Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
meganrm opened a pull request on allen-cell-animated/cell-line-catalog
add conversion script
i started added the genes and the basic info for the cell lines to the new cell line catalog
meganrm pushed 1 commit to feature/conversation allen-cell-animated/cell-line-catalog
- add conversion script bf8027d
meganrm pushed 1 commit to feature/normal-cell-lines allen-cell-animated/cell-catalog
- update filter a02eeed
meganrm pushed 1 commit to feature/normal-cell-lines allen-cell-animated/cell-catalog
- change status of normal cell lines c7ce66c
meganrm pushed 1 commit to feature/normal-cell-lines allen-cell-animated/cell-catalog
- make text smaller a7a9f20
meganrm created a branch on allen-cell-animated/cell-catalog
feature/normal-cell-lines - Cell catalog 2.0 [beta]
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
I realized after I wrote this that it's a styled component. But from line 239 to 256 could def be its own function within this file if not its own component. especially since there are 4 places you...
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
this hint doesn't make sense with the table is empty
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
also could be it's own file
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
why do these need to be two lines? couldn't it be `<CheckOutlined /> "Done editing" : <TagOutlined /> "Create and edit"? also since it's already it's own component, why not send in `isAnnotationMod...
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
take it or leave it: push `currentLabelIdx !== null` check into `setLabelColor` so you can just call it without always having to wrap it in an if statement
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
can you type `hex` here instead of coercing it below?
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
for readably it would be good to move this and the `onChange` on 277 into named functions like you did on 306. also is there a reason 277 is an arrow function and this isn't?
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
are these names enforced somewhere?
meganrm created a review comment on a pull request on allen-cell-animated/timelapse-colorizer
nit: should "ff00ff" be a constant?
meganrm created a review comment on a pull request on mesoscope/cellpack-client
same with all the other strings, like "example_recipes", and "configs".