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

Adding date filtering feature and other improvements #1

Merged
merged 5 commits into from
Feb 15, 2024

Conversation

Vicbi
Copy link
Collaborator

@Vicbi Vicbi commented Feb 13, 2024

Adding date filtering feature and other improvements

⚙️ Release Notes

*Incorporation of the date filtering feature for the visualisation tool.

📝 Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

@Vicbi Vicbi added the enhancement New feature or request label Feb 13, 2024
@Vicbi Vicbi closed this Feb 13, 2024
@Vicbi Vicbi reopened this Feb 13, 2024
@Vicbi Vicbi closed this Feb 13, 2024
@Vicbi Vicbi reopened this Feb 13, 2024
@Vicbi Vicbi closed this Feb 14, 2024
@Vicbi Vicbi reopened this Feb 14, 2024
@Vicbi Vicbi assigned Vicbi and PSchmiedmayer and unassigned Vicbi Feb 15, 2024
Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

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

Thank you for the changes; I just reviewed the code and it is great that you added some more context and surfaced more of the core functionality.

I only had a minor comment that the file name should stay generic in the open-source version of the notebook and we should not contain any project-specific code in there.

SpeziDataPipeline.ipynb Outdated Show resolved Hide resolved
Copy link
Member

@PSchmiedmayer PSchmiedmayer left a comment

Choose a reason for hiding this comment

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

I had some comments for the README to improve the formatting and some issues in the formatting how this will be displayed in the repo 👍

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Vicbi Vicbi requested a review from PSchmiedmayer February 15, 2024 18:33
@PSchmiedmayer PSchmiedmayer merged commit d6b481a into main Feb 15, 2024
@PSchmiedmayer PSchmiedmayer deleted the date-filtering-feature-branch branch February 15, 2024 21:01
Vicbi added a commit that referenced this pull request Feb 28, 2024
*Incorporation of the date filtering feature for the visualisation tool.

By submitting creating this pull request, you agree to follow our [Code
of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md):
- [x] I agree to follow the [Code of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md).
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 this pull request may close these issues.

2 participants