[WIP] Release ReactiveUI v8 (netstandard)

Too much time has passed since the last release (integration) so it's time to ship some code that way the maintainers get feedback from consumers. If feedback is 👍 then relax. If feedback is 👎 then do a release pretty much immediately/asap to resolve issues encountered - always be rolling forward.

[ ] Final call for merging into anything potentially breaking in this release. I'd like to see Platform Registration Splitting and Netcoreapp merged before release. Maybe a couple other bugfixes from the community as well. Need to keep this scope restricted.
[ ] Investigate compatibility for consumers who are currently using PCL's and see if we can make things less breaking - https://github.com/reactiveui/ReactiveUI/issues/1421
[ ] All breaking changes need to be re-reviewed and discussed in the comments below because they will need be added to a section at the bottom of the RELEASENOTES with information of how to migrate codebases.
[ ] Validate each platform by hand (bring back the playground projects) and ensure clean package installation. https://www.youtube.com/watch?v=HJOQqF-H6wk&t=973s
[ ] Kick off the samples project and use the alphas in this project for couple of sprints to to integration testing.
[ ] Sign (identity signing, not strong naming) our packages/assemblies via the .NET foundation service to make it impossible for rogue actors to ever masquerade as ReactiveUI.

Coverage decreased (-0.2%) to 66.635% when pulling f33b96c830e7359ea67cccc95a756af824f28ff4 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage decreased (-0.1%) to 66.75% when pulling 5d614827ddd21e95ff9ea270db713cef6595ed97 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage decreased (-0.1%) to 66.75% when pulling d9a066f5a0933f0b6374255046d69494cdf65754 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage decreased (-0.1%) to 66.75% when pulling e1d5fb608f1c1a7ff3fd7866ba4eb549858d15ba on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage decreased (-0.1%) to 66.75% when pulling 72b4921d0b60d55b795474c2f7a03918a85fb150 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage decreased (-0.1%) to 66.75% when pulling dea8a4b2bc4fedaeecdc59328c1fa27d9e90c069 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+0.1%) to 67.024% when pulling e5187bd60d77a0255e603e9205a0f4ae88b2051c on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+0.2%) to 67.056% when pulling 72531f1fce10189c03b35f518eb5ec74f3b0ae3b on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+0.2%) to 67.056% when pulling c2981d65b7b611b0b05c4a3d53390c2aca5f0a01 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+0.2%) to 67.056% when pulling e8227e80b7ffbd521099842c59219c7703a9c8a5 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+0.1%) to 67.024% when pulling c1285aa5848cd0bea6ffcef98ec1b878ff185d24 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.5%) to 70.424% when pulling ece4833e2712048851ac01e7b6285fd33681ebeb on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 0ba0916867400dc950c69b5adc7ca0a0eb56549c on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 43288ef59d8d5ffe77c2a5dd331fb1af6033c774 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling a08a185f897be49a7c579eb5ed131bdecb3d02ad on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 31bf583bf8a71094b8e2af40fe945361e9d45961 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 8d19db2ace827e5e29555bb4ec8075987b36b3d9 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 1b3a5a24c3990a9ffa90710bbdb380c7792f9269 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 6c381c1b20bb940fc4d59061bd66ade6ac1dc3fa on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling b551dfe13c21090b7529bd69539433bbe91b63fb on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+3.6%) to 70.458% when pulling 8c3a0bb6ecdd42dbb3f924d221a432057b245a13 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling ce83b07e9b731f988344147a8ec2579355d31422 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling fd1ca6830d5df9df27535c9e669924ffb768db34 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 2826dc174feaf43fc56e0ec3c2ddf76790e55502 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 3333348ac0ef467722485374bf8beeeb744cf627 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling d0ddd07df437ef02121aed2c90a1436f5994a0c3 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 095ecb268760c1f60690b6cc13cde442e8be3321 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 22b992c68daff5c263cd31815dc572ea8655ea17 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling b4bee167119ad302a174565c6123725d2b851b7a on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 0af6b0a844a240e816580c64b10956f728427f1e on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 48310adcc5217f9a0899432f7b18f0e9a4e27613 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling fe51eb5777c7bff0e1be2e90f560cbbac9ed091e on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.467% when pulling 81db6c727e0d60fca4ebeb925c94b74e856eeae0 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.454% when pulling 785340753bcb1619296127ba260ae5893f334fbc on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling dac6f7db4177c5cc4a8c50e1edd624860d02d198 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 12187bb81db445577cdeb4b2ae08a53a42e19120 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 863053c93de6b44c5be95997bd908b66d9e5833c on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 995f8508c8fd7a1314f2cc9c6a8881a66c7cf7e6 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling f67ba03f795afb2d2e2fee0834aa44434c7d7bd1 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 416b0088fc7458be057b1a47e6eae9523bc932f3 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling f3ac8192df7d7a7414813cf76f41349d1919a213 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 2a65584f4e73da472958d3e9ce5d8578748a0f4a on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling f23538577ea7e567ab372c550adebebe9fdbb7f7 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 08d9e58425174cedcbb9073ae0d8fd2b7c2cea7a on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling f3441ef83b318edfec0c18478693ae8bfb1ff3b9 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.454% when pulling 995e973acfe27890d9974a77947676f9e10b3be3 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.487% when pulling 7bf86b9db689bf7567d1aabcfde0efa525c03b24 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.497% when pulling<PHONE_NUMBER>35fde4fbba270f8665502644082be2 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.497% when pulling c289a97f75b53c509e092b9c8ad3680e6d04a020 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.464% when pulling 3ddb295f09ecd77a05d72496f7e3b08dc9ed92ff on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.497% when pulling 4aef04696bf98a2cc55b08bf1432083ec376e8cc on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.464% when pulling 83cbc4b33e8a44039f3d9d6f9d3cf9e62b3bf7ea on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.6%) to 69.497% when pulling fe73608c2f8165fd9775b2cbe105a940112b3a29 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.

Coverage increased (+2.7%) to 69.589% when pulling f94ba3fde4067b7c1fe64c90e358d57dd9557ed5 on develop into 522170ebbe6a3e0c284136a473c18ba1b82a66bf on master.


Add Powered By Fate copyright information

As this application utilises concepts from Fate RPG it would be appropriate to reference the original material in the credits/about page.
Licensing information can be found here:
http://www.faterpg.com/licensing/
At a minimum, the application should include the following text and I'd also like to include the Powered by Fate logo.

This work is based on Fate Core System and Fate Accelerated Edition (found at http://www.faterpg.com/), products of Evil Hat Productions, LLC, developed, authored, and edited by Leonard Balsera, Brian Engard, Jeremy Keller, Ryan Macklin, Mike Olson, Clark Valentine, Amanda Valentine, Fred Hicks, and Rob Donoghue, and licensed for our use under the Creative Commons Attribution 3.0 Unported license (http://creativecommons.org/licenses/by/3.0/).

Using the Powered by Fate logo requires the following text

Fate™ is a trademark of Evil Hat Productions, LLC. The Powered by Fate logo is © Evil Hat Productions, LLC and is used with permission.


Add Nautobot App migration docs for Region and Site to Location data migration

Closes: #2958
What's Changed

Added region-and-site-data-migration-guide.md in docs/plugins folder and added references to it in upgrading-from-nautobot-v1.md and version-2.0.md
TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Having a reverse migration that is a no-op can be dangerous. I would recommend providing a real reverse migration with Example App instead of the no-op. If Site were a required field, reversing migrations may be impossible with a No-op.

Having a reverse migration that is a no-op can be dangerous. I would recommend providing a real reverse migration with Example App instead of the no-op. If Site were a required field, reversing migrations may be impossible with a No-op.

It is exceedingly unlikely that we will restore their database in as-good of a state with a reverse from 2.0. If you make any data changes at all post-migration, you will be re-inventing data and relationships that didn't exist in the roll-back.

Having a reverse migration that is a no-op can be dangerous. I would recommend providing a real reverse migration with Example App instead of the no-op. If Site were a required field, reversing migrations may be impossible with a No-op.

It is exceedingly unlikely that we will restore their database in as-good of a state with a reverse from 2.0. If you make any data changes at all post-migration, you will be re-inventing data and relationships that didn't exist in the roll-back.

But, if someone tries to uninstall a plugin, and the rollbacks fail due to a missing reverse migration, that would be a concern.

Having a reverse migration that is a no-op can be dangerous. I would recommend providing a real reverse migration with Example App instead of the no-op. If Site were a required field, reversing migrations may be impossible with a No-op.

It is exceedingly unlikely that we will restore their database in as-good of a state with a reverse from 2.0. If you make any data changes at all post-migration, you will be re-inventing data and relationships that didn't exist in the roll-back.

But, if someone tries to uninstall a plugin, and the rollbacks fail due to a missing reverse migration, that would be a concern.

Very valid point! Definitely need to consider the uninstall case, but that might need to be a whole separate PR after this one?

Sure! I just want to be careful recommending noop. I have been bitten in the past with migrations I could not reverse.

Opened up an issue to track this. https://github.com/nautobot/nautobot/issues/3374

Nits only at this point.
(I just noticed the commit message on your last commit - what happened?)

Oh no, I did id in my commit message, and it looks like it did a command substitution :(

Nits only at this point.
(I just noticed the commit message on your last commit - what happened?)

Oh no, I did id in my commit message, and it looks like it did a command substitution :(

Yeah if you're going to use backticks in the commit message, make sure you wrap the message in single-quotes rather than double-quotes. :-)


Specify supported Python versions

I'm not sure if this is has already been decided, but I think that it's worth considering and specifying which versions of Python this will support.
For instance, I was about to post an issue in regards to using the abc module in the Python standard lib to implement the base abstract classes, but I don't know if the goal is to support versions of Python older than 2.6 (I'd like to hope not, but I'm a bit divorced from the reality of legacy systems).

I'm trying to make it compatible with 2.X - 3.X. I haven't used 3 so I'm just guessing. My guess is that it will be 2.6 - 3.X. I was using some 2.6 specific stuff in it earlier today. That has since been replaced, but I think it's going to happen inevitably.

The stated goals seem like 2.7 (or at the very least 2.6) and up make sense, though I'd advocate focussing on supporting 2.6 and 2.7 as legacy if possible (I don't think that it should be too hard if it's not a goal to reach too far back into Python 2.x versions).
If it's helpful, I can submit a pull request setting up a test suite that runs against both versions, but I don't want to do that until this is further out of an exploratory phase.

That would be nice, FWIW current fontTools is 2.7+.
Looking at the code, the incompatibilities are:

bPoints: tuple arguments in functions, this was dropped in Python 3 to discourage superfluous packing/unpacking.

def foo((x, y)):
    ...

may be replaced with
def foo(x, y):
    ...

which is the more py3thonic way, or:
def foo(tuple):
    x, y = tuple

Obviously, I'd advocate for the former.

you put has_key method in kerning.py and font.py but x.has_key(k) was dropped in favor of k in x (you can implement it with the __contains__ builtin)
python 3 has dropped most redundant syntax (except for string formatting, which is getting a fourth syntax f"foo"!)

Also, generally my past py23 pull requests are a good overview of what should be taken into account if that helps. We can always correct down the road of course.

Changing away from tuples in the methods relating to points will be pretty breaking for existing scripts, going with def foo(tuple) is going to break a lot fewer things.

Oh right, I missed that.

bPoints: tuple arguments in functions, this was dropped in Python 3 to discourage superfluous packing/unpacking.

Yeah, I'm removing those as I go through. The files in there are extracts from the old RoboFab and I've only worked through Font, Glyph and Contour so far. The format will be foo(tuple) for backwards compatibility and general cleanliness.

you put has_key method in kerning.py and font.py but x.has_key(k) was dropped in favor of k in x (you can implement it with the contains builtin)

I'm implementing it as __contains__ the way that defcon does. I think RoboFab predates __contains__ (!!!!) so there is a lot of legacy has_key stuff. I'm mapping has_key to __contains__ just to keep scripts from breaking.

py26 is no longer officially supported by PSF. It's very difficult to maintain support for the full range of pythons from 2.6 to 3.5 with a single code base. I'd recommend to drop support for python 2.6 now that the project is starting.

I meant 2.7. I forgot where the 2.X series stopped.

Currently we're testing 2.7, 3.4, and 3.5. That seems like a reasonable set to support, so I'm going to close this.


bug(COMPONENT): bug(mat-menu/mat-icon-button/radio-button/checkbox): Unable to reduce ripple size

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
angular material 14
Description
For a mat-menu icon or a  button of any size, the default ripple size is displayed.
In certain scenarios, we need to fit in the button in smaller UI. As the ripple effect is taking more space than the button, it is creating a UI glitch.
If we reduce the size of button, still the ripple effect is taking unwanted space and hence I had to redesign the application.
Is there any way to override the ripple size "THROUGH CSS" as the other option (density) is not working as expected for our use cases
Issue is happening after migrating angular and angular material from 14 to 15 versions.
Reproduction
[StackBlitz link:
Steps to reproduce:

(https://stackblitz.com/edit/angular-rumxmw?file=src%2Fapp%2Fbutton-overview-example.html)

Expected Behavior
The ripple effect should be relative to the button, if the button size is 20px height and width, the ripple size of the button or the mat-menu-icon should be also 20px and should be contained within the mat-button . It should not overflow within the button wrapper .
Actual Behavior
if the button size is 20px height and width, the ripple size of the button or the mat-menu-icon is still 40px and its not contained within the mat-button . It is still overflowing outside the button wrapper and causing UI issues being distorted
Environment
Angular CLI: 15.2.4
Node: 16.15.0
Package Manager: npm 8.5.5
OS: win32 x64

Browser(s): chrome, firefox etc

Angular: 15.2.4
... animations, cdk, cli, common, compiler, compiler-cli, core
... elements, forms, language-service, material
... material-moment-adapter, platform-browser
... platform-browser-dynamic, router
@angular-devkit/architect       0.1502.4
@angular-devkit/build-angular   15.2.4
@angular-devkit/core            15.2.4
@angular-devkit/schematics      15.2.4
@angular/flex-layout            15.0.0-beta.42
@schematics/angular             15.2.4
ng-packagr                      15.2.2
rxjs                            7.8.0
typescript                      4.9.5

At this time we don't support customizing the size. In the future this will be something that can be done through tokens. Although, I would be hesistant to make a button under 48px for accessibility reasons. Here is a quick article talking more about touch targets: https://web.dev/articles/accessible-tap-targets.
It seems on Stack Overflow there are a few solutions that have worked for other users.


Preparation

We might want to select specific files from previous processing.
Preparation should allow to select files and pass them directly to commands (without any intermediate operation like filtering file content) by providing just:
...
"preparation": {
    "filterFile": "*"
},
...

Tests missing for this feature.


Shadowrocket not Working.

Shadowrocket after updating to support vision flow, is not working with the shadowrocket.conf file in this repo.
For example, it will proxy google.com but it will block Instagram and youtube.
something is wrong with the config file.
Thanks

Any response or idea?
Or anyone tested this?

@hawshemi please test the new config from newest release and lemme know if that works.

Issue is fixed.
Thanks.


How do I add the Group.Read.All permission?

I don't see this as an option anywhere....

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

ID: 0a9d928d-fab5-99af-105f-86cb0520dcfa
Version Independent ID: ded494e8-1cc3-04bb-fec9-1c44e7231d06
Content: The SharePoint modernization scanner
Content Source: docs/transform/modernize-scanner.md
Product: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Hello @jmuehlenbein
the documentation provides this link about Granting access via Azure AD App-Only (recommended) which shows how to define applications in Azure AD and how to grant permissions to SharePoint. However, this application can be granted permissions to Graph as well.
In your PnP Modernization Scanner Azure AD app, click on "API permissions" in the left menu bar, and click on the "Add a permission" button. A new blade will appear. Here you choose the permissions that you will grant to this application. Choose i.e.:

Microsoft Graph

Application permissions

Group

Group.Read.All

Now, click on the blue "Add permissions" button at the bottom to add the permissions to your application, then on "Grant admin consent for Contoso".
I hope it helps to clarify your question.
Regards,
Jarbas

Group.Read.All is set in Azure AD App, you can firstly register Azure AD App:

Then set Application Permissions:

Reference:
Manage app registration and API permission for Microsoft Graph notifications


Api Error Code: -1021 Message: Timestamp for this request is outside of the recvWindow.

This issue happened just 2 days ago, is this new problem from Binance API?

Same error

i got similat error that was "timestamp for this request 1000ms ahead of server" and i just closed the visual stdio and open it again and the problem was gone.

It is using your machines time to generate a timestamp. This has to be within acceptable range of Binance's server. You just need to set your time either manually - or sync with an internet time server like nist.gov - this can be found under you clock settings in windows. Just hit update - sometimes takes a few attempts.
If it keeps failing, then utilize Binance's API method GetServerTime and compare it with your own.

Solved by disabling auto timezone on Windows 10.


Add a more informative error message for documentLoader issues.

throw new Error(`Document loader unable to load URL "${url}".`);

Addresses: https://github.com/digitalbazaar/vc-js/issues/39
@iainphd does this help?

When I saw the error I didn’t realise it was coming from lib/documentLoader.js - so something to that end would help, explaining that it needs to be customised to find the DIDdoc would be useful

From: Andrew Jones<EMAIL_ADDRESS>Sent: Thursday, February 6, 2020 9:34 pm
To: digitalbazaar/vc-js
Cc: Iain Barclay; Mention
Subject: Re: [digitalbazaar/vc-js] Add a more informative error message for documentLoader issues. (#40)
@aljones15 commented on this pull request.

In lib/documentLoader.jshttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdigitalbazaar%2Fvc-js%2Fpull%2F40%23discussion_r376094626&data=01|01|barclayis%40cardiff.ac.uk|60e585e582dc49c07cdf08d7ab4c613f|bdb74b3095684856bdbf06759778fcbc|1&sdata=XgShzEic1PXEcS9Wie3Tu5eJViV82HQJK6hJ9ugawQc%3D&reserved=0:

throw new Error(vc-js documentLoader could not resolve the ${url}. +

'You might need a custom documentLoader.');

@iainphdhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fiainphd&data=01|01|barclayis%40cardiff.ac.uk|60e585e582dc49c07cdf08d7ab4c613f|bdb74b3095684856bdbf06759778fcbc|1&sdata=SQq8qKhg%2BpLP%2FNeSfpybopAY4HbSELa%2Bws0VPeyO8Oc%3D&reserved=0 this PR is intended to help you. So what error wording would work best for you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdigitalbazaar%2Fvc-js%2Fpull%2F40%3Femail_source%3Dnotifications%26email_token%3DALKMA6OJNK2ELEWXOSADRKLRBR67FA5CNFSM4KQ7UAZ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCUS2GFY%23discussion_r376094626&data=01|01|barclayis%40cardiff.ac.uk|60e585e582dc49c07cdf08d7ab4c613f|bdb74b3095684856bdbf06759778fcbc|1&sdata=gkekxtEaSIUROqHP6FuZPz1U2TnJgykN7W3ozmElbFg%3D&reserved=0, or unsubscribehttps://eur03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALKMA6MXYMXXPIAIGKROTNTRBR67FANCNFSM4KQ7UAZQ&data=01|01|barclayis%40cardiff.ac.uk|60e585e582dc49c07cdf08d7ab4c613f|bdb74b3095684856bdbf06759778fcbc|1&sdata=B1DRS1xd7Q2kdTRkPi9dkuF8FAaefJJQM%2BMWK0Yu30Q%3D&reserved=0.

Hmm yeah, we do need a more informative error message. I don't think 'Document loader unable to load URL' is quite it -- we do want to get across that the url in question is specifically not authorized (by not being included in the loader).

Hmm yeah, we do need a more informative error message. I don't think 'Document loader unable to load URL' is quite it -- we do want to get across that the url in question is specifically not authorized (by not being included in the loader).

the issue with using authorized is that the documentLoader also looks up verificationMethods and not just contexts. This PR is specifically for this issue;
https://github.com/digitalbazaar/vc-js/issues/39#issuecomment-582942088


Remove Windows-Machine-Learning SampleSharedLib's dependency on BatchSupport sample project

Remove Windows-Machine-Learning SampleSharedLib's dependency on BatchSupport sample project

I think there's some sort of constant circular dependency going on with the samplesSharedLib. Updating SampleSharedLib.vcxproj does successfully remove the dependency, but when I go to build the BatchSupport solution, I get a build error saying that the SamplesSharedLib project can't build with the error: "This project references NuGet package(s) that are missing on this computer.... The missing file is ..\packages\Microsoft.Windows.CppWinRT.2.0.220131.2\build\native\Microsoft.Windows.CppWinRT.props", even though it's clearly in SamplesSharedLib.vcxproj as seen in this change.
The only way I've found to get BatchSupport to build is to manually delete these references in SamplesSharedLib.vcxproj, then install the cppwinrt nuget package into BatchSupport the solution itself, thus creating SamplesSharedLib's original dependency on BatchSupport.
There may be a better way that I don't know about to fix this build issue without re-introducing SamplesSharedLib's dependency on BatchSupport. But if there isn't I suppose we make SamplesSharedLib into just a collection of include-able headers instead of a separate project.

I can build BatchSupport now, but I haven't figured out yet why I get the following error when trying to build CustomTensorzation (the other sample that uses SamplesSharedLib):
LNK2038	mismatch detected for 'C++/WinRT version': value '2.0.201201.7' doesn't match value '2.0.220131.2' in main.obj ..\Samples\CustomTensorization\CustomTensorization\SampleSharedLib.lib(FileHelper.obj)

I can build BatchSupport now, but I haven't figured out yet why I get the following error when trying to build CustomTensorzation (the other sample that uses SamplesSharedLib): LNK2038 mismatch detected for 'C++/WinRT version': value '2.0.201201.7' doesn't match value '2.0.220131.2' in main.obj ..\Samples\CustomTensorization\CustomTensorization\SampleSharedLib.lib(FileHelper.obj)

That's really odd, I searched the entire repo for 201201 but found nothing, did you try cleanup build?

That's really odd, I searched the entire repo for 201201 but found nothing, did you try cleanup build?

I did, but it doesn't seem to help. Do you get this error? It could just be something with my local build.


Permit can be ended if current period end time same day or less than

Context

PV-786
How Has This Been Tested?

Unit tests
Manual Testing Instructions for Reviewers
In Webshop, create a new open ended permit. When cancelling the permit, you should have both options available ("immediate", or "end of current period"). In both cases the permit should be closed correctly or the end date set to end at specified time.

Codecov Report
Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (453dcb3) 84.26% compared to head (4ecaf37) 84.30%.

Files
Patch %
Lines

parking_permits/models/parking_permit.py
60.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #473      +/-   ##
===========================================
+ Coverage    84.26%   84.30%   +0.03%
===========================================
  Files          160      160
  Lines         9761     9792      +31
===========================================
+ Hits          8225     8255      +30
- Misses        1536     1537       +1

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


messageObject always nil

I lifted the example code and I do get notified, but the messageObject is always nil in the block handler.
In my watch kit extension, I send:
[self.wormhole passMessageObject:@{@"buttonNumber" : @(1)} identifier:@"button"];
But in my view controller, messageObject is always nil.
Any ideas? I assume it's something simple.
[self.wormhole listenForMessageWithIdentifier:@"button" listener:^(id messageObject) {
// The number is identified with the buttonNumber key in the message object
NSNumber *number = [messageObject valueForKey:@"buttonNumber"];
self.numberLabel.text = [number stringValue];
}];

Make sure you've enabled the App group in the Watch Kit Extension capabilities list. That did the trick for me.

I have enabled the app group. It's really weird and it's a problem on the app side. The extension is correctly sending the object. The problem is happening with this call: containerURLForSecurityApplicationGroupIdentifier
Now, the application does have the correct app group.

Weird. No changes, works today.


Catch HTTP errors

This logs all request errors via httpx that can happen from time to time (connect error, timeout, ..) and continues updating other feeds instead of terminating the task.
See also https://www.python-httpx.org/exceptions/
This should greatly improve stability of the update feeds task which is related to #10

Can be manually tested by adding an invalid feed like https://foo.bar. I noticed in my test runs that an actual feed threw a httpx.ConnectTimeout

Thanks! Quite the oversight on my side to not check for exceptions there.
Fixes #10


LiteDataLoader wrapper improvements

What does this PR do?
Follow up to #10279
Related: #10329
I accidentally approved #10279 (instead of comment), but the PR was not finished and pushed to master due to priorities of the Lightning v1.5 release.
This PR adds several improvements and corrections. With the new Lite package in Lightning, we want to achieve the highest level of code quality, documentation polish, test coverage and very important, readability. Lite is not Lightning: Everyone should be able to dive into the code and understand it immediately.
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@awaelchli do you want to land this before we move this code to Lightning?


Make buttons for links in WayPoint 'Pair program on bonfires'

Way too many people are coming into the LetsPair chatroom not knowing how to get signed up for Screen Hero. They ask for an invite even though they don't need one. The problem seems to me to be related to the visibility of the downloads for Screen Hero in number 5 in the WayPoint 'Pair program on bonfires'. Below is a typical exchange (the first one of the sort that I've participated in) illustrating the problem and my proposed solution which is to:
Make the links more visible by turning them into buttons.
nunya01 21:40
Is an invite the only way to get on Screen Hero? If so, can somebody invite me?
Cylocation 21:44
My gorsh, everyone is asking that. Follow the directions on the http://www.freecodecamp.com/challenges/Waypoint: Pair Program on Bonfires page, download it and sign in from inside the app.
b-a-chua 21:44
@Cylocation is there any way to make that message into a sticky?
That way, people can just see it easily.
Cylocation 21:47
@b-a-chua I don't think Gitter has that sort of functionality.
What would be better is to make the Mac and Windows links in number 5 more visibly clickable, perhaps make them buttons.

Closing as stale, please reopen if this is still an issue.


performance of font-lock-fontify-keywords-region

I'm not sure if this is scala-mode doing this, but Emacs started running really slowly for me in a big file and I took a profile-report:
- redisplay_internal (C function)                                3129  74%
 - jit-lock-function                                             3121  74%
  - jit-lock-fontify-now                                         3121  74%
   - funcall                                                     3121  74%
    - #<compiled 0xb3e4c9>                                       3121  74%
     - run-hook-with-args                                        3121  74%
      - font-lock-fontify-region                                 3121  74%
       - font-lock-default-fontify-region                        3121  74%
        - font-lock-fontify-keywords-region                      3093  74%
         + eval                                                   112   2%
         + scala-font-lock:mark-pattern1-part                      39   0%
           scala-font-lock:mark-reserved-symbols                   36   0%
         + scala-font-lock:mark-integerLiteral                      8   0%
           scala-font-lock:mark-underscore                          8   0%
        + font-lock-fontify-syntactically-region                   28   0%

it all seems to be internal work in the font-lock-fontify-keywords-region can you think of anything that might be causing a problem in scala-mode? I assume it is scala-mode that is defining the keywords for the mode.
Sadly I can't share the file with you, but if I can ever reproduce in a public project, most definitely. Probably for the best, this file would burn your retinas.

This issue was moved to ensime/ensime-emacs#227


Feature: Everytime click on navbar content it open newtab

Feature description
Description
Every time when we click on the navbar content (like About, Blog, Resources) it opens a new tab that consumes more ram than a single page, I want to work on this issue, Also it affects the user experience.
##Video
https://github.com/SauravMukherjee44/Aec-Library-Website/assets/83147410/e0866be4-9214-429f-adb9-323ed1ae7bd7
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Closing this


Allow colour-coded tables.

Human Practices:

SWOT Analysis
NICE Guidelines

What colours and which elements of the table?


[grpc] java-client -> go-server interop test fail

The response by go-server does not contain tag "method".
SEVERE: Client received wrong TagContext. Got [Tag{key=TagKey{name=operation}, value=TagValue{asString=interop-test}}, Tag{key=TagKey{name=project}, value=TagValue{asString=open-census}}], want [Tag{key=TagKey{name=method}, value=TagValue{asString=interop.EchoService/echo}}, Tag{key=TagKey{name=operation}, value=TagValue{asString=interop-test}}, Tag{key=TagKey{name=project}, value=TagValue{asString=open-census}}].

With @songy23 's help, I found the behavior change is due to https://github.com/census-instrumentation/opencensus-go/pull/566. Which behavior is correct? Should the "method" be propagated or not? If not, we should probably change the java code.

FYI the method tag will be automatically propagated in gRPC Java: https://github.com/grpc/grpc-java/blob/master/core/src/main/java/io/grpc/internal/CensusStatsModule.java#L332. While in Go, it's no longer propagated after https://github.com/census-instrumentation/opencensus-go/pull/566.
/cc @dinooliva @sebright @rakyll @ramonza @bogdandrutu

IIUC C++ also propagates method tag: https://github.com/census-instrumentation/opencensus-cpp/blob/master/opencensus/plugins/grpc/internal/server_filter.cc#L114.

I think gRPC should record stats with the method tag but not propagate it, since the server already knows what method was called and can add the tag back.

We deliberately stopped propagating it because it doesn't really make sense outside the scope of the original request.

Make sense. I think we need to specify this in gRPC propagation spec (https://github.com/census-instrumentation/opencensus-specs/issues/66) if not propagating method tag is the desired bebavior.

The grpc client doesn't need to propagate the method tag but the grpc server does need to add it to the context where the rpc is being processed so there should be a method tag in scope on the server side.
Also, if the grpc server subsequently makes a call as a grpc client, then the method tag will be propagated since it is in scope (I don't believe gRPC has any logic for explicitly removing the method tag).


Attaching a tab delimited file

Attaching a tab delimited file is displaying like this.
PFA for the tab delimited file sample and the screenshot.

tabDelimited.txt

With right mime type it works perfectly:
@Attachment(value = "TSV file", type = "text/tab-separated-values")

Please, try it.

Thank you


[BUG] System.UnauthorizedAccessException: Access to the path 'RobloxCrashHandler.exe' is denied.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
2.8.1
What problem did you encounter?
Roblox launcher can open but bloxstrap cant
Bloxstrap Log
2024-10-31T14:28:46Z [App::OnStartup] Starting Bloxstrap v2.8.1
2024-10-31T14:28:47Z [App::OnStartup] Compiled Wednesday, 30 October 2024 at 5:35:23 AM from commit 2be2f4d0d275e60eabf2f63ecbbe4e38b2aa0773 (tag/v2.8.1)
2024-10-31T14:28:47Z [App::OnStartup] Loaded from %UserProfile%\AppData\Local\Bloxstrap\Bloxstrap.exe
2024-10-31T14:28:47Z [App::OnStartup] Temp path is %UserProfile%\AppData\Local\Temp\Bloxstrap
2024-10-31T14:28:47Z [App::OnStartup] WindowsStartMenu path is %UserProfile%\AppData\Roaming\Microsoft\Windows\Start Menu\Programs
2024-10-31T14:28:47Z [Logger::Initialize] Initializing at %UserProfile%\AppData\Local\Bloxstrap\Logs\Bloxstrap_20241031T142847Z.log
2024-10-31T14:28:47Z [Logger::Initialize] Finished initializing!
2024-10-31T14:28:47Z [JsonManager<Settings>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Settings.json...
2024-10-31T14:28:47Z [JsonManager<Settings>::Load] Loaded successfully!
2024-10-31T14:28:47Z [JsonManager<State>::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\State.json...
2024-10-31T14:28:47Z [JsonManager<State>::Load] Loaded successfully!
2024-10-31T14:28:47Z [FastFlagManager::Load] Loading from %UserProfile%\AppData\Local\Bloxstrap\Modifications\ClientSettings\ClientAppSettings.json...
2024-10-31T14:28:47Z [FastFlagManager::Load] Loaded successfully!
2024-10-31T14:28:50Z [LaunchHandler::LaunchRoblox] Initializing bootstrapper
2024-10-31T14:28:50Z [LaunchHandler::LaunchRoblox] Initializing bootstrapper dialog
2024-10-31T14:28:50Z [Bootstrapper::Run] Running bootstrapper
2024-10-31T14:28:50Z [Bootstrapper::SetStatus] Connecting to Roblox...
2024-10-31T14:28:50Z [Deployment::TestConnection<https://setup.rbxcdn.com>] Connecting...
2024-10-31T14:28:50Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup.rbxcdn.com/versionStudio
2024-10-31T14:28:50Z [Deployment::InitializeConnectivity] Testing connectivity...
2024-10-31T14:28:52Z [Deployment::TestConnection<https://roblox-setup.cachefly.net>] Connecting...
2024-10-31T14:28:52Z [Deployment::TestConnection<https://setup-aws.rbxcdn.com>] Connecting...
2024-10-31T14:28:52Z [Deployment::TestConnection<https://setup-ak.rbxcdn.com>] Connecting...
2024-10-31T14:28:52Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup-aws.rbxcdn.com/versionStudio
2024-10-31T14:28:52Z [HttpClientLoggingHandler::ProcessRequest] GET https://setup-ak.rbxcdn.com/versionStudio
2024-10-31T14:28:52Z [HttpClientLoggingHandler::ProcessRequest] GET https://roblox-setup.cachefly.net/versionStudio
2024-10-31T14:28:53Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://roblox-setup.cachefly.net/versionStudio
2024-10-31T14:28:53Z [Deployment::InitializeConnectivity] Got https://roblox-setup.cachefly.net as the optimal base URL
2024-10-31T14:28:53Z [Bootstrapper::Run] Connectivity check finished
2024-10-31T14:28:53Z [Deployment::TestConnection<https://setup.rbxcdn.com>] Connectivity test cancelled.
2024-10-31T14:28:53Z [Deployment::TestConnection<https://setup-aws.rbxcdn.com>] Connectivity test cancelled.
2024-10-31T14:28:53Z [Deployment::TestConnection<https://setup-ak.rbxcdn.com>] Connectivity test cancelled.
2024-10-31T14:28:53Z [Bootstrapper::CheckForUpdates] Checking for updates...
2024-10-31T14:28:53Z [HttpClientLoggingHandler::ProcessRequest] GET https://api.github.com/repos/bloxstraplabs/bloxstrap/releases/latest
2024-10-31T14:28:54Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://api.github.com/repos/bloxstraplabs/bloxstrap/releases/latest
2024-10-31T14:28:55Z [Bootstrapper::CheckForUpdates] No updates found
2024-10-31T14:28:55Z [Bootstrapper::GetLatestVersionInfo] Got channel as production
2024-10-31T14:28:55Z [Deployment::GetInfo] Getting deploy info for channel production
2024-10-31T14:28:55Z [HttpClientLoggingHandler::ProcessRequest] GET https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer
2024-10-31T14:28:56Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://clientsettingscdn.roblox.com/v2/client-version/WindowsPlayer
2024-10-31T14:28:56Z [RegistryKeyEx::SetValueSafe] Writing '' to HKEY_CURRENT_USER\SOFTWARE\ROBLOX Corporation\Environments\RobloxPlayer\Channel\www.roblox.com
2024-10-31T14:28:56Z [HttpClientLoggingHandler::ProcessRequest] GET https://roblox-setup.cachefly.net/version-b7eebc919e96477a-rbxPkgManifest.txt
2024-10-31T14:28:56Z [HttpClientLoggingHandler::ProcessResponse] 200 OK https://roblox-setup.cachefly.net/version-b7eebc919e96477a-rbxPkgManifest.txt
2024-10-31T14:28:56Z [Bootstrapper::SetStatus] Installing {product}...
2024-10-31T14:28:56Z [LaunchHandler::LaunchRoblox] Bootstrapper task has finished
2024-10-31T14:28:56Z [LaunchHandler::LaunchRoblox] An exception occurred when running the bootstrapper
2024-10-31T14:28:56Z [App::FinalizeExceptionHandling] (0x80070005) System.UnauthorizedAccessException: Access to the path 'RobloxCrashHandler.exe' is denied.
   at System.IO.FileSystem.RemoveDirectoryRecursive(String fullPath, WIN32_FIND_DATA& findData, Boolean topLevel)
   at System.IO.FileSystem.RemoveDirectory(String fullPath, Boolean recursive)
   at Bloxstrap.Bootstrapper.UpgradeRoblox()
   at Bloxstrap.Bootstrapper.Run()
   at Bloxstrap.Bootstrapper.Run()
2024-10-31T14:28:56Z [HttpClientLoggingHandler::ProcessRequest] POST https://bloxstraplabs.com/metrics/post-exception

I'm assuming this has been fixed with v2.8.2. If not, please open a new issue.


fix: update frame script reference

What

This PR fixes a broken link.
Changes
Commit 4c92e49891bfdd8b7ee8f375cd6f5f536460037d changed the location of Frame.ts. This PR adjusts sources to changes.
Tested on

Related issues

thanks!


[Performance Page] Meshery UI restructuring

Description
This PR Adds a static page for the UI performance page
Signed commits

[x] Yes, I signed my commits.

@kayceeDev We are no longer using makestyles please use styled component like in the other PRs

Alright. Thanks. Adjustments will be made.

@kayceeDev are you circling back to this PR?

I am definitely.


Variable renaming in custom match is not identified by case

In this line of code https://github.com/trueagi-io/hyperon-experimental/blob/8ee0262a1469c29094d49fd51c515646ae59a212/python/sandbox/numpy/numme.py#L39
while running this line of script https://github.com/trueagi-io/hyperon-experimental/blob/8ee0262a1469c29094d49fd51c515646ae59a212/python/sandbox/numpy/nm_test.metta#L34
the following bindings are obtained
{'y1': array([2, 3]), 'y2#33': array([3, 4]), 'y3': array([4, 5])}
{'x#32': array([[1, 1],
       [2, 2],
       [3, 3]]), 'y1': array([2, 3]), 'y2#33': array([3, 4]), 'y3': array([4, 5])}

and the result of whole evaluation case is (np.add $x#32 $y2#33)). That is, exactly those variables, which are used in the result of case are got renamed and not used to form the actual output (i.e. (np.add $x $y2))).
How to reproduce: run python/sandbox/numpy/nm_test.metta script.

My guess it can be an issue of passing Bindings between Python and Rust. But it needs more analysis to say for sure.

@Necr0x0Der  could you please retest it. May be it is fixed after switching to minimal MeTTa.

Yes, it seems to be fixed (although some other minor changes in the behavior also happened)


feat: avoid error out on frame with invalid manufacturer code

I am trying to move my project from pymeterbus and some heatmeters my project has to detect don't have filed out manufacturer code (it contains 0x00 bytes) with leads to m-bus-parser to error out and not be able to parse any DataRecords. I modified the code to not error out and be able to continue.
I suspect storing a Result inside a struct isn't the nicest option but this way the information that something happened while parsing the manufacturer code is still retained. I am open to other ideas how to handle that.

Hey, thanks for the PR! Sorry I only just saw it now. I will take a look :)

I want to understand you issue, you would like to be able to get the parsed result for an illegal manufacturer, am I correct?

Exactly,
the payload that does this can be found in my other PR https://github.com/maebli/m-bus-parser/pull/29/files#diff-cd9539fdb6e41539f62bd90927ed31026413e7c20f5ecf794429cd1c6a33e5cbR847
when you put that payload into https://dev-lab.github.io/tmbus/tmbus.htm you get this:


Errors on macOS 10.15 Catalina

Following features don't work on macOS 10.15 Catalina.

iTunesHelper.PlayPause();
iTunesHelper.Stop();
Get/Set iTunesHelper.PlayerPosition

Still works

iTunesHelper.OnStatusChanged

Sorry, I'm still using Mojave and no plan to upgrade to Catalina right now...😑


CONTRIBUTING links to deliciousinsights/brunch-guide/issues

Is this the actual intent or should it point to the brunch/brunch-guide/issues now?

It should be updated


Ephemeral room versions

Suggestion
Matrix has resolved moderated instant messaging over open federation(s) just fine. However, existing room versions 1 through 7 are all persistent rooms. To make Matrix one true communication medium and messaging protocol, we need ephemeral room versions too, alongside all the persistent room versions we have now. However, we will have to tackle with the moderation problem over federation in an ephemeral information exchange situation, which is more difficult as servers would not be able to ask for older state in ephemeral rooms.

To make Matrix one true communication medium and messaging protocol, we need ephemeral room versions too

You again jump to conclusions without elaborating the logic behind them. I, for one, fail to grasp how the right part of the quoted statement implements the left part of it - especially with the caveat you leave after "However".

I thought it was obvious — simply put, not everything is worth persisting.

What is an "ephemeral room version"?

An ephemeral room version is a room version that only accepts ephemeral events and rejects persistence. Ephemeral communications, even in moderated group chat settings, have significant use cases. Not everything is worth or even suitable for persisting.

I think it could be useful for the PubSub use case (#951)

I forgot about this issue.

An ephemeral room version is a room version that only accepts ephemeral events and rejects persistence.

First of all, we can't have a room that has absolutely no persistence.  Some things must be persisted (e.g. m.room.member).  Secondly, even if we make an exception for state events, we don't need a room version that disallows persistence.  Instead, we can have rooms that allow both ephemeral and persisted events, and allow the room to be configured to allow one or the other, or both. MSC2477 does this by allowing power levels of ephemeral events to be specified separately from regular events.


GLContext: incorrect free when mediapy imported

I've stumbled upon a confusing error when mediapy is imported and an renderer instantiated, MWE:
$ python -c "import mediapy; import mujoco; model = mujoco.MjModel.from_xml_string('<mujoco/>'); renderer = mujoco.Renderer(model)"
Exception ignored in: <function GLContext.__del__ at 0x7f215f6a4b80>
Traceback (most recent call last):
  File "/nix/store/r325k7kv6ra6yh74dvh4x3g1xl692ma3-python3.11-mujoco-3.0.0/lib/python3.11/site-packages/mujoco/glfw/__init__.py", line 41, in __del__
  File "/nix/store/r325k7kv6ra6yh74dvh4x3g1xl692ma3-python3.11-mujoco-3.0.0/lib/python3.11/site-packages/mujoco/glfw/__init__.py", line 35, in free
TypeError: 'NoneType' object is not callable

Removing the mediapy import, or simply not assigning the renderer to a variable fixes it.
I experience this with v3.0.0 in an in-progess package for NixOS (https://github.com/NixOS/nixpkgs/pull/266099). Trying to figure out where this error originates. mediapy is version 1.1.9. Python is 3.11.

Can reproduce this on my end as well on Ubuntu and Mac.

We're looking into this, will have an update soon.

Hi @alberthli and @tmplt, can you check if the latest release (3.1.0) fixes your issue? We've added a close method that you should call to free up resources associated with the renderer object. You can use it as follows:
# Option 1.
renderer = Renderer(...)
# Use renderer.
renderer.close()

# Option 2.
with Renderer(...) as renderer:
# Use renderer.

Hi @kevinzakka, I can confirm it works (I only tested option 1).

Awesome, thanks @alberthli. Marking as resolved then.

Explicitly calling renderer.close() or del rendered does resolve it on 3.1.0 for me, but omitting it yields errors. But perhaps __del__ isn't invoked on the OP example?

Hi @alberthli and @tmplt, can you check if the latest release (3.1.0) fixes your issue? We've added a close method that you should call to free up resources associated with the renderer object. You can use it as follows:
# Option 1.
renderer = Renderer(...)
# Use renderer.
renderer.close()

# Option 2.
with Renderer(...) as renderer:
# Use renderer.

hello @kevinzakka , this method doesn't work. My code is like this:
...
import mediapy as media
...
renderer = mujoco.Renderer(model)
...
renderer.close()
When i run it, the problem reappears agian:
Exception ignored in: <function Renderer.del at 0x000001D063353CE0>
TypeError: 'NoneType' object is not callable

Hi, I still have the same issue on 3.1.6 when I have a persistent renderer in my env class. Am I supposed to create it on the fly using the with statement? I thought this would slow down the rendering.
Thanks.


on upgrade - UI does not reflect new version information

The UI appears to need to be restarted before it notices the updated version information

lol -wrong repo


Add feature: Scale selection

Add the ability to select scale for the whole room with synchronisation.
Available scales should harmonize well even if all notes play simultaneously.
Good candidates for this are:
Pentatonic scales
Hang

D3 Ding, A3, B♭3, C4, D4, E4, F4, A4

Ancient Solfeggio Scale

174 Hz, 285 Hz, 396 Hz, 417 Hz, 528 Hz, 639 Hz, 741 Hz, 852 Hz, 963 Hz

Koshi tunings

G, C, E, F, G, C, E, G (Terra)
A, D, F, G, A, D, F, A (Aqua)
A, C, E, A, B, C, E, B (Aria)
G, B, D, G, B, D, G, A (Ignis)

Ideally, scales can also be rotated and slightly detuned to achieve unique sound characteristics.

See https://codepen.io/jakealbaugh/full/qNrZyw/


S2 non-gray-800 colors update

Updated S2 non-gray-800 tokens in dark theme, and all aliases impacted to refer to the updated tokens:

silver-800

magenta-800

red-800

orange-800

brown-800

yellow-800

chartreuse-800

celery-800

green-800

seafoam-800

turquoise-800

cyan-800

blue-800

indigo-800

purple-800

fuchsia-800

pink-800

cinnamon-800

accent-background-color-default

accent-background-color-hover

accent-background-color-down

accent-background-color-key-focus

informative-background-color-default

informative-background-color-hover

informative-background-color-down

informative-background-color-key-focus

blue-background-color-default

negative-background-color-default

negative-background-color-hover

negative-background-color-down

negative-background-color-key-focus

red-background-color-default

positive-background-color-default

positive-background-color-hover

positive-background-color-down

positive-background-color-key-focus

green-background-color-default

seafoam-background-color-default

cyan-background-color-default

indigo-background-color-default

purple-background-color-default

fuchsia-background-color-default

magenta-background-color-default

pink-background-color-default

turquoise-background-color-default

brown-background-color-default

cinnamon-background-color-default

silver-background-color-default

Motivation and context

These updates to colors ensure we have a color stop that is 4.5:1 with white and 3:1 with gray-100. These color updates will fix accessibility issues for selected, accent variants in some components, like action button and tag.

Types of changes

 [ ] Patch (bug fixes, typos, mistakes; non-breaking change which fixes an issue)
 [x] Minor (add a new token, changing a value, deprecating a token; non-breaking change which adds functionality)
 [ ] Major (deleting a token, changing token value type, renaming a token by deprecating the old one; fix or feature that would cause existing functionality to change)
Checklist:

 [x] I have signed the [Adobe Open Source CLA](https://opensource.adobe.com/cla.html).
 [x] I updated the token in all applicable sets. This applies if updating, adding, or deleting a token that has data across different sets (for example, if the value differs across color themes.)

TESTING spectrum2
READING spectrum-tokens: /spectrum-tokens/packages/tokens/src/
READING   tokens studio: /spectrum-tokens-studio-data/src/tokens-studio/spectrum2-colors/
READING   tokens studio: /spectrum-tokens-studio-data/src/tokens-studio/spectrum2-non-colors/

WARNING - STUDIO INCLUDES DEPRECATED TOKEN: overlay-opacity/dark [31d5b502-6266-4309-8f8a-3892e6e158da]
WARNING - STUDIO INCLUDES DEPRECATED TOKEN: overlay-opacity/light [26b9803c-577f-4a29-badd-dfc459e8b73e]

SPECTRUM SYNC SUMMARY
CHANGED VALUES: (48) accent-background-color-default/dark, accent-background-color-hover/dark, accent-background-color-down/dark, accent-background-color-key-focus/dark, informative-background-color-default/dark, informative-background-color-hover/dark, informative-background-color-down/dark, informative-background-color-key-focus/dark, positive-background-color-default/dark, positive-background-color-hover/dark, positive-background-color-down/dark, positive-background-color-key-focus/dark, negative-background-color-default/dark, negative-background-color-hover/dark, negative-background-color-down/dark, negative-background-color-key-focus/dark, blue-background-color-default/dark, green-background-color-default/dark, red-background-color-default/dark, brown-background-color-default/dark, cinnamon-background-color-default/dark, cyan-background-color-default/dark, fuchsia-background-color-default/dark, indigo-background-color-default/dark, magenta-background-color-default/dark, pink-background-color-default/dark, purple-background-color-default/dark, seafoam-background-color-default/dark, silver-background-color-default/dark, turquoise-background-color-default/dark, blue-800/dark, green-800/dark, orange-800/dark, red-800/dark, celery-800/dark, chartreuse-800/dark, cyan-800/dark, fuchsia-800/dark, indigo-800/dark, magenta-800/dark, purple-800/dark, seafoam-800/dark, yellow-800/dark, pink-800/dark, turquoise-800/dark, brown-800/dark, cinnamon-800/dark, silver-800/dark impacting 75 tokens

TEST COMPLETE


amplify mock not setting API_*_GRAPHQLAPIENDPOINTOUTPUT env var correctly in lambda resolver

Describe the bug
I have a graphql lambda resolver that as part of its processing does a create mutation via the same graphql API.
I am currently testing this lambda resolver via amplify mock. Everything works until the lambda attempts to do a graphql mutation.
The error I get is the following:
Lambda Data source failed with the following error

{
  type: 'Lambda:Unhandled',
  message: {
    message: 'Request failed with status code 403',
    name: 'Error',
    stack: 'Error: Request failed with status code 403\n' +
      '    at createError (/Users/hisham/src/ess-app/amplify/backend/function/EssGraphQLResolverFn/src/node_modules/axios/lib/core/createError.js:16:15)\n' +
      '    at settle (/Users/hisham/src/ess-app/amplify/backend/function/EssGraphQLResolverFn/src/node_modules/axios/lib/core/settle.js:17:12)\n' +
      '    at IncomingMessage.handleStreamEnd (/Users/hisham/src/ess-app/amplify/backend/function/EssGraphQLResolverFn/src/node_modules/axios/lib/adapters/http.js:260:11)\n' +
      '    at IncomingMessage.emit (events.js:327:22)\n' +
      '    at IncomingMessage.EventEmitter.emit (domain.js:467:12)\n' +
      '    at endReadableNT (internal/streams/readable.js:1327:12)\n' +
      '    at processTicksAndRejections (internal/process/task_queues.js:80:21)',
    config: {
      url: 'https://<myapi>.appsync-api.us-east-1.amazonaws.com/graphql',
      method: 'post',
      data: '{"query":"mutation....

Looking at the url above, it is supposed to be http://<IP_ADDRESS>:20002 since I am running amplify mock, but instead it's my actual non-mock aws appsync endpoint. It does not appear that amplify mock sets the API...GRAPHQLAPIENDPOINTOUTPUT env var correctly.
Amplify CLI Version
4.41.2
To Reproduce
Create a lambda resolver that in turn calls the same graphql api with a mutation or query, and test via amplify mock.

@hisham So you have a Lambda resolver as a part of your schema which is calling the GraphQL endpoint as well?
Currently we don't set any environment variables locally for you - but you can set it locally as a process environment for local mocking purposes. We're working on a larger project for setting environment variables locally for lambda functions - cc @edwardfoyle

Yes that's correct. Ok I'll try your suggestion next time I use amplify mock.

@kaustavghosh06 running the following does not work:
API_<myapi>_GRAPHQLAPIENDPOINTOUTPUT=http://localhost:20002 amplify mock
It seems the amplify mock code overrides it with the incorrect url.
I would have to set this env var in the lambda code instead, and then remove it when I do amplify push. Is there a way within lambda to know if it's in a mock environment? I see you set ENV to NONE and REGION to us-east-1-fake, and I can check other things in the env, but I wonder if there's an official way.

Got it. This would PR would potentially fix your issue @hisham - https://github.com/aws-amplify/amplify-cli/pull/6551

A fix for this has been released in version 4.44.1 of the CLI. Please upgrade to the latest version to try it out.


Doesn't consider Artistic-1.0-Perl valid

https://spdx.org/licenses/Artistic-1.0-Perl.html#licenseText
$ node
> require('spdx').valid('Artistic-1.0-Perl')
null

Okay using your package https://www.npmjs.com/package/spdx-expression-validate works


[Feature Request]: Add .env.example file

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Add .env.example file
Use Case
Including a .env.example file provides new contributors with a clear reference for the MongoDB secrets the application uses and how the code interacts with environment variables.
Benefits
No response
Add ScreenShots
No response
Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I want to work on this issue

Sure, you can add the .env.example
Use this for reference


Request for Documentation: Mattermost as a OAuth 2.0 Service Provider - Supported Grant Flows and URLs

Mattermost user joram from https://community-release.mattermost.com has requested the following be documented:
We support the [authorization code](https://oauth.net/2/grant-types/authorization-code/) and [implicit](https://oauth.net/2/grant-types/implicit/) grant flows for OAuth 2.0 applications. The authorize URL is `<your-mattermost-url>/oauth/authorize` and the token URL is `<your-mattermost-url>/oauth/access_token`

See the original post here.
This issue was generated from Mattermost using the Doc Up plugin.

@jwilander - This ticket has been idle for some time. Would you or a member of your team be open to creating a PR with the requested documentation updates?

I can submit it


HTML export doesn't show plots

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
If I want to export a notebook as html. The result shows no plots, made with "plotly.express".
"matplotlib.pyplot" plots are shown.
VS Code Version
1.75.0
Jupyter Extension Version
v203.1.2000312134
Jupyter logs
nothing in output

Coding Language and Runtime Version
Python 3.9.0
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Sorry for never getting back to you on this issue, somehow this slipped through the cracks.
We have made a lot of improvements and fixes in this space and believe this may have been resolved.
Please ensure you update to the latest version of VS code and the extensions.
Closing as I believe it may have been resolved,
If not please ping here and I'll reopen this or feel free to create a new issue.


odb: dbMarker

Adds:

dbMarker, dbMarkerCategory, dbMarkerGroup to support tracking of markers in database
Line as a geom type (this has come up a few times now, so it was easier to add a class).

Changes:

PDN, PSM, GRT to use dbMarker
moved parsing of report information from DRCWidget to dbMarkerGroup
Updates DRCWidget to handle switching between marker information.

Discussion:

currently getting DRC information into images for SiliconCompiler and ORFS rely on reading the drc report back in, which is not needed anymore, but also is also not working at the moment and needs to be reworked how to do this (ie. select the DRC datasource).
Added gui::select_marker_category
Should be database provide callbacks for the GUI to be able to automatically update the DRCWidget or should we just rely on manually updating the list?
DRT breaks the connection between odb objects and DRT objects so obstruction information is lost at the moment, therefore that implementation is not included for now (I can include it if the loss of this information is acceptable, but I don't think it would be a good idea until the losses can be avoided).
Searches for obstructions in block, better solution requires fixing the frObjects
The JSON file format is a little more descriptive for reporting (compared to the TR file), but there isn't a "spec" for this file, so I did my best to handle it, but we can discuss what makes sense for this file.
The DRC widget is also tracking if a marker has been viewed or is visible (it might make sense to move the viewed/visited into the database? and the visibility can continue to be handled in the GUI.

@maliberty marking this draft until discussions points are resolved.

I don't follow the bit about drt and "obstruction information".   Please elaborate.

If you load a drc file can you assign it to a group?  For loading drcs from an external tool it would be helpful to put them in a specific category.

@maliberty The file IO still needs to be fixed, but I was able to remove the group and just have recursive categories.

@gadfort where does this stand with respect to being ready for review/merge?

@maliberty I think it's ready, it contains what we discussed the other week.

Conflict to resolve...

@maliberty conflict resolved. I added a waived field to mirror what klayout has

4 compile errors on MacOS Sequoia M1
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/src/db/dbMarker.cpp:177:10: error: invalid operands to binary expression ('dbIStream' and 'std::size_t' (aka 'unsigned long'))
177 |   stream >> item_count;
|   ~~~~~~ ^  ~~~~~~~~~~
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/include/odb/dbTypes.h:147:12: note: candidate function not viable: no known conversion from 'std::size_t' (aka 'unsigned long') to 'dbGDSSTrans &' for 2nd argument
147 | dbIStream& operator>>(dbIStream& stream, dbGDSSTrans& t);
|            ^                             ~~~~~~~~~~~~~~
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/src/db/dbMarker.cpp:236:10: error: use of overloaded operator '<<' is ambiguous (with operand types 'dbOStream' and 'size_type' (aka 'unsigned long'))
236 |   stream << obj.sources_.size();
|   ~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/include/odb/dbStream.h:86:14: note: candidate function
86 |   dbOStream& operator<<(bool c)
|              ^
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/src/db/dbMarker.cpp:243:10: error: use of overloaded operator '<<' is ambiguous (with operand types 'dbOStream' and 'size_type' (aka 'unsigned long'))
243 |   stream << obj.shapes_.size();
|   ~~~~~~ ^  ~~~~~~~~~~~~~~~~~~
OpenROAD-flow-scripts/tools/OpenROAD/src/odb/include/odb/dbStream.h:86:14: note: candidate function
86 |   dbOStream& operator<<(bool c)
|              ^
4 errors generated.

Its hard to see how dbGDSSTrans  relates to this PR.  Did it work on master prior to this merge?

OR compiled just fine last night ...

@maliberty who is fixing this? I cannot compile on a Mac, but I could make some changes later today, but without a mac in the loop it would be impossible to ensure the changes cover everything

git bisect points to this and I'm willing and able to test ...
fe52a377858ba3e5e7125fd3d73dea36cb5e4281 is the first bad commit
commit fe52a377858ba3e5e7125fd3d73dea36cb5e4281
Author: Peter Gadfort<EMAIL_ADDRESS>Date:   Sun Oct 6 11:18:15 2024 -0400
odb: add dbMarkerGroup/Category and markers

Signed-off-by: Peter Gadfort<EMAIL_ADDRESS>

I have no mac either.  I would just make a branch with a best guess and ask @stefanottili to test it.  I can try if you want.

@stefanottili  can you try: https://github.com/gadfort/OpenROAD/tree/casting

this compiles
@@ -173,7 +173,7 @@ dbIStream& operator>>(dbIStream& stream, _dbMarker& obj)
   stream >> obj.line_number_;
   // User Code Begin >>
   // handle shapes
-  std::size_t item_count;
+  int item_count;
   stream >> item_count;
   for (std::size_t i = 0; i < item_count; i++) {
     std::string db_type;
@@ -233,14 +233,14 @@ dbOStream& operator<<(dbOStream& stream, const _dbMarker& obj)
   stream << obj.line_number_;
   // User Code Begin <<
   // handle sources
-  stream << obj.sources_.size();
+  stream << static_cast<int>(obj.sources_.size());
   for (const auto& [db_type, dbid] : obj.sources_) {
     stream << std::string(dbObject::getTypeName(db_type));
     stream << dbid;
   }

   // handle shapes
-  stream << obj.shapes_.size();
+  stream << static_cast<int>(obj.shapes_.size());
   for (const dbMarker::MarkerShape& shape : obj.shapes_) {
     if (std::holds_alternative<Point>(shape)) {
       stream << _dbMarker::ShapeType::Point;


Protected events

UI reflects that only admins can apply protected labels and edit protected events

Coverage decreased (-9.06%) to 24.777% when pulling 73381d807229fd63e338e678b19600db1dddef37 on osteele/dev into a65d44614c6b5f141990694d1b0778363fd49b23 on dev.


Kanban button text hard to read

The world rejoiced when Kneecaps said "its lobster'in time" then proceeded to lobster everywhere. Actual peak theme, 10/10 would reinstall with extra lobsters.
OK real feedback now, its hard to see the buttons on Kanban because they're the standard shade of grey

Uh oh, what's going on here??

Yeah I was going to mention that! Only glitches out when in windowed mode on Mac. This is what it looks like expanded (1) and collapsed (2). Looks normal fullscreen (3)

Awh you are too sweet! I am so glad you are enjoying the theme 🥰 I went ahead and made some small tweaks to the contrast, but it's certainly difficult to balance user accents --how would you feel about removing the accent color from the buttons?

Wouldn't mind removing colours for Kansan buttons! Whatever you feel fits :)

Also, sorry, I haven't forgotten this; I'm still looking into it (slowly, and I pushed a sort-of fix...) --I don't have a Mac so I had no idea there was an issue there >~< Also, let me know if those Kanbans look better! I think it should be more minimal, now?

Also, sorry, I haven't forgotten this; I'm still looking into it (slowly, and I pushed a sort-of fix...) --I don't have a Mac so I had no idea there was an issue there >~< Also, let me know if those Kanbans look better! I think it should be more minimal, now?

Dig it. Button text looks much more visible now! The dashed table borders does take some getting used to, as the vertical border thickness is different from the horizontal border thickness, not sure if thats a bug or intentional

Anyways, Twiddling thumbs eagerly awaiting your next update and style settings (and any obviously any cheekily-crustaceous, carcinisation-related eastereggs)

noted the Menubar was fixed:

Tables also look clean:


scope launch doesn't tell you if you got it right

scope launch looks like it always works:
$ ./scope launch aouauenthsonteuhaosue.com
1819f0be0e5ea962250c3ec5f0098f5737ffb651a2165b09e0640c48f2c96852
Weave Scope is reachable at the following URL(s):
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/

$ ./scope launch
2d612f472a71f51a082349d87522835f793b00c7773bbd3d1b506a0341968a23
Weave Scope is reachable at the following URL(s):
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/

$ ./scope launch ===============
dc6bb878ea4fa57c55451fee4326b427f0c4cfb471c20471c8722b64b9dc5412
Weave Scope is reachable at the following URL(s):
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/
  * http://<IP_ADDRESS>:4040/

Each of these cases has exit code 0.
When you browse to the given URLs, there is no indication that the input was erroneous.
Thie lack of good feedback hinders a user's ability to learn scope by exploration.

Increases impact of #1556 and #1557.


Setting mouse mode to Input.MOUSE_MODE_CONFINED hides the cursor

Godot version:
3.0.3 RC3
OS/device including version:
Windows 10, Intel(R) HD Graphics 620
Issue description:
Setting Input.set_mouse_mode to Input.MOUSE_MODE_CONFINED hides the cursor. The documentation at http://docs.godotengine.org/uk/latest/classes/class_input.html#enum-input-mousemode says "[Confined mouse mode] Makes the mouse cursor visible but confines it to the game window."
Full disclosure I initially saw this behaviour with a C# script and assumed it was a C# issue, but I also tested it with GD script and the same issue occurred. If you alt-tab away from the window and back to it the cursor then shows as expected and is confined to the window
Steps to reproduce:
Call Input.set_mouse_mode(Input.MOUSE_MODE_CONFINED)
Minimal reproduction project:
test.zip

@jamestrotter You're using mono version? I can't reproduce this with 3.0.3 RC3.

Weird. Maybe its my machine. Mono 5.12.0, but using the MSBuild build tool. I assume mono wouldn't affect the gdscript version also not working though?

I have tested this too and am looking for an answer. I have a HP touchscreen laptop with Intel 4600 GPU. The mouse cursor disappears when confined (GDScript). Which is useful for my game, but when I set mouse back to visible it stays hidden till I move outside of Viewport. It then becomes visible again. I am going to try on different dev machine with Nvidia card and see if same happens.
I am using Godot 3.05, with GDScript.

I have tested on another machine with a NVidia 1050 GPU instead of Intel Graphics with same results. Mouse disappears when confined. When set back to visible you must move outside of viewport to get mouse cursor back.

I am having the same issue on mono 3.0.5. Setting the mouse mode to confined will hide the mouse. I have an option button with various windowed options, and I am setting fullscreen modes and mouse input like so:
private void OnOptionItemSelected(int i)
        {
            switch (i)
            {
                case 0:
                    OS.SetWindowFullscreen(true);
                    Input.SetMouseMode(Input.MouseMode.Confined);
                    break;
                case 1:
                    Input.SetMouseMode(Input.MouseMode.Visible);
                    OS.SetWindowFullscreen(true);
                    OS.SetBorderlessWindow(true);
                    break;
                case 2:
                    Input.SetMouseMode(Input.MouseMode.Visible);
                    OS.SetWindowFullscreen(false);
                    OS.SetBorderlessWindow(false);
                    break;
                default:
                    Input.SetMouseMode(Input.MouseMode.Visible);
                    OS.SetWindowFullscreen(false);
                    OS.SetBorderlessWindow(false);
                    break;
            }
        }

Same issue here. Running Godot 3.0.5 not the mono version.

I have the same Issue in newest Mono version, but the cursor just stays invisible. If i set the cursor to captured and then to confined without moving the cursor intermediately it stays invisible. If i move it before the second change it gets visible.

Hi everyone,
Same issue for me : when I switch the mouse capture mode on Windows from Captured to Confined, the cursor remains invisible. But seems to move correctly in the window (I have a raycast feedback in the console, which tells me when I hover over an object).
Under Linux no problem.
I use Godot in version 3.2.1 Stable and 3.2.2 Alpha.

This seems fixed in 3.4/4.0.

Note that in the master branch, Input.MOUSE_MODE_CONFINED_HIDDEN is available. It's not backported to 3.x yet though.


Typescript support

Hello there,
I appreciate very much this awesome tool that fits very well with the project I am working on.
I am wondering if there somewhere is planned a integration with font-end (Angular/Typescript) side to validate policies.
Can be very powerful to restrict component access based on roles for example.
Thanks a lot for a such awesome tool and keep going!

Thank you for the issue! We've done a lot of thinking about this topic, and we have some similar work upcoming on our near-term roadmap to enable authorization-dependent UI elements from the back end. Making those policy decisions on the front end is entirely possible (our WASM-backed Node.js integration will take a bit of tweaking to get it to run in browsers), and is definitely something we'll look at as a follow-up to the initial work.
Do you think authZ-dependent UI elements driven from the back end would work for your use case, at least until we get to the front end work?
We're still working out the order in which we'll prioritize languages and frameworks for the initial work, but any additional details you can provide about your specific use case would be super helpful to us in designing the feature.
You could also join our Slack workspace if you'd like. We've had people ask similar questions before and have shown them how to achieve it currently.

A couple of updates on this:

As @gj mentioned, it might be that this use case can be handled from the backend. We're doing some work on that currently, and will update https://github.com/osohq/oso/issues/612 based on that.
We did manage to get the typescript library working as a browser module too. We will look into distributing that so people can use it. If you have a use case for it, let us know and we can share with you an early version that you can try out!

A couple of updates on this:

As @gj mentioned, it might be that this use case can be handled from the backend. We're doing some work on that currently, and will update https://github.com/osohq/oso/issues/612 based on that.
We did manage to get the typescript library working as a browser module too. We will look into distributing that so people can use it. If you have a use case for it, let us know and we can share with you an early version that you can try out!

Hello :+1:  thanks again !
We (me and the company I work for) plan to buid a oauth2 authentication flow using SSO soon. We don't have yet a complete infrastructure to implement authentication flow but is on our Road-map.
Once we'll get there I will be glad to test the typescript library with our Front-End team in order to provide some useful feedback.

Hello :+1:  thanks again !
We (me and the company I work for) plan to buid a oauth2 authentication flow using SSO soon. We don't have yet a complete infrastructure to implement authentication flow but is on our Road-map.
Once we'll get there I will be glad to test the typescript library with our Front-End team in order to provide some useful feedback.

Hey @alexvuta, I'm gonna close this one in favor of #612.
Would love to hear any feedback you have over there or on Slack!


Template Weather 3 option not automatically filling in when I create Daily Note

Thank you so much for the plugin.
I am new-ish to Obsidian and was excited to implement your plugin in my Daily Note template.
I was able to set it up properly - I know this because I was, one time only, able to use the "Replace template strings" command to fill in "Weather 3", which I've placed in my template as %weather3%.
However, it doesn't replace the string automatically when I create the Daily Note, and the Open Weather commands only show up sometimes in the command palette.
I would appreciate your help in getting this working.
Thank you!

First check that you have excluded your template folder in the settings. If that is not set then the %weather3% string may have been removed already. Do note that the default %weather3% and %weather4% strings were designed to be used in a <div> block (See the Example.md file for a demonstration). If you want to use it in a different way you can edit it in the settings. If you still can not get it working, post a copy of your template so I can see if there are any problems in it.
Also the plugins commands will only be available while in edit mode, the commands mostly are for inserting text into the current note and that can not be done while in preview mode so the commands will not be displayed. You should see a notification stating this when you open the command button from the sidebar.
Thanks, William.

Hello William,
Thank you so much for the quick reply! Putting the string in a div block and excluding my templates folder worked.
I'm seeing a stray "%desc" in front of the icon. Any suggestions to fix that?
Thanks!

Glad you got it working. "%desc" is a macro for the current weather description but it is missing the trailing "%". It should be "%desc%". Check what you have in your template.
Thanks William.

Glad you got it working.
Thanks, William.


virt-handler, network: Remove pod interface cache

What this PR does / why we need it:
The virt-handler store at memory the pod cache interface that is
retreived form the shared file with virt-launcher to set the VMI status
IP and IPs from there, to do so it implement a mutex to lock the memory
map. The amaunt of time needed to read the file is not that much. This change remove the cache so the lock
can be removed too.d
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

@AlonaKaplan

/cc @AlonaKaplan

/cc @davidvossel @rmohr
How does this look from your perspective ?

/cc @davidvossel @rmohr
How does this look from your perspective ?

i don't get why we would want to remove this optimization.
The alternative is literally opening a file on disk, reading the contents, and closing the file every time the reconcile loop pops for all VMIs. I understand that it's not that expensive in the grand scheme of things, but if i knew this was happening, i'd create a task to optimize it at some point.
In this pr, we're removing an optimization that i'd approve to re-introduce if someone posted it as a PR. Is there some functional reason for why we'd want to consider this that I am missing?

/cc @davidvossel @rmohr
How does this look from your perspective ?

i don't get why we would want to remove this optimization.
The alternative is literally opening a file on disk, reading the contents, and closing the file every time the reconcile loop pops for all VMIs. I understand that it's not that expensive in the grand scheme of things, but if i knew this was happening, i'd create a task to optimize it at some point.
In this pr, we're removing an optimization that i'd approve to re-introduce if someone posted it as a PR. Is there some functional reason for why we'd want to consider this that I am missing?

The performance difference between opening/read/closing file vs serialize goroutines at the mutex
has to be very similar, that's why this PR reduce a little the complexity removing the map/mutex, but since I don't have numbers either I will close the PR.

The performance difference between opening/read/closing file vs serialize goroutines at the mutex
has to be very similar, that's why this PR reduce a little the complexity removing the map/mutex, but since I don't have numbers either I will close the PR.

The background is normally that the kernel keeps the file in the disk cache if there is no memory pressure. For instance greenhouse (our bazel cache) has no built-in caching mechanism at all. It just reads and writes files transferred via http GET/PUT from/to disk. However the performance is great, since for as long as you have enough spare memory, it basically never touches the real filesystem to retrieve cache entries. The kernel caches it all.

The kernel caches it all.

that makes it seem like reads are super efficient. what about writes though?

The kernel caches it all.

that makes it seem like reads are super efficient. what about writes though?

This cache is only for reads.

/retest

The performance difference between opening/read/closing file vs serialize goroutines at the mutex
has to be very similar, that's why this PR reduce a little the complexity removing the map/mutex, but since I don't have numbers either I will close the PR.

The background is normally that the kernel keeps the file in the disk cache if there is no memory pressure. For instance greenhouse (our bazel cache) has no built-in caching mechanism at all. It just reads and writes files transferred via http GET/PUT from/to disk. However the performance is great, since for as long as you have enough spare memory, it basically never touches the real filesystem to retrieve cache entries. The kernel caches it all.

Then I will not close the PR since looks like reading from file is quite efficient.

@qinqon I tested this. It's going to be  more efficient to use a read through cache with a sync map.
Below is how i tested this. On my machine,  the file read took 21 seconds and the map access for the same number of iterations/goroutines took < 1 second.
package main

import (
	"fmt"
	"io/ioutil"
	"sync"
	"time"
)

func readFile(num int, wg *sync.WaitGroup) {
	for i := 0; i < num; i++ {
		_, err := ioutil.ReadFile("fake-read-file.txt")
		if err != nil {
			panic(err)
		}
		wg.Done()
	}
}

func readMap(num int, wg *sync.WaitGroup, m *sync.Map) {
	for i := 0; i < num; i++ {
		_, ok := m.Load("some-file")
		if !ok {
			panic("no key")
		}
		wg.Done()
	}
}

func main() {
	var wg sync.WaitGroup

	now := time.Now().UTC().Unix()
	for i := 0; i < 100; i++ {
		wg.Add(100000)
		go readFile(100000, &wg)
	}
	wg.Wait()
	newNow := time.Now().UTC().Unix()
	fmt.Printf("File read duration seconds: %d\n", newNow-now)

	m := new(sync.Map)
	m.Store("some-file", "some contents")

	now = time.Now().UTC().Unix()
	for i := 0; i < 100; i++ {
		wg.Add(100000)
		go readMap(100000, &wg, m)
	}
	wg.Wait()
	newNow = time.Now().UTC().Unix()
	fmt.Printf("Map read duration seconds: %d\n", newNow-now)
}

@davidvossel thanks so much for taking the time to do some benchmarking, I have modify it a little so we do propertly the wg adding, since we have to add per gorutine not per VM
package main

import (
    "fmt"
    "io/ioutil"
    "sync"
    "time"
)

func readFile(num int, wg *sync.WaitGroup) {
    wg.Add(1)
    defer wg.Done()
    for i := 0; i < num; i++ {
        _, err := ioutil.ReadFile("fake-read-file.txt")
        if err != nil {
            panic(err)
        }
    }
}

func readMap(num int, wg *sync.WaitGroup, m *sync.Map) {
    wg.Add(1)
    defer wg.Done()
    for i := 0; i < num; i++ {
        _, ok := m.Load("some-file")
        if !ok {
            panic("no key")
        }
    }
}

func main() {
    var wg sync.WaitGroup

    now := time.Now().UTC().Unix()
    for i := 0; i < 100; i++ {
        go readFile(100000, &wg)
    }
    wg.Wait()
    newNow := time.Now().UTC().Unix()
    fmt.Printf("File read duration seconds: %d\n", newNow-now)

    m := new(sync.Map)
    m.Store("some-file", "some contents")

    now = time.Now().UTC().Unix()
    for i := 0; i < 100; i++ {
        go readMap(100000, &wg, m)
    }
    wg.Wait()
    newNow = time.Now().UTC().Unix()
    fmt.Printf("Map read duration seconds: %d\n", newNow-now)
}

@davidvossel does the last numbers make sense to you?

@davidvossel does the last numbers make sense to you?

it shows that both methods occur in under a second in our simulated environment that more closely matches virt-handlers behavior
My tests that use a huge number of iterations and go routines are meant to expose that the kernels file cache isn't on par with the in-memory map. I wasn't trying to replicate a realistic representation of virt-handler.
Here are my thoughts. We don't have control over what the kernel caches or not, especially under memory pressure. Even if we did, we know (based on my program) that the performance of a map out performs opening a file, reading, and closing. Realistically, this optimization of using a map probably doesn't improve much at all under normal conditions, but we have it. so i don't get why we'd remove it.
I don't care enough to block this change or anything, but like I said earlier, i'd also approve someone adding an optimization back in with a sync map... So if I'd approve the optimization, why would I approve removing it?

@davidvossel does the last numbers make sense to you?

it shows that both methods occur in under a second in our simulated environment that more closely matches virt-handlers behavior
My tests that use a huge number of iterations and go routines are meant to expose that the kernels file cache isn't on par with the in-memory map. I wasn't trying to replicate a realistic representation of virt-handler.
Here are my thoughts. We don't have control over what the kernel caches or not, especially under memory pressure. Even if we did, we know (based on my program) that the performance of a map out performs opening a file, reading, and closing. Realistically, this optimization of using a map probably doesn't improve much at all under normal conditions, but we have it. so i don't get why we'd remove it.

As I see it, under memory pressure adding a map caching all the VMs interfaces is going make things worse, for sure it would be safer to delegate the cache to the kernel.

I don't care enough to block this change or anything, but like I said earlier, i'd also approve someone adding an optimization back in with a sync map... So if I'd approve the optimization, why would I approve removing it?

The idea is to do both, first we remove unneeded caching and then prepare a PR using sync.Map as you suggested since is clear from the numbers at your benchmarking that it does not serialize at multiple reads, similar to RW mutex.

So looks like under memory preasure as @davidvossel commented is not a good thing to depend on reading a file since kernel will skip the cache and the map is not a big deal under those conditions, I am going to close this PR.
But since there is not going to be two goroutine handling the same VM is safe to have just one sync.Map with VMI UUID as key and something like VMIContext as value it will include the three cached stuff and without critical regions.


go script not working

Filing an issue that the go script seems to be broken at https://github.com/subscriptions-project/encryption/tree/master/golang/cmd/gcp_key_gen
Is there an alternative we can provide?
cc @jeffochoa

Seems that the Tink team changed their Google Cloud API months ago: https://github.com/google/tink/commit/55a65a01ef9e466152ceb51e43e3658e0f6e0870#diff-03579a9e37886100a9fd05ae2288d60eL96
I'll update the script to use the new functions.

Just verified that the script is working again after the PR submitted last week.