search

CARTAvis / ICD-RxJS

This is the carta-backend ICD tests based on the RxJS library
1 stars 0 forks source link

Add new test for stream region_histogram.proto #15

Closed acdo2002 closed 1 year ago

acdo2002 commented 1 year ago

Based on the Mark's new cube histogram PR. protobuf PR backend PR

There is a new sub-message under regionHistogramData(Stream), so we need at least one new ICD test to test this sub-message (modify the current ICD test is another option, I will check which way is better)

acdo2002 commented 1 year ago
acdo2002 commented 1 year ago

@kswang1029 I add one test (SET_HISTOGRAMCONFIG.test.ts) to test sub-ICD message: HistogramConfig (sub-message) Including set_histogram, set_channel, and re_set_histogram. Please check the document (link) and please let me know is there any test can be improved.