[FLINK-13421] [runtime] Do not allocate slots in a MultiTaskSlot when it’s releasing children

What is the purpose of the change
ConcurrentModificationException happened in the children release loop in MultiTaskSlot#release.
It happened because new slot allocations happened, adding a new child to the MultiTaskSlot, thus modified the children field.
New slot allocations happened because a task failover is caused by a child slot's payload releasing. The failover canceled tasks and returned allocated slots to SlotPool. The returned allocated slots were assigned to some tasks that were not canceled yet. The slot assignment satisfies the input location preference constraints of some downstream tasks, and triggered the slot allocation of these tasks.
*To fix this, this PR changes MultiTaskSlot to be not available for slot allocation when it is in releasingChildren state. The change is in SlotSharingManager#listResolvedRootSlotInfo.(SlotSharingManager#getUnresolvedRootSlot is not necessary for a change as unresolved multi-task slot is simple and does not lead to a slot allocation.) *
Brief change log
(for example:)

in SlotSharingManager#listResolvedRootSlotInfo

Verifying this change
This change added tests and can be verified as follows:

Added uint test SlotSharingManagerTest#testResolvedSlotInReleasingIsNotAvailable

Does this pull request potentially affect one of the following parts:

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

Documentation

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

Thanks for opening this PR @zhuzhurk. I have to admit that I cannot completely follow how the ConcurrentModificationException occurs. Could you maybe show me a stack trace of the problematic behaviour? How does it happen that we assign a new child to a MultiTaskSlot while it is being released?
At the moment I don't see how the behaviour you've covered with the test could occur.
Do you have an example which reproduces the reported problem? With such an example I could better understand the problem or one could generate logs on DEBUG log level.

Moreover, it would be helpful to know which revision you used for generating the log snippet which is attached to the JIRA issue.

Thanks for opening this PR @zhuzhurk. I have to admit that I cannot completely follow how the ConcurrentModificationException occurs. Could you maybe show me a stack trace of the problematic behaviour? How does it happen that we assign a new child to a MultiTaskSlot while it is being released?
At the moment I don't see how the behaviour you've covered with the test could occur.
Do you have an example which reproduces the reported problem? With such an example I could better understand the problem or one could generate logs on DEBUG log level.

Hi, @tillrohrmann, I added a case to repro this issue at https://github.com/zhuzhurk/flink/commit/7ef30934841854ed7711a5a0eabf89ecefb0ec2d.
For the log, unfortunately we ran that round of stability test is running with some local path for FLINK-13241, and cannot find the revision number. But I think it's not a new issue, which may happen since 1.7 or 1.8.

Thanks a lot @zhuzhurk. I'll check out the test case and try to reproduce the problem locally.

Thanks a lot @zhuzhurk. I think with your test case I was able to understand the problem. Here is the description for future reference:
The underlying problem is that a releasing MultiTaskSlot might be reused by other slot requests as @zhuzhurk stated. This can happen in the following case: Assume we have the following topology with slot sharing enabled, eager deployment and two allocated slots s1, s2.
src1 \
src2 - sink1
src3 /

Flink would assign s1 -> src1 and s2 -> src2. Since sink1 has three predecessors it will wait for src3 . to get a slot assigned before it can get scheduled.
Now we fail s1. This will trigger a global failover which cancels src2. The cancellation of src2 will return s2 to the SlotPool where it is used to fulfill the slot request from src3. With the assignment of s2 -> src3, sink1 learns about the locations of all of its inputs and will be scheduled. Since slot sharing is enabled, s1, which is currently being released, will be assigned to sink1. Due to this assignment we modify the MultiTaskSlot#children map which causes the ConcurrentModificationException.

I've updated the PR @zhuzhurk. Please take a look and say if you are ok with it.


Doing both length 0 and containskey checks for x5c subject

Fixes #340
Made the code check that the subject field exists before checking the length of it to generate the a better exception being thrown by the verify code.

LGTM! Anything you want to comment on @aseigler?

I'll close this then as it has been overwritten by #344 which  has the same functionality.


Event mode round names too long

The new event mode names are long and are clipping in the overlay.  Seems to happen only on Roll Out and Jump Club that I can tell.

I would rather them just show with the normal name on both the overlay and the stats page.
p.s. Thanks for all your work on this.

No problem. Taken care of in v1.89


Use const ref for grpc_slice

We are copying the slice on every call creation, which is hurting
ping/pong traffic.
As per new style recommendation, we should use a const ref
for grpc_slice.

Thank you for the approval. The failures are unrelated and failing on all other PRs I have.


Question: Makefile, GNUmakefile & Makefile.toml

We currently have 3 makefile-like files:

Makefile
GNUmakefile
Makefile.toml

The Makefile just ensures that GNU make is used and then refers to GNUmakefile and the Makefile.toml is for cargo make. So my question is: do we need all of these?

I tried to run the GNUmakefile with a non-gnu make (NetBSD) and it seemed to work just fine, so can we just use Makefile or are there any GNU extensions we're using?
cargo make is handy, but the usage is undocumented, so I'm not sure how much it's used. I was actually surprised to see that most of the commands in there still work, because the last commit is 2 years old. Can we remove this without losing functionality? And if not, can we port some commands to normal make instead to remove the duplication? @rivy, you have made most (all?) commits to that file. Are you still using it?

@Arcterus had the Makefile/GNUmakefile set up when I came on board. I haven't really touched it much other than to add and keep a bit of basic WinOS support so that at least a quick make would work.
From the Makefile setup, I assume @Arcterus wanted to force use of gmake to avoid surprises between platforms.
Personally, I usually use cargo ... directly and depend on CI for most things these days.
But I set up cargo make as builder which was platform portable and friendly to new devs/users (including me at the time 😃; see cargo make help). It is a bit more generally capable/flexible than GNUmakefile, allowing use of the util scripts and features to automatically and dynamically determine build content (the makefile is hard-coded). I got distracted by other project considerations never took the time to add it to the README on top of the standard cargo ... instructions. But, it could be noted as a third build method for newer devs/users.
That being said, cargo make ... certainly hasn't taken off as the top cross-platform builder in this space, and I'd be happy if we could combine it all into make. But the makefile is only partially WinOS compatible at the moment. And make is crufty. Porting all the Makefile.toml capability to the GNUmakefile, while certainly possible, is complicated and will take some significant coding time and machinations to get a working, friendly, and cross-platform product. I have built a couple for some C/C++ projects if someone wants a reference to take a crack at it.
I would love a more capable, cross-platform, and friendly builder if someone can find/build something. I just haven't found it yet.

Thanks!
The Windows support is something I hadn't considered yet. That's indeed something to keep in mind.

Personally, I usually use cargo ... directly and depend on CI for most things these days

Yeah, me too, but it's not ideal. I often find myself typing long commands like:
cargo test --no-default-features --features ls

The cargo make command for that is certainly much easier :)

But the makefile is only partially WinOS compatible at the moment. And make is crufty.

I definitely agree here. make is very crufty and it's very easy to put a unix command in there by accident, breaking the Windows build.

I would love a more capable, cross-platform, and friendly builder if someone can find/build something. I just haven't found it yet.

There's just, which looks interesting, but otherwise I'm also not sure if there's anything that offers a significant improvement over make.

Question has been answered and this doesn't bother me anymore :)


Update Kubernetes External Secrets app to the latest version

This issue is a:

[ ] Bug Report
[x] Enhancement suggestion
[ ] Feature request

If the issue concerns an existing Marketplace application, please tag the maintainer of the application (found in manifest.yaml) below:
Enhancement suggestions / feature requests
Currently, we have Kubernetes External Secrets version 0.8.3 installed by Civo marketplace as a newer version 0.8.5 is available, so it needs to be updated to the newer version. We kindly request an update to the latest version of Kubernetes External Secrets on the Civo marketplace to ensure that we have access to the most recent features and improvements.
@dirien is the maintainer for this app, so I request you to please take a look into this.

go for it!🎉

Hello! 👋🏼 This requested version has been satisfied via https://github.com/civo/kubernetes-marketplace/pull/715

https://github.com/civo/kubernetes-marketplace/blob/42eb55a95ab8c7283451b0ddc07fa16666a0f8ab/kubernetes-external-secrets/manifest.yaml#L4
cc: @kunal-kushwaha


Add 'varnum' to proc contents statements in SAS script

Change any use of PROC CONTENTS as follows.
Current:

proc contents data=slice_grids;
run;

New:

proc contents data=slice_grids varnum;
run;

This orders the SAS output by variable position instead of alphabetically (by variable name). This is my preferred style.

@mrueschman is this only for slice_grids, or is for the other proc contents as well?

For both 'slice' and 'slice_grids'.


Can i use self hosted in local emulator

i have a config to create a self hosted expo with my local emulator and have following error
I don't have enough qualifications to rent a server :((
my app: https://github.com/Chuazz/phone-book-for-old-man
my clone server: https://github.com/Chuazz/self-host-expo-clone

Hello,
never seen this error, are you running a production .apk file on the emulator? it is not supposed to work in development.
About the server URL,  it may need to be in https, so that is another thing to check

i also try it on real device, i think my problem i server URL

looks like, maybe you can get a domain, point it to your public ip address and then use let's encrypt?

i just rent a domain then get same error with #26


Added the bayesian optimization code

Added the optimization code, documentation and an example notebook.

LGTM. Thanks!


Improved Explanation of Labels and Annotations

This Pull Request resolved the issue mentioned in #42524. I have added explanation and it makes clear that both labels and annotations can be used together.

Thank You @k8s-ci-robot for your guidance. I have consulted with the PR process documentation and also signed a CLA document.

@aj11anuj Thank you for pointing out this. In the new commit, I have revert it back to the original one.

/remove-language zh

/label tide/merge-method-squash

@tengqm Thank You for your guidance. In the latest commit, I fixed those mistakes. You can review now.

/lgtm

/approve


Support SingleApps for buildProducts

we don't need to create the workspace for single apps

frontend:
clone ...
cd ..
yarn install
copy .env to .env
yarn run webpack:prod -> bundles
backend:
clone ..
cd ...
make install


Simplify status handling in loadMetadata

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10454/
Test FAILed.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10456/
Test FAILed.

jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10458/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10460/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10474/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-PR-Builder/10476/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.3 please


Roughly apply horizontal navigation layout for desktop

I saw this PR late after I pushed our first release. For now, we have the back button. Let's wait to hear feedback before making further changes.


No window on first startup

When the user data dir does not exist yet, no window opens.

@jrieken Please review the fix.


Mobile view is scewed

Somehow the map shows a bit scewed in moblie phone views. Tested with Android.
Edit: added screenshots

I cannot replicate the problem with iOS or android. Do you have a screenshot?

This happens only in vertical mode on Android. In vertical mode, also the zoom buttons etc. are very small. Probably Maplibre interprets the Android phone viewport resolution somehow wrong.
When you turn the phone to horizontal, the shape is alright.
Probably this is an issue with MaplibreJS.

Potential fix in 4217a84ab81e6aa909e73bac1177fe172a31d677


How to import a subpackage and its report into an existing parent package?

Hello,
we exported an ABAP package to a ZIP archive. Afterwards we wanted to import the package to a different system.
Structure of export system:

Parent Package A

subpackage B (to export; this contains an ABAP report)

We would like to import it like this:

Parent Package C

 (thus a subpackage B is created and the report objects are stored here)

The parent package exists in both systems. The import system doesn't have the subpackage to import into; thus it must be created on importing the DEVC object.
We did export not just the report inside the package but also the package itself. We are able to see the DEVC object of the package of the exported report in the ZIP file. When we try to import this - regardless of PREFIX or FULL mode used - the content of the subpackage is directly imported to the parent package. Thus the content lays in package 'C' and the original subpackage where the package was inside is lost. On import the metadata, e. g. package description of the parent ABAP package was overwritten and all objects were laying in the parent package. No subpackage was created and thus the objects were not stored one level deeper (in the subpackage). The exported metadata contains no information about the original parent package.
Is there a way to import a report in an ABAP package into a SAP system so that the parent system is not holding the imported objects but the imported objects are kept in a subpackage and thus the package hierarchy/structure is kept as on the export system?
I hope it's not too vague what we want to achieve. We read the documentation and searched for similar issues but found none. This happens on a NW 731 system (export) and it's imported to a NW 740 system.
Thank you for your help!
Best Regards,

Your repos should be created for package A and C. Then export repo A.  Then import the zip into repo C.
If your packages are named as above, then both repos should be FULL mode. If you subpackage is calls A_B, then use PREFIX mode. It will then create subpackage C_B in the target system.

I will review this with a colleague and get back to you end of next week. Thank you for your help @mbtools.


Add logging, improve BaseStream, change to qt5

Codecov Report

:exclamation: No coverage uploaded for pull request head (dev@fc5a4ac).

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 8de88db...fc5a4ac. Read the comment docs.

In the future, do not make broad PRs like this one.

Resolves #15


What is the mask tensor during test?

As illustrated in the paper, the mask tensor is generated from ground truth. But how should I get the mask tensor during test?

In the online tracking stage, we use the predicted results to generate masks.
DiMP only updates the tracker when the current result is reliable. Similar to DiMP, we only generate masks in reliable frames and add them to the memory.


Update New-CsTeamsVoiceApplicationsPolicy.md

Update New-CsTeamsVoiceApplicationsPolicy.md with documentation for new settings

Learn Build status updates of commit f394eda:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a538080:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@JohanFreelancer9

@dariomws @tseward @get-itips

@JianpingChen Gentle reminder

@JianpingChen Please implement the suggested changes, so that we can proceed to merge this PR. Thanks

@JianpingChen Please respond and implement the suggested changes, in the even of no reply we may close this PR. Thanks

@tiburd kindly help. Thanks

Learn Build status updates of commit 4e9f640:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit e6b1b03:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 57978fd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsTeamsVoiceApplicationsPolicy.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@dariomws @get-itips @tseward


Issue with registering a new user.

I cloned your repo, set it up, started the server, redis, and sidekiq. I then went to register and it gave an error saying to look below for the actual errors but nothing else was showing. I traced that back on the command line to an issue with the role not being present. Some more digging led me to a possible fix by changing line 2 in user.rb to before_validation :set_default_role. What's happening is that it wants the role to exist when the validation happens. By changing the callback to a before_validation it happens and then the validations pass. I'll do a PR if this is agreeable.
Second thing, I can't login as an admin and I'm not sure why. From what I can tell, I need to login to the main site as the admin user, then switch to the admin url, and I should get the admin dashboard. However, when trying to login as admin to the main site, I get Completed 401 Unauthorized in 22ms (ActiveRecord: 4.2ms).
Third thing, right now the repo is hard coded to use Gmail as an smtp server, even in develop, and that fails. Thus the error log gets spammed with Sidekiq failures. There's a couple of gems available for handling emails in dev. Mailcatcher and LetterOpener are two of them. I typically use Mailcatcher because it's separate from my project and I just leave it running and all of my projects can hit it.
Lastly, how is this project managed? Is there a roadmap? Is anyone actively working on it currently? I know two things are getting it running Rails 5.2 so it's current, and updating to Bootstrap 4.

@blimey85  Thank you .
PR are welcome and will be accepted. I used to main this project in cloud9 but after it required paid membership and due to financial problem I stopped using it .
Here is the roadmap of this project
https://github.com/bloggervista/Bee-Connect#future-of-this-project
Currently I am on windows and planning to buy mac after having money and will upgrade to rails 5.2 and bootstrap 4.
Currently I am only maintaining it . If somebody want to maintain it I am happy to collaborate . For that please send email to<EMAIL_ADDRESS>And I want to write tests for it also but let see :)

If any problem feel free to reopen issue.


[Workflow] try/except not working correctly in workflow functions

Expected Behavior
Consider the following logic inside a workflow:
try:
    yield ctx.call_activity(submit_order_to_shipping, input=order)
except Exception as e:
    # Shipping failed, so we need to refund the payment
    yield ctx.call_activity(notify, input=f"Error submitting order for shipping: {str(e)}")
    yield ctx.call_activity(refund_payment, input=order)

If the submit_order_to_shipping activity fails, one would expect both the notify and refund_payment activities to run inside the except block.
Actual Behavior
However, as of the 0.2.0 workflow extension release, only the notify activity executes, and then the workflow fails.
The root cause is a bug in the Durable Task Python SDK: https://github.com/microsoft/durabletask-python/issues/20
Steps to Reproduce the Problem
Write a workflow where an activity failure triggers logic in an except block and tries to perform multiple steps inside the catch block.
Release Note
RELEASE NOTE: FIX try/except not working correctly in workflow functions

FYI @DeepanshuA @XavierGeerinck

Not actionable until https://github.com/microsoft/durabletask-python/issues/20 is fixed.

seems https://github.com/microsoft/durabletask-python/pull/21 is merged, so yeah will do dapr-py change to accomodate that


[e2e]: nodejs push main disttag slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5595634943
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.main.disttag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.main.disttag.slsa3.yml
Trigger: push
Branch: main
Date: Wed Jul 19 06:11:35 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5605167577
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.main.disttag.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.main.disttag.slsa3.yml
Trigger: push
Branch: main
Date: Wed Jul 19 23:49:20 UTC 2023
Tests are passing now. Closing this issue.


Updated dependencies + v0.1.8

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

Comparison is base (02780d9) 80.90% compared to head (05906af) 80.58%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #95      +/-   ##
==========================================
- Coverage   80.90%   80.58%   -0.32%
==========================================
  Files          15       15
  Lines        1000      984      -16
  Branches      100       97       -3
==========================================
- Hits          809      793      -16
  Misses        191      191

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


CLOUDSTACK-10434:Some APIs should have access check

Description
This PR fixes the CLOUDSTACK-10434. I think some APIs in  VolumeApiServiceImpl  lack access check. I will list them in below table. I also give the pattch to add the access check for the api in this table. Anyone chould change this table, If you think the APIs do not need access check and change their lable as "no".

API
Lack?

VolumeApiServiceImpl # updateVolume
yes

VolumeApiServiceImpl # detachVolumeViaDestroyVM
yes

VolumeApiServiceImpl # takeSnapshot
yes

VolumeApiServiceImpl # migrateVolume
yes

AccountManagerImpl#createApiKeyAndSecretKey
yes

LoadBalancingRulesManagerImpl#applyLBStickinessPolicy
yes

LoadBalancingRulesManagerImpl#applyLBHealthCheckPolicy
yes

TemplateManagerImpl#createPrivateTemplate
yes

SnapshotManagerImpl#updateSnapshotPolicy
yes

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[x] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

@lujiefsi why do you think this "solves" CLOUDSTACK-10434? I can not find that, it redirects me to CLOUDSTACK-10431, the highest ticket number to date.
Note that the jira board is only used for GSoC currently and issues are at github
this is in no way a comment on the code or the functionality of your PR!

@lujiefsi can you change base branch to 4.15 as it's marked bug fix

@lujiefsi why do you think this "solves" CLOUDSTACK-10434? I can not find that, it redirects me to CLOUDSTACK-10431, the highest ticket number to date.
Note that the jira board is only used for GSoC currently and issues are at github
this is in no way a comment on the code or the functionality of your PR!

@lujiefsi why do you think this "solves" CLOUDSTACK-10434? I can not find that, it redirects me to CLOUDSTACK-10431, the highest ticket number to date.
Note that the jira board is only used for GSoC currently and issues are at github
this is in no way a comment on the code or the functionality of your PR!

The issuse link is:
https://issues.apache.org/jira/browse/CLOUDSTACK-10434

@lujiefsi can you change base branch to 4.15 as it's marked bug fix

YES, i will!

@lujiefsi can you change base branch to 4.15 as it's marked bug fix

done see https://github.com/apache/cloudstack/pull/4859/files

@lujiefsi this PR may be closed in favour of https://github.com/apache/cloudstack/pull/4859
Generally if a PR exists for older branch, we can review/test merge it and forward-merge it until master branch.

@rhtyd i will close this this pull request and move the description to https://github.com/apache/cloudstack/pull/4859


DBZ-2193 Investigate the replacement of LB services with port redirects in OpenShift test-suite

Good job as a PoC. Two things which I would like to see

Move the port forwarding into Controller. Ideally into a standalone method which would be called from initialisation method.
The port forwarding should be stopped at the end of the test class -- a teardown method of Fixture class comes to mind as the first place where to put it.

@MartinMedek  pleas rebase this. It's about time we get this merged. Apology for the delay.

@jcechace Martin will also add kafka external listener with route and ssl to this PR.


A few glitches with IEEE template and bibilography

I worked out a minimal example that can reproduce the problem:
main.tex
\documentclass[conference,10pt]{IEEEtran}
\usepackage{lipsum}

\title{Example Paper}

\author{\IEEEauthorblockN{Author1 Name1 \IEEEauthorrefmark{1}, and Author2 Name2 \IEEEauthorrefmark{2}}
\IEEEauthorblockA{\IEEEauthorrefmark{1}University of One, City, State, Country
\IEEEauthorrefmark{2}University of Two, City, State, Country}}

\begin{document}
\maketitle

\begin{abstract}
\lipsum[1]
\end{abstract}

\begin{IEEEkeywords}
Keyword 1, Keyword 2
\end{IEEEkeywords}

\section{Example Section}

\lipsum[2]~\cite{paper1}

\bibliographystyle{IEEEtran}
\bibliography{ref}

\end{document}

and ref.bib
@article{paper1,
  title={{Paper Title}},
  author={Author1, Name1 and Author2, Name2},
  journal={Journal 1},
  volume={1},
  number={1},
  year={2023}
}

Except for the font issue, mentioned in dginev/ar5ivist#3, the glitches are highlighted in the red circles:

@SubtleMuffin took a quick look - the arXiv submission pipeline doesn't actually process bibliography files (via bibtex, biber, etc) but instead expects the authors to do so locally, and submit the final .bbl file that is produced from the bibliography runs.
If you had a main.bbl file alongside main.tex, the ar5ivist run should work out. I have explicitly disabled the bib processing there, to stick close to the arXiv pipeline. In a way it is a feature, but maybe we need a better warning message to signal why the .bib was ignored.

Whoa thanks for the explanation!
Re bib: is it possible to enable the bibtex feature if we intend to?
Re author mark: cool! Can't wait for it to merge!


Error from server

After fixing the error mentioned in #16 I get the following error:
Error from server (Forbidden): error when creating "controller.yaml": roles.rbac.authorization.k8s.io "sealed-secrets-key-admin" is forbidden: attempt to grant extra privileges: [{[get] [] [secrets] [sealed-secrets-key] []} {[create] [] [secrets] [] []}] user=&{client  [system:authenticated] map[]} ownerrules=[] ruleResolutionErrors=[]
Error from server (Forbidden): error when creating "controller.yaml": clusterroles.rbac.authorization.k8s.io "secrets-unsealer" is forbidden: attempt to grant extra privileges: [{[get] [ksonnet.io] [sealedsecrets] [] []} {[list] [ksonnet.io] [sealedsecrets] [] []} {[watch] [ksonnet.io] [sealedsecrets] [] []} {[create] [] [secrets] [] []} {[update] [] [secrets] [] []} {[delete] [] [secrets] [] []}] user=&{client  [system:authenticated] map[]} ownerrules=[] ruleResolutionErrors=[]

I have just created a fresh cluster on azure, via acs-engine running on k8s 1.6.6 - not quite sure where the issues is.

Ah, I think https://github.com/Azure/acs-engine/issues/680 is probably the issue here.

Thanks for reporting the issue.  I did wonder whether including the RBAC rules by default would cause problems for anyone, and it seems "yes" :(
The easy workaround is to download the controller.yaml and remove all the RBAC rules.  Please let me know if https://gist.github.com/anguslees/cd81816ada739258dd493114818753a6 works for you (actually verify that it can create secrets, etc not just that kubectl create works).  If it's useful, I can include this simpler manifest in future releases.

@anguslees Yeah including that in future releases would indeed be useful, at least for a while, at some point it would probably good to push everyone to use RBAC. Anyway, I updated your snippet to 0.3.0 and then ran the commands from the readme and all working. Thanks for posting this so quickly.


fix(behavior_path_planner): check modified goal uuid in isEgoOutOfRoute (#3678)

Description

cherry-pick
https://github.com/autowarefoundation/autoware.universe/pull/3678
Related links

Tests performed

psim in main
Notes for reviewers

Interface changes

not applicable
Effects on system behavior

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

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

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

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

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

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

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

merged in https://github.com/tier4/autoware.universe/pull/433


Undertale Yellow AFTERLIFE Guitar Metal Remix

Song Name
Undertale Yellow AFTERLIFE Guitar Metal Remix
Artist Name
TobyFox
Source
Youtube
Youtube Link
2T7bvfs6eWk
Direct File Link
No response
Song ID
711157
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 63166980
Downloaded: No

Hie


Logout option ?

Hello,
I am using Netbird on Windows 11 and I can't find a way to log out.
I want to disconnect my client and reconnect it to another account.
I have already tried to reinstall the client, but the configuration seems persistent.
Any idea to deal with this case ?

Hey @Sykursen
We definitely need to add this option. Here is a quick workaround:
The only possible way now is to remove conf file located at C:\ProgramData\NetBird\config.json

Hi, thanks for your answer.
And I don't have a use case with several accounts. I just recreate a new one :)

I would like to raise this issue again, since it appears to be forgotten. There is still no way to log out from the CLI. I had to delete /etc/netbird/config.json with sudo.


getCharsFromUtf8() asymmetry with putCharsToUtf8()

@leventov
This is more of a comment than a real issue, but here FYI.
getCharsFromUtf8() originally returned void.  But understanding how many characters were actually decoded can be valuable to know, especially if the user needs to increment a pointer into the destination Appendable.  But it is not possible for the method to know how many characters already existed in the destination before appending.
I have updated the method to return an integer of the number of characters decoded in the current PR.
The asymmetry is that  putCharsToUtf8() returns the position in the WritableMemory after the last byte has been written from the encoding process instead of the number of bytes encoded.
A question for you is should we change the putCharsToUtf8() to return the number of bytes encoded?  This would make the two methods more symmetric.
We had no tests for the case where the destination Appendable already had some content so I added a test for that.

Yes, makes sense to make symmetric

Will do.  Thank you!


accordion - Talk to the other parent

last bullet item is indented where it should be a bullet point in line wtih the other 7

I don't think it should be another bullet, the 8th point is more of a note on the 7th bulleted item. I will fix it now but want to check w jeannie.

Change intro and bullet instead of number


reduced doubled code

the if (child.get('opened')) { statement is redundant because in the IF and ELSE statement are identical

Will be merged in reverse-proxy-adjustments branch, since we are refactoring the frontend. Keep in mind that this branch will not work with your current backend.


Question about inter eye distance (IEM)

Hi Patrik,
I want to ask about rcr-train and rcr-detect files.
Algorithm calculates inter eye distance, understand that it's needed to better calculate eye landmarks.
But if it's really required to use IEM to train model in rcr-train? For example, I only want to detect nose or mouth to get better speed but don't know if I can easily rebuild rcr-train without using IEM functions.
To sum up, I want to ask:
If inter eye distance is needed ONLY to calculate eye landmarks or whole algorithm depends on that(in some way)?
Thank you in advance for your time.

The IED is not used to better calculate eye landmarks. It's for computing a resolution/size-independent shape update. So basically you want to compute the shape update relative to the current face size, and not in some arbitrary absolute units (e.g. pixels). See our RCRC paper for more details.


revert back to unstacked nft cards

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[x] Bugfix

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #7480

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

🤖 Generated by Copilot at 2676612
This pull request removes the concept of stacks from the NFT gallery UI and backend. It simplifies the data structure and query for fetching NFTs with metadata, and updates the components that display and interact with NFTs accordingly. It also deletes or disables some unused or redundant code related to stacks.

🤖 Generated by Copilot at 2676612

The stacks of NFTs are no more
They were removed from the useNft store
The NeoNftCard and ItemsGridImage too
Were updated to use the new data view
And useItemsGrid got a refactor and more

I hope we can see this stacks again~

we will :wink:

Does not make sense imo.
Squids run on full power anyway.

@vikiival
thought you said yes to revert.
np. close it

thought you said yes to revert. np. close it

Yup, but loooks like @yangwao is in favour of revert


Move shortcuts implementation into feature module

Moving functionality out of :client module

This PR was released with 0.22.1


PoliCheck support files

Adding PoliCheck support files.

Hi @anmenaga, I would like to help get this PR merged for you, but as far as reviewing it goes, I have no context whatsoever for these changes. Can you please provide some background so that I can understand the intent of these changes and then review them, to move this forward?

Thank you.
This repo was planned to move to a new build system. That system has some automated checks; one of them - PoliCheck; files in this PR enable this repo for it.


Comp Current Season High Request needed

Could there be a request to view a comp rank from the current season high? We use this in our league to ensure fair play. Thank you for what you've already created, simply awesome!

Hi @j0nny55555, thank you for submitting a new issue, this will help the Bot to improve.
However as far as i know, there are no websites/API providing Season High. If you know one, please don't hesitate to report me that!
Also thank you for the compliment! I'm sorry about this problem, but without a source, i can't provide a command which returns Season High :(

Hi @j0nny55555, thank you for submitting a new issue, this will help the Bot to improve.
However as far as i know, there are no websites/API providing Season High. If you know one, please don't hesitate to report me that!
I'm happy you like what i did, this means a lot for me!


Context pooling is causing server panics

We deployed go 1.7 + chi 2 yesterday and noticed occasionally a machine would go out of service with:
panic: context: internal error: missing cancel error

goroutine 156 [running]:
panic(0x675540, 0xc42012f880)
	/opt/go-17/src/runtime/panic.go:500 +0x1a1
context.(*cancelCtx).cancel(0xc42001a600, 0x476900, 0x0, 0x0)
	/opt/go-17/src/context/context.go:339 +0x24b
context.(*timerCtx).cancel(0xc42001a600, 0x0, 0x0, 0x0)
	/opt/go-17/src/context/context.go:413 +0x4a
context.propagateCancel.func1(0x803660, 0xc42001a540, 0x800f20, 0xc42001a600)
	/opt/go-17/src/context/context.go:264 +0x128
created by context.propagateCancel
	/opt/go-17/src/context/context.go:267 +0x1dc

It appears chi is pooling custom context objects, but these objects can live longer than a request due to this snippet in context.go
func propagateCancel(parent Context, child canceler) {
	if parent.Done() == nil {
		return // parent is never canceled
	}
	if p, ok := parentCancelCtx(parent); ok {
		p.mu.Lock()
		if p.err != nil {
			// parent has already been canceled
			child.cancel(false, p.err)
		} else {
			if p.children == nil {
				p.children = make(map[canceler]bool)
			}
			p.children[child] = true
		}
		p.mu.Unlock()
	} else {
		go func() {   /// <<<<<<< This may live longer than a request, and the parent.Err() may have been reset between Done() and Err().
			select {
			case <-parent.Done():
				child.cancel(false, parent.Err())
			case <-child.Done():
			}
		}()
	}
}
```

This PR includes a failing test and the removal of the pooling.

It appears that this wouldn't be a problem if chi didn't create its own type of context object and just used Values, perhaps that would be a better approach? It would then fall into the `parentCancelCtx` part of `propagateCancel` and save a goroutine.

@Vektah thanks for the report and test case! I'll give this a bit more thought to consider what is the best way approach to solve this in chi. If you have other ideas please let me know too.

I'll see if we can maintain RouteContext pooling, and instead storing it as a value in the context instead of as a context.Context type, like you suggested.

The context API has been designed to discourage mutating contexts, which seems to contradict any kind of context reuse.

@lwc for sure, the current code doesn't reuse request context objects, it just reused the routing object, but clearly the way its adding itself to the context value chain is being problematic. I could get rid of the pool entirely, and I might.. I'm just trying a few things right now but I've been at a conference all day. Will get this in soon.

Let me know what you guys think of https://github.com/pressly/chi/pull/75 -- this does the job, and it maintains pooling of the route context, which isn't really a context object, its just the routing details. The benefit is the benchmarks are better than removing the pool altogether.


Tablet media query not working?

I am new to Neat 2.0 and having problems with media queries.
My code:
`$neat-grid: (
columns: 12,
gutter: 2em,
);
$mobile: (
columns: 12,
gutter: 1em,
media: 'screen and (min-width: 320px) and (max-width: 767px)'
);
$tablet: (
columns: 12,
gutter: 1.1em,
media: 'screen and (min-width 768px max-width 1024px)'
);
.element {
@include grid-column(3);
@include grid-media($mobile){
@include grid-column(4);
}
@include grid-media($tablet){
@include grid-column(6);
}
}`
It only shows desktop and mobile but tablet is never rendered.

Could you reformat your code?

'screen and (min-width 768px max-width 1024px)'
Should be
'screen and (min-width: 768px) and (max-width: 1024px)'

Oh that was it, thank you so much :)


Please add function "Export To" or "Save As"  xlsx Excel format in Editting Toolbar

Please add function "Export To" or "Save As"  xlsx Excel format in Editting Toolbar

Please add function "Export To" or "Save As" xlsx Excel format in Editting Toolbar

This feature is hard to implement.


P1 Only let users save facility addresses in supported states

If a user tries to save an address in a state where we are not live, they should not be able to save and should see an error message.
Design options:
-Limit dropdown to states where we are live
-Show the user an error message on save
@katiealoisi & Heather can we get a design for this?

We decided to show the full list of states, but provide an error as soon as a user selects a non-supported state (rather than after clicking save).
We did not want to limit the list of states because it would not be clear to users why the other states or not listed.

Looks good! Where is the link for what states are supported going to be?
I worry about direct linking to resources in the UI because it's easy for them to get out of sync and break :(

The link goes to the resources page - specifically here: https://cdcgov.github.io/SimpleReport_Public_Site/resources/using-simplereport/manage-facility-info/find-supported-jurisdictions/
What are you concerned about getting out of sync?

If we change the link it will break the app, and sometimes that's not obvious.

I have this concern in general in linking directly to documentation. So I think it's a bigger thing on do we do direct links, or just generally say go to simplereport.gov/resources

We can just be sure to set up redirects if we change URLs

Yeah I don't think directing them to simplereport.gov/resources will be
helpful since they'd have to wade through that page to find the section on
supported jurisdictions. Are you okay with Heather's suggestion to set up
redirects instead?
On Fri, Feb 26, 2021 at 2:12 PM hmyers-cms<EMAIL_ADDRESS>wrote:

We can just be sure to set up redirects if we change URLs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CDCgov/prime-simplereport/issues/915#issuecomment-786840786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYEZRDVM6IXRGXEVXMEPRDTA7XDXANCNFSM4YDEL3JQ
.

I think this is going to end up being painful for us, but I’m willing to go with it for now.
Redirects are also work, so if  this ends up being more than a small effort I’ll want to remove the links at that point.
From: Katherine Aloisi<EMAIL_ADDRESS>Reply-To: CDCgov/prime-simplereport<EMAIL_ADDRESS>Date: Friday, February 26, 2021 at 1:46 PM
To: CDCgov/prime-simplereport<EMAIL_ADDRESS>Cc: "Beckett, Alicia (CDC/OCOO/OD)"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [CDCgov/prime-simplereport] P1 Only let users save facility addresses in supported states (#915)
Yeah I don't think directing them to simplereport.gov/resources will be
helpful since they'd have to wade through that page to find the section on
supported jurisdictions. Are you okay with Heather's suggestion to set up
redirects instead?
On Fri, Feb 26, 2021 at 2:12 PM hmyers-cms<EMAIL_ADDRESS>wrote:

We can just be sure to set up redirects if we change URLs
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CDCgov/prime-simplereport/issues/915#issuecomment-786840786,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYEZRDVM6IXRGXEVXMEPRDTA7XDXANCNFSM4YDEL3JQ
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/CDCgov/prime-simplereport/issues/915#issuecomment-786910920, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMCKUGDSNVXVKFWSSRUY6UDTBAJERANCNFSM4YDEL3JQ.

Some feedback on the proposed design:
I'm assuming this is an auto-error rather than an error when submitting the form. Is that assumption accurate?
It's not accessible to put this alert box after the alert bc you want to give the person a chance to correct it. And specifically in that location, it skips the subsequent form field too.
If the error is on submit, the alert should go at the top of the form. If it's an auto-error, it should use an inline error. When this is single column #1231, there'll be space to add more text.
/cc @vz3
Also, I did not see figma link or would share feedback in there.

Yes, this is an auto-error. We don't want the user to fill out the entire form only to realize they can't proceed.
It sounds like the way to proceed is to first make the facility form single column (#1231) then move it to an inline error.

FYI the single-column form was merged, not sure if you're still working on this @nathancrtr?
tagging @Rebecca-LM for content review of the error message.

@maya I would change "is not currently..." to "isn't currently..." (The aim is to get language towards writing like we speak/plain language.) Also add a period after the word "supported" in the line below (but do not hyperlink the period).


notification for incoming commits

It would be very useful any kind of notification when there were incomming commits in git repositories.

Automatically, VSCode shows when there are incoming commits but an outstanding notification would be great

The git extension exposes API so it would be trivial for you to implement an extension which could provide that.


PoS milestones update & additional PoS contents Outline

Can you run a Reflow (vs code / cursor plugin makes it easy) to get line-length to 80 chars please, so we're linted. I'll also be adding Vale for linting MS Style soon, that will also be part of the build process.


Updates to dashr-studybrowser

This PR proposes to relocate data for dashr-studybrowser, and to remove some commented out lines. Eventually all data in this repo will be 🚚 to the datasets repo.

@rpkyle i am guessing my review for this app is obsolete ?

As per https://github.com/plotly/streambed/issues/13854, I am closing this as the new deployment process is in place.
If you need any changes from this branch, please recreate your changes against master and create a new pull request. Let me know if you need help and I am happy to jump in.


Added secure score operations

Waiting for input on ExternalSecuritySolutionsTests and AdaptiveApplicationControls_Put

@bquantump , I see you've been assigned to review this PR.
Please review since it is prerequisite for PS/CLI enhancements for secure score.
Thanks

@bquantump , I see you've been assigned to review this PR.
Please review since it is prerequisite for PS/CLI enhancements for secure score.
Thanks

Please ensure all the following are done (for example no swagger is linked here):
https://github.com/Azure/adx-documentation-pr/blob/master/engineering/adx_netsdk_process.md#appendix-a-pr-review-checklist

@giromm2ms are you planning updating this soon? If we do not hear anything by the end of the week, we will be closing this.

@bquantump , I intend to work on it tomorrow.
Please don't close it yet

As this one contains lots of merge content, I am closing it.
Please refer to https://github.com/Azure/azure-sdk-for-net/pull/16985


Fixes

Fix json formatting in optionals.json
Moved "Dislike Bad Dual Audio Groups" to ignored.
Fixed RlsGroup -C.A.A regex. (thanks @fabricionaweb @mvanbaak)

Closes #691

About the EOL this may helps
https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings


Use .venv for catalog virtualenv instead of venv

Fixes

Fixes #4109  by @dhruvkb
Description

Changed openverse/catalog/justfile so that the catalog automatically creates virtualenv at /.venv instead of /venv. Also updated documentation/catalog/guides/quickstart.md so that it includes the new instructions to create virtualenv (utilizing just) and deleted the old ones.
Testing Instructions

First run rm -R catalog/venv catalog/.venv to get rid of any existing environments to start from scratch. Then run just catalog/venv to create the virtualenv. Utilizing ls -a ensure that catalog/.venv exists and not catalog/venv.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[ ] I ran the DAG documentation generator (just catalog/generate-docs for catalog
PRs) or the media properties generator (just catalog/generate-docs media-props
for the catalog or just api/generate-docs for the API) where applicable.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Ahh actually, I think we'll need one more change - we need to add the .venv folder to the .gitignore, it can probably be added here:
https://github.com/WordPress/openverse/blob/3d6e8b5ab96a31ba6da84feb292790f97b39f4a8/.gitignore#L19-L18

I did as you instructed and did git pull origin main on my local catalog_virtualenv branch after resetting to remove the .gitignore change, but my local version of gitignore does not include .venv.


Services created using the screen command cannot request

hello everybody
I have a problem.When i use the screen command  to create a backend terminal.I can't request this port on another machine.I'm sure this port is open.And the system and OPENNMT are normal.Could you please explain this to me, thank you very much.

Hello,
I'm not sure what is the issue you are describing. Is it an OpenNMT issue?
If yes, please include the commands and options you used.
If not, you can still post on the forum, maybe some people are using the same tools as you.

hi， @guillaumekln
this command: th tools/rest_translation_server.lua -joiner_annotate -port 4069 -model

Try setting -host <IP_ADDRESS> on your command line.

yeah, it is ok.  thanks @guillaumekln


remove mima checks

Relates to https://github.com/apache/incubator-pekko/issues/15

accidentally created the branch on the apache repo - I will restart this PR using a branch on my fork


[PowerToys Run] Add the ability to set shortcuts for launch PowerToys with specific Action key

It would be much more useful and conveniently to have the ability to add custom shortcuts for launch an app with some prefilled action key. In this case you don't need to press additional keys to get what you need.
Expected
PowerToys should have a setting for defining nultiple keyboard shortcuts for run an app with filled action key.
Best spot for this is place where you can define shortcut for launch the PowerToys.
Example:

Shortcut
Action Key

CTRL+SPACE
default run without action key

ALT+SPACE
<

CTRL+ALT+SPACE
>

CTRL+SHIFT+SPACE
?

...
...

this is being tracked in the plugin setting part in #5273


feat: :fire: TorBox

[x] Add the torbox sdk #48
[x] Add torbox to the user config
[x] Test stream and cache.
[x] Private tracker logic to review fo DLL process
[x] Check the TorBox search enpoint capability
[x] Check the Usenet capability
[ ] Update the docs

Notes 👍
TorBox are not yet ready for usenet integration, since cache-key mess-up on torbox side and cannot, for the moment, be calculated from indexer.
Search endpoint are too slow and have only public torrent that can be support much faster with bot magnet integration.

This PR should improve the proxied stream and sort of fix #57.
Since a few changes have been made to the streaming process, it should be tested again.


snoflake_table resource is not renaming the column

Provider Version:0.43.0
The provider version you are using.
Describe the bug
When we rename the column in snowflake_table resource file, its not renaming the column in table (i.e. alter table rename column...)
but its dropping existing column and creating new column with new name. Existing data of old column is lost
Expected behavior
A clear and concise description of what you expected to happen.
it should run alter table rename column. Data should not be lost from the column.
resource "snowflake_table" "tf_test" {
database = "TF_DEMO_DB"
schema   = "TF_DEMO"
name     = "TF_TEST"
column {
name     = "id"
type     = "NUMBER(38,0)"
nullable = true
comment  = "id column comment"
}
column {
name     = "newdata"
type     = "VARCHAR(16777216)"
nullable = false
comment  = "data column comment"
}
column {
name     = "create_ts"
type     = "timestamp"
nullable = false
comment  = "ts column comment"
}
comment = "table comment"
}
Snowflake DDL:
ALTER TABLE "TF_DEMO_DB"."TF_DEMO"."TF_TEST" DROP COLUMN "data"
ALTER TABLE "TF_DEMO_DB"."TF_DEMO"."TF_TEST" ADD COLUMN "newdata" VARCHAR(16777216) NOT NULL COMMENT 'data column comment'
This use case failed, as there are rows in the table. The first alter table dropped the column, the second alter failed as it could not create column as not null with rows

Duplicates of https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/753


Grammar Only Correction

grammar changed from "Let's now revisit the our chat sample from the" to "Let's now revisit our chat sample from the"
Motivation and Context

Why is this change required?

This change is required to fix the grammatical error in the sentence.

What problem does it solve?

This change solves the problem of incorrect grammar in the sentence.

What scenario does it contribute to?

This change contributes to the scenario where we want to have correct grammar in our chat sample.

If it fixes an open issue, please link to the issue here.

Since this change does not fix an open issue, there is no link to provide.
Description

I have made a change to the grammar in the sentence by removing the word ‘the’ which was causing the sentence to be grammatically incorrect. The overall approach I took was to identify the error in the sentence and then make the necessary changes to fix it. The underlying design of this change is to ensure that our chat sample has correct grammar and is easy to understand for our users.
Contribution Checklist

[X ] The code builds clean without any errors or warnings
[X ] The PR follows SK Contribution Guidelines (https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
[ X] The code follows the .NET coding conventions (https://learn.microsoft.com/dotnet/csharp/fundamentals/coding-style/coding-conventions) verified with dotnet format
[ X] All unit tests pass, and I have added new tests where possible
[X ] I didn't break anyone :smile:

You win the best PR description award!
Next time, check out PRSkill: A skill that can generate feedback, commit messages, and pull request descriptions based on git diff or git show output. The PRSkill uses the CondenseSkill as a dependency and implements chunking and aggregation mechanisms to handle large inputs.
https://github.com/lemillermicrosoft/skonsole
You'll never have to write another PR description again!
CC @lemillermicrosoft
Thanks for the change!

Thank you! I will need to check that out. It was fun to write though!


Refreshing request

Hi Gwendal,
One day, one question 😇
Since you wanted concrete cases of use... 😅
I'm still evaluating SwiftUI / GRDBCombine, and i have 2 questions (in fact, one question, it's the same topic)

I have a list of TV channels with EPG datas displayed...
In the iOS version, i'm refreshing the list every minute because the EPG data (time / program) can change every minutes.
Is there any way to refresh the query / request for the same View / @ObservedObject ? And then manually send a refreshed request / datas to the view via something like objectWillChange.send() ?

Second question (the same)...
User can change the order of the channel list, or filtering the list via many options...
So it's the same question, rebuilding the query with new .filter or other things like .order

I've tried several methods but none of them work, I'm blocked because when I do a new query and plug to DatabasePublished(publisher), it loses (i think) the link with the @ObservedObject variable of the view, which is normal.
I haven't found a way to redo a query for the same @ObservedObject variable.
On the other hand, the associations works, and that's great !!! 👍🏻
Thanks

Hello @skrew!
You're entering uncharted territory, I admit. I lack experience about SwiftUI so I can't really answer your first question.
I may help for your second question, though. It its current implementation, @DatabasePublished tracks a single request, which can not change. If the request changes over time, you may have to put DatabasePublished aside, and instead create a publisher of requests (driven by your UI), and convert it to a publisher of database values with switchToLatest. Gross untested sample code:
let request = ... // Publisher of QueryInterfaceRequest<Channel>
let channels = request.switchToLatest {
    $0.observationForAll().publisher(in: dbQueue)
}
channels.sink(
    receiveCompletion: { _ in },
    receiveValue: { channel in print("fresh channels: \(channels)") })

This is basically the technique used for the "toggle ordering" button in the RxGRDB demo app.
Maybe @DatabasePublished could be made able to track a publisher of requests instead of a single request. Your own experience, suggested changes, maybe motivated pull requests will be welcomed!

No news is good news: I assume the previous answer was able to unblock you.


[DataModel] Define Data Model through classes in datamodel.js

In order to clarify what the Data Model is, this commit:

Extracts existing BmcDataAPI and KeyScheme classes from storage.js into datamodel.js
Add definition of what a "Comic" is as BmcComic class
Add definition of what a "Source" is as BmcComicSource class
Add required utilities in KeyScheme class
Rework BmcDataAPI to fit the clarified model
Update dependant code to provide the missing parameters

Looks good to me so unless you want to do something else in here, you can merge. :)

nope, I'm good on this side :)


Error in data in ProjectionGeoKey.js

My geotiff file has the following (somewhat mysterious) geoKeys:
GTCitationGeoKey: "UTM Zone 32, Northern Hemisphere"
GTModelTypeGeoKey: 1
GTRasterTypeGeoKey: 1
GeogAngularUnitsGeoKey: 9102
GeogCitationGeoKey: "GCS Name = GRS 1980(IUGG, 1980)|Datum = unknown|Ellipsoid = GRS80|Primem = Greenwich|"
GeogEllipsoidGeoKey: 32767
GeogGeodeticDatumGeoKey: 32767
GeogInvFlatteningGeoKey:<PHONE_NUMBER>01
GeogPrimeMeridianLongGeoKey: 0
GeogSemiMajorAxisGeoKey: 6378137
GeogTOWGS84GeoKey: [ 0, 0, 0 ]
GeographicTypeGeoKey: 32767
ProjLinearUnitsGeoKey: 9001
ProjectedCSTypeGeoKey: 32767
ProjectionGeoKey: 16032

which convertGeoKeys2proj4 converts to this string:
+proj=tmerc +lat_0=0 +lon_0=0.1570796326794895 +k_0=0.9996 +x_0=500000 +y_0=0 +a=6378137 +b=6356752.314140356 +towgs84=0,0,0, +no_defs

This is not quite right - the lon_0 value should be 9, as listgeo lists the tiff file as follows:
eotiff_Information:
   Version: 1
   Key_Revision: 1.0
   Tagged_Information:
      ModelTiepointTag (2,3):
         0                 0                 0
<PHONE_NUMBER>09888  6301417.47768047  0
      ModelPixelScaleTag (1,3):
         0.356706451650175 0.356706451650175 0
      End_Of_Tags.
   Keyed_Information:
      GTModelTypeGeoKey (Short,1): ModelTypeProjected
      GTRasterTypeGeoKey (Short,1): RasterPixelIsArea
      GTCitationGeoKey (Ascii,33): "UTM Zone 32, Northern Hemisphere"
      GeographicTypeGeoKey (Short,1): User-Defined
      GeogCitationGeoKey (Ascii,86): "GCS Name = GRS 1980(IUGG, 1980)|Datum = unknown|Ellipsoid = GRS80|Primem = Greenwich|"
      GeogGeodeticDatumGeoKey (Short,1): User-Defined
      GeogAngularUnitsGeoKey (Short,1): Angular_Degree
      GeogEllipsoidGeoKey (Short,1): User-Defined
      GeogSemiMajorAxisGeoKey (Double,1): 6378137
      GeogInvFlatteningGeoKey (Double,1):<PHONE_NUMBER>01
      GeogPrimeMeridianLongGeoKey (Double,1): 0
      GeogTOWGS84GeoKey (Double,3): 0                0                0
      ProjectedCSTypeGeoKey (Short,1): User-Defined
      ProjectionGeoKey (Short,1): Proj_UTM_zone_32N
      ProjLinearUnitsGeoKey (Short,1): Linear_Meter
      End_Of_Keys.
   End_Of_Geotiff.

Projection = 16032 (UTM zone 32N)
Projection Method: CT_TransverseMercator
   ProjNatOriginLatGeoKey: 0.000000 (  0d 0' 0.00"N)
   ProjNatOriginLongGeoKey: 9.000000 (  9d 0' 0.00"E)
   ProjScaleAtNatOriginGeoKey: 0.999600
   ProjFalseEastingGeoKey:<PHONE_NUMBER>00 m
   ProjFalseNorthingGeoKey: 0.000000 m
TOWGS84: 0,0,0
Projection Linear Units: 9001/metre (1.000000m)

The problem seems to be in data preparation, because the string for 16032 in ProjectionGeoKey.js has the wrong value.  On http://epsg.io/16032.gml, the latitude is 9.   The problem seems simply a mixup between degrees and radians:  0.1570796326794895 is 9 degrees in radians.

Hello again, thanks for the report, I really appreciate the effort!
I've got a lot going on lately, so it'll take some time for me to look into this issue, sorry :)

Looks like it's fixed. It passes my tests with geokeys from this issue, but I'm a bit unsure if I've covered everything while refactoring.
If you have a larger dataset, may I ask you to extract geokeys and whatever Proj4 strings GDAL outputs, and send it to me?

The library now returns this proj4 string for my geokeys:
+proj=tmerc +lat_0=0 +lon_0=8.999999999999991 +k_0=0.9996 +x_0=500000 +y_0=0 +a=6378137 +b=6356752.314140356 +towgs84=0,0,0, +no_defs

It turns out that openlayers cannot correctly parse this.   The problem is the +towgs84=0,0,0, part.   If I change it to +towgs84=0,0,0 (that is, remove the final comma), then it works correctly.
Also, somehow when preparing the data you are changing the 9 degrees in the specification into 8.999999999999991 degrees.  It would be nicer if you could round to 12 digits or something like that, so that the degrees would exactly match the specification.

Thanks, I can confirm that now it works!

Thanks, been a pleasure working with you!


Picture orientation is systematically changed to portrait after cropping on iOs devices only

After cropping a picture in landscape, the orientation is systematically changed to portrait.
This problem is only on iOs (AMHA is issue resolved in #14 for Android devices).

Any updates on this? I can confirm the issue. @VolodymyrLykhonis are you looking into it? Otherwise I could also try to debug this.

@wwwdata I have not had chance look at this yet. You can definitely give it a shot

I had this problem too.

I'm also having this problem.

Added PR with fix
https://github.com/VolodymyrLykhonis/image_crop/pull/33

Added PR with fix
#33

Thanks <3

Closing as per https://github.com/VolodymyrLykhonis/image_crop/commit/f4562e1b8e17ed53ee2b641e7b33f8c739bb361e

In my case if i try cropping a picture from gallery then it turns to landscape for some pictures and the crop area that i chose is not being cropped but showing some other region of the picture


Modeling relation that doesn't follow naming strategy

My application follow the EntityCaseNamingStrategy and I don't wish to change that.
There are two tables that I didn't design and I would like to accommodate.
Basically Account has many AccountSegments. Here is the abridged DDL
CREATE TABLE "Account" (
    "accountNumber" text PRIMARY KEY,
    ...
);

CREATE TABLE "AccountSegment" (
    "segmentId" text PRIMARY KEY,
    "accountNumber" text,
    ...
);

What options do I have to pass to @ManyToOne and @OneToMany to accommodate this? I've been trying with no luck along the lines of:

  @ManyToOne(() => Account, { inversedBy: 'segments', })
  account?: IdentifiedReference<Account>;

  ...

  @OneToMany(() => AccountSegment, segment => segment.account, { referenceColumnName: 'accountNumber', })
  segments = new Collection<AccountSegment>(this);

Can you please retest with v3.4? Could you ideally prepare a reproduction repo?

Still doesn't work. Minimal reproduction
PS: congrats on all of the amazing new features you've added since I last checked

Ok this helps a lot, thanks for a real MVCE! So the real issue is that you were trying to map single column (AccountSegment.accountNumber) to multiple properties (AccountSegment.account and BaseSegment.accountNumber).
This is currently not supported, each column is mapped to a single property only. So what happens here is that your db column accountNumber is mapped to account defined at the AccountSegment level, but not to the BaseSegment.accountNumber, which is a PK.
Honestly mapping a column to multiple properties feels like a bad design to me, there was already another issue mentioning this but there was a quite easy workaround as it was not about primary keys so one could use a getter. The reason why it feels bad is that the reverse mapping, you would have multiple properties that could potentially have conflicting options. But with the PK use case you are having it sounds more reasonable...
A workaround for this example is to not use the BaseSegment entity that is causing the problem here. Something like this should work (not 100% sure if it works with 3.4.1, but it will work once #395 is merged).
@Entity()
export class AccountSegment {

  constructor(args?: Partial<BaseSegment>) {
    Object.assign(this, args);
  }

  @ManyToOne(() => Account, { fieldName: 'accountNumber', wrappedReference: true, primary: true })
  account?: IdentifiedReference<Account>;

  @Property({ persist: false })
  get accountNumber(): number | undefined {
    console.log(this.account);
    return this.account?.unwrap().accountNumber;
  }

  @Property()
  yearBuilt: number;

}

Note that I also added wrappedReference: true, that is needed if you use ReflectMetadataProvider so the ORM will hydrate a reference rather than an entity directly.

Thank you for taking another look.
This doesn't work as is, but I will check again after #395 is merged.

v3.5.0 is out. I was thinking about this again, and I believe you should define both account and accountSegment in the BaseSegment entity (in the way I described above).

I brought the definitions to the subclass level and it's working. Thanks for everything.


Can Pholcus Set Specific Unique Index?

How to set custom unique index at specific fields, for example "url" at mgo output? please tell me how.

You can modify it in database


Excalidraw error on deploy

When I try to deploy an Excalidraw stack from App Templates, I get this error:
failed to deploy a stack: listing workers for Build: failed to list workers: Unavailable: connection error: desc = "error reading server preface: http2: frame too large"

See: https://github.com/Lissy93/portainer-templates?tab=readme-ov-file#raising-issues
Issues relating to specific images need to be raised upstream.
This repo just collates templates listed in the Sources into a single manifest, so once a fix has been pushed out upstream, it will be available here.


Attempt to run on Amazon Sagemaker results in init error for PudlTabl

Describe the bug
I've created a Python 3.7 instance of SageMaker and done the necessary conda installs to run PUDL.  I've created a .yaml file and copied other patterns of success to get my PUDL notebook running.  However, it fails here:

TypeError                                 Traceback (most recent call last)
 in
9
10 #pudl_engine.table_names()
---> 11 pudl_out = pudl.output.pudltabl.PudlTabl(pudl_engine, freq='AS') #annual frequency
TypeError: init() got multiple values for argument 'freq'
Bug Severity
How badly is this bug affecting you?

High: This bug is preventing me from using PUDL.

To Reproduce
Start with a conda_mxnet_latest_p37 kernel and uncomment these conda installs:
%conda install geopandas
%conda install -c conda-forge catalystcoop.pudl
%conda install contextily
Then try to run the PUDL sample notebooks to initialize PUDL.
Expected behavior
I expect it to work, like it does in my desktop docker world and on 2c2i
Software Environment?
Amazon Sagemaker, installed via conda (above)

How did you install PUDL? (e.g. git clone, pip, or conda)
When I install everything via pip, I have different problems: the eia_860_generators function is out of step with my pudl.sqlite dictionary, referencing balancing_authority_code instead of balancing_authority_code_eia.

Additional context
A surprising amount of stuff works even with that failure, but critical stuff doesn't work, so I'd like to fix it.

The PyPI / conda-forge version (0.3.2) of PUDL is quite out of date at this point. If you want current dependencies and data, the only option right now is to install from the repo, or pull down our draft Docker image + archived data as explained in the PUDL Examples repo. This is not ideal. We need to get the data build automated so there's always a release candidate ready to go. The current version of PUDL requires Python 3.8 or 3.9. There are a couple of minor data validation errors that still need to be tracked down. The docs are done. And then I think I just need to turn the crank on running the ETL pipeline and publishing the packages. And yet it still seems hard to get it over the finish line.

Hey there, do you want to give catalystcoop.pudl=0.4.0 which has more modern requirements and should work with the current data... let us know if it works!


Use couch_file:delete/3 in views cleanup

Use couch_file:delete/3 directly instead of one-op front function.
All the tests moved in couch_file_tests.
Depends on apache/couchdb-couch#161

@iilyak Can you take a quick look? It's a pretty basic stuff, just switching on now merged couch_file:delete/3 and removing the tests, they are now in couch_file_tests.

+1


FRs: Commit lint and changelog generation

This issue is an extension of #6298 (version bumping) for related enhancements. Combining into one issue so as not to inundate the team (but can split into multiple issues if that is preferred).
Standards-compliant project management actions like auto-version bumping (SemVer), changelog generation (Keep a Changelog) and commit message linting (Conventional Commits) might be good tasks for the project management tool to tackle and promote best practices for development. It would consolidate the tooling even further, save time for the developer, and reduce the cognitive load. Defining the rules and formats in pyproject.toml could provide consistency and configurability for any codebase.
Example
pyproject.toml
[tool.uv.bump]
version-files = ["src/project/__init__.py:__version__"]
version-format = "semver2"

[tool.uv.commit]
lint-commits = true
commit-format = "conventional-commits"

[tool.uv.changelog]
generate-on-bump = true
include-date = true
date-format = "iso8601"

[tool.uv.changelog-map]
feat = "⭐ Features"
fix = "🐛 Bug Fixes"
chore = "🧹 Chores"
ci = "🚀 Continuous Integration"
docs = "📚 Documentation"
perf = "🔥 Performance Improvements"
refactor = "🔨 Refactoring"
revert = "🔄 Reverts"
style = "🎨 Styling"
test = "🧪 Tests"

Command line usage
Commiting via uv will lint the commit message for applicable conventional commit tags and reject if there isn’t one. This can just shell out to git detected on the path.
uv commit “fix: Always invoke found interpreter when `uv run python` is used (#6363)”

or
uv commit fix “Always invoke …”

Bumping will bump versions for all listed files or file sections (e.g. __version__.py or __init__.py:__version__). Not sure if there’s a consensus on the best place to put the version so version-files provides flexibility.
uv bump

Changelog generation can also be its own command if generate-on-bump is not desired.
uv changelog

Generated changelog
CHANGELOG.md
# Changelog

## 0.3.1 (2024-08-21)

### 🐛 Bug Fixes

- Always invoke found interpreter when `uv run python` is used ([#6363](https://github.com/astral-sh/uv/pull/6363))

### 📚 Documentation

- Remove the preview default value of `python-preference` ([#6301](https://github.com/astral-sh/uv/pull/6301))
- Update env vars doc about `XDG_*` variables on macOS ([#6337](https://github.com/astral-sh/uv/pull/6337))

See also: Commitizen, a tool that provides these capabilities.

This would be cool, of course. We use rooster right now, but it's not polished at all.
I think we need to focus on a lot of other things before this though.

By the way, there is git-cliff, and it works for any projects, not just python ones.

https://github.com/astral-sh/uv/issues/6298#issuecomment-2350938745

Using uvx hatch version ... is mostly fine, but I can't use it when it's a stub only package because there is no *.py.

I need a way to work around this case.
It would be great if this was considered.
As someone else pointed out, it can be solved by doing the following, but it's ugly.
https://github.com/astral-sh/uv/issues/6298#issuecomment-2411738585

As a workaround, I use this in my project:
sed -i -e "s/0.0.0/${GITHUB_REF#refs/*/}/" pyproject.toml


[auto-bump] dependency by andrew-wm-arthur

:coffee: An Automated Dependency Version Bump PR :crown:
Initial Changes
The initial changes contained in this PR were produced by go geting the dependency.
$ cd ./go
$ go get github.com/dolthub/<dependency>/go@<commit>

Before Merging
This PR must have passing CI and a review before merging.
After Merging
An automatic PR will be opened against the LD repo that bumps the bounties version there.

This PR has been superseded by https://github.com/dolthub/bounties/pull/74

This PR has been superseded by https://github.com/dolthub/bounties/pull/75


Support sending keep-alive pings

Describe the bug
We have a customer reporting a 16 minutes delay for the service to detect Disconnect when iOS client loses TCP connection https://github.com/Azure/azure-signalr/issues/1137
Expected behavior
The issue can be resolved if Swift client sends out ping messages every 15 seconds, as similar to C# or JS client, so that the service could detect the Disconnect after 30 seconds when Ping timeout.

JS client logic for sending pings
https://github.com/dotnet/aspnetcore/blob/master/src/SignalR/clients/ts/signalr/src/HubConnection.ts#L607-L617

Correct, this is currently not implemented.

Thanks @moozzyk for the quick response, any plan to support it?

Yes, I am planning to add support for this in a future release.

Nice! Any ETA for the future release?

I am working on this in my free time so I am unable to promise anything.

try the best, we all love u

try the best, we all love u

This should be now fixed and will be included in the next release.


Add a way to end a blind mode game and reveal all items on the card

This could be useful for players who don't want to play a blind mode game until all items are collected, and are interested in seeing the remaining items on the card.

Duplicate of #182 (closed as it's implemented for next version)


【AT】 rollback transaction of phase two

What would you like to be added:
Why is this needed:

assign to me


Segmentation fault while trying to install pylxd with pip3 install pyld (on a Raspberry Pi 4B)

Hello.
When i am trying pip3 install pylxd to my raspberry pi 4 B with ubuntu 18.04 it happens this:
Collecting pylxd
Collecting six>=1.9.0 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/ee/ff/48bde5c0f013094d729fe4b0316ba2a24774b3ff1c52d924a8a4cb04078a/six-1.15.0-py2.py3-none-any.whl
Collecting ws4py!=0.3.5,>=0.3.4 (from pylxd)
Collecting pbr>=1.6 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/96/ba/aa953a11ec014b23df057ecdbc922fdb40ca8463466b1193f3367d2711a6/pbr-5.4.5-py2.py3-none-any.whl
Collecting requests-toolbelt>=0.8.0 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/60/ef/7681134338fc097acef8d9b2f8abe0458e4d87559c689a8c306d0957ece5/requests_toolbelt-0.9.1-py2.py3-none-any.whl
Collecting requests-unixsocket>=0.1.5 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/d0/63/97662a6f7175c08381447a09f6bc35464075f0ea6549cf6daf2668b51f04/requests_unixsocket-0.2.0-py2.py3-none-any.whl
Collecting requests>=2.20.0 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/45/1e/0c169c6a5381e241ba7404532c16a21d86ab872c9bed8bdcd4c423954103/requests-2.24.0-py2.py3-none-any.whl
Collecting cryptography!=1.3.0,>=1.0 (from pylxd)
Collecting python-dateutil>=2.4.2 (from pylxd)
  Using cached https://files.pythonhosted.org/packages/d4/70/d60450c3dd48ef87586924207ae8907090de0b306af2bce5d134d78615cb/python_dateutil-2.8.1-py2.py3-none-any.whl
Collecting urllib3>=1.8 (from requests-unixsocket>=0.1.5->pylxd)
  Using cached https://files.pythonhosted.org/packages/e1/e5/df302e8017440f111c11cc41a6b432838672f5a70aa29227bf58149dc72f/urllib3-1.25.9-py2.py3-none-any.whl
Collecting idna<3,>=2.5 (from requests>=2.20.0->pylxd)
  Using cached https://files.pythonhosted.org/packages/a2/38/928ddce2273eaa564f6f50de919327bf3a00f091b5baba8dfa9460f3a8a8/idna-2.10-py2.py3-none-any.whl
Collecting chardet<4,>=3.0.2 (from requests>=2.20.0->pylxd)
  Using cached https://files.pythonhosted.org/packages/bc/a9/01ffebfb562e4274b6487b4bb1ddec7ca55ec7510b22e4c51f14098443b8/chardet-3.0.4-py2.py3-none-any.whl
Collecting certifi>=2017.4.17 (from requests>=2.20.0->pylxd)
  Using cached https://files.pythonhosted.org/packages/5e/c4/6c4fe722df5343c33226f0b4e0bb042e4dc13483228b4718baf286f86d87/certifi-2020.6.20-py2.py3-none-any.whl
Collecting cffi!=1.11.3,>=1.8 (from cryptography!=1.3.0,>=1.0->pylxd)
Collecting pycparser (from cffi!=1.11.3,>=1.8->cryptography!=1.3.0,>=1.0->pylxd)
  Using cached https://files.pythonhosted.org/packages/ae/e7/d9c3a176ca4b02024debf82342dab36efadfc5776f9c8db077e8f6e71821/pycparser-2.20-py2.py3-none-any.whl
Installing collected packages: six, ws4py, pbr, idna, chardet, certifi, urllib3, requests, requests-toolbelt, requests-unixsocket, pycparser, cffi, cryptography, python-dateutil, pylxd
Successfully installed certifi-2020.6.20 cffi-1.14.0 chardet-3.0.4 cryptography-2.9.2 idna-2.10 pbr-5.4.5 pycparser-2.20 pylxd-2.2.11 python-dateutil-2.8.1 requests-2.24.0 requests-toolbelt-0.9.1 requests-unixsocket-0.2.0 six-1.15.0 urllib3-1.25.9 ws4py-0.5.1
Segmentation fault (core dumped)

When i am doing at my code from pylxd import Client
lxdmanager = Client()
everything works ok.
When i am doing retval = lxdmanager.containers.exists("CameraSvc")
, it seems not working. CameraSvc container exists but if a print retval it gives me False.

The key part is this:
Segmentation fault (core dumped)

i.e. it didn't install properly.  That's pip3 giving up.  Core dumps occur either with a memory violation, an illegal instruction, or a security violation.  Unfortunately, there's not enough info in the bug report to determine what is going on?
To help you:

How much memory does the Pi have?
Is there a coredump file you can analyse?

ubuntu@ubuntu:~/Desktop$ free -m               total        used        free      shared  buff/cache   available Mem:           1848         358         255           7        1233        1459 Swap:             0           0           0
At the beginning libffie and libssl was missing and i installed them with sudo apt-get.
Is there any info which i can provide ?

It's a pip3 / python issue on the raspberry pi; you need to search in that space.  It could be binary issues with the ARM32, etc.

Any other way of installing pylxd ?

It's (almost certainly) not pylxd.  You need to resolve why pip3 is producing a segfault.  For example: use gdb to look at the coredump to see where it is going wrong, is one possibility.  Try installing each of the dependencies listed individually (i.e. pip3 install pycparser==2.20) starting with the last one) and see if that's the problem.  It may be that there's not sufficient memory to run pip3 and install all the modules.  Do you have a swap file set up?

I installed Ubuntu OS to a bigger SD card and pylxd installed perfectly and runned also perfectly at the rpi.


Migrate dates to UTC

Migration script to migrate dates to UTC, plus app changes to handle the new dates properly and remove the moment.js dependency.
Many thanks to @colingourlay who did all the work here!

Great to see this land. Thanks for taking care of business, @jasonrhodes!


Zero touch (preview) does not deploy the MDE app

[Enter feedback here]
Docs say "Defender for Endpoint is automatically installed without the need for the user to open the app" which leads to the interpretation, that an explicit app deployment profile is not necessary. But an MDE app deployment profile is necessary. The following  activation after installation is silent as documented.

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

ID: f9d7718e-f707-9e4d-8583-4a6385b9b40c
Version Independent ID: e5abdc8e-1b61-ca7c-d73c-b4d9d3a6de48
Content: App-based deployment for Microsoft Defender for Endpoint on iOS
Content Source: microsoft-365/security/defender-endpoint/ios-install.md
Product: m365-security
Technology: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Agree with WolfgangSauer.  Whole page is confusing to me.

@WolfgangSauer Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/7391
Thanks
Sri

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


BFI mist missing information

https://whatson.bfi.org.uk/Online/default.asp?default.asp?doWork::WScontent::loadArticle=Load&BOparam::WScontent::loadArticle::article_id=B9EE0D56-2F8A-4452-9DD9-74FEDC39A279
Entry has director, actors, year and duration -- but that's not present in the JSON

Same for Quo Vadis? - https://whatson.bfi.org.uk/Online/default.asp?default.asp?doWork::WScontent::loadArticle=Load&BOparam::WScontent::loadArticle::article_id=62421D77-62A5-4A0B-8BAE-EABCC2A798DD

Not sure why mist didn't, but Quo Vadis? had suffixed text on that line that the regex needed to account for


Add tests and implementation to fix persistentvolume support in jobmanager

Funciton stubs for volume discovery
Implement findvolumes
Pass in pvs
Remainder of create job with container
Fix CreatePvCMountReference
Update readme
Fix linting
Update readme
Update readme

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

27 of 60 (45.0%)  changed or added relevant lines in 2 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.5%) to 54.88%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/localrepomanager/localrepomanager.go
0
2
0.0%

pkg/kubelayer/kubelayer.go
27
58
46.55%

Files with Coverage Reduction
New Missed Lines
%

pkg/kubelayer/kubelayer.go
1
52.14%

pkg/webhooklistener/webhookreceiver.go
5
82.69%

Totals

Change from base Build<PHONE_NUMBER>:
-1.5%

Covered Lines:
343

Relevant Lines:
625

💛  - Coveralls


Suggestion: support view images inside folder

support view images inside folder
It is very common to view images inside a folder

@rsun-thoughtworks
Thank you for your suggestion.
Do you mean to show thumbnails of the images in the folder?
I have tried to implement this in the past, but abandoned it because the performance degraded to an extent that could not be overlooked.
I'm sorry, but I have no intention of implementing this feature in the future.

Grid view functionality is implemented in v2.2.0 now.
Try it out, if you like it.


Breadcrumbs for database operations

:scroll: Description

Create breadcrumbs for database operations where also spans are created.

Set the following data:

type

query for database queries
error for errors
nothing in other cases

category

same as span operation

data

status

either ok or error

database system
database name

message

sqlite queries/commands from internal buffer (e.g: 'UPDATE Product SET title = ?')

Also bumped the iOS simulator to 17, because the 16.4 is invalid in the most recent macOS-13 runner update.

:bulb: Motivation and Context

Closes #1341
:green_heart: How did you test it?
Added unit tests
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[ ] No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
[ ] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps
@buenaflor @krystofwoldrich Please check if the breadcrumb data makes sense.

lgtm although we first need clarification on Stefano's error comments. To me it seems like error looks fine as type when catching the exception

the breadcrumb is still about the query, not the exception, so the type shouldn't change

@stefanosiano Updated everything as suggested. Just a question if type query should be set for write operations like commit & apply

@stefanosiano WDYT, can we merge or is there something left here?

@denrase There are just a couple of type: 'error' left that i'd remove
after that we can merge

@stefanosiano Sry about that, they are removed now.


Custom rules should be above Routing rules

Hello
Just found an issue in the default .htaccess implementation:
Using the Custom rules to force https and www do not function if placed below the Routing rules. I'm barely know .htaccess, but It seems that the [L] tag prevents those rules from firing. So routing via index.php prevents the Custom rules from working.
I resolved this by moving those rules above the Routing section.

Excellent catch once again – thank you and sorry for the delay!
So far, the custom rules (including the www and https redirects) have only been invoked when the path component of the URL had a length of zero or one, i.e. if the path was either completely empty or if it consisted of a single slash only. Whenever you requested a non-empty path (e.g. /css/styles.css), the routing would somehow prevent the custom rules from working, as you said.
The fix is more or less what you did, perhaps the inverse. The directives for the routing part have just been moved to the very bottom: https://github.com/delight-im/PHP-Foundation/commit/c8aece33fa480c3c847d782e5a247cbec6ffef10 This should be equivalent to your solution, so just keep what you prefer. No regressions were found in any tests so far with this fix. I hope that remains true.

Thanks!
Moving the router to the end makes more sense. Now I can move my custom Bad Robot User Agent blockers to the custom section.


This statement very very hard to understand / explain

[Immetti qui i commenti]
"La parola "data di lavoro", nella lingua impostata in Lingua imposterà la data sulla data di lavoro attualmente impostata e specificata nella pagina Impostazioni personali."
The word for 'workdate', in the language set by Language setting, will set the date to the currently set work date that is specified on the My Settings page. Instead of entering the entire word, you can enter part of the word, starting from the beginning, such as 'w' or 'work'.

Dettagli del documento
⚠ Non modificare questa sezione. È necessaria per i collegamenti relativi ai problemi tra docs.microsoft.com ➟ GitHub.

ID: e75fc0c2-d5f5-4652-56de-44fbdc9d8343
Version Independent ID: ad0e08e1-cc52-f2e6-1ce9-cbb1c3efdc7b
Content: Immissione di date e ore in Business Central - Business Central
Content Source: business-central/ui-enter-date-ranges.md
Service: dynamics365-business-central
GitHub Login: @SorenGP
Microsoft Alias: edupont

Hi,
Thanks for reporting this issue. We have reworked the description to make it more clear, and added a little more information about what a work date is and an example of when they're useful.
Though the content will be published soon in English, there will be some delay before the translated version becomes available.


Does Llama Parse skip reprocessing already parsed documents? T

Does Llama Parse detect documents or parts of documents that are already parsed and does not reparse them? If not, how to implement this on user's end? This would prevent unnecessary reprocessing and would save costs on enterprise end.

I am also interested to know this. I have tried to make use of parsing instruction to "suppress" repeated data being shown in the output but it does not seem to be useful.
Suppose I have a document that has its header/summary section repeating in every page of the document, its going to be costly to embed them for large documents in RAG applications.
Using the below sample document as an example. Given prompts that tells the parser to not output information that are repeated between pages (e.g. the document date and number, etc.). It is just not effective. The parser sometimes is also not perfect at understanding the relationship between the parties in the document (despite having prompts that explains how to identify them).
MultiPageInvoice.pdf
This is the markdown result (not using gpt-4o):
# DOCUMENT TYPE
- Invoice

# SUPPLIER DETAILS
- NAME: Abstractors and Design Co.
- ADDRESS: Suite 8, 611 Maine St, San Francisco CA 94105
- TAX IDENTIFICATION NUMBER: N/A

# CUSTOMER DETAILS
- NAME: Ronald Davis
- ADDRESS: N/A
- TAX IDENTIFICATION NUMBER: N/A

# DELIVERY/SHIPPING LOCATION
- NAME: N/A
- ADDRESS: N/A

# BILLING ADDRESS
- Full Billing Address: Suite 8, 611 Maine St, San Francisco CA 94105

# DOCUMENT DETAILS
- DOCUMENT PROCESSING COUNTRY: N/A
- DOCUMENT DATE: 12 March 2020
- DOCUMENT NUMBER: INVOICE 00000135
- REFERENCE DOCUMENT NUMBER: F0016

# CURRENCY
- CURRENCY CODE: N/A
- CURRENCY EXCHANGE RATE: N/A

| Qty | Item | Amount |
|-----|------|--------|
| 3 | ACP101 Accounting Package | $1,350.00 |
| | Annual Subscription to Premier Version with Tax, Inventory and Payroll Plugins | |
| 4.5 | ACP101T Online Training | $495.00 |
| | Hours of Training in Premier Version - Interactive Demos with Q&A Sessions | |
| 10 | ACP101S Standard Support | $1,100.00 |
| | Initial Hours allocated for access to email and phone support for Premier Version | |
| 6 | ACP101C Screen Customization | $660.00 |
| | Hours spent customizing screens in Premier Version for client requirements | |
| 4.5 | ACP101R Report Customization | $495.00 |
| | Hours spent customizing reports in Premier Version for client requirements | |

# TOTALS
- TOTAL AMOUNT EXCLUDING TAX: N/A
- TOTAL TAX AMOUNT: N/A
- TOTAL DISCOUNT AMOUNT: N/A
- TOTAL AMOUNT INCLUDING TAX: N/A
---
# DOCUMENT TYPE
- Invoice

# DOCUMENT PROCESSING COUNTRY
- United States

# DOCUMENT DATE
- 12 March 2020

# DOCUMENT NUMBER
- 00000135

# REFERENCE DOCUMENT NUMBER
- F0016

# SUPPLIER DETAILS
- NAME: Metal Legal Finance
- ADDRESS: 154-164 The Embarcadero, San Francisco, CA 94105
- TAX IDENTIFICATION NUMBER: NA

# CUSTOMER DETAILS
- NAME: NA
- ADDRESS: NA
- TAX IDENTIFICATION NUMBER: NA

# DELIVERY/SHIPPING LOCATION
- NAME: NA
- ADDRESS: NA

# BILLING ADDRESS
- 154-164 The Embarcadero, San Francisco, CA 94105

# CURRENCY CODE
- NA

# CURRENCY EXCHANGE RATE
- NA

| Qty | Item                                      | Amount  |
|-----|-------------------------------------------|---------|
| 2   | ACP101I System Imports                    | $220.00 |
|     | Hours spent importing customer records    |         |
|     | into Premier Version                      |         |
|-----|-------------------------------------------|---------|
| 3   | ACP100 Accounting Package                 | $900.00 |
|     | Annual Subscription to Standard Version   |         |
|     | of Accounts System                        |         |
|-----|-------------------------------------------|---------|
| 4.5 | ACP100T Online Training                   | $495.00 |
|     | Hours of Training in Standard Version -   |         |
|     | Interactive Demos with Q&A Sessions       |         |
|-----|-------------------------------------------|---------|

# TOTAL AMOUNT EXCLUDING TAX
- $5,715.00

# TOTAL TAX AMOUNT
- NA

# TOTAL DISCOUNT AMOUNT
- NA

# TOTAL AMOUNT INCLUDING TAX
- $5,715.00

I'm wondering if I hosted a commercial app with this, and if the users are uploading their same files along with new files, I think that would make a huge cost incurrence on our end!

That's what it claims to do.
invalidate_cache=False, # If set to true, the cache will be ignored and the document re-processes. All document are kept in cache for 48hours after the job was completed to avoid processing 2 time the same document.
which is False by default.
I have not been able to skip already parsed documents though, even when explicitly setting invalidate_cache=False.
Would be great to get some pointers @logan-markewich if you have the time.