APA 102 DRIVER

Rebooting...
f[06][1E][06][18]†fàž[00][18]x†[18][18]x[00]˜à˜ø[18]˜à[18]x˜[18]x[06][18][18][1E]æ€˜€æ€˜€[18]æø[06]~˜[18][18]æà[06][18]†[1E]ffþf€[06]ffæf˜f[1E]f[06][06]f††à†[18]æžþž†~[00][18]f˜x[00][18]†[1E]ff[18]æf[06]æ˜[06]fþfà˜[06]æ˜[06]˜þf€æ˜þ˜f˜˜[1E]˜€[1E]æžøžxfžæøfž[00]†[00][18][18]†[00]˜˜[06]f[18]æf[00]f[00][18]fxff€˜€[1E][06]žžžþfx[18]†˜[18][18][06][18][18]†[06]æfx[18]†˜[18][18][06][18][18]†æfx[18]†~˜[18][18][06][18][18]†[1E]æø[06]˜fx[18]†~˜[18][18][06][18][18]†€žæfx[18]†˜[18][18][06][18][18]†[06]æfx[18]†˜[18][18][06][18]˜[1E]˜€æžþžfx˜[18]†˜þ˜à†[00]˜à[06]žþž[1E]ææ[06][18]x†žx†[06]˜[1E]˜€àžþž[06][06]x˜[18][18]æxx†xx[06][18]˜à[18][18]†àžf†ž˜[18][1E]æ€˜€àžþž[06][06]x˜[18][18]æxx†xx[06][18]˜à˜à[06]†àžf†ž˜˜à[18]x[06][18][1E]æ€˜€€žþž[00][18][18][18][06]fžàž[00]˜à[18]x[00][18]øæžþžæž[00][18][1E]æ€˜€àžþž[06][06]x˜[18][18][06][1E][00][18]˜[1E]€[18][18][18][18]†àžf†ž˜˜ø˜[1E][18][18]˜[1E]˜€€žþž[00][18][18][18][06]fžàž[00]˜à[18]x[00][18]øæžþžæž[00][18][1E]æ€˜€àžþž[06][06]x˜[18][18][06][1E][00][18][18][18][18][1E][00][18][18]†àžf†ž˜[18][1E][1E]˜æ[18]xæ€˜€f†ž`†øf[06][18][18][06][1E][00][18][18][18][18][1E][18][06]˜[1E]˜€in d
in d1
in d2
in d3
in d4
in d5
100000
in d6
in d7
61
42
-4
31
div_b 31
bck_div_num 8
in d5
in d6
//*****************************************************************************************************************************
*/
#include "Log.h"
#include "FastLED.h"
#define NUM_LEDS_PER_STRIP 340
#define NUM_STRIPS 9
#define NUM_LEDS NUM_STRIPS * NUM_LEDS_PER_STRIP
#define CLOCK_PIN 21 //needs to be higher than 16
#include "I2SAPA102.h"
#define FRAMES_PER_SECOND 120
int Pins[9] = {4,13,14,25,26,33,32,15,27 };
CRGB leds[NUM_LEDS];
I2SAPA102 controller(0);
void setup() {
Serial.begin(230400);
controller.initled(leds,Pins,CLOCK_PIN,NUM_STRIPS,NUM_LEDS_PER_STRIP,2); //for 2MHZ
fill_solid(leds, 100, CRGB::Black);
controller.showPixels();
controller.setBrightness(31); //to be used instead of fastled.setbritghness value 0->31 the default ligtning of the apa102
delay(200);
controller.showPixels();
}
void loop() {
fill_solid(leds, 339, CRGB(255, 0, 0));//r
fill_solid(leds + 340, 679, CRGB(0, 255, 0));//g
fill_solid(leds + 680, 1019, CRGB(0, 255, 0));//b
fill_solid(leds + 1020, 1359, CRGB(0, 255, 255));//cy
fill_solid(leds + 1360, 1699, CRGB(255, 0, 255));//mag
//**************************************************************
//causes irratic behavour core error crash leds size 9*340(3060)
fill_solid(leds + 1700, 2039, CRGB(255, 255, 0));//yel
fill_solid(leds + 2040, 2379, CRGB(255, 255, 255));//wh
controller.showPixels();
delay(50);
}

Can you please point me to a url for latest APA102 drivers

thanks got the invitation is 107 and apa102 timing same

It’s not a matter of timing but matter of data structure :)
Yes they are the same

New basic code for hd107 build and tested now to test with wifi then integration to ledsynth main engine. I love the ability to set an individual pixel brightness Im sure this opens up posibilities for new type effects

this is not the final version I have another one a bit better but I need to polish it

Please keep me informed. I am also helping a friend with a product he manufactures IP rated pixel strips aluminium extrusion with internal power supply original APA102 he adds optics to the pixels to tighten the beam. I have a unit here for test at 4mhz clock there are issues. If I set the speed to 2mhz all is ok. I need to investigate further as it could be my RS485 Hardware. In your driver did you do the timing from scratch

Prior to integration I setup this simple sketch, unless I am doing something stupid there is no output on
pins26,33,32,15,27,12. pins 4,13,14,25 no issue
/*
Name:		HD107NEWDRIVER.ino
Created:	11/21/2021 11:23:36 PM
Author:	LaserLaptop
*/
#include "FastLED.h"
#define NUM_LEDS_PER_STRIP 40
#define NUM_STRIPS 10
#define NUM_LEDS NUM_STRIPS * NUM_LEDS_PER_STRIP
#define CLOCK_PIN 21 //needs to be higher than 16
#include "I2SHD108.h"
int Pins[16] = { 4,13,14,25,26,33,32,15,27,12 };
HD107Led leds[NUM_LEDS];
CRGB workingbuffer[NUM_LEDS_PER_STRIP];
I2SHD108 controller(0);
//FORWARD DECS
void RGBfill_workingbuffer(int start, int end, int r, int g, int b, bool order, uint8_t speed);
void RGBfill_ledsbuffer(int stripno, int start, int end, int r, int g, int b, bool order, uint8_t speed);
void RGBfill_workingbuffer(int start, int end, int r, int g, int b, bool order, uint8_t speed)
{
for (int j = start; j < end; j++)
{
	workingbuffer[j] = CRGB(r, g, b);
}

}
void RGBfill_ledsbuffer(int stripno, int start, int end, int r, int g, int b, bool order, uint8_t speed)
{
int stripoffset = stripno * NUM_LEDS_PER_STRIP;
for (int j = start; j < end; j++)
{
leds[j + stripoffset] = CRGB(r, g, b);
}
}
void striptest() {
RGBfill_ledsbuffer(0, 0, 10, 255, 0, 0, true, 0);
RGBfill_ledsbuffer(0, 10, 20, 0, 255, 0, true, 0);
RGBfill_ledsbuffer(0, 20, 30, 0, 0, 255, true, 0);

RGBfill_ledsbuffer(1, 0, 10, 0, 255, 255, true, 0);
RGBfill_ledsbuffer(1, 10, 20, 255, 0, 255, true, 0);
RGBfill_ledsbuffer(1, 20, 30, 255, 255, 0, true, 0);

RGBfill_ledsbuffer(2, 0, 10, 255, 0, 0, true, 0);
RGBfill_ledsbuffer(2, 10, 20, 0, 255, 0, true, 0);
RGBfill_ledsbuffer(2, 20, 30, 0, 0, 255, true, 0);

RGBfill_ledsbuffer(3, 0, 10, 0, 255, 255, true, 0);
RGBfill_ledsbuffer(3, 10, 20, 255, 0, 255, true, 0);
RGBfill_ledsbuffer(3, 20, 30, 255, 255, 0, true, 0);

RGBfill_ledsbuffer(4, 0, 10, 255, 0, 0, true, 0);
RGBfill_ledsbuffer(4, 10, 20, 0, 255, 0, true, 0);
RGBfill_ledsbuffer(4, 20, 30, 0, 0, 255, true, 0);

RGBfill_ledsbuffer(5, 0, 10, 0, 255, 255, true, 0);
RGBfill_ledsbuffer(5, 10, 20, 255, 0, 255, true, 0);
RGBfill_ledsbuffer(5, 20, 30, 255, 255, 0, true, 0);

RGBfill_ledsbuffer(6, 0, 10, 255, 0, 0, true, 0);
RGBfill_ledsbuffer(6, 10, 20, 0, 255, 0, true, 0);
RGBfill_ledsbuffer(6, 20, 30, 0, 0, 255, true, 0);

RGBfill_ledsbuffer(7, 0, 10, 0, 255, 255, true, 0);
RGBfill_ledsbuffer(7, 10, 20, 255, 0, 255, true, 0);
RGBfill_ledsbuffer(7, 20, 30, 255, 255, 0, true, 0);

RGBfill_ledsbuffer(8, 0, 10, 255, 0, 0, true, 0);
RGBfill_ledsbuffer(8, 10, 20, 0, 255, 0, true, 0);
RGBfill_ledsbuffer(8, 20, 30, 0, 0, 255, true, 0);

RGBfill_ledsbuffer(9, 0, 10, 0, 255, 255, true, 0);
RGBfill_ledsbuffer(9, 10, 20, 255, 0, 255, true, 0);
RGBfill_ledsbuffer(9, 20, 30, 255, 255, 0, true, 0);

//leds[0] = Brightness(2);

long ts2 = __clock_cycles();
controller.showPixels();
Serial.printf("fps:%f\n", (float)240000000 / (__clock_cycles() - ts2));
delay(400);

}
void setup() {
Serial.begin(230400);
controller.initled((uint8_t*)leds, Pins, CLOCK_PIN, NUM_STRIPS, NUM_LEDS_PER_STRIP, HD107, 2); //default Speed 2MHz
//controller.initled(leds,Pins,CLOCK_PIN,NUM_STRIPS,NUM_LEDS_PER_STRIP,freq in MHZ);
//controller.initled(leds,Pins,CLOCK_PIN,NUM_STRIPS,NUM_LEDS_PER_STRIP,2); //for 2MHZ
delay(200);
//set the brightness to max for all of the leds
for (int i = 0; i < NUM_STRIPS*(NUM_LEDS_PER_STRIP); i++)
{
	//this affect only the brightness of the leds
	leds[i] = Brightness(31);
}

}
void loop() {
striptest();
}

on line
35 of file of I2SHD108. h
add
#define NUMSTRIPS 16
line 39

Please keep me informed. I am also helping a friend with a product he manufactures IP rated pixel strips aluminium extrusion with internal power supply original APA102 he adds optics to the pixels to tighten the beam. I have a unit here for test at 4mhz clock there are issues. If I set the speed to 2mhz all is ok. I need to investigate further as it could be my RS485 Hardware. In your driver did you do the timing from scratch

Hello on that part it’s not your rs485 it’s the speed of the esp32. In the new library I am working on being able to push at 4mhz maybe more. I need to work on it a bit more

wheres the latest hd10 fix cant see it in shared repository ?

SORRY, misunderstanding I thought you had release an update to the hd108 driver. yes the #define fixed that problem all 10 RS485 outputs working

Hello
Sorry I did not have the time to look at it. I will have a look later today or tomorrow

The problem of the LEDSynth (HD107 DRIVER) is Hardware and maybe your driver as YOU pointed out.
At 1mhz I can drive APA102 over screened CAT5 RS485 transmitter --> receiver . If I use an RJ45 female to female unscreened joiner and use a section of unscreened CAT5 I get errors. I will get a longer length of  Screened cat 5 and test again

Update using current version of your hd driver clocked at 1mhz driving apa102 at 37metres away
no screened cat5 a poor quality joiner linking 2 cat5 cables

Hello is it working well ?

JUST AN OCCASIONAL GLITCH THEN IT CORRECTS ITS SELF WONT RUN FASTER THAN 1MHZ RELIABLY
im using 37mts unshielded cat 5 so could still he a noise issue even as it is twisted pair will continue to do tests will try and pickup a very long shielded cat 5

hello good to hear.
I am busy with a critical go to production at work I will get to it asap

Thanyou Yves
On 02/12/2021 16:43:17, hpwit @.***> wrote:
hello good to hear.
I am busy with a critical go to production at work I will get to it asap
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [https://github.com/hpwit/FASTLEDVIRTUALDRIVER/issues/8#issuecomment-984639213], or unsubscribe [https://github.com/notifications/unsubscribe-auth/AS6GHF6IRCNE6PLLO5DCPQ3UO5ZXFANCNFSM5IPBX2HQ].
Triage notifications on the go with GitHub Mobile for iOS [https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675] or Android [https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub].


Create a binary_sensor using the gpiozero library

Description:
The gpiozero library provides a unified interface for both local GPIO
pins and remote GPIO pins (via pigpiod).  This allows it to replace
the standard rpi_gpio component while also adding new capabilities to
easily interact with GPIO pins on networked Raspberry Pis.
I chose to use pigpiod over other alternatives because it is the de-facto standard that is installed via raspi-config.  Until (this issue)[https://github.com/joan2937/pigpio/issues/189] is resolved in pigpio there is a possibility for the component to enter a deadlock state if network connectivity between Home Assistant and pigpiod is lost.  I have provided a pull request to address this issue; as a workaround my fork of pigpio can be installed instead.
I chose to use gpiozero instead of directly using the pigpio library so that one unified component could interact with both local and remote ports, reducing duplicate code.
**Documentation has been started (here)[https://github.com/maihde/hass-rpi-gpiozero] and will be migrated to home-assistant.io when this component is accepted.
Example entry for configuration.yaml (if applicable):
binary_sensor:
  - platform: rpi_gpiozero
    ports:
      12: Local Input
  - platform: rpi_gpiozero
    host: <IP_ADDRESS>
    ports:
      12: Remote Input 1
      13: Remote Input 2

Checklist:

[X] The code change is tested and works locally.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[X] New dependencies have been added to the REQUIREMENTS variable (example).
[X] New dependencies are only imported inside functions that use them (example).
[X] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[X] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hi @maihde,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @maihde,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @maihde,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @maihde,
can you integrate also gpiozero-switch?
So I'm able to switch some relay on a remote pi.
Tanks...

So the standard rpi_gpio component talks only to local pins.  I wanted to
be able to read GPIO pins on a remote Raspberry PI.  This requires some
form of daemon running on the remote PI and a component within home
assistant to talk to that remote daemon.
For the daemon, I chose to use pigpiod because it is a included with
Raspbian and can easily be enabled with raspi-config.  I implemented the
component using gpiozero because it provided a high-level API that worked
both with local and remote pins; the objective would be that eventually
this implementation (once feature complete) could replace rpi_gpio so
that only one component needed to be maintained.
I'm not partial to the name itself and only used that name to disambiguate
it from rpi_gpio.  If you would prefer that I finish the implementation
so it is a drop-in replacement to rpi_gpio I can work down that path and
submit a new pull request.
On Wed, May 2, 2018 at 2:17 PM, Paulus Schoutsen<EMAIL_ADDRESS>wrote:

However, I'm a bit confused. rpi_gpio seems to talk to the pins, this PR
just talks to a daemon ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/pull/12105#issuecomment-386072108,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOCIgiK8lGyL06INygSpaajsI0zKJKcks5tufg0gaJpZM4R0_a5
.

Hmm. I wonder if we would want this at all or that we should just wait for #13876 to land so that someone can just connect Home Assistant on the remote Pi too.
I wouldn't want to replace the current implementation because it would mean extra dependencies and usage of resources for no extra gain.

This PR seems to gone stale. Please re-open it when you want to proceed.


[HELP] Deploying Blazor Hero to Linux with Nginx and SignalR support

Hi everyone,
I'm trying to deploy this application to linux using NGINX as a reverse proxy.
I managed to get access to the site using the IP Adress but when I connect to it, I recieve a signalR 404 connection error.
When I try to access https://MyIpAddress/signalRHub I get a 404 page.
When I try to access https://localhost:5000/signalRHub on my local windows machine (in development) everything works as expected.
Could anyone help me deploy this project on a linux server using nginx?

This is what I Find when debugging on my local computer

this is in production

It's like SignalR is not working on my linux server. I have no idea why

Do you have Kestrel as web server behind NGINX?
If it is so, take a look to this tutorial:
https://www.nginx.com/blog/tutorial-proxy-net-core-kestrel-nginx-plus/

Hi @nbiada, Thanks for your answer.
I already have a successful kestrel server working.

I can access it with http://<IP_ADDRESS> over the network and everything is working except for the signalRHub.

It's only when I try to access SignalR endpoint that I recieve a 404 not found page.

I believe that my Nginx server is working (at least for the website page) but not for the signalR part.
I don't know how to activate signalR on the production server. When I run the project from my development computer on windows, SignalR is running and working as expected.
I think I'm going to have to remove signalR from the project since this is the only thing blocking me.
I would of course prefer to make it work

What is your nginx config?  You have to add a few lines to a default nginx config.
https://docs.microsoft.com/en-us/aspnet/core/signalr/scale?view=aspnetcore-5.0#linux-with-nginx

Hello, Thanks for your answer and sorry for the late response, I didn't had access to my server during the weekend.
Here is my nginx.conf file.
user www-data;
worker_processes auto;
pid /run/nginx.pid;
include /etc/nginx/modules-enabled/*.conf;

events {
	worker_connections 768;
	# multi_accept on;
}

http {

	##
	# Basic Settings
	##

	sendfile on;
	tcp_nopush on;
	tcp_nodelay on;
	keepalive_timeout 65;
	types_hash_max_size 2048;
	# server_tokens off;

	# server_names_hash_bucket_size 64;
	# server_name_in_redirect off;

	include /etc/nginx/mime.types;
	default_type application/octet-stream;

	##
	# SSL Settings
	##

	ssl_protocols TLSv1 TLSv1.1 TLSv1.2 TLSv1.3; # Dropping SSLv3, ref: POODLE
	ssl_prefer_server_ciphers on;

	##
	# Logging Settings
	##

	access_log /var/log/nginx/access.log;
	error_log /var/log/nginx/error.log;

	##
	# Gzip Settings
	##

	gzip on;

	# gzip_vary on;
	# gzip_proxied any;
	# gzip_comp_level 6;
	# gzip_buffers 16 8k;
	# gzip_http_version 1.1;
	# gzip_types text/plain text/css application/json application/javascript text/xml application/xml application/xml+rss text/javascript;

	##
	# Virtual Host Configs
	##

	include /etc/nginx/conf.d/*.conf;
	include /etc/nginx/sites-enabled/*;
}

#mail {
#	# See sample authentication script at:
#	# http://wiki.nginx.org/ImapAuthenticateWithApachePhpScript
#
#	# auth_http localhost/auth.php;
#	# pop3_capabilities "TOP" "USER";
#	# imap_capabilities "IMAP4rev1" "UIDPLUS";
#
#	server {
#		listen     localhost:110;
#		protocol   pop3;
#		proxy      on;
#	}
#
#	server {
#		listen     localhost:143;
#		protocol   imap;
#		proxy      on;
#	}
#}

and my site-enable/default file:
server {
    listen        80;
    server_name   ppt-lab.com *.ppt-lab.com;
    location / {
        proxy_pass         http://<IP_ADDRESS>:5000;
        proxy_http_version 1.1;
        proxy_set_header   Upgrade $http_upgrade;
        proxy_set_header   Connection keep-alive;
        proxy_set_header   Host $host;
        proxy_cache_bypass $http_upgrade;
        proxy_set_header   X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header   X-Forwarded-Proto $scheme;
    }
}

I tried following the guides on your links but with no success.
I'm not sure if I have to add a second server with location /signalRHub.
I tried everything with no success. I will, for now, sadly remove signalR from the project.

Sorry, I did have this working at one time but it was done only for future planning and not current use.  So, it has not been maintained.  I had to redo my nginx setup and have made many project updates along the way.  Now I'm not able to get it working.
I do not remember adding the additional location.  My notes from when I previously did it referenced the guides previously referenced and
sudo vi /etc/nginx/nginx.conf
and add in http section
proxy_buffer_size   128k;
proxy_buffers   4 256k;
proxy_busy_buffers_size   256k;
large_client_header_buffers 4 16k;

I got mine going.  I made a change in my nginx.conf to enable debugging.
error_log /var/log/nginx/error.log debug;
Then get your app to crash and go look at the logs.  I found the follow error which helped me find the solution.
[info] 143898#143898: *16 client exceeded http2_max_field_size limit while processing HTTP/2 connection
I had to also add http2_max_field_size 64k; because of the version of nginx that I was using.

I added the debug to the error file. There doesn't seem to be an obvious error...

I have now deployed a clean Blazor Hero application without any modification on my side. I still get the same 404 error.
I have added to nginx.conf the following statements like @jnalley20 has suggested without any success.
proxy_buffer_size 128k;
proxy_buffers 4 256k;
proxy_busy_buffers_size 256k;
large_client_header_buffers 4 16k;
http2_max_field_size 64k;

Are you sure that your kestrel in Linux is working correctly?
It seems an unreachable address.

Hi @RHaughton
Did you already solve this issue?


Support arc tracks (GenericJSON)

Support for arc tracks with GenericJSON parser, since Eagle/Fusion 360 Electronics are capable of curved traces.

Clicking to highlight will break. Need to handle arcs there too.

Good point, I had sort of forgotten about the click-to-highlight capability.  That was a fun little Sunday morning math quiz, but it seems to be working now.
For reference: Eagle/Fusion arc angles (internally) are in the range [0.0, 719.9], following the usual mathematical convention where angle is measured counterclockwise from the positive X axis.  Since the Canvas API evidently follows the opposite angle convention, in my Eagle script I simply negated both angles so in the JSON they're always in the range [-719.9, 0.0].  Those values render the arcs in the canvas just fine as-is, but I had to do modulo arithmetic within pointWithinDistanceToArc() when checking proximity.  I think all that logic should also work for values outside that numerical range (assuming Kicad will likely be different) but I haven't personally tested that.
(side note: a comment in DATAFORMAT.md indicating the angle convention wouldn't hurt)


Dark stuff

So when I made the torches whiter and brighter, and the night time brighter, things like chests, beds, crafting tables, and such are kinda dark. It looks odd. How do I fix that?
— Sincerely, n00b

Those things are rendered by an entity shader which is still unmodified in my shader pack (#54). To change how they are rendered, you need to modify its code! There's no configuration values for it yet.

I'm closing this issue as it's basically a dupe of #54.

Oh, okie. Thanks my favorite depressed boi.


where is the tutorial?

https://github.com/Qihoo360/Quicksql/blob/master/doc/BUILD_doc.md  page not foud

This file has been deleted, please visit our new document

Tutorial


Uncaught error in expression analysis with stats::t.test

Expected Behavior
Expression analysis with the default stats::t.test following normalization and imputation should return a tidyproteomics object.
Actual Behavior
Expression analysis with the default stats::t.test executes unsuccessfully, returning an error after attempting to extracting a p-value from the stats::t.test output. Requesting a modification of calc_pv in analysis_expressiondiff_test.R to handle p-value errors and return 1 or NA in such instances (identical data across groups) if appropriate for this test and others if applicable.
Steps to Reproduce the Problem
The reprex
hela_proteins %>% normalize(.method = "median") %>% impute() %>% expression(control / knockdown)
produces output

ℹ Normalizing quantitative data
✔ ... using median shift [495ms]
✔ Selecting best normalization method ... done
ℹ  ... selected median
✔ Imputing by row using the function ::, base, and min [1.8s]
ℹ ... 1838 values imputed
Error in dplyr::mutate():
! Problem while computing p_value = purrr::map2(x, y, calc_pv, .method, ...).
ℹ The error occurred in group 166: identifier = "A0A2R8Y783".
Caused by error in t.test.default():
! data are essentially constant

Specifications

Version: tidyproteomics 1.5.5, R 4.2.2
Platform: Windows

addressed in update 1.5.6 - 2023.09.07, essentially wrapping a try-catch block around the test and reporting 1 on error

Works great. Thanks, Jeff!


why the amphetamine not update in amost one year

why the amphetamine not update in amost one year

Because volunteer effort works on volunteer time.


Fix develocity plugin version update script

It only matches x.y.z but not x.y.
Also, sed is quite tricky on different OSes, use java instead.

Can't cherry-pick due to merge conflict. You have to cherry-pick by yourself.

Sorry some internal error occurs, please contact the administrator @gradle/bt-developer-productivity


Update analysis status

Doesn't look like the analysis status is being updated correctly. Only when I click "edit" from the homepage will the analysis update I believe.
It should probably check every x seconds, or everytime a tab changed if the status is "PENDING"

Yeah, it currently doesn't poll the server to see if the analysis is still "Pending". It's a good idea though. Should we be doing this on both the Analysis Builder page and homepage (which would normally have a list of analyses)? How long does it typically take for the analysis run to complete?

I'm still working on compiling... and that may vary. For now it's simple so maybe a minute but in the future we may do heavier validation. Everytime you do an API call to analysis, I update that field, FYI, so need to check a special "status" route.

Ok, related to this, have you also figured out what the server will return as the analysis results? Perhaps for now you can just return a JSON field that I'll just blindly display in the review or status tab? Once I know this I can implement polling and updating the status/results.

No all we need is to tell the user it is compiled. It means they can now go run the actual analysis on their computer, so we will add those instructions later. at /api/analysis/{id}/bundle the full analysis can be fetched by the command line interface, so compiling makes that end point available, but I don't think you need to worry about it aside from alerting the user to the status for now.

Ok, that makes more sense now. I thought the analysis would automatically run on the server. Do you already have a route for just querying the status? If not, I think at least for now it should be fine to just poll /api/analyses/{analysis_id}. The payload is pretty small and it happens in the background so a slightly faster route is unlikely to increase the perceived speed of the UI.

Yeah that works for now. If we hit performance issues we can change approach.

fixed in #283


Feature/search

Codecov Report

Merging #1 (b77925d) into main (a247fdc) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #1   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         8    +1
  Lines          439       467   +28
  Branches       101       103    +2
=========================================
+ Hits           439       467   +28

Impacted Files
Coverage Δ

src/commands/download.ts
100.00% <ø> (ø)

src/commands/info.ts
100.00% <100.00%> (ø)

src/commands/search.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[📑 Docs]:  feat asyncapi typedoc

What Dev Docs changes are you proposing?
More details about the documentation structure is at:
https://github.com/asyncapi/website/issues/602
Creating this issue, more for tracking it in cli itself and working on it.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Any update on this @imabp, Are you still working on this?

closing as there was no reply

Thanks @derberg we are tracking it in cli repo


paths to db in conf.json

i installed a newer version of golang via a PPA which broke go get in several ways, so i cloned the project manually, ran npm install successfully after editing the conf.json file (which by the way needs to replace the tania-server references with tania-core) but when i try to run go run main.go i get this error:

2018/04/13 08:14:02 Using 'conf.json' configuration
2018/04/13 08:14:02 Using sqlite persistance engine
2018/04/13 08:14:02 Creating database file /home/me/go/src/github.com/Tanibox/tania-server/db/sqlite/tania.db
2018/04/13 08:14:02 Using SQLite at /home/me/go/src/github.com/Tanibox/tania-server/db/sqlite/tania.db
2018/04/13 08:14:02 Executing DDL file for /home/me/go/src/github.com/Tanibox/tania-server/db/sqlite/tania.db
panic: unable to open database file

goroutine 1 [running]:
main.initSqlite(0xc4201fe230)
	/home/me/.local/share/Trash/files/tania-core.3/main.go:365 +0x368
main.main()
	/home/me/.local/share/Trash/files/tania-core.3/main.go:49 +0xd0b

but i had started from a fresh install and had edited the conf.json to replace tania-server with tania-core, where is it getting this path to tania-server?

@bobombolo From what we can saw in your log, your path in your conf.json still refers to .../Tanibox/tania-server/... instead of tania-core.
Then the unable to open database file error message indicates that you have a problem in your sqlite file. It could be missing because of wrong path or permission problem.

but this is my conf file:

{
    "tania_persistence_engine": "sqlite",
    "demo_mode": true,
    "upload_path_area": "/home/me/go/src/github.com/Tanibox/tania-core/uploads/areas",
    "upload_path_crop": "/home/me/go/src/github.com/Tanibox/tania-core/uploads/crops",
    "sqlite_path": "/home/me/go/src/github.com/Tanibox/tania-core/db/sqlite/tania.db",
    "mysql_host": "<IP_ADDRESS>",
    "mysql_port": "3306",
    "mysql_dbname": "tania",
    "mysql_user": "root",
    "mysql_password": "root",
    "redirect_uri": "http://localhost:8080/",
    "client_id": "f0ece679-3f53-463e-b624-73e83049d6ac"
}


Please add a license file and a screen shot

I'd like to add this to awesome-zsh-plugins, but I prefer to add themes with screen shots in the readme so that users can see what they look like without having to install them.
Also, please add a license file, some people won't use any code that doesn't have an open source license. If you don't have one in mind, https://choosealicense.com is a good resource to help you select an appropriate one.
Thanks!

hi
I'm still struggling with #1 since I got no response from ohmyzsh on discord and this seems too trivial to open an issue on the repo.

readme added with screenshot
and license also added


chore: updating formatting via Prettier

updating formatting via Prettier

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


From s3

A few changes.  Going by commit is probably the cleanest way.
Also, personal request, we need to keep testing speed down.  Dask.bag used to test in a few seconds.  With from_url and from_s3 it's up to above a minute.  I tend to be anal about this because I code with continuous feedback from running tests at every save.  The difference between 3 seconds and 50 feels huge.  Perhaps this can be done by marking things as @pytest.mark.slow and using pytest magic?

We seem to be married to py.test so I am for adding the slow markings. I'll add them.


perf(css): improve postcss config resolve

Description
With many parallel CSS file requests (especially with preTransformRequest), each CSS will try to resolve the postcss config because the cache has not been filled. This is slow as we only need one CSS file to resolve it, while others can wait for that one result. Otherwise all CSS files will resolve their own postcss config at once, causing slowness.
On top of that, postcss-load-config takes strangely long to load (2s for me), especially if you don't have a config at all. So we need to reduce calling it.
This PR does the above by caching the promise of the config resolve, instead of it's result.
Additional context

Debugging the slowness in https://github.com/sveltejs/vite-plugin-svelte/tree/main/packages/playground/big

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

This is awesome 🔥


Ci ignore trivial pasindu

Hey @PasinduDineth, thanks for the PR. Unfortunately, this branch is not yet ready for a merge. It was a testing playground for both @Sherlouk and @pedrovereza. Sorry, but I'll have to close it for now.


Better direction? (Rename repo?)

Thinking about submitting to Awesome but that would require substantial changes, such as renaming the master branch, and also a better direction for the project.
Currently the name of the list is awesome-unique-id  which seems language-agnostic, however almost every submission is for JavaScript. What about renaming the list to something like awesome-unique-id-js, awesome-js-unique-id or awesome-javascript-id (my personal favorite)?

I made this list when I was working with JS heavily. I had it as a private file on my computer initially. Once the whole Awesome thing popped up, I created this repo, but to be an official Awesome repo there were a couple of requirements I didn't quite meet. I lost interest as I moved on to other things.
The RoboHash site is just fantastic and I love it. I use it now and again. My current work for example has a CI/CD automation user who didn't have a profile image. It's great for that sort of thing.
I planned to add more ID technologies, but life got in the way. For example, my current struggle is getting screenshots working on Wayland. It is such a dogs breakfast that I'm going to put that off until support has improved I think. I digress.
Is there an awesome id project written in Golang? What about Rust? I would say there are a heap of other tech id packages we could add.
Some of those research papers you linked to could be added also.
I don't think I want to turn this into JAJSR (just another js repo). Id technology and all that surrounds it is fascinating.
Let's keep it generic. In fact, please add research papers as a heading @tzwel.
I will rename the master branch right now.

The master branch has been renamed to main. Please update your local clones.

Got it, added in ffa9e05
I'll keep this open in case we want to turn this completely Awesome-friendly and submit in the future. Maybe then people will contribute libs for other languages

So to be sure, it's fine if I make changes to comply with the Awesome list guidelines?

Yes, make it your own. If you are keen on making it a JS only repo, let me know and I'll rename it. I'm not too invested in this. Happy to have someone else run with it and improve what they can. Thanks for you help.

Let's stick with your idea and not make this js-only, it might be beneficial in the future.

Thanks for you help.

You are welcome, it's very fun!

  README.md:1:84
  ×   1:1   License was not detected by GitHub  remark-lint:awesome-github
  ×   1:1   Forbidden license section found     remark-lint:awesome-license
  ×   1:84  #contributors                       remark-lint:double-link
  ×  13:3   #contributors                       remark-lint:double-link

Here's what we have to fix to adhere to awesome list guidelines. Relicensing mandatory, and you choose what to do with the contributors sections

I'm not running the linter tzwel, but that should do it.

All good now!


How to use for Apollo Client 2.0

The ApolloClient object in the new 2.0 version requires that both link and cache be passed in. How is this library supposed to be used in 2.0?

Please see https://github.com/apollographql/apollo-link/issues/158

@af maybe you can take this code if the author doesn't mind and release it as a new version of this package?
v1 - for apollo v1
v2 - for apollo v2
This will be super useful.

I've been using this package with apollo v1 and would be happy to update and use it with v2.

Thanks @VladShcherbin, I considered repurposing the code, but it looked like @seeden was intending to publish it himself. I'm definitely on board for having the v2 code live here if Apollo doesn't include it.

Thank you. It looks like I actually do not need a local Apollo client as I now realize that I can query directly as follows: graphql(schema, query).then((result) => console.log('Got result', result))

It looks like this is now in core: https://www.apollographql.com/docs/link/links/schema.html

Yes, I added a note about that to the readme this morning. Let me know if it can be worded more clearly.
We had a good run, apollo-local-query


Fixed typo: remove duplicate scroll_delta_x

Renamed the second scroll_delta_x to scroll_delta_y

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


When Importing vue-next with rollup inner components do not mount

Version
3.0.0-alpha.6
Reproduction link
https://github.com/vue-leaflet/vue-leaflet/blob/rollup-bug/src/main.js
Steps to reproduce

npm install
npm run dev
open localhost:5000
main app onMounted is executed
foo markup is missing and onMounted is not executed

What is expected?
Both main app and child components are rendered
What is actually happening?
The child component is not rendered, both by using a template string or a render function

I've prepared another branch where by using vue-next from cdn directly in the html everything works as expected:
https://github.com/vue-leaflet/vue-leaflet/blob/rollup-global/src/main.js
Since this bug, I've migrated the whole repo to use the webpack-vue-next-starter template and with SFC everything works as expected.
As a side note, I've tried to import all the version of present in dist/vue/ thinking that I was lacking the live template compiler, but then I noticed that the issue is the same with render functions
Apologize for the messy repo, I've tried to reproduce on codesanbox but It keeps failing to load vue-next.

See https://github.com/vuejs/vue-next/tree/master/packages/vue#which-dist-file-to-use
The default build used with a bundler is the runtime-only build so it doesn't support on-the-fly compilation.
Also you probably want to wait until rollup-plugin-vue support vue-next.

@yyx990803 thanks for the clarification! One question, the issue happens even just with render functions. Aren't those separate from the template compilation?

Your root instance is still using in-DOM template (in index.html) which requires runtime compilation.


BUG SimpleHTMLEditorField errors in some cases

Closing as this was caused by another module and it's affect on the Requirements portion of the framework.


Implement GeoShape query

Thank you very much for working on this!
When I add new queries, I typically use the Java source as a basis. It often has more fields than documented at the official manual. E.g. GeoShapeQueryBuilder.java has these fields, serializes in this way. Can you look into that and complete the query?

Sure, I'll do it soon!

What is the status of this PR? I really need GeoShape support :/

@timonmasberg Creating all the different builders is a lot of work. It has been mentioned in several issues how one can do that. I'm struggling with time to do it for a while now.
The workaround is to build the query yourself. You can e.g. always you RawStringQuery like so:
q := elastic.NewRawStringQuery(`{
	"geo_shape": {
	  "location": {
	    "shape": {
	      "type": "envelope",
	      "coordinates": [ [ 13.0, 53.0 ], [ 14.0, 52.0 ] ]
	    },
	    "relation": "within"
	  }
	}`)
res, err := client.Search().
	Index("YOUR_INDEX_NAME").
	Query(q).
	Do(ctx)
...

... or build your own Query implementation.

Okay, thank you!


Mac_OSX_Startup.pdf no longer available

Mac_OSX_Startup.pdf no longer available

It's available in the Intenet Archive, the link just need to update.


EKF2 - GPS fusion fault?

Hi guys,
after a few days of testing EKF2 something odd happened. While executing some simple mission, just after take off, my copter started to run away. At first I thought that I made something wrong during mission planning because it seemed as if the copter began heading in the direction of second WP, but it accelerated and passed the WP about 8m from that. I took over control but it was too late and it had hard landing. I have examined the log and don't know what to think of it. In estimator_status there were several GPS glitches reported, problems with velocity and position estimates. No filter_fault_flags, no gps_check_fail_flags. Seems that the estimator 'knew' that something was wrong but did nothing about it.
A few days later I was testing again. Everything was okay, but after approx 5 flights the same happened. Both logs look the same, there is a huge drift of estimated position. From what I observed I can say that GPS position and GPS velocity (up to 12m/s despite MPC_XY_VEL_MAX = 1m/s) where correct.
The issue seems to be similar to #6388 but in real flights, not in SITL.
I cannot upload whole logs due to some agreements that I'm involved in but I'd just like to report the strange behaviour and release the brain storm.
FIRST FLIGHT

SECOND FLIGHT

@pfkrp Could you please add plots of the estimator innovations?
It's this topic https://github.com/PX4/Firmware/blob/master/msg/ekf2_innovations.msg

Here's a recent log from our flight test people where something similar happened: http://logs.px4.io/plot_app?log=4fa1b2b4-8b19-4f77-815b-81fb8855834c

I can't do ekf analysis based on screen shots, please provide the log file.

@tumbili @priseborough Could you please email me on<EMAIL_ADDRESS>?

While bench testing sitting still with GPS connected a Pixracer several days ago, I watched the ship icon drift considerably in a straight line on the QGC map. After a couple seconds, It jumped instantly back to where it started and I saw "EKF GPS fusion timeout - reset to GPS" warning. Not sure it is related but I thought it was odd. If you think this is related, I can spend more time seeking this condition?

@pkocmoud You wouldn't have a log of that would you?

I've seen a similar issue on several occasions and marked it up to bad GPS.  All I can provide is a few anecdotes that may or may-not be helpful:

Had an aircraft with a Pixracer and a very-very cheap "ublox m8n" (likely a knock-off) GPS/Mag.  On several occasions the aircraft position on the Qgc map would jump around and looked a lot like the first x-y plot shown at the top of this thread.  It "seemed" like it had a very good GPS position with lots of sats. This same aircraft had flown dozens of times previously without issue.  After a couple outings where this was happening the cheap GPS was replaced with a much better MRO brand M8N unit and never saw the issue again on that aircraft (replaced the wires to the GPS too).

Had a different aircraft, again with a Pixracer and MRO M8N GPS.  Aircraft had previously flow great then one day starting behaving very badly with position and heading jumping all over the place.  Discovered that the wires going to the GPS/Mag had been damaged and may have been shorting on a metal screw.  Not sure if the GPS or Mag wires were the ones shorting or what was going on exactly.  Replaced the wires and haven't seen the issue again on that aircraft.

I've seen drift like this when set up on a bench inside and I have a very poor in-and-out GPS lock.

@pfkrp I've been noticing the same issues!  What version of the PX4 firmware have you been using?  Mine only started doing this recently, so I'm wondering if it had something to do with when I merged the newest commits from Master.  I'm going to do a couple of builds to test this out, but my last big update was from Jan. 14th.

@brdgtannj Check how your magnitude of magnetic field behaves. It seemed that it was my problem when I compared to the other flights that were successful. It was caused by grounded metal shield in the vicinity of the gps module. Since removal of the shield I haven't observed that problem and the magnitude of magnetic field looks flat as it should.

We've changed so much (and improved so much) that I expect this to be solved on a correctly calibrated setup. Please re-open if it persists.

Seeing something similar flying an S1000 octo:
http://logs.px4.io/plot_app?log=d6944989-20ef-4007-bf1e-ca0b3732347f
Multiple warnings:
2:24:865 | WARNING | [lib__ecl] EKF GPS fusion timeout - reset to GPS
Have tried two different GPS units (ublox M8N, M8T) with the same results.
Looking at the code it appears this warning shows up when EKF decides GPS data is now valid. I'll look at whether GPS data stalled somehow, but it should not have.

It looks more like an inertial sensor issue / EKF2 tuning issue for your specific airframe. You have a relatively high angular rate noise: http://logs.px4.io/plot_app?log=d6944989-20ef-4007-bf1e-ca0b3732347f#Nav-Raw-Angular-Speed-Gyroscope

Output from ./Tools/ecl_ekf/process_logdata_ekf.py
Uploading d6944989-20ef-4007-bf1e-ca0b3732347f.ulg.pdf…
It looks like the compass is at least part of the problem.


Added offset_x and offset_y to IMouseEvent

I added links to the w3c spec as requested.
About the return type,
https://developer.mozilla.org/en-US/docs/Web/API/MouseEvent/offsetX#Browser_compatibility
shows only chrome and android webview use doubles and the other browsers use longs.
Combined with https://github.com/koute/stdweb/issues/164 I thought it might be better to wait till the next semver to sort that out. Do you still want me to change it to a double?

@Tacklebox Browser compatibility isn't a problem, because f64 (double) is a strict super-set of i32 (long). In other words, it's possible to losslessly convert from i32 to f64 (but not vice versa).
And in fact stdweb should automatically do that conversion (since all numbers in JavaScript are f64 anyways).
Given #164, I personally vote to make them f64.

Yep. Basically what @Pauan said.
LGTM; thanks for the PR!


Can the TensorFlow  Android object detection example support MobileNetV2 and MobileNetV3 tflite models?

Issue type
Support
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.8
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Can the TensorFlow  Android object detection example https://github.com/tensorflow/examples/tree/master/lite/examples/object_detection support MobileNetV2 and MobileNetV3 tflite models?It seems that it only supports MobileNetV1?
Standalone code to reproduce the issue
......

Relevant log output
No response

Hi @libofei2004 ,
I tried plugging in the mobilenetv2 in the object detection example. It is giving error but i think with some code changes you can get it to work.

@sawantkumar Maybe，but do you know how?

Hi, @libofei2004
I apologize for the delayed response, I see at the moment we support for object detection models mentioned in this file https://github.com/tensorflow/examples/blob/master/lite/examples/object_detection/android/app/download_models.gradle for object detection TFLite examples, if you're looking to use ssd_mobilenet_v2 then you can download TFLite model from here https://www.kaggle.com/models/tensorflow/ssd-mobilenet-v2 and you may have to change your code as per the ssd_mobilenet_v2 model (input and output), please check this github repo which may help you.
Thank you for your cooperation and patience.

@gaikwadrahul8 github repo this repo is just provides classification, but what I want is object detection.

Hi, @libofei2004
I apologize for the delayed response, you can refer this MediaPipe Tasks Object Detection Android Demo with the option to use a quantized MobileNetV2 please refer this Object detection guide for Android
Thank you for your cooperation and patience.


fix: bump kcl-go version to 0.11.0-alpha.1

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[ ] Y

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 41.884%

Totals

Change from base Build<PHONE_NUMBER>6:
0.06%

Covered Lines:
4136

Relevant Lines:
9875

💛  - Coveralls


[TFLite] Fix TFLite export Batchnorm & Activation Fusing Op

In this PR
For Now, We have problem that NNTrainer <-> Exported TFLite has difference Output and Inference Result
this PR Will solve that issues
[TFLite] Revisit tflite_opnode.cpp
For Fix TF Lite export method
- remove unnecessary transpose due to tflite_interpreter logic change
- change transpose direction according to equation change

**Changes proposed in this PR:**
- Change tflite_opnode.cpp

Resolves:
- TFLite Export ( BatchNormalization Fusing, Activation Fusing)

**Self evaluation:**
1. Build test:   [X]Passed [ ]Failed [ ]Skipped
2. Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Donghak PARK<EMAIL_ADDRESS>

[TFLite] Revisit TF Lite Fusing Operation
TF Lite export feature has issue that exported tflite file has different
Output with NNTrainer original model.
This commit will fix that issue. with modified Fusing Operation

**Changes proposed in this PR:**
- Update tflite_interpreter.cpp

**Self evaluation:**
1. Build test:   [X]Passed [ ]Failed [ ]Skipped
2. Run test:     [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Donghak PARK<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2350. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


windows python service error: (-2147467259, 'Shell_NotifyIcon', 'Unspecified error')

I wrote code below:
    import pythoncom
    import win32serviceutil
    import win32service
    import win32event
    import servicemanager
    import socket
    import time
    import sys
    import pyodbc
    from win10toast import ToastNotifier

    class AlertSertvice(win32serviceutil.ServiceFramework):
        _svc_name_ = "Alert"
        _svc_display_name_ = "Alert"
        _svc_description_ = "Notification of Windows"
        def __init__(self, args):
            win32serviceutil.ServiceFramework.__init__(self, args)
            self.hWaitStop = win32event.CreateEvent(None, 0, 0, None)
            socket.setdefaulttimeout(600)

        def SvcDoRun(self):
            #servicemanager.LogMsg(servicemanager.EVENTLOG_INFORMATION_TYPE,  servicemanager.PYS_SERVICE_STARTED, (self._svc_name_, ''))
            #self.main()
    #    def main(self):
            rc = None
            while rc != win32event.WAIT_OBJECT_0:
                try:
                    toaster = ToastNotifier()
                    toaster.show_toast('test',duration=10)
                except:
                    servicemanager.LogMsg(servicemanager.EVENTLOG_INFORMATION_TYPE, servicemanager.PYS_SERVICE_STARTED, (self._svc_name_, sys.exc_info()[0]))
                rc = win32event.WaitForSingleObject(self.hWaitStop, 10*1000)

        def SvcStop(self):
            self.ReportServiceStatus(win32service.SERVICE_STOP_PENDING)
            win32event.SetEvent(self.hWaitStop)

    if __name__ == '__main__':
        if len(sys.argv) == 1:
            servicemanager.Initialize()
            servicemanager.PrepareToHostSingle(AlertSertvice)
            servicemanager.StartServiceCtrlDispatcher()
        else:
            win32serviceutil.HandleCommandLine(AlertSertvice)

Then compile and install service (python AlertService.py install) then start it.
It dump an error on Event Viewer:
The instance's SvcRun() method failed
Traceback (most recent call last):
File "C:\PYTHON27\lib\site-packages\pypiwin32-219-py2.7-win-amd64.egg\win32serviceutil.py", line 835, in SvcRun
self.SvcDoRun()
File "D:\DB\Python\Test\AlertService.py", line 33, in SvcDoRun
toaster.show_toast('test',duration=10)
File "build\bdist.win-amd64\egg\win10toast_init_.py", line 115, in show_toast
Shell_NotifyIcon(NIM_ADD, nid)
error: (-2147467259, 'Shell_NotifyIcon', 'Unspecified error')
%2: %3
Please help me fix, thank so much

Hi, as mentioned in #18  this requires some serious understanding of Win API's which unfortunately I don't possess. Could you please post it on stack overflow?

I tested this code on a Windows 10:
from win10toast import ToastNotifier
toaster = ToastNotifier()

toaster.show_toast("TEST", duration=5)

The code runs fine and got expected results when being run on native shell.
However I got the error when I run the code through a SSH connection from other machine.


Removes omitempty from stream chatcompletionrequest

Describe the change
More info in this issue: https://github.com/sashabaranov/go-openai/issues/435
This PR removes the omitempty json tag from the ChatCompletionRequest.
Issue: #435

Closing due to commits made to the upstream fork that weren't meant to go here.


refactor: avoid illegal status in state machines

What this PR changes/adds
Avoid illegal status on TransferProcess state machine by filtering by type on state machine processes.
Why it does that
Avoid wasting state machine cycles.
Further notes

use correlationId instead of id on ConsumerContractNegotiationManagerImpl, practically it's the same thing, but it's conceptually more correct.

Linked Issue(s)
Closes #3404
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

Codecov Report
Patch coverage: 100.00% and project coverage change: -0.01% :warning:

Comparison is base (7d86721) 72.09% compared to head (108b80b) 72.08%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3438      +/-   ##
==========================================
- Coverage   72.09%   72.08%   -0.01%
==========================================
  Files         834      834
  Lines       16847    16827      -20
  Branches      953      953
==========================================
- Hits        12146    12130      -16
+ Misses       4289     4287       -2
+ Partials      412      410       -2

Files Changed
Coverage Δ

...iation/ProviderContractNegotiationManagerImpl.java
98.70% <ø> (-0.20%)
:arrow_down:

...egotiation/AbstractContractNegotiationManager.java
100.00% <100.00%> (ø)

...iation/ConsumerContractNegotiationManagerImpl.java
98.83% <100.00%> (-0.17%)
:arrow_down:

...r/transfer/process/TransferProcessManagerImpl.java
93.45% <100.00%> (+1.05%)
:arrow_up:

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


SPM Support

This will close issue #20

@AppliKey please please merge this!! I'd love to see this in the master repo


4.8.1 can not recover from WebGL context loss second time

4.8.1 can not recover from WebGL Context loss due to the following reasons:

All webgl textures have to be released on context loss.
This is done by TextureManager using TextureManager.managedTextures list.
However,  WebGLRenderer.handleContextRestored() calls _initContext() which is creating new instance of TextureManager, thus managedTextures list gets emptied. All textures created before context loss are not tracked anymore, especially textures created by PIXI.Text.

Solution: Pass old ManagedTextures list to new TextureManager instance in handleContextRestored():
                let temp = thistextureManager._managedTextures;
                this._initContext();
                this.textureManager._managedTextures = temp;

PIXI.mesh.NineSlicePlane should rebuild vertex and index buffers.

gl.canvas.addEventListener( "webglcontextrestored",
				( e ) =>
				{
					this._glDatas = {};
					this.dirty++;
					this.indexDirty++;
					this["vertexDirty"] = this["vertexDirty"] + 1;
				}
				, false
			);

PIXI.Graphics should delete all gl* data:

gl.canvas.addEventListener( "webglcontextrestored",
				( e ) =>
				{
					let a: any = this;
					a._webGL = {};
					a.dirty = a.dirty + 1;
				}
				, false
			);

Steps to Reproduce
WebGL context loss can be tested in Chrome using extensions:
ext = gl.getExtension( 'WEBGL_lose_context' );
ext.loseContext();
ext.restoreContext();

Environment

pixi.js version: 4.8.1
Browser & Version:  Chrome *
OS & Version:  Windows*

Any follow-up @RomanLut ?

Not fixed in 4.8.7. Does not survive lost context.
I think you can easily reproduce it in any project with:
ext = gl.getExtension( 'WEBGL_lose_context' );
ext.loseContext();
ext.restoreContext();


Omni DB inaccessible due to high cpu usage used by Omni DB process

Omni DB Web URL sometimes fails to load with connection timeout errors. Most of the times this is faced when omni db process is having more than 100% cpu usage.

@wind39 Could you advice on above issue.

@wind39 @rafaelthca  Currently we are using v2.16 and we get a lot of compalins from users that OmniDB isn't loading. Appreciate your advice on this.

hi @dewapurag,
How many users access OmniDB concurrently? Just a rough estimation.
Is CPU usage 100% all the time? Do you have to restart OmniDB or does it go back to normal by itself?
How often do you experience that? This is a critical issue and we will work together to solve it.
Rafael.

Another question, does one of your users use the SSH terminal feature?

I just pushed a fix to the issue related to SSH terminal. When the SSH tunnel to the remote machine gets disconnected, there is a corner case where the thread waiting for console responses remains stuck in an infinite loop, resulting in 100% CPU.
We are going release 2.17 tomorrow with this fix.
I would still appreciate if you can answer my previous questions so we can investigate if your issue is happening due to something else.

Hi @rafaelthca
Currently around 25 users access concurrently.
Not everytime but sometimes this cpu hit occurs and it resolves automatically too.
If it doesn't resolve automatically  have to restart OmniDB to get back to normal.
Atleast once per day this occur.

Ok.
Is there anything useful in the logs at around the time you are experiencing high CPU usage? Please share it with us after masking sensitive data.
Please test 2.17 after tomorrow's release and let us know if issue is fixed. If not, we'll have to do a more in depth analysis.
What OS are you on?

@rafaelthca
We don't use SSH Tunnel, currently we use only below values to connect;
Technology : mysql
Server -
Port -
Database -
User -
SSH Port -
Currenly syslog looks fine, is there any other log you want me to share

Please find below OS version
root@xxx:~# cat /etc/os-release  NAME="Ubuntu" VERSION="16.04.4 LTS (Xenial Xerus)" ID=ubuntu ID_LIKE=debian PRETTY_NAME="Ubuntu 16.04.4 LTS" VERSION_ID="16.04"

Hi @dewapurag ,
Today we released OmniDB 2.17.0 with a bunch of fixes that might be related to your problem.
Please try upgrading OmniDB to 2.17.0 and see if the issue persists.
Best regards,
William

Hi @ rafaelthca,
I also had a problem with webSocket disconnecting when the CPU was 100% and needed to restart the service to reconnect, but since I was new to Python, I haven't found a solution yet
The omnIDB version is 2.17.0
Do you have any Suggestions?

@rafaelthca
I also had a problem with webSocket disconnecting when the CPU was 100% and needed to restart the service to reconnect, but since I was new to Python, I haven't found a solution yet
The omnIDB version is 2.17.0
Do you have any Suggestions?

Hi all,
OmniDB 3 beta is now live, coming with several improvements.
For example, websocket was replaced with a different strategy using long polling, which fixed quite a few connectivity and performance issues.
I'm closing this issue to organize the roadmap, but feel free to re-open issues if you need.
Thanks for your contribution.
Best regards


[release-1.5] fix imagePullSecret type (#21359)

manual cherry-pick of https://github.com/istio/istio/pull/21359

manual cherry-pick passes visual review.

@richardwxn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
42f33bd9a3951eb1d0160f2ec26902ce54c751f5
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

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

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

Test name
Commit
Details
Rerun command

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
42f33bd9a3951eb1d0160f2ec26902ce54c751f5
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

pilot-multicluster-e2e_istio_release-1.5
42f33bd9a3951eb1d0160f2ec26902ce54c751f5
link
/test pilot-multicluster-e2e_istio_release-1.5

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

/retest

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

Test name
Commit
Details
Rerun command

pilot-multicluster-e2e_istio_release-1.5
42f33bd9a3951eb1d0160f2ec26902ce54c751f5
link
/test pilot-multicluster-e2e_istio_release-1.5

pilot-e2e-envoyv2-v1alpha3_istio_release-1.5
42f33bd9a3951eb1d0160f2ec26902ce54c751f5
link
/test pilot-e2e-envoyv2-v1alpha3_istio_release-1.5

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

/retest


[BUG] PIL.UnidentifiedImageError: cannot identify image file

Description
I'm unable to get the PDF report for my tile extraction due to an image identification error. I'm not sure if that's related with the version of my Pillow package (10.0.1) or Slideflow. Any suggestion would be great, thank you very much
To Reproduce
I used the following code
P = sf.load_project(
        os.path.join(
            "/projects/aieng_pmb/AI_HRD/slideflow_ML/CLAM",
            data_description,
            backbone_model,
            magnification,
        )
    )
dataset_all = P.dataset(
        tile_px=256,   # Tile size, in pixels.
        tile_um=64,  # 0.2632 * 256 Tile size, in microns or magnification.
        min_tiles=8,
    )
dataset_all.tfrecord_report("/path/to/report")

I also tried to generate the report directly when I extracted the tiles, with
    P.extract_tiles(
        tile_px=256,  # size of tile in pixels
        tile_um=magnification,  # size of tile in micro-meters
        whitespace_fraction=0.5,  # discard tiles with this fraction of whitespace
        num_threads=4,  # number of threads
        qc="otsu",
        grayspace_fraction=1,
        save_tiles = True,
        report = True,
        # by default tfrecords = true and image_format = '.jpg'
        # save files to tfrecords in .jpg format
    )

But the method tfrecord_report or extract_tiles produces the following error:
[19:00:14] INFO     Generating PDF (this may take some time)...
/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/fpdf/fpdf.py:1918: UserWarning: Substituting font arial by core font helvetica
  warnings.warn(
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/slideflow/dataset.py", line 3316, in tfrecord_report
    pdf_report = ExtractionReport(reports, title='TFRecord Report')
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/slideflow/slide/report.py", line 433, in __init__
    pdf.image(
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/fpdf/fpdf.py", line 261, in wrapper
    return fn(self, *args, **kwargs)
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/fpdf/fpdf.py", line 3748, in image
    name, img, info = self.preload_image(name, dims)
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/fpdf/fpdf.py", line 3843, in preload_image
    info = ImageInfo(get_img_info(name, img, self.image_filter, dims))
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/fpdf/image_parsing.py", line 121, in get_img_info
    img = Image.open(img_raw_data)
  File "/home/user/.conda/envs/slideflow/lib/python3.9/site-packages/PIL/Image.py", line 3280, in open
    raise UnidentifiedImageError(msg)
PIL.UnidentifiedImageError: cannot identify image file <_io.BytesIO object at 0x2afe65a2bc20>

Environment:

Slideflow Version (e.g., 1.0): 2.1.1
Pillow version: 10.0.1
Slide image format: .svs
OS (e.g., Ubuntu): Linux

Thanks for the report! It's a bit difficult to tell at this point if it's a bug, an environmental issue, or a problem with a corrupt image. I've just pushed an update that should gracefully handle this error and skip images that raise this error.
Try pulling the latest update on the master branch and see if this helps. If the problem is a corrupt image, you should see only a single error message and the rest of the report should generate successfully, If you see hundreds of error messages and the PDF report is empty, it could be an environmental error with mismatched package version or a bug.
We have environments using Pillow 10.01, so I don't suspect that package is the problem.

Hi James, apologize for the delay and thank you for the quick fix. I pulled the update and tried to use pip install dist/slideflow* cupy-cuda11x from source, but I got a conflict message:
Processing ./dist/slideflow-2.1.1+1.gc74c2314-py3-none-any.whl
Processing ./dist/slideflow-2.1.1+1.gc74c231-py3-none-any.whl
Processing ./dist/slideflow-2.1.1+9.ga105e2cb.dirty-py3-none-any.whl
ERROR: Cannot install slideflow 2.1.1+1.gc74c231 (from /user/slideflow/dist/slideflow-2.1.1+1.gc74c231-py3-none-any.whl) and slideflow 2.1.1+1.gc74c2314 (from /user/slideflow/dist/slideflow-2.1.1+1.gc74c2314-py3-none-any.whl) because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested slideflow 2.1.1+1.gc74c2314 (from /user/slideflow/dist/slideflow-2.1.1+1.gc74c2314-py3-none-any.whl)
    The user requested slideflow 2.1.1+1.gc74c231 (from /user/slideflow/dist/slideflow-2.1.1+1.gc74c231-py3-none-any.whl)

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

There seems to be two versions. My current version of slideflow is 2.1.1+9.ga105e2cb.dirty and I'm using cuCIM and pytorch backend. I'm wondering which one of the versions should I choose for the update? Thank you!

In this case, specifying the --upgrade flag should work:
pip install --upgrade dist/slideflow*

Thank you! I'm able to resolve this using the current update.


Temporary hack: overwrite public kernel's final state update using simulator's output

Do #1616, #1617 and #1622 first. Might need to do this at the same time as #1622

Will be closed by https://github.com/AztecProtocol/aztec-packages/pull/1685


solid_queue:install doesn't take config.active_record.pluralize_table_names in consideration

Hi,
I normally use this configuration config.active_record.pluralize_table_names = false, which brings me to this issue. I've upgraded an application to Rails 8 to use Solid Queue. However, when I run the command solid_queue:install, it just copies the template with the table names in the plural.
But in development, Solid Queue uses async so I only found out the issue in production when I got "could not find the table solid_queue_blocked_executions..."
Of course, it's an easy fix by just changing the name to singular inside of queue_schema.rb, but I'm wondering if this could be done already in the generator.

Oh yes! This came up in https://github.com/rails/solid_queue/pull/354 and, in Rails too, as Active Storage or Action Mailbox behave in the same way: https://github.com/rails/rails/pull/52970#discussion_r1770733592. I think for now this should be documented instead of handling it explicitly, just like it was suggested over in that Rails PR.

Thank you, @rosa. I'll close this issue.


Who will maintain this repo?

Now that I am no longer with ChartBoost, who will be maintaining this repo?
Perhaps we should give some permissions and/or redirect for the sake of the many users reliant on this?
ex, more than 1.2K public usages of 'chartboost/ruff-action' -->  https://github.com/search?q=chartboost%2Fruff-action&type=code
https://github.blog/2013-05-16-repository-redirects-are-here/

@CB-GuangyaoXie ?

Sorry for the off-topic comment: Could you please review/merge #24? Thanks :)

I'll take a look when at computer:

unclear whether I still have permissions.

need to look at your code, and consider risks if this introduces new dependencies

Update -- ya, def. don't have permissions.
I have suggested I am willing to accept transfer [ https://docs.github.com/en/repositories/creating-and-managing-repositories/transferring-a-repository ] to continue to maintain for the good of the community of users.

Is there any point of contact at ChartBoost to initiate the transfer?

@charliermarsh would it make sense to pull this under the astral umbrella? Seems like a core functionality for the linter.

Yeah, we can maintain an official GitHub Action.

We can transfer it under Astral if someone has permissions to do so.

@daniel-barros ?

👍 yeah I'm happy to maintain if it's tranferred

Tbd if you need to fork, unclear that chartboost legal will be ok with transfer [ had requested ]
Great to also get in gh action marketplace, while I was employed there I failed at getting needed approvals to get gh developer license signed which was a prerequisite
Ideally transfer occurs to better serve/transition the community of users.   That is why I hadn't forked myself...  without transfer, the best place for this to fork/live is likely in Astral.

I will give it a week or so. If we don't hear back, we'll just create a new repo in the Astral org.

I had permission to merge the PR. I think there is a bit of risk in adding an extra dependency, but it seems to be working fine. I'd like to wait until the repo is transferred so we can generate the release.

Hopefully I'm not getting too ahead of myself - but can Astral also press the necessary buttons to become a "verified" publisher in the actions marketplace? One of the security options for GitHub actions within paid accounts is to restrict 3rd party actions to only those published by verified publishers, and some organizations require this for various supply-chain-security reasons.
They document the necessary steps here: https://docs.github.com/en/apps/github-marketplace/github-marketplace-overview/applying-for-publisher-verification-for-your-organization

@ffernandez92 -- ideally this winds up transferred to https://github.com/astral-sh ...  I think GY has permissions to do that [ I imagine you do not? ] but perhaps waiting on [ lack of? ] legal approval?
There isn't anything company specific/proprietary in here.  Transfer would be the cleanest way to save the company from maintenance, AND esp. be kindest to the community by way of redirect, etc...  But 🤷 .

Any updates from someone who works at ChartBoost? It'd be great to provide uninterrupted service for the users of this.

Thanks all for your thoughtful consideration of ensuring a smooth transition for this action! Just adding appreciation and encouragement as part of a team that uses it for our open source web application. :)

@charliermarsh Can you help us on merging this https://github.com/ChartBoost/ruff-action/pull/29

@nitishxp we cannot merge pull requests here — ChartBoost has not taken any action towards transferring the project.

@zanieb  can we shift that repo under astra, that defect is literally blocking us for most of the things

@nitishxp we do not have control over transfer of this repository, as discussed above — ChartBoost needs to take action for a seamless transition.
We'll create our own action eventually, if it comes to that.

The action is released under Apache 2, which is fairly permissive. Why not fork it under astral?

If the repository is transferred instead, there won't be an interruption for existing users (usage be redirected).

Last I heard from employees [ I am no longer there ], this was going
through legal approval process to transfer.   I can ask whether stuck, a
determination, or in-progress still.  Also, I believe one guy who can
merge/maintain is on vacation currently, so tbd.  Forking is permitted, not
the worst idea, but certainly much better if this can be transferred.
On Sat, Aug 10, 2024, 9:03 AM Amin Sadeghi @.***> wrote:

The action is released under Apache 2, which is fairly permissive. Why not
fork it under astral?
—
Reply to this email directly, view it on GitHub
https://github.com/ChartBoost/ruff-action/issues/26#issuecomment-2282194925,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABGMTJDAUUZYMUGJ7AQTMN3ZQY2UTAVCNFSM6AAAAABGF4AQJGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBSGE4TIOJSGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

I think if we find time to prioritize it before the transfer completes, and there's no clear timeline, we'll just fork. But if the transfer completes before we get to it, that's obviously preferable.

This repo has a github-actions bot actively closing open PRs because they're "stale" in the amount of time it's taking to transfer this. Can we just accept that there will be some loss of continuity and fork? When Chartboost finally gets back to this they can change the readme to direct people to the fork.

Yeah we're just planning to fork it at this point. The bottleneck is finding time for us to prioritize taking over the action.

@charliermarsh -- if you want to fork it, I can commit to starting to maintain.  This is from the original author that no longer has permissions at ChartBoost.  I am confident I can get them to write a README informing of the fork.  Sounds like unlikley to get prioritization to get attention of lawyers.   If that makes sense, later this week I should also be able to update docs via PRs to point to new location.
Steps:

FORK
Grant Me Permissions
I'll work with ChartBoost to update the README to point to new location
I'll submit PR to update docs to point to new location
etc

Do advise if you want me to help move this forward.

Also happy to help.

Creating a fork is the best course of action for this project. Please submit a README update PR when you have the new location created.

@charliermarsh sorry for my impatience, I know everyone here is donating their time, but it feels like we have willpower to make this change and are getting mired down in process.
It sounds like @brucearctor is willing and able to manage these changes for astral, so we should be able to achieve this with 3 actions in the github UI that should take no more than a few minutes:

Fork the repo to astral-sh
Add @brucearctor as an outside collaborator to astral-sh (this link won't resolve until you fork)
Change permissions on collaborator as needed. I think "maintainer" is right but "write" may be enough.

I'll try to get to it today.

I've forked the action here: https://github.com/astral-sh/ruff-action. I'd like to make some substantial changes to it but I don't want that to block the fork.

I've forked the action here: https://github.com/astral-sh/ruff-action. I'd like to make some substantial changes to it but I don't want that to block the fork.

@charliermarsh -- I never saw any message after you forked it about my offer.  Is it safe to assume there is not a need/want for me to maintain the fork in astral-sh org?  [ from the original author who lost permissions to this repo in April ]

As per discussed, https://github.com/astral-sh/ruff-action is the new home of this project. Thanks for all the helps.
This repository will be archived(read-only) very soon. For anyone referencing this repo in your Action jobs, please update them to use astral-sh/ruff-action.


Update Contributors.md

add my name, Andrei Shauko
issue #490

@lapayx, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


get_safe_redirect_to: Fix handling of redirect_host.

5c9d56d2f7b3ec5b3daa42d7c11f000ff7a1885f fixed a bug with calling set()
on a string, but completely removed the utility of allowed_hosts in
is_safe_url.
This commit fixes up the whole handling of redirect_host more
comprehensively.
We need to support passing redirect_host with a scheme because the
typical usage of this function is something like
get_safe_redirect_to(redirect_to, user_profile.realm.uri)
where user_profile.realm.uri will contain the scheme, and it would be
very messy to require the caller to extract the host. That's why this
should be handled in the function.
@andersk Could you take a look too?

No strong use case, but the argument for this would be to avoid astonishment, or bugs, due to absolute redirects within the same hostname not working.
Without this PR:
In [2]: get_safe_redirect_to('https://chat.zulip.org/test', 'https://chat.zulip.org')
Out[2]: 'https://chat.zulip.org'

I think a call like this should be expected to work and return 'https://chat.zulip.org/test'. Do we consider that reason good enough to want this PR?

Yeah, that seems worth having to handle that situation.  So we should just add a test for that.

When would such a situation occur? Do we ever generate login links with absolute next parameters?

Yeah, I'm sorta leaning more in the direction of just adding an assertion that it wasn't called with anything absolute in next.


fix the layout issue in migration assessment dialog

I was testing the migration UI and noticed the scroll is going out of control, the 3 pars, databases/issues/issue details should scroll independently. also, for db list, the scroll should happen inside the list itself, the search box and the server warning list should always be visible as shown in the 'after' screenshot
before

after:

Pull Request Test Coverage Report for Build 682292761

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

Totals

Change from base Build 682291170:
0.0%

Covered Lines:
25464

Relevant Lines:
53684

💛  - Coveralls


Darcula theme being applied instead of Dark/New UI theme

Is there an existing issue outlining your problem?

[X] I have searched the existing issues and they do not solve my problem.

Have you followed the instructions in the the pinned issue?

[X] I have followed the instructions in the pinned issue and they do not solve my problem.

Describe your problem.
After updating to v3.2.0, the UI dark mode themes applies the Darcula theme rather than the Dark/New UI-based theme as it had previously done.
This affects Frappe, Macchiato and Mocha; Latte enables light mode and applies the Light with Light Header-based theme as expected.
The editor colour schemes update/apply as expected for all themes in all modes.
Attach screenshots.
What I see (Darcula)

What I expected to see (Dark/New UI)

What IDE(s) are you seeing the problem on?
IntelliJ IDEA 2023.3 EAP (Ultimate Edition)
Any additional comments?
No response

Isn't that related to the pinned issue?
https://github.com/catppuccin/jetbrains/issues/97

From what I see, no.
As you can see in the screenshots, Catpuccin is selected (and restarts and re-installs were done); and, as noted, the editor colour scheme isn't out of sync with the theme.

Oh ok that's the UI, sorry.
I cannot definitely reproduce the issue on the latest EAP. The issue is not here on 2023.2.

It was working fine on EAP before I updated the plugin to 3.2.0.
It's possible, though unlikely, that there was an EAP version update that took effect at the same time.

Yep can confirm, it works when downgrading to 3.1.1. As a workaround you can download it from the marketplace in the versions tab. Really strange that it works fine on 2023.2...


Add ability to filter alpha correlation results based on metadata category

This would save time by not having to run alpha correlation separately for different metadata categories of interest.

👍

@ceolder: I don't think this issue is relevant anymore --- alpha correlation supports multiple metadata categories in the interactive visualizations (merged in #44, available since 2.0.5). If I am misunderstanding this issue feel free to reopen and provide some additional details. Thanks!


provider fees calculation is broken

Something we're doing is breaking the provider fee calculation. I keep getting failed bids like this from the mainnet providers
Mar 08 19:11:34 mainnet-ams1-provider-master akash[3315893]: I[2021-03-08|19:11:34.572] broadcast response                           cmp=client/broadcaster response="Response:\n  TxHash: 1296B663CC13AB411D1869A60BCC878A94F69408A1A792BE4477198F7BA72E0E\n  Code: 13\n  Raw Log: insufficient fees; got: 0uakt required: 0uakt: insufficient fee\n  Codespace: sdk" err=null
Mar 08 19:11:34 mainnet-ams1-provider-master akash[3315893]: I[2021-03-08|19:11:34.572] bid complete                                 module=bidengine-order order=akash1vn06ycjjnvsvl639fet9lajjctuturrtx7fvuj/1715/1/1
Mar 08 19:11:34 mainnet-ams1-provider-master akash[3315893]: E[2021-03-08|19:11:34.573] submitting fulfillment                       module=bidengine-order order=akash1vn06ycjjnvsvl639fet9lajjctuturrtx7fvuj/1715/1/1 err="broadcast tx error: response code 13 - (&types.TxResponse{Height:0, TxHash:\"1296B663CC13AB411D1869A60BCC878A94F69408A1A792BE4477198F7BA72E0E\", Codespace:\"sdk\", Code:0xd, Data:\"\", RawLog:\"insufficient fees; got: 0uakt required: 0uakt: insufficient fee\", Logs:types.ABCIMessageLogs(nil), Info:\"\", GasWanted:0, GasUsed:0, Tx:nil, Timestamp:\"\"})"

We must be doing this to ourselves, because the other CLI utilities work fine.
Things known not to work
Setting env. vars. like this
AKASH_GAS=auto
AKASH_GAS_PRICES=0.025uakt
AKASH_GAS_ADJUSTMENT=4.5

Setting these env. vars
AKASH_FEES=6123uakt

Things known to work*
Adding --fees=5123uakt to the command line

remek's got the same issue when using --gas=auto https://discord.com/channels/747885925232672829/771909807359262751/949327496799739905
I think, IIRC, I have been getting the same.
Looks like the only solution is not to use --gas=auto and either use static --fees (minimum 5000uakt) as explained here https://discord.com/channels/747885925232672829/771909807359262751/949423492179296336
or, since --fees & --gas cannot be specified together, the following is also working:
AKASH_GAS_PRICES=0.025uakt
AKASH_GAS=200000

@arno01 this is fixed with your patches right?

@hydrogen18 right, fixed in the v0.14.2-rc1. (There is also v0.14.2-rc2 I've just noticed)
When can it get officially released?
As v0.14.1 has basically stopped working without that fix no matter how high you set the scalar gas value to.

@arno01 there are a few more things we want to get into v0.14.2. you can rely on v0.14.2-rc2

@arno01 if you use v0.14.2-rc2, you must run this first: https://github.com/ovrclk/infra-ops/blob/a74edcca530d888b7ee2e7b9cf73c7fb2499008b/roles/akash-provider-run/templates/orphan-manifest-purge

@arno01 if you use v0.14.2-rc2, you must run this first: https://github.com/ovrclk/infra-ops/blob/a74edcca530d888b7ee2e7b9cf73c7fb2499008b/roles/akash-provider-run/templates/orphan-manifest-purge

@boz I don't have an access to https://github.com/ovrclk/infra-ops

here you go @arno01:
#!/usr/bin/env bash

kubectl get mani -n lease -o 'custom-columns=name:{.metadata.name}' --no-headers | \
  while read ns; do

    # if ns exists, it's a fresh manifest.
    if kubectl get ns "$ns" >/dev/null 2>&1; then
      echo "FOUND: $ns"
      continue
    fi

    # make sure it's a NotFound error before deleting.
    nserr=$(kubectl get ns "$ns" 2>&1)
    if [[ "$nserr" == *NotFound* ]]; then
      echo "GONE: $ns"
      kubectl delete mani "$ns" -n lease
      continue
    fi
  done

@troian your changes to make auto work should have fixed this right?

correct


Tag model

Make sure that the tag model matches https://github.com/unitaryfund/metriq-api/blob/main/metriq-api/models/tagModel.js

Done


Provide output when creating or updating DNS records in flarectl

Since CreateDNSRecord now (#28) return the created record when called, we should provide it in the output of flarectl.

Added in 6fd9089a169e309fb3c8a31187517fdab8d566ee.


Add access control object to query filter factors. Update error messages

Issue: https://github.com/ytsaurus/ytsaurus/issues/294
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Fixed comments

Rebased to resolve a conflict (a file was renamed)


Should there be an EVP_PKEY_todata()?

We have the API EVP_PKEY_fromdata() which creates a new key based on a set of OSSL_PARAMs. It is possible to get param data from an existing key using EVP_PKEY_get_params. However this requires you to know up front all the params data that you might want to get and to set up the OSSL_PARAM array accordingly. It would be nice to have an EVP_PKEY_todata() function which creates an OSSL_PARAM array for you and populates it with all the relevant parameters required for a subsequent EVP_PKEY_fromdata() call to be successful.

Why not use the output from the EVP_PKEY_gettable_params call to get the params?
The gettable params would need to be cloned to avoid them being read only.

Are all required parameters (sufficient to re-create the key using EVP_PKEY_fromdata()) exposed by EVP_PKEY_gettable_params?

Are all required parameters (sufficient to re-create the key using EVP_PKEY_fromdata()) exposed by EVP_PKEY_gettable_params?

I'd consider that to be a bug in the provider if they aren't.
There is however a problem with usability of the simple "cloning" - for bignum parameters you do not know how big the data will be in advance and you'll have to call the getparams twice with preallocations. This is clearly visible in the EVP_PKEY_get_bn_param() implementation.

The internal export function passes a callback, which is passed an OSSL_PARAM array constructed by the provider. Just wrap that with a public API, where the callback comes from the caller. Done deal.

I think it would be more user/application friendly, to clone the OSSL_PARAM array for the user -- gets the memory allocated on the proper side of the provider boundary, lets the app take control of the life time etc.

I think that duplicating the OSSL_PARAM array should be a decision for the caller, not a unilateral decision that we make for them.

You need to look at what the export does before making this comment.
By the time the export has finished the params returned from the  callback are destroyed.

I agree with @slontis here.  We need persistence,

You need to look at what the export does before making this comment.

I'm fully aware how it works.

By the time the export has finished the params returned from the callback are destroyed.

The params recieved by the callback only exist until the callback ends, yes.  And?
Could there be a misunderstanding here?  When I said that we could simply make the currently internal export function (evp_keymgmt_util_export()) public, I really meant that, and that means that the application would provide the callback.  Therefore, it would be entirely up to the application what they want to do with the params that the callback receives.

In other words, I was seeing this:
int EVP_PKEY_todata(const EVP_PKEY *pk, int selection,
                    OSSL_CALLBACK *export_cb, void *export_cbarg);

But, from seeing discussions here and in #14785, it looks like we have very different ideas of what EVP_PKEY_todata() should offer.  Would you mind showing your idea in a bit more concrete form?

I'm fully aware how it works.

What you have above is fine.. I want the param data to be available after EVP_PKEY_todata() has been called. i.e within the
export_cb the user can use the OSSL_PARAM_dup() if they want to.

@levitte that is a pretty poor interface compared to the fromdata (other side).
That interface existing doesn't hurt to have (and exposed) - but we should be able to offer a simple interface that returns a usable OSSL_PARAM array and expect our users to actually use that interface in preference to having to run an iterator and arrange collection themselves.

I'm fine with both.  We could expose evp_pkey_util_export() as EVP_PKEY_export() if that makes everyone feel better.

A callback is a dreadful API for this.  An OSSL_PARAM array is what we accept as input, we should be able to output one.  It is much friendlier and easier to use.

I would still like to offer the choice to the user / application.  We do not know what the resulting OSSL_PARAM will be used for, but you can be sure that there are some who'd prefer not to waste more memory allocation and copying than necessary.
Can we settle on both?

Both is okay since we've got to live with the callback form internally, it shouldn't be a major maintenance burden.

Closed as #14800 is merged.


Check GraphicsCaptureSession.IsSupported

Not all SKUs have screen capture enabled (e.g. Xbox, Hololens, etc). Check IsSupported to make sure we don’t try to call out to the rest of the API.

Fixed with commit 9f6dd36e73bfd62f6515fd45cfa52552c64de07b


Improve ACL processing error handling

Ensure links are not kept lingering on all errors.

/test

/test

/test

/test


Syncing gcgrid S3 bucket with Dalhousie's FTP server?

Maggie Marvin at the University of Edinburgh asked whether the data on Dalhousie's FTP can be also uploaded to S3.
My questions are:

Which part of Dalhousie's data are not on S3 right now?

My reference is
http://wiki.seas.harvard.edu/geos-chem/index.php/Downloading_GEOS-Chem_source_code_and_data#Dalhousie_data_directory_archive
Looks like most of them are metfields and should largely duplicate s3://gcgrid/ ?
Further, custom nested fields can be cropped from the global data:
s3://gcgrid/GEOS_0.25x0.3125/GEOS_FP/

How large are those data? If it is small we can probably just upload them. If not then it involves talking to Amazon again.
Who is managing Dalhousie's FTP server currently? Is it possible to coordinate?

Dal is storing the native-resolution GEOS-FP (0.25 x 0.3125) and MERRA-2 data (0.5 x 0.625) on the ComputeCanada server.  I believe this is too large to store on Randall's server rain.ucis.dal.ca.
Jun Meng would be the best one to talk to re: Dal server support.

I'm happy to use s3://gcgrid/GEOS_0.25x0.3125/GEOS_FP/ and crop down to my domain (SE Asia), but I'm intending to run a nested simulation at 0.25x0.3125 resolution for 2017-present, and I was not able to find met fields for this resolution and timeframe on S3. However, it looks like met fields for these years might be available for some of the nested domains from the Dalhousie server at rain.ucis.dal.ca?

@MargaretMarvin as a temporary solution you can transfer those data from rain.ucis.dal.ca to EC2, and even upload them to your own S3 bucket for later use. Transferring data into AWS is free. The storage cost on S3 is $23/month and I don't think nested metfields would exceed 1TB.

I am interested in running a nested simulation at 0.25x0.3125 over China. I found that s3://gcgrid/GEOS_0.25x0.3125_CH/GEOS_FP/ only contains 2013 and 2016, whereas ftp://rain.ucis.dal.ca/ctm/GEOS_0.25x0.3125_CH.d/GEOS_FP/ contains met data from 2012 to now. Yes, it is not hard to transfer those data to EC2 or our own S3, but would be great if Amazon agree with you to upload those missing data.

Closing this issue since the Dalhousie server is no longer in use.


No such file or directory (os error 2) in Wiki.luau

Um this is my first issue so it may not be so good.
I tried to use lune run pilot-generate -- --wiki and it failed, because it is trying to remove a file that is not existing, which errors on my system.
OS: macos
Output: `✔ Accessing ROBLOX documentation took 467.22 ms
✔ Parsing Pilot.lua documentation took 31.00 ms
✔ Generating definition took 6.38 ms
✔ Generating documentation took 14.61 ms
✔ Generating modules took 3.62 ms
✔ Generating vscode took 1.29 ms
No such file or directory (os error 2)
[Stack Begin]
Script '[C]'
Script '__mlua_async_poll', Line 4
Script '.lune/generate/generators/Wiki.luau', Line 157
Script '.lune/generate/generators/Wiki.luau', Line 167
Script '.lune/pilot-generate', Line 65
Script '.lune/pilot-generate', Line 120
[Stack End]
`
What i did so it fixed:
155 if typeof(value) == "table" then
			if fs.isDir(`{path}/{name}`) then
				fs.removeDir(`{path}/{name}`)
			end
			fs.writeDir(`{path}/{name}`)
			recursiveWrite(value, `{path}/{name}`)
		else
			if fs.isFile(`{path}/{name}`) then
				fs.writeFile(`{path}/{name}.md`, value)
			end
		end```

Looks like my file system is less strict than yours, and lets me remove non existent directories (seems bizarre, we are both running macos, though, I am running catalina).
Your solution seems sound, besides the if fs.isFile({path}/{name}) then check (would this not just prevent all generation from occurring?). I have implemented the first half of the solution, but excluded this aspect.


Update version from 3.1.0-SNAPSHOT to 3.1.1-SNAPSHOT

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to Kylin?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[ ] I have create an issue on Kylin's jira, and have described the bug/feature there in detail
[ ] Commit messages in my PR start with the related jira ID, like "KYLIN-0000 Make Kylin project open-source"
[ ] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If this change need a document change, I will prepare another pr against the document branch
[ ] Any dependent changes have been merged

Further comments
If this is a relatively large or complex change, kick off the discussion at user@kylin or dev@kylin by explaining why you chose the solution you did and what alternatives you considered, etc...

Pull Request Test Coverage Report for Build 6100

0 of 0   changed or added relevant lines in 0 files are covered.
8 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.002%) to 27.977%

Files with Coverage Reduction
New Missed Lines
%

tool/src/main/java/org/apache/kylin/tool/query/ProbabilityGenerator.java
1
83.33%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
3
78.42%

core-metrics/src/main/java/org/apache/kylin/metrics/lib/impl/MetricsSystem.java
4
61.67%

Totals

Change from base Build 6065:
0.002%

Covered Lines:
26111

Relevant Lines:
93331

💛  - Coveralls


Explore CLI tool

Describe the solution you'd like
It would be nice to generate components from a cli tool. This way the sketch files can live next to the components.
Use case I think of is probably most value when people have a sort of ui kit/ designsystem.
Issue created for the discussion

Scenario: export to xLayers Labs, or share on xLayers Labs.

Oh yes, I can imagine something like this:
npm i -g xlayers-cli
xlayers-cli --in myFile.sketch --template angular --out ./path/folder

Hi @Jefiozie and @manekinekko , If it is open and still in the roadmap, I would like to work on it after completing #431

Hi @sumitparakh this feature is blocked by #320.
@Jefiozie what's the status of #320?


Angular MsalInterceptor: add access token to custom header

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.19.0
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.5
Description
Is it possible to configure the interceptor to add the access token in a custom header in addition to the Authorization header?
 I need to keep the token in another header.
Alternatively, is it possible to create a second custom interceptor in parallel to the MsalInterceptor for this need?
Thanks in advance.
MSAL Configuration
No response
Relevant Code Snippets
No response
Identity Provider
No response
Source
External (Customer)

@eliott-loreal We recommend adding the accessToken to the custom interceptor and call it after MsalInterceptor if you need to customize this experience. We do not support adding accessToken to other headers yet.