Skip to content

Update tests model for MOD to C as well as C++ translation#1747

Merged
olupton merged 14 commits into
masterfrom
pramodk/models-update-c-cpp-compatibility
Apr 1, 2022
Merged

Update tests model for MOD to C as well as C++ translation#1747
olupton merged 14 commits into
masterfrom
pramodk/models-update-c-cpp-compatibility

Conversation

@pramodk

@pramodk pramodk commented Mar 29, 2022

Copy link
Copy Markdown
Member
  • All CI here in master should pass to make sure model changes are
    backward compatible with MOD to C translation.
  • If CI passes, merge PRs and update all submodules

@pramodk pramodk force-pushed the pramodk/models-update-c-cpp-compatibility branch from 2ea8397 to bf6e866 Compare March 31, 2022 15:26
@olupton olupton force-pushed the pramodk/models-update-c-cpp-compatibility branch from f4e8b76 to d7543aa Compare March 31, 2022 21:21
@codecov-commenter

codecov-commenter commented Mar 31, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1747 (2546fa3) into master (a527c23) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1747   +/-   ##
=======================================
  Coverage   45.42%   45.43%           
=======================================
  Files         551      551           
  Lines      113144   113144           
=======================================
+ Hits        51399    51403    +4     
+ Misses      61745    61741    -4     
Impacted Files Coverage Δ
src/ivoc/ivocvect.cpp 82.22% <0.00%> (+0.13%) ⬆️
src/parallel/bbs.cpp 76.27% <0.00%> (+0.42%) ⬆️

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@olupton olupton merged commit 710d2ca into master Apr 1, 2022
@olupton olupton deleted the pramodk/models-update-c-cpp-compatibility branch April 1, 2022 10:24
@olupton olupton mentioned this pull request Apr 1, 2022
6 tasks
@alexsavulescu alexsavulescu mentioned this pull request Jun 29, 2022
19 tasks
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.

5 participants