Fix some files not being deleted

Questions
Answers

Description?
Solves a problem of some files not being deleted during upgrade. The common culprit was that these files had /admin in their paths.

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes https://github.com/PrestaShop/PrestaShop/issues/34385 (partially, don't close)

Sponsor company

How to test?
Upgrade from <IP_ADDRESS> to 8.1.2 and check that these files from the issue are gone.

How I fixed it

During my findings like in https://github.com/PrestaShop/autoupgrade/pull/651, I got to the step where it builds a list of files to get removed. It builds this list by comparing current version XML to a XML of the target version.
Files that are on that list, are scheduled for deletion.
However, I found out that because these files had universal /admin path in them, it was replacing it by our own admin folder. $list_files_diff[$k] = str_replace('/' . 'admin', '/' . $admin_dir, $path);
However, it was replacing it not only in the beginning, but also in the middle of the paths, resulting in paths of some files to be changed to nonsense.

Before

After

Hi @Hlavtox
Thank you for your PR, I tested it and it seems to works as you can see :
Untitled_ Nov 23, 2023 5_44 PM.webm
But currently we can see that we can't go on the BO, we can go in FO.
When we want to go again in BO to enable debug mode or anything else, we've this error :

Thank you

@AureRita As discussed privately, caused by not filling a session table, fixed here - https://github.com/PrestaShop/autoupgrade/pull/655
A workaround is to clear cookies. :-) This will create a new session and the user will be able to log in. This PR is not the problem.


[JENKINS-30600] let CliGitAPIImpl rely on FilePath+Launcher remote API

https://github.com/jenkinsci/git-client-plugin/pull/212 rebased on master

This purports to fix JENKINS-30600 IIUC?

not only for  JENKINS-30600, but also would fix this one

too much conflicts to rebase this 14 months old PR.


fix: send attributes in all caps to API

Complete each step to get your pull request merged in. Learn more about the workflow this project uses.

[ ] Assign members of your team to review the pull request.
[ ] Wait for pull request status checks to complete. If there are problems, fix them until you see that all status checks are passing.
[ ] Wait until the pull request has been reviewed and approved by a teammate
[ ] After pull request is approved, and you determine it's ready add the label "Ready to merge" to the pull request. A bot will squash and merge the pull request for you after the label is added.

Pull request title looks good 👍. If this pull request gets merged, it will cause a new release of the software.
Example: If this project's latest release version is 1.0.0. If this pull request gets merged in, the next release of this project will be 1.0.1.
To merge this pull request, add the label Ready to merge to this pull request and I'll merge it for you.

Closing. Tests exist in production that verify the SDK does not modify given attributes. Including when key is all caps.


add temporary file to prove ownership

adding a html file to the repo is useless, those are not published.
What is the purpose here? @pohlt


Gather feedback from Uk Gov PaaS maintainers

We'd like to ask maintainers of the UK Gov PaaS repo about what they liked or didn't like about the technical aspects of the product, to gauge how reusable it might be for our needs.
We'll first try reaching out to them through the our TTS shared slack channel with UK Gov.

Summary / Takeaways

Architecture may be outdated and retain some of its experimental and minimal features from its beginnings
May be better for proof-of-concept over using its code as a basis for our work
Metrics design may be good to draw from

From a former maintainer
“Realistically it is more a repository of what worked and what didn't work; and a history of work accompanied by a lot of user research (and some Hail Marys).
From my experience, the product (ie the cloud itself) didn't really kick off until there was a web UI, and Pazmin (our nickname for it) did some things really well (I really enjoyed the metrics and logs features. and some things less well (like billing).
There was a desire to keep it experimental and minimal, at least at the beginning (both Pazmin and the platform itself), it was really centred around billing and usage and less around the platform and apps and services.
Beginning with a more ambitious scope would probably have caused us to change the architecture of pazmin (and maybe integrate backing services differently). As such some of the user interactions suffered (eg we never wrote any automated flows for onboarding, or billing with email integrations etc). What I missed building was also integrating app health back into the user team (eg downtime detection, or crashing apps etc)
Having loads of control over the UI was great for metrics integration (big challenge to make all the graphs simple and relevant to users, also challenge of cardinality versus simplicity, eg a hot app instance versus a general shortage of cpu allocation). Metrics was very research led which led to some nice metrics descriptions in the page.”
From a current maintainer:
“What I would say is fork it, play around with it as a proof of concept but I wouldn't build on top of it. As it's a bit old and lacked proper investment, the stack and software patterns are a bit outdated.”
When asked what they would change about it:
“[it’s] running on older version of express, so this is something to update. Would utilise React server components as currently all pages are rendered static html so require a page refresh on actions. You may even consider NextJS as a base to develop this on.”


Add option to ignore RIS update messages from specific peers

Describe what you would like to achieve
I need to ignore one / multiple RIPE RIS feeder (either the ASN or the IP of the feeder).
Describe why the current solution (if any) is not satisfactory
Some RIPE RIS feeder push stale/wrong data causing false-positive alerts for us. Unfortunately, this specific feeder does not reply to any of our messages to solve it that way. To decrease the amount of false-positive alerts, it'd be cool to exclude either specific ASNs as feeder or specific source feeder IPs.
In our case, it causes alerts like "ASxyz is announcing PREFIX but this prefix is not in the configured list of announced prefixes".

Hi @tomsiewert,
Can you elaborate on this? In particular:

What stale data means in this case?
Did you check the single BGP updates the feeder sent?
Did you contact RIPE RIS?
Can you share the feeder AS? (also in private if you prefer)

Thanks

Hi @massimocandela

We have withdrawn two prefixes several weeks ago and they're also not visible in the DFZ anymore however, this feeder is still sending BGP UPDATE messages to RIPE RIS.
I did
Contacting RIPE is tbd
I'll send you an email

This will go out in the next release (see blacklistSources in config.example.yml). A private conversation is going on to better understand the situation that triggered this ticket.


Rework handling of ingress.class annotation

What this PR does / why we need it:
This PR expands the controller's logic for handling the ingress.class annotation, introducing three handling modes:

"Lazy", the existing behavior for all resources. we load resources if they match the ingress class or if they have no class and we're using the default ingress class ("kong").
"Require", used for resources that must have a matching ingress.class annotation (KongClusterPlugin and TCPIngress).
"Ignore", used when we do not care about the resource class, because the resource can only be loaded if referenced by something else that meets one of the other two handlers' requirements. This means that a KongPlugin loaded by an Ingress or a Secret loaded by a KongPlugin do not require ingress.class, which fixes the bug that prevented the controller from following their updates.

Which issue this PR fixes
Fix #733
Fix #734
Fix #735
Special notes for your reviewer:
This is a draft in part because it needs #751 and #764 merged first, and in part because I have some questions, primarily around what tests we need to add. Draft questions/comments go!
Implementation

The "Ignore" handler currently allows empty-class resources when using non-standard classes. It does not permit mismatched class resources. Should it? We essentially assume that the class annotation doesn't matter for these resources, since the controller only loads them if they're referenced by some non-empty-class resource via the parser. However, pulling in actual mismatches seems a bit wrong, much more so than empty-class resources.

Tests
I checked for existing similar coverage based on an unscientific run of find ./ -name "*_test.go" | xargs grep -ni class.

main_test.go and flags_test.go don't need additions since they're just testing the class annotation itself, not its interaction with resources.
status_test.go shouldn't need changes since it only interacts with Ingresses afaik, and passes them through lazy handling (the old behavior for all resources).
parser_test.go has some existing test with KongClusterPlugins, but those should continue to have them. Everything else presumably omits them and relies on lazy handling, which is fine for the parser, since it's only concerned with resources that it was already able to track updates for (global KongPlugins excepted, but those are going away). Will see after #175 for sure.
annotations_test.go is updated, with some caveats about one of the tests, see ---
not clear if store_test.go or fake_store_test.go. This is where I'd expect to add tests for tracking resource changes, but we don't have any such existing tests. we may require them, or may not--we could consider the annotations_test.go additions sufficient, to show that we can now find emptyMatch/IgnoreClassHandling resources independent of their attachment to something in the parser.

Naming
I'm not super-fond of the naming I chose for the class handling system, and it's something that ultimately matters more how others interpret it, so please bikeshed with abandon as to whether the Require/Ignore/Lazy naming scheme (and some of its minor variants) makes sense for describing what those modes actually do.
Comments
Some of these still refer to the old system and need to be replaced. I'd like to handle that once the actual code/naming is finalized rather than trying to rewrite them in advance.

Current failure is
--- FAIL: TestKongSkipClasslessIngress (0.00s)
    parser_test.go:1714:
                Error Trace:    parser_test.go:1714
                Error:          Not equal:
                                expected: 0
                                actual  : 1
                Test:           TestKongSkipClasslessIngress
                Messages:       expected zero service to be rendered
FAIL

From https://github.com/Kong/kubernetes-ingress-controller/blob/4bf2405e2c9c7ff72b25edbd4dac96575de4e85d/internal/ingress/controller/parser/parser_test.go#L1675-L1727
Need to think over whether that's something the new system actually does wrong or a contrived test that shouldn't run that way in practice and needs adjusting.

Tests now passing with the --skip-empty-annotation semantics from #664 preserved in the new system. IMO this is ready for review by others, with the caveat that it probably needs work after my trying to address several different concerns at once and getting them confused in my head.
Left un-squashed for now because, assuming that there will be change requests, I want to keep the messy history for the benefit of my own memory when I address them. Apologies for the lack of commit atomicity for the PR; please rely on the file diff and assume it will be wrapped up into a single big "revamp class annotation" commit at the end.

The blanket ingressClassHandling inside store doesn't work.
The code needs to be changed on a per-method basis.
Examples:

The class handling logic for global KongClusterPlugin and referenced KongClusterPlugin is different.
Class handling logic for secrets is different based on the type of secret: tls certs, ca cert, consumer credential etc.

I think this is ready now. All of the class filter logic has been ripped out of REH and moved into the store. Kicking the tires on that with a classless Ingress-attached plugin with secret config (and modifying that config) and a misconfigured/no-class global plugin didn't appear to indicate a change in end functionality, though there should now be more events processed, many of which are no-ops. Not comprehensive, but it does not appear that the switch went horribly wrong (I was expecting it to and was pleasantly surprised).
There weren't comments on the tests previously, but I (and reviewers) should give them another once-over tomorrow to confirm they cover what we want.

Longpost partly discussing https://github.com/Kong/kubernetes-ingress-controller/pull/767#discussion_r465982190 further, but also some other things discussed in the Google doc spec. Putting it here since it kinda fits and I didn't want to comment on the doc since the inline comments there don't work well for longer prose:
https://gist.github.com/rainest/36abdb908315e4007a38fdde73188504
Main blocking question there is: should I indeed revert the string behavior argument to a boolean (and as a consequence, add a new lazy boolean argument to isValidIngressClass)?
The behavior decision logic should probably move out of that function into store, assuming we do want separate controls for Ingress and KongConsumer lazy handling even if they have the same default.

Refactoring done, should be ready now. Caveats:

KongConsumer handling currently hardcodes the default, as that's planned for a later PR. The scaffolding to support the flag is there but I haven't added the flag yet.
The Ignore case is not tested, and I do want to add that, but I figure it's a fairly minor point for review (especially since the case is just "return true, always").

Went ahead and added several additional tasks around changing the default class handling to this by means of renaming and flipping the flag default, along with the new consumer class flag. The latter replaces some placeholder code here, and IMO that's good albeit at the cost of increasing the PR scope.
That should also bring the to cover #731 and clear most of the major changes to class handling/matching--rest should be either relatively minor changes, effectively done by virtue of other changes, or expansion of docs&tests. Remaining tests related to that, I think those should probably be in a new PR before closing out those cards, as I'd like to get the lengthy history in this one squashed.

03fefa7 adds varied class tests for annotations, but not for the parser. fakestore (which the parser tests rely on) currently instantiates its store with a hard-coded class (which is the current default, "kong").
It shouldn't be necessary to test a non-default class there, however, as any special behavior was either in the annotations code or REH filters. The former is tested independently. The latter no longer has any class filter, so I don't see much point in trying to test for special behavior there.
CA Certificates currently have (non-special) behavior in the store, but it will go away as part of #739. Knative stuff is more of an issue: https://github.com/Kong/kubernetes-ingress-controller/issues/731#issuecomment-677931057

Remaining skipClassless was missed in the rework, but obsoleted by https://github.com/Kong/kubernetes-ingress-controller/pull/767/commits/1bac620d4c51f6f21acb7123e1cf9c87a00a4fdd for https://github.com/Kong/kubernetes-ingress-controller/pull/767#discussion_r475686003


[mle] fix MLE Router handling errors

This PR tries to fix the assertion failure that occurred when simulating 8x8 nodes forming one partition by OTNS form_partition.py.
This bug was found when trying to resolve #4508.

This PR has been simulated for more than 200 days without any assertion failure.
Any more comments ? @jwhui @abtink @librasungirl

It's strange that simulation tests failed a lot for this PR. I am marking this PR as WIP for further investigation. @abtink @jwhui @librasungirl

any further comments? @mwilniewiec


Provide guidance on contributing

It would be great to have a documentation that helps contributors know what is expected from their PRs or what will help streamline integration of any proposed changes. One thing I found when working on this recently was running ./gradlew spotlessApply before a PR. Please let me know if there's anything else that would help and I'll put together some guidance.

Also using the assemble command to verify the build outside of project targets.

A contributing guide is definitely nice to have, I'll add it soon.


Removed unused resources

One too much commit here in this PR ;)

Opps, let me fix that.

Fixed in another PR: https://github.com/commons-app/apps-android-commons/pull/912

Codecov Report

Merging #911 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #911   +/-   ##
======================================
  Coverage    6.36%   6.36%
======================================
  Files          97      97
  Lines        5105    5105
  Branches      479     479
======================================
  Hits          325     325
  Misses       4753    4753
  Partials       27      27

Continue to review full report at Codecov.

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


Official Docker images

I have #204 working, but currently using a testing Docker repo under my own Docker Hub name. I'm happy to host images for now, but should there be an official Entropic team with official images, maybe built and pushed as part of CI? I'd also be happy to set up a team on Docker Hub with a service account, as long as one of the Entropic core developers takes it over once it's set up.

Yep, that's me! Thanks!


iOS: stillThe image picker does not close when tapping cancel or selecting an image

https://github.com/NativeScript/nativescript-imagepicker/issues/251
With the same issues in the link and solved versions, I still face the problem recently.
It is not happening on the simulator but in real devices, sometimes but not all time.
version 6.2.0
tns-ios version 5.2.0

@tingfong214
Can you try to provide a more information that can help us reproduce this issue. What type of device are you using, which is its iOS version, how often does it occur, etc?

It's tested on iPhone 6, iPhone 7, iPhone XR.
So I assume most of the iOS devices face this problem.
For real devices, it's using ios 12.2
For the simulator, it's using ios 12.1 on iPhone X
It's exactly the same problem with #251 stated above.
I can't tell how often it is, it happens randomly, it's unpredictable.

Closing due to inactivity


Development

Added build and webconfig artifacts, creatic static-web-app folders

Commit

Commit

@MissAlleyCat @Spyntek I believe your repositories/PRs are misconfigured and requesting to merge into the official Azure Static Web Apps community repository. Please verify, closing this PR

approve merge


Issue 12

Finished all tests for available methods.

Codecov Report

Merging #14 into master will increase coverage by 3.53%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage      86%   89.53%   +3.53%
==========================================
  Files           4        4
  Lines        1386     1386
  Branches       20       20
==========================================
+ Hits         1192     1241      +49
+ Misses        187      136      -51
- Partials        7        9       +2

Impacted Files
Coverage Δ

src/HttpService.ts
83.33% <ø> (+40%)
:arrow_up:

src/traveler.ts
46.94% <0%> (+13.74%)
:arrow_up:

src/OAuthError.ts
75% <0%> (+25%)
:arrow_up:

Continue to review full report at Codecov.

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


Updating readme from branch example

Updates
I updated readme file. Please review and accept.

Thoughts
I probably should't, but okay. 👍


small summary in the README

it would be good to have a small summary of the features of the analyzer at the top of the README, like we have for the lucene-zh

resolved in efa7667485313aad3d68c7ea96675841a24b3c2e


OV5642 problem with SPI

Hi,
i have problem with my ArduCAM. I connect with RaspberryPi and level voltage in SPI is higher than 3 V. When camera is connect with RaspberryPi in SPI pin is 3V3 but when I started program this level up to 4 V. What can I do with this?
This problem is with power camera woth 5V and 3V3.
I tried use camera in few RaspberryPi so this isn't problem with RaspberryPi.
Thank you for help :)

@ArduCAM

@pavik1994
Hi,
Sorry for my later reply.
Please connect the camera to 3.3V instead of 5V .
Let us know if you need more help.
Regards,
ArduCAM support team.

Hi,
I connect the camera to 3.3V but still in SPI is 4V or more. What can I do
more?
Regards
2017-06-15 2:11 GMT+02:00 supprot<EMAIL_ADDRESS>
@pavik1994 https://github.com/pavik1994
Hi,
Sorry for my later reply.
Please connect the camera to 3.3V instead of 5V .
Let us know if you need more help.
Regards,
ArduCAM support team.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ArduCAM/Arduino/issues/212#issuecomment-308592492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abmr5o_L_V7J40KaOdJ_Yt-6nWL_KFIeks5sEHaYgaJpZM4Nzk00
.

Hi, try adding 100 ohm resistors between the SPI lines and ground.


Support Hex equal

One very easy and common feature Waffle should include in it's base Chai matchers is a hex equals.
This will match hex values forcing lowerCase.
Example:
expect(ethers.utils.hexZeroPad('0xaaBB')).to.be.hex(ethers.utils.hexZeroPad('0xaabb'));

These two values are essentially the same, and thus using a special hex equal, we can easily compare. This can be used to compare things like addresses to daw data, like create2 situations for example.

I think it is a good idea, would you like to prepare a PR?

I assume zero in front can be ignored, so that following will succeed:
expect('0xaaBB').to.be.hex(ethers.utils.hexZeroPad('0xaabb'));

Otherwise we are just talking about case insensitive comparision.

@marekkirejczyk or @sz-piotr This issue can be closed now I think.

@marekkirejczyk or @sz-piotr This issue can be closed now I think.

Released in 3.2.2

Released in 3.2.2


Global CSS cannot be imported from within node_modules error with Next.js

./node_modules/react-colorful/dist/index.css
Global CSS cannot be imported from within node_modules.
Read more: https://err.sh/next.js/css-npm
Location: node_modules/leva/dist/leva.cjs.dev.js

Next.js gives this error when trying to use Leva.
Here's Next.js' doc page regarding this issue: https://github.com/vercel/next.js/blob/master/errors/css-npm.md
I wonder if Leva would consider follow its recommendation.
Or is there a workaround for this?
Thanks.

Hi! The package is still under heavy development and we don't have a release strategy in place yet.
Bu the good news is that this is already fixed on the master branch!

Should be fixed in 0.3.x, let me know if otherwise!

@dbismut
It works now. Thanks for making this great widget.


Add Stripe example

Test Plan

Checklist

[ ] Tests updated
[ ] Docs updated

Screenshots

imo this example needs a rework

it gives a bad user experience because of the multi redirect here https://github.com/saurabhp75/epic-stripe/blob/baffb612ced3d53f53d17681baf31e62a2387b34/app/routes/account.tsx#L30

I don't get the value of wrapping the stripe calls into endpoints https://github.com/saurabhp75/epic-stripe/blob/baffb612ced3d53f53d17681baf31e62a2387b34/app/services/stripe/api/create-customer.ts

finally the duplication of data between prisma and stripe for the pricing plans makes thing complicated and is not what strip puts in its examples

I will fix first two items.
As for third item, there are few points to consider :

The app needs to keep some data locally (which will be synced using webhook). This is essential for better UX and avoid network latency. IMHO this is a pretty standard approach when integrating with payments platform.

The data to keep in Prisma depends very much on the type of products being sold like one time purchases, subscriptions or the one with licences or usage base plans. See item below for the use cases this example handles.

This example provides 3 subscription based products with option of monthly or yearly payments. Each subscription variant can be configured with a limit on number of units user can consume.

Do let me know which Stripe example you are looking at for DB schema.

README updated based on the discussion.


chore: bdd-gh-1584385067 to 0.0.1

chore: Promote bdd-gh-1584385067 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Add documentation for WordPress.DB.RestrictedFunctions

Related to #1722

@paulgibbs Just checking in: did you have any questions about the review or are you good to proceed ?


Add maps geojson upload getting started example files

Adds sample files for the Getting Started docs to test the new Geojson Upload functionality in the Kibana Maps Application

It looks like there's a bit more I could add to this based upon the contribution guidelines for this repo. For now, if possible, could we get these files into this repo to support our Getting Started docs and generate a follow-up issue to add further materials and descriptions?
The files in this PR were create manually as examples of the type of data we can upload to the Maps application

@jamiesmith I see you've been a reviewer for a few other PRs in this repo. I've added you as a reviewer on this PR but happy to work with whoever makes sense here!

It couldn't hurt tbh. I was thinking I'd follow up with another PR that:

Adds a readme file
Updates the main readme for the examples repo with links to Maps data
Anything else worth adding. I can review the other examples and make sure we're up to speed with everyone else


Glowing Login UI by Pranith

Added a glowing login UI with a black and pink color combination.

source branch deleted, closing PR


Checking running the double update script.

Killed the camera twice by running the update.

improved in #274


React on communication on a background thread

Currently the ClientDolphin provides the following method:
void send(String commandName, OnFinishedHandler onFinished = null)
By using this method the OnFinishedHandler is always called on the UI thread. Since this is great for mostly all use cases there are some special cases that need a different handling. Currently the biggest problem is a logout method in JavaFX that will be called on the Application.stop() method. By doing so the OnFinishedHandler will never be called since JavaFX has already stopped. For this special type we need a callback in a background thread. Maybe the send methods can return CompletableFuture<Void>.

PR https://github.com/canoo/dolphin-platform/pull/356 prepares the code base for this change

PR https://github.com/canoo/dolphin-platform/pull/358 prepares the code base for this change


Ported code to Swift 3 included with the sample Xcode project

There's actually already a Swift port of this linked at the top of the readme.


How to run it on Mac OS?

Please give us more details, like the CPU (intel or Apple Silicon), what app you want to run, and why it doesn't run on your computer.

Thank you for reply!
I follow the tutorial to study, eg. https://rcore-os.cn/arceos-tutorial-book/ch01-01.html or https://rcore-os.cn/rCore-Tutorial-Book-v3/index.html. Neither mentioned how to run it on MacOS.

The steps to run on macOS are similar to those on linux, please refer to README, but you need to install the corresponding toolchain (e.g., QEMU) for macOS.

After an afternoon, the hello app was successfully run.

But qemu-riscv64 was not compiled, so I don’t know what impact this will have.
Configuration steps on MacOS

curl https://sh.rustup.rs -sSf | sh
cargo install cargo-binutils
brew install qemu will failed, so compile from source code:
3.1  brew install libffi gettext glib pkg-config pixman ninja meson // reference https://wiki.qemu.org/Hosts/Mac - Building QEMU for macOS
3.2  tar xvJf qemu-7.0.0.tar.xz
3.3 ./configure --target-list=riscv64-softmmu
3.4 make
git clone https://github.com/rcore-os/arceos.git
make ARCH=riscv64 A=apps/helloworld run

Thank you again!!!

QEMU 8.2.1 was provided by the latest homebrew, which includes qemu-system-riscv64. You may need to update your brew version.

qemu-riscv64 was only available on Linux, which requires Linux syscalls.


coo_matrix.diagonal doesn't works with non canonical format

Description
coo_matrix.diagonal doesn't works with non canonical format:
Due to this piece of code:
https://github.com/cupy/cupy/blob/69f9d1111f643c2a663669ddaeaac82e4f665581/cupyx/scipy/sparse/_coo.py#L204-L210
There doesn't seems to be a function named _sum_duplicates in class coo_matrix.
This is untested as well:
https://github.com/cupy/cupy/blob/69f9d1111f643c2a663669ddaeaac82e4f665581/tests/cupyx_tests/scipy_tests/sparse_tests/test_coo.py#L1115
To Reproduce
import cupy
from cupyx.scipy.sparse import coo_matrix
import numpy as np

a = cupy.testing.shaped_random((8, 5), np)
A = coo_matrix(cupy.array(a))

# This works
A.diagonal()

# This does not works
A.has_canonical_format = False
A.diagonal()

Error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
/tmp/ipykernel_129864/2543193609.py in <module>
----> 1 A.diagonal()

/work/aktech/miniconda3/envs/myenv/lib/python3.8/site-packages/cupyx/scipy/sparse/coo.py in diagonal(self, k)
    206             data = self.data[diag_mask]
    207         else:
--> 208             row, _, data = self._sum_duplicates(self.row[diag_mask],
    209                                                 self.col[diag_mask],
    210                                                 self.data[diag_mask])

AttributeError: 'coo_matrix' object has no attribute '_sum_duplicates'

Installation
Built from GitHub source
Environment
OS                           : Linux-4.15.0-159-generic-x86_64-with-glibc2.10
Python Version               : 3.8.12
CuPy Version                 : 10.0.0
CuPy Platform                : NVIDIA CUDA
NumPy Version                : 1.22.0
SciPy Version                : 1.9.0.dev0+1330.6e375d6
Cython Build Version         : 0.29.25
Cython Runtime Version       : 0.29.26
CUDA Root                    : /work/aktech/miniconda3/envs/myenv
nvcc PATH                    : /work/aktech/miniconda3/envs/myenv/bin/nvcc
CUDA Build Version           : 11020
CUDA Driver Version          : 11020
CUDA Runtime Version         : 11020
cuBLAS Version               : (available)
cuFFT Version                : 10401
cuRAND Version               : 10203
cuSOLVER Version             : (11, 1, 0)
cuSPARSE Version             : (available)
NVRTC Version                : (11, 2)
Thrust Version               : 101000
CUB Build Version            : 101000
Jitify Build Version         : 7db7a57
cuDNN Build Version          : None
cuDNN Version                : None
NCCL Build Version           : None
NCCL Runtime Version         : None
cuTENSOR Version             : None
cuSPARSELt Build Version     : None
Device 0 Name                : TITAN RTX
Device 0 Compute Capability  : 75
Device 0 PCI Bus ID          : 0000:08:00.0
Device 1 Name                : TITAN RTX
Device 1 Compute Capability  : 75
Device 1 PCI Bus ID          : 0000:43:00.0

Additional Information
No response

@anaruse please do it!!!!
Thanks :)


Add more models to benchmark_score

Description
This PR aims to add more CNN models to benchmark scripts like squeezenet and mobilenet and make it more user-friendly. Also add a CNN benchmark script for gluon.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Add 'inception-v4', 'inception-resnet-v2', 'mobilenet', 'densenet121', 'squeezenet1.1' to benchmark_score.py
[x] Add '--network' to script for specific and run all models by default to keep same with before.
[x] Add '--batch-size' to script for specific and run [1, 2, 4, 8, 16, 32] by default to keep same with before.
[x] Add script for gluon to benchmark more CNN models.

Comments

'densenet121', 'squeezenet1.1' in benchmark_score are converted from gluon in order to cover more cases.
These models are benefit a lot from #12530

@pengzhao-intel @juliusshufan

@sandeep-krishnamurthy Thanks for suggestions. It can handle multi-GPUs now:)

@sandeep-krishnamurthy Please take a review again. But my question is that why should we multiply batch size with num of GPUs, what's the define of batch size here?

@szha simplify done, please take a review, thanks:)


Deserialization of default_value fails for strings

It is currently not possible to change default_value using xlsx. The reason is that default_value may be in fact of a JSON type (null, object or array) or a string. Currently, it is always parsed as JSON.

Couldn't replicate the issue. I managed to change the default_value via xlsx using:
![image](https://user-images.githubusercontent.com/4363@qiq /86138736-1d50b500-baef-11ea-9174-ef5a9785dd30.png)
Did you mean something else? @qiq

Closing for inactivity.


what is this error

/usr/bin/env: ‘node’: No such file or directory

man... your nodejs is not currently installed.. please check your installation ... if you use ubuntu or debian... run $ dpkg -l | grep node to check this ... for example..


compared with ShuffleNet V2

@sacmehta
hi, after read your paper and run the code, I've tried compared it with shufflenet v2 1×(implemented in tensorflow) for image classification, their GPU(1080Ti) Speed as follow:
shufflenet v2 1×: 3ms/per image
ESPNet V2: 13ms/per image
I want to make sure my test process is right? (or the actual forward infer is the above speed), thanks

ESPNetv2 which version?

ESPNetv2-2.0 full 32-bit precision network  (not quantized one) image classification on iPhone7 is 1000+ FPS, which means it is running at close to 1ms per image. So, you should get similar speed on your GPU.
Also, it is not good to compare PyTorch and TensorFlow models directly.

@sacmehta
how about the PyTorch model speed on 1080Ti graphics card?


Suggestion: Add a way to update lockbot via an API

It would be really great if Lockbot had some kind of REST Api, this would allow developers to use their CI/CD tools to trigger locks, and the locks could fail if someone else is using that lock.

Thanks for the suggestion @greg-cann (& @bobthemighty)!
My brain is asynchronously having a think about how to implement it in a sensible, maintainable and secure way.
Initial thoughts/notes to self:

Add a Slack command /lockbotapi which generates api key for the current user which has their permissions and performs actions as them (if users want to add bot/service account they need to register one in their slack workspace and generate an API key for them that way)
Create api to perform actions, will need a way to expose data in REST like format instead of Slack Markdown messages, might get my Lockbot abstraction to return a message and data so that domain logic is encapsulated

Anonymous feedback:

Nice tool to help with teams working on shared repos! Would have rated higher if there was more integration with CI pipelines.

E.g., if nobody has locked an environment, then when approving a deploy to that environment, lockbot would send a message that the user who approved the deploy has locked that environment.

E.g. if somebody has already locked the environment and another person attempts to deploy to there, they would be blocked by lockbot somehow


Incomplete Test: project 1 last step, I made all the code edits.

Tests all pass as of now, haven't added new tests yet. Edited the format for some files, so it seems like I changed a lot of files... (Just ignore all the indentation changes.)

Just curious what did you use to format the code?

@kevmo314 I just did command-A and then command-I in Eclipse since some lines weren't indented, just thought it made the code easier to read.

Can you try using google-java-format? It has an eclipse plugin which should reduce the number of changes. Since different people are using different IDEs I think just using the default formatter will result in a lot of formatting changes back and forth so it would be good to standardize on a tool.

@kevmo314 For sure! Sorry I assumed everyone was using eclipse and just manually indenting which resulted in non-standardized formatting. I'll try using the tool you mentioned!


Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance.

Hello,
I'm trying to fix the Microsoft 365 learning pathways has a configuration issue. Ask your administrator for assistance. error.
I've read previous issues and restored broken permissions to CustomConfig list.
Additionally, when this did not work, we removed the solution from our environment and redeployed it. The error persisted.
Please let me know what information you might need to help with troubleshooting and restoring the full functionality of the solution. I'll gladly provide those.

I am seeing this issue too. Started this week.

From the errors I am seeing it looks like the tenant app property isn't set correctly. Depending on how you removed it from your environment things may or may not be set up correctly. In general uninstalling the entire application isn't something we recommend unless you do it 100% (e.g. removing the web part, deleting the site collection, removing the tenant app property).
The error is saying that it can't get a valid configuration. That means that either the tenant app property is not set properly and the web part doesn't know where to get the configuration or that it knows but it can't access it. Check the tenant app property using PowerShell and make sure that it matches the site you are using for Learning Pathways. Also check that the permissions are set up properly for the config lists in the Learning Pathways site https://github.com/pnp/custom-learning-office-365/issues/686#issuecomment-1218201343

Sorry mine was a dumb issue as I wasn't on our corporate network correctly.

I've just checked the permissions and those are as listed in the https://github.com/pnp/custom-learning-office-365/issues/686#issuecomment-1218201343
Also App properties are as follows:

The site is the correct one, as Learning Pathways is deployed there.

@fiszuu The webpart is not getting the configuration from github. Is it possible that github is being blocked in your organization?

I doubt that. Reason behind the doubt is that some users in the same network location are experiencing this issue and others can access the Learning Pathways without any issue.

@fiszuu if you could check the network tab on a page that isn't working and show any errors or items that are not loading from there that would be appreciated.

Hi, sorry for the delay. Here is the network tab outcome:

This is the only error I see

Close due to lack of activity. If this is still an issue please create a new issue and reference this one.


Add skip option for linter

Added --skip option for linter to skip one or more rules by name from
linting.
Fixes #5

I took a shot at this, it would be useful for me as well. Let me know if you had something else in mind.
I looked at adding tests, but would either need to make a new profile or change up what's in a fixture to add a skip array there.

This is awesome! Thank you so much.
Could you have a think about tests, and see what you come up with after another try? Changing how things work is quite alright.

Sure I can see about changing it up.


release: clickstream Android 0.11.0

[x] PR title and description conform to Pull Request guidelines.

Description of changes:

release: clickstream Android 0.11.0

Documentation update required?

[ ] No
[x] Yes (Please include a PR link for the documentation update)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.26%. Comparing base (18c6b8c) to head (03321f0).

Additional details and impacted files
@@            Coverage Diff            @@
##               main      #66   +/-   ##
=========================================
  Coverage     87.26%   87.26%
  Complexity      455      455
=========================================
  Files            42       42
  Lines          1665     1665
  Branches        152      152
=========================================
  Hits           1453     1453
  Misses          135      135
  Partials         77       77

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Doc: MP/SE content for Open API

Environment Details

Helidon Version: 2.0.x
Helidon SE and Helidon MP

Problem Description
We need to provide additional content for both the SE and MP versions of Open API per Dmitry's doc epic.
Specifically we need to:
Reflect Helidon 2.0 changes (if any)
Cover more use cases.
Steps to reproduce
Existing doc can be found here:
https://helidon.io/docs/v2/#/mp/openapi/01_openapi
https://helidon.io/docs/v2/#/se/openapi/01_openapi

There are no changes required to the Helidon 2.0 doc for OpenAPI in SE or MP.
We could add more use cases, but that would greatly expand the length and go against our goal of keeping the one-pagers short and sweet. If we really feel we need to add further use cases, we might consider adding a guide rather than expanding the current one-pager.


Passing {{test}} within the body of a request

I'd like to pass the following body in a POST
{
  "template": "My {{someVerb}} template!!"
}

It, of course, complains that {{someVerb}} is not a variable.
Is this possible?

My original plan in this case was to use an external file without variable substitution.
POST http://httpbin.org/anything
< body.json

body.json
{
  "template": "My {{someVerb}} template!!"
}

Possibly one should still include some kind of encoding
{
  "template": "My \{\{someVerb\}\} template!!"
}

Do you possibly have a suggestion, I find mine currently not good?

I have now implemented it with this escape logic. I don't think it looks any better with other characters.

Thanks @AnWeber. Will you be releasing it soon?

Sorry for the delay. I was sent another feature request, but unfortunately that turned out to be more time-consuming than expected.

I published a new release.

thx for using my extension. Feel free to open issue again, if it is not working as expected.


Feature request: Dropdown to change climate HVAC modes without turning on the primary zone or group

Hi @Noltari,
It's me again.
I've seen many posts/issues about HVAC mode, primary and basic zones, and I’d like to submit a simple idea.
First, I want to show you that I’ve already tried different workarounds for this issue with my limited skills. I created a script that saves the state of the primary zone, changes the mode on it, and then turns it off again if it was off before the change. It works better with the local integration due to latency issues (even with added delays and logic retries).
The script is executable via one button per mode in Lovelace, with the mode passed as an input variable. While this workaround functions, it’s not very clean.
To improve this, I’d like to ask if it would be possible to add a workaround as a new feature :
A dropdown list populated with 'mode_available' to control 'mode' on group's device and/or primary zone's device
Mode parameter Documentation

Example:

curl -X PUT "https://m.airzonecloud.com/api/v1/installations/{installationId}" \
 -H "accept: application/json" \
 -H "authorization: Bearer {token}" \
 -H "content-type: application/json" \
 -d '{"params":{"mode":2}}'

I’m aware that various topics have been opened on this issue, but I wanted to submit this idea regardless, as it seems that modifications to the Climate card in Home Assistant may not be arriving anytime soon.
Thanks again.
Best regards,

PRs created for the Cloud integration:

https://github.com/home-assistant/core/pull/125918
https://github.com/home-assistant/core/pull/125917


vscode: add libGL.so.1 to vscode executable

This should be backported to 23.11.
GPU acceleration broke again for vscode, this time complaining about missing libGL.so.1.
For past issues, see:
https://github.com/NixOS/nixpkgs/issues/243372
https://github.com/NixOS/nixpkgs/issues/248200
PRs that fixed those issues, respectively:
https://github.com/NixOS/nixpkgs/pull/245705
https://github.com/NixOS/nixpkgs/pull/249853
The same issue fixed for chromium/electron apps and backported, respectively: https://github.com/NixOS/nixpkgs/pull/269345 https://github.com/NixOS/nixpkgs/pull/269414
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

For me on Wayland this also requires libEGL.so, but with that it works.

@lorenz Do you mean wayland needs a --add-needed ${libglvnd}/lib/libEGL.so ?


merge package template from Kronos-Tools/npm-package-template

Updated standard to latest version

Current coverage is 87.77% (diff: 100%)

Merging #71 into master will not change coverage

@@             master        #71   diff @@
==========================================
  Files             2          2
  Lines            90         90
  Methods           8          8
  Messages          0          0
  Branches         23         23
==========================================
  Hits             79         79
  Misses           11         11
  Partials          0          0

Powered by Codecov. Last update 19fa8ec...32b85d2

Current coverage is 87.77% (diff: 100%)

Merging #71 into master will not change coverage

@@             master        #71   diff @@
==========================================
  Files             2          2
  Lines            90         90
  Methods           8          8
  Messages          0          0
  Branches         23         23
==========================================
  Hits             79         79
  Misses           11         11
  Partials          0          0

Powered by Codecov. Last update 19fa8ec...32b85d2


Workaround for blank window with WebKit/DMA-BUF/NVIDIA/X11

WebKit 2.42 along with X11 and NVIDIA GPU driver triggers bug: https://bugs.webkit.org/show_bug.cgi?id=261874
The bug causes a blank window. Instead of waiting for the problem to be fixed, this work attempts to check for these conditions before setting the environment variable WEBKIT_DISABLE_DMABUF_RENDERER to 1.
If the variables WEBKIT_DISABLE_DMABUF_RENDERER or WEBKIT_DISABLE_COMPOSITING_MODE have already been set then the workaround is not applied.
I would have preferred not to implement this workaround but we don't know for how long we'll have to wait for a proper fix or a workaround to land in everyone's favorite Linux distro.
For now we can implement a workaround and then monitor the situation.

@powellnorma @raphael10-collab If you have the time to check whether this PR solves your issue in a satisfactory way then that would be much appreciated. I'm open to suggestions for better ways to implement this workaround.

I have done:
git fetch origin pull/1060/head:webkit-nvidia-dmabuf-workaround
git switch webkit-nvidia-dmabuf-workaround
cd examples
g++ basic.cc -std=c++11 -I../ $(pkg-config --cflags --libs gtk+-3.0 webkit2gtk-4.0) -o basic && ./basic

And the issue still occurred (on 70cd5d7b6f3ac). When I used WEBKIT_DISABLE_DMABUF_RENDERER=1 ./basic it worked fine. Maybe the environment variable is not set early enough to have an effect on webkit? Or it isn't set in the first place?
Hmm.. I see that nvidia-smi is used, but I don't have that installed on my system. Maybe it would be better to check if the nvidia kernel module is loaded, e.g. via isdir("/sys/module/nvidia")?

Also for some reason, on my system:
$ echo $XDG_SESSION_TYPE
tty

We believe we could check $DISPLAY instead: https://stackoverflow.com/a/637083

If nvidia-smi isn't installed or if XDG_SESSION_TYPE isn't x11 then it will not set WEBKIT_DISABLE_DMABUF_RENDERER to 1. I guess sd_session_get_type() would return tty as well in your case instead of x11.
Potential alternatives for checking for X11 according to other answers in the link you provided: xset q, xprop -root
Do you know why you don't have nvidia-smi installed? I'll do some testing with /sys/module/nvidia, and if if that works then it seems like a better idea than using nvidia-smi.

Here's what I observed on Ubuntu 22.04.3 and Fedora 39. I have both an NVIDIA GPU and AMD GPU installed in this computer, and select one by blacklisting the driver for the other.
/sys/module/nvidia

When using the NVIDIA GPU and NVIDIA driver then /sys/module/nvidia exists.
After switching to an AMD GPU then /sys/module/nvidia does not exist.

It appears that we can check this path instead of invoking nvidia-smi.
X11

When using Xorg/X11 then xset q and xprop -root return 0 on Ubuntu. xset command not found on Fedora.
When using Wayland then the above commands also return 0 on Ubuntu. xset command not found on Fedora.
$DISPLAY has a value both when using Xorg/X11 and when using Wayland.

These appear to be unreliable to me.

We could perhaps invoke something like pidof -q Xorg because it'll return 0 when using Xorg/X11 and return 1 otherwise (e.g. with Wayland).
If it doesn't hurt too much then maybe it's better to simply skip the check for X11.

If it doesn't hurt too much then maybe it's better to simply skip the check for X11.

That or maybe check if /tmp/.X11-unix/ exists / is not-empty? (An alternative could be /tmp/.X0-lock)
Or $WAYLAND_DISPLAY, from https://unix.stackexchange.com/a/559950
Hm, we could also try to find out how WebKit detects this.. Since I guess it also has to decide at some point if it should open an X11 window or a wayland window

fwiw,  I tested 54dc5ce and it works for me

There are some hints in the WebKit (2.42.4) source code.
IsWayland() in ThirdParty/ANGLE/src/libANGLE/renderer/driver_utils.cpp.
Source/WebCore/platform/graphics/PlatformDisplay.cpp:
#if PLATFORM(GTK)
    if (gtk_init_check(nullptr, nullptr)) {
        GdkDisplay* display = gdk_display_manager_get_default_display(gdk_display_manager_get());
#if PLATFORM(X11)
        if (GDK_IS_X11_DISPLAY(display))
            return PlatformDisplayX11::create(display);
#endif

#if PLATFORM(WAYLAND)
        if (GDK_IS_WAYLAND_DISPLAY(display))
            return PlatformDisplayWayland::create(display);
#endif
    }
#endif // PLATFORM(GTK)

bindX11() in Source/WebKit/UIProcess/Launcher/glib/BubblewrapLauncher.cpp.

Does the latest commit work as expected?

FYI, XOpenDisplay returns non-null when using Wayland.

Here's what I observed after logging in/out of the desktop with Wayland and X11 without rebooting.

/tmp/.X11-unix/ is non-empty on my system when using Wayland.
/tmp/.X0-lock — is this name tied to a specific display? Both when using Xorg/X11 and Wayland I have lock files with numbers 1 and 2, but not 0.
$WAYLAND_DISPLAY has a value when using Wayland and is empty when using X11; however, we can't know for sure that X11 is used when Wayland isn't used.

Also, after rebooting the machine and immediately using Xorg/X11 then there were no files matching /tmp/.X*-lock.

Maybe commit ab8339f910d9405bbe8d2c26495ce8cc98a474cc can be used but I don't feel confident enough. I don't know if we also need to consider Xwayland but I would rather not spend more time on this.

but I don't feel confident enough. I don't know if we also need to consider Xwayland but I would rather not spend more time on this.

I thought abut Xwayland too. It seems that GTK Applications that had wayland and x11 enabled during compile time, will prefer wayland by default:
The routine gdk_display_manager_open_display just uses the first backend that "works":
https://github.com/GNOME/gtk/blob/1dbd74e181b0/gdk/gdkdisplaymanager.c#L380
And the default order of the backends gets defined here:
https://github.com/GNOME/gtk/blob/1dbd74e181b0/gdk/gdkdisplaymanager.c#L257
The only way to change this seems to be setting the environment variable GDK_BACKEND. We could check if that is set, and if so always assume x11.

I just realized that the is_wayland_display function still doesn't do the right thing. :man_facepalming:

If I set GDK_BACKEND to wayland then GDK_IS_X11_DISPLAY(display) returns 0. If GDK_BACKEND is either not set or it's set to x11 then GDK_IS_X11_DISPLAY(display) returns 1.
It almost seems like X11 is preferred on my system. :man_shrugging:
Tested this on Fedora 39 without an NVIDIA GPU.

I just realized that the behavior is different depending on whether I'm launching the app from the terminal or debugging it from Visual Studio Code.

I just realized that the is_wayland_display function still doesn't do the right thing

I thought it was intended that all of the conditions have to be met - now only one has to match. But that should be fine too I think!
Generally a false positive (wayland detected but x11 is used) seems to be  more "costful" than a false negative (x11 detected but waylenad is used) in this case, since the former may lead to the the user just getting a whitescreen (and the latter "only" would lead to potential performance impacts)

If GDK_BACKEND is [..] not set [..] then GDK_IS_X11_DISPLAY(display) returns 1

Hm, does that also happen in the a normal Termianl (not VSC)? If so that seems odd, given the mentioned source code. If we can't figure out why this is, then I guess we will have to always assume x11 ..

I thought it was intended that all of the conditions have to be met - now only one has to match. But that should be fine too I think!

The check for DESKTOP_SESSION was failing because the value doesn't containwayland on my system. The only possible values I've seen on my system are gnome, gnome-classic, gnome-classic-xorg and gnome-xorg.
libANGLE's checks look like this:
// There are multiple environment variables that may or may not be set during Wayland
// sessions, including WAYLAND_DISPLAY, XDG_SESSION_TYPE, and DESKTOP_SESSION
bool IsWayland()
{
    static bool checked   = false;
    static bool isWayland = false;
    if (!checked)
    {
        if (IsLinux())
        {
            if (!angle::GetEnvironmentVar("WAYLAND_DISPLAY").empty())
            {
                isWayland = true;
            }
            else if (angle::GetEnvironmentVar("XDG_SESSION_TYPE") == "wayland")
            {
                isWayland = true;
            }
            else if (angle::GetEnvironmentVar("DESKTOP_SESSION").find("wayland") !=
                     std::string::npos)
            {
                isWayland = true;
            }
        }
        checked = true;
    }
    return isWayland;
}```

does that also happen in the a normal Terminal (not VSC)?

Processes launched from VS Code has GDK_BACKEND set to x11. This appears to be an issue with Electron.

New observations when launching the app from the terminal and not VS Code on Fedora 37.

Windowing system
GDK_BACKEND
GDK_IS_X11_DISPLAY(...)

The latest commit only assumes that the windowing system is X11 when it isn't Wayland, but perhaps this could be an acceptable solution for the time being.

It works for my system. LGTM!

Thank you for taking a look at this, @powellnorma!


The feature is fully supported for both Windows and Linux web apps. All of the behaviors act the same between Windows apps and Linux apps.

Found Windows / Linux app different config when UDR used in app subnet and pointed to a NVA sit in peering VNET
For Windows app , app vnet peering property AllowForwardedTraffic can be any (T/F)
For Linux app , AllowForwardedTraffic has to be TRUE on app vnet peering .
[Enter feedback here]

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@yull1860 Thanks for the feedback! We are currently investigating and will update you shortly.

@yull1860 Thanks for the feedback! I have assigned the issue to the content author to review further and update the document as appropriate

Hi @yull1860. This should have been corrected, and I have not been able to reproduce it. Can you please confirm and proceed to close the issue?
Thanks

@yull1860, if you do happen to reproduce the issue, please feel free to reopen the issue and tag us in your comment.


Document prod_builders.json and try_builders.json

Is there any documentation anywhere that talks about prod_builders.json, what it's schema is, how it's used, where manifest.yaml fits in, and so on? the README in the devicelab directory doesn't seem to mention it, and the files themselves have no comments (and can't have comments, since they're JSON).
It would be great to migrate these to a format (like YAML) that can be self-documenting, and it would be great to have a README (and update the devicelab/README) that explains the context of these files, how to edit them, what the schema is, etc.

prod and try builders are deprecated in favor of .ci.yaml, which is documented at https://github.com/flutter/cocoon/blob/master/CI_YAML.md


[Parallelism] Implement vocabulary parallelism

Balancing Pipeline Parallelism with Vocabulary Parallelism introduces a way to handle vocabulary scaling together with PP. While context parallelism splits the sequence dimension across devices, Vocabulary Parallelism specifically splits the vocabulary dimension of embedding layers.

Results:

cc: @H-Huang


Update grade progress

Added in Index notations

Codecov Report

Merging #67 (3511b16) into master (c02f4e1) will decrease coverage by 0.01%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
- Coverage     71.32%   71.30%   -0.02%
  Complexity      488      488
============================================
  Files            89       89
  Lines          1583     1582       -1
  Branches        157      157
============================================
- Hits           1129     1128       -1
  Misses          402      402
  Partials         52       52

Impacted Files
Coverage Δ

...c/main/java/seedu/address/model/person/Person.java
84.12% <ø> (-0.25%)
:arrow_down:

.../java/seedu/address/model/util/SampleDataUtil.java
50.00% <0.00%> (ø)

.../seedu/address/model/person/GradeProgressList.java
78.26% <100.00%> (ø)

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


la-go-live

resolves #1271

Docs Build status updates of commit 40de7b7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/deploy-and-publish/appsource/publish.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/deploy-and-publish/overview.md
:white_check_mark:Succeeded
View

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.


Fix a race condition caused by other threads calling mapper methods while mappedStatements are being constructed

In projects using the Spring framework, we often use @PostConstruct to perform some operations after bean initialization. In some scenarios, we will create asynchronous thread to perform some time-consuming SQL operations to avoid blocking the application startup process. We observed that the use of mappers in asynchronous thread may broken due to the resize of the underlying StrictMap of mappedStatements.
Example code for using @PostConstruct with asynchronous thread, MybatisRaceConditionApplication.java:
@Autowired
private Mapper1 mapper1;

private volatile boolean applicationReady;

@PostConstruct
public void raceConditionTest() {
    new Thread(() -> {
        int successfulCalls = 0;
        while (!applicationReady) {
            try {
                mapper1.select1();
                successfulCalls++;
            } catch (Exception e) {
                System.err.println("Number of successful calls before application ready: " + successfulCalls);
                e.printStackTrace();
                return;
            }
        }
    }).start();
}

@EventListener(ApplicationReadyEvent.class)
public void doSomethingAfterStartup() {
    applicationReady = true;
    System.out.println("Context Ready Event received.");
}

In this scenario, the main thread is still building the mapper of other beans, that is, it keeps putting mappedStatement to mappedStatements, and in this asynchronous thread, it keeps getting mappedStatement from mappedStatements , because StrictMap inherits from HashMap, when the underlying HashMap performs the resize operation, before the old array element migration is completed and the asynchronous thread obtains the latest table reference, At this time, the mappedStatement that has been put before cannot be obtained from mappedStatements.
This is the source code of HashMap#resize in JDK8:
transient Node<K,V>[] table;

final Node<K,V>[] resize() {
    Node<K,V>[] oldTab = table;

    // omit newCap calculation

    Node<K,V>[] newTab = (Node<K,V>[])new Node[newCap];
    table = newTab;
    if (oldTab != null) {
        for (int j = 0; j < oldCap; ++j) {
            // omit elements reference copy
        }
    }
    return newTab;
}

In the source code, a new array newTab is allocated first, and the empty array newTab reference is assigned to table. Before the element migration is completed, the asynchronous thread calling mappedStatements.get(id) may not be able to get it mappedStatement(depends on seeing the latest table reference, note that table does not contain the volatile modifier. however, it can be obtained normally before and after resize), instead throws the following exception:
org.mybatis.spring.MyBatisSystemException: nested exception is org.apache.ibatis.exceptions.PersistenceException:
### Error querying database.  Cause: java.lang.IllegalArgumentException: Mapped Statements collection does not contain value for me.tianshuang.mapper.Mapper1.select1
### Cause: java.lang.IllegalArgumentException: Mapped Statements collection does not contain value for me.tianshuang.mapper.Mapper1.select1
	at org.mybatis.spring.MyBatisExceptionTranslator.translateExceptionIfPossible(MyBatisExceptionTranslator.java:96)
	at org.mybatis.spring.SqlSessionTemplate$SqlSessionInterceptor.invoke(SqlSessionTemplate.java:441)
	at com.sun.proxy.$Proxy47.selectOne(Unknown Source)
	at org.mybatis.spring.SqlSessionTemplate.selectOne(SqlSessionTemplate.java:160)
	at org.apache.ibatis.binding.MapperMethod.execute(MapperMethod.java:87)
	at org.apache.ibatis.binding.MapperProxy$PlainMethodInvoker.invoke(MapperProxy.java:145)
	at org.apache.ibatis.binding.MapperProxy.invoke(MapperProxy.java:86)
	at com.sun.proxy.$Proxy50.select1(Unknown Source)
	at me.tianshuang.MybatisRaceConditionApplication.lambda$raceConditionTest$0(MybatisRaceConditionApplication.java:25)
	at java.lang.Thread.run(Thread.java:750)
Caused by: org.apache.ibatis.exceptions.PersistenceException:
### Error querying database.  Cause: java.lang.IllegalArgumentException: Mapped Statements collection does not contain value for me.tianshuang.mapper.Mapper1.select1
### Cause: java.lang.IllegalArgumentException: Mapped Statements collection does not contain value for me.tianshuang.mapper.Mapper1.select1
	at org.apache.ibatis.exceptions.ExceptionFactory.wrapException(ExceptionFactory.java:30)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:153)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:145)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:140)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.selectOne(DefaultSqlSession.java:76)
	at sun.reflect.GeneratedMethodAccessor34.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.mybatis.spring.SqlSessionTemplate$SqlSessionInterceptor.invoke(SqlSessionTemplate.java:427)
	... 8 more
Caused by: java.lang.IllegalArgumentException: Mapped Statements collection does not contain value for me.tianshuang.mapper.Mapper1.select1
	at org.apache.ibatis.session.Configuration$StrictMap.get(Configuration.java:1054)
	at org.apache.ibatis.session.Configuration.getMappedStatement(Configuration.java:844)
	at org.apache.ibatis.session.Configuration.getMappedStatement(Configuration.java:837)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.selectList(DefaultSqlSession.java:150)
	... 15 more

The root cause of this problem is that multiple threads access mappedStatements concurrently, and the main thread has made structural modifications to mappedStatements, which does not conform to the usage specification of HashMap, so in order to solve this problem, we should make StrictMap inherited from ConcurrentHashMap, at the same time, in order to be compatible with the changes of the containsKey method and adapt to the NULL key, I rewrote the containsKey method.
This project can reproduce the race condition: GitHub - tianshuang/mybatis-race-condition
Reference:
java - HashMap resize() while single thread writing and multiple threads reading - Stack Overflow
Unit Test
All unit tests pass.

Hello @tianshuang ,
It seems like the same issue as https://github.com/mybatis/spring/issues/696 which may not be resolved by this change.

we will create asynchronous thread to perform some time-consuming SQL operations to avoid blocking the application startup process.

I understand why people do this, but the current MyBatis design may not allow this approach (see my comments on the linked issue for why).
So, let me clarify. Did you verify that this change actually fixes your problem?
If you did, please let me know and I'll look into it.
But it may take some time as I don't have much spare time lately.

Yes, this PR fixed my problem, my problem is similar to https://github.com/mybatis/spring/issues/696, but the problem I am facing has no dependencies between mappers, just because mappedStatements.get and mappedStatements.put are called concurrently by two threads, and at this time the underlying HashMap is executing the resize method. Note that exceptions only occur during resize.
As you say in https://github.com/mybatis/spring/issues/696#issuecomment-1200287147:

The fundamental difficulty is that there is no way to know if all mappers are loaded or not (note that addMapper could be called at any time, even from a user's code).

Since we can't prevent concurrent calls, why don't we make MyBatis more robust? This PR fixed the occasional exception: Mapped Statements collection does not contain value, details can be found here: GitHub - tianshuang/mybatis-race-condition.

Any ideas?

Coverage increased (+0.01%) to 87.309% when pulling 7996f963548106fd1ebaa8b3c52b20c284e6d572 on tianshuang:race_condition into 043aaaa6bc11e3f06e1e85dd4f990b55346612e8 on mybatis:master.

@tianshuang ,
Thank you for the detailed explanation! I could verify the effect.
It should be included in the latest 3.5.12-SNAPSHOT


Can't remove from MacOS "Login Items"

I specified to add MonitorControl to "Login Items" when installing the app.  Now I want to remove it (because it does not work: Seems I have the same issue as reported in https://github.com/the0neyouseek/MonitorControl/issues/61).  But:
[1] When I got to System Preferences > Users & Groups > Login Items, MonitorControl is not listed
[2] Nevertheless, when I reboot, MonitorControl does auto-launch.
Any advice how to prevent this app from auto-launching?

Have you tried checking/unchecking Start MonitorControl at login? in General settings?

Thanks.  That worked: the app no longer launches at login.
Can you clarify why this does not work as expected?  (e.g. When I click in the app UI to set it to "Start at Login", it does not appear in the System Preferences "Login Items" list.)

I honestly have no idea.. my guess is that there are multiple ways to set items as login items and they're not consistent.
I think the login items are a way for you to set specific apps to launch at startup yourself, although we might be able to set it there too, not sure.
You also have, for example, /Library/LaunchDemons/ and `/Library/LaunchAgents' who also manage startup items that won't show up in that list.

Thanks --


As a Engineer, I would like to see better validation reports, so I can easily understand where the OpenAPI has errors

Right now the validation reporting is very modest. Let's make it better!

Accepted!


add javascript operators tutorial

Please fill in this pull request template before submitting
1. This pull request resolves #20
2. Description
JavaScript operators tutorial added (down below 👇 is the list of concepts covered in this tutorial)

Assignment operator ( = )

Arithmetic operators

Addition operator ( + )
Subtraction operator ( - )
Multiplication operator ( * )
Division operator ( / )
Exponentiation operator ( ** )
Remainder operator ( % )
Increment operator ( ++ )
Decrement operator ( -- )

Comparison operators

Equality operator ( == )
Strict equality operator ( === )
Inequality operator ( != )
Strict inequality operator ( !==)
Greater than operator ( > )
Greater than or equal to operator ( >= )
Less than operator ( < )
Less than or equal to operator ( <= )

Logical operators

Logical AND ( && )
Logical OR ( || )
Logical NOT ( ! )

3. Fill in checklist by marking [x]

[x] I've read the CONTRIBUTING.md
[x] I was assigned to this issue
[x] My code is formatted using required linting
[x] I've run my code locally and checked it works

@sabderemane  ahh my bad 😅, thanks for pointing this out.
Check now


Launcher syntax, Exllama backend requires all the modules to be on GPU

I want to use GPTQ models, but I'm struggling with the text-generation-launcher syntax. I was able to see inference working in Python, and have a gist every relevant detail I could think of. It feels like I'm just missing something basic about the launcher syntax:
$ CUDA_VISIBLE_DEVICES=0 text-generation-launcher --dtype float16  --model-id TheBloke/CapybaraHermes-2.5-Mistral-7B-GPTQ  --port 8041 --max-input-length 3696 --max-total-tokens 4096  --max-batch-prefill-tokens 4096
...
ValueError: Found modules on cpu/disk. Using Exllama or Exllamav2 backend requires all the modules to be on GPU.

I'm running TGI in a conda environment (not the docker container) with TheBloke/CapybaraHermes-2.5-Mistral-7B-GPTQ weights to try TGI out.
I got inference working in python (shown in the gist), but text-generation-launcher is throwing ValueError: [..] Exllama or Exllamav2 backend requires all the modules to be on GPU.
I'm already using CUDA_VISIBLE_DEVICES=0 because I just want it to use the GPU alone and not my iGPU. I don't seem to need to disable exllama for inference to work in Python, so I'm not sure why the launcher would throw that? I'm under the assumption that I want to use exllama for performance reasons, and the model's main branch was quantized for it. My gist shows lots of relevant stuff like torch.cuda.get_device_name(0)
Versions:

Python 3.9.18
cargo 1.75.0
libprotoc 25.1
Linux 6.1.71 (archlinux, btw)

Cuda:
$ nvcc --version
Cuda compilation tools, release 12.3, V12.3.103
Build cuda_12.3.r12.3/compiler.33492891_0

I'll go ahead and close this with the assumption that I just need to do more research before I can run gptq models using TGI.


BE-25: Research on Moderator Flags

What's up?
A research on superuser and isstaff flags for the moderator should be done.
To Do

[x] Research on superuser flag
[x] Research on isstaff flag
[x] Share findings with the backend team

Deadline
22.11.2022 @16.00
Deadline got postponed because of the busy schedule
Additional Information
No response
Reviewers
No response

In our application, we have multiple types of users and their ability to use some of the functionalities our app offers are different. To implement these differences we can use django flags. We have already made use of the login_required flag while implementing art item and comment-related API endpoints.
However, we need to use other flags to differentiate between a moderator and level1 user.

There are two flags we can make use of: is_staff and is_superuser.
is_staff flag designates if the user can log in to the Django Admin pages. You can specify permissions to this user and decide what they can do.
The is_superuser flag is an additional flag to assign all permissions without having to add them one by one.

I think we should use the is_superuser flag for the functionalities that only the moderator should be able to do.

To create a superuser in Django first reach the same directory as manage.py.
Run the following command: pyhton manage.py createsuperuser

Reviewers: @KarahanS @BElifb @serdarakol

Outcome of this task will be used in #277 and probably #279. We can close this task after we see that the implementation is working properly.


fixes the overflow issue cause by a long subtitle on a narrow screen …

…device
Description
This fix wraps the content column of the vertical stepper in Expandable() to avoid potential overflow.
Before:

After:

@chunhtai Thank you for the review. I took over this PR from @ayushin by his request, was easier just to make another one - https://github.com/flutter/flutter/pull/61623
I've addressed these comments you've made over there.

@iJimmyWei sounds good to me, i will close this one.


Add job for building machine-api-operator image

Job for building machine-api-operator image

/cc @Rajalakshmi-Girish

Currently, these changes are just building things for ppc64le platform, I will have to work on building these images on x86 as well and push fat manifest.

Currently, these changes are just building things for ppc64le platform, I will have to work on building these images on x86 as well and push fat manifest.

Do you mean someone who is working from x86 and would like to have machine-api-operator for/on power can use this image?
Apologies if I am framing the question wrongly :)

/lgtm

Currently, these changes are just building things for ppc64le platform, I will have to work on building these images on x86 as well and push fat manifest.

Do you mean someone who is working from x86 and would like to have machine-api-operator for/on power can use this image?
Apologies if I am framing the question wrongly :)

It's like someone can trigger this code from x86 box as well..

Currently, these changes are just building things for ppc64le platform, I will have to work on building these images on x86 as well and push fat manifest.

Do you mean someone who is working from x86 and would like to have machine-api-operator for/on power can use this image?
Apologies if I am framing the question wrongly :)

It's like someone can trigger this code from x86 box as well..

Infact ONLY from x86 box as we have image only built on iks prow cluster and not on ppc64le.


404 Error when using pagination on an entries tag inside reefine tag

When using pagination on an entries tag inside a reefine tag I always get a 404 when going to the "second" page (or further pages in the pagination). I am using the url-method and I think it has something to do with the segment parsing!?

For anyone else reading this, the issue was using Structure and the problem was fixed using Freebie https://github.com/Patrick64/reefine#structure


Include adapters such as mysql2 and jdbc-mysql

Looking at the code, it doesn't seem like anything would not be compatible with any adapter that uses mysql, so making the check a little more flexible to make sure that whatever the adapter may be, it's connecting to a mysql database.

I'm not maintaining this gem anymore.  I'd be happy to give it to someone else.


Display Brc20 & Rune prices

You might need a backend proxy, a DB, a caching mechanism etc ... for this to hide the API key of the prices provider.
It might be a big task but I think it would really boost the user experience (mine at least :) ).
again, happy to work on this.

I believe this data is all provided by the Best in Slot API, which is already integrated so it'd be a matter of adding support for the relevant endpoint: https://docs.bestinslot.xyz/reference/api-reference/ordinals-and-brc-20-and-runes-and-bitmap-v3-api-mainnet+testnet+signet

Nice. I can give it a go after the manage-tokens pr merges.

BRC20 prices were previously supported but a later update broke it, should be back up soon: https://github.com/leather-io/extension/issues/5626
Fiat value for Runes we have an existing request here: https://github.com/leather-io/extension/issues/5300
Runes is a highly requested feature! Would love to see this happen.

I'm closing out this issue in favor of the others linked above.


compartment model (SEIR) example request

Dear Lao,
Given that COVID-19 starts to impact countries globally, people have grown interest in using epidemic models to model COVID 19 infectious.
Could you please share an example of the compartment (SEIR) model in PyMC3? That is, using MCMC to estimate the parameters in SEIR model.
I have been looking around online, but haven't found an example that actually work.
Best regards,
Yijun

@Dpananos started to work on it in https://github.com/Dpananos/WuhanFlu

Hi, Dpananos' github hasn't updated for four months. I wonder how would this go. Any suggestion?

If you're interested in doing ODE inference in PyMC3, have a look at DifferentialEquation.  The SEIR model has many variants, and I'm not sure which variant you're interested in looking at.  Consequently, I can't really advise on how to implement without more details and can only point you towards the documentation.

@yijunwang0805 There are multiple effort on modeling SEIR like model, you should check out:
https://github.com/Priesemann-Group/covid19_inference_forecast/
https://github.com/Priesemann-Group/covid19_inference/
You can find some minimal example to illustrate the idea here: https://discourse.pymc.io/t/how-to-sample-efficiently-from-time-series-data/4928/3?u=junpenglao


Create factory functions to allow for element modification.

I'd like to propose that elements that are displayed in the editor be created using factory functions.
I am currently attempting to adjust the way a node is presented on screen.  My requirement is that I be able to adjust size and position attributes of a node's background, border, labels, icon, and sublabels.
In the current the-graph-node.js file, the React elements are created directly:
React.DOM.rect({
  className: "node-bg", // HACK to make the whole g draggable
  width: TheGraph.nodeSize,
  height: TheGraph.nodeSize + 35
}),
React.DOM.rect({
  className: "node-border drag",
  width: TheGraph.nodeSize,
  height: TheGraph.nodeSize,
  rx: TheGraph.nodeRadius,
  ry: TheGraph.nodeRadius
}),

My proposal is that many (all?) React.DOM calls be abstracted away to allow for interception:
Proposed code attached to root TheGraph object:
TheGraph.createNode = function(options) {
    return React.DOM.g(
        TheGraph.createNodeBackground({ className: 'node-bg' }),
        TheGraph.createNodeBorder({ className: 'node-border drag' }),
        // etc...
    );
}
TheGraph.createNodeBackground = function(options) {

    // Here could be some code for defaults, such as:
    if (!options.width)
        options.width = TheGraph.nodeSize;

    return React.DOM.rect(options);
}

TheGraph.createNodeBorder = function(options) {
    return React.DOM.rect(options);
}

Then the file the-graph-node.js would have a simple call in it's render function:
render: function() {
    // other code...
    return TheGraph.createNode(options);
}

Usage
With that in place, I could intercept each call and adjust all attributes accordingly:
var baseCreateNodeBackground = TheGraph.createNodeBackground;
TheGraph.createNodeBackground = function(options) {
    options.width = 200; // Increase width
    options.height = 50; // Decrease height
    return baseCreateNodeBackground(options);
}

Looks to be fixed in PR #157


'X-Frame-Options' to 'sameorigin' with CloudFront

Hi,
I'm trying to follow the instructions provided in the documentation. I have S3 bucket behind the CloudFront, and I whitelisted the CloudFront URL on Amazon connect. But when opening the page I created (as per instruction) I get an error:
Refused to display ' in a frame because it set 'X-Frame-Options' to 'sameorigin'.
I tried to whitelist the S3 bucket as well but with no change, but according to amazon-connect-streams documentation the whitelisting the CloudFront should be sufficient. I used HTTPS of course and embedded agent panel from the documentation example.
What am I missing? I'm using the latest 1.6.0 version of amazon-connect-streams. I see the documentation is not up to date as the whitelisting looks a little different on Amazon Connect now but I don't think thats the issue.
Thanks

Hello Piotr,
A few things: Make sure you are using the right ccpUrl(the one finishing with ccp-v2/ or ccp#/), make sure you are logged in your ccp in an other tab on the same browser (checkout #192 for details, logging in via iframe is not supported) and that you are not using "Incognito Window" as authentication token doesn't seems to work within the tabs in that mode. I got it working yesterday night via Cloudfront as well. Thanks

Hello Piotr,
A few things: Make sure you are using the right ccpUrl(the one finishing with ccp-v2/ or ccp#/), make sure you are logged in your ccp in an other tab on the same browser (checkout #192 for details, logging in via iframe is not supported) and that you are not using "Incognito Window" as authentication token doesn't seems to work within the tabs in that mode. I got it working yesterday night via Cloudfront as well. Thanks

Hello Piotr,
A few things: Make sure you are using the right ccpUrl(the one finishing with ccp-v2/ or ccp#/), make sure you are logged in your ccp in an other tab on the same browser (checkout #192 for details, logging in via iframe is not supported) and that you are not using "Incognito Window" as authentication token doesn't seems to work within the tabs in that mode. I got it working yesterday night via Cloudfront as well. Thanks

Yes, it is working if you are logged in on another tab already. But why the login window itself is not working? Is there any workaround?

Hello Piotr,
A few things: Make sure you are using the right ccpUrl(the one finishing with ccp-v2/ or ccp#/), make sure you are logged in your ccp in an other tab on the same browser (checkout #192 for details, logging in via iframe is not supported) and that you are not using "Incognito Window" as authentication token doesn't seems to work within the tabs in that mode. I got it working yesterday night via Cloudfront as well. Thanks

Yes, it is working if you are logged in on another tab already. But why the login window itself is not working? Is there any workaround?

Hi,
Any workaround for this or is there any best practice recommendation on this problem?
Thanks

Hi,
Any workaround for this or is there any best practice recommendation on this problem?
Thanks

Facing the same issue even with the domains added as allowed origin. I've tried almost all the suggested solutions in all the open and closed issues in this repo but none of those seem to work. For me it initially looked like it worked with ccp-v2/, but the login popup would open, and once I logged in, the whole CCP console would open up in the popup itself; is this expected behavior? Is it supposed to open in a new tab or is it supposed to load inside of the iframe? I can't tell since I haven't seen it working properly, and the docs aren't helpful. That was yesterday, when I started to work on it today, I am noticing that it has stopped working entirely, and right now, the login popup is also not opening. I've tried ccp#/ , tried loginPopup: false but no dice.

Facing the same issue even with the domains added as allowed origin. I've tried almost all the suggested solutions in all the open and closed issues in this repo but none of those seem to work. For me it initially looked like it worked with ccp-v2/, but the login popup would open, and once I logged in, the whole CCP console would open up in the popup itself; is this expected behavior? Is it supposed to open in a new tab or is it supposed to load inside of the iframe? I can't tell since I haven't seen it working properly, and the docs aren't helpful. That was yesterday, when I started to work on it today, I am noticing that it has stopped working entirely, and right now, the login popup is also not opening. I've tried ccp#/ , tried loginPopup: false but no dice.


Write app using SampleProblemDetailsWriter

Need complete app using Custom IProblemDetailsWriter
The code is at https://github.com/dotnet/AspNetCore.Docs.Samples/tree/main/fundamentals/middleware/problem-details-service
This code is needed for this comment
@sammychinedu2ky  can you do this?

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

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: riande

Associated WorkItem - 89655

We've already done this before 😅

We've already done this before 😅

Do you know where the code is at?

Here is it @Rick-Anderson
https://github.com/dotnet/AspNetCore.Docs.Samples/pull/25#event-7309770452

@sammychinedu2ky  right, but we never register SampleProblemDetailsWriter in Program.cs

Need complete app using Custom IProblemDetailsWriter
The code is at https://github.com/dotnet/AspNetCore.Docs.Samples/tree/main/fundamentals/middleware/problem-details-service
This code is needed for this comment
@sammychinedu2ky can you do this?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 0e03fdf4-b19b-17c3-04f4-336ab6d2ee06
Version Independent ID: 38515dfb-91a5-b395-db9d-084bbaf095c8
Content: Handle errors in ASP.NET Core
Content Source: aspnetcore/fundamentals/error-handling.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: riande

Associated WorkItem - 89655

So I should edit this code write?

So I should edit this code write?

Yup, just add a #define  and new snippet tag around it. Above that in the comments list instructions to test it.

Update the following
https://github.com/dotnet/AspNetCore.Docs.Samples/blob/main/fundamentals/middleware/problem-details-service/Program.cs


Update to Mbed Crypto v2.0.0

We're currently using Mbed Crypto 1.1.0 which exposed an old version of PSA Crypto API.
After the interface work is done, we should update the version of mbed we pull to 2.0.0 and modify the way we create and handle keys (where necessary).
Changes that will probably be needed:

Updating the way Mbed Provider handles key creation (as per the new PSA spec)
Updating the way we deal with key lifetimes (deciding how this will be handled is WIP)
Adding permit_copy as a flag for key attributes

If providers will be forced to only support one key lifetime, a new issue shall be raised.

This will also open up the possibility of changing the storage path of keys by defining PSA_ITS_STORAGE_PREFIX at compile time: https://github.com/ARMmbed/mbed-crypto/blob/development/library/psa_its_file.c#L48

Based on the work done in #52, there is no intention to make the current mbed provider work with volatile keys.
A new/joint provider that will only work with such keys will be created at some point, but we'll have a new issue for that.

We're going to find a workaround for a shortcoming in bindgen: https://github.com/rust-lang/rust-bindgen/issues/1344
Most of the functions for populating key attributes structures are defined as static inline. So we either have to populate things by hand or write our own FFI wrappers.


remove scroll from side-by-side records

This removes the scrolling feature for individual records on the side-by-side record comparison.

This looks good, and I can confirm that it works as planned in my local environment.  I'm willing to merge it if nobody else has any further observations.

:rocket:


Daily Motivational Quote extension

Is your feature request related to a problem? Please describe.
I'm always frustrated when I lack motivation or need a quick boost of inspiration during my day. It would be beneficial to have a way to receive daily motivational quotes, and the ability to get additional quotes when needed, directly within our application.
Describe the solution you'd like
I would like to implement a feature that displays a daily motivational quote to users. Additionally, there should be a button that users can click to receive another random quote. This feature should include an array of at least 15 different motivational quotes.
Describe alternatives you've considered
I have considered integrating third-party APIs that provide motivational quotes, but this approach may introduce unnecessary dependencies and potential costs. Another alternative is to send quotes via email or notifications, but this might be intrusive for some users.
Additional context
Here's a brief outline of the feature:
Daily Quote Display: Show a motivational quote that changes daily based on the date.
Get Another Quote Button: Include a button that, when clicked, provides a random motivational quote from a predefined list of at least 15 quotes.
contributing from Gssoc24

@Sulagna-Dutta-Roy please assign


Disable flserver datamodel in Fluent

Reading cases with large no. of zones from PyFluent consumes too much time/memory as flserver datamodel is enabled in Fluent. Setting the env. var. APP_LAUNCHED_FROM_CLIENT (added for lbapp/gpuapp) in Fluent disables the flserver datamodel.

Reading cases with large no. of zones from PyFluent consumes too much time/memory as flserver datamodel is enabled in Fluent. Setting the env. var. APP_LAUNCHED_FROM_CLIENT (added for lbapp/gpuapp) in Fluent disables the flserver datamodel.

flserver will be required for icing/aero, but we can disable this line when fluent is launched with icing/aero as an option.
... i had some work related to that in https://github.com.mcas.ms/pyansys/pyfluent/pull/417 put this is months old and i'll need to revisit it soon.


Use array for local variables

Local variables resolved by the resolver pass are still stored in nested environments by the interpreter and looked up by name. They should instead be associated with a unique ID by the resolver and added to a single 1D list in the interpreter for better efficiency.

what's an array?

what's an array?


chore(labeler): add component for audio matches

can you post a screenshot, if it makes sense then I will just merge

While Bo was reviewing, he found some problems. We'll put this in draft as we figure it out.

please refer to this ticket in core.


Fix the backspace key that sometimes eats clicks

The SVG image on the backspace key needs "pointer-events: none" so that clicking directly on the image will pass through the click event to the button, instead of eating it.

Really clean solution. Thanks a lot. I added data-delete to the SVG and the PATH but this is way cleaner.


Revert "Disable failing print actions integration test in Firefox"

Reverts mozilla/pdf.js#14367, since https://bugzilla.mozilla.org/show_bug.cgi?id=1741698 has now been fixed.

/botio integrationtest

/botio integrationtest

Thanks and happy new year :).

Happy new year to you too!


Add hidpi support for Windows

With the Spacegray thing on ST3 (running on a Surface Pro 4 at 200% scaling), all of the assets are blurry. These include the arrows, tab close icon, tab overflow icon, and scrollbar:

Is it possible to introduce higher-res versions of these assets on Windows, or is this a ST3 bug?

Can you try restarting the editor? If that doesn't solve it, try reinstalling the theme and then restarting the editor right after that as well. Let me know how it goes.

I added dpi_scale: 2.0 in my user preferences and restarted the editor to no avail. I also tried reinstalling the theme with Package Control and restarting the editor, which also didn't work.

Is this happening to you with the other themes in Materialize as well? Or is it a Spacegray specific thing?

I tried Brogrammer and Solarized with the same results.

This is strange. The image you posted is usually what happens when the theme updates or when you haven't restarted the editor after installing it.
Does this problem occur when you use the original Material Theme?

Sadly, yes (I installed it, applied it, and restarted the editor). Should I report this to them instead?

Sorry to hear that. Unfortunately, Materialize is basically nothing more than a fork of Material Theme with some minor modifications. equinusocio is the mastermind behind the creation of all of this, so he might be better suited to deal with this problem. So I would definitely recommend raising an issue there, and then if he is able to fix his themes, I can try to replicate his fix within Materialize.

Okay. As a side note, I did run Ctrl+H on Material Spacegray.sublime-theme and replaced .png with @2x.png, and it did properly force the usage of hiDPI assets (with some missing ones on the scrollbar).

I can try out defaulting to the 2x assets, once I get home, I'll keep you updated.

Thanks! I've reported this to ST core: https://github.com/SublimeTextIssues/Core/issues/1073
The only potential issue I can see with forcing 2x assets is that margins are not respected on Windows (at least for the thumb_* images), so I have to downgrade to normal assets with those (not sure how the margins work) to get correct proportions. Not sure how that can be dealt with other than introducing additional margins.

Hi, just wanted to give you an update. I looked at the issue you made on the original Material theme. I really want to keep Materialize as closely aligned with the original Material theme as possible, so I'm going to hold off on making any changes until I see how he responds to it (I believe he has still left the issue open). Sorry about any inconvenience this may cause in the mean time!

It's fine! I do have my temporary workaround, so it's not all lost


The error is not thrown if there are cyclic dependencies

In the plugin parameters there is an option circleImportThrowErr. It does not work. Circular dependencies are simply shown in the console, but the build continues and completes successfully.
I think this is happening because the argument data is an object and does not have a length property. I think the problem can be solved quite simply – you need to check the length of the object keys:
function validateCircleData(ctx: Context, data: CircleData){
    if(ctx.circleImportThrowErr && Object.keys(data).length){
        throw new Error('has circular dependencies in this project')
    }
}

Thank you very much for reporting this issue, it was my mistake. I have solved this problem in this commit. https://github.com/threedayAAAAA/vite-plugin-circular-dependency/commit/faa301dc7c45470faf75335ab2c3f556b613179a
I will release a new version later and you can try to update to the latest version.
Thanks again sincerely for pointing out this problem for me.