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

Add helper scripts for creating and deleting large numbers of tecplot files #225

Merged
merged 7 commits into from
Jul 28, 2023

Conversation

A-CGray
Copy link
Contributor

@A-CGray A-CGray commented Jun 30, 2023

I have a bunch of helper scripts that I use for processing the large number of f5 files that get output from my optimisations. This PR would add them to the repo so other people can use them too.

Summary of scripts:

  • f5convert_all: Runs f5totec in parallel on all .f5 files in the current folder for which there is not an existing tecplot file (or there is a tecplot file but it's older than the f5 file)
  • f5convert_subdirs: Finds all directories from the current directory down that contain .f5 files and runs f5convert_all in each one.
  • f5clean_all: Deletes all tecplot files in the current directory for which there is a corresponding f5 files (useful for freeing up disk space)
  • f5clean_subdirs: Finds all directories from the current directory down that contain .f5 files and runs f5clean_all in each one.

@timryanb
Copy link
Collaborator

timryanb commented Jul 5, 2023

Thanks @A-CGray, I think these features would be a good addition.
Is there anyway we could implement them under f5totec as command line options?
We could also potentially add this to f5tovtk

@A-CGray
Copy link
Contributor Author

A-CGray commented Jul 5, 2023

Thanks @A-CGray, I think these features would be a good addition. Is there anyway we could implement them under f5totec as command line options? We could also potentially add this to f5tovtk

I'm not sure how easy it'd be to add these capabilities to the f5totec executable as that would require re-implementing all the filesystem operations in C.

However, it should be pretty easy to add a command line-option to the scripts to switch between using f5totec and f5tovtk. I could probably also make the subdirectory traversal a command-line option, so we'd just end up with two scripts f5convert and f5clean

@timryanb
Copy link
Collaborator

timryanb commented Jul 5, 2023

Thanks @A-CGray, I think these features would be a good addition. Is there anyway we could implement them under f5totec as command line options? We could also potentially add this to f5tovtk

I'm not sure how easy it'd be to add these capabilities to the f5totec executable as that would require re-implementing all the filesystem operations in C.

However, it should be pretty easy to add a command line-option to the scripts to switch between using f5totec and f5tovtk. I could probably also make the subdirectory traversal a command-line option, so we'd just end up with two scripts f5convert and f5clean

Alright, that works for me

@A-CGray
Copy link
Contributor Author

A-CGray commented Jul 6, 2023

I think this is good to go now @timryanb, but maybe check that everything works as expected on your machine

Copy link
Collaborator

@timryanb timryanb left a comment

Choose a reason for hiding this comment

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

Let's give @gjkennedy a chance to review

@gjkennedy gjkennedy merged commit 888061a into master Jul 28, 2023
@gjkennedy gjkennedy deleted the f5totecHelpers branch July 28, 2023 13:39
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.

3 participants