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.

Sign up for GitHub

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

Jump to bottom

Merge dev into main #67

Merged
merged 30 commits into from
Apr 23, 2024
Merged

Merge dev into main #67

merged 30 commits into from
Apr 23, 2024

Conversation

AlvaroCubi
Copy link
Member

No description provided.

psauvan and others added 28 commits March 20, 2024 17:27
@psauvan
update Fuzzy write function
3d4ad1f
@psauvan
several stp file update
d123e67
@psauvan
script to color CAD document following material label
f7cdf77
@psauvan
fix rounding error of z value acos(z) evaluation
db451b8
@AlvaroCubi
Add a bug report template and .gitignore (#44)
f106652
@repositony
Removed unused imports and variables (#35)
4db7ce2
@shimwell
raise value error instead of exit (#41)
2d6e263
@shimwell
Update to testing path (#40)
fdcd29d 
* changed hardcoded path to work for all users

* path update
@psauvan
Update bug_report.md
f14702d
@shimwell
raising catchable error instead of exit (#47)
59731d8 
* raising catchable error instead of exit

* added new line
@shimwell
added try for freecad import (#48)
377156b
@AlvaroCubi
Solve SyntaxtError producing a bug (#49)
dcb2c66 
* Solve SyntaxtError producing a bug

* Limit the amount of decimals printed in the enclosure comment
@psauvan
fixed error units in OpenMC output
b132266
@akolsek
GEOUNED training presentation added (#53)
b0742fd
@shimwell
Adding minimal ci to check cad converts (#52)
9173938 
* added minimal CI

* all step files

* added ci yml

* removed freecadcmd arg
@psauvan
Update pyproject.toml
7a81367 
Update url links to "GEOUNED-org"
@shimwell
typo correction to test if CI is working (#54)
6ed02e9 
* typo correction

* [skip ci] added ci status badge

* opening up ci to all os systems
@psauvan
fix small bug with torus (bad identification of closed mozaic faces)
4cbd230
@shimwell
Adding minimal volume checking CI comparing CAD and CSG volumes (#56)
6e7f883 
* added volume test to ci

* removed if os cond

* added csg vs cad volume for **most** stp files
@shimwell
Added mkdocs (#57)
1a46bcd 
* added example docs

* added docs deps
@shimwell
Reverting mkdocs PR 57 (#58)
d57bf81 
* Revert "added example docs"

This reverts commit bd1c67a.

* Revert "added docs deps"

This reverts commit f31d241.
@shimwell
Adding minimal particle transport tests on resulting csg files (#59)
44ee8fa 
* added transport tests

* removed files that break, raised issue

* [skip ci] remove prints

* added link to issue
@ecasglez
Add the name to the cells in OpenMC to improve identification (#61)
a96e19d 
The name is the same that is used in MCNP inputs cell.Comments
@shimwell
converting with no simplification = more geometry works (#62)
a4cc2cd
@shimwell
reduced number of skiped geoms (#63)
f741417
@shimwell
[skip ci] corrected bade to main branch (#64)
8e0e465
@shimwell
Improved import statements (#65)
af70963 
* changed single file from package to relative imports

* 2nd file converted to relative imports

* converted more fiels to relative import

* relative imports everywhere

* isort on all imports
@shimwell @AlvaroCubi
Add bug report and .gitignore to main (#45) (#66)
6160952 
Co-authored-by: AlvaroCubi <[email protected]>
@AlvaroCubi AlvaroCubi requested a review from psauvan April 23, 2024 13:01
@shimwell
Copy link
Collaborator

super to see this PR,
the badge on the main README should go green after these changes get pushed to main and the push trigger the CI on main 📗

@psauvan psauvan merged commit 8bc7bd0 into main Apr 23, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@psauvan psauvan psauvan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@AlvaroCubi @shimwell @psauvan @repositony @akolsek @ecasglez