fabilab / cell_atlas_approximations_HI

Human interface for cell atlas approximations
0 stars 1 forks source link

Handle invalid genes errors and generate dotplot for lacustris' markers #38

Closed YingX97 closed 10 months ago

YingX97 commented 10 months ago

Hi Fabio,

Here are the updates:

Q:"_Show 10 markers of myopeptidocyte in slacustris whole".

Since some of the markers have weird names (e.g:'c95490_g21-to-1slc31a2solutecarrierfamily31member2'). Although it returns the markers, but failed to make an api call to the dotplot endpoint, and gives invalid_parameter error.

I have updated the code to handle this error, and generate dot plot for the rest of the marker.

I think it's ready for merging.

iosonofabio commented 10 months ago

I'm already fixing that 😉

On Thu, Nov 23, 2023, at 13:21, YingX97 wrote:

Hi Fabio,

Here are the updates:

Q:"Show 10 markers of myopeptidocyte in s_lacustris whole".

Since some of the markers have weird names (e.g:'c95490_g21-to-1slc31a2solutecarrierfamily31member2'). Although it returns the markers, but failed to make an api call to the dotplot endpoint, and gives invalid_parameter error.

I have updated the code to handle this error, and generate dot plot for the rest of the marker.

I think it's ready for merging.

You can view, comment on, or merge this pull request online at:

https://github.com/fabilab/cell_atlas_approximations_HI/pull/38

Commit Summary

• 3b27408 https://github.com/fabilab/cell_atlas_approximations_HI/pull/38/commits/3b2740844099f841d48cc6d645717b97d1b68c52 fix bug in marker genes (sponge) • 8935ee0 https://github.com/fabilab/cell_atlas_approximations_HI/pull/38/commits/8935ee0e39f9e4d47e17a6ce49544828caeb2c6f remove console.log File Changes

(1 file https://github.com/fabilab/cell_atlas_approximations_HI/pull/38/files)

• M src/utils/chatSideEffects.js https://github.com/fabilab/cell_atlas_approximations_HI/pull/38/files#diff-556552af24c01aae05484f6c19de80cd1d89c0078eedffaa4c3e9563ef025d6e (15) Patch Links:

• https://github.com/fabilab/cell_atlas_approximations_HI/pull/38.patch • https://github.com/fabilab/cell_atlas_approximations_HI/pull/38.diff — Reply to this email directly, view it on GitHub https://github.com/fabilab/cell_atlas_approximations_HI/pull/38, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJFEAHA6J424UEKE3YDS6TYF2XLXAVCNFSM6AAAAAA7XDAWIKVHI2DSMVQWIX3LMV43ASLTON2WKOZSGAYDOMZYGE3DIMY. You are receiving this because you are subscribed to this thread.Message ID: @.***>