forked from petrobras/ccp
-
Notifications
You must be signed in to change notification settings - Fork 1
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 data processing #5
Open
raphaeltimbo
wants to merge
576
commits into
main
Choose a base branch
from
add-data-processing
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
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
Owner
raphaeltimbo
commented
May 15, 2023
- Fix library name
- Replace master with main in the docs
- Add processing module to filter data
When we run the script from excel the log would be saved to the documents folder.
This adds an option to select a different anaconda environment.
Xlwings is used with the xl files in the scripsts folder.
This file can be used to update the code with git.
If a file is modified the merge will fail, so we have to restore all files before we try to make the git pull.
This reverts commit dfa0e36.
Add point class for back-to-back compressor.
Since we split arg_names in "_", it is not possible to automatically check for arguments with flow_v or flow_m in their name.
This method allow us to write the toml to a string. This is useful to send this string to a .zip file directly without the need to save the file on disk.
The use of pickle can be unsafe, so we had to replace it with the toml files.
The .ccp files now have the compressor objects serialized to a .toml file instead of a pickle one due to security issues.
When converting an impeller to a different suction condition, the value of volume_ratio_ratio is kept constant and a new speed is found. This speed would have a different value from the speed in the original curve. This is not always what the user wants, since some curves would even go out of the compressor speed range. With the speed='same' option, the impeller is converted and them new curves are generated by interpolating the converted curves to the speed values of the original impeller.
Ubuntu 18.04 is deprecated.
The use of pickle can be unsafe, so we had to replace it with the toml files.
Before moving the whole app to a multipage app, we serve a main page with links to each app.
This adds help messages that the user can see when they hover the mouse over the question mark next to the rendered markdown.
The processing module will have functions that will be used to process data to be used in performance calculation. For now, the main use is to filter a DataFrame based on maximum fluctuation values as per ASME PTC 10.
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.