Update B9PartSwitch.netkan

would you kindly update this listing with a correct forum url?
https://forum.kerbalspaceprogram.com/index.php?/topic/140541-*/
sorry for the inconvenience.  🌺

The forum URL seems to be fine, and it's in an author-controlled metanetkan anyway:
https://github.com/blowfishpro/B9PartSwitch/blob/master/NetKAN/B9PartSwitch.netkan

https://github.com/KSP-CKAN/CKAN/wiki/Suggested-Tags

resources - Mods focused on ISRU and resource management

As far as I know, B9PartSwitch doesn't have much to do with ISRU. It's about swapping out different versions of parts.

B9PS also swaps out resource tanks.
and Now the link is working. 😵💫


Deprecating tasks self API

Description
As per discussion with @ankushdharkar, he suggested deprecating self API from tasks as it is hard to maintain separate routes for this
Suggested-Solution

Can create an API once it hit we can store the information in local storage.

More solutions need to be discussed

Closing this issue as other issue ticket is already created which do the same task

Closing this issue as other issue ticket is already created which do the same task


License?

Would it be possible to add a license to your software? The article just state it is freely available.
Thanks.

I'm discussing this with my co-authors, we'll update it soon. It will be some flavor of open-source at the least.

We've now added an explicit license.


Update NPM Dependencies

Attempting to update the following NPM dependencies:

crypto-js from 4.1.1 to 4.2.0

Resolved by #1742

Resolved by https://github.com/PowerDNS-Admin/PowerDNS-Admin/pull/1742


Allow routers to be configured with a list of identifiers

Allow routers to be configured with a list of identifiers.  If an identifier cannot assign a dest to a request, it falls back to the next one in the list.

A question worth considering while we're here:
exceptions tend to be much costlier code paths than other graceful returns (i.e. stack tracing, etc).
We're admitting now that an unidentifiable request isn't so exceptional: we expect that identifiers won't always identify requests.
What if we made Identifiers return an ADT like:
sealed trait RequestIdentification
case class IdentifiedRequest(dst: Dst, request: Request) extends RequestIdentification
object UnidentifiedRequest extends RequestIdentification

This makes the interface much clearer, though it's an API breakage (worth it imo)

UnidentifierRequestException mixes in NoStackTrace, but I agree with you that it's not the best to do control flow this way.
And API break would be ideal, (either to ADT as your suggest or just plain Option) but I was hesitant because we know there are identifier plugins in the wild that would need to be updated.

Still think the API break is worth it? 😉

It's my preference that we push forward with the API break and communicate the change.  I think this puts the Identifier in a much better place moving forward.
I have a slight preference for an ADT over an Option:

Option[Tuple] screams for want of a proper type
We could conceivable support additional states (idk what, just thinking)
We could conceivably extend IdentifiedRequest with additional information (on that note, we probably should avoid case classes here -- case classes cause all field additions to be API breakages (on unapply).

:star: my name is oliver and i endorse this message.


Tab bar sorting function

First of all, this extension is amazing.
I'm a bit of a freak when it comes to organization and this piece of code is just what I needed.
I might have an idea or feature request.
It would be great to have an option where all groups jump to the beginning of the tab bar.
Maybe through a switch with a filter or a sorting function where you can sort the created groups (A-Z, custom, ...) and have that order reflected in the tab bar.

Hey @DasFlogetier, thanks for the suggestion.
This is the feature I want to implement next, although I'm not sure when I'll find the time to work on the project.
I'll close this issue though, as it is a duplicate of #24 and #12.


Add RouteShapeFormat for polyline with precision 6

For completeness’ sake, the RouteShapeFormat enumeration should have a polyline6 case for Polyline encoding with the usual 1×10⁶ precision, to match the geometries=polyline6 option supported by Directions API v5 (but not v4).

+1


Detach repository from tonerdo/pose

I believe this fork has diverged significantly from tonerdo/pose to warrant detachment. Additionally, given the lack of activity in said repository, I don't have high hopes of getting the changes merged into the master branch.
Detaching a fork can  be done here: https://support.github.com/contact?tags=rr-forks&subject=Detach Fork&flow=detach_fork
Although I'm a bit unsure as to the meaning of "detach" vs. "extract". Will probably create a ticket with GitHub support instead of going through the bot.
UPDATE 2024-01-28: I've created a ticket with GitHub support to detach the repository while keeping all issues and PRs intact.

The repository has been successfully detached from tonerdo/pose.


Can not ignore the tearing effect

If without background, it works pretty good. With background, you can find in this gif.

Maybe double buffer can fix this?

Hi,
You can improve the quality by increasing the size of VDB (LV_VDB_SIZE in lv_conf.h)
If you set it to LV_HOR_RES * LV_VER_RES the screen will be rendered in one piece. It's like double buffering.

@kisvegabor emm, not working.
more buffering just improve the speed, but tearing is not disappear.
Looks like the tearing position is fixed at a height(maybe at 240?) very weird.

@kisvegabor current my screen is 480x800, once I set it to 480x240, the tearing effect is not showing anymore. Any idea?

I can confirm it is tearing at height = 250px~300px
Looks like first frame only render to around 250px then next frame render the full screen.

I'm not sure but I suppose it comes from the PC simulator's monitor driver. When the monitor driver  (monitor.c) copies the pixels (in monitor_flush()) to the frame buffer then there is no good triggering to not refresh the screen while copying.
However it sets the sdl_refr_qry flag it remains true after the first run and maybe SDL will update  the screen and clear the flag while you are copying the next frame.
You can try to set sdl_refr_qry = false in the beginning of monitor_flush to be sure it won't be refreshed during the flush.

@kisvegabor I do not think so, please check this video: https://youtu.be/_MouNVwW46k
It is on a framebuffer device.

@Vonger Your video is blocked？

@sirius65535 Updated 👍

I see. It's sure that if you create a (LV_HOR_RES * LV_VER_RES) sized VDB then the whole screen will be refreshed in one piece. So disp_flush will get the full screen in the buffer.  I think it comes from the asynchronous update of the frame buffer.
I found it: https://stackoverflow.com/questions/13907471/linux-framebuffer-graphics-and-vsync

@kisvegabor Why the size should be LV_HOR_RES *LV_VER_RES rather than the answer above times (color depth)/8?

@WangZhihua1998 unit is pixel.

Here is nice example of linux framebuffer animation with FBIO_WAITFORVSYNC https://github.com/rst-/raspberry-compote/blob/master/fb/fbtestXIV.c
It is for Raspberry Pi,  but it should work on any LinuxFB. You can also try fbtestXIVb.c in the same repo.
It is using real double buffering and just switching buffers at the right time. You can try this example on your platform if your driver supports wayting for vsync and tearing free animation.
Next step can be to incorporate this mechanism to lvgl driver for LinuxFB

As there was no activity in this topic for a few months I close this issue.
If you made some progress or have further questions don't hesitate to comment and reopen the issue.


GROOVY-8305: Fix default Ivy settings file for local m2 path

worth being tested on Linux
not sure how to write a test for that

What can we do to ensure that this PR is reviewed/merged as soon as possible?

+1
@paulk-asert  What do you think about the PR?

I get different results to the summary above. Perhaps I am running a different test. I don't have a space in my user.home directory either. For me I get:
{noformat}
file: Windows OK, Ubuntu OK (status quo)
file:/ Windows OK, Ubuntu KO
file:// Windows KO, Ubuntu OK
file:/// Windows OK, Ubuntu OK
{noformat}

with the provided script on Windows I have the same results:

file: Windows OK --> this one was not working in the reported case but working with provided script
file:/ Windows OK
file:// Windows KO
file:/// Windows OK

I tried also with this script to try to be as close as possible to intiial usecase but still cannot reproduce the problem with only file:

import groovy.grape.Grape;
import groovy.lang.GroovyClassLoader;
Grape.grab([classLoader : new GroovyClassLoader(),group :'org.apache.camel', module : 'camel-catalog-provider-karaf', version : '2.19.2B'])

the code calling Grape: https://github.com/apache/camel/blob/camel-2.19.2/platforms/camel-catalog-maven/src/main/java/org/apache/camel/catalog/maven/MavenVersionManager.java
I'm wondering what is causing the issue, any ideas on what can be the possible root cause in order that i can try to reproduce are very welcome :s

@apupier, In the IVY JIRA you filed originally, can you add the steps that I can follow to reproduce this issue?

@apupier, In the IVY JIRA you filed originally, can you add the steps that I can follow to reproduce this issue?

Just to be sure that you noticed it, I provided the steps few days ago: https://issues.apache.org/jira/browse/GROOVY-8305?focusedCommentId=16167801&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16167801

@apupier, I wasn't following that JIRA so hadn't noticed your update which included the steps to reproduce. Thanks for letting me know. I'll give it a try in the next few days and see how it goes.

@jaikiran i'm wondering if the openjdk and oracle jdk are not behaving differently for this "file:" stuff, on a server we had a lot of errors of ivy cache in log, I changed from openjdk to oracle jdk and there are no more errors. I think that it might explain the difference of result that we have.

Apache Ivy 2.5.0-rc1 is out. Maybe we can bump Ivy to 2.5.0-rc1 and see whether the issue exists.
http://ant.apache.org/ivy/

Groovy is using Ivy 2.5.0-rc1 on master and 2.6.x branches. Feel free to give it a try.

When I have used a filesystem-based ibiblio in the past on Windows, the format looked like
<ibiblio name="Local" m2compatible="true" root="file:///C:/Users/..."

As stated above, ${user.home} is going to resolve to "C:..." on Windows and "/home/..." on Linux.  So, I don't think you can unconditionally set a file url prefix that will work for both.  Is there a variable that can be used to get a file: url for the user's directory?

Another approach would be to set the "ivy.ibiblio.default.artifact.root" property from the code and remove the root attribute from the settings file.  I thinnk this defaults to Maven Central's URL.  So you would need to add a root attribute to the ibiblio below jcenter, which is intended to point to maven central.
https://ant.apache.org/ivy/history/master/resolver/ibiblio.html

Last line below is proposed addition to GrapeIvy.groovy that would allow you to use "${user.home.url}" in the default settings xml.
    GrapeIvy() {
        // if we are already initialized, quit
        if (enableGrapes) return

        // start ivy
        Message.defaultLogger = new DefaultMessageLogger(System.getProperty('ivy.message.logger.level', '-1') as int)
        settings = new IvySettings()
        settings.setVariable('user.home.url', new File(System.getProperty('user.home')).toURI().toURL() as String)


additional canary service configuration

Currently we are migrating our services to flagger as a delivery tool. We are using istio as a service mesh. For our Rest based services istio AB and canary deployments are very good, but some services are build upon amqp. For canaries we played around with the scaling of our apps to make progressive traffic shifting work as desired. But for the AB deployment scenario we didn't find a solution.
Is it possible with flagger to provide the canary an additional configuration to override the configuration of primary, so that in our case the canary connects to a different queue (like in the picture)?

You need to make some adjustments to your app code:

create a configmap that contains two keys queue: B and queue-primary: A
at application startup get the HOSTNAME env var
parse the hostname and if it contains -primary then use the queue-primary entry from the configmap, if not then use the queue value.

Ok, this works. But this rather a workaround in my opinion, because it pushes infrastructure knowledge into service.


message pre-checks?

hi, what's the correct way to see if we have enough data on the stream to decode a message?
decode returning a boolean doesn't let one distinguish between:

incomplete
invalid
wrong message type

which could need to be handled differently

Protobuf as a format does not allow precise detection of any of these conditions.
In fact, empty message (of 0 bytes) is a valid encoding for any proto3 message type.
To verify the integrity of a message, external means are needed (like a checksum or a signature).

Protobuf as a format does not allow precise detection of any of these conditions. In fact, empty message (of 0 bytes) is a valid encoding for any proto3 message type.
To verify the integrity of a message, external means are needed (like a checksum or a signature).

not even using tag+length to determine if we need more bytes?

Only if the message gets cut in middle of a field. But because protobuf messages are not self-delimiting, you must already have some method of knowing when the message ends.
stream->errmsg will give a more detailed reason for an error return, but in general this is meant only for debugging purposes. I try to keep the messages stable, so it's not terribly wrong to strcmp() with e.g. "end-of-stream".
You can perform sort-of validation without decoding by repeatedly calling pb_decode_tag() and pb_skip_field(), but that will only detect some errors and not all, as many corruptions and truncations of protobuf messages result in a valid message.

my primary goal is to know when I have enough data to pb_decode or disconnect someone sending nonsense data.
checking stream->errmsg seems like a fair work-around to answer if it's incomplete and if it's definitively not the kind of wrapper message we expect

I don't see a clear use case where a best-effort distinction between "truncated message" and "otherwise corrupted message" would be useful.

I don't see a clear use case where a best-effort distinction between "truncated message" and "otherwise corrupted message" would be useful.

It's the case of asynchronous io, you are notified there Is something you read, and you read it into a buffer.
Now you need to decide if you process it or wait. if pb_decode succeeds, all good. But If it doesn't, what do you do? keep appending and retrying. but if it was the wrong type of message or complete non-sense, you'll append data for ever

As previously mentioned, the Protocol Buffer wire format is not self delimiting so the data stream needs to include a message delimiting mechanism so that you can know when you have received a complete encoded message. A common method for doing this is prefixing an encoded message with the size of the encoded message (using pb_encode_ex() with the PB_ENCODE_DELIMITED flag and pb_decode_ex() with the PB_DECODE_NULLTERMINATED flag is one way of doing this).

thank you


Showcase Submission for challenges/30-phyllotaxis

Thank you Corné for your contribution! A member of the Coding Train team will review it shortly.

3D Phyllotaxis
Corné

This link is giving me a 404 sadly! The commit is not associated with a GitHub account but if anyone can help find or re-submit I'd love to consider it!

I don't think there's a mistake with the url, the link was deleted by the user. The ids are sequential on openprocessing.org and you got another submission with id 1977351 (this one is 1977421 which is only 70 away).
I suggest we close this PR. The author can always resubmit.

Agreed!


Replaced ARGS by ARGS_GET in rules in phase:1

This PR fixes #1949.

Thank you very much for this fix. Merging now.


Multiple Target Depedencies - Only first embedded.

If there are multiple Targets with (the same?) Framework dependency, only the first Target is linked and embedded correctly.
The build dependency is correctly configured for both targets, but it is not getting linked and embedded in the second one.
eg: Target1 gets linked and embedded, Target2 does not. - if Target1 is deleted althogether Target2 works like expected.
targets:
  Target1:
    dependencies:
      - carthage: CarthageFramework
      - target: dependency/dependencyTarget

  Target2:
    dependencies:
      - carthage: CarthageFramework
      - target: dependency/dependencyTarget

Keep up the great work!

Did you mean to exclude dependencies: in the second target?

sry, it got lost in the abbreviation - it's correct in the actual yml-file. (edited the initial posting)

As written it should work. What are the types of these two targets? Do they depend on each other in any way?

both targets are application are of type application. They don't depend on each other - the second target is more or less a slight modification (with some additional features and one more carthage dependency) than the first.

@Skymax1986 if you could post a fuller spec that has all the required definitions, we can debug this


disable informative status by default

Improves performance. Informative status config should be left upto user.
See https://github.com/fish-shell/fish-shell/issues/7367#issuecomment-702596671

@bobthecow this is a single line config change, could you please review and merge?


[Feature request] Filer support for multiple remotes

This is a new feature requests.
It would be a useful if Filer would support multiple remotes, for example (aws and azure) that could be kept in sync with their respective remotes.
The use case would be to write once to Filer, and Filer would sync that file to BOTH Aws and Azure
Spoke with @chrislusf and thinks it only needs a few tweaks.
Let me know if you need a script that can reproduce this

This function is implemented, but you will need to configure, for example, rabbitmq and three queues, one incoming from the filer, then rabbitmq itself copies all messages into two queues, one for Amazon and the second for azure, and two weed replication applications are launched
See https://github.com/seaweedfs/seaweedfs/wiki/Async-Replication-to-Cloud

That's exactly what I am looking for! Thank you.


use binary gpg signatures instead of ASCII armored ones (.asc -> .sig)

makes the signatures downloadable by default
instead of showing them in the browser

What's the point of targeting stable here?

So it'll be in the docs for 2.068.1.

You mean in the download.html included with the zip files? Wouldn't think anyone would have any reason to open it.

You're right, I think it's even excluded from the release. Still this is going to be the only signatures for the next release, so it kinda makes sense to me to do it on stable.


just a test

TEST

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-038f4106140e14596.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-038f4106140e14596

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-08e14c8c9ffdd7fb6.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-08e14c8c9ffdd7fb6

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: https://i-0aa2ba1183e9b20b9.test.spinmint.com
Test Admin Account: Username: sysadmin | Password: sysadmin
Test User Account: Username: user-1 | Password: user-1
Internal IP: <IP_ADDRESS>
Instance ID: i-0aa2ba1183e9b20b9

Spinmint test server destroyed


UserSession, ApiSettings and tests

I'd just started to play around with this file too :-) glad I saw your slack! lets have a good coordination meeting this morning
Here's where I was going with this file. I think it can be pretty simple. I haven't tested it and it requires updating tp.TransportSettings to not assign a default value of None to headers which I think is fine. What do you think? (also, I couldn't quite get ConfigParser.read_file to work. the docs seem to say use it first for any required settings then use read for optional settings?)
"""Settings for configuring API Access
"""
from __future__ import annotations

import configparser
import dataclasses as dc
from typing import Optional

from looker.rtl import transport as tp

# pylint: disable=too-many-instance-attributes
@dc.dataclass(frozen=True)
class ApiSettings(tp.TransportSettings):
    """API settings uses `looker.ini`."""
    client_id: str
    client_secret: str
    embed_secret: str
    user_id: str

    @classmethod
    def configure(cls, filename: str,
                  section: Optional[str] = None) -> ApiSettings:
        """Read a .ini file and return an ApiSettings instance.
        """
        cfg_parser = configparser.ConfigParser()
        with open(filename) as cfg_file:
            cfg: str = cfg_parser.read_file(cfg_file)

        section = section if section else cfg.sections[0]
        return cls(**cfg[section])


Can't craft vanilla sticks with modded planks

MC 1.19.2
FAPI 0.62.0
VSAS 2.1.7
When VSAS is installed together with a mod that adds new trees, it disables its crafting recipes for vanilla sticks. To test this, I installed VSAS together with Promenade. There's another mod that causes the same issue - MineGate, and in this case it prevents all mods from adding recipes for vanilla sticks. I didn't test VSAS with all such mods, but it's reasonable to assume that it also affects all mods that add new trees too.
There are no errors showing up in logs, so I'm not including any, but let me know if you need more info from me.

Should be resolved in latest release now up on Curseforge, lmk if not.
#

It's fixed, but just an FYI the behavior from https://github.com/xanthian/variant_sticks_and_stuff/issues/8 seems to be back. I tried crafting acacia sticks, but it prioritized vanilla sticks instead. That's not an issue for me personally, because I use Polymorph, but just letting you know.

arg

Yeah, it's highly likely to be caused by another mod, because I tested it in a modpack, rather than on its own, but the person who reported the issue was also using some modpack. Again, it's not an issue for me personally, so if they encounter it with this version, they should just reopen their issue.


B1 task

no baker id : 723

rejected:B1
It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing
We’ve thus marked B1 as rejected, however if this is a mistake, please upload the log summary and we can re-review.


[WIP] Revert "Disable parallel restore"

This reverts commit 0acc9d64fe3e9776f344f6a945278ea163a3875e.

Let's see whether parallel restore works now...

@dotnet-bot test this please

@dotnet-bot test this please

https://github.com/dotnet/cli/pull/1411

@dotnet-bot test this please


Deployment Unsuccessful without RBAC's Key Vault

Describe the bug
Unable to deploy the platform. This is due an issue with the deployment of the KeyVault, for security purposes I require it to have RBAC (Role Based Access Control). Without this the deployment will not run in my organisation.
However my team and I are struggling to find the configuration options for this in the cloned repo. Appreciate this may be a simple fix / pointer in the right direction but would be gladly appreciated.
Script ran in powershell:
./deploy.ps1 -aadConfig aad.json -branchName poc/pubsub_binary -type app
To Reproduce
Steps to reproduce the behavior:

Try to deploy on a managed subscription that only allows RBAC key vaults

Expected behavior
The deployment to be successful.
Error Message
"properties": {         "statusCode": "BadRequest",         "serviceRequestId": null,         "statusMessage": "{\"error\":{\"code\":\"InvalidTemplateDeployment\",\"message\":\"The template deployment failed because of policy violation. Please see details for more information.\",\"details\":[{\"code\":\"AppendPoliciesUndefinedFields\",\"message\":\"Found policies that refer to an undefined field property for api-version '2016-10-01'.  		Fields: 'Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*]'. 		Policy identifiers: '[{\\\"policyAssignment\\\":{\\\"name\\\":\\\"Policy Initiative to enforce RBAC in KeyVault to meet IRM requirements\\\"
Additional context
Add any other context about the problem here.

@NooraKubati I updated the KV apiVersion in the file azuredeploy.minimum.json. Could you please clone and deploy using branch lcantero/updatekvapi, then post any errors that you might get here.

Thanks @luiscantero can you please do the same on the poc/pubsub_binary branch?

@luiscantero Hi Luis thanks for your response. I have tried deploying from that branch and hit the same error again:
`Deployment will use 'lcantero/updatekvapi' branch in 'https://github.com/Azure/Industrial-IoT'.
Please specify a name for your application.

: #####
Start time: 11:21
Using preview images from mcr.microsoft.com.
Starting deployment...
11:21:53 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed because of policy violation. Please see details for more information.`

and further detailed in the activity log:
"properties": {         "statusCode": "BadRequest",         "serviceRequestId": null,         "statusMessage": "{\"error\":{\"code\":\"InvalidTemplateDeployment\",\"message\":\"The template deployment failed because of policy violation. Please see details for more information.\",\"details\":[{\"code\":\"AppendPoliciesUndefinedFields\",\"message\":\"Found policies that refer to an undefined field property for api-version '2019-09-01'. Fields: 'Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*]'. Policy identifiers: '[{\\\"policyAssignment\\\":{\\\"name\\\":\\\"Policy Initiative to enforce RBAC in KeyVault to meet IRM requirements\\\"
I am running this deployment under a managed subscription but after also creating a new tenant and AAD global admin, as it is a POC. It is possible this set up is also contributing to the failures.

@NooraKubati We are investigating and will get back ASAP.

@NooraKubati Please pull to update your lcantero/updatekvapi repo, then try to deploy again.
If it succeeds, you will need to perform the following manual steps to complete the deployment:

Go to the resource group, find the resource of type Managed Identity and copy its name
Select the Key vault, click on Access control (IAM) -> Add -> Add role assignment
Role: Key Vault Administrator
Select:
Save
Select the App service (for the Web app, not the services) -> Overview -> Restart
Allow up to ~5 min for the web app to be ready and refresh the page a few times

@luiscantero Hi Luis, I have tried your suggestions from scratch however the deployment is still hitting the same error unfortunately.
Please review my attempts below:
PS C:\WINDOWS\system32> cd "C:\Users\Noora.Kubati"
PS C:\Users\Noora.Kubati> git clone --single-branch --branch lcantero/updatekvapi  https://github.com/Azure/Industrial-IoT
Cloning into 'Industrial-IoT'...
remote: Enumerating objects: 21, done.
remote: Counting objects: 100% (21/21), done.
remote: Compressing objects: 100% (16/16), done.
remote: Total 42824 (delta 7), reused 13 (delta 5), pack-reused 42803
Receiving objects: 100% (42824/42824), 41.19 MiB | 7.88 MiB/s, done.
Resolving deltas: 100% (29102/29102), done.
Updating files: 100% (3102/3102), done.
PS C:\Users\Noora.Kubati\\Industrial-IoT> git branch
PS C:\Users\Noora.Kubati\Industrial-IoT> cd "C:\Users\Noora.Kubati\Industrial-IoT\deploy\scripts"
PS C:\Users\Noora.Kubati\Industrial-IoT\deploy\scripts> git branch
* lcantero/updatekvapi
PS C:\Users\Noora.Kubati\Industrial-IoT\deploy\scripts> ./aad-register.ps1 -TenantId YYYYYYYYYYYYYYYYYYYY -Output aad.json -Name doorsiot1009 -ReplyUrl https://doorsiot1009.azurewebsites.net/
Importing module AzureAD
Save credentials in .user file? [y/n]: y

Overwrite File?
Overwrite existing file at 'C:\Users\Noora.Kubati\Industrial-IoT\.user'?
[Y] Yes  [N] No  [S] Suspend  [?] Help (default is "Y"): y
Using context (Account<EMAIL_ADDRESS>Tenant XXXXXXXXXXXXXXXXXXXXXXX)
Selected Tenant 'DoorsIOT.onmicrosoft.com' as authority.
'doorsiot1009-service' updated with required resource access, app roles and known applications.
User<EMAIL_ADDRESS>already has all app roles assigned.
Admin consent granted to native client application.
'doorsiot1009-client' updated with required resource access.
Registering https://doorsiot1009.azurewebsites.net/ as reply URL ...
Admin consent granted to web application.
'doorsiot1009-web' updated with required resource access.
PS C:\Users\Noora.Kubati\Industrial-IoT\deploy\scripts> ./deploy.ps1 -aadConfig aad.json -branchName lcantero/updatekvapi -type app
Signing in ...

Please choose a subscription from this list (using its index):

Index Subscription                                    Id
----- ------------                                    --
    1 XXX YYYYYYYYYYYYYYYYYYYYYYYYYYYYYY
    2 XXX YYYYYYYYYYYYYYYYYYYYYYYYYYYYYY
    3 XXX YYYYYYYYYYYYYYYYYYYYYYYYYYYYYY

>: 3
Azure subscription XXX (YYYYYYYYYYYYYYYYYYYYYYYYYYYYYY) selected.

Please provide a name for the resource group.
>: XXX-IOT
Using existing resource group XXX-IOT...
Deployment will use 'lcantero/updatekvapi' branch in 'https://github.com/Azure/Industrial-IoT'.

Please specify a name for your application.
>: doorsiot1009
Start time: 14:39
Using preview images from mcr.microsoft.com.
Starting deployment...
14:39:53 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed because of policy violation. Please see details for more information.

Deployment failed.
Try again? [y/n]: n
New-AzResourceGroupDeployment : 14:39:53 - Error: Code=InvalidTemplateDeployment; Message=The template deployment failed because of policy violation. Please see details for more information.
At C:\Users\Noora.Kubati\Industrial-IoT\deploy\scripts\deploy.ps1:925 char:27
+ ... eployment = New-AzResourceGroupDeployment -ResourceGroupName $resourc ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [New-AzResourceGroupDeployment], Exception
    + FullyQualifiedErrorId : Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDeploymentCmdlet

in the portal:
 "statusMessage": "{\"error\":{\"code\":\"InvalidTemplateDeployment\",\"message\":\"The template deployment failed because of policy violation. Please see details for more information.\",\"details\":[{\"code\":\"AppendPoliciesUndefinedFields\",\"message\":\"Found policies that refer to an undefined field property for api-version '2016-10-01'. Fields: 'Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*],Microsoft.Keyvault/vaults/accessPolicies[*]'. Policy identifiers: '[{\\\"policyAssignment\\\":{\\\"name\\\":\\\"Policy Initiative to enforce RBAC in KeyVault to meet IRM requirements\\\",\\\"id\\\":\\\"<censored>"},\\\"policyDefinition\\\":{\\\"name\\\":\\\"Enforce RBAC in KeyVault to meet IRM requirements\\\",\\\"id\\\":\\\"/<censored>"},\\\"policySetDefinition\\\":{\\\"name\\\":\\\"Apply RBAC policies to Key vault\\\"
Internally we have removed all the chunks of code that refer to the key vault in the files: azuredeploy.minimum.json, azuredeploy.json and deploy.ps1
This has led to all the components being created successfully and the only failure point is the resource "infrastructure.minimum"
Please see below:

The error within this is the same as we have been experiencing (Enforce RBAC in KeyVault to meet IRM requirements).
For our use case we may be able to make this work as all the components are there bar the secrets for the vault so we will attempt to manually add these in.
Hope this helps your work.

@NooraKubati Could you please contact your IT and ask about more details regarding the policy? (the error is not standard, so I cannot check myself).
In the repo that we are using, we set the enableRbacAuthorization=true, which results in a Key vault with this setting:

@NooraKubati I just updated the template again and removed all access policies, which are not needed when using RBAC. Could you please try to deploy again? Thanks.

@luiscantero Hi Luis, unfortunately this did not succeed either. The key vault was made correctly with the required RBAC access, however this has not gotten rid of the error and there were no secrets/keys to be found.
Have just had a long call with some of your colleagues and I think this issue will take a lot of investigating so I am to happy to continue working on that together privately and maybe this issue can be closed for now. We can update later on with a solution if there is one to be found.

We will continue offline


MID  missing: This attribute is not passed. It is a mandatory attribute

Getting the error message for the following code,
        $payTMwallet = PaytmWallet::with('receive');
        $payTMwallet->prepare([
            'order' => 123,
            'user' => 11111,
            'mobile_number' => '77777 77777',
            'email' =><EMAIL_ADDRESS>            'amount' => '10',
            'callback_url' => 'http://LocalCalbackURL'
          ]);
          return $payTMwallet->receive();
    }

Which return an HTML page and which automatically submit and return this error.
http://i.prntscr.com/UQMIJb55S26WYwCJ1GjmYg.png

Check config, you might have missed adding.
Ref : https://github.com/anandsiddharth/laravel-paytm-wallet#configuring

@anandsiddharth I checked it still getting the same error.
steps I have done.

Added package by composer command.
Registered in bootstrap\app.php since I am using Lumen.
Added paytm-wallet array to config\services.php

<?php
return array(
    'paytm-wallet' => [
        'env' => 'local',
        'merchant_id' => 'paytmTestID',
        'merchant_key' => 'paytmTestKey',
        'merchant_website' => 'mysite.com',
        'channel' => 'WEB',
        'industry_type' => 'Retail',
    ],
);

controller code in here.

public function initiatePay(Request $request){
        $payTMwallet = PaytmWallet::with('receive');

        $payTMwallet->prepare([
            'order' => 123,
            'user' => 11111,
            'mobile_number' => '77777 77777',
            'email' =><EMAIL_ADDRESS>            'amount' => '10',
            'callback_url' => 'http://LocalCalbackURL'
          ]);

          return $payTMwallet->receive();
}

still getting the same error

MID
missing: This attribute is not passed. It is a mandatory attribute

In Lumen, services not registered by default. After adding the following line of code in bootstrap\app.php It works.
$app->configure('services');


PR description to include "upstream"/"origin" (remote) info

Type: Bug
Create two repos in GitHub, an upstream and a forked origin. Locally create a branch off origin/main, make changes, and push back to origin. Now start the flow to PR back to upstream -- the UX shows the remote names, but not their aliases (such as "origin" and "upstream") which is what usually the user works with from the git CLI.
Extension version: 0.70.0
VS Code version: Code 1.81.1 (6c3e3dba23e8fadc360aed75ce363ba185c49794, 2023-08-09T22:22:42.175Z)
OS version: Windows_NT x64 10.0.22621
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.92GB (1.20GB free)

Process Argv
. --crash-reporter-id 401b0534-16b6-4f87-b5bc-8667e33d2a44

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
282f8724:30602487
f6dab269:30613381
showlangstatbar:30737416
03d35959:30757346
7ij38806:30736111
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxptcf:30805731
pythonnoceb:30805159
dsvsc013:30795093
dsvsc014:30804076
diffeditorv2:30821572

Since it's GitHub related, we decided to show the GitHub remote names (ex. "Microsoft/vscode-pull-request-github" and "alexr00/vscode-pull-request-github") instead of their git remote names ("upstream" and "origin"). I'm going to leave this issue open for discussion for a bit.

@zivcaspi can you post a screenshot of what it looks like for you when creating the PR?

Closing due to lack of activity.


⚠ ClamAV has degraded performance

In 1e25656, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 130 ms

Resolved: ClamAV performance has improved in 72993a1.


[WIP][SPARK-34736][K8S][TESTS] Kubernetes and Minikube version upgrade for integration tests

What changes were proposed in this pull request?
This PR upgrades Kubernetes and Minikube version for integration tests and removes/updates the old code for this new version.
Details of this changes:

As discussed in the mailing list: updating Minikube version from v0.34.1 to v1.7.3 and kubernetes version from v1.15.12 to v1.17.3.
making Minikube version checked and fail with an explanation when the test is started with on a version <  v1.7.3.
removing minikube status checking code related to old Minikube versions
in the Minikube backend using fabric8's Config.autoConfigure() method to configure the kubernetes client to use the minikube k8s context (like it was in one of the Minikube's example)

Why are the changes needed?
With the current suggestion one can run into several problems without noticing the Minikube/kubernetes version is the problem.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
It was tested on Mac with this script which installs each Minikube versions from v1.7.2 (including this version to test the negative case of the version check) and runs the integration tests.
It was started with:
./check_minikube_versions.zsh > test_log 2>&1

And there was only one build failure the rest was successful:
$ grep "BUILD SUCCESS" test_log | wc -l
      26
$ grep "BUILD FAILURE" test_log | wc -l
       1

It was for Minikube v1.7.2  and the log is:
KubernetesSuite:
*** RUN ABORTED ***
  java.lang.AssertionError: assertion failed: Unsupported Minikube version is detected: minikube version: v1.7.2.For integration testing Minikube version 1.7.3 or greater is expected.
  at scala.Predef$.assert(Predef.scala:223)
  at org.apache.spark.deploy.k8s.integrationtest.backend.minikube.Minikube$.getKubernetesClient(Minikube.scala:52)
  at org.apache.spark.deploy.k8s.integrationtest.backend.minikube.MinikubeTestBackend$.initialize(MinikubeTestBackend.scala:33)
  at org.apache.spark.deploy.k8s.integrationtest.KubernetesSuite.beforeAll(KubernetesSuite.scala:163)
  at org.scalatest.BeforeAndAfterAll.liftedTree1$1(BeforeAndAfterAll.scala:212)
  at org.scalatest.BeforeAndAfterAll.run(BeforeAndAfterAll.scala:210)
  at org.scalatest.BeforeAndAfterAll.run$(BeforeAndAfterAll.scala:208)
  at org.apache.spark.deploy.k8s.integrationtest.KubernetesSuite.org$scalatest$BeforeAndAfter$$super$run(KubernetesSuite.scala:43)
  at org.scalatest.BeforeAndAfter.run(BeforeAndAfter.scala:273)
  at org.scalatest.BeforeAndAfter.run$(BeforeAndAfter.scala:271)
  ...

Moreover I made a test with having multiple k8s cluster contexts, too.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40621/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/40621/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/40621/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136038/

Hi, I'm experiencing in the same issue and and to resolve the issue I've applied the same changes. Could you please merge it ?

@fbalicchia Could you please share with us the error you run into (the one on which this PR helped on)?
By the way I cannot merge my own PR without an expert's approval (expert of the area where the PR belongs to). But your input will help the case.

Using minikube v1.18.1 default driver is Docker and at the moment of writing masterUrl not necessary bind on port 8443 https://github.com/apache/spark/blob/master/resource-managers/kubernetes/integration-tests/src/test/scala/org/apache/spark/deploy/k8s/integrationtest/backend/minikube/Minikube.scala#L69.
Instead using default config or using factory with autoConfigure("minikube") library help us

Test build #137071 has started for PR 31829 at commit de32835.

Test build #137071 has finished for PR 31829 at commit de32835.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137071/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41649/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41649/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41649/

Test build #137499 has started for PR 31829 at commit 1f594aa.

Test build #137499 has finished for PR 31829 at commit 1f594aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137499/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42073/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42073/

rebased on master

Test build #137502 has started for PR 31829 at commit 855273a.

Test build #137502 has finished for PR 31829 at commit 855273a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42076/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/42076/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/42076/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137502/

Test build #137566 has started for PR 31829 at commit 855273a.

Test build #137566 has finished for PR 31829 at commit 855273a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137566/

Test build #137602 has started for PR 31829 at commit 855273a.

Test build #137602 has finished for PR 31829 at commit 855273a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137602/

@dongjoon-hyun actually I am pretty sure this helps (even with the skipping of the PVTests which when @shaneknapp sets the Minikube driver to Docker will fix the Kubernetes integration tests).
So could you please review this as it is?


Cpp solution for Hoop Jump

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #574
Replace issue_no with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have created a helpful and easy to understand README.md
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works

@Amit366 @Ayush7614 Please review this PR and merge


Delete invalid reference in docs/reference/index.md

"api/index.md" or "docs/api/index.md" not exist, delete the reference.

Closing this PR because this is a dupe of https://github.com/docker/docker/pull/28921


cleanup: Enforce a line length of '100'

Description
I want to start cleaning up neofetch. This will include various PRs from me starting with this one. This PR limits the length of every line in the script to 100 characters max. This is done through breaking commands into multiple lines and using more variable where necessary.
I was originally going to limit lines to 80 characters but this seems impossible, hell limiting the lines to 100 proved difficult in places.
Later changes I'd like to make are as follows:

Make all variables local to their functions.

The top of each function will include: local var1 var2 var3.
This makes it easier to see where variables were declared.

Remove the dynamic variable use.
Make each function return a value.

Instead of setting a variable we'll print the value.

Document what each function takes as input and what it outputs.

This needs a thorough review as I could've very easily broken something in the process of doing this. Edit: I've already found some bugs in this PR.
Changes
This includes some smaller changes.

Add /usr/libexec to $PATH.

TODO

[x] Fix the remaining 6 or so lines that are still too long.
[x] Add checking for line length to Travis.

Shellcheck might have an option for this.

I can't see anymore bugs in this PR so I'm going to merge it.


ngOnChanges hook does not work in themes

Describe the bug
When overriding a stock component in a theme, the changes hook is never called.  The instance variable compRef is used in src/app/shared/theme-support/themed-component.ts:ngOnChanges but that variable is not set until ngInit runs, and that happens after the first call to ngOnChanges.
I note that none of the stock themeable components implements onChanges so this issue has not caused a problem until now, when I want to make section-upload-file.component.ts themeable.
To Reproduce
Steps to reproduce the behavior:

Write a themed replacement component that depends on change notification.
Notice that no changes notifications are received.

Expected behavior
All component lifecycle hooks should function in themes.

SubmissionSectionUploadFileComponent breaks as soon as I introduce ThemedSubmissionSectionUploadFileComponent, before actually creating a new implementation in my theme.

This is caused because the ThemedComponent.ngOnChanges() here updates the @Input() variables from the SubmissionSectionUploadFileComponent directly. By updating those variables manually, the ngOnChanges() method is not triggered. The fix would be to call the ngOnChanges() method with these changes instead.
Atmire would like to claim this issue

@alexandrevryghem : Sounds good & thanks. Assigning to you and pulling over to the 7.6 board

#2202 may be useful in testing a fix for this issue.

@mwoodiupui : a possible fix for this issue was just created in #2235.  So, you may want to try out #2235 to make sure it fixes the issue you've encountered

I found out that the ngOnChanges is also not called components who don't use the regular theming method but use the custom decorators instead:

ListableObjectComponentLoaderComponent: @listableObjectComponent
MetadataRepresentationLoaderComponent: @metadataRepresentationComponent
ClaimedTaskActionsLoaderComponent: @rendersWorkflowTaskOption

I can fix those too in this PR 🤷♂


Update what are the necessary internet endpoints used for Azure databricks clusters

The customer reported missing information about internet endpoints used for Azure databricks clusters on github.Please refer below links:
https://github.com/MicrosoftDocs/azure-docs/issues/103912

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

ID: a44cbeaf-af2c-ef7e-b411-24556279d898
Version Independent ID: aaced3ee-7d7c-a9d6-a81b-b132674fa055
Content: User-defined route settings for Azure Databricks - Azure Databricks
Content Source: databricks/administration-guide/cloud-configurations/azure/udr.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

@mssaperla
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

Hi @RamanathanChinnappan-MSFT
Thanks for bringing this issue to our attention.
I've delegated this to content author @mssaperla, who will review it and offer his insightful opinions.

Thanks for your dedication to our documentation. We've created an internal work item (DOC-7909) to address your feedback. The timeline for resolution varies based on resourcing. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


Add Windows Slider.

Summary: Adding the Windows implementation of this module.
Implemented the following properties:

maximumValue, minimumValue, value, step
inverted, disabled, vertical (Windows-specific)
minimumTrackTintColor, maximumTrackTintColor
thumbTintColor, thumbImage
onValueChange, onSlidingStart, onSlidingComplete

Closes microsoft/react-native-windows#4612
Updated the README with information on how to run the test app on Windows, and how to link this module to existing projects.
(Also moved the example app's one into the main file in order to improve coherence).
Updated the version of React Native for this module to >=62 - I'd like to begin a discussion on whether this is fine for this module, or should be done only for Windows, which requires that version.
Also, spotted dead code in Slider.js - the default prop values don't get picked up at all - changing them doesn't alter the app's behavior. Given this and the fact that a default value of 0 for the step prop seems mistaken, I'm also updating it by placing it in the component definition (can be overridden easily). (defaultProps is not available on components returned by forwardRef, as is the case here - https://github.com/facebook/flow/issues/7467)
Test Plan:
As a subsequent PR, I'd like to add a basic CI loop for this. Since there are issues with Windows builds on CircleCI, it'd have to be through the addition of Github actions.
You can run the test app on Windows to see it in action.

@wojteg1337 @michalchudziak
My colleague has signed off on this one. Can you guys take a look as well?

@michalchudziak Thanks for the approval!
Before you merge it, I just wanted call out the fact that I bumped up the version of RN to 62, and took a guess as to this being a breaking change, which would result in the release number getting change to something like 3.1.X (indicated in README).
Let me know if this is fine, or whether you'd prefer to keep this module on an older version of React Native, with a note that on Windows the latest is required.

@michalchudziak What do you think about my last comment? :)

@michalchudziak @wojteg1337 Can you merge this one, plz? Do you want to address the question from my previous comments?


jmxtrans run_interval in bcpc-hadoop is overridden

The run_interval of jmxtrans is currently set in bcpc-hadoop/attributes/default.rb. Due to chef compile order, it is overridden and doesn't take effect. We may need to move it to somewhere else, such as bcpc-jmxtrans.

We should probably move all jmx related attributes into bcpc_jmxtrans cookbook's attributes


WIP refactor to separate functions and add tests

Relatively large refactor to improve ease of testing functions separately. Will add more details when ready for review.

Going to have to update the tests since the new refactor returns data in a different format.


Windows Analytics uninstallation

Hello,
how can the Windows Analytics client be uninstalled or disabled on the clients?

@officedocsbot assign @mypil

@Kofl - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket so this can get resolved ASAP. Based on outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for Windows 10 products:  https://support.microsoft.com/en-us/hub/4338813/windows-help?os=windows-10
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@officedocsbot close

Sorry, but that wont help out. Windows Anlaytics is no software which can be uninstalled by a default way for Windows 10. If you have a look at the installation script, uninstall can only be provied by the product team of Windows Anlaytics.

@officedocsbot open

@officedocsbot open

@Kofl - May we know which Microsoft document link you used in reference to this issue? Please share it with us so we can investigate further. Thank you.

Hi,
https://support.microsoft.com/en-us/hub/4338813/windows-help?os=windows-10 wont help on uninstalling the software. We need a script from the product team to clean out the changes to disable Windows Anlaytics.

@Kofl - Sorry to hear that you are having issues. Please understand that this site is a product documentation site and not a support nor a feature request site.
As such, the feedback section here is intended for feedback that improves the content in the Microsoft documentation and not intended for user feedback on product features nor troubleshooting.
So, with this said, what is the Microsoft article that you are referring to? Can you please share the link so we can have it investigated?
Thank you.

Sure, thats clear. The documentation is missing how to uninstall/disable Windows analytics, its not mentioned under the FAQs https://docs.microsoft.com/en-us/windows/deployment/update/windows-analytics-faq-troubleshooting nor mentioned elsewhere.
A complete documentation should could installation AND uninstallation/disabling of the service.

@jaimeo - We need your insights on this issue. Do you have any information that we can share with @Kofl to answer his query?
Thank you.

@AndreaBarr - please assist to re-open this issue. thank you.

Is this not what you're looking for? https://docs.microsoft.com/en-us/windows/deployment/update/windows-analytics-faq-troubleshooting#disable-upgrade-readiness

@Kofl - kindly check on the response and confirm if the link shared by jaimeo helps to resolve your issue. Thank you.

Thanks, that clarified it. The confusing part is " Disable Upgrade Readiness" and "Enrolling" for the installation.

@Kofl - Is there something within this article that you suggest to be added or edited?
If yes, please share the appropriate text and where it should be placed, so that a Pull Request can be created to get it added. Thank you!

I think we can stick to the original text, thanks for asking.


Broken for python3.10

It fails to execute on python3.10
❯ aws-okta-processor --help
Traceback (most recent call last):
  File "/Users/dacha204/.local/bin/aws-okta-processor", line 5, in <module>
    from aws_okta_processor.cli import main
  File "/Users/dacha204/.local/pipx/venvs/aws-okta-processor/lib/python3.10/site-packages/aws_okta_processor/cli.py", line 25, in <module>
    from . import commands
  File "/Users/dacha204/.local/pipx/venvs/aws-okta-processor/lib/python3.10/site-packages/aws_okta_processor/commands/__init__.py", line 1, in <module>
    from . import authenticate # noqa
  File "/Users/dacha204/.local/pipx/venvs/aws-okta-processor/lib/python3.10/site-packages/aws_okta_processor/commands/authenticate.py", line 30, in <module>
    from aws_okta_processor.core.fetcher import SAMLFetcher
  File "/Users/dacha204/.local/pipx/venvs/aws-okta-processor/lib/python3.10/site-packages/aws_okta_processor/core/fetcher.py", line 5, in <module>
    import aws_okta_processor.core.prompt as prompt
  File "/Users/dacha204/.local/pipx/venvs/aws-okta-processor/lib/python3.10/site-packages/aws_okta_processor/core/prompt.py", line 4, in <module>
    from collections import Mapping
ImportError: cannot import name 'Mapping' from 'collections'<EMAIL_ADDRESS>
It works fine on python3.9.
Steps to reproduce:
brew install<EMAIL_ADDRESS>pipx install --python<EMAIL_ADDRESS>aws-okta-processor
aws-okta-processor --help

I've changed the following line, locally
https://github.com/godaddy/aws-okta-processor/blob/7c5d98ee0f6f3f3d64d7bdf9eccae596406e06cc/aws_okta_processor/core/prompt.py#L4
to
from collections.abc import Mapping

and it's working with Python 3.10

@rwebler-godaddy Thanks for checking into this! I'll see about providing a patch that does that in a backward-compatible fashion. 👍🏻


Integration Test Fixes

This PR contains a variety of fixes to the integration test framework to address recent issues. Fixes included:

graphql ingress race condition (nodes failing to initialize, but initialization log is present)
context deadline timeout on network destructions
skip deploying graphql ingresses for tests that do not require it
pod assignment race condition (very rare, but became more frequent with changes introduced in this PR)
seed node artifact generation race condition (nodes failing to initialize, unable to find initial peers and restart in time)

!approved-for-mainnet


Please add ARM version in ECR Public Gallery

Currently https://gallery.ecr.aws/xray/aws-xray-daemon does not have an ARM version available, but ARM version of xray exists on the Docker repo.
Please add it to the gallery so we can pull from there.

Hi @kitkit201 ,
Thanks for opening this issue. We are currently investigating this issue. I will let you know once I have more updates to share!


How much gpu memory does the InternVL-Chat-ViT-6B-Vicuna-13B  model take up when running?

How much gpu memory does the InternVL-Chat-ViT-6B-Vicuna-13B  model take up when running? And what is the approximate running speed if I use A100?

你好，如果是运行llava code里的在线demo的话，InternVL-Chat-ViT-6B-Vicuna-7B占用28G显存，InternVL-Chat-ViT-6B-Vicuna-13B占用47G显存


Understanding the components page translated

The understanding the components page has been translated into Spanish

Done @diemol !


Honour network settings in podman play/generate kube

/kind feature
Description
I am uncertain whether the kube yaml standard supports network settings.
Either podman play kube should honour network settings, e.g. --network or the generated yaml file using podman generate kube should take into account network settings for pods and/or containers.
As far as I can see in #5241 network opts have been added to podman pod create but there seems to be nothing for either podman play kube nor podman generate kube.
For single containers and systemd generation this is already supported using
podman generate systemd {ctrID} --new
Thanks a lot for this.
I checked versions 1.6.4 (CentOS 8), 1.8.0 (Fedora 30) and 1.8.2 (Kubic Repos for CentOS 8)

Interested in opening a PR to fix?


Use Re-entrant locks instead of synchronized blocks to make driver loom freindly

Loom project: https://openjdk.java.net/projects/loom/
Using synchronized keyword pins the virtual thread on the platform thread degrading the performance. Using ReentrantLock should prevent it from happening.
Sample code:
void foo() {
  synchronized(this) {
     ... // performs IO or something blocking
  }
}

With code like:
// loom friendly ...

private final Lock lock = new ReentrantLock();

void foo() {
  lock.lock();
  try {
     ... // performs IO or something blocking
  } finally {
     lock.unlock();
  }
}

@mthakur77 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree company="Agoda services pvt ltd"

raising new one.


x/build/dashboard: update Debian Sid builders (linux-amd64-sid, linux-386-sid)

Debian Sid, also known as "Debian Unstable", is an ever moving target. Our host-linux-amd64-sid is described as "Debian sid, updated occasionally."
It was last updated in Nov 2019. Given this builder doesn't explicitly try to provide coverage for older Linux versions, I've updated it to a newer image built in Nov 2022 as part of builder maintenance work this week. If it causes problems, we can revert or follow up, otherwise it being more up-to-date should provide more value (more signal, less noise).
CC @golang/release.

It found that TestGdbBacktrace, TestGdbPython, and GdbPythonCgo tests in package runtime aren't passing on a newer Debian Sid:
https://storage.googleapis.com/go-build-log/237a723c/linux-amd64-sid_5ce55b68.log
https://storage.googleapis.com/go-build-log/237a723c/linux-386-sid_d3d75d30.log
The failure looks very similar (if not identical) to #54352, and the explanation in https://github.com/golang/go/issues/54352#issuecomment-1210781002 lines up—the update caused gdb version 12.1 to be tested, which is newer than 11.1 mentioned there.
Reverted to the previous version for now to avoid creating disruption.


Make checking too many errors optionally

Support ignore too many error as advanced configuration, instead of system configuration.

Add finished with warning status.

Support ignore too many error as advanced configuration, instead of system configuration.

good job^^


App randomly stopped scanning sensor

Was working perfectly until just over an hour ago - since then it hasn't taken any readings. I'll send you the log file.

Thank you for the email and the log file. I was able to correct the error through the data and have already committed a fix. If you built the app yourself with Xcode you can already get the update. But I will also submit an update to Apple tonight.
Many greetings


Update readme

@hrehman-sphereon
Some questions:

Where does the JwtObject under step 3 come from? And please check if that part is still correct after my changes.
Last release step, final sentence of document -  am I pointing to the correct readme?
Please read the new background carefully as I made some clarifications after our last talk

There were conflicts which have been resolved and pushed in the next PR. This is obsolete.


PP-10862 Combine organisation details Cypress tests

Cypress best practices suggest that having tests rely on the state of previous tests is an anti-pattern.
Combine tests and remove use of Cypress.Cookies.preserveOnce() so that tests can be run independently.
Remove the .within(() => {}) blocks from the tests as they made it difficult to read once the tests were combined.
Add in { delay: 0 } to the options when calling .type() on an element to speed up text input to make runtime less.
Review note
Hiding whitespace changes from the diff will make this a lot easier to review

Great refactor and nice optimisations 👍 I wonder if its worth doing something like (from a Cypress question thread):
Cypress.Commands.overwrite('type', (originalFn, subject, text, options = {}) => {
  options.delay = options.delay || 0;

  return originalFn(subject, text, options)
})

And then explicitly slow it down if it needs to simulate typing speed (testing client side js behaviour or something). It does feel a little brittle (assuming the command type doesn't change name!) but I don't think there's a global config -- specifying per command looks fine to me as well

That seems like a great suggestion, thanks! I was looking for a global config.
As we don't use much client-side javascript in selfservice, it should probably be fine in most cases never having a delay?


Respect ANDROID_AVD_HOME environment variable

[x] This is a small change

Description:
Android SDK tools respect the ANDROID_AVD_HOME environment variable to locate installed emulator instances. This change adds support for that setting when fixing up the config.ini file.

Could you check this? https://github.com/wix/Detox/pull/1573#discussion_r314241163

Looked at #1573 and I investigated a bit. I ran emulator -list-avds in different configurations and got the following to list the avds correctly:

With avds inside /tmp/foo/.android/avd: ANDROID_SDK_HOME=/tmp/foo emulator -list-avds
With avds inside /home/len/android/avd: ANDROID_EMULATOR_HOME=/home/len/android emulator -list-avds
With avds inside /home/len/android/avd: ANDROID_AVD_HOME=/home/len/android/avd

I will update the branch to reflect this.
Reading #1573 I am not sure why ANDROID_SDK_HOME did not work.

Hmm, I have a hard time deciphering the Jenkins logs to figure out what went wrong during the checks of the second commit. I also fail to understand why it would break while the first commit succeeded. Any pointers are welcome.

Because the original code also contained /, but I can fix that. Also gives me a way of retriggering the checks :) Or is there some easier way than pushing a new commit?

If nobody looks earlier, I can look on Monday.

That would be nice. As an experiment I force-pushed my initial commit (which completed fine earlier) and the checks also failed, so something seems to have changed in the test setup after my initial commit?
For now I re-pushed my other commits.

@lvanengelen, thank you for your contribution!

Just out of curiosity, could you explain why CI kept failing after my earlier pushes?

I am not sure about everything, but, in particular, the Jenkins OSS CI had undergone some configuration changes and only the recent commits to master have fixed android-rn-59 problems and maybe something else. I have noticed that your branch has been not in sync with master for enough time to start getting some of those issues.


Add user agent infos when auth with AzureClientFactory

Add user agent infos when auth with AzureClientFactory, for #1015

Codecov Report

Merging #1021 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             develop   #1021   +/-   ##
=========================================
  Coverage       38.6%   38.6%
  Complexity       626     626
=========================================
  Files            112     112
  Lines           4696    4696
  Branches         728     728
=========================================
  Hits            1813    1813
  Misses          2736    2736
  Partials         147     147

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 bdc2257...271a7e5. Read the comment docs.


fix(pipeline) commented datadog-a7 to unblock windows builds

Fixed windows build failure due to transitive dependency that added files named in some asian language

Superseded  https://github.com/StackVista/stackstate-agent/pull/243


JarSearchableOptionsTask cache miss

What happened?
The jarSearchableOptions task is exhibiting a cache miss caused by its sandboxDir input property being set to an absolute path. Since sandboxDir is a String property, the path it is set to should either be relative, or, if changes to the contents of the directory affect the outcome of the task, sandboxDir should be converted to a DirectoryProperty with relative path normalization, so that its contents can be properly tracked by Gradle.
Relevant log output or stack trace

Task input comparison showing sandboxDir input difference
Custom value comparison showing the sandboxDir input difference is caused by an absolute path

Steps to reproduce
In a project using gradle-intellij-plugin, run a clean build executing the jarSearchableOptions task. Clone or copy the project into another file system location, and run the same build again. The jarSearchableOptions task in the second build should come from cache, but it does not.
Gradle IntelliJ Plugin version
1.17.3
Gradle version
8.7-rc-1
Operating System
None
Link to build, i.e. failing GitHub Action job
No response

I noticed that on the main main branch, the JarSearchableOptionsTask has been restructured to be SandboxAware, and that the sandboxDir property no longer exists - looks like it has been split into multiple @Internal DirectoryPropertys. In that case, not sure if you will want to fix the existing implementation, but based upon the updated implementation, perhaps sandboxDir should just be marked @Internal?

Thanks for your input, Tyler! And sorry for the pretty late reply ;-)
The sandbox story in 2.0 faced a couple of iterations and currently it's indeed Internal. The prepareSandbox task (the "sandbox producer") has in addition the sandboxDirectory path set as an input property (String, to do not consider its content)
https://github.com/JetBrains/intellij-platform-gradle-plugin/blob/fe3be513ad3ff265fae1d255662b27b369f253b9/src/main/kotlin/org/jetbrains/intellij/platform/gradle/tasks/PrepareSandboxTask.kt#L316
Creating of all subdirectories within the sandbox is now also properly handled and tested.


Update expo-deps workflow to run every other even day

What kind of PR is this?:

/kind chore

don't understand why is this workflow failing

@fuxingloh this won't work because react-native-qrcode-svg has a different RN dependency than Expo. expo upgrade is running on background, then it fails because npm install doesn't work. It needs to be npm install --force.

react-native-qrcode-svg

We need to resolve it somehow, something for another day.


update docker_remote_api_v1.19.md

Signed-off-by: Yuan Sun<EMAIL_ADDRESS>

Looks like we would not be able to merge this PR because of merge conflicts. Please fix them and force push to your branch.

Nice catches! LGTM

@SvenDowideit @fredlf @jamtur01

Oh, good catch, can't believe I missed those; Looking at this; the same typos are also in the 1.18 API docs. Would be great to have them included as well, but if you can't, don't worry, it can also be done in a follow-up PR

Hi @thaJeztah ,
No problem. They have been updated。^_^
sed -i 's/containg/containing/g' *.md
sed -i 's/othercontainers/other containers/g' *.md
sed -i 's/neworking/networking/g' *.md
sed -i 's/capabilties/capabilities/g' *.md
sed -i 's/byets/bytes/g' *.md
sed -i 's/exection/execution/g' *.md

Oh, that's awesome @sunyuan3! :heart: (I was careful suggesting, because not all contributors are experienced Git users, which can be troublesome)
Thanks so much. LGTM (not a maintainer)

LGTM


Run receiver on first call to Halogen.Aff.Driver.runUI

I wrote my first Halogen app assuming that running runUI would automatically run the receiver of the ComponentSpec with the input I provided it and trigger a query for my component that would initialise the state after sending an AJAX request.
But this did not work. Instead, I had to do:
io <- runUI Home.component unit body
io.query $ H.action $ Home.SomeQuery

to trigger the initialisation which did what I wanted.
Cases like these where you want to initialise an app's state through an effect (which you clearly can't do with initialState, which is pure) seem quite common - why not make it a default through invoking the receiver?
If not made default, the docs should be made more clear that the even though both initialState and receiver take an input, only initialState is executed when you run the component with an input. This information is not clear at all from just looking at the types.

You can use a lifecycleComponent and have it raise a query using init to achieve this - the initial state will be set before Init fires. This applies to all components, not just the top level :)

Thanks! Do you think it's worth clarifying the point I mentioned in the docs?

Yeah definitely, I have a TODO in the readme to add docs about the lifecycle stuff, I was going to add something in there to explain it, and link across from the section on receivers. I just need to actually write it!

Great! Just wanted to make sure it's tracked somewhere. Closing.


Add install instructions to Readme

Thanks for opening this. I'll get on adding this to the readme, but you can install this package through package control in the standard way

I installed the package before I found this, but thought it was another once since there is no highlighting of lines exceeding 50chars and no highlighting for the diff

I'll add diff highlighting to the roadmap.
Do you have an example of it not highlighting on lines that are too long?

I've updated the README, and I've included diffs in the syntax. Just waiting on package control to update with my latest release


feat: modify attachments to be compatible with pages based on folders

now attachements supports:

content

page.files

attachment.pdf

page.md
_index.md

and

content

page

index.md
files

attachment.pdf

_index.md

Can you update documentation about attachments shortcode ?
Thanks a lot

Off course, I can :)
Is that enough to modify exempleSite or there is other changes to do ?

Only exampleSite

done in #10


dts/bindings: document use of value 'use-prop-name'

'use-prop-name' is not documented.
Update dts/bindings/device_node.yaml.template to fix this.
Fixes #9971
Signed-off-by: Erwan Gouriou<EMAIL_ADDRESS>

@SebastianBoe: changes done. Thanks for the review!


⚠ Bofrost.at has degraded performance

In 9362b9f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1479 ms

Resolved: Bofrost.at performance has improved in 41ddd13.


Ability to restart a build.

This is a feature request for your consideration.
Something that's possible to do in Travis CI is to restart a given build, for a PR or for a commit.

Sometimes a build fails due to temporary issues, such as a network connectivity issue, or because one of the dependencies wasn't pushed, causing build errors.
With GopherCI, as far as I know, it's not currently possible to restart a build, so if it fails, you're no longer able to check if that commit has issues after resolving the temporary problem, and you're stuck with a permanent failed status on the commit or PR.
This has happened to me recently.
Commit
https://github.com/shurcooL/home/commit/a71dbf64bef731e9272bb6d36b40fe8ea83e0fe7.

https://gci.gopherci.io/analysis/1228
https://travis-ci.org/shurcooL/home/builds/308193327

PR
https://github.com/shurcooL/githubql/pull/27.

https://gci.gopherci.io/analysis/1215
https://gci.gopherci.io/analysis/1214
https://travis-ci.org/shurcooL/githubql/builds/306504263
https://travis-ci.org/shurcooL/githubql/builds/306504256

Thanks, I'm surprised I didn't have an issue for this already.
You're right it's needed, it'll be simpler if I simply allow anyone to restart a build, the challenge was making sure I kept enough information around to restart a build.


Add option withoutMinutes to time field.

For time field I need to be able to select only hour, not minutes.
I added option withoutMinutes that if set to true eliminate showing
minutes overlay.

If you need to select just hours, why don't you just create a drop down with hours only. Or just an input field?

I think it would be an usefull option for time field. And I don't think in worth to create another control type for this reason.

I took a closer look and it looks good. I think, however, it would be better to rename "withoutMinutes" to "noMinutes" as it is shorter. I will do that.
Thanks for your contribution.

I also added it for datetime field.


User can login without RBAC permissions

Describe the problem you'd like to have solved
I use Auth0 and RBAC for an API. Only users with the required permissions are allows to access the API. This works on the server side by validating the scopes of the access token. But in react users without the required permissions still successfully login and get an access token. The token does not contain the required claims (scopes) which are requested in the Auth0Provider. Also when using getAccessTokenSilently and specify a specific scope and the user does not have permissions of that scope, an access token is return without the required scope, which is useless to make a request to the API.
When the user logs in there is no hint or notice that he/she has not the required permission, which were requested by the application.
Describe the ideal solution
getAccessTokenSilently should throw an exception instead of returning an access token which does not have the requested scope.
Alternatives and current work-arounds
One must manually check if the requested scope is contained in the IdToken claims using the getIdTokenClaims function.

Hi @Legion2 - thanks for raising this

One must manually check if the requested scope is contained in the IdToken claims using the getIdTokenClaims function.

Yep, this is our current best practice for implementing RBAC in a SPA
Closing this for now, feel free to reopen if you have any other questions

for reference https://community.auth0.com/t/checking-user-scopes-in-react-spa/41410

for reference https://community.auth0.com/t/checking-user-scopes-in-react-spa/41410


Disabled auto discovery still throws exception on createMessaging

Describe the bug
When disabling auto discovery in the factory exceptions are still thrown when creating the messaging instance.
To Reproduce
$sa = ServiceAccount::fromJsonFile('/path/to/sa.json');
$sa->withProjectId('project-id');
$factory = new Factory();
$factory->withServiceAccount($sa)->withDisabledAutoDiscovery();
$messaging = $factory->createMessaging();
/** Throws:
Kreait\Firebase\ServiceAccount\Discovery\FromEnvironmentVariable: The environment variable "FIREBASE_CREDENTIALS" is not set.
Kreait\Firebase\ServiceAccount\Discovery\FromEnvironmentVariable: The environment variable "GOOGLE_APPLICATION_CREDENTIALS" is not set.
Kreait\Firebase\ServiceAccount\Discovery\FromGoogleWellKnownFile: The well known file is not readable or invalid
*/

Expected behavior
No required environment variables or well known file.
Environment (please complete the following information):

OS: Docker php-fpm:7.2
PHP version: 7.2.24
Firebase SDK Version: 4.35.1

Additional context
Perhaps I am doing something wrong here - but I would expect this code to fully disable auto discovery.

Each call on the factory will return a new instance of the factory, so when you're calling $factory->createMessaging(), you're actually using the Factory from the $factory = new Factory(); line.
This should work:
$factory = new Factory();
$factory = $factory->withServiceAccount($sa)->withDisabledAutoDiscovery();

$messaging = $factory->createMessaging();


BFGS could accept an initial inverse hessian like TF

#4594 could be solved and performance improved if it were possible to supply an the initial state of the hessian or it's inverse as in the TF probability implementation.
Would you consider a PR for this?

I'm open to this, but I guess the natural question is why this feature makes sense for JAX if it doesn't exist in SciPy?

@davidia I experimented with this when implementing the BFGS for JAX and it's not a good idea in many situations. Firstly, not all functions have well-defined Hessian inverses. Secondly, even when there is, this can lead to poor BFGS convergence behaviour. I determined this empirically. At the moment, I think this feature should not be added for stability reasons and because it would deviate from SciPy. However, I'd be open to exploring this more.

#4594 is already solved by #4594. It's just waiting to be merged @shoyer can you poke it?

I tried the Jax implementation of the BFGS but the algorithm doesn't find a proper line search at the first iteration whereas the Scipy implementation works very well. By digging in both implementations, it seems that the inverse Hessian is initialized with the identity matrix but  Nocedal and Wright in "Numerical Optimization p.143 formula (6.20)" proposed an heuristic method to scale the Hessian matrix at the first iteration. This is implemented in Scipy FullHessianUpdateStrategy but it seems is not used by their BFGS algorithm. What do you think? Is it worth testing?

@Joshuaalbert got it, thanks.

@LoicRaillon Can you post a minimal example showing this behaviour? Also, keep in mind line-search behaviour isn't necessarily related to the inverse Hessian. Line search only requires a search direction and then it approximately minimises a scalar function along that line within some constraints.
The BFGS as it exists in scipy, and here in JAX starts with an initial inverse Hessian of identity. While there are other options, often the identity is a good enough starting point (this is also said by Nocedal and Wright). The reasoning is that the extra costs typically don't improve the solution. Remember BFGS and it's variants far variable metric methods, and start of as gradient descent and asymptote to approximate Newton methods. The speed of convergence is often fast (only a few iterations). However, I am interested in exploring this a bit more. Therefore, I would need some examples.


Provide system and gettext as ESM

Both system and gettext are currently available to be imported as ECMAScript Modules, I think since GJS supports ESM. It would be nice to be able to import them in such a way using ts-for-gir, since the default GNOME Builder template imports system that way :)

I agree, we should support the following:
import gettext from 'gettext';
import system from 'system';

And something like that:
import { exit } from 'system';
import { ngettext as _ } from 'gettext';

Cairo is another one like this, it doesn't use the gi:// format as far as I know. However, Cairo does not provide named exports according to the docs.

Added in #106


Databricks Runtime Version 5.2 no longer available

Runtime version 5.2 is no longer available for selection in the selectionlist when creating a new cluster. 5.3 is the current oldest 5.X version you can select. There are 4 different 5.3 runtimes available, 5.3 with or without GPU and 5.3 ML with or without GPU. Please specify which one to choose in the context of the article and perhaps a link to documentation explaining the differences.

@mdmaarschalk : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@mdmaarschalk - Thank you for your feedback and contribution! I will sign off on this change and see to making the changes you recommended. #sign-off


Add "start cluster" before showing admin ui link in Brew install caveats

During onboarding research, one user clicked on the admin ui link (marked below) before starting a cluster. Could we add "start a cluster" commands in the caveats or simply removing the admin ui portion at this stage?

cc @benesch

We don't really document what gets printed there. In other words, our homebrew installation docs don't mention there's a launchd configuration to start a single-node cluster. Should they, @benesch?

Sorry for the confusion! I told @kuanluo to open the issue here because we don't have anywhere else to file it now that the Homebrew formula lives in the Homebrew project proper. It's not exactly a docs issue.
Anyway, we probably don't want to call out the launchd configuration in our Homebrew docs. It's shown to be confusing if you don't know how brew services work.

Got it. I'll just assign to you then, @benesch. Let me know if there ends up being something to do for docs.

Oops, sorry for the misunderstanding, @jseldess!
If we can submit a better copy for this, it would be great!


Problem with printer

Sometimes let me use it but sometimes didnt, idk why, isnt a crash.
Sry if my explanation isnt good im french.

Btw im using the lastest 1.8.0 version

Does that happen on a server? If so, he server owner most likely disabled the feature.

Yes, happens on a server (also in sp), but sometimes let me use it or not.. so i don't think that the owner disabled that feature

I'll need the fml-client-latest.log file from when the printer is disabled. Also, some of the earlier Schematica builds had some issues that didn't properly block some features due to a 1.7 -> 1.8 quirk.

I solved the problem, just works when you open the schematic for the first time and then u can enable the printer mode but when you open it again you can't use printer mode.


A.12.IR15.spatialdataservicetype.node

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
This test case only needs to be executed in the context of 'scenario 2'. Prerequisites should reference other test cases in the same ATS or a conformance class, not introduce additional assertions. I.e. What should the test result be in case of 'scenario 1' (skipped, not-applicable, etc-)?
Proposed Change
Incorporate this 'precondition' into the test method. Alternatively, it may be better to merge test cases A.04, A.05, and A.07-A.024.

Agreed that a scenario 2 response is a precondition for this test.  Not sure if this test can be merged with others.

Implemented precondition that test case metadataURL.node must indicate scenario 2


[Bug]: Did not receive verificatio email when registering

What happened?
The latest main branch, on your website https://librechat-librechat.hf.space/ and my local development, have not received the verification email.
p.s. I can’t receive it with other Gmail accounts either.
Steps to Reproduce

Click [sign up]  to register and fill in the information, such as<EMAIL_ADDRESS>Click [continue] to be prompted to check the verification email and jump to the login page.
I checked the<EMAIL_ADDRESS>email address and did not receive the verification email, nor did I click Resend.

What browsers are you seeing the problem on?
Chrome
Relevant log output
My local log:

2024-06-08 10:55:01 info: [sendVerificationEmail] Verification link issued. [Email<EMAIL_ADDRESS>2024-06-08 10:56:16 error: [sendEmail] connect ETIMEDOUT 74.xxx.xxx.109:465

Screenshots
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

I'm not sure why email password reset / verification isn't working on our demo, I've set ALLOW_UNVERIFIED_EMAIL_LOGIN=true to bypass the requirement for verification in the meantime
(It's working from my home server deployment)


How could we declare a variable or pointer of lmrtfy::thread_pool

How could we declare variable lmrtfy::thread_pool, for example,
in a header h
lmrtfy::::thread_pool pool_var;
//Or
lmrtfy::::thread_pool * pool_ptr;
in a cpp
pool_var = lmrtfy::thread_pool();
lmrtfy:::thread_pool pool
pool_ptr = &pool;
Thanks!

I'm not sure why exactly you want a pointer to it, you might have to explain more. Perhaps you want a reference? For example, if you're passing a lmrtfy::thread_pool to a function, you could declare it something like
void foo(lmrtfy::thread_pool<>& pool) { ... }

(I believe the <> are necessary when passing an empty-template-list template into a function)
Or perhaps you want it to be global?

For example,
In a head file - myclass.h, I will declare a pool variable:
lmrtfy::thread_pool<> pool_vars;
In a cpp file - myclass.cpp, I will initialize the pool variable when myclass running:
pool_vars  = lmrtfy::thread_pool ;
How can I do that?
Thanks!

If it's a global variable, and you are okay with it being initialized at startup and destroyed at exit, an easier way is to put
inline lmrtfy::thread_pool pool_vars;

in your header, you don't need to define it in a separate .cpp file.
If you need to delay initialization or perhaps end it early, you could do something similar:
inline std::unique_ptr<lmrtfy::thread_pool> pool_vars;

Then, when you want to initialize:
pool_vars = std::make_unique<lmrtfy::thread_pool>();

And when you want to destroy it:
pool_vars.reset();

If you don't need it to be global, perhaps a member of a class, just declare it as a member of the class and it will be correctly default initialized in any constructor.
Let me know if you are trying to do something else.


False Positive with Style/CollectionCompact on Hash

There are times when using compact on a Hash is not appropriate, and reject is, as they can return different results:
{ key1: :foo, key2: false, key3: nil }.compact == { key1: :foo }

{ key1: :foo, key2: false, key3: nil }.reject { |_, value| value.nil? } == { key1: :foo, key2: false }

There are times where the second one is the desired output (and reject must be used), but Rubocop flags it.

Expected behavior
To not flag reject on a hash in the situation described above.
Actual behavior
Rubocop indicates that compact should be used, even though it returns a different result than reject.
RuboCop version
$ bundle exec rubocop -V
1.9.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.4.1, running on ruby 2.7.2 x86_64-linux)
  - rubocop-performance 1.9.2
  - rubocop-rails 2.9.1

This doesn't seem to be valid:
{ key1: :foo, key2: false, key3: nil }.compact
#=> {:key1=>:foo, :key2=>false}

I've tested in ruby 2.6, 2.7 and 3.0.


Current reading of Alfen connected to 1 phase is wrong

Describe the bug
My Alfen Eve charger is setup and configured in 1 Phase mode. Unfortantly this causes an error in the UI.
Failed to parse web socket data: JSON.parse: unexpected character at line 1 column 32 of the JSON data [{"loadpoints.0.chargeCurrents":}]
Rootcause is that the charger is reporting nothing on P2 and P3:
[lp-1  ] DEBUG 2022/09/24 20:04:50 charge currents: [6.13 NaN NaN]A
Steps to reproduce
Connect an Alfen Eve charger that is connected to only 1 phase.
Open the UI.
Configuration details
levels:
  site: debug
  lp-1: debug

network:
  # schema is the HTTP schema
  # setting to `https` does not enable https, it only changes the way URLs are generated
  schema: http
  # host is the hostname or IP address
  # if the host name contains a `.local` suffix, the name will be announced on MDNS
  # docker: MDNS announcements don't work. host must be set to the docker host's name.
  host: evcc.local
  # port is the listening port for UI and api
  # evcc will listen on all available interfaces
  port: 7070

sponsortoken:

site:
  - title: Home # display name for UI
    meters:
      grid: my_grid # grid meter reference
      pvs: my_pv # first pv meter reference
      batteries: my_battery # battery meter reference
    residualPower: 120
    bufferSoC: 80
    prioritySoC: 50

loadpoints:
  - title: Garage # display name for UI
    charger: my_charger # Wallbox Referenz
    #vehicle: my_car # Referenz auf Standardfahrzeug
    mode: pv # charge mode (off, now, minpv, pv)
    maxcurrent: 32
    phases: 1

chargers:
  - name: my_charger
    type: template
    template: alfen

    # Modbus TCP
    modbus: tcpip
    id: 1
    host:  # Hostname
    port: 502 # Port

meters:
  - name: my_grid
    type: template
    template: solaredge-hybrid
    usage: grid
    host:  # IP-Adresse oder Hostname
    port: 1502 # Port 502 (SetApp) oder 1502 (LCD) # Optional
    timeout: 10s # Optional
  - name: my_pv
    type: template
    template: solaredge-hybrid
    usage: pv
    host:  # IP-Adresse oder Hostname
    port: 1502 # Port 502 (SetApp) oder 1502 (LCD) # Optional
    timeout: 10s # Optional
  - name: my_battery
    type: template
    template: solaredge-hybrid
    usage: battery
    host:  # IP-Adresse oder Hostname
    port: 1502 # Port 502 (SetApp) oder 1502 (LCD) # Optional
    timeout: 10s # Optional

Log details
Using config file:  /config/evcc.yaml
starting evcc: 'evcc --config  /config/evcc.yaml'
[main  ] INFO 2022/09/24 20:10:32 evcc 0.103
[main  ] INFO 2022/09/24 20:10:32 using config file: /config/evcc.yaml
[main  ] INFO 2022/09/24 20:10:32 listening at :7070
[lp-1  ] WARN 2022/09/24 20:10:34 locking phase config to 1p for switchable charger
[site  ] INFO 2022/09/24 20:10:34 site config:
[site  ] INFO 2022/09/24 20:10:34   meters:      grid ✓ pv ✓ battery ✓
[site  ] INFO 2022/09/24 20:10:34     grid:      power ✓ energy ✗ currents ✗
[site  ] INFO 2022/09/24 20:10:34     pv 1:      power ✓ energy ✗ currents ✗
[site  ] INFO 2022/09/24 20:10:34     battery 1: power ✓ energy ✗ currents ✗ soc ✓
[lp-1  ] INFO 2022/09/24 20:10:34 loadpoint 1:
[lp-1  ] INFO 2022/09/24 20:10:34   mode:        pv
[lp-1  ] INFO 2022/09/24 20:10:34   charger:     power ✓ energy ✓ currents ✓ phases ✓ wakeup ✗
[lp-1  ] INFO 2022/09/24 20:10:34   meters:      charge ✓
[lp-1  ] INFO 2022/09/24 20:10:34     charge:    power ✓ energy ✓ currents ✓
[lp-1  ] DEBUG 2022/09/24 20:10:34 phase timer inactive
[lp-1  ] DEBUG 2022/09/24 20:10:34 pv timer inactive
[lp-1  ] DEBUG 2022/09/24 20:10:34 max charge current: 6A
[site  ] DEBUG 2022/09/24 20:10:34 ----
[lp-1  ] DEBUG 2022/09/24 20:10:34 charge power: 1433W
[site  ] DEBUG 2022/09/24 20:10:34 pv power: 10W
[site  ] DEBUG 2022/09/24 20:10:34 battery power: 0W
[site  ] DEBUG 2022/09/24 20:10:34 grid power: 1649W
[site  ] DEBUG 2022/09/24 20:10:34 battery soc 0: 0%
[site  ] DEBUG 2022/09/24 20:10:34 site power: 1769W
[lp-1  ] DEBUG 2022/09/24 20:10:34 charge currents: [6.16 NaN NaN]A
[lp-1  ] DEBUG 2022/09/24 20:10:35 charger status: C
[lp-1  ] DEBUG 2022/09/24 20:10:35 charge start energy: 6.21e+03kWh
[lp-1  ] INFO 2022/09/24 20:10:35 start charging ->
[lp-1  ] DEBUG 2022/09/24 20:10:35 charger disable
[lp-1  ] DEBUG 2022/09/24 20:10:35 wake-up timer: stop
[lp-1  ] DEBUG 2022/09/24 20:10:35 phase timer inactive
[lp-1  ] DEBUG 2022/09/24 20:10:35 pv timer elapse
[lp-1  ] DEBUG 2022/09/24 20:10:35 pv timer inactive
[lp-1  ] DEBUG 2022/09/24 20:10:35 switched phases: 1p
[site  ] DEBUG 2022/09/24 20:10:44 ----

What type of operating system are you running?
Docker container
Version
0.103

Loadpoint should convert all NaNs to zero.

I would expect the same but with a Modbus Poller connected I get following information on the registres containing the currents (320, 322, 324). No data is available on the registers matching P2 and P3.

Its the same for the Phase voltages (306,308,310):

Please
evcc charger —log trace

[main  ] INFO 2022/09/25 10:45:48 evcc 0.103
[main  ] INFO 2022/09/25 10:45:48 using config file: /config/evcc.yaml
Power:          6W
Energy:         6223.1kWh
Current L1..L3: 0.04A NaNA NaNA
Charge status:  B
Enabled:        false

Log trace!

I understand, but setting log level to trace in config shows only an output up-to debug. Connection to sh on docker shows this output. I'm running evcc as Home Assistant Add-on.

  However I have the same setup as above and receiving the same error. Do I need to change a file somewhere with the changes as provided in the fix via the HA file editor?

Please open new issue- this has been resolved months ago


[DX-1813] Upgrade node matrix for Node 18

https://immutable.atlassian.net/browse/DX-1813
Summary
Allowing for Node 18 when consuming and also for testing workflows.
Why the changes
Customers want it! https://immutable.atlassian.net/browse/DX-1813
Things worth calling out
Expecting build / tests pass on Node 18
Before merging

[ ] For Immutable developers: Do the changes in this PR require documentation updates? Please refer to this SDK documentation guide and list links to documentation update PRs (they don't have to be merged) below (or write N/A):

[ ] Add documentation update 1
[ ] Add documentation update 2

Since this repo has a dependency of itself, the test suite using Node 18 won't pass as the existing package doesn't allow for Node 18.
Therefore I'm going to remove this CI test, and once we re-publish the package, we can add this back. To verify this upgrade won't break anything, we can locally run the tests and build/test with yalc.