iSSB icon indicating copy to clipboard operation
iSSB copied to clipboard

Samus reopen

Open Alekemon opened this issue 2 years ago • 28 comments

Alekemon avatar May 17 '22 19:05 Alekemon

Can you also add this PR?:

https://github.com/lufinkey/iSSB/pull/26

Alekemon avatar May 18 '22 14:05 Alekemon

Are you still here?

Alekemon avatar May 21 '22 19:05 Alekemon

Yes. I'm still here. I have things I'm working on and don't have time to clone this down and test all the time so please don't be rude. I'll check this out when I get the chance but please make sure this is actually fully tested by you, fully working, looks good and correct, and matches the Smash moveset before I test it.

lufinkey avatar May 23 '22 04:05 lufinkey

Ok, fine

Alekemon avatar May 23 '22 12:05 Alekemon

Trying to build in Visual Studio gives me unresolved external symbol "public: __cdecl SmashBros::Samus::Samus(float,float,unsigned char,unsigned char)" (??0Samus@SmashBros@@QEAA@MMEE@Z) referenced in function "public: static class SmashBros::Player * __cdecl SmashBros::CharacterLoader::createPlayer(float,float,unsigned char,unsigned char,int)" (?createPlayer@CharacterLoader@SmashBros@@SAPEAVPlayer@2@MMEEH@Z)

Nightcaat avatar May 23 '22 22:05 Nightcaat

@Nightcaat you need to add the Samus.h/cpp files to the project. I assume he didn't commit the project file changes but that should fix that specific issue I think

lufinkey avatar May 24 '22 01:05 lufinkey

Not trying to be rude, but I've already playtested everything and it seemed to be correct. The only possible issue would be Samus' final Smash ray placement whose animation should be slightly to the left...

Alekemon avatar Jun 08 '22 18:06 Alekemon

Apologies, I've set a reminder to get to it tonight. It's just been work and very little free time. I was the rude one in this case for letting it sit so long, sorry about that

lufinkey avatar Jun 08 '22 19:06 lufinkey

By the way, you have my total permission to re-release this app with your changes if you want, as long it's named something different from this game and credits the original to this one, and doesn't obscure the fact that it's a fork

lufinkey avatar Jun 08 '22 21:06 lufinkey

Just tested:

  • down special gives an error special_prep_down_right animation does not exist
  • releasing the neutral special seems to trigger the animation for her grapple
  • there's no charging feedback on the neutral animation. ie there's nothing to indicate that the ball is charging or how much its charged
  • side special doesn't seem to launch a guided missile
  • up special not working
  • there's a bit of animation leaking through the top of her up air standard
  • up air standard should slow her fall a tiny amount for a bit or boost her a bit up
  • side air standard should slow and hold her fall while doing the attack, and there should be fire feedback
  • down air standard is a bit choppy, are there any transitional frames you could use?
  • neutral standard should be a combo, but it seems to be doing all the combo moves at once
  • final smash beam needs to be lined up with her gun

lufinkey avatar Jun 08 '22 22:06 lufinkey

Thanks!

Alekemon avatar Jun 12 '22 12:06 Alekemon

Finally got some time to polish Samus and fix what was stated on the feedback, giving Samus a more accurate moveset to her Smash iterations. Would love to see her in a new update of iSSB, and the request is still active. I look forward to your reply.

Seriously, this felt awesome to me playing around with a jailbroken iPad back in 2014 (one year before I've got my 2DS and a better PC) lol and I've never imagined contributing to this back then. Thanks for considering updating the game roughly 9 years after!

Alekemon avatar Feb 03 '23 23:02 Alekemon

Hey, thanks again for taking the time to work on this! I'll try to review it and post feedback as soon as I'm able

lufinkey avatar Feb 07 '23 03:02 lufinkey

Your welcome, I'll appreciate any feedback!

Alekemon avatar Feb 08 '23 19:02 Alekemon

Looks like there were some compile errors: Screenshot_20230218_125616

What are you using to test this? I didn't see any project files updated in the PR

lufinkey avatar Feb 18 '23 17:02 lufinkey

I've finally been able to find the error

Alekemon avatar Mar 16 '23 20:03 Alekemon

Gotcha, I still don't see any updates to any project files, so I'm curious how you compiled/ran it?

lufinkey avatar Mar 17 '23 18:03 lufinkey

I finished solving everything and Samus.obj gives some weird external errors

Alekemon avatar Apr 01 '23 20:04 Alekemon

If you post the error here, I might be able to help you

lufinkey avatar Apr 01 '23 20:04 lufinkey

I want to emphasize i appreciate your contribution here, but in the future please do not submit untested code in a Pull Request. As I've mentioned before, it wastes my time to have to pull down and test code that you yourself have not verified through testing. I understand errors slip through, but please take the time to make sure that your code is actually fully tested and verified as working.

lufinkey avatar Apr 01 '23 20:04 lufinkey

Yeah, of course, I'll test everything before posting. This is the error: image

Alekemon avatar Apr 01 '23 22:04 Alekemon

Ok, I've fixed that error, I'm going to playtest a bit

Alekemon avatar Apr 01 '23 22:04 Alekemon

Glad you were able to fix that issue! I'm a little worried looking through this code because most of it look straight up copy-pasted from other characters. Most of the projectiles and attack code looks like it was taken straight from Ichigo or Mario. Part of the process of adding a new characters is making sure that they handle and play uniquely from the other characters. There are different priorities, damage levels, attack/vulnerable sides, knockback velocity, durations, etc. Please make sure that you're intentionally choosing those values with the character in mind, and not just copying them from other characters.

lufinkey avatar Apr 01 '23 22:04 lufinkey

I've chosen those values intentionally, just to match Samus' moveset better without having to write entire chunks of code from scratch.

Alekemon avatar Apr 01 '23 22:04 Alekemon

Awesome, glad to hear it

lufinkey avatar Apr 01 '23 22:04 lufinkey

Now I'm getting this error, seems it's not related to Samus:

could not open haptic feedback: Haptic: There are 0 haptic devices available Loading image Images/icon.png Error BufferedImage::loadFromFile(String) Couldn't open Images/icon.png Error loading image Images/icon.png Loading image Images/loading.png Error BufferedImage::loadFromFile(String) Couldn't open Images/loading.png Error loading image Images/loading.png error loading font Fonts/arial.ttf LoadContent Loading image Images/Game/Controls/button_a.png Error BufferedImage::loadFromFile(String) Couldn't open Images/Game/Controls/button_a.png Error loading image Images/Game/Controls/button_a.png Error: Attempt to add image Images/Game/Controls/button_a.png to animation failed!

Alekemon avatar Apr 01 '23 22:04 Alekemon

More screenshots

image

Alekemon avatar Apr 01 '23 22:04 Alekemon

It looks like the assets folder isn't getting copied to the build folder. Check the build scripts section on visual studio, otherwise I'll take a look when I'm free

On Sat, Apr 1, 2023, 6:56 PM Alekemon @.***> wrote:

More screenshots

[image: image] https://user-images.githubusercontent.com/42167718/229319776-61ced495-4a90-40af-bca6-98dfad569306.png

— Reply to this email directly, view it on GitHub https://github.com/lufinkey/iSSB/pull/30#issuecomment-1493156240, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB3VGUPOAZQXBO4XN6SHWADW7CXALANCNFSM5WGBGPHA . You are receiving this because you commented.Message ID: @.***>

lufinkey avatar Apr 01 '23 23:04 lufinkey