Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Teach flywire_cave_query about query filter dicts #162

Merged
merged 1 commit into from
Aug 1, 2022

Conversation

jefferis
Copy link
Collaborator

@jefferis jefferis commented Aug 1, 2022

  • convert R lists to python dicts using cavedict_rtopy
  • with test

* convert R lists to python dicts using cavedict_rtopy
* with test
@codecov
Copy link

codecov bot commented Aug 1, 2022

Codecov Report

Merging #162 (651651d) into master (b15e710) will increase coverage by 0.06%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #162      +/-   ##
==========================================
+ Coverage   49.84%   49.91%   +0.06%     
==========================================
  Files          33       33              
  Lines        3956     3963       +7     
==========================================
+ Hits         1972     1978       +6     
- Misses       1984     1985       +1     
Impacted Files Coverage Δ
R/cave.R 62.30% <88.88%> (+1.33%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us.

@jefferis jefferis merged commit bc2d569 into master Aug 1, 2022
@jefferis jefferis deleted the feature/flywire_cave_query-filter branch August 1, 2022 20:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant