-
Notifications
You must be signed in to change notification settings - Fork 2
Refactor of Weac #8
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
Merged
Merged
Changes from all commits
Commits
Show all changes
173 commits
Select commit
Hold shift + click to select a range
c98ebd7
Starting Refactor: weac_2
pillowbeast bfeb9da
Refactor: slab/scenario -> now system_model solve.
pillowbeast 2e681a6
Refactor: SystemModel + Eigensystem + Constants + Cached_Properties
pillowbeast 2ec1e91
Refactor: Start with SlabTouchdown
pillowbeast 68e43e5
Refactor: Touchdown Implementation
pillowbeast d9a12bc
Refactor: Functional + Tested until Unknown Constants
pillowbeast 220df3c
Minor: l -> length
pillowbeast 60cc766
minor
pillowbeast 5465a2e
Ruff Formating + Default ModelInput
pillowbeast 71a4eec
Refactor: Analysis Implementation.
pillowbeast de54d82
Refactor: Plot + Config + Analyzer
pillowbeast 0689f08
Refactor: Coupled Criterion
pillowbeast 442b006
Refactor: CriteriaEvaluator
pillowbeast 8cd9398
StreamLit: Implementation
pillowbeast 884a439
Proper Slab Deformed Plot
pillowbeast 8cf4c16
Demo + Streamlit + Integration Tests
pillowbeast ac4ad1b
Minor: Test modification
pillowbeast 3524dca
Profiling: Analyzer
pillowbeast 1ada3a2
Plotting: Envelopes
pillowbeast 750acd7
Plotting + Streamlit Analysis
pillowbeast 724d050
minor
pillowbeast 9b40fef
Backup: Misc Data
pillowbeast f463313
Minor: Cleanup
pillowbeast 0627e7c
Plotting: Criteria for Scenarios
pillowbeast c5947a3
Streamlit: Structural changes
pillowbeast ba03bf3
Visualization: Concept
pillowbeast 24ddd95
Streamlit: Analyzer Extended & Minor: Logging changes
pillowbeast fc6287b
.gitignore updates
pillowbeast a6ab257
Streamlit UI: for non-technical user
pillowbeast 285f3f4
Touchdown: Calculation also without PST for DERR calculations
pillowbeast 9706c58
Validation: between weac and weac_2
pillowbeast 4937272
Formatting: Cleanup via Ruff
pillowbeast 45efc5f
Streamlit: Simple Traffic Light Feedback
pillowbeast 3551d16
Rename: crack_l to crack_length (for coherency)
pillowbeast c700024
Streamlit: Steady-State ERR
pillowbeast 3704c56
Attribute Change: Collapse Factor -> Collapsed Height
pillowbeast a172247
Mamba: Provide Environment
pillowbeast 856b1df
Streamlit: Touchdown Distance as Traffic Light for Propagation Potential
pillowbeast eea110d
CleanUp: Formatting + Utils Folder + RM API Folder (un-used code: fas…
pillowbeast df91a2a
Bug Fix: Forgotten change for last commit
pillowbeast 6684369
Bug Fix: Forgotten change from last commit
pillowbeast e92517c
CleanUp: minor
pillowbeast 4eba23e
CAAML: Parser from CAAML using snowpylot to Weac Objects
pillowbeast c9fab76
Attribute Change: Collapse Height calculation based on A.Herwijnen da…
pillowbeast 9a79f15
Minor: Rename snowpilot_parser
pillowbeast f347486
minor: print statement removal / pydantic validation removal for weak…
pillowbeast eda1256
feature: snowpilot_parser implementation from caaml to weac objects v…
pillowbeast 41b9f40
constant: changed rho_ice from 917 to 916.7 to align with new impleme…
pillowbeast 9dff7cc
tests: changed directory structure to mimic weac_2 structure / tests …
pillowbeast 25ea693
feat: read out density measurements from snowpilot files / test: writ…
pillowbeast 76d5037
feat: extended layers with grain information
pillowbeast 7593047
admin: clean-up
pillowbeast 38815fd
bugfix: robustify geldsetzer
pillowbeast 9996111
bugfix: hand_hardness,grain_type,grain_size can be None / feat: let l…
pillowbeast 1f0612f
feat: hand_hardness to density / grain_type to density <- based on ge…
pillowbeast b9b1d20
work_in_progress: snowpilot_parser with new geldsetzer relations
pillowbeast 4c44018
feat/bugfix: evaluate PSTs and fit distributions & plots
pillowbeast dab381a
update: TODO
pillowbeast 33a89b9
feat: change default values
pillowbeast 5c2fb71
analysis: pst + weac_layer spaced
pillowbeast c872f5c
feat: eval weak layer every 50 mm
pillowbeast bdcba88
attri: match crack_height calcualtion between old and new
pillowbeast ad58824
feat: SSERR & TD calculation / proper root search (brentq) for find_m…
pillowbeast a9d1cb8
application: weac layer analysis with TD + SSERR & plotting
pillowbeast 191d96d
Bug Fix: Fix collapse Height according to: collapse height is the hei…
pillowbeast 79da639
Minor: cosmetics, rm commented section
pillowbeast 0722ac4
feat: old_weac also output steady-state energy release rate (SSERR/Gd…
pillowbeast 1592a3a
Plotting: Updates for weac layer evaluation
pillowbeast 434f6d5
Minor: comment out parser functions + print_call_stats flag addition
pillowbeast 1c44326
Analysis: LayerWise + Avalanche / Plotting: LayerWise
pillowbeast eb0aab2
ReStructure: Moving Applications to LayerWise / App-Hub + Exclude: Da…
pillowbeast 082c51e
Minor: Typing fixes
pillowbeast 3bd868a
Mv: image
pillowbeast f110112
Rm: Data -> Data Hub Submodule
pillowbeast 81ed469
Add: data via Data Hub + Gitmodules
pillowbeast 058fcfe
Update: README
pillowbeast deeee4e
Ruff: Formatting
pillowbeast 4cb61d6
CleanUp/Readability/ErrorCatching
pillowbeast 3f6ffb6
Tests: Comprehensive Test Suite -> everything except for regression test
pillowbeast f42b550
Minor/Logging
pillowbeast 864b6bb
Tests: Regression Tests
pillowbeast 4a1f7bb
RENAME: weac -> old_weac & weac_2 -> weac
pillowbeast 4220069
RENAME: old script -> old_ / combined scripts -> test_comparison_
pillowbeast 8bfdf04
Move: demo to demo folder
pillowbeast f99c991
REMOVE: old Weac
pillowbeast 8fdc466
REMOVE: old criterion check
pillowbeast 13caa5e
Minor: Bug Fixes
pillowbeast dd88b80
Dependency Update: provide requirements for venv / environment for Ma…
pillowbeast ee78cbc
Update .cursorignore
zacczakk 4f53977
Update .gitignore
zacczakk d5f90ab
Fix typo
zacczakk 1478bcd
Fix typo
zacczakk d41c39f
Remove unused import
zacczakk 342f180
Remove unused variable assignment
zacczakk 04d7ef2
Fix typo in error message
zacczakk 23b2799
Remove unused import
zacczakk 1b2d7fb
Remove unused import
zacczakk cb56039
Remove unused imports
zacczakk e005f95
Fix indentation and ensure pip is installed
zacczakk dfbc87e
Remove unnecessary f-string prefix
zacczakk f1fb215
Remove unused import
zacczakk 3dea82d
Fix typo
zacczakk 2c63795
Bug Fix and Remove np.array wrapping around mass
pillowbeast 04c9ab9
Add Snowpylot dependency / logger instead of print
pillowbeast 9ded51a
Submodules Shallow + Track Main Branch
pillowbeast a4057df
Catch unwanted modification of layers
zacczakk 42c9d1b
Refactor snow types: Replace Literal types with Enum classes for Grai…
zacczakk 48341f3
Update type hints: Change Tuple to built-in tuple and specify return …
zacczakk 2dc99ea
Remove environment.yml and requirements.txt; update optional dependen…
zacczakk 94a3951
Refactor SKIP_VALUE: Replace hardcoded string with a constant for bet…
zacczakk e4fdc8c
Enhance error handling in UnknownConstantsSolver: Add detailed diagno…
zacczakk 580aee1
CodeRabbit Convos
pillowbeast 3cfa6e9
Refactor constants and improve slab calculations: Change ROMBERG_TOL …
zacczakk bb6d71c
Improve documentation in Segment class: Clarify argument descriptions…
zacczakk 8845f28
Refactor slab parameter calculations: Remove redundant variable and o…
zacczakk f71329d
Refactor: make sure update_from_config also updates crack_length
zacczakk 79739ea
Part2: CodeRabbit Convos
pillowbeast b2fb8a6
Docstring
pillowbeast 2214642
Name Change: crack_length -> cut_length regarding scenario/scenario_c…
pillowbeast 52c0280
Print -> Logger / Recursion Stop Logic / Bug Description + Error Hand…
pillowbeast 2755e46
CodeRabbit Convos
pillowbeast b4a8988
Refactor imports and update type hint for iterations in FindMinimumFo…
zacczakk ea2052f
Refactor criteria evaluation calculations: Introduce local results ar…
zacczakk d05ea99
Refactor tests to utilize isolated weac environment: Introduce weac_r…
zacczakk d2e80f6
Update .cursorignore and .gitignore for improved file management; enh…
zacczakk bafb439
Modification to be able to import weac version for integration tests
pillowbeast fa0e568
Update reference version in tests to weac==2.6.2 for consistency acro…
zacczakk dd7b322
CodeRabbit Review #2
pillowbeast a412801
Testing Dev/Interactive Environment
pillowbeast 7ffe440
feat: Run tests against weac v2.6.3
zacczakk 4b1d362
Add: other tests
pillowbeast 9065998
chore: Update default WEAC reference version to 2.6.4 in weac_referen…
zacczakk a1a65a9
Tests: Comparison of Z instead of C because C can dependent on the sy…
pillowbeast 4ff489b
Minor
pillowbeast 7982298
Tests: Extension of Comparison Test to physical properties Sxx/Szz/Tx…
pillowbeast a5d50f6
Demo: Clean-up
pillowbeast ab92559
Plotter: Minor
pillowbeast eff630a
README: Overhaul
pillowbeast 64b1851
chore: Update pylint configuration to disable naming convention warnings
zacczakk 4449033
Ruff: formatted
pillowbeast b7ddeb9
chore: ruff & pylint
zacczakk d52b05d
Merge branch 'feat/researcher-focus' of https://github.com/2phi/weac …
zacczakk bdfc4a5
chore: Remove push triggers from workflow files for formatting, linti…
zacczakk cd8050d
Docstrings: dummy docstrings at Top of Files
pillowbeast e0a3f5d
chore: Ruff format
zacczakk 10644c5
chore: Ignore import statement lint messages in tests
zacczakk b9a4577
Docstrings: Concise Class Docstrings
pillowbeast b098597
Pylint: Errors
pillowbeast d16a4d0
Pylint: Fixing
pillowbeast afae46f
Final Pylint Resolve & Ignore
pillowbeast ff4d093
chore: Update GitHub Actions to trigger on specific pull request even…
zacczakk 35c9c4e
chore: Improve GitHub Actions with manual triggers and correct format…
zacczakk e9e33b0
rm: unused imports
pillowbeast 40645ad
Use Pyproject.toml to ignore certain errors.
pillowbeast 883aa89
Ruff: Comments
pillowbeast 9c6c465
Raise Error if Old Weac unavilable
pillowbeast cb42561
Install Venv for CI Test
pillowbeast c3ff760
Figure out wihy workflow fails.
pillowbeast 34bfa91
UnitTest Error fix
pillowbeast 770af47
Coderabbit Comments
pillowbeast b9d8ba0
Remove installation step for Python 3.12 venv in GitHub Action
zacczakk 4d9d3ca
Enhance GitHub Actions workflow by adding pip caching and updating Py…
zacczakk 4f4b33a
doc: Update requirements
zacczakk 08378f4
chore: Update pylint configuration to use pyproject.toml for comprehe…
zacczakk b153761
Merge branch 'main' into feat/researcher-focus
zacczakk be7c760
fix: Version number
zacczakk 9ffb31f
fix: Remove unnecessary rcfile option from pylint command in GitHub A…
zacczakk 32ff41e
doc: Fix typos and improve clarity in README.md
zacczakk 803ca37
doc: Enhance README.md with updated version details and improved code…
zacczakk 2a13014
chore: Remove unused import
zacczakk 42f349a
docs: add Sphinx config; set version/release as strings and drop depr…
zacczakk cf0b44d
chore: Ruff
zacczakk a72b6d9
refactor: Remove main.py and validation_cc.py scripts to clean up pro…
zacczakk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| /docs/ | ||
| /.venv/ | ||
| /data/ | ||
| /img/ | ||
| /demo/ | ||
| /LICENSE |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| [submodule "data"] | ||
| path = data | ||
| url = https://github.com/2phi/weac-data-hub.git | ||
pillowbeast marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| shallow = true | ||
| branch = main | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧹 Nitpick (assertive)
Optional: drop grep -P for portability.
GNU grep -P is fine on ubuntu-latest, but can be disabled elsewhere. Consider awk/sed for portability if you reuse locally.
Example replacement for the three grep -oP pipelines:
awk '{ if (match($0, /[A-Z][0-9]+([a-z-]+)/, a)) print a[0] }' pylint-report.txt | sort | uniq -c | sort -nr || true
awk -F: 'NF>1 &&
awk -F: 'NF>1 &&
🤖 Prompt for AI Agents