Skip to content
This repository has been archived by the owner on May 9, 2023. It is now read-only.

Fix division by zero bug + generalize components #15

Merged
merged 2 commits into from
Jan 25, 2022
Merged

Fix division by zero bug + generalize components #15

merged 2 commits into from
Jan 25, 2022

Conversation

cvauclair
Copy link
Contributor

subgrounds.plotly_wrappers:

  • Generalize TraceWrapper object

subgrounds.query:

  • Fix bug in Query.get_argument
  • Fix bug in Seleciton.get_argument

subgrounds.subgraph:

  • Add optional argument default to SyntheticField constructor

subgrounds.transform:

tests:

  • Remove outdated tests

examples:

  • Update Olympus subgraph API URL

@cvauclair
Fix division by zero bug + generalize components
37d75ba 
subgrounds.plotly_wrappers:
- Generalize `TraceWrapper` object

subgrounds.query:
- Fix bug in `Query.get_argument`
- Fix bug in `Seleciton.get_argument`

subgrounds.subgraph:
- Add optional argument `default` to SyntheticField constructor

subgrounds.transform:
- Fix unhandled `ZeroDivisionError` exception in `LocalSyntheticField`
  transform
- Start removing old, unused code

tests:
- Remove outdated tests

examples:
- Update Olympus subgraph API URL
@cvauclair cvauclair added bug Something isn't working enhancement New feature or request labels Jan 25, 2022
@cvauclair cvauclair merged commit d5382ba into main Jan 25, 2022
@cvauclair cvauclair mentioned this pull request Jan 25, 2022
Closed
@cvauclair cvauclair deleted the bugfix/sfield-division-by-zero branch February 13, 2022 17:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Synthetic fields division by zero
1 participant
@cvauclair