Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

88 add options for saving in sa #90

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

maypink
Copy link
Collaborator

@maypink maypink commented Apr 27, 2023

WIP

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

aim-pep8-bot commented Apr 27, 2023

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
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