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

more gradual and less effective BLD_LIGHTHOUSE effect for planets/buildings #3977

Merged
merged 1 commit into from Nov 13, 2022

Conversation

agrrr3
Copy link
Contributor

@agrrr3 agrrr3 commented Jul 13, 2022

was -30, now between -20 to -30 for ships and -0 to -10 for all other objects

@agrrr3 agrrr3 changed the title more gradual and less effective for planets/buildings BLD_LIGHTHOUSE effect more gradual and less effective BLD_LIGHTHOUSE effect for planets/buildings Jul 13, 2022
@Vezzra Vezzra added component:content scripting The Issue/PR deals with the FOCS language, turn events or the universe generator. category:tweak The PR contains insignificant code changes, like code style grooming or value tweaking. labels Jul 17, 2022
@geoffthemedio
Copy link
Member

Is there a discussion thread or does this need testing? I don't have an opinion on the balance aspects.

@agrrr3 agrrr3 marked this pull request as draft July 24, 2022 02:14
@MichaelCourtney
Copy link
Contributor

Is there a discussion thread or does this need testing? I don't have an opinion on the balance aspects.

https://www.freeorion.org/forum/viewtopic.php?f=15&t=12509

@agrrr3 agrrr3 force-pushed the 20220713_LIGHTHOUSE_nerf branch 2 times, most recently from ad7e377 to 82e73e2 Compare October 11, 2022 20:23
@agrrr3 agrrr3 marked this pull request as ready for review October 11, 2022 20:24
@agrrr3
Copy link
Contributor Author

agrrr3 commented Oct 13, 2022

note, this is intended for 0.5 in order to make it a viable strategy to use stealth as a defense for planets

@Vezzra Vezzra added this to the v0.5 milestone Oct 14, 2022
@geoffthemedio
Copy link
Member

this causes parse errors:

Error: 0:06.866641 {0x00007ff37a7fe6c0} [error] server : ReportParseError.cpp:92 : /freeorion/build/default/scripting/buildings/LIGHTHOUSE.focs.txt:132:12: Parse error.  Expected (?-i:effects) here:
3: 
3:         // takes the maximum age of all lighthouse buildings in the system regardless of owner (by just triggering for the oldest ones)
3:         EffectsGroup
3:             scope = WithinDistance distance = 0.00001 condition = Source
3:             stackinggroup = "LIGHTHOUSE_CATALOG_STEALTH_STACK"
3:             activation = (Source.Age >= (Statistic Max value = LocalCandidate.Age condition = And [
3:             ^
3:                 InSystem id = Source.SystemID
3:                 Building name = "BLD_LIGHTHOUSE"
3:             ]))
3:             accountinglabel = "BLD_LIGHTHOUSE_CATALOG_STEALTH_LABEL"
3:             effects = SetStealth value = Value - min(Source.Age, NamedReal name = "BLD_LIGHTHOUSE_CATALOG_STEALTH_MALUS" value = 10)

…anets/buildings

was -30, now between -20 to -30 for ships and -0 to -10 for all other objects
@agrrr3
Copy link
Contributor Author

agrrr3 commented Nov 8, 2022

Is anything stopping the merge?

@Vezzra
Copy link
Member

Vezzra commented Nov 11, 2022

Is anything stopping the merge?

No objections so far, but also no one has done the merge yet. I'm going to create the release branch on Sunday (Nov 13th), unless there are objections againt it, this PR should be merged until then.

If there are no objections, but the branch still not merged when I create the release branch, I'm going to do the merge myself before (unless I forget, which is always a possibility considering my age... 😉).

@Vezzra Vezzra mentioned this pull request Nov 13, 2022
48 tasks
@Vezzra Vezzra merged commit eb53f48 into freeorion:master Nov 13, 2022
@Vezzra Vezzra added the status:merged All relevant commits of this PR were merged into the master development branch. label Nov 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:tweak The PR contains insignificant code changes, like code style grooming or value tweaking. component:content scripting The Issue/PR deals with the FOCS language, turn events or the universe generator. status:merged All relevant commits of this PR were merged into the master development branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants