[BUG] - Prop id did not match. Server: "react-aria-1" Client: "react-aria-2" for Input component with Label

Describe the bug
When I'm using the Input component with a label prop to display, I'm always getting a hydration error that the id for the label component (which renders automatically) on the server and client doesn't match.
Upon debugging I find that the issue is related to the <label> element which is being handled via the React-Aria package of Adobe Spectrum Library.
A possible solution could be to allow passing more label props (such as htmlFor, id, etc) in the Input component itself.
<Input
      clearable
      placeholder="Your Full Name"
      label="Name"
      width="300px"
      helperText={errors.name?.message}
      helperColor={errors.name ? 'error' : 'default'}   />

Currently I'm passing an id attribute in props for Input component. If id is not passed in props, then I get the error of htmlFor mismatch on server and client.
NextJS version: 13.0.3

Your Example Website or App
https://stackblitz.com/edit/nextjs-rlmewg?file=pages%2Findex.js
Steps to Reproduce the Bug or Issue

Install next-ui in your next.js project.
Use the Input component and pass a label prop to it.
Check the browser console for the error.

You can check here also - https://stackblitz.com/edit/nextjs-rlmewg?file=pages%2Findex.js
Expected behavior
The client and server id and other attribute htmlFor (or for) should remain same after hydration.
Screenshots or Videos

Operating System Version
macOS
Browser
Chrome

I had a similar issue but I forgot what I did to solve my issue. The one difference I made was switching to the canary build of Next.JS via npm install next@canary.

I had a similar issue but I forgot what I did to solve my issue. The one difference I made was switching to the canary build of Next.JS via npm install next@canary.

Which canary version did you use? NextJS has a latest stable release (2 days ago) as of now after the latest canary (3 days ago) Check here

Which canary version did you use? NextJS has a latest stable release (2 days ago) as of now after the latest canary (3 days ago) Check here and I am using this stable release only and still got the error.

I didn't realize NextJS released a new stable version. Regardless, I'm sorry for misleading you. I will update my comment above. When I tested to see if my issue went away, I was currently on a branch that I am debugging a different issue I was facing, and I took out any NextUI components in that branch. So when I checked to see if I was still getting your error, it wasn't there.
My apologies for unintentionally misleading you.

Which canary version did you use? NextJS has a latest stable release (2 days ago) as of now after the latest canary (3 days ago) Check here and I am using this stable release only and still got the error.

I didn't realize NextJS released a new stable version. Regardless, I'm sorry for misleading you. I will update my comment above. When I tested to see if my issue went away, I was currently on a branch that I am debugging a different issue I was facing, and I took out any NextUI components in that branch. So when I checked to see if I was still getting your error, it wasn't there.
My apologies for unintentionally misleading you.

It's okay. There is a definite bug and I'm also getting errors for mismatch in classnames on server and client for some of the components.

I am also running into this issue

I am running into the same issue for Navbar Item

Hi! I have the same issue to unfortunately

Same

I am running into the same issue for Radio

"@nextui-org/react": "1.0.0-beta.12",
I have the same issue.

Same issue for here.

I have same issue in Table Header

The same

Moreover, this problem has occurred on every component of <li>

You can lazy load the Input component by using the dynamic import provided by the nextjs
import dynamic from 'next/dynamic';
const Input = dynamic(() => import('@nextui-org/react/input'), { ssr: false });
This one has worked for me.

Curious to know if this warning has been fixed. Warning: Prop id did not match. Server: "react-aria8623762345-8-name" Client: "react-aria3264912956-1-name"
Dynamic Component works for me as well; however, it does load like its glitchy

Related: https://github.com/lyleshaw/TweetAnalyzer/pull/5

import { useId } from 'react'

<Input id={useId()} />

Same
@nextui-org/react": "^1.0.0-beta.13, next": "13.4.10, react": "18.2.0

I upgraded to v2.0.1, but this problem is occured. Should I open new Issue?

My issue has related with hydration of next-theme library. It has resolved. Thanks you.


string library

[ ] length
[ ] trim(s)
[ ] pad(s)
[ ] number to string formatting
[ ] interpolation

For interpolation be reasonable, we almost always want as is and interp {}, so make that default for all "", be literal, if there are newlines take them, if there's a { that isn't followed by a { then that's a meant to be a replacement.

Interp is in and standard, we'll add these as we find them required rather than ahead of time


ace: use actual GOOS and GOARCH when building validator ACIs

Currently, os of built validator ACI is hardcoded to linux, which makes ugly patches necessary to make it run properly on Jetpack. This patch makes the build script use current GOOS and GOARCH values for the os and arch tags.

Great, thanks, makes sense to me. (Btw, know I owe you an email...)


RR object_id changes on execute_remote fetch or save, AKA duplicate objects

I believe the idea is that all RR records are kept track of by their Opal object_id.
r = Record.new
r.object_id #=> 36748

r.save
# { saved_models: [[36750, "Record", ...]] }

On save, it seems to add 2 to the object_id, aka it's instantiating an extra object in the save process rather than reusing the one you the programmer created.
On fetch it does the opposite which is doubly weird, the ID of the record being rendered in my code is 2 higher than the ID of the record being fetched. Like every RR instantiation actually instantiates 2 objects, then hyper-mesh uses the wrong one remotely.
Maybe there are 2 datastructures, one for rendering and one for hyper-mesh that aren't sharing objects.
This seems inconsequential, but it's causing me many rendering issues. I have an association rendering a list of components. There's a button to add a new one. I'm using key as recommended to ensure order, doing key: record.object_id. Adding a record and saving leads to a blank item in the list, or the item disappearing, or sometimes I even get a duplicate saved. I assume due to the above object_id changing and messing up key or there being duplicate objects in the RR system.

@sfcgeorge is this closed actually?

@sfcgeorge is this still an issue?  otherwise I'm closing


Changing days to cache, or amount to cache, doesnt clear old builds

As above, if I updated a project I expect it to then clear out old builds.
Currently I have to manually clear old builds when disk space becomes an issue.
Cheers

@azz0r there are two mechanisms currently

before creating new build every time the count is checked if it exceeds than old one is removed
old test variations are cleared every midnight

could you double check and specify what exactly is not working with steps?

Hey thanks for the response, it works as you mentioned, had thought maybe updating the cache amount would trigger.
Thanks


[WIP] Add NoEmptyStatementFixer

.

TODO; fix HHVM :(

And 5.3 ;)

ref https://github.com/FriendsOfPHP/PHP-CS-Fixer/blob/52ca3f008fffb402400a3db9cad90459c9ac098d/Symfony/CS/Tests/Tokenizer/TokensTest.php#L852

fabbot is wrong on this ;)

@SpacePossum what is the state of this PR ?

waiting for clearTokenAndMergeSurroundingWhitespace method in https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1705  and this one can be cleaned upped (it is as good as finished)

last TODO is check priority with no_useless_return

Could anyone make a final review please?

Please add this test case:
interface TestInterface1 extends TestInterface2, TestInterface3 {
};

I think it will fail.

Nice spotted @gharlan !
Test added, expecting failures.

do you want to handle this?
class Foo {
    use A, B {
        B::smallTalk insteadof A;
        A::bigTalk insteadof B;
    }; // <- could be removed
}

love to, but I'm not sure if it is valid? I tried https://3v4l.org/VXVMg

I tried https://3v4l.org/VXVMg

Parse error, so it is inavlid.

TODO

fix anonymous class handling
fix priority related to CombineConsecutiveUnsetsFixer

is it done then?

Proposal for new priority mapping.

Current Priority
New priority
Fixer name

100

encoding

98 | | short_tag
50 | | indentation
50 | 55 | linefeed
30 | 40 | unneeded_control_parentheses
26 | 28 | duplicate_semicolon
25 | 28 | no_empty_statement
25 | | phpdoc_to_comment
24 | | combine_consecutive_unsets
21 | | class_definition
20 | | phpdoc_indent
16 | | phpdoc_types
15 | | phpdoc_scalar
10 | | phpdoc_no_empty_return
10 | | phpdoc_no_access
10 | | phpdoc_no_package
 5 | | phpdoc_order
 5 |  7 | no_empty_phpdoc
 1 | | double_arrow_multiline_whitespaces
 0 | | namespace_no_leading_whitespace
 0 | | phpdoc_inline_tag
 0 | | phpdoc_separation
 0 | | short_scalar_cast
 0 | | ternary_spaces
 0 | | no_blank_lines_after_class_opening
 0 | | self_accessor
 0 | | hash_to_slash_comment
 0 | | trim_array_spaces
 0 | | no_empty_lines_after_phpdocs
 0 | | single_quote
 0 | | operators_spaces
 0 | | unalign_double_arrow
 0 | | print_to_echo
 0 | | no_blank_lines_before_namespace
 0 | | header_comment
 0 | | short_array_syntax
 0 | | ereg_to_preg
 0 | | phpdoc_var_to_type
 0 | | newline_after_open_tag
 0 | | long_array_syntax
 0 | | php_unit_strict
 0 | | php4_constructor
 0 | | strict
 0 | | new_with_braces
 0 | | unalign_equals
 0 | | unary_operators_spaces
 0 | | spaces_before_semicolon
 0 | | spaces_after_semicolon
 0 | | short_echo_tag
 0 | | strict_param
 0 | | multiline_spaces_before_semicolon
 0 | | function_typehint_space
 0 | | concat_without_spaces
 0 | | lowercase_keywords
 0 | | switch_case_semicolon_to_colon
 0 | | function_call_space
 0 | | method_argument_space
 0 | | list_commas
 0 | | array_element_no_space_before_comma
 0 | | lowercase_cast
 0 | | method_argument_default_value
 0 | | object_operator
 0 | | pre_increment
 0 | | switch_case_space
 0 | | php_closing_tag
 0 | | parenthesis
 0 | | visibility
 0 | | multiple_use
 0 | | function_declaration
 0 | | single_line_after_imports
 0 | | elseif
 0 | | lowercase_constants
 0 | | trailing_spaces
 0 | | no_trailing_whitespace_in_comment
 0 | | single_array_no_trailing_comma
 0 | | include
 0 | | blankline_after_open_tag
 0 | | phpdoc_short_description
 0 | | native_function_casing
 0 | | single_blank_line_before_namespace
 0 | | phpdoc_var_without_name
 0 | | heredoc_to_nowdoc
 0 | | standardize_not_equal
 0 | | array_element_white_space_after_comma
 0 | | multiline_array_trailing_comma
 0 | | join_function
-5 | | remove_lines_between_uses
-5 | | phpdoc_trim
-9 | | phpdoc_type_to_var
-9 | | short_bool_cast

-10 | -4 | unused_use
-10 | | php_unit_construct
-10 | | align_equals
-10 | | logical_not_operators_with_spaces
-10 | | spaces_cast
-10 | | concat_with_spaces
-10 | | align_double_arrow
-10 | | phpdoc_single_line_var_spacing
-10 | | psr0
-10 | | logical_not_operators_with_successor_space
-10 | | echo_to_print
-11 | | phpdoc_params
-17 | | empty_return
-18 | | no_useless_return
-19 | | whitespacy_lines
-19 | -20 | return
-20 | | line_after_namespace
-20 | | remove_leading_slash_use
-20 | | extra_empty_lines
-25 | | braces
-30 | | ordered_use
-50 | | eof_ending

Please keep it in separated thread, this is not connected to the feature that this PR adds.

what is the state of this PR?

one more item (https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/1671#discussion_r55608895), I hope/think that is it

reworked to make the code more simple following the idea of @gharlan

:+1:

ping @keradus let me know if there is something todo on this one :)

:+1: except 2 minors

:+1:  done :)

:+1:

Thank you @SpacePossum.


add emit option for asset modules

fixes #12474

What kind of change does this PR introduce?
feature

Did you add tests for your changes?
yes

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?

the parser options for asset modules have a new emit option. Defaults to true. Setting it to false will opt-out from writing assets from asset modules.

How about the 'new URL()' syntax for assets? Can I apply this option as well?


Feature/107 Unsubscribe from emails

Added ability for users to unsubscribe from emails (#107).

Added Email Subscription section to settings:

After clicking "Unsubscribe" button, show the /account/settings/change_email_subscription page

Clicking the button shows the success message

For unsubscribed users the settings section looks like this

Added Unsubscribe link to the welcome email, which points to {WebsiteBaseUrl}/account/settings/change_email_subscription:

text:

Welcome to SaaS Starter!
This is a quick sample of a welcome email. You can customize this email to fit your needs.
To unsubscribe, visit: https://saasstarter.work/account/settings/change_email_subscription

Issue to be fixed: If user it not logged in they will not be redirected to /account/settings/change_email_subscription after login, this could be implemented later?

Summary by CodeRabbit

New Features

Added functionality for users to manage their email subscription preferences directly from their account settings.
Introduced an unsubscribe link in welcome emails to allow users easy access to update their subscription status.

Bug Fixes

Enhanced error handling when toggling email subscription status to ensure smoother user experience.

Documentation

Updated README to provide clearer setup instructions for new and existing Supabase projects.

Tests

Added comprehensive unit tests for the email subscription management functionality to ensure reliability and correctness.

Thanks! I’m a bit slammed but will take a look when I have some time!

Really fantastic PR. Thanks @evgenyneu.
Merging as is. Few nitpick things like wording, but those are optional and can come after. I might quickly do them now.


"Microsoft.MixedReality.Toolkit" has dependency on editor assemblies

Overview
The core "Microsoft.MixedReality.Toolkit" assembly has a dependency on
"Microsoft.MixedReality.Toolkit.Core.Utilities.Editor",
"Microsoft.MixedReality.Toolkit.Core.Definitions.Utilities.Editor",
It seems to use a number of utility classes from the editor assemblies, wrapped in #if UNITY_EDITOR. Apart from good design principle, this also prevents the editor assemblies in turn from referencing the "Microsoft.MixedReality.Toolkit" assembly, which should be possible.
Expected Behavior
Core should not have a reference on editor assemblies.
Actual Behavior
It has.
Steps to reproduce

Look at MixedRealityToolkit.asmdef
Try to add dependency on "Microsoft.MixedReality.Toolkit" in MixedRealityToolkit.Core.Utilities.Editor.asmdef

Unity Editor Version
Mixed Reality Toolkit Release Version
MRTK v2 RC1

The reason for this is so the editor scripts are compiled and ran first. Without this then some of the initial editor setup steps will fail.

Assigning to @lukastoenneMS for the time being.

No runtime assemblies should have editor assembly dependencies.

MRTK3 is being very careful to ensure proper package and assembly separation and dependency management. If you find an issue there, please file issue(s) pointing out the specific instances. Thanks!


No MPL file once the experiment is complete

Work on extracting data from file other than MPL for the start time of the experiment.

Added to Master


Target dnx451

Reverts 6bea51909239f5dd6c5b9f44b0575b9f7227a770 on DNX

:shipit:

Recalling. After chatting with @Eilon, we decided that adding the facades to user's project.json is an acceptable workaround.


docs: clarify bw_per_day

The docs are pretty light on this.
I assume this is bandwidth per day measured in the unit gigabytes?

Yes! Exactly. It's gigabytes per day. I really don't document SporeStack well at all. Would you want to see this in sporestack-python, the limited API documentation section, and/or anywhere else in particular?

@teran-mckinney
Are the docs on GitHub? I wouldn't mind making the edits myself really - makes it easier for you.

Unfortunately not, I'll probably add a section for /options and update those. This would be here: https://sporestack.com/api_documentation
I actually have a V2 API in the works that hopefully will be much better about this. I'll add a quick note there but not invest too much in the current setup unless requested.

A bit more of a meta question but I'm plan on writing a provisioning tool for kontena.io. Basically you'll be able to automatically spawn a Kontena node with SporeStack as the provider.
Should I make an issue about it here - or is there a way I can document this so SporeStack users know this?
Also the refence code for the implementation is linked below for those interested, it will be at least another 3 months before I get to this task but it's actually quite easy. SporeStack supports passing a cloudinit - just like the provider below. Adding SporeStack should be a piece of cake.
https://github.com/kontena/kontena/pull/748/files
Is there perhaps a test environment for sporestack?  I'm pretty sure all I need to do is pass the original cloudinit file as provided by that PR, but if there's another way, I'd rather not purchase a new server for a day each time I fail to deploy.

Thanks for addings it to the docs!

Wow, that is really neat. Hmmm...
So the best setup for this would probably be a launch profile: https://github.com/sporestack/node-profiles
Would it fit that design fairly well? I don't think those even support cloudinits, though. I haven't used cloudinit.
Does Kontena have any sort of existing iPXE configuration? The new API will focus primarily on iPXE. And given your 3 months out figure, the V2 API may be the defacto. If you have notable interest in this, I can see about including cloudinit in V2 but a lot of V2s design is in being able to host on my own baremetal, of which I only support iPXE.
I also need to update the node-profiles to V2. V2 isn't even tested yet heavily so still a little ways out.
There's no test environment as of yet but I can help you with developing for use on the platform. I can give you a development token for trying it out. Just send me an email for that closer to the date.
PS: I updated the API docs: https://sporestack.com/api_documentation

@teran-mckinney
Actually - I was thinking of implementing SporeStack into kontena rather than implementing kontena into SporeStack. Kontena has a pretty broad userbase, I think having SporeStack as a plugin there will be more beneficial for your service. As far as I know you'll be the only Bitcoin accepting server provider there.
The current API has a cloudinit field - which I intended to use.
I have an interest in using SporeStack for all my servers - but I want (dare I say, need) a well integrated cluster manager for my servers.
Here's a plugin example library for AWS, the juicy bits are in this directory:
https://github.com/kontena/kontena-plugin-aws/tree/master/lib/kontena/machine/aws
Kontena solely relies on cloudinit for initializing itself, I would like to keep that for maintainability.
A workaround using CoreOS / RancherOS should allow me to harmonize iPXE and cloudinit.
Once I get to implementing it I will send you an email.

#!ipxe
# Boot a persistent RancherOS to RAM

# Location of Kernel/Initrd images
set base-url http://releases.rancher.com/os/latest

kernel ${base-url}/vmlinuz rancher.state.dev=LABEL=RANCHER_STATE rancher.state.autoformat=[/dev/sda] rancher.cloud_init.datasources=[url:http://example.com/cloud-config]
initrd ${base-url}/initrd
boot

PXE is not an issue.

Sounds great, thank you! Definitely curious to talk more about this over email when you have the time.


[Self-attack vulnerabilities] possibilities/list

The bugs below work only if an admin is logged and is tricked into pasting JavaScript code or uploading SVG's
WonderCMS comes with some security features and some responsibilities.
1. A logged-in user (admin) can execute JavaScript anywhere on their website.

This has always been a WonderCMS feature.
I personally don't consider this needs fixing, since a logged-in admin can do much more damage than just XSS attacks (including website defacement, malware distribution, cryptominers, ...)

2. A logged in user can upload a SVG (containing code other than an image, such as JavaScript).

SVG's are generally not just images, they can also include code such as JavaScript, XML, these are awesome  features of SVG's.

If there are enough wishes for this action, the SVG uploading functionality can be completely removed from WonderCMS.
If we already allow JavaScript to be executed at any part of the CMS, would removing the SVG functionality make any difference?

3. Host header attack.

This will not be considered a vulnerability until we see a live exploit of this (not local).
Using the Burp Suite Tool to create/show a local attack is not enough, since there needs to be a way to exploit a WonderCMS installation (and not just locally attack one-self).

How to prevent self-attack vulnerabilities

Avoid pasting random JavaScript code.
Avoid uploading random SVG's.
Install themes and plugins only from wondercms.com

The list above is subject to change. All discussions are welcome.
Reporting the above issues/bugs/vulnerabilities will not include you in the WonderCMS reward system.

I'm trying to think of a reason to counter myself and ask should we remove SVG uploading functionality and why?

even if we remove SVG uploading, users can still INLINE SVG's with HTML, which is a good thing
this way we can prevent users being "tricked" into uploading malicious SVG's onto their site, since they can't see the SVG content (if someone tricks them).

Please share your opinion on this topic.
TLDR: Remove SVG uploading or not? (even if we remove it, users can inline SVG's)

please note we would be still supporting executing JavaScript in any part of WonderCMS as we already do.

Added additional warning for uploading SVG's on the WonderCMS download page.
Here's the safety tips:
5 safety tips to avoid being hacked

Change the default password and default login URL immediately.
Install themes and plugins only from this website.
Regularly update WonderCMS. Always create backups.
WonderCMS supports running JavaScript anywhere. Be careful not to paste random/malicious JavaScript code.
WonderCMS supports uploading SVG's which could also contain JavaScript code. Don't get tricked into uploading malicious SVG's. If in doubt, avoid uploading any SVG file extension.

Change the default password and default login URL immediately.

Better: generate a random password from start so in the (likely) event that the user doesn't change the password, it's different on all installs.

@NicolasCARPi since WonderCMS doesn't pick up the users email, we would still have to display the password on the users homepage.
This would however prevent any automating attacks with the default password.

Closing this thread, create a seperate wiki page: https://github.com/robiso/wondercms/wiki/[Self-attack-vulnerabilities]-possibilities-list


LogContext.PushProperties not working with this sink?

This is a similar question to what was asked from before but I'm trying to get Serilogs LogContext.PushProperties to work when this sink is setup. Basically I'm setting up a collection of Enrichers with values that I want to track (these values come into the service from the controller and I want every log entry down the stack to have them)
        using (LogContext.PushProperties(enrichers.ToArray()))
        {
            try
            {
               // Stuff happens here
            }
         }

None of these properties show up in AI (everything previously worked in Seq). The examples given for this in a previous post seemed to modify the TelemetryClients context on startup. I need this to work per call. Any examples that you guys know of that can get me pointed in the right direction?
Thanks

Hi Travis,
Do you have Enrich.FromLogContext() set for the new configuration?
Cheers,
Nick

Has this been resolved? We're having the same issue.


Add Eurolite-LED-Flood-Light-252-RGB-19

I could not find this fixture so I created it with the fixture creator.

Hi, thanks for your submission. This fixture looks a lot (channel mapping wise) like the existing "Eurolite-LED-PAR64-RGB.qxf". Massimo, do we have a policy about fixtures from the same manufacturer, different model name but same behavior? Can we define aliases?

@kripton  unfortunately we don't.
I think there are already some kind of "duplicates" in the library.
The alias idea is not bad at all actually

Thanks @karrika :+1:


v3: get rid of org.json and use Gson for performance

https://github.com/cookpad/puree-android/issues/31

[x] get rid of org.json
[x] benchmark with v2 - about 2 times faster in average

I think it is good enough

[x] documentation

benchmark
Note it is not fair enough because Puree.flush() invokes asynchronous tasks. That is because the first try is so fast.
v2

v3

Good improvement :+1:

Thanks!

@rejasupotaro
One more thing: changed class JsonConvertible into interface PureeLog to make things clear.

@gfx OK, I noted.

It's time to merge!


build: improve performance

Summary

replaces tailwindcss with windicss
replaces microbundle(babel) with tsup(esbuild)
fixes eslint and prettier linting

On my workstation, build performance increased by 60%, dropping from roughly 53 seconds to 17 seconds.

Thanks for this, really appreciate the tooling improvements 🚀


Extension to specify simple styling/processing expressions

We have been experimenting with how to style coverages in CMSS, e.g. being able to select which bands to use and how to use them and potentially combine their values (e.g. allowing to compute vegetation index), and apply colormaps, etc. It occurred to me that part of this could also be used as part of a simple coverage request to define bands to return with the additional capabilities of very simply defining arithmetic to perform to compute the values, e.g. something like:
http://maps.ecere.com/ogcapi/coverage/s2?bands=evi(2.5*(B08/4000.0-B04/4000.0)/(1+B08/4000.0+6*B04/4000.0+-7.5*B02/4000.0))
or
http://maps.ecere.com/ogcapi/coverage/s2?bands=r(B04/4000),g(B03/4000),b(B02/4000)
Below is the syntax we are considering in CMSS (mostly intended for either client-side rendering, or server-side e.g. for Maps or Maps/Tiles API).
#Sentinel_2_L2A
{
   // For normal sentinel-2 L2A imagery (if no band named r, g, or b: single band (other than 'a') is an index; multi-band is assigned to r,g,b in order)
   bands =
   {
      a = (B02 != null | B03 != null | B04 != null) ? 1.0 : 0.0,
      r = B04 / 4000.0,
      g = B03 / 4000.0,
      b = B02 / 4000.0
   };

   // For EVI:
   /*
   bands =
   {
      a = (B02 != null | B03 != null | B04 != null) ? 1.0 : 0.0,
      evi = 2.5 * (B08 / 4000.0 - B04 / 4000.0) / (1 + B08 / 4000.0 + 6*B04 / 4000.0 + -7.5*B02 / 4000.0)
   };
   */

   // Generating a multi-band coverage (processing use case)
   /*
   bands =
   {
      b1 = B04 / 4000.0,
      b2 = B03 / 4000.0,
      b3 = B02 / 4000.0,
      evi = 2.5 * (B08 / 4000.0 - B04 / 4000.0) / (1 + B08 / 4000.0 + 6*B04 / 4000.0 + -7.5*B02 / 4000.0)
   };
   */

   // For a tint to apply to the monochrome index, or the default single band value:
   //color = lime;

   // For a color map applied to the index (e.g. EVI) instead of grayscale monochrome:
   /*
   colorMap = [
      { 0.0, 0x533300 },
      { 0.5, sienna },
      { 0.6, darkGoldenrod },
      { 0.7, oliveDrab },
      { 0.8, yellowGreen },
      { 0.9, greenYellow },
      { 1.0, lime }
   ];
   */
}

CQL2 with extensions might also be considered for expressing this, and possibly some extension could allow to define aggregation capabilities.
Coverage access/processing capabilities:

Downsampling resolution
Subsetting (Trimming and (reducing) Slicing)
Select only some values from the range (range subsetting)

Additional capabilities which could expressed e.g. with CQL:

Band arithmetics (and defining new bands) -- this original issue
Filtering (by range values, by geometry) -- i.e. filter
Reduction (e.g. summation, average, minimum, maximum resulting in one less dimension)
Aggregation ( group result ) downsampling to lower-resolution, e.g. daily averages

See also OpenEO description https://openeo.org/documentation/1.0/datacubes.html


Fix Samsung first character selection

Problem reported here: https://github.com/braintree/braintree-web/issues/245
Samsung keyboards when inputting to text input fields reports an incorrect range selection on first entry: {start: 0, end: 1}. This value gets passed through restricted input, and maintained during formatting. The result is the selection highlighting the first character of entry.
This proposal is to always reset the selection range after formatting to the formatted selection end.
Tested this on:

Samsung Keyboard, Galaxy S4 (SPH-L720T) running Android 5.0.1, Chrome 55.0.28883.91
Gboard (Google Keyboard), Nexus 6P running Android 7.1.1, Chrome 55.0.28883.91

Tested this on a device that was having problems with the previous bug. Still works great. 👍

The integration tests were flaky. Kicked them off again and they actually seem to be running now.


Add minimal replacements for default 400/500 pages

This grabs the image from https://github.com/codeforamerica/somerville-teacher-tool/issues/279 and uses it to make minimal 400 and 500 pages, replacing the default Rails ones.
It might be a good improvement to update the branding to match the rest of the product (not sure it will be obvious to users why there's a planet), but the intent here is just to close the work-in-progress from https://github.com/codeforamerica/somerville-teacher-tool/issues/279.
404:

500:

Live in production: https://somerville-teacher-tool.herokuapp.com/takemetoyourleader


Add unique error messages for different "Invalid Signature" cases.

Hi,
Would it be possible to improve the "Invalid signature" errors ? In real life, it seems we have this error in many case, and it's a pain to debug. More so with the fact that it's often the end user that see this error, and not me (since i use passport-saml in my code, but it's the end user that does the request when he connects to my app through saml).
One who be really easy to change, it's the one when there is multiple assertions.
And for the others, it would be nice to be just a bit more precise, so we know in which usecase we are (this should be quick). And also to display e.g. "signature 'aPoeK...' not compatible with certificate 'vVm8...'", so user can verify that he didn't sign with the wrong certif or whatever (but this would be longer to do).

Agreed. This has made SAML integrations incredibly painful for me.
Is this error intentionally opaque? I understand that you do not want to reveal such low level information to end users, but application developers should be masking all errors with a generic error page.

saml.js contains 6 unique cases which all result in the same Invalid Signature diagnostic message.
Would it simply work to extend the strings to be more unique: Invalid Signature: More than one assertion. Just spell out some more detail in each case. Unless there's a reason that it /needs/ to have the diagnostic in each case, this seems like the simplest solution.

Thanks for fixing this!

@markstos Thank's a lot, this is useful !


[Metamask] TransportError: Failed to sign with Ledger device: U2F DEVICE_INELIGIBLE

Trying to sign a transaction interacting with a smart contract, I get this error:
"message": "Error: TransportError: Failed to sign with Ledger device: U2F DEVICE_INELIGIBLE",
"stack": "Error: TransportError: Failed to sign with Ledger device: U2F DEVICE_INELIGIBLE\n
at chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/background.js:1:1227754\n
at chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/background.js:1:1229201\n
at n (chrome-extension://nkbihfbeogaeaoehlefnkodbefgpgknn/background.js:1:205012)"

Hello, what was the solution to this?

Which browser/browser version/OS are you using ?


unknown error when group by has couldn't push down predicate

Steps to reproduce the behavior (Required)
Table supplier was SSB DataSet
select sum(S_NATIONKEY) a, sum(S_ACCTBAL) as b, S_ADDRESS as c from supplier group by S_ADDRESS
having a < b*1.2 or c not like '%open%'"

Expected behavior (Required)
success
Real behavior (Required)
unknown error
StarRocks version (Required)

2.0.0-GA 2.0.1

Reopen it for other test only.


Error 99 returned when mraa_i2c_write(m_i2c, data, length); is called, but i2cdetect can see device.

I am using mraa on the Intel Edison with Arduino breakout board. I am using the library in its c++ distribution. I initialized an mraa::I2c object using the argument for bus 6. I compiled with g++ using the c++11 standard, and -lmraa argument. libmraa is installed on the Edison I am using. I would love to provide any other information you would need to help me. You can view my code at here, but specifically I am using mraa in the file named Wire.cpp. I am trying to make an Arduino library work for the Intel Edison. The library interfaces with an I2C device called Pozyx which does indoor localization and is visible at address 0x4B on bus 6. You can check out Pozyx here if you are interested, and their documentation on how to interface with it over I2C here. Using gdb, I was able to see that the code I wrote can step through Pozyx, my Wire.cpp, and mraa::i2c, but when mraa_i2c_write(m_i2c, data, length); is called on line 206 in i2c.hpp, code 99 is returned which means error unspecified. How can I learn how to fix this error, or what causes it? Using gdb I have determined that m_i2c is not null. Please tell me what more I can tell you to help me.
I am a college student doing this to get my bachelors degree. This is a very small part of my project, but it is holding my team up. We would very much like to be able to interface with Pozyx over I2C on the Intel Edison, and we hope that mraa is able to help us reach our goal. Thank you for the work you have done on mraa, and the help you will provide to us.

Where in your code are you getting stuck in the wire.cpp? I've only had a quick glance and couldn't quite work out what you're doing. But maybe start with something super basic in python like this:
import mraa as m
bus = m.I2c(6)
bus.address(0x4b)
print (int) bus.readReg(0x00)

From a quick read of the datasheet this should return 0x43 - https://www.pozyx.io/Documentation/Datasheet/RegisterOverview. At least this way we know if your code or if mraa is not working right. If it's your code then I suggest you send a list of only the mraa calls you are making and their order. But starting with checking the device responds correctly with a basic register read is always my first recommendation.
You also don't mention the version of mraa you're using, I'd recommend using 1.x or newer on an edison board. 1.5.1 is obviously recommended ;-)
Good luck with your project!
@intel-iot-devkit/upm-core-dev have you seen this device before? Looks quite cool

In Wire.cpp it gets stuck in Wire::endTransmission(uint8_t), specifically line 28 where i2c->write(txBuffer, (int)txBufferLength); is called. Wire::endTransmission(uint8_t) is actually called by Pozyx::i2cWriteRead(uint8_t*, int, uint8_t*, int) in Pozyx_core.cpp on line 605. The program I am using to test the code is compiled from ready_to_localize.cpp, where the first thing it does is exactly what that python script you provided does (line 102).
After making sure I had mraa 1.5.1, I ran the python script you provided. Here was the output:
Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/usr/lib/python2.7/site-packages/mraa.py", line 1175, in readReg     return _mraa.I2c_readReg(self, reg) ValueError: Unknown error in I2c::readReg()
I assume the underlying error is code 99. Could it be because the way Pozyx wants I2C done? There documentation on their website is pretty thorough, but I do not know how mraa does its I2C.

Well that's good news really, we can reproduce in 4 lines :) This would indicate we're simply not seeing your i2c device.
I think actually this may due to mraa not respecting the arduino 'shield' behaviour. Try connecting your i2c lines on your shield to the i2c SCA/SCL lines directly. See if that works.

@pillager225, you mentioned that i2cdetect can see the board, so it's unlikely to be the voltage mismatch, but just to be sure - please check that IOREF jumper is set to the 5V position and not the 3.3V one.

Well, we are in the electronics land with this issue, so I guess yes, 0.4 volts may make a difference here (especially if there are further dropouts when the setup tries to communicate)

So I wired the ioref pin to 5.1 volts. That 5.1 volts was relative to the same ground that the edison was being powered by. I still got error code 99.
I figure that the best thing I can do is install the mraa c library with the source included so that I can debug that portion of the code. I'll let you know of any progress I make in that area.
Also, I know Intel is very close to my school, Santa Clara University, so if there is somebody willing to come by the SCU Robotic Systems Lab to check things out, please let me know.

Sadly I don't think we have anyone in the teams in SC, development of mraa is in Europe, the UPM team is in Portland, US.
The only thing I can recommend is grabbing a logic analyser and having a look at what's happening on the SCL/SDA lines. If you don't have one, you can gab very cheap USB based ones online, but I'm sure your university will have something you can use, especially in a robotics lab!

I have yet to use a logic analyzer, but I will do that sometime this week most likely.
Do you guys know compile mraa so that when you debug a program using mraa using gdb you can step through the c code that is executed? I would like to chase the error down as far as I can in the mraa library to get more information as to why I am getting the error.

Look at docs/building.md. there's instructions for building a debug version
On 23 Jan 2017 21:59, "Pillager225"<EMAIL_ADDRESS>wrote:
I have yet to use a logic analyzer, but I will do that sometime this week
most likely.
Do you guys know compile mraa so that when you debug a program using mraa
using gdb you can step through the c code that is executed? I would like to
chase the error down as far as I can in the mraa library to get more
information as to why I am getting the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/intel-iot-devkit/mraa/issues/624#issuecomment-274614999,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfZZukuqGWOnXVRRkdtCUMNAyPMO3-Yks5rVRSwgaJpZM4LV74i
.

@Pillager225, is this still actual, how can we help you further on this if yes?

I don't know how you guys can help. I have been unable to find the time to use the logic analyzer at my school and because this is part of a project related to my schoolwork, I needed to move forward. I did get images of the waveforms caused by the edison when trying to talk to pozyx, and they matched the ones that come out of a arduino which is able to communicate with pozyx over i2c. The only difference is that pozyx does not acknowledge the edison. I cannot, at this time, invest more time into trying to get i2c to work. Thank you guys for your bits of advice and help when trying to solve this issue. It is likely that the problem has more to do with pozyx then mraa.


Add pyproject.toml for Custom Node Registry

We are building a global registry for custom nodes (similar to PyPI). The registry makes using custom nodes more reliable by reserving globally unique names for each custom node, and supporting semantic versioning.
Here’s some more information on the registry.
Action Required:

[x] Go to the registry. Login and create a publisher id. Comment it here and we will update the PR (or just merge it and change it yourself).
[x] Write a short the description.
[x] Merge the separate Github Actions PR and run the workflow.

If you want to publish the node manually, install the cli and run comfy node publish
Please message me on Discord if you have any questions!

Hey @Limbicnation, Saw that you merged the PR which is amazing!!!!! Just a few last steps to get you onboarded 🚀
Steps:

Go to the registry. [https://www.comfyregistry.org/]
Login and create a publisher id.
Update the toml file with your publisher ID, and publish your node either through the CLI or with GH Actions!

There's also like a 1minute video we made too teehee - https://youtu.be/WhOZZOgBggU
Let me know if there is anything else I can help with!!"

Thank you for your help!
I apologize for the delayed response. When I enter my API token to publish my node, I encounter the following error:
Please enter your API Key (can be created on https://registry.comfy.org):
Publishing node version...
{'Failed to publish node version: 400 {"message":"Failed to validate token"}\n'}

hey @Limbicnation ! Can you give me your publisher ID? You also have to enter your publisherID into the pyproject.toml file before trying to publish your node too! if you can try both of these things I can try to help debug on my end too!

hey @Limbicnation ! Can you give me your publisher ID? You also have to enter your publisherID into the pyproject.toml file before trying to publish your node too! if you can try both of these things I can try to help debug on my end too!

Hey @haohaocreates, thanks for the quick heads-up! I realized I had the wrong publisher ID in my pyproject.toml.
Just a quick question: Are you available to help with debugging custom nodes for ComfyUI?
I'm just getting started and find it a bit intimidating.


Remove additional  item from filter list

Hello and thank you for this amazing table library. :wave:
This PR removes the <li> from the list of filtering options. Not sure if you this is something intended from your side, but since we already have the Filter by in the input, we probably can get rid of adding it to the list of options.

At one point, this was needed in order for the placeholder to actually show in the textbox/select part. Does it still show a placeholder without this menu item?


id transfer problem

Dear soren121,
Thank You so much for this app! I love it.
I try to transfer my id to the public version of the app but get the notification "unable to decrypt id. Something's fishy here."
I think I did everything right, tried it different times today. I would be really sad if I loose my yak karma :( any tips or hints for me?
Thank you very much.

Fixed in 10168a5. I pushed a new update to the Store just now, version <IP_ADDRESS>. Hopefully it should fix your problem. The update should be available in the next few hours.
If you've updated and it's still broken, please reopen this issue.

Works now. Thank you so much. Incredibly fast response!


Day321

Resolves #651.

Pull Request Test Coverage Report for Build 1222

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

Totals

Change from base Build 1220:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Skip text proccessing by action button.

Hey, thanks for the PR! This is incorporated in the new beta now, see here.


Docker Image Permission Issues

Hello maintainers,
I've encountered a problem when trying to deploy the h2ogpt Docker image on a Kubernetes/OC cluster. The deployment fails due to a Permission denied error. I believe the root cause is the way the Docker image is set up.
Issue Details:

Python is installed and run from the /root directory. This is problematic as many Kubernetes environments restrict running containers as root for security reasons.
The Docker image does not seem to be set up in a manner that's conducive for professional deployments, especially in environments that have strict security policies.

Deployment Configuration:
We are using the following deployment.yaml:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: h2ogpt-deployment
  labels:
    app: h2ogpt
spec:
  replicas: 1
  selector:
    matchLabels:
      app: h2ogpt
  template:
    metadata:
      labels:
        app: h2ogpt
    spec:
      containers:
      - name: h2ogpt-container
        image: gcr.io/vorvan/h2oai/h2ogpt-runtime:latest
        ports:
        - containerPort: 7860
        env:
        - name: HOME
          value: /workspace/
        volumeMounts:
        - name: cache-volume
          mountPath: /workspace/.cache
        - name: save-volume
          mountPath: /workspace/save
        command:
          - tail
          - '-f'
          - /dev/null
      volumes:
      - name: cache-volume
        persistentVolumeClaim:
          claimName: cache-pvc
      - name: save-volume
        persistentVolumeClaim:
          claimName: save-pvc

Temporary Solution:
As a quick fix, we built an additional layer on top of the existing image:
FROM gcr.io/vorvan/h2oai/h2ogpt-runtime:latest

RUN chgrp -R 0 /root && \
    chmod -R g=u /root

RUN chgrp -R 0 /workspace && \
    chmod -R g=u /workspace

However, this is not a sustainable long-term solution.
Request:
I kindly request that the Docker image be restructured to avoid such permission issues, especially when deploying in environments like Kubernetes. Ideally, the image should not rely on the /root directory for operations and should be set up in a way that respects common security practices.
Thank you for your attention to this matter.

Thanks, we will take a look at it ASAP

Please take a look @achraf-mer and @ChathurindaRanasinghe

Over at https://github.com/h2oai/h2o-llmstudio/blob/main/Dockerfile they're using an "llmstudio" user in the container. Perhaps it would help to do the same here?

I don't think such a user would help, this is from llmstudio container:

The user is llmstudio:
llmstudio@h2o-llmstudio1:/workspace$ whoami
llmstudio

Directory permissions under /workspace:
llmstudio@h2o-llmstudio1:/workspace$ ls -la
total 328
drwxr-xr-x 1 root root    598 Aug  2 22:31 .
drwxr-xr-x 1 root root    240 Aug 12 05:41 ..
...
[Trimmed for brevity]
...
-rw-r--r-- 1 root root   5576 Aug  2 22:23 train_wave.py

However user + changing dir + changing owner could work. Something like this:
# ... (existing Dockerfile content up to the point of Miniconda installation)

# Create a new user 'llmstudio'
RUN useradd -m llmstudio

# Install Miniconda into /usr/bin/miniconda3
RUN wget https://repo.anaconda.com/miniconda/Miniconda3-py310_23.1.0-1-Linux-x86_64.sh && \
    bash ./Miniconda3-py310_23.1.0-1-Linux-x86_64.sh -b -p /usr/bin/miniconda3

# Update PATH environment variable
ENV PATH="/usr/bin/miniconda3/bin:${PATH}"

# Change the ownership of the directories to 'llmstudio'
RUN chown -R llmstudio:llmstudio /workspace && \
    chown -R llmstudio:llmstudio /usr/bin/miniconda3

# Set the working directory to /workspace
WORKDIR /workspace

# Switch to 'llmstudio'
USER llmstudio

# ... (rest of your Dockerfile content)

By making these changes, you ensure that the container runs as the user llmstudio and that this user has the necessary permissions to execute the application and its dependencies from the /workspace directory.

Nice. I'd recommend setting chmod a+rwX so that we avoid permissions errors when the docker container is run as a existing host user, too!

@ffalkenberg Can you share your manifest files on getting this to work on k8s?

Hello @lamw,
Certainly! Below is an example of the Kubernetes manifest I used to deploy the h2ogpt :
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: cache-pvc
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 30Gi
---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: save-pvc
spec:
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 10Gi
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: h2ogpt-deployment
  labels:
    app: h2ogpt
spec:
  replicas: 1
  selector:
    matchLabels:
      app: h2ogpt
  template:
    metadata:
      labels:
        app: h2ogpt
    spec:
      containers:
      - name: h2ogpt-container
        image: gcr.io/vorvan/h2oai/h2ogpt-runtime:latest
        ports:
        - containerPort: 7860
        env:
        # proxy address, adjust or remove as necessary
        - name: HTTPS_PROXY
          value: http://your-proxy-address:8080
        - name: HOME
          value: /workspace/
        volumeMounts:
        - name: cache-volume
          mountPath: /workspace/.cache
        - name: save-volume
          mountPath: /workspace/save
        # This command is used to keep the pod running. Replace with h2ogpt command when needed.
        command:
          - tail
          - '-f'
          - /dev/null
        resources:
          limits:
            cpu: '4'
            memory: 25G
            # The NVIDIA MIG configuration specifies how GPU resources should be allocated
            # This depends on the Kubernetes cluster environment and can be skipped if not relevant.
            nvidia.com/mig-7g.40gb: '1'
          requests:
            cpu: '4'
            memory: 25G
            nvidia.com/mig-7g.40gb: '1'
      volumes:
      - name: cache-volume
        persistentVolumeClaim:
          claimName: cache-pvc
      - name: save-volume
        persistentVolumeClaim:
          claimName: save-pvc
---
kind: Service
apiVersion: v1
metadata:
  name: h2ogpt-service
spec:
  ports:
    - protocol: TCP
      port: 80
      targetPort: 7860
  selector:
    app: h2ogpt
---
apiVersion: route.openshift.io/v1
kind: Route
metadata:
  annotations:
    haproxy.router.openshift.io/balance: roundrobin
    haproxy.router.openshift.io/disable_cookies: 'true'
  labels:
    app: h2ogpt-route
  name: h2ogpt-route
spec:
  to:
    kind: Service
    name: h2ogpt-service
    weight: 100
  port:
    targetPort: 7860
  wildcardPolicy: None

You might need to adjust it based on your specific requirements and environment.
Additionally, there's now a Helm chart available as part of the repository. It can greatly simplify the deployment process on Kubernetes. I'd highly recommend checking it out if you haven't already.
I hope this helps! Let me know if you have any further questions or need additional details.

Thank you @ffalkenberg - I was almost close in my YAML, but used args rather than command
I'm able to get basics up and running but what is the strategy for pre-downloading the models and simply referencing them, same goes with generated DB files, which I've done manually and wanted to include them. Since there's no k8s examples, Its been a bit challenging to figure out the right arguments/flags ...

Hello @lamw,
The nature of Persistent Volume Claims (PVCs) in Kubernetes guarantees that once the models and DB files are downloaded, they will persist and survive pod restarts and evictions. This means that once the files are on the PVC, they won't be downloaded again in subsequent deployments or pod restarts.
However, if you want to avoid downloading the models and DB files entirely within the Kubernetes environment, you can use an initContainer. The initContainer can be set up to copy the files from a predefined location (e.g., a mounted storage or another container) to the PVC before the main application starts.
Here's a basic approach:

Create the PVC: This PVC will store your models and DB files.
Deploy with initContainer:

The initContainer accesses the PVC.
It copies the models and DB files from a predefined location to the PVC.

Run the Main Application: With the data already on the PVC, the main container(s) start without needing to fetch or download the models and DB files.

This approach ensures that your models and DB files are made available to your application without any download operations within the Kubernetes environment.
To determine exactly where the data gets stored, it would typically be in directories like /workspace/.cache or /workspace/save, depending on your application's configuration.

OK, this was extremely helpful and I've been able to copy my files which I'm putting into separate PVC (not sure if this is actually needed). I'm almost there but getting a permission denied when it attempts to upload new data ...
I've got a mount point /workspace/userdata which contains the generated DB (MyDdata) + source files in mydata directories:
k exec wget-init-pod -- ls -la /workspace/userdata
Defaulted container "main-container" out of: main-container, userdata-init-container (init)
total 6993420
drwxrwxrwx   12 root     root          4096 Sep 29 14:46 .
drwxr-xr-x    3 root     root          4096 Sep 29 14:46 ..
drwxr-xr-x    3 root     root          4096 Sep 29 14:46 db_dir_MyData
drwxr-xr-x    2 root     root        155648 Sep 29 14:52 mydata

I'm then using the following arguments ... but 100% sure this is correct:
- --user_path=/workspace/userdata
- --langchain_mode=MyData
- --langchain_mode_paths="{'MyData':'/workspace/userdata/mydata'}"

When I try to add a few file, I see following in the console:
Traceback (most recent call last):
  File "/h2ogpt_conda/lib/python3.10/shutil.py", line 816, in move
    os.rename(src, real_dst)
OSError: [Errno 18] Invalid cross-device link: '/tmp/gradio/089a37315889eb217144e299cdd51c0690eb151d/test.html' -> '/workspace/userdata/mydata/test.html'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/workspace/src/gpt_langchain.py", line 4796, in update_user_db
    return _update_user_db(file, db1s=db1s,
  File "/workspace/src/gpt_langchain.py", line 4941, in _update_user_db
    shutil.move(fil, new_fil)
  File "/h2ogpt_conda/lib/python3.10/shutil.py", line 836, in move
    copy_function(src, real_dst)
  File "/h2ogpt_conda/lib/python3.10/shutil.py", line 434, in copy2
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "/h2ogpt_conda/lib/python3.10/shutil.py", line 256, in copyfile
    with open(dst, 'wb') as fdst:
PermissionError: [Errno 13] Permission denied: '/workspace/userdata/mydata/test.html'

I can manually exec and touch a file but seems the process isn't able to and not sure if there's some additional permissions or settings required

Hello @ffalkenberg
I am trying to run h2ogpt using openshift but getting this error "exec /workspace/generate.py: exec formate error". This error is because of architecture differences but I have built the image on amd64 and tried to run it on minikube based on amd64. I can run the image using docker but not with k8s or openshift. What might be causing this error?

The only arch specific binary wheels we have are llama_cpp_python, auto-awq, auto-gptq, and exllama.  If you are using llama_cpp_python, you can do this kind of thing and rebuild the docker image and it should work: https://github.com/h2oai/h2ogpt/issues/1440#issuecomment-1975173876
But instead of the cuda line you should use AMD one from here:
https://github.com/abetlen/llama-cpp-python?tab=readme-ov-file#supported-backends
CMAKE_ARGS="-DLLAMA_HIPBLAS=on" pip install llama-cpp-python


How to implement new protocol in your existing code

Hi ,
I have checked all your protocol and its really very helpful for china device but i have once device which is not suited with any one of your protocol. so i would like to add custom protocol for that .
So how can i do that ? or are you able to create protocol for it if i will provide you protocol documentation ?
Thanks

Please read official documentation:
https://www.traccar.org/implement-protocol/
If you need help with implementing it, send me an email to discuss options.

Ok thanks.  Please share your email id. so we can discuss it .

It's in the profile.

Ok. i have mailed you . please check it.

I don't see any emails from you.


Can't flash OLED with keymap

Linking: .build/helix_rev2_oled_default.elf                                                         [ERRORS]  |   | /usr/bin/avr-ld: .build/obj_helix_rev2_oled_default/keyboards/helix/rev2/rev2.o:/home/draven/qmk_firmware/keyboards/helix/local_drivers/ssd1306.h:69: multiple definition of display'; .build/obj_helix_rev2_oled_default/local_drivers/ssd1306.o:/home/draven/qmk_firmware/keyboards/helix/local_drivers/ssd1306.h:69: first defined here
| /usr/bin/avr-ld: .build/obj_helix_rev2_oled_default/keyboards/helix/rev2/keymaps/default/keymap.o:/home/draven/qmk_firmware/keyboards/helix/local_drivers/ssd1306.h:69: multiple definition of display'; .build/obj_helix_rev2_oled_default/local_drivers/ssd1306.o:/home/draven/qmk_firmware/keyboards/helix/local_drivers/ssd1306.h:69: first defined here  | collect2: error: ld returned 1 exit status  |  make[1]: *** [tmk_core/rules.mk:306: .build/helix_rev2_oled_default.elf] Error 1 Make finished with errors make: *** [Makefile:584: helix/rev2/oled:default:flash] Error 1

This is the error I get when trying to flash a keymap with oleds enabled

problem solved, had to downgrade gcc from 10.0.1 to 8.3


Autocomplete broken on a couple of packages (gg, ui, ...)

Describe the bug
Autocomplete is only partially working on gg and not at all on ui.
Expected Behavior
It would suggest methods, not only enums and structs.
Current Behavior
Only suggests structs and enums
Reproduction Steps
Install v and v-analyzer, create a new project and import gg and try completions on it
Possible Solution
No response
Additional Information/Context
No response
Environment details (v doctor output)
V full version: V 0.4.6 769e914.aa34047
OS: linux, Linux Mint 21.3
Processor: 8 cpus, 64bit, little endian, Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz
getwd: /home/sudokit
vexe: /home/sudokit/langs/v/v
vexe mtime: 2024-07-15 14:18:04
vroot: OK, value: /home/sudokit/langs/v
VMODULES: OK, value: /home/sudokit/.vmodules
VTMP: OK, value: /tmp/v_1000
Git version: git version 2.34.1
Git vroot status: weekly.2024.28-22-gaa340472
.git/config present: true
CC version: cc (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
thirdparty/tcc status: thirdparty-linux-amd64 a0799a5b
Editor name
vsocde and helix
v-analyzer Version
v-analyzer version 0.0.3-beta.1.a796f25
VS Code Extension Version
[0.0.1]

Hey
Thank you very much for taking the time to create this issue, I think you should try installing version from https://github.com/vlang/v-analyzer

Yeah it happened both in helix (using v-analyzer with the latest version) as well as vs code (ext installed from GitHub link) plus with zed (using v-analyzer as well). Cleared cache and tried again and still the same issue

i installed it from the link you provided, and now gg works. ui not so much...
better than nothing :D

haha trueee .p


[question] How to keep the boilerplate updated, but change the app files?

I really liked your project, but the problem is that if i forked it, or create a new project with the current version of the repository i will lose any updates of the rest of the files.
There is a way to keep this updated even if i change all the files inside app?
Thanks
Claudio

If your application is a fork from this repo, you can add this repo to another git remote:
git remote add upstream https://github.com/chentsulin/electron-react-boilerplate.git

Then, use git to merge some latest commits:
git pull upstream master

Thanks men

wow. you should add this in the readme


Support tmux-weather plugin.

Source
Supporting that plugin would be great.

Here is a screen shot of me doing just that. I replaced the date (which I do not need) with the weather:

All working but for the e at the end of the status. No idea how to get ride of it…

Hey @kierun I have a very similar solution without problem with the 'e' at the end
Also I -ga option to split the lines. It helps a lot when you playing with the parameters.
#+--- Bars ---+
set -g status-left "#[fg=black,bg=blue,bold] #S "
set -ga status-left '#[fg=black,bg=blue]#{?window_zoomed_flag, Z ,}'
set -ga status-left "#[fg=blue,bg=black,nobold,noitalics,nounderscore]"

set -g status-right  "#{prefix_highlight}"
set -ga status-right "#[fg=brightblack,bg=black,nobold,noitalics,nounderscore]"
set -ga status-right "#[fg=white,bg=brightblack] %a %b %d "
set -ga status-right "#[fg=white,bg=brightblack,nobold,noitalics,nounderscore]"
set -ga status-right "#[fg=white,bg=brightblack] #{weather}"
set -ga status-right "#[fg=cyan,bg=brightblack,nobold,noitalics,nounderscore]"
set -ga status-right "#[fg=black,bg=cyan,bold] %H:%M "

#+--- Windows ---+
set -g window-status-format  "#[fg=black,bg=brightblack,nobold,noitalics,nounderscore] "
set -ga window-status-format "#[fg=white,bg=brightblack]#I "
set -ga window-status-format "#[fg=white,bg=brightblack,nobold,noitalics,nounderscore] "
set -ga window-status-format "#[fg=white,bg=brightblack]#W #F "
set -ga window-status-format "#[fg=brightblack,bg=black,nobold,noitalics,nounderscore]"

set -g window-status-current-format  "#[fg=black,bg=cyan,nobold,noitalics,nounderscore] "
set -ga window-status-current-format "#[fg=black,bg=cyan]#I "
set -ga window-status-current-format "#[fg=black,bg=cyan,nobold,noitalics,nounderscore] "
set -ga window-status-current-format "#[fg=black,bg=cyan]#W #F "
set -ga window-status-current-format "#[fg=cyan,bg=black,nobold,noitalics,nounderscore]"

I hope it will help

Hey @kierun I have a very similar solution without problem with the 'e' at the end

Thank you! I removed all my lines, added yours, and the "e" vanished. Sadly, it was replaced by "n"!
Running tmux 3.1, in case that's an issue…

Also I used -ga option to split the lines. It helps a lot when you playing with the parameters.

Now, that is a superb option! Thank you.

Itrestingly enough, messing around some options, I got this:

Colour me confused…

Running tmux 3.1, in case that's an issue…

@kierun I'm running tmux 3.1 on macos catalina 10.15.4. I don't have the issue.
However I'm using a self-made weather plugin. Out of curiosity I switched to plugin by xamut and I immediately noticed some problems with the alignment.
self-made:
by xamut:

@nikita-y Yes, there is some weird alignment going on. When I maximise then minimise the window, I get a 1/2 character gap which is filled with black where the extra character was. Trying xamut's minus Nord does not show this problem.

Added dedicated support for every plugin would mean that the bar will burst when there are more requests from other users. That's why this plugin only provides a minimal default content to prevent bloat.
Of course I'm very aware that users should be able to customize the content to fit their needs, but unfortunately the current implementation of the plugin doesn't allow to simply modify single elements but only disable the default content at all.
There are more open requests in this repository for other plugins and a way to design the content per user configurations, but I haven't got the time to refactor the plugin to allow this. I'd appreciate every PR and contribution to make this possible.
So for the weather plugin the only way is to customize the plugin sources on your own or disable the default content at all like described in the documentations I've linked above.
Sorry for not being able to provide this feature at the moment.

@arcticicestudio  No worries whatsoever. Thank you for you work and explanations. Sadly, I am at the limits of my knowledge of tmux configuration so would not even know where to start to help even if I had the time. ☹ However, do let me know if you want anything tested in the future.
All the best and thank you for a super theme! ☺

The problem seems like your terminal, font and/or libc disagreeing about the width of the Unicode codepoints you are using for the weather symbols. You could try rebuilding tmux with --enable-utf8proc.

@nicm I use st and that should support uft8… Of course, it is a trivially simple terminal which means it might not work.
I tried re-compiling tmux with --enable-utf8proc and get the same issue:
; ldd `which tmux`
        linux-vdso.so.1 (0x00007f85e1672000)
        libutil.so.1 => /lib/x86_64-linux-gnu/libutil.so.1 (0x00007f85e0f95000)
        libutf8proc.so.2 => /usr/lib/x86_64-linux-gnu/libutf8proc.so.2 (0x00007f85e0d4c000)
        libtinfo.so.5 => /lib/x86_64-linux-gnu/libtinfo.so.5 (0x00007f85e0b22000)
        libevent-2.1.so.6 => /usr/lib/x86_64-linux-gnu/libevent-2.1.so.6 (0x00007f85e08d1000)
        libresolv.so.2 => /lib/x86_64-linux-gnu/libresolv.so.2 (0x00007f85e06b6000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f85e02c5000)
        libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f85e00a6000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f85e144c000)

It is not about general support for UTF-8 it is about the widths of the characters, if tmux and the terminal have different ideas about the width of characters that you use then the status line will not be aligned properly. Either it will be too long and wrap onto the following line, or it will be too short and there will be stray output at the end (like in your case).
Are you sure you restarted tmux entirely after rebuilding? tmux kill-server.
What is the actual character that is causing a problem?

@nicm

Are you sure you restarted tmux entirely after rebuilding? tmux kill-server.

Yes, I did.

What is the actual character that is causing a problem?

So far it seems to be any of the weather icons. @nikita-y seems not to have problems when they are not using them.

Right, but I don't know what weather icons are, can you copy and paste one of them here?

Right, but I don't know what weather icons are, can you copy and paste one of them here?

Ah, I see. Here are some examples:
; curl -s 'wttr.in/{Paris,london,moscow,tokyo}?format=3'
Paris:⛅ +17°C
london: ☀ +15°C
moscow: 🌩 +24°C
tokyo:⛅ +19°C

And how it looks on my terminal:

I see the same problem on OS X, libc doesn't know the width of this symbol so tmux assumes it is width 1. If you run for example tmux -Ltest -f/dev/null -vv new then do:
printf '⛅\n'
exit

Do you see this in the log?
$ grep Unicode tmux-server*.log
1588597364.430575 Unicode 26c5, wcwidth() -1

If I build with libutf8proc then tmux gets the correct width 2.
What version of utf8proc do you have? I have 2.5.0.

I think you will have trouble actually if st is also using wcwidth(), so you may need to use a different terminal.

Do you see this in the log?

Sadly, I see nothing in the logs.

What version of utf8proc do you have? I have 2.5.0.

I have 2.1.0-1 which is the default Ubuntu one.

I think you will have trouble actually if st is also using wcwidth(), so you may need to use a different terminal.

St does use that. Using gnome-terminal, I do not get the problem.

If you look for top_bit you will see the width tmux is using:
$ grep top_bit tmux-server-*.log
1588599649.505904 input_top_bit_set 3 '\342\233\205' (width 2)

If tmux is using width 2 (with or without utf8proc), and it works in gnome-terminal, then it sounds like an st problem and you would be best talking to them about it.

Ah, it does look like a st problem then:
; grep top_bit tmux-server-*.log
13206:1588599861.114720 input_top_bit_set 3 '\342\233\205' (width 2)
13426:1588599861.851887 input_top_bit_set 3 '\342\233\205' (width 2)
13855:1588599862.339703 input_top_bit_set 3 '\342\233\205' (width 2)

I shall open a bug (if I can figure out how) with them and link to this issue…


Higher Resolution RGB With NN Overlay

Start with the why:
As requested in the Luxonis Community, it is often useful to be able to show the neural results overlaid on a higher resolution RGB preview.  Say for demos, testing at a distance, and for direct visualizations in products/etc.  As it stands now the RGB preview output is the size of the neural network itself and is not configurable (where as the left and right streams are request-able at full resolution).
And as it stands now the RGB stream is selectable at 1080p and 4K resolution, but this is without neural inference result overlay.
Move to the how:
Either the H.264 stream (will need to decode and show the frames in depthai.py), or the uncompressed NV12 output of the our internal pipeline (that can be converted to BGR before displaying in python).
Move to the what:
Allow higher resolution RGB with neural inference result overlay.

[ ] Document this example on docs.luxonis.com

[ ] Document this example on docs.luxonis.com


Add barebones UVPM3 support

I did this in an attempt to add bare bones basic UVPM3 support to cats-blender-plugin
I'm not sure what
context.scene.uvpm3_props.lock_overlapping_mode = '0' used to be, but there seems to only be mode 1 and 2 now. Where mode 1 is equivalent to "Any Part" in the UI and 2 is equivalent to "exact". I'm also not sure exactly what this does on the cats side of things. Please let me know more about this and I can reassess the situation on my end and possibly fix what is wrong.
I noticed the default margin in uvpm3 is
context.scene.uvpm3_props.margin = 0.003
but I think cat's defaults to much higher...
I initially considered hard coding this at 0.003 but left it set to margin for now.
There's also rotation options which might enable even tighter packing
context.scene.uvpm3_props.rotation_enable = True
context.scene.uvpm3_props.rotation_step = 15

but I'm not sure what kind of pixel blur/quality loss might be caused as a result of this. Initially I hard coded these to be on, but I removed it after considering the quality implications. It might be worth adding these options to cats configuration somewhere at some point? I think as it is, the script might default to using whatever the user has picked under the UVPM3 menu in the UV editor? I'm truthfully not very familiar with blender plugin development or how it works.
Please feel free to let me know what you think or let me know of any changes I should make.

change the rotation step to one, and keep rotation on. The rotation rotates the UV's and doesn't rotate the pixels therefore it won't run the quality due to anti-aliasing done by usual image editors

change the rotation step to one, and keep rotation on. The rotation rotates the UV's and doesn't rotate the pixels therefore it won't ruin the quality due to anti-aliasing done by usual image editors

I've made this change. When it rotates the UV's though, won't it have to rotate the texture to match?

change the rotation step to one, and keep rotation on. The rotation rotates the UV's and doesn't rotate the pixels therefore it won't ruin the quality due to anti-aliasing done by usual image editors

I've made this change. When it rotates the UV's though, won't it have to rotate the texture to match?

Nope, because the texture is resampled during the bake process. It doesn't have to rotate the texture it completely recalculates the texture in it's entirety into a rotated state. It's like getting an artist to draw you the texture again in a rotated form and not just using an image editor to rotate it and make it all blurry
To explain it further in more complicated terms, it uses ray tracing to shoot rays onto the surface of the model and using the color samples from the texture it draws those colors onto the texture of the new UV based on what the rays find and then saves that texture into the bake folder


Cache world in VisibilityNotifier3D to avoid crash

Fixes #45508
Not sure why the world was null, but if it works...

In set_aabb() it checks is_inside_world() before accessing get_world(), maybe that's what is needed here too?

In set_aabb() it checks is_inside_world() before accessing get_world(), maybe that's what is needed here too?

No, there's still crash on exit.

No, there's still crash on exit.

Thanks!

Thanks!

Cherry-picked for 3.2.4.

Cherry-picked for 3.2.4.


"Do you want to continue anyway?"

Double check to see if the quoted message look good.

Addressed as part of text review


COVID19 API

Hi friends
First of all ...
I greatly appreciate the effort you are making to make the publications of the different api that provide information from the COVID19
Several days ago, I created an api that you might be interested in and added to the api list, for the benefit of devs.
URL COVID19 API

@ChrisMichaelPerezSantiago Thanks for your kind words.
For adding this repository to the list, please read the contribution guidelines first.


macOS "About ecode" version is inconsistent

The macOS "ecode -> About ecode" menu displays this version:

Whereas ecode's "Help -> About ecode..." menu displays this version:

Admittedly a trivial issue, apologies!

Oh, trivial but important! Thanks for reporting it.

Fixed!


[Docs] whether it supports communication betwreen upf

As general mode, traffic is flow from upf to target. But i have a lab in much traffic scene. The much traffic flows to one upf, it will increase the pressure of the server, so i want to know if it's possible to support horizontal traffic between upfs?
Thanks!

If you want to distribute the load across multiple UPFs, you can refer to the load control procedure described in TS 29.244 Section 6.2.3. In this procedure, the UPF can send its load status to the SMF via an N4 message. Additionally, in cases of heavy loading, the overload control procedure in TS 29.244 Section 6.2.4 allows the UPF to proactively send a signal to the SMF. For further details on how to manage multiple UPFs through the SMF, you can refer to TS 29.501 Section 6.3.3 on UPF selection.

Thanks!
If it means free5gc now doesn't support it ? If so, i will try to code it.

OK，Thansk


Github PR Template

Add pr template discussed in the working group.
https://github.com/camaraproject/WorkingGroups/blob/main/Commonalities/documentation/Issue and PR template Howto.md

Please @ShutingQing, do not include so far the update in Maintainers. Explanation is provided within
https://github.com/camaraproject/CarrierBillingCheckOut/pull/82

Modified.

Please @ShutingQing can you address Rafal's comments?

Resolved Rafal's comment for file changes.
About the label, hi @rartych, do you mean by label the issue to subproject management? @PedroDiez  I don't think i have the option to label the issue. Could you pls check the repository settings?

@rartych How this labelling has been performed in Identity and Consent Management. I have been checking in their repo and do not find something related to
_In order to fully support automatic issue labeling 2 labels need to be added (created): correction , subproject management.
(In Identity and Consent subprojects the label colors were set to #DB2F1F and #515C8B respectively)_

How this labelling has been performed in Identity and Consent Management?. I have been checking in their repo and do not find something related to
_In order to fully support automatic issue labeling 2 labels need to be added (created): correction , subproject management.
(In Identity and Consent subprojects the label colors were set to #DB2F1F and #515C8B respectively)_

@PedroDiez  The labels should be added manually (I haven't found a way to automate it - I guess Codeowner rights are enough) in:
https://github.com/camaraproject/CarrierBillingCheckOut/labels
compare with:
https://github.com/camaraproject/IdentityAndConsentManagement/labels

Think now PR is ready, any case please take a look @rartych

I do not think that the Bug template will be used as it is intended for Provider implementation repositories.
So the issue_bug_template.md file can be removed from this PR.
But we can keep it and simply do not use the template.

I do not think that the Bug template will be used as it is intended for Provider implementation repositories. So the issue_bug_template.md file can be removed from this PR. But we can keep it and simply do not use the template.

Please consider this @ShutingQing


Fix the first line so that it is judged as a ja doc

The current help file is not judged as Japanese help because there is no Japanese written in the first line.
This is why I added Japanese to the first line.

Thank you for sending the PR.
I'm sorry, but this issue has been addressed in the earlier PR #3 , so this one is closed.


[Event Request] Codeunit 448 Job Queue Dispatcher - OnAfterCalcNextRunTimeForRecurringJob

Please add a new publisher event to the function CalcNextRunTimeForRecurringJob as follows:
.
.
  NewRunDateTime := CREATEDATETIME(DT2DATE(StartingDateTime) + 1,0T);
END;

//>> Start New Publisher Call
OnAfterCalcNextRunTimeForRecurringJob(JobQueueEntry,NewRunDateTime,Handled);
IF Handled THEN
  EXIT(NewRunDateTime);
//<< End New Publisher Call

EXIT(CalcRunTimeForRecurringJob(JobQueueEntry,NewRunDateTime));
.
.

New Publisher Event:
LOCAL [IntegrationEvent] OnAfterCalcNextRunTimeForRecurringJob(VAR JobQueueEntry : Record "Job Queue Entry";VAR NewRunDateTime : DateTime;VAR Handled : Boolean)

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. We will update this issue with information about availability.


Support additional uplink port types in the LogicalInterconnectGroupProvider

Scenario/Intent
After SDK issue #216 gets implemented/fixed, we'll need to add the same support for our cookbooks.
Related resource:

api200/logical_interconnect_group_provider

Environment Details

oneview Cookbook Version: 2.1.0
OneView Version: Synergy

Expected Result
Support Q ports, etc., where we need to specify an enclosure_index to calculate the offset value.
Actual Result
Not currently supported

I know that oneview-sdk with these enhancements ( HewlettPackard/oneview-sdk-ruby#230  ) needs to be released first but i will preliminary look at it ;)

Theoretically we can get interconnect module type from already loaded interconnects as they always have type and bay number and enclosure_index

compile some easy readable list from @item.data['interconnectMapTemplate']['interconnectMapEntryTemplates']

['logicalLocation']['locationEntries'].find { |entry| entry['type'] == 'Bay' }
['logicalLocation']['locationEntries'].find { |entry| entry['type'] == 'Enclosure' }
['permittedInterconnectTypeUri'] (and fetch the type or name from OneView)

edit method load_uplink_sets so this part will be finding needed values in compiled structure

I can see a few challenges here ... for example these 16Gb Synergy FC switches and their enclosure_index = -1 . I need to create some mockup and test in the real environment :)

Ahhh, I've just realized that API300::Synergy contains @data['enclosureIndexes'] by default so we have to do following:

check whether @item.data['enclosureIndexes'] exists (otherwise enclosure_index is 1 or nil, does not matter)
check the length of @item.data['enclosureIndexes'], when the length is 1 - enclosure_index will be @item.data['enclosureIndexes'].first
if length > 1 - uplinks connections info has to contain enclosure_index, example below

uplink_connections = [
  { bay: 3, port: 'Q1', enclosure_index: 1 },
  { bay: 6, port: 'Q1', enclosure_index: 2 }
]

hmmm .... I can see how it has been solved in Puppet library, that's actually great and straightforward solution.
parsed_uplink_info[:connections].each do |link|
  link[:type] ||= nil
  link[:enclosure_index] ||= 1
  up.add_uplink(link[:bay], link[:port], link[:type], link[:enclosure_index])
end

I will try to implement & test this week.


Tab content scrolls out of view in Edge

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Tab group should not scroll when clicking a different tab
What is the current behavior?
When a tab group is below on the page, the page scrolls back to the top of the tabset, hiding the content of the tab and you have to scroll again to view the content of the tab.
What are the steps to reproduce?
https://stackblitz.com/edit/angular-material2-issue-ejf4cu?file=app/app.component.html
Browse all the way to the bottom so that the content of tab 1 is visible. Click on tab 2. The content of the tab scrolls out of view.
What is the use-case or motivation for changing an existing behavior?
If you are viewing Tab 1 and click on Tab 2 you don't want to scroll again. Tab 2 should become visible directly.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 6.1.1
Material 6.4.3
Typescript 2.9.2
Edge
Is there anything else we should know?

@crisbeto Were you looking at this already?  I seem to remember you working on preventing changing tabs changing scroll position?

I did https://github.com/angular/material2/pull/6812 a while ago, but that was for the scroll position inside the tabs, rather than the position on the page.


Add dark mode for AMOLED displays

It will save power on devices use AMOLED displays. (It has to be true black RGB: 0, 0, 0)

This feature has been removed in 2.1.3 due to compatibility problems with the system default dark and light theme.
Also the black theme doesn't save much more battery than dark theme : Link
So it won't be re-implemented in the near future


Fix - Logging in progress bar should converted to float.

What does this PR do?
Fixes #5481
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

It is not the progress bar responsibility of knowing how to convert objects to a float, especially when there is device specific logic embedded in there.

You are entirely. It was a temporary fix for people using this release/1.2-dev branch. I need to refactor LoggingConnector to handle things properly, but it would take me more time.
If you want to block this temporary fix, let's wait for me to properly fix it then :)
Best,
T.C

It is not the progress bar responsibility of knowing how to convert objects to a float, especially when there is device specific logic embedded in there.

You are entirely. It was a temporary fix for people using this release/1.2-dev branch. I need to refactor LoggingConnector to handle things properly, but it would take me more time.
If you want to block this temporary fix, let's wait for me to properly fix it then :)
Best,
T.C


Update job error

After updating to v7.3.3, an error message keeps popping up after starting DBeaver (with a delay of a few seconds):
Während "Update Job" ist ein interner Fehler aufgetreten. [my translation: error running "Update Job"]
Registry Directory not available: C:\Users\Ulrich%20Demlehner.eclipse\org.jkiss.dbeaver.product_7.3.2_1535670467_win32_win32_x86_64\p2\org.eclipse.equinox.p2.engine\profileRegistry.
But this very directory does exist and has a subdirectory named <DefaultProfile.profile> so I'm a bit lost what DBeaver is trying to tell me.
My environment: Win10 64 bit, German language version, DBeaver v7.3.3 also German language version.
Thanks for your help!

seems to be a duplicate of #11046 .
anyway thanks for the report

seems to be a duplicate of #11046 .
anyway thanks for the report


Remove PLRI closeOnException in computeNext

This appears to be causing a double-close on timeouts. In other iterators in this wrapping set, we don't self-close on exception, letting the top-level caller catch and close, which moves back down the wrapping set of iterators.
The RangeConcatIterator path depended on this close on exception. The second commit updates this path to close any already added ranges if building the iterator throws an exception.

No new tests failed. The code coverage is so low because we added catch blocks that are not easy to reach. I think we're good to merge it. Do you agree @jkni?


Added loading state to table body

This adds a loading state to the table body, which follows PatternFly's examples. This is displayed during pagination, when a new report is fetched.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1636   +/-   ##
=======================================
  Coverage   78.53%   78.53%
=======================================
  Files         240      240
  Lines        3936     3936
  Branches      719      719
=======================================
  Hits         3091     3091
  Misses        747      747
  Partials       98       98

Impacted Files
Coverage Δ

src/components/charts/costChart/costChart.tsx
7.93% <0.00%> (ø)

src/components/charts/trendChart/trendChart.tsx
78.44% <0.00%> (ø)

src/components/charts/usageChart/usageChart.tsx
80.95% <0.00%> (ø)

...rc/components/charts/costChart/costChart.styles.ts
100.00% <0.00%> (ø)

.../components/charts/trendChart/trendChart.styles.ts
100.00% <0.00%> (ø)

.../components/charts/usageChart/usageChart.styles.ts
100.00% <0.00%> (ø)

...charts/historicalCostChart/historicalCostChart.tsx
82.90% <0.00%> (ø)

...arts/historicalTrendChart/historicalTrendChart.tsx
83.80% <0.00%> (ø)

...arts/historicalUsageChart/historicalUsageChart.tsx
83.46% <0.00%> (ø)

.../historicalCostChart/historicalCostChart.styles.ts
100.00% <0.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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


Postgres schemas are not supported

db.py will identify all tables across all tables, but can't query them because it only tries to access the unqualified table, without the schema prepended.
For example, if I have a schema s and a table t, calling db.all() tries to execute SELECT * FROM t instead of SELECT * FROM s.t. Unless users have configured a postgres search_path (and have unambiguous table names) this query will fail.

Ran into the same issue. For schemas that are not in the search_path, tables are identified and can be described, but its methods fail b/c the sql select statement that's composed does not qualify table names with a schema name. For example, for table mytable that's not in a schema in the search_path, db.tables.mytable.count fails with this error:
DatabaseError: Execution failed on sql 'select * from mytable;': relation "mytable" does not exist

Any workarounds or tricks? Thanks.

Does this commit address the schema problem reported in this issue? I haven't tested it, but it'd be helpful to know if the problem is resolved.


[Feature] Editor support for multiple matching rules

Is your feature request related to a problem? Please describe.
I use Stylebot for an intranet system used by many different teams. I have a global site-wide style, and page-specific styles for team workspaces. Stylebot correctly matches and applies all rules, however the editor lacks an option to edit a particular rule, forcing me to use the Options screen and refresh, which is cumbersome and error-prone.
Describe the solution you'd like
When matching multiple URLs, I'd like to have an option to edit the stylesheet for a particular rule. This could be done, for example, by adding a drop-down to select the stylesheet to edit. Another option is to display, in the editor itself, the sum of all applied stylesheets, with color coding or some other visual indication as to the rule they originate from, ideally also with the ability to easily move CSS styles between Stylebot rules.
Additional context
Mock-up

@xianath makes sense - An earlier version used to support this. I will try and bring this back.


Container image build and push to quay.io/tinkerbell

Would be nice to have the CLI built into a container and an image pushed to https://quay.io/organization/tinkerbell.
Expected Behaviour

Current Behaviour

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Operating System and version (e.g. Linux, Windows, MacOS):

How are you running Tinkerbell? Using Vagrant & VirtualBox, Vagrant & Libvirt, on Packet using Terraform, or give details:

Link to your project or a code example to reproduce issue:

not something that has been needed or requested. Closing.


NBA player.minutes_played error

Describe the bug
I am trying to get minutes played for all players on all teams (NBA). When I get to a player that shows up in the roster but doesn't have any minutes played (ie, Jontay Porter, Memphis Grizzlies, 2019-20) I get an error -> AttributeError: 'Player' object has no attribute '_minutes_played'
To Reproduce
for player in roster.players:
        dict1 = {}
        dict1 = {
            'id' : player.player_id,
            'name' : player.name,
            'team' : team.abbreviation,
            'minutes' : player('2019-20').minutes_played,
            'games' : player.games_played,
            'points' : player.points,

        }
        rows_list.append(dict1)
        print(dict1)

Desktop (please complete the following information):

OS: Windows 10

For anyone running into the same issue I am just using a check on hasattr to see if player.minutes_played exists:
`
if hasattr( player , 'minutes_played' ) :
`


Add Rule 2001 to Security Hardened Shoot Ruleset

What this PR does / why we need it:
This PR implements rule 2001 of the Security Hardened Shoot Ruleset. It evaluates the SSH access field (based on it's value or availability) of the shoot spec.
Which issue(s) this PR fixes:
Part of #304
Special notes for your reviewer:
Release note:

Implementation for rule `2001` from the `security-hardened-shoot-cluster` ruleset for provider `garden`.

/assign


Add additional websockets for bookTickers

Simply adds an interface for the bookTicker(s) websocket streams.

Thank you for the contribution. Fixed in #122. Please try it there and provide feedback.


[ci] Bump version for development

@jameslamb
Could you please help with updating R-package/configure?

@jameslamb
Could you please help with updating R-package/configure?

done! I've also updated cran-comments.md, since 3.1.1 was accepted to CRAN overnight.

https://cran.r-project.org/web/packages/lightgbm/index.html


Deep linking in .Net MAUI WinUI

I'm developing a desktop application for Windows with .Net MAUI, I want to host the login in websites, so I redirect my app to:
await Launcher.OpenAsync(new Uri("https://dev.myapp.io/login/user?redirect_uri=myapp:///callapp?token=token)"));
And I have registered my app for Windows:

<uap:Extension Category="windows.protocol">
<uap:Protocol Name="myapp">
uap:DisplayNamemyapp</uap:DisplayName>
</uap:Protocol>
</uap:Extension>

In this case, when I type: myapp:///callapp?token=token in any browser, it redirects to my app, this is great, but each time it opens a new instance, so I solve the single instance issue by implementing the workaround in the last comment here:
https://github.com/dotnet/maui/issues/9973
But my issue is:
1.The uri didn't open the existing app instance, I mean make it visible
2.How can I redirect to the HomePage after login successfully, I'm not able to navigate the pages because it's written in WinUI specifically?

I need the deep linking to be smoothly running in MAUI WinUI because in my app we have quite a few cases to communicate with the websites, for some specific reasons.

Hi @OnepunchmanSven
Afraid I'll have to let your question be answered in the engineering issue. There's no deep linking doc yet because of the issues with it on all platforms.


cannot connect to azure stack using self signed cert

Storage Explorer Version:
Platform/OS Version:
Architecture:
Regression From:
Steps to Reproduce:

import self signed azure stack AAD cert
select target azure stack
restart program
you try and connect to the endpoint.  It doesn't even try.

if you try to you a connection string, it gives a cert err, and if you try and connect to a azure stack instance with a trusted certificate from global CAs you get a login prompt.
this app does not work with azure stack self signed or I suspect an enterprise signed certificates. (which it should)
Expected Experience:
to be consistent, If I choose to use a self signed cert that is my choice and the program should work
Actual Experience:
with the connection, there is no useful error at all, using a connection string it at least popups

Closing due to lack of response. Please open a new issue if you're still having problems. Thanks.


Fix UI artifacts

Fix UI artifacts
Fixes contact cards in /support overlapping the navigation dropdown and add styling to /legal.
Pull Request Checklist:

[x] Please make sure all jobs pass before requesting a review.
[x] Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).
[ ] Have you lint your code locally before submission?
[ ] Did you write tests where appropiate?

Related Issue?
Screenshots (if appropriate):

Contact cards in /support overlapping the navigation dropdown
Styled /legal

Embedded screenshots

@aphilas If you look at the legal page in master. It looks a lot better. Can you please confirm fi you can see updated designs in master?

The legal page in master is perfect, thanks.
I will revert those changes.


Do not follow redirects for the health check URLs

What does this PR do?
This PR avoids that the health checker follows HTTP redirects.
Motivation
The default behavior of the go HTTP client is to follow redirects and stop after 10 consecutive requests (see). I wouldn't expect this from a health checker. A redirect could point to a completely different host which is broken. The result of this would be that traefik removes a healthy backend.
More

[x] Added/updated tests
[ ] Added/updated documentation

Additional Notes

Hello @coder-hugo,
We think that not following the redirect for the health check makes sense but we can't integrate it like that without risking breaking existing users relying on the current behavior.
WDYT about integrating it as an health check option?


default callback to undefined

request currently does not allow you to skip providing a callback to the request function. In the logic, it knows how to handle undefined for promises, but we have to provide the undefined value explicitly

This has been merged in v2.2.0.


Update ci-cd.yml

Try:

using name with uses to label a step
new lines in file to split up steps

Codecov Report

Merging #2 (2f09080) into main (a8a6ee8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #2   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines           35        35
=========================================
  Hits            35        35

Continue to review full report at Codecov.

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


Available checkpoint?

Great work. Do you plan to release pretrained model? It would be easier for followers to try the inference:)

Hi, thanks for the interest in our work. Yes, we are working to release the checkpoint soon enough!

Hi @hjwdzh we are making the checkpoint available through hugging face. You will find the link to the model in the README.


replace constant_time_compare with hmac.compare_digest

Hi Mike,
My commit should fix the problem that I reported on #57.
It would be great if you can review it and release a new version to pypi because latest itsdangerous version on pypi breaks this package.
Thanks!

Hi @jllopezpino,
Thanks for your patience.
I believe @invisiblek is correct, and it won't work as-is.
I think we ought to:

Use hmac.compare_digest as you did
Use hmac to do the signing for simplicity (ideally allowing use of the same settings that itsdangerous did so that signatures remain compatible
Remove all of the nonsense required for Py2 to work; I'm fine with only supporting >=3.3.

Remove all of the nonsense required for Py2 to work; I'm fine with only supporting >=3.3.

I would go that way. But you are the maintainer here : )


lchown error pulling kube-cross

docker pull fails pulling kube-cross image. Most other images work though.
docker pull k8s.gcr.io/kube-cross:v1.11.2-1
v1.11.2-1: Pulling from kube-cross
bc9ab73e5b14: Already exists
193a6306c92a: Already exists
e5c3f8c317dc: Already exists
a587a86c9dcb: Already exists
1bc310ac474b: Already exists
87ab348d90cc: Already exists
786bc4873ebc: Already exists
251302927e9c: Pull complete
5651f8073a75: Pull complete
8a85eca14b20: Pull complete
ac20f6dc4bc0: Pull complete
683f835d2e99: Pull complete
bf4a87ea5477: Pull complete
0954fe016605: Extracting [==================================================>]  10.69MB/10.69MB
failed to register layer: ApplyLayer exit status 1 stdout:  stderr: lchown /usr/local/src/etcd/etcd-v3.2.24-linux-amd64: invalid argument

docker info:
Containers: 2
 Running: 0
 Paused: 0
 Stopped: 2
Images: 72
Server Version: dev
Storage Driver: vfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host ipvlan macvlan null overlay
 Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: 9f2e07b1fc1342d1c48fe4d7bbb94cb6d1bf278b
runc version: a00bf0190895aa465a5fbed0268888e2c8ddfe85
init version: fec3683
Security Options:
 seccomp
  Profile: default
 rootless
Kernel Version: 4.18.16-300.fc29.x86_64
Operating System: Fedora 29 (Workstation Edition)
OSType: linux
Architecture: x86_64
CPUs: 8
Total Memory: 15.39GiB
Name: host
ID: 4C7X:657N:XBEM:NGW3:76BW:OW55:EEQE:QCZ7:W6XE:FU3Y:Y44M:3WK4
Docker Root Dir: /home/naadir/.local/share/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
Labels:
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

For k8s.gcr.io/kube-cross:v1.11.2-1 you need more UID/GID numbers, because /usr/local/src/etcd/etcd-v3.2.24-linux-amd64 is owned by UID 478493, GID 89939 (which seems due to https://github.com/coreos/etcd/releases/download/v3.2.24/etcd-v3.2.24-linux-amd64.tar.gz ).

naadir:100000:65536 in both /etc/{subuid,subgid}
I also checked selinux, just in case, and didn't find anything interesting there
Usernetes actually works (with some minor fixups for Fedora I'll PR), and was trying to see if kind would work instead.
I'll dig into the source of that image in a bit

Ah, just seen your new comment.
I'll do that, thanks.

Ah, just seen your new comment.
I'll do that, thanks.