Remove specialist topic code from repo

Ticket: https://trello.com/c/gbfKtIeY/2401-remove-specialist-topic-code-from-search-m
WHAT
Remove some/all references to specialist_sectors from Search API.
WHY
As we are retiring specialist topics.

@kevindew The million dollar question. I don't think so.

All specialist topics have been archived now https://www.gov.uk/api/search.json?filter_format=specialist_sector
There was no boosting being applied to documents tagged to specialist topic, boosts are on document type or supergroup
So if we don't reindex, I think it means that existing documents will still have the specialist-sector field, where as new ones will not (as the mapping has updated). But I don't think there's any negative consequences of that.
Plus SearchAPI V1 isn't being used for Site Search anymore, so I guess this reduces the risk further.

@leenagupte do you agree?


Massive drops in AdX spend on Label 2-5

The large drops in AdX spend on label 2-5 (~35% or so) are well documented. However, what isn't documented is that disabling PAAPI in GAM doesn't fix them. Large drops in AdX cpm and Impressions remain for GAM accounts with PAAPI disabled. This appears to be because DV3 and Google Ads are refusing to bid in the label 2-5 traffic much of the time. DSP spend from TTD and Amazon are consistent across the five labels. Is this an attempt to elicit certain supply side behavior, or perhaps a technical oversight, that the DSP is directing funds away the contextual auction from 6% of Chrome traffic regardless of if a device auction will occur subsequently?
Here is some revenue and cpm from AdX in a recent time period in November 2024 for some subset of users. The particular length of time and subset of users not disclosed to protect company monetization information.
<html>
<body>
<!--StartFragment--><google-sheets-html-origin><style type="text/css"><!--td {border: 1px solid #cccccc;}br {mso-data-placement:same-cell;}--></style>
Mode A label | Protected Aud Delivery | Revenue | Impressions | cpm
-- | -- | -- | -- | --
[cookdep:label_only_1] | N | 1200.559526 | 575207 | 2.087178226
[cookdep:label_only_2] | N | 748.310124 | 475074 | 1.575144344
[cookdep:label_only_3] | N | 759.266972 | 473568 | 1.603290281
[cookdep:label_only_4] | N | 755.501655 | 474836 | 1.591079141
[cookdep:label_only_5] | N | 716.617527 | 469509 | 1.526312652

<!--EndFragment-->
</body>
</html>

Based on some private communication, this appears to be a technical oversight at dv3/ Google ad but side entities.

Steps to resolution appear in progress, thank you to anyone who engaged!


Unable to asynchronously load data into Blazor components

I started using Blazor recently and was trying to figure out how to asynchronously load data onto the page. Normally I would do this with ajax in javascript, put up a loading message or something and render the data from the server when it was ready. This functionality is suppose to be easily achievable with OnInitializedAsync. But regardless of my attempts the page always loads synchronously. Clicking the tab causes the browser to wait until the page is fully loaded from the server. I thought I was doing something wrong with my code, so I decided to load up the default Blazor Server App project and even that is failing to do what its supposed to. The code below is from the FetchData page of the blazor default project, I tweaked the number of elements to render from 5 to 5000 for longer delays to test this.
FetchData.razor
@page "/fetchdata"

@using BlazorApp1.Data
@using System.Diagnostics
@inject WeatherForecastService ForecastService

Weather forecast

This component demonstrates fetching data from a service.

@if (forecasts == null)
{

Loading...

}
else
{

                Date
                Temp. (C)
                Temp. (F)
                Summary

            @foreach (var forecast in forecasts)
            {

                    @forecast.Date.ToShortDateString()
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

            }

}

@code {
	private WeatherForecast[] forecasts;

	protected override async Task OnInitializedAsync()
	{
		forecasts = await ForecastService.GetForecastAsync(DateTime.Now);
	}
}

WeatherForecastService.cs
using System;
using System.Linq;
using System.Threading.Tasks;

namespace BlazorApp1.Data
{
    public class WeatherForecastService
    {
        private static readonly string[] Summaries = new[]
        {
            "Freezing", "Bracing", "Chilly", "Cool", "Mild", "Warm", "Balmy", "Hot", "Sweltering", "Scorching"
        };

        public Task<WeatherForecast[]> GetForecastAsync(DateTime startDate)
        {
            var rng = new Random();
            return Task.FromResult(Enumerable.Range(1, 5000).Select(index => new WeatherForecast
            {
                Date = startDate.AddDays(index),
                TemperatureC = rng.Next(-20, 55),
                Summary = Summaries[rng.Next(Summaries.Length)]
            }).ToArray());
        }
    }
}

My expectation was that the Loading... text in FetchData.razor is supposed to display until the call ForecastService.GetForecastAsync(DateTime.Now) completes. After which the page re-renders and shows the table with the data. But when clicking the FetchData tab in the app, the browser hangs and waits for the code to fully execute. I'm running this in VS 2019 on Windows 10. Am I missing something? Shouldn't Loading... show while the async function executes?

Maybe it is connected to lack of actual multithreading in Blazor.

The problem you're seeing is not with Blazor, it's that GetForecastAsync isn't actually asynchronous, so if you increase the Enumerable.Range size, it just blocks for longer.
If, instead of increasing the size of the enumerable, you change it to this:
        public async Task<WeatherForecast[]> GetForecastAsync(DateTime startDate)
        {
            await Task.Delay(5000);
            var rng = new Random();
            return Enumerable.Range(1, 5).Select(index => new WeatherForecast
            {
                Date = startDate.AddDays(index),
                TemperatureC = rng.Next(-20, 55),
                Summary = Summaries[rng.Next(Summaries.Length)]
            }).ToArray();
        }

You'll see the effect you're expecting.  In reality, if GetForecastAsync was waiting on an async call to a database or a HTTP service, then this would work out OK.
Maybe the template should be better in this regard?

If this is server-side blazor then I think the prerender will wait for the init code to complete before sending your initial html.
If you want to see UI before the days loads, put the call in OnAfterRender when firstRender == true

@mrpmorris answer is correct. Prerendering will always wait for the async work to complete before producing the HTML output so that it can produce meaningful results. The alternative suggested is a valid option to avoid delaying the rendering until the content has loaded.


[1.19] Add support for passing certain methods thru to the Agglomerated Trinkets

PR Contains ways of using scrolling to select a given trinket and then passing thru certain methods to certain trinket.
Was designed to fix issue with Artifacts that can be disabled never being able to be re-enabled causing it to be useless

?


Unknown Firewall nexusguard

nexusguard.com

ˋˋ`Server: <IP_ADDRESS>.20.973a686
...

 Will be addressed to shortly. :)

Addressed to via 1a09bfe in #77.


App: Clean up render loop

I noticed some dead code in the render animation loop. I also want to generally look it over for simple optimizations.

The biggest optimizations would probably be:

dead code removal (there is tons)
reduce the size of functions (check profile for ones that don't opt)
reduce allocations, replace with local variables


Fix e2e test error on page transition

Codecov Report

Merging #589 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #589      +/-   ##
==========================================
+ Coverage   85.38%   85.42%   +0.03%
==========================================
  Files          25       25
  Lines        2361     2497     +136
  Branches      203      203
==========================================
+ Hits         2016     2133     +117
- Misses        345      364      +19

Impacted Files
Coverage Δ

browser/services/platform.js
86.71% <0%> (-1.75%)
:x:

browser/directives/breadcrumb.js
10% <0%> (-1.12%)
:x:

browser/services/request.js
20% <0%> (-0.84%)
:x:

browser/model/virtualbox.js
67.6% <0%> (-0.4%)
:x:

browser/directives/pathValidator.js
82.97% <0%> (-0.36%)
:x:

browser/pages/start/controller.js
44.79% <0%> (-0.04%)
:x:

browser/model/cdk.js
100% <0%> (ø)
:white_check_mark:

browser/model/helpers/version.js
100% <0%> (ø)
:white_check_mark:

browser/pages/location/controller.js
100% <0%> (ø)
:white_check_mark:

browser/services/data.js
100% <0%> (ø)
:white_check_mark:

... and 13 more

Continue to review full report at Codecov.

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


Add Notification menu support

The Notification section including the following features:

Create/Edit/Delete Notifications/Subscriptions
Display Notification/Subscription/Transmission list
Transmission by Notification Slug View

Closing the PR due to DCO checking failed. Will open another one.


Feat/make platforms2

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] The changes are appropriately documented (if applicable).
[x] The changes have sufficient test coverage (if applicable).
[x] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
This pull request is a continuation of #144 -- @anulman's work plus a fix to how we determine makers' supported platforms as discussed here and here, and then rebased onto the latest master with conflicts resolved.
NOTE: This changes the maker API in a backwards-incompatible way. All makers in the project have been updated, but external makers' signatures will need to updated to be compatible.
Many thanks to @anulman for doing the heavy lifting here, and for being cool with me touching things up while he was traveling 😄

I eagerly await the major version bump :smile:
I wonder why code coverage dropped.

The tests are passing, but coveralls is complaining. I don't really have much experience with coveralls, so I poked around a bit but couldn't really figure out why the coverage dropped...anybody able to provide some insight?

Will play with this locally for manual verification but looking good 👍


impl Drop flush for Journal, resolve cyclic Arcs on Drop

See https://github.com/fjall-rs/fjall/discussions/44

https://github.com/fjall-rs/fjall/commit/8787345a84e58c0160e01bdf5baf3df22e3b8339
https://github.com/fjall-rs/fjall/commit/5cd33a51722b08ec4457bfff0d4ab825401b7190


tmux support

These features don't seem to work when in a tmux session. Is this a limitation of iTerm2 or is there some way support can be added to this project?

Hello @amdprophet I have no knowledge about tmux. I think it's feasable since I made a Exit+Save button for vim that way:
~/.vimrc

autocmd VimEnter * silent !echo -ne "\033]1337;PopKeyLabels\a"
autocmd VimEnter * silent !echo -ne "\033]1337;SetKeyLabel=F1=Save & Quit\a"
autocmd VimEnter * map <F1> :wq<CR>
autocmd VimEnter * silent !echo -ne "\033]1337;SetKeyLabel=F2=Quit\a"
autocmd VimEnter * map <F2> :q!<CR>

Does that can help you?

so looks like this actually works when tmux is started using the tmux iterm integration thing. Might have something to do with the zsh-hook thing.

Merged 👍

It doesn't work for me in tmux. I'm inside a box via ssh that has tmux, could that be a problem?

Not working for me in local tmux.


Add support for RelatedManager.add() through_defaults (Django 2.2)

This is a proof of concept intended to address https://github.com/jazzband/django-taggit/issues/629.
Problems:

I don't think this change would be compatible with Django < 2.2, at least not in its current state.
I am not sure how to test this properly (I made an attempt but haven't gotten it working).

I would be happy to put some more work into this if it's a desired change. If so, I would appreciate some guidance on how to go about testing it.

Hi @soulshake Thanks for the PR. Are you interested in cleaning up the failing tests?

@jdufresne I would be, but I wasn't sure how to go about it.

I did some digging - each of the test case classes use specific models to test. The tests for using custom through models all use the OfficialFood model, and the unit test for testing through_defaults should be in the TaggableManagerOfficialTestCase test class.

Testing with Django 1.11 seems to work fine, since the through_defaults are passed into get_or_create() as the defaults kwarg, which has support across all the targeted Django versions.

It would be good to change the function signature for add() so that through_defaults is pulled out of keyword args. With it defined as a positional argument you would have to explicitly pass in None when using any kwargs, such as in #653

please fix the merge conflict


After alexaPi setup I cannot access :5050 --> HTTP500

Thank you for reporting an issue with AlexaPi.
Make sure you are running the latest version of AlexaPi, and look through issues before submitting.
Check out our Wiki, and make sure to look at Debugging, and Audio debugging in particular before filing an issue.
In order to help troubleshooting, be sure to include the following information:
Output of the audio debugging script.
--> Not yet at that stage
Your OS (including version) where you are running AlexaPi:
RASPBIAN STRETCH WITH DESKTOP
Image with desktop based on Debian Stretch
Version:June 2018
Release date:2018-06-27
Kernel version:4.14

Your hardware platform and model you are running on:
Rasberry Pi2B

Python release (python2 --version):
2.7.x

Description of problem:
Hi,
I tried this approach several times.
Initially setup fails on the issue reported in #415.
The setup runs fine after installing CherryPy 10.2.1

However, there is no way that I can get to http://<IP_ADDRESS>:5050
It returns http500 ... No clue what is going wrong.
My Python is 2.7.x and I run the latest Raspbian

I reran setup several times, but no succes.
Even after getting clean OS on SD, the failure is reproducible.
Any clues are helpful. I want to get this up and running this weekend.
From everything I read, most people are able to run the setup on Raspbian without issues, but I am apparently doing something wrong all the time :-(

Expected:
Setup runs without issues and  I can get to http://<IP_ADDRESS>:5050

Problem-relevant config.yaml entries:

Steps to reproduce:

start setup
enter amazon keys&secrets when asked
navigate to  http://<IP_ADDRESS>:5050

Traceback (if applicable):

Additional info:

FIXED IT
Ok, I was able to fix this....
Here are the steps that are different from the standard install:

First I did this: https://github.com/alexa-pi/AlexaPi/issues/415#issuecomment-418217508

Next I continued as stated here: https://github.com/alexa-pi/AlexaPi/issues/415#issuecomment-427680722

In the previous step (2) was stated to manually run the authorisation script using sudo python3 auth_web.py to get through. This did not work in my case due to a missing reference to yaml.

Install pyyaml. Note that I did this for both v2 and v3 so sudo pip install pyyaml and sudo pip3 install pyyaml

Now when I run sudo python3 auth_web.py the webpage does not load due to an http 500.... It took some extensive debugging to find out that this was caused by an error with the 'six' library: 2018-10-12 10:42:22,910 cherrypy.error ERROR    [12/Oct/2018:10:42:22] ENGINE AttributeError("module 'six.moves.urllib_parse' has no attribute 'unquote_to_bytes'",)
That is why I had to do step 6)

I installed the latest version of six for both v2 and v3 so sudo pip install six==1.11.0 and sudo pip3 install six=1.11.0

After step 6 running the sudo python3 auth_web.py still failed with the six error. Possibly due to path issues.
I was finally able after step 6 to simply run sudo python auth_web.py and I got a token from Amazon.

I think actually things started working because of the 'six' version. My suggestion is to make sure you have the proper 'six' version (1.11.0) installed and the proper cherrypy version (10.2.1) and then run the original install script without the change mentioned in step 2. If that does not work, also do the change from step 2, but I am not sure if it is really needed
I hope this is helpful to anyone facing the same issue.


Add page selection per pdf

Add page selection per PDF that goes into the merge.
The user can use a input and file in the pages like: 1,4,7-9 so the merge will only select 1,4,7,8,9 to merge.

Fixed in React refactor #7


Not calling the callback

It could be on my end, but any help would be appreciated!
This query is not calling the callback, as it doesn't even print "
Phases"
mysql_tquery(MySQLHandle, "SELECT * FROM houses", "OnStartLoadHouse", "");
`forward OnStartLoadHouse();
public OnStartLoadHouse()
{
print("Phase 1");
if(cache_num_rows())
{
    print("Phase 2");
    for(new i, j = cache_num_rows(); i < j ; i++)
    {

		// cache stuff

    }
}
print("Phase 3");
return 1;

}`
Screenshot of the log

Is that really the whole log? Also the query seems to be different in the log from what you described here (SELECT * FROM kuce vs. SELECT * FROM houses).

Translated it for you kuce = houses.
There is a bit more on the top I think, but that's the connection part. Not really a problem in this right?

Okay I am closer to the solution. As you've seen, it puts OnStartLoadHouse (the query is under OnGameModeInit) callback on a delay. No "Phase" prints whatsoever...
BUT, look what happens when I start the server and login. You can clearly see it puts OnAccountCheck on a delay and then continues the OnStartLoadHouse callback
The log.txt

This is weird, indeed. Give me some time to look into that log.
Can you try to delay the house select query for a second?

I haven't got into the game, but looking at the log it successfully loads the house when using a delay of 2 seconds (tickcount). Here is the log
Log 2.txt

Sorry for the late response.
The query worker threads seem to initialize very late in your case. Is your PC maybe too slow to keep up? Also disable the connection pool if you don't need it.

A very late response indeed. I am no longer playing SA:MP or creating a gamemode.
My PC wasn't slow at that time so I really don't know what was causing it. Well, now it doesn't matter much, eh?

Any fixes? I have the same problem for some time now, havent found any fix.

@thesten22 Do your worker threads also initialize way too late, or are your query callbacks just not called? If it's the latter, that's a too generic issue. Open up a new issue with some code and debug log output.

@maddinat0r They aren't called at all. They are called only when a player joins the server


GEOMESA-1317 Document adding new copyright license files

Adds brief description of using the License Maven plugin.

looks good to me


Package request: GameConqueror

Project description
GameConqueror is the graphical (GTK) frontend for scanmem, a memory scanner.
Scanmem is already packaged for NixOS.
Metadata

homepage URL: https://github.com/scanmem/scanmem
source URL: https://github.com/scanmem/scanmem
license: GPL3
platforms: linux

Here is Arch Linux's package for GameConqueror: https://archlinux.org/packages/community/x86_64/gameconqueror/

I'll give this a try

Any news for this one? Scanmem fortunately already in the library, don't know about this one.

@neonfuz sorry for tagging, but any news on this package? No pressure, I also want to try it, might be nice if you have a base that I could work on

Whenever I try to build gameconquerer, I keep getting:
Traceback (most recent call last):
  File "/nix/store/64ffx4937mw9506x7myfa534ixdy11bd-scanmem-0.17/share/gameconqueror/GameConqueror.py", line 34, in <module>
    import gi
ModuleNotFoundError: No module named 'gi'

I'm very new to nixpkgs, so any pointers to help make this work would be welcome.

@jgonyea the error means a dependency is missing.
When searching the error message in nixpkgs, i found this commit: https://github.com/NixOS/nixpkgs/commit/88f749545c8a3299bdf6f856285c32058063f6cf
Maybe you should also use buildPythonApplication.
The use of python3 and python310Packages might lead to mixing different python versions and cause issue.

I found how to wrap the python package in the buildInputs stage, but now I'm running into gtk issues:
Traceback (most recent call last):
  File "/nix/store/9gd22ly8hyq7z7zirycgyrc73frsn48n-gameconqueror-0.17/share/gameconqueror/GameConqueror.py", line 35, in <module>
    gi.require_version('Gtk', '3.0')
  File "/nix/store/kwsw90jz5adz7swvr58k8gi9pkgz6v4l-python3-3.10.9-env/lib/python3.10/site-packages/gi/__init__.py", line 126, in require_version
    raise ValueError('Namespace %s not available' % namespace)
ValueError: Namespace Gtk not available

I have gtk in the buildInputs stage, as well as wrapGAppsHook in thenativeBuildInputs, but nothing else I try seems to make it get past the error above when I attempt to run the application.  My latest attempt is at the link above

@jgonyea based on this PR https://github.com/NixOS/nixpkgs/pull/189505/files , I think you need to add gobject-introspection to nativeBuildInputs.

Any progress on this?

I've dug into this, and the problem seems to stem from 2 things, as I can get it to compile, but not fully run. The first problem is that the program seems to still be running on Python2, which NixOS already ree's at when you try to install it. The second problem is that, if you use a newer version of python, it complains about name spaces, which is a common issue when pygobject is running on a version of python that is "too new".
The solutions seem to be to either, patch out the check for gtk3, but patch files are voodoo magic to me, I do not understand how they work, and this seems like a terrible idea. Or to make a pipx venv within the nix profile, and just add everything needed for python 2.7, and it should theoretically work just fine from there, but I'm unsure if the pyvenv would create issues with sandboxing. Regardless, attached is the modifications I've made to the existing scanmem Nix Package, to get gameconqueror to compile and install.
It does install and compile it, but the above issues are where the roadblock comes in.
default.nix.zip
You can install it and see the result yourself using
nix-build -E 'with import  {}; callPackage ./default.nix {}'
In a terminal, but make sure the terminal is in the same directory when you extract it.

https://github.com/NixOS/nixpkgs/pull/244030

As the PR is merged, GC works as expected in unstable. Unless someone wants to request a backport, this issue can probably be closed.


Wrong execution order in afterStep/afterScenario

With this configuration
beforeScenario: function(scenario) {
    console.log(scenario.name);
  },

afterStep: function(stepResult) {
    console.log('Status:', stepResult.status);
    if (stepResult.status === 'failed') {
      browser.saveScreenshot(
        './errorShots/' +
          stepResult.feature +
          '_' +
          stepResult.scenario +
          '_' +
          stepResult.text +
          '.png'
      );
      console.log('Screenshot created');
    }
  },

afterScenario: function(scenario) {
    console.log(scenario.name);
  },

I hope to obtain this:
GOOGLE
Status: passed
Status: passed
GOOGLE
YANDEX
Status: passed
Status: failed
Screenshot created
Status: skipped
YANDEX

But sometimes I obtain this:
with image created
GOOGLE
Status: passed
Status: passed
GOOGLE
YANDEX
Status: passed
Status: failed
Status: skipped
YANDEX
Screenshot created

And sometimes this:
without create image
GOOGLE
Status: passed
Status: passed
GOOGLE
YANDEX
Status: passed
Status: failed
Status: skipped
YANDEX

@christian-bromann This is the issue i was talking about in ticket https://github.com/webdriverio/webdriverio/issues/3074
The issue is seen in the latest wdio-cucumber-framework.
This issue is not in the version wdio-cucumber-framework: 1.0.3. I have a project and confirmed.
I have created a sample git project for you to reproduce the isuse:
https://github.com/NaveenThiyagarajan/wdio_cuke_hookIssue.git
Thanks.
Naveen

@NaveenThiyagarajan see my comment in https://github.com/webdriverio/webdriverio/issues/3068#issuecomment-444011064
it will help you.

Thanks @BorisOsipov I tried using the cucumber default hooks
After(() => {
    if(Status.FAILED === 'failed'){
        browser.screenshot().value;
        console.log('Screenshot taken');
    }
});

... But the allure-reporter was acting up weird with that .. :( it was showing skipped tests as passed.

... But the allure-reporter was acting up weird with that .. :( it was showing skipped tests as passed.

I haven't seen such issue in allure repo...

... But the allure-reporter was acting up weird with that .. :( it was showing skipped tests as passed.

I haven't seen such issue in allure repo...

Ok .. I got it.. It fixed once i moved to 0.8.3 allure-reporter. Thank you.. good for now.


Feature/175- social log in

Is your feature request related to a problem? Please describe.
Not applicable
Describe the solution you'd like
The social log in using google has been implement it in the backend. Now, the corresponding part of the frontend should be implement it.
Describe alternatives you've considered
Not applicable

Story Points: 8
See the discussion in Frontend S3
Powered by Parabol


WSO2 IS 5.3 - I can not create new IDPs on Solaris.

I have been testing IS on solaris for several weeks with jdk 1.8 144. Everything works correctly.
However, when I try to register a new IDP from the console the browser accesses to the url:
https://myHost-fqdn:9443/carbon/idpmgt/idp-mgt-add-finish-ajaxprocessor.jsp?X-CSRF-Token=QHG4-GFEI-78V8-LKLH-5690-QYI8-YO7Y- IT3Y
and it gives the error "Non secure connection".
I added the exception to the browser to accept the self-signed carbon certificate.
I tried a clean installation where I only changed carbon.xml to adjust the host name.
I have tried the same in linux and it works correctly.

https://wso2.org/jira/browse/IDENTITY-6905

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[Feat] implement share functionality

Could you please assign this issue to me?

Hi, @subrat611 , you may work on this and please refer to the Contribution guidelines on the readme once.
All the best!

Closing this because of no response or pr from your side.

Due my end sem examination I am not able to work on this issue.
If the issue has been fixed let me know.


Suggestions concerning the world map

I have some suggestions concerning the way data is displayed on the world map as well as its performance:

Comparability of markers
Currently, some markers on the map show the number of cases in a country (e.g. South Korea, UK, France) whereas others only represent the number of cases in a specific region of a country (e.g. in Japan, Germany, US). This results in the markers not being very well comparable to each other. This problem could be solved by summarizing the numbers of cases of all regions of each country and displaying them as a single marker.

Loading times
The largest bottleneck of the website is Plotly.js. Loading this file asynchronously is likely to result in a considerable performance boost. However, this is currently not possible because the world map is processed by the server, which requires the client to load Plotly.js before the page finishes loading. To allow Plotly.js to be loaded asynchronously, the world map needs to be processed on the client.

Comparability of markers

One bullet 1, it is something I've come to consider because it skews the true comparison of the visualization among countries. I had decided to just wait it out, because JHU is constantly updating the datasets. Some months back there was no data on South American cities and now there's plenty.
So again, unless we could source other countries' data, I would say leave it as such.

Loading times

I'm open to whatever change you deem appropriate on this!

Alright. I'm going to start working on improving the loading times then.

#57, thanks to @Rahmsauce


[BUG]Azure.Identity 1.13 Breaks DefaultAzureCredential On DevBox

Library name and version
Azure.Identity 1.13
Describe the bug
My code is using DefaultAzureCredential(true) to authenticate to remote azure resources. After upgrading Azure.Identity to 1.13 a handful of users are noting that they are getting the following error when running inside of Microsoft DevBox:

'ManagedIdentityCredential authentication failed: [Managed Identity] Authentication unavailable. Either the requested identity has not been assigned to this resource, or other errors could be present

There are no assigned identities to these DevBoxes (we are using the devbox.microsoft.com instances not selfhosted). Downgrading to 1.12 seems to have resolved this issue but putting this bug here as it seems like the detection logic around DefaultAzureCredential may be incorrect in 1.13.
I was under the impression that DefaultAzureCredential was a chained credential and that any failures in the chain would only be thrown back to the user if none of the credentials in the chain succeeded. However, the interactive popup never showed up, also some users have the VSCredential and the MI credential error happens seemingly as soon as it fails.
Expected behavior
Using DefaultAzureCredential(true) inside a DevBox VM with no managed identity would attempt other credentials in the chain before throwing an exception back to the user.
Actual behavior
DefaultAzureCredential(true) throws an exception on DevBoxes if there is no MSI attached to the devbox and doesn't attempt any further credential operations.
Reproduction Steps

Create a DevBox at (https://devbox.microsoft.com/)
Create a c# project with Azure.Identity 1.13
Make a call to new DefaultAzureCredential(true).GetTokenAsync(<some token request context>)
Observe the ManagedIdentity exception

Environment
Microsoft Devbox (https://devbox.microsoft.com/)

This is possibly related to #46653

We're also running into this error running on our DevBox. We're using the Azure.Identity in combination with the Microsoft.Configuration.ConfigurationBuilders.AzureKeyVaultConfigBuilder class. Using version Azure.Identity 1.12 there were no problems. Upgrading to 1.13 gives the following error:

Description: An error occurred during the processing of a configuration file required to service this request. Please review the specific error details below and modify your configuration file appropriately.
Parser Error Message: The configBuilder 'AzureKeyVault' failed while processing the configuration section 'appSettings'.: 'AzureKeyVault' GetValue() Error: ManagedIdentityCredential authentication failed: [Managed Identity] Authentication unavailable. Either the requested identity has not been assigned to this resource, or other errors could be present. Ensure the identity is correctly assigned and check the inner exception for more details. For more information, visit https://aka.ms/msal-managed-identity.
Status: BadRequest
Content:
{"error":"invalid_request","error_description":"Identity not found"}
Headers:
x-ms-request-id: 6b9bc649-6a39-4e2d-9e72-a6b8b23abc6d
Date: Thu, 17 Oct 2024 07:21:26 GMT
Server: IMDS/150.870.65.1475
[Managed Identity] Error Code: invalid_request Error Description: Identity not found
See the troubleshooting guide for more information. https://aka.ms/azsdk/net/identity/managedidentitycredential/troubleshoot

Thanks for reporting - I am investigating.

Hi @keggster101020
I am unable to reproduce this. Could you try creating the credential the following way to see if you still do not get prompted for interactive credentials?
Also, could you enable logging and share that with me for the failure scenario? For console output, you would just need to add the following line to your program.
using AzureEventSourceListener listener = AzureEventSourceListener.CreateTraceLogger();

If you need to log the output somewhere other than the console, this is described in these docs.
            var options = new DefaultAzureCredentialOptions
            {
                ExcludeEnvironmentCredential = true,
                ExcludeWorkloadIdentityCredential = false,
                ExcludeManagedIdentityCredential = false,
                ExcludeAzureDeveloperCliCredential = true,
                ExcludeSharedTokenCacheCredential = true,
                ExcludeVisualStudioCredential = true,
                ExcludeVisualStudioCodeCredential = true,
                ExcludeAzureCliCredential = true,
                ExcludeAzurePowerShellCredential = true,
                ExcludeInteractiveBrowserCredential = false
            };
            var cred = new DefaultAzureCredential(options);

Same issue is happening from Azure DevOps pipelines while connecting to an Azure SQL Database within an AzureCli@2 task.
Pipeline is executing EF Core SQL migrations executable against Azure SQL connecting as the pipeline identity (federated). Upgrading to 1.13.0 breaks with a very similar error. {"error":"invalid_request","error_description":"Identity not found"}
Downgrading back to 1.12 resolves the issue.
Not sure which token source is being picked up in 1.12 but sounds like it should use WorkloadIdentityCredential before ManagedIdentityCredential?
2024-10-17T14:01:54.5569573Z Status: BadRequest
2024-10-17T14:01:54.5569889Z Content:
2024-10-17T14:01:54.5570233Z {"error":"invalid_request","error_description":"Identity not found"}
2024-10-17T14:01:54.5570484Z
2024-10-17T14:01:54.5570793Z Headers:
2024-10-17T14:01:54.5571125Z Server: IMDS/150.870.65.1475
2024-10-17T14:01:54.5572761Z x-ms-request-id: 0778ae6f-ee8f-480c-bf61-f6b9baace128
2024-10-17T14:01:54.5573303Z Date: Thu, 17 Oct 2024 14:01:52 GMT
2024-10-17T14:01:54.5573518Z
2024-10-17T14:01:54.5573849Z [Managed Identity] Error Code: invalid_request Error Description: Identity not found
2024-10-17T14:01:54.5574524Z    at Microsoft.Identity.Client.ManagedIdentity.ImdsManagedIdentitySource.HandleResponseAsync(AcquireTokenForManagedIdentityParameters parameters, HttpResponse response, CancellationToken cancellationToken)
2024-10-17T14:01:54.5575130Z    at Microsoft.Identity.Client.ManagedIdentity.AbstractManagedIdentity.AuthenticateAsync(AcquireTokenForManagedIdentityParameters parameters, CancellationToken cancellationToken)
2024-10-17T14:01:54.5575688Z    at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.SendTokenRequestForManagedIdentityAsync(ILoggerAdapter logger, CancellationToken cancellationToken)
2024-10-17T14:01:54.5576253Z    at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.GetAccessTokenAsync(CancellationToken cancellationToken, ILoggerAdapter logger)
2024-10-17T14:01:54.5576752Z    at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.ExecuteAsync(CancellationToken cancellationToken)
2024-10-17T14:01:54.5577398Z    at Microsoft.Identity.Client.Internal.Requests.RequestBase.<>c__DisplayClass11_1.<<RunAsync>b__1>d.MoveNext()
2024-10-17T14:01:54.5578181Z --- End of stack trace from previous location ---
2024-10-17T14:01:54.5578616Z    at Microsoft.Identity.Client.Utils.StopwatchService.MeasureCodeBlockAsync(Func`1 codeBlock)
2024-10-17T14:01:54.5579228Z    at Microsoft.Identity.Client.Internal.Requests.RequestBase.RunAsync(CancellationToken cancellationToken)
2024-10-17T14:01:54.5579838Z    at Microsoft.Identity.Client.ApiConfig.Executors.ManagedIdentityExecutor.ExecuteAsync(AcquireTokenCommonParameters commonParameters, AcquireTokenForManagedIdentityParameters managedIdentityParameters, CancellationToken cancellationToken)
2024-10-17T14:01:54.5583764Z    at Azure.Identity.MsalManagedIdentityClient.AcquireTokenForManagedIdentityAsyncCore(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5585033Z    at Azure.Identity.MsalManagedIdentityClient.AcquireTokenForManagedIdentityAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5585841Z    at Azure.Identity.ImdsManagedIdentityProbeSource.AuthenticateAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
2024-10-17T14:01:54.5587148Z    at Azure.Identity.ManagedIdentityClient.AuthenticateCoreAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
2024-10-17T14:01:54.5587757Z    at Azure.Identity.ManagedIdentityClient.AuthenticateAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
2024-10-17T14:01:54.5588623Z    at Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5589250Z 	StatusCode: 0
2024-10-17T14:01:54.5589606Z 	ResponseBody:
2024-10-17T14:01:54.5589931Z 	Headers:
2024-10-17T14:01:54.5590858Z    --- End of inner exception stack trace ---
2024-10-17T14:01:54.5591497Z    at Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex, String additionalMessage, Boolean isCredentialUnavailable)
2024-10-17T14:01:54.5592044Z    at Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5592606Z    at Azure.Identity.ManagedIdentityCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5593429Z    at Azure.Identity.DefaultAzureCredential.GetTokenFromSourcesAsync(TokenCredential[] sources, TokenRequestContext requestContext, Boolean async, CancellationToken cancellationToken)
2024-10-17T14:01:54.5595453Z    at Azure.Identity.DefaultAzureCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5596006Z    at Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex, String additionalMessage, Boolean isCredentialUnavailable)
2024-10-17T14:01:54.5601350Z    at Azure.Identity.DefaultAzureCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5603146Z    at Azure.Identity.DefaultAzureCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5604000Z    at Microsoft.Data.SqlClient.ActiveDirectoryAuthenticationProvider.GetTokenAsync(TokenCredentialKey tokenCredentialKey, String secret, TokenRequestContext tokenRequestContext, CancellationToken cancellationToken)
2024-10-17T14:01:54.5604610Z    at Microsoft.Data.SqlClient.ActiveDirectoryAuthenticationProvider.AcquireTokenAsync(SqlAuthenticationParameters parameters)
2024-10-17T14:01:54.5605321Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.<>c__DisplayClass148_1.<<GetFedAuthToken>b__1>d.MoveNext()
2024-10-17T14:01:54.5606264Z --- End of stack trace from previous location ---
2024-10-17T14:01:54.5606886Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
2024-10-17T14:01:54.5607569Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.GetFedAuthToken(SqlFedAuthInfo fedAuthInfo)
2024-10-17T14:01:54.5608044Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OnFedAuthInfo(SqlFedAuthInfo fedAuthInfo)
2024-10-17T14:01:54.5609251Z    at Microsoft.Data.SqlClient.TdsParser.TryRun(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj, Boolean& dataReady)
2024-10-17T14:01:54.5610134Z    at Microsoft.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj)
2024-10-17T14:01:54.5611372Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.CompleteLogin(Boolean enlistOK)
2024-10-17T14:01:54.5612107Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.AttemptOneLogin(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, TimeoutTimer timeout, Boolean withFailover)
2024-10-17T14:01:54.5613420Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.LoginNoFailover(ServerInfo serverInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString connectionOptions, SqlCredential credential, TimeoutTimer timeout)
2024-10-17T14:01:54.5614565Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds.OpenLoginEnlist(TimeoutTimer timeout, SqlConnectionString connectionOptions, SqlCredential credential, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance)
2024-10-17T14:01:54.5615832Z    at Microsoft.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, SqlCredential credential, Object providerInfo, String newPassword, SecureString newSecurePassword, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling, String accessToken, DbConnectionPool pool, Func`3 accessTokenCallback)
2024-10-17T14:01:54.5617153Z    at Microsoft.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
2024-10-17T14:01:54.5617819Z    at Microsoft.Data.ProviderBase.DbConnectionFactory.CreatePooledConnection(DbConnectionPool pool, DbConnection owningObject, DbConnectionOptions options, DbConnectionPoolKey poolKey, DbConnectionOptions userOptions)
2024-10-17T14:01:54.5618544Z    at Microsoft.Data.ProviderBase.DbConnectionPool.CreateObject(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
2024-10-17T14:01:54.5619089Z    at Microsoft.Data.ProviderBase.DbConnectionPool.UserCreateRequest(DbConnection owningObject, DbConnectionOptions userOptions, DbConnectionInternal oldConnection)
2024-10-17T14:01:54.5619904Z    at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, UInt32 waitForMultipleObjectsTimeout, Boolean allowCreate, Boolean onlyOneCheckConnection, DbConnectionOptions userOptions, DbConnectionInternal& connection)
2024-10-17T14:01:54.5620581Z    at Microsoft.Data.ProviderBase.DbConnectionPool.TryGetConnection(DbConnection owningObject, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal& connection)
2024-10-17T14:01:54.5621265Z    at Microsoft.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
2024-10-17T14:01:54.5621967Z    at Microsoft.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
2024-10-17T14:01:54.5622649Z    at Microsoft.Data.ProviderBase.DbConnectionClosed.TryOpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
2024-10-17T14:01:54.5623221Z    at Microsoft.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry, SqlConnectionOverrides overrides)
2024-10-17T14:01:54.5623663Z    at Microsoft.Data.SqlClient.SqlConnection.Open(SqlConnectionOverrides overrides)
2024-10-17T14:01:54.5624077Z    at Microsoft.Data.SqlClient.SqlConnection.Open()

@christothes  I can look to grab a more official repro when I get some time (I tore down the devbox as part of the testing). However, part of my scenario was also access key vault using configuration builder extensions and the error @rdvanbuuren-gasunie  posted above should be reflective of the error that I saw as well.
Also, I did test the DefaultAzureCredential scenario disabling all providers except VS and interactive and I can confirm that that did resolve my issue. However, I still see this as a regression since prior to 1.13.0 the logic was as I described above: failures wouldn't break the entire chain they would only be thrown back if nothing in the chain resolved.

@keggster101020 I totally agree that a change of behavior would be a regression, I was only asking about the alternative config for troubleshooting purposes since I cannot reproduce this locally. When I fetch a token using DefaultAzureCredential it makes it past the ManagedIdentityCredential and selects the VisualStudioCredential. I presume it would continue on past that to the interactive credential if I wasn't running it from VS.

@keggster101020 or @rdvanbuuren-gasunie - could you share the relevant parts of your configuration for the credential that reproduces the error in that scenario?

@keggster101020 or @rdvanbuuren-gasunie - could you share the relevant parts of your configuration for the credential that reproduces the error in that scenario?

Yes, of course!
We're runnig a .NET 4.8 website using IISExpress on a DevBox, with the following configuration in the Web.config:
<configuration>
  <configSections>
    <section name="configBuilders" type="System.Configuration.ConfigurationBuildersSection, System.Configuration, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" restartOnExternalChanges="false" requirePermission="false" />
  </configSections>
  <configBuilders>
    <builders>
      <add name="AzureKeyVault" vaultName="${KEY_VAULT_NAME}" type="Microsoft.Configuration.ConfigurationBuilders.AzureKeyVaultConfigBuilder, Microsoft.Configuration.ConfigurationBuilders.Azure, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35" />
    </builders>
  </configBuilders>
  <appSettings configBuilders="AzureKeyVault">
    <!-- From KeyVault -->
    <add key="ClientSecret" value="" />
  </appSettings>
</configuration>

This should load the ClientSecret from the KeyVault secrets into the appSetting. If you need anymore information, I'm happy to help!

@christothes for the Azure DevOps Workload/Federated Identity issue (see my comment above), shall I open a new issue or keep it in this thread?

@christothes for the Azure DevOps Workload/Federated Identity issue (see my comment above), shall I open a new issue or keep it in this thread?

I believe that issue is fixed by #46693

seeing the same issue with 13 in devbox

seeing the same issue with 13 in devbox

Hi @aherrick - Could you clarify which issue specifically?

We are seeing a similar issue currently within an Azure DevOps pipeline.
As part of the pipeline an integration test is running which connects to AzureAppConfiguration service.
To make sure we are authenticated to read the AzureAppConfiguration resource, we perform an az login ...  first, using a service principal dedicated for these integration tests.
Within the TestFixture, the connection to AzureAppConfiguration is authenticated using new DefaultAzureCredential()
With Azure.Identity.1.12.0 this process is running fine.
After updating to 1.13.0 it is failing at

Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)

with

Azure.Identity.AuthenticationFailedException : ManagedIdentityCredential authentication failed: [Managed Identity] Authentication unavailable.

The issue is, we are not using ManagedIdentity Authentication anywhere in the devops pipeline, but instead rely on Azure CLI Credentials.
I assume that these mentioned breaking changes are somewhat related, but I really cannot find any EnvironmentVariable or other parameter in the pipeline that would match any of these...

1.13.0 (2024-10-14)
Breaking Changes
Previously, if a clientID or ResourceID was specified for Cloud Shell managed identity, which is not supported, the clientID or resourceID would be silently ignored. Now, an exception will be thrown if a clientID or resourceID is specified for Cloud Shell managed identity.
Previously, if a clientID or ResourceID was specified for Service Fabric managed identity, which is not supported, the clientID or resourceID would be silently ignored. Now, an exception will be thrown if a clientID or resourceID is specified for Service Fabric managed identity.

An alternative explanation would be that, some other errors in the authentication flow of ManagedIdentityCredential that were silently caught in the previous version, eventually leading to the factory to traverse further through the configured identity providers, now actually make the process fail early, and not providing the factory with the chance to continue...
From my local dev-machine everything is running as expected, so there must be something on the MS Hosted agent in devops, that now make the library think it should use ManagedIdentity...

I can confirm this issue.
Create a simple console application
IdentityTest.csproj
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net8.0</TargetFramework>
    <ImplicitUsings>enable</ImplicitUsings>
    <Nullable>enable</Nullable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Azure.Identity" Version="1.12.1" />
  </ItemGroup>

</Project>

Program.cs
Console.WriteLine((await new Azure.Identity.DefaultAzureCredential(false).GetTokenAsync(new Azure.Core.TokenRequestContext(["<Valid scope for your tenant>"], null))).Token);

Run in a pipeline using AzureCLI@2 task;
Pipeline:
pool:
  name: Azure Pipelines
  vmImage: "ubuntu-latest"

trigger: none

steps:
- task: AzureCLI@2
  displayName: Get credentials for pipeline agent principal
  inputs:
    azureSubscription: <Name of Azure Resource Manager Service Connection>
    scriptType: pscore
    scriptLocation: inlineScript
    addSpnToEnvironment: true
    inlineScript: dotnet run --project IdentityTest.csproj

The pipeline will succeed when using Azure Identity 1.12.1
When using Azure Identity 1.13.0 will produce this error:
Unhandled exception. Azure.Identity.AuthenticationFailedException: ManagedIdentityCredential authentication failed: [Managed Identity] Authentication unavailable. Either the requested identity has not been assigned to this resource, or other errors could be present. Ensure the identity is correctly assigned and check the inner exception for more details. For more information, visit https://aka.ms/msal-managed-identity.
Status: BadRequest
Content:
{"error":"invalid_request","error_description":"Identity not found"}

Headers:
Server: IMDS/150.870.65.1475
x-ms-request-id: 85ec817c-d0b7-42f6-892d-c13b6c136982
Date: Wed, 23 Oct 2024 13:20:55 GMT

[Managed Identity] Error Code: invalid_request Error Description: Identity not found
See the troubleshooting guide for more information. https://aka.ms/azsdk/net/identity/managedidentitycredential/troubleshoot
 ---> MSAL.NetCore.<IP_ADDRESS>.MsalServiceException:
	ErrorCode: managed_identity_request_failed
Microsoft.Identity.Client.MsalServiceException: [Managed Identity] Authentication unavailable. Either the requested identity has not been assigned to this resource, or other errors could be present. Ensure the identity is correctly assigned and check the inner exception for more details. For more information, visit https://aka.ms/msal-managed-identity.
Status: BadRequest
Content:
{"error":"invalid_request","error_description":"Identity not found"}

Headers:
Server: IMDS/150.870.65.1475
x-ms-request-id: 85ec817c-d0b7-42f6-892d-c13b6c136982
Date: Wed, 23 Oct 2024 13:20:55 GMT

[Managed Identity] Error Code: invalid_request Error Description: Identity not found
   at Microsoft.Identity.Client.ManagedIdentity.ImdsManagedIdentitySource.HandleResponseAsync(AcquireTokenForManagedIdentityParameters parameters, HttpResponse response, CancellationToken cancellationToken)
   at Microsoft.Identity.Client.ManagedIdentity.AbstractManagedIdentity.AuthenticateAsync(AcquireTokenForManagedIdentityParameters parameters, CancellationToken cancellationToken)
   at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.SendTokenRequestForManagedIdentityAsync(ILoggerAdapter logger, CancellationToken cancellationToken)
   at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.GetAccessTokenAsync(CancellationToken cancellationToken, ILoggerAdapter logger)
   at Microsoft.Identity.Client.Internal.Requests.ManagedIdentityAuthRequest.ExecuteAsync(CancellationToken cancellationToken)
   at Microsoft.Identity.Client.Internal.Requests.RequestBase.<>c__DisplayClass11_1.<<RunAsync>b__1>d.MoveNext()
--- End of stack trace from previous location ---
   at Microsoft.Identity.Client.Utils.StopwatchService.MeasureCodeBlockAsync(Func`1 codeBlock)
   at Microsoft.Identity.Client.Internal.Requests.RequestBase.RunAsync(CancellationToken cancellationToken)
   at Microsoft.Identity.Client.ApiConfig.Executors.ManagedIdentityExecutor.ExecuteAsync(AcquireTokenCommonParameters commonParameters, AcquireTokenForManagedIdentityParameters managedIdentityParameters, CancellationToken cancellationToken)
   at Azure.Identity.MsalManagedIdentityClient.AcquireTokenForManagedIdentityAsyncCore(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.MsalManagedIdentityClient.AcquireTokenForManagedIdentityAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.ImdsManagedIdentityProbeSource.AuthenticateAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
   at Azure.Identity.ManagedIdentityClient.AuthenticateCoreAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
   at Azure.Identity.ManagedIdentityClient.AuthenticateAsync(Boolean async, TokenRequestContext context, CancellationToken cancellationToken)
   at Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
	StatusCode: 0
	ResponseBody:
	Headers:
   --- End of inner exception stack trace ---
   at Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex, String additionalMessage, Boolean isCredentialUnavailable)
   at Azure.Identity.ManagedIdentityCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.ManagedIdentityCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.DefaultAzureCredential.GetTokenFromSourcesAsync(TokenCredential[] sources, TokenRequestContext requestContext, Boolean async, CancellationToken cancellationToken)
   at Azure.Identity.DefaultAzureCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex, String additionalMessage, Boolean isCredentialUnavailable)
   at Azure.Identity.DefaultAzureCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Azure.Identity.DefaultAzureCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
   at Program.<Main>$(String[] args) in /home/vsts/work/1/s/Program.cs:line 1
   at Program.<Main>(String[] args)

@phelm1-landolakes Would you mind verifying with your repro also using this latest build? 1.14.0-alpha.20241023.3

@christothes  Using package version 1.14.0-alpha.20241023.3  succeeds.

This is now available on the public nuget as 1.13.1

Can someone please confirm if this problem is resolved in version 1.13.1? I have recently gotten this error and my Azure.Identity is updated to version 1.13.1

@MHDBST - If you are seeing the same error with 1.13.1, please open a new issue with the specific details of your issue. It may be a different problem.

This is now available on the public nuget as 1.13.1

I ran into issue with this on Azure DevOps instance after upgrade to version 1.13.1 it was resolved. thanks


Links broken when viewing directly in GitHub.

Viewing the docs directly in GitHub results in broken links. e.g. https://github.com/PagerDuty/incident-response-docs/blob/master/docs/getting_started.md
I ran into issues using relative links, so changed to absolute links. Unfortunately this breaks viewing the docs in GitHub. I found it hard to get the links to work in GitHub and on the rendered website at the same time. I'd fix one, and the other would break. For now, I focussed on making sure the rendered website at https://response.pagerduty.com works properly.
This issue is to find a way to get the GitHub links working again at the same time as the website.

Changing this to a "Won't Fix". We'll ensure that the website always works, but having it work natively in GitHub will be considered a "Nice to have" feature rather than a requirement.
We'll fix whatever issues we can, but we aren't going to invest significant effort to get it to work in both places.


What is the watcherID set to when there is an error?

  let watcherId = await BackgroundGeolocation.addWatcher(...
Two questions:

If the plugin fails (throws an error in the callback) what is watcherId set to?

do we need try/catch blocks? or is the error always returned in the callback?

addWatcher can fail due to permission problems. If it fails, the watcher is never created and so there is no need for a watcher ID. You should handle the failure in case permissions are not granted.

Let's think about your example. Suppose the promise returned by addWatcher rejects. Execution of the let statement is aborted, and control is transferred to the nearest catch block in the call stack. Because there is no possible way for any code to access watcherId, its contents do not matter. Does that make sense?


Inventory Setups: When renaming a setup, the bank layout isn't retained

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 11
Bug description

In inventory setups
Create an inventory setup
Select inventory setup
Open bank
Drag items in bank to rearrange
Rename the inventory setup
Open bank

Expected: The layout of the items in the inventory setup should persist even though the name of the inventory steup has changed.
Actual: They bank layout is messed up.
Screenshots or videos

RuneLite version
Runelite: 1.10.7
Launcher 2.4.5
Logs
No response

https://github.com/geheur/bank-tag-custom-layouts/issues/21 that issue already exists on the correct repo, please close this issue.


VSTS giving "Unexpected token \r in JSON during ARM template deployment

Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Environment
https://rankengineering.visualstudio.com/RES/_releaseProgress?_a=release-environment-logs&releaseId=5241&environmentId=13868

Agent - Hosted or Private:
Private agent running on Win 2012 server (VMDUBUILDAGNT01).

Issue Description

As in the screenshot you can see that the JSON validation is failing during the VSTS task deployment for the APIM based on the "Azure Resource Group Deployment" task. However when i use the same template through VS Code and deploy using Powershell using New-AzureRmResourceGroupDeployment command, it works fine and i am able to deploy the API (including policy content as well). I understand that there is a lot of escaping required for inline xml in JSON template, it will be much easier if we can allow the policy file to be deployment using some better and easy way.
I noticed that there is way suggested to deploy the policy file using external URL, but this requires hosting the file on a web server which is not a suitable solution for me.
Error logs
The validation error for the task "Event Notification API" is below:
Error: Ensure the Template file ( 'C:\Agent_work\r1\a_ResAzureIac\RES Templates\CreateAPIM\RESV2\api-eventnotification-template.json' ) is valid. Task failed while parsing with following error: "Unexpected token \r in JSON at position 6190"

@ankurkapoor Azure Resource Group Deployment task simply using JSON.parse() to parse the JSON template. Can you please try parsing your JSON template using powershell or some online tool to figure out the point of parsing failure.
Else please provide us (on<EMAIL_ADDRESS>with your release logs making sure that you've added system.debug=true as a release variable. Also share any sample template similar to api-eventnotification-template.json.

@ankurkapoor we've seen few similar issues in past and mostly its a syntax issue in json template. Powershell is quite lenient over json syntax but you can verify your syntax using some JSON parsers like http://json.parser.online.fr/
Looking forward for your response over the same.

I have come across this issue myself.
In my case it turned out to be an additional comma in the Template parameters, eg.
"parameters": {
    "someProperty": {
        "type": "string"
    },   <--
}

Powershell seems to be fine with it, but the Agent Task does not (As described above)


documentation for M.phi() added

Do the checklist before submitting the PR:

[x] Have you read the guidelines mentioned in CONTRIBUTING.md?
[x] Have you squashed your commits?

Q: What version of Node.js you've used in the project? (e.g. Node v10.11.0)
A: v10.15.3
Q: Mention the Issue Number! (e.g. Fixed #8)
A:  #167

Give additional information regarding the PR below:

documentation for M.phi() added

Codecov Report

Merging #178 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #178   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          78     78
  Lines        1378   1378
=====================================
  Hits         1378   1378

Continue to review full report at Codecov.

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


Access managment

𝗜𝘀 𝘆𝗼𝘂𝗿 𝗳𝗲𝗮𝘁𝘂𝗿𝗲 𝗿𝗲𝗾𝘂𝗲𝘀𝘁 𝗿𝗲𝗹𝗮𝘁𝗲𝗱 𝘁𝗼 𝗮 𝗽𝗿𝗼𝗯𝗹𝗲𝗺? 𝗣𝗹𝗲𝗮𝘀𝗲 𝗱𝗲𝘀𝗰𝗿𝗶𝗯𝗲.
We often would like to know who is connected to a XAF application.
We often ask users to logout from the app and if they do not, it would be nice to force a signout.
𝗗𝗲𝘀𝗰𝗿𝗶𝗯𝗲 𝘁𝗵𝗲 𝘀𝗼𝗹𝘂𝘁𝗶𝗼𝗻 𝘆𝗼𝘂'𝗱 𝗹𝗶𝗸𝗲

The solution must show a list view of who (permissionpoliucyuser) is connected and from what device : Desktop or mobile. Maybe the ip address.
From that list view, an admin can force signout.

𝗛𝗼𝘄 𝘄𝗲 𝗽𝗿𝗶𝗼𝗿𝗶𝘁𝗶𝘇𝗲 𝗶𝘀𝘀𝘂𝗲𝘀
We will try to answer all questions that do not require research within 24hr.
To prioritize cases that require research we use the following labels in order.

❇ Exclusive-Services
❤ Bronze Sponsor
❤ Sponsor
❤ Backer
Installation
ShowStopper
Nuget
Contribution
BreakingChange
ReproSample
Deployment
Must-Have

For all other issues the posting time is respected.

how u imagine the logout? how about a iis restart? will sure log out them all :)

How about : https://stackoverflow.com/questions/41629691/how-to-sign-out-other-user-in-asp-net-core-identity/42389162

cannot answer if this applicable without researching it deep, there should be other ways to do xafApplication.LogOff() using SignalR or Magic.Onion the tasks is not as easy as it sounds though.

I did ask DX in order to see.
Let's wait the answer.
https://supportcenter.devexpress.com/ticket/details/t978442/xaf-blazor-access-management


Steven/cutetimeline

Updated the timeline view
Also added some bugfixes in detail and new event activities handling null values.

eff it. merged.


ossf scorecard: Branch-Protection

Description:
This issue is to drive discussion around the ossf scorecard results for the Branch-Protection section. This is part of improving the project's security posture #653.
The Branch-Protection section of the ossf scorecard highlights the hardening recommendations for GitHub branches. Some of them may conflict with parts of our CI/CD automation, which may mean we won't be able to mitigate.
Result: branch protection is not maximal on development and all release branches
Details:

[ ] Linear history disabled on branch 'master'

[ ] Status checks for merging disabled on branch 'master'

[ ] Stale review dismissal disabled on branch 'master'

[ ] Number of required reviewers is only 0 on branch 'master'

[ ] Owner review not required on branch 'master'

[ ] 'administrator' PRs are exempt from reviews on branch 'master'

@saschagrunert here's my views/questions on the 6 points:

I can't think of any issues enabling linear history for the master branch. What do you think?
This is a false positive, as status checks are enabled for master. No action required here.
The state dismissal can be a problem for us whilst we have some long running/flake tests. I think we could keep as is and review it after further improvements on test suite.
/ 5. / 6. This may be due to the fact that "Include Administrators" is disabled - probably the same is for number 2. Do you think doing the below may break our merge automation?

Enable Include Administrators
Enable Require a pull request before merging
Add k8s-ci-robot to Restrict who can push to matching branches

I can't think of any issues enabling linear history for the master branch. What do you think?

We already have it enabled, right? :)

This is a false positive, as status checks are enabled for master. No action required here.

Agree :+1:

The state dismissal can be a problem for us whilst we have some long running/flake tests. I think we could keep as is and review it after further improvements on test suite.

About which kind of dismissal are we speaking about here? Something like the integration of a stale bot?

/ 5. / 6. This may be due to the fact that "Include Administrators" is disabled - probably the same is for number 2. Do you think doing the below may break our merge automation?

Enable Include Administrators
Enable Require a pull request before merging
Add k8s-ci-robot to Restrict who can push to matching branches

We could give it a try! :)

@pjbgf changed the branch protection rule for master, let's see what it breaks.

@saschagrunert nice one, let's hope it all works out.
By the way, the linear history for master branch still shows as disabled for me:

So we are probably not refering to the same thing. :)

On Thu, Nov 11, 2021 at 02:24:22AM -0800, Paulo Gomes wrote:

@saschagrunert nice one, let's hope it all works out.
By the way, the linear history for master branch still shows as disabled for me:

So we are probably not refering to the same thing. :)

But we don't have those horrible merge commits that GH uses by default
for whatever silly reason. Isn't that it?

@pjbgf I changed it to require a linear history.
It looks like that the other options from the branch protection were gone, so I hope that there is nothing like a nightly sync which enforces a certain branch protection ruleset within this org.

Will keep on until changes can be manually confirmed.

But we don't have those horrible merge commits that GH uses by default for whatever silly reason. Isn't that it?

@jhrozek sorry missed your message here. But I also did not understand exactly what do you mean here, can you clarify please?

After a delay the changes were applied:

Let's keep an eye in the comming days as to the impact this may have.
Closing it for the time being.

But we don't have those horrible merge commits that GH uses by default for whatever silly reason. Isn't that it?

@jhrozek sorry missed your message here. But I also did not understand exactly what do you mean here, can you clarify please?

No problem,  I probably meant something else than you, Sascha and the scorecard meant. I thought we already did have linear history because I don't see merge commits in SPO like I see e.g. in selinuxd (https://github.com/containers/selinuxd/commits/main)


Please add a LICENSE file

https://docs.github.com/en/communities/setting-up-your-project-for-healthy-contributions/adding-a-license-to-a-repository

Thanks for the reminder!
Added in https://github.com/westonb/open-pmic/commit/9f616459827b5ca0565903c2c924ba74b4a80395#diff-c693279643b8cd5d248172d9c22cb7cf4ed163a3c98c8a3f69c2717edd3eacb7


brew cask info gives Not installed for some installed casks (chefdk, vagrant, virtualbox)

Some apps are listed with brew cask list, but brew cask info $app shows Not installed. I have read "brew cask list shows wrong information" but AFAIK, my problem is with cask info.
$ brew cask list
betterzipql  chefdk   java         soundcleod   vagrant  virtualbox   xquartz

However:
$ for app in chefdk vagrant virtualbox; do; echo ---- $app ----; brew cask info $app; done
---- chefdk ----
chefdk: 0.16.28-1
Chef Development Kit, ChefDK
https://downloads.chef.io/chef-dk/
Not installed
https://github.com/caskroom/homebrew-cask/blob/master/Casks/chefdk.rb
==> Contents
  chefdk-0.16.28-1.pkg (pkg)
---- vagrant ----
vagrant: 1.8.5
Vagrant
https://www.vagrantup.com/
Not installed
https://github.com/caskroom/homebrew-cask/blob/master/Casks/vagrant.rb
==> Contents
  Vagrant.pkg (pkg)
---- virtualbox ----
virtualbox: 5.1.2-108956
Oracle VirtualBox
https://www.virtualbox.org
Not installed
https://github.com/caskroom/homebrew-cask/blob/master/Casks/virtualbox.rb
==> Contents
  VirtualBox.pkg (pkg)

It also happens to me, but with different apps:
crashplan, java, tunnelblick.
I also found that brew cask is unable to update applications. I think that cask has in repository newer versions and, if we have older one installed, it states that it is not installed, so it can't update it.
As an example: I have installed java using brew cask, version 1.8.0_92-b14.
Now, output of brew cask info java:
java: 1.8.0_102-b14
Java Standard Edition Development Kit
http://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html
Not installed
https://github.com/caskroom/homebrew-cask/blob/master/Casks/java.rb
==> Contents
  JDK 8 Update 102.pkg (pkg)

On the contrary, as VLC seems OK:
~ brew cask info vlc
vlc: 2.2.4
VLC media player
https://www.videolan.org/vlc/
/usr/local/Caskroom/vlc/2.2.4 (10 files, 252K)
https://github.com/caskroom/homebrew-cask/blob/master/Casks/vlc.rb
==> Contents
  VLC.app (app)
  /usr/local/Caskroom/vlc/2.2.4/vlcwrapper (binary)

Yes, this is a known problem when there is a newer version in the repo than is installed.
Hopefully most of it will be fixed in https://github.com/caskroom/homebrew-cask/pull/23328. But this is still very much a work in progress.

FYI, the not-really-a-fix solution is brew cask uninstall {{cask_name}} --force && brew update && brew cask install {{cask_name}}, but follow https://github.com/caskroom/homebrew-cask/pull/23328 for updates.

Still errors a la https://github.com/caskroom/homebrew-cask/issues/23762 after updated with latest merge. Should that have been fixed too?

@wickles, could you please create a separate issue with your output?

it's AFAIK exactly the same as the other issue, so I'll just move conversation over there.


Prefix label onboarding form

✨ Codu Pull Request 💻
Fixes #(issue)

Pull Request details

Remove prefix as it's currently not possible to do it using the available Catalyst components

Any Breaking changes
Associated Screenshots

@John-Paul-Larkin free for you to re-review


Dataset creation

Because a classification model is just a graph its not just words you need for !KW but a balance of the pronounciation that creates spectra.
https://github.com/StuartIanNaylor/ProjectEars/tree/main/dataset/ml-commons
I was using the ml-commons dataset but its so full of dross its a shame it the only spoken word dataset we have.
I was importing the words into sqllite and using
# Natural Language Toolkit: Tokenizers
#
# Copyright (C) 2001-2022 NLTK Project
# Author: Christopher Hench<EMAIL_ADDRESS>
My hack is here https://github.com/StuartIanNaylor/ProjectEars/tree/main/dataset/ml-commons
The idea is to balance phoneme and sylable as in a classification you have this seesaw effect where if you over bias to a particular phoneme then a lesser phoneme will rank lower.
Phonemes by sylable count where the nearest thing I could think of that directly map to resultant spectra.
Its a gist of likely how the big guys do it but they will have professional linguists to advise on how to create balanced classifications.

PS just to add whilst I came across it, but someone is arguing Multitaper-mel spectrograms for keyword spotting provides more accuracy https://arxiv.org/pdf/2407.04662
Dunno


Create the testing script

Description
Create a script for running Unit Tests
Subtasks

[x] Create the script
[x] Add a badge to the readme

Total: 10m


[FLINK-34883] Fix postgres uuid column as PK

A majority of Postgres databases use UUIDs as primary keys.
When we enable 'scan.incremental.snapshot.enabled = true', Flink-CDC will try to split into chunks.
The splitTableIntoChunks function relies on the queryMinMax function, which fails when trying to calculate the MIN(UUID) and MAX(UUID), as that is not supported in Postgres.
This change those queries to cast the uuid to a String value.

@GOODBOY008 can you help here to find someone that can help review this?

@leonardBang can you help here?

Thanks @eskabetxe for the contribution, I'd like to find someone  who is more familiar to Pg to review this PR.

@loserwang1024 would you like to review this PR?

@GOODBOY008 any chance to get this merged?


Create quantitative analysis script

@brendaju The only further work (maybe) to explore with this would be evaluating the system's performance on an additional scene (maybe like scene 0355 or something). May not have time, but if we do, this would be good to have as well.


fix percent escape not working when not at the beginning of the line

Fixes: https://github.com/sqlalchemy/mako/issues/323
The Lexer now will generate wrong result when there is no \n before %%.
Case:
from mako.template import Template

template = """%% do something
%%% do something
if <some condition>:
    %%%% do something
        """

print(Template(template).render())

The result before fix:
%% do something
%% do something
if <some condition>:
   %%%% do something

The result after fix:
% do something
%% do something
if <some condition>:
   %%% do something

Hi,  @zzzeek  can you please take a look? The fix passed all the tests on my local machine。

this does change the output of rendering so I'm a little concerned about old templates relying on the broken behavior.   I can just put it out there as 1.3.1 and see what happens

New Gerrit review created for change 773b6bdaf3418436a2f3fb0a3052d833c0fedcfd: https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111

At the same time, I found that the current fix is incorrect and generates results that are missing Spaces in some escape percent cases.
case：
from mako.template import Template

template = """
    %%%% do something    # 4 spaces before %
"""
print(Template(template).render())

result only has 3 spaces before "%":
   %%% do something

The logic of the code here will consume one spcae:
https://github.com/sqlalchemy/mako/blob/dc66614a2b9ba19ee880c855471c37cd3307c22a/mako/lexer.py#L360-L362
https://github.com/sqlalchemy/mako/blob/dc66614a2b9ba19ee880c855471c37cd3307c22a/mako/lexer.py#L74-L75

This regex block fixes the issues mentioned above and works correctly.
(?<=\n)(?=[ \t]*(?=%|\#\#))
rewrite to
(?<=\n)(?=[ \t]*(?=%(?!%)|\#\#))
The regex block now:
        match = self.match(
            r"""
                (.*?)         # anything, followed by:
                (
                 (?<=\n)(?=[ \t]*(?=%(?!%)|\#\#)) # an eval or line-based
                                             # comment preceded by a
                                             # consumed newline and whitespace
                 |
                 (?<!%)(?=%%+) # consume the first percent sign out of a group of percent signs
                 |
                 (?=\${)      # an expression
                 |
                 (?=</?[%&])  # a substitution or block or call start or end
                              # - don't consume
                 |
                 (\\\r?\n)    # an escaped newline  - throw away
                 |
                 \Z           # end of string
                )""",
            re.X | re.S,
        )

However, there might be room for improvement, as I am not very familiar with regular expressions.

The test code and regex has been updated.

Patchset 5d3e74223b30a2d3a996255a4df4d5f9a267a5b4 added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111

However, there might be room for improvement, as I am not very familiar with regular expressions.

you're working at expert level already so consider yourself familiar!

Patchset f2820295dd5b391864c416a3d0ffc92fecc5c73b added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111

for the formatting, use the tooling we have :
cd /path/to/mako
pre-commit install
pre-commit run --all

Thanks, it has been updated!

Patchset ab8e74756d1ddfae6584854b2765e3706ba87ad5 added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111

Michael Bayer (zzzeek) wrote:
thank you!
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111

Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5111 has been merged. Congratulations! :)

Hi -
I've reverted the change.   Can we please try again, adding new tests that test the case in #384
$ pip install mako==1.3.0
...
Successfully installed mako-1.3.0
$ mako-render - <<< foo%%bar
foo%%bar
$ pip install mako==1.3.1
...
Successfully installed mako-1.3.1
$ mako-render - <<< foo%%bar
foo%bar

The change needs to be limited to only percent signs as the first non-whitespace character, not any percent signs.

Mako 1.3.1 is yanked

New Gerrit review created for change ab8e74756d1ddfae6584854b2765e3706ba87ad5: https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

Michael Bayer (zzzeek) wrote:
please add tests for double percents in the middle of non-whitespace lines
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

Oh sorry, I will revise and test this part again.

Patchset db9309737277d46976413d90c13811a9c61c46df added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

Michael Bayer (zzzeek) wrote:
wow a whole new method.  OK.   I dont have a lot of time today so ill try to look more closely at this soon.
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

All has been updated . Thanks for taking the time to review !

Patchset ff8ce2e128098560e3988bb2375a49fa8649d3d5 added to existing Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

Michael Bayer (zzzeek) wrote:
thank you again!  we try again.   can keep yanking/reverting til it works :)
View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141

Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/mako/+/5141 has been merged. Congratulations! :)

That positive lookbehind appears to do nothing. Your tests still pass if you use just r"^(\s*)%%(%*)"

Ahh, it seems that there is no difference between the two in MULTILINE mode, if the positive lookbehind will affect performance, maybe it should be changed here.

I believe the positive lookbehind does not get exercised because a pair of percent signs preceded by characters on a line would have been consumed by the "text" regexp.   However, if that were not the case, then this regexp would still need to confirm the percent signs occur subsequent to the beginning of a line with only whitespace in between.
>>> import re
>>> x = "    %%  \n   %%   \n hello %%"
>>> re.compile(r"(?<=^)(\s*)%%(%*)", re.M).match(x, 24)
>>> re.compile(r"(\s*)%%(%*)", re.M).match(x, 24)
<re.Match object; span=(24, 27), match=' %%'>

will affect performance,

it would be negligible, and in fact the lookbehind version is slightly faster (note this is one million calls to the match):
>>> re1 = re.compile(r"(?<=^)(\s*)%%(%*)", re.M)
>>> re2 = re.compile(r"(\s*)%%(%*)", re.M)
>>> import timeit
>>> timeit.timeit("re1.match(x, 24)", "from __main__ import x, re1")
0.08006502594798803
>>> timeit.timeit("re2.match(x, 24)", "from __main__ import x, re2")
0.1264837539056316

the overhead of a new python call to "match_percent()" is going to be much more significant than the reg itself, and this is fine.   compilation performance is not really at stake here these are all minimal changes.

maybe it should be changed here.

definitely not


fix(mobile): fix styling issues in mobile version

remove wrong search bar padding
remove negative margin on banner and horizon (was causing scrolling to the right beyond 100%)
remove gray area on pages

Approving doesn't work somehow ^^' LGTM, though


Can you release the processed real bulk data for model validation?

Hi Kevin,
I come across your paper and find it really useful to deal with the bulk RNAseq data, which is exactly what I need. I wonder if you could release the processed real bulk data listed in Table S2 (PBMC1, PBMC2, Xin, ROSMAP, and Ascites)? So that I can explore the existing real bulk RNAseq data first, and repeat your experiment on the real bulk data.
Thanks ahead.
Best,
Fan

Hi Fan,
unfortunately I can't just distribute that data, as some of it was given to me directly by the authors or has access restrictions. However, I did not do any processing to the bulk RNA-seq data, and I provided links to all datasets in the paper. So you can just download the count/ or normalized data and apply Scaden if you like.
Let me know if you have any issues or trouble downloading the data.
Best,
Kevin

@KevinMenden  Hi Kevin, thanks for your reply. And I'm afraid that I have to ask you the details of each real bulk data collection process. I believe that it will also help others who have the same question.

Hi @yfzon,
sure, if you have a look at the Methods section of the Scaden paper you should find all the details you need  (see below for the part of interest).
For instance, the GEO accession for the PBMC2 dataset: GSE107011
Note also the part at the end:
"For the RNA-seq datasets analyzed in this study, we did not apply any additional processing steps but used the obtained count or expression tables directly as downloaded for all datasets except the ROSMAP dataset"
As I said, we basically used the raw counts, except for the ROSMAP dataset.
Okay - let me know where and with which dataset you are struggling and then I surely can help you out!
Cheers,
Kevin
Text from the Methods section:
"Tissue datasets for benchmarking. To assess the deconvolution performance on real tissue expression data, we used datasets for which the corresponding cell fractions were measured and published. The first dataset is the PBMC1 dataset, which was obtained from Zimmermann et al. (21). The second dataset, PBMC2, was downloaded from GEO with accession code GSE107011 (10). This dataset contains both RNA-seq profiles of immune cells (S4 cohort) and from bulk individuals (S13 cohort). As we were interested in the bulk profiles, we only used 12 samples from the S13 cohort from these data. Flow cytometry fractions were collected from the Monaco et al. publication (10).
In addition to the above mentioned two PBMC datasets, we used Ascites RNA-seq data. This dataset was provided by the authors, and cell type fractions for this dataset were taken from the supplementary materials of the publication (18).
For the evaluation on pancreas data, artificial bulk RNA-seq samples created from the scRNA-seq dataset of Xin et al. (20) were used. This dataset was downloaded from the resources of the MuSiC publication (8). The artificial bulk RNA-seq samples used for evaluation were then created using the “bulk_construct” function of the MuSiC tool.
To assess how Scaden and the GEP algorithms deal with the presence of unknown cell types, we generated PBMC bulk RNA samples from the four scRNA-seq datasets (6000 each). The undefined amount of unknown cells that was generated by this approach was removed to be replaced by defined amounts of 5, 10, 20, and 30% of unknown cells, respectively. Cell fractions of all four samples were predicted with Scaden trained on the other three.
Performance on these samples was then assessed to test robustness against unseen cell types in the bulk mixture. Scaden was trained on samples from all datasets but the test dataset, while CSx and MuSiC used data8k as a reference.
The microarray dataset GSE65133 was downloaded from GEO, and cell type fractions were taken from the original CS publication (6).
Last, we wanted to get insights into neurodegenerative cell fraction changes in the brain.  Here, we use the ROSMAP study cortical RNA-seq dataset along with the corresponding clinical metadata, to infer cell type composition over six clinically relevant stages of neurodegeneration (22). Furthermore, to assess deconvolution accuracy on postmortem human brain tissue, we used 41 samples from the ROSMAP, for which cell composition information from immunohistochemistry (23) was recently released and for which fractions for all cell types were reported. The ROSMAP RNA-seq data were downloaded from www.synapse.org/. The cell composition values were provided by the authors of the study (23).
RNA-seq preprocessing and analysis. For the RNA-seq datasets analyzed in this study, we did not apply any additional processing steps but used the obtained count or expression tables directly as downloaded for all datasets except the ROSMAP dataset. For the latter, we generated count tables from raw FastQ files using Salmon (33) and the GRCh38 reference genome. FastQ files from the ROSMAP study were downloaded from Synapse (www.synapse.org)."


Fixes onDestroy been called multiple times.

LGTM, beware this is a breaking change on flame as we expose ordered_set api

Yeah, but I don't think this is problematic. We can just make this 0.19.0 imo


fix: do nothing when validating webhooks is empty

Description of your changes
Wait do validatingWebhooks will block when validatingWebhooks is empty, we should skip it.

lgtm @Arvintian


SAKIII-4797 validate CLE cookie

https://jira.sakaiproject.org/browse/SAKIII-4797

Hi Stuart, please see the comment in the JIRA about using the /current path to get only the current session for the logged-in user as in:
https://server/direct/session/current.json

New pull request against 1.2.0 at https://github.com/sakaiproject/3akai-ux/pull/1812


test: Prefer ObjectId to Literal '........'

Add testcase for ensuring the absence of any '........' strings within trigger regexes.

Thanks!


Fix bug with if nested, false condition is evaluated if it is an if function

Fix bug with if nested, false condition is evaluated if it is an if function , related to issue : #210
This PR  use a different way to fix this bug  that PR #211
as asked in comment https://github.com/uklimaschewski/EvalEx/pull/211#pullrequestreview-273455041 and https://github.com/uklimaschewski/EvalEx/pull/211#issuecomment-520212442

I hope to find some time this weekend to merge this PR and have a look at the other issues.
I have just started working at a new job, so I am quite busy at the moment. 🙂

I just verified that these changes are passing all tests (including those in my project).

@RobertZenz @uklimaschewski Could you create a new version 2.2 with this fix ?

@laurentyhuel Version 2.2 was built and should pop up at Maven Central in the next while.


[HelpTopic] Update readme for content authors

Explain explicitly which properties are required, defaults, and what formats are accepted.

Readme includes prop descriptions: https://github.com/patternfly/patternfly-quickstarts/tree/main/packages/module#help-topic-type-definition
May revisit extra documentation for content authors in the extension documentation spike on org.


add scheduler plugin ClusterUnschedulable

What type of PR is this?
kind/feature
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Codecov Report

Merging #673 (ec63856) into main (e2ffa5f) will decrease coverage by 0.32%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #673      +/-   ##
==========================================
- Coverage   13.60%   13.29%   -0.32%
==========================================
  Files          66       71       +5
  Lines        7492     7825     +333
==========================================
+ Hits         1019     1040      +21
- Misses       6391     6701     +310
- Partials       82       84       +2

see 40 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Rebase feature/remote-inference to 2.x

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Does this PR only contains rebase and fixing conflicts?

Closing this PR since there's new codes checked in to feature/remote-inference branch and re-rebased in this PR: https://github.com/opensearch-project/ml-commons/pull/969


Wiki API docs have broken links

The DotNetMDDocs-generated documentation links are all broken because GitHub wikis squash all files into a single folder.
Putting our docs into the git repo with the source is a non-starter -- at least not before https://github.com/clcrutch/DotNetMDDocs/issues/7 is fixed. Even then, I'm not sure... I might rather have a site we publish to.
CC: @clcrutch

I've removed the wiki.


Figure out how to handle spaces in filenames for cc65

All filenames passed to cross-assemblers are quoted, but for some reason cl65.exe doesn't work correctly.  Merlin 32 and 64tass quote filenames the same way and work fine.
C:\Src\cc65-2.17\bin\cl65.exe --target none "Name WithSpaces.S"
ExitCode=1 -

----- stderr -----
ca65: Don't know what to do with `WithSpaces.S'

I tested cl65 on Linux and had no problems, but running the command manually from the git shell had the same issue.
Issue filed vs. cc65: cc65/cc65#788

For whatever reason, the cl65.exe I built has this problem, but the pre-built Windows binary (download linked from https://cc65.github.io/) doesn't.  So this isn't something I need to work around.


Profiling updates

Fixes

Fix the destination for annotated files when generating a coverage report where some source files are included via relative path.

Proposed changes

Improve coverage report's rendering of annotated source code:

Add line numbers.
Do not color leading whitespace.
Change DOCTYPE to HTML5.

Add a coverage legend within annotated source files.
Tidy the HTML output in profile report (and in swish-test's HTML mat output)
Adapt to a change from version 9.5.1 of Chez Scheme that maintains profile counts even when the collector reclaims source code.
Strip the common prefix from source files in the coverage summary and sort based on real path.

According to the HTML5 validator:

The type attribute for the style element is not needed and should be omitted.
The summary attribute on the table element is obsolete. Consider describing the structure of the table in a caption element or in a figure element containing the table; or, simplify the structure of the table so that no description is needed.

We made some updates to address HTML5 validator warnings. Turns out they have a container image that makes it a bit handier. There is a CSS warning that looks like a known bug in the validator.
We also ran HTML Tidy 5.8.0.

Thanks for the reviews, Bob!


The thief objective to steal the detective's lighter is impossible if no detective room is mapped.

See: Saltern. Might apply to other lowpop maps. I know Packed has a room.

Spawns in their coat pocket. (which is not intuitive, a common theme in thief objectives)

Also not sure what you mean about Packed, the only mapped one is on Reach, which I don't even think is in rotation.

As for the coat itself, the only mapped one is on Origin. The rest are found inside a detective's locker, or on the detective themselves.

Spawns in their coat pocket. (which is not intuitive, a common theme in thief objectives)

If this is true I'll close this off. I inquired about it earlier and was told that it might've been an oversight or I misinterpreted.

Although, if a detective never spawns, that effectively means any maps without a det room mapped are impossible for the thief to complete their objective. Least that's what happened to me.)


Add market data service for FTX

Describe the solution you'd like

Modify include/ccapi_cpp/service/ccapi_service.h so that market data service class for FTX is enabled there. Modify /include/ccapi_cpp/ccapi_macro.h and /include/ccapi_cpp/ccapi_session_configs.h accordingly. Create /include/ccapi_cpp/service/ccapi_market_data_service_ftx.h and add appropriate logics.
End-to-end test it so that market depth and trade data can be successfully streamed.

Special thanks to @donzthefonz who contributed 99.9% of the code.


Create signal to calculate budgets

Purpose
Project's total_estimated_budget and actual_cost should be updated in the Budget model's signal.
Furter Info
Related ticket: https://github.com/Humanitec/ActivityAPI/issues/419

Done! @Menda


Adding dotnet tool

Adding ability to install each tool from dotnet tool
Should close #11

Really love this addition. Spanning MacOS as well with this addition! :)


Meetings schedule

I am not sure where the discussion happened, but i believe it was agreed to do the commcomm meetings every two weeks instead of every week.
It would be awesome if someone has the time to update the recurring event on google calendar to accommodate for this change.

@Bamieh I think the Node.js Calendar has been updated to reflect the meeting schedule. I recommend subscribing to http://nodejs.org/calendar . The calendar currently shows Thursday, 2018-01-04 as the next scheduled meeting.

@bnb @rachelnicole I'd love your confirmation that the CommComm calendar has settled at least for January and that the Node.js Calendar is up-to-date. As the Champion for @nodejs/user-feedback I need to schedule time to attend CommComm meetings and have a number of agenda items that need to be addressed. Thank you. 🙏 A little bit of clarity now will make the holiday break less stressful since the 4th occurs vacation time.

@dshaw Oops my bad. I have already subscribed to the calendar but it was still showing today's meeting for some reason 😄 . Seems that i have the meeting copied to my calendar and it doesnt reflect updates on the original. Feel free to close the issue.
Cheers!

@Bamieh Yeah, I had the same thing happen to me with the previous CommComm meeting. I had copied it to my calendar.

@dshaw the next meeting should be this Thursday - I assume the meetings were entered manually and not as a recurring meeting, because I moved the last one and none of the upcoming meetings were updated - noted, and I'll set them up as recurring for the future.
@Bamieh as per our schedule, there should normally be a meeting every other week - hopefully the calendar wasn't too confusing, an oversight on my part. Since the last meeting was December 14th, the next meeting should be December 18th - I'll be creating the issue tomorrow 👍

@bnb It is clear, it's just that i have copied the old calendar so i got confused, my bad.

Since the last meeting was December 14th, the next meeting should be December 18th

28th*


Move Future into seperate Interop

Feel free to close this if it doesn't make sense.
But some (maybe many) of us are looking at IO as what Future should have always been. Unfortunately some existing libraries e.g HTTP Clients have Future return types and so it makes sense to include the interop module. Including the interop library I assume brings alongs all the scalaz/cats/etc dependencies as well.
Might be worth treating Future as a seperate case ?

Both dependencies are optional and won't be pulled by interop if the user's project doesn't yet depend on them
https://github.com/scalaz/scalaz-zio/blob/1af623cce75d20fc09feace67295c69b781ec8e3/build.sbt#L80-L81


CI: cleanup test databases automatically

The CI machines we using are not necessarily single-use. The tests currently leave behind lots of LocalDB databases in the USERPROFILE. This can cause test failures on subsequent builds of CI.
For example: https://dev.azure.com/dnceng/internal/_build/results?buildId=41727&view=logs

System.Data.SqlClient.SqlException : Cannot create file 'C:\Users\dlab14\CreateTest.mdf' because it already exists. Change the file path or the file name, and retry the operation.
CREATE DATABASE failed. Some file names listed could not be created. Check related errors.

I tried adding an additional build step which executed tools/CleanMSSQLLocalDB.cmd script before and after tests, but this failed with "sqlcmd not found."

@smitpatel @bricelam Any ideas here?

Is anything calling sqllocaldb d(elete)? It's unusual that the files exists but the databases don't...

Might be a good opportunity use a different localdb instance and put all the database files under the bin folder instead.

Install sqlcmd ? https://www.microsoft.com/en-us/download/details.aspx?id=53591

@bricelam and @AndriySvyryd will look into how we manage our databases on the new C.I. machines.

Note that we don't need to continue using localdb if using some other SQL Server edition would be better.

Fixed by #14522

See also #15122