End of audio being truncated with 22kHz sample rate on portaudio

When I play 32-bit integer 22kHz audio, via portaudio and a custom .asoundrc, the end of the sound is truncated. It seems to be okay for 44.1kHz and 48kHz.
The reason for the selection of 32-bit integer samples (rather than 16-bit integer) was to get resampling to work with portaudio, otherwise I'd get the following errors (line numbers will be slightly off, as I've sprinkled in additional debugging code):
Expression 'framesAvail' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 3645
Host error description: Unknown error
Expression 'PaAlsaStreamComponent_GetAvailableFrames( self, &framesAvail, xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 3994
Expression 'PaAlsaStreamComponent_RegisterChannels( &self->playback, &self->bufferProcessor, &playbackFrames, &xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 4124
Expression 'PaAlsaStream_SetUpBuffers( stream, &framesGot, &xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 4364
CallbackThreadFunc: Thread -1381649952 is canceled due to error -9999

Likewise, the rate plugin was needed (and then plug to get around an error playing mono sounds, and then route to handle mono sounds in the wrong speaker), so my .asoundrc got a little complicated:
pcm.bluetooth {
	type route
	ttable.0.0 0.5
	ttable.0.1 0.5
	ttable.1.0 0.5
	ttable.1.1 0.5
	slave.pcm {
		type rate
		converter "speexrate"
		slave {
			rate 48000
			pcm {
				type plug
				slave.pcm {
					type bluealsa
					device "6B:F4:54:06:AC:D1"
					profile "a2dp"
				}
			}
		}
	}
	hint {
		show on
		description "Bluetooth (bluez-alsa)"
	}
}

It seems possible that the myriad plugins I have to use to get this to work might be contributing to the problem.
I saw #67, however in my case, aplay works fine... probably because it's not being routed through portaudio. Based on that, I tried increasing ba-transport.c usleep(200000) to usleep(1000000), but it didn't seem to make a difference.
Please note that playing to a non-bluez-alsa device works fine in portaudio and I don't need a custom .asoundrc or sample size for those. I also tried playing via portaudio with a similar .asoundrc to a hw device, and that didn't truncate.
Any ideas on this would be very much appreciated!

commit 2cd6e46 fixes an issue bluealsa had with repeatedly opening/closing/reopening a pcm device. As portaudio uses this technique to test the capabilities of a device before selecting it, it is possible that this commit may also improve the behaviour of bluealsa with portaudio applications.
I'm not sure how much of your complex configuration is just to overcome issues, but it would be great if you could test the latest bluealsa code with a default configuration and with the config above to compare results with the bluealsa code version you have used so far.

Thanks! I've gone ahead and given that commit a try, but I'm still unable to play audio with a non-crazy .asoundrc. Here are some details if you'd like to try to recreate:
PortAudio example program (modified by me to try out different sample rates and formats, but it should do everything the original does and more, although I'm not sure I got mono playback correct):
https://pastebin.com/CHtV2gWG
Just in case, here's the original PortAudio example (NOTE: works because it's using 32-bit floats... why that matters remains a mystery to me. To run this one you'd need to modify my OP .asoundrc to include !default):
https://github.com/EddieRingle/portaudio/blob/master/examples/paex_sine.c
Compile:
gcc -W -Wall -o paex_sine_ext paex_sine_ext.c -lm -lportaudio

.asoundrc reverted to what I hope is standard:
pcm.bluetooth {
	type plug
	slave.pcm {
		type bluealsa
		device "E9:08:EF:2D:40:33"
		profile "a2dp"
	}
	hint {
		show on
		description "Bluetooth (bluez-alsa)"
	}
}

Run (NOTE: 2 happens to be the bluetooth device for me, it might be different for you. i16 has always failed for me with a non-48000 sample rate, but i32 or f32 should work with my OP .asoundrc):
pa/paex_sine_ext 2 i16 stereo 5 44100 64

paex_sine_ext output (NOTE: I've added a bit of debug code to PortAudio to look into the framesAvail error, trivial patch at https://pastebin.com/U2ZDWVBh, unfortunately it didn't help me understand what went wrong):
PortAudio Test: output sine wave. f = 8, stereo, NS = 5, SR = 44100, BufSize = 64
before paHostApiInitializers[0].
ALSA version (build): 1.1.8
ALSA version (runtime): 1.1.8
BuildDeviceList: Ignoring ALSA plugin device [cards] of type [unknown]
BuildDeviceList: Found plugin [default] of type [unknown]
BuildDeviceList: Found plugin [sysdefault] of type [unknown]
BuildDeviceList: Found plugin [front] of type [unknown]
BuildDeviceList: Found plugin [rear] of type [unknown]
BuildDeviceList: Found plugin [center_lfe] of type [unknown]
BuildDeviceList: Found plugin [side] of type [unknown]
BuildDeviceList: Found plugin [surround21] of type [unknown]
BuildDeviceList: Found plugin [surround40] of type [unknown]
BuildDeviceList: Found plugin [surround41] of type [unknown]
BuildDeviceList: Found plugin [surround50] of type [unknown]
BuildDeviceList: Found plugin [surround51] of type [unknown]
BuildDeviceList: Found plugin [surround71] of type [unknown]
BuildDeviceList: Found plugin [iec958] of type [unknown]
BuildDeviceList: Found plugin [spdif] of type [unknown]
BuildDeviceList: Found plugin [hdmi] of type [unknown]
BuildDeviceList: Found plugin [dmix] of type [unknown]
BuildDeviceList: Ignoring ALSA plugin device [dsnoop] of type [unknown]
BuildDeviceList: Found plugin [modem] of type [unknown]
BuildDeviceList: Found plugin [phoneline] of type [unknown]
BuildDeviceList: Ignoring ALSA plugin device [hw] of type [hw]
BuildDeviceList: Ignoring ALSA plugin device [plughw] of type [plug]
BuildDeviceList: Ignoring ALSA plugin device [plug] of type [plug]
BuildDeviceList: Ignoring ALSA plugin device [shm] of type [shm]
BuildDeviceList: Ignoring ALSA plugin device [tee] of type [file]
BuildDeviceList: Ignoring ALSA plugin device [file] of type [file]
BuildDeviceList: Ignoring ALSA plugin device [null] of type [null]
BuildDeviceList: Found plugin [bluealsa] of type [plug]
BuildDeviceList: Found plugin [bluetooth] of type [plug]
BuildDeviceList: Filling device info for 21 devices
FillInDevInfo: Filling device info for: Artik-I2S-MAX98090: - (hw:0,0)
GropeDevice: collecting info ..
GropeDevice: collecting info ..
Default input device: Artik-I2S-MAX98090: - (hw:0,0)
Default output device: Artik-I2S-MAX98090: - (hw:0,0)
FillInDevInfo: Adding device Artik-I2S-MAX98090: - (hw:0,0): 0
FillInDevInfo: Filling device info for: sysdefault
GropeDevice: collecting info ..
GropeDevice: Limiting number of plugin channels to 128
GropeDevice: collecting info ..
GropeDevice: Limiting number of plugin channels to 128
FillInDevInfo: Adding device sysdefault: 1
FillInDevInfo: Filling device info for: front
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.front
OpenPcm: Opened device 'front' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: front, all channels == 0
FillInDevInfo: Filling device info for: rear
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
OpenPcm: Opened device 'rear' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: rear, all channels == 0
FillInDevInfo: Filling device info for: center_lfe
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
OpenPcm: Opened device 'center_lfe' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: center_lfe, all channels == 0
FillInDevInfo: Filling device info for: side
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
OpenPcm: Opened device 'side' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: side, all channels == 0
FillInDevInfo: Filling device info for: surround21
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround21
OpenPcm: Opened device 'surround21' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround21
OpenPcm: Opened device 'surround21' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround21, all channels == 0
FillInDevInfo: Filling device info for: surround40
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround40
OpenPcm: Opened device 'surround40' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround40, all channels == 0
FillInDevInfo: Filling device info for: surround41
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround41
OpenPcm: Opened device 'surround41' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround41, all channels == 0
FillInDevInfo: Filling device info for: surround50
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround50
OpenPcm: Opened device 'surround50' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround50, all channels == 0
FillInDevInfo: Filling device info for: surround51
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround51
OpenPcm: Opened device 'surround51' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround51, all channels == 0
FillInDevInfo: Filling device info for: surround71
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.surround71
OpenPcm: Opened device 'surround71' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: surround71, all channels == 0
FillInDevInfo: Filling device info for: iec958
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958
OpenPcm: Opened device 'iec958' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: iec958, all channels == 0
FillInDevInfo: Filling device info for: spdif
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958
OpenPcm: Opened device 'spdif' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.iec958
OpenPcm: Opened device 'spdif' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: spdif, all channels == 0
FillInDevInfo: Filling device info for: hdmi
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
OpenPcm: Opened device 'hdmi' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
OpenPcm: Opened device 'hdmi' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: hdmi, all channels == 0
FillInDevInfo: Filling device info for: modem
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
OpenPcm: Opened device 'modem' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
OpenPcm: Opened device 'modem' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: modem, all channels == 0
FillInDevInfo: Filling device info for: phoneline
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
OpenPcm: Opened device 'phoneline' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
OpenPcm: Opened device 'phoneline' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: phoneline, all channels == 0
FillInDevInfo: Filling device info for: bluealsa
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bluealsa.device'
ALSA lib conf.c:4568:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5036:(snd_config_expand) Args evaluate error: No such file or directory
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM bluealsa
OpenPcm: Opened device 'bluealsa' ptr[(nil)] - result: [-2:No such file or directory]
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bluealsa.device'
ALSA lib conf.c:4568:(_snd_config_evaluate) function snd_func_refer returned error: No such file or directory
ALSA lib conf.c:5036:(snd_config_expand) Args evaluate error: No such file or directory
ALSA lib pcm.c:2565:(snd_pcm_open_noupdate) Unknown PCM bluealsa
OpenPcm: Opened device 'bluealsa' ptr[(nil)] - result: [-2:No such file or directory]
FillInDevInfo: Skipped device: bluealsa, all channels == 0
FillInDevInfo: Filling device info for: bluetooth
ALSA lib bluealsa-pcm.c:718:(_snd_pcm_bluealsa_open) Couldn't get BlueALSA PCM: PCM not found
OpenPcm: Opened device 'bluetooth' ptr[(nil)] - result: [-19:No such device]
bluealsa-pcm.c:589: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Setting constraints
GropeDevice: collecting info ..
GropeDevice: Limiting number of plugin channels to 128
bluealsa-pcm.c:309: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Closing
FillInDevInfo: Adding device bluetooth: 2
FillInDevInfo: Filling device info for: default
GropeDevice: collecting info ..
GropeDevice: Limiting number of plugin channels to 128
GropeDevice: collecting info ..
GropeDevice: Limiting number of plugin channels to 128
Default input device: default
Default output device: default
FillInDevInfo: Adding device default: 3
FillInDevInfo: Filling device info for: dmix
GropeDevice: collecting info ..
FillInDevInfo: Adding device dmix: 4
BuildDeviceList: Building device list took 0.128747 seconds
after paHostApiInitializers[0].
PaAlsaStreamComponent_Initialize: Host Chans P 2
AlsaOpen: Opening device bluetooth
bluealsa-pcm.c:589: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Setting constraints
PaAlsaStreamComponent_InitialConfigure: device MMAP SND_PCM_ACCESS_MMAP_INTERLEAVED: YES
PaAlsaStreamComponent_InitialConfigure: device MMAP SND_PCM_ACCESS_MMAP_NONINTERLEAVED: YES
PaAlsaStreamComponent_InitialConfigure: device can MMAP: YES
PaAlsaStreamComponent_DetermineFramesPerBuffer: user-buffer (frames)           = 64
PaAlsaStreamComponent_DetermineFramesPerBuffer: user-buffer (sec)              = 0.001451
PaAlsaStreamComponent_DetermineFramesPerBuffer: suggested latency (sec)        = 0.190000
PaAlsaStreamComponent_DetermineFramesPerBuffer: suggested host buffer (frames) = 8443
PaAlsaStreamComponent_DetermineFramesPerBuffer: suggested host buffer (sec)    = 0.191451
PaAlsaStreamComponent_DetermineFramesPerBuffer: periods min = 3, max = 8738, req = 4
PaAlsaStreamComponent_DetermineFramesPerBuffer: suggested host buffer period   = 2112
PaAlsaStreamComponent_DetermineFramesPerBuffer: device period minimum          = 441
PaAlsaStreamComponent_DetermineFramesPerBuffer: device period maximum          = 3764
PaAlsaStreamComponent_DetermineFramesPerBuffer: host buffer period             = 2112
PaAlsaStreamComponent_DetermineFramesPerBuffer: host buffer period latency     = 0.047891
bluealsa-pcm.c:319: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Initializing HW
bluealsa-pcm.c:346: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: FIFO buffer size: 4096
bluealsa-pcm.c:352: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Selected HW buffer: 5 periods x 9192 bytes <= 45976 bytes
bluealsa-pcm.c:367: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Initializing SW
bluealsa-pcm.c:367: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Initializing SW
bluealsa-pcm.c:367: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Initializing SW
bluealsa-pcm.c:389: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Prepared
bluealsa-pcm.c:367: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Initializing SW
PaAlsaStream_Configure: Playback period size: 2112, latency: 0.191565
OpenStream: Stream: framesPerBuffer = 64, maxFramesPerHostBuffer = 2112, latency i=0.000000, o=0.191565
bluealsa-pcm.c:389: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Prepared
bluealsa-pcm.c:240: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Starting
bluealsa-pcm.c:134: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Starting IO loop: 7
PaUnixThread_New: Waited for 0.0288379 seconds for stream to start
Play for 5 seconds.
src/hostapi/alsa/pa_linux_alsa.c 3719 PaAlsaStream_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3719 PaAlsaStream_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3971 PaAlsaStreamComponent_RegisterChannels
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3719 PaAlsaStream_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3971 PaAlsaStreamComponent_RegisterChannels
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3971 PaAlsaStreamComponent_RegisterChannels
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
src/hostapi/alsa/pa_linux_alsa.c 3971 PaAlsaStreamComponent_RegisterChannels
src/hostapi/alsa/pa_linux_alsa.c 3611 PaAlsaStreamComponent_GetAvailableFrames
Expression 'framesAvail' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 3622
Host error description: Unknown error
Expression 'PaAlsaStreamComponent_GetAvailableFrames( self, &framesAvail, xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 3971
Expression 'PaAlsaStreamComponent_RegisterChannels( &self->playback, &self->bufferProcessor, &playbackFrames, &xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 4101
Expression 'PaAlsaStream_SetUpBuffers( stream, &framesGot, &xrun )' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 4341
CallbackThreadFunc: Thread -1960680992 is canceled due to error -9999
 OnExit: Stopping ALSA handles
bluealsa-pcm.c:280: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Stopping
bluealsa-pcm.c:104: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: IO thread cleanup
AlsaStop: Dropped frames
OnExit: Stoppage
Stream Completed: No Message
CallbackThreadFunc: Thread -1960680992 exiting
 Stopping callback
PaUnixThread_Terminate: Joining thread -1960680992
Callback thread returned: -9999
bluealsa-pcm.c:280: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Stopping
bluealsa-pcm.c:359: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Freeing HW
bluealsa-pcm.c:309: /org/bluealsa/hci0/dev_E9_08_EF_2D_40_33/a2dp: Closing
TerminateHostApis in
TerminateHostApis out
Test finished.

bluealsa output during run:
/usr/bin/bluealsa: bluealsa-dbus.c:328: Called: org.bluealsa.PCM1.Open()
/usr/bin/bluealsa: ba-transport.c:722: New transport: 20 (MTU: R:672 W:895)
/usr/bin/bluealsa: bluez.c:1330: Signal: org.freedesktop.DBus.Properties.PropertiesChanged()
/usr/bin/bluealsa: bluez.c:1364: Signal: PropertiesChanged: org.bluez.MediaTransport1: State
/usr/bin/bluealsa: ba-transport.c:601: State transition: 0 -> 2
/usr/bin/bluealsa: io.c:2576: Created new IO thread: A2DP Source (SBC)
/usr/bin/bluealsa: io.c:469: Starting IO loop: A2DP Source (SBC)
/usr/bin/bluealsa: ba-transport.c:601: State transition: 2 -> 2
/usr/bin/bluealsa: io.c:126: PCM read buffer flushed: 1512
/usr/bin/bluealsa: io.c:126: PCM read buffer flushed: 0
/usr/bin/bluealsa: io.c:111: PCM has been closed: 16
/usr/bin/bluealsa: ba-transport.c:882: Closing PCM: 16
/usr/bin/bluealsa: ba-transport.c:882: Closing PCM: 16
/usr/bin/bluealsa: io.c:520: Keep-alive polling: 0
/usr/bin/bluealsa: io.c:520: Keep-alive polling: 0
/usr/bin/bluealsa: ba-transport.c:749: Releasing transport: A2DP Source (SBC)
/usr/bin/bluealsa: ba-transport.c:778: Closing BT: 20
/usr/bin/bluealsa: ba-transport.c:928: Exiting IO thread: A2DP Source (SBC)
/usr/bin/bluealsa: bluez.c:1330: Signal: org.freedesktop.DBus.Properties.PropertiesChanged()
/usr/bin/bluealsa: bluez.c:1364: Signal: PropertiesChanged: org.bluez.MediaTransport1: State
/usr/bin/bluealsa: ba-transport.c:601: State transition: 2 -> 0

I don't get any sound at all when using your paex_sine_ext program.
I have some sample mp3 files with sample rates of 22.05kHz, 44.1kHz, and 48kHz, and sample formats of s16le, s32le, f32le. I tested them all with mpg123-portaudio (which is the only portaudio application I have available right now) and they all appear to work fine with a default install of bluealsa. So I'm not convinced this is a bluealsa issue.
The thing is, bluealsa does not do any pcm format conversions - the a2dp link will always be stereo s16le (defined by the a2dp spec) and the rate is nearly always 48kHz (although could possibly be 44.1kHz, but the application does not get to choose, its fixed when the bluetooth connection is made). The bluealsa plugin will only accept this one format  when the application sets the hardware params. As a convenience, a default bluealsa install also defines an alsa pcm device called "bluealsa" which uses the alsa "plug" plugin to do any necessary conversions. If you have a file which is not converted correctly when playing to this device, then does that not suggest it is the converter which is failing rather than the bluealsa code?

Thanks for taking a look! Is there any way to use the bluealsa pcm device and its conversions via .asoundrc? The reason I need that is to be able to specify the bluetooth address. Should my .asoundrc be able to use the bluealsa device resampling abilities, or is it flawed in some way?

I'm no expert in alsa configuration, but your .asoundrc looks ok but rather over-complicated as far as I can tell. You wrap the bluealsa plugin within a "plug" plugin which does the automatic channels/format/rate conversion for you; but you then wrap that in "speexrate" and "route" which seems rather redundant? Have a look at the section "Automatic Conversion Plugin" in https://www.alsa-project.org/alsa-doc/alsa-lib/pcm_plugins.html - that shows how to use the "plug" plugin to do the required conversions.
My concern is that this complex setup all resulted from a perceived failure of port audio to handle 16-bit samples at 22kHz. Yet I can see no reason why port audio should fail there. Is the failure only when using bluealsa, or is it with all devices? If all devices, then I would concentrate on solving it with your on-board card before adding the extra complexity of third-party plugins such as bluealsa.

My apologies, I was referring to the reverted .asoundrc from this comment, was it correct?
https://github.com/Arkq/bluez-alsa/issues/243#issuecomment-531912301
Audio plays fine with the internal non-bluealsa devices via PortAudio.
FWIW, I refactored my application to play directly to ALSA and it plays 16-bit audio fine now, at all sample rates. Go ahead and close if you want, although I do believe there is something weird happening between PortAudio and bluealsa, perhaps in the PortAudio code? Hard to say.

@killerkalamari - if your application works fine with alsa hw devices and not bluealsa, then clearly I was wrong - this is a bluealsa issue.
I'm not the bluealsa developer, just someone interested in understanding issues with it for my own benefit. I'm afraid that at present, I can't see where or how bluealsa can cause the "framesAvail" error you are seeing. It may need someone with knowledge of portaudio internals to explain that.
So I think this issue should be left open because, as you say, there is possibly some unexpected interaction between bluealsa and portaudio that might well impact other applications.

This issue has been idle for over 18 months now, Given the significant changes in bluez-alsa in that time, and the fact that I am unable to reproduce the issue, I am closing.
If you experience problems using the latest bluez-alsa code with portaudio please open a new issue, including error/debug logs from bluealsa if possible.


s2n-quic 1.45 is semver breaking

The s2n_quic::application::Error type implements From<VarInt> from s2n-quic-core. In 1.44, VarInt was from version 0.44 of s2n_quic_core. In 1.45, VarInt is from the semver incompatible s2n_quic_core 0.44 resulting in a breaking change for any users of the public From<VarInt> impl.
The only correct fix is to provide an implementation of From<VarInt> for the 0.44 version of VarInt as well. Turning back time, VarInt should either be hidden entirely from public APIs or reexported from the s2n-quic crate.

We don't currently have any guarantees around breaking changes for our 0.x crate, in this case s2n-quic-core. Only types accessible by the s2n-quic crate are considered for semver.
Is there a reason you're needing to depend on s2n-quic-core?

This a breaking change in s2n_quic, not s2n_quic_core, though it is precipitated by a change to the latter. The use of core occurs through the h3 integration.

Our s2n-quic-h3 crate should not have depended on s2n-quic-core, given the recommendation from the readme:

This is an internal crate used by s2n-quic. The API is not currently stable and should not be used directly.

I've gone ahead and removed that dependency in #2325, and would suggest you make a similar change to your code.


MTR-46: create client with passed id

Ebert has finished reviewing this Pull Request and has found:

1 possible new issue (including those that may have been commented here).

You can see more details about this review at https://ebertapp.io/github/Nebo15/mithril.api/pulls/53.

Changes Unknown when pulling e8b945dd021474876a01c4ee31de96fd3f385512 on MTR-46 into ** on master**.


index 400000 is out of bounds

When I ran the script on glove.6B.300d.txt, I got IndexError that index 400000 is out of bounds for axis 0 with size 400000. Adding 1 to num_lines seems to fix the problem.
num_lines = check_num_lines_in_glove(glove_file) + 1

Hi,
Thanks for raising an issue. The other file, glove2word2vec.py handles this problem elegantly.
There's probably one less extra field, which is the token itself (50 numbers for dimensions, 1 string for token, 51 in total). So it should be enough to do + 1 in the code.


Creating Docz entries from other data sources?

Question
I'd like my Docz site to pull in pages from Google Docs (using the gatsby-source-google-docs plugin) as well as MDX files. When creating nodes and creating pages for the Google Docs content, how would I also add "entries" for Docz so that I can render these new pages using the base layout component from Docz?
Any information pointing me in the right direction would be greatly appreciated!

@rsexton404 did you ever figure this out? trying to get something working myself

@colbyfayock I never did get an answer to this and didn’t have the time to dig into Docz’s data structures which were kind of on obscured by the way Gatsby’s theme system puts a lot of code that you might want to build on top of in node_modules. We just ended up building our own props table component that used gatsby-transformer-react-docgen nodes directly. Then we used React Live for our live code examples. Ended up being easier and more customizable than Docz. Hope that helps! I’d be happy to chat more about it sometime!

ah okay. not quite the same but i got Netlify CMS working simply because it commits the files into git 🤷♂ https://github.com/colbyfayock/gatsby-starter-docz-netlifycms
but would love to get a data source for a more advanced cms at some point. thanks for the reply


Scroll to active navigation item

Fix https://github.com/executablebooks/sphinx-book-theme/issues/608

It would be great to have this fix merged and released before the big upgrade to current pydata-theme. @choldgraf, could you have a look?

Thank you @choldgraf for reviewing and merging.
The upgrade to pydata-sphinx-theme~=0.10 will bring breaking changes. Maybe a 0.3.x release before this upgrade would be a good idea. What do you think?

agreed - I created an issue to cut a patch release here:

https://github.com/executablebooks/sphinx-book-theme/issues/611

However I don't have much time for this so I welcome others to give a shot at releasing


feature request: options for 'delimiter' and 'capitalize'

Hi! This is great - thank you! a couple suggestions...

Some places do not allow whitespace in passwords, with a 'delimiter' option we could pick something like - or _, or better yet - none at all! Which leads me to...
An option to capitalize the first letter of each word option would mean we would not need a delimiter at all!

With both of these implemented it would actually be safer too; whilst it does not add any more entropy - it does protects against entropy loss, for that reason, a password like OneTwoThreeFour is better than one like one two three four, but most importantly - it is accepted everywhere 👍

Hi and thanks for the suggestions! Sounds pretty reasonable, I will try to implement them on Sunday.

Done! See the PR #2.

Thank you!
I love the 'presets' idea too!
And you even added an entropy calculator! 🔥
This is now perfect! - all the features you'd ever need 🤘


TypeError: 'FetchResponse' object is not an iterator

I tried the example snippet under https://web.deta.sh/home/user/project/bases:
# 1) pip install deta
from deta import Deta

# 2) initialize with a project key
deta = Deta("project key")

# 3) create and use as many DBs as you want!
users = deta.Base("users")

users.insert({
    "name": "Geordi",
    "title": "Chief Engineer"
})

geordi = next(users.fetch({"name": "Geordi"}))[0]

users.delete(geordi["key"])

But had a TypeError:
>>> from deta import Deta
>>> deta = Deta("<my_token>")
>>> users = deta.Base("users")
>>>
users.insert({
    "name": "Geordi",
    "title": "Chief Engineer"
})>>> ... ... ...
{'key': 'qzb6preouoau', 'name': 'Geordi', 'title': 'Chief Engineer'}
>>> geordi = next(users.fetch({"name": "Geordi"}))[0]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'FetchResponse' object is not an iterator

pip show deta

Name: deta
Version: 1.0.0
Summary: Python SDK for Deta Base & Deta Drive.
Home-page: http://github.com/deta/deta-python
Author: Deta
Author-email<EMAIL_ADDRESS>License: MIT
Location: /Users/user/.pyenv/versions/3.8.5/lib/python3.8/site-packages
Requires:
Required-by:

the last time i use deta, that was working perfectly, i really don´t why but that is not working, but i have find a simple solution
this is what works for me
geordi = users.fetch({"name": "Geordi"})
return user._items

hi, sorry the code snippet on the page is for earlier versions of the sdk. For the latest version, please check the docs.
https://docs.deta.sh/docs/base/sdk/#fetch


Cannot edit record VAC3073-M-00654

None of the edits I do to this record using GEOMG or the spreadsheet import get applied. I am not sure what it is about this record - it does have a faulty thumbnail.

org.apache.solr.common.SolrException: ERROR: [doc=VAC3073-M-00654] Error adding field 'dct_publisher_sm'='[Indiana Department of Geology and Natural Resources, William B. Burford]' msg=Multiple values encountered for non multiValued copy field dc_publisher_sort: William B. Burford

Looks like we've got an Aardvark schema.xml bug. The schema includes a <copyField source="dct_publisher_sm"     dest="dc_publisher_sort"/> but you cannot sort on multivalued (string) fields.
We don't use a publisher sort in Geoportal or GEOMG, so removing this copyField and reindexing should help resolve a lot of the "gone missing" items from Solr.

Reindexes properly with Solr schema.xml file change in place: https://github.com/BTAA-Geospatial-Data-Project/geoportal-solr-config/releases/tag/4.1.1


What is happening with this extension now that this feature is being included in VSCode soon

Didn't see this discussed anywhere, and figured the answer might be nice to pin somewhere.
Anyway, was curious how the upcoming sync feature built in to VSCode would be impacting this extension?  Will this still exist?  Is @shanalikhan getting the support they need?  Will the configurations for this extension need to change (looks like vscode is using the same sync.* settings)?  @sandy081 (or whoever on the VSCode team) will there be a way to easily migrate?

There are no plans for migration.

I actually came to this repo today looking to ask much the same question..
So just to add my 2 cents' worth, I would be very happy if this extension continued to exist after VSCode get's it's own "settings sync" functionality... I don't have, nor do I want, a personal online MS account (which will obviously be a requirement for their own solution).


mgatk-del for other species: array length error

Hello! Thank you for making such a useful tool!
I am reaching out about an issue when using mgatk-del for other species. In the clifind.py file (mgatk--> deletion calling folder), it creates the clip_pos_count_0 and other arrays with the length of 16569, which is the length of the human mtDNA genome.
However, this then only works for human application. I was writing to suggest adding the length as a user-specified parameter, or maybe finding cov and cov_out first, then use cov_out.shape to determine the length for clip_pos_count_0.
My solution for anyone with the same problem:

Clone the github repository
Change the length to match the mtDNA length of your species. This change is necessary in clifind.py file, lines 79-82 and line 120
Build the package.

Best,
Christy

thanks for the note! we definitely had some tunnel vision when writing this. your edits make perfect sense to me and I will try to update the package in the coming weeks. Thank you !


Add Certified k8s Logo to kubeadm overview

See https://github.com/kubernetes/kubeadm/issues/616

This change is

https://deploy-preview-6805--kubernetes-io-master-staging.netlify.com/docs/reference/setup-tools/kubeadm/kubeadm/


500: INTERNAL SERVER ERROR

I get the Flamegraph tab in the Toolbar, but I get a: 500: INTERNAL SERVER ERROR
Details:
Traceback (most recent call last):
File "/home/usr/.pyenv/versions/pq/lib/python3.4/site-packages/django/core/handlers/base.py", line 132, in get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/usr/.pyenv/versions/3.4.3/lib/python3.4/contextlib.py", line 30, in inner
return func(*args, **kwds)
File "/home/usr/.pyenv/versions/pq/lib/python3.4/site-packages/debug_toolbar/views.py", line 19, in render_panel
content = panel.content
File "/home/usr/.pyenv/versions/pq/lib/python3.4/site-packages/djdt_flamegraph/djdt_flamegraph.py", line 55, in content
'flamegraph': flamegraph.stats_to_svg(self.sampler.get_stats())
File "/home/usr/.pyenv/versions/pq/lib/python3.4/site-packages/djdt_flamegraph/flamegraph.py", line 15, in stats_to_svg
out, _ = proc.communicate(stats)
File "/home/usr/.pyenv/versions/3.4.3/lib/python3.4/subprocess.py", line 960, in communicate
stdout, stderr = self._communicate(input, endtime, timeout)
File "/home/usr/.pyenv/versions/3.4.3/lib/python3.4/subprocess.py", line 1602, in _communicate
input_view = memoryview(self._input)
TypeError: memoryview: str object does not have the buffer interface

Thank you for the report. Python 3 wasn't tested very well, but now it is. New release is on PyPI.

Great, thanks, it works now!


.c source changed to .cpp - gcc interprets file differently

Hi,
this package seems to be copying my .c source file to /tmp/ with an extension of .cpp.
My older gcc-4.6 is interpreting this file differently depending on the extension.
Can the file extension of the source file be preserved?
thanks

Thank you for opening the issue. I will fix this issue ASAP and publish a new release.

Hello. The bug has fixed. Please update to the latest version of linter-gcc. Let me know if if works for you now.

Works for me! Thanks.
@hebaishi A comment on a different issue got me thinking. Is it possible to set more than 1 gcc/g++/etc compiler per directory? For example, if the source contained a mix of C,C++,ASM it seems like it would choose single compiler executable.


arranging box of passward and reenter password

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.

issue no?

It's been already done!


Fix typo TrianglesStrip -> TriangleStrip

According to https://api.skia.org/classSkVertices.html#a7676d1d74f49cdf5f21a6b9450fea939 correct spelling is TriangleStrip (not plural). Currently have to use @ts-ignore to make it work correctly. When using trianglesStrip (according to TS enum), getting issue with rendering.
<Vertices
    // @ts-ignore
    mode={'triangleStrip'} // <--- this working, even though it's "incorrect"
    vertices={vertices}
    textures={uvs}
    blendMode='srcOver'
>
</Vertices>

I pushed my branch to the origin remote by mistake, you can accept the changes at https://github.com/mrEuler/react-native-skia/pull/1

Thanks @mrEuler :)


KubeDB Install Redis Sentinel and Redis Shows Empty Status Without Errors

Hi all.
I followed the guides from the links below to install KubeDB Community edition, and thereafter install Redis Sentinel and Redis Cluster:
https://kubedb.com/docs/v2023.10.9/setup/install/community/
https://kubedb.com/docs/v2023.10.9/guides/redis/sentinel/redis-sentinel/
I installed KubeDB successfully with the following command:
helm install kubedb appscode/kubedb --version v2023.10.9 --namespace kubedb --create-namespace --set-file global.license=kubedb-license.txt
The installation of both Redis Sentinel and Redis Cluster show created, but when I try to get the Sentinel or Redis, I see an empty status.
My manifest files are as follows
Redis Sentinel
apiVersion: kubedb.com/v1alpha2
kind: RedisSentinel
metadata:
  name: my-redis-sentinnel
  namespace: demo
spec:
  version: 7.2.0
  replicas: 3
  storageType: Durable
  storage:
    resources:
      requests:
        storage: 250Gi
    storageClassName: "apix04scn"
    accessModes:
    - ReadWriteMany
  terminationPolicy: WipeOut
  podTemplate:
    spec:
      resources:
        requests:
          memory: "512Mi"
          cpu: "250m"
        limits:
          memory: "1Gi"
          cpu: "1.0"

Redis Cluster
apiVersion: kubedb.com/v1alpha2
kind: Redis
metadata:
  name: my-redis
  namespace: demo
spec:
  version: 7.0.5
  replicas: 3
  sentinelRef:
    name: my-redis-sentinnel
    namespace: demo
  mode: Sentinel
  storageType: Durable
  storage:
    resources:
      requests:
        storage: 250Gi
    storageClassName: "apix04scn"
    accessModes:
    - ReadWriteMany
  terminationPolicy: WipeOut
  podTemplate:
    spec:
      resources:
        requests:
          memory: "750Mi"
          cpu: "250m"
        limits:
          memory: "2Gi"
          cpu: "2.0"

Applying these files shows
redissentinel.kubedb.com/my-redis-sentinnel created and redis.kubedb.com/my-redis created respectively.
Running kubectl -n demo get redissentinel, returns the following nothing else (secrets, replicasets, pods etc) gets created, no errors whatsoever
NAME                     VERSION   STATUS   AGE
my-redis-sentinnel   7.2.0              17m
Same for kubectl -n demo get redis
NAME                VERSION   STATUS   AGE
my-redis              7.0.5              20m11
I have tried all configurations, without a storageType of Durable and storageClass. I have tried the exact examples given on the above pages and I still get the same behavior.
Any help will be appreciated.
Thanks.

Can someone help on this please


compile failure with pre-AMP base < 4.8

see below; I've already revised the affected release (see https://hackage.haskell.org/package/word-wrap-0.3.3/revisions/)
Configuring library for word-wrap-0.3.3..
Preprocessing library for word-wrap-0.3.3..
Building library for word-wrap-0.3.3..
[1 of 1] Compiling Text.Wrap        ( src/Text/Wrap.hs, /tmp/matrix-worker/1502046925/dist-newstyle/build/x86_64-linux/ghc-7.8.4/word-wrap-0.3.3/build/Text/Wrap.o )

src/Text/Wrap.hs:89:20:
    Not in scope: ‘<$>’
    Perhaps you meant ‘<>’ (imported from Data.Monoid)

Thanks!


Help with Client mocking / setupTestClientAndCreateIndex example

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[ ] elastic.v5 (for Elasticsearch 5.x)
[X ] elastic.v6 (for Elasticsearch 6.x)
Please describe the expected behavior
Following the example in setup_tests, I expect to be able to create a mock client with a simple index.
Please describe the actual behavior
I return an error:
panic: runtime error: invalid memory address or nil pointer dereference [recovered]         panic: runtime error: invalid memory address or nil pointer dereference [signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x1326ab6]
Any steps to reproduce the behavior?
Create and a reduced version of setupTestClientAndCreateIndex as follows:
const ( 	testIndexName  = "elastic-test" 	testIndexName2 = "elastic-test2" 	testIndexName3 = "elastic-test3" 	testMapping    =
{
"settings":{
"number_of_shards":1,
"number_of_replicas":0
},
"mappings":{
"doc":{
"properties":{
"user":{
"type":"keyword"
},
"message":{
"type":"text",
"store": true,
"fielddata": true
},
"tags":{
"type":"keyword"
},
"location":{
"type":"geo_point"
},
"suggest_field":{
"type":"completion",
"contexts":[
{
"name":"user_name",
"type":"category"
}
]
}
}
}
}
}
`)
type logger interface {
Error(args ...interface{})
Errorf(format string, args ...interface{})
Fatal(args ...interface{})
Fatalf(format string, args ...interface{})
Fail()
FailNow()
Log(args ...interface{})
Logf(format string, args ...interface{})
}
func setupTestClient(options ...elastic.ClientOptionFunc) (client *elastic.Client) {
//var err error
client, _ = elastic.NewClient(options...)
//if err != nil {
//	t.Fatal(err)
//}

//client.DeleteIndex(testIndexName).Do(context.TODO())

return client

}
func setupTestClientAndCreateIndex(t logger, options ...elastic.ClientOptionFunc) *elastic.Client {
client := setupTestClient(options...)
cl, _ := client.CreateIndex(testIndexName).Body(testMapping).Do(context.TODO())
fmt.Println(cl)
//if err != nil {
//	t.Fatal(err)
//}
//if createIndex == nil {
//	t.Errorf("expected result to be != nil; got: %v", createIndex)
//}

return client

}
func TestFind(t *testing.T) {
cl := setupTestClientAndCreateIndex(t)
fmt.Println(cl)
}`
Execute the test

There were some issues already regarding testing. I personally think that if you need to mock *elastic.Client, you're actually testing on the wrong layer of abstraction. See https://github.com/olivere/elastic/issues/795#issuecomment-398376126 for a discussion and some hints.

Fair enough. Thank you


Speaker detail module

should

show all speaker(s) details expanded (name, company, company role, social links, img, bio)
gracefully handle multiple speakers
have a simple link to session(s)

Could this just be a module that goes on the session page?

in my current design IA, a session page already has a summary module about the speaker(s) with link(s) to their pages, and vice versa, where a speaker page has a summary module with link(s) to the session(s) they're in. i considered merging them too, but felt that leaf/detail pages for each collection item would scale more to the various speaker/session combos out there. thoughts?

i also think it's really nice for a speaker to have a dedicated page to share, that's not just about their session(s). also, if a session has multiple speakers, it's a nice gesture to give them each their own detail pages instead of making them merge their notes about themselves or something.

portable as a module, no need for dedicated pages if we have dedicated sessions 👍
updated the description with some updated expectations

i'm thinking now we skip this page entirely and link any "profile clicks" to the respective profile on chrome developers, or some other entity where our bios already exist. thoughts?


[TESTS] Need to add BindingData tests for Microsoft.Azure.WebJobs.Extensions.Blobs/Queues

Query/Question
Since the package Microsoft.Azure.WebJobs.Extensions.Blobs was migrated to the Azure SDK in the track1 to track2 move, the BindingData tests from the WebJobs repo also need to be migrated.
These tests are currently in the WebJobs repo.
There are some Queue BindingData tests in the Azure SDK here, but these tests should be cross-referenced with the WebJobs repo to ensure test coverage remains the same.
I don't see similar tests for blobs extensions. These will need to be added to test the logic here.
Environment:

Name and version of the Library package used: Microsoft.Azure.WebJobs.Extensions.Blobs and Microsoft.Azure.WebJobs.Extensions.Queues

The Queues tests are covered here.
https://github.com/Azure/azure-sdk-for-net/blob/c13662bd5365cb66a13beb9ffa32337965e8c2a7/sdk/storage/Microsoft.Azure.WebJobs.Extensions.Storage.Queues/tests/QueueTriggerTests.cs#L822-L910

Blobs https://github.com/Azure/azure-sdk-for-net/pull/24391 .

What about Queues?

@mathewc please see comment above. Queues are already covered.


fix order by with cte cannot bind

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run clickbench


Add checks for interface arguments

On W-007, I glanced through the output scanning against upstream refpolicy and all the examples of issues found I checked were the false positive on mls arguments.  I'm hesitant to include a check that is so false-positive happy.  Any thoughts on ways to eliminate that?
New functional test files need to be included in tests/Makefile.am FUNCTIONAL_TEST_FILES list or else functional tests won't work out of a distribution tarball.  Looks like there were already some missing before this PR.  I'll get those added in and add a pipeline check for that so that case gets caught earlier.

On W-007, I glanced through the output scanning against upstream refpolicy and all the examples of issues found I checked were the false positive on mls arguments. I'm hesitant to include a check that is so false-positive happy. Any thoughts on ways to eliminate that?

I am not sure whether these are actual false-positives, do we not want to get warned about such arguments? (see #19)

New functional test files need to be included in tests/Makefile.am FUNCTIONAL_TEST_FILES list or else functional tests won't work out of a distribution tarball. Looks like there were already some missing before this PR. I'll get those added in and add a pipeline check for that so that case gets caught earlier.

Oops. A pipeline check would be nice.

This feels different to me than the sort of issue I was thinking of in #19.  In the example there, the developer's intent seems likely to be to substitute foo as $1 and bar as $2, when in fact they're substituting "foo bar" as $1 and "" as $2.  In the mls case, the developers intent is to substitute "mls_systemlow - mls_systemhigh" in its entirety as the argument, which is in fact what they're doing.
It seems to me like in this case, the argument for quoting the whole string or whatever the proposed alternative is comes down to making things easier for selint, which doesn't seem like a reasonable demand to make on policy developers.  I could definitely be wrong about that though, but my impression is that the community doesn't see anything wrong with usages like foo(s0 - s1).
An ideal solution might be to compare to the expected number of arguments, but that requires parsing the comments above the interface, which is something I'd like to do eventually, but seems decidedly non-trivial.  A more hacky approach might be to not report this warning if the argument includes a '-' character.  That would also omit "domain -unconfined_t" and the like, which seems desirable to me.
I'm open to hearing arguments on the other side though.

I'm okay with this.  Go ahead and do the squashing and then I'll merge.


[BEAM-1101] Remove the pipeline_type_check option

Default value is True for this option and --pipeline_type_check is a
noop. (Still keeping the --no_pipeline_type_check for optionally
disabling this check.)

R: @robertwb

LGTM

Thank you.


Translation Improvement Request: Microsoft Official Documentation

[Enter feedback here]
Hello,
I've noticed that the translation of the content from the Microsoft official documentation into Korean appears to be a bit unnatural. Specifically, I found the following table that compares Microcontrollers (MCUs) and Microprocessors (MPUs):

Feature
Microcontroller (MCU)
Microprocessor (MPU)

CPU
Less
More

RAM
Less
More

Flash
Less
More

OS
Bare Metal / RTOS
General Purpose (Windows / Linux)

Development Difficulty
Harder
Easier

Power Consumption
Lower
Higher

Cost
Lower
Higher

Deterministic
Yes
No - with exceptions

Device Size
Smaller
Larger

I believe that "Less" and "More" in the CPU, RAM, and Flash columns might not convey the intended meaning clearly in Korean. Additionally, the terms used for "Deterministic" could be improved for better understanding. I'd like to contribute to improving this translation by suggesting more natural alternatives and providing a better representation of the original content. I can work on this and provide a revised version of the table.
Please assign me as the assignee for this issue so that I can begin working on the necessary changes.

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

ID: a3b64e2b-c6b1-df1a-706a-6708915e5ba6
Version Independent ID: 863e28f7-244b-8ea6-19a4-2fa62ff696c2
Content: Overview of Azure IoT device types
Content Source: articles/iot-develop/concepts-iot-device-types.md
Service: iot-develop
GitHub Login: @ryanwinter
Microsoft Alias: rywinter

@Cor5in
Thanks for your feedback! We will investigate and update as appropriate.

Oh, I modified it and sent a full request

@Cor5in
I've delegated this to @ryanwinter, a content author, to review and share their valuable insights.

Oh, I see, thanks!


Split up the CSS in a stylesheet for colors and one for structure

When talking about Material Design and that it may look aweful on a Mac (for example) we thought it might help to support theming.
To help implement this, split up the CSS in a stylesheet for colors and one for structure (padding, margins etc).
@JeroenVinke:

That way we can replace the stylesheet that has colors with a materialize one, while keeping the structural css the same)
I don't mean "replace" but we can have multiple stylesheets with different theming

We could even use the "accent/primary" color principle (for all themes - maybe more colors).. This way it's easy to keep the UI streamlined..
...
A colors.less file or something similar should be enough.

I chose not to use the colors.less approach, because themes should be able to specify more than just the base color. The materialize theme would be able to use the accent/primary approach though

themes should be able to specify more than just the base color

You're right.
I had something different in mind, though. I'll try to be more specific. :smile:

define a bunch of colors which are used in the application (try to limit them to a few)
put them in colors.less (in addition to other theme files)

Then we have a classical palette definition and a reference for future themes.

Correct me if i'm wrong, but in the less files of the themes we would work with variables out of the colors.less file of the theme?

Now it makes less sense to me. :smile:
It's just a shortcut or a reference to which colors are used. Also it's a help to streamline color usage.
Nothing more, actually. Of course themes can still use their own colors.
Let's say I'm writing a simple theme which does nothing more than change the colors (because pink looks alien on my device or I simply don't like pink).
I copy/import Monterey's colors.less and change the colors.
In another example I'd like to keep the colors but add some more things. I can use the colors defined in Monterrey's colors.less  file there and be happy.
This is just to explain what I thought. Everytime I read this, it somehow becomes less "important" but I like the approach of having a color guide for ourselves.

OK, we are on the same page now :)
I'm working on the theme switcher now so I'll make sure that this is possible as well


Cleanup of test-related terminology

What this PR does / why we need it?
Previously, the structure that contained metadata used for testing purposes was named XXStructure. However, the name XXStructure did not convey that it should only be used for testing purposes, leading to misuse in some cases. To address this, we have established the following rules:

The XXForTest type is an object used for debugging or testing purposes, representing a lossy transformation from XX. Its values can change frequently.
The XXStructure type is an object used to store XX, representing a lossless transformation from XX. It can be transformed back to XX from the XXStructure.
The obj.toXX method is used when obj is converted to the XX format.

In accordance with these rules, this PR have updated the names as follows:

Tree.toStructure() -> Tree.toTestTreeNode().
Element.getStructureAsString() -> Element.toTestString().

Any background context you want to provide?
What are the relevant tickets?

Fixes #
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

CC) @7hong13, @skhugh, @humdrum


VS Code just deleted my file.

VSCode Version: Version 1.29.1 (1.29.1)
OS Version: OSX 10.14.1 Beta (18B50c)

One file, gone.
Unfortunately I didn't save to git.
It was my entire days work.
I'm not sure what happened.
I didn't do anything.
My terminal said ENOSPC no space left on device as an error in the file on question. And the file went blank. I checked trash but nothing. The actual file is still there but there is nothing in it. It like someone just copied all the text in the file and deleted it.
I have about 2GB on my hard drive so space isn't an issue.
I love you vs code but this is pretty hard core.
I don't understand why this is even possible.

The same issue happened to me. I was lucky that the file was commit to git. I can restore the file easily.
I just checked logs at ~/.config/Code/logs, but didn't find any related message.
How can I provide more information about this issue?
VS Code: 1.29.1-1542309157
OS: Debian GNU/Linux 9.6 (stretch)

I think this is a duplicate of https://github.com/Microsoft/vscode/issues/34915. It is certainly possible that the OS reserves a little bit of disk space to operate so that you see this error even though there is still some disk space. Sorry for that, we should get https://github.com/Microsoft/vscode/issues/34915 addressed asap.


Can't import constants in next.config.mjs/js

Describe the Bug
When I import constants like this in my next.config.mjs:
import { constants } from '@sitecore-jss/sitecore-jss-nextjs';

The build will break:
(node:8856) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
 ⨯ Failed to load next.config.mjs, see more info here https://nextjs.org/docs/messages/next-config-error

> Build error occurred
Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\chalk\source\index.js from<EMAIL_ADDRESS>not supported.
Instead change the require of index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.
    at mod.require (C:\Projects\Redacted\turborepo\node_modules\next\dist\server\require-hook.js:65:28)
    at Object.<anonymous> (C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\@sitecore-jss\sitecore-jss-nextjs\dist\cjs\services\component-props-service.js:16:33) {
  code: 'ERR_REQUIRE_ESM'
}

But when I change the import to:
import { constants } from '@sitecore-jss/sitecore-jss';

It works fine.
It makes no difference if it's called next.config.js or next.config.mjs.
This is in an update from 22.0.0 to 22.2.1, and there was nothing in the changelog about this, even though it's clearly a breaking change.
To Reproduce
Use the import as described.
Expected Behavior
It should not complain about import bollocks. Those are among the most frustrating errors you could possibly run into.
Possible Fix
No response
Provide environment information

Sitecore Version: 10.4
JSS Version: 22.2.1
Browser Name and version: Firefox latest
Operating System and version (desktop or mobile): desktop, Windows 10
Link to your project (if available): N/A

Just to add: it's not that constants is that much of an important thing to be able to import. I'm just thinking "what else doesn't import correctly, should we need it in the future?"
This is a disconcerting thought, and that's why this merits some thorough testing, now and in the future.

After going through the migration and no errors remaining, a build now produces this:
   Collecting page data  ..Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\chalk\source\index.js from<EMAIL_ADDRESS>not supported.
Instead change the require of index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.
    at mod.require (C:\Projects\Redacted\turborepo\node_modules\next\dist\server\require-hook.js:65:28)
    at Object.<anonymous> (C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\@sitecore-jss\sitecore-jss-nextjs\dist\cjs\services\component-props-service.js:16:33) {
  code: 'ERR_REQUIRE_ESM'
}
Error [ERR_REQUIRE_ESM]: require() of ES Module C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\chalk\source\index.js from<EMAIL_ADDRESS>not supported.
Instead change the require of index.js in<EMAIL_ADDRESS>to a dynamic import() which is available in all CommonJS modules.
    at mod.require (C:\Projects\Redacted\turborepo\node_modules\next\dist\server\require-hook.js:65:28)
    at Object.<anonymous> (C:\Projects\Redacted\turborepo\apps\e-commerce\node_modules\@sitecore-jss\sitecore-jss-nextjs\dist\cjs\services\component-props-service.js:16:33) {
  code: 'ERR_REQUIRE_ESM'
}

> Build error occurred
Error: Failed to collect page data for /[[...path]]
    at C:\Projects\Redacted\turborepo\node_modules\next\dist\build\utils.js:1268:15 {
  type: 'Error'
}

So it's going wrong at "Collecting page data".
 It doesn't say where this include is actually coming, and it is incorrectly telling me to replace a require inside node_modules, which I'm obviously not going to do.
Please provide a fix at your earliest convenience. This is now blocking us. Critical priority please.

If I go to<EMAIL_ADDRESS>and actually make the change it wants me to make:

It appears to work. Or at least the build doesn't explode.
But again, this is horrible and unsustainable workaround that will only work on my machine, and will be replaced at the whim of an npm install, or indeed a deployment to any environment beyond my local dev.
The real solution is to not transpile to arcane require statements. Or set up the exports in the (meaning JSS's) package.json correctly.

Doing some more investigation, I see JSS uses require in next.config out of the box - import is not (and wasn't as of v22.0) usable in next.config.js as it's not an ESM module.
We have plans to switch all of JSS to ESM completely and that should take care of this and similar problems.

import in next.config.js (or .mjs) is fully supported. It's working perfectly for other imports as well.
What version of Next.js are you using that doesn't support this? We are on 14, switching to 15 shortly.


Can't install plugin on cordova 9

Hi!
I can't install this plugin on a cordova 9.0.0 project. I just can see the start of a warning:

E:\Cordova9>cordova -v
9.0.0

E:\Cordova9>cordova plugin add cordova-plugin-background-mode
Installing "cordova-plugin-background-mode" for android
warn

E:\Cordova9>

Best regards
Michael

I was also having the same issue. The plugin does appear on the list when I do "cordova plugins". That is probably the reason why I'm having "Uncaught TypeError: Cannot read property 'backgroundMode' of undefined".

We are having the same problem...
C:\Projects\pp\test>ionic cordova plugin add cordova-plugin-background-mode

cordova plugin add cordova-plugin-background-mode --save
Installing "cordova-plugin-background-mode" for android
warn

C:\Projects\pp\test>

same problem for ios

I have some probem too.

Yeah got the same problem when trying to re-add the plugin - (the src etc doesn't get added). I had a copy of the plugin on my machine it works fine even on Android 9.0 - so a workaround could be to manually download the plugin off the git and add from your local machine?

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

But I have the same issue in Mac OS.
Installing "cordova-plugin-background-mode" for android warn CordovaError: warn     at /usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:230:23     at Array.forEach (<anonymous>)     at getEngines (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:208:13)     at Function.runInstall (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:285:22)     at /usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:76:35     at process._tickCallback (internal/process/next_tick.js:68:7)     at Function.Module.runMain (internal/modules/cjs/loader.js:757:11)     at startup (internal/bootstrap/node.js:283:19)     at bootstrapNodeJSCore (internal/bootstrap/node.js:622:3)

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

Confirmed! For Android.

For me, I had to change the plugin.xml and change
        <source-file             src="src/android/BackgroundModeExt.java"             target-dir="src/de/appplant/cordova/plugin/background" />
to
        <source-file             src="src/android/BackgroundExt.java"             target-dir="src/de/appplant/cordova/plugin/background" />

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

aparently work here

Not working for me. ¿Can you fix that bug? Thank you very much.

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

just compile after delete that line?
if I run again the "plugin add" of course the source will be overwrites :/

I tried it but it does not work

Deleting the line below works, as others said. The rest of the windows code is commented out.

For me, I had to change the plugin.xml and change
 <source-file src="src/android/BackgroundModeExt.java" target-dir="src/de/appplant/cordova/plugin/background" />
to
 <source-file src="src/android/BackgroundExt.java" target-dir="src/de/appplant/cordova/plugin/background" />

this file "BackgroundExt.java" not exist in 0.7.2 version. any alternative fix?

For me, I had to change the plugin.xml and change
 <source-file src="src/android/BackgroundModeExt.java" target-dir="src/de/appplant/cordova/plugin/background" />
to
 <source-file src="src/android/BackgroundExt.java" target-dir="src/de/appplant/cordova/plugin/background" />

this file "BackgroundExt.java" not exist in 0.7.2 version. any alternative fix?

my fork fix cordova 9
https://github.com/sercatven/cordova-plugin-background-mode

How can I install de plugin without de Cordova-CLI so I can delete the indicated windows line of code?

For me,

run: cordova plugin add cordova-plugin-background-mode  normal
$root_dir/plugins/cordova-plugin-background-mode/plugin.xml delete line

run: ionic cordova platform rm android ; ionic cordova platform add android
done

Same issue here, seems to be related to the windows-sdk line on plugin.xml.
Can someone fix it?

@sercatven  : still facing same issue for Background Mode plugin not installed error

For me, IONIC 4

run: cordova plugin add cordova-plugin-background-mode  normal
$root_dir/plugins/cordova-plugin-background-mode/plugin.xml delete line
<engine name="windows-sdk" version=">=10.0.14393.0" />
run: ionic cordova platform rm android ; ionic cordova platform add android
done

Does it work for ios too?

@Divyanshu30 : I am using ionic3. I also delete the line in plugin.xml but when I say this.backgroundMode.on('Activate') error showing for me Plugin not installed

@All : I uninstall cordova version 9.0 and tried to install cordova version 8 and installed plugin called Background Mode and no error shown and working fine

I have the same problem with Cordova 9 & iOS. I'm waiting for the solution. I hope it arrives early because I can't release the app.

It is going to be almost 2 months since this error appearded. Is there a way to fix it? I think the pull requests are still waiting there.

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

works for me! Thanks

It works but only on iphone simulator; not on real device!

It works but only on iphone simulator; not on real device!

that just happened to me. please help

For me, I had to change the plugin.xml and change
 <source-file src="src/android/BackgroundModeExt.java" target-dir="src/de/appplant/cordova/plugin/background" />
to
 <source-file src="src/android/BackgroundExt.java" target-dir="src/de/appplant/cordova/plugin/background" />

this file "BackgroundExt.java" not exist in 0.7.2 version. any alternative fix?

my fork fix cordova 9
https://github.com/sercatven/cordova-plugin-background-mode

This fork worked for me. My environment is Ionic 4 and Android 9.0.
Also note that you need to call disableWebviewOptmizations for your app to actually keep running on background.
Thanks @sercatven .
Also, please @katzer , accept the PR's to fix these bugs.

#452

Hey everyone!
I'm too face this problem when trying to install this plugin in Cordova 9.0.0.
However, I just found a way to make it work despite I don't know why doing this make this work.
Just delete the following line in plugin.xml
<engine name="windows-sdk" version=">=10.0.14393.0" />

Worked for me, thanks a lot!
P.S.: Only works with cordova-plugin-background-mode version: 4c157f4 (= commit of Feb 3, 2019). If i use the latest version i do not get errors, but background mode never gets activated.

Is there a fork that we can use in ionic instead of this plugin? Cant beleive that in all this time the issue and pull requests are still there

Delete the next line in plugin.xml
 only works on virtual device.
You could remove the webview plugins for ionic if you do not need it. this worked for me

For me, IONIC 4

run: cordova plugin add cordova-plugin-background-mode  normal
$root_dir/plugins/cordova-plugin-background-mode/plugin.xml delete line
<engine name="windows-sdk" version=">=10.0.14393.0" />
run: ionic cordova platform rm android ; ionic cordova platform add android
done

You're the boss! many thanks!

i have the same problem

+1

For me, I had to change the plugin.xml and change
 <source-file src="src/android/BackgroundModeExt.java" target-dir="src/de/appplant/cordova/plugin/background" />
to
 <source-file src="src/android/BackgroundExt.java" target-dir="src/de/appplant/cordova/plugin/background" />

this file "BackgroundExt.java" not exist in 0.7.2 version. any alternative fix?

my fork fix cordova 9
https://github.com/sercatven/cordova-plugin-background-mode

it works. thanks!

it didn't work for me,
in plugins for cordova background mode, this line was not there

but I still rm android platform and again added that. But still not working,
still getting the following issue,
vendor.js:51847 ERROR Error: Uncaught (in promise): TypeError: Cannot read properties of undefined (reading 'enable')
TypeError: Cannot read properties of undefined (reading 'enable')
    at Channel.<anonymous> (main.js:939:61)
    at Channel.subscribe (cordova.js:802:21)
    at document.addEventListener (cordova.js:125:34)
    at document.addEventListener (cordova.js:1460:13)
    at AppComponent.<anonymous> (main.js:932:46)
    at step (vendor.js:106845:23)
    at Object.next (vendor.js:106826:53)
    at vendor.js:106819:71
    at new ZoneAwarePromise (polyfills.js:3272:29)
    at Module.__awaiter (vendor.js:106815:12)
    at resolvePromise (polyfills.js:3193:31)
    at new ZoneAwarePromise (polyfills.js:3275:17)
    at Module.__awaiter (vendor.js:106815:12)
    at main.js:923:100
    at push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (polyfills.js:2785:31)
    at Object.onInvokeTask (vendor.js:53413:33)
    at push../node_modules/zone.js/dist/zone.js.ZoneDelegate.invokeTask (polyfills.js:2784:60)
    at push../node_modules/zone.js/dist/zone.js.Zone.runTask (polyfills.js:2557:47)
    at push../node_modules/zone.js/dist/zone.js.ZoneTask.invokeTask (polyfills.js:2860:34)
    at ZoneTask.invoke (polyfills.js:2849:48)
defaultErrorLogger @ vendor.js:51847

Can you help me out please ?And anyone else can help also pl. Thanks.
My project versions :
ionic: 4.11.13
angular : 7.2.4
@ionic-native/background-mode: "^5.0.0-beta.24",
project type: angular


A set of basic selection / transformation operators

What about flatMap?

I might do a kind of concatMap later, perhaps also a merging concurrent variant, but I don't want to turn the whole thing into a full library either :-)


Allow collective/host admins to filter by drafts

As a collective/host admin, I can filter expenses by draft (e.g. on https://opencollective.com/babel/expenses).

Hey @Betree this one looks interesting to me.
Is the scope of this ticket to add an option to the status filter for expenses that lets a collective/admin filter expenses by draft?
Let me know if you think this is an appropriate one to take on now.

Is the scope of this ticket to add an option to the status filter for expenses that lets a collective/admin filter expenses by draft?

Exactly ✅ I've assumed that it's doable with frontend-only changes, but maybe we'll discover we need to make small changes in the API too.

Let me know if you think this is an appropriate one to take on now.

It is! I'll assign it to you, feel free to ping me or the engineering team if you need anything.

Duplicate of https://github.com/opencollective/opencollective/issues/6224


WebPortal Delete Job Question

Organization Name: SitonHoly

Short summary about the issue/question:
i submit some jobs in webportal,but some jobs error
Too much wrong task make me feel angry
i want to delete these jobs，what should i do?

Brief what process you are following:
submit some tasks

How to reproduce it:

OpenPAI Environment:

OpenPAI version: 0.11.0
Cloud provider or hardware configuration: SitonHoly Server machine
OS (e.g. from /etc/os-release):ubuntu 16.04
Kernel (e.g. uname -a):4.4.0-116-generic
Hardware (e.g. core number, memory size, storage size, GPU type etc.):
master:30core,125G,60T,gpu=none
workerA:30core,376G,20T,gpu=v100
workerB:30core,376G,20T,gpu=1080ti
master1,workerA2,workerB*7
Others:

Anything else we need to know:

we do not expose API to delete a job. you can only stop jobs.

we do not expose API to delete a job. you can only stop jobs.

i try deploy  nni on openpai and  found very trial will generate a job,so when  trial number=1000,there will be 1000 jobs!


Fix has_velocities again

This PR redoes the work of #701, except it adds a few more methods to make it even more robust.
After this is merged we should revisit #707.

PS: You wrote resolve #707 in your original PR comment so merging this will automatically close #707 but I don't think that's what you want?

Hmmm, when we start eliding operations for 1D and 2D models I guess we will still have all three velocity components as you can have v-velocity through an xz model?

The Flat annotation implies that derivatives are zero along a certain direction, but it does not imply that the velocity components are zero.
Some people call this "two and a half" - D model. In other words the other dimensions are still there; it is just the problem is homogeneous along them.
In general you do want all 3 components even when derivatives are zero. This is especially important for problems in x-z with rotation (and thus non-zero v component), which is a common case in oceanography.
We could also support "true 2D" at some point, but we would need additional abstraction other than just topology for users to specify this behavior.


Adding null-conditional operator on IEnumerable.GetEnumerator()

Summary of the changes (Less than 80 chars)

Added ?. operator to prevent null reference exception (_items?.GetEnumerator())

Addresses #16938

 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.:x: smaglio81 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Looks like there can also be a null ref in CopyTo
https://github.com/aspnet/AspNetCore/blob/d66db3b0fffdcb010914a4d5a8cb4426f5f1c7ce/src/Http/Http/src/Internal/ItemsDictionary.cs#L98-L104

Okay, I've attempted to integrate the recommended updates. Let me know how it looks.

Are you interested in fixing CopyTo also?

In the code snippet, it seemed like line 98 was doing a null check? Also, I'm kind of new to the github resolution system ... since that comment didn't have the conversational display (and resolution button) like the other comments I thought it might have been a momentary concern that was self-resolved.
Let me know if you still want it updated after reviewing line 98?
https://github.com/aspnet/AspNetCore/blob/d66db3b0fffdcb010914a4d5a8cb4426f5f1c7ce/src/Http/Http/src/Internal/ItemsDictionary.cs#L98-L104

line 98 was doing a null check?

Yeah, but the after the null check line 104 uses _items which would null ref.

Great Catch! Since it seems like .CopyTo wants to copy the items into the dictionary, what if around line 102 EnsureDictionary() was called to initialize _items?

It's copying from _items to the passed in array, we don't want to allocate the dictionary.
I think just adding an else would be good.

Can do!

So, I started going down the route of using an else statement, but I wanted to format the curly braces in the way the code normally does it. After poking around for an example, it started to seem like else statements weren't used a lot. So, I went back to your original suggestion and used the null-conditional operator.
I also added a unit test for it. I'm not sure if I hit the standards on it.

@aspnet-hello triage https://dev.azure.com/dnceng/public/_build/results?buildId=422315&view=logs

This comment was made automatically. If there is a problem contact<EMAIL_ADDRESS>I've triaged the above build. I've created/commented on the following issue(s)
https://github.com/aspnet/AspNetCore-Internal/issues/2730
https://github.com/aspnet/AspNetCore-Internal/issues/3241

Thanks @smaglio81 !


titlewolds failing for unknown reason.

After the second time launching with titleworlds it gives me an error saying failed to connect to server.
error:

latest.log
tw folder

Thank you, I was hoping someone would report this
It's an error I get about 0.1% of the time when developing so it is a pain to debug it but this customized crashlog should make it easier


edit pass: Protobuf chapters

@shirhatti, @mairaw - Please review the copy edit to these articles and let me know if you have any feedback. This is for Azure DevOps work request 1646344.
Contributes to #15889.

@shirhatti Hi, this edit pass is still waiting on your signoff. Can you help wrap this up? Thanks!


fix: Icefall Mantle incorrectly applying surge buffs to non-Stasis weapons

When Icefall was enabled in perks, it'd apply its damage bonus to any weapon even if it wasn't Stasis.

Good catch, lgtm and @BarmonHammer can probably just merge this


Prevent placeholder from creating a DOM element on dropped event

I want to prevent ui-tree default behavior on dropped event, specifically the creation of an empty element in the DOM, that is the placeholder.
I'm using ui-tree between two connected trees but on dropped I see the placeholder instantly creates an empty element in the tree, while I want to have control over the tree view model since I update the tree view model once I receive the websocket message from the server.
How the tree will be sorted is not defined, so I would like to prevent ui-tree to update the tree view model on its own.
Thanks in advance for any hint.

After trying different things I came with a work around, to prevent placeholder from showing due to the creation of a DOM element on dropped.
I've used the beforeDrop function that is called before the dragging node is dropped to remove the dragging node from the destination tree.
The code looks like:
 $scope.treeOptions = {
        beforeDrop: function(e) {
            var draggedInfobitId = e.source.nodeScope.$modelValue.id;
             doSomething ()
            e.dest.nodesScope.removeNode(draggedInfobitId);
            return false;
        }
    }


Too loud!

Really like it but the sound is too loud! When you return to it from YT, your media etc you can go deaf so loud is the white noise. Could you please address this issue?

Hello @rezogo,
I've incorporated the requested functionality in my pull request. Prior to having @lazy-guy merge the changes, you have the option to temporarily use my fork at https://github.com/Weizhe-Chen/tomodoro. It's hosted on this website: https://wchen-robotics.com/tomodoro/

Fantastic! Thank you

I have merged the changes done by @Weizhe-Chen (#7), which adds a volume slider for controlling the loudness.


google places autocomplete: The provided api key is invalid

I receive the following error when trying to use this package:

As per the directions, I have enabled the following APIs:

Places API for Web
Geocoding API

I also added the following restrictions to see if that would help, but it did not:

Any help would be appreciated.

You need to get the key here https://developers.google.com/places/web-service/intro
And use the component like this:
<GooglePlacesAutocomplete query={{ key: 'API_KEY' }} />

None of the above working for me .


himalaya: 0.7.1 -> 0.7.3

Description of changes
https://github.com/soywod/himalaya/blob/master/CHANGELOG.md
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 229320 run on x86_64-linux 1

2 packages built:

himalaya
vimPlugins.himalaya-vim


Query params are being cut off with the default redirect rule

Recently, we migrated our front-end from S3 + CloudFront to Amplify, and one of our functions stopped working.
We integrated with a third-party service (TradeStation) and after the successful authentication, it should have redirected us back to our website and provide us the code in query params.
But then we noticed that for some reason all the query params are being cut off and instead of https://our-website.com/profile?code=XXX we received simply https://our-website.com/profile and therefore it broke our logic.
After a deep investigation, we figured out that it most likely was related to the "Rewrites and redirects" rule that we have by default in our Amplify app. To be precise, this one: /<*> -> /index.html
I was able to overcome this issue by simply adding an extra slash after the redirect URL that we provide. So the https://our-website.com/profile/?code=XXX works fine and doesn't cut of query params.
Please let me know if this solution is how it suppose to work or probably you have some better solution like adjusting the redirect rule or anything else.
Thanks in advance!

Hi @YelyseiLukin, thanks for reaching out. I will look at this and get back to you with the recommended solution.

Is there any movement on this? Unfortunately adding an extra slash is not an option for us.

@tomasswood @YelyseiLukin we just launched support for this: https://docs.aws.amazon.com/amplify/latest/userguide/redirects.html#query-strings-and-path-parameters. Let us know if this works for you.


Polish translation improvements

Corrections and removal of Swedish values from Polish translation.

@all-contributors add @m-ochyra for translation


ContentDialog does not follow RequestedTheme properly in XAML Islands

Describe the bug
When a ContentDialog is shown within an XAML Island and it's RequestedTheme does not match the current Windows theme, it does not apply properly (See screenshot below) - mainly the buttons appear to use the wrong RequestedTheme.
Steps to reproduce the bug

Create an app that uses WinUI 2 with XAML Islands (Tutorial here: https://docs.microsoft.com/en-us/windows/apps/desktop/modernize/host-standard-control-with-xaml-islands).
Set Windows' app mode to Dark
Create a ContentDialog with it's requested theme set to Light in XAML and show it.

Expected behavior
The buttons should be in light mode, like the rest of the dialog. This is what happens with the same setup in a UWP app:

Screenshots
Here is a ContentDialog in an XAML Island with its RequestedTheme set to Light while the Windows theme is set to dark. Notice that the buttons don't seem to realise they should be in light mode, making the text very hard to read:

NuGet package version
Microsoft.UI.Xaml 2.8.0
Windows app type

[ ] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
No response

I found some more information about this, it seems the Windows Terminal team has come across this issue as well.
See these for some more details about this problem:
https://github.com/microsoft/terminal/pull/5224
https://github.com/microsoft/terminal/issues/12356
https://github.com/microsoft/terminal/issues/3917
https://github.com/microsoft/terminal/issues/3654
https://github.com/microsoft/terminal/issues/5195

Still happens in the latest version of WinUI 2 and Windows 11.

@bpulliam Y'all may want to dedupe with #2331. My limited reading of these two is that they're effectively the same.


New package: Pyro v0.1.0

Registering package: Pyro
Repository: https://github.com/femtomc/Pyro.jl
Created by: @femtomc
Version: v0.1.0
Commit: 9decd9ac197c3dd9446970a97cc9955ada4fdffb
Reviewed by: @femtomc
Reference: https://github.com/femtomc/Pyro.jl/issues/1#issuecomment-675567302

As in, leave the repo as BDD.jl but have it on the general registry as BinaryDecisionDiagrams?


Fix package dependencies

See #654

@alpmestan My apologies about making this PR difficult to review. To figure out the failure in #654 I had to read some code I didn't write and my hands were automatically fixing some style inconsistencies with the rest of the Hadrian code base. (I'd like to emphasise that I make no judgement about which style is better, it's merely an attempt to make the code base look more homogeneous, which I think is useful.)
There are only two important changes.

The first one fixes missing dependencies:

-pkgDependencies = fmap (fmap PD.dependencies) . readPackageDataFile
+pkgDependencies = fmap (fmap (map pkgName . packageDependencies)) . readCabalFile

Here PD.dependencies returned versioned package names, e.g. ghc-8.7, which then failed to match with non-versioned package names such as ghc in contextDependencies. Switching from PD.dependencies to packageDependencies fixes this.

I clearly remember that we didn't have this bug before, so I added some tests for our package dependency infrastructure to prevent such regressions in future:

testDependencies :: Action ()
testDependencies = do
    putBuild "==== pkgDependencies"
    depLists <- mapM (pkgDependencies . vanillaContext Stage1) ghcPackages
    test $ and [ deps == sort deps | Just deps <- depLists ]
    putBuild "==== Dependencies of the 'ghc-bin' binary"
    ghcDeps <- pkgDependencies (vanillaContext Stage1 ghc)
    test $ isJust ghcDeps
    test $ pkgName compiler `elem` fromJust ghcDeps
    stage0Deps <- contextDependencies (vanillaContext Stage0 ghc)
    stage1Deps <- contextDependencies (vanillaContext Stage1 ghc)
    stage2Deps <- contextDependencies (vanillaContext Stage2 ghc)
    test $ vanillaContext Stage0 compiler `notElem` stage1Deps
    test $ vanillaContext Stage1 compiler `elem`    stage1Deps
    test $ vanillaContext Stage2 compiler `notElem` stage1Deps
    test $ stage1Deps /= stage0Deps
    test $ stage1Deps == stage2Deps

Everything else are cosmetic changes, fixing minor issues in comments, and adding TODOs.
Hope this helps!

@alpmestan Many thanks for the review! Merged.


change kata-operator-daemon container image

When code for the new operator was merged first we missed that
there is still a private test container image referenced in the
code.
This patch fixes it and changes the container image to
quay.io/isolatedcontainers/kata-operator-daemon:4.7
Signed-off-by: Jens Freimann<EMAIL_ADDRESS>

/lgtm

Maybe also relevant in this PR. Noticed that the links for the kata-operator-daemon does not work.


Android error, Working fine iOS

What went wrong:
A problem occurred evaluating project ':react-native-insta-story'.

Plugin with id 'maven' not found

"react-native": "0.67.2",
"react-native-insta-story": "^1.1.7",

Tested on "react-native" : 0.68.1 and it worked without problems. (Android Version )
try to delete the node_modules & package-lock.json then run npm i then expo r -c

I face same issue in  "react-native": "0.68.2",

Any solution on this?

I face same issue in "react-native": "0.68.2" Anyone have Solution Please Share Here.....

you can use new version(1.1.8)


[Docs] Document enabling Introspection data providers for UMA

Refer the content at [1]
[1] https://github.com/wso2/product-is/issues/4769

Updated doc:
https://docs.wso2.com/display/IS580/User+Managed+Access+with+WSO2+Identity+Server


load_in_8bit

model = AutoModelForCausalLM.from_pretrained("Model/Dolly_v2_3b",
device_map="auto",
load_in_8bit=True,
# torch_dtype=torch.bfloat16
)
My system is windows. Torch version - 1.13.1+cu116.
For above code execution I am getting error as:
RuntimeError:
CUDA Setup failed despite GPU being available. Please run the following command to get more information:
    python -m bitsandbytes

    Inspect the output of the command and see if you can locate CUDA libraries. You might need to add them
    to your LD_LIBRARY_PATH. If you suspect a bug, please take the information from python -m bitsandbytes
    and open an issue at: https://github.com/TimDettmers/bitsandbytes/issues

And Bitsandbytes Bug Report:
===================================BUG REPORT===================================
Welcome to bitsandbytes. For bug reports, please run
python -m bitsandbytes
and submit this information together with your error trace to: https://github.com/TimDettmers/bitsandbytes/issues
bin c:\workspace\nimish\environments\dolly_test\lib\site-packages\bitsandbytes\libbitsandbytes_cpu.so
False
CUDA_SETUP: WARNING! libcudart.so not found in any environmental path. Searching in backup paths...
CUDA SETUP: WARNING! libcuda.so not found! Do you have a CUDA driver installed?
CUDA SETUP: Loading binary c:\workspace\nimish\environments\dolly_test\lib\site-packages\bitsandbytes\libbitsandbytes_cpu.so...
LoadLibrary() argument 1 must be str, not WindowsPath
CUDA SETUP: Problem: The main issue seems to be that the main CUDA library was not detected.
CUDA SETUP: Solution 1): Your paths are probably not up-to-date. You can update them via: sudo ldconfig.
CUDA SETUP: Solution 2): If you do not have sudo rights, you can do the following:
CUDA SETUP: Solution 2a): Find the cuda library via: find / -name libcuda.so 2>/dev/null
CUDA SETUP: Solution 2b): Once the library is found add it to the LD_LIBRARY_PATH: export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:FOUND_PATH_FROM_2a
Please help.

As the error states, your CUDA library was not detected. Did you install cuda?


Simple Maze tutorial doesn't load

pxtapp.js:1 GET https://pxt.azureedge.net/api/md/arcade/tutorials/simple-maze?targetVersion=0.15.7&cdn=20200102 404
Repro:
Open arcade.makecode.com
Click "Simple Maze"
See error

@jwunderl  can you take a look?

Weird, opening on local serve. Worth noting something else is going wrong too as the card is out of date - this is how it should look / how it looks on localserve

I think it is something to do with the rename.

Oh wait, that makes this make sense; it's requesting /simple-maze instead of /maze and we wanted to change the title because simple sounded wrong. Anyone have any idea why the homepage is stale?

well it's no longer stale, updated for me on live in an incognito window; must have been a small delay?

New incognito window, first load:

CDN caching is for a day.
https://pxt.azureedge.net/api/md/arcade/tutorials?targetVersion=0.15.7&cdn=20200102
Unless your date is incorrect, it is going to serve the old URL.

Weirdly enough non-incognito chrome and safari show the right thing:

Ahh, seems like it worked cause i went to beta first in the new window, my bad. Is there anything to be done here other than wait? I suppose I can always rename the files back to simple maze and just call it maze everywhere?

Right solution is to add the old files too in addition to new files. And delete them after couple of days.

I don't understand what is happening. Why would the incognito mode see the old thing and yet the non-incognito see the new thing?

I think if you bump live it will fix it too, as beta is requesting with the new version so it was skipping the cache (i think) Ill put up a pr to just temporarily dup the files now though ~

Capture the network trace and see what https://pxt.azureedge.net/api/md/arcade/tutorials?... URL is being loaded


Add at-import-partial-extension autofix only when "never"

This PR adds the autofix support to the scss/at-import-partial-extension rule only when the "never" option is given.
I believe we cannot support autofix when "always" is given because it's impossible to determine file extensions to be added.

Thanks! More stuff for my review queue :D


Framing of sentence for PowerShell installation module

[Enter feedback here]
After the instruction of enabling SharePoint Embedded, the next line discusses about the SharePoint PowerShell Module. But, it is not mentioned explicitly about SharePoint PowerShell Module and directly instructs to Upgrade or Install. Please check the highlighted section in the below screenshot.

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

ID: b469eb4a-8a12-a1c2-aa28-052e4e8a3eeb
Version Independent ID: b469eb4a-8a12-a1c2-aa28-052e4e8a3eeb
Content: Hands on Lab - Setup & configure the SharePoint Embedded environment and web app project
Content Source: docs/embedded/mslearn/m02-03-hol.md
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

These docs are now moved to the new location, so closing this from here - to avoid confusion.
Valid finding and good feedback - will bypass to the owning team.


Added support for iAsyncDisposable

Closes #43

@BrentCarter could you have a look whether this changes would suffice for you? Thanks a lot!
Don't worry about the amount of changes. I needed to do an upgrade to .NET 5 in order to achieve this. The relevant Part is here
This will also result in a breaking change and therefore in an increase of the major version.

@BrentCarter could you have a look whether this changes would suffice for you? Thanks a lot!
Don't worry about the amount of changes. I needed to do an upgrade to .NET 5 in order to achieve this. The relevant Part is here
This will also result in a breaking change and therefore in an increase of the major version.

Thanks for getting into this.  I just took a look and see that IAsyncDisposable support was added to MvvmComponentBase, which is great, but I guess I was sort of expecting it would be on ViewModelBase as well.  Is there a reason to have it on the component but not the VM given the VM does implement the non-async version?  I'm still evaluating and learning different patterns for Blazor so I don't have an active project using this library yet, so I can't point a specific use I need it for...  but it seems like that might be useful.

Thanks for getting into this.  I just took a look and see that IAsyncDisposable support was added to MvvmComponentBase, which is great, but I guess I was sort of expecting it would be on ViewModelBase as well.  Is there a reason to have it on the component but not the VM given the VM does implement the non-async version?  I'm still evaluating and learning different patterns for Blazor so I don't have an active project using this library yet, so I can't point a specific use I need it for...  but it seems like that might be useful.

You're right, I missed that. Actually the IDisposable implementation in ViewModelBase serves no real purpose from looking at it. It seems like it was used at some point but I left the empty implementation. Gonna remove it then.

You're right, I missed that. Actually the IDisposable implementation in ViewModelBase serves no real purpose from looking at it. It seems like it was used at some point but I left the empty implementation. Gonna remove it then.


'update' function implementation

Describe the problem

While working with @devin-petersohn on implementing a progress bar, it seemed like the implementation of df.update() or other in-place operations might want to use different functions or names. Currently the progress bar says computation is being performed by a "caller" function.
Source code / logs

Hi @noah-kuo , thanks for reporting!
Looks like the progress bar says correctly because this function is being performed if we call df.update() or series.update(). You suppose that the progress bar should say computation is being performed by a "update" function?

Yes @YarShev, it would be better to show update, which is the pandas API that was called. We wrap the pandas.DataFrame.update in the caller function because it is inplace and Modin does not allow inplace updates.
I am not sure that there is a nice solution to this problem.

@devin-petersohn , okay, I will take a look on this.

We could do it this way (link). @devin-petersohn , what do you think?

#2092

Left a comment on your commit @YarShev

I have created the PR to simplify discussion. Current comments:
Devin: what about func.__name__ instead?
Yaroslav: There are some cases when we pass lambda as a parameter func to the copy_df_for_func. func.__name__ will be lambda in that case. We already have the such example for series_update. Perhaps, we will worth adding some docstring on parameter func_name in order to facilitate better understanding of other people what is func_name needed for. What do you think?


Challenge accepted.

Hi there, this is the code and web with the challenge solved.
Code:
https://github.com/matew17/random-faces
Web:
https://matew17.github.io/random-faces/
Thanks.

Don't call methods or put logic in constructors, use ngInit instead 😄

See you in the next challenge!


Change colour

Play around with themed colour examples.

The Browse page (https://ubc-lib-geo.github.io/maps-collection/browse.html) has a green search button when the page is loaded, but turns blue when the page is selected. No idea why that is, but the green color is hex code: #278838. Search for this color in the project files and change occurrences to desired blue.


Do you have a Keras 2 version？

This is a very useful project for me, I spent some time to modify some details to make sure it can run on Keras to like add "require_flatten=include_top". However, if you can provide a Keras 2 version that will be better.

But this is a Keras 2 model. Where does it show warnings that it is using Keras 1 format?

In Densenet.py line 108 : "include_top=include_top"
but in Keras document
https://github.com/fchollet/keras/blob/master/keras/applications/imagenet_utils.py
def _obtain_input_shape(input_shape,
                        default_size,
                        min_size,
                        data_format,
                        require_flatten,
                        weights=None):

Then when I run cifra100.py, met I an error said “"include_top=include_top" is an unexcept parameter. So   I modify it as "require_flatten=include_top" and make it to work.

This was changed in a commit 13 days ago. Thanks for bringing it up. I'll have to take some time to go through all of my model builders for that.

Most probably, scale layers wont be needed. Keras BatchNorm has scaling inbuilt. I will have to port the weights though, which will take time.