Skip to content

Several error fixes #32

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

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

SigmaDolphin
Copy link

i have been working on SkyReader for some time and been working to improve it, i found several errors in it and managed to fix them
as a summary:

added SetOutputReport since most portals communicate by reports rather than raw hid writes
added a delay on the portal initialization because the portal has a slight delay to start polling data and allowing for commands
fixed an oversight in the write commands that only allowed to write to position 0 skylanders
added a couple of functions to allow raw reading and raw writing to figures

as a note, most portals work properly and read properly, except TrapTeam, which for some reason duplicates the listings of each skylander
All my testing was done in Windows using PS/Nintendo Portals

most portals communicate with reports rather than raw writes
the portal has a startup delay, so i added a delay after activating to allow for the portal to startup and allow for the next commands to run properly
in addition, changed the hid_write for hid_set_output_report
fixed an oversight in the write functions that only let you write to position 0 skylanders (bNewSkylander = false)
also added a couple of functions to raw read or raw write data to the figures
added raw read and raw write capabilities
@SigmaDolphin
Copy link
Author

also i am a big dumdum for not adding the modified .h files to the pull request

@silicontrip
Copy link
Owner

Hi,
I'm not quite sure how to manage this now. Both my kids have out grown Skylanders. The last version they played was on the xbox, which I know the portal has extra encryption (thanks Microsoft for the walled garden) I did keep hold of my old Wii portal. But honestly my interests have moved on. As the beauty of open source I'd really like to hand this over to someone else to work with. I know that capull0 has done extensive work in a linux port and (strangely enough) other have made windows ports, I say strange because I ported this from a windows app to work on OSX initially (oh and being a command line addict fix up and make the command line options consistent).
But not sure how to move on. Thanks for the interest.

@SigmaDolphin
Copy link
Author

SigmaDolphin commented Apr 17, 2019 via email

@hegyak
Copy link

hegyak commented Apr 25, 2019

Looking forward to testing these changes/fixes on a Windows 10 System. With both PS4 Trap Team portal, and XBox 360 Portal (I forget which ones I have).

I made a GUI editor for Skylander Figure Dumps. myself. Does a pretty good job too. It doesn't do everything I want, like Skills in depth but most of the things you want to edit, you can.

My language focus is on .Net so I can't really help with the C++ side of the code but I got all the games for XBox 360 (retail) and Trap Team on PS4. And I want to help this out as best as I can as well.

@hegyak
Copy link

hegyak commented Apr 26, 2019

After testing this against my PS4 Trap Team Portal. This is a good Pull request. It works great. Reads in the Traps in the trap hole, and the Skylanders too.

The only request I have is this:
Please fix the following line in checksum.cpp
fprintf(stderr, "Checksum failure for checksum type %d, data area %d", type, dataArea);
to
fprintf(stderr, "Checksum failure for checksum type %d, data area %d\n", type, dataArea);

It adds a new line for the Checksum is invalid text output. It should work on OSX, Windows and Linux.

@SigmaDolphin
Copy link
Author

to be honest i haven't messed up with the checksums in SkyReader, they are a tad outdated as well since its missing the Type 4 checksum and some other things, i did add a function in my VB6 app to check for all the checksums, i haven't uploaded it yet because i am a bit scared Activision might go berserker on me

@hegyak
Copy link

hegyak commented Apr 28, 2019

I get your desire to keep that stuff hidden away.

@SigmaDolphin Is there a way we can chat about this more? Sharing information and data. Swapping apps and code as well.

If you want to Email me, my Username here on GitHub @ Gmail is a good place to get a hold of me.

I am willing to share what I know and hopefully we can make some new app(s) together.

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

Successfully merging this pull request may close these issues.

3 participants