Issue 29997

I am opening this up to verify the pipeline tests and will finalize the PR once the pipeline is green.

@ZeeshanTamboli Yes, I will be continuing with this. Is there an urgency to wrap this up ?

No urgency, just wanted to make sure its active.

Hi @ZeeshanTamboli Would it be possible to get this reviewed ? It addresses this issue

Thanks @ZeeshanTamboli. This was insightful, I will make the changes and push 'em up.

@alzaar Since I couldn't make changes to your PR, I added the modifications in a different PR, which is #39318. Thanks for your PR and your interest in the issue.


ActiveField override input(): options lost

What steps will reproduce the problem?

create a custom ActiveField class which extends yii\widgets\ActiveField
create a new method which uses parent::input() to render the field
use Html::addCssClass() in this method to add custom CSS classes (as an example to utilise the $options)

What is the expected result?
In this case, yii\widgets\ActiveField() has the $inputOptions = ['class' => 'form-control']. The extending class' emailInput() method adds the email CSS class.
The custom CSS classes shall be merged with the existing, thus giving the field both CSS classes: form-control and email.
What do you get instead?
The parent class' options (yii\widgets\ActiveField::$inputOptions) get overriden by child class.
This happens in https://github.com/yiisoft/yii2/blob/master/framework/widgets/ActiveField.php#L365.
A quick test with array_merge_recursive() or ArrayHelper::merge() led to the expected result.
Code / Screens
class CustomActiveField extends \yii\widgets\ActiveField
{
    /**
     * @param array $options
     * @return ActiveField
     */
    public function emailInput($options = [])
    {
        Html::addCssClass($options, ['input' => 'email']);
        return parent::input('email', $options);
    }
}

Additional info
As I cannot overview all implications or do not know whether the override IS actually expected behaviour, I did not create a pull request with a fix.
Can do if you like.

Q
A

Yii version
2.0.15

PHP version
7.1

Operating system
Ubuntu 1604

https://github.com/yiisoft/yii2/blob/6590feef0db1e994a42ffdc3536bccbe2f737ee4/framework/helpers/BaseHtml.php#L1953-L1954
use
    public function emailInput($options = [])
    {
        Html::addCssClass($options, 'email');
        return parent::input('email', $options);
    }

Thanks for getting back and pointing out.
But using the string variant of adding the CSS class leads to the exact same result.
Html::addCssClass($options, 'email');
The original options from $this->inputOptions are not merged with $options.
Another project, same case:

Try making a pull request fixing it. We'll see if tests are affected or not.

Ok. Thanks for checking


Fix endless loop for bad sql like "select * from table where id in ()"

From wechat group

Codecov Report

Merging #825 into dev will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev     #825      +/-   ##
==========================================
+ Coverage   63.23%   63.24%   +<.01%
==========================================
  Files         493      493
  Lines        9713     9712       -1
  Branches     1599     1599
==========================================
  Hits         6142     6142
+ Misses       3226     3225       -1
  Partials      345      345

Impacted Files
Coverage Δ

.../core/parsing/parser/clause/WhereClauseParser.java
91.42% <100%> (-0.05%)
:arrow_down:

...ce/OrchestrationShardingDataSourceFactoryBean.java
77.77% <0%> (+11.11%)
:arrow_up:

Continue to review full report at Codecov.

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


First route in base Component is overwritten when extending its routes

OpenUI5 version:  1.36.9-
Browser/version (+device/version):

Chrome 50.0.2661.102 m for Windows
Microsoft Edge 25.10586.0.0
Probably affecting others as well

Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):
https://github.com/fabiopagoti/ui5-extend-navigation/commit/947ed7b9c5bab483218bba357d04cd1891d7450d
User/password (if required and possible - do not post any confidential information here):
As this issue is related with Component extensions, there are two components
base.app -> inside folder "Base"
ext.app -> inside folder "ExtensionDifferentDefaultRoute"
Inside the base Component, there are three views and the navigation between them work as follows
A* <---> B <---> C

= default route (pattern: "" in the route object)

Inside the extension Component, the objective is to have the same views and the same navigation, just changing the default route to make B as the default view
A <---> B* <---> C
Steps to reproduce the problem:
0. Open app inside folder Base and navigate between all views

Open app inside folder ExtensionDifferentDefaultRoute -
Click in the "Go to C" button (it navigates normally to page C)
Click in the "Go to B" button (it navigates normally to page B)
Click in the "Go to A" button (it displays an error in the console - Can not navigate to route with name r1 because the route does not exist)

What is the expected result?
It should navigate to page A
What happens instead?
Navigation is not triggered
Any other information? (attach screenshot if possible)
Inside method onPressButtonA of controller Second, inspect available routes using the following commands
var router = this.getOwnerComponent().getRouter()
router._oRoutes
You will see that route r1 is not available.
Now, open Component.js from the Base folder and remove the comment in line
routes: [
// {}, // uncomment this to make r1 available in the extension component
{
    pattern: "",
    name: "r1",
    target: "t1"
},

There will be a empty and first route in the base component.
After that, repeat the steps and you will see that the error is not displayed in the console anymore, but the navigation to page A is still not triggered.
This proves that the "First route in base Component is overwritten when extending its routes".

Hello Fabio,
thanks for the very detailed description of the problem and also providing a git repository to reproduce the issue. I will have to ask our component expert to take a look as this will need some deeper analysis from our side.
Stay tuned, we will get back to you soon here.
Internally tracked as: #1670331623
Kind Regards,
Michael

Hi Fabio,
this is not really an error furthermore a design decision in the routing configuration. The routes have been modeled as array. The extension configuration of the Component is merged via jQuery.extend (as deep extend): https://github.com/SAP/openui5/blob/master/src/sap.ui.core/src/sap/ui/core/ComponentMetadata.js#L362
By using jQuery.extend an array will not be merged furthermore the entries of the source array (routes from extension app) overrides the entries of the target array (routes for the base application). This is what you explain above. By adding a dummy route in the base application you just ensured that when the routes configuration of the extension is applied it will override the dummy value.
The main idea for the decision here was to allow to override routes or restructure them for extensions. We wanted to avoid more complexity in merging the configuration which would require additional instructions to decide whether to merge the entries or to overwrite them.
Basically what you can to in such cases is to add undefined values in the extension application configuration:
[undefined, undefined, undefined, { ... }] or just [,,, { ... }]
This would make sure to keep the routes configuration from the extended application and adds your route in addition.
HTH and best regards,
Peter

Hello Peter,
Thanks a lot for the answer.
After doing some tests using jQuery.extend I have questions:
Supposing my "base" app is a standard Fiori app - I shouldn't be changing the code in order to include dummy routes. Should I include the routes always with the same name from the original app? I would not assume there will be controller enhancements to trigger navigation.
Would there be any impacts in my extension app if the base app gets updated and more routes are included?
Thanks!

Hi Fabio,
the solution we have currently with the array for the routes is not optimal - but for now there is no better alternative and we have to stay compatible right now.
Regarding your question about extending the Fiori app: I would not include the routes of the extended app and just leave them empty and just include the additional routes.
When the routes of the extended app gets updated the possible issue would be that you overwrite routes from the extended app. But your route would work. This means you need to re-check your extension once a new extended app version is used.
Apologize that there is no better alternative available for now. We need to reconsider this...
Best regards,
Peter

Thanks Peter!
I agree that it's not optimal, but in any case most companies would do a regression test after a system upgrade.
More important than the limitation itself is to have it documented in this issue.
Thank you so much, I believe this issue might be closed.


Check precision of incomplete gamma

Following comment in incomplete gamma implementation does sound very reassuring
    -- For very large `p' normal approximation gives <1e-10 error
    | p >= 2e5           = norm (3 * sqrt p * ((x/p) ** (1/3) + 1/(9*p) - 1))

Implementation of incomplete gamma provides about 10 signiticant digits. Sometimes worse

Mostly fixed in dd10135d14a0da2a8b66da275de569490bb32442
There's still room for improvements but these will be filed as separate issues.


Will there be a backport on 1 16 5

Will there be a backport on 1 16 5

No (physically impossible because Automobility needs Minecraft's new code to work)

does this code work from version 1 18?
it won't work on 1 17 either?


Full screen alpha widgets causing FPS drop

Description
The full screen widgets that are alpha overlays (ie darkness or lightness at TOB/GWD) take a lot of CPU to render, causing FPS to drop even in GPU mode.
To reproduce
Using the GPU plugin at any darkened area like TOB or GWD will result in an abnormal FPS drop.
Expected behavior
Instead of the CPU taking the performance hit of these overlays, remove the widget and re-darken the region via GPU.
Media
These clips show the effects of the fullscreen darkening overlay reducing FPS at the Theatre of Blood.
https://streamable.com/rwxpl
In this first clip, simply entering a darkened area reduces the FPS from 50 to 38.
https://streamable.com/1ovbd
In this second clip, hiding the offending widget brings the FPS back up to 50.

The first column is CPU usage, and the second column is GPU usage. See that, since I have a quad-core processor, Runelite is consuming one core, and the GPU is not maxing out.
Environment (please complete the following information):

OS: Windows 7 x64
RuneLite version: 1.4.27
Custom launcher running latest shaded snapshot; 32-bit Java 1.8.

fixed in 246b0f8a863d3e1973628c2ef58f967a32ec5c87


Update db.py

Hi,
a small modification to avoid IndexError if 'BU_' field is empty in the DBC file.
What do you think ?

I merged a pull request today which should fix this issue. Please install release 2.0.0 from PyPi and close this pull request if it works.
Upgrade cantools with: pip install -U cantools

I think I was too fast with some renamings. My fault, sorry. There's a typo in the code. Happily you travisci reveals this.

The newly created release should support an empty BU_ field. Install it and we'll close this pull request if it works.

I'll check your new code and close the PR if it is Ok


fix: typings of flatmap operator

Codecov Report
Base: 99.57% // Head: 99.57% // No change to project coverage :thumbsup:

Coverage data is based on head (546561d) compared to base (3bccee7).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #152   +/-   ##
=======================================
  Coverage   99.57%   99.57%
=======================================
  Files          43       43
  Lines        1398     1398
  Branches      291      291
=======================================
  Hits         1392     1392
  Misses          2        2
  Partials        4        4

Impacted Files
Coverage Δ

src/types/index.ts
100.00% <ø> (ø)

src/ops/flat-map.ts
100.00% <100.00%> (ø)

src/ops/flat.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

failes with this:
test/ops/flat-map/types/async.test-d.ts:36:0
  ✖  36:0  Parameter type AsyncIterableExt<string | number> is declared too wide for argument type AsyncIterableExt<123 | "bla">.

Maybe your code was bind from the master?
This test gives some problems...
const test3 = pipeAsync(
    iterable3,
    flatMap((value, index) => {
        expectType<number | string | boolean>(value);
        return index ? 123 : 'bla';
    })
);
expectType<AsyncIterableExt<number | string>>(test3);

I'm not sure why it's calculated the return type as 123 | "bla" instead of number | string.
I'll just add a quick fix for it.


Will go-zero support static resource services in future versions?

Will go-zero support static resource services in future versions?

Static resource can just use nginx  or  oss.

Static resource can just use nginx or oss.

Yes, this is the only way to do it at present.
Will it not be supported in the future version of go zero?

https://github.com/zeromicro/zero-examples/tree/main/http/download
Is this what you want?

https://github.com/zeromicro/zero-examples/tree/main/http/download
Is this what you want?

Dear Kevwan, thank you for your reply!!
Sorry，This is not what I want.
What I want is this:
package main
import "net/http"
func main() {
	http.Handle("/static/", http.StripPrefix("/static/", http.FileServer(http.Dir("assets"))))
	http.ListenAndServe(":8080", nil)
}

But I don't know how to realize it with go-zero

https://github.com/zeromicro/zero-examples/tree/main/http/static

https://github.com/zeromicro/zero-examples/tree/main/http/static

Thank you very much for your help！

https://github.com/zeromicro/zero-examples/tree/main/http/static

No recursion！There are more folders in the folder！


Add HOST_INFO.INCLUDE_CPU_INFO setting

On my machine, py-cpuinfo is quite slow. Seems it's a known issue: https://github.com/workhorsy/py-cpuinfo/issues/155
This setting cuts the time it takes me to run the test suite in half.

Oh, I didn't know that this is an issue. This could also explain the long startup times I observe sometimes. Looks good to me!

BTW, it looks like the username or email configured in your git is not associated with your GitHub profile (there are 3 participants listed).

Thanks for merging and for the heads up on the email situation.


What format is SecKey supposed to be in?

Hi,
let publicKey: SecKey = /* ... */
What format is the public key supposed to be supplied in here? I have the PEM and JWKS formats, but passing those in do not seem to work. I am unable to figure out what format I need to convert the public key in to. Here is an example of the public key I have
let jwk = """
{
"kty": "EC",
"kid": "JoO-sJHpheZboXdsUK4NtfulfvpiN1GlTdNnXN3XAnM",
"use": "sig",
"alg": "ES256",
"crv": "P-256",
"x": "GsriV0gunQpl2X9KgrDZ4EDCtIdfOmdzhdlosWrMqKk",
"y": "S99mZMCcJRsn662RaAmk_elvGiUs8IvSA7qBh04kaw0"
}
"""
            let publicKey = """
            -----BEGIN PUBLIC KEY-----
            MFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEGsriV0gunQpl2X9KgrDZ4EDCtIdf
            OmdzhdlosWrMqKlL32ZkwJwlGyfrrZFoCaT96W8aJSzwi9IDuoGHTiRrDQ==
            -----END PUBLIC KEY-----
            """

Thanks

Ok guys, I had it correct but for one key part. It expects a SecKey, but my problem was that I was not using the "as SecKey" notation after calling SecKeyCreateWithData. So here is an example
let attributesECPub: [String:Any] = [
                    kSecAttrKeyType as String: kSecAttrKeyTypeEC,
                    kSecAttrKeyClass as String: kSecAttrKeyClassPublic,
                    kSecAttrKeySizeInBits as String: 256,
                    kSecAttrIsPermanent as String: false
                ]

                var xStr = "GsriV0gunQpl2X9KgrDZ4EDCtIdfOmdzhdlosWrMqKk"
                var yStr = "S99mZMCcJRsn662RaAmk_elvGiUs8IvSA7qBh04kaw0"
                xStr = xStr.replacingOccurrences(of: "-", with: "+").replacingOccurrences(of: "_", with: "/")
                if xStr.count % 4 == 2 {
                    xStr.append("==")
                }
                if xStr.count % 4 == 3 {
                    xStr.append("=")
                }
                yStr = yStr.replacingOccurrences(of: "-", with: "+").replacingOccurrences(of: "_", with: "/")
                if yStr.count % 4 == 2 {
                    yStr.append("==")
                }
                if yStr.count % 4 == 3 {
                    yStr.append("=")
                }

                let x : [UInt8] = (Data.init(base64Encoded: xStr)?.bytes)!
                let y : [UInt8] = (Data.init(base64Encoded: yStr)?.bytes)!
                let i : [UInt8] = [0x04]

                let pubBytes = i + x + y
                var error: Unmanaged<CFError>?
                let pk = SecKeyCreateWithData(Data.init(pubBytes) as CFData, attributesECPub as CFDictionary, &error)! as SecKey


Add flushing check to WriteUpgradeAckChannel

Description

closes: #3929
Commit Message / Changelog Entry
N/A Feature branch

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

pushed some commits with cian's blessing!


[ios] fix attr name camelized error.

the input attribute name is camelized.

This PR/issue doesn't received response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.


feat: Improve checkNotUpdated action

The action now treat URL repo (starts with (https://)?github.com/) as GitHub plugin
Handle HTTP error or GraphQL error and print its messages.

OK.  But it seems not support SSH style.
git@github.com:Shougo/dpp-ext-installer.git

I will improve the routine.


Integrate with a reliable Windows CI (other than AppVeyor)

Is there actually another Windows CI that we can use?

@sigmavirus24 Good question. I haven't done any thorough research yet. Suggestions welcome.
I've removed the AppVeyor integration because it's been broken for a while. I'm unable to make even basic things like pip install work anymore.  And their support is unresponsive.

I don't think there is another one sadly

Solved via Github Actions CI, thanks everyone.


Add vivaldi snapshot <IP_ADDRESS>

The version number should be in the name (example). Please squash your commits.
But why is this snapshot here? What’s the value in this specific version?

Like iterm and iterm-beta, google-chrome and google-chrome-beta, vivaldi snapshot is also an unstable version.

Ah, got it, that is fine, then. Can you squash the commits or do you need me to do it?

Thank you for the contribution. It needed some fixes, so they were made in the below referenced pull request. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Allow User-Agent To Be Configurable Via Custom Function

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is.
The issue is that we would like the ability to overwrite the user-agent. There is a way to add a suffix extension but we want to be able to have it a system level where this config applies to all calls of the boto clients. Right now the work requires monkey patching a solution to do this.
Describe the solution you'd like
What we want is a way to have a config to be able to pass a custom user_agent logic (it will use env variables to construct) and we want this to apply to all boto calls so we can more easily figure out access patterns from different systems.
I can work on this if we can get assurance that this can fit in here and get guidance on what things to watch out for.

Hi @tim-finnigan! Thank you for the additional response!
I agree appending to the user agent would be great. I think the case for us, is that we want to even allow for a suffix extension, but based off a function to get the information since it is dynamically changing depending on who is submitting the request. The issue as a platform maintainer (such as a workflow platform in our case), is that we have many users who can commit their own workflows and call the boto3.client without passing the Config. We want to somehow at a platform level, make sure that information is passed without having to fork the botocore library if that makes sense.
Can you think of ways to support this?

Hi @Acehaidrey thanks for following up. Can you set up different profiles for your users? If you’re trying to figure out access patterns you could try configuring profiles and using a service like IAM Access Analyzer collect access information.


dataset

Thank you very much for your contribution, can you provide us with the bone sequence dataset corresponding to the casia_b dataset

Thank you very much for your contribution, can you provide us with the bone sequence dataset corresponding to the casia_b dataset

您好，同学，请问这个问题您解决了吗？我现在也卡在这里了。

Thank you very much for your contribution, can you provide us with the bone sequence dataset corresponding to the casia_b dataset

请问你训练过Gait3D或GREW吗，其精度怎么样呢


Black box and theano tutorials are apparently not tested at all

Based on the fact that the tests didn't break prior to this PR: https://github.com/tensorflow/cleverhans/pull/158

Oh, I misunderstood the PR. I thought it was saying the warnings crashed rather than successfully printing warnings.


Clarify help text for --all and --status flags in esi lease list and offer list

Updated the help text for openstack esi lease list command to specify that --all shows all projects and --status any shows all statuses.
Ensured similar clarity for openstack esi offer list command.

@tzumainn Updated that change.

https://github.com/CCI-MOC/esi-leap/blob/master/docs/esi-leap-api-ref.md
https://github.com/CCI-MOC/esi-leap/blob/master/docs/esi-leap-requirements.md
I referred to these two files and tested the command, which is what I understand.
Am I missing something?

Yeah - the code I linked shows the default statuses that will actually be returned.

https://github.com/CCI-MOC/esi-leap/blob/master/esi_leap/common/statuses.py
It was bit confusing for me.


Create Playground Tasks

Create 2 tasks from the below, verify and assign them to Mohan and Sobi, add any additional comments to the tasks.
Add Get and Post Methods to Program.cs
Mogan

GetVersionsAsync()
GetVersionAsync(string version)
GetChangesAsync(string version)
GetRelatedAsync(List ids)

this one has rate limit in place

GetSkillsAndTraceFromDocumentAsync(string version) from ../extract/trace endpoint

Sobi

GetSkillsAsync(string version)

don't worry about typeIds, fields, limit in the query for now

GetSkillsAsync(List ids)

don't worry about typeIds, fields, limit in the query for now

GetSkillAsync(string version, string id)
GetSkillsFromDocumentAsync(string version) from ../extract endpoint

Playground tasks created and assigned. This is done.


Import Profiles

I read some where a a while back that because of issues with regards to licenses it perhaps wouldn't work to distribute images with software!
But using cloud-config within profiles has been an awesome tool to help automate the deployment of services (IMO) so instead of deploying pre-made images could it not be added to import profiles from remote ?
Instead of having a remote LXD node to sync to, it could work over http to allow for easier deployment / integration with existing services (github, gitlab or any http server) as an lxd profile appears to be just a flat file?
The profile could be extended to include the base image (ubuntu, alpine, centos etc) the cloud-config (profile including a cloud-config) is intended to be run on.
There would be problems such as varying network configurations and attached devices, but they should be up for the user to decide!

lxc profile copy lets you copy profiles between remotes

Alternatively you can always provide a .yaml file and do:

lxc profile create SOME-NAME
lxc profile edit SOME-NAME < your-profile.yaml


usage for shadow apiserver

What would you like to be added:
need docs  for shadow apiserver featuregate
Why is this needed:
I found a featuregate flag in clusternet-hub  deployment file

but i didn't found any doc to explain  the function of this ShadowAPI，and when should we set it to true/false；
i read the code，this shadow apiserver probably  launch an apiserver to make a copy of raw apis，but i can't get why we need to do this；
can somebody please  explain the feature-gate of  ShadowAPI？ thanks

@snstaberah  In short, if you want to deploy multi cluster resources, you need to enable this feature gate ShadowAPI. With this feature gate, you can create a Deployment in kube-apiserver, while this deployment is only visible to Clusternet. That's why I call it shadow api.
You can view this deployment with clusternet plugin.
kubectl clusternet get deploy

thanks，but why i need to create a deployment is only visible to Clusternet？and how to create such a deployment？i see examples in  examples/applications/，and the deployment  seems just like the normal deployment；  Does it means when i create a  normal deployment in kube-apiserver，the clusternet will auto create another shadow one in clusternet schema and scope？

thanks，but why i need to create a deployment only visible to Clusternet

Clusternet runs as an add-on to manage multi clusters. We do need to way to distinguish those objects from those in kube-apiserver. For example, if you create a deployment with kubectl create deployment xxx, then this deployment will run in current cluster immediately. But this is not what we want. We want to deploy this deployment to multi clusters, not current single cluster.

and how to create such a deployment？i see examples in examples/applications/

kubectl clusternet apply -f examples/applications/

Does it means when i create a normal deployment in kube-apiserver，the clusternet will auto create another shadow one in clusternet schema and scope？

No.

or create shadow objects only with thekubectl clusternet apply command？

Yes. kubectl clusternet create/delete/edit/apply/scale/xxx works the same with those in kubectl.

understood，thanks


Atomic merge buffer acquisition for groupBys

This patch fixes #3819.
After this patch, when a group-by query is submitted, all needed merge buffers are first acquired atomically before query execution.

This change is

it looks like this will solve first case reported in #3819 too, however description seems to be very explicit about case two only. no ?

@jon-wei, @himanshug thanks for your review. I addressed your comments.
@himanshug, yes this PR covers the case 1 as well. I updated the pr description.

@leventov thanks for your review. Addressed some of your comments.
I don't know why I can't add an inline reply for your comment. That part is not what I changed. I just changed that take() method throws a RuntimeException instead of an InterruptedException.

@jihoonson and about poll vs take() - before you explained, that timeout 0 means "no wait", but take() is blocking.

@leventov ah, I got your point. That's also not what I changed, but I clarified it.

@jon-wei I simply added a new takeBatch() method to BlockingPool which is checkAndDrain() you mentioned. After thinking about this for a while, I decided to add it because this issue focuses on guaranteeing the atomicity when acquiring merge buffers, and it cannot be achieved without takeBatch(). But I still believe that we need to improve and maybe redesign BlockingPool.
Sorry for going back and forth. Would you and other reviewers @himanshug @leventov mind reviewing again please?

Hmm, travis has been done for a while but the PR hasn't figured that out yet. Going to bounce it to see if that helps.

:+1:

@himanshug thank you for the review and merge.
@gianm @leventov @jon-wei if you have more comments on the newly added part, please feel free to tell me. I'll do in a follow-up pr.


Export OMI_collider from UnityGLTF

I've got this working for box colliders now. Waiting on support to get mesh colliders working.

This now works for all collider types. I'll open source it as soon as we have a method of separating our assets from the code.


Wip/release1.1 cleanup

Pull Request Test Coverage Report for Build 75

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.206%

Totals

Change from base Build 74:
0.0%

Covered Lines:
2773

Relevant Lines:
5096

💛  - Coveralls


Windows taskbar label "forgets" custom tab names after switching tabs

Environment
Windows build number: 10.0.19041.0
Windows Terminal version: 1.1.2021.0
Additional info: In Windows Settings > Personalization > Taskbar, my "Combine taskbar buttons" setting is set to "When taskbar is full" instead of the default "Always, hide labels" option.
Steps to reproduce

Give a WT tab a custom name by right-clicking and selecting "Rename Tab"
Observe that custom name is respected in the Windows taskbar label.
Open a new tab in WT.
Observe that the new tab's default name is now shown in the Windows taskbar because it is the active tab, as expected.
Close the new tab.
Observe that the original tab's custom name is still in place, but the Windows taskbar has gone back to displaying its default, path-based name.

Expected behavior
If an active tab has a custom name, that custom name should always be displayed in the Windows taskbar.
Actual behavior

Hey @PankajBhojwani, your change fixed this one right?

Yep! Closed by #7119

Sorry for the duplicate! I'll look in closed issues as well as open ones before submitting next time.

Oh, no worries! This one was only recently closed, and the more issues we fix the harder it'll be to find ones in the Closed list :smile:


DRILL-6422: Update guava to 23.0 and shade it

@vrozov, could you please take a look at this PR?

@vvysotskyi  there are merge conflict warnings showing up. Some of these, appear to affect files that haven't been updated since July 2017 (e.g. exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillOperatorTable.java ). Can you check if you need to rebase?

This PR was on the last master before the last batch commits. DrillOperatorTable.java class contains changes, merged in https://github.com/apache/drill/pull/1268 (it was merged in this last batch commit)  and perhaps this PR caused merged conflicts in this class.
I'll resolve all merge conflicts once the active review stage comes.

@vrozov can you review this today?

@priteshm I'll start to review the PR if time permits today, but with the number of changes involved, I doubt that it can be done in a day or two.

@vrozov would you be able to wrap up the review this week? I am assuming most of the changes are related to shading the library.

@vrozov, I made requested changes, could you please take a look again?

I'd recommend splitting this PR in two. The first PR to shade newer version of guava and the second PR to change drill code to use the shaded jar. The first PR needs to be merged first, so shaded jar can be published to maven.

Shaded jar needs to be published to maven no matter what. In case of a single PR, publishing to maven needs to happen before PR is merged. In the case of two PR, there is no such dependency. The first PR can be merged without publishing shaded jars to maven. Otherwise LGTM.

@vrozov, I don't mind of creating a separate PR, but I still think that it is unnecessary, since we need to merge the first PR, publish shaded guava package to the maven manually and then merge the second PR.
Could you please explain how to manually publish shaded guava package to the maven repository?

@vvysotskyi Correct, but note that with a single PR it is necessary to complete almost the same actions by a single committer in a short period of time (due to merge conflicts) while two PRs allow avoiding such dependency and one person can merge the first PR, another person can publish artifacts to maven and a third person can wait for the artifacts to become available before resolving conflicts and merging the second PR.
To manually publish artifacts, see https://repository.apache.org or use maven deploy. I don't remember who (all committers or only PMCs) are allowed to publish to the Apache maven repository.

@vvysotskyi Please make sure to publish the artifact to the Apache repository (not a MapR repository that is not mirrored).


[SPARK-48526][SS] Allow passing custom sink to testStream()

What changes were proposed in this pull request?
Update StreamTest:testStream() to allow passing a custom sink. This allows writing better tests covering streaming sinks, in particular:

reusing a sink across calls to testStream.
passing a custom sink implementation.

Why are the changes needed?
Better testing infrastructure.
Does this PR introduce any user-facing change?
No
How was this patch tested?
N/A
Was this patch authored or co-authored using generative AI tooling?
No

thanks, merging to master!


cxxopts

I'm using this:
https://github.com/jarro2783/cxxopts
works great :)

Thanks for recommendation. For now, I will stay with ArgumentViewer.


E commerce example

Boom 💥 first commits \o/ @mthuret 👍

Let's merge this, we will add some linting to avoid 4 spaces.


[navtree] keyboard navigation

[x] introduce arrow navigation to navtree so that it can be much more quickly traversed from the keyboard without stepping through all menus
[x] up and down arrows should traverse tree items in order
[x] left and right arrows?

[x] expand/collapse on branch items?

navigating to items via keyboard seems to have regressed


Problem to open draw.io diagrams saved file on owncloud

Preflight Checklist

[ x ] I agree to follow the Code of Conduct that this project adheres to.
[ x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
I'm having trouble editing drawio diagram files after April 18th. When I save any diagram, whether an existing diagram or a new one, I can no longer open it for editing, as the following error appears: "Error: This is not a Drawio file!". I'm using owncloud server 10.0.10 with drawio 0.0.9 plugin.
I don't know what else to do to safely edit and save my diagrams.
To Reproduce
Steps to reproduce the behavior:

Create or open an existing diagram inside the owncloud server (Web with google chrome)
Edit the diagram
Save and close the diagram
I try to open the saved diagram and the following error appears: "Error: This is not a Drawio file!". The diagram screen opens in white but with the loading animation and it doesn't come out of it.

Expected behavior
The diagram should open normally for editing.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 0.0.9 (plugin)

Desktop (please complete the following information):

OS: W10
Browser Chrome
Version 112.0.5615.138

Smartphone (please complete the following information):

Device: Xiaomi 11T Pro
OS: Android 12
Browser Chrome
Version 112.0.5615.136

Additional context
Add any other context about the problem here.

We're not the author of the owncloud plugin. If there is a listing page there might be a support page link.

https://marketplace.owncloud.com/apps/drawio


Removes the legacy app support and all the altcoins

Building on top of #31, to be merged after that.

I just noticed that not all CI workflows are being run because the PR is not against master or develop.
It could be nice to have them running

@sgliner-ledger in 4c55c310840db25d66ea4336311ffe930e067bd4 I added the workflows for codeql and sonarcloud to the experimental branch.

Codecov Report

Merging #61 (4c55c31) into experimental (ee99834) will decrease coverage by 41.24%.
The diff coverage is 86.95%.

@@                Coverage Diff                @@
##           experimental      #61       +/-   ##
=================================================
- Coverage         75.17%   33.92%   -41.25%
=================================================
  Files                15       17        +2
  Lines               882     2084     +1202
=================================================
+ Hits                663      707       +44
- Misses              219     1377     +1158

Flag
Coverage Δ

unittests
33.92% <86.95%> (-41.25%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/common/buffer.h
100.00% <ø> (ø)

src/common/wallet.c
9.24% <ø> (-43.36%)
:arrow_down:

src/debug-helpers/debug.h
0.00% <0.00%> (ø)

src/common/buffer.c
89.34% <100.00%> (+0.17%)
:arrow_up:

src/common/script.c
98.63% <100.00%> (+0.26%)
:arrow_up:

src/common/wallet.h
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[DR-9195-custom-heading] adds custom heading styles

adds custom heading styles

The headline- legacy classes should only be styled within the .wysiwyg namespace. Also please combine reused properties into one declaration.


Consider using rule-based part-of-speech tagging

Thank you for writing such a great library!
The purpose of this issue is to explore and perhaps implement the use of rule-based POS tagging.
I found RDRPOSTagger while searching for a faster tagger than NLTK provides. I was consider implementing it, but found spaCy was sufficient for my needs. I wanted to make note of this tagger for spaCy.
According to the RDRPOSTagger project website, the algorithm achieves:

90K words/second
96.49% accuracy as compared to the "state of the art" TnT tagger accuracy of 96.46%
93.42% accuracy for Vietnamese using 28k corpus

To-do:

[ ] Compare accuracy of RDRPOSTagger and spaCy perceptron tagger against gold standard. Note: define what is gold standard.
[ ] Study RDRPOSTagger code for scope of work to Cython. Note: RDRPOSTagger implemented in Python.

I think you'll find that spaCy's tagger achieves more than 90k words per second. TnT tagger is also far from state of the art.
spaCy's tagger achieves between 97 and 97.3% accuracy on the standard evaluation. There are more accurate taggers in the literature, but not by much. Accuracy generally tops out at about 97.45.
Feel free to implement a rule-based tagger if you want one, but I doubt we'll accept the pull request. Having more than one tagging model will cause a lot of problems.

Thank you for the feedback and clarification! I'm definitely not very familiar with taggers nor the literature. 90k per minute is definitely the kind of performance I'm looking for. Cheers!

On Feb 10, 2016, at 11:58 AM, Matthew Honnibal<EMAIL_ADDRESS>wrote:
I think you'll find that spaCy's tagger achieves more than 90k words per second. TnT tagger is also far from state of the art.
spaCy's tagger achieves between 97 and 97.3% accuracy on the standard evaluation. There are more accurate taggers in the literature, but not by much. Accuracy generally tops out at about 97.45.
Feel free to implement a rule-based tagger if you want one, but I doubt we'll accept the pull request. Having more than one tagging model will cause a lot of problems.
—
Reply to this email directly or view it on GitHub.


Transforming quadratic systems into matrix form

Here is how I would imagine this working:
using DynamicPolynomials
@polyvar x y
F1, F2 = get_linear_square_parts([
    (x, x + y^2),
    (y, x - y + x * y + x^2)
])

where, in each tuple, the first entry is a variable and the second is the corresponding right-hand side. For example, the input above would describe the ODE system
x' = x + y^2
y' = x - y + x * y + x^2


Update HTML tags & HTTP verbs

I do agree with these - skipping asking for source ;)

Thanks for the suggestion @cosad3s !


Add IgnoreSpecificDefaultLibraries to GpioTestTool

fixes dependency resolution failure on latest builds

Codeflow

Hi @jordanrh1, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jordan Rhee). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Upgrade to SynapseAI 1.19 version

We made a benchmark to compare the perf with 1.18, using 5 models listed below:

BAAI/bge-large-en-v1.5
sentence-transformers/all-mpnet-base-v2
sentence-transformers/all-MiniLM-L6-v2
sentence-transformers/paraphrase-albert-small-v2
intfloat/e5-mistral-7b-instruct

Best throughput for each model are:

1.18
1.19

BAAI/bge-large-en-v1.5
1521
1370

sentence-transformers/all-mpnet-base-v2
2662
2698

sentence-transformers/all-MiniLM-L6-v2
5175
4539

sentence-transformers/paraphrase-albert-small-v2
3251
3181

intfloat/e5-mistral-7b-instruct
153
156

basically aligned with 1.18, except 2 larger perf drop for BAAI/bge-large-en-v1.5 and sentence-transformers/all-MiniLM-L6-v2, this is due to the Pytorch 2.5 Performance Degradation issue mentioned in Optimum-habana, we added torch._C._set_math_sdp_allow_fp16_bf16_reduction(True) for WA.

@regisss , pls help review


Proposal - add log message in results exemple for better clarity

I propose to add another line in console writeline result return to display count of anomalies found, which is the goal of the exercise

@mrbullwinkle
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@mrbullwinkle,
Can you review this old PR and determine whether it needs to be closed or merged?

I sent an email to the author today.
@MicrosoftDocs/public-repo-pr-review-team

sent email and Teams msg to author

#sign-off


ADFS Support

Hello,
are the above commands available in ADFS 2016 as well? Specifically, I would like to change the Access Token lifetime.
Thank you.

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

ID: 02ff3d62-dc0e-ad0c-f5ae-418337cf31e4
Version Independent ID: d50952b5-6b98-c40d-d3a3-f9cbec58dd28
Content: Configurable token lifetimes in Azure Active Directory
Content Source: articles/active-directory/active-directory-configurable-token-lifetimes.md
Service: active-directory
GitHub Login: @hpsin
Microsoft Alias: hirsin

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

@ilio78 Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@ilio78 We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


CURLOPT_FILE

Hey guys,
I got a strange behavior when using Jyggen Curl in Laravel to download some JPEGs.
Problem was, that the file got downloaded, but was corrupted (the headers i think)
I took the Basic curl usage, made the same options and request - and it worked!
Had no time for further debugging, but here's what I did:
NON working example:
$fp = fopen ($dirTarget.$filename, 'wb+');
$request = new Request($uriSrc);
$request->setOption(CURLOPT_BINARYTRANSFER, 1);
/* Protected Option. Unable to set...
$request->setOption(CURLOPT_RETURNTRANSFER, 1);*/
$request->setOption(CURLOPT_FILE, $fp);
$request->execute();
fclose($fp);

WORKING example:
$fp = fopen ($dirTarget.$filename, 'wb+');
$curl = curl_init();
curl_setopt($curl, CURLOPT_URL, $uriSrc);
curl_setopt($curl, CURLOPT_BINARYTRANSFER, 1);
curl_setopt($curl, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($curl, CURLOPT_FILE, $fp);
$result = curl_exec($curl);
fclose($fp);

Any Ideas?
Regards,
Steffen

Strange, haven't encountered this, but can't say I've uploaded files with the library either. Is this also an issue when using the CurlFile class (see #15 for example usage)?
I'll look into it.


Google Season of Docs/Summer of Code

Please share any additional details on this topic
https://opensource.googleblog.com/2023/01/announcing-google-season-of-docs-2023.html
https://opensource.googleblog.com/2023/01/mentor-organization-applications-are-open-for-google-summer-of-code-2023.html
Detail what actions or feedback you would like from the TAC
Awareness of these programs, see if there is interest in the projects.

Projects aware of both programs, but no bandwidth to participate.


Task/98

In this PR, the README.md has been modified and the file HOW_TO_RUN.md has been created. In this one all the content of our old readme has been copied.
In the new one we have added the corresponding badges as well as a reference to how to run the project and the content taken from the first-contributions repository.
In addition we added missing images and an initial icon to improve the aesthetics of the file
Translated with www.DeepL.com/Translator (free version)

Algunas cosas:

Quitar de la descripción Translated with www.DeepL.com/Translator (free version)
Hace solo un commit, en vez de 9 (ya que el trabajo es solo una cosa), si queres remarcar todo lo que hiciste lo podes comentar en la descripción del PR.


Removing all items from installs array does not fully remove the array

Steps to reproduce:

Create an installs array and save.
Remove all items from the installs array and save.
Open pkginfo and notice that an empty installs array (<array/>) still exists.

Expected behavior:
Removing all items from the installs array would result in the array being fully removed, not simply empty.
Why this is important:
Until this commit, Munki treated an empty installs array as evidence of an item being installed in some cases. This could have undesirable consequences if such an item is included in a manifest as a managed_uninstall.

Thanks for reporting, fix will be included in the next release. I've now changed the behaviour only for installs array. I need to think if this is needed for other array type pkginfo keys.

Fixed in 1.6.0


Replace Ember.keys with Object.keys

This removes a deprecation warning in Ember.js 1.13.

Thanks for the effort on this!


[BUG] Version 2.0.6 broke API chatmessage

Describe the bug
Previously (2.0.5 and lower versions) I was able to retrieve the entire chat history, with api key authorization enabled, for previously chat ids. Basically, it worked just fine. Now, using 2.0.6, the api fails to provide the conversations history.
Additional context
I had to revert back to version 2.0.5 and without any changes made to my workflow, it magically started working again.
I am talking about this api endpoint: https://docs.flowiseai.com/api-reference#/chatmessage-id

works fine for me:

Hmm... then it might be because I am running it on docker with apache as a reverse proxy?

im not sure, you can also use our docs page to test it: https://docs.flowiseai.com/api-reference/chat-message#/chatmessage-id

Closing for now. Feel free to re-open if issue persists


add int32 default for Tensor(int) and Tensor(List[int)

#2709
TODO

need to check if this interferes with Tensor.default usage
add tests

for reference: torch.zeros, torch.ones, torch.full
# defaults to float32
In [8]: torch.ones(5).dtype
Out[8]: torch.float32
In [9]: torch.zeros(5).dtype
Out[9]: torch.float32

# full defaults to int
In [16]: torch.full(size=(5,), fill_value=4).dtype
Out[16]: torch.int64

note:
if we do decide to do this
all Tensor() inits will have to be
Tensor([1.]) instead of Tensor([1])
to init a float tensor
(saw torch do this a few times in test_indexing)

There are a lot of changes around the code base that would need to be made to adapt to this
I only changed enough to get a green CI cuz green CI is nice to look at
This is also kinda an invitation for bugs since just having everything in float makes it so that we don't have to worry about dtype shannanigans (I vaguely remember geohot saying something a long time ago about having everything in float)
idk
if the current approach is ok (doing Tensor(1.) for float or adding dtype=dtypes.float), and having int or list of int default to int32 is what we want, I could go through the codebase and spot places that need to be changed to float.
otherwise feel free to close

thanks for initiating this. there's a deeper problem here that we are not auto cast int to float for the ops like DIV, EXP etc. changing the test callers does not fix the issue. i am looking into that now

https://github.com/chenyuxyz/tinygrad/pull/28  🚀 🚀 🚀


Next.JS Implementation

Hey,
Thanks for the great repo. I found that I cant use this in my next.js app because all global CSS has to be declared on the root component. if there a way to use this without the CSS? I think that should make it work.

Please let me know if you need any more details. thank you!

@jonrrivera https://github.com/uiwjs/react-md-editor/issues/52 Hope it helps you.

@jonrrivera Exmaple: https://codesandbox.io/s/nextjs-example-react-markdown-preview-vuxih?file=/pages/index.js

https://github.com/uiwjs/react-md-editor/issues/52#issuecomment-848969341
I created a nextjs package next-remove-imports to solve the problem.

@jonrrivera

I made it work by adding this to my next.config.js file:
const removeImports = require("next-remove-imports")({});

const nextConfig = removeImports({
 your config here...
 ...
 ...
});

module.exports = nextConfig;

And the following within my page:
import { MarkdownPreviewProps } from "@uiw/react-markdown-preview";

const MarkdownPreview = dynamic<MarkdownPreviewProps>(
  () => import("@uiw/react-markdown-preview"),
  {
    ssr: false,
  }
);

const MyPage: NextPage = () => {
  return (
<MarkdownPreview source={"Some **markdown** text"} />
)
}

Example: @uiwjs/next-remove-imports/example


Use Legendre as a PRF for the proof of custody

I am opening this issue in order to track the progress of my research on using Legendre as a pseudo-random function (PRF) for the proof of custody. This is to give everyone the opportunity to contribute their knowledge as well as get input where people think we need more information before finalizing this decision.
Why Legendre?
The rationale for doing this is that this would enable validators to use an MPC to compute the proof of custody, so that validator secrets can be shared across multiple parties in a trustless way, without a risk of a minority getting the validator slashed.
Suggested construction
This is my currently favoured construction: legendre_proof_of_custody_minimal.pdf
It was designed based on the goals of (1) trying to minimize the number of Legendre invocations (it turns out this is still a major bottleneck in the MPC) and (2) trying to improve the safety of the construction (so I use the output of many Legendre PRFs as the input of another Legendre symbol as a kind of "MPC-friendly one-bit hash" in order to mask the Legendre outputs from cryptanalysis).
A python prototype is available here: https://github.com/ethereum/research/blob/master/proof_of_custody/legendre_aggregate_minimal.py
Performance
The python prototype currently runs in 65 ms on my PC. The "old" implementation using sha256 needs 8 ms (4096 sha256 invocations on 560 data bytes). So while it is slower, cleartext performance still seems very reasonable and not like a major issue. Performance improvements on my current python version can almost certainly be made as only ca. 20 ms are spent on computing the Legendre symbol itself (for which I use GMP).
MPC friendliness
It appears that Legendre is by far the most MPC-friendly PRF known, especially if only one bit of output is needed as in our case [1]. Even given this, however, getting reasonable performance on 2 MB inputs requires some optimization. The construction above uses the property that the XOR of several Legendre "bits" is actually the same as multiplying the inputs and then computing one Legendre of the product, which enables us to cut down the number of actual Legendre invocations by a factor of 10 to only 4096 invocations.
Here is a reference implementation of this using the SCALE-MAMBA system [2]: https://github.com/ethereum/research/blob/master/proof_of_custody/legendre_aggregate_minimal.mpc
The online phase with 3 players runs in 14s locally and in 29 s under WAN conditions (usingtc qdisc add dev lo root netem rate 50000kbit delay 100ms to emulate). There is significant optimization potential by utilizing vectoring inside the SCALE-MAMBA system and the current program is not optimized for rounds, so I think latency will not be the main issue. Currently the per-player communication required (each node sends and receives this amount of data) is 22.86 MB (of which 4.52 MB in the online thread, the remainder can be precomputed to reduce latency).
Likely the MPC can be further optimized using the MPC-method developed in [4]. This is especially promising as it only needs a constant amount of computation for any number of parties.
Safety
Safety in our case means that there is no way someone can extract the proof of custody secret from the output of the custody bits.
Classical safety
Currently, there are no known classical attacks against the Legendre PRF. The best known classical algorithm is O(p) [5], i.e. exponential. So far research is limited and the problem has not been covered as much as many other more well-known crypto primitives such as RSA or EC.
According to Dan Boneh, "Generally, the Legendre PRF is a secure PRF", and the iterative construction makes the problem harder [6]. Of course, it would be nice to have some more information on that, and I am currently looking for someone who can prove that the iterative construction is actually harder to analyse, in a similar way as Damgård did in his original paper when suggesting the Legendre PRF [7], where he showed stronger security of the Jacobi symbol given weak assumptions on the Legendre symbol.
Quite a lot of work has been done on the Legendre symbol as a random number generator. In that respect it seems to have almost ideal properties (very low correlations and a perfectly flat spectrum as it is an eigenvector of the Fourier transform). Note however that this does not indicate cryptographic security by itself.
Quantum safety
Currently, all known quantum algorithms ([5], [8]) are based on a quantum evaluation of the Shifted Legendre symbol (whereas in our case, the quantum algorithm would only have a finite number of classical evaluations to work with). From this perspective, the quantum safety of the Legendre PRF is currently unknown. According to Wim van Dam (author of [8]), a couple of people have thought about the problem and didn't find an algorithm for the classical query problem.
Computing the secret
The proof of custody secret is currently one of the "derived" secrets of the phase 0 spec, which means it is the BLS signature of a given epoch. This signature can be represented by two numbers a and b ∈ 𝔽q^2. Since one can be computed using the other (modulo one extra bit), a can be used as the canonical representation of the signature. Now a itself as an element of 𝔽q^2 is represented by two 𝔽q-numbers s1, s2. My construction uses these as the secrets for the Legendre-based proof of custody, which comes very natural as they are already 𝔽q numbers.
Assuming that the the participants have t-threshold Shamir shares of their secret, we still need a way to securely compute s1 and s2. A proof of concept for doing this in an MPC in order to get shares of s1 and s2 can be found here: https://github.com/ethereum/research/blob/master/proof_of_custody/bls_sign.mpc
Alternatives
While there are other MPC-friendly PRFs ([1] has a good summary except for some more recent and very experimental ones), none of them are even close to Legendre in performance. Given that MPC performance is still a bottleneck at the current stage of research, I think none of these will be viable.
An alternative proposed by Dan Boneh and others are so-called key-homomorphic PRFs, which are homomorphic with regards to addition of the key. This would of course be a very natural way to have a shared proof of custody without needing an MPC. The most promising one is the DDH PRF, which is basically hash(x)^s in a Diffie-Hellman group. If we do this inside BLS12-381, this is actually just a BLS signature :) However, we have considered this scheme before and rejected it due to poor cleartext performance (state of the art is 3 ms for a BLS signature, so the cleartext computation would take 10s of seconds). Using modular exponentiation is similar at 10 ms per exponentiation for 2048 bit primes, which seems to be the recommended size.
[1] https://eprint.iacr.org/2016/542.pdf
[2] https://github.com/KULeuven-COSIC/SCALE-MAMBA
[3] https://eprint.iacr.org/2011/535.pdf
[4] https://eprint.iacr.org/2018/570.pdf
[5] https://arxiv.org/pdf/quant-ph/0212016.pdf
[6] Private communication
[7] https://link.springer.com/content/pdf/10.1007%2F0-387-34799-2_13.pdf
[8] https://arxiv.org/pdf/quant-ph/0011067.pdf

Update: BLS signature MPC proof of concept is now done. Doing the full ECMUL in an MPC is actually not too expensive (similar cost to one proof of custody computation), so no need for fancy tricks.

An update on the MPC implementation: I now found a way to reliably test the total communication in the 2 out of 3 MPC case using the SCALE-MAMBA system:

proof of custody computation: 22.86 MB
BLS signature MPC: 23.83 MB
(The original 60 MB figure was due to the system doing some extra "preparation work" that was not actually needed).

Note that these figures scale with n^2 in the number of parties, so this system would still be quite unattractive for more than about 5-10 parties. There are protocols (e.g. https://eprint.iacr.org/2018/570) that do this constant in the number of parties, at least for the multiplications, and my back-of-the-envelope computation indicates that we would also end up with somewhere between 20-30 MB. I am currently investigating how to best implement this paper in order to benchmark it.
Overall, I am now very confident that this construction allows an efficient computation of the proof of custody using an MPC.

So there have been some small updates to the construction I suggested, mainly to correct for a possible attack vector that Justin suggested: Setting many of the subchunks to the same value introduces a lot of "squares" in the Legendre evaluation, which means while their value itself cannot be predicted, their combined value would always be 1. This would allow an attacker to "pierce through" the iterated Legendre construction, if they had full control of all the data in the proof of custody, and proble single Legendre bits (which is still safe if Legendre is a good PRF, but remove the additional safety layers).
Anyway, this attack vector can be removed by adding multiples of the secret, which removes the ability of an attacker to create squares unless they know the derived secret.
One further change I have made is that the subchunks are now 1048 and 132 bytes (total of 11 subchunks per chunk), which is more in line with the general spec in terms of byte sizes.
Here is an updated writeup: legendre_proof_of_custody_minimal_protected.pdf - I am still looking for cryptographers who are interested in examining this further.
Updated MPC and Python code: https://github.com/ethereum/research/tree/master/proof_of_custody
(Now the SCALE-MAMBA takes 24.66 MB of communications, slight increase due to increasing subchunks from 10 to 11)

@dankrad Do you want to share a summary of what's happened in the last ~6 months regarding Legendre?
(Off the top of my head: Dmitry found an attack, you setup a bounty program, three teams independently found an improvement to Dmitry's attack, some of the easier bounties were claimed, a paper was published, and recently a further small attack improvement was found. As I see it, using Legendre remains the plan as it is still far from being broken for our use case and we don't have much choice anyway in the context of the new per-slot crosslinks.)

@dankrad Can we close this issue? :)


windows: convert wstr to str

SHGetKnownFolderPath returns wide-character strings, which need to
be converted to UTF-8 with WideCharToMultiByte.
WideCharToMultiByte is in Kernel32 which will always be loaded in
the process. No need to dlopen it.

Thanks. The patch is in as @5c02647. As it was easier than rebasing. ;)


Inactive users are still visible and accessible

When a user becomes inactive, it should disappear from all channels, from search, shold not accept messages

Fixed by rocketchat team


Build fails: error: no viable overloaded '='

Build fails on FreeBSD:
In file included from /usr/ports/math/metasmt/work/metaSMT-3-471-g6623580/tests/test_result_wrapper.cpp:2:
/usr/ports/math/metasmt/work/metaSMT-3-471-g6623580/tests/../src/metaSMT/result_wrapper.hpp:47:18: error: no viable overloaded '='
          ret[i] = vb[i];
          ~~~~~~ ^ ~~~~~
/usr/local/include/boost/logic/tribool.hpp:72:7: note: candidate function (the implicit copy assignment operator) not viable: no known conversion from 'std::__1::vector<bool, std::__1::allocator<bool> >::const_reference' (aka '__bit_const_reference<std::__1::vector<bool, std::__1::allocator<bool> > >') to 'const boost::logic::tribool' for 1st argument
class tribool
      ^
1 error generated.

hoangmle/metaSMT still fails to build:
In file included from /usr/ports/math/metasmt/work/metaSMT-3-622-gc6e4985/tests/test_result_wrapper.cpp:2:
/usr/ports/math/metasmt/work/metaSMT-3-622-gc6e4985/tests/../src/metaSMT/result_wrapper.hpp:122:16: error: no viable overloaded '='
        ret[0] = t;
        ~~~~~~ ^ ~
/usr/include/c++/v1/__bit_reference:56:22: note: candidate function not viable: no known conversion from 'boost::logic::tribool' to 'bool' for 1st argument
    __bit_reference& operator=(bool __x) _NOEXCEPT
                     ^
/usr/include/c++/v1/__bit_reference:66:22: note: candidate function not viable: no known conversion from 'boost::logic::tribool' to 'const std::__1::__bit_reference<std::__1::vector<bool, std::__1::allocator<bool> >, true>' for 1st argument
    __bit_reference& operator=(const __bit_reference& __x) _NOEXCEPT
                     ^
In file included from /usr/ports/math/metasmt/work/metaSMT-3-622-gc6e4985/tests/test_result_wrapper.cpp:2:
/usr/ports/math/metasmt/work/metaSMT-3-622-gc6e4985/tests/../src/metaSMT/result_wrapper.hpp:135:17: error: no viable overloaded '='
         ret[i] = vt[i];
         ~~~~~~ ^ ~~~~~

rev. c6e4985


Add AVFeature type to build.rs

The build script has gotten pretty complex over time and implements a decent amount of workarounds and other cruft.
I plan on doing small-ish changes and PR them bit by bit. @shssoichiro if you prefer, I could also consolidate a rework into one big PR for review.

I think smaller PRs are easier to review, so sending them in one at a time would be ideal. Either way, the fixes are definitely appreciated!


IntelliJ IDEA plugin development does not keep pace

Hi.
I am very concerned about @mplushnikov 's team seems not to keep their plugin in active deveplopment.
Just bumping versions and that is it.
Without proper IDE support Lombok users will struggle.
Is there any possibility to give a watchword (if correct to say so) to the community to find one(s) who's freely familiar with IntelliJ IDEA plugins ecosystem to quickly fork and continue or smth like that,
I am not familiar at all, that is why I post this.

Maybe the best way is not to fork it but help contribute to that project? I'm sure the plugin would appreciate a pull request or 2.

@hc-codersatlas  Sure. The poit is not "how" but "what". I a kind of overestimate to eliminate the problem.

@mplushnikov silently released current stable version support plugin though keeping almost total silence e.g. not responding to most of the issue tracker messages. Might be busy, but not clear why so non-comunicative in issue tracker. Great they made it overall. Thanks.


Create 'pg_attrdef' Catalog Table

We need to store attribute default values in the catalog. Postgres uses the pg_attrdef catalog table. We need to add this:
https://www.postgresql.org/docs/9.6/static/catalog-pg-attrdef.html
We need to support the following type of default values:
CREATE TABLE foo (
   val1 INT DEFAULT 1,
   val2 INT DEFAULT 1 + 1 + 1 +1,
   val3 TIMESTAMP DEFAULT NOW(),
   val4 INT DEFAULT ASCII('x')
);

The first step is to create the catalog object and then support populating it when we create a table. Then the next step is to add support for using this information in DataTable. We should make sure that we support the DEFAULT keyword:
INSERT INTO foo VALUES (DEFAULT, DEFAULT, DEFAULT, DEFAULT);

Sure, will look into that during weekend.


8.5.2RC1 - File manager: Number of items per page can't be changed in the modal

The selected number of items per page in the file manager seems to be remembered only per session. So, for example, if you log in and want to select an image for the image block, the file manager modal will always show the default 10 items per page, but you may want to change that value when selecting a file. Currently this value can't be changed in the modal.
I don't believe that this behavior is by design.
To reproduce:

Log in.
On any frontend page, add an image block to an area.
In the block's modal click the button "Choose Image".
The file manager modal opens.
The select field for items per page shows 10.
Click on the select field and try to select any other value - it's not possible.
Moreover, while you do this for the first time, the file manager modal jumps down the screen quite a bit for whatever reason.

Tested on a clean install with Elemental theme.

The issue is gone in 8.5.2


fix(scaffolder,bitbucket): support custom defaultBranch for Bitbucket Cloud

Before this change, the returned repoContentsUrl of this scaffolder action
was hardcoded to branch master which is the default value for defaultBranch.
The change will ensure to use the provided value instead.
Closes: #9759
Signed-off-by: Patrick Jungermann<EMAIL_ADDRESS>Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

FYI: I believe Bitbucket Server might not work as well, but as I have no setup for it, I kept it untouched.


Some of the readmes in here are out-of-date - we've reorganized the repo layout and forgotten to update them.

I was just looking around and found that:

main repo readme has a link for "HTS Precompile methods" that's a 404.

the contracts subdirectory has a readme which names several subdirectories that don't exist there, they've been moved elsewhere:

Likely other links are bad too, I haven't checked.

addressed in this PR https://github.com/hashgraph/hedera-smart-contracts/pull/966


Missing partial shared/views/ems_common/_vms error

I ran into this exception while navigating to VM list page from the summary page of an infra provider.
Missing partial shared/views/ems_common/_vms with {:locale=>[:en], :formats=>[:html], :variants=>[], :handlers=>[:erb, :builder, :raw, :ruby, :haml, :jbuilder, :rjs]}. Searched in: * "/var/www/miq/vmdb/app/views" [ems_infra/show]
Steps to reproduce:

Add an infra provider
Navigate to the summary page of the provider.
Click on the 'VMs and Instances' link in the Relationships section

this is dup of https://github.com/ManageIQ/manageiq/issues/4272


[ENH]: Remove the cmdlet compatibilty logic

Describe the feature/enhancement you'd like added
We barely make use of this logic and it's more annoying than it is useful.
A better option is to simply have a check in each OS specific script to ensure it's in the right place.
We could even make a basic cmdlet to do so if we wish.

This should be done by way of #35


Adding a score_permission method.

initial stab at scoring.
I have a branch of policyuniverse that uses this.  More to come.

Looks sensible, might be worth some comments describing what each of the tags means.


DVC tracked file rename fails when file is not pulled yet

After trying to rename the images in the get-started-experiment:

File and directory are completely missing in the DVC tracked view.

Logs
```
ERROR: failed to get diff - Unable to acquire lock. Most likely another DVC process is running or was terminated abruptly. Check the page  for other possible reasons and to learn how to resolve this.
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:30:10.240Z, pid: 8062] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc status --show-json - COMPLETED (7189ms)
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:30:34.872Z, pid: 8071] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . --dvc-only -R --show-json - COMPLETED (31813ms)
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:34:09.044Z, pid: 9486] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . data --dvc-only --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:34:09.572Z, pid: 9486] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . data --dvc-only --show-json - COMPLETED (530ms)
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:35:26.889Z, pid: 10054] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc pull data/images.tar.gz - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:35:29.514Z, pid: 10054] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc pull data/images.tar.gz - FAILED with code 255 (2629ms)
ERROR: unexpected error - [Errno 22] The provided token is malformed or otherwise invalid.: An error occurred (InvalidToken) when calling the ListObjectsV2 operation: The provided token is malformed or otherwise invalid.
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:39:06.877Z, pid: 11773] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc diff --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:39:06.879Z, pid: 11774] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc status --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:39:06.887Z, pid: 11783] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . --dvc-only -R --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:39:09.817Z, pid: 11773] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc diff --show-json - FAILED with code 1 (2944ms)
ERROR: failed to get diff - Unable to acquire lock. Most likely another DVC process is running or was terminated abruptly. Check the page  for other possible reasons and to learn how to resolve this.
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:39:14.659Z, pid: 11774] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc status --show-json - COMPLETED (7782ms)
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:05.360Z, pid: 11783] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . --dvc-only -R --show-json - COMPLETED (58476ms)
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:18.505Z, pid: 12299] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc move data/images.tar.gz data/images1.tar.gz - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:19.518Z, pid: 12326] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . data --dvc-only --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:20.648Z, pid: 12326] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list . data --dvc-only --show-json - FAILED with code 1 (1132ms)
ERROR: failed to list '.' - The path 'data' does not exist in the target repository '/Users/ivan/Projects/get-started-experiments' as an DVC output.
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:20.681Z, pid: 12331] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list .  --dvc-only --show-json - INITIALIZED
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:21.427Z, pid: 12299] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc move data/images.tar.gz data/images1.tar.gz - FAILED with code 255 (2925ms)
ERROR: unexpected error - [Errno 2] No such file or directory: '/Users/ivan/Projects/get-started-experiments/data/images.tar.gz'
[version: 0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0, 2021-09-14T05:40:21.938Z, pid: 12331] > /Users/ivan/Projects/get-started-experiments/.env/bin/python -m dvc list .  --dvc-only --show-json - COMPLETED (1260ms)
```

Environment
0.1.0-e1bf948c5cc02edf6260c675401d500502abe9b0
Version: 1.60.0
Commit: e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff
Date: 2021-09-01T10:54:53.442Z
Electron: 13.1.8
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Darwin x64 19.6.0

DVC version: 2.7.3+788b4d
---------------------------------
Platform: Python 3.8.9 on macOS-10.15.6-x86_64-i386-64bit
Supports:
        azure (adlfs = 2021.9.1, knack = 0.8.2, azure-identity = 1.6.0),
        gdrive (pydrive2 = 1.9.3),
        gs (gcsfs = 2021.8.1),
        hdfs (pyarrow = 4.0.1),
        webhdfs (hdfs = 2.6.0),
        http (aiohttp = 3.7.4.post0, aiohttp-retry = 2.4.5),
        https (aiohttp = 3.7.4.post0, aiohttp-retry = 2.4.5),
        s3 (s3fs = 2021.8.1, boto3 = 1.17.106),
        ssh (sshfs = 2021.8.1),
        oss (ossfs = 2021.8.0),
        webdav (webdav4 = 0.9.1),
        webdavs (webdav4 = 0.9.1)
Cache types: reflink, hardlink, symlink
Cache directory: apfs on /dev/disk1s1
Caches: local
Remotes: s3
Workspace directory: apfs on /dev/disk1s1
Repo: dvc, git

First workaround to try: disable this in situations that DVC can't handle (e.g. when files are not physically present yet).

Let's delete the rename command.


JsonSerializationException on CreateChatCompletion

Describe the bug
Calling CreateChatCompletion results in a JsonSerializationException when attempting to deserialize the CreateChatCompletionResponse object. Full error: JsonSerializationException: Could not find member 'system_fingerprint' on object of type 'CreateChatCompletionResponse'. Path 'system_fingerprint', line 21, position 23.
To Reproduce
Steps to reproduce the behavior:

Call the following:

var completionResponse = await openai.CreateChatCompletion(new CreateChatCompletionRequest()
{
Model = "gpt-4-1106-preview",
Messages = [list of messages],
Temperature = 0.0f,
}

Expected behavior
A populated completionResponse.
Environment:

OS: MacOS Ventura version 13.4
Unity Version [e.g. 2022.3.7f1]
Plugin Version [e.g. 0.1.5]

Context:
This was working yesterday. Maybe the OpenAI API updated?

Hi @mifu67 my guess also is that Open AI might have made system_fingerprint mandatory in the response type. Can you add
public SystemFingerprint object { get; set; } in CreateChatCompletionResponse here to see if that fixes the issue?
I do not have an API Key, so I wont be able to test it unfortunately.

@srcnalt yes, that fixes the issue. Thanks!


Bug report: m365 pa app list

Description
m365 pa app list command is now not returning data but error "Error: The received access token has been obtained from wrong audience or resource."
Seems some changes done by Microsoft at the API level were the token with aud "https://management.azure.com/" does not work to get the list of PowerApps and related commands.
This was working fine for a couple of days.
Looking at the Original PowerApps Powershell it seems now the aud "https://service.powerapps.com/" would be required...because when I used the Token that we are getting from PowerApp Powershell with https://service.powerapps.com/ we are able to make calls from postman also but another access token with resource "https://management.azure.com/" is no longer working.
Wondering if there is a way to get a token or log in with aud "https://service.powerapps.com".
Steps to reproduce
m365 login
m365 pa app list

Expected results
It should return list of PowerApps for currently logged-in user.
Actual results
Error: The received access token has been obtained from wrong audience or resource.
Diagnostics
Executing command pa app list with options {"options":{"debug":true,"output":"json"}}
Existing access token eyJ0eXAiOiJKV1QiLCJhbG......................[Remove token from here] still valid. Returning...
Request:
{
  "url": "https://api.powerapps.com/providers/Microsoft.PowerApps/apps?api-version=2017-08-01",
  "method": "get",
  "headers": {
    "common": {
      "Accept": "application/json, text/plain, */*"
    },
    "delete": {},
    "get": {},
    "head": {},
    "post": {
      "Content-Type": "application/x-www-form-urlencoded"
    },
    "put": {
      "Content-Type": "application/x-www-form-urlencoded"
    },
    "patch": {
      "Content-Type": "application/x-www-form-urlencoded"
    },
    "user-agent": "NONISV|SharePointPnP|CLIMicrosoft365/6.2.0",
    "accept-encoding": "gzip, deflate",
    "X-ClientService-ClientTag": "M365CLI:6.2.0",
    "accept": "application/json;odata.metadata=none",
    "odata-version": "4.0",
    "authorization": "Bearer eyJ0eXAiOiJKV1QiLCJhbGciO...................[Removed token from here]"
  },
  "responseType": "json",
  "decompress": true
}
Request error:
{
  "url": "https://api.powerapps.com/providers/Microsoft.PowerApps/apps?api-version=2017-08-01",
  "status": 401,
  "statusText": "Unauthorized",
  "headers": {
    "connection": "close",
    "content-type": "application/json; charset=utf-8",
    "date": "Fri, 10 Feb 2023 04:30:39 GMT",
    "cache-control": "no-store, no-cache",
    "transfer-encoding": "chunked",
    "vary": "Accept-Encoding",
    "strict-transport-security": "max-age=15724800; includeSubDomains",
    "x-ms-islandgateway": "GA0000003",
    "x-ms-request-id": "india.centralindia:b5575623-bb92-4aeb-a334-b881b4a5a487",
    "x-ms-correlation-request-id": "5e6cd9dd-417e-4dbc-9879-71bd8f7e9565",
    "x-servicefabric": "NoRetry",
    "x-ms-service-request-id": "b6e1f326-a705-4b7a-bc26-103b3f9ab776",
    "x-ms-correlation-id": "5e6cd9dd-417e-4dbc-9879-71bd8f7e9565",
    "x-ms-activity-vector": "<IP_ADDRESS>",
    "server-timing": "x-ms-igw-upstream-headers;dur=4.6,x-ms-igw-req-overhead;dur=0.8",
    "x-content-type-options": "nosniff"
  },
  "error": {
    "error": {
      "code": "InvalidAuthenticationToken",
      "message": "The received access token has been obtained from wrong audience or resource."
    }
  }
}

CLI for Microsoft 365 version
"v6.2.0"
nodejs version
v16.15.1
Operating system (environment)
Windows
Shell
cmd
cli doctor
{
  "os": {
    "platform": "win32",
    "version": "Windows 10 Home",
    "release": "10.0.22000"
  },
  "cliVersion": "6.2.0",
  "nodeVersion": "v16.15.1",
  "cliAadAppId": "31359c7f-bd7e-475c-86db-fdb8c937548e",
  "cliAadAppTenant": "common",
  "authMode": "DeviceCode",
  "cliEnvironment": "",
  "cliConfig": {},
  "roles": [],
  "scopes": [
    "AllSites.FullControl",
    "AppCatalog.ReadWrite.All",
    "AuditLog.Read.All",
    "Bookings.Read.All",
    "ChannelMember.ReadWrite.All",
    "ChannelMessage.Read.All",
    "ChannelMessage.Send",
    "ChannelSettings.ReadWrite.All",
    "Chat.Read",
    "Chat.ReadWrite",
    "Directory.AccessAsUser.All",
    "Directory.ReadWrite.All",
    "ExternalConnection.ReadWrite.All",
    "Group.ReadWrite.All",
    "IdentityProvider.ReadWrite.All",
    "Mail.Read.Shared",
    "Mail.ReadWrite",
    "Mail.Send",
    "Notes.Read.All",
    "Place.Read.All",
    "Policy.Read.All",
    "RecordsManagement.ReadWrite.All",
    "Reports.Read.All",
    "SecurityEvents.Read.All",
    "ServiceHealth.Read.All",
    "ServiceMessage.Read.All",
    "ServiceMessageViewpoint.Write",
    "Tasks.ReadWrite",
    "Team.Create",
    "TeamMember.ReadWrite.All",
    "TeamsAppInstallation.ReadWriteForUser",
    "TeamSettings.ReadWrite.All",
    "TeamsTab.ReadWrite.All",
    "TermStore.ReadWrite.All",
    "User.Invite.All",
    "User.Read.All",
    "User.ReadWrite.All",
    "profile",
    "openid",
    "email",
    "user_impersonation",
    "user_impersonation"
  ]
}

Additional Info
No response

Hi @siddharth-vaghasia
Thanks you for logging this bug and doing already a lot of research! On first sight, this seems to be related to #4460.

thanks @milanholemans for the response. putting my comments in the PR

Hi @siddharth-vaghasia,
thanks for reporting this!
On my end the command is still working. But that may be some release process that's generally kicking over things here and there, in which case it might stop working for me soon as well.

@pnp/cli-for-microsoft-365-maintainers is anyone else experiencing this issue?

Still working for my tenant in first release.

@siddharth-vaghasia now #4460 has been merged in our stable version. Can you confirm that it now works for you?

Closing this issue due to lack of response.


luci-mod-admin-full: Fix wrong icon when noise=0

#1101 The wireless client signal strength is not displayed
The existing code depends on subtraction of two negatives being a positive.
However, sometimes noise comes into this code as 0.
This creates a negative subtraction result, which selects the wrong icon.
Changing the formula from a multiply by -1, to a Math.abs()
ensures that the result is always a positive value.
Signed-off-by: Bob Meizlik<EMAIL_ADDRESS>

Any progress here?

Closing
No progress since a half year


Error in the path of the predict step!

Paul and I tried to test with interactive python with the "_MAS.fasta" file, and it seems the script can not find the absolute path of this file.
Traceback (most recent call last):
  File "/home/morrellp/llei/BAD_Mutations/BAD_Mutations.py", line 335, in <module>
    main()
  File "/home/morrellp/llei/BAD_Mutations/BAD_Mutations.py", line 315, in main
    out = predict(arguments_valid, loglevel)
  File "/home/morrellp/llei/BAD_Mutations/BAD_Mutations.py", line 250, in predict
    lrt.prepare_hyphy_inputs()
  File "/panfs/roc/groups/9/morrellp/llei/BAD_Mutations/lrt_predict/Predict/predict.py", line 102, in prepare_hyphy_inputs
    infile.write(os.path.abspath(self.nmsa) + '\n')
  File "/soft/python2/2.7.8/lib/python2.7/posixpath.py", line 367, in abspath
    if not isabs(path):
  File "/soft/python2/2.7.8/lib/python2.7/posixpath.py", line 61, in isabs
    return s.startswith('/')
AttributeError: 'MultipleSeqAlignment' object has no attribute 'startswith'

Fixed in 9f55398ecc8ace1a46a7afdaf8c39ca412576352
predict.py was trying to get the path for a MultipleSequenceAlignment object, when it should have been trying to get the path for a str object representing a filename


Sanitization issues in custom HTML infowindows

Hey, people have reported a couple of issues these days when they use code inside their custom infowindows:

IMG tags which contain URL with & characters are broken due to escaping
target="_blank" tags have stopped working too

@viddo is there a reason to now allow target and img tags with &?

All attributes are escaped by default, but a browser should be capable of handling that. I've been fiddling around in local dev env and have not been able to break it. Can you give some example of an img URL, @iriberri ?
The target attribute is disallowed by default, but I can't find any good motivation for why that is. I can change the policy to allow it.

Sure, I won't share them publicly but: SB/4981277 -- they can be found in the first message.

Example (target blank): https://team.cartodb.com/u/iriberri/viz/bb8a2cb4-dc52-11e4-bdcc-0e4fddd5de28/map

http://gis.stackexchange.com/questions/141473/links-will-not-open-to-new-window-cartodb

The target issue will be solved here, https://github.com/CartoDB/cartodb.js/pull/428 I'll reply to that thread on stackexchange, thx!
Re: the img isssue far it seem to be an issue on embeds only for some reason, still investigating...

The img-issue is due to the Mustache's escaping, using the unescaped output (triple angle brackets, i.e. <img src={{{ img_url }}} ...) makes the image to be rendered as expected. Custom templates are filtered through the sanitize step anyways so it's safe to do. No code change is necessary.

Extra info about '&' escaping in URLs, they get converted to:

Great, I'll let the users know. Thanks!


CMK in KV with private endpoint

Can storage accounts & Azure Database for PostgreSQL use CMK in a KV which uses private endpoint?
I am guessing from #85277 it is not possible. If this can be updated in the docs it would be great!
PostgreSQL Content: Docs

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

ID: 6dc2f1a2-3f5a-baf3-94e7-37bf4539530b
Version Independent ID: 13dd580d-f249-62eb-13e2-da76f2f7f7dc
Content: Configure encryption with customer-managed keys stored in Azure Key Vault - Azure Storage
Content Source: articles/storage/common/customer-managed-keys-configure-key-vault.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

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

@SaurabhSharma-MSFT Is there an update if CMK can be used with a key vault with only private endpoint?

#reassign: @tamram

@ramki88 - Thanks for your question. I checked with the engineering team, and this is a supported scenario. You need to configure the key vault to allow trusted access to Microsoft services, as described here:
https://docs.microsoft.com/azure/key-vault/general/network-security#key-vault-firewall-enabled-trusted-services-only
#please-close

@tamram -- that was not the question, so your answer is entirely irrelevant!
The question is if PRIVATE ENDPOINT can be used. Enabling trusted services to bypass the firewall is not the same.
Can somebody from Microsoft who understands the question please weigh in here? Based on lab testing I'm doing, I have not found a way to make this work. Not sure if I'm missing a piece, or if this configuration is not supported.

Can confirm we have the same issue

Funny that if we temporary set allow access to key vault for certain IPs then we are able to set the CMK on storage account and then switch KV back to public access disabled.
But not sure if it's even working on background.

Funny that the problem occurs when we create new Key Vault with public access disabled.

I think this is not so funny, as it suggests that someone just implemented a simple check of "Allow trusted" and just forgot that this is not the only thing that should be also checked :| making private endpoint usage effectively possible with your workaround.
So I'm still not so sure if it is safe to enable communication over private endpoint or not :(

I think it's the safest way Azure can offer right now but keep in mind that due to the nature of SaaS services private endpoint is not the same as having application deployed in private on-prem network. Only subset of operations (data plane) goes through the private endpoint, control plane operations will always be (must be) accessible through public internet network. Moreover the domain names of resources with private endpoint is still resolvable to public IP and is publicly routable.
This article summarizes it pretty well https://medium.com/version-1/the-realities-of-azure-private-endpoints-45bf01986b7d

So is it generally safe to turn on public access and then turn it off at a later step? In the case of cosmos db for postgres, even the network limited public access will not work. Can anybody weigh in? I feel it's not optimal to have the key vault for production publicly reachable.

@ozgeeg FYI


Create content card 5 blue

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow  figma design and to make the button use Button Design System
After building the component please raise a PR with a screenshot of the component and add the component in Content Cards/src/Content-Card-10-blue.html. Please do not add hover effect in the main div
If you need to use any icon please use it from Hero Icons
Good luck.

@RavindraP04  I want to contribute

@RavindraP04  I want to contribute

@RavindraP04  I want to contribute

#1768 Created
@RavindraP04 Check it out


UI: apply timezone settings

Background
At present, the time displayed in the UI is set to UTC by default. Consequently, users of the UI are required to individually convert the displayed time to their respective local time zones, which can be rather inconvenient.
Overview
In this pull request, I intend to modify the UI so that the displayed time aligns with the server's TimeZone setting, thereby enhancing user experience.
To achieve this, the React components have been updated to obtain TimeZone settings from Redux. I was uncertain about the most appropriate Redux Store for storing the TimeZone settings and ultimately decided on the domains store. If you believe there is a more suitable location for these settings, kindly share your insights.
Additionally, as the TimeZone will be utilized across the majority of pages within the UI, I have incorporated the necessary changes into _app.js. If there is a more efficient or effective approach to implementing this feature, please do not hesitate to share your suggestions.

I apologize for the large commit. If using redux is not appropriate, I'm thinking of exploring other methods. I would appreciate any comments you may have.

Apologies for the delay. Our team has started looking into it but they haven't completed the review yet. I will check with them this week. Thank you for your patience :)

 My time zone is IDT (Israel) and I get a different time zone. What I don't understand is why I don't get the same time zone for the second member's expiration and the second member's review reminder

@mendi160 that's actually correct and is not a different time zone. Both are Pacific Time Zone - however, based on the date, it correctly reports the daylight savings time in effect on that date:
PDT - Pacific Daylight Time zone
PST - Pacific Standard Time zone

@mendi160 I apologize for the confusion, my explanation was not clear. This PR does not change the display according to each user's timezone. In our environment, we basically operate in JST and our users are also familiar with JST. Therefore, we wanted to convert the timezone displayed in the UI to JST, which led to this change.

@mendi160 I apologize for the confusion, my explanation was not clear. This PR does not change the display according to each user's timezone. In our environment, we basically operate in JST and our users are also familiar with JST. Therefore, we wanted to convert the timezone displayed in the UI to JST, which led to this change.

If so, it LGTM


vpnc does not work with systemd-resolved

vpnc (and other VPN tools, e.g. wireguard) uses resolvconf (as provided by openresolv) to edit /etc/resolv.conf. This does not work with systemd-resolved, for which /etc/resolv.conf is a symlink into the nix store.
systemd can handle /etc/resolv.conf in four different modes, but I think the key decision is whether it should be a symlink or a regular file:

If it is a regular file, vpnc can continue to use openresolv to manage /etc/resolv.conf. We would just need to ensure that <IP_ADDRESS> is added as a nameserver when using systemd-resolved (perhaps using resolvconf -a in the activation script?).
If it is a symlink, vpnc should instead be set up to use systemd's resolvconf compatibility shim. Then via this shim it will talk to systemd directly (over DBus?), circumventing the need for /etc/resolv.conf to be mutable.

I think my preference is for (2), though there is a slight problem in that the resolvconf shim is currently broken in our fork of systemd (https://github.com/NixOS/nixpkgs/issues/54617). But long term we can assume that this will be fixed.
Then the question would be how to configure vpnc to use systemd instead of openresolv. I can confirm that vpnc.override { openresolv = systemd-patched; } works for me, where systemd-patched is systemd patched with a fix for (https://github.com/NixOS/nixpkgs/issues/54617).
But this seems a bit ugly - perhaps there is a better way? The way Arch does it is with a systemd-resolvconf package that provides and conflicts with openresolv.
Note that this could also lead to a fix for the resolvconf collisions that appeared in 18.09.
@fpletz @vcunat

I think this question is related to what we're discussing in #62955, which introduces a new NixOS module for resolvconf. An open question there is, if resolvconf is supposed to be disabled, how can we be sure none of the various packages like vpnc will actually call it?

We recently moved to 243. Is this working for you now?


Layout as template parameter

Containers in xtensor should take an additinoal layout template parameter with three possible values:

row_major
column_major
dynamic_layout

This would facilitate the integration of BLAS libraries and the use of SIMD intrinsics for most common layouts. Doing this, the following items should be taken into account:

in place transpose should be disabled for row_major / column_major. In order to keep consistency, I think we could remove these functions from the xcontainer interface and add a transpose view. This would break backward compatibility.
the reshape(const shape_type&, const strides_type&) overload should be disabled for row_major / column_major.

Implemented in #255.


Synchronous Connection Pool not working

Bug Report

Current Behavior

Stack trace
/Users/brivera3/src/perf-test-data-service/app/src/main/java/com/intuit/v4/devtest/perftestdataserviceapp/util/LettuceConnection.java:44:21
java: no suitable method found for createGenericObjectPool(()->client[...]ect(),org.apache.commons.pool2.impl.GenericObjectPoolConfig)
    method io.lettuce.core.support.ConnectionPoolSupport.<T>createGenericObjectPool(java.util.function.Supplier<T>,org.apache.commons.pool2.impl.GenericObjectPoolConfig<T>) is not applicable
      (cannot infer type-variable(s) K,V
        (actual and formal argument lists differ in length))
    method io.lettuce.core.support.ConnectionPoolSupport.<T>createGenericObjectPool(java.util.function.Supplier<T>,org.apache.commons.pool2.impl.GenericObjectPoolConfig<T>,boolean) is not applicable
      (cannot infer type-variable(s) T
        (actual and formal argument lists differ in length))

Input Code

Input Code
package com.intuit.v4.devtest.perftestdataserviceapp.util;

import io.lettuce.core.RedisClient;
import io.lettuce.core.RedisURI;
import io.lettuce.core.api.StatefulRedisConnection;
import io.lettuce.core.api.sync.RedisCommands;
import io.lettuce.core.support.ConnectionPoolSupport;
import org.apache.commons.pool2.impl.GenericObjectPool;
import org.apache.commons.pool2.impl.GenericObjectPoolConfig;
import org.springframework.beans.factory.annotation.Value;
import org.springframework.stereotype.Component;

import javax.annotation.PostConstruct;
import javax.annotation.PreDestroy;

/**
 * This is a wrapper class around Lettuce library.
 */
@Component
public class LettuceConnection {

    @Value("${redis.server.host}")
    private String redisHost;

    @Value("${redis.server.port}")
    private int redisPort;

    private RedisClient client = null;

    private GenericObjectPool<StatefulRedisConnection<String, String>> pool;

    LettuceConnection() {

    }

    @PostConstruct
    private void init() throws Exception {
        try {
            RedisURI redisUri = RedisURI.Builder.redis(redisHost)
                    .withPort(redisPort)
                    .build();
            client = RedisClient.create(redisUri);
            pool = ConnectionPoolSupport
                    .createGenericObjectPool(() -> client.connect(), new GenericObjectPoolConfig());
        } catch(Exception e) {
            e.printStackTrace();
            throw e;
        }
    }

    public StatefulRedisConnection<String, String> getRedisConnection() throws Exception {
        try (StatefulRedisConnection<String, String> connection = pool.borrowObject()) {
            return connection;
        }
    }

    public RedisCommands<String, String> getRedisCommands() throws Exception {
        try (StatefulRedisConnection<String, String> connection = pool.borrowObject()) {
            return connection.sync();
        }
    }

    @PreDestroy
    public void close() {
        if(client != null) {
            pool.close();
            client.shutdown();
            System.out.println("client shutdown");
        }
    }

}

Expected behavior/code
Its supposed to instantiate the connection pool so that the other functions can acquire and release a connection upon need.
Environment

Lettuce version(s): 6.1.4.RELEASE
Redis version: 6.2.5

Possible Solution

Additional context

Please check the method declarations vs. usage in your code. We have quite a few tests for connection pooling that should give you an idea how the code is supposed to look like.