Add Unit Tests for Author, MusicAlbum, and Genre

I did the following in this PR:

Edited all files before starting the test units.
Added music unit test
Added author unit test
Added genre unit test
Made sure that all tests are passed for the implemented methods and according to requirements

Status: APPROVED 🟢
✋
Hi @GeekyHacks
Your project is complete! There is nothing else to say other than... it's time to merge it: ship it:
Congratulations! 🎉 👏 👏 💯


Duplicate defintion  in generated rust code for ra4m1 SVD

Hello,
When running svd2rust with the latest commit on Renesas ra4m1 svd, there is a lot of duplicated definitions error.log
Here is the generated code:
lib.rs.txt
I'm a newbie in embedded development, I hope that's enough information.

duplicate of #852

Sorry, I didn't see it when I created the issue. Thanks for the followup.


Error parsing Java packages

I'm testing with prettier-java:0.0.2 with a specific patch for handling #31
I have lots of parsing errors, it looks like Java packages are not correctly parsed, here is one example:
[error] src/main/java/io/jhipster/demo/aop/logging/LoggingAspect.java: SyntaxError: ';' expected. (1:9)
[error] > 1 | package io.jhipster.demo.aop.logging;
[error]     |         ^
[error]   2 |
[error]   3 | import io.github.jhipster.config.JHipsterConstants;
[error]   4 |

In order to give you a full project, which might also have more issues, please have a look at the attached Zip file.: tmp.zip
This is a @jhipster application, made from the current master branch (which passes our test suite, so it should be quite good).
You need to run yarn prettier:format and it should format all .ts and .java files. It should be pretty comprehensive, so hopefully it will help you catch more issues.

Something seems to be very different when releasing it instead of using it from within the project.
yarn prettier --write ../jhipster-example/**/*.java 2> errors

I get two errors from the parser for code that combines strings, but everything else it runs through like crazy... Will keep this open though for the great example :)

Please check version 0.0.4, if you remove all comments from the file it should parse! I tried that. For the status on the comments see: https://github.com/thorbenvh8/prettier-java/issues/32

Sorry but I upgraded to 0.0.4 and removed the comments, but still got the same error:
[error] src/main/java/io/jhipster/demo/aop/logging/LoggingAspect.java: SyntaxError: ';' expected. (1:9)
[error] > 1 | package io.jhipster.demo.aop.logging;
[error]     |         ^
[error]   2 |
[error]   3 | import io.github.jhipster.config.JHipsterConstants;
[error]   4 |

Can you checkout this project and run 'yarn test' just to give me an idea if that works.
And after that can you run out of that project 'yarn prettier --write "../tmp/**/*.java"', imagining your tmp project is in the same workspace as prettier-java.
Can't test your project in total because I have not the correct node version, but I did this example project https://github.com/thorbenvh8/jhipster-prettier here, 'yarn prettier LoggingAspect.java' works for me, prints it into the console, add --write to in place replacement.
Can you check if that works, thanks for investing the time solving this with me :)

yarn test works, and indeed I have different errors:
[error] ../../tmp2/src/main/java/io/jhipster/demo/domain/User.java: Error: Sad sad panda, parsing errors detected in line: 222, column: 36!
[error] Expecting: one of these possible Token sequences:
[error]   1. [';']
[error]   2. [';']
[error] but found: '' +
[error]             ", firstName=''

So that's definitely an improvement!

If you could create an issue with the minimal code that we need to reproduce that error?
And we still have to figure out why its not working when its standalone with comments in the other issue.
Im gonna close this issue! Thanks for helping

Can't test your project in total because I have not the correct node version

If you haven’t already, check out nvm @thorbenvh8.


Missing function to set estimate

I'm missing a function in IssuesClient to set estimate time, like described here: https://docs.gitlab.com/ee/api/issues.html#set-a-time-estimate-for-an-issue.

Closing, probably dead.


Update readme to reflect current CBDT/CBLC support

Sorry, took me a few shots to get the PR in order :)

Thanks


controller's onunload method on route change never called

var module =  {
controller: function() {
this.onunload = function() {
console.log("unload controller")   // do not see this.
};
this.modal = submodule(modal);
},
view:    //return some view
};

Tried and it works fine for me: http://jsfiddle.net/c1jbdb8g/
Can you post a jsfiddle demonstrating the issue?

hmm...
strange, could you check the following fiddle
http://jsfiddle.net/7j4xgeye/86/

Your mainModule.controller has no onunload method. This is the actual instantiated controller by the route. So either change this or dynamically ad the onunload of your submodule.controller to the mainModule.controller

Better solution would be imho this

What @StephanHoyer said. Or you could setup an onunload in mainModule controller that calls the submodule's onunload


Homebridge Govee Plugin Feedback

Feedback as requested:
[Govee] [Shop_01-Ceiling] device supports 'colorwc' command, let me know on Github with model number [H619E].
-Kevin

Hi @kmechols60
thanks for reporting this, please can you update to the newest version of the plugin and check that colour control is still working from homebridge?
thanks

Working great!
-Kevin
On Sunday, February 27, 2022, 03:02:19 AM CST, Ben @.***> wrote:
Hi @kmechols60
thanks for reporting this, please can you update to the newest version of the plugin and check that colour control is still working from homebridge?
thanks
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>


Update from original

Making sure I sync with eugenp/tutorials before starting another article's code from my github fork.

Please ignore this pull request. It was a trial.


Add a flag to avoid parameter checking.

https://github.com/dwavesystems/dwave_micro_client/blob/a425ff6984f6369514359df8fdbfa52c8d5033d8/dwave_micro_client.py#L734
This can be useful when some solvers have x parameters that are not advertised yet, for example x_reinitialize_state, etc...

I looked into this.
A Solver is typically created from a Connection object (at least that's how its used in dwave_micro_client_dimod). So the flags for the solver would need to be given as arguments to the connection.get_solver method. However, there is some caching based on solver name happening there. So providing the same name with different flags could be problematic.
May want to fragment what is happening in that function in a few parts. This is more refactoring than I am comfortable doing (since I am not aware of design choices / priorities).

@frechette-alex  Sure, we can skip checking of x_* parameters. Those will be available only for internal testing anyhow, and discarded otherwise (no security concerns).

That sounds like an even better solution that what I had in mind.

Closed by #211.

@frechette-alex heads up, version 0.4.6 (just released) adds support for experimental parameters (pass-through the x_* params).


any way to change the labeltext to dynamic labeltext from pauload

is there anyway i change the labeltext to dynamic labeltext to a incoming string in a  payload?
i have a string that tells me when my mail ic comming and i want to have the day in label depend on with day the mail ic comming .
best regard pehr

Hi @teknikrummet,
Sorry for the delay!  Completely overlooked this issue...
Could the mustache expressions solve your question?
Bart

its this tect inside the button i want to find a way to do it to dynamic. i can figureout to do it with your metod on the text to the left on the buttons but not inside the buton

@teknikrummet,
Unfortunately it is not possible to set the options via an input message.  I thought that we had added that feature at the time being, but seems we didn't.
Adding such a feature would be a lot of work.  And since AngularJs (on which the current Node-RED board is build) is end of life, such a feature won't be added anymore to this node.  It will be more useful if we focus now on a new dashboard.  As you can see here, we started yesterday discussing ui node support in the new Flexdash dashboard...
Kind regards,
Bart


conditional select mode #1829

This PR addresses:
conditional select mode #1829
TEST:

Ensure select mode works properly.
Make sure select/deselect mode does not work in edit verse and comment.

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


End of partition flag always coming false

Hi @TsuyoshiUshio
we are trying to read the End of partition flag but it is always coming as false. Can you please check and confirm if this is a bug or we are missing some setting
consumeResult.IsPartitionEOF
please let me know if any other info is required.
thanks

I guess i figured it out closing


Proposal: using env variable USQLDSN

Just like psql uses PGHOST, PGDATABASE etc. maybe we can have a variable USQLDSN(for example) which sets the datasource name
For example:
export USQLDSN=pg://user:pass@host/database
./usql
Connected with driver postgres (PostgreSQL ....

@atdichev Thanks for the suggestion. I don't think this fits in well with how psql and usql recognize environment variables.
If this is something you'd like to enable on your command-line, you could set a bash alias to pass that environment variable as the first parameter to usql, and/or just pass it directly:
$ usql $USQLDSN


Do not delete keys with empty values

Do not delete keys with empty values, sometimes it is important to have a key with empty values, it would be great to not delete it!
Like:
{"key":""}
(By the way, congratulations on the solution, very nice!!!)

You can un-comment the lines 309 and 310 to get the null values as blanks


Updated Ogma to 5.1 (round 2)

Trying (second attempt; #104) to update Ogma to 5.1.

https://github.com/Linkurious/ogma-build-systems/pull/104/commits/9ca493534d96584c54ff060462a05e76515a5ffd


[UI] Selected Items disappear in Table view, Even when deletion denied in modal

Current Behavior

Currently at /configuration/filters, /configuration/applications, /configuration/patterns. On clicking the delete icon all the selected items get disappeared. Even when NO is selected in the delete modal.
https://user-images.githubusercontent.com/103871275/208772597-e638472e-c9f0-47f7-9fb4-1ecbb1d6288e.mp4
Desired Behavior

Items should appear after NO is selected in the delete modal.
Environment

Browser: Chrome Safari Firefox
Host OS: Mac Linux Windows
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker Kubernetes

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Hey @mentors. I figured out that items disappear from the frontend due the delete icon of the mui data table. If we add the functionality to reload the page if NO is selected. The things will work.

Hi @hs05june, are you in the community slack workspace already? Will you say "hi", if so?
Thanks for filing this.

@hs05june good eye. Thanks for filing this issue. We don't to force a page reload, but instead prevent the removal of these items from the data table in the first place.

@leecalcote Yes I am there in the slack community.

https://user-images.githubusercontent.com/103871275/208922890-10e2841c-27e8-42df-8cfe-09d501a9f1de.mp4
Hey @leecalcote. It was due to the default behaviour delete button in thr toolbar select of MUI Data Table. So I have overidden it with customised toolbar select

Hey @leecalcote I have opened a PR #6758 in which I have solved this issue for design page. Kindly review it

@hs05june thanks. Three others are assigned to review.

Hey @leecalcote . I have received review for the PR made to solve this issue on /configuration/patterns. I will be making a PR for /configuration/filters, /configuration/applications also.

@hs05june ok, that’s great. Do update the name of your PR to be represent the specific work.

@Chadha93 fyi


ERROR TypeError: Cannot read property 'style' of undefined

The component works fine, but after switching, it complains about the style that cannot be found. I tried to import the css, as mentioned in the README, but it keeps giving the same error.
Any thoughts on what could cause this?

Hi Sconix,
Not that I am aware of. So I used the component approach, which worked fine, but switched to the directive approach for more control. The slides come from an array that is initalized in the ngOnInit. I initalize the Swiper in ngAfterViewInit. After serving, it immediately shows the "cannot read property 'style' of undefined error.
I am using Swiper 4.4.1. as a dependency, tried with an older version as well, but to no avail.

What do you mean that you initialize Swiper in ngAfterViewInit? If the slides array is updated on ngOnInit then that is most likely the problem. The swiper wrapper initializes in ngAfterViewInit since Swiper needs DOM to be ready, so try moving your slides initialization to the ngAfterViewInit.

Looks like that solved it! Thanks for the help and of course for providing this wrapper!


Make sure spoilers do the right thing in email and mobile push notifications

We should make sure that the new spoilers markdown feature does the right thing in desktop, email and mobile push notifications.  I think the right behavior is to have our logic that translates content into the supported formats for email/push notifications should keep the spoiler header but not the spoiler content.

Push notifications: get_mobile_push_content, tested in zerver/tests/test_push_notifications.py.
Email notifications: build_message_payload, tested in zerver/tests/test_email_notifications.py
Desktop notifications: process_notification in notifications.js.  This is easy to test with an incognito tab sending messages to another tab; it doesn't work.

@aero31aero can you do a similar audit with the new !time feature to make sure it does something sane on this front?  If not, please open a blocker issue for that as well.

@zulipbot claim

Sigh, zulipbot looks for folks with at least 1 commit for that notice, but apparently ignores co-authored-by.

@opheliasdaisies just wanted to check in -- is this something you're planning on fixing?

@timabbott Yes, @Dylnuge and I are still planning on tackling this. That being said, we don't have a ton of time at the moment, so if it's high-priority we're okay with you giving it to someone else.

Fixed in #15798.  Thanks @aero31aero!


assertion left != right failed. Can it be the sidecar feature?

Describe the bug
I am using bsc-reth and my node instance is currently syncing the chain from scratch (node started with --full flag)
Currently at the execution stage. Today I have rebuild bsc-reth with the latest develop branch, that now includes the sidecar support feature (#45)
After restarting the node with the new binary, The node panics with the message assertion left != right failed: A static file <> database inconsistency was found that would trigger an unwind to block 0.
After going back to the previous binary (without #45) the node picks up where it left and continue syncing/executing.
Steps to reproduce

Build bsc-geth with a commit prior to #45 commit.
Start as a full node (--full flag)
Wait for it to sync and get to execution stage
Upgrade bsc-reth with latest develop (or #45 commit)
Start the node

Node logs
2024-07-10T14:37:33.247800Z  INFO Verifying storage consistency.
2024-07-10T14:37:33.308331Z  INFO Setting unwind target. checkpoint_block_number=40103919 unwind_target=0 segment=Sidecars
thread 'main' panicked at /node/bsc-reth/reth/crates/node/builder/src/launch/common.rs:372:13:
assertion left != right failed: A static file <> database inconsistency was found that would trigger an unwind to block 0
  left: Unwind(0)
 right: Unwind(0)
stack backtrace:
   0: rust_begin_unwind
   1: core::panicking::panic_fmt
   2: core::panicking::assert_failed_inner
   3: core::panicking::assert_failed
   4: reth_node_builder::launch::common::LaunchContextWith<reth_node_builder::launch::common::Attached<reth_node_builder::launch::common::WithConfigs,DB>>::create_provider_factory::{{closure}}
   5: <reth_node_builder::launch::DefaultNodeLauncher as reth_node_builder::launch::LaunchNode<reth_node_builder::builder::states::NodeBuilderWithComponents<T,CB>>>::launch_node::{{closure}}
   6: reth_cli_runner::run_to_completion_or_panic::{{closure}}
   7: reth::cli::Cli<Ext>::run
   8: bsc_reth::main
note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.

Platform(s)
Linux (x86)
What version/commit are you on?
e11ee4898
What database version are you on?
2
Which chain / network are you on?
bsc (56)
What type of node are you running?
Full via --full flag
What prune config do you use, if any?
[prune.parts]
sender_recovery = { distance = 2_592_000 }
transaction_lookup = { distance = 2_592_000 }
receipts = { distance = 2_592_000 }
account_history = { distance = 2_592_000 }
storage_history = { distance = 2_592_000 }

### If you've built Reth from source, provide the full command you used

cargo build --bin bsc-reth --features "bsc jemalloc asm-keccak" --profile "maxperf"

### Code of Conduct

- [X] I agree to follow the Code of Conduct

Yes, this new version is not forward compatible.
You could:

sync from 0 using the newest version
revert feat: add support of block sidecars for bsc (#45) commit and build the node
waiting for a fix version


Fix gpg download url for fedora keys

Fedora renamed branches to get rid of master. This broke some download
paths for us for gpg keys.
Here the announcement<EMAIL_ADDRESS>Release note:
NONE

/test pull-kubevirt-verify-rpms

/approve


Define interface for diagnosis key verification server

The publish API current has a placeholder for accepting verifications codes that can be validated against an external authority to verify the keys should be accepted.
That "code server" is not and should not be part of this project, but defining a well formed interface so that this server can integrate with code servers is within the scope of this project.
/kind enhancement

/close in favor of #349
Instead of having the exposure notification server call out, we’re going to define a signed, inbound payload.

/close


Wrong property-will-be-overwritten-by-spread error when property is any

TypeScript Version:  3.9.0-dev.20200328
Search Terms:
property-will-be-overwritten-by-spread
Code
type A = {
    a?: any
}

const a: A = {};
const b = {  a: 'zzz', ...a, }; // 'a' is specified more than once, so this usage will be overwritten

Using string instead of any as the type of a property in A works as expected.
Expected behavior:
No error
Actual behavior:
'a' is specified more than once, so this usage will be overwritten.(2783)
input.ts(6, 24): This spread always overwrites this property.

Playground Link:
Using any errors -  bug
https://www.typescriptlang.org/play/?ts=3.9.0-dev.20200328&ssl=6&ssc=32&pln=1&pc=1#code/C4TwDgpgBAglC8UDeAoK6oEMD8AuLAdiCgL4ooDGA9gQM7Bb5yJIkDclN9UARgsukz4A5AC9xwgDRQAdHMzT2QA
Using string no error - works as expected
https://www.typescriptlang.org/play/?ts=3.9.0-dev.20200328#code/C4TwDgpgBAglC8UDeAoK6oEMD8AuKAzsAE4CWAdgOYoC+KKAxgPblFb5yJI0DcjLbAEYJk6TPgDkALxkSANFAB0yzAt5A
Related Issues: #36779

Might be the fact that a is an optional property but doesn't have undefined in the type (because any eats it).


Feature: Read crowdsecLapiKey from file

Quick thought, it might be a good idea to have this variable readable from a file on the local filesystem, since dynamic traefik configurations usually reside in git repositories, and pushing API keys to git services is a big no no.

Resolved when #53 merges
I added the ability to read from files for certificates and as well for the LAPI key

Awesome! I've also been looking into using tls for the traefik<->crowdsec communication, so this kills two flies with one swing.

Version 1.1.6 has this feature, closing to clean a bit


Game Ready Driver and Driver Studio specific check

Useful program, but should be able to distinguish whether the installed driver version is GRD (Game Ready Driver) or SD (Driver Studio) and consequently check the availability of updates related to the installed driver type.

Good call!
I'll investigate this.

I wrote a very basic AutoIt code to automatically check and retrieve the latest Nvidia Studio driver version.
I followed this guide:
https://github.com/lord-carlos/nvidia-update
And the resulting query is:
https://www.nvidia.com/Download/processFind.aspx?psid=120&pfid=933&osid=135&lid=1&whql=&lang=en-us&ctk=0&dtcid=1

I've encountered the same issue while writing a Powershell installation script for nvidia drivers. How were you able to find which API call was responsible for GRD vs SD? I have tried using dtid=18 to no avail. And i cant find the Studio Drivers anywhere from changing the API calls around

Some GPUs don't have Studio Drivers available.  Which GPU are you using? I just tried with data for a 1080ti and dtid=18 and got the studio driver.
https://www.nvidia.com/Download/processDriver.aspx?psid=101&pfid=845&osid=57&lid=2&dtcid=1&dtid=18

The main issue I am having is that I can't differentiate between the two when they are installed on the computer. I also do not have a recent enough Nvidia card to test with.
To implement an automatic check for SD vs GRD, I would need to find a registry key or file that is different between the driver versions.

I run Win11 with a RTX 3090 Ti card + Studio driver.
If you registry contain the following DCHUVen key stored in:
Computer\HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\nvlddmkm\DCHUVen
Then the system is probably run a DCH driver release, read here for more info:
https://gist.github.com/fyr77/0ac16813c8f469d08789c9785f4368d2

Yes, but both SD and GRD run as DCH drivers. I am already using this registry key as well as a file to check for DCH vs Standard drivers.

I will try later to help you in retrieving the needed info...

I am sorry but I have not been able to find any useful information (either on the web or on my PC) to determine whether the driver installed is the GRD or SD version.

Oh thats interesting.
I had been playing with the variables for a while, but wasnt able to get Studio Drivers.

I've encountered the same issue while writing a Powershell installation script for nvidia drivers. How were you able to find which API call was responsible for GRD vs SD? I have tried using dtid=18 to no avail. And i cant find the Studio Drivers anywhere from changing the API calls around

Some GPUs don't have Studio Drivers available. Which GPU are you using? I just tried with data for a 1080ti and dtid=18 and got the studio driver. https://www.nvidia.com/Download/processDriver.aspx?psid=101&pfid=845&osid=57&lid=2&dtcid=1&dtid=18

I'm working with multiple different GPUs, but i dont think that will cause any issues as the Studio Drivers appear to be the same for every card.

@pruneaue I attempted to document as much of the API as I could on the wiki of this repository. https://github.com/fyr77/EnvyUpdate/wiki/Nvidia-API
Maybe that can help you.

I was unable to find a reliable way of differentiating. Since this is a very minor issue and switching between the drivers is supported, I will close this as not planned.


Windows: Support for heading on windows devices (tablets e.g.)

🚀 Feature Requests
Support to get the heading of the device on a Windows machine. Like when making using it for maps on a windows tablet.
Contextualize the feature
There is already an option for the heading in Android and iOS and there are alot of Windows tablets in use. Since flutter support Windows officially now it would be a great addition to the plugin.
Describe the feature
So it would return the heading of the Position so it can be used in maps for example. This is already an option but returns NaN when used on Windows device. (The device has heading in the Windows offical map app so gps is working properly).

Dear @Cooliolio,
It seems that heading is forwarded when a heading is available on windows...
 if (location.Coordinate().Heading() != nullptr) {     position.insert(std::make_pair(EncodableValue("heading"), EncodableValue(location.Coordinate().Heading().GetDouble())));   }
You could help us a lot by checking out the repository and then try to debug the issue geolocation_plugin.cpp:260. I'll label this issue as a bug for now until we are able to reproduce this. Furthermore, can you tell us something about the testing environment like e.g. brand of the tablet and the output of flutter doctor -v.

Hi @Cooliolio,
The changelog of the Windows package seems to imply this has been fixed in recent versions of the plugin: https://github.com/Baseflow/flutter-geolocator/blob/main/geolocator_windows/CHANGELOG.md.
This should be in effect starting from flutter-permission-handler: ^10.0.0.


--update flag is not value

The --update flag is not valid in this document

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

ID: 2af5c40c-cd4e-f680-2c9a-ac1b0145b3a9
Version Independent ID: 0d3ff836-0f48-1888-92c3-4014bc14b9e8
Content: How to upgrade Azure Dev Spaces tools - Azure Dev Spaces
Content Source: articles/dev-spaces/how-to/upgrade-tools.md
Service: azure-dev-spaces
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@AceHack Thank you for the feedback!  We are investigating this and will get back to you shortly.

@AceHack I was able to get the command to work successfully.  Where exactly did you run into a problem?

This command did not work until I ran
az extension update --name dev-spaces-preview

Then this command worked.  I was not able to find that in the documentation that I needed to run that first command.

it looks like if you call the "az aks use-dev-spaces" it will install the dev-spaces-preview extension if it does not exist but it will not updated older ones.  So if this is the first time you've done it then everything works fine.  But if your on an older extension then things might not work well.
Thanks.

I believe this is no longer an issue. Please reopen if you continue to see issues.
#please-close


Migrating from Redux

This would be very helpful to me since I'm still learning Redux and how it all works.


Printing a decorated function with RemoteOmnisci causes an AttributeError

There is a defect in the __repr__ of RemoteJIT.
>>> from rbc.omniscidb import RemoteOmnisci
>>> omni = RemoteOmnisci()
>>> def farenheight2celcius(f):
...     return (f - 32)* 5 / 9
...
>>> omni('double(double)')(farenheight2celcius)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tupui/rbc/rbc/remotejit.py", line 323, in __repr__
    self.signature, self.local)
  File "/home/tupui/rbc/rbc/remotejit.py", line 319, in local
    return Caller(self.func, self.signature.local)
  File "/home/tupui/rbc/rbc/remotejit.py", line 113, in local
    sig = Signature(self.remotejit.local)
  File "/home/tupui/rbc/rbc/remotejit.py", line 458, in local
    localjit = type(self)(local=True, debug=self.debug)
  File "/home/tupui/rbc/rbc/omniscidb.py", line 277, in __init__
    self._init_thrift_typemap()
  File "/home/tupui/rbc/rbc/omniscidb.py", line 288, in _init_thrift_typemap
    for typename, typ in self.thrift_client.thrift.__dict__.items():
AttributeError: 'LocalClient' object has no attribute 'thrift'

It's actually a problem with the LocalClient:
>>> from rbc.omniscidb import RemoteOmnisci
>>> omni = RemoteOmnisci(local=True)

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/tupui/Documents/Quansight/Projects/Omnisci/rbc/rbc/omniscidb.py", line 277, in __init__
    self._init_thrift_typemap()
  File "/Users/tupui/Documents/Quansight/Projects/Omnisci/rbc/rbc/omniscidb.py", line 288, in _init_thrift_typemap
    for typename, typ in self.thrift_client.thrift.__dict__.items():
AttributeError: 'LocalClient' object has no attribute 'thrift'


AWESOME BOOKS NAVIGATION

**##AWESOME BOOKS NAVIGATION AND CONTACT INFORMATION#**🌳
#IMPLEMENTATIONS🧾
-Navigations were included (Add list, Add New, Contact)✔
-When any of the Nav element are clicked, it displays that particular section on the screen✔
-Styling Of the entire project ✔
-Implementation of some Transition✔

STATUS: INVALID CODE REVIEW REQUEST 🛑 🛑
Hi @citec-47 and @jossi30,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You are required to maintain the headers in the template (as you had before) but customize the contents of the README. Kindly use the template and ensure all the headers are there and complete.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


exception in deferred followed by failure to transmit clock updates to TCP/IP clients

Exception in deferred
Log entry:
2015-11-17 11:50:50.754615500 2015-11-17 11:50:50-0600 [-] <IP_ADDRESS> - - [17/Nov/2015:17:50:50 +0000] "POS
T /RPC2 HTTP/1.1" 200 15460 "-" "xmlrpclib.py/1.0.1 (by www.pythonware.com)"
2015-11-17 11:50:50.755011500 2015-11-17 11:50:50-0600 [-] Unhandled error in Deferred:
2015-11-17 11:50:50.755892500 2015-11-17 11:50:50-0600 [-] Unhandled Error
2015-11-17 11:50:50.755893500   Traceback (most recent call last):
2015-11-17 11:50:50.755893500     File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1192, in run
2015-11-17 11:50:50.755894500       self.mainLoop()
2015-11-17 11:50:50.755894500     File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 1201, in mainLoop
2015-11-17 11:50:50.755895500       self.runUntilCurrent()
2015-11-17 11:50:50.755895500     File "/usr/lib/python2.7/dist-packages/twisted/internet/base.py", line 824, in runUntilCurrent
2015-11-17 11:50:50.755900500       call.func(*call.args, **call.kw)
2015-11-17 11:50:50.755901500     File "/usr/lib/python2.7/dist-packages/twisted/internet/task.py", line 218, in __call__
2015-11-17 11:50:50.755901500       d = defer.maybeDeferred(self.f, *self.a, **self.kw)
2015-11-17 11:50:50.755901500   --- <exception caught here> ---
2015-11-17 11:50:50.755902500     File "/usr/lib/python2.7/dist-packages/twisted/internet/defer.py", line 139, in maybeDeferred
2015-11-17 11:50:50.755904500       result = f(*args, **kw)
2015-11-17 11:50:50.755904500     File "/usr/local/lib/python2.7/dist-packages/txtrader/tws.py", line 342, in EverySecond
2015-11-17 11:50:50.755905500       self.CheckPendingResults()
2015-11-17 11:50:50.755905500     File "/usr/local/lib/python2.7/dist-packages/txtrader/tws.py", line 240, in CheckPendingResults
2015-11-17 11:50:50.755905500       cblist.remove(cb)
2015-11-17 11:50:50.755907500   exceptions.ValueError: list.remove(x): x not in list
2015-11-17 11:50:50.755910500
2015-11-17 11:50:50.794655500 2015-11-17 11:50:50-0600 [-] error: <error id=-1, errorCode=2100, errorMsg=API client has been unsubscribed from account data.>
2015-11-17 11:51:53.241222500 2015-11-17 11:51:53-0600 [HTTPChannel,0,<IP_ADDRESS>] xmlrpc_status()

Re-entrancy not guaranteed by LoopingCall unless the called function returns a deferred, then the library will not reschedule untll the deferred has fired.
The function passed to LoopingCall should create a deferred, then pass the deferred using CallLater with 0 delay to the the work function.  LoopingCall's function returns the deferred immediately, and the framework won't call it again until the work function calls callback() on the deferred.

solved with https://github.com/rstms/txTrader/commit/5b8539544c2b6d5bb73708ca00b99440638181b4


Create "Search Results" page/s

Create "Search Results" page/s

Working on ngx-translate for search table and component

some UI issues are resolved in search component and implemented carbon icon and dropdown for table

Task is completed.


read content from external file for gsoc page.

Describe the bug
Remove all hardcoded things from gsoc.

Really intrested to work on the issue, Can I?


updates codecov workflow

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.02%. Comparing base (32f8f95) to head (4a669d9).
Report is 4 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
- Coverage   75.08%   72.02%   -3.07%
==========================================
  Files           7        8       +1
  Lines         602      579      -23
==========================================
- Hits          452      417      -35
- Misses        150      162      +12

Flag
Coverage Δ

unittests
72.02% <ø> (-3.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


feat: Update HERE Wallet and add topLevelInjected

Reverts near/wallet-selector#1091
This is the original PR:  https://github.com/near/wallet-selector/pull/1077
It was reverted at the direction of Pagoda leadership pending an internal review. We needed to release another change first. The feature is now approved and this revert will release the changes for the feature.

feat:  https://github.com/near/wallet-selector/pull/1077


Presentation Submission: Custom Credential Handling Logic

Following https://github.com/TBD54566975/ssi-service/issues/403
Be able to add processing logic and/or webhooks for additional processing after a Presentation Submission has been validated.
Could work in a manner simple to what Credential Gate does.

I'm trying to discern what is different from what our current webhooks allow for. Currently, we already have the ability to add webhooks around any service endpoint defined in https://developer.tbd.website/docs/apis/ssi-service
Can you elaborate more on the use case that you're considering?

This would be enabling processing in the service as opposed to needing an external call with a webhook. E.g. check for a revocation without needing to fire a webhook off if there's a credential status in a cred

This would introduce two solutions for the same problem. I would like to understand better what the benefits would be. Can you elaborate? See below for how this could be done with existing webhooks.
If you need to do registration in code (similar to what credential gate does), then you would also be able to start a simple http server with your customhandler function. This handler could be called with a webhook that hits localhost. The webhook could be registered at startup. That call would not cross a network boundary.


Implement RFC: Harmonize profile location targets

Cookbook version
2.x
Scenario:
It should be easy for customer to copy profile locations from inspec.yml to chef attributes. In https://github.com/chef/inspec/issues/1227 we defined the target format for audit cookbook. We change from
"profiles" => {
    # org / profile name from Chef Compliance
    "base/linux" => true,
    # supermarket url
    "brewinc/ssh-hardening" => {
      # location where inspec will fetch the profile from
      "source" => "supermarket://hardening/ssh-hardening"
    },
    # local Windows path
    "brewinc/win2012_audit" => {
      # filesystem path
      "source" => "E:/profiles/win2012_audit"
    },
    # github url
    "brewinc/tmp_compliance_profile" => {
      "source" => "https://github.com/nathenharvey/tmp_compliance_profile"
    },
    # disable profile
    "brewinc/tmp_compliance_profile-master" => {
      "source" => "/tmp/tmp_compliance_profile-master",
      "disabled" => true
    }
  }

to
{
  "profiles": [
    {
      "name": "hardening/ssh-hardening"
    },
    {
      "name": "os-hardening",
      "url": "https://github.com/dev-sec/tests-os-hardening/archive/master.zip"
    },
    {
      "git": "https://github.com/dev-sec/tests-ssh-hardening.git"
    },
    {
      "git": "https://github.com/dev-sec/tests-os-hardening.git"
    }
  ]
}

  "profiles": [
    {
      "name": "hardening/ssh-hardening"
    },
    {
      "name": "os-hardening",
      "url": "https://github.com/dev-sec/tests-os-hardening/archive/master.zip"
    },
    {
      "git": "https://github.com/dev-sec/tests-ssh-hardening.git"
    },
    {
      "git": "https://github.com/dev-sec/tests-os-hardening.git"
    }
  ]
}

profiles_array = []

profiles[:profiles].each { |profile| profiles_array.push(profile[:url] || profile[:git]) }

profiles[:profiles].each { |profile|
  if profile.length == 1 && profile[:name]
    profiles_array.push("supermarket://" + profile[:name])
  end
}

profiles[:profiles].each { |profile|
  if profile.include?(:supermarket_url)
    profiles_array.push(profile[:supermarket_url] + "/" + profile[:name])
  end
}

profiles[:profiles].each { |profile|
  if profile.include?(:compliance_url)
    profiles_array.push(profile[:compliance_url] + "/" + profile[:name])
  end
}

final_profiles = profiles_array.reject { |profile| profile.nil? }

puts final_profiles```

i think i'm probably over-complicating this....but does this make sense?

Lets not nest profiles. This should be enough:
"profiles": [
    {
      "name": "hardening/ssh-hardening"
    },
    {
      "name": "os-hardening",
      "url": "https://github.com/dev-sec/tests-os-hardening/archive/master.zip"
    },
    {
      "git": "https://github.com/dev-sec/tests-ssh-hardening.git"
    },
    {
      "git": "https://github.com/dev-sec/tests-os-hardening.git"
    }
  ]


Is it possible to make Kimera Semantics instance-aware?

If I use something like maks RCNN to generate images having instance segmentation, i.e. same class have different colors, will the Kimera-Semantics be able to handle it? If not, any suggestion about how to do that?
I noticed that in kimera semantics config file kimera_semantics_ros/cfg/tesse_multiscene_office1_segmentation_mapping you have something like
...
Painting_A,246,188,6,255,12
Painting_B,246,188,6,255,12
Painting_C,246,188,6,255,12
Painting_D,246,188,6,255,12
...

I am just wondering for the same class 12 if those paints can have a different color setting

Hi @zhaozhongch, that's right, you can use MaskRCNN, the semantic label corresponds to the last column (number 12 above).
I guess a quick approach to get this working with MaskRCNN would be to connect Kimera-Semantics with the semantic class output of MaskRCNN converted to a grayscale image where the pixel values correspond to the semantic class id.
You may want to reserve as well a special id for non-segmented pixel.
You can also change the covariance matrix here.
I just set them to be isotropic (constant diagonal matrix).
I had to add the decoding from colors to semantic labels because our segmentation (semantic mask) is given as RGBA colors instead of ids.
The only good thing about this approach is that the semantic images and the 3d reconstruction have the same colors.
For anything else, this is suboptimal because I am constantly mapping semantic ids to colors and vice-versa...


Fix various typos in some training docs

Fixes #4943.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/153f7d555967f79e7d6d7ec05eda398dea2ea292/
Edited pages:

v19.1/training/locality-and-replication-zones.md
v19.1/training/planned-maintenance.md
v19.2/training/locality-and-replication-zones.md
v19.2/training/planned-maintenance.md
v2.1/training/planned-maintenance.md

@mattcrdb let me know if this fixes your issue.  Thanks for finding (and filing)!

Additional typo, this one raises an error in the training module:
https://www.cockroachlabs.com/docs/stable/training/under-replication-troubleshooting.html
On step 2 the instruction states to reduce the amount of time the cluster waits before considering a node dead to just 1 minute.
The minimum time to do so is 1 minute and 15 seconds, both the instruction and the code snippet should be changed.
./cockroach sql --insecure --host=localhost:26257 --execute="SET CLUSTER SETTING server.time_until_store_dead = '1m0s';"
should be
./cockroach sql --insecure --host=localhost:26257 --execute="SET CLUSTER SETTING server.time_until_store_dead = '1m15s';"

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ccea8dd365cc6c1eaf5230664e93e29269fc7625/
Edited pages:

v19.1/training/locality-and-replication-zones.md
v19.1/training/planned-maintenance.md
v19.1/training/under-replication-troubleshooting.md
v19.2/training/locality-and-replication-zones.md
v19.2/training/planned-maintenance.md
v19.2/training/under-replication-troubleshooting.md
v2.1/training/planned-maintenance.md

Thanks Matt, missed that one.

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c98b14e9529c8d0dffcc0247bbf4c5107c5d3480/
Edited pages:

v19.1/training/locality-and-replication-zones.md
v19.1/training/planned-maintenance.md
v19.1/training/under-replication-troubleshooting.md
v19.2/training/locality-and-replication-zones.md
v19.2/training/planned-maintenance.md
v19.2/training/under-replication-troubleshooting.md
v2.1/training/planned-maintenance.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/c98b14e9529c8d0dffcc0247bbf4c5107c5d3480/


Migrate to PocketBase

PocketBase seems to be a perfect choice for this app.
Will migrate completely to PocketBase when I got time.

Done


Add address validation

Now the validate parameter includes address validation by default; it can be turned off using --no_address_validation


[Event Request] Codeunit 80 "Sales-Post" - procedure PostItemTrackingForShipment - OnPostItemTrackingForShipmentOnBeforeGetATOItemLedgEntriesNotInvoiced

Please add an Event OnPostItemTrackingForShipmentOnBeforeGetATOItemLedgEntriesNotInvoiced in procedure PostItemTrackingForShipment in Codeunit 80 "Sales-Post"
[Event Request] Codeunit 80 - Sales-Post - procedure PostItemTrackingForShipment
    var
        ItemEntryRelation: Record "Item Entry Relation";
        SalesShptLine: Record "Sales Shipment Line";
        RemQtyToInvoiceCurrLine: Decimal;
        RemQtyToInvoiceCurrLineBase: Decimal;
        QtyToBeInvoiced: Decimal;
        QtyToBeInvoicedBase: Decimal;
        ShouldAdjustQuantityRounding: Boolean;
        IsHandled: Boolean;
    begin
        with SalesHeader do begin
            SalesShptLine.Reset();
            case "Document Type" of
                "Document Type"::Order:
                    begin
                        SalesShptLine.SetCurrentKey("Order No.", "Order Line No.");
                        SalesShptLine.SetRange("Order No.", SalesLine."Document No.");
                        SalesShptLine.SetRange("Order Line No.", SalesLine."Line No.");
                    end;
                "Document Type"::Invoice:
                    begin
                        SalesShptLine.SetRange("Document No.", SalesLine."Shipment No.");
                        SalesShptLine.SetRange("Line No.", SalesLine."Shipment Line No.");
                    end;
            end;

            IsHandled := false;
            OnPostItemTrackingForShipmentOnBeforeGetATOItemLedgEntriesNotInvoiced(SalesHeader, IsHandled); <-- NEW EVENT
            if not IsHandled then

                if not TrackingSpecificationExists then
                    HasATOShippedNotInvoiced := GetATOItemLedgEntriesNotInvoiced(SalesLine, TempItemLedgEntryNotInvoiced);

            SalesShptLine.SetFilter("Qty. Shipped Not Invoiced", '<>0');
            ...

[IntegrationEvent(false, false)]
local procedure OnPostItemTrackingForShipmentOnBeforeGetATOItemLedgEntriesNotInvoiced(SalesHeader: Record "Sales Header"; var IsHandled: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 44285.


UserAgent with runtime env

We should try to add to our user agent header in
https://code.google.com/p/google-api-dotnet-client/source/browse/Src/GoogleApis.
Core/Apis/Http/ConfigurableMessageHandler.cs#51 the current runtime (WP, WinRT,
etc.)

See the following class for more details:
http://mvvmlight.codeplex.com/SourceControl/network/forks/onovotny/MvvmLightPort
able/changeset/view/f356af74426f#GalaSoft.MvvmLight/Portable/GalaSoft.MvvmLight/
Helpers/DesignerPlatformLibrary.cs

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 11 Nov 2014 at 2:02

Closing out as stale.


Add styles and animation for navigation bar

Enhance the navigation bar by adding styles (hover animation)

add an flow animation when user moves to another page.


Extend the current standardisation functionality

Extend the current standardisation functionality

An initial demo of how to do this, as well as some extensions, was done in #16


Delete .codeclimate.yml

Pull Request Test Coverage Report for Build 42

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 46.667%

Totals

Change from base Build 40:
0.0%

Covered Lines:
14

Relevant Lines:
30

💛  - Coveralls


HMR with docker and apache proxy

Environment

Operating System: Linux
Node Version:     v22.3.0
Nuxt Version:     3.12.1
CLI Version:      3.12.0
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      -
Runtime Modules:  -
Build Modules:

Reproduction
Complicated to produce a reproduction because it happens with docker.
Describe the bug
When updating to version 3.12.1, HMR no longer works.
Here is my docker development environment:
An Apache proxy container in https with self-signed certificates
A node container with nuxt
Here is my nuxt.config file:
  vite: {
    server: {
      hmr: {
        port: 443,
      },
    },
  },

Worked great in version 3.11.
Now the error message in console browser:
[vite] failed to connect to websocket (SecurityError: The operation is insecure.).

Performing a clean install from Nuxt 3.11 and 3.10 causes the same error.
If I revert my package-lock.json file with version 3.11, everything works again. This makes me think that it is a Nuxt dependency which causes problems, certainly quickly. But I see there were changes to the hmr in version 3.12 so that may also be the reason.
Additional context
No response
Logs
No response

Would you ensure you're using the latest version of nuxi?

Yes, I have the latest version of nuxi:
3.12.0
Ok, I deleted vite.server.hmr and port 443, but I got the same error.
I also tried adding vite.server.hmr.protocol = wss, same error
The strange thing is that it worked before updating to 3.12
I access nuxt via https://example.test (configured in /etc/hosts file).
Then apache proxy redirects traffic to my node container.
It listens for ws connections and redirects them to the node:3000 container
I also tried listening for wss requests in the apache config, but still no.

Would you ensure you're using the latest version of nuxi?
Also, you shouldn't configure vite.server.hmr. Nuxt configures Vite HMR to be through the same port as the dev server.

I think you do need to configure hmr port when using Docker, at least that's what I do in my project
server: {
    hmr: {
        clientPort: (process?.env?.NUXT_PUBLIC_RUNNING_IN_DOCKER === 'false') ? 3000 : 17365
    },
},

I have the same issue, but using nginx. I'm using Nuxt on port 3010 and nginx as a proxy for https. It was working fine until upgrading to Nuxt 3.12. Nuxt 3.12.1 has the same issue for me.
My config:
    // ...
    vite: {
        server: {
            hmr: {
                protocol: 'wss',
                clientPort: 443,
                path: 'hmr/',
            },
        },
    },
    // ...
})```

I tried using it without this config, but got the same issue, just with a different port.

The error I'm getting in Safari:
```[blocked] The page at https://my_domain/my_path/ was not allowed to run insecure content from ws://my_domain:443/app/hmr/.```

As you can see, it tries to use `ws://` instead of `wss://`

- Operating System: Darwin
- Node Version:     v20.11.0
- Nuxt Version:     3.12.1
- CLI Version:      3.12.0
- Nitro Version:    2.9.6
- Package Manager<EMAIL_ADDRESS>- Builder:          -
- User Config:      nitro, runtimeConfig, modules, typescript, components, app, security, routeRules, devServer, imports, colorMode, build, css, vite, vue, experimental
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

FYI: when I manually hardcoded wss:// as protocol in the vite package in setupWebSocket function, everything started working as expected. This makes me think that it's a nuxt/vite configuration issue.

Also experiencing this issue and that vite ignores the vite.hmr config and still uses ws://. Downgrading nuxt doesn't seem to fix the issue, possibly related to some other dependency?

FYI: when I manually hardcoded wss:// as protocol in the vite package in setupWebSocket function, everything started working as expected. This makes me think that it's a nuxt/vite configuration issue.

Yes, change at node_modules/vite/dist/client/client.mjs line 507 protocal hard coded "wss" work for me too.

I'm not entirely sure which PR introduced it (I think either https://github.com/nuxt/cli/pull/420 or possibly https://github.com/nuxt/nuxt/pull/27326), but there is a quick workaround available until this is resolved properly:
Just add the following hook code to your nuxt.config.ts.
  hooks: {
    'vite:extendConfig': (config) => {
      if (typeof config.server!.hmr === 'object') {
        config.server!.hmr.protocol = 'wss';
      }
    },
  },

Really appreciate this workaround and for sharing.  This took me a couple of hours to figure out.  Stabling this and the documentation around changes, how they affect reverse-proxy configs, etc., is hopefully, a priority.
Until such time as this gets figured out...for me:
nuxt.config.js
vite: { // @see https://github.com/nuxt/nuxt/issues/27558
    server: {
      hmr: {
        protocol: "wss",
        clientPort: 443,
        path: "hmr/",
      },
    },
  },
  hooks: { // @see https://github.com/nuxt/nuxt/issues/27558
    'vite:extendConfig': (config) => {
      if (typeof config.server.hmr === 'object') {
        config.server.hmr.protocol = 'wss';
        config.server.hmr.clientPort = 443;
        config.server.hmr.path = "hmr/";
      }
    },
  },

nginx.conf (relevant snippets)
upstream tres_rios_app_upstream {
  server app:3000;
  keepalive 64;
}

  location /_nuxt/hmr/ {
    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection "upgrade";
    proxy_pass http://tres_rios_app_upstream/;
  }

It is a regression introduced with nuxt cli 3.12.2
It took me hours to understand the issue. Nuxt is using Vite for HMR. Usually, Nuxt run on a Port (3000) and Vite has its own port. On this port, Vite exposes a Websocket that is used to updated the website client side.
Nuxt changed that behaviour with a trick and is sharing the same port for Nuxt and Vite. This is done with some hook in nuxi when starting Nuxt.
Last week this behavior was changed and now it breaks if you run your frontend with HTTPS behind a reverse proxy.
Setting custom options in vite has no impact, because the settings are overridden with from Nuxt (except you set en environment variable called NUXI_DISABLE_VITE_HMR=TRUE
But if you do so, Vite and Nuxt will no longer share on the same port and then our reverse proxy setup no longer works. So this is not a way to go.
Example that does NOT work:
vite: {
  server: {
    hmr: {
      protocol: 'wss',
    },
    }
  }
}

Add this hook to you nuxt.config.ts until this is fixed upstream
  hooks: {
    'vite:extend'({ config }) {
      if (config.server && config.server.hmr) {
        // @ts-ignore
        config.server.hmr.protocol = 'wss'
      }
    },
  },

I'm fairly sure it was something about this change that has caused the regression https://github.com/nuxt/cli/commit/1bbd196a931c855e6b1cc526f9f0809d0defaa22
in src/utils/dev.ts changing config.server.hmr = {...} (Lines 193-199) to the following
config.server.hmr = {
+  protocol: undefined,
  ...(config.server.hmr as Exclude<typeof config.server.hmr, boolean>),
-  protocol: undefined,
  port: undefined,
  host: undefined,
  server: this.listener.server,
}

fixes it for me and allows setting the protocol with the normal config option again
vite: {
  server: {
    hmr: { protocol: 'wss' }
  }
}`

This can be done with the current release by editing node_modules\nuxi\dist\chunks\dev2.mjs  at lines 6853-6859
config.server.hmr = {
+  protocol: void 0,
  ...config.server.hmr,
-   protocol: void 0,
  port: void 0,
  host: void 0,
  server: this.listener.server
};

Would you provide a reproduction please? 🙏

Probably quickest way to reproduce is using Caddy.
Add <IP_ADDRESS> nuxt.local to hosts file or change the domain to any local domain below:
caddy reverse-proxy --internal-certs --from nuxt.local --to localhost:3000
Start nuxt dev server (no nuxt.config changes needed, so a new empty project will suffice), then visit https://nuxt.local and notice that it will complain about insecure protocol used in the devtools console. Also changes in any of the .vue files will not trigger HMR in browser.
This works fine when nuxi is downgraded back to 3.11.1 or earlier via package.json:
  "overrides": {
    "nuxi": "3.11.1"
  }

When using pnpm as package manager, you'll probably need to add this to your package.json, using this notation:
"pnpm": {
    "overrides": {
      "nuxt>nuxi": "3.11.1"
    }
  }

Thank you for the reproduction! This should be resolved in the next nuxi release 🙏

I've been banging my head against the wall on this for 2 days. The workaround seems to move the needle on host and protocol but i cannot for the life of me get port to map.
I've resorted to disabling HMR for now.

Dear @danielroe, I see that you mention this should be fixed in the next release.

How does one know when the next release is?
Would it make sense to keep this issue open and close it once it is actually fixed and released?
I just tried to update nuxt/nuxi, and it appears not released at this time.

I, personally, have seen these "closing it's going to be release in the next release" on a few packages (Firebase for example) and then no one knows when that release will actually happen.
Maybe I am missing something or don't comprehend how we would know, but when tracking an issue like this, I find it difficult to stay up to date when it's closed without actually being fixed & released. Which is also signified by others opening new issues like https://github.com/nuxt/nuxt/issues/28082 which is probably a duplicate of this one here.

@chrisspiegl This doesn't fully answer your question, but a helpful tip is that you can watch repositories and receive notifications specifically when new releases come out.
It's still up to you to check the notification and see if the fix you are interested is included in the release, but it's makes it a lot easier to keep track of. I also just find it quite useful to enable for all larger packages I use regularly to see what features are being added.

Click the Watch button in the repository, select Custom

Select Releases (and any other notifications you want to receive),

I know about getting notifications upon GitHub release tags, but I wonder if there is a better way, like:

closing issues when they are actually released instead of fixed
or maybe when it's fixed and closed, stating that there will be an update comment upon release.

I know this is something that at least pull requests do show with the "This PR’s merge commit doesn’t appear in any tags". Perhaps something similar can be achieved with issues upon release?
I realize, this is "off-topic", so I'll leave it at that. But I hope this can be improved for projects like this.

I think it’s a good idea and have been thinking about a bot to comment when a release incorporating a fix is made.

Reproduction over at https://github.com/nuxt/nuxt/issues/27558#issuecomment-2188504331 still fails with nuxt 3.13.1 & nuxi 3.13.1
It looks like the value evaluated here becomes the default, which did not happen before the changes in https://github.com/nuxt/cli/pull/420
https://github.com/nuxt/nuxt/blob/6c98cdc645060b0eea311dcc42f8834a10117011/packages/vite/src/client.ts#L178-L182
Still not quite sure of the order of operations here, when which part of the configuration gets applied, but I think one way to work around this would be to not assume that devServer.https being falsy automatically means that we are using an insecure protocol in the browser. Leaving the false branch undefined here should let vite automatically detect based on the url protocol in browser whether to use ws or wss as the previous commenter tried to explain:
const socketProtocol = __HMR_PROTOCOL__ || (importMetaUrl.protocol === 'https:' ? 'wss' : 'ws')
@danielroe any thoughts?


DOC: cupyx.interpolate: document limitations on ROCm

Splines use banded linear algebra, which we emulate with sparse solvers, and those are not available on ROCm, so document the limitation.

Thanks, @ev-br!


Add files do not trigger re-configurate with glob & CONFIGURE_DEPENDS flag

Brief Issue Summary
Add files do not trigger re-configurate with glob & CONFIGURE_DEPENDS flag
For my buildscript:
file(GLOB_RECURSE CORE_SOURCES CONFIGURE_DEPENDS SOURCES "Core/*.*")

What I expected is to re-configurate automatically when I add some files under Core/ directory.
While cmake --build actually does the re-configuration, it still makes IntelliSense run into mistakes as it re-configurates too late.
CMake Tools Diagnostics
No response
Debug Log
No response
Additional Information
No response

@ArcticLampyrid Thanks for the notification! This is an interesting case, that I don't believe we currently have a solution for. Therefore, I will mark this as an enhancement request.
I think that the possible solution for this would either be a user-configurable setting where you could tell us what source directories you care about, and we react to changes there. However, this is somewhat of a band-aid type fix. Additionally, if the FileAPI response contained information about something like this, then we could import it and use it for a feature like this.
Putting this on the backlog and we will continually monitor the upvotes and interest in this idea!

Track:

Additionally, if the FileAPI response contained information about something like this, then we could import it and use it for a feature like this.

CMake v3.30 is released with such API.

The cmake-file-api(7) "cmakeFiles" version 1 object's version field has been updated to 1.1. It gained a globsDependent field to report file(GLOB) calls using CONFIGURE_DEPENDS.

See also: https://cmake.org/cmake/help/v3.30/manual/cmake-file-api.7.html#cmakefiles-version-1

@ArcticLampyrid I've moved this item from our "Backlog" to "On Deck", thanks for pointing out that the FileAPI has been updated in CMake v3.30.


Use static_cast to cast to int in some places.

This is recommended by the Google C++ style, to use C++ style casts instead of
C-style casts.
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Casting#Casting
Signed-off-by: Thiago Farina<EMAIL_ADDRESS>

It's true the style guide recommends that form, but I've always found it hard to read for the specific case of list length.  I think the old code is better.


Crash Opening 1.5 MB HTML File

Created by Bokeh when opening in 1.0.7 on a Windows 7 x64 machine. Also occurred in Safe Mode. I was able to open a small ~25 KB webpage without issues.

FILE

Thanks for the report, @MatthewSteen, but it would be much more helpful if you provided more information.
Also, can you share the file in question? Can you reproduce the crash in safe mode? Did Atom actually crash or did you observe an exception?

@izuzak see above for additional info.

@MatthewSteen Thanks for the update! Looking at that file it seems like this is happening because of a very long line, which is the same issue as https://github.com/atom/atom/issues/979 - I'm gonna close this out in favor of that issue.

sounds good @mnquintana, I did a search for my specific case but didn't see that one.


How to deal with maps loaded but hidden on current page?

Hi Mourner,
If a map is loaded hidden on a page and then gets displayed via toggle link or tab change for example, the map is not shown properly and you have to call invalidateSize() in order to fix this.
The issue here for me is that within my Wordpress plugin I cannot always detect if the map is loaded in a hidden div - I built a workaround to support jQuery tabs which is widely used and supports events which are fired when a tab is changed and as a followup calls invalidateSize(). Unfortunately many non-standardized solutions exist where the map does not show up properly and which do not fire events I could use in order to fire invalidateSize().
It would be great if a future version of leaflet could support proper map display even if it is loaded in a hidden div, but as I am pretty sure that this is not an option, I´d appreciate any hint on how to approach this issue.
thx,
Robert

I cannot get invalidateSize() to fire, it keeps telling me
"Uncaught TypeError: this.$.map.invalidateSize is not a function"

@lismore sounds like a problem with your setup, since this.$ isn't something Leaflet uses.
If you think this is a Leaflet issue anyway, please make a JSFiddle or similar that illustrates the problem, and create a new issue.

thanks for the quick response

@lismore you should probably ask/file a bug with the authors of the custom element, and they can create a Leaflet issue of there's really a problem in Leaflet.
It's unfortunately impossible for the Leaflet developers to keep track of how all projects that wrap and use Leaflet work.

invalidateSize works, but it does not position open marker popups very well when they are already open.


Added example command for restoring db.sqlite3

That's not really how it should work. The file should just be put into the host folder that is mounted as data folder into the Docker container. I see no need to provide a command to copy the file into the container, which is potentially confusing and just a convoluted way of putting it into the folder on the host system.

No worries. I was working with volumes instead of bind mounts, so it's not quite as straight forward to copy files directly into the filesystem.  The advantage of using docker cp from a documentation point of view is that it works for both bind mounts and volumes.


[WIP]feat: Support user list in admin.

Admin can allow common user's auth to public, and common user's articles can be reached in home page.


stepping through lines in the debugger gets really slow when there are particles in the scene

Godot version
4.0beta15
System information
windows10x64
Issue description
It becomes impossible to debug when there are particle nodes in the scene because stepping through lines gets really slow for some reason, even on empty scenes. For 2d particles this is happening with high particle count, but for 3d it happens even with low particle count.
Steps to reproduce

open the attached project and run the only scene it has
try to step through the script's code and see how slow it gets

Minimal reproduction project
test.zip
Update
I have updated the attached project to add two new 2D scenarios, one with gpu particles e other with cpu particles.
With a slightly higher particle count (I'm testing with 2048), poor debugger performance happens in any scenario, regardless of using CPU or GPU particles, regardless of being 2D or 3D, regardless of using opengl3, vulkan mobile or vulkan forward plus renderer. Previously I tought that vulkan with gpu particles was the only case where the poor performance didn't happen, but this is only true for low particle count. With a higher particle count every scenario has poor debugger performance.

Can you reproduce this with a GPUParticles2D node, or a CPUParticles3D node (+ a Camera3D node)?

It is reproducible both on vulkan and opengl, with both cpuparticles2d and gpuparticles2d. But it doesn't happen with 3d particles, neither cpuparticles3d nor gpuparticles3d.

I tested on another machine. The only particle that doesn't affect the debugger is the GpuParticles2D on vulkan (since opengl doesn't support gpu particles). The poor debugger performance was reproducible with all other particles, regardless of using vulkan or opengl.
I'm uploading another test project with four scenes, one per particle type.
NewGameProjectG4.zip

still reproduces on beta 4

I have what seems to be the same issue. Extremely slow stepping through/resuming, and yes I have particles in the scene

I have what seems to be the same issue. Extremely slow stepping through/resuming, and yes I have particles in the scene

Could you please test the attached project on both vulkan and opengl backend and see what particles cause slow debug stepping on your machine? In my setup debug stepping only happens without problems with GpuParticles2D on vulkan.

I'm using GPUParticles3D and I think one CPUParticles3D, both exhibit the issue

I have updated the issue description and the attached project. I have found that, at least on my machine, all scenarios shows poor debugger performance.

I noticed a similar bug happens just by using the TIME uniform in a shader. I reported it here:
https://github.com/godotengine/godot/issues/68232

still reproducible on beta 5

The current state of this issue in beta 13 is that, regardless of rendering backend (fordward+, mobile or compatibility), stepping through lines in the debugger while running 2d particles (either cpu or gpu) causes no problems, but with 3d particles (either cpu or gpu) the stepping is really slow.

@clayjohn Why remove this from 4.0 and the issues roadmap? I got your other message that 4.x still includes 4.0, but these actions certainly look like "we're not going to fix this before releasing 4.0". Yet this issue makes debugging virtually impossible to use. Since we're using TIME on our foliage shader, which we have in most scenes, debugging is effectively broken. It pauses like 5 seconds between clicks when browsing the scene tree, and stepping through code doesn't work.

@TokisanGames We're releasing 4.0 in a matter of (a couple of) weeks. If you've been living with this annoying issue until now, you might need to keep living with it when 4.0 is released too.
It will be fixed when someone has time to get to it but it wasn't assessed to be a showstopper for release. Once fixed, the fix can be backported to the 4.0 branch for a patch release.

@TokisanGames We're releasing 4.0 in a matter of (a couple of) weeks.

@akien-mga OMG what? It's barely beta software, right now. I suppose you could call being unable to debug because it's broken, "living with."

it wasn't assessed to be a showstopper for release.

Why are you guys rushing broken software out the door? It's not just the debugger. There are countless issues that are still in a beta state at best. The API is still changing every week! There are new features coming out regularly. What happened to feature freezes to work on bugs? Don't get me wrong, there has been a lot of great progress. But there's no way I would call this ready for release in 2-3 weeks. Game breaking issues keep coming out on each release so I only test every 2-3 of them, and I know I'm not alone in that.

See https://godotengine.org/article/release-management-4-0-and-beyond/
We think Godot 4 is now in a state where more users can start using it to prototype new projects, and that's what 4.0 will be for.
It will take time to reach the level of polish that you're expecting, which will happen over frequent 4.x releases later this year.

Ok, good luck. Thanks for the blog post.

It will take time to reach the level of polish that you're expecting

Yeah.
Though I'm not expecting perfection. How about just a feature and api freeze and bug fixing cycle?

We have been in feature freeze since mid-August

Not really! The freeze hasn't even started yet. The API changed 7 days ago, if not more recently, and new features were added 3 days ago!

As much as we appreciate your passionate concerns, this is not really the place to discuss this. Let's not pollute a legitimate issue ticket with a tangential talk.

Following up from https://github.com/godotengine/godot/issues/72118#issuecomment-1405701525
This bug appears to be related to situations where the engine makes a redraw request to ensure that the editor updates continuously by calling RenderingServerDefault::redraw_request()
This request is validated in the main loop by checking RenderingServer::get_singleton()->has_changed():
https://github.com/godotengine/godot/blob/73b78bf749a2f599a23e0982fdd5e3a3060f30dd/main/main.cpp#L3132-L3144
So the difference between these cases and the normal case is that the engine is actually drawing a frame instead of spinning an empty loop:
RenderingServer::get_singleton()->draw(true, scaled_step); // flush visual commands
 Engine::get_singleton()->frames_drawn++;

To help with further debugging, does this issue persist if "update_continuously" is enabled in the editor settings?

Thanks for looking at it.
For me, Enabling update_continuously makes no difference. Hopefully it will help others.
I have some scenes where the remote scene tree responds quickly and other larger scenes where clicking on nodes takes upwards of 5 seconds with the editor window not responding in between. The largest scene has foliage with TIME, particles, and many static meshes. The remote scene tree gets faster as I delete objects, but it's not clear that it's caused by any specific object or particular type from those three. I can get more speed by deleting some, but if I undo that and delete the others, it behaves the same. Even if I delete all of the foliage and particles, but have many static objects, it's still slow, but not as extreme.
The only thing that works is deleting most of the meshes out of my scene. The editor reports ~9900 objects in my viewport from a birds eye view.
A different scene with a fast remote scene tree only shows 2100 objects. However when the remote scene tree is open, the editor pauses every couple of seconds, until I switch back to the local scene tree.

I can confirm that setting "editor settings > interface > editor > update continuously" to true works around the issue. At least on my setup: godot 4 beta 15, windows 10, nvidia graphics.
Doing this I can step through lines in the debugger normally. This works for this (https://github.com/godotengine/godot/issues/67696) bug, the one with TIME (https://github.com/godotengine/godot/issues/68232) and the one with animated tiles (https://github.com/godotengine/godot/issues/72118)

Activating #update continuously# does not work for me as fix for #72118.
(Godot 4 Beta ??/12/13/14/15, windows 10, nvidia graphics)
Debugging becomes unuseable as soon the animated tiles are displayed on screen.
Debugging speed is directly affected by the amount of displayed animated tiles on screen**.
One single animated tile didnt changed a lot, but with 3 tiles a jump between code lines already takes 3 to 5 seconds.

I created also a minimal test project and it worked there ?!

I have just noticed this too. On a minimal test project the "update continuously" workaround works fine for animated tiles, but not on my main project.
While investigating the issue I have noticed that, by removing from my script all member properties that held references to tilemap nodes, the debug stepping become as quick as in an empty test project. But the more properties I had referencing tilemap nodes, the worse the debug step performance. Even if multiple properties reference the exact same tilemap node. Even if the referenced node doesn't have any animated tile, just normal static tiles.
For the time being, this could be worked around by replacing the node typed property with just it's instance id. That is, replacing this:
@onready var map:TileMap = $Map

with this:
@onready var map:int = $Map.get_instance_id()
func get_map()->TileMap: return instance_from_id(map)

Since this happens even without using animated tiles, I think this deserves a separate issue.

Experimented a bit more with this issue while doing something else. It looks like this is a threading issue somewhere in the renderer or in the debugger. As adding a print statement in the main loop causes the issue to go away entirely.
I added a print statement like so:
if ((!force_redraw_requested) && OS::get_singleton()->is_in_low_processor_usage_mode()) {
 	if (RenderingServer::get_singleton()->has_changed()) {
 		RenderingServer::get_singleton()->draw(true, scaled_step); // flush visual commands
 		Engine::get_singleton()->frames_drawn++;
 	}
	print_line("print statement");
 } else {
 	RenderingServer::get_singleton()->draw(true, scaled_step); // flush visual commands
 	Engine::get_singleton()->frames_drawn++;
 	force_redraw_requested = false;
 }

And the issue went away completely.
CC @RandomShaper and @Faless Do either of you have an intuition for why a threading issue would appear here? For reference if the first branch is run (if ((!force_redraw_requested) && OS::get_singleton()->is_in_low_processor_usage_mode()) { ) the debugger takes 10-20 seconds to go to the next line while if the else condition is run the debugger runs fine.

Does this magic print also work with https://github.com/godotengine/godot/issues/68232 , https://github.com/godotengine/godot/issues/72118 and https://github.com/godotengine/godot/issues/72180 ?

I have a hunch that one side of the debugger needs to be notified somehow, but the low CPU mode is imposing a wait. This is a very abstract idea, though. I don't get why precisely stuff that forces update (TIME, particles, animated tiles) is the one with the issue. I'd need to investigate a bit myself.


2.6 UX: Apps & Marketplace chart install polishing

Tested on Rancher master-head version cb94b9c
QA questions and feedback on chart install changes made here: https://github.com/rancher/dashboard/issues/2670

[ ] Choosing the chart version before installing is not obvious. Users may mistake this for a link to the github version. Is there a way to make this more intuitive to users?
[ ] What are these dates next to the versions? Why are they the same date?
[ ] It is not obvious what chart link downloads.
[ ] For OPA, the home and related link redirects the user to the same link.

[ ] The monitoring chart version is different from the application version. What is the difference? Is there a way to make this clearer?

[ ] During install step 1, the version number is too small. Can this be made bigger? Or have the chart name smaller?
[ ] Is there a reason "enable additional deployment options" is shown on step 1? Should this be hidden by default? The selection is far away from the rest of the fields as well.

[ ] In dark theme, some options look disabled and the font is hard to read.

@richard-cox Let's discuss these

The UI has changed a bit since this issue was opened. For example, the bug with the chart version dates has been fixed:

Since this issue is so old, I'm closing it in favor of the newer https://github.com/rancher/dashboard/issues/2671


Unnecessary dependencies

Hi
Your component lists two dependencies (react, mathjax). I belive you should remove both dependencies, as MathJax is available via the script import (and so doesn't require any local files) and react is already listed as peer dependency, which is the better choice.
I removed those dependencies and your component still works perfectly, as long as there is react in the project (as peer).

@poiuqwer78 sorry for delay,
should be solved now


Ability to automatically shorten classnames used inside emails

Currently we do declare readable and usually long classnames for the MJML components.
This is very good for development phase, however this increases the final HTML size (making gmail client cut the emails).
Maybe it's possible to add a post processing step to shorten them?
mjml('<mjml>...</mjml', {postProcessing: {shortenClassNames: true}});

So in development mode we would use false and for production one it would be true.
What do you think?

I think that's not the job of MJML to provide this. The minify tool that we use should do the trick to avoid any client clipping issue for like 99% of projects.
I think here, you want to do a "deep" optimization and that would require to have an external deps like clean-css to do that for you.


[Bug]: vllm not support fp8 kv cache when use flashinfer

Your current environment

Collecting environment information...
PyTorch version: 2.3.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.27.6
Libc version: glibc-2.35

Python version: 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-5.4.119-19.0009.44-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 12.2.140
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA L20
GPU 1: NVIDIA L20
GPU 2: NVIDIA L20
GPU 3: NVIDIA L20
GPU 4: NVIDIA L20
GPU 5: NVIDIA L20
GPU 6: NVIDIA L20
GPU 7: NVIDIA L20

Nvidia driver version: 535.161.07
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.9.5
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Address sizes:                   52 bits physical, 48 bits virtual
Byte Order:                      Little Endian
CPU(s):                          384
On-line CPU(s) list:             0-383
Vendor ID:                       AuthenticAMD
BIOS Vendor ID:                  Red Hat
Model name:                      AMD EPYC 9K84 96-Core Processor
BIOS Model name:                 3.0
CPU family:                      25
Model:                           17
Thread(s) per core:              2
Core(s) per socket:              96
Socket(s):                       2
Stepping:                        0
BogoMIPS:                        5200.07
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid amd_dcm tsc_known_freq pni pclmulqdq monitor ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch osvw topoext perfctr_core invpcid_single ibpb vmmcall fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 avx512_bf16 clzero xsaveerptr wbnoinvd arat avx512vbmi umip avx512_vbmi2 vaes vpclmulqdq avx512_vnni avx512_bitalg avx512_vpopcntdq rdpid fsrm
Hypervisor vendor:               KVM
Virtualization type:             full
L1d cache:                       6 MiB (192 instances)
L1i cache:                       6 MiB (192 instances)
L2 cache:                        192 MiB (192 instances)
L3 cache:                        768 MiB (24 instances)
NUMA node(s):                    2
NUMA node0 CPU(s):               0-191
NUMA node1 CPU(s):               192-383
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Spec store bypass: Vulnerable
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Full AMD retpoline, IBPB conditional, STIBP disabled, RSB filling
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected

Versions of relevant libraries:
[pip3] flashinfer==0.1.0+cu121torch2.3
[pip3] numpy==1.22.2
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] onnx==1.14.0
[pip3] pytorch-quantization==2.1.2
[pip3] torch==2.3.0
[pip3] torch-tensorrt==0.0.0
[pip3] torchdata==0.7.0a0
[pip3] torchtext==0.16.0a0
[pip3] torchvision==0.18.0
[pip3] transformers==4.42.3
[pip3] triton==2.3.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.1
vLLM Build Flags:
CUDA Archs: 5.2 6.0 6.1 7.0 7.2 7.5 8.0 8.6 8.7 9.0+PTX; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    GPU1    GPU2    GPU3    GPU4    GPU5    GPU6    GPU7    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      NODE    NODE    NODE    SYS     SYS     SYS     SYS     0-191   0               N/A
GPU1    NODE     X      PIX     NODE    SYS     SYS     SYS     SYS     0-191   0               N/A
GPU2    NODE    PIX      X      NODE    SYS     SYS     SYS     SYS     0-191   0               N/A
GPU3    NODE    NODE    NODE     X      SYS     SYS     SYS     SYS     0-191   0               N/A
GPU4    SYS     SYS     SYS     SYS      X      NODE    NODE    NODE    192-383 1               N/A
GPU5    SYS     SYS     SYS     SYS     NODE     X      PIX     NODE    192-383 1               N/A
GPU6    SYS     SYS     SYS     SYS     NODE    PIX      X      NODE    192-383 1               N/A
GPU7    SYS     SYS     SYS     SYS     NODE    NODE    NODE     X      192-383 1               N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

🐛 Describe the bug
export CUDA_VISIBLE_DEVICES=1
export VLLM_ATTENTION_BACKEND=FLASHINFER
python -m vllm.entrypoints.openai.api_server
--port=8006
--host=<IP_ADDRESS>
--served-model-name=vllm_fp8_kvcache
--model=deepseek-ai/DeepSeek-Coder-V2-Instruct
--quantization fp8
--kv-cache-dtype=fp8
--tensor-parallel-size=1
--dtype=bfloat16
--max-model-len=8192
i run vllm use the script, i want to use fp8 attention which Q*transopse(Key) or softmax * value in attention support fp8 compute by FLASHINFER, such as this issue https://github.com/vllm-project/vllm/issues/6246 . the  error .

i set vllm version to 0.5.2 and flashinfer version to 0.8, same error, who can tell me why .

any aone care it ?  @DarkLight1337

@Yard1 can you help with this?

I'm not sure if this is supported yet?

thanks, when this feature will be support? @Yard1

Met the same issue, is there any plan to support this feature?

same error, hope flashinfer to support fp8 kvcache + cuda graph


missing udev cause failing on rosbuildfirm

http://build.ros.org/view/Ksrc_uX/job/Kbin_uX64__realsense_camera__ubuntu_xenial_amd64__binary/92/console
23:26:48   4857 '/var/lib/dpkg/info/ros-kinetic-librealsense.postinst: 40: /var/lib/dpkg/info/ros-kinetic-librealsense.postinst: udevadm: not found'
23:26:48   4858 'libkmod: ERROR ../libkmod/libkmod.c:586 kmod_search_moddep: could not open moddep file '/lib/modules/4.15.0-20-generic/modules.dep.bin''
23:26:48   4859 'modinfo: ERROR: Module alias uvcvideo not found.'
23:26:48   4860 'dpkg: error processing package ros-kinetic-librealsense (--configure):'
23:26:48   4861 ' subprocess installed post-installation script returned error exit status 1'
23:26:48   4862 'Setting up ros-kinetic-roslint (0.11.0-0xenial-20180222-193217-0800) ...'

this pr requires https://github.com/ros/rosdistro/pull/18111

Hi @k-okada
Thanks for the PR. From reading the linked issues, I still don't understand if the fix will be sufficient to solve the problem. If follow-up modifications will be required please open new PR


Space initiative

Tidy the spacing between the text within our initiatives that was placed on each other.
Screenshot Before

Screenshot After

Thanks, @hjdesigner!!! :heart:


Installation and version control for Kiln

Problem: developers might want to install and handle different version on Kiln through nom. Currently, they handle that through an entirely different interview process that would let them change the installed version.
Solutions:

better couple the installation interview with the running interview, and make it easier to discover from the running tests interview.
complete couple the installed version of Kiln with the installed version of this package; then, to change the installed version of kiln, you do it through the same interface you handle different docassemble packages with

Thanks for the issue! Adding a little commentary:

better couple the installation interview with the running interview, and make it easier to discover from the running tests interview.

Sorry I didn't describe clearly - currently the installation and test running are completely coupled. The presentation didn't make that clear and I should add a slide. I was actually suggesting making them less coupled and having two separate interviews, but somehow making the installation interview very discoverable (as the suggestion above describes)

complete couple the installed version of Kiln with the installed version of this package; then, to change the installed version of kiln, you do it through the same interface you handle different docassemble packages with

The pro here is that it would be the same workflow as folks use for their other packages. The con would be that it would be hard for folks to go back to previous versions. Occurred to me later: Currently this worries me as I think ALKiln is still in its infancy, or at least still young, and I want devs to be able to easily revert if they need to. Especially once we have shrinkwrap in place - that way they'll be sure of the dependencies as well.


teamspeak3: Update to Version 3.6.2

Closes #XXXX

Relates to #XXXX

[x] I have read the Contributing Guide.

/verify

/verify
teamspeak has now prioritized 5.00-beta on the Downloads Page and teamspeak3 on Archives. so I've removed Checkver and Autoupdate for now. If anyone would like to help me fix Checkver and Autoupdate I would appreciate it!

/verify
@chriba Thank you

Hmm, apparently teamspeak doesn't like it when I access their file structure directly. As of today, only direct access to the files is allowed and not to the folder structure. However, the archive page still only offers an outdated version...

pre_releases File Directory is also no longer available. It was still available a few hours ago.
It seems that TeamSpeak wants everyone to go to the Microsoft Store and download their teamspeak5-beta version.
😓Or maybe they will fix the link to the file structure?

Therefore my solution no longer works, I currently see no viable solution to fix this.

Thank you anyway😄


Health check endpoint

/health endpoint returns:

enabled models
cache status
cache size
cache limit

https://github.com/deadbits/vector-embedding-api/commit/838037273ad0d60b98b2d8ff195931be4507ca2f


Add fast_executemany option to executemany

Note the issue with inserting Decimal objects into Float columns.

This is closed by #115
The issue above is tracked here


updated plex-media-server (<IP_ADDRESS>.1714-7be11e1)

Thank you for the contribution. It was merged directly as 8ddcfb2837b3c07b5b41bc8b2e83d3d43a3f9a3f to keep commit history cleaner. Your contribution is still credited to you.


Update 10 HA Configuration Issues.ps1

Make Cluster HA Admission Control configurable

What doesn't work till now, that the message output in case of an issue is misleading currently. That comes from the $pLang variable contents. Those should be more generic or possible to set by Variable to.

Looks good, but can you please submit to the dev branch? Thanks


docs: Refactor enhanced plural docs

Hey @karimshalapy,
as discussed, here's a suggestion for how I'd restructure the pluralization docs.
While reworking the content, I think generally where things went in a slightly different direction, is that your proposed docs were very on the correct and elaborative side. I think it's very important to be correct and precise, but I feel like this sometimes gets in the way of making the docs easy to read and skim. My impression is that developers can quickly catch up something like Hello {name} and understand that name is a variable that can be interpolated without digging too much into the terminology and syntax.
I think the link to the ICU docs that you've included is quite important for those users who are interested in a deep dive, but I think I'd focus the next-intl docs a bit more on what we provide on top of the syntax, with expandable deep dives where there's a strong need for more clarity.
Therefore this proposal moves a few pieces back into place where they were before and adds supplemental deep dives for the pluralization features that were the original aim of this change.
Can you have a look and let me know what you think? I'd also really appreciate if you can confirm the statements about Arabic in the docs, they're based on a quick research on my side! 🙂 Thanks!

Changes:

Remove danger type for Callout
Remove invalid markers in configuration
Move plural and enums docs back to messages page and restructure content

@amannn
Regarding your request about the Arabic statement review.
The only statement about the Arabic language that I found in the changes is the following.

For example, English has two forms: One for the singular (e.g. "1 follower") and one for everything else (e.g. "0 followers" and "2 followers"). In contrast, Chinese only has one form, while Arabic has six

It's a bit complicated, to be honest, so this statement is correct in a way.
When it comes to cardinal pluralization, we have 5 forms of words:

Singular
Dual (I don't know how it translates to English but it's used when the count of something is 2)
Sound Masculine Plurals
Sound Feminine Plurals
Broken Plurals

So, if you count the 0 state of the word we'll have 6 forms, but the 0 state shares the form with one of the plurals.
It doesn't stop there, all these forms are not going to be used in this situation because the language is gendered, so you'll most likely at max use 3 forms of the word in the same sentence while interpolating plurals.
Just to clarify something, by forms I mean how the word will look, and by state I mean in which range of the count.
So, if you're talking about how many forms of a word can be used in a sentence while interpolating plurals, then it should be 3.
If you're talking about the different states (0, 1, 2, few, other) then they should be 5.
If you're talking about all the states that can be used in the Arabic language then it should be 5 as well (the list mentioned above).
That being said, I am not a linguistic genius and I might have dropped one form somehow, so if you need more info regarding this try sharing whatever document you found that listed these 6 forms.
I hope this was helpful, and if this wasn't the statement you wanted me to review, please let me know which one you meant.

@amannn You're totally correct, it should not be ignored, but knowing that # corresponds to {variable, number} is a valuable piece of information that I would miss without explicitly reading it somewhere because you might need to have an edge case where you'll use a custom format for this particular instance as follows: {variable, number. <custom-format>}. and without knowing that you can replace the # and use any other variable.
This can happen for artistic reasons, the designer for some reason in a special case wants custom formatting for it, or whatever.
It's like a worthy mention that can be added in a collapsible menu just like you did with the plural ranges keywords.
That being said, I agree with you, and not a whole lot of people will face such scenarios, but if some face it they won't find the answer easily.
Regarding, the Arabic language statement, you can keep it, especially because you already linked: https://www.unicode.org/cldr/charts/43/supplemental/language_plural_rules.html
I'll merge this PR for now, and if you think that the # is worthy of adding as a sidenote or a collapsable menu, it can be added later before merging my branch onto your main repo.


Prevent yarn install

As part awsmobile init there's run a step that invokes yarn install, but why?
My usecase:
For building in CI I've made awsmobile-cli a part of my devDependencies to ease the use of the commands in my npm run-script's.
I have a command that somewhat looks like this awsmobile configure aws $AWS_ACCESS_KEY_ID $AWS_SECRET_ACCESS_KEY $AWS_DEFAULT_REGION && awsmobile init -y $AWS_MOBILEHUB_PROJECT_ID that first configures awsmobile-cli then init's the MobileHub project and downloads it from AWS. So far so good but after successful initialisation yarn install is invoked and I can't see that step provides anything but extended build time so at least making it optional would be a nice thing. BTW, I'm not hosting my app with AWS and I'm using npm everywhere else so the resulting yarn.lock file in my tree is a bit annoying.

One of the last steps of the init command is to install the dependency of the aws-amplify library.
If your project is a react or react-native project, and the system has yarn installed, the cli will use yarn to add the dependency, otherwise it will use npm.
I will mark this as a feature request to make this part of the cli configurable when used in a script.

I'm also facing this problem. Build time is longer and there is no way to fix aws-amplify and aws-amplify-react versions in package.json. That can cause issues. Maybe one option would be to check the presence of those packages, if they exist just skip this step.

I'm currently having issues with CI because the version of aws-amplify installed by awsmobile init is different from the version we specify in package.json. So when our build server runs this, it pushes up a different version of aws-amplify with breaking changes.
I haven't seen a way to specify the AWS Mobile Project ID without doing an init.

Looking at the API a little more, it would be awesome if I could pass the AWS Mobile Project ID to awsmobile pull. Or pass it to awsmobile configure aws and then run awsmobile pull.


fixing the dependencies

When the original optional dep was added to EMT and Explorations, we forgot to add circular dependency protection to Thaumic Boots despite the fact we added it to EMT and Explorations

compile is deprecated. you shouldnt really use that at all. read here for the options: https://github.com/GTNewHorizons/ExampleMod1.7.10/blob/master/dependencies.gradle

Sure


Time validation?

Currently, date and time values are not validated on read.
Validation, on the other side, should occur.
The problem is, why?
The best way I figured out up now is to extend the argument list of toTime function with an optional output value, lIsValid, which on output is .true. if the date was found valid, .false. otherwise.
This does not safeguards against improper assignments by using the standard constructor Time(...). This suggests also introducing an isValid logical function, applicable to Time values.
Please when you do, start with the new function and then reference it in a second change proposal.

Other functions are likely to produce invalid dates: all converting to and from string.
The same treatment, via optional argument lIsValid, would be beneficial.


[BUG] Assertion failed: t.data.dptr != nullptr. Input x is not allocated!

While running RMS norm, I got the following exception:
/workspace/TransformerEngine/transformer_engine/common/transformer_engine.cpp:39 in function CheckInputTensor: Assertion failed: t.data.dptr != nullptr. Input x is not allocated!
  File "/usr/local/lib/python3.11/dist-packages/transformer_engine/pytorch/module/rmsnorm.py", line 50, in forward
    rmsnorm_out, rsigma = tex.rmsnorm_fwd(inputmat, rmsnorm_weight,
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/torch/autograd/function.py", line 553, in apply
    return super().apply(*args, **kwargs)  # type: ignore[misc]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

However, clearly, all input tensors are allocated — I verified this with debugger

Can you provide a reproducer? The following runs for me:
import torch
import transformer_engine.pytorch as te

# Options
batch_size = 128
hidden_size = 128
dtype = torch.float32
device = torch.device("cuda")

# TE module
layer = te.RMSNorm(hidden_size, params_dtype=dtype, device=device)

# Synthetic data
x = torch.randn([batch_size, hidden_size], dtype=dtype, device=device, requires_grad=True)

# Forward and backward pass
y = layer(x)
y.sum().backward()

Can you provide a minimal reproducer? The following runs for me:
import torch
import transformer_engine.pytorch as te

# Options
batch_size = 128
hidden_size = 128
dtype = torch.float32
device = torch.device("cuda")

# TE module
layer = te.RMSNorm(hidden_size, params_dtype=dtype, device=device)

# Synthetic data
x = torch.randn([batch_size, hidden_size], dtype=dtype, device=device, requires_grad=True)

# Forward and backward pass
y = layer(x)
y.sum().backward()

Hey!
This appeared when I tried to use Fp8Tensor. I'll try to write a minimal example, but this could be rather hard

Most of our kernels don't handle Float8Tensor directly. Also, our RMSNorm kernel doesn't support FP8 input at the moment, just FP8 output. As a quick fix, you could manually cast your Float8Tensor to higher precision with:
x = Float8Tensor(...)
y = layer(x.from_float8())