No history visible in eve app

Hey I tried to add this cool plugin to my weather station extended plugin but I'm facing some problems.
I added your plugin v0.3.8 according to your description but the history data in the eve app is empty.
Do you have any idea what I might be doing wrong?
You can see my implementation here: https://github.com/naofireblade/homebridge-weather-station-extended/blob/master/index.js
I debugged the fakegato-history service and there seems to be history data in it:
displayName: 'Now History',
UUID: 'E863F007-079E-48FF-8F27-9C2605A29F52',
subtype: undefined,
iid: 25,
characteristics:
 [ Characteristic {
     displayName: 'Name',
     UUID: '00000023-0000-1000-8000-0026BB765291',
     iid: 26,
     value: 'Now History',
     eventOnlyCharacteristic: false,
     props: [Object],
     _events: [Object],
     _eventsCount: 1 },
   S2R1Characteristic {
     displayName: 'S2R1',
     UUID: 'E863F116-079E-48FF-8F27-9C2605A29F52',
     iid: 27,
     value: '8AAAAAAAAACdvCMgAwECAgIDAgQAwA8AAAAAAAAAAAEB',
     eventOnlyCharacteristic: false,
     props: [Object],
     _events: [Object],
     _eventsCount: 1 },
   S2R2Characteristic {
     displayName: 'S2R2',
     UUID: 'E863F117-079E-48FF-8F27-9C2605A29F52',
     iid: 28,
     value: null,
     eventOnlyCharacteristic: false,
     props: [Object],
     _events: [Object],
     _eventsCount: 2 },
   S2W1Characteristic {
     displayName: 'S2W1',
     UUID: 'E863F11C-079E-48FF-8F27-9C2605A29F52',
     iid: 29,
     value: null,
     eventOnlyCharacteristic: false,
     props: [Object],
     _events: [Object],
     _eventsCount: 2 },
   S2W2Characteristic {
     displayName: 'S2W2',
     UUID: 'E863F121-079E-48FF-8F27-9C2605A29F52',
     iid: 30,
     value: null,
     eventOnlyCharacteristic: false,
     props: [Object],
     _events: [Object],
     _eventsCount: 2 } ],
optionalCharacteristics: [],
isHiddenService: false,
isPrimaryService: false,
linkedServices: [],
size: 4032,
minutes: 4,
accessoryName: 'Now',
log:
 { [Function: bound ]
   debug: [Function],
   info: [Function],
   warn: [Function],
   error: [Function],
   log: [Function],
   prefix: 'Wetterstation' },
accessoryType116: '03 0102 0202 0302',
accessoryType117: '07',
accessoryType: 'weather',
firstEntry: 0,
lastEntry: 3,
history:
 [ ,
   { time:<PHONE_NUMBER>, setRefTime: 1 },
   { temp: 1.9, ressure: 998, humidity: 94, time:<PHONE_NUMBER> },
   { temp: 1.8, ressure: 998, humidity: 94, time:<PHONE_NUMBER> } ],
memorySize: 4032,
usedMemory: 3,
currentEntry: 1,
transfer: false,
setTime: true,
refTime: 539212957,
memoryAddress: 0,
dataStream: '',
IntervalID: null,
_events:
 { 'service-configurationChange': [Function: bound ],
   'characteristic-change': [Function: bound ] },
_eventsCount: 2 }````

launch homebridge with the -D option and see if Eve is requesting data in the log. In your addEntry call you have “ressure” instead of “preassure”, finally wait some time to let the history fill up, and refresh data in Eve.

Thank you for your fast reply. I fixed my typo and added the -D flag. I can see in the log that eve is requesting data. But still no data available. Last night this ran for some hours without any result.
Recent log:
[2/1/2018, 9:59:59 PM] [Wetterstation] **Fakegato-timer: addData  Now { time:<PHONE_NUMBER>, temp: 2.1, pressure: 998, humidity: 94 }  immediate:  false
[2/1/2018, 10:00:00 PM] [Wetterstation] **Fakegato-timer: executeCallbacks**
[2/1/2018, 10:00:00 PM] [Wetterstation] First entry Now: 0
[2/1/2018, 10:00:00 PM] [Wetterstation] Last entry Now: 2
[2/1/2018, 10:00:00 PM] [Wetterstation] Used memory Now: 2
[2/1/2018, 10:00:00 PM] [Wetterstation] 116 Now:<PHONE_NUMBER>00000060c5232003 0102 0202 03020300c00f00000000000000000101
[2/1/2018, 10:00:00 PM] [Wetterstation] **Fakegato-timer: emptyData ** Now
[2/1/2018, 10:00:48 PM] [Wetterstation] Data request Now:<PHONE_NUMBER>0000
[2/1/2018, 10:00:48 PM] [Wetterstation] Address requested Now: 1
[2/1/2018, 10:00:49 PM] [Wetterstation] Data Now:<PHONE_NUMBER> 0100 0000 8160c523200000 0000 00 0000
[2/1/2018, 10:00:49 PM] [Wetterstation] Now Entry: 2, Address: 2
[2/1/2018, 10:00:49 PM] [Wetterstation] Data Now:  10<PHONE_NUMBER>00000007cc00b824fc26
[2/1/2018, 10:01:22 PM] [Wetterstation] Data request Now:<PHONE_NUMBER>0000
[2/1/2018, 10:01:22 PM] [Wetterstation] Address requested Now: 1
[2/1/2018, 10:01:22 PM] [Wetterstation] Data Now:<PHONE_NUMBER> 0100 0000 8160c523200000 0000 00 0000
[2/1/2018, 10:01:22 PM] [Wetterstation] Now Entry: 2, Address: 2
[2/1/2018, 10:01:22 PM] [Wetterstation] Data Now:  10<PHONE_NUMBER>00000007cc00b824fc26
[2/1/2018, 10:03:59 PM] [Wetterstation] **Fakegato-timer: addData  Now { time:<PHONE_NUMBER>, temp: 2.1, pressure: 998, humidity: 94 }  immediate:  false
[2/1/2018, 10:04:00 PM] [Wetterstation] **Fakegato-timer: executeCallbacks**
[2/1/2018, 10:04:00 PM] [Wetterstation] First entry Now: 0
[2/1/2018, 10:04:00 PM] [Wetterstation] Last entry Now: 3
[2/1/2018, 10:04:00 PM] [Wetterstation] Used memory Now: 3
[2/1/2018, 10:04:00 PM] [Wetterstation] 116 Now: f00000000000000060c5232003 0102 0202 03020400c00f00000000000000000101
[2/1/2018, 10:04:00 PM] [Wetterstation] **Fakegato-timer: emptyData ** Now
[2/1/2018, 10:04:08 PM] [Wetterstation] Data request Now:<PHONE_NUMBER>0000
[2/1/2018, 10:04:08 PM] [Wetterstation] Address requested Now: 3
[2/1/2018, 10:04:09 PM] [Wetterstation] Now Entry: 3, Address: 3
[2/1/2018, 10:04:09 PM] [Wetterstation] Data Now:  10 03000000f000000007d200b824fc26

ok, data is downloaded by Eve. Is the time on both iOS device and homebridge server correct? Data in the “future” will not be shown. Other than that, it is possibile that Eve does not like the way in which you expose your services and characteristics. Have a look at my fork of your plugin, I’m using it with no problem since months.

I see now that your Raspi is not on time: yes that could be a problem.

[2/1/2018, 10:00:00 PM] [Wetterstation] **Fakegato-timer: emptyData ** Now [2/1/2018, 10:00:48 PM] [Wetterstation] Data request Now:<PHONE_NUMBER>0000 [2/1/2018, 10:00:48 PM] [Wetterstation] Address requested Now: 1 [2/1/2018, 10:00:49 PM] [Wetterstation] Data Now:<PHONE_NUMBER> 0100 0000 8160c523200000 0000 00 0000 [2/1/2018, 10:00:49 PM] [Wetterstation] Now Entry: 2, Address: 2 [2/1/2018, 10:00:49 PM] [Wetterstation] Data Now:  10<PHONE_NUMBER>00000007cc00b824fc26 [2/1/2018, 10:01:22 PM] [Wetterstation] Data request Now:<PHONE_NUMBER>0000 [2/1/2018, 10:01:22 PM] [Wetterstation] Address requested Now: 1 [2/1/2018, 10:01:22 PM] [Wetterstation] Data Now:<PHONE_NUMBER> 0100 0000 8160c523200000 0000 00 0000 [2/1/2018, 10:01:22 PM] [Wetterstation] Now Entry: 2, Address: 2 [2/1/2018, 10:01:22 PM] [Wetterstation] Data Now:  10<PHONE_NUMBER>00000007cc00b824fc26
this part is strange: Eve is requesting data it already has, it should not happen

I'm also having a hard time to see history in Eve.  Not sure if it's related to Eve 3.1, which they just released, or to my inexperience with fakegato.  As far as I can tell from the debug log, at least one entry is stored every 10 minutes.  I'm only seeing sporadic entries in Eve.  I'm using the fakegato version from npm.
The funny thing is my iPhone and iPad show a different history for the same sensor.  I've got multiple motion sensors, multiple door sensors, and multiple weather sensors, all exposed to HomeKit the same way, but behaving differently with respect to showing the history.
I support for Eve devices, the BTLE protocol would probably make sure only one client connects to the log concurrently.  With WiFi, multiple clients will be connected.  Could that be an issue?
As I mentioned on Slack, my Eve door has two additional attributes in the logging service.  I checked my Eve Energy plugs, and they have the same two attributes.
When I trigger motion, or open the door and then look at the events (the ... button underneath the graph), Eve consistently shows the event.

If you are using more than one iOS devices, make sure that you don’t have multiple Eves open at the same time: this is a thing the I suppose would create issue.
During test, it is probable that you repeatedly restarted homebridge with the history just started, so Eve will see every time new histories with low entry number, very similar to those that it already has download. This also could crate issue. Delete all the entries in Eve (not one by one, by with the remove all button, this will reset the history), force close Eve e restart it.
Inviato da iPad

Il giorno 02 feb 2018, alle ore 00:08, Erik Baauw<EMAIL_ADDRESS>ha scritto:
I'm also having a hard time to see history in Eve. Not sure if it's related to Eve 3.1, which they just released, or to my inexperience with fakegato. As far as I can tell from the debug log, at least one entry is stored every 10 minutes. I'm only seeing sporadic entries in Eve. I'm using the fakegato version from npm.
The funny thing is my iPhone and iPad show a different history for the same sensor. I've got multiple motion sensors, multiple door sensors, and multiple weather sensors, all exposed to HomeKit the same way, but behaving differently with respect to showing the history.
I support for Eve devices, the BTLE protocol would probably make sure only one client connects to the log concurrently. With WiFi, multiple clients will be connected. Could that be an issue?
As I mentioned on Slack, my Eve door has two additional attributes in the logging service.  I checked my Eve Energy plugs, and they have the same two attributes.
When I trigger motion, or open the door and then look at the events (the ... button underneath the graph), Eve consistently shows the event.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

just checked on my installation. I have 0.3.8 as well, running from 22/1. No gap and all data is there. Also tried to delete all history, and everything reload.

Ok I will fix the time on my pi later on. If that not works I will take your fork and try to swap code to mine until it not works anymore. I hope it's not the platform API. Because that's much more flexible then single accessories.

Platform should have been fixed by @NorthernMan54 in a previous release. I see that you are using a Temperature service to which you add air pressure and humidity, instead I use the custom Elgato weather service. Eve expects the right history type from the right service and characteristics structure. It has some tolerance, but not too much. We know that a temperature service with only the temperature characteristic is ok, but don’t know if adding also humidity and air pressure makes Eve demanding for the custom weather service.

I see that you are using a Temperature service to which you add air pressure and humidity, instead I use the custom Elgato weather service. Eve expects the right history type from the right service and characteristics structure. It has some tolerance, but not too much. We know that a temperature service with only the temperature characteristic is ok, but don’t know if adding also humidity and air pressure makes Eve demanding for the custom weather service.

I'm doing the same, yet so does the Eve Degree, according to the Gist, see: https://gist.github.com/simont77/3f4d4330fa55b83f8ca96388d9004e7d#elgato-eve-degree.  It does list an additional custom characteristic (130) for the custom Pressure service (00A), which I don't yet expose, because I don't know what type it should be.
It's not easy for me to combine three REST resources on the Hue bridge (it shows different sensor resources for temperature, humidity, and pressure) into a single weather service, as the current design of homebridge-hue is based on a 1:1 mapping of REST resources to HomeKit services.  I could try to expose only temperature in the weather service and put 0's in the log entries for humidity and pressure.

During test, it is probable that you repeatedly restarted homebridge with the history just started, so Eve will see every time new histories with low entry number, very similar to those that it already has download. This also could crate issue. Delete all the entries in Eve (not one by one, by with the remove all button, this will reset the history), force close Eve e restart it.

I left homebridge (test environment) running for some time, adding weather log entries every minute.  I deleted Eve from my test iPhone (using a different iCloud account, so I'm sure it's the only device connect to this homebridge), that should take care of any old history in the app.  Then I re-installed it and connected it to homebridge.  Judging from homebridge's debug output, fakegato returns plenty of entries to Eve, yet Eve only shows 1 or two entries.
[2018-2-2 10:00:44] [Hue] Data request Balcony Air Pressure:<PHONE_NUMBER>0000
[2018-2-2 10:00:44] [Hue] Address requested Balcony Air Pressure: 1
[2018-2-2 10:00:44] [Hue] Data Balcony Air Pressure:<PHONE_NUMBER> 0100 0000 81fe4f24200000 0000 00 0000
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 2, Address: 2
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 3, Address: 3
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 4, Address: 4
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 5, Address: 5
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 6, Address: 6
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 7, Address: 7
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 8, Address: 8
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 9, Address: 9
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 10, Address: 10
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 11, Address: 11
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 12, Address: 12
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 020000001f10000007b603e81cfc26 10 030000001f10000007b603e81cfc26 10 040000001f10000007b603e81cfc26 10 050000001f10000007b603e81cfc26 10 060000001f10000007b603e81cfc26 10 070000001f10000007b603e81cfc26 10 080000001f10000007b603e81cfc26 10 090000001f10000007b603e81cfc26 10 0a0000001f10000007b603e81cfc26 10 0b0000001f10000007b603e81cfc26 10 0c0000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 13, Address: 13
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 14, Address: 14
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 15, Address: 15
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 16, Address: 16
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 17, Address: 17
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 18, Address: 18
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 19, Address: 19
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 20, Address: 20
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 21, Address: 21
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 22, Address: 22
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 23, Address: 23
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 0d0000001f10000007b603e81cfc26 10 0e0000001f10000007b603e81cfc26 10 0f0000001f10000007b603e81cfc26 10 100000001f10000007b603e81cfc26 10 110000001f10000007b603e81cfc26 10 120000001f10000007b603e81cfc26 10 130000001f10000007b603e81cfc26 10 140000001f10000007b603e81cfc26 10 150000001f10000007b603e81cfc26 10 160000001f10000007b603e81cfc26 10 170000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 24, Address: 24
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 25, Address: 25
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 26, Address: 26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 27, Address: 27
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 28, Address: 28
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 29, Address: 29
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 30, Address: 30
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 31, Address: 31
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 32, Address: 32
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 33, Address: 33
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 34, Address: 34
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 180000001f10000007b603e81cfc26 10 190000001f10000007b603e81cfc26 10 1a0000001f10000007b603e81cfc26 10 1b0000001f10000007b603e81cfc26 10 1c0000001f10000007b603e81cfc26 10 1d0000001f10000007b603e81cfc26 10 1e0000001f10000007b603e81cfc26 10 1f0000001f10000007b603e81cfc26 10 200000001f10000007b603e81cfc26 10 210000001f10000007b603e81cfc26 10 220000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 35, Address: 35
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 36, Address: 36
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 37, Address: 37
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 38, Address: 38
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 39, Address: 39
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 40, Address: 40
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 41, Address: 41
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 42, Address: 42
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 43, Address: 43
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 44, Address: 44
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 45, Address: 45
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 230000001f10000007b603e81cfc26 10 240000001f10000007b603e81cfc26 10 250000001f10000007b603e81cfc26 10 260000001f10000007b603e81cfc26 10 270000001f10000007b603e81cfc26 10 280000001f10000007b603e81cfc26 10 290000001f10000007b603e81cfc26 10 2a0000001f10000007b603e81cfc26 10 2b0000001f10000007b603e81cfc26 10 2c0000001f10000007b603e81cfc26 10 2d0000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 46, Address: 46
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 47, Address: 47
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 48, Address: 48
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 49, Address: 49
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 50, Address: 50
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 51, Address: 51
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 52, Address: 52
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 53, Address: 53
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 54, Address: 54
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 55, Address: 55
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 56, Address: 56
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 2e0000001f10000007b603e81cfc26 10 2f0000001f10000007b603e81cfc26 10 300000001f10000007b603e81cfc26 10 310000001f10000007b603e81cfc26 10 320000001f10000007b603e81cfc26 10 330000001f10000007b603e81cfc26 10 340000001f10000007b603e81cfc26 10 350000001f10000007b603e81cfc26 10 360000001f10000007b603e81cfc26 10 370000001f10000007b603e81cfc26 10 380000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 57, Address: 57
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 58, Address: 58
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 59, Address: 59
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 60, Address: 60
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 61, Address: 61
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 62, Address: 62
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 63, Address: 63
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 64, Address: 64
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 65, Address: 65
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 66, Address: 66
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 67, Address: 67
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 390000001f10000007b603e81cfc26 10 3a0000001f10000007b603e81cfc26 10 3b0000001f10000007b603e81cfc26 10 3c0000001f10000007b603e81cfc26 10 3d0000001f10000007b603e81cfc26 10 3e0000001f10000007b603e81cfc26 10 3f0000001f10000007b603e81cfc26 10 400000001f10000007b603e81cfc26 10 410000001f10000007b603e81cfc26 10 420000001f10000007b603e81cfc26 10 430000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 68, Address: 68
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 69, Address: 69
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 70, Address: 70
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 71, Address: 71
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 72, Address: 72
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 73, Address: 73
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 74, Address: 74
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 75, Address: 75
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 76, Address: 76
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 77, Address: 77
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 78, Address: 78
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 440000001f10000007b603e81cfc26 10 450000001f10000007b603e81cfc26 10 460000001f10000007b603e81cfc26 10 470000001f10000007b603e81cfc26 10 480000001f10000007b603e81cfc26 10 490000001f10000007b603e81cfc26 10 4a0000001f10000007b603e81cfc26 10 4b0000001f10000007b603e81cfc26 10 4c0000001f10000007b603e81cfc26 10 4d0000001f10000007b603e81cfc26 10 4e0000001f10000007b603e81cfc26
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 79, Address: 79
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 80, Address: 80
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 81, Address: 81
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 82, Address: 82
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 83, Address: 83
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 84, Address: 84
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 85, Address: 85
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 86, Address: 86
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 87, Address: 87
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 88, Address: 88
[2018-2-2 10:00:45] [Hue] Balcony Air Pressure Entry: 89, Address: 89
[2018-2-2 10:00:45] [Hue] Data Balcony Air Pressure:  10 4f0000001f10000007b603e81cfc26 10 500000001f10000007b603e81cfc26 10 510000001f10000007b603e81cfc26 10 520000001f10000007b603e81cfc26 10 530000001f10000007b603e81cfc26 10 540000001f10000007b603e81cfc26 10 550000001f10000007b603e81cfc26 10 560000001f10000007b603e81cfc26 10 570000001f10000007b603e81cfc26 10 580000001f10000007b603e81cfc26 10 590000001f10000007b603e81cfc26

I create door and motion log entries when an event happens, and when nothing has happened for 10 minutes.  Again, judging by the log, homebridge returns plenty of entries to Eve, but Eve shows none.
[2018-2-2 10:37:47] [Hue] Clock adjust Dining Room Motion: eb682420
[2018-2-2 10:37:47] [Hue] Data request Dining Room Motion:<PHONE_NUMBER>0000
[2018-2-2 10:37:47] [Hue] Address requested Dining Room Motion: 34
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 52, Address: 52
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 53, Address: 53
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 54, Address: 54
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 55, Address: 55
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 56, Address: 56
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 57, Address: 57
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 58, Address: 58
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 59, Address: 59
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 60, Address: 60
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 61, Address: 61
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 62, Address: 62
[2018-2-2 10:37:47] [Hue] Data Dining Room Motion:  0b 34000000f11600000200 0b 35000000f11600000200 0b 36000000f11600000200 0b 37000000f11600000200 0b 38000000f11600000200 0b 39000000f11600000200 0b 3a000000f11600000200 0b 3b000000f11600000200 0b 3c000000f11600000200 0b 3d000000f11600000200 0b 3e000000f11600000200
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 63, Address: 63
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 64, Address: 64
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 65, Address: 65
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 66, Address: 66
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 67, Address: 67
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 68, Address: 68
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 69, Address: 69
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 70, Address: 70
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 71, Address: 71
[2018-2-2 10:37:47] [Hue] Dining Room Motion Entry: 72, Address: 72
[2018-2-2 10:37:47] [Hue] Data Dining Room Motion:  0b 3f000000f11600000200 0b 40000000f11600000200 0b 41000000f11600000200 0b 42000000f11600000200 0b 43000000f11600000200 0b 44000000f11600000200 0b 45000000f11600000200 0b 46000000f11600000200 0b 47000000f11600000200 0b 48000000f11600000200

When looking at the entries while motion is detected, an entry appears in Eve.
[2018-2-2 10:45:24] [Hue] Dining Room Motion: state changed event
[2018-2-2 10:45:24] [Hue] Dining Room Motion: sensor presence changed from false to true
[2018-2-2 10:45:24] [Hue] Dining Room Motion: set homekit motion from 0 to 1
[2018-2-2 10:45:24] [Hue] Dining Room Motion: add history entry {"time":1517564724,"status":1}
[2018-2-2 10:45:24] [Hue] First entry Dining Room Motion: 0
[2018-2-2 10:45:24] [Hue] Last entry Dining Room Motion: 4c
[2018-2-2 10:45:24] [Hue] Used memory Dining Room Motion: 4c
[2018-2-2 10:45:24] [Hue] 116 Dining Room Motion: b61a000000000000fe4f242002 1301 1c014d00c00f00000000000000000101
[2018-2-2 10:45:24] [Hue] Dining Room Status: state changed event
[2018-2-2 10:45:31] [Hue] Data request Dining Room Motion: 01144c00000000
[2018-2-2 10:45:31] [Hue] Address requested Dining Room Motion: 4c
[2018-2-2 10:45:31] [Hue] Dining Room Motion Entry: 76, Address: 76
[2018-2-2 10:45:31] [Hue] Data Dining Room Motion:  0b 4c000000b61a00000201
...
[2018-2-2 10:45:51] [Hue] Dining Room Motion: state changed event
[2018-2-2 10:45:51] [Hue] Dining Room Motion: sensor presence changed from true to false
[2018-2-2 10:45:51] [Hue] Dining Room Motion: set homekit motion from 1 to 0
[2018-2-2 10:45:51] [Hue] Dining Room Motion: add history entry {"time":1517564751,"status":0}
[2018-2-2 10:45:51] [Hue] First entry Dining Room Motion: 0
[2018-2-2 10:45:51] [Hue] Last entry Dining Room Motion: 4d
[2018-2-2 10:45:51] [Hue] Used memory Dining Room Motion: 4d
[2018-2-2 10:45:51] [Hue] 116 Dining Room Motion: d11a000000000000fe4f242002 1301 1c014e00c00f00000000000000000101

@naofireblade, Looks like there's a p missing in pressure:
history:
 [ ,
   { time:<PHONE_NUMBER>, setRefTime: 1 },
   { temp: 1.9, ressure: 998, humidity: 94, time:<PHONE_NUMBER> },
   { temp: 1.8, ressure: 998, humidity: 94, time:<PHONE_NUMBER> } ],

Some hints:

Eve Degree may have a different signature for history, so it may well be that temp+press+hum need a weather service and not a temp service. I don't have history signature for Degree.
-In door and motion you should expose also the additional characteristics. From your log, communication is ok, but then Eve does not recognise the proper pair history-type/accessory-type and shows nothing.
For motion and Door Eve shows only change events, to see all the data you sent you have to export data to xls, there you will see also the repeated entries. All the entry I see in the first part of your log is for the same state. Then you actively change the state and data appear: this is correct

@ebaauw in your Weather log I see strange data, many entries (if not all) have the same time. In this case Eve will show only one point, but the full list if you export to xls

Thanks, @simont77.  I hadn't even noticed the export to Excel function.  Indeed multiple entries show the same time.  Lemme check what I've done wrong.

In door and motion you should expose also the additional characteristics.

Indeed, exposing Eve's custom Duration (12D) and Sensitivity (120) characteristics on the Motion Sensor service instead of my own custom characteristics seems to do the trick for the Motion sensor.  The Hue bridge uses different values than the Eve devices, but nothing that cannot be handled through some additional mapping between Hue bridge and (Eve) HomeKit values.  The Door sensor also works, exposing Times Opened (129), Last Activation (11A), Open Duration (118) and Closed Duration (119), and Reset Total (112) on the logging service.

Eve Degree may have a different signature for history, so it may well be that temp+press+hum need a weather service and not a temp service. I don't have history signature for Degree.

I don't think it has.  Eve is just very particular about the characteristics provided in combination with the logging service, as you mentioned before.  It requires TemperatureDisplayUnits on the TemperatureSensor service and the custom Elevation (130) characteristic for the custom AirPressure Service (00A).  Elevation is used to calibrate the Air Pressure - Eve shows it under the accessory details, like the Temperature Display Units.  I think Elevation is an INT16 - is there any way to hack HMCatalog to verify this?  The unit is meters.  Eve also shows a Detect button for the Elevation.  It asks for permission to access the current location when you press it, so it probably uses the GPS for detection.  I've currently defined Elevation as read-only, so cannot yet test whether it works.
I'll leave my test homebridge running for a couple of hours, but so far it's looking good.

in your Weather log I see strange data, many entries (if not all) have the same time (0x1f100000 i.e. 0x101f after the reftime). In this case Eve will show only one point, but the full list if you export to xls

I have no clue what caused this, especially since most entries show the date of the most recent entry.  I can only theorise it was the mismatch between logging service and the characteristics of the  sensor services.  It seems to be OK now.

After hours of comparing your weather station code with mine, swapping parts and testing, I found the problem. It's the / in the manufactor characteristic of my information service "github.com/naofireblade".
Now I just must find a way to force homebridge to update this information without clearing the IdentifierCache.

...that’s new! We already knew that Eve does not like “:” in the serial number, but likes “-“. I must admit that I was very lucky in getting it working at the beginning.

Yeah "." is also fine. Can you add the problematic characters in the readme? That would help other a lot.

...uhm, maybe the problematic character was / also for the serial number

Did you found a solution to fix that without readding homebridge to homekit?

unfortunately not, it was @NorthernMan54 that had that problem with /. I updated the readme

My long-running test seems to go fine now.  Published homebridge-hue v0.5.71 and installed it to my “production” environment.
Thanks for the info on the ‘/‘ in the SerialNumber!  I don’t think I would have figured that out myself.
It looks like the weather logging also works when the Air Pressure Sensor service is missing.  I still write 0 values for pressure, but the accessory only has Temperature Sensor and Humidity Sensor services.  You see two lines in the Rooms view instead of one, and in the accessory details Eve doesn’t conclude that it’s an Eve Degree device.


Stricter type checking among object types

Hi there! I've been getting some unexpected results when types change between different flavors of object:
var dd = require('deep-diff');

dd([], {})
// => undefined

dd({}, /surely these are different!/gi)
// => undefined

dd(['a','b','c'], {a:1, b:1, c:1})
// => [ { kind: 'D',
// =>    path: [ 0 ],
// =>    lhs: 'a' },
// =>  { kind: 'D',
// =>    path: [ 1 ],
// =>    lhs: 'b' },
// =>  { kind: 'D',
// =>    path: [ 2 ],
// =>    lhs: 'c' } ]

dd({a:1}, [1])
// => [ { kind: 'D',
// =>     path: [ 'a' ],
// =>     lhs: 1 },
// =>   { kind: 'N',
// =>     path: [ '0' ],
// =>     rhs: 1 } ]

Here's a patch that inspects objects a little more closely and reports type changes such as {} -> [] as kind E.

Very nice. Thank you for taking the time to supplement the tests.

Absolutely, thanks for the merge!

Sure thing; It will be a few days before I update the release packages.


Delete preexisting dynamic library first

Fix #128. Copying the build library on top of a preexisting build can result in a segfault if a node is currently using the already-built library.

There is common trick how to perform an atomic operation on *nix filesystem. If you copy into a temporary file at the same volume first and then move temporary to a destination. Then move operation is an atomic operation which means kernel performs unlink and move at once. There is not a time window where the file is missing or incomplete. I would suggest this approach to use here.

@pichi do you know if :file.rename/2 from erlang maps to atomic mv? I did not check that function, as I assumed that atomicity is not an issue here.

I'm not familiar with Elixir enough to know how to make random file name so you should fill it.
compiled_lib = Path.join([target_dir, Atom.to_string(build_mode), src_file])
destination_lib = Path.join(priv_dir(), dst_file)
tmp_lib = Path.join(priv_dir(), random_name())

# Perform atomic copy
File.cp!(compiled_lib, tmp_lib)
File.rename(tmp_lib, destination_lib)

I hope File.rename is implemented using glibc rename() or mv command which does it correctly. I don't see File.rename! variant which makes me a little bit nervous.

@evnu I'm not sure about :file.rename/2. I will see.

Yes, :file.rename/2 for prim_file can be tracked down to
https://github.com/erlang/otp/blob/76c4821cff1e91c26dd2f812fc8cb40aa7d48e57/erts/emulator/nifs/unix/unix_prim_file.c#L813
seems OK.

Elixir's File.rename delegates to file:rename/2:
# lib/elixir/lib/file.ex
  def rename(source, destination) do
    :file.rename(source, destination)
  end

I am unsure how to generate a random file name without falling back to mktemp. I did not find functions which are guaranteed to generate unique values across different nodes. make_ref/0 does not produce unique references across instances, and getting the time also seems to allow race coditions. Ideas?
Note that the proposed change in https://github.com/hansihe/rustler/pull/129#issuecomment-363755568 might leave temporary files around on some systems. If moving a file onto an opened file is not allowed on a certain operating system (I bet this is the case on Windows), the temporary file might not be removed. The original implementation in this PR fails gracefully if deleting an opened file is not allowed, as it does not generate additional files in priv/.


Different image sizes in dataset

Hi,
Thank you for a great repo :D
is there a way to deal with differing image sizes in the dataset? This is the error I get:
  File "/Users/mareksubocz/it/keypoint-detection/keypoint_detection/data/coco_dataset.py", line 235, in collate_fn
    images = torch.stack(images)
RuntimeError: stack expects each tensor to be equal size, but got [3, 1080, 1920] at entry 0 and [3, 1122, 1998] at entry 7

The image's width and height are included in the COCO's json.

Hi @mareksubocz
This is indeed not supported in the dataloader at the moment, and stacking these tensors will give the error you have encountered above. What you will need to do is make sure that all image tensors have the same shape before they are collated in the dataloader.
How to bring them to the same shape? there are many ways to achieve this. One would be to select the largest dimensions in the dataset and to pad all other images to this size, but this will make training more expensive if you have big differences.  Another, more common, strategy is to resize all images to a specific size, e.g. 1920x1080. If you want to keep aspect ratio, you can resize and then pad where needed. This is the approach that YOLO takes for example.
How to implement this? You can achieve this in multiple ways. Albumentations has a number of transforms that you could use to achieve this: Resize while keeping aspect ratio and padding or directly resizing.
You can apply these transforms upfront on your dataset before training, or you can do it at runtime in the keypoint detector.
The latter can be achieved by adding these augmentations to all dataloaders, similar to how it is now done for the training augmentations here. For the former, I think you might find a part of my lab's coco-tooling codebase useful, it allows you to run a number of albumentation transforms on a coco dataset and create updated images/annotations.
Let me know if you encounter issues! I'm rather busy the next two weeks, but afterwards I could even help implementing this, as  it seems like a nice feature to have.

I think I might try to do this on my own and submit a pr, we'll see. Thank you for the tips :D. I guess the keypoint values should also be updated to match the resized image, right?
All the best
Marek

Do you think it would be a good idea to apply resize in detectory.py in order to have it work for inference as well?

I think I might try to do this on my own and submit a pr, we'll see .

Looking forward to it!

I guess the keypoint values should also be updated to match the resized image, right?

Yes, Albumentations can do this. But a slight modification is needed to accomodate for a possibly varying amount of keypoints in each channel. You should use this class instead of the normal Compose.

Do you think it would be a good idea to apply resize in detectory.py in order to have it work for inference as well?

That would be an option as well. You could add a 'preprocess' step to the forward() function. My gut feeling says to do this in the dataset instead of in the detector to keep predicitions and preprocessing separate,  but this indeed implies you need to also do it manually at inference.

I kept the preprocessing inside the datamodule, adding the resize option to all dataloaders, including validation and test ones. It's a very bare bones version of this feature, but worked with some basic tests I ran.
Feel free to change it up or even throw out altogether. Let me know what you think :)


please produce a library version

This is the most beautiful thing I've seen in years and I think it's almost close to zen.
  I only wish you have a library version that can be linked to my program.
Nice job!

Thanks for the feedback. We're glad you have a keen eye for beauty and a distaste for spending tons of time worrying about your linker.
It wouldn't be too crazy to make it a library (although it'd be impossible to multithread right now):

replace all references to alloc with malloc (they won't play nicely at all) and change the Makefile to not have -nostdlib
remove _start since that'll definitely confuse stuff
then write an interface to mainTrain / mainTest. You could probably just pass in (int argc, char **argv) and it'd work.


feat: add support for created and modified methods to AzureBlobFileSystem

@hayesgb would you have time to look at this at some point?

Apologies for the delay @Mause.  This looks good.

Thankyou!!


Testing, please ignore. (nzsgls)

This is a bug bounty test.. Please DO NOT APPROVE THIS! (nzsgls)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.nzsgls nwfbu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Bofrost.be has degraded performance

In 11b9792, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1145 ms

Resolved: Bofrost.be performance has improved in e735d2c.


No warning is printed when @prisma/cli and @prisma/client have different versions in package.json

It's possible to specify different versions of the @prisma/cli and @prisma/client in package.json and users will not get a warning that they did this and might run into weird behaviour because of this (it's also unclear which version of Prisam Client would be generated when the packages have different versions). For example, I have this package.json:
{
  "name": "script",
  "license": "MIT",
  "devDependencies": {
    "@prisma/cli": "2.0.0",
    "ts-node": "8.10.2",
    "typescript": "3.9.6"
  },
  "scripts": {
    "dev": "ts-node ./script.ts"
  },
  "dependencies": {
    "@prisma/client": "2.2.2",
    "@types/node": "13.13.14"
  },
  "engines": {
    "node": ">=10.0.0"
  }
}

When running npm install and npx prisma generate, the CLI output looks normal and doesn't give me a heads-up that the packages have different versions. After the client generation, I'm also not sure if the client is now on version 2.2.2 or 2.0.0.

Is that trivial to achieve? Are there precedents in doing these kinds of things?
I agree that can happen, and don't argue this could be a nice improvement, but wonder if we should handle that ourselves.
If it's trivial to do, we could definitely consider it, otherwise I would rather park that as this is more a general dependency management problem than a Prisma problem per se.
One main consideration: should that be done only upon prisma generate, a subset of other commands (e.g. prisma doctor) or all commands?

Until now, we don't have a practical reason to do this.
The Prisma Client can tell the CLI, which versions of the binaries it needs and downloads the correct ones.
Once that protocol has a breaking change, we can start adding this.

Couldn't that simply warn? I don't think we'd need to go as far as downloading etc. Just checking the version number would be enough?

Yes it could also warn, then we would just not rely on the preexisting tools and conventions (peer dependencies) for that anymore.

I think peer dependencies just give a warning anyway

After further discussion, it looks like the binary version of the client is not correctly resolved. This should be qualified as a bug and fixed.
We can reconsider adding warnings if this still happens after fixing that.

I want to know if this warn may lead to unexpected behavior ?

Which warn exactly? Mismatch of Prisma CLI and Prisma Client? Yes, there are combinations where this is not desirable. We suggest to upgrade so both versions match.

Which warn exactly? Mismatch of Prisma CLI and Prisma Client? Yes, there are combinations where this is not desirable. We suggest to upgrade so both versions match.
If no upgrade to match both versions is done and this warn stopped being displayed? What does it mean? Do you still suggesting to upgrade?

You should be getting a warning on prisma generate on mismatching versions.
Yes, we suggest to use the same versions for CLI and Client so that the warnings goes away and no problems can occur.


Setup Github actions

Automatically check:

[ ] ESlint
[ ] TSC
[ ] Tests
[ ] Prettier

Deploy:

[ ] Auto builds for each pr (needs separate subdomain)
[ ] Nightly build for master branch
[ ] 1 click deploy for new versions

We would also need to set up prettier. Which also means embracing multi-line imports which i hate, but am willing to live with


Reset Button Implementation

When the reset button disappears, the height of the layout it's inside becomes 0 because the reset button was the only visible view inside it. Since android:animateLayoutChanges only affects views inside the layout, the width of the button is animated but not the height. This makes the transition seem a bit jumpy.
Not too urgent to fix as the dialog is still usable in this state, however it would look a lot nicer if this didn't happen.

The reset button is going to be absent from 0.1.0. Helps simplify things a little. I'll add it back later.

I'm not adding this back. It's kind of useless, and I can't come up with a situation where you would actually want this over a button separate from the dialog with the same purpose.


Move pyrealm_build_data documentation into an autodoc style setup

Description
This PR moves all of the text out of the existing pyrealm_build_data.md file and into appropriate __init__.py docstrings within the pyrealm_build_data package. It then adds autodoc directives to pull those docstrings back into the same file. This keeps the package documentation actually within the package.
The PR also removes a couple of outdated files and adds some missing documentation.
Fixes #271
Type of change

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[ ] Make sure you've run the pre-commit checks: $ pre-commit run -a
[ ] All tests pass: $ poetry run pytest

Further checks

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.70%. Comparing base (1f315ba) to head (941f5af).
Report is 206 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #331      +/-   ##
===========================================
+ Coverage    95.29%   95.70%   +0.41%
===========================================
  Files           28       34       +6
  Lines         1720     2469     +749
===========================================
+ Hits          1639     2363     +724
- Misses          81      106      +25

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Created issue #332 to document need to clean up two leaf build data as noted in init.py


Feedback about Sorald's dependency management

Hello, we built a tool to identify bloated dependencies on Java dev trees.
DepClean was used on Sorald's codebase and built a visual rendering of all the project's dependencies.
We would love it if you could take 7min of your time to look at this rendering and provide feedback.
If you are interested you can go to this link
Thanks a lot
DepClean team

@khaes-kth @fermadeiral I assigned myself to this, but I realized it might be better if one of you do it as I have the same task to do over in Spork :)

@khaes-kth Are you done with this? If so, you can close the issue.

Yes, I did it last week.

Great!


[Feature Request] Simple mechanism for setting/accessing ALS from flows and tools

When building tool plugins, it's useful for the tools to be as broadly defined as possible; however, when actually using those tools it's useful for the LLM to have as narrow and concise a definition as possible.
It would be great to be able to do something like:
const myTool = defineTool({name: "myTool"}, async input => {
  const client = new WikiClient(getContext("wiki.server"));
});

defineFlow({name: "myFlow"}, async () => {
  setContext("wiki.server", {...});
  return generate({tools: [myTool], ...}); // passes context via ALS
});

The reason this is different from plugin config is that I discovered there are basically three kinds of config for tools:

Initialization-time config (plugin config handles this)
Inference-time config (LLM provides these arguments)
Contextual config (e.g. Auth, or in this case wiki server config)

It would be great to have facilities for (3).

@mbleigh status? Close if no longer applicable.


PRD: [JB] Test command

Description:
As a user, I want Cody to generate unit tests for code that I've selected.
Objectives:
Provide an easily discoverable, pre-packaged prompt that allows Cody to generate unit tests for code that a user has selected.
### Acceptance Criteria:
- [ ] https://github.com/sourcegraph/jetbrains/issues/406
- [ ] https://github.com/sourcegraph/jetbrains/issues/407
- [ ] https://github.com/sourcegraph/jetbrains/issues/408

Design Tasks:
### Design Tasks

Engineering Tasks:
### Engineering Tasks
- [ ] https://github.com/sourcegraph/jetbrains/issues/118
- [ ] https://github.com/sourcegraph/jetbrains/issues/279
- [ ] https://github.com/sourcegraph/jetbrains/issues/280
- [ ] https://github.com/sourcegraph/cody/pull/2719

Related Docs:
PRD: Cody Chat and Commands

done as of 6.0.0


blur jstree node

I want to use the mouse to click around the tree node.All tree nodes are not selected.
What should be done?

I am afraid I do not understand the question - please clarify and I will reopen. What is the problem you are facing? Is the mouse selection not working?

I want all nodes not to be selected by clicking on the nodes nearby.


Null pointer exception when sample usage testing

Prerequisites

[ ] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filled: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aidealista
Checked that there is not already provided the described functionality

Description
I got a null pointer exception when sample usage testing
Steps to Reproduce

Add this project library to maven dependency
Ran the below code
// with default values
FormatPreservingEncryption formatPreservingEncryption = FormatPreservingEncryptionBuilder
.ff1Implementation()
.withDefaultDomain()
.withDefaultPseudoRandomFunction("laki".getBytes())
.withDefaultLengthRange()
.build();

//usage
String cipherText = formatPreservingEncryption.encrypt("12121", "mytweak".getBytes());
String plainText = formatPreservingEncryption.decrypt(cipherText, "mytweak".getBytes());
    System.out.println(cipherText);
    System.out.println(plainText);

Expected behavior:
Should print cipher text and plain text
Actual behavior:
got a null pointer exception
Exception in thread "main" java.lang.NullPointerException
at com.idealista.fpe.config.GenericTransformations.transform(GenericTransformations.java:37)
at com.idealista.fpe.config.GenericDomain.transform(GenericDomain.java:25)
at com.idealista.fpe.FormatPreservingEncryption.encrypt(FormatPreservingEncryption.java:28)
at experiment.EncryptionTest.main(EncryptionTest.java:81)
Reproduces how often:
100%
Versions
1.0.0

The problem is it can't encrypt a number sequence. when I use letters in alphabet for the input stream I can encrypt and decrypt it without any issue.

I am also facing the same issue, any updates on this?

I have created below class and added to casspath
package com.wso2telco.dep.user.masking;

import com.idealista.fpe.config.Alphabet;

public class BasicNums implements Alphabet {

    private static final char[] NUM_CHARS = new char[] {'1', '2', '3', '4', '5', '6', '7', '8', '9' , '0'};

    @Override
    public char[] availableCharacters() {
        return NUM_CHARS ;
    }

    @Override
    public Integer radix() {
        return NUM_CHARS.length;
    }
}

and used it as in below code
BasicNums numAlphabet = new BasicNums();
        TextToIntTransformer textToIntTransformer = new GenericTransformations(numAlphabet.availableCharacters());
        IntToTextTransformer intToTextTransformer = new GenericTransformations(numAlphabet.availableCharacters());

        // with default values
        FormatPreservingEncryption formatPreservingEncryption = FormatPreservingEncryptionBuilder
                .ff1Implementation()
                .withDomain(new GenericDomain(new BasicNums(), textToIntTransformer, intToTextTransformer))
                .withDefaultPseudoRandomFunction(secretKey.getBytes())
                .withDefaultLengthRange()
                .build();

Will be possible for this encryption tool to encrypt character wise based on domain, as in my case the
string might be alphanumeric?

I also encountered this problem in the beginning, and solved it in the following way.
1, if the content to be encrypted contains only numbers, then construct NUM_CHARS like this
private static final char[] NUM_CHARS = new char[] {'1', '2', '3', '4', '5', '6', '7', '8', '9' , '0'}；
2, if the content to be encrypted contains numbers and uppercase letters, then construct NUM_CHARS
private static final char[] NUM_CHARS = new char[] {'1', '2', '3', '4', '5', '6', '7', '8','9','0','A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z'};
3, other situations are changed with steps 1 and 2
then
BasicNums numAlphabet=new BasicNums();
TextToIntTransformer textToIntTransformer = new GenericTransformations(numAlphabet.availableCharacters());
IntToTextTransformer intToTextTransformer = new GenericTransformations(numAlphabet.availableCharacters());
String secretKey = "privateKeyText";
// with default values
FormatPreservingEncryption formatPreservingEncryption = FormatPreservingEncryptionBuilder
.ff1Implementation()
.withDomain(new GenericDomain(new BasicNums(), textToIntTransformer, intToTextTransformer))
.withDefaultPseudoRandomFunction(secretKey.getBytes())
.withDefaultLengthRange()
.build();
    String cipherText = formatPreservingEncryption.encrypt("1212321ASDSAD", "2088".getBytes());
    System.out.println(cipherText);
    String plainText = formatPreservingEncryption.decrypt(cipherText, "2088".getBytes());
    System.out.println(plainText);

output:
WU69RQ3AB5IRR
1212321ASDSAD


fix: update url for v2 config

This minor update resolves #3048

Please let me know if any changes are required or if additional feedback is needed to move this forward. Thanks!

just needs some time from the maintainers :crossed_fingers:  I'm keeping on top of removing the stale label for peoples contributions :sweat_smile:


Conflict between Sentry and Ignition

Not sure if this is your issue or a Sentry specific issue, but it wasn't a problem previously with the older version.
The reported error we're seeing is Cannot assign Sentry\Laravel\Tracing\ViewEngineDecorator to property Spatie\LaravelIgnition\Views\ViewExceptionMapper::$compilerEngine of type Illuminate\View\Engines\CompilerEngine

me too, same problem

getting the same:
Cannot assign Sentry\Laravel\Tracing\ViewEngineDecorator to property Spatie\LaravelIgnition\Views\ViewExceptionMapper::$compilerEngine of type Illuminate\View\Engines\CompilerEngine

It seems like it only happens on errors from blade only
to reproduce:
laravel new app
composer require sentry/sentry-laravel

open welcome.blade.php
add the code: @dump() anywhere.
visit app.test
see:
Cannot assign Sentry\Laravel\Tracing\ViewEngineDecorator to property Spatie\LaravelIgnition\Views\ViewExceptionMapper::$compilerEngine of type Illuminate\View\Engines\CompilerEngine

test repo: https://github.com/atmonshi/sentry-ignition
live site: http://<IP_ADDRESS>

This is fixed in the latest release.

@freekmurze Similar problems persist. For example, in exceptions that occur in Livewire components.
I created a PR #42


Passive Persuasion instead of Passive Perception displayed in sheet.

Seems like an autocomplete mistake? ;-)

(Both string and value are showing "Persuasion")

Not my brightest moment. 🤦
Thanks for the report!

Fixed in 0.10.4


add custom npm registry to config

In some case, we need use company's private registry, and I think there are people who have the same same needs，so i hope add custom npm registry to config

We should add the flag in the CLI too https://github.com/saojs/sao/blob/master/bin/sao

see #91


Instant speculative retries for JAVA-1490

Allow 0ms for the constant speculative execution policy and make it schedule it immediately.

@adutra made changes mentioned, I did end up removing the manager.executor on the immediate execution like you mentioned. Even pushing to executor the execution to the others will be more or less at the "same time" depending on the connection callback, other queries on connections, what the netty threads are doing, and a fair dice roll.

@clohfink thanks looks very good 👍


Disabling replication startup jitter in Windows makefile

Overview
A similar change has already been made to *nix Makefile already.
This is for the Javascript integration test suite to not timeout
when running in Travis. We don't run Windows tests in Travis but
this should speed things a bit, and it's nice to keep both in
sync.
Jira: COUCHDB-3324
Testing recommendations
Javascript replication test suite should run faster than before this change.
Related Pull Requests
https://github.com/apache/couchdb/pull/470
Checklist
This needs manual testing from someone with access to Windows machine

@wohali Thank you for noticing this. Sorry I missed your comment in the main PR about adding it and noticed it only in reading my email later in the evening. So I made a separate PR.
But I might need your help to test this. I don't have any windows machines handy.

Drop the quotation marks since there is no space in what is being passed, and everything should be fine. +1

Makes sense. Thank you!


classic portal no longer available "data-lake-store-end-user-authenticate-using-active-directory"

"also called the client ID in the Azure classic portal" can be dropped, because classic portal no longer available and the phrase can introduce some misunderstanding

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 47ecf3a3-177e-8547-67f3-6a3dea058b14
Version Independent ID: 6c5ab6e4-d4ce-47b7-7ca8-36c01092a4be
Content: End-user authentication: Data Lake Store with Azure Active Directory
Content Source: articles/data-lake-store/data-lake-store-end-user-authenticate-using-active-directory.md
Service: data-lake-store
GitHub Login: @nitinme
Microsoft Alias: nitinme

@SychevIgor Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@SychevIgor Thanks for your feedback. This issue is now fixed in PR - https://github.com/MicrosoftDocs/azure-docs-pr/pull/40733.
@vasivara-MSFT We can now close this issue.


Fix typo in 'xterm-true-color' section

Hi,
I found that escape sequences for t_8f and t_8b in xterm-true-color section are wrong.
I fixed it.

I noticed that it's intentional. I'll close this.


OFX (v1.x) headers

Thanks for all of the work on this project.
Looking at the OFX output, it appears that the DATA:OFXSGML and ENCODING:UTF-8 headers are added. What do you think of adding other common OFX headers such as:
OFXHEADER:100
VERSION:102
CHARSET:1252
SECURITY:NONE
COMPRESSION:NONE
OLDFILEUID:NONE
NEWFILEUID:NONE

That's a good idea. Do you have a link to somewhere that documents these OFX headers as standard/common?

It appears the ofx.net site isn't responding, but the specification of the 1.02 version is available in archive.org in this zip file: https://web.archive.org/web/20170620010052/http://www.ofx.net:80/downloads/ofx1.0.2spec.zip, in the file Ofexfin1.doc.
That lists the following headers:
DATA:OFXSGML
ENCODING:
OFXHEADER:100
VERSION:102
SECURITY:
CHARSET:
COMPRESSION:
OLDFILEUID:
NEWFILEUID:

Sounds good - to clarify @reubano, there's a ENCODING header which is typically utf-8 or usascii, however, all the CHARSET examples I've seen have used 1252.

1252 implies windows-1252 though. Maybe the examples are wrong? Either that, or CHARSET doesn't mean what it seems?

It all around seems a bit of a mess 😀
# NOTE: Encoding in OFX, particularly in OFX 1.02,
# is kind of a mess.  The OFX 1.02 spec talks about "UNICODE"
# as a supported encoding, which the OFX 2.0 spec has
# back-rationalized to "UTF-8".  The "US-ASCII" encoding is
# given as "USASCII".  Yet the 1.02 spec acknowledges that
# not everyone speaks English nor uses UNICODE, so they let
# you throw any old encoding in there you'd like.

(source)
What do you think about a pair of ENCODING:UNICODE and CHARSET:UTF-8? example

Of all those, it seems like wasabe has the most sane reasoning. I think what you suggest is good (taking into account the link above). There should also be some mapping (encoding -> charset) so that if a different output encoding is selected, the appropriate headers will automagically get set.

Hey guys, I happened to stumble upon this project and this discussion.  As it happens, I've spent way too much time combing through the OFX specs while working on ofxtools.  I'd be glad to help if I can.
If I understand correctly, the scope of your work here is to generate OFX for importing into various financial management applications (e.g. GnuCash).  There are a few good reasons to prefer generating OFXv2 (i.e. XML) rather than OFXv1 (i.e. SGML) - character encodings being one of them.  If you want to generate OFXv1, you should really prefer to adhere to the OFXv1.6 spec.  As Jay notes in the ofxparse code linked above, OFXv1.0.2 has many warts that were fixed in subsequent versions.
You should always include OFX headers for OFXHEADER and VERSION, even if you're outputting OFXv1.0.2 for some reason.  You are wise to set ENCODING: UTF-8, and really you want to output the full set of OFX headers (setting the rest of them to NONE) for an easy gain in parser compliance.
You can look at ofxtools.header for some logic.
Indeed, you might be able to make all this easier to develop & maintain by just using ofxtools to generate the OFX you want... an example of the API can be found in ofxtools.Client
I find it more pleasant to generate OFX like this:
sonrs_status = STATUS(code=0, severity='INFO')
sonrs = SONRS(status=sonrs_status, dtserver=datetime.today(), language='ENG')
signon = SIGNONMSGSRSV1(sonrs=sonrs)
bankmsgs = BANKMSGSRSV1()
ofx = OFX(signonmsgsrsv1=signon, bankmsgsrsv1=bankmsgs)
header = OFXHeader(version=220)
print(str(header) + str(ofx))

P.S. relevant to issues like #17 - ofxtools has fairly extensive support for all kinds of investment transactions - that's what I mostly use it for, personally.  BUYDEBT, SELLSTOCK, EXERCISE, SPLIT, RETOFCAP, INVPOSLIST, SECINFO... it's all in there.


not fetching when no internet

flag and country list is not getting when i have no internet connection

Yes, this is waste of time. In order not to waste people's time, it should be mentioned that there will be no flag and filter data when there is no internet.


Heredocs can only be used as the last parameter in a method call

The use case is a multiline interpolated text where indents don't affect the whitespace at the beginning of each line to be used directly in a method call.
This isn't possible due to a conflict where the heredoc delimiter needing to be the only thing on the final line, but the comma to separate parameters also needs to be on the same line as the final line of the parameter. This also doesn't work if we leave a space between the delimiter and the comma.
The current alternatives are by either assigning the text outside the call, a begin/end pair, or %/%q.
This does work when it is the last parameter since the closing bracket can be on a newline, whereas the comma cannot.
Reproducible code
Carc link
def foo(foo1 = nil, foo2 = nil)
  foo1
  foo2
end

foo(<<-HEREDOC
  words
  HEREDOC,
  "test",
)

syntax error in eval:10
Error: Unterminated heredoc: can't find "HEREDOC" anywhere before the end of file

Current Undesired Solutions
Assigning outside the call.
bar = <<-HEREDOC
  words
  HEREDOC
foo(
  bar,
  "test",
)

begin/end is a bit messy, and it shouldn't be necessary.
foo(begin<<-HEREDOC
  words
  HEREDOC
  end,
  "test",
)

%/%q doesn't let us use whitespace to indent the text without also including it in the text. It also parses lines differently
foo(%(line1
line2
line3),
  "test",
)

Tested in both latest 0.35.1 and nightly as of 18th Jan

Can't do to this? This works.
foo(<<-HEREDOC, "test")
  words
  HEREDOC

Carc

Can't do to this? This works.
foo(<<-HEREDOC, "test")
  words
  HEREDOC

Carc

That works, but it really feels like it shouldn't. Also, this would only support 1 heredoc in the method.
It is also limited to having the other parameters on a single line, which I do not want for my example since there are many parameters for the method call in my use-case

That works, but it really feels like it shouldn't. Also, this would only support 1 heredoc in the method.
It is also limited to having the other parameters on a single line, which I do not want for my example since there are many parameters for the method call in my use-case

Can you show us your real code?

Can you show us your real code?

Discord::Embed.new(
    title: title,
    description: desc,
    timestamp: event.start.time,
    thumbnail: lookup_image(event.summary),
    colour: lookup_colour(event)
  )

Here is a snippet of my use-case. The description is the HEREDOC with interpolation based on variables. This happens inside some iteration nesting too. I am choosing to assign to desc beforehand in the meantime.

Discord::Embed.new(
    title: title,
    description: desc,
    timestamp: event.start.time,
    thumbnail: lookup_image(event.summary),
    colour: lookup_colour(event)
  )

Here is a snippet of my use-case. The description is the HEREDOC with interpolation based on variables. This happens inside some iteration nesting too. I am choosing to assign to desc beforehand in the meantime.

So something like this?
Discord::Embed.new(
  title: <<-HEREDOC,
  One
  Two
  HEREDOC
  description: <<-HEREDOC,
  Three
  Four
  HEREDOC
  timestamp: event.start.time,
  thumbnail: lookup_image(event.summary),
  colour: lookup_colour(event)
)

Basically heredoc is letters, numbers or underscore. Any whitespace or symbol stops the heredoc start but then regular code can follow. Next lines upto the heredoc identifier are the heredoc body. So if you need a comma you need to put it in the first line.
Other examples:
foo(a, b, <<-HEREDOC.upcase, c)
  FOO BAR
HEREDOC

bar(a, <<-HEREDOC + " world!")
  Hello
HEREDOC

By the way, this is exactly how it works in Ruby too.

So something like this?
Discord::Embed.new(
  title: <<-HEREDOC,
  One
  Two
  HEREDOC
  description: <<-HEREDOC,
  Three
  Four
  HEREDOC
  timestamp: event.start.time,
  thumbnail: lookup_image(event.summary),
  colour: lookup_colour(event)
)

Basically heredoc is letters, numbers or underscore. Any whitespace or symbol stops the heredoc start but then regular code can follow. Next lines upto the heredoc identifier are the heredoc body. So if you need a comma you need to put it in the first line.
Other examples:
foo(a, b, <<-HEREDOC.upcase, c)
  FOO BAR
HEREDOC

bar(a, <<-HEREDOC + " world!")
  Hello
HEREDOC

By the way, this is exactly how it works in Ruby too.

Ah! Thank you, I was not aware of needing to put the comma in the first line. It seems very confusing/counter-intuitive. Maybe the reference could be expanded to include an example like this

Ah! Thank you, I was not aware of needing to put the comma in the first line. It seems very confusing/counter-intuitive. Maybe the reference could be expanded to include an example like this

I agree, the reference doesn't make this clear: https://crystal-lang.org/reference/syntax_and_semantics/literals/string.html
@oprypin By the way, can the doc nav table be expanded inline? I find it a bit hard to see at a glance what's the table of contents. Previously I would expand "Syntax and semantics", then "Literals", then "String". In fact it was expanded beforehand. With the new approach I feel pretty restricted.

I agree, the reference doesn't make this clear: https://crystal-lang.org/reference/syntax_and_semantics/literals/string.html
@oprypin By the way, can the doc nav table be expanded inline? I find it a bit hard to see at a glance what's the table of contents. Previously I would expand "Syntax and semantics", then "Literals", then "String". In fact it was expanded beforehand. With the new approach I feel pretty restricted.

@asterite there are a few things that could be done. But most of them will be blocked on https://github.com/squidfunk/mkdocs-material/issues/2155 because the nav is too huge. You can thumb up that issue :D

@asterite there are a few things that could be done. But most of them will be blocked on https://github.com/squidfunk/mkdocs-material/issues/2155 because the nav is too huge. You can thumb up that issue :D

Fixed by crystal-lang/crystal-book#454

Fixed by crystal-lang/crystal-book#454


Resolves issue 555

Description
Describe the change simply. Provide a reason for the change.
Include screenshots of any new or modified screens (or at least explain why they were omitted)
This pull request is categorized as a:

[ X] New feature
[ ] Bug fix
[ ] Code refactor
[ ] Documentation
[ ] Other

Checklist

[ ] I’ve run pytest and made sure all unit tests pass before sumbitting the PR

If you modified or added functionality/workflow, did you add new unit tests?

[ ] No, I’m a fool
[X ] Yes
[ ] N/A

I have tested this PR on the following platforms/os:

[ ] Raspberry Pi OS Manual Build
[ ] SeedSigner OS on a Pi0/Pi0W board
[X ] Other

Note: Keep your changes limited in scope; if you uncover other issues or improvements along the way, ideally submit those as a separate PR. The more complicated the PR the harder to review, test, and merge.

Fixed missed import sentence on tools_screens.py

#555 link

First of all, thank you @fedebuyito !
As of https://github.com/SeedSigner/seedsigner/pull/603/commits/3c967a0e02aa1ebdcad523b556bc84755637d13a

ACK for the concept, looks cleaner to have the low end of the keyboard filling the screen space.
ACK for the screenshot generation. All the keyboard screens looking good.
ACK tested on Raspberry Pi OS Manual Build.

Pending code review regarding the KEY_BACKSPACE object.

concept ACK

Code adapted in order to resolve conflicts from last massive merge (PR https://github.com/SeedSigner/seedsigner/pull/620 )


add west europe to ACI regions

@jluk : Thanks for your contribution! The author, @mmacy, has been notified to review your proposed change.

#sign-off

@mmacy didn't realize i was on the public repo - could you do a quick sign off


Rapid Test Profile: Add extra field for gender

Feature description
Currently the Rapid Test Profile does not include a extra field where the user could enter their gender. An extra field should be added where the user can enter their gender.
Problem and motivation
When a rapid test station uses the rapid test profile to register the user, the staff from the testing station has to manually add the gender. So the motivation is to make the process more easy for testing stations.

Internal Tracking ID: EXPOSUREAPP-12174

@thomasaugsten
Will the gender field be added soon?

Thanks for your feature request @Ein-Tim. We have created an internal ticket for it and will raise this topic internally.
Internal Tracking ID: EXPOSUREAPP-12174

Corona-Warn-App Open Source Team

As the CWA project went into ramp-down mode, I don't expect this feature to be implemented. I'm therefore closing this issue.


better games counting

I think this will include games like they are on /user/games into the
count given as response to the /user, that used to decide if Games button
appeared.
NOTE: It builds and it can return 0 without an issue in my
test environment, how it works when there are games, I don't
know / need help setting up a fuller test environment some day
after the LD is over.

Do we need to hide the games button?

I'm still waiting to hear back, but this is sounding awfully familiar.
https://twitter.com/DaFluffyPotato/status/893337787394662400

i haven't managed to setup events and so i haven't managed to create games

It seems like this isn't necessary, because /node/getmy will already return you a list of games you're an author on. (refs.author). And it's not related to the tweet pov linked (which is a bug we already fixed at the very tail end of LD39)

Yeah my bad for not being more on the ball here.

Yeah I think this might have been something I solved with a 1-line fix. I seem to remember a bug that made games not show up on the game page if you didn't have any games, and doing a quick-fix change it to return some better total than what I was returning before.
Sorry again. I think this might have been the result of an early communication problem, something I should have made more clear I was dealing with.
Or if it's not, haha, we can just keep talking about this issue until we get bored. :laughing:

I was thinking about closing it, but in general to have defaults that would stress ppl of they ever show all seems like a bad idea to me


[REF-3562][REF-3563]Replace chakra usage

In this PR, we do the following:

[x] Move all chakra related integration tests to reflex-chakra repo
[x] Move the chakra_color_mode_provider.js file/logic from reflex core to reflex-chakra repo

There's still a few chakra references left over:

benchmarks/test_benchmark_compile_pages.py can we make the input to radix?
integration/test_client_storage.py
reflex/style.py just a comment referencing chakra
reflex/components/radix/themes/base.py - can we remove the chakra-specific logic here?
tests/components/test_component.py
tests/components/datadisplay/test_table.py - this one is testing the chakra table, we can probably just move it to the reflex-chakra repo
tests/components/forms/test_form.py
tests/components/media/test_icon.py (similarly can be moved)
tests/components/typography/test_markdown.py

[x] benchmarks/test_benchmark_compile_pages.py can we make the input to radix?
[ ] integration/test_client_storage.py
[ ] reflex/style.py just a comment referencing chakra
[ ] reflex/components/radix/themes/base.py - can we remove the chakra-specific logic here?
[x] tests/components/test_component.py
[x] tests/components/datadisplay/test_table.py - this one is testing the chakra table, we can probably just move it to the reflex-chakra repo
[x] tests/components/forms/test_form.py
[x] tests/components/media/test_icon.py (similarly can be moved)
[x] tests/components/typography/test_markdown.py


RunPodSandbox() fails due to network error.

Sandbox is created successfully but  RunPodSandBox()  fails with below error.
I tried separately with sample code too, same error occurs. It is reproducible with latest containerd and cri-containerd
rpc error: code = 2 desc = failed to get network status for sandbox "fb58bf5ab96b7e1197616d5406eda1a5845b39adfc322f1efbd49d6e36472ef7": Unexpected command output Device "eth0" does not exist.
 with error: exit status 1

@kunalkushwaha which cni plugin are you using ? Can you post your CNI network config file ?

If you do not have the network config in the default location prior to starting the CRI service, you will need to using the Status() to ensure the networkstatus is ready and loaded.
We moved to https://github.com/containerd/go-cni for the cni library. A Status() check is needed to load the cni config. There is no background watch being done.
So:

If cni config is added prior to starting CRI , it will be loaded.
If cni config is added after CRI is started, It will loaded on the next Status() check.

@kunalkushwaha Does this resolve your problem?

@abhi @Random-Liu below are my config files for CNI
$ cat /etc/cni/net.d/99-loopback.conf
{
        "cniVersion":"0.3.1",
        "type":"loopback"
}

~/go/src/github.com/moby/buildkit/examples/buildkit4 on  cni-network-worker! ⌚ 9:25:46
$ cat /etc/cni/net.d/crio-bridge.conf
{
    "cniVersion": "0.2.0",
    "name": "crio-bridge",
    "type": "bridge",
    "bridge": "cni0",
    "isGateway": true,
    "ipMasq": true,
    "ipam": {
        "type": "host-local",
        "subnet": "<IP_ADDRESS>/16",
        "routes": [
            { "dst": "<IP_ADDRESS>/0" }
        ]
    }
}

Also, I am using go-cni for one of my WIP PR https://github.com/moby/buildkit/pull/309 and it works fine with these CNI configuration.
Will try to debug more on this.

I see similar issue in containerd test:
E0327 02:43:26.722169    3440 remote_runtime.go:92] RunPodSandbox from runtime service failed: rpc error: code = Unknown desc = failed to setup network for sandbox "ba2e941b7dae2f7dcb17439e673a1a14147757154119c3698b5548191a78914d": failed to set bridge addr: could not add IP address to "cni0": file exists
Mar 27 02:43:26.723: INFO: Unexpected error occurred: rpc error: code = Unknown desc = failed to setup network for sandbox "ba2e941b7dae2f7dcb17439e673a1a14147757154119c3698b5548191a78914d": failed to set bridge addr: could not add IP address to "cni0": file exists

@kunalkushwaha may be you are running into this ? https://github.com/kubernetes/kubernetes/issues/25281

@Random-Liu your issue is different from @kunalkushwaha issue.
This is an error from bridge plugin. Its a netlink error. I will checkout the possibilites for bridge plugin.

Thanks ! Closing the issue.


Listener for date change

I've had a quick look at PersianDatePicker.java and have found that there is no way for a class to be notified when the date of PersianDatePicker changes. If that is the case I can send you a pull request that adds that feature.

This is very kind of you!


fix: acli output has changed, capture the output file correctly, fixes #5697

The Issue

#5697

The acli output for downloaded database has changed.
How This PR Solves The Issue
Parse the new output
Manual Testing Instructions
Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

I should have created this PR on ddev/ddev so it would be tested properly.


memory overflow

There are still more memory leak issues present. Switching from test1.json to test2.json increases memory usage by 0.1MB.

Switching from test1.json to test2.json adds 1Mb of memory.

Hey @cn17173,
Thanks for the report!!
As soon as I have time, I'll have a go at finding the problem !!
Cheers,
Gus

The following two lines of code in the function UpdateTreeFromNode in the unit LJV.Forms.Main may cause memory leaks:
treeNode^.NodeName:=TJSONObject(AJSONData).Names[index];
treeNode^.NodePath:= treeNode^.NodePath + '.' + treeNode^.NodeName;.
lt seems that assigning dynamically allocated string objects to treeNode·.NodeName without releasing the originalobjects may lead to memory leaks. However,l don't know how to modify it.
SetLength(treeNode^.NodeName, 0); SetLength(treeNode^.NodePath, 0); can solve the memory overflow problem, but it will lose the values of NodeName and NodePath.

Hey @cn17173,
I'm very, VERY grateful for your initial leg work!!!
This will, for sure, narrow my attention and will be quicker !!
Again, sorry I don't have time right now to pursue it, but it's on my todo list and it will not be forgiven!!
Cheers,
Gus


fix(style) - Refactor timeout of hiding subnav using js

Fixes https://github.com/international-labour-organization/designsystem/issues/968
Notes :-

Currently there is an odd animation when closing the subnav

Fix :-

Refactor javascript set timeout function to sync timing with the transition

Hey @Shashika6, were you able to replicate this problem in Storybook?

Will move it back to ready for review after testing in bb8


Short solution needed: "How to rename data frame column" (python-pandas)

Please help us write most modern and shortest code solution for this issue:
How to rename data frame column (technology: python-pandas)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments to make solution explained.
Link to this issue in comments of pull request.

How to rename dataframe column


length problem of faa aligner

When I use the faa aligner, an error is thrown due to the length problem. Specifically, I intend to align 1000 amrs to 1000 sentences. However, the faa aligner returns 907 alignments, causing the error. I locate the problem that happened in faa_aligner.py (about line 98).
fwd_out, fwd_err = self.fwd_align.communicate(fa_in, timeout=self.timeout)
rev_out, fwd_err = self.rev_align.communicate(fa_in, timeout=self.timeout)

I have found the reason. When an AMR only contains stopwords, it will turn into an empty string, which will cause the problem.

I have found the reason. When an AMR only contains stopwords, it will turn into an empty string, which will cause the problem.


Cannot typing on table

hi, after i upgrade flutter to version 1.26.0-17.3.pre i cannot typing anything inside table. and i try to open your documentation and cannot typing anything inside table.
this link https://weblaze.dev/pluto_grid/build/web/#feature/number-type-column
can you help me?

I notice this on the example website wasn't accepting input for editable fields and thought I was going mad!
Found it works in my dev environment Flutter 1.25.0-8.3.pre so I guess something has broken in the newer release.

oke note, so i must downgrade flutter version?

Downgrading is hopefully one workaround but obviously needs to be investigated and fixed by the Pluto devs.

ok thanks, i try to downgrade flutter first.

Since flutter's web and desktop are not yet stable versions, these problems have often been encountered.
In case of this problem you will have to wait for the problem to be resolved in the master channel.
Or, as mentioned above, you should run it on a non-issue version.

thanks, i try to downgrade flutter

@bosskmk does the web demo always use the latest version of flutter beta?  Would it be OK to simply check if the web demo is working to determine if it is safe to upgrade flutter?

Operates based on the master channel.  Integration tests should be added for each platform.

@bosskmk sorry, I mean is the web demo built using the head of the beta channel of flutter?

web demo is built on master channel.

@bosskmk OK, I'm just starting out with flutter web and still building my knowledge.  I assumed from the flutter docs that flutter web is only available in the beta channel.
Thanks.

Yes, it can also be used in develop and master channels, which are the latest channels than beta channels.
For reference, I plan to change my development channel to beta when the desktop is supported in beta.

when i switched to master channel i can type everything in table, but i can't move my focus column by keyboard even i press enter to change my value.

@BangPei
I had the same problem on the master channel last time, but it was resolved after a while.
I don't know if this is the same problem or if the way keyboard events are handled has changed overall.
I think I could solve it myself, but I hope that the flutter version will be changed and it will be solved automatically.

I'm in a situation where another package I'm using has issues in the beta channel.  I tried upgrading to master branch and the charts package now works but I have the input issue with pluto grid.
Weirdly if I open developer tools the issue goes away and I can edit cells again .

i have same problem, i cannot edit cell when typing something, on web i meet same problem. hope you can fix it soon

Hi, I have the same problem on flutter 2 (build for web). I cannot make any filter on the table because it does not accept any key typing. I've tested it on chrome for pc.


0.4.0

Allows clients to pass exec a custom event name to emit stdout on.  Event name will default to 'stdout' if the event name is left null.
Will require a small change to how existing clients call exec, so I've linked to a (yet to be written) upgrade guide at the top of the README as well.

And here's the upgrade guide: https://github.com/500px/gunter/wiki/Upgrading-from-0.3.x-to-0.4.x
@pwyliu CR?

Cool, I'll do some functional testing for this and the other one from yesterday.

Poke poke can this be moved forward?

Note that I've rolled in the changes from #19 since they ended up being breaking.


Universal Windows Platform Compatible

@BlueRaja is it much effort to make you NuGet package compatible with the UWP?  I have considered forking your source and taking what I need to make my project work but I figured that might be duplicate effort if this might have already been on your horizon.  Just thought I would ask, thanks for your   ##time!
In Christ,
Rock

Is it currently not compatible?  I'm not aware of any API changes to anything that this project uses.

@BlueRaja,
As a side not I have your code running just fine under a UWP project without modification.  The only work required was to create the new project and move the source to a Shared folder so that both projects can access the files intuitively.  You can observe my changes here: rockthethird/High-Speed-Priority-Queue-for-C-Sharp.
I have no experience with NuGet submissions so I don't want to imply that the required changes would be trivial. My intention is to simply inform you that you have done well in making the code itself compatible with many .NET versions.  And that just maybe the community can benefit from the future proofing of your repository.

@BlueRaja,
To answer your question directly, the NuGet is currently not installable through NuGet.  As for its compatibility, I believe the code would work as is.  My opinion stems from my previous comment which states the fact that my only change required was the need for adding the source files to specifically to a UWP project.
(Sorry my reply was already typed out before and sent before I fully understood you comment).

I have no idea how to make this library compatible with UWP (there's absolutely no reason it wouldn't be already).  I tried playing with it for a few hours today but didn't get anywhere.
I've asked this question on StackOverflow for help, but it doesn't seem to be getting much attention.  I'll put a bounty on it in a few days to see if that helps.

Thanks for giving so much attention to the matter. I should have time to
look into this next week and will provide my findings.
In Christ,
Rock
On Sat, Apr 16, 2016 at 6:57 PM BlueRaja<EMAIL_ADDRESS>wrote:

I have no idea how to make this library compatible with UWP (there's
absolutely no reason it wouldn't be already). I tried playing with it
for a few hours today but didn't get anywhere.
I've asked this question
http://stackoverflow.com/questions/36670422/how-do-i-make-an-existing-c-sharp-library-uwp-compatible
on StackOverflow for help, but it doesn't seem to be getting much
attention. I'll put a bounty on it in a few days to see if that helps.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/BlueRaja/High-Speed-Priority-Queue-for-C-Sharp/issues/4#issuecomment-210920827

--
(sent from my phone)

I put a 200 rep bounty on it, but it didn't draw any extra attention :( Any luck on your end?

Sorry I have been so swamped with work.  I will hopefully be able to check things out soon.

After looking into your posting a little more it appears that you must add another project that targets the UWP.  After that the NuGet packaging magic should just work as described by a couple commenters.  If it is any consolation your code seems to be fully UWP compatible and require no modification.  From my ability to simply import your code to a UWP, my suggestion would be to link all source files to the new UWP project in your solution rather than allowing the Add->Existing Item dialog to copy the files.
In Christ,
Rock

Completed in #13


Revised Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Thank you! I will submit the A2 and A3 tomorrow. I'm having some
difficulties pushing it to the right branch. I have cloned too many
repositories with the updates.
On Mon, Jun 17, 2024 at 7:36 PM amanda-ng518 @.***>
wrote:

@.**** approved this pull request.
A1: good
—
Reply to this email directly, view it on GitHub
https://github.com/ValeriaGuima/applying_statistical_concepts/pull/3#pullrequestreview-2124077807,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BH5ZYGBTJ3BY4UH24OGFZJTZH5XHRAVCNFSM6AAAAABJNEXMHOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMRUGA3TOOBQG4
.
You are receiving this because you authored the thread.Message ID:
<ValeriaGuima/applying_statistical_concepts/pull/3/review/2124077807@
github.com>


Spell checking on long documents

I've noticed that MDE stops spell-checking after about one page of text (if the document is long; "long" in my case is ~1500 words which is about 5 pages when pasted into MS Word). Is that a technical limitation of the editor component MDE uses? It is quite unpleasant and I'd happily sacrifice some of the raw power of my PC for full spell-checking.

Actually, I was wrong. It seems that the spell checking depends on context – in my case, the spell checking doesn't work in a nested list:

I have a theory: there's an option Ignore Code Blocks and this paragraph is indented by four spaces (to create a nested list item). Maybe this confuses MDE?


Make our AJAX app crawlable

For background on this issue see:
https://developers.google.com/webmasters/ajax-crawling/docs/learn-more
This may be a challenge, but if our app is not crawlable then our sites will be hidden from much of the web. The suggestions in the google link do not provide much comfort. We need to evaluate the issue and propose some solutions first.
Not sure who wants to take this on: @svoinea @howdyjessie @crisaless

@karlcz adding karl to the discussion because we may need to think broadly about this issue (i.e., it may not just be a client-side chaise issue).

This might not be an issue according to this link, which says that Googlebot should be able to render the full page.

That could be good news. We should look into Google’s Webmaster Tools to understand how Google renders our app.
http://googlewebmastercentral.blogspot.no/2014/05/understanding-web-pages-better.html

If I search google with stuff found on the default data browser page
of results, it serves it as the first result:
facebase investigator steve potter
However, I don't think we're following linked-data practices very well
so the crawler has nowhere to go after that page of 25 results. I
don't think a crawler will press arbitrary buttons in a UI.  I think
that the page needs links that can be followed to reach related
search result views, page through results, etc.
karl

This is related to #1 since we need to have individual document URLs that show meaningful pages of stuff that the search engine can index.
We also need to think about how to present many such URLs so they are discoverable by the crawler, e.g. in a site-map and/or as related content links embedded in each data browser screen.

@hongsudt this is important for driving traffic to facebase, but it might have to move to wishlist.

We can have a student do this, I think. Assuming this information is still current (and we could find out at google's instructions for web masters):
http://blog.woorank.com/2014/05/how-to-create-a-robots-txt-file-with-sitemap-location/
Essentially we just need a student to create a sitemap app that dynamically generates a sitemap when a crawler hits it. I suppose this might be more efficient if ERMrest did it so that it was served up only when it changes.

Based on our most recent conversation, we have decided that having a dynamically updated sitemap is not necessary and maybe not possible anyway. What we need is to generate a static site map from a script that can be rerun on some schedule and/or manually. This will probably be a 10 line bash script. It should take the following parameters:

URL: the URL of the ermrest catalog, e.g., https://www.facebase.org/ermrest/catalog/1
schemaName: the name of the schema to target, e.g., legacy
tableName: the name of the table to target, e.g., dataset
keyColumn: the name of the column to retrieve data from, e.g., id

We suggest using the EMRrest ATTRIBUTE interface and request the CSV content type for just the key column of the table. We also think we can start by assuming the table has a single key column (not a composite key made of multiple columns).

The script for generating a sitemap has been added to the chaise application inside the sitemap/ folder at the root of the project. There is a readme in there explaining the nuances of the script and how to apply that to other applications in the future.
The next step would be to take the script and add it to our recipe for the chaise application and generating the robots.txt file in that recipe.

Let's take the defaults out. Actually that forces this script to only generate a sitemap for facebase. So this line:
https://github.com/informatics-isi-edu/chaise/blob/master/sitemap/generate-sitemap.sh#L19
should be replaced with:
record_urls ${1} ${2} ${3} ${4} ${5} ${6}

leave the > sitemap.txt off and let the caller simply redirect.
For the readme, similarly remove the note on defaults and replace with and example of using the script in a code block.

I modified the script to take in 6 arguments per @robes's suggestion. When running the script, the user needs to specify the file to output stdout to. The readme has been updated to reflect this as well.

i made one minor fix, to include the trailing / here ...record/#...


chapter_recurrent-modern/deep-rnn

Job d2l-zh/PR-731/1 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-731/

Job d2l-zh/PR-731/4 is complete.
Check the results at http://preview.d2l.ai/d2l-zh/PR-731/


Better support for tf-keras and Keras 3 models

Follow-up PR after Keras <> Hugging Face integration (https://github.com/keras-team/keras/pull/19854)
This PR:

Defines a "new" library tf-keras which correspond to the legacy Keras 2.x. TF-Keras is the official naming and has a separate Github repo here.
Moves the previous keras snippet to tf-keras (the legacy huggingface_hub.from_pretrained_keras)
Moves the previous keras download count rule to tf-keras (counting "saved_model.pb")
Defines a new keras snippet using keras.saving.load_model
Defines a new keras download count rule using "config.json"
tf-keras will not be listed as a library in https://huggingface.co/models. => we should only promote keras with the proper integration

At the moment, models are automatically tagged as keras by moon-landing when a saved_model.pb file is detected. I will open a separate PR to tag them as tf-keras instead. New models that will be pushed to the Hub using the new Keras<>HF integration will automatically be tagged as keras in the model card.
EDIT: I opened https://github.com/huggingface/moon-landing/pull/10501 (private) in moon-landing to correctly detect tf-keras models.

Thanks everyone for the reviews and comments! I addressed them all. Could you have a look at it before merging? Once that's done, we should be able to move forward on the moon-landing part.
In parallel I'm updating docs + api inference repos.

Thanks for the reviews! Let's get this merged :)


Change Scholarship page to use Submission directive

The scholarship page was copied and refactored into the submission directive for a few other pages.  Now these changes should be pulled into the scholarship page so that all the submission pages can run using the same code.

Completed in e3488910c4be664b89b7b1184b080a52a00f787b


fix: skip adding @SerialName when it's already there

Hello and thank you for the lib! This is to solve the following issue:

As soon as we start using @SerialName, like downbelow for example:

@JsonApiX(type = "address")
@Serializable
data class Address(
    @SerialName("street") val streetName: String,
    val number: Int,
    val country: String,
    val city: String
) : JsonApiModel()

Our code gen looks like this (please notice annotation doubling) :

@Serializable
@SerialName(value = "Attributes_address")
public data class Attributes_Address(
  @SerialName(value = "city")
  public val city: String,
  @SerialName(value = "country")
  public val country: String,
  @SerialName(value = "number")
  public val number: Int,
  @property:SerialName(value = "street")
  @SerialName(value = "streetName")
  public val streetName: String
) : Attributes {
  public companion object {
    public fun fromOriginalObject(originalObject: Address) = Attributes_Address(city =
        originalObject.city, country = originalObject.country, number = originalObject.number,
        streetName = originalObject.streetName)
  }
}

With the fix, the code generated will look like this:
@Serializable
@SerialName(value = "Attributes_address")
public data class Attributes_Address(
  @SerialName(value = "city")
  public val city: String,
  @SerialName(value = "country")
  public val country: String,
  @SerialName(value = "number")
  public val number: Int,
  @property:SerialName(value = "street")
  public val streetName: String
) : Attributes {
  public companion object {
    public fun fromOriginalObject(originalObject: Address) = Attributes_Address(city =
        originalObject.city, country = originalObject.country, number = originalObject.number,
        streetName = originalObject.streetName)
  }
}

I understand it's not perfect solution, but it works and I'm lazy :)

Hi! Thanks for the lib! Ran into the same issue, would be nice to have it asap.


Demo notebook for updated metrics functions

Demonstrating updated functionality for metrics functions in xgcm

set_metrics()
interp_like()
get_metric()

I am not sure what is going on. My best guess for now. The first notebook failure is some temporary glitch (https error) and the mld notebook failure might be due to the install command you still had in the notebook? If fixing that doesnt help, we will need to dig deeper into what has changed on the docker image recentyl

Hmm I'm not sure if this affects anything, but to update the notebook I run it in Pangeo Cloud, then download it to my computer so I can then push changes to this repository. Is there a way to push directly from Pangeo Cloud?

Oh yeah for sure. You can open a terminal on there:

Just click on the big blue + button up top and choose Terminal on the lower left.

This is looking great. I really liked how you show the big result up front (grid.average(uo) works) and dive in to all the effort it took to get it to work.
Some higher level comments:

I think you should show 1 or maybe 2 years of data so people can see patterns.
I think you need a simple 4-cell version of this figure with T-points and u-points to make it really clear to the audience what needs to happen. you could illustrate what "averaging" or "interp"-ing does using a version of the 4-cell image too

The "setup" (reading, subsetting, plotting) is a little long. While presenting I think you may want to do this quickly mostly highlighting "this is a regional subset of the global model and I want to analyze it".

Do you mean a graphic of 4 boxes with the dot in the middle for temperature, and arrows at the sides for u-velocity? Will this be embedded in the notebook as well?

Yes. You can highlight a "u cell" and a "T cell" and show that areas are potentially different.


Add Virtual Machine tasks

Add tasks to install virtual machine items (via deb-get possibly as well as apt)

added virtual machines task and tested