Coder Social home page Coder Social logo

Comments (9)

KJeff01 avatar KJeff01 commented on May 2, 2024 1

@maxsupermanhd Can you confirm if these problems happen on the latest master?

from warzone2100.

KJeff01 avatar KJeff01 commented on May 2, 2024 1

Closing since trucks don't require an HQ with 9f5168c. Reopen or make a new issue if you find anything else odd happening.

from warzone2100.

Forgon2100 avatar Forgon2100 commented on May 2, 2024

Clearly, that's a bug, likely introduced by #378. Thanks for reporting.

[...] Somebody can build new trucks and somebody can't until command center. [...]

Could you be more precise as to when building trucks is possible while
no HQ has been built? In skirmish games with no HQ being built, I have
only reproduced the absence of truck templates, but not their presence.
Your screenshots give me no additional information about this.
Or did I misunderstand your comment?

Another bug by that PR is that the Machinegun Viper Wheels template is
always available in Alpha 1. Removing the following line of code from
data/base/script/campaign/cam1a.js does not change this:

enableTemplate("ViperLtMGWheels");

from warzone2100.

maxsupermanhd avatar maxsupermanhd commented on May 2, 2024

Could you be more precise as to when building trucks is possible while
no HQ has been built?

So Vaut, me and some other people decided to test master because of my pull request with old weapons accuracy and other good stuff to play with. On second or third game when i built only 2 labs and factory i decided to produce more builders but when opened factory tab screenshots... So one or two games i cant build any units before HQ, when other players can. Next games in voice chat another few players (not all) reported that problem. Unfortunately i still don't know how to fix this.
And i found some logs for this:

error   |23:37:32: [loadDroidTemplates:457] Design id:221480 (Truck Viper Wheels) *NOT* added to UI list (duplicate), player= 1
error   |23:37:32: [loadDroidTemplates:457] Design id:221480 (Truck Viper Wheels) *NOT* added to UI list (duplicate), player= 2
error   |23:37:32: [loadDroidTemplates:457] Design id:221480 (Truck Viper Wheels) *NOT* added to UI list (duplicate), player= 3

EDIT:
Maybe adding enableTemplate("ViperLtMGWheels"); to warzone2100/data/mp/multiplay/skirmish/rules.js can fix this?

from warzone2100.

KJeff01 avatar KJeff01 commented on May 2, 2024

Another bug by that PR is that the Machinegun Viper Wheels template is
always available in Alpha 1. Removing the following line of code from
data/base/script/campaign/cam1a.js does not change this

Campaign doesn't enforce this rule. Like the multiplayer transporter, it has the available flag set to true. So, if you want that, remove that line from the templates.json base file. It seems the default truck and mg viper are available so using enableTemplate on them is unnecessary except for the tutorial.

I believe why some see their designs and some not depends on if they built an HQ in previous games. Setting it back to false when games load should do the trick.

Patches here should mostly fix this. I made researched truck templates always available now. The one drawback is T2/T3 no bases allows players to use their non-default trucks. Still wondering how to stop that or if there is a better approach.

from warzone2100.

KJeff01 avatar KJeff01 commented on May 2, 2024

Trucks will now be available without the HQ.

from warzone2100.

Forgon2100 avatar Forgon2100 commented on May 2, 2024

Another bug by that PR is that the Machinegun Viper Wheels template is
always available in Alpha 1. Removing the following line of code from
data/base/script/campaign/cam1a.js does not change this

Campaign doesn't enforce this rule. Like the multiplayer transporter, it has the available flag set to true. So, if you want that, remove that line from the templates.json base file. It seems the default truck and mg viper are available so using enableTemplate on them is unnecessary except for the tutorial. [...]

Thanks. I had gotten used to the buggy behavior before #378 was pushed.
The problem has been fixed in #383, which is now ready to be merged.

from warzone2100.

maxsupermanhd avatar maxsupermanhd commented on May 2, 2024

We will test latest master this evening. (22:00 GMT+3)

from warzone2100.

maxsupermanhd avatar maxsupermanhd commented on May 2, 2024

So me and Vaut tested this for now (works) but there was only two players in here. Cant say that its absolutely tested but hadn't noticed any bugs.

from warzone2100.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.