LeKaitoW / raox

Rao modelling language written in Xbase
http://raox.ru
MIT License
12 stars 26 forks source link

Заливать площадь под графиком #73

Open aurusov opened 9 years ago

aurusov commented 9 years ago

Удобно для графика загрузки. Опциональная возможность. Надо встроить как команду popup-menu.

vbm_lix1leo

qqq

aurusov commented 9 years ago

@LeKaitoW Оля, ты находила настройку, когда изучала графики ?

LeKaitoW commented 9 years ago

Сейчас это настроено в коде: https://github.com/LeKaitoW/rdo-xtext/blob/feature/game5/ru.bmstu.rk9.rdo.ui/src/ru/bmstu/rk9/rdo/ui/contributions/PlotView.java#L139 Думаю, если в стандартном меню этого нет, то можно переписать меню, добавив туда такой пункт. (Когда думали делать экспорт, то столкнулась с проблемой, что в стандартное меню JFreeChart'а нельзя добавлять какие-то пункты, только переписывать меню.)

aurusov commented 9 years ago

Посмотри пожалуйста в стандартном меню.

LeKaitoW commented 9 years ago

Нашла в стандартном меню. Properties... -> Plot -> Appearance -> Background paint.

LeKaitoW commented 9 years ago

Занятость выглядит неплохо (или мне так только кажется), хотя надо бы убрать квадратики как-то: default А вот для графиков типа "количество в очереди" такое лучше не делать: default

Честно, не знаю как сделать это в виде настройки, потому что сейчас я просто исправила метод .createXYStepChart на createXYStepAreaChart вот тут: https://github.com/LeKaitoW/rdo-xtext/blob/feature/game5/ru.bmstu.rk9.rdo.ui/src/ru/bmstu/rk9/rdo/ui/contributions/PlotView.java#L134.

aurusov commented 9 years ago

Уже неплохо. Посмотри как убрать квадратик.

aurusov commented 9 years ago

А почему у загрузки пропали названия по оси Y ?

LeKaitoW commented 9 years ago

Потому что это ещё один забытый минус - параметр занятость_парикмахера является boolean, а не enum. А boolean так и остался со значениями 0 и 1.

LeKaitoW commented 9 years ago

Про квадратики - все методы оказались deprecated, кроме одного: http://www.jfree.org/jfreechart/api/javadoc/org/jfree/chart/renderer/xy/XYItemRenderer.html#setSeriesShape-int-java.awt.Shape- Но этот метод хочет в качестве второго параметра awt-фигуру. Если передать ему null - опять квадратики.

aurusov commented 9 years ago

Потому что это ещё один забытый минус - параметр занятость_парикмахера является boolean, а не enum.

Что у тебя за модель ?

aurusov commented 9 years ago

Если передать ему null - опять квадратики.

Передай ему прямоуголник http://docs.oracle.com/javase/7/docs/api/java/awt/Rectangle.html без ширины и высоты.

LeKaitoW commented 9 years ago

Прямоугольник без ширины и высоты сработал. Но, может, не будем заливать? default

aurusov commented 9 years ago

А что тебя смущает ? Все верно, показана площать под графиком. Фигово, что нет линии, идущей по нулю, но и на двух пред. графиках она отсутствует.

bogachev-pa commented 9 years ago

Если кому-то интересно мое мнение, то заливка без нижней линии выглядит странно, будто график определен не на всей оси, а только на ее частях. Мало того, непонятно как отличить постоянно меняющийся график от заливки (тут что-то может решить прореживание, но его же пока нет), а еще теряются моментальные изменения графика вниз-вверх внутри заливки.

В общем, по умолчанию должно быть явно как раньше. На мой взгляд, можно разрешить пользователю включить такое, конечно, то тогда как минмум, нужно научиться проводить нижнюю линии и научиться как-то отличать скачки внутри области заливки.

aurusov commented 9 years ago

Заливка без нижней линии - это вообще не то, что, нам надо. И да, все говорят правильно. Но есть ли другой режим или придется писать отображение с нуля ? Да еще без прореживания ? Такое слово, как компромис есть в словаре ?

aurusov commented 9 years ago

но тогда как минмум, нужно научиться проводить нижнюю линии и научиться как-то отличать скачки внутри области заливки.

Ну если как минимум, то научи, а я пул-реквест попринимаю.

bogachev-pa commented 9 years ago

Ок, когда закончу более приоритетные задачи, примусь за эту.