deephaven / deephaven-core

Deephaven Community Core
Other
235 stars 79 forks source link

Restore deephaven-learn python tooling #1133

Open niloc132 opened 2 years ago

niloc132 commented 2 years ago

CC @chipkent @jmao-denver @jjbrosnan

Unfortunately, https://github.com/deephaven/deephaven-core/pull/974 had to be reverted (in https://github.com/deephaven/deephaven-core/pull/1096) as the build broke after it merged. Internal slack discussion found that other tests were failing, this ended up being due to #1067 having merged earlier, but not been merged into the #974 branch, so the failure was not detected as part of the pull request.

A two step process is proposed to fix this:

chipkent commented 2 years ago

Updating status.

1) Learn library without dependencies has gone in. #1140 2) Learn library dependencies have been blocked by base+ image work. @niloc132, @devinrsmith, or @jjbrosnan can comment on status. 3) We still need to run testing on the base+ images.

I'm going to assign to @devinrsmith and @niloc132, since their work has been blocking.