[Discussion] A suggested change: Don't use the user's location

I encourage the following change to be made for the next
version of Pokemon Essentials.

Replace the code in the self.getLatLong method in PBDayNight (in PokemonUtilities) with:

Replace the code in the getRoughLatLon method (in PokemonUtilities) with:
Currently the getLatLong functions are only used for day/night tinting. This is
only an aesthetic use, so using the user's real location is, strictly speaking,
not necessary.

Poccil, hi! Long time no see. How are you doing?

Essentials has moved on a bit since you handed it over to the community/wiki/me (as you can see in the wiki's Downloads page). One of the first things that was changed was the day/night system - I downgraded it all to that of an older version, before latitudes and longitudes came about. Now it only depends on the time of year (and I'm considering removing even that dependency, because I don't think it needs to be that complicated).

If you have any spare time, I would love to hear any other suggestions and comments you may have about Essentials.

OMG. I never thinks that I will be see poccil again where.

Your latest day/night system are also removed because this considerably lags the game in field for some PCs that gets annoying 0.3 ms pauses. You can see better in this video.

I am a big fan of your kit. Can you play my game? Can I have an autograph?

I see that the day-night system was already taken care of.

On another note, I encourage removing the PokemonBoxManager script section from Pokemon Essentials. It has never served any real purpose in the starter kit.

poccil said:

On another note, I encourage removing the PokemonBoxManager script section from Pokemon Essentials. It has never served any real purpose in the starter kit.

Click to expand...

The PokemonBoxManager is the biggest script in the your last release (in last Maruno version it is surpassed by 6k lines PokeBattle_MoveEffects) that can be used to communicated to several pokémon games and even with Shoddy Battle that uses Java (using the Serializable interface with a very interesting way). Nothing of it can be reused, even with some work?

FL . said:

The PokemonBoxManager is the biggest script in the your last release (in last Maruno version it is surpassed by 6k lines PokeBattle_MoveEffects) that can be used to communicated to several pokémon games and even with Shoddy Battle that uses Java (using the Serializable interface with a very interesting way). Nothing of it can be reused, even with some work?

Click to expand...

If you want to put the work in, go ahead.

I agree with poccil. No one's ever used it, and it's really quite out of date now anyway so it's all but unusable.

As a note of interest, PokemonBoxManager hasn't been spaced out properly. If it was, it would definitely be longer than any other script section.

Maruno said:

If you want to put the work in, go ahead.

I agree with poccil. No one's ever used it, and it's really quite out of date now anyway so it's all but unusable.

As a note of interest, PokemonBoxManager hasn't been spaced out properly. If it was, it would definitely be longer than any other script section.

Click to expand...

You know, it never even occured to me what PokémonBoxManager is even for. I assumed it was related to the PC boxes, and even though I've edited PokémonStorage many times, I never thought about the fact that PokémonBoxManager was never of any importance.

So, that entire script section is unused? That's definitely interesting. I'm going to look at it for the heck of it.

Maruno said:

If you want to put the work in, go ahead.

I agree with poccil. No one's ever used it, and it's really quite out of date now anyway so it's all but unusable.

As a note of interest, PokemonBoxManager hasn't been spaced out properly. If it was, it would definitely be longer than any other script section.

Click to expand...

At moment it is above my skills.

But I don't think that is a good idea to throw away a very big script done by a genius. Something probably can be reused.