Remi Perenon
Remi Perenon
Hi Dominik, some questions / remarks: - Would it be feasible to use directly the list of possible operators defined in the API instead of defining `possibleOperators` ? - Would...
Hello, Sorry for the delay. We reviewed your pull request with @MathieuTillet @gui-co @eurydice76 and Miguel. We have numerous comments. **1- The most important comment here is that you created...
We totally get that. Just remember that your code is an algorithm and not a loader! In that case an important question is: do you need to merge this PR...
Hello everyone! Concerning the question about merging or not, it was just a (maybe clumsy) way to say that at some point you will have to go through a similar...
Hello, Some feedback: - We think the length of the `data_x` should be `self.spectrum_amount()+1` instead of `2*self.spectrum_amount()`, because you need to have `N+1` values of "X" if you have `N`...
Works also with [this script](https://docs.mantidproject.org/nightly/tutorials/python_in_mantid/solutions/03_pim_sol.html#a-direct-matplotlib-with-sns-data) ``` # import mantid algorithms, numpy and matplotlib from mantid.simpleapi import * import matplotlib.pyplot as plt import numpy as np # Load the data run...
Hi Andrei. Thanks, that solves the whole problem for the two issues described above. Apparently, there is a PR [here](https://github.com/mantidproject/mantid/issues/36938) to change the code generation in that way. We should...
In [this page](https://developer.mantidproject.org/Testing/Core/Core.html) (referenced in the main documentation) the link to "3D and volumetric data" goes directly to [matplotlib documentation](https://matplotlib.org/stable/plot_types/3D/index.html#d-plots) Also, the data used in [this section](https://docs.mantidproject.org/nightly/tutorials/python_in_mantid/plotting/01_basic_plot_scripting.html#d-surface-and-wireframe-plots) `PG3_733.nxs` does not...
Hi Danny, thanks for your comment. On ILL side, I take back the responsibility of this PR. The RawDataExplorer widget can be put behind the workspace one if you want...
Considering the discussions of [this thread](https://app.slack.com/client/T02J4MMEW/C02J4MMGA/thread/C02J4MMGA-1679406837.530229), I change the reviewer to Tom. Indeed, since this PR is large and has a huge impact, it would be nice of someone at...