Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt to the structure of the UI elements (DronesWindow) #79

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

focabr
Copy link
Contributor

@focabr focabr commented Mar 3, 2023

Adapt examples for EVE Online to changes in the game client.

Adapt to the structure of the UI elements (DronesWindow) found in the sample 'session-2023-03-03T04-09-36-event-(72,110,122)-eve-online-memory-reading.json'.

With this adapt, there is a list of children with group names.
image

I sent 3 session-memory-reading.json

If you have any question, feel free to contact me.

Best regards

…is left for the cluster to shut down.

+ Adapt to the structure of the UI elements found in the sample session-2023-02-27T09-59-39-event-4966-eve-online-memory-reading.json
Adapt to the structure of the UI elements (DronesWindow) found in the sample 'session-2023-03-03T04-09-36-event-(72,110,122)-eve-online-memory-reading.json'.
@focabr
Copy link
Contributor Author

focabr commented Mar 3, 2023

I will need to improve my test cases...

I just found out that the anomaly and industrial mining BOTs crashed after this adapt, the BOT can no longer identify the "idle drone", I still can't identify what is causing it.

@focabr
Copy link
Contributor Author

focabr commented Mar 3, 2023

I sent a new file with the name drones_withgroup_session-2023-03-03T15-54-31-event-66-eve-online-memory-reading.json, it will have the following drone configuration.

Being 10 drones in total, 5 in space as follows.

  • 1 Mining and 1 Combat without a group.
  • 2 Combat within a group called "Combat".
  • 1 Mining within a group called Mining (it does not appear because it was out of sight)

@focabr
Copy link
Contributor Author

focabr commented Mar 3, 2023

Sorry, maybe I took something bigger than me.

After this last adjustment it is correctly bringing the drones and groups, but the hierarchy is still incorrect, the topmostGroupEntry should be "Drones in Space".

Still under investigation.

Sending new files and images.

drone_group_session-2023-03-03T20-28-32-event-56-eve-online-memory-reading.json

image

image

…oups, but the hierarchy is still incorrect, the topmostGroupEntry should be "Drones in Space".
@Viir
Copy link
Collaborator

Viir commented Mar 7, 2023

Thank you very much for the investigation!
I see the drone status in the tree view, which is encouraging!
The hitpoints seem missing on the drone entries, but that might be a consequence of the switch to Photon UI last month.

@focabr
Copy link
Contributor Author

focabr commented Mar 9, 2023

If my investigation is correct, the hierarchy is not working because it was based on totalDisplayRegion.x placement, today the group names are aligned, as shown in the images below.

Drones in Space

image

Combat and Mining Groups

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants