fix(core): validate amount_to_capture in payment update

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description

If capture method is automatic then amount must be equal to amount_to_capture, if amount_to_capture is sent in payment_create request.
This validation works in payments_create call but doesn't work as expected in payments_update call.
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Manual
Payment create with confirm false

Update payment amount (fails with validation error)

Send correct amount_to_capture field and update is successful

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

Your changes seems to affect other connectors (In CI, it is found to affect Stripe): https://github.com/juspay/hyperswitch/actions/runs/8053593266/job/21996336428?pr=3830.

Also, can you please check the same for other connectors i.e., adyen, bluesnap, cybersource, checkout?


Trix indicator fillna value is wrong

Hello, there is probably an issue with Trix indicator, when used with fillna=True parameter,
it fills the missing value with 100, but I believe it should be 0, because then it is extreme value, that is never reached with this indicator. See image bellow:

Hi @Hokyjack ,
Thanks for your message.
TRIXIndicator and all the indicators that use ema internally have the same effect because the nan values are filled by pd.ewm. So let me study a solution for all the cases.
For the moment, try to work filling with the value that you prefer:
ta.trend.TRIXIndicator(all_symbols_preds_train["NEOUSDT"]["df_eval"]["close"], fillna=False).trix().fillna(0)

Best,
Dario


Supervisor raffle merge backend

Rebasetin tuon mun tekemän set-raffled-supervisors -endpointin Henrin branchista ja sitten rebasetin koko paskan developista. Eli tässä on nyt rafflen kaikki backend muutokset.

Codecov Report

Merging #135 (fa62958) into develop (80dcad8) will decrease coverage by 1.69%.
The diff coverage is 33.43%.

@@             Coverage Diff             @@
##           develop     #135      +/-   ##
===========================================
- Coverage    53.04%   51.34%   -1.70%
===========================================
  Files           86       98      +12
  Lines         3369     3710     +341
===========================================
+ Hits          1787     1905     +118
- Misses        1582     1805     +223

Flag
Coverage Δ

back-end
46.94% <33.43%> (-2.99%)
:arrow_down:

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

Impacted Files
Coverage Δ

back/services/raffle.js
8.60% <8.60%> (ø)

back/middlewares/members.js
21.42% <21.42%> (ø)

back/services/raffleSupervisors.js
26.56% <26.56%> (ø)

back/models/members.js
27.27% <27.27%> (ø)

back/controllers/members.js
28.57% <28.57%> (ø)

back/services/members.js
40.00% <40.00%> (ø)

back/validators/raffleSupervisors.js
42.85% <42.85%> (ø)

back/controllers/raffleSupervisors.js
45.45% <45.45%> (ø)

back/middlewares/raffle.js
50.00% <50.00%> (ø)

back/validators/members.js
50.00% <50.00%> (ø)

... and 10 more

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 80dcad8...fa62958. Read the comment docs.


Browse documents: Change default sort and enable manual sorting

Stories

When I need to quickly access a document I recently added, I can easily find it because the Documents list is automatically sorted with the most recently added documents at the top.
Conversely, if I know a program document is one of the first ones I ever added, I can quickly find it by reverse sorting.

Acceptance criteria
Default sorting of documents

[x] Documents are automatically sorted by date added, in reverse chronological order. I.e. the most recently added document is first.
[x] Regardless of any manual sorting I may do, the next time I visit Browse > Documents, the default sort order is restored.

Manual sorting by Date added

[x] The Date added column can be manually sorted chronologically in either direction.
[x] It's visually obvious that those two columns are manually sortable.
[x] It's visually obvious what the current primary sort is, and in which direction. (A common technique is to display up and down carets next to the column name, like these:

Manual sorting by Document name

[x] The Document column can be manually sorted in alphanumeric order, in either direction.

Testing:

[x] IF results provide enough to paginate, ensure Pagination functions properly.
[x] Manual sort function works for all columns
[x] Results properly reflect information being sorted
[x] Multiple searches need to be completed to confirm functional implementation based on acceptance criteria.

Estimated Test Time w/ Acceptance:
2  hours
Housekeeping

[ ] Unit test/s written wherever applicable
[ ] i18n tags included wherever applicable
[ ] Functionality is accessible to people using screen readers

Deployment Verification

[x] Confirmed “Ready for Deploy” status
[x] Media Verifying:

There's a bit of weirdness with the manual sort by Document. When I click it once, I get the following which doesn't look right from that first document title COATE but when I clicked into the document it looks like maybe the user was using Russian. I dunno.

But then when I click on the Document header again to sort the other direction there's also two weird docs that I wouldn't expect (row 1 and 2).

And then, when I click on the document header multiple times, that document YDP appears multiple times. Each time I click on the document header, the doc YDP creates another duplicate. And for the record, I checked and confirmed there's only one doc with the name YDP.

As discussed with Emily and Sanjuro in Slack: This is not a release blocker, so moved this back into the backlog instead of Failed, to avoid confusion over what needs to be worked on right now.

The COAT thing is definitely using Unicode characters (Cyrillic?) even though they look like normal ASCII characters:
>>> u'СОАТЕ'
u'\u0421\u041e\u0410\u0422\u0415'

That row duplication thing is really wonky though...

QA Verified

[x] Performed sorting on multiple programs, and didn't see any duplicates, or inconsistencies in the sorting. No longer can I reproduce the issue reported by @esinn.

Pre-Deploy Check:

[ ] Non-regressed
[ ] Feature/fix can be found in release branch in staging environment, functioning as expected.

If above is checked, this feature/fix is DEPLOY READY.

Fixed in Hucklepatch


Missing retries is a problem

Add faraday retry middleware

Does https://github.com/lookout/elementary-rpc/pull/34 solve this, @thedebugger?

@ismith Only partially. We can add the retry middleware by default. That should fix it.


Appstudio update test-custom-default-branch-igdd

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mwia-on-pull-request-
creating pipelinerun test-annotations-mwia-on-pull-request- in namespace build-e2e-zysn-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-yvpd-on-pull-request-
creating pipelinerun test-annotations-yvpd-on-pull-request- in namespace build-e2e-zysn-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-kqtd-on-pull-request-
creating pipelinerun test-simple-build-kqtd-on-pull-request- in namespace build-e2e-zysn-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-sorp-on-pull-request-
creating pipelinerun test-simple-build-sorp-on-pull-request- in namespace build-e2e-zysn-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


If a player folds or has folded for a game round, this should be indicated to the other players

closed in: e32ab71c34bca3726e2500695f0847d857ed126d


OpenCV dll won't recognize holds

At some point it appears we messed up the opencv plugin. When building with the code found on master, the dll runs without error but returns zero holds. The DLL from pat's computer (posted in the slack) works but was last built in early november

In commit: https://github.com/patxu/cs98-senior-project/commit/73a0c3aff6ea1e063e8594ee31afd3f16433d961 we change the function name to its current name of classifyImage. Since pat's dll runs when calling this function name we know it must have been built post this commit.

#123


Implementing storage usage doc.

Updating auto-generated docs in the process, including inherited-members for Blob and Bucket so reload and patch show up.
This is an updated version of #761.
@jgeewax I see this as a discussion point for design of ACL stuff. It is labeled don't merge because there is some "broken fourth wall" statements in there and methods that aren't implemented yet.
At the very least it covers #1037 for storage, which should really be addressed.

The new doc looks great.  I replied inline to the "fourth wall" comments.

FYI @tseaver @daspecster this still exists if someone wants to take it over

@dhermes I wouldn't mind after speech is in.

@dhermes I am operating under the assumption that this must be so ridiculously out of date as to be entirely untenable (see: gcloud/). Closing. Re-open if you think I am wrong (and explain why).

You are both wrong and right.
You are wrong because the surface hasn't really changed since this was sent, and the surface (especially regarding ACLs) could use some love.
You are right because storage is 1.0 and we can't really make sweeping changes any longer.


🐛 [Bug report]: there is a "loading..." string that gets visible sometimes

Descrcibe the bug?
On the reactplay.io/plays page, there is a "loading..." string that gets visible sometimes after navigating between reactplay.io/ and reactplay.io/plays pages a few times.
Steps to reproduce the bug?

Go to reactplay.io
Navigate a few times between the root and /plays page
Note the "loading..." under the header

Expected behavior
The "loading..." string should not be shown and the top banner should have been visible
Desktop (Please provide your system information)
OS: Windows 11
Browser: Chrome
Version: 111.0.5563.64
Mobile (Please provide your device information)
No response
Screenshot / Screenshare

Relevant log output
none

Code of Conduct

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

can i work on this issue?

can i work on this issue?

Assigned to you

Can I also try.......

The issue still exists

Hi, I want to contributing to this issue, can anybody please tell me is this issue still persisting

as i can see the issue is still there i would like to work on it

hey if no one is working on this issue, I would like to work on it.

If this issue is still open, I would like to work on this. Can you assign me this?


Kindly Implement CompletionService

As it is done in Infinispan

It has been done already in 3.0.0 branch

If at all blocking queue is used in the implementation, kindly make sure a truly distributed blocking Queue is used in the implementation, otherwise the implementation would be broken


Ability to specify ignored files and folders

Example: https://yamllint.readthedocs.io/en/latest/configuration.html#ignoring-paths

Need this as well.

This feature has been implemented with v1.2.0 thanks to @poppinlp.


Documentation - Clarify upgrade wording

At present, the manual upgrades page states the following:

To upgrade K3s from an older version you can re-run the installation script using the same flags

However this is an unclear statement, namely what is "the same flags" ?
For example, if I originally installed using these flags:

server --secrets-encryption --flannel-backend=wireguard-native --cluster-init --tls-san="foo.example.com"

Am I expected to re-specify those ? Because my reading of "the same flags" would be ... yes, you do.
Or if I installed worker nodes using:

K3S_URL=https://foo.example.com:6443 K3S_TOKEN="foobar" sh -s -

Am I expected to re-specify those ? Because again, my reading of "the same flags" would be ... yes, you do.

Yes, the same flags as you used when originally installing on that node. Is there a different way that "the same" could be read?

Thanks @brandond
Well, the problem then is that your example is confusing.  Because you say "using the same flags, for example" and then your example shows no flags:

curl -sfL https://get.k3s.io | sh -

@udf2457 I think the implication is that the exact same flags should be specified, and the example is left blank to avoid confusion as users can specify any number of flags upon installation. I will move this to the Backlog for further consideration.


cargo update --breaking doesn't update Cargo.lock in some corner-cases

Problem
cargo update --breaking (and cargo metadata, cargo check, etc) fail to resolve a dependency requirement that seems resolvable.
Steps
I tried to create a minimal reproduction example but I failed. So here is the reproduction example (I'm forcing +nightly to avoid polluting your rustup, I don't think the exact version matters):
git clone https://github.com/google/wasefire.git
cd wasefire/crates/cli-tools
git checkout 247c701928e080530d1bae953c8bdb209608a330
cargo +nightly -Z unstable-options update --breaking
cd ../cli
cargo +nightly -Z unstable-options update --breaking

I get the following output:
error: failed to select a version for `syn`.
    ... required by package `thiserror-impl v2.0.3`
    ... which satisfies dependency `thiserror-impl = "=2.0.3"` of package `thiserror v2.0.3`
    ... which satisfies dependency `thiserror = "^2.0.3"` of package `cargo_metadata v0.19.1`
    ... which satisfies dependency `cargo_metadata = "^0.19.1"` of package `wasefire-cli-tools v0.2.1-git (/tmp/wasefire/crates/cli-tools)`
    ... which satisfies path dependency `wasefire-cli-tools` (locked to 0.2.1-git) of package `wasefire-cli v0.2.1-git (/tmp/wasefire/crates/cli)`
versions that meet the requirements `^2.0.87` are: 2.0.90, 2.0.89, 2.0.88, 2.0.87

all possible versions conflict with previously selected packages.

  previously selected package `syn v2.0.66`
    ... which satisfies dependency `syn = "^2.0.8"` (locked to 2.0.66) of package `clap_derive v4.5.4`
    ... which satisfies dependency `clap_derive = "=4.5.4"` (locked to 4.5.4) of package `clap v4.5.4`
    ... which satisfies dependency `clap = "^4.5.4"` (locked to 4.5.4) of package `wasefire-cli v0.2.1-git (/tmp/wasefire/crates/cli)`

failed to select a version for `syn` which could resolve this conflict

I don't understand why cargo doesn't select syn v2.0.90. Is there a way to print debug info during resolution?
Possible Solution(s)
No response
Notes
Similar situations work, so I'm assuming this should always work. But maybe this assumption is wrong and this corner-case is not supported. But I'd like to understand what characterizes this corner-case then.
Version
cargo 1.84.0-nightly (e75214ea4 2024-10-25)
release: 1.84.0-nightly
commit-hash: e75214ea4936d2f2c909a71a1237042cc0e14b07
commit-date: 2024-10-25
host: x86_64-unknown-linux-gnu
libgit2: 1.8.1 (sys:0.19.0 vendored)
libcurl: 8.9.0-DEV (sys:0.4.74+curl-8.9.0 vendored ssl:OpenSSL/1.1.1w)
ssl: OpenSSL 1.1.1w  11 Sep 2023
os: Debian n/a (rodete) [64-bit]

iiuc If you run cargo update foo --precise 3.2.1, then only foo will be updated. If that version of foo needs a newer version of syn to work, then it will error because we lock all other dependencies.  If you pass --recursive, we then only unlock the dependency tree which helps in most cases but not all, and will cause dependencies to update that didn't necessarily need to.
I'm guessing cargo update --breaking is leaving the rest of the dependencies locked so they can't update, unlike manually editing Cargo.toml.  In a way --breaking is an update "force" flag  and a package selection flag.  If we remove the package selection flag part, then non-breaking updates will happen to other dependencies which might not be what the user wants.
I'd be curious if this works with --recursive.
Personally, I feel like there are quite a few pitfalls with cargo updates defaults and wonder if we should evaluate what can be changed and what the workflows should be.

Potentially related issues

#5484
#10189
#10307


Show when channel becomes settleable/settled

User Story
As a user I would like to know how many blocks are left until I can settle a closed channel. I think the current settle timeout is 500 blocks and we already save openBlock and closeBlock for each channel in the local state.
Acceptance criteria

Progress bar or just a 288/500 blocks label that indicates how many blocks are left until channel can be settleable/settled

Tasks

Already implemented. Closing :)


🐛 [Bug] torch._export.verifier.SpecViolationError when using Torch-TensorRT

Bug Description

got the export error with torch_tensorrt 2.2.0 use dynamo IR
torch._export.verifier.SpecViolationError: Node.meta max_pool2d_default is missing val field.

To Reproduce
optimized_model = torch_tensorrt.compile(
                        self.model,
                        ir="dynamo",
                        inputs=inputs,
                        enabled_precisions=enabled_precisions,
                        debug=debug,
                        workspace_size=(2147483648) * 4, # 8GB,
                        min_block_size=7,
                        torch_executed_ops=torch_executed_ops,
                        truncate_long_and_double=True,
                        enable_experimental_decompositions=True
 )

Steps to reproduce the behavior:

Expected behavior

Environment

Build information about Torch-TensorRT can be found by turning on debug messages

Torch-TensorRT Version 2.2.0:
PyTorch Version 2.2.1:
CPU Architecture: x86_64
OS (e.g., Linux): Linux
How you installed PyTorch (conda, pip, libtorch, source): pip
Build command you used (if compiling from source): no
Are you using local sources or building from archives: no
Python version: 3.10
CUDA version: cuda 12.3
GPU models and configuration: A30
Any other relevant information: no

Additional context

how to fix this error

use the latest nightly build on 20240110 solve this problem.
python -m pip install --pre torch==2.3.0.dev20240110+cu121 torch-tensorrt==2.3.0.dev20240110+cu121 --extra-index-url https://download.pytorch.org/whl/nightly/cu121
python -m pip install --pre torchvision==0.18.0.dev20240110+cu121 --extra-index-url https://download.pytorch.org/whl/nightly/cu121
python -m pip install --pre torchtext==0.17.0.dev20240110+cpu --extra-index-url https://download.pytorch.org/whl/nightly/cu121

but the nightly build after 20240110  is not avaliable
ERROR: Could not find a version that satisfies the requirement torch-tensorrt==2.3.0.dev20240222+cu121 (from versions: 0.0.0.post1, 1.3.0, 1.4.0, 2.2.0.dev20231010+cu121, 2.2.0.dev20231106+cu121, 2.2.0.dev20231107+cu121, 2.2.0.dev20231108+cu121, 2.2.0.dev20231109+cu121, 2.2.0.dev20231110+cu121, 2.2.0.dev20231111+cu121, 2.2.0.dev20231112+cu121, 2.2.0.dev20231113+cu121, 2.2.0.dev20231114+cu121, 2.2.0.dev20231115+cu121, 2.2.0.dev20231116+cu121, 2.2.0.dev20231117+cu121, 2.2.0.dev20231118+cu121, 2.2.0.dev20231119+cu121, 2.2.0.dev20231120+cu121, 2.2.0.dev20231121+cu121, 2.2.0.dev20231122+cu121, 2.2.0.dev20231123+cu121, 2.2.0.dev20231124+cu121, 2.2.0.dev20231125+cu121, 2.2.0.dev20231126+cu121, 2.2.0.dev20231127+cu121, 2.2.0.dev20231128+cu121, 2.2.0.dev20231129+cu121, 2.2.0.dev20231130+cu121, 2.2.0.dev20231201+cu121, 2.2.0.dev20231202+cu121, 2.2.0.dev20231203+cu121, 2.2.0.dev20231204+cu121, 2.2.0.dev20231205+cu121, 2.2.0.dev20231207+cu121, 2.2.0.dev20231208+cu121, 2.2.0.dev20231209+cu121, 2.2.0.dev20231210+cu121, 2.2.0.dev20231211+cu121, 2.2.0.dev20231212+cu121, 2.2.0.dev20231213+cu121, 2.2.0.dev20231219+cu121, 2.2.0.dev20231220+cu121, 2.2.0.dev20231221+cu121, 2.2.0.dev20231222+cu121, 2.2.0.dev20231223+cu121, 2.2.0.dev20231224+cu121, 2.2.0.dev20231225+cu121, 2.2.0.dev20231226+cu121, 2.2.0.dev20231227+cu121, 2.2.0.dev20231228+cu121, 2.2.0.dev20231229+cu121, 2.2.0.dev20231230+cu121, 2.2.0.dev20231231+cu121, 2.2.0.dev20240101+cu121, 2.2.0.dev20240102+cu121, 2.2.0.dev20240103+cu121, 2.2.0.dev20240104+cu121, 2.2.0.dev20240105+cu121, 2.2.0.dev20240106+cu121, 2.2.0.dev20240107+cu121, 2.2.0.dev20240108+cu121, 2.2.0.dev20240109+cu121, 2.3.0.dev20240110+cu121)
ERROR: No matching distribution found for torch-tensorrt==2.3.0.dev20240222+cu121

This should be fixed by https://github.com/pytorch/TensorRT/pull/3002


Consider using ingress

In the use case where we want to expose the server to the public, I think Kubernetes Ingress would be better than exposing the service as NodePort, for domain name, pathes etc .. ?

Most ingress controllers only support http protocol and Minecraft is a dedicated protocol requiring generic TCP.
Because of this, sadly, most ingress approaches would not work.

Agreed with @billimek . nginx ingress has some non-standard options for bare TCP ingress; however, even then it won't help much since it's not aware of the server selection in the proprietary Minecraft handshake packet.
I implemented and use https://github.com/itzg/mc-router as my compromise for lack of ingress support.

In the use case where we want to expose the server to the public, I think Kubernetes Ingress would be better than exposing the service as NodePort, for domain name, pathes etc .. ?

You really should use a LoadBalancer for this.  hacking up the ingress isn't the way to go

Ah yes, excellent point @jfmatth . Once I saw mention of ingress I was thinking layer-7 routing was part of the use case.
To be more specific, when I have deployed on GCP or DigitalOcean I'd use mc-router and then a cloud LB in front of that to pay only once for the LB regardless of how many server hostnames were being routed :)

Thanks for the clarification !


Raw Option for Joins?

Is there a way to do a raw join, like WhereRaw?  I'd like to be able to add a join specifying the entire predicate for the join instead of the exact columns to join on.
Instead of:
var query = new Query("Posts").Join("Authors", "Authors.Id", "Posts.AuthorId");
Do this:
var query = new Query("Posts").JoinRaw("Authors", "Authors.Id = Posts.AuthorId");

yes you can use the same WhereRaw in the lambda overload
query.Join("Table", j =>  j.WhereRaw(""));

Excellent.  I'll give that a try.  Thanks.


Cleanup dead templates

LMK if you think any of these are still in use. I think they're all dead. There's a --printUnusedTemplates option to hugo but it gets false positives from the promos we aren't using.


Two way binding ngmodel is not working

This editor two way binding is not working how I am binding data in update time I need to display preview once user update data
ngModel and formcontrol is not working
<email-editor ngDefaultControl [(ngModel)]="emailTemplate.templatestring"
name="templatestring" #templatestring="ngModel" [ngClass]="{'is-invalid': f.submitted && templatestring.invalid}" />

I believe you need to call exportHTML to get the current design.
this.emailEditor.editor.exportHtml((data: any) => {   message.htmlMessage = data.html;   message.unlayerDesign = data.design; });

data.design is working for me thanks @dvdnhm


Update javadoc: it incorrectly said to return 'true'

@amaembo Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@amaembo Thank you for signing the Contributor License Agreement!

Merged in 2a12048 thanks.


Turned the theme upside down

T1 Issue report:
Turned the theme upside down
check the image here ----> https://prnt.sc/JxqTiabpz8Gv

T1 version:
v0.8.1
Discord client:
PTB
Platform:
Windows
Discord Mod:
BetterDiscord
Additional notes:
This happen after i update discord

Yeah, the update broke it, but there is a good chance they'll revert it again.
Stable seems unaffected.
If you need to access the settings menu, do Ctrl + Comma

even normal discord broke the theme with updates ;-;
can you make update of the theme maybe this prevent to stop broking

should work again

yes!
thank you, it work now
appriciated your work !

I also get the same issue, thank you


Handle degenerate curves

If all points are colinear, we should detect that and generate curves to span the same footprint.  This is much lower priority than the cusp issue though.

Actually, I convinced myself that just handling cusps would be enough.  So, dupe of https://github.com/googlei18n/cu2qu/issues/6


Expose webpack stats config on webpack dev middleware

Why?
I want to be able to manage what is the output of dev server. I think passing webpack.stats options to dev middleware is a good way. What do you think?

Thanks! Would be nice to document it with some use cases.

Now there are even more lint errors.

I use spaces in my editor :trollface:

Use EditorConfig and run npm t before commit. It’s easy.

Yes! 🍏


Update configure-exchange-server-for-hybrid-modern-authentication.md

Made changes to step 1 as we need AutodiscoverServiceInternalUri to be added to Azure AD Service Principal's SPN's and removed the Addition of activesync URI as Outlook app won't use that

Issue#https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/913

please copy review @JohanFreelancer9

@kelleyvice-msft Please merge this PR. Thanks.


Upgrade vcr gem to 3.x.

Let's see if we can upgrade to VCR 3.x which added better thread safety, among other things. When I ran the full suite locally, I only saw 1 failure and it appeared to be unrelated.
For the VCR changelog, please see https://github.com/vcr/vcr/blob/master/CHANGELOG.md.

@jrafanie @Fryguy Whaddya think?


[Dialog] disableEnforceFocus prop not working in Electron App

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/material-ui/react-modal/
Kind of issue
Other
Issue description
I was looking to disable the focus on the Dialog and it appears it doesn't work with my application.
Docs mention:
<Modal disableEnforceFocus />

My usage:
<Dialog open={true} hideBackdrop={true} disableEnforceFocus={true}>
...
</Dialog>

This above snippet is making the Dialog Backdrop invisible but still the focus on component is not being disabled.
Context 🔦
Dialog component displays the list of errors returned from the server. The user should be provided to edit the settings with the dialog open itself.

Could you possible try to replicate the behavior on codesandbox and post a link? Or have you verified that this only occurs on Electron?

Same problem. If you start tabbing from outside the dialog, it will navigate through the page setting the focus on all the different elements, but once the focus enters the dialog, it will never go back to the page again regardless of the value of disableEnforceFocus. The only difference is that with disableEnforceFocus as false you cannot even start tabbing from outside the dialog, it will always trap the input, but the fact that the focus can't scape the dialog once it's in, is not desirable either.


Update to use template files

Pull Request (PR) description

Update appveyor.yml to use the default template.
Added default template files .codecov.yml, .gitattributes, and .gitignore, and
.vscode folder.

This Pull Request (PR) fixes the following issues

Task list

[x] Added an entry under the Unreleased section of the change log in the README.md.
Entry should say what was changed, and how that affects users (if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Resource Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
[ ] New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

This change is

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@4668715). Click here to learn what that means.
The diff coverage is n/a.

@@         Coverage Diff          @@
##             dev     #8   +/-   ##
====================================
  Coverage       ?   0.8%
====================================
  Files          ?      3
  Lines          ?    250
  Branches       ?      0
====================================
  Hits           ?      2
  Misses         ?    248
  Partials       ?      0

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 4668715...0a85c92. Read the comment docs.


Update poi panel on login/logout

On login or logout refresh the boolean isPoi and update the panel

Maybe it's too late but as store is a singleton can we use method instead of spaguetto event ?
whe should evaluate the size of the change before make a choice
Tell me

We could have store extend EventTarget  #134


Remarks

This PR is used to discuss remarks on the code.
A guide to Remarks can be found here

NOTE: this PR should never be merged into master.

Codecov Report
Base: 40.18% // Head: 39.95% // Decreases project coverage by -0.24% :warning:

Coverage data is based on head (bd93c40) compared to base (57c6bbd).
Patch coverage: 19.46% of modified lines in pull request are covered.

:exclamation: Current head bd93c40 differs from pull request most recent head a363e87. Consider uploading reports for the commit a363e87 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1484      +/-   ##
==========================================
- Coverage   40.18%   39.95%   -0.24%
==========================================
  Files         147      147
  Lines        9651     9737      +86
==========================================
+ Hits         3878     3890      +12
- Misses       5144     5222      +78
+ Partials      629      625       -4

Impacted Files
Coverage Δ

pkg/config/genesis/config.go
0.00% <ø> (ø)

pkg/config/genesis/presets.go
69.23% <ø> (ø)

...nsensus/blockgenerator/candidate/blockgenerator.go
0.00% <0.00%> (ø)

...core/consensus/blockgenerator/candidate/genesis.go
0.00% <0.00%> (ø)

...kg/core/consensus/blockgenerator/candidate/mock.go
0.00% <0.00%> (ø)

pkg/core/data/ipc/transactions/fixtures.go
13.76% <0.00%> (-0.21%)
:arrow_down:

pkg/core/data/ipc/transactions/provider.go
0.00% <0.00%> (ø)

pkg/gql/query/blocks.go
63.22% <0.00%> (ø)

pkg/gql/query/mempool.go
30.00% <0.00%> (ø)

pkg/gql/query/types.go
9.75% <0.00%> (-4.04%)
:arrow_down:

... and 10 more

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

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


Correct first bullet point in Note

[Enter feedback here]

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

ID: 65e527db-a4c2-782e-5a5f-ad8b3159a977
Version Independent ID: f2de1272-5e37-c59f-fcb9-34fa66b0d268
Content: Connect to Exchange Online PowerShell using multi-factor authentication
Content Source: exchange/docs-conceptual/exchange-online/connect-to-exchange-online-powershell/mfa-connect-to-exchange-online-powershell.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

Currently Note section has bullet point:
"• You can't use the Exchange Online Remote PowerShell Module to connect to Exchange Online PowerShell and Security & Compliance Center PowerShell in the same session (window). You need to use separate sessions of the Exchange Online Remote PowerShell Module."
This is incorrect.  Here's a better explanation:
"In order to use both Security and Compliance powershell commands with Exchange Online commands in the Exchange Online Remote Powershell Module for MFA, you need to first connect with connect-ippsession commandlet.  Once that imports the commands, you'll have a new commandlet available, new-exopssession.  Here's an example of connecting to both within the same window and using a proxy defined by NETSH:
netsh winhttp set proxy <IP_ADDRESS>:8888 "localhost;*.fabrikam.com"
$ProxySetting = New-PSSessionOption -ProxyAccessType WinHttpConfig
Connect-IPPSSession -PSSessionOption $ProxySetting
#Then you can add the Exchange Online commands too, and add a prefix to prevent any conflict with connection to on premises or the security and compliance commands which share a few of these commands.
$EXOSession = New-ExoPSSession -pssessionoption $ProxySetting
Import-PSSession $EXOSession -Prefix EXO
"

@officedocsbot assign @yogkumgit

Hello @austinmccollum
Thanks for your feedback and contribution.
I created a pull request to update this article and the Security & Compliance one.
The author @chrisda will review it and if he agrees, it will be merged.
Thank you

@chrisda Please merge the PR #3868

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @austinmccollum Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


[HB-7031] Handle Memory Leak

Fixes a memory leak for fullscreen ads.
Updated CHANGELOG, README, Adapter version to <IP_ADDRESS>.1

Will be pushing another commit that moves the listener to an internal class.


Howto Use a handler for multiple requirements?

Issues with Existing Topics
I'm trying to understand Use a handler for multiple requirements.  I don't understand how to create my requirement(s).  Can this example be expanded?

In the one to one scenario, I do this: public class MinimumAgeHandler : AuthorizationHandler<MinimumAgeRequirement>.  Cool 😎.  That must be the glue between the two of them.
What about with public class PermissionHandler : IAuthorizationHandler?  Where's the glue?
public void ConfigureServices(IServiceCollection services)
{
    services.AddMvc();

    services.AddAuthorization(options =>
    {
        options.AddPolicy("???", policy =>
            policy.Requirements.Add(new ???()));
    });

    services.AddSingleton<IAuthorizationHandler, PermissionHandler>();
}

@spottedmahn there is no glue at the interface level, your permission handler can handle any requirements you want.  Its app specific at this point, what requirements you have, and which handler class is responsible for which requirement.

At the end of the day authorization will succeed as long as all requirements have been met, whether that's one handler succeeding them all, or a bunch of different handlers is up to the app.

You guy are amazing!  That is an amazing response time ⚡!
Processing 🤔 your response...

@spottedmahn I've created issue https://github.com/aspnet/Docs/issues/6033 to clarify some things in the doc. Anything else you'd like to see addressed in there?

I think it would help to see the ConfigureServices of ReadPermission, EditPermission and DeletePermission
I think it would help to see the Policies used in a controller
In general, provide a more complete sample.

I'm thinking I can do this... not 100% at this point though.
public void ConfigureServices(IServiceCollection services)
{
    services.AddMvc();

    services.AddAuthorization(options =>
    {
        options.AddPolicy("Read", policy =>
            policy.Requirements.Add(new ReadPermission()));
        options.AddPolicy("WriteDelete", policy =>
            {
                policy.Requirements.Add(new EditPermission());
                policy.Requirements.Add(new DeletePermission());
            });
    });

    services.AddSingleton<IAuthorizationHandler, PermissionHandler>();
}

[Authorize(Policy = "Read")]
public class DocumentController : Controller
{
    public IActionResult GetDoc() => View();

    [Authorize(Policy = "Delete")]
    public IActionResult DeleteDoc() => View();
}

@spottedmahn Thank you for the feedback. In the meantime, here's the sample app associated with the doc: https://github.com/aspnet/Docs/tree/master/aspnetcore/security/authorization/policies/samples/PoliciesAuthApp1.

Oh, nice!  Perhaps this sample can be update for the Use a handler for multiple requirements example?
Looking at ConfigureServices in that sample I don't see it.  Please correct me if I'm wrong.

@spottedmahn Yeah that's from the sugar default implementation, TRequirement isn't on the interface

AuthorizationHandler<TRequirement> is sugar to make it easy to handle on requirement

@spottedmahn See the following file for the handler: https://github.com/aspnet/Docs/blob/master/aspnetcore/security/authorization/policies/samples/PoliciesAuthApp1/Services/Handlers/PermissionHandler.cs

Thanks @scottaddie.  Looks like the same code I see on the docs page.  I was hoping for a complete code sample.

AuthorizationHandler<TRequirement> is sugar to make it easy to handle one requirement

Ahh, I see, thanks @HaoK!

I really need to follow up on this with a sample, but I’ve implemented this and does feel good to me. 🚫
Before, I had handler logic in individual classes.  Not I’ve got 1⃣ handler class handling many cases.
Seems to go against the single responsibility principal.
It was manageable for our simple case of 3 or 4 IFs, but it doesn’t seem to scale well.
Thoughts 💭?

Isn't that by definition what you are trying to accomplish? (single handler handling multiple requirements), if you want single responsibility, have one handler per requirement

Composability is really what I want to accomplish.  I only headed down this path to achieve that.
It is really a separate issue now that I think 🤔 about it.  This thread is about how-to handle multiple requirements in one handler, like you stated.
I’ll create a good sample, on a different issue, to describe it, thanks for the quick ⚡ reply!

Did you get chance to figure out the complete working code from the solution? I am struggling the same issue now. Different examples have different approaches. Really need one to get better understanding.


Handle an existing spantree in exactmatch

This may happen if a field which is indexed is used as an input for another field indexed as exact match.

Ping @arnej27959


Testing OP-TEE Remote Attestation with Veraison compiled natively

Hello, I am trying to run your project with Veraison verification. While trying to compile Veraison with docker deployment, I encountered many errors that I couldn't fix, so I tried to compile it natively and it worked. At this point I want to ask whether it is possible to run the OP-TEE remote attestation with Veraison verification compiled natively.
Thank you in advance.

Do you mean that you buiid OPTEE PTA for remote attestation?
Ather that you need to build
CA and TA https://github.com/iisec-suzaki/optee-ra/tree/main/attester/remote_attestation
Relying Party on other machine 1 https://github.com/iisec-suzaki/optee-ra/tree/main/relying_party
and set up Verifier (Variasion) on other machine 2.

Thank you. When I run the optee_remote_attestation command, I receive the following errors:
On the relying party:
Error forwarding request: Post "http://verification-service:8080/challenge-response/v1/newSession?nonceSize=32": dial tcp: lookup verification-service: Temporary failure in name resolution
On the normal world:
Failed to allocate Veraison client session. Error/log message: error decoding response body: expected value at line 1 column 1 Disposing client session
I guess these errors are related with the fact that the attester cannot connect to the verifier, even though it is active and running and the URL matches.


Add xRetry package and apply to tests

Add xRetry package to Directory.Packages.props and IntegrationTests.csproj. Update StepwisePlannerTests to use RetryTheory attribute for improved test reliability.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

/sk generate-pr-description


AccessibilityManager uses wrong values

Upon experimenting with the AccessibilityManager I noticed weird behavior when setting the width and height of the overlay div in the update method. The width and height are set using the renderers width and height, instead of the DOM elements getBoundingClientRect().
This leads to wrong values being set, as borders are not accounted for.
The Canvas has a border of 10px:

Current code:
const rect = this.renderer.view.getBoundingClientRect();
const sx = rect.width / this.renderer.width;
const sy = rect.height / this.renderer.height;

let div = this.div;

div.style.left = `${rect.left}px`;
div.style.top = `${rect.top}px`;
div.style.width = `${this.renderer.width}px`;
div.style.height = `${this.renderer.height}px`;

Simply changing the last two lines to use the rect instead solves the issue:
div.style.width = `${rect.width}px`;
div.style.height = `${rect.height}px`;

Is there any reason the renderer's width and height are used instead of the getBoundingClientRect()? If not and it's simply a miss I can create a PR that fixes it :)

Hi @nikkop - Did you find a solution for this?


DistantanceMatrix should have a method to find max_pair

Method should return the two strings that correspond to the maximally divergent pair.
Working on the underlying numpy array is likely the most performant way to implement this.
sketch:
def max_pair(self):

  max_index_flat = np.argmax(self)
  max_index_1, max_index_2 = np.unravel_index(max_index_flat, self.shape)
  max_pair = (self.names[max_index_1], self.names[max_index_2])

  return max pair

The type hint for this will be
def max_pair(self) -> typing.Tuple[str, str]:

I also think that a min_pair() method should be implemented.
Examples of these should be added to doc/cookbook/calc_genetic_distance.rst. Add a new h2 level headings for Get names of sequences with max pairwise distance, same for minimum.

I'll work on this!! 💪


OpaqueSpan: eradicated

The OpaqueSpan class was originally created to wrap SpanData objects when exposed through the external facing custom span API. Once we began sampling, we needed to have nullSpan functionality inside the hooks. To achieve this, OpaqueSpans were used everywhere SpanData objects were needed. Since OpaqueSpan was needed everywhere, it no longer makes sense to wrap SpanData objects.

@ofrobots PTAL

This is great! LGTM.


Energy Tablet from Mekanism Not able to be used in auto crafting

Issue description:
Energy Tablet from Mekanism Not able to be used in auto crafting.
What happens:
When going to auto craft it says missing item.
What you expected to happen:
Energy tablet in storage and auto craft to happen.
Steps to reproduce:

Pattern an energy tablet.
Pattern an item that uses the tablet in a recipe (induction casing)
Try to craft.
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: <IP_ADDRESS>60
Refined Storage: 1.6.4 (one behind)

Does this issue occur on a server? [yes/no]
More than likely
PS: I did notice the NBT tags are different from the crafted item to the JEI item. Even when creating the pattern from a crafted Energy tablet, the craft still says missing item.
so my guess is once the craft is stored on a pattern, the NBT tag goes back to the JEI tag
{mekData:{energyStored:0.0d}} is the NBT tag when crafted and in the system
{mekData:{}} is the NBT tag from JEI

Read prev issues


Meter doesn't register dmg

The meter connects to the server:
[1] INFO  CasualMeter.App - Checking for updates.
[1] INFO  CasualMeter.App - Current Version: v<IP_ADDRESS>
[1] INFO  CasualMeter.App - Starting up.
[17] INFO  CasualMeter.ShellViewModel - Sniffer has been enabled.
[18] INFO  CasualMeter.ShellViewModel - Connected to server Mount Tyrannas.
As you can see here.
But the meter doesn't register dmg from me or anybody in the party.

I'd suggest restarting.  I've never gotten this issue.

i restarted, looked internet adapters, getting off antivirus, runing as admin, manually connecting to server via "server override txt" and i checked with a tera snifer if i got packets sending.

Don't know what to tell you.  There's no way I can fix something that no one else can reproduce.


feat: add support to set Type for AWS parameter store

Problem Statement
Currently, we don't have the option to set which parameter type to use for the AWS parameter store, and as a result, all secrets are stored remotely as plain text String. This PR adds the possibility to work with the following types String, SecureString, and StringList through PushSecret's metadata.
Related Issue
Fixes #3422
Checklist

[x] I have read the contribution guidelines
[X] All commits are signed with git commit --signoff
[X] My changes have reasonable test coverage
[X] All tests pass with make test
[X] I ensured my PR is ready for review with make reviewable

Nicely done!

/ok-to-test sha=28ab26f


The order of requires clauses is not respected

I tried this code:

#[kani::requires(x == 0)]
#[kani::requires(x * y < 10)]
fn foo(x: u8, y: u8) {}

#[kani::proof_for_contract(foo)]
fn check_foo() {
    foo(kani::any(), kani::any());
}

using the following command line invocation:
cargo kani -Zfunction-contracts

with Kani version: d2f5dbe04ad
I expected to see this happen: Verification successful

Instead, this happened: the multiplication overflow check failed:
SUMMARY:
 ** 1 of 49 failed
Failed Checks: attempt to multiply with overflow
 File: "src/lib.rs", line 2, in foo::{closure#2}

VERIFICATION:- FAILED

I would expect it to succeed because of the previous requires clause.
Interestingly, if I flip the order of the requires clauses, verification succeeds:
#[kani::requires(x * y < 10)]
#[kani::requires(x == 0)]
fn foo(x: u8, y: u8) {}

#[kani::proof_for_contract(foo)]
fn check_foo() {
    foo(kani::any(), kani::any());
}

SUMMARY:
 ** 0 of 49 failed

VERIFICATION:- SUCCESSFUL

I don't think we have established the evaluation order of contract predicates though.

I understand. It seems intuitive to expect that the order is respected though. This was the source of confusion in https://github.com/model-checking/verify-rust-std/discussions/99.

We decided not to fix this issue. If a user writes this:
#[requires(a)]
#[requires(b)]
fn foo() {...}

it's not obvious that b has a control flow dependency on a (i.e., that we should not evaluate b unless a is true). If the user wants to introduce such a dependency, they should do so explicitly, e.g.:
#[requires(a && b)]
fn foo() {...}


Bump up dependency analyze to 1.10.0

Summary
This PR upgrades the Gradle dependency version to 1.10.0 to facilitate the ongoing Java upgrade.
Background
We initially planned to upgrade the Gradle dependency to version 1.9.2. However, after implementing this upgrade, we encountered "unusedDeclaredArtifacts" errors. Further investigation revealed that these errors were due to a known bug (https://github.com/gradle-dependency-analyze/gradle-dependency-analyze/issues/527) in version 1.9.2. This bug has been resolved in version 1.10.0.
Changes Introduced
Upon upgrading to version 1.10.0, several build.gradle files were found to be missing dependencies that are now required due to changes introduced in this newer version (as per commit 2bc05cf9df5690c4). This PR addresses these errors by adding the necessary dependencies to the relevant build.gradle files. Additionally, some of these dependencies were not needed for Java 11. However, when running analyzeDependencies for Java 21, new "usedUndeclaredArtifacts" errors were encountered. To resolve these issues, the necessary dependencies were added to the relevant build.gradle files.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes.
[ ] I have added documentation for the Reference Guide

* What went wrong:
Execution failed for task ':solr:prometheus-exporter:analyzeTestClassesDependencies'.
> Dependency analysis found issues.
  usedUndeclaredArtifacts
   - org.apache.solr:solrj-zookeeper:10.0.0-SNAPSHOT@

* What went wrong:
Execution failed for task ':solr:modules:gcs-repository:analyzeClassesDependencies'.
> Dependency analysis found issues.
  usedUndeclaredArtifacts
   - com.google.auth:google-auth-library-credentials:1.23.0@jar

* What went wrong:
Execution failed for task ':solr:modules:hdfs:analyzeClassesDependencies'.
> Dependency analysis found issues.
  usedUndeclaredArtifacts
   - io.dropwizard.metrics:metrics-core:4.2.26@jar

* What went wrong:
Execution failed for task ':solr:modules:s3-repository:analyzeClassesDependencies'.
> Dependency analysis found issues.
  usedUndeclaredArtifacts
   - software.amazon.awssdk:http-client-spi:2.26.19@jar

I was curious about why these new errors appeared immediately after the upgrade. From the community, I learned that the new changes in gradle-dependency-analyze (as detailed in gradle-dependency-analyze/gradle-dependency-analyze#527) have introduced these warnings. While some of these dependencies seem to be transitive, I cannot see them being directly used in the project. I wonder if there might be another reason for these warnings beyond them being transitive dependencies.

These I think seem reasonable from a quick glance.

@dsmiley Should I create JIRA for this one?

@risdenk I know you introduced this plugin but IMO it's more annoying & burdensome than helpful.  Maybe we should back out this plugin?  I was somewhat appreciative of the Maven equivalent, which had some imperfections but mostly worked.  The Gradle one seems more troublesome, requiring that we explicitly list dependencies that are not in fact directly used.  Nowadays, I just want the build to be simpler.
JIRA issues aren't required for build improvements in general.

Planning to merge this shortly!

@risdenk Should I backport this change to 9_x?

@iamsanjay yes

@risdenk I know you introduced this plugin but IMO it's more annoying & burdensome than helpful. Maybe we should back out this plugin? I was somewhat appreciative of the Maven equivalent, which had some imperfections but mostly worked. The Gradle one seems more troublesome, requiring that we explicitly list dependencies that are not in fact directly used. Nowadays, I just want the build to be simpler.
JIRA issues aren't required for build improvements in general.

since we talked about this during the community meeting - I agree the verbosity is a pain but this basically moves runtime issues into build time issues. Every case I've looked into the classes/dependencies in question were used in the code somewhere. There could be bugs in the plugin but they seem to get fixed overtime. It usually is the case the plugin seems to miss things that needed to be included (which is what this upgrade addresses). I haven't seen a case where a dependency needed to be declared but then wasn't actually used.


Error while updating the installer

Release Type: Official Release
Version: Current
Platform(s): Windows 10
Hello there,
I have tried to install xenko (first time) but when opening the launcher there always pops up the window saying "An error occurred while updating the launcher. If the provlem persists, reinstall this application"
Is there any way I can find out what the issue is eg. Log Files? Does the launcher use an IP on Port 80? IPv4 is not allowed to connect to port 80, but I guess that you are using 443 if I can trust the network manager.

The last IP cannot be connected. I don't know whether there is some kind of load balancer that disallows pings but I cannot ping it. I don't know if its a bad sign.

Probably duplicate of #1 #131 and #130.
I have just uploaded a new launcher that should fix those.
Please download latest launcher from https://xenko.com/download/ and let us know if it fixed the issue.
Otherwise, it should at least display more info.

Yeah it was that issue. I thought it should have been fixed when I saw the date.


Resizable nested inside of draggable creates drag with resize handler

Given the following code:
<Draggable bounds="parent">
  <Resizable className="box" width={150} height={150} onResizeStart={this.onStart}>

Test

  </Resizable>
</Draggable>

Where <Draggable /> is the react-draggable module, when resizing using the react-resizable-handle, it also triggers the drag of the parent element. Is this an issue or is there a way to 'cancel' the resize from bubbling upwards to the parent draggable?

Same problem here, any resolution?

I guess I just found some clues, first you should put cancel prop in the draggable, like this
<Draggable  bounds="parent" cancel=".react-resizable-handle">

Then you need to have a customised onResize function, which draws the resized box, like this:
onResize: function (event: Event, data: Object) {
	  let {element, size} = data;
	  let {width, height} = size;
	  let widthChanged = width !== this.state.width, heightChanged = height !== this.state.height;
	  if (!widthChanged && !heightChanged) return;

	  //[width, height] = this.runConstraints(width, height);
	  console.log(width, height);

	  this.setState({width, height}, () => {
	    if (this.props.onResize) {
	      this.props.onResize(event, {element, size: {width, height}});
	    }
	  });
	},

the Resizable component will be like this:

    text

This is very roughly implementation, I found the code in this link:
https://github.com/STRML/react-resizable/blob/master/lib/ResizableBox.jsx

Hey. I've just pushed #82 for this. Please check it out and let me know your comments.
Cheers!

I know this is quite old but I'm having a very similar issue.
I managed to have a resizable/draggable box following what is done on #82 but if my Draggable has the prop bounds="parent", I'm unable to drag my box on the Y axis.
If someone have a solution, thanks.

I reproduced my scenario in here.
I think it is a special case. I use Resizable inside a useDrag element (react-dnd). And when resizing, the drag event is propagates to the useDrag-element.
I tried to use e.stopPropagation(), but it didn't work. Then I gave e.preventDefault() a try and It worked !!
Anyone can help me know why?


BETA 5: Login error

I am seeing this failure consistently on our build server post WWDC.  Have updated sigh to the most recent beta.
[09:34:12]: ------------------
[09:34:12]: --- Step: sigh ---
[09:34:12]: ------------------
/Library/Ruby/Gems/2.0.0/gems/spaceship-0.0.6/lib/spaceship/client.rb:150:in `login': #<Faraday::Response:0x007f83ccf96940> (Spaceship::Client::InvalidUserCredentialsError)
	from /Library/Ruby/Gems/2.0.0/gems/spaceship-0.0.6/lib/spaceship/client.rb:44:in `login'
	from /Library/Ruby/Gems/2.0.0/gems/spaceship-0.0.6/lib/spaceship.rb:32:in `login'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.0.0.beta6/lib/sigh/spaceship/runner.rb:11:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/sigh-1.0.0.beta6/lib/sigh/manager.rb:8:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/actions/sigh.rb:20:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/fast_file.rb:112:in `block (2 levels) in method_missing'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/actions/actions_helper.rb:51:in `execute_action'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/fast_file.rb:94:in `block in method_missing'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/fast_file.rb:93:in `chdir'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/fast_file.rb:93:in `method_missing'
	from (eval):46:in `block (2 levels) in parse'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/runner.rb:35:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/runner.rb:35:in `block in execute'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/runner.rb:25:in `chdir'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/runner.rb:25:in `execute'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/lib/fastlane/lane_manager.rb:33:in `cruise_lane'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/bin/fastlane:39:in `block (2 levels) in run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/command.rb:178:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/command.rb:153:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/runner.rb:428:in `run_active_command'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/runner.rb:68:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/commander-4.3.4/lib/commander/delegates.rb:15:in `run!'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/bin/fastlane:118:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fastlane-1.3.2/bin/fastlane:124:in `<top (required)>'
	from /usr/bin/fastlane:23:in `load'
	from /usr/bin/fastlane:23:in `<main>'
[09:34:12]: Starting login
[09:34:14]: Variable Dump:
[09:34:14]: {:ENVIRONMENT=>nil, :PLATFORM_NAME=>nil, :LANE_NAME=>"deploy", :BUILD_NUMBER=>0, :PRODUCE_APPLE_ID=>"957304011", :CERT_FILE_PATH=>"/tmp/cert/3LMSG3CHK3-R58UK2EWSO.cer", :CERT_CERTIFICATE_ID=>"3LMSG3CHK3"}
[09:34:14]: #<Faraday::Response:0x007f83ccf96940>
[09:34:14]: fastlane finished with errors
Build step 'Execute shell' marked build as failure
Build did not succeed and the project is configured to only push after a successful build, so no pushing will occur.
Finished: FAILURE

Also seeing this error when running from the command line

oops, I'm actually on beta 6
buildserver:iOS robwithhair$ sigh -v
sigh 1.0.0.beta6

I see you're working on this already https://github.com/fastlane/spaceship/pull/44

@robwithhair When you update spaceship it should work again :+1:

@KrauseFx Thanks, all seems to be working again.

Enjoying the WWDC coverage?  Really interested in the open sourcing of swift, if it happens.


WebpackBundleLookupError in version 1.0.0 when files outside of entry points change

I've recently upgraded to version 1.0.0 of django_webpack_loader and webpack-bundle-tracker.
I have a project set up something like:
- frontend/
  - webpack.config.js
  - assets/
  - cypress/

The config includes:
entry: {
  index: ['./assets/js/index.jsx'],
},

When I edit files in the cypress directory, while running webpack-dev-server, no rebuild is triggered since the files aren't within the index entry point, but a new stats JSON is generated, and django_webpack_loader blows up with: WebpackBundleLookupError: Cannot resolve bundle index
Python error looks like this:

Do I need to change my configuration to somehow avoid new stats files getting generated when I edit files in frontend/cypress, or is this a bug? I couldn't find any exclude or ignore options in webpack-bundle-tracker. Didn't seem to be an issue with previous versions of these packages.

Didn't seem to be an issue with previous versions of these packages.

Confirmed downgrading to "webpack-bundle-tracker": "^1.0.0-alpha.1" and git+https://github.com/oleggromov/django-webpack-loader.git@164e7e8753 works correctly.
Unrelated, but I also noticed I need to change relative path location for the stats file between 1.0.0-alpha.1 and 1.0.0 though I didn't see any mention of that anywhere:
-  new BundleTracker({ filename: '../webpack-stats.dev.json' }),
+  new BundleTracker({ filename: './webpack-stats.dev.json' }),

Not sure if any issue should be created for adding that to the docs, or if it's a regression/bug?

Thanks, we'll be checking this tomorrow

@DrMeers could you please inform the webpack version you're using?

this was on<EMAIL_ADDRESS>

@DrMeers thanks for the info.
Would you be able to share more configurations about your setup? Like webpack.config.js and webpack-dev-server setup? I'm having some trouble reproducing this issue.

webpack.dev.config.js:
const webpack = require('webpack')
const BundleTracker = require('webpack-bundle-tracker')
const HtmlWebpackPlugin = require('html-webpack-plugin')

const config = require('./webpack.base.config.js')

const webpack_dev_url = process.env.WEBPACK_DEV_URL || 'http://localhost:8080'

config.devtool = 'inline-source-map'

config.entry.index = ['react-hot-loader/patch', ...config.entry.index]

config.output.publicPath = `${webpack_dev_url}/`
// config.devServer =  -- see server.js WebpackDevServer options

config.resolve = {
  ...(config.resolve || {}),
  alias: {
    ...config.resolve.alias,
    'react-dom': '@hot-loader/react-dom',
  },
}

config.module.rules[0].use[0].options.plugins.unshift('react-hot-loader/babel')

config.plugins = config.plugins.concat([
  new webpack.HotModuleReplacementPlugin(),
  new webpack.NoEmitOnErrorsPlugin(), // don't reload if there is an error
  new BundleTracker({ filename: '../webpack-stats.dev.json' }),
  new HtmlWebpackPlugin({
    templateContent:
      '<!DOCTYPE html><meta charset="UTF-8"><title>Site Title</title>

',
    favicon: './assets/img/favicon.ico',
  }),
])

config.mode = 'development'

module.exports = config

webpack.base.config.js:
const webpack = require('webpack')
const path = require('path')
const MiniCssExtractPlugin = require('mini-css-extract-plugin')
const autoprefixer = require('autoprefixer')

module.exports = {
  context: __dirname,

  entry: {
    index: ['./assets/js/index.jsx'],
  },

  output: {
    path: path.resolve('./webpack_bundles/'),
    filename: '[name]-[hash].js',
  },

  plugins: [
    new webpack.LoaderOptionsPlugin({
      options: {
        eslint: {
          failOnWarning: false,
          failOnError: true,
        },
      },
    }),
    new MiniCssExtractPlugin({
      // Options similar to the same options in webpackOptions.output
      // both options are optional
      filename: '[name]-[hash].css',
      chunkFilename: '[id]-[hash].css',
    }),
  ],

  module: {
    rules: [
      {
        // keep jsx first
        test: /\.jsx?$/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'babel-loader',
            options: {
              plugins: [
                // BEWARE use config.module.rules[0].use[0].options.plugins
                // in webpack.dev.config.js!
                '@babel/proposal-class-properties',
                '@babel/proposal-object-rest-spread',
              ],
              presets: ['@babel/env', '@babel/react'],
            },
          },
          {
            loader: 'eslint-loader',
            options: {
              // fix: true,
            },
          },
        ],
      },
      {
        test: /\.tsx?$/,
        use: 'ts-loader',
        exclude: /node_modules/,
      },
      {
        test: /\.(scss|css)$/,
        use: [
          // fallback to style-loader in development
          process.env.NODE_ENV !== 'production'
            ? 'style-loader'
            : MiniCssExtractPlugin.loader,
          'css-loader',
          {
            loader: 'postcss-loader',
            options: {
              plugins: () => [autoprefixer],
            },
          },
          {
            loader: 'sass-loader',
            options: {
              data: '@import "variables";',
              includePaths: [path.resolve(__dirname, './assets/css')],
            },
          },
        ],
      },
      {
        test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        loader: 'url-loader?limit=10000&minetype=application/font-woff',
      },
      {
        test: /\.(jpg|png|pdf|eot|svg)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        loader: 'file-loader',
      },
      {
        test: /\.(otf|ttf)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        loader: 'base64-inline-loader?limit=1000&name=[name].[ext]',
      },
    ],
  },

  resolve: {
    alias: {
      '@': path.resolve(__dirname, 'assets/js'),
      components: path.resolve('assets/js/components'),
    },
    modules: ['node_modules', 'bower_components'],
    extensions: ['.js', '.jsx', '.ts', '.tsx', '.json'],
  },
}

server.js:
const webpack = require('webpack')
const WebpackDevServer = require('webpack-dev-server')
const config = require('./webpack.dev.config')

const PORT = parseInt(config.output.publicPath.split(':')[2], 10)

new WebpackDevServer(webpack(config), {
  publicPath: config.output.publicPath,
  index: 'index.html',
  disableHostCheck: true,
  sockHost: 'localhost',
  sockPort: PORT,
  hot: true,
  inline: true,
  historyApiFallback: true,
  headers: {
    'Access-Control-Allow-Origin': '*',
  },
}).listen(PORT, '<IP_ADDRESS>', (err, result) => {
  if (err) {
    console.log(err)
  }

  console.log(`Listening at <IP_ADDRESS>:${PORT}...`)
})

Awesome, thanks for the quick reply! Will be giving a look at this soon.

Thank you!

Closing as not reproduced. Please feel free to provide more info if you face a similar issue.


Raphael Pierzina - Kickstarting projects with Cookiecutter

00:00 - Welcome!
00:10 - Introduction
01:22 - Agenda
02:18 - What is Cookiecutter?
06:00 - How does Cookiecutter work?
09:20 - Create you own Cookecutter template
17:01 - Popular Cookecutter templates
19:02 - Demo: A custom Jupyter widget

Thank you!!!


Add "getting started" instructions for Wasm

We should update TemporaryGettingStarted.md to include instructions for testing on Wasm.

Follow-up to #584.


bridge: Exception handling in prepare_confirmation_transaction looks fishy

The prepare_confirmation_transaction method catches a ValueError with the following code:
        try:
            ...
        except ValueError as e:
            if "failed due to an exception" in e.args[0]["message"]:
                self.logger.info(

If the try block raises a ValueError with something like raise ValueErrror("foo"), the catch block would  throw a TypeError: string indices must be integers.
The code in question was introduced in 3bdb7ede0bb80cff90e812dd1e1e73b3054c5a03
My guess is this error we're trying to catch is raised in buildTransaction, so we should move the other code out of the try block.
Also, the test 'failed due to an exception' in error message looks very generic. I fear this may match more exceptions than we actually want.

@saschagoebel Any comments on this one? I guess you did actually test this with a running program, right?

Yes, this was part of the docker compose task, so it ran on a real chain with a deployed contract.
I was not able to find any better error message in this case, because the transaction is never actually executed, so there's no stack trace either.
I found some StackOverflow posts and issues with similar questions, but no usable answer.
If you come up with something that gives us the actual error, I'd be happy to change it.
I left the signing in the try block, because I want execution to stop at this point in any case. But I've heard rumors that exceptions in python are used differently though.

Thanks for the clarification. We should at least clean that up a bit and add some clarifying comments.

Closing as we don't handle this error anymore (we specify the gas limit explicitly now, so the error shouldn't ever be thrown).

This has been removed in d000a668e5990edbddd71539867280a4fa599f67.


Update CONTRIBUTING.md

Occurrences of the word pre commit are now pre-commit. The document possesses both hyphenated and non-hyphenated iterations, and this PR makes the usage consistent.
Note: CLA has been signed.

Thanks @glennon -- have you signed the CLA?

Verified @glennon has signed the CLA. Merging.


Feat/account stats

Statistics
Account Table creation
Top Account
Accounts data export

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: raghukapur9:x: shivam-25You have signed the CLA already but the status is still pending? Let us recheck it.

We will push with optimizations


add sourmash to private and public CLI

This follows from #166, where we added the initial sourmash workflow.
In this PR, I:

wrote an initial parsing method for sourmash compare to extract estimated ANI values,
provided logging to the DB via the private CLI, and
added sourmash to the public CLI.

The logic for logging to the DB via the private CLI and adding sourmash to the public CLI follows the approach used in other methods like aniM, fastANI  etc.
Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[X] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)
[ ] This change requires a documentation update
[ ] This is a documentation update

Action Checklist

[X] Work on a single issue/concept (if there are multiple separate issues to address, please use a separate pull request for each)
[X] Fork the pyani-plus repository under your own account (please allow write access for repository maintainers)
[X] Set up an appropriate development environment (please see CONTRIBUTING.md for this repository)
[X] Create a new branch with a short, descriptive name
[X] Work on this branch

[X] style guidelines have been followed (please see CONTRIBUTING.md for this repository)
[X] new code is commented, especially in hard-to-understand areas
[ ] corresponding changes to documentation have been made
[X] tests for the change have been added that demonstrate the fix or feature works

[X] Test locally with pytest-plus -v non-passing code will not be merged
[ ] Rebase against origin/master
[X] Check changes with linting/formatting tools before submission (please see CONTRIBUTING.md for this repository)
[X] Commit branch
[X] Submit pull request, describing the content and intent of the pull request
[X] Request a code review
[X] Continue the discussion at Pull requests section in the pyan-plus repository

Things that we still need to discuss/change:

Currently, with sourmash, we are only able to report estimated ANI values, not coverage, the number of aligned bases, or similarity errors. I need to investigate if there is a way to retrieve this information.
In the public CLI, the argument extra is set to scaled=300 for now, but this will need to be changed to scaled=1000 to match the default settings in sourmash. It is currently set to scaled=300 because scaled=1000 fails to estimate ANI values for our set of viral genomes. I can't figure out how to change the parse settings to scaled=300 for testing when the default is set to scaled=1000.

It appears there is no built-in way in typer to do mutually exclusive options (although this is possible in argparse), although there are various workarounds - none actually show this in the help. I would therefore keep in simple and define two optional integer arguments and check within the function body:
@app.command(rich_help_panel="ANI methods")
def sourmash(  # noqa: PLR0913
    fasta: REQ_ARG_TYPE_FASTA_DIR,
    database: REQ_ARG_TYPE_DATABASE,
    # These are for the run table:
    name: REQ_ARG_TYPE_RUN_NAME,
    *,
    # The mode here is not optional - must pick one!
    mode: OPT_ARG_TYPE_SOURMASH_MODE = method_sourmash.MODE,
    create_db: OPT_ARG_TYPE_CREATE_DB = False,
    executor: OPT_ARG_TYPE_EXECUTOR = ToolExecutor.local,
    scaled: OPT_ARG_TYPE_SOURMASH_SCALED = method_sourmash.SCALED,  # this is 1000
    num: OPT_ARG_TYPE_SOURMASH_NUM = None,
    kmersize: OPT_ARG_TYPE_KMERSIZE = method_sourmash.KMER_SIZE,
) -> int:
    """Execute sourmash calculations, logged to a pyANI-plus SQLite3 database."""
    if scaled and num:
        msg = "ERROR: Options --scaled and --num are mutually exclusive."
        sys.exit(msg)
    elif scaled:
        extra = f"scaled={extra}"
    else:
        extra = f"num={num}"
    check_db(database, create_db)
    ...

Mutually exclusive arguments and default values is fiddly. Comment above updated with two ways that would work, with slightly different end-user experiences.

Thanks so much, @peterjc, for the such detailed review—really appreciate it, especially the tips on handling two mutually exclusive arguments. It would’ve taken me ages to figure that out on my own, and it’s great to see how proper (and elegant!) argument parsing is done.
I'll get to this as soon as I can, as I still need to prep for tomorrow's talk.

Also we need to add sourmash to the third-party requirements - any particular version?

Oh sugar! I added sourmash to requirements.txt rather than the third-party requirements. I will change this now. I have been working on the newest sourmash version (4.8.11), so I guess I will set this as the minimum for now?

We should put a comment block at the top of requirements.txt that is it python-only things (and in principle should work with pip install or conda install), like https://github.com/peterjc/thapbi-pict/blob/master/requirements.txt

Hi @peterjc. Unfortunately, I wasn't able to finish all of this before going on annual leave. I lost a bit of time trying to get the MacStudio to behave again.
I will be back in the office on the 18th, so don't worry if you can't get to this by then. I can handle what’s still left to do when I'm back.

That's great - have a good break 👋

See 35d335ce45ce9f1f5ac72e54be4a8a76f702ba45 for updating the repr.


Support Ruby 2.2.

Latest rubies area ready for support. Would help getting pry-doc updated, which is my motivation.

Thanks, I released the new version of lates_ruby, but currently I have no time to update pry-doc.


docs: close all v2 related issues and PR's from ionic-site repo

From @adamdbradley on November 9, 2015 15:0
https://github.com/driftyco/ionic-site/issues
https://github.com/driftyco/ionic-site/pulls
Copied from original issue: driftyco/ionic2#477

This is no longer an issue - we want docs issues to go on ionic-site now


Code example for dotnet core library is wrong

Issue moved from MicrosoftDocs/feedback#3245

Please respond to @zlidzen.

From @zlidzen on Wednesday, November 4, 2020 1:01:14 PM
Is your feature request related to a problem? Please describe.
Where :
https://docs.microsoft.com/en-us/dotnet/core/tutorials/library-with-visual-studio-code#add-a-console-app-to-the-solution
In pp.3 'Open ShowCase/Program.cs and replace all of the code with the following code.'
What :
Presented code example doesn't compile.
Why :
This method "void ResetConsole()" doesn't contain declaration or input parameter of "row".
Describe the solution you'd like
add input parameter in "void ResetConsole()" -> "void ResetConsole(int row)"
update call this method "ResetConsole();" -> "ResetConsole(row)"

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

ID: a3cbb75b-e0aa-13c6-c07d-65f0f0f68da2
Version Independent ID: 06ecbd1e-dad7-fbac-9160-dfce34df4d2c
Content: Create a .NET class library using Visual Studio Code - .NET
Content Source: docs/core/tutorials/library-with-visual-studio-code.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

Issue moved from MicrosoftDocs/feedback#3245

Please respond to @welcome[bot].

From @welcome[bot] on Wednesday, November 4, 2020 1:01:16 PM
Thank you for creating the issue! One of our team members will get back to you shortly with additional information. If this is a product issue, please close this and contact the particular product's support instead (see https://support.microsoft.com/allproducts for the list of support websites).

ResetConsole is defined within the Main method, so row is accessible in it.  Is it possible you placed an extra end curly brace before the ResetConsole declaration?

ResetConsole is defined within the Main method, so row is accessible in it.  Is it possible you placed an extra end curly brace before the ResetConsole declaration?


l20n grammar: allow {{.attr}} references

In the l20n toolbox, see the Entity Properties section:
<buttonClick "Click me"
    info: "{{buttonClick.title}}"
    title: "In order to press the button use ctrl+{{buttonClick.accesskey}}"
    accesskey: "c">

The l20n markup would be simpler if we could use:
<buttonClick "Click me"
    info: "{{.title}}"
    title: "In order to press the button use ctrl+{{.accesskey}}"
    accesskey: "c">

i.e. {{.foo}} would refer to the entity’s “foo” attribute. Is that possible?

I don't see why not. Sounds like a good addition!


Update styles and flow on create apps page

@bantic for you to review/merge. Pretty simple.
Adds styles and cancel functionality to the app creation page.
Also filters the count of apps and databases by if they are not dirty. This keeps the count from going up by one when createRecord is called. It may break in the case of edit (making it go down by one) but let us cross that when we get to it. Probably just a better prop to filter on.

@mixonic looks good. I'm curious about the isSaved vs isDirty for persisted apps/dbs CPs, but otherwise good to merge imo

@bantic isSaved is not a flag on models. I looked at some source and it seems isNew is what I really want to filter by. Updated here, and I've also added the component for matching sub-routes.


feat: F3 - RETURN opcode

Halt execution returning output data
Kakarot v0 reference:

implem: https://github.com/kkrt-labs/kakarot/blob/44dabfcab05a87f85bcfe0a457809059d11d83fb/src/kakarot/instructions/system_operations.cairo#L154
test: https://github.com/kkrt-labs/kakarot/blob/44dabfcab05a87f85bcfe0a457809059d11d83fb/tests/src/kakarot/instructions/test_system_operations.py#L60

Ethereum Foundation EELS:

implem: https://github.com/ethereum/execution-specs/blob/master/src/ethereum/shanghai/vm/instructions/system.py#L233

### Tasks
- [ ] Take reference to Kakarot v0 codebase and tests
- [ ] Take reference to Ethereum Execution Layer specs codebase
- [ ] Think of all the edge cases of RETURN opcode: memory expansion, offset + len > msize, offset > msize, len > msize
- [ ] Write tests for all edge cases
- [ ] Respect consistency with repo in terms of notation and naming conventions
- [ ] Implement the opcode

SinceGroupFrontierLogging OperationsIndex 1 is top of the stack. See PUSH.NotesThis instruction has no effect on the EVM state. See here.Stack inputoffset: byte offset in the memory in bytes.size: byte size to copy.topic1: 32-byte value.topic2: 32-byte value.topic3: 32-byte value.topic4: 32-byte value.Error casesThe state changes done by the current context are reverted in those cases:Not enough gas.Not enough values on the stack.The current execution context is from a STATICCALL (since Byzantium fork).Dynamic gasstatic_gas = 375
dynamic_gas = 375 * topic_count + 8 * size + memory_expansion_cost
For LOG4, topic_count is 4.
The memory expansion cost explanation can be found here.Estimate your gas costInput: offsetInput: sizeState: current memory sizeStatic gas + dynamic gas = 1875

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this link everytime you take an issue and check your issue isn't targeted by a known bug.

Important update: we are gathering some bugs in the Kakarot v0 codebase, we need to make sure each issue and each PR in Kakarot-ssj is aware of the lists of known bugs. Look at this tracking issue everytime you take an issue and check your issue isn't targeted by a known bug.

c'est pas comme ca qu'on imagine la vie


Distributed training

Hi,
thanks to all maintainers of this project, that's a great tool to streamline the building and tuning of a Faiss index.
I have a quick dumb question about the training of an index in distributed mode. Am I correct that the training is done on the host, i.e non distributed, and that only the adding/optimizing part is distributed ? After a quick look at the code and doc, I feel like that's the case, right ? If that's the case, would there be a possibility of training the index in a distributed fashion?

hey!
glad you like autofaiss
currently indeed autofaiss train the index on a single node.
Usually this is not a problem because the number of points that are used for training is a small part of the whole embedding set, usually up to 32x the number of clusters so for example 3M, even for a billion size index.
So it takes only up to an hour for training
However, it is indeed a technical possibility to distribute the training. If you have an use case that requires it, I advise you look into the pointers I put in that issue https://github.com/criteo/autofaiss/issues/101
What is your use case?

Well usually I tend to train my indexes with a number of points on the higher end of the recommended range, about 128 to 256 * nCentroids.
For about half a billion base vectors, I generally try indexes with 217, 218 centroids, that's around 20M to 80M points to cluster, for, hopefully, the best possible coverage of the distribution of my points.
Training with this many points takes a while ! Maybe that's being a bit too cautious and I could take far less points to train.
Another thing to factor in is that I generally can't use all the cores on the host, as it's generally under medium to heavy use. I usually only get a fraction of those cores, while the cluster cores are plenty and less frequently occupied.
Thanks a lot for your answer and your pointer to the distributed kmeans, It looks very promising.

Did you measure better knn recall by using so many training points ?
I used 2^17 centroids and 64x training points for a 5B embeddings index and it's working well.
In the past we did experiments with the number of training points and didn't see a big impact in using much more.


Feedback bug

In feedback page, the app shows navigation bar above the feedback fragment and the three buttons are not working on that page.

Issue is sort of fixed. While it works, the layout is messed up, instead of using the whole screen it uses the remaining space.


#Create a smash the  balloon game

create a game where user can click on the balloon to increase the points .
balloon should emerge at some random position each time after 1 second
you are free to set the time limit .

As Contributor in GSSoC"23 , I Would like to Work on This game . Can you Assign me this issue ?

hey, @vibgyor786  can you assign me this issue? I have experience in reactjs.

@shubham-shinde-442 kindly complete within 10 days

Sure

is it completed or still available

Please assign me this issue under GSSOC'23


About Visualization Code

Good Work!
But could you provide the code for visualization of patch embeddings which shown in Fig.4 in your paper.
Thanks a lot.

Thank you for your interest! I apologize for the current state of the visualization code—it’s still somewhat unorganized, and I haven't had the time to properly restructure it. However, I’m attaching the core part of the code below for your reference. I hope this helps to address your issue:
    def visualize_feat_map(self, patch_tokens):
        concat_patch_tokens = torch.cat(patch_tokens, dim=2)
        B, L, C = concat_patch_tokens.shape
        H = int(np.sqrt(L))
        concat_patch_tokens = concat_patch_tokens.squeeze(0).cpu().numpy()

        n_components = 1
        F_reshaped = concat_patch_tokens.reshape(-1, C)
        pca = PCA(n_components=n_components)
        F_pca = pca.fit_transform(F_reshaped)

        scaler = MinMaxScaler()
        F_pca_normalized = scaler.fit_transform(F_pca)

        visualized_image = F_pca_normalized.reshape(H, H, n_components)

        visualized_image = visualized_image * 255
        visualized_image = visualized_image.astype(np.uint8)

        return visualized_image


Add --config flag to pass the configuration file

I'm opening this issue because:

[ ] I have found a bug
[x] I want to request a feature
[ ] I have a question
[ ] Other

User should have the ability to choose the location of the configuration file to be used.
Example:
Default configuration file, ~/.qrcp.json, is used:
qrcp

Custom configuration file is used:
qrcp --config ~/my/dotfiles/qrcp.json

Released in version 0.6.3


It's 2020, remove support for Python 2

Library already supports Python 3, let's remove the expectation that we support Python 2 now that it's officially deprecated.
Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

FYI closed to make additional changes if we decide to do this (i.e. remove six dependency)


install python

like which python python npm package or python setup
Originally posted by @DevAndromeda in https://github.com/DevSnowflake/discord-player-downloader/issues/7#issuecomment-957913072

I am guessing this https://www.python.org/downloads/


Add NetworkPrefabs.Init() back to Plugin.cs

A mistake was made in this commit: https://github.com/EvaisaDev/LethalLib/commit/8469f19a2f2cf00464c632d4d168abd4bba76ed2
which removed NetworkPrefabs.Init(); in Plugin.cs, causing Network Prefabs to never be registered.

This is the root cause of v1.4.0s issues.

💀


Add support for libxml2 version 2.10.2

Add include for xlink.h to solve compilation error unknown type name ‘xlinkType’. See commit ebb17970 ("Remove unneeded #includes") in the libxml2 source code which removed this include.
FTP support and legacy modules have been disabled by default (see commit 776b0028 in libxml2). The test was updated accordingly.
Docbook has been disabled by default (see commit aeaf02c0 in libxml2). The test was updated accordingly.

Hi,
I am unable to install libxml-ruby on Arch Linux since libxml2 has been updated to version 2.10.2. I got the following error during compilation. This was also reported in this bug
ruby_xml_node.c:792:3: error: unknown type name ‘xlinkType’

I've tried to fix this problem but I'm not very familiar with C-programming. All errors and failing test can be explained from changes in the libxml2 library (see my commit message). Compilation seems to work (rake compile) and the test run successfully. Please have a look if these changes make sense.
Kind regards,
Martijn

Can this file just be included in ruby_xml_node.c then?

Can this file just be included in ruby_xml_node.c then?

Yes. I'm on arch linux and encountered the same error. I fixed it locally by adding #include <libxml/xlink.h> to ext/libxml/ruby_xml_node.c only.

I can verify that. Closing this PR in favor of a less invasive change.

I have the same issue on OpenBSD/adJ 7.2.

@cfis What is the less invasive change? Is there any progress on this?

Did you try the latest version? 4.0.0? Should be fixed there. If not let me know.

Did you try the latest version? 4.0.0? Should be fixed there. If not let me know.

It worked for me. Thank you.

Great!


README is not rendered properly on PyPI

See: https://pypi.org/project/pykwalify/
According to the packaging guide, it should be possible to create GitHub Flavored Markdown that works on PyPI: https://packaging.python.org/guides/making-a-pypi-friendly-readme/

Removed the history file that is likley the culprint for this problem. Next release should have a proper one rendered on pypi


Spacing feature request

Is your feature request related to a problem? Please describe.

The spacings being able to be modified would be fantastic.
from this:

to this:

It seems trivial, but this height bar in a wide screen page makes it a bit eye glairing.
setting the:

.theme-dark .title-31SJ6t.themed-Hp1KC_ {
  background-color: transparent;
}

to
.theme-dark .title-31SJ6t.themed-Hp1KC_ {
  background-color: transparent;
  height: 35px; /* <--- */
}

is a small tweak, but the only reason I now need to use a manual setup instead of an auto update system
Describe the solution you'd like

Perhaps have a Spacing option for certain windows, like the left panel, server panel, and the main page as 3 keys would be fantastic.

Additional context

Nope, thanks for the wonderful theme

I won't be adding options for this, however you can continue using the auto file and instead save this snippet (and other ones) in your Custom CSS to make changes.


[Fonts] Update icons

Reland #95007. It was reverted in #95966 to fix an issue with the font.
This adds a regression test for that issue

Gold has detected about 2 new digest(s) on patchset 3.
View them at https://flutter-gold.skia.org/cl/github/96115


custom font don't stretch up

Describe the bug
Recently I have decided to use custom font instead of default font. I have a little problem because some glyphs don't stretch up and it looks wierd when I display it. I don't know if there is a problem with my font or there is a bug in the library.
To Reproduce

Download font from link https://drive.google.com/file/d/1eYEoNWXoonjhSVx_AN_H0pghTOnjT8WC/view?usp=sharing
AddOverride above font
You will see that some glyphs don't stretch up ( Newton symbol, square roots)

Expected behavior
All glyphs should stretch up.
Additional context
Screens (Newton symbol, square root)

Does your font set OTF math tables? Without correct information you will not get stretched glyphs.

Ah a TTF font. You should edit your font to use OTF math tables instead.

Hey. I converted my font to OTF and I have a little problem. When I add my font, sometimes text is not visible. I thought that there is something wrong with my font, but I added default CSharpMath font and this bug also occurs. See screen below.
Here is the code:
Assembly assembly = Assembly.GetExecutingAssembly();
using (Stream stream = assembly.GetManifestResourceStream("latinmodern-math.otf"))
{
CSharpMath.Settings.GlobalTypefaces.AddOverride(new Typography.OpenFont.OpenFontReader().Read(stream));
}
textView.LaTeX = "Sample text $$x+3$$";
Is there something wrong with above script?


Cryptic error messages when providing a config file without the expected section

This accidentally happened to me and is easy to reproduce. Do the following in an empty directory, using pydocstyle 2.0.0:
echo '[pydcstyle]' > config
touch test.py
pydocstyle test.py --config=config

Then I get the following error message:
Traceback (most recent call last):
  File "/home/toon/.local/bin/pydocstyle", line 11, in <module>
    sys.exit(main())
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/cli.py", line 68, in main
    sys.exit(run_pydocstyle())
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/cli.py", line 43, in run_pydocstyle
    conf.get_files_to_check():
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/config.py", line 171, in get_files_to_check
    config = self._get_config(name)
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/config.py", line 262, in _get_config
    config = self._create_check_config(options)
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/config.py", line 383, in _create_check_config
    if cls._has_exclusive_option(options) or use_defaults:
  File "/home/toon/.local/lib/python2.7/site-packages/pydocstyle/config.py", line 497, in _has_exclusive_option
    cls.BASE_ERROR_SELECTION_OPTIONS])
AttributeError: 'NoneType' object has no attribute 'ignore'

It is too difficult to figure out what this error message means.

Thanks for reporting this @tovrstra. I agree there should be a better error message.

Implemented in #280 .

Thanks!


Prevent JSON parse error

Users can pass in { enabled: true } as the sortOptions, which means initialSortBy would be undefined and JSON.parse throws an error when you hand it undefined. So we need to default the value if it is not present.

Thanks @TheJaredWilcurt !


detecting name collisions during path serialization

The feature I would like to see, is testing for whether there are serialized name collisions.
This collisions can occur when the when one hierarchy contains an instance named foo_bar and also a sub hierarchy named foo and containing instance bar. When using get_path(hier_separator='_')  both of the above branches would result in foo_bar thus causing a name collision.
test.rdl file
addrmap test {
    default accesswidth = 32;
    regfile {
       reg {
           field {
           } val [32];
       } value;
    } conf;
    reg {
        field {
        } val [16];
    } conf__value;
};

peakrdl c-header test.rdl -o test

Since the latest version of PeakRDL-cheader is using a double underscore __ as a separator, I used the same in the example.
The generator will create a single field definition instead of two separate definitions due to a name collision.
I think there should be a way to detect such collisions.
It might be a global check for the given separator, or a check should be done each time get_path(hier_separator='_') is called.

I agree that exporters should check for such collisions.
Since this is specific to how each unique exporter will generate its content, and not something that would be appropriate for the systemrdl-compiler project to enforce, I will close this issue. I have created enhancement issue tickets in the three projects I own that are susceptible for such a collision and will fix individually.


"Try" doesn't seem to handle Odbc.Query() error

Hi,
I don't seem to be able to stop an Odbc.Query() error message from bubbling up.
fails = try Odbc.Query(ConnectionString, "SELECT * FROM NonExistentTable"),
handle = if fails[HasError]
             then Odbc.Query(ConnectionString, "SELECT * FROM ExistingTable")
             else fails[Value],

The above try statement fails to catch the error and I always end up with the "Table or view does not exist" error stemming from the first query. If I replace the first line with
fails = try error "forcing an error",

That explicit error is caught properly as described in the rather humble documentation.
Use case is a connector trying to retrieve additional metadata from catalog queries and falling back to a more limited metadata-scavenging method if the catalog query doesn't exist (connecting to an older version).

This is a common problem for the types where creating the value doesn't always mean accessing the underlying source.  If you know that the table size is relatively small, you can say
try Table.Buffer(Odbc.Query(...))
This will force the data to be read, which would trigger the error.


fix(record): update to use new FrooxEngine.Store assembly

This will address the new location of FrooxEngine's Record which is now located in FrooxEngine.Store.dll.

fixed in https://github.com/art0007i/CustomInspectors/releases/tag/2.1.1


Unicodeescape Error

python fix-host-save.py "C:\Program Files\uesave\uesave.exe" "C:\Temp\973569E445D6C70B5497E7A80B2D83F3" A3C50D3D000000000000000000000000 24DEFDD9000000000000000000000000 True
That is the command I am entering into Python 3.12 Command Prompt.
This is what I get in return:
Python 3.12.1 (tags/v3.12.1:2305ca5, Dec  7 2023, 22:03:25) [MSC v.1937 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.

python fix-host-save.py "C:\Program Files\uesave\uesave.exe" "C:\Temp\973569E445D6C70B5497E7A80B2D83F3" A3C50D3D000000000000000000000000 24DEFDD9000000000000000000000000 True
File "", line 1
python fix-host-save.py "C:\Program Files\uesave\uesave.exe" "C:\Temp\973569E445D6C70B5497E7A80B2D83F3" A3C50D3D000000000000000000000000 24DEFDD9000000000000000000000000 True
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: (unicode error) 'unicodeescape' codec can't decode bytes in position 16-18: truncated \uXXXX escape

Any idea what I am doing wrong?

Sorry, realized I forgot the bin folder, but same results.

I also tried both 3.10 and 3.12 of Python.  They both give the same errors.

Found a UI tool that does the conversion for you.  No need to worry about this post anymore.  Thanks!

fyi the problem you were having was you were entering the command in the Python interpreter instead of Command Prompt, but I'm glad you were able to find a solution anyway!


Drop custom serializer class in favor of Pydantic

Moved serializer into BaseModel.model_dump.
Unsure on this API because it requires using .model_dump(mode="json") to trigger using serializer because the Python mode is used internally for dumping the data into the prepared context.

Wait, I'm dumb, I can just add a new function.


[fix] flattening of self-join with constants

Stumbled upon this in my CBMA two-term conjunctive query testing.
Note that this branch is based on
https://github.com/NeuroLang/NeuroLang/pull/562, which should be merged first.
Let the single-rule program be
P(x) :- R(x)

and the query be ans :- P(a), P(b).
The flattening of the conjunction P(a), P(b) currently gives
R(a), x = a, R(b), x = b

which is not what we want.
That happens because the flattening does not handle well the self-join with
constants.
My strategy for fixing this is to create a new mixin that also freshens all
head variables of the rules of the program before the application of the MGU.
Now, the flattening of ans :- P(a), P(b) is
R(_f1_), _f1_ = a, R(_f2_), _f2_ = b

This seems to be a shattering problem not a flattening problem. Could you check that @tgy ?

I don't understand why the equalities need to be added instead of just replacing the variables in the head by the terms used when the head appears in the body of another rule.

You're right, I think the issue here was that the x = a and x = b equalities were added, which was not wanted.

Now the flattening correctly outputs R(a), R(b)


Where can I download the dataset of this network

Where can I download the dataset of this network

http://paintschainer.preferred.tech/downloads/

Dataset is not open because I dont have copyright.
you can download pre-trained model. ( copyright is reserved by me )