Tamas Gal

Results 151 comments of Tamas Gal

I’d also like to add that we should differentiate between unit tests and higher level tests. I write unit tests for my packages but for scientific projects I do not...

> @tamasgal can you explain in more detail what you mean by "high level tests"? Yes sure. Unit tests are tiny bits of tests as you know, so you have...

I am not sure either. It's quite specific to the usecase 😉 Earlier I just used the `test/` folder, which was removed. So now I just create it manually. I...

> How does having a test folder changes things...? You still have to do manually everything :S I never said it does 😉 I just said: earlier it was there,...

> * the goal with a science project is to lead to some result about the world, presented in numbers, tables, figures, etc. > * Ideally, this result would be...

Can you post the output of the code above?

The problem is that this par in `allpix_3a3a_PixelHit` could not be parsed: ```python _raise_notimplemented('TStreamerSTL', "{'_classversion': 4, '_fOffset': 0, '_fName': b'mc_particles_', '_fTitle': b'', '_fType': 500, '_fSize': 24, '_fArrayLength': 0, '_fArrayDim': 0,...

This is the relevant part of the generated interpretation code from `uproot`: ```python allpix_3a3a_Object._readinto(self, source, cursor, context, parent) self._pixel_5f_ = allpix_3a3a_Pixel.read(source, cursor, context, self) self._time_5f_, self._signal_5f_ = cursor.fields(source, cls._format1) self._pixel_5f_charge_5f_...

Have you tried it with `uproot4`? The code-gen was greatly improved in the new version `:)`

Sorry for the very late response, I was very busy and abroad and just got a little time window to look over the notifications. These are awesome news and it...