Jump to content

Motion to deprecate the happy lucky whatever pack


Wisp

Recommended Posts

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.
Link to comment

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.

Link to comment

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.

Link to comment
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.

Link to comment

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.

Link to comment

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.

Link to comment

What it does:

  1. Append IDS symbols to animate.ids, anisnd.ids, gtimes.ids, shout.ids and spell.ids
  2. Add a header (IDS V1.0) to damages.ids, happy.ids, slots.ids, soundoff.ids and state.ids
  3. Add the file moraleai.ids
  4. Append the CD_STATE_NOTVALID symbol to state.ids
  5. Add a blank mus file to songlist.2da
  6. Write references to non-existent area scripts to areas wihout a reference
  7. Fix nulled map-note offsets in a bunch of areas (with 0 map notes)
  8. Remove undetectable, undisarmable "traps" with no effect (no script) from containers in ar0311.are. ar0327.are, ar0417.are and ar0510.are
  9. Conformise spell level and spell type with the spell res for many memorised spells
  10. Write duration = 0 for many spell effects with non-temporary timing modes (to appease DLTCEP)
  11. 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.

Link to comment

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.

Link to comment
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?
Link to comment

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...