Skip to content

88 add options for saving in sa#90

Open
maypink wants to merge 8 commits into
mainfrom
88-add-options-for-saving-in-sa
Open

88 add options for saving in sa#90
maypink wants to merge 8 commits into
mainfrom
88-add-options-for-saving-in-sa

Conversation

@maypink

@maypink maypink commented Apr 27, 2023

Copy link
Copy Markdown
Collaborator

WIP

@maypink maypink linked an issue Apr 27, 2023 that may be closed by this pull request
@aim-pep8-bot

aim-pep8-bot commented Apr 27, 2023

Copy link
Copy Markdown
Collaborator

Hello @maypink! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 1:1: F401 'datetime' imported but unused
Line 14:1: F401 'golem.core.optimisers.timer.OptimisationTimer' imported but unused
Line 80:80: F821 undefined name 'timer'

Line 2:1: F401 'typing.Sequence' imported but unused

Line 61:45: E251 unexpected spaces around keyword / parameter equals
Line 171:47: E226 missing whitespace around arithmetic operator
Line 342:76: E226 missing whitespace around arithmetic operator

Line 171:47: E226 missing whitespace around arithmetic operator
Line 280:76: E226 missing whitespace around arithmetic operator
Line 352:47: E226 missing whitespace around arithmetic operator

Comment last updated at 2023-05-05 14:32:25 UTC

@nicl-nno

Copy link
Copy Markdown
Collaborator

Актуально или можно закрыть?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add options for saving in SA

3 participants