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

CodeCarbon #350

Merged
merged 44 commits into from
Mar 28, 2023
Merged

CodeCarbon #350

merged 44 commits into from
Mar 28, 2023

Conversation

carraraig
Copy link
Collaborator

Added Codecarbon output for each pipeline. We need to change the format of the column of emission in scientific format.
@sylvchev do you know where to change this option?

@carraraig carraraig requested a review from sylvchev March 21, 2023 14:52
@carraraig
Copy link
Collaborator Author

carraraig commented Mar 21, 2023

The package codecarbon need to be added, @bruAristimunha can you add it using poetry?

@carraraig
Copy link
Collaborator Author

carraraig commented Mar 21, 2023

At the moment seems that the codetracker is not reinitialize at each computation. Try to solve

moabb/analysis/results.py Outdated Show resolved Hide resolved
moabb/analysis/results.py Outdated Show resolved Hide resolved
moabb/evaluations/evaluations.py Outdated Show resolved Hide resolved
moabb/evaluations/evaluations.py Outdated Show resolved Hide resolved
moabb/evaluations/evaluations.py Outdated Show resolved Hide resolved
moabb/tests/analysis.py Outdated Show resolved Hide resolved
moabb/tests/analysis.py Outdated Show resolved Hide resolved
moabb/tests/analysis.py Outdated Show resolved Hide resolved
moabb/tests/evaluations.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
carraraig and others added 12 commits March 22, 2023 13:42
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Co-authored-by: Sylvain Chevallier <[email protected]>
Copy link
Collaborator

@bruAristimunha bruAristimunha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @carraraig and @sylvchev,

Looks amazing for me =)

@sylvchev
Copy link
Member

I updated the requirements and pyproject.toml. There is still an error in the documentation due to the update of Sphinx version. Once this is solved, we could merge!

@sylvchev
Copy link
Member

The doc build error is strange, I could not reproduce it locally. Maybe we could merge this PR and I'll open a new one to make some correction on the documentation (there are some error and warning) to further investigate this error.

@carraraig
Copy link
Collaborator Author

I agree with this proposal @sylvchev

@sylvchev sylvchev merged commit c9c4043 into NeuroTechX:develop Mar 28, 2023
@sylvchev
Copy link
Member

sylvchev commented Mar 28, 2023

thanks @bruAristimunha and @carraraig, it so amazing to have carbon footprint integrated in MOABB.

@bruAristimunha
Copy link
Collaborator

Super cool!

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.

3 participants