Bluetooth: host: bt_conn_auth_cb callbacks are not called when pairing to BLE 4.1 central in BT_SECURITY_L4 mode.

I'n using peripheral_lbs from Nordic nRF Connect SDK on nRF52840dk board.
I have Android phone Samsung Galaxy J1. It has BLE 4.1 only.
I've set secuirity level to BT_SECUIRITY_L4
static void connected(struct bt_conn *conn, uint8_t err)
{
    if (err) {
	printk("Connection failed (err %u)\n", err);
	return;
    }

    printk("Connected\n");

    int sec_result = bt_conn_set_security(conn, BT_SECURITY_L4);
    printk("Security level, err %d\n", sec_result);

    dk_set_led_on(CON_STATUS_LED);
}

On a smartphone I run nRFConnect app and try to pair with DK board.
Pairing fails because BLE 4.1 doesn't support LESC pairing (BT_SECURITY_L4) but
Board shows that the "secuirity_changed" callback called and then immediately "disconnected" callback.
Passkey for d0:1d:c1:85:77:9b (random): 786779
[00:00:12.933,227] [1;31m<err> bt_hci_core: Failed to set required security level
[00:00:12.941,314] [1;31m<err> bt_conn: not connected
[00:00:12.946,960] [1;31m<err> bt_conn: not connected
Security changed: d0:1d:c1:85:77:9b (random) level 3
Disconnected (reason 22)

Expected behavior
I expect bt_conn_auth callbacks like "cancel" or "pairing_failed" to be called but they are not.

Same issue exists also when required BT_SECURITY_L3 and peer does not have MITM flag set.
Can be reproduced with twe shell instances when peer side has not set any authentication callbacks that makes it MITM capabable
Remote:
bt init
bt advertise on identity

Local:
bt init
bt auth all
bt connect <addr>
bt security 3

Original issue can be reproduced with two shell instances, remote with BT_TINYCRYPT_ECC=n
Remote:
bt init
bt auth all
bt advertise on identity

Local:
bt init
bt auth all
bt connect <addr>
bt security 4


Remove duplicate checks for no scenarios/steps found

Duplicate checks are present for No scenarios found in spec and No steps found in scenario in parser and execution packages. We can remove the above checks from the execution.

No scenarios found in spec
https://github.com/getgauge/gauge/blob/7321d9552894c5acb5a0ffcc24b65d06617d56d7/execution/specExecutor.go#L93
https://github.com/getgauge/gauge/blob/7321d9552894c5acb5a0ffcc24b65d06617d56d7/parser/specparser.go#L550

No steps found in scenario
https://github.com/getgauge/gauge/blob/7321d9552894c5acb5a0ffcc24b65d06617d56d7/execution/scenarioExecutor.go#L63
https://github.com/getgauge/gauge/blob/7321d9552894c5acb5a0ffcc24b65d06617d56d7/parser/specparser.go#L554

Duplicate checks have been removed via e573988a8fd8ed193af7b9c5557e87eda1f3e37a. Functional tests are passing.
@sguptatw Can you verify if No scenarios found in spec and No steps found in scenario parse errors are working as expected?

Getting an exception Running from an IDE with no scenario heading
[ParseError] /Users/sswaroop/work/gauge/getgauge-examples/java-maven-selenium/specs/one.spec:1 Spec heading not found => ''
[ParseError] /Users/sswaroop/work/gauge/getgauge-examples/java-maven-selenium/specs/one.spec:7 Scenario should be defined after the spec heading => 'Scenario Heading'
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Starting ChromeDriver 2.32.498537 (cb2f855cbc7b82e20387eaf9a43f6b99b6105061) on port 28846
Only local connections are allowed.
Aug 31, 2017 8:58:55 AM org.openqa.selenium.remote.ProtocolHandshake createSession
INFO: Attempting bi-dialect session, assuming Postel's Law holds true on the remote end
Aug 31, 2017 8:58:56 AM org.openqa.selenium.remote.ProtocolHandshake createSession
INFO: Detected dialect: OSS
runtime error: invalid memory address or nil pointer dereference
goroutine 1 [running]:
runtime/debug.Stack(0x421d695b0, 0x15ec8a0, 0x199ecf0)
	/usr/local/Cellar/go/1.8.3/libexec/src/runtime/debug/stack.go:24 +0x79
github.com/getgauge/gauge/execution.recoverPanic()
	src/github.com/getgauge/gauge/execution/execute.go:128 +0x57
panic(0x15ec8a0, 0x199ecf0)
	/usr/local/Cellar/go/1.8.3/libexec/src/runtime/panic.go:489 +0x2cf
github.com/getgauge/gauge/execution.newSpecExecutor(0x421be0900, 0x1979ae0, 0x421acfbc0, 0x19718c0, 0x421bc81b0, 0x421e31aa0, 0x0, 0x0)
	src/github.com/getgauge/gauge/execution/specExecutor.go:51 +0x9d
github.com/getgauge/gauge/execution.(*simpleExecution).executeSpecs(0x421d1e070, 0x421bfc4e0, 0xed1397970, 0x1b111191, 0x19b10a0)
	src/github.com/getgauge/gauge/execution/simpleExecution.go:129 +0x160
github.com/getgauge/gauge/execution.(*simpleExecution).execute(0x421d1e070)
	src/github.com/getgauge/gauge/execution/simpleExecution.go:91 +0x125
github.com/getgauge/gauge/execution.(*simpleExecution).run(0x421d1e070, 0x196d340)
	src/github.com/getgauge/gauge/execution/simpleExecution.go:70 +0x39
github.com/getgauge/gauge/execution.ExecuteSpecs(0x421c76db0, 0x1, 0x3, 0x0)
	src/github.com/getgauge/gauge/execution/execute.go:123 +0x9b0
github.com/getgauge/gauge/cmd.execute(0x421c76db0, 0x1, 0x3)
	src/github.com/getgauge/gauge/cmd/run.go:113 +0x15a
github.com/getgauge/gauge/cmd.glob..func11(0x19a8ca0, 0x421c76db0, 0x1, 0x3)
	src/github.com/getgauge/gauge/cmd/run.go:54 +0x9c
github.com/spf13/cobra.(*Command).execute(0x19a8ca0, 0x421c76d20, 0x3, 0x3, 0x19a8ca0, 0x421c76d20)
	src/github.com/spf13/cobra/command.go:651 +0x23a
github.com/spf13/cobra.(*Command).ExecuteC(0x19a7760, 0x421c2d580, 0x169ec01, 0x421c05dc0)
	src/github.com/spf13/cobra/command.go:726 +0x339
github.com/getgauge/gauge/cmd.Parse(0x16a446f, 0xa, 0x0)
	src/github.com/getgauge/gauge/cmd/cmd.go:95 +0xa3
main.main()
	src/github.com/getgauge/gauge/gauge.go:88 +0x68

Process finished with exit code 1

should be available in nightly >= 31-08-2017

The following are the various types of parse errors

[x] Step is not defined inside a concept heading
[x] Table doesn’t belong to any step
[ ] Scenario heading is not allowed in concept file
[ ] Concept should have at least one step

[ ] Does not show this parse error even with no step in the concept

This is a concept file with following syntax for each concept.
# Concept <Heading>

# Concept1 <Heading>
* step1 <Heading>
* step2

[x] Concept heading can only have dynamic parameters
[ ] Duplicate concept definition found
Not getting the implementation suggestion for one of the concept steps. What should be the expected behavior?

This is a concept file with following syntax for each concept.
# Concept1 <Heading>
* unimplemented1

# Concept1 <Heading>
* unimplemented2

[x] Teardown should have at least three underscore characters
[x] Table location not specified
[x] Scenario heading should have at least one character
[x] Table header should be not blank
[x] Table header cannot have repeated column values
[x] Multiple spec headings found in the same file
[x] Scenario should be defined after the spec heading
[x] Could not resolve table from file
[ ] Spec does not have any element

[ ] This is throwing null pointer exception while execution

[ ] Spec heading not found
[x] Spec heading should have at least one character
[x] Data table should at least have 1 data row
[x] Dynamic param could not be resolved
[ ] Step should not be blank

java.lang.StringIndexOutOfBoundsException: String index out of range: 1
	at java.lang.String.substring(String.java:1963)
	at com.thoughtworks.gauge.processor.ValidateStepProcessor.getMethodName(ValidateStepProcessor.java:72)
	at com.thoughtworks.gauge.processor.ValidateStepProcessor.validateStep(ValidateStepProcessor.java:53)
	at com.thoughtworks.gauge.processor.ValidateStepProcessor.process(ValidateStepProcessor.java:38)
	at com.thoughtworks.gauge.connection.MessageDispatcher.dispatchMessages(MessageDispatcher.java:91)
	at com.thoughtworks.gauge.GaugeRuntime.dispatchMessages(GaugeRuntime.java:105)
	at com.thoughtworks.gauge.GaugeRuntime.access$100(GaugeRuntime.java:37)
	at com.thoughtworks.gauge.GaugeRuntime$2.run(GaugeRuntime.java:86)
	at java.lang.Thread.run(Thread.java:745)
java.lang.StringIndexOutOfBoundsException: String index out of range: 1
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example.spec:20 Connection closed [<IP_ADDRESS>:50872] cause: EOF => ''
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example.spec:30 write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection => 'Almost all words have vowels <table>'
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example1.spec:3 write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection => 'unimplemented1'
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example1.spec:21 write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection => 'sdf "def"'
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example1.spec:23 write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection => 'sdfsdf'
[ValidationError] /Users/sswaroop/Downloads/untitled 2/specs/example1.spec:31 write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection => 'Almost all words have vowels <table>'
  Failed to initialize suite datastore. Error: write tcp <IP_ADDRESS>:50868-><IP_ADDRESS>:50872: use of closed network connection

[x] Circular reference found in concept
[x] Duplicate scenario definition found in the same specification

For spec should have at least one element try with an empty spec.
Is this parse error expected in any other scenario as well?

Fix will be available in Gauge nightly >= 01-09-2017 and Gauge Java nightly >= 01-09-2017

Except the case when there are duplicate concept definitions - Validation error and implementation suggestion are not shown for one of the concept steps, which is being tracked here.
All the other cases works fine.


searchset from excel

Hello,
i am a new user. I'm working on a navisworks integration with a large database.
I'm trying to set up search sets from an excel file.
Specifically I would need a function that allows me to take the values of an entire column of excell and not change the number of row.
Is it possible ?
Thanks
Marco

Hello Marco, thank you for posting. Have tried with our Excel nodes? If it's so and it didn't work, could you give us more detail about what you want to achieve?


Blank page with enabled javascript

Environment

Technology
Version

Flutter version
2.6.0

Plugin version
5.3.2

Android version
31.0.0

iOS version
15.2

Xcode version
13.0

Hello everyone!
When clicking on the links on the site whatsapp.com blank pages open.
The URL in "onLoadStart" returns an empty string. No errors  in "onLoadError".
If  paste the URL into "initialUrlRequest", then everything works fine.
If disable JavaScript, then everything works fine too. This problem is only with JavaScript enabled.
if open the site and click link in a regular browser, then everything is fine
Thank you for your help!

I'm getting the same problem. For me it's appears both when JavaScript is off and when it's on (Android). Hope this gets fixed. Is there a version of this package that we could roll back onto?


allow eslint v9 in peer dependencies

hi, @hiro08gh
I tried to fix #8
please take a look
ref

https://eslint.org/docs/latest/use/migrate-to-9.0.0
https://typescript-eslint.io/getting-started

did

expand peer deps
upgrade dev deps
migrate this package's own eslint config to flat config format
remove duplicate test case as eslint started to raise error ( https://eslint.org/docs/latest/use/migrate-to-9.0.0#stricter-rule-tester )

Big thanks! I'll check it out after my work🙏

This PR will not be a problem. Thank you! Node 16.x CI is fail, but the test is passing, so we will merge it.  I will remove it with my other PR.


Token classification labels are strings and we don't have the list of labels

Not sure if this is an issue we want to fix or not, putting it here so it's not forgotten. Right now, in token classification datasets, the labels for NER, POS and the likes are typed as Sequence of strings, which is wrong in my opinion. These should be Sequence of ClassLabel or some types that gives easy access to the underlying labels.
The main problem for preprocessing those datasets is that the list of possible labels is not stored inside the Dataset object which makes converting the labels to IDs quite difficult (you either have to know the list of labels in advance or run a full pass through the dataset to get the list of labels, the unique method being useless with the type Sequence[str]).

Indeed. Pinging @stefan-it here if he want to give an expert opinion :)

Related is https://github.com/huggingface/datasets/pull/636

Should definitely be a ClassLabel 👍

Already done.


Adding my data

Adding my data

There are some conflicts, resolve them and update the PR. @GeraAlcantara

Hi I've resolve de conflic, can you help me review it


The provider need to set use_inline_resources

FC059 is turned off for now

At this point, the resource needs to be rewritten to use custom resources. I'm going to go ahead and close this.


Q: How are child (e.g. ffmpeg) processes killed if the main process is killed (by SIGTERM|SIGINT)

Is the signal propagated to the children?
Asking for #871

Signals are propagated to children, just did a test :)


bottom bar typography change

i hope now it shows my name in contributors. I had the wrong email :sob:

Merging this request but in future, any changes to Mendel.css or Mendel.js will have to be done in Mendel project and then be used here

sure! can we add that in some sort of guideline in readme?

Yes, we should! Mendel is a different project which is basically a fork of SimpleMDE customized to our requirement.


Make an exception/rules which app needed for update

Reporting a bug 🐛
Please make sure the version of deb-get you are using is up to date:
deb-get update
deb-get reinstall deb-get

Expected behavior
I have one app called Sunshine that is broke on recent update. So, I need to downgrade it and it will be better if I can make parameters which app needed to be updated to recent version
Actual behavior
recent release on Sunshine broke the application
Steps to reproduce
sudo deb-get upgrade and when upgrade all apps to the recent release.  specific app does not worked
System information
Run the following a paste the output below:
/etc/os-release
PRETTY_NAME="Ubuntu 22.04.1 LTS"
NAME="Ubuntu"
VERSION_ID="22.04"
VERSION="22.04.1 LTS (Jammy Jellyfish)"
VERSION_CODENAME=jammy
ID=ubuntu
ID_LIKE=debian
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
UBUNTU_CODENAME=jammy

/etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=22.04
DISTRIB_CODENAME=jammy
DISTRIB_DESCRIPTION="Ubuntu 22.04.1 LTS"

deb-get version
deb-get version

deb-get update
deb-get update

Hi @dtantono , thanks for letting us know. I suspect a broken upstream release is not the root of the problem.  The definition for sunshine needs tightening up.  If you put the following into /etc/deb-get/99-local.d/sunshine it will override the definition and can confirm if my guess is correct:
DEFVER=1
ARCHS_SUPPORTED="amd64 arm64"
CODENAMES_SUPPORTED="bullseye focal jammy"
SUNSHINE_RELEASE="${UPSTREAM_RELEASE}"
if [ "${UPSTREAM_ID}" == debian ]; then
    SUNSHINE_RELEASE=$UPSTREAM_CODENAME
fi
get_github_releases "LizardByte/Sunshine" # "latest"
if [ "${ACTION}" != "prettylist" ]; then
    URL=$(grep "browser_download_url.*-${UPSTREAM_ID}-${UPSTREAM_RELEASE}-${HOST_ARCH}\.deb\"" "${CACHE_FILE}" |grep -v -e 'nightly-dev' |head -1 |cut -d'"' -f4)
    VERSION_PUBLISHED="$(echo "${URL}" | cut -d'/' -f8 | sed 's|^v||')"
fi
PRETTY_NAME="Sunshine"
WEBSITE="https://docs.lizardbyte.dev/projects/sunshine/"
SUMMARY="Sunshine is a Gamestream host for Moonlight."

You may have been getting the arm or debian .deb ....
If you really need to skip a version you could then edit the above and add the version you don't want to the list of grep -v  expressions - so something like   ... | grep -v -e 'nightly-dev' -e '0.18.1' | ...
Hope that helps.  Please let me know if it works for you as I'll be putting the above in a PR shortly.

@dtantono it would be useful to see a  deb-get show sunshine before and after putting the local override in place.

noted for that. that exception workaround is just skipping certain version and then if newer version comes up, then it will download newer version right? Just for future reference,  Is it possible to exclude an app from the future updates on deb-get update list?

The raison d'etre of deb-get is to follow new upstream releases and keep up-to-date.  If you put an exception workaround in place as above then the next newer release will be picked up.  If you need to downgrade you might need to re-install to get there. If you find a release is broken you can let upstream know and get them to fix it or withdraw it, or make a local interim override as above (and ideally let us know so we can assess if we need to put an exclusion in place).
It is not possible to exclude an app that way (you could add a feature request for that by all means but ..).  I cannot think of a simple way of achieving that off the top of my head, except to say for apps using apt repos or PPAs the standard pinning mechanisms of apt should still work.   A local "exclude these apps from update" list would be a possible but non-trivial enhancement.

Apt pinning : https://help.ubuntu.com/community/PinningHowto
sudo apt-mark hold <package_name>

thank you it works


Thanks for your contribution @bitbrain-za, looks very cool.


ENH: add 'reflect' extension mode

This reflect extension mode was requested in #187.   It is also supported in recent Matlab versions.  It is nearly the same as edge mode symmetric but doesn't duplicate the value at the very edge upon reflection.
I have added a couple of tests to verify the expected behavior.  One of the new tests also confirms proper behavior when the DWT filter length exceeds the signal length for various edge modes.
A table comparing the mode naming conventions of PyWavelets as compared to Matlab's dwtmode and numpy.pad was added to the documentation.
I don't think it is a blocker, but ideally one would also update the Matlab comparisons to include this edge mode too (it is called symw in Matlab), but I am not currently working from a PC with Matlab available.

There are also two anti-symmetric edge modes in Matlab's dwtmode that I believe correspond to 'symmetric' and 'reflect', but with the values negated upon reflection.  I did not add those here, but they would be pretty straightforward as well if there is a use for them.

Current coverage is 86.40% (diff: 100%)

Merging #193 into master will increase coverage by 0.08%

@@             master       #193   diff @@
==========================================
  Files            17         17
  Lines          2200       2221    +21
  Methods           0          0
  Messages          0          0
  Branches        242        242
==========================================
+ Hits           1899       1919    +20
- Misses          249        251     +2
+ Partials         52         51     -1

Powered by Codecov. Last update 108f08f...5cd8292

I updated the precomputed Matlab results.  For simplicity, I just replaced the previous .npz file with an updated one that includes the new mode  (file size increased from ~2MB to ~2.4MB).
The alternative would be to leave the original binary untouched and add a supplementary one for the  reflect case.

updating sounds fine. if the tests still pass that gives enough confidence I'd say. and we can always go back to figure out differences at the Nth decimal if really needed.

Any other comments for this one?  If not, will merge soon.

Looks good to me, nice enhancement.


VIH-9843 upgrade demo to 2 wowza vms

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/VIH-9843
Change description
Increase demo to 2 vms and update readme to include new envs
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ X] No

/azp run

/azp run


fix: remove tailwind preflight from api client app

This should fix the leaky reset styles that were coming out of @scalar/api-client. It's worth doing some testing to make sure removing the Tailwind preflight didn't break anything.

couple of things in the api client in workspace mode, but that can be tackled in another pr! other than button, seems good to go!

Sidebar items shouldn't be hitting the end - not sure if related to this PR

Sidebar items shouldn't be hitting the end - not sure if related to this PR

Looks like it's the same on main, I'll do a fix though


execute migrations

The commands have been split across several files, isolating functionality now, instead of having a single file with everything.
New commands

kallax migrate up --dsn "database source" [--steps N] [--version V] [--all]
kallax migrate down --dsn "database source" [--steps N] [--version V]

Bug fixes

Inverse and non-inverse relationship foreign keys were being swapped.
Sort CREATE TABLEs and DROP TABLEs with a dependency graph to ensure it's valid when running the migration. Order of both types of changes is deterministic. So, order of the migration is as follows:

Sorted create tables.
Sorted drop tables.
Rest of the changes, unordered.

@erizocosmico Does this execute migrations in a transaction? Is that handled by github.com/mattes/migrate ?

That's executed by mattes/migrate. I guess they do, I'll check
On Mon, 8 May 2017 at 12:23, Santiago M. Mola <notifications<EMAIL_ADDRESS>wrote:
@erizocosmicohttps://github.com/erizocosmico Does this execute migrations in a transaction? Is that handled by github.com/mattes/migratehttp://github.com/mattes/migrate ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/src-d/go-kallax/pull/147#issuecomment-299830112, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABQFF3J0aDs22XNVurJrU5W5BvDXgt5oks5r3u0MgaJpZM4NR7fi.

Done @Serabe

@smola they did not, in the end. So I added that.


Upgrade to hawkular-parent 11

I merged in the PR that takes me to 13. So this PR is obsolete. Therefore, I closed it.


Automate release notes and precise Dockerfiles source

Problem:
Currently MD file points at the master blob from this repository, what's not entirely true for every historical release, as the Dockerfiles files change.

Will be handled as #44


[SDK] Fix: Wallet avatar styles in ConnectButton and Modal

i just want my cat back

PR-Codex overview
This PR focuses on updating the styling of the img element in the DetailsModal component to ensure it covers the available space more effectively.
Detailed summary

In the first img element:

Changed -width: "100%" to +width: "100%"
Added +objectFit: "cover"

In the second img element:

Changed -width: iconSize.xxl to +width: "100%"
Changed -height: iconSize.xxl to +height: "100%"
Added +objectFit: "cover"

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5422  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @kien-ngo and the rest of your teammates on  Graphite

Merge activity

Nov 15, 6:08 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


--security-group-ids

I can't seem to get this param to work trying multiple formats:
--security-group-ids '[sg-e985168d,sg-e7f06383,sg-ec1b7e88]'
--security-group-ids 'sg-e985168d,sg-e7f06383,sg-ec1b7e88'
--security-group-ids sg-e985168d,sg-e7f06383,sg-ec1b7e88
This is when making a spot instance request in a VPC subnet.
Every time it defaults back to the default security group. The security group IDs are definately correct and exist in the VPC listed.
Any ideas what I'm doing wrong?
Full command:
knife ec2 server create -x 'centos' -i '~/.ssh/work-aws'  -N 'Staging_Skeleton' -r 'role[qa-staging-skeleton],' -E 'staging' -j '{ "name": "Staging_Skeleton" }' --flavor 'm3.medium' --spot-price '0.1' --ssh-key 'work-aws' --subnet 'subnet-363d446f' --security-group-ids '[sg-e985168d,sg-e7f06383,sg-ec1b7e88]' --ebs-volume-type 'gp2' --ebs-size '40' --iam-profile 'chef-client' --associate-public-ip'
Thanks,
Darryl

Hi @darrylb-github ,
Format for providing the security group ids is --security-group-ids 'sg-e985168d,sg-e7f06383,sg-ec1b7e88' .
Please verify if your spot price request is fulfilled.

@darrylb-github, did the solution work for you?

I had the same issue. Surrounding the Security Group IDs with single quotes worked. It would be great if --help indicated this.
--security-group-ids 'X,Y,Z'

@appanchor  will update the help for this.

We're discussing in https://github.com/chef/knife-ec2/pull/439 switching to --security-group-ids sg-e985168d --security-group-ids sg-e7f06383 --security-group-ids sg-ec1b7e88 or -g sg-e985168d -g sg-e7f06383 -g sg-ec1b7e88 to avoid issues with comma separated values.
We can keep supporting 'x,y,z' too depending on how much code there is to support that.

@btm we have fixed the issues related multiple security group ids here #439. Its still not released.  Should we close this one ?

This has been released in knife-ec2 0.14.0
https://discourse.chef.io/t/knife-ec2-0-14-0-released/9972

As per https://github.com/chef/knife-ec2/issues/406#issuecomment-238940775, in https://github.com/chef/knife-ec2/pull/439, we added a new --security-group-id option that allows you to use the argument multiple times. This behavior is more consistent across our tools and avoids having to parse different string delimiters.


Efficient way to test if a TarInputStream is a valid stream to a Tar archive?

First thing that comes to mind is to simply extracting every TarEntry in the archive. But is there a solution without actually extracting any data? For example - would listing all of the tar entries via TarInputStream.GetNextEntry() be sufficient for validating if the stream is indeed a stream to a valid Tar archive?

Yeah, I think that would be the easiest test. There isn't much else to verify in tar files except for the header CRC and content lengths, both of which are verified by iterating over the entries.

Thanks for the prompt response! Closing this issues as I believe it is resolved.


Call to a member function registerLinkTag() on null

Error
Call to a member function registerLinkTag() on null

fix in https://github.com/Julian-B90/yii2-seomanager/commit/0e25f90b3e7b1f4b75afcdde89c7919e694dbef0


Update tokio and warp crates to v0.3

warp: https://github.com/seanmonstar/warp/issues/641

updated at 0.5.0

updated at 0.5.0


Bug: Black Screen when opening Files

Description
Hello,
I get a black screen for 1-2 seconds when i open Files.
Steps To Reproduce

Open Files

Requirements
I get a black screen for 1-2 seconds when i open Files.
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 24H2
User ID
No response
Log File
debug.log

Thanks for the feedback, This is normal as Files has to load, startups improvements are being worked on but will take time.


Could not resolve URL for hyperlinked relationship using view name "plugins-api:netbox_sync-api:sync-detail"

Plugin version
1.4.5
NetBox version
3.5.3
Python version
3.9
Steps to Reproduce

In Netbox open edit backup job view and press "Save" button or delete backup job.

Expected Behavior
The backup job must be updated or deleted.
Observed Behavior
An exception is thrown:
<class 'django.core.exceptions.ImproperlyConfigured'>

Could not resolve URL for hyperlinked relationship using view name "plugins-api:netbox_sync-api:sync-detail". You may have failed to include the related model in your API, or incorrectly configured the `lookup_field` attribute on this field.

Python version: 3.10.7
NetBox version: 3.5.3

+1
I have installed the plugin today and got the exact same issue. i´d love to have a solution, i really want to try that plugin.

Looks like it might be pulling in from an internal plugin.
I will take a look and push out an update later today.

Hi!
Thank you for the quick fix. I could need some help for updating the plugin, i´m kind of a noob here. I tried to update via pip install --upgrade but it did not change a thing. Could someone maybe point me in the right direction?
Thanks!

Hi! Thank you for the quick fix. I could need some help for updating the plugin, i´m kind of a noob here. I tried to update via pip install --upgrade but it did not change a thing. Could someone maybe point me in the right direction? Thanks!

Hi @HTLBraunau !
Do you activate netbox virtualenv before upgrade plugin?
# source /opt/netbox/venv/bin/activate

Also you can check installed version of the plugin in virtualenv:
# pip show netbox-config-backup

Try upgrade the plugin with this command:
/opt/netbox/venv/bin/pip install -U netbox-config-backup

Yes i did enable the venv! But i didn´t quite get the pip command right. with the flag -U it did work flawlessly. Thank you very much @exec-to!


feat!: Implement Requirement 1.1.3

This PR
Only the last four commits will be merged in with this PR. The others are waiting on #77 and #78.

Breaking: Renamed API.provider=(provider) to API.set_provider(provider, domain: nil) to better adhere to the specification. I decided to move away from the attr_writer naming convention as the domain parameter would be surprising. I would love others' thoughts on this.
Provider instances are now stored inside a provider hash in the config instance, with the default provider using nil as the key. The other option I tried here was to store @default_provider in addition to the hash, but this approach was simpler to the slight detriment of readability.
Refactored API.provider to receive domain as a keyword argument. If no domain is provided, it will return the default provider. This is backward compatible.
Implemented Requirement 1.1.3 from spec version 0.7.0.

Follow-up Tasks
In subsequent PRs, I'll continue on the flag evaluation specifications.
How to test
Open a console and play around with the set_provider method on the API. If no name is provided, the default provider will be replaced. If a name is provided, the provider will be registered using that name. If the same name is provided, the provider will be overridden.

@open-feature/sdk-ruby-maintainers Would love to get this merged in soon as well if anyone has some time for a review!

I updated the PR title to mark it as a breaking change.


CharacterState/Descriptor - add key and document (description) parallels to name

As background to a more andvanced rendering/layout system include new fields that represent name as it should be seen in 1) keys, and 2) taxon descriptions.

@proceps You only need the alternate names for Descriptor?  Not CharacterState?

@mjy I need for both. I put two separate issues.

Closing, will add new issue for where they appear.


Why i can't use this library..

Since I did migrate androidX, I can't use auto-scroll-view-pager library.
i did implementation in build.gradle(app) file. like below code.
implementation ('cn.trinea.android.view.autoscrollviewpager:android-auto-scroll-view-pager:1.1.2') {         exclude module: 'support-v4'     }
also, i did sync project and clean code, rebuild.
but i couldn't find 'cn.trinea.android.view.autoscrollviewpager.AutoScrollViewPager' in my xml.
how can i solve it..?

I've created kotlin version of this library and there AndroidX is supported.
Feel free to use it: https://github.com/pzienowicz/androidx-auto-scroll-view-pager


What are "external sources", exactly?

Would be helpful if the article clearly specified what "external sources" are, exactly.

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

ID: 4bf3b48c-94bb-67d5-25d9-135f4160fd78
Version Independent ID: 14d65fdf-f6d1-4b0c-ada3-b06d04c4b72a
Content: Windows Defender Application Control and .NET Hardening (Windows) - Windows security
Content Source: windows/security/threat-protection/windows-defender-application-control/use-windows-defender-application-control-with-dynamic-code-security.md
Product: m365-security
Technology: windows-sec
GitHub Login: @jsuther1974
Microsoft Alias: dansimp

Thanks @Rvvhub for the feedback, and thanks @MaratMussabekov for submitting an edit to the article.


Follow git-flow for the repo

Hi guys!
Would it be possible for you to follow git-flow for the repo?
It can be a bit confusing, if you've got not-yet-released features on your master branch.
E.g. I wanted to peek into your sample module, and saw completely different PermissionListener being used. And I needed to perform a small investigation to see if I was doing something wrong or if it was just something not yet released.
Thanks

Hi @blipinsk
We have discussed this internally and we've got to the conclusion of keeping the repository as simple as possible. We will use master for small non-breaking changes/fixes and create other branches for bigger features (like adding support for perma-denied permissions & multirequest). The fact that the repository is not following this rule right now is just a mistake we are trying to fix. We are sorry for the inconvenience :(
Thanks for your feedback though, we will think about it for bigger projects.

Fair enough :smirk: , thanks for the response.
Great job on the library by the way, relatively simple, but highly useful. :+1:
I'm hoping you'll manage to implement the callback on the perma-denied permissions into the library. It would be the icing on the cake.


Debian package of GUI app

What has to be done
There should be a debian package of our GUI app, so it can be updated using the standard sudo apt update && sudo apt upgrade mantra. The package should also install all apt-gettable dependencies if needed.
How do you know it has to be done
Currently, docker pull takes about 10 minutes to complete, and is multiple 100s of MBs. The GUI app is in reality much smaller and this much effort and time is not reasonable, especially with our rapid development.  It has been proposed that we could have a debian package.
Code pointers
@paulovap has noted that this should be pretty easy.

Yes, making the metadata do generate the .deb the file is rather easy. The only thing I don't know is how to build the project on CI targeting arm. Maybe @jlebar can help with this

Do you know how to build locally targeting ARM?
The CI scripts are pretty straightforward, they just set up the circleci container and then execute the build.  See .circleci/config.yml.

This probably has knock-on effect on #765 #88 #539 and possibly closes some of them as obsolete.
We probably want one custom image of Raspbian that we still need to script the creation of and store somewhere.
This image should come preconfigured to apt get the latest GUI app.


Feature Request: Add demo link to documentation

What
This is a request to

Create at least one dedicated "demo" wordpress website with this ActivityPub plugin enabled, and
Link to ^ this website somewhere on [a] this repo's README.md file and [b] this plugin's main page

Why
It's really unclear what this plugin does and does not do, but a demo says a thousand words
How

Spin up a new wodpress site, enable this plugin, publish some content
Update the docs to link to it

I added a playground link to the WordPress.org repo, so that you can (at least) check the settings screens directly in your browser.

I added a playground link to the WordPress.org repo, so that you can (at least) check the settings screens directly in your browser.

But did you also enable the feature on your plugin page @pfefferle ?
Because it’s not there, unfortunately.

I can't find any links to any demos. I don't understand what was changed. Can you please paste a link that shows what you mean?

Sorry, WordPress.org changed something and I had to opt-in! Now you can see a "Live Preview" button at the top!

For clarity, @pfefferle updated the contents of this page:

https://wordpress.org/plugins/activitypub/

The image that he uploaded in the last comment shows a screenshot of the above page, showing a button that says Live Preview next to a button that says Download
If you click on the Live Preview button, then it takes you to this page:

https://wordpress.org/plugins/activitypub/?preview=1

Unfortunately, when I click the Live Preview button, I get an empty white page.

The thing is, that there is no change to a site if it activates the plugin! It still looks the same as before. Sure, you can check out the blocks but they simply help with following or shows the followers!
The plugin makes no mastodon out of you blog, it simply enables ActivityPub so that others can follow your blog.

Here is my demo site: https://pfefferle.wordpress.com
And here you can see the blocks: https://pfefferle.wordpress.com/fediverse/

The thing is, that there is no change to a site if it activates the plugin! It still looks the same as before. Sure, you can check out the blocks but they simply help with following or shows the followers!
The plugin makes no mastodon out of your blog, it simply enables ActivityPub so that others can follow your blog.

This is great info, and I think it should be clearly documented on this repo's README.md and the wordpress.org plugin page

Here is my demo site: https://pfefferle.wordpress.com
And here you can see the blocks: https://pfefferle.wordpress.com/fediverse/

This is exactly what I'm looking for, thank you. Can you please add links to these (the second one is especially important) on this repo's README and the wordpress.org plugin page?


reverted whatwg-fetch version 3.0.0 back to 2.0.4

reverted whatwg-fetch version 3.0.0 back to 2.0.4 as it was breaking the UI (nothing loaded)

Adding @Oliver-I


💡 [REQUEST] -Adding  FAQ page

[x] I am from GSSoC'23.
[x] I would like to be assigned this task.

Section
Description

Concise Description
I would like to add FAQ page to the website

Solution Proposed
This page would answer common doubts of the customers(farmers).

Eg:

How does CropForesight determine the best crop recommendation for a specific piece of land?
What factors does CropForesight consider when suggesting a crop for cultivation?
Can CropForesight provide crop recommendations for different types of soil conditions?
How accurate are the crop recommendations provided by CropForesight?
Is CropForesight applicable to all regions and climates?  |

Code of Conduct
By submitting this issue, you agree to follow our Code of Conduct.

[x] I follow the Contributing Guidelines of this project.
[x] I have checked the current issues for duplicate problems.
[x] I have read the readme of this repository.

I have resolved the issue in CropForesight-Frontend # 71. Plz, review and merge.


Scaling Operators

DISCLAIMER: Note that need for scaling operators disputable problem, putting it here, since this is a quite simple solution that is feasible to implement. We do not plant to implement this, it's rather part of discussions and explore the topic, also as part for the options we have on table regarding performance improvements and scaling.
The Problem
Currently we can run only 1 instance of operator. The fundamental problem is that, operator instance receives all the events, so multiple instances would process same events in parallel. In the following will build up and see how to overcome this problem with a simple approach with drawbacks and then taking it a step further  that might handle the event more efficiently  (but added complexity).
Solution - First Step
The solution in short is to deploy instances of operator as a StatefulSet. So every pod / instance (for sake of simplicity let's talk about 3 instances) will receive and ordinal index: 0,1,2. So we will have three pods, with operators running inside will be aware of their ordinal index and the number of replicas (in our case 3).
The trick is that the operator will filter the events. All the events received (either related to the custom resource or dependent resources - mostly from informers) eventually are further propagated as an internal event, with a ResourceID (a name + namespace of the target custom resource ) in it. This ResourceID identifies the target custom resource we want to reconcile. What we can do is hash this resource id, and mod it by the number of replicas:  hash(ResourceID) mod [number of replicas] in our case 3. and only process the events when this number is equal the ordinal index of the pod.
In this way we will receive all the events from K8S API but will just drop the events which are not valid for us based on the approach above, so not process them at all, not even cache them.

This works nicely with a set of predefined number of instances. To have it auto scaled, we would need to reconfigure the instances (mainly the number of replicas input, this is doable quite nicely using config maps, or just watching the StatefulSet inside the operator).
Improvement: Pre-Hashing with Mutating Webhook
In the previous approach we would receive still all the events regarding all the resources. What we could do is use label selectors and already ask for the events related to the particular operator instance. For this we need to provide a mutation webhook that will add the labels to the custom resources. Again we can use the same formula just adding a label:
metadata:
  labels:
    operatorid: [hash(name+namespace?) mod replica-count ]

(Or just in a round-robin fashion?)
This can be done on creation of the resource. Note that on the secondary resource the operator can add the labels when it's creating them, we need the admission controller just for the main custom resource. We have to make sure also that the label
is not removed (but can be updated), that might be done via validation admission controller.
Although for auto-scaling (mainly downscaling) this might mean reprocessing the resources, which is not desriable.

Remarks

If a pod crashes and restarts and will produce the events it just missed, since will receive the same ordinal index.
Note that this is not necessarily about auto scaling, with little tweaks that works too.

Implementation
To implement this we just need filters or label selectors added to the Event Sources, which are already possible.
And have a simple webhook implemented, for the improved case.
(Maybe we can think about a generic filtering mechanism (think of Servlet filter) that is cross cutting through all event sources, will create a separate issue for that)
Notes

Getting the id to the pod https://stackoverflow.com/questions/50750672/is-there-a-way-to-get-ordinal-index-of-a-pod-with-in-kubernetes-statefulset-conf
An analogy could be spotted on the improved variant with sticky sessions
(see also: https://en.wikipedia.org/wiki/Consistent_hashing for autoscaling and redistributing ids)

We talked about sharding approaches on https://github.com/fabric8io/kubernetes-client/issues/3587#issuecomment-1014652601 and the related pr.
Instead of a webhook we were talking about a "coordinating" operator being responsible for assigning indexes to the resources.  The idea was the same, you need something to monitor create / update to ensure the necessary index label is applied.
Some of the additional thoughts:

you'd need a good hashing function wrt the number of statefulset replicas - @manusa suggested using the uid.
you need to transitively set the necessary index label on dependent resources, so you can filter all the way down.  Otherwise each operator replica ends up watching all dependent resources
if you ever needed to modify the replica count (or need to be defensive to any modification), the index can be changed - which would be seen as a delete event by each operator replica informer looking for that label value.  You may not want to process that rehashing as delete/recreate - in that case there would need to be an additional lookup or other mechanism to establish the difference between a changed index, and a full deletion.

thx @shawkins for response!
The hashing is over ResourceID because that is always at hand (note that these are not necessarily only Kubernetes resources), but can be external resources. And this is how the primary resource addressed internally.

Instead of a webhook we were talking about a "coordinating" operator being responsible for assigning indexes to the resources.

This is definitelly the way to go when scaling up or down, on creation probably safer with the mutation hook. Maybe the combination of both is the ideal (and most complex :) )

you need to transitively set the necessary index label on dependent resources, so you can filter all the way down. Otherwise each operator replica ends up watching all dependent resources

Yes, so this can be done in creation time by the operator. The only problem I see is with the auto-scaling, how are the dependent resource re-indexed - this cannot be a generic coordinating operator neither a web-hook, since its not aware of the dependent resources:

One way to do it in general, is just re-index them by the new target operator itself, since the informer won't have them in the cache, read it from API server and change the index directly in the handling operator.
An other is to have it done by the original operator (when coping with the delete event you mentioned) in a two phase manner. Mark the custom resource to stop for processing, change the dependent resource indexes too, then unmark it.

Anyways that is the harder part, but also definitely doable. Just the point is that the main operator should handle the dependent resources, since that is the only component aware of them. This still could be done a quite nicely.

I think we should better describe what is the problem we are trying to solve and where the bottleneck is.
As example, it is not clear how having multiple replicas as described in option 1 would help, it looks to me it can even exacerbate the problems (but I may have not understood well the solution so, pardon me if what I'm saying is wrong) as:

the k8s api server has to notify N more clients with the same set of events
each event need to be de-serialized to a Java object before being processed and discarded
in fact we have just burning some CPU cycles and we put lot of pressure on the garbage collector as well to the CPU caches

The only reasonable solution to me, seems the one based on an label selection however this does not seems very trivial and it is also business sensitive (in some case you can just stop the process, in some others - as mentioned - you may need to wait for the reconcile loop to go to a safe point) so I wonder if this class of problems should be better solved by an external operator.

@lburgazzoli thanks for the feedback!
I agree that using only the part 1 of the approach would need some careful measurement, and definitely does not sound that nice without the part 2.
With the second part made a little addition, IMHO the only little more problem there is with the dependent resources re-indexing, changed the wording, it's actually does not need stop, rather a skip in the other instance, while the dependent resources are changed. Will describe this more in depth later.
I think I did not articulate enough that this issue just like this we have some topic we talk about and create an issue just initiate
a discussion and explore the topic. Added the "exploratory" label also to this issues and added a disclaimer regarding to this. Just to make it also explicit: I don't think we should implement such issues until there is explicit request from users and a strong rational behind it backed by numbers. We recognize that the currently the Kubernetes setup does not scale the internal operators, while capable of handling very large clusters.
This scaling and performance aspects sometimes pops up, more like in discussion level, so again this solution seemed to be simple enough, to take a step further and actually create an issue from it.
The problem this could help is memory utilization, and some other aspects on startup speed of the operator (since we do cache sync on startup). So currently in operators to have fast reconciliation loops we cache all the dependent resources (if possible), in order to limit the hits on the Kubernetes API server and make reconciliation execution faster. Obviously this could lead to large memory consumption especially in case there is a large amount of custom resources and related dependent resources. So basically, what is described is basically sharding, that could helm with this problem, of memory utilization (but also processor)  in a single instance.
(Caching is an interesting topic, it's been an ongoing discussion, how to limit caches or have them in a smart way. Recently controller-runtime also made improvements regarding to this subject. Will probably create issue regarding to that, since we staring to get to the point when it could be feasible to implement. )

I think I did not articulate enough that this issue just like this we have some topic we talk about and create an issue just initiate a discussion and explore the topic. Added the "exploratory" label also to this issues and added a disclaimer regarding to this. Just to make it also explicit: I don't think we should implement such issues until there is explicit request from users and a strong rational behind it backed by numbers. We recognize that the currently the Kubernetes setup does not scale the internal operators, while capable of handling very large clusters.

I  understand, no worries :)

The problem this could help is memory utilization, and some other aspects on startup speed of the operator (since we do cache sync on startup). So currently in operators to have fast reconciliation loops we cache all the dependent resources (if possible), in order to limit the hits on the Kubernetes API server and make reconciliation execution faster. Obviously this could lead to large memory consumption especially in case there is a large amount of custom resources and related dependent resources. So basically, what is described is basically sharding, that could helm with this problem, of memory utilization (but also processor) in a single instance.

I guess here we need to define what we mean by memory utilization because it is true that with sharding you can reduce the memory consumed by a single operator but overall, you may end up requiring more memory as each operator has it s own memory overhead :).
To be clear, I have nothing against this exploration and eventually with an implementation (quite the opposite) but I think that we need to recognize that there's no single solution to solve the problem as often it boils down to the business so maybe, a good start would be to provide some recipes for common scenarios.

I guess here we need to define what we mean by memory utilization because it is true that with sharding you can reduce the memory consumed by a single operator but overall, you may end up requiring more memory as each operator has it s own memory overhead :).

Made it more clear that it is a memory utilization of a single instance of operator, not the overall memory usage.

cc @salaboy   (the discussion on our side regarding scaling mentioned on JBCN :) )


GridItem heigth not calculated correct when resizing a GridItem

Expected Behavior
I expect the item height to be the actual calcalution the documentation describes: The value is a number that is multiplied by rowHeight.
Please describe the behavior you are expecting
When the rowHeight is set to 30px, I expect that the calculation of the rowHeight to be calculate correct. So when the rowHeight is set to 30px and you resize a GridItem to double the size should be 60px.
Current Behavior
Currently the calculation is off. When you double the height it is calculated to: 70px.
Failure Information (for bugs)
It looks like the height calculation is also taking the gap values into account.

but if the height remains 60px, it would not be able to align to the second line.
it will miss the height of gap value.
simple example:

is there any other way to avoid the align question?

but if the height remains 60px, it would not be able to align to the second line. it will miss the height of gap value.
simple example:
is there any other way to avoid the align question?

You are correct @UTing1119 .
I will keep this open while i do some checks about the calculation for grid items having h > 1.

Closing this based on the feedback from @UTing1119 above.


Multiple versions of python fail to install with asdf install

Steps to reproduce
On a brand new install of asdf, I tried to execute cd ~ && asdf install with this ~/.tool-versions file:
python 3.7.3 2.7.16

Expected behavior
python-build 3.7.3 /home/kevin/.asdf/installs/python/3.7.3
Downloading Python-3.7.3.tar.xz...
-> https://www.python.org/ftp/python/3.7.3/Python-3.7.3.tar.xz
Installing Python-3.7.3...
Installed Python-3.7.3 to /home/kevin/.asdf/installs/python/3.7.3
python-build 2.7.16 /home/kevin/.asdf/installs/python/2.7.16
Downloading Python-2.7.16.tar.xz...
-> https://www.python.org/ftp/python/2.7.16/Python-2.7.16.tar.xz
Installing Python-2.7.16...
Installed Python-2.7.16 to /home/kevin/.asdf/installs/python/2.7.16

Actual behavior
python-build 3.7.3 2.7.16 /home/kevin/.asdf/installs/python/3.7.3 2.7.16
python-build: definition not found: 3.7.3 2.7.16

Environment
OS: Arch Linux
asdf version: 0.7.2

https://github.com/asdf-vm/asdf/issues/540 was re-opened.

Definitely an issue with asdf itself.

So, I'll close this one.


Support notary servers from providers other than IBM

We assume that notary server for an image runs on port 4443 on the same host as the registry (https://github.com/IBM/portieris/blob/master/helpers/image/image.go#L118). That assumption isn't valid for other registries - particularly Docker Hub which runs on notary.docker.io:443.
The GetContentTrustURL function should return the correct value for images hosted in either Docker Hub or the IBM registries. Bonus points for it being easy to link to other registry providers' notaries too.

We will also need to look at the ParseToken() oauth helper. There is regex for only matching IBM Cloud tokens.

Would it be possible to allow users to override the configured notary server?
(possibly on a per policy basis?)

@mikebryant To add to what @molepigeon said. This should probably be a separate issue so feel free to raise one with your specific use case.

@austbot Would it be possible to configure the Notary server that Portieris speaks to at the Helm chart or Kubernetes manifest level? That way assuming the Notary server supports Docker Registry auth (as you've mentioned,) then shouldn't it be just as simple as that?

We don't necessarily want to force Portieris to go to a particular Notary server for all images. We can work such a requirement into @mikebryant's suggestion above if you want to go to your Notary for every image, but I think there's a deeper problem here too regarding how to find trust data out of the box.
Trust data for images in different registries might be in different Notary servers, and so Portieris needs to know where to go to get that information on a per image basis. We don't want to require that users roll their own Notaries and manually sign releases (unless they want to of course), because then they'd have to keep an eye out for updates to the images from the original provider and sign the update before Portieris would allow it.
In IBM's case, our setup involves multiple regional registries with different domain names, each with its own accompanying Notary server. Our model is that signatures for images in, say, the European regional registry go into the European Notary, and only that Notary. So Portieris needs to know that for images from the European IBM Registry it goes to the European IBM Notary, and for ones in the US region it goes to the US Notary.
Expanding to Docker Hub, notary.docker.io is where their trust server lives. But that Notary cant be expected to have knowledge of images in, say, the IBM registry.

I think 2 would be quite a lot of work and adds an extra CRD for the user to have to configure.
The solution I'm currently leaning towards is as follows:

Having a fixed set of Notaries we "support". By support I mean we have explicit mappings between the registry and notary URLs. These would be limited to public cloud offerings such as IBM Cloud, Quay, DockerHub, etc.

Add a field in the existing CRD to allow a user to override a notary URL for a given repository. This would give us maximum flexibility and allow us to support any Notary server.

Not sure on default behaviour. I guess we should just reject with a helpful error message asking the user to use one of the supported Notaries or explicitly provide one in the CRD.
I think this covers everyones requirements and would be the simplest to implement/configure.
@molepigeon @zparnold @austbot @mikebryant @ericavonb

Trust data for images in different registries might be in different Notary servers, and so Portieris needs to know where to go to get that information on a per image basis.

This exact problem is what TAP4 of the TUF spec is intended to address: https://github.com/theupdateframework/taps/blob/master/tap4.md
I think this would be best solved by implementing TAP4 in Notary - as you can see from reading the TAP, there are lots of cases that need to be considered when you introduce multiple sources of TUF metadata. The CRD would then essentially take the map.json field data as input.

Sounds great!
On Thu, May 17, 2018 at 2:24 PM Liam White<EMAIL_ADDRESS>wrote:

If there are no objections to my proposal above I'll take a crack at
getting most of the way to the first point by trying to come up with some
kind of sensible mapping/interface.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IBM/portieris/issues/5#issuecomment-390016642, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AE81SAGzBLEFXIoUbWKfTBj90BOhL6Qqks5tzepygaJpZM4Teyo7
.

What about going to DNS for this? So if an image is myreg.io/image/name:bla, Portieris will look up notary.myreg.io and use that if it works? Using a SRV record would also allow different ports on different servers.
We would need to be careful that this doesn't open up new attack vectors via DNS poisoning (force DNSSEC?)

@amouat This should now be supported via https://github.com/IBM/portieris/pull/33 and https://github.com/IBM/portieris/pull/19. Although I haven't had a chance to manually verify.
Let us know if you have any issues with it!


Selectors that contain '-' mask error about unknown native or directive property.

DomElementSchemaRegistry claims to have any property for tag names that contain a '-'. My suspicious is this was included for Polymer interoperability, but this masks very useful errors about incorrect inputs (which can later lead to runtime NoSuchMethod exceptions).
  bool hasProperty(String tagName, String propName) {
    if (!identical(tagName.indexOf("-"), -1)) {
      // can't tell now as we don't know which properties a custom element will get
      // once it is instantiated
      return true;
    } else {
      var elementProperties =
          schema[tagName.toLowerCase()] ?? schema["unknown"];
      return elementProperties[propName] != null;
    }
  }

Is this still necessary, and if it is, could we remove this condition and instead provide a way at compile time to register custom element properties?

I think it's unlikely we will get to this by 5.0 at this point.

Use case, what if Twitter for example adds a custom element, twitter-login:
<twitter-login [email]="myEmail"></twitter-login>

... will need to generate ...
void detectChanges() {
  if (currVal0 != ctx.myEmail) {
    js.setProp(el0, currVal0 = ctx.myEmail);
  }
}

So, based on our discussion:

Current behavior is a huge regression from old versions of AngularDart
Re-enabling validation for foo-bar components would be a breaking change, but ~easy-ish
We don't currently have a way of opting out, i.e. "this is a custom element, ignore it". We could:

<twitter-login @suppressValidation @customElement etc...>

We would like to at minimum change the plugin to use this suppression format instead.

@matanlurey Are you sure custom elements need to support [property]=value syntax? I don't see anything regarding that here: https://developer.mozilla.org/en-US/docs/Web/Web_Components/Using_custom_elements

Do we need to support setting custom element properties via the []-syntax? In @matanlurey's example, wouldn't it just work if we set email as an attribute? At least with normal element attribute and properties work interchangeably.

@leonsenft: It looks like while it is good style to reflect properties as attributes, it's not strictly speaking required - for example even the best practices for Chrome states:

Aim to only accept rich data (objects, arrays) as properties.
...
Do not reflect rich data properties to attributes.


Documentation: replace RawGit with jsDelivr

RawGit, used in README.md, is sunsetting in October 2019; it should be replaced with something else, e.g. jsdelivr; use https://www.jsdelivr.com/rawgit for an easy migration.

Fixed in PR #9.


Drift web related build errors

@simolus3 I tried drift example app tried web related commands. It's working fine, but showing weird errors. Please see attached images.  Did I miss something?

How where those files generated, why is there a copy of your entire dependency graph in web/packages? :D They probably shouldn't be there - and since they're not part of the project they shouldn't be analyzed in the project's context.

@simolus3 thank you for this awesome project. I am new to drift, so not sure which steps to follow. So initially I checked out the official app example and then ran the command flutter build web then flutter run -d chrome. Chrome launched successfully. But showed the loading icon infinitely. when I checked the dev console found that workers were not loaded. Then I followed the official web tutorial executed following commands
flutter pub run build_runner build --delete-conflicting-outputs -o web:build/web/
cp -f build/web/worker.dart.js web/worker.dart.js
files 'shared_worker.dart.js' was generated. After that when I ran flutter run -d chrome everything worked successfully but got the above-mentioned error in vs code. Am I missing something or following the wrong steps?
My second question is when I try to bring drift web to my existing project by copying 'sqlite.wasm', 'worker.dart',tools folder, 'lib/database' folder, and'build.yaml'   I get sqlite related compilation errors . But if I start from the official app project as a template and copy all my existing code to the same , it works fine . I am missing some thing here?  Please help.

Am I missing something or following the wrong steps?

I think the documentation is just not ideal here, I'll update it. We use build_web_compiler to compile the shared web worker to JavaScript, something that needs to happen outside of the regular Flutter compilation toolchain.
You can look at this example for a script (generate_js.sh) that runs the build with -o web:build/web, copies over the shared worker and deletes everything else (so that you don't get those warnings).
An alternative is the setup used in the app example itself, which uses a custom builder (tool/builder.dart and build.yaml) that will automatically copy the worker when running build_runner build (no -o flag is necessary here at all).

I get sqlite related compilation errors

Could you post them? Be aware that you need conditional imports like these since we can't import the regular sqlite3 on the web at all.

@simolus3 I created a new project and followed above said steps and executed generate_js.sh as an extra step, then ran flutter -d chrome which launched successfully without any compilation error. Thank you for the help.

@simolus3 official app example doesn't work in safari. Please see attached pics.

@simolus3 above-mentioned issue, can you cross check?

I can reproduce the problem. This only happens in workers in Safari. It seems like the problem is using the http client to load sqlite3, which uses XMLHttpRequests. Those generally aren't available in workers, but it looks like the request is still made. However, we then receive XMLHttpProgressEvents, even though that class isn't defined in the global scope for workers (which is pretty weird, but again we're not supposed to use that API in workers).
I'll take a look if using the fetch API could fix this.

Seems like using fetch fixes the problem - I've made those changes on the wasm-improvements branch which fixes the problem in Safari for me.
I've also opened a Dart SDK issue for this (https://github.com/dart-lang/sdk/issues/51918), although I guess this is a Safari problem that they won't be able to do much about. There should also be a way to do this without requiring a new dependency, I'll take a look at making these changes as well.

@simolus3 Thank you for the reply. I checked out wasm-improvements branch, ran flutter pub get then flutter run -d chrome I get a compilation error. Please see the attached image. I removed .pub-cache/hosted/pub.dev/drift-2.6.0 folder and tried again, getting the same error. Please let me know your thoughts.

I've used the branch to work on some improvements that will make drift much easier to use with workers soon. That introduced some new APIs used in the example that haven't been published yet.
Regarding the original problem of using fetch, you can see 697da450bce51c63ae8c6326c85cf9c4adab097d for the original fix that doesn't require the new version yet.
The good news is that all of this will become much easier soon :D

@simolus3 worked fine after merging 697da45 changes.  Thank you.


Fix std::result_of removed in C++20

Fix for #530 based on comment by @arolson101.
I used __cpp_lib_is_invocable feature test macro to detect the presence of std::invoke_result.

VS2015 and 2017 failed to build... close for now...


Production Release - December 18 / 2024

These are the stories that should be included in this PR:

#2277
#1712
#1412
#2171
#1943
#2279
#2265
#2028
#2089
#2252
#2214
#2272
#2253

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request] show tasks that are Due for a given Day In Weekplanner

Id like to see not only the already scheduled tasks but also the tasks that are Due in the weekplanner maybe just at the top (like the normal unscheduled tasks in dayplanner)
Sorry for posting it as a bug dont know how to change the lable now ....

I dont know if it would be possible but maybe one would even be able to create multiple views where dayplanner shows not only tasks scheduled for a given day but also due tasks and so on ? Should also be doable withthe dataview filters that are already in place


JSON Output Example Gives Extra Answers

The bug
The program produces extra answers even though not specified. Also, the program does not follow the keys that I have specified.
To Reproduce
Give a full working code snippet that can be pasted into a notebook cell or python file. Make sure to include the LLM load step so we know which model you are using.
guidance.llm = guidance.llms.transformers.Vicuna("eachadea/vicuna-7b-1.1")
os = ["windows", "linux", "mac"]

# define the prompt
program = guidance("""Generate a character profile for each intern in JSON format.
```json
{
    "name": "{{name}}",
    "strengths": {{gen 'strengths'}},
    "weaknesses": {{gen 'weaknesses'}},
    "likes": {{gen 'likes'}},
    "dislikes": {{gen 'dislikes'}},
    "favourite operating system": "{{select 'fav_os' options=os}}",
    "favourite food": [{{#geneach 'food' num_iterations=3}}
        "{{gen 'this'}}",{{/geneach}}],
}```""")
sy = program(name = "sy", os=os)

Results:

System info (please complete the following information):

OS (Ubuntu)
Guidance Version (guidance.__version__): tried 0.0.61 and 0.0.63, both has the same issue

change "strengths": {{gen 'strengths'}}, to "strengths": {{gen 'strengths' stop=","}},

@aniquetahir Can you explain why {{gen 'strengths' stop=","}} would prevent the outer structure from being repeated?  As I read that, it suggests that it should generate a single strength in an array (which would be the same as num_iterations=3, i imagine)
I image its early and there aren't formal docs, but if there were, I'd love to read 'em.  (I checked the README and https://guidance.readthedocs.io/en/latest/api.html , which mention 'stop' but don't provide a description of what it does that fits with your suggestion.)

Right. I overlooked that its an array.
"strengths": [{{gen 'strengths' stop="],"}}], might work. Basically the problem with the original generation is that it doesn't know when to stop so it produces the rest of the json.

Guidance version 0.0.57 and previous stops correctly. Those versions seemed to use the given prompt's next tokens after the gen call to determine stop tokens.

@alexb1200 explained it correctly. We'll add some helper for valid json to the new release, hopefully soon.


Saving and Loading .obj of spatial mesh on runtime for Hololens spatial mapping

How about scanning the environment through hololens, saving it using voice or gesture commands and loading the saved .obj on runtime using voice or gesture commands for Hololens spatial mapping applications. Is it possible to use this service this way?

@sprakhar12 I have loaded in dynamic models using the glTF binary format. It ran as Windows Store App so it should also run ok as HoloLens UWP app. I haven't yet tried the .obj format, but providing there is a way to do this in Unity it should be another potential file format.

Thanks for giving the gITF suggestion. Have to look into it.
Will keep posting and updating on .obj if something works

@sprakhar12 I created a separate Unity sample for loading glTF models using blob storage. https://github.com/Unity3dAzure/StorageServicesGLTFDemo  I included it as a separate sample for now due to the external plugin binary dependencies of the UnityGLFT project.


☂ Release v0.10.0 requirements

We are targeting the release for the first week of Dev 2024. Since the date is close and 0.9 was not so long ago we may skip release candidate.
### Must Haves
- [ ] https://github.com/kubernetes-sigs/kueue/pull/3487
- [ ] https://github.com/kubernetes-sigs/kueue/issues/3533

### Nice To Haves
- [ ] https://github.com/kubernetes-sigs/kueue/issues/3232
- [ ] https://github.com/kubernetes-sigs/kueue/pull/3616
- [ ] https://github.com/kubernetes-sigs/kueue/issues/3589
- [ ] https://github.com/kubernetes-sigs/kueue/issues/1833
- [ ] https://github.com/kubernetes-sigs/kueue/issues/2936

It is going to be a smaller release than before, but we make it to release the TAS rank-based ordering which is awaited by users. Let me know if we you know about some other candidate features that could make it
cc @tenzen-y @mwielgus @mwysokin @dgrove-oss @kannon92

@dgrove-oss @tenzen-y Maybe we could consider graduating ConfigurableResourceTransformations?

Maybe we could consider graduating ConfigurableResourceTransformations to Beta?

It's relatively low risk (changing the feature gate to enable by default) and doesn't add/change any API.  There hasn't been much time to get user feedback, but that's probably ok.

@dgrove-oss @tenzen-y Maybe we could consider graduating ConfigurableResourceTransformations to Beta?

Does this indicate that we will graduate both ConfigurableResourceTransformations and WorkloadResourceRequestsSummary to beta?

Yes, I see no blockers for that

Yes, I see no blockers for that

SGTM, let us collect feedbacks by graduating those to beta

@dgrove-oss feel free to submit a PR for that

I have updated the nice-to-haves with 3 related KEPs:

https://github.com/kubernetes-sigs/kueue/issues/3589
https://github.com/kubernetes-sigs/kueue/issues/1833
https://github.com/kubernetes-sigs/kueue/issues/2936

Also, updated the list with TAS-related pending issues.

I'm ok to slightly postpone the original release target date to try to include the PRs which look almost ready:

https://github.com/kubernetes-sigs/kueue/issues/1833
https://github.com/kubernetes-sigs/kueue/issues/2936

cc @yaroslava-serdiuk @KPostOffice

/close
We are in progress of preparing 0.10 already.


The hal compute example has an incorrect alignment for memory mapped buffers

https://github.com/gfx-rs/gfx/blob/master/examples/compute/main.rs#L104
The vulkan spec says

If size is not equal to VK_WHOLE_SIZE, size must either be a multiple of VkPhysicalDeviceLimits::nonCoherentAtomSize, or offset plus size must equal the size of memory.

I am not sure what the correct action here should be considering that this should work across all backends. Should this be aligned with the value from get_buffer_requirements?

Technically this should not be required because the buffer is host coherent, but         device.release_mapping_writer(writer); will always flush even if the buffer is coherent.


Glass tiles are completely transparent in the dark

Glass tiles are simply too transparent even in the light

perhaps give the same blue tint as full windows.


Added samples for Azure CAPI types

This PR adds 3 more CRD examples related to Azure CAPI CRDs.
It also disables publication of CRD documentation for the legacy Azure CRDs AzureConfig and AzureClusterConfig (the customer don't have access to those CRs anyway.
Please remember to

[X] Give the PR a meaningful title -- it will be shown in the release notes
[X] Add (or remove) user questions associated with any updated docs

Did you manually edit these pages? Normally you would

update https://github.com/giantswarm/docs/blob/master/scripts/update-crd-reference/config.yaml#L6 to the latest apiextensions version
Run make update-crd-reference
Commit the results

You also added two CRDs to the skipped list, but the actual CRD pages still exist. With the execution of the above make command, that should also be solved.

yes I did it manually, my apologies for the mistake.
Will open a new PR.


Python >= 3.5 support

All Submissions:

[x] Have you added a note in the CHANGELOG.md for your change?
[ ] Have you added unit tests for your change?

I expect the PR test to fail since it's only running with Python 3.4 right now. Will need to fix it.

retest this please

retest this please

retest this please

retest this please

retest this please

retest this please


Default newton absolute error for dune seems wrong

Is your feature request related to a problem? Please describe.
The current examples (and expected inputs) have mesh sizes that are very small in absolute values. This, turns into absolute values that are also small for convergence criterions. The dune simulator in the editor has a default that doesn't seems suited for these kind of inputs: 1.00000e-12. Basically all of the examples that I tried[^1] "stay" in a steady state when simulated with the dune simulator due to this option.
Describe the solution you'd like
I would like that the dune simulation works "out-of-the-box" and doesn't need to be fine tuned for the common case. The absolute error checking in dune-copasi is optional, meaning whenever this value is not set, there won't be a check against the absolute value. Since most of the simulations here are expected to be dynamic, I would set this to empty as a default and let people change it when they know a sensible value for their system.
[^1]: except of the simple diffusion, probably because this does not use the non-linear solver

Thanks for looking into this - this sounds like a good solution to me

Hmm, I was thinking about this again. When the solution converges towards the steady state, the absence of an absolute tolerance will make the problem more and more difficult to resolve until it cannot reach the relative tolerance :thinking:. How do you solve this in the pixel simulator? How about an absolute tolerance that is a percentage of the total amount of species in the whole domain when they are equal to one or another expected value that the user can set up?

I agree this is a problem - Pixel uses explicit solvers so this issue doesn't come up there.
I guess it's not possible to have a heuristic for a good absolute tolerance for any model in general, but scaling it with species amount sounds like a good idea to me.
Are the units of the absolute tolerance species amount?
The user units in the GUI are typically biology concentration units, e.g. mmol/mL - so 1 in these units can be some crazy number in amount/volume in the dune-copasi model

The tolerance check should come evaluating the weak formulation integral with the current trial. So if I am not messing up something, it should be something like mmol/s (assuming the transient part has no scaling units, is that correct?). Sometimes one changes the equation to multiply by the dt instead of dividing by it but that must be handled internally by scaling the tolerance too. I need to double check this in our solvers.


improve straightening: directly use ITK in physical coordinates (instead of pixelar)

implement thin plate spline

can we close this?


Add git-co-authors extension

Description

Simplify adding co-author attributions to git commits. Add your frequent collaborators, and copy the correct syntax to paste into your commit.
Screencast

Add a co-author

Select a co-author

Paste into your commits

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for the review @pernielsentikaer, I'll try to make some time to make the changes requested this week!

Hi @nickradford 👋
I hope you don't mind I pushed some improvements, do you mind checking that I didn't break anything 🙂

@pernielsentikaer Looks good to me! Just pushed up the lint fix. Thanks for improving the UX :D


electronize is not fount in macOS

I install the cli tool with dotnet tool install ElectronNET.CLI -g in macOS, but the command electronize is not found. But it's worked on Windows.
> dotnet tool list -g
Package Id                            Version      Commands
------------------------------------------------------------------------------
dotnet-aspnet-codegenerator           2.2.2        dotnet-aspnet-codegenerator
electronnet.cli                       0.0.11       electronize
microsoft.web.librarymanager.cli      1.0.172      libman

I have the same problem.
command not found: electronize

May be the same solution as for Windows:
Rename or clone the electronize.exe to be called dotnet-electronize.exe

Any update ? I'm facing same issue also

I have the same issue On MacOS

@seanmars  @LuciferJun1227 @phamquocthang You must add dotnet tools to your $PATH. Most likely, you are using Bash, so you will have a ~/.bashrc or ~/.bash_profile that gets loaded in your terminal.
In your file, you must add
# .NET
# For tools installed with `dotnet tool` command
export PATH="$HOME/.dotnet/tools:$PATH"

and then you must either source that file (eg source ~/.bash_profile) or create a new terminal window/tab.


Maintenance updates

Fix data type of 'ScaleType'
Do not send disconnect events, if the remote ski is not found as an existing remote device

coverage: 93.684% (+0.005%) from 93.679%
when pulling 5d63cf6098720a1325ed80d26be04e9d7caaa57d on feature/maintenance
into 0589320cecedd40c010c1152b4cf20a5a16e2589 on dev.


Docs: (Epic) Editorial/style review of plugin-tools website

Review content of plugin-tools for adherence to Grafana style guidelines and fix errors and typos. Ensure that content contains appropriate links to all relevant related pages. Ensure that content is appropriately categorized within the site's hierarchy.
### Tasks
- [ ] Review "Get started" for editorial style
- [ ] Review "Key concepts" for editorial style
- [ ] Review "Tutorials" for editorial style
- [ ] Review "How-to guides" for editorial style
- [ ] Review "E2E test" for editorial style
- [ ] Review "Publish a plugin" for editorial style
- [ ] Review "Troubleshooting" for editorial style
- [ ] Review "Reference" for editorial style

Question: Who should review these PRs? Do we need to get another member of the doc squad set up to review PRs in this repo?

Per conversation with Fiona, editorial style review will be done "ad hoc" on other PRs rather than in a separate PR.


accept a fromServer param for restangularizeCollection

restangularizeColleciton is actually mapped to restangularizeCollectionAndElements which does not accept a parameter for "fromServer".  This causes elements in the collection to perform "POST" requests when element.save() is called.  I would like to set fromServer to true so that a proper "PUT" request will be executed.
A current hack to get around this is do to something like this:
Restangular.restangularizeCollection(test, test.samples, 'samples');
_.each test.samples, (sample) -> sample.fromServer = true
I would like to merge this unless there is good reason for why "false" is explicitly being passed.
Thanks

+1

Great!! Rebase and I'll merge!

@bearfact Can you please rebase and I'll merge this in?

Actually I'm going to go ahead and close this in favor of #1389 since it's up to date with master, includes documentation changes and unit tests. Thanks for your work tho @bearfact


Reorg the sidebar with guidance

Reorganize the sidebar based on feedback from @sarah11918
Give guidance on new pages to add, convert, merge together, etc.

There's nothing here I don't like.  :)  Proposed, commented out changes look good, too.
The only thing I'm still not 100% sold on is that "API" is the best label to be opposite Learn.
Gatsby uses "Reference" for all that same kind of info that we have in "API" main section. My gut is that's more descriptive and accurate for us.
Nextjs and React use API, but then, the ONLY thing in their API section is actually API stuff, not other guides/config etc.
11ty has all sections in one big sidebar including separately Configuration, Plugins, API Services (*of interest for future, they also have a Glossary!)

@sarah11918 +1 from me! I meant to change that here based oh your earlier feedback and forgot :)

So, I didn't need to round up the troops to get others to weigh in then?  ;)

I agree with Sarah that reference is a much more descriptive label for that section.
I do feel like it's weird having some sidebar-headings being two words and some only being one word.
I also feel like the "basics" section name isn't the best name for that section (although I can't think of anything better, so feel free to ignore this opinion).

So, I think we like this!  3 things (though only the 3rd is blocking before we can actually switch up the sidebar on the live site. The other 2 can always be changed later):

Learn / Reference instead of Learn/API  (I'm not sure if I can JUST change type: api here to reference, or whether it's somewhere in the sidebar component itself this should (also?) be changed, so that's on @FredKSchott to take care of.

We've had a preference indicated for changing BASICS to ESSENTIALS. Anyone have strong feelings on this? (will also ping @delucis ) I have no strong preference for either header.

The "Why Astro?" get-to-the-heart-of-Astro page doesn't exist yet. Sarah has notes; Fred has thoughts. This one's on us.

Just opened #236, which ought to make this question easier to answer!

I think either Essentials or Fundamentals could both be good.

🧑🚀 ⏩ ❤ — ✅

I agree with Sarah about keeping header words simple for accessibility & i18n. I do personally like Fundamentals though, but I think a11y and i18n win out over personal preference :)
Cleaning up and merging! The comments in the config file are left as TODOs to tackle over the next few weeks


Training of CRAFT model

Can you provide help/examples on how to perform training of CRAFT model?

Hello Asad, I'm looking for the same answer. Did you manage to find it ?

@tahakoudri  No , I am trying Keras implementation of CRAFT for training. They have provided documentation for training.
You may find this comment also useful.

Thank you so much for your advice, I am also looking for fine tuning this type of solution. So am wondering is it possible to maybe have a call with you this week, to discuss about our work ?

@tahakoudri did you manage to fine tune the CRAFT model. If Yes, can you share how to do.

Please check this comment!


add missing --graceful option

On newer Laravel versions, the built-in MigrateCommand comes with an extra option:
{--graceful : Return a successful exit code even if an error occurs}

This will cause php artisan migrate-data to fail with The "graceful" option does not exist..
This PR will fix that.

Thank you for the contribution @jeffhuys


Location of response message section in transactions.

Section Number Identify the most specific section number the issue occurs (e.g. 4.1.2)
3.110, 3.112, 3.113
Issue Describe your issue. Don't write a book, but do include enough to indicate what you see as a problem.
The response messages should be right after the request message when there are multiple messages for the transaction instead of all the requests followed by all the responses.
Proposed Change
Place the response sections directly after the request sections
Priority:

Low: Typo or other minor classification that an editor can manage. Requires no group discussion.

Ok, this would mean that
ITI-110 will have the following new order:

2:3.110.5 Create Subscription Request Message
2:3.110.6 Create Subscription Response Message
2:3.110.7 Update Subscription Request Message
2:3.110.8 Update Subscription Response Message

ITI-112 will have the following new order:

2:3.112.5 Handshake Notification Request Message
2:3.112.6 Handshake Notification Response Message
2:3.112.7 Heartbeat Notification Request Message
2:3.112.8 Heartbeat Notification Resonse Message
2:3.112.9 Event Notification Request Message
2:3.112.10 Event Notification Response Message
2:3.112.11 Subscription Deactivation Notification Request Message
2:3.112.12 Subscription Deactivation Notification Response Message

ITI-113 will have the following new order:

3.113.5 Subscription Search Request Message
3.113.6 Subscription Search Response Message
3.113.7 Subscription Status Search Request Message
3.113.8 Subscription Status Search Response Message
3.113.9 Subscription Events Search Request Message
3.113.10 Subscription Events Search Response Message


default not set on option

The default is not being set on the option:
    use ConfigureWithAttributes, RunsCommands, RunsProcesses;

    public function __invoke(
        #[Option(description: "create the project records")]
        bool $createProjects = true,

        $application = new Application($kernel);
        $command = $application->get($commandName);
        $defintion = $command->getDefinition();
        $option = $defintion->getOption('createProjects');
        assert($option->getDefault() === true); // fails

I'm digging into it now, and will submit a PR.

There's something wrong with the condition:
        if ($value->mode ^ InputOption::VALUE_NONE && $parameter->isDefaultValueAvailable()) {
            $value->default = $parameter->getDefaultValue();
        }

The problem is that $value->mode is InputOption::VALUE_NONE, so the bitwise OR return false, and the default value isn't set.
@kbond any suggestions?

Ah, the caret is a bitwise XOR, I think that line should be
if ($value->mode | InputOption::VALUE_NONE && $parameter->isDefaultValueAvailable()) {

Perhaps I'm confusing value_none with value_negatable?
When I try to explicitly declare the mode, I get
Cannot use $mode when using Zenstruck\Console\Attribute\Option as a parameter attribute, this is inferred from the parameter's type.
I get a similar error when explicitly declaring the default in the Option attribute.

I'm going to close this, as my thinking was wrong on VALUE_NONE options.


Undefined Field :gsub

How are you using the lua-language-server?
NeoVim
Which OS are you using?
MacOS
What is the issue affecting?
Diagnostics/Syntax Checking
Expected Behaviour
str:gsub() should be regarded as valid
Actual Behaviour
str:gsub() triggers a warning undefined field. It looks like this is a regression of #567. It's pretty much the same issue that was reported there already.

Reproduction steps
keymap("n", "<leader>i", function ()
	local lineContent = fn.getline('.')
	if lineContent.find("!important") then
		lineContent = lineContent:gsub("!important", "")
	end
end)

Additional Notes
No response
Log File
No response

local lineContent = fn.getline('.')  ---@type string

I mean, this does work, but feels a bit cumbersome, considering how commonly used :gsub and family are.
could there maybe be an option to exclude them from the undefined-field rule? Or a code action to add ---@type string to the respective line?

issue fixed via neodev actually.


Unable to do method calls or edit fields

The latest commit (well since 07f0b07 which fixes a crash issue) is unable to do method calls via the explorer, everything is no-oping. All the properties/fields are defaulting to 0 and trying to set them does nothing (setting the frame for instance).

In that commit, not calling [invocation invoke]; seems to be culprit for the default values but not sure about method calls being no-oped.

I tried checking out the latest commit with 07f0b07 reverted and that seems to fix this issue completely. This means that if statement is the cause of the method calls not working too.

Nah, it's because I removed that -invoke call like you said. Originally I meant to just remove the comments above it I think.
The if statement I added is to avoid calling methods we can't get the return values of anyway.
Thanks for catching this! Dunno how I overlooked this...

Pretty sure the methodReturnLength is 0 for methods that have return type of void which is why I wasn’t able to set the frame even after re-adding invoke.

Wow, I goofed again. I'm gonna force-push all three of these fixes into one commit for cleanliness, so you'll have to do a force-pull. Thanks again!


Be more explicit about what's happening in the require.ensure block

I'm seeing links out to this warning (http://bluebirdjs.com/docs/warning-explanations.html#warning-a-promise-was-created-in-a-handler-but-was-not-returned-from-it) in a project that is enabling blue bird warnings.  I think we need to return null in our require.ensure.

I am seeing this issue with translatiosnLoader.js as well


Add a flag to Column so user can choose to strip line returns or tabs from within column values

    # LMDTODO: Add a flag which allows the user to say if line returns or tabs should be stripped from INSIDE
    # values of this column.  It's a frequent data error when data is edited or provided in Excel, to have
    # a multiword value like "United\nKingdom" or "United \nKingdom".  Yet we don't want to strip line returns
    # from long text values.  Also move this explanation to spec/docs

@YuliaS  this would be a good one


Need a list of allowed extensions for non-image binaries

Issue #113 allows editors to include and link to binary files such as PDFs, but we obviously need a highly-constrained list of file types that we're willing to allow. I'm inclined to say that the only file type we should allow is PDF, until we're given a strong case for a different type. Definitely no Office docs or anything like that; they can be turned into PDFs if required. @JanelleJenstad @LEMDO-PM can you think of any other type we should consider at this point?

I agree. PDF only.

PDF sounds good to me as well!

Fix committed in rev 12248. If all is well, this can be closed.

Added Schematron to enforce this in rev 12275. Closing now.