Two dates reforecasts dask warnings and takes ages

It takes ages to load two dates from reforecasts. Shouldnt that work better and faster? @floriankrb @b8raoult
>>> ds = cml.load_dataset("s2s-ai-challenge-training-input", origin="ecmwf", date=["20200102","20200109"], parameter='t2m')
By downloading data from this dataset, you agree to the terms and conditions defined at https://apps.ecmwf.int/datasets/data/s2s/licence/. If you do not agree with such terms, do not download the data.
ds.to_xarray()
>>> ds.to_xarray()
WARNING: ecmwflibs universal: found eccodes at /work/mh0727/m300524/conda-envs/s2s-ai/lib/libeccodes.so
Warning: ecCodes 2.21.0 or higher is recommended. You are running version 2.12.3
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 20
  **blockwise_kwargs,
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 11
  **blockwise_kwargs,
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 46
  **blockwise_kwargs,
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 20
  **blockwise_kwargs,
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 11
  **blockwise_kwargs,
/work/mh0727/m300524/conda-envs/s2s-ai/lib/python3.7/site-packages/dask/array/core.py:4299: PerformanceWarning: Increasing number of chunks by factor of 46
  **blockwise_kwargs,
... takes more than 10 minutes

I think the challenge here is that we have 20 forecast_reference_times per file and they cannot be stacked easily. As it is now, this is a huge bottleneck.

works nicer now. thanks for fixing @floriankrb @b8raoult


Cannot specify ContainerEnvironmentVariable via the command line using -p:ContainerEnvironmentVariable=?

For automating builds and deploys it would be useful if we can specify container environment variables via the command line. Such as:
dotnet publish --os linux --arch x64 -p:ContainerEnvironmentVariable=EnvironmentVariable=Value /t:PublishContainer -c Release
How can this be done via the command line since that property is a complex property?
<ContainerEnvironmentVariable Include="ASPNETCORE_ENVIRONMENT" Value="Development" />
But when building for other environments i would like to set it to a different value such as the below:
<ContainerEnvironmentVariable Include="ASPNETCORE_ENVIRONMENT" Value="Staging" />
Are there any workarounds or other means to achieve this?

Hi @lymberl - currently this is not possible. MSBuild doesn't allow passing Items on the command line. @rainersigwald are there patterns that we could rely on here? Or I suppose in the worst case we could do something like

have a property that users could set
have a target that explicitly 'splits' that property into Items by parsing a specific format.

have a property that users could set
have a target that explicitly 'splits' that property into Items by parsing a specific format.

This is the best I can think of too; it's nice and explicit.

How confusing would it be if we chose the Property name of ContainerEnvironmentVariable as well? Does MSBuild handle the ambiguity correctly based on how the name is used, either in @() for items or $() for Properties?
Otherwise I suppose that would impact the choice of name for the property to use here.

Yes, we have totally separate lookup tables so you can have a property and an item with the same name. It tends to be pretty confusing to debug (even more than getting it wrong when there's only one but it's the other kind) but in this case I think that's outweighed by the consistency of naming.

We need something like this for CI/CD, as is we need to stick to docker based container builds to inject environment variables.

Is it resolved?, we are facing the same issue on providing ContainerEnvironmentVariable in the command line to set ASPNETCORE_ENVIRONMENT Environment variable.


Feature/admin level time dimension

Added support for time-dimension to admin-level-data and allowed date range with startDate and endDate.

@wadhwamatic there was an issue with wms layer with unmatching dates with admin-level-data was causing the app to freeze due to infinite loop. I had a discussion with @JorgeMartinezG and he told me that we are aware of the issue and plan to have a discussion on how to better handle it. In the mean time I just implemented his fix.

When I load a WMS layer (10-day rainfall) with the latest data (11-Nov-2021) and then activate the 'Admin Level With Date Range' layer, the message 'No data was found for the layer 'Admin Level With Date Range' on 2021-11-11' appears. The WMS layer changes to 1-April-2021 but the related admin level data does not load. The timeline and calendar picker are also empty. See screenshot #1 below.
When I do the same as above but choose the 'Admin Level With Dates' layer instead, the layer does load but the user isn't made aware of the change in date to 1-April-2021. In addition, the timeline and calendar picker are empty. See screenshot #2

Closing this PR as we'll revisit dates more holistically in the near future


[CWS] fix deadlock when calling RemoveDump from a function where ad is already locked

What does this PR do?
Here is what is happening:
adm.resolveTagsPerAd locks ad
and calls adm.RemoveDump which locks adm

adm.SendStats locks adm
and then calls ad.SendStats which locks ad

this different order of locking can mean that we end up with a deadlock on ad, and thus we block the whole event goroutine which is catastrophic.
After finding that https://github.com/DataDog/datadog-agent/pull/24459 is bad. This PR is much better because it keeps the natural hierarchy of locks:

adm.Lock -> ad.Lock is fine
ad.Lock -> adm.Lock is really a bad idea and should basically never happen

here we simply take advantage of the finalization step what moves the state to Stopped, we also add a cleanup sub-function that will simply remove those stopped ADs from the activeDumps (sadly this slight delay requires a small change to the tests). This allows the finalization step to not need to edit the manager thus allowing to never deadlock on trying to lock an adm with an ad already locked.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


[FEAT] support for aapt2-compile '--legacy' to skip "soft" errors

https://developer.android.com/studio/command-line/aapt2#compile_options
it might help as a workaround for some quirks of https://developer.android.com/studio/command-line/aapt2#aapt2_changes which were fine in aapt (v1).

Ps. I'm using this pre-built aapt2
from https://maven.google.com/web/index.html#com.android.tools.build:aapt2
(the exe is in the jar, no x64 but that's fine..)

and:

 -a,--aapt           Loads aapt from specified location.

nevermind I saw it in already in androlibresource.java


Update stens to 0.7.1. Fix clippy lints.

I need to have stens on 0.7.1 so I can pull this crate into code I'm using that also uses bp-core and rgb-std.
I updated stens, but it invalidated the test assertions.

I'm not sure what the best decision is for that.

@dr-orlovsky For more context on why I need to use these things, I basically want to create a wrapper library around RGB that uses BDK for signing and can be cross-compiled to WASM and an AWS Lambda. For the portions that can't be run in WASM yet (like libsecp256k1) we'll make shim in calls to the Lambda service (NYI).
https://github.com/diba-io/bitmask-core/pull/56
Then, this can be easily pulled in to make a desktop wallet also, and eventually we can deprecate the AWS part once libsecp256k1-zkp can be compiled to WASM someday.

Haha, I guess you were on it. I merged your latest, but you'll still need to update the test assertions... But I don't want to do it blindly since you have more context as to why those changed.

Perfect, would love to! And make all the changes you like, Rust is a perfect guide. I've been able to keep up with everything so far, your code is very sensible, and very good, properly idiomatic Rust. 🤠


Edburns msft dd 1617770 functions quarkus tutorial public

@edburns, if this PR is ready for review and merge, it needs to be moved out of darft mode. To do so, selected the "Ready for review" button.

Docs Build status updates of commit 1a2b527:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/azure-functions/functions-quarkus-tutorial.md
:warning:Warning

Details

articles/azure-functions/index.yml
:white_check_mark:Succeeded

articles/azure-functions/media/functions-quarkus-tutorial/azure-function-app.png
:white_check_mark:Succeeded

articles/azure-functions/media/functions-quarkus-tutorial/azure-function-app-detail.png
:white_check_mark:Succeeded

articles/azure-functions/media/functions-quarkus-tutorial/azure-function-app-ready.png
:white_check_mark:Succeeded

articles/azure-functions/media/functions-quarkus-tutorial/quarkus-crud-running-locally.png
:white_check_mark:Succeeded

articles/azure-functions/TOC.yml
:white_check_mark:Succeeded

articles/azure-functions/functions-quarkus-tutorial.md

Line 15, Column 94: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-us/services/functions/' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 365, Column 1: [Warning: file-not-found - See documentation] Invalid file link: 'overview.md'.
Line 370, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'configure-language-java.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 or suggestions 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

@edburns
Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
We'll close this PR. Thank you.


Dashboard Functionality Removed - API's Not Updated

Description of bug
When DNN Platform was migrated to using the Persona Bar, starting with version 9.0.0, the older Dashboard extension point was not brought forward.
However, all of the dashboard API's, methods, and implementation remain part of the DNN API and could be used by developers with no real results
Steps to reproduce
Create a new Dashboard Extension and notice that it is not displayed anywhere.
Current behavior
Developers can build a Dashboard Extension without any warnings, and the functionality will not work.
Expected behavior
Developers should get a build warning and notice that the API's will be removed.
Discussion Point
Traditionally, these API's would be something that should be flagged for removal in version 11.0.0 given the current version.  However, since this functionality is not working now, I would propose that we flag for removal in 10.0.0 to coincide with all other breaking changes already planned.
Affected version

[x] 09.07.01 latest supported release

I agree with this special case and removing it in 10.

Just updating the history here, this was actually removed in 8.0.7


How to split "Accent" information of the speaker?

Thanks for the codebase. Good work!
In the paper, Speech is split into -- timbre (using speaker embedding), pitch, rhythm, content. If I am not wrong, the accent information of the speaker is not captured by the speaker embedding. (I know this because when I experimented with AutoVC codebase, the speaker embedding did not capture the accent info. It accent info of the source speech was always seen in the voice conversion output.)
Any ideas on how to split the accent information from speech?
Thanks,
Pravin

Good question. AutoVC can disentangle accent to some extent but not sufficient. Disentangling accent is another interesting research problem. We do not have a solution for that.

Good question. AutoVC can disentangle accent to some extent but not sufficient. Disentangling accent is another interesting research problem. We do not have a solution for that.

Thanks @auspicious3000
After Speech decomposition into timbre, pitch, rhythm, content -- Would you expect the accent information to be a part of the content embedding?

Thanks @auspicious3000
After Speech decomposition into timbre, pitch, rhythm, content -- Would you expect the accent information to be a part of the content embedding?

Accent may be a part of each component.


Add Coveralls support and badge

Changes in this pull request
Adding Coverall support and badge in README. Waiting on tests. Fixes #37
Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I have reviewed the contributing guide

Changes Unknown when pulling d61a01fbad00f6c22acc3babb65049671dbff8e7 on coverall into ** on master**.

🎉


:sparkles: Version 2.1.0

Codecov Report

Merging #65 (c25ea71) into main (21b0b7d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #65   +/-   ##
=======================================
  Coverage   99.81%   99.81%
=======================================
  Files          76       76
  Lines        1060     1065    +5
  Branches       52       49    -3
=======================================
+ Hits         1058     1063    +5
  Partials        2        2

Impacted Files
Coverage Δ

src/surfaces/home-tab.ts
100.00% <ø> (ø)

src/surfaces/message.ts
100.00% <ø> (ø)

src/surfaces/modal.ts
100.00% <ø> (ø)

src/surfaces/workflow-step.ts
100.00% <ø> (ø)

src/md/index.ts
100.00% <100.00%> (ø)

src/methods/other-methods.ts
96.55% <100.00%> (+0.39%)
:arrow_up:

Continue to review full report at Codecov.

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


Windows detects Trojan Win32/Zpevdo.A

All Windows Releases get detected as Trojan Win32/Zpevdo.A.

It's a problem with the .exe
Sorry for the inconvenience...
This exe is generated by another tool in order to start a bat script file which runs PMM silently. The other way is to use the .bat file in the root folder which do the same thing (but with an annoying black cmd terminal...)

Thanks for the fast response, I expected something like this.
For me a cmd terminal is nothing unusal.^^
From my perspective a disclamer that it probably gets detected as a Trojan would be enouth, or that people who dont want take the risk, can use the bat script instead.
I just searched for it and didn't found an answer.


PowerToy Run UnitConverter float number precision is not enough

Microsoft PowerToys version
"0.57.2"
Running as admin

[ ] Yes

Area(s) with issue?
PowerToys Run
Steps to reproduce
Type %% 1nm in m --> 0m
Any small enough unit will be took as 0 in larger unit.

✔ Expected Behavior
Type %% 1nm in m --> 1e-9m or 0.00000001m
❌ Actual Behavior
Type %% 1nm in m --> 0m
Other Software
No response

Besides, it will be better if result could be displayed as scientific notation formation. Thanks in advance.

Calculator is also affected by this.
If I try =1/100000000000 I got 0,0000000000.

Also we have 1 km to nm =<PHONE_NUMBER>99,9998 and 1 mile to nm =<PHONE_NUMBER>999,9998
The reason is we always use Math.Round(convertedValue, _roundingFractionalDigits) with _roundingFractionalDigits = 4 in UnitHandler class.
We need to make it more intelligent to handle four significant digits, not four digits after decimal separator.


Fix specs for Array pack with format J and j specs

on big-endian platforms.

@eregon Does this look ok to merge?

It sounds fine although the diff is hard to read.
I suppose you wrapped the _be in big_endian and _le in little_endian?
Thanks for the fix! Feel to free to merge it :smiley:

Yes the _ and ! options are `native endian' so change with platform endianness :smiley:


Update README.md

Configuration in order to use Sendria with Laravel (and to avoid the Swift_TransportException 🙄 ).

Hi @tuxfamily ,
Awesome! Thank you for your PR, going to merge :)

Hi @msztolcman ,
Thanks for the merge ( and Sendria!).
I lost an hour because the 'trick' MAIL_ENCRYPTION=tcp is undocumented.
So if it can help someone I would be glad 😄


Leaderboard with wallet integration feature

Something went wrong with PR preview build please check

Please run npm run format:mdx

Please run npm run format:mdx

It doesn't fix anything. It seems like the problem with permissions

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Something went wrong with PR preview build please check

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/

Preview is available here:http://soroban-docs-pr598.previews.kube001.services.stellar-ops.com/


Improve docs to clarify that blocks are loaded and used in a flow rather than being created in a flow

Clarifies confusion as here: https://discourse.prefect.io/t/attributeerror-nonetype-object-has-no-attribute-call-on-sync-portal/1638
Checklist

[x] Summarized PR's changes in CHANGELOG.md

I think the README has the appropriate docs now about blocks + loading:
https://github.com/PrefectHQ/prefect-dbt#installation
Closing this for now. Please re-open this if you believe it could be better!


Not populate 0 value in input type number

I'm aware of superform populate every form field with its default based on the zod schema, but I'm dealing with the situation when I'm not using label to point out which value must be typed each case, so I cannot populate the 0 value in input type number and hide the placeholder. Is there any way to accomplish this?
Here my solution, it works but I end up with many number-like strings on the server side that must be converted to store in the database.
const createNumberLikeSchema = (maximum: number) =>
	z // number-like pipeline.
		.string()
		.trim()
		.superRefine((value, errorQueue) => {
			if (value === '' || isNaN(Number(value))) {
				errorQueue.addIssue({
					code: z.ZodIssueCode.custom,
					message: PART_INVALID_NUMBER_MESSAGE
				});
				return z.NEVER;
			} else {
				const transformedValue = Number(value);
				if (transformedValue <= 0 || transformedValue >= maximum) {
					errorQueue.addIssue({
						code: z.ZodIssueCode.custom,
						message: `${PART_OUT_OF_RANGE_MESSAGE} ${maximum} expected`
					});
				}
			}
		});

const partVendorSchema = z.object({
	// Particular constraints should be added instead of .min(1) each case.
	uuid: z.string().trim().uuid(),
	name: z.string().trim().min(1, PART_STRING_TOO_SMALL_MESSAGE),
	cost: createNumberLikeSchema(PART_COST_LAYOUT_CONSTRAINT)
});

Try an intProxy or numberProxy and experiment with their options: https://superforms.rocks/api#proxy-objects

It works man :) but I got another svelte question fo u instead, sorry if not related with superform :(
I'm using svelte form to manage add/remove action over child components based on REPLs,
	const handleRemovePartVendor = (index: number) => {
		let updatedWizardStore = { ...$superPartStore };
		updatedWizardStore.vendors = [
			...updatedWizardStore.vendors.slice(0, index),
			...updatedWizardStore.vendors.slice(index + 1, updatedWizardStore.vendors.length)
		];
		// Assign update to trigger re-render.
		superPartStore.set(updatedWizardStore);
	};

{#each $superPartStore.vendors as vendor, index (vendor.uuid)}
<Vendor {index} {() => handleRemovePartVendor(index)} />
{/each}

Vendor.svelte
...
<Input
  type="text"
  placeholder="Vendor's name"
  bind:value={$superPartStore.vendors[index].name}
/>

But despite the fact of it works on deletion action, it raises an error when I tried to delete the boundaries items, it's like it's trying to render items that was deleted already. My guess here is that is not a good idea to manage item detetion via svelte store index and it's better to find the items to be deleted by their uuid instead.

It worked perfectly man, thanks a ton


Enables the codeQL.codespacesTemplate flag

This will let us use some specific extension features.
Sibling PR: https://github.com/github/vscode-codeql/pull/2040

Note: We don't want to merge this until we've implemented the full "create skeleton pack" functionality.


Cleanup KVO when view is deallocating

May cause crash when OAStackView was deallocated but still observing subview's hidden property.

Is there anything I can do to help get this PR merged?

@samsymons Hey, thanks for the interest.
Do you think you can get the travis build green again :) that would be super helpful!

@oarrabi Absolutely, I'll take a look into that — thanks! 😄

I have not yet had time to fix up the CI issues with this PR but hope to look into that this week; sorry about that!
I also found an edge case with this fix, where views added multiple times to the stack view will not get properly deallocated after this fix in dealloc:
[self removeObserverForViews:self.subviews];

In this case, I had a label being added to the stack view multiple times, but that line above will only remove the observer for subviews. I fixed this crash in our project by updating the fix to remove the observer for all arranged subviews, not just self.subviews.
It seems that addArrangedSubview: is assuming that it is always getting a new item, so you can have a situation where the same view is added multiple times. Does it makes sense to have addArrangedSubview: check whether the newly added subview already exists in the arranged subviews array before adding it?


Cannot delete roles from a service template on Sunstone

Description
After create a service template with multiple roles, it's impossible to delete one of the existing roles.
To Reproduce

Create a simple service template with two roles, e.g:

{
  "name": "testing-update",
  "deployment": "straight",
  "description": "",
  "roles": [
    {
      "name": "master",
      "cardinality": 1,
      "vm_template": 0,
      "elasticity_policies": [],
      "scheduled_policies": []
    },
    {
      "name": "minion",
      "cardinality": 1,
      "vm_template": 0,
      "elasticity_policies": [],
      "scheduled_policies": []
    }
  ],
  "ready_status_gate": false
}

Click to update button:

Click remove one of roles:

Then click to update button, and check that the template hasn't changed

Expected behavior
Be able to delete roles from Sunstone
Details

Affected Component: Sunstone
Hypervisor: [e.g. KVM]
Version: development

Progress Status

[x] Branch created
[x] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

@tinova PRs to be approved:

Code
Docs

Needs testing

Tests


I need help

Hello,
I tried to reach you by sending mail. please reply to me as soon as possible.

@alahammad need help?


Windows: Try to make the dev-setup script more robust

Stop assuming where Visual Studio is installed (instead, try to detect any of the available installations).
Do a better job of waiting for the Visual Studio installer to terminate.
Try to run everything in parallel, but in a way such that each individual task is in a readable block.
Don't reinstall libjpeg-turbo (to avoid a dialog box complaining about it already being installed).
Clobber existing files on GTK installation.

This PR is slightly lower priority — please review after #164 and friends.

You need to run this as Administrator.
In it run:
Set-ExecutionPolicy Unrestricted
.\scripts\windows-setup.ps1

Then open a new window, and run npm i -- that works.


Prevent the default progress bar

Is there any way to remove the default progress bar the shows inside the player?
I'm thinking it could potentially confuse users of the App to think something is loading.

Hi, unfortunately it's not possible to remove basic components from the web UI.

Thanks mate!

Thanks mate!


Disable Verification is missing

[Enter feedback here]
I made mistake. The "local account sign up page" in the pages table did not look like a clickable link.. it is.
Docs are good!
Select User flows.
Select the user flow for which you want to disable email verification.
Select Page layouts.
Select Local account sign-up page.
Under User attributes, select Email Address.
In the Requires Verification drop-down, select No.
Select Save. Email verification is now disabled for this user flow.
When you click page layouts, there is no option to select the page. I am trying to disable disable email verification during sign up in the page layouts section but I think the docs may be outdated.

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

ID: d4f2f68f-0b69-2156-990d-c21937d52e65
Version Independent ID: b93c92d9-6635-5c03-c728-fe8339f1747f
Content: Disable email verification during customer sign-up - Azure AD B2C
Content Source: articles/active-directory-b2c/disable-email-verification.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@psierak,  Thanks for leveraging Azure docs feedback channel to raise this question. Since you have closed the thread I presume the issue is resolved, if it's not the case or you need further assistance kindly just let us (@shashishailaj) know.
-As a side note, this channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.


Add chapter 5 of Rethinking 2

@aloctavodia @AlexAndorra Finished but I'm stuck on a couple things I could use help on. To do them one at a time.
Do you guys know how to do code example 5.19 to 5.21 in the book cleanly in PyMC3? There's two parts

Putting two models under one model context
Simulated counterfactuals in a particular order (In this case age -> Marriage rate first, then (Age, Marriage)

In overthinking 5.25 Richard shows how to do this using just plain arrays, which I am able to follow, but not sure if it's possible within Theano/PyMC3 framework

What about this?
with pm.Model() as model:
    A = pm.Data('A', df.A)
    ## A -> D <- M
    a = pm.Normal('a', 0 ,0.2)
    bM = pm.Normal('bM', 0 ,0.5)
    bA = pm.Normal('bA', 0 ,0.5)
    sigma = pm.Exponential('sigma', 1)
    mu = a + bM*df.M + bA*A
    D = pm.Normal('D', mu, sigma, observed=df.D)

    aM = pm.Normal('aM', 0 ,0.2)
    bAM = pm.Normal('bAM', 0 ,0.5)
    sigma_M = pm.Exponential('sigma_M', 1)
    mu_M = aM + bAM * A
    M = pm.Normal('M', mu_M, sigma_M, observed=df.M)

    trace = pm.sample(1000)

with model:
    seq_A = np.linspace(-2, 2, 50)
    pm.set_data({'A': seq_A})
    pps = pm.sample_posterior_predictive(trace, model=model)

_, ax = plt.subplots(1, 2)
ax[0].plot(seq_A, pps['D'].mean(0))
az.plot_hpd(seq_A, pps['D'], ax=ax[0])

ax[1].plot(seq_A, pps['M'].mean(0))
az.plot_hpd(seq_A, pps['M'], ax=ax[1])

Wow....... How did you get that to work? Youre amazing. I swear I tried that but it didnt work but yet here it is. Thank you

I don't know why reviewnb didn't pipe up, but can y'all see this? https://app.reviewnb.com/pymc-devs/resources/pull/73/

As a former US president would say: yes, we can -- at least I can

Aside from 5.39 where I'm getting different results from the book for some reason ready for review!
Not sure if the different results are from the QuadPotential method  from MCMC inference

This is really nice work, thank you @canyon289 ! Especially when considering that there is a lot of new content in this chapter compared to the 1st edition.
I tried to add useful and clear comments, but please tell me if you have any question.
Just one last comment: don't forget to run black_nbconvert on the NB when it's finalized 😉

Thanks for the new version @canyon289 ! I just left a few comments above but I think this is almost ready to merge 🎉
Before merging, don't forget to run black_nbconvert on the NB 😉
Just two notes, for this or future NBs:

If you want to get rid of the FutureWarnings, you can do:

import warnings
warnings.simplefilter(action="ignore", category=FutureWarning)

You can add ; after plots to get rid of matplotlib axes

Ooh I also forgot: don't forget to add the last watermark cell, as Osvaldo did for the first three chapters


issue 41: initial support for lua 5.3

tested on mongo-c-driver 1.3.5, tests are running flawlessly.
Lua 5.3 is supposed to be quite different, but the only change I've found so far is the introduction of the Integer (which is a subtype of a number). The rest doesn't affect this driver as far as I'm concerned.
should fix #41 !

Neat!


Change property name.

OK, I just saw the bug associated with it. This seems to have happened during refactoring to open source the code. I'd suggest to do a search and replace the string in the scripts instead (the change in the config options was intentional).

OK, I just saw the bug associated with it. This seems to have happened during refactoring to open source the code. I'd suggest to do a search and replace the string in the scripts instead (the change in the config options was intentional).

Sounds good, I didn't know which way you had changed it but I came across it when running some experiments and noticed the mismatch.


ER_NO_DEFAULT_FOR_FIELD

Below is the log outputs:
RowDataPacket { id: 205 } 60c8040e4afcf7001cfa4a04
{
  status: undefined,
  event: 'UpdateDueAt',
  created_at: 2021-06-19T12:56:32.000Z,
  updated_at: 2021-06-19T12:56:32.000Z,
  sub_ticket_id: 205
}
RowDataPacket { id: 210 } 60d1d79d4afcf7001cfa4a7a
{
  status: 'Assigned',
  event: 'UpdateStatus',
  created_at: 2021-06-22T12:29:17.000Z,
  updated_at: 2021-06-22T12:29:17.000Z,
  sub_ticket_id: 210
}
2021-06-25 14:55:08 error: ER_NO_DEFAULT_FOR_FIELD: Field 'sub_ticket_id' doesn't have a default value Error: ER_NO_DEFAULT_FOR_FIELD: Field 'sub_ticket_id' doesn't have a default value

record:60c8040e4afcf7001cfa4a04 insert succeed , but record:60d1d79d4afcf7001cfa4a7a insert failed.  Why?

Hi @shengzhou1216 sorry there was no response for a while. The error you are getting is from your MySQL server, that your table does not define a default value for the sub_ticket_id column, so you cannot perform an INSERT on that table without supplying a value for the sub_ticket_id column in the new row.


feature: enable reordering of hasMany relationship and select fields

Description
It allows you to sort relationship, and select fields using drag, and drop if you set both hasMany, and sortable to true in the field config

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

Solution to #691 & #538


Support HTML DOM element object as selector in Scene constructor (useful for unit testing)

Some unit testing frameworks such as Karma create a virtualized, in-memory DOM environment that isn't the real DOM.  It would be valuable to be able to pass in an element object as the selector to Famous' Scene constructor.  Currently it takes only a string selector which searches the real DOM.  Being able to pass in an element object as the selector is crucial to unit testing.  It allows for better isolation of components that can be tested in any environment, whether the actual DOM or a virtualized one.

@peacechen this is currently supported.
FamousEngine.createScene('#myFamousApp');

Passing in '#myFamousApp' as a selector still searches for a matching element in the actual DOM.  It just selects by ID.  Is there another trick to this?
The issue at hand is that in certain test environments, the element doesn't exist in the real DOM.  Therefore a DOM selector string will fail because the Famous engine is searching for a match in the DOM.

@michaelobriena please consider re-opening this.  The current Scene constructor doesn't accept an element object as the selector.

This is not something I believe we should support since it breaks the idea that the scene is able to run in a non UI thread.  I understand this is useful but our current offering shouldn't make anything impossible for you to do.

Will keep open until both sides are expressed

I would like to hear if there's any way to unit test a Famous widget in Karma.  Specifically a native Famous widget, not via the Angular-famous integration library.

Although I agree that it would be useful to pass in an element object into the Scene constructor, it isn't really needed for unit testing.
Famous already has "kind of a virtual DOM". The engine has been designed to be able to run in a web worker. Since we don't have access to the document within the worker thread, we send messages (or commands - however you want to call it) to the UI thread (e.g. INIT_DOM in order to initialize a DOM element). As a sirect consequence of that, everything that happens within the worker, can work in pretty much any environment (including Node.js). In fact, you could run the /core tests completely in Node.js (and we are actually doing that: https://github.com/Famous/engine/blob/master/package.json#L8).
In short, this means that in order to unit test Famous (or your Famous app), you don't necessarily need to use the UI thread at all. Just don't do FamousEngine.init() and instead manually do FamousEngine.step(time) if needed. You can assert state using Node#getValue.  The channel abstracts the way we communicate with the UI thread within the web worker. I will look into making this easier by providing some sort of "unit testing channel" if that's something that is wanted.
To the second point: Passing in an actual DOM element instead of a selector in the Scene constructor would be something that we could support pretty easily, but there is absolutely no way this could possibly work in a web worker. Therefore the decision has been made not to implement this.

Specifically a native Famous widget, not via the Angular-famous integration library.

The angular integration is based off of 0.3 and doesn't use the current engine.

@alexanderGugel :  Thanks for explaining the inner workings of Famous. Still much for me to learn about.  As far as unit testing, the context is as a consumer of a Famous widget (e.g. the carousel).  The widget is a black box as far as the consumer is concerned. The widget takes care of instantiating the engine and setting it up.  By design the consumer has no control over that.  When the consuming project instantiates the widget and directs it to render to a selector, the widget (and thus Famous) tries to render to the DOM which in a unit testing environment may not exist.
So far the only way to test this from a consumer's viewpoint is with end-to-end testing.  It sounds like the Famous architecture currently doesn't support transparent unit testing outside of the DOM (by transparent I mean without passing additional options to tell the widget to operate on a different code path). Perhaps it's just me who feels it's not good practice to set the testee on a different code path since it can alter the outcome.  I certainly wouldn't want you to rip up everything just to support this scenario, but it's worth keeping in mind for future re-architecting.


[MM-51925] Migrate to Github Actions and support ARM builds

Summary

Migrate CircleCI to Github Actions
Support ARM builds using Docker Buildx

Ticket Link
https://mattermost.atlassian.net/browse/MM-51925

Ping @mattermost/core-build-engineers @mattermost/cloud-sre

Addressed comments

Nice thanks!

Looks like we also need to unregister this repo from the CircleCI dashboard. Otherwise it keeps trying to find the config file and fails.
cc @toninis

Looks like we also need to unregister this repo from the CircleCI dashboard. Otherwise it keeps trying to find the config file and fails.
cc @toninis

CircleCI config removed 🚀


Dataset

Hi, thanks for your very interesting work. Can you provide the dataset you developed?

Hi @sieme97,
unfortunately, at the moment we do not have a quick way to share the generated data (several GBs). However, you can use the images listed in samples/dCOCO_file_list.txt to generate depth maps with MiDaS and create the flow labels.


restorable: "pixels must not be stale when restoring" error is reported on Android

Previously this was solved by #860, but now the same issue was newly reported. This time,
mSurfaceView.onResume()
mSurfaceView.onPause()

caused this issue, but I am not sure.

This can theoretically happen when Ebiten functions are called on a different goroutine. Hmm, I don't know how to fix this issue correctly 🤔
Metal / Vulkan would remove the necessity of restoring anyway...

My current idea is to block all image operations between the end of the frame-update and the start of the frame-update.

Huh, GC and finalizers were to blame, which were executed on a different goroutine.

Confirmed this was fixed.


Add Contributors

@all-contributors please add @GeekMasher for code and review

@all-contributors please add @GeekMasher for code and review


Overhauled Ubuntu install steps via new page

This PR also:

adds links for the new Ubuntu install page to the navigation pane and the deployment landing page for easy access
removes the Ubuntu/Debian tab from the deployment landing page

@nab-77 - Please update this PR to indicate where the Marketing CTA links should be added. Thanks!
Note: The docs link via mattermost.com/download to the Ubuntu install docs will need to be updated from https://docs.mattermost.com/install/installing-ubuntu-2004-LTS.html to https://docs.mattermost.com/install/install-ubuntu.html. Docs will create client-side redirects to ensure users with legacy Ubuntu install URLs are redirected to the correct page.
@emdecr - This PR is now ready for you to add the Marketing CTAs and code formatting. Thank you! Can we get your help with the note above too to update the website URL?

Missed the tag on this one; for some reason I'm not getting notifications for this PR via the GitHub <> MM integration 😞
@cwarnermm I took a stab at the merge conflict in deployment.rst that was happening when I pulled from master, but worth a double-check to make sure that's all still good. Sorry if I mucked things up 😅
Lemme know, if anything - TY!

Oh and yes, once we merge, happy to change the URL on mm.com/download – just keep me in the loop or ping me, and I'll make the change.


Update to Typesense 0.25

Change Summary
Regenerated Typesense Models to updated API spec.
Updated Search Parameters for new models.
Fixed any type errors resulting from regenerated models.
PR Checklist

[x] I have read and signed the Contributor License Agreement.

I've implemented Analytics and added the associated tests. 3/4 analytics test pass on my device with only delete failing due to a decoding error. The JSON response is simply { name : nameValue } while the decoder expects values for each element in the AnalyticsRuleSchema. If this is a valid response, I can simply change the AnalyticsRuleSchema to have optional values -- or implement a different workaround if this is not desired.

The JSON response is simply { name : nameValue }

Oh yeah that's the right response. We need to update the Typesense API spec accordingly.

Gotcha. Made that change. Still failing all multi search and synonym tests due to "collection already exists" errors.

I see the test failures when I run on my Mac while emulating Typesense via docker. The compiling errors I believe are due to an older version of swift. That should be fixed with the latest commit.

Yup, I see those HTTP 409 errors now in GitHub action as well...
I wonder how the same test suite works on master 🤔
May be we should add logic to the collection creation code snippets in the tests, to not error out if the collection already exists?

Just did that on multi search tests and all tests passed locally (didn't need to do it for synonyms oddly enough).
Unfortunately I had to remove highlight from SearchResultHit at least for the time being. Since Any does not conform to decodable, I tried to represent it as data first and then use JSONSerialization, but unfortunately the decoder recognizes the highlight field as a dictionary, throwing an error (and the compiler throws an error if highlight is declared a dictionary). I'll take a few more stabs at it if any lightbulbs click.
getNextNode() is now failing -- and has been since I implemented AnalyticsTests. Oddly enough, it seems related to analytics tests occurring first. When I run the APICallTests alone they pass. Doesn't make much sense as everything is a value type.

getNextNode() is now failing -- and has been since I implemented AnalyticsTests

Looks like the last run of the test suite passed on CI?

Awesome, thank you @willtempleton!


[New architecture] Upgrade react-native-picker-select

We need to upgrade react-native-picker-select in order to use the new architecture in the App. We use Expensify's for of the library, and the PR has already been merged - https://github.com/Expensify/react-native-picker-select/pull/12. It would just be the case of changing the commit hash to this one: https://github.com/Expensify/react-native-picker-select/commit/da50d2c5c54e268499047f9cc98b8df4196c1ddf
More context: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293299589539
Discussion here: https://expensify.slack.com/archives/C01GTK53T8Q/p1707293531588409

Just an upgrade, not a new lib. Recategorizing...

@abekkala unassigning you as this task was just forked off from a greater ongoing initiative to unable the new React Native Architecture in E/App.

Hey! I’m Wojtek from Software Mansion, an expert agency, and I’d like to work this issue.

Hey @abekkala, @shubham1206agra should be paid $500 for their review of https://github.com/Expensify/App/pull/36117 (assuming no regressions)

@shubham1206agra offer sent
once the regression waiting period has passed, I'll submit payment

@shubham1206agra payment sent and contract ended - thank you! 🎉


Fix broken links

Hello,
I just fixed the broken links.
Happy coding,
Darius

I signed it!

Jeremy: Be aware that you'll need to merge this manually into the internal tree (you can export it as a patch and re-import it).
Using GitHub's automatic merge will cause this change to get lost the next time the automerger runs, unfortunately.


It seems lsquic does not comply with the requirements of RFC 9000 when processing the "MAX_STREAM_DATA frame"

Dear all:
The story begins here:
When lsquic serves as the server and handles chromium's(client) ietf quic v1 requests, we encountered a problem.
The error log from client-side is as follows:
I chromium: [918:1415:1119/151718.781863:INFO:quic_connection.cc(2153)] Client: Received Transport ConnectionClose for connection: 7330ae0c3284cfd2, with error: QUIC_IETF_GQUIC_ERROR_MISSING (received MAX_STREAM_DATA frame on never-opened stream 0), transport error code: STREAM_STATE_ERROR, error frame type: 0

It seems lsquic close the connection, as stream 0 was not opened, but the MAX_STREAM_DATA frame arrived first.
Following is the code:

When we captured the packet, we saw that the MAX_STREAM_DATA frame encoded before the request for stream 0

It seems normal.
My question:
But, when I read the RFC 9000, it writes:

So, I think when checking whether the stream is already opened at the point receiving the MAX_STREAM_DATA frame, the lsquic need to check if the stream belong to its side first?
We can discuss more about this.
Wait for your reply.

Yours sincerely
Michel Kong

I will submit a fix later.


revert_client_build_for_windows_x64

Can't build client on jdk8 for x64

LGTM.

@karianna I don't have the rights to merge so that's up to you :)


Allow setting analog period on pins

Addresses the servo scenario on https://github.com/microbit-foundation/micropython-microbit-v2-simulator/issues/76

Main risk for review is probably that I'll break the existing pin support.


PR: Modify color scheme preferences

Related to #820
Description
This PR aims to simplify the user experience for editing the color schemes.
It also incorporates  the creation/deletion of an unlimited number of custom schemes.
This PR however does not (yet) add the custom schemes to the options in the other plugins (editor etc...) as the idea is to unify the different color schemes into a single global one, located in this updated preferences page.
The custom name is generated automatically (custom-N) and cannot be modified for now, as the modifications needed for this would add too much to the PR, which is already getting big.
Screenshots

@nodd, @SylvainCorlay, @blink1073 comments are welcome

I love the comments on your own PR, very schizophrenic.  I am +100 on this change.

Hehe, I wanted to save some type, and I am almost sure someone had something to say in that respect,
Would you mind testing it @blink1073 ?

A couple of quick suggestions:

I don't like the Create new scheme button to expand the two columns. I'd like to see it at the bottom but only on the left column (i.e. the buttons column :-)
In the dropdown menu please change Scheme to Scheme:

:+1:

The first thing I thought when viewing the screenshots is that the preview is really a great addition. Just the line numbers are missing.
About the edition of the colors, the entries should be grouped differently, ideally with many group widgets. The minimum would be to group the text values together so that the checkboxes are not all over the place. One group for "text" and one for "background" for example. Or "text", "text highlighting" and "background".

Look update
@ccordoba12

@Nodd

Plus now we have instant preview, so any changes on an edited theme will reflect in the preview right away (without the need to apply the changes)

It is looking pretty good already, but I guess the final touch would be to have the preview in the editor dialog, and have it update in every single change inside the actual dialog.
What do you think?

Here is my habitual flow of nitpicks:

Maybe current cell and current line should be in the Background group, since the are applied to blocks of code and not particular words or characters.
Background group could be before Highlight group since it's more general (not sure about this one)
Normal text should be on top
builtin and keyword should be next of each other

having the preview in the same dialog would be nice too, but you could also put the edition widgets in the main panel next to the current preview. Maybe it's too big to be practical, I'm not sure.
There is too much space between the "Preview" label and the preview widget, I'd completely remove the label since it's already quite obvious.

After a very quick overview of the code, it seems that all the definitions are stored in the main configuration file.  Ideally each scheme should be in its own file so that it's easier to share, but I'd leave that for another PR (along with import/export)

I guess the final touch would be to have the preview in the editor dialog

It's fine where it is for me. I think few people will go into the trouble of customizing themes, but most will like to see what a theme looks like instead.

Maybe current cell and current line should be in the Background group

I don't know about this one, but I agree with the other @Nodd suggestions.

Maybe current cell and current line should be in the Background group, since the are applied to blocks of code and not particular words or characters.

I do not agree, cause the idea is to have a highlight group.... independent of what it highlights

Background group could be before Highlight group since it's more general (not sure about this one)

I am not sure what general mean? you mean would be changed more often?

Normal text should be on top builtin and keyword should be next of each other

Sure, it make sense, I was waiting for you to give me the full order, I just grouped them together...

There is too much space between the "Preview" label and the preview widget, I'd completely remove the label since it's already quite obvious.

Sure

having the preview in the same dialog would be nice too, but you could also put the edition widgets in the main panel next to the current preview. Maybe it's too big to be practical, I'm not sure.

No no no no and no, the idea was to take away so much visual noise from the user when selecting, a theme. The casual user really just selects a default, a power user would either import one, or make one.
The edition dialog is already too large to bring it back to the main menu.

After a very quick overview of the code, it seems that all the definitions are stored in the main configuration file. Ideally each scheme should be in its own file so that it's easier to share, but I'd leave that for another PR (along with import/export)

:-1:
To do this I think we need a common use color schemes format, which probably exists and stores stuff as XML or JSON, that is the point of having the Export options, so that you can share them in any format we decide to support, that will come in another PR when those "standards" are pointed out by someone. Any preferences in Spyder is built on the CONF idea, which makes life easy, but restricts us to keep using it, so having separate files would be another layer of possible bugs...

@nodd, @ccordoba12
Look update

I would leave out the create new button for now, and leave that for another PR, as it is just inviting confusion at the moment.  Otherwise, I am :+1: for merging and iterating.

@blink1073, I will remove the button but leave the code that does the work ok, as it is perfectly functional?

Yep, makes sense to me.

@blink1073 by the way did you actually test it :-p ?

Yes, hence my comment about it being confusing :)

@blink1073 was having some issues with the dialog being too tall, so now is a two columns layout

Perfect, thank you, I am no longer confused.  :)

My main concern about the config file full of everything is that when there is a problem in the config we have to reset everything. I'd bet that it's not the color definitions that will cause spyder to fail, but they will be reset anyway along everything else.
Way too many issues are solved by spyder --reset, we have something to solve here (on the long term).
It's very nice on 2 columns, but actually I'd change the order again (sorry, I was on my phone before):

Normal text

Comment

String

Number

Keyword

Builtin

Definition

Instance (should be renamed to self, or do whe use this color for something else ?)

Current cell

Current line

Occurrence

Matched parens

Unmatched parens

Link

I agree that technically cells and current line are highlighted by the color, but when working on code it's more of a general visual help rather than an important information about what I'm doing in my code. I don't want  to start a big debate, as putting them in highlight works too.

@Nodd cool, thanks for the input
I understand your config concerns and reset, but that is a whole different discussion, so...
Pas maintenant!

Gimme a  :+1: and go to bed @Nodd

The screenshots look good to me ! :+1:

Ok, @ccordoba12 fixed all comments, and added editable naming for all the color schemes

Ok, so @ccordoba12
Only custom schemes can edit the name of the scheme

@ccordoba12 made the fixes, thanks for the review

There's one little error I discovered with live testing: even though you can change a custom scheme name in the scheme editor dialog, that name is not reflected in the schemes combo box of the new preferences pages.
I think it should be changed there too :-)

@ccordoba12 fixed the issue and related comments

@ccordoba12 merged with master....

I can't open our Preferences after your update. This is the traceback
Traceback (most recent call last):
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/spyder.py", line 2614, in edit_preferences
    widget.initialize()
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 70, in initialize
    self.setup_page()
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 995, in setup_page
    self.update_combobox()
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1030, in update_combobox
    self.scheme_choices_dict[self.get_option('{0}/name'.format(n))] = n
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 49, in get_option
    return CONF.get(self.CONF_SECTION, option, default)
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/config/user.py", line 380, in get
    raise cp.NoOptionError(option, section)
ConfigParser.NoOptionError: No option 'Custom/name' in section: 'color_schemes'

@ccordoba12 I dont get  that error, can you reset your ini ???

@ccordoba12 ?

I added as comments how to fix the bug I'm seeing. The problem is the old Custom scheme has not name attribute (added by you to the other schemes). And Custom  is returned as a valid scheme name when you use names = self.get_option('names'), if you are coming from 2.3.
I think the best thing to do (as I did) is to remove Custom from names. Another option would be to remove all Custom/* options from users spyder.ini. But that can't be done by a bump in CONF_VERSION because there is no Custom section in conf/main.py. Besides, removing those options must be done once and only once.
Of course, a potential problem will occur if a user decides to name a color scheme Custom :-s

@ccordoba12 thanks for the suggested fix.
There is no potential problem, 'cause all the custom names are stored in a different key.

Oh yeah! But now I'm seeing this error when trying to create a new scheme:
Traceback (most recent call last):
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1081, in update_preview
    scheme_name = self.current_scheme
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1016, in current_scheme
    return self.scheme_choices_dict[self.current_scheme_name]
KeyError: u''
Traceback (most recent call last):
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1055, in update_buttons
    current_scheme = self.current_scheme
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1016, in current_scheme
    return self.scheme_choices_dict[self.current_scheme_name]
KeyError: u''

And this error when trying to delete it (after closing and opening Preferences again after the error mentioned above):
Traceback (most recent call last):
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1158, in delete_scheme
    self.set_scheme('spyder')
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1182, in set_scheme
    dlg.set_scheme(scheme_name)
  File "/home/carlos/Projects/spyder/github-repo/spyderlib/plugins/configdialog.py", line 1231, in set_scheme
    self.stack.setCurrentIndex(self.order.index(scheme_name))
ValueError: 'spyder' is not in list

Are this errors because of the changes you suggested?

I don't think so. My changes only remove Custom as a possible scheme. These errors are in your code.


Fix travis ci

Looks like ruby 2.4 added the sum method to Enumerable module, which causes the test to fail. https://ruby-doc.org/core-2.4.1/Enumerable.html#method-i-sum

Is Rubinius not supported by travis anymore?
As for the second commit - probably our sum should be changed to behave like the one in Ruby 2.4 for the sake of consistency.

I read through travis ci and found we were missing Trusty to be able to use Rubinus :)
Also updated the sum method to behave more like current one. Also added two more test cases that reflect the given examples.

The changes look good now. You should just squash all the commits into two - 1 fixing rbx and 1 fixing the tests and the definition of sum.

Squashed them!


Package request: Spleeter

Project description
Spleeter is the Deezer source separation library with pretrained models written in Python and using Tensorflow. It makes it easy to train music source separation models (assuming you have a dataset of isolated sources), and provides already trained state of the art models for performing various flavours of separation.
Metadata

homepage URL: https://research.deezer.com/projects/spleeter.html
source URL: https://github.com/deezer/spleeter
license: mit
platforms: linux, darwin

Add a :+1: reaction to issues you find important.

there was an old PR of it #133039


Simplify demos

@surma I was thinking it might make it easier to work with the project if we simplify the demos a bit.
For the AOM branch I just wrote an HTML file that includes anything I need (web components polyfills, maybe an AOM polyfill someday(?), howto-label, etc) and doesn't worry about things like %PATH%
https://github.com/GoogleChromeLabs/howto-components/blob/aom/elements/howto-checkbox/demo.html
I mainly did this because I want to be able to share the demo easily with other folks who might check out the repo and I was concerned about them getting hung up with a build step. Also on numerous occasions I've edited the demo file so it looks like the above so I could hack on something. Then when I reach a stopping point I delete all of the stuff and put things like %PATH% back so it works for the build process.
I realize the reason we have the current build setup is because of the way things live in Web Fundamentals. One idea I had was to leave the docs and everything as they are on web fundamentals, but for live demos, it could just link to our demo pages on GitHub, instead of needing to embed on the site itself.

Talked with Surma about this over hangouts and we agreed that we can switch over to hosting the demos on gh-pages. For web fundamentals we'll have a button that links to the live demo. That will cut down on a lot of the weirdness in getting stuff on /web.


README.md: Fix a broken table

That bit at the end of the file happened by itself; I suspect the file was missing a newline there.

Thanks @SamB


/web invokes LLM, causing it to repeat prior request

Issue
/web appears to add the page content to the context and then hit the LLM, which looks at the text and the previous request, thinking the previous request must be a new one.
/web should hold that content locally prepend it to the following request instead.

Version and model info
No response

Thanks for trying aider and filing this issue.
You can always just include a URL in a chat message, so you can explain what you want to do with the URL. Aider will ask if you want to include it.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


New pre-release of numpy breaks an old repository with pinned versions

Since a few days ago our CI started failing for one of the 'frozen in time" repos we use to check if we can still build old images.
The problem is that a new version of numpy was pre-released a few days ago. Since then one of our "check we don't break old repos in the wild" tests is now installing the latest version of numpy, which requires Python 3 but the repo wants Python 2.
The weird thing is that the repository specifies a version of numpy that it would like to install, a Python version and a pinned matplotlib version as well.
This is the end of the trace back we see:
      File "/srv/conda/envs/kernel/lib/python2.7/site-packages/setuptools/sandbox.py", line 45, in _execfile
        exec(code, globals, locals)
      File "/tmp/easy_install-VBFt7K/numpy-1.17.0rc1/setup.py", line 31, in <module>

    RuntimeError: Python version >= 3.5 required.
    ----------------------------------------
ERROR: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-sgij5M/matplotlib/

This is while it is trying to install matplotlib-1.5.1. It is odd that this is trying to install numpy 1.17 from source instead of 1.10.4 which is what is specified in requirements.txt
Figuring out why this happens is high priority as it will block all PRs.

Don't use pip regularly; according to the docs the order of installation process is as follows.

Identify the base requirements. The user supplied arguments are processed here.
Resolve dependencies. What will be installed is determined here.
Build wheels. All the dependencies that can be are built into wheels.
Install the packages (and uninstall anything being upgraded/replaced).

From the Travis logs it seems like Matplotlib is downloaded and installed first, which seems odd because pip should install packages in topological order meaning that NumPy (and any other Matplotlib deps) should be installed first.

It's related to setup_requires since there's no wheel for matplotlib 1.5.1 which has numpy in setup_requires, which is handled by setuptools (not pip). Since there's no wheel for matplotlib, it's building from source, pulling in numpy during build. At the time it does this, it doesn't know that numpy is topologically before matplotlib (it can't know when building from source, since the install_requires metadata hasn't been determined yet). Since setup_requires and easy_install are deprecated functionality, they've never been fixed to support things like python_requires metadata that would forbid the new prerelease which has dropped py2.
The basic answer is: pinning matplotlib 1.5.1 doesn't work for Python 2.7 unless numpy is installed first, and for packages installed from source at least, requirements.txt must itself be in topological order.
Related: https://github.com/pypa/setuptools/issues/917 which proposes PEP 518 as the replacement for setup_requires, but no real plan to fix the deprecated setup_requires to use pip (since it could break other things).

That all makes sense to me but not sure where it leaves us. Should we open a PR to update the repo's requirements.txt file to put numpy ahead of matplotlib? Not sure what else can be done...other than dropping the repo from the test suite.

So we've been building numpy from source for matplotlib all along but now the version of numpy that gets pulled in for that doesn't work with Python 2 any more which is why we see the failure now?
We can't edit the repository because it is a Zenodo record that is immutable.
What we could do is take the repo and create a new Zenodo entry where the order in the requirements.txt is switched.

So we've been building numpy from source for matplotlib all along but now the version of numpy that gets pulled in for that doesn't work with Python 2 any more which is why we see the failure now?

Yes, that's exactly right. This is why truly reproducible builds are hard!


Fix LookupObjects SQL injection warning

Use AR instead of SQL

Coverage remained the same at 91.967% when pulling 21e371450d816c53c365d5811b2da2dfb9b12873 on jdc-lookup_objects-sql-injection into 5f02de245e5cc7848dcd7d19d677e793bbd29dcd on master.

@JoeCohen Can you show me how you do your benchmarks?

I did a crude Benchmark in the console, arbitrarily setting local variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do your
benchmarks?
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for the article. Did you write out the test as a file, and execute it in the console as she suggests? Or just type the code directly into console...
sorry for my ignorance! I don't even know how to execute a file from the console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do your
benchmarks?
—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it. I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and execute
it in the console as she suggests? Or just type the code directly into
console...
sorry for my ignorance! I don't even know how to execute a file from the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Got it, thanks. Do you use a debug console, or just rails console?
I find that rails console acts unpredictably if I paste in multi-line code, at least in VSC. I had better luck with pry, when I had that gem installed, and also with debug.

On Jul 13, 2022, at 5:32 PM, Joseph D. Cohen @.***> wrote:
I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it. I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and execute
it in the console as she suggests? Or just type the code directly into
console...
sorry for my ignorance! I don't even know how to execute a file from the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

I use rails console. I hadn't thought of other ways to do it. And VS Code
does display unpredictably when the lines are too long.
On Wed, Jul 13, 2022 at 5:38 PM andrew nimmo @.***>
wrote:

Got it, thanks. Do you use a debug console, or just rails console?
I find that rails console acts unpredictably if I paste in multi-line
code, at least in VSC. I had better luck with pry, when I had that gem
installed, and also with debug.

On Jul 13, 2022, at 5:32 PM, Joseph D. Cohen @.***> wrote:
I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it. I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and
execute
it in the console as she suggests? Or just type the code directly into
console...
sorry for my ignorance! I don't even know how to execute a file from
the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183818266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFEXISTSRX2Y5YNXJ7TVT5OPTANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Roger. If i find a better way, I’ll share it!
nimmo
On Jul 13, 2022, 5:46 PM -0700, Joseph D. Cohen @.***>, wrote:

I use rails console. I hadn't thought of other ways to do it. And VS Code
does display unpredictably when the lines are too long.
On Wed, Jul 13, 2022 at 5:38 PM andrew nimmo @.***>
wrote:

Got it, thanks. Do you use a debug console, or just rails console?
I find that rails console acts unpredictably if I paste in multi-line
code, at least in VSC. I had better luck with pry, when I had that gem
installed, and also with debug.

On Jul 13, 2022, at 5:32 PM, Joseph D. Cohen @.***> wrote:
I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it. I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and
execute
it in the console as she suggests? Or just type the code directly into
console...
sorry for my ignorance! I don't even know how to execute a file from
the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183818266,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALDFEXISTSRX2Y5YNXJ7TVT5OPTANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.Message ID: @.***>

Your way works great and there really is no clear reason to ever use the
command line, but since that is the way I do it, I thought I would briefly
share how I write command line scripts that run the MO env.
Basically, just follow the scripts.  In particular, I usually use
scripts.mushroom_mapper.rb as the template:
#!/usr/bin/env ruby
frozen_string_literal: true
require(File.expand_path("config/boot.rb", dir))
require(File.expand_path("config/environment.rb", dir))

exit(0)
Save that as, say, /vagrant/mushroom-observer/test.rb, then
vagrant/mushroom-observer> chmod a+x test.rb
vagrant/mushroom-observer> ./test.rb
That's it!
(Note that I removed the ".." from the path of boot and environment.rb
because the script lives in the main MO dir, not scripts.)
On Wed, Jul 13, 2022 at 9:01 PM andrew nimmo @.***>
wrote:

Roger. If i find a better way, I’ll share it!
nimmo
On Jul 13, 2022, 5:46 PM -0700, Joseph D. Cohen @.***>, wrote:

I use rails console. I hadn't thought of other ways to do it. And VS Code
does display unpredictably when the lines are too long.
On Wed, Jul 13, 2022 at 5:38 PM andrew nimmo @.***>
wrote:

Got it, thanks. Do you use a debug console, or just rails console?
I find that rails console acts unpredictably if I paste in multi-line
code, at least in VSC. I had better luck with pry, when I had that
gem
installed, and also with debug.

On Jul 13, 2022, at 5:32 PM, Joseph D. Cohen @.***> wrote:
I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it.
I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and
execute
it in the console as she suggests? Or just type the code directly
into
console...
sorry for my ignorance! I don't even know how to execute a file
from
the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
<
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/>
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how
you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183818266
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFEXISTSRX2Y5YNXJ7TVT5OPTANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183834624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNLJCL4H3WC7HILZ2GTVT5RFJANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ahaa. This is helpful!
Definitely preferable to have the script in a fiie. It can get annoying running a multi line bit of code more than once in the console, because I have to keep pasting it in.
I may add this to the developer docs.
nimmo
On Jul 13, 2022, 7:14 PM -0700, Jason Hollinger @.***>, wrote:

Your way works great and there really is no clear reason to ever use the
command line, but since that is the way I do it, I thought I would briefly
share how I write command line scripts that run the MO env.
Basically, just follow the scripts. In particular, I usually use
scripts.mushroom_mapper.rb as the template:
#!/usr/bin/env ruby
frozen_string_literal: true
require(File.expand_path("config/boot.rb", dir))
require(File.expand_path("config/environment.rb", dir))

exit(0)
Save that as, say, /vagrant/mushroom-observer/test.rb, then
vagrant/mushroom-observer> chmod a+x test.rb
vagrant/mushroom-observer> ./test.rb
That's it!
(Note that I removed the ".." from the path of boot and environment.rb
because the script lives in the main MO dir, not scripts.)
On Wed, Jul 13, 2022 at 9:01 PM andrew nimmo @.***>
wrote:

Roger. If i find a better way, I’ll share it!
nimmo
On Jul 13, 2022, 5:46 PM -0700, Joseph D. Cohen @.***>, wrote:

I use rails console. I hadn't thought of other ways to do it. And VS Code
does display unpredictably when the lines are too long.
On Wed, Jul 13, 2022 at 5:38 PM andrew nimmo @.***>
wrote:

Got it, thanks. Do you use a debug console, or just rails console?
I find that rails console acts unpredictably if I paste in multi-line
code, at least in VSC. I had better luck with pry, when I had that
gem
installed, and also with debug.

On Jul 13, 2022, at 5:32 PM, Joseph D. Cohen @.***> wrote:
I typed/inserted the text in the console.
I'm equally ignorant, so that seemed like the easiest way to do it.
I was
also unclear about how to do it as a file in the context of MO.
On Wed, Jul 13, 2022 at 5:29 PM andrew nimmo @.***>
wrote:

Thanks for the article. Did you write out the test as a file, and
execute
it in the console as she suggests? Or just type the code directly
into
console...
sorry for my ignorance! I don't even know how to execute a file
from
the
console, in the context of rails. Do you use rails runner?

On Jul 13, 2022, at 1:02 PM, Joseph D. Cohen @.***> wrote:
I did a crude Benchmark in the console, arbitrarily setting local
variables.
See How to Benchmark my code in Ruby or Rails
<
https://www.lynnbright.com/learn-how-to-use-ruby-benchmark-module/>
On Wed, Jul 13, 2022 at 12:07 PM andrew nimmo @.***>
wrote:

@JoeCohen https://github.com/JoeCohen Can you show me how
you do
your
benchmarks?
—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183578840

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFHAHZP4NPRINRP5EMDVT4HXHANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<

https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183812419

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAALDFBHCYTGJPNLTI533QTVT5NODANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.

—
Reply to this email directly, view it on GitHub
<
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183818266
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAALDFEXISTSRX2Y5YNXJ7TVT5OPTANCNFSM53MRS57Q

.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/MushroomObserver/mushroom-observer/pull/1084#issuecomment-1183834624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYTNNLJCL4H3WC7HILZ2GTVT5RFJANCNFSM53MRS57Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because your review was requested.Message ID: @.***>


WSAECONNRESET: Vulkan program closes immediately after launch because of other Vulkan layer

Description

I created a triangle program using Rust and Ash, and it runs fine when launched on its own. However, when launched through Renderdoc Launch Application, the program opens and immediately closes, making it impossible to debug. Everytime renderdoc's log end with "WSAECONNRESET"
RDOC 046344: [00:03:30]    win32_process.cpp( 526) - Log     - Running process F:\SourceCodes\Rust\learn-vulkan-rs\target\debug\triangle.exe
RDOC 046344: [00:03:30]    win32_process.cpp( 610) - Log     - Injecting renderdoc into process 42656
RDOC 042656: [00:03:30]             core.cpp( 499) - Log     - RenderDoc v1.26 Windows 64-bit Release (4524cddca999d52aff790b626f92bb21ae9fe41f) capturing application
RDOC 042656: [00:03:30]      crash_handler.h( 150) - Log     - Timer Waiting for crash handling server - 58.853 ms
RDOC 042656: [00:03:30]      crash_handler.h( 156) - Log     - Created crash-handling server \\.\pipe\RenderDocBreakpadServer176133708544
RDOC 042656: [00:03:30]      crash_handler.h(  87) - Log     - Connecting to server \\.\pipe\RenderDocBreakpadServer176133708544
RDOC 042656: [00:03:30]         settings.cpp( 460) - Log     - Loading config from C:\Users\xxxx\AppData\Roaming\renderdoc\renderdoc.conf
RDOC 042656: [00:03:30]         settings.cpp( 311) - Log     - Android.SDKDirPath has been customised from  to C:/Users/xxxx/AppData/Local/Android/Sdk
RDOC 042656: [00:03:30]         settings.cpp( 311) - Log     - Android.JDKDirPath has been customised from  to C:/Users/xxxx/scoop/apps/openjdk/current
RDOC 042656: [00:03:30]   win32_libentry.cpp(  68) - Log     - Loading into F:\SourceCodes\Rust\learn-vulkan-rs\target\debug\triangle.exe
RDOC 042656: [00:03:30]  sys_win32_hooks.cpp(  84) - Log     - Registering Win32 system hooks
RDOC 042656: [00:03:30]      d3d11_hooks.cpp(  44) - Log     - Registering D3D11 hooks
RDOC 042656: [00:03:30]        amd_hooks.cpp( 154) - Log     - Registering AMD hooks
RDOC 042656: [00:03:30]      nvapi_hooks.cpp( 141) - Log     - Registering nvidia hooks
RDOC 042656: [00:03:30]      d3d12_hooks.cpp( 179) - Log     - Registering D3D12 hooks
RDOC 042656: [00:03:30]       d3d8_hooks.cpp(  36) - Log     - Registering D3D8 hooks
RDOC 042656: [00:03:30]       d3d9_hooks.cpp(  45) - Log     - Registering D3D9 hooks
RDOC 042656: [00:03:30]       dxgi_hooks.cpp( 250) - Log     - Registering DXGI hooks
RDOC 042656: [00:03:30]        egl_hooks.cpp(1073) - Log     - Registering EGL hooks
RDOC 042656: [00:03:30]         gl_hooks.cpp( 280) - Log     - Registering OpenGL hooks
RDOC 042656: [00:03:30]        wgl_hooks.cpp( 655) - Log     - Registering WGL hooks
RDOC 042656: [00:03:30]         vk_layer.cpp(  99) - Log     - Registering Vulkan hooks
RDOC 042656: [00:03:30]      crash_handler.h(  51) - Log     - Re-using crash-handling server \\.\pipe\RenderDocBreakpadServer176133708544
RDOC 042656: [00:03:30]      crash_handler.h(  87) - Log     - Connecting to server \\.\pipe\RenderDocBreakpadServer176133708544
RDOC 046344: [00:03:30]   target_control.cpp( 626) - Log     - Got remote handshake: triangle [42656]
RDOC 042656: [00:03:30]          app_api.cpp( 363) - Log     - Initialising RenderDoc API version 1.6.0 for requested version 10100
RDOC 042656: [00:03:30]             core.cpp(2001) - Log     - Adding Vulkan device frame capturer for 0x000001F03612BAC0
RDOC 042656: [00:03:30]  vk_device_funcs.cpp( 887) - Log     - Initialised capture layer in Vulkan instance.
RDOC 042656: [00:03:30]          vk_core.cpp(3815) - Log     - RegisterSurface() window<PHONE_NUMBER>331042
RDOC 042656: [00:03:30]             core.cpp(2037) - Log     - Adding Vulkan frame capturer for 0x000001F03612BAC0 / 0x0000000000331042
RDOC 042656: [00:03:30]  vk_device_funcs.cpp(1455) - Log     - physical device 0: NVIDIA GeForce RTX 4090 (ver 531.41 patch 0x0) - 10de:2684
RDOC 042656: [00:03:30]   vk_next_chains.cpp(1568) - Error   - Unhandled struct VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_ACCELERATION_STRUCTURE_FEATURES_KHR in pNext chain
RDOC 042656: [00:03:30]   vk_next_chains.cpp(1568) - Error   - Unhandled struct VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_RAY_TRACING_PIPELINE_FEATURES_KHR in pNext chain
RDOC 042656: [00:03:30]   vk_next_chains.cpp(2738) - Error   - Unhandled struct VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_ACCELERATION_STRUCTURE_FEATURES_KHR in VkDeviceCreateInfo pNext chain
RDOC 042656: [00:03:30]   vk_next_chains.cpp(2738) - Error   - Unhandled struct VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_RAY_TRACING_PIPELINE_FEATURES_KHR in VkDeviceCreateInfo pNext chain
RDOC 042656: [00:03:31]  vk_device_funcs.cpp(4028) - Log     - Created capture device from physical device 0
RDOC 042656: [00:03:31]     vk_serialise.cpp(1938) - Error   - No support for VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_ACCELERATION_STRUCTURE_FEATURES_KHR is available in this build
RDOC 042656: [00:03:31]     vk_serialise.cpp(1938) - Error   - No support for VK_STRUCTURE_TYPE_PHYSICAL_DEVICE_RAY_TRACING_PIPELINE_FEATURES_KHR is available in this build
RDOC 042656: [00:03:31]        vk_memory.cpp(  71) - Log     -   Memory type 0: VkMemoryPropertyFlagBits(0) in heap 1 (VkMemoryHeapFlagBits(0)) (31.8 GB) []
RDOC 042656: [00:03:31]        vk_memory.cpp(  71) - Log     -   Memory type 1: VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT in heap 0 (VK_MEMORY_HEAP_DEVICE_LOCAL_BIT) (23.6 GB) [GPULocal]
RDOC 042656: [00:03:31]        vk_memory.cpp(  71) - Log     -   Memory type 2: VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT | VK_MEMORY_PROPERTY_HOST_COHERENT_BIT in heap 1 (VkMemoryHeapFlagBits(0)) (31.8 GB) [upload]
RDOC 042656: [00:03:31]        vk_memory.cpp(  71) - Log     -   Memory type 3: VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT | VK_MEMORY_PROPERTY_HOST_COHERENT_BIT | VK_MEMORY_PROPERTY_HOST_CACHED_BIT in heap 1 (VkMemoryHeapFlagBits(0)) (31.8 GB) [readback]
RDOC 042656: [00:03:31]        vk_memory.cpp(  71) - Log     -   Memory type 4: VK_MEMORY_PROPERTY_DEVICE_LOCAL_BIT | VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT | VK_MEMORY_PROPERTY_HOST_COHERENT_BIT in heap 0 (VK_MEMORY_HEAP_DEVICE_LOCAL_BIT) (23.6 GB) []
RDOC 046344: [00:03:34]    win32_network.cpp( 264) - Warning - recv peek failed: WSAECONNRESET: An existing connection was forcibly closed by the remote host.

After spending a day troubleshooting, I found a log message indicating that my program had inserted a layer called "VK_LAYER_GAMEPP" from GamePP.
[2023-04-13T15:14:10Z INFO  vulkan::debug] [General] [Loader Message (0)] : Inserted device layer "VK_LAYER_KHRONOS_validation" (F:\SDKs\VulkanSDK\<IP_ADDRESS>\Bin\.\VkLayer_khronos_validation.dll)
[2023-04-13T15:14:10Z INFO  vulkan::debug] [General] [Loader Message (0)] : Inserted device layer "VK_LAYER_GAMEPP" (C:\Users\xxxx\AppData\Local\GamePPSDK\common\vulkan\.\GPP_VKLayer64.dll)
[2023-04-13T15:14:10Z INFO  vulkan::debug] [General] [Loader Message (0)] : Inserted device layer "VK_LAYER_NV_optimus" (C:\Windows\System32\DriverStore\FileRepository\nvlei.inf_amd64_8f312ba3620ee346\.\nvoglv64.dll)
[2023-04-13T15:14:10Z INFO  vulkan::debug] [General] [Loader Message (0)] :        Using "NVIDIA GeForce RTX 4090" with driver: "C:\Windows\System32\DriverStore\FileRepository\nvlei.inf_amd64_8f312ba3620ee346\.\nvoglv64.dll"

I tried deleting the corresponding DLL and found that I was able to debug the program through Renderdoc successfully.
Steps to reproduce

I have compressed the GPP_VKLayer64.dll layer file and my triangle program into this zip file: temp.zip

install GamePP from its website (or insert their layer directly maybe?).
Launch triangle.exe in Renderdoc

Environment

RenderDoc version: 1.26
Operating System: Windows 11
Graphics API: Vulkan

NVIDIA GeForce RTX 4090
NVIDIA Game Ready: 531.61

This sounds like it's clearly a bug with the other layer. I will find the necessary variable to disable it, but uninstalling the software will also work.

Of course, one reason why I submitted the issue was to help others who may encounter the same problem. I scoured the internet (issues, Vulkan Discord, Reddit, Stack Overflow, etc.), but could not find an answer to the mysterious "WSAECONNRESET" error. Perhaps it could serve as a reminder for people to check their Vulkan layers when they encounter this type of issue.
I can close the issue if you prefer.

I appreciate you trying to help in that way, but that is very likely to do more harm than good. The reason that the diagnostic log is not directly accessible by users is because it's intended to be interpreted and viewed only by RenderDoc developers. That message has nothing to do with vulkan or the problem you encountered and will have no correlation to problems people have in the future. It is likely to cause confusion if anyone tries to use it to investigate a problem because their problem may be with an entirely different API and they may waste time investigating the wrong thing.
The root problem here here is that you had a vulkan layer installed which has bugs and caused the crash, there's nothing more to investigate there and unfortunately there is no good way to signpost this in future because it can't be detected. I'll edit the title to try and avoid confusing people in future.


[BUG] Intermittent failure in test proxy

Library name and version
Azure.Core.TestFramework
Describe the bug
See this PR which only updates autorest.csharp and doesn't carry over any code change. Its corresponding build pipeline failed intermittently.
Here is the error in the first attempt. It appeared in every failed test case.
Error Message:
   Azure.RequestFailedException : Service request failed.
Status: 400 (Bad Request)

Content:
{"Message":"Unable to parse the line taLake_369e19e7ec","Status":"BadRequest"}

After re-trying 2 times, the pipeline passed without error.
Expected behavior
There should not be intermittent failure in test proxy.
Actual behavior
Intermittent failure appears in test runs. It can be eliminated after several re-try.
Reproduction Steps
Run the pipeline, check the result. The error will go away after retrying several times.
Environment
No response

Several users have reported similar issues. Like this: https://github.com/Azure/azure-sdk-for-net/pull/37490

@JoshLove-msft :  Would you take a look please?

This is a duplicate of https://github.com/Azure/azure-sdk-tools/issues/6562


Filtering for empty strings

There's currently no way to filter based on empty strings (other than S-strings). This should work, but doesn't:
from employee
filter name != ""

instead, it fails with:
Error: during parsing
   ╭─[:2:17]
   │
 2 │ filter name != ""
   ·                 ┬
   ·                 ╰── expected string_inner
───╯

Yeah, that's a bug with string parsing.


New analyse task: Function Nested depth

Is your feature request related to a problem? Please describe.
Too many depth in using combinations of for, foreach, if/elseif/else and while blocks nested are reducing the understanding
Describe the solution you'd like
Adjusting the nested depth to a maximum. Example: 3

Done.


POST Query foreign IPS

💡 Summary
Add new REST endpoints that mirror existing sql queries and replace existing sql query with REST
Specifically: Convert "query_foreign_IPs()" TSQL query to an API endpoint
Motivation and context
There are two major reasons to use endpoints in place of sql queries.

Replace the need to know t-sql
Off load data aggregation to fastapi parallel workers

Implementation notes

Gather existing queries from code.
Create new endpoints at fastapi.
Secure new endpoints in same fashion as existing endpoint.

Acceptance criteria

[X] home/models.py model created
[X] dataAPI/schemas.py schema created
[X] dataAPI/views.py functions created
[X] dataAPI/tasks.py function created
[X] Verified endpoint is working
[X] Awaiting push to api-extended branch

Endpoint has been finished and added to the CD-pe-reports-api-extended branch