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

Add info to log files #295

Closed
wsnoble opened this issue Feb 15, 2024 · 0 comments · Fixed by #343
Closed

Add info to log files #295

wsnoble opened this issue Feb 15, 2024 · 0 comments · Fixed by #343
Assignees
Labels
enhancement New feature or request

Comments

@wsnoble
Copy link
Contributor

wsnoble commented Feb 15, 2024

I am looking at the log file when I run casanovo in sequence mode, and it's not very informative. Here is what I see, after the long print-out of all the config options:

2024-02-14 17:18:42,503 INFO [casanovo/MainProcess] casanovo.sequence : Sequencing peptides from:
2024-02-14 17:18:42,503 INFO [casanovo/MainProcess] casanovo.sequence :   immunotest_orbitrap.mgf
2024-02-14 17:18:42,869 INFO [depthcharge.data.hdf5/MainProcess] hdf5.__init__ : Reading 1 files...

I think there is some easy information we could add to make it more useful. This includes

o the full command line that was issued,

o the name of the host machine,

o the wall clock time at the start and end of the run, as well as the elapsed wall clock time,

o a final count of the number of spectra that were sequenced, and

o some statement at the end indicating that the run completed without error.

If it's possible to also include info about the total CPU and/or GPU usage, that would be great.

@wsnoble wsnoble added the enhancement New feature or request label Feb 15, 2024
@bittremieux bittremieux added this to the Casanovo v5.0.0 milestone May 14, 2024
@Lilferrit Lilferrit linked a pull request Jun 21, 2024 that will close this issue
@Lilferrit Lilferrit self-assigned this Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants