dodon4 / vtkvisit Goto Github PK

@Dodon4

1. просьба по возможности придерживаться стандарта https://www.python.org/dev/peps/pep-0008/ - примеры хорошего python - кода тут https://github.com/dryuri92/openmc/blob/develop/openmc/trigger.py;
2. непонятно, где ты читаешь картограмму, делать это необходимо, вышлю тебе несимметричный случай, чтобы проводить тесты на нем;
3. про уменьшение размерностей, будем полагаться на 2 вещи:

1. пользователь знает что-то о python
2. ты лишь отслеживаешь, что его индексация верная, то есть получившийся результат пригоден для отрисовки, т.е массив одномерный, не происходит выходов за пределы и если хотим отрисовать картограмму, то проверить соответствие с нумерацией.
   .. пример приведу в pull request

4. добавить какие-либо опции к отрисовке, например, логарифическое масштабирование, можно посмотреть как в VisiT реализованы разные варианты закраски.

В целом уже намного лучше, правильно, что разделил реализацию и тестовую часть, но все таки надо поработать над индексацией более внятной, сейчас многовато функций для отображения

Вот так делать не то что нельзя, это противозаконно
def _read_cart(self):#читаем картограмму self.readcart = reader() self.data_cart = self.readcart._read_cart("./Cart.txt")
Входные данные для того и входные, что подаются извне, здесь необходима явная интерфейсная функция для входа

в репозитории должен быть source.py а не файл jupiter!
Что имел в виду под индексацией. Функции должно быть 2:
plot_1d(data, ind, **kwargs) и plot_2d(data, ind, **kwargs) отличие в них в обработке данных на ошибки. В случае одномерных данных ind может быть None. Под kwargs подразумеваю опции отрисовки какие-либо.
Как это работает. Пусть у нас есть одномерный, двумерный и трехмерные массивы
arr1d = np.random.rand(10) arr2d = np.random.rand(100).reshape(10, 10) arr3d = np.random.rand(1000).reshape(10, 10, 10)
Тогда для одномерного случая отрисовка будет вестись для массива result:
ind = [1, 5, 8] if ind is None: result = arr1d else: result = arr1d[ind] array([ 0.31664533, 0.62822625, 0.8490838 ])
Двумерный случай для всей "строки":
ind = (2,) result = arr2d[ind] array([ 0.64820333, 0.11194976, 0.78621426, 0.80403238, 0.36598413, 0.30349365, 0.69646116, 0.9769087 , 0.9635891 , 0.20261581])
Двумерный случай для некоторого набора в третей строке :
ind = (2,[2, 5, 8]) result = arr2d[ind] result array([ 0.78621426, 0.30349365, 0.9635891 ])
По сути тоже самое и в трехмерном случае:
` ind = (2, 3)
result = arr3d[ind]
result
array([ 0.53312721, 0.23668674, 0.46408177, 0.12980727, 0.70448937,
0.34130949, 0.45726152, 0.45339598, 0.33126448, 0.10061749])

ind = (2, 3, [1, 4, 5])
result = arr3d[ind]
result
array([ 0.23668674, 0.70448937, 0.34130949])`
То есть result - это тот самый массив, который отправляется на отрисовку на картограмму или график, а ind - задается пользователем, ты проверяешь между этими действиями