Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
toloudis opened an issue on allen-cell-animated/vole-app
multiscene: load zarr group as scenes
## Use Case In the absence of a mature ome-zarr spec for collections, (see the ro-crate subtask) try to detect if a zarr url holds a group of zarr arrays. ## Solution Can we detect this scenario...
toloudis opened an issue on allen-cell-animated/vole-app
multiscene: Parse csv exported from BFF
## Use Case Allow BFF to determine multiscene collection to load in VolE. ## Solution Get some example data from Antoine. Use BFF to export CSV. Load CSV into app, each row is one scene. use Fil...
toloudis created a comment on an issue on allen-cell-animated/vole-core
Consider adding a different ticket to load a format that can be exported from BFF
toloudis created a review on a pull request on bioio-devs/bioio
LGTM? I also agree about #80 but this is a good enough interim thing
toloudis created a comment on a pull request on allen-cell-animated/agave
> LGTM, my brain hurts I understand it's a totally different language and adding some nontrivial stuff. I can give you a verbal walkthrough on this if interested.
toloudis created a review comment on a pull request on allen-cell-animated/agave
Yes, VertsCode has basically a position and a color and an id number used for mouse selection. They hold the vertices of all the lines we are drawing. And v is a collection of those.
toloudis created a review comment on a pull request on simularium/simularium-viewer
should "start" and "watch" be the same thing? I think what I was going for in my head was to have "start" always watch the source files so you could have automatic updates. It seems like you cou...
toloudis pushed 20 commits to main allen-cell-animated/agave
- add clip plane changes a516843
- try update tensorstore to fix builds f8cb4e7
- Revert "try update tensorstore to fix builds" This reverts commit f8cb4e78e47835d98fe97cbff1937459d68ac37b. 5e76c89
- try continue-on-error to get past cmake generate error 2a3bdf4
- bump min cmake for tensorstore 3224527
- remove continue-on-error c19fa14
- try tensorstore 0.1.69 d138dd5
- Merge branch 'feature/add-clip-plane' of https://github.com/allen-cell-animated/agave into feature/add-clip-plane # ... ac7be36
- bump tensorstore again e2c85a3
- try windows 2019 c47d76e
- try explicit template args 5a0b663
- more explicit array ctor? 8b52f08
- one more try at windows compile f2d5680
- backtrack 3b796ab
- windows-2019 ca53291
- remove assertion 1dc4402
- bump gcc for ubuntu-latest 2fe5ced
- fix linux build compile error 59f87b1
- remove fixed parameter 417d45d
- Merge pull request #230 from allen-cell-animated/feature/add-clip-plane add clip plane changes 2d04cb9
toloudis closed an issue on allen-cell-animated/agave
user clip plane
One additional user-controlled clipping plane can let users see into cross sections of specific interest. Add a clip plane as a grid that can be drawn in viewport. Allow the clip plane to be rotat...
toloudis closed a pull request on allen-cell-animated/agave
add clip plane changes
Resolves #228 Add user clipping plane functionality. * Add SceneClipPlane which represents a clipping plane in the scene. * Add functionality to volume rendering shaders to apply the clipping...
toloudis created a review comment on a pull request on simularium/simularium-viewer
I think this line should basically be "start" and you maybe don't need a separate serve-test-bed, right?
toloudis pushed 3 commits to feature/serialize-clip-plane allen-cell-animated/agave
toloudis created a review on a pull request on bioio-devs/bioio-bioformats
LGTM ... Regarding the bf_version check for 6.7, do we have a min version for bioformats in this reader?
toloudis created a review comment on a pull request on simularium/simularium-viewer
or even css:build and css:watch ! just riffing...
toloudis created a review comment on a pull request on simularium/simularium-viewer
Adding all these things makes me nervous about complexity of build. I wonder if the ones that match could be put close to each other, like `watch:css` next to `processCSS` and maybe rename process...
toloudis created a review on a pull request on allen-cell-animated/timelapse-colorizer
LGTM, I suggest one possible minor refactor - optional!
toloudis created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
is there a more elegant way of doing these little style tweaks or is this pretty standard?
toloudis created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
Do you no longer need a second function `setLabelOnId` since you can just pass single or multiple ids as arrays?
toloudis created a review on a pull request on allen-cell-animated/timelapse-colorizer
LGTM, I suggest one possible minor refactor
toloudis created a review on a pull request on allen-cell-animated/timelapse-colorizer
LGTM, I suggest one possible minor refactor
toloudis created a review comment on a pull request on allen-cell-animated/agave
Sorry, this is noise you can ignore because it should stabilize once tensorstore is settled again. It's just me thrashing on it. This should not be changing this often.