Pinentry Serve returned error: EOF

Describe the bug
When running echo "GETPIN" | pinentry-touchid I get the following response
OK Hi from pinentry-touchid!
OK
2022/06/06 10:54:16 Pinentry Serve returned error: EOF

pinentry-mac runs as expected - I have tried to unsinstall & re-install pinentry-mac & pinentry-touchid with no sucess at this point. I have also been looking for similar issues but haven't found a solution as of yet.
System information
macOS

Architecture: Intel
Version: 12.3.1

GPG

Output of gpg --version

gpg (GnuPG) 2.3.6
libgcrypt 1.10.1
Copyright (C) 2021 Free Software Foundation, Inc.
License GNU GPL-3.0-or-later <https://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Home: /Users/<USER>/.gnupg
Supported algorithms:
Pubkey: RSA, ELG, DSA, ECDH, ECDSA, EDDSA
Cipher: IDEA, 3DES, CAST5, BLOWFISH, AES, AES192, AES256, TWOFISH,
        CAMELLIA128, CAMELLIA192, CAMELLIA256
AEAD: EAX, OCB
Hash: SHA1, RIPEMD160, SHA256, SHA384, SHA512, SHA224
Compression: Uncompressed, ZIP, ZLIB, BZIP2

Installed via Homebrew?

Yes.
Configuration

Please attach the output of the command gpgconf.

gpg:OpenPGP:/usr/local/Cellar/gnupg/2.3.6/bin/gpg
gpgsm:S/MIME:/usr/local/Cellar/gnupg/2.3.6/bin/gpgsm
keyboxd:Public Keys:/usr/local/Cellar/gnupg/2.3.6/libexec/keyboxd
gpg-agent:Private Keys:/usr/local/Cellar/gnupg/2.3.6/bin/gpg-agent
scdaemon:Smartcards:/usr/local/Cellar/gnupg/2.3.6/libexec/scdaemon
dirmngr:Network:/usr/local/Cellar/gnupg/2.3.6/bin/dirmngr
pinentry:Passphrase Entry:/usr/local/opt/pinentry/bin/pinentry

gpg-agent.conf

cat /Users/<USERNAME>/.gnupg/gpg-agent.conf
debug-level basic
log-file /Users/<USERNAME>.gnupg/gpg-agent.log
pinentry-program /usr/local/opt/pinentry-touchid/bin/pinentry-touchid

Logs
gpg-agent:
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK Pleased to meet you, process 36805
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- RESET
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- OPTION ttyname=/dev/ttys004
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- OPTION ttytype=xterm-256color
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- OPTION lc-ctype=UTF-8
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- reloadagent
2022-06-06 10:53:55 gpg-agent[36329] SIGHUP received - re-reading configuration and flushing cache
2022-06-06 10:53:55 gpg-agent[36329] reading options from '/Users/<USERNAME>/.gnupg/gpg-agent.conf'
2022-06-06 10:53:55 gpg-agent[36329] enabled debug flags: ipc
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 -> OK
2022-06-06 10:53:55 gpg-agent[36329] DBG: chan_8 <- [eof]

pinentry-touchid:
2022/06/06 10:49:49 main.go:105: Ready!
2022/06/06 10:49:49 main.go:211: Confirm was called!
2022/06/06 10:49:50 main.go:105: Ready!
2022/06/06 10:49:50 main.go:211: Confirm was called!
2022/06/06 10:50:15 main.go:105: Ready!
2022/06/06 10:51:10 main.go:105: Ready!
2022/06/06 10:51:45 main.go:105: Ready!
2022/06/06 10:54:16 main.go:105: Ready!
2022/06/06 10:57:46 main.go:105: Ready!
2022/06/06 10:57:59 main.go:105: Ready!

@stffndtz can you give it a try to the latest release (v0.0.3) and check if it fixes the issue?

@jorgelbg It seems I am still getting the same error after the update:
echo "GETPIN" | pinentry-touchid
OK Hi from pinentry-touchid!
D hello
OK
Pinentry Serve returned error: EOF

Post update (via brew) I ran
/usr/local/opt/pinentry-touchid/bin/pinentry-touchid -fix && gpg-connect-agent reloadagent /bye

Let me know if I can help!

I have the same issue. I can use pinentry-touchid for gpg-agent, but I cannot use it generally for other programs.
$ echo GETPIN | pinentry-touchid
OK Hi from pinentry-touchid!
D abc
OK
Pinentry Serve returned error: EOF

Could this be the culprit?
https://github.com/jorgelbg/pinentry-touchid/blob/0235243df4d2795475cc2067b89a0f69f1e6b798/main.go#L428-L431
Note the response from pinentry-mac is not Hi from pinentry-mac!. In fact it is Pleased to meet you. Viz.
$ echo GETPIN | pinentry-mac
OK Pleased to meet you
D abc
OK

So maybe line 428 should be changed to:

Just to be clear, if you are only checking directly with:
$ echo GETPIN | pinentry-touchid

This is rightfully showing the pinentry-mac GUI. This happens because you haven't followed the full assuan protocol to talk to pinentry. Yes, GETPIN is a valid assuan command that pinentry-touchid understands but since no information about your GPG key has been provided, it assumes that there is no item in the keychain and asks you to provide one (as a fallback mechanism).
You need to set pinentry-touchid as your pinentry-program in ~/.gnupg/gpg-agent.conf by adding the following line:
pinentry-program /usr/local/bin/pinentry-touchid

and also reload your gpg-agent:
$ gpg-connect-agent reloadagent /bye

For checking that it is working you can use:
$ echo 1234 | gpg -as -

You can also interact directly via stdin with pinentry-touchid, but you need to do the full dance of providing the potions that the gpg-agent usually sends to any pinentry program.

Just to be clear, if you are only checking directly with:
$ echo GETPIN | pinentry-touchid

This is rightfully showing the pinentry-mac GUI. This happens because you haven't followed the full assuan protocol to talk to pinentry. Yes, GETPIN is a valid assuan command that pinentry-touchid understands but since no information about your GPG key has been provided, it assumes that there is no item in the keychain and asks you to provide one (as a fallback mechanism).
You need to set pinentry-touchid as your pinentry-program in ~/.gnupg/gpg-agent.conf by adding the following line:
pinentry-program /usr/local/bin/pinentry-touchid

and also reload your gpg-agent:
$ gpg-connect-agent reloadagent /bye

For checking that it is working you can use:
$ echo 1234 | gpg -as -

You can also interact directly via stdin with pinentry-touchid, but you need to do the full dance of providing the potions that the gpg-agent usually sends to any pinentry program.

This comment doesn't appear to have any relation to this bug report.

What I meant is that just sending GETPIN to pinentry-touchid is not enough and it does not considered a valid assuan session, the gpg-agent (and I would expect the same from the program that is trying to use it) should do the valid session.
In this case EOF is just a report of an early disconnection. That being said it is true that pinentry-mac does not consider this early termination as an unexpected error. I've adjusted the behaviour of the the error check in 82cc231b007b822b4eb11a9d8ba2e773aaae063c and also set the default prompt in case none is passed, which provides a more similar experience to what pinentry-mac does. Can you give it a try?

What I meant is that just sending GETPIN to pinentry-touchid is not enough and it does not considered a valid assuan session, the gpg-agent (and I would expect the same from the program that is trying to use it) should do the valid session.
In this case EOF is just a report of an early disconnection. That being said it is true that pinentry-mac does not consider this early termination as an unexpected error. I've adjusted the behaviour of the the error check in 82cc231 and also set the default prompt in case none is passed, which provides a more similar experience to what pinentry-mac does. Can you give it a try?

Thanks, I'll give it a try soon.
To clarify, the problem (for me at least) is that echo GETPIN | pinentry-touchid is a perfectly valid way to use a pinentry program, however it returns an error in this case even though a PIN can be fetched from the user. So, the program succeeds in collecting a PIN, but it prints an error message and, more importantly, returns a non-zero exit code.
In my case, I am attempting to use software called passage which in return uses software called rage which calls pinentry to ask for a PIN. I symlinked pinentry to pinentry-touchid because I want to use it for everything. It fails however. I assume what is happening is that pinentry-touchid falls back to pinentry-mac but doesn't return the exit code from the latter. And it also prints that error.

I don't know anything about Assuan commands, but the way that rage uses pinentry triggers this error, so I assume that something about pinentry-touchid is not conforming to spec here. I'll try the fix soon.

Sorry, I don't use go and don't know how to compile this project, so I can't try your fix.
If you add a Makefile for dummies, I can make.

I've added a step to the CI that should build & upload the binaries. You could grab the built binaries from the summary page of the workflow. For instance for the latest commit: https://github.com/jorgelbg/pinentry-touchid/actions/runs/2831484675 (at the bottom).

I had a chance to take a look under the hood at what rage sends to the pinentry program and I think that pinentry-touchid will keep falling back always to pinentry-mac.
When used with PGP the gpg-agent sends some identifying metadata about the key that is requesting the passphrase. This metadata is used by pinentry-touchid to tag the passphrase when it is stored in the keychain, since it is possible for a user to have multiple keys. Since this metadata it is not provided at all by rage pinentry-touchid cannot identify (nor currently store) an item in the keychain and falls back to pinentry-mac for the user to provide a password.
One way to go is to use a default label/name for tagging the keychain item. The downside of this is that you wouldn't be able to have more than 1 keys stored in the keychain (for rage) because there wouldn't be any way of distinguishing them apart.
Anyhow, this is my interpretation of what I've seen on the interaction between rage and whatever pinentry program is configured.

Excellent, thanks for that clear explanation. I had wondered how pinentry-touchid knows which secret is being requested. That explains it.
The gpg-agent must pass an identifier string to the Assuan GETPIN instruction, or use a different instruction. Is it possible for pinentry-touchid to support caching of secrets with arbitrary identifiers/tags? Could you give an example of a shell command which does this, like echo GETPIN MY_ID | pinentry-touchid?
This would not help with rage specifically, but it would allow other programs to easily use pinentry-touchid. This may pose an additional security risk, if programs supplied predictable (guessable) identifiers. This could be somewhat reduced by having the pinentry-touchid program display clearly which program is requesting which secret (process name, tag/identifier requested).

The majority of the information is passed via the SETDESC command. The pinentry program is expected to extract the metadata from the description. For instance: https://github.com/jorgelbg/pinentry-touchid/blob/1170eb6bc7b23313aee622887b47b77be6e5fb5f/main.go#L52-L53
If the description contains any of the info that matches those regexes (plus the key ID) it will be used to label the keychain item.
If rage passes down some sort of key id or anything else that could be used to label and retrieve the password, pinentry-touchid should work similarly to how it works with the gpg-agent. This could be also shared via the description that rage sets when calling the pinentry program.

This would not help with rage specifically, but it would allow other programs to easily use pinentry-touchid. This may pose an additional security risk, if programs supplied predictable (guessable) identifiers. This could be somewhat reduced by having the pinentry-touchid program display clearly which program is requesting which secret (process name, tag/identifier requested).

The info passed down to pinentry-touchid is essentially provided with the intention of helping the user identify which passphrase is needed (or to automatically fetch it from the keychain) so it shouldn't be insecure by default. In the gpg-agent case, the private key is never shared. In any case, is responsibility of the caller that sends the commands to pinentry.
Having said that, I'm not familiar enough with age/rage to say which info could potentially be shared with pinentry-touchid 😅.


Added link to Kubestronauts in README.md

Suggestion that we add a link to the Kubestronaut page on cncf.io to the README.md

Thanks!


optuna: n_jobs is getting deprecated

It appears that with the latest optuna, parallelism within an experiment viz n_jobs is getting deprecated. This issue is to explore the implications and research any alternatives

In the latest stable optuna version 2.10, n_jobs is deprecated. But, there is a PR about re-introducing it back.
concurrent.futures is one of the alternative.
Deprecate n_jobs: https://github.com/optuna/optuna/pull/2560
Reintroduce the feature of n_jobs: https://github.com/optuna/optuna/issues/2937
About concurrent.futures : https://github.com/optuna/optuna/pull/2269


Support capacitor 6.1.0

Describe the bug
The package cannot be installed with capacitor core version > 6.0.0. Also the the package on npm is behind. Version 6 is only available in the next tag.
To Reproduce
Steps to reproduce the behavior:
Install the package with<EMAIL_ADDRESS>Thank you for your support!

+1

Thanks for the Issue. This was a complete failure on my part.
Fixed and released v6.0.2. Please try this. Thanks.

No worries, thanks for the quick fix :)


drop :open styles

Description
Older versions of Chrome with experimental web platform features turned on (Chrome 113) supported the :open pseudo class to determine if a popover was open or not. We have support code for this in the form of CSS that checks @supports ([popover]:open).
The problem is that now causes issues with certain versions of WebKit, which claim to support :open and pass the CSS check, only to unapply the popover styles but not actually support popover. In other words, there are versions of WebKit - such as Gnome Web - for which this breaks.
Given the popularity of Gnome Web (a fairly small % of total market share) compared with Chrome 113 with Experimental Web Platform Features enabled (an even smaller %), it makes sense to remove this CSS in order to support browsers like Gnome Web.
Steps to test/reproduce
Please explain how to best reproduce the issue and/or test the changes locally (including the pages/URLs/views/states to review).
Visit the demo page in Gnome Web. Everything is broken.
Visit this PR demo page in Gnome Web. Everything should work.
/cc @WebReflection
I've manually tested this patch in the following browsers:

Modern WebKit (Safari 14.1)
Latest Edge
Latest Chrome
Latest Firefox
Chrome Dev
Gnome Web
QuteBrowser v3.0.0
Palemoon (works but has some CSS issues unrelated to this PR)
Chrome 113 (w/ Experimental Web Platform Features) - it doesn't work here - this is the trade-off.
Chrome 113 (w/o Experimental Web Platform Features)
Chrome 114 (w/ Experimental Web Platform Features)
Chrome 114 (w/ Experimental Web Platform Features)
Latest Safari iOS

Thank you very much for this and I can confirm everything works as expected now in Web: LGTM
If worth anything, here my thoughts:
/* This older `:open` pseudo selector is deprecated and support will be removed
in a later release. */

I think that comment says it all, hopefully the lesson learned here is to drop ASAP any experimental hack for forever green browsers (such as Chrome/ium).
On the other hand, a @support([any-attribute]:pseudo) seems to be a way too generic check to me, and I wonder if there's any way that @support could be improved to know if such support is experimental ... is this maybe something to discuss with the CSS working group? an :experimental selector that exists only if a feature is, effectively, experimental?
This would allow more fine-tuning experiments targeting earlier versions (as it was in this case) without breaking browsers that implemented the "not experimental anymore" behavior.
/cc @tabatkins for thoughts about this (in case is not possible already)

@keithamus apologies for the late ping but ... any ETA when this will be deployed on GitHub ??? I still see the popover and it keeps being annoying :sweat_smile: thanks!

It’ll go out likely this week.


[Bug] [broker]Disk iops cannot be higher than 4k，seems to be affected by Direct I/O

Search before asking

[X] I searched in the issues and found nothing similar.

Version
3 Aliyun ecs，broker and bookie share a disk
disk have 16800 iops
pulsar-version：3.0.0
CentOS Linux release 7.9.2009 (Core)
3.10.0-1160.88.1.el7.x86_64
Minimal reproduce step
Through the fni tool, when testing disk performance, it is found that turning on/off direct has a great impact on disk iops

What did you expect to see?
I hope to see that the disk io of pulsar can exceed 4k
What did you see instead?

Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I think this is relate to aliyun ecs and disk io. not a pulsar problem.


Verbose unchanged after initialization

The module initializes with verbose = 2, and is not changing to verbose = 0 even after setting the parameter.

Hi @dragarok 👍
Can you please upgrade to the latest version and check - I have made a lot of changes and this should be fixed.
pip install feature --upgrade
Please confirm
Thanks
AutoViML


ADFS Support?

Any plan to add ADFS support as the.net library seems to be working on it.?
https://github.com/AzureAD/microsoft-authentication-library-for-dotnet/pull/143

Agreed - it would be nice to move to MSAL from ADAL - with ADFS support being the only missing feature for me.

I second this idea.
Unfortunately, ADFS 2016 cannot support JavaScript at this time, because they lack CORS-Features - therefore you cannot read config data or key-material from client-based js.
It would be nice, if the MSAL Team would engage the ADFS-Team with that Problem.

Doesn't work (yet) with ADFS 2019 either even if cors is supported, msal says: "ERROR Error: InvalidAuthorityType" when providing an adfs url instead of aad b2b. Just use ADAL.js for now with ADFS.

Works like a charm using oidc-client.js with ADFS 2019 with the dropped "resource" requirement and CORS support.

@BatmanAMA  @Only1MrAnderson We are testing our latest MSAL JS against this and will update the documentation on ADFS supportability based on the same. Please expect this to be after our current release though.

@negoe can you please follow up on the ADFS support for MSALs? We do know that we do not support the endpoint directly, Clear documentation will help folks who come to this route.

@sameerag this is something that would be really useful, teams report that all is working with our .net clients, just the js that its a problem with still.

@DarylThayil well, its the end of September, whats the word on MSAL for ADFS???
I really would like to ditch my ADAL JS,, which should be sent back to hell where it came from

@LastTribunal  I have done some digging on this and looks like there is more dev work needed to ensure ADFS 2019 is working in MSAL JS. Our initial assumption was that it is more of guidance issue than dev work, which is not true. Most of this work is was heavily contributed by ADFS team on other MSALs and I have started getting together the details to have this supported in MSAL JS.
Unfortunately this may not be finished in September, but I will keep updating the progress here.

@sameerag thanks for the update.  I really hope I can get my hands on it soon, i have a hybrid app that works both with Azure AD and ADFS, using ADAL for angular.  There are many issues that are making my app almost unusable..

@sameerag  any update on the ETA?

Not taken up yet as we are closing a major release, unfortunately,  but it is in my list for October. Will update here soon.

@sameerag update?

@LastTribunal Here is what I gathered so far on ADFS as this thread already mentions above:

OnPrem ADFS is not yet supported and may not be for a while
ADFS 2019 should be supported out of the box.

In this context, I fixed the bug that was throwing the "ERROR Error: InvalidAuthorityType" in msal js. Can you please try checking with the latest version of msal : msal-1.2.0-beta.3 and we can take it forward as it comes?

@sameerag, I dont get it.  If msal with adfs is not supported, what is the purpose of testing for this error?
Are you saying that msal should work with adfs 2019 by default?
Can you clarify??

Yes. Msal with ADFS 2019 should work out of the box.

do you have a demo msal angular app that i can test against ADFS?

Just note, from what I understand, the js library is still not supported for adfs, only the .net library

@LastTribunal Can you please try this using msal-core sample substituting the authority with an ADFS 2019 one? Currently we are working on upgrading angular to msal-core and currently do not have an available sample.
msal latest beta version in npm is the recommended version.

@sameerag you guys need to come up with an MSAL Angular implementation and demo app that showcases interfacing with both AAD and ADFS ASAP  No one cares about MSAL otherwise... what problem does it solve as of right now that ADAL doesnt? The only sick patient is ADALJS/ADALAngular with near fatal symptoms when interfacing with ADFS.  I hope the powers at MS are reading this.

@LastTribunal We are working on Angular release already, please check the project board angular for the progress. We are also working on samples improvements the remaining part of the year. We will follow up with an ADFS 2019 sample once Angular is available.
msal js adds support for v2 endpoints, combines support for school, work accounts, B2C etc, more details on this here. In msal js we also enhanced the API interface and new features which are not present in adal js.
The request to try msal-core meanwhile is to unblock those set of people who are already using msal-angular with custom improvements on the msal-core. Hope this clarifies the strategy.
Please note that ADFS on prem is not supported and we do not have line of sight on when that would be supported from microsoft client libraries.

Hello:
I've seen that the version 1.2.0 support adfs, but only url microsoft? ¿Is possible use a url adfs private and use Msal?
I try do this but I've a error why the library try to redirect a url that it not the mine
https://login.microsoftonline.com/common/discovery/instance?api-version=1.0&authorization_endpoint=https://****.**********.com/adfs/oauth2/v2.0/authorize 400 (Bad Request)

Hi @marisaoliva We would need some time to address this as this would need some significant changes in our authority code which determines the end point to call for token acquisition. I am tagging this as a feature request and we will update the roadmap when we can invest on this.

Hi,
Just wondering if there is any update for MSAL and on-premise ADFS? We have a React application that would need ADFS access tokens and we got that working using react-adal, however, we're switching to MSAL as ADAL is not supported anymore.
Thanks!

I'd also like to pile on this one. Considering that MSAL.NET supports direct connections to on-prem AD FS 2019+ (with some minor tweaking of generated JWTs, it turns out) I had more or less assumed that MSAL.js would also have similar support, until reading this issue. My colleagues and I would be very keen to see this becoming a reality so that we have similar approaches available for SPAs.

I've been waiting for support from this so called MSAL team, all I got was empty promises and excuses.  Wasted my time

Update: We expect to add support for ADFS 2019 in 1.4.0. Follow the above pull request for more. Sorry this took so long, appreciate the patience!


Does it work for unicode strings ? (solved

It only works for ASCII but not unicode for me. (it returns results of all the same distance) I'm trying to write a dictionary

I enabled damerau and now it works

It’s a serialization issue. I added serde derive to it, and used flexbuffers. It doesn't work when deserializing from disk.
Something must have changed during the process. It works with yaml

now use cbor


Swagger UI URL issue when Netbox is published behind a reverse proxy

Issue type: feature request

Python version: 3.0
NetBox version: 2.0b3

We publish Netbox behind a reverse proxy in HTTPS. But in the Swagger UI, all the links and actions have a protocol set to http. So it doesn't work.
It would be nice if Swagger could use relative URIs or if we could specify the exposed URL of Netbox in the config file.

I also run NetBox on HTTPS but don't have this problem. Can you please share you configurations and the version of Swagger you have installed?

Here is the version of Swagger :
django-rest-swagger (2.1.2)
My server is named "netbox" and served behind https://netbox, but :

Here is my Apache configuration for this vHost :
<VirtualHost *:80>
  ServerName netbox
  Redirect permanent / https://netbox/
</VirtualHost>

<VirtualHost *:443>
  ServerName netbox
  ErrorLog logs/netbox_ssl_error_log
  TransferLog logs/netbox_ssl_access_log
  LogLevel warn
  Include /etc/httpd/includes.d/ssl_cert.conf

  Alias /static /opt/netbox/netbox/static

  <Directory /opt/netbox/netbox/static>
    Options Indexes FollowSymLinks MultiViews
    AllowOverride None
    Require all granted
  </Directory>

  <Location /static>
    ProxyPass !
  </Location>

  ProxyPass / http://<IP_ADDRESS>:8001/
  ProxyPassReverse / http://<IP_ADDRESS>:8001/
</VirtualHost>

I'm not able to replicate this, but I'm using nginx as a frontend. Can you inspect the AJAX request in the browser to determine if it's being redirected?

I managed to fix it. I just added RequestHeader set X-Forwarded-Proto "https" to the Apache configuration and it started working.


Dependency cleanup

See commit messages for details.

Thanks for the review!

One concern is about rustls-tls switch, but looks like the tests are fine, so, fine?

Not sure how good our test coverage of TLS is. But I think we can assume that the crates we depend on work correctly with rustls, so I'm not really worried about it. And we can easily switch back later if we want to.


Http Client Issue

Laravel Version
11.9
PHP Version
8.3
Database Driver & Version
No response
Description
I am trying to use HTTP client in a do while loop for pagination. Somehow after calling the first url even after using the updating the url is it calling the same url in the first instance of the loop.
Steps To Reproduce
This is where i setting up some basic things.
public function __construct(protected SourceControl $sourceControl)
    {
        $this->http = Http::withToken($sourceControl->access_token)
            ->retry(2, 0, function (Exception $exception, PendingRequest $request) {
                if (! $exception instanceof RequestException || $exception->response->status() !== 401) {
                    return false;
                }

                $request->withToken($this->getNewToken());

                return true;
            })
            ->acceptJson()
            ->asJson()
            ->timeout(15)
            ->throw();
    }

Here i call the paginated urls.

public function getPaginatedData(string $url): true
    {
        $currentUrl = $url;

        while (!!$currentUrl) {
            $response = $this->http->get($currentUrl, [
                'role'   => 'member',
                'pagelen' => 20
            ]);
            $currentUrl = $response->json('next');
            dump($response->json());
        }

        return true;
    }

even though we get he current url and the new url is passed. The first loop url is getting used.

But,

If I directly use like Http::get then i am not getting this issue. This is really confusing.

Can you provide a sample from dump($response->json())?
Can you post a public repo that makes it easier to reproduce?

I was trying it in my private project and found this issue. If I could create a public repo with the same code I will do.
the dump response would be some like

If next url is available then
$data = ['values' => [], 'next' => 'example.test?page=2']
Now as per the code in the next iteration of the loop it has to call the url of
'example.test?page=2'
But it calls the url
'example.test'
and so the next url is still present and it will be the same url and thus ends in an infinite loop.

If I could create a public repo with the same code I will do.

You can create a second endpoint on the sample public repo that mimics the external endpoint.

Here, I made it for you:
https://github.com/rodrigopedra/bug-report
Instructions are on README.md. The relevant code is in ./routes/web.php.
The issue is that you are using both a query parameter in the URL (?page=2) and sending query parameters through the request's options (['role' => 'member','pagelen' => 20]).
When the options are populated, Laravel does not parse the URL for parameters.
https://github.com/laravel/framework/blob/57d6188a6b768654d702fadce76d59af8bb01a01/src/Illuminate/Http/Client/PendingRequest.php#L1144-L1169
It seems to me that Laravel should handle both cases (query parameters in the URL and in the request options).
I can't work on a PR right now, as I am leaving for lunch and will be the whole day off. If you can, be my guest.
As a workaround, you can parse the next URL query parameters and add it to the request options.


Time picker does not open in a modal on mobile

https://github.com/malzariey/filament-daterangepicker-filter/assets/22561290/16329c86-424a-46a6-b35b-acc41afc276f
On mobile, in a modal, tapping the time picker results in some odd behaviour. The time picker does not open, instead the entire date picker moves up. This does not happen outside a modal. See the attached screen recording.

Thank you for your issue,
I tried to repreducre the issue and it was not visible
Screencast from 2024-07-02 01-12-40.webm
Please share a minimal issue repository to inspect further.
Closing the issue as of now


Update README.md

Incorrect config file location

@wagoodman No, not the latest release.
After some looking around, it seems this was updated very recently:
https://github.com/wagoodman/dive/pull/152/files#diff-04c6e90faac2675aa89e2176d2eec7d8
However, the latest release, which is v0.6.0 for deb, rpm, and docker hub, doesn't not read ~/.config/dive/config.yaml.  So yeah, big waste of time going by the documentation not related to he latest release.
Most people will be on the release, not using the master branch.

Good catch


fix: delete multiple resources, close #1986

Fix #1986

I looked at the backend code, maybe we can just exec delete sql statement instead of use a transaction.
I'm not sure if I'm right, because I rarely code with sql and golang.
https://github.com/usememos/memos/blob/8bcc2bd715778ef093bb640ab162b9eec4fe1809/store/resource.go#L169-L189

@0x-jerry For read-only operations, we can query directly through the database without a transaction. But for write operations, it is best to execute them in a transaction.


IDF v5.0 regression: mqtt not providing user_context field (IDFGH-9261)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hi,
I'm working on porting the NodeMCU mqtt module from IDFv4 to IDFv5, but I've run into what seems to be a big issue.
Previously*, it was possible to set a "user context" pointer when initialising the client, and that pointer would then be handed back in each event. This followed the typical pattern for event driven libraries in C, where the caller is free to associate whatever context with a session and have it accessible at the time of handling the event.
In IDFv5**, I cannot find any support for this. Is it really intentional that such a fundamental aspect of the library has been changed/removed? I'm looking at a significant rewrite of the module code here if that's the case. By the looks of it I would have to establish and maintain an external esp_mqtt_client_handle_t to user context lookup structure in order to achieve what the old user_context field did. Obviously, I'd prefer to not have to do that.
Have I overlooked something here? I can't find any mention of this change in the API documentation, release notes or migration notes.
I originally posted this on the forum but got no response there. This seems like quite a regression to me.
Thanks in advance.
*) See https://docs.espressif.com/projects/esp ... parameters
**) See https://docs.espressif.com/projects/esp ... figuration

Hi @jmattsson
Yes, the field user_context was removed intentionally, since mqtt client uses IDF standard esp-event, which allows adding its own user context. It's briefly described in the examples:
https://github.com/espressif/esp-idf/blob/236fa5e6698a262aed534d2a46c78711204b903a/examples/protocols/mqtt/ssl/main/app_main.c#L135-L136
(note that user_context was used before migration to esp-event when notifications were posted as callbacks, and was kept for compatibility up to v5.x)

Oh! Okay, that explains a lot. Thanks @david-cermak !
I hadn't seen any deprecation messages about the callbacks, so I was oblivious to the fact it'd been changed. I went back and checked all the release notes, and there is no mention of that anywhere that I can find. May I suggest adding a note to at least the 4->5 migration doc about this change before someone else gets similarly confused? :)
On my end I only saw errors about missing user_context and the changed esp_mqtt_client_config_t structure, which made me stare at the wrong things and miss the bigger picture.
Thanks again for the quick response. Feel free to close the issue (unless you want it to hang around for doc update reasons).

migration-guides should mention this change.


[Backport] [2.x] Bump org.bouncycastle:bc-fips from <IP_ADDRESS> to <IP_ADDRESS> in /distribution/tools/plugin-cli (#13446)

Backport of #13446 to 2.x

❌ Gradle check result for 485ec8e: FAILURE

https://github.com/opensearch-project/OpenSearch/issues/11374

@dblock if you have a sec, mind approving this backport for 2.x? thank you


Phantom go

Add the game of Phantom Go based on Go
Tagging @michalsustr for review

@googlebot I fixed it.

Hmm, if this doesn't work, maybe you can try getting it directly from https://cla.developers.google.com/

The CLA still didn't work. Can you copy the files and start a new PR?

Ok, instead of another commit, I will make a new PR, I hope the CLA will work this time.

@Syor maybe now is a good time to close this, copy the files, and open a new PR? It would be good to run the tests. And we can iteratre more on the new PR.


Insufficient documentation on this API

Type of issue
Missing information
Description
[Enter feedback here]
There is no description on what this does, how it works, or under what circumstances on how you would use it. We're looking at how we can dynamically update the connection string when using Power BI embedded for a customer-centric connection, and we think this might be the answer, but... there's literally no information on this.
Page URL
https://learn.microsoft.com/en-us/dotnet/api/microsoft.powerbi.api.datasetsoperations.updatedatasourcesingroupwithhttpmessagesasync?view=azure-dotnet#microsoft-powerbi-api-datasetsoperations-updatedatasourcesingroupwithhttpmessagesasync(system-guid-system-string-microsoft-powerbi-api-models-updatedatasourcesrequest-system-collections-generic-dictionary((system-string-system-collections-generic-list((system-string))))-system-threading-cancellationtoken)
Content source URL
https://github.com/Azure/azure-docs-sdk-dotnet/blob/master/xml/Microsoft.PowerBI.Api/DatasetsOperations.xml
Document Version Independent Id
1582c48b-ef59-41d7-4e36-ebc490edb491
Article author
@azure-sdk
Metadata

ID: 11d33961-5ff3-9456-9d78-ff14e1757b88
Service: powerbi

Hi @jeremy-holovacs-sp.  Thanks for reaching out and we regret that you're experiencing difficulties.   Unfortunately, the repository where the Microsoft.PowerBI.Api library is maintained does not accept issues, so I'm unable to transfer this to the PowerBI team on your behalf.  To ensure that the right team has visibility and can help, your best path forward for would be to open an Azure support request or inquire on the Microsoft Q&A site.
I'm going to close this out; if I've misunderstood what you're describing, please let us know in a comment and we'd be happy to assist as we're able.


DropdownButton has no splash effect

Tapping on a DropdownButton has no splash/ink effect or any touch feedback.
Potential simple fix would be to replace GestureDetector with InkWell here:
https://github.com/flutter/flutter/blob/db096cdfd84851ccbd39f57f4af44d36ea28729e/packages/flutter/lib/src/material/dropdown.dart#L919-L923
Would be nice to add custom padding as well.

i use like that, look smooth :)
                    OutlineButton(
                            shape: BeveledRectangleBorder(borderRadius: BorderRadius.circular(2)),
                            onPressed: () {
                              print('aaaa');
                            },
                            padding: EdgeInsets.only(top: 4, bottom: 4, left: 12, right: 12),
                            highlightColor: Colors.blue.shade50,
                            splashColor: Colors.blue.shade200,
                            borderSide: BorderSide(color: Colors.black26),
                            child: DropdownButtonHideUnderline(
                              child: DropdownButton(
                                  isExpanded: true,
                                  elevation: 2,
                                  value: selectedItem != 99 ? selectedItem : null,
                                  hint: Text('hint),
                                  items: _items.map((f) {
                                    return DropdownMenuItem(
                                      child: Text(f),
                                      value: _items.indexOf(f),
                                    );
                                  }).toList(),
                                  onChanged: (val) {
                                    setState(() {
                                      selectedItem = val;
                                    });
                                  }),
                            ),
                          ),

Huh, so I guess you can wrap it with something that does have splash effect. I thought I tried that before posting the issue. Oh well, I still think DropdownButton should provide its own splash effect because adding your own empty onPressed (or onTap with InkWell) in the parent isn't a very good solution.


fix #1361 inclusive language comment

Fixed: #1361
See issue for context.

Nah, thee tests break all the time. These server-adapter tests are very unstable unfortunately. I kinda regret adding this feature instead of just documenting how to use the different servers.

Thanks all!!

@kathytafel thank you for bringing a more inclusive view to the bottle


Animations not triggered when onRequestClose prop absent

Looks like these lines cause the timeout not to trigger when onRequestClose is missing: https://github.com/rackt/react-modal/blob/master/lib/components/ModalPortal.js#L113-L114
So, I am needing to put in an empty function:
        <Modal
          closeTimeoutMS={150}
          isOpen={this.props.isModalOpen}
          onRequestClose={() => {}} >

          <LoginForm />

        </Modal>

Should we make a change? I could do a PR, but unsure on the best direction.

+1

Any thoughts on this? I would happily create a PR if that sounds good

@quicksnap Yes, sir. It seems right to make it optional. +1
Maybe (?) ModalPortal.js#L87-L88

I'd happily whip up a PR if a maintainer would give the OK.

+1 to that @quicksnap go right ahead and I'll get it in the next release.

Oh cool! I'll try and get a PR today or in the next few. If I forget for some reason, someone just ping me. Or make the PR themselves =P

Also, it's odd that you don't have some sort of a collaborator badge, @claydiffrient! shrug

Indeed @quicksnap... I don't know why I don't have one... but oh well... I am one :)


Add validation of Github workflows to pre-commit

This pull request adds a new hook to the pre-commit config to validate GitHub workflows. It downloads the schema directly from the GitHub vendor (which is updated periodically) and checks if the schema is compliant. Closes #383.
Checklist

[X] This PR is meant for the lincc-frameworks/python-project-template repo and not a downstream one instead.
[X] This change is linked to an open issue
[X] This change includes integration testing, or is small enough to be covered by existing tests

This looks like a great addition. Just to be sure, did you confirm that all of our current workflows pass the pre-commit check?

Yes, I created a repository with the latest PPT release just to make sure, and the workflows were validated!


Manage new consumer type (kafka 0.10.x)

Hi,
Kafka >0.10.x introduce a new consumer type. No problem with Kafdrop, except that you no longer see the owner. We can see the new consumer type with this command:
./kafka-consumer-groups.sh --new-consumer --describe --group MYGROUP --bootstrap-server localhost:9092

Is this being addressed by the 2.0 release?

Kafdrop is still using Kafka 0.8, so it still has no way to discover the new non-ZK based consumer types. We hope to upgrade to Kafka 0.10 later this year, at which point Kafdrop will be compatible.

Guys, thats a very nice application you have. I'm thinking about migrating it to gradle(latest version) and putting kafka 0.10 to work. I have some ideas on some features to add to :).

Hi @michael-pratt, is there any ETA for supporting newer kafa versions?
Suri

Hi @thammuio, we've had some other higher priority tasks the past few months, but we're still hoping to do our Kafka upgrade this year.

Thank you, @michael-pratt. I will look forward for this.
Suri

hi @michael-pratt just wanted to check if you are able to upgrade the Kafdrop to support 0.10.x version of Kafka? and do you have any timelines to get the code pushed here?
Thanks in advance...

There is no real development taking place on Kafdrop 2.x. Consider updating to the next major version – Kafdrop 3.

@ekoutanov  Thanks, Let me check. Does Kafdrop3 works with SASL_SSL (both kerberos and ssl enabled kafka clusters)?


Questions for: High-throughput molecular imaging via deep learning enabled Raman spectroscopy

Dear Dr. Horgan:
Good morning!
I have read your paper High-throughput molecular imaging via deep learning enabled Raman spectroscopy and your code GitHub - conor-horgan/DeepeR: DeepeR: deep learning enabled Raman spectroscopy.
They are of great help for my current research, which is about Raman spectroscopy and bacteria metabolism .
I wonder if you can help me in several questions:

In your training dataset 159618X500, how can I transfer the 500-length data array back to actual Raman shift? (What is the formula to project X-axis from 0 to 1800 Ramen Shift(cm-1) to 0-500 data array in the Training_Input?

In our research, a D2O/H2O peak between 1700-2700 Raman shift is important to predict the metabolism of the bacteria. However, in your dataset there is only 500 datapoints (I assume your data’s Raman shift is about 500-1800).
Do you have a untruncated dataset that have longer data length (for example data with 500-4000 Raman shift)？
We would be very grateful if you can share with us such longer dataset.

We have about 10K Raman data for E-coli (500-4000 Raman shift). I am currently considering randomly concatenate our 10K data with longer Raman shift with your 160K data (500-1800 Raman shift), do you think this is a viable solution for data augmentation? Radom sampling the 10K longer Raman shift dataset and concatenate with your 160K shorter Raman shift dataset would produce another 160K+ longer dataset.

What about overfitting? I found my current encoder-decoder DL model tend to memorize the average spectrum from the training set. If we encounter a totally new bacteria, the denoising will produce false image (which is understandable). Encoder-decoder models produce the clear output half by encoder-side information, half by decoder side memorization.

In our use cases, we can often find some new bacterium that is out of the training set on the slices.
However, my current model will still produce the spectrum of E-coli (bacteria in the training set) rather than the spectrum of Lactobacillus (new bacteria) in the real test samples.
Do you have any suggestions to prevent the DL model from producing output just by memory, when a incoming sample is clearly a new specimen？
It might be better to give up some accuracy to prevent the model from producing every spectrum according to the training set specimen average.
Maybe putting more weights on the encoder side information and trying to reduce the memorization capacity on the decoder side?
I currently don’t have good solution for this problem, hoping to hear your insight.
Regards!
WEI LI

3. nd see if there is separation between spectra with/without deuterium.

Thank you very much!
I am very grateful for your help.
So far, I haven't find a good solution for the overfitting problem albeit I have done all the things you have suggested. The nature of my task requires my model to be able to reconstruct/denoising cell signiture that is out of all training/testing/validation set (i.e. a completely new cell-type). My current solution is to give up full-length transformer model, limit the scope(context window size) of the CNN filter or use local self-attention, so that the model can only learn short distance feature, proventing it from memorzing the entire cell spectrum. Full length transformer has the best result so far but tend to learn long distance feature, which I think is quite corelated with known cell-types.
I will share with you our research progress in the future once we submit our draft.


Plugin incompatible with IntelliJ IDEA 2020.2

Short description
Seeing this error with the plugin after upgrading to IntelliJ IDEA 2020.2
9:35 AM	Plugin Error: Plugin "Lombok" is incompatible (until build 201.SNAPSHOT < IU-202.6397.94).

Expected behavior
Plugin should work with the latest IntelliJ IDEA (2020.2) which is no longer beta or release candidate.
Version information

IDEA Version: IntelliJ IDEA 2020.2 (Ultimate Edition) Build #IU-202.6397.94, built on July 27, 2020
JDK Version: OpenJDK 64-Bit Server VM by JetBrains s.r.o. / jdk1.8.0_144
OS Type & Version: Windows 10 10.0
Lombok Plugin Version: 0.30-2020.1
Lombok Dependency Version: 1.16.18

Steps to reproduce

Install IntelliJ IDEA 2020.1.x
Install latest Lombok plugin (0.30-2020.1)
Upgrade to IntelliJ IDEA 2020.2
Notice plugin is no longer compatible

Sample project
N/A
Additional information
N/A
Stacktrace
N/A

Duplicate of 855 (sorry did not see the existing issue)

Also for now, as a workaround, I updated the "until-version" from "201." to "202." in the META-INF/plugin.xml inside the lombok plug-in jar file and restarted IntelliJ IDEA. It seems to be compatible now, though I am not sure if will work 100%, as there were breaking changes introduced in IntelliJ 2020.2, but we'll see. So far so good.

Also for now, as a workaround, I updated the "until-version" from "201." to "202." in the META-INF/plugin.xml inside the lombok plug-in jar file and restarted IntelliJ IDEA. It seems to be compatible now, though I am not sure if will work 100%, as there were breaking changes introduced in IntelliJ 2020.2, but we'll see. So far so good.

thx，it works .


Remove marked data after validation

Since removing data is not a validation action, I think it is reasonable
to remove data after validation, and reflect validation error if desired.

#55

Had provided $ncformGetValue API, refer to: https://github.com/ncform/ncform/blob/master/CONFIG.md#ncform-api


Sign out

When I try to sign out of GitHawk (github profile in settings), the app crashes and doesnt sign out of the account.

Bug Report Dump (Auto-generated)

Version 1.27.2<PHONE_NUMBER>)
Device: iPad Pro (iOS 12.4.1)
TestFlight: false

This has been fixed in #2762 and will be available in the next update.


Ensure responses to encapsulated requests use the same encapsulation

Fixed in https://github.com/zwave-js/node-zwave-js/pull/4832/


Handle dependencies of service providers

I have some libraries which contain service providers:

libfoo with FooService and FooServiceProvider
libbar with BarService and BarServiceProvider

BarService depends on FooService. Therefore, BarServiceProvider uses the foo service from pimple:
function register(Container $c) {
    $c["bar"] = function ($c) {
        return new BarService($c["foo"]);
    };
}

Now, an application wants to use libbar and has no interest in libfoo. In order to use bar from the container, the app needs to register not only BarServiceProvider but also FooServiceProvider. This is inconvenient for applications, because they need to know the dependencies and they might change. It's also inconvenient for authors, because they first need to learn the dependencies, digging trough service providers.
In order to ease usage of services, there needs to be a way to let providers specify the providers it needs to have registered. I could imagine introducing a new interface ServiceWithDependencyInterface that providers can implement to return the providers it needs:
class Bar implements ServiceProviderInterface, ServiceWithDependencyInterface {
    public function register(Container $c) {...}
    public function getDependencies() {
        return [FooServiceProvider::class];
    }
}

Before a service providers register method is called, Pimple would fetch the dependencies and let them register recursively.
There's the issue of passing data to the service providers via the second argument to Container::register. Of course it's nice for dependency injection, but incompatible with getDependencies. So go full circle back to manually specifying all providers? I'd rather let getDependencies specify the dependencies with more detail:
class FooServiceProvider implements ServiceProviderInterface, ServiceWithDependencyInterface {
    public function register(Container $c) {...}
    public function getDependencies() {
        return [
            new DependencyOnServiceProvider(ConfigurationProvider::class),
            new DependencyOnParameter("foo.name")
        ];
    }
}
class BarServiceProvider implements ServiceProviderInterface, ServiceWithDependencyInterface {
    function getDependencies() {
        return [
            new DependencyOnServiceProvider(FooServiceProvider::class)
        ];
    }
}
interface DependencyInterface {

    /** @return boolean */
    public function dependencyFulfilled();

    public function resolveDependency();
}
class DependencyOnServiceProvider implements DependencyInterface{...}
class DependencyOnParameter implements DependencyInterface{...}

This allows different strategies to be applied for dependency resolution. Dependency classes might also accept a docstring parameter, specifying why the dependency is needed. If resolution fails, the docstring can be used in the exception message. With this approach, ease of usage is achieved and dependency injection preserved.

I started working on this in a fork, will see how it goes.


Map does not initially render in correct size

Hello!
I am creating a Dash app that includes a map made using the following code:
def map(df):
    fig = px.scatter_mapbox(df, lat="Latitude", lon="Longitude", size="Log(TotalRevenue)",
                            color = "Log(TotalRevenue)",  color_continuous_scale=px.colors.sequential.YlOrRd,
                            zoom=2, size_max=10, hover_name='Location', animation_frame='Year',
                            category_orders={'Year':[2010, 2011, 2012, 2013, 2014, 2015, 2016]})

    fig.update_layout(margin=dict(l=15, r=15, t=10, b=10),
                      autosize=True)

    return fig

The map is embeded in the Dash app like so:
dcc.Graph(
            figure=map(df),
            style={'width':'55vw', 'height':'inherit'}
        )

The strange thing is that on start-up of the app, the map almost always shows up in a smaller size.

When I click on the animation button, the map then pops into the right size.

Is there a way to correct this?
Thank you very much!

Hi @nchelaru, I have observed the same with other plotly_express chart types use in Dash. I don't have an explanation for it but what worked best for me was to have the Output of the callback be a html.Div's children element and return the whole dcc.Graph object rather than just the figure.
Hope this can help.

@DrGFreeman Thank you so much for your reply, and apologies for my late reply!
I think that my callback does return the entire dcc.Graph object, though not of a html.Div.
@app.callback(
    Output('map-container', "children"),
    [Input('datatable-interactivity', "derived_virtual_data"),
     Input('datatable-interactivity', "derived_virtual_selected_rows")])

def update_graphs(rows, derived_virtual_selected_rows):
    if derived_virtual_selected_rows is None:
        derived_virtual_selected_rows = []

    dff = merged_df if rows is None else pd.DataFrame(rows)

    grouped = dff.groupby(['ZipCode', 'OrderDate', 'Latitude', 'Longitude', 'Location', 'City', 'State', 'Year'])['TotalAmount'].sum().reset_index()

    grouped = grouped[grouped['TotalAmount'] > 0]

    grouped['Log(TotalRevenue)'] = np.log(grouped['TotalAmount'])

    return [
        dcc.Graph(
            figure=map(grouped),
            style={'width':'55vw', 'height':'inherit'}
        )
    ]

dbc.Col(dbc.Card([
                    dbc.CardHeader("Geographic distribution of sales territories",
                                   style={'backgroundColor':'#F3AE4E', 'color':'white', 'fontWeight':'bold'}),
                    dbc.CardBody(
                        id='map-container'
                    ),
                ], className='h-100', color="light", outline=True), width=7)

This app single-page, but with multiple tabs. Curiously, if the tab that contains this map is the first one to be loaded, the map renders in the correct size.
This is not a serious problem, but a strange one.


[Mobile View] Images in Full Screen are obscured by the Table of Contents Header

Found in Dev Environment (RAMP Storyline Release)
Affected sites:

NPRI Sector Overview: Oil Sands Extraction - Canada.ca (https://ecdc-webmappingappsstoryramp-asv.azurewebsites.net/index-ca-en.html#/en/410b88da-0ed1-4749-903f-5e76c24e2e5f)
NPRI Sector Overview: Pulp and Paper - Canada.ca (https://ecdc-webmappingappsstoryramp-asv.azurewebsites.net/index-ca-en.html#/en/f6f7baf4-cccb-4521-a037-b4691b0f0d49)
National Pollutant Release Inventory Substance Overview: Ethylene Glycol - Canada.ca (https://ecdc-webmappingappsstoryramp-asv.azurewebsites.net/index-ca-en.html#/en/ea24000c-7dc3-49a9-baac-c55d28dcaeb9/)

Platforms:
Issue has been observed on Chrome Mobile 91
Summary:
In the Mobile view of any of the RAMP Storylines, when an image is put into Full Screen mode, they are partially obscured by the Table of Contents Header.
Steps to Reproduce:

Using a Mobile Device(for reference, the device used to test was an iPhone 12 Pro Max), open National Pollutant Release Inventory Substance Overview: Ethylene Glycol - Canada.ca (ecdc-webmappingappsstoryramp-asv.azurewebsites.net)

Scroll down to any image.

Tap Full Screen on the image.

⚠Notice that the Full Screen image is obscured by the Table of Contents Header.

Current version is 105, and I cannot reproduce this issue.


Create osdctl config file

This PR creates a new config file for osdctl to use at ~/.config/osdctl. It's a key-value format, with the key and value separated by a colon, and each key-value pair on a new line.
This PR has a few other small changes:

Renames the directory the osdctl env code lives from config to envConfig. The package name is the same
Creates a function that returns the current OCM Environment (prod, stage, int) for a given connection
Refactors generating credentials for CCS clusters to get the Jumprole account IDs from the new config file

Is there any specific reason to roll our own config file here instead of using something like viper which can abstract a lot of this for us and would be a bit more standard between codebases? Another benefit of using a package like viper is that it's relatively simple to allow default behavior -> user configs -> env vars in order of precedence so that you can override things during individual command runs, and you get that out of the box as well.

As for Viper, I wasn't aware of it before. After looking at it, I'm not sure its the best choice as of now, simply because using JSON, YAML, TOML, etc, I think is too complex for what we currently are using this config file for, a key-value store. Even simple changes to data in these formats tend to become "find the formatting issue" rather than "make sure the data is correct", so I wanted to stay away from them unless absolutely needed. So far, the use cases of the config file I've heard are confined to AWS Account IDs and profiles, and a PD api key, and a key value pair with a colon and a new line work for these. Its entirely possible there are use cases that I've missed, and if JSON, YAML or whatever make more sense given those use cases, then I'm happy to refactor this.

Re: Viper
I know that osde2e uses it heavily, and I'm using it in the v2 rewrite of ocm-container (which is an all-go rewrite).  I've found it to be extremely simple to add and really like the flexibility (specifically that around the precedence to how flags can be overwritten).
My only real concern here about using "the simple key-value store" is that if we do need more functionality later that a structured markup language would provide we're more prone to try to get it working within the current system rather than making the migration, leading to an even more complex config parsing code than necessary, and then getting users to migrate from one config file to the other would be a pain unless we build some kind of automatic migration process into the codebase itself, which again is just more bespoke code for us to maintain.
I'll leave the decision up to you, though, your point about the simplicity does make sense overall.
If we do keep the custom config file, can we add some tests with a few use cases so that if we do end up adding more complex validation logic we can at least prevent regressions?

I'd be another +1 for viper (backplane and ocm-agent also use it) if only because given osdctl is already using cobra for the CLI and viper is a natural companion to it - allowing for things like extremely straightforward binding between CLI parameters to viper config (which is practically a 1-liner). But I'm equally fine the approach already already taken.

/hold
I'll do a spike of viper today. I'm all for using the right tools for the job, and I agree that changing the config file will be annoying once established, so I'll give it a fair try.

/label tide/merge-method-squash

/retest

I refactored to include Viper and specified YAML as the format, since that seems to be the best compromise between human manageable and easily parseable, and works with the key: value format out of the box.

/lgtm


Idea: Generate a summary of used licenses

It would be great if along with finding the licenses there were a  short summary which could be easier to review.
This would require finding a way to 'detect' what type of license the project uses. Very much similar to GitHub does, maybe this can be a good source for regex or similar!
Appears GitHub use this: https://github.com/licensee/licensee
Extension
As an extension to this feature could generate a diagram for use in documentation. using DOT and Graphviz  maybe etc.

https://i1.wp.com/www.embeddeduse.com/wp-content/uploads/2016/04/qt-licenses-up-to-5.6.png?ssl=1

Thanks for the suggestion, that does sound like a very useful addition to the project!


compile error: x29 cannot be used in asm here

I am trying to compile on Kunpeng 920-4826 machine, but got this error
g++ -c -fPIC test_temp_L1.c -o test_temp_L1.o
test_temp_L1.c: In function 'void SGEMM_NN_L1(float*, float*, float*, long int, long int, long int)':
test_temp_L1.c:2727:1: error: x29 cannot be used in asm here
 }
 ^
Makefile:14: recipe for target 'test_temp_L1.o' failed

It seems like register x29 is the frame pointer and cannot be used directly. Can you check this problem, Thanks.
The infomation of the CPU
Handle 0x002B, DMI type 4, 48 bytes
Processor Information
        Socket Designation: CPU01
        Type: Central Processor
        Family: ARM
        Manufacturer: HiSilicon
        Version: Kunpeng 920-4826
        Voltage: 0.9 V
        External Clock: 100 MHz
        Max Speed: 2600 MHz
        Current Speed: 2600 MHz
        Status: Populated, Enabled
        Upgrade: Unknown
        Core Count: 48
        Core Enabled: 48
        Thread Count: 48
        Characteristics:
                64-bit capable
                Multi-Core
                Execute Protection
                Enhanced Virtualization
                Power/Performance Control

Hello! Thank you for your interest in this work.
Please check your GCC version.

Thanks, g++-8 works for me.
By the way, I notice that the interface of gemm miss some arguments like lda, will you support such arguments later?


Fix lint issue

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #1816
Special notes for reviewers:

Release note:

NONE

@ramineni @lingxiankong please help check this

Thanks for fixing.
Looks good.
/lgtm

approving it as small change and to unblock gate
/approve


User assigns organisation category to single user

As a user
I want to associate him/her with an agency/organisation

This issue was moved to dumparkltd/hr-nmrf-client#60


grpo  has not   prm

grpo has  the step level  reward deal,also known as progress reward model,but not seen in the code,  can  you tell the reason or how to use step level deal ? thanks

Sorry for the late reply.
PRM or ORM are similar. The current code here simply extracts scores at the end of each sentence. You can modify the model interface to utilize scores at all positions (or at step level, such as scores outputed at all  "comma" tokens), just like how we use values in PPO.
This example may also be helpful.
We'd like to help you if you encounter any issues during implementation.

thanks,i will run on this repo

have seen the example from you ,it is about to use ppo to optimize sentiment class. in my knowledge and with other colleague  communication, none has got the benefit so far. can you tell me it is just a example to learn,or you have got benefit to train the classification task with rl like ppo?

Sorry for the late reply.
PRM or ORM are similar. The current code here simply extracts scores at the end of each sentence. You can modify the model interface to utilize scores at all positions (or at step level, such as scores outputed at all "comma" tokens), just like how we use values in PPO.
This example may also be helpful.
We'd like to help you if you encounter any issues during implementation.
about this case
https://github.com/openpsi-project/ReaLHF/blob/main/examples/customized_exp/ppo_sentiment.py

It's just an example to learn. You can customize the interface to do what you want to do, either training the PRM or using the PRM for PPO. We feel sorry that we don't have the bandwidth to provide all reference implementations.

thanks

It's just an example to learn. You can customize the interface to do what you want to do, either training the PRM or using the PRM for PPO. We feel sorry that we don't have the bandwidth to provide all reference implementations.

thanks

can you build a  wechat  group for tech,and let me in ? my: yiyepiaoling0715

Sorry for the late reply. Just requested in wechat.


Find not reaching last elements

I believe lwrb_find is not reaching all the way to the end of the buffer to find the needle.
The function is looping buff full - input len - start_offset, but I believe the condition in the foor loop should be <=.

Thanks for reporting. Do you have min reproducible code?

I confirm bug. It was fixed with release of 3.0.0


fix: publish.yml

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


SSO configuration example?

How can I configure Authelia to login users via SSO OIDC provider like Google or GitHub?

Hello @omerlh , unfortunately OpenID is not supported yet in Authelia. I'd be glad to review any PR on this subject though if you want to give a hand.

I see. Thanks!


JSON highlighting corrupted by green background in Windows Terminal

Checklist

[X] I've searched for similar issues.
[X] I'm using the latest version of HTTPie.

Minimal reproduction code and steps

Request a JSON file using HTTPie in Windows Terminal, e.g. http -j GET https://raw.githubusercontent.com/httpie/httpie/master/tests/fixtures/test.json
Observe corrupt/incorrect green background in JSON syntax highlighting

Windows Terminal version 1.11.3471.0
Windows 10 21H2 (19044.1415)
HTTPie 2.6.0
Python 3.10.1
JSON highlighting used to work correctly for me, but I'm not sure in exactly which version(s). I tested in Command Prompt, too, with the same result, so I don't think this is specific to Windows Terminal.

Debug output
>http -j --debug GET https://raw.githubusercontent.com/httpie/httpie/master/tests/fixtures/test.json
HTTPie 2.6.0
Requests 2.27.1
Pygments 2.11.2
Python 3.10.1 (tags/v3.10.1:2cd268a, Dec  6 2021, 19:10:37) [MSC v.1929 64 bit (AMD64)]
C:\Users\will\scoop\apps\python\current\python.exe
Windows 10

<Environment {'colors': 256,
 'config': {'default_options': []},
 'config_dir': WindowsPath('C:/Users/will/AppData/Roaming/httpie'),
 'devnull': <property object at 0x000002A31B077A10>,
 'is_windows': True,
 'log_error': <function Environment.log_error at 0x000002A31B097010>,
 'program_name': 'http',
 'stderr': <colorama.ansitowin32.StreamWrapper object at 0x000002A31B089D80>,
 'stderr_isatty': True,
 'stdin': <_io.TextIOWrapper name='<stdin>' mode='r' encoding='utf-8'>,
 'stdin_encoding': 'utf-8',
 'stdin_isatty': True,
 'stdout': <colorama.ansitowin32.StreamWrapper object at 0x000002A31B089690>,
 'stdout_encoding': 'utf-8',
 'stdout_isatty': True}>

<PluginManager {'adapters': [],
 'auth': [<class 'httpie.plugins.builtin.BasicAuthPlugin'>,
          <class 'httpie.plugins.builtin.DigestAuthPlugin'>],
 'converters': [],
 'formatters': [<class 'httpie.output.formatters.headers.HeadersFormatter'>,
                <class 'httpie.output.formatters.json.JSONFormatter'>,
                <class 'httpie.output.formatters.xml.XMLFormatter'>,
                <class 'httpie.output.formatters.colors.ColorFormatter'>]}>

>>> requests.request(**{'auth': None,
 'data': '',
 'headers': {'User-Agent': b'HTTPie/2.6.0', 'Accept': b'application/json, */*;q=0.5', 'Content-Type': b'application/json'},
 'method': 'get',
 'params': <generator object MultiValueOrderedDict.items at 0x000002A31B178AC0>,
 'url': 'https://raw.githubusercontent.com/httpie/httpie/master/tests/fixtures/test.json'})

HTTP/1.1 200 OK
Accept-Ranges: bytes
Access-Control-Allow-Origin: *
Cache-Control: max-age=300
Connection: keep-alive
Content-Encoding: gzip
Content-Length: 180
Content-Security-Policy: default-src 'none'; style-src 'unsafe-inline'; sandbox
Content-Type: text/plain; charset=utf-8
Date: Tue, 11 Jan 2022 15:00:58 GMT
ETag: W/"020a89035cfd7a956c3a3db63baedb50bec31c5b8516170321eeb60c2f338f55"
Expires: Tue, 11 Jan 2022 15:05:58 GMT
Source-Age: 79
Strict-Transport-Security: max-age=31536000
Vary: Authorization,Accept-Encoding,Origin
Via: 1.1 varnish
X-Cache: HIT
X-Cache-Hits: 1
X-Content-Type-Options: nosniff
X-Fastly-Request-ID: 3f9a95965264c43c85ce6b1c6b891280811ce375
X-Frame-Options: deny
X-GitHub-Request-Id: B0CA:18E9:14CFB7:1AFE2C:61DD9B58
X-Served-By: cache-iad-kcgs7200037-IAD
X-Timer: S1641913258.082057,VS0,VE1
X-XSS-Protection: 1; mode=block

{
    "name": "Jakub Roztočil",
    "unicode": "χρυσαφὶ 太陽 เลิศ ♜♞♝♛♚♝♞♜ оживлённым तान्यहानि 有朋"
}

Additional information, screenshots, or code examples

@wjrogers does other styles work for you (http --help can show other alternatives)?

@isidentical yes, actually! Some don't seem to do anything (i.e. the output is not colored), but auto works:

As as a quick workaround (until this issue gets resolved) you can actually set it as the default by adding the following to your config file (possibly located at %APPDATA%\httpie\config.json):
{
    "default_options": [
        "--style=auto"
    ]
}

We normally use fruity (a custom theme) as the default on windows (and auto on linux/mac os) because we had reports in the past that it was the best looking, though we probably should re-investigate this and perhaps use auto on all platforms.
https://github.com/httpie/httpie/blob/508788ca5688bbdfba24b1a810d3091c78eccf46/httpie/output/formatters/colors.py#L30-L34

I'm having this same issue and I can't seem to figure out how to get --style to work for me. I also can't find the config.json file. I'm operating on windows.

Hey @FftyShadesofCode! Would you mind trying http --style=auto pie.dev/get and sending the output as a screenshot?

[image: image.png]
On Thu, Jan 13, 2022 at 7:14 AM Batuhan Taskaya @.***>
wrote:

Hey @FftyShadesofCode https://github.com/FftyShadesofCode! Would you
mind trying http --style=auto pie.dev/get and sending the output as a
screenshot?
—
Reply to this email directly, view it on GitHub
https://github.com/httpie/httpie/issues/1266#issuecomment-1012082062,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHOQ6T5EZLMXNIN33K55MPDUV262PANCNFSM5LWSZ4OA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update predicate endpoint to handle !=

Related to #13215
As per the following comment, the UI should allow for users to support the following boolean logic for key-value pairs (a = "b" AND c = "d" AND w != "s"). Currently the newly integrated API throws an error whenever a != predicate is submitted.
https://github.com/influxdata/influxdb/issues/15423#issuecomment-542728048
Blocking Issue #15423

Blocked because it currently isn't supported by storage. Need to figure out next steps


Package Maintainership

What this PR does / why we need it
This PR introduces documentation covering package maintainership.
Which issue(s) this PR fixes

Fixes: #2736

Problems with the branch, created package-maintainer-docs-2736 to replace it. Will create a new PR as well.


Current page grows when new items are added (asynchronously)

When I click the Refresh button to get more items from the feed, new items are stuffed on the first line of the current page making it grow beyond it's page length.
Furthermore, the records are stuffed without respect for current ordering.

Current page should only accept those items within current page range and remove those bumped out of page length.
Item needs placed on list in proper place.


Feedback forum is closed

The link given right before the table of contents points to a closed forum.

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

ID: 8e47bfea-cf8e-d716-9880-6c15c31fcbdf
Version Independent ID: ba95989d-2cb0-1644-7b38-24b2eab3b8c7
Content: Microsoft Store Policies - UWP applications
Content Source: windows-apps-src/publish/store-policies.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

Thanks for reporting this. The link has been updated to point to: https://docs.microsoft.com/en-us/answers/topics/partner-center-general.html


Add numpydoc validation to tests

Could use numpydoc validation within the unit tests to confirm all docstrings being produced and tested are conformant.

Resolved via #21.


A basic implementation of ONNX exportation.

A basic implementation of ONNX exportation(see onnx_convert.py)
A basic usage of the exported ONNX model(see onnx_usage.py)

Hello, thank you for the PR!
(Sorry for the late response)


LUCENE-10560: Speed up OrdinalMap construction a bit.

I benchmarked OrdinalMap construction over high-cardinality fields, and lots of
time gets spent into PriorityQueue#downHeap due to entry comparisons. I added
a small hack that speeds up these comparisons a bit by extracting the first 8
bytes of the terms as a comparable unsigned long, and using this long whenever
possible for comparisons.
On a dataset that consists of 100M documents and 10M unique values that consist
of 16-bytes random bytes, OrdinalMap construction went from 9.4s to 6.0s. On
the same number of docs/values where values consist of the same 8-bytes prefix
and then 8 random bytes to simulate a worst-case scenario for this change,
OrdinalMap construction went from 9.6s to 10.1s. So this looks like it can
yield a significant speedup in some scenarios, while the slowdown is contained
in the worst-case scenario?
Unfortunately, this worst-case scenario is not exactly unlikely, e.g. this is
what you would get with a dataset of IPv4-mapped IPv6 addresses, where all
values share the same 12-bytes prefix.

I kinda feel like in this case we are trying to outsmart the JIT compiler with optimizations it has for Arrays.equals().
I understand the idea that we could be smarter based on the data ... maybe ... but I don't think we "know enough".
I'd rather us know exactly what that threshold is and write appropriate metadata (e.g. min/maxTermLength) if we can figure it out, better than causing regressions for some use-cases. bonus if any additional metadata metadata can be utilized by checkindex.

As a data point, I ran the current version of the benchmark at https://github.com/mikemccand/luceneutil/pull/170 and got the following numbers:
Before:
id: 12556.30082 msec
name: 12797.48203 msec
country_code: 3.79370 msec
time_zone: 9.48330 msec

After:
id: 6939.94252 msec
name: 9675.48024 msec
country_code: 3.05867 msec
time_zone: 8.84499 msec

Superseded by #1052.


Task 287: CaseHOLD

Incorrect answer generation task based on CaseHOLD dataset
Also, I changed the category of CaseHOLD and overruling tasks to legal.

Thanks! I'll fix them now.
Should I add incorrect answer generation in the README file as well?

Thanks! I'll fix them now.
Should I add incorrect answer generation in the README file as well?

Yes add in the readme as well. Thanks!

Thanks, Done.


SqlPackage does not deploy referenced DAC package

Summary
Although IncludeCompositeObjects is set to true, the  SqlPackage deploys only "main" DAC package.
Environment
OS: Windows 10 Pro x64
SQL Server version: 13.0.4001
SqlPackage version: 15.0.5176.1
Steps to reproduce
I have created a solution with two database projects: Dependent.sqlproj and Dependency.sqlproj. The Dependent.sqlproj references to Dependency.sqlproj:
<ItemGroup>
  <SqlCmdVariable Include="Dependency">
    <DefaultValue>Dependency</DefaultValue>
    <Value>$(SqlCmdVar__4)</Value>
  </SqlCmdVariable>
</ItemGroup>
<ItemGroup>
  <ProjectReference Include="..\Dependency\Dependency.sqlproj">
    <Name>Dependency</Name>
    <Project>{d635d16a-0d11-4c0a-b4c3-a646e442d8f4}</Project>
    <Private>True</Private>
    <SuppressMissingDependenciesErrors>False</SuppressMissingDependenciesErrors>
    <DatabaseSqlCmdVariable>Dependency</DatabaseSqlCmdVariable>
  </ProjectReference>
</ItemGroup>

After compitation of Dependent.sqlproj the bin folder looks fine, because it contains following files:

Dependency.dacpac;
Dependency.dll;
Dependency.pdb;
Dependent.dacpac;
Dependent.dll;
Dependent.pdb.

To deploy databases I use following PowerShell script:
sqlpackage /a:Publish ^
  /sf:Dependent.dacpac ^
  /tsn:"(localdb)\MSSQLLocalDB" ^
  /tdn:Dependent ^
  /dsp:Deployment.sql ^
  /df:Deployment.log ^
  /p:IncludeCompositeObjects=true ^
  /v:Dependency=Dependency

Expected behaviour
It should deploy Dependent and Dependency databases to the target server.
Actual behaviour
It deploys only Dependent database. Deployment.sql also contains nothing about Dependency database objects.
Attachments

Deployment.log;
Deployment.sql.

This behavior is by design. If the referenced project is a "same project" reference, then it also gets published along with the main project getting published when IncludeCompositeObjects=true. The publish action only publishes to one database and external databases are effectively read-only.
In this case, the referenced project is to a different database, so the referenced project is expected to be published first before the project that references it.


Refactor: reimplement in Python

Closes #30
Closes #24
Closes #4
Testing this PR

Check out this branch
Rebuild the devcontainer image docker compose build --no-cache dev
Rebuild and Reopen in Devcontainer in VS Code
Run all the tests
Print the version information in a terminal:

hashfields --version

Print the help/usage in a terminal:

hashfields --help

Try the example from the README:

echo "column1,column2,column3,column4
r1c1,r1c2,r1c3,r1c4
r2c1,r2c2,r2c3,r2c4
r3c1,r3c2,r3c3,r3c4
r4c1,r4c2,r4c3,r4c4" | hashfields --alg sha256 --drop column1 column3 --skip column2

Expected output:
column2,column4
r1c2,fb66e41761a74ea0c042e1c226c04fa2ce1a1334d7063d86230d17f33f109b68
r2c2,6051c006caee661a6ccb390b8cf7a43230c5cd7b54861f7306a598b612f924b9
r3c2,7e32c53b7729f5dce7ac54232b7f2d93d6c78ed19fc8d096b0fde948f513e9dc
r4c2,f8d0624d128daf97c61ec28f4396e8f14be2ca2940d18fdf33e939cda9bd1824

@angela-tran

Did you ever find why the old code might've been causing https://github.com/cal-itp/hashfields/issues/24? Or is the thought that we've hopefully fixed it by having a more concise and readable implementation?

I never did figure it out, or really look that much further. It felt easier to just pass the job off to pandas which has already put the effort in to dealing with column-based operations and reading/writing CSV. This implementation is definitely much more concise and readable!


Adding kube pod and container labels to containerd

Currently we have the pod and container labels part of
containerd metadata extensions. However for third party users
like cadvisor that depend on standard kube labels will need
to be aware of the way metadata is stored in containerd to
fetch the labels.
Signed-off-by: abhi<EMAIL_ADDRESS>

/ok-to-test

/ok-to-test

@abhi

Test is failing.
What label do we need in cadvisor? Do we control the label key name or does cadvisor have any presumption on it? If it's the former, I prefer we add the labels we need ourselves, instead of passing through labels from kubelet.

@Random-Liu yes will correct the test cases.
I am not adding any labels from kubelet. I am only putting the labels passed by the kubelet to CRI into the labels section of containerd. These are the pod_name , pod_namespace and pod_uid labels. Currently we put in all these as a part of metadata and store it using extensions in containerd which is totally fine. But in cadvisor I need to know the type of data stored as metadata extensions to unmarshallany. For this I need to import cri-containerd code into cadvisor. For the purpose of well known labels I feel we could keep labels as easily readable labels by third party user than stored as metadata extensions. cadvisor relies on these labels to identify pod. WDYT ?

@abhi
I mean we can use labels created by kubelet, or we could also apply the required labels ourselves.
I feel like we could create our own label io.cri-containerd.sandbox.name or something.

@abhi Wait. Does cadvisor in kubelet get information based on the labels? If so, then we have to pass through.
Is that the case?

@Random-Liu  Yep something like this -
https://github.com/kubernetes/kubernetes/blob/440884be9ae54a500139697890889e89dabe25cf/pkg/kubelet/stats/cadvisor_stats_provider.go#L183

LGTM


mstsc.exe (RDP Client ) does not work in windows after installing pgina

Hello , I've done pgina configuration for my Ldap and everything is okay ...
the only problem is that i can not use mstsc.exe in my windows server anymore and it will crash when i want to use it to remote to other servers ...
this is the crash dump , I hope you could fix it
thanks team

0:014> !analyze -v

                                                                        *

                   Exception Analysis                                   *

                                                                        *

KEY_VALUES_STRING: 1
Key  : Analysis.CPU.Sec
Value: 4

Key  : Analysis.DebugAnalysisProvider.CPP
Value: Create: 8007007e on DESKTOP-SR6L3RV

Key  : Analysis.DebugData
Value: CreateObject

Key  : Analysis.DebugModel
Value: CreateObject

Key  : Analysis.Elapsed.Sec
Value: 194

Key  : Analysis.Memory.CommitPeak.Mb
Value: 150

Key  : Analysis.System
Value: CreateObject

Key  : Timeline.Process.Start.DeltaSec
Value: 2663

ADDITIONAL_XML: 1
NTGLOBALFLAG:  0
APPLICATION_VERIFIER_FLAGS:  0
CONTEXT:  (.ecxr)
rax=000000dc44e0f168 rbx=00000000c0000374 rcx=00007ffcb564b000
rdx=0000000000000000 rsi=0000000000000000 rdi=00007ffcb715ed40
rip=00007ffcb7121cd0 rsp=000000dc44e0f600 rbp=0000000000000000
r8=0000000000000003  r9=00007ffcb715eda8 r10=00007ffcb7083e57
r11=0000000000000000 r12=0000000000000000 r13=0000000000000000
r14=0000000000000008 r15=000000dc4530b070
iopl=0         nv up ei pl nz na po nc
cs=0033  ss=002b  ds=002b  es=002b  fs=0053  gs=002b             efl=00000206
ntdll!RtlReportCriticalFailure+0x8c:
00007ffcb7121cd0 eb00            jmp     ntdll!RtlReportCriticalFailure+0x8e (00007ffcb7121cd2)
Resetting default scope
EXCEPTION_RECORD:  (.exr -1)
ExceptionAddress: 00007ffcb7121cd0 (ntdll!RtlReportCriticalFailure+0x000000000000008c)
ExceptionCode: c0000374
ExceptionFlags: 00000001
NumberParameters: 1
Parameter[0]: 00007ffcb715ed40
PROCESS_NAME:  mstsc.exe
ERROR_CODE: (NTSTATUS) 0xc0000374 - A heap has been corrupted.
EXCEPTION_CODE_STR:  c0000374
EXCEPTION_PARAMETER1:  00007ffcb715ed40
ADDITIONAL_DEBUG_TEXT:  Followup set based on attribute [Heap_Error_Type] from Frame:[0] on thread:[PSEUDO_THREAD] ; Followup set based on attribute [Is_ChosenCrashFollowupThread] from Frame:[0] on thread:[PSEUDO_THREAD]
FAULTING_THREAD:  ffffffff
STACK_TEXT:
0000000000000000<PHONE_NUMBER>000000 pGinaCredentialProvider!unknown_function+0x0
SYMBOL_NAME:  pGinaCredentialProvider!unknown_function
MODULE_NAME: pGinaCredentialProvider
IMAGE_NAME:  pGinaCredentialProvider.DLL
STACK_COMMAND:  !heap ; ** Pseudo Context ** ManagedPseudo ** Value: 1da56d46b50 ** ; kb
FAILURE_BUCKET_ID:  HEAP_CORRUPTION_ACTIONABLE_BlockNotBusy_DOUBLE_FREE_c0000374_pGinaCredentialProvider.DLL!unknown_function
OS_VERSION:  6.3.9600.17031
BUILDLAB_STR:  winblue_gdr
OSPLATFORM_TYPE:  x64
OSNAME:  Windows 8.1
FAILURE_ID_HASH:  {927cac54-99e4-969b-f83f-602b2e43372c}
Followup:     MachineOwner


Grammar

Going through the app and looking for any grammar/spelling errors.

Made all grammar corrections.

Made all grammar corrections.


cuSolver support

AFAIK there isn't any support in coriander for cuSolver, are there any plans to support cuSolver?
I assume this should be done through a plugin like Coriander-dnn, is this a difficult process?

I'm not planning on adding cuSolver myself. However, does sound like an interesting project.
You'd need to either write your own solver in OpenCL (eg Cholesky), or else find an existing OpenCL implementation. The former sounds like more fun to be honest :) but if there is an existing implementation, that existing implementation might be already highly optimized, and portable.

I'll see what I can do. I'm not yet to experienced, so I'll think I'll use an external lib. No need to reinvent the wheel.


Watch for changes in omnisharp.json

This PR introduces live change tracking in omnisharp.json. There are a couple of issues open for that, will attach them to this PR when I locate them 😄
At the moment changes in formatting settings in omnisharp.json are only read by OmniSharp at startup - so if you change anything you need to restart the server to have them take effect which is not an ideal experience.
Note that we do not roll out a file watcher by hand, but rather rely on change tracking capabilities of Microsoft.Extensions.Configuration. On OmniSharp side, technically, the way it's implemented, is that we let each of our IWorkspaceOptionsProvider run as soon as the configuration change is reported.
Here is the change in action:

related issue: (see comment 1) https://github.com/OmniSharp/omnisharp-roslyn/issues/366#issuecomment-268984041

What options does this work for?

it deals with the strongly typed OmnisharpOptions so at the moment that's formatting only

related issue: https://github.com/OmniSharp/omnisharp-vscode/issues/1290#issuecomment-284827335

one more related issue: https://github.com/OmniSharp/omnisharp-vscode/issues/1080


Latest commit to paperclip master breaks gem

https://github.com/thoughtbot/paperclip/commit/09d6bb7865c06de9e4b90328f507977ae6146814

@jeffblake Thanks. This should be now fixed on master


Update Configuration Scope

Allows for a hierarchy of changes

It seems that you want to merge to the Master branch, could you instead select the Module branch ?
https://github.com/blog/2224-change-the-base-branch-of-a-pull-request

Hello! We've recently moved to a brand spanking new module architecture with updated tests. I'm going to close this particular PR (which references the old code base). If you want to submit another one for the new module format, please do! ❤ 👍


[Event Request] Codeunit 10 "Type Helper"- OnBeforeGetField

Hi, in the codeunit 10 "Type Helper", would it be possible to get an event OnBeforeGetField, like this :

    procedure GetField(TableNo: Integer; FieldNo: Integer; var "Field": Record "Field"): Boolean
    var
        IsHandled: Boolean;
    begin
        IsHandled:= false;
        OnBeforeGetField(IsHandled);
        if IsHandled then
            exit;
        exit(Field.Get(TableNo, FieldNo) and (Field.ObsoleteState <> Field.ObsoleteState::Removed));
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeGetField(var IsHandled: Boolean)
    begin
    end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Syntax error in generated DOT file

The graph construction component given here generates a dot file which throws syntax errors when trying to view it using Graphviz.
We ran the code scripts ourselves and are able to reproduce the dot file but run into a syntax error in line 1054 in both files, which prevents us from viewing the resultant graph. Could you please provide instructions on how we can fix this?

Try to add missing " and " for each of the entity. This is caused by a bug of networkx and need to be manually fixed by adding "" for all entities.

Was able to fix most of the errors by adding "" for all the entities, except this particular one.

I am unsure what is the exact problem in this line. But try to replace special characters in this line, it also cause such error sometimes.


Clicking pencil icon on collection view pages should open edit form, but instead pencils disappear

The collection in the video is https://sul-h2-stage.stanford.edu/collections/1080
https://images.zenhubusercontent.com/272543337/69f6867b-3d27-4b84-b46f-82cec318167d/disappearingpencils.mov

This is now working.


west: Update bash autocomplete file

Update the existing bash autocomplete with the latest commands and
command-line switches.
Signed-off-by: Carles Cufi<EMAIL_ADDRESS>

Will post a zsh autocomplete file next.

@carlescufi twister is failing :(

Rebased to fix CI failure

One missing feature I noticed today is that west init -m path/to/local/repository doesn't complete path/to/local/repository. Would it be possible to add that at some point?

I assume you mean west init -l ? This works fine for me locally.


Administrative template admx/adml source?

In this section https://learn.microsoft.com/en-us/windows-server/security/tls/manage-tls#configuring-tls-cipher-suite-order-by-using-group-policy I see the statement
"From the Group Policy Management Console, go to Computer Configuration > Administrative Templates > Network > ..."
I don't think the Network administrative template is installed by default (I'm running GPME on Windows 10, the AD DC is Windows Server 2019); where is it supposed to be found?  I've done a fair amount of searching and the only administrative templates I have found that address schannel config are unofficial.
thanks!
Martin

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

ID: e40eec07-5000-8af5-37f9-c2908bea3c67
Version Independent ID: 4540c962-9294-e70d-de5b-8094e608768c
Content: Manage Transport Layer Security (TLS)
Content Source: WindowsServerDocs/security/tls/manage-tls.md
Product: windows-server
Technology: security
GitHub Login: @Justinha
Microsoft Alias: justinha

Hello @martinbherbener,
We'd like to express our gratitude for your commitment to our documentation. Unfortunately, we’ve recently fallen short of our usual standards in addressing documentation concerns in a timely manner and offer our apologies for any inconvenience we’ve caused.
Because this issue is now outdated, we’re closing this request. If we've addressed the documentation issue, this thread will also be closed. However, if you believe there’s still a documentation concern, you're encouraged to open a new GitHub issue. We welcome any additional suggestions for improving our documentation to provide a more satisfactory experience in the future. Thanks for your understanding and continued support.
#please-close


TypeError: 'NoneType' object is not callable

I didn't run run.h, directly run codec_train.py, because I wanted to know the architecture of the whole program. But I have this problem. Do you know the reason?
Traceback (most recent call last):
File "E:\00\FunCodec-master\funcodec\bin\codec_train.py", line 48, in
main(args=args)
File "E:\00\FunCodec-master\funcodec\bin\codec_train.py", line 23, in main
GANSpeechCodecTask.main(args=args, cmd=cmd)
File "E:\00\FunCodec-master\funcodec\tasks\abs_task.py", line 1130, in main
cls.main_worker(args)
File "E:\00\FunCodec-master\funcodec\tasks\abs_task.py", line 1239, in main_worker
model = cls.build_model(args=args)
File "E:\00\FunCodec-master\funcodec\tasks\gan_speech_codec.py", line 310, in build_model
frontend = frontend_class(**args.frontend_conf)
TypeError: 'NoneType' object is not callable

I think this error is due to the uncompleted configuration. I recommend you first run the run.sh, then the log file will be created automatically. Then, you can find the python command at the first line in  the log file. You can use this python command to learn the whole framework.

I think this error is due to the uncompleted configuration. I recommend you first run the run.sh, then the log file will be created automatically. Then, you can find the python command at the first line in the log file. You can use this python command to learn the whole framework.

The reason why I don't run run.h is because it will download the dataset and it will take a long time.
In fact, there is already a dataset, I can copy it, so your run.h code can add a judgment: if there is a data set, don't download it. But I don't know how to edit this code, can you help me?
https://github.com/alibaba-damo-academy/FunCodec/blob/3435d870126a48fe61a87fac1af3b6530be388ea/egs/LibriTTS/codec/run.sh#L77-L88

First, you should download, unzip and place the dataset as expect of the stage 0. Then, you can skip the download stage and start from stage 1 as follows:
bash run.sh --stage 1 ....

First, you should download, unzip and place the dataset as expect of the stage 0. Then, you can skip the download stage and start from stage 1 as follows: bash run.sh --stage 1 ....

Wow, so easy, so cool, nice job.


Flash cleanup and RJS removal.

Goals:

Limit the number of different methods that are used to display flash messages
Use JSON instead of RJS where possible

Steps for Testing/QA
Click around trying to make the UI show flash messages. Try some with sparkle, one w/o sparkle...

@martinpovolny I'm also seeing
app/controllers/container_node_controller.rb
15:      render_flash_and_stop_sparkle(node.unsupported_reason(:launch_cockpit), :error)

app/controllers/vm_common.rb
95:      render_flash_and_stop_sparkle(vm.unsupported_reason(:launch_cockpit), :error)

good find, thx, I messed up rebase

@himdel : fixed and tested (go to a VM that has no IP and press the Cockpit icon).


SIMD implementation for Poly1305

This is taking me a little longer than I though it would because it's a bit tricky, so I probably won't finish it until later. If people have strong feelings, please leave a comment so I can prioritize accordingly.


bump lucide icons version to 0.323.0

Merge this only after PR #54 has been merged.

available in<EMAIL_ADDRESS>


get drivefile query - don't support cyrillic filenames

When i download files using command
gam user $EMAIL get drivefile query "'me' in owners" format microsoft targetfolder $pathtofoldier
filenames have empty space.

This is working as intended. Google Drive supports a much wider range of filename characters than most operating systems do. In order to successfully download the file GAM has to choose a filename for the file that is valid on the OS GAM is running and this varies widely across Windows, Linux and MacOS. So as a safety precaution, GAM removes any characters from the Google Drive filename that aren't in the English alphabet (26 characters, upper and lower), numbers 0-9 or the exact characters -_.() . (note that space character is included there).
Admittedly, while this is working as intended it's not terribly user friendly and I'm sure it completely mutilates some filenames, especially in the non-English-speaking majority of the world. It was a decision made to keep things simple and reliable if far from perfect.
What we could look at doing is pulling in a 3rd party Python library like pathvalidate which can handle the challenge of converting a filename from Google Drive's allowed characters to something the local OS/filesystem supports much more gracefully and with far less mangling.
Marking this as an enhancement and help wanted if anyone is interested in switching GAM to using pathvalidate or something similar. The relevant code would be found around:
https://github.com/jay0lee/GAM/blob/master/src/gam/__init__.py#L3860

thx a lot for your response.
i'm are not developer, so i can't do it by my self. Hope, someone will help with it

yes, this is quite strange, that only english chars are supported.

finally i managed to check it deeper, and found, that editing gam/var.py  with something like that:
FILENAME_SAFE_CHARS = FILENAME_SAFE_CHARS + 'АБВГДЕЁЖЗИКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯабвгдеёжзиклмнопрстуфхцчшщьыъэюя'
solves issue locally.
now need to understand how to use pathvalidate

looks like it was easier, than i thought. @jay0lee  please check #1408

closing this as PR was committed long ago...


add link to privacy mode

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/10142

@MikePlumleyMSFT
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@tiburd Can you help us here? Thank you

confirmed path for link in GitHub office-docs-powershell repo


view in browser option

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.10586
Extensions:

Extension
Author
Version

csharp
ms-vscode
1.10.0

why there is no option for view in browser for web pages ex. html ?

I believe this is a duplicate of #15928

Duplicate of #15928


Bump xmlbf to 0.4

There was an issue in xmlbf-0.3 when parsing nested elements. xmlbf-0.4 fixes that.

Please be aware that I haven't tried building the project, I just changed this file.


Add discovery of Python versions installed with uv-toolchain

Completes #2842 by fixing Windows tests.
Replaces our current bootstrapped Python version loading in tests with some naive toolchain Python version discovery.
There are no user facing changes here and there should be no change in test experience. I confirmed the test suite only fails on the following cases when the toolchain path is not populated:
uv::pip_compile_scenarios python_patch_override_no_patch
uv::pip_compile_scenarios python_patch_override_patch_compatible
uv::pip_install_scenarios python_greater_than_current_patch
uv::venv create_venv_python_patch

Merged into #2842


AuthorizationService is not disposed

the AuthorizationService created inside AppAuthWebView should be disposed when not used anymore otherwise it creates a leak.
also, the service creates a binding to a CustomTabManager which may be problematic (and is also something you wished not to use in the first place, but use the web view instead). i think the AppAuthConfiguration should be set with a browser matcher that will always return false.

I am really busy at the moment, i can't edit this library soon, i will be grateful if you fix these problems and send me PR.

For the second part I think this could work:
AppAuthConfiguration appAuthConfig = new AppAuthConfiguration.Builder()
.setBrowserMatcher(descriptor -> false)
.build();


[release-1.22] Update Helm chart version to 0.22.0

/retest

/retest

/lgtm
/approve

/check-cla


Add tests for GeoIP functionality

This functionality depends on MaxMind DB and they seem to like changing what their functions return, so tests are needed to catch that.

Since today the test script checks for geoip record: https://github.com/sematext/logagent-js/blob/master/test/test.js#L40

So.... closable?

Closing. We still have https://github.com/sematext/logagent-js/issues/30 open


Convert FNF to HTML?

Describe your problem here.
Is there any way to convert FNF to an HTML file so It can be put in a web version?
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
HELL NAH

Psych no longer supports HTML


Update to support version 2203.2

This PR updates
--Removed step to 'Clone Landingzone repository', as Landingzone repository is already included (from version 2203.1)
--Updated instruction to refer latest branch 2203.2 (yet to merge).
Screen shot is not updated. Updated instruction as a note.

This PR is to support the 2203.2.
https://github.com/Azure/caf-terraform-landingzones/pull/406