Jump to content


Photo

Motion to deprecate the happy lucky whatever pack


15 replies to this topic

#1 Wisp

Wisp
  • Modders
  • 968 posts
  • Gender:Male

Posted 06 February 2012 - 03:26 PM

i) It serves no discernible purpose,
ii) it is pretty much only ever installed because it is there,
iii) it's presence is inconvenient and
iv) all the other reasons it should be axed.

#2 Miloch

Miloch

    Barbarian

  • Modders
  • 4715 posts
  • Gender:Male
  • Location:At Large

Posted 06 February 2012 - 05:31 PM

I used to use it but pretty much manually do the things that it does when I mod. Though it can still serve a purpose... I guess anyone who wants it can always undeprecate it.

What are "all the other reasons"?
Mod Contributions
Aurora (new release!) * BG1 NPC * Gnomes * Haiass * Level 1 NPCs * Lost Items * P5Tweaks
PnP Free Action * Thrown Hammers * Unique Containers * In Progress: Adjusted Portraits * DSotSC (Tutu)
================================================================
Player & Modder Resources
BAM Batcher * Creature Lister * PSPad Highlighters * Tutu/BGT Area Map & List * Tutu Mod List
================================================================
"Wherefore do ye toil; is it not that ye may live and be happy? And if ye toil only that ye may toil more,
when shall happiness find you?" -H.P. Lovecraft, The Quest of Iranon

#3 Daulmakan

Daulmakan

    Comfortably numb

  • Members
  • 530 posts
  • Location:The Houses of the Holy

Posted 06 February 2012 - 05:31 PM

Disabling DLTCEP's startup error messages makes it worthwhile for me. Is there an actual technical reason for avoiding it?
I ain't evil, I'm just good looking.


#4 Salk

Salk
  • Members
  • 2347 posts
  • Gender:Male
  • Location:Sweden

Posted 06 February 2012 - 08:39 PM

I think it should be deprecated but I wonder if there's code there to be salvaged and imported into the Core fixes?

#5 berelinde

berelinde

    The Typo Queen

  • Gibberlings
  • 8594 posts
  • Location:New Jersey, USA

Posted 06 February 2012 - 11:18 PM

Disabling DLTCEP's startup error messages makes it worthwhile for me. Is there an actual technical reason for avoiding it?

If you're a player, not really, but there's no reason to include it, either. If you're a modder, you should never, ever install it. It does things that your mod *should* do, patching in CD_STATE_NOTVALID if you haven't done it already, patching ids, assigning area scripts to areas where the vanilla game did not, patching in a blank song reference to allow mods to use PlaySong(0), etc. Basically, it's there to prevent players from suffering at the hands of lazy modders.
Must. Write. Faster.

cmorgan: "None of us get old around here, just more proficient at doing more stuff with less braincells!"

berelinde's mods
TolkienAcrossTheWater website
TolkienAcrossTheWater Forum

#6 Ardanis

Ardanis

    A very GAR character

  • Members
  • 2225 posts
  • Gender:Male
  • Location:Saint-Petersburg, Russia

Posted 07 February 2012 - 08:27 AM

Ninde and Xulaye mods over SHS either didn't install at all or installed with warning without this component. It's been three years ago, though, so I'm don't know if Kae had updated them since then.
"Uguu~ Boku Ayu."

Before you start breaking wall tiles with your bare fists, ask yourself first - do you truly need it?

#7 ericp07

ericp07

    Journeyman Modder

  • Members
  • 1262 posts
  • Gender:Male
  • Location:Sacramento, CA, USA

Posted 07 February 2012 - 08:54 AM

I've never had such issues when installing Ninde or Xulaye, although both mods required me to manually edit a bit to get them to install with no errors.

As for the SHFLM component, I'm with those who'd like to see the component removed.

Edited by ericp07, 07 February 2012 - 08:54 AM.

Working and playing under Mac OS X 10.4.11 Tiger on a hot-rodded PPC G4 Quicksilver Mac.
~Buion na 'ell! I serve with joy! Your eyes and ears I shall be. Let us hunt together!~
- Erysseril Gwaethorien: a joinable, romanceable NPC mod for BGII - SoA/ToB, in active development.

A female elf warrior and a Bhaalspawn cross paths during their quests, joining forces to share adventure and companionship. Will they find more?

#8 devSin

devSin
  • Fixpackers
  • 3017 posts
  • Gender:Male

Posted 07 February 2012 - 02:06 PM

Yes, it needs to be removed.

Somebody will have to review the component to see if there's anything salvageable, and there may be things in there that deserve some sort of community release (maybe a tutorial or something), but it doesn't belong in the fixpack.

There was a reason for it in the beginning, but the lack of maintenance and the reality of mod development simply make it more harmful than beneficial.

#9 plainab

plainab

    Sasha al'Therin

  • Members
  • 1717 posts
  • Gender:Male
  • Location:Lost on the Sword Coast

Posted 07 February 2012 - 02:22 PM

It does things that your mod *should* do, patching in CD_STATE_NOTVALID if you haven't done it already, patching ids, assigning area scripts to areas where the vanilla game did not, patching in a blank song reference to allow mods to use PlaySong(0), etc.

that right there is the reason it was developed. If every mod under the sun had to put all this stuff in prior to the actual mod content... well it stands to reason to have a singular install for all mods rather than some modders doing it and other modders not and mods not playing nice with each other cause of it.

If nothing else, a tutorial explaining what each thing does and why and where you should put it in your mod if your mod touches on any of the stuff.

Where it becomes a problem is those mods made with the fixpack and the happy lucky component installed and players installing said mod without using the fixpack. BUT it becomes the modder's responsibility to inform their users that the mod requires the fixpack and the happy lucky component.
My working mods:
an AI Party Script for BG2 game engine DOWNLOAD LINK ONLY!
Interactive Tweaks for BG series with some IWD support. DOWNLOAD LINK ONLY!
Rest For 8 Hours an IWD mod
-------------------------------------------
My contributions: BG1Fixpack, BG1Tweaks
On Hold: Solestia an NPC for SOA
-------------------------------------------
My website: http://sasha-altheri...s.com/index.htm

#10 Ardanis

Ardanis

    A very GAR character

  • Members
  • 2225 posts
  • Gender:Male
  • Location:Saint-Petersburg, Russia

Posted 07 February 2012 - 03:47 PM

You might be surprised to hear this, but all too often players don't really follow install orders. They're lazy to read and to study all the technical aspects of BG modding. Lately it was somewhat mitigated by BWP, but you may never be certain.

As usual - if you want something done, do it yourself.
"Uguu~ Boku Ayu."

Before you start breaking wall tiles with your bare fists, ask yourself first - do you truly need it?

#11 Daulmakan

Daulmakan

    Comfortably numb

  • Members
  • 530 posts
  • Location:The Houses of the Holy

Posted 07 February 2012 - 04:34 PM


Disabling DLTCEP's startup error messages makes it worthwhile for me. Is there an actual technical reason for avoiding it?

If you're a player, not really, but there's no reason to include it, either. If you're a modder, you should never, ever install it. It does things that your mod *should* do, patching in CD_STATE_NOTVALID if you haven't done it already, patching ids, assigning area scripts to areas where the vanilla game did not, patching in a blank song reference to allow mods to use PlaySong(0), etc. Basically, it's there to prevent players from suffering at the hands of lazy modders.

Or aspiring, modders-to-be. It seems it's been more useful than I thought.

Since the component is optional, I don't really see the harm. If it's the lack of updates, there are several other mods at G3 that could do with an upgrade as well.
I ain't evil, I'm just good looking.


#12 Wisp

Wisp
  • Modders
  • 968 posts
  • Gender:Male

Posted 08 February 2012 - 09:52 AM

What it does:
  • Append IDS symbols to animate.ids, anisnd.ids, gtimes.ids, shout.ids and spell.ids
  • Add a header (IDS V1.0) to damages.ids, happy.ids, slots.ids, soundoff.ids and state.ids
  • Add the file moraleai.ids
  • Append the CD_STATE_NOTVALID symbol to state.ids
  • Add a blank mus file to songlist.2da
  • Write references to non-existent area scripts to areas wihout a reference
  • Fix nulled map-note offsets in a bunch of areas (with 0 map notes)
  • Remove undetectable, undisarmable "traps" with no effect (no script) from containers in ar0311.are. ar0327.are, ar0417.are and ar0510.are
  • Conformise spell level and spell type with the spell res for many memorised spells
  • Write duration = 0 for many spell effects with non-temporary timing modes (to appease DLTCEP)
  • Remove ability headers of type 0 from many items (from back when this was thought to stop the drain-charge opcode from destroying the whole item)

1, 4 and 5 should be done by the mod that needs it, as per standard practice today.
7 may be valid, but I assume it was banished to the modder pack because it was, in fact, not.
The rest are either bunk or smell strongly of it. Maybe 3 and 10 can be spun off as some make-DLTCEP-happy library.

Edited by Wisp, 08 February 2012 - 10:01 AM.


#13 aVENGER_(RR)

aVENGER_(RR)

    Sneaksie!

  • Fixpackers
  • 1008 posts
  • Gender:Male

Posted 11 February 2012 - 03:13 AM

1, 4 and 5 should be done by the mod that needs it, as per standard practice today.


Agreed.

Maybe 3 and 10 can be spun off as some make-DLTCEP-happy library.


That would probably be for the best.
Retired modder

Rogue Rebalancing - Author
aTweaks - Author
Wizard Slayer Rebalancing - Co-Author
BG2 Fixpack - Contributor

#14 CamDawg

CamDawg

    Executive Delivery Boy

  • Gibberling Poobah
  • 8709 posts
  • Gender:Not Telling

Posted 29 July 2012 - 09:42 AM

The SHFLM was originally planned as a new home for detectable spells until we discovered what a mess that would actually be. As a result it ended up being a largely ignored, largely useless bag of tricks.

Deprecate the whole thing, but leave the code intact in case some modder wants to grab some bits or, alternatively, move the whole damn thing to the debugging suite.
Don't you worry about Planet Express, let me worry about blank.

#15 cmorgan

cmorgan

    journeyman investigator

  • Gibberlings
  • 6901 posts
  • Gender:Male
  • Location:Glencoe, IL, USA

Posted 29 July 2012 - 10:01 AM

1, 4 and 5 should be done by the mod that needs it, as per standard practice today.


Agreed.

Deprecate the whole thing, but leave the code intact in case some modder wants to grab some bits or, alternatively, move the whole damn thing to the debugging suite.

I agree - move into debugging suite as feedback for modders, and reinforce the docs and to indicate findings without fixing, then allow fixing?
I love deadlines. I love the whooshing noise they make as they go by. - Douglas Adams



Reply to this topic



  


0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users