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 groundwork for data flag & outlier marking #22

Merged
merged 49 commits into from
Jan 21, 2021

Conversation

davidclemens
Copy link
Owner

@davidclemens davidclemens commented Jan 14, 2021

This PR adds the basics for data flags & outlier marking by implementing the underlying architecture.

It is part of DataKit and linked to gearDeployments with the markOutliers method.

Further implementation into the plot methods and the subsequent data analysis is still required.

This PR is part of #3 and #4.

Also add abbreviations to worldDomain.
- Also improve the performance of by only accessing the get.Index method once.
Also includes some new unit tests.

Part of #4 & #9.
Removing empty groups caused the indexes no longer to be the correct size when used in the relative time calculations. Fixed by moving the block to the end.
If an axis held other objects, such as text, they'd be returned as a chart as well. This is now fixed.
@davidclemens davidclemens added new feature New feature or request UI User interface kit: GearKit kit: DataKit labels Jan 14, 2021
@davidclemens davidclemens self-assigned this Jan 14, 2021
@davidclemens davidclemens added this to the v0.1b1 milestone Jan 14, 2021
@davidclemens davidclemens marked this pull request as draft January 19, 2021 22:44
@davidclemens davidclemens marked this pull request as ready for review January 21, 2021 12:13
@davidclemens davidclemens merged commit 9665ba8 into development Jan 21, 2021
@davidclemens davidclemens deleted the refactor-outlier-marking branch January 21, 2021 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kit: DataKit kit: GearKit new feature New feature or request UI User interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant