Cannot save script file if no scene file is open

Godot version:
3.1.1 stable

Windows 10
Issue description:
I am editing a .gd file, it is a global file I have for singleton autoload. If I do not have a scene file open (i.e. something showing in the scene pane) I cannot save the script file, I get a 'a root node is required to save the scene'.
The .gd file is not associated with any scene, and I am not saving a scene, just a script.
Additionally to compound this, if I have scene1.gd script file open (associated with scene1.tscn), but I have open the scene file scene2.tscn (which does not associate with scene1.gd), then it lets me save.
i.e. The IDE isn't bothered what you have open as a scene, it will not let you save a gd script file if you have something, anything, open.
Pretty sure this is a historical thing when script files were (possibly) only embedded in scenes so having an external script was not applicable. However, if this really is 'as designed' then perhaps a less cryptic message would be useful (e.g. instead of ' root node is required to save the scene', 'you must have a scene open to save a script').
btw, possibly linked to this is when you open a .gd file the tab always displays the name of the currently open scene. So regardless of whether the .gd file is associated with the scene or not, you get the name of the open scene as the tab for the script file. Or 'empty' if you have no open scene. Indeed, as a side issue, I think related to this issue is the reason why you cannot have more than one .gd script file open, you have to use another pane that sucks up screen width just to open another script file and replace the current one...
Steps to reproduce:

open a .gd file
close the current scene
press ctrl-s
profit

Minimal reproduction project:
A project with a gd and a tscn file.

Duplicate of #22730.

I did try to search, honest! but found nothing.
I will raise another suggestion for my other points.


Route developer links /developers/:id/links/:field

Goals:

Only signed in users can message a developer and visit their website, Twitter, GitHub, and LinkedIn
Track clicks on these links

Which enables:

Better messaging when trying to message a developer when not signed in (vs. the button not showing up at all)
Upselling a company to pay to message a developer

I know that moving links to the social profiles doesn't make a ton of sense right now.  But I'm hoping a redesign to Developers#show will only show those links as buttons and generic labels (e.g. "GitHub", not "@joemasilotti").
To-do

[x] Route links through app
[x] Require authentication
[ ] Analytics
[ ] Better messaging when not signed in

Putting this on via #72.


Range and mulitple mode broken on IE 11

The range and multiple mode is not working on IE

11.
When I select date in those two modes,which should be the first/start date, they react just like single mode: just the one date I selected is input and the calendar closed.

Hello - this was a bug back in Dec 2017
I use "npm" to manage my libraries, and flatpickr is currently at 4.5.2 according to "npm".
How do I go about getting a newer version including patches like this one???
Please - this is fairly urgent as we rolled out a new version tested with Chrome (which worked perfectly) but with IE11 is failing miserably.

@chmln would you like to publish a new version into npm?  Please?

@murraycollingwood sorry for your troubles. The release you're on already includes that bugfix. I will get a new release out in a few days, see if that will help resolve your issues.

Hi Gregory
We're also looking at #1241 wondering if that is also a related issue.
Although out problems generally seem to be playing out on IE11 more than
any other browser.
Anyways, looking forward to a fresh cut on 'npm'
Cheers
Murray
On Mon, 4 Feb 2019 at 17:42, Gregory<EMAIL_ADDRESS>wrote:

@murraycollingwood https://github.com/murraycollingwood sorry for your
troubles. The release you're on already includes that bugfix. I will get a
new release out in a few days, see if that will help resolve your issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/flatpickr/flatpickr/issues/1139#issuecomment-460129452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFioKH7oUSQR_ScN1TUGB9s2MAuROwWCks5vJ7o4gaJpZM4Qx_Vm
.

--
Murray Collingwood
Focus Computing
Australia ph 07 3175 0575
New Zealand ph 03 928 1699
http://www.focus-computing.com.au

@chmln  We are reasonably sure #1241 is also giving us trouble.  Currently we have removed all of our modal dialogs so the current version is working, however if you could push a new npm version that would help us out so we could re-enable our modals.
Cheers
Murray

@chmln
We are still looking forward to that fresh cut on 'npm'.
Cheers
Murray

@murraycollingwood there have been 4 new releases since. Please check if the issue still exists. If it does, I will have it fixed asap

I want to share a workaround I've done for the issue of range selection not working correctly IE (Version: 11.615.17763.0).
Here is the issue i'm referencing:
"...they react just like single mode: just the one date I selected is input and the calendar closed."
Here is the workaround:
Setting the option "inline: true" when you are using the range mode in IE allows for the range functionality to work correctly in IE. Hope this helps someone.


i18n: add Chinese (Simplified) translation

Thanks for such a nice mod!

thanks for your contribution, will merge! :)


Update circleci docker image

as per: https://support.circleci.com/hc/en-us/articles/23614965074459-2024-Image-Deprecations-and-Brownouts-Android-Linux-VM-Remote-Docker-Linux-VM
see: https://discuss.circleci.com/t/remote-docker-image-deprecations-and-eol-for-2024/50176
We will not see in the CI here whether or not this works, but only once we create a "merge commit" and let the CI run on master.

The CI did fail, albeit at a step prior to the one I introduced changes for here: https://app.circleci.com/pipelines/github/bids-standard/bids-specification/7492/workflows/064d2593-2005-45ae-8c23-f32cdf846a41/jobs/22350


#vector celebrity_problem

clebebrity problem is solved by me.
problem statement available in internet

Please read contribution guidelines.


Custom development

Hi,
I need your system but I need some modifications. I need men having to pay for Tokens and then use 1 token per message. Is this possible and if yes what would it cost? I also need to keep track of the amount of messages woman receive in order to pay them a commission on messages received. Then also if possible I need a referral system. If a woman invites another woman to sign up, she also receives commission on the messages the second woman receives.
Looking forward to your reply,
Regards

Yes this is possible but is a lot of work! What is your budget?

Give me a quote and I will see if I can raise the budget
Sent from my iPhone

On 11 Jun 2020, at 17:00, Undefined-Variables<EMAIL_ADDRESS>wrote:

Yes this is possible but is a lot of work! What is your budget?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Update README.md

A humble typo. Amazing guide Patrick, you are doing great work!

Thanks for the patch & kind words!


flutter 3.22: linux gl texture crashes

Steps to reproduce
It's found in video players when some animation starts, for example fvp and media-kit. I also create a simple example to reproduce the bug. no crash in flutter <= 3.19.
Expected results
no crash
Actual results
crash after a few seconds
Code sample
https://github.com/wang-bin/flutter_linux_texture
Code sample
static gboolean fl_render_texture_gl_populate(FlTextureGL *texture,
                                              uint32_t *target, uint32_t *name,
                                              uint32_t *width, uint32_t *height,
                                              GError **error) {
  if (!flutter_texture_populate(texture, target, name, width, height)) {
    *error =
        g_error_new_literal(g_quark_from_static_string("gl_widget_flutter"), 0,
                            "populate texture failed");
    return false;
  } else {
 //  required to reproduce the bug
    fl_texture_registrar_mark_texture_frame_available(flutter_texture_registrar,
                                                    (FlTexture *)fl_texture);
    return true;
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

master channel works

Thanks for confirming that. Closing this as fixed on master channel.


Ensure /usr/local/bin is in the path for consul binary

Checks the current path prior to running the consul binary.

If no PATH is set, then a default value is set that includes /usr/local/bin
If PATH already includes /usr/local/bin then no change is made to the path
Otherwise, /usr/local/bin is added to the PATH environment variable

Tested change on CentOS 6.8 and verified that it fixes the problem I had encountered documented in #290

I would like the path to be set back after. The reasoning here is that I don't want weird bugs to crop up because the path variable has changed during the puppet run when operators are not expecting that.

Alright, I made the change so that the PATH environment variable is restored after consul is run.  Good catch!

Can you double check the tests? It looks like they are failing on something fact-related.

Can you try rebasing on master to get current working tests?

Hi Matt, maybe you should save the result of the exec over the last line restoring the ENV :)

Thanks for the tip.  I've fixed the code, tests are passing.

Ok here I go!

Thank you!


Investigate additionalDetails field for CloudStack machine API

Coming from this conversation

What is this field supposed to be used for?
Can the user specify only random custom UI info (like labels) or can some of this affect who CloudStack behaves? Or both?
If the user needs to pass here configuration that actually affects the CloudStack logic, can we parametrize them into actual API fields?

https://github.com/shapeblue/cloudstack/blob/main/api/src/main/java/com/cloud/vm/VmDetailConstants.java lists out some of the explicit labels/details that are allowed by CloudStack. However users can also submit their own labels/details for cloudstack management systems to use when managing the CloudStack VM's

After discussing more about this, we will be exposing userCustomDetails that would allow non-standard key value inputs that are not defined here. Any standard inputs required from user would be evaluated case by case basis, and will be exposed through definitive cluster spec inputs.

Issue has been addressed - please resolve it


How to keep default bootstrap layout

I am using this plugin but using ng-intl-tel-input directive changes the default bootstrap layout. Can you tell me how to preserve the default layout.
Before

After

Fixed by changing display value in CSS to display:block from display:inline-block


Test Explorer pane in sidebar remains after Test Explorer UI extension uninstalled

But the "Test Explorer UI" (and Python Test Explorer) have been uninstalled, and vscode restarted.

Strangely, reinstallling gives an even stranger result...

The 2 views that can be seen in your first screenshot are not from this extension - they don't use the state icons that this extension uses (note that the Test side bar is used by many testing extensions).
In your last screenshot the view in the middle is from this extension.

Sure enough, it's from the Microsoft Java Test Runner.  Sorry.


Update kube-prompt to v1.0.4

Update kube-prompt to v1.0.4
https://github.com/c-bata/kube-prompt/releases/tag/v1.0.4

This was resolved in v2.1


[Bug]: Exception Caught

What happened?
Dont know some modules are not getting downloaded properly there is temp folder and undecrypted videos

l
Branch
master/main
What operating systems are you seeing the problem on?
Windows
Relevant log output
esktop/udemy-downloader-master/udemy-downloader-master/out_dir/wifi-hacking-penetration-testing-from-scratch/12 - ARP Pd isoning Detection & Security/./002 Detecting Suspicious Activities using Wireshark. encrypted.m4a. part-Frag17 .aria2
01/19 17:02:39 [ERROR] Exception caught
Exception: [DefaultBtProgressInfoFile.cc:213] errorCode=1 Failed to write into the segment file C:/Users/Administrator/I esktop/udemy-downloader-master/udemy-downloader-master/out_dir/wifi-hacking-penetration-testing-from-scratch/12 - ARP Pc isoning Detection & Security/./002 Detecting Suspicious Activities using Wireshark. encrypted.m4a.part-Frag18. aria2
01/19 17:02:39 [ERROR] Exception caught
Exception: [DefaultBtProgressInfoFile.cc:213] errorCode=1 Failed to write into the segment file C:/Users/Administrator/I esktop/udemy-downloader-master/udemy-downloader-master/out_dir/wifi-hacking-penetration-testing-from-scratch/12 - ARP P
isoning Detection & Security/./002 Detecting Suspicious Activities using Wireshark. encrypted.m4a. part-Frag20. aria2
01/19 17:02:39 [ERROR] Exception caught
Exception: [DefaultBtProgressInfoFile.cc:213] errorCode=1 Failed to write into the segment file C:/Users/Administrator/I esktop/udemy-downloader-master/udemy-downloader-master/out_dir/wifi-hacking-penetration-testing-from-scratch/12 - ARP P
isoning Detection & Security/./002 Detecting Suspicious Activities using Wireshark. encrypted.m4a.part-Frag21. aria2
01/19 17:02:39 [ERROR] Exception caught
Exception: [DefaultBtProgressInfoFile.cc:213] errorCode=1 Failed to write into the segment file C:/Users/Administrator/I esktop/udemy-downloader-master/udemy-downloader-master/out_dir/wifi-hacking-penetration-testing-from-scratch/12 - ARP Po isoning Detection & Security/./002 Detecting Suspicious Activities using Wireshark. encrypted.

Other information
No response

path too long, move closer to root of drive and use --id-as-course-name


Add version/client data to multiplayer hello packet

this was done in the initial multiplayer implementation years ago https://github.com/etternagame/etterna/commit/5fb876e557f4d41e0fc18b3582ee5736fec7cea3


Implement the relative phase Toffoli gates

Summary
Implement the relative phase Toffoli gates (CCX and CCCX), which only existed as functions and not Gates. These gates are used in the definition of other multi-controlled gates and are blocking their implementation.
Details and comments
These gates follow https://arxiv.org/pdf/1508.03273.pdf. The relative-phase CCX is shown in Fig. 3, dashed box and the relative-phase CCCX is shown in Fig. 4, the whole circuit.
Things left to decide:

[x] How should the gate class be called? The method used to be rccx, which conflicts with the naming scheme we introduce in #3633 where we use R to denote a rotation.
Edit: Named rccx/RCCXGate since the method is already called rccx.
[x] How should the relative phase Toffolis be drawn in the circuit drawer? Drawing them with X as base gate suggests that they are the same as the Toffolis -- which they are not.
Edit: Resolved by not being a controlled gate, thus the name is drawn in a box (composite gate style).
[x] Do these gates have a base_gate? The first idea might be to use the XGate as base gate, but this is not accurate, as the base gate implies that the controlled gate can be written as
controlled_gate = |0><0| \times id + |1><1| \times X.
Also this runs into problems with the default mechanisms of Qiskit to create a controlled gate.
If they don't have a base_gate this must be accordingly dealt with in the circuit drawer.
Edit: Resolved by not being a ControlledGate but Gate type for now.

Ok, I left the method name as rccx/rcccx and named the classes RCCXGate/RCCCXGate. The docstrings describe them as simplified Toffoli or Margolus, or simplified 3-controlled Toffoli, respectively. If that naming is fine, the PR is ready to go from my side. 👍


Delay connection attempts without addresses.

Cherry pick:
https://gitlab.com/swift-server-community/RediStack/-/merge_requests/147
Motivation
In some circumstances users may have connection pools configured without any SocketAddresses ready to go. This is particularly likely in service discovery configurations. Right now, the effect of attempting to use such a pool is two fold.  Second, we'll fall into the exponential backoff phase of connection establishment.
The first property is annoying, but the second one is actively harmful. If your construction is timed incorrectly, we'll have the awkward problem of burning a bunch of CPU trying to create connections we know we cannot, and then a lengthy delay after the addresses are actually configured before we start trying to use them. That's the worst of all worlds.
This patch adds logic to detect the attempt to create connections when we don't have any configured addresses and delays them. This path should improve performance and ergonomics when in this mode.

Oh we need to change this a tiny bit to not break API.

@fabianfett I assume you mean the change from class to struct?


Fix #1689

Use Oscar's core method get_model to get Category model in data migrations.
Explanation:
Documented way to data migration in Django 1.7+ is using apps.get_model in forward/backward migrations. But this is not the same method as get_model from django.apps.apps. In migrations operations used ModelState instances which actually is:

Represents a Django Model. We don't use the actual Model class
as it's not designed to have its options changed - instead, we
mutate this one and then render it into a Model as required.

It's from here.
So, apps.get_model return fake model, which actually is deconstructed model class instance (ModelBase) with no properties, no class methods, only with model fields.
Side effect of using Oscar's get_model is requirement to Category model's properties full_name and _slug_separator will not be changed/renamed in future.

Thanks a bunch for digging into this. I'm not sure if actually loading the model is safe at this point though. IIRC the Django docs example for data migrations does what the original code does. Maybe we should only use Oscar's get_model to be able to fetch the _slug_separator? What do you think?

We probably have to put the slug separator in a setting and use that instead of the model.

First, we should decide how to be with backward migration, because it using full_name property. This property is strongly depends on treebeard's MP nodes, and therefore using RawSQL query (as one of probably solutions) for backward migration is complicated and may be wrong way.
And yes, separators can be moved to settings. But seems to it's so low-level settings that I don't think it will be used anywhere else. And again, general part of issue is backward migration.

Seems to be already fixed with 60b13


Parcelable error when reloading app back from stack and has been cleared by the os

Caused by: java.lang.RuntimeException: Parcel android.os.Parcel@1b179a5d: Unmarshalling unknown type code 7471204 at offset 1112
STR:
Enable Don't keep activities in Developer options. Start the app and go to activity with the rc progressbar.
Get out of app and go back to it. crash.
From what i gathered
Happens due to progaurd
Need to add
-keepclassmembers class * implements android.os.Parcelable {
static ** CREATOR;
}
in progaurd-rules

It should be fixed with this #16
Idk why @akexorcist hasn't accepted the PR yet as Travis says the build passed


Draggability on windows

(Don't know if that's a word...)
When I have Uc on AlwaysOnTop mode, I obviously want to reduce it in size for taking notes, right? Well, when the window size is reduced, half of the screen is small buttons, and there is nowhere to grip the window to drag it around...
I know this would be a lot of work to fix, but implementing a form of responsive design for Uc would help quite a bit while taking notes on Uc.
Thanks.

Hi @codesharkasm,
I invistigated a while on this issue and the cause of this is ridiculous : by default, the toolbar is transparent and above the window draggable region. It prevent as well the draggability of this region, and in the case of a small window, the whole draggable region...
I will fix this ASAP and release a v.0.10.2
Thanks a lot for the feedback

Oh, this issue is also caused by tabs, they prevent as well the draggability of the window draggable region.

Issue fixed & updated title bar with responsive for small (Un)colored window.
OK in next 0.10.2 release.

@n457 Thanks!


Grab delta while climbing does not respect parent rotation

Environment

Source of VRTK: Github
Version of VRTK: 3.0.1
Version of Unity3D: 5.5.0f3

Steps to reproduce
In the 037_CameraRig_ClimbingFalling example scene:

Create an empty GameObject and put the [CameraRig] and [VRTK] objects inside:
[Empty GameObject]
-- [CameraRig]
-- [VRTK]
In my case the purpose of this was to create a prefab for these components.

Set the Rotation Y of the parent object to 90.

Play the scene and try to climb something. While hanging onto the wall, try moving close to and away from the wall, or side to side - the transform between the player and the wall is incorrect.

I have fixed the problem locally by changing this line to rotate the delta, but I'm not sure whether it's an actual solution or whether it just happens to fix my use case:
https://github.com/thestonefox/VRTK/blob/e1e7f700238af2106324a05c73711ce7762341f3/Assets/VRTK/Scripts/Locomotion/VRTK_PlayerClimb.cs#L74
playArea.position = startPosition - (GetPosition(grabbingController.transform) - startControllerPosition);
to
playArea.position = startPosition - **transform.rotation** * (GetPosition(grabbingController.transform) - startControllerPosition);
If this is the correct solution I can submit a pull request for it.

Doesn't this PR resolve this issue?
https://github.com/thestonefox/VRTK/pull/903


[QUERY] How to add new access policy to key vault without deleting all existing access policies?

Library name and version
Azure.ResourceManager.KeyVault 1.2.0
Query/Question
As the title says, how do I add new access policy to key vault without deleting all existing access policies in the vault?
According to this bug report, it should be possible, but I just cannot figure out how: https://github.com/Azure/azure-sdk-for-net/issues/32872
This sample doesn't give any answers either. https://github.com/Azure-Samples/azure-samples-net-management/blob/master/samples/keyvault/manage-key-vault/Program.cs. If I modify the sample to update access policy for a new principal id, the existing access policy is deleted
Environment
No response

I tried the following code in an attempt to iterate over all existing access policies to add those to the update patch:
                var patch = new KeyVaultPatch();                 patch.Properties = new KeyVaultPatchProperties();                 var existingPolicies = keyVault.GetPolicyAssignments();                 foreach (var policy in existingPolicies)                     patch.Properties.AccessPolicies.Add(new KeyVaultAccessPolicy(tenantId, policy.Data.ManagedIdentity.PrincipalId.Value.ToString(), permissions));                 patch.Properties.AccessPolicies.Add(new KeyVaultAccessPolicy(tenantId, principalId1, permissions));                 var UpdateKeyVault = await keyVault.UpdateAsync(patch);
But existingPolicies does not contain any elements :(

Hi @trygvelo you could try this code. Let's say there's already an instance keyVault :
IdentityAccessPermissions accessPermissions = new IdentityAccessPermissions()
{
    Secrets =
    {
        IdentityAccessSecretPermission.All
    }
};
KeyVaultAccessPolicy accessPolicy = new KeyVaultAccessPolicy(tenantId, ObjectId, accessPermissions);
var accessPolicies = new List<KeyVaultAccessPolicy>() { accessPolicy };
var updateProperties = new KeyVaultAccessPolicyProperties(accessPolicies);
var updateParameters = new KeyVaultAccessPolicyParameters(updateProperties);
keyVault.UpdateAccessPolicy(AccessPolicyUpdateKind.Add, updateParameters);

The ObjectId could be either a user account or a managed identity.

Hi @cxznmhdcxz
Thanks, that works exactly as expected!

Hi @trygvelo you could try this code. Let's say there's already an instance keyVault :
IdentityAccessPermissions accessPermissions = new IdentityAccessPermissions()
{
    Secrets =
    {
        IdentityAccessSecretPermission.All
    }
};
KeyVaultAccessPolicy accessPolicy = new KeyVaultAccessPolicy(tenantId, ObjectId, accessPermissions);
var accessPolicies = new List<KeyVaultAccessPolicy>() { accessPolicy };
var updateProperties = new KeyVaultAccessPolicyProperties(accessPolicies);
var updateParameters = new KeyVaultAccessPolicyParameters(updateProperties);
keyVault.UpdateAccessPolicy(AccessPolicyUpdateKind.Add, updateParameters);

The ObjectId could be either a user account or a managed identity.


wrong-dtype

The previous 'fix' to force create nodata values instead of nodata bits had a /very/
unintended side effect of causing int64 output arrays instead of uint8 when
processing using dask.
This PR adds extra tests for wofs calculation using both numpy and dask arrays,
and both GA style ARD data, and USGS Collection 2 style data (needs scaling).

It works for me :)


SNOW-701257: Remove error log

Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes #SNOW-701257

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Remove the code that logs en exception as an error.

Do we still have the log of failed query + exception after this change?

Good question. I'm thinking of adding the problematic SQL to SnowparkSQLException so people can get the SQL from the exception. Simply removing the error log will make it harder to support because users only see errors but don't see the SQL.

Should we have a parameter to control this? I feel logging error is still useful when running tests, so it can be default to false


⚠ RPC server - Binance BSCT has degraded performance

In 790d4c7, RPC server - Binance BSCT ($BINANCE_BSCT_RPC_URL) experienced degraded performance:

HTTP code: 504
Response time: 25308 ms

Resolved: RPC server - Binance BSCT performance has improved in a273303.


the cmd dialog left when use the win+R to start up the vscode

VSCode Version: 1.21.3
OS Version: windows10 1803 17134.81

Steps to Reproduce:

press Win + R to Run
type "code" and enter
the vscode start-up normally, but the cmd dialog left there, which I expect to disappear automatically

PS.
I once thought that there maybe some process prevent the cmd to exit when running "code". So I do the following step:

start-up the cmd
type "code" and enter
nothing block the cmd, I can type another sentence immediatly

Does this issue occur when all extensions are disabled?: Yes

I cannot confirm this issue, the cmd closes on my machine as expected. Addressing the first video: there is something blocking the cmd from exiting, otherwise it would close / show something similar to the 2nd video. The second video is expected behavior. (similar to starting notepad etc.)

@Heldenkrieger01
I try again on my conputer. This time I not only test the start-up of vscode but also the notepad. The Following is the screen-capture:

When start the vscode, there is still something block the cmd from exiting. But the notepad doesn't have this problem.

I will do this test on other computers later.

I do the test on another two computers which run windows 10. And I reproduce the bug.
Which operate system do you use? @Heldenkrieger01

I get EXACTLY the same problem and this has happened throughout a whole year. Used system versions 1803 and 1809.
Something may help: system default language setting: Chinese (simplified), since that's also on @sigmarising 's system.

This is still happen in version 1.37 and 1.37.1, Windows 1903. Well my Windows language is also Chinese Simplified.
I will provides some details here:

double click code.cmd in the bin folder, the Command Prompt will not automatically exited.
Close VS Code will close the Command Prompt but close command prompt will not close the opened VS Code.
If you delete "@echo off" in the bat file you will see the Command Prompt is blocked at "endlocal" command. In fact it always blocks at the last command.
If you already have VS Code running, the first point is not true now, the bat will open a new window and exit normally.

duplicate: #66750

It's 2020 and this issue is still present on a cleanly installed, US-English Windows 10 Build 2004, on a clean, freshly installed VSCode build.
One thing I noticed is that if you run "code" again, with an already open iteration of VSCode, the cmd window will disappear.


1.6 Different row sizing (rows of varying height)

Row height is determined by the height of its cell contents.
Functionality of <iron-list> will be used to allow different heights of rows.

A demo example should be provided.

Done.


Add Content-Type to sigv4 header allowlist

Will need to keep in sync with these latest changes here https://github.com/grafana/grafana/pull/30115

Originally posted by @wbrowne in https://github.com/grafana/grafana-aws-sdk/issues/4#issuecomment-757750871
Fixes grafana/grafana#32548

Great catch @Madh93! Obviously slipped my mind 🤦


feat: v0.2.0

Feature

New creator function createMongoDBDataAPI

Fix

fix #1
#3 Adjust typescript generics
#4 Disable parameter inference

@maxfi Hi! there

Hi @surmon-china and @lzfee0227. In this release, shall we also include:

https://github.com/surmon-china/mongodb-data-api/issues/2
A BSON parser?

If so I'll create PRs for both tomorrow. Thanks.

@maxfi

About #2 is of course PR welcome when we make sure there is no problem with fetch compatibility
Regarding A BSON parser, you need to describe in detail its purpose and design

We can release these features in the next major version. ❤


Add support for switching render backend at runtime

In particular, switching between OpenGL, DirectX, and whatever other APIs may be supported by gfx-rs (Mantle, Metal, ...) in the future.

I remember the game engines in the latter half of the 90's. You had Direct3D and OpenGL and even software rasterizers, and you could actually see the difference between the API's. E.g. software rasterizers would render these blocky textures, since bilinear filtering would take too much CPU, and a lot of the time vendor implementations of OpenGL vs Direct3D would differ such that for example some 3D accelerators had artefacts in OpenGL, but not in Direct3D, or things like glass materials didn't look like glass in Direct3D because it didn't support something and you could change the rasterizer in the settings. I haven't seen an engine that allows the user to change the rasterizer for over a decade now. I figure it's because things like the aforementioned artifacts are all but gone now, so I'm wondering: what would be the use in having gfx support switching rendering apis at runtime?

@eugene2k Unity supports it, for example. Though of course not at runtime. I originally opened this issue as a, "can we do this?". I am not any more sure that it's valuable as I was when I opened it :) One thing that might be nice is shipping a single binary for all the various APIs, which could be quite useful - Vulkan where it's supported, OpenGL or DX11 where it's not.
What is clearly out of scope is changing the backend of a currently-running renderer - you'll definitely need to tear it down and build it back up with the new API in place. I'm not sure whether and to what extent windowing APIs support reusing windows in this way (the biggest concern really being Windows, which is where multiple APIs are usually present, though in the future I guess it will be OpenGL vs Vulkan on Linux, or OpenGL vs Metal on OS X/iOS).

Hmm... I didn't think about api switching in that context, but it should be possible without runtime switching, shouldn't it? Theoretially all you need is to create two (or more) renderers instead of one, place them into separate dlls, and choose between them at runtime, before initialization.

Yes, that's a fine theory, but that's not how it works out in Rust, especially how we use generics. You need to have everything going through the same ABI or otherwise via trait objects, which looks exactly the same as runtime switching.

If I may offer my use case and experience, we desperately need this feature:

One thing that might be nice is shipping a single binary for all the various APIs, which could be quite useful - Vulkan where it's supported, OpenGL or DX11 where it's not.

We are trying to reimplement our visualization front-end for long-running distributed computing applications and our end-users are generally non-programmers or at least not keen enough to configure and build the application such that it fits their specific machine. We definitely need the ability to select the particular implementation we'd like to use at runtime. Being able to switch that backend at runtime would be nice but being able to deploy a single binary for each platform and allow the end user to specify rendering backends via config file is a serious must.

@dkushner it all comes down to narrowing down the part of the program that knows about different backends. E.g. it may just be your main() function doing:
match config.graphics_backend {
  Backend::OpenGL3 => {
      gfx_window_glutin::init(...);
      gfx_device_gl::init(...);
      run(window, device, factory);
  },
  Backend::Direct3D11 => {
      gfx_window_dxgi::init(...);
      gfx_device_dx11::init(...);
      run(window, device, factory);
  },
}

Another (simpler) solution is to use config flags, like Amethyst does. It used to have enums for some of the types, where enumeration values are basically backend versions of the same concepts (GL Device, D3D11 Device, etc), but this approach turned out to be pretty verbose.

I did a PoC for an additional backend some time ago, which would handle runtime dispatching.
https://github.com/msiglreith/gfx-frontend/blob/master/src/lib.rs
An application would then only interface this backend or it could be, additionally, coupled with compile time switching as done in Amethyst.

@kvark, oh yeah I'm up to here at least, my issue is mostly the fact that I need a service like RenderManager to be in charge of creating GPU-bound resources at any arbitrary time over the lifetime of the application. For this reason, RenderManager needs to hold a reference to the Factory and Device but these traits cannot be trait objects, so I'm thinking the absolute only way to accomplish this is to similarly type paramaterize the RenderManager implementation? Is my thinking correct?
Basically, it's like your example but instead of passing the device and factory off to a run() I need to store it in a struct.

With LL transition we are getting more traits and associated types. We are full on the generic backend API, and providing run-time switching is escaping the scope. If that's badly needed, there are 2 ways to achieve it that I can think of:

Enclose render-dependent code into a separate executable (or dynamic library). For example, original Unreal shipped with a launcher app that allowed you to configure the settings and choose a graphics backend. Technically, the game could have multiple executables for different backends, and I think some of the games do.
Wrap every type into an enum at the type level like @msiglreith proposed. This is rather tedious, and I believe should be explored from the point of view of procedural macros or compiler plugins, generating all the boilerplate automatically.

On this note, I believe the issue can be closed.


implement jekyll layout for jupyter lesson

This was suggested in #26 . I think teaching this way will be more structured, and more easily timed correctly.
The first exposure to a notebook workflow is now the compute-pi idea from @rkdarst , while the word-count exercise is instead an exercise.
Left to do:

adjust timing for each episode
include final exercise on creating a notebook repository on GitHub and connecting it to mybinder, and add a mybinder batch to the README

the resulting lesson can be viewed at https://wikfeldt.github.io/jupyter/

Looks very good! The one question I was asking myself is whether the exercises should not be separate episodes. Also the timing is too optimistic in some episodes. One suggestion I will make on our group chat is to show episode lengths instead of episode start times in the overview.

thanks for checking. So the episode timings are currently meaningless, i will update them.
I'll think about moving the exercises into the other episodes, it's not straightforward. my thinking around that is that people will have different usecases for jupyter (which is different from e.g. git that everyone uses in similar ways), so it's good to have a smorgasbord of exercises that participants can choose between

Good point about smorgasbord - I like that :-) So let's only adjust timings and then we merge. There are minor things I would like to change but I can do that as follow-up PRs.

huh, i accidentally merged the PR by pushing to this repo instead of my fork. Oh well, timing has been updated, exercises have solutions and instructor guide is updated (but should be expanded/polished).


Feature/progress enhancements

changed prop name from percentage to value
added hideValue prop
added variant prop with possible values of  'default' and 'indeterminate'
added tests

:tada: This PR is included in version 3.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


saveLayout is not serializable/deserializable

saveLayout includes entire widget objects and not just the layout:
https://github.com/phosphorjs/phosphor/blob/23b9d075ebc5b73ab148b6ebfc20af97f85714c4/packages/widgets/src/docklayout.ts#L1201-L1216
I was expecting the following capability:

User opens app
User messes with layout
App saves layout to DB
User closes app
...
User opens app
Previous layout exists in DB and its layout is restored
User messes with layout
App saves layout to DB
User closes app
...rinse and repeat...

Something more like:
    createConfig(): DockLayout.ITabAreaConfig {
      let widgets = this.tabBar.titles
          .filter(title => title.owner && title.owner.createConfig)
          .map(title => title.owner.createConfig());
      let currentIndex = this.tabBar.currentIndex;
      return { type: 'tab-area', widgets, currentIndex };
    }

I suspect the current implementation is more for a quick undo/redo type scenario?

 It wouldn't be right for Phosphor to make those decisions for you.


Touchable underlay fix

Before the highlight underlay color would always be light gray, with this patch it will adjust to whatever color the button has.

Oops PR #1 got in here too, but I guess that one is a no brainer.


Cleanup volumes tutorial

Minor fixes to style, with a focus on simplification.
QA Steps: checked rendered output, ran linter.
This commit addresses FFTK-2514
This commit applies to FFTK-2510
This commit applies to FFTK-988
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Cleanup and simplification
Checklist
Optional. Add a 'x' to steps taken.
You can fill this out after opening the PR. "Did I..."

[x] Run spelling and grammar check, preferably with linter.
[x] Avoid changing any header associated with a link/reference.
[] Step through instructions (or ask someone to do so).
[x] Review for wordiness
[x] Match tone and style of page/section.
[x] Run make linkcheck.
[x] View HTML in a browser to check rendering.
[x] Use semantic newlines.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Comments
Any thing else that a maintainer/reviewer should know.
This could include potential issues, rational for approach, etc.

Docs for f457a50f513beff0053c6a565b61a6fa39eb0372 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2250/docs/artifacts/html/index.html

Docs for 9c1a06cf903b71c872d845be92b025d65934f6bf are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2257/docs/artifacts/html/index.html

Docs for b74688bf28ba328eeb952adfa3862a4c6c5886d3 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2260/docs/artifacts/html/index.html


Create Pages “test-page”

Automatically generated by Netlify CMS

👷 Deploy Preview for cute-twilight-a58b68 processing.

Name
Link

🔨 Latest commit
32fc48596669eff4b21edfa0a0d572481e6c54cb

🔍 Latest deploy log
https://app.netlify.com/sites/cute-twilight-a58b68/deploys/62b432dcdd48cb0008a58c83


Unknown error when move inconsecutive columns

Describe the bug
When I tried to move inconsecutive columns, "unknown error" had occurred.
To Reproduce
Steps to reproduce the behavior:

Import & wrangle sales_2011_01.txt
Select column3, column5 with cmd-clicking.
Select move from the column header context menu.
Select first.
See error.

Expected behavior
More intuitive error message should come out, so that user can know what made wrong.

This has been resolved.

Done.

Integrated test done.


[Feature Request]: Animated Album Art is bad.

Choose the extension for feature request.
Full Screen Mode
📝 Provide a description of the new feature
ngl, just really not a fan of it. its just a giant spinning album cover, and it it shows off the bottom right corner (my friends tab) of spotify while spinning. as a general concept, it looks pretty silly.
i was expecting it to be some sort of automatically generated animated colour palette. one that matches the palette of the album artwork, or even just applying some swirly blend filters to the album art background would be cool.
➕ Additional Information
No response

You don't have to use the feature if you don't like it., it's experimental for a reason. :)
Next time maybe don't share feedback by being so rude and condescending.


how to create a self-signed server certificate

I can build this framework but my question is how I can create a self-signed server certificate? Appreciate if anyone could share some snippet code

Sorry this project is for building OpenSSL for iOS, not for OpenSSL issues. Please refer to the OpenSSL Mailing List or Stackoverflow.


Use new api to save RNG state every time an AES key is generated

"seifnode" module has a new api to save the RNG state (https://github.com/paypal/seifnode/issues/14)

Feature has been added with #2


Support MVE on ARMv8.1-M

Is your enhancement proposal related to a problem? Please describe.
The Zephyr ARMv8.1-M port currently does not support the M-Profile Vector Extension (MVE; aka. Helium).
Describe the solution you'd like
Add the ARM arch configurations for the M-Profile Vector Extensions.
Additional context
The CMSIS-DSP integration can make use of the MVE instruction set on the supported processors (e.g. Cortex-M55) once this feature is implemented:
https://github.com/zephyrproject-rtos/zephyr/blob/f3435b75efbfd7614ff873f8bfa90a031d6d27e8/modules/Kconfig.cmsis_dsp#L639-L663

cc @ioannisg @microbuilder @galak


Drop Python 3.7/3.8 support; add Python 3.12

Sibling of https://github.com/LiberTEM/LiberTEM/pull/1607
Also use tox-uv for installing packages in CI
Contributor Checklist:

[ ] I have added or updated my entry in the creators.json file
[ ] I have added a changelog entry for my contribution
[ ] I have added/updated documentation for all user-facing changes
[ ] I have added/updated test cases

Reviewer Checklist:

[ ] /azp run libertem.libertem-live-data passed

/azp run libertem.libertem-live-data

/azp run libertem.libertem-live-data

tox-uv failures are due to our sdist name being completely wrong. Citing from https://packaging.python.org/en/latest/specifications/binary-distribution-format/#escaping-and-unicode, which is referenced from https://peps.python.org/pep-0625/#specification:

As the components of the filename are separated by a dash (-, HYPHEN-MINUS), this character cannot appear within any component. [..]

The issue can be reproduced by running python -m build --sdist in the LiberTEM-live source tree.
Next step would be finding out why this is the case, how that name is generated, if we are at fault or if this is an upstream bug (setuptools?) etc.

/azp run libertem.libertem-live-data

/azp run libertem.libertem-live-data

/azp run libertem.libertem-live-data

/azp run libertem.libertem-live-data

/azp run libertem.libertem-live-data


Build for multiple versions of urdfdom

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

I am not sure what's going on with the one Windows build. Seems like a mismatch of debug/release in bullet?

The actual error is actually this:
CMake Error in dart/utils/urdf/CMakeLists.txt:
  Imported target "urdfdom" includes non-existent path

    "include"

  in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:

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

  * An install or uninstall procedure did not complete successfully.

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

Any ideas how to fix that in the urdf feedstock?

Related: https://github.com/ros/urdfdom/issues/140

@traversaro - I think you are the cmake Windows expert, any clue? :)

No need because of conda-forge/urdfdom-feedstock#16


Added progress bar and fixed bugs

PR Type
[Feature | Fix | Documentation | Other ]
Short Description
Added an optional progress bar to basic_client. Also fixed two bugs that I found in nnunet client.
-Optional progress bar shows training progress
-Fixed a bug nnunet client would train for way too many steps per epoch
-Fixed a bug where the lr scheduler did not anneal the LR fast enough

sorry about the messy commit history. I used rebase instead of merge without knowing what I was doing.

sorry about the messy commit history. I used rebase instead of merge without knowing what I was doing.

I am also terrible at using rebase. I think my rebase success rate is like 60%. 😂


Add a fullscreen login screen

Currently, the login screen is very small. It would be nice when the login screen fills up the whole screen. Maybe only when the GPU plugin is active?
Nice to have (my dream):

From #8739, this might be made easier by leveraging the login screen code from mobile:


HtmlBlock and newline

I have the following input:
hello

    Foo

world

And in the following method:
    public void visit(HtmlBlock node)
    {
        getListener().onRawText(String.valueOf(node.getChars()), Syntax.HTML_4_01);
    }

I get the following for String.valueOf(node.getChars()):

\n
\n    Foo	\n
\n
\n

I'm concerned about the last \n. In pegdown I wasn't getting any trailing newline, which seems better IMO since this means the newline char would be issued in another Node.
What's the rationale for including the trailing newline in the HTML node?
Thanks

@vmassol, in pegdown \n inclusion was an inconsistent mess across nodes. In flexmark, the \n is included  in block nodes since they start and end on full lines. The HTML block contains the \n since it is part of the text that makes up the node.
Can you give an example where this \n would make sense to make part of the next  node? You can exclude the trailing \n if you don't need it but for rendering HTML it makes sense to render with the trailing \n so hat the next element starts on a new line.

Can you give an example where this \n would make sense to make part of the next node

At some point you need to decide when the text is no longer HTML. Imagine the following input hellow. I'm sure you'd agree that w should not be part of the HTML node and should be part of the next node, right? For me the same could apply for hello\n (i.e. \n not being part of the HTML node). This seems more logical to me to not consider it to be HTML. My rationale is that you should have the smallest possible HTML content and to the maximum consider the characters to be non-HTML. Since /n is valid as non-HTML content, it should not be part of the HTML node IMO.
WDYT?

FTM, I'll ignore the trailing new line. Thanks

@vmassol, \n is only part of HTML block. Inline HTML is a separate node and like any inline does not include the \n. Hence there is no need to ignore the \n in HTML blocks.

In flexmark, the \n is included in block nodes since they start and end on full lines

In the XWiki AST model, block nodes (as opposed to inline nodes) are always separated by 2 newlines. Thus in the XWiki AST, if you have:
hello

    Foo

world

You'd get:

Paragraph

Word ('hello')

HTML('....')
Paragraph

Word('world')

And if there was 3 new lines as in:
hello

    Foo

world

Then you'd get:

Paragraph

Word ('hello')

HTML('....')
Newline (1)
Paragraph

Word('world')

What's the rationale for ending all Block nodes with '/n'?

I'm closing this issue since I'm now trimming the node content and it's fine with me. I'm just curious to understand the rationale. Thx.

ok I guess the answer is probably that MD supports Block nodes that are not always separated by 2 new lines. Is that it?

@vmassol, in markdown blank lines are ignored other than to be used to mark a block end, at it is always 1 blank line that does that. Pegdown did the same. Blank lines were not part of the AST.
The AST has no blank lines, they are discarded during parsing. For my plugin I infer them from the gaps in the AST node ranges and the content of original source. Since each node has a range in original source and from each BasedSequence access to the original source through BasedSequence.getBaseSequence() and BasedSequence.baseSubSequence() methods it can be done with an AST walk or from any node by checking if previous node endOffset < this node startOffset, or this endOffset < next node's startOffset.
I never bothered to add the option of rebuilding the blank lines in the AST since it can be easily done with what is already there.

@vsch
how can I get inline HTML output (without \n between blocks) from a markdown string?

@Zhen-hao, HTML inline does not output \n after the element.
Can you please give me the sample of your markdown with HTML inline which causes an issue?

@vsch I realized that it was soft breaks that producing the newlines. problems have been solved. thanks a lot!


Check for dashboard readiness after cluster is ready

Issue link

Resolves #355
What changes have been made

The check for is_dashboard_ready is now performed after the cluster is ready.
Verification steps

Run cluster.up() and cluster.wait_ready() together.
See no errors.

Checks

[x] I've made sure the tests are passing.
Testing Strategy

[x] Unit tests
[x] Manual tests

Also, looks like pre-commit test failed, may need to run. black on changed files (or let the pre-commit module make the change locally)


Can't KYC

Hello,
I've been mining pi off and on for the last year or so. I only have 64 pi, but in real life I have -100 USD and I'm drowning in debt. This mere $2400 would allow me to relax while I finish my coding bootcamp (fullstack web dev) for the next 4 months.
I was wondering if it would be possible for anyone to check the KYC status for @shawazonfire and determine why my account has been flagged for scripting. I submitted two requests but didn't receive any confirmation email or response. I've never attempted to take advantage of the pi network, I've only used a low-end cellphone to mine (and I forgot to mine for quite a few months).
I would appreciate any information regarding how I can get my account verified so that I can transfer my pi.
Thank you,
Shawaz Imam

Vào Th 6, 24 thg 3, 2023 lúc 07:22 Shawaz @.***> đã
viết:

Hello,
I've been mining pi off and on for the last year or so. I only have 64 pi,
but in real life I have -100 USD and I'm drowning in debt. This mere $2400
would allow me to relax while I finish my coding bootcamp (fullstack web
dev) for the next 4 months.
I was wondering if it would be possible for anyone to check the KYC status
for @shawazonfire and determine why my account has been flagged for
scripting. I submitted two requests but didn't receive any confirmation
email or response. I've never attempted to take advantage of the pi
network, I've only used a low-end cellphone to mine (and I forgot to mine
for quite a few months).
I would appreciate any information regarding how I can get my account
verified so that I can transfer my pi.
Thank you,
Shawaz Imam
—
Reply to this email directly, view it on GitHub
https://github.com/pidiscovery/pi/issues/14, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUF5KLWUKTEEJ4IUNOCXSODW5TSMRANCNFSM6AAAAAAWF4QFVA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I don't understand what you want to talk about.

Hi, thank you for the response! I apologize for the misunderstanding - I was hoping to find any clarification regarding the status of my account (@shawazonfire) and if it's possible to determine why it has been "flagged for scripting". I saw that KYC has been opened for all pioneers, but I am unable to begin the KYC process.
It says that my account has not yet been selected for an invite slot, but the FAQ says that if an account has not been offered KYC initiation, it means that account has been flagged for scripting.
I sent some messages to customer support, but didn't get a response. I've only mined pi on my low-end cell phone, and never attempted to do anything except click the "mine" button whenever I rememebered.
Is there some other reason that I cannot initiate the KYC process? Thank you so much for your time!


CARBON-15247 - Upgrade Hazelcast version to 3.4.2

Depends on https://github.com/wso2/orbit/pull/95

Need to be merged to Carbon 4.5.0 branch since this adds API changes.


Tag foreignkey doesn't affect db.Modle.Related

As the belongs_to Foreign Key document shows, my example code below should be work. But it always notice ** invalid association ** error.
What's wrong here? Thanks for help!
package main

import(
	"log"
	"github.com/jinzhu/gorm"
	_ "github.com/jinzhu/gorm/dialects/mysql"
)

type ApplicationGroup struct{
	gorm.Model
	Name   string
}

type Application struct{
	gorm.Model
	Name string

	GroupID int
	Group   ApplicationGroup `gorm:"foreignkey:GroupID"`
}

func main() {
	db, err := gorm.Open("mysql", "root:root@tcp(localhost:3306)/dbname?charset=utf8")
	if err != nil {
		log.Fatal(err)
	}

	app := Application{GroupID: 123}
	var group ApplicationGroup
	q := db.Model(&app).Related(&group)
	log.Println(q.Error) //invalid association []
}
`

maybe you can try this
q := db.Model(&app).Related(&group, "GroupID")

Yes, this works.
But I think it should get GroupID from the gorm tags, instead of manually specified.

I ran into this too, I have to specify it manually. @jinzhu shouldn't it use the gorm:"foreignkey" tag value?

same problem @jinzhu
as per the source code:
// Related get related associations
func (s *DB) Related(value interface{}, foreignKeys ...string) *DB {
	return s.NewScope(s.Value).related(value, foreignKeys...).db
}

func (scope *Scope) related(value interface{}, foreignKeys ...string) *Scope {
	toScope := scope.db.NewScope(value)
	tx := scope.db.Set("gorm:association:source", scope.Value)

	for _, foreignKey := range append(foreignKeys, toScope.typeName()+"Id", scope.typeName()+"Id") {
		fromField, _ := scope.FieldByName(foreignKey)
		toField, _ := toScope.FieldByName(foreignKey)

		if fromField != nil {
			if relationship := fromField.Relationship; relationship != nil {
				if relationship.Kind == "many_to_many" {
					joinTableHandler := relationship.JoinTableHandler
					scope.Err(joinTableHandler.JoinWith(joinTableHandler, tx, scope.Value).Find(value).Error)
				} else if relationship.Kind == "belongs_to" {
					for idx, foreignKey := range relationship.ForeignDBNames {
						if field, ok := scope.FieldByName(foreignKey); ok {
							tx = tx.Where(fmt.Sprintf("%v = ?", scope.Quote(relationship.AssociationForeignDBNames[idx])), field.Field.Interface())
						}
					}
					scope.Err(tx.Find(value).Error)
				} else if relationship.Kind == "has_many" || relationship.Kind == "has_one" {
					for idx, foreignKey := range relationship.ForeignDBNames {
						if field, ok := scope.FieldByName(relationship.AssociationForeignDBNames[idx]); ok {
							tx = tx.Where(fmt.Sprintf("%v = ?", scope.Quote(foreignKey)), field.Field.Interface())
						}
					}

					if relationship.PolymorphicType != "" {
						tx = tx.Where(fmt.Sprintf("%v = ?", scope.Quote(relationship.PolymorphicDBName)), relationship.PolymorphicValue)
					}
					scope.Err(tx.Find(value).Error)
				}
			} else {
				sql := fmt.Sprintf("%v = ?", scope.Quote(toScope.PrimaryKey()))
				scope.Err(tx.Where(sql, fromField.Field.Interface()).Find(value).Error)
			}
			return scope
		} else if toField != nil {
			sql := fmt.Sprintf("%v = ?", scope.Quote(toField.DBName))
			scope.Err(tx.Where(sql, scope.PrimaryKeyValue()).Find(value).Error)
			return scope
		}
	}

	scope.Err(fmt.Errorf("invalid association %v", foreignKeys))
	return scope
}

the code tries to figure out the first possible field name that containing the foreign key in the following order:
the second argument when calling the function Related()
the type name of the owner or owned type, appended by Id
we can see it definitely ignores the tags foreignkey and association_foreignkey

@jinzhu
the official documentation is basically wrong:

To define a belongs to relationship, the foreign key must exists, default foreign key uses owner’s type name plus its primary key.
For the above example, to define a model that belongs to User, the foreign key should be UserID.
GORM provides a way to customize the foreign key, for example:

type User struct {
  gorm.Model
  Name string
}

type Profile struct {
  gorm.Model
  Name      string
  User      User `gorm:"foreignkey:UserRefer"` // use UserRefer as foreign key
  UserRefer uint
}

in fact, foreignkey has no use here!

Yes, there is an issue, when I design the Related method, I thought it as a method to query associations with user-specified foreign key, the auto guessing foreign key is just sugar.
It should be used like db.Related(&user, "User"), db.Model(&app).Related(&group, "Group")
But I think I am not going to fix this one,  as we dropped the Related support in v2, refer:
https://github.com/go-gorm/gorm/wiki/GORM-V2-Release-Note-Draft#association-mode

Yes, there is an issue

you should at least point it out in the documentation. so many people encountered this bug before.

Actually it is mentioned in the document, but maybe not quite clear.


IsGrasped and HoloLens 2

For HoloLens 2, this property also triggers when pinching or closing your hand.  This is described in the developer guide here: https://docs.microsoft.com/en-us/windows/mixed-reality/hands-and-motion-controllers-in-directx#cross-device-input-properties

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

ID: a0667364-0d31-b547-8b6b-fc85b72a1872
Version Independent ID: df41789d-bcb4-0e17-d761-ffaac27fa6e3
Content: SpatialInteractionSourceState.IsGrasped Property (Windows.UI.Input.Spatial) - Windows UWP applications
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.ui.input.spatial.spatialinteractionsourcestate.yml&version=GBlive&_a=contents
Product: uwp
Technology: winrt-reference
Git Login: Karl-Bridge-Microsoft
Microsoft Alias: kbridge

@caseymeekhof - Thanks for the feedback. I've updated the topic with more info.


Cosmological distance vectorization

Currently the cosmological distances can be computed passing the scale factors a1 and a2, or two lists with the same size.
We need a third option: one single scale factor (lens) and a n-size list (sources) as inputs.
@elisachisari

I think what needs to be done is to add a case similar to _vectorize_fn5 in this file: https://github.com/LSSTDESC/CCL/blob/master/pyccl/pyutils.py

I would simply insert the following code here: https://github.com/LSSTDESC/CCL/blob/46692a38afd249946ac004cec391f47fd7c34f63/pyccl/background.py#L135
if (np.ndim(a2) == 0) and (np.ndim(a1) != 0):
    a2 = np.full(len(a1), a2)

@pennalima @elisachisari fixed in #950 . Tagged you for a quick PR if you have time.


fix fix illegal import of guava's Throwables

What problem does this PR solve?
fix illegal import of guava's Throwables
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[x] No need to test or manual test. Explain why:

[x] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall


[FLINK-33346][runtime][test] Removes timeout

What is the purpose of the change
The instability is caused by the timeout. The fatal error hides the actual cause (i.e. the TimeoutException). It can be reproduced by setting the timeout of the test to 0ms. The actual cause is revealed when disabling the fatal error handling or not throwing a AssertionError in case of the fatal error.
I checked the rest of the class: The other test methods that use timeouts need it.
Brief change log

Removes timeout

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Error caused by FLINK-32523

@flinkbot run azure


Feature request: Support Access-Control-Expose-Headers

It would be great if we could define custom headers that browsers are allowed to access in config/cors.js
It's all about Access-Control-Expose-Headers.
I forked and added very basic implementation.
If you want I can make PR.
config/cors.js looks now
module.exports.cors = {

  // Stripped code for readability

  /***************************************************************************
  *                                                                          *
  * Which headers should be allowed for CORS requests? This is only used in  *
  * response to preflight requests.                                          *
  *                                                                          *
  ***************************************************************************/

  headers: 'content-type, Total-Count',

  /***************************************************************************
  *                                                                          *
  * Which headers in CORS response should be allowed for borwser? This is    *
  * only used in response to preflight requests.                             *
  *                                                                          *
  ***************************************************************************/

  exposeHeaders: 'Total-Count'

};

My solution: https://github.com/hinok/sails/commit/81ed85ec914c542afd8e9fcac3472c044ea11253
@gaprl mentioned that earlier: https://github.com/balderdashy/sails/issues/2146

I like your idea @hinok. We would love you to write a pull request of course! Thank you for helping us making Sails a better framework.

+1

@loicsaintroch https://github.com/balderdashy/sails/pull/2712

@ArtworkAD Good, I didn't have time to write tests for that feature but I'm glad that someone made PR ;)


inject Fastly annotations if Fastly operator is present on the cluster

We have the Fastly Operator now available on k8s clusters. When it is present, all client domains should route via the CDN, so we need to inject the necessary annotations to ingresses for this.
fastly.amazee.io/watch
fastly.amazee.io/service-id
We would need to store the service-id along with the cluster information in the API. Also these settings need to be able to be overridden at the project or environment level too, pulling in a customer provided api-secret-key from a BUILDTIME env var

The first part of this is merged in #2457
The lagoon-builddeploy controller has had the addition merged https://github.com/amazeeio/lagoon-kbd/pull/32, but hasn't been released yet.

The first part of this is merged in #2457
The lagoon-builddeploy controller has had the addition merged https://github.com/amazeeio/lagoon-kbd/pull/32, but hasn't been released yet.


The electric-sql generate --proxy flag is broken in<EMAIL_ADDRESS>

It just creates a url from the other possible env variables resulting in something like: Proxy URL: postgresql://prisma:proxy_password@localhost:65432/garmin-app

actually, this means it doesn't work anywhere. I was testing locally and it seemed like it was working but then I realized it was because it was connecting to a local running electric process.

Hey @KyleAMathews
Thanks for the report, yes there's a bug in the generate command. --proxy should be overriding the env vars and defaults but isn't. I will have a PR fixing this ready today.
Until then you can use --pg-proxy-host, --pg-proxy-port etc to override individual parts. To see the full list run npx electric-sql help generate.

Thanks!


Update build scripts

This pull request addresses the installation issue discovered by @vmagnin.  The issue was traced back to the CMake scripts in the forimage library.  I have submitted a pull request fixing that issue.  Once that pull request is merged into main, this request should complete the fix.

Thanks a lot! The installation seems OK:
[100%] Built target forcolormap
Install the project...
-- Install configuration: "Release"
-- Installing: /usr/local/lib/libforimage.a
-- Up-to-date: /usr/local/include
-- Installing: /usr/local/include/forimage.mod
-- Installing: /usr/local/include/forimage_parameters.mod
-- Installing: /usr/local/include/lut.mod
-- Installing: /usr/local/include/pnm.mod
-- Installing: /usr/local/include/forcolor.mod
-- Installing: /usr/local/lib/cmake/forimage/forimageTargets.cmake
-- Installing: /usr/local/lib/cmake/forimage/forimageTargets-release.cmake
-- Installing: /usr/local/lib/cmake/forimage/forimageConfig.cmake
-- Installing: /usr/local/lib/cmake/forimage/forimageConfigVersion.cmake
-- Installing: /usr/local/lib/pkgconfig/forimage.pc
-- Installing: /usr/local/lib/libforcolormap.a
-- Up-to-date: /usr/local/include
-- Installing: /usr/local/include/matplotlib_colormaps.mod
-- Installing: /usr/local/include/colormap_parameters.mod
-- Installing: /usr/local/include/forcolormap_info.mod
-- Installing: /usr/local/include/forcolormap.mod
-- Installing: /usr/local/include/miscellaneous_colormaps.mod
-- Installing: /usr/local/include/scientific_colour_maps.mod
-- Installing: /usr/local/lib/cmake/forcolormap/forcolormapTargets.cmake
-- Installing: /usr/local/lib/cmake/forcolormap/forcolormapTargets-release.cmake
-- Installing: /usr/local/lib/cmake/forcolormap/forcolormapConfig.cmake
-- Installing: /usr/local/lib/cmake/forcolormap/forcolormapConfigVersion.cmake
-- Installing: /usr/local/lib/pkgconfig/forcolormap.pc

Outstanding!

Thanks, @jchristopherson.
I have tested the ForImage examples and ForColormap examples, as well as the ForColormap test, and they function correctly on my system!


fix(cert-manager): Upgrade cert-manager to v1.5.3

Fixes #276

Thanks Janelle! Would you mind updating the cert-manager version in the README as well? It should be 1.3.0 or newer.


Cooperative multi-task / event loop with co_await

I'd like to create something like an event loop that schedules tasks and do some logic when a task (or sub-task) is co_awaited.
Basically I want something like sync_wait that runs in a loop running just a piece of a task (until its next co_await).
For example this one is a code working, but not given what I want:
#include <cppcoro/task.hpp>
#include <cppcoro/sync_wait.hpp>
#include <iostream>

cppcoro::task<int> f2()
{
	std::cout << "f2.1\n";
	co_return 2;
}

cppcoro::task<int> f1()
{
	std::cout << "f1.1\n";
	auto co = f2();
	std::cout << "f1.2\n";
	auto r = co_await co;
	std::cout << "f1.3\n";
	co_return r;
}

int main()
{
	std::cout << "main.1\n";
	auto co = f1();
	std::cout << "main.2\n";
	auto r = sync_wait(co);
	std::cout << "main.3: " << r << '\n';
	return 0;
}

I want something like this:
int main()
{
	std::cout << "main.1\n";
	auto co = f1();
	std::cout << "main.2\n";

	auto r = start_task(co);
	do
	{
		std::cout << "main loop\n";
	} while (r.continue());

	std::cout << "main.3: " << r << '\n';
	return 0;
}

Can I achieve this with this library?
I need that all the code run in the same thread.

I have the same question.  Looking for a way to have coroutinues run atop an event loop.  Some events may be ui-event handles, other events may be the signal to resume a suspended coroutine.
The closest concept I see in the current api is to have a blocking call use an single_consumer_event to suspend itself, and then the resuming event to call .set() to continue it.  But how, from the event loop we start a task until it suspends, and then get control back to process the next message, I haven't figured out.


update the Cloud 2.0 SDK to use the latest spec

Rename "shared" messages and fields back to "serverless".

NOTE: this is dependent on an open PR in the managed-service repo. It's purely mechanical, based on the openapi.json spec generated there.


Add equivalent definitions of star_lang and use induction

This is still a draft: the build is failing, because I slightly changed the "main" definition of star_lang.
If you agree that this definition of star_lang is best (I've added an explanation in the code; happy to discuss), then we can go through the trouble of adapting the proofs that were using the old version of star_lang and are now failing. (It will not be difficult to change: it's just a matter of rearranging some stuff.)

We can fix all the effected proofs on Saturday and talk about how you proved the equivalent star definitions, if that works for you.

I think you forgot to push StarLang.v

Thank you, this looks great :)


Clean Up (dev)-dependencies

We should have another look at the dependencies and devDependencies to clean up and categorize.

see #22

@deniaz what do you think about this: https://github.com/namics/generator-nitro/commit/673b68778b7e085a5df87acde1cdb81a277080e5

I'll close this. But if you think this needs more work don't be afraid to open the task again.


fix: rename maven-settings-picker to gav-picker

This PR was deployed with mocked data in the following URLs for your appreciation:

Storybook: https://fabric8-launcher-frontend-pr-1330-storybook.surge.sh


Support bulk operations extension methods on DbSet instead of just DbContext

The standard EF Core AddRange() and RemoveRange() methods are available on either the DbSet or DbContext classes. Is there a compelling reason to not offer both interfaces for bulk operations as well?

Not particularly.
Methods naming are Insert, Read, Update, Delete so CRUD convention, except Create->Insert.

Sorry, I don't really mean the naming of the methods - I mean that because BulkInsert() is available on DbContext it should be available on DbSet as well. This makes the bulk operations consistent with the standard EF operations which can be invoked on either object.
So I should be able to do:
dbContext.BulkInsert(entities);
dbContext.BulkDelete(entities);

As well as:
dbContext.SomeTable.BulkInsert(entities);
dbContext.SomeTable.BulkDelete(entities);

Would you accept a PR to add this interface?

Any further comment? Here's an example from EF core itself, on AddRange():
https://github.com/dotnet/efcore/blob/ec3df8fd7e4ea4ebeebfa747619cef37b23ab2c6/src/EFCore/Internal/InternalDbSet.cs#L289-L290
https://github.com/dotnet/efcore/blob/ec3df8fd7e4ea4ebeebfa747619cef37b23ab2c6/src/EFCore/DbContext.cs#L1497-L1502

Looking into this.  I'd also like to be able to access Bulk* methods from a DbSet, rather than via DbContext.
This is because we split our DataContext collections across repositories, where each Repository class has a DbSet that it performs operations (including bulk operations) on.

Doesn't look promising.  The static methods exposed by DbContextBulkExtensions.cs use a DbContextBulkTransaction which depends on DbContext to function.
I had a go at adding a new DbSetBulkExtensions class, you'd have to just pass in the context anyway making it redundant/pointless to implement.

Does anything here work? https://stackoverflow.com/a/50864373

Library already has it's method for getting DbContext.
So implementation could be like this (example for one method):
public static class IQueryableBulkExtensions
{
    public static void BulkInsert<T>(this IQueryable query, IList<T> entities, BulkConfig bulkConfig = null, Action<decimal> progress = null, Type type = null) where T : class
    {
        var context = BatchUtil.GetDbContext(query);
        context.BulkInsert(entities, bulkConfig, progress, type);
    }

    / ...
}


sunwait: fix darwin build

Motivation for this change
ZHF: #122042
cc @NixOS/nixos-release-managers
Hydra failure: https://hydra.nixos.org/build/142959762/nixlog/1
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build sunwait

Result of nixpkgs-review pr 123560 at 755fa6d3 run on aarch64-linux 1

1 package built successfully:

sunwait

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/misc/sunwait/default.nix:16:3:
   |
16 |   installPhase = ''
   |   ^


New version: KNITRO v0.7.3

Registering package: KNITRO
Repository: https://github.com/JuliaOpt/KNITRO.jl
Created by: @frapac
Version: v0.7.3
Commit: a444f426fe038c0426f37ca470679becfd5d6daa
Reviewed by: @frapac
Reference: https://github.com/JuliaOpt/KNITRO.jl/issues/131#issuecomment-546877828

Your new version pull request does not meet all of the guidelines for automatic merging.
Specifically, your pull request does not meet the following guidelines:

I was not able to install and import the package. See the Travis logs for details.

If you want to prevent this pull request from being auto-merged, simply leave a comment.
(If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.)

It is important to note that if your pull request does not meet the guidelines for automatic merging, this does not mean that your pull request will never be merged. It just means that your pull request will require manual review by a human.

These guidelines are intended not as requirements for packages but as very conservative guidelines, which, if your new package or new version of a package meets them, it may be automatically merged.

On a separate note, I see that you are registering a release with a version number of the form v0.X.Y.
Does your package have a stable public API? If so, then it's time for you to register version v1.0.0 of your package. (This is not a requirement. It's just a recommendation.)
If your package does not yet have a stable public API, then of course you are not yet ready to release version v1.0.0.

[noblock]

[noblock]
I have some trouble figuring out why the checks are not successful...

Building KNITRO → `/tmp/jl_wMsS95/packages/KNITRO/BbXEV/deps/build.log`

┌ Error: Error building `KNITRO`:

│ ERROR: LoadError: Unable to locate KNITRO installation, please check your enviroment variable KNITRODIR.

│ Stacktrace:

│  [1] error(::String) at ./error.jl:33

│  [2] top-level scope at /tmp/jl_wMsS95/packages/KNITRO/BbXEV/deps/build.jl:43

│  [3] include at ./boot.jl:328 [inlined]

│  [4] include_relative(::Module, ::String) at ./loading.jl:1094

│  [5] include(::Module, ::String) at ./Base.jl:31

│  [6] include(::String) at ./client.jl:431

│  [7] top-level scope at none:5

│ in expression starting at /tmp/jl_wMsS95/packages/KNITRO/BbXEV/deps/build.jl:42

└ @ Pkg.Operations /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.2/Pkg/src/backwards_compatible_isolation.jl:647

But maybe this package is not able to be installed automatically?

Indeed: there is a dependence to an external commercial solver. If the binaries of this solver are not found in the system, then the build fails.

Thanks for your help!


TryValidateModel and TryUpdateModel don't revalidate the model if model is already validated after model binding

If I have an action
public object UpdateModel(Product product)
{
   product.Name = null;
   TryValidateModel(product)
 }

TryValidateModel(product) does not re-validate model since it is already validated after model binding.
We should clear model state errors and change the ValidationState to Unvalidated in ModelStateDictionary for TryValidateModel and TryUpdateModel

Fix checked in - 7b18d1d3f1f04ba8952b13fd2b0f903e684bf78d


"Max Hover Light Count (2)" Exceeded Warning when playing Scene with Default HL2 profile

Describe the bug
Steps to reproduce the behavior:

Import MRTK 2.5
Configure blank scene with MRTK Default HoloLens 2 profile
Hit play
Bring up hand

Observer following warning in console window:
Max hover light count (2) exceeded.
UnityEngine.Debug:LogWarningFormat(String, Object[])
Microsoft.MixedReality.Toolkit.Utilities.HoverLight:AddHoverLight(HoverLight) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Utilities/StandardShader/HoverLight.cs:103)
Microsoft.MixedReality.Toolkit.Utilities.HoverLight:OnEnable() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Utilities/StandardShader/HoverLight.cs:55)
UnityEngine.Object:Instantiate(GameObject, Transform)
Microsoft.MixedReality.Toolkit.Input.BaseControllerPointer:SetCursor(GameObject) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/SDK/Features/UX/Scripts/Pointers/BaseControllerPointer.cs:84)
Microsoft.MixedReality.Toolkit.Input.BaseControllerPointer:OnEnable() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/SDK/Features/UX/Scripts/Pointers/BaseControllerPointer.cs:150)
Microsoft.MixedReality.Toolkit.Input.LinePointer:OnEnable() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/SDK/Features/UX/Scripts/Pointers/LinePointer.cs:93)
Microsoft.MixedReality.Toolkit.Teleport.TeleportPointer:OnEnable() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/SDK/Features/UX/Scripts/Pointers/TeleportPointer.cs:136)
Microsoft.MixedReality.Toolkit.Teleport.ParabolicTeleportPointer:OnEnable() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/SDK/Features/UX/Scripts/Pointers/ParabolicTeleportPointer.cs:33)
UnityEngine.Object:Instantiate(GameObject)
Microsoft.MixedReality.Toolkit.Input.BaseInputDeviceManager:CreatePointer(PointerOption&) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Providers/BaseInputDeviceManager.cs:303)
Microsoft.MixedReality.Toolkit.Input.BaseInputDeviceManager:RequestPointers(SupportedControllerType, Handedness) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Providers/BaseInputDeviceManager.cs:223)
Microsoft.MixedReality.Toolkit.Input.BaseInputSimulationService:GetOrAddControllerDevice(Handedness, ControllerSimulationMode) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Providers/InputSimulation/BaseInputSimulationService.cs:139)
Microsoft.MixedReality.Toolkit.Input.BaseInputSimulationService:UpdateControllerDevice(ControllerSimulationMode, Handedness, Object) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Providers/InputSimulation/BaseInputSimulationService.cs:78)
Microsoft.MixedReality.Toolkit.Input.InputSimulationService:LateUpdate() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Providers/InputSimulation/InputSimulationService.cs:385)
Microsoft.MixedReality.Toolkit.BaseDataProviderAccessCoreSystem:LateUpdate() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Services/BaseDataProviderAccessCoreSystem.cs:67)
Microsoft.MixedReality.Toolkit.<>c:<LateUpdateAllServices>b__69_0(IMixedRealityService) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Services/MixedRealityToolkit.cs:1035)
Microsoft.MixedReality.Toolkit.MixedRealityToolkit:ExecuteOnAllServicesInOrder(Action`1) (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Services/MixedRealityToolkit.cs:1109)
Microsoft.MixedReality.Toolkit.MixedRealityToolkit:LateUpdateAllServices() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Services/MixedRealityToolkit.cs:1035)
Microsoft.MixedReality.Toolkit.MixedRealityToolkit:LateUpdate() (at Library/PackageCache/com.microsoft.mixedreality.toolkit.foundation@2.5.0-preview.20200910.3/Core/Services/MixedRealityToolkit.cs:700)

Expected behavior
No warnings should appear when playing empty scene configured with MRTK.
Your setup (please complete the following information)

Unity 2019.4.10f1 Personal
MRTK Version 2.5 UPM

Target platform (please complete the following information)

Run in Unity Editor, target platform was set to Standalone

Also note, this is happening with .unitypackages as well

Marking release blocker based on our policy of not introducing any new warnings when playing using default MRTK profiles.

@RogPodge Was the HoverLight added as part of your teleport cursor upgrades in https://github.com/microsoft/MixedRealityToolkit-Unity/pull/8386? I don't remember having one on the cursor before

Yeah I was looking too, looks like the HoverLight is getting pulled in when the TeleportPointer starts up with hands (i.e. in editor simulation hands).

We could increase the hover light capacity (though this I think reduces our perf). It's possible that this might be a spurious warning (i.e. if the teleport pointer isn't visible then I assume there's no hover light involved, same if the teleport pointer is causing the other things to not show).

yeah part of the teleporter pointer improvements involved adding the Hoverlight to better illuminate the teleport area, I'm not sure what the best call to make here is with regards to perf vs. improving the user experience. I'd lean towards it being ok from a performance perspective, since the lights aren't active unless you are teleporting, and upping the count to say 4 should be alright.

We already seem open to the user manually increasing the number of hoverlights used here, so I feel like it should be ok to bump this number up a bit, thoughts?
https://microsoft.github.io/MixedRealityToolkit-Unity/Documentation/Rendering/HoverLight.html

@Cameron-Micka for HoverLight feedback

This is a warning that more hover lights are active in the scene than the shader will accept. This could be likely spurious as @wiwei mentions if it's impossible to have two hands active while teleporting?
If this is a feature that is needed, it would align with another request: https://github.com/microsoft/MixedRealityToolkit-Unity/issues/5888 I think I have a solution in mind that would keep performance the same for existing setups and allow users to add more lights if needed. (The only drawback would be more shader permutations, aka increase build times and memory footprint. But, I don't anticipate it would be drastic.)

@Cameron-Micka is it ok to up the limit to 4 for now so the user doesn't encounter the warnings? This is to unblock the 2.5 release. I don't really want to remove the hover lights from the teleport cursors because who knows if we'll ever get around to remembering to add them back after you make your performance changes.

Hi @RogPodge, I did a quick test to see how many additional instructions would result from increasing the count from 2 to 4:
 // 2 Hover Lights Max
 // Stats for Vertex shader:
 //        d3d11: 23 avg math (8..88)
 // Stats for Fragment shader:
 //        d3d11: 62 avg math (1..145), 0 avg texture (0..1)

 // 4 Hover Lights Max
 // Stats for Vertex shader:
 //        d3d11: 23 avg math (8..100)
 // Stats for Fragment shader:
 //        d3d11: 72 avg math (1..161), 0 avg texture (0..1)

You can see the average case introduces 10 extra pixel shader instructions (and 16 in the worst case). It's hard to quantify these numbers since I bet it would introduce little performance loss in our example levels (since many of them are rather simple). But, might have adverse effects in apps which are already close to being GPU bound.
Afaik we don't have any test harnesses to stress test changes like this (especially since our tests don't happen on device). But, they would be nice to have!
How close is the 2.5 release? Or, how soon would the change I describe above need to be committed?

I believe we're trying to release before the end of the month if I'm not mistaken  @wiwei @Cameron-Micka

I also ran into this issue when using the Unity in-editor player on the Android platform with the Hand Interaction Examples scene. I had the Default XR SDK Profile enabled when the issue occurred.

Just opened a PR to (hopefully) address this: https://github.com/microsoft/MixedRealityToolkit-Unity/pull/8596


Does not work with Firefox: "No data received"

What OS?

Code Server: Linux Ubuntu 22.04 with Docker
Client: Windows 11 with Firefox as web browser

Description
When pressing "Edit csv" on Firefox an error "No data received" appears at the location the table containing the CSV data is supposed to appear. On Google Chrome everything works as expected. Using Firefox is essential for my development workflow and so switching to Google Chrome is unfortunately not an option. I would love to use this awesome extension as I constantly have to edit CSV files. I even tried on a fresh installation of Windows with the latest version of Firefox and its default browser configurations and experienced the same issue. Please let me know if there is anything I can provide to help you debug this issue.
Expected behavior
To see the CSV file represented as table when using Firefox as I do when using Google Chrome.
Steps to reproduce

Create a new docker container using the lscr.io/linuxserver/code-server:latest image.
Make sure to setup a valid SSL certificate or use a HTTPS reverse proxy. This is required for most extensions (including this one) to work and expected behavior.
Install "Edit csv v0.8.1 from janisdd" from the default extension store (https://open-vsx.org/extension/janisdd/vscode-edit-csv)
Open any small CSV file and as you can see it won't work with Firefox while it works without any issues using Google Chrome.

Screenshot
Here a screenshot of this issue which also includes the inspector output containing all requests and errors occurring after pressing the "Edit csv" button on Firefox. Please let me know if there is anything else I can provide.

Well, works for me ;D

But seriously, I found in https://stackoverflow.com/questions/62475212/securityerror-cssstylesheet-cssrules-getter-not-allowed-to-access-cross-origin that there might be some firefox extension injecting some css files. As of the last extension update I need to iterate all css files and find two specific files (dark.css and light.css) to change some css variables. From my understanding the extension script is not allowed to access css styles from firefox plugins/extensions/add-on.
Do you have any firefox plugins installed? If so, can you give me the link to reproduce this issue?

I don't have any Firefox plugins installed. I tried it on a clean installation of Windows 11 with a clean installation of Firefox and still got the issue. I even tried Firefox on a fresh installation of Ubuntu 22.04 and experienced the same issue there as well. I also tried disabling CORS using the CORS Everywhere extension which also wasn't successful and still resulted in a cross-origin stylesheet exception. I don't experience any Firefox specific issues with any other VSCode extensions. The reason it worked you might be because you connected over localhost. Most modern web browsers disable some security features when connected over localhost.

I tried to create the simplest possible docker-compose file to replicate this issue. I came up with the following docker-compose.yml using which I was able to reliable replicate this issue by just installing vscode-edit-csv as only visual studio code extension. The ssl folder should contain a ssl.cer and ssl.key file for the PROXY_DOMAIN and the domain used to access the VSCode server. In my case it's a wildcard certificate for *.nico.re.
version: "3"

services:

  code-server:
    image: lscr.io/linuxserver/code-server:latest
    container_name: code-server
    hostname: code-server
    restart: unless-stopped
    environment:
      - PUID=0
      - PGID=0
      - TZ=Europe/Zurich
      - SUDO_PASSWORD=root
      - PROXY_DOMAIN=code-server.nico.re
    volumes:
      - ~/.vscode-server:/config
      - ~/ssl:/root/ssl
    ports:
      - "8443:8443"
    security_opt:
      - seccomp:unconfined
    cap_add:
      - SYS_PTRACE

With above docker-compose file the issue also occurred when connecting over localhost using an SSH tunnel to the docker host. If you are still having troubles recreating this, I could host a public VSCode server with this issue for you.

I found a temporary workaround for this issue: Disabling the setupAndApplyInitialConfigPart1 function skips the code which triggers this issue and make this extension work with Firefox.
How to skip the setupAndApplyInitialConfigPart1 function: Inside extensions/janisdd.vscode-edit-csv-0.8.1-universal/csvEditorHtml/out/util.js after the line function setupAndApplyInitialConfigPart1(initialConfig, initialVars) { add a return; statement.
There are likely very good reasons for this function to exist so disabling it might cause a lot of side effects but I haven't noticed anything too bad. At least the core features seam to all still be working without this function so disabling it until this issue is resolved is for sure a better option than not being able to use this extension at all with Firefox.

Because you are having troubles recreating this issue, I created a Linux VM which includes booth the VSCode Server and the Firefox Client so everything required to reproduce this issue is inside this VM. Start the VM with the latest version of VirtualBox and log in using the password debian, open Firefox and open the bookmarked page, click on "Edit csv" to reproduce the issue. The VSCode server starts automatically on boot and was created using the docker-compose.yml file on the desktop. While setting up this VM I also realized that SSL and PROXY_DOMAIN can be skipped using this setup due to localhost having an elevated level of trust.
Here you can get the VM to easely reproduce this issue: https://drive.google.com/file/d/1xNEUicaD-PZtpSw_QZ70Uq2w9_cbkt8x/view?usp=sharing

Yeah, I couldn't get ssl working locally with code-server...
Thx for the vm, I'm able reproduce the issue there.

The fix is relatively simple: put crossorigin="anonymous" in the link tag in the getHtml.js file
<link rel="stylesheet" crossorigin="anonymous" href="${darkThemeCss}">
<link rel="stylesheet" crossorigin="anonymous" href="${lightThemeCss}">

However, I have to verify that this does not break the extension if used normally.

The fix is relatively simple: put crossorigin="anonymous" in the link tag in the getHtml.js file
<link rel="stylesheet" crossorigin="anonymous" href="${darkThemeCss}">
<link rel="stylesheet" crossorigin="anonymous" href="${lightThemeCss}">

However, I have to verify that this does not break the extension if used normally.

Thanks a lot! I can confirm that this fixes the issue.


Deploy module "se.jbee.inject" via JitPack

Set Silk DI's version to "19.1-ea"
Generate consumer pom.xml
Add JitPack configuration file

Initial results from this branch:

Usage: https://jitpack.io/#sormuras/silk/jitpack-SNAPSHOT
Build log: https://jitpack.io/com/github/sormuras/silk/jitpack-SNAPSHOT/build.log
API documentation: https://javadoc.jitpack.io/com/github/sormuras/silk/jitpack-SNAPSHOT/javadoc

@sormuras anything keeping this from being merged?
Another question: Didn't you have the actions hooked in for PR in your fork? What is needed for that?

anything keeping this from being merged?

None. Press Play on Tape.

Didn't you have the actions hooked in for PR in your fork?

I did. They are here: https://github.com/jbee/silk/pull/87/checks ... trigger on each push to master and for each pull request.
https://github.com/jbee/silk/blob/9a0496141d027cc198823b14cc3b9cbe517a82f8/.github/workflows/bach.yml#L8-L12

What is needed for that?

Put/modify/delete a workflow configuration file in .github/workflows directory. Find the official documentation here: https://help.github.com/en/actions

Branch protection rules was what I wanted to get at. Found it and hopefully configured it now to require the checks to pass before allowing to merge.

Build just finished over at https://jitpack.io/com/github/jbee/silk/master-SNAPSHOT/build.log
Module se.jbee.inject is usable via https://jitpack.io/#jbee/silk/master-SNAPSHOT
API Documentation: https://javadoc.jitpack.io/com/github/jbee/silk/master-SNAPSHOT/javadoc