⬆(project) upgrade cypress to version 10.11.0

Purpose
We want our project to be up to date with the latest cypress release.
Proposal

[X] upgrade cypress version to 10.11.0
[X] upgrade Javascript dependencies in package.json/yarn.lock
[X] add cypress linter plugin

Includes changes from #22. Blocked until is #22 merged.

I think review comments applies to #22! Sorry for that. Consider #22 and this PR as accepted.


Contribution bar improvements

Follow-up PR for #534

Further improvements I think we can make in a separate PR:

Render the avatars on a different place in the markup so that the overflow-hidden doesn't cut them off when the block amount is low.

Tested on desktop and mobile on chrome.
There seems to be some issues with dynamic spacing on text. But I think that is beyond the scope of the PR. The contribution bar seems to well sized though.

For mobile, the sizing seems off as well but the contribution bar seems to work well.

@ecurrencyhodler yeah those issues are not really related to the ContributionBar itself like you'd assumed, they will be addressed when we start working on #504, #505, and #506.


Pilot wrongly reports endpoint_no_pod after 1.0.4 upgrade

Describe the bug
After an update from istio 1.0.2 to 1.0.4 pilot (curl http://istio-pilot.istio-system:8080/debug/push_status) reported a lot of endpoint_no_pod errors.
However the reported services were definitely having pods attached that were healthy and ready.
Using istioctl deregister did not help as it was reporting Could not find ip %s in svc %s endpoints [...].

Restarting pilot did not clear the errors (is there a persistent cache somewhere?).
Redeploying the affected services and pods did not clear the errors
removing istio via kubectl delete and installing it again via kubectl apply cleared all errors

Expected behavior

An upgrade should not break service connectivity
If there is a cache there should be a way to clear it

Steps to reproduce the bug
Install istio 1.0.2 via kubectl apply
Upgrade to istio 1.0.4, by applying the CRDs first, followed by another kubectl apply.
Sidecars were not updated at first.
We saw a number of issues happen after this, i.e. a lot of services being reported as not ready and returning 503s. We used istioctl proxy-status (before we saw that it causes deadlocks), which reported a sync-status of 98% for all pods still having the 1.0.2 sidcar. After restarting pilot, sync got up to a 100% again. We then redeployed all pods to upgrade the sidecar.
As this did still not resolve all problems we checked curl http://istio-pilot.istio-system:8080/debug/push_status which lead us to this issue.
Version
istio 1.0.4
kubernetes 1.11.2-gke.18
Installation
Via rendered helm manifest and kubectl apply
Environment
GKE

duplicate with #10116


Add version Control

Add version control for releasing versions and add it to Documentation

Hello! Could you tell it in detail? I don't understand what version control for releasing versions mean. Thank you!

@hu-yan2023
Something like this
https://github.com/hotosm/osm-fieldwork/wiki/Version-Control
We can use python packages to manage versions efficiently ! It would be good to have proper documentation and version control on releasing new version of package


Look into switching from multipart library to better supported python-multipart

The API for this is not as clean for our simple use case.


Fixed ill output of ArrangeColumn: Use strdisplaywidth instead of s/./x/g

I'm a Japanese user of your useful CSV plugin.
When I'm editing .csv files which includes Japanese (multi-byte) characters,
the results of :ArrangeColumn were ill because of the mismatch of estimated string width within the column.
I replaced the logic of counting with the built-in function strdisplaywidth(v:val).
Then :ArrangeColumn worked very well.
Additionally, the comment " do not strip leading whitespace can be fulfilled by this change.
Thank you.
Before:

After:

Is this because of a ambiwidth characters? Can you please amend the commit and make this change only if the strdisplaywidth() function really does exist?

I see.

strdisplaywidth is a feature from Vim 7.3 (2010).

Oh, so long already? I remember when it was pretty new.
Thanks. Will merge it once #130 has been included.

thanks, merged.


Allow setting more deployment properties

Reference PR: #236
Adding deployment spec values for coordinator deployment and worker deployment:
 - progressDeadlineSeconds
 - revisionHistoryLimit
 - strategy

This is needed for us in case of an update of Trino Cluster that requires the deployment to bring pods up and down.

Sorry @nineinchnick , I re-opened a PR because I messed up the rebase and squash from #236 .

@nineinchnicklLet me know if the commit message is correct.
I looked in the previous merged PRs and saw that usually commit message and PR title were matching.

"more deployment properties" is to vague
You can have a short title and then a paragraph with more info if desired. But is should not be vague. See more info at https://trino.io/development/process#pull-request-and-commit-guidelines-


Missing Cases in _parseDateTime()

Hello,
If you have the dateTimeFormat set to "MM-dd-yyyy hh:mm AA", it is not popping up the date-time picker with the pre-populated date (only the time).  The date gets filled in with the default.
I think you are missing some cases when parsing the date portion in your if-else block within the _parseDateTime() function.  For example bArrMatchFormat[11] would (I think) cover this one but there are others missing too.

Thanks for reporting. Conditions for DateTimeFormats without seconds component were missing.


Refractor/symbol table

✋Hi guys,
Please check and review my changes:

I've removed the SymbolTable object
All the tests passed, except 3 that @melkouri is going to remove.

✋Hi guys,
Please check and review my changes:

I've removed the SymbolTable object
All the tests passed, except 3 that @melkouri is going to remove.

TO DO: After the merge, don't forget to update the changes of @melkouri in MulDiv


also to include in the instructions

Another thing we may want to consider adding to the instructions is the request that somewhere
in the report or their code they state which lab question they are referring with their research question to or how they came up with their question. This could help me with grading if they had the question or the concept wrong from the start on the prior lab. Alternatively, if we want to go with the 'please tell Dr. Gilligan or Jen your topic by Friday' route, then I can always just have a record of their topic choice and reference point that way

That's a good idea. I didn't look at this before I wrote the instructions yesterday, or I would have put it in, but I think we can finesse this by asking them to do two things:

Include it in their "tell us your topic by Friday" thing.

Incorporate it into the context in their introduction: Previous work and why is this interesting? "In a previous lab, we did this and saw that, and this raised the following questions for further investigation, which we describe here."


Backport "HBASE-26521 Name RPC spans as $package.$service/$method (#4024)" to branch-2

Signed-off-by: Duo Zhang<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+0 :ok:
mvndep
0m 26s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
4m 36s
branch-2 passed

+1 :green_heart:
compile
3m  5s
branch-2 passed

+1 :green_heart:
shadedjars
7m 26s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
4m 32s
branch-2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
4m 13s
the patch passed

+1 :green_heart:
compile
3m  1s
the patch passed

+1 :green_heart:
javac
3m  1s
the patch passed

+1 :green_heart:
shadedjars
7m 26s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
4m 24s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
181m 51s
root in the patch passed.

226m 23s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4103

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 80553cc47ca3 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / cbbb1a0542

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/testReport/

Max. process+thread count
6578 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client hbase-server . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 38s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+0 :ok:
mvndep
0m 27s
Maven dependency ordering for branch

+1 :green_heart:
mvninstall
3m 56s
branch-2 passed

+1 :green_heart:
compile
2m 54s
branch-2 passed

+1 :green_heart:
shadedjars
7m 10s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 54s
branch-2 passed

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 22s
Maven dependency ordering for patch

+1 :green_heart:
mvninstall
3m 47s
the patch passed

+1 :green_heart:
compile
2m 46s
the patch passed

+1 :green_heart:
javac
2m 46s
the patch passed

+1 :green_heart:
shadedjars
6m 50s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
3m 46s
the patch passed

_ Other Tests _

-1 :x:
unit
297m  4s
root in the patch failed.

337m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4103

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux bb6e79cc5499 4.15.0-65-generic #74-Ubuntu SMP Tue Sep 17 17:06:04 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / cbbb1a0542

Default Java
AdoptOpenJDK-1.8.0_282-b08

unit
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-root.txt

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/testReport/

Max. process+thread count
3241 (vs. ulimit of 12500)

modules
C: hbase-common hbase-client hbase-server . U: .

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-4103/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix redundant double-quoting in pgvector filters #268

This change seems to fix the issue #268 , but I want to note that it's likely a symptom, not a core issue. The current design forces the user to have a "strongly"-typed metadata implementation instead of relying on rust datatypes with macro-base Serialize/Deserialize implementations. As a thought, it may be better to update the VectorStore trait with the associated type for Metadata.

I am not sure if it’s related but running the example https://github.com/Abraxas-365/langchain-rust/blob/main/examples/vector_store_postgres.rs
has super inconsistent results. even if you pass in a list of documents and ask it questions it never gives you any close results


feat: support extended pow arithmetic

Which issue does this PR close?
Closes #4166
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?
Yes

Sorry for taking so long to get back to you on this, I'm honestly not sure about this. The current support for pow is pretty half-baked and I kind of feel rather than plowing on and piling on yet more complexity, it is worth taking a step back and reassessing how these kernels are currently setup (#3999).
I have filed #4187 to remove the powf_scalar kernel, as to be completely honest I'm not really sure why it is there at all :sweat_smile:


authentication failed

Hi, Im trying to run the example on the README, and I saw that when I tried to clone, git is asking me for authentication. I looked at the source code and I saw there were passwords being set via an environment variable, so I tried running the server again by giving an explicit user/pass:
JOSH_USERNAME="josh" JOSH_PASSWORD="josh" ./target/release/josh-proxy --local ./tmp1/ --remote=https://github.com --port=8000

but when I tried:
git clone http://localhost:8000/git/git.git:/Documentation.git

I still got:
Cloning into 'Documentation'...
Username for 'http://localhost:8000': josh
Password for 'http://josh@localhost:8000':
fatal: Authentication failed for 'http://localhost:8000/git/git.git:/Documentation.git/'

What is the right way to authenticate?

Hi Nikita,
You are raising a good point here 👍
We never used the proxy without authentication so it will always require credentials even if the upstream host does not
require them itself. I consider this a bug and will fix it.
Anyway the credentials you need to provide are the same as those you would provide for the upstream. So in this case your github http username and password.
The environment variables JOSH_USERNAME and JOSH_PROXY are not used for configuration but as a means to pass on those values down to it's own grandchild process:
josh-proxy sets them, calls git-http-backend which in turn calls an update-hook that reads the variables.

Hey thanks for the quick response, just wondering about this:

Anyway the credentials you need to provide are the same as those you would provide for the upstream. So in this case your github http username and password.

Is that necessary for it to work? I thought since the github repository is public that I should be able to clone without providing my user/pass

No it should not be necessary. It's a bug. I never noticed it because in our production deployment there are no repos with public access.


Improve dashboard and alarms

Resolves: https://linear.app/snowfork/issue/SNO-833

add units to the metrics
Link to dashboards in some operational page or Slack links

More information/details added to the dashboard for

latency
balance

Alarm rules
https://github.com/Snowfork/snowbridge/pull/1196#issue-2288992655
Alarm flow
Demo Alarm first raised in slack channel, click the alarm will link to pagerduty page, in Alarm Details there is the link to dashboard for the details.

I still "No Unit" in the diagram

Actually the unit here is blocks while the valid enums in api are as follows:
Seconds | Microseconds | Milliseconds | Bytes | Kilobytes | Megabytes | Gigabytes | Terabytes | Bits | Kilobits | Megabits | Gigabits | Terabits | Percent | Count | Bytes/Second | Kilobytes/Second | Megabytes/Second | Gigabytes/Second | Terabytes/Second | Bits/Second | Kilobits/Second | Megabits/Second | Gigabits/Second | Terabits/Second | Count/Second | None
So I think neither make sense for our use case, so instead I just add a label for the Y axis in https://eu-central-1.console.aws.amazon.com/cloudwatch/home?region=eu-central-1#dashboards/dashboard/Latency

Please check if that make sense.


[ja] Translate content/en/blog/_posts/2024-08-22-Fine-grained-SupplementalGroups-control into Japanese

This is a Feature Request

Translate "Kubernetes 1.31: Fine-grained SupplementalGroups control"

content/en/blog/_posts/2024-08-22-Fine-grained-SupplementalGroups-control into Japanese (not yet published)

What would you like to be added

Why is this needed
There is no Japanese translate for this page
Comments

/area localization
/language ja
/assign

Thanks!
/triage accepted


[,FULL VIDEO,] Subhashree Sahu Leaked Video Goes Viral On Social Media Original Video

Subhashr𝚎𝚎 Sahu L𝚎aked Original's Video 'MMS' On Social Media (X)
🌐 CLICK HERE 🟢==►► WATCH NOW
🔴 CLICK HERE 🌐==►► Download Now
Subhashree Sahu, a young and rising digital content creator, recently became the subject of a viral video that has captivated audiences across social media platforms. The video, which has taken the internet by storm, has sparked widespread conversation and engagement on platforms like X (formerly Twitter) and Instagram.
Subhashree Sahu's viral video gained immense attention due to its intriguing content, and it quickly spread across various social media channels. As a talented digital creator, Subhashree’s work has always been admired, but this recent video has pushed her popularity to new heights. Fans have taken to X Twitter to share their thoughts, reactions, and appreciation for her creativity.
Within hours of the video's release, hashtags related to Subhashree Sahu started trending on X and Instagram. The buzz around the video led to countless shares, retweets, and comments, making her a trending topic worldwide. Subhashree's followers, along with curious new viewers, flocked to her official social media handles to watch the video and stay updated on her latest posts.
On Instagram, Subhashree Sahu regularly shares snippets of her daily life, behind-the-scenes footage from her creative projects, and interacts with her fans. Her official Instagram page has seen a significant rise in followers since the video's release, with fans eagerly awaiting her next update.
Likewise, Subhashree’s official Twitter account has also seen a surge in activity. Fans have been tagging her, asking about the inspiration behind the viral video, and expressing their support. Her tweets and posts continue to engage her growing fanbase, solidifying her position as a rising star in the digital world.
Subhashree Sahu’s viral video has undeniably become a pivotal moment in her career. As more people discover her content, her fanbase continues to expand, making her one of the most talked-about digital creators of the moment. If you haven’t seen the video yet, head over to Subhashree’s official Instagram or X Twitter account to catch a glimpse of what everyone’s talking about!
Belle Delphine video Instagram
Aishah Sofey video official Twitter
,
,
,
,Subhashr𝚎𝚎 Sahu L𝚎aked Original's Video 'MMS' On Social Media (X)
🌐 CLICK HERE 🟢==►► WATCH NOW
🔴 CLICK HERE 🌐==►► Download Now
Subhashree Sahu, a young and rising digital content creator, recently became the subject of a viral video that has captivated audiences across social media platforms. The video, which has taken the internet by storm, has sparked widespread conversation and engagement on platforms like X (formerly Twitter) and Instagram.
Subhashree Sahu's viral video gained immense attention due to its intriguing content, and it quickly spread across various social media channels. As a talented digital creator, Subhashree’s work has always been admired, but this recent video has pushed her popularity to new heights. Fans have taken to X Twitter to share their thoughts, reactions, and appreciation for her creativity.
Within hours of the video's release, hashtags related to Subhashree Sahu started trending on X and Instagram. The buzz around the video led to countless shares, retweets, and comments, making her a trending topic worldwide. Subhashree's followers, along with curious new viewers, flocked to her official social media handles to watch the video and stay updated on her latest posts.
On Instagram, Subhashree Sahu regularly shares snippets of her daily life, behind-the-scenes footage from her creative projects, and interacts with her fans. Her official Instagram page has seen a significant rise in followers since the video's release, with fans eagerly awaiting her next update.
Likewise, Subhashree’s official Twitter account has also seen a surge in activity. Fans have been tagging her, asking about the inspiration behind the viral video, and expressing their support. Her tweets and posts continue to engage her growing fanbase, solidifying her position as a rising star in the digital world.
Subhashree Sahu’s viral video has undeniably become a pivotal moment in her career. As more people discover her content, her fanbase continues to expand, making her one of the most talked-about digital creators of the moment. If you haven’t seen the video yet, head over to Subhashree’s official Instagram or X Twitter account to catch a glimpse of what everyone’s talking about!


USB HID host trouble with optical mouse ICs (IDFGH-9140)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.1 master
Operating System used.
Windows
How did you build your project?
VS Code IDE
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32-S3 Korvo 3
Power Supply used.
USB
What is the expected behavior?
Hid example  mouse connect ok and show X Y values
What is the actual behavior?
, but other mouse result to fail and reboot after as minute
I (318) example: HID HOST example
E (1528) HUB: Bad transfer status: CHECK_SHORT_LANGID_TABLE
E (1528) HUB: Stage failed: CHECK_SHORT_LANGID_TABLE
I (1528) example: Interface number 0, protocol Mouse
E (1538) hid-host: hid_control_transfer(677):
E (1538) hid-host: hid_host_claim_interface(986):
Steps to reproduce.

New project from template example USB HID HOST
Build run
Connect into USB mouse
one work ok , but second with IC , that we need , fail
IC : PAW352x ...

Debug Logs.
I (318) example: HID HOST example
E (1528) HUB: Bad transfer status: CHECK_SHORT_LANGID_TABLE
E (1528) HUB: Stage failed: CHECK_SHORT_LANGID_TABLE
I (1528) example: Interface number 0, protocol Mouse
E (1538) hid-host: hid_control_transfer(677):
E (1538) hid-host: hid_host_claim_interface(986):
E (61538) USBH: Device 1 gone
E (61538) hid-host: hid_host_release_interface(1019):
E (61538) hid-host: hid_host_release_interface(1019):
E (61538) hid-host: hid_host_uninstall_device(1096):
ESP_ERROR_CHECK failed: esp_err_t 0x103 (ESP_ERR_INVALID_STATE) at 0x4200886d
0x4200886d: app_main at C:/Users/meracik/ESP_Proj1/Masterpokus/main/hid_host_example.c:374 (discriminator 1)

file: "./main/hid_host_example.c" line 374
func: app_main
expression: hid_host_uninstall_device(hid_device)

abort() was called at PC 0x40379f77 on core 0
0x40379f77: _esp_error_check_failed at C:/Users/Public/esp_master/esp-idf/components/esp_system/esp_err.c:47

Backtrace: 0x4037572e:0x3fcf3540 0x40379f81:0x3fcf3560 0x4037fd5e:0x3fcf3580 0x40379f77:0x3fcf35f0 0x4200886d:0x3fcf3620 0x4202343f:0x3fcf36a0 0x4037c52d:0x3fcf36d0
0x4037572e: panic_abort at C:/Users/Public/esp_master/esp-idf/components/esp_system/panic.c:452

0x40379f81: esp_system_abort at C:/Users/Public/esp_master/esp-idf/components/esp_system/port/esp_system_chip.c:77

0x4037fd5e: abort at C:/Users/Public/esp_master/esp-idf/components/newlib/abort.c:38

0x40379f77: _esp_error_check_failed at C:/Users/Public/esp_master/esp-idf/components/esp_system/esp_err.c:47

0x4200886d: app_main at C:/Users/meracik/ESP_Proj1/Masterpokus/main/hid_host_example.c:374 (discriminator 1)

0x4202343f: main_task at C:/Users/Public/esp_master/esp-idf/components/freertos/app_startup.c:208 (discriminator 13)

0x4037c52d: vPortTaskWrapper at C:/Users/Public/esp_master/esp-idf/components/freertos/FreeRTOS-Kernel/portable/xtensa/port.c:162

ELF file SHA256: a4a8e9e7172b9450

Rebooting...

More Information.
Demo works ok with usb keyboard and mouse ASUS ...

Hi @mmar22 ,
Thank you for the detailed description of the issue and a serial debug output log attached.
May I ask you what is the mouse model you are trying to use?

Sorry RJ we use only pcb and sensor removed from mouse case long time ago,,,
maybe Genius or Logitech. I report chip id in first post

@mmar22 No problem, thank you for the quick response.
We have a bunch of devices that can help to reproduce the scenario. Just want to make statistics and add one more device to the list.
We can use them for sure to fix the bug but I can't say how long does that take, as the problem (if there is a problem) is in the usb-related part of the code.
Will be back as soon as possible with more detailed information.

@mmar22 Hi,
We have a fix, it is in a review state right now.
Update will be delivered soon enough, stay connected.
Thanks for your contribution.

Thanks for try fix, i report this because it mark some problem in usb stack.
One more question.
I try implement this demo into lvgl platformio arduino project.
Seems work , but i need some recommendations for task priorities.
Now i combine tasks with loop and in hid host example is started next threads.
Can you explain me levels in detail ?
`    task_created = xTaskCreate(handle_usb_events, "usb_events", 4096, NULL, 2, &usb_events_task_handle);
assert(task_created);
// hid host driver config
const hid_host_driver_config_t hid_host_config = {
    .create_background_task = true,
    .task_priority = 5,
    .stack_size = 4096,

`

@mmar22 No problem.
Fixes is already in a review state, so we are waiting them pretty soon, I hope.
Yes, sure.
Briefly: usb_events task is for usb related events and process usb related tasks (such as data exchange), hid_host driver task is for events, that is connected to HID level (e.g. HID device attaching or device removing).
As far as that question is not really related to the problem with the hub and optical mouse it is better to open new issue in case there are still questions about level distribution in USB to get a better or more detailed answer.
Thanks for understanding.

Hi @mmar22 ,
The changes is in the master right now.
Now the mouse should work well.

UPD: That changes has been merged internally and it may take a few days to get public in GitHub.

@roma-jam Is this fixed? Could you share the commit id?

@AxelLin There were a bunch of issues with the similar problems at a first glance. But they are a little bit different.

For "HUB: Bad transfer status: CHECK_SHORT_LANGID_TABLE" this one: 770961d2b8973c86602044077e478b12bd9a0555
For "HUB: Bad transfer status: CHECK_FULL_DEV_DESC" this one: be899f665390ee8805062d129ae1aa0a966a3795

But in my previous comment I am talking especially about fix 1.


Cannot import imod on MacOSC (arm)

In GitLab by @hydrocomputing on Nov 29, 2023, 14:48
I installed imod on my Mac with am M1 processor.
Python version: 3.11.6
Install command: mamba install imod
After:
import imod

I get this error message:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[1], line 1
----> 1 import imod

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/__init__.py:4
      1 import pkg_resources
      3 # exports
----> 4 from imod import (
      5     couplers,
      6     data,
      7     evaluate,
      8     flow,
      9     mf6,
     10     msw,
     11     prepare,
     12     select,
     13     util,
     14     visualize,
     15     wq,
     16 )
     17 from imod.formats import gen, idf, ipf, prj, rasterio
     19 # version

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/couplers/__init__.py:1
----> 1 from imod.couplers.metamod import MetaMod

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/couplers/metamod/__init__.py:1
----> 1 from imod.couplers.metamod.metamod import MetaMod
      2 from imod.couplers.metamod.node_svat_mapping import NodeSvatMapping
      3 from imod.couplers.metamod.rch_svat_mapping import RechargeSvatMapping

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/couplers/metamod/metamod.py:6
      2 from typing import Optional, Union
      4 import tomli_w
----> 6 from imod.couplers.metamod.node_svat_mapping import NodeSvatMapping
      7 from imod.couplers.metamod.rch_svat_mapping import RechargeSvatMapping
      8 from imod.couplers.metamod.wel_svat_mapping import WellSvatMapping

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/couplers/metamod/node_svat_mapping.py:4
      1 import numpy as np
      2 import xarray as xr
----> 4 from imod import mf6
      5 from imod.couplers.metamod.mappingbase import MetaModMapping
      6 from imod.msw.fixed_format import VariableMetaData

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/mf6/__init__.py:28
     21 from imod.mf6.ims import (
     22     Solution,
     23     SolutionPresetComplex,
     24     SolutionPresetModerate,
     25     SolutionPresetSimple,
     26 )
     27 from imod.mf6.ist import ImmobileStorageTransfer
---> 28 from imod.mf6.model import GroundwaterFlowModel, GroundwaterTransportModel
     29 from imod.mf6.mst import MobileStorageTransfer
     30 from imod.mf6.npf import NodePropertyFlow

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/mf6/model.py:15
     12 import xugrid as xu
     14 import imod
---> 15 from imod.mf6 import qgs_util
     16 from imod.mf6.pkgbase import Package
     17 from imod.mf6.validation import validation_model_error_message

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/mf6/qgs_util.py:10
      7 from matplotlib import cm, colors
      9 import imod
---> 10 import imod.qgs as qgs
     11 from imod import declxml as xml
     14 def _get_color_hexes_cmap(n, cmap_name="magma"):

File ~/mambaforge/envs/imode/lib/python3.11/site-packages/imod/qgs.py:30
     21     # srsid: int = 0
     22     # srid: int = 0
     23     # authid: str = ""
     24     # description: str = ""
     25     # projectionacronym: str = ""
     26     # ellipsoidacronym: str = ""
     27     geographicflag: bool = False
---> 30 @dataclass
     31 class DestinationSrs(Aggregate):
     32     spatialrefsys: SpatialRefSys = SpatialRefSys()
     35 @dataclass
     36 class ProjectCrs(Aggregate):

File ~/mambaforge/envs/imode/lib/python3.11/dataclasses.py:1230, in dataclass(cls, init, repr, eq, order, unsafe_hash, frozen, match_args, kw_only, slots, weakref_slot)
   1227     return wrap
   1229 # We're called as @dataclass without parens.
-> 1230 return wrap(cls)

File ~/mambaforge/envs/imode/lib/python3.11/dataclasses.py:1220, in dataclass.<locals>.wrap(cls)
   1219 def wrap(cls):
-> 1220     return _process_class(cls, init, repr, eq, order, unsafe_hash,
   1221                           frozen, match_args, kw_only, slots,
   1222                           weakref_slot)

File ~/mambaforge/envs/imode/lib/python3.11/dataclasses.py:958, in _process_class(cls, init, repr, eq, order, unsafe_hash, frozen, match_args, kw_only, slots, weakref_slot)
    955         kw_only = True
    956     else:
    957         # Otherwise it's a field of some type.
--> 958         cls_fields.append(_get_field(cls, name, type, kw_only))
    960 for f in cls_fields:
    961     fields[f.name] = f

File ~/mambaforge/envs/imode/lib/python3.11/dataclasses.py:815, in _get_field(cls, a_name, a_type, default_kw_only)
    811 # For real fields, disallow mutable defaults.  Use unhashable as a proxy
    812 # indicator for mutability.  Read the __hash__ attribute from the class,
    813 # not the instance.
    814 if f._field_type is _FIELD and f.default.__class__.__hash__ is None:
--> 815     raise ValueError(f'mutable default {type(f.default)} for field '
    816                      f'{f.name} is not allowed: use default_factory')
    818 return f

ValueError: mutable default <class 'imod.qgs.SpatialRefSys'> for field spatialrefsys is not allowed: use default_factory

In GitLab by @JoerivanEngelen on Dec 1, 2023, 12:03
Thanks for posting this issue.
I unfortunately botched the conda-forge recipes for Python 3.11. In theory 3.11 should be supported by iMOD Python 0.15.0, but as I haven't thoroughly tested it, I limited it to 3.10. A side-effect of this I didn't think was that this caused an older version of iMOD Python to be installed (which has not set the upper limit yet).
I guess I have to fix a bunch of versions to set the upper limit properly. Relevant documentation: https://conda-forge.org/docs/orga/guidelines.html#fixing-broken-packages
For now, I advise you to install iMOD Python on Python 3.10, until this is resolved.
BTW: This is independent of the OS

In GitLab by @JoerivanEngelen on Dec 4, 2023, 14:32
marked this issue as related to #676

In GitLab by @JoerivanEngelen on Dec 1, 2023, 12:05
marked this issue as blocked by #350

In GitLab by @JoerivanEngelen on Dec 11, 2023, 14:02
Fixes patched in https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/606


OSS-Fuzz Integration

My name is McKenna Dallmeyer and I would like to submit dateparser to OSS-Fuzz. If you are not familiar with the project, OSS-Fuzz is Google's platform for continuous fuzzing of Open Source Software. In order to get the most out of this program, it would be greatly beneficial to be able to merge-in my fuzz harness and build scripts into the upstream repository and contribute bug fixes if they come up. Is this something that you would support me putting the effort into?
Thank you!

Sounds great, please feel free to go ahead!

Sounds great, please feel free to go ahead!

Thank you so much! Could you give me an email address that I can include in the project.yaml so a maintainer/contributor can be given access to the bug reports and OSS-Fuzz dashboard for the project?

You can use<EMAIL_ADDRESS>

Hi @Gallaecio could you please merge my latest OSS-Fuzz pull request? An import issue caused it to stop building and is now performing poorly on the oss-fuzz dashboard. Thank you!


long delay before Open Scripture Editor dialog appears

Describe the bug
Sometimes after clicking Open Scripture Editor on the menu, there is a very long delay before the dialog appears. I have only ever seen this when I have the Toggle Include My Paratext 9 Projects on, and I'm pretty sure it is related to that, since I have a lot of projects and it takes a long time to load them all. However, there seems to be a timing issue. See details below.
To Reproduce
Steps to reproduce the behavior:

If Toggle Include My Paratext 9 Projects is not already on, toggle it on and restart Platform.Bible
Open the main Platform.Bible menu
On the Project menu, click Open Scripture Editor
Sometimes the dialog appears quickly and displays a message that it is loading the projects, but other times it does not. There is no indication that it is doing anything, but if you wait a minute or so, it will eventually appear.

Expected behavior
Ideally, I would expect the dialog to appear very quickly and then either display the message about loading projects or (better yet), load them lazily so that the list begins to populate more quickly. (Of course, it would be nice to find a way to load them more quickly, but maybe this is faster when not running through the debugger.) As a last resort, if we can't make the dialog display right away, the pointer should change or there should be some other visual cue that it is working.

I see this violation in the console, so maybe this is related:
useTouchMove.js:154 [Violation] Added non-passive event listener to a scroll-blocking 'wheel' event. Consider marking event handler as 'passive' to make the page more responsive. See https://www.chromestatus.com/feature/5745543795965952


zen-mode compatiblity

I'm using zen mode and it's z-index is set to 40. nvim-treesitter-context's z-index is 20 so context doesn't show up in zen mode. Can the zindex of nr be set to a value higher than 40, or will this have side effects?

Sorry don't use zen mode.


Schema for v2.0.0 does not allow to extend the components object with specification extensions

Describe the bug
The schema for v2.0.0 (and all older versions) does not allow to extend the components object with specification extensions. This is inconsistent with the specification which states that the components object can be extended with specification extensions.
How to Reproduce
Validating the following exemplary AsyncAPI document against the v2.0.0 schema with any JSON schema validation library results in an error like Property 'x-extension' has not been defined and the schema does not allow additional properties or similar:
{
    "asyncapi": "2.0.0",
    "info": {"version": "1.0", "title": "Example API"},
    "channels": {},
    "components": {
        "x-extension": "xxx"
    }
}

Expected behavior
Above AsyncAPI document should be validated without errors.

@balogal you are right, components should also be "extendable". Can you open a PR with the fix?

just 2.0.0 please, 🙇🏼


SUGGESTION: few typos and indentation fixes for README

This has been merged. The change will propogate shortly.


Multidimensional array functionality for Session class

As it is you can only store variables in the Session class with regular variables.
There should be functionality for multidimensional arrays as such:
Session::setMulti
(
    array ("orderid" => 1, "productdescription" => "A nice table", "date" => "29-03-2016"),
    array ("orderid" => 2, "productdescription" => "A nice chair", "date" => "30-03-2016"),
    ...
);

It should accept any length of arguments.
PHP currently does not have any implementation of a method being able to accept any variable number of arguments but it's still achievable through func_get_args ().

This is how we ( or just me ) do it now.
$array = array (
    array('key' => $val, 'key2', $val),
    array('key' => $val, 'key2', $val)
);

Session::set ( 'myMultiArray', $array );

@NathanBastiaans Some users just aren't that "agile".

@bobdesaunois But then again, some users manage to get memory warnings out of a simple SQL statement...

TMFRHHHHHHHHHHH
UYBGHDEXWXSWZSSXWXSWXSWSXWSEWXSWDWSXSWXSWSWDSWXWWDSXSXWS

DJKSAHDSAHDKJSAJHASDGFASHDGFASDFGASHFGAHSDFGALSDFGLHASDF
Is dat wat je bedoelde?

Nee, ik bedoelde DJKSAHDSAHDKJSAJHASDGFASHDGFASDFGASHFGAHSDDDDDDDDDDDD


Reduce benchmark requests

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
A single inflight is generated by a single go routine. Additionally, there is one main go routine. Therefore instead of requesting 2 * Inflight CPUs we can have Inflight + 1 instead.
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
/assign @mborsz

/lgtm
/approve


pushState does not work in nested tabs

Tested with the official kitchen sink:

set pushState: true
open kitchen sink with npm run dev
navigate to nested routes > tabs > third tab

Then I see no update in the URL, "tab2" remains.
Refresh the page will load tab2 instead of tab3.

And it won't at the moment because this "Tabs Routes" thing not using F7's router which doesn't support it

Yeah, it doesn't currently work. The problem is that only the Pages component calls router.load, which is what calls into the History API. When only a tab route changes with no page change, Pages ignores the route change event and doesn't call router.load, and therefore the tab route change doesn't affect History.
The other issue with not calling router.load on nested route changes is that view.history doesn't get updated either. When switching between tabs, using the back button to go back through the tab selection history is not usually desirable anyhow, but when switching components within a tab (which is sort of like page changes), it potentially is desirable.
I suppose the tab components could call router.load as well when a tab route changes, or perhaps the F7 Vue router could call it after all components have updated themselves after the route change event. When a link click triggers the tab change, it seems like it should default to replacing the current history item if changing to a different tab and should default to adding a new history item if changing the component displayed within a tab. It doesn't seem like that simple of a change, but maybe there is an easier way to do it.

Is it in scope to complete the routing / push state behaviour?

Getting History working is a pretty essential feature for the routing features in F7 Vue / React to be on par with Vue Router and React Router. It definitely belongs near the top of our priority list in the roadmap in my opinion.


Moved document milestone into document

When we initially created the document table, we assumed there will be more fields to add that are specific to certain document types. However, no other field other than milestone was ever needed. So it made the extra table completely unnecessary.
Changelog:

Delete DocumentMilestone model
Added milestone column into the Document model
Adjusted tests, api and front as necessary
Changed url secure_file_submissions to part_3_agreements

 SFS | Move Milestone to Document |2|

 SFS | URL references Secure Document Upload


fix repo creation

https://github.com/SUSE/spacewalk-testsuite-base/issues/407

@ncounter yop, i wads thinking that we could just hardcode and creating all the metadata for all repo.
Maybe for sles15 if os-release change , we could do it.
Thx :panda_face:

thx @ncounter , the stuff was that we couldn"t imagine, that sles11sp4 was not CONSISTENT. :rofl:


ModuleNotFoundError: No module named 'distill_default_utils'

Hello, nice work.
While attempting to replicate your findings, I encountered a problem in the "buffer.py" file, line 6. It appears that a crucial module and function, "from distill_default_utils import load_or_process_file," are missing. I have been unable to locate this module elsewhere or determine the appropriate installation method.
I kindly request your assistance in resolving this issue. Thank [you.]

Hi! I'm sorry that the module distill_default_utils was moved to src.vl_distill_utils. Please try again with the latest buffer.py.

Hi! I'm sorry that the module distill_default_utils was moved to src.vl_distill_utils. Please try again with the latest buffer.py.

Thank you.  Now I can run this.


Downgrade core to java 8 via Jabel

This patch does the same thing as https://github.com/substrait-io/substrait-java/pull/95, but using jabel

Seems like small enough refactoring to accept. Let's see long term how much Java8 this is a burden and reevaluate based on that.
Thanks for the patch @baibaichen !


[Announcement] Semantic-UI now is a part of new ASP.NET Core templates

Hi,
After my request (aspnet/Mvc/issues/2709), I thought that the proposal was ignored. But today, I was watching a tutorial on how generate an ASP.NET project via Yeoman (ASPNET-Monsters-Episode-38-Yeoman), and I discover that Semantic-UI is now a part of new ASP.NET Core templates (OmniSharp/generator-aspnet) :smile: .

So I return to my request 2709 and I see that finally the community react.
So thanx to @jackjwilliams for the hard work, the integration was completed because of him via this pull request.
So @jlukic dont forget to react with the ASP.NET community if there are some break changes. and Who knows, maybe Semantic-UI will be the default template for ASP.NET and not bootstrap :wink:

Apparently I'm a little slow because I'm just now seeing this (over a year later). No problem, was a fun learning experience!


[css mode] teach it about @supports

Add support for the @supports rule
Fixes part of https://bugzilla.mozilla.org/show_bug.cgi?id=977665

Merged as 6b9511d11d2, test in 3469f317b5eb.
If we add another expression language (like the one in @supports and @media) we should generalize it so that we don't end up copy-pasting all the boilerplate stuff.


change issue_id to issue_number

Instead of issue_id (which is a thing in the api) this value should be issue_number which directly correlates to the number attribute in the issue.  (e.g if this is issue #6 then issue._number == 6")
https://github.com/Python-Community-News/GH-Issues/blob/3484064721da0b04d85b13bfa3d76fabe4c45884/src/gh_issues/__init__.py#L40

so basically we just have to replace issue_id with issue_number or did I misinterpret it?


Ion Inputs don't have underlines inside of Cards

Ionic version:  (check one with "x")
[ ] 1.x
[ ] 2.x
[x] 3.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request
Current behavior:
I'm noticing that Ion Inputs inside a Card don't have the normal underline beneath them unless they're active.

Expected behavior:
Inputs should have an underline beneath them

Steps to reproduce:
http://plnkr.co/edit/I5vapkRia0I3yyP37hgf?p=preview
<ion-card>
  <ion-card-content>
    <ion-card-title>Title</ion-card-title>

    <ion-item>
      <ion-label floating>Test input</ion-label>
      <ion-input type="text"></ion-input>
    </ion-item>

  </ion-card-content>
</ion-card>

Other information:
It looks like it's caused by some styling on ion-card.
.card-md .item-md.item-block .item-inner {
    border: 0;
}

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages: (~/Github/ionic-bug-inputs/node_modules)

    @ionic/cli-utils  : 1.8.1
    ionic (Ionic CLI) : 3.8.1

local packages:

    @ionic/app-scripts : 2.1.3
    Ionic Framework    : ionic-angular 3.6.0

System:

    Node : v6.11.1
    npm  : 3.10.10
    OS   : macOS Sierra

Hello! Thank you for opening an issue with us! This issue is a duplicate of #11640 so I will close this one. Please track issue #11640 for further updates.
Thank you for using Ionic


Change to readTime calculation

The estimated read time for blog posts now uses the number of words instead of number of characters.

Used main branch, will reopen pr with a separate fix branch.. My bad


Create an Index HTML file

Create an HTML file named Index.html that has the following attributes:

Index.html:

Has a welcome slogan
Has the option to either Sign Up or Log In
Sign Up redirects to the URL name = signup and Log In redirects to the URL name = login

This file goes inside templates/Main directory

@SameeranB I would like to work on this issue, but I am new to Python and following the Setting started steps in the project README I have a problem with step 1 $ pip install -r requirements.txt because there is no requirements.txt file in the project folder.
Any pointers to what I might be doing wrong or is a new issue needed to create the requirements.txt file too?

@SameeranB I would like to work on this issue, but I am new to Python and following the Setting started steps in the project README I have a problem with step 1 $ pip install -r requirements.txt because there is no requirements.txt file in the project folder.
Any pointers to what I might be doing wrong or is a new issue needed to create the requirements.txt file too?

Hey @dickyw71 , glad to know that someone is trying Python out. The absence of the requirements.txt was a mistake on our part, it has been resolved. Do check it out!

Steps I have taken:

I git cloned the project:
git clone https://github.com/IEEE-VIT/Django_To_Do.git
I created a new python 3.7 virtual env in a separate directory at the same level as the project directory thus:

mkdir Django_To_Do_env && cd Django_To_Do_env
python3 -m venv env

I activated the new virtual environment with source Django_To_Do_env/env/bin/activate
Then I ran pip install -r requirements.txt which returned
Could not open requirements file: [Errno 2] No such file or directory: 'requirements.txt'

Hey @dickyw71 , glad to know that someone is trying Python out. The absence of the requirements.txt was a mistake on our part, it has been resolved. Do check it out!

Ok @SameeranB, I pulled in the new file am carrying on, thanks.

@SameeranB, just to clarify, the templates/Main directories do not exist right now. So I will create those directories too as well as the Index.html file?

@dickyw71 yes, you are meant to create the Main folder under templates

Can I take this issue?

I would like to take this issue.


Release 0.0.16

New features

New version.py and setup.cfg-based setup (#40, #42)
Single file autorelease import for other repos (f01bb19)

Merging despite the Appveyor failure, which is because of a version skip. I expect I'll be cutting another release version soon anyway.


Improve contributing workflow: transition to simplified git flow with automatic release of beta versions

See https://github.com/thomvaill/log4brains/discussions/108 and the ADR created in this merge request to understand the context and the rationale behind this change. But the key idea is to streamline contributions and release more often.
The goal is to make contributing easier, streamline the release process, and allow more automation. This will make it simpler to delegate tasks to new maintainers and make it easier for contributors to follow the process. It also provides early feedback on beta features from the community, which will help us catch issues before stable releases.
WORK IN PROGRESS
Changes to the contributing workflow

We’re transitioning to a Simplified Git Flow with the following changes:

The main branch for ongoing work is now develop: all the feature branches should be merged into this branch
Any change on develop triggers the automatic release of a beta version (testable immediately by running npm install log4brains@beta)
When we are satisfied with the current version of develop, we merge it into the stable branch and release the stable version manually to npm

We introduce 2 maintainer roles in the CONTRIBUTING.md:

Canary Maintainers can merge PRs to the develop branch, triggering beta releases for testing.
Core Maintainers can merge develop into stable for stable releases (for now, I’ll handle this).

How to review?
The best is probably to read first:

The new ADR "Transition to Simplified Git Flow"
CONTRIBUTING.md
Then the other files

Your Feedback Matters
I’d love to hear your thoughts on this new workflow. Do you think this will make contributing easier? Are there any concerns or suggestions you have about this approach? Your feedback is invaluable to make sure this change benefits the community.

After reading through the code, I think that the proposed flow (a.k.a. "the process") strikes a good balance between making things easier for contributors without compromising quality or introducing too many hurdles.
The only thing I am wondering about is whether the project would only want developer contributors or if more high-level individuals might also be needed, as developers quite often prefer clear work task over fuzzy issue triage and back-and-forth support requests.
I mention this specially, as I've seen this in other open-source project where contributors (and maintainers) spend time on non-code work. People started to feel like they were burning a lot of fuel just spinning their wheels without actually getting anywhere, which eventually led to loss of interest and people leaving or burning out.


Only a ReactOwner can have refs

addComponentAsRefTo(...): Only a ReactOwner can have refs. You might be adding a ref to a component that was not created inside a component's render method, or you have multiple copies of React loaded (details: https://fb.me/react-refs-must-have-owner).
using "react": "15.3.2"
I played games trying to figure out if it was a dependency / version issue, but gave up when everything started breaking.
Pretty sure this explains what you will need to do to resolve this issue if you can repro it: https://facebook.github.io/react/warnings/refs-must-have-owner.html

Thanks for that, will take a look and try to rectify it asap

Had a play, looks like the issue is actually more the end comment on that help page - my plugin required react 15.4.0 and up whereas you were using an exact version, so you ended up with two react version in your codebase (I tested both cases and got your error when I switched my example to your exact react version only). I've downgraded the react version to 15.3.0 and up and it seems to work in my tests, can you try it again and see if it resolves your issue please?

Closing this as I haven't heard back. if you still have this issue let me know and i'll look into it again. thank you


MRP April 2021 Updates

Fixed year

@willduff : Thanks for your contribution! The author(s) have been notified to review your proposed change. @hferrone


4.0.0 Docs refer to incorrect afAuth.subscribe method

Hi, simple fix for documentation on 4.0.0 RC0:
https://github.com/angular/angularfire2/blob/4.0.0-rc.0/docs/Auth-with-Ionic2.md
refers to
this.authState = afAuth.authState;
    afAuth.subscribe((user: firebase.User) => {
      this.currentUser = user;
    });

This caught me out, after learning more about rxjs the simple solution was to subscribe to this.authstate instead:
this.authState = afAuth.authState;
        this.authState.subscribe((user: firebase.User) => {

            console.log('user is: ' + user);
            this.currentUser = user;
        });

I'll create a PR
AngularFire 4.0.0 RC0
Ionic Framework: 3.1.1
Ionic App Scripts: 1.3.6
Angular Core: 4.0.2
Angular Compiler CLI: 4.0.2
Node: 6.10.3
OS Platform: macOS Sierra
Navigator Platform: MacIntel
User Agent: Mozilla/5.0 (iPhone; CPU iPhone OS 9_1 like Mac OS X) AppleWebKit/601.1.46 (KHTML, like Gecko) Version/9.0 Mobile/1

If you have 4.0.0rc0 or greater I belive the correct code is:
afAuth.authState.subscribe((user: firebase.User) => {
see https://github.com/angular/angularfire2/pull/917

Hi, your code line would work.
That #917 PR used
this.af.auth.subscribe(authState => {

...which is wrong.  The documentation is inconsistent and plain wrong. I've focussed on the one docu page that is largely correct as then folk can focus on one working example.
The documentation author decided to assign the local variable this.authState, which I used for left line readability. Otherwise you have a hodge podge of 'lets grab that variable, oh, this time we'll grab the original'.
I've now spent multiple hours diving into learning how rxjs works, checking source to find out what the simple solution was. Then went jumping through the hoops to submit a fix for the next poor person. If someone feels strongly about which variable to use, then so be it, lets just not spend a week debating it.

Hey @markterrill I'm sorry these docs aren't more clear or clear at all! 👎
Is there anything about this issue that might be unique to Ionic or is it just general Auth methods / variable naming / best practice in general?

It simply was that the docu was wrong. afAuth.subscribe() doesn't exist.
Correct solutions are either:
this.authState = afAuth.authState;
this.authState.subscribe(....

or as @gianpaj suggests, going to the constructor argument directly:
afAuth.authState.subscribe(

On a more broader note, the ionic docu has some odd references to objects not defined, ie this FirebaseAuthState interface:
signInWithFacebook(): firebase.Promise<FirebaseAuthState> {

A wish list:

consistent login/logout naming, most things are login/logout, not signin/signout. If going with a specific provider then I'd suggest loginWithFacebook()
inclusion of information on route guards on the ionic demo. IE using a service like suggested at https://blog.khophi.co/angularfire2-authentication/ and using that as a function on canActivate within Ionic2 route guards.
update the Auth-with-Ionic2.md with the fix as explained in this PR as it as the wrong subscribe

I well understand and appreciate it's pretty hard for projects to keep up with documentation when things are moving fast. The trick is there are a number of tutorial sites out there charging ~120USD for their howto and they must be doing some business. The project team has left the door open to them by getting it wrong in the first place. Then the tutorial sites get it wrong and you end up with 5 different (and wrong) ways of doing it! Anyways!
I'll likely submit a new version of the AuthService on that page when I finish doing it.

As far as I can tell setting this.authState = afAuth.authState; is completely pointless. Unless I'm missing something....which is possible.

Seems to be a convention throughout most documentation implementations. It makes sense from the perspective that a local shorthand variable decouples user code (view and controller) from how its accessed from angularfire. Which hasn’t exactly been consistent during the great refactor.
I really don’t mind either way though its more convenient to use shorthand rather than a big find/replace each time its changed.
The key point is the docu is wrong, it points to the wrong object. As long as that’s fixed we’ll have less folk scratching their heads.
I’m going to bow out of further discussion, up to you guys.
Mark
From: Mark Goho<EMAIL_ADDRESS>Reply-To: angular/angularfire2<EMAIL_ADDRESS>Date: Thursday, 11 May 2017 at 12:35 am
To: angular/angularfire2<EMAIL_ADDRESS>Cc: Mark Terrill<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [angular/angularfire2] 4.0.0 Docs refer to incorrect afAuth.subscribe method (#960)
As far as I can tell setting this.authState = afAuth.authState; is completely pointless. Unless I'm missing something....which is possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/angular/angularfire2/issues/960#issuecomment-300502129, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACfaUWHaZNfPBOTA1hAB_AbNMfa_Vkr_ks5r4ctIgaJpZM4NU_zM.

I have used the below code and I was able to retrieve the user profile in v4.0.0:
`
user: Observable<firebase.User>;
displayName: string;
email: string;
photoURL: string;
constructor(public afAuth: AngularFireAuth, private db: AngularFireDatabase) {
this.user = afAuth.authState;
afAuth.authState.subscribe((user: firebase.User) => {
  this.displayName = user.displayName;
  this.email = user.email;
  this.photoURL = user.photoURL;
});

}
`

fixed

There change so much... nothing from that what i found on google works... i try now to make sience 14 hours, a function that output IsLoggedIn true / false.. nothing works. i tryed every single service that is listed in google...

@BamiGorengo as mentioned above you can use this.afAuth.authState to identify if the user is logged in or not

this.afAuth.authState outputs everytime false.. thats my problem.


Use backports under 2.7

a quick afterthought: do you think it'd make sense to use ruby-next for better backwards compatibility for dry-rb gems?

@Morozzzko probably not, we'll be dropping support for 2.7 soon anyway.


DO NOT MERGE - [DeploymentScripts] Added SystemData support

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run

Thanks @jianyexi. @jorgecotillo , can you follow Jianye's comment to correct the suppression in readme?

/azp run automation - sdk

@raych1 all checks passed, updated examples and included the proper suppress information. Please proceed to merge the code if there are no other outstanding comments.


Cleanup RBD contexts after exceptions to prevent potential agent crash

We noticed that when an exception occurs within the cleanup loop inside
the deletePhysicalDisk routine that the previously allocated contexts
are not cleaned up.  This seemed to cause an eventual crash of the host
agent after multiple exceptions within the loop.
In addition to ensuring the contexts are always freed we also improved
the logging when exceptions do occur to include the actual return code
from the underlying library in deletePhysicalDisk and deleteSnapshot.

Thank you! Good catch :)
Could you look into the logging, since it would show the user more which is happening.

Absolutely, I'll add a bit more to the logging when I get into the office.

Tests passed.

Thank you for adding the log messages!
LGTM

CI RESULTS
Tests Run: 85
  Skipped: 0
   Failed: 0
   Errors: 0
 Duration: 3h 57m 12s

Associated Uploads
/tmp/MarvinLogs/DeployDataCenter__Jul_15_2016_21_09_16_5GQ7Y8:

dc_entries.obj
failed_plus_exceptions.txt
runinfo.txt

/tmp/MarvinLogs/test_network_H9099Z:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_vpc_routers_62MXN1:

failed_plus_exceptions.txt
results.txt
runinfo.txt

Uploads will be available until 2016-09-17 02:00:00 +0200 CEST
Comment created by upr comment.

We have QA'd the patch and confirm that it prevents the agent from crashing when an unexpected exception occurs from upstream RBD libs.
LGTM


MQTT issues

Hello Rhasspy devs,
I've noticed some weird behaviour with mqtt... the problem here is, that I use IoBroker as my backend for everything, I use the integrated mqtt server addon that they provide (all updated to current version)

there is still some weird behavior, I only use an remote rhasspy instance for kaldi reco (there I did not setup the mqtt --> so no collision problems)
so in conclusion: main rhasspy: on a Raspberry Pi setup with docker, Latest Version 2.4.20, that does mqtt, wakeword and tts and an rhasspy instance in an vm. also dockerized 2.4.20 for kaldi only.

This screenshot is my setup on the raspberry pi that does the wakeword and so on... but not the kaldi part...
Still are there Issues, where suddenly it reconnects to the mqttbroker without any disconnect message...
[INFO:2669544] HermesMqtt: Connected to <IP_ADDRESS>:1884 [INFO:2487798] HermesMqtt: Connected to <IP_ADDRESS>:1884 [DEBUG:2342556] PorcupineWakeListener: started -> listening [DEBUG:2342552] DialogueManager: ready -> asleep [INFO:2342548] DialogueManager: Automatically listening for wake word [DEBUG:2342542] DialogueManager: handling -> ready
And caused by this, sometimes very often, the intents will not be send to the broker, causing IoBroker not to update e.g. a state of a light...
If you have any further questions regarding logs or so on, then I can post it.

Hello Rhasspy devs,
I've noticed some weird behaviour with mqtt... the problem here is, that I use IoBroker as my backend for everything, I use the integrated mqtt server addon that they provide (all updated to current version)

there is still some weird behavior, I only use an remote rhasspy instance for kaldi reco (there I did not setup the mqtt --> so no collision problems)
so in conclusion: main rhasspy: on a Raspberry Pi setup with docker, Latest Version 2.4.20, that does mqtt, wakeword and tts and an rhasspy instance in an vm. also dockerized 2.4.20 for kaldi only.

This screenshot is my setup on the raspberry pi that does the wakeword and so on... but not the kaldi part...
Still are there Issues, where suddenly it reconnects to the mqttbroker without any disconnect message...
[INFO:2669544] HermesMqtt: Connected to <IP_ADDRESS>:1884 [INFO:2487798] HermesMqtt: Connected to <IP_ADDRESS>:1884 [DEBUG:2342556] PorcupineWakeListener: started -> listening [DEBUG:2342552] DialogueManager: ready -> asleep [INFO:2342548] DialogueManager: Automatically listening for wake word [DEBUG:2342542] DialogueManager: handling -> ready
And caused by this, sometimes very often, the intents will not be send to the broker, causing IoBroker not to update e.g. a state of a light...
If you have any further questions regarding logs or so on, then I can post it.

For reference, this is the moment, when Rhasspy looses the connection, but I can't work out, why this happens...

I think you have the wrong project for 2.4.20. This is the github of the new 2.5 version. The 2.4 github is here.

Ok, never mind, this was the official docker install described in the docs...


Custom latency format

1. What version of Caddy are you running (caddy -version)?
Custom build from matser a24e36176105d9d42166156c1147e5e0465e2064
2. What are you trying to do?
log latency in custom format instead of 30.021232ms
3. What is your entire Caddyfile?
:80 {
    log / internal/access.log "{remote} - [{when}] {method} {path} {status} {size} {latency} {host}"
}

4.  What did you expect to see?
I want some latency format like 33.33 without trailing ms, or event more flexible use go time format.
6. What did you see instead (give full error messages and/or log)?
<IP_ADDRESS> - [04/Aug/2016:12:28:12 +0800] GET /product/type 200 347 18.869068ms <IP_ADDRESS>

7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?

If we just round this number, is that sufficient? Will anyone mind?

< 1s: Round to nearest whole number

= 1s: Round to nearest thousandth

I can take a look at this one.

@definitelycarter @mholt I stumbled across this while trying to figure out how to use GoAccess with caddy for log analysis, and believe that the original issue here was misunderstood.
Many log analysis tools expect the "latency" param to be just a number (e.g. 123 or 123.45, but not 123ms), which is what I believe @wenerme was trying to get at. (cf. the %L param here: https://goaccess.io/man#custom-log)
I'd be interested in seeing caddy support this so that the barrier of entry for pairing caddy with common log analysis tools is lowered, and can happily submit a PR if there's agreement that this is a valuable addition (and that the list of placeholders wouldn't become too polluted by the addition of something like a {latency_ms} placeholder).

@seedifferently Hmm, let's discuss that in a new issue then. Reference this one in the new issue. Thanks!


sprint: Basic-Tests kick-off

Basic tests sprint kick-off
Dates
GanttStart: 2018-10-01
GanttDue: 2018-10-12
People

@diegodelemos (10d)
@dinosk (10d)
@tiborsimko (10d)

Goals

Implement the minimum test infrastructure for all REANA components, involving mostly mocking of:

AMQP
Celery
REST API (OpenAPI specs)
Filesystem

Extract the test fixtures to a common repository to share across components.

Kanban

kanban board https://waffle.io/reanahub/reana?milestone=Basic-Tests

Tasks

Basic testing infrastructure:

[x] (https://github.com/reanahub/reana-pytest-commons/issues/1) Bootstrap new repo containing pytest fixtures
[x] (https://github.com/reanahub/reana/issues/91) REST mock
[x] (https://github.com/reanahub/reana-workflow-controller/issues/131, https://github.com/reanahub/reana-commons/issues/46) AMQP mock
[x] (https://github.com/reanahub/reana-workflow-controller/issues/132) Code including Celery tasks calls (currently we are "hacking" it, see here)
[x] (https://github.com/reanahub/reana-pytest-commons/issues/4) Try FS fixture on a different component (i.e. REANA-Server) so when we extract it it will work.
[x] (https://github.com/reanahub/reana-pytest-commons/issues/3) Try DB fixture on a different component (i.e. REANA-Job-Controller) so when we extract, it will work.
[ ] POSTPONED until Rich-Tests sprint. (https://github.com/reanahub/reana/issues/90) Get a Kubernetes cluster running on CI environment with service account token (needed by REANA-Job-Controller) and test that examples work using reana CLI (reana install-client, reana install-cluster, reana run-example...).

Closing sprint, everything has been accomplished but the CI setup.

POSTPONED until Rich-Tests sprint. (https://github.com/reanahub/reana/issues/90) Get a Kubernetes cluster running on CI environment with service account token (needed by REANA-Job-Controller) and test that examples work using reana CLI (reana install-client, reana install-cluster, reana run-example...).


chore: import repository https://github.com/martinmlouis/jenkinsnew.git

this commit will trigger a pipeline to generate the CI/CD configuration which will create a second commit on this Pull Request before it auto merges

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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


Resp view

Changes
Issue: #1421

CSS fix
Add responsive view for both tablet and mobile
Add Scroll to Top button[^1]
Collapsible sidebar for small devices[^1]

Context

Current website is not responsive and mobile-ready.

[^1]: <noscript> functionality

@peff
I hope you've checked the responsive view which doesn't break the functionality of website.

@adsingh14 Sorry it took so long for me to get back to this. The result looks pretty good to me!
A few things I noted:

there was a funky noop merge on your branch, as well as some small whitespace issues (trailing whitespace on lines noticed by git show --check, dropping newlines from end of text files, and one or two funny indentations). I fixed those up and force-pushed onto your branch, as it makes this and future diffs a little smaller. I hope that isn't disruptive to any ongoing work (if it is, you can force-push back over it, though I hope to just merge this soon anyway).
the <noscript> sidebar works fine for me if I disable javascript entirely, but not when using a blocker like ublock. AFAICT this is a ublock thing, and there's not much we can do about it on our end (it just doesn't respect the tags except for in a few cases). The non-js version of the sidebar works so well, though, it makes me wonder: what is the advantage of the js version? I.e., would it be worth just always using the no-js version?
I have mixed feelings on the "scroll to top" button, as it can sometimes cover actual content. It's not too bad, but I'm also not really sure what it's accomplishing. Thoughts on getting rid of it?

A few things I noted:

there was a funky noop merge on your branch, as well as some small whitespace issues (trailing whitespace  ...

That's Windows CRLF issue, I tried to fix it several time but it changes files while I commit. Just in case, if it generate any issue, I will fix it later.

The non-js version of the sidebar works so well, though, it makes me wonder: what is the advantage of the js version? I.e., would it be worth just always using the no-js version?

I don't think so. Javascript actually decrease lines of code and helpful to create better UI/UX experience. CSS (<noscript>) part is just a tweak.

It's not too bad, but I'm also not really sure what it's accomplishing. Thoughts on getting rid of it?

That's helpful for long pages when someone scrolled down and don't want to refresh the whole page. It makes better UX experience and better for tablet/small devices. If you still don't want to remove, then please tell from which version should I remove :

No Javascript version only
From both JS and non-JS version ?

I don't think so. Javascript actually decrease lines of code and helpful to create better UI/UX experience. CSS () part is just a tweak.

They seem identical to me in function, with the exception of the "X" to close in the javascript version. I do agree that fewer lines is nicer in general, but if we're carrying both anyway, that's more total lines (plus the seldom-used one will be subject to bitrot).

Should I remove Javascript code for collapse-able sidebar and consider as a final change to get this merge ?

@peff  Should I remove Javascript code for collapse-able sidebar and consider as a final change to get this merged ?

Should I remove Javascript code for collapse-able sidebar and consider as a final change to get this merged?

I think that's my preference, but if you (or anybody) has a strong opinion, we could go the other way. As you noted, there are already parts of the site that don't work without javascript, so I'm not sure how common it is.

I'm not against the CSS but most of code runs best with JS which is supported by all web browsers (incl. IE).

My main concern here is people who disable javascript with extensions like ublock. It doesn't run the javascript, but also doesn't use the noscript bits, so you end up with the navbar disappearing but no way to bring it back.
I guess another alternative is to make the javascript more progressive: do the bits in noscript by default (without a noscript tag), and then if the javascript is run, have it mark up the DOM to do whatever it needs. That sounds like work, though, and I don't want to make you do more work.

@peff I keep CSS sidebar functionality and removed 'no-js' functionality. 'Scroll-to-top' works okay with/without JS. It'd better to squash my all commits and use the recent one (05934c5).

@peff I keep CSS sidebar functionality and removed 'no-js' functionality. 'Scroll-to-top' works okay with/without JS. It'd better to squash my commits and use the recent one (05934c5) one in your repo.

@peff I keep CSS sidebar option and removed extra lines from .js and other files. 'Scroll-to-Top' <noscript> won't affect the core functionality. So, the recent change is more viable.

@peff  Thank you for whitespace fixes. May I know how to fix this issue for in future changes ? I've used VS Code for all changes.
I also checked the live website but nothing has changed there !

Thank you for whitespace fixes. May I know how to fix this issue for in future changes ? I've used VS Code for all changes.

I'm not sure how to prevent the "no newline at end of file" issues. Most editors I've seen include it by default; I'm not sure if there's some VS Code setting at work here (I don't use it myself).
For trailing whitespace on lines, you can view your commits with git show --check (or git diff --check before making a commit) to see which ones Git takes note of.

I also checked the live website but nothing has changed there !

Yeah, there's a big Cloudflare cache in front of the Heroku deployment, so changes may take up to 4 hours to be seen via https://git-scm.com. You can visit https://git-scm.herokuapp.com directly to see the deploy in the meantime (and indeed, it looks great, including the search results!).

@peff After I sync fork repo on local &  typed ./script/server, 'Terminal' shows this error:
-bash: ./script/server: /bin/sh^M: bad interpreter: No such file or directory

-- Currently using WSL/Windows Terminal on Windows 10

I'm not sure how to prevent the "no newline at end of file" issues. Most editors I've seen include it by default; I'm not sure if there's some VS Code setting at work here (I don't use it myself).

If it seems worth doing, we could consider adding editorconfig files which I perceive to be a universal solution for formatting requirements (new line at end of file, trimming trailing whitespaces). You could find more information regarding Editorconfig in their site: https://editorconfig.org/
My understanding is that it works by relying on built-in support in editors and existence of plug-ins where built-in is lacking.

I'm not sure how to prevent the "no newline at end of file" issues.

I used WSL Terminal for ruby based projects. To clean the CRLF mess, I cloned the repo, again, in WSL home folder. My recent commit(47ff215) is the result of 'no-headache' PR.

My understanding is that it works by relying on built-in support in editors and existence of plug-ins where built-in is lacking.

On Windows, WSL Terminal works best to avoid extra packages/plugins. Instead of using Windows drive, use of home or Ubuntu directory is recommended for this project.


Include stdlib.h in json-builder.c

In json-builder.c there are calls to malloc and free, but stdlib.h is not included.
It used to be included via json-builder.h from json.h
But json.h doesn't include it anymore
https://github.com/json-parser/json-parser/commit/b5ce13ee6437f1ccc49eb360e0ab465ba2d89cc6

Thanks! I'll merge this after figuring out a proper way to handle the fact that these are two separate repos. A submodule for json-parser is one way to ensure intended versions are matched together, but maybe it would be worth the effort to merge the repos.

json-builder is so small it might be worth just pasting it onto the end of json-parser's json.c, to be honest.
I wouldn't recommend renaming json-parser though, as the name is very well established and it's packaged as jsonparser in repositories.

Actually gonna merge this now, I haven't had time to look into doing what I said previously but this is apparently a bigger issue than I realized. It results in crashes without this patch!


Support LookaheadDecoding?

Hi,
A recent technique called could speed up inferencing further than almost 100%
https://github.com/hao-ai-lab/LookaheadDecoding
That repo show we mostly need to do this for any models adapted to this technique (now onloy LLaMA models and greedy search generation are supported)
import os
os.environ["USE_LADE"]="1"
import lade
lade.augment_all()
lade.config_lade(LEVEL=5, WINDOW_SIZE=7, GUESS_SET_SIZE=7, DEBUG=0)

It would be great if the same technique is implemented to exllamav2 as well!
Thanks!

Feel free to submit a PR. Most immediate obstacle for lookahead decoding is the lack of support from Flash Attention which doesn't allow for custom attention masks.
Currently ExLlamaV2 supports speculative decoding which gives a similar speedup to LADE or Medusa. It also allows sampling, and it takes advantage of flash-attn.

Yep, LADE would need do_Sample turning off. Seems speculative decoding is the goto from the get-go.


fix: helmfile -f dir/ -l k=v sync fails on first helmfile w/ no match

Fixes #187

thanks @mumoshu! that was really fast. much appreciated.

@alebabai Thanks for the feedback! I've added trailing filenames as you can see in f884a72. WDYT?

Looks good, thanks!

Would love to get this merged & released =)


Failed to read NVS keys (0x1117) when pre-encrypted binary with secure boot and flash encryption enabled (IDFGH-12468)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Chip: ESP32C3
ESP-IDF version: release/v4.4
We have the next production cycle at the factory:

read MAC, register the device, generate flash encryption key (unique for each device), burn the key into efuse BLOCK_KEY0
Download test firmware (both firmwares are huge and can't fit into flash simultaneously), encrypt it with generated key (bootloader, partition table, app), upload encrypted binaries to device with esptool, burn SPI_BOOT_CRYPT_CNT
Perform tests
Download prod firmware, encrypt it with the same key, upload it to device via esptool.

In this routine we get ESP_ERR_NVS_CORRUPT_KEY_PART for prod firmware. I looked into the docs, they suggests to completely erase flash before flashing firmware with different partition table. Unfortunately esp32c3 ROM doesn't support flash erasing when secure boot is enabled.
If I disable NVS encryption - everything is ok.
If I upload plaintext binary and chip performs key generation internally - everything is ok
We flash both firmwares encrypted because want be able to reflash it just in case, and don't want to wait till MCU performs self-encryption (1 min) because operator may disconnect device from jig and soft-brick device
How to solve this issue? What additional info could I provide?

we write empty (all 0xff) nvs keys partition of size 4k to flash, it shud work . ..

@aectaan Did you try above suggestion? Flashing (flash writes are supported with secure boot) NVS keys partition with all bytes set to 0xff should help to fix the problem.

Yes, tried, looks helpful. Repeating this procedure with more devices now, to be 100% sure


Pressing enter to post comment reloads page

...and doesn't post the comment.  Pressing the "babble" button works fine.  This is very likely caused by default form submit not being prevented.

Fixed in a4faf73a403c1edbf0b991bf3906e7ab26ed316c.


Model doesn't seem to include tool output in its response

This issue is built off of the code I detailed in #345. After running effectively the same code I included there (with the one alteration being requesting the model to perform "alfalfa" on 2 and 3, rather than 2 and 4), I follow python example on generating a response based on the outcome of the tool call as follows:
if response.message.tool_calls:
    for tool in response.message.tool_calls:
        if function_to_call := toolfuncs[tool.function.name]:
            print('Calling function:', tool.function.name)
            print('Arguments:', tool.function.arguments)
            output = function_to_call(**tool.function.arguments)
            print('Function output:', output)
            messages.append(response.message)
            messages.append({'role': 'tool', 'content': str(output), 'name': tool.function.name})
final_response = oclient.chat(model="llama3.2:3b-instruct-q8_0", messages=messages)

This prints the following:
Calling function: alfalfa
Arguments: {'a': '2', 'b': '3'}
Function output: 72340

This indicates that the tool was chosen and called successfully. For multiple attempts at making the same request this remained consistently true. However, when I then print final_response.message.content I find that the model either doesn't understand what "alfalfa" means (despite understanding it well enough to call the tool), or simply makes up its own meaning for it.

So it truly depends on what your initial prompt is for the model, what your prompt is to get the final response, and the size of the model you're using.

Initial prompt - to get the best performance you should probably tell the model what all tools it's has access to even in the system prompt (although it does get passed in later as well)
Would recommend the final prompt to be something along the lines of "the tool output is attached and the function run was {function_to_call}, tell the user the final output of the function"
Larger models will inherently perform better - would recommend at least an 8b model for this


[Bug]: Properties faulty identified as not declared for types in react-hook-form

Is there an existing issue for this?

[X] I have searched the existing issues and my issue is unique
[X] My issue appears in the command-line and not only in the text editor

Description Overview
Lint returns errors for types from react-hook-form
Example of problematic code
import React from 'react'
import { Controller, useForm } from 'react-hook-form'
import { TextField } from '@mui/material'

interface IFormTestProps {
    number: number
}

const FormTest = () => {
    const { control, register } = useForm<IFormTestProps>({})
    return (
        <form>
            <Controller
                name="number"
                control={control}
                render={(props) => {
                    console.log(props.field.ref)
                    return (
                        <TextField
                            label="Number"
                            type="number"
                            {...register('number')}
                            onChange={(e) => {
                                // This is what we want to do
                                props.field.onChange(parseInt(e.target.value, 10))
                            }}
                        />
                    )
                }}
            />
        </form>
    )
}

export default FormTest

Lint error
> npm run lint
...
c:\...\FormTest.tsx
  17:39  error  'field' is missing in props validation           react/prop-types
  17:45  error  'field.ref' is missing in props validation       react/prop-types
  25:39  error  'field' is missing in props validation           react/prop-types
  25:45  error  'field.onChange' is missing in props validation  react/prop-types

I believe these are the relevant dependencies
{
  "devDependencies": {
    "@types/react": "18.2.79",
    "@types/react-dom": "18.2.25",
    "@typescript-eslint/eslint-plugin": "7.7.0",
    "@typescript-eslint/parser": "7.7.0",
    "eslint": "8.57.0",
    "eslint-plugin-import": "2.29.1",
    "eslint-plugin-react": "7.34.1",
    "eslint-plugin-react-hooks": "4.6.0",
    "typescript": "5.4.5"
  },
  "dependencies": {
    "@mui/material": "5.15.15",
    "react": "18.2.0",
    "react-dom": "18.2.0"
  },
  "scripts": {
    "lint": "eslint src --cache --ext .ts,.tsx,.snap"
  },
}

Expected Behavior

I expected that eslint-plugin-react would detect imported types correctly and not report these errors.
eslint-plugin-react version
v7.34.1
eslint version
v8.57.0
node version
v20.10.0

The FormTest component doesn't have any types declared, so these warnings are correct.


installation error

Hi,
I am using your medialibrary pachage already,
now I need to install laravel-glide
but when i run
composer require spatie/laravel-glide

it gives me following error
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - spatie/laravel-medialibrary 3.17.2 requires spatie/laravel-glide ^2.2.4 -> satisfiable by spatie/laravel-glide[2.x-dev].
    - spatie/laravel-medialibrary 3.17.2 requires spatie/laravel-glide ^2.2.4 -> satisfiable by spatie/laravel-glide[2.x-dev].
    - spatie/laravel-medialibrary 3.17.2 requires spatie/laravel-glide ^2.2.4 -> satisfiable by spatie/laravel-glide[2.x-dev].
    - Can only install one of: spatie/laravel-glide[3.0.0, 2.x-dev].
    - Installation request for spatie/laravel-glide ^3.0 -> satisfiable by spatie/laravel-glide[3.0.0].
    - Installation request for spatie/laravel-medialibrary == <IP_ADDRESS> -> satisfiable by spatie/laravel-medialibrary[3.17.2].

Am I doing something wrong?
I am on laravel 5.2

The medialibrary still uses v2 of laravel-glide. So for now you can only use that version.
I'll create a new major version of the medialibrary that'll ise laravel-glide v3 in the coming months.

thanks.


Interactable Profile Theme text at incorrect indentation?

Describe the bug
I'm not sure this is a bug, but in the Interactable script (for a checkbox) under the Profiles for each of Background, Button, Label, CheckBoxCheck there are four items.
CheckboxCheck
    Target
    Theme
    > ToggleSomething
       Theme
    >ToggleSomethingElse

Visually I find this confusing: either
a) , the two toggles and the second theme are meant to be in some sort of nested relationship with the first theme, and the toggle buttons icon (>) are offset to the left of the (otherwise) similarly indented items, or
b) there is no nested relationship, and so the toggle button icon (>) is at the same level as the first Theme, but then the second Theme is incorrectly indented to the right
If (a) the nature of the nested relationship is unclear. If (b) the second Theme needs to be moved left.
To reproduce
See in Inspector window
Steps to reproduce the behavior:
Expected behavior
?
Your Setup (please complete the following information)

Unity Version 2018.3.12f1
MRTK Version v2.0 latest

Target Platform (please complete the following information)
Unity Editor

@Railboy / @killerantz, can you clarify the interactable property nesting?
Thanks!

@genereddick is correct, it is confusing and a bug. The second theme, for a single target, should not be tabbed in, but flush with the first Theme. All the themes effect different properties of the same object equally and are not meant to be nested. Thanks for catching this.

Maybe the first Theme in the list needs to be indented with the second, so that they imply more that they are effecting the target.

@killerantz I like the idea of indenting the first theme to the same level as the others as I think it clarifies the relationship

@genereddick feel free to open a PR to fix this!

I created a PR #4143


Implement Language Cfgu Type

This PR adds Language type to configu
Resolves #143

Great job! 🥇 Thanks for your contribution @Sadaf-A 💯
Please merge main to this branch

@RonConfigu Thanks for the review,
done!

@Sadaf-A Please tag future PRs with correct label. feat in this case.
The correct label will present on the issue for ref.
Thanks


Quote the path to the MSI given to msiexec

This way, spaces in the path will be correctly handled.
Without the quotes, msiexec hangs (presenting a GUI usage statement) if there are spaces in the path.

Is there anything I can do to get this merged?

I really need this so that I can start using this plugin. This seems like a really low risk change. Can we get this merged?

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
Non-GitHub Verified Committers
There are 1 commit author(s) whose commits are authored by a non-GitHub verified email address. Chef will have to manually verify that they are authorized to contribute.
Please sign the CLA here.

Why did the CLA bot get involved a year after this was submitted?

The CLA bot wasn't watching this repo. It's been turned on now.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

Bump?

Sorry for the delay. Thanks


Add support for SHOW DATABASES/SCHEMAS/TABLES/VIEWS in Hive

This PR adds support for the above statements in the Hive dialect, making the previously MySQL-only structs a bit more generic.

cc @iffyio

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

90 of 92 (97.83%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 89.4%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/mod.rs
31
33
93.94%

Files with Coverage Reduction
New Missed Lines
%

src/ast/mod.rs
1
82.2%

Totals

Change from base Build<PHONE_NUMBER>0:
0.02%

Covered Lines:
30498

Relevant Lines:
34114

💛  - Coveralls

Sorry, did a last change and forgot cargo fmt... Changes committed now, if you can run the CI again that would be great @alamb

🚀

@iffyio @alamb why isn't this merged?

@iffyio @alamb why isn't this merged?

Sorry -- am behind.

Thank you!


Login Not working with special characters

For special characters such as % in the password, login is not working, corrected in masterthermconnect rc9 to be implemented.

@valentapat - Login should be fixed in latest commit to main

@valentapat - Don;t forget to update masterthermconnect manually :)

I'm confused, wait for Release 1.0.0 or not ??
Must I again  enter pip install -I masterthermconnect, or new version r09
pip install -I masterthermconnect  .. show error in instalation, and now hass dont show mastertherm

With pip you have to specify the release if its a pre release so pip install -i masterthermconnect==1.1.0rc9
1.1.0 is not final yet

ok now is ready, Im happy
by the way - can you public "nowhow" for connectin to mastertherm-online ?
My effort was read this data by node-red, across login form, bud failure about the Auth, cookies, redirect, etc.

Not sure what you mean? I will put detail about my project but as it's still work in progress it's not done yet.
I won't document about how the login process works it's using openid so lots of good information on that on the web for developers


[CI] Upgrade build image to CentOS 7 + GCC 8; require CUDA 10.1 and later

Closes #6791
Closes #6642
May help  #7111
This PR raises the minimum CUDA requirement to 10.1, in order to support GCC 8.

@hcho3 I pulled the changes from your branch and was able to pass tests in https://github.com/dmlc/xgboost/pull/7130 .

Also tested https://github.com/dmlc/xgboost/issues/7111 with the workaround removed.


"Ghindeni" is listed in the drop-down menu

Description:
When user writes ”Ghindenii" the location appears in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Ghindenii"

Expected results
The location appears in the drop-down menu as "Sat Ghindenii, DOLJ (Ghindenii)".


[BACKLOG-40939] - Fix for jdk-17 unit test for salesforce plugin

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 10m 18s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl \
plugins/salesforce/core

:ok_hand: All tests passed!
Tests run: 88, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


I can't generate the files in current working directory

HI @jondot !
Currently, I'm trying to generate files in the current working directory. Following the docs, I have tried this:
---
to: <%= cwd %>/<%= module.toLowerCase().replace(/\s+/g, '-') %>/routes.definition.js
unless_exists: true
---

However, <%= cwd %>, it doesn't find dynamically the current directory that I'm sitting, but instead my project's root folder.
Let's say my project is in /webapp/, but I want to navigate to /webapp/clients/<cwdFromWhereIRunTheScript>/<moduleDynamic>/<filesGoHere> why I can't accomplish it with <%= cwd %>.
Thanks in advance!
EDIT Update:
I have been trying to use __dirname  like so:
to: <%= `${__dirname}` %>/<%= module.toLowerCase().replace(/\s+/g, '-') %>/routes.name.js
unless_exists: true

But it throws an error: __dirname is not defined
I still looking for way to solve this.
EDIT UPDATE 2:
:1st_place_medal: :tada: I have a created the first Hygen tag on stackoverflow. Link here

Bit late but for others
try process.env.INIT_CWD
https://stackoverflow.com/questions/12238477/determine-command-line-working-directory-when-running-node-bin-script


Embedding Variable Implement.

Description
In Large scale learning, It stores embedding data by EmbeddingVariable instead of tf.Variable. We define a set of ops to connect EV ops.
Issue: https://github.com/tensorflow/tensorflow/issues/19324
rfc: https://docs.google.com/document/d/1odez6-69YH-eFcp8rKndDHTNGxZgdFFRJufsW94_gl4/edit?usp=sharing

@rhdong


Bug - button display and functionality

checklist:

[x] 1
[x] 2
[x] 3
[x] 4
[x] 5

Describe the Bug:

Continue button displays

 Person responsible when person responsible is empty

Cancel button displays

Attachments

Back button is missing

Attachments
Allocation of Emission

Save button is missing

Attachments

Save and Save and Continue perform same routing function

Production Data
Allocation of Emission

Steps to reproduce the behaviour:

Continue button displays

 Person responsible when person responsible is empty
http://localhost:3000/reporting/reporting/reports/3/person-responsible

Cancel button displays

Attachments
http://localhost:3000/reporting/reports/3/attachments

Back button is missing

Attachments
http://localhost:3000/reporting/reports/3/attachments

Allocation of Emission
http://localhost:3000/reporting/reports/3/facilities/f486f2fb-62ed-438d-bb3e-0819b51e3aff/allocation-of-emissions

Save button is missing

Attachments
http://localhost:3000/reporting/reports/3/attachments

Save and Save and Continue perform routing same function

Production Data
http://localhost:3000/reporting/reports/3/facilities/f486f2fb-62ed-438d-bb3e-0819b51e3aff/production-data
Allocation of Emission
http://localhost:3000/reporting/reports/3/facilities/f486f2fb-62ed-438d-bb3e-0819b51e3aff/allocation-of-emissions

Screenshots:
Additional information:
OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional Context

Thanks @shon-button - I think for Person Responsible, we don't want them to leave this blank so should not show the continue button either, if it is blank.
I also observed that clicking on Save on the alumina page sent me to the next page (non-attributable emissions). I'm not sure if this was specific to this activity page or happens anytime it's the last activity before moving to the next section. Will try to recreate.
Additional reporting data - when I click save and continue it gets stuck on re-directing and keeps coming back to this page instead of moving forward.


Set protobuf_MODULE_COMPATIBLE before any find_package call

🦟 Bug fix
Fixes https://github.com/ignitionrobotics/ign-gazebo/issues/956
Summary
In some environments where Protobuf is compiled using CMake (so not in debian-based distributions, as there protobuf is compiled via autotools), it is necessary to set the protobuf_MODULE_COMPATIBLE variable before any call to find_package, as any call to find_package could call transitively find_package(protobuf).
Checklist

[x] Signed all commits for DCO
[x] Added tests
[x] Updated documentation (as needed)
[x] Updated migration guide (as needed)
[x] codecheck passed (See contributing)
[x] All tests passed (See test coverage)
[ ] While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge

Rebased to Citadel, just waiting for CI.
I'll also cross-reference https://github.com/ignitionrobotics/ign-gazebo/pull/501, which originally added protobuf_MODULE_COMPATIBLE to Edifice, and hasn't been backported because it includes other breaking changes.