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

Incomplete Shipoutfit file export of EDMC.exe #2228

Closed
Eventure-TCE opened this issue May 7, 2024 · 11 comments · Fixed by #2231
Closed

Incomplete Shipoutfit file export of EDMC.exe #2228

Eventure-TCE opened this issue May 7, 2024 · 11 comments · Fixed by #2231
Assignees
Labels
Projects
Milestone

Comments

@Eventure-TCE
Copy link

Eventure-TCE commented May 7, 2024

Hello Rixxan,

I received a report from a user of TCE that references the outfitting export file for the ship in use where jump drive data is missing. I think this has to do with the new drives, which are not yet fully mapped by EDMC.exe.

Here is the incomplete export text:
[Vulture]
L: 3C/Gimballed Beam Laser
L: 3C/Gimballed Beam Laser
U: 0D Frame Shift Wake Scanner
U: 0I Point Defence
U: 0I Chaff Launcher
U: 0F Electronic Countermeasure

BH: 1I Lightweight Alloy
RB: 4A Power Plant
TM: 5C Thrusters
FH: 3C Frame Shift Drive
EC: 3C Life Support
PC: 5A Power Distributor
SS: 4C Sensors
FS: 3C Fuel Tank (Capacity: 8)

MC: 5D Module Reinforcement Package

5: 5E Cargo Rack (Capacity: 32)
4: 4C Shield Generator
2: 1B Collector Limpet Controller
1: 1A Fuel Scoop
1: 1C Frame Shift Drive Interdictor
1: 1E Supercruise Assist
1: 1E Advanced Docking Computer

Cargo : 32 T
Fuel : 8 T
Mass : 339.70 T empty
379.70 T full

The range information (unladen/laden) is missing under the mass information.

@Eventure-TCE Eventure-TCE added bug unconfirmed An unconfirmed bug labels May 7, 2024
@Rixxan
Copy link
Contributor

Rixxan commented May 7, 2024

Which EDMC cmdline flag does TCE use for that? I remember last time you mentioned you could replicate with EDMC.exe --loglevel DEBUG -o %TEMP%\test.csv. Will that also replicate here?

@Rixxan
Copy link
Contributor

Rixxan commented May 7, 2024

Issue ID'd by using the -e flag.

Cause of issue is that the modules are not in the Coriolis data files which is what feeds our internal reckoning. Will see what workarounds I can make :)

@Eventure-TCE
Copy link
Author

Yes, it's the -e flag.

Maybe you add a zero for unladen and laden, like

Range : 0 LY unladen
0 LY laden

as long as no data is available, if you don't have another solution up your sleeve.
Hope that they can fill the gap at some point.
Thanks for your help, again!

@Rixxan Rixxan removed the unconfirmed An unconfirmed bug label May 7, 2024
@Rixxan Rixxan self-assigned this May 7, 2024
@Rixxan Rixxan added this to Needs triage in Bug Triage via automation May 7, 2024
@Rixxan Rixxan moved this from Needs triage to High priority in Bug Triage May 7, 2024
@Rixxan Rixxan added this to the 5.11.0 milestone May 7, 2024
@Rixxan
Copy link
Contributor

Rixxan commented May 7, 2024

So this isn't going to be an instant fix. To provide the output of the ship's range and loadout (EDSM.exe's -e flag), the program needs to know all the available details. We generally source this data from Coriolis's files and read the journal files.

Unfortunately, there are two bits of data not present in the game data we have access to that seem to preclude us from just adding them ourselves - fuelmul and fuelpower. Historically, the community has reverse-engineered these values to be constants based on the size and rating of the drive. However, when I use the historical numbers for these formulas, they do not match what I observe in the game. Therefore, it is likely that either my math sucks (possible) or FDEV has altered these numbers for the new SCO drives.

Without these updated numbers, we won't be able to determine the exact numbers of your laden, unladen, and other jump drive details. As such, we'll be filling these values with temporary placeholders of 0.0 if we can't figure out the range from the existing data.

Of course, as these numbers are known we'll add the exact info. (Currently, I think there are 43 modules whose details are unknown to Coriolis and need these numbers to get added, and of course, anything added in this update may not be initially supported such as any Python Mk II specific modules like armor) There may be bugs when encountering Python Mk II specific modules or the new FSDs, among other details.

I will continue to add things as I find them. Look for a 5.10.5 update to come out sometime in the next few hours to at least suppress these issues and add some of the data we know. Further updates will come as more information becomes available.

@Rixxan
Copy link
Contributor

Rixxan commented May 7, 2024

@Eventure-TCE Is the language of "X.X LY Laden" exactly expected, or simply to have some content there? Because the "Laden" and "Unladen" output by EDMC seem to be misnomers - they assume maximum fuel and only differ based on if cargo is present. I want to correct that but not if it'll break TCE Or other readers.

@Eventure-TCE
Copy link
Author

Eventure-TCE commented May 7, 2024

As long as only the values change, but not the name, you can adjust it from my part of view. Otherwise, I would just have to know the new name in advance and I would then adapt TCE accordingly, if that is what you have in mind.

@Rixxan
Copy link
Contributor

Rixxan commented May 7, 2024

Potential minor change:
https://github.com/HullSeals/EDMarketConnector/blob/690523c9169a88c90fd5b7073c898e216cd8551d/edshipyard.py#L181-L182

but it seemed to be okay, the program seemed to adapt cleanly. If that does break something let me know and I'll revert it in the next release.

@Rixxan
Copy link
Contributor

Rixxan commented May 9, 2024

EDCD/coriolis-data#98 contains the information we need. Once that's merged in we can pull the data to EDMC and release an update with SCO module support.

Rixxan added a commit to HullSeals/EDMarketConnector that referenced this issue May 9, 2024
@Rixxan
Copy link
Contributor

Rixxan commented May 9, 2024

As long as only the values change, but not the name, you can adjust it from my part of view. Otherwise, I would just have to know the new name in advance and I would then adapt TCE accordingly, if that is what you have in mind.

Update, this did break things. Will revert in 5.10.6. (See e5f99c2)

@Eventure-TCE
Copy link
Author

Thank you for the tip and nice to hear that you have now received the missing data. I have a TCE update in the pipeline with a new, configurable interface for EDMC. I'll wait for your release for a final check as I've rewritten the file's reader which should work with both versions so nothing breaks on my end.

@Rixxan
Copy link
Contributor

Rixxan commented May 11, 2024

Released in 5.10.6

@Rixxan Rixxan closed this as completed May 11, 2024
Bug Triage automation moved this from High priority to Closed (Fixed) May 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Bug Triage
Closed (Fixed)
Development

Successfully merging a pull request may close this issue.

2 participants