fix: add deploy errors to validate error message

What does this PR do?
Move some error discovery code to a common function so it can be used by the command or the formatter
What issues does this PR fix or reference?
@W-15207335@

QA

linked plugin-deploy-retrieve to sf
create scratch org in dreamhouse-lwc, sf org create scratch --edition developer --alias scratch-org
Referenced an unknown class in force-app/main/default/classes/FileUtilities

✅  Failed (human readable)
❯ sf project deploy validate -d force-app -o scratch-org
 ›   Warning: @salesforce/plugin-deploy-retrieve is a linked ESM module and cannot be auto-transpiled. Existing compiled source will
 ›   be used instead.
Validating Deployment of v60.0 metadata to<EMAIL_ADDRESS>using the v60.0 REST API.
Deploy ID: 0Af5300001CF1liCAD
Status: Failed | ████████████████████████████████████████ | 92/92 Components | Errors: 3

Component Failures [3]
=====================================================================================================
| Type  Name              Problem                                                         Line:Column
| ───── ───────────────── ─────────────────────────────────────────────────────────────── ───────────
| Error FileUtilities     Invalid type: ContentVersion2 (9:45)                            9:45
| Error FileUtilitiesTest Dependent class is invalid and needs recompilation:             9:45
|                          Class FileUtilities : Invalid type: ContentVersion2 (9:45)
| Error propertyCarousel  Unable to find Apex action class referenced as 'FileUtilities'.

Test Results Summary
Passing: 0
Failing: 0
Total: 0
Error (1): Failed to validate the deployment (0Af5300001CF1liCAD). Due To:
Error in FileUtilities - Invalid type: ContentVersion2 (9:45)
Error in FileUtilitiesTest - Dependent class is invalid and needs recompilation:
 Class FileUtilities : Invalid type: ContentVersion2 (9:45)
Error in propertyCarousel - Unable to find Apex action class referenced as 'FileUtilities'.

3 component error(s)

✅  Failed (--json)
❯ sf project deploy validate -d force-app -o scratch-org --json
 ›   Warning: @salesforce/plugin-deploy-retrieve is a linked ESM module and cannot be auto-transpiled. Existing compiled source will
 ›   be used instead.
{
  "code": 1,
  "context": "DeployMetadataValidate",
  "commandName": "DeployMetadataValidate",
  "data": {
    "deployId": "0Af5300001CF1jNCAT"
  },
  "message": "Failed to validate the deployment (0Af5300001CF1jNCAT). Due To:\nError in FileUtilities - Invalid type: ContentVersion2 (9:45)\nError in FileUtilitiesTest - Dependent class is invalid and needs recompilation:\n Class FileUtilities : Invalid type: ContentVersion2 (9:45)\nError in propertyCarousel - Unable to find Apex action class referenced as 'FileUtilities'.\n\n3 component error(s)",
  "name": "FailedValidationError",
  "status": 1,
  "stack": "FailedValidationError: Failed to validate the deployment (0Af5300001CF1jNCAT). Due To:\nError in FileUtilities - Invalid type: ContentVersion2 (9:45)\nError in FileUtilitiesTest - Dependent class is invalid and needs recompilation:\n Class FileUtilities : Invalid type: ContentVersion2 (9:45)\nError in propertyCarousel - Unable to find Apex action class referenced as 'FileUtilities'.\n\n3 component error(s)\n    at Messages.createError (/Users/mdonnalley/repos/salesforcecli/plugin-deploy-retrieve/node_modules/@salesforce/core/lib/messages.js:431:16)\n    at DeployMetadataValidate.run (file:///Users/mdonnalley/repos/salesforcecli/plugin-deploy-retrieve/lib/commands/project/deploy/validate.js:203:18)\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\n    at async DeployMetadataValidate._run (/Users/mdonnalley/repos/salesforcecli/plugin-deploy-retrieve/node_modules/@oclif/core/lib/command.js:304:22)\n    at async Config.runCommand (/Users/mdonnalley/.volta/tools/image/packages/@salesforce/cli/lib/node_modules/@salesforce/cli/node_modules/@oclif/core/lib/config/config.js:426:25)\n    at async run (/Users/mdonnalley/.volta/tools/image/packages/@salesforce/cli/lib/node_modules/@salesforce/cli/node_modules/@oclif/core/lib/main.js:85:16)",
  "exitCode": 1,
  "warnings": []
}


Enhance so that it works with lm-studio's OpenAI API

lm-studio is a relatively common way to run local LLMs and present an OpenAI API endpoint however (maybe I should open an issue with them - I will) does not implement model properly so this would break for example
tools: sys.http.html2text
Model Name: "local-model"

And I get
export OPENAI_BASE_URL="http://localhost:1234"
❯ gptscript agd.gpt
13:39:07 started  [main]
2024/03/15 13:39:07 model not found: "local-model"

If I curl the lm-studio endpoint I see
curl http://localhost:1234/v1/models
{
  "data": [
    {
      "id": "/Users/tok/.cache/lm-studio/models/bartowski/merlinite-7b-GGUF/merlinite-7b-Q8_0.gguf",
      "object": "model",
      "owned_by": "organization-owner",
      "permission": [
        {}
      ]
    }
  ],
  "object": "list"
}%

And their sample code suggests they don't implement model fully
# Example: reuse your existing OpenAI setup
from openai import OpenAI

# Point to the local server
client = OpenAI(base_url="http://localhost:1234/v1", api_key="not-needed")

completion = client.chat.completions.create(
  model="local-model", # this field is currently unused
  messages=[
    {"role": "system", "content": "Always answer in rhymes."},
    {"role": "user", "content": "Introduce yourself."}
  ],
  temperature=0.7,
)

print(completion.choices[0].message)

This works today if you follow the instructions here: https://docs.gptscript.ai/alternative-model-providers


Update to 4.2.0

Update to libzmq 4.2.0

Seems something has changed in the library name in Linux & Win. I will have a detailed view when possible. Also, I think I will open a branch in the repo to support this, or maybe one for 4.1.5. Thanks!

Opened 4.1.5 branch, will continue 4.2.0 in master.
Something changed in the lib naming scheme in Win, and also now some dependencies to win libraries like "wsock32","Iphlpapi". I fixed it and submitted to CI, lets see if improves. When fixed, I'll continue with linux and OSX.


.coffee files become excluded if a .js version exists in the same directory

I have my repo setup so that potato.coffee compiles in the same directory to potato.js. When potato.js exists, my code coverage always shows 0% for potato.coffee. If I remove potato.js, code coverage for potato.coffee goes back to a correct percentage.
example structure where I'm running into issues
project
│  README.md
└───lib
│   │   potato.coffee
│   │   potato.js   <- when this exists, coverage of project/lib/potato.coffee is always 0%
│                      when it doesn't exist, coverage of project/lib/potato.coffee is normal
└───test
│    │
│    └───lib
│    │    │   potato.coffee
│    │    │   potato.js

Here's my project/test/mocha.opts in case it matters:
--compilers coffee:coffee-script/register
--require coffee-coverage/register-istanbul
--reporter spec
--recursive
--growl
test/lib/potato.coffee

This isn't really a coffee-coverage problem, it's more of a node.js problem.  If you're using coffee-coverage to dynamically instrument files, then the files get instrumented when node.js loads them, but if you require "potato", then node.js will always try to require potato.js before potato.coffee, and will just ignore the coffee file if the js file is there.  I'm not sure there's anything we can do to fix this.

Closing, since I haven't heard back.


This package Microsoft.Azure.Management.Automation 2.0.7 is reported as NotSupported NetStandard2 although it has a dependency with NetStandard 1.4 is this correct?

This package Microsoft.Azure.Management.Automation 2.0.7 (https://www.nuget.org/packages/Microsoft.Azure.Management.Automation/2.0.7) is reported as NotSupported NetStandard2 although it has a dependency with NetStandard 1.4 is this correct?
I've installed the package a a netstandard2.0 library and it compiles with no warnings.

https://www.nuget.org/packages/Microsoft.Azure.Management.Automation/2.0.7 only supports .NET standard 1.4, However, looks like https://www.nuget.org/packages/Microsoft.Azure.Management.Automation/3.8.1-preview supports .NET standard 2.0. Looks like APICatalog that APIPort getting .NET API information may not be up to date for this library. @safern, can you take a look?

Closing as API Port was deprecated in favor of binary analysis in .NET Upgrade Assistant.


store initial configuration of the service during service deployment

As part of service configuration update, we must start with storing initial service configuration properties during the service deployment.

[ ] Add new table SERVICE_CONFIGURATION with foreign Key as the ID of the DEPLOY_SERVICE table. and CONFIGURATION column
[ ] CONFIGURATION column stored configuration as JSON.
[ ] Add created and updated time columns
[ ] Extract initial configuration as below -

JSON Key is name of ServiceConfigurationParameter
Value is the initialValue of ServiceConfigurationParameter
If the same name is present in the flavor properties, then that value from the deployed flavor must be used as the initial value.

[ ] Use  kafka service properties from bitnami kafka docker image and update the service template
https://github.com/bitnami/containers/tree/main/bitnami/kafka#111-debian-9-r205-220-debian-9-r40-111-ol-7-r286-and-220-ol-7-r53
To start with, we can only provide the following configuration parameters in the template - KAFKA_CFG_MESSAGE_MAX_BYTES, KAFKA_CFG_LOG_DIRS, KAFKA_CFG_NUM_IO_THREADS, KAFKA_LOG_FLUSH_INTERVAL_MESSAGES, KAFKA_OFFSETS_TOPIC_REPLICATION_FACTOR

let me do it


webots_ros2: 1.1.3-2 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository webots_ros2 to 1.1.3-2:

upstream repository: https://github.com/cyberbotics/webots_ros2.git
release repository: https://github.com/ros2-gbp/webots_ros2-release.git
distro file: foxy/distribution.yaml
bloom version: 0.10.7
previous version for package: 1.1.2-2

As PR for galactic, I had to change by hand the file distribution.yaml

Closed in favor of a new PR with correct settings of version.


ocs: Update Volume Snapshots secret name generation

For external restriced mode the csi-secrets name will
not be default, so updating the name accordingly in the
volume snapshot class so snapshots can work
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2091998
Signed-off-by: parth-gr<EMAIL_ADDRESS>

/cherry-pick release-4.11

/retest-required

The testing looks good the volume snapshot class is been updated to use the desired secret name,
@iamniting @nb-ohad can you have a look onto the cahnges,
And also see it wouldn't affect the managed services.

@parth-gr @agarwal-mudit This change has a very high probability to break consumer mode.
I would highly suggest to test a custom build of this change in consumer mode before merging.

/hold

@parth-gr @agarwal-mudit This change has a very high probability to break consumer mode. I would highly suggest testing a custom build of this change in consumer mode before merging.
Putting a hold until we have clarity

It just matches the secret name prefix which would not affect the by default secrets,
I tested this on external cluster, do you also wanted to test me the manged services mode?

@nb-ohad @Madhu-1 @iamniting any update??

It just matches the secret name prefix which would not affect the by default secrets,
I tested this on external cluster, do you also wanted to test me the manged services mode?

My main concern is not the change to the secret name, as @Madhu-1 already confirmed that in consumer mode we are using a different set of secrets. The concern is the structure of the data in externalOCSResources.
@iamniting @Madhu-1 I would like your opinion on the effect of the storagecluster UID indexing change on the externalOCSResources map. Is that going to cause us problems in consumer mode?

@nb-ohad @Madhu-1 @iamniting any update??

PS: I feel like there is no other parameter added so the total number of values will be the same in the map so no change in ordering, may that's what your question means

/lgtm

/retest

/hold cancel

/hold cancel


Clarify in storage FAQ item that it's about Prom 1.x

Fine with me as well if nobody objects in 3... 2... 1...

oh shoot, should have looked at the code, I thought you made the change. Will do in master.


uwsgi: 2.0.21 -> 2.0.22

https://uwsgi-docs.readthedocs.io/en/latest/Changelog-2.0.22.html

plugins/php: Fix PHP 8.2 compilation

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Haven't tested the plugins yet.

Haven't tested the plugins yet.

ok nixosTests.uwsgi passed

@GrahamcOfBorg build uwsgi

@drupol I quickly fixed a typo in your edit. Do you want to squash it, or should I do it?

Please do, I'm having coffee with colleagues atm

let me just amend the commit message

I like this kind of collaboration :) thanks for being proactive!


Update multithreading.c

It looks more clearly like this <3

Thank you for your contribution @uysalibov :)


Feature/read commands from file

🎉 open with -@ file.data, create writetable stream to file.data, read metadata by writing to the data file, close exiftool.


Show change links for foreign keys

https://github.com/django/django/commit/9d9f0acd7e245c9f9c30727a003666618754c924

That's actually never interesting for us -- content types are never registered as admin models on their own...


Testcafe browser differs from local browser

What is your Test Scenario?

I would like to make sure that testcafe fails when it should in this particular case as its passing when it should fail.
What is the Current behavior?

The scenario is at my company we raise instances in AWS put our product on the instance then run the automation. When I try to go to the instance I get stuck with a spinner at the login page. I tried firefox, chrome, chromium, safari, incognito, tor, etc... They all get stuck at this spinner and you cant get to the login page because of this.
When I run some tests via testcafe on my computer using chrome on this instance it gets past this spinner, logs in, then just resumes tests like nothing is wrong. I have tried using localhost as the host, different ports, skip js errors, and other flags.
I believe I have proved this is related to the proxy server as if i try some online proxy servers, some of them perform like testcafe does and successfully gets to the login page.
The developers have found out that this was due to bad headers in the code.  I am not sure but my guess is that testcafe is filtering out these headers with the proxy server thus allowing the test to bypass the error in the browser.
What is the Expected behavior?

When i run a test via testcafe it will also get stuck on this spinner then fail.
What is your web application and your TestCafe test code?

Your website URL (or attach your complete example):
Please let me know if this is needed and when someone will look into this and I will raise an instance for a short period of time and provide the IP.  Also is there a way to keep the IP private?

Please let me know if this is needed and when someone will look into this and I will raise an instance for a short period of time and provide the IP. Also is there a way to keep the IP private?

The developers have found out that this was due to bad headers in the code.

We cannot debug the issue in the way you wish to proceed. We need to be able to run the example locally. It looks like your programmers figured out the area where the issue might be. So, please share a simple example that we can use to reproduce and research the issue.

Hi you can use this:
https://<IP_ADDRESS>:8443

Thank you for sharing the example. I was able to reproduce the described behavior.

Thanks i have torn down the environment as of now.  Please let me know if you need another instance of this.

@AndrewRedisLabs
To investigate the issue, we need a testing page that should be reachable at any time.  Feel free to reopen this issue if you can make such an example.


Prepare data before passing to custom parser

Custom parser can be provided in Schema.
It is ok to parse data in dict before passing to custom parser or provide a way to call use factory inside custo mparser.

Pass factory instance
Check custom parser anotations
Add zero-step parsing setting (e.g. link to other type)

Probably can be replaced with #24 and post parse step

Does not apply to the new major version, now we have chaining


MailChimp.php still says about version 2.2

See here: https://github.com/drewm/mailchimp-api/blob/master/src/MailChimp.php#L11
I think it should be 2.4.

If it means a lot to you, I'd welcome a PR 😄

Done :)


refactor(chips): remove 6.0.0 deletion targets

Removes the deletion targets for 6.0.0 in the material/chips entry point.
BREAKING CHANGES:

remove which was deprecated in 5.0.0 has been removed. Use removed instead.
destroy which was deprecated in 5.0.0 has been removed. Use destroyed instead.

Caretaker note: we need to remove google usages of the old API. This might require a cleanup before syncing this PR.


Keep contextTypes and childContextTypes

Make sure that JSC does not think these are dead code which happens when
it collapses them to Comp$contextTypes. By tagging them with
@nocollapse JSC keeps these as static properties.
The only one we can remove is propTypes which is not checked in
production mode anyway.

TBR

LGTM


Window title only repainting when window is resized

VSCode Version: Code 1.14.2 (cb82febafda0c8c199b9201ad274e25d9a76874e, 2017-07-19T23:34:09.706Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author (truncated)
Version

Bookmarks
ale
0.15.2

cucumberautocomplete
ale
2.3.1

vscode-eclipse-keybindings
alp
0.9.6

xml
Dot
1.9.2

LogFileHighlighter
emi
1.1.1

tortoise-svn
fan
0.1.1

vscode-javac
geo
0.0.46

wordcount
ms-
0.1.0

seti-icons
qin
0.1.3

java
red
0.7.0

vscode-icons
rob
7.11.0

velocity
sod
0.1.0

(1 theme extensions excluded)

Steps to Reproduce:

Open a file with vscode - its name is not placed into the window title
Resize the vscode window by either dragging one of the application borders or double-clicking the window title
Window title will update

This also applies to tabs - changing tab does not repaint the window title, only a resize will. I had previously changed my window.title setting but since removed it from my settings.json and since I've been seeing this problem I'd even explicitly put the default window.title setting into settings.json, but it didn't make any difference.

Reproduces without extensions: Yes

This bug seems to be fixed in Electron 2.0.4.
"Windows
Fixed windows causing repaint issues when DWM composition is disabled. #12501"


Allow running in place on an existing checkout

Ran into this when I was setting up https://github.com/rubygems/rubygems.org/pull/5019 -- I want to just run frizbee on the current checkout, so it can run on every PR

Hello @segiddins!  I'm excited about this PR -- anything we can do to help you out in revisitng this?

I'll try to circle back next week!

@evankanderson This should be ready for another look, along with the other two PRs!

@segiddins hey, it looks like the linter is complaining about the cyclomatic complexity of the init function now. I'll leave it up to you if you want to reduce the complexity or just slap a nolint above (I've done both in about equal amounts...)

Let's see if this helps?


Add a way to remove an attachment from the scope

There should be a way to remove an attachment, which was previously added, from the scope.
This prevents that the same attachment is sent for multiple crash reports.
It's also the last missing piece needed for finishing https://github.com/getsentry/sentry-dart/pull/576

That depends on https://github.com/getsentry/sentry-dart/issues/510 by the way.
Discussed solutions are making it available on beforeSend via hints or adding a boolean to the add attachment method, sendOnlyOnce or a new method called removeAttachment(...), we're going to discuss it.

If the goal is to capture a single event with an attachment the solution is to capture an event with a local scope.

@ueman does https://docs.sentry.io/platforms/dart/enriching-events/scopes/#local-scopes help to add an attachment to a single scope?

I think so. That would make it possible to do
Sentry.captureException(
    exception,
    withScope: (scope) {
        scope.addAttachment(ScreenshotAttachment());
    },
);

#1136 is going to unblock this via hints

You can do https://github.com/getsentry/sentry-dart/issues/711#issuecomment-1106119591
or https://github.com/getsentry/sentry-dart/blob/fa16e53b4bb6d070a637610acb48612b82700d3c/dart/lib/src/scope.dart#L211-L213


Stochastic Inference for Classification, Fixing Z

What is the purpose of fixing and then unfixing "Z" in the documented classification example? If one were to implement stochastic variational optimization for the SVGP classification example, should Z remain fixed during optimization and if so why?

It's a pretty fickle optimization problem since Z introduces many degrees of freedom - fixing Z and allowing the remaining hyperparameters to converge first gets the optimization routine into a more well-behaved part of the parameter space where the optimization routine can then be run as normal.

Thanks, that makes a good bit of sense. Second question (and I apologize, the documentations parameters are difficult to map to the referenced papers), when using one of the tensorflow optimization methods (e.g. ADAM), in the documentation maxiters is set to np.inf. Is there a reason that this is the case in that example but in all others the number of iterations is a (relatively small) constant?

That sounds like an unusual choice! I guess the reason is that GPflow behaves nicely when you interrupt it (ctrl-c) so perhaps the author of that document planned on letting it run until after breakfast 😀, when he could interrupt it manually.
Sent from my iPhone

On 1 Mar 2017, at 04:00, Charles Noyes<EMAIL_ADDRESS>wrote:
Thanks, that makes a good bit of sense. Second question (and I apologize, the documentations parameters are difficult to map to the referenced papers), when using one of the tensorflow optimization methods (e.g. ADAM), in the documentation maxiters is set to np.inf. Is there a reason that this is the case in that example but in all others the number of iterations is a (relatively small) constant?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closed due to inactivity


Escaped password working with psql but not Prisma Client

Bug description
Hello I have a generated password that is )u&rI5]FlgG|1)q57rpB8-4{S3~5?7G}. Obviously it needs to be properly escaped, so I've made to the point where I've escaped the password enough to be working with psql
psql postgres://user:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432

\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G} ✅ with psql
\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G} ❌ with prisma
)u&rI5]FlgG|1)q57rpB8-4{S3~5?7G} ❌ with psql and prisma

I've tried several escaping combinations, as well as URLEncoding the password, but I've never got to the point where what was working with psql was also working with prisma client.
Thanks in advance for any help! 🙏
How to reproduce

Prisma version
   "@prisma/client": "2.16.1",
Setup a PostgreSQL docker container
  postgresql:
    image: postgres:latest
    ports:
      - 5432:5432
    environment:
      POSTGRES_USER: user
      POSTGRES_PASSWORD: )u&rI5]FlgG|1)q57rpB8-4{S3~5?7G}

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

datasource db {
  provider = "postgres"
  url      = env("POSTGRES_URL")
}

.env
POSTGRES_URL="postgres://user:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432"

Run prisma migrate reset --preview-feature
The command above fails with the following error:
$ prisma migrate reset --preview-feature
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database ")u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\", schema "public" at "user::undefined"

Error: undefined: error: Error validating: This line is not a valid definition within a datasource.
  -->  schema.prisma:4
   |
 3 |                 provider = "postgresql"
 4 |                 url = "postgres://user:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432"
 5 |             }
   |

Expected behavior

I'd expect the escaped password working with psql to work with prisma client
Prisma information

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

datasource db {
  provider = "postgres"
  url      = env("POSTGRES_URL")
}

Environment & setup

OS:  Mac OS
Database: PostgreSQL
Node.js version: v15.12.0
Prisma version:

prisma               : 2.16.1
@prisma/client       : 2.16.1
Current platform     : darwin
Query Engine         : query-engine 8b74ad57aaf2cc6c155f382a18a8e3ba95aceb03 (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli 8b74ad57aaf2cc6c155f382a18a8e3ba95aceb03 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core 8b74ad57aaf2cc6c155f382a18a8e3ba95aceb03 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt 8b74ad57aaf2cc6c155f382a18a8e3ba95aceb03 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Studio               : 0.346.0

You should be able to urlencode the unescaped password and use it with Prisma I think. I am not sure if the same format would also work with psql though.

@janpio  Thanks for the quick answer, I've tried urlencode the unescaped password )u&rI5]FlgG|1)q57rpB8-4{S3~5?7G} which gives me )u&rI5%5DFlgG%7C1)q57rpB8-4%7BS3~5?7G%7D} no luck the error is
$ prisma migrate reset --preview-feature
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database ")u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\", schema "public" at "user::undefined"

Error: undefined: error: Error validating: This line is not a valid definition within a datasource.
  -->  schema.prisma:4
   |
 3 |                 provider = "postgresql"
 4 |                 url = "postgres://user:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432"
 5 |             }
   |

I've also tried with urlencode the escaped password \)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G} giving me )u&rI5%5DFlgG%7C1)q57rpB8-4%7BS3~5?7G%7D and it also does not work
$ prisma migrate reset --preview-feature
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database ")u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\", schema "public" at "user::undefined"

Error: undefined: error: Error validating: This line is not a valid definition within a datasource.
  -->  schema.prisma:4
   |
 3 |                 provider = "postgresql"
 4 |                 url = "postgres://ReviewToolDbUser:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432/reviewtool?schema=reviewtool"
 5 |             }
   |

Hm, what if you use this urlencoded string %29u%26rI5%5DFlgG%7C1%29q57rpB8-4%7BS3~5%3F7G%7D? (via https://www.urlencoder.org/)

@janpio Same error unfortunately with the string you shared
$ prisma migrate reset --preview-feature
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database ")u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\", schema "public" at "user::undefined"

Error: undefined: error: Error validating: This line is not a valid definition within a datasource.
  -->  schema.prisma:4
   |
 3 |                 provider = "postgresql"
 4 |                 url = "postgres://user:\)u\&rI5]FlgG\|1\)q57rpB8-4{S3~5\?7G}@localhost:5432"
 5 |             }
   |

Can you try this again @emyann? This is what I am getting today:

With the original plain text password DATABASE_URL="postgresql://user:)u&rI5]FlgG|1)q57rpB8-4{S3~5?7G}@localhost:5432":

C:\Users\Jan\Documents\throwaway\6505>npx prisma db pull
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db"

Introspecting based on datasource defined in prisma\schema.prisma …
Error: Error parsing connection string: invalid port number in database URL

This error message is obviously not great, but we are tracking that in an issue already.

With the url/percent-encoded password DATABASE_URL="postgresql://user:%29u%26rI5%5DFlgG%7C1%29q57rpB8-4%7BS3~5%3F7G%7D@localhost:5432":

C:\Users\Jan\Documents\throwaway\6505>npx prisma db pull
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": PostgreSQL database "postgres", schema "public" at "localhost:5432"

Introspecting based on datasource defined in prisma\schema.prisma …
Error: P1000

Authentication failed against database server at `localhost`, the provided database credentials for `user` are not valid.

Please make sure to provide valid database credentials for the database server at `localhost`.

And that is expected as I do not have a user with that password on my local server.

@1323ED5 We can't encode it ourselves because then we won't be able to determine termination points of various parts of the URL that are determined by special characters like @


Bulk import tasks

This could be a
tod task import /path/to/file.md
And then the file could be in format
- This is a main task p1
 - This is a subtask p2
- This is another task

And then the user is prompted for which project they should be inserted into
Maybe we could get a little crazy and also find or create the project as well?
# ProjectOne

- This is a main task p1
 - This is a subtask p2
- This is another task

# ProjectTwo

- This task is in another project

I think that Todoist has an excellent "complex" import option, and maybe we can just support the simple case.
Would you consider using this? How about you @titoOdUA ? I do have situations where I have a list of tasks in my text editor and would love to just select them and use a command to pipe them straight into Todoist... but I wouldn't say that this is a critical feature for me.

I like this suggestion. I currently copy/paste delimited lists (1 per newline) into Todoist as tasks. I would use an import feature.

I would start by adding a simple-as-possible plaintext bulk import. No markdown, no lists. Just a line by line import, every line is a task. This would give us a foundation on the import logic, with the possibility to extend it in the future

Let's do that, then spawn another request for subtasks.
From what I can tell, we don't really specifically support subtasks at the moment anyway (just treat them as another/regular task) so that's a future item.

I will start working on this (if Alan agrees on my solution) then!

I like it! I really appreciate you guys putting so much thought into it :+1:

Implemented and released, check out list import


wojcikowski.pl is down

Hi, I am trying to test this scoring function, but the server is down and, as a result, binaries not accessible, nor the *.pickle files. Maybe they can be uploaded as a GitHub release for permanent storage? Thank you, Jaime.

Everything works fine on my end. Could you check if the server resolves correcty? Maybe your DNS fails to do so.
As suggested I have uploaded a release v1.0 on Github for anyone who cant access the external server.

I checked on https://downforeveryoneorjustme.com/wojcikowski.pl and it looks like it can't be reached from there either.
This is the output from ping and tracepath:
> ping wojcikowski.pl
PING wojcikowski.pl (<IP_ADDRESS>) 56(84) bytes of data.
^C
--- wojcikowski.pl ping statistics ---
10 packets transmitted, 0 received, 100% packet loss, time 134ms

> tracepath wojcikowski.pl

 1?: [LOCALHOST]                      pmtu 1500
 1:  _gateway                                            106.134ms
 1:  _gateway                                              3.646ms
 2:  <IP_ADDRESS>                                        1.497ms pmtu 1488
 2:  <IP_ADDRESS>                                         57.832ms
 3:  <IP_ADDRESS>                                           58.477ms asymm  6
 4:  <IP_ADDRESS>                                          75.306ms asymm  5
 5:  <IP_ADDRESS>                                           58.104ms
 6:  ae8-1.mad10.core-backbone.com                        83.450ms asymm  5
 7:  ae10-2021.fra20.core-backbone.com                    94.367ms asymm  6
 8:  core-backbone.hetzner.com                            89.358ms asymm  6
 9:  core22.fsn1.hetzner.com                              95.076ms asymm  8
10:  ex9k1.dc6.fsn1.hetzner.com                           94.264ms asymm  9
11:  no reply
12:  no reply
13:  no reply
14:  no reply
15:  no reply
16:  no reply
17:  no reply
18:  no reply
19:  no reply
20:  no reply
21:  no reply
22:  no reply
23:  no reply
24:  no reply
25:  no reply
26:  no reply
27:  no reply
28:  no reply
29:  no reply
30:  no reply
     Too many hops: pmtu 1488
     Resume: pmtu 1488

Hope it helps! And thanks for the upload!

I do have access from one of my servers. Weird, indeed!

It is the provider's fault - there was a switch issue, they had resolved it, but I guess some caches need to be flushed... As you can see there seem to be a loop on one of Hetzner's switches.
Anyhow, for now we have redundancy on Github Releases. Thanks for the suggestion and detailed information from your network endpoint.
Feel free to close the issue unless there is anything I could do for you.

That was fast! Thanks a lot!


High security vulnerability reported in cloned gatsby-starter-netlify-cms

Hi there,
I cloned this repo, and my npm is reporting a high security vulnerability.
npm audit:
High            Denial of Service
Package         serialize-to-js
Patched in      >=2.0.0
Dependency of   parcel-bundler
Path            parcel-bundler > serialize-to-js
I don't see the vulnerability listed under this repo security tab, yet it occurs when I clone and install it. What do you guys recommend?

Run "npm i serialize-to-js" to update the package. A PR should be made to update package.json.

Thanks for responding Xayaan,
I tried  "npm i serialize-to-js", but the<EMAIL_ADDRESS>dependency is still showing a dependency on<EMAIL_ADDRESS>which is triggering the security warning.
Do you know of a way to force that sub dependency to update?
Here's the output I got:
command: npm list serialize-to-js
+--<EMAIL_ADDRESS>| --<EMAIL_ADDRESS>--<EMAIL_ADDRESS>

The packages are regurarly getting updated, i suggest to either pull down those changes from this repo to your cloned repo. Or setup a renovate bot and a dependa-bot, just as this repo does.

Installing the problem dep direct won't fix the issue, parcel-bundler will still get its own version.
@illulli are you still having this issue if you clone fresh today? Our dependency bots were in flux during a bit of experimentation over the last two weeks but everything should work great now.

@illulli are you still having this issue if you clone fresh today?

@erquhart I am not illulli but I just use Netlify CMS today and have this issue.

Hi, I tried to update the packages by npm update but get

Field "featuredimage" must not have a selection since type "String" has no subfields

with the image query in graphQL, is it a must to stay with the current package version?

Removed the parcel-bundler dependency.


Go Lang examples for Recurly.js

Examples using Go + Web.go (Web framework for Go)

:elephant:

I'm not sure what the elephant means, but I've got to clean up this PR by getting rid of the PHP related commits. The indenting on the main application file is also b0rked.

'huge'


Perf: Cloudflare Pages

Cloudflare Pages vs Netlify.
Perf and DX.

Guess I will stick with netlify for now

Completed, our websites will now be faster


Adapt plugin for general Membrane use cases

This PR:

removes Membrane.RTSP.Source.TCP and instead uses Membrane.TCP.Source, which was tweaked to use socket's active mode and has much better CPU performance.
replaces Membrane.RTSP.Source.Decapsulator with Membrane.RTSP.TCP.Decapsulator, which has since been updated to pick up interleaved RTSP messages more reliably and forward packets from all channels (including RTCP packets).
removes Membrane.RTSP.Source.Transport.TCPWrapper, because the functionality of passing socket control has since been added to Membrane.RTSP.
removes retrying mechanisms, as it would be more appropriate to handle connection failures by letting the element crash and handling it on a higher level with crash groups.
changes behavior of choosing ports for UDP.Sources in case of :udp transport, from choosing randomly from a fixed, wide range, to choosing free ports successively from a range passed via options. This decision was made to prevent situations where a port is free during the SETUP phase but taken by a different, unrelated process during PLAY phase. The ability to pass the port range via options is also needed when the ports available are limited by things like docker mappings or firewall.

Notes:

Passing control of the TCP socket from the RTSP session to the TCP.Source goes as follows:

TCP.Source sends {:request_socket_control, socket, pid} notification to RTSP.Source
RTSP.Source calls RTSP.Source.ConnectionManager.transfer_socket_control/2 with TCP.Source's pid
RTSP.Source.ConnectionManager calls RTSP.transfer_socket_control/2with TCP.Source's pid
RTSP calls :gen_tcp.controlling_process/2 with the socket handle and TCP.Source's pid

Hi @Noarkhh,
Thanks for the PR. Can you rebase the PR or merge master to this PR. I added two changes that's not relevant with your PR

I don't check for 200 for get parameter request (since some AXIS camera returns 401 but still keep sending media data). Since we don't pass the rtsp session to tcp decapsulator, this won't be an issue.
I restart the rtsp connection when it's closed, that's also not an issue since you removed the retry logic from this plugin.

I accidentally opened this PR from my master, I'll close it and open another one


subgraph of each question

my question is general and I greatly appreciate it if u answer me.
I think questions' subgraphs are built over the whole KGs based on a specific strategy(e.g. PPR). If I want to create embeddings of subgraphs' entities, should I do according the main dataset?
For example, I want to generate vector representations of the CQW-subgraph-entities  based on an algorithm, should I use the Freebase dataset or is there a sub set of Freebase to do it?

There are some pretrained embedding for Freebase entities, e.g. http://<IP_ADDRESS>/index/toolkits#pretrained-embeddings

On Mar 11, 2021, at 2:54 AM, dinani65 @.***> wrote:
my question is general and I greatly appreciate it if u answer me.
I think questions' subgraphs are built over the whole KGs based on a specific strategy(e.g. PPR). If I want to create embeddings of subgraphs' entities, should I do according the main dataset?
For example, I want to generate vector representations of the CQW-subgraph-entities based on an algorithm, should I use the Freebase dataset or is there a sub set of Freebase to do it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/OceanskySun/GraftNet/issues/21, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADE5XL345YV4NOHVGUWRXUDTDBZJVANCNFSM4Y7YLK2Q.


update examples and getting started for provider and api package

Finishes the provider rename in examples and getting started
Updates examples and getting started for api package update

@open-telemetry/javascript-approvers please review this. Until it is merged examples, getting started, and benchmark are all failing on master.


Use the realpath of original filename in _AtomicFile

This way, the code becomes resilient to the chdir() changing mid-way, between
the open() and close() of _AtomicFile.

bump?

Thanks! merged.

Thanks! Appreciated.


Take it graceful if the icon file cannot be found

In case the icon cannot be found we should not get a fatal error in the logging.
Addresses #174

I think this isn't enough. If the case happens where HasEmbeddedIcon is true but no icon is found, there's something more wrong than just a missing icon. Either the database or the file system would be inconsistent, which would be a way bigger problem than just one crashing package download, in which case BaGetter shouldn't just return no icon but prompt people to fix it.
The HasEmbeddedIcon property isn't just used in this one case either. The icon also is displayed as part of the metadata in several places, depending on that property. The data being inconsistent with the files would break those too, even though that probably wouldn't be critical bugs but just display errors. Still errors though.
I would be open to try/catching this error on the web UI though in order to provide actionable information about database and file system being inconsistent. That way the core code, which is in itself consistent, stays like it is, but the error is still caught and (hopefully) fixed.


release: v1.0.0b1

Signed-off-by: Nicolas Harraudeau<EMAIL_ADDRESS>

@lnielsen The travis build already passed on my personal account. I am also preparing the PR for invenio-files-rest which will enable again devel mysql build


Imported functions not working

Tell us about your environment:

Puppeteer version: The latest version
Platform / OS version: Visual studio code windows 10
Node.js version: The latest version

What steps will reproduce the problem?
Importing a long JS function from a different file:
let startTheShow = require('./scraper.js');
And then trying to use that function on the page.
await page.evaluate(() => {

    startTheShow('info','hi','hi');

})

What is the expected result?
The imported function will run on the page
What happens instead?
"startTheShow is not defined"
Placing the whole script inside the evaluate {} instead of importing the function, fixes the issue. Unfortunately this option makes a lot of mess in the index file. Any idea why I can't use this imported function?

Hi, I realized that I first have to expose the function like this:
await page.exposeFunction("startTheShow",startTheShow);
await page.evaluate(async function() {
   await startTheShow('info','hi','banana');
})

But it will return this error:
Error: Evaluation failed: TypeError: this._pageBindings.get(...) is not a function
any idea?

I have also tried this syntax : ```
await page.exposeFunction("startTheShow",async function() {await startTheShow;});
await page.evaluate(async () => {
await startTheShow('hi','hi','hi');

It does not throw an error any more, but it will not run the function on the page

Hey @ErezShahaf , are you running the latest published Puppeteer from npm or from this GitHub repo directly?

Going to close this, but please open a new issue if you're still seeing this issue.

I am using the latest published Puppeteer which is v1.18.1 and node v10.15.0.
With this code, I am getting the exact same error, "VM1571:2 Uncaught (in promise) TypeError: this._pageBindings.get(...) is not a function".
        await page.exposeFunction('finder', finder);
        await page.evaluateOnNewDocument(()=> {
            document.addEventListener("click", function callback(e){
                const selecotr = finder(e.target);
                console.log(selecotr)
            } , true /* capture */);

        });

Can confirm same error, got no idea how to fix this

Quick fix:


Feature: Support Dim Mode

Is it possible to support the dim bit for the Embrava lights?  It looks like there is hardware support per the API, but it doesn't look like setting this bit is currently possible.
I'm looking to not glow as much in darker conditions when the light is on :)

Ha, well I'll just close this one.  I discovered I could just set the second bit of the hex for the relevant color to 0 and it is nicely dimmed by 50%:
red = 0xff0000 (bright) | 0x0f0000 (dim)
green = 0x00ff00 (bright) | 0x000f00 (dim)

Should anyone have an opinion, my proposed implementation is:
$ busylight --dim N [on|blink|fli] [colors]

Where N is an integer from 1 to 100, representing a scaling value to apply to each channel of the requested color.

The global --dim option is now available in version 0.19 of busylight and honored by all the subcommands: on, blink, throb, fli, and rainbow. Additional work done polishing the CLI (catching exceptions, harmonizing global option handling, worked on debug output and other small UX fixes).
$ busylight --dim 100 on green   # no effect, full value green
$ busylight --dim 80 on green    # green is 80% intensity
$ busylight --dim 50 on green    # green is 50% intensity
$ busylight --dim 10 on green    # green is 10% intensity

I know this issue is a little stale, but please re-open it if you have any other thoughts about dimming :)


Color palet has only one/two column

When choosing a new color for either text of backgroun of text, the color palet is way too long it stretches.
Description
OS - Tested on Windows and Ubuntu
Browser - Brave v1.49.132
Screenshot:

This is happening if the editor is not in modal window.

Outch, such an edge case here!
It's because they are on the right, which is not a usage I had thought about.
I cannot fix it, but feel free to move those to the left to get rid of this bug :)


Repo is from the future

krisajenkins/AdventOfCode-2019

Thank you @lpil. I was in a hotel in Sweden when I set up this repo, and I think I was a bit disoriented. :-D


Validate the driver pod begins running before exiting spark-submit

In fire-and-forget mode, we should check that the driver started running before exiting. We should also report on the driver's status if it fails to run.

@mccheah is this still applicable?


drivers: tee: Introduce shm memory registry API

Tee subsystem can receive request from TEE OS to allocate memory,
which is represented by tee_shm object. Tee subsystem should keep
track of the allocated tee_shm object in terms of some session to be
able to clean remaining objects, which where not released by TEE OS
before closing session.
This API is tracking only objects, allocated in terms of some session
and do not track objects, allocated using syscalls tee_shm_alloc and
tee_shm_register because they are not connected to the specific
session and should be managed by caller.

After the discussions it was decided to close this PR as not relevant. That's because of the following statements:

OP-TEE OS is more secure than Zephyr, so we can rely on the OP-TEE OS and expect that resources are managed by OP-TEE os and cleanup is it's responsibility.
Resources, requested using syscalls, as was mentioned in the description - are not related to session. So those resources can't be tracked by driver because there is no deinit implementation in Zephyr. Also, as we expect clients to be running in the userspace threads - we expect userspace threads to manage allocated resources.


landsat-api cron job for updating the meta data is failing

The public version of the api has not been updated for sometime because the cron job is failing on our server. Has to be fixed

@scisco were you able to fix the cron job?

yes! 130K new records were added  :)

Great, I think we can close https://github.com/developmentseed/landsat-util/issues/24


Exclude Storage Partition from the available image flash area

Addresses #27471
The problem:
The storage partition is not visible to the linker, When code-partition is NOT used, the image code area may step into the storage partition, which may lead to problems hard to debug.
Proposed solution (in this PR):

Describe in Boards DeviceTree the reserved flash area that shall not be accessible to the image
Define the corresponding Kconfig symbols
Use the Kconfig symbols in the Cortex-M linker to exclude the area from the image code space

Notes

Only required when the build does not use the DT CODE PARTITION
Only addressed for Nordic nRF DKs - should be extended to all in-tree board definitions

Limitations:

Efficient only when the reserved area is at the end of the flash area (majority of the cases)

@ioannisg, when code-partition is not used, what is the meaning of the flash division ? IMO if you want to use the flash division as described by the dts you should mandatory use code-partition. If the board dts file does not describe what is needed, it should be overriden.

@ioannisg, when code-partition is not used, what is the meaning of the flash division ? IMO if you want to use the flash division as described by the dts you should mandatory use code-partition. If the board dts file does not describe what is needed, it should be overriden.

@Laczen thanks for the feedback, I also had the same thought. However, there's a challenge: mandatory use of code partition means mandatory using a bootloader as well, and this cannot be accepted as a default solution.

when code-partition is not used, what is the meaning of the flash division ?

To elaborate more: users might not want to use the code partition, together with bootloader, and instead use the whole available flash. But they need to exclude the storage partition at the end.
We need to find a solution that is backwards compatible with the current situation and still addresses the rather serious bug of having the zephyr code overlap with the storage space.
Forcing to use code-partition is not backwards-compatible.
I am open for alternative suggestions of course.

when code-partition is not used, what is the meaning of the flash division ?

To elaborate more: users might not want to use the code partition, together with bootloader, and instead use the whole available flash. But they need to exclude the storage partition at the end.
We need to find a solution that is backwards compatible with the current situation and still addresses the rather serious bug of having the zephyr code overlap with the storage space.
Forcing to use code-partition is not backwards-compatible.
I am open for alternative suggestions of course.

Maybe we can just define the default behavior using code-partition: The default flash layout could just be: code-partition and storage-partition. I never understood why mcuboot is dictating the flash layout, maybe it is time to correct this.

Maybe we can just define the default behavior using code-partition: The default flash layout could just be: code-partition and storage-partition.

So, my patch is sort of doing this thing:

defines a reserved flash area (for storage and/or for anything else that the image cannot use)
rest of flash for code.

This is still the default. MCUboot enables USE_DT_CODE_PARTITION, and forces the flash layout, but that is not default.

I never understood why mcuboot is dictating the flash layout, maybe it is time to correct this.

I guess the reason is that the partitions need to be statically defined in DeviceTree (which runs before Kconfig) and we need to account for whether we do need to use bootloader and image slots or not.
@Laczen if you can share a draft of your proposal and how it should work, feel free to post it here.
@mbolivar-nordic @nvlsianpu I hope you could give some feedback here, as well.

Forcing to use code-partition is not backwards-compatible.

I don't follow this. The storage partition is defined in DT. Arguably, using the storage partition but not the code partition was always wrong.

Forcing to use code-partition is not backwards-compatible.

I don't follow this. The storage partition is defined in DT. Arguably, using the storage partition but not the code partition was always wrong.

Code partition is needed only when we deal with chain-loadable zephyr images, e.g. together with MCUboot, and/or in the case of Secure/Non-Secure images.
Storage partition may also be used outside this scenario; one may not care at all about bootloaders, but still wants to reserve some area for storage.
If we now force to use DT CODE PARTITION whenever storage partition is used, we will break compatibility with the above use cases.

I guess the reason is that the partitions need to be statically defined in DeviceTree (which runs before Kconfig) and we need to account for whether we do need to use bootloader and image slots or not.

@mbolivar-nordic this is where i see the problem. All partitions are defined statically, and the are not aware of whether they are being used or not. This is decided later, with Kconfig.

Code partition is needed only when we deal with chain-loadable zephyr images, e.g. together with MCUboot, and/or in the case of Secure/Non-Secure images.

No it's not. It's the partition that code goes into.

@ioannisg I think you're not understanding the purpose of the code partition.
There's nothing that's chain-loading specific about it.
You can read the documentation here: https://docs.zephyrproject.org/latest/reference/devicetree/index.html#chosen-nodes

I don't think this is the right approach for the reasons I mentioned above.

@mbolivar-nordic  I assume you insist on forcing the use of DT CODE PARTITION, whenever the storage partition is to be used by the application, is that correct?
Here's a typical default Zephyr board flash layout:
partitions {
		compatible = "fixed-partitions";
		#address-cells = <1>;
		#size-cells = <1>;

		boot_partition: partition@0 {
			label = "mcuboot";
			reg = <0x00000000 0xc000>;
		};
		slot0_partition: partition@c000 {
			label = "image-0";
			reg = <0x0000C000 0x32000>;
		};
		slot1_partition: partition@3e000 {
			label = "image-1";
			reg = <0x0003E000 0x32000>;
		};
		scratch_partition: partition@70000 {
			label = "image-scratch";
			reg = <0x00070000 0xa000>;
		};
		storage_partition: partition@7a000 {
			label = "storage";
			reg = <0x0007a000 0x00006000>;
		};
	};

So, if a Zephyr application wants to use the storage partition, it will have to use the code-partition as well. Code partition is usually slot0_partition. This partition does not start at address 0x0, so the image needs to be chain-loadable by a parent image, at least for Cortex-M.
How would this work without a bootloader? This is the challenge I am talking about, otherwise, like I said, I am fine going with this.

Forcing to use code-partition is not backwards-compatible.

I'm in line with this. But maybe we can switch to use the code-partition silently
Problem here is that partition layout with and without mcuboot differs each other.  Second problem is that Kconfig goes after DT - so build system is unable to choice proper partition layout basing on CONFIG_BOOTLOADER_MCUBOOT.
We never really defined layout without MCUBoot in DTS. Solution might be to switch to:

provide non-mcuboot layout for each boards as the default
provide overlay with mcuboot-enabled partition layout for each board (we can)
set default for CONFIG_BOOTLOADER_MCUBOOT basing on existence of the boot-partition in DTS
start to always use code partition.

This will keep backward compatibility while building. Building for MCUBoot will require to inject overlay file (but no need to set CONFIG_BOOTLOADER_MCUBOOT=y explicitly anymore), would be great if path in relation to the board directory will might be used.

@nvlsianpu I have been working on just this thing earlier today. Let me post a new draft PR, to share this with the reviewers. :)


[core] Improve object table for fileIO, Privileged and parent_path

Purpose

Improve object table:

FileIO can be customized in AbstractCatalog.
Privileged should handle ObjectTable.
Add parent_path in Object table schema.

Tests

API and Format

Documentation

+1


Fade In / Fade Out

Some Patterns or images might benefit from a fade in / fade out option

Indeed they would. I thought of putting in an "envelope" to allow brightness shaping based on a grayscale image, but maybe that is too complex. It would be pretty easy to add a fade in/out based on a time or frame count setting. Would that be good enough?

Frame based would be harder for the internal patterns, I guess, so I would vote for time based

It is much harder to do for the internal ones. They are all kind of stand-alone functions. The BMP display function can easily handle it. Although... thinking... maybe I could use an interrupt timer to do it. Might be kind of messy. I'll give it some thought this evening.
I'll come up with something, I think the fade in/out is a good idea. Shall we have adjustable start/end or is black always that value. I vote for black.

another vote for black.

I added a fade in/out count to the  "file Image settings" menu. It is the number of frames to fade and out. 0 to 255 is allowed. I could make it longer but it would require floating arithmetic and I don't think it is really necessary. GIve it a try.
I haven't figured out any way to add it to the built-ins. It is of difficult to fade out the ones that run forever, but I suppose it could be done when the long press cancels the playing. I can probably figure out a way to do it. I'll keep giving it some thought, but for now it only works with BMP files.

I added a fade in/out count to the  "file Image settings" menu. It is the number of frames to fade and out. 0 to 255 is allowed. I could make it longer but it would require floating arithmetic and I don't think it is really necessary. GIve it a try.
I haven't figured out any way to add it to the built-ins. It is of difficult to fade out the ones that run forever, but I suppose it could be done when the long press cancels the playing. I can probably figure out a way to do it. I'll keep giving it some thought, but for now it only works with BMP files.

After I wrote the above I thought of a reasonable way to do the fade in/out for everything. It's a bit of work, but not too bad. I'll probably work on it next week.

After I wrote the above I thought of a reasonable way to do the fade in/out for everything. It's a bit of work, but not too bad. I'll probably work on it next week.

I have updated the code with improved handling of the fade in/out. It should now be possible to add fading to the built-ins. I haven't done this yet, but I will work on it soon.

I have updated the code with improved handling of the fade in/out. It should now be possible to add fading to the built-ins. I haven't done this yet, but I will work on it soon.

Re-readding this I noticed you liked the time based setting. Frame/column based was a lot easier to code. I could still make it time based, but I would have to map the time onto columns. I'm not sure that would be any better. It could also be a percentage of the total size, that almost makes more sense. For example: 10% of the total width. Anyway, let's play with the frame count based one and see if that works ok.

Re-readding this I noticed you liked the time based setting. Frame/column based was a lot easier to code. I could still make it time based, but I would have to map the time onto columns. I'm not sure that would be any better. It could also be a percentage of the total size, that almost makes more sense. For example: 10% of the total width. Anyway, let's play with the frame count based one and see if that works ok.


change DAO's intro message

It feels off User, ...

remove the user word from this message

The message is now updated and the user part was removed.

and the info icon leads to more details about DAO proposals and voting.

Verified, Devnet bd6c7c5.

Verifed:

Devnet
d2d2622


Update for Python 3.11 support

Codecov Report
Base: 95.65% // Head: 95.65% // No change to project coverage :thumbsup:

Coverage data is based on head (3eae7b3) compared to base (893d35d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   95.65%   95.65%
=======================================
  Files           3        3
  Lines          23       23
=======================================
  Hits           22       22
  Misses          1        1

Flag
Coverage Δ

unittests
95.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Rexec example: Golang server does not work with jsChan client

Hello everyone,
Based on an email conversation I had with Matteo, he suggested I open an issue here, as his assumption was that the Node client should work with the Golang server, based on last time he tested it.
Basically, I am testing the Golang libchan rexec server:
https://github.com/docker/libchan/blob/master/examples/rexec/rexec_server/server.go
Against the Node.Js jsChan rexec sample:
https://github.com/GraftJS/jschan/blob/master/examples/rexec/client.js
The versions tested are the ones in master repo as of January 15, 2015.
I had two console windows open, on the same server. One running a compiled golang version of libchan rexec server, waiting on port 9323.
The other console would be for the client:

the golang client run with "rexec echo hi" works correctly
the node client, run with "node index.js echo hi" does not work correctly

I added a few log lines in the server, and the approximate location where the server stops being responsive is right after:
t, err := tl.AcceptTransport()  (line 63, it passes this one)
and before
receiver, err := t.WaitReceiveChannel()  (line 71, it doesn't seem to pass this one, no errors thrown)
In other words, the t.WaitReceiveChannel()  does not appear to be responsive to jsChan's channel.
My regards to the team, great work,
Silviu

This is bad :(. I will try to have a look, but if in the meantime you would like to get it fixed, it might be awesome. I am a little low on bandwidth, so it might take a while :(.
@AdrianRossouw would you like to help tracking this down?

i don't think keeping in sync with libchan is one of our priorities anymore, since work there seems to have stalled.
we'll probably remove the references to it eventually.


Added Chrome extension (Icon in next pull :p)

I thought this was unsupported, so I added a chrome extension. Users need to load unpacked extension (please work on the icon(s).)
Many thanks
Jamie

I prefer having a css file only so it's easy to use on any browser that has Stylish or similar :)

Ok, I'm leaving this in my fork


Tag release 1.2

Hello!
Would you mind tagging a new release 1.2 ? So I can create the conda-forge packages with the latest commit.
Thanks!

Done, and I gave you write access too so you wont have to wait after me in the future :)

Thanks!


Search Stream Logging: Request Id Empty?

I recently noticed that with my TraceUtilities settings, Search requests include Request Id information while SearchStream requests do not. This behavior feels inconsistent and can pose some friction when providing information to Support.
For example, when I try to get the Information level summary logs:
TraceUtilities.Configure(TraceUtilities.SUMMARY_REQUEST_LOGS_SOURCE, listener, System.Diagnostics.SourceLevels.Information);

The Search logs look something like:
[2021-05-04 12:00:40Z] - Request made: Host: https://googleads.googleapis.com, Method: /google.ads.googleads.v6.services.GoogleAdsService/Search, ClientCustomerID: [Redacted],  RequestID: [Redacted, but present], IsFault: False, FaultMessage:
While the SearchStream logs from the same execution look like:
[2021-05-04 12:00:40Z] - Request made: Host: https://googleads.googleapis.com, Method: /google.ads.googleads.v6.services.GoogleAdsService/SearchStream, ClientCustomerID: [Redacted],  RequestID: , IsFault: False, FaultMessage:
Does anyone have a work-around for this issue?
-- Kevin

Hi, I'm still having trouble with this. Is there another way to get tracking information on SearchStream requests?

I'll amend this to say that sometimes I get the RequestId. I can't seem to figure out what triggers it to appear or not.

@AnashOommen This has come up again when talking with support on the Google Ads Forum. There is no request id to provide support with in many of my SearchStream calls, especially in failure cases. Help on this would be much appreciated.

I enabled some more detailed logging and the logs without Request Id look something like this when more verbose logging is enabled:
[2021-06-08 21:28:38Z] -
---------------BEGIN API CALL---------------
Request
Method Name: /google.ads.googleads.v6.services.GoogleAdsService/SearchStream
Host: https://googleads.googleapis.com
Headers: {
"x-goog-api-client": "gl-dotnet/3.1.0 gapic/6.1.0 gax/3.2.0+002f6637d17e4170f5db7ec7ca6f0163e6df1481 grpc/2.34.1",
"developer-token": "REDACTED",
"login-customer-id": "//** OMITTED **//"
}
{ "customerId": "//** OMITTED //", "query": "SELECT ad_group.cpc_bid_micros, ad_group.cpm_bid_micros, ad_group.effective_target_cpa_micros, ad_group.final_url_suffix, ad_group.id, ad_group.name, ad_group.status, ad_group.targeting_setting.target_restrictions, ad_group.tracking_url_template, ad_group.type, ad_group.url_custom_parameters, campaign.id, campaign.name, metrics.average_cpc, metrics.clicks\nFROM ad_group\nWHERE campaign.id IN (// OMITTED **//)\nORDER BY ad_group.id ASC", "summaryRowSetting": "NO_SUMMARY_ROW" }
Response
Headers: {
"content-disposition": "attachment",
"date": "Tue, 08 Jun 2021 21:28:38 GMT",
"alt-svc": "h3-29=":443"; ma=2592000,h3-T051=":443"; ma=2592000,h3-Q050=":443"; ma=2592000,h3-Q046=":443"; ma=2592000,h3-Q043=":443"; ma=2592000,quic=":443"; ma=2592000; v="46,43""
}
{ "results": [ //** RESULTS OMITTED **// ], "fieldMask": "adGroup.cpcBidMicros,adGroup.cpmBidMicros,adGroup.effectiveTargetCpaMicros,adGroup.finalUrlSuffix,adGroup.id,adGroup.name,adGroup.status,adGroup.targetingSetting.targetRestrictions,adGroup.trackingUrlTemplate,adGroup.type,adGroup.urlCustomParameters,campaign.id,campaign.name,metrics.averageCpc,metrics.clicks" }
----------------END API CALL----------------

This issue is fixed in v11.0.0.


Remove some old and broken build stuff

Just deleting some files that must be unused, because they're broken.

Hulk. Smash!
Thank you!


Access control on the explore page

I want to build dashboards for the users, but don't want them to send custom queries. If I take away their can_save and can_edit permission, they still can sent custom queries. So I want to prohibit the user from entering the pages like /caravel/explore/druid/3 . But if I take away their can_explore permission, they will be unable to see the slices on the dashboard. So this is what I am trying to add in view.py:
def explore(self, datasource_type, datasource_id):
    ...
    slice_custom_query_perm = self.appbuilder.sm.has_access(
        'can_custom_query', 'SliceModelView')

    if request.args.get("json") == "true":
        ...
    elif request.args.get("csv") == "true":
        ...
    else:
        if request.args.get("standalone") == "true":
            template = "caravel/standalone.html"
        else:
            if not slice_custom_query_perm:
                flash(_("You don't seem to have the permission "
                        "to customize the slices"), "danger")
                return redirect(error_redirect)

            template = "caravel/explore.html"

Do you think it is a good idea to design it this way? How can I create the new permission "can_custom_query" in caravel init? Thanks

Have you had any further feedback on this @x4base - we're trying to understand how to do the same (new to superset!)

Seems like removing can_explore from a role should work now since we now have a different view for the json (can_explore_json)

Tried it out @mistercrunch but not sure if my requirement was the same as described here. I'm looking for user to have restricted access to explore view to only be able to view the chart in greater detail + download the CSV data behind the chart. Right now if I give them can_explore permission they can do everything in the explore view including changing the slice configuration from the left menu which we don't want them to do

@shrutimkrishnan I'm guessing if you could export CSV from the dashboard view you wouldn't need explore at all

@mistercrunch yup exactly, saw you're taking this feature on in the other entry so thanks a lot!
Sent from my Google Pixel using FastHub


Support new encryptor image name format

The encryptor image name is used to determine valid images for AWS.
The image name is also used to determine the encryptor type (NetBSD
vs. FreeBSD) to make certain decisions (for cryptor policy and
sriovNetSupport). The image name format for FreeBSD encryptor
image was changed from brkt-avatar-freebsd- to
metavisor--. This change supports the new image name
format, but ignores the ealier image name format (brkt-avatar-freebsd).

It's probably best to match both "brkt-avatar-freebsd-* and "metavisor-*". This to remain compatibility with previous image names.

I don't see any reason to match "brkt-avatar-freebsd-*" and have to remove it later on, especially given the fact that we haven't even release FreeBSD MV for Beta yet. Most folks anyways use the latest encryptor and the only downside of not supporting the old image name format is that the encrypted images will not have sriovNet support enabled (if the source image had it enabled).


Metal: Validation fails if a shader program samples from textures in both fragment and vertex shaders

Describe the bug
Certain shaders cause Metal API Validation to fail. The condition seems to be:

The vertex shader samples from a texture, AND
The fragment shader samples from a texture.

To Reproduce
I couldn't find any shader program in the BGFX example that samples from a texture in both the vertex and fragment shaders, so I've added a modified version of 27-terrain that does so.
Steps to reproduce the behavior:

Apply the patch to 27-terrain
Compile the example.
Compile the shader for Metal.
Run the example with Metal API Validation enabled (steps to enable are below)
Choose the last option in the ImGui window to switch to heightmap texture mode.

Expected behavior
The example should continue to run as normal, but with the terrain textured with the new pattern texture.
Observed behaviour
The program aborts with a Metal API Validation error:
validateFunctionArguments:3531: failed assertion `Fragment Function(xlatMtlMain): missing sampler binding at index 1 for s_patternTextureSampler[0].'

Additional context
Why this is important

If Metal API Validation fails, you can't capture a frame in the Metal graphics debugger.
I've heard it also causes Apple to reject apps for the App Store.

To run the examples with Metal API Validation enabled
These instructions are for XCode 11.3.

Start a new empty XCode project
Product -> Scheme -> Edit Scheme...
Choose "Run" on the left
Choose "Info" tab on the right

Under "Executable", choose "Other" and then choose the bgfx examples.app

Choose "Options" tab on the right

Set "Metal API Validation" to "Enabled"
Set "Working Directory" to bgfx/examples/runtime

Close the scheme editor
Product -> Run
The examples app should run

Test Case
You can find the test case on my branch here: https://github.com/rogual/bgfx/commit/6ace366de3a0d2eb7ce109869d01af61313fa3e8
The diff is here:
diff --git a/examples/27-terrain/fs_terrain.sc b/examples/27-terrain/fs_terrain.sc
index aff04bea0..022fb9c27 100644
--- a/examples/27-terrain/fs_terrain.sc
+++ b/examples/27-terrain/fs_terrain.sc
@@ -7,7 +7,9 @@ $input v_position, v_texcoord0

 #include "../common/common.sh"

+SAMPLER2D(s_patternTexture, 1);
+
 void main()
 {
-	gl_FragColor = vec4(v_texcoord0.x, v_texcoord0.y, v_position.y / 50.0, 1.0);
-}
+	gl_FragColor = texture2D(s_patternTexture, v_texcoord0.xy);
+}
\ No newline at end of file
diff --git a/examples/27-terrain/terrain.cpp b/examples/27-terrain/terrain.cpp
index 3b493304a..2ef3a5a1c 100644
--- a/examples/27-terrain/terrain.cpp
+++ b/examples/27-terrain/terrain.cpp
@@ -113,7 +113,9 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)
 		m_dvbh.idx = bgfx::kInvalidHandle;
 		m_dibh.idx = bgfx::kInvalidHandle;
 		m_heightTexture.idx = bgfx::kInvalidHandle;
+		m_patternTexture.idx = bgfx::kInvalidHandle;
 		s_heightTexture = bgfx::createUniform("s_heightTexture", bgfx::UniformType::Sampler);
+		s_patternTexture = bgfx::createUniform("s_patternTexture", bgfx::UniformType::Sampler);

 		m_oldWidth  = 0;
 		m_oldHeight = 0;
@@ -167,12 +169,18 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)
 		}

 		bgfx::destroy(s_heightTexture);
+		bgfx::destroy(s_patternTexture);

 		if (bgfx::isValid(m_heightTexture) )
 		{
 			bgfx::destroy(m_heightTexture);
 		}

+		if (bgfx::isValid(m_patternTexture) )
+		{
+			bgfx::destroy(m_patternTexture);
+		}
+
 		bgfx::destroy(m_terrainProgram);
 		bgfx::destroy(m_terrainHeightTextureProgram);

@@ -289,6 +297,16 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)
 				m_heightTexture = bgfx::createTexture2D(s_terrainSize, s_terrainSize, false, 1, bgfx::TextureFormat::R8);
 			}

+			if (!bgfx::isValid(m_patternTexture) )
+			{
+				m_patternTexture = bgfx::createTexture2D(2, 2, false, 1, bgfx::TextureFormat::RGBA8);
+				static unsigned char mem[] = {
+					0,0,0,255,        255,255,255,255,
+					255,255,255,255,  0,0,0,255
+				};
+				bgfx::updateTexture2D(m_patternTexture, 0, 0, 0, 0, 2, 2, bgfx::makeRef(mem, 2 * 2 * 4));
+			}
+
 			mem = bgfx::makeRef(&m_terrain.m_heightMap[0], sizeof(uint8_t) * s_terrainSize * s_terrainSize);
 			bgfx::updateTexture2D(m_heightTexture, 0, 0, 0, 0, s_terrainSize, s_terrainSize, mem);
 			break;
@@ -478,6 +496,7 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)
 				bgfx::setVertexBuffer(0, m_vbh);
 				bgfx::setIndexBuffer(m_ibh);
 				bgfx::setTexture(0, s_heightTexture, m_heightTexture);
+				bgfx::setTexture(1, s_patternTexture, m_patternTexture);
 				bgfx::submit(0, m_terrainHeightTextureProgram);
 				break;
 			}
@@ -499,7 +518,9 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)
 	bgfx::ProgramHandle m_terrainProgram;
 	bgfx::ProgramHandle m_terrainHeightTextureProgram;
 	bgfx::UniformHandle s_heightTexture;
+	bgfx::UniformHandle s_patternTexture;
 	bgfx::TextureHandle m_heightTexture;
+	bgfx::TextureHandle m_patternTexture;

 	float m_viewMtx[16];
 	float m_projMtx[16];

@attilaz seems to get CC'd on every Metal issue, so, cc: @attilaz :)

Update: If I modify the example so that mode 2 is the default, instead of having to change to it with the ImGui controls, the bug doesn't manifest — but then, if you change to one of the other modes, and therefore stop sampling both textures, the bug triggers.
diff --git a/examples/27-terrain/terrain.cpp b/examples/27-terrain/terrain.cpp
index 2ef3a5a1c..854ed9825 100644
--- a/examples/27-terrain/terrain.cpp
+++ b/examples/27-terrain/terrain.cpp
@@ -127,7 +127,7 @@ ExampleTerrain(const char* _name, const char* _description, const char* _url)

                uint32_t num = s_terrainSize * s_terrainSize;

-               m_terrain.m_mode      = 0;
+               m_terrain.m_mode      = 2;
                m_terrain.m_dirty     = true;
                m_terrain.m_vertices  = (PosTexCoord0Vertex*)BX_ALLOC(entry::getAllocator(), num * sizeof(PosTexCoord0Vertex) );
                m_terrain.m_indices   = (uint16_t*)BX_ALLOC(entry::getAllocator(), num * sizeof(uint16_t) * 6);

Sorry, I keep making stupid mistakes editing this. Now it's working. There is an issue here somewhere, but I'll reopen it when I can make a proper issue report.

You should not open issues if you're not sure you have actual issue... Use Discussions or Discord instead.

I was sure! I was just wrong 🙁

Alright, I've fixed my test case. It's when vertex and fragment shaders sample from specifically the same texture.
I won't reopen until I've confirmed on Discord that this is a proper bug.

Might be a similar issue to https://github.com/bkaradzic/bgfx/issues/2410 (which is fixed btw, that issue should be closed)

OK, it sounds like this is a bug then. Reopening.

I am encountering the same issue. Has there been any more discussion on this issue?


Req.params and req.swagger.params, need to duplicate code in controller?

Hello,
i am trying to use swagger using swagger-tools in a little test project but i have some difficults to use it with express.
When i make a request with curl, in my controller i have to use req.params but when i use swagger, i have to use req.swagger.params.
Is there a way to uniform this like using only req.swagger.params for both or req.params?
If not, do i have to check myself if req.swagger exists ... and duplicate code ?
Sorry for bad english.

No.  req.params is populated by whatever web framework you're using.  req.swagger.params is populated by swagger-tools and it contains a lot more detail and type-coerced values.

No. req.params is populated by whatever web framework you're using. req.swagger.params is populated by swagger-tools and it contains a lot more detail and type-coerced values.

Thank you for yout reply.
I'm agree with your response but my problem is:
- req.swagger.params only is filled when executing the request from swagger-ui

req.params only is filled when executing the request from CURL

In other posts of swagger-express-mv i saw whe have to add :
- swagger_params_parser in a yaml config file to get req.swagger.params filled when calling with CURL or  Swagger-ui
Is there a way  to use swagger_params_parser with swagger-tools ?
Is there a way to use a config file like in swagger-express-mv SwaggerExpress.create(config, function(err, swaggerExpress) {... ?
Is there another thing to do to get req.swagger.params filled with both Curl call or Swagger-ui?

swagger-ui has nothing to do with swagger-tools.  If req.swagger is not populated, the swagger-metadata middleware did not match the request to a Swagger operation.  It might make sense to turn on debugging to see the request come in, and the swagger-tools debugging output to verify.  (My hunch is if swagger-ui works but direct API calls don't, it's a basePath issue.)

You are all right. That was a basePath issue. Thank you so much.


Adding arrows and labels to links

Hi, is it possible to add arrows and labels to links?
example:

In theory yes -  you have the possibility to modify path (link) svg element in onNodeUpdate and onLinkUpdate method.

@bumbeishvili dou you have any example where the Arrow and Label is available on links. If yes, please share link of it.
Thanks!

Hi @bumbeishvili, I can see the [defs](https://github.com/bumbeishvili/org-chart/blob/cb1f376656bc83360a4fea6872597fd427fcd775/src/d3-org-chart.js#L264) function which is setting the marker for the connections, but not sure how to enable it? can you help me with that?


Stop Octoprint from logging errors during startup

Description:
Switched to discovery.load_platform however the following Traceback is now occurring. Even though this Traceback is thrown all of the sensors still get setup. @balloob
Traceback (most recent call last):
  File "/home/pi/.virtualenvs/hass_dev/lib/python3.4/site-packages/homeassistant-0.32.0.dev0-py3.4.egg/homeassistant/helpers/entity_component.py", line 148, in _async_setup_platform
    entity_platform.add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/pi/.virtualenvs/hass_dev/lib/python3.4/site-packages/homeassistant-0.32.0.dev0-py3.4.egg/homeassistant/components/sensor/octoprint.py", line 49, in setup_platform
    for octo_type in monitored_conditions:
TypeError: 'NoneType' object is not iterable

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

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

You'll need to delete the constants that you're no longer using: DISCOVER_BINARY_SENSORS, DISCOVER_SENSORS.
I'm surprised it was even working before as I can't see those constants to be defined?

The exception makes sense. The platform is now setup with discovery_info which you set to an empty dict. Passing in the config is only used for setting up the component if it was not set up.

@balloob I figured it out, when I added octoprint it was never designed to to work with discovery, you have to manually add which sensors you want to the config. So... I am assuming the way it works was discovery was trying to setup the components and failed, but then the hard coded sensors in the config got hit and setup successfully? I removed discovery altogether and now it is working without any errors. I think it would probably be possible to make discovery work, but since that isn't how it was originally developed I think that should be an enhancement at a later date.

OK 🐬


Cluster_loss and Datasets

Hi @ada-shen , thanks for sharing the code! I am very interested in your algorithm and I tried the code.
But I have the following two questions and I hope you could help:
(1) Is Cluster Loss (Loss(θ, A) in the paper) negative and should it be in the range [-1, 0)? I tried the code and it was smaller than -1. I thought the value of numerator should be <= denominator.
(2) Could you share more details about how you preprocess the dataset? For instance, I am now try the code CelebA, but I am not sure how you cropped your image? So I just resized the input to (224,224).
It would be great if you can add more instructions of different datasets.
Thank you in advance!

Hi @ada-shen. I am unable to run the code for any dataset. There seems to be a "cub_file" and "voc_file" variable in the vgg_iccnn_train.py file. What file path do I need to provide here if I want to run the code for training?

Hi @yaorong0921. Could you please share some details as to how you were able to run the code?

We will release a tutorial to show how to train the iccnn from scratch, and we will provide the data we use for training and testing.


Pin babylon 6.8.0

@alexlande @ianobermiller It looks like babel-eslint is having some trouble with babylon 6.8.1 and pinning to 6.8.0 fixes the build issue in #727 for now.

:bow: thanks! will merge this as soon as it goes green.

Coverage remained the same at 96.236% when pulling e5b10a8876e3827ec68f12df620bdc9474cabf01 on jmcriffey:bug-pinBabylon into e9429037aecf98cbfdb38cd26c83ca818665cdca on FormidableLabs:master.

@jmcriffey: actually, one thing I'm not sure about. Should babylon be added to our move-babel-to-dependencies script? We've got a script that moves babel packages to dev dependencies on npm publish: https://github.com/FormidableLabs/radium/blob/master/npm-scripts/move-babel-to-dependencies.js#L6-L10
Presumably babylon should be included there too.

Good catch @alexlande. Looking at it closer, I believe I can just pin it in devDependencies since it is an issue with eslint.

Coverage remained the same at 96.236% when pulling 0fcf86db7a6177c68d9cbee1c45442b4f85f6846 on jmcriffey:bug-pinBabylon into e9429037aecf98cbfdb38cd26c83ca818665cdca on FormidableLabs:master.

LGTM, thanks!


[5.0.2]: const-like inference is not preserved in conditionally requirable intersected object

Bug Report
🔎 Search Terms
const-like inference, const type parameter, conditionally require
🕗 Version & Regression Information
5.0.2

I was unable to test this on prior versions because const type parameters are a new feature

⏯ Playground Link
Playground link with relevant code
💻 Code

type StringWithSpace = `${string} ${string}`;
type FooBar<Foo extends string, Bar extends Record<string, string>> = {
  foo: Foo;
} & (
  Foo extends StringWithSpace // 💡if changing to simply `Foo extends string`, const-like inference is used
  ? { bar: Bar }
  : { bar?: never }
);
//                                                            ⬇ use const like inference
declare function requireBarIfFooHasSpace<Foo extends string, const Bar extends Record<string, string>>(foobar: FooBar<Foo, Bar>): Bar;
const foobar = requireBarIfFooHasSpace({
  foo: 'has space',
  bar: {
    hello: 'world'
  }
});
type inferredFooBar = typeof foobar;
//    ^? type inferredFooBar = {␊hello: string;␊}

🙁 Actual behavior
Using an extends check to conditionally intersect an object to make a property required or unallowed does or does not preserve const-like inference, depending on the complexity of that check.
If it is a simpler check, such as extending a primitive, const-like inference is used. If it is a check extending a template literal, the inference is lost.
🙂 Expected behavior
The complexity of the check to conditionally intersect the object should not determine whether const-like inference is used.
Potentially related to https://github.com/microsoft/TypeScript/issues/53307

This is working as intended. Inference for Foo and Bar happens at the same time, so when determining the contextual type for { hello: 'world' } it is not yet known that the inferred type for Foo will be 'has space'. Therefore, evaluation of the conditional type is based on the constraint of Foo, which is string. Since string doesn't extend StringWithSpace, the resulting type is { bar?: never }. This type doesn't reference a const type parameter, thus no const behavior.

Thank you for the clarification!


Problem with 'select' event in piecharts

Google charts dispatch the error when some piechart is clicked.
core.es5.js:1084 ERROR Error: Invalid column index null. Should be an integer in the range [0-1].
Checking the code source, the directive uses select event to dispatch emit event, okay! But, inside of method:
            google.visualization.events.addListener(self.wrapper, 'select', function () {
                          ...
                        if (selectedItem.row !== null) {
                            selectedRowValues.push(self.wrapper.getDataTable().getValue(selectedItem.row, 0));
                            selectedRowValues.push(self.wrapper.getDataTable().getValue(selectedItem.row, selectedItem.column));
                          ...
            });

That error is happening because the piechart does not have column property and always is null and dispatching the error.

The problem will be fixed in my pull request:
https://github.com/vimalavinisha/angular2-google-chart/pull/47/files#diff-3d20a0f02912cfb1634a3096ed238530R56

This same error is impacting my application. I can verify that the fix above does work. Can this pull request (or maybe one with just that one line change) be merged and a new version published?
Thanks

@matthewdickinson the repo author needs accept my PR :/


carry_supplies => check_answers

Previous path: dietary_requirements

my new hobby => racing bill to review PRs
... and losing.

🏎


Warn against redefining global variables  in the code editor

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no): Closes #7270

Description

Adds window prop names to a predef list used for codemirror linting.
Linter warns that you are 'redefining ${variableName}'

Code LGTM.

cc @freeCodeCamp/moderators

@Bouncey Can you update this to resolve the merge conflict?

merged!!!!!
https://github.com/freeCodeCamp/freeCodeCamp/commit/1126bf7004ec6cabfeb0641eefc823175d33c4ba
🍺 🍻
Happy Coding!


3D Model Disappears After Zooming Beyond a Certain Level in Android

Describe the bug
When using the 1.53.4 version of the Filament library for rendering 3D models on Android, the model loads and displays fine under normal conditions. However, when zooming in beyond a certain point, parts of the model—or sometimes the entire model—disappear.
To Reproduce
Steps to reproduce the behavior:

Load a 3D model using Filament on an Android device.
Zoom in on the model using pinch-to-zoom.
After a certain zoom level, parts or the entire model disappear from view.

Expected behavior
The 3D model should remain fully visible and not disappear regardless of the zoom level.
Screenshots
I have attached a video that demonstrates the issue for better clarity. - Video Link
Logs
There are no relevant logs for this issue.
Desktop (please complete the following information):

OS: N/A (Android issue)
GPU: N/A
Backend: N/A

Smartphone (please complete the following information):

Device: All devices
OS: Android (all versions)

Additional context
The issue is reproducible across all Android devices. Please refer to the attached video for further context.

It does look like your object's bounding boxes are incorrect. Please check, and reopen this bug if you're sure they are correct.


Compatibility with macOS Mojave

Really appreciate the work you do, awesome screensaver.  Doesn't seem to work with Mojave, any plans on making an update?  And 4K?

You cannot use the Aerial screen saver with this version of MacOS. Please contact the vendor to get a newer version of the screen saver.

Desktop & Screen Saver

Duplicate of #464

Closing as duplicate of #464


error : not working on windows 8.1

Hello,
I'm unable to make it work. I've got an error on the first launch.

An unhandled exception occurred! The app has attempted to recover and carry on, but you may experience some weirdness. Report error?.

I can't create a new project neither.

I just pushed out a new build that fixes a load of bugs, and also shows more information in the exception message. Would you mind trying that and see if it fixes your issue? And if it doesnt could you tell me what the exception message says.

An unhandled exception occurred!
Méthode introuvable : '!!0[] System.Array.Empty()'.
The app has attempted to recover and carry on, but you may experience some weirdness. Report error?

That means you need to install .Net 4.6. Why thats the error it reports I do not know. But the solution should be installing .Net 4.6

I confirm it's working now.


nixos/fail2ban: improve module documentation

Description of changes
The fail2ban documentation was not clear in some area

default backend is journald, the apache example can't work because it's using a logpath, you need backend = auto to not use the default journald
adds a simple example for dovecot
mentions iptables-ipset-proto6-allports ban action which supports IPv4 and IPv6 for blocking IP on all ports

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1227


Build(deps-dev): Bump @typescript-eslint/parser from 5.25.0 to 5.26.0 (#3532)

Pull request opened by the merge tool on behalf of #3532

Merged in 04b2e493edec531896133a9f2178d06f91b5a479


Why does MajoranaOperator not subclass SymbolicOperator?

It seems odd that MajoranaOperator is not a sub-class of SymbolicOperator. A notable side effect is that contrary to QubitOperator and other SymbolicOperators, MajoranaOperator does not auto-remove terms that are below EQ_TOLERANCE. Not the end of the world but a strange inconsistency...

SymbolicOperator stores (index, action) pairs. For MajoranaOperator, there is no action, so it does not quite map onto the SymbolicOperator class.


Feature: added way to specify a custom database connection

I had the necessity to move this incredible dataset on a different database to improve performance on the main DB, so I decided to contribute directly on the package to give the possibility to do it.
This new feature doesn't introduce any breaking changes since, by default, it works with the default connection.
To customize the database connection it's just needed to add a new variable in the .env file name WORLD_DB_CONNECTION.
The only issue is that to make it work properly, the user must specify the connection when seeding via the --database option, I came with a solution to this problem: add a world:install command I already wrote (but not pushed yet) which I'll better explain in a comment on this PR.

As said in the PR description, I felt the urge to make the installation process as simple as possible, so I added a world:install command that I think should come out in pair with this feature.
This new command corresponds to these ones:
// publish migrations
Artisan::call('vendor:publish --tag=world');
// migrate new tables
Artisan::call('migrate');
// re-seed the world data
Artisan::call('db:seed --class=WorldSeeder --database=' . config('world.connection'), array(), $this->getOutput());

By calling this command, the user should never worry about manually selecting the database when seeding.
I'm even thinking to to add the --database option to the migration too, so only the selected connection will be migrated, without messing with other user migrations

@rinodrummer thank you for this major contribution.
this addition makes sense, however please help in formatting the code. i have noticed some wrong code formatting. please help on fixing this. i would suggest that you push the World:install helper i would like to pull the branch and test it locally before merging it.
thank you

@nnjeim I committed the world:install command!
I take the opportunity with this comment to ask you an opinion on adding the --database option to the php artisan migrate command too, so only the selected connection will be migrated, without messing with other user migrations.


Cow: Watch mcc-mnc.com

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

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

Approved by tripleee in Charcoal HQ


fix: tidyup of mm_getauthinfo.sas

:tada: This PR is included in version 4.9.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


concourse.suse.dev: Investigate why concourse-worker statefulset couldn't connect

Is your feature request related to a problem? Please describe.
Problem:
concourse-worker statefulset on ns concourse was trying to reach TSA at the service concourse-web (which doesn't have an sshd listening in tcp 2222), instead of trying reach concourse-web-worker-gateway:2222, which does have an sshd listening.
We fixed it by editing the concourse-worker statefulset, and pointing the env CONCOURSE_TSA_HOST from concourse-web:2222 to concourse-web-worker-gateway:2222.
This may have happened because when upgrading the helm chart to recreate the workers, I mistakenly forgot to helm update, therefore I did a downgrade from 6.4.0 to 6.0.0, which failed with:
$ helm upgrade --namespace concourse concourse concourse/concourse --values values.yaml
UPGRADE FAILED
Error: Service "concourse-web" is invalid: spec.ports[0].nodePort: Forbidden: may not be used when `type` is 'ClusterIP'
Error: UPGRADE FAILED: Service "concourse-web" is invalid: spec.ports[0].nodePort: Forbidden: may not be used when `type` is 'ClusterIP'

And then an upgrade from 6.0.0 to 6.4.1, which suceeded.
Describe the solution you'd like
We need to investigate why this has happened, as we don't want it to happen again if we helm upgrade.
Describe alternatives you've considered
Uninstall the concourse chart and reinstall cleanly, redeploying the pipelines and losing history.
Additional context
Add any other context or screenshots about the feature request here.

@viccuad I am closing 'cause we all agree to move quickly to GHA and leave Concourse.


Changelog v1.15.9

BUGFIXES

Revert "Fix delete u2f keys bug (#18042)" (#18107)
Migrating wiki don't require token, so we should move it out of the require form (#17645) (#18104)
Prevent NPE if gitea uploader fails to open url (#18080) (#18101)
Reset locale on login (#17734) (#18100)
Correctly handle failed migrations (#17575) (#18099)
Instead of using routerCtx just escape the url before routing (#18086) (#18098)
Quote references to the user table in consistency checks (#18072) (#18073)
Add NotFound handler (#18062) (#18067)
Ensure that git repository is closed before transfer (#18049) (#18057)
Use common sessioner for API and web routes (#18114)

TRANSLATION

Fix code search result hint on zh-CN (#18053)

Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Awaiting merge of #18114

BTW: could we take another backport in?

https://github.com/go-gitea/gitea/pull/18095

Wait #18121's backport.

Wait #18121's backport.

Sorry, I need to do more test for #18121. So please ignore that.

make L-G-T-M work