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

New Monitr #354

Merged
merged 132 commits into from
Dec 6, 2022
Merged

New Monitr #354

merged 132 commits into from
Dec 6, 2022

Conversation

pfafflabatuiuc
Copy link
Contributor

A new version of Monitr with more features. For a guide on how to use see: https://toolsforexperiments-manual.readthedocs.io/en/latest/plottr/apps.html#monitr

# Conflicts:
#	plottr/data/datadict.py
…ree working. It keeps up to date with watchdog events. First version of filtering is also implemented.
…ittle the documentation of watchdog_classes.py. fixed bug when updating parent folders that didn't contain any ddh5 but folders containing ddh5.
…the program session.

Added the parameter tittle to the widget Collapsible.
…l size. Tags are now shown on top and with color. Comments are now created with time-stamps. Minor code renaming and documentation.
@pfafflabatuiuc
Copy link
Contributor Author

I have added watchdog as a testing requirement in test_requirements.txt. Should I add it as a requirement in requirements.txt and the setup file?

Copy link
Contributor

@wpfff wpfff left a comment

Choose a reason for hiding this comment

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

this works well, but please fix typos in filenames (and elsewhere if there are any ;)) and add watchdog to the requirements.

@@ -0,0 +1,316 @@
"""
Copy link
Contributor

Choose a reason for hiding this comment

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

fix typo in filename

@marcosfrenkel marcosfrenkel linked an issue Dec 5, 2022 that may be closed by this pull request
Copy link
Contributor

@wpfff wpfff left a comment

Choose a reason for hiding this comment

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

looks good now.

@wpfff wpfff merged commit 72a8f4c into toolsforexperiments:master Dec 6, 2022
@pfafflabatuiuc pfafflabatuiuc deleted the monitr_revamp branch December 6, 2022 17:21
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.

concept for improving monitr
4 participants