TC-BINFO-1.1 needs to be updated as per the test-plan

Summary Title:
TC-BINFO-1.1 test script needs to be updated as per the Test-plan
Description:
TC-BINFO-1.1 testcase is updated  the Attribute list , Hence the Yaml script is also needs to be updated as per the Test-plan
Additional Info:
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/basic_information.adoc#tc-binfo-1-1-global-attributes-with-dut_server

This testcase is removed from the test-plan and test-script hence closing this issue.


2nd Commit 4 BX Bitphoenix x Nov17

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

i am watitng for PS merge

Please read the SPL-token README, and ensure your SPL-token info uses the correct Uniswap format.


"Enter a name for your Microsoft DevBox" TextBox expands to the width of the error message

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
Select "Microsoft DevBox" from the "Configure your environment" flow.
enter "1" in the textbox for "Enter a name for your Microsoft DevBox"
Expected result
The textbox size is consistent.
Actual result
The width of the TextBox expands to the width of the error message.

Included System Information
No response
Included Extensions Information
No response

The other elements of the page should also have a constant width :


Setting desired_capabilities

I'm running a crawler with the selenium_firefox engine but I got the following exception Selenium::WebDriver::Error::InsecureCertificateError:. As I see on this link https://github.com/SeleniumHQ/selenium/wiki/Ruby-Bindings#ssl-certificates it's something I need to configure on firefox.
Is it possible to set the option desired_capabilities through Kimurai?

@vifreefly Hi, I created this PR https://github.com/vifreefly/kimuraframework/pull/46 just as note about how I fixed the issue. I would like to know how you think is the best way to add this option and of course, to know if makes sense to add this option to the project.
Thank you.


spaces in $HOME

Hy!
My $HOME env variable contains spaces... it's an osx installation with an ssd and a hdd. I call the hdd: "Data HD" and it's the host for my home folder..
Both install.sh and the default .zshrc fails with this setup. I fixed it by replacing ZSH=~/.oh-my-zsh lines in both to ZSH="$HOME/.oh-my-zsh"
Now it seems to work :)

Oh crap! Spaces in directories should totally be supported.
We usually are not completionists about quoting, because it's not required in zsh*. But the install script runs under sh, so we need to quote aggressively there. Will fix this. Thanks for the heads-up!
*Unless you have certain options set, but emulate -L zsh is the way to protect against that.

You are totally right, reverted $ZSH in my .zshrc to the original and it still works!
I was confused because I first changed the installer to $ZSH="~/.oh-my-zsh" and then it cloned the config into "/path to my home/~/.oh-my-zsh"

After thinking about this a bit, I'm not sure OMZ should support it after all. In general, paths with spaces and other special characters should be supported by our code. But the particular case of a home directory with spaces in its name may be different. We could fix the installer and ~/.oh-my-zsh to work with this. But a lot of other things, both in OMZ and other other software, are probably going to break in this case, with failure modes that could include significant data loss. Lots of software doesn't expect or protect against this case, especially anything that uses tilde expansion in shell scripts.
We might be better off having OMZ check for spaces in the home directory, and refusing to install or run in this case. Thoughts, @mcornella and/or @robbyrussell?
If I were you, I would rename my HD or make some other configuration change so the path to your $HOME directory doesn't have spaces in it. You're likely to run in to a lot of other Unix things that break under that setup, even if OMZ works.

Well... what you're saying is totally makes sense, but on the other side, I've been using this setup for years and had no serious problems after all. Once or twice, I had a problem when trying to build software from source, but with some quotation marks, I was always able to fix it. The biggest problem I can remember that the Qt developer package does not work when installed on a path that contains spaces, but I found a neat warm place for it under /opt :) It seemed to be easier than changing my $HOME. I'm a bit afraid of what could possibly break if I suddenly change my home to a different location.
After all it seems to be a good reminder for me when working on my own sw projects, not to forget the quotation marks :))
Anyway in my opinion, refusing the installation might be a bit too strict, but a warning with an additional confirmation seems to be a good idea.
p.s.: OMZ seems to work correctly on this setup, I've already enabled some plugins and played around with it since yesterday, and found no additional problems.

On Oct 9, 2015, at 2:42 AM, Andrew Janke<EMAIL_ADDRESS>wrote:
After thinking about this a bit, I'm not sure OMZ should support it after all. In general, paths with spaces and other special characters should be supported by our code. But the particular case of a home directory with spaces in its name may be different. We could fix the installer and ~/.oh-my-zsh to work with this. But a lot of other things, both in OMZ and other other software, are probably going to break in this case, with failure modes that could include significant data loss. Lots of software doesn't expect or protect against this case, especially anything that uses tilde expansion in shell scripts.
We might be better off having OMZ check for spaces in the home directory, and refusing to install or run in this case. Thoughts, @mcornella https://github.com/mcornella and/or @robbyrussell https://github.com/robbyrussell?
If I were you, I would rename my HD or make some other configuration change so the path to your $HOME directory doesn't have spaces in it. You're likely to run in to a lot of other Unix things that break under that setup, even if OMZ works.
—
Reply to this email directly or view it on GitHub https://github.com/robbyrussell/oh-my-zsh/issues/4470#issuecomment-146726515.

Once you have it installed, core OMZ itself should work pretty well, because it uses zsh internally, which normally doesn't care about spaces in variables. It's the things its various functions and plugins call out to that might use other shells or parsing logic internally that would be problems. And we've got a lot of plugins. I'd just be really worried that some installer or something is going to do a -e ~/Dropbox existence check, it'll return a spurious false, and it'll feel free to clobber a bunch of your files because it thinks it's working with a blank slate. I seem to remember a system updater that had a bug like that a few years back which would wipe your entire hard drive if it got triggered. (And just in general, I'd be nervous with a setup that required extra care in bash or sh. But hey, it's your machine...)
On the other hand, if it's working for you, and especially if we put out a warning in the OMZ installer, yeah, we probably shouldn't be that nannyish. And it wouldn't be a ton of work to do. Curious to hear what the core OMZ developers think.


When one of the Multiple push notification present in the tray is touched others vanish in android

If there are three or more notifications in the notification Tray (received when app is in background) and when one of them is touched app goes into active state and similarly when second is touched .But after the second is touched from the tray all other notifications from the tray are vanished.

The patch in pull request #333 fixes this.

Could you please give me details on how to go this step?
Integrate the PushApps SDK into your app

Add PushApps.framework to your Xcode project. Please make sure that the "Copy items into destination group's folder (if needed)" is NOT checked.

How should I add the PushApps.framework in my project? Could you please give me step by step on where I click in the Xcode environment?
Thanks
—
Sandro
On Wed, Jan 14, 2015 at 6:52 PM, Jesper We<EMAIL_ADDRESS>wrote:

The patch in pull request #333 fixes this.
Reply to this email directly or view it on GitHub:
https://github.com/phonegap-build/PushPlugin/issues/443#issuecomment-69989317


Trigger Happy new face

I'm testing a lot of things arround the UI, which is pretty basic today :)
So I made course on https://www.codecademy.com/ and after AngularJS, now its ReactJS :)
I try to find something, that I can understand quickly, to start to write something for TriggerHappy.
But whatever the choice I'll do to make a new UI, with such a "modern" Javascript project, I wont throw in the bin the actual work I did with the wizard.
At the end, my goal will be to propose a project that could be :

use with a *JS Front + DRF as back
use as it is today, simple and efficient, with every needed things for its work

If someone is interested final choices are:

VueJS
DjangoRestFramework

if you feel good with that, let's start :)

Hi,
short question, hadn't a look at it right now.
Is there already a REST-API in place, or does this needs to be written from the ground up?
So this would lead to two tasks:  Frontend| Backend...
Would be happy to help you with this!

Actually DRF is used to handle 2 webhook from Taiga and Slack/Mattermost.
All have to be done :)
Then, yes two tasks to do.
I try all of this with a notes taken app, but nodejs and webpack are not my friends :) I need to dive into webpack at least.

Could be useful https://ariera.github.io/2017/09/26/django-webpack-vue-js-setting-up-a-new-project-that-s-easy-to-develop-and-deploy-part-1.html

Won't be used , plan to rewrite everything with starlette


Requests Subpackage Not Supported by gopkg.in

The README recommends grabbing the V3 version of the code at gopkg.in/dgrijalva/jwt-go.v3.
go get gopkg.in/dgrijalva/jwt-go.v3 and import jwt "gopkg.in/dgrijalva/jwt-go.v3" works for the main jwt package but will not work for the requests sub package as it performs an absolute import on the main jwt package via github.
I found this out by running http_example_test.go and the compiler erroring that  requests.ParseFromRequestWithClaims was not finding the "github.com/dgrijalva/jwt-go" import.

Never-mind this is now working for me

I falsely closed this due to a cached library

This is a limitation of the gopkg.in tool, which I don't maintain. With tools like dep quickly becoming mainstream, I'd prefer to target those. gopkg.in will continue to work as long as the maintainer keeps it running, but I'm going to remove it as a recommendation for future versions of this library.


BLEPeripheral.begin function not executing

Having powered up my new nRF8001 module, it isn't showing up on any bluetooth devices ( I am primarily using the Bluefruit LE app recommended by ADA Fruit, on iPhone 5s, iOS 8.3 ).
Having traced the code, blePeripheral.begin (); isn't working in main.cpp (approx. line 68)
This is because nRF8001::begin isn't working in nRF8001.cpp
Because waitForSetupMode(); isn't working in nRF8001.cpp (approx. line 201)
I have traced this to the "while (!setupMode){" loop in nRF8001.cpp (approx. line 778)
Where and why does "setupMode" get activated is this a software issue, or is it triggered by a hardware input? I will continue tracing, but any thoughts would be much appreciated.
bool lib_aci_event_get(aci_state_t *aci_stat, hal_aci_evt_t *p_aci_evt_data)
{
  bool status = false;

  status = hal_aci_tl_event_get((hal_aci_data_t *)p_aci_evt_data);

  /**
  Update the state of the ACI with the
  ACI Events -> Pipe Status, Disconnected, Connected, Bond Status, Pipe Error
  */
  if (true == status)
  {
    aci_evt_t * aci_evt;
  ...

What causes the “lib_aci_event_get(&this->_aciState, &this->_aciData” condition to be met?

I'd try starting at a higher level.
What are you passing to BLEPeripheral()? Consult the pinouts chart for recommendations. For the Adafruit nRF8001, stick with the default BLE_REQ 10, BLE_RDY 2, BLE_RST 9 to start.
What does your wiring look like? Does it match the pinouts?
Can you get the test example to run on your hardware?

Thanks @don!
@georgeedwards for further assistance, please provide answers to the questions Don asked in the previous comment.

Hi everyone, I have done some further fault finding and although the wiring was correct, my SPI ports on Mbed were faulty. As such, I have bought a new one and now have the library working!
Thanks for your help.

@georgeedwards great!


Re design error handling

Codecov Report
Attention: Patch coverage is 53.88016% with 939 lines in your changes are missing coverage. Please review.

Project coverage is 85.11%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #266      +/-   ##
==========================================
- Coverage   89.40%   85.11%   -4.29%
==========================================
  Files         123      122       -1
  Lines        7995     8668     +673
==========================================
+ Hits         7148     7378     +230
- Misses        847     1290     +443

Files
Coverage Δ

include/async_mqtt/client.hpp
100.00% <ø> (ø)

include/async_mqtt/endpoint.hpp
100.00% <ø> (ø)

...sync_mqtt/impl/client_acquire_unique_packet_id.hpp
100.00% <ø> (ø)

...mpl/client_acquire_unique_packet_id_wait_until.hpp
100.00% <ø> (ø)

include/async_mqtt/impl/client_impl.hpp
91.05% <100.00%> (ø)

...l/endpoint_acquire_unique_packet_id_wait_until.hpp
97.91% <100.00%> (ø)

...de/async_mqtt/impl/endpoint_get_stored_packets.hpp
96.42% <100.00%> (ø)

include/async_mqtt/impl/endpoint_impl.hpp
96.72% <100.00%> (+0.25%)
:arrow_up:

...de/async_mqtt/impl/endpoint_register_packet_id.hpp
96.87% <100.00%> (+0.32%)
:arrow_up:

...e/async_mqtt/packet/impl/copy_to_static_vector.hpp
96.96% <ø> (ø)

... and 62 more

... and 1 file with indirect coverage changes


Add heroku and eslint configurations to project

here too, please be more specific and add as much details as possible.


[E2E] Add Tenderly support and E2E tests for Sei Network

This task is to add e2e tests to check that the Sei v2 network deployment is working correctly.

Tenderly for multichain added. Deleting for now as e2e tests with Ethereum already test most of the functionality without potentially highlighting chain-specific issues such as RPC issues found in Sei


Which bits to ignore when the secret agreement is finished

Hello !
Computer science student here, I was fidgeting around with the library, and i saw that on the secret agreement method using p751, there is the following comment : Output: a shared secret SharedSecretB that consists of one element in GF(p751^2) encoded in 188 bytes.
Now, 751 * 2 = 1502, and 188 * 8 = 1504. That would mean that two bits are redundant, right ?
However, when running the following mock program :
#include <stdio.h>
#include "P751_api.h"

#define BTBP "%c%c%c%c%c%c%c%c"
#define BTB(byte)  \
  (byte & 0x80 ? '1' : '0'), \
  (byte & 0x40 ? '1' : '0'), \
  (byte & 0x20 ? '1' : '0'), \
  (byte & 0x10 ? '1' : '0'), \
  (byte & 0x08 ? '1' : '0'), \
  (byte & 0x04 ? '1' : '0'), \
  (byte & 0x02 ? '1' : '0'), \
  (byte & 0x01 ? '1' : '0')

int main() {

	unsigned char APri[47];
	unsigned char APub[564];
	random_mod_order_A_SIDHp751(APri);
	EphemeralKeyGeneration_A_SIDHp751(APri, APub);

	unsigned char BPri[48];
	unsigned char BPub[564];
	random_mod_order_B_SIDHp751(BPri);
	EphemeralKeyGeneration_B_SIDHp751(BPri, BPub);

	unsigned char secretAgreementA[188];
	unsigned char secretAgreementB[188];

	EphemeralSecretAgreement_A_SIDHp751(APri, BPub, secretAgreementA);
	EphemeralSecretAgreement_B_SIDHp751(BPri, APub, secretAgreementB);

	printf(BTBP"\n", BTB(secretAgreementA[0]));
	printf(BTBP"\n", BTB(secretAgreementA[187]));

	return 0;
}

we see no particular pattern in either the first or last byte. My question is, where are the two missing bits ?
Thanks in advance,
MRandl

Elements (a+b*i) over GF(p751^2), where "a" and "b" are defined over GF(p751), are encoded as {a, b}, with "a" in the lowest memory portion. In turn each of these elements over GF(p751) are encoded as 94 octets in little endian format (i.e., the least significant octet is located in the lowest memory address).
This means that the top bits of elements "a" and "b" are clear (in your example top bits of the bytes secretAgreementA[93] and secretAgreementA[187] should be zero).

Thanks a lot! Sorry to have bothered!


Add blur event on mat-row in mat-table

Bug, feature request, or proposal:
In Material table, we con only set click event on mat-row. When working with inputs and dynamicals rows with formarray, It should be great to have possibility to add a blur event on the row.
What is the expected behavior?
Add a blur event on mat-row in Material table.
What is the current behavior?
Only click event is possible.
What are the steps to reproduce?
`

                <ng-container matColumnDef="select" sticky>

                            <mat-checkbox (change)="$event ? masterToggle() : null" [checked]="selection.hasValue() && isAllSelected()"
                                [indeterminate]="selection.hasValue() && !isAllSelected()">
                            </mat-checkbox>

                            <mat-checkbox (click)="$event.stopPropagation()" (change)="$event ? toggleRow(row) : null"
                                [checked]="selection.isSelected(row)">
                            </mat-checkbox>

                </ng-container>

                <ng-container matColumnDef="product">
                     P.U. HT

                        {{ row.value }}

                </ng-container>

`

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
For next release of Angular Material.
Thanks ! Best regards

I suspect if we add blur to this, then it would follow that we should add blur to every element in our component library. Im not sure its feasible to do so

@andrewseguin , I understand perfectly, thanks for your attention.
Best regards


To be discussed: Do we really need a whole system-upgrade in install_prereq.sh?

Hi,
Just for discussion:
I'm unsure if it's a good idea to execute "sudo apt-get upgrade -y" within install_prereq.sh. Not everyone wants a full system upgrade just to install tuya-convert. Some users won't even look into this file and just excute it, which may lead to long execution time.
I would only state in the readme, that you should have a more or less uptodate system.
Thanks!

This is a fair point, one I've considered myself. The fear is that out-of-date systems will lead to more frivolous bug reports. We can try it without for some time and see if it leads to an increase in issues. Thanks for your feedback.


Added desired solution

Naive approach

this is amazing @Anshumanshukla46
thanks for the contribution in my repo and please share in social media and tag me on twitter


feat(harborcluster): update harborcluster cr status

Signed-off-by: wangcanfeng<EMAIL_ADDRESS>

please squash the commits


How to use rpi5 iommu for passthrough?

I noticed that there are IOMMU device nodes in the device tree of the Raspberry Pi 5. In Xen, how can I enable these IOMMU devices?
When starting a DomU with Dom0, if I want to perform device passthrough, I need to enable IOMMU in Xen to achieve it, right? And it is necessary to have "I/O virtualisation enabled" printed when Xen initializes.

Hi @LJKingd
Problem with RPi5 is that it does not have IO-MMU that support Stage 2 translations. So, on this platform it is not possible to get classic device-passthrough.

iommu@5100 {
compatible = "brcm,bcm2712-iommu";
reg = <0x10 0x5100 0x00 0x80>;
cache = <0x2b>;
#iommu-cells = <0x00>;
phandle = <0x53>;
}

@lorc Thank you very much for your reply.
I noticed that there is such an IOMMU node in bcm2712-5-b.dts. Does this mean that the Raspberry Pi 5 has an IOMMU module? If so, why can't it be used?
iommu@5100 {
compatible = "brcm,bcm2712-iommu";
reg = <0x10 0x5100 0x00 0x80>;
cache = <0x2b>;
#iommu-cells = <0x00>;
phandle = <0x53>;
}

Yes, it in fact has IOMMU module, but it is for Stage 1 translation only. I.e. it only can be used by Linux kernel.
This IOMMU covers only small part of address space. Its purpose is to create physically contiguous memory regions from discontiguous buffers for devices that does not support scatter-gather DMA operations. It is more like TILER module from TI chips than real IO-MMU like ARM's SMMU or Renesas's IPMMU.

@lorc  Thanks for your answer. I understand now.


gRPC Unity package still doesn't work after 2.26

What version of gRPC and what language are you using?
grpc_unity_package.2.35.0-dev202012021242 + grpc-protoc_macos_x64-1.35.0-dev from here.
What operating system (Linux, Windows,...) and version?
MacOS Catalina 10.15.6
What runtime / compiler are you using (e.g. python version or version of gcc)
Unity 2019.4.16f1 LTS
What did you do?
I downloaded the last Unity package (2.35.0) from https://packages.grpc.io/, extracted it to the Asset/Plugins folder.
What did you expect to see?
No errors, successful MacOS build.
What did you see instead?
Error: Could not load signature of Google.Protobuf.ByteString:get_Span due to: Could not load file or assembly 'System.Memory, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. assembly:System.Memory, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 type: member:(null) signature:
Unloading broken assembly Assets/Plugins/Google.Protobuf/lib/net45/Google.Protobuf.dll, this assembly can cause crashes in the runtime

I get those errors on every version after 2.26.

@virtumonde Could you suggest a workaround while 2.35 unity package broken?
I have a .proto file and need to generate a client for unity.
https://packages.grpc.io/archive/2019/12/a02d6b9be81cbadb60eed88b3b44498ba27bcba9-edd81ac6-e3d1-461a-a263-2b06ae913c3f/index.xml
This package does not raise any errors in the unity console, but the generated file is broken.
protoc -I=./Assets/Protos --csharp_out=./Assets/Protos Assets/Protos/router.proto
protoc --version
libprotoc 3.14.0

I think this error occurs because I use the latest protoc version.

@virtumonde Could you suggest a workaround while 2.35 unity package broken?
I have a .proto file and need to generate a client for unity.
https://packages.grpc.io/archive/2019/12/a02d6b9be81cbadb60eed88b3b44498ba27bcba9-edd81ac6-e3d1-461a-a263-2b06ae913c3f/index.xml
This package does not raise any errors in the unity console, but the generated file is broken.
protoc -I=./Assets/Protos --csharp_out=./Assets/Protos Assets/Protos/router.proto
protoc --version
libprotoc 3.14.0

I think this error occurs because I use the latest protoc version.

Hi @rdcm. I think you are not compiling using the protoc corresponding to grpc_unity_package.2.26.0-dev. Download the protoc corresponding to your OS from here (one of these). I am also using 2.26 and after typing ./protoc --version I get libprotoc 3.8.0, not libprotoc 3.14.0.

Hi @rdcm. I think you are not compiling using the protoc corresponding to grpc_unity_package.2.26.0-dev. Download the protoc corresponding to your OS from here (one of these). I am also using 2.26 and after typing ./protoc --version I get libprotoc 3.8.0, not libprotoc 3.14.0.

@virtumonde Yes, you right I did not see the protoc in the archive only plugins for different languages. Thanks!

@virtumonde Yes, you right I did not see the protoc in the archive only plugins for different languages. Thanks!

@rdcm Np! Is there a chance you're familiar with grpc-web? I have a problem with settings it up. It think it is related to Envoy.

@rdcm Np! Is there a chance you're familiar with grpc-web? I have a problem with settings it up. It think it is related to Envoy.

Nope, for load balancing, I'm going to set up Nginx and I haven't any experience with Envoy.
If it will be interesting for you I can share the config when my application will be done.

Nope, for load balancing, I'm going to set up Nginx and I haven't any experience with Envoy.
If it will be interesting for you I can share the config when my application will be done.

That would be great!

That would be great!

Looks like this issue is a duplicate of https://github.com/grpc/grpc/issues/21908

Looks like this issue is a duplicate of https://github.com/grpc/grpc/issues/21908


[Workshop] Feature Management for your App's Deployments feat. LaunchDarkly |  March 2nd, 2023

🎤 Speaker Info

Name:  Tali Friedman
GitHub Handle:
Bio:  Solutions Engineering Manager at LaunchDarkly
LinkedIn:
🔗 Workshop Links
Zoom Link: https://github.zoom.us/j/97013759503?pwd=aDd5NmNKQTNsbGlLQWhuSm4vVEFGdz09
Eventbrite link:  https://www.eventbrite.com/e/feature-management-with-deployments-with-launchdarkly-tickets-532718885527?aff=ebdsoporgprofile
Notion Card Link: https://programequity.notion.site/Feature-Management-with-Deployments-with-LaunchDarkly-061fc508716144cbb6587c7776343c33

For Speakers:
Presentation Overview

Presentation Materials

[ ] Link to slides:
[ ] Link to additional resources (optional)
[ ] Link to post-session feedback survey (optional)

Operations

[ ] Send the following copy in chat following your presentation: https://github.com/ProgramEquity/open-source-mentorship/blob/main/Workshops/post_workshop_links.md
[ ] Send organizer links to slide deck and materials to put in this repo

For Organizers:
Pre-workshop

[ ] Speaker confirmed for the date and time with calendar invite
[ ] Slide content in shared drive
[ ] (optional) zoom poll set up
[ ] (optional) dress rehearsal if requested

Post-workshop

[ ] content updated in https://github.com/ProgramEquity/open-source-mentorship/blob/main/Workshops/
[ ] speaker filled out post-workshop survey
[ ] Content updated in Notion card and recording uploaded

moved to https://docs.google.com/document/d/1I5pTYN7ZiqQjWNSZAOPG2z-HCEPL9SFjXeQzyRnkwWc/edit#


update training link

Preview: https://45-293151248-gh.circle-artifacts.com/0/intersect-training.github.io/training-links/index.html


Support = in record initialisation

Frontend tests are failing. They format code (using unparser.rs), so even though they haven't been updated, they're getting the updated syntax.
I think we should update the TS to generate and expect the new syntax.


Add Linkin Park - No More Sorrow by Linkin Park

Song Name
Linkin Park - No More Sorrow
Artist Name
Linkin Park
Source
Youtube
Youtube Link
https://youtu.be/rW4uBvP2Dqc?si=TlAiMPtOBUVaxHh7
Direct File Link
No response
Song ID
814267
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 110849063
Downloaded: No

accept


feat: Added validation for input of extract function and var

Also performed some refactoring of tests to make it easier to create these test cases. Might move validation into get input, but that might break tests in current structure. Also noticed that extract function to file has no tests, let me know if I should make issue for that.
Addresses #62

I am happy you tested the empty input.  always needed.
Thanks for all the contributions.


Added more expansive orientation support

The current Orientation enum isn't really accurate - there are many types of orientations that iOS supports so I think those should be properly represented in the enum.

Hello @AdiAyyakad! A few days ago we have finally created the 2.0 release and I have taken over the maintenance of DeviceKit from Dennis. I will take a look at your pull request and discuss it with @Zandor300 - our main contributor - on how to proceed further. Many thanks in advance for your time and effort!

These can also be used for the implementation of isPortrait and isLandscape.
var isPortrait: Bool {
    return UIDevice.current.orientation.isPortrait
}

var isLandscape: Bool {
    return UIDevice.current.orientation.isLandscape
}

var isFlat: Bool {
    return UIDevice.current.orientation.isFlat
}

@Zandor300 So my hesitation with adding those at all is because they're not binary values. Like orientation isn't either portrait, or it isn't. You can be flat, but still have to orient for a portrait view.
https://github.com/AdiAyyakad/TestUIDeviceOrientation
I just whipped up a sample project that listens to orientation changes and it doesn't always report the right value.

@AdiAyyakad Oh yeah! I've installed your project on my iPhone.
What I think we should do, is listen to value changes, and remember what the last option was: Portrait and Landscape. When we call isLandscape, and UIDevice.current.orientation.isFlat == true, instead, we should return what the last known landscape vs portrait value was.
Personally think this should be an opt-in functionality since this isn't relevant for some apps and we shouldn't be wasting unnecessary resources on this. Additionally, the above functionality contradicts the way UIDevice works and DeviceKit is supposed to be a value-type replacement for UIDevice. Not everybody wants .isLandscape == true + .isFlat == true at the same time.
So when you opt-in for this functionality, you can enable the monitoring for the value:
var isOrientationMonitoringEnabled: Bool {
    didSet {
        if isOrientationMonitoringEnabled {
            NotificationCenter.default.addObserver(...)
        } else {
            NotificationCenter.default.removeObserver(...)
        }
    }
}

to match UIDevice.current.isBatteryMonitoringEnabled

Hmm I see where you're going with this, but yeah to me it just feels like DeviceKit should be more value typed and this might add some extra logic that it doesn't really need. But overall up to y'all!

I have found though that UIApplication.shared.statusBarOrientation.isLandscape is a more consistent API for landscape detection since it's related to the content orientation on the screen rather than the physical device. Maybe we can just use that?

@denisenepraunig and I discussed this and we think we should split up the two types of orientations:

.physicalOrientation
.screenOrientation

That way we reduce confusion and allow the user to choose which one they want/need.
The current variable .orientation would become an alias of .physicalOrientation and then be deprecated to push users to move to one of the two.
I think that would be the best option moving forward. Obviously this is still up for discussion if you come up with a better implementation @AdiAyyakad.

@Zandor300 no that sounds great to me!

@AdiAyyakad Okay, great! Then feel free to implement that.


[Feature] pro-layout compatible with ant-design@v4

🥰 需求描述 Description of Requirement
The current version of ant-design-vue is 4. One of the changes was to remove less and all the less styling files. In the migration guide there is a description on how to move away from it and use the old variables defined in less.
pro-components is currently using version 3 of ant-design-vue and imports less files directly. As we use both, we cannot update ant-design-vue until pro-components is also compatible with  it. Would you please update to v4?
🧐 解决方案 Solution
Update to ant-design-vue v4
🚑 其他信息 Other information
(https://antdv.com/docs/vue/migration-v4#less-migration)

我改了一版v4的
npm i antdv-pro-layout


Add Microsoft translate app to power toys run plugins

Description of the new feature / enhancement
I would like to see the option to translate from another language as a plugin pulled from the Microsoft Translator app into Power Toys Run. Being able to globally translate instantly without having to open any app would be amazing.
Scenario when this would be used?
I would be able to use this constantly anytime that I come across a language I don't speak which is most of them as I only speak English. However, I'm constantly running into other languages such as Chinese and and Russian. Being able to quickly hold down alt and space to copy and paste to translate would not only be convenient but would escalate and enhance my workflow.
It would also increase my usage of Microsoft Translate instead of relying on 3rd party translating as it would be convenient and within a few hot keys away.
Supporting information

you can try PowerTranslator

/dup #18014


add setup docs and firmware script

implements setup.sh, which allows one-liner installation of dependencies by streaming the script to bash
adds firmware setup instructions to SETUP.md
creates DEVELOPMENT.md doc for describing important dev info, such as running tests
removed map extract section - this needs to be part of general usage docs

Todo

Has not been tested on a Pi yet. CAUTION: very likely to brick your Pi!
Add installation of curb-wheel server, and code to run the daemon on start
Recommend reboot at end of docs?

adding #28 to enable dev mode configuration that uses local wifi network instead of pi hosted AP

merged startup scripts into a single npm start command in commit
317fb0bb0266b4103fcd43edf238390e5fb8393f


Animated SVG slows down Safari

This is when using latest Safari on latest macOS to view the repo home/readme with MacBook Pro M2 Pro.
It really is unbearable. I resorting to reading the Markdown sources without rendering.

Thats not great, we can probably replace it with a GIF.

I think GitHub also supports mp4 now in Markdown. Though not sure if these videos would be autoplaying...

It also performs poorly on chrome

woopsie! Will try a video instead.


Add option to have info buttons

Similar to #521, but instead of tooltips this adds info icons that you can hover over to see the text.
{
  "type": "string",
  "title": "Name",
  "options": {
    "infoText": "Your full name"
  }
}

This pull request was made from the wrong branch (master instead of feature/info-buttons) so I had to recreate the pull request to be able to use the master branch. Replacement pull request: #719


Default thumbnail is broken for file (not image) uploads

Vue2-dropzone Version<EMAIL_ADDRESS>Node Version (node -v):  v6.11.1
NPM Version (npm -v): 5.3.0
OS: Ubuntu 16.04.2 LTS

Description:
Adding file manually brakes default (blue) thumbnail. I'm uploading a pdf file, so I don't generate the preview.

Steps To Reproduce:

Use code from demo
Change showSuccess function to this

showSuccess: function (file, response) {
    this.$refs.uniqueID.manuallyAddFile(
        { name: response.filename, size: response.size },
        null,
        null,
        null,
        {
            dontSubstractMaxFiles: true,
            addToFiles: true
        })

    console.log('A file was successfully uploaded')
}

Thanks for the report @kmaxat , I'll take a look into it and see what I can find.

@kmaxat It's really weird, see following image, which I reproduced from your given step. Completely fine!

Hi folks,
Version 3 is now released. Check out the new docs for more info.

I have the same issue. When you add manually a pdf file, the preview is broken. How can we have the default blue thumbnail?

This PR is the solution for this error.
If you are getting error on preview for non image then you can use the above PR.
https://github.com/rowanwins/vue-dropzone/pull/343

This issue has been fixed in the recent release. Check V3.3.0


fix: move taskresult reconcile earlier to avoid workflow stuck. Fixes: #13085

Fixes #13850
Motivation

Modifications
The root cause is that taskresult has not been reconceiled. So move taskresult reconcile earlier to avoid workflow stuck.
time="2024-11-03T17:44:15.553Z" level=info msg="Processing workflow" Phase=Running ResourceVersion=1753330 namespace=argo workflow=workflow-template-hello-world-gfbjh
time="2024-11-03T17:44:15.554Z" level=debug msg="Task results completion status: map[workflow-template-hello-world-gfbjh-2798872602:false]" namespace=argo workflow=workflow-template-hello-world-gfbjh
time="2024-11-03T17:44:15.554Z" level=debug msg="taskresults of workflow are incomplete or still have daemon nodes, so can't mark workflow completed" fromPhase=Running namespace=argo toPhase=Error workflow=workflow-template-hello-world-gfbjh
time="2024-11-03T17:44:15.554Z" level=error msg="Unable to set ExecWorkflow" error="WorkflowSpec may not change during execution when the controller is set `templateReferencing: Secure`" namespace=argo workflow=workflow-template-hello-world-gfbjh

Verification
The scenario in the issue is verified and the workflow is set to error.

/retest

I am a bit worried we are just moving this around, this was an earlier change that moved the calling of the taskresult reconcile function to where it is now.

I am a bit worried we are just moving this around, this was an earlier change that moved the calling of the taskresult reconcile function to where it is now.

Thank you for your reply. After testing, if there is a problem between processing logs and taskResultReconciliation execution (for example, the template in the templateReferencing: Secure scenario is modified), the workflow will be stuck because the taskresult of the created pod will never be reconciled and returned.

Yeah that makes sense, thanks for taking a look.

@shuangkun thanks for your work.  @isubasinghe anything we can do to move this along?

@sstarcher we cannot move the taskresult reconcile to earlier, it will result in the workflow getting stuck. So we need to somehow address this before we can get this merged.

Theoretically, after entering Processing, the earlier the taskresult is processed, the less likely it is to get stuck.


Create Engineering Feedback Guidance

How to prepare for customer engagement
Customer engagement lifecycle: when should you do CSE Feedback?
What are the steps to submit feedback? What information should I include in the feedback (e.g. customer scenarios, repro steps, examples of quality feedback, etc.)

This has been covered by other work items.


openid using gitlab

Hi,
I am trying to use gitlab as openid provider which ends in http 500 response.
Traceback (most recent call last):
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/app.py", line 2292, in wsgi_app
    response = self.full_dispatch_request()
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/app.py", line 1815, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/app.py", line 1718, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/_compat.py", line 35, in reraise
    raise value
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
  File "/opt/web/powerdns-admin/flask/lib/python3.5/site-packages/flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/opt/web/powerdns-admin/app/views.py", line 456, in login
    oidc_username = me["preferred_username"]
KeyError: 'preferred_username'

Is there any documentation I can refer to?
Thanks in advance.

Found the solution to my issue in app/views.py
Just in case someone else is facing this issue, here's what I did.
diff --git a/app/views.py b/app/views.py
index b94cbff..535251f 100755
--- a/app/views.py
+++ b/app/views.py
@@ -451,8 +451,8 @@ def login():
         return redirect(url_for('index'))

     if 'oidc_token' in session:
-        me = json.loads(oidc.get('userinfo').text)
-        oidc_username = me["preferred_username"]
+        me = json.loads(oidc.get('user').text)
+        oidc_username = me["username"]
         oidc_givenname = me["name"]
         oidc_familyname = ""
         oidc_email = me["email"]

preferred_username should be in the OpenID userinfo response. Do you have any idea why it is not with Gitlab?

@ngoduykhanh preferred_username is not in the userinfo response for a lot of providers... I ran into this issue while trying to use Dex as a provider. Being able to set what field is used similar to scopes, would be very appreciated.

Same problem when using keycloak as an identity provider.

Hello,
Same issue with Keycloak 17.0.0   KeyError: 'preferred_username'  with this
Mappers set:

Thank you.

Samle Problem with Keycloak 18.0.0
May 28 17:27:01 gunicorn[1886]:   File "/opt/web/powerdns-admin/powerdnsadmin/routes/index.py", line 369, in login May 28 17:27:01  gunicorn[1886]:     oidc_username = me[Setting().get('oidc_oauth_username')] May 28 17:27:01 gunicorn[1886]: KeyError: 'preferred_username'
preferred_username is defintily mapped and for testing username is also mapped but nothing works.
I may have set the wrong API URL in PDNS-Admin... Can anyone tell me what URL is expected there?

@vmarkop @RGanor
Have you checked the keycloak notes ?

This issue is very old an it using fixed oidc claims, since then claims can now be changed so its needed to be tested again.

Updated in #717

If other people having problems i think it is better to create a new issue since this is very old and not relevant anymore.
Anyway, as a first step, i need @rtoenniges to dump the content of 'me'

I'm closing this issue since it has been addressed with this PR.


refactor(staking): use votingPowerThreshold

Description / Change(s) / Related issue(s)
Ticket
https://emurgo.atlassian.net/browse/YV-34

One commit for the feature, a dozen for the types and tests to behave 😹

One commit for the feature, a dozen for the types and tests to behave 😹

I love when it is like this!


feat/seeded_rng: add SeededRng type

This exposes a helper type which wraps an XorShiftRng and seed together, allowing tests containing
random data to be reproducible by using the same seed repeatedly.

This change is

r? @afck

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

Comments from Reviewable


Possible redundancy in the code

Hi, I notice that when constructing the Transformer, you always return the intermediate features at this line. However, after feeding them to MLP for corr regression, you only take the prediction over the last layer at this line. So I guess maybe you can set return_intermediate=False to save some memory/computation?

I have another (maybe stupid) question. In your positional embedding here, the order of x/y seems to be reversed? If I concat them by xy_embed = torch.stack([x_embed, y_embed],dim=-1), and print their values:
>>> xy_embed.shape
torch.Size([2, 8, 8, 2])  # this is fine
>>> xy_embed[0, 0, 0]
tensor([1., 1.])  # this is also fine
>>> xy_embed[0, 0, 1]
tensor([2., 1.])  # shouldn't it be [1., 2.]?
>>> xy_embed[0, 5, 1]
tensor([2., 6.])  # shouldn't it be [6., 2.]?

I find their values a bit confusing. Any insight here?

Hey, yes, it sounds a good idea to set return_intermediate=False. I would appreciate it if you can create a PR.
I'm not sure about the positional encoding question, will the permute function fix this ordering problem?


[docs] update translations-zh for Popover and Modal

[x] I have followed (at least) the PR section of the contributing guide.

translations are managed in https://translate.material-ui.com/, not git


Feature Request: Add impl Middlware to support simple stateless middlewares

I read a blog post that included a simplified middleware so I wrote one of my own:
/// Middleware that disables browser caching by default.
async fn no_store<'a, Data>(req: tide::Request<Data>, next: tide::Next<'a, Data>) -> tide::Result<Response>
where Data: Clone + Send + Sync + 'static
{
    use tide::http::cache::{CacheControl, CacheDirective};
    let mut response = next.run(req).await;

    if let None = response.header("Cache-Control") {
        let mut header = CacheControl::new();
        header.push(CacheDirective::NoStore);
        header.push(CacheDirective::MaxAge(Duration::from_secs(0)));

        response.insert_header(header.name(), header.value());
    }
    Ok(response)
}

Unfortunately, that didn't work.  Off to the API docs, and I find that Middleware only has an impl for:
impl<State, F> Middleware<State> for F
where
    State: Clone + Send + Sync + 'static,
    F: Send + Sync + 'static + for<'a> Fn(Request<State>, Next<'a, State>) -> Pin<Box<dyn Future<Output = Result> + Send + 'a>>,

At this point, as a merely intermediate Rust user, my eyes glazed over.  So I generated my own impl like this: (Though I had to just try out different lifetimes until the compiler was happy. Because I'm still a bit vague on the how async_trait transforms the code, and so what signature I need to implement to match the one in the Rust docs.)

struct NoStore {}

#[async_trait]
impl <State: Clone + Send + Sync + 'static> tide::Middleware<State> for NoStore {
    async fn handle<'a, 'b>(&'a self, req: tide::Request<State>, next: tide::Next<'b, State>) -> tide::Result<Response>
    {
        // same body.
    }
}

But, AFAICT, there's no reason that Tide couldn't just add an impl Middleware for these type of bare/stateless middlewares.  Then you could just:
app.with(no_store);

// instead of having to use a struct and manually impl the trait.

app.with(NoStore{});

I've implemented a solution to this and simplified the code base to move State into extensions as well as remove the for<'a> requirement on middleware so this is now possible in my branch https://github.com/http-rs/tide/pull/895
use tide::RequestState;
use tide_derive::RequestExt;

#[derive(Clone, RequestExt)]
struct State {
    foo: String,
}

#[async_std::main]
async fn main() -> tide::Result<()> {
    let mut app = tide::with_state(State { foo: "asdf".into() });

    app.with(|req: tide::Request, next: tide::Next| async move {
        use tide::http::cache::{CacheControl, CacheDirective};
        let mut res = next.run(req).await;

        if let None = res.header("Cache-Control") {
            let mut header = CacheControl::new();
            header.push(CacheDirective::NoStore);
            header.push(CacheDirective::MaxAge(std::time::Duration::from_secs(0)));

            res.insert_header(header.name(), header.value());
        }
        Ok(res)
    });
    app.at("/").get(index);
    app.listen("<IP_ADDRESS>:7000").await?;

    Ok(())
}

async fn index(req: tide::Request) -> tide::Result {
    println!("{}", req.state().foo);
    Ok(tide::Response::new(200))
}


delete namespace feature in fleet

Is your feature request related to a problem? Please describe.
Fleet has added a new feature to delete namespaces when gitrepo is deleted, reference.
Describe the solution you'd like
Rancher UI has this check box item Keep resporces for gitrepos. I propose a similar checkbox for delete namespace.
One think would be important to mention here is that keep resources overrides delete namespace option. When keep resources is active, fleet completely ignores the delete namespace option.
Describe alternatives you've considered
n/a
Additional context
n/a

@richard-cox, the backend will be in the 2.9 release. Could this UI implementation be done also for 2.9?

@mmartin24 There needs to be an issue in rancher/dashboard for UI changes containing as much information as possible about the functionality. UX will then take a look and figure out the best way to present them.
The Always Keep Resources and Delete Namespaces concepts do seem contradictory, how come the functionality wasn't provided in a single option?
In terms of changes in 2.9.0 it's 50:50, otherwise it'll be 2.10.0

@mmartin24 There needs to be an issue in rancher/dashboard for UI changes containing as much information as possible about the functionality. UX will then take a look and figure out the best way to present them.

Thanks @richard-cox , we will create or maybe transfer this issue to rancher/dashboard. Besides the info provided above and in its reference, is there anything else you think is missing?

The Always Keep Resources and Delete Namespaces concepts do seem contradictory, how come the functionality wasn't provided in a single option?

If I understand correctly, this part prevents deleteNamespace to work if keepResources is active:
(@rubhanazeem , please feel free to correct me if I am mistaken)
Is this what you meant, @richard-cox?

In terms of changes in 2.9.0 it's 50:50, otherwise it'll be 2.10.0
Ok, I will let the team know

Besides the info provided above and in its reference, is there anything else you think is missing?

Think I was just hoping for a bit clearer. What resource and page the properties are on.

If I understand correctly, this part prevents deleteNamespace to work if keepResources is active:

Was wondering why it wasn't a single option, something like resourceRetention all | delete namespaces | nothing. Not important, as it's all done now. Was more just to understand the changes.

@sbulage Thanks, that helps a lot. What resource are these properties on?

Thanks @richard-cox :smile:
I found this when correctDrift a.k.a Self Healing introduced: https://github.com/rancher/dashboard/pull/9310/files#diff-4f0062bbf4f47b90a1d0579a65a908e99394b18c1c434700da4729f11a304980
You can find more information on this issue: https://github.com/rancher/fleet/issues/1564.


prisma2 generate always uses yarn even if you're using NPM in your project

If you're an NPM user, you'll get a package-lock.json and package.json. Running prisma2 generate does not pick up that we're using NPM and always uses yarn. This ends up creating both a package-lock.json file and a yarn.lock file.
Full repro here: https://github.com/prisma/prisma-client-js/issues/560

We may want to use https://github.com/sindresorhus/has-yarn in generate

Related issue: https://github.com/prisma/prisma-client-js/issues/560

fixed in alpha 920!


[FR] Interface to edit db config

Crecto should provide an interface where we can define db config elements like poolsize, connection count, retries, timeout etc.
Even if it is provided in the code (and not a separate config file), it will help a lot with dev tasks like data seeding.

Connection pool stuff is handled by crystal-db via url params you pass to the database url.  Check the params listed in the table on this page under Configuration: https://crystal-lang.org/docs/database/connection_pool.html
I've thought about providing a separate configuration for crecto, but using the method above is already pretty easy so I haven't made up my mind yet.

Working on a change right now so Repo works how ecto does, something I should have done to begin with but missed.
Now the user defines their own Repo to use, instead of calling Crecto::Repo.  Inside the repo is where your database connection configuration goes.  This will also allow for an application to use multiple data sources, which currently wont work.
Repo definitions look like:
module Repo
  extend Crecto::Repo

  config do |conf|
    conf.adapter = Crecto::Adapters::Postgres
    conf.database = "crecto_test"
    conf.username = "test"
    conf.password = "123"
    conf.hostname = "localhost"
    conf.port = 5432
  end
end

module MysqlRepo
  extend Crecto::Repo

  config.do |conf|
    conf.adapter = Crecto::Adapters::Mysql
    conf.database = "mysql_test"
    conf.port = 3306
  end
end

# Then use it in your application like normal:
Repo.insert(user)
MysqlRepo.insert(user)

The branch is repo_config, (diff link).
Branch is working, but is still a WIP.  One thing in particular I'd like to figure out a better way to do is: Right now I'm passing the connection (or transaction connection) around through several methods so that by the time it gets to execute in the adapter it has the connection.
Tagging you all because you've been contributing or using crecto the most lately, and this will definitely be a breaking change.
@Zhomart @neovintage @metacortex @huacnlee @jipiboily


next:{cdn:true} is no work

{
  "name": "mistat",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "next": "latest"
  },
  "next":{"cdn":false}
}

when I run next build & next start or run next ,the cdn is not dispair,shall we style the cdn link in our  service ?

@ckken here's a tip so your code displays better :wink:
Use 3 backticks before and after your code to preserve spaces and line breaks. Like so:
#```
code
#```

But without '#'

@flybayer got it, thanks


Allow suppression of style. metadata in first slide

If you define enough classes - and I personally have - seeing them all show up in the metadata list can be unhelpful.
It should be possible to suppress these metadata lines from the table - perhaps on a switch.

Pushed change so that specifying hideMetadata: style suppresses style. metadata items from the Processing Summary slide.

Moved filling in of Processing Summary slide to the end - to allow gathering of when statically set metadata items were overridden dynamically.
Any dynamically overridden items are coloured blue and an asterisk placed after the value.


Release v0.3.0

Description
Formal release PR for PCNS v0.3.0.
https://apps.nrs.gov.bc.ca/int/jira/browse/PADS-170

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

As discussed FRONTEND_CHES_BCC need to be changed to FRONTEND_CHES_ROADMAP_BCC in nr-permitconnect-navigator-service-config to align with line #110 in RoadMap.vue.

First....there so many good things about this!  But what you want to know:

Will Developer tab be hidden by default unless it is us?
Last Edited Date is not shown on the submission list until after you get past the first tab (save only first tab, view draft, no date)
We are missing an "s" in the word successfully on the confirmation page after submitting.
We need a "coming soon" when you click the Status button before we could release this to proponents.
The Files tab seems broken...list view is not working and I believe a file I uploaded disappeared on subsequent login.
If you click on the BF note from the notification bar or from the BF calendar, you get a 422.
There is no search option on the Enquiry list view
We were hoping to have Address and Business lookups included on the Navigator view of submissions
If a Navigator creates a submission in their view and enters an address it is still forcing them to enter lat/long.  It shouldn't.
The Assigned To field is acting odd.  It doesn't do a dropdown and if you want to switch people you have to do a bunch of backspacing first and then enter a new set of 3 characters.

First....there so many good things about this! But what you want to know:

Will Developer tab be hidden by default unless it is us?
Last Edited Date is not shown on the submission list until after you get past the first tab (save only first tab, view draft, no date)
We are missing an "s" in the word successfully on the confirmation page after submitting.
We need a "coming soon" when you click the Status button before we could release this to proponents.
The Files tab seems broken...list view is not working and I believe a file I uploaded disappeared on subsequent login.
If you click on the BF note from the notification bar or from the BF calendar, you get a 422.
There is no search option on the Enquiry list view
We were hoping to have Address and Business lookups included on the Navigator view of submissions
If a Navigator creates a submission in their view and enters an address it is still forcing them to enter lat/long.  It shouldn't.
The Assigned To field is acting odd.  It doesn't do a dropdown and if you want to switch people you have to do a bunch of backspacing first and then enter a new set of 3 characters.

The developer tab is only visible to those with the developer role. Nobody outside the dev team will ever see, or be able to access it.

First....there so many good things about this! But what you want to know:

Will Developer tab be hidden by default unless it is us?
Last Edited Date is not shown on the submission list until after you get past the first tab (save only first tab, view draft, no date)
We are missing an "s" in the word successfully on the confirmation page after submitting.
We need a "coming soon" when you click the Status button before we could release this to proponents.
The Files tab seems broken...list view is not working and I believe a file I uploaded disappeared on subsequent login.
If you click on the BF note from the notification bar or from the BF calendar, you get a 422.
There is no search option on the Enquiry list view
We were hoping to have Address and Business lookups included on the Navigator view of submissions
If a Navigator creates a submission in their view and enters an address it is still forcing them to enter lat/long.  It shouldn't.
The Assigned To field is acting odd.  It doesn't do a dropdown and if you want to switch people you have to do a bunch of backspacing first and then enter a new set of 3 characters.

Resolved most noted issues. After discussion, we will not be making adjustments to the Assigned To dropdown, or adding the Address and Business lookups to the Nav submission view at this time.


gollvm:  cross compiling

Dear Sir,
I want to compile a program in the x86_64 environment, this program can run in the aarch64 environment, and I try " go build -o output -gccgoflags="--target=aarch64-linux"", but it cannot work.

CC @thanm
I believe that GoLLVM does not not current support aarch64-linux anyhow.

Thank you for your reply, but the documents of gollvm writes that gollvm is supported for aarch64 linux.
Which architectures and operating systems are supported for gollvm?
Gollvm is currently supported only for x86_64 and aarch64 Linux"

OK, thanks.
What is the exact output that you get from your go build command?

It outputs as follows:
/usr/local/bin/llvm-goc: unable to determine target CPU features for target aarch64-unknown-linux

This is caused by the inconsistency between the triple_ lists in ArchCpusAttrs.h and the normalized triple_ that we get through the --target option, and the root cause is clang. In fact, if you set --target=x86_64-linux, the same error will be reported. I can fix this problem.

Thank you very much!

Thanks @erifan. I took a look and I can reproduce as well.  I assume this will involve changing and/or rerunning tools/capture-fcn-attributes.go -- are you planning of sending a patch?

@thanm Yes, I plan to send a patch, but it will be next week, is that ok?

Yes, I plan to send a patch, but it will be next week, is that ok?

Yup, that's fine. Thanks.

@wyw900807 , there are some common files for both Clang and llvm-goc.
They define which CPU architectures are supported and how specific CPU features/instruction sets are related.
Some other files are responsible for the calling conventions, in LLVM.
As for now - you can do the following (this duplicates what Clang itself would do):

Create a C++ source code file named, say, llvm_cpu_features_investigation.cpp
Populate it with the following content (you could improve a bit, to format the output):

#include "llvm/ADT/StringRef.h"
#include "llvm/ADT/StringMap.h"
#include "llvm/MC/SubtargetFeature.h"
#include "llvm/Support/Host.h"
using namespace llvm;
SubtargetFeatures Features1;
int main (int argc, char **argv)
{
sys::getHostCPUName();
StringMap HostFeatures;
if (sys::getHostCPUFeatures(HostFeatures))
for (auto &F : HostFeatures)
Features1.AddFeature(F.first(), F.second);
printf("test %s", Features1.getString().c_str());
printf("\nsomething else\n");
return 0;
}

Compile it:

$ clang++-11 -I/usr/include/llvm-c-11 -I/usr/include/llvm-11 -L/usr/lib/i386-linux-gnu/ -lLLVM-11 ~/workarea/llvm_cpu_features_investigation.cpp

Run your resulting executable:

$ ./a.out
test
(...your CPU instruction sets are listed here...)
something else

Run the following command:

$ llc-11 --version
LLVM (http://llvm.org/):
LLVM version 11.0.0
Optimized build.
Default target:
(...that would be your current target, not for cross-compiling)
Host CPU:
(...there would be your CPU model, as LLVM understands it...)

You shall see some CPU features, show during (4), with a "-" prefix. Those are non-supported features, for a specific CPU model (5). You can't expected that to be used during -mtune
Since you are cross-compiling - it would be required to get corresponding CPU features (4) and the CPU model(5) definitions, on Aarch64 (probably you would either run under qemu or would use a system-on-chip/micro-controller, for that purpose).
Please provide specific information about targeted Aarch64 hardware.

Github Actions give access to a relic Clang 9 - so I used it for an example.
Here is the workflow configuration file:

name: LLVM
on:
create:
branches: [ main ]
pull_request:
branches: [ main ]
jobs:
build:
runs-on: ubuntu-latest
steps:
  - uses: actions/checkout@v2

  - name: Get Clang's version
    run: clang --version

  - name: Check our path
    run: pwd
  - name: Extracting llc's host target
    run: llc-9 --version | grep CPU
  - name: Compile test tool, for retrieveing CPU features
    run: clang -I/usr/lib/llvm-9/include/ -I/usr/lib/llvm-9/include/llvm-c -L/usr/lib/llvm-9/lib -lLLVM-9 -lstdc++ $GITHUB_WORKSPACE/llvm_cpu_features_investigation.cpp

  - name: Run the CPU feature extractor
    run: ./a.out

@advancedwebdeveloper  Thank you for your help, I will try it

Hi, @thanm I thought gollvm already supports cross compiling, but it doesn't seem to be. Because we did not generate standard libraries for multiple architectures, the corresponding libraries can't be found during cross compiling. Updating ArchCpusAttrs.h is easy, but adding support for cross compiling requires much more effort. It seems that gccgo does not currently support cross compiling ?
@wyw900807 Maybe you need to compile the gollvm toolchain on Linux aarch64, and then compile your program.

OK, Thank you!

@erifan yes, there is almost certainly a lot of work to do to set up Gollvm as a cross compiler with respect to the libraries. The first step is getting the compiler to work properly however.
Gccgo does support cross compilation, but not using the same model as vlang-- with gccgo you need a separate installation and compiler binary per target (as opposed to the clang model in which a single compiler binary supports multiple targets).


Spike refactor to use new Path from roc-lang/path

Hit a bit of a snag with this upgrade.
thread 'main' panicked at /private/tmp/nix-build-roc_cli-0.0.1.drv-0/cargo-vendor-dir/inkwell-0.2.0/src/types/enums.rs:475:13:
Found StructType(StructType { struct_type: Type { address: 0x1338c91b8, llvm_type: "{ [0 x i64], [3 x i64], i8, [7 x i8] }" } }) but expected the PointerType variant
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Investigated a bit, it might be best to wait and maybe land LLVM upgrade first, this looks to be unrelated to the Path upgrade.

Closing as this has gone a bit stale and it will be easier to start from scratch.


Failed to load shared object: libicui18n.so.61

The error occurred after updating Manjaro. Installed icu is 62.1-1.
Hydrogen looks for libicui18n.so.61 which was replaced by libicui18n.so.62.

Updating "stringi" in R has solved the issue.


Python client type hint error with column-oriented upsert

I'm trying to do an upsert without hard-coded data and I get some type hint errors. Everything works fine so I'm assuming its just a typing issue.

The error on ids is:
Pyright: Argument of type "list[int]" cannot be assigned to parameter "ids" of type "List[ExtendedPointId]" in function "__init__"
  "list[int]" is not assignable to "List[ExtendedPointId]"
    Type parameter "_T@list" is invariant, but "int" is not the same as "ExtendedPointId"
    Consider switching from "list" to "Sequence" which is covariant

I can fix this one by typing the ids declaration as ids: list[ExtendedPointId]. This may be just a pyright issue since ExtendedPointId is defined as int | str.
The error on vectors is:
Pyright: Argument of type "Iterable[ndarray[Unknown, Unknown]]" cannot be assigned to parameter "vectors" of type "BatchVectorStruct" in function "__init__"
  Type "Iterable[ndarray[Unknown, Unknown]]" is not assignable to type "BatchVectorStruct"
    "Iterable[ndarray[Unknown, Unknown]]" is not assignable to "List[List[StrictFloat]]"
    "Iterable[ndarray[Unknown, Unknown]]" is not assignable to "List[List[List[StrictFloat]]]"
    "Iterable[ndarray[Unknown, Unknown]]" is not assignable to "Dict[StrictStr, List[Vector]]"

Can't seem to get around this one without a cast. Like I said, everything works but the type errors hamper the DX.

Hi @affanshahid
Thanks for reporting this, we'll look into it
Could you please tell us qdrant-client version you're using along with the version of pyright?

qdrant-client==1.11.2
pyright language server 1.1.381

Hey @affanshahid
We took a look at the issue, we might need more time to fix the issue with ids
However, embeddings type hint issue seems to be correct, since it does not expect numpy arrays

I see, i imported TextEmbedding from fastembed in the example, what would be the correct way of doing this? Cast to regular list?


Wire up profit and loss totals

Currently profit and loss values are stubbed out, need to get from augur-node.

added summing up realized for ETH change in user's balance.

This tix isn't blocked, not sure how to remove the dep.

UI is already wired up and gets the realized percentage.

Looks to be working correctly now - closing


Babyon Toolkit Name Refactor

Update 'Unity Toolkit' references to Babylon Toolkit'

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Oops... is duplicate

Snapshot stored with reference name:
refs/pull/15285/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15285/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15285/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15285/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15285/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15285/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15285/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15285/merge#BCU1XR#0

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15285/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Issue with executing the PowerShell

I'm currently getting the problem with executing the PowerShell to get Nuget Tools installed on my notebook.
error message
"
Unable to find package 'Microsoft.CrmSdk.XrmTooling.PackageDeployment.PowerShell'
move : Cannot find path 'C:\Tools\tools' because it does not exist.
At line:49 char:1

move .\Tools$pdPoshFolder\tools*.* .\Tools\PackageDeployment.PowerS ...

  + CategoryInfo          : ObjectNotFound: (C:\Tools\tools:String) [Move-Item], ItemNotFoundException
  + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.MoveItemCommand

"
Any idea what has been changed ?

Szczegóły dokumentu
⚠ Nie edytuj tej sekcji. Jest ona wymagana dla docs.microsoft.com ➟ Dołączanie problemu w usłudze GitHub.

ID: 9d7d5f36-11fc-9ac5-778b-c03823bbb072
Version Independent ID: 814e151a-2018-285a-697f-f929a73957d4
Content: Download tools from NuGet (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/customerengagement/on-premises/developer/download-tools-nuget.md
Product: d365ce-op
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@losi127 This should be fixed now. The sample PowerShell script in the topic needed updating, so grab a new copy and try it.


Conversion error

The following code has UNREACHABLE executed at lib/SPIRVProducerPass.cpp:2179! error.
const sampler_t sampler =
CLK_NORMALIZED_COORDS_TRUE | CLK_ADDRESS_REPEAT | CLK_FILTER_NEAREST;

kernel void foo(global float* out,
                global float* in,
                __read_only image2d_t inImage) {
  uint i = get_global_id(0);
  float sum = 0.0f;
  for (int j = 0; j < 4;++j)
    sum += read_imagef(inImage, sampler, (float2)(0.05 * (float)i, 0.05 * (float)i))[j];
  out[i] = in[i] + 100.0f * sum;
}

I think it is caused by 0.05 * (float)i in the third argument of read_imagef().
If I changed it as 0.05f * (float)i, the error disappeared.
Is it an intended behavior e.g., no implicit converting from double to float ?

The floating point literals are of type "double" in C.    Use the flag -cl-single-precision-constant to force the compiler to treat them as "float".   Either that or append 'f' to the number, e.g. 0.05f.
This is a known issue.   Normally you don't want to imply 64-bit floats just for the literals when the rest of the program is using only 32-bit floats.


adjust for leantime 2.4 config

When accessing leantime 2.4 an error appears due to a breaking change described here: https://github.com/leantime/leantime/releases/tag/v2.4-beta-2
This adopts the change

Many thanks for this - This is tagged as 1.0.35 and creditted to ArjenR@github


Drop support for EOL Python 3.6

Contributor checklist

[ ] Provided the tests for the changes.
[x] Assure PR title is short, clear, and good to be included in the user-oriented changelog

Maintainer checklist

[ ] Assure one of these labels is present: backwards incompatible, feature, enhancement, deprecation, bug, dependency, docs or skip-changelog as they determine changelog listing.
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

Fixes https://github.com/jazzband/pip-tools/issues/1562.
Includes and closes https://github.com/jazzband/pip-tools/pull/1560.
Includes and closes https://github.com/jazzband/pip-tools/pull/1561.

Closing in favour of https://github.com/jazzband/pip-tools/pull/1567.


EACCES: permission denied, open '../amplify/deployment-secrets.json'

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.16.0
Amplify CLI Version
10.8.1
What operating system are you using?
mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
AWS CLI amplify add auth
Describe the bug
(base) % amplify add auth
Using service: Cognito, provided by: awscloudformation
The current configured provider is Amazon Cognito.
Do you want to use the default authentication and security configuration? Defau
lt configuration
Warning: you will not be able to edit these selections.
How do you want users to be able to sign in? Email
Do you want to configure advanced settings? No, I am done.
🛑 EACCES: permission denied, open '/Users/dboe/.aws/amplify/deployment-secrets.json'
Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Session Identifier: 1a463445-7eea-4731-80f3-937180a095d0
Expected behavior
it should work
Reproduction steps
(base) % amplify add auth
Using service: Cognito, provided by: awscloudformation
The current configured provider is Amazon Cognito.
Do you want to use the default authentication and security configuration? Defau
lt configuration
Warning: you will not be able to edit these selections.
How do you want users to be able to sign in? Email
Do you want to configure advanced settings? No, I am done.
🛑 EACCES: permission denied, open '/Users/dboe/.aws/amplify/deployment-secrets.json'
Resolution: Please report this issue at https://github.com/aws-amplify/amplify-cli/issues and include the project identifier from: 'amplify diagnose --send-report'
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Session Identifier: 1a463445-7eea-4731-80f3-937180a095d0
Project Identifier
report-1679672674127.zip
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @dboe1107 :wave: thanks for raising this! Was the CLI installed with sudo by chance? Can you try assuming ownership over that file with chown ~/.aws/amplify/deployment-secrets.json?

Closing due to inactivity


A new filter pattern is needed

Keycloak needs a filter pattern that supports multiple criteria. There are some reasons that the current Faceted filter and Multiple filters don't fit our use case.

There can be more than 5 criteria, so there aren't enough spaces for multiple filters.
For each attribute, there can be thousands of options so a faceted filter cannot cover that.

I have also considered applying multiple attribute-value filters. But it is unpractical due to the technical limitation. Iterated filtering needs too much effort on the server-side to be implemented.
An assumed pattern for us is, there will be a search box allowing the free search. A dropdown with a form will be attached to it. Users can fill in the form to limit all the criteria. And the search will be conducted by the Search button in the form.
What's more, the filter will support the search operator. That is, users can directly input "username:hwang firstname:haley" instead of filling the form.
I'm not sure if the Expression Builder would fit our use case. As there isn't a guideline for it.

@yih-wang I've seen this type of use case come up before and it's true that we do not currently have a component that satisfies this. Just a couple of questions. What is the required relationship between multiple attribute-value terms. Is it assumed to be a Boolean AND, i.e. show everything that satisfies the first criteria AND the second criteria, etc? Or does this need to be more flexible? Depending on that, an expression builder may or may not be needed.

@mcarrano I think all the relationships should be Boolean AND. Maybe we don't need an expression builder, just another new filter pattern.


Surfacing mcf required filters in the backend

Description
This pull request surfaces changes made to the cpr_sdk to support filtering for the MCF corpus data.
Proposed version
Please select the option below that is most relevant from the list below. This
will be used to generate the next tag version name during auto-tagging.

[ ] Skip auto-tagging
[ ] Patch
[x] Minor version
[ ] Major version

Visit the Semver website to understand the
difference between MAJOR, MINOR, and PATCH versions.
Notes:

If none of these options are selected, auto-tagging will fail
Where multiple options are selected, the most senior option ticked will be
used -- e.g. Major > Minor > Patch
If you are selecting the version in the list above using the textbox, make
sure your selected option is marked [x] with no spaces in between the
brackets and the x

Type of change
Please select the option(s) below that are most relevant:

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] GitHub workflow update
[ ] Documentation update
[ ] Refactor legacy code
[ ] Dependency update

How Has This Been Tested?
Please describe the tests that you added to verify your changes.
Reviewer Checklist

[ ] DB_CLIENT DEPENDENCY IS ON THE LATEST VERSION
[ ] The PR represents a single feature (small driveby fixes are also ok)
[ ] The PR includes tests that are sufficient for the level of risk
[ ] The code is sufficiently commented, particularly in hard-to-understand areas
[ ] Any required documentation updates have been made
[ ] Any TODOs added are captured in future tickets
[ ] No FIXMEs remain

A thought - add multiple geographies to test data to ensure we're ready for that as I think they're all just a list of the single geo currently.

Have you spun up the backend locally outside of tests and ran a query?

Functionally this looks great to me, and I think the tests you've added look good. I am wondering if we might need one more test in the search tests that makes sure we get metadata back? Like how we are currently doing for geographies. This would give us good confidence in this all fitting together correctly. This is good to go from my view otherwise!

Definitely!


lowercase string macros?

Perhaps the string macros should be lowercase.
static"Hello World"20

Closed by   #7


Improve validation and error handling

Some improvements suggested in conversation with @coretl

in getattrs make a nice error when a non-existent attribute is requested (I am type X with attrs ... but you have asked for Attr Y)
also remove need to say object.name instead render object as its id
Provide a validation capability for defs e.g.

defs:
  - name: Psc
    args:
      - name: carrier
        type: object
        description: IPAC carrier name

      - name: interrupt_vector
        type: object
        description: Interrupt Vector reserved with epics.InterruptVectorVME, count=3

    validate:
      - assert: interrupt_vector.count == 3
        message: Interrupt_vector must be reserved with epics.InterruptVectorVME, count=3

Do this after

The first two bullets are now covered in 3.0.0b5
Leaving this open as the suggestion of a validate is still useful.


Unable to compress Png file in both android and ios.

Hey @nomi9995 thanks for reply.There is no any error log in console while compress image but image size is larger than original image size.Like my original image size is 13.12 kb and after applying image compression it will be 43 kb.

@krupalikevadiya can you try some bigger image like 1 to 5 mb

No i just check it for small images like kb.But let you know result for large images after test.

@krupalikevadiya you should check this package on minimum 50kb image

Hey @nomi9995 what i have to do to compress images which have size lower 50kb?

@krupalikevadiya you have to play with these three options

Yes i used this all three options and use same height and width of original image with 0.5 quality but png image become double in size after compression.

Hey @nomi9995 did you have any another suggestion for png image compression?

@krupalikevadiya you can add native code for less than 50kb image. I will welcome your PR


⚠ SceneTime has degraded performance

In 4f43f67, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1277 ms

Resolved: SceneTime performance has improved in 7ff10c4 after 6 minutes.


RN: Switch EventEmitter to Array.from(...)

Summary:
Switches EventEmitter#emit to use Array.from instead of the spread operator.
This should be functionally identical (with marginally less overhead of the runtime having to determine the type of registrations), but there seems to be some unexpected Babel configurations in the community that causes this line of code to do the wrong things.
Although we should independently root cause the Babel plugin configuration problems, this change might provide immediate relief and is also not any worse (e.g. in terms of code readability). This also adds a descriptive comment explaining the intention of the call to Array.from.
Changelog:
[Fixed][General] - Fix a potential bug in EventEmitter when used with certain Babel configurations that incorrectly polyfill the spread operator for iterables.
Differential Revision: D49389813

@yungsters is react native planning a release with this in react native 72 version

@yungsters is react native planning a release with this in react native 72 version

Thanks for the poke; I've requested this fix for the next patch release in https://github.com/reactwg/react-native-releases/discussions/89.


fixed transparency

Should it be done this way ?

No, still some problems:
http://grab.by/S1Hg

seems to work ~ok

@vikbez
Thank you! Pull Requests!!!!!!!
However , "blend mode " did not work...
I have received a very big hint .
thank you very very much!

Glad it helped !
Thanks !
(you should add a donate button 👍 )


Simple password generates 'password invalid'

Fresh after installation I tried adding a user with the password "111111111111". Not the most secure password, but ought to work. Doesn't work, gives "password invalid" message. Took a while to figure out that the password validation contains a regex that looks like it might be limiting characters to upper and lower case letters and digits:
validates :password, allow_nil: true, length: { minimum: 6 }, format: { with: /(?=.*[a-z])(?=.*[A-Z])(?=.*[0-9])/ }

If that's the case it can be done with a much simpler regex, this one doesn't work! More to the point, why is this here at all? Isn't the restriction of characters in a password a feature that only some will want? Leave it behind a '#' if you want to let it be a reminder that such a thing can be done.

I want to promote good security practices…

@lazaronixon running bundle exec brakeman -q -o tmp/brakeman.html with your regex shows the following error:

I found a possible solution, but some tests break.
Nice work.

This regex /\A(\w|[ !@#$%*])*\z/ seems to work fine.

It's weird that brakeman think it's risky, I'm just following this wiki https://github.com/heartcombo/devise/wiki/How-To:-Set-up-simple-password-complexity-requirements @yorsant

Looking over this warning, it's a false positive for our case. The password must contain this regexp, I don't care which characters can be present before or later in the matching regexp. In summary, it's not a risk. An alternative would be \A(?=.*[a-z])(?=.*[A-Z])(?=.*[0-9]).*\z but it doesn't bring any benefict.
https://cwe.mitre.org/data/definitions/777.html

I removed the format validation from the password, I realized that hey.com validates through pwned only. Also now we run Brakeman on CI.


ArrayModel to produce array config files

A first version of the array_model module was implemented.
An application called produce_array_config.py is provided as an example of usage of the ArrayModel class.
The sim_telarray config files produced were tested (for the given example) with sim_telarray using an external script. This kind of test will be implemented here in a future PR.

As a general comment on reviewing, as this is one of the first pull requests I review: on what level of detail do we expect the review? I am certainly not familiar enough with all the details of gammasim-tools to do a line-by-line review, also I don't know python enough to give you good feedback on the language level.

Ideally the review should be as deep as possible. But of course we cannot do that and need to find a good balance in order to optimize our time use. I think the most important items to review are: testing the applications, overview of the code to check for obvious problems and reviewing the application documentation.

As a very general point - something we have discussed before but I think we should do again:

is the code generic enough? I read a lot of 'SST', 'LST', and very specific names. This might hurt on the long term if CTA evolves or if we find other applications for gammasim-tools.

This is something to be discussed in a call again. The code is being designed to be as modular and flexible as possible. That does not mean that it is completely generic. I honestly don't think it is realistic to expect a fully generic software. The goal should be to make it easily expandable, but also efficient and usable for our needs now (and soon).

sim_telarray dependence: I wonder if we should review the current implementation of putting sim_telarray-specific code into the model classes, or if it would be better to completely separate this and have something like a sim_telarray-writer class. On the long term, this might help if we want to use other simulation codes.

Do you consider the name of the parameters as sim_telarray specific code?
Apart from the name of the parameters, I'm trying to make specific signatures for sim_telarray related functions. We have, for example, exportSimtelConfigFile(). In the future we can expand by writing another function exportOtherSoftwareConfigFile().
We could add modules in between (interfaces) to avoid that. However, I think that is an unnecessary over complication, given the size of our software.

Should discuss this, even if it adds another module: separating out the sim_telarray specific exporters would make sense in my opinion.
But also clear that we have an extremely strong dependency on sim_telarray. But let’s think hard if and where we can reduce it.

On 7. Apr 2021, at 11:03, RaulRPrado @.***> wrote:
As a general comment on reviewing, as this is one of the first pull requests I review: on what level of detail do we expect the review? I am certainly not familiar enough with all the details of gammasim-tools to do a line-by-line review, also I don't know python enough to give you good feedback on the language level.
Ideally the review should be as deep as possible. But of course we cannot do that and need to find a good balance in order to optimize our time use. I think the most important items to review are: testing the applications, overview of the code to check for obvious problems and reviewing the application documentation.
As a very general point - something we have discussed before but I think we should do again:
• is the code generic enough? I read a lot of 'SST', 'LST', and very specific names. This might hurt on the long term if CTA evolves or if we find other applications for gammasim-tools.
This is something to be discussed in a call again. The code is being designed to be as modular and flexible as possible. That does not mean that it is completely generic. I honestly don't think it is realistic to expect a fully generic software. The goal should be to make it easily expandable, but also efficient and usable for our needs now (and soon).
• sim_telarray dependence: I wonder if we should review the current implementation of putting sim_telarray-specific code into the model classes, or if it would be better to completely separate this and have something like a sim_telarray-writer class. On the long term, this might help if we want to use other simulation codes.
Do you consider the name of the parameters as sim_telarray specific code?
Apart from the name of the parameters, I'm trying to make specific signatures for sim_telarray related functions. We have, for example, exportSimtelConfigFile(). In the future we can expand by writing another function exportOtherSoftwareConfigFile().
We could add modules in between (interfaces) to avoid that. However, I think that is an unnecessary over complication, given the size of our software.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Dr Gernot Maier
CTA
Deutsches Elektronen-Synchrotron DESY
Ein Forschungszentrum der Helmholtz-Gemeinschaft
Platanenallee 6, D-15738 Zeuthen, Germany
Tel.: +49  33 7627 7598
Internet: www.desy.de/cta
Besucheradresse: 1X07

On the definition of the array model as described in the header of produce_array_config.py: is there a possibility to disable any default settings and that each telescope model for each telescope has to be set explicitly? There are use cases where we want to do this, but at this point the system will fall back (unintentionally) to defaults if a telescope has been forgotten. Maybe just an option like 'setexplicitelyallModelParameters' (find a better name).

Should discuss this, even if it adds another module: separating out the sim_telarray specific exporters would make sense in my opinion. But also clear that we have an extremely strong dependency on sim_telarray. But let’s think hard if and where we can reduce it.

Yes, that is right. I created an issue and will work on this later.

On the definition of the array model as described in the header of produce_array_config.py: is there a possibility to disable any default settings and that each telescope model for each telescope has to be set explicitly? There are use cases where we want to do this, but at this point the system will fall back (unintentionally) to defaults if a telescope has been forgotten. Maybe just an option like 'setexplicitelyallModelParameters' (find a better name).

Since yesterday, you can set a telescope model for each telescope in the arrayConfigFile, you don't need to give a default.
You can then set a base telescope model and change any parameter you want.
If you mean setting a completely new telescope model, ignoring the base model, no, that is not possible.
You can play around with a completely new telescope model at a lower level, using the TelescopeModel. But at the level of the array_model, we always require a base model.
Note that creating a new telescope model from the grounds is not supposed to be a simple task and should never be done at this level. The ideal process would be creating the new model using a telescope_model module, uploading it to the DB and then, at the array_model level, building it from the DB.
We intentionally don't want to give this much freedom at the high level stage of the array_module.


Simple AdBlock for OpenWrt

Just a possible heads-up. Feel free to close it if it's not relevant here.
https://github.com/stangri/openwrt-packages/blob/simple-adblock/net/simple-adblock/files/README.md
People reported that thing is holding up on ~40K hosts, while common adblock is causing the router to crash on just ~20K hosts.
I hope it could be useful to examine and compare it with DNScrypt's performance, if someone is willing to test it.
Possible related issue: https://github.com/jedisct1/dnscrypt-proxy/issues/162

Looks like it is just a script fetching data from different sources and deduplicating entries?
There is something similar in the dnscrypt-proxy package: https://github.com/jedisct1/dnscrypt-proxy/tree/master/utils/generate-domains-blacklists

https://github.com/stangri/openwrt-packages/blob/simple-adblock/net/simple-adblock/files/simple-adblock.init#L10
wget --no-check-certificate

I stopped reading here.


Profile for performance bottlenecks

Before I try to refactor anything for theoretical speed gains, figure out what the actual slow parts of the build process are.

Ran a couple tests, including adding a NullWriter to check how much time is spent writing output.
time dorchester plot tests/data/suffolk.geojson tests/data/suffolk.csv --key POP10 --progress
100%|██████████████████████████████████████████████████████| 9170/9170 [00:24<00:00, 368.40features/s]
       25.89 real        25.80 user         0.42 sys

time dorchester plot tests/data/suffolk.geojson /dev/null --key POP10 --progress --format null
100%|███████████████████████████████████████████| 9170/9170 [00:21<00:00, 428.71features/s]
       22.62 real        22.51 user         0.39 sys

So there's definitely a difference, but it's not actually as much time as I'd expected. Lots of time really is spent in Python, doing math and loops.


cask/quarantine: don't try to sudo during tests.

Fixes #15958

What is to here and why is it not writeable? Seems like there's a problem here if we're demanding sudo and it doesn't fail without it.

What is to here and why is it not writeable?

It's explicitly set to not be writable in the test.

Seems like there's a problem here if we're demanding sudo and it doesn't fail without it.

If you can reproduce the same behaviour in Not The Test Environment: sure. This just feels like an omission when underlying functionality was changed, though.

Right I understand the situation now.
In the test we pass NeverSudoSystemCommand  to install_phase. This down the line gets passed, in this case, to Cask::Artifact::Moved#move as the command parameter, where we run command.run! instead of system_command!.
The issue here is we don't pass this further down to the Quarantine.copy_xattrs call when it looks like we should be.

I tested out @Bo98's idea and the failing test now succeeds for me locally.
diff --git a/Library/Homebrew/cask/artifact/moved.rb b/Library/Homebrew/cask/artifact/moved.rb
index d41e62cec..a9e3470d7 100644
--- a/Library/Homebrew/cask/artifact/moved.rb
+++ b/Library/Homebrew/cask/artifact/moved.rb
@@ -91,7 +91,7 @@ module Cask
             command.run!("/bin/cp", args: ["-pR", *source.children, target],
                                     sudo: true)
           end
-          Quarantine.copy_xattrs(source, target)
+          Quarantine.copy_xattrs(source, target, command: command)
           source.rmtree
         elsif target.dirname.writable?
           FileUtils.move(source, target)
diff --git a/Library/Homebrew/cask/quarantine.rb b/Library/Homebrew/cask/quarantine.rb
index 227790629..e038011e6 100644
--- a/Library/Homebrew/cask/quarantine.rb
+++ b/Library/Homebrew/cask/quarantine.rb
@@ -174,11 +174,11 @@ module Cask
       raise CaskQuarantinePropagationError.new(to, quarantiner.stderr)
     end

-    sig { params(from: Pathname, to: Pathname).void }
-    def self.copy_xattrs(from, to)
+    sig { params(from: Pathname, to: Pathname, command: T.class_of(SystemCommand)).void }
+    def self.copy_xattrs(from, to, command:)
       odebug "Copying xattrs from #{from} to #{to}"

-      system_command!(
+      command.run!(
         swift,
         args: [
           *swift_target_args,

@Bo98 Good catch, will try to take a look, thanks.

Thanks @Bo98 @apainintheneck have updated accordingly based on your feedback and diff.

Thanks for the help with fixing this! I thought it was maybe a local problem when I first saw it pop up since CI has been green the whole time.


Compute the time in milliseconds to decide when to send heartbeat in can

CAN was checking ticks instead of milliseconds, yet the configuration parameters are all in milliseconds.
One possible further change (for discussion) would be to remove the conditional of only updating the send time in case of success. If there is a can issue and we could not send, the current code would try in a very tight loop and could cause timing issues elsewhere.

 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.

Thanks for the PR!  The sys tick happens to be 1 ms in our code so it worked but you indeed have a better solution.
As for the CAN re-try , it should only be retrying once per cycle (125 microseconds), so minimal overhead.

@Ibarria In the current devel branch, CAN uses HAL_GetTick() instead of osKernelSysTick(). The HAL function should always return ticks in milliseconds. Unless there are other concerns, I'm closing this PR.


Should I import the cocoapods version or the one on the master branch?

Which version of Hero should we use if we want to have the most stable version? The one on the cocoapods or the one that is on the master branch. Maybe there should be an indication about this in the README.

Most stable version should always be the one on cocoapods. There might be new fixes and patches that are not yet available in the cocoapods version but exists in the master branch. But keep in mind that I might break the master branch sometimes.

Ok, clear. Do you plan to publish the master branch on the cocoapods soon? I need to use it in production and I want the most stable version possible

Expecting  the master branch on the cocoapods soon.