Error : Unable to resolve module child_process

"react": "16.2.0",
"react-native": "0.53.3",
"react-native-version": "^2.6.1",
I get a red screen in IOS simulator with this error:
error: bundling failed: Error: Unable to resolve module child_process from /Users/bhakta/dev/F/drprax/node_modules/react-native-version/index.js: Module does not exist in the module map

@bhakta0007 Hey, looks like you're importing this module into your app code. This module is meant for versioning your app in your Git project.

I saw an example usage on the site and I thought I could use the module to read the versions back into the code.
https://www.npmjs.com/package/react-native-version#api

That API is for developers who need to version their app as part of their own script. It only sets version/build numbers.
To display the app version inside your app to your users, you'll need something like: https://www.npmjs.com/package/react-native-version-number

Yes, I am now using react-native-version-number.
It would help if the README can clarify this point about the package.


Internal Data Model

The data model should be optimized for execution speed of the various FFT operations we need to do. For this it might be advantageous to align the data such that the dimension along which we do Fourier Transforms are aligned sequentially. Since Numpy uses C-style indexing by default, we could stick to that and do:
Azi x Ele x Pol x Freq x Element
This way the first three dimensions are also the "external" variables and the last two are those related to the array itself. For this it might be helpful to use xarray in order to get cleaner and more readable code. Xarray can also handle interpolation, which is nice for our needs.

This was changed to Azi x Ele x Freq x Pol x Element


root case - snap installation

The case in which you have chosen to install the snap package as root user(e.g when running default lxd container, hosting juju client only)

Lovely! Thank you


Check if localStorage works when extending Model.Local

https://github.com/spine/spine/issues/583.
This will check to see if localStorage is functional before extending Spine.Model.Local.

hm. is there as reason you didn't go with the seemingly simpler if window.localStorage check?
Curious what modernizr uses to check for this...

This is what modernizr using, https://github.com/Modernizr/Modernizr/blob/master/feature-detects/storage/localstorage.js#L37-L46.
When I tested on private Safari, window.localStorage is there but not functional.

okay, looks good. I might add a console log in the catch so developers would get some feedback on why it is not functioning now that the error will be swallowed up.

:+1:


Unable to create Linux datascience vm using azure cli

Hi Team,
I am trying to create a Linux data science vm using azure cli .
using the below commands I was able to get the urn for the image.
azure vm image list-publishers [options]
the command to create the vm:
azure vm create -g johan -n  datascience1 -Q  microsoft-ads:linux-data-science-vm:linuxdsvm:1.0.2 -o johandisks187 -l westeurope -f newnic -y Linux --plan-name microsoft-ads --plan-publisher linux-data-science-vm --plan-product microsoft-ads:linux-data-science-vm:linuxdsvm:
where we have given in the plan and publisher name and the product name.
But we are failing with the below details:
error for command1.txt
we have also tried using the below command:
azure vm create -g johan -n  datascience -Q  microsoft-ads:linux-data-science-vm:linuxdsvm:1.0.2 -o johandisks187 -l westeurope -f newnic -y Linux
info:    Executing command vm create

Looking up the VM "datascience"
Enter username:  azure
Enter password for azure: ************
Confirm password: ************
info:    Using the VM Size "Standard_DS1"
info:    The [OS, Data] Disk or image configuration requires storage account
Looking up the storage account johandisks187
Looking up the NIC "newnic"
info:    Found an existing NIC "newnic"
info:    Found an IP configuration with virtual network subnet id "/subscriptions/5eefe034-2471-4f40-96ea-e5e8a7dac305/resourceGroups/johan/providers/Microsoft.Network/virtualNetworks/johan-vnet/subnets/default" in the NIC "newnic"
info:    This is an NIC without publicIP configured
info:    The storage URI 'https://johandisks187.blob.core.windows.net/' will be used for boot diagnostics settings, and it can be overwritten by the parameter input of '--boot-diagnostics-storage-uri'.
Creating VM "datascience"
error:   Long running operation failed with error: 'Creating a virtual machine from Marketplace image requires Plan information in the request. OS disk name is cli5448053ac97934f2-os-1486473668342.'.
error:   Error information has been recorded to /root/.azure/azure.err
error:   vm create command failed

Here it shows the plan and publisher information.
error2command.txt
Could you please assist me in creatin the Linux data science vm,

Hi @kailashms, you are using Azure Xplat CLI, which is a different repo.  Please repost your issue here:
https://github.com/Azure/azure-xplat-cli
and they should be able to better assist you.


How do you install Alpha? Newb here

I don't know how to install it, beside a apk file

You can ask to join alpha testing on HA Client Discord server


crypto/rand/rand_win.c: include "e_os.h" to get the default _WIN32_WINNT

This helps decide if the BCrypt API should be used or not.
Fixes #8635

This looks ok, but I've got to wonder whether the default is right for 3.0. Should we really be compiling for XP support always?

Merged.
cc8926ec8fcecae89ceab91ef753de93e49568f9 crypto/rand/rand_win.c: include "e_os.h" to get the default _WIN32_WINNT

Oh, and cherry-picked to 1.1.1 (almost forgot...)
7a3c4b374bf5e7aa990ac473acba7db1c941f876 crypto/rand/rand_win.c: include "e_os.h" to get the default _WIN32_WINNT

This looks ok, but I've got to wonder whether the default is right for 3.0. Should we really be compiling for XP support always?

This is a good question, and one we should ask elsewhere.
(how far back do we want to support?  In general?  Per platform?)

This looks ok, but I've got to wonder whether the default is right for 3.0. Should we really be compiling for XP support always?

This is a good question, and one we should ask elsewhere.
(how far back do we want to support? In general? Per platform?)

Please take a look at  #8640 (and #8639).


Etcher freezing on "Starting..." dialog

Etcher version: 1.0.0-beta.17
Operating system and architecture: macOS 10.12.1 64-bit
Do you see any meaningful error information on DevTools? No

When trying to flash an image to an SD Card, Etcher freezes on the "Starting..." dialog on the flash button:

No errors come up in DevTools or the GUI...

...but if I try running the command from the DevTools in Terminal.app it comes up with this error message:

Any help would be appreciated. Thanks!

Hm, very weird. So the command exits with an error code other than zero, but we don't know anything else about it.
BTW, did you run the command after closing Etcher? If so, the error might be due to the IPC server ID not longer being valid, and that might be obscuring the original issue.
If you can reliably reproduce (let me know if not), you should be able to execute the command while Etcher is frozen, but without closing it, and see if that reveals anything else.

Okay, I have a lot to say, so bear with me:
Thank you for your reply (especially this close to Christmas! 🎄)! I did run the command in Etcher while the Terminal was closed. So, I ran it while Etcher was open as you said and that just freezes too (it freezes on 'diskutil' at the top'). No errors:

As for reproducibility, I know I used Etcher to flash the iso of the Debian+PIXEL image to my USB stick and this worked successfully. In this case (where Etcher is freezing), I am trying to flash an img (LibreELEC for RPi Zero) to my SD card. The download from LibreELEC came as a img.gz. I have tried flashing the img.gz as it is and unzipped as a img. In both cases Etcher freezes. I have also tried flashing the Debian+PIXEL image to the SD card. It Freezes 😢 ! And if I try to plug in my USB stick from earlier (to reproduce the scenario where it worked before) , macOS doesn't recognise it (probably because it has partitions etc after flashing Debian+PIXEL onto it). This suggests to me there's a problem with flashing to SD cards. My SD card in particular is a micro SD to SD card adapter. The micro SD card inside is a micro SDHC I 8GB class 6 (It doesn't say the brand). Let me know if you need any more info.

Have you tried, or can you try, to flash a different SD card? Perhaps the
issue is with the card itself? Etcher shouldn't behave this way for sure,
but i wonder if it's to do with all SD cards on your machine or just this
specific one has some special factor that's causing the behavour.
--
Alexandros Marinos
Founder & CEO, Resin.io
+1<PHONE_NUMBER>
@alexandrosm
On Sat, Dec 24, 2016 at 10:57 AM, Facel3ss1<EMAIL_ADDRESS>wrote:

Okay, I have a lot to say, so bear with me:
Thank you for your reply (especially this close to Christmas! 🎄)! I did
run the command in Etcher while the Terminal was closed. So, I ran it while
Etcher was open as you said and that just freezes too (it freezes on
'diskutil' at the top'). No errors:
[image: screen shot 2016-12-24 at 10 07 29]
https://cloud.githubusercontent.com/assets/16763503/21466362/e42e98e6-c9c0-11e6-9478-e564a628f2dd.png
As for reproducibility, I know I used Etcher to flash the iso of the Debian+PIXEL
image https://www.raspberrypi.org/blog/pixel-pc-mac/ to my USB stick
and this worked successfully. In this case (where Etcher is freezing), I am
trying to flash an img (LibreELEC for RPi Zero
https://libreelec.tv/downloads/) to my SD card. The download from
LibreELEC came as a img.gz. I have tried flashing the img.gz as it is and
unzipped as a img. In both cases Etcher freezes. I have also tried flashing
the Debian+PIXEL image to the SD card. It Freezes 😢 ! And if I try to
plug in my USB stick from earlier (to reproduce the scenario where it
worked before) , macOS doesn't recognise it (probably because it has
partitions etc after flashing Debian+PIXEL onto it). This suggests to me
there's a problem with flashing to SD cards. My SD card in particular is a
micro SD to SD card adapter. The micro SD card inside is a micro SDHC I 8GB
class 6 (It doesn't say the brand). Let me know if you need any more info.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/resin-io/etcher/issues/985#issuecomment-269079620,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLUCPk9wXVopwYZcvMkh5i1feYbM21tks5rLPq3gaJpZM4LUsF2
.

Can you show us the exact diskutil error you see?
I guess this could indicate some issues when unmounting the drive. What happens if you manually unmount using diskutil unmountDisk /dev/diskN? Does it hang as well?
Regarding recovering your drive, you should be able to reformat using the Disk Utility.app.
Let us know how that goes! Merry Christmas!

If manually unmounting works fine, I'll create a custom build with extra debugging information.

Hello again, sorry for the long reply (with Christmas & New Year etc). @alexandrosm I have tried using a different SD card and it flashed successfully 😄 !! The SD card in question was a Maplin 8GB micro SD card (I am using the same adapter as before)

ygu

Sorry about the close and reopen, my computer spazzed :/

@jviotti I have tried with the original SD card and it's working again as before 😕 . I know this doesn't help in terms of reproducability. One difference I noted was that the disk it was using was different. Before when it wasn't working is ws showing /dev/disk4 (You can see this in the screenshot above). Now, it is showing my SD card as /dev/disk1. And the diskutil 'error from before wasn't an error. Terminal.app shows the current running process in the title bar, and it had froze on 'diskutil' when it wasn't working before. I thank you for your help anyway! Keep up the good work!

Ugh, it sucks to not be able to reliably reproduce. I guess this could have happen if there was a file descriptor opened to the drive which diskutil was unable to close even with the force option we pass.
Do you remember if any application was using the drive in question while you were trying to flash to it?

@jviotti I don't think so. I cannot remember using another program apart from Etcher... If I was using another program (which is unlikely), it would be Finder. If you have any other queries, I would be happy to respond. Also, if this behaviour occurs again, then I can record that here if you want me to.

The diskutil command we run is:
/usr/sbin/diskutil unmountDisk force /dev/diskN

where N corresponds to the disk number.
How long did you wait for diskutil before killing the command? It might be worth to try manually running that command and see if you can manage to reproduce that way.


Missing json requirement

irb(main):001:0> require 'packagecloud'
NameError: uninitialized constant JSON
	from /root/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/packagecloud-ruby-0.2.22/lib/packagecloud/client.rb:1:in `<top (required)>'
	from /root/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /root/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
	from /root/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/packagecloud-ruby-0.2.22/lib/packagecloud.rb:2:in `<top (required)>'
	from /root/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:128:in `require'
	from /root/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:128:in `rescue in require'
	from /root/.rbenv/versions/2.2.3/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:39:in `require'
	from (irb):1
	from /root/.rbenv/versions/2.2.3/bin/irb:11:in `<main>'

We just released 0.2.23 which should solve this issue: https://rubygems.org/gems/packagecloud-ruby/versions/0.2.23


Contribute best Python Scripts written by you

Contribute best Python Scripts written by you.

I want to take up this issue.

@1PunMan sure go ahead :+1:

Should we raise PR after forking project?

do you take NLP type stuff?

Wanna work on it!!


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/iobroker-community-adapters/ioBroker.nello

[ ] [E019] Invalid repository URL: git+https://github.com/Zefau/ioBroker.nello.git. Expected: git+https://github.com/iobroker-community-adapters/ioBroker.nello.git
[ ] [E150] No common.connectionType found in io-package.json
[ ] [E152] No common.dataSource found in io-package.json
[ ] [E605] No actual year found in copyright. Please add "Copyright (c) 2018-2022 zefau<EMAIL_ADDRESS>at the end of README.md
[ ] [E701] No actual year found in LICENSE. Please add "Copyright (c) 2022 zefau<EMAIL_ADDRESS>at the start of LICENSE

I have also found warnings that may be fixed if possible.

[ ] [W400] Cannot find "nello" in latest repository

Thanks,
your automatic adapter checker.

obsolete


Ktor 3 compatibility

I have attempted to port Kodein to Ktor 3 myself, as only two changes were required, however one test is failing: https://github.com/stashymane/Kodein/commit/89a4d60fd72ecd24d4e0cd5c2caad05660559413

It seems that the call pipeline is being skipped for some reason. Otherwise the update is rather simple.
Here's the run for details: https://github.com/stashymane/Kodein/actions/runs/8818984587/job/24209096046

I get errors with kodein 7.22.1 and ktor 3.0.0
Found interface io.ktor.server.routing.Routing, but class was expected
java.lang.IncompatibleClassChangeError: Found interface io.ktor.server.routing.Routing, but class was expected
	at org.kodein.di.ktor.ClosestKt.closestDI$lambda$2(closest.kt:32)

I still get the same errors:
Found interface io.ktor.server.routing.Routing, but class was expected
java.lang.IncompatibleClassChangeError: Found interface io.ktor.server.routing.Routing, but class was expected
	at org.kodein.di.ktor.ClosestKt.closestDI$lambda$2(closest.kt:32)

Do you need anything?

Ktor 3 Compatibility has not been released in a stable release.
You can try it on the snapshot channel.

Ah, thanks, I somehow thought it was part of 7.23.1. Sorry!

@romainbsl, any news about stable version compatible with ktor 3.0.+ ?

It is available on SNAPSHOT, and it will probably come as is. I plan to work on Ktor plugin registry and compose related features before releasing a new version.
In the meantime, you can use it from the snapshot channel.
maven repository: https://s01.oss.sonatype.org/content/repositories/snapshots
target version of Kodein: 7.24.0-main-SNAPSHOT


[Go] When using the dotprompt plugin with a ModelName config, it will panic

Describe the bug
When creating a prompt with a ModelName Config, the provided model name is not considered and it will panic. This is due to a bug when looking up the model in:
https://github.com/firebase/genkit/blob/2940fd520ebde799c74581921546b655b2cab898/go/plugins/dotprompt/genkit.go#L207-L225
The model is shadowed inside the if statement and result to the lookedup model going out of context after the if statement.
To Reproduce
prompt, _ := dotprompt.Define("menu", `Suggest an item for the menu of a {{ type }} themed restaurant`, dotprompt.Config{ModelName: "googleai/gemini-1.5-flash", InputSchema: r.Reflect(Input{}), OutputSchema: os, OutputFormat: ai.OutputFormatJSON})

resp, _ := prompt.Generate(ctx,
			&dotprompt.PromptRequest{
				Variables: input,
				Config:    &ai.GenerationCommonConfig{Temperature: 1},
			},
			nil,
		)

Expected behavior
When a model name is provided, either use the provided model name or return an error if the model lookup fails.
Runtime (please complete the following information):
Not relevant
** Go version
Not relevant
Additional context
The panic will happen in line:
https://github.com/firebase/genkit/blob/2940fd520ebde799c74581921546b655b2cab898/go/plugins/dotprompt/genkit.go#L227
This is similar to https://github.com/firebase/genkit/issues/847 but not quite the same issue.

I think the fix is pretty simple, just replacing
model := ai.LookupModel(provider, name)

with
model = ai.LookupModel(provider, name)

Please confirm and I can open a PR with the fix.

@dhiaayachi Hi, I was able to reproduce the error you mentioned. It seems you're right about both the cause and the solution. I think it's a good idea to create a PR for it.
It would also be better to write an additional test, similar to the one below, to ensure that passing a ModelName config works as expected:
https://github.com/firebase/genkit/blob/main/go/plugins/dotprompt/genkit_test.go#L46


Provide Empty state for when there are no results

Description
When there are no results in the filter, there is no empty state to allow the user to know that the page is blank because there is nothing there.  We should show an empty state, and announce this state to screen reader users.

Acceptance Criteria

[ ] Show an empty state (ex. No results found) when there are no results after applying filters.
[ ] Use aria-live region to announce the state

Can I get assigned to this task?

@KenMain5 assigned!

Is there anyone I can talk to on how to design this? or what it should look etc?

Is there anyone I can talk to on how to design this? or what it should look etc?

HI @thansidwell and @marvieqa, do you have wireframes or suggestions for how this should look like in the UI?

@CodeWritingCow @marvieqa Yes! Here's the design. https://www.figma.com/design/NAFkgq34abW6uJ0R7PW24T/Prototype---Clean-%26-Green-Philly?node-id=6265-35268&t=B1Q1D5C9HzoiZWLE-1

mb I've been busy, I'll get it done this weekend!

@KenMain5 no worries! You have 2 other tickets on your plate already. If you feel like you're taking on more than your time capacity allows, I can take this off your plate so you can focus on the other tickets. Also, @AZBL is looking to pick up a ticket, so maybe you and Alex can chat on Slack and work something.

@CodeWritingCow okay I'll work something out with him!

Per conversation with @KenMain5, I've unassigned Kenny from this ticket. He's working on other tickets, so this one is open for other developers to take.

Hello, can I pick up this issue?

@Michael-Dratch it's yours!


Animation go continuous and smooth?

Hello,
is it possible to make the transition/animation
go continuously and smooth? (Like a news ticker) from right to left.
Thanks

You can set rn-carousel-easing to change the easing function.
By doing so you could set it to a linear function and set rn-carousel-duration and rn-carousel-auto-slide to an equal duration to get a smooth transition (not tested but should work).
I hope you get the idea (if you have any problems just ask back)

Thanks for your reply. Doesnt work :(  -
This is my setup:
rn-carousel-auto-slide="0.1"
rn-carousel-duration="0.1" //cause 0 doesnt work
rn-carousel-loop
rn-carousel-easing="easeFromTo"
Any other suggestions ?
I really like this slider

Could you set up a fiddle for this?


Unexpected line and block comment behaviour in last case of switch

What version of Go are you using (go version)?

$ go version
go version go1.11.4 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/local/ZOHOCORP/mani-pt2396/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/local/ZOHOCORP/mani-pt2396/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build688297931=/tmp/go-build -gno-record-gcc-switches"

What did you do?
Commented out the last case of switch with line or block comment(s).

https://play.golang.org/p/fkseigRrxYV
What did you expect to see?
The last case(s) to be in their place and not go inside the scope of the previous case.
	switch val {
	case foo1:
		foo()
	case foo2:
		foo()
	// case bar:
		// bar()
	}

	switch val {
	case foo1:
		foo()
	case foo2:
		foo()
	/* case bar:
		// bar()*/
	}

What did you see instead?
The last case(s) is aligned as if it is inside the scope of the previous case.
	switch val {
	case foo1:
		foo()
	case foo2:
		foo()
		// case bar:
		// bar()
	}

	switch val {
	case foo1:
		foo()
	case foo2:
		foo()
		/* case bar:
		// bar()*/
	}

Maybe good to fix, but the heuristics are complicated. How do we know whether we are not actually commenting out code inside the last case ?
Although I agree we should be consistent in the indentation whether we are in the middle of a switch-case or at the end. Currently, it seems there is an inconsistency.
package main

func main() {
	val := 1
	switch val {
	case 1:
		_ = 1
	// case 5: // gets aligned to the case
	// 	_ = 5
	case 2:
		_ = 2
		// case 3: // gets aligned to the code block inside the case
		// 	_ = 3
	}
}

@griesemer

I’m 95% confident this is a dup. On my phone and can’t find the original (apologies?), but there has been significant discussion of indent, gofmt, and switch cases on some other issue.

I’m 95% confident this is a dup. On my phone and can’t find the original (apologies?), but there has been significant discussion of indent, gofmt, and switch cases on some other issue.

Are you referring to this one #28057?

Yes. And in a related vein there's also https://github.com/golang/go/issues/20681 and https://github.com/golang/go/issues/9910. And perhaps others.

Leaving open for future consideration.  Unfortunately, gofmt work is not currently a priority for us as long as code is not broken by it, so theses issues will linger a bit. The good news is that when we eventually get to addressing these, we have a lot of corner cases to test against. And once fixed, it's trivial to "fix" huge corpuses of code in one fell swoop.


SSH auth with password?

Hello, I was wondering if there are plans to support ssh/& openssh connections with password(with keys)
Is it intentionally not supported?
I suppose I can build my own AuthMethod to patch that in, but it would be nice to have it in the core config.

I believe that you have two options:

use ssh-agent with the key added
use your own AuthMethod as you suggested.

I think this is something that should not be done, ssh has already solved this with ssh-agent.

I probably badly worded it there. I meant without keys(eg. if the only option to connect is with passwords) I suppose I'll setttle on the AuthMethod. Thanks

I do wonder if I'm able to do something similar to an auth method with the openssh?
AFAIK ssh-agent can't deal with password only auth(and for various reasons, keys might not always be an option)


[Bug]: 0 spending balance & Loss of funds when restoring wallet (LN Channel closed)

Describe the bug
some UX issues not sure if its a bug but when I open a channel, I receive notification that Instant Payments are available - its says open connection but on the home screen it says pending balance. The app was ON during the entire time. Positive side is it took less than 5 min for Bitkit to open a channel. But then under spending balance I can see 0 - mixed messages (see screenshots).
I backed up wallet and try to wipe and restore. Initially app crashed without providing option to send a report. I then restored the wallet only to find out there was no channel opened (see screenshot), amount was taken and now missing after restore (wallet has not indicated it was even LN tx).
Reproduce

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots / Recording

bitkit_ldk_logs_1692108700214.zip

Operating system
iPhone 6S
Bitkit version
v63
Log output
No response

I encountered the same issue. Even though the transaction status had been confirmed of 11 times, my spending balance was 0 sats. After the 11th confirmation, the balance updated, and I am now able to perform Lightning transactions without any issues.

I think the zero spending balance issue might be just the app state not auto updating and being in sync with LDK. The restore bug is separate.

some UX issues not sure if its a bug but when I open a channel, I receive notification that Instant Payments are available - its says open connection but on the home screen it says pending balance. The app was ON during the entire time. Positive side is it took less than 5 min for Bitkit to open a channel. But then under spending balance I can see 0 - mixed messages (see screenshots).

The first part is a known issue and tracked here: https://github.com/synonymdev/bitkit/issues/1261
Since the channel wasn't actually open yet (it is under pending in the screenshot) we can't add the local channel balance to the overall spending balance since you cannot use it yet. So working as expected here.
Assigning to @Jasonvdb for the second part

Closing this, seems outdated.


Optimize ParagraphText's background and underline shapes

Coming from Tomas' comment in 411, optimize ParagraphText to only instantiate its backgroundShape and underlineShape objects when needed.

This was more properly closed by #558. The shapes were further optimized by only using one shape per shared value between multiple consecutive TextExt nodes rather than one per segment.


Deprecate channel pool

Publishing with emque-producing will fail in certain instances due to the implementation of the channel pool.

When producing more than the 20 messages quickly, there will not be a channel available and the message will fail.
After losing a connection to rabbit (or when rabbit fails and stops), the connection may be able to be re-established but the channels will then need to be re-opened too. This may be unnecessary complicating Bunny and we can look to with_channel or other ideas for improving channel management.

Note the original reason for the channel pool was that under heave load without the pool, too many channels would be created on the Rabbit server and it would cause memory issues on the Rabbit server.

We are seeing producing errors in production and I'm working on getting a better, more specific error for this. Some more discussion here
https://groups.google.com/forum/#!topic/ruby-amqp/y-qeN-3c2fw

Closing with the version 1.1.1 fix.


Add maven exclusion for netty transitive dependency in artemis

Fixes issue noted in https://github.com/openmessaging/benchmark/issues/363 , which was blocking me in production as well as locally
Currently the benchmark-framework module is unable to create the worker pool when given a list of workers due to a transitive dependency on netty-transport==4.1.77.Final which is pulled in from the artemis client, which conflicts with asynchttpclient's dependency on netty
This excludes all netty dependencies that the artemis library pulls in which resolves the issue. The artemis driver seems to be functional still without this transitive dependency.
You can reproduce this issue locally by starting benchmark-worker and attempting to run any benchmark using those workers. Adding the exclusion and rebuilding allows the benchmark to work as intended.
This also seems to resolve an issue I was encountering where the benchmark will fail to terminate the java process once it completes. The bin/benchmark executable now terminates successfully upon completion.

@merlimat @teabot Thank you for any attention to this


Website displaying incorrect info and broken images

Your install steps are incorrect and your hero image is broken.

Hah, we still don't have a logo for it so far.
Will update the installation though, thanks!


Update example-usage.md to remove reference to simd and rust nightly.

Which issue does this PR close?

Minor doc update
Rationale for this change
Doc cleanup
What changes are included in this PR?
example usage doc cleanup.
Are these changes tested?
Nothing to test.
Are there any user-facing changes?

I think arrow-rs used to have a simd feature that required nightly, but the auto vectorization of the rust compiler is now good enough there is no need to do it manually.

Thanks @comphead  and @Omega359


CUDA_ERROR_NOT_INITIALIZED error when using Multiple GPUs with multiprocessing

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 1,13
Python version: 3.7
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA 10, cuDNN 7.7
GPU model and memory: 2x NVIDIA Tesla T4, 16GB GDDR5

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
Hello, I am trying to parallelize a machine learning algorithm by using multiprocessing for my TensorFlow session. This way, I can delegate one GPU for each process (and it doesn't allocate all of the memory and combine the two GPUs into one at the beginning.) However, my methods are currently yielding a CUDA_ERROR_NOT_INITIALIZED.
Describe the expected behavior Each multiprocessing Process should be only able to see one GPU and be forced to use that: however, neither are being detected.
Code to reproduce the issue
import multiprocessing

class AlignProcess (multiprocessing.Process):
    def __init__(self, gpu ):
        multiprocessing.Process.__init__(self)
        self.gpu = gpu

def run(self):
    if self.gpu is not None:
        os.environ["CUDA_VISIBLE_DEVICES"]=str(self.gpu - 1)

        import tensorflow as tf
        session = tf.Session()
        session.run(tf.global_variables_initializer())
        session.close()

# Later on in code

AlignProcess(1).start()

Other info / logs
Using cuDNN version 7600 on context None
Mapped name None to device cuda: Tesla T4 (0000:00:04.0)
INFO:root:Working on recording /home/user/program/media/recordings/6.wav
Running source separation for recording: /home/user/program/media/recordings/6.wav
Testing...
WARNING:tensorflow:From /home/user/program/src/align/wavenet/Models/UnetAudioSeparator.py:99: conv1d (from tensorflow.python.layers.convolutional) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.conv1d instead.
WARNING:tensorflow:From /home/user/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
2019-06-17 14:05:59.513204: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 AVX512F FMA
2019-06-17 14:05:59.518744: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-06-17 14:05:59.519588: I tensorflow/compiler/xla/service/service.cc:150] XLA service 0x55ebf146ee30 executing computations on platform Host. Devices:
2019-06-17 14:05:59.519622: I tensorflow/compiler/xla/service/service.cc:158]   StreamExecutor device (0): <undefined>, <undefined>
2019-06-17 14:05:59.623164: E tensorflow/stream_executor/cuda/cuda_driver.cc:300] failed call to cuInit: CUDA_ERROR_NOT_INITIALIZED: initialization error
2019-06-17 14:05:59.623244: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:161] retrieving CUDA diagnostic information for host: wavenet-base-2gpu-t4-4
2019-06-17 14:05:59.623251: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:168] hostname: wavenet-base-2gpu-t4-4
2019-06-17 14:05:59.623412: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:192] libcuda reported version is: Not found: was unable to find libcuda.so DSO loaded into this program
2019-06-17 14:05:59.623465: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:196] kernel reported version is: 418.67.0
Num of variables64
WARNING:tensorflow:From /home/user/anaconda3/lib/python3.7/site-packages/tensorflow/python/training/saver.py:1266: checkpoint_exists (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
INFO:tensorflow:Restoring parameters from /home/user/program/src/align/wavenet/../../models_english/wavenet_full_44KHz/full_44KHz-236118
Using cuDNN version 7600 on context None
Pre-trained model restored for song prediction
Mapped name None to device cuda: Tesla T4 (0000:00:04.0)
INFO:root:Working on recording /home/user/program/media/recordings/7.wav
Running source separation for recording: /home/user/program/media/recordings/7.wav
Testing...
WARNING:tensorflow:From /home/user/program/src/align/wavenet/Models/UnetAudioSeparator.py:99: conv1d (from tensorflow.python.layers.convolutional) is deprecated and will be removed in a future version.
Instructions for updating:
Use keras.layers.conv1d instead.
WARNING:tensorflow:From /home/user/anaconda3/lib/python3.7/site-packages/tensorflow/python/framework/op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
2019-06-17 14:06:04.528660: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 AVX512F FMA
2019-06-17 14:06:04.551547: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-06-17 14:06:04.552053: I tensorflow/compiler/xla/service/service.cc:150] XLA service 0x55ebf3709e10 executing computations on platform Host. Devices:
2019-06-17 14:06:04.552076: I tensorflow/compiler/xla/service/service.cc:158]   StreamExecutor device (0): <undefined>, <undefined>
2019-06-17 14:06:04.670544: E tensorflow/stream_executor/cuda/cuda_driver.cc:300] failed call to cuInit: CUDA_ERROR_NOT_INITIALIZED: initialization error
2019-06-17 14:06:04.670736: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:161] retrieving CUDA diagnostic information for host: wavenet-base-2gpu-t4-4
2019-06-17 14:06:04.670788: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:168] hostname: wavenet-base-2gpu-t4-4
2019-06-17 14:06:04.670981: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:192] libcuda reported version is: Not found: was unable to find libcuda.so DSO loaded into this program
2019-06-17 14:06:04.671078: I tensorflow/stream_executor/cuda/cuda_diagnostics.cc:196] kernel reported version is: 418.67.0
Num of variables64
WARNING:tensorflow:From /home/user/anaconda3/lib/python3.7/site-packages/tensorflow/python/training/saver.py:1266: checkpoint_exists (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
INFO:tensorflow:Restoring parameters from /home/user/program/src/align/wavenet/../../models_english/wavenet_full_44KHz/full_44KHz-236118
Pre-trained model restored for song prediction

Hello,
I figured it out! This error was happening because I was importing TensorFlow, and thusly, cuDNN, from earlier on in my program.

How did you fixwd it?

Hello,
I figured it out! This error was happening because I was importing TensorFlow, and thusly, cuDNN, from earlier on in my program.

hello, How do you slove this problem?

I've got this warning too (along with the frozen training process) when I tried to train maskRcnn on coco dataset.
It's strange but I removed import tensorflow as tf from the begining of my code and after getting a lot of this error again, finally they stopped and the model continued to training!
Of course it's not a good solution because I need to import tf anyway, but maybe that helps somebody.


update method for firebase.service

hi, here is a pull request for firebase update method, (straight forward, analogous to the save method)
the update method is needed in addition to save method as the user may want to change only some of his data. which is not possibile in save method witch replaces whole object completely.

It looks good. Do you think you could write a couple simple tests for your update method? The tests should just make sure that the service actually returns a promise, and handles the Firebase callback properly.
They should be very easy to implement, you could probably even copy the setData() tests  from the FirebaseService Spec.
If you don't want to, I'd still be happy to merge your changes. :)

@pawelgorka  Whoops, it looks like I missed the fact that you already added tests. I'll Merge it right now!


move the attribute processing to save()

WHY
BEFORE - What was wrong? What was happening before this PR?
As reported in https://github.com/Laravel-Backpack/CRUD/issues/5579 calling the makeSureXXXHasAllAttributes in the constructor would lead to "unecessary" calls to that function. In a sense that only the first call does the job, the others would just return early from a bunch of functions.
AFTER - What is happening after this PR?
I've removed the duplicated calls by not calling the method in the CrudField/CrudColumn constructor, it will still run the "mandatory" call to initialize the field/column in the CrudPanel.

Hi @pxpm,
Just to clarify some things regarding the incident reported in https://github.com/Laravel-Backpack/CRUD/issues/5579
The admin panel was loading slow mainly because the database was responding slow.
The thing was backpack was trying to guess the rest of the attributes based on the field name although the column was fully configured in the controller.
To give you an example:
CRUD::column('id')->type('text')->label('ID');

I expect here that backpack would guess the remaining of the attributes (type and label are set by me).
What really happens is backpack start guessing at this point CRUD::column('id') and then the rest are overwrites ->type('text')->label('ID');
Again, the main issue for me was the database, but still, backpack, when using fluent syntax, should ONLY guess the necessary attributes after i stop with the column configuration, at this point CRUD::column('id')->type('text')->label('ID'); to be specific.
To summarize, that function should be called when the column is displayed not configured.

Indeed, that's because Backpack assumes the whole column is inside column() call. All the other function calls are only setters.
This was done to keep backwards compatibility while slowly introducing the fluent methods. Ideally in the future (we already discussed and planned something for that), when we have "operation hooks" (just made this name, we have no name for it yet), we can do like you said, and only call those methods at the end of the operation setup.
For now, the "fix" for your problem is not let the database get queried by defining identifiableAttribute() in your model. https://backpackforlaravel.com/docs/6.x/crud-fields#identifiable-attribute
Cheers


Hash map exception thrown when applying a toggle text attribution command using custom attributions

I have a demo where I toggle some custom color attributions with the following code:
_editor.executeCommand(ToggleTextAttributionsCommand(
  documentSelection: selection,
  attributions: attributeToggles,
));

If I try and toggle attributes on text that already has some attributes I get the following exception:
The following ConcurrentModificationError was thrown while dispatching notifications for
ValueNotifier<TextStyle>:
Concurrent modification during iteration: Instance of '_CompactLinkedHashSet<dynamic>'.

When the exception was thrown, this was the stack:
#0      _CompactIterator.moveNext (dart:collection-patch/compact_hash.dart:473:7)
#1      AttributedSpans.hasAttributionsWithin (package:super_editor/src/infrastructure/attributed_spans.dart:70:33)
#2      AttributedText.hasAttributionsWithin (package:super_editor/src/infrastructure/attributed_text.dart:56:18)
#3      ToggleTextAttributionsCommand.execute (package:super_editor/src/default_editor/text.dart:792:25)
#4      DocumentEditor.executeCommand (package:super_editor/src/core/document_editor.dart:38:13)
#5      _SuperTextEditorState._toggleStyleForSelection
(package:flutter_folio/styled_widgets/super_text_editor.dart:170:13)
#6      _SuperTextEditorState._handleStyleChanged
(package:flutter_folio/styled_widgets/super_text_editor.dart:180:5)
#7      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:308:24)
#8      ValueNotifier.value= (package:flutter/src/foundation/change_notifier.dart:412:5)
#9      _NetworkedScrapboardState._handleScrapStyleChanged
(package:flutter_folio/views/editor_page/networked_scrapboard.dart:142:20)
#10     _NetworkedScrapboardState.build.<anonymous closure>.<anonymous closure>
(package:flutter_folio/views/editor_page/networked_scrapboard.dart:77:48)
#11     _ScrapPopupEditorState._handleFgColorChanged
(package:flutter_folio/views/editor_page/scrap_popup_editor/scrap_popup_editor.dart:223:26)
#12     ScrapPopupPanelColor.build.<anonymous closure>.<anonymous closure>
(package:flutter_folio/views/editor_page/scrap_popup_editor/scrap_popup_panel_color.dart:43:57)
#13     _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:989:21)
#14     GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:193:24)
#15     TapGestureRecognizer.handleTapUp (package:flutter/src/gestures/tap.dart:608:11)
#16     BaseTapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:296:5)
#17     BaseTapGestureRecognizer.acceptGesture (package:flutter/src/gestures/tap.dart:267:7)
#18     GestureArenaManager.sweep (package:flutter/src/gestures/arena.dart:157:27)
#19     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:444:20)
#20     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:420:22)
#21     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:287:11)
#22     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
#23     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:338:5)
#24     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
#25     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
#29     _invoke1 (dart:ui/hooks.dart:182:10)
#30     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:282:7)
#31     _dispatchPointerDataPacket (dart:ui/hooks.dart:96:31)
(elided 3 frames from dart:async)

The ValueNotifier<TextStyle> sending notification was:
  ValueNotifier<TextStyle>#57e3a(TextStyle(inherit: true, color: Color(0xff29fd2e), backgroundColor:
  Color(0x00000000), family: Caveat, weight: 400, style: normal))
════════════════════════════════════════════════════════════════════════════════════════════════════

The color attribution class:
class ColorAttribution extends Attribution {
  @override
  String get id => 'color ${background ? "bg" : "fg"}';

  Color color;
  bool background;

  ColorAttribution({this.color = Colors.black, this.background = false});

  @override
  bool canMergeWith(Attribution other) {
    if (other.id == id && (other as ColorAttribution).color == color && other.background == background) {
      return true;
    }
    return false;
  }
}

It seems that this also can happen with NamedAttributions as well but I don't have a repro for that.

@cenomla can you provide a minimal reproduction of the issue?


Add adversrially regularization loss for ARGVA exmaple

Currently, examples/argva_node_clustering.py doesn't consider the adversrial regularization loss.
Done
[Major Fix] Add model.reg_loss(z)
Without model.reg_loss(z), it runs same as VGAE.
[Minor Fix] Remove discriminator.train()
model.train() executes train methods of its children which include discriminator.
Test Results
Consol Log
log.txt
Plot

Thanks :)


add acados and replay on Darwin

Thanks for the PR, but we generally like to build generated things that get checked in ourselves out of caution. #23048 is a more complete version of this.


test render template

Hi @todak2000,
I'm glad that you found this example repo useful as a starting ground for your own project, but I don't think it is appropriate to make your app be the example that others build off of.
If you run into any hurdles running your project on Render, be sure to reach out to<EMAIL_ADDRESS>and we can help you get things sorted. Happy hacking!


Fix child_process event name typo in nodejs typings

disconnet -> disconnect

I did a simple find + replace within the Github interface. Hope that's all that's necessary / that's okay.
There was no corresponding issue for this PR.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@joaomoreno This seems to have come in with the Electron update.

Thanks!


feat(pipes): add steamid or object id pipe

TBD: should this fetch data, or only validate data?
this should only validate the types of data, not the underlying database access (I think?) per nest's standards. Either needs research or needs approval.

Implemented by #1454

ty


Colors and icons

Besides some cleanup this mainly includes three things:
a) Launch screen - shows logo while loading without adding any noticeable overhead

b) Updated location markers and new accent color - I'm not convinced about the choice of colors here. Maybe @zutrinken can give some input here :)

merging this for now - color changes can be made later :)


Add my name for Challenge 1!

Hi @aspittel!
Once you have the challenge result, you will compare it to the challenge you created. And after make all the necessary changes in your code, we will close this PR.
Remember that all challenges must be developed in your repository. That's why is very important to fork this repository.
And thank you for your awesome contribution @aspittel!

We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This PR is now closed! 🎉


Refactor Relays Model for  v2 architecture

We must improve the model and refactore the tests.

Solved in commit


ERROR:mteb.evaluation.MTEB:Error while evaluating QuoraRetrieval: 'SimCSEWrapper' object has no attribute 'stop_multi_process_pool'

Hi! Thanks for this easy-to-use repo!
While I'm getting this error when running the example script https://github.com/embeddings-benchmark/mtebscripts/blob/main/run_array_simcse.py on retrieval benchmarks like QuoraRetrieval.
How do I evaluate on retrieval tasks when using my own model with a wrapper?

Hi! Thanks for this easy-to-use repo! While I'm getting this error when running the example script https://github.com/embeddings-benchmark/mtebscripts/blob/main/run_array_simcse.py on retrieval benchmarks like QuoraRetrieval. How do I evaluate on retrieval tasks when using my own model with a wrapper?

Hey sorry for the trouble, can you try with beir==1.0.0?

Hi! Thanks for this easy-to-use repo! While I'm getting this error when running the example script https://github.com/embeddings-benchmark/mtebscripts/blob/main/run_array_simcse.py on retrieval benchmarks like QuoraRetrieval. How do I evaluate on retrieval tasks when using my own model with a wrapper?

Hey sorry for the trouble, can you try with beir==1.0.0? I also just updated the mteb package not to use parallel encoding, which should solve the issue if you upgrade to the latest mteb version. Let me know if you still face problems!

beir==1.0.0 solves the problem! Thanks for the help!


Issue when running prettier from command line/npm

Hi,
After installing the package in our repo - anytime i run prettier --config .prettierrc --write . I get this error:
[error] Unexpected token '?'
When I check the prettier output on vscode everything seems fine - but for some reason when trying to run prettier through the terminal (or through an npm command) it just fails.
See here the .prettierrc config
{
  "plugins": ["./node_modules/prettier-plugin-multiline-arrays"],
  "printWidth": 120,
  "singleQuote": true,
  "useTabs": false,
  "tabWidth": 2,
  "semi": true,
  "bracketSpacing": true
}

Some questions:

Does this happen on every file you try to format? (Try specifying a specific file instead of . in your command.)
What version of Node.js do you have installed in your terminal? (node --version)

Based on the error alone, I'm guessing what's happening is that VS Code is running an up-to-date version of Node.js (cause it ships with it), but your terminal is using an old version of Node.js that does not support optional chaining, which this package's code utilizes.

Thank you for the quick answer - Indeed node 14 did the trick :) and fixed the issue


fix: correctly handle all empty HeadObject/HeadBucket responses

Previously we only checked for empty bodies with HTTP 404, but it seems like S3 also returns empty bodies for HTTP 403. This PR expands the handler logic to catch all empty bodies rather than just 404's.
Issue #

Closes https://github.com/awslabs/aws-sdk-kotlin/issues/1368
Description of changes

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

question: do S3s 403 responses frequently contain payloads? I think we would still fail with the same Failed to parse response as ... exception if that's the case

HeadObject 403 never contains a payload. Here is a sample codegen:
private fun throwHeadObjectError(context: ExecutionContext, call: HttpCall, payload: ByteArray?): kotlin.Nothing {
    val wrappedResponse = call.response.withPayload(payload)
    val wrappedCall = call.copy(response = wrappedResponse)

    val errorDetails = try {
        if (payload == null) {
            if (call.response.status == HttpStatusCode.NotFound) {
                S3ErrorDetails(code = "NotFound")

            } else {
                S3ErrorDetails(code = call.response.status.toString())

            }
        } else {
            checkNotNull(payload) { "unable to parse error from empty response" }
            parseS3ErrorResponse(payload)

        }
    } catch (ex: Exception) {
        throw S3Exception("Failed to parse response as restXml").also {
            setS3ErrorMetadata(it, wrappedCall.response, null)
        }
    }
    val ex = when(errorDetails.code) {
        "NotFound" -> NotFoundDeserializer().deserialize(context, wrappedCall, payload)
        else -> S3Exception(errorDetails.message)
    }

    setS3ErrorMetadata(ex, wrappedResponse, errorDetails)
    throw ex
}

I'm not sure how a 403 with a non-null payload will get an Failed to parse response as restXml error exception thrown unless it's malformed.


Vim script can overwrite builtin function

let g:.foo = 1 let 1 to global variable foo. And this way can overwrite builtin functions.
Describe the bug
let g:.foo = 1 let 1 to global variable foo. And this way can overwrite builtin functions.
To Reproduce
let g:.trim = {x-> ' '..x..' '}
echo '['..trim(' foo ')..']'

This get [  foo  ].
Expected behavior
[foo]

This is also wrong.
let g:['trim'] = {x-> ' '..x..' '}
echo '['..trim(' foo ')..']'

This should fix the issue.
diff --git a/src/userfunc.c b/src/userfunc.c
index f5d906302..e463d73cd 100644
--- a/src/userfunc.c
+++ b/src/userfunc.c
@@ -1436,6 +1436,12 @@ deref_func_name(

     cc = name[*lenp];
     name[*lenp] = NUL;
+
+    if (builtin_function(name, -1)) {
+	name[*lenp] = cc;
+	return name;
+    }
+
     v = find_var(name, &ht, no_autoload);
     name[*lenp] = cc;
     if (v != NULL)

Besides that check, it should not even be possible to add that variable in g: or s:

Actually, we cannot use builtin_function(), because in lambda's using a lower case name is possible.

@brammool Thank you. But still wrong.
call extend(g:, #{foo: { -> 'foo' }})

@brammool another degrade with this fix.
let s:trim = {x-> ' '..x..' '}

This makes error E704.

I prefer checking for the referencing to the symbol, Not checking for letting variables.

@brammool another degrade with this fix.
let s:trim = {x-> ' '..x..' '}

This makes error E704.

That looks correct, it should use "s:Trim".
Did this work OK before?  Then we might have to check this only in Vim9
script, where script-local variables can be used without s: prefix.
Actually, I cannot reproduce the error in legacy script.  And I do get
the error in Vim 9 script, looks OK to me.  Did you have some context to
trigger the error?
--
ARTHUR:  No, hang on!  Just answer the five questions ...
GALAHAD: Three questions ...
ARTHUR:  Three questions ...  And we shall watch ... and pray.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

:let s:.foo = { -> 'foo' }
:echo s:foo()

Vim(let):E704: Funcref variable name must start with a capital: foo

:let g:s = s:
function! CallS(name)
  return s:[a:name]()
endfunction
:let g:s.foo = { -> 'foo' }
:echo CallS('foo')

:help E704 says:
<							*E704* *E705* *E707*
A Funcref variable must start with a capital, "s:", "w:", "t:" or "b:".  You
can use "g:" but the following name must still start with a capital.  You
cannot have both a Funcref variable and a function with the same name.

so, I think "s:" should be able to be followed by a lower letter.
And, I think that "s:" in this part should be changed to "l:"
https://github.com/vim/vim/commit/3d9c4eefe656ee8bf58c0496a48bd56bac180056#diff-3452ba3ede8c12266b78c613c3171f5a19f42244731749b3bc60072cdf45be46R2687-R2688
as follows:
  call assert_fails('let l:["trim"] = {x -> " " .. x}', 'E704:')
  call assert_fails('let l:.trim = {x -> " " .. x}', 'E704:')

:let s:.foo = { -> 'foo' }
:echo s:foo()

Vim(let):E704: Funcref variable name must start with a capital: foo

Looks like I missed the dot.
--
BRIDGEKEEPER: What is your favorite colour?
GAWAIN:       Blue ...  No yelloooooww!
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

:help E704 says:
<							*E704* *E705* *E707*
A Funcref variable must start with a capital, "s:", "w:", "t:" or "b:".  You
can use "g:" but the following name must still start with a capital.  You
cannot have both a Funcref variable and a function with the same name.

so, I think "s:" should be able to be followed by a lower letter.

In legacy script, yes.  But in Vim9 script a script-local variable can
be accessed without s:, so there we won't allow it.
But in Vim9 script assigning to s:.trim won't work, it tries to parse
".trim" as a type.  Same for s:['trim'].

And, I think that "s:" in this part should be changed to "l:"
https://github.com/vim/vim/commit/3d9c4eefe656ee8bf58c0496a48bd56bac180056#diff-3452ba3ede8c12266b78c613c3171f5a19f42244731749b3bc60072cdf45be46R2687-R2688
as follows:
  call assert_fails('let l:["trim"] = {x -> " " .. x}', 'E704:')
  call assert_fails('let l:.trim = {x -> " " .. x}', 'E704:')

Except that this doesn't fail yet.
--
BRIDGEKEEPER: What is your favorite colour?
LAUNCELOT:    Blue.
BRIDGEKEEPER: Right.  Off you go.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

:let g:s = s:
function! CallS(name)
  return s:[a:name]()
endfunction
:let g:s.foo = { -> 'foo' }
:echo CallS('foo')

After the recent changes this also works correctly.  For legacy script
this is allowed, s:foo can be a function. For Vim9 script it gives the
E704 error.
--
BEDEVERE: How do you know so much about swallows?
ARTHUR:   Well you have to know these things when you're a king, you know.
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


BaseOutputParser breaks pydantic PrivateAttr

System Info
langchain== 0.0.340
pydantic==2.4.2
pydantic_core==2.10.1
Who can help?
@hwchase17 @eyur
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[X] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain.schema import BaseOutputParser
from pydantic import PrivateAttr

class MyParser(BaseOutputParser[list[int]]):
    _list: list[int] = PrivateAttr(default_factory=list)

    def parse(self, text: str) -> list[int]:
        return self._list

parser = MyParser()
assert parser.parse("test") == []

Expected behavior
I expect PrivateAttr to still work within BaseOutputParser.

Ah the v1 vs v2 is subtle, thank you very much @169! 🙏
Closing this out, if there's some sort of other resolution you can think of, feel free to reopen


Iteration through graph nodes and set devices

There is a rather simple way to iterate graph nodes and set a device in Java TF:
    String deviceString = String.format("/gpu:%d", deviceIdx);
    GraphDef.Builder builder = GraphDef.parseFrom(graphDef).toBuilder();
    for (int i = 0; i < builder.getNodeCount(); i++) {
        builder.getNodeBuilder(i).setDevice(deviceString);

Is there a corresponding options in the TFC#.

I'm also looking for a way to set devices on ALL nodes of an imported graph (due to this issue). Python and C++ both have their own way to set device in this scenario.
Python uses with tf.device..., which can be easily mimicked in TF#:
var graph = new TFGraph();
using (var device = graph.WithDevice(deviceName))
{
     graph.Import(File.ReadAllBytes(path));
}

However this does not work, it seems TFGraph.WithDevice is just a superficial approach which does not set device for TFGraph.Import-ed nodes.
The C++ approach is similar to the Java one @serjl mentioned, iterate through all the nodes and set device for them. But after messing with TFGraph and TFOperation and TFOperationDesc for an hour I realized it's not trivial to do in TF#.

I found a way to do it through python. Import your frozen *.pb in python and run over all nodes and rename them all to the desired GPU. Save it again into a new *.pb. Use it in TF# with adding!!!! into sessionOptions the allow_soft_placement=True, so that non-GPU nodes can still run on CPU.

@serjl do you have sample code you can share please?

Later today will put here. But just to stress, it does work one session *.pb per device. So if you want to run on 2 gpus you need to initiate two sessions with its own *.pb uploaded.

@serjl yeah I'm aware that you can do it in python, C++ and even Java, but I just want to make my code as C# as possible.

@hillin Have  you tried TensorFlow.NET ?

@serjl do you have sample code you can share please? graph.WithDevice does work, but only if the TF# apps are running in separate processes. Your trick of remodelling a frozen pb will simplify things enormously.

detection_graph = tf.Graph()
with detection_graph.as_default():
od_graph_def = tf.GraphDef()
with tf.gfile.GFile(PATH_TO_PB, 'rb') as fid:
serialized_graph = fid.read()
od_graph_def.ParseFromString(serialized_graph)
for i in range(2):
for nod in od_graph_def.node:
# if nod.name in nodeList:
nod.device = "/device:GPU:%d" %i
tf.train.write_graph(od_graph_def, PATH_TO_NEW_PB, 'frozen_inference_graph_GPU_'+ '%d' % i +'.pb', False)

@AldoZ Here the Python code for two GPU's:
detection_graph = tf.Graph()
with detection_graph.as_default():
od_graph_def = tf.GraphDef()
with tf.gfile.GFile(PATH_TO_PB, 'rb') as fid:
serialized_graph = fid.read()
od_graph_def.ParseFromString(serialized_graph)
for i in range(2):
for nod in od_graph_def.node:
# if nod.name in nodeList:
nod.device = "/device:GPU:%d" %i
tf.train.write_graph(od_graph_def, PATH_TO_NEW_PB, 'frozen_inference_graph_GPU_'+ '%d' % i +'.pb', False)

@serjl Fantastic, thank you

Here is my approach (as I said, as C# as possible):
[DllImport("libtensorflow")]
private static extern void TF_ImportGraphDefOptionsSetDefaultDevice(IntPtr opts, string device);

private static void SetDevice(TFImportGraphDefOptions options, string device)
{
    TF_ImportGraphDefOptionsSetDefaultDevice(options.Handle, device);
}

private static TFGraph LoadGraph(string path, string deviceName)
{
    var graph = new TFGraph();

    var options = new TFImportGraphDefOptions();
    SetDevice(options, deviceName);

    graph.Import(File.ReadAllBytes(path), options);

    return graph;
}

public TFSession RunSession(string graphPath, string deviceName)
{
    var graph = LoadGraph(path, deviceName);

    // follow this guide to use ConfigProto: https://github.com/migueldeicaza/TensorFlowSharp/issues/206
    var config = new ConfigProto
    {
        LogDevicePlacement = true,
        AllowSoftPlacement = true
    };

    using (var stream = new MemoryStream())
    {
        config.WriteTo(stream);
        var configBuffer = stream.ToArray();

        var options = new TFSessionOptions();

        unsafe
        {
            fixed (byte* pConfigBuffer = configBuffer)
            {
                options.SetConfig((IntPtr)pConfigBuffer, configBuffer.Length);
            }
        }

        return new TFSession(graphPath, options);
    }
}

So RunSession(graphPath, "/device:GPU:1" gives you a session which will run on GPU1.

@hillin Cool, thank you very much! Regarding the return in CreateSession: shouln't it be
return new TFSession(graph, options);
otherwise I don't see where you use the graph variable.

@serjl yeh it was a typo. Fixed!


Remove in-coordinator (SGv1) GraphQL-api

As part of Stargate V2 clean up we can now remove:

graphqlapi main-level Stargate V1 implementation (all ITs removed earlier), replaced by Quarkus-based apis/sgv2-graphqlapi
RESTv2 half of restapi (need to leave RESTv1), replaced by Quarkus-based apis/sgv2-restapi. Module will remain for time being, until we can deprecate RESTv1.

Ok: already merged PR for GraphQL removal (#2119) but based on further discussions, reverted -- we need to wait until successful deployment to production: this way we have easy/easier rollback path, having the old implementation still available.
Leaving the issue itself open.

Fixed on 2.1 branch in #2640.


[Snyk] Security upgrade electron from 12.0.9 to 12.0.10

Snyk has created this PR to fix one or more vulnerable packages in the `yarn` dependencies of this project.

Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

packages/desktop/package.json

⚠ Warning
Failed to update the yarn.lock, please update manually before merging.

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

651/1000  Why? Recently disclosed, Has a fix available, CVSS 7.3
Heap-based Buffer Overflow SNYK-JS-ELECTRON-1296553
No
No Known Exploit

651/1000  Why? Recently disclosed, Has a fix available, CVSS 7.3
Heap-based Buffer Overflow SNYK-JS-ELECTRON-1296555
No
No Known Exploit

726/1000  Why? Recently disclosed, Has a fix available, CVSS 8.8
Use After Free SNYK-JS-ELECTRON-1296557
No
No Known Exploit

833/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 8.8
Type Confusion SNYK-JS-ELECTRON-1296559
No
Proof of Concept

726/1000  Why? Recently disclosed, Has a fix available, CVSS 8.8
Use After Free SNYK-JS-ELECTRON-1296561
No
No Known Exploit

651/1000  Why? Recently disclosed, Has a fix available, CVSS 7.3
Race Condition SNYK-JS-ELECTRON-1296563
No
No Known Exploit

651/1000  Why? Recently disclosed, Has a fix available, CVSS 7.3
Heap-based Buffer Overflow SNYK-JS-ELECTRON-1296565
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

@mareknovotny Please re-trigger the build with an empty commit.


make all 3 gemms in Float8Linear support configurability, not user facing

Stack from ghstack (oldest at bottom):

-> #315

Summary:
This PR adds some plumbing for how to eventually make all 3 gemms in a linear fwd/bwd configurable:

add LinearMMConfig to Float8Tensor to tie together the three ScaledMMConfig objects, one per gemm
add GemmInputRole to Float8Tensor to specify how to pick the right config
plumb all of these throughout the codebase

Note that none of this is user facing, and there is no logic change.  Planned follow-ups:

a future PR will make the per-gemm behavior configurable in a user facing way, which will hook up to the objects introduced in this PR
a future PR will update the naming from x/w/dL_dY to input/weight/grad_output throughout the codebase

Test Plan:
./test/test_everything.sh

Reviewers:
Subscribers:
Tasks:
Tags:
Differential Revision: D59973551

@vkuzo has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Analysis: spacer matches to host genome

This is a bit of a rough draft, feel free to add thoughts in the comments.
Motivation
Research has suggested that there may be cases when CRISPR systems are used for something besides immunity to foreign DNA - perhaps they could be regulating the host genome, or they might simply be inactive. A clue that one of these things might be happening is if there are spacers that come from their own host genome. To check this, we can use BLAST to look for matches between spacers and their host genome.

Here's roughly what needs to happen:

[ ] Make table in database to store matches between organisms and spacers. Possible fields:

id
organism_id
spacer_id
genomic_start
genomic_end

Note that genomic_start and genomic_end aren't the original start and end of the spacer, but the start and end of any matches found with BLAST.

[ ] BLAST spacers against host genome (c.f. #60). We could do this directly with files containing spacers and fasta files for genomes, for example:

python parserscripts/blast.py -t blastn -q data/spacers/NC_000853.fasta -s data/NC_000853.fasta -e 10 -o NC_000853.xml

OR we could work directly from the database: pull spacer sequences from the organismspacerrepeatpair table, download the corresponding genome, and BLAST.
Either way, the output will look very similar. Here's an example of the BLAST output for organism NC_000853.

[ ] Analyze BLAST output (c.f. #62). We expect that each spacer will produce at least one match to itself, since we know it's a sequence already in the genome. We're interested in matches that aren't direct self-matches. From the database, the way to check this is in the organismspacerrepeatpair table: if a match falls inside the genomic_start and genomic_end from the table, then it's a self-match and we should ignore it.

For example, I blasted spacer 7513 ("ATCTTGTGATGCCGGCGATGGGAAAGCCGAAAAACA") against its host genome, NC_000853, and found a genomic start and end of 1428113, 1428148. Since this range falls inside the genomic start and end of the original spacer-repeat pair in the CRISPR locus, we would ignore that match.
Here's the organismspacerrepeatpair entry for that spacer:
    "organism_spacer_repeat_pairs": [
        {
            "id": 7986,
            "organism": 657,
            "spacer": 7513,
            "repeat": 144,
            "order": 1,
            "genomic_start": 1428083,
            "genomic_end": 1428149
        }

filterByExpect.py is another script that parses BLAST output, for comparison.

Pinging @cemyld and @aays - what's the best workflow for this? I feel like we should just work directly from the database, right? How do we deal with needing to download genomes - is fetch.py appropriate for this?

I think the workflow would be as such:
iterate through each spacer in the database:
iterate through each unique organism that has the spacer:
blast spacer vs organism genbank file
if more than one match:
store results
Populating the organismcas table already downloads the genbank files however fetch.py can make sure they're up to date or are not missing. I'm wondering if this data should be stored in the database or just a separate script that outputs the matches is enough.

I think @cemyld's workflow above seems pretty reasonable! I don't think the output necessarily needs to be stored in the database - seems to be a bit more on the analysis side of things to begin with.
Given the findings in #199, though, we might have to work with genomic contigs instead of single GenBank records hypothetically containing an entire genome. That should be manageable though, and would just add another iterative layer to the workflow:
for spacer in database:
for each organism containing spacer:
for genomic contig from organism:
blast spacer vs contig file
if more than one match:
store results
Thoughts?


Model Snapshot code: Generated code for non well-known types (like enum)

Model snapshot code is a serialized form of the model therefore it should include correct type as the model. But properties that are not well-known types (like user defined enum) comes from application code, so to including them in model snapshot code requires the reference to them to be added in C# code. Moreover it can cause problem if application code is changed. Therefore presently model snapshot code includes underlying type of non well-known types.
Using a JSON mode snapshot may help.

We can swap to representing enums in the model if we need it in the future. For the moment using the base type of the enum is fine. We can change this when another feature requires it.