-
Notifications
You must be signed in to change notification settings - Fork 18
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
FLASC v1.4 #129
Merged
FLASC v1.4 #129
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
…/flasc into feature/port_to_polars_4
Feature/add options on count computation in energy ratio
* Remove unused var and clean a little * Split the energy ratio calculation a little * Clarify test comments * Add check of df_freq and specify datatype binning * Update test for passing in frequency * Allow passing in of frequency table * Change count weight to weight * Include details of missed bins * Add enforcement all ws/wd appear in all dataframes * Redo tests and add for missing bin on one side * Add more tests for df_freq * Add checks on ws/wd missing on df_freq * Add tests * Polars deprecating groupby to group_by * Computing df_freq explicitly. * Clearer about resampling on first bootstrap call. * df_freq now wired through to Output object. * Compute ws freqs alone, if needed. * Now handling uplift plots correctly. * Option to overlay df_freq weighting. * Renaming to frequency for clarity. * Remove commented code and nomralize weights * Update comment * Rerun example notebook. --------- Co-authored-by: misi9170 <[email protected]>
…warnings when NaNs occur.
Add calculations for total uplift
* Initiation of branch * Lower the noise on this example * Incremental progress * Update noteboook * Add emgauss models * Add support changing fi parameters within call * sort imports * small updates * small changes * Sort imports * Small cleaning * Add a function to replicate nan values * Add a test of replicating nan values * Add docstring * Remove redundant class * Add test script * Implement more of tuning script * Update notebook * Update test script * Update utils * Update tuner * Add a tuning notebook * Add option for absolute uplift * Use absolute uplift * Chain through absolute uplift * Updates * updates * Update notebook * refactoring * Building out pseudocode for the wake loss eval case. * Move step of generating df_fi into the specific evaluators, if required. * setting up floris candidates. * Commit to remerge develop. * small refactoring * refactoring * A little progress * Small edits * updates * fix check call * re-run * Updates * some clean up * Starting smarteole version * yaw right side * yaw one side * One way steering applied. * Typo fix. * Updated emgauss defaults; artificial tuning notebook running as expected. * Adding small readme. * Point users to existing pp fitting. * Removing sklearn dependency. * Fix yaw angles * Delete floris tuner * Update examples * Rename artificial example * Update examples --------- Co-authored-by: misi9170 <[email protected]>
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.
What's Changed
Full Changelog: v1.3.1...v1.4