Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add validation on watchset importing (estimate RAM needed) #22

Open
vaspa opened this issue Dec 4, 2015 · 2 comments
Open

Add validation on watchset importing (estimate RAM needed) #22

vaspa opened this issue Dec 4, 2015 · 2 comments

Comments

@vaspa
Copy link
Contributor

vaspa commented Dec 4, 2015

A watchset that does not fit the watch memory should not pass the validation.
An estimation of RAM needed and actual RAM limit should be shown to the user. So the user could adjust his watchset definition to fit the RAM requirements.

@althink
Copy link
Member

althink commented Dec 4, 2015

I'm aware that this is quite important but unfortunately at this stage is
not possible. Memory usage is changing from day to day and the biggest
issue is that malloc is allocating a little bit more memory that is
required and there's no standard how much will it be. So there are many
factors that affect that. And I have to change it one more time to run your
weather watchset because it's too big for latest snapshot.

On Fri, Dec 4, 2015 at 3:41 PM, Pavel Vasilyev [email protected]
wrote:

A watchset that does not fit the watch memory should not pass the
validation.
An estimation of RAM needed and actual RAM limit should be shown to the
user. So the user could adjust his watchset definition to fit the RAM
requirements.


Reply to this email directly or view it on GitHub
#22.

@vaspa
Copy link
Contributor Author

vaspa commented Dec 4, 2015

Of course it will wait for a stable version.
BTW I don't use the weather watchset, it is just a challenge for testing purposes ;)

Is it right that the external properties values are stocked in the external RAM now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants