Switched from wireless lan logger LSW-3 to cable logger LSE-3

I first tried the LSW-3 logger and I could connect with home assistant to the logger and get data.
But I don't have good WLAN in the cellar where the inverter is installed. So I switched from the wireless version to the LAN cable version called LSE-3. LSE-3 is reporting to solarmanpv website successfully. But Home Assistant doesn't want to connect anymore. Does somebody know if there is a setting I need to change or have any other idea what's wrong.
I have changed the serial number of the logger in the config. Otherwise I use the deye_sg04lp4.yaml file.
I entered the IP address of the logger and setup the network that it is always the same IP address.
I can also reach the configuration web page of the logger.
The number of the serial number starts with 21...

Hi, I second this -  got LSE3 wired logger and so far was unable to connect it to HASS at all. I Will gladly serve as a sandbox for LSE3 integration testing :) (SN 21xxxxxxxx...) firmware MW4C_14_CSIP_1.14

It looks like the stick has port 8899 open:
Starting Nmap 7.80 ( https://nmap.org ) at 2023-03-20 15:43 CET Nmap scan report for solarman-logger.fritz.box (<IP_ADDRESS>) Host is up (0.0029s latency). Not shown: 998 closed ports PORT     STATE SERVICE 80/tcp   open  http 8899/tcp open  ospf-lite

I read in other issues that this integration supports only the WiFi dongles, some had success with the beta version, but most not. Search for "ethernet" in the issues. If you find a way, please tell me!

Ik have the same issue :(

I still did not get it to work? Did anyone find a solution?

If you have the LAN adapter, just try the SolaX Power Modbus custom_component

Ow, Ok, lets try that!!  I'll let y'all know the results

I have the same logger LAN and I cannot as well use this integration, values always unknown. right now the only solution I found, as reported also above, is with SolaX Power modes integration.

I have the same problem. I cannot read Modbus from Deye SUN 5K sg03lp1 inverter with a LSE-3 Stick logger. The LAN LSE-3 logger (s/n 21xxxxxxxx) works well with Solarman website but returns incorrect data from local requests based on Solarman V5 protocol.
For example a request for battery voltage (reg:587) gives me :
Tx Packet: a5 1700 1045 b200 <s/n> 02 0000 00000000 00000000 00000000 01 03 024b 0001  15
Rx Packet: a5 1700 1045 03 00 bc 02
The return packet is way too short , should have 1015 instead of 1045 and length is wrong.
Does anyone have any idea where to look?

McLP11, could you tell more about the use of SolaX Power modes integration with LSE-3 and Deye ? Thanks

@adnovea right now SolaX Inverter Modes looks like the only integration I was able to connect with my LSE-3 LAN dongle. It exposes 159 sensors, including also some switches or specific tools for configuration but not tested yet at all. It's pretty stable, I've just noticed that sometimes is not back online when you restart HA, new restart and back again

Thanks McLP11 for your reply. I'm trying to understand what is different between SolaX and Solarman to make the LSE-3 working. Is SolaX using a portal and an API ? What I'm looking for, is local communicate with my Inverter using the LSE-3 on my local network only (not using the Solarman portal API). This was possible with the LSW-3. Which version of SolaX have you installed, the one I used with HAC is SolaX Inverter Modbus and expose only 4 sensors.

I can communicate with the inverter using the LSE-3 Stick Logger and the Modbus RTU/TCP tester (free) application (https://www.gineers.com/media/downloads/GModbus.zip). The LSE-3 seems to support RAW Modbus RTU/TCP without the V5 Solarman encapsulation. This has to be confirmed. I'm waiting from Solarman feedback about a LSE-3 F/W version supporting V5 protocole.

This is great to hear. I am following this development!

Dear Stephane,
Would you mind to add this protocol to your component in order to support the Ethernet LSE-3 (S/N 21xxxxxxxx) as a selectable option. We will be grateful for this update and again a big thank you for your great work.

I have an zcs azzurro 80 ktl-v3 and the lse-3 Ethernet adapter, but I can't connect. can you help me, I tried your solution but it dosent work.

Is your LSE-3 S/N in 21xxxxxxxx ?
Do you have a LSW-3 module and is it working ?
Python indentation is very critical and you better replace Tabs by Spaces.
You can enable the HA debug mode and check the log to see if there is problem with the Solarman modified code
To check if the LSE-3 communication is working  download the GModbus application (https://www.gineers.com/media/downloads/GModbus.zip) and run :

Enter inverter IP address and port (8899) and press Connect TCP
Check "Holding", set Address to "1156" (0x484) and Count to "1" to read Grid Frequency x0.01
Press Read registers
Few seconds later you have the result in the Grid on the right.
In decimal column you should read a value around 5000

If this fails, my mods cannot work for you.

My LSE-4 S/S is in 21xxxxx, I don't have a LSW-3 module because there is no WLAN at this place. I tried the GModbus application with your settings and get something back, look at the picture.

The GModbus results are encouraging. Your communication to the LSE-3 stick logger is successful.
If you found in the HA log errors such as  "Querying [0x0400 - 0x042B] failed..." it's probably a typo issue.
Check the code indentation.

I checked the code and make it all new, now it works for me, thanks for you solution. is there a way to read the modbus register? In the azzurro.yaml there are only two sensors of the strings, and the 80ktl-v3 have some more. By google I didn't found the full modbus register for zcs azzurro

Great ! I'm glad the mods works.
You can read any Azzuro Modbus Register using the GModbus application:

Connect, select Holding, enter start address and the number of register to read (max 30 otherwise it could fail repetitively)

Your string sensors are registers are :

0x0584: PV1 Voltage
0x0585: PV1 Current
0x0586: PV1 Power
0x0587: PV2 Voltage
0x0588: PV2 Current
0x0589: PV2 Power

You can try to read register from 0x058A (1418) to 0x58F (1423) and check if the data pattern looks like the PV1/PV2 ones.
Often, strings data are next to each other.
Sorry, I cannot help you more about the Azzurro.

Modification for LAN LSE-3 Stick Logger module using raw Modbus RTU protocol
I failed to make the mods working because I have no experience in Python coding.
In config_flow.py I added the following line:
 vol.Optional(CONF_LOOKUP_PROTOCOL, default=data.get(CONF_LOOKUP_PROTOCOL)): vol.In(LOOKUP_PROTOCOL),

but I cannot correctly retrieve the protocol radio-button value in the Solarman code.
Could someone help ? Thanks.

I have the same issue. I can't communicate in any way, all parameters remain at "Unknown". I tried to follow different configurations but I can not solve.
My configuration is:

Inverter "ZCS HYD-6000-HP" with firmware "V40022_2023-05-22".
Ethernet logger "LSE-3" with firmware "ME_0D_270A_1.09".

Does anyone have an idea how to fix it?

folder and then apply the modifications below using

@adnovea Does this solution work?

May I suggest you to follow the steps below:

Find the Modbus register list (Internet search) corresponding to your Inverter (https://github.com/StephanJoubert/home_assistant_solarman/files/10187824/solarman-ZCS_HYD-6000.txt)
Use GModbus utility to test some registers and validate the list.
Check if the file zcs_azzurro-ktl-v3.yaml has the correct register values. If not, update the YAML file according to your Inverter registers (you can create a copy of the zcs_azzurro-ktl-v3.yaml file into a file named custom_parameters.yaml and select this file in the HA Solarman module configuration).
Modify the Solarman module files parser.py and solarman.py as I mentioned above (create backup file to roll back if needed).

If you succeed in the steps above, you'll be able to retrieve data from your inverter.
N.B. I you encounter some troubles connecting with GModbus, may be your stick logger F/W need to be updated. You can email the customer service at Solarman<EMAIL_ADDRESS>and they will do the update remotely.
see also https://github.com/StephanJoubert/home_assistant_solarman/discussions/148

Yes my mods works with Deye and has been used with Azzuro inverter by gmbh07.
Copy the solarman-ZCS_HYD-6000.txt above into custom_parameters.yaml,
apply the modifications,
then select the _custom_parameters.yaml_file in HA Solarman module
and full reboot HA.

the solution from adnovea for my azzurro and LSE-3 is still working with no problems

@adnovea with your suggestion my configuration is working.
My configuration is:

Inverter "ZCS HYD-6000-HP" with firmware "V40022_2023-05-22".
Ethernet logger "LSE-3" with firmware "ME_0D_270A_1.09" and SN "21xxxxxxxx".

I'm using the configuration "zcs_azzurro-ktl-v3.yaml" is the only one that works, but I see only few values updated.
What is the best configuration I can use?
Since your code works, can't the developer @StephanJoubert upload it to the official library?

Hi Michael,
Glad to see that the mods is working with your config.
I hope @StephanJoubert will update the code for supporting the LSE-3.
Have you tried to download the configuration file at https://github.com/StephanJoubert/home_assistant_solarman/files/10187824/solarman-ZCS_HYD-6000.txt ?
Did it expose more values ?

I tried to put the content fo your txt file in "custom_parameters.yaml" (27 entities) but the communication does't work.
I wait for some minute but it remain Disconnected.

Once you have :
1-  created the "custom_parameters.yaml"
2- fully restarted HA
3- cleared the browser
4- and re-logging to HA,
you should have 27 entities exposed by the integration.
If this still fails, try to communicate with you inverter using GModbus (see above how to run this utility).
Attention: register values shall be entered as decimal (not Hex) values.

For ME3000-SP and the LSE-3, using the sofar_wifikit preset and changes by @adnovea (https://github.com/StephanJoubert/home_assistant_solarman/issues/247#issuecomment-1591817882) worked wonders.

Change seems to be incompatible with 1.5.0 because of a switch to pysolarmanv5. @adnovea not sure if you know how to work around that? I'm happy to open a pull request with a enable/disable V5 encapsulation toggle, though for 1.5.0 I'm not sure how to disable V5 anymore.

Hi maartenba,
I made the mods for 1.0.0.
Basically the LSE-3 support a RAW Modbus protocol.
To support the LSE-3 module, the V5 solarman protocol that encapsulate the RAW Modbus data just need to be optional.
My mods was only commenting the code line of the encapsulation.
I'm sure Stephan know how to do this but I'll be please to help when time permits.

I would like to ask if someone would be willing to upload modified files here for use by LSE, somehow I can't do it. Thank you very much .

Hi Mocky158, if you can wait a about a week, I'm creating a repo (solarman_lse) with a fork from the nice Stephan Add-on to support the LSE stick loggers. The purpose of this repo is mainly to share a quick solution for those with LSE Stick Logger. It is compatible with the inverter_definitions and provides a more flexible solution for Deye Sun 5K with possibility to write registers. For other inverters, owner should dig in the inverter Modbus register documentation to find the corresponding codes and values (BE CAREFUL because this can definitely damage the inverter).

@adnovea
You describe here (https://github.com/StephanJoubert/home_assistant_solarman/issues/247#issuecomment-1590665761
) that you adapt two files (parser.py and solarman.py) and the LAN dongle then works on the DEYE SUN?
Maybe I'm being very stupid, but I can't find the values in the two files.
My dad has the DEYE SUN and I want to help him integrate it into HomeAssitant.
THX

Niklas1E, the last version of Solarman use the Solarman library instead of the files mentioned above.
That's the reason why I had to fork this project into the repo solarman_lse.

            Thank you very much!Can I damage anything on the inverter with the add-on?Or does it only read data or does it also write data?Sorry, I'm an amateur.Am 20.01.24 um 08:23 schrieb AdNovea®

                Von: "AdNovea®"<EMAIL_ADDRESS>20. Januar 2024An: "StephanJoubert/home_assistant_solarman"<EMAIL_ADDRESS>"Comment" ***@***.***>,"Niklas1E"<EMAIL_ADDRESS>Re: [StephanJoubert/home_assistant_solarman] Switched from wireless lan logger LSW-3 to cable logger LSE-3 (Issue #247)

Niklas1E, the last version of Solarman use the Solarman library instead of the files mentioned above.
That's the reason why I had to fork this project into the repo solarman_lse.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

There is always a risk when you don't perfectly understand what you do.
Read carefully all the instructions. This is not a "plug&play" module. It has not been tested with many inverters nor fully debugged.
I just share what I'm currently using for my Deye Sun 5K but you must understand what you are doing.
As long as you avoid writing to the inverter registers, the risk is limited.
Do not use switch entities to prevent sending unexpected commands and keep only read registers.

Hello @adnovea . I'm in the same situation.
Deye Sun 6K, with LSE-3 SN: 21*******
1- created the "custom_parameters.yaml"
2- IP ans SN:

3- Configuration in HA:

4- Result:

Why? What I'm doing wrong?
Thank you very much!

Imanol82 did you read the whole post ?
I explain above why the Stepan module does not work with LSE module and what can be done to solve the problem.
The current version cannot be modified for LSE and you need to tune the add-on with my solarma_lse fork.

Thank you very much!
I'm very new in HA... I'll try mod.
:)

Hello. I have an Afore BNT004KTL inverter with LSE-3 logger serial number 21xxxxxxxx and no configuration works in homeassistant. Does anyone have this inverter connected? Please help.

There is always a risk when you don't perfectly understand what you do. Read carefully all the instructions. This is not a "plug&play" module. It has not been tested with many inverters nor fully debugged. I just share what I'm currently using for my Deye Sun 5K but you must understand what you are doing. As long as you avoid writing to the inverter registers, the risk is limited. Do not use "switch/select" entities to prevent sending unexpected commands and keep only "sensor" registers.

Hi! I am using the Solarman LSE for my SUN-10K-SG04LP3-EU. Everytime I set up the interation, in less then 24 HRS, my inverter falls into COM error and has to be restarted. That never happens when the integration is off. As I am using LSE version and my neigbor literally has the same solution, but with standard Solarman integration and wireless logger working just fine, I am just asking  -  where to look, who to ask? Or ditch it? Many thanks, Martin


Alsa is not working as it should with headphones

I have some issues related to Alsa. I'll describe what I do and what I'm getting
Expected behavior:
Module should enable/disable sound after one press on the volume icon; should show actual volume and state (muted or not) for current source (headphones or speaker) and current volume
Actual behavior:
It works randomly. Let's see what actually happens.
First, making a clean state of Alsa
alsactl restore
Then I see in Polybar actual volume (no headphones) and can change it via mouse wheel
Then I'm pressing on the volume icon
No sound
Pressing again
Sound appears. Everything is fine
Attaching headphones
Music is working and I can change volume, label the same as ramp-headphones-0
Pressing on the icon
No sound and label the same as label-muted
Pressing again. NO SOUND and label the same as label-muted
Mouse wheel doesn't help - no sound
Pressing again on the icon
Label now from ramp-volume with number % but HEADPHONES are attached
Detaching headphones
Sound is playing but icon shows muted state and I CAN adjust the volume with mouse wheel
Pressing on icon
Shows ramp-headphones-0 label with wrong volume and sound is playing
Pressing again and sound muted and label from muted state. Ok.
Attaching headphones again
No sound, icon shows muted state
Impossible to change volume. Icon after a few presses shows ramp-headphones-0 label with 0%
Detaching headphones. Sound appears.
To enable sound in headphones I should restart Alsa since without that action sound will never work.
Polybar Config
[module/alsa]
type = internal/alsa

master-soundcard = hw:0
speaker-soundcard = hw:0
headphone-soundcard = hw:0

master-mixer = Master
speaker-mixer = Speaker
headphone-mixer = Headphone
headphone-id = 4

; the same issue with  = false
mapped = true
interval = 5

format-volume = <ramp-volume> <label-volume>

label-muted = 

ramp-volume-0 = 
ramp-volume-1 = 
ramp-volume-2 = 

ramp-headphones-0 = 
ramp-headphones-1 = 

Environment:
polybar 3.1.0-140-g067aac1
Features: +alsa +curl +i3 +mpd +network +pulseaudio
X extensions: +randr (+monitors) -render -damage -sync -composite +xkb +xrm +xcursor
Build type: RelWithDebInfo
Compiler: /usr/sbin/c++
Compiler flags: -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -Wall -Wextra -Werror -Wno-noexcept-type -O2 -pedantic -pedantic-errors
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Exact same problem, it would be great to have a fix


Documentation and Testing

Integrate with Blockly source, setup tests and basic documentation

Progress can be tracked on branch webpack-integration


Handle bounds in the SelectAndScatterOp shape function

SelectAndScatterOp's bound inference is very simple. There are two inputs: operand and source. The source contains the indices to the result tensor to place the reduced values from operand, so it does not affect the bound of the result tensor. As for operand, the type is equal to the result tensor. There is only a movement of values happening in this op (after reduction is applied, but this also does not affect the bounds), so we can propagate the bounds with no further inference done.
closes #754

After further look, adding AllTypesMatch/AllShapesMatch traits do not work when we need compatible types, and #900 is migrating away from them.

"Technically, shapes don't have to match and just need to be compatible but we require equal shapes for binary ops so should apply here as well". Overall, the trend in the last 6 months was to require compatible shapes wherever possible to better support dynamic programs. For example, binary ops say HLO_CompatibleOperandsAndResultType instead of SameOperandsAndResultType, and we've made similar relaxations to other ops too. Here's an example: https://github.com/openxla/stablehlo/blob/067750f1bfaf3361611571fb2f545f5db0351d2e/stablehlo/dialect/StablehloOps.td#L675-L676.

Ah, right. This makes sense.
I was looking at StableHLO_BinaryElementwiseOp and got confused. That is still adding the SameOperandsAndResultShape trait. This trait is named same but actually checks for compatibility. I looked at the 15 ops deriving from this and only MHLO_ComplexOp needs that. Even for that we should probably defined and use  HLO_CompatibleOperandsAndResultShape trait. Similarly, for unary and other ops using SameOperandsAndResultShape trait but that is more involved as the element types differ.


Setup integration tests for CDS 3.3

Is your feature request related to a problem? Please describe.
We have a shim layer for CDS 3.3 now, we should setup integration testing on a cluster.

it's added as part of regular CI jobs with internal cluster. Close it now


Incompatibility with nette/forms v3.1.0 - Undefined index: perPage

Hello :)
After today upgrade, I got Undefined index: perPage - File: .../vendor/ublaboo/datagrid/src/DataGrid.php:1509
1509:            $this->saveSessionData('_grid_perPage', $values['perPage']);
Error occurs after pressing Save button in inline edit.
The problem is not in new version of Datagrid, but in some incompatibility with new Nette Forms (which new version is from 2020/01/19). It appears after upgrade nette/forms from v3.0.7 to v3.1.0. Datagrid v6.6.1 with Nette 3.0.7 is ok, with Nette 3.1.0 is not.
Problem is (probably) in nette/forms BC break mentioned in release notes (Form::getValues() returns only controls in scope (BC break)). Inline editor submit buttons don't validate the perPage control, so new nette/forms doesn't return it's value.

Looking into that 👍

Fixed in https://github.com/contributte/datagrid/releases/tag/v6.8.0


Script is incorrect [Cognitive Services]

I receive the following runtime error in PowerApps when attempting to call LanguageDetect:
"TextAnalytics.DetectLanguage failed: {"code":"Bad Request", "message":"Invalid request","innerError":{"code":"InvalidRequestQueryString","message":"Request contains a query string. Request content should be placed in the request body, not in the url as a query string."}}

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

ID: e58cccc6-9234-ec33-b4b0-3303ebe282ef
Version Independent ID: a556f95f-240d-206b-6dd0-c2fc47b1bf11
Content: Use Cognitive Services in PowerApps - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/cognitive-services-api.md
Service: powerapps
GitHub Login: @gregli-msft
Microsoft Alias: gregli

Also stumbled across this issue. The problem seems to be related to the language detection call. KeyPhrases and sentiment work fine.

@Microsoft - Is anybody working on this request. Also, please add codes to clear the collection and fields. Everytime I run the app the labels are prepoluated with information from the previous session.

Use DetectLanguageV2 call:
If( chkLanguage.Value = true,
ClearCollect( languageCollect,
TextAnalytics.DetectLanguageV2({text:tiTextToAnalyze.Text}).detectedLanguages.name
)
);


Unable to parse client secret file to config: oauth2/google: missing redirect URL in the client_credentials.json exit status 1

Error
I started using your example. and I get this error.
2020/03/19 17:06:41 Unable to parse client secret file to config: oauth2/google: missing redirect URL in the client_credentials.json
exit status 1
Any suggestions please

Go version (go version) : go version go1.13.7 windows/amd64
OS (Windows)

For the moment I solved this issue. First I checked both the go get parts. The same error result. Then I repeated step 1 in order to create a new version of my credentials.json.  This was the issue apparently: the quickstart.go responded as is described.
Now I can go further with developing my program to filter certain events.
For the moment thanks to this simple tutorial.

How to generate credentials.json file , the same error result with you


update crossbeam to free memory early

What problem does this PR solve?
Issue Number: close #10037
Check List
Tests

Unit test
Integration test

Release note

Speed up memory freeing

This pull request is closed because it's related version has closed automatic cherry-picking.
If it's still needed, you can reopen it or just regenerate it using bot,
see:

https://prow.tidb.io/command-help#cherrypick
https://book.prow.tidb.net/#/plugins/cherrypicker

You can find more details at:

https://internals.tidb.io/t/topic/785


SSL Not Working in Latest Rust

Overview
When trying to use the examples from crates.io and github or from around the issues list, Lapin fails to connect using SSL. Non-SSL connections do work.
It appears that the return value from connection unusable. Be warned though, I am pretty new to Rust. The std::iter::Iterator seems to be unimplemented?
Expected Result
When using amqps as the key, it is expected that lapin can connect to RabbitMQ using lapin futures with the latest Rust and Tokio.
Actual Result
An unimplemented error is thrown related to the trait.
Report
Technologies Overview

Rust Stable 1.39.0
lapin-futures 0.28.3
lapin 0.28.3
tokio 0.2.0-alpha.6

Cargo Dependecies:
[dependencies]
env_logger = "0.7.1"
log = "0.4.8"
futures = "0.3.1"
tokio = "0.2.0-alpha.6"
lapin-futures = "0.28.3"
lapin = {version="0.28.3", features=["futures"]}
futures-util = "0.3.1"
amq-protocol = "3.1.0"
native-tls = "0.2.3"
tokio-tls = "0.3.0-alpha.6"

Code:
use futures::future::Future;
use lapin_futures as lapin;
use crate::lapin::{BasicProperties, Client, ConnectionProperties};
use crate::lapin::options::{BasicPublishOptions, QueueDeclareOptions};
use crate::lapin::types::FieldTable;
use log::info;
use tokio;

fn main() {
    env_logger::init();

    let addr = std::env::var("AMQP_ADDR").unwrap_or_else(|_| "amqp://<IP_ADDRESS>:5672/%2f".into());
    let rt = tokio::runtime::Runtime::new().unwrap();
    rt.block_on(async move {
        Client::connect(&addr, ConnectionProperties::default()).map(|client| {
            println!("Connected");
        }).map_err(|err| println!("Error {:?}", err))
    });
    rt.shutdown_on_idle();
}

Error:
error[E0599]: no method named `map` found for type `lapin_futures::client::ClientFuture` in the current scope
  --> src\main.rs:15:65
   |
15 |         Client::connect(&addr, ConnectionProperties::default()).map(|client| {
   |                                                                 ^^^ method not found in `lapin_futures::client::ClientFuture`
   |
   = note: the method `map` exists but the following trait bounds were not satisfied:
           `&mut lapin_futures::client::ClientFuture : std::iter::Iterator`
   = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope; perhaps add a `use` for it:
   |
1  | use futures::future::Future;
   |

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
error[E0599]: no method named `map` found for type `lapin_futures::client::ClientFuture` in the current scope
  --> src\main.rs:15:65
   |
15 |         Client::connect(&addr, ConnectionProperties::default()).map(|client| {
   |                                                                 ^^^ method not found in `lapin_futures::client::ClientFuture`
   |
   = note: the method `map` exists but the following trait bounds were not satisfied:
           `&mut lapin_futures::client::ClientFuture : std::iter::Iterator`
   = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope; perhaps add a `use` for it:
   |
1  | use futures::future::Future;

Is there a way to solve this?
Thanks and keep up the good work.

Hi,
have you tried what rustc tells you?
help: the following trait is implemented but not in scope; perhaps add a `use` for it:
   |
1  | use futures::future::Future;


circul takes two vectors but uses entries of only the first one

It uses the length of the second one. This is kind of weird, but I guess not technically a bug. Is this interface choice deliberate?

Hi @argriffing ,
Here is the interface for circul matrix:
julia> matrixdepot("circul")
Circul matrix:

 Input options:

 vec, n: a vector and the column dimension

 vec: a vector

 [type,] dim: the dimension of the matrix

 ['symmetric', 'pos-def', 'eigen']

 Reference:  P.J. Davis, Circulant Matrices, John Wiley, 1977.

So basically, we could do
julia> matrixdepot("circul", [1,2,3])
3x3 Array{Int64,2}:
 1  2  3
 3  1  2
 2  3  1

julia> matrixdepot("circul", [1,2,3], 2)
3x2 Array{Int64,2}:
 1  2
 2  1
 1  2

julia> matrixdepot("circul", 3 )
3x3 Array{Float64,2}:
 1.0  2.0  3.0
 3.0  1.0  2.0
 2.0  3.0  1.0

julia> matrixdepot("circul", Int, 3 )
3x3 Array{Int64,2}:
 1  2  3
 3  1  2
 2  3  1

```
Inputting two vectors is not recommended. But ``matrixdepot("circul", [1,2,3], [1,2])`` is equivalent to ``matrixdepot("circul", [1,2,3], length([1,2]))`` if you are wondering.

Best,

Weijian

OK I see. Only the reasonable call signatures are documented in that way, and the weird call signatures which are technically part of the public interface are not documented, or at least are not documented as prominently. Thanks!

Hi @argriffing,
Thanks to Julia's new documentation system, all matrix generators are now properly documented. See https://github.com/weijianzhang/MatrixDepot.jl/blob/master/src/higham.jl
Just let you know.
Thanks,
Weijian

Also here is the documentation of circul from the master version:
julia> matrixdepot("circul")
     Circulant matrix
    ≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡≡

  A circulant matrix has the property that each row is obtained by cyclically
  permuting the entries of the previous row one step forward.

  Input options:

    •  [type,] vec, n: a vector and the column dimension;

    •  [type,] vec: a vector;

    •  [type,] dim: the dimension of the matrix.

  ['symmetric', 'pos-def', 'eigen']

  Reference: P. J. Davis, Circulant Matrices, John Wiley, 1977.

Thanks! I don't use the Julia language much yet, but I'm sure I will in the future.


Fix: Error if the user/role with Postres already exists

Issue
There is an issue with postgresql installation, more specifically creating the user role postgres if the user role already exists.
The part of the script with the issue:
setup_postgres:
pg_ctl -D "$HOMEBREW_PREFIX/var/postgres" start
  "$HOMEBREW_PREFIX/opt/postgres/bin/createuser" -s postgres
  pg_ctl -D "$HOMEBREW_PREFIX/var/postgres" stop

I already had a postgres role/user installed so this caused the following error:
server started
2022-07-22 16:48:40.978 +07 [42818] ERROR: role "postgres" already exists
2022-07-22 16:48:40.978 +07 [42818] STATEMENT: CREATE ROLE postgres SUPERUSER CREATED CREATEROLE INHERIT LOGIN;
createuser: error: creation of new role failed: ERROR: role"postgres" already exists
failed

Would this also cause a problem if although the role didn't exist but the service was also already running, ie from installation outside of brew, such as via Postgres.app?
I had to manually remove the above part so I could continue to the asdf installation step.
Should the postgres setup and role creation be optional? Or shouldhave pre-checks if postgres is already running or the user role already exists?
Expected
I expected that it would not cause errors if the script was ran  multiple times.
Steps to reproduce

Have Postgres already installed with a user/role postgres
Run the script

Addressed in same PR - #38


[DOCFIX] Update Alluxio Storage

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18553/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18556/
Test PASSed.


Sending a function name to the chat spills JS code into chat

!mmm chat: ${chat} spills JS code into the chat:
phyllo:function functionImplWithVariables(...args)
{
return value.apply(variables, args);
}

!mmm   debug chat: ${playerMessage} ${chat} makes it look more confusing:
(From Mych's Macro Magic):
🔎 "/w "Yerrick MacRothgar" Ahhhh -- da ist dir gerade ein kalter Schauer den Rücken hinuntergelaufen!" function functionImplWithVariables(...args)
Mych's Macro Magic: {
return value.apply(variables, args);
}
(From phyllo): Ahhhh -- da ist dir gerade ein kalter Schauer den Rücken hinuntergelaufen!
I have a vague idea now what I was trying to do there, and I think at the time ${chat} just told me the function name, if it was overwritten with whisperback() or not. I'm not doing that any more, anyway, and the current version of the senseLogic script does not reproduce the bug.

Ah, overwriting chat with  whisperback doesn't work anymore anyway (because chat() has different parameters since the recent "impersonate sender" change).
Is/was this a useful feature for you? I could try to revive it then (by matching up whisperback()s parameters again).

No, please don't. I never seriously used it, just played around with it once and apparently forgot to remove debugging code from the version that ended up in the live game 👎

Okay.
I'll still do something that makes the result of stringifying of chat less useless – e.g. make it stringify to "chat(p1,p2)", or in case of custom functions, the same but with the actual parameter names.


Create HttpClient without Base URL when running from Electron

Is your feature request related to a problem? Please describe.
I modified the Client-Side example to run in Electron which works fine, save for the injected HttpClient.
Describe the solution you'd like
When I do
@inject HttpClient http

I get an exception:
System.ArgumentException: Only 'http' and 'https' schemes are allowed.

This happens because Electron calls a local index.html file which makes the Base URL a file:// URL which HttpClient is not happy with. It happens here in WebAssemblyHostBuilder:
services.AddSingleton<HttpClient>(s =>
{
    // Creating the URI helper needs to wait until the JS Runtime is initialized, so defer it.
    var navigationManager = s.GetRequiredService<NavigationManager>();
    return new HttpClient
    {
        BaseAddress = new Uri(navigationManager.BaseUri) // <- boom!
    };
});

I suggest checking if the BaseUri is http or https, and if not, just returning a new HttpClient(); and maybe issuing a warning to the log.
Workaround
I discovered that you can add services which already exist in the ConfigureServices method of Startup.cs and they will override the old definition. So, if you add this to your ConfigureServices method, it works as I suggest:
services.AddSingleton(delegate (IServiceProvider s) {
    var nm = s.GetRequiredService<NavigationManager>();
    var lf = s.GetRequiredService<ILoggerFactory>();
    var l = lf.CreateLogger(typeof(Startup));
    var baseUri = new Uri(nm.BaseUri);
    var scheme = baseUri.Scheme.ToLowerInvariant();
    switch(scheme) {
        case "http":
        case "https":
            l.LogInformation("Configuring HTTP Client with Base URI {0}", baseUri);
            return new HttpClient {
                BaseAddress = baseUri
            };
        default:
            l.LogWarning("Scheme is {0} instead of http or https, returning HTTP Client with default settings", scheme);
            return new HttpClient();
    }
});

So when the HttpClient is requested, it is first checked if the scheme of the base URI is http or https. If it is, the client is configured with the base URI, otherwise a client with the default settings is returned.
When now doing @inject HttpClient http there will be no crashes, but when running from Electron, you get a client which is not configured with a Base URI which can be used to query remote APIs.

@rabejens thanks for contacting us.
We plan to move the registration of the HttpClient instance outside of the default configured services. We used to do this in the past due to the need to replace the default HttpClientHandler, but that's not the case anymore.
In the future this won't be an issue and users will get to register the HttpClient in DI in whatever way they want.

@SteveSandersonMS in case he wants to add anything.


Many snippets do not work in VS Code on Mac OS

Many snippets e. g. tf-vsphere* or tf-openstack* do not seem to work on VS Code on Mac OS.
Steps to reproduce:

Create a .tf file associated with Terraform Syntax Highlighting
Start typing "tf-vsp", nothing happens :(

Tested with:

VS Code 1.19.3 on Mac OS 10.12.6 (Sierra)
Advanced Terraform Snippets for Visual Studio Code 2.0.1

@pyroxde thanks, I'll look into it. cheers

Openstack isn't working for me on High Sierra, VSCode 1.25.1


[WIP] Add bundle add command to add gems into the Gemfile

Continuing the work done to implement bundle add GEM from bundler/bundler#4546
To do:

[x] Cherry-pick commit from @ognevsky
[x] Run specs
[x] Use Bundler.definition to resolve last_version_number
[x] Pattern bundle add implementation after bundle inject
[x] Add option to specify added gem's group
[x] Add option to specify added gem's source
[x] Modify existing specs & add additional specs

@RochesterinNYC

Is there any reason this couldn't just be a plugin?

I actually want this on a regular basis, and have planned on adding it (when I get around to it 😝) for years... this is also a standard-ish feature in RubyGems, NPM, etc. I feel like it's got a pretty good case for being in core.

Should a source argument be supported? And if the Gemfile is organized in source blocks ... I guess this command wouldn't be able to add the line within an existing source block very reliably (esp. in the case with a mix of blocks and same-line and no specified source).

@chrismo, that's a fair point. For the baseline implementation, I might just expect a source argument to add the gem with the proper source explicitly at the end. I'm not sure the current Bundler classes have the ability to inject a gem into a source block for a Gemfile. Are there any use cases where the injection of the added gem into the source block is necessary as opposed to being a nice-to-have?

@RochesterinNYC Made a few changes. Can you review this? :smile:

hey @b-ggs, some things that @indirect and I discussed:
There is a bundle inject command. bundle add will be like a more user-friendly, advanced version of bundle inject. You've already covered the two main UX differences:

version is not required to be passed to bundle add
the default version constraint is ~> <latest_version>
However, it'd be nice if the bundle add and bundle inject commands could use the same underlying class (maybe the Injector https://github.com/bundler/bundler/blob/master/lib/bundler/injector.rb) as the overall functionality is very similar. Does that make sense?

:umbrella: The latest upstream changes (presumably #4608) made this pull request unmergeable. Please resolve the merge conflicts.

@RochesterinNYC Still need to add specs, but how's this?

@RochesterinNYC Code Climate seems to be complaining that this is too similar to the code in bundle inject. Maybe I should just call Inject.new([...]).run from within add.rb? Do you have any suggestions for this?

A --pre flag sounds compelling.
Yup, I don't think the Add class needs to exist. Rather, inject with certain flags/options passed in could serve for both commands.

@RochesterinNYC Removed the Add class, and moved everything into the Inject class. I also implemented the --pre flag for adding prerelease gems. Also, I added the appropriate specs. Could you check it out and let me know if I've missed anything? Thanks!

Great, what does the output for this look like? Additionally, does this print out the timestamp line in the Gemfile like bundle inject will? We'd want to avoid printing that line for bundle add.

@RochesterinNYC It originally also appened the timestamp. To remove the timestamp, I had to edit the Injector class to add an option to omit the timestamp from the lines appended to the Gemfile.
Here's the output for this:
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile
source 'https://rubygems.org'
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  dbundle add rack 1.5.0
Fetching gem metadata from https://rubygems.org/
Fetching version metadata from https://rubygems.org/
Added to Gemfile:
  rack (= 1.5.0)
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile
source 'https://rubygems.org'

gem 'rack', '= 1.5.0'

I also made sure that it didn't affect how bundle inject appended lines to the Gemfile.
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  dbundle inject rack 1.5.0
Fetching gem metadata from https://rubygems.org/
Fetching version metadata from https://rubygems.org/
Added to Gemfile:
  rack (= 1.5.0)
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile
source 'https://rubygems.org'

# Added at 2016-06-16 12:36:59 +0800 by boggs:
gem 'rack', '= 1.5.0'

@RochesterinNYC For some reason, if I don't include a version number, it seems to fetch gem metadata twice. I'll look into this.
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile                                                                                1 ↵
source 'https://rubygems.org'
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  dbundle add rack
Fetching gem metadata from https://rubygems.org/
Fetching version metadata from https://rubygems.org/
Resolving dependencies...
Fetching gem metadata from https://rubygems.org/
Fetching version metadata from https://rubygems.org/
Resolving dependencies...
Added to Gemfile:
  rack (= 1.6.4)
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile
source 'https://rubygems.org'

gem 'rack', '= 1.6.4'

But, so far, do you have any comments regarding the implementation?

Added some comments, I think source and group flags are also very useful for bundle add for specifying a source or group on the new gem line in the Gemfile.

How is this progressing?

@RochesterinNYC I've yet to add specs for --groups and --source, but it's pretty much working now.
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  touch Gemfile
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  dbundle add thin 1.0 --groups foobar barbaz --source file:///home/boggs/dev/bundler/tmp/gems/remote1
Fetching source index from file:/home/boggs/dev/bundler/tmp/gems/remote1/
Resolving dependencies...
Added to Gemfile:
  thin (~> 1.0)
╭─boggs@archbook ~/dev/4632-bundle-add-test ‹ruby-2.3.0› ‹node-v4.4.1›
╰─➤  cat Gemfile

gem 'thin', '~> 1.0', :group => [:foobar, :barbaz], :source => 'file:///home/boggs/dev/bundler/tmp/gems/remote1'

@RochesterinNYC I've added the appropriate specs. I cover too much on issues such as gem duplicates, etc. since inject_spec pretty much covers those kinds of issues.
As soon as the specs pass, I think this is good to go. Do you have any feedback or is there something you'd like me to add?

@RochesterinNYC This is odd - Travis seems to have stopped running specs for this PR.

@b-ggs that is odd, try to force push the PR to update it and see if that triggers a new build?

@b-ggs travis is running right now....

@segiddins @RochesterinNYC Yeah, it just showed up on my end. A few minutes ago it had only listed codeclimate as one of the checks, and was telling me that it had passed all 1 of 1 checks.

Great, I'll take another look at this and review later tonight.

Added a few more comments/small feedback points. This looks really good and about ready :+1:

@segiddins Oops, slight misunderstanding of the --pre flag. gem install cocoapods --pre gives the latest stable despite it not being a prerelease, since it's the latest available.
I've reverted back to how I was originally checking for prereleases, and have added the appropriate specs.

@segiddins @indirect can I have homu access on this PR?

@homu delegate=RochesterinNYC

:v: @RochesterinNYC can now approve this pull request

@homu r+

:pushpin: Commit ecd55e5 has been approved by RochesterinNYC

:hourglass: Testing commit ecd55e5 with merge 2439ac8...

:sunny: Test successful - status


Error: export 'EmailBuilderHistoryHostDirective' (imported as 'i2$1') was not found in '@wlocalhost/ngx-email-builder'

this is the error that happens when using i18n in angular 16...
"./node_modules/@wlocalhost/ngx-md-email-builder/fesm2022/wlocalhost-ngx-md-email-builder.mjs:3237:127-164 - Error: export 'EmailBuilderHistoryHostDirective' (imported as 'i2$1') was not found in '@wlocalhost/ngx-email-builder' (possible exports: AIPAlign, AIPBackground, AIPBorder, AIPColor, AIPEmailBody, AIPEmailBuilderAside, AIPEmailBuilderBlock, AIPEmailBuilderComponent, AIPEmailBuilderHistoryService, AIPEmailBuilderMiddlewareService, AIPEmailBuilderRestService, AIPEmailBuilderService, AIPEmailBuilderStorageService, AIPFon t, AIPImageUpload, AIPLineHeight, AIPLink, AIPMargin, AIPPadding, AIPStructure, AIPTemplateList, AIPValueChanged, AIPWidthHeight, ApplyMiddlewarePipe, ButtonBlock, ColumnStylesPipe, DefaultEmailBuilderHistoryService, DefaultIPEmailBuilderMiddlewareService, DefaultIPEmailBuilderRestService, DefaultIPEmailBuilderService, DividerBlock, HtmlBlock, IPE mail, IPEmailBuilderColumnDirective, IPEmailBuilderDynamicDirective, IPEmailBuilderHistoryActionDirective, IPEmailBuilderHistoryHostDirective, IPEmailBuilderSettingsDirective, IPEmailBuilderStructuresDirective, IPEmailBuilderTextEditorDirective, IPEmailBuilderTextEditorMenuDirective, IPHistoryModelDirective, ImageBlock, IpCanPipe, IpPreviewLinkPipe, NavigationBlock, NgxEmailBuilderModule, SocialBlock, SocialPathPipe, SpacerBlock, Structure, TemplateThumbPathPipe, TextBlock, ToBodyBlockPipe, ToBodyStructurePipe, ToHistoryOptionsPipe, addDefaultBlock, addNewIPEmailBuilderBlock, addToStore, cloneDeep, createBackground, createBorder, createFont, createLineHeight, createMargin, createPadding, createWidthHeight, debounce, defaultColumnsOptions, defaultsDeep, getFromStore, mergeObjects, randomString, withConfig)"

Hey @h-pires, I'm not sure I can reproduce it! what version of @wlocalhost/ngx-md-email-builder do you try to install?


[문서 수정] docs/computeds

Code change checklist

[ ] Added/updated unit tests
[ ] Updated /docs. For new functionality, at least API.md should be updated
[ ] Verified that there is no significant performance drop (npm run perf)

computeds 수정 사항 반영

@iChenLei How have you been? It's nothing to do with the PR, but I suddenly got curious. I want to know the statistics of the number of people entering mobx kr, is there a page that I can check? Personally, when I looked at the code, it seemed like you were using Google analytics!

kr.mobx.js.org copy/paste google analytics trackingId from mobx main repo , you can generate a trackingId your own and replace it, I think it's simple and don't need much time (Google Analytics is free). @kimyoungjae96

@iChenLei Thank you so much :)


Setting X-Forwarded-Host header

Summary
There may be other components in front of the gatekeeper that process the original request.
The step of adding headers on the gatekeeper sets the X-Forwarded-Host header without checking if it is present in the incoming request.
This causes backend services that use the X-Forwarded-Host header to not work.
Environment

Expected Results
The original value of the X-Forwarded-Host header must be preserved if it was passed.
Actual Results
GoGatekeeper rewrites the value of the X-Forwarded-Host header in an incoming request to the hostname where GoGatekeeper is installed.
Steps to reproduce

Additional Information
Source code:
	// @step: add the proxy forwarding headers
	req.Header.Set("X-Real-IP", utils.RealIP(req))
	if xff := req.Header.Get(constant.HeaderXForwardedFor); xff == "" {
		req.Header.Set("X-Forwarded-For", utils.RealIP(req))
	} else {
		req.Header.Set("X-Forwarded-For", xff)
	}
	req.Header.Set("X-Forwarded-Host", req.Host)
	req.Header.Set("X-Forwarded-Proto", req.Header.Get("X-Forwarded-Proto"))

UP:
problem in line
req.Header.Set("X-Forwarded-Host", req.Host)

@gt814 Hi thanks for opening issue! Actually as you can see it checks if header is present and sets it back to current value in else condition (this else branch is redundant in code) so nothing is overwritten

@p53 Hi
I totally agree that there is redundancy in handling the X-Forwarded-For header.
The main problem is in the line of code
req.Header.Set("X-Forwarded-Host", req.Host)
X-Forwarded-Host is a header to identify the original host requested by the client.
The host of the entry point and the host where the gatekeeper is installed can be different.

The problem can be fixed like this
if xfh := req.Header.Get("X-Forwarded-Host"); xfh == "" {
    req.Header.Set("X-Forwarded-Host", req.Host)
}

so that the X-Forwarded-Host header is only set if it has not been set previously.

:) ah ok sorry for misunderstanding didn't read your description properly you were refering to X-Forwarded-Host not X-Forwarded-For,

@gt814 hi created 3.0.1-rc1 release with fix, there is also docker image

@gt814 hi created 3.0.1-rc1 release with fix, there is also docker image

@p53 Thank you very much!
In the test environment, version 3.0.1-rc1 does not start.
An error occurs: exec /opt/gatekeeper/gatekeeper: exec format error.
I am trying to figure out what is causing the error.
I used version 2.9.4 before and it runs successfully.
Since version 2.9.5 an error appears.
I think the current ticket can be closed.
I can create a separate one for the problem with the application startup.

I figured out why it didn't launch.
I downloaded the image under the ARM architecture and pushed it to the local storage, but the test environment works under the AMD architecture.


[WIP] Added info modal in meshery

Notes for Reviewers
This PR fixes #8493
This PR adds new info modal in Pattern and Filters page to expose catalog data so that users can get information of design and filter and can edit at the same time if they are owner of that design or pattern
This information will be mostly same as when you publish a design, I'll add more screenshot after completing this
Signed commits

[ ] Yes, I signed my commits.

Task linked: CU-85zttdnfm [UI] Expose Catalog Request Data

@Yashsharma1911 still WIP or ready for review?

Let's discuss this on the Meshery dev call tomorrow,
Please add this as your agenda item in Docs: https://docs.google.com/document/d/1Dg4Uz9rGTuGHmYZwIt47z1j5x3LmnpxD1M1SgypAkQ0/edit

https://github.com/meshery/meshery/assets/71271069/6878c16b-407b-4b5e-b0db-05fc9157d536

@theBeginner86 thanks more making changes let's merge this if it is working

@theBeginner86 thanks more making changes let's merge this if it is working

yes, I was just tying this off. Would merge as soon as I'm done...
Thanks for confirming the status from your side 👍

We are ready now, @Yashsharma1911


Ingestion issues - duplicate data - strange results in PowerBI reports

🐛 Problem
I've deployed FinOps hub a few times. I think going from 0.2 to .0.3 RC, then 0.3 0.4 and now 0.5.
When I look at my reports I have some strange data that must be the result of bad imports into the storage account. Wondering if this is a hole I can dig myself out of - but really I could just wipe the entire setup and redeploy.
It would be nice to retain my original historical imports, as I think this data would be lost due to age?
👣 Repro steps
I have these cost exports which I created back in February with 1.0-preview(v1).

My main daily FinOpsHubMonthtodate export

msexports storage account

msexports/account/ - Duplicate data or ok?

Data factory triggers

Data factory linked services (deploy failed to clean up??)

Pipelines

🤔 Expected
Probably not worth the time digging this detail up for screenshots but I have very inaccurate cost details vs Azure Cost Analysis. Some months look ok, some look way too high.
My ingestion powerbi report had strange details, but now on 0.5 it doesn't work at all. Keeps all the 'demo data'.

@psilantropy You shouldn't see any duplication as of 0.4, which changed to use the manifest for identifying export scopes. To confirm, you'd need to look at the parquet files in the ingestion container. If you see any folders that duplicate other folders, then that will cause data duplication because the pipelines copy to a destination folder. I suspect the cases where you might be seeing that are from before 0.4 because of how we identified scopes based on the export path.
The easiest way to fix that is obviously to clear the ingestion container and re-export data for up to the last 7 years via the Start-FinOpsCostExport PowerShell command. That said, you shouldn't need to do that.
I'm not sure if we have a way to identify duplication cleanly, but we can definitely put some thought into it. I could see that going into the Data ingestion report to facilitate troubleshooting like this.

Hi @flanakin, thanks for the information and advice.

My inactive (single use exports) exports look ilke this;

My month to date export, untouched since Feb-24
Good feb

March ok with 2 files?

All folders up to August look like this with 2x parquet files.
Then a new folder structure as per the below;
August (again)

September

Honestly I don't care if I have to export again to get this clean. So delete these two folders?

Delete/stop my cost exports job? (finopshubmonthtodate)
Then run ?
Start-FinopsCostExport -Name 'CostExport' -Backfill 24

@psilantropy Those very short paths look suspicious, but it's hard to say without seeing the values. Let's setup some time so we can look at them together. Can you email ftk-support@ms so we can setup some time?

Happy to do that, but only if you gain something from reviewing it. More than happy to wipe it out.

Short answer: Keep "providers" and delete the other, then backfill 1 export. Don't forget to set the correct export name and specify the scope of the billing account in the command call:
Start-FinOpsCostExport -Name 'finopshubmonthtodate' -Scope '/providers/Microsoft.Billing/billingAccounts/###' -Backfill 24
Long answer...
I'm assuming you only have one billing account. If that's true, then you should only see one path to parquet files in the ingestion container:
providers/microsoft.billing/billingaccounts/###/focuscost/yyyyMM/*.parquet
If you have any others, check the months to make sure you don't have duplicate months. If you do, delete the other folder and keep the one at the path above.
The one other thing I'm noticing in your files that seems odd is that you seem to have one parquet file created on the first and another created later in the month. That seems suspicious. Without confirming, I almost wonder if there's a different file naming pattern for daily vs. monthly exports, which could be causing 2 versions of the file to be created. Hubs overwrites files based on name and would miss this case if the same file had a new name, which would definitely result in duplicated files. But that's just a guess at this point...

I just wiped it all. Had some issues importing after the delete.
Ingestion report looks ok now. Folder structure clean.
CostSummary report looks good.
Commitment reports has a null value error, but i'll review and log after reviewing the cost summary in detail.
Appreciate your assistance.


Feature request - Translatable fields

Hello,
Do you plan to implement translatable fields with a package such as spatie/laravel-translatable ?
Thanks
System GA

Hi @InfinityWebMe,
thanks for the question.
We will take this issue in the planning of the next sprint.
Best,
Yevhen


Commit-status-and-panel

Fixes

https://issues.redhat.com/browse/HACBS-1136
Description

Adds status and task side panel to commit details
Type of change

[x] Feature
[ ] Bugfix
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

Screen shots / Gifs for design review

How to test or reproduce?

Added unit tests for commit-utils, CommitSidePanelHeader, CommitSidePanelbody
Browser conformance:

[x] Chrome
[x] Firefox
[ ] Safari
[ ] Edge

@abhinandan13jan looks very cool so far!
Few comments:

Let's move the GitHub icon and the status label after the page title. I think @christianvogt mentioned it is doable now
Did we include a "read more" after a long commit comment? I think we set up a max character of 72 and "read more" will just display the rest.
On the side panel:
I'm not sure what is "go-3-on-push..." is that a build pipeline placeholder?
We should have a side panel title, displaying the full name of the node
Below that, we should have an icon and generic name of the node type (in grey, body weight)
For the status below, we should start that string with the corresponding icon (green/red/yellow/blue)

Thank you!

@abhinandan13jan looks very cool so far!
Few comments:

Let's move the GitHub icon and the status label after the page title. I think @christianvogt mentioned it is doable now
Did we include a "read more" after a long commit comment? I think we set up a max character of 72 and "read more" will just display the rest.
On the side panel:
I'm not sure what is "go-3-on-push..." is that a build pipeline placeholder?
We should have a side panel title, displaying the full name of the node
Below that, we should have an icon and generic name of the node type (in grey, body weight)
For the status below, we should start that string with the corresponding icon (green/red/yellow/blue)

Thank you!

Moved the GH icon to header
Yes, I updated to the read more, but this string is only 56 char long so it doesn't show. However, for demo I've set the read more length to 45 in screenshot
Go-random-number is the actual kubernetes object name for the pipeline. I've updated to use logical name "build/Release/Integration Test" , Buld in this case instead. We currently don't have the logical mapping/definitions in place with the real object(kubernetes PipelineRun CR) vs the display texts, which will be taken care of once @karthikjeeyar data model for the workflow visualization is in

Updated PR

Modularised CommitSidePanel into Two new components CommitSidePanelHeader and CommitSidePanelBody
Updated StatusIconWithText to use Label instead of Badge
Added test suites for CommitSidePanelHeader, CommitSidePanelBody & commit-utils
Moved initialStatus as commitStatus inside useEffect

/retest

/retest

updated @christianvogt @karthikjeeyar

/lgtm

/approve

/retest


Add Spliddit to Fix Emojis [Resolves #94]

Implements the spliddit (https://github.com/essdot/spliddit) function to resolve emoji issues with jquery.lettering. Also modifies the example to include the spliddit library and make the appropriate attributions.

Hey, thanks for this PR!
If possible I'd prefer that splitddit be optional to minimize the amount of dependencies. We can include a section in the readme for emoji support.

Please do whatever you want with it. Had the problem, made the solution, shared it with you since there was a related issue open.


Rule 'array-bracket-even-spacing' does not seem to be working anymore

Since I upgrade my packages, Eslint is complaining about my brackets.:Here are the (most relevant?) packages I'm using:
"babel-eslint": "^10.0.1",
"eslint": "^5.15.1",
"eslint-config-standard": "^12.0.0",
"eslint-loader": "^2.1.1",
"eslint-plugin-standard": "^4.0.0",

And this one the error I get everywhere:
error  A space is required after '{'   object-curly-spacing

It seems like the rule is working as intended?
Also, we have moved away from this rule in standard 13, so if you upgrade to this version then your errors should go away because we don't even use this rule anymore


Tweaks to data formats

This PR is the Rust counterpart to automerge/automerge#296. The individual commits on this PR link to their counterparts in the JS implementation.
Due to my poor Rust skills I couldn't figure out how to write the Rust counterpart for automerge/automerge@6fa8b5a5b9bbfb178750c629fc6107b30b9c35a3, so that change is missing from this PR. Could one of you please help with that? If you check out the predecessor commit automerge/automerge@7f2e9f4897a420ca6ebd36421e935dc09484d1d4 on the JS repo, and compile d320ba8b23fc7e0aa2501f4ff98feffcd32d4e98 on this repo, the JS-Wasm tests should be passing, so this PR should only be missing the Rust counterpart to automerge/automerge@6fa8b5a5b9bbfb178750c629fc6107b30b9c35a3.
Commit d320ba8b23fc7e0aa2501f4ff98feffcd32d4e98 is incomplete: although yarn release is able to build the wasm, I have not finished updating the Rust tests. I'm not sure what the status of the Rust tests is: running cargo test on the current HEAD 83145b82c49809aaccf7e6463e164de59225045d does not even compile, let alone run the tests. Maybe we should first fix those tests, and then finish the changes in d320ba8b23fc7e0aa2501f4ff98feffcd32d4e98.
@orionz @alexjg help please :)

This looks straightforward to integrate. The failing tests are to do with the removal of the ChangeRequest type, which has broken the Rust frontend. I should have a fix for that merged today.

This looks straightforward to integrate. The failing tests are to do with the removal of the ChangeRequest type, which has broken the Rust frontend. I should have a fix for that merged today.

Don't feel bad @ept , the encode/decode code is some of the hardest to read in the whole codebase since I really prioritized zero-copy semantics over code readability.  I'm experiencing the same thing now as I implement the whole document compression... :/  I'll get this sorted.

Don't feel bad @ept , the encode/decode code is some of the hardest to read in the whole codebase since I really prioritized zero-copy semantics over code readability.  I'm experiencing the same thing now as I implement the whole document compression... :/  I'll get this sorted.


[fix](docker case) Fix test_sql_mode_node_mgr and add cloud multi f…

…ollower fuzzy test
What problem does this PR solve?

fix start docker cluster slow, double cycle use one break, can't break out
Fix the test_sql_mode_node_mgr case that fails to meet testing objectives, as the container exits after dropping the follower.
Add fuzzy add/drop follower/observer cases on the cloud.
Fix docker compose error java.lang.Exception: doris compose output is empty, err:
Fix show frontends slow query when change fe nodes, due to show frontends use is isJoin function call getHostName(https://bugs.openjdk.org/browse/JDK-8143378#:~:text=getHostName()%3B takes about 5,millisecond on JDK update 51.)

Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

run buildall

run buildall


Do not use PLINQ in CompositeGlob

Threadpools are a scarce resource in VS, do not use them.
We should instead consider other optimisations: #2044

@lifengl Look good?

looks good to me.  In the future, maybe composition glob can merge regular expressions...

From: Andy Gerlicher<EMAIL_ADDRESS>Sent: Tuesday, May 2, 2017 1:49 PM
To: Microsoft/msbuild
Cc: Lifeng Lu; Mention
Subject: Re: [Microsoft/msbuild] Do not use PLINQ in CompositeGlob (#2045)
@AndyGerlicher approved this pull request.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/msbuild/pull/2045#pullrequestreview-35886182, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ALGWwkw4sYnekPskdXjnH3oP7JdcNYHEks5r15a9gaJpZM4NOkai.


fix several issues when building a docker image

/lgtm

/lgtm

there are two no empty files in the pr. please remove it and then it will be merged. thanks


Different benchmark results in the board vs. detailed view

Hi all,
When I look at my submissions in the profile, I get an average score of 0.352 as in the image:

But when I click on the model to see the details, I get different values:

Any ideas why this is happening and which score I should trust?
Many thanks,
Amin

Hi @heydarshahi,
The table in your profile also shows scores on benchmarks that are not (yet) public, for instance MajajHong2015.IT-rdm in IT.
The model view only shows scores on benchmarks that are public, notice there are only 5 benchmarks in IT rather than 7 in your profile.
Sorry for this confusion, it's on our list of things to fix and make more intuitive.

I see. Thanks for the clarification :)


Amplify init truncates profile names after the first occurrence of a "."

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v14.9.3
Amplify CLI Version
8.3.1
What operating system are you using?
Ubuntu 22.04 (via WSL2 on Windows 10)
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes mad
Amplify Categories
Not applicable
Amplify Commands
init
Describe the bug
Profiles that contain a dot are truncated before the dot.
Expected behavior
The list of profiles should look like:
? Please choose the profile you want to use (Use arrow keys)
❯ sandbox-dan.hook
Reproduction steps
In ~/.aws/config Create a profile with a name that contains a dot:
[profile sandbox-dan.hook]
....
Run  amplify init and answer the questions in the following way:
Note: It is recommended to run this command from the root of your app directory
? Do you want to use an existing environment? No
? Enter a name for the environment dhook
? Choose your default editor: Visual Studio Code
Using default provider  awscloudformation
? Select the authentication method you want to use: AWS profile
The list of profiles will look like:
? Please choose the profile you want to use (Use arrow keys)
❯ sandbox-dan
Selecting it results in
"Failed to get profile: Cannot read property 'accessKeyId' of undefined"
GraphQL schema(s)
No response
Log output

# Put your logs below this line

Additional information
No response

This appears to be a limitation of the .ini parser that's being used to read the profiles. When parsing .ini files, the "." must be escaped, here is the issue reported on the .ini parser's github page: https://github.com/npm/ini/issues/60.


Unable to respond normally when using MemoryStore Postgres.

Describe the bug
I used ./deploy-azure.ps1 to deploy Chat Copilot to Azure, using the parameter -MemoryStore Postgres. The deployment process went smoothly. However, the following error was reported when I started using it:
Unable to generate bot response. Details: Error: 400: Bad Request => The CLR type Pgvector.Vector isn't natively supported by Npgsql or your PostgreSQL. To use it with a PostgreSQL composite you need to specify DataTypeName or to map it, please refer to the documentation.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Platform
Additional context

Thank you for reporting this.  Pgvector is an extension for Postres:  https://github.com/pgvector/pgvector
Did you install and configure this extension?

Thank you for the reminder.  I referred to this Microsoft document https://learn.microsoft.com/en-us/azure/cosmos-db/postgresql/howto-use-pgvector and ran the following commands, which seem to have resolved the issue.
SELECT CREATE_EXTENSION('vector');
I suggest that this situation should be mentioned in the deployment documentation.
However, I have encountered a new error, which seems to be the same issue as #215
Unable to generate bot response. Details: Error: 400: Bad Request => 42P01: relation "public.ce001975-756a-4dc5-9c09-bdfa5a8146b6-LongTermMemory" does not exist POSITION: 115

Yes, I think this may be related: https://github.com/microsoft/chat-copilot/issues/215

Yes, I think this may be related: #215

same

I checked and the main connector readme does have a pointer to the extension info:
https://github.com/microsoft/semantic-kernel/tree/main/dotnet/src/Connectors/Connectors.Memory.Postgres
I've created an issue to update our own documentation to refer to Postgres (currently only ACS and Qdrant):
https://github.com/microsoft/chat-copilot/issues/237


Fix undefined error in server tree data source

Fixes https://github.com/microsoft/azuredatastudio/issues/12615
This was "broken" in https://github.com/microsoft/azuredatastudio/commit/503090856a291232cc564e397d258e7a2ae03f4f#diff-7159ef3349b8a0d975ddfc700bd99a12
We aren't really using the old tree control correctly (it doesn't really support passing undefined as an item) but refactoring it to be correct is probably not worth it given that this will be replaced with the async server tree at some point anyways.
Using an empty string as the id here should be safe since it'll be the only item in the tree so no chance of collisions.

Pull Request Test Coverage Report for Build 271062664

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 38.081%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/objectExplorer/browser/serverTreeDataSource.ts
0
1
0.0%

Totals

Change from base Build 271004996:
-0.003%

Covered Lines:
19144

Relevant Lines:
45868

💛  - Coveralls


(maint) Update for Rubocop 0.77.0

This commit updates the project for Rubocop 0.77.0
Due to there being so many differences in rubocop for Ruby < 2.3.0, this commit
also stops bundling rubocop for old Ruby versions.

Self merging as master is broken.


Export map as svg?

Is it possible to export maps/layers as svg file?

Yes, but not using built-in functionality. There have been a few examples of this using other libraries, including the https://github.com/hanbyul-here/svg-exporter repo, but I'm not sure if any of them are still working. :/
But that's where I'd start – good luck!
(For other questions feel free to drop by our gitter channel: https://gitter.im/tangrams/tangram-chat)

Note that the SVG example linked is geared towards the underlying Tilezen vector tiles, but not Tangram specifically, so wouldn't include any Tangram styling, etc. Tangram renders with WebGL, which has a very different method of expressing geometry and styling compared to SVG, and unfortunately the two aren't easily compatible.


moer-mpg-data-analysis_unit08

Hello,
I have some questions about this unit:
What are knots? Are GAMs the same as LOESS, with only the difference, that the points are not weighted? Why do we set px?
Best regards, Kathi

When tuning the model I get the Error:
Error in gam(dep ~ s(ind, k = knots[a], fx = TRUE), data = df) :
Not enough (non-NA) data to do anything meaningful
Does anybody else have this problem?
Best regards

When tuning the model I get the Error:
Error in gam(dep ~ s(ind, k = knots[a], fx = TRUE), data = df) :
Not enough (non-NA) data to do anything meaningful
Does anybody else have this problem?
Best regards


Textfield text not visible on flutter web

TextField text color defaults to transparent when used in flutter web.
Same applies to TextFormField. The cursor doesn't move and text typed is not visible.
Looking through chrome developer tools (Version 76.0.3809.132 (Official Build) (64-bit)), the text
defaults to transparent despite efforts to change its color through vscode. On unchecking the transparent box on chrome developer tools, the text becomes visible but cursor doesn't move from starting point. I have also tried setting enabled to true among other code settings but it does not work.
Steps to Reproduce
'''
import 'package:flutter_web/material.dart';
void main() => runApp(MyApp());
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
title: 'Flutter Demo',
theme: ThemeData(
primarySwatch: Colors.blue,
),
home: MyHomePage(title: 'Flutter Demo Home Page'),
);
}
}
class MyHomePage extends StatelessWidget {
MyHomePage({Key key, this.title}) : super(key: key);
final String title;
@override
Widget build(BuildContext context) {
return Scaffold(
appBar: AppBar(
title: Text(title),
),
body: Center(
child: Column(
mainAxisAlignment: MainAxisAlignment.center,
children: [
Text(
'Hello, World!',
),
ListTile(
title: Text('Main header'),
subtitle: TextField(
style: TextStyle(color: Colors.black),
enabled: true,
),
),
],
),
),
);
}
}
'''

##Flutter Analyze:
flutter analyze
No issues found! (ran in 6.8s)
##Flutter Doctor
! Doctor found issues in 1 category.
PS C:...herro> flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.7.8+hotfix.4, on Microsoft
Windows [Version 10.0.18362.295], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK    version 29.0.2)
[√] Android Studio (version 3.5)
[√] VS Code, 64-bit edition (version 1.38.0)
[!] Connected device
! No devices available

Please do not include screenshots of code. This makes it extremely tedious to reproduce the issue, as we would have to manually transcribe your example.

Sorry for that, it's just the basic 'hello world' demo web app with the Textfield as a ListTile's child. I have edited the file to include the code above.

I can't reproduce on 5a3a46ada2417957415c58a5a1adff0fc11f7fc5.
Please try the unforked workflow at https://github.com/flutter/flutter/wiki/Building-a-web-application-with-Flutter  and see if you still have the issue

I can't seem to run my project with the unforked webflow at the moment.

Thanks. Will do.

Any solution for people that can't run the unforked webflow?

@xeu100 A fix for this Issue is provided in PR https://github.com/flutter/flutter_web/pull/125 :-)

Due to some limitations, I also can't use the unforked work flow at the moment

I can't reproduce on 5a3a46a.
Please try the unforked workflow at https://github.com/flutter/flutter/wiki/Building-a-web-application-with-Flutter and see if you still have the issue

Due to some limitations, I also can't use the unforked work flow at the moment.

The same error, there is invisible text in text flutter form. It is web-app, I have tested it in firefox 69.0 and chrome 76.0.3809.132 . I tried use TextFormField from flutter Developoers guide(https://api.flutter.dev/flutter/widgets/Form-class.html) and tried reinstall flutter and also generate new project by stagehand but nothing happened.
Its look like a invisible font - I pressing the key, on the screen nothind display. But if I will select  - that text suddenly appear! and if I make that region  copy to other text window(other application, like Notepad), text also became visible.
Flutter 1.9.1+hotfix.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 2d2a1ffec9 (3 days ago) • 2019-09-06 18:39:49 -0700
Engine • revision b863200c37
Tools • Dart 2.5.0

I am using https://flutter.dev/docs/get-started/web with the version 1.9 is working all well :)

I'm facing the same issue as @alex89607

The unforked workflow works. Thanks for the pointers.

Thanks for the response. I'm closing the issue.


Feature/cdk helper

Implement CDK-Helper Deployment.
The solution is now cdk-deploy native and uses CDK-Helper to grab the packaged lambda code when deployed via Amazon internal pipelines to support 1-click deploy.
All unit tests pass, and both cdk-deploy and internal-pipeline deploy have been verified as deploying successfully with all lambda contents present
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

will be merged in another PR that fixes a build issue


Permissions not working: "Environment is unreachable"

Bug description
I installed Portainer CE 2.18.2 in my Kubernetes cluster. As soon as I create a non-administrator user, this user can't access the environment:

However, if I assign this user the administrator privilege it works.
Expected behavior
I should be able to access the environment with this non-admin user.
That's my user:

That's the environment:

Those are the environment access settings:

Portainer Logs
2023/05/09 07:37AM INF portainer/main.go:494 > encryption key file not present | filename=portainer
2023/05/09 07:37AM INF portainer/main.go:517 > proceeding without encryption key
2023/05/09 07:37AM INF boltdb/db.go:124 > loading PortainerDB | filename=portainer.db
2023/05/09 07:37AM INF ssl/ssl.go:80 > no cert files found, generating self signed SSL certificates
2023/05/09 07:37:21 server: Reverse tunnelling enabled
2023/05/09 07:37:21 server: Fingerprint ad:7b:cc:ac:1d:5d:b5:ba:64:f8:82:4f:b1:9b:ef:32
2023/05/09 07:37:21 server: Listening on <IP_ADDRESS>:8000...
2023/05/09 07:37AM INF portainer/main.go:765 > starting Portainer | build_number=30140 go_version=1.19.4 image_tag=linux-amd64-2.18.2 nodejs_version=18.16.0 version=2.18.2 webpack_version=5.68.0 yarn_version=1.22.19
2023/05/09 07:37AM INF http/server.go:342 > starting HTTPS server | bind_address=:9443
2023/05/09 07:39AM INF endpointutils/endpointutils.go:186 > attempting to detect storage classes in the cluster
2023/05/09 09:40AM http: proxy error: namespaces "portainer" not found
2023/05/09 09:41AM http: proxy error: namespaces "portainer" not found
2023/05/09 09:41AM http: proxy error: namespaces "portainer" not found
2023/05/09 09:41AM http: proxy error: namespaces "portainer" not found
2023/05/09 09:47AM http: proxy error: namespaces "portainer" not found
2023/05/09 07:47AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:47AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:47AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 09:47AM http: proxy error: namespaces "portainer" not found
2023/05/09 07:47AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:51AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:52AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:53AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 09:53AM http: proxy error: namespaces "portainer" not found
2023/05/09 07:53AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:53AM ERR client/client.go:94 > unexpected status code | status_code=403
2023/05/09 07:53AM ERR client/client.go:94 > unexpected status code | status_code=403

Technical details:

Portainer version: 2.18.2
Kubernetes version (managed by Portainer): v1.22.5
Platform (windows/linux): linux
Browser: Google Chrome
Use Case (delete as appropriate): Using Portainer at Home
Have you reviewed our technical documentation and knowledge base? Yes

@pieveee
Thank you for the information.  I am going to investigate.  I will update you as I learn more.
Thanks!

@pieveee
I wanted to follow up on this request.  I am unable to reproduce the issue.
https://github.com/portainer/portainer/assets/20426210/4661df57-d69c-4c27-9ce4-7d2b83bad7dc
Can you post how you installed portainer?  I am seeing in the logs http: proxy error: namespaces "portainer" not found
Thanks!

@tamarahenson
I can confirm, that installing portainer in a different namespace than portainer causes this issue. I think we should consider this as a bug, as the namespace should be independent of running portainer in a Kubernetes cluster.

@pieveee
Thank you for the additional information.  I am currently investigating.  I will update you as I learn more.
Thanks!

@pieveee
Upon further review, we are not supporting installing Portainer into a namespace other than portainer
Thanks!

@tamarahenson Thank you for the information. Is there any intention to support installing Portainer in other namespaces? Depending on the predefined architectural structure, some organisations could require this.

@pieveee
We have it documented here:  https://docs.portainer.io/start/install-ce/server/kubernetes/baremetal
"You will be using the portainer namespace for Portainer. At present this is a requirement - other namespaces are currently unsupported"
Thanks!

@ Ah all right. I hope other namespaces will be supported in the near future. However, I am going to close this issue for now. Thank you for your support!


Bug: profile icon is not in the correct position

Improve the Css to maintain icon position.
Expected behavior
Icon should be placed in the good position.
Screenshots

@Itz-Manu fix this!!


[mvp] Add check of activity state before getting instance of holder fragment in onDestroy()

HolderFragment.getInstance(activity/fragment) is wrapped into try/catch because isDestroyed() was added in 17 API.


Allow autocomplete and maxlength attributes for more elements

The following attributes are being ignored by these element view helpers and they should be allowed by adding them to $validTagAttributes:

autocomplete

Zend\Form\View\Helper\FormTextarea
Zend\Form\View\Helper\FormSelect

maxlength

Zend\Form\Element\Number

Firefox pre-fills textarea and select elements too and we need a way to turn that off.
We may need to start this issue together with issue #6464 which is also related to the autocomplete attribute.

@Ocramius this issue can be closed ? #6854 was merged and closed :-)


Handle cases when rel param is path with colons

Okay I ended up doing my own simpler fix in 6d4657be003030711e14835f750df81fc86d95d1 for the bug, although I used your test cases to do it (and merged in your cases.json so you at least get the commit credit for it). Thanks!

Thats ok for me. Could you make pr in rss-bridge to update src/urljoin.php for fixing https://github.com/RSS-Bridge/rss-bridge/issues/866 ?

Sure, although I had thought you were pulling my library in as an upstream dependency via composer or whatever. But I'll take care of that.


Remove unused dependencies for velox_hive_iceberg_splitreader library

Duplicate of https://github.com/facebookincubator/velox/pull/8808/files


Can't load a binary on OS X 10.10.3

Trying to use gdb 'load', with a pyocdgdbserver (the target is a K64F board):
> load
Loading section .isr_vector, size 0x198 lma 0x0
Loading section .flash_protect, size 0x10 lma 0x400
Loading section .text, size 0x2b8e8 lma 0x420
Loading section .ARM.extab, size 0xdc lma 0x2bd08
Loading section .ARM.exidx, size 0x1b0 lma 0x2bde4
Loading section .data, size 0xa60 lma 0x2bf94
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.9/Frameworks/Python.framework/Versions/2.7/lib/python2.7/threading.py", line 810, in __bootstrap_inner
    self.run()
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 153, in run
    [resp, ack, detach] = self.handleMsg(data)
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 220, in handleMsg
    return self.flashOp(msg[2:]), 1, 0
  File "/usr/local/lib/python2.7/site-packages/pyOCD/gdbserver/gdbserver.py", line 380, in flashOp
    self.flash.init()
  File "/usr/local/lib/python2.7/site-packages/pyOCD/flash/flash.py", line 45, in init
    self.target.writeBlockMemoryAligned32(self.flash_algo['load_address'], self.flash_algo['instructions'])
  File "/usr/local/lib/python2.7/site-packages/pyOCD/target/cortex_m.py", line 646, in writeBlockMemoryAligned32
    self.transport.writeBlock32(addr, data[:n/4])
  File "/usr/local/lib/python2.7/site-packages/pyOCD/transport/cmsis_dap.py", line 191, in writeBlock32
    dapTransferBlock(self.interface, len(data), WRITE | AP_ACC | AP_REG['DRW'], data)
  File "/usr/local/lib/python2.7/site-packages/pyOCD/transport/cmsis_dap_core.py", line 227, in dapTransferBlock
    max_pending_reads = interface.getPacketCount()
  File "/usr/local/lib/python2.7/site-packages/pyOCD/interface/interface.py", line 48, in getPacketCount
    return self.packet_count
AttributeError: 'HidApiUSB' object has no attribute 'packet_count'

@c1728p9 This was introduced in the commit d2b5c06d4cde08c2ed8a11

Pull request #121 should fix this

Thanks @c1728p9 for looking at this so quickly :+1:


Tensorflow with ROCm support won't see my GPU

System information

Ubuntu 18.10
TensorFlow installed from source
TensorFlow version: 2.0
Python version: 3.6
Installed in virtualenv with pip
Bazel version: 0.23.0
GCC/Compiler version: 8.2.0
ROCm/MIOpen version: not sure. should be latest with respect to this post
GPU model and memory: Radeon Rx 580 and 16 GB of RAM

So, I successfully built tensorflow to have ROCm support, I took a simple keras model from a tutorial online to see if I could get it to use the GPU and instead I noticed it used the CPU. I am not sure if I need to install drivers for it or not. I installed the ROCm libraries using the experimental build since I am running Ubuntu 18.10.
When I used ./configure I made sure to say yes to ROCm support. I also said yes to XLA JIT support.
I compiled Tensorflow with this command:
bazel build --config=opt --config=rocm //tensorflow/tools/pip_package:build_pip_package
This is what I get when I run the model. It only shows CPU stats. I have used Tensorflow with an Nvidia graphics card in the past and it normally will check for the GPU and it if doesn't see it it will default to the CPU. I am not sure if the ROCm version will do that. I would like to think that it will.

This is what happens when I run the Session() function that Tensorflow provides. It is supposed to show all mapped devices and the card doesn't seem to show up but my CPU sure does.

I am not sure if I built it properly or not. Is there something that I could be missing?

Hi @Shad0wSt4R , Ubuntu18.10 is not in ROCm2.2 support list, doc is here:
https://github.com/RadeonOpenCompute/ROCm#supported-operating-systems---new-operating-systems-available
The error you've observed might be caused by the base driver configuration.
If you don't have plan to change your OS, you might try to upgrade your base kernel to 4.20 or newer (please review this doc) and use our public docker image, e.g. the following one:
rocm/tensorflow:rocm2.2-tf1.13-python3


Add noise and pattern properties on Fill

fix #228

Warnings

:warning:

Source/dom/style/Border.js changed, but not:

📚 its docs
🧪 its tests

That's OK as long as you're refactoring.

:warning:

Source/dom/style/Fill.js changed, but not:

📚 its docs

That's OK as long as you're refactoring.

:warning:

Source/dom/style/Shadow.js changed, but not:

📚 its docs
🧪 its tests

That's OK as long as you're refactoring.

:warning:

Source/dom/style/Style.js changed, but not:

🧪 its tests

That's OK as long as you're refactoring.

  Generated by :no_entry_sign: dangerJS


How to preserve weight after power restart?

I put 1kg of weight on the scale, after a power restart the weight resets to 0 and should show 1kg.
Scale should show the current weight after power reset in my examle 1kg..
Any idea how to solve the issue?

When you restart your device, you are using Tare function again.
When you use Tare, the MCU waits fro stabilization time (default 400ms afaik) and then does a reading.
Then that reading is accepted as "0" point, this is what you are experiencing.
I myself never tried without Tare but you can disable tare, enter a known calibration factor and that way you can prevent this.
The possible issue with this approach would be, readings change slightly with environmental changes, so you might lose accuracy.
Try and see if this is okay for you.

There is an approach to this in the library example file Persistent_zero_offset.ino, with functions for saving and restoring the tare/zero offset value using the internal EEPROM. Basically you just send 't' from the Serial terminal to perform tare function and save the offset to EEPROM. The offset value will be restored from EEPROM at next boot.

Thanks a lot, will try it...


FakeCUDAAtomic Add, Min and Max methods do not return the old value

FakeCUDAAtomic Add, Min and Max methods do not return the old values, as expected:
class FakeCUDAAtomic(object):
    def add(self, array, index, val):
        with addlock:
            array[index] += val

Instead these methods always return None.  Only compare_and_swap works as expected.

Hi i want to solve this bug . As this is my first issue , can u please give me some guidelines?


/back location is lost on server restart

Would it be possible to save the location to disk so that the /back location is persistent?

theoretically, yes, with some minor code changes, but i really don't see it being practical...

Why not? In most Spigot/Paper servers I've been in with Essentials, people tend to rely on /back as a persistent temporary warp point and it often ends up quite useful in that regard.

wait, so how do you mean persistent? i though persistent meant that it persists across server restarts...

Yes, that is what I mean. Especially in servers that restart often, it's convenient to know that you won't lose your /back.

alright, i'll consider making it persistent


chore(deps): update all non-major dependencies

This PR contains the following updates:

Package
Type
Update
Change

@antfu/eslint-config
devDependencies
patch
^2.8.0 -> ^2.8.3

@commitlint/cli (source)
devDependencies
minor
^19.0.3 -> ^19.2.0

@commitlint/config-conventional (source)
devDependencies
minor
^19.0.3 -> ^19.1.0

@types/node (source)
devDependencies
patch
^20.11.25 -> ^20.11.28

@vitest/coverage-v8 (source)
devDependencies
minor
^1.3.1 -> ^1.4.0

vitest (source)
devDependencies
minor
^1.3.1 -> ^1.4.0

Release Notes

antfu/eslint-config (@antfu/eslint-config)
v2.8.3
Compare Source
🚀 Features

Improve cli  -  by @injurka and @antfu in https://github.com/antfu/eslint-config/issues/420 (bc13b)

View changes on GitHub
v2.8.2
Compare Source
🐞 Bug Fixes

Cli-suggest-remove-files  -  by @injurka in https://github.com/antfu/eslint-config/issues/418 (30229)
Ignore .yarn folder  -  by @HigherOrderLogic in https://github.com/antfu/eslint-config/issues/421 (d41a5)
import/newline-after-import: Disable considerComment option  -  by @antfu (80f8e)

View changes on GitHub
v2.8.1
Compare Source
🚀 Features

Upgrade vue and ts plugins  -  by @antfu (e36ed)

View changes on GitHub

conventional-changelog/commitlint (@commitlint/cli)
v19.2.0
Compare Source
Features

cli: introduce new --last flag, to stop recommending HEAD~1 (#3916) (99f4f3f)

v19.1.0
Compare Source
Note: Version bump only for package @commitlint/cli
19.0.3 (2024-02-28)
Note: Version bump only for package @commitlint/cli
19.0.2 (2024-02-28)
Note: Version bump only for package @commitlint/cli
19.0.1 (2024-02-27)
Bug Fixes

drop resolve-from, resolve-global and import-fresh, resolve global packages correctly (#3939) (8793c63), closes #3938

conventional-changelog/commitlint (@commitlint/config-conventional)
v19.1.0
Compare Source
Note: Version bump only for package @commitlint/config-conventional
19.0.3 (2024-02-28)
Note: Version bump only for package @commitlint/config-conventional

vitest-dev/vitest (@vitest/coverage-v8)
v1.4.0
Compare Source
🚀 Features

Throw error when using snapshot assertion with not  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5294 (b9d37)
Add a flag to include test location in tasks  -  by @sheremet-va in https://github.com/vitest-dev/vitest/issues/5342 (d627e)
cli:

Support wildcards in --project option  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5295 (201bd)

config:

Add shuffle.files and shuffle.tests options  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5281 (356db)
Deprecate cache.dir option  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5229 (d7e8b)

coverage:

Support --changed option  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5314 (600b4)

vitest:

Support clearScreen cli flag  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5241 (e1735)

🐞 Bug Fixes

Repeatable --project option  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5265 (d1a06)
--inspect-brk to pause before execution  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5355 (e77c5)
Correct locations in test.each tasks  -  by @sheremet-va (4f6e3)
api:

Use resolvedUrls from devserver  -  by @saitonakamura and @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5289 (2fef5)

browser:

Add magic-string to optimizeDeps.include  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5278 (8f04e)

coverage:

Expensive regexp hangs v8 report generation  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5259 (d68a7)
V8 to ignore type-only files  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5328 (c3eb8)
Respect source maps of pre-transpiled sources  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5367 (6eda4)
Prevent reportsDirectory from removing user's project  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5376 (07ec3)

expect:

Show diff on toContain/toMatch assertion error  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5267 (8ee59)

forks:

Wrap defines to support undefined values  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5284 (5b58b)

typecheck:

Update get-tsconfig 4.7.3 to fix false circularity error  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5384 (bdc37)

ui:

Escape html in error diff  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5325 (ab60b)

vitest:

Loosen onConsoleLog return type  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5337 (6d1b1)
Ensure restoring terminal cursor on close  -  by @hi-ogawa in https://github.com/vitest-dev/vitest/issues/5292 (0bea2)
Ignore timeout on websocket reporter rpc  -  by @sheremet-va (38119)
Correctly override api with --no-api flag  -  by @sheremet-va in https://github.com/vitest-dev/vitest/issues/5386 (51d1d)
Logs in beforeAll and afterAll  -  by @fenghan34 in https://github.com/vitest-dev/vitest/issues/5288 (ce5ca)

workspace:

Throw error when browser mode and @vitest/coverage-v8 are used  -  by @AriPerkkio in https://github.com/vitest-dev/vitest/issues/5250 (29f98)

View changes on GitHub

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Hywax Assistant.

👋 @hywax-assistant
💖 Thanks for opening this pull request!
Please follow the contributing guidelines. And we use semantic commit messages to streamline the release process.
Examples of commit messages with semantic prefixes:

fix: interval for sending a request in the base service
feat: add new NAME service
docs: example of using the NAME service

👋 @hywax-assistant Congrats on merging your first pull request! 🎉


Add code quality checking tools / linters to Travis

As suggested by @mnquintana, to prepare the app for more open contributions (and to clean up some of our mess), we should add code quality tools like rubocop and other linters to Travis to make sure we're writing good code. It will also give us ideas for refactoring / improvement.

I think as part of that we're gonna have to implement some code quality tools and make sure we run them on travis to enforce code style and stuff (ie. rubocop, jscs, jshint, any other linters I can't think of)

I'm tweaking the rubocop settings to be more permissive to account for our current code state (e.g. increasing the max method line length, etc) and we've merged in the JS linters. I'd like to merge all of the open PR's into master and merge that into here before making any changes to the code itself (to avoid conflicts), but once that's done we can put these to use.
In terms of the JS linters, since they won't be passing unless we fix our JS code, do we want to maybe comment them out of the build to avoid failing builds for now? Otherwise I think the GitHub - Travis integration will be a little useless. Thoughts?

I'm tweaking the rubocop settings to be more permissive to account for our current code state (e.g. increasing the max method line length, etc) and we've merged in the JS linters. I'd like to merge all of the open PR's into master and merge that into here before making any changes to the code itself (to avoid conflicts), but once that's done we can put these to use.
In terms of the JS linters, since they won't be passing unless we fix our JS code, do we want to maybe comment them out of the build to avoid failing builds for now? Otherwise I think the GitHub - Travis integration will be a little useless. Thoughts?

We could definitely comment them out for now (the JS linters) – we'll just have to remember to uncomment them once we're ready again. :)

We could definitely comment them out for now (the JS linters) – we'll just have to remember to uncomment them once we're ready again. :)

Sounds good, I'll add a comment to #1061 as we can bundle in a general refactoring of the JS code in there as well.

Sounds good, I'll add a comment to #1061 as we can bundle in a general refactoring of the JS code in there as well.

Ok, I didn't follow my own advice and have been cleaning up files all day :-). I've gone through the controllers, decorators, and helpers and have been ignoring rubocop where I felt that properly refactoring would take too much work. I also removed two files left over from the log purge (a helper and associated specs). 676 offenses to go!

Ok, I didn't follow my own advice and have been cleaning up files all day :-). I've gone through the controllers, decorators, and helpers and have been ignoring rubocop where I felt that properly refactoring would take too much work. I also removed two files left over from the log purge (a helper and associated specs). 676 offenses to go!

Whew, /app now passes rubocop :-). Only 430 more offenses to go!

Woo hoo! We now pass rubocop :-D.


Greenkeeper

There is a way to make the greenkeeper updates once every week or something like that?
I think they are kind of spamy and most of them are minor updates.
Maybe one update every weekend in one PR with all the updates.

Funny, you should ask 😆
I'm arriving at the same conclusion. There appears to be no way for GK to open a weekly update PR.  ref GK #247
I'm working on an update deps roll-up as we speak.

@mxstbr I understand you are traveling? But, is it possible for you to disable Green Keeper?
It's actually less work, less spam and less confusion to manually do weekly roll-up updates, which I'm willing to continue doing.

Done, it's disabled!

It's actually less work, less spam and less confusion to manually do weekly roll-up updates, which I'm willing to continue doing.

Thanks for all the work you've been doing in keeping this repo sane, you rock! :tada:

@mxstbr And You and this repo likewise rock - that's why I'm here 💯