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

basket.py #262

Merged
merged 53 commits into from
Mar 15, 2022
Merged

basket.py #262

merged 53 commits into from
Mar 15, 2022

Conversation

Ksteeds
Copy link
Contributor

@Ksteeds Ksteeds commented Feb 18, 2022

Development & completion of Basket Class.

@Ksteeds Ksteeds self-assigned this Feb 18, 2022
@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 18, 2022

Intermediary stage.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 18, 2022

Believe code is complete - requires testing.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 18, 2022

Initial testing complete.

@Ksteeds Ksteeds changed the title Basket_Class basket.py Feb 18, 2022
@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 21, 2022

Transitioning on to the Unit Test redesign.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 22, 2022

Intermediary commit - will potentially fail.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 23, 2022

Unit Test complete assuming there are not any hidden bugs.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 23, 2022

Changes made to make_relative_value unit test can be ignored.

@Ksteeds Ksteeds linked an issue Feb 23, 2022 that may be closed by this pull request
@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 24, 2022

Target Positions is nearing completion.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 28, 2022

Following discussions / comments - changes are required to target_positions.py

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Feb 28, 2022

basket.py complete.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Mar 1, 2022

target_positions.py complete.

Will require an extensive Unit Test.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Mar 4, 2022

Latest version of Basket.py file required for target_positions.

Merged the pull requests.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Mar 15, 2022

Final changes to unit_target_positions being made.

@Ksteeds
Copy link
Contributor Author

Ksteeds commented Mar 15, 2022

Work complete following ToDo assignments.

assert isinstance(df_c_wgts, (list, pd.DataFrame)), error_4

basket_names = []
ctypes_baskets = ctypes + basket_names
Copy link
Contributor

Choose a reason for hiding this comment

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

This is equivalent to ctypes_baskets = ctypes.


basket_names = []
ctypes_baskets = ctypes + basket_names
if baskets:
Copy link
Contributor

Choose a reason for hiding this comment

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

baskets is of type baskets: dict = None, not bool

@rsueppel rsueppel merged commit 895e265 into develop Mar 15, 2022
@Ksteeds Ksteeds deleted the basket branch March 16, 2022 09:37
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.

CategoryRelations xcat_trims argument with single None gives empty df
2 participants