pjf / masterwork-dwarf-fortress

Patches and bugfixes applied to Meph's amazing Masterwork Dwarf Fortress
33 stars 22 forks source link

Dwarf Manual Update Notes (v4g): #89

Closed TonyHargrove closed 10 years ago

TonyHargrove commented 10 years ago
- Matched building construction skill from building_masterwork.txt
    - Many are Architect/Mason, but some are not.
- Updated descriptions on Apothecary, Biologist.
- Trouble with Alchemist & Chemist
    - Alchemist Lab is built by Chemist & Soap Maker (POTASH_MAKING),
        not Alchemist (LYE_MAKING)
    - Chemist Lab is built by Alchemist.
    - I'm looking at skills from the image in
masterwork-dwarf-fortress/MasterworkDwarfFortress/repository/changes.html#aa
    - I'll let someone else decide when to update building_masterwork.txt
- Toxicist built by WAX_WORKING (Waxer & Chandler), should this be TRAPPER?
- Morul's and Trade Storehouse do not require Mason, just Architect.
- Corrected some typos.
pjf commented 10 years ago

Merged with thanks! I wish I had the answers to your questions, although it sure seems that Alchemist and Chemist have their build labours swapped. I certainly wouldn't hesitate to correct that if you feel comfortable editing the raws.

TonyHargrove commented 10 years ago

I can do simple edits of the raws, but it's more a matter of ensuring that the Alchemist and Chemist are incorrect versus being swapped for some specific reason. Same for Toxicist.

On Mon, Jan 6, 2014 at 7:36 AM, Paul Fenwick notifications@github.comwrote:

Merged with thanks! I wish I had the answers to your questions, although it sure seems that Alchemist and Chemist have their build labours swapped. I certainly wouldn't hesitate to correct that if you feel comfortable editing the raws.

— Reply to this email directly or view it on GitHubhttps://github.com/pjf/masterwork-dwarf-fortress/pull/89#issuecomment-31657210 .

pjf commented 10 years ago

Aha! Meph would be the definitive person to ask about that, although be aware he's super busy with doing Warlocks right now. Having said that, I can't really imagine the current functionality being as-intended. ;)