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

QPPC-6017: Docker Source cleanup, Update measures, mvp data #763

Merged
merged 7 commits into from
Feb 8, 2024

Conversation

mike-tekavec
Copy link
Contributor

Related Tickets & Documents

https://jira.cms.gov/browse/QPPC-6017
https://jira.cms.gov/browse/QPPC-6018


Description

Simplify the Docker image for creating the single source json file from Mathematica's single source CSV.
Update measures data


What type of PR is this?

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore
  • ⏩ Revert

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📜 README.md
  • 📓 Confluence
  • 🙅 no documentation needed

[ ⌥ Optional ] Are there any post-deployment tasks we need to perform?


Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mathematica's new file (v7.0_1.29.24.csv) contains the additionalProcedure code we need for C2Q

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The single source json file created from the new CSV adds the additionalProcedureCode. There are additional differences in the order of elements in the peformanceOptions array but this does not impact (will confirm) C2Q results

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah it appears that way. We (claims) are the only ones that care about this file. The ordering causing more diffs in these might cause some pause, but otherwise I think mostly claims items were touched overall and looking good.

measures/2023/measures-data.json
mvp/2023/mvp-enriched.json

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updates from the single source json file, similar additions/modifications here that occurred in that file.

Copy link
Contributor

@ChristopherMTeixeira ChristopherMTeixeira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice. The claims items were due for an overhaul. Order of codes being mangled around in some of the shared jsons, is probably pretty typical.

Copy link
Contributor

@ChristopherMTeixeira ChristopherMTeixeira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NB but thought of something, if we're not ready to remove, should we put a note in the old claims related items, that they're just for historical purposes? The old README for creating the single source, the py script for the same, and the few odds and ends. Auxilary scripts like clean_csv.py might not be such a big deal, but wouldn't want someone to catch the old README and make single source with old stuff.

Also, will help see if we can get the other eyes we'll need on this from outside claims.

Copy link

sonarcloud bot commented Feb 6, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

6 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@mike-tekavec mike-tekavec merged commit 532cc79 into develop Feb 8, 2024
4 checks passed
@mike-tekavec mike-tekavec deleted the QPPC_6017_dockerSourceCleanup branch February 8, 2024 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants