Skip to content

Integrate SQLite, CRT/IAT workflows, and NEcost examples - #58

Merged
nstauff merged 3 commits into
accert-dev:developmentfrom
JiaZhou-PU:combine-pr-clean
Jul 31, 2026
Merged

Integrate SQLite, CRT/IAT workflows, and NEcost examples#58
nstauff merged 3 commits into
accert-dev:developmentfrom
JiaZhou-PU:combine-pr-clean

Conversation

@JiaZhou-PU

@JiaZhou-PU JiaZhou-PU commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Merge Request Description

Summary

This PR integrates the recent ACCERT workflow updates into development.

Added

  • SQLite-backed ACCERT database setup and procedures.
  • CRT workflow package, examples, sampling utilities, and visualization support.
  • IAT API/data updates and ACCERT-to-IAT/CRT bridge utilities.
  • Combined CRT/IAT GUI workflow and tutorial examples.
  • Expanded NEcost tutorial cases and report-check examples.

Changed

  • Reorganized Workbench templates into ACCERT, NEcost, and common template namespaces.
  • Renamed CRF workflow references to CRT.
  • Updated documentation for install, examples, CRT, IAT, GUI workflow, and NEcost usage.
  • Refreshed integration gold outputs and regression coverage.

Removed

  • Legacy MySQL test/workflow pieces.
  • Old generated documentation and obsolete template locations.

Testing

pytest                                 34s  accert 03:58:42 PM

================================================== test session starts ===================================================
platform darwin -- Python 3.12.13, pytest-9.1.1, pluggy-1.6.0
rootdir: /Users/jia.zhou/projects/ACCERT
configfile: pyproject.toml
plugins: anyio-4.9.0
collected 77 items

test_crt_api.py ......                                                                                             [  7%]
test_gui_workflow.py ..                                                                                            [ 10%]
test_iat_api.py ................                                                                                   [ 31%]
test_integration.py ......                                                                                         [ 38%]
test_main.py ..............................                                                                        [ 77%]
test_monte_carlo_sampler.py ...                                                                                    [ 81%]
test_necost_workflow.py ..........                                                                                 [ 94%]
test_sqlite.py ...                                                                                                 [ 98%]
test_util.py .                                                                                                     [100%]

Notes

This branch is intended to merge into accert-dev/ACCERT:development.


For Change Request Review

  • 1. Is the merge request reference an issue? If the issue is closed, the issue close checklist shall be done.
  • 2. Is the merge request reference a task? If the task is fulfilled, the task close checklist shall be done.
  • 3. If any changes to the database, there must be an accompanying change to the all the linked tables.

@JiaZhou-PU
JiaZhou-PU changed the base branch from main to development July 29, 2026 20:39
@JiaZhou-PU JiaZhou-PU changed the title Combine pr clean Integrate SQLite, CRT/IAT workflows, and NEcost examples Jul 29, 2026
@JiaZhou-PU

Copy link
Copy Markdown
Collaborator Author

@nstauff Ready for review

@nstauff nstauff left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This looks great and will be a major release for ACCERT. In the future, I recommend trying to break down the different MR into smaller changes to avoid such large review.

Comment thread docs/source/examples/ap1000.rst Outdated
Comment thread docs/source/reference/table.rst
Comment thread docs/source/user/iat.rst Outdated
Comment thread src/cost_escalation.py Outdated
Comment thread src/Main.py Outdated
Comment thread tutorial/crt/crt_showcase.ipynb Outdated
Comment thread tutorial/gui/crt_iat_gui.py Outdated
Comment thread tutorial/necost/ap1000_lcoe_scenarios.py Outdated
Comment thread tutorial/necost/EG07.OT05.son
Comment thread CHANGELOG.md
@JiaZhou-PU
JiaZhou-PU marked this pull request as ready for review July 30, 2026 14:13
@JiaZhou-PU

Copy link
Copy Markdown
Collaborator Author

Thanks! I will address the comments today

@JiaZhou-PU JiaZhou-PU left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Resolved most comments

Comment thread docs/source/reference/table.rst
Comment thread docs/source/examples/ap1000.rst Outdated
Comment thread docs/source/user/iat.rst Outdated
Comment thread src/cost_escalation.py Outdated
Comment thread src/Main.py Outdated
Comment thread tutorial/crt/crt_showcase.ipynb Outdated
Comment thread tutorial/gui/crt_iat_gui.py Outdated
Comment thread tutorial/necost/ap1000_lcoe_scenarios.py Outdated
Comment thread tutorial/necost/EG07.OT05.son
Comment thread CHANGELOG.md
@JiaZhou-PU

Copy link
Copy Markdown
Collaborator Author

@nstauff, except for the NECOST request, all comments are addressed.

Comment thread docs/source/reference/table.rst
Comment thread tutorial/necost/EG07.OT05.son
@nstauff
nstauff merged commit 941dd0f into accert-dev:development Jul 31, 2026
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.

2 participants