-
Notifications
You must be signed in to change notification settings - Fork 272
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
revert: Obsolete useless hub01 basement #2488
base: main
Are you sure you want to change the base?
Conversation
Rationale from issue to restore hub01 basement.
|
...oh, yeah. used wrong section. |
I guess we need to ask @Coolthulhu |
"We could" I guess could be rejected as a rationale unless someone demonstrates they want to port more, though? As it stands, it doesn't do much |
there were some serious request to port over hub01 contents from korean community so i was investigating. haven't checked with dda's hub prs, however. |
Oh, in that case I retract my objection since it seems there's genuine interest <3 |
I think it's fine either way I suppose. The main reason to remove it is if it actually impacts performance, and I think by now performance should be fairly a bit better such that we can afford to keep it. The other reason to remove it is the whole "you currently can't access it unless you're playing the game wrong" thing, but this seems like a thing that will be negated by any future plans if someone gets around to implementing a valid reason to access the location. Being given permission to visit them in the future to assist with something on-site, or tasked with breaking in by another faction antagonistic to the faction, for examples. While that does hinge on someone going and actually doing it, they can't exactly begin to without the location also existing to be fleshed out. @Zlorthishen also brought up in DMs the idea of fleshing out the refugee center and giving it a working underground layer, if the performance impact of it proves to be tolerable. This however is likely a "if it gets implemented it'll likely be directly in service to actually using it" thing instead, whereas here we already have the location but just need to rig it to factor into gameplay/story. |
I'm fine with un-removing if it is going to become playable, but the PR should only be merged along with the PR that makes it playable. |
@NobleJake, could you help me find which PR adds access to hub 01 basement? |
Sorry for late reply. First of all, I am looking for pr related to hub 01 basement on cdda github, and additionally posted a question about quest related with hub 01 basement on reddit. If there are results, I will report them. |
@scarf005 |
@NobleJake no worries. we could wait until related related DDA PR, but if you could create relevent quest that leads to hub01 basement, we're more than welcome for contribution! |
There is a possibility that the underground hub agents will eventually release their hostile state someday, so I think it would be good to watch for a while. |
A PR was just made to CDDA CleverRaven/Cataclysm-DDA#64712 to add an achievement for hub01, so it's not totally useless now ;) |
still, we'll have to wait until basement. |
This reverts commit 8c5a757.
1abe9df
to
ec83219
Compare
Purpose of change
Describe the solution
Describe alternatives you've considered
keep the status as-is. however, we could port more hub01 contents to fill the feature parity between DDA.
Testing
reverted then spawned hub01 on overmap. played for a while. quest was quite fun. maybe someone should port more DDA hub01 contents.