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

Cargo containers should be bays, not equipment #1594

Open
Saklad5 opened this issue Aug 18, 2024 · 8 comments
Open

Cargo containers should be bays, not equipment #1594

Saklad5 opened this issue Aug 18, 2024 · 8 comments
Labels
(RFE) Enhancement Requests for Enhancement, new features or implementations

Comments

@Saklad5
Copy link
Contributor

Saklad5 commented Aug 18, 2024

According to page 239 of the sixth printing of TechManual, Cargo Containers are a type of transport bay. This is a significant distinction, as it means that bay doors may be allotted to containers specifically.

It also means that MegaMekLab should allow designers to adjust the quantity of cargo containers in the "Transport Bays" tab, rather than requiring them to add containers one by one. That can become quite tedious when adding a large number of containers, and having dozens makes the "Assign Criticals" tab quite a bit harder to read.

@pavelbraginskiy
Copy link
Member

What unit types is this about? CVs, SVs, SCs, all handle transports somewhat differently.

@pavelbraginskiy pavelbraginskiy added the Waiting on Author This issue is waiting on a response from its author. label Jan 22, 2025
@Saklad5
Copy link
Contributor Author

Saklad5 commented Jan 22, 2025

What unit types is this about? CVs, SVs, SCs, all handle transports somewhat differently.

No they don't. All of those units handle bays the same way, with the only distinction being whether they have slot limits occupied by bays.

@pavelbraginskiy
Copy link
Member

All of the units that can carry cargo have very different interfaces in MML, so I'm not sure what units this suggestion refers to:

It also means that MegaMekLab should allow designers to adjust the quantity of cargo containers in the "Transport Bays" tab, rather than requiring them to add containers one by one.

@Saklad5
Copy link
Contributor Author

Saklad5 commented Jan 22, 2025

All of the units that can carry cargo have very different interfaces in MML, so I'm not sure what units this suggestion refers to:

It also means that MegaMekLab should allow designers to adjust the quantity of cargo containers in the "Transport Bays" tab, rather than requiring them to add containers one by one.

I mean that containers should be a row in the Transport section of the Structure/Armor tab for Combat Vehicles, and an option in the Available Bays section of the Transport tab for other units.

@pavelbraginskiy pavelbraginskiy added (RFE) Enhancement Requests for Enhancement, new features or implementations and removed Waiting on Author This issue is waiting on a response from its author. labels Jan 22, 2025
@psikomonkie
Copy link
Collaborator

psikomonkie commented Jan 22, 2025

If equipment can transport they should probably implement Transporter in some way. This would integrate them with the rest of MegaMek's loading stuff.

EDIT: Have a look at CargoBay.

@Saklad5
Copy link
Contributor Author

Saklad5 commented Jan 22, 2025

Now that I think about it, "Cargo" and "Liquid Storage" are in the same boat: they're actually transport bays, and only implemented as equipment for the sake of 'Mechs. So they really shouldn't be allowed on other unit types.

@pavelbraginskiy
Copy link
Member

So as far as I understand, the Cargo and Cargo Container equipment should be made unavailable to non-mech units (CVs should instead use the Transport section of the Structure tab, SVs should instead use the Transport tab), and there should be a new kind of Cargo Container bay added for CVs, SVs, and DSs, while keeping the existing Cargo and Cargo Container equipment for Meks.

I got that all right?

@Saklad5
Copy link
Contributor Author

Saklad5 commented Jan 22, 2025

Basically. Fighters and Small Craft get transport bays too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(RFE) Enhancement Requests for Enhancement, new features or implementations
Projects
None yet
Development

No branches or pull requests

3 participants