Skip to content

Clean references#2

Merged
rhandberg merged 10 commits intoSNflows:develfrom
emirkmo:clean_references
Jun 30, 2020
Merged

Clean references#2
rhandberg merged 10 commits intoSNflows:develfrom
emirkmo:clean_references

Conversation

@emirkmo
Copy link
Contributor

@emirkmo emirkmo commented Jun 26, 2020

These are my initial change to make a clean reference star list. This involves the nonlinear cut-off parameter added to image class. And then using the DAOStarFinder comparison more strictly for now, with a back-up in case it fails.

I also added a function for removing targets within X distance and commented out reference code for filtering based on daofind parameters.

I also added Jupyter notebooks/lab misc to .gitignore.

@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2020

Codecov Report

Merging #2 into devel will decrease coverage by 0.22%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel       #2      +/-   ##
==========================================
- Coverage   19.08%   18.85%   -0.23%     
==========================================
  Files          14       14              
  Lines         964      981      +17     
==========================================
+ Hits          184      185       +1     
- Misses        780      796      +16     
Impacted Files Coverage Δ
flows/load_image.py 20.45% <0.00%> (-1.50%) ⬇️
flows/photometry.py 9.31% <0.00%> (-0.30%) ⬇️
flows/download_catalog.py 17.54% <0.00%> (+0.27%) ⬆️

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 7cc8f9e...7549840. Read the comment docs.

@rhandberg rhandberg merged commit 8e90620 into SNflows:devel Jun 30, 2020
@emirkmo emirkmo deleted the clean_references branch July 1, 2020 09:24
rhandberg pushed a commit that referenced this pull request Apr 22, 2021
Rasmus Comments, flake8 things. Correct Branch
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.

3 participants