rigrig

Most recent activity

• Possible memory leak in Bangle.js gadgetbridge?

  in Bangle.js • 2 days, 3 hours

  

  Actually, we might get away with just making it an attribute, and only calling Volley.newRequestQueue for the very first request (and cleaning it up in dispose). That way there is no need to sprinkle stop over all response/error handler code, and we don't need a full-blown singleton class either.

  Looking at 400-line switch statements makes me itchy though, so I'll try to submit a PR with some refactoring Soon, and include a fix for this.

  edit: PR

• Bangle.js 2 button stuck

  @rigrig started

  • 16 comments
  • 983 views

  @markcra replied
• Possible memory leak in Bangle.js gadgetbridge?

  in Bangle.js • 6 days, 23 hours

  

  I wonder if it's related to Volley.newRequestQueue - maybe that should just be called once?

  I think that's it: looking at this, we should either call stop() on the queue once we have a response, or set up a singleton. Otherwise every request leaves a new "thread pool" lying about.

  If you just need to make a one-time request and don’t want to leave
  the thread pool around, you can create the RequestQueuewherever you
  need it and call stop() on the RequestQueue once your response or
  error has come back, using the Volley.newRequestQueue() method
  described in Sending a Simple Request. But the more common use case is
  to create the RequestQueue as a singleton to keep it running for the
  lifetime of your app, as described in the next section.

  (Calling stop() seems by far the easiest solution.)

• Wind speed not showing in weather app

  in Bangle.js • 1 month

  

  I'm using Weather Notification myself (It's briefly mentioned at the bottom of the app readme), but wouldn't recommend it.
  (In 2020 it still looked maintained, and QuickWeather wasn't an option. Guess I'm just lucky that it kept working for me.)

• Wind speed not showing in weather app

  in Bangle.js • 1 month

  

  Using the Weather Notification app, wind speed shows up ok.
  Using QuickWeather, the wind speed is set to 0.

  So it looks to be an issue with the QuickWeather<->GadgetBridge connection.

  (Thanks for posting this, I didn't know about QuickWeather, and it looks pretty neat)

• Minimum messages code?

  in Bangle.js • 2 months, 2 weeks

  

  Yes, if you put that code in a boot file it should work, except you also want to check that

  1. type === 'text': only show notifications/sms
     
  2. msg.t !== 'remove': only show new/modified messages
     
     

  The messages library README has some example code using E.showMessage().

• Font upscaling / new font discussion

  in Bangle.js • 2 months, 3 weeks

  

  setFontMono(bool)

  That seems like a good idea (maybe also add setFontMono(int width)?)

  g.setFontSize(...)

  That looks good. (and font packs could override Graphics.prototype.setFontSize)

  try and find a font that would display the given text in the area. It might be good for messages...

  On the one hand: that seems overly complicated. I think messages should just use a fixed size and handle overflow: that will happen anyway, and showing short messages in a larger font doesn't seem worth the hassle. (with variable character size, I think we'd need to "try all sizes until one fits", not sure that is a good idea¹)
  On the other hand: if authors really want to use it, we don't want apps to "roll their own" if Bangle could do it for them.

  ¹ even though I wrote that code myself, regretting it now 😉

• Font upscaling / new font discussion

  in Bangle.js • 2 months, 3 weeks

  

  A new font format seems interesting, especially if you could add e.g. emojis.
  I don't get how it would be smaller than indexed bitmaps though?
  Edit: oh, variable width and just a \n after the last black pixel per line would help.

  I guess sometimes you still want to pick an exact font, for e.g. clock layouts? Maybe add Bangle.selectFont(size), and it gives you the best font to use for that size? Custom "font packs" could then override that function.
  Seems a bit clunky though g.setFont(Bangle.selectFont('30px')); :-(

• Font upscaling / new font discussion

  in Bangle.js • 2 months, 3 weeks

  

  For me it's always kind of hard to tell from pictures instead of the real watch, but I think it looks good in the menu bar.
  To me it looks just too small/large in the scroller, but we could maybe resize scroll items?

  say: "I want a bitmap font 20px high" and let Espruino choose?

  This would be nice to have. (And people could customize fonts globally without messing up layouts.)

• Message App UI simplicity

  in Bangle.js • 5 months, 1 week

  

  Sorry for being a bit quiet about issues, somehow I expected to get things done during the holiday, but so far I haven't actually gotten round to coding at all :-(
  Hopefully I'll get some fixes in next week though.