Merge dev into main #171
Merged
Merge dev into main #171
Conversation
This file contains 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
* camel case for all classes * corrected file imports for georeverse * Apply suggestions from code review variable back to snake case * variable back to snake case * back to snake case * back to previous var name
* snake case for CellDefinitions file, tests pass * vars in snake_case 2 more files * working still
* Add comments to Geometry_GU.py * Add comments to booleanFunction.py * Add comments to Geometry_GU.py
* improved import statements flake8 hints * removed extra line
* Add a github workflow that prevents PRs to main that is not comming from dev (#92) (#93) * Add scripting interface Change to allow setting users parameters by either config file or geouned set method * minor fix * Changes in geounedClassCall.py --------- Co-authored-by: AlvaroCubi <55387701+AlvaroCubi@users.noreply.github.com>
* added format checker * run black . locally
* replaced all .format with f strings * corrected f string conversion
* snake case im more varibles * formatting --------- Co-authored-by: Patrick Sauvan <psauvan@ind.uned.es>
* removing slowest two geometries * removing slow files from CI * format
@psauvan comment has been taken into account. Unreachable code that was there for future consideration has been changed to comments. * if statement includes False so never runs code * if statement includes True so always run and never reaches else * line after return never reached * unused arg * unused arg * unused arg calling function * commented unused code as requested * formatting * format
* also writting openmc python, phits and serpent files * formatting * decreased lcoal run time and added all outFormats * black using same version as in pyproject * py3.11 in formatting ci * matched exactly black versions * simpler black pinning * format
* converted more functions * more snake case functions * more snake case function names * more snake case functions * more snake case functions * more snake case functions * formatting * formatting * formatting * formatting * converting function names to snake class
* started rtd * testing docs action * update pip * corrected python version * added s to docs * publish every time * added static folder * skipping ci for non code files * changed to rerad the docs theme * started install section * added more sections * added api class * docs ci using conda * added -y for freecad * trying py 3.12 * mimic working conda ci * using rst so no markdown convertion needed * added github edit link * removed path prints * added readme link, badge * format
* pushing triggers doics build * added if push
* replaced print statements with log * added logger * formatting * removed encoding arg * removed verbose as all gets logged * removed unused logger * removed , from logging cmd * sorted imports * added another tqdm loop * isort * format
* fixed multi comma log statements * capital letters for progress bar sections * formatted
* instances of Options class passed by arg * updated usage to show how to use Options * format pep8 * added missing options arg * added missing args options * passing in options as arg * bug fix for compsolid * format
* passing tolerance as arg not global var * added tolerance example to usage section * one set of argument names * moved atr and types getting out of loop * format * passing missing options and tolerances * format * found bug hidden in try: except statement * using tolerance for more local varibles * formatting
* replaced global with class passed by arg * format
* added dev install instructions * added method section
…ormat class (#148) * added type ching setters * added low limit checking for reals * format * added type checkers for Tolerances * format
* settings being passed through the code * all the attributes show in usage example
* added export csg method * format * leaving comments in tests
* testing from json * returned file to pre PR status * returned file to init * returned file to init * temp fixing for serpent universe, issue raised * making use of stp file already on repo
* split usage into two sections * added minimal config from cli example to ci * improved cli docs * format
* moved core logic from init * format
* added test that currently fails * fixed bug with freecad attribute usage * added missing arg * format
* added missing args * format
* added tests to check writing to folder * back to org * back to org * back to org * creating output folder prior to writing * isort on the imports * format
* renamed subfolders to lowercase * renamed files to lower case
* openmc_py and openmc_xml lower case * removed outdated code comment
* added input checking on export_csg * compacted same types * Update src/geouned/GEOUNED/core.py --------- Co-authored-by: Patrick Sauvan <psauvan@ind.uned.es>
* added windows install instructions * improved install instructions
psauvan
approved these changes
May 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.