Results 16 comments of Charbel Jacquin

> nice, this works for me with Orthanc thanks for starting the JSON to Dicom decoder, I will see if I can add to this > > I wonder why...

> Thank you for reaching out, @vsaase. It is true that this project is currently not covering conversions to/from JSON, but it makes sense to have that eventually, potentially as...

@Enet4 @vsaase please have a look on the last commit of #174

Hum, this might be fixed with one of my pending PR (the one about instant feedback). Hum, actually not, but I rewrote the sound preview here for similar reasons (playing...

> > Hum, this might be fixed with one of my pending PR (the one about instant feedback). Hum, actually not, but I rewrote the sound preview here for similar...

> * Open the `SampleEditor` -> enter a large value in the spinbox of the "Start" and press enter -> both the values in the spinboxes of "Start" and "Loop"...

> Hi, is this causing any actual problems? > > The library isn't currently expected to be used by anything other than the tools included with hydrogen itself. It's been...

> Hey @charbeljc, > > We finally managed to talk about this PR internally. Sorry for the delay. > Hi @theGreatWhiteShark no worries about the delay, I understand that introducing...

> Ah, I see. We mistook the testing use case for an arbitrary example instead for the main purpose. > > So, `pytest` is so much more powerful than the...

> Ah, I see, this makes a bit more sense of the motivation. Thanks for clarifying! > > I think the lack of existing tests speaks more to the nature...