Optimize divide and hystrix plugin to use common plugin template

[x] Optimize divide and hystrix plugin to use common plugin template
[x] Optimize common plugin template to support for handle array

Optimize divide   plugin to use common plugin template.
The hystrix plugin has no change.


style: add metainformation for linux menus

Now, Etcher, on the basis of the .desktop file, is placed in the menus category 'Other'.
With this changes, Etcher will be moved to the menu category for system utilities ('System' category).

Related to #1914 - which implies that this issue has been fixed already?

I am not sure, but electron-builder build deb package, using own meta information from electron-builder.yml, and electron-create-appdir.sh and electron-create-appimage-linux.sh create app image for Linux, and their .desktop generate separetly and with using different infromation (electron-builder get info for .desktop from electron-builder.yml and electron-create-appdir.sh just using  options from script argument and string template inside script), isn't it?
So #1914 add Category for deb building, but not for app image building, because for app image .desktop is generated by electron-create-appdir.sh. And then you use app image on Linux and install .desktop file, Etcher appears in category 'Others'.

Hmmm, I guess this also partially relates to #1625  (which is why the AppImages aren't also being built by electron-builder).
@mmmm1998 It might be better to do something like https://github.com/resin-io/etcher/blob/master/Makefile#L111 inside electron-create-appdir.sh to ensure that it always uses the same category as electron-builder.yml ? *shrug*

Well, @ lurch, maybe i have invented better solution.
I have added new argument to electron-create-appdir.sh, that takes menu categories.
And have added categories variable, using shyaml (like https://github.com/resin-io/etcher/blob/master/Makefile#L111), in Makefile. So now, menu categories pass to electron-create-appdir.sh in the same way as APPLICATION_NAME variable .

Closing as we moved on to https://github.com/balena-io/scripts.git usage (related file here)


update use of argument parser to latest API

motivation: Address warnings
changes: use newer API for retrieving unparsed arguments

@swift-ci smoke test

@natecook1000 this is an attempt to update the new allUnrecognized API but testCommandPluginArgumentsNotSwallowed which worked previously is failing with this, basically seems like the --help and --version arguments get swallowed again. any advise?

@swift-ci Please smoke test

🤔 Although, thinking of it now, using .allUnrecognized to re-parse will probably ditch when it sees the help flags. You'll want to add this to the PluginArguments type so that it doesn't try to parse any help flags:
struct PluginArguments: ParsableCommand {
    static var configuration: CommandConfiguration {
        .init(helpNames: [])
    }
    // etc...
}

@swift-ci smoke test


docs: add distro installation instructions and include IDevID also in sidebar

docs: point to installation guide from RHEL and SLE Micro
docs: include IDevID page also in the sidebar

Thanks for adding RHEL and SL new instructions.


Documenting src/components/SearchBar/Searchbar.jsx to explain functionality

Documenting src/components/SearchBar/Searchbar.jsx to explain how the Searchbar functionality and how the state management flow works in this file to reduce technical debt.

I would like to work on this please!

Great! I assigned you (@jerhe3) to the issue. Have fun working on it!


Release 0.14.2

This release branch is focused on addressing API breaking changes in the manager and providing updated documentation to fit those changes.
Todo:

[ ] Changelog and package.json updates (docs and manager)

Included:

https://github.com/linode/manager/pull/2610 / https://github.com/linode/manager/pull/2612 Fix resize pricing
https://github.com/linode/manager/pull/2591 Rescue accepts devices
https://github.com/linode/manager/pull/2609 Client PAT changes
52232f0203509b42d47d3dd825a7cd8ff183fc09 Closes #2607 Add more information related to Linode types
2378a847b8a066c30bce8d2f8dbead9fce7bc277 Closes #2606 fix render of required property attribute in docs
5b4b2f7dc7f22a8cfb90eb86f006f3eac32e6230 Closes #2588 New blockstorage event names
348ebace1739e9938139e91178220847650806b5 Closes #2589 Test data changes for config changes
764c3d0afcdc8efad30e3a9286612a8c39e8bc7a Closes #2593 Linode object changes
941e0d2050cafba1e4c57dceaebf49c8a9026385 Closes #2577 Docs Changes

4a69f5d34023e5b8c81511507191e80fbc72c779 Closes #2590 (API Changelog 2017-10-4)
4f32d34ec38e75538579f60a23a1beaf800db69e Closes #2521 (docs) add description for linode type id
a9fd051057176792adfcc787ed39ecde36074df2 Closes #2584 (docs) Changed root_ssh_key to authorized_keys
71e738e990f8b2c10e95ce5bcebb470e7f35d1d7 Closes #2496 Added docs for booted option
2f27eab7f02ccfea5d8821dba30ab675cac9e693 Closes #2587 document oauth app
d5214b6bc8c7f0c030db1bcad51982344f4fd83c Closes #2578 (feature/add-public-to-clients)
5ed335725151bce155239622e30ee8033867ed38 Closes #2579 (feature/oauth-apps)
831ef48603736c5ba70a700406071a08923b1b40 Closes #2567 api45Docs
b58f2a78b92a125b5da486a2d3cfe43e36361a6f Closes #2568 jamesottinger/volumeclone
6dbe1f4e4204cbd28f67237b5549a95a1a6ecc1f Closes #2569 yoshizzle/configHelpers
9db3acb49b2706597ef233fff4936aa6e16e04f3 Closes #2573 jamesottinger/diskstatus
feab2381aee5a5cc7d376ac22253ed4be4a2108a Closes #2575 jamesottinger/rescuedevices

Coverage increased (+0.03%) to 71.375% when pulling 52232f0203509b42d47d3dd825a7cd8ff183fc09 on release-14.2 into edd99dc4e1be5ab8190f243c3dbf8b830716255e on master.

Coverage decreased (-0.02%) to 71.329% when pulling 452ea3360c444ca632bfa8edf70c4757509b4eda on release-14.2 into edd99dc4e1be5ab8190f243c3dbf8b830716255e on master.

Coverage increased (+0.008%) to 71.356% when pulling 44fd5f6013db2b8f5157668929c8d06cd6657d8a on release-14.2 into edd99dc4e1be5ab8190f243c3dbf8b830716255e on master.

Coverage decreased (-0.02%) to 71.329% when pulling eb035f160380c39c43518ca435b4168f2d85108f on release-14.2 into edd99dc4e1be5ab8190f243c3dbf8b830716255e on master.

Coverage decreased (-0.02%) to 71.329% when pulling 76eeb3fa8a3cf37f02f9c6711ae95aa19255651c on release-14.2 into edd99dc4e1be5ab8190f243c3dbf8b830716255e on master.


Static installation: dyld: Library not loaded: /usr/local/opt/icu4c/lib/libicui18n.66.dylib

Version: code-server-3.3.1-macos-amd64
OS: Mac OS Mojave 10.14.6

Description: I need to install statically, as I need to install supported versions in my automation environment. However, after my last issue was solved, I tried to install the latest, but I get the following error:
(base) Sujeilys-MBP:code-server-3.3.1-macos-amd64 sfonseca$ ./code-server
dyld: Library not loaded: /usr/local/opt/icu4c/lib/libicui18n.66.dylib
  Referenced from: /Users/sfonseca/Downloads/code-server-3.3.1-macos-amd64/bin/../lib/node
  Reason: image not found
Abort trap: 6

Everything is up to date in my environment.

See https://github.com/cdr/code-server/issues/1640#issuecomment-629966663
Will fix with in #1701 and then should be in the next release in the next few days.

So I just pulled down https://github.com/cdr/code-server/releases/download/v3.4.1/code-server-3.4.1-macos-amd64.tar.gz and am seeing this exact issue still. Are we sure this has been fixed?
/Users/username/code-server-3.4.1-macos-amd64/bin/code-server
dyld: Library not loaded: /usr/local/opt/icu4c/lib/libicui18n.67.dylib
  Referenced from: /Users/username/code-server-3.4.1-macos-amd64/lib/node
  Reason: image not found
Abort trap: 6

Hmm can reproduce, not sure what happened. Reopening.

Found it, had $PATH badly set in one of the CI scripts.

Will be fixed in the next release, sorry 🤦

I'll also just replace the existing macOS release.

Updated. Will still be sure to mention in next release's notes.

OK pulled down the refreshed build and it's working for me now. Thanks!

v3.5.0 with VS Code v1.48.2 has been released, fixing this issue! 🍻


Remove float multiplication logic as it's never needed

:wave: :fist_oncoming:
Unless I'm mistaken, every actions performed on a Dinero leading to a call on calculator.multiply  is always a product of an int and an int or an int and a float.
That would mean the logic supporting the float multiplication is useless.
As a proof you can apply this diff:
@@ -42,7 +42,7 @@ export default function Calculator() {
      * @return {Number}
      */
     multiply(a, b) {
-      return isFloat(a) || isFloat(b) ? floatMultiply(a, b) : a * b
+      return a * b
     },

And all the dinero.js tests will be green.
As this could also be a sign of a lack of exhaustive tests, I've checked and ruled out this possibility.
I suggest removing the logic in question which will also remove the isFloat function now that it'll be unused. Less complexity to maintain.

Hey there 😎 This logic was added while fixing #8 and it's being used.
First off, the Calculator should remain agnostic from Dinero. It may very well be used somewhere else (in another service for example). So there definitely could be calls like Calculator().multiply(1.5, 2.5).
Secondly, you can definitely use a float multiplier when calling Dinero#multiply, just like #8 shows us. In this case, you need to turn the multiplier into an integer to avoid doing floating point math.


Taipower issue new energy cost

Taipower issue new energy cost from 2023/4/1, the program need be modified to fit it.

Modified program to v0.52.1 to fit new Taipower energy cost.


Move "Link Binary With Libraries" before "Copy Bundle Resources"

I use external frameworks that need to be linked and signed.
At distribution(uploading app) time I faced signing issues with these frameworks.
Then found the solution:
had to move "Link Binary With Libraries" before "Copy Bundle Resources" in "Build Phases".

Currently did it manually in Xcode. But didn't find the way to do it in the project spec - YML file.
Could anyone help?

Thanks for the report @mrtsamma. Fixed in #768 👍

Released in 2.13.0


Add CI config for modeldb-backend

Adding a config that contains the CI database password

Closing in favor of #61


Update index.js

Issues:

Whenever you try to run Builder.run(), a folder named --enable-logging is created in the application source directory.
concat results in the same output as above but with a run folder instead.

Sorry for taking so long to get to this. I can't say I've ever seen that happen. Can you clarify?


list() offset

This patch allows archive.list() to accept an opts.offset parameter instead of the default value of 0. This way, I can more easily write a hyperdrive indexer that tails the internal feed without having to implement the machinery already defined in .list().

+1

https://github.com/substack/hyperdrive-index

Great feature. 5.2.0


[DONOTMERGE] Add initial support for ciao external IPs API

Modify ciao-controller to add a stubbed out api for external IPs and pools. Modify ciao-cli to use the API to allow the user to interact with the pools and external IPs resources.

Coverage decreased (-0.2%) to 65.678% when pulling 6ab53c6ccf33df753c438fbfb3919fdcbc4172e3 on kaccardi:topic/api-stub into 497f542435574c0fc0bf09f4d02e8e1e4bf61e64 on 01org:master.

@markdryan we are so close to being able to submit a real PR for all the external IP support that I think I'd rather wait to merge this until we have everything together. But please let me know if you would rather have it earlier and I'll revisit.

@kaccardi No rush.  I can wait.

@kaccardi we can close this right?


add formatter for robot user agents

Not sure if it should also be added to the random user agent method.

Hi, and thanks for your contribution. Crawlers aren't faked that often, so I'll pass on adding this formatter to the core. But you're welcome to publish it as a standalone package, and to add a link to it in Faker's README!


Mousing over uncreated feed source causes issues

 Issue by evansiroky
Wednesday May 23, 2018 at 23:39 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/issues/178

if you have entered some text in the new feed source, mousing over that uncreated feed source causes the text to get cleared
if you click the create button on a feed source with empty text it acts like it gets saved, but it doesn't really

 Comment by landonreed
Friday Nov 16, 2018 at 15:42 GMT

Closing because this will be fixed once #41 is merged.


Recommend to provide Deconstructor Overloads with distinct Number of Parameters and Discards

Hi, in addition to warning about Deconstructor Overloads with the same Number of Parameters I would propose when a conflict happens to provide Overloads with more Parameters and discard them on deconstruction.

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

ID: ec91dfa8-265f-a220-fe14-14047375ee93
Version Independent ID: b2afe72f-5b30-8938-7b64-9f45056df36a
Content: Deconstructing tuples and other types
Content Source: docs/csharp/deconstruct.md
Product: dotnet-csharp
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Closing as a product suggestion.


Update DECIMAL_RE to allow scientific notation in auto inferred schemas

Which issue does this PR close?

Closes #1215.
Rationale for this change
Allow the csv schema inference to handle scientific notation in floating point columns.
What changes are included in this PR?
Update DECIMAL_RE to match on numbers in the form 2e2, -2e-3, 3.4e-5.
Update some csv tests.
Are there any user-facing changes?
Columns that would have been inferred to be utf8 might infer to Float64

@pjmore thank you for this.  i tried out the regex and looks good.

Codecov Report

Merging #1216 (b5a9981) into master (fcd37ee) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1216   +/-   ##
=======================================
  Coverage   82.70%   82.71%
=======================================
  Files         175      175
  Lines       51711    51712    +1
=======================================
+ Hits        42769    42773    +4
+ Misses       8942     8939    -3

Impacted Files
Coverage Δ

arrow/src/csv/reader.rs
88.12% <100.00%> (+0.01%)
:arrow_up:

arrow/src/array/transform/mod.rs
85.56% <0.00%> (ø)

parquet_derive/src/parquet_field.rs
66.43% <0.00%> (+0.22%)
:arrow_up:

arrow/src/datatypes/field.rs
54.10% <0.00%> (+0.30%)
:arrow_up:

arrow/src/datatypes/datatype.rs
66.80% <0.00%> (+0.42%)
:arrow_up:

Continue to review full report at Codecov.

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

@pjmore  it appears there is a lint failure on this PR https://github.com/apache/arrow-rs/runs/4900531577?check_suite_focus=true
Can you please fix it?

@pjmore  please fix lint.
LGTM

Thanks @pjmore  and @liukun4515


chore: add eslint config @typescript-eslint/consistent-type-imports

Description
This pull request changes the following:

add eslint config @typescript-eslint/consistent-type-imports to reduce chances of circular dependencies

Related Issues

Relates to #928

:tada: This PR is included in version 0.32.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Selective command permissions

Pretty low priority, would be cool though.
Some examples
/lp user <user> promote <track>
Currently checks for luckperms.user.promote
Should be changed to check:

luckperms.user.promote.<track>.<group promoting to> (if false return, if true allow, if undef continue)
luckperms.user.promote.<track> (if false return, if true allow, if undef continue)
luckperms.user.promote (if true, allow, otherwise deny)

Should be the same for demote.
/lp user <user> parent add <group>
Currently checks for luckperms.user.parent.add
Should be changed to check:

luckperms.user.parent.add.<group> (if false return, if true allow, if undef continue)
luckperms.user.parent.add (if true, allow, otherwise deny)

Should be the same for permission equivalents.
How to determine whether players can actually use the command
I guess this should be achieved through parsing the command string directly. Instead of doing the check at execution, it will be done ahead of time.

This would be really useful imo.

I would love this to be added, but also with /lp user  parent remove  and as permission luckperms.user.remove.


Provide command line arg to disable remote state bucket updates

Hi,
with v0.37.0 introduced enforced remote state bucket updates our pipelines start to fail. The root cause is the permission model we use. Our CI pipelines which execute terragrunt are not allowed to update the S3 state bucket (which is managed centrally), therefore we would like to use the newly introduced disable_bucket_update = true. Unfortunately, we have many terragrunt.hcl files for all the different pipelines distributed across many repositories which would make updating all the remote state configurations cumbersome.
As an alternative, we would like to use this option as a command-line argument, since our terragrunt execution is configured in a central share pipeline template which we could updated easily.
Looking forward if this feature request would be an option.
Best,
David.

Would it also make sense to expose this via an environment variable ?

Would it also make sense to expose this via an environment variable ?

Absolutely, we build a tooling image which includes Terragrunt; given that we have the same behaviour across all of our repos, we'd be able to set the env var in that image rather than having to update all of our repos


[Introspection] Different order of ENUMs when introspecting Postgres DB

When introspecting the same Postgres DB, the found ENUMs are sometimes returned in different orders which leads to diffs like this:
https://github.com/prisma/introspection-engine-output/commit/40fbdf6235204a209c9b7e34b99009f0135a2889#diff-c4afa267928e1af8a66e5a825f013d50

I did note this once, at any layer in the introspection (SQL response set or rust array that handles it), it could be that the result order is ambiguous.
SQL result set's order is not guaranteed without an order by, probably this is that.

Maybe this: https://github.com/prisma/prisma-engine/blob/a8da337aac05d40da968e657bfdee398f2bee651/libs/sql-schema-describer/src/postgres.rs#L488-L492

Not sure if I should mark this as a bug as this doesn't break the tool. Marking it as repro available for now.


Created a new character sheet for Stars Without Number Revised Edition

Stars Without Number is getting a revised edition, and most of the rules are now Draft Final, meaning they probably won't change before release, and almost all of them are in there. So I cloned the character sheet from Stars Without Number 1st Edition and rewrote it for Revised Edition.

Just want to verify for Roll20 that this is, indeed, for the new upcoming edition.

Thanks for chiming in @kevinsearle


⚠ Bofrost.de has degraded performance

In aeef1a3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 8051 ms

Resolved: Bofrost.de performance has improved in 76dd498 after 6 minutes.


Quickstart "Error: Server closed the connection" for any command

Search before asking

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

Version
kvrocks version 2.6.0 (commit c0451df0)
macOS Ventura 13.4 on Apple M2 Pro Macbook Pro 14-inch 2023
Kvrocks:
Minimal reproduce step
As per the docs:
docker run -it -p 6666:6666 apache/kvrocks

In a separate terminal:
redis-cli -p 6666
> GET foo

What did you expect to see?
Return nil or key value.
What did you see instead?
On docker terminal:
I20231123 11:59:40.362723     1 main.cc:334] kvrocks version 2.6.0 (commit c0451df0)
I20231123 11:59:40.410612     1 storage.cc:332] [storage] Success to load the data from disk: 16 ms
I20231123 11:59:40.413663     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414007     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414085     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414157     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414258     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414347     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414443     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414533     1 worker.cc:72] [worker] Listening on: <IP_ADDRESS>:6666
I20231123 11:59:40.414712     1 worker.cc:515] [worker] Thread #281473666143544 started
I20231123 11:59:40.414757     1 worker.cc:515] [worker] Thread #281473654420792 started
I20231123 11:59:40.414800     1 worker.cc:515] [worker] Thread #281473654244664 started
I20231123 11:59:40.414839     1 worker.cc:515] [worker] Thread #281473654068536 started
I20231123 11:59:40.414875     1 worker.cc:515] [worker] Thread #281473653892408 started
I20231123 11:59:40.414916     1 worker.cc:515] [worker] Thread #281473653716280 started
I20231123 11:59:40.414952     1 worker.cc:515] [worker] Thread #281473632924984 started
I20231123 11:59:40.414999     1 worker.cc:515] [worker] Thread #281473632748856 started
I20231123 11:59:40.415161     1 server.cc:209] [server] Ready to accept connections

In redis-cli terminal:
Error: Server closed the connection
not connected>

Anything Else?
Maybe I need to do some auth first, but the Quickstart guide does not specify this.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Hi @theronic, thanks for your report.
It's a fixed bug in Docker image configuration file, you can workaround by overwriting the bind address:
docker run -it -p 6666:6666 apache/kvrocks:2.6.0 --bind <IP_ADDRESS>

refer: https://github.com/apache/kvrocks/issues/1851

Please update the docs :) If someone tries the Quickstart for the first-time and it doesn't work, immediately you think "oh oh - is this maintained?" which reduces chance of adoption in risk-averse companies.

Sure, thanks for your kind suggestions.


Add support for inline comments

In regard to issue #206 ("Attach comment to an empty object"):
I have an initial modification of attachComments() and addComments() which supports comments in empty object/block.
I've called them inline comments, and they are attached to the node which represents the empty object/block (i.e. ObjectExpression/BlockStatement).

You'll need to add tests before this can be accepted.

Sure, I'll add tests, and will fix my code so the existing tests which are failing will pass.

Whats the status on this.  I have a node module that exports a set of constants and all my inline comments are lost because they exist at the body level (no node to attach to).  I could really use this functionality.

@Cleanshooter Looks like it still needs tests.

@Cleanshooter Add tests, so this can get merged

@tonyobanon You can add tests, too. It's open source after all.

@tonyobanon why would I?  I last posted a comment on this project 3 years ago.  I don't even know what this thing is anymore.  Probably be a good idea to not call out people that haven't been involved with a project for years to do stuff for you in the future... 🙄


Make all fields in response consistent, e.g. in camelCase or underscore style, etc.

Should we consider to make the fileds in response consistent across the whole api-gateway system?

@chenzongxiong yes, with camelCase, in the index.ts file of the gateway we just need to move the underscores from the prefixes.

Is this still relevant @chenzongxiong?

@geastwood this is feature good to have. I'm not sure whether anyone implements this feature or not.
When I extended api-gateway with discord, twitter, discourse, etc, I found inconsitent naming style in the response.
That's why I create this issue. And I think we need to decide it as soon as possible since api-gateway is still not used by mcp.
@bdmason I found rn-app has its own Query in api-gateway. Before we decide which style to choose, we should also ask @franjs . Otherwise, we may break their dependencies.

We are using camelCase for the substrate-api-proxy. (#79)

It seems camelCase should be used.


Add support for SPARQL bodies

This PR adds support for SPARQL bodies:

SPARQL bodies are integrated into bruno-lang v2, but not in v1, is this ok or should v1 also be supported?
Example query:
## All predicates, ordered by number of subjects
SELECT ?predicate (COUNT(DISTINCT ?subject) AS ?count) WHERE {
  ?subject ?predicate ?object .
}
GROUP BY ?predicate
ORDER BY DESC(?count)
LIMIT 10

Fantastic !!
Thank you @dozed for adding this in Bruno!


[feature] Add network layer for a mobile client

User story
As a user, I want to scan a barcode and get all information about this machine
Tech details
Interactions with the backend will require a network layer. It is a common task for all use cases, but it is linked to the first, main user story.
Likely it will be based on coroutines, but it will be good to look through recent publications and conferences to spot new trends and may be even new standards. If you will decide to go with coroutines, avoid to spawn a publisher-subscriber model where it is unnecessary.
Also implement a proxy to mock API request until the backend will be ready.
Add DI framework as a part of this task.

Backend prototype is ready within scope of #10, it mocks search by barcode API and packed into a docker container.

I had an intent to write a test case for a repository across real dev backend, but postponed this task for another time, because it might waste time on unnecessary experiments with new approaches.


Add support for e.g. /usr/lib/bootc/kargs.d or equivalent

(Edited by @cgwalters )
This issue is basically the bootc equivalent of https://github.com/ostreedev/ostree/pull/2217
which has a lot of history.
I think it makes sense to revisit this in a container-native flow.  Having a drop-in dir for kernel arguments that can be applied alongside kernel updates would make sense to me.

Original issue text:
rpm-ostree kargs --append='audit=0' is not working:
podman build -t quay.io/flightctl/flightctl-agent:latest -f packaging/Containerfile.fedora ./
STEP 1/13: FROM quay.io/centos-bootc/fedora-bootc:eln
STEP 2/13: COPY rpmbuild/RPMS/x86_64/flightctl-agent-0.0.1-1.el9.x86_64.rpm /tmp/
--> Using cache d6c77d8351a0e8777c7786cf4218fd16f120ec240b778a00bfa69eb93106db99
--> d6c77d8351a0
STEP 3/13: COPY packaging/flightctl-custom-assets/flightctl_rsa.pub /usr/etc-system/root.keys
--> Using cache e9d47ae7f9756a2a45a3a803f5817599f0a4c8f8d166fade365bfd5e06d3669f
--> e9d47ae7f975
STEP 4/13: RUN touch /etc/ssh/sshd_config.d/30-auth-system.conf;     mkdir -p /usr/etc-system/;     echo 'AuthorizedKeysFile /usr/etc-system/%u.keys' >> /etc/ssh/sshd_config.d/30-auth-system.conf;     chmod 0600 /usr/etc-system/root.keys
--> Using cache f094fb7d73fbc90d709f1d43dffc5b9b44f4739d6fae3b7f44018a48dbcb1cb7
--> f094fb7d73fb
STEP 5/13: VOLUME /var/roothome
--> Using cache 5c199e0456b55e4feb83f4bbd7afa08ef3bb8e360b141f6e203c793b1f15403b
--> 5c199e0456b5
STEP 6/13: ADD packaging/flightctl-custom-assets/config.yaml /etc/flightctl/
--> Using cache 08068ad64e01a17b5deaf1f17a5b111441b081994aa19341c1430f70c44b1935
--> 08068ad64e01
STEP 7/13: ADD packaging/flightctl-custom-assets/ca.crt /etc/flightctl
--> Using cache 6cf50e3e940d43a151d8df1efa69adb4f1ab78be5f278bba851b7bc4791ae076
--> 6cf50e3e940d
STEP 8/13: ADD packaging/flightctl-custom-assets/client-enrollment.* /etc/flightctl/
--> Using cache 467f88db436db8c62bf93775f271861275679c2e5ec8bdbed7f4e10eaf9cbb44
--> 467f88db436d
STEP 9/13: RUN rpm-ostree install -y /tmp/flightctl-agent-0.0.1-1.el9.x86_64.rpm
--> Using cache 51a5903e3b71a19a548282e35e933c153d03470071b02525b622877e3e5b2c69
--> 51a5903e3b71
STEP 10/13: RUN ln -s /usr/lib/systemd/system/podman.socket /usr/lib/systemd/system/multi-user.target.wants/
--> Using cache 886a0421c6f532094be9c8298f451e63e241c59316ad61907e06e6c6187650b6
--> 886a0421c6f5
STEP 11/13: RUN ln -s /usr/lib/systemd/system/flightctl-agent.service /usr/lib/systemd/system/multi-user.target.wants/
--> Using cache e83c1f9965b89846251ea2b1e4df3c969a52c9984e7f42c50e855035ee39fa79
--> e83c1f9965b8
STEP 12/13: RUN rpm-ostree kargs --append=audit=0
error: This system was not booted via libostree.
Currently, most rpm-ostree commands only work on ostree-based host systems.

Error: building at STEP "RUN rpm-ostree kargs --append=audit=0": while running runtime: exit status 1

This is the containerfile:
FROM quay.io/centos-bootc/fedora-bootc:eln

COPY rpmbuild/RPMS/x86_64/flightctl-agent-0.0.1-1.el9.x86_64.rpm /tmp/

COPY packaging/flightctl-custom-assets/flightctl_rsa.pub /usr/etc-system/root.keys
RUN touch /etc/ssh/sshd_config.d/30-auth-system.conf; \
    mkdir -p /usr/etc-system/; \
    echo 'AuthorizedKeysFile /usr/etc-system/%u.keys' >> /etc/ssh/sshd_config.d/30-auth-system.conf; \
    chmod 0600 /usr/etc-system/root.keys
VOLUME /var/roothome

ADD packaging/flightctl-custom-assets/config.yaml /etc/flightctl/
ADD packaging/flightctl-custom-assets/ca.crt /etc/flightctl
ADD packaging/flightctl-custom-assets/client-enrollment.* /etc/flightctl/

RUN rpm-ostree install -y /tmp/flightctl-agent-0.0.1-1.el9.x86_64.rpm
RUN ln -s /usr/lib/systemd/system/podman.socket /usr/lib/systemd/system/multi-user.target.wants/
RUN ln -s /usr/lib/systemd/system/flightctl-agent.service /usr/lib/systemd/system/multi-user.target.wants/
RUN rpm-ostree kargs --append='audit=0'
RUN ostree container commit

kargs support needs to be added. Current workaround (thanks @miabbott ) is here: https://containers.github.io/bootc/install/#operating-system-install-configuration-required

Anything around having rpm-ostree emit errors should probably start as an issue in that repository.  However, it's fine...I reworded and edited this issue.
Ultimately when executed in a container image, rpm-ostree args and tools like grubby would basically write files here.
Note a large difference versus the bootc install configs is that those kargs are only applied at bootc install time today, there's no "day 2 changes".

I know bootc wants to leave the door open for non-ostree backends, but given that kargs are intimately linked to the thing that updates the bootloader, isn't the right place to implement this at the ostree level? (I.e. revive https://github.com/ostreedev/ostree/pull/2217 and then users can put things in those directories in their derived builds.)
I can certainly imagine /usr/lib/bootc/kargs.d being a symlink to /usr/lib/ostree/kargs.d so that users don't have to type ostree.

Yeah, it's a fair point.  Mainly for things like this I just want to ideally do new features in Rust in this project, having it more of a mono-repo style.

As part of whatever kargs support is implemented, we should consider how to support per-architecture kargs.
For example, console settings; see the discussion on the MR from @trgeiger - https://gitlab.com/bootc-org/examples/-/merge_requests/11#note_1783996244

This is also relevant for bootc-image-builder where it is nice to support customizing the kernel commandline


[MINOR] Refactored Pipelines::bulkInsert

Change Logs
Small refactoring of Pipelines::bulkInsert, which reduces code duplication, and makes branching inside this method more explicit.
Impact
No
Risk level (write none, low medium or high below)
No
Documentation Update
No need
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[ ] CI passed

Started to search reason of TestSecondaryIndex @Test Secondary Index With Updates Compaction Clustering Deletes flakiness, https://github.com/apache/hudi/pull/12408. Will retry to restart CI for other PRs.

@hudi-bot run azure


Updated dat file mappings.

Apologies for cramming mutliple things into 1 pull request here.
Added new Rhapsodies zones and a new category to hold them. Put "The Threshold" region into WotG's list and moved its zones into it. Added the mission lists for Rhapsodies. Did both English and Japanese, didn't touch French or German (thought those got removed from retail anyway..) and noticed I forgot to add the Japanese dialog table entry for GM Home, so got that as well.
I auto re-formatted your xml document, it was pretty messy which lead to a parsing error.
You will want to review this with whitespace set off by appending ?w=1 to the url.
If anyone is worried about git blame, you can disable whitespace comparing for that too.

https://github.com/Windower/POLUtils/pull/17/files?w=1
Like so.


updates won't work when fields are set to default values - null as default possible?

When updating items the way as suggested at https://developers.google.com/google-ads/api/docs/client-libs/java/field-masks there seems to be a problem, when the value is set (back) to the default value.
e.g. I have a CustomerConversionGoal that is biddable and I want to deactivate that:
        CustomerConversionGoal goal = CustomerConversionGoal.newBuilder()
                .setResourceName("/ACCOUNTID/customerConversionGoals/DEFAULT~WEBSITE")
                .setBiddable(false)
                .build();
        System.out.println(FieldMasks.allSetFieldsOf(goal));

only returns
paths: "resource_name"

the "biddable" path is missing, because false is the default value. This behavior leads to undesired effects, if you do not remember that default values have to be considered separately every time.
The moment I create the goal I don't have the original item, so FieldMasks.compare(original, goal) is no option for me. As far as I found out the only option for my use case would be to not only set the values for the field but also collect the paths by myself, e.g.
        Set<Integer> changedFieldNumbers = Sets.newHashSet(CustomerConversionGoal.BIDDABLE_FIELD_NUMBER);
        List<String> changedFieldNames = CustomerConversionGoal.getDescriptor().getFields().stream()
                .filter(descriptor -> changedFieldNumbers.contains(descriptor.getNumber()))
                .map(FieldDescriptor::getName)
                .collect(Collectors.toList());
        System.out.println(FieldMask.newBuilder().addAllPaths(changedFieldNames).build());

But that's a lot of code and quite an overhead to not only set the value, but also keep the field (number) in order to create the field mask. I liked the way the old adwords lib worked: if a value different from null was set, this value was transferred, no matter if it was a default value or not.
Wouldn't it be possible to always use objects as fields and set the default to null and if the field value is different from null to add it to the field mask? Getters of the default items could stay null safe, by checking for a null value and returning the default value in case of a null value  instead.

Unfortunately this is a known issue and there is not a good way to automate this. The problem stems from using proto3 to represent the API. Fields by default do not have presence, so there is no difference between a field that is not set and a field set to its default value. The guidelines prohibit us simply making all mutable fields optional.
It looks like we need to update our guidance to move away from FieldMasks.allSetFieldsOf() I'll raise this as a suggestion to update our guidance around constructing mutates.
I'd recommned that you specify the fields manually by doing:
FieldMasks.newBuilder().addPaths("biddable").build()

I was afraid that would be the answer. Too bad. I don't like using potential changing strings, so I think I'll use the static field numbers to determine the paths.
By the way. Maybe it's a wanted behaviour and I don't mind if it stays like it is, so I didn't wanted to open a bug report for it:
I have a Convertion Action defined in manager account A and use this action in client account B. When I query the CustomerConversionGoal of account B, I get resource names like customers/B/customerConversionGoals/DEFAULT~WEBSITE when I try to update the goal with this resource name I get a mutate_error: RESOURCE_NOT_FOUND. e.g. requests kWbuEMzt7OPlL3dS62nwfQ, gm08Zppe7L31nTr-8JZniw.

My guess would be that it's working as intended. @jradcliff IIRC we discussed this recently, can you confirm the behaviour here?

Yes, that's intended if A is the effective conversion account for B. In that case, in queries on B, you'll be able to retrieve the conversion actions, customer goals, and custom goals defined in A, but you will only be able to mutate those objects via account A.
This is reflected in the UI when you look at the Conversions --> Summary page for B. It'll have a message like:
Conversion goals and actions are managed by manager account ([customer ID of A]). To make changes, contact the manager of that account or disconnect from that manager account in your access settings.


Indy: Update the shared site.yaml playbook with Indy

Add the Indy main deploy network playbook to the shared site.yaml playbook.

Closed by #59


Traefik Pilot, Duplicate proxies and Request Duration only

Do you want to request a feature or report a bug?

Bug

What did you do?

Integrated with Pilot via Connect with Traefik Pilot
What did you expect to see?
Integration was succesful and everything seems nice. Integrated to get basic metrics eg: RPS etc but once i enabled it, it shows 2 proxies per Instance and on restart, it doubles up. Also, only Request Duration was showing up no other metric.
What did you see instead?
There should be only single proxy created and every metrics should work.
Output of traefik version: (What version of Traefik are you using?)

Version:      2.3.2
Codename:     picodon
Go version:   go1.15.3
Built:        2020-10-19T18:36:22Z
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
[global]
  checkNewVersion = true
  sendAnonymousUsage = true

[entryPoints]
  [entryPoints.http]
    address = ":12345"
  [entryPoints.traefik]
    address = ":9090"

[log]
  filePath = "/home/gitlab/ArtifactService/traefik/logs/traefik.log"
  format = "json"

[api]
  dashboard = true
  insecure = true

[ping]

[pilot]
    token = "xxx"

[providers]
  [providers.file]
    directory = "/home/gitlab/ArtifactService/traefik/conf"
    watch = true

Hello,
Thanks for using Traefik Pilot.
At startup a UUID is generated for the proxy, when you restart Traefik it's not possible to detect if it's the same proxy or a new proxy, so Traefik Pilot display 2 proxies temporary. A proxy is removed 10 min after the last heartbeat signal, this explains you see 2 proxies during a reboot.
Metrics are sent every 5 minutes by the instances, during the first 5 min Pilot don't have enough information to display the metrics.
https://doc.traefik.io/traefik-pilot/metrics/#metrics-granularity
I hope I have clarified your understanding.


Add TritonTilingExt dialect to support ops not representable in linalg

The TritonTilingExt dialect leverages linalg's TilingInterface to add tiling & fusion support for operators that cannot be represented in linalg Much of the tiling implementation is borrowed from linalg's tiling code. As an example, I have added a barebone version of cumsum that represents the cumulative sum of the inner-most dimension of a tensor. Other backends can then lower the op to lower-level implementation as needed after applying tiling & fusion.

The TritonTilingExt dialect leverages linalg's TilingInterface to add tiling & fusion support for operators that cannot be represented in linalg Much of the tiling implementation is borrowed from linalg's tiling code. As an example, I have added a barebone version of cumsum that represents the cumulative sum of the inner-most dimension of a tensor. Other backends can then lower the op to lower-level implementation as needed after applying tiling & fusion.


Unexpected behavior of UI on resizing file information display

When resizing file information display to the left tree view and list view are slides to the right too fast.
How to reproduce:

move cursor to the line between list view and file information display until left-right arrow appears
while holding left mouse button move cursor to the right

What happens:

file information display shrinks to the left
file view and tree view are shrinks to the right too fast

What should happen:
I am not sure what is intended behavior but there is my assumptions:

file information display shrinks to the left
file view and tree view are shrinks to the right with same speed as file information display

or

file information display shrinks to the left
file view grows to the left

what the heck? … lol … yea that's strange 😄

I'll close this once i tested it again

I seriously did not test this fixed issue for more than 1½ years?


Failed to deploy path that does not exist:

Repro steps:

Action: azureFunctions.deploy
Error type: Error
Error Message: Failed to deploy path that does not exist: c:\Users\francis.ricafort\repos\AzureFunctions\bin\Release\netcoreapp2.1\publish
Version: 0.17.0
OS: win32
Product: Visual Studio Code
Product Version: 1.34.0
Language: en

Call Stack
Object.<anonymous> extension.bundle.js:332:5208
o extension.bundle.js:332:4796

@ricafort can you add repro steps? Did the "publish" task succeed or did you select "Deploy Anyway"?


Check for update if no update then no restart

Is there a way to make it so the Mac will not need a restart if no update is found?

Scott, this is the default behavior if no update is found. You must have some other option like --policy-triggers also enabled.


sklearn.PCA not in API reference docs?

Users would expect to find a section for sklearn in the API docs, but won't find it. I think it's because it's under the private _extrinsics.
Maybe we rearrange the packages? There are configuration options for autodoc, but I think that path would involve revealing _extrinsics in the docs, which I don't think we want.

This will be resolved by #1452.

There's a deeper problem we have with figuring out how to get code that needs optional dependencies to generate documentation correctly, but there's not a page in the api ref that can get people started: https://docs.opendp.org/en/stable/api/python/opendp.extras.sklearn.decomposition.html
Original scope of this issue is satisfied.


chore: add Zagazig university

@nadaaskora I am sorry, the domain fci.zu.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Maximum call stack size exceeded

For some reason your package does not seem to work fine with matb33:collection-hooks. Here is the error it is throwing on startup.
Exception in template helper: RangeError: Maximum call stack size exceeded
    at Tracker.Dependency.depend (http://localhost:3000/packages/tracker.js?7776276660c988c38fed448d8262b925dffb5bc3:415:48)
    at _.extend.userId (http://localhost:3000/packages/ddp-client.js?250b63e6c919c5383a0511ee4efbf42bb70a650f:4143:24)
    at AccountsClient.userId (http://localhost:3000/packages/accounts-base.js?37ee275552e2b0b0f518492f393eb55f395001ce:443:30)
    at Object.Meteor.userId (http://localhost:3000/packages/accounts-base.js?37ee275552e2b0b0f518492f393eb55f395001ce:298:19)
    at Object.signedUpUserIdFunc (http://localhost:3000/packages/brettle_accounts-patch-ui.js?e1612f9934003fba77a820dc4caafe77a2a67c91:48:39)
    at http://localhost:3000/packages/matb33_collection-hooks.js?57812df73cfb573558060484c53c71840fc10dcc:63:40
    at Object.Tracker.nonreactive (http://localhost:3000/packages/tracker.js?7776276660c988c38fed448d8262b925dffb5bc3:615:12)
    at Object.getUserId (http://localhost:3000/packages/matb33_collection-hooks.js?57812df73cfb573558060484c53c71840fc10dcc:62:13)
    at CollectionHooks.extendCollectionInstance._.each.collection.(anonymous function) [as findOne] (http://localhost:3000/packages/matb33_collection-hooks.js?57812df73cfb573558060484c53c71840fc10dcc:151:25)
    at Object.signedUpUserIdFunc (http://localhost:3000/packages/brettle_accounts-patch-ui.js?e1612f9934003fba77a820dc4caafe77a2a67c91:52:27)

Thanks for the report. Can you provide a reproduction as either a github repo or a meteorpad?

Nevermind. I've managed to reproduce it and am working on a fix.

This is fixed in version 0.1.5 which I just published.
@khaigh, this is a more general purpose fix than your fork.

Great, thanks!


Notebook package update version

Update civis-jupyter-notebook version to v1.0.1

waiting for the R package repository to recover so that the image can build... :sigh: Otherwise, looks good to me!


Huge temperatures

Hello, I've tried to use this module but I got huge temperature values, around 1000ºC. Which could it be the reason? Does this module work with single zone MLX90614 sensor?

I found a solution: https://forum.micropython.org/viewtopic.php?f=14&t=2433&p=61684#p61684

I am having this same issue. All reads from the sensor return \xFF\xFF.
My I2C frequency is set to 100kHz and I have the required pull-ups on the data lines.
Any suggestions?

Temp 1037.55 C is indeed caused by read16() returning \xFF\xFF.
\xFF\xFF == 65535
(65535 * 0.02) - 273.15 == 1037.55
Since the sensor temp maxes at 125 and the object temp at 380, perhaps I should make the read16 throw an error instead of returning a seemingly valid >1000 result.
If you are using the GY-906 module, it has 2x 4.7kΩ pull ups on the back, so you shouldn't need extra ones.
You could try dropping the I2C frequency even further to see how it reacts.
Does the module show up in an i2c.scan()?
What microcontroller are you using? and which I2C pins?
Are you powering it with 3V or 5V?
What version of micropython?
Do you know specifically which version of the MLX90614 sensor your module is using? eg. BAA

I am using the sensor from Adafruit (https://www.adafruit.com/product/1747) with the included 4.7k pullups. The specific sensor is the MLX90614ESF-BAA and I'm powering with 3.3V.
I have tried a frequency of 50kHz, but will try lower...
Yes, the sensor shows up fine on i2c.scan() and the correct address (0x5a).
I am using the PyBoard D-series (https://pybd.io/hw/pybd_sfxw.html) with the i2c = I2C('Y', freq=100000) command.
I will have to check version when I'm in the office tomorrow and report back. It's a fresh PyBoard but don't know if they come with latest or legacy versions.

OK, have tried adjusting the frequency all the way down to 10kHz, with no change in the output.
The micropython version is: MicroPython v1.10-445-ga9b1d3ca3
I had a look and the current version is 1.17 from what I can see, so this one seems fairly old. I'll figure out how to update and give it another test.

That did the trick! All seems to be working now.
In case others have the same issue, I found this to be the best explanation of how to update the firmware:
https://learn.sparkfun.com/tutorials/how-to-load-micropython-on-a-microcontroller-board/pyboard
It's a bit old, but still worked for me.
One last gotcha that had me stumped for a bit was that, with the newer firmware, you need to explicitly enable the 3.3V output on the PyBoard to power the I2C peripherals using this command: machine.Pin('EN_3V3').on()

Glad you got it sorted!
Power control in the docs:
https://pybd.io/hw/pybd_sfxw.html#power-control
Theres onboard 5.6k pull ups you can use too:
https://pybd.io/hw/pybd_sfxw.html#i2c-buses

Thanks! Those are both really useful to know.


ODK-X REST Protocol links broken in docs

There are mentions of rest protocol, that when clicked returns dead-end 404 errors.

Hi @chinenyelilian I think there's already a related issue to fix broken links and typo here: https://github.com/odk-x/tool-suite-X/issues/207

Thanks for pointing this out  @iamhks
I guess odk-x/docs/pull/109 closes this.


lmdb.Error: training/.ipynb_checkpoints: No such file or directory

!python train.py --train_data='training' --valid_data='training' --Transformation='TPS' --FeatureExtraction='ResNet' --SequenceModeling='BiLSTM' --Prediction='Attn'

Filtering the images containing characters which are not in opt.character
Filtering the images whose label is longer than opt.batch_max_length
['1']
dataset_root: training
opt.select_data: ['/']
opt.batch_ratio: ['1']
dataset_root:    training	 dataset: /
Traceback (most recent call last):
File "train.py", line 305, in
train(opt)
File "train.py", line 33, in train
train_dataset = Batch_Balanced_Dataset(opt)
File "/content/drive/My Drive/internship/deep_text_recognition_benchmark_master/dataset.py", line 42, in init
_dataset, _dataset_log = hierarchical_dataset(root=opt.train_data, opt=opt, select_data=[selected_d])
File "/content/drive/My Drive/internship/deep_text_recognition_benchmark_master/dataset.py", line 118, in hierarchical_dataset
dataset = LmdbDataset(dirpath, opt)
File "/content/drive/My Drive/internship/deep_text_recognition_benchmark_master/dataset.py", line 135, in init
self.env = lmdb.open(root, max_readers=32, readonly=True, lock=False, readahead=False, meminit=False)
lmdb.Error: training/.ipynb_checkpoints: No such file or directory
but when i do ls -a command in training directory then their it shows .ipynb_checkpoints
please help me out!!!!

You need to first run the create_lmbd_dataset.py with your own custom dataset and then train the model.
See this issue. It's mentioned in the README.

Perhaps there are .ipynb_checkpoints in your training/ folder.
Not visible in colab. Go into Google drive and check then you'll see it.
Remove that and try running it again.

Hi, @Tamoghna-Saha  I'm facing the exact same issue, I have run the creat_lmdp_dataset.py already, and got the outputfile as results.mdp I changed the and made it as training.mdp.
after that I moved the file to have the directory  data_lmdb_release/training/training.mdp
but the same issue still appears
self.env = lmdb.open(root, max_readers=32, readonly=True, lock=False, readahead=False, meminit=False)
lmdb.Error: data_lmdb_release/training/: No such file or directory
I have also looked at this https://github.com/clovaai/deep-text-recognition-benchmark/issues/85 and edit the code accordingly but the issue is still there, I hope you can help in this thanks a lot.

@AbdelsalamHaa did you solved your error?

@AbdelsalamHaa even I am having the same error can you help me if you solved your error?

write this line of code in colab/jupyter whatever you are using and it will return True, meaning that the .ipynb_checkpoint folder is probably hidden.
import os
os.path.isdir('.ipynb_checkpoints')
Once detected use os.remove to delete it

**os.removedirs('.ipynb_checkpoints'), removedirs since, it's a directory we have to remove


Incorrect Jacobian used for orientation IK

The gradient used here is weird,
https://github.com/deepmind/dm_control/blob/master/dm_control/utils/inverse_kinematics.py#L165
quat2vel does axis-angle conversion of the error quaternion q q_t^*, but the jacr here is actually mapping to 2 vec(\dot{q} q*) = (0, \omega). it's unclear what the loss function is, or how the dual (of an undefined loss) is pulled back to id.
Edit:
Ideally, you'd just take the gradient of the vec(err) and use the jacobian for vec(err) (not jacr).
Let me know if this is the intention and I'll send a PR.

The loss, or rather the objective, is actually given on page 10 in the paper cited here:
https://github.com/deepmind/dm_control/blob/master/dm_control/utils/inverse_kinematics.py#L251
In the code you are referring to, there is no gradient calculation of this objective per se. In other words, err does not represent the loss in the usual sense and its gradient is not needed. Instead, the objective is minimized directly with respect to \Delta\theta given the Jacobian and the error (see Eqs.(10),(11)).
Also, this inverse kinematics problem can be formulated in different ways, including the one you are suggesting, but I think the rotational component of the Jacobian is always required if orientations are considered in the loss.

I realize that this is using LM for solving non-linear least squares. The issue is that for the residual err = \theta vec(q q_t^*)/Z, currently being used, J*err is clearly not equal to {\partial err \over \partial q}. It appears that there is no first principle 'objective'+approximation I'm missing, and this is indeed a hack/error.
To reiterate, the Jacobian maps T_q Q -> \mathfrak{so(3)}; this is not the same as {\partial err \over \partial q}.
My only suggestion is to fix this. This can be done by using the correct Jacobian in LM; changing the residual term makes this Jacobian calculation simpler (the axis-angle residual is in any case an approximation of this).

Hi @akssri,
It's not entirely clear to us what you are saying, likely it's a notation thing. In order to avoid further confusion, I'm going to close this comment and as you propose you can send a PR, which should make the discussion much more concrete.
Thanks you so much for your time!


updates documentation around the invalidate content api

What does this PR do?
Fixes inaccurate documentation regarding the create invalidation requests api endpoint.
Which TC components are affected by this PR?

[x] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[ ] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
Build the documentation by running make in the docs/ directory.
If you're missing dependencies, first run sudo -H pip install -r docs/source/requirements.txt
Check all that apply

[ ] This PR includes tests
[x] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2513/
Test PASSed.


Feature(messaging): 15380 - preferred language

Ticket:

15380

Description
Set the preferred language and read it from the DB
Type

[ ] Dependency upgrade
[ ] Bug fix
[x] New feature
[ ] Dev change

Why I am adding everybody here?
Because at the moment I implemented this for the messaging, but in future we will use a shared language across building blocks, using the one stored in the user profile


New Feature setEmail

For projects where users authenticate and verify their email, it would be safer to allow the assignment of email through a "setEmail" function. This prevents users from mistyping their email addresses and avoids all the potential issues that could arise in the future (not seeing invoices, being unable to cancel subscriptions).

Hi @teury,
Sorry for the late response. You can pass a verified email in the purchase method as part of the PurchaseParams object. If you pass it that way, we won't ask the user to input it again.
We do however require the user to verify the email by clicking on the button on the first email sent to continue receiving invoices/emails. This is to make sure the email is valid and doesn't fall into spam traps.
Let us know if that answers your question!

Hi @teury, Going to tentatively close this but please let us know if you have further questions!


Consolidate game-playing bots

We have at least three bots related to two-player games: chess, tictactoe, connectfour.
We should try to consolidate some of the code related to two-player games, so that it's easy to create future two-player games without duplicating logic for inviting players to games, managing message flow, etc., that is really the same for all two-player games, apart from small details that could be delegated to game-specific classes.
The best example we have so far in terms of high level design is the connect-four game.  The connect-four game is nice, too, because it's pretty moderate in terms of programming difficulty and fun to play.
The connect-four game has two flaws that I know of:

It seems to have a model that there's only one active game at any time.
It plays all games privately, instead of using streams.

If you take this ticket, you'll want to discuss ideas on the "integrations" stream.

@amanagr How hard do you think it would be to make the GameHandler for multi-player games?

I am working on it, will update in 2-3 days.

Yeah! #324 (Port Merels) and #325(Playing the game with a range of players) are already merged. Maybe I can work more on it after Summer.


One-Definition-Rule compliancy

If you use the header in multiple compilation-targets, errors due to symbol-duplication occur.
In this PR I marked all functions outside the Classes inline so that cannot happen again.
To verify that it is actually working, I also separated the Tests into multiple files, which link without errors now.
You can test the change of behavior if you try to separate your tests with the current version and try to run them.

Hey, appreciate the PR here! I'll pull it in now. Thanks!


[FEATURE] Include any NTFS/ACL setting module.

Hi,I would like to propose including to DSC Resources modules for managing NTFS Rights on filesystems:
https://github.com/SNikalaichyk/cNtfsAccessControl

Unfortunately the built modules are not likely to use "c" (community) DSC resources, however you can follow the instructions on rebuilding the module to include custom resources. HTH
Also curious where the puppetlabs/acl module falls down? It's quite a powerful module that can really do some very granular permissions.

Just had a look and confirmed that the puppetlabs/acl module has more features than this, but as I mentioned, you can always build custom DSC resources if you choose. Give BUILDING a read. Thanks!

Thanks. I'll try with puppetlabs/ACL. I just wanted to be with DSC for the whole time.

Thanks for the suggestion @marcinbojko, but Puppet is unlikely to include the community resources (those with c prepended) for the same reasons as issue #272. You can always include them in your distribution, or in this case use the puppet module which covers more use cases. I am going to close this now, but feel free to open if issues arise


minor gdb syntax update

There are two things that I wasn't sure how to add:

alias - actually a statement, but wants a statement name behind (maybe ignore that and just add as statement)?
server only usable in GDB scripts (kind of  aprefix, not a command) to not ask yes/no and to not pollute the history, can take "anything" afterwards and should ideally not interfere with the syntax highlighting (example: server source myfile - "source" should be highlighted as it commonly is

Things that are completely unknown to me:

Would it by possible to have a python range (python\n...\nend) highlighted with the python highlighter?
Would it be possible to fold define/if/python?

Forgot to add save:
(gdb) help save
Save breakpoint definitions as a script.

List of save subcommands:

save breakpoints -- Save current breakpoint definitions as a script.
save gdb-index -- Save a gdb-index file.
save tracepoints -- Save current tracepoint definitions as a script.

Have you contacted the maintainer?  His email is in the header.

Have you contacted the maintainer? His email is in the header.

No, but if it works that way I'll do this now :-)

Have you contacted the maintainer? His email is in the header.

No, but if it works that way I'll do this now :-)

Many maintainers do not check github and need to be contacted by email.
--
MESKIMEN'S LAW
There's never time to do it right, but always time to do it over.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

No reply from the maintainer?

No, sadly none, asked by mail.

Let me include it then.  Hmm, some abbreviations can be made more generic.  Please check with the next runtime files update.

There are two things that I wasn't sure how to add:
* `alias` - actually a statement, but wants a statement name behind (maybe ignore that and just add as statement)?

* `server` only usable in GDB scripts (kind of  aprefix, not a command) to not ask yes/no and to not pollute the history, can take "anything" afterwards and should ideally not interfere with the syntax highlighting (example: `server source myfile` - "source" should be highlighted as it commonly is

One approach is to use syn-nextgroup instead of a containing pattern like gdbStatmentContainer (sic).  Something like the following (untested):
syn match gdbLineStart "^" nextgroup=gdbStatement,gdbServerPrefix skipwhite
syn keyword gdbServerPrefix server nextgroup=gdbStatement skipwhite contained

alias is a little more complicated because it allows an option.  You could add the option to the nextgroup sequence or use a match for alias that includes the option handling.

Things that are completely unknown to me:
* Would it by possible to have a python range (`python\n...\nend`) highlighted with the python highlighter?

* Would it be possible to fold define/if/python?

Yes to both.  See :help syn-region, :help syn-fold and :help syn-include
If that doesn't help I can take a look at these when Bram sends out the next update as it sounds like he might be making some modifications.

If that doesn't help I can take a look at these when Bram sends out the next update as it sounds like he might be making some modifications.

@dkearns That helps but it would be good if you can check that then, because I'd rather like to work on #9065.

OK.  Ping me in a couple of weeks if I haven't done anything.


Printing

Related Issue or Background Info
Fixes #426
(Epic in #535?)
Changes Proposed
Adds a popup menu in the test results section, at the moment it will only have the "print results" choice.

When the user selects this choice it will show a preview of the printed report in a modal dialog:

If they click Print it takes them to the system/browser print dialog which allows them to save to PDF or print directly to a printer:

Here is the resulting PDF:
test-result-sample.pdf
Questions (mostly for @hmyers-cms and @aliciabeckett-gov)

UX: We could conceivably skip our own dialog (the first one) and take them directly to the system one, but that would take some additional work. We have to render what we need somewhere in order for the system print dialog to print it, but we might be able to do it off screen. It would take a bit more research to figure that out.

Layout issues: The two-column format for the Patient and Facility causes problems because addresses in particular are often long. Do we need them to be labeled? Things like the names, addresses, and phone numbers are pretty easy to identify without labels and the sample we got in #426 named "Example Antigen Lab Result.pdf" does something similar.

Missing information: I don't see a place to obtain the Specimen ID, Test Name, or Collection Date. We do have a time for the Test Date if you want to print it. The Test Result is just not showing in this shot but it's there. I need some real text for the Next Steps section or we can remove it.

I'm fine with having our own modal pop up for now if it makes things way easier. Eventually we will want to build bulk printing, and this pattern won't really work anymore, but we can address the issue then.
For the layout issues, it looks like it is fine everywhere except the facility address. In theory we may not need those labels but I think they help keep the layout visually consistent and makes it more clear that this is the facility address and not the patient or ordering providers address.
You should be able to take the next steps text from my mockup. Copy/pasted here so you don't need to type it all up from the image:
Antigen tests can return inaccurate or false results and follow up testing may be needed. Continue social distancing and wearing a mask. Contact your healthcare provider to determine if additional testing is needed especially if you experience any of these COVID-19 symptoms.
•  Fever or chills
•  Cough
•  Shortness of breath or difficulty breathing
•  Fatigue
•  Muscle or body aches
•  Headache
•  New loss of taste or smell
•  Sore throat
•  Congestion or runny nose
•  Nausea or vomiting
•  Diarrhea

I'm fine with having our own modal pop up for now if it makes things way easier. Eventually we will want to build bulk printing, and this pattern won't really work anymore, but we can address the issue then.
For the layout issues, it looks like it is fine everywhere except the facility address. In theory we may not need those labels but I think they help keep the layout visually consistent and makes it more clear that this is the facility address and not the patient or ordering providers address.
You should be able to take the next steps text from my mockup. Copy/pasted here so you don't need to type it all up from the image:
Antigen tests can return inaccurate or false results and follow up testing may be needed. Continue social distancing and wearing a mask. Contact your healthcare provider to determine if additional testing is needed especially if you experience any of these COVID-19 symptoms.
•  Fever or chills
•  Cough
•  Shortness of breath or difficulty breathing
•  Fatigue
•  Muscle or body aches
•  Headache
•  New loss of taste or smell
•  Sore throat
•  Congestion or runny nose
•  Nausea or vomiting
•  Diarrhea

A couple styling notes:

The ellipsis icon is a bit small, any chance we can bump that up a bit?
From your screenshot w/ the action menu open, I actually can't tell whether you're about to print the negative or the positive result. The design has the entire row highlighted to make it more obvious which one you've selected the action on. If that's gonna be really tough to implement we can chat about simpler alternatives for the short term and do the full row highlighting once we move to bulk actions

A couple styling notes:

The ellipsis icon is a bit small, any chance we can bump that up a bit?
From your screenshot w/ the action menu open, I actually can't tell whether you're about to print the negative or the positive result. The design has the entire row highlighted to make it more obvious which one you've selected the action on. If that's gonna be really tough to implement we can chat about simpler alternatives for the short term and do the full row highlighting once we move to bulk actions

@hmyers-cms would it help to stack the boxes vertically to get around the address issue? Otherwise it all looks good to me!

@hmyers-cms would it help to stack the boxes vertically to get around the address issue? Otherwise it all looks good to me!

Thanks for the input, @hmyers-cms and @aliciabeckett-gov ! Sorry about these long replies, I just want to make sure I bring things to your attention and address any concerns.
I think the remaining high-priority question is whether the Specimen ID, Test Name, and Collection Date are important enough that we need to address them before releasing this, and whether to remove them if we don't. All we know really is the date the test was reported, which can be back dated. I can't find a place where we describe the test name or specimen.
I've included a shot below of something like what I was proposing in terms of removing the labels. It gives us plenty of room for the address and facility names, which would also be problematic for "Arroyo Hills Convalescent Care". Your call on the formatting, I can keep it as is, just wanted to show an idea of what I meant.

The new Next Steps is there now. I made them two columns because we were getting close to the end of the page and I assume we want to keep this to a single page. To keep it to a page we'd have to buy back space with tighter spacing or smaller font size. I haven't actually seen this printed (I don't have a printer) so I'm not sure what this looks like on paper.
Also, is Next Steps dependent on test result? It seems like the text might be different in those cases.

Good call on making the menu dots bigger, they were smaller than 32px which is the minimum recommended hit target. I've enlarged them but hopefully not comically. I added the hover styles as well. I didn't want to pull in all the work for the test results changes here so I hadn't tried to address all the work to get in sync with the engineering prototype.

Thanks for the input, @hmyers-cms and @aliciabeckett-gov ! Sorry about these long replies, I just want to make sure I bring things to your attention and address any concerns.
I think the remaining high-priority question is whether the Specimen ID, Test Name, and Collection Date are important enough that we need to address them before releasing this, and whether to remove them if we don't. All we know really is the date the test was reported, which can be back dated. I can't find a place where we describe the test name or specimen.
I've included a shot below of something like what I was proposing in terms of removing the labels. It gives us plenty of room for the address and facility names, which would also be problematic for "Arroyo Hills Convalescent Care". Your call on the formatting, I can keep it as is, just wanted to show an idea of what I meant.

The new Next Steps is there now. I made them two columns because we were getting close to the end of the page and I assume we want to keep this to a single page. To keep it to a page we'd have to buy back space with tighter spacing or smaller font size. I haven't actually seen this printed (I don't have a printer) so I'm not sure what this looks like on paper.
Also, is Next Steps dependent on test result? It seems like the text might be different in those cases.

Good call on making the menu dots bigger, they were smaller than 32px which is the minimum recommended hit target. I've enlarged them but hopefully not comically. I added the hover styles as well. I didn't want to pull in all the work for the test results changes here so I hadn't tried to address all the work to get in sync with the engineering prototype.

I can make peace with this layout if it avoids engineering headaches. Can we bold the person and facility name, and also bold the labels for sex, DOB, ordering provider, NPI, and CLIA?
The style updates on the actions menu look great!
The other questions I'll leave to Alicia to answer.

I can make peace with this layout if it avoids engineering headaches. Can we bold the person and facility name, and also bold the labels for sex, DOB, ordering provider, NPI, and CLIA?
The style updates on the actions menu look great!
The other questions I'll leave to Alicia to answer.

Specimen ID, Test Name, and Collection Date:
Specimen ID- we should have this but could ship without it
Test Name- we definitely need this
Collection Date- we don't need this since we don't know it, we only know result date. It should be the same as the test date because it is a rapid test, and if it's not we don't know that. So let's not include Collection date.

Specimen ID, Test Name, and Collection Date:
Specimen ID- we should have this but could ship without it
Test Name- we definitely need this
Collection Date- we don't need this since we don't know it, we only know result date. It should be the same as the test date because it is a rapid test, and if it's not we don't know that. So let's not include Collection date.

Also, I think we do want different guidance if someone tests positive. Let me follow up with Alex at CDC to see if we can get some guidance.

Also, I think we do want different guidance if someone tests positive. Let me follow up with Alex at CDC to see if we can get some guidance.

Closing because this branch is stale and the ticket is blocked on stakeholder feedback. Most of this code has been merged but not visible in the UI.


Fatal error: Connection attempt timeout when connected to Archive Node

{"exn":{"sexp":"<IP_ADDRESS>:3086","tags":[{"tag":"connection attempt timeout","loc":"src/tcp.ml:120:11"}],"backtrace":["Raised at file \"src/error.ml\" (inlined), line 9, characters 14-30","Called from file \"src/error.ml\" (inlined), line 5, characters 2-50","Called from file \"src/tcp.ml\", line 120, characters 11-76","Called from file \"src/job_queue.ml\" (inlined), line 131, characters 2-5","Called from file \"src/job_queue.ml\", line 171, characters 6-47"]},"host":"<IP_ADDRESS>","peer_id":"12D3KooWDYcmHyn8bW3n8MM2VMBkJiDfQ5NgaLGgPbTcjAUyTujE","pid":13,"port":8302}

Full exported logs are here (contains two instances of this crash): https://garethtdavies.keybase.pub/mina/logs/2020-12-18_00-07-58.tar.gz

Another instance of this running 41db20648b566908881a17c8b0248d05f517f25a

I just realised this is related to running the archive node as <IP_ADDRESS>:3086.

Did this make it into the testworld builds as have another instance of this running Commit [DIRTY]226c5d976dad8f4b8f9b1c0614bee3bf85757ad2 on branch HEAD
2020-12-30T09:59:02.000Z 833bf7163f01 explorer_coda.1.sr99jjpqwvokfdne2xugqypom {"timestamp":"2020-12-30 09:59:01.398780Z","level":"Fatal","source":{"module":"Init__Coda_run","location":"File \"src/app/cli/src/init/coda_run.ml\", line 564, characters 2-26"},"message":"Unhandled top-level exception: $exn\nGenerating crash report","metadata":{"exn":{"sexp":"<IP_ADDRESS>:3086","tags":[{"tag":"connection attempt timeout","loc":"src/tcp.ml:120:11"}],"backtrace":["Raised at file \"format.ml\" (inlined), line 242, characters 35-52","Called from file \"format.ml\", line 469, characters 8-33","Called from file \"format.ml\", line 484, characters 6-24"]},"host":"<IP_ADDRESS>","peer_id":"12D3KooWRQdVRep7QKAqWgKr91UHVDsFyZ3aVcdo5G5mu9nsHVLs","pid":12,"port":8302}}

A subtle difference to the original issue in that it is being called from format.ml rather than queue.ml.

It did, yes. I'm debugging this now.

I misread the git log, apparently the cut was made just before the PR for this was merged. I've opened some PRs to fix the weird backtrace, but this should otherwise be fixed.


[ENH] Added useful attributes to extracted shapelets for RDST

Reference Issues/PRs
Fixes #1950

What does this implement/fix? Explain your changes.
Added a list of starting points & class values for the transformer's extracted shapelets.

Does your contribution introduce a new dependency? If yes, which one?
No.

Any other comments?
I'm pretty sure this introduction doesn't harm any functionality across the module. I searched for all uses of RDST and checked for calls to RDSTClassifier.transformer.shapelets

I can't seem to figure out how the error relates to the code I have added

I dmed you about the issue, someone else had it previously and it seems its just a random chance to pass it or not. Worth raising it as an issue. Haven't looked at the algorithm myself yet but id assume it most likely needs to have set a seed.

Just noting that there will be an impact on #1949 when this goes though

I'm fine with the addition of classes, although it might not be obvious to use. For example, consider a shapelet extracted from class 0. It might be more discriminative toward class 2 due to the absence of a common pattern present in class 0 and 1 for example.

Yes I totally agree, that's exactly what made me want to use it in my notebook haha, being able to visualise shapelets and their source class should improve explainability


Tumblr Sharing from Permalink on IE 8

TSG reported, DS verified. Pop up window opens but nothing ever loads.
Windows 7 / IE 8

@sbsmith86 marked as low priority since our IE8 traffic currently is pretty tiny and this is a pretty specific use case (being on IE 8/reporting back/sharing on Tumblr specifically), but if you have some downtime and have a second to check out this bug that'd be great.
Might just be browser stack acting funny?

Closing this actually -- it shows eventually! Just takes a hot 5 mins. Sorry!


update all dependencies to latest semvar

Hey @ZakariaRidouh, thanks for your contribution; I'm sorry that I didn't respond earlier. 👍


fix some tests by adding delay and moving some tests

hi @paulmillr,
By adding some delays and moving some tests, it is a little better.
My tests are ok but some continue to failed.
The failed are not link to my fix. Some failed seems to be random...
The #1024 continue to failed on my mac with or without my changes, but with my changes it is a little better.
The "close" test involve lot of errors but if it move to the end of the file, it is ok...
Can you test this fix on your mac?

thx @paulmillr, if I can help on something let me know.


use single info point that is extendable

Hi @tuarrep , thanks for this amazing plugin, I love it :-)
In this pull request I would like to make a proposal to use hoverpoints that:

extend LatLng so that the point list can be used both in the polyline and in the Elevation
change the ElevationHoverNotification to a InfoHoverNotification, and pass that as hover notification instead of only the coordinate. That would allow to extend that InfoPoint and pass through it also other information like progressive distance, speed and timestamp.

I hope you like the idea.

Thank you for this PR!
I need to review it deeper later, but here some first impressions.
I'm convinced by the InfoPoint class. It's smarter than mine.
I'm not convinced by the name. "Info" is too vague. What about ElevatedLatLng?
I don't understand the benefits of the changes on the notification. If I've well understood, it's just a name change.
BTW if it's really needed, it should be a second PR to split responsibilities and allow reverting if needed.
Thanks again for your interest.

I'm convinced by the InfoPoint class. It's smarter than mine.

ok, that is great.

I'm not convinced by the name. "Info" is too vague. What about ElevatedLatLng?

I chose info (even if I am also not really happy about it), because it can carry anything. But then you are right. In fact downstream apps will anyways extend the ElevatedLatLng if they add more information.
I agree, most probably then your original name would be best to keep?

I don't understand the benefits of the changes on the notification. If I've well understood, it's just a name change.

Yes, I changed the name, and after the discussion above, I think it should stay as it was.
The important thing is that the notification carries the complete point and not only the LatLng information.
I could redo the PR reflecting this discussion. What do you think?

Closing this pull request and opening a new one.

Just let this one open for the new ElevationPoint, remove the notification renaming

To prevent BC we can leave the point named ElevationPoint.

@tuarrep I had made the new pull request in order to not revert changes in this one (I am not sure that it would be so readable). Would you mind give a quick check to the other PR? I think it reflects almost everything you commented.
I would have to add as per your comments:

Add a getter latLng which return the current instance for backward compatibility
Add a doc comment that deprecates it

OK.
Closing in favor of #2


Share blob or base64 encoded image

Hello i want to share an image created with domtoimage library .
The goal here is to :
1- create a to-do list & show it as ion-list [DONE]
2-  then use dom-to-image  to render it as an image [DONE]
3- share it using Social Sharing plugin[.....]

I can't use capacitor because one of the packages i need  is outdated .....
domtoimage :
Save and download a compressed JPEG image
Or
Get a PNG image blob

@midomario
it has been fixed in my forked repo:
https://github.com/EYALIN/community-cordova-plugin-social-sharing
you can install it as an npm package:
cordova plugin add community-cordova-plugin-social-sharing --save


How to send bitmap to platform channel

How to send bitmap data to the platform channel?

The bitmap class has a .buildHeaded method that returns a list of bytes. That list is transferrable via the platform channel.
In order to keep the issues clear, I am closing this for now.


Not a directory: 'hipconfig'

File "/tmp/pip-req-build-qa8r3e9n/setup.py", line 8, in
from torch.utils.cpp_extension import BuildExtension, CppExtension, CUDAExtension, CUDA_HOME
File "/home/zhenhengdong/anaconda3/lib/python3.9/site-packages/torch/utils/cpp_extension.py", line 19, in
from .hipify import hipify_python
File "/home/zhenhengdong/anaconda3/lib/python3.9/site-packages/torch/utils/hipify/hipify_python.py", line 34, in
from .cuda_to_hip_mappings import CUDA_TO_HIP_MAPPINGS
File "/home/zhenhengdong/anaconda3/lib/python3.9/site-packages/torch/utils/hipify/cuda_to_hip_mappings.py", line 34, in
rocm_path = subprocess.check_output(["hipconfig", "--rocmpath"]).decode("utf-8")
File "/home/zhenhengdong/anaconda3/lib/python3.9/subprocess.py", line 424, in check_output
return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
File "/home/zhenhengdong/anaconda3/lib/python3.9/subprocess.py", line 505, in run
with Popen(*popenargs, **kwargs) as process:
File "/home/zhenhengdong/anaconda3/lib/python3.9/subprocess.py", line 951, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/home/zhenhengdong/anaconda3/lib/python3.9/subprocess.py", line 1821, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
NotADirectoryError: [Errno 20] Not a directory: 'hipconfig'

It's not meant to work with HIP (e.g. AMD GPUs).

解决了吗


Markdown Preview

VSCode Version: Code 1.12.1 (f6868fce3eeb16663840eb82123369dec6077a9b, 2017-05-04T21:18:32.269Z)
OS Version: Darwin x64 16.5.0
Extensions:

Extension
Author
Version

csharp
ms-vscode
1.9.0

pdf
tomoki1207
0.2.0

markdown-pdf
yzane
0.1.7

Steps to Reproduce:

The following sentences are described by Markdown.
こんな__文章__だとしっかり__プレビュー__されない
If you have double-byte characters after __aaa__, your character will not be bold.

Thanks. The bug is that 文章 and プレビュー are not bolded in the markdown preview, correct?
I believe this is by design. The common mark spec for strong emphasis requires __ be surrounded by whitespace or punctuators.  This is also why b in this string is not rendered as bold text:
a__b__c

This is also consistent with github's rendering:

a__b__c

Closing as-designed


Added Invoice Related definitions to Glossary

I have now rearranged the terms alphabetically and added links to other definitions as required (https://github.com/killbill/killbill-docs/pull/323/commits/08a93fbc68378d5fc8bb140f3891f48f469a273a).


Fix NullPointerException in primitive double assertions

Check List:

Fixes NullPointerException in AbstractDoubleAssert
Unit tests : YES
Javadoc with a code example (on API only) : NA

The Javadoc for the isEqualTo(double) and isNotEqualTo(double) (with primitive parameter) states that an AssertionError will be thrown if actual == null, however it was actually throwing a NullPointerException when casting actual to primitive in the == and != checks.
Javadoc:
* @throws AssertionError if the actual value is {@code null}.

Reproduction:
assertThat((Double)null).isEqualTo(5.0d);

Threw NPE here when it tried to cast boxed null actual to primitive double
if (expected == actual) return myself;
                ^-- npe

Integrated thanks @goughy000!

No problem - thanks for merge @joel-costigliola :)


Make oidc-client ReactNative-friendly

These changes makes this lib RN-friendly, meaning that it is possible to require it from a RN app (#306). The app must still create navigators that suits RN of course.
One change is that the buffer lib is not included in the minified file(s).
The other change is to not use 'for (.. of ..)' constructs since that seems to be troublesome for JavaScriptCore (https://github.com/facebook/react-native/issues/4587) .

I don't know if saw my code comment? I asked:
should this be keys.length? there's a for loop that it looks like it's missing.

No I didn't see any comment, but you're correct, .length was certainly missing.

Is there an ETA regarding when this branch will be merged ?

@brockallen any idea on a timeline for merging this one into a new release?

Why would you use that from a React Native app - native apps don't use implicit flow - they follow
https://tools.ietf.org/html/rfc8252
see https://github.com/FormidableLabs/react-native-app-auth

That'd work too, was just going off of this pull request.
Thanks for the pointer!


Unable to insert link in richeditor

Winter CMS Build:  1.2.3
Pages Plugin Version: 2.1.2
PHP Version: 8.1.16

Description:
Unable to insert link to selected text
Steps To Reproduce:

Select any static page( The select page must define richeditor varriable)
Input texts to the richtexteditor
First try is succesfull
Second time or later are unable to input thel ink because the input link modal is closed immediately.

Attached the screencast:
https://github.com/wintercms/wn-pages-plugin/assets/2065400/bd36f065-9db9-4564-b907-5c07951eb651

Duplicate of #35, fixed by https://github.com/wintercms/winter/commit/23ba829f19e6062932f61efb390e36c91b8bc497. Will try to release 1.2.4 of Winter soon, but in the meantime you can switch to dev-develop in your composer.json

@bachvtuan is this fixed for you?

@AIC-BV I haven't tried that. I'm still waiting for the next WinterCMS release. Meanwhile, I copied 2 js files from this commit https://github.com/wintercms/winter/commit/23ba829f19e6062932f61efb390e36c91b8bc497 and it works

Oh I see it, I don't use richeditor, I have multilingual richeditor which probably has the same issue, thx for the reply!


Add ModelFactory

All SDK Contribution checklist:
This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please open PR in Draft mode if it is:

Work in progress or not intended to be merged.
Encountering multiple pipeline failures and working on fixes.

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:
Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

[ ] Include updated management metadata.
[ ] Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

If the check fails at the Verify Code Generation step, please ensure:

Do not modify any code in generated folders.
Do not selectively include/remove generated files in the PR.
Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:
  `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup
Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

API changes have been detected in Azure.Monitor.Query. You can review API changes here
API changes
+         public static LogsQueryResult LogsQueryResult(IEnumerable<LogsTable> allTables);
+         public static LogsQueryResult LogsQueryResult(IReadOnlyList<LogsTable> allTables, JsonElement statistics, JsonElement visualization, JsonElement error);
+         public static MetricsQueryResult MetricsQueryResult(string timespan, IEnumerable<MetricResult> metrics);
+         public static MetricsQueryResult MetricsQueryResult(int? cost, string timespan, TimeSpan? granularity, string @namespace, string resourceRegion, IReadOnlyList<MetricResult> metrics);

https://github.com/Azure/azure-sdk-for-net/issues/24128


When making zValidator a shared variable, type inference occasionally fails on the client side

What version of Hono are you using?
4.6.2
What runtime/platform is your app running on?
Cloudflare Workers
What steps can reproduce the bug?
I have some code where an authorization header validator is being shared like this:
export const authHeaderValidator = zValidator(
  'header',
  z.object({
    authorization: z.string(),
  })
)

When calling this, type inference always works on the backend (e.g., const { authorization } = c.req.valid('header') // ok), but on the frontend, I occasionally get an error saying that the header parameter cannot be specified.
Defining the validator inline on the backend resolves the issue, but I’m curious why this error occurs sporadically. Since I want to use this validator across almost all routes, I would prefer to keep it shared. I would appreciate any guidance on how to resolve this issue.
What is the expected behavior?
No response
What do you see instead?
No response
Additional information
No response

Hi @mayone-du
This depends on the structure of your project. For example, whether it is monorepo or not. And this may be related to https://github.com/honojs/hono/issues/2955

Thanks for the reply, seems to be the same problem as 2955, so I'll close it.


String interpolation with input parameters

Some of the benefits of using String interpolation for queries is that one doesn't have to track the number and positions of params making it easier to develop, refactor and maintain. sqltyped is already using interpolation for SQL fragments. It might be worth it to implement both. For instance both Slick and Anorm use $ for parameter interpolation and #$ for "string splicing".
Any arguments for/against?

I don't see any other cons but then the interpolated String is no longer copy-pasteable SQL which can be directly fed to a database (or into some other tool). But I'm not against this feature as it is completely optional to use interpolation if it suits better.
A bit off topic but I had one experimental feature on a roadmap to try out. Related to query composition. As SQL itself does not compose very well I planned to try if bigger SQL statements could be composed from smaller pieces by providing a small set of combinators. Like Compose.and in the example below.
 AND age>?
Compose.and(q1, q2)

For an unmaintained project your responses sure are fast :)
The composition is quite interesting, it looks like a hybrid between a type-safe query DSL and plain SQL. How does this work, I imagine through the parser combinators but I guess not with the JDBC fallback?
One thing that would seem like a bit of a drawback is that it seems you can only compose valid queries, which seems like a bit of duplication if you only want to add a small fragment to a long query. At any rate, you don't often see an approach like that- I haven't.

Haha yes I've responded today perhaps faster than normally :)
I imagined to implement it so that the sql function here would return an AST of the statement. Then and combinator would merge these ASTs and then a new SQL could be generated from the merged AST. To be honest, I'm not sure how easy it would be to implement that and get merging and typing semantically correct, and as you said this approach has its drawbacks as it requires some duplication and boilerplate. This might be a novel idea (hence a strong indicator that it might suck too :) but it would allow some interesting compositions which would not be possible to do with simpler interpolations.
Compose.join(q1, q2, ("id", "person"))

Anyway, I never saw strong enough need for a complex feature like that. It is an interesting idea but could be unnecessarily complex to get right.

SQL syntax has some specifics which make String interpolation a bit less useful: in INSERT statements, where one is usually using a bigger number of parameters and consequently more likely to mix it up, the parameters are not located next to the column names anyway: INSERT INTO table (colname1, colname2, colname3, ...) VALUES (val1, val2, val3, ...).
Besides, it's more difficult to set different options (jdbcOnly, useInputTags, keys) here- granted, useInputTags might not be useful at all anyway, but still.

OK, I've got 2 use cases for interpolation:

For queries- it's much more common to combine whole SQL fragments and create dynamic queries. Also, it's usually necessary to refer to the result type if you want to pass it to other methods. On the other hand, there are often not that many input parameters. For this case the existing String interpolation works well.

For updates and inserts- these more often than not have a simple structure and do not need to be constructed dynamically. However, there could be many input parameters, making it more important to match correctly the params to the placeholders in the query. Here interpolation as params is more useful. This means that the query must be defined in the scope of the variables that are used as interpolation arguments and as a result it's not always possible to refer to the return type of the query, but this is rarely needed for updates/inserts- I haven't found a use case where I need to pass the result of these to another function and it's not a Shapeless record anyway.

1 and 2 are practically impossible to get in the same interpolator anyway. Based on this, I've created another interpolator:
https://github.com/jonifreeman/sqltyped/compare/master...vdichev:master
If you think this makes sense and looks good, I can push it directly to the original repo here.

Looks good to me!
Yes the return values for updates/inserts are sometimes not used at all. There's basically two cases I did use in my own project, 1) to get updated row count (needed quite rarely) 2) to get the autogenerated key values when inserting new rows (quite common need).


Move all saved searches to new kibana

Production

[x] Blink: 401, renamed to Blink: Debug => HTTP 401
[x] Blink: Broadcast Incoming Receipt, renamed Blink: Twilio SMS Broadcast => Status Callback Webhook
[x] Blink: Broadcast Relay expected skip, renamed Blink: Twilio SMS Broadcast => Expected Skip
[x] Blink: Broadcast Relay Retries, renamed Blink: Twilio SMS Broadcast => Relay Retry
[x] Blink: Broadcast Relay Success, renamed Blink: Twilio SMS Broadcast => Relay Success
[x] Blink: c.io profile updates, renamed Blink: Customer.io => Profile Update => Success
[x] Blink: Campaign Signup Posts -> Customer.io, renamed Blink: Customer.io => Track Campaign Signup Post => Success
[x] Blink: Campaign Signups -> Customer.io, renamed Blink: Customer.io => Track Campaign Signup => Success
[x] Blink: Crashes, renamed to Blink: Debug => Crashes
[x] Blink: Customer.io -> Blink, renamed to Blink: Customer.io => Email Activity Webhook
[x] Blink: Customer.io errors, renamed to Blink: Customer.io => All Errors
[x] Blink: Dropped messages, renamed to Blink: Debug => Retry Manager => Limit Reached, Reject Message
[x] Blink: Errors, renamed to Blink: Debug => All Errors
[x] Blink: Gambit Retries, renamed to Blink: MobileCommons => Gambit Campaigns => Retry
[x] Blink: Gambit successful proxy requests, rebame Blink: MobileCommons => Gambit Campaigns => Success
[x] Blink: Missed cio Backfill updates
[x] Blink: MoCo message data, renamed to Blink: MobileCommons => Message Data Webhook
[x] Blink: Production, renamed Blink: Debug => Production
[x] Blink: Validation errors, renamed to Blink: Debug => Validation Errors
[ ] Blink: Warnings

Staging
This potentially could be moved by exporting prod searches:

Blink: Staging
Blink: Staging Broadcast Relay Retries
Blink: Staging Broadcast Relay statuses expected to be skipped (non 'delivered').
Blink: Staging Broadcast Relay Success
Blink: Staging Cio Customer Update
Blink: Staging Incoming Broadcast Reciept
Blink: Staging Warnings

Visualisations

[ ] Blink: Customer.io => All Errors
[ ] Blink: Customer.io => Email Activity Webhook
[ ] Blink: Customer.io => Profile Update => Success
[ ] Blink: Customer.io => Track Campaign Signup => Success
[ ] Blink: Customer.io => Track Campaign Signup Post => Success
[ ] Blink: Debug => All Errors
[ ] Blink: Debug => Crashes
[ ] Blink: Debug => HTTP 401
[ ] Blink: Debug => Production
[ ] Blink: Debug => Retry Manager => Limit Reached, Reject Message
[ ] Blink: Debug => Validation Errors
[ ] Blink: MobileCommons => Gambit Campaigns => Retry
[ ] Blink: MobileCommons => Gambit Campaigns => Success
[ ] Blink: MobileCommons => Message Data Webhook
[ ] Blink: Twilio SMS Broadcast => Expected Skip
[ ] Blink: Twilio SMS Broadcast => Relay Retry
[ ] Blink: Twilio SMS Broadcast => Relay Success
[ ] Blink: Twilio SMS Broadcast => Status Callback Webhook


SLES Update Management Dashboard does not display hotfixes based on classification

Hi,
In SUSE Linux Enterprise Server (SLES) 12 SP4 for SAP Applications environment, I manage updates in cooperation with Update Management.
As a result of scanning that are missing updates, it was confirmed that “Critical” and “Security” are not displayed.
Is this behavior correct?

To isolate, I ran "zypper lp" and "zypper listupdate" commands on SLES.
The results of "zypper lp" command are categorized but not the "zypper listupdate" command.
However, Update Management seems to refer to the result of "zypper listupdate".
I understand that this is not a problem with the documentation, but if you have a suitable feedback location, you would contact me.
zypper_lp.txt
zypper_listupdate.txt

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

ID: db47901d-1664-058d-9407-47e932dc9661
Version Independent ID: e90ec9ee-e7da-4f19-248f-4c825aaa8b9f
Content: Update Management solution in Azure
Content Source: articles/automation/automation-update-management.md
Service: automation
Sub-service: update-management
GitHub Login: @bobbytreed
Microsoft Alias: robreed

@kogesaka Thanks for the comment. We are actively investigating on the issue and will get back to you shortly with an update.

Looks like the feedback is related to update management solution's profound backend configuration. Assigning to @bobbytreed for better analysis / insights.

I have passed the feedback to the engineering team #please-close