eternaldensity / Sandcastle-Builder

xkcd: 1190: Time: The Game
Other
83 stars 68 forks source link

Finishing 4.0 #1385

Closed TheSnacker closed 8 years ago

TheSnacker commented 9 years ago

Admittedly, I have been kinda missing from all of this for a while. Sorry about that. It is kinda hard to get myself to program after working all day. I also got kinda intimidated when it came time to start making new stuff and not just fix what was already there. That said, I am going to try to pitch in and help fix what needs it. Given that there hasn't been an update in over two weeks, it sounds like it is needed. I do need to know what needs fixing though. I have tried looking through the other topics and all I have found is one item. Please reply with any others that still need fixing, and I'll try to add them to this list and work on them.

https://docs.google.com/spreadsheets/d/1ry0g-n26zvJiuPVQr4AnTlcY8B2F3_6FTAUa5sYloaA/edit#gid=493290078

edit: replaced list with speadsheet link

pickten commented 9 years ago

Dunno if what is fixed vs what is wrong on the spreadsheet, but, on the spreadsheet:

Breath still seems to lock on use.

Knight attacks become too hard too fast

Glass lightning notif spam

Also, the overview is listed as fixed on the spreadsheet.

TheSnacker commented 9 years ago

Okay, that helps. I didn't find the spreadsheet before. I have added a link to it in the main post here. It isn't the most straightforward thing, but I will see if I can figure out what still needs to be done through it and see what I can do to fix things.

eternaldensity commented 9 years ago

Cool, I was actually thinking of sending out a reminder that this still exists :D

EPSIL0N commented 9 years ago

I think the breath fix is waiting to be merged in, I am going to look into the knight fight issues.

eternaldensity commented 9 years ago

There's no open pull requests on my end.

EPSIL0N commented 9 years ago

Ah, I think there is a request to merge into LL's branches that I will probably end up completing this weekend if not tonight.

Not trying to take over things but just because few other people change the issues when you look at the colours of the issues they generally reflect my opinion on the issue, green being solved, yellow for minor stuff or just needing some testing and red/orange for more major concerns.

I think we were are aiming for all green.

TheSnacker commented 9 years ago

Sorry, I tried to work on this over the weekend, but I had no idea how far thing have come, and I'm a little lost at this point. I'm thinking I'm going to switch over to the beta version soon, and start working my way through just so that I can see what is actually being talked about so that I may even be able to fix something.

EPSIL0N commented 9 years ago

Ooh, another person. I think I have the absolute most up to date stuff on my local copy as I seem to have fallen into the position of the person who merges all the different collaborators work along with about two weeks of bug fixes. I could push it to somewhere on github if you wanted.

TheSnacker commented 9 years ago

I was going to start playing through the one linked in the "Tired of this" thread in the Reddit. If that is not the most up to date version though, it would help to have access to that.

I still hurt for time to actually program, but I can QA at least, since I can play the game at work without it interfering too much.

EPSIL0N commented 9 years ago

Ah, what is linked is not too out of date but I wish I had pushed an update a week ago as pretty much as soon as I offered to update last night we found that one of the fixes has created an incorrect save loading behavior, hopefully it won't be too hard to work it out.

TheSnacker commented 9 years ago

Running through that version, I did find another bug. It looks like Robotic Hatcher is not working. I added it to the doc.

EPSIL0N commented 9 years ago

Status update. I am sitting on a lot of fixes and tweaks but atleast one of them changes the way saves are read which causes issues for the beta testers as it makes current version 4 not load cleanly into possible upcoming version 4.

On that note I wanted to confirm that we are aiming for version 4 to be the number on release and to ask if we want beta saves to be carried over or not. I like the idea of a flat 4 and have no strong feelings either way. I would like to keep my progress and I took steps to avoid using any thing we found as broken to get an advantage.

We are currently waiting on a few more tests: Does the WotT/WotP one or the other system work properly? Does aperture science cause a loot page overflow with 1000 copies / Does that only happen before save and reload? Does Blue Hints effect both per click and per mnp like the code suggests it should?

And the remaining unclaimed issures on the todo list are: Make kitten reward system for dragon drum and sea mining and leo. Add notifs and effects to those three as needed. Green Sun needs a unlock condition. Honor among serpents notification cancel bug appears to not be fixed, once fixed Honor among serpents will need an unlock condition.

The Honor among serpents bug looks pretty simple so I will likely move on to that shortly. I believe pickten is working on a few issues and I am trying to help.

TheSnacker commented 9 years ago

I tried running through the new stuff, but there is so much that is new, and this being sandcastle builder, it is hard to figure everything out on my own. I could dig through the code to see what the new boosts are, and the unlock conditions or whatnot, but that would take a lot of time. I'd ask you guys to help me through it, but at this point, I wouldn't be helping as much as I would be holding you guys back.

I came in here thinking that it would just be squashing bugs, and that I was good at that. Sadly I would need to learn the new game before I could do any of that, and by the time I learned that, I'm sure you guys will have the bugs beat. For now, I'm going to go back to the game as it is, and try to get infinite bonemeal, because at the point I was at in 4.0 when I stopped, though the diamonds were there, winning all the dragon fights that I would need to get all the GDLP unlocks would have been next to impossible.

EPSIL0N commented 9 years ago

Ah, thanks for the help, every extra viewpoint moves us forward even if it is just by making me a little more confident I've not been over looking some behavior for an entire month.

I'm not sure if it is meant to be possible to get to infinite bonemeal, I'm trying to figure out what it supposedly breaks besides infinite bonemeal to infinite eggs which is not a good idea.

Yeah the knight attack fights in 4 are pretty broken because of some code involving princesses but that is one of the things I have a change for. I think it is something like the fight after Lords is a sextillion times harder and the next one like an octillion times that?

TheSnacker commented 9 years ago

The problem is that by the time I stopped, the best I could do was the early Lord fights. Even without princesses pushing it past the Lords stage, Wyverns get attacked by late Lords by default. Princesses break it worse, but it kinda seems to be broken even before that, unless I am missing something?

EPSIL0N commented 9 years ago

I see where you are coming from. I think I just ignored knights when I was playing and I also changed that a little while doing the rework. I think I could only reliably beat 2 lords but I haven't taken any of the specializing super boosts yet.

TheSnacker commented 9 years ago

Oddball question. I notice that the cats are now devoid of chromatic heresy, and I was wondering if it was just that, to make it black and white like everything else, or if there was more to it?

pickten commented 9 years ago

What it was was that I don't know the colored variant's address, and didn't see it anywhere in the existing code. Does anyone know?

TheSnacker commented 9 years ago

Oh. Looking through the code, you just need to take out the /g to get color versions. It looks like this is set up in gui.js?v=1.

EPSIL0N commented 9 years ago

We are still plugging away at it, I didn't make much progress this last week due to real life stuff.

We still need suggestions for unlock conditions for Green Sun and Honor among serpents.

We also need clarification if people in general or the authors feel that Dragon Drum and Leo should be available the moment they can have effect or require additional conditions be met.

I was also looking for opinions if the kitten only rewards should be effected by double department as twice as many could be nothing of note like 2 panes instead of 1 or it could be pretty big for doubled dragon drum. I don't think Sea Mining is effected by doubling.

charmsRace commented 9 years ago

Hey, so. Sorry I haven't been around in forever, my computer sort of... died. It won't charge, and I haven't been able to get a new one in a few months, so no programming for me. I've been following along on mobile but I'm not sure what I can contribute. Regarding Leo, though, the idea is that it's a backup in case the player spends panes frivolously, so whatever unlock condition it has, it shouldn't be too difficult to meet, and it shouldn't require access to very many panes.

pickten commented 9 years ago

I feel like Green Sun should probably unlock when you get your Nth Dimensional Keyhole for some N, probably one of 2, 5, or 11 (first t1i discovery).

Honor Among Serpents seems weak to me, though possibly because I'm only a 'newt player, so maybe just a wyvern fight reward? Alternately, maybe just make it be when you fledge on top of hiding wyverns? IDK, it just doesn't seem like it needs to be any harder than that.

For the Double Department thing, I think it would be a bit weird for DD to affect the kitten only rewards, since the kitties are separate from the DoRD in theory, but I don't know how the balance would work out.

EPSIL0N commented 9 years ago

Ah good to hear from you Cal.

I think that is a good idea related to some milestone in Dimensional keys or progress in t1i (I had missed the note in the code that says that)

Yeah, I think if we've managed to fix the bug with Honor it is no longer overpowered.

I think the kitten issue was mainly in my head, it should be pretty easy to have 1 kitten reward or if none are available than 1 or 2 depending on department DoRD rewards.

EPSIL0N commented 9 years ago

Aiming to try and get something together for this weekend if not Friday but I was wondering how many people here are interested in testing more, the save issue I might have mentioned before still exists but it is an isolated case to the testers so it is hard to estimate if I should put time into writing a persist clause for it or just offer to convert the saves for the few people who will be effected.

pickten commented 9 years ago

I'd say we should just write a separate script, but I'm not sure how many people would use it/need it.

EPSIL0N commented 9 years ago

Progress report. Just about ready to push the update, ended up having to alter persist anyways so I've gone ahead and done the relatively small apparently changes to load testing 4 saves into 4.001 (or whatever we want to call it that is >4). Just hunting a bug that we need to deal with as it will persist into the future and be a relative annoyance to all new players and doing some final testing.

charmsRace commented 9 years ago

How's shard/pane stuff doing? What are the outstanding tasks (if any)? I've got no idea what's happened since my computer gave up. Has it actually been playtested?

EPSIL0N commented 9 years ago

The update as a whole has not really been play tested but I think I have tested each of the smaller fixes as I merge or write them into my local copy.

I believe everything shard/pane related is in working order but there may be balance issues. Green sun gets pretty crazy powerful within about two days of t1i progress.

EPSIL0N commented 9 years ago

Looks to be ready for the next round of testing, just going to give people a little bit longer to back up their saves. I think the issue is fixed but if I tested with ten different saves, the eleventh could still break.

EPSIL0N commented 9 years ago

Good news is I had the foresight/good fortune to push the month's worth of content from my laptop to github, bad news is an hour later my laptop battery then reached the end of the 'end of life' stage and apparently this is not a model that tolerates 'AC only' running well. I'm not completely out but my next best choice is a desktop so I won't be able to check the spreadsheet or github and reddit threads as spread out during the day on weekdays.

erplsf commented 9 years ago

Greetings. inb4: sorry, for jumping right into the testing, and maybe, not reading previous discussion. I've imported my savefile into last version of Leopards master, and noted that I can't Time Travel, or Negator-flip into negative NPs. Is it known bug, or intended behaviour?

pickten commented 9 years ago

I know what's wrong with Negator -- that's from a check I added, not realizing Negator uses the same function as time travel (edit: nevermind, that's not right). What do you mean for Time Travel, though? What NP were you trying to travel from and to?

EPSIL0N commented 8 years ago

Additional details like which NP to which NP would help a lot as I may have broken something while trying to fix something else.

erplsf commented 8 years ago

Here's save file for the case study: http://sprunge.us/eHNF I'm trying to travel backwards via Time Travel from NP 1 and recieve 'Divide by zero error'. Jumping via discovery to NP -1 puts me into flipped NP 0, traveling forward via TT from NP -1 does the same. Using Camera on that NP reports that I already have the discovery for 'Minus In The Beginning'. Then, trying to travel from flipped NP 0 forward also reports divide by zero error.

EPSIL0N commented 8 years ago

Ah, Thank you. I'll try to look at it some time today, I wonder what the issue is this time.

Edit: So far is just looks like a small visual error for negative NPs displaying as 0 for -1, -1 for 2 and so on as the divide by zero and similar warnings are intended, you shouldn't be able to manually time travel across timelines including NP 0 as it's own 'line'.

EPSIL0N commented 8 years ago

Found it and fixed it, it was just a visual bug but I also added in pickten's changes to my working copy, so I am wondering when I should push an update?

erplsf commented 8 years ago

I think you can push updates into LL's repository, whenever you're done.

EPSIL0N commented 8 years ago

There are a few little things on my list to address which I will try to do in the next few days but most of what is not quite satisfactory to me is the stuff that has been sitting in opinion limbo for weeks now.

Unless someone comes up with a larger complaint or issue I think the only thing left to consider is when we would want to release it in terms of people being available to fix any major bugs or balance issues that appear with the bigger audience.

eternaldensity commented 8 years ago

I was thinking of asking how things were going, and I see that you got there first :D Yeah it's about time we got something out the door soon. So long as there's nothing game breaking or save breaking or really really dumb... (more than the norm, I mean. ;) )

erplsf commented 8 years ago

I'm eager to all new stuff that's coming, and a few bugs won't matter so much :)

EPSIL0N commented 8 years ago

I have pushed the version with the last of the things from my todo list and a few more typo fixes from pickten to my EPS-fixes branch.

I don't know of anything that should break but some of the dragon fight mechanics relating to princesses are pretty much placeholders as the fights went from completely unwinnable to currently a touch too easy.

pickten commented 8 years ago

There seems to be something weird causing the prey.indexof bug that occasionally shows -- I had it briefly, refreshed, and now it's gone.

Because it's bugging me, I'm trying to get the negpix overview to actually display negpix info (wasn't an issue before because it was unfledgeable anyways). Edit: got jumping working. Edit2: got display working. Just need to fix the overflow. Edit3: Just need to test that my css changes work, but github is failing to sync the pages directory. Edit4: all good now.

Also, fledging negative is broken, looking to fix it.

EPSIL0N commented 8 years ago

I didn't think we were at the stage to fledged in the negatives yet.

pickten commented 8 years ago

Oh, in that case the message should be much clearer (it says "0 have been fledged at NP"). I have been focusing more on the display issues, so I'll just adjust the message in the case of negative newpixes to say something along the lines of "you can't do that" and leave it at that.

Edit: I may change this later, but for now, it reads "I'm sorry Cueball, but I'm afraid you can't fledge there." The check is just if you are actually fledging a positive number of dragons, so it should be fine.

TheSnacker commented 8 years ago

Change it to: "The underworld is no place for living dragons"

I have an idea for using negative NPs as a place to fledge undead dragons to, using a Necromancer boost. I think it would be an interesting idea for a use for Dragons lost fighting and Starved Dragons, as well as an interesting use for Negative pix. I do not want it to be something that 4.0 is delayed any further for though. Far too many people have waited far too long for this update. We should not make them wait any longer if they do not have to.

eternaldensity commented 8 years ago

nods

pickten commented 8 years ago

That makes sense, and I'm very curious to see how that turns out. Made a PR to change that.

elswyyr commented 8 years ago

What's missing now? Is there anything that still needs to be done before the update is pushed to live?

eternaldensity commented 8 years ago

I was gonna ask the same. If I went live today, what terrible things woukd result?

pickten commented 8 years ago

There was the bug identified on the sub a couple days back allowing unlimited negative downgrading of the refinery/chiller that I want to check/remove, but haven't gotten to yet (I'll probably get it done tonight, though). While not a bug, per se, there is also something that seems to cause button-presses to scroll back up to the top of the page. Besides that, I think the only known bug is the daylight savings time ONGs, but that can easily be pushed back. EDIT: PR is out to fix glass downgrade.