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

vehicles.json overhaul and new obstacle parts (Ready for review) #3733

Merged

Conversation

Projects
None yet
4 participants
@NaturesWitness
Copy link
Contributor

commented Oct 18, 2013

Typed up by Mechanoid on the forums, here is an overhaul of vehicles.json, fixing in a massive amount of (literal) holes in Cata's fleet of vehicles. Also, I replaced the last of the underbody wide wheels with the correct normal wheels, as they were no longer needed.

Original forum thread:
http://smf.cataclysmdda.com/index.php?topic=3739.0

EDIT - This has grown from a small patch job to a thorough rebuild of nearly every vehicle with an interior in Cataclysm. Tested and everything works fine, so this is ready for review.

vehicles.json overhaul
Courtesy of Mechanoid on the forums, here is an overhaul of
vehicles.json, fixing in a massive amount of (literal) holes in Cata's
fleet of vehicles.  Also, I replaced the last of the underbody wide
wheels with the correct normal wheels, as they were no longer needed.

Original forum thread:
http://smf.cataclysmdda.com/index.php?topic=3739.0
@i2amroy

This comment has been minimized.

Copy link
Member

commented Oct 18, 2013

While I certainly appreciate the work, I'd really like to see some of these new windshields/boards be replaced by a new part, "moulding", which would function similar to a low hp board that allows you to see through it.

@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 18, 2013

Hmm that's a good thought I'll take a look at it.

@ianestrachan

This comment has been minimized.

Copy link
Contributor

commented Oct 18, 2013

Is "moulding" the right term? It brings images of art classes to me, but I don't know what specific car parts are called (which doesn't help when it's traditional to explain programming concepts by giving car analogies).

Would "panel" be more accurate for a transparent obstacle part, or is that not quite right?

@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 18, 2013

I was thinking "fender" might be a possibility, as these are probably going to be over wheels most of the time. A "quarter panel" is a vehicle part, that could be a good name. I'll try and put together a quick proof-of-concept for this, so we can see how it looks.

quarterpanel proof of concept and example design (car)
As suggested in the comments, added quarterpanels.  A quarterpanel is an
obstacle, but does not block vision.  It has 250 durability, or
one-quarter of a board.  Also, as an experiment, turned all boards on
the "car" into quarterpanels, added quarterpanels over wheels, and moved
headlights to the top two corners of the vehicle, eg. where the wheels
are.  I think this makes a more realistic looking car, putting it up so
people can comment.
@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 19, 2013

okay here is a quick concept thing I typed up. Added quarterpanels that are obstacles but don't block vision. Name, durability, and item used are placeholder for now, this is mostly just to get input on concept, so changed title to WIP.
Also, tried a quick redesign on the "car". All boards are now quarterpanels, wheels and front have quarterpanels on them, and headlights are moved to top two corners of vehicle, more like an actual car. Fired it up in game and I think the end result looks pretty nice. Please post your thoughts on these changes, especially on what the name and stats of the quarterpanels should be.

@ianestrachan

This comment has been minimized.

Copy link
Contributor

commented Oct 19, 2013

I think 'quarter panel' might be too long to fit in the vehicle interaction window. There's not a lot of space there, but I'm not sure. "Panel" might be better if that is the case.

Otherwise, I like the sounds of it, particularly moving the headlights up.

@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 19, 2013

Good catch on the name too long thing, I'll be sure to double-check that in-game. There's actually two other things I'd like your thoughts on:

One, what do you think of having the panels over the wheels? When I was doing this, it occurred to me that practically every modern car has body panels over the wheels, so it was kinda strange cars in Cata all had exposed wheels like a 50's hot rod. Adding the panels on the corners gives the car a much sleeker look I think, although it could be viewed as a little uninteresting.

Two, what are your thoughts on the durability values? I've been thinking a little about it, and 250 may be too low. Maybe 500 would be better?

I'll try to work on this more later tonight, I have to go over to a buddy's house today and fix an interior door and try to get a recalcitrant circuit breaker to work.

NaturesWitness added some commits Oct 19, 2013

Improve quarterpanel durability an new beetle design
doublechecked whether quarterpanel fits in the menus and it does.
Raised quarterpanel's durability to 500, 250 felt too low.  Also tweaked
design of beetle, take a look and comment your opinion.
car chassis design update
updates car chassis to new design
@ianestrachan

This comment has been minimized.

Copy link
Contributor

commented Oct 19, 2013

Unless we add a Formula One car to the game, every wheel should have a quarter panel over it. The z-level is currently rigged so that the tire's glyph shows up above frames but below the 'center' layer, so it'd cover up the wheel, which is fine if the panels are angled properly.

I have no idea what good durability values are. I always just look at similar parts and ballpark 'em. Boards are 1000, so a quarter panel should be in the 250-500 range.

@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 19, 2013

Okay I have another question for everybody; Should windshields extend the entire width of the vehicle, or should the outermost edges be quarterpanels?

@kevingranade

This comment has been minimized.

Copy link
Member

commented Oct 19, 2013

Probably the forward corners should have windshield all the way across. I don't think the little pillars justify blocking an entire tile. Boards should be for fairly large areas of blocked off vision.

NaturesWitness added some commits Oct 19, 2013

More quarterpanel shapes and big windshields
When I started on the semi-truck, I realized just putting horizontal
quarterpanels on the front would make it look really boring.  So, I made
some more quarterpanel variants to go with the decorative frame
variants.  Also gave the previously redesigned vehicles full width
windshields, per Kevingranade's suggestion.
RV
@NaturesWitness

This comment has been minimized.

Copy link
Contributor Author

commented Oct 20, 2013

Whew, okay that's everything except the armored car and the military truck. I didn't do those because they use armor plating for walls, and there's another PR up that's tweaking armor plates. I'd rather not get into it right now and have these changes possibly conflict with the other PR, so I'll wait until that one lands and do a new PR with the last two vehicles. Just to summarize, here's what we've added:

1 - Quarter panels added, essentially a half-board that prevents movement but doesn't block vision. Has variations to match every frame tile shape except cross.

2 - Quarter panels added to edges of almost every vehicle, including over wheels. Gives vehicles a more defined shape and mass, you can't just walk over the hood of a car.

3 - Headlights moved from on the windshields of vehicles to the front, usually over the front wheels (I mean c'mon, who sticks headlights on a windshield??)

4 - Underbody wide wheels removed from all vehicles and replaced with normal wheels (with the new vehicle design system, they're no longer necessary, and in fact can be removed altogether).

5 - Back wheels on RV moved to edges of vehicle to match front ones (now that wheels can go under boards, there's no reason for them to be in the middle of the frame)

6 - Windshields now span the entire width of vehicles.

I think that's everything, feel free to comment if anything looks like it should be changed.

@kevingranade kevingranade merged commit b65a204 into CleverRaven:master Oct 21, 2013

1 check passed

default Merged build finished.
Details
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.