Wasm: question: how best to make CoreRT a drop in replacement regards WebAssembly.Runtime.InvokeJS

The Uno Platform makes use of https://github.com/mono/mono/blob/59a1eade7ce4fb79c56a47d043151fbdd5363b1b/sdks/wasm/src/driver.c#L323 for JS interop.  It does this here: https://github.com/unoplatform/uno/blob/1dede9e628108c7c7b71d6aee4cd759c0f9bf9d9/src/Uno.Foundation/Runtime.wasm.cs#L21.
How would that be best handled so that CoreRT could be a drop in replacement for mono?  Could it intercept the call to WebAssembly.Runtime.InvokeJS in https://github.com/dotnet/corert/blob/f52a8e504b0485dff9e4737a6fb9535d5eccb77d/src/ILCompiler.WebAssembly/src/CodeGen/ILToWebAssemblyImporter.cs#L1575 and replace with a call to a similar method perhaps in https://github.com/yowl/corert/blob/uno-js-llvm/src/CoreRT.WebAssembly.Interop/InternalCalls.cs.  Not asking how to actually get into JS, but what would be the right way to start.
Thanks.

Yes, redirecting in ImportCall around the existing places that handle IsInternalCall sounds right. The simplest thing would be to redirect the InternalCall to a managed method elsewhere like you propose. I assume the managed implementation will just call a DllImport method implemented in C/C++ that delegates to Emscripten. The C/C++ method could live under src\Native, like the other System.Private.*.Native libraries.


Make an Android version of YouTube-dl

Here's a long-term wish list item: make an Android version of YouTube-dl.

(Dentex's no longer works.)

You know, just like there was originally only a desktop version of the Chrome browser. And then they made an Android version to!

It works just fine on android.

I mean an .APK!

I mean an .APK!
Like VLC, available for both desktop and mobile.

I wouldn't hold high hopes, at least for Play Store: I checked, there're exactly 0 downloaders that claim to support YT, but there's numerous downloaders.

Yes, it would never be allowed on Play Store. Because Play Store is owned by Google which owns YouTube.
But that shouldn't stop hopes for a future .APK.


Scala Hadoop Enrich: bump Scala Common Enrich to 0.12.0

Duplicate of #1369

Not a duplicate - see titles


list_tables_sql_db is not a valid tool, try another one.

System Info
langchain: 0.0.215
python: 3.10.11
OS: Ubuntu 18.04
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
While querying a SQL database, the agent gets stuck in an infinite loop due to list_tables_sql_db not being a valid tool.
> Entering new  chain...
Action: list_tables_sql_db
Action Input:
Observation: list_tables_sql_db is not a valid tool, try another one.
Thought:I should look at the tables in the database to see what I can query.  Then I should query the schema of the most relevant tables.
Action: list_tables_sql_db
Action Input:
Observation: list_tables_sql_db is not a valid tool, try another one.
Thought:I should look at the tables in the database to see what I can query.  Then I should query the schema of the most relevant tables.
Action: list_tables_sql_db
Action Input:
Observation: list_tables_sql_db is not a valid tool, try another one.
Thought:I don't know how to answer this question.
Thought: I now know the final answer
Final Answer: I don't know

> Finished chain.

Expected behavior
The agent should get the list of tables by using the list_tables_sql_db tool and then query the most relevant one.

yes it is still not resolved
I am uning the latest version

Hi , Im also facing the same issue , is there any solution available ?


Use TTA Mode with OpenCL?

Hello,
unfortunately, I have a little problem to use waifu2x-caffe, because I use Intel HD Graphic via windows and not Nvidia, so it isn´t possible for me to use Cuda.
But I read, with OpenCL there would be another way, to use the GPU, but don´t know, if I could us waifu2x-caffe with it.
I would appreciate every little hint.
And thank you very much in advance.
nightingale

https://github.com/DeadSix27/waifu2x-converter-cpp supports OpenCL but it does not support latest waifu2x model.

Thank you very much.
I realized a little bit too late, that it would be also possible to run it with the CPU.
It lasts pretty long, but it doesn´t matter, because, it is such a wonderful little program and I can´t thank you enough for your efforts, which you invested in this project.:)
Dear Greetings
nightingale

Thank you


Command to install xcsync fails as tool is still in preview

Type of issue
Code doesn't work
Description
The article states to install the tool globally to use this command:
dotnet tool install -g dotnet-xcsync
However, that command will not work as the nuget package is in preview. The command will fail with:
dotnet-xcsync is not found in NuGet feeds https://api.nuget.org/v3/index.json.
Both this documentation and the documentation at https://learn.microsoft.com/en-us/dotnet/maui/whats-new/dotnet-9?view=net-maui-9.0 indicate this tool was released with .Net 9 but it does not appear this was the case. It is still in preview and the tool cannot be downloaded as a released tool.
The nuget package can be found here in preview and contains the command that can be used to download the preview package:
https://www.nuget.org/packages/dotnet-xcsync/0.1.0-preview.24509.2
The documentation in both places should be updated with the fact that this is in preview and with information on the correct way to download the preview.
Page URL
https://learn.microsoft.com/en-us/dotnet/maui/macios/xcsync?view=net-maui-9.0&tabs=visual-studio-code
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/macios/xcsync.md
Document Version Independent Id
d5f511c6-2870-5ca6-400e-afa302e40146
Article author
@haritha-mohan
Metadata

ID: d5f511c6-2870-5ca6-400e-afa302e40146
Service: dotnet-mobile
Sub-service: dotnet-maui

Related Issues

Associated WorkItem - 356253

@Bowman74 Agreed that you're correct, and thanks for pointing this out.

@davidbritch Thank you sir!


WebRequestLoader does not support proxy server

Hello,
Currently HttpClient in WebRequestLoader  is private,
And WebRequestLoader does not provide any method to support proxy server.
If the program is running behind the proxy, then there is no way to download the asset from web.
Thank you.
example code,
proxy = new WebProxy
{
	Address = new Uri(proxyUrl),
	UseDefaultCredentials = true,
};

httpClientHandler = new HttpClientHandler
{
	Proxy = proxy,
};

httpClient = new HttpClient(handler: httpClientHandler, disposeHandler: true);

Under proxy few web links will be blocked, so you can check if github is allowed. Try loading a gltf model from git url, if that doesn't work ask the network admin to unblock the links for the gltf.

I mean that gltf library is needed to provide the method to set the proxy settings in API.


Shortest Common Supersequence

Submission Title
Shortest Common Supersequence
Submission Type
Algorithm
Submission Description
This algorithm finds the shortest common supersequence of 2 strings - the shortest string that has both str1 and str2 as subsequences. If 'm' is the length of the first string and 'n' is the length of the second string, then this algo takes O(m * n) time and O(m * n) space.
Submission Directory
Algorithms => Dynamic_Programming => Shortest_Common_Supersequence => shortest_common_supersequence.ext
Contributors

[x]  C++ -
[x]  C -
[x]  Python -
[x] Java -
[x] JavaScript -
[x] TypeScript -
[x] Golang -
[x] Ruby -
[x] php -
[x] Swift -
[x] C# -

@ahampriyanshu Please assign this issue to me for C++. Will be happy to help.

@ahampriyanshu I would like to work on this issue in python, It will be great if you can assign it to me,

Can I work in java,pls ?

Please assign this to me I'll work on this in C

@ahampriyanshu  Please assign this issue to me for C++

Hi, if not assigned to anyone, can i work on C++?

@Swarn10  please assigned me this in python

hi @sanjiti0606 , @IamJayshree , @aakashsoni-cloud  Please opt different language

@Mr-Vaibhav-Shukla  assign me for C#

can I do golang? @Mr-Vaibhav-Shukla

i would like to contribute for javascript solution

thanks everyone :v:


no error logging when capistrano task can't clone the repo

if you give the github auth key incorrect permissions (and so cloning the repo 403s), then the gist output is empty, and there is no log anywhere of the error.
Two suggestions:

always log stdout/stderr to Rails.logger when a job fails (I can PR my code for this if you'd like)
This shouldn't result in no gist output. Entirely unsure why that's the case.

I believe this should be resolved with #115.


Use predicates to handle failed devworkspaces

What does this PR do?
Update how failed workspaces are handled

Predicates filter events so that failed workspaces do not trigger reconciles
...unless the routingClass is updated.

What issues does this PR fix or reference?
Don't recall if we had an issue for this one
Is it tested? How?
Easiest to test on k8s:
make docker install # or restart if already running
kubectl apply -f samples/cloud-shell
kubectl get dw # cloud shell is failed
kubectl edit dw cloud-shell # editing devworkspace does not trigger reconciles
kubectl patch dw cloud-shell --type merge --patch '{"spec": {"routingClass": "basic"}}'
kubectl get dw # cloud-shell is starting

@sleshchenko Good catch on the error -- I've fixed it (and other common issues) by reconciling whenever the spec is changed, so now we can either update routingClass, or fix a plugin ID, etc.

/test v5-devworkspaces-operator-e2e

so now we can either update routingClass, or fix a plugin ID, etc.

Well, we can update whatever we need to fix workspace spec but it won't help workspace to recover startup. See below for instructions on how to reproduce.
cat <<EOF | kubectl apply -f -
kind: DevWorkspace
apiVersion: workspace.devfile.io/v1alpha2
metadata:
  name: theia
spec:
  started: true
  template:
    projects:
      - name: project
        git:
          remotes:
            origin: "https://github.com/che-samples/web-nodejs-sample.git"
    components:
      - name: theia
        plugin:
          id: eclipse/che-theia/next
      - name: terminal
        plugin:
          id: eclipse/che-machine-exec-plugin/non-existing
EOF

# 2. Check it's failing
sleep 10
kubectl get dw theia -o=yaml | yq '.status'

# 3.
kubectl patch dw theia --type merge --patch '
spec:
  template:
    components:
      - name: theia
        plugin:
          id: eclipse/che-theia/next
      - name: terminal
        plugin:
          id: eclipse/che-machine-exec-plugin/nightly
'

# 4. Check it's not failing anymore (IT IS IN THIS PR)
sleep 10
kubectl get dw theia -o=yaml | yq '.status'

OUTPUT:
devworkspace.workspace.devfile.io/theia created
{
  "conditions": [
    {
      "lastTransitionTime": "2020-12-22T09:09:15Z",
      "message": "Failed to download plugin meta.yaml for terminal",
      "status": "True",
      "type": "FailedStart"
    }
  ],
  "phase": "Failed",
  "workspaceId": "workspaceb93dcb174bfe4c2f"
}
devworkspace.workspace.devfile.io/theia patched
{
  "conditions": [
    {
      "lastTransitionTime": "2020-12-22T09:09:20Z",
      "message": "Failed to download plugin meta.yaml for terminal",
      "status": "True",
      "type": "FailedStart"
    }
  ],
  "phase": "Failed",
  "workspaceId": "workspaceb93dcb174bfe4c2f"
}

It's caused by the way we update components statuses. I've pushed the fix https://github.com/devfile/devworkspace-operator/pull/229/commits/98a7275d9f56cd1df3f2b8cacc8bae6de575ee60 but feel free to improve it if you see any issue there.

/test v5-devworkspaces-operator-e2e

Thanks @sleshchenko for your patience -- I didn't read your original issue thoroughly enough. That's what I get for not testing the specific case before pushing.

/test v5-devworkspaces-operator-e2e


Some completions do not work well on VS 2022.

Environment

Actual result

When I input "ctxt", do not find "Content"

Member completions do not appear automatically

I also encountered a similar problem. When I selected Module2, the previous print was replaced.

decimal.Context exists, decimal.ctx will bring it up. please file separate issue if they still occur


Cannot Install Mitaka r13.1.2

Do NOT include any confidential information, this issue will be created on a public repo
This section is completed by the issue reporter
On which version(s) do you encounter this issue?
Mitaka r13.1.2
Please provide the output to 'git describe --all' for  all deployments where you have confirmed it occurs.
13.3.8-1-g14762c3
What type of issue is this? Please tick the appropriate box.

[X ] bug - existing functionality that does not work for you as documented

What is the expected behavior?
setup-infrastructure.yml completes without error and OpenStack is installed
What is actually happening?
setup-infrastructure.yml fails to complete. Specifically, the play fails at the repo_build.yml task during creation of the OpenStack-Ansible requirement wheels
What were you attempting to do when you encountered this error (give as much detail as possible)?
Attempting to install Mitaka r13.1.2 at the point where it was time to run the setup-infrastructure.yml play after following these steps:
cd /opt/rpc-openstack/openstack-ansible/playbooks
openstack-ansible setup-hosts.yml
cd ../../rpcd/playbooks ; openstack-ansible pip-lockdown.yml
cd - ; openstack-ansible setup-infrastructure.yml
How can we reproduce this issue?
After git cloning Mitaka 13.1.2, bootstrap the deployment host and then proceed to run either deploy.sh or the individual plays until you reach setup-infrastructure.yml
How can we verify your issue is addressed?
Mitaka is successfully installed and no errors are encountered during the various plays
If you are using or know of a workaround please describe the necessary steps?
Edit /opt/rpc-openstack/rpcd/playbooks/roles/horizon_extensions/defaults/main.yml and change the following variable from 'master' to "mitaka-13.1" like this:
horizon_extensions_git_install_branch: "mitaka-13.1"
What is your evaluation on the importance of this issue, and why?
Critical, self-explanatory
This section is completed during the issue resolution process. If you are reporting an issue, please do not modify the section below.
Which branches are impacted by this issue?
Remove branches that are not impacted.
For each pull request, update the branch line to reference it.

[ ] master
[ ] mitaka-13.1
[ ] liberty-12.2
[ ] kilo

Upgrade impact. Consider the impact on moving between major, minor and patch releases.

[ ] This change does not impact upgrades
[ ] This change addresses a potential upgrade impact
[ ] The upgrade impact is being tracked by another issue.

The issue tracking the upgrade impact is:
Testing impact

[ ] An issue/pull-request has been create or exists to track the testing gap, and is referenced below
[ ] This change includes additional testing
[ ] This change is tested by the existing setup

Additional testing links:
Release note

[ ] This change includes a release note.
[ ] This change does not require a release note.

Doc Impact
Does your change require documentation?

[ ] Yes, and I submitted an issue to Docs issues
[ ] Yes, and I contributed a documentation patch to docs-rpc
[ ] No documentation required for this change

Additional documentation links (please add links if you have ticked yes):

This is my fault. The problem was fixed in https://github.com/rcbops/rpc-openstack/pull/1658, I just forgot that I needed to create a release after the fix was made.
I'll create a release soon.

Blocked by https://github.com/rcbops/rpc-openstack/pull/1679

Unblocked now

Blocked again by: https://github.com/rcbops/rpc-openstack/issues/1663

Please use the latest release, r13.1.3. This problem is fixed in that tag.
https://github.com/rcbops/rpc-openstack/releases/tag/r13.1.3


Error during training validation: AttributeError: 'NoneType' object has no attribute 'item'

Dear author, thank you very much for your excellent work on this project. When I train my own SGDet model, I encounter two errors during the validation phase.
No.1 is as follows:
Traceback (most recent call last):   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 994, in <module>     main()   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 973, in main     model, best_checkpoint = train(                              ^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 704, in train     run_val(cfg, model, val_data_loaders, args['distributed'], logger, device=device)   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 843, in run_val     if len(dataset_result) == 1:        ^^^^^^^^^^^^^^^^^^Traceback (most recent call last): ^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 994, in <module> TypeError: object of type 'float' has no len()     main()   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 973, in main     model, best_checkpoint = train(                              ^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 704, in train     run_val(cfg, model, val_data_loaders, args['distributed'], logger, device=device)   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 848, in run_val     dataset_result[k1][k2] = torch.distributed.all_reduce(torch.tensor(np.mean(v2)).to(device).unsqueeze(0)).item() / torch.distributed.get_world_size()                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'item'
No.2 is as follows:
Traceback (most recent call last):   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1514, in <module>     main()   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1493, in main     model, best_checkpoint = train(                              ^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1253, in train     val_result = run_val(cfg, model, val_data_loaders, args['distributed'], logger)                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1363, in run_val     if len(dataset_result) == 1:        ^^^^^^^^^^^^^^^^^^^ TypeError: object of type 'float' has no len() Traceback (most recent call last):   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1514, in <module>     main()   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1493, in main     model, best_checkpoint = train(                              ^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1253, in train     val_result = run_val(cfg, model, val_data_loaders, args['distributed'], logger)                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Dpan/wyc/realtime_rwsg/SGG-Benchmark/tools/relation_train_net.py", line 1368, in run_val     dataset_result[k1][k2] = torch.distributed.all_reduce(torch.tensor(np.mean(v2)).to(device).unsqueeze(0)).item() / torch.distributed.get_world_size()                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "/Dpan/wyc/anaconda3/envs/rtrw_sg/lib/python3.11/site-packages/torch/distributed/c10d_logger.py", line 72, in wrapper     return func(*args, **kwargs)            ^^^^^^^^^^^^^^^^^^^^^   File "/Dpan/wyc/anaconda3/envs/rtrw_sg/lib/python3.11/site-packages/torch/distributed/distributed_c10d.py", line 1992, in all_reduce     work = group.allreduce([tensor], opts)            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ RuntimeError: No backend type associated with device type cpu
Could you tell me how to solve them???Thank you very much!!!!!!!!

Yes there may be some issues if you want to train with multiple gpus. I will investigate that another day, in the meantime try to run the training on a single gpu, it should work.


Added MSI Path parameter

Added a parameter to specify the path to the Unified Agent's MSI file as well as adding some additional error checks to handle if the msi (dev or regular) cannot be found at the path and stop the script's execution.
Script's signature was not updated.

@pahuijbr adding the path for the MSI does not specifically enable a blocked scenario but does align with the way the onboarding script's path can be provided, and it does allow customers to place the MSI in the same location they may host other installer files in their organization.  This also help clarify where the MSI should be loaded, otherwise users of the script have to read the entire script to determine the right location for placing the MSI.
The other change is that the original script only checks if it is not in dev mode and the MSI exists.  However, if the MSI does not exist then the script assumes the use of Dev Mode and attempts to execute the Dev MSI (regardless of if $DevMode is present or if the devmode msi exists).  The update checks the status of DevMode along with the DevMode MSI path to validate the intent and existence and if those checks fail then the script will report an error rather than proceeding.

@pahuijbr - Another issue if the customer is putting the script into SCCM there is no way to control where the MSI is going to be located.  Solution is the customer must manually edit the script to the location where SCCM could access it.

Please check latest script revision, should solve for same/similar issue


#bodhi Missing EID in Telemetry event JSON created on submitting feedback for content

When user submits feedback after viewing content, the value for the key eid in the telemetry event JSON  is blank.

Feedback of the content is not generating any event in portal.
Content feedback functionality will work only in mobile(sending feedback telemetry to Genie-SDK)
@harishgilimi @vrayulu @AmiableAnil

@harishgilimi what are the next steps for this issue?

@kochhar I have narrowed down the error the sendFeedback method in TelemetryV2Manager. It is making a call to createEvent with empty EID which is the first parameter
The sendFeedback method definition:
this.sendFeedback = function(eks) {
        return this.createEvent("", eks);
    };

The createEvent method signature
this.createEvent = function(eventName, body, consumer)

@vinukumar-vs @harishgilimi I believe the correct EID should be "OE_FEEDBACK". Could you please confirm?

@kochhar
As I commented above, Genie-Canvas(ContentRenderer) is not logging any OE_FEEDBACK event(from end page rating). Content player is logging only GE_FEEDBACK when it is playing in mobile.
When the content is playing in portal(using ContentRenderer), there is no event generating feedback(OE_FEEDBACK) from the end-page as of now.
@mathewjpallan @harishgilimi @anujtech46 @AmiableAnil @sferoz

@vinukumar-vs thanks for the confirmation that the eid should be OE_FEEDBACK. let's file a GenieCanvas bug.

Issue moved to project-sunbird/sunbird-portal #227 via ZenHub


Multiple output regression

Do you guys have a multiple output regression? Lets say input is (x^0,...,x^n) output is (y^0,...,y^m), where the training table consists of (x^0,...,x^n,y^0,...,y^m, someother stuffs)
Or do you guys have matrix utilities like svd svm mat inverse etc?

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

ID: 008a45d2-e86b-1ea9-ef79-10f79f6cba40
Version Independent ID: 94c0d9c3-3c9e-8b9c-2288-6215f9f8c122
Content: Tutorial: Predict prices using regression with Model Builder - ML.NET
Content Source: docs/machine-learning/tutorials/predict-prices-with-model-builder.md
Product: dotnet-ml
GitHub Login: @luisquintanilla
Microsoft Alias: luquinta

Hi @hoksource
Thank you for your question. ML.NET currently does not support this scenario. What you can do is something like the following sample.
https://github.com/zeahmed/DeepLearningWithMLdotNet/tree/master/NYCTaxiMultiOutputRegression
If you have a TensorFlow / ONNX model that performs this task, you can use that model inside ML.NET. Keep in mind, this sample was created before version 1.0 of the API, so a few things may not work when you use the current version of ML.NET. However, the concepts should still apply.
I would encourage raising an issue on the dotnet/machinelearning repository requesting this feature.

Thank you for the response. Looking forward for this feature within the year or 2.


[Blockly][Codegenerator] for loop with variable not working

The variable in the for loop does not get initialed and causes compilation errors.

lead to the following code:
for (i = 1; i <= 10; i += 1) {
}

initialization of i is missing.

closed #318


Github SSO button

Is your feature request related to a problem? Please describe.
We need a Github SSO button in Accounts-UI login similar to how Google SSO button was implemented
Describe the solution you'd like
A button with a link to these endpoints that is based upon the environment:

https://auth.api.dev.zesty.io/github/login
https://auth.api.stage.zesty.io/github/login
https://auth.api.zesty.io/github/login

Additional context
Project board
Tech Spec
@darwin808 Let me know if there's any questions. Tagging @zcolah and @shrunyan here as well.

@darwin808
If the github user has no name in github public profile page, this error is being returned:
{"message":"Name of Github account has not been set.","status":"Bad request","data":null,"meta":{},"code":400}
Could this be handled by the UI to show this error message?

@shrunyan
Tagging you here. See my comment above, UI needs to handle the error {"message":"Name of Github account has not been set.","status":"Bad request","data":null,"meta":{},"code":400} which occurs when the github user has no name in [github public profile page](https://github.com/settings/profile).
Can you advise the best way to handle this error in the UI?

@mattezra @giseleblair @ardeay
Im getting an error in  https://beta.zesty.io/

@darwin808 @giseleblair @ardeay
I tested this and confirmed that it has been fixed when the env var was added to prod.
Can you test again?


Add ramp up/down for matmul

Feature
For di/dt testing, we want to be able to configure matmul to ramp up/down at the beginning and end to smooth out current draw. This is an example of what it will look look:

loop 1: 8 cores active
loop 2: 16 cores active
loop 3: 32 cores active
loop 4 to N-3: all cores active
loop N-2: 32 cores active
loop N-1: 16 cores active
loop N: 8 cores active

For implementation, we will group matmul cores together and cores within a group sync with another (ie. only one core within the group can run compute). As we increase active cores, we will "shrink" the group size and vice versa. Ideally, we can configure two things:

Maximum number of active cores at the beginning and end of the matmul
How fast we ramp up/down (ie. by factor of 2, 4, etc...)

Implementation
Branch: bliu/didt
For sharded 1D or 2D matmuls, there are two orthogonal features that you can try out by adding these settings in your environment variables:
TT_MATMUL_RAMP_CONFIG='{"enable_dummy_loops": true, "enable_ramp": false, "initial_cores": 8, "ramp_multiple": 2}'

enable_dummy_loops: If true, insert additional loops of dummy matmul_block compute at the beginning and end of actual matmul loops to gradually turn on/off cores. In the middle, run actual matmul at full capacity.
enable_ramp: If true, puts the actual matmul loops into a start, middle, and end phase where cores are grouped and are gradually turned on/off within the group.
initial_cores: How many cores to start/end with. This is used to determine the group size. ie. If 64 cores total, initial cores of 8 means group size of 8; if 56 cores total, initial cores of 8 means group size of 7.
ramp_multiple: How fast we turn on/off cores. If initial cores of 8 and ramp multiple of 2, you the second loop 16 cores will be running, etc... This applies to both enable_dummy_loops and enable_ramp feature, although they are implemented slightly differently.

NOTE: You can probably run both features together, but I haven't tested that.
Test Results with PMON Capture
Test are run on sjc-snva-t3005 with chip 0, since this is the chip that consistently hangs in the 8-chip hang repro.
PMON visualization:

PMON fit:

Enable ramp tests
I modified the test to run only 5000 loops for captures instead of 10000. Test command is this (initial_cores is varied for experiments):
TT_MATMUL_RAMP_CONFIG='{"enable_dummy_loops": false, "enable_ramp": true, "initial_cores": 8, "ramp_multiple": 2}'
WH_ARCH_YAML=wormhole_b0_80_arch_eth_dispatch.yaml pytest tests/didt/test_sharded_ff1.py::test_specific_chip_reproduce_matmul_2d_hang_t3000[logical_chip0]

Baseline (ie. don't set anything related to TT_MATMUL_RAMP_CONFIG):

Baseline with stagger (TT_ENABLE_MATMUL_STAGGER=1):

Initial cores 1:

Initial cores 8:

Initial cores 32:

Enable dummy loops tests
IMPORTANT EDIT: I think I may have messed up my implementation of these dummy loops. Each core acts independently of one another. So apart from some basic overhead of arithmetic on risc, the cores may be starting all at roughly the same time, with the "staggered" cores turning off near the end of ramp up. During ramp down, this should be fine as is since cores all start off doing work and slowly drop off. To properly implement the ramp up, we need to do some syncing between cores. We can do this by using simple mcasting:

Add the same dummy loops to reader kernel of all cores
Add syncs between reader and compute
Control the loops by using core 0 (since it should always be running every dummy loop). Core 0 is sender, all other cores are receivers. Every time core 0 is about to start compute, mcast VALID to all other cores to let them know it is the next loop.

Single chip tests
Same as before except I turn on enable_dummy_loops and turn off enable_ramp:
TT_MATMUL_RAMP_CONFIG='{"enable_dummy_loops": true, "enable_ramp": false, "initial_cores": 8, "ramp_multiple": 2}'
WH_ARCH_YAML=wormhole_b0_80_arch_eth_dispatch.yaml pytest tests/didt/test_sharded_ff1.py::test_specific_chip_reproduce_matmul_2d_hang_t3000[logical_chip0]

Baseline:

Initial cores 1:

Initial cores 8:

Multichip tests
Intent here is to see if voltage is different when running multi-chip workloads vs. single chip. PMON by default captures from chip 0, which happens to be the chip I am running my single chip tests on. If you are running single chip tests on another chip, you need to switch which device you are running PMON from.
Same environment setup as single chip tests, but switch the pytest option to the 8 chips one:
TT_MATMUL_RAMP_CONFIG='{"enable_dummy_loops": true, "enable_ramp": false, "initial_cores": 8, "ramp_multiple": 2}'
WH_ARCH_YAML=wormhole_b0_80_arch_eth_dispatch.yaml pytest tests/didt/test_sharded_ff1.py::test_reproduce_matmul_2d_hang[8chips-ff1-hang]

Baseline:

Initial cores 1:

Initial cores 8:

TLDR:

enable_ramp: This features makes cores alternate compute for initial and tail-end matmul loops, which may not be ideal for power usage. So I started with a simpler approach of adding dummy loops at the beginning and end.
enable_dummy_loops: Feature was not implemented correctly. We need to sync the dummy loops across cores with mcasting:

Add the same dummy loops to reader kernel of all cores
Add syncs between reader and compute
Control the loops by using core 0 (since it should always be running every dummy loop). Core 0 is sender, all other cores are receivers. Every time core 0 is about to start compute, mcast VALID to all other cores to let them know it is the next loop.

multi-chip tests: Voltage usgae for multi-chip vs. single chip tests looks different. Needs more testing.

The example used in the issue description (ff1 matmul) triggers AICLK throttling, so nominal voltage is not constant. I repeated the experiment (enable ramp experiment, not the dummy loops one) with fixed voltage and AICLK (850mV and 650MHz). I did the experiment on both N150 and N300 (both chips), and observed similar behavior, so I am adding results from N150 only (T3008 board 0).
The biggest droops are always the first time all 64 cores start working, regardless of whether we had some ramp up work before or not. Since we are doing synchronization between cores in ramp-up iterations, they all more or less finish at the same time, and when all 64 start working we have similar droop as we do in the baseline case. Seems that the fact that we had some workload running on less cores before we turn on all 64 of them doesn't reduce the droop meaningfully. @TT-BrianLiu @davorchap @ttmtrajkovic
To confirm this, in each case I took 6 captures with PMON script which sum up to ~200-300 iterations of matmul, and I didn't see a meaningful reduction in the droop nor any correlation between the ramp-up step/starting point and droop we see when all 64 cores finally start.
Baseline case
The red square shows one iteration of FF1 matmul (48 blocks). The biggest droop is always when all 64 start doing the first block, although it isn't the same for every iteration. Minimum of the iterations I captured is 782.6mV.

Ramp-up starting with 32 cores
The red square shows 2 phases of 32-core iteration. Still, the biggest droop that happens is the one when all 64 cores start, and the minimal one I saw in captured iterations is 783.8mV.

Ramp ups starting with 8 or 16 with step 2 behave in a similar way. Ramp-up starting with 16 cores and step 4 showed worse droop - minimal was around 780mV.
Dummy loops
Regarding the dummy loops solution, the results show that we increase the overshoot problem when running these loops in the beginning. This is with current solution. Regarding sync you mentioned, where all cores in group would sync for each new iteration - I am not sure I understand the reasoning. With sync, seems to me that all cores would turn off at the same time again before the 64-core matmul starts. The difference I see is that we would switch from 32+ cores to 64 in the last step, instead of going 32->64, but I am probably missing something - let me know. @TT-BrianLiu

Update: Running ramped-up FF1 without GELU workload on a board that fails still results in a fail, although it happens in a later iteration. Tried 32 and 16 ramp up with multiple 2.

LM head example still hangs as well, the failing rate being unchanged.


Fix guards in a few files

This fixes most outstanding issues in #89. There's plenty of outstanding issues with cudaMemCpy but those are less likely to fail silently compared to cudaMalloc calls.

Those silent fails are also due to exceptions not passing through thread termination. This is something @kpu might want to take a look at when he's back.

@emjotde : I need to find time and converse nth_element into a node look-alike object and withdraw memory allocation in this place. I need to have some workspace memory in this node. It seems to be ease to implement.


Allow DefaultLogger to integrate better with newer versions of SLF4J

Description
Have DefaultLogger in azure-core implementation implement Logger instead of MarkerIgnoringBase as it is deprecated in SLF4J 2.0.0+. This should allow for azure-core to integrate better with newer versions of SLF4J in case MarkerIgnoringBase is removed in later versions of SLF4J.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


IdP locales, style & help

[x] updated locales
[ ] add link to help
[ ] update style

@oyo idp fields

should both show up like that - we need to reduce the user input to ensure it is as easy as possible for the beginning:
Please display them like the URI


Expected behavior
Grass ramps have different texture to look more like man-made elevation modifications.
Screenshots

Hey Xezon i made a ew changes do you think this ok

Did you tweak the look of the ramps?

it's better i guess

Grass ramp is ok but ideally it looks natural. I have seen other maps with grass ramps. I think you just need to smooth out the elevation changes. I would need your updated map files to review your changes.

Tournament Himalaya V3.zip
Here my version 3 and please tell me anything if there something need to fix

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/9c3d6405a7e5aee03da0eba37f37615db63c09e7
Grass ramps look natural now.
There still is issue #75
And I added new issue #125


Partial fix for installation issue when My Documents on network drive

Summary of the Pull Request
Partial fix for Installation does not complete in the case of
a machine install. A fix for the user install needs deeper analysis.

PR Checklist

[ ] Closes: #xxx
[ X] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Ran the machine installer to successful completion in the following environment:

Windows 10 (as reported in issue)
Language: German (as reported in issue)
My Documents moved to a network drive

Note that I also verified that the bug repros with the following versions:
PowerToysSetup-0.80.0-x64.exe
PowerToysUserSetup-0.80.0-x64.exe
PowerToysUserSetup-0.80.1-x64.exe

/azp run

/azp run


Issue #1223 Supports both GET/PUT, restructured to used same executio…

Added matching RESTful call /device/api/v1/device/name/{device-name}/command/{command-name}, finding the corresponding device and command ids and passing them to the methods that operate off of IDs used by /device/api/v1/device/{device-id}/command/{command-id}.
To convert command name into appropriate ID for the device, used the device's profile and iterate through names looking for a match.  Unless the number of commands is substantial - and not sure how large that would be - seemed that overhead of creating the map would be more than just walking through the array.  Not opposed to making a map but didn't seem necessary (but remember, I'm new here).
Tested by following camera control example in docs https://docs.edgexfoundry.org/Ch-WalkthroughData.html and trying various combinations of commands or invalid commands.  Also updated .raml with new calls (basically cut-and-paste of existing ID calls with changes to reflect the use of names).

recheck

recheck

Rebase the PR branch and that should clear the docs-verify failure above.

Cleaned up commit history after making a mess of things and should have consolidated changes, both original and PR comments, included in this one, if I didn't mess it up.

recheck

@scsosna99 Build is now failing due to a PR that was merged today, so you'll need to rebase again.
13:51:40 ?   	github.com/edgexfoundry/edgex-go	[no test files]
13:52:04 # github.com/edgexfoundry/edgex-go/internal/core/command
13:52:04 internal/core/command/restDevice.go:77:31: d.Profile.Commands undefined (type models.DeviceProfile has no field or method Commands)

On the other build failure you called out in the #devops channel, I believe that had to do with no ARM base image being available for a recent Go version upgrade that happened yesterday. When there's no obvious cause you can retrigger a build by typing 'recheck' in the comment.

need to check field name change you made, will update my code this evening.

recheck

recheck

recheck

recheck

Re-assigning to @difince for closure. A new PR will be opened.


Button names confusing

Not exactly a bug. not exactly a feature request, so I put it here as lower priority
In the extension (quite useful to me, BTW, to open tabs for monitoring all my 3D printers at one time), you have TWO buttons named "Open List" - one to load, and one to open them all.  This is not only confusing to some, but as a former programmer/beta tester of decades, it also goes totally against all HIGs (Human Interface Guidelines) and Standard Practices... two buttons named the same but for different purposes in one dialog is not a good thing.
They should probably be named something such as "Load List" and "Open List" or "Open Loaded List" to differentiate between them and their functions.

Thanks for creating the issue with the suggestion, and I am happy that you find the extension useful.
It is something that I have been meaning to change for a while now, and thanks for your suggestions of what to change the text to. I think I will change it to something like 'Open links' for the button that opens the links, and 'Load List' for the button that loads the list from browser storage, as this will make it a lot clearer as you mentioned.

Thanks for creating the issue with the suggestion, and I am happy that you find the extension useful.
It is something that I have been meaning to change for a while now, and thanks for your suggestions of what to change the text to. I think I will change it to something like 'Open links' for the button that opens the links, and 'Load List' for the button that loads the list from browser storage, as this will make it a lot clearer as you mentioned.

Sounds good to me.

One other suggestion, when you get time... Allow a "Default List" checkbox that would select that list in the "Your Saved List" control box automatically, so it would be a two-click effort (Load the default list, as it would already be selected) and Open the list) instead of fiddling with the actual popup each time.  99% of the time I open the same list.  If the there were no "default list" selected, then the behavior would be as it is now.

One other suggestion, when you get time... Allow a "Default List" checkbox that would select that list in the "Your Saved List" control box automatically, so it would be a two-click effort (Load the default list, [as it would already be selected] and Open the selected list) instead of fiddling with the actual popup each time. 99% of the time I open the same list. If there were no "default list" selected, then the behavior would be as it is now.

Do you mean that when you open the popup, the links are already loaded into the textarea? Or that when you open the popup the "Default List" is the chosen option in the select dropdown?
If you wanted the list to be loaded into the textarea when you load the popup there is an option currently in the settings: "Default list to display", you can select one of your lists, and when you open the popup the chosen list will be loaded into the textarea.

One other suggestion, when you get time... Allow a "Default List" checkbox that would select that list in the "Your Saved List" control box automatically, so it would be a two-click effort (Load the default list, [as it would already be selected] and Open the selected list) instead of fiddling with the actual popup each time. 99% of the time I open the same list. If there were no "default list" selected, then the behavior would be as it is now.

Do you mean that when you open the popup, the links are already loaded into the textarea? Or that when you open the popup the "Default List" is the chosen option in the select dropdown?
If you wanted the list to be loaded into the textarea when you load the popup there is an option currently in the settings: "Default list to display", you can select one of your lists, and when you open the popup the chosen list will be loaded into the textarea.

Oh, ::duh::, I COMPLETELY missed that.  ::Nevermind::


Identity: Unable to log into multiple accounts using the Token Cache

Package Name: "@azure/identity"
Package Version: 4.0.1
Operating system: Ubuntu 23.10
[x] nodejs

version: 18.17.1 (VS Code Extension)

[ ] browser:

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
I am developing a VSCode extension and try to use Microsoft Graph to access Outlook resources of two different Microsoft accounts and cache the tokens using the @azure/identity-cache-persistence package (v1.0.0) and the InteractiveBrowserCredential.
With the token cache enabled, I can log into the first account (the browser opens) and give access to an OAuth application to some Outlook resources. I do this by manually calling authenticationRecord = credential.authenticate(...)
Since this is the first log in, no AuthenticationRecord can be provided to the InteractiveBrowserCredential.
Now I try to log into the second account (by setting InteractiveBrowserCredential.loginHint to another value) and again without providing any AuthenticationRecord because it is not available yet. What happens then is that the AuthenticationRecord of the first account is being returned without opening the browser and thus without giving me the chance to log into that second account.
My workaround for now is to disable the token cache when no authentication record is available (i.e. for the first log in) and enable it for all subsequent login attempts. But that leads to the inconvenience that I need to log in twice into every account instead of only once because only for the second log in, the token cache is enabled.
To Reproduce
Below is the code that does not work for me, i.e. the token cache is always enabled. Run it twice with different user accounts (set loginHint to a different account each run). Azure Identity will ask the user only once to log in.
const loginHint = `<the Microsoft account>`;
const authenticationRecordFolderPath = path.join(os.homedir(), ".IdentityService");
const authenticationRecordFilePath = path.join(authenticationRecordFolderPath, `cortex_notes@${loginHint}.json`);

let authenticationRecord: AuthenticationRecord | undefined;

try {

    const jsonString = await fs.readFile(authenticationRecordFilePath, {
        encoding: 'utf8'
    });

    authenticationRecord = JSON.parse(jsonString);
} catch (err) {
    // ignore
}

const scopes = ['Mail.ReadWrite', 'Calendars.Read'];

const credential = new InteractiveBrowserCredential({
    tenantId: 'common',
    clientId: 'f470bc86-5748-46ef-8d92-450964420fb9',
    tokenCachePersistenceOptions: {
        enabled: true,
    },
    loginHint: loginHint,
    authenticationRecord: authenticationRecord
});

if (!authenticationRecord) {
    authenticationRecord = await credential.authenticate(scopes);

    try {

        await fs.mkdir(authenticationRecordFolderPath, {
            recursive: true
        });

        let jsonString = JSON.stringify(authenticationRecord);

        await fs.writeFile(authenticationRecordFilePath, jsonString);
    } catch (error) {
        // ignore
    }
}

const authProvider = new TokenCredentialAuthenticationProvider(credential, {
    scopes: scopes,
});

const graphClient = Client.initWithMiddleware({ authProvider: authProvider });

Expected behavior
I expect a possibility to enforce authentication, i.e. ignore the token cache, but at the same time update it when the authentication was successful so it is populated when the user needs to authenticate the next time.
More specifically, I would expect that the token cache is (optionally) bypassed when no AuthenticationRecord is available so that the user is forced to authenticate manually.
Screenshots
Additional context

Hello, I have figured what the problem is. I'll be working on a fix for this soon.

Great, thank you!

@Apollo3zehn Can you provide me with the logs of the above sample. You can do this by
import { setLogLevel } from "@azure/logger";
setLogLevel("verbose");

Also, I did try the sample above. For me it does prompt me with the dialog box when I try to create a credential with a different email address (loginHint). I would recommend you to compare your two authentication records, if they have different contents when created. If they have the same content it means - it's taking in the same account (tenant, client id and email) credentials for login. Compare the two authentication records created locally and let me know what your content really looks like.

I will test again today and report my findings!

Please find the logs below. I tried first to access<EMAIL_ADDRESS>for which I got a login request which succeeded. Then I tried to access<EMAIL_ADDRESS>for which I did not got asked to login (no browser page opened). And the end result is that I am logged into only into<EMAIL_ADDRESS>where I can access my mails.
Here is the comparison of the authentication records. They are equal:

false
azure:core-client:warning The baseUri option for SDK Clients has been deprecated, please use endpoint instead.
azure:identity:info MsalClient => getToken() => Attempting to acquire token interactively
azure:identity:info InteractiveBrowserCredential => getToken() => Creating new PublicClientApplication with CAE disabled.
azure:identity:info InteractiveBrowserCredential => getToken() => No cached account found in local state, attempting to load it from MSAL cache.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Info - getTokenCache called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Info - getAuthCodeUrl called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - createAuthority called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata  from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Verbose - Found cloud discovery metadata from hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get endpoint metadata from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find endpoint metadata in the config... Attempting to get endpoint metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Info - Building oauth client configuration with the following authority: https://login.microsoftonline.com/common/oauth2/v2.0/token.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - Auth code client created
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:40:53 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - createAuthCodeUrlQueryString: No account, adding login_hint from request
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Info - acquireTokenByCode called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - createAuthority called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata  from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Verbose - Found cloud discovery metadata from hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get endpoint metadata from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find endpoint metadata in the config... Attempting to get endpoint metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:21 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Info - Building oauth client configuration with the following authority: https://login.microsoftonline.com/common/oauth2/v2.0/token.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:21 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - Auth code client created
azure:core-rest-pipeline retryPolicy:info Retry 0: Attempting to send request bb3a26b2-e48b-4907-8a54-f79144e4ddf3
azure:core-rest-pipeline:info Request: {
  "url": "https://login.microsoftonline.com/common/oauth2/v2.0/token?client-request-id=REDACTED",
  "headers": {
    "content-type": "application/x-www-form-urlencoded;charset=utf-8",
    "x-anchormailbox": "REDACTED",
    "accept-encoding": "gzip,deflate",
    "user-agent": "azsdk-js-identity/4.5.0 core-rest-pipeline/1.18.1 Node/20.18.0 OS/(x64-Linux-6.12.4-arch1-1)",
    "x-ms-client-request-id": "bb3a26b2-e48b-4907-8a54-f79144e4ddf3"
  },
  "method": "POST",
  "timeout": 0,
  "disableKeepAlive": false,
  "withCredentials": false,
  "abortSignal": {},
  "requestId": "bb3a26b2-e48b-4907-8a54-f79144e4ddf3",
  "allowInsecureConnection": false,
  "enableBrowserStreams": false
}
azure:core-rest-pipeline:info No cached TLS Agent exist, creating a new Agent
azure:core-rest-pipeline:info Response status code: 200
azure:core-rest-pipeline:info Headers: {
  "cache-control": "no-store, no-cache",
  "pragma": "no-cache",
  "content-type": "application/json; charset=utf-8",
  "expires": "-1",
  "strict-transport-security": "max-age=31536000; includeSubDomains",
  "x-content-type-options": "nosniff",
  "p3p": "CP=\"DSP CUR OTPi IND OTRi ONL FIN\"",
  "x-ms-request-id": "97d5aa6b-5fec-47da-9c82-07856b610803",
  "x-ms-ests-server": "2.1.19683.3 - NEULR1 ProdSlices",
  "x-ms-clitelem": "1,0,0,,",
  "x-ms-srs": "1.P",
  "content-security-policy-report-only": "object-src 'none'; base-uri 'self'; script-src 'self' 'nonce-DEvL3U_YDeUZAZQmCbLbgg' 'unsafe-inline' 'unsafe-eval' https://*.msauth.net https://*.msftauth.net https://*.msftauthimages.net https://*.msauthimages.net https://*.msidentity.com https://*.microsoftonline-p.com https://*.microsoftazuread-sso.com https://*.azureedge.net https://*.outlook.com https://*.office.com https://*.office365.com https://*.microsoft.com https://*.bing.com 'report-sample'; img-src 'self' data: https://*.msauth.net https://*.msftauth.net https://*.msftauthimages.net https://*.msauthimages.net https://*.msidentity.com https://*.microsoftonline-p.com https://*.microsoftazuread-sso.com https://*.azureedge.net https://*.outlook.com https://*.office.com https://*.office365.com https://*.microsoft.com https://*.bing.com 'report-sample'; report-uri https://csp.microsoft.com/report/ESTS-UX-All",
  "x-xss-protection": "0",
  "set-cookie": "fpc=AkZTDNjysiJGg8ElvhMqQfQqIoFUAQAAABDW9N4OAAAA; expires=Fri, 17-Jan-2025 14:41:21 GMT; path=/; secure; HttpOnly; SameSite=None",
  "date": "Wed, 18 Dec 2024 14:41:21 GMT",
  "connection": "close",
  "content-length": "3459"
}
azure:core-rest-pipeline retryPolicy:info Retry 0: Received a response from request bb3a26b2-e48b-4907-8a54-f79144e4ddf3
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing 2 retry strategies.
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing retry strategy throttlingRetryStrategy.
azure:core-rest-pipeline retryPolicy:info Retry 0: Skipped.
azure:core-rest-pipeline retryPolicy:info Retry 0: Processing retry strategy exponentialRetryStrategy.
azure:core-rest-pipeline retryPolicy:info Retry 0: Skipped.
azure:core-rest-pipeline retryPolicy:info None of the retry strategies could work with the received response. Returning it.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - setCachedAccount called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - Persistence enabled, calling beforeCacheAccess
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [021744dd-203f-4115-bddd-4580f82d8ea0]<EMAIL_ADDRESS>: Verbose - Persistence enabled, calling afterCacheAccess
azure:identity:info InteractiveBrowserCredential => getToken() => SUCCESS. Scopes: Mail.ReadWrite, Calendars.Read.
azure:identity:info MsalClient => getToken() => Attempting to acquire token interactively
azure:identity:info InteractiveBrowserCredential => getToken() => Existing PublicClientApplication found in cache, returning it.
azure:identity:info InteractiveBrowserCredential => getToken() => Attempting to acquire token silently
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [b92a19cd-eb10-430b-9a47-5881b376d43c]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [b92a19cd-eb10-430b-9a47-5881b376d43c]<EMAIL_ADDRESS>: Verbose - createAuthority called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata  from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - Found cloud discovery metadata from hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get endpoint metadata from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find endpoint metadata in the config... Attempting to get endpoint metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:22 GMT] : [b92a19cd-eb10-430b-9a47-5881b376d43c]<EMAIL_ADDRESS>: Info - Building oauth client configuration with the following authority: https://login.microsoftonline.com/common/oauth2/v2.0/token.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:22 GMT] : [b92a19cd-eb10-430b-9a47-5881b376d43c]<EMAIL_ADDRESS>: Verbose - Silent flow client created
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getAccessToken - Returning access token
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:22 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getIdToken - Returning ID token
azure:identity:info InteractiveBrowserCredential => getToken() => SUCCESS. Scopes: Mail.ReadWrite, Calendars.Read.
azure:core-client:warning The baseUri option for SDK Clients has been deprecated, please use endpoint instead.
azure:identity:info MsalClient => getToken() => Attempting to acquire token interactively
azure:identity:info InteractiveBrowserCredential => getToken() => Creating new PublicClientApplication with CAE disabled.
azure:identity:info InteractiveBrowserCredential => getToken() => No cached account found in local state, attempting to load it from MSAL cache.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - getTokenCache called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getIdToken - Returning ID token
azure:identity:info InteractiveBrowserCredential => getToken() => Attempting to acquire token silently
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [30dd37d3-e23c-44c9-8939-b3a6e35fcd3c]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [30dd37d3-e23c-44c9-8939-b3a6e35fcd3c]<EMAIL_ADDRESS>: Verbose - createAuthority called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata  from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Found cloud discovery metadata from hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get endpoint metadata from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find endpoint metadata in the config... Attempting to get endpoint metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : [30dd37d3-e23c-44c9-8939-b3a6e35fcd3c]<EMAIL_ADDRESS>: Info - Building oauth client configuration with the following authority: https://login.microsoftonline.com/common/oauth2/v2.0/token.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [30dd37d3-e23c-44c9-8939-b3a6e35fcd3c]<EMAIL_ADDRESS>: Verbose - Silent flow client created
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getAccessToken - Returning access token
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getIdToken - Returning ID token
azure:identity:info InteractiveBrowserCredential => getToken() => SUCCESS. Scopes: Mail.ReadWrite, Calendars.Read.
azure:identity:info MsalClient => getToken() => Attempting to acquire token interactively
azure:identity:info InteractiveBrowserCredential => getToken() => Existing PublicClientApplication found in cache, returning it.
azure:identity:info InteractiveBrowserCredential => getToken() => Attempting to acquire token silently
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - initializeRequestScopes called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [3ff6ba92-a506-4e43-95c7-f5c5dd5ec13c]<EMAIL_ADDRESS>: Verbose - buildOauthClientConfiguration called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [3ff6ba92-a506-4e43-95c7-f5c5dd5ec13c]<EMAIL_ADDRESS>: Verbose - createAuthority called
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get cloud discovery metadata  from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find cloud discovery metadata in the config... Attempting to get cloud discovery metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Found cloud discovery metadata from hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Attempting to get endpoint metadata from authority configuration
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Verbose - Did not find endpoint metadata in the config... Attempting to get endpoint metadata from the hardcoded values.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : [3ff6ba92-a506-4e43-95c7-f5c5dd5ec13c]<EMAIL_ADDRESS>: Info - Building oauth client configuration with the following authority: https://login.microsoftonline.com/common/oauth2/v2.0/token.
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 verbose message: [Wed, 18 Dec 2024 14:41:35 GMT] : [3ff6ba92-a506-4e43-95c7-f5c5dd5ec13c]<EMAIL_ADDRESS>: Verbose - Silent flow client created
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getAccessToken - Returning access token
azure:identity:info InteractiveBrowserCredential => MSAL Node V2 info message: [Wed, 18 Dec 2024 14:41:35 GMT] : []<EMAIL_ADDRESS>: Info - CacheManager:getIdToken - Returning ID token
azure:identity:info InteractiveBrowserCredential => getToken() => SUCCESS. Scopes: Mail.ReadWrite, Calendars.Read.

To it looks like the login-hint being ignored.

To it looks like the login-hint being ignored.

Yeah, first, you need the prompt with the specified login hint right? My PR will address that. That way it won't appear like silent auth is happening, when not needed.
Second, I wanted to check with you is - Are both the emails approved for logging into the app registration account (the tenant-id and client-id provided above in the sample)? If your second email doesn't have the permission to authenticate to that app registration, then the login may fail even with the fix. So make sure you have the appropriate set up or use the correct tenant and client id required for the second email account.

Yeah, first, you need the prompt with the specified login hint right? My PR will address that. That way it won't appear like silent auth is happening, when not needed.

That is great, it would solve my problem!

Second, I wanted to check with you is - Are both the emails approved for logging into the app registration account (the tenant-id and client-id provided above in the sample)? If your second email doesn't have the permission to authenticate to that app registration, then the login may fail even with the fix. So make sure you have the appropriate set up or use the correct tenant and client id required for the second email account.

Yes, both are approved. If I switch the order of the emails I use to log in, I can access mails of that other email address. It is always the second address which does not work, no matter what order they have.

@Apollo3zehn
I am addressing the second issue that you mentioned in my current PR.
One more thing, while addressing the first issue, it will ALWAYS show the login dialog box as well. Is that the experience you are looking for? I vaguely remember that a few customers had complained over the amount of times they saw the login dialog in the past.


/network/options fails asserter

➜  rosetta-cli git:(master) ✗ rosetta-cli version
v0.5.8
rosetta-cli view:networks --configuration-file rosetta-cli/filecoin//filecoin-testnet-asserter-config.json
loaded configuration file: rosetta-cli/filecoin//filecoin-testnet-asserter-config.json
{
 "blockchain": "Filecoin",
 "network": "ignition"
}
Error: error details populated in /network/options: /network/options: unable to get network options

According to the asserter, we shall not populate errors.details. I think the following 2 errors populates details
 {
                "code": 15,
                "message": "unable to submit transaction",
                "retriable": false,
                "details": {
                    "lotusErr": "not enough funds (required: 1.300794209895606336 FIL, balance: 1 FIL): not enough funds to execute transaction"
                }
            },

            {
                "code": 22,
                "message": "unable to get tipset",
                "retriable": true,
                "details": {
                    "lotusErr": "looking for tipset with height greater than start point"
                }
            },

any reason for this restriction? It makes easier to understand the actual cause of the problem..

I can get back to you on reasoning behind. But can we just remove these for now to unblock?

I will leave this issue open, so we can enable these error details once the asserter accepts them


Different behavior with similar  elements in same page

Hi,
I'm experiencing a weird behavior with similar  elements present in the same page:

On the one hand, a  Lista Alquileres element in the main section of the page, triggers, as expected the function vaadinRouterGlobalPopstateHandler(event)

while, on the other one, a Lista on the sidebar section of the same page doesn't trigger anything and results with the browser trying to request the page from the webserver, with a resulting "404 Page not found"

The only difference is that while the first one was hardcoded to better understand this error, the second one was programatically generated during page load time.
When hovering over both elements, both produce the "localhost:5000/alquileres/list" in the browser's bottom bar, but with unexpected different results.
I'm using Chrome Version 84.0.4147.89 (Official Build) (64-bit)
Oriol

Hi @OriolRF, thanks for reporting. Could you help to attach a sample project that we can use to reproduce the problem?

Hi,
I found the problem: the sidebar handler included a e.stopPropagation() which prevented the proper functioning of the router. Once removed, the behavior is now the expected one.
Sorry to bother you.
Thank you


No details of last update in details screen

On the old website there are details of when the relation was last updated and when the website last updated the information from the route relation. The new website does not have this anymore.
It would be helpful if that information was included.

Situation on: 2018-05-12 22:01:02
Network last update: 2018-05-07 Traildino
Relation last update: 2018-01-17 dvdhoven

@MaartenDeen thanks for reporting this.
About the timestamps
The new website shows details about the network, route and node history. The idea at the time was that it was no longer necessary to also show the timestamps in the summary information. The history is displayed in reverse order, so ou can just look at the start of the history to see the timestamp of the latest change. to this information
But I can see how it could be helpful to include that information in the summary again.
The following information is still present in the data that is sent by the server to the browser, and requires minimal work to include again in the user interface:

"situation on" timestamp (valid for networks, routes and nodes)
network last update timestamp
network relation last update timestamp
route last updated timestamp
node last updated timestamp

The logic to record the route relation last update timestamp was removed, but can be added again (requires a rebuild of the entire analysis database - no problem).
I will add all timestamps again.
About the user names
In release v2.0.4 all references to user names and user identifiers were removed from the system.
This was done as a first step in preparing for changes that are expected in the distribution of the metadata collected in OpenStreetMap. See Appendix A in the OSM GDPR position page. The second step in this preparation was to put the changeset information behind a login in release v2.0.7. Depending on the outcome of the GDPR related changes to OpenStreetMap, more information may be removed in the future, or the user information may be re-added (although this would be a major change again, because at this moment the user information is completely absent in the analysis database).


Metro 134 subscription pods rebalance

Description
https://razorpay.atlassian.net/browse/METRO-165
https://razorpay.atlassian.net/browse/METRO-134
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[x] I have added unit tests that prove my fix is effective or that my feature works (if applicable)
[ ] I have added integration tests for the new feature (if applicable)
[ ] I have manually tested my code to the best of my abilities.

Closing this since we do not intend to add this functionality anytime soon.


HttpCache warm up does not work with SEO URLs

PHP Version
7.4.26
Shopware Version
<IP_ADDRESS>
Expected behaviour
When using SEO templates for products and categories like {{ product.translated.name | lower }}-{{ product.productNumber | lower}} and {% for part in category.seoBreadcrumb %}{{ part | lower}}/{% endfor %} no http cache entries will be generated for those URLs. The warmup task only generates cache entries for the ID-based URLs.
Actual behaviour
If SEO URLs are used, the HTTP cache warmer should generate cache entries for the SEO URLs and not for the ID-based URLs.
How to reproduce
Activate SEO URLs. Clear the cache (bin/console cache:clear), warm up the HTTP cache (bin/console http:cache:warm:up), wait until the warm-up has finished.
Open the shop in a browser with disabled browser cache and surf around. If you look at the response headers, you will see that Age is always zero. If you visit the same page again Age is greater zero.

After diving deeper into the code, I found that the issue is not caused by SEO URLs. Before the request reaches the HTTP cache, the SEO URL is replaced with the ID-Based URL by Shopware\Storefront\Framework\Routing\RequestTransformer.
The culprit was the nginx configuration from the production template, which adds the path of the URL as a query parameter to the request. The query parameters are part of the cache key (Shopware\Storefront\Framework\Cache\HttpCacheKeyGenerator). Since the cache warmer does its requests without the query parameter, it creates cache entries with different keys.
See https://github.com/shopware/production/pull/170


mmv: 2.1 -> 2.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/rrthomas/mmv/releases.
meta.description for mmv is: "Utility for wildcard renaming, copying, etc"
meta.homepage for mmv is: "https://github.com/rrthomas/mmv"
meta.changelog for mmv is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
Warning: no invocation of /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2/bin/mcp had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2/bin/mln had a zero exit code or showed the expected version
Warning: no invocation of /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2/bin/mmv had a zero exit code or showed the expected version
0 of 3 passed binary check by having a zero exit code.
0 of 3 passed binary check by having the new version present in output.
found 2.2 with grep in /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2
found 2.2 in filename of file in /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
mmv

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mmv https://github.com/r-ryantm/nixpkgs/archive/ba020033c50130fba785bf699c574d98c099ddc7.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2
ls -la /nix/store/vdid3d20l9v17zzk0lmrmffy2hx4c8mg-mmv-2.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

mmv

Maintainer pings
cc @siraben for testing.

Version 2.3 is available. Please consider #163037.


Create device trees

[x] https://github.com/hermit-os/loader/issues/274
[x] https://github.com/rust-vmm/vm-fdt/issues/61

[x] https://github.com/rust-vmm/vm-fdt/pull/68

[x] https://github.com/hermit-os/loader/pull/287

Resolved in https://github.com/hermit-os/loader/pull/287.


Added comma to line 17

That first section of that sentence is an introductory phrase

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @a-Daley for your contribution and congratulations on your first PR merge to this repo! 🎉
Hope to see more contributions from you in the future.


Add subclass GroupHab of Hab to better encapsulate differences

There is a lot of code in Hab of the form
if self.mhab ....
to execute group hab specific code.  this is begging to be put in a sub class that has the GroupHab code.
Only the subclass GroupHab would then have a .mhab  .merfers .migers .smids .rmids where all the group specific stuff is going

Closed with PR #438


Feature Request: Option to rewrite source ip dynamically to match incoming packet source address

Hi there,
I'd like to start with THANK YOU. Your program allows us to run some client software over a VPN that inexplicably uses a broadcast packet to locate its host server every single time you open it.
Any chance you would be able to implement an option for the source IP to be dynamically rewritten based on the origin of the broadcast packet? Everything works great as-is, but I can only use it for one host at a time. I would love to help out, but unfortunately I don't know Go.
Thanks.

Hey, thanks for the positive feedback :)
Just to make sure I understand your request correctly: You want to run the program on some host in your network. It receives broadcast packets from other hosts in the network and forwards them over a VPN tunnel. Reply packets could go directly from the VPN tunnel to the original hosts, but the source address rewriting gets into your way (for more than one other host). So, you would basically need an option to just keep the original source IP address in the packets, right?

Yep, you got it. That's exactly what I'm hoping for.
Other than the fact that it can only point at one original IP right now, it's working perfectly.
Thanks again :)

I added the command line argument "-keep-source-ip" that should disable the source address rewriting. Could you test if it works for you?

Great, closing this issue for now. If anything breaks, feel free to re-open or create a new one :)


Werror enabled for CMake CI builds

Enables Werror for CMake based builds on Travis ci.

Coverage remained the same at 99.875% when pulling c72d8a2be6493eb5555cbf4cd441735f1c553f23 on offa:ci_cmake_werror into 5a197bcf2ff0a2f87d293b8100a7d7c5c3afe9f4 on cpputest:master.


core dump when controller isn't found

Code isn't falling out properly with ignore_ace.

PR #79 should fix this.


Objective-C nonnull and nullable keywords aren't highlighted

Apple introduced the nonnull and nullable keywords into Obj-C in early 2015, but these still haven't been added into Prettify. They currently are not highlighted at all. It would be great if these could get added!

The way it is implemented, there is no specific language handler for objective-C, rather all C-like languages are highlighted in a shared way:

https://github.com/google/code-prettify/blob/master/js-modules/prettify.js#L734

with a list of common keywords (search for CPP_KEYWORDS).
So adding those keywords specific for Obj-C means they also show up as keywords in plain C/C++ codes...


feat: Added Bucket Sort in Go

Fixes: #2110
Added Bucket Sort in Go

@atarax665 @HarshCasper Please review this pr.

@m-code12
Please open another PR if you plan to work on it.

@m-code12
Please open another PR if you plan to work on it.

Yeah I'll raise another PR.


Make sonar work without user defined configurations

Some user will most probably just install and do sonar <url>, and will get the following error:
TypeError: Path must be a string. Received null
    at assertPath (path.js:28:11)
    at Object.resolve (path.js:1167:7)
    at Object.exports.load (/Users/alrra/.nvm/versions/node/v8.1.3/lib/node_modules/@sonarwhal/sonar/dist/src/lib/config.js:87:31)
    at Object.exports.execute (/Users/alrra/.nvm/versions/node/v8.1.3/lib/node_modules/@sonarwhal/sonar/dist/src/lib/cli.js:94:27)
    at run (/Users/alrra/.nvm/versions/node/v8.1.3/lib/node_modules/@sonarwhal/sonar/dist/src/bin/sonar.js:28:34)
    at Object.<anonymous> (/Users/alrra/.nvm/versions/node/v8.1.3/lib/node_modules/@sonarwhal/sonar/dist/src/bin/sonar.js:31:1)
    at Module._compile (module.js:569:30)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)

This constitutes a bad user experience, and I think we should change things so that, if no configurations are found, sonar will just use it's defaults (+ maybe also print a message, such as: Configurations not found, using defaults...).

Thoughts @sonarwhal/contributors?

Thoughts @sonarwhal/contributors?

If we are going to do that then we should review which rules are on by default. I'm thinking more specifically about the ssllabs. It takes a long time to run if the site is not cached.
We could turn it off in this case or look for a way to output more information to the user so "Finishing" is not on the screen for 1min.
Another alternative could be to tell the user that no configuration has been found and if they want sonar to create one and scan the site. Then we launch init. That might be easier.

I'm thinking more specifically about the ssllabs. It takes a long time to run if the site is not cached.

:+1:

Another alternative could be to tell the user that no configuration has been found and if they want sonar to create one and scan the site. Then we launch init. That might be easier.

That also works for me.

Good catch! 👍

This constitutes a bad user experience, and I think we should change things so that, if no configurations are found, sonar will just use it's defaults (+ maybe also print a message, such as: Configurations not found, using defaults...).

Another alternative could be to tell the user that no configuration has been found and if they want sonar to create one and scan the site. Then we launch init. That might be easier.

👍

Changed the title to a more accurate one.

shouldn't it  log no .sonarrc file found. Someone who uses sonar should be used to read docs, that implicit mention having such a file.

@cxoltero it's all about making the first run experience as easy as possible :)

@qzhou1607 have you done a PR for this? It is in the review column but I don't see anything related to this.


ADD DCO signing as a pre-commit check

We should see if we can add DCO signing as a pre-commit check , a lot of new developers get stumped by this, we should make sure that this process is seemless.
CC: @ryantwolf

Moving to next sprint, not a customer request.


Performing an update w/o the cluster stopped messed up the CloudFormation stack and now I can't do any mods.

I was trying to do an update of the cluster.  I added the ClusterConfig/SlurmSettings/ScaledownIdletime to my cluster config file:
  ParallelClusterConfig:
    Version: 3.11.1
    Architecture: x86_64
    Image:
      Os: rocky8
      CustomAmi: ami-0d68c6538XXXXXXX
    DisableSimultaneousMultithreading: true
    ClusterConfig:
      SlurmSettings:
        ScaledownIdletime: 20

I had noticed earlier that when I downsized some node count, you STOPPED the cluster.  I assumed you would do the same here.
Maybe not.
The cluster was not stopped.  The update failed.  The UPDATE_ROLLBACK_FAILED.
I stopped my cluster.
Tried the update again, but it can't do the update as the cluster if in a bad state.  How does one get out of a bad state to continue using aws-eda-slurm-cluster install.sh?
That said, I've discovered my above syntax is not valid.  the ${cluster}-config gets created, but the actual cluster stack doesn't.  Need to document how to add things like SlurmSettings better please.  I just can't figure out. Will file another issue for that.

Usually what you'll need to do is resume the rollback. When you do that you'll get an option to skip the failed resource. I'm assuming that the failure can from a custom resource like the ParallelCluster or UpdateHeadNode resource. Expand the section for the failed resources, click that, and then continue the rollback.


Add path support for capture checking

I tried this version of the old subsumes and compared its outcome with the new subsume for all the tests in testCompilation.
  final def subsumesOld(y: CaptureRef)(using Context): Boolean =
    (this eq y)
    || this.isRootCapability
    || y.match
        case y: TermRef =>
            y.prefix.match
              case ypre: CaptureRef =>
                this.subsumes(ypre)
                || this.match
                    case x @ TermRef(xpre: CaptureRef, _) =>
                      x.symbol == y.symbol && xpre =:= ypre
                    case _ =>
                      false
              case _ => false
          || y.info.match
              case y1: SingletonCaptureRef => this.subsumes(y1)
              case _ => false
        case MaybeCapability(y1) => this.stripMaybe.subsumes(y1)
        case _ => false
    || this.match
        case ReachCapability(x1) => x1.subsumes(y.stripReach)
        case x: TermRef =>
          x.info match
            case x1: SingletonCaptureRef => x1.subsumes(y)
            case _ => false
        case x: TermParamRef => subsumesExistentially(x, y)
        case x: TypeRef => assumedContainsOf(x).contains(y)
        case _ => false

There was no difference in outcomes, and the version here is shorter and clearer. So we should probably use this, unless we have tests that show there is a difference and the version here is wrong.

I would have expected more changes in the recheckSelect rule. Indeed, recheckSelect seems to not make use of path types. Here is a simple test program:
class IO

class C(val f: IO^)

def test(io: IO^) =
  val c = C(io)
  val f = () => println(c.f)

This types f as () ->{c} Unit. But it should be () ->{c.f} Unit.

Generally, what's missing is a suite of tests that shows how path-dependent types give a more expressive language. Show code that did not typecheck before, but now does.

There's a problem with pattern matching shown in the path-patmat-should-be-pos.scala test.

Rebased to main


add forceInheritance option

This feature is no longer required. The extension now displays a message to the user if there are parent folders with tabaqa.json files.


fix typo

This change is

Pull Request Test Coverage Report for Build 158

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 62.844%

Totals

Change from base Build 151:
0.0%

Covered Lines:
137

Relevant Lines:
218

💛  - Coveralls


Remove the word experimental in readmes

We have the word "Experimental" in the titles in the readme files for our core packages.
My guess is that we started with this when we were working on core v2, forgot to remove it when core v2 went GA and all our newer core packages from them did a copy/paste :)
@xirzec, @joheredi Am I missing something?

The presence of alpha/beta in the package version should reflect the supportability aspect of things.
The only case I can think of where having experimental listed in the readme is useful is if it is an unpublished package and someone landed on the readme page in the github repo wondering what it is

The only case I can think of where having experimental listed in the readme is useful is if it is an unpublished package and someone landed on the readme page in the github repo wondering what it is

That seems to be a good use. In this case, perhaps leave it in core-crypto?

Fair point. Left the term experimental in core-crypto and core-util


POM Generator

@ty-desj I assigned it to you to add some detail to the issue description. You can unassign yourself after if you prefer.


Realm Locking Main Thread Indefinitely

When I attempt to migrate my default realm the main thread locks indefinitely. I set a break point just before the call to setSchemaVersion: and attempt to print out the path in LLDB:
(lldb) po [RLMRealm defaultRealmPath]
/Users/jim/Library/Developer/CoreSimulator/Devices/39EF0DDD-8D57-4F70-BC5C-283B84AE4D7D/data/Containers/Data/Application/38784568-3D9C-40BE-B514-F6071C152D8B/Documents/default.realm

But then even trying to instantiate the realm and print the path (which I believe should throw an exception because my realm has not been migrated yet causes the application to lock up:
(lldb) po [RLMRealm defaultRealm].path
error: Execution was interrupted, reason: signal SIGTERM.
The process has been returned to the state before expression evaluation.

I get no actual error. The error printed out is thrown once I terminate the simulator after a few minutes. Is there any way I can further diagnose this error?

@jimjeffers is it possible you're trying to use the Realm from multiple processes? Could you possibly share the backtrace that happens when the main thread hangs?

That was the issue. There was no backtrace I had to actually press stop in Xcode to terminate the process. It was very bizarre. I deleted the application from the simulator and everything is fine again. I will post back if I can recreate the instance again. Otherwise, it may be an issue with the simulator.

If it happens again, instead of pressing stop in Xcode, press pause and type bt in the debugger, that'll show you the backtrace (and certainly let us know!).

Got it. Well just a heads up. It happened again but this time I closed the Realm Browser and reran the app. Now I'm getting the expected error instead of a hanging process! I'll be sure not to have that running in the background as I test the app.


A new function required to handle browser history pushState and navigation for static or SPA sites.

Added new function called navigateInternal.


ISDK-2408: Add Network Quality indicator to the local participants view

This PR adds the Network Quality functionality to the MultiPartyExample application.
More specifically:

The TVIConnectOptionsBuilder.isNetworkQualityEnabled property is set to true when creating the TVIConnectOptions. This will enable the Network Quality functionality when connected to a group room.
The func localParticipant(_ participant: TVILocalParticipant, didChange networkQualityLevel: TVINetworkQualityLevel) delegate method is implemented on MultiPartyViewController and the MultipartyViewController instance is set as the delegate of the TVILocalParticipant.
A Network Quality level indicator graphic has been added for each of the six possible states: 0-5
When the func localParticipant(_ participant: TVILocalParticipant, didChange networkQualityLevel: TVINetworkQualityLevel) method is fired, it calls the new updateLocalNetworkQualityLevel(networkQualityLevel: TVINetworkQualityLevel) method which will chose the appropriate graphic and color to use to display at the top right of the local participant's video view.
Colors.swift was added to house an extension to UIColor that makes it easier to access the standard Twilio status colors.
Updates Podfile to consume the 2.9.0 sdk.

As a follow on PR, possibly after the initial shipping of NQ, I would like to make a LocalParticipant xib and view class to manage the UI similar to that of the remote participants. That would ensure that the content within that view is laid out in the same fashion as the remote participant views during rotations

I made a small change to the selection of the image and tint color for better swiftiness and clarity.

Still have a constraints layout issue on the LocalParticipantView that I need to resolve.

There was discussion with @aymenn on the document about having the NavBar displayed with the room name. If there is agreement to remove it, I am happy to do so.

There was discussion with @aymenn on the document about having the NavBar displayed with the room name. If there is agreement to remove it, I am happy to do so.

I guess that would be okay in portrait, but it really doesn't make sense to waste the space in landscape (on any device that has constrained height in that orientation).

I agree with removing it in Landscape. Just was more questioning what we do in Portrait.

I agree with removing it in Landscape. Just was more questioning what we do in Portrait.

My opinion is that it would be easiest to display the Room name in Portrait (update in roomDidConnect:). And on any device orientation which is vertically constrained, hide the navigation bar.

NavBar is autohidden for vertically constrained layouts


fix(*): remove OvhApiServices dependency.

Remove OvhApiServices dependency

Provide also some informations into CHANGELOG.md about future major release