Component chart state should be updated on call(component), not call(component.event)?

Currently the zoom behavior and the brush update their __chart__ state on their respective event listeners. However, it would be possible for the brush to update __chart__ state when the component is redrawn (or transitioned), independent of whether we want to dispatch events for those changes.
This would ensure that the __chart__ state is consistent with the display of the brush, even if the brush extent is changed within a brush event handler without subsequently calling brush.event, as in the Brush Snapping II example.
It is unclear how this would affect the design of the zoom behavior, which has no displayed state, and thus is not typically “redrawn”. Alternatively, __chart__ should be documented as part of the public API so that when the behavior of the brush is customized, there’s a supported way of keeping the persisted state of the brush consistent with its appearance.
/cc @wycats

Seems handled in the new D3 4.0 modules.


chore: update quarkus patch

update of quarkus patch. Upstream PR https://github.com/knative/func/pull/1531

/cc @matejvasek

/approve
/lgtm


R-T5.4-8: The data pipeline module of the   Orchestrator should support deploying data pipelines expressed using TOSCA models into the AWS data pipeline service.

ID:
R-T5.4-8

Type:
PORTABILITY

User   story:
As an Operations Engineer, I want to   deploy data pipelines to AWS data pipeline service

Requirement:
The data pipeline module of the   Orchestrator should support deploying data pipelines expressed using TOSCA   models into the AWS data pipeline service.

Extended   Description:
The goal is to support deploying data   pipelines into an existing data pipeline service provided by Public clouds.   AWS data pipeline service is a reference case.

Priority:
Must have

Affected   Tools:
ORCHESTRATOR

Means   of Verification:
Tests to verify that deployment to AWS   data pipeline service is successful

Take a look at for my first comment here: #17.

A number of TOSCA node types are developed that deploy the data pipeline into AWS data pipeline service. For instance, AWSCopyS3ToDynamodb TOSCA node type can be used to create and deploy data pipeline into AWS data pipeline service to move data form S3 bucket to dynamo db. Some other related TOSCA node types are AWSCopyDynamodbToS3, AWSCopyS3ToS3, AWSShellCommand, and AWSSqlActivity.
This issue can be closed now. @gcasale

Approved


the code logic of getting kafka topic at oap and agent side is not consistent

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[x] Bug
[ ] Requirement
[ ] Feature or performance improvement

Bug

Which version of SkyWalking, OS, and JRE?
Skywalking8.6.0 mac jdk1.8.0_191

What happened?
the code logic of getting kafka topic at oap and agent side is not consistent. Let's see.

OAP side

// the AbstractKafkaHandler class
 public String getTopic() {
       StringBuilder sb = new StringBuilder();
       if (StringUtils.isNotBlank(config.getMm2SourceAlias())) {
           sb.append(config.getMm2SourceAlias()).append(config.getMm2SourceSeparator());
       }

       if (StringUtil.isNotBlank(config.getNamespace())) {
           sb.append(config.getNamespace()).append("-");
       }
       sb.append(getPlainTopic());
       return sb.toString();
   }

Agent side

//  the KafkaProducerManager
  String formatTopicNameThenRegister(String topic) {
        String topicName = StringUtil.isBlank(KafkaReporterPluginConfig.Plugin.Kafka.NAMESPACE) ? topic
                : KafkaReporterPluginConfig.Plugin.Kafka.NAMESPACE + "-" + topic;
        topics.add(topicName);
        return topicName;
    }

Requirement or improvement
We can see the getting logic among oap and agent isn't consistent; Getting topics in oap depend on "mm2SourceAlias" and "mm2SourceSeparator" besides of namespace and origin topic; but only namespace and origin topic in agent; the two fields "mm2SourceAlias" and "mm2SourceSeparator" don't have other usage;
Possiable fix
First change the logic of getting kafka at oap side
 // the AbstractKafkaHandler class
  public String getTopic() {
        StringBuilder sb = new StringBuilder();

        if (StringUtil.isNotBlank(config.getNamespace())) {
            sb.append(config.getNamespace()).append("-");
        }
        sb.append(getPlainTopic());
        return sb.toString();
    }

Second delete the mm2SourceAlias and mm2SourceSeparator fields in KafkaFetcherConfig class

There 2 configurations were added through https://github.com/apache/skywalking/pull/5949. The PMC team doesn't use or test it directly. @osiriswd was the code contributor.

This seems caused by a Kafka mirror system.

The doc has included this

When use Kafka MirrorMaker 2.0 to replicate topics between Kafka clusters, you can set the source Kafka Cluster alias(mm2SourceAlias) and separator(mm2SourceSeparator) according to your Kafka MirrorMaker config.

This was reported through #7316

There 2 configurations were added through #5949. The PMC team doesn't use or test it directly. @osiriswd was the code contributor.

@wu-sheng Is the function of replicating  topicsKafka in  MirrorMaker 2.0 still using in product environment? Can I change the code logic of getting kafka topic at oap side and remove the two fields?

Is the function of replicating topicsKafka in MirrorMaker 2.0 still using in product environment?

Is this contributed by contributors. I don't know how the community uses it. This would have a side effect. We could accept a feature only part of users need this, but has no harm.

Can I change the code logic of getting kafka topic at oap side and remove the two fields?

What do you mean? I am confused.

Is the function of replicating topicsKafka in MirrorMaker 2.0 still using in product environment?

Is this contributed by contributors. I don't know how the community uses it. This would have a side effect. We could accept a feature only part of users need this, but has no harm.

Can I change the code logic of getting kafka topic at oap side and remove the two fields?

What do you mean? I am confused.

@wu-sheng ok, I see. the two fields only used in the logic of getting topic name at oap side.

Is the function of replicating topicsKafka in MirrorMaker 2.0 still using in product environment?

Is this contributed by contributors. I don't know how the community uses it. This would have a side effect. We could accept a feature only part of users need this, but has no harm.

Can I change the code logic of getting kafka topic at oap side and remove the two fields?

What do you mean? I am confused.

@wu-sheng ok, I see. the two fields only used in the logic of getting topic name at oap side.

These 2 are only used when you need MirrorMaker, nothing more.


Questions about paper

Hello,
I've read your paper and have a few questions that I wanted to discuss:

In Figure 1 of the paper, pooling is done through token averaging. Have you tried any other pooling methods? I understand that the approach averages out patch tokens for each split (for instance, averaging [0~3, 0, 0] if the patch token at the very edge on the left of the first picture is [0, 0, 0]). However, there seems to be no ablation study on why this design was chosen. While averaging appears to be a reasonable approach, I wonder if there have been any experiments regarding this.

In Figure 7(a), when comparing the ViT-L and ViT-B-S^2 models, it's noted that ViT-L exhibits slightly better generalization. How about to interpret this from a shape-texture bias perspective? (curious about your thoughts!!) For instance, a television predicted as a chestplate might look very similar in texture to an actual chestplate, and a flute predicted as a triceratops could also be perceived as similar in texture. It seems that the inclusion of multi-scale features might enhance more local (closer to texture) features, and I'd like your insight on this.

I keep coming back to the concept of token average pooling :). In the case of the picture in Figure 1, it seems possible to adjust the number of tokens using a resampler for the 32x32x768 setup. I'm curious if any experiments have been conducted regarding this.

Thank you for sharing these interesting results!
Best,
Jihwan

Thanks for the questions Jihwan!

Yes we've tried 2x2 conv instead of 2x2 average pooling. Turns out there's no significant difference in terms of performance.

Yeah that's a good point. My intuition is more image scales brings larger model capacity and thus better generalization (given sufficient data). We didn't test the shape-texture bias of the model, and it's not clear what's the relation between shape-texture bias and generalization.

Yeah a resampler can extract relevant information from lots of tokens without introducing too much computation cost. It can also avoid the information loss caused by average pooling the 32x32 feature map to 16x16. We only tried on LLaVA-like model and didn't try the resampler. I think this could help and welcome to try it out! I will be interested to know about the results of that:)

Best,
Baifeng


Add tests for BuildMerklePath

Summary
see comment

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned

Closed by #7160


Phoenix OS not working

used the installer and got an  windows boot error when choosing android as a boot option

im using an Acer Switch Alpha 12 64-bit UEFI


circular symlink breaks typescript intellisense

Steps to Reproduce:

Before creating symlink, intellisense works:

Inside some project folder, create symbolic link to parent folder
$ ln -s ../../ baselibs

Restart vscode

Now intellisense fails:

My project folder has 16942 files and 2944 folders. I try to reproduce the issue in a simpler folder structure, but i couldn't.
I also tried to ignore the symlink in the settings.json, but this doesn't fix the issue.
Gladly i'll provide more info.

VSCode Version:
Versión 1.5.1
Confirmación 07d663dc1bd848161edf4cd4ce30cce410d3d877
Fecha 2016-09-08T21:34:58.760Z
Shell 0.37.6
Representador 49.0.2623.75
Node 5.10.0
OS Version:
Linux os006724 4.6.0-1-amd64 #1 SMP Debian 4.6.4-1 (2016-07-18) x86_64 GNU/Linux

@bedorlan thank you for reporting this issue. Is it possible for you to share your project so we can try and reproduce on our end?
@dbaeumer this is a tough one. He wasn't able to reproduce with a simpler structure. Any thoughts?

Finally, i was able to reproduce the issue. project attached.
project.zip
Steps to Reproduce:

uncompress project.zip & project.tgz (automatic symlinks)
open the folder project
open the file 0/1/2/test.ts
try intellisense

tip: i'm using a custom typescript sdk:
{
    "typescript.tsdk": "./node_modules/typescript/lib"
}

@waderyan pretty sure this is a tsserver issue. Since they load the whole project I don't know whether they 'normalize' links to avoid traversing the cycle endlessly.

This issue was moved to Microsoft/TypeScript#10873


Revert "Allow unordered properties"

It is always easier to find a property when they are sorted instead of some colleague arbitrarily placing it somewhere in a properties file.

As discussed in the CoP. We do not think that we would benefit from an alphabetical ordering. Of course we can always opt-in to sort our properties this way but the company-wide policy will not enforce this.


Markdown is being parsed twice

I believe there is a bug with https://github.com/tuchk4/storybook-readme/commit/99f8fe19b831c3c44a486dc820de9bf0ab11388e.
if (type === DOC) {
  // parse 1: args is already parsed
  return (...args) => {
    // parse 2: normalizedDocs is parsed again
    const normalizedDocs = normalizeDocs(args);
    return handler.doc({
      docs: [
        ...normalizedDocs.docsBeforePreview,
        ...normalizedDocs.docsAfterPreview,
      ],
      config,
    });
  };
}

Markdown that is being parsed twice (This is a snippet that is being consumed by import { doc } from 'storybook-readme';):
import Grid from '@oct/ui/grid';
import GridColumn from '@oct/ui/grid-column';

<Grid>
  <GridColumn className="medium-2 large-4">Charmander</GridColumn>
  <GridColumn className="medium-4 large-4">Bulbasaur</GridColumn>
  <GridColumn className="medium-6 large-4">Squirtle</GridColumn>
</Grid>

The first pass parses the < and > into < and > respectively. The second pass identifies the newline above <Grid> and converts the newline into a p—which it shouldn't.
With 4.0.2:

With 4.0.3:

got it. Will publish new version today

Will be released with https://github.com/tuchk4/storybook-readme/issues/118

It is fixed<EMAIL_ADDRESS>Note that right now there are breaking changes at alpha version. I wrote about here https://github.com/tuchk4/storybook-readme/issues/118#issuecomment-472684902
BTW if you really need I can make this fix and release it for version ^4.0.5 or wait for 5.0.0
cc @PatrickChevalierADSK as reporter of https://github.com/tuchk4/storybook-readme/issues/115

fixed in 5.0.0


Internal public API review for Preview 1

[x] Review the public API and hide any members that are not intended to be part of the customer-facing API.
 Submit public API (.swiftinterface) for architect review
 Make adjustments and pass architect review
Moved to issue #322

Moved the arch-board review to separate issue #322 so as not to block Preview 1, per @mayurid.


Troubleshooting Microsoft Defender for Identity known issues

Troubleshooting Microsoft Defender for Identity known issues
a.	[https://learn.microsoft.com/en-us/defender-for-identity/troubleshooting-known-issues](url)
b.	The troubleshooting article is specifically targeted toward resolving general issues – particularly hard-to-troubleshoot problems.
c.	I would like a section for diagnosing and resolving this scenario where “NT Service\All Services” is not granted “Logon as a Service” permissions.
i.     This link here looks to have specific steps for diagnosing this error
d.	[https://techcommunity.microsoft.com/t5/core-infrastructure-and-security/microsoft-defender-for-identity-azure-atp-deployment-and/ba-p/1676122#:~:text=Clustered%20to%20Flat.-,Sensor,-service%20%22Azure%20Advanced](url)

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

ID: 3de7f0b7-1c29-6714-51b9-75df6731ce25
Version Independent ID: c6ada0cd-4fb8-b1eb-db34-206660e33e9f
Content: Troubleshooting known issues - Microsoft Defender for Identity
Content Source: ATPDocs/troubleshooting-known-issues.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thank you for your comment. We'll investigate and get back to you.

Hi @dcurwin checking for updates on this. Thanks in advance!

@shellymoore23  - Thanks again for your suggestion. We've updated the document.

Thank you so much!
Shelly Moore
Incident Manager US Public Sector
Hours: M-F 8am-5pm CST
Office: +1<PHONE_NUMBER>
Microsoft Services Support:
https://serviceshub.microsoft.comhttps://serviceshub.microsoft.com/
Support ((24/7): 1-800-936-3100
Support (US NAT): 1-800-936-3200
GCC High: 1-877-966-7787
[cid:1e215f5a-59e4-4eb2-8ad6-a7715edae4f6]
Please @ me for Escalations and High Priority issues.
For feedback or concerns, please email my manager,  Kevin Dozier– @.@.>).

From: David Curwin @.>
Sent: Thursday, January 26, 2023 11:45 AM
To: MicrosoftDocs/ATADocs @.>
Cc: Shelly Moore @.>; Mention @.>
Subject: Re: [MicrosoftDocs/ATADocs] Troubleshooting Microsoft Defender for Identity known issues (Issue #587)
@shellymoore23https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fshellymoore23&data=05|01|shellymoore%40microsoft.com|df308a94a5884eacd6e208daffc5252c|72f988bf86f141af91ab2d7cd011db47|1|0|638103519468691442|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=uW%2BEq0OuUQVqnfuDzDSCIPY6ZpfsMotY9Jq8ikTlanc%3D&reserved=0 - Thanks again for your suggestion. We've updated the document.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FATADocs%2Fissues%2F587%23issuecomment-1405370599&data=05|01|shellymoore%40microsoft.com|df308a94a5884eacd6e208daffc5252c|72f988bf86f141af91ab2d7cd011db47|1|0|638103519468691442|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=I0Qx4pgL4tYU5cWyiyRQreIAUoljsV%2F0q2PeT2fFtJ4%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA5KRNTKDKWU7WBUQKDLFR7DWUKZ4LANCNFSM6AAAAAAT7OSWPI&data=05|01|shellymoore%40microsoft.com|df308a94a5884eacd6e208daffc5252c|72f988bf86f141af91ab2d7cd011db47|1|0|638103519468691442|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=vJW3PmysCunSz6JEO%2BzOpbrS8qop%2By0O8hsNjPoxjM0%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Feat/set max concurrent streams

Description
Adds the ability to set the max concurrent stream option from grpc. Also increases the max workers default value to 300.
Type of change

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

Codecov Report
Base: 55.19% // Head: 55.20% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (285eec6) compared to base (8badaed).
Patch coverage: 66.66% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #380   +/-   ##
========================================
  Coverage    55.19%   55.20%
========================================
  Files           24       24
  Lines         1683     1690    +7
========================================
+ Hits           929      933    +4
- Misses         688      690    +2
- Partials        66       67    +1

Flag
Coverage Δ

unittests
55.20% <66.66%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

core/pkg/membrane/membrane.go
64.25% <66.66%> (-0.22%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


IsSystemdService: make check stricter to avoid false positives

Checking the INVOCATION_ID envvar is causing false positives.
Instead, we now return true only when the direct parent process is 'systemd'.
Fixes https://github.com/aspnet/Extensions/issues/2525
cc @Tragetaschen @anurse @onyxmaster @Tratcher

Technically breaking so we should run this by the API/Breaking Change review group (cc @aspnet/asp-net-api-reviews). I think it's warranted though (and the set of people it would break is pretty small: scenarios where INVOCATION_ID is set, the direct parent is not systemd, and the desired behavior is to act as a systemd service).

Thanks @tmds! @anurse do you think we should backport this?

The fact that it is technically a breaking change makes me a little nervous. But it is customer-reported and has no workaround. We also strongly believe there is no intended scenario that would be broken.
Let's take it to Tactics. @halter73 could you open a backport PR? I can fill in the justification details.


PoET does not include all miners in the membership proof

Nodes register to the PoET, they print:
challenge submitted to PoET proving service (PoET id: f2faab4ce2abd0cdfb35e029f2a4c3bd1f4117ac48b1f9e8db0044c038a4b94a, round id: 8788)
Later they get the PoET proof, but are not included in it:
cannot create atx in epoch 2: cannot create Nipst: not a member of this round (poetId: f2faab4ce2abd0cdfb35e029f2a4c3bd1f4117ac48b1f9e8db0044c038a4b94a, roundId: 8788)
In addition, if this happens, the miner should remove this NIPST immediately, instead of try again next layer (and outputting the same error)

Closing this although the bug was not found. If it happens again, please reopen.


Unresponsive when vMix is unreachable

Websocket connection is almost unreachable when device is trying to connect to vMix.

Changes tcp connection to async


Command-line - Linux platform version works on windows platform

What version of Flyway are you using?
4.2.0
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Command-line
What database are you using (type & version)?
Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
What operating system are you using?
windows
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
I typed command "flyway migrate" sqls by using flyway-commandline-4.2.0-linux-x64 on windows platform.
What did you expect to see?
Some erorrs should be prompted, like "wrong platform".
What did you see instead?
Seems Linux platform version works on windows platform. Could anyone confirm this? Thanks in advance.

Each download ships with both a flyway (Linux and macOS) and flyway.cmd (Windows) launch script. Only the bundled JVM is different. However the startup scripts do fall back to a system JVM if available.
Works as designed.


Make flow-sensitivity understand instance_of?

This makes sure that, after calling instance_of? on an object, Sorbet
knows about the type.
Motivation
Fixes #48
Test plan
Added automated tests.

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build result here:
→ https://go/builds/bui_HptdmlRUXPlQhe


Remove hardcoded broker class from tests

This will actually run namespaced Broker tests

/cc @aliok @Cali0707

/retest


Setting cmdheight breaks command_line_delay logic

This is an issue for the problem we discussed earlier where setting cmdheight for a value bigger than 1 will result in the command_line_delay not getting executed as it only accounts for last row in the editor grid, which isn't the case for cmdheight values over 1

I think this has been fixed in main. Please reopen if you come back to this and the issue remains.


Aliases with quotes, single or double, aren't working

These commands fail:
alias l="ls -l"
alias l='ls -l'
alias l="ls"
alias l='ls'
This succeeds:
alias l=ls
I believe it treats l="ls -l" as 2 arguments: l= and "ls -l", but they should be one.

Are you talking about in your .bashrc?

No: see the screenshot.

Cool, thanks for the update.

I'd like to try to fix this.
I think the root cause is a bug in the Scanner: every quoted string literal is a separate token, but it shouldn't be. Example: echo a"b"'c'. Expected output: abc, actual: a b c. The simplest way to fix it is to do this: if there are 2 string literals next to each other with no space characters between them, then combine them into one. Does this make sense?

Yeah, that does. The scanner code always did look a little suspicious to me. We would definitely appreciate the help! (I would also appreciate a test case in scanner_spec.ts if you do fix this)


ScatterLayout's export_to_png doesn't center properly

Here's an album of screenshots and png exports showing what I mean: http://imgur.com/a/jpc1h

Could you share a simple example for easier testing? I could not reproduce it with examples/widgets/scatter.py.


Using Krakend's JWT validator

Hello! Always i'm trying to use the auth/validator middleware, i'm getting the following error:

This is my template file:

And my root file:

It was a problem with docker-compose's environment variables.


[release-calendar] ChannelTable Component

The ChannelTable Component will display the different channels- long term stable, stable, beta 1%, experimental 1%, beta opt-in, experimental opt-in, and nightly- along with newest RTVs that have gone through those channels.

This PR has become too large and complicated. It has now been split into several smaller prs in order to allow for an easier time to review.


Resolve: Refactor Context.GenericTag to Context.Tag

Close #359

Thank you! But I already refactored it in a brach.


Add tests

The following components should be tested:

geojson import
comparison function
running the entire program?

Done. The tests could be expanded, but at least the first steps have been taken


Image copy, subprocess.CalledProcessError returned non-zero exit status 3.

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az image copy Extension Name: image-copy-extension. Version: 0.2.7.
Errors:
Command '['/usr/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Windows-2019-Crowe-Template-NCUS-PRD1_os_disk_snapshot', '--location', 'northcentralus', '--resource-group', 'IS-CoreSystem-PRD1', '--source', '/subscriptions/<<removed>>/resourceGroups/TEMP-image-transfer/providers/Microsoft.Compute/snapshots/Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.
Traceback (most recent call last):
...

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az image copy --source-object-name {} --source-resource-group {} --target-location {} --target-resource-group {} --target-subscription {} --target-name {}

Expected Behavior
Environment Summary
Linux-3.10.0-1127.19.1.el7.x86_64-x86_64-with-centos-7.8.2003-Core
Python 3.6.8
Installer: RPM

azure-cli 2.13.0

Extensions:
log-analytics 0.2.1
image-copy-extension 0.2.7

Additional Context

hi @qwordy could you pls help to have a look? thanks

@dadams0641 Can you show the traceback? Thanks.

Sure!
`Getting OS disk ID of the source VM/image
Creating source snapshot
command failed: ['/usr/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Windows-2019-Crowe-Template-NCUS-PRD1_os_disk_snapshot', '--location', 'northcentralus', '--resource-group', 'IS-CoreSystem-PRD1', '--source', '/subscriptions/<>/resourceGroups/TEMP-image-transfer/providers/Microsoft.Compute/snapshots/Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus', '--output', 'json', '--tags', 'created_by=image-copy-extension']
output: ResourceNotFoundError: Resource Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus is not found.
CLIInternalError: The command failed with an unexpected error. Here is the traceback:
Command '['/usr/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Windows-2019-Crowe-Template-NCUS-PRD1_os_disk_snapshot', '--location', 'northcentralus', '--resource-group', 'IS-CoreSystem-PRD1', '--source', '/subscriptions/<>/resourceGroups/TEMP-image-transfer/providers/Microsoft.Compute/snapshots/Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.
Traceback (most recent call last):
File "/usr/lib64/az/lib/python3.6/site-packages/knack/cli.py", line 215, in invoke
cmd_result = self.invocation.execute(args)
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 654, in execute
raise ex
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 718, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 711, in _run_job
six.reraise(*sys.exc_info())
File "/usr/lib64/az/lib/python3.6/site-packages/six.py", line 703, in reraise
raise value
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 688, in _run_job
result = cmd_copy(params)
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/commands/init.py", line 325, in call
return self.handler(*args, **kwargs)
File "/usr/lib64/az/lib/python3.6/site-packages/azure/cli/core/init.py", line 784, in default_command_handler
return op(**command_args)
File "/home/resadmin/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 105, in imagecopy
run_cli_command(cli_cmd)
File "/home/resadmin/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 45, in run_cli_command
raise ex
File "/home/resadmin/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 21, in run_cli_command
cmd_output = check_output(cmd, stderr=STDOUT, universal_newlines=True)
File "/usr/lib64/python3.6/subprocess.py", line 356, in check_output
**kwargs).stdout
File "/usr/lib64/python3.6/subprocess.py", line 438, in run
output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['/usr/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Windows-2019-Crowe-Template-NCUS-PRD1_os_disk_snapshot', '--location', 'northcentralus', '--resource-group', 'IS-CoreSystem-PRD1', '--source', '/subscriptions/<>/resourceGroups/TEMP-image-transfer/providers/Microsoft.Compute/snapshots/Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.`

It says "Resource Windows-2019-Crowe-Template_os_disk_snapshot-northcentralus is not found". Can you check if it exists?

So, that's what I was thinking at first, maybe I got the name, subscription, or RG incorrect, but so far as I can tell, I got them right.
Here's the command I ran:
az image copy --source-object-name "Windows-2019-Crowe-Template-NCUS-PRD1" --source-resource-group "IS-CoreSystem-PRD1" --target-location "North Central US" --target-resource-group "IS-CoreSystem-QA1" --target-subscription "Crowe QA 1" --target-name "Windows-2019-Crowe-Template-NCUS-QA1" --verbose

With ResourceID:
/subscriptions/<<removed>>/resourceGroups/IS-CoreSystem-PRD1/providers/Microsoft.Compute/images/Windows-2019-Crowe-Template-NCUS-PRD1

I am running into this same issue.  What is likely happening is that the original OS disk that was used to create the image no longer exists.  This error is occurring for me as well because I am using packer to generate the image.  Packer creates a temporary resource group, creates a VM, takes an image, then puts the image in the specified resource group.  Finally, Packer cleans up the temporary VM and resource group.  When the copy image command runs it tries to locate the original OS disk that no longer exists and fails.  Perhaps there would be a way to output a better error message to help people who run into this issue as a short term fix.  For a long term fix it would be nice to not have to require the original OS image, but I realize there are likely some azure constraints to work around.  It would be create if azure had the ability to just clone an image rather than these other games that are being played.

A similar issue https://github.com/Azure/azure-cli-extensions/issues/1756. I am still trying to find the cause.

I am running into this same issue. What is likely happening is that the original OS disk that was used to create the image no longer exists. This error is occurring for me as well because I am using packer to generate the image. Packer creates a temporary resource group, creates a VM, takes an image, then puts the image in the specified resource group. Finally, Packer cleans up the temporary VM and resource group. When the copy image command runs it tries to locate the original OS disk that no longer exists and fails. Perhaps there would be a way to output a better error message to help people who run into this issue as a short term fix. For a long term fix it would be nice to not have to require the original OS disk, but I realize there are likely some azure constraints to work around. It would be great if azure had the ability to just clone an image rather than these other games that are being played.

Totally agree. This extension is full of magic. It is a very long path to copy a image. I suggest Azure service support image copy operation. But unfortunately, they are all devoted to shared image gallery. It is an advanced version of image management. You can have a try. Need some learning time.

I can't reproduce it and find cause so far.

I can't reproduce it and find cause so far.

image-copy extension need to be improved systematically, add feature-request to this one.

This bug was never fixed and it appears that the az image copy command isn't able to function at all - over a year later from these comments. It isn't a feature request - I don't think that is in dispute. It is a feature that exists but doesn't work. az image copy has been broken for years now without being fixed.
PS /home/csp> az image copy --source-resource-group CustomImagesRG `

          --source-object-name Win10MultiV9.5-20220107 `
          --target-resource-group CustomImagesRG `
          --target-location eastus `
          --target-subscription 12345678-1234-1234-1234-605185bb3005 `

Getting OS disk ID of the source VM/image
Creating source snapshot
command failed: ['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Win10MultiV9.5-20220107_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'CustomImagesRG', '--source', '/subscriptions/12345678-1234-1234-1234-605185bb3005/resourceGroups/AVDImageBuildRG/providers/Microsoft.Compute/disks/AVDImageBuild_OsDisk_1_4125c892f47a42c09530358ab9632b88', '--hyper-v-generation', 'V1', '--output', 'json', '--tags', 'created_by=image-copy-extension']
output: ERROR: (NotFound) Resource AVDImageBuild_OsDisk_1_4125c892f47a42c09530358ab9632b88 is not found.
Code: NotFound
Message: Resource AVDImageBuild_OsDisk_1_4125c892f47a42c09530358ab9632b88 is not found.
The command failed with an unexpected error. Here is the traceback:
Command '['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Win10MultiV9.5-20220107_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'CustomImagesRG', '--source', '/subscriptions/12345678-1234-1234-1234-605185bb3005/resourceGroups/AVDImageBuildRG/providers/Microsoft.Compute/disks/AVDImageBuild_OsDisk_1_4125c892f47a42c09530358ab9632b88', '--hyper-v-generation', 'V1', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.
Traceback (most recent call last):
File "/opt/az/lib/python3.10/site-packages/knack/cli.py", line 231, in invoke
cmd_result = self.invocation.execute(args)
File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/init.py", line 663, in execute
raise ex
File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/init.py", line 726, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/init.py", line 697, in _run_job
result = cmd_copy(params)
File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/init.py", line 333, in call
return self.handler(*args, **kwargs)
File "/opt/az/lib/python3.10/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
return op(**command_args)
File "/home/csp/.azure/cliextensions/image-copy-extension/azext_imagecopy/custom.py", line 108, in imagecopy
run_cli_command(cli_cmd)
File "/home/csp/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 45, in run_cli_command
raise ex
File "/home/csp/.azure/cliextensions/image-copy-extension/azext_imagecopy/cli_utils.py", line 21, in run_cli_command
cmd_output = check_output(cmd, stderr=STDOUT, universal_newlines=True)
File "/opt/az/lib/python3.10/subprocess.py", line 420, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/opt/az/lib/python3.10/subprocess.py", line 524, in run
raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['/usr/bin/../../opt/az/bin/python3', '-m', 'azure.cli', 'snapshot', 'create', '--name', 'Win10MultiV9.5-20220107_os_disk_snapshot', '--location', 'eastus', '--resource-group', 'CustomImagesRG', '--source', '/subscriptions/12345678-1234-1234-1234-605185bb3005/resourceGroups/AVDImageBuildRG/providers/Microsoft.Compute/disks/AVDImageBuild_OsDisk_1_4125c892f47a42c09530358ab9632b88', '--hyper-v-generation', 'V1', '--output', 'json', '--tags', 'created_by=image-copy-extension']' returned non-zero exit status 3.


feat: Wrap API Calls in try/catch Blocks in Assets.tsx

Description
The current implementation in Assets.tsx may not consistently handle errors from API calls. To improve error handling and ensure that any issues are properly caught and logged, we should wrap all API calls in try/catch blocks.
Proposed Solution
Refactor the code in Assets.tsx to wrap all API calls in try/catch blocks. This will ensure that any errors are caught and can be handled appropriately, improving the robustness of the application.
Detailed Steps

Identify API Calls:

Locate all instances in Assets.tsx where API calls are made.

Wrap in try/catch:

Wrap each API call in a try/catch block.
Ensure that any errors are logged or handled appropriately within the catch block.

Testing:

Ensure that the refactored code is thoroughly tested.
Write unit tests to verify that errors are properly caught and handled.
Test the application to ensure that it behaves correctly when API calls fail.

Benefits

Improved Error Handling: Wrapping API calls in try/catch blocks ensures that errors are caught and handled, preventing unhandled exceptions.
Better Debugging: Logging errors provides valuable information for debugging and resolving issues.
Robustness: The application becomes more robust and resilient to failures in API calls.

Additional Notes

Consistency: Ensure that all API calls throughout Assets.tsx are consistently wrapped in try/catch blocks.
Logging: Consider using a logging library or service for better error tracking and monitoring.

By wrapping API calls in try/catch blocks in Assets.tsx, we can enhance the error handling capabilities of the application, making it more reliable and easier to debug.

Hi @Arindam200 I love to contribute this issue can you assign this to me

Go Ahead @dhirajudhani

hii @Arindam200 I have created a pr request please review and if there are any changes please let me know


Suggestion -  bootswatch support

Suggest to support bootswatch from https://bootswatch.com in theme.
This could be easily done with switching with the theme name by changing the css or cdn url.
Use-PodeWebTemplates -Title 'Some Title' -Theme cyborg

I probably won't support bootswatch internally, as some themes will require custom logic to work properly, but this could be a good chance to get custom themes setup. Something like:
Use-PodeWebTemplates -Title Example -Theme Custom

Add-PodeWebCustomTheme -Name Cyborg -Url<EMAIL_ADDRESS>Add-PodeWebCustomTheme -Name Lux -Url<EMAIL_ADDRESS>
If there are multiple, the first custom theme is the default.

I probably won't support bootswatch internally, as some themes will require custom logic to work properly, but this could be a good chance to get custom themes setup. Something like:
Use-PodeWebTemplates -Title Example -Theme Custom

Add-PodeWebCustomTheme -Name Cyborg -Url<EMAIL_ADDRESS>Add-PodeWebCustomTheme -Name Lux -Url<EMAIL_ADDRESS>
If there are multiple, the first custom theme is the default.


refactor: unify the error handling methods in the crypto package that are different from the project style

agree


Bump typescript to v5.5.3

LGTM. Let’s just inform the rest about the version bump and check if they have any objections.


feat(infiniteQuery): add possibility to decide which pages should be refetched

refs #1866
missing:

[x] finalize api
[x] tests
[x] docs

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed floor_divide deprecation warnings seen in pytest output

LGTM, thanks!

I can't merge the PR, I don't have write access to this repository.

I can't merge the PR, I don't have write access to this repository.

I know :) Thanks again!

I can't merge the PR, I don't have write access to this repository.

I know :) Thanks again for fixing this!


add pod and node chaos for HA sample clusters

add pod and node chaos for HA sample clusters

https://gitlab.com/gitlab-org/charts/gitlab/-/blob/master/scripts/deploy_chaoskube.sh


ebiten: add KeyF13 to KeyF25

Operating System

[ ] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Many platforms support F13 and more, so why not? :-)
Why is this needed?
Actually I found this when I'm working on #2703. This makes it easy to merge GLFW packages a little.

Oops, Android does not support this...


Add drag-and-drop to SASL webpage

I can't seem to be able to drag and drop anything. I tried with Safari, Chrome, and Firefox. All I see is the convenient checkboxes replaced with a clumsy text field with no sanity checking for typos.

Merged manually to avoid merge commits in https://github.com/znc/znc/pull/912


Typescript

Do you have typescript definitions for the package?

I just committed typed definitions. Please take a look at code (index.d.ts) or update npm to latest version. thank you so much


Uncaught (in promise) TypeError: Cannot assign to read only property 'aroundRadius' of object '#'

🐛 Current behavior
Since the version 7.7.1 of react-instantsearch-core when using Configure a crash and a type error appear when aroundRadius is set at "all"
Stack trace:
merge.js:48 Uncaught (in promise) TypeError: Cannot assign to read only property 'aroundRadius' of object '#'
at _merge (merge.js:48:1)
at merge (merge.js:78:1)
at push../node_modules/algoliasearch-helper/src/SearchParameters/index.js.SearchParameters._parseNumbers (index.js:303:1)
at new SearchParameters (index.js:98:1)
at Object.getRenderState (connectConfigure.js:72:1)
at index.js:414:1
at Array.forEach ()
at Object.render (index.js:412:1)
at index.js:431:1
at Array.forEach ()
at Object.render (index.js:422:1)
at InstantSearch.js:105:1
at defer.js:18:1
The above error occurred in the  component:
at Configure (http://localhost:3000/static/js/0.chunk.js:259274:83)
at Index (http://localhost:3000/static/js/0.chunk.js:259434:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at http://localhost:3000/static/js/0.chunk.js:314906:24
at ScrollView (http://localhost:3000/static/js/0.chunk.js:113683:37)
at ScrollView
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at SearchSuggestions__StyledScrollView
at SearchSuggestions (http://localhost:3000/static/js/main.chunk.js:62877:27)
at InstantSearch (http://localhost:3000/static/js/0.chunk.js:259495:23)
at form
at P (http://localhost:3000/static/js/0.chunk.js:353453:8)
at SearchLanding (http://localhost:3000/static/js/main.chunk.js:66173:55)
at Search (http://localhost:3000/static/js/main.chunk.js:65889:24)
at ErrorBoundary (http://localhost:3000/static/js/0.chunk.js:253671:35)
at withErrorBoundary(Unknown)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at withScreenWrapper__View
at ComponentWithScreenWrapper (http://localhost:3000/static/js/main.chunk.js:44339:54)
at StaticContainer (http://localhost:3000/static/js/0.chunk.js:52912:16)
at EnsureSingleNavigator (http://localhost:3000/static/js/0.chunk.js:52573:23)
at SceneView (http://localhost:3000/static/js/0.chunk.js:52796:21)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at CardSheet (http://localhost:3000/static/js/0.chunk.js:63170:22)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at Dummy (http://localhost:3000/static/js/0.chunk.js:61818:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at Card (http://localhost:3000/static/js/0.chunk.js:62444:171)
at CardContainer (http://localhost:3000/static/js/0.chunk.js:62904:33)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at NativeScreen (http://localhost:3000/static/js/0.chunk.js:310686:171)
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at MaybeScreen (http://localhost:3000/static/js/0.chunk.js:62331:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at MaybeScreenContainer (http://localhost:3000/static/js/0.chunk.js:62321:22)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at Background (http://localhost:3000/static/js/0.chunk.js:56545:20)
at CardStack (http://localhost:3000/static/js/0.chunk.js:63337:171)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at SafeAreaProviderCompat (http://localhost:3000/static/js/0.chunk.js:57581:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StackView (http://localhost:3000/static/js/0.chunk.js:63770:171)
at http://localhost:3000/static/js/0.chunk.js:54341:13
at StackNavigator (http://localhost:3000/static/js/0.chunk.js:61378:17)
at SearchStackNavigator (http://localhost:3000/static/js/main.chunk.js:44376:31)
at ErrorBoundary (http://localhost:3000/static/js/0.chunk.js:253671:35)
at withErrorBoundary(Unknown)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at withScreenWrapper__View
at ComponentWithScreenWrapper (http://localhost:3000/static/js/main.chunk.js:44339:54)
at StaticContainer (http://localhost:3000/static/js/0.chunk.js:52912:16)
at EnsureSingleNavigator (http://localhost:3000/static/js/0.chunk.js:52573:23)
at SceneView (http://localhost:3000/static/js/0.chunk.js:52796:21)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at Background (http://localhost:3000/static/js/0.chunk.js:56545:20)
at Screen (http://localhost:3000/static/js/0.chunk.js:57637:107)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at NativeScreen (http://localhost:3000/static/js/0.chunk.js:310686:171)
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at MaybeScreen (http://localhost:3000/static/js/0.chunk.js:52032:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at MaybeScreenContainer (http://localhost:3000/static/js/0.chunk.js:52021:22)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at SafeAreaProviderCompat (http://localhost:3000/static/js/0.chunk.js:57581:23)
at BottomTabView (http://localhost:3000/static/js/0.chunk.js:51861:29)
at http://localhost:3000/static/js/0.chunk.js:54341:13
at BottomTabNavigator (http://localhost:3000/static/js/0.chunk.js:51003:17)
at TabNavigator
at ErrorBoundary (http://localhost:3000/static/js/0.chunk.js:253671:35)
at withErrorBoundary(Unknown)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at withScreenWrapper__View
at ComponentWithScreenWrapper (http://localhost:3000/static/js/main.chunk.js:44339:54)
at StaticContainer (http://localhost:3000/static/js/0.chunk.js:52912:16)
at EnsureSingleNavigator (http://localhost:3000/static/js/0.chunk.js:52573:23)
at SceneView (http://localhost:3000/static/js/0.chunk.js:52796:21)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at CardSheet (http://localhost:3000/static/js/0.chunk.js:63170:22)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at Dummy (http://localhost:3000/static/js/0.chunk.js:61818:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at Card (http://localhost:3000/static/js/0.chunk.js:62444:171)
at CardContainer (http://localhost:3000/static/js/0.chunk.js:62904:33)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at NativeScreen (http://localhost:3000/static/js/0.chunk.js:310686:171)
at AnimatedComponent (http://localhost:3000/static/js/0.chunk.js:327913:173)
at AnimatedComponentWrapper
at MaybeScreen (http://localhost:3000/static/js/0.chunk.js:62331:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at MaybeScreenContainer (http://localhost:3000/static/js/0.chunk.js:62321:22)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at Background (http://localhost:3000/static/js/0.chunk.js:56545:20)
at CardStack (http://localhost:3000/static/js/0.chunk.js:63337:171)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at SafeAreaProviderCompat (http://localhost:3000/static/js/0.chunk.js:57581:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StackView (http://localhost:3000/static/js/0.chunk.js:63770:171)
at http://localhost:3000/static/js/0.chunk.js:54341:13
at StackNavigator (http://localhost:3000/static/js/0.chunk.js:61378:17)
at http://localhost:3000/static/js/1.chunk.js:1065:31
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at withWebWrapperweb__SiteContainer
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at withWebWrapperweb__SiteWrapper
at ComponentWithWebWrapper
at main
at http://localhost:3000/static/js/0.chunk.js:319476:25
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at RootNavigator__Main
at _TabNavigationStateProvider (http://localhost:3000/static/js/main.chunk.js:44596:57)
at RootNavigator (http://localhost:3000/static/js/1.chunk.js:1075:40)
at EnsureSingleNavigator (http://localhost:3000/static/js/0.chunk.js:52573:23)
at BaseNavigationContainer (http://localhost:3000/static/js/0.chunk.js:52231:27)
at ThemeProvider (http://localhost:3000/static/js/0.chunk.js:58585:20)
at NavigationContainerInner (http://localhost:3000/static/js/0.chunk.js:57999:25)
at AppNavigationContainer (http://localhost:3000/static/js/1.chunk.js:405:37)
at Suspense
at ScreenErrorProvider (http://localhost:3000/static/js/main.chunk.js:19482:23)
at OnboardingWrapper (http://localhost:3000/static/js/main.chunk.js:71398:23)
at SubscriptionContextProvider (http://localhost:3000/static/js/main.chunk.js:30842:23)
at CulturalSurveyContextProvider (http://localhost:3000/static/js/main.chunk.js:18402:23)
at SnackBarProviderComponent (http://localhost:3000/static/js/main.chunk.js:106906:23)
at SearchWrapper (http://localhost:3000/static/js/main.chunk.js:63891:23)
at SearchAnalyticsWrapper (http://localhost:3000/static/js/main.chunk.js:76169:23)
at FavoritesWrapper (http://localhost:3000/static/js/main.chunk.js:20696:23)
at AccessibilityFiltersWrapper (http://localhost:3000/static/js/main.chunk.js:4519:23)
at LocationWrapper (http://localhost:3000/static/js/main.chunk.js:83191:23)
at ErrorBoundary (http://localhost:3000/static/js/0.chunk.js:253671:35)
at AuthWrapper (http://localhost:3000/static/js/main.chunk.js:5687:23)
at GoogleOAuthProvider (http://localhost:3000/static/js/0.chunk.js:64189:23)
at SettingsWrapper (http://localhost:3000/static/js/main.chunk.js:5826:23)
at QueryClientProvider (http://localhost:3000/static/js/0.chunk.js:339008:21)
at ReactQueryClientProvider (http://localhost:3000/static/js/main.chunk.js:85880:23)
at div
at http://localhost:3000/static/js/0.chunk.js:319476:25
at NativeSafeAreaView (http://localhost:3000/static/js/0.chunk.js:310158:23)
at SafeAreaProvider (http://localhost:3000/static/js/0.chunk.js:310320:24)
at StyledNativeComponent (http://localhost:3000/static/js/0.chunk.js:358950:29)
at react-native-save-area-providerweb__SafeAreaProvider
at SupportedBrowsersGate (http://localhost:3000/static/js/main.chunk.js:129078:23)
at ThemeProvider (http://localhost:3000/static/js/0.chunk.js:358925:76)
at ThemeProvider (http://localhost:3000/static/js/main.chunk.js:86895:23)
at Le (http://localhost:3000/static/js/0.chunk.js:353370:67)
at ThemeProvider (http://localhost:3000/static/js/main.chunk.js:86862:23)
at ServiceWorkerProvider (http://localhost:3000/static/js/main.chunk.js:129395:23)
at RemoteConfigProvider (http://localhost:3000/static/js/main.chunk.js:82236:39)
at App (http://localhost:3000/static/js/main.chunk.js:66:24)

React will try to recreate this component tree from scratch using the error boundary you provided, ErrorBoundary.
🔍 Steps to reproduce
clone this repo:  https://github.com/pass-culture/pass-culture-app-native
run: git checkout PC-27546/fix-algolia-search
run:  yarn
run:  yarn start:web:testing
(be carefull and do not share your location)
got to search
click on the search box
This probleme appear in file src/features/search/components/SearchSuggestions/SearchSuggestions.tsx line 103
Live reproduction
/
💭 Expected behavior
not to crash
Package version
"@algolia/client-search": "^4.23.3", "algoliasearch": "^4.23.3", "instantsearch.js": "^4.66.1", "react-instantsearch": "^7.7.1",     "react-instantsearch-core": "^7.7.1", "search-insights": "^2.13.0"
Operating system
macOs 14.4
Browser
arc, chrome
Code of Conduct

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

Is this still  open?

Is this still open?

I opened this ticket 30 mins ago so yes ;)

Can you please assign it to me?

More info for who will debug this: previous version was 7.1.0, for InstantSearch.js that means 4.57.0->4.66.1, helper 3.14.2->3.17.0
I don't yet have an idea what could cause this change in behaviour though, I don't see those parts of the code as being changed in that version range on first glance


Bracketed types don't work in cast expressions

I would like to be able to go (^u8)(thing), but it crashes the compiler.

Parsing wise this "should" work.
Taking a look at it.

The problem occurs as we fail to convert a call expr to a cast at resolve, due to ambiguity between deref expression and pointer types. A fix shouldn't be too bad though.


Some dependency updates that require manual work

Update to rstest v0.18.2, which requires another toolchain bump for a transitive dep. We're now at 1.66 for the older nightly tests.
Update zerocopy v0.7.32, which changed public traits

This obviates #149 and #156


This repo isn't being maintained anymore..

It looks as though t4t5 is no longer active in this repo. I've tried reaching out to him on twitter without a response. In the meantime I have a fork at gb-native-router that has a few improvements made to it.
If you guys have issues or changes you'd like to make, feel free to put them on my repo and I'll take care of it for you. I'll be maintaining my repo until t4t5 comes back, at which point I'll just submit a PR here and merge it in to this one.
~Cheers!

Thanks @MikaelCarpenter

the updated and maintained version is now on npm, so you can install with npm install gb-native-router --save

Thanks @MikaelCarpenter

Thanks @MikaelCarpenter


Update and optimize image

Update LMDB to version 0.9.24
Move all build processes to build image
Install as few dev packages as possible to final image.
All these changes makes final image for about 120MB smaller than before.
Use curl in final image (instead of wget). Curl is smaller.

Note that the target image (target branch) is probably going to be obsolete in the foreseeable future.
See issue #32 for details.


Multiple BC server instances

Hi,
We would like to run multiple BC server instances in same container using port sharing.
Right now we are using BC instances with domain Windows authentication - one instance in each container. We need additional BC server instance along with its own web client instance for testing web services with NavUserPassword auth.
The only workaround we have right now is to create a separate container with NavUserPassword auth. But then we would have additional containers and additional ports we need to manage.
I tinkered with this problem a little. Port sharing works in container. External database also works (I reused existing encryption key). The only things I could not get working are: web server instance (for conveniently generating web service addresses) and after restarting container it would show error about not started server instance.
There should be commands for managing server instances inside container - it should do this in organized manner: it should setup port sharing by default, create separate web client instance, add firewall rule if needed, etc.
Something like this:
New-BcContainerServerInstance -containerName $ContainerName -serverInstance $ServerInstance -auth NavUserPassword

# Plus management commands
Set-BcContainerServerInstance -containerName $ContainerName -serverInstance $ServerInstance [-restart] [-stop] [-start] [-configuration @{ ... }]
Remove-BcContainerServerInstance -containerName $ContainerName -serverInstance $ServerInstance

Automatically created variable $ServerInstance should be left as it is - it should point to the default instance.
I think port sharing should be enabled by default - it works with System account without changes in SMSvcHost.exe.config file.
Thanks.

But then we would have additional containers and additional ports we need to manage.

To make managing ports easier you can checkout the traefik support. This will allow you to set up something like https://my.example.com/my_container which gets rid of all the port handling.
If you have an "external" sql server and a reverse proxy. Multiple containers will basicly act like multiple instances with the added bonus that they are far better seperated from each other and this is something you can do today, without the need to wait for any changes @freddydk might or might not do.

Hi, our problem with Traefik was, and I don't know if it is fixed, that it did not properly pass Windows authentication information. When you have multiple users connecting to the same container, web client would correctly recognize the first connecting user, say SOFTERA\USER1, but the second user, say SOFTERA\USER2, would be still be recognized as SOFTERA\USER1.
The reason was that IIS was tracking web session according to the client IP - each client is a separate session. But Traefik has a single client IP, so from IIS perspective it would nearly always use the same session. Windows credentials are only sent once for the first request.
Trafik supports passing so X-Real-IP and X-Forwarded-Host headers but IIS ignored them.
Traefik did work correctly with NavUserPassword authentication, because then each request has user & password pair so each individual request is autenticated property.
And finally Traefik that comes with BcContainerHelper is outdated. There is this reverse proxy from Microsoft that probably should be used as a replacement: https://github.com/microsoft/reverse-proxy. YARP does not seem to have built-in support for docker, but it is programable, so that support can be added.

This is not something I will have time to do.
You might be able to do this with Invoke-ScriptInBcContainer and then normal PowerShell commands - but it is not something I will add to ContainerHelper.

I see. I will try to find solution on my own then.
Thanks.


Clarifying Redirect Support levels

What type of PR is this?
/kind cleanup
/kind documentation
What this PR does / why we need it:
This clarifies support levels in the Redirect filter and documents how overlapping support levels should be interpreted.
Does this PR introduce a user-facing change?:
NONE

/cc @hbagdi

/lgtm


Support use with Composable Pinia stores

Clear and concise description of the problem
From the documentation, this seems to not support usage with Composable Pinia stores.
Suggested solution
Would be great to have documentation on how - or simply the ability - to use this library within Composable Pinia stores.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

What do you mean by "composable pinia store" ? 🤔  Setup stores ? We support them already (with maybe some limitation due to pinia #244 )

Yes, sry...setup stores.  I am trying to use these through Nuxt btw.

And to further explain what I am trying to do: I am using Nuxt and using setup Pinia stores.  I would like them all to be persisted in localStorage (IndexedDb would be better, but doesn't look like this supports that).
Reason I need this is to be able to set a store variable and then use it in a new window.

https://prazdevs.github.io/pinia-plugin-persistedstate/frameworks/nuxt-3.html

@prazdevs This does not really answer the question.
Setup store doesn't export an object, but a function.
Here is an exemple:
export const useCounterStore = defineStore('counter', () => {
  const count = ref(0)
  const name = ref('Eduardo')
  const doubleCount = computed(() => count.value * 2)
  function increment() {
    count.value++
  }

  return { count, name, doubleCount, increment }
});

Is setup syntax still an option in de docs? I cannot seem to find it.


Add Item Properties view

The theia-trace-extension currently has a separate view in the explorer (left side) that populates upon selection in the graphs. For example, selecting an event in the Events Table the details will listed there. This should be supported in this project as well. An additional web-view needs to be implemented for that. Note that the traceviewer-react-components needs to provide the view implementation so that the web-view only need so use it and provide UI controls.

The implementation requires a refactor of the corresponding class of the theia-trace-extension into a Theia indenpendent react component and webview code in vscode to use the newly create react component. A similar exercise was done with the open traces widget. See refactoring code https://github.com/eclipse-cdt-cloud/theia-trace-extension/pull/295 and the update for that in this repo
[3] https://github.com/theia-ide/vscode-trace-extension/pull/20.


txn: add configurations and enable pessimistic-txn by default

Signed-off-by: youjiali1995<EMAIL_ADDRESS>
What have you changed? (mandatory)
Make some hard code configuration to be configurable and enable pessimistic-txn by default.
What are the type of the changes? (mandatory)

Improvement (change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
unit tests
Refer to a related PR or issue link (optional)
https://github.com/tikv/tikv/pull/4698

@zhangjinpeng1987 @AndreMouche @coocood PTAL

LGTM

ping @zhangjinpeng1987


[DEV] Failing to release pokemon on init. Exits with runtime error "Pokemon Bag is full!"

Expected Behavior
Start to release pokemons by config.
Actual Behavior
Starting to catch pokemon having bag full and getting an error.
Steps to Reproduce
Have full pokemon bag initialise bot on top of pokemon.
Other Information
OS: win 10

please open your config file and tweak it with better settings to release pokemon that you dont care to keep.


Pip install git path modified pylint - simplescheduler

Describe your changes :

Worked on fixing the pylint simplescheduler fetching issue.

Type of change :

[x] Bug fix

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have performed a self-review of my own.
[x] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@sureshms

+1


Error when using the local/localhome storage driver

Describe the bug
When using the "local" or "localhome" storage driver, an error occurs and logged in users can't upload files.
The following error occurs a few times in the log:
ERR failed to list storage spaces error="error: not supported: list storage spaces" filters=[{"Term":{"SpaceType":"project"},"type":4},{"Term":{"User":{"idp":"https://...","opaque_id":"26996839-94fc-4158-a7b6-fb00e1d20f92"}},"type":6}] line=github.com/cs3org/reva/v2@v2.13.2/internal/grpc/services/storageprovider/storageprovider.go:554 pkg=rgrpc service=storage-users status={"code":14,"message":"not implemented","trace":"00000000000000000000000000000000"} traceid=00000000000000000000000000000000
Steps to reproduce
Steps to reproduce the behavior:

Configure the environment variable - "STORAGE_USERS_DRIVER=local" or - "STORAGE_USERS_DRIVER=localhome" in the docker-compose.yml file.
Log in using the web UI.

Expected behavior
After logging in, the user should be able to upload files. No error should appear in the log.
Actual behavior
An error is repeated in the logs and the user can't upload files:

Setup
docker-compose.yml:

services:
  app:
    image: owncloud/ocis:3.0.0-rc.3
    container_name: ocis
    environment:
      - "OCIS_URL=https://..."
      - "OCIS_LOG_FILE=/dev/stdout"
      - "OCIS_LOG_PRETTY=true"
      - "OCIS_LOG_LEVEL=info"
      - "OCIS_LOG_COLOR=true"
      - "PROXY_HTTP_ADDR=<IP_ADDRESS>:9200"
      - "PROXY_TLS=false"
      - "OCIS_INSECURE=true"
      - "PROXY_OIDC_INSECURE=true"
      - "PROXY_ENABLE_BASIC_AUTH=false"
      - "IDM_ADMIN_PASSWORD=..."
      - "IDM_CREATE_DEMO_USERS=false"
      - "PROXY_AUTOPROVISION_ACCOUNTS=true"
      - "OCIS_OIDC_ISSUER=https://..."
      - "PROXY_OIDC_REWRITE_WELLKNOWN=true"
      - "WEB_OIDC_CLIENT_ID=ocis-web"
      - "PROXY_OIDC_ACCESS_TOKEN_VERIFY_METHOD=none"
      - "STORAGE_USERS_DRIVER=local"
    ports:
      - "9200:9200"
    volumes:
      - "./persistent/config:/etc/ocis"
      - "./persistent/data:/var/lib/ocis"
    entrypoint: ["/bin/sh"]
    command: ["-c", "ocis init || true; ocis server"]
    user: "1000:1000"
    restart: unless-stopped

Additional context
The use case behind using the local/localhome driver is to have the directories and files stored as-is on the filesystem instead of blobs (decomposed fs).

@MrPixel1 As you can see in our docs https://owncloud.dev/services/storage-users/
the local or localhome drivers are not supported in ocis. They are part of reva and maintained by the CS3 community.


X11 Apps don't work on double hop

If I ssh to a PC with X11 forwarding I can run GUI apps.
But if I then laumch dev-c7 I get
X11 connection rejected because of wrong authentication.

This is a bit critical as it means this does not work from home even via nx since you have to ssh from the nxservers to run podman.
The other possibility is that this is completely broken for some reason - I can't test that right now as I am at home.

OK - I now understand the issue.
I broke X11 when I went to a background container
This does not work:
$ podman run --rm --name test -dit -v HOME:$HOME --userns keep-id --net host  -e HOME -e DISPLAY ghcr.io/dls-controls/dev-c7 sleep 100d
$ podman exec -it test bash -l
bash-4.2$ xeyes
X11 connection rejected because of wrong authentication.
Error: Can't open display: localhost:12.0

But this does
$ podman run -v $HOME:$HOME --userns keep-id --net host -it -e HOME -e DISPLAY ghcr.io/dls-controls/dev-c7
$ xeyes

Now I need to work out how to provide the single container solution without breaking X11

This works and should be transferable to the run-dev-c7
podman run -dit --rm --name test  -v $HOME:$HOME --userns keep-id --net host  -e HOME -e DISPLAY ghcr.io/dls-controls/dev-c7 bash
podman exec -it test bash -l

You set the environment variable before for HOST; could that work for X11?

The obvious fix for this is to not change the host name.
I only change it so that the prompt looks different - so we Could change the prompt easily enough instead. (well quite easily - it is set by the bash login and rc files so I'd have to work out how to overcome that.)

Hmm upon further investigation from inside DLS today I now feel that:

X11 just works when launching the container from a local terminal
If I then exec into that container from an SSH session, still all good.
X11 in a container that is launched from an SSH session always gets X11 connection rejected because of wrong authentication.
HOWEVER: the last statement cannot be true because I somehow made it work from home yesterday (Huh?)

UPDATE: the below does work. But dev-c7 with hostname = host does not so there is some other setting in the dev-c7 launcher that X11 does not like.

This works and should be transferable to the run-dev-c7
podman run -dit --rm --name test  -v $HOME:$HOME --userns keep-id --net host  -e HOME -e DISPLAY ghcr.io/dls-controls/dev-c7 bash
podman exec -it test bash -l

I believe this is fixed in #19

removed the setting of the hostname in the container
added docs on how to get your prompt to show you are in the container

UPDATE: the below does work. But dev-c7 with hostname = host does not so there is some other setting in the dev-c7 launcher that X11 does not like.

This works and should be transferable to the run-dev-c7
podman run -dit --rm --name test  -v $HOME:$HOME --userns keep-id --net host  -e HOME -e DISPLAY ghcr.io/dls-controls/dev-c7 bash
podman exec -it test bash -l

Final note: the above was failing because I did not use the fully qualified hostname
run-dev-c7.sh -s pc0116.cs.diamond.ac.uk

Did work - so it is just hostname that is the issue.


[explore][time-series] marker option is not work when rich tooltip option is unselected.

Description
In time series chart, marker option is not work when rich tooltip option is unselected.
Screenshot or Video
https://user-images.githubusercontent.com/81597121/122186275-bd0f6c00-ce42-11eb-8886-448806d93993.mov
Repro steps
1, create time series chart using random_time_series data
Select "random_time_series" dataset
Select "Time Series Chart" as your chart type
In Time, select Time Column "ds" and Time Grain "week"
Select "No Filter" in Time Range
Select count as metrics
run query
2, go to CUSTOMIZE panel and select/unselect 'MARKER'
3, Unselect 'Rich tooltip' and try select/unselect 'MARKER'
Expected results
'MARKER' option should work
Actual results
'MARKER' option only work when 'Rich tooltip' is selected
Environment
(please complete the following information):

superset version: master

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

working on it.

fixed https://github.com/apache-superset/superset-ui/pull/1187. @jinghua-qa cc

Verified fixed


Add isNotNull() and isNull() predicates

This closes #1221

Current coverage is 96.10%

Merging #1234 into master will increase coverage by +0.01% as of e7def98

@@            master   #1234   diff @@
======================================
  Files           78      78
  Stmts         9168    9170     +2
  Branches      1717    1717
  Methods          0       0
======================================
+ Hit           8810    8813     +3
+ Partial         93      92     -1
  Missed         265     265

Review entire Coverage Diff as of e7def98

Powered by Codecov. Updated on successful CI builds.


I wonder if there is a way to 'hook' the Clipboard so if new text is placed there it automatically gets added to your 'database'???

I wonder if there is a way to 'hook' the clipboard so that every time text is added it gets automagically added to your 'database'

Hello. Thanks for your idea. I had thought of something similar to that, where the app would essentially save everything - or some things - you copied to the clipboard, but then some concerns started creeping in, such as how I would handle sensitive data. I'm not yet ready to deal with the privacy and security concerns that would open up, so I'm keeping that feature out for now.


fix: Use sequential and post order for vite artifact deletion

Sometimes writeBundle is executed before the upload task,  need to ensure the order of code execution

No other plugins are considered, currently only for reference

Lgtm! Thank you!

Too fast , Hope there are no bugs 🥲
By the way, it seems that my electron vite project(used vite-plugin-electron) has the same problem( Triggered multiple deletion task ), Looks like it's all solved.
Thanks again


Make the Dark  and light mode  working

What would you like to share?
Note: Please visit to V1.0.0 branch and make that changes in that branch only and push changes into v1.0 0 branch only not in the main branch.
All the communication regarding the designing part will be done in Discord only.
Additional information
No response

Hey  @ vikash-8090-yadav i would like to work on this issue .It would be great if you can assign this issue to me .
Thank you !!..

Great start working on this but please raise the pr asap. Thanks!
@SimpleKumari

@SimpleKumari Are u working on this?

Ya sure sir actually I have an OA and an interview back to back of a
company so I didn't get time to complete it I will do it by tomorrow or a
day after that for sure .
And sorry for your inconvenience.
On Mon, 7 Aug 2023, 22:40 Vikash Yadav, @.***> wrote:

@SimpleKumari https://github.com/SimpleKumari Are u working on this?
—
Reply to this email directly, view it on GitHub
https://github.com/Vikash-8090-Yadav/SaL--dApp/issues/198#issuecomment-1668276520,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYYC4X6JEO4JPVX2WSQ6WSDXUEORFANCNFSM6AAAAAA24M5OWQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Serverless full static mode.

Is your feature request related to a problem? Please describe.
As a user I want the ability to host a full-statically generated site on a CDN and have the ability to turn off the Drupal backend without effecting the generated content.
Describe the solution you'd like

[ ] Prevent requests to backend if site is statically generated and in serverless mode.

Describe alternatives you've considered
N/A
Additional context
The only place that appears to need changes is the DruxtRouter component's middleware. By preventing the middelware query, Druxt will not be able to support redirects.

Currently the Middleware makes a request, which causes an error in a Full static site when the Drupal backend has been disabled.
One benefit of the middleware is the ability to process redirects.
The middleware will only be active when the Wildcard router is enabled.
Adding a Fetch hook to the DruxtRouter component will allow the component to be invoked directly, which will allow for more control over routing.
e.g.,
~/pages/blog/_slug.vue
<template>
  <DruxtRouter />
</template>

Using the component directly will also allow the ability to define the route path to lookup, which is not support by the middleware approach:
<template>
  <DruxtRouter value="/node/1" />
</template>

Or connecting to a model:
<template>
  <DruxtRouter v-model="path" />
</template>


Reconfirm location and jurisdiction preferences

Feature
Ask the candidate to reconfirm their location and jurisdiction preferences in case these have changed since their application was submitted.
Task list
[ ] Form component
[ ] Review component
[ ] Amend guidance text to "Please update this section to take into account any changes since you submitted your application"

Hey team! Please add your planning poker estimate with Zenhub @drieJAC @warrensearle @HalcyonJAC


Updated Spanish translation

Fixed unfortunate translation from "secuestrar" (kidnap) to "suplantar" (supplant, closer to the intended hijack meaning).

Codecov Report

Merging #360 (ce226ed) into master (7679cea) will decrease coverage by 0.45%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            master     #360      +/-   ##
===========================================
- Coverage   100.00%   99.54%   -0.46%
===========================================
  Files           12       12
  Lines          229      221       -8
===========================================
- Hits           229      220       -9
- Misses           0        1       +1

Impacted Files
Coverage Δ

hijack/contrib/admin/admin.py
97.22% <0.00%> (-2.78%)
:arrow_down:

hijack/views.py
100.00% <0.00%> (ø)

hijack/signals.py
100.00% <0.00%> (ø)

hijack/templatetags/hijack.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Thank you, @uxDaniel. Much appreciated.
It's released in 3.1.2 🎉


Timestamp should be of type timestamp

Fixed https://github.com/brad-richardson/osm-pbf-parquet/commit/4f2bd39b212041cf5bcac724200d4da1ef37adee#diff-30a95dd9fa29d65f066feaebc38ad8ac2d8cd37eb1107d7ca8e7ee7519928bb1R90


Update README.md with run instructions

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Other (updates to documentation, formatting, etc.)

Description

Issues addressed

All submissions:

[ ] My code follows the code style of this project.
[ ] Do existing samples that are effected by this change still run?
[ ] I have added samples for new functionality.
[ ] I raise detailed error messages when possible.
[ ] My code does not rely on labels that have the option to be hidden.

Which browsers was this tested on?

[x] Chrome
[ ] Firefox
[ ] IE
[ ] Edge

Thanks @gregdegruy . I've approved these changes. Please merge at your convenience.
Regards,
Tyler

Thanks for the contribution @gregdegruy . I've approved and merged this change.
-Tyler


Add support for exponential back-off on retry

Describe the solution you'd like
AWS ElastiCache documentation suggests an exponential-backoff strategy on retry, which Redisson doesn't currently support. It would be nice to support this, in addition to the current fixed retryInterval. This could either be a fixed option, or the ability to pass in a function to compute the retryInterval based on what retry number it's on. The latter would make the timeout calculations in PublishSubscribeService#subscribe() and RedissonPatternTopic#acquire() a bit harder, but give more flexibility if anyone wanted a more complex approach (e.g. back-off with limit).
This has been mentioned before in #1047 and #4278 but not explicitly requested.
Describe alternatives you've considered
A perhaps simpler approach would be a flag that always computed retryInterval as initialRetryInterval * retryNumber. This would make the documentation simpler, but be less flexible.
Both approaches may have challenges with the secondary use of retryInterval to check if the connect has failed in RedisExecutor#scheduleRetryTimeout, unless the initial connect timeout check was always done after initialRetryInterval (if I'm understanding the goal of that code correctly).

I'm happy to submit a PR for this if we can come up with a design that you're happy with.

Will it be similar to backoff implementation in org.redisson.api.LockOptions ?

It could be -- it really depends upon what interface you want to expose, which is tied to how we mutate the existing interface. As I mentioned under alternative approaches, the least impactful change from an API perspective would be a new flag, like increaseRetryInterval or something like that, which causes subsequent retryIntervals to be computed as initialRetryInterval * retryNumber.  Using an implementation like LockOptions implies that we add both a flag and a maxDelay option, as well as potentially a flag about whether or not to add jitter (which is critical for locks, but may or may not be good on reconnects -- you want to avoid thundering herd with many clients, but with a single client it doesn't matter).

retryInterval setting could be deprecated and replaced with ConstantDelay implementation.
config.retryDelay = new ConstantDelay();

Below is the interface used for retryDelay setting:
public interface DelayStrategy {
  Duration nextDelay(int attempt);

}
I removed RedissonPatternTopic#acquire() since it's not used. As for PublishSubscribeService, a subscriptionTimeout setting needs to be introduced. Do we still need to introduce a flag in this case?

@mrniko Any update on this? Does Redisson supports exponential-backoff strategy on retry?


LFS: Fix SSH authentication for trailing arguments

Fixes #1296
This PR works around an issue where the git-lfs client sends trailing arguments, possibly OIDs, after the SSH authentication command defined in the specs.
Needs testing by @kubatyszko since I can't reproduce the error locally.

The code change seems reasonable, I'll test it tomorrow at work - about 24 hours from now. Thanks for your work on this!

I've verified and confirmed that it now works okay, per #1313

LGTM

LGTM


Explore element availability per station

Can we throw out stations not providing all elements for simplicity?

We'll try to parse all data for now


Brush the log when used s3 plugin start

fluentd td-agent 1.0.2
OS：Linux version 3.14.27-25.47.amzn1.x86_64 (mockbuild@gobi-build-60001) (gcc version 4.8.2 20140120 (Red Hat 4.8.2-16) (GCC) ) #1 SMP Wed Dec 17 18:36:15 UTC 2014.
configuration

 @type tail
 read_from_head true
 path /data/service/async_jump/log/count.log.%Y-%m-%d-%H
 pos_file /var/log/td-agent/count.log.pos
 format json
 tag count.log
 keep_time_key true
 time_format  %Y-%m-%dT%H:%M:%S.%L%Z
 utc 8

 @type s3
 s3_bucket mob-ad
 s3_region us-east-1
 path 3s/async_jump
 flush_interval 60s
 store_as gzip_command
 buffer_chunk_limit 256m
 buffer_type file
 buffer_path /var/log/td-agent/buffer/count.buffer
 format json
 time_key time
 utc  8
 include_time_key  true
 time_format  %Y-%m-%dT%H:%M:%S.%L%Z
 s3_object_key_format %{path}/count/%{time_slice}/<IP_ADDRESS>-count.log.%Y-%m-%d-%H_%{index}.%{file_extension}
 time_slice_format %Y/%m/%d/canada/%H

- - problem when i used "td-agentd -vv" to start fluentd,I find some questions when "starting SSL for mob-ad.s3.amazonaws.com:443..." upon complete it will reading the log files from s3 path.These log files were first uploaded.He didn't start working until he had read the first log files
like this：
opening connection to mob-ad.s3.amazonaws.com:443...
opened
starting SSL for mob-ad.s3.amazonaws.com:443...
SSL established
<- "GET /?········································
-> "HTTP/1.1 200 OK\r\n"
-> "x-amz-id-2: BUR7AqmR4wXf4DJFFXen5r3Z2M+Q0Goi1i4e9JB+kXVxUV5nMrz7tqvcWAzODcf/H/D+YORUA9M=\r\n"
-> "x-amz-request-id: 1DA6873467DE7D02\r\n"
-> "Date: Wed, 18 Apr 2018 09:12:36 GMT\r\n"
-> "x-amz-bucket-region: us-east-1\r\n"
-> "Content-Type: application/xml\r\n"
-> "Transfer-Encoding: chunked\r\n"
-> "Server: AmazonS3\r\n"
-> "\r\n"
-> "2000\r\n"
reading 8192 bytes...
--》················· log information in s3
reading 8192 bytes...
--》····················log information in s3
reading 8192 bytes...
-->··················log information in s3
······························
························

I do not understand your question, please could you write more clearly?
（If it is easier, you could try writing the question in a different language, maybe we can understand.  不太懂你的問題， 可以寫清楚一點嗎？）

No response. And the question is about S3 plugin? closed.


Output alert applayer v4

Tickets
Redmine ticket:

https://redmine.openinfosecfoundation.org/issues/6501
https://redmine.openinfosecfoundation.org/issues/6500
https://redmine.openinfosecfoundation.org/issues/5977

Requires Suricata PR https://github.com/OISF/suricata/pull/9797
https://github.com/OISF/suricata-verify/pull/1465 wihtout unneeded SSH commit, and all checks restricted to target Suricata version 8

Merged in #1488, thanks!


Update minisign public key / incorrect minisign signatures (2.0.24) ?

Goog day
All attempts to verify signatures fault in current(2.0.24) release.
I can assume that either the key with which the signatures were created has changed, or the signatures themselves are incorrect for some reason.

D:\Soft\Sec\dnscrypt-proxy>minisign -Vm dnscrypt-proxy-win64-2.0.24.zip
Signature verification failed
D:\Soft\Sec\dnscrypt-proxy>minisign -Vm dnscrypt-proxy-win64-2.0.23.zip
Signature and comment signature verified
Trusted comment: timestamp:1556487635   file:dnscrypt-proxy-win64-2.0.23.zip

There is also a question: is it possible in any way to get the public key from signature, which was created by that key?
Did not find the answer to this question in minisign docs.
Best regards,

Thanks for reporting this and for having taken the time to check the signatures.
Not sure what happened, but I triggered a new build of the same revision, and looks like the files and signatures match now.
The signature doesn't include include the public key itself, however it contains a short key identifier, that you can also find in the public key file.
The minisign tools currently doesn't have an option to print just that identifier, but that is a trivial addition I can add.

The minisign tools currently doesn't have an option to print just that identifier, but that is a trivial addition I can add.
Thx Frank, it would be great!

Now verifying successful, but I noticed that some signatures are missing:
dnscrypt-proxy-android_arm64-2.0.24.zip
dnscrypt-proxy-freebsd_amd64-2.0.24.tar.gz
dnscrypt-proxy-freebsd_arm-2.0.24.tar.gz
dnscrypt-proxy-freebsd_armv7-2.0.24.tar.gz
dnscrypt-proxy-linux_mips-2.0.24.tar.gz
dnscrypt-proxy-linux_mips64le-2.0.24.tar.gz


Add Shell Completions

These commits add a subcommand takes a shell as an argument and generates shell
completion scripts for that shell to stdout.
i.e.
$ innernet completions bash
  OR
$ innernet-server completions zsh

The supported shells:

Bash
ZSH
Fish
PowerShell
Elvish

This is further used to generate some completion scripts for shells that
have standard locations to look for completion scripts by default (Bash and Fish)
at DEB package time.

Even without putting these into  packaging, a user could enable completions on the fly:
$ source <(innernet completions bash)

ZSH requires the completions be in a specific place under the user's
home dir, so the following would allow ZSH users access to
completions (assuming ~/.zfunc is in their $FPATH or substituting
for an appropriate dir if not):
$ innernet completions zsh >
~/.zfunc/_innernet && exec zsh

TODO

[ ] ACTION: Someone more familiar with the packaging should ensure there isn't anything incorrect or missing, or point me in the right direction
[ ] DISCUSS: The packaging portion could be pulled out entirely if so desired.
[ ] DISCUSS: Another option is to check the scripts into the repo source control, and use them as a starting point that is further customized or tweaked/perfected.

Edit: adds another TODO bullet for discussing checking the completion scripts into source control instead of/in addition to packaging, or using a subcommand

A fellow skydiver :)

Yep! Most people think my profile picture is a motorcycle helmet.

[ Generated Completion files ]

I agree that option 1 is better. It allows you to tweak the scripts if needed. When the completion feature of clap was originally developed it was meant to do exactly this, and function as a starting point. However, it turned out to be good enough that people just use them as is without much real need for tweaking.
I'd still include the completions subcommand, which is nice in that it allows people who install through other means access to completion scripts in a pinch. If worried about polluting the CLI, we can also always just make the completions subcommand hidden so that it doesn't appear in the --help message.

[ Aliases ]

This is depending on the shell. Both ZSH and Fish will transparently handle and give you the correct completions for aliased commands, even when using the alias. i.e. $ inn --<tab> will display the correct options. Bash however, does not. There are hacky workarounds, but this is at the user level, not the application level.  Unfortunately, renaming the completion script doesn't help with Bash, you have to also add a line to bottom of the completion script.
For example, if you alias inn=innernet you could run:
$ echo 'complete -F _innernet -o bashdefault -o default inn' >> /etc/bash_completion.d/innernet.sh

(replacing the /etc/bash_completion.d/innernet.sh with wherever you've saved/sourced your completion script).
There is no reason clap couldn't add that line for you when you've defined aliases on your top level command, but currently it does not as no one has asked for it.
i.e.
#[derive(Debug, StructOpt)]
#[structopt(name = "innernet", about, alias = "inn")]
struct Opts { /* .. */ }

Would at least generate the appropriate line at the end of the Bash script. However, only for the inn alias, if someone used something else you'd be back in the same boat.

Option 1 sounds good! I'm not yet too concerned about the subcommand overload, I think we'll eventually want to re-organize the StructOpt layout or manually write our own usage text at some point as the commands build up, but that's for another day.
As for keeping the completions synced with code changes, I've been adding those types of tasks to the release.sh script, which is what is also responsible for re-generating the manpages at every release (but not every commit).

Rebased on the latest master and adds the generate scripts to the repo.
The only two TBD item I'm still tracking is whether or not to include the Bash and Fish scripts in the DEB/RPM packaging, and I (or someone) still needs to ensure the DEB/RPMs grab/place those scripts correctly.

Looks great, thanks so much!


FCAF3D evaluation result mAP question

Hi ! Thanks for you wonderful work! I have learned a lot from your job，but i have a simple question
For evaluation，you say you report the best metrics accross 5*5 trails，do you mean
1、the best mAP0.25 and mAP0.5 in your paper is come from the same pth and the same test result or
2、select the best mAP0.25 and mAP0.5  respectively from the 25 results，they are not come from one result

Hi @wenchened ,
Your second option is correct.

Thank you very much！


improve ParameterTree handling

TODO:

[ ] general documentation improvements
[ ] make flat dict version accessible to user, as well as methods to flatten and unflatten it? One use-case is that imagine if we export a model for simulink, it will only take array as inputs for the parameters, how does the user know how to flatten a large paraemter tree to flat array?
[ ] there are places where we call apply_and_forward_with_arrays, during jax tracing or casadi code-gen, it will basically set the parameter of the model to the jax tracer or casadi symbolics, which are undesired artifacts, which we should ensure to set them back to the numeric values (otherwise when the user calls the model again, it won't work, but usually in OCP, the user doesn't call the model, only the jit or compiled function calls, so we don't see it)

to ensure the model parameters are again set back to numpy types, we should create a test:

in ocp test
create compiled model calls (or maybe just do this as a model test). use both jax and casadi
after the compilation, ensure that the model parameters are still numpy types

model_params.set_param(...) modifies the parameters in the model where model_param comes from in place (probably already a dangerous behavior?! maybe should force the use of model.set_recursive_params afterwards?)
model_params.replace_subtree(..) does not modify the parameters used in the model in place!


Polling API lacks support for timers and mailboxes...

More thought needs to be put into the polling API.  As an example, the polling interface lacks direct support for timers and mailboxes.  One has to stand on their head to fit these constructs into the polling abstraction.

Please provide more context and examples as to the need for standing on your head.

I'm unsure how to respond to this follow-up.  As I mention in my initial comment, "the polling interface lacks direct support for timers and mailboxes."  This means that one can't directly use k_poll to determine the expiration of a timer, for example.  There are likely many workarounds (i.e. standing on one's head), but one way is to have timer expiration trigger a signal event (which can be polled).
While there may be valid reasons to have excluded timers and mailboxes from being k_poll'ed, those reasons aren't clear and from a high level seem to have been a design oversight.


Emojis too big in gnome-terminal

Describe the bug
When I have emojis on the terminal inside gnome-terminal, these are shown by far too big. Their height is about 6 to 7 lines of characters.
To Reproduce
Steps to reproduce the behavior:

Configure NixOS like shown below
Open gnome-terminal
Press several times the return key
Copy the character “💩” into the console
Press several more times the return key

Expected behavior
I would expect the Emojis being shown at the size of a double width character (i.e. same height as text, double the width of typical western characters.
Screenshots

Additional context
The font “Noto Emoji” is installed in /etc/nixos/configuration.nix like that:
  fonts.fonts = with pkgs; [ noto-fonts noto-fonts-cjk noto-fonts-emoji liberation_ttf fira-code fira-code-symbols mplus-outline-fonts dina-font proggyfonts roboto roboto-mono dejavu_fonts carlito source-code-pro ];
  fonts.fontconfig.ultimate.enable = true;
  fonts.fontconfig.defaultFonts = {
    monospace = ["Noto Sans Mono" "Noto Sans Mono CJK TC Regular"];
    sansSerif = ["Noto Sans" "Noto Sans CJK TC Regular"];
    serif = ["Noto Serif" "Noto Sans CJK TC Regular"];
  };

No other font related settings I am aware of have been done.
My locale is:
LANG=zh_TW.UTF-8
LC_CTYPE="zh_TW.UTF-8"
LC_NUMERIC="zh_TW.UTF-8"
LC_TIME="zh_TW.UTF-8"
LC_COLLATE="zh_TW.UTF-8"
LC_MONETARY="zh_TW.UTF-8"
LC_MESSAGES="zh_TW.UTF-8"
LC_PAPER="zh_TW.UTF-8"
LC_NAME="zh_TW.UTF-8"
LC_ADDRESS="zh_TW.UTF-8"
LC_TELEPHONE="zh_TW.UTF-8"
LC_MEASUREMENT="zh_TW.UTF-8"
LC_IDENTIFICATION="zh_TW.UTF-8"
LC_ALL=

Metadata

system: "x86_64-linux"
host os: Linux 4.19.82, NixOS, 19.09.1172.2d896998dc9 (Loris)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3
channels(root): "nixos-19.09.1172.2d896998dc9, unstable-20.03pre199897.471869c9185"
channels(matthias): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
- gnome3.gnome-terminal
- noto-fonts-emoji
# a list of nixos modules affected by the problem
module:

We've actually dropped fonts.fontconfig.ultimate.enable = true; on master because it's seriously outdated. Can you try disabling it? When we added a default emoji font to NixOS we had to switch to using default fontconfig configuration.

Thanks @worldofpeace
I removed this line and I can confirm, that this fixes the problem.
(I added this setting on 2019-09-15 to fix a problem I had with terminal fonts after switching the channel to nixos-19.09. But the problems I had in September don't exist anymore even with this setting removed. So I agree, that its better to disable fontconfig-ultimate.)


Add cmake configuration for external projects

Add cmake configuration for mlir-hlo so that external projects that want
to depend on it can imports its cmake targets (via -DMHLO_DIR=... for
example).

CC: @stellaraccident

Can you update the doc to refer to find_package?

Can you update the description to refer to find_package?

Done.

Done.

CI failures (at least the one that I can see) is unrelated to this PR.


Get rid of component.io for browser build

Should use webpack (or browserify), which uses NPM instead of an obsoleted, parallel universe packaging system.
Should allow us to get rid of the forks we have, like:

https://github.com/microflo/buffer
https://github.com/microflo/ieee754
https://github.com/microflo/base64-js

Have had some successes with webpack. One thing is that microflo is pulled into https://www.npmjs.com/package/fbp-protocol-client so that should also be fixed. And that is used by fbp-spec and noflo-runtime, which again is used by noflo-ui.

@bergie do we use the component.io version of fbp-protocol-client anywhere now? If not then that unblocks this

@jonnor I don't think we do. Some 3rd parties might but in general component.io is anyway semi-broken by now

Done since 0.3.54