cloudy

Author Topic: Build 43 .zip update: less memory use and now drag and drop install  (Read 1071 times)

0 Members and 1 Guest are viewing this topic.

Offline Weather Display

  • Posts: 64,538
just discovered something big
while I was making it so that all the cron files are in the application bundle, in the resources folder...and so WD could be install to where ever...
I discoverd that the main start up procedure was being run 2x
which meant 2 lots of the ini files and other resources loaded into memory!
so now build 43 uses only half the amount of memory as before :) :)

also build 43 .zip update is now the application bundle zipped up...as it has the cron files in the new location
so unzip that to where you have WD installed (i.e /Application/wdisplay/), to replace the WeatherD icon
it means though it could be unzipped to anywhere and run from anywhere now

(as long as the library files are in /usr/local/lib/  (thats the next thing to try and localise))

this double loading of the ini files could be the cause of  the access violations too
I will check if this problem also is in the Linux version too
« Last Edit: January 31, 2010, 07:03:38 PM by windy »

Offline jangel1

  • Posts: 116
Re: Build 43 .zip update: now uses half the memory
« Reply #1 on: January 31, 2010, 04:20:12 AM »
Great Brian -- I have it up an running.  Should we delete the cron files in the /Application/wdisplay/ folder -- or does it hurt to leave them there in case we need to go back a version? 

Thanks
JP

Offline Weather Display

  • Posts: 64,538
Re: Build 43 .zip update: now uses half the memory
« Reply #2 on: January 31, 2010, 08:20:46 AM »
you can leave them there, it will not matter  :wink:
note that people using the usbreader or usbreaderirox need to keep WD installed to where it is now...as those programs are still writing the usb.txt file to the default location (I can change that though)

Offline weathertwp

  • Posts: 171
  • Madison Twp, Pa. USA
    • The Farmstead  2dogs2
Re: Build 43 .zip update: now uses half the memory
« Reply #3 on: January 31, 2010, 10:19:40 AM »
Brain,

Do I replace the Applications/wdisplay directory    or   Do I do the show package contents on WeatherD/Contents/MacOs/ and over write the existing file ?

Thanks...
Research-aholic by Nature ..

Offline Hugh Whalen

  • Posts: 91
  • Flowers Cove, Grand Lake, NB, Canada
    • Flowers Cove Weather
Re: Build 43 .zip update: now uses half the memory
« Reply #4 on: January 31, 2010, 11:50:50 AM »
I downloaded the file, unzipped and ran it.

The memory footprint does not appear to have changed. When it starts it shows

91 MB real memory (increased to 104 MB within 5 minutes)
71 MB private memory (increased to 79 MB within 5 minutes) and
11 MB shared memory (increased to 11.7 within 5 minutes)

This is approximately the same as previous builds.

Is this what others are showing?

Hugh
« Last Edit: January 31, 2010, 04:09:07 PM by Hugh Whalen »

Offline Weather Display

  • Posts: 64,538
Re: Build 43 .zip update: now uses half the memory
« Reply #5 on: January 31, 2010, 05:07:54 PM »
you replace the WeatherD file in/Applications/wdisplay/

start up memory use I am sure was lower....than before....oh well...

Offline jmar

  • Posts: 1,405
  • Halifax, Canada
    • HRM Weather
Re: Build 43 .zip update: now uses half the memory
« Reply #6 on: January 31, 2010, 06:46:34 PM »
you can leave them there, it will not matter  :wink:
note that people using the usbreader or usbreaderirox need to keep WD installed to where it is now...as those programs are still writing the usb.txt file to the default location (I can change that though)

CONGRATULATIONS WINDY! YOU DID IT!
This is the biggest improvement to Weather Display Mac since the change to the data folder last year! (and that was great as well!)

Now Weather Display is a completely self contained Mac application!  Since I don't use the usbreader or usbreaderirox, I deleted the wdisplay directory in the Applications folder altogether and Drag/installed WeatherD right into the Applications folder!. It started up and is running like a charm!

What does this mean to users now?

1. To install Weather Display Mac there is only an installer necessary on the first install, just to install the library files. With your permission I will work on that now. So a new user can now drag install the app to Applications folder (or wherever else they want) install the lib files, and set up the program!

2. To install an update to Wd MAc, just drag and replace the old WeatherD icon with the new icon, that is all! :)

This is great for you also Windy, now you never have to have the users working in the application bundle to install a new executable!.

Glad to see part one of your strategy to make WeatherD a true Mac app a reality. The new structure is the same as Apple Apps like Garageband, and others if you look in their "show contents" folders.

Once you cheange the usbreader and usbreaderirox to function out of the Resources folder in the app bundle, all users will enjoy ridding themselves of the Applications/wdisplay folder as well :):)

I know how hard you worked on this. I can't believe how you integrated so much in such a short time! As users come to the site to update they will see the simplicity you introduced by this change in structure that is making WD Mac a great experience to install now!

All the best,
Jmar

PS - You should change the title of this post to "Build 43 and up. Half the memory and now Drag to Install!"

 :greenjumpers:
iMac, 2 x Mac Mini, Mac OS X 10.6.7, Indigo integration, Web - WDL, Saratoga templates, Lightbox

Offline pipross

  • Posts: 12
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #7 on: February 01, 2010, 02:29:38 AM »
Hi all,
Replaced my WeatherD application in the applications/wdisplay folder, and it came up with weather display, unregistered and not setup properly, brought back the 41 weatherD and it works normally. I presume it must be something to do with my setup... advise would be welcome..

Cheers
Ross

Offline Weather Display

  • Posts: 64,538
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #8 on: February 01, 2010, 03:00:31 AM »
make sure you do not have the ini files in the /WeatherD.app/contents/MacOS folder
they should be in
/Library/Application Support/wdisplay/
along with the webfiles, datafiles/logfiles folders

if not then copy them all to there and delete from the MacOS folder
very early versions had the files in the MacOS folder

Offline pipross

  • Posts: 12
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #9 on: February 01, 2010, 03:42:07 AM »
got it going thanks...

Offline artpics

  • Posts: 46
  • Calabasas California
    • Calabasas  CA Weather
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #10 on: February 01, 2010, 03:51:05 AM »
i have 120mb now was around 240mb and i have seen over 300 good job everyone  :D keep up the good work.
Mac 10.6 on an Imac

Offline weathertwp

  • Posts: 171
  • Madison Twp, Pa. USA
    • The Farmstead  2dogs2
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #11 on: February 01, 2010, 06:41:58 AM »
After 12 hours I'm up to 256mb real Memory 295 Virtual Memory ... (Via Activity Monitor)
Research-aholic by Nature ..

Offline Weather Display

  • Posts: 64,538
Re: Build 43 .zip update: less memory use and now drag and drop install
« Reply #12 on: February 01, 2010, 08:51:59 AM »
it was more the start up memory was less (as the ini files were not loaded 2x, which hopefully will mean less access violations too)