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

Even more folding parts! #10405

Merged
merged 8 commits into from Dec 12, 2014

Conversation

Projects
None yet
4 participants
@jokermatt999
Copy link
Contributor

commented Dec 9, 2014

I've added alternators, batteries, floodlights, and a folding version of the shopping cart basket (and recipe). The basket is the only one that actually gets smaller when folded, however.

On the subject of balance, I was considering adding "foldable" to motors (edit: and perhaps even solar panels) to allow them as part of carry-able vehicles, but giving them no reduction in volume for folding_volume (like batteries and alternators). Does that seem alright/balanced? I don't really see any issues, but does anyone have a problem with it?

@kevingranade kevingranade added the unknown label Dec 9, 2014

@jokermatt999

This comment has been minimized.

Copy link
Contributor Author

commented Dec 9, 2014

Don't worry, I tested the json this time too.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 9, 2014

kevingranade added the unknown label 21 minutes ago

Twilight Zone music plays

@jokermatt999

This comment has been minimized.

Copy link
Contributor Author

commented Dec 10, 2014

Just curious, any reason for the "unknown" flag? Also, I'd do a separate pr for the engines/solar panels, if others think it's a good idea.

@NaturesWitness

This comment has been minimized.

Copy link
Contributor

commented Dec 10, 2014

I like the idea of folding solar panels. I can see having sets of folding solar arrays for your mobile base. You set them up and hook them to your base with jumper cables when you make camp, and fold them and put them in the trunk when you want to move.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 10, 2014

NaturesWitness has it right. I'm curious about the unknown flag too.

@jokermatt999

This comment has been minimized.

Copy link
Contributor Author

commented Dec 11, 2014

Kevin, could I at least get some feedback about the "unknown" tag? It's kind of frustrating having this stuck in PR limbo when it's been validated and tested.

@NaturesWitness

This comment has been minimized.

Copy link
Contributor

commented Dec 11, 2014

I'm actually curious about this too. For some bizarre reason I thought it was related to the Pokemon "Unown" from the gold and silver games.

BTW, why does everyone like Pikachu anyway? Everyone KNOWS Farfetch'd is the best Pokemon in the entire series.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Dec 11, 2014

It's a meaningless bug from waffle.io, ignore it.

@jokermatt999

This comment has been minimized.

Copy link
Contributor Author

commented Dec 11, 2014

Heh, I was worried it was due to me falling to test previous prs. >>

And yes, Farfetchd is the best due to delicious leeks.
On Dec 11, 2014 11:36 AM, "Kevin Granade" notifications@github.com wrote:

It's a meaningless bug from waffle.io, ignore it.


Reply to this email directly or view it on GitHub
#10405 (comment)
.

"components": [
[
[ "wire", 6 ],
[ "basket", 1]

This comment has been minimized.

Copy link
@KA101

KA101 Dec 11, 2014

Contributor

Watch your spacing there.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 11, 2014

After looking at this, I'm not quite sold on making all sizes of alternator fold up: a volume 8 alternator seems kinda large/bulky to be folded neatly. Wire baskets can be compressed, sure--I've seen and used 'em on a bicycle IRL--but the alternators, etc don't feel workable to me.

If I'm missing something important, let me know and I'll rethink.

@jokermatt999

This comment has been minimized.

Copy link
Contributor Author

commented Dec 11, 2014

Should be all good to go. I'd like to leave the floodlights (as they were specifically requested) and the recharging station (for utility) as foldable, although neither really get any smaller.

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 12, 2014

I'm not sold on the recharge station but they can be small enough when not vehicle-mounted, so wev.

@KA101 KA101 self-assigned this Dec 12, 2014

@KA101

This comment has been minimized.

Copy link
Contributor

commented Dec 12, 2014

Looks like we forgot something in construction.cpp a while back: done_vehicle didn't have anything for folding frames, so using a folding frame to start vehicle construction created a standard, nonfolding one.

Was straightforward to fix though.

@KA101 KA101 merged commit 214531d into CleverRaven:master Dec 12, 2014

1 check passed

default
Details

@kevingranade kevingranade removed the unknown label Dec 12, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.