[Build 136] Better Show differences in Strategy Config

In following picture, the only difference is last 3 lines, but all conditions are shown in red.

Please re-write so that only different parts be shown in red

Attachments
Screenshot 2022-06-09 063911.png
(51.90 KiB)
image-0.png
(83.27 KiB)
  • Votes +3
  • Project StrategyQuant X
  • Type Bug
  • Status Fixed
  • Priority Normal

History

AA
#1

Alex

09.06.2022 05:43

Task created

b
#2

bentra

09.06.2022 12:13
Voted for this task.
AA
#3

Alex

18.06.2022 11:07
Voted for this task.
E
#4

Emmanuel

06.08.2022 15:39
Voted for this task.
g
#5

Lee Guan Chuan

16.08.2022 11:03

Status changed from New to Fixed

AA
#6

Alex

11.10.2022 15:18

Attachment Screenshot 2022-10-11 132643.png added

Screenshot 2022-10-11 132643.png
(59.16 KiB)
Thanks for solving this issue.

It's now much better, and it is much easier to distinguish differences.

By the way, I have two suggestions:


1. Is it possible to separate ranking and test parameters (using a line e.g.)?

2. Is it possible to show each test parameters in one line? It's hard to find difference.

3. As you see, it seems it contains some bugs. (Red parts does not differ from black part)

g
#7

Lee Guan Chuan

12.10.2022 12:49

Attachment image-0.png added

image-0.png
(83.27 KiB)
Hi Alex,


I understand your problem. Unfortunately, I couldn't make it a new line for every parameter. If I do that, some users with many additional markets will have a lengthy page. 


However, I have updated it to display it like this. I hope it can help you identify the differences easily. Please look at the attached image.


Yes. That is a bug. I couldn't reproduce it from my side. Can you please send me a strategy with that strategy config?


Thank you.

AA
#8

Alex

12.10.2022 14:21
Hi Lee.

The work you done is great. It really helps us.

Regarding the bug, I do not know where it was created.

I'll send if I could reproduce it.

Thanks

AA
#9

Alex

17.10.2022 09:50

Attachment Screenshot 2022-10-17 104752.png added

Screenshot 2022-10-17 104752.png
(28.02 KiB)
Hi Lee.

It seems in old version, backtest on additional markets(1) , (2), etc were separated with an empty line, but now as in image-0 they are connected.

Please kindly check it.

g
#10

Lee Guan Chuan

17.10.2022 10:56
Hi Alex,


Sorry for that. My bad. I have added the gap between the lines.


Thank you.

AA
#11

Alex

29.10.2022 09:59

Attachment Screenshot 2022-10-29 105557.png added

Screenshot 2022-10-29 105557.png
(47.19 KiB)
Hi Lee,

It seems some un necessary line spacing is added in [136Dev6].

the first two is OK, but the latter ones are not required.

g
#12

Lee Guan Chuan

31.10.2022 06:01

Attachment image-0.png added

image-0.png
(26.49 KiB)
Hi Alex,


I have updated the gap.




Thank you.

AA
#13

Alex

03.12.2022 04:45

Attachment Screenshot 2022-12-01 071147.png added

Screenshot 2022-12-01 071147.png
(58.00 KiB)
Hi Lee,

Thanks for solving the issue, we can now find differences much easier.

There is another issue that will be great if can be solved.

In attached photo, there is only one difference in filters (one shown by arrow is missing in the right side) but much differences shown than exists.

g
#14

Lee Guan Chuan

04.12.2022 01:50
Hi Alex,


I have updated this.


Thanks.

AA
#15

Alex

20.02.2023 04:38

Attachment Screenshot 2023-02-20 053614.png added

Attachment Screenshot 2023-02-20 052740.png added

Screenshot 2023-02-20 052740.png
(97.07 KiB)
Screenshot 2023-02-20 053614.png
(103.22 KiB)
Hi Lee,

Please note a minor bug when all thing is fine.

there is no space between additional markets.

AA
#16

Alex

18.06.2023 09:48

Attachment diff.png added

diff.png
(84.92 KiB)
A reminder for above comment.

Please look at below screenshot.

At places where we do not have differences, we do not have empty lines which is not good. It s not readable.


Votes: +3

Drop files to upload

or

choose files

Max size: 5MB

Not allowed: exe, msi, application, reg, php, js, htaccess, htpasswd, gitignore

...
Wait please