Update 07-cluster-scaling.md

Cluster autoscaler required the cluster to be created with VMSS. Standalone cluster cannot be enabled for autoscaling.

seems like installing the az extension az extension add --name aks-preview changes the default behavior.. having the extension installed the cluster will be created with vmss support and you'll not run into this problem.. do you think we should add az extension add --name aks-preview to our initial build intructions? i'd like to keep the autoscaler if possible..

I tried that but it still threw the error.

From: heoelri<EMAIL_ADDRESS>Sent: Wednesday, February 27, 2019 6:19 PM
To: heoelri/container-bootcamp
Cc: sujitdmello; Assign
Subject: Re: [heoelri/container-bootcamp] Update 07-cluster-scaling.md (#36)
Assigned #36https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fheoelri%2Fcontainer-bootcamp%2Fpull%2F36&data=02|01||73d8edb415d54da75da108d69c9d13c0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636868595778822080&sdata=w732j7gH9ntAvZU2UE%2BU9Pm2ZZnQAu2WQIm%2FGb9DXrc%3D&reserved=0 to @sujitdmellohttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsujitdmello&data=02|01||73d8edb415d54da75da108d69c9d13c0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636868595778832085&sdata=LIHXMDJR6tEgrF8DUuo28f6h%2Bamt5xZ2QpKzXzJSLrA%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fheoelri%2Fcontainer-bootcamp%2Fpull%2F36%23event-2167519784&data=02|01||73d8edb415d54da75da108d69c9d13c0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636868595778842096&sdata=6EIsOCGRVlEooPvWBkqsyKBl9OkUs6VCTGHC2GDtA64%3D&reserved=0, or mute the threadhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEIoklgff28uMIYEkOyvo9fkfQb0TpJxks5vRlu4gaJpZM4bRvuS&data=02|01||73d8edb415d54da75da108d69c9d13c0|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636868595778852114&sdata=%2BjW4Pqqmrnb2VVW2x1CDoLo1FhhjlnuHDvhJ7eALbfA%3D&reserved=0.

ok @sujitdmello what about adding --enable-vmss directly at the beginning while creating the cluster?

I will test it to make sure it does not impact the az aks scale command portion of the lab.

Operation failed with status: 'Bad Request'. Details: The AvailabilityProfile type of agentpool 'nodepool1' is set to VirtualMachineScaleSets, but the Kubernetes version is 1.11.6 which doesn't support VMSS.

One suggestion:
I think we can have this as a separate exercise on day 3/4 where candidates can spin up a new cluster on VMSS to test this.
From: sujitdmello<EMAIL_ADDRESS>Sent: Thursday, February 28, 2019 3:39 PM
To: heoelri/container-bootcamp<EMAIL_ADDRESS>Cc: Deepu S S<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [heoelri/container-bootcamp] Update 07-cluster-scaling.md (#36)
Operation failed with status: 'Bad Request'. Details: The AvailabilityProfile type of agentpool 'nodepool1' is set to VirtualMachineScaleSets, but the Kubernetes version is 1.11.6 which doesn't support VMSS.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fheoelri%2Fcontainer-bootcamp%2Fpull%2F36%23issuecomment-468169833&data=02|01|Deepu.SS%40microsoft.com|2220735da7154ea8874d08d69d4fcf06|72f988bf86f141af91ab2d7cd011db47|1|0|636869363438567673&sdata=cFgV3YEgdK%2F%2FdxhC6jdukZtkeOqKzgfICDwCOMvgxFI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAmexqodr59DPX6MH3I563-rwL_KVL8SXks5vR4eVgaJpZM4bRvuS&data=02|01|Deepu.SS%40microsoft.com|2220735da7154ea8874d08d69d4fcf06|72f988bf86f141af91ab2d7cd011db47|1|0|636869363438567673&sdata=oCYwY3b4HLuSv9yJVEr4LSXgz25HsqopsG0aDI32rfQ%3D&reserved=0.

@desreela yeah.. I agree.. I think that's the easiest solution for now

az extension add --name aks-preview
az feature register --name VMSSPreview --namespace Microsoft.ContainerService
az feature list -o table --query "[?contains(name, 'Microsoft.ContainerService/VMSSPreview')].{Name:name,State:properties.state}"
Wait for status to change to 'registered'
az provider register --namespace Microsoft.ContainerService
// Now create AKS cluster
az aks create --name $CLUSTER_NAME --resource-group $NAME --node-count 2
--kubernetes-version 1.12.4 --generate-ssh-keys --location $LOCATION --enable-vmss
az aks scale --name $CLUSTER_NAME --resource-group $NAME --node-count 1 works.
az aks upgrade --resource-group containers2 -n k8s2 --kubernetes-version 1.12.5 works
az aks update --resource-group $NAME --name $CLUSTER_NAME --enable-cluster-autoscaler --min-count 1 --max-count 3 works

thanks @sujitdmello should we change it that way (it's pretty complicated and hard to understand on day 2) or should we move the autoscaler part (temporarly into the advanced ones)? i think it will become default very soon..
@sujitdmello @desreela @blueflightx7

Yeah let’s keep it simple. We can move the autoscaling to its own lab.

From: heoelri<EMAIL_ADDRESS>Sent: Thursday, February 28, 2019 6:44 PM
To: heoelri/container-bootcamp
Cc: sujitdmello; Mention
Subject: Re: [heoelri/container-bootcamp] Update 07-cluster-scaling.md (#36)
thanks @sujitdmellohttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsujitdmello&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823571198&sdata=vr14fw%2FsS2MTC26iz4Ja0sI37Szi%2FnIAURVnb5e8VMc%3D&reserved=0 should we change it that way (it's pretty complicated and hard to understand on day 2) or should we move the autoscaler part (temporarly into the advanced ones)? i think it will become default very soon..
@sujitdmellohttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsujitdmello&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823581209&sdata=xLe73OfwVr58futOcluDTsebeRazFAQdrK9JFvO2VEg%3D&reserved=0 @desreelahttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdesreela&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823591220&sdata=jt6Ah%2BpgwQw0%2B8kWBAWF0MbPTqVJgsQw9Ei5tZ3stcc%3D&reserved=0 @blueflightx7https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fblueflightx7&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823601225&sdata=sugwvvXH1xJR71yoeqxHqcJdnlt9C%2FSyu4XmqprJULA%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fheoelri%2Fcontainer-bootcamp%2Fpull%2F36%23issuecomment-468225867&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823611236&sdata=%2FYeh2lLyXZ7iy3rUL5E7kt%2Bu1AWVPePOIt1IzUJlNSw%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAEIokkFnf8Yb_xS6I-V_uEYliL3aeHwsks5vR7MYgaJpZM4bRvuS&data=02|01||a3698e7cb0b04876bdcd08d69d69bebe|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636869474823621241&sdata=C7uLA1YWMu2aGDp02olx3HhyyeCov192UvxCSgDUP9w%3D&reserved=0.

fixed by removing the autoscaler part temporarily @sujitdmello FYI


🚀 [Feature Request] - Adding hover effect to Our Services section images

Is your feature request related to a problem ? Please describe. ✍
Enhancement: Adding a hover effect to the images in the "Our Services" section of the website. This enhancement aims to improve user-friendliness and provide a more interactive experience for visitors.
Describe briefly how you would implement this enhancement.
Some steps to implement this enhancement-
Step 1: Identify the CSS class or selector for the images in the "Our Services" section.
Step 2: Create a new CSS rule or modify an existing one to define the desired hover effect. This can include changes in opacity, color, or any other visual effect that enhances the user experience.
Step 3: Apply the CSS rule to the identified images or their parent container to activate the hover effect when the cursor is over the images.

Additional context or screenshots
screenshots:

Please assign this issue to me so that I can contribute to this amazing project...
Are you contributing for a program?
GSSoC2023
Code of Conduct

[X] I have read and agree to the Contributing Guidelines of this project.

Can't u check that this issue is alreadt raised here #303


NTR: regulation terms for GO:0032782 bile acid secretion

Hello,
Can I please request all of the regulation terms for GO:0032782?
For annotation of PMID:22767443
Please add GO Refs:
GOC:rph
GOC:BHF
GOC:BHF_miRNA
thanks,
Rachael.

Hi @rachhuntley
Again this is 'acts upstream of'; can you not use this in P2GO now ?
Thanks, Pascale

So are we supposed to be using acts upstream of (or it's +/- children) instead of regulation now?
The documentation for acts upstream of says "The acts upstream of gene product to GO term relation is used when the experimental evidence is sufficient to determine the mechanism relating a gene product's activity (MF) to a biological process and that activity occurs before that biological process, is not an integral part of the process, nor does it regulate the process. "
The authors of this paper say:
"Here we test the proposal that miR-33 also modulates hepatic bile metabolism by decreasing the expression of specific sterol transporters in the canalicular membrane of hepatocytes.",
which sounds like regulation by the miRNA to me.
I often create interaction networks for miRNAs and their targets and then overlay GO terms, specifically separating the regulation of a MF or BP from involvement in the process itself. If I start using this qualifier then when I make the networks using Cytoscape/BinGO, the regulation will be lost and it will appear as if the miRNA is part of the process, since these tools don't take account of these qualifiers.
Maybe I missed where this was discussed, but I'm not sure of the goal here.

@pgaudet - Are you objecting to these regulation terms being made?

Hi @krchristie
To be consistent with current practices we can create them. We can revisit when we finalize the guidelines for the GP2term relations.
Thanks, Pascale


Bump up versions for staging eks-a release

Issue #, if available:
Description of changes:
Bump up versions for staging eks-a release
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: mitalipaygude
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [mitalipaygude]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Non-production deployments causing task to fail

Vercel deployments are succeeding to non-prod (production: false), but Azure is reporting task failure:
To deploy to production (mywebsite.com), run `vercel --prod`
##[error]Cannot read properties of undefined (reading 'length')
Finishing: verceldeploymenttask

Only difference between the below YAML examples is the production input.
Successful YAML (deployment and Azure task succeed):
trigger:
  branches:
    include:
      - main
  paths:
    include:
      - my-project

pool:
  vmImage: ubuntu-latest

steps:
- task: vercel-deployment-task@1
  inputs:
    vercelProjectId: 'my-proj-id-here'
    vercelOrgId: 'my-org-id-here'
    vercelToken: '$(VERCEL_TOKEN)'
    vercelCWD: 'my-project'
    production: true
    debug: true

Failing YAML (deployment succeeds, azure task reports failure):
trigger:
  branches:
    include:
      - main
  paths:
    include:
      - my-project

pool:
  vmImage: ubuntu-latest

steps:
- task: vercel-deployment-task@1
  inputs:
    vercelProjectId: 'my-proj-id-here'
    vercelOrgId: 'my-org-id-here'
    vercelToken: '$(VERCEL_TOKEN)'
    vercelCWD: 'my-project'
    production: false
    debug: true

Both of these will successfully deploy to Vercel (one to prod, another to a preview), but the deployment task is being reported as failing in Azure in the latter.

It's happening to us too,  since we configured the pipeline, a month ago
we tried but failed to find a workaround
It isn't very pleasant because our CI/CD notification reports the successful pipeline as a failed (with red)
Fingers crossed a fix is easy to implement

I'm getting a similar issue--the deploy job runs until it times out, but the Vercel preview build was fully deployed.

I encountered a similar issue where the deployment was successful on Vercel, but it failed on Azure. Changing the version as mentioned here resolved the problem for me for now.
https://github.com/vercel/vercel-azure-devops-extension/issues/32#issue-2482471330

Hey all, apologies for the error, the latest version 1.6.0 should be working for you now and I'm working on adding a preview version of the extension for more thorough testing in the future!

This error still appears to be happening on 1.6, just tested now.  The deployment ends with the error:
To deploy to production (home.withjoy.ai), run `vercel --prod`
##[error]Cannot read properties of undefined (reading 'length')
Finishing: verceldeploymenttas

This still appears to be happening on 1.6. Just tested now, and the error message is present with the task failing:
To deploy to production (my.website.domain), run `vercel --prod`
##[error]Cannot read properties of undefined (reading 'length')
Finishing: verceldeploymenttask

Ah okay, sorry about that @general-adhoc, I assumed it was related to an issue we had to rollback for 1.6.0. I'll reopen this issue

I encountered a similar issue where the deployment was successful on Vercel, but it failed on Azure. Changing the version as mentioned here resolved the problem for me for now. #32 (comment)

@Paulshaju thank you for the workaround here 🙏
@codybrouwers  Would you happen to know why an Azure DevOps pipeline would not be recognizing a specific task version?
When I attempt to pin my task at 1.3.0:
- task<EMAIL_ADDRESS>
I receive an error:
A task is missing. The pipeline references a task called 'vercel-deployment-task'. This usually indicates the task isn't installed...

I believe the reason might be that the Vercel Deployment Extension I have installed from the marketplace is set to 1.6.3?

Thank you for your help (I'm new to Azure...) and for maintaining this extension.

Is there a bypass for the users who use this in the GUI release pipelines and can't change the version as they please?
or is there any solution in sight for a new version with this issue fixed?
thx

I think I've figured out the root cause!
TLDR: As a quick fix make sure to use your Team ID (starting with team_) instead of your Personal Vercel ID for the vercelOrgId input.
You can find your Team ID by navigating to Account Overview and opening the submenu for the team associated with your application, and selecting Manage.

Scroll down and you will find your Team ID:

To the actual issue:
If run with a Personal Vercel ID as vercelOrgId the extension tries to retrieve the stagingPrefix from the Vercel REST API endpoint /v2/user for the creation of your preview deployment's alias URL. The response varies depending on the token's scope:

If the scope is set to a specific team the response will contain the object labelled AuthUserLimited in the docs and doesn't contain a prop stagingPrefix, thus the pipeline crashes in vercel-deployment-task-source/src/index.ts:275 with the exception Cannot read properties of undefined (reading 'length') – see this run.
If the scope is set to Full Account the response will contain the object labelled AuthUser and actually contain the stagingPrefix, but the creation of the alias will fail because the pipeline provides the vercelOrgId input as scope to the Vercel CLI as well. This is not supported – see failed run here.

With a Full Account scope token you can retrieve a defaultTeamId from the /v2/user endpoint and and some additions to the plugin we could possibly make it work, but this seems unsafe in case the account is linked to multiple teams. To add some speculation from my side: in the past Vercel didn't assign teams to single user/personal accounts. Therefore there might have been no alternative to using the Personal Vercel ID. This seems to be changed with the 'Northstar' migration and now all projects should be tied to a Team ID – I can't find any details on this but found multiple references in API schemas.
If this assumption is correct and all accounts are migrated (@codybrouwers – maybe you can confirm?) I would propose to remove support for the Personal Vercel ID altogether in favor of only allowing usage of a Team ID. I'll prepare a PR adjusting the docs to match the quick fix I've described in the beginning and prepare a second PR to remove the functionality tied to the Personal Vercel ID.
As this is a breaking change (at least config-wise) I would consider to publish this version as a V2, even though production deployments should still work just fine.

Addition: This is issue not related to #32 and thus can't be avoided pinning your version to an older version!

Adjusting Docs: #42


[gym] Only one .app artefact is copied over

Hi!
I have a Mac app which contains another .app as a dependency & build artefact (it's a helper app to allow the app to boot on startup, documented here.
The BuildProductsPath/Release directory listing looks like:
Foo.app
Foo.app.dSYM
Foo.swiftmodule
FooHelper.app
FooHelper.app.dSYM
FooHelper.swiftmodule
... // other frameworks

So far, so good...
What happens
Currently, copy_mac_app() only copies the last found .app in the folder (in this case it's FooHelper.app) during a build.
My CLI output is:
[12:29:02]: Successfully exported and compressed dSYM file
[12:29:02]: Successfully exported the .app file:
[12:29:02]: ./FooHelper.app

What should happen
I'd expect Foo.app to be available at the end of the build. It's .dSYM is, so that's a start :)
Solutions/discussion
There are a couple of solutions I can think of to this:

Copy all .apps. This is probably not good (since you probably never need only the *Helper.app on its own), but it's easiest to implement
Specify an env var to state which .app to copy
Look into the chosen scheme's Info.plist to find the CFBundleExecutable, and copy only that (defaulting to the last .app if not found?)

I'd say the last one is the best way to do this. I'll take a look at how easy that is, but would appreciate input!

fastlane env output: https://gist.github.com/notjosh/e68a1afff659e798e1d79d078faf81f0

@notjosh i did a few PR's to improve gym

https://github.com/fastlane/fastlane/pull/7090
https://github.com/fastlane/fastlane/pull/7091

this include a fix for this issue, using the last option you described (fetching EXECUTABLE_NAME)

Whoa nice! Any idea when that'll make it to a release?

@notjosh hopefully soon - you may +1 the PR's - and probably you can try it out?


Automatic update of Newtonsoft.Json to 12.0.2

NuKeeper has generated a patch update of Newtonsoft.Json to 12.0.2 from 12.0.1
Newtonsoft.Json 12.0.2 was published at 2019-04-22T01:21:49Z, 23 days ago
1 project update:
Updated NuKeeper\NuKeeper.csproj to Newtonsoft.Json 12.0.2 from 12.0.1
Newtonsoft.Json 12.0.2 on NuGet.org
This is an automated update. Merge only if it passes tests
NuKeeper: https://github.com/NuKeeperDotNet/NuKeeper

LGTM


input box of select component

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When I add an input field to Select component, I cannot edit the input box as it closes the menu immediately when the input box is focused.
Expected Behavior 🤔
It should be able to edit the input field.

Steps to Reproduce 🕹

const handleFooterClicked = (e) => {
        e.preventDefault()
        e.stopPropagation()
}
...
<Select value={value} onChang={handleChange}>
          <MenuItem value={10}>Ten1</MenuItem>
          <MenuItem value={9}>Ten2</MenuItem>
          <MenuItem value={8}>Ten3</MenuItem>
          <div onClick={handleFooterClicked}>
                <input />
          </div>
</Select>

Steps:

Click the input box and it closes the select menu

I guess this is impossible for now, right?

Why do you want to have an input in a Select? If you want users to be able to select one of pre-defined values, or a custom value, you can use the Autocomplete compontent - see the Creatable demo.

FWIW, you can make it work by moving the event handler to the input elment - see demo.

@michal-perlakowski Thank you very much. It worked! ❤ ❤ ❤


Send Email to Host Admins when Wise Balance is too Low

This was discussed in today's ops meeting where host admins has to manually go to Wise account and check for remaining balance or keep checking the host dashboard to notice that the Wise balance needs to be topped up.

It would be nice if we could send a reminder email to host admins when the Wise balance is low.
Suggestion: A weekly cron job that checks the transferwise balance for each connected account and sends an email to the host admins if the balance is less than a threshold.

It would be extra nice if the system calculated the threshold based on average weekly throughput

Could we close this with the new statuses showing on the expense page? @BenJam


revel/harness: app timed  out

←[32mINFO ←[0m 22:07:21    app     run.go:26: Running revel server
Revel proxy is listening, point your browser to : 9000
ERROR 22:08:20 harness.go:232: Killing revel server process did not respond after wait timeout. processid=15704
ERROR 22:08:21 watcher.go:270: Could not start application              error="revel/harness: app timed
out"

This error appears when I run through revel run. At the same time, I am working on laptop battery.
If I run a project with power from 220w, then the project starts.
If I turn off watch mode. That project is launched from both the battery and 200w.
Platform: Windows 10 v2004
go version go1.14.1 windows/amd64
[[override]]
name = "github.com/revel/modules"
version = "0.21.0"
[[constraint]]
name = "github.com/revel/revel"
version = "0.21.0"
Where to find the reason for this behavior?

did it manage to bind to that port?

same issue, any advice? @alexyalinc


Are there plans to manage entries?

Hello there, thanks for the work. Are there plans to extend the package for manage the entries of the CHANGELOG.md? Something like https://pypi.org/project/changelog-cli/, for instance?
Namely, these tasks:

Add a new entry in the unreleased section
Figure out the next version based on the entries (assuming they conform to something like https://www.conventionalcommits.org/en/v1.0.0/ or https://gitmoji.dev)
Given a version, change the unreleased section to that version and create a new empty unreleased section (and possibly change Project.toml as well)

If you are doing these tasks with some tool already, please let me know.

#11 could be a step towards

Figure out the next version based on the entries (assuming they conform to something like https://www.conventionalcommits.org/en/v1.0.0/ or https://gitmoji.dev/)

since it would allow parsing the changelog to know which entries already exist.


[Feature] tooltip's trigger support series for Pie chart

What problem does this feature solve?

echarts: 5.5.1

I found the pie chart only support item tooltip.
But if i got a nest pie chart, and i wanted to show then series data when hover the item(highlight current one), it was difficult to do.
I could use my chartData instead of get params from formatter. But it gets some problems when hidden by legend.
Example
What does the proposed API look like?
tooltips:{
    trigger: 'series', // ONLY support for Pie
    formatter:(params: Array) => {
        // do something
        return '';
    }
}

What do you expect by setting tooltip.trigger to 'series'?

What do you expect by setting tooltip.trigger to 'series'?

I can achieve something similar to the Example.
There is one issue with this example is when i hide the data by click legend, it still remains visible on the tooltip.

Items with the same name are grouped and controlled together in legend. So you may set the same data.name for the items you wish to group together and data.itemStyle.color to set different colors.

Sorry, I couldn't get your point very much.
As for me, If without 'series' for nested pie chart, I only use a hack way to get the series data and format it to tooltip, so that I could render my series data on tooltip instead of just one item's data.
If echarts could support 'series', then I could get the series data in a formal way and by the way I could fix the issue which I mentioned before.

here is one issue with this example is when i hide the data by click legend, it still remains visible on the tooltip.


Retrieve resource templates from Sinopia platform

Includes:

Add configuration for disabling/enabling spoofing

Set default value to enable spoofing and explicitly set this value in the editor service in docker-compose to retain current behavior

...

connects to #309

Coverage increased (+0.1%) to 77.36% when pulling ce1d330b33e56e460dd03327c66c0c04e063cc7d on integrate_with_platform_309 into ca0d7c7bb7630812a111aeee4112bc3eda2494a0 on master.

@mjgiarlo LGTM!

@mjgiarlo, @jermnelson added "security/detect-object-injection": "off" to quiet the Generic Object Injection Sink  security/detect-object-injection warnings during the build. You may want to rebase off master now, or wait until PR #453 gets merged and then rebase, which will also clean up some of the unreferenced and duplicative methods.

@jermnelson @jgreben Given that #453 closes #309, what in this PR is still worth keeping around?

@mjgiarlo @jermnelson: Seems like all the env stuff having to do with REACT_APP_* would be good additions that I did not cover.
I think we should only really ditch the changes to ResourceTemplateForm and reducers/index.js here.
The changes I made to sinopiaServer.js use the client api instead of superagent, which I think is the way we originally wanted to go, but either way, I think the result is the same. Your tie-in to the Config is a good addition that I did not cover.
LMK how you want to go forward with rebasing/merging/etc..

@jgreben super cool, thank you! I'm inclined to wait for #453 to be merged, and then rebase and polish this off.


Ensure admin information is used

If there is no primary geo that is lat/lng but there is admin 0, 1, 2 info etc this must be set in the Causemosified output.

Resolved with #13


BUG: GrapesJS Studio: Importing code breaks css of other pages

GrapesJS version

[X] I confirm to use the latest version of GrapesJS

What browser are you using?
Firefox 126.0.1 (64-Bit)
Reproducible demo link
https://studio.grapesjs.com/
Describe the bug
(I don't know if this is the correct place to report bugs for GrapesJS Studio, I know it is still in Alpha, but I still wanted to report this. It destoyed some hours of work and I don't want anyone else suffer from this)
Example Code (simple site built and exported with GrapesJS Studio): HTMLCodeWithStyling.txt
How to reproduce the bug?

Create new Page
Import attached example code with css
see styling of first page
Create new Page
Import code again
Check first page, notice the differnce

What is the expected behavior?
Styling imports correctly and does not break other pages.
What is the current behavior?
Styling does not import correctly (background color missing?) and messes with styling of other pages.
Thank you for staying open-source!
Code of Conduct

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

The latest Studio (new location: https://app.grapesjs.com/studio) should contain the related fix


Ping trace throws exception

If source and destination IP addresses provided from 2 different networks, throws exception.

Fixed via #31


update globalhub namespace

Related Issue:  stolostron/backlog#<ISSUE_NUMBER>
https://issues.redhat.com/browse/ACM-9153
Description of Changes:
The globalhub namespace in incorrect
What resource is being added:  | N/A
Is this a Hub or Managed cluster change?:
Hub Cluster | Managed Cluster | N/A
Notes:

FYI @clyang82

/lgtm
/approved

/assign @dislbenn

/lgtm


feat(host): skip calculating checksum for cached image if need

What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

release/3.11
/area host

/cc @swordqiu @ioito


Fix breaking change in Filament 3.0.9

Deleted file breaking testbench from composer update
https://github.com/filamentphp/filament/commit/67c053a89e88853adc472b504e479b30bbfe76c2

Not needed anymore due to fix applied in Filament core
https://github.com/filamentphp/filament/commit/dfe563d5dc21701a6b0a184154d5e9c657df15b9


SqlNotificationInfo.None enum value missing, causing unnecessary exceptions.

Describe the bug
In SqlDependencyListener, when an incoming XML message is parsed, sometimes these messages look like this:
<qn:QueryNotification xmlns:qn="http://schemas.microsoft.com/SQL/Notifications/QueryNotification" id="4579" type="change" source="timeout" info="none" database_id="9" sid="0x01050000000000051500000002AAFCB5FFBA57356D2C83B7E9030000"><qn:Message>52cdaaaf-f5e5-4342-8c10-dfec05cf4950;ec06ef71-ec94-4e56-a11e-c1905ebf4cf9</qn:Message></qn:QueryNotification>

The info attribute is parsed at https://github.com/dotnet/SqlClient/blob/main/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlDependencyListener.cs#L1133-L1167. However, the enum that the value is parsed into does not included a value "None", causing an exception to be thrown. It is ultimately handled by the containing try/catch, but there are a few issues with this:

Exception flow and handling incurs a performance cost.
The exception is System.ArgumentException and is thrown from mscorlib, which means that suppressing it in Visual Studio's Exception Settings causes all such exceptions to be supressed, making debugging more difficult when there are other AgumentExceptions thrown by mscorlib that I do want to break on.

To reproduce
This is a bit weird to reproduce. The first few notifications where will be of Info == Error. But sometimes, it will be "none" in the XML, which then throws and gets turned into Unknown.
Set exception settings in Visual Studio to break on all CLR exceptions.
// See https://aka.ms/new-console-template for more information
using Microsoft.Data.SqlClient;

var connectionString = "Server=localhost;Database=BugReproDb;Trusted_Connection=True;Command Timeout=5";
SqlDependency.Start(connectionString);
var connection = new SqlConnection(connectionString);
connection.Open();
var cmd1 = connection.CreateCommand();
cmd1.CommandText = "IF OBJECT_ID('TestTable') IS NULL CREATE TABLE TestTable (c1 int);";
cmd1.ExecuteNonQuery();
cmd1.Dispose();

Console.WriteLine("Usually the notification here will be 'Change Timeout Error'.");
Console.WriteLine("But sometimes, it will be 'Change Timeout Unknown'. This is the error.");
while (true)
{
    var tcs = new TaskCompletionSource();
    var cmd = connection.CreateCommand();
    cmd.CommandText = "SELECT c1 FROM dbo.TestTable";

    var dependency = new SqlDependency(cmd, null, (int)1);
    dependency.OnChange += (o, e) => {
        Console.WriteLine($"{e.Type} {e.Source} {e.Info}");
        if (e.Info == SqlNotificationInfo.Unknown)
        {
            Environment.Exit(0);
        }
        tcs.SetResult();
    };
    cmd.ExecuteNonQuery();
    await tcs.Task;
    cmd.Dispose();
}

Expected behavior
SqlNotificationInfo.None is defined so exception flow isn't needed.
Further technical details
Microsoft.Data.SqlClient version: 3.0.1
.NET target: net6.0
SQL Server version: 15.0.2000.5
Operating system: Win 10

Interesting. According to the schema that it claims that it implements the value none is invalid, http://schemas.microsoft.com/SQL/Notifications/QueryNotification/QueryNotification.xsd either the value must be missing or one of the values specified in the restriction which doesn't include none.
We could handle the none case in this library probably by using TryParse and then checking for the literal "none" value but the SQL Server either shouldn't be sending that value of should be using a schema that allows that value and those changes would need to be made at the server side.

@Wraith2 where does it state that it implements that schema? There are already some values in the enum not present in the schema.

<qn:QueryNotification xmlns:qn="http://schemas.microsoft.com/SQL/Notifications/QueryNotification" id="4579" type="change" source="timeout" info="none" database_id="9" sid="0x01050000000000051500000002AAFCB5FFBA57356D2C83B7E9030000">qn:Message52cdaaaf-f5e5-4342-8c10-dfec05cf4950;ec06ef71-ec94-4e56-a11e-c1905ebf4cf9</qn:Message></qn:QueryNotification>

Right there in the xml the user provided. It uses xmlns to declare a short namespace prefix qn and identifies that it implements the schema at the url. That url exists and has a short and easy to read (a rarity) xsd schema.

Oops, guess I'm blind. If none is not a supported value, should we be adding it to the enum? This could be by design and we can just improve it by not throwing that exception but ultimately keeping it as unknown.

I think we have to add handling of it somehow but I don't know whether we want to add it to the enum. If we're going to change the enum (which is a breaking change then it would probably be a good idea to ask the Sql Server team what possible values it can take from their side and make sure everything is covered. It's clearly no longer limited to the originally defined values.

Server team has confirmed the server shouldn't be sending that value and this is likely a server-side bug.

Excellent news. I think it is still best to handle it in this library without throwing since we can't be certain users will ever upgrade or even be on a supported version. What do you think?

Yeah, I like the idea of at least changing this to a TryParse rather than a Parse with try/catch.

I agree, I'll make a PR for it.

@ascott18 I'm unable to reproduce it on 15.0.2080.9. I ran it for several minutes, how long should it take?

@johnnypham When I originally made the repro it consistently happened on the third notification. I just ran it again and got it on the 4th.
But sometimes when I'm developing the actual codebase where this is used, I can go 30 minutes or more until it happens and VS breaks on that exception.

Had a hunch and did a little more checking. As I thought, the timing of the "none" comes from when SQL Server started up.
PS> Stop-Service MSSQLSERVER; Get-Date; Start-Service MSSQLSERVER; Get-Date; while ($true) {dotnet run}
WARNING: Waiting for service 'SQL Server (MSSQLSERVER) (MSSQLSERVER)' to stop...
WARNING: Waiting for service 'SQL Server (MSSQLSERVER) (MSSQLSERVER)' to stop...
WARNING: Waiting for service 'SQL Server (MSSQLSERVER) (MSSQLSERVER)' to stop...
WARNING: Waiting for service 'SQL Server (MSSQLSERVER) (MSSQLSERVER)' to stop...

Wednesday, November 3, 2021 5:16:27 PM
Wednesday, November 3, 2021 5:16:28 PM
Repro starting: 11/3/2021 5:16:29 PM
Usually the notification here will be 'Change Timeout Error'.
But sometimes, it will be 'Change Timeout Unknown'. This is the error.
11/3/2021 5:16:44 PM Change Timeout Error
11/3/2021 5:16:59 PM Change Timeout Error
11/3/2021 5:17:14 PM Change Timeout Error
11/3/2021 5:17:28 PM Change Timeout Unknown
Repro starting: 11/3/2021 5:17:29 PM
Usually the notification here will be 'Change Timeout Error'.
But sometimes, it will be 'Change Timeout Unknown'. This is the error.
11/3/2021 5:17:44 PM Change Timeout Error
11/3/2021 5:17:59 PM Change Timeout Error
11/3/2021 5:18:14 PM Change Timeout Error
11/3/2021 5:18:28 PM Change Timeout Unknown

Hmm still unable to repro. I'll try to find someone who can. If I can't, would you mind validating the changes?

Sure!

@ascott18 you can download the nuget here and try it out: https://dev.azure.com/sqlclientdrivers-ci/sqlclient/_build/results?buildId=43533&view=artifacts&pathAsName=false&type=publishedArtifacts

Thanks, it works (added a review to the PR).
On an unrelated note, I had to add "TrustServerCertificate=True" to my connection string (connecting to localhost). I'm guessing the defaults around cert trusting are being changed in 4.0?

Yes they are.

Closing as #1378 has been merged.


ModuleNotFoundError: No module named 'PIL'

I generated step 1
step1:
python3 step1_layers_to_spritesheet/build.py
and got this error code
ModuleNotFoundError: No module named 'PIL'

Did you run make first_time_setup? You may need to install python dependencies. Also if you're on windows you might have to replace python3 with python

I'm using a windows PC , ohh I have been using python3 but it has been working, I tried to replace the python3 to py but it still gave this same error message

Closing as resolved let me know if you have any issues in Discord


Some Material Icons previews are broken

Description
When scrolling through the icons in the Visual Studio Code or Android Studio, it is easier to check the preview in the IDE, and looks like some icons have broken previews.
https://user-images.githubusercontent.com/48603081/161268603-07c6cd75-9fc2-4eb1-90bc-ae0081554dd4.mov

ex: manage_history
Visual Studio Code

Android Studio

flutter doctor -v
[✓] Flutter (Channel master, 2.13.0-0.0.pre.357, on macOS 12.3 21E230 darwin-arm, locale
    en-GB)
    • Flutter version 2.13.0-0.0.pre.357 at /Users/tahatesser/Code/flutter_nevercode
    • Upstream repository git@github.com:NevercodeHQ/flutter.git
    • Framework revision 7971b412c1 (5 hours ago), 2022-04-01 05:31:08 -0400
    • Engine revision c3274494b0
    • Dart version 2.17.0 (build 2.17.0-261.0.dev)
    • DevTools version 2.12.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc2)
    • Android SDK at /Users/tahatesser/Code/android-sdk
    • Platform android-32, build-tools 33.0.0-rc2
    • ANDROID_SDK_ROOT = /Users/tahatesser/Code/android-sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] VS Code (version 1.66.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.36.0

[✓] Connected device (3 available)
    • iPhone 13 Pro Max (mobile) • 3EB061AB-0B1E-41E3-8445-2A179E0402D0 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-15-4 (simulator)
    • macOS (desktop)            • macos                                • darwin-arm64   •
      macOS 12.3 21E230 darwin-arm
    • Chrome (web)               • chrome                               • web-javascript •
      Google Chrome 100.0.4896.60

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

cc: @guidezpl

Hi @TahaTesser, Thanks for filing the issue. I am able to reproduce the issue on the latest master channel. Some Icons do not  load the Icon preview such as

Icons.manage
Icons.manage_history
Icons.manage_history_outlined
Icons.manage_history_sharp

flutter doctor -v
[✓] Flutter (Channel master, 2.13.0-0.0.pre.357, on macOS 12.3 21E230 darwin-arm, locale en-IN)
    • Flutter version 2.13.0-0.0.pre.357 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 7971b412c1 (5 hours ago), 2022-04-01 05:31:08 -0400
    • Engine revision c3274494b0
    • Dart version 2.17.0 (build 2.17.0-261.0.dev)
    • DevTools version 2.12.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-32, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.10.2 out of date (1.11.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 61.2.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.65.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.37.20220301

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 535317A4-921C-488F-B234-C2CC6D4AE5A1 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-5
      (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.3 21E230 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 100.0.4896.60

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

I would recommend filing this issue in https://github.com/Dart-Code/Dart-Code/issues and https://github.com/flutter/flutter-intellij/issues, they use different mechanisms for pulling in the new font and displaying inline icons.

duplicate of https://github.com/Dart-Code/Dart-Code/issues/3641


Handle of errors that occur during conversion

If an error is encountered mid conversion, the client continues to poll. Resolve this issue and report the failure to the failure listener

@daniel-warren looks good, can you merge and release


Take screenshots doesn't work

Currently if you run you get an error about input file being undefined. The problem is due to this refering to the global object instead of to the options object.
The fix is to use self instead of this, the author already set var self = this up and just forgot to use it; just a simple typo. The fix will be applied to Processor.js line 242.
Here is the change with some context
https://gist.github.com/1728346

Also while I was at it I added support for incrementalOutputFiles, so you can optionally have output files named like tn_1.jpg tn_2.jpg ... instead of tn_77.4s.jpg.
Incrementally naming files is a lot easier to work with so you can just iterate over them and upload to s3 without having to print out the files in the directory or something.
https://gist.github.com/1728409
Sorry for the gists I didn't feel like branching

fixed by merging #34


[Smart Contract] Smart contract verification

Usefull link : https://forum.openzeppelin.com/t/how-to-verify-a-contract-on-etherscan-bscscan-polygonscan/14225

Great @Le-Caignec, can you add the verify task for the upgrade script to verify the new implementation?

done ✅


Try to allow user to disable alerts

This PR is trying to allow user override to the env variables that controls the behavior of the workflow in check_plugin.yml.

codes lgtm. just wanna know how it works. how should we give the input to control it?

That is why I need it to be in master for testing.
Repository_dispatch events can send parameters through payloads, but only works in master branch, and I can only try to push as little PR as possible to playtest it.
If my initial test did not work, aka this PR, I will use a personal fork to try it out.


"Stay on" when continuing on the same road

Along the same lines as #30, it would be nice if we could say "Keep left to stay on Main St" if Main St. is the street both before and after the fork in the road.
/cc @freenerd @bsudekum @mayagao

Isn't this the purpose of the continue maneuver type? If not, what is the continue maneuver type for?
The OSRM docs say:
continue | Turn in direction of modifier to stay on the same road

I'm perplexed by the continue maneuver. Based on what the docs say about "staying on the same road", I'd expect both of these to be continue because they both involve staying on the same road:

Continue left onto South Airport Boulevard

Confusing because this feels like a left turn
Ideal instruction would be "Turn left to stay on South Airport Boulevard"

Keep right at the fork onto I 395

Confusing because this is staying on the same road (I'm already on I 395
Ideal instruction would be "Keep right at the fork onto I 395"

Does continue make sense as a maneuver type or is it a flag that belongs on all other maneuver types to indicate when a road is staying the same?
/cc @MoKob

Here's another example:

We get a "Continue right onto Marsh Road"
Then we get a "Turn left onto Marsh Road"

Why is the first one a continue and second a turn?

Why is the first one a continue and second a turn?

This is a bug that should be fixed in https://github.com/Project-OSRM/osrm-backend/pull/4400, but is not yet deployed.
Per voice with @daniel-j-h, the next action is to adjust the phrasing in our continue instructions to be more smooth. For example, instead of Continue slightly right onto {way_name} it should be Turn slightly right to stay on {way_name}.
This is immediately actionable and we should do this now.
However, I don't see how this will address the original fork case described in the OP. fork and continue are both maneuver types, but you can have a fork that continues. How we do handle that case? This leads me back to my original question:

Does continue make sense as a maneuver type or is it a flag that belongs on all other maneuver types to indicate when a road is staying the same?

This would allow any instruction to be rephrased with "stay on" instead of "onto" which is confusing when you're already on the road in question.

Here's an example of a fork instruction where the name is staying the same:
https://www.mapbox.com/get-directions/#16/36.9879/-122.0236?coordinates=-122.022312,36.989434;-122.02360473016304,36.987874443471256
Since this isn't a continue, I'm not sure we have enough information to make this instruction read "Keep right at the fork to stay on CA 1".

Next action is to improve whatever we can by rephrasing continue instructions to use "stay on" language.
Unless, I'm missing something we might need some fixes upstream in OSRM in order to support fork instructions that continue on the same road.

@willwhite are these the instructions you are saying need to be changed?
https://github.com/Project-OSRM/osrm-text-instructions/blob/master/languages/translations/en.json#L82
https://github.com/Project-OSRM/osrm-text-instructions/blob/master/languages/translations/en.json#L87

@willwhite in making this change I noticed a couple test failures worthy of additional consideration:

https://github.com/Project-OSRM/osrm-text-instructions/pull/142 is a big improvement but the instructions will still be bad for some of the example routes.

Will be "Continue left to stay on South Airport Boulevard" but should be "Turn left to stay on South Airport Boulevard"
Will be "Continue right to stay on Marsh Road" and "Continue left to stay on Marsh Road" but should be "Turn right to stay on Marsh Road" and "Turn left to stay on Marsh Road"

Thoughts on changing the "Continue" to "Turn" ?


Hook execution in current shell

I'm trying to take advantage of the BASE16_SHELL_ENABLE_VARS exports in tinted-theming/tinted-shell. I can set the BASE16_COLOR_##_HEX exports correctly if I source the script from my current shell like so:
source "$(tinty config --data-dir-path)/base16-shell-scripts-file.sh"
However, when I apply a scheme via tinty apply these exports are not set. I assume this is because hooks are executed via shell command instead of executing in the current shell. Is there a workaround for this that will allow the hook to run in the current shell?
What version of tinty are you using?
0.14.0

Hello! Yes there is a workaround: https://github.com/tinted-theming/tinty/blob/main/USAGE.md#sourcing-scripts-that-set-environment-variables

Thanks!


add missing files to XCode project

fixes the ticket http://answers.soom.la/t/compilation-error-in-cocos2d-levelup/1925/3

@vedi  thank you


Pressing l on a leaf entry is executing C-i in neovim

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md
[X] I have updated 'mini.nvim' to latest version

Module(s)
Mini.files
Description
When I press l (go in entry) in a leaf entry (a file and not a folder) it is going to next position in the jump list (C-i)
Neovim version
0.10
Steps to reproduce

nvim -nu minimal.lua
Be sure to have a next item in the jump list (C-i) (moves somewhere)
Open mini.files and navigate to a file
Press l

Expected behavior
No response
Actual behavior
I don't expect the current buffer to move to the next position in jump list

Thanks for the issue!
I can not reproduce and pretty sure this is not an issue with 'mini.files'. Could you try reproducing this with only 'mini.files' installed and no your user config loaded? You can use init-deps-example.lua for minimal config and :h $NVIM_APPNAME for how to test cleanly without affecting your current config.

I am closing this as I still can not reproduce. If you'll create a robust reproducible example, please, still post it here.


Humble - nav2 params file has not updated with new server name

nav2 has  renamed "recoveries server" to "behavior server".
https://navigation.ros.org/configuration/packages/configuring-behavior-server.html

Note: pre-Rolling/Humble this was the Recovery server, not behavior server.

Need to update the turtlebot nav2 params file with the new name.
for example:
https://github.com/ROBOTIS-GIT/turtlebot3/blob/humble-devel/turtlebot3_navigation2/param/waffle.yaml#L317

Thank you for your interest and advice.
We always strive to provide prompt updates to ensure the best quality in our products and resources.
The TurtleBot3 currently works well with the existing source and manual.
If a parameter name change becomes necessary, we will update it accordingly


Add URL for targeting Github Enterprise instances

Allow setting of a URL so vcpkg can download repositories hosted on GitHub Enterprise
instances. Additionally provides a way to set the authorization token when downloading
private repositories.

I haven't written the documentation for the new values yet. I wanted to know if there was any preference to the variable names. Currently its GITHUB_URL which matches whats in the GitLab equivalent. Also I was just using TOKEN for the OAuth token. If there are any preferences here for names let me know.

@vicroms not sure what the best naming of things are.
My only concern with GITHUB_URL is that it might imply that you need to set that which is not the case. GITHUB_ENTERPRISE_URL might be better?
With TOKEN I'm just not sure if that's clear enough. Maybe ACCESS_TOKEN? Whatever the consensus is it should probably be added to the gitlab command. I don't use GitLab so I wouldn't know how to check that though.

@Rastaban @ras0219-msft any feedback on this?

@vicroms @Rastaban @ras0219-msft is this something that you all don't want to support? The PR has been open for almost 2 months now and I'm not sure what is going on.

@vicroms @Rastaban @ras0219 any comments?

Hey @vicroms @Rastaban @ras0219-msft this PR has been open for over 5 months any feedback?

Thank you for your patience and thank you for the PR!
The design LGTM; I renamed the two parameters to GITHUB_HOST and AUTHORIZATION_TOKEN to be slightly more descriptive. I've also added basic documentation.
Assuming no CI issues, this is ready to merge.


Respect http(s)_proxy env vars in AsyncClient (like in Client)

requests uses the http(s)_proxy environment variables, aiohttp by default doesn't. This change allows the AsyncClient to work behind a proxy.

Thanks for this pull request @L3viathan. Good catch, we certainly had different behavior in the Client and AsyncClient here.
Happy to merge this and will provide a patch release afterwards.

Looks like the actions are running without secrets to me... 🤔


Add CMAQ data loader

 Gov4Git notices
On Friday, 23-Feb-24 21:20:56 UTC by Gov4Git dev
Notice 3y4lq5
Started managing this issue as Gov4Git concern 28 with initial priority score of 0.000000.
This project is managed by Gov4Git, a decentralized governance system for collaborative git projects.
To participate in governance, install the Gov4Git desktop app.
Notice 7usomf
This issue's priority score is now 0.000000.
The cost of priority is 0.000000.
The projected bounty is now 0.000000.
Notice osjyzj
The set of eligible proposals claiming this issue is empty.

 Gov4Git notices
On Monday, 26-Feb-24 15:00:15 UTC by Gov4Git dev
Notice vgcmol
This issue's priority score is now 3.000000.
The cost of priority is 9.000000.
The projected bounty is now 9.000000.
Notice 75zlgz
The set of eligible proposals claiming this issue is empty.


date range functionality idea

Hi there,
I had an idea for this plugin: what if we added a function that allows the user to check how many days/weeks/years are in a date range?
The idea stemmed from when I was filling in paperwork (e.g., how many months did you work here for?) and I needed a quick way to calculate that.
The plugin could also be a more general date plugin instead of just fetching the day of the week (e.g. more functionality such as converting Unix time to days).
I would also be happy to contribute to the code.
Kind regards,
Sheshydev

Hi Sheshydev,
That sounds like a great idea for a plugin! Adding a function that allows users to calculate the duration in days, weeks, or years for a given date range would definitely be useful in various scenarios.
Expanding the plugin to include more date-related functionality, such as converting Unix time to days, would make it even more versatile and valuable.
and we can add timezones function also.
If you're interested in contributing to the code, that would be fantastic! Open-source projects often thrive on collaboration, and your contributions would be highly appreciated.
Kind regards,
Ahmed ElSaeed


Add postgres package autobumping

we want to keep postgres packages up to date for the supported major
version lines. This task does exactly that. It checks the postgres git
repo for REL_* tags matching our used postgres versions and creates bosh
blobs from the src. It then replaces and uploads the packages used in
the release.
Implenents: [#182288132]
Fixes: CVE-2022-1552

LGTM


Embed job does not accept an array of texts

I'm using the embed job to embed my documents, and I have an array of strings. I tried to use the text field but it only supports strings, and the texts field isn't supported.
In comparison the regular embed api supports texts by default

did you get this sorted @rgb-panda ?


onDidOpenTextDocument Extension API fires twice for files in git repos

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

VS Code Version: 1.87.1
OS Version: macos sonoma 14.3.1

Steps to Reproduce:

Open a folder inside of vscode as a workspace
Run an extension that contains the following handler:

vscode.workspace.onDidOpenTextDocument((document) => {
  console.log(document.fileName);
})

files that are also a part of the git HEAD will execute this handler twice on open, one with the normal filepath, and the other with the same filepath but .git added at the end of it.

console output:
opened /path/to/file.ts
/path/to/file.ts
/path/to/file.ts.git

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.87.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


Test new upstream build config

Checklist

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

See https://gitlab.com/ljn917/rdchiral_cpp/-/issues/1#note_718833726 for context.

Let's wait for the CI to be green and I'll merge here.

Thanks @ljn917 for the new build config and thanks @maclandrol for the OSX confirmation!


How to use separate controllers?

I must be crazy or doing something wrong.

I mean, i enter quickadmin panel through /admin, but then all links in the menu points to /users or something always without the /admin prefix...? So all the routes are overwriting my default routes on my project?
Is there an easy way to put all quickadmin routes within the "admin" prefix, so it doesnt mess up my default routes?

Sorry for long waiting time.
There is an option in config file to do so:
We are using this to prefix our routes:
  'prefix'     => config('quickadmin.route'),


The performance of Haproxy drops a lot when using OpenSSL 3.0 for higher number of threads

Parameters:
Cipher suite: AES128-SHA
TLS version: TLSv1.2
CPS test result:

Flame Graph:
Haproxy with Openssl 1.1.1

Haproxy with Openssl 3.0.12

Root Cause:
tls1_change_cipher_state is consuming more CPU cycles due to "rw_lock" contentions from evp_generic_fetch

Will this be optimized in the future release?

of which the evp_generic_fetch path is one

At least one of the read locks here is namemap related - see also #23370.
There's another one in the method store (see crypto/property/property.c) that also gets hit in this codepath and should be another candidate for RCU (#22729).
So this looks like at least 2 read locks every time we do a fetch.

@abasapax  it would be interesting if you could get equivalent figures for 3.2. There are numerous performance improvements in 3.2 compared to 3.0 that might make some difference here.

We have measured the performance with 3.2 and couldn't see much improvement when compared to 3.0

would you be willing to try with this PR:
https://github.com/openssl/openssl/pull/23680
It contains several performance improvements that I'm working on

ping, any response here to the tests of #23680 ?

No response, marking as inactive, to be closed at the end of 3.4 dev barring further input

I apologize for the delayed response. With these improvements, we've started haproxy validation. Performance data results will be updated by EOW.

Hi,
We have checked Haproxy with TLSv1.2 & TLSv1.3 using "nhorman:handshake-test" branch and below are the observations.
In TLSv1.2 case seen significant drop:

TLSv1.2 test details:
Cipher suite: AES128-SHA
TLS version: TLSv1.2
Certificate: RSA 2048
In TLSv1.3 case seen some improvement:

TLSv1.3 test details:
Cipher suite: TLS_AES_128_GCM_SHA256
TLS version: TLSv1.3
Key exchange: X25519
Certificate: RSA 2048

I'm not sure I understand the details of what you tested
You indicate that you tested the handshake-test branch with HAProxy.  can you, for comparison, test without HAProxy (just to isolate the slowdown somewhat)?

If I am not wrong, You are asking to check with other applications nginx or speed right?

@bjayanax Sorry scratch what I said above, for some reason I read your results as a connection test between two endpoints with haproxy in the middle, not for a moment realizing that haproxy is in fact where you are measuring the performance of openssl.
That said, I'm hard pressed to understand these results.  Is it possible for you to create a flame graph of haproxy when used with the handshake-test branch?

From https://github.com/haproxy/haproxy/issues/2454#issuecomment-2353243899
it sounds like this is still a problem in 3.3 and 3.4.

It does, but re: my last comment here https://github.com/openssl/openssl/issues/23388#issuecomment-2210659090
A flame graph would help us identify where the continued slowdowns exist

Remeasured haproxy perf with the most recent nhorman:handshake-test branch and did not see a significant drop as previously, but it did not achieve the OpenSSL 1.1.1 value of 82k, only reaching around 60k with this development branch.

Along with this, we measured haproxy performance with the latest "openssl-3.4.0-beta1" branch and saw significant improvement for the P256 and P384 curves and their combinations.

In comparison to the openssl-3.4.0-beta1 branch, the nhorman:handshake-test * branch produced higher RSA2K values. However, these numbers are still lower than OpenSSL 1.1.1.


Update the documentation for the tool

Delete old docs and write new documentation that reflects V2 of the tool.

One minor update: https://github.com/log2timeline/PlasoScaffolder/wiki/Using-the-Tool
This is not supposed to live in the wiki, but be checked in under docs.

#50 should add sphinx api documentation to the project as well as moving all the Wiki pages to a separate source directory that can be used by sphinx, etc.


URL is wrong

The URL for Fix modified default rules in Azure AD Connect,  goes nowhere.
[Enter feedback here]

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

ID: cf9169c8-ec81-9342-c93e-963647d70c6c
Version Independent ID: 0ea93b27-477a-84e5-3d93-6707f5b9cb88
Content: Azure AD Connect: Upgrade from a previous version - Microsoft Entra
Content Source: articles/active-directory/hybrid/how-to-upgrade-previous-version.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@MattITMan
Thanks for your feedback! We will investigate and update as appropriate.

@MattITMan
Thanks for contributing towards enhancement of document. Required changes have been made and once approve by author it will reflect in the document itself.
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
Duplicate of #99450
Duplicate of #99497


Possible to Remove Notification from Status Bar?

Would it be possible to have an option to disable to the entry int eh status bar that says whether or not Stille is on?
Thanks :)

Hey @ReaderGuy42 thanks for the suggestion. So you want to be able to remove the status bar label that indicates whether Stille is on or off. Is that correct?

Yes, exactly :) I just prefer to have as little as possible in the status bar, that's why I ask. Thanks!

Ok, great! Now I understand @ReaderGuy42 thanks :)
Let me look into it.

@ReaderGuy42 I've just release 1.1.0 which adds a new settings toggle to turn the status bar label on and off. Give it a shot and let me know if that works!

Yes, perfect, thank you!!


Fix S3Reader skipping every other key

Copy self.keys since it's going to be modified after
This was causing every other key in s3 to not be read, the list first
was being added to last_position by reference and then removed the read
keys from it, causing the loop to skip one key for every readed key

Man!
How come this was passing the tests?

Thank you, @Youwotma !


selectMarksByValueAsync doesn't work with Measure Names and Measure Values fields

Hello,
when I am trying to select marks by selectMarksByValueAsync method on Worksheet object using Measure Names or Measure Values as field name nothing happens and marks are not selecting.
Bartosz

Thank you @bartoszZamojcin
We have duplicated the issue internally and found that Measure Names and Measure Values are improperly handled by internals in selection handling. We have narrowed down the issue and found a root cause. I created a defect and assigned it to the selections team. I have also reported that it is a blocking issue for you.
Thank you,
John

Internal defect is #1372577.

Good news @bartoszZamojcin
The selections team has fixed this. I have verified that I can use selectMarksByValueAsync on both Measure Names and Measure Values. This will be released in version 2022.2.
John

@johnDance
Thanks for dealing with the issue. When can we expect version 2022.2 to be released?
Bartosz

Normally it is at the end of that quarter. So for .2, it would be right around the end of June.

@johnDance
Unfortunately within our organization we wont get version 2022.2 this year, but instead it will be 2022.1, so the fix wont affect us. Is there any way to backport the solution to the version 2022.1 like you proposed in one of the other issues? This would be highly appreciated.
Bartosz


Temporarly disable DN Search Converter to fix issue #636.

This is a quick fix to be able to be able to login into Designer News, while a better fix is implemented.

Issue fixed meanwhile by #642
Thanks for investigating this, @pfmaggi!


Status logging

What this PR does / why we need it:
Sorry but I just had to fix the message regarding the progress after reading it several times a day in the past days...
Regarding the ellipsis ligature: I know this is to some degree a matter of personal preference. I'm not a fan of it as it distracts my eyes from the code. It's just weird to see those special characters sprinkled around. Also, if I were to add a new log statement with ellipsis I would have to go and copy one from somewhere else since I have no idea how to type one out on my keyboard.
Lastly, if your terminal doesn't support unicode they will print some mush instead of the ellipses.
There are some cool code fonts that present ellipsis (and some special characters) as ligatures, e.g. FiraCode.
Special notes for your reviewer:
I can easily pop off the commit that changes the ellipses. Let me know what you think.
Does this PR introduce a user-facing change?: no
NONE

/ok-to-test

/lgtm

@xmudrii PTAL

Thanks @s-spindler!


Media Manager per folder Roles/Permissions?

Hi there,
is it possible to set roles/permission on folders in the media manager? So that can have different folders and in each folder only certain roles can upload files (and also only these roles can see the folder)?
Example:
Role 1 can see folder "summer party" and can upload files into this folder.
Role 2 can see folder "christmas" and can upload files into this folder. But Role 1 could not see the "christmas"-folder and cannot upload files into this folder (and vice versa).
So something like Policies (where you can define that a user must be in Role 1 to see/edit/delete a Post for example) for the CRUD/BREAD part. But for the Media Manager.
I hope it's clear what I want to accomplish?

That's not currently possible, and is somewhat difficult to build with how the media manager is currently built.  It uses physical directories on the host as the directories in media manager, so we would need a whole new way of tracking the folders.
We'll look into it (and are open to pull requests if you wish to give it a try yourself!), but it likely won't get merged in until a new major/minor release (depending on implementation) as it would likely require breaking changes

This will be possible in version 1.2 (https://github.com/the-control-group/voyager/pull/3889).
It allows you to change the base_path during runtime.


fix assertion on the number of bits for the offset

The assert becomes an issue when the Offset was computed from an alloca instruction (address space 5), which is 32-bit, but the GEP instruction has an address expression in generic address (address space casted from an alloca), which is in 64-bit.

what if the GEP has 32 bit pointer but the offset is 64 bit? Is that possible?

Not 100% sure if that could happen (probably unlikely?) but definitely would be wrong code and will get caught by the assert if it occurs.
This was exposed by sroa so I'd think that it's unlikely that we'd start from an expression with generic address (where the offset was computed) and then generate a new GEP with a different address space with less bits?

LGTM. Thanks.


3016 list add pinned items

Closes #3016
Summary
Added feature to lock rows and prevent them from being reordered
Change List (commits, features, bugs, etc)

Added lockedIds prop to List and HierarchyList
Added List story showing lockedIds
Updated Hierarchy story to show knob for lockedIds

Acceptance Test (how to verify the PR)

Go to https://deploy-preview-3017--carbon-addons-iot-react.netlify.app/?path=/story/1-watson-iot-hierarchylist--with-nested-reorder

Verify that Jeff McNeil can't be directly reordered and that there is a lock icon on the row

Turn on multi-nesting and make sure it looks and works ok

Turn on hasVirtualList and make sure it looks and works ok

Go to https://deploy-preview-3017--carbon-addons-iot-react.netlify.app/?path=/story/1-watson-iot-list--with-reorder-and-locked-rows

Verify that the two top rows can't be directly reordered and that there is a lock icon on the rows

Verify that it is not possible to drag and drop another row above the locked row

Turn on hasVirtualList and make sure it looks and works ok

Regression Test (how to make sure this PR doesn't break old functionality)
All normal List and Hierarchy List stories work as before

Things to look for during review

[ ] Make sure all references to iot or bx class prefix is using the prefix variable
[ ] UI should be checked in RTL mode to ensure the proper handling of layout and text.
[ ] (React) All major areas have a data-testid attribute. New test ids should have test written to ensure they are not changed or removed.
[ ] All strings should be translatable.
[ ] The code should pass a11y scans (The storybook a11y knob should show no violations). New components should have a11y test written.
[ ] Unit test should be written and should have a coverage of 90% or higher in all areas.
[ ] All components should be passing visual regression test. For new styles or components either a visual regression test should be written for all permutations or the base image updated.
[ ] Changes or new components should either write new or update existing documentation.
[ ] PR should link and close out an existing issue

Nice work! Looks great to me. Just a couple iotPrefix replacements, and a couple questions:
Should we still be able to move a locked row in multiple-nesting using the checkbox in the HierarchyList with nested reorder story?

I don't know, probably not. It is not an issue for #2603 since it doesn't use the multiple-nesting but I will disable that checkbox if the row is locked since that makes more sense. Good call.

Do we need to test a scenario with a locked rows that items cannot be dragged above in the HierarchyList (ie. scenario 2 from the acceptable instructions, but with a HierarchyList instead)?

That is a valid question. There is no clear spec for this as a general List/HierarchyList functionality but you can test the scenario by using the knob lockedIds in the story "1-watson-iot-hierarchylist--with-nested-reorder". In order to prevent anything from being dragged above a topmost locked child item the parent needs to be locked as well. That is okay for my purpose with #2603.

It could be a storybook oddity but... If you turn on multi-select and you have the initial selected item still selected and then choose one more item. Move them both and only the second item is still selected.

Initial state
choose a second
once moved

It could be a storybook oddity but... If you turn on multi-select and you have the initial selected item still selected and then choose one more item. Move them both and only the second item is still selected.

I think this is a bug in the component, but it is unrelated to this PR and also happens in next. I created a new issue for it here https://github.com/carbon-design-system/carbon-addons-iot-react/issues/3042

The two locked rows do not show as draggable but I am unable to drag and drop any rows on the second link

I think you can drag all the other items (and that should be made visible by the blue line that indicates the drop zones) but nothing will happen once you drop since the List is stateless. The HierarchyList is stateful and that story also shows what happens after the actual drop. Let me know if I misunderstood you and you actually can't actually drag and drop any rows at all.

The two locked rows do not show as draggable but I am unable to drag and drop any rows on the second link

I think you can drag all the other items (and that should be made visible by the blue line that indicates the drop zones) but nothing will happen once you drop since the List is stateless. The HierarchyList is stateful and that story also shows what happens after the actual drop. Let me know if I misunderstood you and you actually can't actually drag and drop any rows at all.

We could add an action to the onItemMoved callback to make it clear something is happening.

The two locked rows do not show as draggable but I am unable to drag and drop any rows on the second link

I think you can drag all the other items (and that should be made visible by the blue line that indicates the drop zones) but nothing will happen once you drop since the List is stateless. The HierarchyList is stateful and that story also shows what happens after the actual drop. Let me know if I misunderstood you and you actually can't actually drag and drop any rows at all.

We could add an action to the onItemMoved callback to make it clear something is happening.

Yes, good idea. I'll add that.


Blog and documentation site michenriksen.com no longer has a domain?

Browsing http://www.michenriksen.com/ yields a 404, and nslookup michenriksen.com reports an NXDOMAIN (no such domain) error. Are the docs available elsewhere?

I think there has never been much more than the README file.

I forgot to renew my domain, but was luckily reserved by Gandi 😆


Feature: reference customization (i.e. classes)

We can wrap a full citation (possibly including multiple citation items) in a <span> element with a customizable class. Add configuration to enable/disable wrapping wrapCitation (default: true) and for customizing classes citationClasses (default: "citation").
Unlike #3, we don't need control per citation item to implement this.

This is, for the most part, implemented in #8


KfProducerSettings.java:20: test producer construction

The puzzle 26-7c38feaf from #26 has to be resolved:
https://github.com/eo-cqrs/eo-kafka/blob/4da907a416b37b1445aefc91045db61212d76378/src/main/java/org/eocqrs/kafka/KfProducerSettings.java#L20-L20
The puzzle was created by @h1alexbel on 22-Feb-23.
Estimate: 20 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 26-7c38feaf has disappeared from the source code, that's why I closed this issue.


prefer cabal when there is a cabal.project.{local, freeze}

I propose that cabal-install be the preferred backend if there is a cabal.project.local or cabal.project.freeze file, even if there is a stack.yaml, as this implies that the user has taken steps to prefer cabal.
Generally it would be good to allow the user to provide an envvar to choose the preference when there is ambiguity. e.g. many simple projects have a stack.yaml because somebody requested it. An envvar might just be the simplest option all around instead of getting into complex rules.

A configuration file should be specified if the automatic configuration fails. The cabal configuration is ranked below the other configuration options as cabal has the worst support for working out which component you need to start the repl (ie, there's no way to map a filepath to a component). If you use cabal it is likely you have to specify a config file anyway to specify the right component.

I assume you've seen http://hackage.haskell.org/package/cabal-helper ?

Yes, cabal-helper needs to be integrated into hie-bios but hie-bios was created to avoid depending on only cabal-helper.

btw my usecase is for the batch compiler not the repl. I would like to be able to launch tools that have access to ghc-lib (i.e. to see .hi files, source code bases, and in the future .hie files).

Yes. The guiding principal behind hie-bios is to not link against any specific library. This shifts the responsibility for describing the right environment to the build tool rather than the tooling author.
Anyone can write their own tool using whatever libraries they like and provide a suitable command line interface to query.


ImportError: cannot import name 'ToTAgent' from 'tree_of_thoughts'

ImportError: cannot import name 'ToTAgent' from 'tree_of_thoughts'
Anyone has the same problem?

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

+1

+1 how to solve it?


Implement hierarchical label management for tasks

As a user
I want to leverage hierarchical label structure to create tasks
so that I can implement the GTD methodology more completely

For example: auto/create/task. This convention could reuse auto/archive and then also be used for "events."


IllegalStateException in MarshmallowReprintModule.java

v3.1.0
IllegalStateException in MarshmallowReprintModule.java:
caused by java.lang.IllegalStateException : Failed parsing settings file: /data/system/users/0/settings_fingerprint.xml

Can you give me more information? What's the full stack trace, what device do you see that on, what version of android?

Issue appears to be happening on several Android versions and Samsung Devices and one HTC 10:
HTC 2PS64 10               v8.0
SM-N920V                     v7.0
SM-G920T                     v7.0
SAMSUNG-SM-G928A v7.0
SM-G925V                     v6.0.1

Exception:     java.lang.IllegalStateException
Ex. Message:   Failed parsing settings file: /data/system/users/0/settings_fingerprint.xml
Thread:        Thread[main,5,main]
Memory Usage:  7 / 3764 MB (0%)
Original stack trace:
java.lang.RuntimeException : Unable to start activity ComponentInfo{...SplashActivity}: java.lang.IllegalStateException: Failed parsing settings file: /data/system/users/0/settings_fingerprint.xml
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2904)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2979)
at android.app.ActivityThread.-wrap11(Unknown Source)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1643)
at android.os.Handler.dispatchMessage(Handler.java:105)
at android.os.Looper.loop(Looper.java:180)
at android.app.ActivityThread.main(ActivityThread.java:6944)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:835)
Caused by: java.lang.IllegalStateException : Failed parsing settings file: /data/system/users/0/settings_fingerprint.xml
at android.os.Parcel.readException(Parcel.java:1959)
at android.os.Parcel.readException(Parcel.java:1897)
at android.hardware.fingerprint.IFingerprintService$Stub$Proxy.hasEnrolledFingerprints(IFingerprintService.java:498)
at android.hardware.fingerprint.FingerprintManager.hasEnrolledFingerprints(FingerprintManager.java:774)
at com.github.ajalt.reprint.module.marshmallow.MarshmallowReprintModule.hasFingerprintRegistered(MarshmallowReprintModule.java:165)
at com.github.ajalt.reprint.core.ReprintInternal.hasFingerprintRegistered(ReprintInternal.java:73)
at com.github.ajalt.reprint.core.Reprint.hasFingerprintRegistered(Reprint.java:78)
at ...
at ...
at ...
at android.app.Activity.performCreate(Activity.java:6986)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1232)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2857)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2979)
at android.app.ActivityThread.-wrap11(Unknown Source)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1643)
at android.os.Handler.dispatchMessage(Handler.java:105)
at android.os.Looper.loop(Looper.java:180)
at android.app.ActivityThread.main(ActivityThread.java:6944)
at java.lang.reflect.Method.invoke(Method.java)
at com.android.internal.os.Zygote$MethodAndArgsCaller.run(Zygote.java:240)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:835)

Interesting. It's definitely not a bug in Reprint, so the best we can do is catch it. I'm not seeing that crash on any devices. How are you using reprint?

Indeed.  I agree, this is something that bubbles up from android.hardware.fingerprint.  I couldn't find anything tracking this specific issue in Android.  I did come across some info on issues with some OEMs breaking the APIs here for certain devices that don't have sensors, but nothing on the parse issue.
We are using reprint for a login feature in our app.  When this crash occurs, it's in our launcher when we try to see if the device supports and if it is set up for fingerprint (so we can add the feature to our login screen and settings menu).  Were you looking for more specifics?

I'll definitely catch the exception and return false. The fact that I've never seen this crash makes me wonder if you're configuring Reprint differently than I expect.

Can you check your merged manifest to make sure <uses-permission android:name="android.permission.USE_FINGERPRINT"/> is included?
Can you check that you're only calling Reprint.initialize before any other calls to Reprint, and only once in the lifetime of the process (e.g. in Application.onCreate, not on Activity.onCreate)?
Are you including the SPass module?


How to use the PreprocessorsFile ？

I don't see any way to use this file
how do i control the text to send server?
it always sends one or two words to the server because the text contains tags ( < b>< /b> or "< i>< /i>")

Entries in that file simply serves as a search-replace before the text is sent to the server.
As for the tags, I do not understand your question.

Which pair separator is used in these files?


Remove provider prefix from builder metadata extension method

Closes #942
Follows https://github.com/aspnet/EntityFrameworkCore/issues/16686
/cc @ajcvickers

Note: comment-related methods have been left as-is, since we're going to be dumping our Npgsql-specific comment support in favor of EF Core relational's (#892).


Make slides more neutral

[x] Remove LMU logo
[x]  No names on slides (instead link to team page)

see also #342

this is our new miminalistic title slide layout:

closing this -- done is better than perfect. logo can come in eventually


Any value can be passed to clientIdFromToken

Describe the bug
Its possible to pass anything to token at runtime
https://github.com/ghostdevv/jellycommands/blob/6cd717cde9ac359b5dbb52514bbe485a8a3ff615/packages/jellycommands/src/utils/token.ts#L7
This will run on any value that is passed to token
Buffer.from(token.split('.')[0], 'base64').toString();

Reproduction
    const notString = clientIdFromToken(1234567890);

    assert.strictEqual(notString, null);

Logs
No response
System Info
N/A

Since this function is internal, we probably don't need any checking of the value given - however it still may be possible to get a value to that function that isn't a string if we aren't checking types in other places
would be worth looking into that

After looking into it more this seems to be the case where it is relying on another function to catch a value that is not a string. Token is caught by cleanToken here
https://github.com/ghostdevv/jellycommands/blob/6cd717cde9ac359b5dbb52514bbe485a8a3ff615/packages/jellycommands/src/JellyCommands.ts#L48
cleanToken ref:
https://github.com/ghostdevv/jellycommands/blob/6cd717cde9ac359b5dbb52514bbe485a8a3ff615/packages/jellycommands/src/utils/token.ts#L3

Ok awesome! It's definitely a bit fragile but I am happy with that for the time beind


Export setByKey function

I have copied some things in the declaration in the comment from the setPath function.
#295

I rebased branch to next. Sorry, you will need to wait next minor release (for now you can just copy-paste method).
Can you fix ESLint warnings by calling eslint .

Thanks!

Thank you!


[AKS] Fixing Failing App Routing Tests

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
az aks approuting tests/commands were failing due to an issue in the AKS RP that has since been resolved, and also due to misconfigurations within the CLI testing suite, which this PR addresses/fixes.
General Guidelines

[x] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[x] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[x] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

AKS

Ran a live suite for all modified tests, all are passing - https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=102370209&view=results


Tests for different versions Keras

I noticed some failures when writing tests in the Lab, depending on the version of Keras.
The only version I tried that made it through the run was Keras==2.0.8.
Keras==1
Successfully installed backports.weakref-1.0rc1 bleach-1.5.0 funcsigs-1.0.2 h5py-2.7.0 html5lib-0.9999999 keras-1.0.0 markdown-2.6.9 mock-2.0.0 model-converters-0.0.3 numpy-1.13.1 pbr-3.1.1 protobuf-3.4.0 pyyaml-3.12 scipy-0.19.1 tensorflow-1.3.0 tensorflow-tensorboard-0.1.5 theano-0.9.0 werkzeug-0.12.2
Traceback (most recent call last):
  File "/usr/local/bin/keras_to_tensorflow", line 10, in <module>
    KerasToTensorflow.convert(sys.argv[1], sys.argv[2])
  File "/usr/local/lib/python2.7/dist-packages/model_converters/keras_to_tensorflow.py", line 20, in convert
    model = KerasToTensorflow.load_keras_model(model_path)
  File "/usr/local/lib/python2.7/dist-packages/model_converters/keras_to_tensorflow.py", line 10, in load_keras_model
    return keras.models.load_model(model_path, custom_objects={
AttributeError: 'module' object has no attribute 'models'

Keras==2
Traceback (most recent call last):
  File "/usr/local/bin/keras_to_tensorflow", line 10, in <module>
    KerasToTensorflow.convert(sys.argv[1], sys.argv[2])
  File "/usr/local/lib/python2.7/dist-packages/model_converters/keras_to_tensorflow.py", line 20, in convert
    model = KerasToTensorflow.load_keras_model(model_path)
  File "/usr/local/lib/python2.7/dist-packages/model_converters/keras_to_tensorflow.py", line 12, in load_keras_model
    'relu6': keras.applications.mobilenet.relu6,
AttributeError: 'module' object has no attribute 'mobilenet'

We might want to consider only supporting 2.0.8 forward at this point.
cc @stephensolis

Mobilenet support was introduced with Keras 2.0.6 (release notes), that's why that was failing.
However, the APIs this converter actually depends on are present since Keras 1.0.7 (relatively trivial patches would be needed for older versions, but considering its age, I think that's a good cutoff point).
(Tensorflow's SavedModels were also introduced in 0.12.0, but since TF older than that was not distributed on PyPI, I don't think that's such a big deal)


Fix division by zero in GuiScrollBar function

When trying to use the floating window example, I got several times some crashes when expanding the window size.  It's caused in the GuiScrollBar function, when the maxValue is equal to the minValue (I don't know if this case is intended in the first place). Because valueRange variable is maxValue minus minValue, then it is equal to 0, and triggers a crash later, when we use it in division (division by 0).
I'm not sure if it's the best fix, maybe minValue == maxValue is not indentend in the first place and we should fix this problem earlier?

@Elkantor thanks for the review.

I'm not sure if it's the best fix, maybe minValue == maxValue is not indentend in the first place and we should fix this problem earlier?

Fix looks ok to me, avoiding the crash is the priority.


Admin panel changes don't allow space entry

This line https://github.com/paulsnar/edaemon/blob/7c7ff87d790eaa40a61c6c94920e44b482393240/management_frontend/src/components/LessonsColumn.js#L33 is the culprit mostly. That trimming should be done only on serialization, not on inputs.

Resolved in staging, will be pushed out together w/ 2.0.2 this week.


✨ Added PollResponse Through Email

Fixes #95
I have implemented the code for sending email to the poll creator whenever someone votes on their poll

Congrats and thanks!

@PushkarDureja I'll take a look once I reach home (in couple hours).
Thanks for contributing!

We will merge it once the endpoint is done.
Thanks for the contribution!

Great !!

@PushkarDureja have you tested this PR with the new endpoint locally? If yes, we can merge this.

@PushkarDureja have you tested this PR with the new endpoint locally? If yes, we can merge this.

Yes I did it yesterday only after adding back the auth condition.

Thanks, merging!

We've deployed this PR to our site, so can you check out the changes and let us know if everything is fine?

We've deployed this PR to our site, so can you check out the changes and let us know if everything is fine?

Sure, Let me check

@AnandBaburajan I don't think it is working!! Did you redeploy the mailer ?

@AnandBaburajan I don't think it is working!! Did you redeploy the mailer ?

Yup.

@AnandBaburajan I don't think it is working!! Did you redeploy the mailer ?

As far as I remember, It was working fine on my local environment. But give me some time to figure it out

@AthulJoshi can you share the mailer logs with Pushkar and/or look into it? Thanks!

@AthulJoshi  I have added a comment, Please have a look

So, the page is reloading before the request is completed right?

So, the page is reloading before the request is completed right?

But even if the request has been initiated (which is actually happening, I checked on the live one), Will it depend on the fact,  whether the browser is being reloaded or not.


Ensure that the Media order in ExportActionMixin is the desired one

Problem
Django's admin/js/jquery.init.js may be loaded after import_export/action_formats.js
Fix #1006
Solution
As stated in the django release notes, added admin/js/jquery.init.js in Media
Acceptance Criteria
Manually tested in versions 2.0.13 and 2.2.8

Coverage decreased (-0.2%) to 95.424% when pulling 141364cc081d97dd4b0413f3c85977bf8c1de812 on Microdisseny:fix-1006 into 3cf5e3f9796a5caf7c5ea3928119af7ce4706c0d on django-import-export:master.

Closing in deference to #1007


update spectacle to version 1.0.5

Changes to a cask
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[ ] brew cask audit --download spectacle.rb is error-free.
[ ] brew cask style --fix spectacle.rb left no offenses.

Check out cask-repair, it makes Cask updates like this quite easy (details in CONTRIBUTING)

Closing, as there are merge conflicts. Will submit an updated PR for you, this time.


Suggestion: consolidate rc-test-template and rc-test-template-new

As of this writing there are two RC test templates, named as described in the issue title.  This seems confusing, and having something like "new" in the title is going to be even more confusing once it gets a little older.  Can we just have one rc-test-template?  After all, creating new versions of things without having to rename them is exactly what git and GitHub are for.
If both versions are still needed, please rename them to something that makes it clear when one should be used over the other.

Above commit changes "rc-test-template" to "old-relic-rc-test-template" and "rc-test-template-new" should be used from now on.  Keeping old test template for time being as a few issues were found with the new markup file and I want it there as a back up in case we decide the new template isn't sufficient.  After a few successful rcs, if the devs like the new template the old will be deleted.


Add tests for loading/error states

Verifying that https://github.com/emberjs/ember.js/pull/14545 does what it needs to in this addon. Should restart the CI processes once the upstream PR lands.

https://github.com/emberjs/ember.js/pull/14545 landed (which I believe this is blocked on), and is cherry-picked into beta, once the Travis build for beta completes we should be able to rerun these builds and get them passing.


Demo: Constant Demo URL

Demo URL changes every time running docker compose down && docker compose up because of Load balancer change so the Demo URL in Readme.md needs to change.
It's good if the demo URL is constant.
Solution ideas:

Manually create Load balancer and reuse it
Configure URL related parameter of load balancer with x-aws-cloudformation (if possible)
Get custom domain and use it

Closing because we started to use ec2 instance.


multi: utilize tx confimation notifications.

This uses tx confirmation notifications from the wallet to confirm the coinbases sourced by payments are spendable before attempting to spend them. The readme now  better explains why multiple pool fee address and multiple mining addresses are important to set in production.
Resolves #222 and #218.

Tested thoroughly on testnet with 6 mining clients, using both PPS and PPLNS. Performed start/stop testing. Everything is looking good :+1:


added client app.json

What this does
Adds an app.json template for basic TN Apps, this is separate than the one we currently have which is set up for the bootstrapper's review apps

[Jira ticket](Paste Jira link here)

Checklist

[ ] Add script that changes the file name in place to app.json and removes the default one created for the bootstrapper pipeline

Deploy Notes
n/a
How to test
n/a

Currently blocked as we can't test this while Heroku's connection to Github is disabled due to that security issue.

Currently blocked as we can't test this while Heroku's connection to Github is disabled due to that security issue.

@oudeismetis  is this still blocked? or even needed?

I don't normally like doing this, but there are a couple of small PRs like this that I'm going to merge into a single PR so we can get these things in.

closing this PR in favor of https://github.com/thinknimble/tn-spa-bootstrapper/pull/129


Correcting translations

This pull request includes the following changes:
Adjusted the displays to accommodate both translations, specifically addressing the length variations between English and French.
Incorporated the requested changes from previous tickets.
These changes aim to improve the user experience by ensuring that button displays are optimized for both English and French translations, preventing text overlap and maintaining a clean interface.
Thank you for your review.

@MohamedRayenJmili  i made some comments on your change can you please check them and update the code thanks


Zap Nylas-N1 configuration files

Signed-off-by: Sébastien Gross <seb•ɑƬ•chezwam•ɖɵʈ•org>

Thank you for the contribution. It needed some fixes, so they were made in aca5659e621c8c60c7b5b106d1c52d91629f5982. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Custom 404 page for static view

I am trying to create a custom 404 page for the static site view.
If I direct 404 errors to the static root, then it serves the message from Server.php:
throw new NotFoundException('The Page you requested is yet to be made. Try again later.');

However, this then directs users to links that are not static (they are effectively now out of the static root) and have lost the search bar.
Is it possible to create  a hidden page in the static view (that doesn't show in the Nav menu) that can be used as a 404 page?

Hello, yes I think we can generate a page for this.

I'm closing this issue as the Daux project has been moved to https://github.com/dauxio/daux.io
For this specific issue, I'll have a look to create it in the project


Add download button to column action in directory listings

Reopening PR. See #904

@k8n thanks for this contribution!


Add Easy Install Script

I noticed https://github.com/zyedidia/eget has a quick install script and I figure this should have it too 👍🏻

Wow ty I really appreciate the effort, however I actually don't want to include this kind of script. Actually, I used to have one and I removed it in this commit. Here are my reasons why:

From a security perspective, I don't want to encourage curling random scripts from the internet. I know it's pretty commonplace now, but I'm always wary about installing things this way.
Adding an install script means I will have to maintain an install script.

Closing this PR, but again I appreciate the effort!


Rendering luminosity difference between Web/Native

I'm doing some shader work and asset testing. Basically, I'm trying to display the same scene on Web and Native. While testing, I've realized that the rendering is way darker on Native compared to Web.

Duplicate here : https://github.com/BabylonJS/ThePirateCove/issues/171 for private assets.

experience.js cube gives the same pixels between web and native.

A NodeMaterial with a texture and convert to linear space on gives the same result.

Fixed in bjs : https://github.com/BabylonJS/Babylon.js/pull/12172


Execution terminal transition

All updates made and merged in the last PR (#76) in this nested chain of collection execution PRs