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.

Sign up for GitHub

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

Jump to bottom

Major Update: Replace Docker with virtual environments and update versions #117

Merged
merged 51 commits into from
May 8, 2024
Merged

Major Update: Replace Docker with virtual environments and update versions #117

merged 51 commits into from
May 8, 2024

Conversation

chipkent
Copy link
Collaborator

@chipkent chipkent commented Apr 25, 2024 •
edited
Loading

A major update to how deephaven-ib is built and deployed plus relevant version updates.

  • Java 11 -> 17
  • IB 10.19.01 -> 10.19.04
  • Deephave 0.28.1 -> 0.34.x
  • Updated examples.
  • Major build and deployment changes. No more docker.
  • Script to create properly configured python virtual environments to run deephaven-ib from.
  • Now use the most recent compatible deephaven version instead of the specific version listed.
  • Fixed the short rate downloader. IB had changed the FTP server.
  • Fixed broken bars_historical calculation.
  • Better error messages.

@chipkent
DH version to 0.33.3
15811a9 
Java version to 17
@chipkent chipkent self-assigned this Apr 25, 2024
chipkent added 23 commits April 25, 2024 15:27
@chipkent
Updated the ib api version to 10.19.04
780185e
@chipkent
Streamline and simplify the IB artifact building.
65b9ea9
@chipkent
Added a script to build deephaven-ib virtual environments.
4d8f657
@chipkent
Made IB wheel building fail on errors.
482a2bb
@chipkent
Updated to server-ui deephaven images.
527e3d7
@chipkent
Debug CI
bdc638d
@chipkent
Debug CI
6a8781d
@chipkent
Debug CI
91972ea
@chipkent
Debug CI
5d1525d
@chipkent
Removed the server-ui change since it requires 0.34
f906127
@chipkent
Made the versions easier to update.
ba169a3
@chipkent
Remove legacy docker
9be1e6c
@chipkent
Updated documentation.
b82bb72
@chipkent
Updated documentation.
01e0ab5
@chipkent
Made ib version handling robust.
eef03c0 
Fixed release build.
@chipkent
Removed all docker requirements.
122a7c7 
Replaced IB docker build with local functionality in the script.
@chipkent
Release the hard version constraint on deephaven.
70c5cb0
@chipkent
Updated the minimum python version to avoid core dump in jpy.
0b51312 
deephaven/deephaven-core#5440
@chipkent
Fix broken dependency version handling.
b44eab1
@chipkent
Fix broken short rate downloads.
7aa5b8c
@chipkent
Fix DH queries that are broken.
2905ca6 
Better error messages.
@chipkent
Fix DH examples.
26fe2d5 
Add a script to start and run deephaven.
@chipkent
Add an option to create a venv for use in pycharm.
018d8f0
@chipkent chipkent added bug Something isn't working enhancement New feature or request labels May 1, 2024
@chipkent chipkent changed the title Update DH version to 0.33.3 and Java to 17 Major Update: Replace Docker with virtual environments and update versions May 1, 2024
alexpeters1208
alexpeters1208 reviewed May 7, 2024
View reviewed changes
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
examples/run_deephaven.py Outdated Show resolved Hide resolved
alexpeters1208
alexpeters1208 approved these changes May 8, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
@chipkent chipkent merged commit 89d9e73 into main May 8, 2024
8 checks passed
@chipkent chipkent deleted the dh_v0333 branch May 8, 2024 18:46
@github-actions github-actions bot locked and limited conversation to collaborators May 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jjbrosnan jjbrosnan jjbrosnan approved these changes

@alexpeters1208 alexpeters1208 alexpeters1208 approved these changes

Assignees

@chipkent chipkent

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@chipkent @alexpeters1208 @jjbrosnan