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

Test fixes #548

Merged
merged 77 commits into from
Oct 10, 2022
Merged

Test fixes #548

merged 77 commits into from
Oct 10, 2022

Conversation

lsimonsen
Copy link
Member

Merging into test.

lsimonsen and others added 30 commits June 1, 2022 14:04
Release of bugfix to version 0.0.13
changing Markdown image reference for Logo as per @Magnus167
naive_pnl some changes for efficiency
lsimonsen and others added 22 commits September 29, 2022 20:01
Updated call to sns.lineplot to use "estimator" instead of "ci"
changed  "ci" keyword to "estimator" in sns.lineplot calls in view_timelines.py
* testing config file

* testing config file
Updated in accordance to inputs from @lsimonsen
…rkflow

Adding feature to publish to TestPyPI
* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)
…ive_pnl_528

Feature/create panel for naive pnl 528
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (test@fc37714). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             test     #548   +/-   ##
=======================================
  Coverage        ?   60.19%           
=======================================
  Files           ?       26           
  Lines           ?     2804           
  Branches        ?        0           
=======================================
  Hits            ?     1688           
  Misses          ?     1116           
  Partials        ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Magnus167
Magnus167 previously approved these changes Oct 7, 2022
Copy link
Member

@Magnus167 Magnus167 left a comment

Choose a reason for hiding this comment

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

Summary

Implemented but still needs work

  • macrosynergy/pnl/naive_pnl.py

Fixed bugs

  • macrosynergy/management/check_availability.py
  • macrosynergy/dataquery/auth.py
  • macrosynergy/init.py
  • macrosynergy/management/shape_dfs.py
  • macrosynergy/panel/basket.py
  • macrosynergy/panel/view_timelines.py
  • README.md
  • setup.py
  • tests/integration/dataquery/test_dataquery.py

Added features

  • .github/workflows/python-test-pypi-package.yml
  • .github/workflows/pythons-docs.yml
  • tests/integration/test_testpypi_version.py

* Merging version 0.0.18@develop to 0.0.18@test (#541)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Feature/publish test py pi workflow (#543)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

* adding line to install twine

* Updating readme for extra explanations, also updating config file to include development status. (#540)

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Feature/publish test py pi workflow (#544)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

* adding line to install twine

* Updating readme for extra explanations, also updating config file to include development status. (#540)

* fixed pypi upload url

* changed comments

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Fix merge conflicts

* Fix additional conflicts

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>
* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Updating test repo (#521)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Added feature to divide naive_pnl plots into a facetgrid

* Removed extra blank space

* Removed extra blank space

* Updating readme for extra explanations, also updating config file to include development status.

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

* Merging version 0.0.18@develop to 0.0.18@test (#541)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Updating readme for extra explanations, also updating config file to include development status. (#540)

* Feature/publish test py pi workflow (#543)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

* adding line to install twine

* Updating readme for extra explanations, also updating config file to include development status. (#540)

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Feature/publish test py pi workflow (#544)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* Created workflow YAML for pushing to testPyPI

* naive_pnl some changes for efficiency

* Added worfklow for publishing to TestPyPI

* updated ms/__init__.py to get version from setup.py

* renamed file

* Create test_testpypi_version.py

* added comment

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

* fixed bug arrising due to type errors

* Adjusting the workflow for Test Pypi in line with the latest advice

* Adding the info link

* Updating test repo (#521) (#522)

* Minor bug in view_ranges.

* Unkwown GitHub error.

* Handle GitHub.

* GitHub Python versions.

* PyPi 0.0.18

* naive_pnl some changes for efficiency

* #506 bugfix

changing Markdown image reference for Logo as per @Magnus167

* Updating rst files

* MSY-500 Naive PnL mis-match of shapes

* Update README.md

* Minor spelling mistake + adding documentation link

* Updating documentation website

* Updating documentation website

* Updating documentation website

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>

* changed  "ci" keyword to "estimator" in sns.lineplot calls

* Updated call to sns.lineplot to use "estimator" instead of "ci"

* updated test that checks version numbers

* Summation bug fixed.

* Changed L11 to avoid an extra '/' in the API URLs

* Feature/codecov (#534)

* testing config file

* testing config file

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536)

* Update test_testpypi_version.py

Updated in accordance to inputs from @lsimonsen

* Removed extra blank space

* Removed extra blank space

* Feature/check availability windows (#538)

* Fixing check_availability.py to work on pycharm as well as notebook

* Adding extra days to integration tests to make sure it works when data has not been updated for a while (#536) (#537)

* adding line to install twine

* Updating readme for extra explanations, also updating config file to include development status. (#540)

* fixed pypi upload url

* changed comments

Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

* Updating readme for extra explanations, also updating config file to include development status.

* Updating readme for extra explanations, also updating config file to include development status.

* Updated naive_pnl.plot_pnls() to create a facetgrid

* Fix merge conflicts

* Fix additional conflicts

Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: Lasse de la Porte Simonsen <lsimonsen@macrosynergy.com>
Co-authored-by: Lasse Simonsen <33926514+lsimonsen@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>

Co-authored-by: Palash Tyagi <ptyagi@macrosynergy.com>
Co-authored-by: Ksteeds <csteeds@macrosynergy.com>
Co-authored-by: rsueppel <rsueppel@macrosynergy.com>
Co-authored-by: costam <15220959+mikiinterfiore@users.noreply.github.com>
Co-authored-by: Ksteeds <89909824+Ksteeds@users.noreply.github.com>
Co-authored-by: CyrusYavariMacro <cyavari@macrosynergy.com>
Co-authored-by: CyrusYavariMacro <113984752+CyrusYavariMacro@users.noreply.github.com>
Co-authored-by: MicheleCosta <michele.costa@barcelonagse.eu>
Copy link
Member

@Magnus167 Magnus167 left a comment

Choose a reason for hiding this comment

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

Changes already reviewed

@Magnus167 Magnus167 merged commit db0af75 into test Oct 10, 2022
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.

None yet

7 participants