IDA 7 idaapi.dt_3byte conflicts with idaapi.dt_byte

In IDA version 7, idaapi.dt_3byte == idaapi.dt_byte.  This is causing all byte operands to be listed as size 3 instead of size 1.
I believe the issue could be fixed by reversing the order to idaapi.dt_byte and idaapi.dt_3byte in the sark.code.base.DTYP_TO_SIZE dictionary.
As an aside, there is no dt_3byte in the ida_ua module, which I believe is the module containing all the dt_* fields in the idaapi reorganization HexRays did.
I've sent an email to HexRays about this issue as well. I can share any updates they give me here.

This also breaks accessing the register name for byte sized registers.
This is because the incorrect size is being passed to base.get_register_name from Operand.reg.

Received a response from Hex Rays:

Yes, we removed the 3byte operand type from IDA. It is not really used in any processor modules except one or two, where we reimplemented it as a custom data type. Feel free to remove all references to it from your source code.

Closed in #68


Issue55

A couple of commits to close issue #55

The systemd unit files will also need to be changed to accommodate this rename.
Also the script files that generate the CNCI images need to comprehend the name change.
TravisCI today not check the validity of the CNCI and CNCI agent, hence it will not catch this error.

Coverage remained the same at 42.919% when pulling 77aa7bdeddf7a4bb1445125355f3c2f1481c8283 on albertomurillo:issue55 into 16e9a4cd8f3ef52c534106b449f16a4870066491 on 01org:master.

Coverage remained the same at 42.919% when pulling d63fb26b55ca648c6a5cba6f5353d6a65a3817c6 on albertomurillo:issue55 into 16e9a4cd8f3ef52c534106b449f16a4870066491 on 01org:master.

@albertomurillo looks good. I will test this on my cluster and merge.

Had to rebase make a few changes to merge this. Code is now merged.
There was a bug in one of the scripts which had to be fixed.

Oh i was just rebasing that one but got into a meeting. Thanks!


[CP-Stable][ios][platform_view] fix platform view clipping path intersection

Issue Link
https://github.com/flutter/flutter/issues/154249
Target
stable
Cherry pick PR Link
https://github.com/flutter/engine/pull/55007
Changelog Description
Fixes issues
https://github.com/flutter/flutter/issues/153904
https://github.com/flutter/flutter/issues/154249
https://github.com/flutter/flutter/issues/154664
Impacted Users
Apps with platform view (e.g. google ads, google maps)
Impact Description
When a platform view is clipped by multiple paths, it was incorrectly clipped. Some examples can be found in the linked issue.
Workaround
No workaround available.
Risk
low
Test Coverage
yes
Validation Steps
Go to any of the linked issue, and see the problem fixed after this PR.

@hellohuanlin can you help fully fill this out?  There shouldn't be any NAs in here, and the validation steps should be filled out.

Cherry pick PR Link
flutter/engine#54820

This isn't the right PR link, and this doesn't cherry-pick cleanly due to conflicts with #54335.  I'll open a new PR.

Thanks. I can also do it on Monday if you haven't yet.

any update on this ? Eagerly waiting for this to land in stable 🤞🏻

The stable PR is open at https://github.com/flutter/engine/pull/55007

@kishangohel I am holding this cherry-pick review as it does not currently have a valid changelog description.  Please follow our Hotfix documentation best practices.

We are blocked by this too. Is there an estimated time when and if this lands in stable?

Please follow our Hotfix documentation best practices.

@itsjustkevin The "Impact Description" section seems close to what's recommended in the link: "When a platform view is clipped by multiple paths, it was incorrectly clipped."


Attitude filter for multirotors: Let users choose between filters, default to old one

@LorenzMeier I didn't want to hijack your branch #2135, so here's a new one (also untested) with the default changed and the param docs updated.
When I read #2013 it sounds like there are still issues to be taken care of?

The open issues had been addressed based on a comment from Anton. But I'm ok with defaulting to the old one - however, the performance of the new estimator is likely significantly better, due to the offset estimation, so we should try to test and move to it as fast as possible.

Let us know when you've tested the change.

Change is tested and working.
We just felt that with these kind of drastic changes it's probably good to give it a few more tests. Underlying is much broader discussion though about "stable" which is a topic we need to address very soon..

I'm not sure we need much discussion about stable, rather, a better process. I think the objectives about the stable branch are pretty clear, however, there are a series of testing / review fails in the last days I'm unhappy about, too. I've already taken measures at ETH to improve the situation, however, we probably need additional help, e.g. from you guys. Mostly in being very quick and consistent about regular testing, and some of it is pretty hard at ETH.

I agree, I also meant the process.


PD not getting dismissed some times

Describe the bug
When dealing with Asuncións method, showing PD with Util class across app.
To Reproduce
Calling update record async Firestore method and showing Updating PD and post method calling Util.dismissProgressBar(). Please refer the code:
Util Class code :
Code :
static showProgressBar(BuildContext context, String message) async {
if(pDialog == null) {
pDialog = ProgressDialog(context,
type: ProgressDialogType.Normal,
isDismissible: false,
showLogs: true);
}
pDialog.style(message: message);
await pDialog.show();
}
static Future dismissProgressBar() async {
if (pDialog != null && pDialog.isShowing()) {
return pDialog.hide();
}
}
Calling Class Code:
try {
NetworkManager().updateChild(updatedChild).then((value) {
if (_profileImageStatus == AppConstants.profileImageUpdated) {
uploadFile(updatedChild).then((value) {
Util.dismissProgressBar().then((value) {
if (this.mounted) this.setState(() {});
});
});
} else {
//loggedUser = value;
Util.dismissProgressBar().then((value) {
if (this.mounted) this.setState(() {});
});
}
});
} catch (e) {
Fimber.e(e.toString());
}
Expected behavior
PD should get dismissed every time post method complete
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
Flutter 1.20
Smartphone (please complete the following information):
Simulator Nexus 6P
Additional context

It's because the pDialog is trying to hide itself when the dialog is not shown completely.
The isShowing flag is set after 200ms (the default  transitionDuration of the progress_dialog).
Some workaround you can wait to 200ms when the dialog failed to hide itself.
static Future dismissProgressBar() async {
if (pDialog != null) {
    bool isHidden = await dialog.hide();
    if (!isHidden) {
      Future.delayed(Duration(milliseconds: 200), (){
        dialog.hide();
      });
    }
}

please use await

Hi!
I'm having the same issue, but I'm using the await keyword.
await state.progress.fold<Future>(
      () => _downloadDialog.hide(),
      (progress) => _downloadDialog
          .show()
          .then((_) => _downloadDialog.update(progress: progress)),
    );
log('showing: ${_downloadDialog.isShowing()}');

The log is showing me false even though the dialog is being shown. Any idea?

Same issue here on Android


(refactor) Changes the WS and HTTP addresses for AscendEx endpoints.

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Changes the WS and HTTP addresses for AscendEx.
Tests performed by the developer:
All test cases pass.
Tips for QA testing:
[ch21228]

Opened in the wrong repo by mistake


[FLINK-11122][core] Change signature of WrappingProxyUtil#stripProxy(T)

What is the purpose of the change
This fixes a type inference ambiguity by the Java 9 compiler.
cc: @StefanRRichter
Brief change log

Change signature of WrappingProxyUtil#stripProxy(T)

Verifying this change
This change is already covered by existing tests, such as SafetyNetCloseableRegistryTest.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for reviewing @StefanRRichter. Merging.


README: add section 'usage on Jupyter notebooks'

Thanks for the heads-up. I think it is useful to have this snippet in a discoverable place

Good idea, thanks for the PR!


query regarding output file

In the genome_abandance.x.bed file:-
chrom | coordinate | gene | read id | read pos | kmer
chr7 | 5567320 | ACTB | e88129423ae1.fast5 | 1257 | AAACA
I want to know the coordinate is m6A modified or not.
Because in the ratio.x.tsv file some coordinates are missing which are present in genome_abandance.x.bed file, I search it in your readme file but I can't find so I just ask you.
Thanks in advance

The ratio.x.tsv is the final output, we fillter the genome_abandance.x.bed and then generate ratio.x.tsv.

ok, thanks but if the ratio.x.tsv file is also like genome_abandance.x.bed file then it's easy to extract all coordinates with gene name. I want to intersect the coordinates but first I have to arrange the coordinates only one in each row. So is possible to make a file like genome_abandance.x.bed file.
Thanks in advance

You can fillter the genome_abandance.x.bed through ratio.x.tsv file.

can you please tell me how because they are two different types of file one file have only 6 columns and a ratio.x.tsv file has multiple columns. I just try the grep command but it didn't help.
Thanks in advance

wget https://raw.githubusercontent.com/gaoyubang/nanom6A/main/fillter_output.py
python fillter_output.py result_final/ratio.x.tsv result_final/genome_abandance.x.bed
The output result_final/genome_abandance.x.bed.filler.bed is the filltered output.


ccx: BMP388 Static Pressure Sensor Driver

Create static pressure sensor driver
Interface: LPSPI1
Sensor: BMP388
Product page
Datasheet
Also needs to generate an interrupt

There's 3 interrupts that this sensor generates:

FIFO Watermark Interrupt
FIFO full interrupt
data ready interrupt

How do we need to react to each interrupt?

Bosch driver


Added troubleshooting for lsb_release not found

Lsb_release isn't in all distribution. Debian Stretch WSL on Microsoft App Store, for example, doesn't include it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/install-azure-cli-apt.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Cannot import name 'Bittrex'

I tried executing this small example and I'm getting the following error:
ImportError: cannot import name 'Bittrex'
Is it something I'm doing wrong?
This is my code:
from bittrex import Bittrex

btx = Bittrex(<key>, <key_secret>)

print(btx.get_balance("BTC"))

Try from bittrex.bittrex import Bittrex

Thank you!


Document the way the Kubernetes scheduler mode impacts Karpenter

Description
What problem are you trying to solve?
I have the following hypothesis.

With the default Kubernetes scheduler configuration of LeastAllocated, Karpenter is statistically more likely to disrupt pods and/or require consolidation than if the scheduler was running in MostAllocated mode.

I'd like to see this tested and documented. I suspect that in some cases, especially where eviction on some pods is disabled, that this would provide a significant price reduction.
How important is this feature to you?
I think having this officially documented would help users understand how to get the most out of Karpenter and would add weight to requests to cloud providers to support customising the scheduler.

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

CC @jonathan-innis (this is what I spoke to you at KubeCon EU about)

I'd like to see this tested and documented. I suspect that in some cases, especially where eviction on some pods is disabled, that this would provide a significant price reduction

Agreed. This sounds like a good feature to explore. Obviously, for us to give a recommendation for those that have direct control of their SchedulerConfiguration and the control plane, but also platforms like EKS should explore this alternative option to see if they get increased performance with Karpenter from this other setting.
I suspect that you are right about the performance, at least that we would be able to act more aggressively with consolidation since we'd have "less cluster churn" disrupting our current consolidation decision-making.

/triage accepted


🐛 fix GHA linter complaints

The introduction of the GHA linter found some issues regarding the shell scripts the workflows contain.  This PR fixes them.


The "Get rid of FAKE_VIDEOS" test doesn't seem to work, it won't clear after I remove the FAKE_VIDEOS array and any use of the variable on the "app.co

The "Get rid of FAKE_VIDEOS" test doesn't seem to work, it won't clear after I remove the FAKE_VIDEOS array and any use of the variable on the "app.component.ts" file
Author: Sean
Slide: Local,  Public

@sean-r-martin Is this still the case? I tried deleting the array and it seems to work

I'm not sure why it wasn't working for me before, but I also tried it and it's working for me now.


Add new HMC internals

Add new statistics from https://github.com/TuringLang/AdvancedHMC.jl/pull/124 to internals

Thanks, @sethaxen!


Update composer install instructions

I thought it would be a good idea to use the composer name that's in the composer.json

Agree, let's do this.


errorCallback from breeze manager.ExecuteQuery not executed

If I call an API (which throws an exception) using breeze's executeQuery method, the errorCallback function is never called.
var eq = new breeze.EntityQuery('TestError');                 instance.manager.executeQuery(             eq,              function(data) {                 console.log('success')             },              function(err) {                 debugger;             });
this is api method
[HttpGet]         public void TestError()         {             throw new Exception("test exception");         }
nor does it work with what is described here (see executeQuery method)
I suppose this is because aurelia-breeze replaces Q with ES6 Promises.
Is it possible to have the errorCallback called anyway? Or am I just doing something wrong?
I am not sure if this has to do with this issue:
https://github.com/jdanyow/aurelia-breeze/issues/44
If so, please feel free to remove this issue.

I think I solved it.
I have added an extra catch block to this original code
requestInfo.config.request.fetch(config.url, init).then(function (response) {
        response.json().then(function (data) {
          var breezeResponse = new HttpResponse(response.status, data, response.headers, requestInfo.zConfig);

          if (response.ok) {
            requestInfo.success(breezeResponse);
          } else {
            requestInfo.error(breezeResponse);
          }
        })**.catch(function (error) { //new catch block
            return requestInfo.error(error);
        });**
      }).catch(function (error) {
        return requestInfo.error(error);
      });

Now my errorCallback method is hit correctly. I could not inspect the .json method though as it was telling me "native code".

Missed this earlier, but it appears to be the same as my #44.
In addition my reinstated code includes error.json processing.


Revert "drop nette/utils dependency (#21)"

This reverts commit 71ecd2d7ebce972fd7d8f94af39f3089aafe84b2.
while working in PossibleTwigMethodCallsProvider  I realized there is indeed usage of nette/utils
this bug should have been caught by CI.. hmm 🤔

Indeed, seems like PHPStan missed something...

nette/utils was available because of a transitive dependency
mstaab@NB-COMPLEX-45 MINGW64 /c/dvl/Workspace/unused-public (recursiv)
$ composer why nette/utils
tomasvotruba/type-coverage 0.0.1 requires nette/utils (^3.2)

I see.
Btw, what is the reason to remove it? Is it blocker in some way? It requires PHP 7.2+ for this version: https://packagist.org/packages/nette/utils

I removed it, as I did not see immediate use of it by searching with the IDE (it seems I had a bug in my search term ;)).
btw: with https://github.com/maglnet/ComposerRequireChecker one could detect whether a project depends on symbols it does not declare direct dependencies to

I see :)
The nette/utils saved my soul so many times with these advances filesystem/json utils with high quality exceptions :)
I'd prefer the other way around, that checks the deps as well: https://github.com/composer-unused/composer-unused
E.g. here is one of setups https://github.com/symplify/symplify/blob/main/composer-unused.php


Talisdam only working on RTB, not digirom?

Was it one of these CircuitPython or lib updates we just did? But it's still really weird that it would work with RTB but not digirom.

RTB might be a bit flaky too. Need to track this down.


feat(module:tree): add custom filter and custom highlight func

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[x] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #4497
What is the new behavior?
User can custom filter search and custom highlight func
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

I prefer to use directive(like nz-tree-highlight-node) instead of function, can you check my PR pushed to your repo (https://github.com/NearZXH/ng-zorro-antd/pull/1)?
we can use it like:

Will this make out in the version 9?


Many bug fixes and misc. improvements

Using Thread.new instead of Process::detach fork for render_pdf because Process fork was causing
database connections to disconnect when the forked process ended
Fixing file corrupted when rendering_time is small
Make it possible to add a header/footer with page numbers
Add some new tests, including some tests that check PDF strings
Allow content disposition (inline or attachment) to be specified with a
X-Pdf-Disposition header.
Allow rendering to an output format other than '.pdf', like '.png' images (PhantomJS isn't just
for PDFs!)
Make the Middleware thread-safe
Add a simpler middleware, SynchronousMiddleware, that runs the phantomjs
command synchronously and doesn't use any of the fancy polling that
Shrimp::Middleware uses.   page, which doesn't even go away after the PDF file has
been downloaded.
Merge the options that were passed in to the middleware with the options
configured in Shrimp.config, so that you can override an option from
Shrimp.config by passing in a different value to the middleware.
Add config.debug option which outputs additional debugging information like the the phantomjs
command being run in case you want to manually run it
Change the default value for the tmpdir option to save to a shrimp* subdir under /tmp instead of
just saving everything directly under /tmp.
Remove some duplication and clean up some code
Improvements to the Readme file

These fixes look great! :)


Pass canvas through stream?

Summary
I am currently using LocalStream.getUserMedia() to stream to subscribers, but I am intending to impliment AR effects like masks/glasses etc... as these are rendered on a canvas, is it possible to pass through the canvas rather than the webcam feed?
Additional context
Heres a basic example;

Wrong repository, sorry.


Wrong constant?

In line 832 of dragonbox.h there is a right shift of 7 bits, whereas in the paper it is a right shift of 71 bits (section 5.1.7). It seems that 71 is indeed correct.

That is a shift by 71. The remaining 64 bits are in the umul128_upper64; that function multiplies, then shifts the return value by 64.
But yes, it certainly looks wrong to the uninitaited.

Oh, my bad! Thanks!


Changes related to Sign Transaction Protocol  #736

This is a WIP pull request which includes following changes related to the "Sign Transaction" Protocol to be implemented.
Below are the related details :
https://github.com/hyperledger/aries-cloudagent-python/issues/736
https://hackmd.io/5LzMhfsMQBevB5V2tKz4hA?view
It contains changes in connection record, connection related classes and new module is added "endorse_transaction"

I'll take a look at this PR tonight but FYI there are some issues with DCO (need to include "--signoff" for all commits) and code formatting that need to be addressed, as well there are some tests that are failing.

Build errors are like:
aries_cloudagent/admin/tests/test_admin_server.py:9: in <module>
    from ...config.default_context import DefaultContextBuilder
aries_cloudagent/config/default_context.py:18: in <module>
    from ..protocols.actionmenu.v1_0.base_service import BaseMenuService
aries_cloudagent/protocols/actionmenu/v1_0/base_service.py:6: in <module>
    from ....connections.models.conn_record import ConnRecord
E     File "/home/indy/project/aries_cloudagent/connections/models/conn_record.py", line 131
E       def __init__(
E       ^
E   SyntaxError: duplicate argument 'their_role' in function definition

Hi @HarshMultani @ankita-p17  thanks for all the great work!  I'd be interested in a walkthrough of the functionality, as soon as you have it ready to demo!

@HarshMultani  please take a look at this PR:  https://github.com/hyperledger/aries-cloudagent-python/pull/836
It adds meta-data to a connection record, we can use this to set the endorser/author roles.

@HarshMultani please take a look at this PR: #836
It adds meta-data to a connection record, we can use this to set the endorser/author roles.

Hi @ianco. This looks good. In this way we can change the key to "transaction_roles" and value as
{
"transaction_my_role" : "Author/Endorser",
"transaction_their_role" : "Endorser/Author"
}

A new pull request for the Sign Transaction Protocol has been raised and this is no longer of any use.
The new pr corresponding to this one is #870


chore: bump Std to leanprover/std4#541

Notable changes: lemmas were added in Std about gcd and lcm, that now have implicit arguments. Mostly this is a positive change in Mathlib, we can just delete the arguments. The one to consider in review is in ModEq.

What was the rationale for making the arguments implicit in Std?

What was the rationale for making the arguments implicit in leanprover/std4#538?

No rationale, it was done without reference to what mathlib was doing. i.e. I wrote them from scratch and took advantage of autoImplicit. :-) I'm happy to switch back if there is a reason to.
I do want a quick resolution on this PR, because I can't update bump/v4.6.0 until Mathlib master is using the HEAD of Std's main again, and this will cause problems the longer it is delayed.
Thus if you, or any maintainer, says "the change to implicit argument is bad", then I will promise to change them back, but not here.

bors merge

bors merge


Add a GitHub Action to use the Mac ARM runners

Our macOS CI runners are Intel (x86_64) based, so we're missing coverage for the ARM-based chips. These are now available via GitHub Actions.
OpenImageIO has got this set up on their project:
https://github.com/AcademySoftwareFoundation/OpenImageIO/blob/master/.github/workflows/macarm.yml
They set it up as a separate Action rather than doing it as part of their CI.  We should probably follow their lead.

The reason OIIO set it up as a separate action is to carefully control how often it's run. It's expensive enough that I was worried about blowing ASWF's GitHub budget if it runs on every push or PR. So I rigged it to only run on tags, releases, and PRs from branches whose name specifically indicates that we want to test on Mac ARM.

I noticed that you've already added a CI workflow for MacOS arm, but might I make a suggestion? Feel free to use FlyCI's M1 and M2 runners. Our runners are on average 2x faster and 2x cheaper than GitHub's AND we have a free tier for OSS projects (see below).
Install Instructrions
Easily replace your M1 runners:
jobs:
 ci:
-    runs-on: macos-latest
+    runs-on: flyci-macos-large-latest-m1
   steps:
   - name: 👀 Checkout repo
     uses: actions/checkout@v4

Or try the M2 runners:
jobs:
  ci:
-    runs-on: macos-latest
+    runs-on: flyci-macos-large-latest-m2
    steps:
      - name: 👀 Checkout repo
        uses: actions/checkout@v4

Pricing

Processor
vCPU
RAM (GB)
Storage
Label
Price on FlyCI
Price on GitHub

M1
4
7
28 GB
flyci-macos-large-latest-m1
$0.06
-

M1
8
14
28 GB
flyci-macos-xlarge-latest-m1
$0.12
$0.16

M2
4
7
28 GB
flyci-macos-large-latest-m2
$0.08
-

M2
8
14
28 GB
flyci-macos-xlarge-latest-m2
$0.16
-

500 mins/month Free for Public Repos
If your repo is public, then FlyCI offers 500 mins/month of free M1 runner usage with the flyci-macos-large-latest-m1 runner.
Best Regards,
Kiril Gantchev
CEO and co-founder of FlyCI


Remove application label

Error:Execution failed for task ':app:processDebugManifest'.
> Manifest merger failed : Attribute application@label value=(@string/label_application) from AndroidManifest.xml:17:7-48
  	is also present at [com.github.demoNo:AutoScrollViewPager:v1.0.1] AndroidManifest.xml:13:9-41 value=(@string/app_name).
  	Suggestion: add 'tools:replace="android:label"' to <application> element at AndroidManifest.xml:6:3-8:37 to override.

hi, it has been fixed. thank you.


Add role of input field as a non-required property

The role of the input field is now input. This is an abstract ARIA role, and should be replaced with a non-abstract role. Today the input field will fail an accessibility audit. https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_01
I've made the role a non-required property for the input field, but with default role still as input. This way the role can be changed if desired.

Cool! Thanks


feat: allow supression of s3 output

Allows suppression of s3 file operations to not flood the logs.

@peter-svensson While I don't see any issues with the code, I would appreciate if you added a test to the coresponding testsuite checking that the variable does make the code pass that option to the aws cli


Correct (if possible) way to convert win-acme certificates to .CRT and .KEY files?

Explanation and Question
On the LetsEncrypt forums I asked about the possibility and permissibility of using the service to create a certificate for foo.bar.co.uk, and then using the generated certificates on a local foo.bar.co.uk web application which would point to a local IP address when accessed within the local network. Fortunately this is apparently okay to do, albeit quite a manual process every 3 months.
The internal web application is Zulip and requires a .CRT file and a .KEY file in place.
How best should I do it? Thanks for any advice.
My incorrect attempt
I've noticed that win-acme creates the following files in ProgramData for a given website.
ca-foo.bar.co.uk-crt.der
ca-foo.bar.co.uk-crt.pem
foo.bar.co.uk.history.json
foo.bar.co.uk-all.pfx
foo.bar.co.uk-chain.pem
foo.bar.co.uk-crt.der
foo.bar.co.uk-crt.pem
foo.bar.co.uk-csr.pem
foo.bar.co.uk-gen-csr.json
foo.bar.co.uk-gen-key.json
foo.bar.co.uk-key.pem

To try to achieve it I followed this guide using the file foo.bar.co.uk-all.pfx that seemed (as a certificate non-expert) the most sensible one although I am unsure.
Using the following commands.
openssl pkcs12 -in foo.bar.co.uk-all.pfx -nocerts -out ~/foo.bar.co.uk-all.key

openssl pkcs12 -in foo.bar.co.uk-all.pfx -clcerts -nokeys -out ~/foo.bar.co.uk-all.crt

I got the output Enter Import Password: which I was not able to get past.
And again, thanks for any advice on this case.
Update
I updated to the latest version of win-acme and noticed that my incorrect attempt is not applicable to how the current system works, it seems that each website has a single .PEM and .PFX file.
Update 2
I was able to achieve this with just a few commands in the end taking the .PEM and .PFX files associated with the site (finding the ID and PFX password in the List scheduled renewals).
Getting the CRT.
openssl.exe pkcs12 -in [ID]-cache.pfx -clcerts -nokeys -out [ID]-cache.crt (then entering the PFX password)
Getting the encrypted key.
openssl.exe pkcs12 -in [ID]-cache.pfx -nocerts -out [ID]-cache.key (then entering password followed by PEM contents)
Getting the unencrypted key.
openssl.exe rsa -in [ID]-cache.key -out [ID]-cache-decrypted.key (then entering the PEM contents)
And it all works well locally.
Thank you for the fantastic project, I'll try get around to the Patreon donation side of things soon! :)

Hi @capavanni, it looks like you upgraded from 1.9.x to 2.0.x somewhere between your original post and your updates.
With version 2.0.9 you don't have to use openssl.exe. Instead you can use the PemFiles store plugin to get when you need. You can activate it from the command line or by using the interactive menu in the "full options" mode.
https://github.com/PKISharp/win-acme/wiki/Store-plugins#pemfiles

Perfect, thanks for the input, and also for the great program.
I see that option now. They use the PEM file extension but can I simply change the relevant files to CRT and KEY? Or I guess that Nginx would simply accept the file due to its correct content and file extension is arbitrary.
Anyhow, I am deviating from win-acme related discussion here, I'll close the case. Thanks!

I expect Nginx ignores the extension, but let me know when it doesn't, because I'm allergic to people having to do manual steps ;)

Hi Wouter. Thanks for the suggestion to use the plugin, it saved a lot of time. As you predicted, nginx didn't care that both file extensions had changed to PEM, and I was simply able to copy the files from the plugin and restart nginx with success.
Again, thanks for the fantastic piece of software :)


TSV output format

Hiya,
I'm here from the BVCN youtube channel and hoping to use KEGG-decoder for a project. I was hoping to use the TSV output to create my own heatmaps but the format I'm getting doesn't seem to be columns of pathways and rows of samples. I attached one of my function_out.list files.
kegg-decoder-output_matanushka1.txt

My guess would be that all of your gene IDs start with NODE_, right? KEGG-Decoder is a simple script and use the formatting in the first column to determine which genome to assign functions to.
Ideally you'd want something like:
GENOME1_GENE1	KO
GENOME1_GENE2	KO
GENOME2_GENE1	KO

KEGG-Decoder will provide output for GENOME1 and GENOME2. Everything before the first "_" is the name of the genome.
If you have something like:
NODE_GENOME1_GENE1	KO
NODE_GENOME2_GENE1	KO

KEGG-Decoder will provide one output for NODE


[DOCFIX]在调试向导中增加Alluxio远程调试章节

在google论坛Alluxio Developers中，我提出了名为“Just Debug it ! Debug it ! Remote Debug with alluxio (eclipse or Idea and others)”的主题，给出了远程调试alluxio的方法，开发者Bin Fan给予了回复，建议我把方法写在调试向导中，这样很多初学者会受益于此。

Can one of the admins verify this patch?

alluxio-bot, check this please

Hey @maobaolong , thanks for this PR. Would you please also add this part into the English documentation? BTW, please express the title and description of this PR in English also. Thanks.

thank you for your reply，the english version is previous PR, its number is 4537.
the link is https://github.com/Alluxio/alluxio/pull/4537

sorry ,forget to @RongGu
thank you for your reply，the english version is previous PR, its number is 4537.
the link is https://github.com/Alluxio/alluxio/pull/4537

One minor comment left. otherwise, LGTM. thanks for this improvement!

@apc999
Thanks for apc999's help, your help is careful and useful. Now I accept all your comment.

LGTM. thanks for the contribution. Looking forward to seeing you more in our community.


Remove hard coded values from dpsim-api + dpsim-worker example

Network files should be stored in sogno file server and the simulation request needs to include the fileID.

[x] translate file ID to URL in S3 storage using sogno file service ID
[x] request to dpsim API with file ID (zip of CIM files)
[x] create job description and send via AMQP to be retrieved by dpsim worker

@richardmarston could you please link the PR and close the issue?

Closed by https://github.com/sogno-platform/dpsim-api/pull/19


fix(curriculum): add two edge test cases to the sorted union problem

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #44588

Added two edge test cases to the problem as previously, even the code which wasn't handling some edge cases was passing the tests.
Fixed some issues with the code in response to some pipeline failures previously.

Hi @huyenltnguyen, thanks a lot for the feedback.
I have made a small change to the solution function as well and tested it on my end.
I believe it's now working as expected and giving correct answers to the edge cases as well. Do let me know if there are any other issues with the code.


Missing space for export default function doX() with spaceBeforeParentheses

Describe the bug
Missing space after the function name when using spaceBeforeParentheses for export default function doX() statement.
https://dprint.dev/playground/#code/GYVwdgxgLglg9mABAgpgCgJSIN4ChEGIBOKUIRSAjLgL664oAeADnEVIqJLAolAO5xMOfIRJkKiAEy16TVu0QATFMACGIADYcu0eEigALEuix5CxUuSQBmWkA/config/N4KABGBEBmCuB2BjALgSwPbwCIFNEBsBDAJ0LUwDoBnAB0MRwCEdp1icAFEneZACxxVBkAFxhkxWDhABfEEA/plugin/typescript
Input Code
export default function three() {
    return 3
}

Expected Output
export default function three () {
    return 3
}

Actual Output
export default function three() {
    return 3
}

The problem is actually with swc and the fact that it reports the exported default function declaration as a function expression. Opened an issue.

This is expected because that's controlled via "functionExpression.spaceBeforeParentheses": true. It's a function expression and not declaration at export default ... (not a bug in swc). It's why you can do export default function() {} without a name.
That said, for formatting concerns, this is often considered to be like a declaration and so maybe this option should change the behaviour of that too.

Oh wait, you might be right. I'll look into it more after I finish eating.

Yeah, this should be treated as a function declaration https://262.ecma-international.org/14.0/#prod-ExportDeclaration


How do I find issues for a specific document that may have already been created?

For example for https://github.com/MicrosoftDocs/visualstudio-docs/blob/master/docs/install/install-and-use-visual-studio-behind-a-firewall-or-proxy-server.md

Never mind, when to the bottom of the article and created an issue there: https://github.com/MicrosoftDocs/visualstudio-docs/issues/1486


Update overview.md

fixed Typescript to TypeScript

This repo is being archived in favor of docs.wormhole.com


Problem with Russian cyrillic characters and rule id 960024

Hi,
Our environment: Windows 2008 Server R2 Standard; IIS 7.5; ModSecurity IIS 2.7.7 (64-bit installer).
We have Russan version of website with pages that contain Russian (Cyrillic) letters. We don't have problem with ModSecurity IIS 2.7.7 when these pages are opened. Problem exist with search query that exist on site. If we put more than two Russian (Cyrillic) letters as search query value than access is blocked by rule 960024 (same is happening if we put more than two non-ASCII characters - e.g š,č,ć,ž,đ). These is problem since search query is blocked with rule 960024 (Meta-Character Anomaly Detection Alert - Repetative Non-Word Characters) when searching the regular Russian terms/expressions.
URL encoding is working. I also checked that t:urlDecodeUni exist in rule 960024. So question is why more than two non-ASCII characters as argument value are always blocked with this rule.
Best regards,
Sasa

Good afternoon.
Try to rule 960024 to replace the regular expression \W{4,} to [^A-Za-zА-Яа-яё0-9_]{4,}.
Just though pcre declared support for the localization of many languages, Russian letters are there apparently are not included.
On my web resources decided.

I will try this and send feedback: replace the regular expression \W{4,} to [^A-Za-zА-Яа-яё0-9_]{4,} in rule 960024.
Thanks for suggestion

Obviously \W{4,} is equal with [^A-Za-z0-9_]{4,}
If I want to include Russian characters question is following: is [АБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯабвгдеёжзийклмнопрстуфхцчшщъыьэюя] equal to [А-Яа-я] or to [А-ЯЁЙа-яёй]?
One of these two options probably needs to be used:

Replace \W{4,} to [^A-Za-zА-Яа-я0-9_]{4,}
Replace \W{4,} to [^A-Za-zА-ЯЁЙа-яёй0-9_]{4,}
Do you know which option is covering all Russian characters?

Useful link: http://stackoverflow.com/questions/14906232/regular-expressions-with-the-cyrillic-alphabet

Problem is deeper since regular expression in rule 960024 (and all other rules) matches only fist byte of hexadecimal value after url encode.
Example:  d1 8f is hexadecimal value of я; after url encode я is represented with %D1%8F.
If we replaced \W{4,} with [^я]{4,} in rule 960024 than all Russian characters which first byte of hexadecimal value starts with d1 will not be matched with this pattern. This means that not only repetitive я (more than 4 times) will be ignored but also other Russian characters which first byte after url encoding is having value d1. This is not good. Regular expression should match both bytes of hexadecimal value after url encoding. This is the main problem here - core rule set or waf engine need to be fixed in order to work correctly in these cases.
Useful link: http://www.utf8-chartable.de/unicode-utf8-table.pl?start=1024

good afternoon
Closer expression is [А-ЯЁЙа-яёй]
Although, I think that you are right and more reliable to use the full alphabet
[АБВГДЕЁЖЗИЙКЛМНОПРСТУФХЦЧШЩЪЫЬЭЮЯабвгдеёжзийклмнопрстуфхцчшщъыьэюя]
It is strange that you did not help. I have a number of false positives decreased somewhere 4.
What is left is not terrible (I use mode Anomaly Scoring Detection Mode).
With regard to your second message. I already wrote about this, too, though not so clever words :) until the response of some clear not received.
https://github.com/SpiderLabs/owasp-modsecurity-crs/issues/220

Basically I think main issue with false positive related to non-English characters are related to utlDecodeUni transformation function which discards higher byte of url encoding
Right solution will be to fix this bug and keep current syntax of relevant  rules. But as I saw from your and other similar open issues solution will be to modify rules in CRS 3.0. They will exclude this characters from matching.
I tried last version modsecurity 2.9.0 for iis with latest crs 2.2.9-9 and problem still exist with mentioned characters from other languages

So still wait for a new set of rules in the kernel release and under these rules. Expand on the test machine, check for bugs and hope that the number of false positives is reduced.
I tried at a few rules of version 3.0. I have a kernel version 2.7.7 under Apache. Such a rule 981 318, see false positives

I just check syntax of rule 960024 in crs 3.0.0-dev and it is the same as in crs 2.2.9.

hey @saymne can you describe the detail with utldecodeuni a little bit more... I apologize as an English speaker i am just dense to this problem :-P.  @zimmerle what are your thoughts here?

Hi,
Our environment:
Windows 2012 Server R2 Standard;
Apache/2.4.12 (Win64) OpenSSL/1.0.1m PHP/5.6.9;
ModSecurity for Apache/2.8.0; OWASP_CRS/2.2.9
We have Arabic version of website with pages that contains Arabic letters. We don't have any problem with ModSecurity when Post any English letters. Problem exist when Anyone POST comments with Arabic letters.
This Message form Apache Log :
Message: Access denied with code 403 (phase 2). Pattern match "\W{4,}" at ARGS:comment.
[file "C:/Apache24/conf/crs/activated_rules/modsecurity_crs_40_generic_attacks.conf"]
[line "37"] [id "960024"] [rev "2"] [msg "Meta-Character Anomaly Detection Alert - Repetative Non-Word Characters"]
[data "Matched Data: \xd8\xb5\xd8\xa8\xd8\xa7\xd8\xad \xd8\xa7\xd9\x84\xd8\xae\xd9\x8a\xd8\xb1
found within ARGS:comment: \xd8\xb5\xd8\xa8\xd8\xa7\xd8\xad \xd8\xa7\xd9\x84\xd8\xae\xd9\x8a\xd8\xb1"]
[ver "OWASP_CRS/2.2.9"] [maturity "9"] [accuracy "8"]
We Add these in httpd.conf m but still the same issue !!
SecUnicodeMapFile crs\unicode.mapping
SecUnicodeCodePage 1256
Best regards,
shadi

It appears that this is a problem with how PCRE handles foreign chars... So our regex's need to take account of this in CRS.

I am supporting site, where exist sub-sites in subdomains on 22 languages. For all languages except english exists problems with rule ID 960024.
best regards, Andrei
upwork.link

Closing this one as it's more related with the CRS rules and it's already being handled at OWASP ModSecurity CRS #220 and OWASP ModSecurity CRS #226.


Added in yolo script for Auto Annotation

regarding the Codacy errors, there's an issue with the way things are implemented, you can't do multilevel method names. See here: https://github.com/opencv/cvat/issues/652

@benhoff , thanks for the great contribution. As always.
I have a couple of questions and requests:

Could you please add information about the contribution to CHANGELOG.md file?
Did you expect any problems with provided instructions for converting yolo to IR? For example, I tried 3 different version of TF (2.0, 1.15, 1.13) and only the 1.13 version of TF worked as expected. The same for networkx python package.
Does your converted model can find something on the image: https://farm3.staticflickr.com/2372/2220900131_59625951f6_z.jpg? I cannot find an image where my converted model can detect something.

I got some matches, probably not as many as there should be.

There appear to be some issues with the accuracy of the converted model.
See: https://github.com/PINTO0309/OpenVINO-YoloV3/issues/32
and: https://github.com/PINTO0309/OpenVINO-YoloV3/issues/34
Intel link: https://software.intel.com/en-us/forums/computer-vision/topic/808504#comment-1938506
I'll add some notes about the conversion issues. Specifically NetworkX and Tensorflow. I was on tensorflow 1.12.3, but had similar issues on a different laptop that was on Tensorflow 2.0. The DLDT repo needs to restrict the dependencies!
See here for NetworkX info: https://github.com/opencv/dldt/issues/294

I'll add some notes about the conversion issues.

OK. I will wait. Please add a note to CHANGELOG.md. In general I really want to merge it ASAP because we had a lot of requests in the past for YOLO support.

Hi, @benhoff @nmanovic thanks for your contribution on this awesome project.
I recently starting work on this repo, I'm trying to apply my pre-trained yolov5(https://github.com/ultralytics/yolov5) model on the auto-annotation module.
Until now, I've converted the yolov5 model to onnx model, I also used the openvino API to convert the onnx model to openvino model, so I have the .bin .json. .xml .mapping files in hand.
What I'm trying to do is to create a python interpretation script similar to yours @benhoff , I adjusted the code, not matched layer name issue has been solved, while it seems that there exists another issue which is no boxes produced. (my original model has a very good mAP/accuracy performance)
Can you help solve this issue, could this be caused by source code changes since last year's PR? @benhoff

On the other hand, I found the codes changed a lot since I pulled it down a few days ago, may I happen to know the reason of it, is that because you are not willing to let the complex customized auto-annotation module exposed to new developers?

@mingweihe , we have changed the implementation of automatic annotation feature. It is open source and now it supports any DL frameworks out of the box. Please see how to enable the feature here: https://github.com/opencv/cvat/blob/develop/cvat/apps/documentation/installation.md#semi-automatic-and-automatic-annotation
Also you can find all DL models which are available for automatic annotation here: https://github.com/opencv/cvat/tree/develop/serverless

@nmanovic Thank you, serverless seems very cool! great job!

Hi @mingweihe
Did you find a way to use your custom pretrained yolov5 model for automatic annotation?
The links shared by @nmanovic  do not contain yolov5 functions if I'm not mistaken.


Moving the claim-formats section out to a distinct, PR-able registry

This would close #340.

I won't have time to review this until next week, please hold off merging it until then.


Cannot integrate in React App

So I'm using the monaco yaml plugin in combination with for my react web app.
I integrated it as described in the manual. Everything looks good in the development build. The workers are working normally.
As soon as I try to deploy my app for production, and I open the editor it can't load the web workers correctly.
It always tells me it could not load the web workers and following that an unexpected usage as in. #1385
Can you tell me if I have any misconfiguration or am I missing something ?
(to Mention i use seperate instances of monaco editor but always destroy them before launching a new one)
I'm also using react-rewired to override the webpack config, as you described in Issue #1385
These are my worker inputs.
// eslint-disable-next-line import YamlWorker from 'worker-loader!monaco-yaml/yaml.worker'; // eslint-disable-next-line import EditorWorker from 'worker-loader!monaco-editor/esm/vs/editor/editor.worker'
window.MonacoEnvironment = { 	getWorker(workerId: string, label: string): string { 		if (label === 'yaml') { 			return new YamlWorker(); 		} 		return new EditorWorker(); 	},
	new MonacoWebpackPlugin({ 			languages: ['yaml'], 			customLanguages: [ 				{ 					label: 'yaml', 					entry: 'monaco-yaml', 					worker: { 						id: 'monaco-yaml/yamlWorker', 						entry: 'monaco-yaml/yaml.worker', 					}, 				}, 			], 		})
Any ideas how I can fix this issue ? As i mentioned it only causes problems in production so I guess there is something wrong with my config.

Yes, sorry I was in a rush didnt expect the fast response. So here is the full formatted code.
config-overrides.js
const MonacoWebpackPlugin = require('monaco-editor-webpack-plugin');
const path = require('path');

module.exports = function override(config, env) {
	const isEnvProduction = env === 'production';
	config.mode = 'production'
	config.output = {
		filename: isEnvProduction
		? 'static/js/[name].[contenthash:8].js'
		: 'static/js/[name].bundle.js'
	};
	config.node = {
		__dirname: true
	};
	config.plugins.push(
		new MonacoWebpackPlugin({
			customLanguages: [
				{
					label: 'yaml',
					entry: 'monaco-yaml',
					worker: {
						id: 'monaco-yaml/yamlWorker',
						entry: 'monaco-yaml/yaml.worker.js',
					},
				},
			],
		})
	);
	return config;
};

MonacoEditor.js
// eslint-disable-next-line
import 'monaco-editor';
// Monaco
import * as monaco from 'monaco-editor';

import { editor } from 'monaco-editor/esm/vs/editor/editor.api';

// eslint-disable-next-line
import YamlWorker from 'worker-loader!monaco-yaml/yaml.worker.js';
// eslint-disable-next-line
import EditorWorker from 'worker-loader!monaco-editor/esm/vs/editor/editor.worker.js';
import { setDiagnosticsOptions } from 'monaco-yaml';

// eslint-disable-next-line
self.MonacoEnvironment = {
  getWorker(workerId, label): string {
	  if (label === 'yaml') {
		  return new YamlWorker();
	  }
	  return new EditorWorker();
  },
};

These are my currently used versions
    "monaco-editor": "^0.34.1",
    "monaco-editor-webpack-plugin": "^7.0.1",
    "monaco-yaml": "^4.0.3",

The Issue I referenced was meant to be this one:
https://github.com/microsoft/monaco-editor/issues/1385
I always get these errors in production, while in development everything works perfectly.

What is also pretty interesting, as I said above I have several tabs in my react app, which load the monaco yaml editor. Sometimes it works, but most of the time the errors shut down the editor.

You seem to be combining two conflicting solutions. One using MonacoEnvironment and worker-loader, and one using monaco-editor-webpack-plugin. You should remove one of them.
Also you mentioned “to Mention I use seperate instances of monaco editor but always destroy them before launching a new one”.
What do you mean by that? Could you share the code?

Alright, which solution would you recommend for my case ?
Yes it's a bit tricky to explain:
We have several Tabs in our Application, which are kept alive by React-Activation.
Once I reload I use the following code to reevaluate the problems list.
	useActivate(() => {
		if(editorItem && typeof editorItem.getModel === 'function') {
			const resource = editorItem.getModel().uri;
			if(resource) {
				const markers = editor.getModelMarkers({resource});
				handleEditorValidation(markers);
			}
		}
	});

But I don't think this affects the editor, because it is also working well in development

Personally I’m not a fan of relying on a framework-specific plugin for your bundler, so I recommend to use the method using MonacoEnvironment. However, if another method works for you, I’d say go with that.
Some people run into problems because they load the editor from a CDN. Some people aren’t even aware, because a wrapper library like @monaco-editor/react does that for them.
Also it’s important that a model and its uri represents a file on a (virtual) file system. You can’t create the same file twice. Also disposing an editor doesn’t dispose its model. I don’t think you’re running into this, but it’s something you should understand and pay attention to.

Alright, I just removed MonacoWebpackPlugin and went back loading the MonacoEnvironment through the URLs.
window.MonacoEnvironment = {
	getWorker(moduleId, label) {
		switch (label) {
			case 'editorWorkerService':
				return new Worker(new URL('monaco-editor/esm/vs/editor/editor.worker', import.meta.url));
			case 'yaml':
				return new Worker(new URL('monaco-yaml/yaml.worker', import.meta.url));
			default:
				throw new Error(`Unknown label ${label}`);
		}
	},
};

My new Webpack config looks like this now

const CssMinimizerPlugin = require('css-minimizer-webpack-plugin');
const HtmlWebPackPlugin = require('html-webpack-plugin');
const MiniCssExtractPlugin = require('mini-css-extract-plugin');

module.exports =  {
	output: {
		filename: '[contenthash].js',
	},
	devtool: 'source-map',
	resolve: {
		extensions: ['.mjs', '.js', '.ts'],
	},
	module: {
		rules: [
			{
				test: /\.css$/,
				use: [MiniCssExtractPlugin.loader, 'css-loader'],
			},
			{
				// Monaco editor uses .ttf icons.
				test: /\.(svg|ttf)$/,
				type: 'asset/resource',
			},
			{
				test: /schema\.json$/,
				type: 'asset/resource',
			},
			{
				test: /\.ts$/,
				loader: 'ts-loader',
				options: { transpileOnly: true },
			},
		],
	},
	optimization: {
		minimizer: ['...', new CssMinimizerPlugin()],
	},
	plugins: [new HtmlWebPackPlugin(), new MiniCssExtractPlugin({ filename: '[contenthash].css' })],
};

As before it works totally fine in Development but I still get these Errors in my Production Build.
Could it be like a Problem with CORS ?

I was able to solve the problem, it hasn't had to do anything with monaco editor or with monaco yaml. The main problem was our setup with keycloak. CORS blocked the worker scripts.


Changed query to detect MySQL storage engine.

update for INFORMATION_SCHEMA.ENGINES sometiomes do not exist!

update for INFORMATION_SCHEMA.ENGINES sometiomes do not exist!

Could you please create a Trac ticket with more details including the circumstances in which the query fails? Thanks!

Closing due to lack of follow up.


Work in Production environment?

I am working in Flutter App, I need this library to work generating new bitcoin adress, and make transactions.
Do you recommend work in production with this library?
Do you know some flutter app using the library in production?
Thanks.

We are using it on production, For basic functions as generating new bitcoin adress, and make transactions.It's safe to use, Another functions i can not make sure 100%. We are trying to improve this lib. Thanks for using this lib, feel free to open if you have any issues.


[BUG] Rich traceback not working proprely within notebook

Describe the bug
When the rich traceback is enabled in a notebook, the traceback is almost unreadable and show a criptic message:
tmp/ipykernel_1565558/3782956317.py:1 in <cell line: 1>                                  │
 [Errno 2] No such file or directory: '/tmp/ipykernel_1565558/3782956317.py'               │

Here is a screenshot that showcase the issue with a very simple traceback.

One could say that there is no point of activating the trackback from inside a notebook. However the problem persist when importing a library that enable rich traceback. (Ex in my case docarray or jina)

thanks in advance for any support :smiley:  !!
Platform

Click to expand
Linux, running from jupyter notebook
from rich.diagnose import report
report()

╭────────────────────── <class 'rich.console.Console'> ──────────────────────╮
│ A high level console interface.                                            │
│                                                                            │
│ ╭────────────────────────────────────────────────────────────────────────╮ │
│ │ <console width=93 ColorSystem.TRUECOLOR>                               │ │
│ ╰────────────────────────────────────────────────────────────────────────╯ │
│                                                                            │
│     color_system = 'truecolor'                                             │
│         encoding = 'utf-8'                                                 │
│             file = <ipykernel.iostream.OutStream object at 0x7f26f700bcd0> │
│           height = 100                                                     │
│    is_alt_screen = False                                                   │
│ is_dumb_terminal = False                                                   │
│   is_interactive = False                                                   │
│       is_jupyter = True                                                    │
│      is_terminal = False                                                   │
│   legacy_windows = False                                                   │
│         no_color = False                                                   │
│          options = ConsoleOptions(                                         │
│                        size=ConsoleDimensions(width=93, height=100),       │
│                        legacy_windows=False,                               │
│                        min_width=1,                                        │
│                        max_width=93,                                       │
│                        is_terminal=False,                                  │
│                        encoding='utf-8',                                   │
│                        max_height=100,                                     │
│                        justify=None,                                       │
│                        overflow=None,                                      │
│                        no_wrap=False,                                      │
│                        highlight=None,                                     │
│                        markup=None,                                        │
│                        height=None                                         │
│                    )                                                       │
│            quiet = False                                                   │
│           record = False                                                   │
│         safe_box = True                                                    │
│             size = ConsoleDimensions(width=93, height=100)                 │
│        soft_wrap = False                                                   │
│           stderr = False                                                   │
│            style = None                                                    │
│         tab_size = 8                                                       │
│            width = 93                                                      │
╰────────────────────────────────────────────────────────────────────────────╯

╭─── <class 'rich._windows.WindowsConsoleFeatures'> ────╮
│ Windows features available.                           │
│                                                       │
│ ╭───────────────────────────────────────────────────╮ │
│ │ WindowsConsoleFeatures(vt=False, truecolor=False) │ │
│ ╰───────────────────────────────────────────────────╯ │
│                                                       │
│ truecolor = False                                     │
│        vt = False                                     │
╰───────────────────────────────────────────────────────╯

╭────── Environment Variables ───────╮
│ {                                  │
│     'TERM': 'xterm-color',         │
│     'COLORTERM': None,             │
│     'CLICOLOR': '1',               │
│     'NO_COLOR': None,              │
│     'TERM_PROGRAM': None,          │
│     'COLUMNS': None,               │
│     'LINES': None,                 │
│     'JPY_PARENT_PID': '1483554',   │
│     'VSCODE_VERBOSE_LOGGING': None │
│ }                                  │
╰────────────────────────────────────╯

platform="Linux"

Hello, we have the same issue, has anyone found a way to avoid this?
I tried with rich 12.6.0, 11.2.0, and 10.16.2 and still get the same problem (No such file or directory: /tmp/... for each line of the traceback wich comes from the notebook) and I'm pretty sure it worked at some point before upgrading but I didn't manage to get it working again.
Thanks!

It appears as though by the time the traceback is displayed, Jupyter has removed the temporary file it created /tmp/.
There's been no relevant changes in Rich, so it looks to me like Jupyter has changed behaviour here. I think the best fix may be for Jupyter to ensure that the code isn't removed from the filesystem until after the traceback handlers is run.
There may be a workaround, which I'm considering. And that would be to snapshot the code at the time of the exception. But that is not without other potential issues.

Yeah indeed it looks like a Jupyter problem. Unfortunately as jupyter lab and notebook are widely adopted especially in ML I tend not to use the rich traceback anymore because it will break in notebook.
We had to remove from DocArray because of this.
If they were any solution even hacky one we would probably bring back the rich traceback

There doesn't appear to be any way to workaround this in Rich. The paths reported in the traceback don't exist, so there is no way to get the relevant code.
I've reported this issue to the Jupyterlab repo.

IIRC instead of using the files directly, Rich could/should use the linecache module. This is also where IPython and/or ipykernel writes to.
I onced patched Rich to use that, but I lost the changes.

An excellent example on how to use linecache is the traceback module in the CPython distribution.

Not sure that would help, since Rich needs the entire file to do Syntax highlighting.

each notebook cell is written to a 'file', which may not exist on disk, but is in the linecache module, see
https://github.com/ipython/ipython/blob/47abb68a6ad12539a161d03cd5a7a751cade9044/IPython/core/compilerop.py#L177
where it gets written to

so, linecache.getlines(filename) should get you the file lines you need.

Ah, well that explains a lot. Thanks.
I can use that to workaround the issue. Although I'm not keen on the implementation in ipython. The cache attribute is not documented or exported in __all__.

I agree, but seeing it being used in traceback.py and ipython, I thought this was probably poured in concrete a long time ago.

Thanks for the explanation @maartenbreddels
Tracebacks are fixed in 13.1.0 (just released).

great news thanks !

Great, fix looks nice, glad I could help!

Just popping on here to say that this issue was killing me for a few days... I had no idea this package existed, my jupyter just randomly broke.


Add movie and series list layout option

Summary
Adds a list layout option to the MoviesView and SeriesView which can be enabled via a new “Layout” settings preference.
Note: The additional top padding on iPhone/iPad looked a bit funky so https://github.com/ruddarr/app/pull/305/commits/4cf4477852faa8e76eab820e7272fe30fe766003 removes it. This is not reflected in the screenshots below!
Movies Layout

title
year | rating | runtime
genre
critic’s ratings
monitor status | profile | size on disk

Series Layout

title
year | rating | number of seasons
genre
next airing date
monitor status | profile | size on disk

Movies (Dark)
Movies (Light)
Series (Dark)
Series (Light)

Closing in favor of https://github.com/ruddarr/app/pull/309


Plugin not working on Windows

Hi I'm a big fan of the project but I have been having some issues on windows:
Vim Version:
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Nov  5 2023 23:49:43)
MS-Windows 64-bit GUI version with OLE support
Included patches: 1-2092
Compiled by appveyor@APPVEYOR-VM

Currently :DockerToolsOpen opens a panel with the following text:
# Press ? for help
"docker container ls" accepts no arguments.
See 'docker container ls --help'.

Usage:  docker container ls [OPTIONS]

List containers

The command docker container ls -a works correctly from any terminal and even from vim doing :!docker container ls -a so I'm unsure of what could be happening.
I tried to debug the plugin, but my understanding of vimscript is not the greatest and couldn't get any obvious leads.
Thanks again for the project and hope there's an easy fix for this one.

Hi @sysnet-ai, thank you for your support to the project!
Unfortunately, I do not have a Windows PC to reproduce this issue, can you please perform the following steps and let me know what is the result?

Switch to branch debug/docker-ls-format
Run :DockerToolsOpen
Paste the debug message here, which starts with docker-tools:

Hi @kkvh - Thanks a lot for the swift response. This is what I'm seeing in the new branch:

Can you execute the same command in terminal? Also, can you run the following vim command?
:set shellslash?

Thank you.

Hi @kkvh - It ended up being the shellslash.
I had it set (for other plugins), if I unset it this works like a charm.
I don't know if there's anything to be done - I'll let you close it then. Thanks a lot.

Hi @sysnet-ai thank you for letting me know that you have resolved the issue by having the shellslash option unset. Currently I prefer not to do anything with this option so I am going to close this issue for now.


Add fedora installation instructions

Here is the command which can be used to install all required dependencies for fedora:
sudo yum install libxkbcommon-devel
This is still very much WIP, so feel free to comment additional libs which are required.

yum is old and isnt used anymore by most people on fedora, instead there is dnf
i could find the dependencies

Hey, What about Flatpak?
This would solve the issue for all Linux distros.
This should actually be the preferred way, then there is no need for an extra thing.

sometimes flatpak software doesnt work properly/as intended (retroarch and discord are examples)

sometimes flatpak software doesnt work properly/as intended (retroarch and discord are examples)

What problems do you have with discord?
The only problem I have with the flatpak version occurs on the deb version either
Flatpak works best for most things, especially leafish

for discord specifically, rich presence doesnt work and for retroarch it didnt detect any of my controllers
flatpak could still work fine though, but some people don't want to install it or bother with it


Add Submit Button to top of Screen

@pbanaszkiewicz  could we add the Submit button to both the top and the bottom of the screen? This would help when I am adding information to the top of a profile instead of having to scroll all the way to the bottom being able to submit without scrolling.
Thanks!

+1!

@sheraaron @karenword this is quite straightforward change, but I need to know which pages would you like to have the top+bottom buttons?

In my experience the training request page is the main problem. Manually changing a status at the top (e.g. accept), then clicking ‘submit’ at the bottom involves a lot of scrolling!

On Oct 10, 2018, at 7:45 AM, Piotr Banaszkiewicz<EMAIL_ADDRESS>wrote:
@sheraaron https://github.com/sheraaron @karenword https://github.com/karenword this is quite straightforward change, but I need to know which pages would you like to have the top+bottom buttons?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/swcarpentry/amy/issues/1369#issuecomment-428600531, or mute the thread https://github.com/notifications/unsubscribe-auth/AVjuiBXpXXHTxEMURgnq8hbs_2LYFVJ1ks5ujgf3gaJpZM4XTYpy.

@pbanaszkiewicz also adding "submit" on the adding organizations, adding persons, accept and create event, workshop request form (event and task).
Thanks!


Add FLAC support without changing raylib source code

These changes should build raylib with -DSUPPORT_FILEFORMAT_FLAC=1.
This enables FLAC support as mentioned in #78.
I tested both TARGET_LINUX and TARGET_WIN64_MINGW and a sample FLAC file just works on both.
Notes:

Maybe the flag could be passed only to the raudio.c module build step. But I don't know if that definition changes anything else and may be needed for all modules anyway.

Maybe this could be done in a different way. I didn't want to assume too much and insert unrequested complexity, so a little more thought could shape this support introduction better than this bare minimum idea.

I don't think an additional configure option for the repo's build is needed (in order to have this be flexible/optional).

For now, the available configure options (hotreloading, bundled come to mind) seem fare more relevant than what this would offer.

I guess it mostly comes down to the downsides of turning this support on.

Don't know much about that, but may need to be looked into.
For now I can say that on my system the binary size increased:

from 5174kB to 5548kB, so ~7% (on TARGET_LINUX)
from 5723kB to 6085kB, so ~6% (on TARGET_WIN64_MINGW)

Looks alright! Thank you!


Improvement for metroae container ssh copyid

Would it be possible to make this command usable without specifying a host so that the ssh key is copied to all required hosts present in the deployment files?
Right now if I want to start using the metroae container for an already existing deployment I have to run metroae container ssh copyid  for all deployed components (except VSCs ofc).

This is a great suggestion. I will create a JIRA ticket to track this internally. Thank you.

METROAE-419 has been created.

Thanks a lot! Will this include all Nuage components (linux based, not VSC of course) as well as target servers? Or only a subset?

We will support Linux-based components and Linux-based target servers. We will use the credentials provided in credentials.yml or default credentials when credentials are not provided.

Okay. I was just looking at the dev branch and could only find this role: https://github.com/nuagenetworks/nuage-metroae/blob/dev/src/roles/make-copy-sshid-automatic/tasks/main.yml
And this one only includes VSD and VSTAT so I was wondering if the other components as well as their target servers are also considered.

@toweber you are correct. I was wrong. The implementation that we delivered to the dev branch only covers VSD and VSTAT. Your question suggests that we should add KVM-based target servers. Since we don't support upgrade of NUH and VNSUtils, it seems that there are no other VSP components to add support for. What am I missing?

One thing that I can think of would be if MetroAE only takes care of the "deploy" but not the "predeploy". For that scenario I guess it would also be valuable / convenient to be able to copy the ssh key to NUH (or vnsutils). Currently seeing this with my customer. They have a separate team that only takes care of VM provisioning in VMWare up until IP connectivity.
What do you think?

In v4.5 (about to be released) we have the following workflows:

metroae vsd copy sshid that copies '~/.ssh/id_rsa.pub' to all VSDs in the deployment. This can be helpful in the scenarios when the VM bring-up is done outside of MetroAE or we are doing an upgrade on something that existed previously.
metroae vstat copy sshid that does the above for the ES nodes defined in the deployment.
metroae container ssh copyid that copies the MetroAE container's ssh key to the specified host

from this discussion, I think additional functionality should be considered:

Add NUH to the list of supported nodes.
Add target servers to the list of supported nodes.
Add support for copying the container's ssh key to VSD, ES, NUH and target servers. (Not a separate workflow.
Add a workflow for copying to all nodes in a single operation.
Add support for specifying the ssh key file to copy - might not be the default in the non-container case.

@toweber what do you think?

METROAE-445 has been created to track this issue.

That sounds great!


[Spark] Makes LogSegment comparison aware of coordinated commits

Which Delta project/connector is this regarding?

[x] Spark
[ ] Standalone
[ ] Flink
[ ] Kernel
[ ] Other (fill in here)

Description
This PR updates the comparison (equals) method of LogSegments to consider coordinated commits. The reason for this change is as follows: The current comparison logic determines whether a LogSegment is equal to another LogSegment, if the deltas match and the modification time of the last delta is equal for both LogSegments. However, on a coordinated commits table, this may not be true anymore. This is because after a commit, we update the LogSegment by appending the last commit to it to create the post commit snapshot. In a coordinated commits table, this will be a UUID commit. However, by the next time DeltaLog.update is called, that UUID commit could have already been backfilled and so the LogSegment determined as part of the update call will contain the backfilled commit, which will have a different modification time compared to its UUID counterpart. As a result, even though the LogSegments represent identical table states, they would be identified as different.
The problem is that if we determine that the LogSegment from the previous snapshot is different to the LogSegment created by update, we will swap the old snapshot with the new snapshot. However, if these are indeed identical, then the swapping is not necessary and leads to Delta losing any cached state on the swapped out snapshot. This can lead to unnecessary slowdowns.
This PR fixes this issue by updating the LogSegment comparison logic. Instead of comparing the last file in the segment, we compare the minimum last backfilled file across both segments. For example, if segment 1 contains files 0.json, 1.json, 2.uuid.json, and 3.uuid.json and segment 2 contains 0.json, 1.json, 2.json, and 3.json, we compare 1.json and if that matches in both segments, we assume that the segments are equal (they also need to match in length of their deltas).
In addition to the LogSegment comparison fix, we also introduce a new member on Snapshot, that captures the correct last known backfilled version. In the example above, segment 1 would report 1.json as the last known backfilled version but segment 2 already contains 3.json as the last known backfilled version. To correctly determine, whether all commits for a specific table version (snapshot) have been backfilled, we update this state on the snapshot whenever we detect a stale LogSegment.
How was this patch tested?
Updated existing and added new unit tests.
Does this PR introduce any user-facing changes?
No

Flaky test? Not sure yet ...
https://github.com/delta-io/delta/actions/runs/10305728119/job/28527235347#step:7:21476
[info] - streaming *** FAILED ***
[info]   Expected exception org.apache.spark.sql.streaming.StreamingQueryException to be thrown, but no exception was thrown (IdentityColumnAdmissionSuite.scala:273)
[info]   org.scalatest.exceptions.TestFailedException:
[info]   at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:472)
[info]   at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:471)
[info]   at org.scalatest.funsuite.AnyFunSuite.newAssertionFailedException(AnyFunSuite.scala:1564)
[info]   at org.scalatest.Assertions.intercept(Assertions.scala:766)
[info]   at org.scalatest.Assertions.intercept$(Assertions.scala:746)
[info]   at org.scalatest.funsuite.AnyFunSuite.intercept(AnyFunSuite.scala:1564)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionSuiteBase.$anonfun$$init$$53(IdentityColumnAdmissionSuite.scala:273)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionSuiteBase.$anonfun$$init$$53$adapted(IdentityColumnAdmissionSuite.scala:272)
[info]   at org.apache.spark.sql.delta.test.DeltaSQLTestUtils.withTempDir(DeltaSQLTestUtils.scala:49)
[info]   at org.apache.spark.sql.delta.test.DeltaSQLTestUtils.withTempDir$(DeltaSQLTestUtils.scala:47)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionScalaSuite.withTempDir(IdentityColumnAdmissionSuite.scala:314)
[info]   at org.apache.spark.sql.delta.test.DeltaSQLTestUtils.withTempDir(DeltaSQLTestUtils.scala:32)
[info]   at org.apache.spark.sql.delta.test.DeltaSQLTestUtils.withTempDir$(DeltaSQLTestUtils.scala:31)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionScalaSuite.withTempDir(IdentityColumnAdmissionSuite.scala:314)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionSuiteBase.$anonfun$$init$$52(IdentityColumnAdmissionSuite.scala:272)
[info]   at org.apache.spark.sql.delta.IdentityColumnTestUtils.$anonfun$withIdentityColumnTable$1(IdentityColumnTestUtils.scala:76)
[info]   at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
[info]   at org.apache.spark.util.SparkErrorUtils.tryWithSafeFinally(SparkErrorUtils.scala:64)
[info]   at org.apache.spark.util.SparkErrorUtils.tryWithSafeFinally$(SparkErrorUtils.scala:61)
[info]   at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:99)
[info]   at org.apache.spark.sql.test.SQLTestUtilsBase.withTable(SQLTestUtils.scala:307)
[info]   at org.apache.spark.sql.test.SQLTestUtilsBase.withTable$(SQLTestUtils.scala:305)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionScalaSuite.withTable(IdentityColumnAdmissionSuite.scala:314)
[info]   at org.apache.spark.sql.delta.IdentityColumnTestUtils.withIdentityColumnTable(IdentityColumnTestUtils.scala:74)
[info]   at org.apache.spark.sql.delta.IdentityColumnTestUtils.withIdentityColumnTable$(IdentityColumnTestUtils.scala:71)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionScalaSuite.withIdentityColumnTable(IdentityColumnAdmissionSuite.scala:314)
[info]   at org.apache.spark.sql.delta.IdentityColumnAdmissionSuiteBase.$anonfun$$init$$51(IdentityColumnAdmissionSuite.scala:270)
[info]   at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.scala:18)
[info]   at org.scalatest.enablers.Timed$$anon$1.timeoutAfter(Timed.scala:127)
[info]   at org.scalatest.concurrent.TimeLimits$.failAfterImpl(TimeLimits.scala:282)
[info]   at org.scalatest.concurrent.TimeLimits.failAfter(TimeLimits.scala:231)
[info]   at org.scalatest.concurrent.TimeLimits.failAfter$(TimeLimits.scala:230)
[info]   at org.apache.spark.SparkFunSuite.failAfter(SparkFunSuite.scala:69)
[info]   at org.apache.spark.SparkFunSuite.$anonfun$test$2(SparkFunSuite.scala:155)
[info]   at org.scalatest.OutcomeOf.outcomeOf(OutcomeOf.scala:85)
[info]   at org.scalatest.OutcomeOf.outcomeOf$(OutcomeOf.scala:83)
[info]   at org.scalatest.OutcomeOf$.outcomeOf(OutcomeOf.scala:104)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:22)
[info]   at org.scalatest.Transformer.apply(Transformer.scala:20)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike$$anon$1.apply(AnyFunSuiteLike.scala:226)
[info]   at org.apache.spark.SparkFunSuite.withFixture(SparkFunSuite.scala:227)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.invokeWithFixture$1(AnyFunSuiteLike.scala:224)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTest$1(AnyFunSuiteLike.scala:236)
[info]   at org.scalatest.SuperEngine.runTestImpl(Engine.scala:306)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTest(AnyFunSuiteLike.scala:236)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTest$(AnyFunSuiteLike.scala:218)
[info]   at org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterEach$$super$runTest(SparkFunSuite.scala:69)
[info]   at org.scalatest.BeforeAndAfterEach.runTest(BeforeAndAfterEach.scala:234)
[info]   at org.scalatest.BeforeAndAfterEach.runTest$(BeforeAndAfterEach.scala:227)
[info]   at org.apache.spark.SparkFunSuite.runTest(SparkFunSuite.scala:69)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$runTests$1(AnyFunSuiteLike.scala:269)
[info]   at org.scalatest.SuperEngine.$anonfun$runTestsInBranch$1(Engine.scala:413)
[info]   at scala.collection.immutable.List.foreach(List.scala:334)
[info]   at org.scalatest.SuperEngine.traverseSubNodes$1(Engine.scala:401)
[info]   at org.scalatest.SuperEngine.runTestsInBranch(Engine.scala:396)
[info]   at org.scalatest.SuperEngine.runTestsImpl(Engine.scala:475)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTests(AnyFunSuiteLike.scala:269)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.runTests$(AnyFunSuiteLike.scala:268)
[info]   at org.scalatest.funsuite.AnyFunSuite.runTests(AnyFunSuite.scala:1564)
[info]   at org.scalatest.Suite.run(Suite.scala:1114)
[info]   at org.scalatest.Suite.run$(Suite.scala:1096)
[info]   at org.scalatest.funsuite.AnyFunSuite.org$scalatest$funsuite$AnyFunSuiteLike$$super$run(AnyFunSuite.scala:1564)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.$anonfun$run$1(AnyFunSuiteLike.scala:273)
[info]   at org.scalatest.SuperEngine.runImpl(Engine.scala:535)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.run(AnyFunSuiteLike.scala:273)
[info]   at org.scalatest.funsuite.AnyFunSuiteLike.run$(AnyFunSuiteLike.scala:272)
[info]   at org.apache.spark.SparkFunSuite.org$scalatest$BeforeAndAfterAll$$super$run(SparkFunSuite.scala:69)
[info]   at org.scalatest.BeforeAndAfterAll.liftedTree1$1(BeforeAndAfterAll.scala:213)
[info]   at org.scalatest.BeforeAndAfterAll.run(BeforeAndAfterAll.scala:210)
[info]   at org.scalatest.BeforeAndAfterAll.run$(BeforeAndAfterAll.scala:208)
[info]   at org.apache.spark.SparkFunSuite.run(SparkFunSuite.scala:69)
[info]   at org.scalatest.tools.Framework.org$scalatest$tools$Framework$$runSuite(Framework.scala:321)
[info]   at org.scalatest.tools.Framework$ScalaTestTask.execute(Framework.scala:517)
[info]   at sbt.ForkMain$Run.lambda$runTest$1(ForkMain.java:414)
[info]   at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
[info]   at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136)
[info]   at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635)
[info]   at java.base/java.lang.Thread.run(Thread.java:840)


Wrong fps label

When displaying the speed slider the knob has a middle position (10fps), but the label shows "1fps". The correct value is displayed as soon as I move the slider manually.
Tested with the following values:
transitionTime: 100
minSpeed:      0.1
maxSpeed:      20

Can you post your code or a jsbin ?

I added an example to the JSBin playground: http://playground-leaflet.rhcloud.com/vor/1/edit?html,output
I didn't add any times - the effect is the same with or without data.
The relevant lines of code are at the very end.

Ok this is because the initial speed is guessed from playerOptions. But with manual intantiation, you don't have that speed.
If you add
       playerOptions:{transitionTime: 100}

it works...
This bug also appear if the player speed is changed programmatically
The label should be updated with the speed when the player changes. For now, it's only updated when the user slide the knob.

:+1:


Go to definition fails for files outside the workspace

Open VSC
Select an interpreter that's outside the workspace
Type the following code

import os
os.getenv

Go to definitino of getenv
Now attempt to navigate in the code in os.py

As this is outside the workspace, go to definition & similar functionality is not available

validated


pullled the launch app step into it's own methed

Thanks for the contribution. Would you be able to make the small change I suggested in the commit comments? Would be happy to take this if so.


[JsonPatch] Add test operation support

From @kirthik on March 20, 2015 9:37
Test operation in code today does not compare anything. Removing the TODO from code. Pasting code snippet for reference.
var conversionResultTuple = PropertyHelpers.ConvertToActualType(
                typeOfFinalPropertyAtPathLocation,
                operation.value);

            // conversion successful
            if (conversionResultTuple.CanBeConverted)
            {
                // COMPARE - TODO
            }
            else
            {
                throw new JsonPatchException<T>(operation,
                    string.Format("Patch failed: provided value is invalid for property type at location path: {0}",
                        operation.path),
                    objectToApplyTo);
            }

Copied from original issue: aspnet/Mvc#2218

From @danroth27 on April 1, 2015 16:16
@kirthik Do we currently return a 400 level error response if someone tries a test operation?

From @kirthik on April 1, 2015 17:4
No. We don't. It does nothing today.

Curious.. Will the Test Operator be implemented at some point? If so, do you know when? I'm requiring functionality that the test operator provides and I'm considering my options here.

As a workaround I use:
// JSON.stringify([{ op: "test", path: "/key", value: "key_value"}])
// [FromBody] JsonPatchDocument<CustomRequestModel> jsonPatchDocument

var operation = jsonPatchDocument.Operations.SingleOrDefault(
a => a.OperationType == OperationType.Test &&
a.path.Equals("/key", System.StringComparison.CurrentCultureIgnoreCase));

// (string)operation.value


Polkadot Parachain Slot - Prepare for Slot Auction

We want to prepare for slot auction date: August 25 (Auction 26-31)
Next round: 25/08 (Auction 26)
Rewards Policy:

Do we launch a crowdloan campaign ?
If so:

Need to have 500 DOT transferrable
Set the crowdloan cap
Set desired end of crowdloan

Closed with Polkadot crowdloan live


[go_router] Remove urlPathStrategy

The urlPathStrategy API is a convenience based on documentation on URL strategies for Flutter apps that target the web. We should remove this API and recommend using flutter_web_plugins instead:
import 'package:flutter/material.dart';
import 'package:flutter_web_plugins/url_strategy.dart';

void main() {
  usePathUrlStrategy();
  runApp(ExampleApp());
}

is this really necessary ? I think having it there is nice

Can't we depend on the package in the go_router instead and fix it in the package?

merged into feature branch. check https://github.com/flutter/flutter/issues/108044 for progress


[CYPRESS]: Use the SDK to process a payment.

Title: Verify Make and View a Test Payment

Description:
This task involves testing the "Make a test payment" feature from the homepage. The user will go through the checkout process, entering payment details and viewing the payment summary upon successful payment.

Test Case: Make and View a Test Payment

Pre-requisites: None

Steps:

Go to the homepage.
Click on "Try it out" in the "Make a test payment - Try our unified checkout" message box.
Verify the user is redirected to the test checkout page.
Select the currency from the dropdown (e.g., "EUR").
Enter the amount (77) in the text box.
Click on "Show preview".
Verify that the SDK preview is rendered with the following elements:

"Card number" text box
"Expiry" text box
"CVC" text box
"Save card details" checkbox
"Pay" button with the currency and amount

Enter the test card details into the SDK:

Card Number: 4242 4242 4242 4242
Expiry: 01/27
CVC: 492

Verify that the card details are displayed correctly in the respective text boxes.
Click on the "Pay" button.
Verify that the "Payment Successful" message is displayed.
Click on "Go to payment".
Verify that the user is redirected to the payment details page.

Test Data:

Currency: EUR
Amount: 77
Card Number: 4242 4242 4242 4242
Expiry: 01/27
CVC: 492

Expected Results:

The user is redirected to the test checkout page after clicking "Try it out."
The selected currency ("EUR") should be reflected in the dropdown.
The amount (77) should be entered and displayed correctly in the text box.
After clicking "Show preview," the SDK preview should be rendered with all necessary elements (card number, expiry, CVC, etc.).
Test card details should be entered correctly, and the payment form should reflect the correct input.
Upon clicking "Pay," a "Payment Successful" message should be displayed.
The user should be redirected to the payment details page upon clicking "Go to payment."

Acceptance Criteria:

The test checkout page is accessible from the homepage.
The currency and amount are correctly entered and reflected in the form.
The SDK preview renders with the appropriate fields.
Payment details are correctly submitted, and the payment is processed successfully.
The user can view the payment details page after successful payment.

This structured format ensures that all steps, test data, and expected outcomes are clearly defined for testing the payment process.

Hi , I would like to work on this issue. Please assign it to me.

Hey @gorakhnathy7, I would like to work on this issue as this is the prerequisite for other issues.
As ankesh has also commented on verification of time range in payment operations page issue already, can I have this one?

Sure @s-vamshi assigning this issue to you!
Hey @Ankesh2004 As i see you have commented on multiple issues, but we won't be able to assign all at once.
let me know if you want to work on any new issues.
Thanks!

Hey @s-vamshi
Please note that for this issue, submit a PR on https://github.com/juspay/hyperswitch-control-center, and link to the issue.

hey @JeevaRamu0104 @gorakhnathy7
i am trying to create a test payment using the try it out button on the home page by spinning up hyperswitch-control-center and hyperswitch server locally,
when i click on show preview button the sdk preview is breaking.
can someone guide me to fix this issue?

Hi @s-vamshi you need to configure the sdk url in the config file or set it has env.You also need to run the sdk locally
export default__endpoints__sdk_url="http://localhost:9050";

Hey if i am not wrong just setting the url in https://github.com/juspay/hyperswitch-control-center/blob/main/config%2Fconfig.toml#L7 right?

yes @s-vamshi

Hey @gorakhnathy7,
Can i fill the form with the following issue also?
https://github.com/juspay/hyperswitch/issues/6125#issue-2551184783

Hey @s-vamshi if you wish to raise more PRs, feel free to wait till all the PRs are merged.


The netebpfext fuzzer should check for memory leaks

The netebpfext fuzzer should check for memory leaks.

Consider the cases where there can be memory leaks on error-paths other than memory allocation.


Fix URI is not hierarchical #59

One proposition for fix "URI is not hierarchical"

if this works, this is the most elegant fix I think out of three all of us proposed for #64

+1 please

+1


Missing subchain, fail when no chain with start block

If given a start block that didn't exist, or there was an unparented block between the start and end blocks, missing_subchain would yield a chain only as far back as the first unparented block, when searching backwards from the end block. The user was none the wiser.
Instead, fail in those cases.
Tested on a Rosetta-generated archive db, by creating an unparented block in the middle of what was a chain between the start and end blocks.
Closes #7911.

!approved-for-mainnet


File not found error

Hi, I am using gdrive-upload-action to upload an .pdf output to google drive. But got an file not found error
:error::creating file: &{AppProperties:map[] Capabilities:<nil> ContentHints:<nil> ContentRestrictions:[] CopyRequiresWriterPermission:false CreatedTime: Description: DriveId: ExplicitlyTrashed:false ExportLinks:map[] FileExtension: FolderColorRgb: FullFileExtension: HasAugmentedPermissions:false HasThumbnail:false HeadRevisionId: IconLink: Id: ImageMediaMetadata:<nil> IsAppAuthorized:false Kind: LastModifyingUser:<nil> Md5Checksum: MimeType: ModifiedByMe:false ModifiedByMeTime: ModifiedTime: Name:resume_Harry_Wu.pdf OriginalFilename: OwnedByMe:false Owners:[] Parents:[***] PermissionIds:[] Permissions:[] Properties:map[] QuotaBytesUsed:0 Shared:false SharedWithMeTime: SharingUser:<nil> ShortcutDetails:<nil> Size:0 Spaces:[] Starred:false TeamDriveId: ThumbnailLink: ThumbnailVersion:0 Trashed:false TrashedTime: TrashingUser:<nil> Version:0 VideoMediaMetadata:<nil> ViewedByMe:false ViewedByMeTime: ViewersCanCopyContent:false WebContentLink: WebViewLink: WritersCanShare:false ServerResponse:{HTTPStatusCode:0 Header:map[]} ForceSendFields:[] NullFields:[]} failed with error: googleapi: Error 404: File not found: ***., notFound

I am sure that that .pdf file is successfully generated and I also tried to upload the source .tex file but it also failed with the same error.
Here is my yml file:
name: Build_and_release
on: push
jobs:
  build_and_release:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v2

      - name: Compile
        uses: dante-ev/latex-action@latest
        with:
          root_file: resume.tex

      - name: upload to google drive
        uses: team-tumbleweed/gdrive-upload-action@main
        with:
          filename: resume.pdf
          name: resume_Harry_Wu.pdf
          folderId: ${{ secrets.folderId }}
          credentials: ${{ secrets.credentials }}
          encoded: false

Really apperciate your work and help!
Thanks!

Hi! Thanks for creating this issue!
I am currently at KubeCon NA 2022, so I'm pretty swamped.
I will try to solve this as soon as I get back!
Thanks for your understanding!

Hi! I did some testing and everything seems fine on our end. Since it's throwing a googleapi error, I think it might be that you either didn't set the right permissions for the Google Service Account or that the folder id is wrong. Can you confirm that this is how you got your folder id?

Thanks!

Thanks for your reply.
Turned out that I didn't share the folder to the service account
it works now, thank you so much!

Great! If you have any other issues, let me know!


Feat: Add link checker to CI workflow

This adds our link checker to the CI workflow. It shows up independently from the slug checker in the PR & push checks.
If any broken links are found, it returns an error code, which causes the CI check to show a red cross. It also attempts to find the Markdown source files containing the broken links and annotates the lines containing the broken links. The annotations can be seen in the "changed files" view of PRs and commits.

PTAL @delucis

Looks good to me! Any reason we should wait before merging?

I added the results of our discussion to the "minor changes" commit. I sent you a screenshot how the annotations look like on Discord. Think we should be ready to merge now. :)


"SMTP data successfully saved" message shows before submitting data.

"SMTP data successfully saved" message shows before submitting data. Notice is not dismissable.

https://github.com/mjones129/private-links/pull/30


Request for new term [vertebral artery dissection]

Label
vertebral artery dissection
Synonyms
No response
Synonym type
None
Definition
Vertebral artery dissection (VAD) is a rare cause of stroke in the general population; however, represents one of the more common causes of stroke in patients younger than 45 years of age. Its signs and symptoms can be vague, and diagnosis can be elusive. Spontaneous dissections have been reported. However, incidental minor trauma often precipitates this potentially dangerous condition. Often some neck distortion such as chiropractic manipulation, bending of the neck, or blunt trauma causes the dissection. The dissection of the artery may ultimately lead to a stroke which often can be delayed for days following the acute dissection.  Source: https://www.ncbi.nlm.nih.gov/books/NBK441827/
Parent term
cervical artery dissection
Children term(s)
No response
ORCID Identifier
No response
Website URL
https://www.ncbi.nlm.nih.gov/books/NBK441827/ https://www.cureus.com/articles/51091-vertebral-artery-dissection-a-pain-in-the-neck#!/ https://www.ncbi.nlm.nih.gov/pmc/articles/PMC2588305/ https://emedicine.medscape.com/article/761451-overview
Additional comments
We received a message from an inquirer who is a stroke survivor asking if we  had this condition listed at NORD.  Currently it is not listed in NORD and it is not in Orphanet either (I will write a ticket to them for their consideration). Orphanet does list familial cervical artery dissection  (note that cervical artery dissection occurs when there's a tear in either a carotid or vertebral artery).  It is a rare condition.
Please let me know your thoughts.  Thank you, Gioconda Alyea

Dear @galyea123,
Thank you for reaching out to the Mondo Disease Ontology, part of the Monarch Initiative!
We have received your ticket and will review it as soon as possible.
If you have any additional information or updates relevant to your inquiry, please add them in a comment on this ticket. Your input is valuable in helping us understand and resolve the issue effectively.
Thank you for your patience and for your contributions to Mondo!
Sincerely,
The Mondo Team

Hi Gioconda,
Thank you for your continued collaboration as we look to improve MONDO.
As stated in your definition for Vertebral Artery Dissection: "Vertebral artery dissection (VAD) is a rare cause of stroke in the general population". VAD is a cause of stroke, and not a disease itself. Further documentation on what are and are not included as disease definitions in MONDO can be found here: https://mondo.readthedocs.io/en/stable/editors-guide/exclusion-reasons/#exclusion-reasons. This would most likely be considered an 'external factor' exclusion reason.
If you have any further questions or comments, feel free to reach out.
Yousif

Hi Yousif!
Thank you for your response.
To my understanding vertebral artery dissection is, in fact, a disease, as it has specific pathogenesis (an expanding hematoma in the vessel wall is the root lesion in vertebral artery dissection) natural history, clinical features, and treatment. It is not just a cause of stroke.  I believe the following article may be helpful to clarify my points. https://www.ncbi.nlm.nih.gov/pmc/articles/PMC2588305/
In addition, please note that there is an ICD-10 code I77. 74 for Dissection of vertebral artery is a medical classification as listed by WHO under the range - Diseases of the circulatory system.
Of note, a disorder in the Orphanet database can be a disease, a malformation syndrome, a clinical syndrome, a morphological or a biological anomaly or a particular clinical situation (in the course of a disorder). We also sometimes include consequences of a medical procedure (ex: Scarring in glaucoma filtration surgical procedures).
Thank you!
Best,
Gioconda Alyea

Hi!  I was looking in 7976 but I am not sure if you deciding on adding the term?  Thank you, Gioconda

Hi Gioconda!
Yes, we will be adding it to our 2024-08 release.

[like]  Gioconda Alyea reacted to your message:

From: Yousif @.>
Sent: Monday, July 22, 2024 10:35:10 PM
To: monarch-initiative/mondo @.>
Cc: Gioconda Alyea @.>; Mention @.>
Subject: Re: [monarch-initiative/mondo] Request for new term [vertebral artery dissection] (Issue #7593)
Hi Gioconda!
Yes, we will be adding it to our 2024-08 release.
—
Reply to this email directly, view it on GitHubhttps://github.com/monarch-initiative/mondo/issues/7593#issuecomment-2243923529, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4QAWDYAKERZBELAJW3NRALZNWCJ5AVCNFSM6AAAAABGEQFMG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBTHEZDGNJSHE.
You are receiving this because you were mentioned.Message ID: @.***>
Any review, use, distribution or disclosure by others is strictly prohibited.


feat(Structures): added context-aware structures

Fixes #14

I like where this is going. I think we need to throw an error on this.set and use super.set everywhere. Also don't we need to export a const instance instead of the bare class?