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

Timestamp utils #703

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Timestamp utils #703

wants to merge 4 commits into from

Conversation

rsagas
Copy link
Contributor

@rsagas rsagas commented Mar 14, 2024

While trying to modularize the qililab portfolio, I started going backwards into things that are not modularized and make it difficult for me to modularize the experiments.

Specifically, the treatment of the whole file system and timestamps is not properly modular, so I went ahead and added a simple module with the functions.

I have also made unit tests.

This is a WORK IN PROGRESS. I created this PR for tracking it, and will notify once it is ready to be reviewed

@rsagas rsagas requested a review from jjmartinezQT March 14, 2024 11:58
@rsagas rsagas self-assigned this Mar 14, 2024
@rsagas rsagas marked this pull request as draft March 14, 2024 12:50
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.

1 participant