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.

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

Standardizing Windows data path and removing folder creation on import #264

Merged
merged 2 commits into from
Jun 1, 2023

Conversation

raph-luc
Copy link
Member

@raph-luc raph-luc commented May 31, 2023

Same user data path naming scheme as in ansys/pyfluent#1615, ansys/pymapdl#2064, ansys/pyprimemesh#485, ansys/ansys-tools-path#44, ansys/pysystem-coupling#158, ansys/pymechanical#257

Original issue tracker ansys/pyfluent#1588

Changes Windows data path from %LocalAppData%\ansys_mapdl_reader\ansys_mapdl_reader to %LocalAppData%\Ansys\ansys_mapdl_reader, no change to the path on Linux

Removed what looks like an unnecessary try-except block

Also changed it so that the examples and data path folders are not created immediately on import regardless of whether they will be used. They are instead created when downloading example files through downloads._retrieve_file(), as that seems to be the only place where these directories were used in this module

@raph-luc raph-luc changed the title Standardizing Windows data path Standardizing Windows data path and folder creation on import May 31, 2023
@raph-luc raph-luc changed the title Standardizing Windows data path and folder creation on import Standardizing Windows data path and removing folder creation on import May 31, 2023
Copy link
Member

@RobPasMue RobPasMue left a comment

Choose a reason for hiding this comment

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

LGTM!

@RobPasMue RobPasMue merged commit f7066a6 into main Jun 1, 2023
@RobPasMue RobPasMue deleted the fix/standardizing_data_path branch June 1, 2023 06:26
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