Entities are not showing when sort option is present in the config

Upgraded to latest version 3.1.1 and something broke, my card was showing only 2 entries. Reverting to 3.1.0 all works as expected.
Describe the bug

How to reproduce
Upgrade to 3.1.1.
Expected behavior

YAML configuration
type: custom:battery-state-card
title: Battery levels (<70%)
tap_action: more-info
secondary_info: '{last_changed}'
colors:
steps:
- '#ff0000'
- '#ffff00'
- '#00ff00'
gradient: true
sort:
by: state
filter:
exclude:
- name: state
value: 70
operator: '>'
- name: entity_id
value: sensor.front_door_battery
- name: entity_id
value: binary_sensor.living_room_motion_sensor_1_battery_low
include:
- name: attributes.device_class
value: battery
collapse: 6
entities:

entity: sensor.xxx_s_iphone_battery_level
name: xxx's phone

please paste here your card config

**Screenshots**
<!-- If applicable, add screenshots to help explain your problem. -->

**Version**
<!-- What is the version of the card? -->
<!-- If regression happened after updating Home assistant please add HA version as well. -->

Just updated and now my card shows no entities, instead of loads:
`type: custom:battery-state-card
title: Batteries
secondary_info: '{last_changed}'
filter:
include:
- name: attributes.device_class
value: battery
sort:
by: state
collapse: 8
bulk_rename:

from: ' Battery'
from: ' level'
colors:
steps:

'#ff0000'
'#ffff00'
'#00ff00'
gradient: true
`

Sort seems to be the issue here.  Once I removed sort from mine everything showed up again.
I tried...
sort: state
I even tried...
sort:
by: state
In both cases, none of the batteries will display.  As soon as I remove sort everything appears again.  So, I'm suspecting this is a code defect in the sort code.

Sort seems to be the issue here. Once I removed sort from mine everything showed up again.
I tried... sort: state
I even tried... sort: by: state
In both cases, none of the batteries will display. As soon as I remove sort everything appears again. So, I'm suspecting this is a code defect in the sort code.

Yep, I can confirm that removing sort fixes the issue. I'll wait for a fix (the next release) before upgrading.

This fix isn't actually working for me. Even removing sort doesn't make the card appear.
It seems like
filter:
  include:
    - name: attributes.device_class
      value: battery

doesn't seem to be generating entities anymore. If I manually add entities then they show up.

the same here.
remove
sort:
  by: state

for now solve problem, but list isnt sorted 📦 waiting for fix

Duplicate of https://github.com/maxwroc/battery-state-card/issues/623 (same root cause / core issue)

ver. 3.1.2 not solved problem. Still exsist

Can you please tell me what is your lang set in HA and how the decimals are being formatted - curious whether comma or dot is being used to separate decimal values: e.g.
98.5  vs.  98,5
I assume the problem happens when comma is used. Please let me know if I'm correct.
The additional thing which I'd like you to try is to use the following in your config (card level)
default_state_formatting: false

I think this will mitigate the issue

Hi Max,
It's GB, I'm in the UK so it's 98.5. Let me try using default_state_formatting: false and I'll come back shortly.

That didn't help. Here is where I added the config.
type: custom:battery-state-card
secondary_info: '{last_changed}'
title: Battery levels
default_state_formatting: false
tap_action: more-info
....

Weird. Are there any errors in the browser dev console?

I have the same problem.
I have both entities and filter to add all the entities I want. Like this.
If I remove the entities part the sort function is working.

If both used together the sort function is not working, and nothing is shown.
entities:
  - entity: vacuum.audun
  - entity: sensor.audun_sin_iphone_battery_level_2
    charging_state:
      entity_id: sensor.audun_sin_iphone_battery_state_2
      state: Charging
  - entity: sensor.edwin_sin_iphone_battery_level
    charging_state:
      entity_id: sensor.edwin_sin_iphone_battery_state
      state: Charging
filter:
  include:
    - name: entity_id
      value: sensor.sensor*_battery*
      state: charging
    - name: entity_id
      value: sensor.*_battery*
      state: charging
    - name: entity_id
      value: sensor.*_batteri*
      state: charging
    - name: entity_id
      value: sensor.battery
      state: charging
  exclude:
    - name: entity_id
      value: sensor.*iphone_battery_state*
    - name: entity_id
      value: sensor.*mobil_battery_state*
    - name: entity_id
      value: sensor.ringeklokke_battery_temperature

Can someone please confirm whether there are any errors in the console?

Hi Max, Does this help? sorting.ts:30  Uncaught (in promise) TypeError: r.replace is not a function    at sorting.ts:30:50    at Array.find ()    at sorting.ts:17:21    at Array.sort ()    at sorting.ts:15:28    at se. (battery-state-card.ts:72:25)    at Generator.next ()    at o (tslib.es6.js:118:58)(anonymous)     @          sorting.ts:30(anonymous)     @          sorting.ts:17(anonymous)     @          sorting.ts:15(anonymous)     @          battery-state-card.ts:72o            @          tslib.es6.js:118Promise.then (async)                   l             @          tslib.es6.js:120(anonymous)     @          tslib.es6.js:121e            @          tslib.es6.js:117(anonymous)     @          lovelace-card.ts:42(anonymous)     @          utils.ts:125setTimeout (async)                       (anonymous)     @          utils.ts:125set hass @          lovelace-card.ts:56(anonymous)     @          hui-view.ts:153value     @          hui-view.ts:151performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          hui-root.ts:607_$AE     @          reactive-element.ts:1378performUpdate @          reactive-element.ts:1343scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          partial-panel-resolver.ts:101value     @          hass-router-page.ts:91performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          hass-base-mixin.ts:56_updateHass      @          context-mixin.ts:95(anonymous)     @          connection-mixin.ts:227s            @          store.js:19(anonymous)     @          entities.js:74e.subscribeMessage.type            @          entities.js:76(anonymous)     @          connection.js:23_handleMessage             @          connection.js:15  From: Max ChodorowskiSent: 08 January 2024 12:04To: maxwroc/battery-state-cardCc: dnedeltchev; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) Can someone please confirm whether there are any errors in the console?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
after add:
sort:   by: state
console shows errors:

remove sort - no errors
 BATTERY-STATE-CARD  3.1.2
slider-entity-row.js?hacstag=1448997001741:1 SLIDER-ENTITY-ROW 17.4.1 IS INSTALLED
kiosk-mode.js?hacstag=497319128520:1 ≡ kiosk-mode             ⋮
version 5.2.0            ⋮
card-mod.js?hacstag=190927524341:1 CARD-MOD 3.4.1 IS INSTALLED
card-mod.js?hacstag=190927524341:1 You may not be getting optimal performance out of card-mod.
 I was suspecting that this might be the problem but I thought that the code cannot be in such state that this particular line breaks...
I'll try to figure out how come this state of the card was possible.
Thanks!

Is there someone willing to install a custom card code (probably it will be the best to temporary replace the existing file) and later paste what is being logged (everything till the first error):
battery-state-card.zip
This one contains additional logging which can help me to find the actual problem.

If it helps, sort by name works, but by state not.

Yeah, I know, it is only sorting by state which is broken.
Looking forward to getting logs from someone who has a repro (see my prev post)

Ok, I just replaced the .js file with the one in the zip and I have the same problem, no entities display. Here are the logs. bubble-card.js?hacstag=680112919163:489  Bubble Card  v1.6.3 card-mod.js?hacstag=190927524342:1 CARD-MOD 3.4.2 IS INSTALLEDclock-weather-card.js?hacstag=522634019221:175   CLOCK-WEATHER-CARD  Version: 2.2.1utils.ts:1  BATTERY-STATE-CARD  3.1.2mushroom.js?hacstag=444350375323:3557 🍄 Mushroom 🍄 - 3.2.3room-card.js?hacstag=45444094910802:30  ROOM-CARD  1.08.02banner-card.js?hacstag=1867657040130:162 banner-card: 0.12.0hui-element.js?hacstag=249942054180547:1 HUI-ELEMENT 1.1.2 IS INSTALLED button-card.js?hacstag=146194325411:426   BUTTON-CARD   Version 4.1.1 fold-entity-row.js?hacstag=150781994220:78 FOLD-ENTITY-ROW 2.2.0 IS INSTALLED sorting.ts:30  Uncaught (in promise) TypeError: r.replace is not a function    at sorting.ts:30:50    at Array.find ()    at sorting.ts:17:21    at Array.sort ()    at sorting.ts:15:28    at se. (battery-state-card.ts:72:25)    at Generator.next ()    at o (tslib.es6.js:118:58)(anonymous)     @          sorting.ts:30(anonymous)     @          sorting.ts:17(anonymous)     @          sorting.ts:15(anonymous)     @          battery-state-card.ts:72o            @          tslib.es6.js:118Promise.then (async)                   l             @          tslib.es6.js:120(anonymous)     @          tslib.es6.js:121e            @          tslib.es6.js:117(anonymous)     @          lovelace-card.ts:42(anonymous)     @          utils.ts:125setTimeout (async)                       (anonymous)     @          utils.ts:125set hass @          lovelace-card.ts:56(anonymous)     @          hui-view.ts:153value     @          hui-view.ts:151performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          hui-root.ts:607_$AE     @          reactive-element.ts:1378performUpdate @          reactive-element.ts:1343scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          partial-panel-resolver.ts:101value     @          hass-router-page.ts:91performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726j             @          lit-html.ts:1883_$AI      @          lit-html.ts:1825v            @          lit-html.ts:1192g            @          lit-html.ts:1540_$AI      @          lit-html.ts:1384z            @          lit-html.ts:2183update  @          lit-element.ts:165performUpdate @          reactive-element.ts:1329scheduleUpdate              @          reactive-element.ts:1261_$Ej       @          reactive-element.ts:1233await in _$Ej (async)                    requestUpdate  @          reactive-element.ts:1208set         @          reactive-element.ts:726value     @          hass-base-mixin.ts:56_updateHass      @          context-mixin.ts:95(anonymous)     @          connection-mixin.ts:227s            @          store.js:19(anonymous)     @          entities.js:74e.subscribeMessage.type            @          entities.js:76(anonymous)     @          connection.js:23_handleMessage             @          connection.js:15 Sent from Mail for Windows From: Max ChodorowskiSent: 08 January 2024 17:45To: maxwroc/battery-state-cardCc: dnedeltchev; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) Is there someone willing to install a custom card code (probably it will be the best to temporary replace the existing file) and later paste what is being logged (everything till the first error):battery-state-card.zipThis one contains additional logging which can help me to find the actual problem.—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Where do you get this log from? It doesn't look like from the developer console in your browser. On the PC you can usually access your browser developer console by pressing F12 (then look for the console tab)
Please make sure that the js file is refreshed in the browser Ctrl+F5. Disabling the browser cache should help as well

Yep, that is from the F12, dev console of Edge, I don’t use Chrome and I don’t have the disable cache, unless it’s somewhere hidden in Edge. Should I install Chrome and try? From: Max ChodorowskiSent: 08 January 2024 20:39To: maxwroc/battery-state-cardCc: dnedeltchev; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) Where do you get this log from? It doesn't look like from the developer console in your browser. On the PC you can usually access your browser developer console by pressing F12 (then look for the console tab)Please make sure that the js file is refreshed in the browser Ctrl+F5. Disabling the browser cache should help as wellimage.png (view on web)—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Found the disable cache, let me try again
On Mon, 8 Jan 2024 at 20:39, Max Chodorowski @.***>
wrote:

Where do you get this log from? It doesn't look like from the
developer console in your browser. On the PC you can usually access your
browser developer console by pressing F12 (then look for the console tab)
Please make sure that the js file is refreshed in the browser
Ctrl+F5. Disabling the browser cache should help as well
image.png (view on web)
https://github.com/maxwroc/battery-state-card/assets/8268674/1d0882f3-756b-4280-8f16-a9b3993238e0

—
Reply to this email directly, view it on GitHub
https://github.com/maxwroc/battery-state-card/issues/630#issuecomment-1881785602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONHCL4RAS5NVF4RXPCNVVDYNRKOHAVCNFSM6AAAAABBNUP2S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRG44DKNRQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

this is weird...
if I am on 3.1.0 and just replace the file with the one in the zip,
everything still works. But if I upgrade to 3.1.2 and then replace the .js
file, nothing works, no entities. Does simply replacing the js file under
www/community/battery-state-card actually does the trick? It doesn't look
like it...
On Mon, 8 Jan 2024 at 20:45, Dimitare Nedeltchev <
@.***> wrote:

Found the disable cache, let me try again
On Mon, 8 Jan 2024 at 20:39, Max Chodorowski @.***>
wrote:

Where do you get this log from? It doesn't look like from the
developer console in your browser. On the PC you can usually access your
browser developer console by pressing F12 (then look for the console tab)
Please make sure that the js file is refreshed in the browser
Ctrl+F5. Disabling the browser cache should help as well
image.png (view on web)
https://github.com/maxwroc/battery-state-card/assets/8268674/1d0882f3-756b-4280-8f16-a9b3993238e0

—
Reply to this email directly, view it on GitHub
https://github.com/maxwroc/battery-state-card/issues/630#issuecomment-1881785602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AONHCL4RAS5NVF4RXPCNVVDYNRKOHAVCNFSM6AAAAABBNUP2S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBRG44DKNRQGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

OK, installed the card manually, not via hacs, now the console logs is going mental, it looks like it’s trying to sort things and here are the logs, they keep going over and over again... it’s almost like it’s in a loop... Sort 2 [sensor.downstairs_battery_level]: 100 (string)battery-state-card.js:1573 Sort 1 [sensor.victor_bedroom_battery_level]: 100 (string)battery-state-card.js:1574 Sort 2 [sensor.side_of_house_battery]: 93 (string)battery-state-card.js:1573 Sort 1 [sensor.victor_bedroom_battery_level]: 100 (string)battery-state-card.js:1574 Sort 2 [sensor.kitchen_battery_level]: 100 (string)battery-state-card.js:1573 Sort 1 [sensor.victor_bedroom_battery_level]: 100 (string)battery-state-card.js:1574 Sort 2 [sensor.downstairs_battery_level]: 100 (string)battery-state-card.js:1573 Sort 1 [sensor.victor_bedroom_battery_level]: 100 (string)battery-state-card.js:1574 Sort 2 [sensor.master_bedroom_battery_level]: 100 (string)battery-state-card.js:1573 Sort 1 [sensor.kitchen_light_switch_2_battery]: 99 (number)battery-state-card.js:1574 Sort 2 [sensor.dimitares_macbook_internal_battery_level]: 98 (string)battery-state-card.js:1577  Uncaught (in promise) TypeError: _a.replace is not a function    at battery-state-card.js:1577:100    at Array.find ()    at battery-state-card.js:1563:25    at Array.sort ()    at getIdsOfSortedBatteries (battery-state-card.js:1561:32)    at BatteryStateCard. (battery-state-card.js:1674:33)    at Generator.next ()    at fulfilled (battery-state-card.js:29:62)(anonymous) @    battery-state-card.js:1577(anonymous) @    battery-state-card.js:1563getIdsOfSortedBatteries      @    battery-state-card.js:1561(anonymous) @    battery-state-card.js:1674fulfilled   @    battery-state-card.js:29Promise.then (async)           step @    battery-state-card.js:31(anonymous) @    battery-state-card.js:32__awaiter       @    battery-state-card.js:28(anonymous) @    battery-state-card.js:263(anonymous) @    battery-state-card.js:186setTimeout (async)               (anonymous) @    battery-state-card.js:186set hass  @    battery-state-card.js:277(anonymous) @    hui-view.ts:153value       @    hui-view.ts:151performUpdate     @    reactive-element.ts:1329scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726value       @    hui-root.ts:607_$AE       @    reactive-element.ts:1378performUpdate     @    reactive-element.ts:1343scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726j       @    lit-html.ts:1883_$AI @    lit-html.ts:1825v      @    lit-html.ts:1192g      @    lit-html.ts:1540_$AI @    lit-html.ts:1384z      @    lit-html.ts:2183update    @    lit-element.ts:165performUpdate     @    reactive-element.ts:1329scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726value       @    partial-panel-resolver.ts:101value       @    hass-router-page.ts:91performUpdate     @    reactive-element.ts:1329scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726j       @    lit-html.ts:1883_$AI @    lit-html.ts:1825v      @    lit-html.ts:1192g      @    lit-html.ts:1540_$AI @    lit-html.ts:1384z      @    lit-html.ts:2183update    @    lit-element.ts:165performUpdate     @    reactive-element.ts:1329scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726j       @    lit-html.ts:1883_$AI @    lit-html.ts:1825v      @    lit-html.ts:1192g      @    lit-html.ts:1540_$AI @    lit-html.ts:1384z      @    lit-html.ts:2183update    @    lit-element.ts:165performUpdate     @    reactive-element.ts:1329scheduleUpdate    @    reactive-element.ts:1261_$Ej @    reactive-element.ts:1233await in _$Ej (async)            requestUpdate      @    reactive-element.ts:1208set   @    reactive-element.ts:726value       @    hass-base-mixin.ts:56_updateHass  @    context-mixin.ts:95(anonymous) @    connection-mixin.ts:227s      @    store.js:19(anonymous) @    entities.js:74e.subscribeMessage.type    @    entities.js:76(anonymous) @    connection.js:23_handleMessage   @    connection.js:15  From: Dimitare NedeltchevSent: 08 January 2024 20:52To: maxwroc/battery-state-cardCc: maxwroc/battery-state-card; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) this is weird...if I am on 3.1.0 and just replace the file with the one in the zip, everything still works. But if I upgrade to 3.1.2 and then replace the .js file, nothing works, no entities. Does simply replacing the js file under www/community/battery-state-card actually does the trick? It doesn't look like it... On Mon, 8 Jan 2024 at 20:45, Dimitare Nedeltchev<EMAIL_ADDRESS>wrote:Found the disable cache, let me try again On Mon, 8 Jan 2024 at 20:39, Max Chodorowski<EMAIL_ADDRESS>wrote:Where do you get this log from? It doesn't look like from the developer console in your browser. On the PC you can usually access your browser developer console by pressing F12 (then look for the console tab)Please make sure that the js file is refreshed in the browser Ctrl+F5. Disabling the browser cache should help as wellimage.png (view on web)—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Yeah this is what I was hoping for to see let me go through it...

Please try to copy paste the beginning of the console log. I'm really interested in lines with "sensor.kitchen_light_switch_2_battery" string. We're almost there!!!

Ok I think I have found the root cause.
Can you please test the below file? It still floods the console with the debug logs though
battery-state-card.fix.zip

Worked! Although I had to clear my browser cookies/cache, otherwise a simple refresh didn’t work. My phone, however, Android, worked straight away. From: Max ChodorowskiSent: 08 January 2024 21:45To: maxwroc/battery-state-cardCc: dnedeltchev; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) Ok I think I have found the root cause.Can you please test the below file? It still floods the console with the debug logs thoughbattery-state-card.fix.zip—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
Ok cool. I'll try to release the fix tomorrow. Thanks a lot for your help!

Credit goes to you! It’s an awesome card!  From: Max ChodorowskiSent: 08 January 2024 22:28To: maxwroc/battery-state-cardCc: dnedeltchev; AuthorSubject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630) Ok cool. I'll try to release the fix tomorrow. Thanks a lot for your help!—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
My card is not displayed, starting from version 1.3.1 with value = battery, without it the data appears
filter:
include:
- name: "attributes.device_class"
value: "battery"

Max will be releasing a fix soon. He's found the root cause. For now, you may downgrade to 3.1.0.

From: CH1PSET1 @.>
Sent: Tuesday, January 9, 2024 4:53:24 AM
To: maxwroc/battery-state-card @.>
Cc: dnedeltchev @.>; Author @.>
Subject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630)
My card is not displayed, starting from version 1.3.1 with value = battery, without it the data appears
filter:
include:

name: "attributes.device_class"
value: "battery"

—
Reply to this email directly, view it on GitHubhttps://github.com/maxwroc/battery-state-card/issues/630#issuecomment-1882411388, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AONHCL7KX33N64SIELEBFATYNTEMJAVCNFSM6AAAAABBNUP2S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBSGQYTCMZYHA.
You are receiving this because you authored the thread.Message ID: @.***>

Fix released. Please confirm that it works

i can confirm that now all working.
thanks for quick fix

Works like a charm! Thank you Max!

From: Max Chodorowski @.>
Sent: Tuesday, January 9, 2024 8:46:55 PM
To: maxwroc/battery-state-card @.>
Cc: dnedeltchev @.>; Author @.>
Subject: Re: [maxwroc/battery-state-card] Entities are not showing when sort option is present in the config (Issue #630)
Fix released. Please confirm that it works
—
Reply to this email directly, view it on GitHubhttps://github.com/maxwroc/battery-state-card/issues/630#issuecomment-1883767815, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AONHCLYJ3KSW4J57PUAOATLYNWUD7AVCNFSM6AAAAABBNUP2S2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBTG43DOOBRGU.
You are receiving this because you authored the thread.Message ID: @.***>

I'm glad to hear that. Thanks for your help guys!

@maxwroc Thanks for fixing! Also working for me!


Use python bindings for tests

This PR begins the migration to pytest for all of our e2e tests using IREE's python bindings for both the compiler and the runtime. I've tested it thoroughly (by running run_matmul.sh and cpu_comparison/run.py ~100~50 times).
Two major changes were required to make this happen:

Move all CLI args to AMDAIEOptions so that they're registered at plugin load time. This was necessary for enabling Session.set_flags in Python. Going forward we should make sure to put all CLI args in the same place;
The XRT HAL had to be drastically refactored because in its current state (prior to this PR), it doesn't support multiple dispatches in a single process - I had to track down all kinds of resource leaks (why this PR took so much longer than I thought);

There's a test 1536x2048x1536 that was prior disabled for Windows but is now disabled entirely. I don't know what the connection is (I wasn't able to track it down) but since every other test passes, including the 1000 run test, and even after repeating overall ~100~50 runs, I think it's safe to conclude there's something wrong with that shape rather than the HAL.

The migration isn't complete because the bindings and the VM itself don't support bf16 yet; I will send them a PR soon.
Note, in its current state the test scripts run ~100~50 times. I will remove this before landing.
TODO (for the next iteration): right now repeat_runs doesn't work because of how the XRT command buffer works (it's free'ed after every dispatch and with it the xrt::kernel and xrt::hw_context). Likely I'm doing something wrong (possibly using the wrong command buffer model) so I think it should be fixable.

Off the top of my head things which we need to reach parity, hopefully we can reuse what's already done in cpu_comparison/run.py:

When the numerics fail, give good feedback
Random seed control
Only sample for very large matmuls, don't perform full CPU computation (maybe not needed)

Also, can you/someone please at some point:

document in the README how to run the pytests in the test dir
create an issue describing what work still needs to be done
keep track of migration of tests from old script to pytest, ideally they shouldn't be run in both places. Or maybe they've all been moved already?

keep track of migration of tests from old script to pytest, ideally they shouldn't be run in both places. Or maybe they've all been moved already?

None have been moved, some have been duplicated.


R2 vec api

Please take a look.

same PR than #1298 (but for r2.Vec)

Ugh, this identified an issue with the approach I'm taking for doing the linting checks; it cannot extend the git history when the PR comes from another user's repo.
I'll look into it.

The issue here is git fetch --shallow-exclude=master origin r2-vec-api uses origin and it should determine the repo from Travis variables (I hope this is possible). It's not.
If you send a PR that outputs the result of git remote during a Travis build, what is that output? I think this is the only possible way to get this information.

Can you rebase this on to master please.

done.


SO-2929: Report error on relationships with inactive source, destination or type concepts

This pull request adds a validation rule to test if a relationship has inactive source, destination or type concept.
https://snowowl.atlassian.net/browse/SO-2929

LGTM!


home-assistant: 0.91.4 -> 0.92.0

Motivation for this change
https://www.home-assistant.io/blog/2019/04/24/release-92/
This was a complex update due to home-assistant/home-assistant#22699.
The parse-requirements.py script needed quite some changes to account for the new manifest.json. But the new code turned out shorter than the old one.
I also have completely rewritten the code that accounts for the platform = "component" way of configuration. The respective commit (fe65a43dff1b3f29943b9b0016b7bcc838427ed5) should be backported. This will make it rather useless to use autoExtraComponents with a pre-0.92 version of Home Assistant. Is this something that should be possible?
Due to the extensive changes, I invite everyone to test thoroughly. Please bear in mind that you will need to apply fe65a43dff1b3f29943b9b0016b7bcc838427ed5 to your nixpkgs checkout if you build your system from one of the release channels.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc @netixx

This breaks transifex-client, which requires python-slugify<3.0.0.
cc @etu

This breaks transifex-client, which requires python-slugify<3.0.0.

 It probably doesn't require that really. It has made lots of absurd claims in the past. I'll look into this tomorrow at work (in about 8-9 hours or so). I already patch the package to get rid of some of the absurd claims of required versions of deps. But yeah, I need it so I'll fix it somehow.

A number of platforms failed to initialize for me:

darksky
pushover
broadlink
onkyo

Error setting up platform pushover

Traceback (most recent call last):
  File "/nix/store/1zvaffg7c9n2nwyqsfnsram172h8jqal-homeassistant-0.92.0/lib/python3.7/site-packages/homeassistant/components/notify/__init__.py", line 71, in async_setup_platform
    platform.get_service, hass, p_config, discovery_info)
  File "/nix/store/kimimvhimclyfzlncpg36zjni3wn70nq-python3-3.7.3/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/nix/store/1zvaffg7c9n2nwyqsfnsram172h8jqal-homeassistant-0.92.0/lib/python3.7/site-packages/homeassistant/components/pushover/notify.py", line 26, in get_service
    from pushover import InitError
ModuleNotFoundError: No module named 'pushover'

Ahh wait, while this is the package used, the module is still the one from stable and thus without the mentioned commit.

@dotlambda I have tested building transifex locally, and changing this line: https://github.com/NixOS/nixpkgs/blob/7e2eb72407934c44cf72694772d346c684a35673/pkgs/tools/text/transifex-client/default.nix#L20 is enough to get it to work. I did set it to 4.0.0 locally.

@etu I have added a commit making the suggested change.

A number of platforms failed to initialize for me:

darksky
pushover
broadlink
onkyo

Since all of these use the platform = ... way of configuration, we should be fine once the commit is applied.

Feel free to merge into staging once it's ready.

@peterhoeg Did you try again?

I didn't, no, and I will not be able to do so before the weekend.

I ve tried to add:
sensor = [ {
      platform = "darksky";
      api_key = "APIKEY";
      monitored_conditions = [
        "temperature"
        "humidity"
      ];
    } ];

but didnt install dependent packages (python-forecastio).

@juaningan I tested your config and added builtins.trace to see which packages are added to componentBuildInputs. python-forecastio was one of them. Are you sure your NixOS system is built from a Nixpkgs checkout with 38a6d3c0426c0d6f37ae814ba49c7c6d2912f27b apllied?

yes, I'm using
let
  unstable = import (builtins.fetchTarball {
     url = https://github.com/dotlambda/nixpkgs/archive/home-assistant-0.92.tar.gz;
  }) {};
in
services.home-assistant.package = unstable.home-assistant

anyways I will double check.

Tried again:
ModuleNotFoundError: No module named 'forecastio'
If I add the component in extra packages then works.

The nixpkgs version used to build your system, not the one Home Assistant is built from, must include the patch. That's because it's the NixOS module's task to detect the used components.
Maybe we could change this so that the package is passed autoExtraComponents and config.

I understand, I really like to select which packages I prefer bleeding edge in my system without affecting system wide. So I can keep with my workaround. Or do you know how can I override only that module?

That patch will be backported to 19.03, so you won't have to change anything. But for testing, you have to build from a local nixpkgs checkout of your channel with the patch appied.

I'll merge on 2019-05-05 20:00 UTC if there's no further comment until then.


reimplement javascript wrapper in rust

Decide if this is doable within this sprint or if its lower priority

@laudiacay we should discuss

@zevkent Rocket should handle almost all of the js wrapper


Fixed #33909 -- Corrected django.dispatch.receiver() signature.

Nice. Thanks for jumping on this. I was planning on it, but also failing to act on those plans!


firefox-pmos-mobile: init at -pmos-2.2.0

Motivation for this change
This PR adds a wrapped version of Firefox with postmarketOS's mobile-friendly configurations. The package version is (browser version)-pmos-2.0.1. It's possible to remove the postmarketOS-branded initial homepage by setting keepHomepage = true;.

Screenshot

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

May I suggest we drop the postmarketOS branded homepage by default?
I'm also not sure what are the guidelines with Mozilla branding and non-default homepages.

May I suggest we drop the postmarketOS branded homepage by default?

Sure, I guess it makes things less confusing.

I'm also not sure what are the guidelines with Mozilla branding and non-default homepages.

That's actually a good point, I'm not sure if we can use the Firefox trademark with such modifications sans approval from Mozilla.

Reminder to committers/contributors: don't remove the ofborg internal error tag! Only a collective member can do so safely!

I don't know if changes that are not changing the "trademarky" bits are or are not fine. And even me saying it might not be fine to change the homepage is entirely conjecture.

Why don't we call the package mobile-config-firefox? That's what the AUR and postmarketOS do: https://repology.org/project/mobile-config-firefox/versions.

@dotlambda

Why don't we call the package mobile-config-firefox? That's what the AUR and postmarketOS do: https://repology.org/project/mobile-config-firefox/versions.

It's the config itself rather than the wrapped browser with the firefox executable. Here I call it mobile-config-firefox as well but don't expose it in all-packages as it's not meant to be consumed alone.

Gentle poke. Can we get this merged?

@zhaofengli i rebased your patches against master, updated 2.2.0 -> 4.0.2, and implemented an updateScript. deployed to a Pinephone, and it's working as expected.
if you care to shepherd this PR through, feel free to pull my patches. i've left them unsquashed so that you can see what changes -- probably want to squash them before merging. i implemented librewolf-pmos-mobile to show that we got the libName stuff right -- probably not worth shipping it though: https://github.com/NixOS/nixpkgs/compare/master...uninsane:nixpkgs:firefox-pmos-mobile

@uninsane Unfortunately I haven't touched mobile stuff in a while so it would take me some time to ramp back up and test things. Could you open a new PR and link it here? I'll then close this one.

@uninsane at https://github.com/NixOS/nixpkgs/pull/206054#issuecomment-1608640136

@bendlas if you have a moment, care to take a look over https://github.com/NixOS/nixpkgs/pull/121356#issuecomment-1608619585 to weigh in? i just posted a patch there that gets us inline with upstream, but it's a pretty different approach than what just got merged, so i'm not sure the best way to proceed.

Hey, sorry, I didn't see we had this, before merging. I've got no problem with updating what I merged, with this PR (including the name, firefox-pmos-mobile), if you're inclined to rebase.
I'll inline comment some points:

the patch i shared above causes firefox to ignore touch input. i'm not sure if this is an issue only when using wayland, or sway, or if it's a packaging bug. either way, i'm finding it more practical to use webkit-based browsers on mobile (Tangram, epiphany), so putting this on hold.
but if anyone else makes progress on updating this package, ping me & i'll be happy to test it on hardware for you.


Why search cfg.SOLVER.IMS_PER_BATCH = 8 ?

cfg.SOLVER.IMS_PER_BATCH = 8
cfg.SOLVER.MAX_ITER = 88888888
cfg.TEST.IMS_PER_BATCH = ngpus_per_node

when I change it, training is very slow.
And when i  start more than one run_server.sh: ERROR RuntimeError: Address already in use.
Thx!!!!

The number of IMS_PER_BATCH depands on your GPU memory.
Set different "master_port"  for different servers, then they can run at the same time.

The number of IMS_PER_BATCH depands on your GPU memory.
Set different "master_port"  for different servers, then they can run at the same time.

Thx！！！！


Double search fields on country search

Country selection is showing two overlapping search fields. Without input focus it look like this (hints overlapping):

With input focus is looks like this:

Same results on:
Emulator: Pixel 3a API 33 in latest Android Studio
Physical: Samsung S21 Ultra API 33 (Android 13)

What theme are you using, you may try to use the same as in the sample folder

I copied the theme from the sample app and the results are largely the same. Layout is a bit different, but still have the search fields doubled:

The dialog is launched from an activity derived from AppCompatActivity with this code:
        Bundle args = new Bundle();
        args.putLong(
            TimeZonePickerDialog.BUNDLE_START_TIME_MILLIS,
            ZonedDateTime.now().toEpochSecond() * 1000
        );
        args.putString(
            TimeZonePickerDialog.BUNDLE_TIME_ZONE,
            ZonedDateTime.now().getZone().getId()
        );

        TimeZonePickerDialog timeZonePickerDialog = new TimeZonePickerDialog();
        timeZonePickerDialog.setArguments(args);
        timeZonePickerDialog.setOnTimeZoneSetListener(onTimeZoneSet);
        FragmentManager fm = getFragmentManager();
        timeZonePickerDialog.show(fm, TimeZonePickerDialog.TAG);

I am not sure why it could be duplicating the hints as well as the search icon, make sure it is importing from the correct package as in the sample app
import com.tyganeutronics.timezonepicker.*

as the base library does not use app compact views and uses a Holo theme
If this is not the issue, just like it's done in the library folder, you can override the library Views with your own Views, make sure you also override the classes so that you use the correct views. Let me know if you still face the same issue after.

I have followed the steps in the README which creates a new TimeZonePickerDialog() instance. That class is not present in com.tyganeutronics.timezonepicker so Android studio auto imported com.android.timezonepicker.TimeZonePickerDialog.
When I switch to create com.tyganeutronics.timezonepicker.TimeZonePickerBaseFragment instead (which is derived from TimeZonePickerDialog) the double search goes away.
Still a few other problems, but I'll build the sample app a test them there before opening tickets.

You are correct, the example in the README is outdated, will adjust it to reflect the ones in the sample folder


Alter linux debug stacktraces handling to support more environments

Use -gdwarf-4 to support both LLVM and GCC when calling addr2line
As discussed in #devel on chat.godotengine.org:

GCC and LLVM debug symbols should be compatible but depending on compiler versions and addr2line version in might be support only some versions of debug symbols, so adding -gdwarf-4 build flag to LLVM build might help. LLVM 14 and latest GCC versions use version 5 by default.

This indeed seems to be true, as without specifying -gdwarf-4 addr2line does not work with clang builds (and llvm-addr2line has to be used instead).

Subtract position-independant execuable relocation when passing the address to addr2line
The PIE executable change shouldn't interfere with any logic for non-PIE executables, since for them the relocation is equal to zero.

Just tested this PR and it works! Thanks @vaartis!

Don't merge yet, gonna make calls to addr2line batched, so it doesn't take so long.

Okay, it's a lot faster now since it does all of them in one call.

Thanks!

Cherry-picked for 3.6.


[20.09] nextcloud: init at 21.0.0

Motivation for this change
Partial backport of #113958.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@berbiche fixed :)


change BLE advertise name case to match Ref VI & C5 BT

Coverage remained the same at ?% when pulling a4028425495be11b499a342c17bdd2f40106c8f6 on ble-name into a25820da9423c719099a52101f6e9e46f9e189c7 on next.


Added link within text of Attributions section

This paragraph under the Attributions section references a link from the code example above, but having an in-text link makes the information easier to access.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @Ttesori for your contribution and congratulations on your first PR merge to this repo! 🎉


Added Dispatcher tests of the following methods:

Q
A

Is bugfix?
no

New feature?
no

Breaks BC?
no

Tests pass?
yes

getTraceLevel()
setTraceLevel()
getFlushInterval()
setFlushInterval()
dispatch()
init()

Ready to reviewing

Thank you!


About use Google translate from chrome://settings/languages

About use Google translate from chrome://settings/languages
I hadn't looked into this menu until recently, but today for the first time I see that this option can be enabled and/or disabled. I activated it, added some languages.
https://i.postimg.cc/Znqbzbgd/Luka.png
I tried to try out this translator that I activated, but no menu appeared on any site to try to translate from. Am I doing something wrong or enabling the translation in the menu is just a trick.
(I remember or I'm wrong, but before this translation option was permanently disabled and/or it said "this setting is managed by your administrator".)
Yes, I know that this project is Ungoogled and disconnected from Google services, but for this very reason I noticed that the translation menu can now be activated, I did it, but I could not translate anywhere.
Used ungoogled-chromium_114.0.5735.134-1.1_windows_x64.zip from https://github.com/ungoogled-software/ungoogled-chromium-windows/releases/download/114.0.5735.134-1.1/ungoogled-chromium_114.0.5735.134-1.1_windows_x64.zip
Can you clarify?

One needs to provide a translate URL for this to work. See https://github.com/ungoogled-software/ungoogled-chromium/pull/1175

I looked at the reference you wrote.
But I really don't understand. Can you be a bit more clear as for a complete beginner?!
I have no idea what to do with "--translate-script-url" - there is clearly no such flag.
I open this site: http://www.baidu.com/ and I want to translate it, for example, into Spanish, German, but the "Translate to" menu does not appear. This is my question and the uncertainty of how to make this "Translate to" menu appear for me with your help?!
Who knows how many others like me don't even know that this translation feature is already available, but we don't know how to activate and use it.
Thanks.

Too bad no one wants to help...

It goes without saying that the activation of the google translator from its settings = pointless activation, because there will be no translation. The function is blocked and no one wants to explain it in simple terms, which is unacceptable when asking a question asking for help. Worse is when instead of help there is no one on the other side, and this nowadays demonstrates that society is demoralized and lacks values.

You could simply install an extension for this.

That's not very helpful for others, changing the topic 🤦🏽♂
Can someone close this then?


[BUG]  comm.py global variable has a typo

https://github.com/microsoft/DeepSpeed/blob/master/deepspeed/comm/comm.py
line:484> def all_reduce_coalesced
line:...>     ...
line:491>     global cbd
line:492>     return cdb.all_reduce_coalesced(tensors, op, group, async_op)

line:492 'cdb' is correct, but
line:491 written as 'cbd', error
other refernece code
line:041> cdb = None
line:438> global cdb
line:439> return cdb.reduce_scatter...

thx all.


SKPaymentTransactionStatePurchased on launch are discarded

I'm doing Auto-Renew Subscription and in sandbox, Apple auto renew every 5 minutes for 6 times. when i close my app and reopen within that 30 minute window. i get a warning No callback registered for transaction with state purchased. (https://github.com/chirag04/react-native-in-app-utils/blob/master/InAppUtils/InAppUtils.m#L59) which make sense because there is no callback tied to that product id since this was invoked in the background.
The app shouldn't be discarding these transactions. Maybe put them in some cache and export a method to retrieve and flush this cache.

i have not done auto renewable subscriptions but this is one more example of why the callback api is not the right way to go.
ideally we should have an event driven api like something like:
inapputils.addListener('purchased', function cb(productId) {
 // unlock store etc here.
});

This is going to be a major breaking change but i think that's the way forward.

@chirag04 we can keep the callback api and add event api too.
I think the issue is with calling addTransactionObserver in objective c init. User should call this in js code as early as possible. This is needed for both callback & event api.


QuoteBars are BROKEN.

BasicTemplateForexAlgorithm throws this out:
ERROR:: QuoteBar.ParseTradeAsQuoteBar(): Data formatted as Trade when Quote format was expected.  Support for this will disappear June 2017.
Here's an even simpler example:
    public class ForexTest : QCAlgorithm
    {
        /// <summary>
        /// Initialise the data and resolution required, as well as the cash and start-end dates for your algorithm. All algorithms must initialized.
        /// </summary>
        public override void Initialize()
        {
            SetStartDate(2014, 5, 7);  //Set Start Date
            SetEndDate(2014, 5, 15);    //Set End Date
            SetCash(100000);             //Set Strategy Cash
            // Find more symbols here: http://quantconnect.com/data
            AddForex("EURUSD", Resolution.Daily, Market.FXCM);

        }

        /// <summary>
        /// OnData event is the primary entry point for your algorithm. Each new data point will be pumped in here.
        /// </summary>
        /// <param name="data">Slice object keyed by symbol containing the stock data</param>
        public void OnData(QuoteBars data)
        {
            Debug("I'm here");
        }
    }

So this means, All forex algo's don't work right now without using Tick data or a consolidator.

Minute seems to work, Daily doesn't

It is something to do with CSVLength. It seems Minute and Hourly is length 5, and the condition to use ParseForex is > 5 (Minute is size 11).

Ok, I think this is all happening because perhaps my machine has old format daily data on it? It seems I accidentally included the data in git.

Nope, it's because this repo has old data in the fxcm/daily folder. lets delete all sample data and let the API deal with it?
Lean/Data/forex/fxcm/daily/eurusd.zip

I'm gonna extend this a little:

Remove all API gettable data from the repo
Allow users to automatically download free data, without having to add it through the checkout every time (Accept fxcm etc data agreements as flag on their account - it's not needed per every download right?)

Closing as this is just issue with your data and not related to LEAN.

Sorry for all the spam, but the issue is the data you've left in this repository, in lean : Lean/Data/forex/fxcm/daily/eurusd.zip
This stops the Api from downloading the latest, correct version - please delete it from the repo. When the data was updated around June last year, this file should have been replaced with the new format (QuoteBar) version.

@jaredbroad  I have a pull request ready, but are we actually allowed to include FXCM's data in a repo?   It's just an update of the latest EURUSD and USDNZD, but perhaps the solution should be to delete the sample data and let the user deal with it


Need 4 Speed? How about a new turbo for UI displaying?

Sandor and Damir have complained more than once to me about the slow UI displaying. It’s mainly the “Post-Processor” page where there are many UI controls.
I have discussed this problem with Professor Schafer. We are clear about the reason: UIs here are generated by .m files but not .fig files. If by .fig files, the displaying will be faster, but the codes will become lengthy due to the corresponding OOP kind of things. It’s not difficult but we don’t like it. We want to keep the codes neat and tidy.
I was considering this problem for several months, and figured out a way to partially solve this problem these days, all based on .m files. It is as such:
The first time when we generate the “Post-Processor” page, the speed will be the same as before, it is still the .m file that produce the UIs. But after that the UIs will be safely stored and not get cleared when switching to other pages,. As the result, whenever we switch back to this page, there will be no need to regenerate it. Things become faster.
I’ve tested this method and it works.
There is only one more thing that could take us a little bit time for coding: we need a new “update” action to the UIs when switching back to this page. Currently when we switch back, every UIs are regenerated and initialized. It should be better if the previous settings of the UIs can be kept (the proposed method can), but of course some necessary updates are needed.
So, what do you think about it? Does it worth a try?

I'm interested. But wary if it confuses things. Maybe do it for just one page and show me, then I can see more deeply.
The post-processor is uniquely slow. Somehow it is more than just all the elements I think.
I have been wondering if part of it is order and how I do things. If the axes for the plots are not updated until after the GUI elements are written, would that be faster. Also, I jump around with the GUI elements is that slowing things down. Even the PMM page on the loads pre-processor is much faster than the post-processor page. I still think, I have made some additional mistake or strange thing beyond just having a lot of elements.

About this issue, I have good news and bad news.
Good news is that this problem has been perfectly solved. Even the first time generating the “Post-Processor” page is quick.
Bad news is for me: the solution has nothing to do with my previous proposal.
I changed the generating order, GUIs first and then plots, as suggested Prof. Schafer, then problem solved.
So, next, I will do it in the comparout.m and compareout_cb.m with minimum modifications to the original codes.
Then, maybe someday we can introduce a mechanism for maintaining the user data of the GUIs, so that it can act more like normal dialog boxes.

OK, let's take the win!
Let's finish the vectorization merge, then start this as the next pull request issue.

Closing this issue, based on latest merged commits.


[StyleCop] Fix all warnings in Bot.Builder.Transcripts.Tests

Add a blank line after closing brace

Replaced with #1509


Support for multiple tags?

Question
Not sure if this is a bug or a feature request, so I'm doing it as a question 😄
I want to use the s2i-build action, but we need to tag the image with multiple tags. So far as I can see, this action does not support it?
My initial thought was to use the push-to-registry action to do all the tagging (since push-to-registry support a list of tags), but it looks like push-to-registry needs a 1 to 1 mapping between tags to push.
Anything I'm missing here, or does this go as a feature request to support multiple tags? 😄

@andmos This seems to be a good feature request :+1:
We will try to get this done :slightly_smiling_face:

From what I can see here, it might be that the tagging must be a separate tag step. Not sure if s2i support multiple tags without doing the tagging separately with e.g docker tag
https://github.com/redhat-actions/s2i-build/blob/4b558c78eab3a55e0316514cbcb5a2d853a76e1c/src/s2iExec.ts#L38

Here is an example of this done in Bash from a custom Action I've used:
set -e
if [ "$5" = "" ]; then
    echo "Skipping login for image pull - username not set."
else
    echo "Credentials for builder image registry detected - logging in."
#    echo "$6" | docker login "$4" --username "$5" --password-stdin
fi
# We will always need to login to the registry we intend to push to
#echo "$9" | docker login "$7" --username "$8" --password-stdin
# Grab builder image
docker pull "$2"
# Build
s2i build "$1" "$2" "$3"

TAGS="${10}"

if [ "$TAGS" = "" ]; then
    echo "Skipping custom tagging, using latest - tags not set."
else
    for TAG in ${TAGS//,/ }
    do
        echo "Tagging image ${3} with ${TAG}"
        docker tag "${3}" "${3}:${TAG}"
    done
fi
# Push to output registry

docker push "$3"

Yeah, we did same way in buildah-build action.
There is no option to provide multiple tags in s2i.

I am going to use docker here to tag, as s2i has dependency on docker
Ref: https://github.com/openshift/source-to-image#dependencies

Also added outputs image and tags which outputs name of image built and list of tags respectively. (similar to what we have in buildah-build action)

@andmos you should be able to use this feature in redhat-actions/s2i-build@v2


place the call stack in Core Coupled RAM (CCRAM)

this implies that memory allocated on the "stack" can't be used with the
DMA peripheral.
closes #45

I want to do some benchmarks and write some examples that use the DMA before deciding if this a good idea or not.

:umbrella: The latest upstream changes (presumably 5a3fda8) made this pull request unmergeable. Please resolve the merge conflicts.

Done in #70


Revert "slack review stats (#1935)"

This reverts commit 292fbd0c379af12ad2ed1c7019f9f712c968b7e8.
Summary of changes
Changes introduced in this pull request:

Turns out Slack notification is a Sponsor/premium-only feature.

Reference issue to close (if applicable)

Other information and links

Oh well.


chore: add project shields github

add some project shield for getting deploy info from github

:tada: This PR is included in version 1.0.9 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


More details in docs of enumerate_adapters

(The RTD CI build fails because Github is glitchy today.)

(The RTD CI build fails because Github is glitchy today.)


Failure in front-end Windows test runner after recent CL

The change https://dart-review.googlesource.com/c/sdk/+/353103
causes the batch compiler for front-end compilation tests to fail on Windows.
There is a comment on the CL:
* Patched the bulk compiler to be able to capture coverage information
  (likely only works in non-Windows as it has to hook into signaling to
  be able to capture data as the process is being killed by the test.py
  script).

The windows front-end compilation tests run on a nightly schedule, and are now broken because
they use the batch compiler. Can the compiler be fixed so it doesn't try to do this new stuff when running
on Windows?
The failure output is:
C:\src\dart\sdk>git reset --hard 8a6b48f5968bd982b941105f400c47cd8ab30607
HEAD is now at 8a6b48f5968 [CFE] First pass at coverage tool

C:\src\dart\sdk>python3 tools/test.py -ncfe-strong-win corelib/apply_test.dart
No build targets found.
Test configuration:
    cfe-strong-win(architecture: x64, compiler: fasta, mode: release, runtime: none, system: win)
Suites tested: corelib

FAILED: fasta-none release_x64 corelib/apply_test
Expected: Pass
Actual: Crash

--- Command "fasta" (took 14.000534s):
set DART_CONFIGURATION=ReleaseX64 & set DART_SUPPRESS_WER=1 & .\out\ReleaseX64\dart .\pkg\front_end\tool\_fasta\compile.dart --verify --skip-platform-verification -o .\out\ReleaseX64\generated_compilations\cfe-strong-win\tests_corelib_apply_test\out.dill --platform .\out\ReleaseX64\vm_platform_strong.dill -Dtest_runner.configuration=cfe-strong-win --nnbd-strong --packages=.\.dart_tool\package_config.json .\tests\corelib\apply_test.dart (working directory: C:\src\dart\sdk\)

exit code:
253

stderr:
Unhandled exception:
SignalException: Failed to listen for SIGTERM, osError: OS Error: The request is not supported.

, errno = 50

--- Re-run this test:
python3 tools/test.py -n cfe-strong-win corelib/apply_test
[00:15 | 100% | +    0 | -    1]

=== 0 tests passed, 1 failed ===

OK, I see some code in the batchEntryPoint which is catching the failure to handle coverage signals and exiting. I think the Windows batch runner will need to not only avoid failing, but must also stay alive and handle multiple compilation tasks before restarting, otherwise the performance will decrease to around 15 seconds per compilation test, which causes the runner to time out when running 30,000 tests.


Allow the test cases to run without Permission problems

Do wrongly and include unnecessary commits. Will do another pull request


Update count_reads.rst

Added line to show appropriate answer for the given commands.

thank you @Careyjmac


Quick question on changes in 0.10.0

In the changelog I see:

The shape of some generated response types changed to be flatter and more ergonomic.

Some of the types I have are complaining that they no longer exist or they have similar names. Unfortunately when I try to "Go to definition" it just links me to the derive macro, so it's hard for me to see what the generated type structure is.
Is there either a way for me to see those generated types or is there somewhere I can read more in depth about what changed between 0.9.0 and 0.10.0? Thanks!

I think I figured it out! The changelog does say it flattened some responses so for some of the missing fields I just removed a property accessor, and for the misnamed ones I just need to change them to camelCase. Hope this helps anybody else!


Better CSS at-rules handling (fixes #2854)

Description of the pull request:
Excludes contents of @keyframes someAnimation {...} blocks from the transformation process.
Fixes the following issues:
#2854
Is breaking:
No.

Note that this doesn't make @keyframes scoped, it only fixes the cases when Ractive would break the CSS. Making @keyframes scoped would most likely be quite complicated and I think the chance of collision is relatively small. Maybe a warnOnceIfDebug() message when we detect @keyframes would be a good idea here?

Having not authored many keyframes myself, this may be a terrible idea, but could the transform just note the names and do a search and destroy with scoping at the end of the process?

@evs-chris if you mean renaming the animation and all its usages the problem is that simple search & replace would break stuff if the animation name was also used in other places (part of property name/value). To do it safely, we'd need more complicated parsing.

Yeah, probably not worth it, but at least there's a solid starting point if it comes up in the future. Good to go?


RailsConf 2017 role call

Hey all,
Is anyone planning anything for RailsConf? I'm headed down and would love to meet some other maintainers and contributors. Drop a comment if you are going to be there!
Lew

Would love to make it, so far no such luck.

I will not be there, but will probably watch some of the talk livestreams.

How was it? Did you meet anyone who uses Exercism @parkerl ?


Rename files to remove spaces

Hi, Git seems to be acting up when there are spaces in the filenames (plus we can't import them in Python). Can we just like rename "Useful code" to helpers or something?

No problem, I'll throw an underscore in


Recognize variables

Recognize variables defined in (a) parameters section and (b) SystemContainer class and provide completion for them when one writes % char.
After such completion, it should also finish the variable with closing percent sign (I guess there is a callback for it)

1+. First, colored variables would be fine.


[BUG]: Baklava select input popover close event trigger problem

Issue description
When typing in the bl select input, pressing the space bar causes the popover to close.
Media & Screenshots

Baklava Version
No response
Operating system
No response
Priority this issue should have
Low (slightly annoying)
Please review the checkboxes that are applicable.

[ ] This issue is directly related to React

merged https://github.com/Trendyol/baklava/pull/796


[HUDI-934] Add processing logic for the decimal LogicalType

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
This PR is to fix a type cast exception when updating a MOR table with decimal type.
JIRA: https://issues.apache.org/jira/browse/HUDI-934
Brief change log

return a HiveDecimalWritable object when the logical type of  fixed type is decimal.

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[x] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Hi, as the JIRA describes, this PR is intend to fix a type cast exception in hive, BytesWritable to HiveDecimalWritable, after updating a MOR table with decimal type.
Steps to reproduce：
1、create a MOR table with decimal type, insert data with hiveSync
2、update this table
3、query the _rt result table with hive
In avro, the decimal is a logical type, and its underlying type is a fixed (or bytes). I add a processing logic when field schema's type is fixed and logical type is decimal, return a HiveDecimalWritable instead of BytesWritable.

@leesf @luoyajun526 The code LGTM. But I think we need to make sure existing integration tests for MOR cover this case. The integration tests use HoodieTestDataGenerator to generate data. If we can modify the trip schema in HoodieTestDataGenerator to add a decimal field as well, I think this case will get covered through existing integration tests which test MOR table as well as perform upserts on them and then query using hive. If you don't want to block this PR on integration tests, we can open a JIRA for this.

@leesf @luoyajun526 The code LGTM. But I think we need to make sure existing integration tests for MOR cover this case. The integration tests use HoodieTestDataGenerator to generate data. If we can modify the trip schema in HoodieTestDataGenerator to add a decimal field as well, I think this case will get covered through existing integration tests which test MOR table as well as perform upserts on them and then query using hive. If you don't want to block this PR on integration tests, we can open a JIRA for this.

@umehrot2 @leesf Hi, I added a Jira for this review. HUDI-987


make DatastoreBatchWriter public

It would be useful for DatastoreBatchWriter to be public for instrumentation purposes. Its sibling interfaces DatastoreWriter, DatastoreReaderWriter, DatastoreReader etc are all public so I assume that it not being public is not intentional.

I signed it!

@wsh Could you take a look?

There are a couple of adjacent weirdnesses worth pointing out: BaseDatastoreBatchWriter is public but it's an abstract class whose sole constructor has no access modifier, so in practice I think that means it can only be overridden by another class in the same package.


MAKING THE Image in the Contact section clearer and add some curves on the corner the corner of the blue section.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The image can be made more clearer and the blue headquarters section can be made made with more curves

@Anushkabh can please assign me this issue .
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

hey @ANIRUDH130903 !
This issue is already addressed and assigned to someone else.
keep looking for new issues in the repo.
Happy coding ✨


VSCode extension?

Hello.
I think this tool is the best thing since sliced bread. However, it is a bit unconfrotable to have to install it globally and then run it from the command line.
I think an VSCode extension wrapping this library will be awesome.
I am not sure if it will be worth wrapping this entire library or just cherry pick some components. For example, things that are not going to be needed are:

command line
ability to read files
ability to execute astx files

So maybe it is possible to leverage the power of astx as a separate package?

Thanks!  Oh yeah I've wanted a VSCode extension since I started this... The thing I haven't figured out yet is if it's possible to make something very similar to the search/replace UIs in extensions (the VSCode extension API is extremely limiting).  That's really the only thing holding me up...
Since this lib exports a node API the VSCode extension could wrap that.  I would probably make the extension capable of loading a locally installed astx version as well so that you wouldn't have to upgrade the extension every time I release improvements to astx.

Anyway I'm about to go on a long weekend vacation, but I'll try to research this next week

I also have a couple of extensions and the extensions API it's indeed very limited.
Indeed you can't show an UI like the normal search and replace, but there are different approaches that other plugin authors are taking.
One is to resort to config.json and ask the user to put them there and turn them into commands that you can call on the commands pallette or trigger them with a hotkey. This is what find and transform does
You can open a webview and put any UI you want there, it may look a bit overkill but for a basic UI it may be possible. Some tools go beyond this and allow users to define their own UI, like ego power tools
Lastly seems that you can contribute with a tree view so maybe you can ask the search and replace using a prompt and then display the results on a tree view. Sadly they don't allow custom inputs there

The fact that people are forced to do things like use a webview 🤦
I need to see how the Atom TS experience is these days.

Okay, I guess the situation is getting better because now they allow WebviewViews which can be rendered in the sidebar or panel areas.

Yes,  they have the usual arrogance of "We built this awesome editor for free, you should be grateful and all that". We all know Microsoft right? I still don't trust them.
I also use the VIM extension and it is too bad the performance hits it gets from time to time.
I also saw that there is a custom editors api, which allows you to build, well, that, custom editors. Maybe it is the right thing for showing the list of affected files and maybe a preview.
I didn't knew you can use WebViews on the sidebars. Where you saw that?

I find a similar extension that can only do search. I rise an issue there nikaspran/vscode-ast-query#15

Huh, some sorta interesting capabilities in esquery, but I think having a syntactic JS/TS pattern syntax is a lot better because you don't need to know node types and attribute names

That would definitely be a good starting point!  Astx find cli output colorizes what variables captured, would be nice to do that in an extension (in addition to supporting replacement).  I'll start working on it soon

@danielo515 @linonetwo I have a beta version of the extension out now: https://github.com/codemodsquad/vscode-astx

@jedwards1211 I give you a star on Github, but I cant find it in vscode extension store?

I haven't put it in the vscode extension store yet because it doesn't support publishing beta versions.  I guess I could use 0.x versions if you really want but right now all you have to do is download the VSIX file from the releases page (here's a link to the latest: vscode-astx-1.0.0-beta.8.vsix) and install that with code --install-extension vscode-astx-1.0.0-beta.8.vsix

Oh I had put those instructions in the vscode-astx README but you didn't see it because I had forgotten to set the default branch to beta


add initial /metadata and /userdata endpoints

Adds endpoints for GET /api/v1/metadata and GET /api/v1/userdata

oops! sorry, I accidentally un-drafted this.

Codecov Report

Merging #18 (ed3929c) into main (229939e) will increase coverage by 8.05%.
The diff coverage is 93.33%.

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   84.84%   92.90%   +8.05%
==========================================
  Files           2        6       +4
  Lines          99      155      +56
==========================================
+ Hits           84      144      +60
+ Misses         14        9       -5
- Partials        1        2       +1

Flag
Coverage Δ

unittests
92.90% <93.33%> (+8.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/middleware/identify_instance.go
88.46% <0.00%> (+6.31%)
:arrow_up:

pkg/api/v1/router_responses.go
62.50% <62.50%> (ø)

internal/httpsrv/server.go
94.04% <100.00%> (+8.13%)
:arrow_up:

pkg/api/v1/router.go
100.00% <100.00%> (ø)

pkg/api/v1/router_instance_metadata.go
100.00% <100.00%> (ø)

pkg/api/v1/router_instance_userdata.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 229939e...ed3929c. Read the comment docs.


Use random state to mitigate CSRF attacks

See: https://github.com/casdoor/casdoor/issues/422#issuecomment-1066095439 and https://github.com/jenkins-infra/repository-permissions-updater/issues/2379#issuecomment-1054046999

@seriouszyx

When cadoor is an idp, it can accept random states, and casdoor will return the states provided by the sp as is. So I think currently casdoor does not need to modify the state policy on apisix, it just needs to generate random states on the apisix plugin.

The solution is roughly to generate a random state in SDK (Java, Go...), then Casdoor receives it and just passes it back as part of the response. Is this OK?

When cadoor is an idp, it can accept random state, and casdoor will return the state provided by the sp as is. So I think currently casdoor does not need to modify the state policy, it just needs to generate random state on the apisix plugin.

@Steve0x2a you are right. @ComradeProgrammer plz just update the plugin code to fix this. Casdoor doesn't need to be updated for this part.

The solution is roughly to generate a random state in SDK (Java, Go...), then Casdoor receives it and just passes it back as part of the response. Is this OK?

@Abingcbc for Casdoor as IdP, we can first update the code in: https://github.com/casdoor/casdoor-js-sdk

Fixed in: https://github.com/casdoor/casdoor-js-sdk/pull/31


Updated Readme (info about virtual memory)

As it was mentioned, we need an information about he vm.max_count-Setting in the README. I added this now.

Looks good!


Add binding to Moment.date

docs: https://momentjs.com/docs/#/get-set/date/
issue: https://github.com/reasonml-community/bs-moment/issues/36

Hey @liubko,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 4a62b1d0-01f8-11e9-9333-8383ba3c9018

🎉
any plans to publish this on npm?


-1 as an invalid mob type

Thanks for pushing a (potential) fix for the invalid sound mapping. Over at Geyser we've gotten a report of an invalid mob type: https://github.com/GeyserMC/Geyser/issues/502. Easy fix but I'm not sure what -1 should default to.

A workaround should be in now, let me know if it solves the problem. Honestly, the proper solution would be to go through adding all the defaults that Minecraft uses in it's registries, but I'm worried that implementing catch-all defaults would end up hiding real errors in value mappings.

Closing as fixed and tracking implementation of a proper fix in #540.

Awesome, thanks.

As an update, I dug into the vanilla client a bit more and realized that actually, while some things like entity types do have defaults, others that have been causing problems like sounds actually do not; if you launch Minecraft with the error log, you'll see that whatever thread is processing the packet is constantly printing NPE traces.
In light of this, I've decided to scrap the idea of default values and instead give library users more control over error handling. The latest PacketLib commit (which is now depended on by the latest MCProtocolLib commit) adds PacketErrorEvent, which has a method "setSuppress" that allows users to suppress the internal error handling and disconnect for any packet read/write errors that occur. Instead, apps are free to handle the error however they wish, perhaps by simply ignoring packets with bad values and printing a warning. I've also added some specific exception types for MagicValues related errors to make them easier to identify.


Please consider adding ability to default or toggle external antenna for Arcadyan G4AR

Upon reboot the Arcadyan G4AR appears to revert to internal antenna. Please add ability to default external and or toggle remotely as this setting is currently only available on the Gateway itself.

@clobberx Are you still experiencing this? I recently tested rebooting my Arcadyan G4AR from the HINT Control app, and found it persisted the external antenna setting.

@aaronsilber, you are correct. This must have been a temporary glitch. Antenna settings persist now. Thanks for the follow up!


Switch to github actions

This repo is experiencing pretty large CI backlogs at the moment. I'm guessing they are caused by issues @ Travis CI: https://news.ycombinator.com/item?id=25338983
This PR switches the CI over to Github actions, and removes Travis CI.
You can see outputs at https://github.com/ghiculescu/cancancan/actions (or at least I can). I think you'll need to enable something for the checks to show against this PR.
Other changes:

I dropped Ruby 2.3 support. It's been EOL'd for over a year. https://www.ruby-lang.org/en/downloads/branches/
I included a comment about this, but it's not clear to me why we have a failing test on jruby / AR 5.2. I haven't investigated yet. On Travis this combination was also allowed to fail, so I've excluded it in the test matrix here.
I've included the changes in https://github.com/CanCanCommunity/cancancan/pull/657 to run tests against Rails master. I've also added Rails 6.1. To get a green build here, I've included the single code change from #657.
I just can't get Nokogiri to work on Truffleruby, so I've excluded it until someone can figure that out.
Same goes for Jruby + Rails 6.1. Here's a sample failure. Rails 6 is fine.

@coorasse this is ready to go, what do you think?

@coorasse FYI

I am very happy of switching to GitHub Actions. I have a personal preference for SemaphoreCI but they lack support for build matrixes and exclude some combinations. 😢
Regarding Truffleruby I'd like to keep it running. @eregon can you help @ghiculescu with the nokogiri issue?

See https://github.com/oracle/truffleruby/issues/62, so just adding this should fix it:
    - run: sudo apt-get -yqq install libxml2-dev libxslt-dev
      if: startsWith(matrix.ruby, 'truffleruby')

Thanks for your help @eregon!
@coorasse we are now green on Truffleruby: https://github.com/ghiculescu/cancancan/actions/runs/431036514

@ghiculescu could you please re-enable also Travis in this PR? I'd like to run both of them until I have a green build on Github Actions. Afterwards I'll remove the Travis configuration. Thank you! 🙏

or maybe is enough to rebase and commit. I enabled Actions now.

Hmmm. That doesn't seem to have worked :/

@ghiculescu could you please re-enable also Travis in this PR?

Done @coorasse

Nice! The first build has started 😄 https://github.com/CanCanCommunity/cancancan/actions/runs/452447924


Environment switch does not seem to work

When I switch environment to local.env, it seems that my_env is still reading from .env.

I tried httpyac-cli, and it works fine.
httpyac -e local -all .\test.http

Version info
vscode: 1.95.3 (user setup)
vscode-httpyac: v6.16.4
httpyac-cli: 6.16.4
os: windows 11

Dotenv files

.env

my_env=foo

local.env

my_env=bar

Requst

test.http

@my_env={{my_env}}

{{
    console.info(my_env);
}}
GET https://catfact.ninja/fact

The documentation for how --env works is lacking. local.env is ignored.
I got it to work with .env.
It would be great if I didn't have to serialize any secret for it to work. It should have an option to read the env vars instead.

local.env should work. This feature I am using in my default setup. The problem in the example I am not sure if it is an order problem with the environments. Shouldn't really be, because I use that too. Strange is the first line which basically sets a=a. But I doubt whether this is reason. I need a pc for this, but I have currently given myself a private coding ban. Coding at work is too exhausting. Maybe next week.

Yes,Coding is really tiring. You should take a break.
I changed the first line, problem still there.