BUGFIX: Fixed issue where the $id variable would be overridden in derived fields

When using derived fields subsequent iterations of the loop on line 527 can in some cases be reset to an id that is not the one being re-indexed resulting in a SQL error.

Looks like the travis build is failing because of an unrelated issue see previous build: https://travis-ci.org/silverstripe-labs/silverstripe-fulltextsearch/builds/117662029

Please rebase on master; https://github.com/silverstripe-labs/silverstripe-fulltextsearch/pull/114 has fixed the broken build.

@tractorcow looks like its passing now :)


close JDBC resources in TlsIntegrationTest

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

Project coverage is 0.00%. Comparing base (59551e4) to head (2a51e72).
Report is 184 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #12752       +/-   ##
=============================================
- Coverage     61.75%    0.00%   -61.75%
+ Complexity      207        6      -201
=============================================
  Files          2436     2387       -49
  Lines        133233   130893     -2340
  Branches      20636    20270      -366
=============================================
- Hits          82274        6    -82268
- Misses        44911   130887    +85976
+ Partials       6048        0     -6048

Flag
Coverage Δ

custom-integration1
?

integration
<0.01% <ø> (-0.01%)
:arrow_down:

integration1
<0.01% <ø> (-0.01%)
:arrow_down:

integration2
0.00% <ø> (ø)

java-11
?

java-21
<0.01% <ø> (-61.63%)
:arrow_down:

skip-bytebuffers-false
<0.01% <ø> (-61.75%)
:arrow_down:

skip-bytebuffers-true
0.00% <ø> (-27.73%)
:arrow_down:

temurin
<0.01% <ø> (-61.75%)
:arrow_down:

unittests
?

unittests1
?

unittests2
?

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

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

ready for review

ready for review @Jackie-Jiang


[beta] remaining backend challenges - no boilerplate gomix project to remix

@Greenheart @Em-Ant It looks like the remaining backend challenges (everything after managing packages with npm) requires a boilerplate project for campers to remix (also missing from npm challenges intro - #13091) that is to be used throughout and continually built upon all the way through the end of the infosec section. Is this right?
Either first challenge or intro should have a link. I was able to find the boilerplate link for the npm section in the CurriculumExpansion branch, but not for this section.
If this is the case, maybe we should also make a clear note in the first landing page (or on each landing page) that these are supposed to be cumulative, so if campers attempt to jump in at a later section, they know they'll need to go back and start from basic node and express?

Ok, so it looks like each section has it's own boilerplate. These need to be added to the landing pages for each section so that others can start working on them as well Thanks for posting this link @Em-Ant!
https://hackmd.io/GYVgTBIIxgtAzATgCYCNYBYCGZ2oBwbqLCJSKIj4DG1y8QA=

@Em-Ant Also, none of these boilerplates will work until the CORS settings are updated as @JosephLivengood pointed out for the NPM challenges in #13095
I forked your repo for the Basic Node and Express challenges and changed this line of index.js and updated the package.json of the GoMix project to reflect my repo, and that all did the trick. I'm assuming this is an issue that is consistent throughout.

@no-stack-dub-sack It turns out he had it fixed, we were using outdated boilerplates.
I can confirm the boilerplate works as intended without edits or fixes on the hackmd.io, however the solution to it is using the outdated version.
@Em-Ant @no-stack-dub-sack I'll edit the solution as soon as I can to share the same working code as the updated boilerplates 👍

@JosephLivengood perhaps only for the NPM boilerplate? The Node and Express boilerplate from hackmd.io pulls from @Em-Ant's repo that I linked to above (in which the CORS error still seems to be an issue)

@no-stack-dub-sack Possible, I only have checked NPM since I'm working in the advanced section right now. Is it all of the boilerplates apart from npm needing that fix?

@JosephLivengood Not sure, I've only moved on to the Node and Express one, but that one for sure does not work as is

@no-stack-dub-sack I'll make that fix real fast

@no-stack-dub-sack Looks like it was just that one missing it, I added it into the boilerplate but I do not have access to Em's solutions!

Oh, ok, great! What did you have to do to fix it, if it wasn't changing the URL to beta.freecodecamp.com in the CORS setting like you did the other day?

@no-stack-dub-sack Just updated the code from by default restricting to just those, to not restricting any request (by default at least) as Em did

@JosephLivengood I don't see that in the boilerplate's code - did you change it in the dependency package? "https://github.com/em-ant/fcc-express-bground-pkg.git"

@no-stack-dub-sack Haha no problem, made me double check 👍
Want to hop on gitter to communicate about these? I'll be there pretty late in the night working on some stuff!

@JosephLivengood Yup! Just IMd you


React to deprecation of Fedora.28.Amd64.Open

The dedicated Fedora.28.Amd64.Open queue will be shutdown on May 29. This updates to use the Docker queue instead.

Getting a consistent failures now in Fedora tests.

mkdir: cannot create directory '/root/helix/work/correlation/sdk': Read-only file system

Must be missing something obvious. @MattGal @HaoK is the entire filesystem read-only when using dockerized Helix? Or just some directories?

Yeah we can't write to the correlation payloads on the dockerized queues, we will need to switch to a different directory for these queues

Alternatively, maybe we should just pull upload the cli/sdk as correlation payloads instead of having each worker install them?

Updated to put the SDK into the working directory instead of the correlation folder.
Looks like test failures now are due to the normal flakiness. Going to retry once more and then will add Flaky attributes.

/azp run AspNetCore-helix-test

Lots of very consistent Blazor test failures plus numerous random-seeming test failures. Error for the Blazor tests is:
System.IO.FileNotFoundException : Could not find file '/home/helixbot/work/309432f4-a992-4f9f-bafa-2a93fbf1a09c/Work/636c0f27-9b3f-4806-9849-6df7f6a3e897/Exec/referenceHints.txt'

In addition, the test view does not show anything for Fedora ☹

@dougbu not sure what's up with the Fedora but the MC view from the logs (https://mc.dot.net/#/user/aspnetcore/pr~2Faspnet~2Faspnetcore/ci/20190521.2 ) shows 100% pass rate for Fedora 28 so that's good.

112 failures overall. Going to take a closer look and start applying [Flaky] attributes.
FYI @aspnet/build

Upon further review, it appears I broke the existing skip conditions. I fixed up the environment variables we are setting and added a comment to code to indicate that the HELIX env var is expected to be set to the 'friendly' queue name (not the fully-qualified version which lists the docker image)

/azp run AspNetCore-helix-test

Only test failure on Helix now is https://github.com/aspnet/AspNetCore-Internal/issues/2490.
I'd like to merge this today. @HaoK can you review and sign off if the changes look ok?


Temp fix for freezing UI

Signed-off-by: Guy Ackermans<EMAIL_ADDRESS>

I think the problem here is two-fold:
1 The asset search is currently throwing an exception.
2 Any uncaught exceptions thrown by the asset search can cause the UI platform to become unavailable because while the spinner is there, the user is unable to even navigate to the UI views that do work. Ideally I think we should have a high level, general UI platform exception catcher which allows the user to still navigate to other UI tabs even though a specific tab may thow an unhandled exception.
I agree my workaround is a bad approach long-term, but right now it is the only way for me to be able to navigate to other views than Asset Search.


No actual value of each bar in the graph is painted

Hi I ran into an possible issue. I am running node 7.4.0 installed with homebrew on osx. I've installed<EMAIL_ADDRESS>and<EMAIL_ADDRESS>globally with npm install -g. When i run either pwmetrics or with the —runs=3 option I get an output like this.

I get an rough idea about the dimension by the 10.000 value on the x-axis but the specific values are missing like in the example screenshot in the readme.md file. If you need any further informations let me know. Best regards r.

@rpkoller thanks for pointing this out. We have fix for it.


proposal: cmd/go: go test flag -skip should be cacheable

Proposal Details
Running tests with the -skip flag should be able to use the test result cache.
In the original design of the test result cache (#11193), the test flag -run is a cacheable flag. However, the subsequent proposal (#41583) that added the -skip flag to go test but did not make it cacheable.
Both -skip and -run are flags used to match or filter the test cases. Since the -run parameter is cacheable, the -skip parameter should also be cacheable, too.

Related Issues

proposal: cmd/go: add flag `go test -no-cache` #39056 (closed)
proposal: cmd/go: go test: allow `-run` and `-skip` flags to specify a package #68401 (closed)
cmd/go: -benchtime flag should be cacheable #44555 (closed)
proposal: testing: flag show skipped #34306 (closed)
proposal: testing: add a flag to detect unnecessary skips #25951
cmd/go: doc mentions "cacheable flag" but never lists them #42544 (closed)
cmd/go: go test flag `-failfast` should be cacheable #47355 (closed)
cmd/go: "help test" caching paragraph is out-of-date and potentially misleading #48484
testing: enforce -skip in example tests #61491 (closed)
cmd/go: -exec prevents test caching  #27207

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)


Alerting shows error

Error while setting alert in Grafana:

tsdb.HandleRequest() error Could not find executor for data source type: hawkular-datasource

Alerting is not available with the hawkular datasource (or any other third-party datasource since there's no public API for that), so you cannot use alerts from within Grafana.
However hawkular has its own alerting engine that you could use instead, though it's not visible from grafana dashboards: example here http://www.hawkular.org/hawkular-services/docs/quickstart-guide/#_step_3_add_alerting


Sending StreamContent greater than 10mb from Container App to Container App Fails

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
I have two dotnet 6 container apps deployed in 1 environment.
When trying to send a file greater than 10mb via StreamContent, http client fails with exception.
Any stream under 10mb succeeds to be retrieved on the 2nd container app.
Steps to reproduce

Deploy 2 dotnet 6 container apps
Have a the first container app send an http request to the 2nd container app, with stream content greater than 10mb
Alternatively you can send it via postman to the first, then have the first send the stream to the second

Expected behavior [What you expected to happen.]
I expected that the 2nd container app would receive the request successfully.
Actual behavior [What actually happened.]
An error ocurred, the exception is as follows:
Message:
"Error while copying content to a stream."
Exception:
{"Unable to write data to the transport connection: An existing connection was forcibly closed by the remote host.."}
CallStack:

"   at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ThrowException(SocketError error, CancellationToken cancellationToken)\r\n   at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.System.Threading.Tasks.Sources.IValueTaskSource.GetResult(Int16 token)\r\n   at System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable.ConfiguredValueTaskAwaiter.GetResult()\r\n   at System.Net.Security.SslStream.<g__CompleteWriteAsync|182_1>d1.MoveNext()\r\n   at System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable.ConfiguredValueTaskAwaiter.GetResult()\r\n   at System.Net.Security.SslStream.<WriteAsyncChunked>d__1811.MoveNext()\r\n   at System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable.ConfiguredValueTaskAwaiter.GetResult()\r\n   at System.Net.Security.SslStream.d__191`1.MoveNext()\r\n   at System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable.ConfiguredValueTaskAwaiter.GetResult()\r\n   at System.Net.Http.HttpConnection.d__78.MoveNext()\r\n   at System.Runtime.CompilerServices.ConfiguredValueTaskAwaitable.ConfiguredValueTaskAwaiter.GetResult()\r\n   at System.Net.Http.HttpContent.<g__WaitAsync|56_0>d.MoveNext()"

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
What I tried:

Deployed the 2nd container app to Azure App Services and sent the same payload to it, no error occurred.
Turned the ingress to public on the 2nd container app, send the payload directly via Postman, no error occurred.
I am wondering if there is some hidden configuration on ACA where it limits how much data we can send to the service, because anything under 10mb is successful.

Sample Code:
    internal static async Task<IResult> Upload(HttpRequest request,IHttpClientFactory factory)
    {
        var myclient = factory.CreateClient("myclient");

        myclient.BaseAddress = new Uri("https://{{host}}.westus.azurecontainerapps.io");

        using var streamCopy = new MemoryStream();
        await request.Body.CopyToAsync(streamCopy);
        streamCopy.Position = 0;
        var uploadContent = new StreamContent(streamCopy);

        uploadContent.Headers.ContentType = new MediaTypeHeaderValue(request.ContentType);
        uploadContent.Headers.ContentLength = request.ContentLength;

        var result = await myclient.PostAsync("/upload", uploadContent);
        var message = await result.Content.ReadAsStringAsync();

        return Results.Ok(message);
    }

I'm unable to repro this. This is what I used
Program.cs

using System.Net.Http.Headers;

var builder = WebApplication.CreateBuilder(args);
// default is 30MB?
builder.WebHost.UseKestrel(o => o.Limits.MaxRequestBodySize = long.MaxValue);
var app = builder.Build();

var client = new HttpClient()
{
    Timeout = TimeSpan.FromMinutes(4)
};

app.MapPost("/upload", async ctx =>
{
    var targetApp = Environment.GetEnvironmentVariable("TARGET_APP") ?? "target-app";

    using var streamCopy = new MemoryStream();
    await ctx.Request.Body.CopyToAsync(streamCopy);
    streamCopy.Position = 0;
    var uploadContent = new StreamContent(streamCopy);
    // or just pass the stream directly to avoid buffering it in memory unless needed
    // var uploadContent = new StreamContent(ctx.Request.Body);

    uploadContent.Headers.ContentType = new MediaTypeHeaderValue(ctx.Request.ContentType);
    uploadContent.Headers.ContentLength = ctx.Request.ContentLength;

    var result = await client.PostAsync(new Uri($"https://{GetInternalFqdn(targetApp)}/upload-internal"), uploadContent);
    var message = await result.Content.ReadAsStringAsync();

    await ctx.Response.WriteAsync(message);
});

app.MapPost("/upload-internal", async ctx =>
{
    await using var file = new FileStream("/tmp/test", FileMode.OpenOrCreate, FileAccess.Write);
    await ctx.Request.Body.CopyToAsync(file);
    await ctx.Response.WriteAsync("success");
});

string GetInternalFqdn(string appName)
{
    var fqdn = Environment.GetEnvironmentVariable("CONTAINER_APP_ENV_DNS_SUFFIX");
    return string.IsNullOrEmpty(fqdn) ? appName : $"{appName}.internal.{fqdn}";
}

app.Run();

created 2 apps with the same image, public-app, internal-app, and a 100MB file
$ az containerapp create --name public-app --image ahmelsayed/upload-app --ingress external ...
$ az containerapp create --name target-app --image ahmelsayed/upload-app --ingress internal ...

$ head -c 104857600 </dev/urandom > 100mb.file
$ curl https://public-app.env.westus.azurecontainerapps.io --data-binary @100mb.file -v

> POST /upload HTTP/2
> Host: public-app.env.westus.azurecontainerapps.io
> user-agent: curl/7.79.1
> accept: */*
> content-length: 104857600
> content-type: application/x-www-form-urlencoded
>
* We are completely uploaded and fine
< HTTP/2 200
< date: Thu, 29 Sep 2022 04:49:59 GMT
< server: Kestrel
< content-type: text/plain; charset=utf-8
<
* Connection #0 to host public-app.env.westus.azurecontainerapps.io left intact

success%

Do you see any particular error on the target app?

If you can share your app url and a rough time frame with me I can check what error our proxy saw between the 2 apps. you can use ahmels at microsoft.com if you don't want to share the details publicly.


Markdown_Paths Array fails if larger than 5 items

The npm run render command will succeed but generate an empty index.html if the markdown_path[] is larger than 5 items:
This works:
"markdown_paths": [
        "header.md",
        "abstract.md",
        "summary.md",
        "motivation.md",
        "scope.md"
      ],

This does not:
"markdown_paths": [
        "header.md",
        "abstract.md",
        "summary.md",
        "motivation.md",
        "scope.md".
        "introduction.md
      ],

I'm suspect about this, because we have two other specs using the tool with 15+ files and it runs fine. It see a period instead of a comma on your fourth item, could that be causing the json to be invalid?

Zooming in:

Agreed. Fixed. Thanks.


training pytorch RT-DETR on single-class object detection

Hello and thanks for your work. I have a question regarding training RT-DETR on a custom dataset with only one object class (e.g. Face). Does this repo work with single-class object detection? If so what should I change in the code?
Thanks.

Yes, It can handle single-class case


Move JSInterop to AspNetCore repo from Extensions

Why? Does it include a namespace change? Can you add some more details about the reasoning?


Redundant if/else statement + potential double transfer issue?

This expression makes no sense:
if (
      offChainDepositAssetBalance.gt(0) &&
      offChainWithdrawAssetBalance.gt(0)
    ) {
      console.warn(
        'Balance exists in both channels, transferring first, then withdrawing'
      );
      await transfer(
        _depositChainId,
        _withdrawChainId,
        _depositAddress,
        _withdrawRpcProvider,
        offChainDepositAssetBalance,
        _evts,
        _node,
        true
      );
      return;
    }
    // if offChainDepositAssetBalance > 0
    if (offChainDepositAssetBalance.gt(0)) {
      // then start transfer
      await transfer(
        _depositChainId,
        _withdrawChainId,
        _depositAddress,
        _withdrawRpcProvider,
        offChainDepositAssetBalance,
        _evts,
        _node,
        true
      );
    }

From this line: https://github.com/connext/vector-modal/blob/main/src/components/Modal.tsx#L910

Fixed in main.


Footer navbar responsive

Modified footer section and Navbar and make it suitable for all devices
previous-

NOW

@Niladri2003 please resolve the conflicts and commit again

Everything is fine from my side .
can you please help me to solve the issue
I think someone also working on same problem that's why conflicts are happening

@Niladri2003 you'll have to sync your branch with the main this branch

@Niladri2003 I'll suggest you to make new PR

Ok .


The presentation video isn't available in France

On this page, the youtube presentation video isn't available for France.
http://www.awesomescreenshot.com/image/750749/117cb04220c38c2c5216cce1ef9be777

It's just a music video for the following song.
REO Speedwagon - Roll With The Changes


How to apply in video gaze correction?

Hi:
Your work of gaze correction in img seems not suitable for video application due to inconsistency and flicker between frames. Do u have any idea about video gaze correction?

Hi: Your work of gaze correction in img seems not suitable for video application due to inconsistency and flicker between frames. Do u have any idea about video gaze correction?

until now , can you provide some paper can instruct video gaze correction ?


Tuple vs Struct - Option in Path Extractor

I have been trying to set up a handler that takes 2 path parameters where the second one is optional.
This only seems to work as I expect when the generic type for the Path parameter is a struct where the second path param is an Option type. But it doesn't work if I try to use a 2 entry tuple type with the second type being an Option.
Is this expected and if so what is the reason?
Here is my attempt at a fairly minimal example
use poem::{handler, test::TestClient, web::Path, Route};

#[derive(serde::Deserialize)]
pub struct TestParams {
    param_1: String,
    param_2: Option<String>,
}

#[handler]
pub fn test_struct_handler(Path(TestParams { param_1, param_2 }): Path<TestParams>) -> String {
    format! {"struct: {param_1} & {param_2:?}"}
}

#[handler]
pub fn test_tuple_handler(Path((param_1, param_2)): Path<(String, Option<String>)>) -> String {
    format! {"tuple: {param_1} & {param_2:?}"}
}

#[tokio::main(flavor = "current_thread")]
async fn main() {
    let app = Route::new()
        .at("/struct/:param_1", test_struct_handler)
        .at("/struct/:param_1/:param_2", test_struct_handler)
        .at("/tuple/:param_1", test_tuple_handler)
        .at("/tuple/:param_1/:param_2", test_tuple_handler);

    let cli = TestClient::new(app);

    cli.get("/struct/foo/bar").send().await
        .assert_text("struct: foo & Some(\"bar\")").await;

    cli.get("/struct/foo").send().await
        .assert_text("struct: foo & None").await;

    cli.get("/tuple/foo/bar").send().await
        .assert_text("tuple: foo & Some(\"bar\")").await;

    cli.get("/tuple/foo").send().await
        .assert_text("tuple: foo & None").await; // This fails with status 400, with message "invalid path params"
}

Thanks!

Same problem here.


How do I monitor hov many events has been raised for a specific event type ?

How do I monitor hov many events has been raised for a specific event type ?
The page doesn't seem to explain any thing about that. That would be very usefull.

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

ID: c5ed2449-4122-2660-56be-78ceebd0de66
Version Independent ID: b5b1c94e-3efe-b234-5366-79823678f237
Content: Monitor Azure Event Grid message delivery
Content Source: articles/event-grid/monitor-event-delivery.md
Service: event-grid
GitHub Login: @spelluru
Microsoft Alias: spelluru

@cjjohansen
Thanks for your Question! We are currently investigating and will get back on this.

@cjjohansen, Thanks for raising this issue. Are you trying to get metrics by event types for System Topics?


feat: receiving Image and other types of message in conversation form

In conversation form, we can receive a text like
const text = await conversation.form.text();
Can we also receive a photo and other types of message in this format?

There are a lot of ideas what to wait for. This will not be done for 1.0. We should first figure out #11 and then we can look into adding more things. I would be happy to take ideas regarding #11.

There are a few issues with the existing form controls already. For example, form.text reacts to callback queries because updates types are not checked properly. This should be made consistent first, before adding more features.
Do you have time to work on this?


Lift

The code for the lift system needs to be written. The lift system will have motors controlled by a PID loop using encoders and potentiometers probably.

I will implement this when we get details on exactly how the lift is going to work.


Unable to pull 1.1.4

~$ docker pull factoriotools/factorio:1.1.4 Error response from daemon: manifest for factoriotools/factorio:1.1.4 not found: manifest unknown: manifest unknown
This reports as being v1.1.3 according to log.
~$ docker pull factoriotools/factorio:latest  latest: Pulling from factoriotools/factorio Digest: sha256:5ed8c9bd2b9ef54ceb867c859881e998c32a9200769c36ca85fdb73fca0adaee Status: Image is up to date for factoriotools/factorio:latest docker.io/factoriotools/factorio:latest

+1
The build job is stuck for 4 hours !

Looks like the "build & pull to docker" job on Travic CI is queud: https://travis-ci.org/github/factoriotools/factorio-docker/jobs/747441340

Done :)

Looks like 1.1.6 is stuck now too?  1hr and counting.

1.1.6 is stuck now too.

Travis is a bit slow currently. If this persists and gets worse over time I might need to rebuild the CI on github actions.


Validator incorectly validate URI

When i was writing manifest for GameSave Manager I experienced scoop validator falsly make https://www.gamesave-manager.com/?s=download&guid={D5312F19-F659-40E1-869A-1C991BB0E95B}&a=dl#/cosi.zip as error when running tests (generic tests from extras bucket).
Installing (powershell 5, 6.1.1) and downloading via browser work good, but test failed with:

From my testing problems are curly brackets.

Use URL encoded string:
{ = %7B
} = %7D


[question] What is the mechanism of "qttools.searchMode": "cmake"?

Excuse me, I want to verify the mechanism of "qttools.searchMode": "cmake". Here is my opinion:

If we use find_package(Qt6) or find_package(Qt5) in CMakeLists.txt, then the Cache Variable Qt6_DIR or Qt5_DIR will be written in the CMakeCache.txt after configuring the CMake project.

Then qttools will use this Qt6_DIR or Qt5_DIR to locate some tools such as designer.exe relatively.

Am I correct?

Correct

I think the best solution (for now) is to use qttools.extraSearchDirectories to add the bin directory of Qt Kits manually. For example:
{
  "qttools.extraSearchDirectories": [
    "C:\\Qt\\6.2.0\\msvc2019_64\\bin",
  ]
}

In other words, I have to install Qt by Official Installer, anyway.
As for Qt installed by Conan, I don't need and had better not to add qt:tools=True option in conanfile.txt because it seems that there are only Binary Packages of Qt with default option of qt:tools=False in conan-center-index. That is, it will take a lot of time to compile the Source Package for the first time if I add qt:tools=True option in conanfile.txt.


feat(ios): return asset id on ios

Thanks for submitting a PR! Please read these instructions carefully:

[ ] Explain the motivation for making this change.
[ ] Provide a test plan demonstrating that the code is solid.
[ ] Match the code formatting of the rest of the codebase.
[ ] Target the master branch, NOT a "stable" branch.

Motivation (required)
What existing problem does the pull request solve?
Test Plan (required)
A good test plan has the exact commands you ran and their output, provides screenshots or videos if the pull request changes UI or updates the website. See What is a Test Plan? to learn more.
If you have added code that should be tested, add tests.

Oh my bad! I was a little confused as code contains both a refactor (which I think is great!) and adding the ID to response. So ignore my last comment about using the ID for search.. So next steps I'd say would be:

Let's separate the refactor out into a separate PR ? (I support this change and using the function fetchPHAsset but I think we should keep any refactoring code separate from this so we can get it merged into the main repository faster, then we can PR the clean-up separately)
Let's include the Android part
Let us decide a new name for includeExif and I can make the change to update that

Let me know if you think this makes sense

@lukebrandonfarrell OK great, thank you. That makes sense to merge them separately!
I don't mind who merges it 😎 Early next week sounds awesome. Happy to do it if it's next week as I'm away this week. Just let me know 😄
No problem, thank you for yours! 🙌

Great @helenaford, I've pushed the updates Johan requested to https://github.com/react-native-image-picker/react-native-image-picker/pull/1795 then I'll come back and do a final CR on this, thanks!

Hi guys, thanks for such a great library and all the awesome work you do :)
Is there any chance including the asset id on the image returned after taking a photo with launchCamera could be added in a future update?  I noticed that and the timestamp are not included, even with includeExtra set to true, in IOS only (on Android I am getting both of those when taking a photo).


Fix memory leak in RaiCore

The API functions xrb_sign_transaction and xrb_hash_transaction requires the caller to free.

Good call, thank you


cmd/list: support using --full-name with --installed-on-request et al.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Fixes https://github.com/Homebrew/brew/issues/18908#issuecomment-2532926860.

Thank you so much ♥♥♥

Thanks @ZhongRuoyu!


Mouse wheel scrolls vertically rather than zooming

Can you tell me more about where you're seeing this?

On the canvas. If I scroll with my mouse's wheel, the canvas view scrolls up and down, when I would expect it to zoom in and out. But maybe my expectation is wrong?

You can cmd/ctrl + wheel to do that if you like. We've been around the houses on this one a few times, with some folks preferring scroll to zoom, others preferring scroll to move 😄

Who wants to scroll with the mouse wheel on a 2D canvas? Absolute madness.
No that's fair. Closing this if this an intentional decision. A setting might be nice at some point tho.


Please mention that everyone can update the PowerBI report and not just read it

Our user would like to use this feature, but they realized that everyone could edit the PowerBI report and just not read it. This limitation makes complicated to maintain a single source of truth for all the one who may access the 'vizualize this list'.
So please mention that everyone can update the PowerBI report and not just read it in the Consideration and limitations: https://learn.microsoft.com/en-us/power-bi/create-reports/service-quick-create-sharepoint-list#considerations-and-limitations
Thanks,
Best regards,
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: be7b35f1-76fc-3c6c-b97c-42b9724c7644
Version Independent ID: edb54e4e-c730-5573-fe30-d331946a7c54
Content: Create a report quickly from a SharePoint list or library in the Power BI service - Power BI
Content Source: powerbi-docs/create-reports/service-quick-create-sharepoint-list.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Thanks for your comment, @charletsjp. This issue has been assigned to the doc author for review.

@charletsjp thanks for your comment! I've added it to the Considerations section of https://learn.microsoft.com/en-us/power-bi/create-reports/service-quick-create-sharepoint-list#considerations-and-limitations. The change should be live later today Fri Oct 28 Pacific time.


Review Enterprise 1.9 auth docs

@lesam @pierwill
Does it makes sense (and does it cover all bases) to replace the entire "User management" section here with a link to Set up users with fine-grained authorization?
If so, let's do that, and then publish. If that doesn't cover all bases, let's create another doc issue to track this change specifically, and push out this PR, which addresses many broken links and starts folks on the auth path.

Originally posted by @kelseiv in https://github.com/influxdata/docs-v2/pull/2778#discussion_r662517992

I think we need to re-organize the security sections of the docs under Administration.  There are a number of submenu items that I believe we need to collect together.  @kelseiv and I were looking at this earlier today.

no... it does not cover all the bases....
If I break this down a bit more --- looking at what you can do with OSS and what you can do with Enterprise:

OSS supports the ability to setup users (READ/WRITE/ALL/ADMIN) permissions.  If you do nothing more, you have that permission for all DBs within the instance.

OSS ALSO allows for setting database level grants.  Meaning, within an instance, you can grant/deny access to the various databases within that instance (i.e. not a blanket grant....like in no. 1)

This is generally covered here :https://docs.influxdata.com/influxdb/v1.8/administration/authentication_and_authorization/#user-types-and-privileges
It appears that the only way to maintain these today is via InfluxQL commands.  This can be issued via the CLI (interactive session) or the Query API or via Chronograf.  However, Chronograf has specific UI pages which supports No. 1, but if you want to do No 2....you will need to submit individual InfluxQL queries through the Data Explorer (in the same way that you would use the CLI or Query API).

Enterprise shifts the number of user permission from 4 to 16.   These are covered here: https://docs.influxdata.com/enterprise_influxdb/v1.9/features/users/

These permissions can be set via the API and there are specific API calls to allow for User Management within InfluxDB Enterprise.  These are documented here: https://archive.docs.influxdata.com/enterprise_kapacitor/v1.5/administration/auth/#user-and-privilege-management-over-the-influxd-meta-api
Chronograf can again be used to set the individual user permissions when connected to InfluxDB Enterprise.
InfluxQL and the interactive mode using InfluxQL via the CLI cannot be used to set the Enterprise user permissions.  However, there is no distinction, warning or other indication that the user should NOT use these metaqueries for user management.

Enterprise appears to allows for TWO mechanisms for layering in addition permissions.  The first of which matches No. 2.  The mechanism exposed for OSS.  The other is the Fine Grained Authorization which is described here: https://docs.influxdata.com/enterprise_influxdb/v1.9/guides/fine-grained-authorization/

What is unclear?

Customers who are upgrading from OSS to Enterprise.... what happens to the existing users and their permissions when the Enterprise upgrade is complete?

What takes precedents the InfluxQL meta queries or the InfluxDB Meta API -- for user management?  Should a user be prevented from using the InfluxQL meta queries against InfluxDB Enterprise?

Both of these scenarios need some hands-on testing, walk-through, and exploration to understand the behavior so that we may document this really, really clearly.
My current guess is that only a subset of our customers are using the finer grained permissions offered by InfluxDB Enterprise and most resort to the broad permission set offered through the InfluxQL meta queries.


Password not working?

Please paste the output of the following command here: sudo edit-chroot -all
We trust you have received the usual lecture from the local System
Administrator. It usually boils down to these three things:

    #1) Respect the privacy of others.
    #2) Think before you type.
    #3) With great power comes great responsibility.

Password:

Please describe your issue:
No matter what I do, it doesn't accept my password? The first time I didn't specify a password for the debugging, and test0000 was not being accepted. The second time I specified a password and that's not being accepted either.
If known, describe the steps to reproduce the issue:

@clownkitty,
Yes, that's the correct behavior if you've enabled Debugging Features.
See if this helps: #2050 (comment)
Hope this helps,
-DennisLfromGA

so i should do it again and then not enable the debugging? :0

@clownkitty,
If the comment I linked above doesn't help and you can't first login as root and supply a valid password then, yes, I suppose you may have to powerwash your device losing any files, folders, & chroots you haven't backed up and start over. Only this time do not enable Debugging Features at the OOBE screen.
FYI: You can enable any or all of the Debugging Features after you sign-in to your account profile by running the below debugd/helper scripts as root:
/usr/libexec/debugd/helpers/dev_features_chrome_remote_debugging
/usr/libexec/debugd/helpers/dev_features_password
/usr/libexec/debugd/helpers/dev_features_rootfs_verification
/usr/libexec/debugd/helpers/dev_features_ssh
/usr/libexec/debugd/helpers/dev_features_usb_boot

Hope this helps,
-DennisLfromGA


Packages with a bit more structure

Packets services and actions now have some sub packets.

Coverage decreased (-0.6%) to 8.903% when pulling 18fae1be0ad0c86f10dad64411ec733feb3cdc09 on package-fix into c1d26c1bad9571f10c23b6701ed42e682a6a6576 on master.


Document Container color constructor argument

I'm not sure how widespread this particular little issue is but I found it for the Container Widget and Container Class pages.
In short, the Container has a color property that is part of the constructor and this is clearly listed at the top of the page for the Container constructor, here:
https://docs.flutter.io/flutter/widgets/Container/Container.html
However, I mistakenly thought that the Container would not have a color property because color is not listed as one of the properties on the right hand side of the page for the Container class, here:
https://docs.flutter.io/flutter/widgets/Container-class.html
After having it pointed out to me that the Container does, in fact, have a color property, I went digging down into the Class page rather than relying on the summary or properties and methods on the right. There, in the text about the constructor, it does clearly state that there is a color property as part of the constructor.
This is not an urgent issue but it's not trivial either, since I'm attempting to put together presentation material on Flutter for Google Developer Groups and meetups and I've created materials based on those summaries in the class pages for each widget.
Interest in Flutter is growing exponentially, and those of us stepping out to speak to groups about Flutter can't afford to be putting out bad information. I'll be extra careful to triple check and not completely trust the summaries from now on but we really do need to ensure that every list of a Class' properties contains... well, all the properties.
Thanks and, as always, I appreciate all the fantastic, hard work that's been going on. Just because "you missed a spot", that doesn't mean that the thousands of spots you didn't miss aren't great. They are!
Scott

This one is tricky. The reason why color didn't show up as a property is that the color argument of the constructor didn't actually become a property, when a Container object gets instantiated. Instead, it gets passed to the constructor of a BoxDecoration object to style the Container. Please see these lines in the source code: https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/widgets/container.dart#L259-L263
One potential solution is to show constructor arguments instead of properties on the sidebar. Cc: @jcollins-g who works on DartDoc.

See, that's what I thought. I thought that the decoration was what had the color but when you read that the container can take a color parameter in it's constructor it leads one to believe the Container has a color property somewhere.
Not an easy fix, I agree. I can see that I'll be bending a lot of ears to triple check my facts before I step in front of an audience.

Closing this because the documentation of the Container class now shows the color property and is well documented.
https://api.flutter.dev/flutter/widgets/Container-class.html


RuntimeError: You must compile your model before using it

Getting the following error in MacOS for cnn
from imagededup.methods import CNN
cnn = CNN()
encodings = cnn.encode_images(image_dir='/Users/test/Downloads/tojpg/')
duplicates = cnn.find_duplicates(encoding_map=encodings)
print(duplicates)
2020-09-30 16:48:50,853: INFO Initialized: MobileNet pretrained on ImageNet dataset sliced at last conv layer and added GlobalAveragePooling
2020-09-30 16:48:50,853: INFO Start: Image encoding generation
Traceback (most recent call last):
File "/Users/subbu/Extras/test/finddup.py", line 4, in
encodings = cnn.encode_images(image_dir='/Users/subbu/Downloads/zip/tojpg/')
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/imagededup/methods/cnn.py", line 175, in encode_images
return self._get_cnn_features_batch(image_dir)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/imagededup/methods/cnn.py", line 96, in _get_cnn_features_batch
self.data_generator, len(self.data_generator), verbose=self.verbose
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 2298, in predict_generator
verbose=verbose)
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training_generator.py", line 354, in predict_generator
model._make_test_function()
File "/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/site-packages/tensorflow/python/keras/engine/training.py", line 716, in _make_test_function
raise RuntimeError('You must compile your model before using it.')
RuntimeError: You must compile your model before using it.

Could you please share the output of pip freeze from the environment you are using?

it's a bug in tensorflow==1.12, you need update your tf
https://github.com/tensorflow/tensorflow/issues/22952

@subramanya2107 Closing the issue as it's a tf bug as mentioned in the comment above.


tutorial

Docs Build status updates of commit 3030e72:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:x:Error

Details

docs-ref-conceptual/azure-cli-bash-tutorial.md

Line 21, Column 1: [Error: include-not-found] Invalid include link: '/azure-docs/includes/quickstarts-free-trial-note.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 449088d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:x:Error

Details

docs-ref-conceptual/azure-cli-bash-tutorial.md

Line 21, Column 1: [Error: include-not-found] Invalid include link: './includes/quickstarts-free-trial-note.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Hi @carlrab could you please put some more information into the description? Giving some context to purpose and goal of this new doc?

@dbradish-microsoft @chasewilson I will defer an answer to your question to Delora Bradish

Docs Build status updates of commit 30c0d9f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:warning:Warning
View
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md

Line 19, Column 22: [Warning: file-not-found - See documentation] Invalid file link: '../articles/guides/developer/azure-developer-guide.md'.
Line 59, Column 55: [Warning: file-not-found - See documentation] Invalid file link: 'format-output-azure-cli'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 378bc7e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 94854f5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 1c884dc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit cc26477:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:warning:Warning
View
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md

Line 106, Column 61: [Warning: file-not-found - See documentation] Invalid file link: 'i'.
Line 353, Column 1: [Warning: multiple-h1s - See documentation] Multiple H1s(H1 'Delete containers by iterating a loop') are not allowed. You can only have one top-level heading.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 98c6130:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

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

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:bulb:Suggestion
View
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md

Line 211, Column 32: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? If so, add &preserve-view=true to the URL: /cli/azure/storage/account?view=azure-cli-latest#az-storage-account-create

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5b66375:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 06e2eaa:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 5b8b33b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 19d286a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 19d286a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-bash-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 80299de:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/TOC.yml
:white_check_mark:Succeeded
View

docs-ref-conceptual/azure-cli-learn-bash.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add graphConv to SAGPool

There was a typo with the GCN option in the SAGPool. Instead of using GCN it was using GraphConv (as the default conv in benchmark currently is GraphConv). This PR fixes the above and also adds GraphConv as a separate option and makes it default to maintain consistency with the kernel benchmarking.
I wasn't able to find a short form for GraphConv as  GCN is already taken :) so I have used gConv as the abbreviation. If we have to use 3 letter then GRC could be a possiblity. Please change the abbreviation if you have a better naming for GraphConv and do let me know too!

Thank you :)


RuntimeError when commit

🐛 Bug Report
I got the error RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7', when try to first commit.
🔬 How To Reproduce
Steps to reproduce the behavior:

cookiecutter gh:TezRomacH/python-package-template
Follow instructions
After git add, git commit and get the error.

Code sample
git commit -m ":tada: Initial commit"
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
An unexpected error has occurred: CalledProcessError: command: ('/home/dbinary/.cache/pypoetry/virtualenvs/smplnotes-P5WiIkN--py3.8/bin/python', '-mvirtualenv', '/home/dbinary/.cache/pre-commit/repobgj1aqa4/py_env-python3.7', '-p', 'python3.7')
return code: 1
expected return code: 0
stdout:
    RuntimeError: failed to find interpreter for Builtin discover of python_spec='python3.7'

stderr: (none)
Check the log at /home/dbinary/.cache/pre-commit/pre-commit.log

Environment

OS: Arch Linux
Python 3.8.2

📈 Expected behavior
Make a commit without errors.
pre-commit.log

Hi! Thank you for the bug report!
According to the error text, you don't have python3.7, which is the version required version for the template.
I can see in the logs that you're running the program with python3.8. Can you please try to do the same, but on python3.7? Just to ensure the 3.7 works properly.
As a hotfix, I can suggest you put it in `.pre-commit-config.yaml':
default_language_version:
  python: python3.8

And I'll make a fix for it tomorrow.  Now it's 1:00 a.m. Moscow time 😄

Slowpoke time!
I'have just made a fix for this. In installation there is now a parameter minimal_python_version 😍


Calculating pseudo likelihoods on GPU

Hi!
I wanted to calculate pseudo likelihoods for my amino acid sequences using antiberty. I run into the following error when trying to use the pseudo_log_likelihood function:
_nll = torch.nn.functional.cross_entropy(
File "C:\Users\anama\miniconda3\envs\plm\lib\site-packages\torch\nn\functional.py", line 3053, in cross_entropy
return torch._C._nn.cross_entropy_loss(input, target, weight, Reduction.get_enum(reduction), ignore_index, label_smoothing)
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument target in method wrapper_CUDA__nll_loss2d_forward)
Looking at your code briefly, I believe it is because the labels stay on the cpu while the logits are on cuda when calculating nll. I have cuda on my laptop so I checked and confirmed that my antiberty instance is by default located on my GPU. Could you please look into this or let me know if I am missing something? Thanks a lot!

Hi!
I wanted to calculate pseudo likelihoods for my amino acid sequences using antiberty. I run into the following error when trying to use the pseudo_log_likelihood function:
_nll = torch.nn.functional.cross_entropy( File "C:\Users\anama\miniconda3\envs\plm\lib\site-packages\torch\nn\functional.py", line 3053, in cross_entropy return torch._C._nn.cross_entropy_loss(input, target, weight, Reduction.get_enum(reduction), ignore_index, label_smoothing) RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument target in method wrapper_CUDA__nll_loss2d_forward)
Looking at your code briefly, I believe it is because the labels stay on the cpu while the logits are on cuda when calculating nll. I have cuda on my laptop so I checked and confirmed that my antiberty instance is by default located on my GPU. Could you please look into this or let me know if I am missing something? Thanks a lot!

Hi, I met the same issue today. A simple change
labels = self.tokenizer.encode(                 " ".join(list(s)),                 return_tensors="pt",             )[:, 1:-1]**.to('cuda')** in line 266 at pseudo_log_likelihood function worked fine for me. Hope it's helpful!


Fix/daef 478 UI/UX fixes

This PR introduces UI/UX fixes.
TODOs

[ ] Inject missing Japanese translation:

wallet.settings.exportWalletButtonLabel: Export wallet

Reported issues

[x] Ada symbol for redemption icon is too big

[x] Dark theme: input field borders and separators should be #334152 (update also borders of images in themes screen)

[x] Dark theme: scroll should be #E9F4FE not #C8CCCE

[x] Dark theme: wallet settings dropdown arrow active/inactive states are mismatched

[x] Dark theme: dropdown is white

[x] Dark theme: all popups are white (shouldn’t we make them dark?)

[x] About dialog MacOS: if expand window to full screen and then close -> it leaves black screen which can be closed only with closing whole app

[x] Dark theme: disabled button issue - https://zpl.io/V1yJRlw

[x] Green theme: dropdown list selection wrong color, should be #EDEEEF

[x] Switch colors: correct https://zpl.io/V40ZGjQ

[x] Terms or use: scroller should be as normal on right side, but menu should be fixed  (also should crop in the end of window (not with 20px padding)

[x] Dialogs should be stylised for each theme (ex. “Cardano” should have gray icons in “add wallet” dialog), same for selections

[x] Theme screen, dark theme chosen, selection in menu should be inverted (selection color = button color)

[x] Receive screen -> “Copy address” should not underline when hovered

[x] “export this wallet” underlined -> strange -> place next to the delete button

[x] Summary page: scroll whole block (name, amount, tx number), because this info is duped in header

@darko-mijic can you please obtain missing translation?
wallet.settings.exportWalletButtonLabel: Export wallet

Thanks! :)

@a-rukin all UI/UX issues are now fixed - please see the result screenshots in the PR's description.
@darko-mijic @DominikGuzei this PR is now ready for review...

I have already requested translations for all open PRs @nikolaglumac. Waiting :)

"Dark theme: input field borders and separators should be #334152 (update also borders of images in themes screen)" : this screenshot shows that difference between active and inactive state is in borders and font weight Light/Regular. Inactive state also should have 50% transparency for text.

Other things are great!

Thanks for reporting! Will fix it ;)

Reviewing this now …

@nikolaglumac when i run npm run dev on this branch im getting this build error:
[0] ERROR in ./~/css-loader?sourceMap&modules&localIdentName=[name]_[local]&importLoaders=1!./~/sass-loader?sourceMap!./app/themes/simple/SimpleSelect.scss
[0] Module build failed:
[0]     border-color: rgba($bg-color, 0);
[0]                  ^
[0]       Argument `$color` of `rgba($color, $alpha)` must be a color
[0]
[0] Backtrace:
[0] 	node_modules/react-polymorph/lib/themes/simple/mixins/arrow.scss:16, in function `rgba`
[0] 	node_modules/react-polymorph/lib/themes/simple/mixins/arrow.scss:16, in mixin `arrow`
[0] 	node_modules/react-polymorph/lib/themes/simple/SimpleSelect.scss:74
[0]       in /Users/dominik/work/clients/input-output/daedalus/daedalus/node_modules/react-polymorph/lib/themes/simple/mixins/arrow.scss (line 16, column 19)
[0]  @ ./app/themes/simple/SimpleSelect.scss 4:14-205 13:2-17:4 14:20-211

Do i have to cleanup something / install anything new?

@DominikGuzei please run npm install - new version of react-polymorph is needed in order to get pass those errors ;)

@DominikGuzei please wait with merging until @a-rukin approves the changes ;)

For now I don't have questions, we discussed mostly everything. So OK.

@a-rukin thanks!
@DominikGuzei we are ready for the merge ;)

@nikolaglumac translation is still missing isn't it?

Ah yeah! Sorry for the confusion :(
We need to wait for the translation first!

@nikolaglumac im adding the missing translation now …

This is ready to be merged as soon as CI is green 👍


chore(deps): update dependency bazel to v5.3.2

run into some error when building with bazel 5.3.2
ERROR: The project you're trying to build requires Bazel 5.3.1 (specified in /private/tmp/ibazel-20221117-33447-o7pxmb/bazel-watcher-0.20.0/.bazelversion), but it wasn't found in /opt/homebrew/Cellar/bazel/5.3.2/libexec/bin.

relates to https://github.com/Homebrew/homebrew-core/pull/116043

Duplicate of #556


Unable to connect to DocumentDB (CosmoDB)

Describe the bug
When we connect o DocumentDB, unable to open connection to cosmoDB.
The same work when we have .Net Core 2.0
Below are the version we are using:
.Net Core 2.2
Package used:
Visual studion: 2017 (15.9.12)
Exception or Stack Trace
InnerException = {System.Net.Sockets.SocketException (10060): A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond
at System.Net.Http.Connec...
["One or more errors occurred. (A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond)","   at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)\r\n   at System.Threading.Tasks.Task.Wait()\r\n   at webapi.Controllers.ValuesController.Get() in C:\XXXi\ConsoleApp1\webapi\Controllers\ValuesController.cs:line 42 System.Net.Http.HttpRequestException: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond ---> System.Net.Sockets.SocketException: A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond\r\n   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)\r\n   --- End of inner exception stack trace ---\r\n   at System.Net.Http.ConnectHelper.ConnectAsync(String host, Int32 port, CancellationToken cancellationToken)\r\n   at System.Threading.Tasks.ValueTask`1.get_Result()\r\n   at System.Net.Http.HttpConnectionPool.CreateConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken)\r\n   at System.Threading.Tasks.ValueTask`1.get_Result()\r\n   at System.Net.Http.HttpConnectionPool.WaitForCreatedConnectionAsync(ValueTask`1 creationTask)\r\n   at System.Threading.Tasks.ValueTask`1.get_Result()\r\n   at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken)\r\n   at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)\r\n   at Microsoft.Azure.Documents.Client.DocumentClient.HttpRequestMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)\r\n   at System.Net.Http.HttpClient.FinishSendAsyncBuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts)\r\n   at Microsoft.Azure.Documents.Client.GatewayServiceConfigurationReader.GetDatabaseAccountAsync(Uri serviceEndpoint)\r\n   at Microsoft.Azure.Documents.Routing.GlobalEndpointManager.GetDatabaseAccountFromAnyLocationsAsync(Uri defaultEndpoint, IList`1 locations, Func`2 getDatabaseAccountFn)\r\n   at Microsoft.Azure.Documents.Client.GatewayServiceConfigurationReader.InitializeReaderAsync()\r\n   at Microsoft.Azure.Documents.Client.DocumentClient.InitializeGatewayConfigurationReader()\r\n   at Microsoft.Azure.Documents.Client.DocumentClient.GetInitializationTask(IStoreClientFactory storeClientFactory)\r\n   at Microsoft.Azure.Documents.Client.DocumentClient.EnsureValidClientAsync()\r\n   at Microsoft.Azure.Documents.Client.DocumentClient.OpenPrivateInlineAsync(CancellationToken cancellationToken)"]
To Reproduce
Open a console application of .Net Core 2.2 with package:
Code Snippet
try
{
                WebProxy proxyObject = new WebProxy("XXXX");
                WebRequest.DefaultWebProxy = proxyObject;

                HttpClientHandler handler = new HttpClientHandler()
                {
                    Proxy = proxyObject,
                    UseProxy = true,
                };

                var accountKey = "XXX";

                var databaseAccountName = "XXX";
                var databaseEndPoint = new Uri(string.Format("https://{0}.documents.azure.com", databaseAccountName));
                var databaseAccountKey = accountKey;
                var connectionPolicy = new ConnectionPolicy
                {
                    ConnectionMode = ConnectionMode.Gateway, //ConnectionMode.Gateway
                    ConnectionProtocol = Protocol.Https,
                    RetryOptions = new RetryOptions()
                    {
                        MaxRetryAttemptsOnThrottledRequests = 6,
                        MaxRetryWaitTimeInSeconds = 5
                    }
                };

                // var client = new DocumentClient(databaseEndPoint, databaseAccountKey, handler, connectionPolicy);
                var client = new DocumentClient(databaseEndPoint, databaseAccountKey, connectionPolicy);
                client.OpenAsync().Wait();

            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
            }

Expected behavior
Connection to DocumentDB (CosmoDB) should open with out error.
Setup (please complete the following information):

OS: Windows
IDE : Visual studion 2017 (15.9.12)
Version of the Library Microsoft.Azure.DocumentDB.Core=> Version="2.4.0"

Additional context
Its working with .Net Core 2.0

FYI @weshaggard @shahabhijeet We'll probably need help routing this to the right place. This issue affects multiple SDKs including Azure Search and Azure table storage, but only on .NET Core 2.2. Any ideas who this should be assigned to? Is it a more general issue with .NET Core?

@csreddyennam, we will take a look.
Can you please kindly file this issue at https://github.com/Azure/azure-cosmos-table-dotnet/issues which is where issues related to Cosmos DB Table SDK are tracked and triaged? Thanks.

Thanks, I have logged the issue @ https://github.com/Azure/azure-cosmos-table-dotnet/issues/6

I will go ahead and close this issue since this is being tracked in the right repo now. Please reactivate if I missed anything


Array destructuring "easy" solution to the challenge

Describe your problem and - if possible - how to reproduce it
I don't know if it's a "fixable" problem but i was surprised my solution without using array destructuring worked. Just had to write :
"a = 6
b = 8"
Then submit and my solution was accepted !
So i don't know if it's a real issue but i wanted to share it, because in the challenge it's specified that you need to "Use array destructuring to swap a and b."
Add a Link to the page with the problem
https://learn.freecodecamp.org/javascript-algorithms-and-data-structures/es6/use-destructuring-assignment-to-assign-variables-from-arrays
Tell us about your browser and operating system

Browser Name: Mozilla Firefox
Browser Version: 52.9.0 (64 bits)
Operating System: Linux - Debian

If possible, add a screenshot here

I've got this one 🙂

@geeseyj Try giving it another stab

@geeseyj, @thecodingaviator I reopened PR #18909, resolved merge conflicts, and replaced the partial solution with a full solution and it seems to fix this issue.  I am awaiting feedback from other reviewers before we can merge it.


Default Terminal application should be Windows Terminal, PowerShell, CMD in that order of Availability

Is your feature request related to a problem?
No.
Describe the solution
When you first install Files, it should default to using Windows Terminal, PowerShell, or CMD, in that order of Availability. This is instead of simply assuming CMD as the default.
Describe alternatives you've considered
N/A
Scope

Capability
Priority

Default to Windows Terminal if available
Must

Default to PowerShell if available
Must

Default to Command Prompt if neither were available
Must

System Information

OS Version: Windows 11 21H2 22000.100
App version: v<IP_ADDRESS>

Additional comment
It's generally recommended by most to consider the order of preference to be Microsoft Terminal > PowerShell > CMD. Windows itself is also doing the same order of preference.

Just adding a comment I personally use windows terminal but know a lot of people will used cmd over powershell and terminal (Most people probably don't even have terminal installed on win10)

Just adding a comment I personally use windows terminal but know a lot of people will used cmd over powershell and terminal (Most people probably don't even have terminal installed on win10)

I'm not saying force people to use it in the preferred order, I'm saying it should default to it. If the user really does want CMD, even though they have Microsoft Terminal or PowerShell, then they can change back to CMD in the settings.

I think it should default to terminal if it's installed but would need files to check that, Which I'm not sure how easy that is

This seems reasonable being that Terminal will be installed by default on Windows 11,.

This seems reasonable being that Terminal will be installed by default on Windows 11.

Yep, also the context menu in Windows 11 explorer "open with terminal" uses it

I would add that there should be a layer added, if pwsh.exe or Cross-Platform PowerShell 7 is detected, it should be used after Windows PowerShell. Some people might know them as Black PowerShell and Blue PowerShell, with the Blue PowerShell being the older however still more reliable version as far as Windows OS Management goes.
Although, that being said this particular piece may be worth having as it's own ticket and also may need to be re-reviewed as Microsoft has pointed a few times that they are looking to replace Windows PowerShell with the newer PowerShell 7/Core when the time is right.


Add a counter and debug log to monitor histogram bucket to shard distribution

What this PR does / why we need it:

This PR adds visibility to the support histogram aggregation feature introduced in this PR.
Adds a debug log that logs the metric id and which shard id it is being forwarded into
Adds a counter metric that counts the number of histogram metrics forwarded to the shard
Fixes #

Special notes for your reviewer:
Does this PR introduce a user-facing and/or backwards incompatible change?:

Does this PR require updating code package or user-facing documentation?:

@jnyi

screenshot of the metric from a local run:

screenshot of debug log from coordinator/aggregator:


Revert "Merge pull request #18859 from grpc/internal_py_proto_library"

This reverts commit 5b720f19c17d623b1b42f9089108295e25f89d88, reversing
changes made to a64ae3c0d54903aa27fcf27296d7bc6c67d1e192.

Approval for pure revert

Going ahead and merging since this is purely a revert.


perldevel: 5.29.3 -> 5.29.4

Motivation for this change
version bumb

Success on aarch64-linux (full log)
Attempted: perldevelPackages.LWP
Partial log (click to expand)
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35/lib  /nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35/bin
patching script interpreter paths in /nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35
checking for references to /build in /nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35...
shrinking RPATHs of ELF executables and libraries in /nix/store/xqyxz037x0k9vnqzcsp6vviigpy42hzz-perl5.29.4-libwww-perl-6.35-devdoc
gzipping man pages under /nix/store/xqyxz037x0k9vnqzcsp6vviigpy42hzz-perl5.29.4-libwww-perl-6.35-devdoc/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/xqyxz037x0k9vnqzcsp6vviigpy42hzz-perl5.29.4-libwww-perl-6.35-devdoc
checking for references to /build in /nix/store/xqyxz037x0k9vnqzcsp6vviigpy42hzz-perl5.29.4-libwww-perl-6.35-devdoc...
/nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35

Success on aarch64-linux (full log)
Attempted: perldevelPackages.LWP
Partial log (click to expand)
  /nix/store/9nryh4jp9ci2xq7gl76lif4ydddrcfpm-perl5.29.4-File-Listing-6.04.drv
  /nix/store/dq77vvgr69yia14kcl3687vi3d1xsxwq-perl5.29.4-HTTP-Negotiate-6.01.drv
  /nix/store/i73gks24s1ip1kjz8w8x6pwj42wyl6is-perl5.29.4-Test-RequiresInternet-0.05.drv
  /nix/store/ja506k0w0n3gf2ldl51vsya1s6x6k3g2-perl5.29.4-Test-Fatal-0.014.drv
  /nix/store/mclma6xz1dk1y0l21lcljjdmll4l69fs-perl5.29.4-HTTP-Daemon-6.01.drv
  /nix/store/p71h9albax95zdgkzkahyn0y773rzkhv-perl5.29.4-WWW-RobotRules-6.02.drv
  /nix/store/sinpqgplg1z07465qcbjdxwn44mx0wbp-perl5.29.4-HTML-Parser-3.72.drv
  /nix/store/r19dia3b6s8mawn6yw7w2h1j4bmf79ni-perl5.29.4-libwww-perl-6.35.drv
waiting for locks or build slots...
/nix/store/g9pb2bhs8j7x2mqwh9rdgs3aa93rgiy6-perl5.29.4-libwww-perl-6.35

Success on aarch64-linux (full log)
Attempted: perldevel
Partial log (click to expand)
/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4

Success on x86_64-linux (full log)
Attempted: perldevel
Partial log (click to expand)
/nix/store/7g7fdypwxnx9pxdgi78h7sw0zdnpfhaj-perl-5.29.4

501+ rebuils I guess are only perldevelPackages.*, so they should be quite lightweight and should have no other dependencies

Success on x86_64-linux (full log)
Attempted: perldevelPackages.LWP
Partial log (click to expand)
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/m6ap8jjgch1xbymfjsxn19scf0ya0l4c-perl5.29.4-libwww-perl-6.35/lib  /nix/store/m6ap8jjgch1xbymfjsxn19scf0ya0l4c-perl5.29.4-libwww-perl-6.35/bin
patching script interpreter paths in /nix/store/m6ap8jjgch1xbymfjsxn19scf0ya0l4c-perl5.29.4-libwww-perl-6.35
checking for references to /build in /nix/store/m6ap8jjgch1xbymfjsxn19scf0ya0l4c-perl5.29.4-libwww-perl-6.35...
shrinking RPATHs of ELF executables and libraries in /nix/store/mhyhi0iq106ykp197yanq6bbrvpz1m85-perl5.29.4-libwww-perl-6.35-devdoc
gzipping man pages under /nix/store/mhyhi0iq106ykp197yanq6bbrvpz1m85-perl5.29.4-libwww-perl-6.35-devdoc/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/mhyhi0iq106ykp197yanq6bbrvpz1m85-perl5.29.4-libwww-perl-6.35-devdoc
checking for references to /build in /nix/store/mhyhi0iq106ykp197yanq6bbrvpz1m85-perl5.29.4-libwww-perl-6.35-devdoc...
/nix/store/m6ap8jjgch1xbymfjsxn19scf0ya0l4c-perl5.29.4-libwww-perl-6.35

I think this is ok for master.

check darwin:
@GrahamcOfBorg build perldevel

Success on aarch64-linux (full log)
Attempted: perldevel
Partial log (click to expand)
/nix/store/n22j0jdr8ka9f7x86x0s379mim8afgi5-perl-5.29.4

Success on x86_64-darwin (full log)
Attempted: perldevel
Partial log (click to expand)
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/kmxhpslk6cfags1ql6x5w701j5slw05h-perl-5.29.4/lib  /nix/store/kmxhpslk6cfags1ql6x5w701j5slw05h-perl-5.29.4/bin
patching script interpreter paths in /nix/store/kmxhpslk6cfags1ql6x5w701j5slw05h-perl-5.29.4
gzipping man pages under /nix/store/r69v7wnkj1kiysjynxsqfb3n251x6b4k-perl-5.29.4-man/share/man/
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/r69v7wnkj1kiysjynxsqfb3n251x6b4k-perl-5.29.4-man
gzipping man pages under /nix/store/300xz2w6a7p4slpn8yg935y55c104hp5-perl-5.29.4-devdoc/share/man/
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/300xz2w6a7p4slpn8yg935y55c104hp5-perl-5.29.4-devdoc
/nix/store/kmxhpslk6cfags1ql6x5w701j5slw05h-perl-5.29.4

Success on x86_64-linux (full log)
Attempted: perldevel
Partial log (click to expand)
/nix/store/7g7fdypwxnx9pxdgi78h7sw0zdnpfhaj-perl-5.29.4


Update logstalgia to 1.0.7

New versions should be sent to Homebrew. Homebrew is merged into Linuxbrew weekly. Please see CONTRIBUTING.md


cmd/go: 'go install' interactively prompts for SSH password when fetching from a private Git repo

The go command intentionally tries to suppress interactive prompts using GIT_TERMINAL_PROMPT=0.
Unfortunately, that isn't actually sufficient to suppress Git terminal prompts: if Git ends up connecting over SSH, then the ssh command itself may hijack the console to ask for a password.
According to various scraps of information on the Internet, we may also need to set  -o·BatchMode=yes in our GIT_SSH_COMMAND, and perhaps also GCM_INTERACTIVE=never.
For me, this manifested as a hang in the mod_get_private_vcs test at CL 297950.
CC @jayconrod @matloob @kevinburke

We discussed this issue with @rsc on 2021-03-23 and decided to give it a try for Go 1.17 (with the understanding that it might be reverted during the beta or in a point release if it turns out to be too invasive a fix).
(I hadn't dusted off the CL until just now because I was focused on landing #36460 before the freeze.)

We discussed this issue with @rsc on 2021-03-23 and decided to give it a try for Go 1.17 (with the understanding that it might be reverted during the beta or in a point release if it turns out to be too invasive a fix).

Is there something that happened that made you bring this up just now? I haven't thought about this since you merged the referenced CL in March

I was going through my outstanding changes and noticed that CL 300157 was still marked wait-issue, and remembered that we had actually discussed the issue enough to move forward.

Curious at to why this was implemented - what is the requirement to disable password prompts unconditionally? It breaks yubikey ssh for me (see above referenced issue)

Honestly, it would be best to leave it to the user to decide how they want to be prompted.
I know some folks love to use SSH-Agent, and others simply don't put passwords on their SSH Keys; however...

SSH Agent introduces security issues as it keeps keys in memory unencrypted for the duration of its operation until the user explicitly removes it from memory, most don't
I've had at least one employer require SSH Keys to have password protection (ironically they later required SSH Agent for their SSH Bastion proxy too).

Personally I neither like using SSH Agent nor do I like having my SSH Keys unprotected. I keep passwords on them for a reason and I want to know whenever the key is used by a process.  If the goal is to reduce the number of prompts (a good goal) then perhaps there are other more secure ways to do so.


Update TPCH Infrastructure

This does a few things:

Generate performance reports for every query if you add the --performance-report flag
Generate altair comparison charts
Change the infrastructure size up and down based on scale
Use Arrow filesystem (although I'm not sure how big a deal this is yet)
Improve charts to use consistent coloring, and drop ibis for pandas

Merging tomorrow morning my time if there are no comments

@mrockling arrow file system doesn't work with dask-expr main. I am guessing that you are still using my hacky branch that avoid the error? Should we invest effort into making this work or roll back for now?

@phofl why not just merge your fixes in dask-expr?

My fix removes the filesystem from our hashing logic, resulting in ugly patches in a few places. This was never meant to get merged, rather to try it out.
We have to implement a tokenize logic for Arrow Filesystems. This might be trivial, I don't know that

@phofl: What branch are you talking about here? There's https://github.com/dask/distributed/issues/8301, for which I'd need the TPC-H tests to work to dig deeper into it.

https://github.com/phofl/dask-expr/tree/testing

@mrocklin arrow file system doesn't work with dask-expr main. I am guessing that you are still using my hacky branch that avoid the error? Should we invest effort into making this work or roll back for now?

I think I'm no longer using arrow filesystem anyway (although I would like to)

We have to implement a tokenize logic for Arrow Filesystems

We don't need to do anything smart here I think.    We might not get good reuse, but that's not a big deal.  Or maybe I don't understand the situation.  Is there somewhere I can learn more?

@hendrikmakait you can apply this diff (I'll fix this shortly)
diff --git a/tests/tpch/conftest.py b/tests/tpch/conftest.py
index d174a78..07a9237 100644
--- a/tests/tpch/conftest.py
+++ b/tests/tpch/conftest.py
@@ -291,6 +291,7 @@ def fs(local):
     if local:
         return None
     else:
+        return None
         import boto3
         from pyarrow.fs import S3FileSystem


Is it possible to use the overview method with chunked?

I have this doubt because the chunked method is the only one that does not vote me the outlook connection, so I wanted to know if you could do something like that, or to search by mail and that the UID is greater than the uid that you register in the database?

Hi @alexv96 ,
many thanks for your question.
Yes you can do this by calling Query::fetchBody(false) to prevent the query from fetching the message body. All you are left with are the message headers and flags - just like the overview method :)
Best regards,


Elm: automate packaging with elm2nix

rewrite documentation
automate the whole process into update.sh
remove all legacy snippets

cc @jerith666 who did a bunch of cool work recently.

I am super curious to try this out!  However, multiple dependencies of pkgs.elmPackages.elm fail to build at this commit -- have you looked into that yet?

Yeah it's due to LTS-13 bump. I'll see what I can do.

@GrahamcOfBorg build elmPackages.elm elmPackages.elm-format

@jerith666 compiles now :)


Experiment: immer patch merge

Summary
Because we collect the commands of the individual keypresses in StrategyState.accumulatedCommands, if you do a long press, this is gonna be an evergrowing list of commands to apply, and on every new keydown we apply all the commands here.
Overall this means this is a quadratic algorithm, and it becomes unusable in a couple of seconds.
Solution idea:
Do not apply all the commands in accumulatedCommands and their patches in every dispatch, but let's just store a merged patch and modify it on every dispatch so it contains the effect of the current command too.
Main problem:
I could not find a readymade solution to merge immutability-helper patches, and it doesn't seem to be a trivial problem. If we only support $set commands, then it is not that complex, but the interaction between all the different commands is quite complex.
Note: the naive solution to just merge the patch objects is not corrent, e.g. when you have a $set command to a part of an object and another set for a subkey of the same part.
Possible solutions:
Use immer
Immer has a really nice feature that it can produce patches for a given change. If you apply multiple changes it will still produce a compact patch.
So if we would modify the commands to return immer patches, we could just apply all the patches from all the commands and get back a compact patch list containing all the effects. We could just store this compact patch list in StrategyState, and next time apply that, apply all the commands too, and store the overall merged patch in StrategyState, etc. So basically immer knows what we need.
Downsides:

we need to rewrite all commands to generated immer patches (or recipe functions, and we can produce the patches themselves using immer)
immer has a performance overhead, but for our editorstate it doesn't seem to be more than an extra 0.2-0.3 ms

Notes about the draft

only the keyboard absolute move is implemented and even that is buggy

I'm closing this, because the other option won: https://github.com/concrete-utopia/utopia/pull/2165


dotnet tool restore often failing on actions

Since moving csharpier up to using 0.26.0 of csharpier I've been seeing versions of this often on the actions that are run.

Could it have to do with having the 3 target frameworks now?
Is there maybe a bug with net8 itself, which is failing to restore the tool sometimes?
Rerunning the action gets past it, but it is frustrating.

Should have done a bit more googling, looks like this is an issue with the net8 rc2 - https://github.com/dotnet/sdk/issues/35989


[MSKINS-242] Replace deprecated prettify JS library

@kwin Here is a very early draft. Problematic. Bootstrap's padding and highlight.js padding add up and make it abysmal:

It will take time to make it look like:

This draft includes these languages:
"bash.js"
"bash.min.js"
"diff.js"
"diff.min.js"
"java.js"
"java.min.js"
"json.js"
"json.min.js"
"powershell.js"
"powershell.min.js"
"shell.js"
"shell.min.js"
"xml.js"
"xml.min.js"

@kwin, do you want to work on this? Before 2.0.p


unmatched shapes in tf.concat()

In my distribution, the example works well:
python -m src.flownet2.test --input_a data/samples/0img0.ppm --input_b data/samples/0img1.ppm --out ./

But when I tried to use my own data, there was a unmatching problem in tf.concat() between conv5 and deconv5. I pasted the error logs below:
> $ python -m src.flownet2.test --input_a data/examples/00000.jpg --input_b data/examples/00001.jpg --out ./

Traceback (most recent call last):
  File "/usr/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/home/workspace/playground/flownet2-tf/src/flownet2/test.py", line 78, in <module>
    main()
  File "/home/workspace/playground/flownet2-tf/src/flownet2/test.py", line 39, in main
    out_path=FLAGS.out,
  File "src/net.py", line 63, in test
    predictions = self.model(inputs, training_schedule)
  File "src/flownet2/flownet2.py", line 22, in model
    net_css_predictions = self.net_css.model(inputs, training_schedule, trainable=False)
  File "src/flownet_css/flownet_css.py", line 18, in model
    net_cs_predictions = self.net_cs.model(inputs, training_schedule, trainable=False)
  File "src/flownet_cs/flownet_cs.py", line 18, in model
    net_c_predictions = self.net_c.model(inputs, training_schedule, trainable=False)
  File "src/flownet_c/flownet_c.py", line 70, in model
    concat5 = tf.concat([conv5_1, deconv5, upsample_flow6to5], axis=3)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/array_ops.py", line 1048, in concat
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_array_ops.py", line 495, in _concat_v2
    name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2508, in create_op
    set_shapes_for_outputs(ret)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1873, in set_shapes_for_outputs
    shapes = shape_func(op)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1823, in call_with_requiring
    return call_cpp_shape_fn(op, require_shape_fn=True)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 610, in call_cpp_shape_fn
    debug_python_shape_fn, require_shape_fn)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/common_shapes.py", line 676, in _call_cpp_shape_fn_impl
    raise ValueError(err.message)
ValueError: Dimension 1 in both shapes must be equal, but are 15 and 16 for 'FlowNet2/FlowNetCSS/FlowNetCS/FlowNetC/concat_1' (op: 'ConcatV2') with input shapes: [1,15,27,512], [1,16,28,512], [1,16,28,2], [] and with computed input tensors: input[3] = <3>.

The shape of input images are the same, and I pasted the output of command file below:
> $ file data/examples/*
data/examples/00000.jpg: JPEG image data, JFIF standard 1.01, resolution (DPCM), density 94x94, segment length 16, baseline, precision 8, 854x480, frames 3
data/examples/00001.jpg: JPEG image data, JFIF standard 1.01, resolution (DPCM), density 94x94, segment length 16, baseline, precision 8, 854x480, frames 3

What's more, I tried to use the same image as --input_a and --input_b, the error still occurs.

I have the same problem.

So how do you fix the problem? i also meet the same problem. Hope for your reply~
@sampepose  @Beanocean  @Co1dAt0m

@Zealoe
When conducting convolution operation with stride 2, the size of feature map will be divided by 2. In order to match the sizes of feature map in corresponding convolution and deconvolution operations, the shape of input image must be an integral multiple of 64. So you need to scale your images to meet this requirement before inputting them into the model. Please refer the code lines in original FlowNet2

@Beanocean thanks very much！


Recurring Error

Not sure why this is happening but I keep getting this error. Path is correct.
Invalid CFML construct found on line 358 at column 79.
ColdFusion was looking at the following text:
[
The CFML compiler was processing:
A script statement beginning with private on line 358, column 9.
The error occurred in /Applications/ColdFusion9/wwwroot/renters/miscellaneous/stripe.cfc: line 358

It looks like you are running ColdFusion 9, and the CFML compiler doesn't like the implicit array listed as the default value for the params agument in the the apiCall() function (which starts on line 358). Are you perhaps running on CF9 without any of the hotfixes or updates? I went back and looked at the first hotfix for CF9 and found bug id 80231 (see here: https://helpx.adobe.com/coldfusion/kb/cumulative-hot-fix-1-coldfusion.html), which states that implicit arrays didn't work as default values for the second argument of a function onward. If this is the version you are on, you would either need to update CF9 with the hotfix, or you could trying changing line 358 from array params = [ ] to array params = arrayNew(1), and see if that works.

Switching to  array params = arrayNew(1) gets rid of the initial error.
But new errors regarding the FOR loops occur.
Invalid CFML construct found on line 378 at column 17.ColdFusion was
looking at the following text:
for
The CFML compiler was processing:

A script statement beginning with for on line 378, column 17.
A script statement beginning with private on line 371, column 9.

Sorry bug you with all of this. If a solution does not jump out at you do
not worry about.
Erik
On Mon, Mar 2, 2015 at 8:10 PM, jcberquist<EMAIL_ADDRESS>wrote:

It looks like you are running ColdFusion 9, and the CFML compiler doesn't
like the implicit array listed as the default value for the params
agument in the the apiCall() function (which starts on line 358). Are you
perhaps running on CF9 without any of the hotfixes or updates? I went back
and looked at the first hotfix for CF9 and found bug id 80231 (see here:
https://helpx.adobe.com/coldfusion/kb/cumulative-hot-fix-1-coldfusion.html),
which states that implicit arrays didn't work as default values for the
second argument of a function onward. If this is the version you are on,
you would either need to update CF9 with the hotfix, or you could trying
changing line 358 from array params = [ ] to array params = arrayNew(1),
and see if that works.
—
Reply to this email directly or view it on GitHub
https://github.com/jcberquist/stripecfc/issues/1#issuecomment-76866866.

Yeah, you're definitely running CF9 with no updates. for...in array looping was fixed in CF9 update 1: https://helpx.adobe.com/coldfusion/kb/issues-fixed-coldfusion-9-0.html. (See Issue ID 80679.) I think at this point if you still want to use stripe.cfc you will need to update to at least CF 9.0.1 (or you can try working through the errors one by one, but that would be a pain). I do note in the Readme here that CF needs to be at least version 9.0.1

OK. I will get the dev enviro updated. Luckily the production enviro is
updated. Thanks for the help and the stripe.cfc code.. I look forward to
getting this working.
Best,
Erik
On Tue, Mar 3, 2015 at 1:45 PM, jcberquist<EMAIL_ADDRESS>wrote:

Yeah, you're definitely running CF9 with no updates. for...in array
looping was fixed in CF9 update 1:
https://helpx.adobe.com/coldfusion/kb/issues-fixed-coldfusion-9-0.html.
(See Issue ID 80679.) I think at this point if you still want to use
stripe.cfc you will need to update to at least CF 9.0.1 (or you can try
working through the errors one by one, but that would be a pain). I do note
in the Readme here that CF needs to be at least version 9.0.1
—
Reply to this email directly or view it on GitHub
https://github.com/jcberquist/stripecfc/issues/1#issuecomment-77007678.