The International Simutrans Forum

Simutrans Extended => Simutrans-Extended paksets => Pak128.Britain-Ex => Topic started by: Junna on June 17, 2018, 03:38:07 AM

Title: 1870-1890's carriages lack class definitions
Post by: Junna on June 17, 2018, 03:38:07 AM
The majority of passenger rolling stock 1870-1890 has no class definitions (defaulting to low). These are the GNR, Midland, LSWR and the LBSCR and essentially all carriages after the 1870's LNWR sets that last had class definitions. Perhaps this is because graphics have not been changed to represent various class carriages?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on June 17, 2018, 10:23:28 AM
I have not yet got around to adding the various classes of passenger carriages in these eras: I was hoping to complete the passenger carriage classes at Christmas, but the early carriages took much longer than I expected in terms of research (and I ended up generally improving early railways (i.e., pre-1850s) a great deal, which were much lacking previously), and did not have the time to finish the carriages. I have been very busy recently with a number of things outside the world of Simutrans, and such Simutrans time as I have had has largely been spent fixing bugs.

These will be done in due course, but it is very hard to predict when I will have the time. If anyone else would like to do this (even for just a limited range of carriages, e.g., GNR carriages), then this would be welcome, albeit some detailed historical research would be necessary to ensure that one is representing the correct kinds of carriages with the correct sorts of graphics, and it is likely to be necessary to re-render the complete set of graphics for that whole type of carriages (including all livery variants), as the new rendering model produces graphics that do not look identical to the old rendering model, so carriages of the same type should not be produced by differing rendering models. (Indeed, it is the necessity to deal with these issues that has made it take so long to produce the full set of classed carriages).
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on November 23, 2018, 05:53:19 PM
I've done a little research on the topic, because I've noticed the same issue present. Here's a bit of information I've found and a pull request in github to incorporate this.

According to a short history section on page 6 (https://pregroupingrailwaysdotcom.files.wordpress.com/2018/06/lswr-coach-instructions.pdf), the following details about LSWR 6 wheel carriages are given: The first design being, drg.2454, a six wheel 4 compartment 1st/2nd composite, a year later, 1st/2nd 6 wheel composite saloons were built. These were closely followed by three types: 4 wheel 3rd brakes, 5 compartment all 3rds and 4 compartment all 1st's.

I've edited the lswr 6 wheel 28ft and created a pull request (https://github.com/jamespetts/simutrans-pak128.britain/pull/53). My amendments are: fixing the missing capacity on the front brake, adding a 1st/2nd class coach (assuming 5 per bench in 2nd, 4 in 1st, comfort increase of 2 and 13 like the lnwr 30ft). I'm unsure whether the overcrowded capacity should be enabled in the higher classes, but I've settled for four standing in per second class compartment. I'm using a substantial amount of data from lnwr coaches of the time.

Let me know your thoughts on my suggestion, I'm happy to hear whether this approach is suitable for the pakset. I've pushed it to James, because I can't seem to push to Rollmaterial's fork.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 01, 2018, 11:47:41 AM
Thank you for your work on this. I have incorporated some of your changes but not all of them: in particular, the LSWR 28ft carriages need more work before they can be broken down into firsts, first/second composites and thirds: the thirds, being five compartments (thank you for researching this) will need new graphics showing five compartments. When creating any new vehicles, new translation texts are necessary, as is updating coupling constraints in all other vehicles in the pakset that have coupling constraints and that might need to couple to this type of vehicle (i.e. all those that already have coupling constraints allowing coupling with any of the other 28ft LSWR carriages).
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on December 24, 2018, 10:11:03 AM
You're most welcome. I'm hoping to contribute in a small fashion, fixing easy issues that require time rather than skill.

Quote from: jamespetts on December 01, 2018, 11:47:41 AMthe thirds, being five compartments (thank you for researching this) will need new graphics showing five compartments.
Forgive me for asking, but isn't the following file a 28ft 6 wheel with 5 compartments? "Pak128.Britain-blends/trains/Carriages/lswr-6-wheel-28ft.blend". I have modified this file to produce two 1st/2nd class composite carriage suggestions. They're located here (http://gofile.me/28Rsd/W47X4ueMm) and here (http://gofile.me/28Rsd/tZFkUyHit), which if found suitable could be used to complete the addition. On one I have widened the compartments to fill the length of the carriage but kept the "inner" compartments as an array not yet confirmed into their position for easy adjustment. The other is modified to resemble the model shown on the page (http://www.orion-models.co.uk/LSWR_C1.html) referenced in the dat file. Are dat file constraint entries are case sensitive or limited in character types (such as "/")?

I've just read your other reply. Merry Christmas to you and your family! I've attached png previews so you don't need to produce graphics to see the changes.

Furthermore I'm offering changes to the 8 wheel LNWR carriage constraints, to allow the front brakes to be coupled to the passenger carriages. See the GitHub pull request (https://github.com/jamespetts/simutrans-pak128.britain/pull/54).
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 24, 2018, 06:15:47 PM
Thank you for this. I have just pushed a commit with an update including new graphics for the 28ft LSWR 6 wheel carriages. You are correct: the Blender file was indeed for a 5 compartment carriage, so I have created a new 4 compartment carriage as the 1st/2nd composite.

The reason that I have done this from scratch myself rather than used your version is that there are a number of complexities in the process of which your version did not take account, and I thought that the easiest way of explaining these complexities was simply to demonstrate how to deal with them by doing this myself.

First of all, the graphics. The graphics are now produced using the new, more efficient process taking advantage of the ability to use transparent pixels: see here (https://forum.simutrans.com/index.php?topic=2401.msg162208#msg162208) for a description of the transparent workflow. The carriages (and other pakset objects) produced using this new workflow look slightly different (and better) to those produced using the old workflow. This means that, when one object in a set (in this case, a variant of a type of railway carriage) is re-rendered using the new method, it is important to re-render all others in the set so that the appearances match. I am not sure how you produced the graphics above; I noted that they were not included in the Github commits.

Secondly, the naming conventions. In the translation texts, the front brakes are just called "brake" (not "brake (front)"), but the rear brakes are called "brake (rear)". This is because "(front)" is unnecessary information when a player is assembling a train in a depot. For the base names, the vehicles should be named "third", "second", "first" and "composite" - I believe that the / and \ characters cause problems in names. An example of the correct naming conventions in the translation texts is given in the commit incorporating these vehicles.

Thirdly, the comfort. Calibrating the comfort for the different classes of vehicles is a somewhat challenging process, as every vehicle's comfort must make sense relative to every other vehicle's comfort. There is a spreadsheet in the pakset here (https://github.com/jamespetts/simutrans-pak128.britain/blob/master/Comfort%20calibration%20overview.ods) which gives the comfort of every vehicle. This will need updating when adding multiple class variants of railway carriages. Using this spreadsheet makes the calibration of comfort values much easier than it would be without the spreadsheet. You will note that I have given lower comfort values for the second/first class than you had specified: that is because your values were closer to existing vehicles that had lavatories than to non-lavatory 6 wheeled first and second class carriages of the 1870s and 1880s.

Fourthly, "none" should not be removed as a next constraint on the front brake: doing this prevents players from having only one carriage in a train if they so desire.

Fifthly, the next/previous constraints. Adding any carriage requires updating all other carriages in the pakset that need to be able to couple to this carriage, not just other carriages of the same type. You will see in my commit the large number of carriages whose coupling constraints had to be changed to accommodate the new composite carriage. This is tedious but necessary given the way in which coupling constraints work in Simutrans.

Sixthly, the names of existing vehicles. You changed, e.g., "lswr-6-wheel-28ft-brake-rear" to "lswr-6-wheel-28ft-brake-rear-3rd". It is best not to change the base names unless this cannot possibly be avoided: just change the translation texts instead. This is because doing this will break all existing saved games that use this carriage (unless you add a suitable renaming entry in compat.tab), and will also break all references to this type of carriage in all other carriages' coupling constraints.

Thank you again for your work on this. If you can merge my version of the changes into your branch and revert any removals of the "none" constraint on front brakes, I can then incorporate your fixes automatically into the main branch.

It is very helpful of you to contribute in this way: the reason that I have gone into detail about the things that need to be rectified is that I very much appreciate your contribution and if you are able to remedy these issues in future work, I will be able to incorporate it automatically very efficiently, which will be extremely helpful to me and to any user of the pakset, as it will greatly speed the completion of the work in adding classes to the passenger carriages so far missing them.

If you are keen to continue this work, may I suggest that you work on LSWR, LBSCR and Southern carriages while I work on LNWR, MR, LMS and GWR carriages (and the others, such as L&TSR and Metropolitan, we can consider later)?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 25, 2018, 12:04:29 AM
Incidentally, I have now recommenced work on adding multiple class variants to carriages of this era, starting with the LNWR carriages, and have so far reached the late 1890s.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on December 27, 2018, 08:38:54 AM
Quote from: jamespetts on December 24, 2018, 06:15:47 PM
I am not sure how you produced the graphics above; I noted that they were not included in the Github commits.
I produced the graphics with the intention of showing a quick overview of my two suggested 4 door variants. They were made in blender using the files I mentioned in the post, but I only posted the two SW images for a quick comparison hoping to save you the trouble of opening each file in blender and generating the images yourself. I didn't create a pull request for these as you may have not wanted to use either model or make adjustments to suit. Please note the images are already using the transparent workflow, I was uncertain as to whether you still merge the images into the side by side image file or whether this workflow intends to not use combined images. I checked both the tutorial (which is great and I hadn't seen it before) and your link which implies keeping the individual images (as do your recent commits).

As to secondly, I've looked into the changes and I'll follow suit. The potential issues with / and \ are what caused me to enquire first.

To thirdly, may I suggest to indicate the comfort values not set prior and allow your revision to evaluate? I am just not that certain about how different operators and coachbuilders compare across the board.

Quote from: jamespetts on December 24, 2018, 06:15:47 PM
Fifthly, the next/previous constraints. ... This is tedious but necessary given the way in which coupling constraints work in Simutrans.
I agree entirely as it's very tedious but essential to the different coupling systems available. Do you use some automated system or a form of reference checking?

Quote from: jamespetts on December 24, 2018, 06:15:47 PM
It is very helpful of you to contribute in this way: the reason that I have gone into detail about the things that need to be rectified is that I very much appreciate your contribution and if you are able to remedy these issues in future work, I will be able to incorporate it automatically very efficiently, which will be extremely helpful to me and to any user of the pakset, as it will greatly speed the completion of the work in adding classes to the passenger carriages so far missing them.
I am very grateful for you going to the extent of answering my questions in such detail and clarity.

Quote from: jamespetts on December 24, 2018, 06:15:47 PM
If you are keen to continue this work, may I suggest that you work on LSWR, LBSCR and Southern carriages while I work on LNWR, MR, LMS and GWR carriages (and the others, such as L&TSR and Metropolitan, we can consider later)?
This sounds like a good start to get things moving. I'll be keeping an eye on your changes and will try to emulate these.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 27, 2018, 03:29:43 PM
Splendid, that is helpful: thank you. Yes, the idea is to keep the individual files per direction, as this eliminates an entire step (and also eliminates reliance on some specialist Windows-only software that was necessary to automate this task), greatly improving the efficiency of production of the graphics.

In relation to comfort, yes, do double check with me whether the values are sensible - but it is always a good start to cross-refer to the spreadsheet. May I suggest adding your values in the spreadsheet in a different colour (e.g. blue) if you wish them to be marked for review?

As to the coupling constraints, I do not have an automated system for checking this: I simply add the additional constraints to the end of the existing constraint pattern by copying/pasting.

In any event, thank you again for your help with this: it is much appreciated.

Edit: I have now merged your fixes to the LNWR 8 wheel constraints, and added some fixes to your fixes to show what needs to be done when re-naming a vehicle: an entry needs to be added in compat.tab and the translation texts need to be amended.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on December 28, 2018, 11:24:45 AM
Quote from: jamespetts on December 27, 2018, 03:29:43 PMand also eliminates reliance on some specialist Windows-only software...)

This is a good step for me as I've made the move to Linux this year, your efforts to stay platform independent are well appreciated. In the Output module in Blender (after the png & RGBA selection), do we use 8 or 16 bits of colour depth on the images? I noticed inconsistencies and want to be certain.

About LBSCR 6 wheel carriages: There are each a third class umber (lbscr-6wheel-third-umber.blend) and a second class umber (lbscr-6wheel-second-umber.blend) carriage in the blend repo. They were added to github at the same time, but don't quite look the same. I'd like to know which one I should edit to adapt liveries.

The LNWR fixes seem to have overfixed the convoy assembly, but a quick scan over the files hasn't revealed any obvious mistake. Have we misprinted a name which the game resolves to a "none" constraint?

I apologize for the snails pace, the holidays are a less productive time for me. If there are pointers to compiling the pakset on linux, I would be grateful. If not I'll fire up a VM and use Windows as it's not that difficult.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 28, 2018, 01:31:04 PM
Interesting that you should have noticed inconsistencies between 8-bit and 16-bit colour rendering: I had not spotted that. I think that 8-bit should be sufficient for our purposes.

In respect of liveries, the definitive Blender colours for each livery in the pakset can be found here (https://forum.simutrans.com/index.php/topic,2401.msg140831.html#msg140831). Thus, if there are any inconsistencies in livery colours in the .blend files, they should be resolved to the colours in that list.

In respect of the LNWR constraints, can you elaborate on what you mean by "overfixed" here?

As to compiling the pakset on Linux, there is a makefile, so you just need to type "make" (or "make -j[X]" where "[X]" is a number of threads to use, e.g., "make -j5" to use 5 threads, one more than on a four core CPU, as one of the threads does very little work) on the command line in the directory where the pakset is situated. This is actually a better build process than using the Python script in Windows (it is faster, too).
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 30, 2018, 12:53:51 AM
I have been working on adding class definitions to carriages to allow a complete timeline of carriages with class definitions. I have now completed all LNWR and LMS carriages (save for the LMS push-pull carriages, which I hope to complete to-morrow). Here are some highlights:

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49161278_707170489670214_8748612515233005568_n.png?_nc_cat=105&_nc_ht=scontent-lhr3-1.xx&oh=22df176344b1db5d739ed7f7387b90c4&oe=5C9EA19D)
LMS non-corridor carriages

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49043055_707170483003548_7904554731790401536_n.png?_nc_cat=102&_nc_ht=scontent-lhr3-1.xx&oh=114312edb8903083febd4f14056dc7a1&oe=5CD07123)
LNWR non-corridor carriages

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49206626_706109403109656_7475803488253902848_n.png?_nc_cat=105&_nc_ht=scontent-lhr3-1.xx&oh=df5508928ae0b07ad4a58f493a65f627&oe=5C9D07FD)
LMS corridor carriages

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49203430_706109413109655_8751374320938254336_n.png?_nc_cat=103&_nc_ht=scontent-lhr3-1.xx&oh=c497e0cb4a6969b0e1e5a8fc82a519e8&oe=5CD5EFEE)
The LMS "Coronation Scot" newly in its original blue livery (the red livery is available as an alternative), with new kitchen and dining saloon cars, all with appropriate class definitions.

The next task is to add class definitions to the LMS push pull set and then to the narrow gauge carriages, which currently entirely lack class definitions, before returning to add class definitions to more carriages of different railway companies (e.g. the Midland Railway).
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 30, 2018, 02:22:48 PM
Here are the LMS push-pull carriages re-rendered and with added class definitions: the centre carriage is a composite. The locomotive (the LMS 3F "Jinty") has also been re-rendered.

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49589775_707491446304785_3553833679342010368_n.png?_nc_cat=106&_nc_ht=scontent-lhr3-1.xx&oh=a474c679df7ca3af34da67652eedf853&oe=5CD854D7)
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 01, 2019, 02:00:12 AM
I have been working especially on narrow gauge vehicles of late. All the narrow gauge vehicles have been re-rendered and re-scaled (they needed to be reduced to 0.89 of their former size, as they were too large), and class definitions have been added for passenger carriages.

A number of entirely new narrow gauge vehicles have also been added, such as a number of different Hunslet quarry locomotives (previously there was only the one, the "Alice" class), and some workers' carriages (the originals were known as "quarrymen's carriages") set to the "very low" class (i.e. 4th class) by default to allow low paid workers to reach industries away from population centres. Standard gauge railways also had what were known as "workmen's trains", but these were generally made up of old third class carriages; the narrow gauge railways (such as the Ffestiniog, on whose carriages these are modelled) had no surplus old carriages, so had to build these from scratch. The original open wagons (which can be overcrowded by the brave quarry workers sitting on the outside rims of the carriages with their boots dangling over the edge as they did in reality, and which is depicted in the loaded graphics) can be upgraded to the enclosed versions (both shown), and both are later superseded by an all-enclosed, continuously braked vehicle.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 02, 2019, 03:35:41 PM
I have now completed work on the narrow gauge vehicles, concluding with Lynton & Barnstaple Railway locomotives (the Manning-Wardle 2-6-2T in both original and superheated versions) and the carriages (a third, a composite and a brake third):

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49898468_709429099444353_517893800866086912_n.png?_nc_cat=101&_nc_ht=scontent-lhr3-1.xx&oh=7c315b83516919d481be77335036b8d8&oe=5CC4F809)

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49572474_709429089444354_8470909121103659008_n.png?_nc_cat=109&_nc_ht=scontent-lhr3-1.xx&oh=4115dfc0210f7fec01317ff1d42c8ba6&oe=5CC5A489)
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 03, 2019, 02:24:00 PM
Returning to standard gauge, I have been working on the carriages of the London, Tilbury and Southend Railway. Here are some of the 6 wheel carriages, re-rendered and now with class definitions. 4 wheeled carriages have also been completed:

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49274294_709933682727228_4014651586797109248_n.png?_nc_cat=107&_nc_ht=scontent-lhr3-1.xx&oh=ae49ee246850f715d706a2b9d47eb3b9&oe=5CD5C7C0)
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 03, 2019, 06:36:37 PM
A quick update on my progress: I've managed to compile the pakset and add classes to the LBSCR 6 wheel carriages. I still need to fiddle with the offsets, but that is what remains. A github pull request should surface on the weekend.

Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 03, 2019, 07:00:38 PM
Thank you for this. May I ask why you need to use offsets? If you align the carriages correctly in Blender, no offsets are required.

The easy way to align the carriage is to select the relevant layers, press "a" to select all in those layers, press numpad "5" to get an orthographic projection, press numpad "1" to get an exact end-on view to align horizontally, then press numpad "3" to align vertically and laterally with an exact side-on elevation. You need to have the buffer stops exactly four large Blender squares to the left of the centre line.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 03, 2019, 11:46:13 PM
The LT&SR bogie carriages have now had added class definitions. Here are the non-corridor carriages:

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49211312_710139759373287_5124711594122543104_n.png?_nc_cat=101&_nc_ht=scontent-lhr3-1.xx&oh=3321f0bc388336ba91e5489421ae563d&oe=5CC5A56B)

and here are the corridor carriages:

(https://scontent-lhr3-1.xx.fbcdn.net/v/t1.0-9/49209918_710151319372131_8901002009344212992_n.png?_nc_cat=105&_nc_ht=scontent-lhr3-1.xx&oh=bf397eec919d0adb4012e3b2d28e9ad9&oe=5CCBE17E)
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 04, 2019, 05:04:36 PM
The LBSCR 6 wheel carriages now feature multiple classes and updated graphics.

Early livery, with a variety of trains. The long train is an express with mail carriages and composites, and the two behind it feature either fast goods or fruit and milk carriages:
(https://i.imgur.com/3QzxoSg.png)
Late livery:
(https://i.imgur.com/5NwClNm.png)

I hope the scaling is correct and the alignments are satisfactory. The difference in colour of the mail and goods carriagescan be attributed to fewer windows. The composite carriages feature a luggage hold, the firsts only four compartments and each goods waggon comes in braked and unbraked variety. I'm looking for feedback in both modelling and costs/capacity.

edit: Thank you James for your patience and support. Once on the right track, your workflow becomes apparent and quite natural. I thought I needed offsets for reversed vehicles, but turning them around in blender does the trick.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 04, 2019, 05:23:26 PM
Excellent - this is definitely progress! These do look good. Do not worry if the locomotives look slightly large for the carriages: these have not been re-rendered with the new workflow yet. Once re-rendered, because the edge pixels will have an alpha channel, and will blend into the background, the locomotives will look the correct size.

The easiest thing to do with reversing end-brake vehicles is to rotate them (select the relevant layers, "a" to select all, "r" to rotate, "z" to confine rotation to the z axis and type "180" to rotate by 180 degrees: much easier than trying to get it precise with the mouse).

Indeed, a good technique for carriages that I have used on some of the later ones (including the LNWR 57ft carriages recently re-rendered) is to have on .blend file for multiple carriages that share the same body profile and length, and just have a different layer for the windows/doors/duckets which can then be switched in and out for rendering multiple carriages. This makes adding multiple liveries much easier. If you do it this way, you can just have one set of windows/doors/duckets for one direction and one for the other, leaving the carriage body, underframe, etc. in one piece.

Thank you very much for helping with this: it is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 04, 2019, 06:12:15 PM
Thank you for the pull request for this. However, the definitions seem to be missing the .dat files for the new carriages - did you forget to issue the "git add --all" command (or equivalent if using a Windows GUI frontend)?
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 05, 2019, 08:04:41 AM
Quote from: jamespetts on January 04, 2019, 06:12:15 PM
Thank you for the pull request for this. However, the definitions seem to be missing the .dat files for the new carriages - did you forget to issue the "git add --all" command (or equivalent if using a Windows GUI frontend)?
I added the new carriages into trains/lbscr-6wheel-carriages.dat in order to keep clutter at a minimum. This link (https://github.com/jamespetts/simutrans-pak128.britain/pull/56/files#diff-d221c5641c75af1d0189a187b46ca9e7) should show the file. Alternatively, one can filter out the .png files and the .dat file diff should not have expanded by default as there are a lot of additions. On the topic of pull requests, should I make a pull request on the Blends repo?

Quote from: jamespetts on January 04, 2019, 05:23:26 PMThis makes adding multiple liveries much easier. If you do it this way, you can just have one set of windows/doors/duckets for one direction and one for the other, leaving the carriage body, underframe, etc. in one piece.
I'm worried I might be missing details in liveries as I believe I know far less about the looks of the vehicles as you or the other contributors. But with enough sources and time, these obstacles can be overcome.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 05, 2019, 01:05:52 PM
Quote from: Spenk009 on January 05, 2019, 08:04:41 AM
I added the new carriages into trains/lbscr-6wheel-carriages.dat in order to keep clutter at a minimum. This link (https://github.com/jamespetts/simutrans-pak128.britain/pull/56/files#diff-d221c5641c75af1d0189a187b46ca9e7) should show the file. Alternatively, one can filter out the .png files and the .dat file diff should not have expanded by default as there are a lot of additions.

Ahh, I see - that seems to have been my error in not fully understanding the Github web interface: for some reason, those files did not show up under an individual commit when I looked at them.

However, now, on inspecting the web interface, I get an error to the effect that there are merge conflicts too complex to resolve at the command line. I suspect that the comfort calibration spreadsheet may be at fault here. Can you see if you can merge my latest changes into your branch, resolve the conflicts as necessary (you might have to do this manually with the comfort calibration spreadsheet by noting your changes in a separate spreadsheet, deleting your version, pulling mine, then manually re-applying yours), and then issuing a fresh pull request for the merged branch?

The comfort calibration spreadsheet can potentially create problems if two people are working on it at the same time, so we may have to think of a way around that.

Quote
On the topic of pull requests, should I make a pull request on the Blends repo?

Yes, please do: that would be most helpful.

QuoteI'm worried I might be missing details in liveries as I believe I know far less about the looks of the vehicles as you or the other contributors. But with enough sources and time, these obstacles can be overcome.

For the LBSCR, I was able to find most of what I needed either from the web or from general books about railway carriages, and likewise the LSWR. If you need information on a livery or details of a particular carriage, please do ask, as I do have a number of books on railway carriages. Sometimes, it is necessary to make an educated guess about how a particular carriage would have been configured.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 06, 2019, 08:10:33 AM
I have issued a pull request for the Blends repository and the pak repository can now be automatically merged.

Quote from: jamespetts on January 05, 2019, 01:05:52 PM
The comfort calibration spreadsheet can potentially create problems if two people are working on it at the same time, so we may have to think of a way around that.
Does github work with databases? It may make entering/extracting data from the .dat files easier if scripted.

Quote from: jamespetts on January 05, 2019, 01:05:52 PMSometimes, it is necessary to make an educated guess about how a particular carriage would have been configured.
I hope my attempts are satisfactory and in any case feel free to adapt/correct further.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 06, 2019, 02:02:51 PM
Thank you very much for this. I have now incorporated the changes. I have made some minor corrections to the translation texts for two of the carriages: you had missed "(third class)" from one of the brake carriages, and the word "composite" should go before "carriage" rather than in the brackets with the class description.

In terms of the .blend files, one or two things that I notice: first of all, make sure not to save the .blend file after the creation of the .png files: always save before this process and then revert to that save immediately after the export process has finished. This is because that process will leave the camera/light sphere aligned incorrectly for exporting again, and this is not easy to correct.

Secondly, I notice that you have left the anti-aliasing at 16, whereas it generally looks better at 11 when using the new workflow.

Thirdly, I notice that you had some objects in layer 3 in at least one of the .blend files (the LBSCR 6 wheel first class carriage): layer 3 is reserved for the light sphere. It is usually best to disable layer 3 when editing, and enable it only when actually exporting/rendering.

In any event, thank you again for this work: it is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 12, 2019, 10:21:14 AM
Thank you for the feedback. I understand it is vital to creating a coherent and well set up repository. I'll try my best to keep in line with the pointers given.

Should I re-render the LBSCR 6 wheel carriages? I understand the desire to get things right while things are still fairly fresh.




Quote from: jamespetts on January 03, 2019, 07:00:38 PMYou need to have the buffer stops exactly four large Blender squares to the left of the centre line.

I've attached a preview of re-rendered LBSCR 4wheel carriages, which I hope to commit later this weekend. Is there anything to be done about the mail carriages' distance to each other? Changing length in the .dat file doesn't help much.


Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 12, 2019, 08:35:38 PM
Thank you for that. It may be helpful if you could re-render the 6 wheel carriages with the anti-aliasing set to 11 to ensure a consistent look; the difference is fairly subtle, but they can seem a little too soft on 16 with the alpha blending enabled.

Thank you for progressing to the 4 wheelers: that is helpful. As to the distance with the mail carriages, can I ask you to clarify what you mean when you state that changing the length in the .dat file does not help much? Do you mean that increasing it by 1 makes the gap too big, or that changing it seems to have no effect at all? If the latter, if you are using a train created when the length was at a different setting, note that you will need to send the train to the depot and out again to see the difference.

Incidentally, it would be very helpful if you could re-upload the reproduction case for the bug in this (https://forum.simutrans.com/index.php/topic,18639.0.html) thread, as there is some reason to believe that fixing this might possibly fix the loss of synchronisation error on the server.

Thank you again for all your work on this: it is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 13, 2019, 01:36:29 PM
Quote from: jamespetts on January 12, 2019, 08:35:38 PM
Incidentally, it would be very helpful if you could re-upload the reproduction case for the bug in this thread, as there is some reason to believe that fixing this might possibly fix the loss of synchronisation error on the server.
I'm sorry I missed that response. The link has been updated and should remain easily accessed for a long time.

I have committed updates to the LBSCR 4 wheel carriages and have created equivalent pull requests. An attached image showcases the vehicle improvements. Again I'm looking for markers missed and criticism. The LBSCR 6 wheel carriages will be updated in due time, I've spent far too long writing a script to update the image descriptors in the .dat files (trying my hand at bash scripting).

Quote from: jamespetts on January 12, 2019, 08:35:38 PMDo you mean that increasing it by 1 makes the gap too big, or that changing it seems to have no effect at all? If the latter, if you are using a train created when the length was at a different setting, note that you will need to send the train to the depot and out again to see the difference.
I meant the former, where adding an increment can provide a large distance between vehicles, although I'm certain this is exacerbated by the small size of carriages in the 19th century.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 13, 2019, 09:22:58 PM
Thank you very much for this. As to the length of the carriages, this is a known issue with the granularity of the length parameter, inherited from Standard. The only thing to do is find the length parameter that seems the least bad.

Thank you for your work on the 4 wheel LBSCR carriages. The first thing that I notice looking at the commit is that you have added a front and rear full brake; may I ask what the reason for this is? The full brake carriages are usually agnostic as to whether they are at the front or rear, as they are symmetrical, unlike brake ended passenger carriages where the brake end needs to be on the outside of the train.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 14, 2019, 05:16:42 PM
I agree that the full brake carriages should be used as found suitable. However in the case of the specific full brake vehicle; the .blend file I found consisted of a rear brake coach and I decided to add a front coach to allow a train formation aesthetically pleasing. If I'm mistaken in the .blend file, then we should remove the newly added vehicle.

Citing the source (https://basilicafields.wordpress.com/2010/12/19/lbscr-carriages/) in the .dat file (https://github.com/Spenk009/simutrans-pak128.britain/blob/master/trains/lbscr-4wheel-second.dat) for LBSCR 4 wheel suburban stock: "These suburban carriages were close-coupled in semi-permanent sets by a central buffing fixture with side chains, and standard buffers were only fitted to the brake end of brake-thirds.", may I suggest we constrain the player into using Stroudley suburban carriages with each other? I would also look into adding a third class carriage for the four wheeled carriages (using the images of the second as they would appear identical), to complete the set, or even some of the nine types introduced if that's sensible to do at this point.

edit: I've re-rendered the 6 wheel graphics for 11x AA. Next up would be the LBSCR/SR 48ft/54ft augmented/sub/cor/non-cor conundrum. Can you remember where you retrieved the information for allocating each type to their respective .dat entry? Also, if you have a system for the .blend graphics I would be most grateful for a short overview of the layer magic.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on January 19, 2019, 11:53:52 PM
Hi, now I'm playing a game with 1870's 4-wheeled carriages and noticed an inconsistency in lengths:

name=4-wheel-1870s-brake-unfitted
length=4
upgrade[0]=4-wheel-1870s-brake-fitted
----
name=4-wheel-1870s-brake-fitted
length=5


Also visually there are 3-different lengths:
4-wheel-1870s-brake (lenght=4) - is the shortest
4-wheel-1870s-tricomposite (length=4) and 4-wheel-1870s-luggage-third (length=5) look visually about the same length (a bit longer than the brake carriage)
all others (composite, suburban, mail, milk, fast freight) are length=5 and look the same, and are longest.
I understand that the granularity is a problem, but at least the 3-composit and luggage-3rd should be consistent - either 4 or 5, and perhaps made visually consistent either with the short mail-brake or the longer suburban and composite carriages). I tried both lengths, 4 is too close, 5 is too far. Also they are slightly misaligned comparing to all others (including the short mail brake van). (If you look at e.g. *_S.png you will see that the 3-composite and luggage-3rd's close end is one pixel closer (to the left).
I also tried to have the mail-brake with length=3, it is too close but looks consistent if the other two are length=4. However they are too close to each other, and I think that at that time carriages were a bit further apart from each other. so length=4 for the brake van and length=5 for all others seems to be better, even if two of them are a bit shorter.

Further thoughts:
I think there is more inconsistencies in this set. I just wonder why they have different lengths at all. And also the capacities do not match. So what we have:
Suburban 3rd (comfort 52): capacity 5x10 + 5x5 (overcrowded) = 50 + 25 (5 compartments - 5 windows), OK
Suburban 3rd (comfort 52) brake: 3x10 + 3x5 = 30 + 15 (3 compartments, but 4 windows, OK 1 can be for the guards compartment?)
Suburban 1st+2nd (comfort 66/60): 2x8 (1st) + 2x8 (2nd) + 2x4 (overcrowded) = 16 + 16 + 8 (4 compartments - 4 windows perhaps more space for stretching legs, OK), but I think the 1st class should be only 2x6.
So far almost OK.
Luggage 3rd (comfort 62) : capacity 50+25, hmm 4 - windows with blank in the middle (luggage compartment + 4 passenger compartments). And shorter. So comparing to suburban we have shorter carriage, less compartments, but same capacity and higher comfort ? Completely wrong. I think this should be rendered to the same length, and capacity only 4x10 + 4x5 = 40+20.
Luggage 1st+2nd (comfort 77/68): capacity 2x6 (1st) + 2x10 (2nd) + 2x5 (over) = 12+20+10. Again we have the same capacity as suburban 1+2, luggage compartment and higher comfort. Really? I think it should be 2x6 + 2x8 + 2x4, and maybe even changed to only 3 compartments (only 1 of 1st class), to have space for luggage and more comfort as in suburban composite.
Luggage tricomposite (comfort (77/68/63): capacity 6+8+10, but according to http://www.vintagecarriagestrust.org/MS&L.htm, it had 4 compartments, so more probably 6+8+2x10. Also this one is shorter. Well it would make sense for 3-compartments, but that again would be more suitable for 1st+2nd composite.

There is nothing like passenger + luggage brake car, so one has to combine the luggage cars either with two mail brakes (which is kind of overkill - too much mail capacity), or suburban brakes. What about having suburban brake with 4 compartmetns (capacity 40+20), and luggage brake with 3 compartments like http://www.preservation.kesr.org.uk/coaching-stock/l-nwr. (And capacity 30+15).

Mail (non brake) and TPO have the same length as suburban 5-compartment coaches, but the mail brake is much shorter. Was it really so? The capacity as roughly 2/3 of the unbraked one, and there is part for the guard. I think again that this should be the same length. Also it would be nice to have a front and rear variant, as the picture is assymetric. The guards compartment is somewhat visible.

I do not know more sources than those above, so there might be some historical evidence for it being so, but this is my proposal
- all 4-wheel 1870s fitted and unfitted should be rendered at the same length=5
- mail brake should have front and rear model (roatated), may be shorter, but scaled properly for length=4 (the current image is even shorter, but too long for length=3)
- there should be suburban brake (4-compartments) and luggage brake (3-compartments) - 3rd class
- luggage 3rd should have only 4-compartments capacity (and full length)
- luggage tricomposite - should have 4-compartments 6+8+2x10 and full length
- luggage composite (1/2) - should have 3-compartments 6+2x8, and may be shorter, but should be rescaled. Current 3-compartment image is too long for length=4. Or it could be a 1st + 3rd composite with 4 compartments?
- suburban composite (1/2) should have capacity 12+16 instead of 16+16
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 20, 2019, 11:05:31 PM
Spenk - thank you for your work on this, and apologies for not having had a chance to look into this yet: I have been very busy this week-end organising my shed, so I have not had any time for Simutrans.

Vladki - thank you for your thoughts. The LBSCR carriages were based on specific known examples, although I think that these were initially done before I had taken to recording the source for each vehicle in a comment in the .dat file.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on January 21, 2019, 12:01:53 AM
I'm not taking about lbscr, but those that have liveries for LNWR, LSWR, GNR, GWR and MR. And the links I have posted are taken from dat files.
Anyway I think a slight derivation from real dimensions to better fit the granularity of simutrans lengths is better than strict adherence to scale.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 22, 2019, 11:09:36 AM
Vladki, thank you very much for your suggestions. I can look into this and play around with the lengths, but I think the carriages are based on real sources and therefore have specific lengths. Is the issue here in aesthetics or convoy assembly? I concur with the issue of passenger capacities in the carriages, these can be amended (maybe very little overcrowding allowance in 2nd class to add an incentive other than slight price difference and valuable comfort advantage).

Maybe we should make more use of luggage carriages. If they can work like a TPO for mail we can give the player the chance to reap the benefits of adding luggage carriages to their long distance trains. An alternative would be to make another (lower) catering class and add luggage sections into them while retaining higher comfort for carriages with internal luggage storage (above or below seats or at exits).
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on January 23, 2019, 07:57:16 AM
Hi spenk,

I started to dig into this issue because of bad appearance. If you use some if the shorter carriages, you'll get irregular spaces between carriages.

However the capacity issue is also important. Especially the 3rd suburban vs. 3rd luggage is striking (shorter, less compartments but higher comfort at the same capacity?)

I wouldn't add mail nor tpo to luggage versions. There are already dedicated mail /tpo carriages and the mail brake van has also quite a lot mail capacity. Catering also does not make much sense when you cannot walk between cars during the ride.

Suburban vs. Luggage is already differentiated by comfort and loading time. That's fine. Luggage cars have higher comfort (no luggage on your knees) and higher loading time.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on January 23, 2019, 12:15:17 PM
I can make the changes and if James sees fit they'll be incorporated. I'll try to post the improvements side by side.

Quote from: Vladki on January 23, 2019, 07:57:16 AMLuggage cars have higher comfort (no luggage on your knees) and higher loading time.

I see your point, but I mean nothing about adding a  mail hold nor  accessible luggage catering during the trip (even if corridored sets allow this). I mean that a dedicated luggage carriage would allow passengers from others to decide that leaving their luggage in the separate car is sensible from a certain length of journey on (trips longer than 2/3 hours aren't usually routine commutes). So if the player adds a luggage carriage to a long distance train, it would be beneficial to longer trip taking pax (class dependent).
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on January 23, 2019, 04:22:52 PM
I think I understand what you mean by luggage catering, but that would have to be a bigger dedicated luggage van. The luggage compartments are imho small to fit only the luggage of passengers in the same car.

Also overcrowding makes sense to me only in the lowest class in train. Would you pay extra to travel in car with nice soft seats if you couldn't sit down? Also standing passengers usually stay in the corridor, not in compartments. So I would be more inclined to reduce the overcrowding even in 3rd class for these carriages
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 24, 2019, 12:02:48 AM
There are some complex issues that have been raised. Firstly, length: it is generally preferable for all carriage graphics to have as accurate a length as possible. The lack of granularity in the length parameter is unfortunate, but all sorts of visual chaos will ensue if we start having carriages of deliberately the wrong length to work around this. Far better would be to work on allowing fractional lengths in the code (although this may be some way off unless someone else codes it).

Secondly, luggage: any representation of luggage in comfort must take account of the fact that the luggage carrying capacity of one vehicle in a train can affect the comfort of other vehicles. Doing this would require a change in the code (catering values are unsuitable, as they generate revenue as well as increasing comfort). For this reason, the simulation of luggage cars has generally been omitted, and, if included in some cases, would require revision of the entire pakset to add luggage vehicles to match all of the various railway carriages, which would be a very large task.

As for overcrowding, the policy so far has been to allow overcrowding in all but first class carriages/compartments. The general rule of thumb is, for non-corridor compartment stock, the overcrowded capacity is half as much again as the seated capacity, as generally as many people can stand in the middle of a compartment as can sit on either side of it. The same holds in corridor compartments, but people can also stand in the corridor, giving a higher ratio of overcrowded to seated capacity, and the same in saloon carriages.

Although people may prefer to stand in the corridor given the chance, people would undoubtedly have stood in compartments in corridor carriages if there were no other space available: it is not plausible that people would have refused to get on the train at all just because they have to stand in a compartment rather than in the corridor. People will stand anywhere with enough space if the train is crowded enough.

Edit: I have now had a chance to look over the code. I see that you have re-rendered quite a lot of vehicles: that is very helpful: thank you for that. I do take the point about the front/rear full brakes, as the graphics do show them having a particular end.
However, one thing that I do notice is that you have in some places (e.g. the LBSCR-4Wheel-First-fitted) added first class without modifying the capacity or comfort. The carriage, although modelled on a first class carriage, had the capacity and comfort that it would have had had it been built as a long-distance third class carriage: before classes were introduced, in the LBSCR carriages, class was used as a proxy for whether the carriages were long distance or suburban.

You will thus need to recalibrate the comfort and capacity of these carriages to match with them being first class (i.e. three a side in each compartment and with a comfort matching contemporaneous first class vehicles). For reference, this is what an LBSCR 4 wheel first class compartment looks like:
(https://farm4.staticflickr.com/3776/9130912532_4e79013c3c_h.jpg) (https://flic.kr/p/eUSjbq)LBSCR 4 wheel first (interior) (https://flic.kr/p/eUSjbq) by James Petts (https://www.flickr.com/photos/14730981@N08/), on Flickr
(https://farm4.staticflickr.com/3689/9130906476_24b8c93855_h.jpg) (https://flic.kr/p/eUSho1)LBSCR 4 wheel first (interior) (https://flic.kr/p/eUSho1) by James Petts (https://www.flickr.com/photos/14730981@N08/), on Flickr
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on February 04, 2019, 06:08:49 PM
James, thank you for noticing this. I've gone ahead and changed the capacity, updated the comfort and removed/reduced overcrowding (1st=0,2nd=12 of 50). I'll be happy to use the gorgeous pictures in the rest of the album for future reference. I've also moved a blend file or two to the Blends repo and removed some of the older non-transparent images.

I was wondering whether we should constrain the LBSCR suburbans as the source provided (https://basilicafields.wordpress.com/2010/12/19/lbscr-carriages/) mentions: "These suburban carriages were close-coupled in semi-permanent sets by a central buffing fixture with side chains, and standard buffers were only fitted to the brake end of brake-thirds."

Vladki:
Quote from: Vladki on January 19, 2019, 11:53:52 PM
- luggage 3rd should have only 4-compartments capacity (and full length) <- I've amended this for the luggage carriage
- luggage tricomposite - should have 4-compartments 6+8+2x10 and full length <- Your source is correct, but our carriage has only three visible compartments as seen here. (https://github.com/jamespetts/simutrans-pak128.britain/blob/master/trains/images/lnwr-4wheel-tricomposite_E.html)
- luggage composite (1/2) - should have 3-compartments 6+2x8, and may be shorter, but should be rescaled. Current 3-compartment image is too long for length=4. Or it could be a 1st + 3rd composite with 4 compartments? <- Is this not a four compartment? Image (https://github.com/jamespetts/simutrans-pak128.britain/blob/master/trains/images/lnwr-4wheel-composite_E.png) In any case this can be rendered.
- suburban composite (1/2) should have capacity 12+16 instead of 16+16 <- It's a suburban vehicle, where we won't have too much reason to splurge on comfort where not needed. I think we should keep this.

Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on February 05, 2019, 04:48:02 PM
Quote from: Spenk009 on February 04, 2019, 06:08:49 PM
- luggage 3rd should have only 4-compartments capacity (and full length) <- I've amended this for the luggage carriage
OK, good
Quote
- luggage tricomposite - should have 4-compartments 6+8+2x10 and full length <- Your source is correct, but our carriage has only three visible compartments as seen here.
- luggage composite (1/2) - should have 3-compartments 6+2x8, and may be shorter, but should be rescaled. Current 3-compartment image is too long for length=4. Or it could be a 1st + 3rd composite with 4 compartments? <- Is this not a four compartment? Image In any case this can be rendered.
I thought about swapping the images for these two. So the tricomposite would be 4-compartments with luggage space in the middle, and 1st/2nd composite would use the 3-compartment image.
Quote
- suburban composite (1/2) should have capacity 12+16 instead of 16+16 <- It's a suburban vehicle, where we won't have too much reason to splurge on comfort where not needed. I think we should keep this.
If there is no reason fir extra comfort, then there is no reason for the 1st classs. I take it that 1st class is 6 seats, 2nd class 8 seats. And the difference betweeen suburban and long distance is only the luggage compartment, that takes approximately the same space as one 3rd class compartment. And 1st/2nd class compartmens have some extra legroom, so only 4 fit the same lenght where 5 3rd class compartments would fit.

So for 3rd class it would be like: 5-compartments (suburban), 4-compartmens (luggage), 4-compartments (brake), 3-compartments (luggage+brake).
Composite would be: 2+2 compartments (suburban), 1+1+2 compartmetns (luggage),  3-compartment composite with luggage could be anything 1+2 or 2+1 or 1st/2nd class only
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 05, 2019, 11:34:34 PM
Thank you very much for that: I have now incorporated that (having added the English translation texts for the class names for the 4 wheel carriages).

For the 4 wheel suburban carriages, I agree that these should have an exclusive constraint set. Also, you might reduce their length to simulate the shorter buffering, which will allow more of them to fit in short platforms.

Your work on adding classes to the pakset is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on February 15, 2019, 06:32:32 PM
I have been scratching my head at some of the configuration of the lbscr carriages. I'm listing my findings with proposed solutions.

Craven Carriages (built 1866 - 1872) - suburban carriages due to their comfort values (lo-38, med-42).
The Cravens don't feature a first class. If this is intended due to the suburban nature, that's understandable.
The Cravens have mail carriages, which seems unusual for a suburban train. If you're supposed to use them to help connect up the mail networks using select stations with mail capability, we could mention it to players in the vehicle comment/copyright section.

Stroudley Carriages (built 1872, improved 1875 - 1901) - main line carriages due to comfort (low-62 but brakes have 52, med-68, hi-77)
As previously mentioned, I've created a third class vehicle and also a composite (source (https://basilicafields.wordpress.com/2010/12/19/lbscr-carriages/)). It is in the newest pull request.
These Carriages retire Jan 1872, 6 wheel improvements don't arrive until December 1872. I suggest moving retire_month to November.
These carriages are main line / long distance. I suggest I lower the capacities for second, if not third class, to reflect the improved comfort. The brake carriage will be fixed to 62 comfort.
I create a set of suburban carriages with higher 1st class capacity, lower comfort across the board, constrained to each other and slightly cheaper purchase price.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 16, 2019, 12:33:06 AM
Thank you for your thoughts in relation to this. A little historical information may be of assistance in working out how these are supposed to work.

Before the 1870s, there were no separate long distance and suburban carriages. It was the LBSCR who first introduced this idea, I think, in the 1870s (or, at least, at around the same time as other railways). Early railway carriages were really extremely uncomfortable: they were small and cramped, had a hard ride, were draughty, had no heating (other than the ability to hire hot water bottles at stations), no lavatories and crude lighting consisting of a rapeseed oil pot lamp (one per two compartments in third and sometimes second class quite often) which produced very little light but a fair bit of smoke.

Second and third class carriages would have had the same capacity: the difference was simply that the second class carriages had padding on the seats: the third class carriages merely had wooden benches. Going back further beyond the 1860s to the 1850s and 1840s, the third class carriages would not even have had proper windows (save for one in the door).

In relation to the retirement date of the Craven carriages, the retirement date is April 1872, the same month as the unfitted Stroudly carriages are introduced. They are available until January 1877. However, fitted Stroudley carriages are available from September 1875 - I am not sure, therefore, what the issue with the dates is: can you elaborate?

In relation to your pull request, I imagine that you will need to modify it to take into account what I wrote above regarding 1860s carriages and the differnce between second/third class (unless you did not incorporate these features into the pull request, in which case, please let me know, and I will review it).

Thank you very much for your work on this.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on February 16, 2019, 08:08:04 AM
To the retirement dates: My apologies. I lost overview of the what was happening when. Of course braked four-wheel carriages are available before the introduction of six-wheel carriages.

I have not incorporated a capacity difference or suburban stock suggestion in the pull request. The intention was to reduce mainline compartment capacity from 10 to 8 (second and third following your description) and create an explicit suburban set (Cravens don't feature a first class which may be a result of being suburban). The main-line carriages shouldn't be constrained but the suburbans in this case could. I have thought about this further and unless players really want it, it was just a small idea.

I've amended files in the pull request to enable automatic merging. The changes are: addition of a composite vehicle, addition of a third class vehicle, fixing first class capacity, moved/removed a few .blend files.

I'd like to share that the 1850s to 1860s transition looks very good, the change in stock, locomotive power and track requirements is very nice. Thank you James for your patience with us. I would continue with lbscr-48/54ft-augmentation carriages next, or are there higher priority tasks in transparent workflow conversion?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 16, 2019, 12:24:56 PM
Thank you for this: that is helpful. It is probably better not to have suburban/mainline Craven carriages, as these did not exist in reality.

In relation to reviewing your pull request, a few things that will need looking at:

(1) you add some non-LBSCR fitted four wheelers (4-wheel-1870s-third-fitted), but the coupling constraints are not updated on all vehicles that can couple to the 1870s 4 wheelers to match;
(2) you change the name of LBSCR-4Wheel-Second to LBSCR-4Wheel-third without adding a compat.tab entry, meaning that any saved game in which players had a LBSCR-4Wheel-Second vehicle will be broken - the better solution here is probably to add a third class as well as the existing second class vehicle and modify the comfort of the second class vehicle;
(3) the full brakes need a "none" constraint as the last in the sequence of "next" constraints so that they can be at the back of trains; also, they should not have previous constraints, since they should be able to couple immediately behind the locomotive if the player desires - generally, it is preferable for full brakes (other than ones that can only go either at the front or back of the train, which are rare) to have no coupling constraints at all; and
(4) the comfort of the 1872 Stroudley carriages needs consideration: the pre-classes scheme had the 2nd class used as the long distance vehicle and 3rd class used as the suburban vehicle - in the comfort calibration overview.ods file, you will see that the suburban 1870s carriages are at a comfort of 52 and the low capacity carriages (i.e. long distance carriages) are at 62; I note that you change the comfort of the brake carriages from 52 to 62 "to match other vehicles in the set", but these vehicles are a different type and should have a different comfort: use the spreadsheet as the guide rather than the .dat files.

Thank you again for your work on this: it is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on February 16, 2019, 01:47:44 PM
Quote from: jamespetts on February 16, 2019, 12:24:56 PMThank you for this: that is helpful. It is probably better not to have suburban/mainline Craven carriages, as these did not exist in reality.
I meant this in regards to Stroudley introduced stock, which were converted main-line carriages.

(1) I must have made an error in the fundamental workflow here. Could you point out the missing additions?
(2) The Second class vehicle is not intended to be replaced by either the third or composite carriages. I supplemented as the source (https://basilicafields.wordpress.com/2010/12/19/lbscr-carriages/) mentions were built. Just in case, I started a savegame from an older nightly source, running a train with the assembly (https://i.imgur.com/RrY0oXH.jpg) as I expect intended, saved and loaded in a freshly compiled pak. I encountered no issues.
(3) I tried to find the source of this. I think it may be due to the fact that I've added a full brake front and amended the rear entry in en.tab, but this has not propagated to your branch.
Quote from: jamespetts on January 24, 2019, 12:02:48 AMI do take the point about the front/rear full brakes, as the graphics do show them having a particular end.
entries between upstream/master (https://github.com/jamespetts/simutrans-pak128.britain/blob/ff3b9c365049f22b5006dad6c1621c8ce764bddf/text/en.tab#L3093) and spenk009/master (https://github.com/Spenk009/simutrans-pak128.britain/blob/6f6ef36f110d4c681929c2fa0779757ae6946830/text/en.tab#L3149).
(4) That is an application I had not thought of. It's just that the brake carriages have comfort=52, there are no third class carriages to make use of as suburban and the comfort value defined across all comforts in the carriages introduced at the exact same time, scheme, type, looks have: comfort[0]=0, comfort[1]=62, comfort[2]=68, comfort[3]=77.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 16, 2019, 11:58:59 PM
(1) My apologies: I seem to have missed in my previous post that you had added these in the commit on at 1736h on the 15th of February.

(2) Looking at this again, I think that I might have misinterpreted the way in which Git for Windows presented the commit: it appeared to involve a deletion of LBSCR-4Wheel-Second and its replacement with LBSCR-4Wheel-third, but in fact it shows this replacement on a copy of the file which you had renamed to provide a new third carriage. My apologies.

(3) This relates to the 6 wheel carriages, in particular the carriage with the name LBSCR-6Wheel-guard: the current version in the pakset does not have any constraints defined, but your commit adds the same constraints as for a carriages such as the LBSCR-6Wheel-4-com-first, which would not allow it to be at either the beginning nor end of a train. The 6 wheel version does not have separate front/rear brake graphics.

(4) The 1870s carriages, unlike the 1860s carriages, need two schemes: one for the suburban carriages and one for the long distance carriages.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on February 17, 2019, 10:34:35 PM
Hello, I have noticed that some of the LBSCR 6-wheel carriages have retire year 1892 (passenger non-break), while others (passenger break and mail) have retire year 1907. Is that intended?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 17, 2019, 11:06:22 PM
I think so - the passenger carriages were replaced with bogie carriages, but, initially, bogie carriages were not built for mail/brake applications.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on February 19, 2019, 08:34:56 PM
But the bogie variants  have same intro (LBSCR-non-cor-lav  in 1889 and LBSCR-sub-bogie-42ft in 1894) for both brake and non-brake (although no mail).
So either the passenger 6 wheelers (both brake and non-brake) should end at same time, probably 1894 or both should be extended to 1907 to match the mail 6-wheelers.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 23, 2019, 12:51:46 AM
Those are the passenger brakes, not the full brakes: the only full brakes available are the 6 wheelers.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on February 23, 2019, 01:55:13 AM
Yes I just thought that the passenger brakes should have the same intro/retire as passenger non-brakes. So that only the mail 6-wheelers will be available longer.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 23, 2019, 10:36:12 AM
That would leave a period without any full brakes available, which would be problematic as it would prevent the formation of an all mail train.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on February 23, 2019, 11:29:04 AM
I'm not sure if I am clear enough. I propose to change the retire date of:

name=LBSCR-6Wheel-brake-front
intro_year=1877
retire_year=1907
freight=Passagiere

name=LBSCR-6Wheel-brake-rear
intro_year=1877
retire_year=1907
freight=Passagiere

to retire_year=1892 to match all other passenger (non-brake) 6-wheelers.

And to keep the full brake (and all mail/fruit/milk 6-wheelers) as is:

name=LBSCR-6Wheel-guard
intro_year=1877
retire_year=1907
freight=Post

Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on February 23, 2019, 12:29:44 PM
My apologies for misunderstanding: I see what you mean now. I have made the adjustment as requested.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on March 04, 2019, 08:48:49 PM
Speaking about lbscr-6wheelers, I have noticed that the lbscr-6wheel-mail*.png is much bigger than other lbscr-6wheel-*.png, so it need re-rendering.
Also lbscr-6wheel-third-umber*.png is a little bit bigger, although the difference is not so remarkable as the mail car.
And the lbscr-6wheel-brake-front_SE/S.png and lbscr-6wheel-brake-rear_N/NW.png are darker than other 6-wheelers in orange livery
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on March 16, 2019, 10:34:21 PM
I think my previous post should be split as separate bug report. Also the lbscr-6-wheel 3rd class in 1904's livery needs rescaling
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on March 17, 2019, 01:49:41 AM
I think that these are more in the category of vehicles that could do with being re-rendered in due course than bug reports - unless there are vehicles in the same set that have not been re-rendered?
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on March 17, 2019, 10:12:59 AM
Most of the lbscr 6-wheelers are already re-rendered. These are a few I have noticed to look different from the rest.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on March 17, 2019, 11:19:30 AM
I wonder whether Spenk009 might like to re-render the remaining LBSCR 6 wheelers, in that case?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on April 22, 2019, 11:30:14 AM
I have now remedied the issue with the LBSCR 6 wheel mail carriage: the problem was in fact caused by using an old .blend file that had not been rescaled. The mail carriages were actually intended to be created from the full brake carriages with the layer containing the guard's lookout duckets hidden, which is what I have now done.

Incidentally, I also note that some of the .blend files for the re-rendered carriages had been saved in their rotated state (i.e., after the automatic creation of the images). It is important that this never be done, as it is very difficult to revert to the original rotation again for the next export: always save then export, then reload the save from before the rendering/export before doing anything else. I had to recover old files from the Github archive to fix this issue.




I have also made some progress in the last few days adding class definitions to some more carriages, including some late 1870s Midland Railway bogie carriages, and the complete set of outstanding Metropolitan and District Railway carriages (including EMUs).
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on April 27, 2019, 11:41:09 AM
My apologies for embarking on some of the maintenance of the pakset. I believe I rebuilt the erroneous mail carriages and pushed them to github. Accordingly, the pak repository would have also been updated on my end.

There vehicles in the works, mainly a few additions to classes. I'm inclined to use the same images for composite and other-class versions of carriages, but this is bad practice and doesn't concur with the level of detail invested in other vehicles. I don't have much documentation on how vehicles looked and decent images of rarer carriages are scarce.

I've found employment in a different company requiring very long hours (including weekends) and 3 hours of daily commute. I'm finding it very difficult to make time for social responsibilities and hobbies. I apologize again for lack of presence and missing the prior two posts.

I will check what progress I've made and report it.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on April 27, 2019, 12:34:32 PM
Don't apologise for working on the pakset - work is much appreciated. Just be careful not to save the .blend files in a rotated, post-export state, as this is quite time-consuming to undo.

Also don't apologise for not having much time: that is a problem common to many of us who are not retired: nobody is obliged to do this work, although it is always appreciated when people do.

If you are having problems finding images of what composite or first or second class carriages of any particular type look like, please let me know on this thread: I have quite a number of books about railway carriages, and I am also a member of a model railway club with an extensive library, so I can fairly easily find pictures or drawings of carriages of a very wide range of types.

It is better to ask than to use the same image for composites as third class vehicles, as having the two look identical might be confusing for players. However, some second class carriages (and some second/third compsoites) would have looked indistinguishable from their third class counterparts (railway practice was generally to give second class passengers the same amount of space as third class passengers, but give them padded seats (or later, slightly better padded seats)), such that the differences on the outside of the carriage would be confined to different lettering/numbering on the doors which would be invisible at our scale.

Thank you again for your work on the pakset: it is much appreciated.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on May 02, 2019, 08:31:20 PM
I dare to continue here - a question about lbsrc-craven carriages. They have brake-front, brake-rear and full-brake (mail), just like many other carriage sets of late 1800's, but lack the constraints forcing particular order. Only constraints are next=any for non-braked carriages. Is it just forgotten, or intentional?
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on May 04, 2019, 11:45:40 AM
I have looked into the changes and updates made, tried to push a commit but failed somehow. I don't think I can recover the changes made. I have created four-wheel suburban carriages, which is in the pull request currently outstanding.

Quote from: Vladki on May 02, 2019, 08:31:20 PMI dare to continue here - a question about lbsrc-craven carriages. They have brake-front, brake-rear and full-brake (mail), just like many other carriage sets of late 1800's, but lack the constraints forcing particular order. Only constraints are next=any for non-braked carriages. Is it just forgotten, or intentional?

The lbscr-cravens follow the same pattern as the normal Four-Wheel carriages, which can be arranged without a brake van adjacent to the locomotive (a single brake van at the rear). We spoke about it around reply 45, so I think that will be kept.

Quote from: Vladki on March 17, 2019, 10:12:59 AMMost of the lbscr 6-wheelers are already re-rendered. These are a few I have noticed to look different from the rest.
Could you list the offending carriages? I will try to fix these then.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on May 04, 2019, 04:59:03 PM
See my post from March 4th
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on May 05, 2019, 12:02:51 PM
I've tried to find the issues in the post from March 4th, but have only been able to find and fix the darker tone in carriage. I've moved example carriages onto a siding here (https://i.imgur.com/03zKZLq.png). Can you point out my error in looking into the issue?

The blend files feature a few vehicles that are designed wider than others. I believe I had fixed these, but the changes have not propagated to the upstream repository due to the bad saving practice. James, can you point out the commit that added the erroneous files?
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on May 09, 2019, 12:19:17 AM
Thank you for your work on this.

First of all, to answer Vladki's question - the coupling constraint scheme for many unfitted vehicles (i.e., those without continuous brakes) is much simpler than for the fitted vehicles: they unfitted vehicles have the same types of coupling constraints as wagons (that is, all but brake carriages have constraint[next]=any).

Spenk - I am a little confused as to what exactly is on your Github repository. Looking at the merge data from your branch, I see a large number of changes going back to February (e.g. "Added LBSCR 4 wheel third class coach") - is this something that you have already done? There are changes for which I do not understand the rationale - for example, you have modified the LBSCR long distance unfitted carriages to be able to be upgradable to the suburban fitted carriages; may I ask what the reasoning was for this?

Also, I believe that I have now fixed the issues with the LBSCR 6 wheel carriages. May I ask what you did by way of re-rendering in the commit "brighter image to match the rest of the set"? Vladki - may I ask which LBSCR 6 wheel carriges, if any, you believe are still incorrectly rendered with the latest nightly build of the set?

You write that you failed somehow to push a commit - can you elaborate? I am not entirely sure that I understand what you are describing here. May I ask to what changes that you are referring when you write of being unable to recover the changes made?

I note that the commit still contains additional suburban versions of the Craven carriages; I believe that I had explained in an earlier post that these carriages did not exist, as there was no distinction between suburban and long distance carriages in the 1860s. You will need to revert these changes before I can merge from your branch.  I believe that there are some other issues discussed above that will likewise need to be dealt with before I can merge this.

I also note that there is a commit with a commit message, "Update on 1870s LBSCR carriages, with two additional carriages made available", but the only thing in that commit is a removal of all the constraints on one of the LBSCR 6 wheel carriages. I presume that this was not intended? May I ask what was intended here?
Title: Re: 1870-1890's carriages lack class definitions
Post by: ACarlotti on May 09, 2019, 03:50:39 AM
I tried investigating what was in the relevant commits, and found it initially quite confusing. I've listed some the things that contributed to my confusion, along with some suggestions for making things clearer in the future.
I expect some of these points could be relevant more generally; they're probably also picking up on the sort of things that people might not notice until they've tried investigating the full history of lots of bits of code (which I have been doing a lot of both while working out how to merge stuff from Standard accurately, and more generally while trying to learn how bits of code were intended to work while debugging or investigating inconsistenties in the code).

Spenk009:
1. In the merge commit 35a2de1 you made some additional changes that weren't present in either parent. This effectively makes them much harder to identify - using 'git log -p' (similar to my normal way of viewing history) doesn't show changes made during merge commits. I had to discover and add the '--cc' flag to view the changes made in that merge. (At least I can now know about that flag for future use). This is an extract of the output:

  LBSCR-4Wheel-Second-fitted
- LBSCR four wheel five compartment carriage (continuously braked) (second class)
+ LBSCR four wheel five compartment carriage (second class) (continuously braked)
+LBSCR-4Wheel-third-fitted
- LBSCR four wheel five compartment carriage (continuously braked) (third class)
++LBSCR four wheel five compartment carriage (third class) (continuously braked)
+LBSCR-4Wheel-composite-fitted
- LBSCR four wheel five compartment carriage (continuously braked) (first/second class)
++LBSCR four wheel five compartment carriage (first/second class) (continuously braked)
  LBSCR-4Wheel-Brake-rear-fitted
- LBSCR four wheel three compartment brake carriage (rear) (continuously braked) (third class)
+ LBSCR four wheel three compartment brake carriage (third class) (rear) (continuously braked)
  LBSCR-4Wheel-Brake-front-fitted
- LBSCR four wheel three compartment brake carriage (continuously braked) (third class)
- LBSCR-4Wheel-full-brake-fitted
- LBSCR four wheel mail/brake carriage (continuously braked) (rear)
+ LBSCR four wheel three compartment brake carriage (third class) (continuously braked)
-LBSCR-4Wheel-full-brake-fitted
-LBSCR four wheel mail/brake carriage (continuously braked) (rear)
++LSCR-4Wheel-full-brake-fitted
++LBSCR four wheel mail/brake carriage (third class) (continuously braked) (rear)
  LBSCR-4Wheel-full-brake-fitted-front
--LBSCR four wheel mail/brake carriage (continuously braked)
++LBSCR four wheel mail/brake carriage (third class) (continuously braked)
  LBSCR-6Wheel-5-com

The lines beginning '++' are ones which were present in neither parent. Obviously it is sometimes necessary to introduce new lines (or new versions of existing lines) to deal with merge conflicts, but in this case I think it would have been better to make those changes in a separate new commit, as this makes it easier to trace their history.

2. The history graph is quite complicated in a couple of places ('--oneline' used here for concision):

$ git log --graph --boundary --oneline  upstream/master..Spenk009/master
* c76627b25 (Spenk009/master) LBSCR 6 wheel carriages
*   60b0015d1 Merge remote-tracking branch 'origin/master'
|\ 
| *   9849fbc64 Merge branch 'master' into master
| |\ 
* | | 50b471bbf UPDATE: LBSCR 4 Wheel Carriages
* | | 578cfe9bc Addded Suburban versions of 4 wheel LBSCR Craven Carriages.
|/ / 
* | 6ef518663 Update on 1870s LBSCR carriages, with two additional carriages made available
* | 6f6ef36f1 Fixing full-brake entries and addition of rear on a brake vehicle
* |   fce4ab076 Merge branch 'master' of github.com:Spenk009/simutrans-pak128.britain
|\ \ 
| * \   35a2de11a Merge branch 'master' into master
| |\ \ 
* | | | c920d6371 Modified comfort value to match other vehicles in the set.
|/ / / 
* | | e07950f34 Added LBSCR 4 wheel composite carriage
* | | 71692e69c Added LBSCR 4 wheel third class coach
o | | 141431bdb Catching up to James, 04.02.19
/ / 
o | 50c3eae6c FIX: Incorrect constraint for the LNWR 42ft brake carriages CHANGE: Add street/stop name

o ece17613c FIX: The previously modified TPOs could not be coupled to the Mark 1 BG

I don't know why this is happening, but it seems that when you are merging from jamespetts' master, you are subsequently making some commits onto the pre-merge version, and then having to create a second merge to include the first one. It would be much cleaner if you only had to merge once. The simplest way to do this in future would be to remember to pull from Github before making further changes (or remembering to commit before carrying out the merge if that was the issue); however, if you forget to do this you might also have the option of rebasing you latest work onto the merge from Github so that only one merge commit is present.

3. I notice that commit 141431b (commit message "Catching up to James, 04.02.19") was in fact a change to the .gitignore file which doesn't seem to fit the commit message, and was subsequently included and then reverted my James. I presume there was just some mistake there.

4. Commits 71692e6 and e07950f made corrections to en.tab that had already been present upstream for a week or two. I guess it's easy to have that happen accidentally - the only thing I might have done differently is edited my commit history to remove my own changes once I realised they were upstream. But that's probably a matter of taste.

James:
I'm confused about this recent merge commit:

$ git log --boundary --graph upstream/master~~..upstream/master~
*   commit 80c7c5f191b529b64ad36c2a46f9d42a53b187bf
|\  Merge: ece17613c d25a25818
| | Author: James E. Petts <jamespetts@yahoo.com>
| | Date:   Thu May 9 01:03:23 2019 +0100
| |
| |     Merge branch 'master' of https://github.com/MatthewForrester/simutrans-pak128.britain.git
| |   
| *   commit d25a25818d10f0af1f4c00d479f0ccfc48b349f9
| |\  Merge: e60dd93c8 6e7e1c2ad
| | | Author: MatthewForrester <38819845+MatthewForrester@users.noreply.github.com>
| | | Date:   Thu Jun 14 05:12:58 2018 +0100
| | |
| | |     Merge pull request #1 from MatthewForrester/hold-fix
| | |     
| | |     Add the same way constraint as this ship's hold
| | |
| | o commit 6e7e1c2ade8fa391be7e250dfac20fd241224e24
| |/  Author: MatthewForrester <38819845+MatthewForrester@users.noreply.github.com>
| |   Date:   Thu Jun 14 05:08:54 2018 +0100
| |   
| |       Add the same way constraint as this ship's hold
| |
| o commit e60dd93c8e3ec2f22b8fc7277911a7b16609b871
|   Author: James E. Petts <jamespetts@yahoo.com>
|   Date:   Mon Jun 11 08:47:44 2018 +0100
|   
|       FIX: Incorrect capacities for the fitted 1870s composite carriage (Rollmaterial)
|
o commit ece17613cc4a58b08bfaee26120ee789e291b028
  Author: Rollmaterial <r.sivonen@tu-bs.de>
  Date:   Mon Apr 29 22:38:16 2019 +0200
 
      FIX: The previously modified TPOs could not be coupled to the Mark 1 BG

It made no change to the source code, and adds only one other new commit - this was a merge between commit 6e7e1c2 and its parent (both already included in your master) which was almost a year old.



I may be assuming too much Git competency here, but my suggestion for how tidy up this branch history to make it clearer (and to avoid having to include seperate commits including and removing suburban versions of the Craven carriages) would be to create a new branch on top of James' master, and cherry pick the commits that you still want to include onto the new branch (I think there're at most five).
As a general point, I'm very much in favour of editing local commit history when doing so can make it less confusing - as long as those commits haven't been shared and (possibly) built upon by someone else, then this shouldn't cause any of the problems caused by more invasive history-editing (such as duplicate commits arising, or forcing other people to rebase onto your amended commits). There're many examples of this sort of thing in commits I've uploaded so far - typical examples are when I add some stuff to an existing commit, or when I realise that I made a mistake porting changes from Standard five commits ago and go back to correct the original commit (instead of making a new one and leaving several broken versions in between).

Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on May 09, 2019, 09:55:07 AM
My apologies for the errant commit: I spotted a commit when looking into this from some time ago that looked as though it ought to have been merged, but I must have corrected this manually in the past.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Spenk009 on May 10, 2019, 03:35:13 PM
Quote from: jamespetts on May 09, 2019, 12:19:17 AM
I also note that there is a commit with a commit message, "Update on 1870s LBSCR carriages, with two additional carriages made available", but the only thing in that commit is a removal of all the constraints on one of the LBSCR 6 wheel carriages. I presume that this was not intended? May I ask what was intended here?
I thought I had made a large set of changes, but became very confused within the command line interface in git, so I believe I ended up resetting everything before committing. The changes I made were brought together over several weekends, so I'm uncertain as to what I was really doing.

Quote from: jamespetts on May 09, 2019, 12:19:17 AMas there was no distinction between suburban and long distance carriages in the 1860s
I misinterpreted the post by basilicafields (https://basilicafields.wordpress.com/2010/12/19/lbscr-carriages/). I thought that Stroudley's suburban carriages would be different to the standard carriages in use.



Quote from: ACarlotti on May 09, 2019, 03:50:39 AMI may be assuming too much Git competency here
I'm not very good with these things. For a while I believed I had understood github's system, but this is proving very wrong indeed. I'll try to learn correct handling, making contributions when I notice issues I could fix.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on May 10, 2019, 10:56:23 PM
Thank you very much for your work on this - don't get discouraged at some initial problems. For reference, the normal command line workflow for Git (assuming that you are not changing branches) is:

(1) make changes to the files;
(2) [only if any new files have been added] "git add --all" to add new files to the staged commit;
(3) "git commit -a" to commit all your changes to your local repository;
(4) type out the Git commit message in the text editor and then CTRL+W to save and CTRL+X to exit; and
(5) "git push origin master" (or substitute "master" for the name of the branch if you are working on a branch other than the master branch) to push your changes to the remote repository.

To merge in changes to your local repository from the remote repository (you do not need to do this with changes pushed from your local repository to the remote repository), just type "git pull origin master" (or substitute "master" for the name of the branch if you want to merge in changes from a branch other than the master branch).

You may need to tidy up your repository a little with some slightly more sophisticated techniques that even I would need to look up now that there are some issues with it, but, once you have done that, the workflow is very straightforward.

As to the suburban carriages - Stroudley's suburban carriages were indeed different to the others; but these came in in the 1870s, not the 1860s. Craven was Stroudley's predecessor, and his carriages (from the 1850s and 1860s) had no distinction between suburban and long-distance. Does this clarify matters?

I hope that this assists.
Title: Re: 1870-1890's carriages lack class definitions
Post by: Vladki on May 14, 2019, 08:55:42 PM
LBSCR 6-wheelers seem to be fixed by the 22 April 2019,commit
There is similar rendering issue with LSWR 6-wheel mail as reported here https://forum.simutrans.com/index.php/topic,18822.0.html
Title: Passenger classes around the 1900s
Post by: Mariculous on September 03, 2019, 09:30:14 PM
Is it intended that most passenger carriages from ~1880 on are third/low class only?
It is pretty hard to build a train around 1880-1900 (don't know about later) that provides carriages for higher classes.
Title: Re: Passenger classes around the 1900s
Post by: Spenk009 on September 04, 2019, 09:27:00 AM
I had originally taken this task upon me. I loaded up a game for 1880:
All listed carriages are available all throughout the 1880s, except for the LSWR carriages (discontinued in 1886). The only carriages not rendered in the transparent workflow are the GNR 6 wheel carriages.

There is a balance between accuracy and playability, which I don't wish to move. Can I ask which carriages should be added, whether they require adaptation visually and what differences they will have to the ones available prior?
Title: Re: Passenger classes around the 1900s
Post by: Mariculous on September 04, 2019, 11:53:49 AM
Well maybe my post was a little bit unclear.
In 1880, everything is still fine as carriages from ~1975 are still available for a few years.
However, after ~1880, most of the newly introduced carriages don't serve second/first class or they don't serve mail, so in ~1890, it will be very hard to create a train with mail and second/first class that is still available for at least the next 10 years.

In 1890 you have the following options:
You can take the Metropolitan eight wheel coaches, which don't have mail carriages.
GNR six wheel cars only have third class.
Four wheel coaches available until 1911 don't have mail, mail is only available until 1894 for these.
LBSCR six wheeler have first class only available until 1892, other classes will be available until 1901.
New LBSCR (1889) cars only have third class.
GWR corridor cars only have third class.
Others won't be available after 1897 at all.

A few years later, in 1895, things will be a bit better:
LNWR corridor cars have all cars that one could need but these are only available for 4 years (is this a typo?)
New L&TSR 6 wheel are available, which are available for 11 years, which is fine, however in 1895 only 7 years are left, which is not much but enough in case one needs more trains than expected on that line.


I don't know what exactly should be added as I don't know pretty much about british railway history.
However, without that in mind, I guess any of the following would solve the problem:
Create some Metropolitan or Metropolitan compatible mail car.
Create some GNR second/first class, may this be a combined car or two different cars.
Increase availability of some four wheel or compatible mail coach to 1911.
Increase availability of LBSCR six wheeler first class car to 1901.
Create some GWR corridor car compatible second/first class car, may this be a combined car or two different cars.

Increase LNWR corridor cars availability to be available for at least 10 years.
Title: Re: Passenger classes around the 1900s
Post by: Spenk009 on September 05, 2019, 08:06:56 AM
Quote from: Freahk on September 04, 2019, 11:53:49 AMHowever, after ~1880, most of the newly introduced carriages don't serve second/first class or they don't serve mail, so in ~1890, it will be very hard to create a train with mail and second/first class that is still available for at least the next 10 years.
Thank you, this is a well placed description of the core issue. I think most (if not all) types of carriages have been added to the game, so we might best make a case for extending availability or improving selection with fictional vehicles.
Quote from: Freahk on September 04, 2019, 11:53:49 AM
I don't know what exactly should be added as I don't know pretty much about british railway history.
I am in the same boat. I also lack the literature to support claims properly, but I've now enrolled in a university course that I hope gives me time for other things and access to better resources.

Re-assigning old coaches for lower classes is the real life practice of the era, but this is difficult to mix if classes (inevitably) overlap. Immediate remedies are to play allowing purchases of obsolete vehicles or amending the pakset for oneself. If someone with good knowledge of the era's carriage selection could weigh in an opinion, we could see about improving the era.
Title: Re: Passenger classes around the 1900s
Post by: Elemental on September 05, 2019, 04:00:56 PM
You can assign individual coaches different classes even if they belong to the same original class - you just can't do it while the convoy is assembled. As an example, using coaches with lavatory for higher class and compatible coaches without lavatory as third class often works well and offers comparable comfort differences to real higher class coaches.

Do it the following way: In the depot window, assemble a convoy including at least all the vehicles you want to assign a different class. Then reassign the classes. Now disassemble the convoy and rebuild it with only the vehicles you want to have the original class. Then reassign classes for them back to standard. Then assemble the convoy the way you actually want it to look. The individual vehicles will remember the classes they were last assigned. While this works, it is indeed needlessly complicated...
Title: Re: Passenger classes around the 1900s
Post by: Mariculous on September 05, 2019, 04:43:21 PM
Sounds extremely complicated but could work :/
Title: Re: Passenger classes around the 1900s
Post by: Ves on September 06, 2019, 06:36:50 PM
Bear in mind that there eventually will come a new window where you can alter, amongst other, these things. It will be possible to select all vehicles of a certain type over your entire network and alter the classes for all those vehicles.
Title: Re: Passenger classes around the 1900s
Post by: Mariculous on September 06, 2019, 09:33:21 PM
Sounds like an awesome usability patch :)
I don't know how exactly this should work but It would be great if one could filter selected vehicles by its lines name.

When I look into the vehicle list, class reassignment would fix these gaps in passenger classes and a better UX would make this usable.
For many coaches in these years, there is not a huge comfort difference in between third class and first class anyways.

Side note: Simutrans extended has many really awesome features but many of these suffer from bad usability.
Title: Re: Passenger classes around the 1900s
Post by: Matthew on September 07, 2019, 08:46:55 PM
Quote from: Elemental on September 05, 2019, 04:00:56 PM
You can assign individual coaches different classes even if they belong to the same original class - you just can't do it while the convoy is assembled. As an example, using coaches with lavatory for higher class and compatible coaches without lavatory as third class often works well and offers comparable comfort differences to real higher class coaches.

Do it the following way: In the depot window, assemble a convoy including at least all the vehicles you want to assign a different class. Then reassign the classes. Now disassemble the convoy and rebuild it with only the vehicles you want to have the original class. Then reassign classes for them back to standard. Then assemble the convoy the way you actually want it to look. The individual vehicles will remember the classes they were last assigned. While this works, it is indeed needlessly complicated...

Quote from: Freahk on September 05, 2019, 04:43:21 PM
Sounds extremely complicated but could work :/

I have been using the approach suggested by Elemental for a while. Like many things with Simutrans, it is mind-bendingly complicated the first time, but after a while it becomes second nature....  ;D
Title: Re: Passenger classes around the 1900s
Post by: jamespetts on September 13, 2019, 11:12:10 AM
The reason for this is that I have not yet completed the very large scale work of adding passenger classes to all the various railway carriages in the late 19th and first half of the 20th century. The early 19th century, second half of the 20th century and all of the 21st century (so far) are complete.

However, there are a number of railway companies whose carriages now do have a full set of classes in all years, as set out above, so there are always some railway carriages with multiple classes in any given era.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on December 30, 2019, 11:42:59 PM
I have merged these two topics together as they were essentially the same topic. I should note that I am at present working on enhancing the GWR offering, which should bring multiple classes of GWR carriages to the pakset in all eras in which the GWR operated standard gauge trains.
Title: Re: 1870-1890's carriages lack class definitions
Post by: jamespetts on January 12, 2020, 11:36:41 PM
I have now completed the GWR project: see here (https://forum.simutrans.com/index.php/topic,19449.0.html) for details. We thus now have GWR carriages with full sets of classes (and liveries) all the way from the early days (circa 1850s) to the 1940s when the GWR ceased to exist as an independent company.