fix: unstate object in Svelte 5 dev mode

...before checking serializability
fixes #11363

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Edits

[ ] Please ensure that 'Allow edits from maintainers' is checked. PRs without this option may be closed.

looks good to me once the test failures are addressed

With https://github.com/sveltejs/svelte/pull/11180 this will need a different approach, since we would need to use runes then.

@dummdidumm Can you not import from svelte/internal?

Feels like there's a deeper question here: when would you ever use $state with pushState? That's not what it's for — it's for associating some current value with a history entry, not some reactive, changing-over-time value. If we're mandating that pushState should be called with serializable data, then that implies that the developer is responsible for calling unstate (or $state.raw or $state.snapshot or whatever it ends up being) themselves. Otherwise we're creating false expectations (that you can pass reactive state into pushState and it can change over time).
In #11363, the argument to pushState comes from preloadData. It feels like a bug that preloadData would return anything other than a static object.

Tinkered locally and I think the right fix for #11363 is to change this line: https://github.com/sveltejs/kit/blob/f71f38149aae0c76b3a58bbbcefbdad16d8392fd/packages/kit/src/runtime/client/client.js#L561
-data: data_changed ? data : page.data
+data: data_changed ? Object.freeze(data) : page.data

$page.data should be immutable, and this conveniently solves the state problem.

I think the underlying problem is that in legacy-client.js in Svelte 5, we're using a proxy to make the props reactive. That has unintended side effects for this.
Another way to solve this is to make devalue more robust against symbolic properties and only error when encountering symbolic properties that are not enumerable (after all, it also ignores those when they're regular string properties).
I don't think that people expect things added to pushState to be reactive - they will never be since they're serialized right away. So I think avoiding the need to call $state.snapshot where possible would be better DX.

Released a new version of devalue with that change

I merged the rennovate PR to bump devalue to v5 here: https://github.com/sveltejs/kit/pull/12141. I have not yet released it


Make uses part of name section

The steps are currently wrong, uses and name are a single section and should not both start with a -.

a

Superseeded by #87


Initial work on compose helper

@martndemus :eyes:

@homu r+

:pushpin: Commit bdd6442 has been approved by poteto

:zap: Test exempted - status

:+1: :trollface:


Fixup the linting.

@sigmavirus24 So I'm unclear why the linter is failing here, do you mind taking a look? The lines that are failing should be # noqad, but somehow that isn't affecting them.


There is no instruction on how to add new languages in the laravel configuration

Feature description
I want to add non english language on lang folder for main laravel directory. But don't see instruction for this. Please add a tutorial to the plugin documentation. Thank you.

Hei! You can follow https://laravel.com/docs/10.x/localization#overriding-package-language-files
But some of menu items has no effect for this setup - is needed some edits

Hei! You can follow https://laravel.com/docs/10.x/localization#overriding-package-language-files
But some of menu items has no effect for this setup - is needed some edits

I am asking the plugin developer about this. Because is not correct to modify the plugin files in the vendor folder.

You no need to modify vendor.
You should add your translation to resources/lang/vendor/filament-blog/<YOUR_SHORT_LANGUAGE>/filament-blog.php
I'm also want to add some translation via Pull Request

You no need to modify vendor. You should add your translation to resources/lang/vendor/filament-blog/<YOUR_SHORT_LANGUAGE>/filament-blog.php
I'm also want to add some translation via Pull Request

You were do not understand me. I'm talking about implementation of multilanguage content in blog plugin (Spatie Translatable) instead of the interface translations.

Thank you for providing a reference @DmitriyDalee
@viterzbayraku you don't need to modify vendor. Follow the guide provided by @DmitriyDalee and I will be happy to merge your PR.

Thank you for providing a reference @DmitriyDalee
@viterzbayraku you don't need to modify vendor. Follow the guide provided by @DmitriyDalee and I will be happy to merge your PR.

Sorry. I have an additional question about Spatie Translatable implementation.


chore(main): release 2.7.1

:robot: I have created a release beep boop
2.7.1 (2023-07-24)
Bug Fixes

upgrade typescript in peerDependencies (#15) (113250b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Maks1mS/nestjs-telegraf/releases/tag/v2.7.1 :sunflower:


fix: fixes the word 'Programados' to 'Programador'

Fixes the word "Programados" to "Programador" on title

LGTM


Docs: new peer usage

Description
This PR updates the documentation with the new liqo peer command

@fra98 Thanks for this nice PR!
One note: I would suggest to add a sentence somewhere in the documentation saying that in-band and out-of-band peerings are no longer available in Liqo 1.0, replaced by a more flexible mechanism (even better to say how the above mechanisms can be implemented right now). Otherwise, we may have people start asking "how do I do in-band peering" or such. Thanks!


xwm: add missing dependency

Existing package

[x] I am the maintainer of this package.

Rolled into a version bump instead (see https://github.com/kisslinux/community/pull/1538)


ServiceRegistration does not unregister a service on destruction of the ServiceRegistration object

While a ServiceRegistration object is bound to the lifetime of the Framework object and the service associated with it will be unregistered when the bundle stops, it would be a good idea to apply RAII to ServiceRegistration and unregister the service in the destructor if it wasn't already.
Minimally an update to the documentation informing users to call ServiceRegistration::Unregister when they are done with the service would be helpful.

There are complications to, at the end of the lifespan of the last ServiceRegistrationBase, calling Unregister on that service.
This call could reasonably be made in the destructor of the ServiceRegistrationBase and the destructor of the ServiceRegistrationBasePrivate. But in either case, it is during the destruction of a ServiceRegistrationBase object (The destructor of the ServiceRegistrationBasePrivate is made during the destruction of the last ServiceRegistrationBase object that owns it).
Within the call to Unregister, as it currently stands, there are calls made to two functions that are potentially problematic:

ServiceRegistry::RemoveServiceRegistration_unlocked(**ServiceRegistrationBase** const& sr)
ServiceFactory::UngetService(Bundle const& bundle,
**ServiceRegistrationBase** const& registration,
InterfaceMapConstPtr const& service)

Both of these functions take a ServiceRegistrationBase object reference as input. The first function, RemoveServiceRegistration_unlocked, does not pose significant issues as it primarily serves to erase ServiceRegistrationBase objects from various data objects, namely ServiceRegistry::services and ServiceRegistry::serviceRegistrations. Additionally it interacts with ServiceRegistrationBasePrivate::properties. The call to Unregister in the new proposal suggests doing so at the end of the lifetime of the final ServiceRegistrationBase object, meaning no more of those Base objects exist within ServiceRegistry::services and ServiceRegistry::serviceRegistrations. The interactions with ServiceRegistrationBasePrivate::properties may need to persist but can be done so without the call to the ServiceRegistry object.
The second function, ServiceFactory::UngetService, is a call made to user implemented code. Either option for implementation will need to pass a reference to a ServiceRegistrationBase object that is currently within its own destructor. Giving the customer a reference to an object within that objects destructor is a poor implementation.
Even if we were to pass the reference back, there are additional challenges in implementation:
If we make the call from Unregister in:

~ServiceRegistrationBase: we must first confirm that this is the last object with a shared_ptr to the same ServiceRegistrationBasePrivate, namely that d.use_count == 1, and then call Unregister only if that is trye. While this is not a problem technically, looking at the use count of a shared_ptr goes against its abstraction.
~ServiceRegistrationBasePrivate: we will need a reference to the parent ServiceRegistrationBase object that is currently being destroyed, but still alive. This could potentially be set in the destructor of that object, but that creates a circular link, first of all, and again breaks abstraction between objects.

While these solutions may work, passing a reference back to a user to an object currently in its own destructor is dangerous, especially considering the user is able to make copies of that object. This in turn increases the references to the ServiceRegistrationBasePrivate object (as each ServiceRegistrationBase object has a shared_ptr to the ServiceRegistrationBasePrivate object) while the ServiceRegistrationBasePrivate object is being destroyed.


Allow subjects to be unretired

From time to time it is desirable to unretire subjects from a workflow.  Typically, this is required when a retirement rule in Caesar is misconfigured, or when a project team wishes to revise a workflow's retirement limit.
Unretirement of a subject on a workflow consists of setting retired_at and retirement_reason back to NULL on the subject_workflow_counts table.  Currently this is accomplished via the Rails console or via direct SQL commands (yikes!).  It would be helpful to add an API endpoint to execute these unretirements given a workflow ID and either a subject ID or a subject set ID.
See this Slack thread for more context.

I'd be up for pairing on this one - @nciemniak this might be a good first issue for you on the API. Thoughts?

@camallen @adammcmaster @nciemniak -- After discussion with @zwolf during Ops Party on 18 Dec 2019, here are some notes and ideas.
Proposed solution would be to provide unretire functionality via the Python Client.  The goal here is to provide a solution that does not require Rails console access (dangerous level of access, even for internal team) but does not require front-end UI tool building (e.g., via Project Builder interface or admin interface).  Client fulfills both of these criteria.  We also discussed whether feature should be admin-only or open, and concluded we might as well allow owners access to the function too.
For this solution, back end work on Panoptes is still required to open up an API update route to subject_workflow_counts (table name) / subject_workflow_status (model name) entries.
Relevant Links:
Route = https://github.com/zooniverse/Panoptes/blob/88fc3c291a0eaeefcefb7dc7dcab8d1aa680b352/config/routes.rb#L141
Model = https://github.com/zooniverse/Panoptes/blob/master/app/models/subject_workflow_status.rb
If this solution is acceptable, I'll issue it up on the Python Client repo.

That sounds fine to me. It should be straightforward to add that to the Client once it's in the API.
Retiring subjects is already supported. My suggestion would be to support a DELETE request to the retired_subjects endpoint

I don't think switching to a HTTP delete verb will work well as DELETE requests may not support a payload body, https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods/DELETE and https://tools.ietf.org/html/rfc7231#section-4.3.5
A payload within a DELETE request message has no defined semantics;
sending a payload body on a DELETE request might cause some existing implementations to reject the request.

Also as we do not have a WorkflowRetiredSubject resource that we reference directly via an ID then we would have to pass the relevant subject ids via URL path (singluar retirement) or using query params (supports batches with list e.g. ?subject_ids=1,2,3,4)
Taking the above into consideration it may be simpler and more intuitive to add a similar POST request to unretire a subject for a known workflow, e.g.
 POST /api/workflows/1/unretire_subject with a payload body of subject_ids={1,2,3,4}

Possibly use the code in https://gist.github.com/camallen/44972b25a3359b649f5338d23fb203d2 as a template for this work

This would be super handy for 1715 Labs stuff too (cc @tingard)

(I would like to 👍 this)

One specific part of this feature request: in addition to passing a subject_ids payload, I'd also like to support passing subject_set_ids too.  As seen in Cam's script code linked above, most of the time we are looking to unretire all or some subject sets linked to a workflow.  A subject_set-based option would be very helpful!


New term: ShinyNGS

Please could @drmarkreuter provide:

homepage
definition (i.e. purpose)
licence info

Apologies for a lack of official term suggestion guidelines - we're volunteer run, and have to budget our time!
Split from #10.
Feel free to add any others to the issue tracker. Thanks!

shinyNGS
https://github.com/pinin4fjords/shinyngs
Shinyngs is an R package designed to facilitate downstream analysis of RNA-seq and similar expression data with various exploratory plots and data mining tools.
GNU Affero General Public License v3.0

Assigned to http://www.ebi.ac.uk/swo/SWO_1100129


[Client connection Problem] - Can't connect to a world -  Fabric 1.19.2

Hey there,
when your mod is installed on the client, i cant connect to a singleplayer world / multiplayer world...
Both not working , client is stuck somehow.
No crash log or console entries
Fabric 1.19.2
Fabric Loader 0.14.9
ToolTip+ 1.3.0

i have no trouble with 1.3.0+1.19/fabric loader 0.14.9 on 1.19.2. perhaps it's another of your mods?
without a crash report or logs i can't really do anything

client latest.log

doesn't seem to be a tooltipsplus issue... if you remove only tooltipsplus does the issue persist?

No, it only happens when i add Tooltipplus

Maybe its a mod of those:
https://www.curseforge.com/minecraft/mc-mods/tooltipfix
https://www.curseforge.com/minecraft/mc-mods/equipment-compare-fabric
https://www.curseforge.com/minecraft/mc-mods/enchantment-descriptions

Or its Anivl Display
https://www.curseforge.com/minecraft/mc-mods/anvil-display

i'm not sure what's causing this, but i don't see how it could be tooltipsplus...
i'd recommend a binary search-- remove half your modlist and see if the problem persists; if it does, remove half of your remaining mods; if it doesn't, put back half of the mods you removed.


Add an option to force decoding buffers to mono

Don't really know how to do that without some extra work, but could be handy in RPi

released in v1.1.0


Bug: NPC to player damage missing?

https://cdn.gamelockerapp.com/semc-vainglory/na/2017/10/07/09/26/948661b6-ab41-11e7-bc6d-0a5864605119-telemetry.json
I can't find any event that has type == DealDamage and payload.IsHero != 1, but minions and turrets deal damage to players

Minions are disabled and turrets as well. 2.9 will enable Kraken DealDamage to anything.


Bug: CD pipeline failing on "Custom Management Group Diagnostic Settings" step due to mission permission for User-Assigned Managed Identity

Is there an existing issue for this?

[X] I have searched the existing issues

Infrastrcuture as Code Type? (Required)
bicep
Module Versions (Required)
ALZ PowerShell Module: 3.0.4
Accelerator Bootstrap Modules: 3.1.2
Terraform Starter Modules: N/A
Bicep Starter Modules: 0.19.2
Input arguments of the ALZ-PowerShell-Module (Optional)
No response
Debug Output/Panic Output (Optional)
No response
Expected Behaviour (Required)
The "02 Azure Landing Zones Continuous Delivery" pipeline should have successfully performed a "Deploy" for the "Custom Management Group Diagnostic Settings" step.
Actual Behaviour (Required)
The "02 Azure Landing Zones Continuous Delivery" pipeline kept looping when performing a "Deploy" for the "Custom Management Group Diagnostic Settings" step.
Manually Formatted Output:
15:12:38 - Error: Code=InvalidTemplateDeployment; Message=Deployment failed with multiple errors:
'Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-platform-management/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-decommissioned/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-landingzones/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-platform/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-sandbox/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-landingzones-corp/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-landingzones-online/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-platform-connectivity/providers/Microsoft.Insights/diagnosticSettings/toLa'.
:Authorization failed for template resource 'toLa' of type 'Microsoft.Insights/diagnosticSettings'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Insights/diagnosticSettings/write' at scope '/providers/Microsoft.Management/managementGroups/alz-platform-identity/providers/Microsoft.Insights/diagnosticSettings/toLa'.'

It looks like the "id-alz-mgmt-eastus2-apply-001" User-Assigned Managed Identity doesn't have the "Microsoft.Insights/diagnosticSettings/write" permission.
Steps to Reproduce (Optional)
No response
Important Factoids (Optional)
This is the first CD pipeline run using the new accelerator.
The Azure resources had been deployed previously using the "classic" accelerator.
References (Optional)
No response

Hi @cspring86, sometimes these permissions take some time to fully propagate. I have seen instances of them taking 30-45 minutes. Can you please re-run the deployment, and let me know if you're still running into the error?

@oZakari I'm still getting the same error. It looks like the "id-alz-mgmt-eastus2-apply-001" User-Assigned Managed Identity needs additional permissions.

I haven't been able to replicate this one after doing multiple deployments over the top of an existing deployment. I have also not heard of others with this permissions issue. Any particular parameters you adjusted, or did you happen to make any changes to the diagnostic module directly? Also, if you add the permission, does the deployment complete or does it error with other missing permissions?

I haven't been able to replicate this one after doing multiple deployments over the top of an existing deployment. I have also not heard of others with this permissions issue. Any particular parameters you adjusted, or did you happen to make any changes to the diagnostic module directly? Also, if you add the permission, does the deployment complete or does it error with other missing permissions?

Interesting.
Doing a bit of searching, I have found that the ALZ-Bicep is creating the relevant diagnostic settings that are causing the error: https://github.com/Azure/ALZ-Bicep/blob/main/infra-as-code/bicep/modules/mgDiagSettings/mgDiagSettings.bicep#L18. The parDiagnosticSettingsName property is "toLa", which can be seen in the error messages.
I've also searched the accelerator-bootstrap-modules repository and can see that the "Azure Landing Zones Management Group What If (alz-mgmt)" custom role is granted the Microsoft.Insights/diagnosticSettings/write permission (here), but the "Azure Landing Zones Management Group Contributor (alz-mgmt)" custom role is not (here).
I think that this is the issue and also explains why the CI (What If) part of the pipeline passes, but the CD part doesn't.
I'd, naively, possibly, expect that the Contributor role would have the same base permissions as the Reader role and then additional "write" permissions on top of these. However, I can see that the Microsoft.Insights/diagnosticSettings/write permission isn't the only one that not also in the Contributor role.

Ok, so I've manually granted the "Azure Landing Zones Management Group Contributor (alz-mgmt)" custom role the Microsoft.Insights/diagnosticSettings/write permission and that step passes now.
However, it is now failing on the "Built-in and Custom Policy Assignments Deployment" step.
The error is this:
Error: Code=InvalidTemplateDeployment; Message=Deployment failed with multiple errors: 'Authorization failed for template resource 'Deploy-MDEndpoints' of type 'Microsoft.Authorization/policyAssignments'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Authorization/policyAssignments/write' at scope '/providers/Microsoft.Management/managementGroups/alz/providers/Microsoft.Authorization/policyAssignments/Deploy-MDEndpoints'.:Authorization failed for template resource 'Deploy-MDFC-Config-H224' of type 'Microsoft.Authorization/policyAssignments'. The client 'xxx' with object id 'xxx' does not have permission to perform action 'Microsoft.Authorization/policyAssignments/write' at scope '/providers/Microsoft.Management/managementGroups/alz/providers/Microsoft.Authorization/policyAssignments/Deploy-MDFC-Config-H224'.'

Funnily enough, this is another one of the permissions granted to the "Azure Landing Zones Management Group What If (alz-mgmt)" custom role and not the "Azure Landing Zones Management Group Contributor (alz-mgmt)" custom role.
I think it makes sense to grant all permissions that the Reader role has to the Contributor role as the baseline.

Adding these 2 permissions from the Reader role to the Contributor role gets the CD pipeline working:
Microsoft.Insights/diagnosticSettings/write
Microsoft.Authorization/policyAssignments/write

It might be also worth adding some timeouts to the pipelines, even if they're oversized because these errors don't appear to cause the step to fail, it just keeps re-trying. I didn't wait to see if it ever did fail though, so it may fail eventually on its own.

Thanks for the detailed information @cspring86. Some of the inconsistencies are due to how the what-if operator works. However,  I have created a PR to add both of these permissions.

Amazing, thanks @oZakari !


[Do not merge] Frontend: Create account page

changes

add createAccountPage
linked from loginPage
validation

required fields
type=email for regex email format check
type=password to hide password and show dots

successfully creates an account on firebase
accompanying success/fail alert messages

LGTM!! thank you for the hard work :-) just have to resolve the merge conflicts!!

this PR/resolving conflict is looking very suspicious.. shouldn't have more deleted lines than new ones...
will create another PR after fresh pull from /main branch just to be extra safe
no. of line changes:


Adding Python bytecode file overlook

Adding Python bytecode file overlook
TODO
https://stackoverflow.com/questions/2998215/if-python-is-interpreted-what-are-pyc-files

Hi, looks like it was already added - https://github.com/776948741/gitignore/blob/1b120f876352da00200c2daff1f3f3173f603abc/Python.gitignore#L3

Yes, I added this ignored item in the warehouse of my fork. There is no ignore item in the version of fork source. When PyC executes Python program, it will automatically generate for accelerating operation. It should be managed without pushing to the version library

Closed as Line 3 should already cover this rule.


Add possibility to write custom regex to convert Terminal path on Ctrl+Click, same as problemMatcher

Adding the possibility to write your own regexps to convert the file paths in the Terminal for a certain workspace would be very useful, and make Ctrl+Click on compiler errors work in cases when it is not possible at the moment.
We use Docker as a container when we build code. The docker is started when we trigger a build and is not active otherwise. This means the paths the compiler sees are not the same as the paths that Visual Studio Code or the rest of the machine sees. Therefore Ctrl+Click on compiler errors don't work.
For example code might be located at:
/home/my_name/git/path/to/code/code_with_error.cpp
The VSC workspace placed at
/home/my_name/git/path/to/
However the compiler that runs in Docker prints:
/home/another_user/folder/application/path/to/code/code_with_error.cpp
Ctrl+Click in the Terminal tries to open the path the compiler printed, and because that is a path inside the Docker it is not found. It is possible to write a ProblemMatcher with a regexp to identify the file and get i listed in the PROBLEMS tab. It would be very useful if it was possible to do something similar for links in the Terminal, per workspace. In the example above the user of VSC could write a custom regexp to extract the part code/code_with_error.cpp and VSC could look for that file relative to ${workspaceFolder} in the same way as the problemMatcher does.

We don't want to add any docker/container smarts into the core, for that it's best to run using the dev containers extension which will also lead to a better overall experience.

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


[BUG] AppCertificateCollection.CreateOrUpdateAsync() throws ArgumentException because of keyVaultId being an empty string in response from Azure, not null

Library name and version
Azure.ResourceManager.AppService (1.0.0)
Describe the bug
AppCertificateCollection.CreateOrUpdateAsync() throws ArgumentException with stack trace:
   at Azure.Core.Argument.AssertNotNullOrEmpty(String value, String name)
   at Azure.Core.ResourceIdentifier..ctor(String resourceId)
   at Azure.ResourceManager.AppService.AppCertificateData.DeserializeAppCertificateData(JsonElement element)
   at Azure.ResourceManager.AppService.CertificatesRestOperations.<CreateOrUpdateAsync>d__15.MoveNext()
   at Azure.ResourceManager.AppService.AppCertificateCollection.<CreateOrUpdateAsync>d__5.MoveNext()

From my investigation, this error occurs as it tries to deserialize the response from Azure (because the certificate does get uploaded successfully).
I believe the cause is that the Azure.ResourceManager.AppService.AppCertificateData.DeserializeAppCertificateData(JsonElement element) method checks if an ID is null - if it is, it will skip to the next property - if it isn't it will create an instance of ResourceIdentifier with that ID passed in as a parameter, but the null check does not account for the possibility that the value might not be null and could just be an empty string - and when an empty string is passed in the constructor of ResourceIdentifier, it throws an exception.
The ID in particular that might be causing this is keyVaultId, as when I send a PUT request to https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Web/certificates/{name}?api-version=2022-03-01 with JSON body:
{
	"location":"UK South",
	"properties": {
		"serverFarmId":"<server farm id>",
                "password":"<pfx password>",
		"pfxBlob":"<pfx blob>"
	}
}

the response from Azure returns an empty string ("") for keyVaultId property in the JSON response, not null.
Expected behavior
The AppCertificateCollection.CreateOrUpdateAsync() method should:

Send a PUT request to the Azure certificates API endpoint with the body from AppCertificateData
Return an AppCertificateResource object with the parsed data from a successful response

Actual behavior

Sends a PUT request to the Azure certificates API endpoint with the body from AppCertificateData
Throws ArgumentException during the deserialization of the response with the stack trace:

   at Azure.Core.Argument.AssertNotNullOrEmpty(String value, String name)
   at Azure.Core.ResourceIdentifier..ctor(String resourceId)
   at Azure.ResourceManager.AppService.AppCertificateData.DeserializeAppCertificateData(JsonElement element)
   at Azure.ResourceManager.AppService.CertificatesRestOperations.<CreateOrUpdateAsync>d__15.MoveNext()
   at Azure.ResourceManager.AppService.AppCertificateCollection.<CreateOrUpdateAsync>d__5.MoveNext()

The certificate was uploaded successfully and shows up in the Azure Portal.
Reproduction Steps
Send a request to create or update a certificate with these properties in AppCertificateData object:
var certificates = resourceGroup.GetAppCertificates();
await certificates.CreateOrUpdateAsync(Azure.WaitUntil.Completed, "MyCertificateName", new AppCertificateData(AzureLocation.UKSouth)
{
    PfxBlob = Convert.FromBase64String("<pfx in base64>"),
    Password = "<pfx password>",
    ServerFarmId = new ResourceIdentifier($"<server farm id>")
});

Then the exception is thrown
Environment
Windows 11 .NET 6 - Visual Studio 17.4.1

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'KeyVault:0.45466357,ARM:0.22375463,Azure.Identity:0.10263796'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

We already have an issue tracking this https://github.com/Azure/autorest.csharp/issues/2802

Duplicate of #31701


Contact photo

Hello, is it possible to get user photo?
like that /ews/Exchange.asmx/s/GetUserPhoto?email=XXXX&size=HR648x648

@akam-it currently not, pull requests are welcome with lib/exchanger/operations/get_user_photo.rb


Crashing after installing Vigil 06

=== Maid Fiddler DUMP ===
Game version: 127
Mod version: BETA 0.9b
Info: Failed to update player values
Error message: System.NullReferenceException: Object reference not set to an instance of an object
at CM3D2.MaidFiddler.Plugin.Gui.MaidFiddlerGUI+PlayerInfo.UpdateDays () [0x00000] in :0
at CM3D2.MaidFiddler.Plugin.Gui.MaidFiddlerGUI+PlayerInfo.UpdateAll () [0x00000] in :0
Game worked fine before, I downloaded and sued the updater for Vigil skills 06, now error. I ahve played around with and searched for a few hours now (usually I can find a way to fix things, but this one has me beaten)
Tried to re-install maid fidler, tried to re-install the vigil pack, tried to re-install C-sharp assembly. Each time reipatcher showed no errors and everything was a success
Game works fine after deleting the files vigil 06 added.
Still breaks when using the vigil 06 files.
Halp?

I downloaded and sued the updater for Vigil skills 06

Which additional content do you mean by "Vigil Skills 06"? If it is the Yotogi Class Skills Vol.6, it requires CM3D2 version 1.31 or later to work.
In addition, when does the error occur? Everything seems to work fine on my end.
In any case, consider updating CM3D2 to the latest version (according to the error dump you have version 1.27 installed).
If the error still persists even after updating CM3D2, either reopen this issue or open a more detailed one.


Errors when building the latest flash-attn (2023-07-29) with Ninja

Errors when building flash-attn with Ninja
I've been using pip install flash-attn --no-build-isolation  (2023-07-29)
Related environment information:

CUDA version: 11.7
PyTorch version: 2.0.1

Reading the installation instructions with the error message, I realized that it was an error in the Ninja compilation
According to a reply under Issue #358, if you don't set MAX_JOBS, it's very likely that Ninja's parallel build will consume so much resource that the system can't handle it.
In fact, the installation instructions also point this out:

If your machine has less than 96GB of RAM and lots of CPU cores, ninja might run too many parallel compilation jobs that could exhaust the amount of RAM. limit the number of parallel compilation jobs, you can set the environment variable MAX_JOBS: `MAX_JOBS=4 pip install flash-attn --no-build- isolation

Information about the devices on the server I'm using:

72 CPU cores
RAM: 197G/376G free/total

              total used free shared buff/cache available
Mem: 376G 62G 197G 12G 116G 298G
Swap: 29G 24G 5.8G

But it seems that the server I'm using has more than 96GB of free CPU RAM, so theoretically it shouldn't cause an error by not setting MAX_JOBS.

ChatGPT:
ninja is an efficient parallel build system that can start multiple compilation tasks at the same time to speed up builds.
By default, ninja attempts to use all available CPU cores to build projects in parallel, which can lead to excessive consumption of system resources, especially on systems with low memory.
MAX_JOBS is an environment variable that limits the number of parallel build tasks for ninja. The number of parallel build tasks can be controlled by setting the MAX_JOBS environment variable to avoid excessive consumption of system resources. Typically, a reasonable value would be twice the number of CPU cores.

In practice, however, error messages do vary with the MAX_JOBS setting.
Errors with different MAX_JOBS settings
MAX_JOBS=1
MAX_JOBS=1 pip install flash-attn --no-build-isolation

Key error message:
/tmp/pip-install-npwm31a8/flash-attn_fe2073454d9241589566a66867aa6cfb/csrc/flash_attn/src/flash_fwd_launch_template.h(118): here
          instantiation of "void run_mha_fwd_hdim128<T>(Flash_fwd_params &, cudaStream_t) [with T=cutlass::bfloat16_t]"
/tmp/pip-install-npwm31a8/flash-attn_fe2073454d9241589566a66867aa6cfb/csrc/flash_attn/src/flash_fwd_hdim128_bf16_sm80.cu(18): here

26 errors detected in the compilation of "/tmp/pip-install-npwm31a8/flash-attn_fe2073454d9241589566a66867aa6cfb/csrc/flash_attn/src/flash _fwd_hdim128_bf16_sm80.cu".
ninja: build stopped: subcommand failed.
ninja: build stopped: subcommand failed.

ninja: build stopped: subcommand failed.
      subprocess.CalledProcessError: Command '['ninja', '-v', '-j', '1']' returned non-zero exit status 1.

A similar error was reported in Issue #358

I tried with MAX_JOBS=1 and see.

      /tmp/pip-install-mvmpcn8m/flash-attn_39e3b6d4eaad444b8beae75cad8aecd2/csrc/flash_attn/src/flash_bwd_launch_template.h
(179): error: expression must have a constant value
      Note #2767-D: the value of *this cannot be used as a constant
                detected during instantiation of "void run_mha_bwd_hdim32<T>(Flash_bwd_params &, cudaStream_t, __nv_bool) [
with T=cutlass::bfloat16_t]"
      /tmp/pip-install-mvmpcn8m/flash-attn_39e3b6d4eaad444b8beae75cad8aecd2/csrc/flash_attn/src/flash_bwd_hdim32_bf16_sm80.
cu(15): here

      /tmp/pip-install-mvmpcn8m/flash-attn_39e3b6d4eaad444b8beae75cad8aecd2/csrc/flash_attn/src/flash_bwd_launch_template.h
(179): error: expression must have a constant value
      Note #2767-D: the value of *this cannot be used as a constant
                detected during instantiation of "void run_mha_bwd_hdim32<T>(Flash_bwd_params &, cudaStream_t, __nv_bool) [
with T=cutlass::bfloat16_t]"
      /tmp/pip-install-mvmpcn8m/flash-attn_39e3b6d4eaad444b8beae75cad8aecd2/csrc/flash_attn/src/flash_bwd_hdim32_bf16_sm80.
cu(15): here

      2 errors detected in the compilation of "/tmp/pip-install-mvmpcn8m/flash-attn_39e3b6d4eaad444b8beae75cad8aecd2/csrc/f
lash_attn/src/flash_bwd_hdim32_bf16_sm80.cu".

EDIT: That looks like #343

where Issue #343 proposes a PR that has been accepted and shouldn't pose a problem
MAX_JOBS=2/4/72
The following is an example of MAX_JOBS=2
Command run:
MAX_JOBS=2 pip install flash-attn --no-build-isolation

Key error message:
...
      ptxas info : Function properties for _Z25flash_bwd_dot_do_o_kernelILb1E23Flash_bwd_kernel_ traitsILi96ELi64ELi128ELi8ELi2ELi4ELi4ELb0ELb0EN7cutlass6half_tE19Flash_kernel_traitsILi96ELi64ELi128ELi8ES2_EEEv16Flash_bwd_ params
          0 bytes stack frame, 0 bytes spill stores, 0 bytes spill loads
      ptxas info : used 48 registers, 696 bytes cmem[0]
      ninja: build stopped: subcommand failed.

subprocess.CalledProcessError: Command '['ninja', '-v', '-j', '2']' returned non-zero exit status 1.

ChatGPT:
This error comes from NVIDIA's CUDA compiler, ptxas, and suggests that the compiler encountered a problem compiling the CUDA code. Specifically, the error message indicates that when compiling a CUDA function named _Z25flash_bwd_dot_do_o_kernel, the compiler was unable to fulfill the resources required for the function.
According to the error message, the function requires 48 registers and 696 bytes of shared memory (cmem). The registers are used to store variables and state within the function, while the shared memory is used to share data within the thread block.

It looks as if MAX_JOBS=2 is over the system resource limit? (weird)

Did you find any solution?

Any solution?

I tried to build the docker in a different machine and It worked.


Potential exception in destructor of http_async_server and async_echo_server classes

Hello,
My friend came across the pattern that exists in the following classes:
examples/http_async_server.hpp
test/websocket/websocket_async_echo_server.hpp
Both repeatedly call join() on std::thread-s from a vector thread_ like this:
~http_async_server()
{
    error_code ec;
    ios_.dispatch(
        [&]{ acceptor_.close(ec); });
    for(auto& t : thread_)
        t.join();
}

This is a potential issue as std::thread::join() can throw an exception:
http://en.cppreference.com/w/cpp/thread/thread/join
and since C++11 all the destructors are implicitly noexcept which means std::terminate will be called should an exception occur.
While both these classes are not part of the library core, it seems this code should be improved to provide better samples as most of the library users will rely on the provided code snippets while writing their own applications.

Thank you for your interest in Beast! However, I believe that the current implementation is correct. Here are the conditions under which std::thread::join can generate an error:

this->get_id() == std::this_thread::get_id() (deadlock detected)
the thread is not valid
invalid_argument if joinable is false

None of these scenarios are applicable. If it makes you feel better, you can add assert(t.joinable()) before the call to join to make it clear that the author of the code understands what they are doing.

Hi Vinnie,
Thanks for the explanation! That makes sense. Please disregard this issue.
And thanks for your library, it is really nice!

I appreciate the kind words!


fix: delete pod who is terminating

What type of PR is this?

/kind bug
What does this PR do?

Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

NONE

/lgtm

/approve


Don't set vim.o.termguicolors = true

Setting vim.o.termguicolors = true in latest nightly causes the background color to change temporarily during loading. By default background is not going to be set temporarily if we usevim.o.termguicolors = false. This has been discussed in this issue https://github.com/neovim/neovim/issues/26372 and wasn't really resolved.
See videos below.
Without termguicolors:
https://github.com/navarasu/onedark.nvim/assets/14967831/b4ce1ff2-4ff0-4eb2-a15f-ac7c4280831b
With termguicolors:
https://github.com/navarasu/onedark.nvim/assets/14967831/b0807aa6-e1e0-4599-9671-6481f8556c3b
By not setting termguicolors = true we comply with the current requirements for neovim for consistent background. If the user needs the termguicolors = true they can always set it in their config. I am not sure why this setting was needed in this plugin in the first place, though. Please let me know if I am missing something.

The upstream issue was fixed: https://github.com/neovim/neovim/pull/28676
But I'd still argue that the termguicolors doesn't necessarily have to be set by plugin.


this not accessible within queue

Hi, apologies, but I've spent an hour banging my head against the desk, as its not clear where the context of this has gone missing.
swal.queue([{
                title: 'Your public IP',
                confirmButtonText: 'Show my public IP',
                text:
                'Your public IP will be received ' +
                'via AJAX request',
                showLoaderOnConfirm: true,
                preConfirm: function () {

                    return new Promise(resolve => {

                        this.http.get('https://api.ipify.org?format=json')
                            .map(res => res.json()).subscribe(data => {
                            console.log(data.results);
                            //this.data = data.results;
                            swal.insertQueueStep(data.ip)

                            resolve();
                        });
                    });
});

                }
            }])

Using Ionic3, Angular 4. How do I insert the this context effectively?  At the moment it doesn't know how to see this.http

If your issue is focused more on 'how do I do this?' rather than 'this is a broken functionality', then either ask on our Gitter or submit it to StackOverflow instead of GitHub, and make sure you add the sweetalert tag to the question.

See Contributing.md

i faced same issue with JQuery before and this simple solution
// used to reference to this
let self :MyClass;

@Component({
  selector: 'my-class',
  templateUrl: 'my-class.html'
})
export class MyClass{

  constructor() {
    self = this ;
  }

doMyStuff(){
        swal.queue([{
                title: 'Your public IP',
                confirmButtonText: 'Show my public IP',
                text:
                'Your public IP will be received ' +
                'via AJAX request',
                showLoaderOnConfirm: true,
                preConfirm: function () {

                    return new Promise(resolve => {

                        self.http.get('https://api.ipify.org?format=json')
                            .map(res => res.json()).subscribe(data => {
                            console.log(data.results);
                            //self.data = data.results;
                            swal.insertQueueStep(data.ip)

                            resolve();
                        });
                    });
                  });

                }
            }])

}

}


UnhandledPromiseRejectionWarning: Error: Invalid TF_Status: 3 - Message: Invalid GIF data (size 20692), failed to slurp gif file: Image is defective, decoding aborted

Hello,
When I use:
const imageBuffer = fs.readFileSync(path);
const image = tf.node.decodeImage(imageBuffer, 3);

I have this error:
(node:309911) UnhandledPromiseRejectionWarning: Error: Invalid TF_Status: 3
Message: Invalid GIF data (size 20692), failed to slurp gif file: Image is defective, decoding aborted
    at NodeJSKernelBackend.executeSingleOutput (/data/nsfwjs/node_modules/@tensorflow/tfjs-node/dist/nodejs_kernel_backend.js:211:43)
    at NodeJSKernelBackend.decodeGif (/data/nsfwjs/node_modules/@tensorflow/tfjs-node/dist/nodejs_kernel_backend.js:376:21)
    at /data/nsfwjs/node_modules/@tensorflow/tfjs-node/dist/image.js:165:54
    at /data/nsfwjs/node_modules/@tensorflow/tfjs-core/dist/tf-core.node.js:4425:22
    at Engine.scopedRun (/data/nsfwjs/node_modules/@tensorflow/tfjs-core/dist/tf-core.node.js:4435:23)
    at Engine.tidy (/data/nsfwjs/node_modules/@tensorflow/tfjs-core/dist/tf-core.node.js:4424:21)
    at Object.tidy (/data/nsfwjs/node_modules/@tensorflow/tfjs-core/dist/tf-core.node.js:10476:19)
    at decodeGif (/data/nsfwjs/node_modules/@tensorflow/tfjs-node/dist/image.js:164:19)
    at /data/nsfwjs/node_modules/@tensorflow/tfjs-node/dist/image.js:211:27
    at /data/nsfwjs/node_modules/@tensorflow/tfjs-core/dist/tf-core.node.js:4425:22
(node:309911) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:309911) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

How can I fix this?
Thanks!

Can you try with a different image? The error says that the image is an invalid GIF. The error is coming from TFJS not our code, so if you need further assistance with debugging it would probably be best to post an issue on their repo.


Correct Lifecycle flow at bootstrap time

This is my intent (choose one)

[x] I want to report a bug
[ ] I want to request a feature or change
[ ] I want to provide or change a feature

The problem
Before Hops V 7, there was a clear order for the calls of lifecycle methods. The async bootstrap had been called and finished before enhanceElement. This is currently used to fetch async data before rendering to the DOM, so Hops 7 it breaks in current applications.
Proposed solution
Three scenarios should be supported:

doing asynchronous stuff before creating  the element (see example above)
doing asynchronous staff before rendering and after creating the element (usecase: apollo graphql client)
composition must still be supported

Possible solution:
Turn enhanceElement into a Promise, that receives a Promise from the chain before and returns again a Promise. This Promise should in any case resolve to a renderable Element.
Example (no runnable code, just to explain the principle):
// using jsx for clarification, should be es5
enhanceElement: function (prev) {
  let data = {};

  return fetchSth()
    .then(remoteData => data = remoteData)
    .then(prev)
    .then(el => <MyProvider data={data}>{el}</MyProvider>)
    .then(el => {
      return doSthAsync()
        .then(Promise.resolve(el));
    });
}

Hey @matthias-reis, thanks for reporting this! I really like your idea, but I thought this was already fixed in https://github.com/xing/hops/pull/254 and released with hops-react v7.3.2. Can you confirm?

Yepp, you"re right ...


Error in decora setup KeyError: 'name'

Home Assistant release with the issue:
0.101.0.dev0
Last working Home Assistant release (if known):
unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Arch Linux, Hass Develpment installation (venv)
Integration:
decora
Description of problem:
I work on #27284 and discovered that if I put the config from the documentation into my configuration.yaml I'll get an KeyError. I don't own a decora device but if I supply the namein the config, It works as expected (see config below)
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
light:
  - platform: decora
    devices:
      00:21:4D:00:00:01:
        api_key: 003026

With the following config, the problem does not appear:
light:
  - platform: decora
    devices:
      00:21:4D:00:00:01:
        name: "DecoraDevice"
        api_key: 003026

Traceback (if applicable):
2019-10-14 17:52:18 ERROR (MainThread) [homeassistant.components.light] Error while setting up platform decora
Traceback (most recent call last):
  File "/home/bouni/projects/home-assistant/homeassistant/helpers/entity_platform.py", line 150, in _async_setup_platform
    await asyncio.wait_for(asyncio.shield(task), SLOW_SETUP_MAX_WAIT)
  File "/usr/lib/python3.7/asyncio/tasks.py", line 442, in wait_for
    return fut.result()
  File "/usr/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/bouni/projects/home-assistant/homeassistant/components/decora/light.py", line 63, in setup_platform
    device["name"] = device_config[CONF_NAME]
KeyError: 'name'

Additional information:
I'm not sure but I think if a device_config.get(CONF_NAME) would be used here the problem could be easily fixed.
Never the less, the documentation of the component is incomplete as it doesn't mention the name at all!

@Bouni the documentation should be updated to include and require the name configuration variable. Otherwise Home Assistant would have to generate a name automatically which is difficult and not very user-friendly by only using the bluetooth address as an input.

I think the formatting for the documentation page is also wrong since not all configuration options are shown as you mentioned.
Compare:
{% configuration %}
devices:
  description: A list of lights to use.
  required: true
  type: map
  keys:
    mac_address:
      required: true
      description: The bluetooth address of the switch.
      type: string
      name:
        description: The name to use in the frontend.
        required: false
        type: string
      api_key:
        description: The API key to access the device.
        required: true
        type: string
{% endconfiguration %}

@springstan I agree, but then the name must be required not optional because device["name"] = device_config[CONF_NAME] fails if no name is set!
Using device["name"] = device_config.get(CONF_NAME) is not an option in my opinion because then HA needs to create a default name.

@Bouni I agree that name must be set to required to solve this probelm.
However, I am not sure if this is the best possible solution as name is optional in most integrations.
Therefore, I believe we have to stay consistent to make it easier for new users.
As a consequence, some default value for name should be used by Home Assistant if no value for name is listed in configuration.yaml.
What do you think?

I'm not sure how to handle this. Maybe the name should feault to decora_<mac_address> or something like that.

Yeah that would be a good idea, not sure if Home Assistant allows colons in an entity name though.

Yeah that would be a good idea, not sure if Home Assistant allows colons in an entity name though.
I would use slugify anyway to create a valid name, that replaces the colons with underscores.
M;aybe not a nice looking name but the user can replace it by setting name.


Add option to bundle code manually

Description
I noticed that regenerator-runtime/runtime.js appears in our client-side JS bundle, which we don't need because we don't support IE11, and does add a significant amount of unnecessary JS. Would be great to have the option to bundle/compile the package ourselves so that our own bundle options can be used.
I'm by no means a npm package expert, so I might be wrong, maybe this is already possible in some way. Also not sure what the the solution would be, maybe if the src directory is also included when installing the package (by adding it to the files array in package.json) I could just include the files straight from that src directory? Or possibly the module option in package.json should be used which points to a version compiled without babel transformations.
Steps to reproduce
Expected Behavior
Actual Behavior
Metadata
typesense: 1.3.0-6
OS: Windows/Firefox

@ArcoMul I'm also not an expert at npm packaging, but happy to include the src folder in the package if that helps.
To verify this, could you clone this repo locally, use npm link and see if you're able to achieve what you have in mind by importing directly from src?

Moving all typesense js packages to pure ESM is the real and modern solution to this, I believe.
Angular for example gives warnings about CJS dependenices: https://angular.io/guide/build#configuring-commonjs-dependencies
Here's a good blog about the why (also linked int the Angular guide): https://web.dev/commonjs-larger-bundles/

@jasonbosco I'm able to import TypesenseInstantSearchAdapter from the src folder, so including that folder would solve it in my case.

@jasonbosco Would be great if you could release an update where the src folder is included? Would benefit us a lot! :-)

@ArcoMul I've published 2.4.1 with the src directory included.


Update documentation for pyedt.downloads.download_file()

Description of the current limitations
The method documentation has no reference to the file source. That is, the example-data repository should be mentioned.
Code sample expected
Update docstrings and method signature for pyaedt.downloads.download_file()
Useful links and references
https://github.com/ansys/example-data/tree/master/pyaedt

Resolved with PR #4511


CSS class binding with class names with dashes

How are class name with dashes handled? for example "my-class-name". Thanks

Closing as rivets handles this fine!


feat(fields): add required attribute to checkbox component

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Just like #501

@RbkCp Thanks a lot for your PR !


WIP: Refactor the Quint specification after #144

What ❔
Following the pseudo-code updates in #144, this PR introduces updates in the Quint specification, namely, moving CommitQC into TimeoutQC and storing only high_commit_qc_view in TimeoutVote instead of storing high_commit_qc.
Why ❔
This update speeds-up model checking times dramatically, as CommitQC fields in TimeoutVote were a major bottleneck.
~This change has broken multiple invariants. Hence, we have to further fix the specification and the invariants before merging.~
Preliminary experiments show that the model checking times have improved dramatically. For instance, we were able to find the expected violation of agreement for N=6, F=1, and B=2.

I have fixed committed_blocks_have_justification_inv today and the model checker has not found counterexamples today. So I will re-run the experiments to see whether all the invariants pass.


Example Link is broken

Example Link is broken

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

ID: 6d4a1a18-d470-6a2e-30d3-8f89a82aaaf8
Version Independent ID: f1e8d3e4-e92e-f8e4-85fd-666101347b2f
Content: How to: Inspect assembly contents using MetadataLoadContext
Content Source: docs/standard/assembly/inspect-contents-using-metadataloadcontext.md
Product: dotnet
Technology: dotnet-standard
GitHub Login: @MSDN-WhiteKnight
Microsoft Alias: gewarren

Yeah, for some reason new added .NET samples weren't pulled to sample browser. I've filed the related Docs site bug recently: https://github.com/MicrosoftDocs/feedback/issues/2595 This is probably the same issue; the sample is merged into live branch, but not available on site both by direct URL and by searching keywords in sample browser. FYI, the Github link for this sample is https://github.com/dotnet/samples/tree/master/core/assembly/MetadataLoadContext
//cc @gewarren @BillWagner


Remove deprecated tasks: XamarinComponentRestoreV0 and XcodePackageiOSV0

These task are deprecated, so they are being removed
Attempted to remove them in #8257 but that resulted in a build error unrelated to this change, which has now been fixed (see #8388).

I assume we'll keep shipping these?

I assume we'll keep shipping these?

The final versions of these tasks have already been broken out of the aggregate Nuget package, so they should still be available if needed.


Generalize Consensus

As @iljakuklic mentioned here his codebase are ready,  and we can generalize BlockchainStorage in struct Consensus, no need to use types directly for it.

Done in #216


fix: efs-provisioner mountOptions is missing toYaml

What this PR does / why we need it:
mountOptions should be a list.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

@scottrigby @viglesiasce
Hi
Sorry to bother you.
Would you review this PR? I would like to use it asap. 😃

/ok-to-test
/lgtm


Support yaml references in declarative config

yaml might be useful to manage complex config files. For example, yaml supports references.
https://camel.readthedocs.io/en/latest/yamlref.html?_ga=2.105617600.217112987.1570526132-738201176.1567149463
We can use the YAML references to combine declarations to DRY config.
_format_version: '1.1'

services:
  identity: &identity
    host: identity.api
    connect_timeout: 60000
    protocol: http
    port: 8000
    retries: 5

  feed:
    <<: *identity
    connect_timeout: 30000
    host: feed.api

@ArtemDerevnjuk dfsdfs #5107

Hi, Kong supports YAML "references" (anchors).
However, Kong is strict about the syntax used. In particular, the services entry needs to be a list, not a hash.
Here's how your declaration would look, using anchors with the expected list syntax:
_format_version: '1.1'

services:
  - &identity
    name: identity
    host: identity.api
    connect_timeout: 60000
    protocol: http
    port: 8000
    retries: 5

  - <<: *identity
    name: feed
    host: feed.api
    connect_timeout: 30000

I have tried it locally with KONG_DATABASE=off KONG_DECLARATIVE_CONFIG=kong.yml bin/kong start and the two services appeared in :8001/services.

@kikito it's my bad) Thanks)

Is this broken in format_version 2.1?


lm_sensors: fix cross-compilation

Motivation for this change
CC and AR need to be specified for cross-compilation to work.
@ofborg build pkgsCross.armv7l-hf-multiplatform.lm_sensors lm_sensors
Things done

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

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

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

cc @dtzWill @Ericson2314

@ofborg build pkgsCross.armv7l-hf-multiplatform.lm_sensors

@ofborg build pkgsCross.armv7l-hf-multiplatform.lm_sensors


IBOM isn't rendering correctly on iOS devices

Hate to post this as an issue as I'm beyond pleased with this excellent tool - but I've looked through the issues and didn't find an answer:
I can't get the resulting ibom.html file to display properly on my iOS devices (iPhone 7+ and iPad Air).   The framework shows up but not the table or board images.   If it's supposed to work with Safari browser I'd appreciate knowing what configuration or other changes I should make.     Thanks for any suggestions and thanks again for this excellent tool.   Bob

I don't have access to apple devices anymore and can not test this. But it works fine on desktop safari so even if I had the access I'm not sure I would put effort into fixing quirks of mobile safari. As a policy I only promise to support modern major browsers, mobile safari isn't one.
Most likely root cause is lack of support for some javascript feature and could have a simple fix. But unless someone investigates the issue and opens a PR I can't help much.

Fair enough, thanks for the reply.  Just thought it might be a setting or something on my end.


ECS Task: Enable security groups

I would like to pass an existing Security Group(s) to enable communication between Orion and generated ECS tasks and other network-isolated services in my VPCs.

Can you share an example of where these would be passed?

https://docs.aws.amazon.com/AmazonECS/latest/APIReference/API_RunTask.html
This is another NetworkConfiguration option, so probably similar to #113 and #112
Additionally, we should be able to disable assignPublicIp.


Thank you for making this

Last year I was looking for a react hot toast alternative in Vue 3 but I didn't find any and had to use vue sonner, but now I can use it!
Big thanks!

Last year I was looking for a react hot toast alternative in Vue 3 but I didn't find any and had to use vue sonner, but now I can use it! Big thanks!

Thank you for your reply. Recently, I am busy with some things at home, and the repo has less time for maintenance. Now I'm back.

Hey! No worries! I'd like to contribute, it's a really cool and helpful project

Hey! No worries! I'd like to contribute, it's a really cool and helpful project

lol. Welcome. Let's build together 🩷🩷🩷

Sure! Any roadmap? Or a discord server? To know what is missing and start working on it

Sure! Any roadmap? Or a discord server? To know what is missing and start working on it

lol. Currently, i don't have a roadmap or a Discord server. I don't think they're necessary since this project isn't very big. We can collaborate by addressing the issues/bugs that others have raised. If you have any new ideas or features you'd like to see, feel free to add them to this repository. Welcome any creative issues and pull requests. @j0rgedev

Uhm okay! I'll take as reference react hot toast

I'd love to join as a OpenSoft dev too! Standard forking procedures?

I'd love to join as a OpenSoft dev too! Standard forking procedures?

Welcome to your arrival, let's build together.
Leocastroz suggested that I create a dashboard where we can jointly propose improvement ideas. Perhaps in the future, there will be some new enhancements to this component, or maybe we will even have a website.
https://github.com/SteveYuOWO/vue-hot-toast-ui/issues/1

I have invited you to join the dashboard I created. Please check your email for details. @j0rgedev @HarryZ10

@SteveYuOWO i haven't got any email

@SteveYuOWO i haven't got any email

Sorry, it seems like Github Projects doesn't require email confirmation. You can already edit the dashboard.

Thanks @SteveYuOWO for creating this cool react-hot-toast alternative for vuejs.
For last few days, I was searching this for swiftwave to replace vue-toast-notification . Finally find it.
I will also try to contribute.


Use run-mode-hooks in show- and search-mode.

Since Elfeed rolls its own major modes instead of using define-derived-mode, it runs its own hooks manually using run-hooks.
However, define-derived-mode uses run-mode-hook, not run-hooks. As a consequence,  Elfeed's major modes do not currently run the universal change-major-mode-after-body-hook and after-change-major-mode-hook, which some packages rely on to detect major mode changes.
This PR replaces run-hooks with run-mode-hooks in elfeed-search-mode and elfeed-show-mode to resolve this issue.
I don't know what the original rationale was for using the regular run-hooks, so please feel free to reject this if there is a good reason!

Merged. The reason for run-hooks instead of run-mode-hooks is simply
because I didn't know any better. Thanks!

Cheers!


Update 1-Installer-le-projet-en-local.md

Sorry I don't speak French but I'd like to propose this change so any developer who use Windows can easyly contribute to project by using Windows port of make. I tried to run Linus command line under Win 10 but its not working (because it tried to run docker under linux)
Probably I need help to translate this into French

Hello @mstrom !
Thanks for your proposal, I will translate it in french when I find the time. It's awesome you took time to create a PR on a project in a language you don't speak!

Hello, here is the french translation of your text :
Note: Pour les utilisateur de Windows, vous pouvez utiliser l'équivalent Windows de make
**Note**: Pour les utilisateur de Windows, vous pouvez utiliser [l'équivalent Windows de make](http://gnuwin32.sourceforge.net/packages/make.htm)

Cheers

Thanks a lot @Stoakes! @mstrom I think you are able to finish this now :) .

@Stoakes thank you!
@tgalopin Translation committed

Thanks @mstrom !


Docs: Document incremental definition via config.toml

Reading incremental parameters from configuration files.

Description

Related Issues

Fixes #...
Closes #...
Resolves #...

Additional Context

@burnash @AstrakhantsevaAA how can we improve wording?

@sultaniman added more context. Please see if it's okay.
Thanks !


Set domain name in rkt pods

#2040 is adding flags for resolv.conf's nameserver, search and options but not domain. Users also don't have any way to set the domain name. rkt sets up the hostname without any domain name.
We should add a flag for setting the domain name.
/cc @steveeJ

To set the domain name, we need to manipulate the entry for <IP_ADDRESS> in /etc/hosts. There are other requests for the ability to modify /etc/hosts as well. This is probably dependent on #3081.

@squeed "need to" or "should"? We can set the domainname and still provide value to anything that expects getdomainname to provide some value.

@euank AFAIK, the standard way of setting the domain name is to add an entry in /etc/hosts (though not for the loopback address, I was mistaken). Some systems allow a FQDN in /etc/hostname, but by no means all. Is there something I'm missing?

We had a bit of offline discussion, all better; conflated domainname with domainname.
I'm not sure if there actually are systems that restrict having an FQDN in /etc/hostname, and you can pass a FQDN to rkt's --hostname flag now just fine. If you pass a fqdn, then that part of this issue could be considered a dupe of #2223 I think, since doing that + --hostname=fqdn solves that problem entirely I think.
I think the dns domain, "domain" inresolv.conf is basically an orthogonal issue to handle separately.

Warning: significant fun ahead: different distros do this differently.
 This is consistent with the max DNS label length, but obviously shorter than the max DNS name length (253). Too-long hostnames are, of course, silently truncated.
Since default rkt hostnames are 40 characters long, that is not a lot of wiggle room.
I'm coming around to the idea of basically ignoring this - and just setting the CNI DNS Domain property in resolv.conf

@squeed
/sysadmin mode on
As you said the host dnsdomainname simply doesn't exists in the "host"/"kernel" (and setting the system hostname with a fqdn is sometimes possible but not a good/correct idea, a related discussion can be found here, and the sethostname call correctly accepts the right size for a hostname without domain).
The host's dnsdomainname is not saved in the host but depends on how an executable resolves is (using the system resolver and so the nssswitch configuration, using its own resolver or the system resolver based on compilation flags like golang does etc..) the solution depends on how the app image is configured. So just changing /etc/hosts will not be the final solutions (it may work most of the time but not always)
I also think that (IMHO also for classic infrastructures) in an app container logic, it doesn't makes much sense trying to force how an app resolves its dnsdomainname mangling /etc/hosts and other files. The app should work without knowing its hostname and its fqdn(s) (since they''ll change every time a pod is executed).

I'm coming around to the idea of basically ignoring this - and just setting the CNI DNS Domain property in resolv.conf

So :+1: for just changing the /etc/resolv.conf domain via cni (obviously not getting it from --hostname).


updated dependency

Codecov Report

Merging #366 (8d5897d) into dev-rc.1 (23acfcb) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           dev-rc.1     #366   +/-   ##
=========================================
  Coverage     31.15%   31.15%
=========================================
  Files            26       26
  Lines          1711     1711
=========================================
  Hits            533      533
  Misses         1161     1161
  Partials         17       17

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Too many memory used

Left lazydocker running for ~3 days, memory use increased to 11.2 GB. There were no running containers for most of this time.
To Reproduce
lazydocker

Expected behaviour
Adequate memory usage
Desktop (please complete the following information):

OS:

arch linux
Linux station75 5.5.3-arch1-1 #1 SMP PREEMPT Tue, 11 Feb 2020 15:35:41 +0000 x86_64 GNU/Linux

Lazydocker Version 0.7.6 (from repo AUR, build: 28 jan 2020 13:43:34)

After install 0.8 memory is OK. Thanks!


Update to webv 1.6.1

with webv 1.6.1, you can run the same test against multiple servers
 webv -s bluebell gelato sherbert -f baseline ...
we should update smokers to have one instance in each region for cost and simplification

Restarted smokers and merged code via PR#48.


[varLib.mutator] Mutator doesn't evaluate FeatureVariations

Mutator doesn't evaluate the instance coordinate against FeatureVariations in GPOS/GSUB. So, if you have a font that makes use of FeatureVariations, and generate an instance at a place where something interesting happens (e.g. currency symbols switch to alternate forms that don't have a slash all the way through), you don't get the alternate behavior.
I'd guess that mutator could just evaluate the condition sets and fix up the references to lookup tables accordingly in the instance. Pruning unused lookups might be tricky, but leaving a few unused (and probably small) lookups might not be too bad in the meantime.

Can you share a test font please?

I could build one, but that would take a while. Maybe @thundernixon or @petrvanblokland have a test font that uses FeatureVariations that's suitable for testing this? Or @irenevl? Or @thomgb?

I'll  make one today.

You can also use Bahnschrift, which has 10 characters that use feature variations (e.g. dollar, oslash, etc).
Email me privately if you need a copy for testing purposes.

Any volunteers to take a look at this?

Thanks!


[sqllab] Added share button to bottom menu

Added share button to sqllab bottom menu for easier discoverability

Codecov Report

Merging #4584 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4584      +/-   ##
==========================================
+ Coverage   71.17%   71.19%   +0.01%
==========================================
  Files         187      188       +1
  Lines       14809    14818       +9
  Branches     1085     1085
==========================================
+ Hits        10540    10549       +9
  Misses       4266     4266
  Partials        3        3

Impacted Files
Coverage Δ

...ssets/javascripts/SqlLab/components/ShareQuery.jsx
100% <100%> (ø)

...assets/javascripts/SqlLab/components/SqlEditor.jsx
74.73% <100%> (+0.26%)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM, feel free to ship as is despite the minor nit


Fb 49494 disable single instance db protection

Rationale

Adds new system property terminateOnExistingConnections=false that allows instance to continue on startup if the application detects more than one labkey instance connected. issueId=49494
Disables synchronousStartup - which should allow healthcheck to complete while system is starting (e.g long running upgrades etc. ) issueId=49761

Related Pull Requests

https://github.com/LabKey/platform/pull/5265

Do we want to limit the use of this flag (terminateOnExistingConnections=false) to just the deployments using CNPG? Maybe this PR does just that...

Do we want to limit the use of this flag (terminateOnExistingConnections=false) to just the deployments using CNPG? Maybe this PR does just that...

Yes, we only use CNPG with docker deployments at this time.


webrtc_streamer - does not save audio and does not display video

I have such a problem. There is a code:
from streamlit_webrtc import webrtc_streamer, WebRtcMode

from aiortc.contrib.media import MediaRecorder

def recorder_factory():
    return MediaRecorder("test.mp4")

streamer = webrtc_streamer(
    key="rec_streamer",
    mode=WebRtcMode.SENDONLY,
    rtc_configuration={"iceServers": [{"urls": ["stun:stun.l.google.com:19302"]}]},
    media_stream_constraints={"video": True, "audio": True},
    in_recorder_factory=recorder_factory,
)

At the output, I need to display the video so that it can be seen and write to 1 video file with an audio track. If possible, an example of such work. I would be very grateful.

For displaying the video, just replace WebRtcMode.SENDONLY with WebRtcMode.SENDRECV.
I also confirmed that the audio track was successfully recorded with this code in my environment.

For displaying the video, just replace WebRtcMode.SENDONLY with WebRtcMode.SENDRECV.
I also confirmed that the audio track was successfully recorded with this code in my environment.

Thanks for the answer! This did start the stream, however now while recording I get an mp4 output file where the audio is recorded and the video is displayed for the first few seconds and then the video freezes and the audio goes on. The output has this message:
[libx264 @ 0x7fc4ef045f00] non-strictly-monotonic PTS
[mp4 @ 0x7fc4f01ddbc0] Application provided invalid, non monotonically increasing dts to muxer in stream 1: -512 >= -512

For displaying the video, just replace WebRtcMode.SENDONLY with WebRtcMode.SENDRECV.
I also confirmed that the audio track was successfully recorded with this code in my environment.

Thanks for the answer! This did start the stream, however now while recording I get an mp4 output file where the audio is recorded and the video is displayed for the first few seconds and then the video freezes and the audio goes on. The output has this message:
[libx264 @ 0x7fc4ef045f00] non-strictly-monotonic PTS
[mp4 @ 0x7fc4f01ddbc0] Application provided invalid, non monotonically increasing dts to muxer in stream 1: -512 >= -512

@DmitryRyumin did you manage to solve this? I am facing the same issue


"No model found for id {x}" when rendering dynamically

shinywidgets version: 0.2.1
Python version: 3.10
Operating System: macOS 13.6

Description
With this app I was trying to have a widget returned from a UI output. But no widget appears, and there's a JS error saying "No model found for id {whatever}".
It looks to me like the fact that the widget registration happens before the output_widget is bound, is something shinywidgets didn't expect?

For now, you can workaround this problem by statically rendering output_widget()'s HTML dependencies, for example:
app_ui = ui.page_fluid(
    output_widget("foo").get_dependencies(),
    ui.output_ui("dynamic_ui"),
    ui.output_text("value")
)

I haven't looked deeply yet at what is causing this, but based on the fact that the simple dynamic UI usage works fine, it seems the issue derives from register_widget() getting called before the output binding has been registered.


Feature request: Add "Calories consumed" Entity

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
As a user, I would like to be able to see the "calories consumed" data that my third party applications write to Google Fit in Home Assistant.
Describe the solution you'd like
When I have a third-party integration with Google fit that adds calorie consumption data to Google Fit, I would like that data to be available in Home Assistant.
Concretely, when I go to the Entities section of the HASS Integtations page for Google Fit, I would like to see, "Calories consumed" as a row in the presented table with values pulled from my calorie tracking app.
Ideally, I'd like it if this integration could auto-discover new metrics in Google Fit as they emerge but I'm not sure if that's possible with the API.
Describe alternatives you've considered
I have looked around for calorie tracking apps with APIs that integrate with Home Assistant and I haven't found anything that works for me. It looks like there is a MyFitnessPal integration but it hasn't been updated in 5 years and I think using Google Fit as a hub API is a better long term solution than relying on individual end-user apps.
Additional context
I'm a passable Python developer and I skimmed the repo but I haven't built a HASS integration before and I'm not sure I'd be much help with the amount of time I have to contribute but this is something I want so, if nothing else, I'm motivated.

Hi @ELind77, I've put this on the backlog because it's not going to be done in the near future as there are other quality of life things I won't to focus on first.
If you're up to it, feel free to have a stab at it yourself. Just fork the repository and as a starting point the calories burnt is probably the best sensor to copy. It's really not too much work, biggest sticking point would be if it needed a new access permission so be sure to check that

Thanks for the response.  I'll explore implementing it myself.  I have some questions along that direction though.

Is this commit a good example for me to follow for what I'm trying to do? https://github.com/YorkshireIoT/ha-google-fit/commit/85c03c05dd2aefc1b8bfa0867cfed32163c33559
How do I find the equivalent of the class path-looking thing for the new data type? https://github.com/YorkshireIoT/ha-google-fit/blob/main/custom_components/google_fit/const.py#L85
After looking at the docs briefly, I'm concerned I won't be able to access this additional data type.  It says:

Note: Custom data types are only available to the apps that created them. Other apps can't see or use your custom data types.

Do you think that means there's no way for HASS to read the "CaloriesConsumed" data?  Is there an easy way for me to test things like this in the package?  Either interactively or with a script I can run to check error messages and whatnot?

1. Is this commit a good example for me to follow for what I'm trying to do? [85c03c0](https://github.com/YorkshireIoT/ha-google-fit/commit/85c03c05dd2aefc1b8bfa0867cfed32163c33559)

Yeh, although if you want an example where only 1 sensor is added commit 364b6de might be better

2. How do I find the equivalent of the class path-looking thing for the new data type? https://github.com/YorkshireIoT/ha-google-fit/blob/main/custom_components/google_fit/const.py#L85

That's a good point, I forgot about that. It's actually a bit of a pain. First you need to make sure you actually have some valid data in your Google Fit account for the sensor type you're looking for.
Then you need to go here, find the sensor data you want to read and look at the OAuth Permission Scope.
Then you have to go to the Google API Playground, authorize using that same permission scope for Fitness v1 and exchange the tokens for you account.
Finally, in the Request URI enter this: https://fitness.googleapis.com/fitness/v1/users/me/dataSources and send the request. In the response you're looking for something that starts with dervied and ends with merged that matches what you want.

3. After looking at [the docs](https://developers.google.com/fit/datatypes/custom) briefly, I'm concerned I won't be able to access this additional data type.  It says:

Note: Custom data types are only available to the apps that created them. Other apps can't see or use your custom data types.

As long as your third-party provider has implemented the data type properly using one of the existing data types then this shouldn't be a problem. Google does have a nutrition sensor type which contains calories consumed.

Do you think that means there's no way for HASS to read the "CaloriesConsumed" data? Is there an easy way for me to test things like this in the package? Either interactively or with a script I can run to check error messages and whatnot?

This is actually a bit of a deficiency in CONTRIBUTING.md docs. I'm going to quickly update this so it's easier to get started with developing.
⚠ Having said all that though, having looked at the sensor type in the Google docs I wouldn't advise tackling this as a good first time change. It looks like the nutrition sensor is a lot more involved than just querying its native type, as the calories data is embedded deeper within the data. That means there will need to be some more fundamental updates to the code base to accomodate this new type.

I'm interested in that one also (using more and more the Health hub and now having also the food in it)
I check quickly and yes seems different. If I can help, I'm willing, but not sure I can do it alone.

Great work guys, Just wanted to say, I would love to see this feature for calories consumed to graph alongside calories burned for comparison 😁


_.runInContext

Thoughts on having _.runInContext to create pristine Underscore instances that are safe to mixin and aid in mocking?
_.mixin({ 'add': function(a, b) { return a + b; } });

var u = _.runInContext();
u.mixin({ 'sub': function(a, b) { return a - b; } });

_.isFunction(_.add);
// => true
_.isFunction(_.sub);
// => false

u.isFunction(u.add);
// => false
u.isFunction(u.sub);
// => true

// using `context` for mocking timers
var u = _.runInContext({
  'Date': function() {
    return { 'getTime': getTimeMock };
  }
});

// or creating a suped-up `defer` in Node
var defer = _.runInContext({ 'setTimeout': setImmediate }).defer;

Naming inspired by http://nodejs.org/api/vm.html#vm_vm_runincontext_code_context_filename.

:+1:

I dunno. It seems like noConflict covers the actual use case for this — if your scripts are cooperating, then you want to share mixins ... and if you're not cooperating, then you want to noConflict.
Perhaps if there was a good use case for this, outside of testing Underscore.


Rapidly switching between tabs causes view size to be wrong

Is there an option to force view size recalculation every time another tab is selected?

@SamusAranX Can you check with the current master?

I updated my project to the new API, but calling .show() on my PreferencesWindowController does absolutely nothing now. My app's open windows lose focus, but no preferences window is actually being opened.

Just tried calling .show() with an identifier, but that doesn't work either. But I've noticed that the window seems to get opened after all:

It's just that for some reason, it's invisible and doesn't participate in App Exposé.

I encountered this only my content views wouldn't have a proper intrinsicContentSize. Using AutoLayout probably is the easiest fix. How are you creating your content views, and can you set their size more rigidly somehow?

I already use Auto Layout. My views are loaded from XIBs and there's nothing special about them.

Since the labels are fully localized, I can't really make their size explicit.

@DivineDominion I can reproduce this by unchecking "Use Auto Layout" for the Advanced preference pane XIB. We need to be able to handle non-autolayout UI too, or if that's too difficult, at least throw an informative error.
The problem here is that https://github.com/sindresorhus/Preferences/blob/891d9df0ee36eb1e0d283051d0e17169371ed529/Sources/Preferences/PreferencesTabViewController.swift#L184 is .zero when Auto Layout is disabled on the view, so the window ends up being zero too. Maybe we could use viewController.view.bounds.size when viewController.view.fittingSize == .zero.

@issuehunt has funded $60.00 to this issue.

Submit pull request via IssueHunt to receive this reward.
Want to contribute? Chip in to this issue via IssueHunt.
Checkout the IssueHunt Issue Explorer to see more funded issues.
Need help from developers? Add your repository on IssueHunt to raise funds.

Just a heads up: Manually setting self.preferredContentSize in my views' various viewDidLoad() methods fixed the problem for me. It's not an optimal solution, but I couldn't find a better way to do it at the moment.

We should also check whether the situation improved in macOS 11.


Support for hinting/completion

I could not find a clear example of this, is there a way to provide hinting/completion with the output being either text or an embedded 'object' similar to https://summernote.org/examples/#hint ?
i,e we would require the most recent few words (at least the last few letters, including the position inside the content) plus an async callback function and ability to populate a drop down.
Similarily a way to underline/mark text (to indicate to a user that a drop down of actions is available at that position).

When implementing the mention feature, I will implement it together.
It will be added after version 3.0.0.

Will keep an eye on it thanks

Were you able to implement this feature?


Passive copy of paginator doesn't show up

http://plnkr.co/edit/007mVhenNIdjAftfMW6K?p=preview
If you take out passive="true" it shows up correctly.

Hmm, that's a regression. Thanks for reporting it. I'll investigate in the next few days.

Thanks, @begriffs !

The regression arose in 2.5.4, which included some small changes to how passive was handled:
https://github.com/begriffs/angular-paginate-anything/compare/v2.5.3...v2.5.4

If I take out the passive === 'true' that was added in b1e04e0, my passive copies show up again.

I think the passive change was made to prevent a problem with too eagerly loading something the first time around. I don't remember the details now -- @mekosan can you help remind me what the problem was?
There's a test for it in 187944b5e14d0e42e44b6487aeb12ade1a622824 but looking at it I can't see what it was all about anymore.

The comment on the commit says "use of passive option on a single control".
That sounds like an unusual use case to me, and maybe not the intended
usage of passive.
On Thu, Mar 12, 2015 at 5:07 AM, Joe Nelson<EMAIL_ADDRESS>wrote:

I think the passive change was made to prevent a problem with too eagerly
loading something the first time around. I don't remember the details now
-- @mekosan https://github.com/mekosan can you help remind me what the
problem was?
There's a test for it in 187944b
https://github.com/begriffs/angular-paginate-anything/commit/187944b5e14d0e42e44b6487aeb12ade1a622824
but looking at it I can't see what it was all about anymore.
--
Reply to this email directly or view it on GitHub
https://github.com/begriffs/angular-paginate-anything/issues/63#issuecomment-78444656
.

That was a while ago, but it think it was to prevent the initial load. I think I was wanting to control the first time the the plugin fired as I was getting multiple requests on the initial page load as some filters were being set.

Hello and thanks for the great work! I tried this in my first meanjs project, looking for the old mirror and the new passive, second pagination doesn't show up if passive, and not in sync with first, if i remove it - https://github.com/mihneasim/journeh/commit/6892852a5d62b45bcc7adb926423c4508b7ee1b1
Dunno if helps, or just duplicating initial report.

@mekosan it sounds like an additional attribute might suit your use case better. Maybe something like ready that would prevent any loading of items until it was true.
@begriffs I think the change from b1e04e0 is causing problems for a couple people. I suggest reverting it.

I reverted the change and brought passive paginators back in new version 4.0.2

Thanks, @begriffs!
On Tue, Mar 17, 2015 at 1:24 AM, Joe Nelson<EMAIL_ADDRESS>wrote:

Closed #63
https://github.com/begriffs/angular-paginate-anything/issues/63.
--
Reply to this email directly or view it on GitHub
https://github.com/begriffs/angular-paginate-anything/issues/63#event-256423143
.

Now requestRange is getting called twice the first time the page is loaded, even though the lower paginator is defined with passive="true".  Restoring the if($scope.passive === 'true') { return; } before the first call to requestRange prevents the double load, but then the passive paginator never renders the numbered page links.

I'm going to live with the fact that two requests fire when the page is first loaded because caching is working and it's a minor problem in the grand scheme of things.  Subsequent clicks to either paginator do behave as expected.  Thanks for continuing to maintain this directive @begriffs.


no "gpt-3.5-turbo" support

please add "gpt-3.5-turbo" support which was introduced few days ago.

Yes. Trying to use this with gpt-3.5-turbo. The completions is different /v1/chat/completions, but there seem to be some parameter changes as well.

please add "gpt-3.5-turbo" support which was introduced few days ago.

You can use https://github.com/HemulGM/DelphiOpenAI

I notice this statement. "Althought not fully tested, it should also work on all supported platforms (Windows, Linux, macOS, Android, iOS)."
I think this means you can compile in Delphi to run on those platforms. I develop in Linux using Lazarus, since Delphi cannot run on Lazarus. This project has allowed me to use the OpenAIClient.pas library in Lazarus on Linux.
If I am incorrect please let me know. From what I saw of the the DelphiOpenAI, it uses fmx which is not compatible with Lazarus on any platform.

It does not use FMX, but yes, the code, although cross-platform, is not compatible with FPC

This repository is fully FPC compatible. Would you mind not hijacking this issue/repo and discuss it in the proper location? Thank you.

still no support for GPT-3.5-Turbo

Is this project dead? There have been no updates for several months now.

No, it's not.

Project has now been updated and supports the new chat completion endpoint, which in turn support the newest models, including of course GPT-3.5-Turbo.
The demos for Delphi and Lazarus were also updated, so you can test and play with it right away.


feat: extra logging

Standards checklist:

[x] The PR title is descriptive.
[x] The PR doesn't replicate another PR which is already open.
[x] The PR has self-explained commits history.
[x] The code is mine, or it's from somewhere with an Apache-2.0 compatible license.
[ ] ~The code is efficient, to the best of my ability, and does not waste computer resources.~
[ ] ~The code is stable, and I have tested it myself, to the best of my abilities.~
[ ] ~If the code introduces new classes, methods, I provide a valid use case for all of them.~

Changes:

User registration steps logging
Video load with recommendations list logging

Other comments:
This pull request is the rebased and actualized version of lost #548 authored by @zanhesl.

In the last commit an additional log was added. It is fired when user opens app via external link.


Where are the latest update for Microsoft Remote Desktop Client? Latest version is 10.2.3000.0

Where are the latest update for Microsoft Remote Desktop Client? Latest version is 10.2.3000.0

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

ID: 67b7b5a4-647e-fee3-6533-6ed88ae45bfa
Version Independent ID: d9102859-0147-346f-8268-6e2a75c8b0b7
Content: What's new in the Microsoft Store client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windows-whatsnew.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

#assign:xelu86

Hey @tamoanxx,
Thanks again for contacting Microsoft. We value your feedback.
I think you may be referring to our "What's new in the Remote Desktop Client" page with the latest version being 1.2.3770 for the Window client itself. As this is a cross-platform product, please refer to the TOC on the left and select the page for the platform you're currently using:

We will now be closing this thread. Thank you and have a most pleasant day.
#please-close


feat: Add support for inky screen in nodemcu device code

@tomsci LGTM. Could you please also add the new configuration to 'docs/architecture/index.markdown'. Thanks!


ceylon doc fails with dependency issues

Create a 'simple' project, then:
In module.ceylon:
native("jvm")
module simple "1.0.0" {
    import ceylon.collection "1.2.0";
    // leave one of the next two lines uncommented:
    import ceylon.ast.core "1.2.1";
    //import com.redhat.ceylon.typechecker "1.2.1";
}

In run.ceylon:
shared void run() {
  print(true);
}

Then, try to run ceylon doc:
$ ceylon doc
error [could not determine type of function or value reference: 'true': Error while loading the ceylon.language/1.2.1 module:
 Declaration 'ceylon.language.true_' could not be found in module 'ceylon.language' or its imported modules but was found in the non-imported module 'ceylon.language'] at 2:8-2:11 of simple/run.ceylon
1 errors, 0 warnings
error [could not determine type of function or value reference: 'true': Error while loading the ceylon.language/1.2.1 module:
 Declaration 'ceylon.language.true_' could not be found in module 'ceylon.language' or its imported modules but was found in the non-imported module 'ceylon.language'] at 2:8-2:11 of simple/run.ceylon
1 errors, 0 warnings
ceylon doc: Parsing failed with 1 error(s)
jvasileff@orion:simple$

@tombentley sounds like related to 1.2.1/1.2.0 because we have mixed imports. Any idea?

No idea @FroMage. And the most recent work to support --target is still in a branch, so it would have to be dist overrides part. It's true that I didn't test ceylon doc specifically though, so we need tests for that ni any case.

The reason this fails when ceylon compile works seems to be because the CeylonModelLoader doesn't return ceylon.language/1.2.0 from lookupModuleByPackageName() due to this:
        // skip modules we're not loading things from
        if(!ModelUtil.equalModules(module,getLanguageModule())
                && !isModuleInClassPath(module))
            continue;

For the compiler, ceylon.language/1.2.0 is not in the classpath, so we hit the continue and go on to find 1.2.1. That logic's not right (because ceylon.language/1.2.0 is a language module, even though it's not the dist one). So we only don't return ceylon.language/1.2.0 by accident.
The CeylonDocModelLoader, otoh doesn't override isModuleInClassPath(), so we don't hit the continue and lookupModuleByPackageName() will happily return ceylon.language/1.2.0.
So the question is, what should the model loader do? There's no way for lookupModuleByPackageName() to know which module to return when there's more than one version available.

Can we really have two language modules loaded at once now? The model loader is not designed for this AFAIK.

No we can't, but I'm trying to figure out the right way for the model loader to deal with the fact that Modules.getListOfModules() can return several language modules. I guess we should just chose the language module matching --target?

Yes, but I do wonder why we have more than one created. IIRC this is not what happens for JDK7/8.


Extension - [nextjs-docs] - Feature caching

Caching and Code Cleanup
Description
Implemented caching for faster performance.
code cleanup.
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

@pernielsentikaer Thanks for the review. I'll get back with the suggested changes.

Could you look into this @PrasadChinwal, seems like there is some merge conflicts  🔥


Add getpid on Redox

@bors: r+

:pushpin: Commit 30f6da8a640b3efd36719d6fd8c4df08ba2383f6 has been approved by alexcrichton

:hourglass: Testing commit 30f6da8a640b3efd36719d6fd8c4df08ba2383f6 with merge f36341e219f033d5fbe867a1d237841845767b88...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing f36341e219f033d5fbe867a1d237841845767b88 to master...