runtime: esbuild sometimes crashes with fatal: morestack on g0 on 64-bit Windows

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

$ go version
go version go1.17.7 darwin/arm64

Does this issue reproduce with the latest release?
Yes, 1.17.7 is the latest release.
What operating system and processor architecture are you using (go env)?
(this is the OS used to build the executable, but not to run it)
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/evan/Library/Caches/go-build"
GOENV="/Users/evan/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/evan/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/evan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.17.7"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/dev/null"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/mj/fcy686697qn53dt7g4rvv2540000gn/T/go-build1751159435=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

This is a new bug report as suggested by https://github.com/golang/go/issues/39457#issuecomment-1045178083. See this issue for full context: https://github.com/evanw/esbuild/issues/2031. Specifically I'm the author of esbuild, a JavaScript build tool written in Go, and I have a user reporting intermittent crashes. The crashes either say fatal: morestack on g0 or something like [signal 0xc0000005 code=0x0 addr=0xffffffffffffffff pc=0x512894].
What did you expect to see?
I expect users to be able to run esbuild successfully without intermittent crashes.
What did you see instead?

The problem appears to happen on 64-bit Windows. Note that esbuild does not make use of cgo. The machine is "a 3970x, i.e. 32 cores, 64 threads" according to the user. I cannot reproduce this issue myself (I don't even have a Windows machine) so I can't help debug this myself.

I'm the user that reported this, but I don't use go directly.  I'd be willing to add whatever instrumentation necessary if that's helpful however; I'm just somewhat out of my depth here.  I don't know how to reproduce the issue consistently; it just ... sometimes happens, and sometimes does not.

Not sure what's going on with the first trace. It looks just like it is waiting for a file read, and the error happened in some other goroutine. We'd need more of the printed stack traces, I think.
That second trace has more information. It's trying to do a string == with presumably a bad string. I'm not sure if it is inside if !strings.HasSuffix(matchKey, "*") { (line 835 at tip) or if strings.HasSuffix(expansion.key, "*") { (line 847 at tip). (Neither line matches the stack trace line - what hash of your esbuild repository was that error capture at?)
The matchKey value in the trace looks maybe bad? It is {0xc00028ad60, 0xc0005623f0} and that second word should be the length of the string, which is way too big. That would cause strings.HasSuffix to make a string with a bad pointer to pass to memeq, which is where we see the crash. The matchKey value isn't necessarily correct in the traceback though, so it is hard to be sure. And the fault address is -1, which doesn't make a lot of sense, unless Windows doesn't report faulting addresses and that's just a "I don't know" fallback.
Have you run this under the race detector?
It would really help if you could catch this in the act under a debugger, so you can inspect the length of matchKey being passed to HasSuffix. Or give us instructions sufficient to reproduce ourselves.
unsafe uses in this module look ok, but I haven't checked the dependencies.
Some debugging things you could do:

Add if len(matchKey) > 1<<30 { panic... } in a few places. Same for the arg to the other HasSuffix call. See if you still get the same crashes.
Run under the race detector
Does this fail with 1.16? the current 1.18 release candidate? Did it ever work with a different Go release? The more you can bisect between a good and bad Go release, the better.

unsafe uses in this module look ok, but I haven't checked the dependencies.

FYI the only dependency is on the golang.org/x/sys library: https://github.com/evanw/esbuild/blob/a1ff9d144cdb8d50ea2fa79a1d11f43d5bd5e2d8/go.mod.


File format for trainMode 3

Hello, I am trying to train sentence embeddings for sentence similarity task. I am trying to train it on the STS Benchmark dataset, which consists of sentence pairs, manually scored for sentence similarity.
As I understood from the wikipedia dataset you provide as example, the training data should consist of rows with sentences delimited by tabs. In accordance with this, I have preprocessed the sentences a bit, picked the ones with high similarity score and created a file with those sentence pairs delimited by \t. First 4 pairs as an example:
b'a plane is taking off .\tan air plane is taking off .\n'
b'a man is playing the cello .\ta man seated is playing the cello .\n'
b'some men are fighting .\ttwo men are fighting .\n'
b'a person is throwing a cat on to the ceiling .\ta person throws a cat on the ceiling .\n'

Unfortunately, when I try to start the training with this file, it throws an error saying that the file did not contain any valid examples. The whole output:
./starspace train -trainFile sts-train.txt -model starspace -trainMode 3

Arguments:
lr: 0.01
dim: 10
epoch: 5
maxTrainTime: 8640000
saveEveryEpoch: 0
loss: hinge
margin: 0.05
similarity: cosine
maxNegSamples: 10
negSearchLimit: 50
thread: 10
minCount: 1
minCountLabel: 1
label: __label__
ngrams: 1
bucket: 2000000
adagrad: 1
trainMode: 3
fileFormat: fastText
normalizeText: 0
dropoutLHS: 0
dropoutRHS: 0
Start to initialize starspace model.
Build dict from input file : sts-train.txt
Read 0M words
Number of words in dictionary:  4340
Number of labels in dictionary: 0
Loading data from file : sts-train.txt
Total number of examples loaded : 0
ERROR: File 'sts-train.txt' does not contain any valid example.

Am I doing something wrong? Does it maybe need more than 2 sentences per line or something like that? Is the trainMode 3 suitable for this problem?
Thank you

So I tried training the model today with the fileFormat flag set to labelDoc. It works!  Closing the issue.

@Tiriar Thank you for posting this question.
Can you help me with a clarification question?
Do you know where it is documented that we need to provide the labelDoc format with spaces around punctuation?
"I 'm ready to transfer , can I do another ? Can I do another t-balance ?"
as opposed to:
"I'm ready to transfer, can I do another? Can I do another t-balance?"
Any guidance would be appreciated.

@mattyd2 I think the wikipedia sentences in the example for learning sentence embeddings were preprocessed this way (the example in examples/wikipedia_sentence_matching.sh).
Also most of the tokenizers I ever used add spaces in front of apostrophes and punctuation. I mostly now use the tweettokenizer from python nltk package. Many algorithms want to have the data preprocessed this way, I guess because it is then easier to just split the sentences by spaces and of course punctuation is a special token than you can often throw away altogether (unless you want the model to learn differences between declarations and questions, then the punctuation might be useful as a token).


koordlet: skip fillExtensionMap if podMetric is nil

Signed-off-by: Jason Liu<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

/lgtm


What  does box encode and decode mean?

And in the file detector/constants.py  meaning of the parameter SCALE_FACTORS?

The use of variable window in detector/util/postprocess_proposals is a little hard to understand ,firstly stack with [0, 0, height, width] and unstack in the prune_outside_windows, if unstack in axis=1, ymin and xmin = 0?


new issue that i opened

we need all sorts of stuff from automatic cup cake dispensers to super computers. Let's not forget Jarvis
check out my kitten

this is a comment

this is a comment

this is a comment


Add: ollama support

I got it as far for it to paste the command in the meantime in the :ex box but when pressing  the :ex box just disappears and the command doesn't get executed.

I'm a bit busy, but I promise I'll look into it this week! 🙏

Forgot to add the config:
  {
    '3v0k4/exit.nvim',
    config = function()
      require('exit').setup({ model = 'llama3.2:3b' })

      vim.api.nvim_create_user_command('ExitOllama', function(opts)
        local prompt = table.concat(opts.fargs, " ")
        require('exit').prompt(prompt)
      end, { nargs = "*" })
    end
  }

The changes you noted are on point.
Not being much of a PR master I pushed some new changes to my local repo that I noticed appear in the 'Commits' section of this conversation. Is this how it's supposed to be done?
PS: It works for me now locally. The only thing I did extra besides the points you mentioned was to extract the whitespace trimmer line into a utils method. I no longer found the v:null extractor to be necessary given the new setup.
Thanks for your time and being so thorough.

At some point, we should dynamically check what models are available in Ollama, but we can leave it out of the scope of this PR and just hardcode them for now like you already did.

Okay.

BTW, in what cases is not response.response true?

Are you referring to utils.system() already handling the case when an error is thrown? After it gets out of utils.system, it's guaranteed that it already has a response. If it doesn't have a response, an error is thrown already. So we don't really need the not response.response check in ollama.lua#prompt, right?
What do you think about adding "Is ollama serve running?" to the Failed to run curl error message? I think it might be helpful to someone reading it and realizing maybe there's something they can do about it.
I tweaked utils.split_once() a bit because it was only returning the part before the first : from the model_id. Now it returns both ollama and : + llama3.2:3b.
So now the returned command is executed as soon as it's returned. Is that how it was intended as? Or should it just print the cmd out so that the user hits ?
Thanks a lot for your help in this!

Are you referring to utils.system() already handling the case when an error is thrown? After it gets out of utils.system, it's guaranteed that it already has a response. If it doesn't have a response, an error is thrown already. So we don't really need the not response.response check in ollama.lua#prompt, right?

utils.lua#system just checks the exit code of the curl command. If the API returned an error, you should check inside the adapter (similarly to openai.luaif response.error then error(response.error.message) end`).
After a quick glance (please double check), I don't see in the ollama docs any mention of errors. If that's the case, we don't need to check for errors in the adapter.

What do you think about adding "Is ollama serve running?" to the Failed to run curl error message? I think it might be helpful to someone reading it and realizing maybe there's something they can do about it.

I see that you added it to the code. Great idea!

So now the returned command is executed as soon as it's returned. Is that how it was intended as? Or should it just print the cmd out so that the user hits ?

It should just fill the command and let the user decide. The command could return !rm -rf /, so it would be problematic if it executed.
Do you have a prompt that triggers that behavior? With what model?

After a quick glance (please double check), I don't see in the ollama docs any mention of errors. If that's the case, we don't need to check for errors in the adapter.
True, there's only 1 mention of error with embedding generation, something I don't understand but something we don't use as far as I can tell.

Do you have a prompt that triggers that behavior? With what model?
Yes, using llama3.2:3b automatically accepted the answer, upon asking: "how to quit vim?"

This looks great. Fantastic job!
Thanks for your guidance! :+1:

Yes, using llama3.2:3b automatically accepted the answer, upon asking: "how to quit vim?"

This is definitely something we should look into.
But it's good we have a way to deterministically reproduce.
I'll be off for the next couple of weeks, so I'm unable to help with this.

I'm back!

Yes, using llama3.2:3b automatically accepted the answer, upon asking: "how to quit vim?"

I tried to reproduce with the following but I cannot (ie, :q is filled and I need to press enter to execute it):
:ExitModel ollama:llama3.2:3b
:Exit how to quit vim?

I'm merging this, thanks for the contribution! 🙏
Please feel free to open a separate issue/pr if you find a way to reproduce the issue mentioned above or have a solution.


Dropbox problem / internal error 1060, 500 internal error

I have a problem with a dropbox storage.
When I first configured it, it worked well.
Then I did some tidying, moving file and directory inside the dropbox share using the linux owncloud sync client.
The client was unable to execute the changes it seems, and started to get errors : internal error 1060, 500 internal error.
Each time it scans the dropbox directory, it ends up with this error and stops the scan.
In the owncloud.log there are lots of this errors :
Debug	OC\Files\Cache\Scanner	!!! No reuse of etag for '' !!! cache: Array ( [fileid] => 7 [storage] => dropbox::0ef0t2zo64sum0jzb0qr1zjybgd7tbz/ [path] => [parent] => -1 [name] => [mimetype] => httpd/unix-directory [mimepart] => httpd [size] => -1 [mtime] =><PHONE_NUMBER> [storage_mtime] =><PHONE_NUMBER> [encrypted] => [unencrypted_size] => 0 [etag] => 54ce5c26e03de [permissions] => 31 ) data: Array ( [mimetype] => httpd/unix-directory [mtime] =><PHONE_NUMBER> [size] => -1 [etag] => 54ce5c4cebea4 [storage_mtime] =><PHONE_NUMBER> [permissions] => 31 )
What seems weird to me is the fact no file name is filled : No reuse of etag for ''
I have another dropbox share configured, and it is working well.
Also, when I  check the dropbox share on the dropbox interface, the changes done with owncloud were correctly done.
I tried several things :

erase content of oc_filecache table : no improvement
configure a new dropbox application : no improvement
erase completely the content of the dropbox share (using the dropbox website) : then the sync worked correctly. I then started to add my files again using owncloud, the problem reappeared....
Each file uploaded though the sync client had a 500 error, even if the file arrived correctly on the dropbox share...

When I try to click on the dropbox share using owncloud web interface, after a few second  I get the root of owncloud...
Would you have an idea of what is going wrong ?
Version : ownCloud 7.0.4 (stable)
I tried the client under windows and linux.
Thanks

Wait, what? I guess you posted to the wrong repository?!

Oups, yes... sorry. I begin with github and I posted in the wrong repository....


[Docs] slideTo is actually swipeTo

In the swiper API docs (http://www.idangero.us/swiper/api/) under the 'Methods' heading there is documentation for mySwiper.slideTo(index, speed, runCallbacks);
I believe this should actually be mySwiper.swipeTo.

I would make the update myself but your docs don't seem to be on GitHub.

Docs are for Swiper 3, you are using Swiper 2.x

There doesn't seem to be a way to get to the API docs for<EMAIL_ADDRESS>Have you got a link to them @nolimits4web ?

Right on repo's home page https://github.com/nolimits4web/Swiper#swiper-2xx

Ah thanks, I was looking for it on http://www.idangero.us/swiper/api/


PROTOCOLS-118 Continuation Response can't handle Login

It seems James does not handle the repeated continuation request correctly.
This is the first part of the fix.
NGINX for example is expecting some text after the initial continuation Tag ("+").
Changed it so an "+ Ok" is send.
It seems to be specified by the RFC 3501, though it is a very vague description.

Ohh, for got the link, here it is:PROTOCOLS-118

Hi @Flyffies ,
Thank you for the contribution, it is an interesting one!
I had a quick look, and this seems like a good move. I will be launching some integration tests on this.
I just have two little comments:

We tend to prefix commit messages with the JIRA tickets. Here:

PROTOCOLS-118 Fixed continuation request not getting recognised by some clients

We tend to avoid comments in the code. These comments can be moved as part of the commit message if needed.

Cheers,
Benoit Tellier

Hi, thanks I didn't saw the comment policy anywhere, so its noted for the future.
As to the second part of my ticket, I Tcpdumped the actual NGINX to James communication with my change applied, here it is:
N -> J: 1 login {8}
J -> N: + Ok
N -> J: testUser {8}
Then James doen not reqest more content.
While attaching to it in Debug I saw the field retry in AbstractNettyImapRequestLineReader chenges to true as AbstractNettyImapRequestLineReader is recreated.
Thats prevents any other continuation request from being send.
But... When I changed the value to false during debug, nothing got written to nginx and James tried to send multiple continueation request that time... Its really strange...

Closed it, because I switched away from james. To many things just dont work

Sorry about that. Can you share a little your experience so that others can benefit? Don't hesitate to open tickets on JIRA, some contributors could be interested fixing them.

Well, the nginx reverse proxy for mail doesn't work, the Mail Clients complain, because james apperently doesn't do some things like its specified in the RFCs.
It caused me a lot of headaches. I found a all in one docker, easy to setup, with postfix and dovecot.
Way less pain.
I opened a Ticket on Jira, but nothing happend there, so I figuered I might as well close it.

Thank you for the feedback. Even if we have not the time to work on it, we are very concerned by the IMAP support, and for the moment pretty happy with it (at least with some MUA). But I agree there are some improvement to do on the authentication and continuation code.
Would you mind to reopen the https://issues.apache.org/jira/browse/PROTOCOLS-118 issue please? Even if you'll not fix it, we'll probably do it one day.


[UN-714] Fix position of 'new' label for secondary card links and adjust padding

Ticket URL: https://national-archives.atlassian.net/browse/UN-714
About these changes
Adjusts the green 'new' label assigned to new stories/records in the secondary cards - makes sure the label doesn't drop below the title when the title is longer than 1 line.
How to check these changes
Review /explore-the-collection/ locally, check a secondary card with a long title and see that the 'new' label stays on the same line as the title and doesn't drop to a new line.
Before:

After:

Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer.
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[ ] Waited for all CI jobs to pass before requesting a review.
[ ] Added/updated tests and documentation where relevant.

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

Hi Jane, nice work - this works for me but I see AJ has left a comment, so happy to approve but also take a look at AJ's comment :) I've tested with just inline and also inline-block and both work in my eyes.

Thanks James I've updated it now to inline-block to support the padding better and avoid any possible cross-browser weirdness - if you're happy to merge please go for it :)


SP_Field - New Resource Proposal

Description
Way to provision Site Columns in sites or webs
Proposed properties
Name
SchemaXml
Special considerations or limitations

Note: Use of SchemaXml and not all other properties that Field objects could take is to reduce the overhead in development and is often used cross on-premises & online with CSOM so should allow developers (or admins) to easily use this where they deem it fit to do so

Usually SharePointDsc focusses on infrastructure components, not content. This because editing content can result in a conflict with user edits (e.g. a user edits the title of a site and your desired state is no longer compliant). This resource is a grey area. I like your idea to also allow managing the Content Type Hub, which is something end users are unable to edit. That is why I think this resource is a valuable one.

To address comments - I totally get that in the majority of cases this seems like a counter intuitive resource to have that could perhaps be better served using other tooling - however there are use cases that I have seen used where this functionality would be incredibly useful and would negate needing to have additional tooling in place, as such I intend to in future raise a PR that implements this.


Emacs support?

Hi @tomjridge , thanks for your interest in Tandem!
Currently, we only support Neovim and Sublime Text.
While we do have plans to extend Tandem to other editors, I don't think we'll be developing an Emacs plugin anytime soon.
However, if you are interested in developing a Tandem plugin for Emacs, we'd be delighted to help out along the way and more than welcome Pull Requests.
We wrote a how-to for developing Tandem plugins. Check out our wiki: https://github.com/typeintandem/tandem/wiki/Interacting-with-the-Tandem-Agent
As long as your plugin conforms to that outline, it should work seamlessly with the Agent!

@rageandqq could you clarify what the Tandem agent is exactly? i see that the neovim plugin requires the neovim Python2 client and node.js, is that how the agent works, with node.js? this repository only has Python code, so I'm a little confused regarding the node.js requirement there...

The crdt is run on node.js

@anarcat The Tandem agent implements all of the common functionality that all plugins need (e.g. managing peers in a session, sending messages between peers, etc.). We actually use the CRDT that powers Teletype under the hood. Since they implemented it in JavaScript, we need to take on the Node.js dependency in order to actually use it with the agent.

I've got something work-in-progress here https://github.com/jscheid/tandem-emacs
It works pretty well when joining a session, hosting a session (and joining it with Sublime) seems broken in the agent?

@jscheid i've run into the same issue when trying to host via the command line and join via sublime. Looking at the log all I get are the following messages:
2018-03-15 00:19 INFO     Tandem UDPGateway is listening on ('', 4001).
2018-03-15 00:19 INFO     Tandem Agent has started.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.
2018-03-15 00:20 DEBUG    Received data from <IP_ADDRESS>:60689.

oh boy, how exciting! didn't expect that to happen so soon :) congrats on the author and keep it up! should we report issues there or here?
(this only makes licensing issues described in #131 even more pressing unfortuantly...)

@anarcat since we are not modifying their agent, and it is their agent that is communicating with the server it might be ok. Alternatively we can point it to another server.

On 2018-03-15 14:50:45, Alexander Griffith wrote:

@anarcat since we are not modifying their agent, and it is their agent that is communicating with the server it might be ok. Alternatively we can point it to another server.

Well, the license on the emacs plugin itself is not a concern here. The
concern is the license of the agent. You still have to use agent to
operate the plugin, and so agree to its license. Changing the server
doesn't resolve the fundamental non-freeness of the license... But I
disgress, this is a discussion for the other issue. :)

the license will change, so nevermind those comments :) whoohoo!

@anarcat @alexjgriffith  The license has been updated! See https://github.com/typeintandem/tandem/pull/135. Hope this quells your concerns 😄
@jscheid That's awesome! I too didn't expect this to happen so soon. What version of python3 are you running locally? We verified that the agent works with python3.6+, and handling versioning issues (and potentially rethinking how we choose, version, and distribute Tandem's dependencies) is one of the issues our team is planning to address.

I have Python 3.6.4 from homebrew.
Could it be a discrepancy between the version of the agent used by the Sublime plugin and the one I'm using?

Does it work with the vim plugin? I have tried using the same agent for both the Emacs extension and Sublime plugin and still could not host on Emacs.

It turns out Emacs hasn't quite achieved sentience yet and didn't know to send the initial buffer when hosting a session without me telling it to... it should work better now.

By the way, what are the criteria for getting it listed as the (or as "an") unofficial Emacs plugin in the README and on the home page?


[AVM Module Issue]: 🐛 role assignments for NIC's does not work

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
0.12.0
(Optional) Correlation Id
No response
Description
Specifying role assignments for network interfaces does not work.
Attempting so causes errors like these:
 Error: Unsupported attribute
│
│   on .terraform/modules/mgmt_windows_vm/main.networking.tf line 93, in resource "azurerm_role_assignment" "this_network_interface":
│   93:   role_definition_name                   = (length(split("/", each.value.role_definition_id_or_name))) > 3 ? null : each.value.role_assignment.role_definition_id_or_name
│     ├────────────────
│     │ each.value is object with 3 attributes
│
│ This object does not have an attribute named "role_definition_id_or_name".

Example of network_interfaces  part of vm declaration:
  network_interfaces = {
    nic1 = {
      name = "nic-my"
      ip_configurations = {
        ip_configuration_1 = {
          name                          = "vm-my-ipconfig1"
          private_ip_subnet_resource_id = var.subnet_id
        }
      }
      role_assignments = { for user_key, principal_id in var.all_vm_users :
        "${user_key}_read_nic_of_vm-my" => {
          principal_id               = principal_id
          role_definition_id_or_name = "Reader"
        }
      }
    }
  }

The issue seems to be here: https://github.com/Azure/terraform-azurerm-avm-res-compute-virtualmachine/blob/862ab3a36ace3192f4fe55e3ad99f3914eaf71ee/main.networking.tf#L92-L93
  role_definition_id                     = (length(split("/", each.value.role_definition_id_or_name))) > 3 ? each.value.role_assignment.role_definition_id_or_name : null
  role_definition_name                   = (length(split("/", each.value.role_definition_id_or_name))) > 3 ? null : each.value.role_assignment.role_definition_id_or_name

should be :
  role_definition_id                     = (length(split("/", each.value.role_assignment.role_definition_id_or_name))) > 3 ? each.value.role_assignment.role_definition_id_or_name : null
  role_definition_name                   = (length(split("/", each.value.role_assignment.role_definition_id_or_name))) > 3 ? null : each.value.role_assignment.role_definition_id_or_name

Notice how each.value.role_definition_id_or_name changed to each.value.role_assignment.role_definition_id_or_name.

@Laffs2k5 - Thanks for submitting this.  Let me get a fix in, and I'll add additional role assignments to the RBAC example so we get good testing of this going forward.  I'll look to push the fix in the next release later this week.

@Laffs2k5 - This issue was corrected in release v0.13.0.  Please re-open this if you determine anything is still missing.


Can I download several types of files?

For example, both pdf and zip in one command? or even any file .?

Why not, It's possible and will be added soon


Hyperdrive Changelog Page

We have updated the Deployment Registry to better support address discovery and a deployment changelog Link to Registry PR. We need a webpage  that reads from the registry contract and generates two tables:

Pools

Factories

Notes:

app.hyperdrive.box/changelog
all addresses are links to etherscan
have a toggle for chain

A model for the registry was added to the SDK in #1252 and draft PR with a WIP changelog page can be found in #1257.

The tables have been added to the UI at the /chainlog route: https://app-staging.hyperdrive.box/chainlog


The noise_file on CIFAR datasets

Hi, thanks to your work. Could you please provide the noise_file on CIFAR datasets?

Hi, they can be found in https://github.com/chenpf1025/IDN and https://github.com/xiaoboxia/Part-dependent-label-noise respectively.


Support 'kAddTo' for SwapAxis Op

I'm currently working on implmenting  facebook fairseq on mxnet, and find SwapAxis doesn't support kAddTo which is needed when accumlating encoder gradient for each attention layer.  Is that easy to add kAddTo for  SwapAxis?  And  is there any other op that doesn't support kAddTo?  By the way, it is very easy to fall in this trap and diffcult to debug.
@eric-haibin-lin @piiswrong

Should be easy to do. Are you talking about forward or backward?

Backward

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!

Hi, I'm looking for simple issues for my first PR. Upon reviewing C++ code, it seems this issue is already resolved, or am I missing anything?
I also made a small test case:
#!/usr/bin/env python
from itertools import product
import numpy as np
import mxnet as mx

CTX_LIST = [mx.gpu(), mx.cpu()]
DTYPE_LIST = ['float16', 'float32', 'float64']
for CTX, DTYPE in product(CTX_LIST, DTYPE_LIST):
    with mx.autograd.record():
        v_x = mx.nd.random.uniform(
           -1., 1., shape=(4,5,6),
           dtype=DTYPE, ctx=CTX)
        v_x.attach_grad(grad_req='add')
        v_x.grad[:] = 0.
        v_y = mx.nd.swapaxes(v_x, 1, 2)
        v_err = mx.nd.square(v_y)
        v_err.backward(retain_graph=True)
        v_y = mx.nd.swapaxes(v_x, 0, 1)
        v_err = mx.nd.square(v_y)
        v_err.backward(retain_graph=True)
        v_y = mx.nd.swapaxes(v_x, 0, 2)
        v_err = mx.nd.square(v_y)
        v_err.backward(retain_graph=True)
    assert(np.allclose(v_x.asnumpy()*6, v_x.grad.asnumpy()))

@khaotik indeed. Thanks for updating the issue.


python3Packages.dpath: 2.0.1 -> 2.0.2

Motivation for this change
Update to latest upstream release 2.0.2
Change log: https://github.com/dpath-maintainers/dpath-python/releases/tag/v2.0.2
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 via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 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 136550 run on x86_64-linux 1

2 packages built:

python38Packages.dpath
python39Packages.dpath


qt statusLog reset field and modal

Needs changes

It would appear that the code in file helpers\authUsers.js is related to identifying system adminitrators. For reference, here is the code contained in the helpers\authUsers.js file:
const authorisedUsers = [
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>];

const authorisedToPerformAction = async (email) => {
  return authorisedUsers.includes(email);
};

export {
  authorisedToPerformAction
};

Consider refactoring the code above as follows:

Rename the file to helpers\sysAdmins.js
Rename authorisedUsers to sysAdmins;
Rename authorisedToPerformAction() to isSysAdmin()

As per comment above re: authUsers.js - as soon as we'll implement user roles (hopefully soon!), we won't need this file.

Ticket to be discussed at Friday stand-up on 2 July. Advised by Tom that we have the option to proceed with this work or roll what has been done back.
We need consensus as to the value of the work and whether its felt we should proceed.

Hi @tomlovesgithub please could you provide some info on how to test? Thanks!

Hi @tomlovesgithub please could you provide some info on how to test? Thanks!

@HalcyonJAC
Apologies!
Added some directions for testing.

@tomlovesgithub Is this ready for testing?


[Feature]: Badge numeric values

Requirements

[X] I have updated Mushroom to the latest available version
[X] I did a search to see if there is a similar issue or if a pull request is open.

Is your feature request related to a problem?
No
Describe the solution you'd like
Add sensor data (for example: number of lights on) in a badge instead of a icon
Describe alternatives you've considered
No response
Additional context
No response

Hereafter a solution based on icon for up to 9 elements (then display 9+)
Here looking to all the lights in a group in a template card:

.
{% set all = expand('group.lumieres_toutes')| list -%}  {% set open1 = all | selectattr('state','eq','on')|list|count%} {% if open1 == 0 %}   mdi:numeric-0 {% elif open1 == 1 %}   mdi:numeric-1 {% elif open1 == 2 %}   mdi:numeric-2 {% elif open1 == 3 %}   mdi:numeric-3 {% elif open1 == 4 %}   mdi:numeric-4 {% elif open1 == 5 %}   mdi:numeric-5 {% elif open1 == 6 %}   mdi:numeric-6 {% elif open1 == 7 %}   mdi:numeric-7 {% elif open1 == 8 %}   mdi:numeric-8 {% elif open1 == 9 %}   mdi:numeric-9 {% elif open1 >= 9 %}   mdi:numeric-9-plus {% else %}   none {% endif %}
It might help for the time being...

Thanks @BBESINET I already used the 'numeric' badges, just hadn't thought of 9+.
I have now:
{% if states(entity) > '0' %}
  mdi:numeric-{{ states(entity) }}
{% elif states(entity) > '9' %}
  mdi:numeric-9-plus
{% endif %}

This works fine for now, but it would be nice if we could just fill this with sensor data.

{% set numLightsOn = expand(state_attr('light.all_lights', 'entity_id')) | selectattr('state', 'eq', 'on') | list | count %}
{%- if numLightsOn > 9 %}
mdi:numeric-9-plus
{%- elif numLightsOn == 0 %}
{%- else %}
mdi:numeric-{{ numLightsOn }}
{%- endif %}

Would like to be able to do this on a custom:mushroom-entity-card


[Feature Request]: Add Notification mechanism for Nightly GH Action

Feature description
The Nightly GH action only notifies the workflow file's most recent updater upon failures, so the rest of the team needs to manually check the results or they will be unaware of sanity/status.  Implement some mechanism that alerts (via email, slack, etc) a list or group of users whenever the workflow fails.
Describe alternatives you've considered
No response
Anything else?
No response

Migrated to Jira: https://issues.redhat.com/browse/RHOAIENG-1677


2.0 Use null instead of false as return type in factories

Ticket: #2561

Thanks @jarednova. I committed your suggestion. With a final approval, this is ready to merge.

Final approval granted, please merge away!

I’m waiting for the changes in #2517 to be merged before merging this PR in.

Thanks @gchtr. To confirm where we are on #2517, is the last pending edit the one described in this comment/discussion, or are there any other items requiring resolution?


Add check to make sure passed rebalancingSetToken into propose is tra…

…cked by Core.

Pull Request Test Coverage Report for Build 5450

3 of 3 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 5427:
0.0%

Covered Lines:
996

Relevant Lines:
996

💛  - Coveralls


Detect isatty when enabling color

This PR is an improvement to https://github.com/microsoft/knack/pull/171.
Only enable color if all of the below conditions stand:
Color is not explicitly disabled
AZURE_CORE_NO_COLOR and this config shouldn't be set:
[core]
no_color = yes

stdout is a TTY
Otherwise (enable color while stdout is redirected), if the downstream command doesn't support color, Knack will fail with BrokenPipeError: [Errno 32] Broken pipe (https://github.com/Azure/azure-cli/issues/13413):
$ az --version | head --lines=1
azure-cli                          2.5.1
The command failed with an unexpected error. Here is the traceback:

[Errno 32] Broken pipe
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 207, in invoke
    self.show_version()
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/__init__.py", line 146, in show_version
    print('\n' + (SURVEY_PROMPT_COLOR if self.enable_color else SURVEY_PROMPT))
  File "/opt/az/lib/python3.6/site-packages/colorama/ansitowin32.py", line 41, in write
    self.__convertor.write(text)
  File "/opt/az/lib/python3.6/site-packages/colorama/ansitowin32.py", line 162, in write
    self.write_and_convert(text)
  File "/opt/az/lib/python3.6/site-packages/colorama/ansitowin32.py", line 187, in write_and_convert
    self.write_plain_text(text, cursor, start)
  File "/opt/az/lib/python3.6/site-packages/colorama/ansitowin32.py", line 196, in write_plain_text
    self.wrapped.flush()
BrokenPipeError: [Errno 32] Broken pipe

To open an issue, please run: 'az feedback'
Exception ignored in: <_io.TextIOWrapper name='<stdout>' mode='w' encoding='UTF-8'>
BrokenPipeError: [Errno 32] Broken pipe

With this RP, color sequences are not printed or sent to downstream command.
$ az --version | head --lines=1
azure-cli                          2.5.1

Also, if color is written to stderr, the terminal color can't revert back (https://github.com/Azure/azure-cli/issues/6080, https://github.com/Azure/azure-cli/issues/8483, https://github.com/Azure/azure-cli/issues/11671, https://github.com/Azure/azure-cli/issues/12084, https://github.com/Azure/azure-cli/issues/13979) :

This is due to colorama issue https://github.com/tartley/colorama/issues/200. In order for color to be reset correctly, the Style.RESET_ALL control sequence must be sent to both stdout and stderr. However, colorama can't send Style.RESET_ALL to stderr correctly, causing the terminal stuck at the previous color:
stdout:  Fore.CYAN  -->    Style.RESET_ALL
stderr:  Fore.RED   -->   (Style.RESET_ALL)   <<< colorama can't send this

stderr is a TTY
Otherwise (enable color while stderr is redirected),
az foo --verbose 2>err.txt

err.txt will have no level information (because there is no color):
az: 'foo' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview
command ran in 0.559 seconds.

With this RP, each message will be prefixed with LEVEL_TAG:
ERROR: az: 'foo' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview
INFO: Command ran in 0.917 seconds (init: 0.077, invoke: 0.840)

CI was broken due to a breaking change in isort https://github.com/timothycrosley/isort/issues/1273:
py36 run-test: commands[2] | pylint knack --rcfile=.pylintrc -r n -d I0013
Traceback (most recent call last):
  File "/home/vsts/work/1/s/.tox/py36/bin/pylint", line 8, in <module>
    sys.exit(run_pylint())
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/__init__.py", line 20, in run_pylint
    Run(sys.argv[1:])
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/lint.py", line 1628, in __init__
    linter.check(args)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/lint.py", line 943, in check
    self._do_check(files_or_modules)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/lint.py", line 1075, in _do_check
    self.check_astroid_module(ast_node, walker, rawcheckers, tokencheckers)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/lint.py", line 1158, in check_astroid_module
    walker.walk(ast_node)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/utils.py", line 1305, in walk
    cb(astroid)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/checkers/imports.py", line 507, in leave_module
    std_imports, ext_imports, loc_imports = self._check_imports_order(node)
  File "/home/vsts/work/1/s/.tox/py36/lib/python3.6/site-packages/pylint/checkers/imports.py", line 664, in _check_imports_order
    isort_obj = isort.SortImports(
AttributeError: module 'isort' has no attribute 'SortImports'
ERROR: InvocationError for command /home/vsts/work/1/s/.tox/py36/bin/pylint knack --rcfile=.pylintrc -r n -d I0013 (exited with code 1)


NotificationListener not working with horizontal ListView.builder in Flutter

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Run the code on Android 11
Scroll the line "text text" to the left

Expected results
Scrolling event should be printed to console
Actual results
Nothing printed to console
Code sample
Code sample
import 'package:flutter/material.dart';

void main() => runApp(const MaterialApp(
      home: MyHomePage(),
    ));

class MyHomePage extends StatelessWidget {
  const MyHomePage({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: SafeArea(
        child: NotificationListener(
          onNotification: (scrollNotification) {
            print(scrollNotification);
            return true;
          },
          child: ListView.builder(
            scrollDirection: Axis.horizontal,  // <= works if replaced by vertical
            itemCount: 2,                      // <= works if replaces by 22
            itemBuilder: (context, index) {
              return const Text("text ");
            },
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.13.3, on Microsoft Windows [Version 10.0.19045.3570], locale    en-US)
    • Flutter version 3.13.3 on channel stable at c:\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision<PHONE_NUMBER> (7 weeks ago), 2023-09-06 14:32:31 -0700
    • Engine revision b8d35810e9
    • Dart version 3.1.1
    • DevTools version 2.25.0

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
    • Android SDK at C:\Users\brand17\AppData\Local\Android\sdk
    • Platform android-33-ext4, build-tools 33.0.2
    • Java binary at: C:\Program Files\Android\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-9505619)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.5.0)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.5.33414.496
    • Windows 10 SDK version 10.0.22000.0

[√] Android Studio (version 2022.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version openjdk version "11.0.15" 2022-04-19

[√] VS Code (version 1.83.1)
    • VS Code at C:\Users\brand17\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.74.0

[√] Connected device (4 available)
    • itel S661LPN (mobile) • 078453724R000308 • android-arm64  • Android 11 (API 30)
    • Windows (desktop)     • windows          • windows-x64    • Microsoft Windows
      [Version 10.0.19045.3570]
    • Chrome (web)          • chrome           • web-javascript • Google Chrome
      118.0.5993.118
    • Edge (web)            • edge             • web-javascript • Microsoft Edge
      118.0.2088.76

[√] Network resources
    • All expected network resources are available.

• No issues found!

Hi @brand17
I'm using Realme 6, Android 11 and still seeing the log printed:
I/flutter (12525): ScrollMetricsNotification(depth: 0 (local), FixedScrollMetrics(0.0..[360.0]..0.0))

But I'm on stable channel 3.13.9. Could you try upgrading Flutter stable channel and confirm?

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-x64, locale en-VN)
    • Flutter version 3.13.9 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (7 hours ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio Giraffe.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.0 23A344 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.117

[✓] Network resources
    • All expected network resources are available.

• No issues found!

[!] Flutter (Channel master, 3.16.0-18.0.pre.91, on macOS 14.0 23A344 darwin-x64, locale en-VN)
    • Flutter version 3.16.0-18.0.pre.91 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e12d1a798c (6 hours ago), 2023-10-29 15:00:30 -0700
    • Engine revision 84dcb4fb93
    • Dart version 3.3.0 (build 3.3.0-71.0.dev)
    • DevTools version 2.28.1
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio Giraffe.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (3 available)
    • iPhone (mobile) • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.7.9 19H365
    • macOS (desktop) • macos                                    • darwin-x64     • macOS 14.0 23A344 darwin-x64
    • Chrome (web)    • chrome                                   • web-javascript • Google Chrome 118.0.5993.117

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Hi @brand17 I'm using Realme 6, Android 11 and still seeing the log printed:
I/flutter (12525): ScrollMetricsNotification(depth: 0 (local), FixedScrollMetrics(0.0..[360.0]..0.0))

But I'm on stable channel 3.13.9. Could you try upgrading Flutter stable channel and confirm?
flutter doctor -v (stable and master)

I am having this same issue, and I am on stable.

@MatthewPeden Could you share the device info (OS, model) along with the output of flutter doctor -v?


[SPARK-8791][SQL] Improve the InternalRow.hashcode

Merged build triggered.

Merged build started.

Test build #36395 has started for   PR 7189 at commit 86be175.

Test build #36395 has finished for   PR 7189 at commit 86be175.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

@chenghao-intel Thanks for working on it, Could you also update equals() and those in GeneratedProjection?

Merged build triggered.

Thank you all for reviewing the code for me, but I think there would be more general way to solve this, closing it for now.


[Perf] improve performance by optimizing getId

Profiler shows it's about 2% faster overall.

Good point.


pypyr-go-terra-aws-k8s

Adding awscli, aws-iam-authenticator and kubectl to image.

HI Rahul, thanks for this. There's getting enough of these little dependencies now that I think it's prob sensible to make a swiss army-knife container for all things devops.
Let's call it pypyr-ops.
it contains

go
terraform
awscli
aws-iam-authenticator
kubectl

pypyr-ops is tagged on an independent version number that increments for each dep update, i.e pypyr-ops:latest, pypyr-ops:5, pypyr-ops:4.5.1 et.
Other than that, a few things to discuss on the Dockerfile itself. . .

RUN statements should be combined.
The example comment is not actually commented - Dockerfile is broken as is
The README copy and paste from pypyr-go doesn't have everything updated


Add Send command to leshan-client-demo.

This aims to re-implement  #1031 since we moved to picocli/Jline (see #1064).
The new commands looks like :
Usage:  send [-c=<contentFormat>] [<paths>...]
Send data to server
      [<paths>...]   paths of data to send.
  -c, --content-format=<contentFormat>
                     Name (e.g. SENML_JSON) or code (e.g. 110) of Content
                       Format used to send data.
                     Default : SENML_CBOR

Example of usage :
send /3/0/1 /3/0/2 -c SENML_JSON
           InteractiveCommands 2021-09-06 15:15:46,013 [INFO] Sending Data to coap://localhost:5683[LWM2M_SERVER 123] using SENML_JSON(110).
                      MyDevice 2021-09-06 15:15:46,018 [INFO] Read on Device resource /3/0/1
                      MyDevice 2021-09-06 15:15:46,020 [INFO] Read on Device resource /3/0/2
           InteractiveCommands 2021-09-06 15:15:46,110 [INFO] Data sent successfully to coap://localhost:5683[LWM2M_SERVER 123] [CHANGED(204)].

@bocajim this is now integrated in master but feedback are still welcomed. :slightly_smiling_face:


nerdctl: 0.10.0 -> 0.11.0

Motivation for this change
Bump nerdctl to 0.11.0
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase 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 130802 run on x86_64-linux 1

1 package built:

nerdctl


Foresight oddity

Not getting adv on any melee attacks, spells attacks seem ok, creatures are not getting dis on attacks against me(i think), saves are working and checks.

Please learn how to report issues. No save, no logs, automatically closed. Reopen once to provide these, steps to repro, etc.


Device only needs .ui.grid

The documentation says items can be hidden on given device with thanks to the container class.

But it seems that the rules target grid instead:
.ui[class*="computer only"].grid.grid.grid:not(.tablet),
.ui.grid.grid.grid > [class*="computer only"].row:not(.tablet),
.ui.grid.grid.grid > [class*="computer only"].column:not(.tablet),
.ui.grid.grid.grid > .row > [class*="computer only"].column:not(.tablet)

This is working, though:

Thanks for the great job anyway.

The grid docs have been updated since.
There's large screen only, computer only, tablet only and mobile only


Fix captured trace length issue when decycling

Motivating example
If a chain of traces is made that looks like this
c -> b -> a

then you add d and e to form two chains
d -> c -> b -> a,
e -> c -> b -> a

decycling from d will not update the length of e
This patch fixes that by not relying on that length for anything else
except as a heuristic whether to decycle or not

todo: need to add a test for this


meta: skip updating mtime/ctime of parent if it's updated recently

Codecov Report

Merging #1960 (37a0a6c) into main (50ca313) will increase coverage by 0.03%.
The diff coverage is 94.11%.

@@            Coverage Diff             @@
##             main    #1960      +/-   ##
==========================================
+ Coverage   61.43%   61.47%   +0.03%
==========================================
  Files         135      135
  Lines       23467    23487      +20
==========================================
+ Hits        14418    14439      +21
  Misses       7370     7370
+ Partials     1679     1678       -1

Impacted Files
Coverage Δ

pkg/meta/base.go
69.61% <ø> (ø)

pkg/meta/redis.go
71.88% <94.11%> (+0.11%)
:arrow_up:

pkg/chunk/cached_store.go
76.05% <0.00%> (-0.19%)
:arrow_down:

pkg/meta/sql.go
64.70% <0.00%> (ø)

pkg/chunk/disk_cache.go
80.33% <0.00%> (+0.48%)
:arrow_up:

pkg/vfs/writer.go
79.66% <0.00%> (+1.01%)
:arrow_up:

Continue to review full report at Codecov.

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


Fix highpass filter issues

Highpass filter and Bandpass filter produce weird data, including ringing after blinks and odd spectrograms.
Hubert has confirmed that the coefficients generated by our filter code are appropriate and that issues are not resulting from our filter interpretation. Likely, the issue is doing to our code collecting data from the Muse, and the sampling rate.

I first tested the sampling rates of data being processed in the dataListener and dataSource runnable. When putting filtering code in the runnable, I was not able to control it's frequency so as to provide good filtering. However, this avenue of investigation lead me to find that the dataListener was being created twice and thus processing data twice as fast as it was expected to.
Making the dataListener final, removing redundant creation calls, and moving the filtering steps into it has lead to data being collected and filtered at a steady 256hz. This produces some pretty good spectrograms and PSDs and looks reasonable on the app.
To further remove noise so as to achieve good data quality, I added another bandstop filter in the range of 55-65hz for 2016 Muses. This completely removes electrical noise and produces the best quality filtered data I've been able to produce thus far.
ex BandPass + BandStop filter (with eyes closed alpha peak)


prevent user from sending more than 1 message a second in chat on the frontend

Fixes #152

instead of timeout based, can you set the date of the last message sent and check against that

@benawad that way we can probably also add timestamps in the chat. maybe in a tooltip.

that'll be a separate thing, timestamped on the server

instead of timeout based, can you set the date of the last message sent and check against that

sure


Alpine Build Broken

ERROR: unsatisfiable constraints:
  php-pear (missing):
    required by: world[php-pear]

Think just need to remove php-pear now? odd worked before.

Seems to only be in edge?
https://pkgs.alpinelinux.org/packages?name=php7-pear&branch=&repo=&arch=&maintainer=

All fixed up now.

Nice job! :+1:


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1043 ms

Resolved: Bofrost.be performance has improved in b52d958.


Met version problems when I recover the st-gcn demo

I followed the st-gcn guide, install the required libraries. The last step is python mmskl.py pose_demo, but when I run this command, the screen shows like this:
(open-mmlab) yangyang@tensorsrv:~/mmskeleton$ python mmskl.py pose_demo
/dm0/home/yangyang/.conda/envs/open-mmlab/lib/python3.7/site-packages/Cython/Distutils/old_build_ext.py:41: UserWarning: Cython.Distutils.old_build_ext does not properly handle dependencies and is deprecated.
"Cython.Distutils.old_build_ext does not properly handle dependencies "
/dm0/home/yangyang/.conda/envs/open-mmlab/lib/python3.7/site-packages/mmcv/utils/registry.py:64: UserWarning: The old API of register_module(module, force=False) is deprecated and will be removed, please use the new API register_module(name=None, force=False, module=None) instead.
'The old API of register_module(module, force=False) '
Load configuration information from ./configs/pose_estimation/pose_demo.yaml
Pose estimation:
Traceback (most recent call last):
File "mmskl.py", line 123, in
main()
File "mmskl.py", line 117, in main
call_obj(**cfg.processor_cfg)
File "/dm0/home/yangyang/mmskeleton/mmskeleton/utils/importer.py", line 24, in call_obj
return import_obj(type)(**kwargs)
File "/dm0/home/yangyang/mmskeleton/mmskeleton/processor/pose_demo.py", line 78, in inference
model = init_pose_estimator(detection_cfg, estimation_cfg, device=0)
File "/dm0/home/yangyang/mmskeleton/mmskeleton/apis/estimation.py", line 20, in init_pose_estimator
device='cpu')
File "/dm0/home/yangyang/mmdetection/mmdet/apis/inference.py", line 33, in init_detector
model = build_detector(config.model, test_cfg=config.test_cfg)
File "/dm0/home/yangyang/mmdetection/mmdet/models/builder.py", line 48, in build_detector
return build(cfg, DETECTORS, dict(train_cfg=train_cfg, test_cfg=test_cfg))
File "/dm0/home/yangyang/mmdetection/mmdet/models/builder.py", line 20, in build
return build_from_cfg(cfg, registry, default_args)
File "/dm0/home/yangyang/.conda/envs/open-mmlab/lib/python3.7/site-packages/mmcv/utils/registry.py", line 167, in build_from_cfg
return obj_cls(**args)
TypeError: init() got an unexpected keyword argument 'num_stages'

https://github.com/open-mmlab/mmskeleton/issues/321

I found that the latest mmdet require pytorch >1.3, and the latest mmskeleton require pytorch ==1.2.0, so their requirements are conflict. My solution is roll back the mmdet to 1.2.0, in this version, mmdet require pytorch >=1.1。


feat: Allow for all kubernetes kinds, accept path as annotation

Description

Added code to accept path as an annotation and use that when calling to vault
Refactored code to be generic enough to accept all resources including CRDs

Closes: #25 #23 #8
Checklist
Please make sure that your PR fulfills the following requirements:

[x] Reviewed the guidelines for contributing to this repository
[x] The commit message follows the Conventional Commits Guidelines.
[x] Tests for the changes have been
[ ] Docs have been added / updated

Type of Change

[ ] Bugfix
[x] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

Will clean up the commented code before turning off draft


Ryan's playbook config + logger fix

This fixes two things:

logger was undefined in smsConfigsLoader and so it'd throw an exception when it'd try to log errors
and then the error it was trying to log was that a Ryan's playbook config was missing

Background info
This is the error the server was throwing:
[01/08 12:58:45 EST][err] ReferenceError: logger is not defined
[01/08 12:58:45 EST][err]     at EventEmitter.app.getConfig (/opt/run/snapshot/package/app/config/smsConfigsLoader.js:66:3)
[01/08 12:58:45 EST][err]     at MCRouting.campaignTransition (/opt/run/snapshot/package/app/lib/ds-routing/controllers/MCRouting.js:65:30)
[01/08 12:58:45 EST][err]     at module.exports (/opt/run/snapshot/package/app/lib/ds-routing/index.js:26:13)
[01/08 12:58:45 EST][err]     at Layer.handle [as handle_request] (/opt/run/snapshot/package/node_modules/express/lib/router/layer.js:82:5)
[01/08 12:58:45 EST][err]     at next (/opt/run/snapshot/package/node_modules/express/lib/router/route.js:100:13)
[01/08 12:58:45 EST][err]     at Route.dispatch (/opt/run/snapshot/package/node_modules/express/lib/router/route.js:81:3)
[01/08 12:58:45 EST][err]     at Layer.handle [as handle_request] (/opt/run/snapshot/package/node_modules/express/lib/router/layer.js:82:5)
[01/08 12:58:45 EST][err]     at /opt/run/snapshot/package/node_modules/express/lib/router/index.js:235:24
[01/08 12:58:45 EST][err]     at Function.proto.process_params (/opt/run/snapshot/package/node_modules/express/lib/router/index.js:313:12)
[01/08 12:58:45 EST][err]     at Function.cls_wrapProcessParams [as process_params] (/opt/run/snapshot/package/node_modules/newrelic/lib/instrumentation/express.js:185:29)

:+1:


Add module support for the OS X framework.

Could you do this PR against master? I'm not planning to do another release from the v3.1.x-fixes branch. Unless something dramatic happens the next release will be 3.2, from master.

Sure.

Actually I can't figure out how to change the pull request to say master, I'll just make a new one.


Optimize the optimizer

Vectorize the graph layout optimization routine. Remove nested for loops and redundant calculations between cost terms and across iterations.
Speeds up layout_plumbing_engine by ~4.8x.
Current master:
mean time: 4.4294932126998905 seconds
max time: 4.631882905960083 seconds
min time: 4.306760311126709 seconds

This PR:
mean time: 0.9118092298507691 seconds
max time: 0.9235303401947021 seconds
min time: 0.8985953330993652 seconds

This change is

Yep everything makes sense, I'll do the plumbing engine comments in the same style then. Preemptive LGTM if it's just comment terminal_graph from here :0


Installation script does not leave the project directory on clean uninstall

When running the install.sh script to perform a clean uninstall (removing the whole installation directory) while inside the /opt/salomon directory it will be removed, but the command prompt will remain there even though it does not exist anymore.
Actually, the script should first leave the /opt/salomon directory and go to /opt as follows:
if [ "$(pwd)" = "$target_dir" ]; then
    cd $(pwd | sed -e "s/\/salomon$//g")
fi

However, this does not work properly.

This is an expected behavior. The script is run inside a subshell and cannot change the working directory of the parent shell.
Since this code is useless, it can be removed. Furthermore, changing the directory is not mandatory in order to uninstall the program.
The shell will remain in the /opt/salomon directory which actually does not exist anymore. Manually changing the directory or closing the shell solves this "issue".

Removed in version 1.16.4.


civic bot not adding "instance of" statement

example: https://www.wikidata.org/wiki/Q61818930
I think this sparql query shows all items with civicIDs without instance of statements: https://w.wiki/6ww
I'm expecting that all items added/edited by the civic variant bot should be on items having instance of sequence variant (Q15304597) or one of its children
probably need to have this issue fixed before the paper goes out...

This one is already on my plate and a tough one. There are basically, two issues here.

Currently, not all of the sequence ontology is in Wikidata yet, simply because it is not available as CC0. This means that if a variant type by its representation in the sequence ontology, is not yet in Wikidata it needs to be added manually. My understand of non-cc0 data is that one can not batch upload all, but adding a reference to a single SO is allowed.
Being able to upload all of SO to Wikidata would solve this.

Some variants don't have a specific variant type annotated in CIViC. The example given is of this type. (https://www.wikidata.org/wiki/Q61818930) Its CIViC record gives "Variant Type:
None specified.". Currently the bot ignores this. An easy fix here would be to add, as you suggest, to add "instance of sequence variant (Q15304597)".

Some time ago we increased the threshold wrt to quality. When we started we added all CIViC records, currently, we only add CIVIC records with high quality indication. This has lead to items in Wikidata that only mention the CIViC ID, plus of which gene it is a variant.

Concluding.

I have a list of missing sequence ontology terms that is already on my TODO list. If we don't add all terms this will be a recurring process, every time curators introduce new sequence variants.
For those terms that miss a variant type in CIViC, we add the default, being "sequence variant". Which leads to the question of what to do once a more specific variant emerges. (Delete the default?)


Water Hardness Calculator

Description
A Water Hardness Calculator helps users determine the hardness of their water supply. Water hardness is typically measured in parts per million (ppm) or grains per gallon (gpg), indicating the concentration of calcium and magnesium ions. This calculator can be useful for homeowners, aquarium enthusiasts, and anyone interested in maintaining water quality.
Key Features
Input Fields: Users can input the concentration of calcium and magnesium ions in their water sample.
Unit Selection: Option to choose between ppm and gpg for the input and output values.
Calculation: Converts ion concentration to water hardness and provides results in both ppm and gpg.
Hardness Categories: Displays the water hardness category (soft, moderately hard, hard, very hard) based on the calculated value.
Screenshots
No response
Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

with hardness, consider mentioning that it is safe for ......
you can find this kind of explanation on the internet, you can hardcode it while coding
for example:
if (ppm< x amt){
print" hard water
hard water is not safe for ......";


Cannot reproduce figures from paper

I tried to use the source code from this repository and variations, but could not reproduce the figures seen in the paper. Could you please help me with directions to reproduce Figure 3.b and Figure 3.d as shown below?

What is the command to be used with the given source code to generate the samples seen above?

Same problem here.


segment fault in activemsg

I experienced a segment fault when I ran dma_random test. The issue is that ContiguousPayload instance tries to free a mem segment with wrong address. The back trace is as follows. (the line numbers refer to dma branch)
[0] Thread 8 (Thread 0x7f4cdd6f7700 (LWP 32548)):
[0] #0  0x00007f4d3acd2619 in __libc_waitpid (pid=32556, stat_loc=stat_loc@entry=0x7f4cdd6f4390, options=options@entry=0) at ../sysdeps/unix/sysv/linux/waitpid.c:40
[0] #1  0x00007f4d3ac571d2 in do_system (line=) at ../sysdeps/posix/system.c:148
[0] #2  0x0000000000d038a8 in gasneti_bt_gdb ()
[0] #3  0x0000000000d0670b in gasneti_print_backtrace ()
[0] #4  0x0000000000d5f241 in gasneti_defaultSignalHandler ()
[0] #5  signal handler called
[0] #6  0x00007f4d3a557471 in opal_memory_ptmalloc2_int_free () from /usr/local/openmpi-1.8.2/lib/libopen-pal.so.6
[0] #7  0x00007f4d3a557bb3 in opal_memory_ptmalloc2_free () from /usr/local/openmpi-1.8.2/lib/libopen-pal.so.6
[0] #8  0x00000000009a59f6 in ContiguousPayload::copy_data (this=0x1a1e8a0, dest=0x7f4d30afcf40) at ../../runtime//activemsg.cc:2341
[0] #9  0x00000000009a43cc in OutgoingMessage::reserve_srcdata (this=0x1a45f40) at ../../runtime//activemsg.cc:1735
[0] #10 0x00000000009a73d5 in ActiveMessageEndpoint::enqueue_message (this=0x1a0d640, hdr=0x1a45f40, in_order=true) at ../../runtime//activemsg.cc:1080
[0] #11 0x00000000009aaf60 in EndpointManager::enqueue_message (this=0x1a0d1a0, target=1, hdr=0x1a45f40, in_order=true) at ../../runtime//activemsg.cc:1870
[0] #12 0x00000000009a5659 in enqueue_message (target=1, msgid=170, args=0x7f4cdd6f6d60, arg_size=24, payload=0x1a46060, payload_size=172, payload_mode=2, dstptr=0x0) at ../../runtime//activemsg.cc:2256
[0] #13 0x00000000009e257c in ActiveMessageMediumNoReply<170, Realm::MetadataResponseMessage::RequestArgs, &Realm::MetadataResponseMessage::handle_request>::request (dest=1, args=..., data=0x1a46060, datalen=172, payload_mode=2, dstptr=0x0) at ../../runtime/activemsg.h:595
[0] #14 0x00000000009e1daa in Realm::MetadataResponseMessage::send_request (target=1, id=3766419464, data=0x1a46060, datalen=172, payload_mode=2) at ../../runtime//realm/metadata.cc:297
[0] #15 0x00000000009e1c63 in Realm::MetadataRequestMessage::handle_request (args=...) at ../../runtime//realm/metadata.cc:248
[0] #16 0x0000000000a39176 in IncomingShortMessage<Realm::MetadataRequestMessage::RequestArgs, 169, &Realm::MetadataRequestMessage::handle_request, 2>::run_handler (this=0x7f4d33a0df20) at ../../runtime/activemsg.h:331
[0] #17 0x00000000009a3e93 in Realm::HandlerThread::thread_main (this=0x1a0e5a0) at ../../runtime//activemsg.cc:741
[0] #18 0x00000000009f9927 in Realm::PreemptableThread::thread_entry (data=0x1a0e5a0) at ../../runtime//realm/proc_impl.cc:1039
[0] #19 0x00007f4d3b9e1182 in start_thread (arg=0x7f4cdd6f7700) at pthread_create.c:312
[0] #20 0x00007f4d3ad0bfbd in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

Zhihao, can you provide the exact command line you're using?

I think the failure could be reproduced by following command line:
GASNETBACKTRACE=1 mpirun -n 2 --bind-to none -H n0000,n0001 ./fluid3d
-input in_300K.fluid -nbx 2
in which case, we try to partition the entire computation into 2 pieces,
and run each piece on one node.
Sincerely,
Zhihao
On Thu, Sep 3, 2015 at 1:11 PM, streichler<EMAIL_ADDRESS>wrote:

Zhihao, can you provide the exact command line you're using?
—
Reply to this email directly or view it on GitHub
https://github.com/StanfordLegion/legion/issues/50#issuecomment-137561165
.

My bad...please ignore the previous email, which leads to a fluid3d failure
case. I found the active message failure case when I run dma_random test in
dma branch. I am not sure if we can reproduce it in master branch.
On Thursday, September 3, 2015, Zhihao Jia<EMAIL_ADDRESS>wrote:

I think the failure could be reproduced by following command line:
GASNETBACKTRACE=1 mpirun -n 2 --bind-to none -H n0000,n0001 ./fluid3d
-input in_300K.fluid -nbx 2
in which case, we try to partition the entire computation into 2 pieces,
and run each piece on one node.
Sincerely,
Zhihao
On Thu, Sep 3, 2015 at 1:11 PM, streichler<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Zhihao, can you provide the exact command line you're using?
—
Reply to this email directly or view it on GitHub
https://github.com/StanfordLegion/legion/issues/50#issuecomment-137561165
.

--
Sincerely,
Zhihao

The following steps could help reproduce #50.
Step 1: check out to recent dma branch
Step 2: enter test/dma_random folder
Step 3: GASNET_BACKTRACE=1 mpirun -n 2 --bind-to none -H n0000,n0001
dma_random -ll:rsize 1024 -ll:dsize 1024 -ll:ahandlers 2
Sincerely,
Zhihao
On Thu, Sep 3, 2015 at 1:31 PM, Zhihao Jia<EMAIL_ADDRESS>wrote:

My bad...please ignore the previous email, which leads to a fluid3d
failure case. I found the active message failure case when I run dma_random
test in dma branch. I am not sure if we can reproduce it in master branch.
On Thursday, September 3, 2015, Zhihao Jia<EMAIL_ADDRESS>wrote:

I think the failure could be reproduced by following command line:
GASNETBACKTRACE=1 mpirun -n 2 --bind-to none -H n0000,n0001 ./fluid3d
-input in_300K.fluid -nbx 2
in which case, we try to partition the entire computation into 2 pieces,
and run each piece on one node.
Sincerely,
Zhihao
On Thu, Sep 3, 2015 at 1:11 PM, streichler<EMAIL_ADDRESS>wrote:

Zhihao, can you provide the exact command line you're using?
—
Reply to this email directly or view it on GitHub
https://github.com/StanfordLegion/legion/issues/50#issuecomment-137561165
.

--
Sincerely,
Zhihao

Zhihao, when I run the above command now, I reliably get this error instead:
dma_random: ../../runtime/realm/mem_impl.cc:604: virtual void Realm::RemoteMemory::get_bytes(off_t, void*, size_t): Assertion `kind == MemoryImpl::MKIND_RDMA' failed. *** Caught a fatal signal: SIGABRT(6) on node 0/2
Can you see if this is a new issue hiding the old one, or maybe the old one is fixed and this one
was hiding behind it?

Hi Sean,
The bug you encountered is a issue happening before #50, which is caused by
check_correctness task trying to read a remote instance (not RDMA-able). I
have pushed a fix into dma branch. You should be able to reproduce #50 on
n0000,n0001. By the way, don't forget to remove disk_file* before every
execution, otherwise, disk memory sanity check may fail.
I am working on merging your threading changes into dma branch. Do you
think if the threading changes may fix #50?
Sincerely,
Zhihao
On Wed, Sep 9, 2015 at 9:23 AM, streichler<EMAIL_ADDRESS>wrote:

Zhihao, when I run the above command now, I reliably get this error
instead:
dma_random: ../../runtime/realm/mem_impl.cc:604: virtual void
Realm::RemoteMemory::get_bytes(off_t, void*, size_t): Assertion `kind ==
MemoryImpl::MKIND_RDMA' failed.
*** Caught a fatal signal: SIGABRT(6) on node 0/2
Can you see if this is a new issue hiding the old one, or maybe the old
one is fixed and this one
was hiding behind it?
—
Reply to this email directly or view it on GitHub
https://github.com/StanfordLegion/legion/issues/50#issuecomment-138962512
.

This has been replaced by #61, which appears to be an issue inside the new DMA code.


Question : draw:editstop:save event ?

Hello,
I am adding extra data in a popup when editing. It would be nice to have an editstop:cancel event and an editstop:save event to known if I have to save these data or delete them. I can't manage the distinction in the current editstop event...
// Triggered on cancel and save
map.on('draw:editstop', function (e) {
    if (e.something == 'save') {
        saveExtraData()
    }
});

Or is there another way ?
Thank you

When will this functions will be added? Is there anyone has an alternative solution for this at the meantime.

Until now, I manage this missing feature with jquery. But this is a pretty ugly hack... It would be better to have separate events.
$('#map .leaflet-draw-section:eq(1) .leaflet-draw-actions li:eq(1) a').on('click', (e) => {
    console.log('cancel event');
    this.cancelEvent = true;
});

I submitted a PR (#651) that expands the info provided by the edit events,
but I am unsure of the status.
The editstop:save event that you want is called editdone. editstop without
a corresponding editdone is your editstop:cancel event.
On Mon, Jan 16, 2017 at 5:50 PM, visernchua<EMAIL_ADDRESS>wrote:

When will this functions will be added? Is there anyone has an alternative
solution for this at the meantime.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Leaflet/Leaflet.draw/issues/545#issuecomment-273001323,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzNEg-UdFZ9jFqJNdSg8egPkVsD0f7nks5rTB5NgaJpZM4ItYet
.

whats the status on this?

@danielduhh I found that if you want to check whenever to save button triggered you can use - draw:edited

It's 2022, does anyone have a solution for this problem?


Fix type url of stats tcp extension

Fix type url of stats tcp extension and use free form name "istio.stats"

Fixes https://github.com/istio/istio/issues/23277

In response to a cherrypick label: new pull request created: #23367


Steam Deck PS4 Controller UI Button Skipping Focus

Godot version
3.5.2 LTS
System information
GLES2
Issue description
I noticed an issue when running my Steam game on my Steam Deck attached to a Dock with a PS4 bluetooth controller connected. My scene tree isn't too advanced. Start with a Control / Hbox / 2 VBox / with Button nodes in each VBox. When the game is played and press either UP / Down D-Pad / Left Joystick Y-Axis the buttons skip focus selection to every other button.
Basically Button Index 0 jumps to 2, then 4, etc

This is my Input mapping

Steps to reproduce
Create a scene starting with Node2D / Control / HboxContainer / 2 VBoxContainers / Buttons, Export to Linux 64bit, In Desktop mode transfer game export to the Steam Deck, mark app as executable, add game to Steam Library, Go back into Game Mode, connect Steam Deck to a Dock, connect Steam Deck to a bluetooth controller (PS4 or Nintendo Switch), Now run game from your Steam Library. You should notice button skipping focus when trying to navigate with the connected bluetooth controller
Minimal reproduction project
Godot_Input_Skip.zip

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/66878.

I can confirm my bug is caused by setting UI input events to ALL instead of 0. For some reason there's a background event that mirrors when using bluetooth on Steam Deck / Linux

As far as I can tell this bug is fixed in 4.1.2+ I know this isn't a great question as it would probably be a huge amount of work but can the new input setup be back ported to the 3.5x as a maintenance release, and If not would there be any guidance / pointers on doing that myself?
Thank you for looking into this and fixing in future versions


More info?

I saw a blurb about this project but having difficulty finding any more information about it. From this Github, I see how the board can be built but as of yet, I have not seen any software to use on the hardware. Is more information forthcoming, such as instructions for programming and pointers on what can be accomplished with it?

I am also interested in programming this.

Hi, this project is a HW platform. The board can be programmed using the Arduino IDE. All the peripherals except the RF have an Arduino library available so you can use it. For the RF IC, you need to modify the library from LSatan (SMARTRC-CC1101-Driver-Lib), or download the library from the repository of Joel Serna Moreno, who has a fork of the library modified for the RP2040.
Thanks!


95 Weekday: Handles 1582 dates incorrectly

The following line in the original code leads to an error if dates before 1582 are supplied (which is fine)
290 IF Y-1582 <0 THEN 1300
However it means the day of week calculation is performed on days in 1582, but before 15 October 1582 (the date the Gregorian calendar started).
I think the test should be changed to check that the supplied date is on or after 15 October 1582.

I am currently looking at tidying up the Javascript implementation, but I am unsure whether I should unilaterally fix this on one implementation.

sure, if it is a bug in the original code please put a note in the readme.md in the game folder!

One of my todo items is to add a "translator / port notes" section to the readme.md of each game..

hurry up~~~~ the translator


chore: migrate benchmarks to Deno.bench

$ deno bench --unstable
Check file:///home/erfanium/Documents/deno_bson/bench/object-id-compare_bench.ts
Check file:///home/erfanium/Documents/deno_bson/bench/simple-serialize-deserialize_bench.ts
running 3 benches from file:///home/erfanium/Documents/deno_bson/bench/object-id-compare_bench.ts
bench object-id-compare (stable) ... 1000 iterations 86,810 ns/iter (77,954..283,496 ns/iter) ok (184ms)
bench object-id-compare (js-bson) ... 1000 iterations 124,280 ns/iter (112,838..236,271 ns/iter) ok (263ms)
bench object-id-compare (local) ... 1000 iterations 89,032 ns/iter (80,158..333,508 ns/iter) ok (185ms)
running 3 benches from file:///home/erfanium/Documents/deno_bson/bench/simple-serialize-deserialize_bench.ts
bench simple-serialize-deserialize (stable) ... 1000 iterations 328,532 ns/iter (297,593..603,777 ns/iter) ok (680ms)
bench simple-serialize-deserialize (js-bson) ... 1000 iterations 509,846 ns/iter (467,672..1,697,680 ns/iter) ok (1s)
bench simple-serialize-deserialize (local) ... 1000 iterations 330,932 ns/iter (303,328..2,282,048 ns/iter) ok (680ms)

bench result: ok. 6 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out (3s)

Is it possible to use the third party lib we used with this one?
To have a more fancy output?

https://discord.com/channels/684898665143206084/689420767620104201/954115734953402449


chore: import repository https://github.com/gaocheng001/wodenode.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add spawn strategies

Hi again.
I made a little bit refactoring)
What was made:

create spawnFactory with registered spawn strategies for each spawnType. Spawn funcions and parsing spawn config moved from Emitter to strategies. SpawnFactory supporting add new strategies and override exists by extend from it. The Emitter has become easier.

removed _spawnFunc() from Emitter. Insteand using strategy.spawn(). At the start of refactoring I don't remove it and mark as deprecated and set _spawnFunc = strategy.spawn for legacy. But this solution has some limitation i.e. if strategy has other function which using in spawn than this functions not available because to _spawnFunc set link to function and strategy context(this) will be missing. Also was additional code for decide which function use _spawnFunc or strategy.spawn(). I decided to remove _spawnFunc() and using only strategy.spawn().

add public getter for rotation property and rename to _rotation protected property. Using in spawn functions on strategies.

I tried to do as less as possible breaking changes on this refactoring.
Potential BC:

using _spawnFunc in extended classes. Early this function was private ( I made this function protected 3 days ago and think than no one had time to use it). Using hack emitter["_spawnFunc"] not in count.
rotation -> _rotation. Can affect if try to set value to 'rotation' property in extended from Emitter class. But to set rotation has rotate() function so this case must be quite rare.

Some examples how it help to customize emitter:
I will use previos example with spawn rect with normal
1) register new spawn strategy 'rectNormal'
Emitter
import {Emitter, SpawnFactory} from "pixi-particles";
import {SpawnFactoryWithNew} from "./newfactory/SpawnFactoryWithNew";

export class ExtendedEmitterWithNewStrategies extends Emitter {

    protected getSpawnFactory(): SpawnFactory {
        return SpawnFactoryWithNew.instance;
    }
}

Factory
import {SpawnFactory} from "pixi-particles";
import {SpawnRectNormalStrategy} from "./strategy/SpawnRectNormalStrategy";

export class SpawnFactoryWithNew extends SpawnFactory {

    protected static _instance: SpawnFactoryWithNew;

    public static get instance(): SpawnFactoryWithNew {
        if (!this._instance) {
            this._instance = new SpawnFactoryWithNew();
        }

        return this._instance;
    }

    protected constructor() {
        super();

        this.strategies["rectNormal"] = new SpawnRectNormalStrategy();
    }
}

Strategy
import {Emitter, EmitterConfig, ISpawnStrategy, OldEmitterConfig, Particle, Temp} from "pixi-particles";
import Rectangle = PIXI.Rectangle;

export class SpawnRectNormalStrategy implements ISpawnStrategy {

    parseConfig(emitter: Emitter, config: EmitterConfig | OldEmitterConfig): void {
        emitter.spawnType = "rect";
        const spawnRect = config.spawnRect;
        emitter.spawnRect = new Rectangle(spawnRect.x, spawnRect.y, spawnRect.w, spawnRect.h);
    }

    spawn(p: Particle, emitPosX: number, emitPosY: number, i?: number): void {
        const emitter = p.emitter;
        //place the particle at a random point in the rectangle borders
        const isHorizontalBorder: boolean = !!Math.round(Math.random());
        if (isHorizontalBorder) {
            const isTop: boolean = !!Math.round(Math.random());
            Temp.point.x = Math.random() * emitter.spawnRect.width + emitter.spawnRect.x;
            Temp.point.y = (isTop ? 0 : 1) * emitter.spawnRect.height + emitter.spawnRect.y;
            p.rotation = isTop ? 270 : 90;
        } else {
            const isLeft: boolean = !!Math.round(Math.random());
            Temp.point.x = (isLeft ? 0 : 1) * emitter.spawnRect.width + emitter.spawnRect.x;
            Temp.point.y = Math.random() * emitter.spawnRect.height + emitter.spawnRect.y;
            p.rotation = isLeft ? 180 : 0;
        }

        p.position.x = emitPosX + Temp.point.x;
        p.position.y = emitPosY + Temp.point.y;
    }

}

2. Change already exists strategy
Emitter
import {Emitter, SpawnFactory} from "pixi-particles";
import {SpawnFactoryWithChanged} from "./changefactory/SpawnFactoryWithChanged";

export class ExtendedEmitterWithChangedStrategies extends Emitter {

    public spawnByNormal: boolean = false;

    protected getSpawnFactory(): SpawnFactory {
        return SpawnFactoryWithChanged.instance;
    }
}

Factory
import {SpawnFactory} from "pixi-particles";
import {SpawnRectExtendedStrategy} from "./strategy/SpawnRectExtendedStrategy";

export class SpawnFactoryWithChanged extends SpawnFactory {
    protected static _instance: SpawnFactoryWithChanged;

    public static get instance(): SpawnFactoryWithChanged {
        if (!this._instance) {
            this._instance = new SpawnFactoryWithChanged();
        }

        return this._instance;
    }

    protected constructor() {
        super();

        this.strategies["rect"] = new SpawnRectExtendedStrategy();
    }
}

Strategy
import {Emitter, EmitterConfig, OldEmitterConfig, Particle, SpawnRectStrategy, Temp} from "pixi-particles";
import {ExtendedEmitterWithChangedStrategies} from "../../ExtendedEmitterWithChangedStrategies";
import {EmitterExtendedConfig} from "../config/EmitterExtendedConfig";

export class SpawnRectExtendedStrategy extends SpawnRectStrategy {

    parseConfig(emitter: Emitter, config: EmitterConfig | OldEmitterConfig): void {
        super.parseConfig(emitter, config);

        const extendedEmitter: ExtendedEmitterWithChangedStrategies = emitter as ExtendedEmitterWithChangedStrategies;
        const extendedConfig: EmitterExtendedConfig = config as EmitterExtendedConfig;
        extendedEmitter.spawnByNormal = extendedConfig.spawnByNormal;
    }

    spawn(p: Particle, emitPosX: number, emitPosY: number): void {
        const emitter: ExtendedEmitterWithChangedStrategies = p.emitter as ExtendedEmitterWithChangedStrategies;
        if ( emitter.spawnByNormal) {
            this.spawnByNormal(p, emitPosX, emitPosY);
        } else {
            super.spawn(p, emitPosX, emitPosY);
        }
    }

    protected spawnByNormal(p: Particle, emitPosX: number, emitPosY: number, i?: number): void {
        const emitter = p.emitter;
        //place the particle at a random point in the rectangle borders
        const isHorizontalBorder: boolean = !!Math.round(Math.random());
        if (isHorizontalBorder) {
            const isTop: boolean = !!Math.round(Math.random());
            Temp.point.x = Math.random() * emitter.spawnRect.width + emitter.spawnRect.x;
            Temp.point.y = (isTop ? 0 : 1) * emitter.spawnRect.height + emitter.spawnRect.y;
            p.rotation = isTop ? 270 : 90;
        } else {
            const isLeft: boolean = !!Math.round(Math.random());
            Temp.point.x = (isLeft ? 0 : 1) * emitter.spawnRect.width + emitter.spawnRect.x;
            Temp.point.y = Math.random() * emitter.spawnRect.height + emitter.spawnRect.y;
            p.rotation = isLeft ? 180 : 0;
        }

        p.position.x = emitPosX + Temp.point.x;
        p.position.y = emitPosY + Temp.point.y;
    }
}

Config
import {EmitterConfig} from "pixi-particles";

export interface EmitterExtendedConfig extends EmitterConfig {
    spawnByNormal?: boolean;
}

Hi.
I made some changes.

Added SpawnFactory.setStrategy(). Now 2 ways to customize spawn strategies: by override classes and by setStrategy(). First way garantie that you will have custom strategies in any time and you not forgot set you custom strategy :) . Second way add ability change or add strategies on the fly. This strategies will apply to new created emitter instances. And you can combine first and second ways.
I don't move properties which using in strategies from emitter to strategies. If someone extend from Emmiter and override some spawn logic - yes, in this case will breaking changes. But exist another cases when this properties changes external, i.e. change spawn area size by emitterInstance.spawnRect property. I think that this case more frequent than override spawn logic. And in this case code will be without breaking changes because all properties stay on old place. As I wrote upper, I tried to do as less as possible breaking changes. And second reason - now created only one strategy instance per spawn type for all emitter instances. So we can not store some concrete emitter data on this strategy instances.
Sharing temp point was not my idea :) . Previously helperPoint was a global variable so I only move it from global space to class static property. But using both cases - not good. I changed it - now every strategy instance has local helper point variable.
<X>Strategy - I think - it is bad idea because if pattern strategy will using in another place than will not clear which strategy where using. Prefix Spawn<X>Strategy say that this strategy about spawn particles.

Ah, I missed that spawn strategies were global, and was thinking that they were one per emitter. In that case, I would suggest that parseConfig() on each strategy returns an object containing the parsed data, and the emitter saves that object and gives it back during spawn(). That way, the Emitter and its properties don't need to know about any of the strategy specifics, only that the strategy exists.

With release 5.0.0, the emitter now uses configurable behaviors for everything, including spawn strategies, which should cover the use cases here.


Fix missing nowait

Did not find any reason for PRIVMSG to not pass nowait.

Coverage remained the same at 97.402% when pulling c79033f9b0565f33707b1d5589cb729c70bc20d5 on adongy:patch-1 into 61459e5d699ff7b68a924b30c1756798e11c0cd5 on gawel:master.

When nowait is false messages are handle by process_queue https://github.com/gawel/irc3/blob/master/irc3/init.py#L200 which take care of flood* parameters. If you use nowait=true, your bot will easily flood the server

What I'm trying to say is that if you try to use privmsg with nowait=True, it will do nothing since nowait is not passed to the underlying send_line.

Oh! Right. I miss read the patch (and the comment was not helping :) ), sorry


clean-repos: Don't delete submodules from index with "clone" method

With pull.method == clone, nlbuild-autotools doesn't require a git repo
in the parent project. So don't run git commands there.
This fixes an incompatibility between the clone method and submodules
where cleaning will leave staged deletions in the index.

This seems to be a regression of desired functionality: post-patch, if I run make -f Makefile-bootstrap clean-repos, the repos present in repos.conf are still in the working tree.

What if we check if there's a submodule ptr at that path, and only leave the empty directory in that case?

Git will put an empty directory at a submodule and if it's missing, the tree is considered dirty. It doesn't really do any harm.


Addressing issue preventing the display of the user in the history table

NIFI-2505:

Updating the user identity field in the ActionDTO.

Reviewing...

Looks good, will merge to master


Stepwise regression doesn't use test/train validation when producing intermediate scores

The current implementation of stepwise regression via neurencoding.utils.SequentialSelector does not use the test data when computing scores at each added regressor.
This is because if the model uses the testing data to select a new regressor during the sequential process this is technically a violation of the test/train paradigm. As such only the improvement on the training data is reported in the .deltas_ and .scores_ attributes.
The real solution is to use the testing data to select the next regressor and store the improvement for the new model on both the testing and training data at each iteration. This needs to be done.

The next feature is chosen by scoring on the train dataset, but the score for the same feature on the test set is also provided and stored. That way we can have a full coverage of the data and report test scores. New attributes of sfs are:
sfs = SequentialSelector(nglm)
sfs.fit(full_scores=True)
sfs.scores_test_
sfs.scores_train_
sfs.deltas_test_
sfs.deltas_train_
# optionally, if the full_scores flag is passed as true:
sfs.full_scores_test_
sfs.full_scores_train_

The full_scores option is to store all scores for the intermediate candidate features, not just the maximum.


use Dockerfile from code-generator for building controller image

Issue #, if available: https://github.com/aws-controllers-k8s/community/issues/1112
Description of changes:

use Dockerfile from code-generator for building controller image
This helps in maintaining only single Dockerfile for building ACK controller images

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

Yep, totally preferable. Thanks
/lgtm