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

CNSGA Output Filenames #252

Merged

roussel-ryan merged 1 commit into xopt-org:main from electronsandstuff:patch-1

Nov 5, 2024

Contributor

electronsandstuff commented Nov 5, 2024

This PR includes the following two changes proposed to allow users on Windows to run the CNSGA generator with file output.

CNSGAGenerator.write_offspring has been updated to avoid the colon character in filenames similarly to write_population in Update cnsga.py #251
The string replacement for colon characters in filenames has been moved to only apply to the timestamp. This avoids causing problems when self.output_path looks like "C:\Users\yourname\project\dir"


          update CNSGA output filenames

894faae

ChristopherMayes approved these changes

View reviewed changes

ChristopherMayes requested a review from roussel-ryan

November 5, 2024 12:54

roussel-ryan approved these changes

View reviewed changes

roussel-ryan merged commit 6de78a7 into xopt-org:main

7 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet