Add support for updating env vars

PR paired with https://github.com/elyra-ai/pipeline-editor/pull/116
What changes were proposed in this pull request?
Add support for refreshing the env vars detected in a notebook or script in the properties for a node
The callback function is designed to support all future properties updates, not just env vars
Also includes Makefile  and doc updates for supporting parallel development of pipeline-editor
Fixes #710
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

Moving this out of WIP, see comments and discussion on the sister PR  elyra-ai/pipeline-editor#116

The CI failures look related to this PR given onPropertiesUpdateRequested is being added.
Error: src/PipelineEditorWidget.tsx(809,11): error TS2322: Type '{ ref: MutableRefObject<any>; nodes: { op: string; description: string; label: string; labelField: string; fileField: string; fileBased: boolean; extension: string; image: string; properties: { ...; }; }[]; ... 7 more ...; onPropertiesUpdateRequested: (args: any) => Promise<...>; }' is not assignable to type 'IntrinsicAttributes & Props & RefAttributes<unknown>'.
  Property 'onPropertiesUpdateRequested' does not exist on type 'IntrinsicAttributes & Props & RefAttributes<unknown>'.

The CI failures look related to this PR given onPropertiesUpdateRequested is being added.

Figured out that this is because the CI tests are pulling the release version of pipeline-editor not the sister PR code that this PR is dependent on. So CI will continue to fail until that PR is merged and released

I can't seem to make my local env to pickup the built package from pipeline-editor. I think we might be missing some sort of link location?

converted this back to draft because since it will need to wait for a release of pipeline-editor then update the versions prior to merging this. It only works now if you install the sister PR code locally.

@ajbozarth Master has been updated with the latest Pipeline Editor and this should be good to be rebased and merged.


Script path type support in editor

In the current editor implementation, you can export classes with custom script types and have the correct type validation be performed for both the inspector drag and drop behaviour and the scene tree selection popup. However, you can also export classes which are not defined by custom classes, but by script path constants, but these will not be validated by either of these editor options. This PR will check the script path of nodes to determine if they match. It also checks the inheritence of scripts.
It's debatable whether this is technically a bug or not, but it deviates from standard expecting operating behaviour given what is permissible in scripting, and is generally quite annoying if you don't like defining custom class names for every minor script.

It should show the script's icon (its own icon if set or base node type). Also instead of full path, which can be very long, I think it should be filename only.

Okay, I'll see if I can edit this to make the result a bit more presentable

There we go, it will now show the filename instead of the full path. It will also attempt to find the most appropiate icon, first checking the inheritence chain, and if it can't find anything, it will fall back to the icon of the base type.

You could've used EditorData's get_script_icon(). Sorry for not mentioning it earlier .-.

Okay, I'll update it again to use this function

Hmm, that function appears to be bugged, doesn't return an icon from scripts that don't have any custom defined icon. I guess I'll open up another PR for that once I've tested things.

Actually, 'fixing' this might or might not be desirable. Allowing that fuction to return the base script type changes how scripts are loaded in the inspector

We could either:
A. Keep it as is
B. Change the script icon return function to return the base types
C. Change it so that when the type is an unknown script type, use the same cog icon as in the inspector.

Maybe try _get_class_or_script_icon()?

@KoBeWi Okay, I've gone back to have a look at this. The solution is basically deriving the icon during the set_valid_types method and storing it in the metadata.
		// Attempt to get the correct name and icon for script path types
		String type_name = type;
		Ref<Texture2D> type_icon = EditorNode::get_singleton()->get_class_icon(type);

		// If we can't find a global class icon, try to find one for the script.
		if (type_icon.is_null()) {
			if (ResourceLoader::exists(type, "Script")) {
				Ref<Script> node_script = ResourceLoader::load(type);
				if (node_script.is_valid()) {
					type_name = type_name.get_file();
					type_icon = EditorNode::get_editor_data().get_script_icon(node_script);
					if (type_icon.is_null()) {
						type_icon = EditorNode::get_singleton()->get_class_icon(node_script->get_class_name());
					}
				}
			}
		}

This is a bit more simplified and also solves an issue with the earlier implementation that it wouldn't use custom icons unless we defined a global class. This time, it simply attempts to get a class icon, and if it fails, it will check if a script exists for that path. If it does, it will change the type name to just the name of the script and will use the get_script_icon method. Since this doesn't work for scripts which have don't have explicitly defined custom icons, I decided instead of trying to determine the base type, to just fall back on the script's class's own icon if a custom icon cannot be found. This is consistent with how scripts are displayed in the inspector and seems pretty consistent.

Updated some of the things pointed out.

Updated now.

The base node class icon is not displayed, generic Script icon is used:

Although that's probably a problem with get_object_icon().


Implement norm(MatrixProductOperator) and norm(other TN models)

norm(MPO) and norm(Composite TN) not implemented for now.
norm(MPS) is implemented.

The norm of any TensorNetwork{<:Quantum} state is already implemented in our code, but you are right that there were problems for the mpo. I have created a new branch with the fixes and am writing the PR now.


Baseline not working for bar charts

The baseline option tag is ignored or not rendered for bar charts when rendering in React.
As per:
http://plnkr.co/edit/senuycDzkMuoxQRl
Tried with and without viewwindow option tag, still doesn't start from below 0.

Upon further reading I think this is linked to this issue.
https://github.com/rakannimer/react-google-charts/issues/335
I can call the chart type ColumnChart and it does regard the baseline option, as well as other options I hadn't noticed weren't being passed. It just doesn't look pretty.
Were there any fixed for allowing material charts to use the options tag, or a different way to get options to the graph? The Material ones are much more pleasing to the eye.
Dan


Contact Page and Email Subscribe

Closes #11 and closes #31.
Notes:

The main addition here is the subscribe netlify function and its corresponding functionality in the footer and on the contact page
The netlify function accepts an email and posts it to the list of leads in Salesforce. This might need a bit of work, but I wasn't sure how to proceed with QA locally since the function needs to be recognized by netlify.
The footer form is setup to use formik/yup as requested, and contains success/error transitions for feedback.

@lougroshek added some more updates here – using netlify dev now as a develop command and everything seems to be working swimmingly – lmk where this should sit in our package.json commands, I'll be fixing up your other requested changes in the meantime! feel free to drop anything else here

@lougroshek some more updates, addressing the spinner, submission feedback/opacity, and validation behavior in the footer

@touchRED Much improved! Thank you!


make API non-throwing

motivation: cleaner API for users
changes:

make request initializer optional instead of throwing
http "sugar" methods (get, post, etc) no longer need to be throwing
fix schema logic to be non-case sensitive
made request object version, method and url immutable
made request object scheme and host internal
adjusted redirect ahndler implementation to stricter request immutabllity
adjust and add tests

Not sure if this was discussed already, but how about this: if the given URL is not suitable for firing off the request, instead of throwing or returning nil, create an appropriate Error and immediately fail the Promise/Task with this error? This way the API is very easy to use (no do/catch, no optional check) and in the rare case that for some reason the URL is incomplete it will be catched by the error handling that is anyhow necessary when doing network requests.

@t089 like in #56 ?

@artemredkin Ok, yes, sorry I got confused. Actually both PRs do this already right? The only remaining question is how to represent a validated URL as an internal type, if I understand correctly?

As @weissi pointed out to me, if you want to replace a function that is throws with one that returns nil, there's a keyword for that: try?. It is always possible to trivially transform a throwing function to a nil-returning one. However, it is not possible to get the throwing information back.
Right now we notify the user of what exactly we think they did wrong, and I think it's simply the wrong thing to do to hide this information.

this is an older proposal, the discussion is happening in #67


Graphql Subscriptions support.

Will be cool have Subscriptions with juniper harnessing the power of rust.

We have experimental support on master:
https://graphql-rust.github.io/juniper/master/advanced/subscriptions.html


Add node 8 install warning

Since only node 9+ is supported.

It's actually 10+ now.
But I don't think this is a particularly huge issue with the new signed release cycle.
CC @Fierent as owner of docs: the node 10 requirement is spelled out there right?


[stable/prometheus-operator] doesn't use multi-arch images

Describe the bug
When helm install stable/prometheus-operator 2 of the pods don't start due to exec format error which is an architecture incompatibility.
$ kubectl get pods --namespace monitoring
NAME                                                   READY   STATUS                  RESTARTS   AGE
monitoring-grafana-5c4dbdb56b-fwqmg                    0/2     Init:CrashLoopBackOff   10         29m
monitoring-kube-state-metrics-96f87d848-9ct98          0/1     CrashLoopBackOff        10         29m
monitoring-prometheus-node-exporter-gf44v              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-jbb7k              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-jlczk              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-nm62p              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-qvx8k              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-rfvkc              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-rk8js              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-rtzql              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-twh92              1/1     Running                 0          29m
monitoring-prometheus-node-exporter-xfmrt              1/1     Running                 0          29m
monitoring-prometheus-oper-operator-6bffc487c5-xqdzl   0/2     CrashLoopBackOff        20         29m

$ kubectl logs --namespace monitoring monitoring-prometheus-oper-operator-6bffc487c5-xqdzl --all-containers
standard_init_linux.go:211: exec user process caused "exec format error"
standard_init_linux.go:211: exec user process caused "exec format error"

$ kubectl logs --namespace monitoring monitoring-kube-state-metrics-96f87d848-9ct98 --all-containers
standard_init_linux.go:211: exec user process caused "exec format error"

$ kubectl logs --namespace monitoring monitoring-grafana-5c4dbdb56b-fwqmg  --all-containers
standard_init_linux.go:211: exec user process caused "exec format error"
Error from server (BadRequest): container "grafana-sc-dashboard" in pod "monitoring-grafana-5c4dbdb56b-fwqmg" is waiting to start: PodInitializing

Version of Helm and Kubernetes:
$ helm version
version.BuildInfo{Version:"v3.2.1", GitCommit:"fe51cd1e31e6a202cba7dead9552a6d418ded79a", GitTreeState:"clean", GoVersion:"go1.13.10"}
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:52:00Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/arm64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:43:34Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/arm64"}
Which chart:
stable/prometheus-operator 0.38.1
What happened:
Didn't fully start, because it pulled x86_64 containers.
What you expected to happen:
It to use the arm64 containers
How to reproduce it (as minimally and precisely as possible):
Take a raspberry pi 4, install ubuntu server 64-bit, install kubernetes from kubernetes.io instructions, install helm, helm install stable/prometheus-operator

Can we reopen closed issues or do I need to make a new issue? I am interested in trying to resolve this.


Fix <SUPABASE-KEY> typo in README.md

Noticed a small typo in the readme. Great library, thanks! 🚀

this is open for a month and simple enough to merge. why is not merged yet?


"Tarball is not in network and can not be located in cache"

When I use docker to deploy VE, my yarn cannot normally read openmbee-angularjs-dropdown-multiselect-v1.0.2.tgz in the local folder ./lib, and the error message shows:
error "./lib/openmbee-angularjs-dropdown-multiselect-v1.0.2.tgz": Tarball is not in network and can not be located in cache (["/opt/mbee/ve/lib/openmbee-angularjs-dropdown -multiselect-v1.0.2.tgz","/usr/local/share/.cache/yarn/v6/npm-@openmbee-angularjs-dropdown-multiselect-1.0.2-d219ab5df1b76b2c1690b20df09119f7746973e3/node_modules/@openmbee/angul arjs-dropdown -multiselect/.yarn-tarball.tgz"])
I've read almost all similar issues on Google. According to the information in these issues, it seems that there is a problem with the reference to the offline cache in the package.json and yarn.lock files, which results in the Tarball not passed to the docker instance.
For example, the reference record in the package.json file is:
"@openmbee/angularjs-dropdown-multiselect": "./lib/openmbee-angularjs-dropdown-multiselect-v1.0.2.tgz"
And The reference record in the yarn.lock file is:
"<EMAIL_ADDRESS>   version "1.0.2"    resolved "./lib/openmbee-angularjs-dropdown-multiselect-v1.0.2.tgz#d219ab5df1b76b2c1690b20df09119f7746973e3"    dependencies:      angular "1.8.3"      jquery "^3.6.0"      lodash  #"^4.17.21"
Refering to  https://stackoverflow.com/questions/40102686/how-to-install-package-with-local-path-by-yarn-it-couldnt-find-package, https://github.com/yarnpkg/yarn /issues/2861, https://github.com/yarnpkg/yarn/issues/2920 and some other issues, I tried various repair methods, including modifying the path and resetting the cache, etc., but none of them could solve the problem .
I'm wondering if I understood the error correctly, or if this tarball file is corrupted. I've spent a long time to solve this problem, could anyone give me some help plz?

They accidentally skipped adding the file to the WORKDIR. To fix add the line:
!lib
to .dockerignore

They accidentally skipped adding the file to the WORKDIR. To fix add the line:
!lib
to .dockerignore

Thank you! I will try.


Add shared assets setup

Assets are now being shared across all 3 frontend app and
are coming from money-to-prisoners-common. This adds the setup needed
to import assets/templates from that package.
This change is part of the following story:
User Story #103077962
As a frontend developer,
I want assets/templates used across all frontend apps to be consistent and shared,
So that I don't have to make changes in multiple locations

retest this please

This is applying the changes from ministryofjustice/money-to-prisoners-cashbook#69 to this repo, so marking blocked until that's resolved.


Use torch._weight_int4pack_mm for W4A16 inference

Since pytorch 2.2, a new mixed mm operation is available to perform a matrix multiplication between float16 / bfloat16 inputs and int4 weights quantized group-wise along the output features axis (which is exactly what gptq, awq and quanto are doing.
This built-in kernel could be used instead of the custom AWQ kernel, that has several restrictions (group_size 128 in particular).
Note: this new operations requires a specific packing of the int4 data, and uses floating point zeropoint.

Starting from a QBitsTensor qweight, the formulas to obtain the data, scales and zero-point seem to be:
data = optimum.quanto.ungroup(qweight._data.unpack(), axis=0, orig_shape=qweight.shape)
scale = qweight._scale.view(qweight.shape[0], -1)
zeropoint = ((2**(4 - 1) - qweight._zeropoint) * qweight._scale).view(qweight.shape[0], -1)


Fix default continuous query lease host

Overview
This commit changes the lease acquisition to add the default host to the meta http handler so that the leader address and http address will match.
re: #5896, #6096
TODO

[ ] CHANGELOG.md updated
[x] Rebased/mergable
[x] Tests pass
[x] Sign CLA (if not already signed)

Needs changelog updated.  :+1:  on green.

Closing and reopening against 0.11.1 branch.


Draft: Add votes to x/act

Source: abstract

Replace Approvers list of Action by Votes list.

message Vote {
  // address is the address of the approver
  string address = 1;
  // voted_at is a timestamp specifying when the voter voted on the action
  google.protobuf.Timestamp voted_at = 2
      [(gogoproto.nullable) = false, (amino.dont_omitempty) = true, (gogoproto.stdtime) = true];
  // vote_type is a type of the vote issued
  VoteType vote_type = 3;
}

//  Type of a vote.
enum VoteType {
  // Unspecified status.
  VOTE_TYPE_UNSPECIFIED = 0;

  // Positive vote for an action
  VOTE_TYPE_APPROVE = 1;

  // Negative vote for an action
  VOTE_TYPE_REJECT = 2;
}

Replace Rule by ApprovalExpression and RejectExpression

message Action {
  ...
  warden.act.v1beta1.Template ApprovalExpression = 13 [(gogoproto.nullable) = false];
  warden.act.v1beta1.Template RejectExpression = 14 [(gogoproto.nullable) = false];
  ...
}

Replace MsgApproveAction by MsgVote. Update MsgServer to handle the message. ApprovalExpression and RejectExpression should be evaluated after each vote.

message MsgVote {
  option (cosmos.msg.v1.signer) = "creator";
  string creator           = 1;
  uint64 action_id       = 2;
  VoteType vote_type = 3;
}

great!
just a note:
warden.act.v1beta1.Template ApprovalExpression = 13 [(gogoproto.nullable) = false];
warden.act.v1beta1.Template RejectExpression = 14 [(gogoproto.nullable) = false];

these shouldn't be templates, they should be the Expression itself (i.e. the AST of the expression)
we'll keep templates completely disconnected from actions


How to remove "example" wording from %collapsible?

I have the following in my .adoc file:
.Title
[%collapsible]
====
. Step 1
. Step 2
====

This produces a collapsed Title in HTML, which when clicked, is expanded, so that step 1 and step 2 are now visible.
The same code in asciidoctor-pdf produces:
Example 1. Title
1. Step 1
2. Step 2

How can I remove the example wording and the number from the produced PDF?

Aha. Asciidoctor PDF is not supposed to be adding the caption prefix in this case. I'll update the code.
In the meantime, you can use this as a workaround.
.Title
[%collapsible,caption=]
====
. Step 1
. Step 2
====

Thank you very much!


Error when running anime dl

Hello!
Was looking for some assistance here.
I use this with Ubuntu headless Server
I have used this without issues in the past and have never had a problem, but recently I have been running into an issue where whenever I run the anime dl command I get the error
ERROR: [Errno 2] No such file or directory: 'node' : 'node'
Any assistance would be appreciated.

-Terra
Edit:
For more information, I had it working until one day this just started happening.
I use this in a Linux virtual machine and since I didn't need the machine anymore I created a new virtual machine that ONLY has Aria2, Python3, and Anime-Downloader on it.
So its a clean install that should have nothing getting in the way.

You are using twist.moe. I am not sure on this, but as far as I am aware twist.moe never worked for me, most likely due to missing Node.js? I might be mistaken on that so what I recommend to you is to change the provider in the config file to something like animepahe. Please take a look at PR https://github.com/vn-ki/anime-downloader/pull/279. You can also take a look at the issue I have linked in it as well.
@vn-ki didn't answer me if this is a bug or just missing Node.js. Considering you are facing this issue my PR has even more reasons to be merged :)

You are using twist.moe. I am not sure on this, but as far as I am aware twist.moe never worked for me, most likely due to missing Node.js? I might be mistaken on that so what I recommend to you is to change the provider in the config file to something like animepahe, or just use the --provider animepahe flag. Please take a look at PR #279. You can also take a look at the issue I have linked in it as well.
@vn-ki didn't answer me if this is a bug or just missing Node.js. Considering you are facing this issue my PR has even more reasons to be merged :)

It is definitely twist.moe.
Whats interesting is that It was literally overnight that it worked then didn't.
Looks like multiple require node.js as well.
animepahe does however seem to be working!
I appreciate the assistance on this as while I did see issues referencing the node.js issue, I did not find anything with my specific error.
Appreciate it! :)

Correction, not working still
When using animepahe it at least lets me search for anime, but when trying to download it now tells me that.
Just to make sure I tried downloading an unrelated file using aria2 and it downloads with no issues.

@terraenvy please take a quick look at https://github.com/vn-ki/anime-downloader/issues/282
It seems something on the animepahe side was off temporarely. Now it should be working for you :)
Also check out my https://github.com/RaitaroH/adl for a better anime dl experience.

Closing because this seems to be resolved.


Connection type "Python remote(debugpy)" missing

Connection type "Python remote(debugpy)" missing. Need infos on debugpy and remote debugging on localhost.

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

ID: 91ab3b84-305b-434b-2155-b288fa98c973
Version Independent ID: 22fb753b-a8a2-5572-03fd-9b98cd118c90
Content: Debug Python code - Visual Studio
Content Source: docs/python/debugging-python-in-visual-studio.md
Product: visual-studio-windows
Technology: vs-python
GitHub Login: @JoshuaPartlow
Microsoft Alias: joshuapa

Hi @ghaefele - Thank you for the feedback. The peer topic Remotely debug Python code on Linux talks about Python remote (debugpy). If that's not what you were looking for and you think this is an issue that should still be addressed, please feel free to reopen.


This implements feature request #103

Added refresh menu item in overflow menu to allow user to refresh the content in *ListFragments and *DetailsFragments.

This conflicts with #135 . Can you check it please.
I also tried applying them in reverse order, but the conflict is also there.

Oops, sorry, I only checked it against master. I should have checked it against #135 as well. Should merge cleanly now.

Ok. Any reason for removing the @Override from the onStop method on AbstractDetailsFragment?

Nope, sigh that's what I get when I do things inbetween...
On Fri, Nov 13, 2015 at 9:43 AM, Synced Synapse<EMAIL_ADDRESS>wrote:

Ok. Any reason for removing the @Override from the onStop method on
AbstractDetailsFragment?
—
Reply to this email directly or view it on GitHub
https://github.com/xbmc/Kore/pull/136#issuecomment-156363355.

fixed it.


Ollie has contentprovider support, do you add contentprovider support?

relates to, #44 . Will say it will be on the way in a near-future release.


Kotlin 1.6.20-RC Support

When using realm in a Kotlin Multiplatform project, im getting the following error:
Compilation failed: Back-end: Please report this problem https://kotl.in/issue
/Users/andrewreed/Documents/workspace/Mobile-Library/src/commonMain/kotlin/com/coolio/mp/storage/models/realm/Alias.kt:-1:-1
Details: Internal error in file lowering: java.lang.IllegalStateException: Cannot find io.realm.internal.interop.ClassInfo on platform Native (general).

The file in question is very simple so i dont think its related to the syntaxes used in it, but here it is:
package com.coolio.mp.storage.models.realm

import io.realm.RealmObject

class Alias : RealmObject {
    var stringValue: String = ""
}

Pretty sure something internal is going wrong with Kotlin and Realm, its probably more likely an issue with Kotlin 1.6.20 but i thought best to raise it here too.

Hi @Reedyuk Thank you for the report. That error looks similar to https://github.com/realm/realm-kotlin/issues/662. What version of the Realm SDK are you using?

0.10.0

Hi @Reedyuk can you please share an example (or your Gradle files), I tried both on an Android and KMM project and both works with 1.6.20-RC2

currently on 1.6.10 for now, ive not got chance to reproduce again.


Add vu-meter with each track, to be able to check if input signal is ok

Each track should have a small vu-meter to check that when armed an input signal is coming to it. The same vu-meter could also be used during playback. Related to another issue: make it logarithmic.

Done.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: KanwarAmrinder sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


(GH-1244) Serialize plan variables as pcore

There are 4 custom types in the Bolt plan language that we want to make
available inside apply blocks: Target, Result, ResultSet, ApplyResult.
For each of the Result objects this is accomplished by serializing the
plan variables to the apply block as pcore instead of as json. We also
needed to add pcore_init methods to the implementation classes of the
objects, because the pcore deserializer will by default call
initialize on the class and pass in the values of the attributes of
the type, which doesn't match the signature of the actual
implementations. Using _pcore_init_hash and _pcore_init_from_hash
lets us define what data gets serialized and how it gets deserialized
for those types.
Target objects are trickier for a few reasons:

They must have an associate inventory
...

Stil TODO:

[] Future proof
[] Tests
[] Docs
[] Threading

CLA signed by all contributors.

Actually, it also needs a changelog entry.

Updated - I also didn't see that we mentioned this in the documentation at all, but should this be documented or have an example somewhere?? CC @hestonhoffman

I think this mostly just makes it work the way it intuitively should, so I'm not sure what there is to document about it.


Release notes for 2.0.15 release

@derekbekoe,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

FYI @sptramer @JasonRShaver

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/release-notes-azure-cli.md
:white_check_mark:Succeeded
View

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

@sankethka Please merge ASAP and send to live.


Tutorial: How to stop all the components correctly

Are there any plans on adding a tutorial with all the steps to correctly stop all the architecture? Or a set of scripts to perform this.
This will be helpful for people starting to experiment with OpenMLOps on AWS, or individual practitoners or students that still can't afford to have all this architecture running all the time. So if they are on a budget they can run experiments, try the architecture, and then stop everything.

Hello! Thanks for your interest in OpenMLOps.
Yes, I think it would be valuable to have a tutorial on that, I will add this to our to-do list. On a more practical note, performing a simple terraform destroy should take down most of your infrastructure, being your DNS records (in route53 if you are using AWS for instance) and your TLS certificates the only things that you should manually take down afterward.


Setup KanBan for fullStack group_capstone

collaborators in this project include;

@Reggeditt
@emmanuelsteven
@bronnie313
@IsmailChangezi
this is the link to our project

Hi @emmanuelsteven
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a link fo the Kanban board that is private.  I cannot access it for review.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @Meltrust,
Wow, you did it 🎉

Your project is complete
To highlight

All cards have descriptions, are fairly pointed, and have assignees✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add OrderSchema tests

⚠ Depends on #1299
Summary

Adds unit tests to OrderSchema.

Changes

Changes .default('unknown') to .catch('unknown') in the OrderSchema as the default value would only be set if no value was provided. If an unknown value is provided we should still set the respective value as unknown.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 93.618%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
6481

Relevant Lines:
6688

💛  - Coveralls


I should see on the resource cards a label that shows if a resource has been verified by the Concourse team

As an OSS Concourse User when I go to https://concourse.github.io/dutyfree/
I can see that Concourse resources have been marked as verified.
So that I can have confidence that the resource does what I expect it do
Acceptance Criteria
When I go to https://concourse.github.io/dutyfree/
I see a way to see which resources the concourse team have verified.
PM Notes: Definition of what verified means and what the list of verified resources would be to confirmed with the Concourse team.
See Figma iteration 4 prototype for placement and styling

Based on feedback from Concourse team can we move the verified label across to the top right hand side of the resource card.

@carlo-colombo @monika-hrda
Accepting.
Thanks!


Step 6 needs updates (6. Add the client address pool)

It looks like the 'Point-to-site Configuration' blade has been changed. Now, it just has three sections: Address Pool, Root certificates, and Revoked certificates.

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

ID: 59e72010-dab4-54d9-1409-c95b0ce60824
Version Independent ID: 0da5450c-6181-1c27-93c0-b0c698d2d598
Content: Connect a computer to an Azure virtual network using Point-to-Site and native Azure certificate authentication: Azure Portal
Content Source: articles/vpn-gateway/vpn-gateway-howto-point-to-site-resource-manager-portal.md
Service: vpn-gateway
GitHub Login: @cherylmc
Microsoft Alias: cherylmc

@npherson Thank you very much for your feedback. We will investigate your inquiry and get back with you.

@cherylmc Here is another VPN content edit due to changes in the Azure Portal workflow.

Thanks! I'll take a look!

#please-close


Better logging to help debug test failure

https://github.com/aspnet/Razor/issues/2571
Looks like all these tests have suddenly timing out during restore on Mac. My guess is either restore is slow on Mac for some reason or there is some underlying problem in MsBuild with Mac. Either way this should help us debug what's really going on

Were some of the failures in release/2.2 as well? If so, should we put this in that branch also?

Updated. @Eilon


Configurable websocket url

Command
serve
Description
I have a scenario when I'm running large system behind a proxy which simulates the production and there are several apps running in a background - that includes several Angular apps. I realized that hot reload would work for only one of them at the time - websocket url is rooted and cannot be changed. Would setting different URL for web-socket connection help to make them all work with hot reload?
Describe the solution you'd like
I would like to have a way to customize the URL for the web-socket connection so that I can distinguish them and proxy them to correct angular dev server
Describe alternatives you've considered
no alternatives considered

Hi @vmachacek,
In this scenario you can use the --serve-path option.
See: https://angular.io/cli/serve for more information about all the available options in ng serve.


[Bug]: npm ERR! code 1 npm ERR! path /home/leiwei/programs/ragflow/web npm ERR! command failed npm ERR! command sh -c umi setup

Is there an existing issue for the same bug?

[X] I have checked the existing issues.

Branch name
main
Commit ID
the latest code
Other environment information
OS: ubuntu 22.04
python: 3.11

Actual behavior
I need to Launch the service from source. When I run the commad "npm install --registry=https://registry.npmmirror.com --force", the bug was occurred.
Expected behavior
I want to solve the bug, how can I do? Thank you.
Steps to reproduce
SyntaxError: Unexpected token '?'
    at wrapSafe (internal/modules/cjs/loader.js:915:16)
    at Module._compile (internal/modules/cjs/loader.js:963:27)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Module.require (internal/modules/cjs/loader.js:887:19)
    at require (internal/modules/cjs/helpers.js:85:18)
    at Object.<anonymous> (/home/leiwei/programs/ragflow/web/node_modules/@umijs/utils/dist/index.js:76:28)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
npm ERR! code 1
npm ERR! path /home/leiwei/programs/ragflow/web
npm ERR! command failed
npm ERR! command sh -c umi setup

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/lw03-10052696/.npm/_logs/2024-09-14T02_41_32_269Z-debug-0.log

Additional information
No response

Run npm install directly in the web directory

Run npm install directly in the web directory

Thank you! I was running nodejs version 12, and the bug was fixed when I upgraded it to version 18.


TryGetFolderFromPathAsync with empty folder path throws

Describe the bug
Calling IStorageProvider.TryGetFolderFromPathAsync(string) throws an ArgumentException when the string parameter is empty.
To Reproduce
Steps to reproduce the behavior:

Call _storageProvider.TryGetFolderFromPathAsync(string.Empty)

Expected behavior
I expected TryGetFolderFromPathAsync to return null if the path was empty since this is a "Try" function that returns a nullable object.
Desktop:

OS: Windows 11
Version 11 RC1.1

Additional context
Stack trace:
System.ArgumentException: The path is empty. (Parameter 'path')
   at System.IO.Path.GetFullPath(String path)
   at System.IO.DirectoryInfo..ctor(String path)
   at Avalonia.Platform.Storage.FileIO.StorageProviderHelpers.TryCreateBclStorageItem(String path) in /_/src/Avalonia.Base/Platform/Storage/FileIO/StorageProviderHelpers.cs:line 13
   at Avalonia.Platform.Storage.StorageProviderExtensions.TryGetFolderFromPathAsync(IStorageProvider provider, String folderPath) in /_/src/Avalonia.Base/Platform/Storage/StorageProviderExtensions.cs:line 34

This change makes sense if anybody can pick it up.

@alexhelms are you willing to provide a PR for it?


Drop to float16 if bfloat16 is not supported

Instead of throwing an error if the GPU compute capability is not supported for bfloat16, vLLM should throw a warning but use float16 instead. This helps in Colab notebooks where the compute of the T4 instance is set to 7.5 and not 8 so vLLM does not work trivially.

I personally prefer to ask users to explicitly specify dtype in this case, since otherwise it can affect the accuracy of the model silently. WDYT @zhuohan123 @simon-mo?

I think the fallback is fine if we explicitly print a warning?

+1. In this case warning is fine. The accuracy difference between bfloat and float should not be too crazy.

I came across this issue while trying to use langchain's vllm support on Colab using a T4 GPU. In case we are not going through with this change do let me know alternatively the best way to go about in the following screenshot. Thanks a lot.

@acebot712 You can simply do this:
llm = LLM(model="mistralai/Mistral-7B-Instruct-v0.1", dtype="half")

I agree with @WoosukKwon, I think in general we should avoid doing any "magic" that can change the outputs, even if slightly. I would suggest to instead modify the exception message to suggest the user to set the dtype to float16 themselves.

I would suggest to instead modify the exception message to suggest the user to set the dtype to float16 themselves.

We can check device capability outside the vLLM and choose the dtype depending on device. The codes could be:
llm = LLM(
    model="mistralai/Mistral-7B-Instruct-v0.1",
    dtype="float16" if torch.cuda.get_device_capability()[0] < 8 else "bfloat16",
)

Hi @acebot712 Thanks for bringing up this issue and submitting the PR! We've decided to keep the current behavior; To avoid silent accuracy changes, vLLM will ask users to set dtype="half" in such a case. Could you revert the change in this PR and add a warning that asks users to set dtype="half"? Thanks.

As a beginner, I just experienced this little setback. Finally, of course, adding the parameter solved it: --dtype="half"
But the premise is that, clever as I am, I found this issue. For the sake of future beginners, it is suggested to provide guidance in the error message, suggesting users to add --dtype="half" to specify the data type. @WoosukKwon

@chuanzhubin A better error message is indeed a great idea? Would you be interested in submitting a PR?


Mounting an engine before {{outlet}} causes error

In app/templates/application.hbs:

Welcome to Ember

{{#link-to "index"}}Home{{/link-to}}
{{#link-to "boom"}}Click-click{{/link-to}}   <-- link to routable engine
{{mount "mountable-engine"}}
{{outlet}}

causes TypeError: undefined is not an object (evaluating 'node.getState().controller.set')
while everything seems fine if this is the case:

Welcome to Ember

{{#link-to "index"}}Home{{/link-to}}
{{#link-to "boom"}}Click-click{{/link-to}}
{{outlet}}
{{mount "mountable-engine"}}

Example repo: https://github.com/alecho/ember-engine-poc Commit 853aa6605ad7057f283e567893d0107032087e40 is working but switching the code to the first example above demonstrates the issue.

Sounds like it's related to https://github.com/dgeb/ember-engines/pull/66

Thanks for reporting. This should be fixed in v0.1.3 (via #66), but please let us know if you experience other problems.


How to activate Style Manager?

First of all. Really great and easy package!
I was wondering if anyone has managed to activate the Style Manager? See below screenshot.

First of all. Really great and easy package!
I was wondering if anyone has managed to activate the Style Manager? See below screenshot.

config/laravel-grapesjs.php
line 109 change of true to false
    'style_manager' => [
        'limited_selectors' => true,
    ],

    'style_manager' => [
        'limited_selectors' => false,
    ],


chore(chart-hook): updating chart to configure hooks for nfs-provisioner

Signed-off-by: mayank<EMAIL_ADDRESS>What this PR does?:
This PR updates helm chart to support configuring nfs hook for nfs-provisioner.
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered::
Any additional information for your reviewer? :
Code changes are covered in https://github.com/openebs/dynamic-nfs-provisioner/pull/93
Checklist:

[ ] Fixes #
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Does this PR change require updating NFS-Provisioner Chart? If yes, mention the Helm Chart PR #
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Closing this PR since updated PR with configmap support is raised.


Camera Bugs with position.y

Bugs

Bug repro on playground

LINE15 & LINE98
VERSION: LATEST

LINE15--ArcRotateCamera - position

Expected result: I was going to create an ArcRotateCamera at Vector3(0, -150, 0)
Current result: It only works if the Vector3.x is not Zero, so I have to change it to Vector3(1, -150, 0) and it works.

LINE98--FreeCamera or UniversalCamera - mouse inputs
-The FreeCamera at Vector3(0, -150, 0) couldn't response to the mouse inputs properly.

You may change the current active camera by editing LINE98 and try other cameras and find them working properly, so I supposed it might be a bug?

This is a known limitation of perspective cameras
Both issues can be fixed by setting x or z to a really small value like 0.0001.

Thanks. That's close to my guess. I'll close the issue.


Do not alter runtime types

Fixes #118 - Do not alter python runtime types on the fly

Correctly create new parameterized types rather than modifying them in place.
Have to use more than typing.Iterable now - hopefully this doesn't cause problems down the line, outside this package.
Still, detect the move to collections.abc.X when there is a matching typing.X, and walk back to typing.X. This is needed to support earlier versions of python, and feels rather fragile!
Fix up versions for github actions to avoid CI build depreciation warnings

Codecov Report

Merging #120 (2db3f81) into master (ace1464) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 2db3f81 differs from pull request most recent head 32e8a73. Consider uploading reports for the commit 32e8a73 to get more accurate results

@@           Coverage Diff           @@
##           master     #120   +/-   ##
=======================================
  Coverage   96.77%   96.78%
=======================================
  Files          15       15
  Lines        1272     1275    +3
=======================================
+ Hits         1231     1234    +3
  Misses         41       41

Flag
Coverage Δ

unittests
96.78% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

func_adl/util_types.py
97.89% <100.00%> (+0.06%)
:arrow_up:

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


[Phcode.dev Bot] Publishing Release 0.0.3 to Extension Store

Thank you for contributing to phcode.dev extension store.
Please track extension/theme publish status by clicking this link.
You may close this issue at any time.

Extension successfully published to store. You can close this issue at any time.


Add sample size and links to the Collex DTO

What and why?
Currently there are a number of endpoints that return a CollectionExerciseDTO object (which then gets converted into json).  This object has a bunch of useful information about a collection exercise, including the action events, but it's missing any information about the sample summary and the sample size.  This matters because we have to make repeated calls to the exercise service to get this information in our other services.
How to test?
Do any request that returns a CollectionExerciseDTO - I would suggest GET /{collex_id}/survey/{survey_id}
Jira
RAS-302

/deploy harrisonj


Failed/blocked measurements wrongly annotated as "accessible" on OONI Explorer & MAT

A community member reported 2 cases where blocked measurements were annotated as "accessible" on OONI Explorer and the MAT:
# Turkmenistan
The following measurement is annotated as "accessible" on OONI Explorer (https://explorer.ooni.org/search?since=2022-05-12&until=2022-06-12&failure=false&domain=ajax.aspnetcdn.com&probe_cc=TM) and the MAT, even though the measurement itself is failed: https://explorer.ooni.org/measurement/20220524T011649Z_webconnectivity_TM_20661_n1_K12GviVnQLLRnJoU?input=https%3A%2F%2Fajax.aspnetcdn.com%2Fajax%2F4.5.2%2F1%2FMicrosoftAjax.js
In this case, it appears that they're blocking both the website and the test helper, which is why OONI Explorer/MAT return the "accessible" annotation (since the results from the local and control vantage points match). However, we should probably expand upon our testing heuristics to account for such cases (when the test helper is blocked).
In the meanwhile, is there a way to annotate these measurements as "anomalous" or "failed" (instead of "accessible") on OONI Explorer/MAT?
# Iran
cdn.ampproject.org was blocked on 23rd May 2022 in Iran on AS3177, but the OONI measurement is flagged as "accessible" on OONI Explorer (https://explorer.ooni.org/search?since=2022-05-12&until=2022-05-24&domain=cdn.ampproject.org&probe_cc=IR&probe_asn=AS3177) and the MAT (because there was no response for DNS requests): https://explorer.ooni.org/measurement/20220523T182859Z_webconnectivity_IR_3177_n1_HqfZB9YzjiFojN0Y?input=https%3A%2F%2Fcdn.ampproject.org%2Fv0%2Famp-iframe-0.1.js
Such measurements should be annotated as "anomalous" (or at least "failed") on OONI Explorer/MAT, but not "accessible".

@hellais @FedericoCeratto what is the status of this?


Added a 'Why Contribute to Open Source?' section in the README

Associated Issue: #48
Summary of Changes

Added a "Why Contribute to Open Source?" section in the README. It is located under the "Who Can Contribute?" section.

Pull Request Test Coverage Report for Build 162

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

Totals

Change from base Build 160:
0.0%

Covered Lines:
97

Relevant Lines:
107

💛  - Coveralls


Date Picker Component Request

Spec reference: https://www.google.com/design/spec/components/pickers.html#pickers-date-pickers
It would be great if this library would have a datepiker component

Stay tuned. I'm extending our component library by adding features not yet available in upstream material-components-web.
First up

Mini variant drawer - https://github.com/trimox/angular-mdc-web/issues/158
Auto-complete (search component)
Date Picker: https://github.com/trimox/angular-mdc-web/issues/392
Expansion panels: https://material.io/guidelines/components/expansion-panels.html

That is a great news
Especially that material-components-web guys don't plan to release these components soon

as a corollary to this would love a date time picker component if possible

@cpru2 Agree. I'm composing engineering documents for the following.
reference: https://www.google.com/design/spec/components/pickers.html#pickers-date-pickers
reference: https://material.io/guidelines/components/pickers.html#pickers-time-pickers
Minor update for this thread.
mdc-data-table and mdc-expansion components are couple weeks out. Should free up time for the Date/Time component design.

bumping this — if anyone knows of any supported component libraries that have a datetimepicker, would greatly appreciate a temporary solution.

Closing for now. Focusing on component refactoring and enhancements.


Improve dfs_edges documentation

Closes #516
Improves the documentation of dfs_edges by:

making explicit that only tree edges from the DFS are returned
adding the pseudocode for the DFS procedure

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
11102

Relevant Lines:
11272

💛  - Coveralls

I am curious about the pseudocode, and why it was added.
Is it meant to explain what depth first search is to someone who is not familiar with the concept?
Is it meant to clarify which specific DFS tree will be returned out of the several possible ones?
What does iterator of G.edges(v) refer to? All incident edges of v? Then is w in w := next(iterator) a vertex or an edge? (I see it's a vertex, but it's confusing that it's taken from an iterator over the edges.)
I think the pseudocode may be slightly incorrect. It does not label the first node as discovered.

I am curious about the pseudocode, and why it was added.

It's just a sketch of how this function works so we can annotate which edges are being returned in case someone is not familiar, or does  not recall, which edges are classified as tree edges in a DFS traversal.

What does iterator of G.edges(v) refer to?

It's an iterator over all neighbors of nodes v.


The use of synchronous document translation service to translate form-data document files directly

Is your feature request related to a problem? Please describe.
I tried using the synchronous document translation sample created here: https://github.com/Azure/azure-sdk-for-js/tree/main/sdk/translation/ai-translation-document-rest/samples/v1-beta/typescript/src/synchronousDocumentTranslation.ts but on the example usage there, a simple plain text was used, and on the project that I'm working on, I need the user to upload their document, it should accept the document and give out the output back as a document or blob.
Describe the solution you'd like
I'd greatly appreciate it if you could help me with an example of a similar implementation that accepts and returns a document file directly,  like a PDF or blob that can be converted back to the right or desired format.
Describe alternatives you've considered
I tried passing in a formData to the body directly but it's throwing error and from the type error that I'm getting which is:
Type 'FormData' is not assignable to type 'DocumentTranslateContentDocumentPartDescriptor | DocumentTranslateContentGlossaryPartDescriptor'., probably because that type of file is not considered for acceptance in the current implementation.
Additional context
Here's an attached screenshot of where I'm using it, though the request is accepting multipart/form-data, but typescript won't just stop yelling at me:

Then when I destructured the file content to match the example used like this:
body: [       {         name: "document",         body: document,         filename: document.name,         contentType: "application/pdf",       },     ],
, I'm getting this error:
error: {       code: 'InvalidRequest',       message: 'The format parameter is not valid.',       target: 'ContentType',       innerError: [Object]     }
Thanks in advance for your help.
cc: @hamshavathimunibyraiah

Hello @rahmlad-aramide , you can provide a file path and then read and feed the contents of that file into the body parameter.
For example:
// Replace 'filePath' with the path to your file
const filePath = 'd:/testdocs/test-input.txt';
// Read the contents of the file asynchronously
const fileContent = await readFileAsync(filePath, 'utf8');

const options: DocumentTranslateParameters = {
  queryParameters: {
    targetLanguage: "hi",
  },
  contentType: "multipart/form-data",
  body: [
    {
      name: "document",
      body: fileContent,
      filename: "test-input.txt",
      contentType: "text/plain",
    },
  ],
};

If you have a file in Azure blob, and want that to be translated into the right format, then you can use the example from here: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/translation/ai-translation-document-rest/samples/v1-beta/typescript/src/batchDocumentTranslation.ts

Hello @rahmlad-aramide , you can provide a file path and then read and feed the contents of that file into the body parameter. For example: `// Replace 'filePath' with the path to your file const filePath = 'd:/testdocs/test-input.txt'; // Read the contents of the file asynchronously const fileContent = await readFileAsync(filePath, 'utf8');
const options: DocumentTranslateParameters = {
  queryParameters: {
    targetLanguage: "hi",
  },
  contentType: "multipart/form-data",
  body: [
    {
      name: "document",
      body: fileContent,
      filename: "test-input.txt",
      contentType: "text/plain",
    },
  ],
};`

If you have a file in Azure blob, and want that to be translated into the right format, then you can use the example from here: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/translation/ai-translation-document-rest/samples/v1-beta/typescript/src/batchDocumentTranslation.ts

That's the challenge I'm facing, because I'm trying to integrate it into my frontend application whereby the user will upload the file and accessing the absolute file path of a user's uploaded file directly in the browser is not possible due to security reasons. Browsers do not expose the information to web applications to protect user privacy and system security. Likewise, using the batch translation, the output file is not produced instantly, or is it possible that I provide the URL to the azure blob storage to the synchronous file translator after uploading it there or there's no solution at all?
Thanks in advance for your help.
cc: @hamshavathimunibyraiah


Disney Plus Clone

Description
Description of the project here:
I will be gone to make Disney Plus Clone using HTML,CSS, JS which will be responsive in nature.
Code of Conduct

[X] I follow Contributing Guidelines of this project.

/assign


EventError fired after suspended Screen resumes

In the past, in tview, I hadn't handled the EventError event. But as one user reported, this gets fired when the user closes the terminal when the application is still running so it should be handled.
Now it turns out I'm getting EventError after a suspended screen resumes. Here's some code reproduce this:
package main

import (
	"fmt"
	"os"
	"os/exec"

	"github.com/gdamore/tcell/v2"
)

func main() {
	s, e := tcell.NewScreen()
	if e != nil {
		fmt.Fprintf(os.Stderr, "%v\n", e)
		os.Exit(1)
	}
	if e := s.Init(); e != nil {
		fmt.Fprintf(os.Stderr, "%v\n", e)
		os.Exit(1)
	}

	for {
		switch ev := s.PollEvent().(type) {
		case *tcell.EventError:
			s.Fini()
			panic(ev) // This fires with EOF after we return from vim.
		case *tcell.EventKey:
			if ev.Key() == tcell.KeyEscape {
				s.Fini()
				os.Exit(0)
			} else if ev.Rune() == 's' { // Press "s" to suspend and start vim.
				if err := s.Suspend(); err != nil {
					panic(err)
				}

				cmd := exec.Command("vim", "/tmp/foo")
				cmd.Stdout = os.Stdout
				cmd.Stdin = os.Stdin
				_ = cmd.Run()

				if err := s.Resume(); err != nil {
					panic(err)
				}
			}
		}
	}
}

Press s to suspend the screen and start vim. When you exit vim, the application panics with "EOF" after the EventError is handled. (It's not vim, this happens with any command.)
Any idea why this happens?

It panics on panic: read /dev/tty: i/o timeout on ArchLinux.
require (
	github.com/gdamore/tcell/v2 v2.4.0
	github.com/lucasb-eyer/go-colorful v1.2.0 // indirect
	github.com/mattn/go-runewidth v0.0.13 // indirect
	golang.org/x/sys v0.0.0-20210903071746-97244b99971b // indirect
	golang.org/x/term v0.0.0-20210615171337-6886f2dfbf5b // indirect
	golang.org/x/text v0.3.7 // indirect
)

go version go1.17 linux/amd64

Wait, did the terminal close?  (E.g. you did this from a subshell?)  I could see that being a problem.  I'm not sure how best to fix this -- because we have to have a way to report this problem to the calling application.  But clearly panic() isn't ideal.

Ok , this isn't specific to Arch.  I've reproduced in WSL.

So the problem is when we suspend, we "close" the channel and rely on a time out to wake us up, but we also send a bogus error upstream.  We can conditionalize sending the error upstream on t.running.

Please have a try with the branch in fix480 -- I suspect this will address the problem.

It works on macOS, thank you.

It works on Arch now too. Thanks!

This change actually created a worse regression -- as some activity to wake and exit the poller is required, and this prevents that from happening.  This causes exiting mouse.go to hang until another key is pressed, and suspending also is not immediate.  :(

I'd be grateful if some macOS user could test the fix for the regression -- see PR #490

I just tried the code above on macOS with 0f66ee8 and it worked as expected.


Context embedding shows anomaly, independent of sentence and token

Problem
While doing some analysis on the pre-trained SciBert transformer networks, we found that there is an anomaly in the context embedding on index 422. Doing some more tests we found that this anomaly was there, independent of the context or the specific token and position. We think that the distance metrics in the contextualized embedding-space, such as cosine similarity, are heavily dominated by this exploding component.
scibert-scivocab-uncased
To see the actual contextual representation of words, we infered a sentence using the pretrained model. We performed this using both a TF and PyTorch version of the model to see if the results were similar.  Below we can see a crude representation of the embedding space (word + positional embedding) simply as a linear plot of the 768 components of the embedding for the word linear in the context of a regular sentence. The values seem nicely distributed around 0 and peaking just a few times just above 2 or -2, which is just as someone would expect the embedding space to look like.

However, if we look at the contextualized embedding, after the last layer of the SciBert model, we find the embedding has turned into this representation below. The representation "explodes" at a particular index (422) very consistently across tokens and sentences.

the token is linear is used in the context of the sentence How are linear regression and gradient descent related, is gradient descent a type of linear regression, and is it similar to ordinary least squares (OLS) and generalized least squares (GLS)?
scibert-scivocab-cased
We now performed the same analysis using the cased version of the model, which gives the following results. Here we have the token: difference  for the sentence:
Computer Vision: What is the difference between HOG and SIFT feature descriptor? The exploding index in this embedding is 421, meaning that the anomaly occurs at a different position.

To compared with the uncased model we did the same for the token linear for the sentence  How are linear regression and gradient descent related, is gradient descent a type of linear regression, and is it similar to ordinary least squares (OLS) and generalized least squares (GLS)? Here we see that the anomaly again is consistent, this also holds for other examples.

From this we see that a similar problem occurs both in the cased and the uncased model, yet the issue is found on a different location from which we figure that something odd is happening in the training process. Again, we have tested this is a behavior in many other tokens and sentences and it's consistent across all circumstances.
Could you explain if this behavior is expected by you and why? And otherwise could you explain what causes this behavior and how it can potentially be overcome?
Thanks in advance!d

Interesting. I have seen the same pattern while training transformers for another project. I don't know why this is happening, but it doesn't seem to be a bug

Can you try this for regular BERT and see if you get the same pattern?

We also tried this for the regular BERT uncased model. This gives us the following results.
The first image show the word difference in the context Computer Vision: What is the difference between HOG and SIFT feature descriptor?. This gives a clear peak at 308. This happens for more tokens but not as consistent as in Scibert.

Another example, the word linear in the context How are linear regression and gradient descent related, is gradient descent a type of linear regression, and is it similar to ordinary least squares (OLS) and generalized least squares (GLS)? creates an output that is much more expected.

Besides results for Bert, we found some tokens in which the pattern doesn't occur when using SciBert. These are for example symbols "),?" and the token The

Conclusion
We see a pattern in both Bert and SciBert, but it is not as strong and therefore we expect it to have less of an impact on the similarity metrics/distances. Not only is it not as consistent, Bert also shows many more peaks which causes a consistent peak to be of less impact.
Do you have any ideas what it can be caused by and what the possible implications are?
Thanks!

Did you already have a look at this?

As I said, I don't think this is a bug, it is just how the model decided to represent your tokens. As to the similarity measures, maybe normalizing the vector somehow will prevent this dimension to dominate the score


doc: update EG control-plane metrics design

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #

/retest

can you split the go file changes into another PR?

sure


Electrum: use protocol://host:port URIs for custom servers

URI can be either tcp://host:port or ssl://host:port for now

A better solution is - if the user is connecting to an .onion address - to add a checkbox in the Electrum dialog to force the certificate verification. By default, it is unchecked.
If the user is not connecting to an .onion address, no checkbox, the app always checks the certificate.


pylsd and glustick package confilication error,  in pip install -r requiremnts.txt

Editor: VScode
host: windows 10 pro
container:  Docker file: FROM ubuntu:jammy
python version in python virtual environment  : python 3.10
Hi,
After I install ceres and eigen in ubuntu, I go into the src code and use a virtual environment of python 3.10
to install requirements, I met an error when running line:25 in  README.md, " pip install -r requirements.txt ":
the following is the error messages:
ERROR: Cannot install -r requirements.txt (line 33) and pytlsd 0.0.4 (from git+https://github.com/iago-suarez/pytlsd.git#egg=pytlsd) because these package versions have conflicting dependencies.
The conflict is caused by:
The user requested pytlsd 0.0.4 (from git+https://github.com/iago-suarez/pytlsd.git#egg=pytlsd)
gluestick 0.0.0 depends on pytlsd 0.0.4 (from git+https://github.com/iago-suarez/pytlsd.git@37ac583)
To fix this you could try to:

loosen the range of package versions you've specified
remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts
I have tried modifying the  last 3 line in the requirements.txt :
pytlsd@git+https://github.com/iago-suarez/pytlsd.git@37ac583
-e git+https://github.com/cvg/DeepLSD.git#egg=deeplsd
-e git+https://github.com/cvg/GlueStick.git#egg=gluestick
but still got errors:
 Building wheel for pytlsd (pyproject.toml) ... error
 error: subprocess-exited-with-error

× Building wheel for pytlsd (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [118 lines of output]
running bdist_wheel
running build
running build_ext
-- The C compiler identification is GNU 11.4.0
-- The CXX compiler identification is GNU 11.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
CMake Deprecation Warning at pybind11/CMakeLists.txt:8 (cmake_minimum_required):
Compatibility with CMake < 3.5 will be removed from a future version of
CMake.
    Update the VERSION argument <min> value or use a ...<max> suffix to tell
    CMake that the project does not need compatibility with older versions.

  -- pybind11 v2.10.4
  CMake Warning (dev) at pybind11/tools/FindPythonLibsNew.cmake:98 (find_package):
    Policy CMP0148 is not set: The FindPythonInterp and FindPythonLibs modules
    are removed.  Run "cmake --help-policy CMP0148" for policy details.  Use
    the cmake_policy command to set the policy and suppress this warning.

  Call Stack (most recent call first):
    pybind11/tools/pybind11Tools.cmake:50 (find_package)
    pybind11/tools/pybind11Common.cmake:180 (include)
    pybind11/CMakeLists.txt:208 (include)
  This warning is for project developers.  Use -Wno-dev to suppress it.

  -- Found PythonInterp: /workspaces/calc_gravity_dir/.env_est_gravity/bin/python3.10 (found suitable version "3.10.12", minimum required is "3.6")
  -- Found PythonLibs: python3.10
  -- Performing Test HAS_FLTO
  -- Performing Test HAS_FLTO - Success
  --     OpenCV_DIR: /usr/local/lib/cmake/opencv4
  --     OpenCV_INCLUDE_DIRS: /usr/local/include/opencv4
  --     OpenCV_LIBS: opencv_calib3dopencv_coreopencv_dnnopencv_features2dopencv_flannopencv_gapiopencv_highguiopencv_imgcodecsopencv_imgprocopencv_mlopencv_objdetectopencv_photoopencv_stitchingopencv_videoopencv_videoio
  --     OpenCV_DIR: /usr/local/lib/cmake/opencv4
  --     OpenCV_INCLUDE_DIRS: /usr/local/include/opencv4
  --     OpenCV_LIBS: opencv_calib3dopencv_coreopencv_dnnopencv_features2dopencv_flannopencv_gapiopencv_highguiopencv_imgcodecsopencv_imgprocopencv_mlopencv_objdetectopencv_photoopencv_stitchingopencv_videoopencv_videoioopencv_calib3dopencv_coreopencv_dnnopencv_features2dopencv_flannopencv_gapiopencv_highguiopencv_imgcodecsopencv_imgprocopencv_mlopencv_objdetectopencv_photoopencv_stitchingopencv_videoopencv_videoio
  -- Configuring done (0.4s)
  CMake Error in CMakeLists.txt:
    Imported target "pybind11::module" includes non-existent path

      "/usr/include/python3.10"

    in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

    * The path was deleted, renamed, or moved to another location.

    * An install or uninstall procedure did not complete successfully.

    * The installation package was faulty and references files it does not
    provide.

  -- Generating done (0.0s)
  CMake Generate step failed.  Build files cannot be regenerated correctly.
  Traceback (most recent call last):
    File "/workspaces/calc_gravity_dir/.env_est_gravity/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
      main()
    File "/workspaces/calc_gravity_dir/.env_est_gravity/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File "/workspaces/calc_gravity_dir/.env_est_gravity/lib/python3.10/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 261, in build_wheel
      return _build_backend().build_wheel(wheel_directory, config_settings,
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 404, in build_wheel
      return self._build_with_temp_dir(
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 389, in _build_with_temp_dir
      self.run_setup()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/build_meta.py", line 311, in run_setup
      exec(code, locals())
    File "<string>", line 65, in <module>
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/__init__.py", line 103, in setup
      return distutils.core.setup(**attrs)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 185, in setup
      return run_commands(dist)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
      dist.run_commands()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
      self.run_command(cmd)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/dist.py", line 963, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/wheel/bdist_wheel.py", line 368, in run
      self.run_command("build")
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/dist.py", line 963, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/build.py", line 131, in run
      self.run_command(cmd_name)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/dist.py", line 963, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/command/build_ext.py", line 88, in run
      _build_ext.run(self)
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/build_ext.py", line 345, in run
      self.build_extensions()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/build_ext.py", line 467, in build_extensions
      self._build_extensions_serial()
    File "/tmp/pip-build-env-06gjzad3/overlay/lib/python3.10/site-packages/setuptools/_distutils/command/build_ext.py", line 493, in _build_extensions_serial
      self.build_extension(ext)
    File "<string>", line 55, in build_extension
    File "/usr/lib/python3.10/subprocess.py", line 369, in check_call
      raise CalledProcessError(retcode, cmd)
  subprocess.CalledProcessError: Command '['cmake', '/tmp/pip-install-9ofndmqr/pytlsd_825a1c16d0d64f67b3681539f05fd5c4', '-DCMAKE_LIBRARY_OUTPUT_DIRECTORY=/tmp/pip-install-9ofndmqr/pytlsd_825a1c16d0d64f67b3681539f05fd5c4/build/lib.linux-x86_64-cpython-310/', '-DPYTHON_EXECUTABLE=/workspaces/calc_gravity_dir/.env_est_gravity/bin/python3.10', '-DEXAMPLE_VERSION_INFO=0.0.4', '-DCMAKE_BUILD_TYPE=Release']' returned non-zero exit status 1.
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pytlsd
Building editable for gluestick (pyproject.toml) ... done
Created wheel for gluestick: filename=gluestick-0.0.0-0.editable-py3-none-any.whl size=5688 sha256=e2dce999515bef5e7de6380de217e154b9081dfc85dc8cae31171cf76e9d1d4b
Stored in directory: /tmp/pip-ephem-wheel-cache-v9ns0oim/wheels/21/76/98/7e6c13d375402b61248de409bcd3eb7eba75f184358e9ec8ea
Successfully built gluestick
Failed to build pytlsd
ERROR: Could not build wheels for pytlsd, which is required to install pyproject.toml-based projects
Any suggestions for solving the conflict  ?
THX
BR

This was solved for me by creating a new venv, then installing only pytlsd with pip install -e git+https://github.com/iago-suarez/pytlsd.git#egg=pytlsd first, and then installing the rest.


[ci] Add appetize workflow to check and deploy new simulator builds

Why
We have almost all of the Snack development workflow in Github, except for Appetize. It's a manual step that requires some attention. This should fix that by adding a check & deploy workflow for Appetize.
How
The workflow contains a couple of steps, with small differences for Android and iOS:

Resolve the current version on Appetize (main, embedded, or both queues)
Resolve the client version it should have from SDK version
Download the client for the specified version
Prepare the client for Appetize (iOS only)
Upload the client to Appetize (main, embedded, or both queues)

With the inputs, we have a "double failsafe" to prevent undesired deployments. You both have to specify the exact SDK version to use, and you have to confirm with "deploy".

When omitting both deploy and sdkVersion, only step 1 is executed.
When omitting deploy but defining sdkVersion, steps 1 and 2 are executed.
When defining both deploy and sdkVersion, all steps are executed.

Required Environment variables

I haven't set these up yet!

APPETIZE_MAIN_TOKEN - account token for Appetize main queue
APPETIZE_MAIN_ANDROID - public key for Appetize main Android app
APPETIZE_MAIN_IOS - public key for Appetize main iOS app
APPETIZE_EMBED_TOKEN - account token for Appetize embed queue
APPETIZE_EMBED_ANDROID - public key for Appetize embed Android app
APPETIZE_EMBED_IOS - public key for Appetize embed iOS app

Test Plan
I created this workflow in my own repository, connected to my own Appetize instance. You can see my test runs here. (note, only main queue is set up)

I'll try to refactor the android and ios client resolving, I don't like that I just copied and pasted it. Plus, it's a bit hard to find out what's going on.


How to stop OUTPUT panel from popping up every time I open a new project?

Super annoying...

It looks like it's the eslint extension doing this. Maybe there's an option to control it?

It pops up for all kinds of weird reasons. Is there an option to just permanently disable it auto-popping up?
I'd still like manual access to it, I just hate that it auto-pops up.

I can only speak for ESLint:
It only opens up once per new workspace / project you open to inform that ESLint will not run. Before we never poped it up or poped it up on every workspace open which was even more unhappy (no awareness  versus to much awareness).
You can disable eslint by default "eslint.enable": false in the user settings. Then you need to enable eslint in the workspace you want to.
No further action planned to ESLint. If you see it with other Output channel things might be different.
Please ping if you are asking for a general setting to never bring the Output Panel to front independent of the channel.

@dbaeumer Yeah any solution to keep that from popping up. I think you guys are right, maybe it is just ESLint, but regardless, it's frustrating to have to close it on every ESLint'd project I work on (almost every project I work on).

Duplicate of https://github.com/microsoft/vscode/issues/105270


Unable to find hash in url, timing out / Unable to complete authoriation as there is no interactive call in progress. This can be due to closing the app while the authorization was in process.

Library

[X]<EMAIL_ADDRESS>
Ionic:

   Ionic CLI                     : 6.12.3 (C:\Users\USER\AppData\Roaming\npm\node_modules\@ionic\cli)
   Ionic Framework               : @ionic/angular 5.5.2
   @angular-devkit/build-angular : 0.1002.1
   @angular-devkit/schematics    : 10.0.8
   @angular/cli                  : 10.0.8
   @ionic/angular-toolkit        : 2.3.3

Cordova:

   Cordova CLI       : 10.0.0
   Cordova Platforms : 6.0.0, android 8.1.0, browser
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.2.1, (and 6 other plugins)

Utility:

   cordova-res                          : not installed
   native-run (update available: 1.3.0) : 0.2.7

System:

   NodeJS : v10.15.2 (C:\Program Files\nodejs\node.exe)
   npm    : 4.6.1
   OS     : Windows 10

Framework

[X] Angular 10
[ ] React
[ ] Other

Description
I have setup the APP in azure like below.

When I use localhost as the redirect URI. It works fine on browser. It will of course not work on the phone. The redirect URI with localhost has theid token and everything. When I put the redirect URI from the android section, the login page redirects to my app correctly. Immediately i get the error number 4 from below list as an alert on the phone. The other first 3 errors i see on the console.
It doesnt matter if i put the useHash: true or false I get the error 1 all the time.
It also cant fire the handleRedirectCallback when i use loginredirect.
Error Message

Error monitorPopupForHash unable to find hash in url, timing out
MSALError {_error: "token_renewal_error", _errorDesc: "Token renewal operation failed due to timeout.", _scopes: ""}
{"errorCode":"token_renewal_error","errorMessage":"Token renewal operation failed due to timeout.","name":"ClientAuthError"}
Alert on Android ( Unable to complete authoriation as there is no interactive call in progress. This can be due to closing the app while the authorization was in process.)

MSAL Configuration
MsalModule.forRoot(
      {
        auth: {
          clientId: "xxxx",
          authority: "https://login.microsoftonline.com/common/",
          validateAuthority: true,
          redirectUri: "msauth://com.XXX",
          postLogoutRedirectUri: "msauth://XXX",
          navigateToLoginRequestUrl: true,
        },
        cache: {
          cacheLocation: "localStorage",
        },
      },
      {
        consentScopes: ["user.read", "openid", "profile"],
        protectedResourceMap: protectedResourceMap,
        extraQueryParameters: {},
      }
    ),

Reproduction steps

  ngOnInit(): void {

    this.authService.handleRedirectCallback((authError, response) => {
      if (authError) {
        console.log("Redirect Error: ", authError.errorMessage);
        return;
      }
      console.log("Redirect Success: ", response.accessToken);
    });
    this.broadcastService.subscribe("msal:loginFailure", (payload) => {
      console.log("login failure " + JSON.stringify(payload));
      this.loggedIn = false;
    });

    this.broadcastService.subscribe("msal:loginSuccess", (payload) => {
      console.log("login success " + JSON.stringify(payload));
      this.loggedIn = true;
    });

    this.authService.setLogger(
      new Logger(
        (logLevel, message, piiEnabled) => {
          console.log("MSAL Logging: ", message);
        },
        {
          correlationId: CryptoUtils.createNewGuid(),
          piiLoggingEnabled: false,
        }
      )
    );
  }

  login() {
    this.authService.loginPopup();
  }

  logout() {
    this.authService.logout();
  }
// For Azure B2C issues, please include your policies.

Expected behavior
Identity Provider

[X] Azure AD

Browsers/Environment

[X] Chrome

Source

[X] Customer request

@nomanshahsyed You cannot use an android redirectUri with msal-angular. Since you can't use localhost you'll need to serve your app over https and register that redirectUri as type web

Hi, can't we make it work? The return URL has the token and the navigation to and from the login page works and it redirects correctly to the app.

@nomanshahsyed What happens when you use redirect instead of popup?

Same behavior. Nothing changes.

Specifically what happens? What errors occur? Can you debug to figure out why there is no hash in the url?

There is a Hash in the URL. I saw the hash. Then tried to enable/disable the useHash with routes. Nothing changed.
The errors were same as I posted above. I followed the logcat too and there were same errors. I'm finished for today but tomorrow morning I'll try to run it again and tell you what i see there.
If only the handle redirect function fires, it will tell more info.

I implemented the login with google and that works fine. The google plus and firebaseX plugins also don't have the Microsoft login functions though they have Twitter, Apple etc login.

If you're seeing the same errors then it's not running redirect...those errors are specific to the popup scenario. If the url returns with a hash but the error says it can't find a hash then you need to determine what is clearing the hash. One thing you can try (for the popup scenario only) is to make the redirectUri a blank page that does not implement msal. If the page used as the redirectUri is either removing the hash or initiating a redirect of its own, the calling page will be unable to retrieve the hash and you'll get an error like the one you've shared above.

Ok so for loginPopup the app crashes now whenever this function is called.
For loginredirect i put the localhost as a redirecturi and the returned url has the # along with the code and client info.
When the URI is for the android app from azure page, the app opens up correctly with the following error Alert on Android
( Unable to complete authoriation as there is no interactive call in progress. This can be due to closing the app while the authorization was in process.)

When the URI is for the android app from azure page, the app opens up correctly with the following error Alert on Android
( Unable to complete authoriation as there is no interactive call in progress. This can be due to closing the app while the authorization was in process.)

@nomanshahsyed Can you determine what is throwing that error? I don't think that's one of ours. Do you see any msal errors in the console?


Type poluttion of ListIterator implementation regresses for-in performance

In the real-world applications ListIterator implementation is usually megamorphic (seen more than 4 classes), this leads to a drastic drop in performance because we don't do a good job removing that megamorphism even if we know the type of iterable at the inlining point.
The test below demostrates the drop in performance of the for-in loop due to this type pollution:
$ xcodebuild/ReleaseX64/dart test.dart
Mono 10
Poly 28
Mega 120

import 'dart:collection';

abstract class MyList extends Object with ListMixin implements List<int> {
  final list = [1,2,3,4,5,6];
  get length => list.length;
  operator[](int i) => list[i];
}

class A extends MyList { }
class B extends MyList { }
class C extends MyList { }

class H {
  final list = new List.generate(1000, (i) => i);
}

class H0 extends H {
  sum() {
    var s = 0;
    for (var i in list) s += i;
    return s;
  }
}

class H1 extends H {
  sum() {
    var s = 0;
    for (var i in list) s += i;
    return s;
  }
}

class H2 extends H {
  sum() {
    var s = 0;
    for (var i in list) s += i;
    return s;
  }
}

measure(name, f) {
  for (var i = 0; i < 10000; i++) f();
  final stopwatch = new Stopwatch()..start();
  for (var i = 0; i < 10000; i++) f();
  print("${name} ${stopwatch.elapsedMilliseconds}");
}

main() {
  measure("Mono", new H0().sum);

  for (var i in [1,2,3,4].map((x) => x));
  for (var i in new A());
  for (var i in new B());
  measure("Poly", new H1().sum);  // ListIterator implemenation is polymorphic.

  for (var i in new C());
  measure("Mega", new H2().sum);  // ListIterator implementation is megamorphic
}

I am assigning to myself because we are going to explore approaches to rectify this inside the optimizing compiler, e.g. an additional round of type propagation and a round of inlining would remove the overhead.
However ideally we would like to have a feature that would allow to duplicate ListIterator implementation per iterable type.

Hows it going ?

Hows it going ?

In  what sense?
FWIW ae8e6705c0e1dd624ec45843879294199c11e368 made the microbenchmark from the issue faster by changing underlying representation of the megamorphic call in IL so there is no longer any performance cliff associated with it.


explicit memory initialization independent of argument parsing

The argument parsing needs further refactoring. The current code does not cover all cases of the argument list specification. For now, I removed any code not related to argument parsing and moved it to the proper initialisation phase.

will send pull request to teaching/selfie


Move deabstraction of non-tensorflow convention functions to the TFPartitioning pipeline.

This PR changes the invocation of deabstraction and partitioning passes as follows:
Diagnostic Pipeline:

Deabstraction of tensorflow-convention functions

Perf Pipeline:

Partitioning of tensorflow-convention functions (the next step is to move this to diagnostic pipeline so that we can have a proper -Onone mode)
Deabstraction and partitioning of non-tensorflow convention functions.

To recover most of the previous behavior with respect to sends and receives for non-tensorflow convention functions, the following optimizations are explicitly invoked after deabstraction (see Deabtraction::doIt()):
  PredictableMemoryOptimizations();
  PropagateSSAValues()

Specifically, the TFPartitionPipeline looks like this now:
SILPassPipelinePlan SILPassPipelinePlan::getTFPartitionPassPipeline() {
  SILPassPipelinePlan P;
  P.startPipeline("TensorFlow Partitioning");
  P.addTFDeabstraction();
  P.addTFPartition();
  return P;
}

Moreover, the perfInliner was modified to prevent inlining of some functions that the deabstraction relies on.
Tests:

Ignore the regressions in send/recv warnings in the tests as they are going away.
In some tests, the basic blocks got re-ordered because the deabstraction sees optimized SILFunction for non-tensorflow convention functions.

@swift-ci please test tensorflow

Please hold off further review on this PR. The changes that I did to the performance inliner (unsurprisingly) causes regressions in swift optimizer tests. I am looking into it.

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

Can one of the admins verify this patch?


Cant sync using the flatpak v0.8.10

In order to be able to sync my website to hashbase I usually have to restart beaker for sync to work.
However, with the flatpak v.0.8.10 It doesnt work. Restarting beaker doesnt sync to hashbase.
So theres no way to sync now with the flatpak v.0.8.10

One of the reasons we rebuilt the p2p stack with the new was because of connectivity issues like this. I may have to suggest waiting for the new release.


TAC.py and TAF.py not geting downloaded

Neither are on my PC

https://github.com/Necrownyx/Twitch-Account-Toolbox/archive/9b86bc40b586bdcc88e1ba79ff6e5b1fbc1edfa4.zip


Figure out whether we can determine the sizes of buffers for encoding data model commands better

Problem
Right now we have to guess at a buffer that will be big enough to hold the results of chipZclEncodeZclHeader or equivalent.  It would be ideal if callers did not have to guess at that.
Proposed Solution
I don't have a proposed solution yet.  It depends on the range of different sizes the encoded data might have.
@tecimovic @bhaskar-apple

@bzbarsky-apple Would if be OK if the caller just pass a PacketBuffer * to the underlying command in src/app and the code encoder code has the responsibility to allocate the PacketBuffer itself ? The necessary size can be directly generated from for the commands.

or a packetbuffer allocator, yeah.

Yeah, that would be fine.  The long-term app API  presumably does not involve PacketBuffer if we can avoid it, so starting to hide it now makes sense.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1589 ms

Resolved: Bofrost.be performance has improved in 33d283b.


Seems make 3-finder tap to look up disable!!!

Seems make 3-finder tap to look up disable!!!
If I quit Scroll-reverser, then 3-finder click works well.
Yosemite 10.10.1

Dupe of #1


Add support for filtering results with a unique size for each status code

When attempting to brute force web servers, they may return similar responses for different URLs.
For example, a web server may redirect us to its homepage, giving us the same final result each time for multiple URLs. If we pipeline gobuster's output into other tools, there may be cases where we only care about URLs that point us to unique content.
For example, brute-forcing a web server with the following command:
./gobuster -m dir -q -k -v -l -w wordlist.txt -u https://example.com
And wordlist:
.
./lib
./bar
./car
../lib
../foo
../../bar

Will yield the following:
Missed: /../../bar (Status: 400) [Size: 357]
Missed: /../foo (Status: 400) [Size: 357]
Missed: /./car (Status: 404) [Size: 332]
Found: /./lib (Status: 301) [Size: 370]
Missed: /./bar (Status: 404) [Size: 332]
Missed: /../lib (Status: 400) [Size: 357]
Found: /. (Status: 200) [Size: 132]

Notice that the paths ../../bar and ../foo (likely) result in the same response since they have content of the same size.
There may be cases where I would like to have the following output instead:
Found: /./lib (Status: 301) [Size: 370]
Missed: /../foo (Status: 400) [Size: 357]
Missed: /./car (Status: 404) [Size: 332]
Found: /. (Status: 200) [Size: 132]

Since I only care about URLs that will give me a unique response.
I have implemented a patch that allows gobuster to filter results with a unique size for each status code to addresses the issue above.
If there is anything I have missed, I will be happy to address it in subsequent commits.

I thought about this again and I think it actually might be better to keep the existing gobuster functionality, but instead write the output to JSON or XML and let another tool parse that output. I am investigating #75 to see if I can get gobuster to output JSON instead.
If any of you think this is still a feature worth adding, let me know since my current solution in its current state will not function properly due to race conditions between threads. Additionally, synchronization checks are missing in my first commit. I also think that with my current solution, the amount of mutex locking/unlocking to achieve synchronization will also impact performance and hence should be designed differently.

@itsbriany I find a lot of use for this feature and I wouldn't see a reason why it wouldn't benefit the whole package itself instead of having a separate tool for it.


"BarcodeScanner" plugin is not implemented on android

Describe the bug
Not at liberty to share the repo at the moment. The bug is the fact that BarcodeScanner plugin itself, not its functions, never get defined or attached to the built CapacitorJS app. I have followed through every step on the README and make sure to use the right version of lib and capacitor (see below).
To Reproduce
I set my codebase to toast error message using vue-toastification So when I mount the vue component with the "BarcodeScanner" the modal pops up and begin scanning. Attached codebase below:
import { BarcodeScanner, SupportedFormat } from '@capacitor-community/barcode-scanner'
import { modalController } from '@ionic/vue'
import { onMounted } from 'vue'
import { useToast } from 'vue-toastification'
...
// funcs
let dismiss = async () => {
  await Promise.all([
    BarcodeScanner.stopScan().catch(() => void 0),
    modalController.dismiss(null, 'cancel')
  ])
}

// lifecycle hooks
onMounted(async () => {
  let hasPermission = await BarcodeScanner.checkPermission({ force: true })
    .then((permission: { granted?: boolean }) => permission.granted)
    .catch((err: {message: string}) => toast.error(err.message) || false)
  if (!!hasPermission) {
    let prepared = await BarcodeScanner.prepare()
      .then(() => true)
      .catch((err: { message: string }) => toast.error(err.message) || false)
    if (!prepared) return await dismiss()
    let mask = document.getElementById('mask')
    let videoNode = document.getElementById('video')
    let container = videoNode ? videoNode.parentElement : null
    if (!!mask && !!videoNode) {
      mask.insertAdjacentElement('beforeend', videoNode)
    }
    if (!!container) {
      document.body.removeChild(container)
    }
    let result = await BarcodeScanner
      .startScan({ targetedFormats: [SupportedFormat.QR_CODE] })
      .catch(err => toast.error('Errored scanning') || false)
    if (!result) return await dismiss()
    // @ts-ignore
    if (result.hasContent) {
      await BarcodeScanner.stopScan().catch(() => void 0)
      // @ts-ignore
      await modalController.dismiss(result.content, 'success')
    }
  } else {
    toast.warning('Not permitted')
    await dismiss()
  }
})

Expected behavior
To make sure this bug does not get dismissed for faulty process setup or ionic visibility, I make sure to get it working on web environment. (Chrome only, not Firefox) Attached is the expected behavior.

Screenshots

Version

@capacitor/core: 4.8.0
@capacitor/android: 4.8.0
@capacitor-community/barcode-scanner: 3.0.3

Desktop (please complete the following information):

OS: macOS Ventura Version 13.0.1
Browser: Chrome
Version: 112.0.5615.137 (Official Build) (arm64)

Smartphone (please complete the following information):

Device: OnePlus 7 (GM1900) and Android Emulator on MacBook Pro M1 ("system-images;android-33;google_apis;arm64-v8a")
OS: HydrogenOS (OP7) and Stock Android (Emulator)
Browser: N/A
Version: Android Version 11 (OP7) and N/A
WebView: Android System WebView (112.0.5615.136)

Additional context
Seems I may be redirected toward having multiple package.json file on my monorepo but I can assure that is not the case.
Attached is the outputs of the npx cap sync process.

Noted with thanks. You can also assume I had been testing "clean-ups" with the following setup permutations as well.
Node version: [16, 18] (unable to use node v14 because vue-tsc does not build)
Package manager: [npm, yarn]
@ionic/core & @ionic/vue & @ionic/vue-router: [6.7.3, 6.7.4, 7.0.3]
@capacitor/android & @capacitor/cli & @capacitor/core: [4.1.1, 4.3.0, 4.5.0, 4.6.0, 4.7.0, 4.8.0]
With no luck.
If you have any idea what is the culprit for the unexpected build behavior, whether it's using an aarch64 build environment, or including a cordova plugin, using latest @capacitor/* libs or latest @ionic/*, it would be very helpful.

The more I look into it, the more I find that my android build has none of the plugins defined in build steps.
Will pursue workaround recommended by Ionic Troubleshooting recommendations instead.

https://capacitorjs.com/docs/android/troubleshooting#plugin-not-implemented
https://github.com/ionic-team/capacitor/issues/1655#issuecomment-579229390
https://stackoverflow.com/questions/55894716/how-to-package-a-hosted-web-app-with-ionic-capacitor

@aekasitt I think this is a good place to start with yes, I am pretty sure it has something to do with the Android setup. One thing came to my mind that helped me once I had this issue was to resync my android studio project with the gradle files. Maybe this might help in your case as well. Unfortunately I don't really have other suggestions.
If you can resolve your issue, it would be awesome if you could post a quick note here, it might really help if someone else runs into this problem in the future!
Thanks!


"Dasherize" resource attributes

Hi there, just recommending that keys given to Resource#attr be dasherized by default. This makes serialization same with the jsonapi-resources gem, and since the official spec hints at this convention anyway. Thanks!

@vinnicc yeah the http://jsonapi.org/recommendations/#naming page recommends separating words an an name with the minus sign (dash). Perhaps an assert can be used to suggest via console warn that the recommended naming convention is to use dasherized attributes.


How to send a message to a single user？

How to send a message to a single user？

You can use Socketcast to send messages to individual sockets.

Is that right? But I can't receive the message.