update node-hid version to support electron 21

Hi MaierN,
I assume version 2.1.2 is the latest version of node-hid in your commit.
If yes, please update your commit to remove the ^ prefix before the version number as I like to distribute this library with a fixed version of node-hid that has been tested.
Speaking of testing, I'll need to run the test suite here locally with my G29 wheel before I approve this merge also.

Testing succesful. ^_^


Inventory Adjustments - Make Both Editable & Deletable

All Inventory adjustment transactions should be fully editable, as well as deletable. We already made an adjustment that contained and error, and now are unable to remove it (eek).

Expected behavior
I would expect that any transaction/entry into Diaper Base can be removed/changed.
Actual behavior
Currently, you can not edit inventory adjustment, and there is no way to delete them.
Steps to reproduce the problem
Try editing an existing inventory adjustment.
Computer Information (Only fill this out if you are a diaper bank)
MacOS High Sierra
10.13.4
Browser: Chrome

For right now, we're going to not add this feature.
Should the need behind this not be addressed with other features or workflows, we can revisit later.


Pipeline tests

Added test cases for classification in tests/test_eval/cases/classification.py
Added classification cases to push_forward test in tests/test_eval/test_push_forward.py
Minor changes in  push_forward test cases for regression in tests/test_eval/cases/regression.py

Merged into a different pull request.


subfolder support for pages

When a page is located in subfolder, the page can't be found.
Can you tell how to support loading page in subfolder, both js and css?

Working on it, give me a couple hours

PR open @bloody-ux

merged


Spine validation

Adds a function isValidSpine which takes GenericSignature and
GenericSpine and checks that the spine conforms to the signature.

:+1: Looks good!

Can you run pulp docs please?

:+1: Thanks!


What's the point of using Encoding UTF8 in Export-csv?

I'm wondering because it ruins characters on a Swedish system. I had to change it to unicode to preserve characters.
So if there's no reason to force use of UTF8 I would suggest having an Encoding parameter passed to Export-Csv.
Edit: To clarify, the broken characters appeared when trying to import the file into Excel, didn't matter which encoding I selected for the import the characters were still broken. Only tried on Microsoft Excel v15 for Macintosh.

I guess this can be closed then thanks to #8


Specify correct folders for isort and flake8 in tox.ini

Also a bunch of flake8 and isort fixes.

:heart_eyes:

@cXhristian Did someone watch It's always sunny in philadelphia? Eletric Boogaloo, haha.


Keyboard Hiding SCLAlertView

First of all, Thank you for this great project.
One little problem: After adding textfield to the alert view, whenever keyboard pops up, it's blocking everything below the textview.
Any advice?

Same issue for me! Had to add some code to the classe to fix this. It's not the best solution but works for now
those two nofification in viewdidLoad
    NSNotificationCenter.defaultCenter().addObserver(self, selector: Selector("keyboardWillShow:"), name:UIKeyboardWillShowNotification, object: nil);
    NSNotificationCenter.defaultCenter().addObserver(self, selector: Selector("keyboardWillHide:"), name:UIKeyboardWillHideNotification, object: nil);

and those 2 functions
  func keyboardWillShow(notification: NSNotification) {
    UIView.animateWithDuration(0.1, animations: { () -> Void in
        let pointMoveY = abs(self.contentView.frame.origin.y - (self.contentView.frame.minY + self.contentView.frame.height/2))
        self.contentView.frame.origin = CGPointMake(self.contentView.frame.origin.x, (self.contentView.frame.origin.y - self.contentView.frame.height/2))
        self.circleBG.frame.origin = CGPointMake(self.circleBG.frame.origin.x, self.circleBG.frame.origin.y - pointMoveY)
    })

}

func keyboardWillHide(notification: NSNotification) {
    UIView.animateWithDuration(0.1, animations: { () -> Void in
        let pointMoveY = abs(self.contentView.frame.origin.y - (self.contentView.frame.origin.y + self.contentView.frame.height/2))
        self.contentView.frame.origin = CGPointMake(self.contentView.frame.origin.x, (self.contentView.frame.origin.y + self.contentView.frame.height/2))
        self.circleBG.frame.origin = CGPointMake(self.circleBG.frame.origin.x, self.circleBG.frame.origin.y + pointMoveY)

    })
}

Solution from Wieseljonas has one bug: The event keyboardWillShow even fires if the keyboard is visible and the autosuggestion is used... this makes the view fading out of the screen...

@ogezue @wieseljonas @devilbags see my pullreq. https://github.com/vikmeup/SCLAlertView-Swift/pull/103

@orangecoding I just tested your pull request, and it works like a charm :) Thanks !

Latest changes pushed to cocopoads. Version: 0.4.3

@PhilippeBoisney no worries ;)


utf8 problem

Hi, i want insert special characters like "ñ" in the json file, i try insert in unicode (\u00F1), but dont works,
i think that in Globals.gd in
file.open(file_name, file.READ)
var file_content = parse_json(file.get_as_text())
how i can convert in utf8 "file_content"?
many thanks

Thank so much too for your work, this dialog system is many flexible for all that i need for my proyects, and i think too that is the most powerfull system godot dialog actually.

Aw, thank you so much! It always makes my day seeing your comments. Best of luck to you on your projects.


Is a proxy server supported?

The app never synchonises my notes, I get the message "Working offline. Changes will sync when connected" although Firefox is obviously online.
My first guess is that there is a problem with my proxy server---is this allowed for in the app?
Thanks

Hmm interesting - I think that the proxy is very likely the issue, I've never tested with a proxy. Sync is done with an AJAX POST to https://notestack.me/sn.php. Try to see where your request is going from firebug.
Where do you have the proxy setup? Is it in firefox, an external app, or in the OS?

Did you change something, because it works now? Thanks.

haha nope, didn't change anything. Glad it works though - Britt


[CO] Line Labels

Hello,
in #16 there was some discussion about inconvenience regarding line labels. I.e. line labels are defined in line.prop_dict as a key/value pair (e.g. "boundary": "rotor_yoke_side").
Now line labels aren't only needed if the lines are boundaries, but also for applying mesh setting, maybe FEA result evaluation etc.
So the key/value pair seem inappropiate. Instead I would suggest to add a label prop. to Line (as in Surface). Further I would rename prop_dict to prop_list that contain all possible properties, e.g. BOUNDARY_PROP_LAB == "Boundary" .
What do you think.
Best regards, Sebastian

Hello,
We used to have line.label but we removed it in #412 (discussion in #344). I don't have time to read the related issue again but I remember that we had long discussions before moving to the prop_dict. This modification was done one year ago, a lot have changed in pyleecan since and we have a better idea of what the prop_dict contains so I agree that we need to take time to improve the current system and write a proper documentation.
If I remember well, we wanted to set too many information in the line label for several purpose so we replaced the label by the prop_dict but maybe a better solution would have been to have label and prop_dict since the label is the most important property to set (and most of the time the only property set). But again different coupling may need different label so it may be tricky to define a generic rule on which label to set, how and why.
Another solution could be to rename BOUNDARY_PROP_LAB  into "LABEL_PROP_LAB" to have a more generic key (or add it as a new key).
I not sure if I understood your point on dict/list and "contain all possible properties" but I don't think that all the lines should have all the property set, since it will create a lot of empty dict or list to handle at each line manipulation. On our side, we have some internal property that we want to set only on few lines, it would be hard to make sure that they are added to all the dict/list (for no purpose other than being there). However, we need to have a better understanding of the possible labels and why they are used for.
Thank you for opening this discussion, it's an important topic to ease current and future coupling :)
Best regards,
Pierre

Hello Pierre,
thank you for your fast answer. I also can't remember the details of this recent discussion.
The point of the list was to store tags (e.g. BOUNDARY_PROP, your_other_internal_prop, ...) for special lines. All other lines will have a line.prop_list = None (or empty list).
But if you got other internal properties, I guess you also use the respective dict value, i.e. line.prop_dict = {internal_prop: prop_value}?! Then a list of properties will be inappropiate.
My other point is that all lines that need some label (e.g. for meshing) need to have the BOUNDARY_PROP_LAB even if they are no boundaries. On the other hand if we use a LABEL_PROP_LAB we could also reintroduce Line.label property.
In general, I think that it is sufficent to have labels that are unique per surface, so every model/coupling can create an overall unique name from surface and line name. Further, I think that 'build_geometry' should include as less information about the models as possible. Otherwise with each new simulation/coupling etc. we need to update all the line labels of all holes/slots etc.
Meanwhile I studied some more code and it seems I'm fine with BOUNDARY_PROP or some other workaround for the moment.
I will post my findings here when I'm finished my work and we will see.
Best regards, Sebastian


Intercepting and Processing SurfaceTexture Video Frames

I am working with Mediapipe and using Android Camera 1. I have set the camera's previewTexture to use the SurfaceTexture as such
  mCamera.setPreviewTexture(mPreviewFrameTexture); // mPreviewFrameTexture is my SUrfaceTexture

Challenge I am having now is I will like to crop this preview frame before it gets sent to the mediapipe graph.
"A SurfaceTexture may also be used in place of a SurfaceHolder when specifying the output destination of the older Camera API. Doing so will cause all the frames from the image stream to be sent to the SurfaceTexture object rather than to the device's display. "
from https://developer.android.com/reference/android/graphics/SurfaceTexture
What I intend to achieve
On a tablet where the camera is positioned on either side, the center of the preview is relative and does not represent to the actual position of the hand relative to the device, this would have been perfect had the camera on the device position at the center.
So I will like to crop out the other half of the preview not tangential to the device surface.
Is there a way I can intercept this and crop the image before it gets sent to the mediapipe graph please for further processing? Please can someone help me?

Just incase anyone comes across this issue, I used afsaredrisy's sample project to understand what I needed to do.
https://github.com/afsaredrisy/MediapipeHandtracking_GPU_Bitmap_Input/tree/master/handtrackinggpu


Out of memory error due to default Row limit

Hi,
I encountered an out of memory error when trying to open a table with rather big rows (including binaries).
PyCharm and IntelliJ, both 2017. Increasing the memory settings might have helped, but it is slow, too.
So reducing the default Row limit to 10 and rebuilding the plugin solved the problem for me.
Perhaps it is worth considering adding the Row limit to the settings.
Btw.: The plugin is great.

Hi,
I will add this in the plugin settings as default row limit

Hi,
you will find a SNAPSHOT here


Add support for custom types?

I'm not too sure about this yet, but maybe FastMatrices should allow storage of there types. This way, we can still easily transpose and print an array of that type easily, albeit maybe not on the GPU.

This will, unfortunately, be limited to structs and primitives on the GPU. This is due to the limitations of how class memory is structured, or so I've been told.

I lied, for now it's going to be structs and primitives for all MatrixOperators. This is due to each FastMatrix having a MemoryBuffer2D, which doesn't support classes. However, this functionality can be added in a separate class, maybe something like UnbufferedFastMatrix.


Update EventsIngress API docs for HIPAA compliance

Update I/O Events EventsIngress API docs to describe HIPAA compliance support.
Description
Describe the new optional header x-event-phidata to be used to indicate that an event payload contains PHI data.
Describes how producers can use the trace-id to retrieve audit logs from I/O Events regarding the data of a patient.
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Please also update the broken support link.


Temperature reversed. Pressure Units

The big temperature gauge is showing the ground temp not the ambient temp.
Possibly make a units switch to choose millibars or inches

Nice catch.  Thanks.


Missing :hover highlight on dropdown in inverted menu

Example (from docs, just added inverted): http://jsfiddle.net/md62o9vc/
Expected result: Hovering over items on open dropdown highlights the item.
Actual result: Hovering over items on open dropdown changes nothing visible.
Browser: Chrome 40.0.2214.111 (64-bit)

Duplicate #1281


Error installing

Hi,
I tried to install tonight and got the following:
The following properties have been set: Property: [AdminUser] = true {boolean} Property: [InstallMode] = HomeSite {string} Property: [NTProductType] = 1 {int} Property: [ProcessorArchitecture] = AMD64 {string} Property: [VersionNT] = 10.0.0 {version} Running checks for package 'Microsoft .NET Framework 4.7 (x86 and x64)', phase BuildList Reading value 'Release' of registry key 'HKLM\Software\Microsoft\NET Framework Setup\NDP\v4\Full' Read integer value 528372 Setting value '528372 {int}' for property 'DotNet47Full_Release' Reading value 'v4' of registry key 'HKLM\SOFTWARE\Microsoft\NET Framework Setup\OS Integration' Read integer value 1 Setting value '1 {int}' for property 'DotNet47Full_OSIntegrated' The following properties have been set for package 'Microsoft .NET Framework 4.7 (x86 and x64)': Property: [DotNet47Full_OSIntegrated] = 1 {int} Property: [DotNet47Full_Release] = 528372 {int} Running checks for command 'DotNetFX47\NDP47-KB3186497-x86-x64-AllOS-ENU.exe' Result of running operator 'ValueEqualTo' on property 'InstallMode' and value 'HomeSite': true Result of checks for command 'DotNetFX47\NDP47-KB3186497-x86-x64-AllOS-ENU.exe' is 'Bypass' Running checks for command 'DotNetFX47\NDP47-KB3186500-Web.exe' Result of running operator 'ValueNotEqualTo' on property 'InstallMode' and value 'HomeSite': false Result of running operator 'ValueGreaterThanEqualTo' on property 'DotNet47Full_Release' and value '460798': true Result of checks for command 'DotNetFX47\NDP47-KB3186500-Web.exe' is 'Bypass' 'Microsoft .NET Framework 4.7 (x86 and x64)' RunCheck result: No Install Needed Launching Application. URLDownloadToCacheFile failed with HRESULT '-2146697208' Error: An error occurred trying to download 'https://raw.githubusercontent.com/coldino/Larkator-release/master/publish/LarkatorGUI.application'.

It looks like your system is blocking the installer's access to the install files. You might need to allow it through your virus/malware detector.
In particular it is failing to reach this link, but this is just the first of the files it needs.

Given we had a couple of reports of this error recently I've updated the installer and its required .Net Framework to v4.8.
Try installing the latest release using (setup.exe)[https://github.com/coldino/Larkator-release/raw/master/publish/setup.exe], and if that doesn't work see if you can use the (LarkatorGUI.application)[https://github.com/coldino/Larkator-release/raw/master/publish/LarkatorGUI.application] file directly.


Create event page for All Things Open

Related to this issue: https://github.com/newrelic/developer-website/issues/659
Description
Overview landing page for our sponsorship of all things open, gold sponsor
Reviewer Notes
Need this image of allthingsopen.png to work: https://github.com/newrelic/developer-website/pull/688
Related Issue(s) / Ticket(s)
https://github.com/newrelic/developer-website/issues/659

issue

Screenshot(s)
If relevant, add screenshots here.

I'm going to close this PR and start from scratch.


Save heap memory by opening two socket connections using the same certificate?

Hi,
I am trying to use two HTTPS connections simultaneously on the ESP8266, both of them with persistent HTTPS connection (so the sockets stay connected). I saw that other people also mentioned the large heap memory footprint and ran into that problem myself too.
Before the first socket is opened, I have 31,648 bytes and it drops to 20,872. The second socket connection drops the free heap space further to 9792 bytes. So about 11K per socket connection.
I was wondering what is the reason for this relatively big footprint? Is it possible to skip the verify() of the fingerprint for one of the servers (for which I don't need authentication) and save memory? Another option I am considering - if the main memory allocation is due to the certificate verification, is it possible to have both sockets use the same certificate verification, so that I would be allocating only 11K instead of 22K?
One thing I've noticed is that the call to ssl_ctx_new() is performed only once, and that the massive memory allocation happens on the two calls to ssl_client_new() - about 8Kb.
Thanks

axTLS heap usage consists of the following components:

TLS fragment buffer. Default size is about 6kB, it is grown dynamically up to the maximum TLS fragment size of 16kB, if the server doesn't support fragment length negotiation extension (RFC 6066).
Certificates received from the server. As an optimization, these are purged as soon as TLS fragment buffer needs to be resized (in HTTPS scenario, this usually happens when the server starts sending HTTP response). Size: depends on the server's certificate chain.
CA certificates, if provided by the client. Size: depends on CA certificate, usually about 1-2kB.
Bigint context for RSA. This is kept around throughout the connection. Size: not sure, but can be determined by instrumenting bigint functions.
The rest of TLS session state: keys, nonces, etc.
Large amount of bigint values needed during key exchange and certificate verification. These are freed after the handshake, but may cause an out-of-memory condition during the handshake. Size: depends on the RSA key length.

Is it possible to skip the verify() of the fingerprint for one of the servers (for which I don't need authentication) and save memory?

As far as i understand, axTLS still loads the whole certificate chain into memory, as it is not aware of the method which will later be used for certificate verification. This is done in process_certificate (tls1.c).

is it possible to have both sockets use the same certificate verification, so that I would be allocating only 11K instead of 22K?

Each concurrent TLS connection needs to do handshake and validate certificates separately. However, if a session has already been established, new connection can perform abbreviated handshake. Unfortunately, axTLS doesn't have support for this.
To summarize, without significant changes to axTLS, you can get biggest memory saving effect if you enable fragment length negotiation extension on the server side.

Each concurrent TLS connection needs to do handshake and validate certificates separately. However, if a session has already been established, new connection can perform abbreviated handshake. Unfortunately, axTLS doesn't have support for this.

Actually axTLS supports abbreviated handshake or SSL resumption as is the official term. What you need to do is keep the SSL session id from a previous session and then use it in a new SSL to the same server providing the SSL session id in the SSL context. We have this implemented in Sming and it is working like charm.
There is even better way to decrease the SSL overhead - inform the server to keep the TCP connection open and then make another request using the same SSL. If you are interested how this can be implemented take a look at the HttpClient in Sming.

Thanks @slaff, looks really useful! Session ID support should be added to Arduino wrapper as well.
I think the equivalent of HttpClient in Arduino does support TCP connection reuse, but the OP was asking about having two connections in parallel (not sure whether they use same server or not).


Results endpoint should return partial data when possible

For example: https://run.biosimulations.dev/simulations/61a5440b4a8b188bb74c1a51#tab=log
The run failed. so it is likey that some or all data was not output.
Trying to get the results (without data) works fine:
https://api.biosimulations.dev/results/61a5440b4a8b188bb74c1a51?includeData=false
However, trying to get the data with results causes a 500 error:
https://api.biosimulations.dev/results/61a5440b4a8b188bb74c1a51?includeData=true
This is by design since the dimensions of that data were probably inconsistent and so the processing throws an error. However, it would be more user friendly to

display a more informative message
return the data that is available even if it incomplete

@jonrkarr I am not sure how we want to handle point 2. Should this be limited to only tasks that have completed or all data?

Multiple things have to be done to properly deal with errors.
This sounds related to the NaN issue with the HSDS. The underlying issue is that we're not properly retrieving NaN and +/-Inf from the HSDS. Currently, the HSDS truncates the data (presumably at the first NaN or Inf). This makes it hard to understand what the data was intended to be. Reshaping the response from the HSDS doesn't fix the problem because data has already been lost, as well as its original shape. I've added a step to raise an error when this truncation happens because when it happens the API doesn't receive enough information to properly interpret the data.
I think the solution is to properly deal with NaN and Inf. At the very least, they need to be converted to null (which would confuse NaN and Inf). Once handled, the error mentioned above would never been raised.
In addition, we need to see what Plotly and Vega do with NaN/null values, and if necessary truncate the data sent to them.

This sounds redundant with #2741.


Composite templater action

Migrate the the bash steps of the updater to a composite action

https://github.com/smartlyio/test-vulcan-js-library/runs/1430066510


Removes bphash

PR Type

[ ] Breaking change
[ ] Feature
[x] Patch

Brief Description

We no longer require objects to be hashable. This PR removes the infrastructure for hashing,
and testing that hashing, from PluginPlay.
Not In Scope

PR Checklist

[x] Documentation (comments, API, user, design, and developer as applicable)

API
User
Design
Developer

[x] Testing (unit, integration, performance, as applicable)

Unit
Integration
Performance

[x] Code adheres to organization standards

C++
Python
Doxygen Conventions
reST Conventions

I think we should be able to merge this without needing to merge https://github.com/NWChemEx-Project/ParallelZone/pull/95.

🚀 [bumpr] Bumped! New version:v0.0.5 Changes:v0.0.4...v0.0.5


Why did you configure the first densely connected layer with GraphConvLayer?

hello :) I ran the code you gave but I would like to ask you why you constructed the first densely connected layer (layers[0]) with GraphConvLayer.
The other layers(layers[1], ..., layers[N-1]) were configured with MultiGraphConvLayer, right?
please let me know any mistake. Thank you !

Hi, you are right.
The reason is that the first densely connected layer takes the dependency tree as the input (we only have one graph). For the latter layer, we constructed multiple graphs based on the node representations. That is the reason why they were configured with MultiGraphConvLayer.


Style like ipfs.io

Minimum styles to make the blog look like the main IPFS site
Adds imagemin image optimization step

Blog home

Post

Also now has footer

woot woot!! Looks so much better.
@alanshaw could you also include a URL to the RSS feed at the top?

@diasdavid:

@alanshaw sweeeeeet! :D woot! :D


Suppress connection and content-length headers

Description
Suppress connection and content-length headers.
In http->gRPC invocation scenario, translating HTTP 1.1 request headers to gRPC request metadata, both headers are copied to GRPC metadata. However, both headers are applicable only for http 1.1. gRPC over Kestrel server validate these headers for http 2 connection unlike the other gRPC implementation. So we can see the problem only in .net gRPC service application.
This PR will added dapr- prefix to two headers to suppress them.
Reference:

https://github.com/aspnet/KestrelHttpServer/issues/175
https://github.com/improbable-eng/grpc-web/issues/568
https://github.com/envoyproxy/envoy/pull/2946/files

Issue reference
#2221
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

Falls into this category:  https://tools.ietf.org/html/rfc7540#section-<IP_ADDRESS>
  This means that an intermediary transforming an HTTP/1.x message to
   HTTP/2 will need to remove any header fields nominated by the
   Connection header field, along with the Connection header field
   itself.  Such intermediaries SHOULD also remove other connection-
   specific header fields, such as Keep-Alive, Proxy-Connection,
   Transfer-Encoding, and Upgrade, even if they are not nominated by the
   Connection header field.

We're not exactly translating HTTP 1.1-> 2.0 - but we are tunneling an individual HTTP message.

Falls into this category: https://tools.ietf.org/html/rfc7540#section-<IP_ADDRESS>
  This means that an intermediary transforming an HTTP/1.x message to
   HTTP/2 will need to remove any header fields nominated by the
   Connection header field, along with the Connection header field
   itself.  Such intermediaries SHOULD also remove other connection-
   specific header fields, such as Keep-Alive, Proxy-Connection,
   Transfer-Encoding, and Upgrade, even if they are not nominated by the
   Connection header field.

We're not exactly translating HTTP 1.1-> 2.0 - but we are tunneling an individual HTTP message.

Let me add these headers to the http/1.1 specific header list.
Keep-Alive, Proxy-Connection, Transfer-Encoding, and Upgrade


The code snippet of the event trigger doesn't explain the use of the TriggerAction properties

The article mentions:

Optionally expose properties that can be set in the XAML when the trigger is declared (such as Anchor, Scale, and Length in this example).

But the accompanying snippet code doesn't illustrate this, and also the following statement:

The properties exposed by the trigger action can be set in the XAML declaration as follows:

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

ID: a3ff3ffd-9177-76ff-ac6e-7cb439750203
Version Independent ID: 8e881672-c9ff-354c-fb22-97a8d1af2d72
Content: Xamarin.Forms Triggers - Xamarin
Content Source: docs/xamarin-forms/app-fundamentals/triggers.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @mshwf
Thanks for pointing this out. My guess is that at some point in the past this doc referenced the VisualElementPopTriggerAction class from the sample application, which does consume the Anchor, Scale, and Length properties. I originally set out to restore this example to the doc, but it's pretty abstract. Instead, I've updated the doc to remove references to the properties concerned.
This change will go live later.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2023-12-19)
Features

add mattermost nodes (0ecd146)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cedricziel/node-red-contrib-mattermost-app/releases/tag/v1.1.0 :sunflower:


Add Education section

Create a section listing educational background, including degrees and certifications.
Acceptance Criteria:
Education entries are properly formatted and easy to read

Issue resolved in aboutme commit forgot to close.


Remove unnecessary ()

coverage: 87.31% (+0.009%) from 87.301%
when pulling e8037eb74d941c95815c5d75be96cdbfbf6486c4 on hazendaz:pom-cleanup
into 260b30f3198e5ee8a1ea9168357ae4ac07486f02 on mybatis:master.


Is shared subscription over cluster available now ?

Environment

OS: ubuntu server
Erlang/OTP: ignore that, i use docker
EMQ: 2.3.3

Description
Recently I am testing the emqtt cluster.
I build the docker with the version 2.3.3 according to the project emq-docker (with tag v2.3.3)
I use etcd (docker) for service discovery and haproxy (docker) for load balancing.
I put all the things in a docker-compose file and the whole cluster works fine.
My goal is when I start 4 mqtt subscriber(mosquitto_sub) as they stick to the same topic and another publisher send msg to that topic, only one of the 4 subscriber can receive it.
I noticed that the emqtt has the feature called shared subscription. BUT it doesn't work over the cluster.
Also i read some old issues, I found one guy suggested trying the topic '$local/$share/group/topic' and it still got some problems.
NEED HELP

What's the implication of mixing local and shared subscription together, versus just shared subscription?


Where to start with this?

I just bumped into this searching CLion as I think I might try it soon (having never done C++ before) and would like to try OpenFrameworks (I've done a little Processing).
I was just wondering if you could suggest any decent 'starting points' with this in mind?
(opened and closed this as it's not really an issue)

You could try this book: http://www.amazon.com/Mastering-openFrameworks-Creative-Coding-Demystified/dp/1849518041


Updated truncate() and empty_table() to accept an array of fields

No ... emptying a table is something that you should be careful with, not do in batches.


chore(TDI-41234): fix vulnerabilities

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-41234
What is the new behavior?
Please check if the PR fulfills these requirements

[ ] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...


Added 🐕 to arf+!

Fixed all issues brought up about previous pull request :)

would you mind changing the emoji to arf:446677431160668161


temporary hotfix to ios getting new preauth

Hi,
This is a proposal of a temporary hotfix to the new auth problem. This is copied from Pandora, so all credits to them. Also uses their public api on this hotfix, as I can't offer full fix.
2023-07-13 更新

咱也不知道OpenAI出于什么心理，在登录时要求验证一个preauth_cookie参数，生成还挺麻烦。
不过不用担心，通过https://ai.fakeopen.com/auth/preauth就能拿到一个可用的preauth_cookie，把它附在登录链接上即可，大概长这样：https://auth0.openai.com/authorize?client_id=...&prompt=login&preauth_cookie=xxx
至于代码，我也已经更新在 Pandora v1.2.7 中了。

--
Example of how to get the proper unique preauth cookie:
curl -A "ChatGPT/1.2023.187 (iOS 16.5.1; iPad1 4,3; build 1744)" \
     -H "Content-Type: application/json" \
     -d '{
        "bundle_id": "com.openai.chat",
        "device_id": "7D1C7D8A-0707-0707-0707-166B8F380ABC",
        "request_flag": true,
        "device_token": "<ios device token>"
     }' https://ios.chat.openai.com/backend-api/preauth_devicecheck

and response cookie: _preauth_devicecheck will contain the appropriate preauth cookie.
I do not know how to generate the device token, someone hopefully has insights on this.
Also sidenote: the device_id in that request can be anything, long as its valid UUID - it's not checked with the device token.

template for more permanent fix here: https://github.com/zutto/OpenAIAuth/blob/preauth_fix/src/OpenAIAuth.py#L73-L99
just need to figure out if we can generate device tokens.. or if we need to just keep re-using someones personal one.

'device_token' is valid for only 24 hours

device_token is valid for only 24 hours

seems like this was true, my original device_token has expired and no longer works.
Pushed update to the go version as well.

Any idea where the device token comes from?

Any idea where the device token comes from?

Somewhere from the apple device, I don't understand this apple ecosystem so not 100% sure:
https://developer.apple.com/documentation/devicecheck/dcdevice/2902276-generatetoken

https://www.marinosoftware.com/insights/how-to-prevent-fraud-without-compromising-user-privacy-devicecheck-on-ios-11

Looks like we might have to fall back onto the standard authentication method which blocks out some VPS IP addresses

Probably good idea to explore alternatives yes.
I have code that can be used to generate these device tokens, but this only runs on mac unfortunately. I'll see if this can run on a docker or something...
https://github.com/zutto/token-generator

As far as I know, docker can run macos, but it depends on the kvm module

https://github.com/sickcodes/Docker-OSX

Probably good idea to explore alternatives yes. I have code that can be used to generate these device tokens, but this only runs on mac unfortunately. I'll see if this can run on a docker or something...
https://github.com/zutto/token-generator

Have you tested if it works?

Since it only needs to run every 24 hours, GitHub actions can do it (MacOS VM is available)

yes, it works.

I'll try to set up github actions, new to these..

well, that didn't work perfectly, but at least the workflow managed to build it.
https://github.com/zutto/token-generator/actions/runs/5568632683/jobs/10171410814
Run ./token-generator
  ./token-generator
  shell: /bin/bash -e {0}
unsupported device

good to know. i got a spare mac laying around but no WIFI so not sure how to do this. could have an endpoint for people to donate device tokens or something

Are you sure you don't need an iOS/iPad device to use it?

Hi,
Yes,
iOS 11.0+ iPadOS 11.0+ macOS 10.15+ Mac Catalyst 13.1+ tvOS 11.0+ watchOS 9.0+ visionOS 1.0+ Beta

Says the documentation, and it works on my mac mini and macbook.
Since this is propietary apple code, can't be sure of what they do, but I'm guessing they actually do check something about the hardware.

x§

It seems that my black apple does not support it, and it needs to read hardware information to be able to generate

Same here.
Might try to find an IOS version of this and get it running via TestFlight

not sure if openai enabled some actual checks on the device ID, but I seem to be getting 400-errors from the preauth_cookie API with the generator token today.

not sure if openai enabled some actual checks on the device token, but I seem to be getting 400-errors from the preauth_cookie API with the generator token today.

Maybe openai is watching us talk

not sure if openai enabled some actual checks on the device token, but I seem to be getting 400-errors from the preauth_cookie API with the generator token today.

Maybe openai is watching us talk

lol

Looks like we might have to fall back onto the standard authentication method which blocks out some VPS IP addresses

Backtracking a bit for alternative solutions, can you describe this issue a bit? Is this caused by cloudflare? And does this happen before, during or after visiting auth0 during the authentication process?
Been implementing standard authentication, which seems to work fine, but I don't have access to any CF blocked IP's myself.

I'm closing this PR in favour of https://github.com/acheong08/OpenAIAuth/pull/47.
Unless someone figures a way to generate these preauth cookies reliably, preferably without depending on external API's, this isn't going to progress.
I'm personally opting for the selenium route, seems quite reliable at this point.
We can re-open this path if the status quo changes in the future :)


[Containerapp] az containerapp create: Fix BadRequest Error about clientType with --bind

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Containerapp

Hi @wangzelin007
The CI failed with error:
Looks not relate to our change and extension, could you help to handle it?
ERROR: test_ref_doc_command-change (__main__.IndexRefDocs.test_ref_doc_command-change)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/connectionpool.py", line 467, in _make_request
    self._validate_conn(conn)
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/connectionpool.py", line 1096, in _validate_conn
    conn.connect()
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/connection.py", line 642, in connect
    sock_and_verified = _ssl_wrap_socket_and_match_hostname(
                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/connection.py", line 782, in _ssl_wrap_socket_and_match_hostname
    ssl_sock = ssl_wrap_socket(
               ^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/util/ssl_.py", line 470, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(sock, context, tls_in_tls, server_hostname)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/site-packages/urllib3/util/ssl_.py", line 514, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock, server_hostname=server_hostname)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/ssl.py", line 517, in wrap_socket
    return self.sslsocket_class._create(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/ssl.py", line 1108, in _create
    self.do_handshake()
  File "/opt/hostedtoolcache/Python/3.11.6/x64/lib/python3.11/ssl.py", line 1379, in do_handshake
    self._sslobj.do_handshake()
TimeoutError: [Errno 110] Connection timed out

Hi @zhoxing-ms
Could you help to review and merge this PR? Thanks!

@Greedygre Please resolve this conflict


Document required ports for OpenNebula services to set up firewalls

Author Name: Jaime Melis (@jmelis)
Original Redmine Issue: 3602, https://dev.opennebula.org/issues/3602
Original Date: 2015-02-17

None

The required ports to open is not documented in the Front-end Installation resulting in " Cannot contact oned, will retry... Error: libcurl failed to execute the HTTP POST transaction, explaining:  Connection timed out after 60062 milliseconds" after strictly following the tutorial if the firewall is set with policy DROP.
It worked for me after I opened ports 9869, 29876 and 2633.


Sync Releases to mattermost-server

Would it be possible to do releases in step with mattermost-server?
Currently there is a gap between mattermost-server (v5.35.0) and mattermost-docker (v5.33.3).

They normally keep the versions in sync. I guess it's paused due to this warning.

Thank you for the information.
I didn't think that would be a problem since the images are tagged with their versions. Even if you run into this you can revert to an older version of the image.

As a temporary solution to this: if you bump the MM_VERSION here and run docker-compose build and up again, the new image with the new release should be built and deployed, right?
But I agree, it would be great if they manage keep it in sync, let's give them a little more time.

As a temporary solution to this: if you bump the MM_VERSION here and run docker-compose build and up again, the new image with the new release should be built and deployed, right?

Good idea. Care to try and let us know if it works? ;-)

Probably not today, but I’d like to try it, I’ll let you know if it works

Good idea. Care to try and let us know if it works? ;-)

Well, I do it this way and it does work. Keep in mind that you need to use Postgres >= v10 to use recent versions. So you eihter need to upgrade the database before or start with a fresh installation and the desired version.

I tried updating the version in docker-compose to 5.35.0 and rebuilding. The app container starts fine, but the update to the web container to work as an unprivileged user doesn't start successfully for me. I get the following error messages, only way I could see to get the container to start again was to revert to the previous nginx base image (nginx:mainline-alpine).
linking plain config
2021/05/20 13:00:09 [emerg] 1#1: bind() to <IP_ADDRESS>:80 failed (13: Permission denied)
nginx: [emerg] bind() to <IP_ADDRESS>:80 failed (13: Permission denied)
ln: /etc/nginx/conf.d/mattermost.conf: File exists
sed: can't create temp file '/etc/nginx/conf.d/mattermost.confXXXXXX': Permission denied
sed: can't create temp file '/etc/nginx/conf.d/mattermost.confXXXXXX': Permission denied
I'm not sure why this happens, because the Dockerfile contains the command "chown -R nginx:nginx /etc/nginx/conf.d && " which should allow sed to create temp files. Any ideas where else I can look?

I just tested changing the server version to 5.35.1 and docker-compose build --no-cache then docker-compose down and docker-compose up -d and everything seems to be working fine :tada: .
I had previously upgraded Postgres to version 12 via a dump and restore of the data.

I previously upgraded Postgres to version 13.2 via the script at https://github.com/mattermost/mattermost-docker/issues/489#issuecomment-790277661
I'll try deleting the volumes for the web container and then running another docker-compose build, in case something weird is happening with the old volumes.

Yep that fixed my issue with the unprivileged nginx build.

I just tested changing the MM_VERSION to 5.35.1 and docker-compose build --no-cache then docker-compose down and docker-compose up -d and everything seems to be working fine 🎉 .
I had previously upgraded Postgres to version 12 via a dump and restore of the data.

I confirm that this works. Just upgraded from 5.33.1 to 5.35.1


Please change MakeFIle rules to allow linux build in a windows machine too

faced a lot if issues wgen trying to build a non VISUAL STUDIO project in windows machine ,

I don't do cross compiling from Windows to Linux. If that's something you want to see in this project, feel free to provide a pull request.

Closing due to inactivity. If this is still important, feel free to provide a PR.


Attempting to run SST with vitest in GitLab pipeline fails during bundle of functions

I ran into an issue where SST and vitest were able to run locally, but when I set up a GitLab pipeline it would consistently fail while bundling the NodeJS functions for testing the Stacks.
TypeError: Cannot read properties of null (reading 'toString')
 ❯ Object.bundle node_modules/@serverless-stack/core/dist/runtime/handler/node.js:166:44
 ❯ Module.bundle node_modules/@serverless-stack/core/dist/runtime/handler/index.js:16:16
 ❯ new Function node_modules/@serverless-stack/resources/dist/Function.js:223:45

I was overriding process.env during the stack tests so I could have a consistent snapshot of the templates. This was causing the childProcess that was trying to spawn "node" by name for the bundling script to fail because "node" wasn't in the path. I'm not entirely sure why it was working locally and not in the container in the pipeline.
My solution for now is to use a spread operator to keep the rest of process.env intact in my tests and just override the variables needed. However it might be better to use process.execPath in the spawn calls for the bundling process, rather than calling "node"
https://github.com/serverless-stack/sst/blob/c19a9b30ab65508e5e1bb43ad5651f293c1c6786/packages/core/src/runtime/handler/node.ts#L197

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Write a specification for the Selector debug panel feature

We need to write a detailed specification for the Selector Debug Panel (#3244) feature and discuss it among the team members.

Subgoals:

[ ] Move the compilator and compiled code to a separate process (https://github.com/DevExpress/testcafe/issues/3810)
[ ] Apply --inspect-brk and other V8 flags to the compilator process (https://github.com/DevExpress/testcafe/issues/3811)
[ ] Implement a synchronous Selector API (https://github.com/DevExpress/testcafe/issues/3812)


WIP: enable TestAuthNJwt e2e test for v2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: chxchx
Assign the PR to them by writing /assign @chxchx in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

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

@quanjielin: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
8f1f732877f901dbf259dce6ca3a6f37a91f9c41
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test istio-unit-tests

prow/e2e-bookInfoTests-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
Assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
tests/OWNERS

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

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-unit-tests.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
8f33c18895624dd391b5fea68ef2e93459caf1dc
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-unit-tests.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test istio-unit-tests

prow/e2e-simpleTests.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test e2e-simple

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-v1alpha3.sh
679a70aba51abb540d6cbcf6a56f4abd23921848
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-bookInfoTests-v1alpha3.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test e2e-bookInfo-envoyv2-v1alpha3

prow/istio-pilot-e2e.sh
da6f4be293f8d3c52d180a5f03740c4291ac8809
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@quanjielin: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
41d5f279ec67d93de7ebc5df5f12dfcfefc7088d
link
/test e2e-simple

prow/e2e-bookInfoTests-v1alpha3.sh
41d5f279ec67d93de7ebc5df5f12dfcfefc7088d
link
/test e2e-bookInfo-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Auth not resolving user for every request

Laravel Version: 10.0.3
PHP Version: 8.1.16
Database Driver & Version: Postgres & I don't know, using sail

Description:
When creating http kernel multiple times within same environment produces that auth user will not resolved anymore. First I used default kernel to make sure passport user can authanticate, but even token is diffrent user not changed for every request.
I solved this problem by making a seperate command to make internal request. Note that this command runs as different process.
$request = Request::create(route('landlord.authentication.tenant.account.show'));

$request->headers->add([
    'X-TENANT-ID' => $this->argument('tenant-id'),
    'Authorization' => sprintf('Bearer %s', $this->argument('token')),
    'Accept' => 'application/json',
]);

$response = app(Kernel::class)->handle($request);

if ($response->isSuccessful())
{
    return $this->line(json_encode(['user' => Arr::get(json_decode($response->getContent(), true), 'data.id')]));
}

$this->line(json_encode(['user' => null]));

Below code is the driver I'm using for the socket server. This driver works fine for first connection. But on the second connection it will not work at all, it will resolve first connections user all the time. Until restarting the server. First I tought maybe the problem is customized kernel and request for websocket but I don't think so. If it was because of customization, default kernel should worked fine.
// Using OpenSwoole.
Auth::viaRequest(
    'socket',
    fn (Request $request) => User::find($manager->findUserIdFromSocketId($request->fd()))
);

To sum it up, dispatching multiple requests within same environment/process does not resolves auth user for every request.
Steps To Reproduce:
Create fresh Laravel project. Add few users. Add test command, for each users create kernel and dispatch request. Return authenticated user in the response. You will see that first will work as expected. But on the other responses you will see that it will return same user again.
Note: I tried Facade::clearResolvedInstances();. In a few hours I will create repo with reproduced issue.

That should be expected. You can review Illuminate\Http\Request::createFrom() as example

@crynobone Thanks, I guess still having the same issue but I will look what I'm doing wrong.


Clearing Filepicker cache on logout

How do we go about clearing the Filepicker cache?
The use case would be that if someone logs out of our app and logs in under another account, they should not be logged in to any Cloud accounts in Filepicker.

In 3.8.1 you can use public Filepicker.clearSession(Context context) method. For now though it clears all cookies so I think it may affect you application (but since you want to use it on user's logout, it should works fine for you, right?).
I will look also for a way to clear only Filepicker's session cookie.

That should work for my current needs. Thanks!

Hello,
When using the clear session method. I get a method not found exception.
https://github.com/Ink/filepicker-android/blob/master/filepicker-library/src/io/filepicker/utils/SessionUtils.java#L41
In this method both methods were introduced at API 21

You're right. Seems to be finally fixed in version 3.8.2


about the initial template

hello!
I have a question about the initial template.
In P2M the mesh is always deformed from the same initial template--Ellipsoid,
Is it possible to be deformed from different template?

Yes, since we build graph locally, P2M++ can deform from other topology meshes. We have done experiments on voxel->marching cubes->mesh dataset, just replace input initial mesh for MDN.

是的，因为我们在本地构建图形，所以P2M ++可以从其他拓扑网格变形。我们已经在体素->多维数据集->网格数据集上进行了实验，只是将输入的初始网格替换为MDN。

excuse me ？  Can you tell me the meaning of “sample_adj” in pkl[‘sample_cheb_dense’]？ What is the meaning of the matrix ？  Why the Adjacency matrix has negative ？
Hope for your reply

Yes, since we build graph locally, P2M++ can deform from other topology meshes. We have done experiments on voxel->marching cubes->mesh dataset, just replace input initial mesh for MDN.

thx for your reply!

Hi @lkf1996 pkl['sample_cheb_dense'] are the laplacian matrix of level1-icosahedron, which is used for chebconv. It is not adjacency matrix, the calculation detail can be found in chebconv and gcn.


DLL crashes on Frontend state (Polaris-CH2)

Compiled with adding Actors to 14.30 SDK. When DLL is injected it freezes then crashes on Frontend state.

@Cubed Can u send the Polaris-CH2 your using?

Nvm was confirmed by Cyuubi that DLL never worked.

Then close the issue

@Cubed close the issue

@Cubed close the issue

Project's dead and we're not offering support for it.

Project's dead and we're not offering support for it.


Document the source for the official Docker library image

This is related to #34. Documents that we are the official Docker library alpine image source and some ideas of why you might use one or the other.

:+1:


[create_package_template] Improve template

Improve metapackage template in create_package_template.py by removing unnecessary and unused code. Note VM-Install-Shortcut already executes VM-Assert-Path $shortcut to verify that the shortcut has been created.
The CI would fail any package created with the previous template as the linter would complain about unused variables.

Looks good! Thanks for updating that. 🙂


[.NET Migration Guide Revamp] [Phase 1] Move .NET Upgrade Assistant tool content

For details on revamp see: https://github.com/dotnet/docs/issues/23561
New TOC: https://github.com/dotnet/docs/issues/23215

[x] .NET Upgrade Assistant tool (MOVE) Move as is to the following toc location...

name: Migration guide
items:

name: Overview
href: ../core/porting/index.md
name: General Information
items:

name: .NET Upgrade Assistant tool
items:

name: Overview
href: ../core/porting/upgrade-assistant-overview.md
name: Windows Presentation Foundation
href: ../core/porting/upgrade-assistant-wpf-framework.md
name: Windows Forms
href: ../core/porting/upgrade-assistant-winforms-framework.md
name: ASP.NET Core
href: ../core/porting/upgrade-assistant-aspnetmvc.md

Completed with #24630


Problem pairing a device from Owon/CasaIA

I'm struggling to get a PIR323 or a CMS323 from Owon.
It looks like the Confirm key in response to the Verify Key is not coming and so the device leave the network.
Here is a sniff .
Transaction key: 82315266be0b67626ac22107f2da3150
CMS323-Pairing-Failing-on-Sonoff.pcapng.zip
Owon stated 5s is the maximum time to get the Confirm Key. If not get it between the 1st verify key in the next 5 seconds the device will leave the network

@Koenkk did you already experienced this type of issue with battery based devices ?

I think this happens because other commands are queued before it. Everytime the device does a Data Request one of the queued commands is being send, if too many commands are queued before it the Confirm Key will come too late. I think the zigpy-znp adapter has to be changed here to wait a bit before sending all kind of commands to the device when it joins.

@puddly what do you think ? I have a similar issue (less often) with bellows as well .

I find it strange that the device announces itself on the network before verifying its key. It even completes the basic device interview and binding before verifying the key.  I can't find anything in the spec referencing the order in which this is supposed to happen but this seems wrong.
zigpy-znp has logic to wait a few seconds for devices to announce themselves after being notified of the key exchange by Z-Stack (since some devices do not communicate at all until they send the announce), but it is unexpected for a device to interleave the two processes.
I'm not sure how much of zigpy's initialization code you're using (if any) but can you try Koen's suggestion of adding a delay before initializing this device? Maybe you can match it by an IEEE address prefix.

I'll try that approach, and will let you know.

Ok. 6 seconds wait after sending the 1st command.
It looks like they start the Key Validation after the first command.
The strange thing is despite the behaviour looks strange, I didn't see this issue on Zigate .

@puddly I got a response from Owon why they are not doing it during the Association time. In fact they wait for the Node Descriptor response from the Coordinator to know which Zigbee stack they are running. if the Zigbee stack is 21 or 22 and above they will do the Key Verification to comply with ZB 3.0
The shame is that we do not see the Node Descriptor Request, nor cannot change the Node Descriptor response (fully embedded into the firmware)

Yes the device does the Node Descriptor request, and I think that from what I saw up to now, most of the devices does that.
Like the Aqara, which then stop working if the Node Descriptor Answer do not provide a Aqara/Xiaomi manufacturer code ;-)

Let's move this conversation back to https://github.com/zigpy/zigpy-znp/issues/149, it doesn't seem related to the Z-Stack firmware itself if it can be solved by adding delays within the radio library.


Phone Directory

A phone directory runs in your local browser and store the numbers you saved in your local storage of browser. You can track them everywhere in the world through your accesssing local browser.

Thank you for your contribution

No problem brother if you making another project and want collaborators or contributers then you can gmail me


[Improvement]: Add github workflow for handwritten connectors for staging and dev release

Suggested improvement
Add github workflow in each connector repo to release to staging-central

Closing this as it is already done


Why are the exports of the failureOf and successOf functions outside the main package?

Well, my question is whether there is a specific reason for this. Honestly, it's quite annoying to have to import twice. I know there are other options, but these functions seem, let's say, cleaner, which is why I'm asking.

Well, actually I just saw that it was removed. Why 🫠😕? Conflict?


[HUDI-3549] Removing "spark-avro" bundling from Hudi by default

Cleaning up
Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
Removing "spark-avro" bundling from Hudi by default.
Primary reason for that is -- "spark-avro" is impossible to be shaded correctly due to its dependencies on other Spark modules (like "spark-sql") which aren't bundled. Bundling just "spark-avro" creates exposure to issues when Spark versions become incompatible w/ each other. More details could be found in HUDI-3549
Brief change log
See above
Verify this pull request
This pull request is already covered by existing tests, such as (please describe tests).
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

@hudi-bot run azure

LGTM. lets test this patch w/ diff spark runtime versions (minor versions) to ensure we are good wrt diff runtime versions against hudi-spark3 bundles.
And I assume, with this patch, we should also be good to rename our spark3 bundles from hudi-spark3.2.1-bundle to hudi-spark3.2-bundles as we discussed.

@hudi-bot run azure

@nsivabalan correct

we should also be good to rename our spark3 bundles from hudi-spark3.2.1-bundle to hudi-spark3.2-bundles as we discussed.

@xushiyan as well. let's avoid renaming bundles . It does cause some busy work and thrashing for users, when they just want to pick up a new version. e.g if they had a HUDI_VERSION in their build/deploy scripts, now they need to all adjust per new naming.
Is the change to bundle names in this PR or a separate one? If so, can we just retain spark2, spark3, spark3.1 as bundle names? whats the plan

@hudi-bot run azure

@hudi-bot run azure

@hudi-bot run azure


uemacs: remove

This crashes with a buffer overflow upon running it. It has likely been
broken for some time. It has no maintainers. It is also unfree.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Could u open a backport to stable?

@worldofpeace Done.


addPath: doesn't POST for missing

please correct me if I misunderstood the addPath option.
was it intended to POST missing locale, namespace, option?
to me that would make a lot of sense

it posts the missing keys - but only if i18next option saveMissing is set to true on init

I can see it gets one language, then a fallback language, but can't see any post request.
both saveMissing=true and saveMissingTo=all props exist in init section

is it possible that your locale is en-US and you just have en in your whitelist?
with fallback en it would result in current = null and use just fallback en?
you can set nonExplicitWhitelist: true on init if you want allow en-US for whitelisted en.
else you might debug https://github.com/i18next/i18next/blob/master/src/Translator.js#L121 and check what ends in lngs.
there are a few tests checking this should work as expected: https://github.com/i18next/i18next/blob/master/test/backward/v1.11.1.compat.js#L1164 so i think it should work

closing...feel free to reopen if still valid issue.


Added Hyperlink to SQLite

HI - Added hyperlink to SQLite in below line
EF Core works with SQL Server/SQL Azure, SQLite, Azure Cosmos DB, MySQL, PostgreSQL, and many more databases through a database provider plugin model.
Thanks,
Ashish

Same resolution as #2688


HTTP: bodyBytes is invalid when response body has more than one chunk

@sandello is this available in the next release?

Check milestone field in right bar.


Adds open content management ui for admins and students

Description of the change
Adds the open content management UI, and fixes a couple things in the backend for visibility. Also moves toast props to common file for access in other places.

Related issues: Closes #409

Screenshot(s)
Admin View/Open Content Management:

Student View/Open Content:

Additional context
There are a few TO DO's I have left in there that are in regards to updating the UI once we have the scraper completed. It is making sure that we have the correct image URL so we can use it-- right now it is hard coded since all libraries are Kiwix. I also would love some feedback on how we should implement different colors for different providers. Should this be a color stored in the database? It will need to be a consideration once we add a new open content provider.
ALSO: I realize that the images look a little funky now. I am attempting to find a solution that makes images look better. Here is an alternative solution:

I don't think it is the best one either. Open to ideas!

Now also closes #410

tes comment

Ok, all comments should have been addressed!

If we look at library.kiwix.org, those thumbnails are just super small. Perhaps we should make the UI display them similarly, meaning off to the side instead of being front+center, which makes it look a little odd when they are so small, or blurry


bump minimum python version to 3.6 to solve FTBFS #4

bump minimum python version to 3.6 to solve FTBFS #4 due to mismatch with dependency (psutil)

Thank you!


add permission

Description
Content1 ...
Content2 ...
Content3 ...

This PR has:

[ ] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[ ] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Hi, please add some description of what this PR does, and better to create a jira issue


ArgoCD application getting out of sync after image update

When we create ArgoCD application, by default we add below ignore differences configuration
  ignoreDifferences:
    - group: '*'
      kind: '*'
      managedFieldsManagers:
        - rollouts-controller

These applications have "AutoImageUpdates" configuration enabled. So whenever there is image update, image updater goes and update the argocd application. And when it updates the application, it's removing
managedFieldsManagers:
- rollouts-controller
This happens every time whenever there is image update. This causes ArgoCD application to be out of sync.

@jannfis Do you think you could take a look at this?

any update

However, there is something odd with the latest code which we have to investigate
@jannfis - is this still relevant? Is there anything that I can help with to get this cut into a release? It seems to affect all of our applications annotated for the updater now that we've made greater use of the ignoreDifferences block that we need to address.


ssh not allowed via go.openly.dev

go get go.openly.dev/pointy
go: downloading go.openly.dev/pointy v1.3.0
go<EMAIL_ADDRESS>verifying module<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>404 Not Found
	server response:
	not found<EMAIL_ADDRESS>invalid version: git ls-remote -q origin in /tmp/gopath/pkg/mod/cache/vcs/3422f09a40f54841cd6a7a264fb1ce7e322a534e37822fe77c51adbcb92f7217: exit status 128:
		fatal: transport 'ssh' not allowed

Also, the README lists this breaking change as v2.0.0 though the breaking change was actually published as 1.3.0.

Met the same issue. @mwielbut

the vanity url issue should be fixed now


Add minimal docs for callHackage and friends

Specifically haskellSrc2nix, callHackage, and callHackageDirect. Other
functions should probably also be documented, but it's at least a start.
Only comments were added.

Motivation for this change
Uncertainty over how to use these functions
Things done
Comments added

Checklist doesn't apply:
- [ ] Tested using sandboxing ([nix.useSandbox](http://nixos.org/nixos/manual/options.html#opt-nix.useSandbox) on NixOS, or option `sandbox` in [`nix.conf`](http://nixos.org/nix/manual/#sec-conf-file) on non-NixOS)
- Built on platform(s)
   - [ ] NixOS
   - [ ] macOS
   - [ ] other Linux distributions
- [ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside [nixos/tests](https://github.com/NixOS/nixpkgs/blob/master/nixos/tests))
- [ ] Tested compilation of all pkgs that depend on this change using `nix-shell -p nix-review --run "nix-review wip"`
- [ ] Tested execution of all binary files (usually in `./result/bin/`)
- [ ] Determined the impact on package closure size (by running `nix path-info -S` before and after)
- [ ] Assured whether relevant documentation is up to date
- [ ] Fits [CONTRIBUTING.md](https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md).

Nice thanks!


Remove static and build-type suffixes

Normally, allegro binaries are built with static and debug suffixes as needed.  But that can make finding them annoying, especially since there's no standard FindAllegro.cmake.  So to simplify it, I removed those suffixes.  That info is provided with the build triplet and config anyway.

@JesseTG,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Wait, no, there's one more thing I have to do

@JesseTG,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Okay, time out, I clearly don't know what I'm doing as well as I think I do.


hypershift input changes

Description

Changes to Hypershift MC and HC install size,
Operator configuration to enable metrics-set
Updated workload to use new P75 and P90
Updated release names per playground, added branch in the cluster name

Fixes

Codecov Report

Merging #213 (9909eb8) into master (a22223a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #213   +/-   ##
=======================================
  Coverage   93.20%   93.21%
=======================================
  Files          24       24
  Lines        1001     1002    +1
=======================================
+ Hits          933      934    +1
  Misses         68       68

Flag
Coverage Δ

gha
93.21% <100.00%> (+<0.01%)
:arrow_up:

python-3.9
93.21% <100.00%> (+<0.01%)
:arrow_up:

unit
93.21% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

dags/openshift_nightlies/models/release.py
95.34% <100.00%> (+0.11%)
:arrow_up:

Continue to review full report at Codecov.

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


fix: Code highlight on --

Describe this PR
Enable double hyphens in code blocks
Changes
Check for double hyphens in code block line starts and exclude from highlighting
What issue does this fix?
Fixes #4138

Just tested and looks good now:


SAK-30510 Bug fixes to LTI additions reporting

Changes introduced in SAK-30601 broke the LTI additions reporting. But also adds a missing resource bundle that broke it.

Thanks @ottenhof


could I use html5 to connect  biggio server?

could I use html5 to connect  biggio server?
how to do that? to subscribe ,to join channel....

Hi @johnhuang669 using HTML5 and Javascript you can certainly connect to a bigq server.  If you check here you'll see a chat client built using Javascript.  I don't have an SDK yet for Javascript but that is doable.

is that node js ??
and I don't know how to subscribe ,,to join channel  in the sample that you mean....

Hi @johnhuang669 yes indeed.  You're correct, I don't have a channel example in there.  I'll work on getting a proper Javascript SDK including channel support over the weekend, rather than a self-contained program (which is what the link is).  Thanks!

Hi @johnhuang669 the JS client end is proving to be a bit more than I'm willing to take on.  I'm happy to provide you the message samples with framing to support channels if you'd like to give it a go.  Just re-open and let me know!  Thanks

Re-opening this, I think I might have found a solution to a blocker that made me give up hope :)

Hi @johnhuang669 I haven't forgotten about this, working on a proper SDK for Javascript.  Will take me a bit more time.

ok , thank your help !!

HI @johnhuang669  you can see progress here: https://github.com/bigqio/sdk-js  I have a long way to go, but ideally we'll be able to handle both channel and direct messaging with this.  It's not anywhere near ready yet but wanted you to see progress.  If you have any feedback please let me know.  Also eventually I'd like to be able to allow the caller to tie callbacks to events so they don't have to handle messages on their end.

Hi @johnhuang669 this is a little early but I've been able to run a series of successful tests.  I'd love your feedback: https://github.com/bigqio/sdk-js  I'll close the issue here - if you have any feedback on issues or enhancements please do open another issue on the sdk-js repo.  Thanks!


Add tap-dbt-artifacts to Hub

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/109
Originally created by @aaronsteers on 2021-08-09 16:25:43

As described in https://gitlab.com/meltano/hub/-/merge_requests/80

View 1 previous comment from the original issue on GitLab


Added Polynomial Rolling Hash for Strings

What does this PR do?
Adds a new Algorithm (String Hashing) to the Algorithms Directory
Test Plan
Add a Readme later , ask other contributors to test.
Related PRs and Issues
Yes, here it is : https://github.com/dubesar/Java-A-Z/issues/460
Have you read the Contributing Guidelines?
Yeah !

@mujtaba1747 please add the problem statement in the code about what the algorithm is.

@mujtaba1747 please add the problem statement in the code about what the algorithm is.

Okay, I'll add a small description.
It isn't based on any problem statement though..

@mujtaba1747 It's ok now.

Thank you for your contribution.


[Event Hubs] Support Development Emulator

Packages impacted by this PR
@azure/core-amqp & @azure/event-hubs
Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/28548
Describe the problem that is addressed by this PR
Supporting local development without hitting the service.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
The service is rolling out their own emulator and this PR adds support for connecting to it
Are there test cases added in this PR? (If not, why?)
Yes
Provide a list of related PRs (if any)
N/A
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists

[x] Added impacted package name to the issue description
[ ] Does this PR needs any fixes in the SDK Generator?** (If so, create an Issue in the Autorest/typescript repository and link it here)
[x] Added a changelog (if necessary)

API change check
APIView has identified API level changes in this PR and created following API reviews.
@azure/event-hubs

Thanks to @l0lawrence help with setting up the emulator and pointing to the right port, this PR has been tested with the emulator and it works. I converted it back to a draft for now because the emulator has not been released yet.


Use the CORE_SCHEMA when parsing entity YAML

By default (an despite the recommendations in the YAML spec), js-yaml uses a DEFAULT_SCHEMA which supports additional "special" types in the YAML document. This behavior differs from the yaml package (used by the core Backstage catalog processor/parser) which defaults to the CORE_SCHEMA instead.
This change adjusts the usage of js-yaml to more closely reflect the actual YAML processor used by Backstage so that (for instance) if an annotation has a value that looks like a date, the value is parsed as a string (like yaml does) instead of a Date object (which is the default behavior under the DEFAULT_SCHEMA when parsing with js-yaml).
:heavy_check_mark: Checklist

[x] Added tests for new functionality and regression tests for bug fixes
[x] Added changeset (run yarn changeset in the root)
[ ] Screenshots of before and after attached (for UI changes)
[ ] Added or updated documentation (if applicable)

Could I get a review?


Installation problem

Hi
i try to install the integration but the only message i get when selecting it after restart is this:

Config flow could not be loaded: {"message":"Invalid handler specified"}

Any ideas?
Core
2024.6.1
Supervisor
2024.06.0
Operating System
12.3
Frontend
20240605.0

2024-06-10 18:35:25.750 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading flow for integration dreame_vacuum: No module named 'imp'


arrow still shown when data-sort-method='none'

I think https://github.com/tristen/tablesort/blob/gh-pages/src/tablesort.js#L113 should move inside the if that follows. That solves the problem in my case.

Have you tried adding the no-sort class to the column ()?
That prevents the arrow from showing when hovering over, and also fixes the cursor (no pointer).

      foo
      bar
      baz

 ⋯

Have you tried adding the no-sort class to the column ()?
I tried its working. Thanks!


Failure to delete temp dir on build-ibmcloud-win2022-x64-1

Ref: https://ci.adoptium.net/job/build-scripts/job/jobs/job/release/job/jobs/job/jdk8u/job/jdk8u-release-windows-x64-temurin/16/console
18:20:46  ERROR: Failed to clean the workspace
18:20:46  jenkins.util.io.CompositeIOException: Unable to delete 'E:\Jenkins\workspace\temp'. Tried 3 times (of a maximum of 3) waiting 0.1 sec between attempts.
18:20:46  	at jenkins.util.io.PathRemover.forceRemoveDirectoryContents(PathRemover.java:87)

Having done some investigation, the e:\workspace\temp directory was being created with an unusual set of permissions, I've removed it manually, and recreated it... Im now running a build to try and determine if the build is corrupting the directory somehow, or if some other process is involved.

Test build running here: https://ci.adoptium.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-windows-x64-temurin/497/console

Having run 2 consecutive builds on the ibmcloud machine, without issues of it cleaning the workspace, Im inclined to switch the machine back online,  I assume a crash or other process, caused the permissions on that temp directory to be wrong(owned by administrator rather than jenkins), and removing the wrong one, and recreating with the correct permissions has resolved the issues in the build

Machine has been returned to service, following 2 x back to back builds without this problem recurring.


Scan file using events

Changes related to https://github.com/topcoder-platform/av-scanner-service/issues/8

@BadalZ This one is good to be merged in


Fix zalgo formatting.

Now that 1.38 has hit stable, we can finally update this rustfmt change.

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 56f8848a51e5db5214ec27bb41c7d99ebac21af8 has been approved by alexcrichton

:hourglass: Testing commit 56f8848a51e5db5214ec27bb41c7d99ebac21af8 with merge 74383b4fdb4f1d29152f64a552c93b7a241f265b...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 74383b4fdb4f1d29152f64a552c93b7a241f265b to master...


infra: Consistent lxml requirements

Update the dependency for lxml to be consistent among different packages; should fix https://github.com/langchain-ai/langchain/issues/19040

https://github.com/docugami/dgml-utils/pull/24 is merged and published as dgml-utils 0.3.3. Let me know if we can help resolve downstream issues. FYI @kenzie28.

why are there so many changes across other lock files?

Another question is whether we need to be pinning to a particular lxml major version. Would we be able to expand the range of major versions we're supporting to users don't get stuck unable to use langchain or some other package because of lxml version mismatch (lxml is a popular package)

@eyurtsev I agree, we can settle for >4.9,<6


Scheduled GitHub Actions Running on Forked Repositories

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
_test_doc_imports.yml and api_doc_build.yml
Error Message and Stack Trace (if applicable)
No response
Description
The scheduled GitHub Actions workflows _test_doc_imports.yml and api_doc_build.yml are also triggered in the forked repository, which is probably not the expected behavior.
System Info
Github Actions

good catch! fixed with a few other things as well


Cannot highstate after salt-the-salt-master

Applying the salt-the-salt-master went fine, but after doing so for any new highstate we now get:
2016-08-18 13:55:29,962 [salt.minion      ][WARNING ][20391] TypeError encountered executing state.highstate: get_pillar() got an unexpected keyword argument 'pillarenv'. See debug log for more info.

cc @edunham @aneeshusa

I restarted the salt-minion and salt-master services and it seems to be working.
The Salt services do NOT auto-restart because of tricky ordering and re-exec issues - it has to be done manually.

Nice work, thanks for figuring that out!


Add blog post on working with confidential data

link to resources already available for managing PII/BII that cannot be committed to GitHub.

Part of strategic plan: https://github.com/noaa-fisheries-integrated-toolbox/FIT-committee/issues/39

@k-doering-NOAA @erinsteiner-NOAA please check my (mostly minor) edits for accuracy; is there a link we could add to the MSA section?
Sensitive information and confidential data blog post
Is the text chunk....

And the exceptions for information collected by observers are: (A) as authorized by a fishery management plan or regulations under the authority of the North Pacific Council to allow disclosure to the public of weekly summary bycatch information identified by vessel or for haul-specific bycatch information without vessel identification;

...specific to the North Pacific Council?  I didn't want to edit bits that seemed like copied legalese, but thought this could maybe be made more generic if possible/applicable to other regions.

Completed Dec 2023


Computed property set via set or setProperties

In the code below
    const Foo = EmberObject.extend({
      first: 'rob',
      last: 'jackson',

      init() {
        this._super(...arguments);
        this.set('fullName', computed('first', 'last', function() {
          return `${this.first} ${this.last}`;
        }));
      }
    });

    let obj = Foo.create();
    let fullName = get(obj, 'fullName');

fullName is expected to be rob jackson however it resolves to
function dec( a, b, c ){
  [code]
}

Discussed this with @ssutar offline, the behavior was deprecated and removed in Ember 3.5, so we won't add support back for it in this polyfill


Allow user to exclude YAML file from processing

Fixes #995.
@lstocchi @rille111 would you be able to give this a bit of a real world test?  I've written unit tests but due to recent shenanigans I'm not in a position to test in a real setup right now.  Thanks and sorry for being so half-baked!

Adding the YAML master to verify this @evidolob

This works very good! Thx @itowlson.

@itowlson we should add a section in the README

Good catch @lstocchi - thanks and done!


State shape

We need to define the state shape.
Here is my proposal, feel free to add your suggestions : propositions.
{
  currentUser:{profile, roles, culture} ,// all the user related information
  routes, // all the router part of the state, see redux react router
  data, // documents ? all the informations from the entities of the app, all the document tree data , we have to find a better name it is not a good one
  layout: {headerCenter, headerExtended, barContentRight, barContentLeft, isExpended}, // all the layout components
  application, // all the application related informations,
  extensions: { commentCenter, notificationCenter, documentCenter, ...},  // all the extensions informations
  dx: {errorCenter, stateCenter} // all dx related infotmations, maybe it should be called devTools
  analytics, // all analytics related informations
  logs // all the application logs
}

@Tommass @Bernardstanislas It would be great to have your opinion on that!

data seems a bit... too general. Maybe dataset ? Or entityStore ? I don't know.
Plus, dx is too function-oriented. I'd rather call it centers
For the rest, it's all cool, except maybe for currentUser that I would better see as user

@Bernardstanislas

I find dataset good
For dx i agree but centers is a bit confusing with the extensions node, the idea of this node is to be removed when PRODUCTION mode is on. Maybe devTools.
user is too general but it's my opinion it can be fine

customData for all project specific reducers.


Several modules use mcrypt which was removed from PHP 7.2.x

mcrypt was removed in PHP 7.2.x, this project uses it in several places. These are mostly in libraries from other people and most of these have been updated as the code for each of these is quite old.
Spamchecker
src/bb-modules/Spamchecker/recaptchalib.php
This is likely as this library (recaptchalib) in this repo is from 2007 and is super out of date.
This has been updated since, but I think an update would require more than just a drop in replacement.
https://github.com/google/recaptcha
OpenSRS
src/bb-library/Registrar/includes/opensrs/openSRS_crypt.php uses mcrypt;
This is likely as the OpenSRS module has not been updated since at least 2013, as this was removed from the official OpenSRS toolkit.
There is likely a stopgap to make this work with newer versions of PHP (they did remove this dependency in 2013) -- but they have significantly updated their "PHP toolkit" since then, so in the future looking at migrating the module entirely to the new thing may be on the cards.
https://github.com/OpenSRS/osrs-toolkit-php

This is likely as this library (recaptchalib) in this repo is from 2007 and is super out of date.
This has been updated since, but I think an update would require more than just a drop in replacement.

Yeah, looking at it, I think it may be recaptcha v1, and there is v3 now
The OpenSRS says it needs PEAR which as far as I can tell is totally abandoned these days, do you know if that's a "requirement" for an easy install?

This is likely as this library (recaptchalib) in this repo is from 2007 and is super out of date.
This has been updated since, but I think an update would require more than just a drop in replacement.

Yeah, looking at it, I think it may be recaptcha v1, and there is v3 now
The OpenSRS says it needs PEAR which as far as I can tell is totally abandoned these days, do you know if that's a "requirement" for an easy install?

I don't think it is.

@BenNottelling May it be easier to remove the OpenSRS lib as I can't see them being used by any registrar adapters?
If we decide to add the adapter in the future, we can just add the library with composer.

Closed in favour of #1013.


how to run this demo with a RGBD camera?

A very nice work. Now I have successfully run this demo, but how to run the demo with a RGBDcamera and how can I configure it?

That depends on your camera driver. Override the fetch interface built upon your camera driver and use it with the main pipeline.

@ bianlongpeng have you solved your problem?

@bianlongpeng  have you solved your problem?


Hot fix to ensure read_set_parameters is not called for a Block type.…

… Based on the C exodusII code, it doesn't look like this is the appropriate method to call.

Addresses issue #161


Fix Bug #960 and enhance overlay images

here are the screenshots of chrome, firefox and Safari browsers

I don't see a watermark for the chrome screenshot. Also, please refrain from using text based browser name comparisons or creating special flows for certain browsers as this introduces inconsistency.

As discussed with @MahmoudElSharkawy in a call we will remove the if/else browser specific and keep it this way to fix the current issue and we will open an enhancement ticket to restructure the GIF builder.
The current issue is in the first image captured by the browser changed from browser to another in terms of dimensions so that we have different in size GIFs across all browsers as we depend on the first screenshot to build GIF.
Currently the GIF of chrome is a bit small but it was originally small as well before the bug but on Safari and firefox more than perfect
Here is the screenshot of chrome :-


Joystick resize

Feature update for issue #53.

I'm closing this since nothing has been done to fix it in a very long time.


Asura scans

Extension information
AsuraScans
Steps to reproduce
Whenever I try to open a manga from asura scans it starts loading then just shows a blank page and the page counter says 1/0

Steps to reproduce:

Open any manga on Asura scans
Click on any chapter

Refavouriting doesn't help and neither does opening the manga directly from the asurascans tab (or from favourites).
Miru version
1.8.1
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.

I found the problem. Due to the update of the website, the data is stored in the script, so the data needs to be retrieved. If you are in a hurry, please replace the following code directly, otherwise wait for the author to update
我找到了问题的所在，由于网站更新，导致数据存在了script中这样需要重新获取数据，如果你着急看的话，请你直接替换如下代码即可，否则就等作者更新
  async watch(url) {
    console.log(url + " url");

    const res = await this.request("", {
      headers: {
        "Miru-Url": "https://asuracomic.net/series/" + url,
        referer: "https://asuracomic.net/",
        origin: "https://asuracomic.net",
        "User-Agent":
          "Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.101 Safari/537.36",
      },
    });
    const regex = /<script>(.*?)\<\/script>/gs;
    const matches = res.match(regex);
    const pageRegex = /\\"pages\\":\[(.*?)\]/gs;
    const pageMatches = matches.join("").match(pageRegex);
    console.log(pageMatches + "pageMatches");
    const httpRegex = /https:\/\/[^\\]+/g;
    const httpMatches = pageMatches.join("").match(httpRegex);
    console.log(httpMatches.length + "httpMatches.length");
    console.log(httpMatches + "httpMatches");

    return {
      urls: httpMatches,
    };
  }


Regular locale naming like "en" breaks the i18n config

What version of this package are you using?
14.1.0
What operating system, Node.js, and npm version?
node-18.18.2 npm 9.8.1
What happened?
Here is a example where it's breaking: https://stackblitz.com/edit/stackblitz-starters-ydrtjx?file=package.json
What's happening is passing locales without a dash so ("en") and not ("en-US") is breaking the entire page. Is this intended behaviour? I've added this example in the next.config.js file:
const badI18nConfig = {
  locales: ['en', 'da'],
  defaultLocale: 'en',
};

const goodI18nConfig = {
  locales: ['en-US', 'da-US'],
  defaultLocale: 'en-US',
};

/** @type {import('next').NextConfig} */
const nextConfig = {
  i18n: badI18nConfig,
};

Don't forget to add /en to the route when accessing the page, wanted to keep the example simple so didn't add middleware for default redirection.
What did you expect to happen?
I can find no documentation regarding this, is this intended behavior?
Are you willing to submit a pull request to fix this bug?
Sure

I think it's better to open this issue in the next.js repo, since this has nothing to with this lib :)


Nephandus vs. Sticks

we have problem in past about Nephandus vs. Sticks,how much counters from stick need to burn due to nephandus text,so if you anounce stick 2 damage,but nephandus text reduce damage,how much counters stick burn-answer is 1,here is rule:  http://www.vekn.net/forum/rules-questions/75865-nephandus-vs-sticks#82119

Thanks ! I'll add this one quickly.

Link above allows you to see the corresponding change --^
It will be available in the Codex on the next release (later today or tomorrow I think)