Exec format Error not solved with wsl.conf in /etc/binfmt. Trying to delete .vscode-server turns entire WSL filesystem to read-only

Windows Version
Microsoft Windows [Version 10.0.19045.3324]
WSL Version
WSL version: <IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
22.04
Other Software
Visual Studio Code latest (8/14/23) and after downgrade to 1.80.2.
Repro Steps
Try to open VSCode from WSL2 with code command (e.g. code <some_folder>
Expected Behavior
It opens VSCode in WSL remote window.
Actual Behavior
It exits with "Exec format error".
Creating wsl.conf file in /etc/binfmt and adding ":WSLInterop:M::MZ::/init:PF" did not work as it did when I encountered the exact same error on another machine running Windows 11.
On this PC, the error persists. Attempting to delete .vscode-server results in the operation failing and turning my entire WSL filesystem into read-only.
Diagnostic Logs
No response

Same "Exec format error" on Fedora 38 and openSUSE Tumbleweed as well. No systemd enabled.
This applies to VS Code 1.8.1 and the latest VS Code-insiders.
Worked once between multiple Windows reboots. But only once among multiple reboots.

Exactly the same, creating wsl.conf file in /etc/binfmt and adding ":WSLInterop:M::MZ::/init:PF" did not work.
I tried to disable systemd but no luck.
When I start WSL the first time I can run Windows executables as usual, but after a minute (more or less) it stops working.

UPDATE I fixed it upgrading to WSL 1.3.15 (wsl --update --pre-release), unfortunately it's not a stable version, so I don't know if it's fit for everyone.
I use VPNs and wsl-vpnkit and just discovered that this tool breaks the possibility to run Windows executables if you run it in distro mode, I had to set it up as a standalone script https://github.com/sakai135/wsl-vpnkit#setup-as-a-standalone-script.
It has always worked both ways, but something weird happened in the last weeks. (Win updates, etc)

I have the same problems (starting about a week ago) on all my distributions:

ubuntu 20.04 LTS systemd disabled
Ubuntu 22.04 LTS systemd enabled
Arch Linux systemd disabled

They all worked flowlessly until a few days ago. I'm not sure what is the workaround of wsl.conf inside /etc/binfmt so I didn't try it.
Thanks

I was able to fix this on Windows 10 by also using fsck on /dev/sdc.

This looks to me like something you've installed has broken the binfmt registration. And unfortunately binfmt_misc is not namespaced. So if you have multiple distros open, and one of them nukes the registration here, then interop won't work anymore (Calling windows executables from within the guest). Do you still have /proc/sys/fs/binfmt_misc/WSLInterop?
cat /proc/sys/fs/binfmt_misc/WSLInterop
enabled
interpreter /init
flags: PF
offset 0
magic 4d5a

Until a fix will be released, if using systemd mask proc-sys-fs-binfmt_misc.mount and systemd-binfmt.service inside each WSL distro. This prevents /proc/sys/fs/binfmt_misc/WSLInterop registration from breaking.


Setup travis

Use CI to ensure integrity of our changes same as regular upstream merges.

Fixed by #12


You should not double backslashes in attribute values

Because of that it is impossible on user side to use gaphviz escape sequences.

Is this still valid today?

Please reopen if this proves to be valid today.


Document: add an example to remove console calls

Refs #1477

Codecov Report

Merging #1744 into master will not impact coverage.

@@          Coverage Diff           @@
##           master   #1744   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files           1       1
  Lines        4038    4038
  Branches      712     712
======================================
  Hits         4038    4038

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 5307e30...e965805. Read the comment docs.

👍


failed to upgrade kubefed from v0.7.0 to v0.8.1

What happened:
helm --namespace kube-federation-system upgrade -i kubefed kubefed-charts/kubefed --version=0.8.1 --create-namespace
Error: UPGRADE FAILED: error validating "": error validating data: [ValidationError(KubeFedConfig.spec): unknown field "statusController" in io.kubefed.core.v1beta1.KubeFedConfig.spec, ValidationError(KubeFedConfig.spec.syncController): unknown field "maxConcurrentReconciles" in io.kubefed.core.v1beta1.KubeFedConfig.spec.syncController]

What you expected to happen:
successfully upgraded kubefed
How to reproduce it (as minimally and precisely as possible):
install kubefed v0.7.0 on k8s v1.21, then upgrade it to v0.8.1
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version)
KubeFed version
Scope of installation (namespaced or cluster)
Others

/kind bug
I think this error is caused by the old crd. If we apply the new kubefedconfig crd manually, we can avoid this error. Any better solutions?

/cc @jimmidyson
/cc @hectorj2f

After I applied the new kubefedconfig crd, another error happened:
F0902 10:52:40.193071       1 controller-manager.go:358] Error: invalid KubeFedConfig "kube-federation-system/kubefed": [spec.syncController.maxConcurrentReconciles: Required value spec.statusController: Required value]

I checked the kubefedconfig and found that the spec.statusController filed was missing.
I think we should patch the kubefedconfig in the post-install-job just like what @hectorj2f did.

I had the same issue. For us, we just deployed the helm chart AGAIN and that time it worked. Apparently the CRD for some reason was not updated in the first run. Interestingly, our pipeline first applies the CRDs each time before doing an helm upgrade - seems like either kubernetes was not fast enough on updating the CRDs or we really just have a timing issue.
Still, I find it interesting that someone else has encountered the same problem as me.

I had the same issue. For us, we just deployed the helm chart AGAIN and that time it worked. Apparently the CRD for some reason was not updated in the first run. Interestingly, our pipeline first applies the CRDs each time before doing an helm upgrade - seems like either kubernetes was not fast enough on updating the CRDs or we really just have a timing issue.
Still, I find it interesting that someone else has encountered the same problem as me.

Interesting.

/remove-lifecycle stale


Providing egg size estimates to OSMOSE

The OSMOSE config file "osm_param-species.csv" provides an estimate of egg size (in cm) for each of the focal functional groups represented in the OSMOSE model.
Egg size is a parameter that can directly be obtained from FishBase/SeaLifeBase. I noticed that there also is a parameter called eggdiam in the "myersrecruitmentdatabase" FishBase table and I am not sure whether we should use this parameter or the LengthOffspringMin parameter of the "spawning" FishBase table.
@Dengaloo What do you think?

@Dengaloo I would also like the bridge to provide egg weight estimates to OSMOSE.
It will be easy to provide estimates of egg weight to OSMOSE from egg size estimates.
Indeed, if we assume that an egg has a diameter of D cm (and, therefore, a radius r = D/2 cm), then under the assumption that the average density of salt water at the surface is 1.025 kg/l = 1.025 g.cm-3, we get an egg mass of 1.0254/3.pi.r3 (in g).
For example, by default, OSMOSE assumes that the size of an egg is 0.1 cm, and, under this assumption, the mass of an egg is 0.00053669 g, considering a sphere with water density. Indeed, if we assume that an egg has a diameter of 0.1 cm (and, therefore, a radius r = 0.05 cm), then under the assumption that the average density of salt water at the surface is 1.025 kg/l = 1.025 g.cm-3, we get an egg mass of 1.0254/3.pi.r3 = 0.00053669 g.

@agruss2 : on eggdiam and LengthOffspringMin, I think eggdiam would more straightforward to use. However, not that the Myers'  database has not been updated since it was absorbed into FishBase. Though, I would think that the data is valid and viable.

@agruss2 : on egg weight, that empirical formula could be used if we had the egg diameter, so we can extract that from the Myers'  database. Then if Myers' database does not have a species or a functional group, we might be able to extract that from the EGGS table. In fact, I think we used Myers' database to fill in the egg size field in the EGGS table. So, yes, this is doable.

@Dengaloo It is good to know that we can use eggdiam estimates.
However, I need to know if we could also use LengthOffspringMin estimates in some case. Indeed, a few OSMOSE models consider focal functional groups that do not lay eggs (e.g., seals). In that case, we would need to look for the LengthOffspringMin parameter in the "spawning" FishBase table, rather than for the eggdiam parameter in the "myersrecruitmentdatabase" FishBase. To do this, some sort of algorithm will need to be implemented in the API.
Please let me know if there would be enough estimates for the LengthOffspringMin parameter in FishBase/SeaLifeBase to derive estimates of "pup size" or "hatchling size" for animals such as seals or sea turtles.

@Dengaloo Regarding my previous comment:
I think that this is not a major issue at all. Actually, only the OSMOSE model of the Strait of Georgia represent seals ...
The thing is that the life cycle and ecology of marine mammals, seabirds and sea turtles is so different from of those of fish that OSMOSE should be substantially recoded to simulate the entire life cycle of marine mammals, seabirds and sea turtles. For this reason, I think that we should emphasize in our project and in the paper that OSMOSE is a modeling platform designed to simulate the entire life cycle of fish and invertebrates currently; and have a bridge between FishBase/SeaLifeBase and OSMOSE that provides estimates for fish and invertebrate functional groups only.

@Dengaloo Looking at your last comment on this issue, I realize that there is an "EGGS" table in FishBase/SeaLifeBase that I am not aware of. Also, I realize that it would make more sense to derive an estimate of egg size for OSMOSE from the "egg size" field of the "EGGS" table.
@Dengaloo and @FIN-JBarile - I am attaching here the version of the "FishBase_DataDictionary" spreadsheet that is currently in our shared Dropbox folder.
FishBase_DataDictionary.xlsx
Could you please send me an updated version of the "FishBase_DataDictionary" spreadsheet providing all of the fields of the all the existing tables in FishBase/SeaLifeBase? Many thanks!

@agruss2 there are 140 fish and 29 nonfish species with LengthOffspringMin values. So, I suppose that is enough for a model?
OK for taking account only of fish and invertebrates. I am seeing a similar pattern for predator prey relationships, and will take not of this. If we are only considering fishes and invertebrates for one parameter, then we have to do the same thing for the other parameters.

@agruss2  Here's the updated data dictionary file. Additional sheets refer to EGGS and EGGDEV tables.
FishBase_DataDictionary.xlsx
I suggest you identify the tables of interest in FishBase before I generate the corresponding data dictionary. Please know that these are not readily available. May I refer you to the FishBase book online: http://www.fishbase.ca/manual/english/contents.htm? You will find the different suite of tables starting from Nomenclature down to Other tables.

@Dengaloo Yes, I agree. We should focus our efforts on fish and invertebrates from now on, since OSMOSE models usually care only about these marine organisms.

@FIN-JBarile Thanks a lot for the new Data Dictionary that will be very useful! Thanks also for the link to the to the FishBase book. OK, from now on, I will identify the tables of interest in FishBase before coming back to you for you to update the Data Dictionary.

@Dengaloo I am looking at the EGGS table right now, and there is no "egg size" field in this table. For the bridge, should we use the "Eggdiammod" parameter from the EGGS table to derive estimates of egg size and egg diameter for OSMOSE? Or should use another parameter from the EGGS table to do that?

@agruss2 : it is the eggdiammod field that measures egg size, we should use that.

@Dengaloo Perfect, thanks ! I just updated the "fishbase-mapping.csv" file.
I think that we can close the present issue now. Do you agree?

I am closing this issue today.

@agruss2 I am assuming that the calculation in https://github.com/jhpoelen/fb-osmose-bridge/issues/86#issuecomment-275958079 accidentally omitted a factor "4" and should read:
egg density * egg volume
~ salt water density * sphere with egg radius
~ salt water density * (4/3) * pi * r^3

@jhpoelen That's correct; thanks!


Release Google.Cloud.Speech.V1P1Beta1 version 2.0.0-beta08

Changes in this release:
Documentation improvements

Use full link in comment (commit dc5c9ea)

Created release for Google.Cloud.Speech.V1P1Beta1-2.0.0-beta08


[editor] Add an Ink editor

Approximate the drawn curve by a set of Bezier curves in using
js code from https://github.com/soswow/fit-curves.
The code has been slightly modified in order to make the linter
happy.

It looks like gulp ci-test failed, i.e. what GitHub action runs, and I'd guess that you need to extend this line to fix that:
https://github.com/mozilla/pdf.js/blob/2fbf14ace86d1d9840124b65a5af3ae2e2abc041/gulpfile.js#L1547

I added external/fit_curve/fit_curve.js  but it doesn't work because external is a the same level as display in lib-legacy.
So changing ../../../external into ../../external is ok but then gulp server isn't.

I added external/fit_curve/fit_curve.js but it doesn't work because external is a the same level as display in lib-legacy.
So changing ../../../external into ../../external is ok but then gulp server isn't.

I suppose that you can temporarily move it back its original position, in the src/display/editor/ folder, but that makes fixing the follow-up issue even more important (in my opinion).

/botio test

/botio unittest

/botio integrationtest


Add Support for React 17

Fixes https://github.com/fridays/next-routes/issues/346

@fridays what was the reason to simply close the pull request? This package cannot be yet installed together with React 17 :/


add ability to change sizes of screens

I would like the middle bar separating the .dot editor and the visualizer to allow me to change the width of both screens :)

It's a nice idea. Would you mind creating a PR for it? Thanks.


fix: the Criterion Transformer correctly handles the rightOperand

What this PR changes/adds
This PR changes the JsonObjectToCriterionTransformer so that the rightOperand is transformed to Object, rather than String.
Why it does that
Right-hand operands other than lists and strings caused a failure in the transformer.
Further notes

I added another end-to-end test to test API calls in conjunction with two connectors and Postgres.

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

Codecov Report
Patch coverage has no change and project coverage change: -53.44 :warning:

Comparison is base (e96e047) 71.91% compared to head (3bf9e7b) 18.48%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3320       +/-   ##
===========================================
- Coverage   71.91%   18.48%   -53.44%
===========================================
  Files         810      810
  Lines       16513    16512        -1
  Branches      964      964
===========================================
- Hits        11876     3052     -8824
- Misses       4236    13371     +9135
+ Partials      401       89      -312

Impacted Files
Coverage Δ

...ansformer/to/JsonObjectToCriterionTransformer.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...form/JsonObjectFromTransferProcessTransformer.java
0.00% <ø> (-100.00%)
:arrow_down:

... and 519 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Invalid prisma.post.findMany() invocation: connect ECONNREFUSED ::1:50413

Bug description
I upgrade my project from @prisma/cli(version 2.13.1) and @prisma/client(version 2.13.1) to prisma(version 2.24.1) and @prisma/client(version 2.24.1) respectively and when i ran npx prisma studio or using prisma client this error appeared to me:
PrismaClientKnownRequestError2 [PrismaClientKnownRequestError]:
Invalid `prisma.category.findMany()` invocation:

  connect ECONNREFUSED ::1:50281
    at cb (/path-to-app-folder/node_modules/@prisma/client/runtime/index.js:35223:17)
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  code: 'ECONNREFUSED',
  clientVersion: '2.24.1',
  meta: undefined
}

The same error appeared when i use the ** Quickstart**(with sqlite).
Note:
After i added this { __internal: { useUds: true }} as any to PrismaClient constructor it work but prisma studio still not working
How to reproduce

Expected behavior
No response
Prisma information
Prisma schema for Mysql
datasource db {
  provider = "mysql"
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = ["native"]
}

model Post {
  id                Int        @id @default(autoincrement())
  title             String     @db.VarChar(256)
  content           String?    @db.MediumText
  published         Boolean?   @default(true)
  publisher         String?    @db.VarChar(256)
  publisher_favicon String?    @db.VarChar(256)
  source_article_id String?    @unique
  link              String?    @db.VarChar(256)
  image_link        String?    @db.VarChar(256)
  language          Language?  @default(en)
  createdAt         DateTime   @default(now())
  updatedAt         DateTime   @default(now()) @updatedAt
  Category          Category[]
  categoryId        Int?
}

model Category {
  id        Int      @id @default(autoincrement())
  code      String   @unique
  createdAt DateTime @default(now())
  updatedAt DateTime @updatedAt
  name      String?
  posts     Post[]
}

enum Language {
  en
}

Prisma schema for sqlite
datasource db {
  provider = "sqlite"
  url      = env("DATABASE_URL")
}

generator client {
  provider = "prisma-client-js"
}

model Post {
  id        Int     @id @default(autoincrement())
  title     String
  content   String?
  published Boolean @default(false)
  author    User?   @relation(fields: [authorId], references: [id])
  authorId  Int?
}

model User {
  id    Int     @id @default(autoincrement())
  email String  @unique
  name  String?
  posts Post[]
}

Environment & setup

OS: Mac OS(Catalina)
Database: MySQL and SQLite
Node.js version: v12.18.3

Prisma Version
prisma               : 2.25.0
@prisma/client       : 2.25.0
Current platform     : darwin
Query Engine         : query-engine c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt c838e79f39885bc8e1611849b1eb28b5bb5bc922 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Default Engines Hash : c838e79f39885bc8e1611849b1eb28b5bb5bc922
Studio               : 0.402.0

I do have a similar issue with a postgresql db on linux. Everything works fine (introspect, migrate, ...), but the first query against the client causes it to panic/crash (I guess) and I get a ECONNREFUSED on varying ports.
Prisma Schema for postgresql:
`generator client {
provider = "prisma-client-js"
}
datasource db {
provider = "postgresql"
url      = env("DATABASE_URL")
}
model players {
id            Int       @id @default(autoincrement())
email         String    @unique @db.VarChar(255)
password_hash String    @db.VarChar(255)
display_name  String    @unique @db.VarChar(255)
}`
Environment & Setup

OS: Linux (opensuse tumbleweed)
Database: Postgresql 13
Node.js version: v16.4.1

Prisma version
prisma               : 2.27.0 @prisma/client       : 2.27.0 Current platform     : debian-openssl-1.1.x Query Engine         : query-engine cdba6ec525e0213cce26f8e4bb23cf556d1479bb (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli cdba6ec525e0213cce26f8e4bb23cf556d1479bb (at<EMAIL_ADDRESS>Introspection Engine : introspection-core cdba6ec525e0213cce26f8e4bb23cf556d1479bb (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt cdba6ec525e0213cce26f8e4bb23cf556d1479bb (at<EMAIL_ADDRESS>Default Engines Hash : cdba6ec525e0213cce26f8e4bb23cf556d1479bb Studio               : 0.410.0

@siemah
Can you please provide a reproduction here?
You can also try using the Node API preview which will be the default engine communication method in the future and studio will work fine with it: https://www.prisma.io/docs/concepts/components/prisma-engines/query-engine#enable-the-node-api-n-api-preview

Hey @pantharshit00
Sorry about the delay, there is a repo follow these instructions:

Install dependencies using yarn or npm i
Run npx prisma studio

This should be automatically resolved now in Prisma v3 since we don't use a sidecar binary process anymore.


Add github action for creating new score versions in s3

Before @esfoobar-usds left, he added score versioning in #1453. However, we need one more github action to actually create where these versions go. He described it as follows:
We need to add "generate version" which will be like Generate Score but it's modeled after deploy_be_to_staging and takes two variables:

J40_VERSION_DATESTRING
J40_VERSION_LABEL_STRING

The difference from deploy_be_staging, it'll put the data in data/score/VERSION instead of storing the score by hash.

In talking to @vim-usds, ideally this would tie into actually creating releases on github, so I should look into that but it's not a must have.


titles at the top

@atomgroomdesign this is minor, but could you put the titles for the images below at the top? It is confusing with them at the bottom, I had to study it for a second to understand which was which.

Labels for images are usually at the bottom.
Sent from my iPhone

On Oct 13, 2015, at 2:06 AM, Brian Moschel<EMAIL_ADDRESS>wrote:
@atomgroomdesign this is minor, but could you put the titles for the images below at the top? It is confusing with them at the bottom, I had to study it for a second to understand which was which.
—
Reply to this email directly or view it on GitHub.

@moschel I can see how you think it's confusing with the label on the bottom.
Although, @justinbmeyer is correct, labels or captions are usually at the bottom.  Moving them to the top will only shift the same exact issue in the other direction for users that are familiar with this pattern.
I think the real issue here is spacing and separation between these two images. Do you think that will help make this less confusing for you?

Nah forget it, its minor. If you guys think its not confusing then its fine.


Email template preview with guest checkout potentially misleading

I think this is a bug, let me know if it's intended. Adding {{ user.get_full_name }} in an email template using the email dashboard will result in the currently logged in user's name going there when used with a guest checkout order. This could be misleading to a developer trying to edit an email template. Not sure how "user" is getting into the context like that. I could look into this more if desired.

user instance automatically added to the template context by auth context processor -
https://docs.djangoproject.com/en/2.0/ref/templates/api/#django-contrib-auth-context-processors-auth.


Various quality edits

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

Project coverage is 4.17%. Comparing base (ab56e96) to head (b14bacf).

Files
Patch %
Lines

...StatsCore/Distributions/MannWhitneyDistribution.cs
0.00%
11 Missing :warning:

...ailDiff/SailDiffReadInBeforeAndAfterDataHandler.cs
0.00%
5 Missing :warning:

source/Sailfish/Utils/DisplayNameHelper.cs
0.00%
3 Missing :warning:

...ysis/SailDiff/Statistics/StatsCore/Ops/StatsOps.cs
0.00%
2 Missing :warning:

...fish/Contracts.Public/Models/TestCaseIdComparer.cs
0.00%
1 Missing :warning:

source/Sailfish/Execution/SailfishExecutor.cs
0.00%
1 Missing :warning:

source/Sailfish/Execution/TestInstanceContainer.cs
0.00%
1 Missing :warning:

.../Extensions/Methods/TableParserExtensionMethods.cs
0.00%
1 Missing :warning:

...ailfish/Extensions/Methods/TypeExtensionMethods.cs
0.00%
1 Missing :warning:

...sh/Registration/SailfishTypeRegistrationUtility.cs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #164       +/-   ##
==========================================
- Coverage   56.17%   4.17%   -52.01%
==========================================
  Files         316     241       -75
  Lines        9833    8385     -1448
  Branches     1277    1158      -119
==========================================
- Hits         5524     350     -5174
- Misses       3983    8021     +4038
+ Partials      326      14      -312

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


Remembering authenticated user

I refreshed a page on my home machine where I was logged yesterday and I was logged out immediately. Could we extend session life to more than one day? With maybe some option to select?

It's still logging me out every day. :<

I genereted passwords, so it should work fine now.


Error: Failed to connect to root. The username and / or password are incorrect.

I'm using
Ubuntu 16.04
OpenBazaar Server v0.12.3
OpenBazar Desktop 2.2.4(as per package.json)
I've downloaded OpenBazaar server using this link.
I've configured username and password using the command which is setapicreds
When I try to connect the desktop with server using valid credentials.
I'm getting following error.
In Desktop:

Error: Failed to connect to root. The username and / or password are incorrect.

In Server:

2018/09/05 11:54:26 http: response.WriteHeader on hijacked connection

I was able to reproduce this error a few minutes ago on 0.12.4.

Logs from a local instance operated by @jjeffryes with the same issue:
C:\GoWork\src\github.com\OpenBazaar\openbazaar-go>go run openbazaard.go start --bitcoincash -v -c asddfafsdfdsfadsfdf
________                      __________
\_____  \ ______   ____   ____\______   \_____  _____________  _____ _______
 /   |   \\____ \_/ __ \ /    \|    |  _/\__  \ \___   /\__  \ \__  \\_  __ \
/    |    \  |_> >  ___/|   |  \    |   \ / __ \_/    /  / __ \_/ __ \|  | \/
\_______  /   __/ \___  >___|  /______  /(____  /_____ \(____  (____  /__|
        \/|__|        \/     \/       \/      \/      \/     \/     \/

OpenBazaar Server v0.12.4
10:29:49.220 [INFO] [cmd/Execute] Peer ID:  QmRzqLhC1KYoZV2u5k5LdpSwcDZM9c9qNMC2sKB8Csp7Qi
10:29:49.224 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.225 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.225 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.225 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.226 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.226 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.227 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.227 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.227 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.228 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.228 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.228 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.229 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/4001
10:29:49.229 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip4/<IP_ADDRESS>/tcp/9005/ws
10:29:49.230 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip6/::1/tcp/4001
10:29:49.230 [INFO] [cmd/printSwarmAddrs] Swarm listening on /ip6/::1/tcp/9005/ws
10:29:49.252 [INFO] [cmd/newHTTPGateway] Gateway/API server listening on /ip4/<IP_ADDRESS>/tcp/4002
10:30:09.223 [INFO] [service/New] OpenBazaar service running at /openbazaar/app/1.0.0
10:30:09.223 [INFO] [cmd/func4] Starting bitcoin cash spv wallet
10:30:09.225 [DEBUG] [recordAgingNotifier/PerformTask] notifications created/records updated: purchaseTimeout: 0/0, purchaseExpire: 0/0, dispute: 0/0, sales: 0/0
10:30:09.234 [DEBUG] [service/SendRequest] Sending STORE request to QmY8puEnVx66uEet64gAf4VZRo7oUyMCwG6KdB9KM92EGQ
10:30:09.234 [DEBUG] [service/SendRequest] Sending STORE request to QmPPg2qeF3n2KvTRXRZLaTwHCw8JxzF4uZK93RfMoDvf2o
10:30:09.596 [DEBUG] [service/SendRequest] Received response from QmY8puEnVx66uEet64gAf4VZRo7oUyMCwG6KdB9KM92EGQ
10:30:09.596 [DEBUG] [service/SendRequest] Received response from QmPPg2qeF3n2KvTRXRZLaTwHCw8JxzF4uZK93RfMoDvf2o
10:30:09.596 [DEBUG] [core/SendStore] Peer QmY8puEnVx66uEet64gAf4VZRo7oUyMCwG6KdB9KM92EGQ requested no blocks
10:30:09.597 [DEBUG] [core/SendStore] Peer QmPPg2qeF3n2KvTRXRZLaTwHCw8JxzF4uZK93RfMoDvf2o requested no blocks
10:30:09.598 [DEBUG] [service/DisconnectFromPeer] Disconnecting from QmY8puEnVx66uEet64gAf4VZRo7oUyMCwG6KdB9KM92EGQ
10:30:09.598 [DEBUG] [service/DisconnectFromPeer] Disconnecting from QmPPg2qeF3n2KvTRXRZLaTwHCw8JxzF4uZK93RfMoDvf2o
10:31:50.043 [INFO] [ipfs/Publish] Published zdj7WXAdxi8RdsVLJs1G4RxbjWaDAczearMBa3932QdeKvdvQ to IPNS
10:40:09.225 [DEBUG] [recordAgingNotifier/PerformTask] notifications created/records updated: sales: 0/0, purchaseTimeout: 0/0, purchaseExpire: 0/0, dispute: 0/0
10:50:09.225 [DEBUG] [recordAgingNotifier/PerformTask] notifications created/records updated: purchaseTimeout: 0/0, purchaseExpire: 0/0, dispute: 0/0, sales: 0/0
2018/09/05 10:50:54 http: response.WriteHeader on hijacked connection
2018/09/05 10:50:54 http: response.Write on hijacked connection
2018/09/05 10:50:57 http: response.WriteHeader on hijacked connection
2018/09/05 10:50:57 http: response.Write on hijacked connection

Thank you for your response. I've corrected it now. it is working perfectly.


Arg Parser + Updated README.md

I have added the argparser, which does not change the logic of the code. It only adds better user compatibility. Users can run: python coinmarketcap-usd-history.py -h, to see the list of available arguments that they can use.

Great!


Allow the user to configure a database-only appliance

Ask whether the user would also want this server to function as an "application server" when configuring an internal database
This includes not starting the evmserverd service as well as not seeding the database with information about a new evm server instance (joining the region).
https://www.pivotaltracker.com/story/show/126134069

@miq-bot add_label core, enhancement

@jvlcek @jrafanie please review.

I don't understand the bit about not seeding...does that mean we can't manage the appliance?

@Fryguy This is just going to be an external datbase server, no evmserverd will be running so I would think we wouldn't want it to be seeded.
This is just to make it easy to get our postgres config and packages on an external database VM to be used in an HA cluster.

LGTM 👍  Nice Job @carbonin

Looks good, maybe we should update the summary screen since the "Database configured", "local database" , "database/region", values don't really make sense in light of having "appliances with an internal database", "appliances with an external database", "database only appliances (no miqserver thing running)"

@carbonin 👍 LGTM

Going to make some changes to the summary screen to better represent the situations brought up by @jrafanie

Okay, I made the change and will do an appliance build and get some screenshots to make this easier to see.

@jrafanie I know this has become quite a bit larger than my original intent. Do you think it would be better to pull the second commit into a separate PR?

@jrafanie I know this has become quite a bit larger than my original intent. Do you think it would be better to pull the second commit into a separate PR?

@carbonin I agree but I think it's reasonable reviewing it commit by commit.  It's up to you if you want to separate it.

So I'll add some screenshots to see what the console refactor looks like.

This is how it used to look before these changes:

This is how it will look now with a local configuration:

And this is how it will look when configured remotely:

LGTM.  There's some followup cleanup but this is good.  cc @gtanzillo you'll probably want to take a look at the screenshots.  Feel free to merge or 👍  and I will.

@gtanzillo what do you think?

👍 Good to go @carbonin

@miq-bot add_label darga/no


Admin-Token for metrics isn't set on docker image

The latest metrics introduces "-Dhawkular.metrics.admin-token". As its not on the base docker image, any calls to /tenants fail.
https://github.com/hawkular/hawkular-metrics/blob/b5e1a337e206c961c3b9e8b47d40e28d081c2c59/api/metrics-api-jaxrs/src/main/java/org/hawkular/metrics/api/jaxrs/filter/AdminFilter.java#L67
https://github.com/hawkular/hawkular-services/blob/master/docker-dist/src/main/resources/startcmd.sh

What is the use case / need for this?

As per the comment on the PR, i dont think its needed anymore. I didn't realise you could use the ADMIN_TOKEN env var before raising the PR.
Happy to close if you dont think its needed as well.


mdx-embed return empty components

Describe the bug
I test mdx-embed with nodejs without browser, and It return an empty component.
To Reproduce
import React from 'react';
import { renderToString } from 'react-dom/server';
import { Tweet } from 'mdx-embed';

const result = renderToString(<Tweet tweetLink="PaulieScanlon/status/1232982448310497286" />);

console.log(result);

Expected behaviour
Have an iframe with a Tweet embedded
Actual behaviour
It return an empty component :

Additional context
node version : v12.21.0

Hey @johackim thanks for opening an issue. I haven't investigated @mdx-js/runtime could you share a little more info on how it might differ from @mdx-js/mdx

Hello @PaulieScanlon ! Thanks for your response :)
I use @mdx-js/runtime directly with nodejs. I build a cli tool to convert markdowns with mdx components to html.

Hello @PaulieScanlon, here a codesandbox to demonstrate the problem : https://codesandbox.io/s/mdx-jsruntime-with-mdx-embed-d73lg

@johackim could it be related to this?
https://github.com/mdx-js/mdx/issues/558
The other thing I've been thinking is that all MDX Embed components reply on the Intersection Observer. E.g when a component enters the viewports the child embed script is loaded. Intersection observer is a browser API so it might be that the components are being passed correctly but they won't load because the intersection observer never enters the viewport. Does that make sense?

@PaulieScanlon,
Yes you're right, the isChildVisible is set to false by default and it is never set to true when used with nodejs (no browser).

Thanks for the update. One thing we could do... and this will probably be part of the ongoing global config work is to allow some kind of config option to turn the intersection observer off. By default it should use intersection observers because this is what keeps MDX Embed "polite" but I guess there's no harm in turning this functionality off if required. I might be able to slip in a quick update that exposes an optional prop on a component that will do a similar thing.
Is there any single component you urgently need? Perhaps I could make a change to just the Tweet component and release an -alpha version for you to test with?

No, there is no urgent for a single component, a global config option to turn the observer off seems to be a good idea, I will wait for this update :)
Thanks @PaulieScanlon

I just ran into this problem one as well. I am using NextJS to update my blogs and migrate off of WordPress. Sometimes the embedded Tweets show up, others times they don't. It seems like the General Observer does not consistently fire. If I refresh the page, the Tweet always loads. However, if I navigate to a page a second time, the General Observer never loads. Though, I can trick it into loading again by scrolling down and then scrolling up.
Barring fixing the logic so that the General Observer always fires properly, I would definitely advocate for a "don't lazy load this embed option".
// simple example
<Tweet tweetLink="bernhardsson/status/1605357323643506695" lazyLoad={false} />

// or taking inspiration from the


improved data plotting

Plotting functionality is mainly used by me to create plots for the paper. I am undecided on whether we will make plotting a core part of the this package yet.

[x] detect if we are doing delays only (ie no risk)
[x] detect if we are doing risky choice only (ie no delays)
[x] scan for any duplicate designs: size of dot = number of times that design was run, colour = proportion of times participant chose B.
[ ] sort colour of data points in legend
[ ] implement the same plotting style for risky choice situations

Now we get better plotting of data. Where there are repeated designs then this is shown by the data point becoming larger. The colour of the datapoint reflects the proportion of times the participant chose B. This is an exaggerated example with a restricted design space to demonstrate the point.


[ci] Automate building artifacts for master branch

This PR attempts to automated the publication as github releases of executables for the latest commit on the master branch.
The executables should be published on the latest release tag: https://github.com/plaans/aries/releases/tag/latest
This is a development build so should be flagged as a pre-release.
TODO:

[x] Add cross compilation for major platform (currently only linux-x86_64 is supported)
[x] Only enable the pre-release job to run once all tests have passed (currently running immediately for facilitate testing)
[x] Before merging, change the test in the pre-release job to only execute on the master (currently on deploy for testing purposes)

@arbimo I have added an up-aries compilation job for the other platforms. Currently, available platforms are,

Linux amd64
Linux aarch64
Windows amd64
Windows aarch64
macOS x86_64
macOS aarch64

The Linux-aarch64 build fails with cross-compilation due to linker problems, so I have added a docker-based build for this platform. Currently, the docker-based build has consumed more time for the previous job but it was fairly quick on the up-aries repository.

@arbimo I have added tests for the cross-compilations.

@franklinselva Thanks for the work on cross compilation ! I just polished this and will merge it now.


Command "kanbn not found"

Hi.
When I'm trying to open a board by clicking on the icons at the botton of VSC, I'm getting a message, "kanbn not found".
That message come's even if the board is already open. Any idea of what is causing that ?
Juste one more thing : I've use the previous version of that extension, the one that was forked for the present one. It's perhaps a explanation ?
Many thanks

Hi
Apologizes for my english
In fact I was just asking why I got this error :)

Hi @dbourrion, I haven't used the bottom panel at all since I've started working on the project, so I'm not too surprised that it's broken. :) The fix should be fairly straightforward, I forgot to rename kanbn.board to kanbn.openBoard in one place. Thanks for reporting, I'll include a fix in the next release!
Out of curiosity, what would you expect the bottom panel to do? It used to open the board for the current workspace in the old extension. Now that there can be multiple boards per workspace the equivalent command will open a list of boards to choose from (same as the Open Board command from the command palette). Do you maybe have a better suggestion for what it should do?

Hi
Oh, cool to learn that a fix is on his way.
Yje option of opening a list of board to choose one looks perfect for me :)

The fix is now live in 0.13.0.


Spawning a thread for each object

I'm wondering if Planck.js is designed for spawning a thread (WebWorker) for each object in the simulations or only for collisions/joints?

To my knowledge planck.js isn't multithreaded. I guess most physic engines are singlethreaded as I would imagine splitting up a world into mostly independent parts is pretty pretty complicated.


add a conditional dataclasses dependency

Codecov Report

Merging #31 (3d515dd) into master (c736c67) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #31   +/-   ##
=======================================
  Coverage   99.26%   99.26%
=======================================
  Files           5        5
  Lines         136      136
=======================================
  Hits          135      135
  Misses          1        1

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 c736c67...3d515dd. Read the comment docs.


Version 184.0 not buildable

54dbe2db129e9ba063916f4d10763b06cd30a660 is missing.
gpg: directory '/home/runner/.gnupg' created
gpg: keybox '/home/runner/.gnupg/pubring.kbx' created
gpg: /home/runner/.gnupg/trustdb.gpg: trustdb created
gpg: error reading key: No secret key
No secret key for<EMAIL_ADDRESS>exists, signing disabled
/bin/sh: 1: cannot create .build/sign.pub: Directory nonexistent
make: *** [sign.pub] Error 2
Makefile:10: recipe for target 'sign.pub' failed

I have cherry picked the fix from the main branch to rel-184: https://github.com/gardenlinux/gardenlinux/commit/3cfe8daefb207c50cca26b3fe0d0da9db6aa0e07

/close

Build fails but for a different reason now. I will investigate.

/assign @gehoern
I assume this is related to moving to a proper repository location!? What do we need to cherry-pick ?

/assign @ccwienk
problem is broken VERSION file in 184 ... contents should be 184 NOT 1.184

This is caused by a wrong version in the VERSION file. It is 1.184 but should be 184.0

Build still fails (make openstack) while installing dracut modules:
dracut-install: ERROR: installing 'growpart'
dracut: FAILED: /usr/lib/dracut/dracut-install -D /var/tmp/dracut.Xk3jOb/initramfs -a grep sfdisk growpart udevadm awk mawk sed rm readlink

Build still fails (make openstack) while installing dracut modules:
dracut-install: ERROR: installing 'growpart'
dracut: FAILED: /usr/lib/dracut/dracut-install -D /var/tmp/dracut.Xk3jOb/initramfs -a grep sfdisk growpart udevadm awk mawk sed rm readlink

/assign @5kt

/assign @5kt

This has been fixed by : 71883cad58d4892306e4239a1f483a27f9f916b9 and 5e5cd88280a951c5d4cb4a7e5e48d4f0f619ac0d
This was caused by the missing cloud-guest-utils package that was getting removed when libdb is uninstalled - dependency issues.

This has been fixed by : 71883cad58d4892306e4239a1f483a27f9f916b9 and 5e5cd88280a951c5d4cb4a7e5e48d4f0f619ac0d
This was caused by the missing cloud-guest-utils package that was getting removed when libdb is uninstalled - dependency issues.

Thanks @5kt. I will close this issue.
/close

Thanks @5kt. I will close this issue.
/close


Partial unique indexes for natural keys

resolve #110
resolve #111
resolve #114
The PR introduces:

PostgreSQL partial indexes for entities' natural keys defined as new Flyway migration scripts
create an utility TestUtils.testEntityUniqueness method used in every entity involved with this campaign to check uniqueness that works like this

create a first entity
try to add a second entity with the same natural key (must fail)
remove the first entity
check the second entity can now be created (check the partial index works with soft delete)
remove the second entity to not alter further tests' execution

Tag entity has a further test added to check the uniqueness is not affecting tags associated with different tag type
ArcUndeclaredThrowableExceptionMapper for managing as requested the error message in case of duplicate key
update FlywayMigrationTest consistently

@mrizzi - Thanks it all looks good to me and it was great for me to understand the solution to this.
I can see the src/test/java/io/tackle/controls/util/TestUtils.java does the heavy lifting and tries, inserting, inserting again (duplicate), deleting then inserting.
I see the database constraint is enforcing the uniqueness and then respond to what is returned from the DB.
The Tag index uses 2 columns so Tag is only unique within Tag Type. Which is what we want.
And the JobFunctionTest.java is a brand new class (were we missing a test for this entity before?) whereas for the other entities we are just adding the testUniqueName() method.
The 6 new sql files that add the unique indices will increase the number of files in
tackle-controls/src/main/resources/db/migration/ to 21, so the changes toFlywayMigrationTest.java reconcile.
As soon as I can test it in our test environment I will merge it.
Super and thanks again!

@mrizzi - Thanks it all looks good to me and it was great for me to understand the solution to this.
I can see the src/test/java/io/tackle/controls/util/TestUtils.java does the heavy lifting and tries, inserting, inserting again (duplicate), deleting then inserting.

yep, the test's approach is always the same so it has been refactored in an util method. What changes is the entity this method tests.

I see the database constraint is enforcing the uniqueness and then respond to what is returned from the DB.
The Tag index uses 2 columns so Tag is only unique within Tag Type. Which is what we want.

:+1:

And the JobFunctionTest.java is a brand new class (were we missing a test for this entity before?) whereas for the other entities we are just adding the testUniqueName() method.

Well, "standard" tests for list, create and delete actions are available in a parameterized set of tests in ServicesParameterizedTest.
Even more, I think testEntityUniqueness should become part of this set of parameterized tests so I'm going to move it there in a further commit.

The 6 new sql files that add the unique indices will increase the number of files in
tackle-controls/src/main/resources/db/migration/ to 21, so the changes toFlywayMigrationTest.java reconcile.

:+1:

As soon as I can test it in our test environment I will merge it.

Hold on, I'm going to refactor the testEntityUniqueness: this won't affect your tests because the changes will only affect the code for tests.

Tested successfully against all of the Controls entities. Excellent!


Avoid sending email in the unit test

Updated the code to prevent sending email when running the apex unit test.
Critical Changes
Changes
Suppress emails for Apex unit tests.
Issues Closed
Fixes #490
New Metadata
Deleted Metadata

Hi @lianeyee , this pull request is about update the code to not sending email in the Apex unit test cases. I don't think user need to do anything about it. So, I just left Critical Changes and Changes part empty. Can you please do a review for it?

Thanks for the ping, @weizsforg. Since I see this item in Changes now, did you decide to mention it, after all? That's fine (I edited it slightly). It's also fine to omit, as long as the Issue Description provides the same info. Your call whether to keep in Changes or leave out.

Thanks for your help @lianeyee . I think we can keep in Changes.

@davidjray @chrismcmahon
This is the pr we discussed in the meeting. Sorry about the confusion I gave. I was thinking about the other work item.
Feel free to let me or @weizsforg know if any questions or concerns.

Hi @weizsforg,
Thank you for jumping on this pull request.
Basically, you need to switch your language to Spanish under your user settings and make sure all the translations are correctly implemented.
The translation file is under the work item.
Feel free to let me know if any questions!


Refine iterator

Small iterator refinition to be a bit more aligned to standard one.

Hi, @bvbfan,
Thanks for taking the time to add your contribution!
I see you've introduced several changes that are not strictly related one to another. I would like to slowly introduce each change because I want to test and understand it. This is more of a learning project. You've addressed some issues that I postponed for a while so I could get the time to analyze them.
Is it possible to incrementally introduce each change on a different pull request? And ideally, there should be a measured or documented motivation for each change. If you don't have time, of course, it's no problem, and I will gradually take each of the aspects you addressed and introduce them to the project (mentioning your contribution).

Feel free to split it. Just std::decay is shorten than existing one.

Great, I will find the time for each one and introduce it.
Thanks again!


Request: Battlemedicine

Adding the person making the check to the immunity effect's name would be useful, since it only applies to the character who used battle medicine, rather than everyone who uses it (and in my experience it is a feat that is often taken by 2+ characters)

There is Ward Medic feat. But it's only for Treat Wounds action, you can't combine with Battle Medicine. If you know feat name it would be great, I not found

Current functionality
https://github.com/reyzor1991/foundry-vtt-pf2e-action-support/assets/6688366/9e672214-8cd7-435e-8eee-97e7d3e60f47
As I understood, you want that when any PC use Battle Medicine target get effect like "Effect: Battle Medicine Immunity (Seoni (Level 1))" and PC get "Battle Medicine was used (Merisiel (Level 1))", is it correct?

I think that was his request, battle medecine's immunity is per PC, treat wounds immunity is global and its duration is depending if the healer had continual recovery or not

https://github.com/reyzor1991/foundry-vtt-pf2e-action-support/assets/6688366/8d9ae63b-552c-45eb-aca1-61408453e85f

I don't understand the clip :
It's not in combat so it's treat wounds, not battle medicine here, so the treat wounds immunity effect they give should be global
Medic dedication gives the chance to use "battle medicine" on someone already immune to the healer's battle medicine (immunity which last 1 day). One battle medicine which bypass the immunity by day, or by hour when master.
Saying that there is still the option to use "battle medicine" out of combat, but that's not what the clip show I think
The pf2e workbench macro for treat wounds/battle medicine already handles immunities effect per PC/ some feats integration (idk if there are not implemented) and offer the choice between treat wounds and battle medicine. Idk if it's possible but running the macro when using the action should be a nice way to automate it

As I understood, you want that when any PC use Battle Medicine target get effect like "Effect: Battle Medicine Immunity (Seoni (Level 1))" and PC get "Battle Medicine was used (Merisiel (Level 1))", is it correct?

Sorry no, you can close the issue. I was sure last time I tried it that when using battle medicine it just put an effect saying "Effect: Battle Medicine Immunity" without the name, but trying it out now it does indeed put the name in brackets and that is fine.
Sorry for the confusion.


"'Future' object is not subscriptable" after persisting an object with dask.distributed

There is a piece of code that runs with the single-machine scheduler but not when dask.distributed is used. The code below adds many computed columns to a dataframe. To prevent RecursionError (see #4360), I call persist once in a while
import dask.dataframe as dd
import pandas as pd
import random
import itertools
import string
import dask.dataframe as dd
random.seed(10)

bigrams = [''.join(b) for b in itertools.combinations_with_replacement(string.ascii_letters, 2)]
data = dd.from_pandas(
    pd.DataFrame(
        {
            'word': [''.join(random.choices(string.ascii_letters,k=4)) for _ in range(100)]
        }
    ),
    npartitions=8
    )

count = 0
K = 500
for bigram in tqdm.tqdm_notebook(bigrams):
    col_name = f'has_bigram_{bigram}'
    data[col_name] = data['word'].apply(lambda w: bigram in w, meta=(col_name, bool))
    count += 1
    if count > K:
        data = data.persist()
        count = 0
if count:
    data.persist()

data.compute()

Everything works OK. However, if I add the following lines to the code above, I get TypeError: 'Future' object is not subscriptable. :
client = Client()

This seems to have been resolved by #2505:
import dask
import distributed
import dask.dataframe as dd
import pandas as pd
import random
import itertools
import string
import dask.dataframe as dd

print("Dask version: %s" % dask.__version__)
print("Distributed version: %s" % distributed.__version__)

random.seed(10)

bigrams = [''.join(b) for b in itertools.combinations_with_replacement(string.ascii_letters, 2)]
data = dd.from_pandas(
    pd.DataFrame(
        {
            'word': [''.join(random.choices(string.ascii_letters,k=4)) for _ in range(100)]
        }
    ),
    npartitions=8
)

count = 0
K = 500
for bigram in bigrams:
    col_name = f'has_bigram_{bigram}'
    data[col_name] = data['word'].apply(lambda w: bigram in w, meta=(col_name, bool))
    count += 1
    if count > K:
        data = data.persist()
        count = 0
if count:
    data.persist()

data.compute()
print("OK!")

$ python test.py
Dask version: 1.1.5+7.g3205dbd
Distributed version: 1.26.1+1.g50dd21f
OK!

Things still aren't as optimal as they could be (higher level fusion as described in #4360 would help), but this specific issue is resolved. Closing.


You can save a quote without entering a field.

@jSylvestre I'm not sure this is a problem -- you can have a 0 acre project so i would assume this would mean no fields as well.

@srkirkland Changing accounts with no fields causes problems:
Just wrap this in a if I think:
https://github.com/ucdavis/Harvest/blob/main/Harvest.Web/Controllers/Api/RequestController.cs#L86

ok, i'll think i'll be better to just initialize to an empty array in the ctor but i'll try both out


Publish code coverage

Please publish your code coverage results to Code Climate

✅ Code coverage published to Code Climate.


tBTC minting step 4/5 error: Transaction was not mined within 50 blocks

Description:
As I saw in the bot, that someone burned their 1 tBTC, I immediately put my URL into browser to continue from step 4. There was no Alert in metamask, I have put quite high gas to proceed - 700. After few minutes I see that transaction still pending. Then I speed it up (1800 gwei) and get this error in 1-2 minutes
Expected behavior:
Receive 0.5 tBTC and see congratulations page
Environment details:
OS: Windows 10 Home, version 10.0.18362, 64 bit OS
Web Browser: Google Chrome, Version 83.0.4103.116 (Official Build) (64-bit)
Wallet: Metamask (Chrome extension), 7.7.9
Additional context:
URL
https://dapp.test.tbtc.network/deposit/0xD72B7c9Cae23dfB7E207fA9FC7C619C9EC893b21/prove
Etherscan:
https://ropsten.etherscan.io/tx/0x592b62c1ecb0c3b10c70639d9baa5386f096e06dda0035e365ae805eff9d1306
Screenshots

tbtc-dapp-console-log-1596459254522.txt
Here's error log. By the way, why did you choose .json format for error log download? It's not very convenient, as github doesn't support that and you would have to change that to some different format, txt for example

By the way, why did you choose .json format for error log download?

We hadn't realized at the time that GitHub was mad about it. It'll be fixed in the next day or so as we roll out a new release.

Whoops, sorry, thought I'd already responded to the rest of the issue :)
Regarding the issue in the description, the issue the transaction ultimately suffered (not at current or previous difficulty) is another instance of the issue I described in this comment, and is an issue with Bitcoin testnet (more or less), so we can't do much about it either right now. Separately, the problem with Metamask not realizing when the transaction went through is out of our hands---and we rely on Metamask to provide the appropriate information to the dApp.
That said, we could handle the “transaction wasn't mined” issue more clearly, though, at least by suggesting a course of action, and I'm going to leave this issue open to capture that corner case so we can make sure to keep it in mind.

@Shadowfiend okay, so if that this issue is not in your scope of influence and happens due to BTC testnet, perhaps we shall close it?

As I noted above, though the issue isn't under our control, how we surface it to the user is, so I'd like to keep it open so we can track that piece of it :)


Bug #658: Missing tooltips on cos and tan block

Video showing that the Bug is fixed https://www.dropbox.com/s/gnjuzr6gt0bl93r/Bug%23658.mov?dl=0

LGTM
thanks!

MERGED!


Remove normalization of paths for macOS

:wave:
Gatsby started using this project in a recent release and unfortunately it lead to a new bug (https://github.com/gatsbyjs/gatsby/pull/12005#issuecomment-468249147) I've dug a bit deeper and found that it is caused by this line:
https://github.com/barsh/true-case-path/blob/a1623002cf442ba7b8bb91c6a981cc46c887b85b/index.js#L16
HFS+ is no longer the default file system for newer Macs and with High Sierra they even upgraded HFS+ systems to APFS if the computer is using a SSD.
APFS seems to not store things with NFD anymore, but if we try to normalize the path, glob will return an empty array for the normalized path. If we remove that line, everything works as expected.
Unfortunately I don't have any old mac with HFS anymore, so I can't test it out right now, but what do you think the best way going forward would be?
There's also #8 lingering around, which I have yet to try.
Thanks for reading this and have a great day :)

Fixed in fork Profiscience/true-case-path, published in 2.0.0


Remove Results template

The Results template in the ModuleSettings is not needed anymore, as results will be displayed using ChartJS.

Setting removed


Workspace.getActiveEditor is deprecated.

Hey Sean, any interest in trying to clean up atom-tidal so that we don't get deprecated calls?
Just a thought!
-Robbie Jacobs
Call ::getActiveTextEditor instead
Workspace.getActiveEditor (/Applications/Atom.app/Contents/Resources/app.asar/src/workspace.js:1022:12)
REPL.getEditor (/Users/robjac/.atom/packages/tidal/lib/repl.coffee:67:19)
REPL.editorIsTidal (/Users/robjac/.atom/packages/tidal/lib/repl.coffee:29:13)
atom-workspace.REPL.atom.commands.add.tidal:boot (/Users/robjac/.atom/packages/tidal/lib/repl.coffee:22:23)

I'll try to help anyway I can //

@robjac
I opened a issue the other day #14
I'll take a look tomorrow, if you want you can send me a pull request, shoule be easy! Thanks

@seansay Oh sorry about duplicating the issue.
I'll send you a pull request if/when I have some free time! Thanks sean.

@robjac
I remove all the old apis, not the warning shoule be gone after update to the new version

@seansay Excellent! great work dude! ; Installing now...
cheers,
-Rob


CAMEL-9047: Replace deprecated boxjavalibv2 with box-java-sdk

Thanks Bill. Just a quick observation, at ASF we dont have @author tags in the javadoc / source code.
Do you mind remove those and push code to this PR

Also I am not sure if you are aware of checkstyle.
Can you run checkstyle to update the code to the code format we use
Run in the box2 folder of the new component
 mvn compile -Psourcecheck

And then fix those errors and push updated PR here, thanks

Hi Claus,
Sure I can pull out the @author tags no problem.
I did already run mvn compile -Psourcecheck before doing the PR and cleaned up all of the validation error.  So I think the source code should be compliant with the Camel project style settings.
William

Updated PR with new commit removing @author tags.

I wonder if you would like to add a documentation page, eg in the src/main/docs there should be a box2-component.adoc file.
See for example the doc to the old component
https://github.com/apache/camel/blob/master/components/camel-box/src/main/docs/box-component.adoc
Mind that those tables with all the options is auto generated, so all you need to do is keep those START END tags and then the build does add the table content itself.

And the pom.xml files should have the ASF license header in the top. See any of the existing pom.xml file to copy the lic header

The box2-component.adoc is already there in PR. Please see: https://github.com/punkhorn/camel-upstream/blob/1b82a0d3b07f47d786660dc81186c7927dacbd86/components/camel-box2/camel-box2-component/src/main/docs/box2-component.adoc

Updated PR with new commit adding ASF headers to pom.xml files.

Thanks for the PR it has been merged. Do you mind closing this?

Thanks.
The code has been merged and replaced the old camel-box, as its going to die anyway when box closes that v1 api.
There is few more things and I logged a ticket. You are welcome to take a look and provide a new PR
https://issues.apache.org/jira/browse/CAMEL-10947


Siop verifier cleanup

deleted/changed console logs with debugs

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature/siop-verifier@975801e). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 6c5e089 differs from pull request most recent head c268d74. Consider uploading reports for the commit c268d74 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                   Coverage Diff                    @@
##             feature/siop-verifier     #104   +/-   ##
========================================================
  Coverage                         ?   42.71%
========================================================
  Files                            ?       76
  Lines                            ?     3186
  Branches                         ?      778
========================================================
  Hits                             ?     1361
  Misses                           ?     1825
  Partials                         ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I've added args for SIOPv2OID4VPRestClient constructor in this pr


Providing empty website url crashes application

After providing empty website url (for the first time - when no default has been stored yet) generator crashes.
? What do you want to name your module? test
? What is your GitHub username? elmccd
? What is the URL of your website?
     _-----_
    |       |    .-----------------------.
    |--(o)--|    |      Bye from us!     |
   `---------´   |       Chat soon.      |
    ( _´U`_ )    |      Yeoman team      |
    /___A___\    |    http://yeoman.io   |
     |  ~  |     '-----------------------'
   __'.___.'__
 ´   `  |° ´ Y `

/home/mccd/apps/node/lib/node_modules/yo/node_modules/rx/dist/rx.js:77
    throw e;
    ^
TypeError: Parameter "url" must be a string, not object

The reason for that is here:
validate: x => x.length > 0 ? true : 'You have to provide a website URL',
filter: x => normalizeUrl(x)

It looks like normalizeUrl, which throws exception for empty string, is being ran before validating provided value.

Did you add a space or just nothing? I guess it should be x.trim().length but that might not be the reason for this failure.

Nothing.
It looks like filter function is executed before validating value.
After removing this line from source code filter: x => normalizeUrl(x) and after submitting empty url I was prompt with 'You have to provide a website URL' as expected.

Seems after the latest yeoman-generator the order of validate and filter changed. // @SBoudrias

Relevant commit (and issue) https://github.com/senshuu/Inquirer.js/commit/8dfc3526b78a1519b73596911c15363f69d2807e.

Can't reproduce this anymore. Must have been fixed in Inquirer.js.


Feature/jstassi/#98 convert scripts to python3

Ricardo, these updates are utility programs that do not have any impact on DAS processing.

Ricardo, all the files that you mention are in the unittests subdirectory. These files are not included in the build. Their only purpose is to test the Python scripts in the GSI_GridComp/etc directory, to verify that the scripts give expected output after being modified. That seems like a useful thing to have. The unittests/outdir subdirectory with its README file were previously in the repository but have been deleted with this PR.

Why can't you design your test so that it runs off the RC files in the ETC directory of the installation? Why should you copy files from gsi/etc into your unit-test env. Sorry, but I still think there is a level of clean up to be done here. You haven't explained to me why you need files that identical to be in the repo w/ different names!

Ricardo, my tests check for expected output from a given input, so if I used the rc files in the etc directory as my input, then I would have to update my tests every time somebody updates the rc files in the etc directory.
Some of my tests read an rc file and then write it back out, which for a successful test means that the output is going to be identical to the input. I think that is what you are seeing when two files with different names are identical.

From: rtodling @.>
Sent: Monday, August 15, 2022 3:22 PM
To: GEOS-ESM/GEOSana_GridComp @.>
Cc: Stassi, Joe (GSFC-610.1)[SCIENCE SYSTEMS AND APPLICATIONS INC] @.>; Author @.>
Subject: [EXTERNAL] Re: [GEOS-ESM/GEOSana_GridComp] Feature/jstassi/#98 convert scripts to python3 (PR #99)
Why can't you design your test so that it runs off the RC files in the ETC directory of the installation? Why should you copy files from gsi/etc into your unit-test env. Sorry, but I still think there is a level of clean up to be done here. You haven't explained to me why you need files that identical to be in the repo w/ different names!
—
Reply to this email directly, view it on GitHubhttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FGEOS-ESM%2FGEOSana_GridComp%2Fpull%2F99%23issuecomment-1215644398&data=05|01|joe.stassi%40nasa.gov|95ceb4ddf7184f12c7eb08da7ef3926e|7005d45845be48ae8140d43da96dd17b|0|0|637961881844256456|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=sdrSWan3bWEMKeLGT5XMckrUJG17f6CRMWCUt6xIdRE%3D&reserved=0, or unsubscribehttps://gcc02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMBAJ2VUL6FVBFKGLF2SFDLVZKKJHANCNFSM56JIPAVA&data=05|01|joe.stassi%40nasa.gov|95ceb4ddf7184f12c7eb08da7ef3926e|7005d45845be48ae8140d43da96dd17b|0|0|637961881844256456|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=cgBXKAFhuzO3LB3NvB7%2BnACqE9ZVgq7JIQTG8uG2seM%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


Cross repo test for foreman provider explorer conversion changes

Testing ManageIQ/manageiq-ui-classic#6782, ManageIQ/manageiq#19949 and ManageIQ/manageiq-schema#464
@himdel please review

@miq-bot add-reviewer @himdel
@miq-bot  add-reviewer @Fryguy


edited Contributing.md to add 'doc_xxx' option

Included language inviting contributors to improve documentation.
(Put an X inside the [ ] to denote check mark [X].)

[ ] If creating a new file :

[ ] added links to it in the README files ?
[ ] included tests with it ?
[ ] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[ ] if done some changes :

[ ] wrote short description in the PR explaining what the changes do ?
[ ] Fixes #[issue number] if related to any issue

[x] other

a minimal edit

Pull Request Test Coverage Report for Build 720

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 72.366%

Totals

Change from base Build 719:
0.0%

Covered Lines:
4245

Relevant Lines:
5866

💛  - Coveralls


Sortable Data Table example renders incorrectly

When we visit the usage page for data table at https://www.carbondesignsystem.com/components/data-table/usage and we click the isSortable prop the styles are incorrectly being applied to column headers on hover.

This has been fixed 👍


Remove Error boilerplate with EitherT

This does look a lot nicer. Thanks!

Glad to perform some janitorial duties. :)


Use of static readonly string properties vs const

Hi,
I noticed that e.g., PropertyNames.cs declares its properties static readonly instead of const.
Based on this (http://stackoverflow.com/questions/755685/static-readonly-vs-const) post it would seem that const might be faster since the literal is burned into the call site. Since this is a safe change to make, I recommend it.
BTW I came across this issue since my own code has an almost identical list, and uses const string instead. I was trying to do away with my code in favor of AngleSharp's declarations.
Oh, and it would be helpful to make class PropertyNames public.
Cheers.
Edit: I just realized that if each assembly has its own copy of the string, fast compare by reference will not work. Is this why they are declared as static readonly? Does that consideration outweigh the overhead of a property access (I guess it does.) Ah well, .net performance never seems to stop being a puzzle even for the most basic things :)

Yes, PropertyNames will be public in the long run. And yes, it is static readonly intentionally. The SO thread should be digested thoughtfully; in micro-benchmarks the direct reference is faster, but for AS static readonly makes more sense.


Attention dropout

Hello,
Was looking at your code and noticed that in the Attention module, you're doing dropout and setting it to x, but on the next line, you're multiplying your att w/ v and setting that to x so dropout doesn't actually get applied. Am I reading that correctly?
x = nn.Dropout(self.dropout_rate)(attn, deterministic=deterministic)

# return weighted sum over values for each query position
x = jnp.einsum('...hqk,...khd->...qhd', attn, v).reshape(B, T, C)
x = nn.Dense(C, dtype=self.dtype, name='c_proj')(x)

Thanks!

you're correct -- good catch! Missed this in my tests since dropout is turned off


Problem with similar hardware

Hi
I have a lenovo ideapad 510-15ISK laptop with the same processor and graphics. (i3 6100u intel hd520). I installed big sur. Wifi, bluetooth, audio works. I patched the battery so battery status is working, generated kext via CPUFriend to manage cpu.
And I have a problem. My computer works up to a certain point.  When they stop, I also have a CPU problem - high kernel_task usage)
In VideoProc, it does not detect the HEVC codec. I can see that on your computer the program detects this codec because you have a screenshot attached. The intel power gadget doesn't work for me either. Does it work for you? And the final problem is:
at some point kernel_task uses 100% of my CPU and that causes huge lags and I have to shut down my computer.
I am sending you my EFI folder.
Can you take a peek?
Do you have any idea what I should do?
https://drive.google.com/drive/folders/1wghC6sO4kpYpC1dwP1c6jeAmFYd-fLY-?usp=sharing

Hi
I have a lenovo ideapad 510-15ISK laptop with the same processor and graphics. (i3 6100u intel hd520).

Hi, try this my config with your configuration.I think it should work fine. Replace EFI folder on my.
EFI.zip

Now everything seems ok even though VideoProc still doesn't detect the hvec codec. Was your screenshot taken on macos catalina? I read that BigSur forces 10 bit hvec in this case. And our CPU only supports 8 bit hvec and therefore VideoProc doesn't detect this codec. Thank you for your help

Now everything seems ok even though VideoProc still doesn't detect the hvec codec. Was your screenshot taken on macos catalina? I read that BigSur forces 10 bit hvec in this case. And our CPU only supports 8 bit hvec and therefore VideoProc doesn't detect this codec. Thank you for your help

In my BigSur VideoProc detect the hvec codec


Is there some documentation/research ppr?

Hey thanks for this work!
Are there any documentation of the whole process of whats actually happening here like some research ppr etc?

It's just a reversed rasterization and bfs to diffuse the colors. Actually there is a much faster method using mesh transform: https://stackoverflow.com/questions/5071063/is-there-a-library-for-image-warping-image-morphing-for-python-with-controlled

It's just an inverted rasterization and bfs to diffuse the colors. Actually there is a much faster method using mesh transform: https://stackoverflow.com/questions/5071063/is-there-a-library-for-image-warping-image-morphing-for-python-with-controlled I'll share the implementation soon.

Hey thanks for the reply I'll look into that and looking forward to the implementation.
Also is there a way to apply the texture/render the texture using meshlab or something on the body model created through smplify-x instead of only the smplx-uv obj file?

It's just an inverted rasterization and bfs to diffuse the colors. Actually there is a much faster method using mesh transform: https://stackoverflow.com/questions/5071063/is-there-a-library-for-image-warping-image-morphing-for-python-with-controlled I'll share the implementation soon.

Hey thanks for the reply I'll look into that and looking forward to the implementation. Also is there a way to apply the texture/render the texture using meshlab or something on the body model created through smplify-x instead of only the smplx-uv obj file?

You can use any renderer, for example pytorch3d has a very easy api to use.

It's just an inverted rasterization and bfs to diffuse the colors. Actually there is a much faster method using mesh transform: https://stackoverflow.com/questions/5071063/is-there-a-library-for-image-warping-image-morphing-for-python-with-controlled I'll share the implementation soon.

Hey thanks for the reply I'll look into that and looking forward to the implementation. Also is there a way to apply the texture/render the texture using meshlab or something on the body model created through smplify-x instead of only the smplx-uv obj file?

You can use any renderer, for example pytorch3d has a very easy api to use.

I imported the smplx model created through smplify-x for my image in meshlab. When I go to filters and apply the texture img file created through your scripts there is no uv map for this model. How do I generate/utilize the uv map file provided by smplx?
sorry I'm a noob to this :(


could you please provide the progard rules?

we found it build failed with error:
Unresolved reference: enableShakeOpen
when in release flavor, but success in debug flavor. So it should be progard rules issue which I could Not find here.

This is a mistake in the iteration of the version, please upgrade to the latest version.
In addition, what I need to explain is that we just add some APIs with empty implementation in Release flavor, so proGuard rules are not required. If you haven't called any APIs of pandora, you don't even need to integrate no-op.


[IMPROVEMENT] Add error reason when replica marked failed or tgt server hits media error

Is your improvement request related to a feature? Please describe (👍 if you like this request)
Currently, it seems the error info is not enough when a replica is marked failed or the tgt server of a volume hits media error to cause the following IO error, especially when the network of a worker node (IM pod running on that) of the cluster env is unstable.
Need to review the data path and enrich the error info/reason to the logs or even related Longhorn resources.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
cc @longhorn/dev-data-plane

@derekbit Please add more info if needed to see what we would like to improve the troubleshooting and debugging.

Reassigned to @james-munson , but need @derekbit follow up.


refactor(types): better null type handling

Use null literal to compose types intersections instead of using NonNullable for not nullable fields. It should lead to a lighter type definition.
BREAKING CHANGE: Not sure if it really breaks anything but won't hurt to create a major version just in case.
fix #33

Looks like a great clean up, and actual schema-to-code correspondence. By visually looking thru the changes I cannot spot any issues with it.
Thanks a bunch!


Error in runing:  goal is None in get_reroute_traj

Hi author~  I launch the simulator with Waymo dataset as  the tutorial in readme. I choose running mode=0 or 1, but when the "step" function in drive.py begins , the following error will occur when env agent runs get_reroute_traj function, it seems like the env agent can't get it's goal so that it fails to search the new route?
I'm sure that my environment dependencies and datasets are installed correctly. So I would like to consult for advice, what is the possible reason?  have you encountered similar error before? many thanks~

Could you please share the following information so that I can repeat and check the problem?

Dataset using
File name
Scenario index or id
Does it crash for one specific scene or every scene?

Thanks so much for your reply which reminds me I set the  tf_example_dir in config.py as  waymo test data, after changing to training or validation data, the above error disappear.
#######################################
But another error occurs during debugging:
I found if I choose Waymo training data,  and choose the first tfrecord file in training part(index=0 in the 1000 tf files) and scenario index = 230 (same as the default value in original code ), then when base planner executes, it print "TEST OFF ROAD!!!!!!!!" :

and the terminal outputs:

Then I continue running, the error occurs:

it seems like ego car fails to find lane sequence as the new route?
Then I choose waymo validation data(index = 0 in the 150 files, while scenario index, same as the default value in original code, is 230 too),  no errors appear
I wonder if this is normal？

The warning of 'TEST OFF ROAD' is normal, but not the crash during simulation. I just updated the planners after additional debugs. Could you please try it again and see if it still crashes for the same scenario?


Document update - ResidualModel & ImplicitModel

Documents show the usage of ResidualModel in the simulation but it is not defined in the latest version of the package. I think it is replaced by ImplicitModel.

@mkaracelebi it looks like you might be reading old documentation. Version 0.8.5 shows the following


Empty value for compare limits changes results, silently without error

Different results between a normal query:
https://bookworm.htrc.illinois.edu/cgi-bin/dbbindings.py?query={"groups":["date_year"],"counttype":"WordsPerMillion","words_collation":"Case_Insensitive","database":"Bookworm2016","search_limits":{"word":["tea"]},"method":"data","format":"json"}
vs one with "compare_limits":[]:
https://bookworm.htrc.illinois.edu/cgi-bin/dbbindings.py?query={"groups":["date_year"],"counttype":"WordsPerMillion","compare_limits":[],"words_collation":"Case_Insensitive","database":"Bookworm2016","search_limits":{"word":["tea"],"publication_country__id":["2"]},"method":"data","format":"json"}

Those appear to be two different queries? (publication_country__id is in the second).
Behavior is anticipated but maybe not explained anywhere.
When compare_limits is undefined, the API uses search_limits but with the word key removed, which is the most common use case. (E.g., count the total number of words in this corpus as the comparison for the number of words my search limit is defined.) When compare_limits is intentionally set to empty, OTOH, it returns the counts for the entire database without regard to the search_limits.
An empty compare_limits should be {}, not [], because you may only have one reference corpus.
If compare_limits is undefined and 'word' is not a key in the search_limits, I think it just starts removing things by some pattern I don't fully know.
There's also an undocumented shorthand where you can write {"search_limits": {"word": ["foo"], "*topic": ["bar"]}} where the asterisk directs the API to drop topic, rather than 'word', when it builds compare_limits.

Those appear to be two different queries? (publication_country__id is in the second).
Behavior is anticipated but maybe not explained anywhere.
When compare_limits is undefined, the API uses search_limits but with the word key removed, which is the most common use case. (E.g., count the total number of words in this corpus as the comparison for the number of words my search limit is defined.) When compare_limits is intentionally set to empty, OTOH, it returns the counts for the entire database without regard to the search_limits.
An empty compare_limits should be {}, not [], because you may only have one reference corpus.
If compare_limits is undefined and 'word' is not a key in the search_limits, I think it just starts removing things by some pattern I don't fully know.
There's also an undocumented shorthand where you can write {"search_limits": {"word": ["foo"], "*topic": ["bar"]}} where the asterisk directs the API to drop topic, rather than 'word', when it builds compare_limits.


PullRequest constructor returns what?

The constructor returns what and wherefore??

Returns a promise, is this an issue?

Perhaps it is just a lack of understanding from my side and is this a novel way of using classes that I haven't come across yet, but it seems to me that you can't (externally) call any of the methods on the PullRequest object if your create an instance of it with new. This because it seems you are not returning an instance of PullRequest but instead an instance of Promise. Granted, you do not expect any of its methods to be called externally...

Yep.
Design wise, could be confusing, in the first instance.
But I will try to elaborate on that:
... IMO...

Choosing Class is just, handy and clean way, to build an Object that relays in the a state, scope, methods (private/public).
Returns a promise, because is clean to consuming and is not that much different from new Promise() + you could await new PullRequest(...) simple has that. Even if I though about the return and I felt I bit guilty in the return, I still claim has valid choice.
The resolve returns the detail of a [PR] (https://developer.github.com/v3/pulls/#create-a-pull-request). You could do something like const { data } = new PullRequest(...)

@remarcmij Have a look what @timneutkens 😎 did in #16
I think is a neat start, and is ready to be consumed with other kinds of implementations/setups.
And then... I do like the concept of service, do one thing good.
Maybe should be documented, with some comments.
But also let's be honest the strong point is the consumable endpoint, this is not a module... yet.. :-)
But I would like to know more how you think can be structured better; and which API deliver.
If you have ideas I will be glad to hear them.
For now, Thanks 🙏

You could use XMLHttpRequest as a model.
So your PullRequest object could follow the same principle, e.g.:
const pr = new PullRequest();
pr.configure(...);
try {
  const result = await pr.execute();
  ...
} catch (error) {
 ...
}

This code would still do what you want, but now using a familiar, trusted scheme. What do you think?

The issue has been resolve.
Thanks @remarcmij 👍


compare content-encoding fails

- compare: {header: "content-encoding", comparator: contains, expected: 'gzip'}

also the extract_test
- extract_test: {header: "content-encoding",  test: "exists"}

both ends in a invalid header name:

ValueError: Invalid header name content-encoding

This error is thrown when no header with that name is defined in the HTTP response.  In this case it means your test is failing.
Remember that to get compressed responses, you must explicitly set the 'Accept-Encoding: deflate, gzip'  header on your request to indicate the client can accept compressed content.   You can also verify this behavior by running the corresponding curl call, with --verbose and --compresssed.

Thanks, gzip encoding has to be enabled first
- test:
    - curl_option_encoding: "gzip"

@thebino That works, but a better way is to set the 'accept-encoding: deflate/gzip' header in the test configuration (there is a 'headers' option).


Program crashes on other systems.

I tried running the program on another system, and it seems to error out. The program is crashing because it's trying to import bookmarks.py which doesn't exist anymore, but I'm not sure why it doesn't error on our current systems. Do you have any idea why this occurs?

Check for an old .pyc or .pyo file.

or the import app.bookmarks on line 24 of app/ci_program.py

oh, that makes sense.

I've removed it. I'll close this -- please let me know if it's not fixed.


Single group

Major upgrade simplifying execution and flow:

containerization of all analyzer tools in use
removing multi-report generation
simplifying report directory structure
bumping up tools to their latest version
adding report validation (./audit report) capability
adding tool container image validation (./audit validate) capability

Closes #13


Get user profile api

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8791ea6) 86.04% compared to head (59ee38d) 86.25%.

Additional details and impacted files
@@                Coverage Diff                @@
##           fyle_orgs_api      #34      +/-   ##
=================================================
+ Coverage          86.04%   86.25%   +0.20%
=================================================
  Files                 30       30
  Lines               1261     1280      +19
=================================================
+ Hits                1085     1104      +19
  Misses               176      176

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


(QENG-3787) Update EPEL package name for EL7

Change the default EPEL package name for EL7 from "epel-release-7-5.noarch.rpm" to "epel-release-7-6.noarch.rpm"

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2391/
Test FAILed.

Have squashed in a commit that updates the spec tests - is there a way to re-run the Jenkins checks?

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2392/
Test FAILed.

😹

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2394/
Test PASSed.


There is no expand animation when clicked on the last item on KitKat

When expanding the last group on Android 4.4, there is no expand animation, the group is opened instantly. The collapse animation for the same group works fine.
Possibly on Ice Cream Sandwich too, because removing the following lines from the expandGroupWithAnimation(int groupPos) method in AnimatedListViewExample.java makes it work again:
boolean lastGroup = groupPos == adapter.getGroupCount() - 1;
if (lastGroup && Build.VERSION.SDK_INT >= Build.VERSION_CODES.ICE_CREAM_SANDWICH) {
    return expandGroup(groupPos, true);
}

+1 to this. Why is the expand animation disabled for 14+, did it cause problems in a specific android version?

+1, I want to know why you disabled the animation of last group.

https://github.com/idunnololz/AnimatedExpandableListView/commit/eae713255fbd7ee2b89dff88479cfa341914dc08
This commit message makes me think it's a simple mistake and condition should be other way around.

@nomyzs
Seems you are right. Testing on Lollipop, I changed to this condition at the beginning of expandGroupWithAnimation():
if (lastGroup && Build.VERSION.SDK_INT < Build.VERSION_CODES.ICE_CREAM_SANDWICH) {
    return expandGroup(groupPos, true);
}

+1 I shield these codes and everything is OK. I have tested in android 2.3, android 4.4 and android 5.1.

+1, I am testing in my tablet ME173X (JellyBean) and seems that the animation works for API level 17 too.

When you include a footer view. The last group stop responding to listener. You click and nothing happens


Whether the library is still being maintained

Whether the library is still being maintained
At present, it has not been updated for half a year, and more than 400 issues have not been processed

Well. this is really werid, have anyone checked  it seems like a version there is 0.8.0 and on githaby it's still 0.7.2 ?
The question remains though... Any good forks ?

@vborbely   Process: com.example.mower_app, PID: 15067
E/AndroidRuntime(15067): java.lang.NullPointerException: Attempt to invoke virtual method 'int android.util.SparseArray.size()' on a null object reference
E/AndroidRuntime(15067): 	at com.pauldemarco.flutter_blue.ProtoMaker.from(ProtoMaker.java:71)
E/AndroidRuntime(15067): 	at com.pauldemarco.flutter_blue.FlutterBluePlugin$2.onScanResult(FlutterBluePlugin.java:779)
E/AndroidRuntime(15067): 	at android.bluetooth.le.BluetoothLeScanner$BleScanCallbackWrapper$1.run(BluetoothLeScanner.java:512)
E/AndroidRuntime(15067): 	at android.os.Handler.handleCallback(Handler.java:873)
E/AndroidRuntime(15067): 	at android.os.Handler.dispatchMessage(Handler.java:99)
E/AndroidRuntime(15067): 	at android.os.Looper.loop(Looper.java:201)
E/AndroidRuntime(15067): 	at android.app.ActivityThread.main(ActivityThread.java:6864)
E/AndroidRuntime(15067): 	at java.lang.reflect.Method.invoke(Native Method)
E/AndroidRuntime(15067): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:547)
E/AndroidRuntime(15067): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:873)
I/Process (15067): Sending signal. PID: 15067 SIG: 9
Lost connection to device.

Have you encountered this problem and how to solve it

Like two years ago, I will am creating a fork that will gather all critical updates.
Please write if you have some important things to add.
You can use:
flutter_blue:
git:
url: https://github.com/boskokg/flutter_blue/
ref: 0.9.0
0.9.0

Android migrate to mavenCentral.
Android support build on Macs M1
Android protobuf-gradle-plugin:0.8.15 -> 0.8.17
Ios example upgrade to latest flutter 2.5
deprecated/removed widgets fixed in example

@boskokg  是否会合并到pub ？

@jahnli It depends on the activity of the maintainer @pauldemarco
Two years ago, he reviewed the commits and updated the main plugin.
If there is no activity from him for some time, we can create a new plugin.

@boskokg  Looking forward to providing a new plugin, flutter_Blue has not been maintained for over half a year

@boskokg  If you release a new plug-in, please inform me here, thank you

@boskokg would you please implement this changes https://github.com/boskokg/flutter_blue/compare/master...MZachmann:updater

@boskokg I'm all for everyone shifting work to a new fork of this repo. If we are migrating to your fork, will you turn on Issues for the fork?

flutter_blue:
git:
url: https://github.com/boskokg/flutter_blue/
ref: 0.10.0
0.10.0
mtuRequest returns the negotiated MTU
Android: functions to turn on/off bluetooth
Android: add null check if channel is already teared down
Android: code small refactoring (fixed AS warnings)
Android: add null check if channel is already teared down
Ios: widen protobuf version allowed

@peekpt I added most of the changes from the MZachmann:updater
I did not added background functionality since there was a discussion about the way it is done. @pauldemarco reviewed it then and did not apply the same changes.
You can read it here: https://github.com/pauldemarco/flutter_blue/issues/510#issuecomment-606466267
I will check for other pull requests to apply.
@Scowley4 I will turn on the issues. Thanks

@boskokg nice job, thank you!
Is there any fix for startScan() that is not filtering withServices: [Guid1,Guid2,...] correctly

Thanks @peekpt
Not sure about startScan.
I am doing now the removal of v1Embedding and removal of Activity usage where possible for the people that use the plugin out of activity.
@peekpt Please let me know if you find any important merge requests.

flutter_blue:
git:
url: https://github.com/boskokg/flutter_blue/
ref: 0.11.0
0.11.0

The timeout was throwing out of the Future's scope #941
Expose onValueChangedStream #882
Android: removed V1Embedding
Android: removed graddle.properties
Android: enable background usage
Android: cannot handle devices that do not set CCCD_ID (2902) includes BLUNO #185 #797
Android: add method for getting bonded devices #586
Ios: remove support only for x86_64 simulators
Ios: Don't initialize CBCentralManager until needed #599

@boskokg
We look forward to you re creating a library for better maintenance
Consider publishing in pub.dev

0.13.0

Supporting Android 12 Bluetooth permissions. #940

0.12.0

Delay Bluetooth permission & turn-on-Bluetooth system popups on iOS #964

There will be a pub.dev push of this branch ?

There will be a pub.dev push of this branch ?

You mean a merge of this fork? @boskokg said the author probably will do that

@peekpt @boskokg Was anyone of you in contact with the author? In case the author is not reachable, moving this branch to pub under a new plugin name should be considered.

@peekpt @boskokg Was anyone of you in contact with the author? In case the author is not reachable, moving this branch to pub under a new plugin name should be considered.

Everything should be considered in open source

@boskokg @peekpt If the fork is going to be used to create a new plugin name, could I suggest that the fork is moved to a github organization, rather than under a specific user's account, and have atleast two maintainers to avoid the same situation happening again in the future?

I published flutter_blue_plus 1.0.0 with the latest code in the fork.
I will add @peekpt as a maintainer. Please suggest if someone else should be the maintainer.
The fork that we used for bug fixes will no longer be maintained since there is no need.

@boskokg   Thank you for your contribution


How to not allow white space in a string

Package version eg. v8, v9:
V8, because using in conjunction with gin gonic.
Issue, Question or Enhancement:
How do I exclude a whitespace from a string?
I have a name that should allow all unicode characters, except a white space. I can't figure it out in the docs.
Code sample, to showcase or reproduce:
type Policy struct {
	Id      string `json:"id"`
	Name    string `json:"name" binding:"required,min=3,max=20, ascii,exclude=<some way of defining whitespace>`
	Version int    `json:"version"`
	Steps   []Step `json:"steps" binding:"required,dive"`
}

https://github.com/go-playground/validator/blob/v8.18.2/examples/custom/custom.go

@tnolet I think your looking for excludesall and does adding a space not work?
PS: you can use v9 with Gin, see here


Implement multi-tenant Vector DB that supports Edge (rm Weaviate)

Weaviate:

Ah.... I got it now. namespace is the tenant.
So you can just use, instead:
const store = await insertDocuments(docs, "", process.env.LONG_TERM_MEMORY_INDEX_NAME)
if you want to run this with multiple tenants, you can create the schema beforehand, and enable multitenancy, like on this recipe:
https://github.com/dudanogueira/recipes-js/blob/main/src/multi-tenancy/example.ts
So whenever you have a new tenant, you  add it to your class, and then provide the same name (in the recipe tenantA and tenantB) as the namespace

Supabase (with https://usebasejump.com schema, RLS policies, and RPC functions), Chroma, Zep are main contenders.

Solved (for now, pending security audit) via filters and keeping the vector db unexposed to the frontend


Adding user current playback and available devices endpoint

Note that https://github.com/plamere/spotipy/pull/182 covers this and more.


feat: Allow timeout configuration in shared/e2e

RMS-2967

@jaredallard thanks for the approval! Now how do we deploy a new version of devbase? 🤔

You ask us! :D
This wouldn't go out until next month though, or we can promote it to rc and get it out with the stable release on Monday. If it's critical.

It's not critical but nice to have as it will solve annoyances in RMS tests (such as having 100+ failed tests when it should only display a few) it's a minor impact change so I'd propose to promote to rc

You ask us! :D
This wouldn't go out until next month though, or we can promote it to rc and get it out with the stable release on Monday. If it's critical.

@jaredallard if we can promote it to rc, it will help to solve this problem on Monday: https://github.com/getoutreach/rms/pull/1349