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

Desolitor shot have not original weapon rendering? #43

Closed
yaaaaa opened this issue Jan 16, 2016 · 21 comments · Fixed by #656
Closed

Desolitor shot have not original weapon rendering? #43

yaaaaa opened this issue Jan 16, 2016 · 21 comments · Fixed by #656
Labels

Comments

@yaaaaa
Copy link

yaaaaa commented Jan 16, 2016

default

last time I used Desolitors in Phrohdoh's repo, and it'll be fine.

@GraionDilach
Copy link
Contributor

Line (contrail, laser) widths have been readjusted since in upstream and the fallback default ended up being scaled with cellsize, which due to RA2 having huge tiles compared to say RA1 ended up wider as well.

Also, Desos never had their RA2 effect, because that one is a sinus wave and not a line.

@yaaaaa
Copy link
Author

yaaaaa commented Jan 16, 2016

maybe I have something confused. probably baffled me that before this wave has a different color

@abcdefg30 abcdefg30 added the Bug label Jan 16, 2016
@MustaphaTR
Copy link
Member

That would be better if we change color of it to green as a start. Purple looks too unreleated.

@phrohdoh
Copy link
Member

Upstream does not have paletted line rendering yet so we cannot remap the line drawn, so there is no point changing it yet.

@abcdefg30 what is the bug here?

@GraionDilach
Copy link
Contributor

The Desolator's beam wasn't remappable, but green - https://github.com/Phrohdoh/oramod-ra2/blob/master/extra-files/rules.ini#L752. Both the width and colors are bug.

@phrohdoh
Copy link
Member

Generate a sine wave and pass the world-space points to WorldRenderer.ScreenPosition then those float2s to the RgbaColorRenderer.

@Mailaender Mailaender changed the title Desolitor shot have not original weapon rendering? Desolitor shot have not original weapon rendering? [$5] Feb 19, 2016
@Mailaender Mailaender changed the title Desolitor shot have not original weapon rendering? [$5] Desolitor shot have not original weapon rendering? Feb 21, 2016
@Mailaender Mailaender removed the Bounty label Feb 21, 2016
@Mailaender Mailaender changed the title Desolitor shot have not original weapon rendering? Desolitor shot have not original weapon rendering? [$5] Feb 21, 2016
@phrohdoh
Copy link
Member

WIP:
deso-test-03

@Mailaender Mailaender changed the title Desolitor shot have not original weapon rendering? [$5] Desolitor shot have not original weapon rendering? Nov 10, 2016
@Mailaender Mailaender removed the Bounty label Nov 10, 2016
@phrohdoh phrohdoh removed their assignment Nov 11, 2016
@forcecore
Copy link
Contributor

forcecore commented Aug 22, 2017

With my Railgun PR in bleed, I think I have a good chance of getting the sine beam into upstream, too. The code won't be too different. (I made the railgun code FROM the sine beam)

@Nihilianth
Copy link
Contributor

I've tried to implement this using the Railgun code. I had to modify it a bit in terms of offsets / using only the sine beam etc. I am unsure though whether I should submit a PR with it, since this mostly a copy paste of the Railgun / RailgunRenderable code.

But here's the result so far:
2019-01-10 12_52_08-openra game 2019-01-10 12-49-42-784 mp4 - vlc media player

And a vid:

@MustaphaTR
Copy link
Member

MustaphaTR commented Jan 10, 2019

We already have a working Sinus Beam rendering written by forcecore in RV, tho looking at this Railgun projectile looks pretty cool too.

@GraionDilach
Copy link
Contributor

GraionDilach commented Jan 10, 2019

For the record: the RV Radbeam is indeed forcecore with a few style updates/cleanup here'n'there.

https://github.com/AttacqueSuperior/Engine/blob/master/OpenRA.Mods.AS/Projectiles/RadBeam.cs
https://github.com/AttacqueSuperior/Engine/blob/master/OpenRA.Mods.AS/Graphics/RadBeamRenderable.cs

If you managed to clean up beyond, do. if you don't then mix the two, and do it thenafter. The RA2 repo ends up better with either.

@MustaphaTR TBH, I'm fairly sure that rendering is doable with RV/AS radbeam though, just use small Amplitude and Wavelength - like 256/64 or so.

@Nihilianth
Copy link
Contributor

Nihilianth commented Jan 11, 2019

Oh I didn't think of checking any forks first. That one by forcecore seems to work pretty much the same, except that it also tracks the target. Though I wasn't able to make the beam go from the desolator's weapon to the target without cs code changes, atm its rendered at the unit's feet (Found out about openra just a couple of days ago, so maybe I am missing something in the rules file).
2019-01-11 11_56_13-window

@MustaphaTR
Copy link
Member

What you are looking for is Armament>LocalOffset.

@Nihilianth
Copy link
Contributor

@MustaphaTR thanks, that's exactly what I've been looking for.
I'm not sure how to handle it from now on tho. Should I submit a PR with that, even though the code is basically from the AS repo?

@MustaphaTR
Copy link
Member

Yes, go ahead and do a PR. There are other traits that are on AS which are already on RA2.dll too.

@pchote
Copy link
Member

pchote commented Jun 17, 2020

There is a $5 bounty on this that can (hopefully still?) be claimed before the timer expires at the end of the month.

@Mailaender
Copy link
Member

Pinging @forcecore who was the original author of the code I submitted.

@forcecore
Copy link
Contributor

Uhmm I'm not sure whether I should take the bounty, even if I did, I'd donate back to some other bounties anyway like the unit waypoint visualization bounty.

@Mailaender
Copy link
Member

If you don't take the bounty, BountySource will collect it as unclaimed and it will be lost.

@forcecore
Copy link
Contributor

forcecore commented Jun 18, 2020

I can't find the bounty, searching for deso, desolator, beam didn't yield any results. I guess it is gone.

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

Successfully merging a pull request may close this issue.

9 participants