Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merge dev into main #171

Merged
merged 48 commits into from
May 16, 2024
Merged

Merge dev into main #171

merged 48 commits into from
May 16, 2024

Conversation

AlvaroCubi
Copy link
Member

No description provided.

AlvaroCubi and others added 30 commits April 26, 2024 11:25
* 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
* Add a github workflow that prevents PRs to main that is not comming from dev (#92) (#93)

* Create feature request template

* improve documentation template

* update feature request template

---------

Co-authored-by: AlvaroCubi <55387701+AlvaroCubi@users.noreply.github.com>
* 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
shimwell and others added 18 commits May 13, 2024 15:18
…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
* Add a github workflow that prevents PRs to main that is not comming from dev (#92) (#93)

* using context managers to write files

* review comments 4/5

---------

Co-authored-by: AlvaroCubi <55387701+AlvaroCubi@users.noreply.github.com>
* 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 psauvan merged commit 692bf43 into main May 16, 2024
11 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants