Reuse the last rendered frame for the next frame

I'm writing a gui application using Skulpin and don't want to draw the whole gui every frame, but only the areas that changed since the last frame. When doing so, I experience heavy flickering (presumably because of double (or so) buffering, so every second frame has the old content still drawn on it). It would be great to have an option to reuse the last frame and draw with Skia on top of it, so you could just overwrite the areas that have changed and save cpu time as well as battery drain.

I think it's a reasonable suggestion and would be happy to review a PR. However I don't plan to add any more features myself because I don't personally use this project anymore and have other projects I would rather work on.


Use dpdk-udp client and pktgen fail.

I want to test performance with the dpdk-udp and pktgen ,I referred to ans_performance_report.pdf ,but the result was not fit.
[Situation]
                         Compute A                                                          Compute B
                                    ----------------------10G sfp-----------------------------
                          run ans                                                                 run pktgen
                          run dpdk-udp
                          ip   : <IP_ADDRESS>                                                     mac : 00:90:0b:6e:8f:94
                        mac : 00:90:0b:6e:8e:f4

                                command                                                        command
                              ./dpdk-udp 7                                                 ./pktgen -c 0x9 -n 1 -- -P -m "3.0"
                                                                                set 0 proto udp
                                                                                set 0 dst ip <IP_ADDRESS>
                                                                                set 0 src ip <IP_ADDRESS>/24
                                                                                set 0 dst mac 00:90:0b:6e:8e:f4
                                                                                set 0 sport 9999
                                                                                set 0 dport 8888.
                                                                                start 0

The result of pktgen Rx only 10 pkts/sec
Thanks

I think your pktgen didn't enable garp. ans didn't learn arp of <IP_ADDRESS>.
You can use ans_cli commands to check.
anscli> ip neigh show.
anscli> ip stats show.


respect new file location

Currently, all newly created files get created in the vault root.
It would be great if the new file location setting could be respected.
app.vault.getConfig('newFileLocation')
'root'
'current'
'folder' -> a user-specified folder
if ('folder') {
  app.vault.getConfig('newFileFolderPath')
  'folder-name'
}

Hi, @kometenstaub.
Various Complements already respect the following options.

Does it satisfy your requirements?

Hi, @kometenstaub.
Another Quick Switcher already respect the following options.

Does it satisfy your requirements?

Huh, that's odd. That's the option I mean.
For me it placed all the files at the vault root although I was in a folder. I'll try to repro.

If you can reproduce it in the sandbox vault, it will be very helpful for me :)

Yes I can reproduce it in the sandbox vault with the same folder as current file option on Obsidian 0.15.2

The code looks whether there is a folder in the query with regex, but it doesn't checkt the default file location setting.
https://github.com/tadashi-aikawa/obsidian-another-quick-switcher/blob/5fea94cd2aadb480b23b125f0a0080e85bb1a976/src/app-helper.ts#L266-L278

Could you tell me the reproduction procedure? (especially the options)

Thanks. What are the following settings?

Released in v5.0.2 🚀

Ah.. I had mistaken 😅

OK. fixed in v5.0.3.

Thank you! :)


Introduce empty primitives crate

Introduce an empty bitcoin-primitives crate.
We were give the name on crates.io and previously a version v0.1.16-alpha was released so we use v0.100.0.
Does not include adding api files since that whole thing is in flux. This is an even smaller step than #2908 that does not include dependencies.

Tagged and published.


test-branch/execute-job/wrong-commit-sha-on-pr/9b533772-de9b-472c-9c10-a95af425814b

PR for testing 765191b70887669d66a1be42d81515af398d69e4

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Keras version update

Keras changed their API annoyingly so need this fix to work with latest Keras and thus bumped the version

Pull Request Test Coverage Report for Build 202

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
12 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.4%) to 70.789%

Files with Coverage Reduction
New Missed Lines
%

fancyimpute/keras_models.py
12
64.86%

Totals

Change from base Build 197:
-1.4%

Covered Lines:
601

Relevant Lines:
849

💛  - Coveralls


Add "transform" helper

Something like:
val transform :
  ('a -> ('b, 's) stream -> ('c * 'a) option) ->
  'a -> ('b, 's) stream ->
    ('c, 's) stream

This would eliminate many situations in which force streams to be constructed using a ref and Markup.stream, e.g. this.

The type of the function given as argument needs more consideration, e.g. maybe it should be
'a -> ('b, 's) stream -> 'c list * 'a option

or something simpler. Putting this on hold until there is more feedback.


Removing a Single Point of Failure caused by the -ExtranetLockoutRequirePDC switch

I would like to thank you for creating this page and sharing this information.  Experience has shown us that it is very beneficial to have an HA ADFS farm with nodes in data centers that are located across the AMER, EMEA and APAC regions.  One of the benefits of this architecture that you mentioned is the “Elimination of single point of failure: With failover capabilities of Azure Traffic Manager, you can achieve a highly available AD FS infrastructure even when one of the data centers in a part of the globe goes down”.  I would like to share one lesson that we learned regarding the use of extranet lockout in this architecture.  If you have it enabled and  don’t run “Set-AdfsProperties -ExtranetLockoutRequirePDC $false”, if some of the ADFS servers in the farm loose internal connectivity to the datacenter that hosts the domain controller with the PDC emulator role, the ADFS servers that lost internal connectivity to the PDC could appear to be up from an internet connectivity\health check standpoint but external users will not be able to authenticate to them via the Web Application Proxy(WAP) server.  It would like to suggest having a reference to the article that explains the –ExtranetLockoutRequirePDC switch because it could help your customers to remove a single point of failure and create an global HA ADFS farm that is even more resilient to either external or internal connectivity failures.
Regards
Devon Henry
References:
Best practices for securing Active Directory Federation Services
https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/operations/configure-ad-fs-extranet-soft-lockout-protection
Configure AD FS Extranet Lockout Protection
https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/operations/configure-ad-fs-extranet-soft-lockout-protection

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

ID: c1af7d15-2153-5e39-0e03-17c44066472b
Version Independent ID: a52da4ab-a8e9-60c3-e67b-bc0b36666c97
Content: High availability cross-geographic AD FS deployment in Azure with Azure Traffic Manager
Content Source: WindowsServerDocs/identity/ad-fs/deployment/active-directory-adfs-in-azure-with-azure-traffic-manager.md
Service: active-directory
GitHub Login: @anandyadavMSFT
Microsoft Alias: anandy;billmath

Thanks for your dedication to our documentation. If we have been unable to review your issue in a timely manner, we sincerely apologize for the delayed response. We have captured the informtion you have provided for review. We are closing this issue for now, but if you feel that it's still a concern, please respond and let us know. If you determine another possible update to our documentation, please don't hesitate to reach out again. #please-close

Hi Ingrid
Thank you very much for the follow-up on this issue.  I would still recommend and reiterate including in this article a reference to the following command:   Set-AdfsProperties -ExtranetLockoutRequirePDC $false
Experience has taught us that setting this command will help eliminate a single point of failure and help ensure that the ADFS\WAP server farm will be able to continue to authenticate external clients if the datacenter that contains the domain controller with the PDC emulator FSMO role is offline and, therefore, unreachable by the other ADFS\WAP servers in the farm that are in the other data centers that are still online.  Please note what is mentioned in the following excerpt from the article in the references section below:
"Server 2016 offers an additional parameter that allows AD FS to fallback to another domain controller when the PDC is unavailable:
ExtranetLockoutRequirePDC  - When enabled: extranet lockout requires a primary domain controller (PDC). When disabled: extranet lockout will fallback to another domain controller in case the PDC is unavailable."
References:
https://docs.microsoft.com/en-us/windows-server/identity/ad-fs/operations/configure-ad-fs-extranet-soft-lockout-protection
Kind regards
Devon Henry


Bitrise integration

Please do not change the current working directory in a step unless it is necessary (and in this case restore the previous value once the step finished: cd -).
If you change the workdir further steps using relative path as input will may fail.
I think in your case you do not need to change the workdir:

i do not see any npm package descriptor file (packages.json), so i think no npm install is needed in the step.sh
the step.js can be called as: $THIS_SCRIPT_DIR/upload.js "${WTU_DEBUG_MODE}" "${WTU_MAILSENDER}" "${WTU_MAILRECEIVER}" "${WTU_FILEPATH}" "${WTU_MESSAGE}" "${WTU_LANGUAGE}"
to install wetransfert simple use: npm install -g wetransfert, this will install wetransfert globally

To pass inputs to upload.js use the  lower case snake case style input IDs, like:
$THIS_SCRIPT_DIR/upload.js "${debug_mode}" "${mail_sender}" "${mail_receiver}" "${path}" "${message}" "${language}"

Ok thanks Krisztián. I'm sending a new PR.

Hi @godrei, i create a new PR, how to rewrite an existing PR ? Just to conserve history ?

Hi @kawaiseb , do you mean a new steplib pr? if you want to share the new version of the step, just drop the existing steplib pr and create a new one.


disableScroll not working on iOS 12

iOS version: 12
Mobile Safari version: 12
Doing the following works in macOS (Mojave) Safari 13 but not iOS mobile Safari 12.
Body still scrolls:
MicroModal.show( 'my-modal', {
  disableScroll: true
});

Thanks for reporting @jq-87. Will investigate.

Should be fixed with #395


twister: platform exclude not working when using prepared artifacts

Describe the bug
When running twister device testing on artifacts ready for testing, --platform and --exclude-platform seem to have no effect.
I would expect that with 3 platforms in the artifacts, already compiled and test cases in the .json files, that running the device-testing twister command to only test a single platform should ignore all the other platforms in the testplan.json and only test the board defined with the flag --platform. Below I am trying to test xmc47_relax_kit but ignore the 2 other boards.
It seems to just be going off the testplan.json/twister.json file and ignoring the flags
To Reproduce
Steps to reproduce the behavior:

python3 scripts/twister -p cy8cproto_062_4343w -p cy8cproto_063_ble -p xmc47_relax_kit -T samples -v -i -j10 -n --prep-artifacts-for-testing (meant to be run on a different computer but fails both ways)
west twister -p xmc47_relax_kit --device-testing --hardware-map map.yml -T samples/ -v -W -n --test-only OR
west twister -p xmc47_relax_kit -P cy8cproto_062_4343w -P cy8cproto_063_ble --device-testing --hardware-map map.yml -T samples/ -v -W -n --test-only
See that twister will show lots of errors and show failures because of No device to serve as cy8cproto_063_ble platform. error.

Expected behavior
Twister should only test the platforms that are defined by the --platform flag when reusing artifacts
Impact
Should be addressed
Logs and console output
Used samples/basic for quicker run time
Doing $ west twister -p xmc47_relax_kit --exclude-platform cy8cproto_062_4343w --device-testing --hardware-map map.yml -T samples/basic -v -W -n --test-only also results in the same logs
$ python3 scripts/twister -p cy8cproto_062_4343w -p xmc47_relax_kit -T samples/basic -v --prep-artifacts-for-testing
ZEPHYR_BASE unset, using "C:\Users\havrylyuk\zephyr\zephyr"
Renaming output directory to C:\Users\havrylyuk\zephyr\zephyr\twister-out.3
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.4.0-3509-g662a14a25e41
INFO    - Using 'zephyr' toolchain.
INFO    - Building initial testsuite list...
INFO    - Writing JSON report C:\Users\havrylyuk\zephyr\zephyr\twister-out\testplan.json
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2\build.log

INFO    - 28 test scenarios (56 test instances) selected, 37 configurations skipped (34 by static filter, 3 at runtime).
INFO    - 18 of 56 test configurations passed (94.74%), 0 failed, 1 errored, 37 skipped with 0 warnings in 411.03 seconds
INFO    - 0 test configurations executed on platforms, 19 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister.json
INFO    - Writing xunit report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister.xml...
INFO    - Writing xunit report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister_report.xml...
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2 on cy8cproto_062_4343w error (Build failure)
INFO    -
INFO    - To rerun the tests, call twister using the following commandline:
INFO    - west twister -p <PLATFORM> -s <TEST ID>, for example:
INFO    -
INFO    - west twister -p cy8cproto_062_4343w -s samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2
INFO    - or with west:
INFO    - west build -p -b cy8cproto_062_4343w -T samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - Run completed

$ west twister -p xmc47_relax_kit --device-testing --hardware-map map.yml -T samples/basic -v -W -n --test-only
Keeping artifacts untouched
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.4.0-3509-g662a14a25e41
INFO    - Using 'zephyr' toolchain.

Device testing on:

| Platform        |           ID | Serial device   |
|-----------------|--------------|-----------------|
| xmc47_relax_kit |<PHONE_NUMBER>09 | COM8            |

INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\sys_heap\sample.basic.sys_heap\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.picolibc.open_addressing.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.newlib.cxx_unordered_map.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.newlib.open_addressing.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.newlib.separate_chaining.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.minimal.open_addressing.djb2\build.log
No device to serve as cy8cproto_062_4343w platform.
see: C:\Users\havrylyuk\zephyr\zephyr\twister-out\cy8cproto_062_4343w\samples\basic\hash_map\libraries.hash_map.minimal.separate_chaining.djb2\build.log

INFO    - 28 test scenarios (56 test instances) selected, 34 configurations skipped (34 by static filter, 0 at runtime).
INFO    - 8 of 56 test configurations passed (36.36%), 8 failed, 0 errored, 34 skipped with 0 warnings in 65.03 seconds
INFO    - In total 16 test cases were executed, 34 skipped on 0 out of total 609 platforms (0.00%)
INFO    - 8 test configurations executed on platforms, 14 test configurations were only built.

Hardware distribution summary:

| Board           |           ID |   Counter |
|-----------------|--------------|-----------|
| xmc47_relax_kit |<PHONE_NUMBER>01 |         8 |
INFO    - Saving reports...
INFO    - Writing JSON report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister.json
INFO    - Writing xunit report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister.xml...
INFO    - Writing xunit report C:\Users\havrylyuk\zephyr\zephyr\twister-out\twister_report.xml...
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) samples\basic\hash_map\libraries.hash_map.minimal.separate_chaining.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 2) samples\basic\hash_map\libraries.hash_map.minimal.open_addressing.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 3) samples\basic\hash_map\libraries.hash_map.newlib.separate_chaining.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 4) samples\basic\hash_map\libraries.hash_map.newlib.open_addressing.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 5) samples\basic\hash_map\libraries.hash_map.newlib.cxx_unordered_map.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 6) samples\basic\hash_map\libraries.hash_map.picolibc.separate_chaining.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 7) samples\basic\hash_map\libraries.hash_map.picolibc.open_addressing.djb2 on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    - 8) samples\basic\sys_heap\sample.basic.sys_heap on cy8cproto_062_4343w failed (No device to serve as cy8cproto_062_4343w platform.)
INFO    -
INFO    - To rerun the tests, call twister using the following commandline:
INFO    - west twister -p <PLATFORM> -s <TEST ID>, for example:
INFO    -
INFO    - west twister -p cy8cproto_062_4343w -s samples\basic\sys_heap\sample.basic.sys_heap
INFO    - or with west:
INFO    - west build -p -b cy8cproto_062_4343w -T samples\basic\sys_heap\sample.basic.sys_heap
INFO    - -+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
INFO    - Run completed

Environment (please complete the following information):

OS: Linux/Windows
Toolchain: Zephyr SDK
Commit SHA or Version used: a7df4295e2539fdbf0938a4e755adb8f00825170

Created a PR with purposed test-only platform filtering
https://github.com/zephyrproject-rtos/zephyr/pull/62671

PR merged


Fix egress routes generation for services of type "ExternalName"

Fixes #13193.
ExternalName support for internal services is already implemented and
TCP case works. However the simple HTTP-case is broken due to missing
alternative hostnames in the routes config.
Extra field had to be introduced to propagate the value of the
"externalName" field. "ExternalName" is really misleading and
Kubernetes-specific, that's why a more generic name was chosen (see
"ServiceAttributes.Target").
I'm planning to add an extra integration test for this using the new integration testing framework once this gets into master branch (as the version in the release branch is lacking certain capabilities).

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

Needs approval from an approver in each of these files:

pilot/OWNERS

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

@philrud: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
00ae6e2ee45ee6f23f8d357cd772fd62b21d0206
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@philrud: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
00ae6e2ee45ee6f23f8d357cd772fd62b21d0206
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
00ae6e2ee45ee6f23f8d357cd772fd62b21d0206
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/hold
The real issue doesn't seem to be the alias

@rshriram thanks for review! I assume you propose to store a set of aliases on the target service instead of storing targets on the alias services. Initially I started with this approach as it seemed more straightforward however there was a couple extra things to take care of:

deleting or updating an "alias" service needs to be taken care of, "AliasHostnames" fields need to be updated on the target services side accordingly
sidecar scoping needs to be taken into account
The result seemed somewhat dirty and fragile, so I went with this approach instead as there's nothing special needs to be done for deletes/updates and sidecar scoping. Also we already traverse through the services while building 'nameToServiceMap', so this isn't adding any considerable overhead.

@rshriram I'm not following what you propose here.
The bug is that ExternalName is not respected. The PR seems to fix ( I agree "Target" is not a good name, and we may want to support a list of external names - but can be done in separate PR later).
The information about external name must be isolated and come from the Service itself - if a Service
is not explicitly imported that name should not become visible or mess up settings of a different namespace.
@philrud  while I appreciate the unit test, this is the kind of feature that must have at least an integration
test and a test that verifies that isolation is respected, we have far too many bugs where Sidecar is not
respected. You can add it to the stability cluster, or the new framework with 2 namespaces.

After more thinking and discussion - I think we should hold until we better understand if this has any security/policy/telemetry implication.
And first we need to make sure we have the equivalent supported by Istio API - ServiceEntry,
since we can't add k8s-only features. Once we confirm that a ServiceEntry with the external name works,
we can follow the same path as k8s Ingress - do a transparent conversion.

closing this based on the input received from @rshriram and @costinm

@philrud I can't seem to find issue 13193. Is there perhaps a different issue # instead? I'm putting together the release notes, that's why I'm asking.

@geeknoid this is weird, 13193 is the correct issue ## actually, but looks like the issue is gone. Either a GitHub bug or somebody with admin privileges must've deleted it.


Display all possible login names on user detail view

With version  > 2.9.0 it is also possible to login with the e-mail address or the phone number. If these features aren't disabled, display also these possible login names on the user detail view. Currently only the username@domain combination and username are displayed.

In order to show the data, the user profile has to be extended with this information backend-wise as a regular user has no access to request the policy.

We need the policy from the backend

Hei @DoGab
As we start with scrum in January, we have created the related user stories to your request.
Feel free to track the releated user stories as we will close this one, to avoid duplicates: #4928 #4929


[SPARK-23523] [SQL] [FOLLOWUP] Minor refactor of OptimizeMetadataOnlyQuery

What changes were proposed in this pull request?
Inside OptimizeMetadataOnlyQuery.getPartitionAttrs, avoid using zip to generate attribute map.
Also include other minor update of comments and format.
How was this patch tested?
Existing test cases.

cc @gatorsmile @cloud-fan

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/testing-k8s-prb-make-spark-distribution/1147/
Test PASSed.

Test build #87775 has started for PR 20693 at commit a3cf3ca.

LGTM

Test build #87775 has finished for PR 20693 at commit a3cf3ca.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class LocalRelation(output: Seq[Attribute],

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/SparkPullRequestBuilder/87775/
Test FAILed.

retest 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/testing-k8s-prb-make-spark-distribution/1152/
Test PASSed.

Test build #87783 has started for PR 20693 at commit a3cf3ca.

Test build #87787 has started for PR 20693 at commit 0a3d84a.

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/testing-k8s-prb-make-spark-distribution/1156/
Test PASSed.

LGTM

Test build #87783 has finished for PR 20693 at commit a3cf3ca.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class LocalRelation(output: Seq[Attribute],

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/SparkPullRequestBuilder/87783/
Test PASSed.

Test build #87787 has finished for PR 20693 at commit 0a3d84a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class LocalRelation(

Merged build finished. Test PASSed.


Jupyterhub server fails

I currently experiencing once more issue 121 that was resolved last time by  increasing from 2GB to 16GB the memory

@erikerlandson @MichaelTiemannOSC

I am experiencing the same issues. I cannot start a server. Is there maintenance going on?

a transitive dependent package: markupsafe has removed a function soft_unicode in version 2.1.0 https://github.com/pallets/markupsafe/blob/b341ea3c59218e25a8ce2c1c2c5ca885e315e41e/CHANGES.rst#version-210
the jijna2 package which is a transitive package of flask and jupyterlab, uses the function in version below 2.11.0 , we would have downgrade the markupsafe package, to fix this issue.

Hey folks, the aicoe-osc-demo image should now be fixed. I am able to spawn a JupyterHub pod on cl1 with that image, please let me know if the error is fixed for you or you're still running into this problem.

Can we close this issue?

After this merge I went back to the JupyterLab hub server, and was immediately given the options I needed to start my notebook correctly.  Thanks!

I wrongly closed this issue, which is against aicoe-osc-demo, a notebook configuration that I don't use.  I thought it was fixing a different issue (https://github.com/operate-first/apps/issues/1918), which I also closed.  If this issue remains a problem, please reopen!


Drop support for EOL Python

Python 2.6-3.5 are EOL and no longer supported by the upstream CPython team.
They're also little used. Here's the pip installs for pyperclip from PyPI for August 2021:

category
percent
downloads

3.7
39.55%
868,110

3.8
24.16%
530,407

3.6
15.43%
338,797

null
8.92%
195,714

3.9
8.54%
187,457

2.7
3.02%
66,251

3.5
0.29%
6,398

3.4
0.05%
1,174

3.10
0.04%
773

3.11
0.00%
5

3.3
0.00%
5

Total

2,195,091

Source: pip install -U pypistats && pypistats python_minor pyperclip --last-month
Dropping allows to use modern Python features and reduces the maintenance burden.

Also upgrade to modern Python syntax using https://github.com/asottile/pyupgrade --py36-plus.

And add python_requires to setup to help pip install the best version for the user's running Python.

This renders obsolete some old issues/PRs:

Closes https://github.com/asweigart/pyperclip/issues/133
Closes https://github.com/asweigart/pyperclip/issues/101
Closes https://github.com/asweigart/pyperclip/pull/131
Closes https://github.com/asweigart/pyperclip/pull/187
Includes and closes https://github.com/asweigart/pyperclip/pull/211

I understand your argument, but the maintenance burden for having Pyperclip continue to run on Python 2.7 is minimal. The entire code base is under 1000 lines long. I don't really see a need to drop support for it. Thank you.


opam-ci hardcodes ocaml version strings

The OCaml version strings for the latest OCaml compiler version, the latest stable version and so on can be retrieved using the ocaml-version rather than being hardcoded in opam-ci itself.
We should port the hardcoded uses to library calls from ocaml-version, and add any utility function to that upstream library if necessary.

Seems to be related to #22


HTTP Client unable to post raw body with more than a thousand ampersands

Laravel Version: 9.12.1
PHP Version: 8.1.5

Description:
In a new Laravel install, running the following command fails with the error message below:
Artisan::command('send', function () {
    Http::withBody(
        str_repeat('A text with a thousand ampersands &. ', 1000), 'text/html'
    )->post('https://some-url.com');
});

  ErrorException

  parse_str(): Input variables exceeded 1000. To increase the limit change max_input_vars in php.ini.

  at vendor/laravel/framework/src/Illuminate/Http/Client/PendingRequest.php:911
    907▕             $laravelData = (string) $urlString->after('?');
    908▕         }
    909▕
    910▕         if (is_string($laravelData)) {
  ➜ 911▕             parse_str($laravelData, $parsedData);
    912▕
    913▕             $laravelData = is_array($parsedData) ? $parsedData : [];
    914▕         }
    915▕

I understand that this could be solved by increasing max_input_vars, but I'm confused why the raw POST body is being parsed as query/form-data parameters at all. I'm not familiar enough with the HTTP Client to understand all details, but the problem might be related to testing-specific code (relevant when faking the client).
By making the following change in PendingRequest.php, the request is successful and the correct data is sent.
protected function parseRequestData($method, $url, array $options)
{
    $laravelData = $options[$this->bodyFormat] ?? $options['query'] ?? [];

>>   // This "solves" the problem, by ensuring the parse_str on line 911 is skipped.
>>    if ($this->bodyFormat === 'body') {
>>        return [];
>>   }

    ...

    if (is_string($laravelData)) {
        parse_str($laravelData, $parsedData);

The Client tests in the framework seem to pass with this change.
Steps To Reproduce:
The example mentioned above is sufficient, and note that a thousand ampersands could commonly appear in HTML code being transmitted (as in our use case). Changing the content type to text/plain doesn't help.
Http::withBody(
    str_repeat('A text with a thousand ampersands &. ', 1000), 'text/html'
)->post('https://some-url.com');

I have to be honest that I don't know myself. But if you can work around this with max_input_vars then I suggest to do that. Otherwise I think it's okay to try a PR with the change you proposed. I also tried this and no tests failed for me but I'm unsure if we have enough coverage for the faking of HTTP requests.

I'll try to make a PR with these changes and we can discuss the validity of the solution there.
I don't think increasing max_input_vars is a viable solution, in cases where user-provided input is sent in a POST request, there could be any number of & symbols and having an upper limit at all is quite strange.

@Krisell agreed, but if it's a workaround for you right now then I suggest to take it. I'm not entirely sure what will break if we implement the change you propose.

I did a bad job searching for previous reports of this issue. Here's the same issue reported for context: https://github.com/laravel/framework/issues/36976
I'll read through the historic discussions and see if I can understand what is going on.


Time Series Cross-Validation

For time series forecsting, we employ PrequentialSplit;
For time series classification, we employ StratifiedKFold;
For time series regression, we employ KFold.


feat(a11y): improve the a11y of the example

It fixes #158.
I've checked the example with different mobiles resolutions and should no be any problems using the table instead of the previous HTML.

Thanks for taking a look at this and diving in!
With Google I/O coming up quickly, I'm going to hold off on reviewing/merging this until after that. I've got a ton of other things that I need to prep for. I really appreciate the work you've done on this!!
thanks!

ok, don't worry (I don't envy your ton of things related with the incoming Google I/O hahaha)  but please don't forget this pull request ;)


Add files via upload

Added colleges domain to the list.

@iiHR5 First stage of verification passed, official domain is gpranchi.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@iiHR5 Thank you for your patience while waiting for a review and processing your request. Pull request merged.

@philipto Thanks for the update. 👍🏻


💡 [DOCS]: ADD PAGESPEED-INSIGHTS SCORE

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

There are a number of advantages to showing the PageSpeed Insights score in your GitHub repository.
First, it can help you identify areas where your website can be improved.
Second, it can help you attract users who are looking for websites with good performance.
Third, it can help you improve your search engine ranking.

Describe the solution you'd like

Google PageSpeed Insights is a free tool that helps you measure the performance of your website. It provides a score from 0 to 100, with higher scores indicating better performance.
The score is based on a number of factors, including the loading time of your website, the use of images and other resources, and the use of caching.
My goal is to add the Score in Svg format, the main advantage is that i can develop the score in animated svg which is way more attractive and gives professional look to  the repository

Example Image

Additional context

Please assign me this issue under gssoc23, i would like to contribute to this repository.

Code of Conduct

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

@0xabdulkhalid thanks for raising this issue, but we don't need it for now.


⚠ Bofrost.gr has degraded performance

In 3ebc4b3, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1571 ms

Resolved: Bofrost.gr performance has improved in 6330838 after 6 minutes.


[BUG] [DOC] Optimize development doc in user-guide

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
StreamPark already supports multiple data sources, and the configuration file has also changed, so it needs to be adjusted
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

please assign me

please assign me


enforce topic on macro and sub issues

requirements
the topic is what is written in square brakets at the beginning of an issue title: [sample view] implement buttons

per new guidelines, the topic should be written if, and only if, the issue is a macro or sub issue

find a way to discourage opening sub-issues without a topic set
do not encourage a topic on standard issues that don't contain sub-issues

specs

disable the "new sub-issue" button if there is no topic in the issue title

explain the reason with a popover or other message

do not put the topic in the default template

otherwise we'd continue writing it everywhere..

misc
{optional: other useful info}

@gabro @FrancescoCioria @francescogior @giogonzo we talked about this last week, check the issue and tell me if this implementation is good in your opinion :)

what about, when one clicks in the add subIssue button a modal/text input is prompt and the user must write the topic if not present?

well that's even better!

what about, when one clicks in the add subIssue button a modal/text input is prompt and the user must write the topic if not present?

IMO this is very complex and leads to the same result than a simple tooltip


block/aoe: use BPF to filter requests for other servers

I added functionality tonight to my raw sockets package to enable attaching a BPF program to the raw socket.  glide gave me some difficulties so that package bump is not included here.  I'd be happy to look into getting that done tomorrow.
This is the improvement I mentioned in #184 , and should stop the AoE server from getting bogged down handling requests that it doesn't care about.

what happens if the mtu changes at runtime?

The interface's MTU is already relied on for:

allocating a byte slice to read in ethernet frames
determining sector size to report to AoE clients

So I imagine this would not be the only problem area if the interface MTU were to change underneath the service.  In this case, we could also just hardcode it to 9000 since the return value just asks BPF to return "up to X" bytes.

Updated, @xiang90 !  @mischief , any more thoughts on the MTU issue?

All looks good to me operationally. I'd say, yeah, you probably want a server type, of which this is roughly a good first implementation. Someday, when you want to bake it into your aoe package, we'll just use it :)
:+1:


error after python run_sdk.py

2022-08-11 12:29:20,507 - modelscope - WARNING - ('PIPELINES', 'image-portrait-stylization', 'unet-person-image-cartoon') not found in ast index file
Traceback (most recent call last):
File "run_sdk.py", line 5, in
img_cartoon = pipeline(Tasks.image_portrait_stylization, 'damo/cv_unet_person-image-cartoon_compound-models')
File "C:\Users\George\anaconda3\envs\dctnet\lib\site-packages\modelscope\pipelines\builder.py", line 250, in pipeline
return build_pipeline(cfg, task_name=task)
File "C:\Users\George\anaconda3\envs\dctnet\lib\site-packages\modelscope\pipelines\builder.py", line 162, in build_pipeline
return build_from_cfg(
File "C:\Users\George\anaconda3\envs\dctnet\lib\site-packages\modelscope\utils\registry.py", line 197, in build_from_cfg
raise KeyError(f'{obj_type} is not in the {registry.name}'
KeyError: 'unet-person-image-cartoon is not in the pipelines registry group image-portrait-stylization'

I reinstall pkgs and run run_sdk.py, everything goes well. It seems modelscope not correctly installed.

@menyifang sorry, seems like i have the same mistake after reinstallation of tensorflow and modelscope.

i`ve reinstalled everything including conda env - still the same error. Maybe i need specific version of tensorflow?

can you download the model binary (*.pb) from modelscope? If so, you can also run from source code with downloaded files.

they are downloadable, but when i try to run.py I have
(dctnet) PS C:\DCT-Net> python run.py

2022-08-14 19:54:37,580 - modelscope - INFO - PyTorch version 1.12.1 Found.
2022-08-14 19:54:37,584 - modelscope - INFO - TensorFlow version 2.9.1 Found.
2022-08-14 19:54:37,584 - modelscope - INFO - Loading ast index from C:\Users\George/.cache\modelscope\ast_indexer
2022-08-14 19:54:37,589 - modelscope - INFO - Loading done! Current index file version is 0.3.4, with md5 d41d8cd98f00b204e9800998ecf8427e
2022-08-14 19:54:39.214081: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX AVX2
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-08-14 19:54:43.809963: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1532] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 1638 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3080 Laptop GPU, pci bus id: 0000:01:00.0, compute capability: 8.6
2022-08-14 19:54:44.073478: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1532] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 1638 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3080 Laptop GPU, pci bus id: 0000:01:00.0, compute capability: 8.6
2022-08-14 19:54:44.304234: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1532] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 1638 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3080 Laptop GPU, pci bus id: 0000:01:00.0, compute capability: 8.6
loading model from damo/cv_unet_person-image-cartoon_compound-models\cartoon_anime_h.pb
WARNING:tensorflow:From C:\DCT-Net\source\cartoonize.py:36: FastGFile.init (from tensorflow.python.platform.gfile) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.gfile.GFile.
2022-08-14 19:54:44.419941: I tensorflow/compiler/mlir/mlir_graph_optimization_pass.cc:354] MLIR V1 optimization pass is not enabled
load model damo/cv_unet_person-image-cartoon_compound-models\cartoon_anime_h.pb done.
2022-08-14 19:54:44.459885: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1532] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 1638 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 3080 Laptop GPU, pci bus id: 0000:01:00.0, compute capability: 8.6
loading model from damo/cv_unet_person-image-cartoon_compound-models\cartoon_anime_bg.pb
load model damo/cv_unet_person-image-cartoon_compound-models\cartoon_anime_bg.pb done.
Could not load library cudnn_ops_infer64_8.dll. Error code 126
Please make sure cudnn_ops_infer64_8.dll is in your library path!

Could not load library cudnn_ops_infer64_8.dll. Error code 126
Please make sure cudnn_ops_infer64_8.dll is in your library path!


android jellybean support

Am new to react native, am having trouble on how to install sdk for android jellybean without android studio

android jellybean without android studio
I think you need Android Studio to do that, but please ask on StackOverflow for alternative methods.


Move the difficulty filter into a block::Header method

Motivation
The difficulty filter check uses <=, and it's easy to get <= confused with <.
Solution
Move the difficulty filter into a block::Header method

This is an API cleanup we might do eventually.


ci(pingcap-qe/ee-apps): bump docker image version

This pull request includes an update to the prow-jobs/pingcap-qe/ee-apps/presubmits.yaml file. The change updates the Docker image version used in the presubmits configuration.

prow-jobs/pingcap-qe/ee-apps/presubmits.yaml: Updated the Docker image from ghcr.io/pingcap-qe/ci/secure-merge-check:v1.0.0 to ghcr.io/pingcap-qe/ci/secure-merge-check:v1.1.0.

/approve

/approve


Immediate error upon re-keying star

The following error occurs on bootup as I attempt to restart this star:
https://gist.github.com/urcades/cfe1439265858ea809540a4ddca552a6

Interestingly, when I tried making a new folder in the console to place this pier, I received the following warning on my ec2 instance:
mkdir: cannot create directory ‘temp2’: No space left on device

That makes sense, the error message points to a failed disk write in the snapshotting system. ~sabbus booted successfully, so you should be fine once you clear some space on that disk.

Gotcha! So there has been no breaking error that'd sink the ship? That's a relief 👍

That's right, you should be able to start that ship back up (urbit sabbus/) once you have free space on disk. It will replay some events first, since the most recent snapshot attempt failed.


tscn format is missing from the filesystem panel

Scenes saved in the tscn format do not appear in the filesystem panel. A potentially related issue is that they do not appear either in the "Quick Open Scene" (Ctrl + Shift + O) dialog. On the other hand they appear fine in the "Open Scene" dialog (Ctrl + O).
If the main scene is a tscn file, it seems however to appear fine in the filesystem.
Edit: Actually no, tscn files appear in the filesystem if they have been saved once during the session.

The same behaviour here on 68d3438a9351498909ffa0bfe79f51b74004e906 build


Create timers.md

Updating Portal with latest DX Roadmap

Seems like a duplicate indeed. I have no permission to change this one though.

closing as duplicate


add loading widgets

instead of displaying fake data or no data; indicate that an API call is initiated and the results are being retrieved; render any api error codes to the UI in an alert panel to clearly communicate issues or mis configurations

Implementation idea; determine if the state contains necessary info. Based on this, if the info is there then render the data. If the info is not present, show a 'loading' widget. Similar to one described here:

https://loading.io/css/
https://cssloaders.github.io/
https://css-loaders.com/flipping/

live on site


SQUBS-818: Fixing Unicomplex to make case classes referenced in Statu…

…sReport public. Also, added proper case class for StatusReport. Allows much better type matching.

up-merge pp 0.6.5 to master


⚠ ClamAV has degraded performance

In 17471d8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 82 ms

Resolved: ClamAV performance has improved in 7270ae2.


Transclusion Support

https://www.reddit.com/r/PKMS/s/ExEET5gWbo

Yeah, I was looking at org-transclusion for such a feature, but I would love something more.
It would be cool if this had advanced transclusion features where you can transclude multiple items into one while sorting or filtering them in some way based on some metadata that each item may have.
For example, you have a bunch of tasks under different projects. Then you could transclude them all under a single heading called Today which filters in all the tasks scheduled for the current day.
I don't know if it's completely possible, but it would be awesome. You could make it a full blown todo app with such a feature alone.

You could make it a full blown todo app with such a feature alone

Well I do something very similar to what you are saying: I link all of my tasks to [[LATER]] then get references to [[LATER]] when I want to view the tasks. This workflow will be improved when markdown-oxide supports queries -- allowing filtering by metadata like you said. Right now, I just get references to [[LATER]] then do a grep style filter if necessary. Of course, a structured filter would be better.
The followup question is how would the transclusion UI be displayed? I think the LSP hover window may be good; it could be like the obsidian reading mode. Given that the language server can send what ever it wants in the hover window, there is a lot of potential. Right now, when hovering on a reference (link) or even a definition (heading), the hover window will show a preview of the written contents in combination with backlinks. This can be made more advanced. What do you think of this option?
Of course, we could also make editor specific plugins to give the best transclusion, but this would be a lot of work and some have said it would be out of scope. I'd say: this is better than making our own text editor! but it certainly does increase the scope of the project significantly and would leave some users out of key features if they are using not fully supported editors.

If you're striving to have the best PKMS, you would want to display the transclusion directly in the buffer, where the user can seamlessly read, navigate and edit its contents, without having to shift context. Otherwise, the user experience wouldn't be the best it could be.
I don't know the limits to which a language server could help implement such a feature, but I imagine it would at least be in charge of retrieving the original contents, updating the transclusion block, handling the edits of a transclusion block, and updating the original contents. If you need editor-specific plugins to complete such a feature, you'll have to decide if doing so is worthwhile in order to achieve the best PKMS. In my opinion, it's at least worth experimenting.
If you need examples of such a feature, there is Org-transclusion as well as Obsidian + MAKE.md plugin with make.MD: Open Flow Blocks in Selection command. Org-transclusion seems to do transclusion pretty well. Obsidian not so much as you have to select and click things to edit them. But a big shortcoming for both is that they can't have multiple transcluded elements under a single transclusion block
Being able to have multiple transcluded elements under a single transclusion block would be a pretty significant as it would allow to sort them, filter them, and modify their presentation in a clean manner. If implemented properly, it'd be like having org-agenda directly in the buffer.  You could transclude todo elements from project A and project B, filter by if scheduled, sort by date, modify the presentation so that their dates are shown at the beginning of the line. Basically, a full blown todo app as I said before.
In terms of implementing the feature, I imagine that for each transcluded element, you'd probably need to in-line their id along with them, in order to track their positions better (for sorting and filtering).
Although, the more I speak about this, the more I think it'll take a significant amount of effort to implement. So it's up to you if you think it's worthwhile.

directly in the buffer, where the user can seamlessly read, navigate and edit its contents, without having to shift context

I don't think it would be possible to have this directly in the buffer, but a user could open an Entity View (as I have been calling it in my notes) where all of those actions (even editing) would be possible. Applied to your case (which is also mine)

One could write a query for getting all task items for the current day, while also giving sort preferences and such
One could open the entity view for this query (as the query is the entity representing your agenda)
In this entity view, you could view transcluded items, reorder items manually, and edit all transcluded items -- while also being able to goto definition on the transcluded item...

For my case, instead of using a query, I would use backlinks to the [[LATER]] entity, which would be included in the entity view. I add a [[LATER]] link to all of my tasks: for example - [[LATER]] fix markdown oxide bugs. To get and edit all of these, I could go the LATER.md file, open the entity view, reorder my tasks etc, ... I have my agenda!
Correct me if I am wrong, but this Entity View (in combination with queries) would fulfill what you are looking for
But still how would we implement this entity view? It can not be view only (we can not leave it at LSP hover). My ideas are...

I make plugins for text editors; however, this by itself is not ideal as some text editors (Helix and Zed) do not have great plugin systems yet. I think the only editors this would be possible for are Neovim and VSCode. That is not too bad!
The LSP directs users to a temp file as the entity view; the user then edits, reorders, ... items in this, the language server listens to all of these changes in detail through the LSP -- keeping track of line numbers for transcluded items -- and updates transcluded items accordingly. I expect that this would take a while to be made robust, but it surely is simple and flexible, so for editors that do not have plugin systems, this could be the experimental solution while I could make robust plugins for the supported editors.

And who knows! Maybe the temp file could be a good solution for all text editors; the LSP with hover, definition, references, rename along with a custom text-based entity view interface is grounds for extreme creativity!

And if one wanted to save an entity view, it would be a simple as saving the markdown file to vault! Markdown-oxide would figure out the rest

I don't think it would be possible to have this directly in the buffer,

It's possible in emacs with org-transclusion. When you add/open a transclusion, it simply replaces the #+transclude line with a copy of the text content it points to. Then you can edit it as if it belonged to you (while behind the scenes org-transclusion handles the editing and any inconsistencies).
I imagine this is also possible in neovim. I don't know about the other editors though.

Correct me if I am wrong, but this Entity View (in combination with queries) would fulfill what you are looking for

Similar to queries except it replaces the query block with an editable copy of the text contents when you open the block.
I'm not sure about Entity View. I'd have to see what you mean. From the sound of it, it doesn't sound like what I'm looking for.
I basically want to stay in the markdown file where I can contain multiple transclusion blocks and write things around or in-between them, rather than switching views. I want the experience to be seamless.

The LSP directs users to a temp file as the entity view; the user then edits, reorders, ... items in this, the language server listens to all of these changes in detail through the LSP -- keeping track of line numbers for transcluded items -- and updates transcluded items accordingly. I expect that this would take a while to be made robust, but it surely is simple and flexible, so for editors that do not have plugin systems, this could be the experimental solution while I could make robust plugins for the supported editors.

Having a temp file is interesting. If for whatever reason, you can't paste the  contents directly into the buffer, then maybe a temp file could hold the transcluded contents as some sort of experimental solution.

Instead of temporary files, having a FUSE file system would be cross platform and editor independent. It could also implement read only views and incorporate the transcluded document with some sort of separator in the buffer itself and incorporate and propagate the changes back to the original files

hm very interesting. I will certainly look into this -- thank you!
Another easy thing we could do is use codelens to give small previews. This may work especially well for embedded block links.

Oh I did mean inlay hints

[ ] Use inlay hints to preview embedded links

(if i forget to do this in v1, please reopen)


Add some benchmarking functionality

I added some benchmarking functionality using Criterion. It's pretty weird because it's using such an old LTS resolver (current versions of Criterion have features like envWithCleanup which would have made this a lot cleaner. Anyway, I ran an echo server using different configured periods and got the results linked in the benchmark.html file. You can see they're all between 50-55 ms. The top graph is the most important; I wouldn't worry too much about the standard dev graphs below since the "outliers" are still below 55 ms.
These were my steps to run the benchmark:
stack build
./.stack-work/install/x86_64-osx/lts-5.4/7.10.3/bin/benchmark --output benchmark.html --time-limit 5

benchmark.html.zip

I was unable to run these with stack directly because when I tried to invoke it as a script a la stack script/benchmark.hs it failed with a parse error on one of the #ifndef directives. The workaround was that I made a package with the benchmark.hs file as an executable. The upshot is now that you have to stack build && ./.stack-work/install/x86_64-osx/lts-5.4/7.10.3/bin/benchmark --output benchmark.html --time-limit 5 (or stack install && benchmark --output benchmark.html --time-limit 5. Does that answer your question?

@dfithian yea for sure. The old stack resolver is definitely a problem. I started upgrading in #8 but haven't had a chance to finalize it.
Either way, I love benchmarks to this is awesome. Thanks!


add sitemap_generator gem

This PR wil fix https://github.com/antleaf/nims-mdr-development/issues/480 .

The workflow does not alter the visibility setting. So works can be in Draft and still be visible. (see example solr record solr_dataset_response_json.txt). We would need to check workflow state is Deposited
The fedora state active or inactive state is used when records are deleted and tomb-stoned.
I will make changes to this PR.
Waiting on PR #411

I got confused between the state in Solr and Fedora. The code is correct. Spotted another issue though.


[FLINK-23430][runtime][checkpoint] Allows taking snapshots for operator coordinators which all corresponding tasks finished

What is the purpose of the change
This PR fixes the issue that for the fully finished operator, if it has used the OperatorCoordinator, then when taking checkpoints, the OperatorCoordinator would report its state first and PendingCheckpoint would store an OperatorState for this operator. Then when we finally fulfill the FullyFinishedOperatorState, the current check that no OperatorState is stored would fail.
Brief change log
We would relax the condition to allow if OperatorState is created, then it should not contains task state.
Verifying this change
This change is tested via the added test.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

Very thanks Roman @rkhachatryan for the review! I also agree with that it would also be solved by not taking snapshot for the coordinator whose tasks are fully finished, and it could reduce the overhead. But since we may have a re-consideration of if we want to snapshot and restore the finished operator coordinator in the next, perhaps we may first solve this issue in a simpler way~

That makes sense, thanks for the explanation @gaoyunhaii .

@flinkbot run azure

Very thanks Roman @rkhachatryan for the review! I'll merge the code~


Fix critical bug introduced by pull #836

Sorry I hadn't fully tested everything yet, since I didn't expect it to get pulled before figuring out the date standards
https://github.com/Nandaka/PixivUtil2/pull/836 tries accessing root when it is part of another method, which is fixed now.
Sketch date is still not working properly which I am going to look at next, but this should be merged asap

already handled in my local commit, going to push now.


Type magic is non-functional in RoomCC code

Type magic does not function and is not recognized in file_kind_yy_KYyRoomCCs
In GMEdit

In GMS 2

Fixed with https://github.com/GameMakerDiscord/GMEdit/commit/af4a5f30b1124783362d495c9048c723089125b4


for ExceptionHandlerMiddleware, move Rollbar to a per-request depende…

…ncy to avoid the error:
Cannot resolve scoped service 'RollbarDotNet.Rollbar' from root provider.

I was getting the following error in ASP.NET Core 2.0 and followed the middleware docs  to change Rollbar to a per-request dependency since it needs other per-request stuff like RequestBuilder
An error occurred while starting the application.

InvalidOperationException: Cannot resolve scoped service 'RollbarDotNet.Rollbar' from root provider.
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteValidator.ValidateResolution(Type serviceType, ServiceProvider serviceProvider)

InvalidOperationException: Cannot resolve scoped service 'RollbarDotNet.Rollbar' from root provider.
Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteValidator.ValidateResolution(Type serviceType, ServiceProvider serviceProvider)
Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType)
Microsoft.Extensions.Internal.ActivatorUtilities+ConstructorMatcher.CreateInstance(IServiceProvider provider)
Microsoft.Extensions.Internal.ActivatorUtilities.CreateInstance(IServiceProvider provider, Type instanceType, Object[] parameters)
Microsoft.AspNetCore.Builder.UseMiddlewareExtensions+<>c__DisplayClass4_0.<UseMiddleware>b__0(RequestDelegate next)
Microsoft.AspNetCore.Builder.Internal.ApplicationBuilder.Build()
Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()

Note to self: ignore failing tests, caused by #59... Pull into staging branch and resolve that.

I originally worked around this by changing my void ConfigureServices(IServiceCollection services) section to public IServiceProvider ConfigureServices(IServiceCollection services) and returning services.BuildServiceProvider(); for the time being. Though you're probably right and it's best to use a per-request scope on the middleware (I should have re-checked docs when I ran into this -- all these docs are from way after this code was written). That should at least get you running if that isn't disruptive to your project and still want to just use the nuget package.
I'll see about getting this pulled in by the end of this weekend at the latest though.

Thanks for the awesome work and super fast response!
FYI, I'm in no hurry to see this merged in. I just copied over the middleware class and modified it in my project.


EditView panics on an obscure set of conditions

The panic apparently happens in edit_view.rs:441:8 and it says
thread 'main' panicked at 'assertion failed: `(left == right)`   left: `16`,  right: `72`: Was promised 72, received 16'
Here's a code example. It crashes when terminal is shrunk horizontally and is too narrow to contain the string in SelectView:
extern crate cursive;

use cursive::Cursive;

use cursive::views::{Dialog, SelectView, EditView, LinearLayout};

fn main() {
    let mut siv = Cursive::new();

    siv.add_layer(
        LinearLayout::horizontal().child(
            Dialog::around(
                LinearLayout::vertical()
                    .child(EditView::new())
                    .child(SelectView::new()
                           .item_str("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"))
            )
        )
    );

    siv.run();
}

If I remove either horizontal LinearLayout or Dialog, it works normally. If I replace SelectView with TextView containing the same string, everything works too.
Tested on Ubuntu 16.04 with both ncurses and termion backends. Cursive version 0.7.2
So, uh, this might be a problem.

Indeed, looks like a bug!
Thanks for the report - I'll look into that.

11ea8a592be2da27cd5ea8ec8b17e0977e575a2e should fix that.

Kind of. The example code works, but it still panics if there's another child in the horizontal layout. Doesn't matter what it is, TextView, DummyView, another EditView etc.
siv.add_layer(
    LinearLayout::horizontal()
        .child(TextView::new("aaaaaaa"))
        .child(Dialog::around(
            LinearLayout::vertical()
                .child(EditView::new())
                .child(SelectView::new()
                        .item_str("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa")))
        )
);

It works now. Thanks.


Missing translations ?

Bug Report
Expected behavior/code
I use richie with the french language. Every thing is translated except the search page.
Actual Behavior
The search page is not translated
Steps to Reproduce

switch to french
go to search page

Environment

Richie version: master branch
Platform: I'm using the docker-compose env provided in this repo.

Additional context/Screenshots

Hey @lunika,
Thanks for your report. This is a known issue. The relevant functionality has not been completely implemented yet.
It is already documented, see #89 for more details.

Close as duplicate.


C++?

Hi, and thank you for making this code available.
I would like to convert it to c++,  as I know dlib is a c++ lib.
Specifically this function

import face_recognition
known_image = face_recognition.load_image_file("biden.jpg")
unknown_image = face_recognition.load_image_file("unknown.jpg")

biden_encoding = face_recognition.face_encodings(known_image)[0]
unknown_encoding = face_recognition.face_encodings(unknown_image)[0]

results = face_recognition.compare_faces([biden_encoding], unknown_encoding)

Have you written custom approaches to the face reidentification? Or is it all dlib functionality?
Thanks!

found some other similar questions, and the answer. thanks


Adjust output file name

Adjust output file name to phenotype basename

@gaow Originally, the output file names are from phenotype basename. I suppose phenotype might be more representative among datasets but I don' know.

Thanks @wangyf1125 this is a good patch.


Bump verison of speedtest-cli

Bump version of speedtest-cli dependency to fix issue ValueError: invalid literal for int() with base 10: ''.

Not too sure why the pytype test is failing looks to be working fine for me locally:
pytype
Computing dependencies
Analyzing 3 sources with 0 local dependencies
ninja: Entering directory `.pytype'
[3/3] check prometheus_speedtest.__init__
Leaving directory '.pytype'
Success: no errors found

@jeanralphaviles is this repo maintained anymore?

is this repo maintained?
Yes, but I haven't had the time to get around to it in a while unfortunately. Full time job takes all my energy, hope you can understand.

Once your PR goes in, I'll re-release the docker images. That should fix them up.


Remove "spell-checker:ignore" lines

Some files have a spell-checker:ignore line (that can be huge sometimes). I guess some editor uses this. However it's cumbersome to maintain these while doing code refactors, etc. and they're annoying while grepping the code. My hope is that these lines are obsolete and can be removed, or is anyone relying on them?
Is there any reason not to remove the spell-checker lines?

I think we should remove them. I haven't added them to any new files I've created anyways, so I think we should just remove them from all files to be consistent. Also, they'll probably become obsolete as the project develops.
@rivy from looking at the git blame you seem to have added those lines. Are they still useful to you?

No, the exceptions are still needed to keep a handle on spelling in the code base. Otherwise, there are just an overwhelming number of "false positive" spelling errors to visually parse when looking for "true" spelling errors. Even with the lack of current spelling maintenance and the resulting "false positives", I can see at least 20 spelling errors, typos, and inappropriate words in the current code base (using a quick cspell --config .vscode\cSpell.json **/*.rs).
I'll probably also be adding an automatic spell-check to the "Style" portion of the CICD in the future as a "soft" failure... when time is available and the code base settles a bit.
Using the VSCode "Code Spell Checker" extension (aka, 'streetsidesoftware.code-spell-checker') will point them out as you're coding to minimize inefficiencies... I'll add that extension to the "recommended" list to prompt for folks to use it. I can move some of the exceptions to the config file or specific exception dictionaries. And I'll move to do that with some other maintenance on the exception lines when I get a bit of time.

Would it be possible to add those allowed words to a global file to the repo, instead of to each file individually? Otherwise, if we want to require this, it should be checked by CI that these lines are always up to date and added to every new file.

@miDeb , certainly!
I made some progress toward that kind of consolidation in earlier commits (see cSpell.json). But I got bogged down by the sheer weight of all the words and changes along side the time crunch that arose for me with this last year's pandemic craziness. Hence all the spell-checker:ignore (ToDO) ... lines.
I'm trying to strike a balance between including reasonable jargon, name, people, and project specific words vs pushing developers to produce readable (and correctly spelled) code and comments. I have developed some expanded word lists/dictionaries (see dxx wordlists). I'm working on pulling them together for a commit here (hopefully in the next few days). I'll try to work through some of the "ToDO" spell exceptions at the same time.
I do want to add a CICD Style spelling check (with soft failure), but it has languished on the lower levels of my priority list (PRs welcome :smile:).
Hopefully, with that global pool of words and CI warnings, developers could then either fix spelling "faults" or add their exceptions as spell-checker:ignore ... lines. That would make it fairly straight-forward to review those spelling exceptions at later dates and refactor when needed.
Sound reasonable?

Sounds good! I'll try and investigate how we could integrate spellcheck to CI. If there's anything else where I could help, please let me know.
Additionally, spell checking is not only an advantage for clean code, but in our case we also get to spell-check the outputs of utilities.

A good portion of the work to move the spell checking more centrally is now done in PR #2315.
I'll start work on more of the ToDO spell checker exceptions next week.
Feedback welcome...

I'm happy with the spell checking right now. It also make sense to have some exceptions specified per file. Can we close this?

I see no objections since Jan 19, so closing this :)


Allow Blackbox exporter to scrape admission webhooks

How to categorize this PR?

/area monitoring
/kind enhancement
What this PR does / why we need it:
PR configures blackbox exporter with http_webhooks module that allows a configurations like:
apiVersion: monitoring.coreos.com/v1alpha1
kind: ScrapeConfig
metadata:
  annotations:
    resources.gardener.cloud/origin: garden/blackbox-exporter
  labels:
    prometheus: garden
    resources.gardener.cloud/managed-by: gardener
  name: garden-blackbox-gardener-admission-controller
  namespace: garden
spec:
  metricRelabelings:
  - action: keep
    regex: ^(probe_success|probe_http_status_code|probe_http_duration_seconds|probe_ssl_earliest_cert_expiry)$
    sourceLabels:
    - __name__
  metricsPath: /probe
  params:
    module:
    - http_webhooks
  relabelings:
  - action: replace
    regex: (.*)
    replacement: $1
    separator: ;
    sourceLabels:
    - __address__
    targetLabel: __param_target
  - action: replace
    regex: (.*)
    replacement: $1
    separator: ;
    sourceLabels:
    - __param_target
    targetLabel: instance
  - action: replace
    regex: (.*)
    replacement: blackbox-exporter:9115
    separator: ;
    targetLabel: __address__
  - action: replace
    replacement: gardener-admission-controller
    targetLabel: job
  staticConfigs:
  - labels:
      purpose: availability
    targets:
    - https://gardener-admission-controller.garden.svc/

and have monitoring for the certificate expiration (among other things).
Which issue(s) this PR fixes:
Makes it easier to add CA certificates monitoring on https://github.com/gardener/gardener/issues/10394
Special notes for your reviewer:
/cc @istvanballok @rickardsjp @vicwicker @chrkl
Release note:

Admission webhooks on the runtime cluster can now define scrape configs for blackbox exporter with module `http_webhooks` and will be scraped with success probes for HTTP codes 2XX - 4XX

Similar to #10405


Error in quickshd example

An error in both debug and release mode for building quickshd (on linux btw, if that makes a difference):
source/app.d(41,3): Error: undefined identifier `cam`

Thanks for reporting this, fixed in https://github.com/bmchtech/rengfx/commit/326683555100abbc2cf73e77a2e0869300ee4ca0


v2.2 breaks create-react-app v1

The changes made in v2.2 cause the following error running npm run build with a react-scripts 1.1 (aka "Create React App"):
Creating an optimized production build...
Failed to compile.

Failed to minify the code from this file:

        ./node_modules/autobind-decorator/src/index.js:7

The above issue went away after downgrading autobind-decorator to 2.1.0.
I'm mostly just posting this here as an FYI in case someone else runs into this...

Also we get a problem with code minimizing through UglifyJSPlugin in Webpack.

https://github.com/andreypopp/autobind-decorator/issues/69#issuecomment-435626439

Please read https://babeljs.io/blog/2018/06/26/on-consuming-and-publishing-es2015+-packages


Added support for 'RABBIMQ_CONFIG_FILE'

Coverage increased (+0.05%) to 60.359% when pulling 2fe5d58ba027a38f327ba436b1681c1be8ef0d61 on feature/config-file into 33eb3e1d83901d4beb0dd4b57d8c1d5fc814a726 on master.


Changing config of active carousel causes carousel to reload

I'm using the settings attribute on the slick directive to pass in configuration for my carousel.  I want to be able to stop autoplay after a user has performed a certain action within the carousel.  When I change the autoplay value in my config to false, the slick carousel will reload on the page causing ~1second of the page completely flashing or looking messed up.
Is it possible to make config changes without reloading the carousel?

@YeomansIII I'm having the same issue.  I ended up using jquery:
$('.carousel').slick("slickSetOption", "swipe", false, false);
Did you work out a better, more angular friendly way?

Any plans to expose this method so we can avoid jQuery?


⚠ Bofrost.lu has degraded performance

In 7a7f0f2, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2719 ms

Resolved: Bofrost.lu performance has improved in edefcac after 13 minutes.


[WinForms] DataGridView-related API validate state and throw IOE

Selected DataGridView-related API validate object state and throw InvalidOperationException.
Selected API now validate object state and throw InvalidOperationException where previously a NullReferenceException could be thrown, if APIs were invoked without being constructing with a valid Owner.
Throwing InvalidOperationException is inline with the behaviours in .NET Runtime, and provides developers with a better debug experience clearly communicating which argument caused the exception.
Old behavior
The listed API would throw NullReferenceException if invoked with null input arguments.
New behavior
The listed API would throw InvalidOperationException if invoked without having a valid Owner property value (in addition to the existing exceptions).
Reason for change
Throwing InvalidOperationException is inline with the behaviours in .NET Runtime, and provides developers with a better debug experience clearly communicating which argument caused the exception.
Recommended action

Review and, if necessary, update the code to prevent constructing the affected types with Owner as null.

Category

Windows Forms

Version introduced: .NET 6.0 preview4
Affected APIs

DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject

[x]  DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject.Bounds Property, property Owner
[x]  DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject.DefaultAction Property, property Owner
[x]  DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject.Name Property, property Owner
[x]  DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject.State Property, property Owner
[x]  DataGridViewTopLeftHeaderCell.DataGridViewTopLeftHeaderCellAccessibleObject.Navigate(AccessibleNavigation) Method, property Owner

Issue metadata

Issue type: breaking-change

I'm going to close this issue now. If there are any more IOE changes for .NET 6, please open a new issue.


Fix and clean up index.html

The spacing for fluid section was slightly off due to incorrect class name used for the row div. Also removed an unnecessary closing div and reformatted index.html using prettier.

Thanks!


add --version command to check the version

add --version command to check the version

adding --version flag does work on local environment but it doesn't work after build.
Key points:
var version = os.Getenv("VERSION") in the main.go
and Get the version step in the release.yml file
      - name: Get the version
        id: get_version
        run: echo "VERSION=${GITHUB_REF#refs/tags/}" >> $GITHUB_ENV

If anybody knows what I am missing please let me know.

@pathcl

Trying uppercase variable

Something is missing the release.yml will work on another more important issue with Episodes meanwhile


Run stop hook before killing the session

Hello! 😄
After noticing that my on_project_stop hook was not running correctly when I exited my session (with tmuxinator stop), I then tried to find someone with the same issue as me. I soon found #555, and though it touched this issue, it went back and forth on other things and eventually reached a bug that was preventing the on_project_exit hook from running at all.
From the issue I saw that the on_project_stop hook would run if I ran the tmuxinator stop command from outside the tmux session, but to me that is not ideal, because in my workflow, while I'm working on a project, I never exit that session until I end my work, so it's natural to stop the project from the session itself (and surely I can't be the only one that does that 😰).
Because the OP was trying to exit with kill-session, this bug being fixed ended up being their solution, as well as moving their command to the on_project_exit hook.
So I searched some more and found #486, as well as this comment https://github.com/tmuxinator/tmuxinator/pull/486#issuecomment-265267067.

I think hook_on_stop may need to be moved before tmux_kill_session_command. On my system (Debian 8.6, Bash 4.3.30, Tmux 1.9, Ruby 2.3.1p112) on_stop commands are ignored unless I reverse these two method calls. Is this working for you?

Bingo. From what I could understand from the source code, the tmuxinator stop command actually execs into a script generated on the fly, which is the file with the commands to which the comment refers, and so by swapping the tmux kill session command with the hook method call on that template (template-stop.erb), I could make my on_project_stop work fine, even when running tmuxinator stop from inside the session itself.
So I decided to open this micro PR implementing the change described in the comment (that appears to have been accidentally left out from the original PR). I tried to write some small test, but since Project#kill is actually exec'd, I couldn't find a way to test that. But the only effect this should have is running the stop command before killing the tmux session instead of the other way around.

@mateusmedeiros thanks for the PR!

As the comment of mine referenced above implies, this is (IMO) the expected behavior

I'm also in favor of this change, as that behaviour is probably more expected than the current one ( and the current one should probably be considered a bug ). @ethagnawl didn't push this and I missed the bugged behaviour.
I'm sorry.

I'd like to merge this as-is, but my one concern is that this change could result in -- potentially destructive -- unexpected behavior (e.g. someone has a hook in place which suddenly starts running). I may be overthinking this, though.

@ethagnawl this is potentially true, as someone could rely on it being run only if stop is run from outside the session.
Could we think adding a new hook along the line of on_tmux_session_end that runs in the position @mateusmedeiros proposed instead?

I was overthinking the scenario I mentioned above re: unexpected behavior. 1) There probably aren't many people using this feature and 2) they've opted in, so the behavior really can't be considered unexpected.
I also think the proposed solution is preferable to adding yet another hook and leaving the semi-broken one in place.
Does anyone else have thoughts about any of the above? Ping @tmuxinator/tmuxinator.

the proposed solution is preferable to adding yet another hook and leaving the semi-broken one in place

I agree.
We may want to do a minor version bump for this, although you could make an argument either way.

We may want to do a minor version bump for this, although you could make an argument either way.

Sure, that sounds reasonable.
@mateusmedeiros Per CONTRIBUTING.md, would you mind adding an entry to CHANGELOG.md? Once that's in, I will merge this and cut a release.


Translate tasks/access-application-cluster/service-access-application-cluster in Korean

This is a Feature Request

What would you like to be added

Translate tasks/access-application-cluster/service-access-application-cluster in Korean
Why is this needed
No translation with tasks/access-application-cluster/service-access-application-cluster in Korean
Comments

Page to Update: https://kubernetes.io/docs/tasks/access-application-cluster/service-access-application-cluster/

/language ko
/assign

/close


CSRF token missing or incorrect

We noticed that applying changes to a zone sometimes gives the following error:
flask.app - WARNING - Forbidden (CSRF token missing or incorrect.): /domain/redacted-domain-name.ext/apply

And found the following at https://flask-wtf.readthedocs.io/en/v0.12/csrf.html#troubleshooting

When you define your forms, if you make the mistake of importing Form from wtforms instead of from flask.ext.wtf, most features besides CSRF protection will work (aside from form.validate_on_submit()), but CSRF protection will fail. Upon submitting forms, you’ll get Bad Request/CSRF token missing or incorrect (and the form.csrf_token in your template will produce no output). The problem is in your broken import statements, not your configuration.

Unsure about the exact fix necessary, but this looks like a bug in PowerDNS-Admin.

The message will be coming from Flask-SeaSurf rather than wtf, but you'd generally expect it to happen either all the time or not at all. It's a bit hard to guess how you might be getting the error only sometimes. Are there any patterns to what you're doing when you get the error, or the zones you get the error on? Do you have a zone that will always give you an error? Are these particularly large zones we're talking about?

I am getting the same randomly upon login. I get a 403 forbidden in nginx and the following in the logfile.log:
2019-08-08 17:06:42,886 - flask.app - WARNING - Forbidden (CSRF token missing or incorrect.): /login

Something very odd going on for that to happen... for some of the other bigger more complicated pages with lots of javascript it's more understandable, but not for the login page. Each time the flask app restarts it generates a random token. When you load the login page that token is served up to you, and when you click login the browser sends the token, user and password back to the server. The only obvious way the token could be incorrect in that case is if the app restarted between you loading the login page and hitting the login button.

We too are seeing the flask.app - WARNING - Forbidden (CSRF token missing or incorrect.) error for different pages on and off:

/template/create
/admin/manageuser
/domain//apply
/login
/template//apply
/admin/setting/dns-records

It happens too rarely to find a pattern and/or make it reproducible.

This looks to be a similar issue: https://github.com/plotly/dash/issues/132
Could you see if passing --preload makes it go away? It looks like PowerDNS-Admin might not be managing the secret key properly - there's a SECRET_KEY configuration option but it doesn't appear to get used for anything that I can see...

there's a SECRET_KEY configuration option but it doesn't appear to get used for anything that I can see...

Haha, the only reference to the SECRET_KEY is in the config.
So I guess using a single worker or --preload in gunicorn is a possible work-around for now.

Interesting. I did increase workers to 4. I will try the preload setting and report back.

From: Robert Kerr<EMAIL_ADDRESS>Sent: Thursday, 8 August 2019 22:09
To: ngoduykhanh/PowerDNS-Admin
Cc: Alun James; Comment
Subject: Re: [ngoduykhanh/PowerDNS-Admin] CSRF token missing or incorrect (#552)
This looks to be a similar issue: plotly/dash#132https://github.com/plotly/dash/issues/132
Could you see if passing --preload makes it go away? It looks like PowerDNS-Admin might not be managing the secret key properly - there's a SECRET_KEY configuration option but it doesn't appear to get used for anything that I can see...
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ngoduykhanh/PowerDNS-Admin/issues/552?email_source=notifications&email_token=AAKZ4GGVTLG2YHJGA2VNPL3QDSDOXA5CNFSM4IJVBX62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD345AKI#issuecomment-519688233, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAKZ4GGDPFJ2K6MGKUY2M2TQDSDOXANCNFSM4IJVBX6Q.

I have been using the system for a few hours so far and have not had a repeat of the CSRF problem with --preload gunicorn setting.

We did see a reoccurrence of the error when using --preload, but only on the /login URI.

I experienced this issue on the login process as well. However, it only happens when the session is expired.
Looking at the code of Flask-Seasurf, the toke is stored in the session, so when it does the comparison between the user's token and server's token, it won't match if the session has expired.

@ngoduykhanh - just testing an upgrade and can confirm it only happens when the session is expired. It does not happen when I logout

We see this continuously happening as well with Kubernetes deployments:
Error en pod whitedns-powerdns-admin=xxxxxyyyyyzzz

2022-03-29 10:05:51,952] [flask_seasurf.py:282] WARNING - Forbidden (CSRF token missing or incorrect.): /login
[2022-03-29 10:06:24,670] [flask_seasurf.py:282] WARNING - Forbidden (CSRF token missing or incorrect.): /login
[2022-03-29 10:06:31,385] [flask_seasurf.py:282] WARNING - Forbidden (CSRF token missing or incorrect.): /login
[2022-03-29 10:07:02,566] [flask_seasurf.py:282] WARNING - Forbidden (CSRF token missing or incorrect.): /register
[2022-03-29 10:07:11,878] [flask_seasurf.py:282] WARNING - Forbidden (CSRF token missing or incorrect.): /login

Hello,
PR #1124 has fixed the problem on first login. Regarding our policy to clean up this project, we're closing this issue. Feel free to open a new issue later.
Regards
PowerDNS Admin Team
See https://github.com/PowerDNS-Admin/PowerDNS-Admin/issues/1153 for the policy

Sorry to resurrect a dead thread, but a fresh installation, and receiving the CSRF issue. Have tried all of the above with no luck.  Unable to register a new account for admin purposes
Nov 22 10:37:15 pdns01 gunicorn[2590]: [2024-11-22 10:37:15 -0500] [2590] [INFO] Starting gunicorn 20.1.0
Nov 22 10:37:15 pdns01 gunicorn[2590]: [2024-11-22 10:37:15 -0500] [2590] [INFO] Listening at: unix:/run/pdnsadmin/socket (2590)
Nov 22 10:37:15 pdns01 gunicorn[2590]: [2024-11-22 10:37:15 -0500] [2590] [INFO] Using worker: sync
Nov 22 10:37:15 pdns01 gunicorn[2593]: [2024-11-22 10:37:15 -0500] [2593] [INFO] Booting worker with pid: 2593
Nov 22 10:37:42 pdns01 gunicorn[2593]: [2024-11-22 10:37:42,132] [flask_seasurf.py:344] WARNING - Forbidden (CSRF token missing or incorrect.): /register

Service file
[Unit]
Description=PowerDNS-Admin
Requires=pdnsadmin.socket
After=network.target

[Service]
PIDFile=/run/pdnsadmin/pid
User=pdns
Group=pdns
WorkingDirectory=/var/www/html/pdns
ExecStart=/var/www/html/pdns/flask/bin/gunicorn --pid /run/pdnsadmin/pid --bind unix:/run/pdnsadmin/socket 'powerdnsadmin:create_app()' --preload
ExecReload=/bin/kill -s HUP $MAINPID
ExecStop=/bin/kill -s TERM $MAINPID
PrivateTmp=true

[Install]
WantedBy=multi-user.target

Service override file
### Editing /etc/systemd/system/pdnsadmin.service.d/override.conf
### Anything between here and the comment below will become the new contents of the file

[Service]
Environment="FLASK_CONF=../configs/production.py"

### Lines below this comment will be discarded

### /etc/systemd/system/pdnsadmin.service
# [Unit]
# Description=PowerDNS-Admin
# Requires=pdnsadmin.socket
# After=network.target
#
# [Service]
# PIDFile=/run/pdnsadmin/pid
# User=pdns
# Group=pdns
# WorkingDirectory=/var/www/html/pdns
# ExecStart=/var/www/html/pdns/flask/bin/gunicorn --pid /run/pdnsadmin/pid --bind unix:/run/pdnsadmin/socket 'powerdnsadmin:create_app()' --preload
# ExecReload=/bin/kill -s HUP $MAINPID
# ExecStop=/bin/kill -s TERM $MAINPID
# PrivateTmp=true
#
# [Install]
# WantedBy=multi-user.target


Wedge configurationa not working

I ran your damBreak tutorial and it works. I then transformed it to a wedge geometry and it crashes at the first refinement giving the following error:
`PIMPLE: iteration 1
Selected 24 cells for refinement out of 2268.
--> FOAM FATAL ERROR:
cell 580 of level 0 does not seem to have 8 points of equal or lower level
cellPoints:8(2916 2917<PHONE_NUMBER> 2959 629 630)
pointLevels:8{0}
From function hexRef2D::setRefinement(const labelList&, polyTopoChange&)
in file hexRef2D/hexRef2D.C at line 3568.

FOAM aborting
#0  Foam::error::printStack(Foam::Ostream&) at ??:?
#1  Foam::error::abort() at ??:?
#2  Foam::hexRef2D::setRefinement(Foam::List const&, Foam::polyTopoChange&) at ??:?
#3  Foam::dynamicRefine2DFvMesh::refine(Foam::List const&) at ??:?
#4  Foam::dynamicRefine2DFvMesh::update() at ??:?
#5  ? at ??:?
#6  __libc_start_main in /lib/x86_64-linux-gnu/libc.so.6
#7  ? at ??:?
Aborted`
Mind you, Lucas code works withe wedges.

Hi, Thanks for checking it out. I did not need wedge cells in my mesh so I did not try.


Tidal support

This is actually awesome! Thanks for doing this, I can finally have the media keys working again.
Is it possible to add support for Tidal as well?

+1

Hi, I tried to add Tidal support but Tidal doesn't have any apple script API which my app uses - so it's impossible to add Tidal at the moment :( You should try http://beardedspice.github.io , maybe it uses some different method.

Hey :)
Any news on the Tidal issue? I don't get it to work with BeardedSpice and your tool is anyway working much better.

Sorry, still no applescript support :(


Allow GVRGazeCursorController to work from any camera position

A long requested bug-fix from @mwitchwilliams, this PR addresses the problem outlined in #1214. It turns out that Rahul was correct in suggesting that setOrigin must be used on the GVRGazeCursorController when the camera is moved, but unfortunately, the gaze-direction was being calculated incorrectly, which is why problems still persisted for Mitch.
This fix corrects the calculation of the gaze-direction and overrides the getOrigin function to always return the main camera rig position. I believe this is a reasonable change since the gaze direction is always calculated with the main camera's head transform, which implies the GVRGazeController is always intended to be used from the point of view of the main camera.

GearVRf-DCO-1.0-Signed-off-by: Jonathan Reynolds<EMAIL_ADDRESS>

@mwitchwilliams I assume you verified this fix works for you?

I haven't tested it, but I provided Jonathan the test case with the problem.  If it fixed that, then only thing left would be to try different camera positions, which can be set by changing x, y, z inside the x3d file <Viewpoint position-'x, y, z'/> node.


Add lookup suggestion to Kusto best practices.

As written in this documentation: https://learn.microsoft.com/en-us/azure/data-explorer/kusto/query/lookupoperator

In terms of performance, the system by default assumes that the $left table is the larger (facts) table, and the $right table is the smaller (dimensions) table. This is exactly opposite to the assumption used by the join operator.

lookup is much faster than join when left table is much large than right table.

#sign-off


Retaining and accessing global values in module

I was hoping to make an application where the user specifies a python script and then for every event in my application it calls one of the functions in the script. So I was kind of thinking of roughly the following state:
x = 0 # Some variable they want to persist across the run

fn handle_event(data=None):
    x += 1

fn finish():
    return x

So that way they can keep track of state that lasts through the run via globals in the script and then process and return some sort of report at the end. However, it seems when I call a function in the module the globals aren't initialised, so I guess they also won't persist?
I tried something like this:
use pyo3::prelude::*;
use std::fs::read_to_string;

fn main() {
    let script = read_to_string("test.py").unwrap();
    Python::with_gil(move |py| -> PyResult<()> {
         let module = PyModule::from_code(py, &script, "my_module", "my_module")?;
         module.getattr("handle_event")?.call0()?;
         module.getattr("handle_event")?.call0()?;
    }).expect("It failed");
}

And it failed as "local variable x referenced before assignment". I'm just wondering if there's a way to do it how I was planning or if I should just bite the bullet and make the user create some Context class in python that gets passed in each time and returned

I think your script has invalid python syntax - try this?

(accidentally sent previous message before typing the script - have edited it to add it.)

That worked thanks, I'll close this :+1:


Cannot read property '...' of null and Cannot read property 'toString' of null

Search is not working. I get this error in the console:
Cannot read property '...' of null
In the code:
if (typeof actual == 'object'){

   // actual can be null here and this is why the error is thrown

    for (var i = 0; i < searchProperties.length; i++){
        if (actual[searchProperties[i]] && actual[searchProperties[i]].toLowerCase().indexOf(expected.toLowerCase()) >= 0){
            return true;
        }
    }
}

good catch, thanks!