Douglas Blank
Douglas Blank
@Nick-Hall, I think we can go ahead and merge this PR (with above tweak) because: 1. It is about 70% faster than before the PR 2. It fixes a bug...
@Nick-Hall, I see in one place in the code where it assumes that all Events have a Date. With the new format is seems that the Date may be None....
@Call-Me-Dave, this is actually a major update of existing functionality. But what might come next is the really exciting stuff! With this change, and changes to the DB, we can...
> By the way - since TEXT is hard-coded in DBAPI, we cannot use [native JSON types](https://www.postgresql.org/docs/current/datatype-json.html), correct? One of the many aspects we learn as we port to different...
I may have misunderstood your question: I think Postgresql can use whatever type you want, as long as it acts like a string in Python.
> It will get more interesting/complicated if/when we start to query into the JSON structure itself That is on the schedule for 6.1 (soon!). We should coordinate now to make...
I assume that the base branch should be master if this is milestone v6.1
I was just going to open an issue on this point. I've used jupyterlab for just a few minutes, but the location of the console input is not just surprising,...
Nice summary of the issues. I hope to play around with some of these ideas soon. Might even be a research topic!
Good news everyone! The [JavaScript PR](https://github.com/comet-ml/opik/pull/1197) has been merged, and we will be adding a specific doc page for it shortly.