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

Initial implementation of gWCS slicing machinery, using ExpressionTree modification #18

Merged
merged 12 commits into from
Aug 31, 2018

Conversation

Cadair
Copy link
Member

@Cadair Cadair commented Aug 22, 2018

This code drops branches from ExpressionTrees to remove inputs (and their corresponding outputs) from the model. The idea is to use this when slicing the gwcs object.

@codecov
Copy link

codecov bot commented Aug 22, 2018

Codecov Report

Merging #18 into master will increase coverage by 0.91%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
+ Coverage   97.04%   97.96%   +0.91%     
==========================================
  Files          12       14       +2     
  Lines         406      589     +183     
==========================================
+ Hits          394      577     +183     
  Misses         12       12
Impacted Files Coverage Δ
dkist/utils/model_tools.py 100% <100%> (ø)
dkist/wcs/slicer.py 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c4a650a...ad7e6e2. Read the comment docs.

@Cadair Cadair force-pushed the gwcs_slicing branch 2 times, most recently from be36092 to afdee44 Compare August 23, 2018 11:23
@Cadair Cadair added this to the v0.1.0 milestone Aug 23, 2018
This code drops branches from ExpressionTrees to remove inputs (and their
corresponding outputs) from the model. The idea is to use this when slicing the
gwcs object.
@Cadair Cadair changed the title Initial implementation of ExpressionTree parsing to remove inputs Initial implementation of gWCS slicing machinery, using ExpressionTree modification Aug 28, 2018
@Cadair Cadair merged commit 486036e into DKISTDC:master Aug 31, 2018
@Cadair Cadair deleted the gwcs_slicing branch August 31, 2018 16:45
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