Improved Documentation and Error-Signaling of VF2PostLayout

Summary
Fixes #11089 as discussed with @mtreinish
Details and comments
VF2PostLayout does not return a solution when presented with an already ideal initial layout for a quantum circuit. This PR extends the VF2PostLayout pass by a 'no better solution found' error code for these situations and changes the documentation accordingly.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

6 of 6 (100.0%)  changed or added relevant lines in 1 file are covered.
1980 unchanged lines in 132 files lost coverage.
Overall coverage increased (+0.002%) to 86.9%

Files with Coverage Reduction
New Missed Lines
%

qiskit/algorithms/evolvers/real_evolver.py
1
92.31%

qiskit/algorithms/minimum_eigen_solvers/minimum_eigen_solver.py
1
97.78%

qiskit/algorithms/phase_estimators/phase_estimation_result.py
1
96.36%

qiskit/circuit/classicalregister.py
1
90.48%

qiskit/circuit/quantumregister.py
1
93.33%

qiskit/opflow/converters/converter_base.py
1
91.67%

qiskit/opflow/evolutions/evolution_base.py
1
92.31%

qiskit/opflow/expectations/matrix_expectation.py
1
96.15%

qiskit/opflow/gradients/circuit_qfis/circuit_qfi.py
1
93.33%

qiskit/opflow/gradients/gradient_base.py
1
96.15%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
74044

Relevant Lines:
85206

💛  - Coveralls


Add experiences archetype

Resolves #51 where the archetype was missing

No problem! You're welcome. I updated it to only one item now.

Thanks for your contribution!


[Coverage] Audit uses of getCurrentCounter() (SR-4453)

Fix a crash in swift code coverage when handling ternary expressions in top-level decls. This has already been committed to master.
rdar://problem/31383534

@swift-ci Please smoke test

@swift-ci Please test Linux platform
@swift-ci Please test OS X platform

@swift-ci Please test Linux platform


added trackResize to default options map

see #62

LGTM, hope there are no downsides to this being true . But it seems doubtful.


Training batches for imbalanced datasets

I have an imbalanced data set with around 100 entries of positive class and 4000 entries of negative class. One way to create the training batches would be to take 100 entries of positive and 100 entries of negative class and then allow the code to proceed as normal. However, this has high chances of overfitting.
I think one thing that can be done is to have equal number of entries of positive and negative entries in the training batch.( batch_size/2 positive and negative entries). How can I do this?

If your data is embedded and labels are available, you can use the following repository:
https://github.com/ms-unlimit/Transformer-Based-Machine-Learning-Framework

这是来自QQ邮箱的假期自动回复邮件。您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

This is an automatic reply,confirming that your e-mail was received.Thank you!-------------------------------------liangting


fix(slide-toggle): disabled theme not working and dragging works if disabled

Quick Summary

If checked and disabled theme is now invalid

Dragging while disabled

Improved dragging for UX

> * It seems like as per the new theming feature, view encapsulation turned off and now the checked theming overwrites the disabled theme (too high specificity)
> * If a slide-toggle is disabled, users are still able to drag the thumb (which is invalid)
> * Fix invalid `user-select` property, and now dragging works without clamps.

Notice: We are two times extra checking for isDragging, that's because we want to keep the renderer as abstract as possible

FYI: I'm trying to get some good HammerJS tests into the slide-toggle in the future.

Huh, autoprefixer must have been accidentally omitted from the build change. How is user-select special?

I'm not really sure about it, If I recall I read a few things about it, e.g it does not get prefixed automatically. That's why I assumed that Material 1.x also does that. Might relate to the prefixer settings.

@jelbourn Added the tests for the dragging functionality.
Please notice that I currently use the Gesture Config from the slider package. I would rather move this config to a more generic place in another PR, because this will bloat the Diff.

LGTM aside from some comment wording


feat(reroute-static-obstacle): rerouting static obstacle using a selected point by the user

Description
This PR is the autoware_launch changes that must be merged along with with autoware_universe changes for rerouting a static obstacle.

Related links
Discussion / Idea
Issue
autoware_universe PR

Tests performed
Using planning simulator :

let the reroute static obstacle point publisher plugin appear for ease of use
select a point through the route of the vehicle
if alternative route is available, vehicle should be able to change route and does not go through the selected point.

This video demo shows this testing procedure.

Notes for reviewers

Please follow the same procedure as show in the video above for testing
:exclamation: This PR mush be merged with autoware_universe PR :exclamation:

Interface changes
N.A

Effects on system behavior
Autoware will be able to reroute for road blockage based on input from the user (human driver/operator)

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

I guess this plugin does not need to be on rviz screen by default. Can be added easily in case of need. So what do you say if we close this PR since its only purpose is to add the plugin into rviz config?

As discussed with @mehmetdogru, we do not need to add this plugin to the default rviz plugins in autoware. If needed, the user can add it using the add plugin sign , then under rviz under tier4_planning_rviz_plugin --> RerouteStaticObstaclePointPublish.
As this PR is just adding the the plugin, then we no longer need it. So, I am closing the PR.


file conversion required to add possible disease genes via SGD

more potential  diesase genes, from SGD/ SGDAlliance
I will supply both lists as there is not much overlap between them?
user-34-187:Desktop val$ cut -f4 DiseaseAnnotationDataDump.20190116.tsv |sort |uniq |wc
210
user-34-187:Desktop val$ cut -f2 alliance_SGD_diseases_downloads.txt |sort|uniq|wc
232
catted togetether
user-34-187:Desktop val$ cat tmp1 tmp2 |sort |uniq |wc
419
???
alliance_SGD_diseases_downloads.txt
DiseaseAnnotationDataDump.20190116.txt

[ ]  @kimrutherford  could you take the  SGD gene names (no orf names, unfortunately)

column 4 in DiseaseAnnotationDataDump.20190116.tsv
column 2 in  alliance_SGD_diseases_downloads.txt
convert to  SGD orf names, pull our pombe ortholog names.
We can then easily upload the list and intersect with our disease annotated to see if we can pick up any more.

(no orf names, unfortunately)

What is an orf name?

use implicated_in from this file
alliance_SGD_diseases_downloads.txt

Do you mean only consider the lines that contain "is_implicated_in"?  There are only 9 of those.

convert to SGD orf names, pull our pombe ortholog names.

At the end are you after a file with pombe systematic IDs in one column and DOIDs in the other column?

I mena they use SGD:S000000737 where we use YEL011W
but I guess you have both internally.
All we need here is a list of any   pombe IDs which  have a disease association in this file but not in ours. It's only a double check and it  shoudl probably wait until we have the new disease data hosted.
I don' think there will be any but it's a good  final check...

Leave this for a while. I will have another look at the files and see if there is anything we can use (I can probably do it myself fairly easily)

I mena they use SGD:S000000737 where we use YEL011W
but I guess you have both internally.

We only have the YEL011W identifiers and the gene names.  In this case that's fine because the files you attached have the gene names.  From the gene name we can find the gene in Chado and then find the pombe ortholog.

Leave this for a while.

OK.  Let me know when you'd like me to look at it again.


[addon fold/foldcode] HTML folding shows multiple arrows when opening tag spans multiple lines.

It looks weird and is confusing. It would be more acceptable if it behaved similar to other languages, even tho ideally it would show the fold arrow for the first line of the open tag and show the collapsed content double-ended arrow starting from the end of the open tag.

Open http://codemirror.net/demo/folding.html
For the HTML example, add a line break on the div before the id attribute. Watch closely to see what happened to the fold arrows.
Look at what happens as one plays with the fold arrows in that section.

The same doesn't occur w/ the other languages, from what I could tell.

I just noticed that this commit makes the result worse, actually. Currently the result is that no fold appears at all when there's a line break; I compared the behavior if I revert the change using DevTools to modify the code.

I see. Does attached patch help?

Uh... now it has the originally-filed behavior where two arrows show up, but that's better than the behavior after the first patch.
To be clear, the originally-filed issue is that the following content:

   some content

gets collapsed into the following appearance:
35▾
↔

rather than the following (note that there is no extra arrow at line 35):
35
↔

I don't see two arrows showing up anymore, when I paste the snippet you show. Are you sure you're testing the current code?

I've tried using DevTools to "Empty cache and hard reload" the folding demo page, and it repros. Not when you immediately paste the snippet- but when you click the arrow in the gutter to collapse the item, the two arrows appear. And clicking on either of those two arrows results in expanding only the div element.

Unfortunately this is not fixed as of 5.47.0, which incorporates the above commit by AlexanderPrendota.  Suggest to re-open this issue.
You can reproduce it with the official code folding demo: https://codemirror.net/demo/folding.html
Make any HTML tag multi-line, and the error persists on folding:


Unable to get any command to work either either sol or ar keypairs

Version: 0.1.35
Ubuntu WSL on Windows 10.
Upload Command:
./arloader upload "*.png" --log-dir "status/asset/"
error: The following required arguments were not provided:
--ar-keypair-path <AR_KEYPAIR_PATH>
./arloader upload "*.png" --log-dir "status/asset/" --ar-keypair-path "./arweave-key"
error: Found argument '--ar-keypair-path' which wasn't expected, or isn't valid in this context
Did you mean --sol-keypair-path?
./arloader upload "*.png" --log-dir "status/asset/" --sol-keypair-path "~/.config/solana/id.json" --with-sol
error: The following required arguments were not provided:
--ar-keypair-path <AR_KEYPAIR_PATH>
./arloader upload "*.png" --log-dir "status/asset/" --sol-keypair-path "~/.config/solana/id.json" --with-sol --ar-keypair-path "./arweave-key"
error: Found argument '--ar-keypair-path' which wasn't expected, or isn't valid in this context
Did you mean --sol-keypair-path?
Estimate Command:
./arloader estimate "*.png"
error: The following required arguments were not provided:
--ar-keypair-path <AR_KEYPAIR_PATH>
./arloader estimate "*.png" --ar-keypair-path "./arweave-key"
error: Found argument '--ar-keypair-path' which wasn't expected, or isn't valid in this context

Can you try using full paths - it doesn't like the ~. I will validate for that in a future release. Can you try putting the keypair paths as environment variables? The command line parser may be expecting those as options to precede the subcommands. I will move those out as options so that they can be passed after the subcommands, e.g., estimate, in the next release.

I'm going to close this since it seems like it's a matter of entering the commands in the order the parser expects, but feel free to open back up or add additional/comments questions if you still have trouble getting it going.

Thanks - I got it to work when putting the options in the correct order and using full paths without the ~.
However, I noticed you pushed out 0.1.36, which is meant to add support for ~ paths, but it's not working for me.
./arloader upload "*.png" --ar-keypair-path ./arweave-key --log-dir "status/asset/" --with-sol --sol-keypair-path "/home/user/.config/solana/id.json"
InsufficientSolFunds
Uploaded 0 KB in 0 files in 0 bundle transaction(s). Run arloader update-status --log-dir "status/asset/" to update statuses.
./arloader upload "*.png" --ar-keypair-path ./arweave-key --log-dir "status/asset/" --with-sol --sol-keypair-path "~/.config/solana/id.json"
error: Invalid value for '--sol-keypair-path <SOL_KEYPAIR_PATH>': Keypair path does not exist.


Address and payment page

Need to have a checkout page with address input fields, and payment method input fields.
(optional) Address and payment information is saved, and used for auto fill.

Completed, including info autofill.


Consider sanitizing all known secret values in text when serializing recordings

Currently we rely on known secret variable names or JSONPath to denote secret values in test recordings. For example, anywhere a variable named "MySecret" is used would be sanitized, or - after spending the compute to deserialize a JSON payload (if JSON; XML not supported currently) - a property matching some JSONPath should be sanitized.
But where secret values exist in text, relying only on JSONPath still requires a developer to input this and is prone to mistakes. While JSONPath also solves problems where secrets are output from services and not known at dev time (i.e. we need to keep support), I think it's worth considering something similar to what I did in Search: https://github.com/Azure/azure-sdk-for-net/blob/71054d50e35c7ebf5eee0a185c48dab702f6fb82/sdk/search/Azure.Search.Documents/tests/Utilities/SearchRecordedTestSanitizer.cs#L93-L125
Effectively, using both an as-is copy and JSON-encoded copy of the secret value search all text matching this value when saving a recording and sanitizing it. Because this could be expensive by default, we could still have developers opt into it, but wouldn't need to know all possible JSONPaths up front (and mitigates the case where the service may add some different patterns).

On a related note, @JoshLove-msft also suggested a great idea that could benefit any language by somehow hooking up CredScan to run on recordings using the test proxy.
/cc @pakrym


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_94cyr3sjhv/clone/recipe/meta.yaml:19: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_94cyr3sjhv/clone/recipe/meta.yaml:26: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_94cyr3sjhv/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
Errors were found


Support for multiple 3D files in addition to 4D filed for bold

https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/bids-discussion/btpHFtu9xWo/dProjNAmAwAJ
It seems the discussion converged to use a _part-xxx key/value pair in the filename for data stored in multiple files. Could the same principle be used to allow 3D NIfTI files as well as 4D NIfTI files? This was already mentioned in another thread:
https://groups.google.com/forum/#!msg/bids-discussion/zFkdHeLcuMc/eDORAobIAAAJ
but the suggestion was lost in the discussion.

Original authors: @gllmflndn

I feel it would be better to offer some samples on how to squeeze 3-D files into one 4-D file or offer a BIDS tool to assist with this. 4-D files are very common in the major software packages.

I also prefer retaining 4D files. Also, I would like to note that _part-XX has been adopted for complex-valued data (i.e., mag/phase or real/imag) and the MEG separation functionality has been moved to _split-XX.


Question of tf_mesh_render.

Hi, thank you for your work.
I already read your README, and you said: "I find many issue happens here, so let's make this more clear.
The tf_mesh_render does not return triangle id for each pixel after rasterise, we do this by our self and add these changes as submodule to mgcnet.".
I have one question for that. Did you use another code instead of tf_mesh_render?
If yes, can you explain where's that code for the change ?
Thanks.

Oh, I first fork from tf_mesh_render, then we change the code for "covisiable map".
So, we provide our version of the tf_mesh_render (tf_mesh_render is excellent).

Oh, I first fork from tf_mesh_render, then we change the code for "covisiable map".
So, we provide our version of the tf_mesh_render (tf_mesh_render is excellent).

Thank you for your explanation. I already complete to install tf_mesh_render. However, when I start to run python3 test_image.py, it shows many errors related to tensorflow (build_graph), even I install the same as your environment.
Anyway, thank you.


Limit access to beta Argo

Description
Beta test team wants to restrict deployment access via argo in beta enviornment so that the products don't break the case study with unapproved changes.
Currently only  Luca Löffler @lucaloeffler must be able to setup applications in beta argo. The products should be removed. Luca will act as "release manager" and decide which apps in whcih versions are supposed to be in Beta.

FYI  @lucaloeffler and @hzierer
Beta is outdated see CoP Channel


Support existing Cloudformation Stack

This follows on from the conversation in #3 . I'd like to be clear that I'm not necessarily asking for a new feature set, but I think there's a couple of interesting workflows that tubes doesn't handle that it potentially could:

I setup some infrastructure by consuming a cloudformation stack provided to my by a third-party for a product that tubes doesn't know or care about.
-1. I am happy with the infrastructure and I have VMs running or something similar that I don't want to be torn down.
I would like to leverage tubes to add a bosh director (or a concourse, or anything else that tubes does support). How can I achieve this goal?

A similar use case would be to take a third-party cloudformation script and add whatever tubes needs to that, to form a composition of third-party plus bosh director or concourse.
I could be persuaded, in both cases, that these use-cases aren't realistic, that why would you want to add a BOSH director or concourse deployment to the same VPC as some existing software that isn't managed by BOSH or deployed via concourse?
I could also be persuaded that this massively increases the complexity of tubes, as it now has to make intelligent decisions about what parts of the existing infrastructure it can piggyback onto, and what parts it can't. In fact this might be essentially impossible without the user explicitly saying things like: "Tubes, you must use VPC x, and re-use subnet Y, but you can't use subnet Z because the security groups don't quite match what you're looking for to deploy BOSH and you must use ELB A but cannot re-use ELB B" etc

My big goal with this project is to minimize the amount of user-facing configuration.  I have the (perhaps naive) belief that a simple, opinionated workflow will meet the needs of a lot of people & teams.  All we really need is test-driven, continuously-integrated plumbing (tubes) between the various tools (cloudformation, route53, bosh-init, bosh) and various sources of truth (bosh.io, github releases).
But I'll readily admit I may have tunnel vision here -- I'm mostly thinking of recent experiences on CF LA & MEGA, and wanting a Concourse server for personal projects.
I'd be very interested to hear about use cases where spinning up a separate VPC is not a good choice and instead one wants BOSH + extras to ride on top of an existing VPC.  I'll probably have follow-up questions from there, aimed at identifying the minimal set of shared config between the existing VPC and the tubes-created infrastructure.

So I've been thinking about this for a few days, and I can't come up with any realistic situation that tubes doesn't solve that I think it should.
At some level I can see a few scenarios like I described above - some third-party provides some of the infrastructure, perhaps I want to add a bosh director to it but I don't control the VPC, etc. But I don't think it's reasonable for tubes to try to solve that, certainly not yet.
I'm going to close this issue. If I have any insights I'll share them in a new issue.


bosh ssh is very slow

bosh ssh is very slow compared to direct ssh access (even with intraDC latencies)
Very unscientific benchmark:
running time bosh ssh cc/0 exit:
real    0m1.921s
real    0m2.855s
real    0m3.844s
real    0m2.820s
real    0m2.841s
real    0m5.849s
running time runsrv cc/0 exit (note: runsrv it's just a custom wrapper around the ssh command)
real    0m0.119s
real    0m0.120s
real    0m0.079s
real    0m0.081s
real    0m0.079s
real    0m0.075s
I agree that bosh ssh does much more than just opening the ssh connection, but the overhead it imposes makes it hard to justify. When doing troubleshooting the latency makes it pretty hard to be effective. Any chance it can be sped up?
BOSH 1.3232.0

@CAFxX Currently there is a roundtrip between CLI and the Director (to authorize SSH access) and between the Director and the Agent (to configure SSH pubkey). It's a bit hard to compete with plain SSH connection since keys are already pre-provisioned vs BOSH configuring the VM with dynamic keys every time you initiate SSH session. There are a few opportunities for optimization at all levels: go cli (upcoming; currently in alpha1) to speed up ruby startup time, cutting out Agent round trip if all keys are already preconfigured, and responding from Director API immediately (instead of from the worker) assuming that keys are preconfigured. With all that we would just end up with a single CLI to Director roundtrip and actual SSH connection. Not sure how you feel about preconfigured keys on all machines?

Here's a random idea (not mine, borrowed from how bitcoin wallets work): pre-provision a pool of single-use keys for each VM. Use one of the keys from the pool when connecting; when the connection ends the key is deleted and a new one generated and added to the pool.

@CAFxX i could see us switching from dynamically generated keys to something more pre-provisioned for the sake of a speedup. this could even help with the case when agent is unresponsive for some reason and you still want to get to the box.

The problem isn't really that you have to generate keys, the problem is that you have to do it every time.
Why not just have BOSH integrate with ssh-agent? That way users can leverage whatever SSH/SCP tooling they prefer and it would allow integrating BOSH with tools like Ansible. Plus SSH and SCP are enormously powerful tools with loads of options and uses that BOSH SSH/SCP will never come close to implementing all of. Also, it would be a waste of time and effort to do that.
Instead BOSH could just focus on sanely/safely handling key and socket creation/termination. You could set TTL for socket/keys through the command and then plug whatever tool you wanted into the socket.

And just as an FYI, most teams that I've spoken with who have created their own releases include steps to either create an SSH user, or change the VCAP password so that they can SSH natively.
This is Shadow Ops, and points to the fact that BOSH ssh/scp instead of being features are detriments. Something entirely different may be called for.

Yup that's we have also done.


don't source control generated contract files

Adding generate-typechain to build script instead so that prettier doesn't complain

PR-Codex overview
This PR removes unnecessary generated contract files and factories, as well as a local prettier config file.
Detailed summary

Deleted multiple generated contract files and factories in src/lib/contracts/generated
Removed a local prettier config file

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

saw you updating generate-typechain script in your PR @Flip-Liquid. I'm removing it from source control completely -- these files shouldn't be here in the first place

nice! thanks for removing these


Replace jcenter with mavenCentral

jcenter has shut down, replacing with mavenCentral
Maven Central is similarly supported by Gradle, so this swap should be sufficient.
It's possible that some packages may not be available on Maven Centeral. So far I have not encountered any errors.

builds passed! merging


Module 1 Exercise 2 [Mohammed Munzamil]

Please "drag & drop" exercise 2 file below.
MM Workshop 4 exercise 2.docx

Please UPDATE THE TITLE of this "Issue" in the title bar above with your name.

Great work on the graphs and observations!

Thank you. It is been a while


Fwd: Missing Palauan word Metekokd

Fwd: Missing Palauan word Metekokd created by<EMAIL_ADDRESS>on 2017-11-23 10:33:09

<EMAIL_ADDRESS>replied,
Begin forwarded message:

<EMAIL_ADDRESS>replied, Metekakl, metekokl...â€™metekakl/metekokl a eianged. Skies cleared up after
the storm.

<EMAIL_ADDRESS>replied, I think this word - metekokd - means persons of prestige, status,
authority, etc.

<EMAIL_ADDRESS>replied, Jelga, I asked for an example sentence and it seems to confirm what you say:Tial ongdibel a mle meseked rar betok el metekoked el ngar bab el chad.The submitter did not attempt a translation.  Perhaps something like "This meeting was crowded with many prestigious people of high status."?


Modal prevents elements outside from being focused

Reproduction link
https://stackblitz.com/edit/angular-bc7gq8?file=src%2Fapp%2Fapp.component.ts
Steps to reproduce

click button to open zorro modal
click button to open custom modal
click input within the custom modal

What is expected?
input can get focused
What is actually happening?
input will never get focused

Environment
Info

ng-zorro-antd
9.3.0

Browser
Chrome latest

the behavior works as expected before 9.1.2

This problem happens in both Modal and Drawer component.
In the release of zorro 9.1.2, the version of angular/cdk bumps to 9.2.4.
But it doesn't seem to be an angular CDK issue as it works well in Angular material.
https://stackblitz.com/edit/angular-wcny5s?file=src%2Fapp%2Fdialog-content-example.ts

Turns out that the change from  FocusTrapFactory to ConfigurableFocusTrapFactory leads to this bug.
Can we change back to FocusTrapFactory for now?
As we can see from https://github.com/angular/components/blob/ba441d44fb8fa725cd2fd9ac2f6574cefc4f4307/src/material/dialog/dialog-container.ts#L32, Angular Material is still using the deprecated FocusTrapFactory in V10


no version of Torch

Describe the bug
Make new conda env with Python=3.8, run pip install -r requirements.txt and get
ERROR: Could not find a version that satisfies the requirement torch==1.9.0+cu102 (from versions: 1.4.0, 1.5.0, 1.5.1, 1.6.0, 1.7.0, 1.7.1, 1.8.0, 1.8.1, 1.9.0, 1.9.1, 1.10.0, 1.10.1, 1.10.2, 1.11.0)
ERROR: No matching distribution found for torch==1.9.0+cu102

Is cu for cuda?  In which case can I run this with another CPU only version of Torch?

Sorry for late response, have you figured it?

No, I have moved on since.


adds synth-midnight-dark colorscheme

This adds a missing color scheme that is available in other base16 template repositories.

Thanks for the PR.


Updated bootstrap-datepicker version to 1.7.0

Coverage remained the same at 100.0% when pulling 0e08f337ea249b22b07950ca1011963cdd235128 on datepicker_1.7.0 into b424427db1c9c94922d2d6c3da058ea05f615471 on master.


fix: example for ResponsiveWrapper class in README

The exmaple for ResponsiveWrapper class in README has two small problems:

It doesn't load the package, which may confuse some new learners.
(context, widget) => ...(child), the params don't match, which also confuse me for a while.

README is quiet important for new users to know about that.
Hope it helps! I am willing to contribute on this project.

Nice catch! This should help make it easier for people to get started!


Remove Go 1.4 darwin hack [phab:D774]

Created by schroederc at 2016-03-14 21:35:59:
The separate arguments map is no longer needed since we've updated to using Go 1.6

zarko accepted this revision at 2016-03-15 17:57:22

schroederc closed via commit 600d3a9dea19a12217733bcfe09e6bbfa6959ef3 at 2016-03-15 18:13:01


Fix MobileNet-V3 bias convert to int32 OverflowError issue

Type of Change
bug fix
API not changed
Description
ILITV-2735: Tensorflow-mobilenetv3, OverflowError: cannot convert float infinity to integer
When strategy trigger 5th time tuning for MobileNet-V3 for test purpose, the freezed min and max value are both 0 of one Conv2D. It caused the scales value become float infinity since the scales value is calculated from the freezed min and max value.
Expected Behavior & Potential Risk
The MobileNet-V3 tuning will not crash.
How has this PR been tested?
Pre-CI and extension test.
Dependency Change?
No.

extension test: https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-mr-extension/4912/


w2ui grid 1.5v multiSelect false scroll grid keyboard down up no effect

Hello,
If i change value multisetect to false in grid,(by example in grid-23.html in demos)
then i can select only one record but only with mouse, if i can do this with keyboard arrows down and up (selecting next row) there is no effect and i scroll inside frame with whole grid.
in 1.4 version i use this function in couple project fine in long datagrids
thanks for your job!

+1 on this.
with multiSelect and reorderRows both set to false, onKeydown property doesn't work, as well as the keyboard navigation mentioned above btw grid rows.
from the code: (w2grid.js)

...

...

...

...
the textarea#grid_{obj.name}_focus won't get focus thus listeners not triggered.
my temp workaround:
tested w "Single or Multiple Selection" Demo
// assume grid name 'grid', as in the case of the demo page
var name = 'grid';
$(w2ui[name].box).mousedown(function(e){$(this).find('#grid_'+ name + '_focus').focus()});

(hope this can save your guys some time :p)
Good Luck!


SDL2 build failure

I'm not sure if I should add more information, please ask.
Host Environment

OS: Arch Linux
Compiler: gcc version 10.2.0
Pacman fully updated, with tslib installed

To Reproduce
Steps to reproduce the behavior:
./vcpkg install sdl2
Failure logs
./vcpkg install sdl2
Computing installation plan...
The following packages will be built and installed:
    sdl2[core]:x64-linux
Detecting compiler hash for triplet x64-linux...
Starting package 1/1: sdl2:x64-linux
Building package sdl2[core]:x64-linux...
Could not locate cached archive: /home/pual/.cache/vcpkg/archives/ec/ec21fb878d7344a5a3e46b23bbd1b5f8bb0738c6.zip
-- Using cached /home/pual/programs/vcpkg/downloads/SDL2-2.0.12.tar.gz
-- Cleaning sources at /home/pual/programs/vcpkg/buildtrees/sdl2/src/SDL2-2-7528f85a66.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/pual/programs/vcpkg/downloads/SDL2-2.0.12.tar.gz
-- Applying patch export-symbols-only-in-shared-build.patch
-- Applying patch enable-winrt-cmake.patch
-- Applying patch disable-hidapi-for-uwp.patch
-- Applying patch fix-space-in-path.patch
-- Applying patch disable-wcslcpy-and-wcslcat-for-windows.patch
-- Applying patch fix-EventToken-header-reference.patch
-- Applying patch 0006-sdl2-Enable-creation-of-pkg-cfg-file-on-windows.patch
-- Applying patch 0007-sdl2-skip-ibus-on-linux.patch
-- Applying patch 0008-fix-macos-metal-test.patch
-- Using source at /home/pual/programs/vcpkg/buildtrees/sdl2/src/SDL2-2-7528f85a66.clean
-- Configuring x64-linux-dbg
-- Configuring x64-linux-rel
-- Building x64-linux-dbg
-- Building x64-linux-rel
-- Fixing pkgconfig file: /home/pual/programs/vcpkg/packages/sdl2_x64-linux/lib/pkgconfig/sdl2.pc
-- CHECK_LIB_ts_RELEASE:CHECK_LIB_ts_RELEASE-NOTFOUND
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:233 (message):
  Library "ts" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:299 (vcpkg_fixup_pkgconfig_check_files)
  ports/sdl2/portfile.cmake:101 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:136 (include)

Error: Building package sdl2:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: sdl2:x64-linux
  Vcpkg version: 2020.11.12-unknownhash

Additionally, attach any relevant sections from the log files above.

@jgilje,Can you look at this question? This looks very similar to the issue you solved before #13551, maybe you will give us some help, thank you

Yeah, it's a similar issue. I just tried on a relatively empty arch vm. It's possible to ignore TSLIB (a touchscreen library for unix platforms it seems) via configuration options, see my proposed patch below.
Also, a sidetrack to this specific issue but related, is that libunwind was detected on my local install, and this is not a configurable option. As the previous conversation in #13551 this should either be simply patched away (like we did in #14275) or added as SYSTEM_LIBRARIES. The former is probably more correct, I can open a separate PR to resolve this if wanted. libunwind appears to only be used by sdl's own unit tests.
diff --git a/ports/sdl2/portfile.cmake b/ports/sdl2/portfile.cmake
index e1c027bf7..10359cbb6 100644
--- a/ports/sdl2/portfile.cmake
+++ b/ports/sdl2/portfile.cmake
@@ -34,6 +34,7 @@ vcpkg_configure_cmake(
     OPTIONS ${FEATURE_OPTIONS}
         -DSDL_STATIC=${SDL_STATIC}
         -DSDL_SHARED=${SDL_SHARED}
+        -DINPUT_TSLIB=OFF
         -DFORCE_STATIC_VCRT=${FORCE_STATIC_VCRT}
         -DLIBC=ON
 )

@jgilje ,Thank you for your suggestions and help, I will try to modify according to the information you provided, thanks again

After reading what @jgilje wrote, I applied their patch to ports/sdl2/portfile.cmake b/ports/sdl2/portfile.cmake
This fixed my issue with ts.
But I got another error, this time with the libunwind.
-- CHECK_LIB_unwind-generic_RELEASE:CHECK_LIB_unwind-generic_RELEASE-NOTFOUND
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:233 (message):
  Library "unwind-generic" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:299 (vcpkg_fixup_pkgconfig_check_files)
  ports/sdl2/portfile.cmake:102 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:136 (include)

Error: Building package sdl2:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: sdl2:x64-linux
  Vcpkg version: 2020.11.12-unknownhash

The only way I could get sdl to build was by replacing the patch file ports/sdl2/0007-sdl2-skip-ibus-on-linux.patch with a modified version:
--- CMakeLists.orig.txt	2021-01-26 22:11:14.047997818 -0500
+++ CMakeLists.txt	2021-01-26 22:11:30.534663451 -0500
@@ -1184,20 +1184,6 @@
         include_directories(${DBUS_INCLUDE_DIRS})
         list(APPEND EXTRA_LIBS ${DBUS_LIBRARIES})
       endif()
-
-      pkg_search_module(IBUS ibus-1.0 ibus)
-      if(IBUS_FOUND)
-        set(HAVE_IBUS_IBUS_H TRUE)
-        include_directories(${IBUS_INCLUDE_DIRS})
-        list(APPEND EXTRA_LIBS ${IBUS_LIBRARIES})
-        add_definitions(-DSDL_USE_IME)
-      endif()
-      if(HAVE_LIBUNWIND_H)
-        # We've already found the header, so REQUIRE the lib to be present
-        pkg_search_module(UNWIND REQUIRED libunwind)
-        pkg_search_module(UNWIND_GENERIC REQUIRED libunwind-generic)
-        list(APPEND EXTRA_LIBS ${UNWIND_LIBRARIES} ${UNWIND_GENERIC_LIBRARIES})
-      endif()
     endif()

     check_include_file("fcitx/frontend.h" HAVE_FCITX_FRONTEND_H)

I don't know if this is appropriate, but it seemed to allow my build to proceed.

After reading what @jgilje wrote, I applied their patch to ports/sdl2/portfile.cmake b/ports/sdl2/portfile.cmake
This fixed my issue with ts.
But I got another error, this time with the libunwind.
-- CHECK_LIB_unwind-generic_RELEASE:CHECK_LIB_unwind-generic_RELEASE-NOTFOUND
CMake Error at scripts/cmake/vcpkg_fixup_pkgconfig.cmake:233 (message):
  Library "unwind-generic" was not found! If it is a system library use the
  SYSTEM_LIBRARIES parameter for the vcpkg_fixup_pkgconfig call! Otherwise,
  correct the *.pc file
Call Stack (most recent call first):
  scripts/cmake/vcpkg_fixup_pkgconfig.cmake:299 (vcpkg_fixup_pkgconfig_check_files)
  ports/sdl2/portfile.cmake:102 (vcpkg_fixup_pkgconfig)
  scripts/ports.cmake:136 (include)

Error: Building package sdl2:x64-linux failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `./vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: sdl2:x64-linux
  Vcpkg version: 2020.11.12-unknownhash

The only way I could get sdl to build was by replacing the patch file ports/sdl2/0007-sdl2-skip-ibus-on-linux.patch with a modified version:
--- CMakeLists.orig.txt	2021-01-26 22:11:14.047997818 -0500
+++ CMakeLists.txt	2021-01-26 22:11:30.534663451 -0500
@@ -1184,20 +1184,6 @@
         include_directories(${DBUS_INCLUDE_DIRS})
         list(APPEND EXTRA_LIBS ${DBUS_LIBRARIES})
       endif()
-
-      pkg_search_module(IBUS ibus-1.0 ibus)
-      if(IBUS_FOUND)
-        set(HAVE_IBUS_IBUS_H TRUE)
-        include_directories(${IBUS_INCLUDE_DIRS})
-        list(APPEND EXTRA_LIBS ${IBUS_LIBRARIES})
-        add_definitions(-DSDL_USE_IME)
-      endif()
-      if(HAVE_LIBUNWIND_H)
-        # We've already found the header, so REQUIRE the lib to be present
-        pkg_search_module(UNWIND REQUIRED libunwind)
-        pkg_search_module(UNWIND_GENERIC REQUIRED libunwind-generic)
-        list(APPEND EXTRA_LIBS ${UNWIND_LIBRARIES} ${UNWIND_GENERIC_LIBRARIES})
-      endif()
     endif()

     check_include_file("fcitx/frontend.h" HAVE_FCITX_FRONTEND_H)

I don't know if this is appropriate, but it seemed to allow my build to proceed.

@Paul-Hubert ,Does this problem still exist?

Thanks for posting this issue. Please reopen this issue if it still be a problem for you.


[feature] question and answer support

https://www.kaggle.com/code/aisuko/question-answering-task-nlp
for question and answer
https://www.kaggle.com/code/aisuko/retrieve-re-rank

We already implement this in PR https://github.com/SkywardAI/chat-backend/pull/151


[MINOR][DOCS] Use SparkSession.builder.getOrCreate() over the deprecated build()

What changes were proposed in this pull request?
This PR replaces SparkSession.builder.build() with SparkSession.builder.getOrCreate()in Spark Connect overview documentation.
Why are the changes needed?
To avoid encouraging users to use deprecated methods.
Does this PR introduce any user-facing change?
Yes, it guides users to use non-deprecated method.
How was this patch tested?
Manually checked via Markdown editor.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master.


Scene name can be saved as extension only: ".tscn"

Godot version
v4.0.beta7.official [0bb1e89fb] and v3.5.1.stable.official [6fed1ffa3]
System information
Windows 10 22H2, OpenGL3, Geforce GTX 550 Ti
Issue description
A scene can be saved without a pre file extension name, leaving only ".tscn".
Steps to reproduce

create a new project
add a 2D scene
save the scene ctrl s
the Save Scene As... window opens, with the pre file extension name selected: [node_2d].tscn
erase the selected text before the dot and extension: backspace
leaving the name as: .tscn
accept the scene name: enter

Expected
If it's by design, then this can be closed.
If the name should have at least one character, then a message should appear, stating the requirement.
Observed
v4.0.beta7.official [0bb1e89fb]
Started with: Godot_v4.0-beta7_win64.exe --rendering-driver opengl3

v3.5.1.stable.official [6fed1ffa3]

Minimal reproduction project
issue, scene name, extension only, v4.0.beta7.zip
issue, scene name, extension only, v3.5.1.stable.zip

The scene saves and loads correctly, so I think it's not really a problem. Although there is inconsistency with New Scene dialog.


Model#destroy() resolves to empty model instead of null.

As noted here. Inconsistent with fetch behaviour.
Original model should be cleared, but promise should resolve null.

The project leadership of Bookshelf recently changed. In an effort to advance the project we close all issues older than one year.
If you think this issue needs to be re-evaluated please post a comment on why this is still important and we will re-open it.
We also started an open discussion about the future of Bookshelf.js here https://github.com/bookshelf/bookshelf/issues/1600. Feel free to drop by and give us your opinion.
Let's make Bookshelf great again


Property 'getInitialProps' does not exist on type 'typeof default'.

I'm trying to convert the example examples/with-react-intl/ to TypeScript, but I bump into the error at line 7 of pages/_document.js:
Property 'getInitialProps' does not exist on type 'typeof default'.

In the last line of<EMAIL_ADDRESS>the class Document is exported as default, thus it is impossible to use module argumentation to add a new method getInitialProps() to the Document class.
How to work around this? Any ideas? Thanks.

I have the same issue.

Sounds like the typescript types require getInitialProps instead of making it optional. We don't keep they're community made and contributed to https://github.com/DefinitelyTyped/DefinitelyTyped

@timneutkens Yes, it's related to typescript typings, seems I have to wait for somebody to fix it since I'm not familiar with typings definitions.

Posted a question on Stackoverflow https://stackoverflow.com/questions/48037609/rewrite-the-next-js-example-with-react-intl-in-typescript/


[Bug]: Order of fields in class implementing Iterable

Version
28
Steps to reproduce
Comparing a class that implements Iterable, fails when fields are initiated in different order.
class IterWrap implements Iterable<[string, number]> {
  a: number;
  b: number;

  *[Symbol.iterator](): Iterator<[string, number]> {
    for (const prop of ['a', 'b'] as ('a' | 'b')[]) {
      yield [prop, this[prop]];
    }
  }

  static order1(): IterWrap {
    const it = new IterWrap();
    it.a = 1;
    it.b = 1;
    return it;
  }
  static order2(): IterWrap {
    const it = new IterWrap();
    it.b = 1;
    it.a = 1;
    return it;
  }
}

expect(IterWrap.order1()).toEqual(IterWrap.order1()); // PASS
expect(IterWrap.order2()).toEqual(IterWrap.order2()); // PASS
expect(IterWrap.order1()).toEqual(IterWrap.order2()); // FAIL, should pass

Before version 28 this used to work. I believe the cause is here: https://github.com/facebook/jest/pull/8359
It uses Object.entries, which means it compares this, which fails:
Object.entries(NoIterWrap.order1()) // [ [ 'a', 1 ], [ 'b', 1 ] ]
Object.entries(NoIterWrap.order2()) // [ [ 'b', 1 ], [ 'a', 1 ] ]

Expected behavior
If all the properties are the same, I expect it to pass. This was the behavior before version 28, and this is also the behavior for a class that doesn't implement Iterable.
Actual behavior
Fails. The reason for failing is also not really helpful:
expect(received).toEqual(expected) // deep equality

Expected: {"a": 1, "b": 1}
Received: serializes to the same string

Additional context
No response
Environment
System:
    OS: Linux 5.15 Manjaro Linux
    CPU: (4) x64 Intel(R)
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 8.5.0 - /usr/local/bin/npm
  npmPackages:
    jest: ^28 => 28.1.2

Wanna send a PR fixing it?


idl list should be sorted and highlight outdated idls

idl list is hard to read because the results aren't sorted.
When idl list is run in a project it show show which ones are outdated if installed in a project. It could also show which are installed and which are not when run in a project.

Perhaps alphabetical sorting?

Also, the date should be formatted


MySQL Test Connection abnormal

Affected module
Does it impact the UI, backend or Ingestion Framework?
NO
Describe the bug
A clear and concise description of what the bug is.
Adding "Connection Arguments" or "Connection Options" parameter serverTimezone=UTC An error occurred
To Reproduce
Screenshots or steps to reproduce

Expected behavior
A clear and concise description of what you expected to happen.
Configure the connection after adding parameters
Version:

OS: Linux
Python version:
OpenMetadata version: 1.4.1
OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.

eg.
<IP_ADDRESS>:3306?serverTimezone=UTC&useSSL=false

@lookingGordo refer to the docs clearly we do not recommend adding those params . This is not JDBC connection. Ask questions in support channel at https://slack.open-metadata.org


If %e specifiers are enabled without enabling %f specifiers, then neither are provided.

Enabling option PRINTF_SUPPORT_EXPONENTIAL_SPECIFIERS but not PRINTF_SUPPORT_FLOAT_SPECIFIERS.
Yields neither..
This may occur if an application really only wants the exponent format.

I don't actually see this happening...
/home/eyalroz/src/mine/mpaland-printf/test/test_suite.cpp:849: PASSED:
  CHECK( !strcmp(buffer, "4.895512e+04") )
with expansion:
  true
with messages:
  test::sprintf_ arguments, replicated ( "arg := arg" ):
  ----
  "%e" := "%e"
  48955.125 := 48955.125
  ----
  Resulting buffer contents: "4.895512e+04"

etc.

@mickjc750 : ping.

@eyalroz
I'm not sure how to reproduce your snippet from the test_suit. I originally saw this issue when compiling the .c and .h files alongside the AVR test. But I can reproduce it on a desktop with the following steps :
Edit the printf_config.h.in to the following:
#cmakedefine PRINTF_SUPPORT_DECIMAL_SPECIFIERS       0
#cmakedefine PRINTF_SUPPORT_EXPONENTIAL_SPECIFIERS   1
#cmakedefine PRINTF_SUPPORT_LONG_LONG                1
#cmakedefine PRINTF_ALIAS_STANDARD_FUNCTION_NAMES    1
Edit the autotest.cpp to include the following on line 118 after the usage message:
sprintf_(txt, "%%e = %e %%f = %f", 8E41, 0.97341);
printf("%s\n",txt);
Then:
cmake .
make
sudo make install
cd test
cmake .
cp ../include/printf_config.h .
make
./autotest
The output after the usage message shows:
%e = e %f = f

Well, it seems I have a bug with the CMake generation of printf_config.h. When I fix that, I see what you're seeing.

@mickjc750 : Please try this out.

@eyalroz confirmed fixed.


Add Roslyn Analyzer for Fody plugin

What kind of change does this PR introduce?

Feature. This adds a new Roslyn code analyzer that ensures that the [Reactive] attribute is added only to classes that implement IReactiveObject.
Unfortunately, I do not have the expertise to wrap this up in a nuget package
What is the current behavior?

When a user tries to add the [Reactive] attribute to a property on a class that does not implement IReactiveObject, the weaving silently fails.
What is the new behavior?

When a user tries to add the [Reactive] attribute to a property on a class that does not implement IReactiveObject, the IDE should give a compile time error.
What might this PR break?
None
Please check if the PR fulfills these requirements

[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: arontsang sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Add client certificate and key to service monitor

Added client certificates based on https://github.com/deads2k/openshift-enhancements/blob/master/enhancements/monitoring/client-cert-scraping.md

/hold

/hold until upstream change merges


createSpeciesPatch

the last row and column of the matrix are equal to NA, except for the diagonal. why?

the last row and column of the matrix are equal to NA, except for the diagonal. why?

this was fixed by replacing bind_rows in the assignNetworks function


[test-triage] Timeout - chip_sw_alert_handler_lpg_sleep_mode_alerts

Hierarchy of regression failure
Chip Level
Failure Description
Job chip_earlgrey_asic-sim-vcs_run_default killed due to: Exit reason: User job exceeded runlimit: User job timed out
Steps to Reproduce

GitHub Revision: 24fcee5a4
dvsim invocation command to reproduce the failure, inclusive of build and run seeds:
./util/dvsim/dvsim.py hw/top_earlgrey/dv/chip_sim_cfg.hjson -i chip_sw_alert_handler_lpg_sleep_mode_alerts --build-seed<PHONE_NUMBER> --fixed-seed<PHONE_NUMBER> --waves fsdb

Tests with similar or related failures

[ ] chip_sw_alert_handler_lpg_sleep_mode_alerts

As this is a V2 test - assigning as P1 priority for now. Given that this is a timeout there may be no underlying issue other than increasing the timeout time for this test.

Fails due to a fatal lc_ctrl alert (FSM state changed to invalid), which prevents software from continuing with just an NMI.


Improve README.md

Fixes #1.

@johnthagen can you review new README.md with simple example please?


Support mapping path prefixes for index stores

This change builds on #17665, which added -debug-prefix-map to rewrite source paths embedded in the debug info of outputs. This change uses same flag to also remap source paths embedded in the index store.
The goal of this is just like the purpose of -debug-prefix-map, to produce artifacts (index store) that contain paths that refer to a known source path, not the build system's chosen build location. The end goal is to have indexes provided from a build cache.
I haven't discussed this with anyone on the Swift team, there are a few things to discuss:

First, are there reasons this can't or shouldn't be done
Which paths in the index should be mapped, currently only source file paths are remapped
Are there any cases where there debug info and index store need separate prefix maps?
Is it ok to expand -debug-prefix-map  to include more than debug info? I think "debug" could be taken to mean outputs used for "development"

TODO

[ ] Add tests
[ ] Add equivalent support to clang

Thanks @allevato. It's unknown whether we could use relative paths in the index store Perhaps someone from Apple might answer this. This pull request would only be useful if Xcode does support relative paths (ex MainFilePath). If not, then indexstore unit files provided by a remote cache will need to be rewritten to be used from Xcode. We're exploring this approach now.

I've learned more here, after making a tool to convert index store unit files. It ends up that other paths need to be remapped, not just source paths, including the OutputFile (the .o path). Since remapping source paths isn't enough, this PR would get more complicated and would be specific to just the index. Closing with the plan to use a separate tool to solve this.

For anyone curious, we've open sourced the tool that remaps paths of an index-store: https://github.com/lyft/index-import


Fix drawing ambiguous-width character

Problem summary
Under the following conditions, Vim does not update the cells of unicode ambiguous-width character properly.

On a terminal which treats ambiguous-width as 1 cell
:set ambiwidth=double

Repro steps
Create 1.txt and 2.txt
vim --clean
:e 1.txt
:normal! izzzzzz
:w
:e 2.txt
:exe "normal!" "iab\u03bbef"
:wq

\u03bb is λ (lambda)

1.txt:

2.txt: (ambw=double)

Open 1.txt and 2.txt
vim --clean +'set ambw=double'
:e 1.txt
:e 2.txt

Result

z of 1.txt remains just behind lambda of 2.txt.
Details
When drawing a content of 2.txt, Vim outputs into terminal:
>>>   CSI 1;1H                CUP / move cursor to (row=1, col=1)
>>>  abλ
>>>   CSI 1;5H                CUP / move cursor to (row=1, col=5)
>>>  ef

Vim treats lambda as 2 cells but terminal treats it as 1 cell, so (row=1, col=4) cell is not updated.
Solution

Overwrite the cells by 2-spaces before output ambiguous-width (when ambw=double)

Codecov Report

Merging #2436 into master will increase coverage by 0.02%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #2436      +/-   ##
==========================================
+ Coverage   74.51%   74.54%   +0.02%
==========================================
  Files          92       92
  Lines      132721   132726       +5
  Branches    29112    29113       +1
==========================================
+ Hits        98903    98940      +37
- Misses      33783    33786       +3
+ Partials       35        0      -35

Impacted Files
Coverage Δ

src/screen.c
73.87% <66.66%> (-0.03%)
:arrow_down:

src/os_unix.c
54.23% <0%> (-0.14%)
:arrow_down:

src/message.c
68.67% <0%> (-0.05%)
:arrow_down:

src/channel.c
82.72% <0%> (-0.05%)
:arrow_down:

src/window.c
81.77% <0%> (+0.03%)
:arrow_up:

src/gui.c
47.5% <0%> (+0.05%)
:arrow_up:

src/if_py_both.h
76.59% <0%> (+0.14%)
:arrow_up:

src/if_xcmdsrv.c
84.71% <0%> (+0.17%)
:arrow_up:

src/gui_beval.c
62.66% <0%> (+0.42%)
:arrow_up:

src/libvterm/src/screen.c
53.34% <0%> (+6.69%)
:arrow_up:

Continue to review full report at Codecov.

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


fullWidth prop should be selected by default

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
For all components that have a fullWidth prop, it would be better to keep it checked by default.

For reference, when I drag some form components on the page, this is what it would look like by default:

this is what it would look like by default:

Yes, this doesn't look good. Clearly, some of the components by default shouldn't be full width. @oliviertassinari What do you think?

The default value could depend on the container. For the top of the page, it's weird, but inside a page, it makes sense to me.
Two extra thought:

I would want to be able to resize components at the top-level, rather than their layout to take the full width, this could be another solution.
When dropping component in a nester container, it doesn't have to take all the space available.

If this is because resizing components in their non-full-width state might be unintuitive, we can try to rethink things so that that the containers don't have to take full-width, or so that the components can be resized inside a container without resizing the whole container.


endpoint: Allow traffic in BPF map when transitioning to allow-all

When an endpoint's policy was updated from default-deny to allow-all,
all traffic was dropped during regeneration between the update of L4
policy BPF maps and the installation of the BPF program which allows
all traffic.
Instead, in this case, explicitly allow all labels in the BPF maps.
This policy map will be applied until the BPF program is installed,
and then ignored by that program.
The CIDR policy may still drop some ingress traffic from IP
addresses outside of the cluster. It wouldn't be sufficient to just
add /0 CIDR prefixes into the CIDR BPF map, since the new /0 prefix
length would be ignored by the BPF program until the BPF program is
regenerated and installed.
This will be fixed when CIDR ingress policy enforcement is
reimplemented to use label-based enforcement like CIDR egress
policy enforcement.
Signed-off-by: Romain Lenglet<EMAIL_ADDRESS>

test-me-please

Note that this will not be backported, because it depends on https://github.com/cilium/cilium/pull/3965, which will not be backported.


Filtering is broken

Describe the Bug
Filter expression are not recognized by the CLI, even when using the exact syntax from juju-jockey info.
Here is an example:
$ juju-jockey machine -f ../juju-status.json 'hostname~juju'
Usage: juju-jockey [-h] [-v] [-f FILE] [-H HOSTNAME] [-U USERNAME] [-S USERNAME] [-T SEC] [-J COMMAND] [-c DIR] [-C SEC] [--refresh] OBJECT [EXPRESSION ...]
juju-jockey: error: unrecognized arguments: hostname~juju

I tried several different filter expression syntaxes with no success.
Expected Behavior
Filters should be applied to the list of queries objects and return the result.

It looks like the tests will need to be expanded since this latest commit passed CLI:
https://github.com/LCVcode/jockey/actions/runs/10890835662/attempts/1#summary-30220415501
I'll take a look today

I am going to update the title and original description here to emphasize tests.  This will help us understand the delta between desired and actual behavior.

@johnlettman Because we have a large delta in tests here, I think that taking a TDD-esque approach would be wise.  I propose that we create a whole new branch in the repo to focus on this task.
This is multiple PR's worth of work, which will necessitate having failing tests on the new branch as we expand our testing coverage.  I do not want to defy the CI that we have built on master.
Whatever abstraction we use to create these matrices of tests, it should be simple.  I do not want this project to become over-engineered.
What do you think?

I am running a manual round of functionality testing to understand what needs to be done and what testing we need to implement now.  Below are tables for my work thus far.  In these tests, I am querying a single object type with a single filter.  This process makes me suspect that our negative filters are just not working.  For example, I could not come up with any case in which ^= or ^~ did what I expected.  This may be responsible for some of the fails below, specifically the one in the first note at the bottom.
Row indicates object type and column indicates the object type of the filter.

    While Jockey looks much better today, I think it is in a less useful state.  I think the best resolution here is to re-implement the generator-based getter and conversion functions from commit c583f7.  Here are the two functions that did the actual filtering of units and machines, named filter_units and filter_machines, respectively.  I particularly like this solution, because expanding this functionality to cover querying applications, hostnames, charms, and IPs would be as simple as adding a correspongind filter_* function and enabling it in the CLI.  Also, generators make everything lightweight and fast, with nearly no internal data model.  Personally, I think that is valuable.
@johnlettman Considering you know the code base best at this point, what do you think is the best path forward to move back to this style of filtering?  What will it take to preserve the functionality of your dot notation with this change?
FYI: I am going to try to prioritize getting Jockey into a more refined state so I can do a live demo in about two weeks.

I am going to compare the current object collectors to see if we missed some nuance.  I suspect we are not parsing subordinates properly in some cases.


Edit imported text when linked by another user

After a user clicks a link to a particular intro, when the intro is complete, the default text is show in the edit boxes rather than the previously displayed custom intro text.

Implemented on 4027583fe


Git tag for 0.1 missing?

Hi!
It seems like there is no Git tag for the 0.1 release found on PyPI. is that intended?
Best, Sebastian

Oops. Fixed.

Excellent.


changing persistency mode in C++

Required Info

Camera Model
D435

Language
C++

Hello,
I would like to know how we can change the persistency mode of the temporal filter, within the code in C++?
Thank you and regards,
Emmanuel

@EmmanuelCos hello,
Selecting persistence mode is done via RS2_OPTION_HOLES_FILL, for example here
    ...
    rs2::temporal_filter    temp_filter;
   ...
    int PersistenceMode = [0..];

    // Set new mode
   temp_filter.set_option(RS2_OPTION_HOLES_FILL,  float(PersistenceMode));

  // Get the human-readable description of the selected persistence mode
  auto descr =  temp_filter.get_option_value_description(RS2_OPTION_HOLES_FILL,PersistenceMode)

Check the documentation for the full range of supported values.


create SynchronousExecutionFlow

Forces project to version 3.9 since using typings module is deprecated from that version.
See PEP-585


Rename to something different than Crux

[crux.land](https://crux.land/) exists which might be confused with crux-js.
Proposals:

cory-js: derived from core, e.g. @cory/event-bus
lou-js: acronym for library of utilities, e.g. @lou/event-bus
love-js: acronym for library of versatile essentials, e.g. @love/event-bus

Open for other suggestions.

lou-js looks good to me (or lou'ks good to me).


fix: fixed issue where the service could return no cases and error no…

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/RIUI-1667
Change description
Ensured we capture the edge case when service returns no cases assigned for logged in user, bubble the error back up.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

all tests passed


privileges missing?

Just tried to install via the README steps.
I execute the following and get what looks like repository access permission errors?

$ sudo git clone --recursive git@github.com:alswl/excalidraw-collaboration.git
Cloning into 'excalidraw-collaboration'...
<EMAIL_ADDRESS>Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.

got any idea's whats going on.
thanks
brian

Hi there, it was simple to solve.
Because Github have two way clone a repo, one way is using SSH keys(SSH protocol), and another way is using HTTPS(HTTP protocol). The error message shows you did not set a Git public key to Github.
How to fix it? There are two solutions:

Update your public SSH key to https://github.com/settings/keys
Replace the repo url with HTTPS Style in .gitmodules.


Added the 'graph groupsetting set' command solving #451

Added the 'graph groupsetting set' command solving #451

Coverage remained the same at 100.0% when pulling 72067243fceec932a6cc485fdab8a2c04bed698a on phawrylak:feature/451-graph-groupsetting-set into 493134fd9e98d85598ef61f2dc886d172f7d0fb4 on pnp:dev.

Thank you! I'll review it shortly!

Merged manually


documentation request: how to update properties

most of the documentation covers initialization of various properties like color() - but it does not show how to update them via code.  Currently, the only way for someone to learn how is to look at the examples and trial and error.
Request: include examples for each component - how to update it after it's been initialized
color --> how to update color?
text --> how to update value and change font or size?
outline --> how to add an outline on say hover over an element?
z --> how to change the z dynamically?
If there is some generic knowledge for updating "anything" please document that.

Yeah this is better explained in the docs on master which will be released next week with the new version.
A kinda hidden thing right now is you're supposed to check the return type to see what it offers, e.g. for the text() component you'll click the returned TextComp type

where all fields on this component will be directly assigned to the game object, and you just operate on these fields normally.


Update howto-sspr-windows.md

Onboarded Windows devices, with self-service password reset for users configured, were unable to reset the user's password when the Reset Password was clicked at the log on screen.
The root cause was that the HKEYLOCALMACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Policies\System\DelayedDesktopSwitchTimeout was set to 1 but its default is 0.
Affected OS: Windows 10

@Justinha
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Fix broken link for error handling

The "runtime error" link on http://wren.io/embedding/ was broken, so I updated it to point to http://wren.io/error-handling.html. I checked all the other links in the Embedding guide to make sure they work too.

Thanks!


[feature] option to only have one search result per line

Currently, a result line is shown for each term found on a given buffer line of text. I found this confusing sometimes, especially when trying to direct-edit the searched term across many lines.
This also maps more clearly to "narrowing" the file, as in filtering the lines.
I'd like to be able to only show one result for each line in the buffer, regardless of how many results are.
Example of how this looks today, the last six results are actually just three lines in the buffer.

+1
It's very annoying when you do inline update of a file(using ex-mode for example) and get notification about multiple changes in one line.

First, I think the current behavior is better than the suggested one.
But I understand an annoyance you guys mentioned when making inline(or direct) edit on the items list.
My assumed workflow is to use ctrl-cmd-g to place selections, then edit.
Or if you are vim-mode-plus's user, you can use occurrence feature to edit multiple words.

We are talking about option(turned off by default), and not a permanent change in behavior.
As I said current behavior creates trouble when you use ex-mode(or sed).

Hmmm, I basically don't want to.
Why can't you apply the same operation to multiple same lines?
Could you give me a practical scenario with a sample and ex-mode operation? (I'm no ex-mode user)
And using ctrl-cmd-g or occurrence operator doesn't work for you?


fix: upgrade repo

upgrade vscode settings
drop .jslintrc, analysis.json, polymer.json
switch to es-dev-server
switch to karma testing
upgrade all deps
drop all polymer/wct deps
add lit-html dev dep to avoid open-wc/testing error
drop mocha/chai/wct-istanbul
linting
drop dayDiff and renderDatetime from date utils (never used)
rewrite moment-based helpers with Date/Intl

TODO:

fix fixture-based test
import Intl.RelativeTimeFormat polyfill for iOS

Signed-off-by: Patrik Kullman<EMAIL_ADDRESS>

@nomego I updated the tests and configuration:

finish mixin + fixture test
test on mobile safari via saucelabs
load needed polyfills conditionally in test environment

@nomego ready for review, 100% tests pass on mobile safari.

@nomego ready for review, 100% tests pass on mobile safari.

Well I can't review since I started the PR :)
If you approve I can merge :)


The commits from previous PR have been isolated and additionally, electron and cljs have been slightly bumped

There are some bugs with Electron UI handling. Needs further investigation before merging.

Additionally, Hound issues have been overlooked. Must fix it this time.

Hmmm, maybe fix optimist and request(new issue) on this same PR.

Concerning the Hound issues, check my comments on #2416.

Sure. :beer:

Needs further investigation with yargs. Maybe it needs to use older version.


Project is not propagagted in asysimg.bat script for Windows

This is the same problem as reported in #5 for Linux.
The solution is to propagate arguments, like in https://github.com/petvana/AutoSysimages.jl/commit/30db5643959a368d45473eac035392155f934e96

Before I'll switch to Windows. I'd guess we need to add %* into the first commend, as used in the second one.
https://github.com/petvana/AutoSysimages.jl/blob/c454cb95fcf70f8573edf566a5a4d273c3204737/scripts/windows/asysimg.bat#L2-L3
 for /f "tokens=1-4" %%i in ('julia.exe -e "using AutoSysimages; print(julia_args()); exit()" %*') do set A=%%i %%j %%k %%l
 @"%~dp0\julia.exe" %A% %*


Fix dumb terminal example

Here's a thought - maybe these should all be TESTED in various environments... via something automated!
Nah, that's too crazy.

I run dumb terminal example on win 10, it works.
PS Z:\src\github.com\Nerdmaster\terminal> go run .\example\dumb.go
I'm thinking of a number from 1-10.  Try to guess it!
(Type 'QUIT' at any time to exit)

Guess a number: 22
Please enter a number in the range of 1 to 10, inclusive
Guess a number: 1
1 is WRONG!  Try again!
Guess a number: 2

I honestly don't know how I have survived this long.
I just tested and it works fine.    And ... not noticed the very obvious error message...?
Thanks for the sanity check!


Possibility to have the .doom folder in .config folder

I am cleaning up in my home directory, and would love to have the emacs and doom config files, in the relevant ~/.config folder that most modern linux distros use.
I might be wrong, but this will movement will comform better with the
Describe the feature

Possibility to move ~/.doom/ to ~/.config/doom/

System information
Linux 4.19.114-1-MANJARO

This is already supported.
https://github.com/hlissner/doom-emacs/blob/develop/docs/getting_started.org#configure
Also you can move your .emacs.d if you have Emacs27+

Thanks sorry!


require a magic response for dev reload server to reconnect

Previously, the dev reload logic would reconnect whenever it got a 200 response, which meant with  a dev server than returns 200 for all routes, the dev logic was constantly reconnecting.
With this, it only reconnects if it gets a response starting with a specific string 'fastui-dev-reload',
Also connect to the endpoint defined by rootUrl.
cc @dmontagu @JonasKs.

Awesome! Thanks 🙏


Checkbox validation bubble not displaying correctly

I've created an HTML5 form with custom error messages and validation through webshim, which is mostly working and displaying correctly, however the bubble isn't displaying below the checkboxes and defaults to the top-left of the page. I'm not sure if my CSS is throwing off the bubble, or if it's trying to display under an element I'm not showing.
Demo (the structure's a little unusual because I need to display the submit button under two separate forms): http://jsfiddle.net/6bkv2vfz/.
Any help would be really appreciated!

Issue was caused by setting the system checkbox to display: none. Please close.


obs-studio-plugins.obs-ndi: fix deprecation errors

Deprecated functions void QDateTime::setTimeSpec(Qt::TimeSpec) and void QCheckBox::stateChanged(int) errors disabled.
Things done

Built on platform(s)

[x ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 361864 run on aarch64-linux 1

1 package built:

obs-studio-plugins.obs-ndi


Ignore lib directory of library files created when linking to other codes

Reason.
Each time I compile BISON with Thermochimica, I get a git status that the submodule Thermochimica has untracked files. The are in a lib/ folder created during compiling.
Proposed Solution
Add lib/ to the .gitignore.
Impact
If anyone ever makes a "lib" directory, then they will have to undo this and be more specific about which compiled files to ignore.
Additional context
I'm going to make it on my own fork and open a PR. So wait for that.

I'd assign myself, but I can't. 😃


wire.NewSet does not generate any code in the wire_gen.go file.

Please answer these questions before submitting your issue. Thanks!
What did you do?
I am trying to use wire.
wire command generates some code but not complete code.
all providers with wire.build get generated correctly in generated code.
but function that provides are wire.NewSet (provider set) is not getting generated code.
What did you expect to see?
I expect to see that either wire will give an exception on why wire.NewSet is not getting resolved.
What did you see instead?
No exception. just generated code has 2 sections..
// Injectors
And
// file.go
with function with wire.NewSet
func ProvideClients(configFilePath string) (error, *dsc.Client, *dsc.NCFile) {
initLogger()
wire.NewSet(provideConfig, provideSSHClient, provideCMDExecutorInstance, provideDSCClient, provideNCFileClient)
return nil, nil, nil
}

Am I missing something here, not much info on provider sets code generation online
And also running go generate doesn't really generate any wire code? is it suppose to?

Does this issue reproduce with the latest release (go1.12.5)?
Haven't tried.
System details
go version go1.12.4 darwin/amd64
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/gb186037/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/gb186037/Projects/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
GOROOT/bin/go version: go version go1.12.4 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.12.4
uname -v: Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64
ProductName:	Mac OS X
ProductVersion:	10.12.6
BuildVersion:	16G29
lldb --version: lldb-900.0.45
  Swift-4.0

Welcome to the Go project @GarimaBathla and thanks for opening this!
I don't think this is the correct repository, perhaps you are looking for  https://github.com/google/go-cloud? Kindly paging @zombiezen

https://github.com/google/wire is the correct repo. Routed to google/wire#182.


Add safe_cast

Ensure no overflow in casts between enums and integral types.
Fixes: RIPD-1702

This PR adds safe_cast which is a more restricted form of static_cast.  It only works between two integral types, or between an enumeration and integral type.  The destination type is restricted to be a type that can hold all values of the source.  For example no signed type can be cast to any unsigned type because the destination can't hold negative values from the source.
When the source or destination is an enumeration type, then the analysis takes place on the enumeration's underlying type.
This cast helps prevent errors that might potentially occur due to different platforms using different implementation-defined choices of underlying type for the same enumeration by catching unsafe casts at compile time.
In places where an unsafe cast is unavoidable, static_cast is used instead of safe_cast, which now serves as a visible marker that this is a place in the code where information loss might be possible (and is often prevented by the surrounding code logic, for example with run-time checks).

Jenkins Build Summary
Built from this commit
Built at 20181227 - 16:49:49
Test Results

Build Type
Log
Result
Status

msvc.Debug
console
no test results, t: n/a
FAIL :red_circle:

msvc.Debug, NINJA_BUILD=true
console
no test results, t: n/a
FAIL :red_circle:

gcc.Debug -Dcoverage=ON, TARGET=coverage_report, SKIP_TESTS=true
logfile
no test results, t: 61s
FAIL :red_circle:

docs, TARGET=docs
logfile
1 cases, 0 failed, t: 0m1s
PASS :white_check_mark:

gcc.Release -Dassert=ON, MANUAL_TESTS=true
logfile
no test results, t: 108s
FAIL :red_circle:

msvc.Debug -Dunity=OFF
console
no test results, t: n/a
FAIL :red_circle:

msvc.Release
console
no test results, t: n/a
FAIL :red_circle:

rpm
logfile
no test results, t: n/a
FAIL :red_circle:

clang.Debug
logfile
1144 cases, 0 failed, t: 3m14s
PASS :white_check_mark:

gcc.Debug
logfile
no test results, t: 62s
FAIL :red_circle:

gcc.Debug -Dunity=OFF
logfile
no test results, t: 52s
FAIL :red_circle:

clang.Debug -Dunity=OFF
logfile
1144 cases, 0 failed, t: 10m4s
PASS :white_check_mark:

gcc.Release -Dassert=ON
logfile
no test results, t: 109s
FAIL :red_circle:

clang.Release -Dassert=ON
logfile
1144 cases, 0 failed, t: 5m42s
PASS :white_check_mark:

gcc.Debug -Dstatic=OFF
logfile
no test results, t: 61s
FAIL :red_circle:

gcc.Debug -Dstatic=OFF -DBUILD_SHARED_LIBS=ON
logfile
no test results, t: 61s
FAIL :red_circle:

gcc.Debug, NINJA_BUILD=true
logfile
no test results, t: 47s
FAIL :red_circle:

clang.Debug -Dunity=OFF -Dsan=address, PARALLEL_TESTS=false, DEBUGGER=false
logfile
1144 cases, 0 failed, t: 10m7s
PASS :white_check_mark:

clang.Debug -Dunity=OFF -Dsan=undefined, PARALLEL_TESTS=false
logfile
1144 cases, 0 failed, t: 11m57s
PASS :white_check_mark:

I'm going to need an assist with the links to errors above, or with a copy/paste of the error messages generated for VS (and thanks!).

@howardhinnant I'm working on that now and expect to have it to you soon :tm:.

Just let me know if these changes are not what people want.


Add a standard bash template to managed scripts

This PR adds a standard bash template to the managed scripts repository.

@feichashao can you please give this a look? Thanks!
/assign @feichashao

/lgtm

@feichashao a quick test in stage and turns out managed-scripts does not support libraries (yet).
WIll revert back to the standard bash template.
/hold

@feichashao this is now ready to be re-reviewed. 🙇🏽
/unhold

/lgtm

/approve


NextJS Link tag causes styling bugs with styled components

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

Current Behavior 😯
With a client side transition using the nextjs Link component, the styles in various components have issues such as the transition css property not being applied and border-radius. Its fixed when you do a full reload, and doesnt happen with regular  tags.
Expected Behavior 🤔
The styling looks the same as on a full reload
Steps to Reproduce 🕹
https://github.com/AndyW22/muissrbug
Steps:

yarn install && yarn build && yarn start

click on the Paper component and notice the differences in the styling between the component you just clicked on and the one that is now shown on a different page, they are the same component but have different styling.

This doesnt happen if you use an <a> tag, so the issue is to do with compatibility with the nextjs Link component. This didnt happen with v4.
I followed all the steps to add styled components, the bug might be due to not doing that correctly.
I also added a key to _app.tsx for the component rendering which should re-render the component on a page transition.
Your Environment 🌎
NextJS
Material UI V5
Typescript
Styled Components

We have same issue.... We should have waited with updating to MUI5 till NextJS was better tested x_x

No, only "workaround" is using  tags instead which ofcourse loses all of the Link tag benefits.

Does it work using the Link component provided as part of the example project?

No, only "workaround" is using  tags instead which ofcourse loses all of the Link tag benefits.

@AndyW22 does it work using the Link component provided as part of the example project?

The example doesnt use any styled components, all it does is setup compatibility (by adding the server styled sheet on _document.tsx) but doesnt use it. all the styles are just made by importing mui components into each react component.
I tried adding the CardContainer component from my example into the /about page, and the same issue occurs.
it would be helpful if the example repo provided examples on using styled components rather than just setting it up, it seems like actually using styled components hasnt been tested but rather just adding it to the project and compiling.

I am closing as a duplicate of https://github.com/mui-org/material-ui/issues/29742 I know that this is an older one, but more discussion happened there in the meantime. Please continue the discussion on the other issue.


As a developer I would like to Create test suite for Razor

have a "test" folder that works with git workflows

Testsuite added(?) unsure if done correctly, since it doesnt work the same way tests in the CLI app does.


Gaia ~1.7 Billion Dataset

Test the limits of performance with of the Gaia dataset using motherduck.
It roughly has 1.7 billion rows with 100 columns.
That's a lot of data!
If the performance testing on this goes well, why not get the flashy 10B rows (truly cracked out insane).

I hope my innovation on this tag will catch on

#61 this is blocking this issue from getting done, will get to this

https://huggingface.co/spaces/donnyb/FalconVis


Implement growing spans

Pull request type

Feature

What is the current behavior?

Issue Number: Fixes #100
What is the new behavior?

Does this introduce a breaking change?

N/A
Other information
Shrinking span ranges is not implemented because there is no easy way to select the spanned cells that make up a span cell, and an extra editor in a new window seems to clumsy.
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added or the PR is neither a bug fix nor a feature
[ ] Docs have been reviewed and added / updated if needed or the PR is neither a bug fix nor a feature
[ ] The Unreleased section in CHANGELOG.md has been amended to reflect the changes in this PR if needed
[ ] Build (mvn verify) was run locally and any changes were pushed
[x] The pull request is against the correct branch (master for bug fixes, develop for new functionality)
[ ] Dependencies and citation templates have been updated where necessary or there are no new dependencies

Checklist for maintainers
Releases

Do NOT push the green merge button on GitHub.
Follow the latest version of the developer/maintainer docs for making releases.
Check that license and citation information for dependencies are complete (each dependency has a folder in /THIRD-PARTY/).

Closing as dead end implementation that has diverged too far from mainline.


Issue 17 Add Microshed testing of default template

Fixes #17

Too much history to read through.. I'm going to merge and we can follow-up later.


How to use in swift 2.0?

I have added pod successfully.
Add header file to bridge.
Now, I write below code get error in second line
let phoneUtil = NBPhoneNumberUtil.sharedInstance()
let number:NBPhoneNumber = phoneUtil.parse("123", defaultRegion: "US")
Error: call can throw,but it is not marked with "try" and the error is not handle
could any one help me out?

Updated readme


DOC: add dask-sql and fugue

Closes https://github.com/dask/dask/issues/6592

FYI to the fugue authors as well (@goodwanghan @kvnkho)

Thanks for tagging @jameslamb
@raybellwaves , I appreciate the fugue mention here. May I suggest linking to the fugue-sql page specifically as well?
https://fugue-tutorials.readthedocs.io/en/latest/tutorials/fugue_sql/index.html
Thanks!

Sorry for my late answer and thanks for the mention @jrbourbeau!
To answer your question: honestly, I do not know. I did receive some good feedback after the talk I gave and on Twitter, that people generally think it is a good idea to have the project, but not a lot about the quality of the tool etc. There are probably still a lot of bugs but without some users, it's hard to find them :-)
I would be very happy to have dask-sql mentioned here. I would propose to add the following:
(still under development and not ready for a production use-case)
But maybe that is clear to anyone seeing the version below 1.0.0.

@raybellwaves seems like this just needs a few tweaks and then it's ready to go!

Thanks Ray!


Notify team admins of quota limit

Regarding this issue: https://dimagi.sentry.io/issues/5986776238/?alert_rule_id=14063560&alert_type=issue&notification_uuid=571a9dea-ea23-4dc3-bcf1-f9b426f90a76&project=4505001320316928&referrer=issue_alert-slack
I think team admins should know when this happens. This and normal rate limits. See OpenAI's error codes page. We can use regex to catch specific 429 messages (they seem to be static) and send an email to all users on the team that have super admin or team admin access.

cc @snopoke

https://python.langchain.com/docs/how_to/chat_model_rate_limiting/#:~:text=Langchain comes with a built,of requests per unit time.


routing: relax penalties for channel disabled errors

It was being considered a misbehaviour from the intermediate hop which
penalized that hop, which is a bit harsh considering mobile nodes. We
change it be considered as other channel update types and only penalize
the channel that failed.
Potential fix for https://github.com/lightningnetwork/lnd/issues/5597

I think we'll minimally want to extend the set of resultTestCases (with a new test case), in the accompanying test folder.

Done!

Remove the draft label?

@Roasbeef done. forgot I'd created it as draft 😅

If I'm understanding it correctly

You do!

Just need to update docs/release-notes/release-notes-0.14.0.md to list this PR to get the final CI check to pass.

After looking at the file, I have no idea where I would place this under 😅 Misc maybe?

After looking at the file, I have no idea where I would place this under 😅 Misc maybe?

You can add a new Mission Control header IMO :)

You can add a new Mission Control header IMO :)

Done! Sorry for the delay.


ModuleNotFoundError: No module named 'flaxformer.architectures.longt5'

Installing the flaxformer repo using pip results in the above error when importing flaxformer.architectures.longt5. I believe the issue is caused by the missing __init__.py file in the respective directory.

Encountered with exact the same issue but from the *.gin configs.
Agree, this is likely because of __init__.py absence, which in case of t5x project has been covered as follows:
https://github.com/google-research/t5x/blob/4c107feeefdcd8013118be841cf857f1d1921e77/setup.py#L44-L46


How can i get all datas？

For example,there are money shown on the recyclerview's each item and i want to calculate the total money,but i can not find the api which can get all the datas on the recyclerview .

@David-Su you must be passing some list to the recycler view's adapter which the recycler view then displays, so you can use that list which you pass to the adapter to get the sum of the money.

@David-Su you must be passing some list to the recycler view's adapter which the recycler view then displays, so you can use that list which you pass to the adapter to get the total sum of the money.

Thanks your answer,but you are talking about the common usage. I want to know the usage with paging3, all i got was PagingData, and there was no list.

You shouldn't be getting the data from recyclerview. you should get this information in your data layer as the total money most likely doesn't depend only on those items displayed on the screen, but on all items.


Problem last item list ng-sortable

if it is last on the list, it adds a new item to undefined and does not change the list order with only ng-sortable
vm.barConfig = {
    animation: 150,
    handle: '.my-handle',
    onSort: function (/** ngSortEvent */evt) {
           console.log(evt.models);
    };
};

Chrome Versão 44.0.2392.0 canary
Windows 8.1
and
Chrome Versão 42.0.2311.135 m
Windows 8.1

resolve problem, conflict com show and hide jqlite, change for ng-show and ng-hide and minus onde level element

@brunoksato I am not quite sure if I understood how you resolved that. Can you still remember?


gNOI-4.1 Adding Nokia deviation

Added devations to accomodate for this test passing.
"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."

Pull Request Functional Test Report for #2462 / 767501ea1c56a677170d9be26dc9ea4b1e5d964a
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

gNOI-4.1: Software Upgrade

Cisco 8000E

gNOI-4.1: Software Upgrade

Cisco XRd

gNOI-4.1: Software Upgrade

Juniper cPTX

gNOI-4.1: Software Upgrade

Juniper ncPTX

gNOI-4.1: Software Upgrade

Nokia SR Linux

gNOI-4.1: Software Upgrade

Openconfig Lemming

gNOI-4.1: Software Upgrade

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

gNOI-4.1: Software Upgrade

Cisco 8808

gNOI-4.1: Software Upgrade

Juniper PTX10008

gNOI-4.1: Software Upgrade

Nokia 7250 IXR-10e

gNOI-4.1: Software Upgrade

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 50.168%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1643

Relevant Lines:
3275

💛  - Coveralls

With the changes test failed on Nokia vendor with the below message.
TestOSInstall Failed with “Missing osfile or osver args” message; when run on image “os_version -> 0.0.0-52412”

The error you've listed above suggests that you've not defined the flags required for this test to operate successfully. However, this is not due to this PR, this would be true even in main branch. Reassigning to @NishaSadhasivam-work


The "brush" interaction doesn't work correctly

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Reproduction link
https://codepen.io/mostopalove/pen/jOBZYRE
Steps to reproduce

Select any part of chart
Expected result: Part of the diagram is displayed
Actual result: Nothing is displayed

Environment
Info

g2
4.1.17

System
-

Browser
-

Sorry, due to problems in the program's architecture design, we are unable to effectively fix the issues. But fixed in G2 5.0, you can have a try.


Generator for json definition of nodes

node-gen --labels=rack=12,geo=Raleigh machine1.example.com machine2.example.com
Should create a json representation of a node which can be loaded into the apiserver

@bgrant0607 @mattf
Thoughts?

It is a little annoying that the smallest 'generatable' json is:
$ node-gen host.example.com
{
  "kind": "List",
  "apiVersion": "v1beta3",
  "metadata": {},
  "items": [
    {
      "kind": "Node",
      "apiVersion": "v1beta3",
      "metadata": {
        "name": "host.example.com",
        "creationTimestamp": null
      },
      "spec": {},
      "status": {
        "nodeInfo": {
          "machineID": "",
          "systemUUID": "",
          "bootID": "",
          "kernelVersion": "",
          "osImage": "",
          "containerRuntimeVersion": "",
          "kubeletVersion": "",
          "kubeProxyVersion": ""
        }
      }
    }
  ]
}

makes we wish for omitempty on creationTimestamp, spec, status....

@eparis I wonder if this will be necessary after #6949 merges? It obsoletes the need for kube-register and may solve your use case as well.

Re. omitempty. Agree: https://github.com/GoogleCloudPlatform/kubernetes/issues/2726#issuecomment-67681313

This could be useful, but we'd probably need more documentation about scenarios where it would work.
It is also the case that self-registration may make this moot.

@bgrant0607 I switched to using Codec, just to learn something new. Hopefully I did it correctly.  I'll add a README

I signed it.

I signed it!

awww, the google bot got it in another pr  :-(

closing. auto registration mostly made me not care.


Readme to update after the change to Poetry

After the change "Move from pip to poetry, rename project to finary_uapi" the readme was not updated as still as to start doing Install requirements. pip install -r requirements.txt which does not work.

Working on a CONTRIBUTING.md file. If you don't plan to contribute, you can now find the project on Pypi.  https://pypi.org/project/finary-uapi/


State supplied to action lifecycle hooks is stale

We're running into some confusion with Microcosm v10.0.0-beta3 and I'm unsure if I'm using the library incorrectly or perhaps it's just a symptom of playing with beta software. Strangely, I don't seem to see this behavior in the examples. Thanks in advance for any guidance!
Results:
listLoading received 0 and state is actually 0.
listCompleted received 0 and state is actually a.
listLoading received c and state is actually c.
listCompleted received c and state is actually a.
listLoading received c and state is actually c.
listCompleted received c and state is actually a.

I expected the received and actual values to match.
Code:
import Microcosm from 'microcosm';
import axios from 'axios';

/******************************************************************
 * Action
 * ***************************************************************/

function list () {
	return function (action) {
		action.open();

		let cancelled = false;
		action.on('cancel', () => cancelled = true);

		axios.get('https://jsonplaceholder.typicode.com/posts')
			.then(response => {
				if (!cancelled) action.close(response);
			})
			.catch(response => {
				if (!cancelled) action.reject(response);
			});
	}
}

/******************************************************************
 * Store
 * ***************************************************************/

const Store = {
	getInitialState() {
		return '0';
	},

	register() {
		return {
			[list.open]: this.listLoading,
			[list.done]: this.listCompleted,
			[list.failed]: this.listFailed,
			[list.cancelled]: this.listCancelled
		};
	},

	listLoading(state) {
		console.log(`listLoading received ${state} and state is actually ${app.state.widgets}.`);

		return 'a';
	},

	listCancelled(state) {
		console.log(`listCancelled received ${state} and state is actually ${app.state.widgets}.`);

		return 'b';
	},

	listCompleted(state) {
		console.log(`listCompleted received ${state} and state is actually ${app.state.widgets}.`);

		return 'c';
	},

	listFailed(state) {
		console.log(`listFailed received ${state} and state is actually ${app.state.widgets}.`);

		return 'd';
	}
};

/******************************************************************
 * Bootstrapping
 * ***************************************************************/

class App extends Microcosm {
	constructor() {
		super();

		this.addStore('widgets', Store);
	}
}

const app = new App();

/******************************************************************
 * Run action
 * ***************************************************************/

app.push(list);
setTimeout(() => {
	app.push(list);

	setTimeout(() => {
		app.push(list);
	}, 1000);
}, 1000);

Runnable repo here (npm install && npm start): https://github.com/Torchlite/microcosm-test

Wow. Awesome issue write up! Thank you! I'll dig into this now.

Ah got it (I think). @ltk this is something that we really need to nail down in our docs.
The short answer:
Microcosm actions are kept in a tree data structure that gets reduced into application state every time an action makes a change. When list moves into done or failed, the open state is no longer within that history, so it never occurs.
The long answer:
Store handlers are isolated instructions for processing the given state of an action (loading, cancelled, done, etc...). They won't know about prior states, or have much context other than how they respond to the state an action is in.
When an action is pushed, it gets appended to an internal tree data structure:

(visualized via microcosm-debugger)
As actions change state, Microcosm runs through this tree again, executing store handlers on the new state of the tree. So what's happening here is that the list action is no longer in the open state, moving to done or failed. The store never gets a request to call the loading state, as if it never occurred.
We've found this to be super helpful for optimistic updates, you get clean up for free, and that it totally eliminates race conditions when determining the proper state for numerous outstanding requests (since actions don't have to worry about cleaning up their own mess). But it can be confusing... I would love to figure out the best way to communicate how this all works.
aside: When an action is marked for clean up (via close, reject, or cancel), the final state gets merged into a cache for performance. This is adjustable by passing a { maxHistory: n } option when instantiating Microcosm.

@nhunzaker, thank you for taking the time to explain so thoroughly. I really appreciate it.
Even before I understood, this was the only Flux-like implementation I've found that actually accounts for the many states of asynchronous activities. Now, understanding the tree, I see the real power of what you guys are making here. Mind = blown. Love it!


which detector of MOT17?

hi,thanks for your nice work!
that's my first time to train MOT17, I don't know how to use MOT17, choose which detector to train?
Looking for your response.

should i delete other detector dataset in MOT17?
then run convert_mot_to_coco.py?
thanks!

Hi~
Please follow the provided steps in README.