[Bug]: from llamaparse import LlamaParse throws PydanticSchemaGenerationError

Bug Description
Running from llamaparse import LlamaParse throws this pydantic error PydanticSchemaGenerationError.
It says:
PydanticSchemaGenerationError: Unable to generate pydantic-core schema for typing.AsyncGenerator[str, NoneType]. Set `arbitrary_types_allowed=True` in the model_config to ignore this error or implement `__get_pydantic_core_schema__` on your type to fully support it.

If you got this error by calling handler(<some type>) within `__get_pydantic_core_schema__` then you likely need to call `handler.generate_schema(<some type>)` since we do not call `__get_pydantic_core_schema__` on `<some type>` otherwise to avoid infinite recursion.

For further information visit https://errors.pydantic.dev/2.10/u/schema-for-unknown-type

Version
0.12.0
Steps to Reproduce
pip install llama-index==0.12.0
from llama_parse import LlamaParse
Relevant Logs/Tracbacks
No response

@chengyin38 seems pydanticv2.10 is breaking. Try downgrading to 2.9 for now, will have to make a patch on llama-index-core separately

Confirming that downgrading Pydantic to version 2.9.2 resolved the issue.

@chengyin38 I also just released llama-index-core==0.12.1 which pins pydantic<2.10, which should save anyone with a fresh install.
Will unpin once the dust settles on pydantics end :)


can't install apple guides and sample code

I'm trying to install apple guides and sample code which is the updated version of swift docs, but can't install it successfully.
It always complains "mv: rename /Users/dr_selump14/.local/share/dasht/docsets/ to /Users/dr_selump14/.local/share/dasht/docsets/Apple_Guides_and_Sample_Code.docset: Invalid argument"
Do you have any idea to fix this? thanks

I tried installing it here but got a zero-length file downloaded instead:
echo Install Apple_Guides_and_Sample_Code docset [y/N] ?...y
--2017-09-17 12:36:58--  https://kapeli.com/feeds/Apple_Guides_and_Sample_Code.tgz
Resolving kapeli.com (kapeli.com)... <IP_ADDRESS>
Connecting to kapeli.com (kapeli.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 0 [application/x-tar]
Saving to: ‘/home/sunny/.local/share/dasht/docsets/Apple_Guides_and_Sample_Code.tgz’

Apple_Guides_and_Sample_Code.tgz            [ <=>                                                                           ]       0  --.-KB/s    in 0s

2017-09-17 12:36:59 (0.00 B/s) - ‘/home/sunny/.local/share/dasht/docsets/Apple_Guides_and_Sample_Code.tgz’ saved [0/0]

gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now

gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now
exit 2

Can you check whether your dowloaded docset file is also zero-length?
-rw-r--r--. 1 sunny sunny 0 Oct 28  2016 Apple_Guides_and_Sample_Code.tgz

Yeah, I got zero-length file too.... what happened actually?

The docset is actually zero-length upstream in Kapeli's repository:
https://kapeli.com/feeds/Apple_Guides_and_Sample_Code.tgz
Downstream users like us have no control over this. /cc @Kapeli

The Apple Guides and Sample Code docset has been removed. Apple no longer distributes it.

Oh, OK .... that's too bad ... close this issue then


increase test-no-self-tracing timeout

Travis intermittently fails on these.
@matthewloring PTAL.

LGTM.


ModuleNotFoundError: No module named 'numpy.typing'

when i run the example using the source code, there is an error, ModuleNotFoundError: No module named 'numpy.typing'

Hi, this is probably due to your numpy version being less than 1.20.0.  (pip freeze | grep numpy will check this.)  Did you pip install . in a venv?

nice! it works after i ungrade the numpy.
------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2021年7月8日(星期四) 中午12:19
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [kensho-technologies/pyctcdecode] ModuleNotFoundError: No module named 'numpy.typing' (#11)
Hi, this is probably due to your numpy version being less than 1.20.0.  (pip freeze | grep numpy will check this.)  Did you pip install . in a venv?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Fix for twistlock issues 57097

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

/test check-ibm-healthcheck-operator


Notebook actions are not rendered until clicking the notebook again

Testing #93742

From the explorer, click on a notebook file
Observe that notebook actions are not rendered
Click the notebook tab title
Observe that notebook actions are now rendered

Maybe a focus issue?


Copy IPV6-mapped-IPV4 addresses correctly in native code

Motivation:
8dc6ad5 introduced IPV6-mapped-IPV4 address support but
copied the addresses incorrectly. It copied the first
4 bytes of the ipv6 address to the address byte array
at offset 12, instead of the other way around.
7a547aa implemented this correctly in netty_unix_socket.c
but it seems the change should've been applied to
netty_epoll_native.c as well.
The current behaviour will always set the address to
<IP_ADDRESS>.
Modifications:
Copy the correct bytes from the ipv6 mapped ipv4 address.
I.e. copy 4 bytes at offset 12 from the native address
to the byte array addr at offset 0.
Result:
When using recvmmsg with IPV6-mapped-IPV4 addresses,
the address will be correctly copied to the byte array
addr in the NativeDatagramPacket instance.

@kbeckmann thanks a lot! Can you also add a unit test or adjust one to verify this ?
Also can you please sing our ICLA: https://netty.io/s/icla ?

@netty-bot test this please

Thanks for the quick response! I am very new to this code base and would prefer not to do further changes :).

@kbeckmann fair enough... then just sign the ICLA and I will do a followup :)

Alright. I as a private person with my private email have signed the ICLA.

@netty-bot test this please

@kbeckmann thanks a lot... merged :)


Failing on dailymotion video

Here's the full output:
$ youtube-dl --verbose http://www.dailymotion.com/video/x397k4q
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--verbose', 'http://www.dailymotion.com/video/x397k4q']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.09.28
[debug] Python version 3.5.0 - Linux-4.2.2-1-ARCH-x86_64-with-arch
[debug] exe versions: ffmpeg 2.8, ffprobe 2.8, rtmpdump 2.4
[debug] Proxy map: {}
[dailymotion] x397k4q: Downloading webpage
[dailymotion] x397k4q: Downloading embed page
ERROR: Unable to extract video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/lib/python3.5/site-packages/youtube_dl/YoutubeDL.py", line 660, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python3.5/site-packages/youtube_dl/extractor/common.py", line 288, in extract
    return self._real_extract(url)
  File "/usr/lib/python3.5/site-packages/youtube_dl/extractor/dailymotion.py", line 201, in _real_extract
    'video info', flags=re.MULTILINE),
  File "/usr/lib/python3.5/site-packages/youtube_dl/extractor/common.py", line 582, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract video info; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Running from Arch linux latest package version 2015.09.28

You are using an outdated version of youtube-dl. Read our FAQ if you have troubles updating.


Ability to use fully qualified names i.e. catalog when specifying db objects

Use case:
PosgreSQL client compatibility (e.g. Trino)
SQL compliance
CrateDB
CREATE TABLE doc.x (txt TEXT)
SELECT crate.doc.x.txt FROM crate.doc.x

PostgreSQL
postgres=# CREATE TABLE public.x (txt TEXT);
CREATE TABLE
postgres=# SELECT postgres.public.x.txt FROM postgres.public.x;
 txt
-----
(0 rows)

From PostgreSQL docs

database.schema.table
can be used too, but at present this is just for pro forma compliance with the SQL standard. If you write a database name, it must be the same as the database you are connected to.

https://crate.io/docs/sql-99/en/latest/chapters/18.html#table-name

To elaborate a bit more on the impact regarding Trino:
Specifically INSERT statements are affected by this. In earlier Trino versions, setting the connection URL to jdbc:postgresql://localhost:5432/ (omitting a database name) used to be a workaround, but in more recent versions, Trino apparently attempts to always put a database name into the table identifier.
Therefore, INSERT INTO doc.t1 VALUES (1); will fail with all possible combinations:

jdbc:postgresql://localhost:5432/ -> INSERT INTO "u1"."doc"."t1" ("a") VALUES (1) (missing database name substituted by Trino with the username 🤔)
jdbc:postgresql://localhost:5432/doc -> INSERT INTO "doc"."doc"."t1" ("a") VALUES (1)
jdbc:postgresql://localhost:5432/crate -> INSERT INTO "crate"."doc"."t1" ("a") VALUES (1)

SELECT statements from Trino still work, they use the simple schema_name.table_name identifier pattern.
Tested with Trino version 387.

Also relevant for dbt
SELECT *
FROM "doc"."dbt_gtraar"."my_first_dbt_model"
WHERE "id" = 1
LIMIT 500

I this case also https://github.com/crate/crate/pull/12652 is relevant
checking against the constant DB_NAME imho is really only possible with the above PR.
Otherwise it should probably just be ignored.


mixer: enforce type discipline for attribute values

enforce StringMap use as the attribute value across the code base
add List, used only by the proto decoder (otherwise, we would have to disable that code) and not surfaced to the user
set L2 cache size to 0
enforce int64 and remove int as a valid attribute value
update code generation to wrap the types accordingly before placing in the bags
update IL/compiler to enforce the types of the computed values and attribute reads

/fixes #10252

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

Needs approval from an approver in each of these files:

install/OWNERS
mixer/OWNERS [kyessenov]

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

This fixes the specific bug pointed out in #9596 . However, we are still disabling the cache, until we install  regression testing to prevent this error from happening in the future.

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-simpleTestsMinProfile

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

Revewers: I'm not fully done, but I want to use the CI machinery to validate e2e behavior with my change. I suspect there's some dependency on the check cache somewhere, but I'm not sure where...

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth-mcp.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth-mcp

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth-mcp.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth-mcp

prow/e2e-mixer-no_auth.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth

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

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

Needs approval from an approver in each of these files:

install/OWNERS
mixer/OWNERS [kyessenov]
tests/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth-mcp

prow/e2e-mixer-no_auth.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth

prow//e2e-simpleTests-minProfile.sh
db4eb2c6bb4d1ab5c77ad85fd8665415d6e138f9
link
/test e2e-simpleTestsMinProfile

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

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
b9f1c15d767d523ab3c085f025e4d89c7bc97592
link
/test e2e-mixer-no_auth-mcp

prow//e2e-simpleTests-minProfile.sh
db4eb2c6bb4d1ab5c77ad85fd8665415d6e138f9
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
db4eb2c6bb4d1ab5c77ad85fd8665415d6e138f9
link
/test e2e-mixer-no_auth

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

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

Needs approval from an approver in each of these files:

mixer/OWNERS [kyessenov]
tests/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
db4eb2c6bb4d1ab5c77ad85fd8665415d6e138f9
link
/test e2e-mixer-no_auth

prow//e2e-simpleTests-minProfile.sh
f820b12df000841844928f175002303ad40c493b
link
/test e2e-simpleTestsMinProfile

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

/test e2e-mixer-no_auth
Seems something is flaky there...
mixer_test.go:843: Query prometheus to get baseline cache hits...
mixer_test.go:849: Baseline cache hits: 15
mixer_test.go:850: Start to call visit function...
mixer_test.go:856: Query promethus to get new cache hits number...
mixer_test.go:862: New cache hits: 15
mixer_test.go:867: Check cache hit: 0 is less than expected: 1

PTAL, the PR is ready for review.

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-mixer-no_auth

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

/test e2e-mixer-no_auth

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-mixer-no_auth

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

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-mixer-no_auth

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

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

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

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth.sh
e8b60382f53fef3016316c39433e8c6ee883a67b
link
/test e2e-mixer-no_auth

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-dashboard.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-dashboard

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

/test e2e-dashboard

Updated the PR to include fixes for the code generation. It used to not wrap net.IP and map[string]string in the right boxes before placing into the bags.
I also replaced int with int64 everywhere in the bag.

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-dashboard.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-dashboard

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-dashboard.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-dashboard

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-dashboard.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-dashboard

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-mixer-no_auth

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-dashboard.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-dashboard

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-multicluster-e2e

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-simpleTests

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-dashboard.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-dashboard

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-local-tests.sh
39c3eb7a06de4111d841dc18f8e5a378f7b678bd
link
/test istio-integ-local-tests

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-bookInfoTests-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
0e5164b140306b1eec2ccdb09fa5f39c3c09f807
link
/test e2e-dashboard

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

/test e2e-dashboard

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-mixer-no_auth.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test e2e-mixer-no_auth

prow/istio-pilot-multicluster-e2e.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
0e5164b140306b1eec2ccdb09fa5f39c3c09f807
link
/test e2e-dashboard

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

/test e2e-dashboard

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/istio-pilot-multicluster-e2e.sh
4226be8b5e960d736ce8ee757fd4084705337120
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
0e5164b140306b1eec2ccdb09fa5f39c3c09f807
link
/test e2e-dashboard

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-dashboard.sh
0e5164b140306b1eec2ccdb09fa5f39c3c09f807
link
/test e2e-dashboard

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

/test e2e-dashboard

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/e2e-simpleTests-minProfile.sh
75651d2877a2e1252887d5ca4af0d32f6bcb5e7b
link
/test e2e-simpleTestsMinProfile

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

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

Test name
Commit
Details
Rerun command

prow//e2e-simpleTests-minProfile.sh
765c813643d43312413922618f0b577d706e82cd
link
/test e2e-simpleTestsMinProfile

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
3ecf9c53dd4c60f010475d40e46513cb8b831791
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-multicluster-e2e.sh
3ecf9c53dd4c60f010475d40e46513cb8b831791
link
/test istio-pilot-multicluster-e2e

prow/e2e-dashboard.sh
3ecf9c53dd4c60f010475d40e46513cb8b831791
link
/test e2e-dashboard

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

/test istio-pilot-multicluster-e2e

/test e2e-dashboard

/test istio-pilot-e2e-envoyv2-v1alpha3

The PR is ready.
Friendly request for a review.

Ping @geeknoid @mandarjog
This PR contains the bit to disable L2 cache, which is one known source of flakiness in mixer tests.

Given all these fixes, do we still need to disable the L2 cache?

@geeknoid I believe we need to add tests to validate L2 cache behavior. Specifically, the original bug could have been caught in a unit-test size set-up.

Well, fine, since the in-proc Go adapter model is on the way out I suppose
it doesn't matter.
/lgtm
/approve
On Mon, Dec 10, 2018 at 12:57 PM Kuat<EMAIL_ADDRESS>wrote:

@kyessenov commented on this pull request.
In mixer/adapter/signalfx/tracing.go
https://github.com/istio/istio/pull/10260#discussion_r240376975:

@@ -28,6 +28,7 @@ import (
octrace "go.opencensus.io/trace"

"istio.io/istio/mixer/pkg/adapter"

"istio.io/istio/mixer/pkg/attribute"

For OOP, the value holds the protobuf dynamic value:
https://github.com/istio/istio/blob/master/mixer/template/tracespan/tracespan_handler_service.proto#L154
This code is for in-process adapter, which doesn't look like it would work
out of the box.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/10260#discussion_r240376975, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHT3arWngAM7wKSttTI4wmraTeBObks5u3srFgaJpZM4Y_rmV
.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, kyessenov
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

mixer/OWNERS [geeknoid,kyessenov]
tests/OWNERS [geeknoid]

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


Grammar/Spelling: Too go directly to the Restricted users page, use https://security.microsoft.com/restrictedusers.

I believe the sentence should read "To go directly to the Restricted users page, use https://security.microsoft.com/restrictedusers."  It wouldn't hurt to have someone proof read the pages for grammar, spelling and accuracy.

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

ID: 4c17e50d-aa8f-bd30-66af-ef4d63f02402
Version Independent ID: 92cbda3f-c3a9-ed20-2038-7e966f491934
Content: Remove blocked users from the Restricted users portal - Office 365
Content Source: microsoft-365/security/office-365-security/removing-user-from-restricted-users-portal-after-spam.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@bt101, fixed. In the future, you too can make fixes like this yourself without opening an Issue first. For instructions, see https://github.com/MicrosoftDocs/microsoft-365-docs/blob/public/README.md.
Thanks for the feedback.
Closing


Extra pre_processor/post_processor input parameter

See #361
Sometimes, for example the yolov5 model, to make it work with Rikai. We have to create a wrapper model to separate these three parts: pre_processing/model.forward/post_processing.
Simply adding extra parameters to pre_processor/post_processor is not enough for the yolov5 model.


Updated DG and added missing UML Diagrams

Added Sequence Diagram for Edit Task
Added Storage UML Diagrams
Edited DG
Fixed bugs in Todo Task
Fixed Validation Regex Bug for Student name: Student Names can now be a single letter

Codecov Report

Merging #260 (20f1026) into master (586bc65) will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #260      +/-   ##
============================================
- Coverage     65.12%   65.07%   -0.05%
  Complexity      852      852
============================================
  Files           137      137
  Lines          3148     3150       +2
  Branches        414      415       +1
============================================
  Hits           2050     2050
- Misses          957      958       +1
- Partials        141      142       +1

Impacted Files
Coverage Δ

...address/logic/parser/AddTodoTaskCommandParser.java
70.58% <0.00%> (-9.42%)
:arrow_down:

...rc/main/java/seedu/address/model/student/Name.java
80.00% <ø> (ø)

Continue to review full report at Codecov.

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


BUG: Fix offbound overview-drag, discrete vertical scrolling

What does this implement/fix?
This fixes the following minor bugs:

[x] Dragging in Overview-Bar off bounds let to an error and unexpected behaviour
[x] Set background-color white as default (relevant for Epochs.plot() without parameter "bgcolor")
[ ] Vertical scrolling with a touchpad scrolled vertically in smaller steps than channels which is unnecessary

@larsoner @hoechenberger With e.g. a touchpad you can currently input vertical scrolling with steps smaller than the common scrollwheel-input. This results in smooth vertical scrolling. Would you rather keep this or do you think it is not necessary or even contraintuitive?
https://user-images.githubusercontent.com/46245704/154279087-a2b72536-4be2-4869-869d-8ff40441ee34.mp4

I think smooth is actually what most people would expect from the touchpad

Yes, smooth, anything else would feel wrong


refactor(batch): Use futures-async-stream to implement limit executor

Signed-off-by: Fedomn<EMAIL_ADDRESS>What's changed and what's your intention?
Use futures-async-stream to refactor LimitExecutor
Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)
close https://github.com/singularity-data/risingwave/issues/1938

bors try


How set action on Closed App

Hello,
I would like to ask if I can set equivalent of .addAction() and .setStyle() of notification or how use app notification builder on closed app.
Because my problem is that when my app is closed does not execute code in onMessageReceived and I can't notification.setStyle or  .addAction().
thank you for share!

Hi,
For Firebase message notification does not support special notifications, use a data message because it call every time onMessageReceived.
$dataBuilder = new PayloadDataBuilder();
$dataBuilder->addData(['a_data' => 'my_data']);

maybe this thread can be help you : http://stackoverflow.com/questions/38504078/firebase-expandable-notification-show-image-when-app-is-in-background.
I hope that will help you . Please give me a feedback if that solve or not your issue.
Best regards
Nicolas

Hello, thankyou for your reply!
But my problem persist.
public function alert($text = null, $token, $action=null)
    {
$optionBuiler = new OptionsBuilder();
        $optionBuiler->setTimeToLive(60*20);
        $notificationBuilder = new PayloadNotificationBuilder('NOTIFY');
        $notificationBuilder->setBody($text)
            ->setSound('default');

        $dataBuilder = new PayloadDataBuilder();

        $dataBuilder->addData(['a_data' => 'my_data']);

        $notification = $notificationBuilder->build();
        $data = $dataBuilder->build();
        $option = $optionBuiler->build();

        FCM::sendTo($token, $option, $notification, $data);
}

I have this function, and when the app is open pass on onMessageReceived corretly, but when I close my app the onMessageReceived is not executed.
am I doing something wrong?

Hello, Try to remove the notification payload of your message like bellow:
public function alert($text = null, $token, $action=null)
    {
        $optionBuiler = new OptionsBuilder();
        $optionBuiler->setTimeToLive(60*20);
        $dataBuilder = new PayloadDataBuilder();

        $dataBuilder->addData(['a_data' => 'my_data']);

        $data = $dataBuilder->build();
        $option = $optionBuiler->build();

        FCM::sendTo($token, $option, null, $data);
}

because when notification is present on the payload, android will handle the message and show it without passing in onMessageReceived when app is in background.
From FCM documentation :
Messages with both notification and data payloads
App behavior when receiving messages that include both notification and data payloads depends on whether the app is in the background or the foreground—essentially, whether or not it is active at the time of receipt.
When in the background, apps receive the notification payload in the notification tray, and only handle the data payload when the user taps on the notification.
When in the foreground, your app receives a message object with both payloads available.
Here is a JSON-formatted message containing both the notification key and the data key:
  {
    "to" : "APA91bHun4MxP5egoKMwt2KZFBaFUH-1RYqx...",
    "notification" : {
      "body" : "great match!",
      "title" : "Portugal vs. Denmark",
      "icon" : "myicon"
    },
    "data" : {
      "Nick" : "Mario",
      "Room" : "PortugalVSDenmark"
    }
  }

Please could you test it and give me a feedback ?

Yes! it Work!
thank you!


Video player

As a user, I want to be able to preview my video files locally. Therefore, I need a video player. At a minimum, I should be able to play and pause files. At best, I should also be able to move forwards and backwards through the file and jump to specific time stamps. The (remaining) duration of the file should be displayed.

For starters, see here: https://flutter.dev/docs/cookbook/plugins/play-video

See here for supported file types: https://exoplayer.dev/supported-formats.html

See here for supported file types: https://exoplayer.dev/supported-formats.html

Added in commit 7cfa186b.

Added in commit 7cfa186b.


chore(main): release 1.1.0-rc

A Release PR has been automatically generated based on activity.

1.1.0-rc (2024-07-08)
Features

support env vars for all flags (#9) (98b298a)

Bug Fixes

use cobra stdout (#11) (9285314)
use correct var for error (2c0a279)

Merge this pull request to cut the next release.

:robot: Created releases:

v1.1.0-rc
:sunflower:


Support EmitMode::ModifiedLines with stdin input

Main motivation is to handle the case where we have an input from stdin and we're mostly interested in a diff rather than whole file and trying to come up with this information ourselves. The EmitMode::ModifiedLines should still support writing to a specified Write sink - we mostly lacked the information on what was the input text so we ask that the passed source_map.
Currently works with https://github.com/Xanewok/rls/tree/rustfmt-modified-lines branch on the RLS side.
related #1173 (although this still has to go through (de)serialization)

CI fails on clippy failure, which in turn is blocked on https://github.com/laumann/compiletest-rs/pull/160

CI failed on ICE in futures-rs:
error: internal compiler error: src/librustc_codegen_llvm/debuginfo/metadata.rs:661: debuginfo: unexpected type in type_metadata: impl core::future::future::Future

@Xanewok looks like there's a PR out already: https://github.com/rust-lang/rust/issues/58888

LGTM, thanks for the update!

Thank you! Do you think you could publish a point release with this or do you prefer to accumulate more changes for the release, first?


WIP: Fix utility functions to branch

The utility functions were using ifelse to evaluate in places where the utility function couldn't be evaluated -- This doesn't work because it evaluates both cases at once and it throws an error.
This is WIP and still needs some tests to check this and potentially some other cases.
For example
u = LogUtility()

# Should just do linear approximation around 1e-10 and extrapolating down, but throws error
u(-0.01)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #184   +/-   ##
=======================================
  Coverage   90.69%   90.69%
=======================================
  Files          24       24
  Lines        1633     1633
=======================================
  Hits         1481     1481
  Misses        152      152

Impacted Files
Coverage Δ

src/modeltools/utility.jl
94.44% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

I'll hold on on review until after you notify me that it is no longer WIP

I think this is ready for now -- I don't have the extrapolating feature ready for elliptical utility yet, but it works for Log/CRRA/CFE


How should the Format should look like?

<IP_ADDRESS> - - [25/Oct/2018:14:27:47 +0200] "GET /shop/caller.php HTTP/1.1" 200 20

goaccess access.log --log-format=COMMON

Closing this. Feel free to reopen it if needed.


refactor(project): Use the new package names

:pencil: Description
Use the new package names.
BREAKING CHANGE: All imports statements need to be updated
Note: The tests won't pass because we won't publish SDK-Core 0.8.2 to maven central.
:link: Related Issues
https://github.com/ExpediaGroup/openworld-sdk-java-core/pull/111

[x] Code is up-to-date with the main branch.
[x] You've successfully built and run the tests locally.
[ ] There are new or updated unit tests validating the change.

#66 replaces this and more.


Python3 type hint annotations are not considered

Hi, thanks for the great project.
Currently it seems that importmagic doesn't look for __annotations__ properties added in recent version of python 3, so python3 syntax-aware checkers complain if I use a type hint without importing the name, but importmagic will delete any import that's only referenced in a type annotation.
Test case:
import importmagic

code = """
from typing import Iterable

def print_it(it: Iterable):
    for i in it:
        print(i)

print_it(['a', 'b', 'c'])
"""

scope = importmagic.Scope.from_source(code)
unresolved, unreferenced = scope.find_unresolved_and_unreferenced_symbols()
print("unreferenced: {}".format(unreferenced))

from typing import Iterable

def print_it(it: Iterable):
    for i in it:
        print(i)

print("function annotation: {}".format(print_it.__annotations__))

Output:
$ python3 importmagictest.py
unreferenced: {'Iterable'}
function annotation: {'it': typing.Iterable<+T_co>}

Well spotted, and great test case, thanks for the bug report. I'll try to get to that this weekend.

(unrelated, but both of our avatars are photographs of Autumn leaves)

Any updates with this? Running into the same issue.

It looks like #49 fixed this?


[HOLD for payment 2023-02-10] Double spaces in IOU confirmation message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Have an IOU request in the chat
Settle it up, either using PayPal, Elsewhere, or Wallet
Notice the text has double spaces, e.g. Settled up  elsewhere.

Expected Result:
There shouldn't be an extra space.
Actual Result:
There is an extra space.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: v1.2.63-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/9680864/216183206-de6feddc-77b5-48c8-a73a-843fb7a47615.mov
Expensify/Expensify Issue URL:
Issue reported by: @youssef-lr
Slack conversation:
View all open jobs on GitHub

@NicMendonca super quick fix I have a PR for ready shortly. No need for C+.

@youssef-lr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@youssef-lr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

No payment to be made as this was worked on and reviewed internally. Closing!


Backwards incompatibility between Microsoft.Data.SqlClient.SqlConnectionStringBuilder and System.Data.SqlClient.SqlConnectionStringBuilder

Describe the bug
There is a significant discrepancy between the two SqlConnectionStringBuilder types as far as their internal keywords are defined.
The keywords can be found in the internal class DbConnectionStringKeywords.  An example:
https://github.com/dotnet/SqlClient/blob/04db1a40a95726af8567b7596653e20acdd7f19a/src/Microsoft.Data.SqlClient/src/Microsoft/Data/Common/DbConnectionStringCommon.cs#L1039
However, in the .NET Framework version of the System.Data.SqlClient, the same keywords class has it defined differently:
https://referencesource.microsoft.com/System.Data/R/6a6a501b9f709c06.html
This is an example keyword, but there are more keywords where this is an issue.
Here's the code excerpt for both:
Microsoft.Data.SqlClient
...
internal const string TrustServerCertificate = "Trust Server Certificate";
...

System.Data.SqlClient
...
internal const string TrustServerCertificate = "TrustServerCertificate";
...

When a connection string is built using the Microsoft.Data.SqlClient.SqlConnectionStringBuilder, it cannot be deconstructed within the System.Data.SqlClient.SqlConnectionStringBuilder as it throws an exception indicating that the keyword is not supported.
To reproduce
Run the following code in a project that is using Microsoft.Data.SqlClient:
var csb = new SqlConnectionStringBuilder { TrustServerCertificate = true };
var cs = csb.ConnectionString;

Take the connection string from that and then apply it to a different set of code using the System.Data.SqlClient:
var connectionString = "Trust Server Certificate=True";
var csb = new SqlConnectionStringBuilder(connectionString);
var cs = csb.ConnectionString;

The following exception will occur:
System.ArgumentException: 'Keyword not supported: 'trust server certificate'.'
Expected behavior
I would expect that the keywords would match and thus the connection strings would be interchangeable.
Further technical details
N/A
Additional context
N/A

@captain-seablade that change was decided and made as a breaking change in earlier stage of M.D.SqlClient in PR #534.

Ok, thank you.  I attempted a search for anything related to breaking changes regarding this issue and I was unable to find the PR you mentioned or other supporting details.  Thank you for the information.  I'll adjust accordingly and add workarounds.

Closing as not an issue,


fix(doc): Update useful_scripts_and_tasks.adoc to include lint script

Note: If there are pending changes to the PR, prefix the PR title with "WIP" and add the label "DO NOT MERGE"
Mandatory

[ ] What does this PR do?

This Pr adds the script to docs which can be used to check for lint errors
Tslint PR: https://github.com/fabric8-ui/fabric8-planner/pull/2673

[ ] What issue/task does this PR references?

[ ] Are the tests Included?

Optional

[ ] Is the documentation Included?

[ ] Are the Release Notes included?

[ ] @mention(s) to expected reviewer(s) included

@sahil143, @jarifibrahim  how about: Task: Lint-check, Sub task: add a hyphen(-), Description: Checks for linter error using TSLint

@sahil143 Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 and visit http://localhost:5000 to access it.

how about: Task: Lint-check, Sub task: add a hyphen(-), Description: Checks for linter error using TSLint

@Preeticp looks good but there is going to be npm run lint-fix also or something similar to this.

@sahil143 Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 and visit http://localhost:5000 to access it.

@sahil143 Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 and visit http://localhost:5000 to access it.

@Preeticp I think this is good. Let's go with this one.

@sahil143 Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2687 and visit http://localhost:5000 to access it.

[test[

[test]

@jarifibrahim is there something that needs to be done? Or can you approve this?

@jarifibrahim is there something that needs to be done? Or can you approve this?

I've added a small comment. I'll approve this PR once it is addressed.


commons-io in redpen-server/pom.xml

Do we need to add commons-io in redpen-server/pom.xml?
mvn jetty:run in my new box works well after adding the follows:
<dependency>
  <groupId>org.apache.commons</groupId>
  <artifactId>commons-io</artifactId>
  <version>1.3.2</version>
</dependency>

I will close this issue since there is no report on this problem from users.


Add categories to Cargo.toml

Hi! crates.io now supports categories, which are a curated list
of topics aimed at helping an end-user coming to crates.io looking for
"a crate to do ______".
We're sending pull requests to selected crates to add categories in order to help
populate the categories and seed their usefulness. We've made a guess at the best
category/categories for this crate; if it doesn't fit, please feel free to take
a look at all the available categories and their descriptions and
the slug values that should be specified in your Cargo.toml and pick
different ones. If you have a category in mind that isn't available, you can
send a PR to this file on crates.io to propose additional
categories.
Crates can have up to 5 categories, and uploading categories to crates.io
currently requires publishing a new version with a cargo nightly from 2017-01-18
or later (it needs to contain this PR).
We've published a blog post with further details about categories.
The blog post also talks about the new crates.io support for CI badges, which
you may be interested in adding as well.
Please let me know if you have any questions!

Do older cargo versions ignore the categories in the toml, that is, is it backwards compatible?
@kamalmarhubi, @posborne I think the proposed category fits perfectly. I was thinking about addingos::macos-apis, but realised that we do not support MacOS-APIs, but *nix-APIs on MacOS.

Do older cargo versions ignore the categories in the toml, that is, is it backwards compatible?

Yep! That's actually the reason you must have a new version of Cargo to publish categories. Cargo has a whitelist of properties that it sends to the crates.io server.

In that case
@homu r+
Thank you

:pushpin: Commit ac57d53 has been approved by fiveop

:zap: Test exempted - status


Using C++ in Bridge.Net

Hi!
I have C++ framework which I would like to use in my Bridge.NET project. Is it possible? Is there any restrictions?
Thanks!

I don't think this would be possible. Bridge can only support parsing C# libraries.

What about managed C++/CLI? Is it possible to refer to C++/CLI project in Bridge.NET project?

Are you going to support parsing C++ libraries?

Hey @xa4atur you might be interested in http://webassembly.org/ - maybe when webassembly is production ready, you could link your javascript libraries to web assembly, just like c# com visible to c++/c

@samuelGrahame thank you, but unfortunately webassembly is not suitable for me

technically, if you had the source code of the c++ library you could convert it to c#

@samuelGrahame I can convert C++ code to C# using SWIG or wrap it using C++/CLI, but I need t o link source C++ classes in that cases. And how Bridge.NET will convert SWIG or C++/CLI code to JavaScript?

Some C++ libraries should not be converted  to JavaScript, beacuse I am going to use them on server side. I hope I can create C++/CLI project and refer to it from Server project, can I?

I don't see a problem in running the c++ libraries from server side, but I believe you'll have a binary wrapper on the server. So they will be pretty much two separate/built projects.
Bridge could build from C# the client-side code, and maybe some server-side nodejs code, but then at some point a call will have to happen to your binary server/side code, linked to the c++ libraries.
But there's no linking from javascript to DLLs, is there?
By current Bridge implementation, it is based on Roslyn, which is a C# and VB analyzer. But can't say it is impossible, just not implemented currently.
So in summary:

bridge can build javascript code out of c# code
it can't from vb, but shouldn't be too far from supporting it, just not tested
there's no implementation to support c++ in bridge currently, but that would be a matter of mapping the roslyn calls to a c++ code analyzer and wiring things up (the challenge here is uncertain)
there's a list of code analyzers and their github project pages from microsoft here: http://microsoft.github.io/CodeAnalysis/ (but I see the c++ part says "it will be placed here", so stay tuned!)
bridge can't build javascript from binaries from any platform -- so we can't just link built DLLs -- that would be a very neat feature, but very low-level-ish and a lot of abstraction like loops, objects, would have to be "guessed" from assembly instructions
it is possible to have Bridge interact with other projects not built by bridge, for example, with server-side calls, ajax and async requests (being a project a Bridge.NET project and then the other, a server-side or web service project

I hope this helps you understand the scenario. I think your goal is very interesting and can be currently attained with what Bridge has to offer -- just not in the same "Visual Studio project", but in the same VS solution. :)

@fabriciomurta thank you for detailed answer.
The point is that I have 3D webgl viewer which has written in C++. I use emscripten to convert it to JavaScript. I thought that I can use Bridge.NET to develop frontend using C# instead of JavaScript.
And I assumed that I can convert my C++ viewer to C# via SWIG or C++/CLI in order to use in client side.


Fix breakage when multiple default routes are declared

About
Force parsing of single interface definition.
The issue
I'm not sure if this is normal / invalid but I was getting the following error due to having multiple default destinations
> npx -p @sitespeed.io/throttle throttle 3gslow
Using profile 3gslow
Error: Command failed: sudo tc qdisc add dev wlp0s20f3wlp0s20f3 ingress
Cannot find device "wlp0s20f3wlp0s2"

    at ChildProcess.exithandler (node:child_process:326:12)
    at ChildProcess.emit (node:events:369:20)
    at maybeClose (node:internal/child_process:1067:16)
    at Process.ChildProcess._handle.onexit (node:internal/child_process:301:5) {
  killed: false,
  code: 1,
  signal: null,
  cmd: 'sudo tc qdisc add dev wlp0s20f3wlp0s20f3 ingress',
  stdout: '',
  stderr: 'Cannot find device "wlp0s20f3wlp0s2"\n'
}

> sudo route
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
default         OpenWrt.lan     <IP_ADDRESS>         UG    302    0        0 wlp0s20f3
default         OpenWrt.lan     <IP_ADDRESS>         UG    600    0        0 wlp0s20f3
<IP_ADDRESS>      <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 docker0
<IP_ADDRESS>      <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 br-672d84f52503
<IP_ADDRESS>      <IP_ADDRESS>         <IP_ADDRESS>     U     0      0        0 br-0f9d5dc90dec
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>   U     302    0        0 wlp0s20f3
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS>   U     600    0        0 wlp0s20f3
<IP_ADDRESS>    <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 br-bb989ac7cf64
<IP_ADDRESS>   <IP_ADDRESS>         <IP_ADDRESS>   U     0      0        0 virbr0

Thank you @andyrichardson for the fix!


check if serverless IS_LOCAL is set

This solves Serverless invoke local command issue #83

@adriangmweb Thank you very much for this PR, I had a change to test it and indeed it solves the issue when running with serverless offline.
Pinging @alixaxel in case he has a chance to review it too ;)
Thanks both

@chemalopezp Thanks, glad this can help others as well.
Probably you are already using some fix, but just in case there is a temporal workaround until @alixaxel has a chance to take a look.
Just set chromium.executablePath to null when IS_LOCAL present, and don't forget to set headless to false, otherwise it will use a normal chrome instance.
const executablePath = (process.env.IS_LOCAL) ? null : await chromium.executablePath
this.browser = await chromium.puppeteer.launch({
  executablePath: executablePath,
  args: chromium.args,
  defaultViewport: chromium.defaultViewport,
  headless: true
});

Thank you so much @adriangmweb , that works like a charm!

I am currently using https://github.com/adieuadieu/serverless-chrome but it doesn't support node 10.x and 12.x, so I'm looking for an alternative.
I'm considering using this, but was wondering if you had to do anything special with regards to using the serverless framework.  Are you using any specific serverless plugins?  webpack, chrome-local, etc?
Thanks for any additional info you can share.

Hi @tcollins, you don't need any plugin to make it work with serverless. Just using a light library to manage it like puppeter-core it works fine.

Thanks for the quick reply.  I'll give this a try.


Deps report bug

All the deps are up to date, and yet we have a wrong warning that 100+ of them are out of date, probably img.shields.io or libraries.io have a bug...

fixed


Error when installing via fusepm: "Found 0 unoproj files. Wanted 1"

When I did the fusepm install command, I got an error:

I just installed fusepm and now tried as can be seen above. Running Win7 64bit.

You need to run fusepm in a directory with a fuse project.


Zsh support?

Hi, I don't know how completions work internally and whether completions for Zsh are something completely different than completions for Bash but it would be awesome if this package worked in both shells. Is that technically possible?
BTW, these Yarn completions are way more advanced than anything else I have seen (for zsh at least), for example, parsing workspaces from package.json as per https://github.com/dsifford/yarn-completion/issues/29 is the right thing to do while most other completions don't support Yarn workspaces at all.

Officially supported zsh completions can be found here.
Unfortunately, the completions for bash and zsh are implemented in an entirely different way, so there's no real great way to interop (that I'm aware of).
See #30 for discussion on some issues with parsing workspaces using package.json.

For anyone interested, I managed to make it work this way:

Add this shopt script somewhere in your $PATH.
autoload bashcompinit && bashcompinit
source yarn-completion.bash

@dsifford if you don't mind I'll use this issue to gather pieces of information about the possible Zsh support. (Disclaimer: I know very little about how the completions work internally in either Bash or Zsh so won't probably be able to help myself.)
Current situation:

At v0.12.0, I'm able to run the completions partially with bashcompinit as per the comment above.
At v0.13.0, it's necessary to manually add a Zsh workaround as described here: https://github.com/dsifford/yarn-completion/commit/4d4089296c032419968da68e4cf8086b602b51a8#r33142870
Even with this, I don't get fully working completions, see https://github.com/dsifford/yarn-completion/commit/4d4089296c032419968da68e4cf8086b602b51a8#r33148720.

What I found in Zsh docs is simply this:

The function bashcompinit provides compatibility with bash’s programmable completion system. When run it will define the functions, compgen and complete which correspond to the bash builtins with the same names. It will then be possible to use completion specifications and functions written for bash.

I also have zsh-users/zsh-completions installed which has this file in it, which seems to implement the compgen and complete functions as well. I'm not sure if it's loaded and how, and how it interacts with the built-in bashcompinit from Zsh.
I also don't know how to troubleshoot issues with autocompletion – the suggestions are simply weird but there are no error messages anywhere.

@borekb Open up a new issue so others can find it easier. That way we might get lucky and have someone stumble upon it and help.
Re: your disclaimer -- I had no idea how bash completions worked and have never wrote a bash completion script in my life prior to making this one. It's tricky at first to understand, but I'm proof that it's possible. Don't sell yourself short.

If it was writing completions for a single known environment, maybe I could try to do that but this sounds like troubleshooting of some (delicate?) interactions / integrations between Zsh and Bash which I really don't think I'd be capable of 😄.
How about re-opening this issue? "Zsh support" as a title sounds pretty good to me.

Whoops, sorry... Thought this was the PR I commented on.
Yep, we can reopen this one. :+1:


Can ECONNRESET errors be retried?

See the screenshot. We get this error occasionally. Can some retry be added to this github action? Thanks!

Yup it certainly can be, I'll try take a look next week

No rush!

That was fast 💨

No problem :)
Let me know if you see the issue again


Odd results in the north

Some north GSPs look like that have some odd results. Is the GSP installed capacity correct?
Are the PV capacity correct in this region?

PV systems over their installed capacity are

21570
26246 - this one was a factor of 1000 out. This produced 2KW today, so i suspect the installsed capacity is wrong. I
31970

26246
This produced 2KW today, so i suspect the installed capacity is wrong. It was 0.046
I changed it to 2.5. Note this pv system was in the region where we saw unusually high values.


feat: add files relevant for the community

Description
As the central point for documentation for the community, this commit moves relevant files into the repository.
Fix https://github.com/kubewarden/kubewarden-controller/issues/685

can we add https://github.com/kubewarden/rfc/blob/main/rfc/0013-governance.md to the PR?

can we add https://github.com/kubewarden/rfc/blob/main/rfc/0013-governance.md to the PR?

Oh, I was looking for the governance file. I did not remember that is a RFC. Nice! I'll added.


Increase BTG mining fee

0.0001 triggers an HTTP 403 Bad Request '66: min relay fee not met. Code:-26' on https://btgexplorer.com/api/tx/send for a transaction having 40 inputs and 1 output (11438 bytes). Increasing the fee to 0.00015 fixes this issue. At currect exchange rates, this fee increase is approx 0.02 USD per transaction: 385 * 0.00005

Closing this to fix a typo.


MULTIPART_UNMATCHED_BOUNDARY=1 functionality is broken

This pull request is not intended for near-term merging, but rather, to illustrate one of the problems with current v3 functionality for MULTIPART_UNMATCHED_BOUNDARY.
(This pull request is not intended to illustrate cases where either a) the value should be set to '2' but is not or b) the value is incorrectly set to '2' when it should be '0'.)
Rule 200004 as provided in modsecurity.conf-recommended tests:
SecRule MULTIPART_UNMATCHED_BOUNDARY "@eq 1" ...

In current v3/master, if there are 'n' apparent boundaries (basically lines beginning with '--'), then if any of the 2nd through n-2-th are non-matches, they will fail to result in MULTIPART_UNMATCHED_BOUNDARY=1.
E.g. if there are a total of 10 apparent boundaries, if any of the 2nd, 3rd, 4th, 5h, 6th, 7th or 8th of them should signal as 'unmatched', MULTIPART_UNMATCHED_BOUNDARY will not get set to 1.
This is illustrated in the 3 new tests in this file. They all fail in current v3/master but they all pass in v3/master as at June 7,2018.

Hi @martinhsv,
thanks for the examples.

This pull request is not intended for near-term merging, but rather, to illustrate one of the problems with current v3 functionality for MULTIPART_UNMATCHED_BOUNDARY.

I've made a quick research about term of "unmatched multipart boundary". Unfortunately I found this expression only at ModSecurity, so I'm a bit disappointed, because I don't know the exact meaning of this expression.

(This pull request is not intended to illustrate cases where either a) the value should be set to '2' but is not or b) the value is incorrectly set to '2' when it should be '0'.)

May be that would be good to see these cases too. I mean I'm curious when this value has set up to 2 but it wouldn't be, or reverse.

Rule 200004 as provided in modsecurity.conf-recommended tests:
SecRule MULTIPART_UNMATCHED_BOUNDARY "@eq 1" ...

In current v3/master, if there are 'n' apparent boundaries (basically lines beginning with '--'), then if any of the 2nd through n-2-th are non-matches, they will fail to result in MULTIPART_UNMATCHED_BOUNDARY=1.

Yes, and I think that's wrong. Consider the line which begins with --, but that's not a boundary, just a signature separator from an e-mail body (eg. you want to protect your webmail with ModSecurity).
Like:
Content-Type: multipart/form-data; boundary=--0000
----0000
Content-Disposition: form-data; name="_msg_body"

Hi Martin,

this is the test message.

Regards,

--
airween
----0000--

I can't consider the line -- as boundary, but the engine does. This is wrong.

E.g. if there are a total of 10 apparent boundaries, if any of the 2nd, 3rd, 4th, 5h, 6th, 7th or 8th of them should signal as 'unmatched', MULTIPART_UNMATCHED_BOUNDARY will not get set to 1.

I think the keyword here is apparent boundaries - no, they are not boundaries. I think we must consider them as regular lines. Every line which begins with --, but not matched with the boundary string is a regular content.

This is illustrated in the 3 new tests in this file. They all fail in current v3/master but they all pass in v3/master as at June 7,2018.

Yes, that's what I fixed.
Please take a look to my example with "e-mail signature separator".
Btw: thanks again your works, I think this is a good chance to re-think the terms - and the expected behavior, of course. As I wrote above, I didn't find any definition of "unmatched boundary" outside of ModSecurity. Also as you see my opinion, these lines are not unmatched boundaries.


Error starting userland proxy: listen tcp4 <IP_ADDRESS>:51821: bind: address already in use

Have an issue while during Installation, OS: Ubuntu 24.04.1 LTS x86_64
root@localhost:~# git clone https://github.com/xtrime-ru/antizapret-vpn-docker.git antizapret
cd antizapret
docker compose pull
docker compose up -d
Cloning into 'antizapret'...
remote: Enumerating objects: 831, done.
remote: Counting objects: 100% (261/261), done.
remote: Compressing objects: 100% (150/150), done.
remote: Total 831 (delta 159), reused 132 (delta 108), pack-reused 570 (from 1)
Receiving objects: 100% (831/831), 124.66 KiB | 5.94 MiB/s, done.
 Thank you

seems like port is already used by another process

nothing else except antizapret is runnig

seems like port is already used by another process

nothing else except antizapret is runnig

Show your docker-compose.wireguard.yml

Show your docker-compose.wireguard.yml

services:
  antizapret-vpn:
    extends:
      file: docker-compose.yml
      service: antizapret-vpn
  wg-easy:
    environment:
      - WG_ALLOWED_IPS=
          <IP_ADDRESS>/8,
          <IP_ADDRESS>/22,
          <IP_ADDRESS>/21,
          <IP_ADDRESS>/24,
          <IP_ADDRESS>/30,
          <IP_ADDRESS>/20,
          <IP_ADDRESS>/21,
          <IP_ADDRESS>/22,
          <IP_ADDRESS>/19,
          <IP_ADDRESS>/19,
          <IP_ADDRESS>/24
      - WG_DEFAULT_DNS=<IP_ADDRESS>
      - WG_PERSISTENT_KEEPALIVE=25
      - WG_PORT=443
    image: ghcr.io/wg-easy/wg-easy
    container_name: wg-easy
    volumes:
      - ./.etc_wireguard:/etc/wireguard
      - ./wireguard/entrypoint.sh:/entrypoint.sh
    env_file:
      - ./wireguard/wireguard.env
    command: /entrypoint.sh
    ports:
      - "443:443/udp"
      - "51821:51821/tcp"
    restart: unless-stopped
    cap_add:
      - NET_ADMIN
      - SYS_MODULE
    depends_on:
      - antizapret-vpn
    sysctls:
      - net.ipv4.ip_forward=1
      - net.ipv4.conf.all.src_valid_mark=1
    logging:
      driver: "json-file"
      options:
        max-size: "100k"
        max-file: "2"root@localhost:~/antizapret#

i think i did not change nothing

Change the Web panel port:
services:
  antizapret-vpn:
    extends:
      file: docker-compose.yml
      service: antizapret-vpn
  wg-easy:
    environment:
      - WG_ALLOWED_IPS=
          <IP_ADDRESS>/8,
          <IP_ADDRESS>/22,
          <IP_ADDRESS>/21,
          <IP_ADDRESS>/24,
          <IP_ADDRESS>/30,
          <IP_ADDRESS>/20,
          <IP_ADDRESS>/21,
          <IP_ADDRESS>/22,
          <IP_ADDRESS>/19,
          <IP_ADDRESS>/19,
          <IP_ADDRESS>/24
      - WG_DEFAULT_DNS=<IP_ADDRESS>
      - WG_PERSISTENT_KEEPALIVE=25
      - WG_PORT=443
      - PORT=51811
    image: ghcr.io/wg-easy/wg-easy
    container_name: wg-easy
    volumes:
      - ./.etc_wireguard:/etc/wireguard
      - ./wireguard/entrypoint.sh:/entrypoint.sh
    env_file:
      - ./wireguard/wireguard.env
    command: /entrypoint.sh
    ports:
      - "443:443/udp"
      - "51811:51811/tcp"
    restart: unless-stopped
    cap_add:
      - NET_ADMIN
      - SYS_MODULE
    depends_on:
      - antizapret-vpn
    sysctls:
      - net.ipv4.ip_forward=1
      - net.ipv4.conf.all.src_valid_mark=1
    logging:
      driver: "json-file"
      options:
        max-size: "100k"
        max-file: "2"

I also recommend looking at what is occupying port 51821:
netstat -ltnp | grep -w 51821

Just now figured out that my vps ssh connection use port 51821 - after changing port number problem was fixed, thank you very much!


auto populate youth profile based on parents

youths profile information such as country and city can be pre filled when they are being created based on their parents information to make set up simpler

@rosalanghammer could be picked for end of year

No such data is required anymore, closing


Fix upgrade for all daemonset type resources

Daemonsets cannot be simply upgraded through a single API call,
regardless of any kubectl documentation. The resource must be
purged and then recreated in order to make any changes.

ci check this


system: recovery script cannot cope with LUA additions in pkg

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I am convinced that mine is new.

Is your feature request related to a problem? Please describe.
FreeBSD package manager started using LUA scripts which we cannot use in the recovery stage because there is no portable way to rerun package installation scripts. For now the serious parts have not been ported to LUA, but when it happens we should be ready.
Describe the solution you like
We cannot install LUA and push the scripts through because a "pkg" object exists with additional functionality which is only provided by libpkg. For this reason we either have to update an existing pkg tool to do what we need or add a new one and link it to libpkg) so we can call the scripts via C functions.
Describe alternatives you considered
Creating a non-pkg LUA env and keeping it up to date which could always break.
Additional context
N/A

Note to self: the issue isn't very pressing as lua only seems to be used for shells and sample files. What we are after are user and group modifications and these continue to work for the time being.

There is https://man.freebsd.org/cgi/man.cgi?query=pkg-lua-script now it seems.


Issue with Number of pages

After enabling scrollbars getting a conflict with pageSize and limit, this issue leading pagination to less reducing the number of pages.
In my case, total rows are 100, per page limit is 10 so I should get 10 pages, but I am getting only 9 pages.
why because it is forcibly setting pageSize as 12, so I am unable to get remaining page number as well as records.

@MorlaRamakrishna The page number starts at 0, so that could explain why you are missing one.

Yes, you are right @mvhecke
In my case, I am getting only 9 pages (i.e. Total number of pages are 9).
Reason:
The number of pages are dividing based on pageSize not limit

Any update on this?

@MorlaRamakrishna Can you reproduce it? I have a starter template made http://plnkr.co/edit/k9NCZWOMzN763ZhF5WkW?p=preview


[US2366055] Annotated Enum Support in LabVIEW MeasurementService

[X] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

Added the "Create Enum Type Specialization.vi" helper VI to allow users to convert enums to Type Specializations.
Updated "Get MetaData from Results.vi" to handle annotations in the same way that "Get MetaData from Configuration.vi" already does.
Updated "Type Specialization Key.ctl" and "Type Specialization.ctl" to contain entries for enum and enum.values.
Updated documentation comments in "Get Type Specializations.vi" in examples to cover enums.

Why should this Pull Request be merged?
AB#2366055
What testing has been done?

Manually tested by calling "Create Enum Type Specialization.vi" from "Get Type Specializations.vi" in both examples.

Added the "Create Enum Type Specialization.vi" helper VI to allow users to convert enums to Type Specializations.
Create Enum Type Specialization.vi

Updated "Get MetaData from Results.vi" to handle annotations in the same way that "Get MetaData from Configuration.vi" already does.
Get MetaData from Results.vi

Updated "Type Specialization Key.ctl" and "Type Specialization.ctl" to contain entries for enum and enum.values.
Type Specialization Key.ctl

Type Specialization.ctl

Updated documentation comments in "Get Type Specializations.vi" in examples to cover enums.
Get Type Specialization.vi.

Create Enum Type Specialization.vi

Get MetaData from Configuration.vi

Get MetaData from Results.vi

Type Specialization Key.ctl

Type Specialization.ctl

Get Type Specialization.vi
Note: The comments added (marked in red) were propagated to existing examples, but the disabled block was not as enums are not used in those examples.

Did Get MetaData from Configurations.vi change? If not, I think you should revert it.

Yes, the change was adding an output terminal for MeasurementPluginService. You made an earlier comment on Get MetaData from Results.vi about the newly added MeasurementPluginService not having an associated output and that adding one would reduce wire forks in the caller and that it should be a required input. Since I had based my changes to Get MetaData from Results.vi off of the existing code in Get MetaData from Configurations.vi, I made corresponding changes by adding an output terminal for MeasurementPluginService to Get MetaData from Configurations.vi as well when addressing that comment.
See usage here:

It looks like the linking changed when you modified the template Get Type Specializations.vi. It expects its typedefs to be in vi.lib. You will have to copy the typedefs to vi.lib, then relink them, then resave.

I think I addressed this correctly by relinking against the installed versions of the VIs in vi.lib (rather than pointing them at the ones in my local git repo), but let me know if that's not the case.
Screenshots of VIs below:
Create Enum Type Specialization.vi

Get Type Specialization.vi Template

Get MetaData from Configuration.vi

Get MetaData from Results.vi

Fixed alignment.

The block diagram comment you updated in the template, you inserted the documentation about enums in the middle of the pin.instrument_type annotation documentation.

The last line that discusses custom instruments should be grouped with the information for the built in instrument types in the pin map. You might consider adding a blank line to denote paragraphs and a natural break in topic from previous text. Since each section begins with a bold heading followed by a colon, I think each section will be easily identifiable even if a section has multiple paragraphs.

If I'm reading things correctly, we end up calling the same VIs (and measurement service plugin VI) to populate type specializations for both the configuration parameters and the output parameters. Is that correct? If so, how does the plugin implementation know whether to populate annotations for the configuration parameters vs. the output parameters? It seems like we either need to have a plugin VI for inputs vs. outputs, add an input to indicate whether we're asking for inputs or outputs, or update Parameter Type Specialization.ctl to include an indication that the specialization applies to an input parameter vs. an output parameter (and then update the server code accordingly). I don't think there's anything that would otherwise prevent an input and output parameter from having the same name. Whatever we do, we need to ensure it maintains compatibility with existing measurement services unless there is realistically no other option other than to break compatibility.

Perhaps this would be better to consider as a follow on PR, but have we considered proving a general purpose VI that given the Measurement Configuration.ctl cluster and/or the Measurment Results.ctl cluster, would automatically generate the correct annotations for enum controls and path controls. Users would still have to manually add pin control annotations since we can't figure that out automatically from the control itself. However, I'm not sure why we're making the user manually do that for path controls or making them manually specify each enum control in the cluster vs. just looking through all of the top level cluster element ourselves for enum controls. It would also eliminate have to hard code control names in multiple places.

Perhaps this would be better to consider as a follow on PR, but have we considered proving a general purpose VI that given the Measurement Configuration.ctl cluster and/or the Measurment Results.ctl cluster, would automatically generate the correct annotations for enum controls and path controls. Users would still have to manually add pin control annotations since we can't figure that out automatically from the control itself. However, I'm not sure why we're making the user manually do that for path controls or making them manually specify each enum control in the cluster vs. just looking through all of the top level cluster element ourselves for enum controls. It would also eliminate have to hard code control names in multiple places.

I think this is a great idea and definitely possible. You can get a refnum for all controls in a cluster, then use the Class Name property to figure out if the control is an enum, then pass the refnum to the VI that Wes wrote to generate the type specialization. We can do all of this without requiring the customer to do anything.
I don't know that it is worth it to go back and do this for paths considering how rare that will probably be for LabVIEW code, but I could be convinced otherwise.

If I'm reading things correctly, we end up calling the same VIs (and measurement service plugin VI) to populate type specializations for both the configuration parameters and the output parameters. Is that correct? If so, how does the plugin implementation know whether to populate annotations for the configuration parameters vs. the output parameters? It seems like we either need to have a plugin VI for inputs vs. outputs, add an input to indicate whether we're asking for inputs or outputs, or update Parameter Type Specialization.ctl to include an indication that the specialization applies to an input parameter vs. an output parameter (and then update the server code accordingly). I don't think there's anything that would otherwise prevent an input and output parameter from having the same name. Whatever we do, we need to ensure it maintains compatibility with existing measurement services unless there is realistically no other option other than to break compatibility.

We could add an annotation that specifies whether the type specialization is an input or output. That would be easy to do from LabVIEW (especially if we generate the enum type specializations automatically). I don't know if that would work from the client side, though.
I'm curious how this would work for paths and pins. Do we just assume those are always inputs? This also doesn't seem like a solely LabVIEW problem. Python would have the same problem, right?

We could add an annotation that specifies whether the type specialization is an input or output. That would be easy to do from LabVIEW (especially if we generate the enum type specializations automatically). I don't know if that would work from the client side, though.

There is no need to add an annotation to indicate whether a parameter is an input or output. The parameter metadata is already split into configuration parameters and output parameters.
The LV APIs for specifying type specializations and other annotations should identify parameters by name + input/output, not name alone.
This sounds like AB#2392402, but in LabVIEW.

I'm curious how this would work for paths and pins. Do we just assume those are always inputs? This also doesn't seem like a solely LabVIEW problem. Python would have the same problem, right?

Paths and pins are only supported for inputs. This is why output parameters didn't support annotations until enums were added.
In Python, you specify type specializations and other annotations using decorator methods which already distinguish between configuration parameters and output parameters, so I don't think the Python API has the same problem.

I don't know that it is worth it to go back and do this for paths considering how rare that will probably be for LabVIEW code, but I could be convinced otherwise.

Just because the service is written LV doesn't mean the UI will be a .vi file. Also, for auto generated UIs like in TS or was done in a Blazor prototype, the annotations might still be useful. I don't think it's necessarily a high priority, but I also don't think it's difficult so we should probably just do it for consistency.

We could add an annotation that specifies whether the type specialization is an input or output. That would be easy to do from LabVIEW (especially if we generate the enum type specializations automatically). I don't know if that would work from the client side, though.

There is no need to add an annotation to indicate whether a parameter is an input or output. The parameter metadata is already split into configuration parameters and output parameters.

The LV APIs for specifying type specializations and other annotations should identify parameters by name + input/output, not name alone.

To be explicit, the annotations at the measurement_service.proto level are a (string, string) dictionary attached to each parameter. Since they are already owned by the definition of the parameter, there is no need to disambiguate them as input/output or even which parameter they apply to. That is already self evident based on the data hierarchy.
However, the LV API we created as a wrapper around that to make it easier to specify the annotations exposed things as an array of annotations for all parameters with only a parameter name as a way to identify which annotation goes with which parameter. This is where we chose poorly since the parameter name alone isn't sufficient to uniquely identify a parameter since input and output parameters can have the same name. So I think we either need to add the direction to the annotations for mapping purposes in GetMetadata where we encode that list into a dictionary of annotations attached to each parameter, or we need to make the call indicate whether annotations for inputs or outputs are being asked for and then maintain two lists for later encoding in GetMetadata.


Fix examples of NN

Examples of NN have been outdated due to the recent advances in 3D architectures

There is also a missing import to Dataset3D in NN

closed by #54


Use System needsCommit to decide whether Commit current transaction? question actually needs to be asked

I appreciate the question when I have work that would be lost if I don't commit. But the dialog is just annoying when I have committed before exiting image:

It is annoying to commit, then immediately exit and get the question.
However, it's not an actual problem and I'm used to being annoyed by this in GBS.
It gets a little more annoying if you have autocommit on.  Shouldn't it automatically commit if so?
However, I don't think this is necessarily worth a lot of effort.

I believe that this dialog only comes up if there are unsaved changes. So, what changes happen behind the scenes after you do an explicit commit?

JadeiteTranscript>>onCloseRequested did not send System>>needsCommit but does as of 3.0.77.


Install flash-attn requirements

Adds flash-attn to be installed as part of the Dockerfile.
I've pointed this to be merged into #2 to keep the diff clean, but this means ideally this should be merged first, followed by single-or-multi-neox-write (or I can just rebase afterwards).

It turns out flash-attn version 1.0.1 is needed to avoid this error (thanks @edchapman88 for documenting the fix):
RuntimeError: Expected is_sm8x || is_sm75 to be true, but got false.  (Could this error message be improved?  If so, please report an enhancement request to PyTorch.)

So I'll update this PR to include this change as well. I'll set it to be Draft in the meantime.

Merged in with @edchapman88's consent.


add getters to private properties, re: #148

so the user can do something with the saved screenshots.

Can you accept this PR, or give me some guidance on how to improve it?  Thx.

@tacman, you can't use the BROWSER_SCREENSHOT_DIR env var?
I think it would be better to update HasBrowser::pantherBrowser() to accept these as options (I don't think the following works right now):
$this->pantherBrowser(['screenshot_dir' => 'some/dir']);

I'm really trying to keep the public api of browsers confined to just actions/assertions if possible.


terminated with exit code: 2.

Type: Performance Issue
I'm encountering a problem while trying to run PowerShell in Visual Studio Code (VSCode). Whenever I attempt to open a terminal or execute PowerShell commands within VSCode, I receive the following error message:
"C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe" terminated with exit code: 2
1)I have tried restarting VSCode, restarting my computer, and reinstalling VSCode, but the issue persists.
2)This issue started occurring recently, and I haven't made any significant changes to my system configuration.
3) i have also try to set terminal.integrated.defaultProfile.windows from default to  windows powershell in vscode settings but it doesn't work
VS Code version: Code 1.87.2 (863d2581ecda6849923a2118d93a088b0745d9d6, 2024-03-08T15:20:17.278Z)
OS version: Windows_NT x64 10.0.22631
Modes:
Extensions (35)

Extension
Author (truncated)
Version

rc-beautify
Ato
1.2.0

blackbox
Bla
1.2.82

vscode-toggle-quotes
Bri
0.3.6

simple-react-snippets
bur
1.2.7

marko-beautify
bus
2.0.0

turbo-console-log
Cha
2.10.4

es7-react-js-snippets
dsz
4.4.3

vscode-html-css
ecm
2.0.9

prettier-vscode
esb
10.4.0

auto-rename-tag
for
0.1.10

code-runner
for
0.12.1

randomeverything
hel
0.0.3

myBeautify
Isa
0.3.7

vscode-gutter-preview
kis
0.30.0

remote-containers
ms-
0.348.0

remote-ssh
ms-
0.109.0

remote-ssh-edit
ms-
0.86.0

remote-wsl
ms-
0.86.0

vscode-remote-extensionpack
ms-
0.25.0

remote-explorer
ms-
0.4.3

remote-server
ms-
1.5.1

java
red
1.28.1

LiveServer
rit
5.7.9

html-to-css-autocompletion
sol
1.1.2

open-in-browser
tec
2.0.0

vscode-lldb
vad
1.10.0

incredibly-in-your-face
Vir
1.0.2

vscode-java-debug
vsc
0.57.0

vscode-java-dependency
vsc
0.23.6

vscode-java-pack
vsc
0.25.15

vscode-java-test
vsc
0.40.1

vscode-maven|vsc|0.44.0
console-ninja|Wal|1.0.297
JavaScriptSnippets|xab|1.8.0
chat-gpt|zha|1.6.63

is there a reason you're using windows powershell? I'd suggest updating and using a different powershell.

you can try changing your terminal.integrated.profiles.windows to fix this, providing the path to a newer version of pwsh

I've updated the terminal.integrated.profiles.windows to point to a newer pwsh version, but the issue is not resolved.


gorm:"auto_preload" struct tag

A version of the documentation found at docs.gorm.io shows the ability to use a struct tag to automatically preload certain associated models via a callback. I can't seem to get that to work. Is this the intended behavior?
The scope tag DOES work, but its easier in my case (a composite) to state what I would like to load instead of everything I wouldn't like to load and it would be best if that happened automatically.
What version of Go are you using (go version)?
1.9.3
Which database and its version are you using?
SQLite3
Please provide a complete runnable program to reproduce your issue. IMPORTANT
Sample code here
Also attached here

@JustonDavies doc.gorm.io is old, please see gorm.io .
For more detail, please see #1451 .

I think Auto Preload behavior has not changed.
#1451 was merged into master branch in 4 May 2017.
This commit(Document Auto Preload) ff8709de is 12 Feb 2018.


Detecting unreachable code via NoReturn does not work for most dunder methods

 Description
Detecting unreachable code via annotations with NoReturn does not work for most dunder methods. It works for __init__, and __call__ and gives another error for __iter__ but does not work for:

context manager methods (both sync and async)
__getattr__, __setattr__ and properties
__getitem__ and __setitem__
__str__, __len__, __bytes__ and friends

 Environment data

Language Server version: 2022.11.30
OS and version: darwin x64
Python version (and distribution if applicable, e.g. Anaconda): 3.10.1

 Code Snippet

from typing import NoReturn

class FailEnter:

    def __enter__(self) -> NoReturn:
        ...

    def __exit__(self) -> None:
        ...

class FailExit:

    def __enter__(self):
        ...

    def __exit__(self, et, ev, tb) -> NoReturn:
        ...

class FailGetItem:

    def __getitem__(self, item) -> NoReturn:
        ...

class FailSetItem:

    def __setitem__(self, attr, item) -> NoReturn:
        ...

class FailStr:

    def __str__(self) -> NoReturn:
        ...

def exampleA():
    with FailEnter():
        print('here')  # should be marked unreachable
    print('there')  # should be marked unreachable

def exampleB():
    with FailExit():
        print('here')
    print('there')  # should be marked unreachable

def exampleC():
    FailGetItem()['hello']
    print('world')  # should be marked unreachable

def exampleD():
    FailSetItem()['hello'] = "world"
    print('here')  # should be marked unreachable

def exampleE():
    str(FailStr())
    print('here')  # should be marked unreachable

This is by design. The code flow analysis employed by pyright (the static type checker that underlies pylance) does not take into account implicit calls to magic methods when determining reachability. Adding such logic would be theoretically possible, but it would be a significant performance hit — likely doubling the time it takes to analyze many functions. Since it's extremely rare for dundered methods to return NoReturn, this wouldn't be a good tradeoff, and we're therefore unlikely to implement such a feature.


--quick-add-key ... encrypt usage

When I try to generate an encryption key using the --quick-add-key command, I get the following error:
gpg: Key generation failed: Wrong key usage

Generating auth and sign subkeys works just fine, but I can't seem to find any documentation about what is going wrong here. Any suggestions?

It seems that the combination of ed25519 and encrypt is what is causing issues. Is it not possible to generate a ED25519 encryption key?

Could you add a transcript for context

On 5 Oct 2022, at 23:35, Will Ruggiano @.***> wrote:

It seems that the combination of ed25519 and encrypt is what is causing issues. Is it not possible to generate a ED25519 encryption key?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

Page 45 of the pdf manual says ….
If algo or usage are given, only the primary key is created and no prompts are shown. To specify an expiration date but still create a primary and subkey use “default” or “future-default” for algo and “default” for usage. For a description of these optional arguments see the command --quick-add-key. The usage accepts also the value “cert” which can be used to create a certification only primary key; the default is to a create certification and signing key
Without knowing what you had on the command line it could be that the entered command was defaulting to creating a certification and signing key
OR …
And a bit later ….
To create an OpenPGP key from the keys available on the currently inserted smartcard, the special string “card” can be used for algo. If the card features an encryption and a signing key, gpg will figure them out and creates an OpenPGP key consisting of the usual primary key and one subkey. This works only with certain smartcards. Note that the interactive ‘--full-gen-key’ command al- lows to do the same but with greater flexibility in the selection of the smartcard keys.
This (the use of “card”) could suggest that if the word “card” was missing, GPG would not look at the smart card and be trying to figure out from the keys on the key ring which key to use (and failing)
Without knowing the command line used and also the contents of the card it is a bit difficult to interpret GPG’s terse errors
With errors like this it is best to give us the output of a GPG card status command to show us what is on the card, a key ring list and a cut and paste of the command entered plus output
Without the context it is difficult with the terseness of GPG

On 6 Oct 2022, at 08:16, Ian Stanley @.***> wrote:

Could you add a transcript for context

On 5 Oct 2022, at 23:35, Will Ruggiano @.***> wrote:

It seems that the combination of ed25519 and encrypt is what is causing issues. Is it not possible to generate a ED25519 encryption key?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.

I am using the exact commands from the guide, except with ed25519 instead of rsa4096.
gpg --batch --generate-key gen-params-ed25519
gpg --quick-add-key $FINGERPRINT ed25519 sign 1y
gpg --quick-add-key $FINGERPRINT ed25519 auth 1y
gpg --quick-add-key $FINGERPRINT ed25519 encrypt 1y

and the full error;
$ gpg --quick-add-key $FINGERPRINT ed25519 encrypt 1y
gpg: Key generation failed: Wrong key usage
gpg: Key not changed so no update needed.

You cannot use ed25519 for encryption, use cv25519 instead.

That is what I suspected. Thanks!


It seems that without a pretrained embedding input, the results become worse

Thanks for sharing your code.
I have run your code for a click-through rate prediction task in item recommendation scenario.  In my setting, each item is seen as a token and users' earlier interactions on items can be seen  as sentences. Then I use DiSAN to encode users' interaction sequence.  If I replace the pre-trained item embedding with a random initializer embedding, the results evaluated by AUC become worse sharply.
SO I wonder whether DiSAN is suitable for training a token embedding meanwhile for sentence encoding?  If not, then a good pre-trained embedding is in deed necessary for DiSAN to get the excellent performance.
For testify the difference before and after tuning the token embedding, I found that the change of embedding is subtle, especially when I calculate the top similar items with the embeddings.

Hi, @aimiyu
Have you tested the LSTM or CNN baselines on your own data? And what's the performance? Were there the same problems?

@aimiyu Have you tried the suggestions by the author?