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

GrFN Omit Sources Toggle Flag #286

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

titomeister
Copy link
Contributor

Modified the GrFN data structure to allow it to have a toggle that determines whether or not the source reference information is included in the GrFN JSON. The source reference information is removed before the GrFN dictionary gets converted to JSON. An additional flag is added to 'run_gcc_to_grfn.py' that allows control of the toggle.

@codecov
Copy link

codecov bot commented Feb 14, 2022

Codecov Report

Merging #286 (ad5675f) into master (f7023ef) will decrease coverage by 0.08%.
The diff coverage is 24.13%.

❗ Current head ad5675f differs from pull request most recent head 21357ee. Consider uploading reports for the commit 21357ee to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #286      +/-   ##
==========================================
- Coverage   66.01%   65.93%   -0.09%     
==========================================
  Files          95       95              
  Lines       18014    18041      +27     
==========================================
+ Hits        11892    11895       +3     
- Misses       6122     6146      +24     
Impacted Files Coverage Δ
automates/model_assembly/networks.py 79.89% <19.23%> (-1.80%) ⬇️
automates/program_analysis/CAST2GrFN/cast.py 81.30% <66.66%> (-0.60%) ⬇️
automates/model_analysis/graph_manipulation.py 70.19% <0.00%> (-0.56%) ⬇️

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 f7023ef...21357ee. Read the comment docs.

@titomeister titomeister requested a review from cl4yton February 14, 2022 17:52
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