[bug]: The "Download Image" button on the txt2img tab can be clicked only directly on the text

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
No response
What happened?
The "Download Image" button on the txt2img tab can be clicked only directly on the text. Clicking on the large empty area next to the text does nothing. Other buttons work: I can click on empty parts of the buttons.
Brave browser, using 125% zoom.
Screenshots
No response
Additional context
No response
Contact Details
No response

Thanks, fix upcoming in PR #2435


Refer to actual generated swift header when making swift hmap

Before, the file in hdrs referred to a non-existant source file
We can see the fix in effect by the xcodeproj build not failing, since the -Swift.h header is no longer getting added to the xcode project !
cc @wileykestner

Was the problem we were trying to solve here that we had an extra -Swift.h file? If so, is it possible to write a test to show that we no longer generate an extra -Swift.h file?

It wasn't actually generated, but the hmap had a reference to a source file with the name ModuleName-Swift.h. That file didn't exist, but we didn't catch it because nothing in the hmap rule actually uses the headers as inputs to any actions, so bazel was happy to continue passing it along, even though it didn't exist.
This showed up in the Xcodeproj rule, since that rule gathers all transitive source (e.g. not generated) files, and the -Swift.h file was showing up there, even though it (should have been a) generated file.


Add AlphaFold optimization examples for PyTorch.

Examples of BladeDISC optimization for Uni-Fold Inference.
A FoldAcc tool for AlphaFold optimization.

 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.

👍LGTM~


Can't use prisma migrate dev

Bug description
I was facing this bug and I posted a comment there but now the issue that I am running npx prisma migrate dev --name init and prisma outputs this:
Drift detected: Your database schema is not in sync with your migration history.

The following is a summary of the differences between the expected database schema given your migrations files, and the actual schema of the database.

It should be understood as the set of changes to get from the expected schema to the actual schema.

If you are running this the first time on an existing database, please make sure to read this documentation page:
https://www.prisma.io/docs/guides/database/developing-with-prisma-migrate/troubleshooting-development

[+] Added tables

# Here prisma shows a list of all the tables in my schema

√ We need to reset the MySQL database "Weknd" at "localhost:3306".
Do you want to continue? All data will be lost. ... no

Reset cancelled.

But I don't have migrations yet (there is not a prisma/migrations folder and there is not a table called _prisma_migrations in the database), and when I run npx prisma migrate status this is the output:
No migration found in prisma/migrations

The initial issue was that I was not able to create a new migration when I ran npx prisma migrate dev --name added_unique_to_some_columns, prisma showed this:
Drift detected: Your database schema is not in sync with your migration history.

The following is a summary of the differences between the expected database schema given your migrations files, and the actual schema of the database.

It should be understood as the set of changes to get from the expected schema to the actual schema.

[*] Changed the `admin` table
  [-] Removed foreign key on columns (userId)

[*] Changed the `associate` table
  [-] Removed foreign key on columns (userId)

[*] Changed the `client` table
  [-] Removed foreign key on columns (membershipId)
  [-] Removed foreign key on columns (userId)

[*] Changed the `clientcouponhistory` table
  [-] Removed foreign key on columns (clientId)
  [-] Removed foreign key on columns (couponId)

[*] Changed the `coupon` table
  [-] Removed foreign key on columns (associateId)
  [-] Removed foreign key on columns (tipoPromoId)

[*] Changed the `paymentshistory` table
  [-] Removed foreign key on columns (balanceAmountId)
  [-] Removed foreign key on columns (userId)

[*] Changed the `schedule` table
  [-] Removed foreign key on columns (venueId)

[*] Changed the `venue` table
  [-] Removed foreign key on columns (associateId)
  [-] Removed foreign key on columns (venueTypeId)

[*] Changed the `venuecoupon` table
  [-] Removed foreign key on columns (couponId)
  [-] Removed foreign key on columns (venueId)

[*] Changed the `venueservice` table
Default Engines Hash    : 8d8414deb360336e4698a65aa45a1fbaf1ce13d8
Studio                  : 0.469.0

But that happened after creating the first migration with prisma, so I wasn't sure why was there a conflict if I was not moving anything manually
How to reproduce
I was using prisma 3 and had my schema and migration history working fine, then I upgraded to prisma 4 following this guide and when I tried to create a new migration that happened.
To reproduce it try

Open a new terminal, create a directory and run npm init -y
Install prisma 4 using npm install prisma@4 @prisma/client@4
Run npx prisma init --datasource-provider mysql
Create the following database in mysql:

create database TestDatabase;

use TestDatabase;

create table table1 (
    id varchar(50) not null,
    field1 varchar(100) not null,
    primary key (id)
);

create table table2 (
    id int not null auto_increment,
    table1Id varchar(50) not null,
    primary key (id),
    unique key table2_table1Id_key (table1Id),
    CONSTRAINT table2_table1Id_fkey FOREIGN KEY (table1Id) REFERENCES table1 (id) ON DELETE RESTRICT ON UPDATE CASCADE
);

Set the DATABASE_URL variable in the .env file
Run npm prisma db pull. At this step I see that the schema that prisma pulled from the database is missing the foreign key in table2, this are the models that prisma shows:

model table1 {
  id     String @id @db.VarChar(50)
  field1 String @db.VarChar(100)
}

model table2 {
  id       Int    @id @default(autoincrement())
  table1Id String @unique @db.VarChar(50)
}

Initialize prisma migrations using npx prisma migrate dev --name init and reset the database
Update the schema using this models:

model table1 {
  id     String @id @db.VarChar(50)
  field1 String @db.VarChar(100)
  table2 table2?
}

model table2 {
  id       Int    @id @default(auto_increment())
  table1Id String @unique @db.VarChar(50)
  table1   table1 @relation(fields: [table1Id], references: [id])

  @@index([table1Id])
}

Run npx prisma migrate dev --name added_foreign_keys
Change field1 String @db.VarChar(100) to field1 String @unique @db.VarChar(100) and run npx prisma migrate dev --name added_unique_constraint_to_field1

When I do these steps I get the following output:
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": MySQL database "TestDatabase" at "localhost:3306"

Drift detected: Your database schema is not in sync with your migration history.

The following is a summary of the differences between the expected database schema given your migrations files, and the actual schema of the database.

It should be understood as the set of changes to get from the expected schema to the actual schema.

[*] Changed the `table2` table
  [-] Removed foreign key on columns (table1Id)

√ We need to reset the MySQL database "TestDatabase" at "localhost:3306".
Do you want to continue? All data will be lost. ... no

Reset cancelled.

Expected behavior
I expect prisma to let me do a migration without telling me that my database schema is not in sync with your migration history when I let prisma take over my database (I mean I am not doing anything manually but creating the database), or also I would expect prisma to let me know what am I doing wrong so I know how to fix it
Prisma information
Prisma Initial Schema (of the example to reproduce the error):
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model table1 {
  id     String @id @db.VarChar(50)
  field1 String @db.VarChar(100)
}

model table2 {
  id       Int    @id @default(autoincrement())
  table1Id String @unique @db.VarChar(50)
}

Prisma last schema (of the example to reproduce the error):
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model table1 {
  id     String @id @db.VarChar(50)
  field1 String @unique @db.VarChar(100)
  table2 table2?
}

model table2 {
  id       Int    @id @default(autoincrement())
  table1Id String @unique @db.VarChar(50)
  table1   table1 @relation(fields: [table1Id], references: [id])

  @@index([table1Id])
}

Prisma schema (of the actual project where I am facing the issue):
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

model associate {
  id        Int     @id @default(autoincrement())
  name      String  @db.VarChar(50)
  balance   Decimal @default(0.00) @db.Decimal(8, 2)
  lastname  String? @db.VarChar(50)
  active    Int     @default(1) @db.TinyInt
  pin       String  @unique @db.VarChar(10)
  userId    String  @unique @db.VarChar(50)
  user      user    @relation(fields: [userId], references: [uid])
  coupons   coupon[]
  venues    venue[]

  @@index([userId])
}

model client {
  id                  Int         @id @default(autoincrement())
  name                String      @db.VarChar(50)
  lastname            String?     @db.VarChar(50)
  active              Int         @default(1) @db.TinyInt
  membershipStartDate DateTime    @db.DateTime(0)
  birthday            DateTime    @db.DateTime(0)
  membershipId        Int
  userId              String      @unique @db.VarChar(50)
  user                user        @relation(fields: [userId], references: [uid])
  membership          membership  @relation(fields: [membershipId], references: [id])
  couponsHistory      clientcouponhistory[]

  @@index([membershipId])
  @@index([userId])
}

model admin {
  id        Int       @id @default(autoincrement())
  name      String    @db.VarChar(50)
  active    Int       @default(1) @db.TinyInt
  userId    String    @unique @db.VarChar(50)
  user      user      @relation(fields: [userId], references: [uid])

  @@index([userId])
}

model user {
  uid         String      @id @db.VarChar(50)
  email       String      @unique @db.VarChar(50)
  phone       String      @unique @db.VarChar(16)
  createdAt   DateTime    @default(now()) @db.DateTime(0)
  associate   associate?
  client      client?
  admin       admin?
  payments    paymentshistory[]
}

model coupon {
  id              Int                     @id @default(autoincrement())
  description     String?
  available       Int
  createdAt       DateTime                @default(now()) @db.DateTime(0)
  expiresAt       DateTime                @db.DateTime(0)
  name            String                  @db.VarChar(50)
  active          Int                     @default(1) @db.TinyInt
  global          Int                     @default(0) @db.TinyInt
  used            Int                     @default(0)
  tipoPromoId     Int
  associateId     Int
  tipoPromo       tipopromo               @relation(fields: [tipoPromoId], references: [id])
  associate       associate               @relation(fields: [associateId], references: [id])
  clientsHistory  clientcouponhistory[]
  venues          venuecoupon[]
}

model clientcouponhistory {
  id        Int       @id @default(autoincrement())
  createdAt DateTime  @default(now()) @db.DateTime(0)
  timesUsed Int       @default(1)
  couponId  Int
  clientId  Int
  coupon    coupon    @relation(fields: [couponId], references: [id])
  client    client    @relation(fields: [clientId], references: [id])

  @@unique([couponId, clientId])
}

model tipopromo {
  id        Int      @id @default(autoincrement())
  label     String   @unique @db.VarChar(30)
  price     Decimal  @db.Decimal(5, 2)
  createdAt DateTime @default(now()) @db.DateTime(0)
  active    Int      @default(1) @db.TinyInt
  coupons   coupon[]
}

model membership {
  id          Int      @id @default(autoincrement())
  label       String   @unique @db.VarChar(30)
  description String?  @db.VarChar(500)
  price       Decimal  @db.Decimal(7, 2)
  createdAt   DateTime @default(now()) @db.DateTime(0)
  active      Int      @default(1) @db.TinyInt
  client      client[]
}

model service {
  id          Int      @id @default(autoincrement())
  name        String   @unique @db.VarChar(100)
  description String?  @db.VarChar(500)
  createdAt   DateTime @default(now()) @db.DateTime(0)
  active      Int      @default(1) @db.TinyInt
  venues      venueservice[]
}

model venuetype {
  id        Int       @id @default(autoincrement())
  label     String    @unique @db.VarChar(50)
  createdAt DateTime  @default(now()) @db.DateTime(0)
  active    Int       @default(1) @db.TinyInt
  venues    venue[]
}

model venue {
  id            Int       @id @default(autoincrement())
  name          String    @db.VarChar(100)
  latitude      Decimal   @db.Decimal(25, 20)
  longitude     Decimal   @db.Decimal(25, 20)
  state         String    @db.VarChar(60)
  city          String    @db.VarChar(60)
  country       String    @db.VarChar(60)
  zipCode       String    @db.VarChar(5)
  street        String    @db.VarChar(200)
  createdAt     DateTime  @default(now()) @db.DateTime(0)
  description   String?
  active        Int       @default(0) @db.TinyInt
  isOpen        Int       @default(1) @db.TinyInt
  associateId   Int
  venueTypeId   Int
  associate     associate @relation(fields: [associateId], references: [id])
  venueType     venuetype @relation(fields: [venueTypeId], references: [id])
  services      venueservice[]
  schedule      schedule[]
  coupons       venuecoupon[]
}

model venueservice {
  id        Int       @id @default(autoincrement())
  createdAt DateTime  @default(now()) @db.DateTime(0)
  venueId   Int
  serviceId Int
  venue     venue     @relation(fields: [venueId], references: [id])
  service   service   @relation(fields: [serviceId], references: [id])

  @@unique([venueId, serviceId])
}

model schedule {
  id        Int       @id @default(autoincrement())
  weekDay   Int
  openTime  DateTime  @db.Time(0)
  closeTime DateTime  @db.Time(0)
  venueId   Int       @unique
  venue     venue     @relation(fields: [venueId], references: [id])
}

model venuecoupon {
  id        Int       @id @default(autoincrement())
  available Int
  createdAt DateTime  @default(now()) @db.DateTime(0)
  used      Int       @default(0)
  couponId  Int
  venueId   Int
  coupon    coupon    @relation(fields: [couponId], references: [id])
  venue     venue     @relation(fields: [venueId], references: [id])

  @@unique([couponId, venueId])
}

model balanceamounts {
  id          Int      @id @default(autoincrement())
  totalPrice  Decimal  @db.Decimal(10, 2)
  price       Decimal  @db.Decimal(10, 2)
  createdAt   DateTime @default(now()) @db.DateTime(0)
  active      Int      @default(1) @db.TinyInt
  users       paymentshistory[]
}

model paymentshistory {
  id                Int             @id @default(autoincrement())
  amount            Decimal         @db.Decimal(8, 2)
  createdAt         DateTime        @default(now()) @db.DateTime(0)
  chargeId          String          @db.VarChar(100)
  userId            String          @db.VarChar(50)
  balanceAmountId   Int             @unique
  user              user            @relation(fields: [userId], references: [uid])
  balanceAmount     balanceamounts  @relation(fields: [balanceAmountId], references: [id])
}

Environment & setup

OS: Windows 11
Database: MySQL 8
Node.js version: 16.13.1

Prisma Version
prisma                  : 4.1.1
@prisma/client          : 4.1.1
Current platform        : windows
Query Engine (Node-API) : libquery-engine 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Introspection Engine    : introspection-core 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Format Binary           : prisma-fmt 8d8414deb360336e4698a65aa45a1fbaf1ce13d8 (at<EMAIL_ADDRESS>Default Engines Hash    : 8d8414deb360336e4698a65aa45a1fbaf1ce13d8
Studio                  : 0.469.0

Hey can you check if there is a database called Weknd already defined at the DATABASE_URL you are defining in the .env file. My guess is that there is already a db schema defined for Weknd. You should be able to resolve the issue I think if you use a different database name.


CH-17: How do I create a new user without an existing one.  No token = no user?

Fixed

@ademars94 How did you fix this issue?

@sahibjotsaggu You need to go into ./app/routes/api.js and find the routing code for '/users'.  In this code, the .post() code block is AFTER the authentication middleware.  To solve the issue, you need to create another route to '/users' BEFORE the authentication middleware and put the .post() code block there.
Like this:
// POST to '/users' (token check happens AFTER this route, so it IS NOT protected)
apiRouter.route('/users')
  .post({
    // Create and save new user
  })

// Route to obtain token
apiRouter.post('/authenticate', function(req, res) {
  // Creates a token for valid user/password
}

// Route middleware to verify a token
apiRouter.use(function(req, res, next) {
  // Checks for valid token on all requests AFTER THIS POINT
}

// GET user index (token check happens BEFORE this route, so it IS protected)
apiRouter.route('/users')
  .get({
    // Sends a response with the user index data
  })

The reason you need to do this is because when you are trying to create a new user, you don't have an authentication token (since only users can get tokens).  Protecting the route for creating new users makes no sense, so we move it before the token check / authentication logic.  HOWEVER, you should keep the .get() block BELOW the authentication.  You don't want unauthenticated users to be able to access the user index, but you want them to be able to create accounts.  If you have any questions let me know.

@sevilayha I am going to create a pull req for this issue, as I think this fix should be made.  I remember having a very hard time figuring this out when I was doing the tutorial.

@ademars94 Thanks for that! Ya, I figured that was the problem but I thought they would've thought about this when writing the book lol


Pack comparer broken after Curseforge site rework

The pack comparer currently only works with packs from FTB (as they are still using the old system).
All other packs just show 0 mods.

Works again
https://refinedstorage.raoulvdberge.com/packcomparer.php


Review custom x-ob-account-id header passed to RS for create payment flows

Background
x-ob-account-id is a custom header that we have added to the API controllers for payments APIs. Example controller: DomesticScheduledPaymentsApi
The value of the header is set to that of the AccountId field in the IDM object for the payment intent. This is an SBAT account identifier (a mongo uuid).
This is a customisation to the OB API which clients need to be aware of. It could cause confusion (if not documented) when making changes to IG routes or if a client wishes to implement their own RS.
Task
Review the purpose of this header and whether we should keep it.

x-ob-account-id this is passed for the following APIs:

DomesticStandingOrder
DomesticScheduledPayments
InternationalStandingOrder
InternationalScheduledPayment

Interestingly, it is not passed for the following:

DomesticPayments
InternationalPayments
FilePayments
DomesticVrp

One would assume that if we needed this AccountId to make a payment then it would be present for all flows.

In the API Controllers for Domestic Scheduled Payments, International Scheduled Payments, Domestic Standing Orders, International Standing Orders the payment data is stored for the Accounts API versus the other APIs where there is no need to store the payment data.
There are designed services in RS for both types of APIs (domestic/international scheduled payments & domestic/international standing orders) to save, retrieve and update both scheduled payments & standing orders for the Accounts API.
In the Accounts API both standing orders & scheduled payments resources can be retrieved for a specific account identified by AccountId or for all accounts that the PSU has consented to.
https://openbankinguk.github.io/read-write-api-site3/v3.1.8/resources-and-data-models/aisp/standing-orders.html#overview
https://openbankinguk.github.io/read-write-api-site3/v3.1.8/resources-and-data-models/aisp/scheduled-payments.html#overview
The value of the x-ob-account-id header is set to that of the AccountId field in the IDM object for the payment intent, used in the services earlier mentioned on the creation & storing of a persisted scheduled payment/standing order within the repository.
In conclusion, the x-ob-account-id header has to be kept.
See also #565.


Added PTPN11 analysis

Currently failing because of a problem finding NP ID in Uniprot
According to Ensembl, the canonical transcript of PTPN11 is ENST00000351677.7 which corresponds to transcript NM_002834.5 and protein NP_002825.3
When I run this cell:
from gpsea.model.genome import GRCh38
from gpsea.preprocessing import VVMultiCoordinateService, \
    configure_default_protein_metadata_service

txc_service = VVMultiCoordinateService(genome_build=GRCh38)
tx_coordinates = txc_service.fetch(PTPN11_transcript_id)

pms = configure_default_protein_metadata_service()
protein_meta = pms.annotate(PTPN11_protein_id)

I get this error:
ValueError: Could not find an entry for NP_002825.3 in Uniprot response

Uniprot lists these as other NP IDs:
NP_001317366.1 NM_001330437.1 [Q06124-1]
NP_002825.3 NM_002834.4 [Q06124-2]
NP_542168.1 NM_080601.2 [Q06124-3]

But same problem with all of these NPs

The UniProt issue started to show up as we started to analyze a larger group of genes/cohorts and in the past days we implemented some workarounds. These include creating ProteinMetadata from UniProt JSON dump or entirely manually.
The documentation to these will arrive with #279 ..
So, please test these out, if possible.

@justaddcoffee
We are putting the notebooks in a separate repo
I moved this notebook there and made a few corrections.
There is one very significant GPC
https://github.com/monarch-initiative/gpsea-cs/blob/develop/notebooks/PTPN11/PTPN11_gpsea.ipynb
I will close this PR, and I invited you to the other repo.


Move Release Pipeline to OIDC

Changes
In https://github.com/databricks/databricks-sdk-py/pull/386, I added a workflow for testing OIDC auth to TestPyPI. This failed due to an existing package: https://github.com/databricks/databricks-sdk-py/actions/runs/6418567925/job/17426619417.
This PR fixes that issue by adding skip-existing: true and at the same time changes the production publishing pipeline to use OIDC as well.
Tests

[ ] make test run locally
[ ] make fmt applied
[ ] relevant integration tests applied

Codecov Report
All modified lines are covered by tests :white_check_mark:
:loudspeaker: Thoughts on this report? Let us know!.


Datagrid OnChange (oninput) search for integers

Is your feature request related to a problem? Please describe.
When using the "search" feature (filters) in a Datagrid, the text (varchar, nchar, etc...) property displays the results as the user inputs the value. However, the same does not occur when using an integer as you have to hit "enter" to get only that one exact parameter.
Describe the solution you'd like
Is it possible to apply the same concept to integers as well?
Describe alternatives you've considered
I've tried to modify the field to a text using a template. However, it did not yield the expected result.
 <RadzenGridColumn TItem="User" Property="UserId" Title="User Id" Sortable="true" Filterable="true">
                <Template Context="data">
                    @data.OwnerId.ToString()
                </Template>
  </RadzenGridColumn>

Additional context
Add any other context or screenshots about the feature request here.

Hey @onemoreuserinthisworld ,
The grid filter will use Contains for strings, this is not possible for numeric types.


Error in the code

Check on https://discord.gg/wtGEjwGsd

Can you explain more here please?

shitty spam


AZ0002 Analyzer

Update Analyzer AZ0002 rule to add more check scenarios:

[ ]  verify if the method has cadncellationToken or requestContext. if not, return failure

[ ] Need to update the AZ0002 description to cover more scenarios

[ ] method signature parameter and response consistence check:  (requestContent must combine with requestContext, response must combine with requestContext

void A (RequestConent content, CancellationToken callellationToken) - Invalid

Response A (CancellationToken callellationToken) - Invalid

Response A (RequestContext requestContext) - Invalid

and so on

[ ] Sync and async method pair check: need to check each sync method coupled with an async method

Duplicate with https://github.com/Azure/azure-sdk-tools/issues/6145


What is reason of undefined reference to symbol 'TLS_server_method@@OPENSSL_1_1_0'?

pkg-config --cflags openssl
-I/usr/local/include
openssl version
OpenSSL 1.1.1d  10 Sep 2019
When i compile my service, it report the following error:
/usr/lib64/gcc/x86_64-suse-linux/4.8/../../../../x86_64-suse-linux/bin/ld: warning: libssl.so.1.1, needed by /usr/local/lib/libevent_openssl.so, may conflict with libssl.so.1.0.0
/usr/lib64/gcc/x86_64-suse-linux/4.8/../../../../x86_64-suse-linux/bin/ld: Module/HttpServer.o: undefined reference to symbol 'TLS_server_method@@OPENSSL_1_1_0'
/usr/local/lib64/libssl.so.1.1: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status

what is the reason?

When you link your service, you may want to consider the output of pkg-config --libs openssl as well.

pkg-config --libs openssl
-L/usr/local/lib64 -lssl -lcrypto
/usr/local/lib64/libssl.so*
lrwxrwxrwx 1 root root     13 Nov 15 18:51 /usr/local/lib64/libssl.so -> libssl.so.1.1
-rwxr-xr-x 1 root root 690488 Nov 15 18:51 /usr/local/lib64/libssl.so.1.1

Did you use that when building your service?

I am not sure, i just add OPENSSL_LIB = -lcrypto -lssl,  but when i issue service, i use libssl.so.1.1 not libssl.a.

You need to include the -L option if there was one, as well.

you mean?
-L/usr/local/lib64 $(OPENSSL_LIB)

I mean it should be part of OPENSSL_LIB's value

Is this resolved now?

Marking as inactive, to be closed at the end of 3.4  dev, barring further input


cplot "points" parameter not explained and not quite right

If you do cplot(exp, points=5) or cplot(exp, points=8) you get a 3x3 image of 9 points.
If you do cplot(exp, points=9), you get a 4x4 image of 16 points.
So it would seem that

The points parameter is not per-axis, but total number of points in the image, which I don't see documented anywhere, and
It's got an off-by-one error somewhere, since points=9 should logically produce a 3x3 image with 9 points.

(I actually wrote this note to myself a few days ago but forgot to submit it.  I've since worked on a  clone of cplot with some changes you might be interested in: documentation, returning axes object, math formatting for axis labels, variable name changes)


Spike/376 - Jessica - Add color tokens to Storybook

Closes #376
Description of Change

Added primitive, semantic, and component color tokens to Storybook
Tokens are documented similarly to Figma / doc site

PR Checklist
Code reviewer validation:

General

[ ] PR is linked to ticket(s)
[ ] PR has changelog label applied if it's to be included in the changelog
[ ] Acceptance criteria:

All satisfied or
Documented reason for not being performed or
Split to separate ticket and ticket is linked by relevant AC(s)

[ ] Above PR sections adequately filled out
[ ] If any breaking changes, in accordance with the pre-1.0.0 versioning guidelines: a CU ticket has been created for the VA Mobile App detailing necessary adjustments with the package version that will be published by this ticket

Code

[ ] Tests are included if appropriate (or split to separate ticket)
[ ] New functions have proper TSDoc annotations

Publish

Changes do not warrant a new version per the versioning guidelines as they are Storybook related.

Deferring to @narin for the actual PR approval as I had setup the general structure so it'd be somewhat self-review to do it myself.
FYI Jessica: pushed another commit checking things out due to pulling in main including Narin's change to rename the tokens import from Colors to colors. Otherwise looked good to me.


[ENH] Circleplot: heatmaps with circles

This needs lots of work but I figured I might as well put it here as a placeholder for now! Edits to come.

Codecov Report

Merging #28 into master will decrease coverage by 1.43%.
The diff coverage is 3.33%.

@@            Coverage Diff             @@
##           master      #28      +/-   ##
==========================================
- Coverage    52.5%   51.06%   -1.44%
==========================================
  Files          19       19
  Lines        1000     1030      +30
==========================================
+ Hits          525      526       +1
- Misses        475      504      +29

Impacted Files
Coverage Δ

netneurotools/plotting.py
7.51% <3.33%> (-0.69%)
:arrow_down:

Continue to review full report at Codecov.

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


Comments can't occur on the same line as variable assignments

Presently, we can add comments in an .env file only if the comment is on a line by itself.  If a comment is added on the same line as a variable assignment, the comment is parsed as part of the value.
Example:
# comment on its own line is ok
FOO=bar # this comment is now included in the value FOO

It would be great if EOL comments were supported.

Closing as a duplicate.  Github hiccupped and I thought the first issue failed to post.


-l with path and without pattern

In ack, we could list all ruby files in /dir/ using:
ack -lf --ruby -- /dir/

Note that no pattern is give. However, I cannot do the same in ag, because ag parses /dir/ as a regular expression. The closest thing I could do in ag is to:
cd /dir/ && ag -l --ruby

The default query when using ag -l --ruby is just ., so you could use that.
ag -l --ruby . /dir/

:+1: interesting.
Also, thanks for the link. I was going to skim the code, I guess that's a good starting point.


Hebrew Translation.

Translate all the string from src/_locale/en/messages.json to Hebrew (in he/messages.json).

Hello, thanks for the translation! I have personally not tested this extension with RTL layouts, is it satisfactory in your opinion?
If you have a few screenshots that would be awesome.
Happy to accept this PR!


rename generated swagger.json

Is it possible to configure SwaggerGen to produce myapi.json instead of swagger.json?

https://github.com/domaindrivendev/Swashbuckle.AspNetCore#change-the-path-for-swagger-json-endpoints


Documentation for batch_x

Hello!
I'm trying to find documentation on the 'batch_" properties in this example: https://github.com/splunk/splunk-library-javalogging/blob/master/src/test/resources/log4j2.xml
Can you point me in the right direction?
Thanks!

Hi @CoryKniefel,
Please find below details of batch* attributes:
batch_interval: The interval, in milliseconds, at which to send batched events.
batch_size_count: The maximum number of events to send per batch.
batch_size_bytes: The maximum size, in bytes, of each batch of events.
Hope this information helps.
Closing this ticket for now.
Please let us know in case of further queries.


TextFormField textAlign center does not center hintText

I been trying everything to center the hintText of a TextFormField but not working, what am I doing wrong?
here's the code
  child: new TextFormField(
        textAlign: TextAlign.center,
        decoration: new InputDecoration(
          hintText: "Full name",
        ),
      ),

Is this supposed to center the hintText? if yes why is it not centering?
whats the supposed behaviour?

Isn't this the same as #16769 ?


CAMEL-15514: Add auto generation of operation IDs

Since operationId is an optional parameter as defined by OpenApi 3 standard, I think that improving the component to be able to auto generate missing operationIds from operation path and http method would benefit users of camel-rest-openapi component.

I wonder if you can end up generate duplicates if that path doesnt take the base path into account eg

user
GET {id}
order
GET {id}

Where user and order are base path, and the the path is {id}, then you have 2 operations with get-id
it should be
get-user-id
get-order-id
And can the id use dash or underscore to seperate the words nicely, otherwise its generated as getid, getuserid etc

Dashes are now used for separating words.
@davsclaus it should not be possible to generate duplicates. OpenApi standard states that all paths are relative to api server url and paths also can not be nested. That guarantees that there can be no duplicate paths. Any base paths are specified at the api server url level.
To expand on your example:
paths

user
GET {id}
order
GET {id}

would be defined in api specification as:

/user/{id} and
/order/{id}


⚠ ClamAV has degraded performance

In 5cc3551, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 68 ms

Resolved: ClamAV performance has improved in 5994053 after 5 minutes.


Chat with friends

If a friend joins u get a toast to start a conversation and this works, but it should also open the chat tab otherwise this is useless.

So you mean when the users clicks on the notification the private message tab doesn't open or the "Chat" tab doesn't open?
It should indeed do that

Yeah thats what i mean. How ever I could also do that, won't be that hard ;)


🐛 [Qbittorrent] Retain old image

Which addon?

Addon name : qbittorrent
Addon version : does not matter

Describe the bug
I wasn't sure it just me or everyone. Only this addons is not deleting the old image over time..
docker image ls | grep qbi
db21ed7f/amd64-addon-qbittorrent                  4.4.5-r0-ls219                                        8f6a23fd78fe   5 days ago      286MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls219                                  da66b6142804   6 days ago      208MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls218                                  7d4bfebd34bf   13 days ago     208MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls217                                  318e8904eff1   2 weeks ago     208MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls215                                  53c0e1eb27e2   3 weeks ago     208MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls214                                  59732be550b8   4 weeks ago     209MB
lscr.io/linuxserver/qbittorrent                   amd64-4.4.5-r0-ls213                                  5b3a16024ea7   5 weeks ago     210MB
lscr.io/linuxserver/qbittorrent                   amd64-<IP_ADDRESS>-r1-ls211                                b048afe5c6c1   5 weeks ago     219MB
lscr.io/linuxserver/qbittorrent                   amd64-<IP_ADDRESS>-r1-ls210                                ca9891fe34ae   8 weeks ago     219MB
lscr.io/linuxserver/qbittorrent                   amd64-<IP_ADDRESS>-r1-ls209                                3920172c68fe   2 months ago    219MB
lscr.io/linuxserver/qbittorrent                   amd64-<IP_ADDRESS>-r1-ls208                                e5842e4a9690   2 months ago    219MB
docker prune does not remove it..
I need to manually remove each.
@alexbelgium, please close it if this is due to home assistant supervisor design.. I remember this was discussed before, i wasn't sure if this has changed.

To Reproduce

Full addon log

Full addon config

System

Supervisor version:
Host system version:

Hi, still due to this bug that prevents me from moving from local build to remote build

If I remember docker prune -a worked, not sure why
Or ha su repair perhaps

docker prune -a  will delete all the container not running.. this includes Let's Encrypt which always stopped.
I will check what ha su repair.. .
Closing this.

I will start with ha su repair next time.

Actually I think they both are the same given this screenshot


Update connectionprofile_getattributednetworkusageasync_1743384794.md

Removing the statement regarding Windows Phone.

Thanks @keith-horton! See also @oldnewthing 's excellent walkthrough here - https://devblogs.microsoft.com/oldnewthing/20210521-00/?p=105234

@msatranjr @mijacobs -

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks!


Erroneous power reported around midnight

Checklist

[X] I have verified that the account and device works in the Comfort Cloud App.
[x] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).

What version of Home Assistant Core?
core-2024.12.3
Describe the issue
looks like there is a consistent report almost every night where the reported current power is unreasonable high >10kW, where the power is reported very high for a few minutes. This throws off the energy calculations when i later integrate over the power curve. I have noticed that there is a different time between 23:00 and 01:00 that this occurs.
I have included the some screen-grabs of the plot. This is based on the sensor:
sensor.NAME_current_power
Different time periods, same behavior.

I have two heat pumps running, both exhibit the same behavior and during the same time.
Running latest version 6.0.1
I noticed that there is a similar report #306
Error/Debug Logs
No response

I had the same than you at the same time. And since then all sensors of my 6 units are not available
only switch on and off are still working but impossible to check the power consumption


M3-1869 Tags going off screen in search bar

Tags on search will wrap now, along with a few other adjustments:

Coverage remained the same at 31.735% when pulling f600b42138493ba7c563a2b593fde49ae4351529 on WilkinsKa1:M3-1869 into 31f3555f56624f484f9c82dea0d6adeaca4c922f on linode:develop.


package.json: Update @babel/polyfill package dependency

Check for any outstanding update of point version of any outstanding NPM package dependency

[x] npm-update

npm-update in progress on bos-tasks-s9p09.
Log: https://<IP_ADDRESS>:8493/logs/npm-update-603-20190322-111515/

npm-update in progress on bos-tasks-3hjf4.
Log: https://<IP_ADDRESS>:8493/logs/npm-update-603-20190322-111515/

npm-update @babel/polyfill done: https://github.com/cockpituous/welder-web/commits/npm-update-@babel/polyfill-20190322-111616

Please manually check features related to these files:
package.json:    "@babel/polyfill": "7.4.0",
webpack.config.js:  // redux-saga uses generators, babel compiles using regenerator runtime included in @babel/polyfill
webpack.config.js:  entry: ["@babel/polyfill", "./main.js"],

The travis failure is ok, the npm module gets updated in #601.


AActor::bActorEnableCollision is not currently handed over

Description
AActor::bActorEnableCollision is not currently persisted on authority handover. This can lead to various physics/collision related issues after an authority transfer.
Workaround
If modifying AActor::bActorEnableCollision from it's default state, re-apply those changes within Actor::OnAuthorityGained.
Internal ticket:
UNR-4146
Environment

HEAD of UnrealGDK master & UnrealEngine 4.26-SpatialOSUnrealGDK
Windows 10

This might have something to do with collision not being replicated by design in UE4 see this discussion.


turbostat number of iterations option

https://github.com/criteo/hwbench/blob/fb1286726d8fc53ff295b5e533191bf8999ceda1/hwbench/environment/turbostat.py#L123
with turbostat 2024.05.10 when using -n1 option to define number iteration, the following error is raised :
turbostat: option '-n' is ambiguous; possibilities: '-num_iterations' '-no-msr' '-no-perf'
using -num_iterations1 resolves the problem.

Solved in https://github.com/criteo/hwbench/pull/20


NetClient a NetMaker client.

NetMaker and NetClient
Netmaker is a networking platform built on the groundbreaking WireGuard protocol. Netmaker unifies your hybrid/multi-cloud, edge, IoT, and Kubernetes environments.
Metadata

homepage URL: https://www.netmaker.org/
source URL: https://github.com/gravitl/netmaker/releases
license: mit, bsd, gpl2+ , SSPL
platforms: unix, linux, darwin,

Would like to request a nix package for NetClient, but both NetMaker and NetClient would also be nice.

Thanks for this fast fix!
Would it be possible to have netclient and netmaker as separate packages?
As netmaker is the server, netclient is the client.
Now it installs both and I think that is not really wanted on all client systems.
Thanks.

someone split the netclient into its own package #244197 recently

Thanks for this fast fix! Would it be possible to have netclient and netmaker as separate packages? As netmaker is the server, netclient is the client. Now it installs both and I think that is not really wanted on all client systems.
Thanks.


[MAIN_LIST] Disappear $

After adding a new item to the main list, all $ icons disappear.

Resolved by other issue.


Investigate ways to target updates from an action

Currently the message endpoint always sends back the whole DOM that got updated. To reduce the payload for performance reasons, it would be useful to target a model update or action to only update a particular fragment of HTML. Currently thinking that could use unicorn:key or element id to target (similar to how loading states work).
Might be particularly useful for child components so that the whole parent component DOM doesn't have to go over the wire.
Not sure if this is an Updater (KeyUpdate?) or an attribute ala loading states. Might need to handle both.

@frbor partial updates have been implemented in the just released 0.17.0. For now, it is based on an unicorn:partial attribute on an action element.
It probably wouldn't be too hard to implement this from Python in the component view by returning a new Updater, but I'm not sure how valuable that might be.
The other potential enhancement would be to be able to target multiple ids/keys (something like unicorn:partial="id1 id2" perhaps?). But, figured that could be an additional improvement in later versions.

@frbor partial updates have been implemented in the just released 0.17.0. For now, it is based on an unicorn:partial attribute on an action element.
It probably wouldn't be too hard to implement this from Python in the component view by returning a new Updater, but I'm not sure how valuable that might be.
The other potential enhancement would be to be able to target multiple ids/keys (something like unicorn:partial="id1 id2" perhaps?). But, figured that could be an additional improvement in later versions.


Add optional package support for phpcs

Instead of requiring a larger image with all the standards enable builds to include optional dependencies or newer/older versions of included standards.

Codecov Report

Merging #290 into master will increase coverage by <.01%.
The diff coverage is 92.3%.

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
+ Coverage   91.21%   91.21%   +<.01%
==========================================
  Files          47       47
  Lines        2868     2903      +35
==========================================
+ Hits         2616     2648      +32
- Misses        252      255       +3

Impacted Files
Coverage Δ

lintreview/tools/phpcs.py
96.9% <92.3%> (-3.1%)
:arrow_down:

Continue to review full report at Codecov.

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


Binary release?

Any plans to release binaries?

I'll try to set up GitHub Actions pipeline later, no promises though.
Also, there's now AUR package available.

I have seen people use goreleaser a lot.

@DannyBen Thanks! Did set it up, now there's binary releases.


chore: Prepare for development, 7.0.2 #trivial

The type of this PR is: chore
Description
Updates version number of the app to prep for the next release!
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

7.0.2 dev prep! - Brian

Cool! So this is a routine patch-level version bump we do after the previous release is out the door?
Should this step be documented in the Release Captain Tasks doc?

This PR contains the following changes:

Dev changes (7.0.2 dev prep! - Brian)

  Generated by :no_entry_sign: dangerJS against 1afa43d5b7b2ba8e38e9a66fc48262333f784973

Cool! So this is a routine patch-level version bump we do after the previous release is out the door?
Should this step be documented in the Release Captain Tasks doc?

Hey @anandaroop yep! It actually is there under Prepare for new release!


[Storage] Improve Download docs

Follow up from https://github.com/Azure/azure-sdk-for-net/issues/22022 .
Enrich docstrings of Download family so that it describes which kind is best for what use case.

Besides docstrings this requires update:
https://github.com/MicrosoftDocs/azure-docs/blob/master/articles/storage/blobs/storage-quickstart-blobs-dotnet.md

https://github.com/Azure-Samples/AzureStorageSnippets/blob/master/blobs/quickstarts/dotnet/BlobQuickstartV12/Program.cs


Where is "_compiled" set up？

Version
2.6.14
Reproduction link
https://github.com/vuejs/vue/blob/dev/dist/vue.runtime.esm.js#L2999
Steps to reproduce
I notice there is a setting here, but I can't find where to set it?
What is expected?
Find positions
What is actually happening?
Did not find

PS: The reason I want to know this is because I have noticed that in some cases when using a functional component,  children are passed a String type. If "_compiled" is set to true, it will return an error.

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


Improve grover with sampler tutorial

Summary
The qiskit doc squad is working on a tutorial checklist based on the Diataxis documentation framework. This PR improves the existing Grover with sampler tutorial using the checklist. A draft of the checklist can be found here.
Details and comments

Use "you" instead of "we"
Add summary section
Grammar fix
add backend option to make sure the tutorial works with ibm quantum instance

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2666

Relevant Lines:
4044

💛  - Coveralls

@Guillermo-Mijares-Vilarino , could you review this?

Of course!

@HuangJunye Any updates to this? Or is it ready to be merged

@kt474 Thanks for pinging. I need to address some of the comments first. After that it should be good to go.

@Guillermo-Mijares-Vilarino Thanks a lot for the positive review comments and discussions1

@kt474 Can you please review the PR? I think it's ready for merging. Sorry for the long delay.


Add shortcuts under GpChat buffer

Right now, we binding keys globally to Gp commands.
If it's better to have shortcuts to trigger the Gp commands under GpChat buffer.
Like r in normal mode in GpChat to trigger GpChatResponse.
Something like Neogit did when under NeogitStatus buffer.

Done, closing.


fix typo "trigger"

Noticed a small typo in the UI. Updated the word "tigger" to "trigger".

Thank you!

Np thanks for pointing that out


Remove git dependencies from tests

You should be able to delete the local tipster git repository and have the tests pass.

Fixed @neelimas.


Cow: Watch motorglo.com

Cow requests the watch of the watch_keyword motorglo\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

motorglo\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Updated clients.go structure Name under (./test/clients.go) to make it exportable to other modules

Changes
Changed Name of  clients structure Client in order to make it exposed to other modules to use easily
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes

/ok-to-test

/ok-to-test

/retest


feat(rules): expose rule.consequence.promote.objectIDs string array

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
yes

Related Issue
Fix #694

Need Doc update
yes

Describe your change
Add support of the new field objectIDs.
The  tricky part here is to make this a discriminate union and not just make objectID & objectIDs optional, one of both needs to be set.
Note: a less aggressive change can be to simply add a new property with checks and logic operations in the setters, but this can be error-prone IMO.

For this change I would go for something simple: having two properties objectID and objectIDs.

Yes I was hesitating about this one, this means moving the error checking from compile-time to runtime (not the best DX IMO), but of course, if you think that the price to pay is too high, then we can go for the two properties solution.

@Aallam
Thanks for the PR and I agree with everything that @Ant-hem already said. Just to give you a bit of context here:

Yes I was hesitating about this one, this means moving the error checking from compile-time to runtime (not the best DX IMO), but of course, if you think that the price to pay is too high, then we can go for the two properties solution.

Yes that's very true indeed. However doing that also has its advantages:

if the engine does not consider it as an error anymore in the future (which already happened), the users won't get a compile-time error for something that now works
it reduces the dev. cost at the company level of having an extra safety net at the API client level which is already implemented at the engine level
if the engine improves its error messages, all users benefit from the change, not just the one who got the latest version of API client X, Y or Z


White screen

changed private const string NAME_MODULE_CLIENT = "client_panorama.dll"; to private const string NAME_MODULE_CLIENT = "client.dll";

I figured out but I want to know its bypass vac.


Implemented sign in flow additionally ( oAuth2 form )

Implemented cellURL input form (only HTML)

and make you can login via oauth2 form.

Applied, thanks!


sc-2059 allow multiple status filters

This allows multiple status filters to be passed into the ListVASPs endpoint in the GET params. The GET requests end up looking like this:
/v2/vasps?status_filters=pending_review&status_filters=verified
I also included the pagination bug fix here, since I was in the area anyway.

Codecov Report

Merging #216 (91d51cd) into main (f9665fd) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main     #216   +/-   ##
=======================================
  Coverage   31.31%   31.31%
=======================================
  Files          50       50
  Lines        5639     5639
=======================================
  Hits         1766     1766
  Misses       3507     3507
  Partials      366      366

Impacted Files
Coverage Δ

pkg/gds/admin.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Should the use_cache setting = True by default?

Currently use_cache=False by default. Should it be set to True? I think so. Changing this default would be a nudge toward good neighbor behavior with the Overpass server by reducing the number of duplicate calls made by a single user re-running code. And it could be easily turned off with ox.config(use_cache=False).

I think so as well. When reviewing the library that was a concern I had about its use in practice. It depends a lot of overpass and it would make sense to have it cache by default. That said, I expect some server based users might experience issues with cache build ups on server machines if it is not flagged. I am not sure how big of a problem that really is though.

Resolved by 7455fae.

A message reporting the current response caching status is now automatically logged (to terminal and/or file) when ox.config() is called, as of 5a3188f.


Python 3.6 sunset end of 2021

Python 3.6 goes out of support, e.g. security updates plus general widespread availability, at the end of this year.
If you are wondering about python's release cadence you can review it here: https://www.python.org/dev/peps/pep-0602/#implementation
The beiwe-backend repo will be shifting to Python 3.8.x - this is purely based on the platform versions available to us on AWS Elastic Beanstalk servers as dictated by this website at this moment in time: https://docs.aws.amazon.com/elasticbeanstalk/latest/platforms/platforms-supported.html#platforms-supported.python.
It is extremely unlikely that such an upgrade to the runtime version will result in any compatibility issues for forest, but I'm not the expert on that.
The forest runtime environment as controlled by the beiwe-backend is trivially portable up to python 3.8, and more recent versions - python 3.9 is the current release, python 3.10 is slated for release on October 4th - is workable given some lead-time.  I am always a proponent of upgrading to the latest and greatest, so if anything at all looks nice for forest in python 3.9 or 3.10 we can do it.
For your perusal and consideration:
https://docs.python.org/3/whatsnew/3.7.html
https://docs.python.org/3/whatsnew/3.8.html
https://docs.python.org/3/whatsnew/3.9.html
The python 3.10's what's-new page does not have an identical url/page structure, but there is this nearly-identical one - I have left this as plaintext for copy-pasting because I suspect it will change on us come Oct 4:
https://docs.python.org/3.10/whatsnew/3.10.html

In a meeting yesterday we decided "might as well do 3.8 because newer is better."
And so it begins....

Note that regressions are a risk because there is no test suite.

python 3.8 is live on the github repo and on all onnela lab servers.  Data processing, including forest, now runs on python 3.8.

Static type checking fails with Python 3.8: #59


HEP 1 implementation tracker

This issue tracks how much of #4593 has been implemented. Since HEP 1 is still in progress, this is not necessarily a complete list of things that need to be done.

[x] hpx_core #4649
[x] hpx_local
[ ] (bonus) use object libraries #4135
[x] allow choosing runtime type at run time #4646
[ ] add CMake options for building only hpx_core or hpx_local

I'll close this, but please reopen if you think this is a better place to track these things. The split of libhpx into multiple libraries has already happened. I think object libraries can be tracked separately, and exposing the different libraries to users is better done in separate projects.


feat: add drag and drop column rearrangement for table viz

Bringing this over from superset-ui (some discussion was on the old PR)
https://github.com/apache-superset/superset-ui/pull/1423
SUMMARY
Add the ability for the end user to drag and drop columns for a table visualization
Table designer can enable column rearrangement in the config options when building the table. Then, user can drag columns around
These are volatile changes, so when the browser is refreshed, the column order goes back to default. Demo is in the chart edit page, which is just for simple demonstration purposes. This is intended for the end user to be allowed to rearrange the columns.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
OFF

ON

ADDITIONAL INFORMATION

[x] Has associated issue: #19380
[x] Fixes #19380
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

@rusackas @junlincc I finally got around to merging in the monorepo, so I'm reviving this PR :)

@ktmud let me know if there's anything else you think should be changed!

@ktmud same with this one. Anything else you think I need to do for this one?

@ktmud same with this one. Anything else you think I need to do for this one?

This looks good. Thanks for your contribution! And sorry for missing your message earlier.

Awesome, no prob, I'm glad you liked it!


order of query reversed from previous version

Minimal Example:
	var stuffWeCite = TAFFY([{
   	 "first": 1,
   	 "second": 2,
   	 "third": 3,
   	 "fourth": 4
	 }]);

	var filterVar = {"first":1};
	var someVar = stuffWeCite()
	.filter(filterVar)
	.select("first", "second", "third", "fourth");

	console.log('someVar: ' + someVar);

Using the older version of TaffyDB, this produces: "someVar: 1,2,3,4" but does not work in Safari.
Using the current version of TaffyDB, this produces: "someVar: 1,4,2,3" and does work in Safari.
I was previously using this[0], this[1], &c. to access the results of my query, so updating broke the way that my program worked. I am probably missing some kind of best practice, but it seems like the previous behaviour should be the desired/intended behaviour.

This does indeed seem related to the Safari workaround.  I will also try this with the regression tests (V8) to see if I get the same results.

From looking at the source, it appears the arguments are being alphabetized (run through a function called sortArgs), which results in the selected data being in the alphabetized order of the property names you give it. IMHO, this is not the correct output for .select, because there is no way to know what the order of the values will be (unless you happen to know that the property names are being alphabetized, which you wouldn't know without looking at the source). This was a breaking change in 2.7 that should be labelled as a bug.

For reference, here's an even simpler example:
var t = TAFFY([{first: 1, second: 2, third: 3, fourth: 4}]);
t().select('first', 'second', 'third', 'fourth');

On older versions, this returns [[1, 2, 3, 4]] as expected. On 2.7, it returns [[1, 4, 2, 3]].

I seem to recall the change also being used to add some predictability into select. Since records can come from a lot of areas (server database, serializing forms, etc) the actual order of values in a given object may not be consistent - or even contain all the same values. You can manually specific the order of columns of course but not having .select() change the order for different subsets of the records was the goal.

Assuming that TaffyDB is built on trying to emulate basic DB functionality, then the order of the columns specified in the select should be the same as the order of the values returned.

I'm open to either. Technically just calling .select() without passing in column names may not be that viable of a use case because of the reasons we both outlined. If you the first record in your selection has the columns in a different order you will see the columns move. Similarly if the first record has missing values you again will see the order change.
The breaking change is unfortunate.
So it seems like we have three options:

Attempt to honor the column order (probably ideal).
Provide some predictable automatic order (such as alphabetical).
Make .select not work unless you provide an order. Either by passing in via .select("id",'first","last") or maybe by setting it global .setColumnOrder("id",'first","last");

Thoughts?

Thinking in DB terms (which I'm very familiar with, as you may have guessed), querying select * from table will return all the columns in the order they are defined on the table. But since TaffyDB doesn't have column definitions, per se, having the ability to specify an optional column order on the table (e.g. .setColumnOrder) could be really handy, although I don't know how much work that would take to implement.
I think the rules around that would be pretty straightforward then:

if a column order is specified in the .select, use it
if no column order is specified in the .select (i.e. just .select() with no params) but there is a column order specified on the table, use the table's column order
otherwise honor the order of the properties on the source object

That sounds good in general. But for your third option:

otherwise honor the order of the properties on the source object

Typically it would only honor the order of the properties of first object in the results set which may very record to record. Additionally I'm insure if this is even possible in Safari anymore.
As an aside, I built TAFFY primarily for use with .get() and .each() intending people to use the objects directly. Methods like .select() and .supplant() were added later to let you have structured ways of getting data out - but unlike SQL it wasn't built around .select(). I prefer your method if we can make it work cross browser, otherwise I'd suggest we move .select() to only return the internal TAFFY record identifier if no global column list is set and no columns passed into .select().
@mmikowski what do you think?


Add --resume arg to continue an interrupted bulk export

Fixes: #333
Notable discussion items:
CLI UX
I was torn about how to have the user initiate a resumed export. For context, the normal bulk export kickoff looks like:
cumulus-etl export URL DOWNLOAD_DIR --options

And that URL can contain export parameters like _type=Patient and the options can also contain parameters like --since=xxx or --task patient (the two sets of parameters get blended). We detect how to do authentication based off the URL too (by hitting the .../metadata endpoint and such).
Now when resuming, few of those args make sense. The normal kickoff URL doesn't really make sense, since the group and all the args have already been decided. Same for args like --since.
The server gives us a URL to hit to resume, but there are no restrictions on that - it could be a different hostname entirely, so we can't rely on it for auth. (So in that sense, the normal kickoff URL would still be helpful, but just for auth.)
So two problems:

How do we get authentication info (i.e. how does the user tell us the base FHIR URL)?
How does the user most sensibly provide us the resume URL?

For (1) there, we do have a pattern used in the ETL of --fhir-url=xxx, used in cases where the user already exported ndjson but we still need to talk to the server (like for DocRefs or Meds). So that's an option.
But (2) is a little tricky. Do we add an arg like --resume=xxx and still require the pointless input URL? If the user provides args like --since=xxx with a resume URL, should we complain? Should the user replace the input URL with a resume URL? If they do, can we autodetect that, or do we need to add a boolean --resume flag?
I went with the following approach:

To resume, the user provides a --resume=xxx flag
The user still provides an input URL (from which we grab authentication info)
We ignore any provided group or _type=xxx params in the input URL
We ignore any --since=xxx style args
We notify the user of this with a Resuming bulk FHIR export… (all other export arguments ignored) message

Benefits:

The user just presses the up arrow and adds a new arg onto the list, rather than entirely transforming the command
We don't need to add a new argument just to get the authentication URL in this one case.
It's the same UX as bulk-data-client uses (though they call it --status=url but I don't feel like that's a very obvious argument name).

Downsides:

It's weird that we accept but ignore all this export info - but that's only truly weird if the user is constructing a new command line from scratch. It feels more natural to me in the "press up, add arg" case.

Thoughts?
Quoting the URL
When an export starts, we now print a message about how to resume if it gets interrupted. It now looks like:

I'm soliciting opinions on:

The wording
The bolding
The fact that I quoted/escaped the URL (see the https%3A// bit). I did this so that you could double click the URL and the whole thing, from --resume= to the end, would be highlighted for copying. Colons (at least on my terminal) break up the terminal's word-detection and you have to manually click and drag around to be able to copy the full thing.

Is it weird that it's escaped?
I did add code so that if you have a resume URL from some other source, we still accept pure URLs. But if it looks escaped, we unescape it first.

Back to comma delineated args
You may remember I recently switched from comma delineated URL params to repeated params, because it was easier at the time and the spec preferred repeated params. Well... after testing it more, Cerner can't seem to handle them, even though it's a SHALL requirement. So. Back to commas.
Checklist

[x] Consider if documentation (like in docs/) needs to be updated
[x] Consider if tests should be added

I'm going to approve this separately from my comments on the PR description, since I think this approach is fine and my take is very subjective.
With that said, from my end:

I don't know that I care about matching the bulk client's CLI UX
We have all the information needed to give a user the exact command they should rerun, rather than just a param to add

So to my mind, we could spit out a command with the necessary resume arguments, and exclude all the extra args, and that would be fine - it's functionally going to be the same action (copy/paste some text), and will be a little more explicit on 'what are these two things actually doing when invoked'. But feel free to disagree.
I also don't hate the idea of writing out a a command or config dict to disk alongside the download, and pointing a user to the path to that file, and have the resume command just load in whatever you need from there, and delete that when the download completes successfully, though that does externalize the functionality in a way that's not terribly clean.

OK we talked offline - we think we're "fine" with this approach as is (the --resume=URL approach). We don't think we're locking ourselves into this, if we want to change how this works in the future. We remain curious about some magic log-reading resume feature - could add in the future maybe. But for now, we'll just go with this (and add a section in the docs about it).


Create a microservice to use for the LLM / VectorStore

@brayanpena530 Please move all the code to https://github.com/LarryBattle/DocuDive/blob/main/v1.1_microservice/
I see code at the root of this directory

Related issue: https://github.com/LarryBattle/DocuDive/issues/2

@brayanpena530 Can you output the timeformat in this
>>> datetime.datetime.now().strftime("%Y-%m-%dT%H:%M:%SZ")
'2018-09-05T14:09:03Z'

sure thing

I moved the code in this PR and make it match the /keepalive and /api/ask endpoints
Changes here: https://github.com/LarryBattle/DocuDive/pull/27


Returned Crystal values are prematurely freed by GC

Currently, if an mruby function returns a nontrivial object which is an instance of a wrapped Crystal class, Anyolite does not ensure that the object is excluded by the Crystal GC, leading to potential memory leaks.
Example: Wrapping a simple addition would create a new mruby object referencing a new Crystal object, but Anyolite does not store a reference to the object, leading to potential deallocation (in constrast to objects created by a constructor method).

Should be fixed by now.
However, this does only work with objects of classes which have a default constructor.


Fix dsa-sha1 signature URLs.

The dsa-sha1 xml namespace urls are malformed, they use a http, instead of http:

Pretty straightforward issue and solution

Thanks for the contribution @rectalogic


[ospec:test] Bug? When several tests or specs share the same name, only one runs.

It has bitten me several times now. The identical names were identical (either oversight after copy/paste or faulty operator precedence assumptions ('foo (' + condition ? 'bar' : 'baz' + ')' => always 'baz)').
o.spec('parent', function() {
    var spy = o.spy()
    o.spec('foo', function(){
        o('bar', spy)
        o('bar', spy)
    })
    o.spec('foo', spy)

    console.log("count", spy.callCount) // 1, when 3 would be expected.
})

I don't know if you want it to work as is, warn the user or fail the suite when some names are identical (one of the last two would make more sense to me).

@pygy IMHO it's usually bad style to have duplicate test names (you're not likely testing a unit if it's ever a good idea), but I do agree that ospec should be smart enough to deal with it and just move on.

@isiahmeadows Agreed, it happened to me by accident, twice, and I would have changed the names had I noticed it differently anyways... I would appreciate a notification/error, personally, but maybe some people wouldn't.

Fixed

Great many thanks (just in case you missed the 🎉  reaction). Note to self: :tada:, at last I found it.


[NFC] Suppress retroactive conformance errors

These warnings are showing up in tests about types and protocols that are defined within this package, so they don't pose a problem for future library evolution. Instead of using the @retroactive attribute, which isn't supported by older compilers, this change fully qualifies the type and protocol in the relevant conformance declarations, which suppresses the issue.
Re: https://github.com/apple/swift-evolution/blob/main/proposals/0364-retroactive-conformance-warning.md

@swift-ci Please test


Feature Request: Please support multithreading analysis!

Even using default option myth can run during days for a single contract but always using a single thread.
Now that the official python implementation has a workaround for the gil since 3.9, is there a way for Myhril to benefit from it?
Please note there are also plans to have a faster data serializer in order to bring expected speed. So the decision shouldn’t be based on the speed for serializing objects using current api.

That sounds great. We will try to use this.

That also means that right now, all new code change should ensure thread safety.

An alternative would be to simply use the Numba packages which can run threads without using the gil at all.
It would also dramatically speed up single threaded mode.

A large portion of the time spent when analysing contract is actually performed in C code in the Z3 solver.
For example, when analysing 0xbb9bc244d798123fde783fcc1c72d3bb8c189413 using -t 1 limit from mainnet, for 2779 queries completing in 2200 seconds on my machine, it took 129 seconds in Z3_optimize_check with only 981 calls and 1798 calls to Z3_solver_check_assumptions taking 449 seconds (and apparently without calling anything else in both cases).
@norhh so the question: is there no way to do something else not depending on the result while those functions are called? Or since I’m seeing often a thread being spawn it’s already happening in multithread mode?

Yes, it was possible. But we implemented multithreading previously but there was a slowdown due to GIL. We can't exactly use multiprocessing as Z3 isn't serialisable friendly. Switching to other versions might help with multi threading.

@norhh I was talking regardless of the Gil of course. Please note Numba doesn’t rely on serialization and can run even old Python versions.
So what can be done while z3 works on Z3_optimize_check or Z3_solver_check_assumptions? Something that doesn’t access the Python api if possible.

@ytrezq The previous approach to multithreading was based on a previous design of laser and mythril, with the current architecture multithreaded symbolic execution would look a little bit different.
Right now the process can generally be seen like this:

There is a single worker thread ( the main thread )
There is a worklist of states that need to be processed
The worker fetches a state from the worklist according to some strategy
The worker computes the successor state(s) for that state and adds them to the worklist

We can parallelize this by having multiple worker threads. (there will need to be some concurrency control added for some of the features & plugins in laser)
I'm not sure what the limitations of Numba are, so I'm not sure how that affects our ability to paralellize things.

@JoranHonig: in the case of Numba here’s the 2 limitations:

The function and it’s subfunctions (including in dependent modules) must not rely on the Python C api (not that it doesn’t code C code if the C/C++ code being called doesn’t).
And the native types of all values in the function and subfunctions (including in dependent modules) can be inferred.

That’s it! and then the process to parallelize is just to add this function decorator:
@njit(parallel=True,nopython=True) before function declaration and it will automatically vectorize it and parallelize it (taking care of thread safety). And as the name jit and nopython=True implies, even if the result can’t be vectorized or panellized, it will run far faster than in normal mode. Numba is even compatible with Python3.4 which mean the approach would work right now.
And basically, the implemented workaround which can be tested right now in the latest version works exactly like multiprocess mode except it remove the ipc overhead.
But what I’m seeing is between 20% to 55% of the time spent in the main thread is performed in the solver. That time spent doesn’t requires the python interpreter at all which means it doesn’t even care about what the gil is about.
We already have part which is performed in a separate thread (I failed to find which one but it represent a 10% increase over single thread mode) from time to time even if the program is bound to main thread speed, so why not decrease a one day analysis time by 7 to 10 hours by spawning works which isn’t related to the solver off the main thread (leaving solver work in the main thread).

@norhh yes I known the codebase is no longer related. But neverless I’m interested about getting more details about in which functions and files it was done.

@norhh any chance this get into mythx?

@ytrezq We are currently working on other issues, we will inform after it's decided.

@norhh might be a good solution for your servers which would allow the previous attempt to just work. I found a gil free version of Python3.6 https://github.com/larryhastings/gilectomy

@norhh might be a good solution for your servers which would allow the previous attempt to just work. I found a gil free version of Python3.6 https://github.com/larryhastings/gilectomy


CI

As nvidiagpubeat is now part of eBay, we might have to explore travis.
I understand we used circle ci earlier and even got it working. But we have explore Travis as eBay organization is activated with Travis.
https://travis-ci.org/eBay/nvidiagpubeat
https://docs.travis-ci.com/user/tutorial/

I can try working on this as I am familiar with TravisCI.


Fix not being able to select the first item in the Wifi list without a crash.

in frame_wifiscan.cpp  idx is incremented before checking if the menu item is that id, causing it never to be found, and over run, causing a crash. Fix was to inc after check.

Do you mean that the crash occurs when selecting the first network in the Wi-Fi list?
Can you provide more information? I can't reproduce the problem.
https://github.com/m5stack/M5Paper_FactoryTest/assets/44396641/d1ad774f-40b6-427e-a251-64074d16af15

Yeah, here you go. Looks like I have a few less networks around too
https://github.com/m5stack/M5Paper_FactoryTest/assets/10201853/12c81174-01dc-4e89-8835-47da4e719efb

Can you send me the log output from the serial port?

--- Terminal on COM6 | 115200 8-N-1
--- Available filters and text transformations: colorize, debug, default, direct, esp32_exception_decoder, hexlify, log2file, nocontrol, printable, send_on_enter, time
--- More details at https://bit.ly/pio-monitor-filters
--- Quit: Ctrl+C | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H
[â�€â�˜â�‚â�‚â�‚ï¿½ï¿½mâ�‘umï¿½ï¿½ï¿½ï¿½2-hal-cpu.c:214] setCpuFrequencyMhz(): PLL: 480 / 2 = 240 Mhz, APB: 80000000 Hz
[   456][I][esp32-hal-psram.c:96] psramInit(): PSRAM enabled
M5EPD initializing...[  2640][D][M5EPD_Driver.cpp:70] begin(): Init SUCCESS.
[  4979][D][GT911.cpp:35] begin(): GT911: Initialization
[  5094][D][font_render.c:71] font_cache_init(): Chace use 51104 bytes
[  5198][W][Wire.cpp:204] begin(): Bus already started in Master Mode.
[  5198][W][Wire.cpp:204] begin(): Bus already started in Master Mode.
[  5471][D][systeminit.cpp:177] SysInit_Start(): done
OK
[  5971][D][epdgui.cpp:131] EPDGUI_MainLoop(): Run Frame_Main
[  6685][D][font_render.c:71] font_cache_init(): Chace use 6820 bytes
[ 15729][D][epdgui.cpp:83] EPDGUI_Run(): Exit Frame_Main
[ 15730][D][epdgui.cpp:131] EPDGUI_MainLoop(): Run Frame_WifiScan
[ 16830][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 0 - WIFI_READY
[ 16938][V][WiFiGeneric.cpp:338] _arduino_event_cb(): STA Started
[ 16939][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 2 - STA_START
[ 21246][V][WiFiGeneric.cpp:381] _arduino_event_cb(): SCAN Done: ID: 128, Status: 0, Results: 2
[ 21247][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 1 - SCAN_DONE
[ 23563][D][epdgui.cpp:83] EPDGUI_Run(): Exit Frame_WifiScan
[ 23565][D][epdgui.cpp:131] EPDGUI_MainLoop(): Run Frame_WifiPassword
[ 38429][D][epdgui.cpp:156] EPDGUI_AddFrameArg(): 1
[ 39206][D][epdgui.cpp:83] EPDGUI_Run(): Exit Frame_WifiPassword
[ 39208][D][epdgui.cpp:131] EPDGUI_MainLoop(): Run Frame_WifiScan
[ 40751][D][frame_wifiscan.cpp:215] Connect(): SSID = [Homes Internet]

[ 40751][D][frame_wifiscan.cpp:216] Connect(): PSWD = [letsgoontheinternet]

[ 40755][V][WiFiGeneric.cpp:97] set_esp_interface_ip(): Configuring Station static IP: <IP_ADDRESS>, MASK: <IP_ADDRESS>, GW: <IP_ADDRESS>
[ 40813][V][WiFiGeneric.cpp:353] _arduino_event_cb(): STA Connected: SSID: Homes Internet, BSSID: 9c:3d:cf:4e:1e:ab, Channel: 10, Auth: WPA2_PSK
[ 40815][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 4 - STA_CONNECTED
[ 42096][V][WiFiGeneric.cpp:367] _arduino_event_cb(): STA Got New IP:<IP_ADDRESS>
[ 42097][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 7 - STA_GOT_IP
[ 42100][D][WiFiGeneric.cpp:991] _eventCallback(): STA IP: <IP_ADDRESS>, MASK: <IP_ADDRESS>, GW: <IP_ADDRESS>
[ 43891][V][WiFiGeneric.cpp:381] _arduino_event_cb(): SCAN Done: ID: 129, Status: 0, Results: 2
[ 43891][D][WiFiGeneric.cpp:929] _eventCallback(): Arduino Event: 1 - SCAN_DONE
Guru Meditation Error: Core  1 panic'ed (StoreProhibited). Exception was unhandled.

Core  1 register dump:
PC      : 0x4008b507  PS      : 0x00060c30  A0      : 0x800e6964  A1      : 0x3ffb2390
A2      : 0xa5a5a5a5  A3      : 0x00000000  A4      : 0xa3e01c5a  A5      : 0xa5a5a5a5
A6      : 0x00ff0000  A7      : 0x00000000  A8      : 0x800fa75c  A9      : 0x3ffb2390
A10     : 0x3ffb244c  A11     : 0x00000000  A12     : 0x00000001  A13     : 0x3ffb23b0
A14     : 0x00000000  A15     : 0x00000004  SAR     : 0x00000011  EXCCAUSE: 0x0000001d
EXCVADDR: 0xa5a5a5a5  LBEG    : 0x4008618d  LEND    : 0x40086195  LCOUNT  : 0x00000027

Backtrace:0x4008b504:0x3ffb23900x400e6961:0x3ffb23a0 0x400def25:0x3ffb23c0 0x400df1be:0x3ffb2410 0x400df799:0x3ffb2480 0x400df816:0x3ffb2750 0x400d2d7a:0x3ffb2770 0x400d3467:0x3ffb27c0 0x400e02
57:0x3ffb2800 0x400fc929:0x3ffb2820

ELF file SHA256:<PHONE_NUMBER>000000

Rebooting...
[â�€â�˜â�‚â�‚â�‚ï¿½ï¿½mâ�‘umï¿½ï¿½ï¿½ï¿½2-hal-cpu.c:214] setCpuFrequencyMhz(): PLL: 480 / 2 = 240 Mhz, APB: 80000000 Hz
[   456][I][esp32-hal-psram.c:96] psramInit(): PSRAM enabled
M5EPD initializing...[  2640][D][M5EPD_Driver.cpp:70] begin(): Init SUCCESS.
[  4987][D][GT911.cpp:35] begin(): GT911: Initialization
[  5102][D][font_render.c:71] font_cache_init(): Chace use 51104 bytes
[  5205][W][Wire.cpp:204] begin(): Bus already started in Master Mode.
[  5206][W][Wire.cpp:204] begin(): Bus already started in Master Mode.


The CrossSporkClient does not work with emulator

For local development, the EVM Gateway uses the Emulator, as a Access Node backend.
However, with the introduction of the CrossSporkClient, there's a panic when trying to run the Gateway locally:
panic: failed to get node version info: client: rpc error: code = Unknown desc = not supported

That's because the Emulator does not support the GetNodeVersionInfo method.

Fixed in https://github.com/onflow/flow-evm-gateway/pull/276


pull request for e2e test at 01/19/2017 19:15:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Have pdf's per part?

In the overall table of contents you can click on a part, which gives you the same information as you already had on the page you were on, but only for this part.
Giving the whole treeview on the part page might be too much for the server though. We should test and check whether this is the case.

Also, we don't have pdf's for parts. Might be useful.

We should go for the middle ground: give a list of chapters and sections on the part page, give sections en lemmas on the chapter page.

Getting @aisejohan involved in the discussion: he has to change the Stacks project Makefile for this after all.

OK with doing this in principle, but I am worried overall with the assumptions that go into this. Would this mean other people using gerby-website would have to do the same?

Those are the kinds of customisations that people who want to use Gerby will have to decide for themselves. It's just a matter of (not) having links on certain pages, Gerby is not responsible for the creation of the pdf's. It's not supposed to be perfect straight out of the box, because every project will be different.
Gerby is free, but support for Gerby isn't. It's how Raymond and I will become millionaires!

Likely story!

Quick reminder for @aisejohan, at the moment this is the only thing left to be done for the release.
The following would be convenient filenames, as they are based on the label in the TeX, so that everything is automated:

preliminaries.pdf
schemes.pdf
topics-in-scheme-theory.pdf
algebraic-spaces

etc.

These is actually a problem this way: there will be two files named schemes.pdf. So the file for part 2 should be called part-schemes.pdf, likewise for the others.

OK, so I am going to put part- in front of all of the names. But I am not sure where you are taking the names preliminaries, schemes, topics-in-scheme-theory, etc from? I guess you are taking these from the current website?

These are in the label, except that the tags for parts are not yet in the actual tags file. So that's another thing that needs to happen.

By, "in the label" I mean that at some point we created a modified tags file, containing tags for parts. These are the labels I am referring to. So

We need to permanently add tags for parts.
The labels associated to these tags are what should be used for the filename of the parts.

Yes, OK, so the only place we store these are going to be in the tags file. Currently I am making these tags and their names from the file chapters.tex in the script add_tags.py which uses the function get_parts to find the parts...

These must be in the tags file, as that is the philosophy of Gerby. So the fact that currently they are not in the tags makes it impossible to run plasTeX on the output of the web directive in the Makefile.

Right, yes, I am going to add them to the tags file soonish... I am first going to update the current website one last time...

OK, I was just looking at this. Of course, it is relatively easy to make parts latex files. But getting the references and the numbering to work out correctly is a huge pain in the neck. Currently, when you compile the book.tex (either the one that is created in tmp which has no tag information or the one created in tags/tmp which does have the tag info) then the internal refs work. Or if you compile all the chapter files into pdfs (either in the main directory of stacks-project or when running the make tags target) then the cross references work and more importantly you don't get ?? when there is a reference. In other words, when you have downloaded only one of the chapter pdfs, then of course the cross refs don't work, but at least you see the chapter name and section.number where you should look for the reference.
Of course it is possbile to get this working with parts also, but I don't want to invent a system where you get crossreferences between the parts pdfs and get a whole new numbering scheme which may confuse readers. Yes, I know I can also get the numbering to work out too, but it is just too cumbersome and not worth it in my opionion. In stead, may I suggest that in stead of the parts.pdf download you offer a list of chapter.pdf downloads in stead? Is this easy to do in the website code?
Let me know.

You are right that doing this is opening Pandora's box. So let's not do this for, I'm okay with that.
We could collate all the chapter pdf's (preferably in the Makefile), but that's a suboptimal solution.
In an ideal world the setup for all the TeX files is sufficiently flexible to handle parts too. But I'm not volunteering to rewrite all the Python and Makefile code to accomplish this :). So I'll remove the part links again for now.

Closing this for now.


Add ability to merge repository branches

As described in the docs.

Thanks!

This was released as part of https://hex.pm/packages/tentacat/0.2.0


Job application review page form

Some things I wanted to get in for now. Some parts are placeholders until the tickets get picked up shortly.
The main thing is the idea that a job application has completed steps. It is very possible that we will want to know this information, and we definitely want to ensure all steps are complete before submitting applications. It will have the added benefit of cleaning up some of the "which steps are complete?" sidebar logic. The final details of how we present incomplete steps to the user needs to be ironed out.
I think this approach is an improvement on the Create a job listing journey where we check that all forms are valid (it's a bit nasty imo)

Review app deployed to https://teaching-vacancies-review-pr-2959.london.cloudapps.digital


how to install it from npm

i run this command:
npm install bootstrap3-wysihtml5-npm
I added this into my app.js (i use laravel 5.4):
require('bootstrap3-wysihtml5-npm');
and i got this error when I run "npm run dev":
These dependencies were not found in node_modules:

bootstrap.wysihtml5.commands
wysihtml5
bootstrap
bootstrap.wysihtml5.templates

@mockiemockiz  Did you ever find a solution for this? Discovering the same problem.

@rbruhn @mockiemockiz
The project is only on bower. I've found a fork on npm, but i don't recommand it.
Information
FYI, You can install Bower dependencies through npm install.
 While Bower is maintained, we recommend using Yarn and Webpack for front-end projects read how to migrate!

So according to this documentation. You can do this :
Solution

Place this line in your dependendecies on your package.json

"dependencies": {
    ...,
    "@bower_components/bootstrap3-wysihtml5-bower": "bootstrap-wysiwyg/bootstrap3-wysiwyg",
    ...
  }

This command looks for the dependecy from Github repository.

In my case it works with yarn (npm equivalent, both use package.json). So i recommand you using yarn install

The result of this command will be :
node_modules/@bower_components/bootstrap3-wysihtml5-bower/

@meyacine  Thanks for the help. I tried your way, and several other variations. Yarn would not work for me as I kept getting an I/O error for some reason. I eventually got all the scripts in using aliases in Webpack but then, for some reason, none of my other jQuery code worked. Even trying out various noConflict solutions nothing helped.
I finally decided to ditch this effort after two days and use Summernote. Installed and worked right from the get go.


[Read Me] Visually improve the ReadMe

The other day, I was referring to PowerToys and try to explain FancyZones to a colleague. He mentioned that the ReadMe file didn't contain any images to show off some of the features (so he had to look for the right link to bring him to the FZ readme).
And I think he's right: the ReadMe looks quite 'boring', and still uses the old app icon. With //Build/ around the corner, I can imagine traffic going up so having an appealing ReadMe might be good to have in place.
I think in terms of content we're fine, but maybe adding some images?

Update icon
Add some sort of 'hero image' at the top?
Add screenshot per feature.

The Microsoft MRTK repo is a nice example:

Maybe some of the designers around here could pitch in with a nice hero image :)?

Also the link to the devdocs is broken.
I miss a description for the steps between install visual studio an opening PowerToys.sln in the devdocs. This is a needed information for poeple start developing/building the first time.

We can change the first sentence of the Contributing section.
Original
This project welcomes contributions of all times.

Proposal
The PowerToys are developed from a large community of different Poeple.
Our goal is making them great for you.

We welcome contributions of all times.

I gave it a first shot here: https://github.com/niels9001/PowerToys/
It's mostly about adding some images here and there to make it visually more appealing :).

Thoughts @crutkas?

@niels9001
I love especially the header image.

I like it a lot. Maybe a quick link horizontal section for quick links to like download / release notes / privacy policy?

One q, why list vs grid of items?

One q, why list vs grid of items?

I was trying to do a grid, similiar to the MRTK sample. However, we have long feature descriptions. If we reduce those to e.g. a single sentence, that could work.

@crutkas For the shortcuts, would something like this work?

@niels9001
I known that this is a first test and I like it. ♥
I have seen that the text rendering on small device should be optimized. The text wrap around images and the buttons is incorrect.
Sory that I can't provide a screenshots.

@niels9001 is that possible to do those style buttons in markdown?

@niels9001 is that possible to do those style buttons in markdown?

Nope. Maybe with some HTML (but I think md support is pretty basic)?
These buttons are just images (screenshots of the Fluent/Fabric UI web toolkit) with a link. Not perfect, but should do the trick.

Let’s do links at first and then improve later. Needs to be accessible and work if someone wants to translate.

Could we use css styled html buttons?

Could we use css styled html buttons?

Doubtful. Rather do a first pass then investigate updates.

Yeah, look into that. Let's stick to default links for now.
@crutkas What links do we need at the top? And can I open a PR to get this in?

@crutkas
Do we need a link to notice.md in the Read Me? (Maybe next to the Release Notes link.)


Functions to inverse and negate private key, commit with 2 blinding factors

This PR adds 3 new convenience functions (and corresponding tests):

secp256k1_ec_privkey_tweak_inv - inverts the private key (x 🡒 x^(-1))
secp256k1_ec_privkey_tweak_neg - negates the private key (x 🡒 -x)
secp256k1_pedersen_blind_commit - pedersen commitment with 2 blinding factors instead of blinding factor and u64

And you also add secp256k1_pedersen_blind_commit function in this PR.
What's it for?

Yeah haven't updated the PR description yet, will do soon. It is to calculate commitment x*G + v*H where v is a secret key instead of a 64 bit integer

I added some tests for the secp256k1_pedersen_blind_commit function, the PR is ready for final review!

Could you provide some general direction as to what those could be used for?

@jaspervdm doing some secret dev work by the looks of it.

These 3 methods are useful in a secret sharing scheme. In particular, the inverse and negate functions are used in calculations for Lagrange basis polynomials.


[STORM-2510] update checkstyle configuration to lower violations

Update from checkstyle-6.11.2 to checkstyle-7.7.
Tweak google_checks.xml from checkstyle-7.7 to have the following changes that
are more inline with the way the storm code is written:

4 space indents instead of 2
line-length limit is 120 instead of 100

Exclude the generated thrift code in storm-client from being checked.
Update all maxAllowedViolations to be in-sync with the current number of
violations, through use of the update-all-pom-violations.bash script that
is attached to STORM-2510.

NOTE: this also fixes STORM-2507.

@vinodkc & @revans2 & @hmcc & @srishtyagrawal : please 👀 when you have a chance.  This is a follow-up to the discussions in PR #2083, and also an offline email thread about the changes in PR #2093.

Like I said before I am fine with whatever convention we have, so long as we have one.
+1

Oops we need a license in storm-buildtools/storm_checkstyle.xml or we need to white list it for rat.

@revans2 : ah, thanks for pointing out the need for license.  I copied how google-cloud-intellij is doing it:

https://github.com/GoogleCloudPlatform/google-cloud-intellij/blob/ba8c85810f65443d9ff411ad30448b377afa6012/google_checks.xml

I also increased the line limit to 140 and adjusted the maxAllowedViolations accordingly.

+1


Impossible to sync Autoplay's playing state using events

Bug is related to

[ ] embla-carousel (core package)
[ ] embla-carousel-react
[ ] embla-carousel-vue
[ ] embla-carousel-svelte
[x] embla-carousel-autoplay
[ ] embla-carousel-auto-height
[ ] embla-carousel-class-names
[ ] embla-carousel-docs (documentation)
[ ] embla-carousel-docs (generator)

Embla Carousel version
v8.0.0-rc19
Describe the bug
"autoplay:play" and "autoplay:stop" events are dispatched before the playing variable is changed, which makes it impossible to synchronise react state with autoplay state
CodeSandbox
Codesandbox
Steps to reproduce

Click the "Stop" button
Observe the carousel stop but state change not being detected

Expected behavior
I would expect those events to be dispatched after the corresponding variable is set. just like with other events.

@mykytabatrak why not do it like this?
Best,
David

@davidjerleke after looking at your approach and re-reading react docs on useEffect, I think you are right. For some reason I was fixated on using useSyncExternalStore for implementing this, it worked just fine for canScrollPrev and canScrollNext, and I just couldn't let it go.
Thank you so much, I believe you can close this issue now.


Roll engine to 5d2f24aa5cbdbe3c5d41b2b261873d26f4077c79

@gspencergoog fyi there's a little script in https://github.com/flutter/engine/wiki/Release-process that you can use when rolling the engine that will include the relevant commits, which makes future archaeology easier. :-)


Fixed button not visible completely #633

Signed-off-by: Ramreiso Kashung<EMAIL_ADDRESS>Description
This PR fixes #633
Notes for Reviewers
Signed commits

[ #633 ] Yes, I signed my commits.

] Yes, I signed my commits.

] Yes, I signed my commits.

@warunicorn19 @YashKamboj
I have updated the PR in response to changes requested.
Additionally, I have also fixed another bug where 'Participate' button was not visible completely at https://meshery.io/programs/gsoc/2020 .
Further requests or suggestions are most welcome!

I think this is ready to merge

I think this is ready to merge

I think this is ready to merge

I think this is ready to merge