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 Literature References #20

Merged
merged 2 commits into from
Dec 4, 2023
Merged

Add Literature References #20

merged 2 commits into from
Dec 4, 2023

Conversation

Scienfitz
Copy link
Collaborator

added some literature references where its appropriate but was missing

Ideally we can have last push form everyone to add references in case there are more places where stuff from other work was used or strongly inspired

@Scienfitz Scienfitz changed the title Add Literature Refernces Add Literature References Dec 1, 2023
@Scienfitz Scienfitz self-assigned this Dec 1, 2023
Copy link
Collaborator

@AVHopp AVHopp left a comment

Choose a reason for hiding this comment

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

Since I basically only worked on the Naive Hybrid which was not sufficiently inspired by a single reference and on the documentation which is already referenced by the footers in the documentation itself, I do not think that I need to add anything here.

baybe/utils/dataframe.py Show resolved Hide resolved
@Scienfitz Scienfitz added the enhancement Expand / change existing functionality label Dec 4, 2023
examples/Backtesting/full_lookup.py Show resolved Hide resolved
baybe/utils/dataframe.py Show resolved Hide resolved
@Scienfitz Scienfitz force-pushed the repo/literature_references branch from 1c6b257 to 7e422b2 Compare December 4, 2023 12:52
@Scienfitz Scienfitz force-pushed the repo/literature_references branch from 7e422b2 to 6fedf3d Compare December 4, 2023 13:58
@Scienfitz Scienfitz merged commit 4c173e2 into main Dec 4, 2023
8 checks passed
@Scienfitz Scienfitz deleted the repo/literature_references branch December 4, 2023 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Expand / change existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants