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

Implements codecarbon plots #330

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

Conversation

sanam2405
Copy link

@sanam2405 sanam2405 commented Nov 11, 2023

While profiling baler with codecarbon, the profiling logs can be stored as a .CSV file. This PR creates plots from the .CSV log file for better visualization of the energy profiles.

For a sample .CSV as shown below

Screenshot from 2023-11-11 14-03-54

The PR plots figures as

perform_training_plot1

perform_training_plot2

perform_training_plot3

perform_training_plot4

These plots are stored at the location workspaces/CFD_workspace/CFD_project_animation/output/profiling
I have created an empty folder profiling under the output directory and added a .gitkeep

Screenshot from 2023-11-11 13-52-11

However it seems that the empty profiling folder is still getting ignored and not being pushed into GitHub. For now, this PR can be reviewed by explicitly creating the profiling folder but this needs to be fixed in the .gitignore which I am unable to figure out and I need help.

The .gitignore present in the root of baler directory has some issues. It is not ignoring as intended, in particular, if we add new contents/folders/files inside the workspaces directory. I fixed the above mentioned strikethrough issue by directly making an empty folder named profiling with a .gitkeep inside and committed the changes on GitHub itself.
However the same doesn't seem to work when I do it locally. Git cannot track the empty folder and cannot push the empty folder into GitHub. Making changes to the .gitignore also doesn't seem to work.

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.

None yet

1 participant