Discussion:  Data sharing and persistence

We are at some point going to run into a need to persist data and share it between users on this project from.  Using APIs to existing services has been serving us well so far, but I think this will end up holding us back.
To ground this in the project I quote @nrobinson2000 in the conversation for PR #27:

@frattaro A voting system should be the next major feature. Just not sure of the best way to implement it.

In PR #13 @frattaro has used Google Plus since it's licence doesn't forbid using +ing as a voting mechanism.  I'm not a fan as it forces people to use Google sign in.   I would accept a GitHub-focused variant if someone can think of one, as we it's pretty central to the project that we use it already, but I can't find a ready way to star or otherwise show appreciation for PRs that we could hook a buttons.github.io button into (generally people seem to use reactions to the top-level comment in the PR conversation to show approval, so that might be an avenue to look into).
This is an ideal case for starting up our own database-type-thing.  We could use a cloud DBMS like Azure or one of Amazon's multitude of PaaS offerings.  However out of a desire to learn it I've stated a Firebase project for my fork dominicbisset/1pr@ca2649a
My arguement for firebase is primarily becuase it is JSON-based rather than table-based.  This should be much easier to migrate every time the project is altered because we're just altering properties which we can do in Javascript, rather than having to script ALTER statements for @benhalpern to run.
As proof of concept for this I've started working on a simple counter thing on my fork, that will eventually morph into a voting mechanism.  If you visit https://dominicbisset.github.io/1pr/ it'll ask you to allow popups (which is a faff and I'm going to change) and to sign in to GitHub. There's also a bug that requires you to refresh the page after you've signed in before the counter starts updating properly, so it has a tendency to reset every time somone logs in.  Despite that, it does in theory count every time someone presses it and broadcasts updates to everyone who happens to be viewing the page.
That's a bit of a wall of text, but I'm mainly trying to stimulate discussion before I waste too much time on it.  Does anyone have a better idea for persistence?

@DominicBisset It looks like @frattaro has left the project.  He closed his pull request and deleted his fork.

To quote @frattaro in #13:

tabs are too frustrating to work with. withdrawing this PR.

I haven't worked with Firebase yet, but it sure seems like a viable option to get started. I agree a JSON-type datastore would be best, considering we're working with Javascript.
I've actually been working on a solution for the voting system that uses only the Github API, tied into the reactions on the top comment like you mentioned @DominicBisset. I'm not 100% sure it will work that way though, so I'll need to do some more testing. If it works, this would create a more integrated solution than using an external database for the votes.
By the way, for my Github oAuth implementation I'm using Auth0, which seems like a more "glamorous" option than your popup. Might be interesting to check it out.
As for the tabs thing @nrobinson2000, I don't wanna derail this issue but if there are more people who have a problem with tabs it might be a good idea to open a separate issue to discuss it?

I also haven't worked with Firebase myself, but based on my familiarity with it, I agree that it seems like the right choice for the issues outlined.

I've now done this in PR #42, so I'm closing this issue.


bbs2gh inventory-report command

Closes #955
Adds the inventory-report command for bbs2gh migrations. The command iterates over projects/repos that the user can access, and generates CSV files (Projects.csv and Repos.csv) listing the assets.
Todo

[x] InventoryReportCommand creates the BBS API client and builds the InventoryReportCommandHandler
[x] InventoryReportCommandArgs specifies these args:

BbsServerUrl
BbsProject - limit the output to this one project
BbsUsername, BbsPassword, and NoSslVerify - make BBS API calls
Minimal - option to generate a minimal report

[x] Validations for required args
[x] InventoryReportCommandHandler generates the inventory report and writes progress logs
[x] Add these methods to BbsApi:

[x] GetRepositoryLatestCommit

Get commits API — defaults to tip of default branch, so we can say limit=1 to only get the newest commit

[x] GetRepositoryPullRequests

Get PRs for repo — will require BbsApi to paginate to get all PRs
response

[x] GetRepositorySize
[x] GetIsRepositoryArchived - This value will not return when fetching all repos. Will need to fetch each repo and query for the archived value. 😢  Supported in BBS v6.0.0 or greater
[x] GetProject - To get the project name if BbsProject was specified

[x] CreateBbsInspectorService with helper methods:

[x] GetProjects
[x] GetProject
[x] GetRepos for a project
[x] GetRepoCount for a project
[x] GetRepoCount for all projects
[x] GetPullRequestCount for a project
[x] GetRepositoryPullRequestCount
[x] GetLastCommitDate for a repo
[x] GetRepositoryAndAttachmentsSize

[x] ProjectsCsvGeneratorService iterates through all projects (or just --bbs-project if specified) and fetches:

[x] Project name
[x] Project URL
[x] Repo count
[x] PR count

[x] ReposCsvGeneratorService iterates through all repos in all projects (or just --bbs-project if specified), and for each repo fetches:

[x] Repo name
[x] Project key
[x] Is repo archived
[x] PR count
[x] Last commit date
[x] Repo size

[x] Write tests
[x] Update RELEASENOTES.md

[ ] Did you write/update appropriate tests
[ ] Release notes updated (if appropriate)
[ ] Appropriate logging output
[ ] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

Functional Tests
The repository archived field was introduced in BBS v6.0.0, so running the bbs2gh inventory-command on earlier versions will exclude the archived state from repos.csv.
Here are the functional test results using our BBS test instance, running v5.14.4:
All projects
src/bbs2gh/bin/Debug/net6.0/bbs2gh inventory-report \
  --bbs-server-url https://test-bbs-o.githubapp.com \
  --bbs-username gohana \
  --bbs-password $BBS_PW

projects.csv

repos.csv

With --minimal flag
projects.csv

repos.csv

Specifying a single project
src/bbs2gh/bin/Debug/net6.0/bbs2gh inventory-report \
  --bbs-server-url https://test-bbs-o.githubapp.com \
  --bbs-project GOH \
  --bbs-username gohana \
  --bbs-password $BBS_PW

projects.csv

repos.csv

I spun up a BBS instance running v8.6.1 to test the command, and saw that the repo archived field is included in the repos CSV.
src/bbs2gh/bin/Debug/net6.0/bbs2gh inventory-report \
  --bbs-server-url https://gohana-bitbucket.expert-services.io \
  --bbs-username gohana \
  --bbs-password $BBS_PW \
  --no-ssl-verify

projects.csv

repos.csv

If a repo's latest commit date can't be fetched, the repos CSV will have an empty entry like this:


Avoid mutating dockerode, and add a few basic unit tests

This module used to promisify and mutate the Docker class from dockerode, which is bad.
It also mutated images and containers by promisifying their prototype.
Now we extend that class and only promisify and add methods to the extension and to images/containers
created by the extension.
We also add mocha (with mochainon) and some basic tests that only cover the basic promisification.
No tests for the docker-toolbelt specific functions were added yet.
Change-Type: patch
Signed-off-by: Pablo Carranza Velez<EMAIL_ADDRESS>

Dockerode natively supports promises since v2.3.3 -- might make sense to use it and not rely on promisification.

@dfunckt the thing is we have a bunch of programs that rely on the Async suffix for everything, and changing all of those would take quite a bit of work. I agree we should use dockerode's Promise support eventually but we need to have the time to change everything else (most of the supervisor, for instance). I'd rather first solve this mutation problem and then address using dockerode's Promise support in a separate major-version-bump PR

Agreed, I forgot this would mean method names would change 👍

and actually, users of this library can also use dockerode's promise support as-is (i.e. just not using Async), so users can use whichever they prefer :)

@pcarranzav "Avoid mutating docker...except for adding some *Async methods" :P
Besides that, it might be worthwhile making this a major version and relying on dockerode's promises - since there might be places that (unintentionally) rely on the mutation we do and be broken without it, so we can kill two birds with one stone

@Page- we really don't mutate Docker here. i.e. if someone requires dockerode and instantiates an docker object it will not have any of the Async methods (which happened before this PR).
But okay, yeah, let's go major. I'll commit soon.

Ah, you're right you did also remove that mutation, but instead you have to promisify every time you do getContainer or getImage which isn't ideal - going with the major bump I think is overall best

@Page- @dfunckt changed to use dockerode's promise support


Resource not found for the segment (...)

Trying to use this library for the first time, but I always get Resource not found for the segment (...). no matter what I try. It's probably something obvious I am missing here.
Following the coding flow example in the readme. $provider->get("me", $token); works, but everything else gives me the following error.
Fatal error:
Uncaught League\OAuth2\Client\Provider\Exception\IdentityProviderException: Resource 'calendars' does not exist or one of its queried reference-property objects are not present. in /vendor/thenetworg/oauth2-azure/src/Provider/Azure.php:315
Stack trace:
#0 /vendor/league/oauth2-client/src/Provider/AbstractProvider.php(628): TheNetworg\OAuth2\Client\Provider\Azure->checkResponse(Object(GuzzleHttp\Psr7\Response), Array)
#1 /vendor/thenetworg/oauth2-azure/src/Provider/Azure.php(173): League\OAuth2\Client\Provider\AbstractProvider->getParsedResponse(Object(GuzzleHttp\Psr7\Request))
#2 /vendor/thenetworg/oauth2-azure/src/Provider/Azure.php(104): TheNetworg\OAuth2\Client\Provider\Azure->request('get', 'd8aa4b66-c36f-4...', Object(TheNetworg\OAuth2\Client\Token\AccessToken), Array)
#3 (...)/test.php(64): TheNetworg\OAuth2\Client\Provider\Azure->get('/me/calendars', Object(TheNetworg\OAuth2\Client\Token\AccessToken))
#4 {main}
thrown in /vendor/thenetworg/oauth2-azure/src/Provider/Azure.php on line 315

Trying to access the calendar API at https://docs.microsoft.com/en-us/graph/api/user-list-calendars?view=graph-rest-1.0 but can't get any request paths to work. Tried a variety of different URL combinations, relative/absolute paths as well as full URLs.
Tried both https://graph.windows.net and https://graph.microsoft.com as access token resources.
Any suggestions?

Seems like I forgot these two lines:
$provider->urlAPI = "https://graph.microsoft.com/v1.0/";
$provider->resource = "https://graph.microsoft.com/";


🧑💻 move build command to the start of initialization process

I suggest moving the "cargo build-bpf" command to the start of the initialization process for trdelnik tests. This is not a critical hotfix, considering other major pull requests are already open; I view this as a minor improvement.
Currently, during initialization, "cargo build-bpf" is executed as one of the last commands. If the build command fails, the project folder will already contain generated files and folders, which I consider to be undesirable behavior from the user's perspective.

Resolved in #129


Update build.gradle

Updated the buildToolsVersion & the targetSdkVersion for compatibility with React Native 0.56.0

@vladrozhnev squashed and merged (a0d4c863a15e8b69fad1b427cd3d80749e3c5899), thank you


Fix spatial_transformer_sampler to behave consistently among cuDNN and non-cuDNN mode

This PR follows a discussion in #2458.
The current implementation of spatial_transformer_sampler in cuDNN mode and non-cuDNN mode are different.
This happens in the case when grid is outside the range [-1, 1].
In the implementation of cuDNN, points are sampled from a zero padded image when this happens.
However, in the current implementation of non cuDNN mode, grid is clipped to range [-1, 1].
Since creating difference between cuDNN and non-cuDNN was not intended, I would like to fix it with this PR.
This also fixes a problem that the documentation is not reflecting the implementation of cuDNN.
The tests became less stable so the tolerance for error is loosened even for cuDNN implementation when grid is outside the range [-1, 1].
Also, I added a test to verify that cuDNN and non cuDNN implementation produce almost same backward and forward passes.

It is a little bit problematic that implementation changes from the already merged one.
However, the original spatial_transformer_sampler has not been included in any release yet as it was merged after the latest version (1.23) was released.

LGTM!


Move enrichments to Akka

Essentially use Akka as a module system (instead of e.g. OSGI).

http://benninger.ca/posts/creating-an-akka-actor-from-a-dynamically-loaded-jar-file-in-scala/
https://gist.github.com/earldouglas/877999

Initially enrichments will still be blocking and sequential. We can parallelize them later.

Came to this conclusion (and googled same links) while thinking about Sauna extension mechanisms.

Cool!

Wontfix.


Initial dry-run in docker produces permission error, root owns ~/dependabot-core/tmp

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Docker
Package manager version
No response
Language version
No response
Manifest location and content before the Dependabot update
No response
dependabot.yml content
No response
Updated dependency
No response
What you expected to see, versus what you actually saw
I am attempting to follow the Setup instructions from README.md and encounter a permission error within the Docker container environment.
The tmp/ and dry-run/ directories do not exist after checking out the repo. They seem to be created by the docker-dev-shell script. Both are created with root:root ownership instead of dependabot:dependabot ownership within the container or myuser:mygroup outside the container.
Changing permissions on the directory within or without the container, or ownership within the container, works around this problem.
Versions/hashes:
Docker 20.10.17
Amazon Linux 2.0.20221210.0
dependabot-core a5e540517b8a5c7e2356ae538e9a4597098479a3
~/src/dependabot/dependabot-core.git $ bin/docker-dev-shell
 > image dependabot/dependabot-core-development already exists

=> running docker development shell

[dependabot-core-dev] ~/dependabot-core $ bin/dry-run.rb go_modules rsc/quote
/usr/local/lib/ruby/3.1.0/fileutils.rb:243:in `mkdir': Permission denied @ dir_s_mkdir - /home/dependabot/dependabot-core/tmp/rsc (Errno::EACCES)
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:243:in `fu_mkdir'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:221:in `block (2 levels) in mkdir_p'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:219:in `reverse_each'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:219:in `block in mkdir_p'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:211:in `each'
        from /usr/local/lib/ruby/3.1.0/fileutils.rb:211:in `mkdir_p'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:591:in `block in _clone_repo_contents'
        from /home/dependabot/dependabot-core/common/lib/dependabot/shared_helpers.rb:168:in `with_git_configured'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:585:in `_clone_repo_contents'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:94:in `clone_repo_contents'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:131:in `load_cloned_file_if_present'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:152:in `fetch_file_from_host'
        from /home/dependabot/dependabot-core/common/lib/dependabot/config/file_fetcher.rb:32:in `block in fetch_files'
        from /home/dependabot/dependabot-core/common/lib/dependabot/config/file_fetcher.rb:28:in `each'
        from /home/dependabot/dependabot-core/common/lib/dependabot/config/file_fetcher.rb:28:in `fetch_files'
        from /home/dependabot/dependabot-core/common/lib/dependabot/file_fetchers/base.rb:75:in `files'
        from /home/dependabot/dependabot-core/common/lib/dependabot/config/file_fetcher.rb:20:in `config_file'
        from bin/dry-run.rb:510:in `<main>'

[dependabot-core-dev] ~/dependabot-core $ ls -ld ~/dependabot-core/tmp/
drwxr-xr-x 2 root root 4096 Jan 17 19:54 /home/dependabot/dependabot-core/tmp/

[dependabot-core-dev] ~/dependabot-core $ whoami
dependabot

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Did you try rebuilding?
$ bin/docker-dev-shell --rebuild

I have removed the two directories owned by root, run the rebuild, and the directories are re-created, still owned by root.

Does this fix things?
diff --git a/bin/docker-dev-shell b/bin/docker-dev-shell
index d5b5e391f..90ceda6d4 100755
--- a/bin/docker-dev-shell
+++ b/bin/docker-dev-shell
@@ -101,6 +101,7 @@ fi
 echo "$(tput setaf 2)=> running docker development shell$(tput sgr0)"
 CODE_DIR="/home/dependabot/dependabot-core"
 touch .core-bash_history
+mkdir -p dry-run tmp
 docker run --rm -ti \
   -v "$(pwd)/.core-bash_history:/home/dependabot/.bash_history" \
   -v "$(pwd)/.rubocop.yml:$CODE_DIR/.rubocop.yml" \

Yes, that appears to resolve the issue.


Golang

📖 Abstract:
In-Depth Exploration of Golang: Principles, Practices, and Optimization.

Slice

https://github.com/golang/go/blob/master/src/runtime/slice.go#L15

Related articles

[ ] 你真的了解 go 语言中的切片吗？
[ ] Getting Friendly With CPU Caches
[ ] 切片(slice)性能及陷阱
[ ] Does Go's range Copy a Slice Before Iterating Over It?

[ ] 深入探究 Go 中的 array 与 slice


Allow test beds to override defaultTestTimeout

Not a breaking change.
An alternative to this would be to create another property for AutomatedTestWidgets binding for debugging specifically. I feel like that's too much of a narrow use case.

Approach LGTM
I don't know how to write a test that exercises that this timeout is used, but in some sense that is orthogonal to changing the default value. I would still write a trivial test case that sets this value to something else though, if only to prevent someone from skipping the comment and changing it to a getter

Done!

The flutter-build has been broken for 5 days (or every time I remember that I need to merge this, it is broken ;) ).  Is it possible to get alerted when I can submit this?

@mehmetf I submitted it for you since I just caught the tree in a green state.

Thanks Michael!


Wrapped footer in block so it can be overridden

@mparent61 Simple change to wrap the <footer> in the base IA template in a block so the age disclaimer can be added by The Onion in its override

Coverage remained the same at 87.107% when pulling e08bca4b94c9366990856cb066247098ca955c13 on article-footer-block into 268ea034a7f8be31d8253eaf14c38157bdfc3f0d on 0.8.x-release.


BRICSDecomposition does not break a particular ligand

Dear all,
I have recently tried to decompose a known drug that is Sertraline Hydrochloride using the function BRICSDecompose(). There is no problem with the code itself but the decomposition fails whereas I would have expected a fragmentation of the ligand into two fragments.
Is there anything you can do to improve the fragmentation algorithm?

RDKit Version: 2017.09.3
Platform: Windows 10

I report the code below:
m = Chem.MolFromSmiles("CNC1CCC(c2ccc(Cl)c(Cl)c2)c2ccccc12")
AllChem.Compute2DCoords(m)
frags = BRICS.BRICSDecompose(m, returnMols=True)

Here's what's going on:
There are BRICS patterns associated with aliphatic carbon rings (it's atom type L15 in the original paper), and there are rules for L15 - L16 (aromatic carbon ring) and L15 - L5 (SP3 nitrogen).
The RDKit encoding of the pattern for L15: [C;$(C(-;@C)-;@C)] requires that the linking atom and its two neighbors are all aliphatic C. That fails in your case since one of the neighbors is aromatic.
I think it's probably ok for us to change that pattern to [C;$(C(-;@C)-;@[#6])] in order to allow an aromatic neighbor. but I will need to think about it a bit more in order to be sure.

I have spotted another one as well: "NS(=O)(=O)c1cc(ccc1Cl)C1(O)NC(=O)c2ccccc12" (Chlorthalidone)
Similar issue.


"superset_worker         Report state: [Errno -2] Name or service not known" when sending email alerts.

Hi, we are having difficulty sending alerts via email in supersets. Every time an alert is triggered, the celery worker will return the following error:
superset_worker         Taking a PNG screenshot or url http://superset:8088/superset/dashboard/2/
superset_worker         [2021-05-25 11:50:09,259: INFO/ForkPoolWorker-1] Taking a PNG screenshot or url http://superset:8088/superset/dashboard/2/
superset_worker         Report state: [Errno -2] Name or service not known
superset_worker         [2021-05-25 11:50:10,348: INFO/ForkPoolWorker-1] Report state: [Errno -2] Name or service not known
superset_worker         [2021-05-25 11:50:10,351: INFO/ForkPoolWorker-1] Task reports.execute[50cbb39a-4a1c-4dac-83e2-56b7be28cf13] succeeded in 10.219615060952492s: None

We have set up supersets in a docker container so we have the celery worker, celery beat, redis and postgres instance up. the following is the docker-compose.yml:
#
# Licensed to the Apache Software Foundation (ASF) under one or more
# contributor license agreements.  See the NOTICE file distributed with
# this work for additional information regarding copyright ownership.
# The ASF licenses this file to You under the Apache License, Version 2.0
# (the "License"); you may not use this file except in compliance with
# the License.  You may obtain a copy of the License at
#
#    http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#
x-superset-image: &superset-image apache/superset:latest
x-superset-depends-on: &superset-depends-on
  - db
  - redis
x-superset-volumes: &superset-volumes
  # /app/pythonpath_docker will be appended to the PYTHONPATH in the final container
  - ./docker:/app/docker
  - superset_home:/app/superset_home
  - /opt/superset/config:/app/pythonpath

version: "3.7"
services:
  redis:
    image: redis:latest
    container_name: superset_cache
    restart: unless-stopped
    #    ports:
    #  - 6380:6379
    volumes:
      - redis:/data

  db:
    env_file: docker/.env
    image: postgres:10
    container_name: superset_db
    restart: unless-stopped
    #    ports:
    #  - 5433:5432
    volumes:
      - db_home:/var/lib/postgresql/data

  superset:
    env_file: docker/.env-non-dev
    #image: *superset-image
    build: .
    container_name: superset_app
    command: ["/app/docker/docker-bootstrap.sh", "app-gunicorn"]
    user: "root"
    restart: unless-stopped
    ports:
      - 8088:8088
    depends_on: *superset-depends-on
    volumes: *superset-volumes

  superset-init:
    #image: *superset-image
    build: .
    container_name: superset_init
    command: ["/app/docker/docker-init.sh"]
    env_file: docker/.env-non-dev
    depends_on: *superset-depends-on
    user: "root"
    volumes: *superset-volumes

  superset-worker:
    #image: *superset-image
    build: .
    container_name: superset_worker
    command: ["/app/docker/docker-bootstrap.sh", "worker"]
    env_file: docker/.env-non-dev
    restart: unless-stopped
    depends_on: *superset-depends-on
    user: "root"
    volumes: *superset-volumes

  superset-worker-beat:
    #image: *superset-image
    build: .
    container_name: superset_worker_beat
    command: ["/app/docker/docker-bootstrap.sh", "beat"]
    env_file: docker/.env-non-dev
    restart: unless-stopped
    depends_on: *superset-depends-on
    user: "root"
    volumes: *superset-volumes

volumes:
  superset_home:
    external: false
  db_home:
    external: false
  redis:
    external: false

as you can see I use a custom superset docker image build from the following docker file. This is the install the Chrome Driver needed for he alerts. The following is my Docker file:
FROM apache/superset:latest

USER root

RUN apt-get update && \
    wget -q https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb && \
    apt-get install -y --no-install-recommends ./google-chrome-stable_current_amd64.deb && \
    rm -f google-chrome-stable_current_amd64.deb && \
    apt-get install -y vim

RUN export CHROMEDRIVER_VERSION=$(curl --silent https://chromedriver.storage.googleapis.com/LATEST_RELEASE_90) && \
    wget -q https://chromedriver.storage.googleapis.com/${CHROMEDRIVER_VERSION}/chromedriver_linux64.zip && \
    unzip chromedriver_linux64.zip -d /usr/bin && \
    chmod 755 /usr/bin/chromedriver && \
    rm -f chromedriver_linux64.zip

RUN pip install --no-cache gevent psycopg2 redis

EXPOSE 8088

HEALTHCHECK CMD curl -f "http://superset:8088/health"

USER superset

the configuration are as follows:
from celery.schedules import crontab

FEATURE_FLAGS = {
    "ALERT_REPORTS": True
}

REDIS_HOST = "redis"
REDIS_PORT = "6379"

class CeleryConfig:
    BROKER_URL = 'redis://%s:%s/0' % (REDIS_HOST, REDIS_PORT)
    CELERY_IMPORTS = ('superset.sql_lab', "superset.tasks", "superset.tasks.thumbnails", )
    CELERY_RESULT_BACKEND = 'redis://%s:%s/0' % (REDIS_HOST, REDIS_PORT)
    CELERYD_PREFETCH_MULTIPLIER = 10
    CELERY_ACKS_LATE = True
    CELERY_ANNOTATIONS = {
        'sql_lab.get_sql_results': {
            'rate_limit': '100/s',
        },
        'email_reports.send': {
            'rate_limit': '1/s',
            'time_limit': 600,
            'soft_time_limit': 600,
            'ignore_result': True,
        },
    }
    CELERYBEAT_SCHEDULE = {
        'reports.scheduler': {
            'task': 'reports.scheduler',
            'schedule': crontab(minute='*', hour='*'),
        },
        'reports.prune_log': {
            'task': 'reports.prune_log',
            'schedule': crontab(minute=0, hour=0),
        },
    }
CELERY_CONFIG = CeleryConfig

SCREENSHOT_LOCATE_WAIT = 100
SCREENSHOT_LOAD_WAIT = 600

# Slack configuration
SLACK_API_TOKEN = "xoxb-"

EMAIL_NOTIFICATIONS= True

# Email configuration
SMTP_HOST = "<email server ip>"
SMTP_STARTTLS = False
SMTP_SSL = False
SMTP_USER = ""
SMTP_PORT = <email server port> # your port eg. 587
SMTP_PASSWORD = ""
SMTP_MAIL_FROM = "<from email>"

# WebDriver configuration
# If you use Firefox, you can stick with default values
# If you use Chrome, then add the following WEBDRIVER_TYPE and WEBDRIVER_OPTION_ARGS
WEBDRIVER_TYPE = "chrome"
WEBDRIVER_OPTION_ARGS = [
    "--force-device-scale-factor=2.0",
    "--high-dpi-support=2.0",
    "--headless",
    "--disable-gpu",
    "--disable-dev-shm-usage",
    "--no-sandbox",
    "--disable-setuid-sandbox",
    "--disable-extensions",
]

# This is for internal use, you can keep http
WEBDRIVER_BASEURL="http://superset:8088"
# This is the link sent to the recipient, change to your domain eg. https://superset.mydomain.com
WEBDRIVER_BASEURL_USER_FRIENDLY="http://<redacted ip>:8088"

SQLALCHEMY_DATABASE_URI = "postgresql://superset:superset@db:5432/superset"

I can't figure out what's wrong and google didn't come up with anything helpful. I can confirm all container can telnet to the redis container via port 6379 and can telnet to superset_app container via port 8088.
Hope someone can give me some ideas on where to look as i am at my wits end. Thank you.

@CollinKendor : have you found a solution? we are having a similar setup and are running into a similar problem where the error log does not make sense...
2021-12-23 07:34:15	 Taking a PNG screenshot or url http://superset.superset-lsy-lsyesp-any:8088/superset/slice/1/?standalone=true&standalone=3 2021-12-23 07:34:15	 [2021-12-23 07:34:15,445: INFO/ForkPoolWorker-7] Taking a PNG screenshot or url http://superset.superset-lsy-lsyesp-any:8088/superset/slice/1/?standalone=true&standalone=3 2021-12-23 07:34:16	 Report state: [Errno -2] Name or service not known 2021-12-23 07:34:16	 [2021-12-23 07:34:16,231: INFO/ForkPoolWorker-7] Report state: [Errno -2] Name or service not known

think it is due to rights in the account.....i can't remember the details. it has been a while.

actually if anyone else comes across here, we fixed it adding a celerybeat worker
      containers:
      - name: superset
        image: "apache/superset:latest"
        imagePullPolicy: IfNotPresent
        command: ["/bin/sh","-c",". /app/pythonpath/superset_bootstrap.sh; celery --app=superset.tasks.celery_app:app beat"]

and then step by step checking the superset-config.py file until we ended with
class CeleryConfig(object):
    BROKER_URL = f"redis://default:{env('REDIS_PASSWORD')}@{env('REDIS_HOST')}:{env('REDIS_PORT')}/0"
    CELERY_IMPORTS = ('superset.sql_lab','superset.tasks', )
    CELERY_RESULT_BACKEND = f"redis://default:{env('REDIS_PASSWORD')}@{env('REDIS_HOST')}:{env('REDIS_PORT')}/0"
    CELERYD_PREFETCH_MULTIPLIER = 10
    CELERY_ACKS_LATE = True
    CELERY_TASK_PROTOCOL = 1
    CELERY_LOG_LEVEL = 'DEBUG'
    CELERY_ANNOTATIONS = {'tasks.add': {'rate_limit': '10/s'},
                          'sql_lab.get_sql_results': {'rate_limit': '100/s',},
                          'email_reports.send': {
                          'rate_limit': '1/s',
                          'time_limit': 600,
                          'soft_time_limit': 600,
                          'ignore_result': True,},
                          }
    CELERYBEAT_SCHEDULE = {
      'email_reports.schedule_hourly': {
          'task': 'email_reports.schedule_hourly',
          'schedule': crontab(minute=1, hour='*'),
      },
      'alerts.schedule_check': {
          'task': 'alerts.schedule_check',
          'schedule': crontab(minute='*', hour='*'),
      },
      'reports.scheduler': {
          'task': 'reports.scheduler',
          'schedule': crontab(minute='*', hour='*'),
      },
      'reports.prune_log': {
          'task': 'reports.prune_log',
          'schedule': crontab(minute=0, hour=0),
      },
      'cache-warmup-hourly': {
          'task': 'cache-warmup',
          'schedule': crontab(minute=0, hour='*'), #hourly
           'kwargs': {
              'strategy_name': 'top_n_dashboards',
              'top_n': 10,
              'since': '7 days ago',
          },},}

CELERY_CONFIG = CeleryConfig
RESULTS_BACKEND = RedisCache(
      host=env('REDIS_HOST'),
      port=env('REDIS_PORT'),
      password=env('REDIS_PASSWORD'),
      key_prefix='superset_results'
)

#adding email report function
FEATURE_FLAGS = {
    "ALERT_REPORTS": True,
    "DASHBOARD_NATIVE_FILTERS": True,
    "DASHBOARD_CROSS_FILTERS": True,
    "ENABLE_TEMPLATE_PROCESSING": True,
    "ENABLE_SCHEDULED_EMAIL_REPORTS": True,
    "EMAIL_NOTIFICATIONS": True
}
# see: https://superset.apache.org/docs/installation/configuring-superset#sip-15
SIP_15_ENABLED = True

# If set to true no notification is sent, the worker will just log a message.
# Useful for debugging
ALERT_REPORTS_NOTIFICATION_DRY_RUN = False

EMAIL_NOTIFICATIONS = True

SCREENSHOT_LOCATE_WAIT = 100
SCREENSHOT_LOAD_WAIT = 600

# Email configuration
SMTP_HOST = "airlapp.isb.lsy.fra.dlh.de" #change to your host
SMTP_STARTTLS = False
SMTP_SSL = False
SMTP_USER = ""
SMTP_PORT = XXX # your port eg. 587
SMTP_PASSWORD = ""
SMTP_MAIL_FROM = " Superset | Reporting Team<EMAIL_ADDRESS>
WEBDRIVER_BASEURL = "http://superset.<namespace>:8088/"
WEBDRIVER_BASEURL_USER_FRIENDLY = WEBDRIVER_BASEURL

SIP_15_ENABLED
SMTP_USER = "" # leaving it empty is important
SMTP_PASSWORD = "" # leaving it empty is important
were important steps - thanks to (among others) https://github.com/apache/superset/issues/11507#issuecomment-841588337


Mapping loaded, pretrained model to actual device.

Required for CPU-only inference. Now models are not mapped with torch.load.

Thank you for pull request! :)


Use type instead of types

rackt/redux dies if an action does not have a type property (see)
I suggest using the same model for solving this as pburtchaell/redux-promise-middleware, namely the use of type suffixes (see)

+1

@Jador In what order are you applying your middleware?

This day in age? I have no clue. Using redux-saga for problems of this nature nowadays.


MoveCodeRefactoring with AnonymousClasses

Description Available here: https://github.com/tsantalis/RefactoringMiner/issues/440

Proof of fix:


tokens_match? issue

on line 10 of DeviseTokenAuth::Concerns::User
i think that
result = @token_equality_cache[key] ||= (::BCrypt::Password.new(token_hash) == token)
should be
result = @token_equality_cache[key] ||= (::BCrypt::Password.new(token_hash).to_str == token)
BCrypt::Password inherits from String, but it doesn't evaluate to true unless i cast it. obviously it's working for others, but i don't think that there's a downside to making the change.

I have been facing the same issue...but only sometimes.
It happens on dev wayyy more than prod

@mackmm145 - I think there may be an issue with what you're trying to do. == means something different in BCrypt:
https://github.com/codahale/bcrypt-ruby#how-to-use-bcrypt-ruby-in-general

Closing for now due to a decent suggested answer.


Hardware Requirements for the HCM

Why isn't there a mention of proper hardware requirements for the HCM? I couldn't find anywhere on the internet that mentions at least minimum required hardware specs or how to calculate the sizing for the server.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 645d01fc-eae4-67c0-ef63-77d34d854d5e
Version Independent ID: 516bc8a7-d927-d080-36a6-69ca95686feb
Content: Hybrid connections - Azure App Service
Content Source: articles/app-service/app-service-hybrid-connections.md
Service: app-service
GitHub Login: @madsd
Microsoft Alias: madsd

@mudithac Thanks for the feedback. We are investigating the issue and will update you shortly.

@mudithac HCM runs on Windows Server 2012 or later and follows whatever minimum hardware requirement they have. We have not seen a need to increase the specification. @jselig95 fyi.

@mudithac We will now proceed to close this thread. If there are further questions regarding this matter, please let us know.


GetProgramAccounts with update authority MemCmp

Premise
Hope this is not due to some error I am making.

Describe the bug
I am using the RPC call GetProgramAccounts/Async on the Token Metadata program ID in order to retrieve all tokens by a certain updateAuthority address. I got to know that the updateAuthority is at offset 1 (is it right?), therefore I am using the following MemCmp:
filters.Add(new()
{
    Offset = 1,
    Bytes = Encoders.Base58.EncodeData(Encoding.UTF8.GetBytes(updateAuthority))
});

however, the call returns no results.
If I do not use the updateAuthority MemCmp and search by something else (for example name <offset 69> or symbol <offset 105>) I am able to retrieve the mint addresses. And, by using:
var bytes = Convert.FromBase64String(pair.Account.Data[0]);
string updateAuthority = ((ReadOnlySpan<byte>)bytes).GetPubKey(1);

I can retrieve the updateAuthority. Thus, I deduced that the Offset for the updateAuthority should be 1.
To Reproduce
Call GetProgramAccounts/Async RPC request with the following MemCmp as filter:
filters.Add(new()
{
    Offset = 1,
    Bytes = Encoders.Base58.EncodeData(Encoding.UTF8.GetBytes(updateAuthority))
});

Expected behavior
I am expecting to be able to filter by updateAuthority.
Desktop (please complete the following information):

OS: Windows x64

This looks to me like it is a minor oversight and not a bug, we've been using GetProgramAccounts with filters without any issues.
After a quick look at the metaplex repo and Solnet.Metaplex it looks like the UpdateAuthority is indeed at offset 1 after the Key which defines the type of the account.
Would it be possible for you to initialize the UpdateAuthority as a PublicKey and try the following?
filters.Add(new() {
Offset = 1,
Bytes = Encoders.Base58.EncodeData(updateAuthority.KeyBytes) });

Or rather, it looks like you're performing conversion string -> bytes -> base58 encoding, this doesn't seem right to me, look at this example from Solnet.Mango where the public key is passed as a string into the method
        public async Task<ProgramAccountsResultWrapper<List<PerpMarket>>> GetPerpMarketsAsync(string mangoGroup,
            Commitment commitment = Commitment.Finalized)
        {
            List<MemCmp> filters = new()
            {
                new MemCmp { Offset = PerpMarket.Layout.MangoGroupOffset, Bytes = mangoGroup }
            };

            return await GetProgramAccounts<PerpMarket>(MangoProgram.ProgramIdKeyV3, filters, PerpMarket.Layout.Length,
                commitment);
        }```

You can probably just do

```filters.Add(new()
{
    Offset = 1,
    Bytes = updateAuthority
});```

This looks to me like it is a minor oversight and not a bug, we've been using GetProgramAccounts with filters without any issues.
After a quick look at the metaplex repo and Solnet.Metaplex it looks like the UpdateAuthority is indeed at offset 1 after the Key which defines the type of the account.
Would it be possible for you to initialize the UpdateAuthority as a PublicKey and try the following?
filters.Add(new() {
    Offset = 1,
    Bytes = Encoders.Base58.EncodeData(updateAuthority.KeyBytes)
});

Or rather, it looks like you're performing conversion string -> bytes -> base58 encoding, this doesn't seem right to me, look at this example from Solnet.Mango where the public key is passed as a string into the method
/// <inheritdoc cref="IMangoClient.GetPerpMarketsAsync(string, Commitment)"/>
public async Task<ProgramAccountsResultWrapper<List<PerpMarket>>> GetPerpMarketsAsync(string mangoGroup,
            Commitment commitment = Commitment.Finalized)
{
    List<MemCmp> filters = new()
    {
         new MemCmp { Offset = PerpMarket.Layout.MangoGroupOffset, Bytes = mangoGroup }
    };

    return await GetProgramAccounts<PerpMarket>(MangoProgram.ProgramIdKeyV3, filters, PerpMarket.Layout.Length,
                commitment);
}

You can probably just do
filters.Add(new()
{
    Offset = 1,
    Bytes = updateAuthority
});

I tried both methods and indeed they both works. I didn't know about the KeyBytes field of the PublicKey class. I guess the class must have an implicit operator such that:
public static implicit operator string(PublicKey key) => key.KeyBytes

Indeed, we have added operators on PublicKey so that operating with byte[] and string is much easier in a number of different places. Are you able to get the tokens by the authority now?

Yes I am able to filter by update authority now, thanks for the fast responses.
Since I am here, may I ask which way would be the fastest in order to retrieve the owner address (of an NFT, token with supply 1) having the mint address? Currently using: GetLargestAccount -> >GetAccountInfo

That does seem like the fastest way, yes, although I've never actually had to do it like that.
I don't know if you've ever seen this, it was a watcher I did for nft marketplaces some time ago just for fun. In this context what I had to do was figure out who was buying/selling/delisting only having the transactions in which the sale occurred and had to figure it out from the destination token account, the following links might help if you ever need something like that.
https://github.com/MonkeDAO/observer/blob/master/Observer/Handlers/DigitalEyesHandler.cs#L31-L35
https://github.com/MonkeDAO/observer/blob/master/Observer/MetaplexHelpers.cs#L44-L64

Thank you very much for the information it was very helpful. Really appreciate it.
I think this issue may be closed then 😃

In the future if you need any help but are unsure if it's a bug feel free to open a discussion :)


What gitlab needs to do to be great again

docker start times (faster jobs)
more stability, less outages
remove the need to have to refresh the MR to see a finished job
navigating back to the mr description should eventually trigger a refresh and shouldnt always be outdated
better menus
job search should be less shitty
the search should suck less ass, always takes a few clicks to tune it right
less outages of the search

I think we kind of gave up on gitlab and are now all on github. Let's close this issue @hendriknielaender ?


broken tracing test checks networks instead of hardcoded name

we skipped some tests by hardcoding cluster name checks: https://github.com/istio/istio/issues/28890
this should skip them properly, but we should see if the actual fix is possible to backport

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

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.8
2e1aabc8afdf2e5173996feebca1d8b399c80bb8
link
/test release-notes_istio_release-1.8

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

/retest

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

Test name
Commit
Details
Rerun command

release-notes_istio_release-1.8
2e1aabc8afdf2e5173996feebca1d8b399c80bb8
link
/test release-notes_istio_release-1.8

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

https://github.com/istio/istio/blob/6d5406628e2fa9bd2fa31e9b4bc5d05186e01bd6/tests/integration/telemetry/tracing/zipkin/servertracing/tracing_test.go#L42-L43
there is no actual fix

/retest


Catch Exception instead of Throwable in dependency-management

Context
In 99% of cases we don't want to catch Errors, which catch Throwable
would do.

Changes applied through #7813 and follow up issue created (#7814)


Allows to disable obfuscation for the project

"enabled — boolean. Allows to disable obfuscation for the project. Default value is true." I think that the default value should be false.

Why? The purpose of the library is to obfuscate the project so the default behavior should be configuration-less if possible.

Obfuscation can be disabled using the following block:
paranoid {
  enabled false
}


feat(Makefile): add --no-cache and --pull flag to docker build

These flags enable two things:

ensure always have the up to date upstream image when building (CI's been caching the image for some time)
ensure we have the latest releases of bash, curl, net-tools and procops

Are the ubuntu-slim images ever updated in place? I thought they were versioned so it should never be a problem.

I believe that is the case. The problem is that ubuntu-slim appears to have a few major CVEs that have not been addressed and there's no 0.5 release out. See https://quay.io/repository/deis/base/image/7baf34926ead6435b63fc8f9952b9d5de1766a38d0badd86028fbaa132141816?tab=vulnerabilities which is for the v0.3.1 release.

removed the --pull commit.

there's no 0.5 release out

@bacongobbler there is, but at aledbf/ubuntu-slim:0.5. Not sure if we need to wait for it to arrive at gcr.io/google_containers/ubuntu-slim:0.5 though.

I built deis/base from this PR and uploaded it to quay.io:
https://quay.io/repository/mboersma/base/image/439f6a565603ba4fcb3fc16cc16e83988e2d616004c1497c0696acfb64d11b30?tab=vulnerabilities
It still contains the two "high" vulnerabilities after scanning though.
Maybe this PR should also add apt-get upgrade to refresh base packages that we don't explicitly install?

@mboersma I think ubuntu slim does a lot of things to clean up after installing packages, so we'd loose a lot of optimizations.

So what's the verdict on this PR? Go or no-go? Upstream having security vulnerabilities is concerning but out of scope for this PR. We'll have to address that later.


FancyZones Layout Editor reverts to Priority Grid after changing active template using hotkey

Microsoft PowerToys version
"0.61.1"
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce

Open Layout Editor, either using hotkey (Windows+`) or from PowerToys/FancyZones Settings tab
Make sure you assign a hotkey to at least one custom layout (i.e. Ctrl-Windows-Alt+1)
Select ANY Layout besides "Priority Grid" (either a built-in one or a custom one)
Close Layout Editor
RE-open layout editor via hotkey (Windows+`) OR from settings
Notice the same layout you chose in step 3 is still considered "active"
Close Layout Editor
NOW, press the hotkey to set the current layout to one of your custom layouts (i.e. Ctrl-Windows-Alt+1) (you will see the layout zones flash on the screen as confirmation)
RE-open layout editor via hotkey OR from settings
The layout editor will NOW "look" as if "Priority Grid" is the currently active layout, but it's really not (which you can prove by closing layout editor and dragging windows around). Unless you select a different layout while in the editor, the REAL active layout is the one you selected with the hotkey, even thought the editor displays a different one.

✔ Expected Behavior
After setting the active layout using a hotkey, the layout editor should still display THAT layout as if it's the active one, instead of reverting back to "Priority Grid"
❌ Actual Behavior
layout editor shows the built-in "Priority Grid" layout as the active layout when launching the editor after selecting one via hotkey
Other Software
No response

Are you still experiencing this issue? /needinfo

sorry, i hadn't tried doing this for a while. i haven't been using different layouts, so it became a moot issue. since i switched to 2 monitors, each with lower resolution than my one previous ultra-wide monitor, whenever i host a meeting, i just share one of my screens, and the dimensions are small enough for my coworkers; therefore, i dont need to switch to a lower resolution for meetings. that said, when i get a chance, i will test this out, and see if the behavior described above is still happening.

sorry, i finally got a chance to see if this behavior is still there, and sure enough, in v0.83.0 it still behaves the way I described in the OP.
When you first open the layout editor, everything looks fine. Whatever layouts you've assigned to your monitor(s), they appear correct. However, once you use a hotkey to change the layout on a particular monitor, the next time you open the layout editor, ALL monitors will show "Priority Grid" as their current layout. And the weird thing is, now it ACTUALLY changes the assigned layouts to all monitors to Priority Grid; it's not just a misrepresentation of the current layout. so, if you ask me, this issue got a little worse. Originally, when i created this issue, i believe it was just a visual thing where the layout editor was saying that the selected monitor was using Priority Grid, even if it wasn't, but this time, when i saw that both monitors "switched" to Priority Grid, and then closed the layout editor, sure enough, both monitors had switched to the Priority Grid layout.
Mind you, if i had never opened the layout editor, the layouts wouldn't have changed on my monitors, so it's the act of opening the layout editor after using a layout hotkey to change the layout on any monitor that makes it so that the next time you open the layout editor, all monitors will have reverted back to Priority Grid automatically.
Please reopen this issue, since it's actually not fixed. Thank you. (i dont open the layout editor very often, if ever, so it's not a critical issue that's constantly causing me anguish; it's just something i noticed and thought, "hey, that's not supposed to happen like that.")


404 error from artifactory when trying to install

Command used: :CocInstall coc-tsserver
Error received: Error on install coc-tsserver: Error: Invalid response from https://artifactory.secureserver.net/artifactory/api/npm/coc-tsserver: 404

was an issue with an unwanted .npmrc in my repo, sorry!


Overhaul terminal io

This PR makes a number of enhancements to terminal io. Because of how the changes stacked on top of each other, it both was difficult to split this into many smaller PRs. Every individual commit in the PR compiles and should likely pass scripted.
The two main changes are:

Adds a new Terminal class which is used to do things like switch the tty mode between canonical and raw mode and also access properties like the height and width of the terminal.
Changes the global java.lang.System.out and scala.Console.out to go through a proxy. This enables us to make supershell work with println. It also allows us to make supershell work when output has been printed without a newline by reseting the cursor back to where the previous output left off.

The impact of item 2 can be seen in some recorded terminal sessions. First I made a simple project that ran a task that depended on 8 other tasks that each just print some random lines to System.out. Here is what it looks like with 1.3.5:

And here is what it looks like with these changes:

The run task didn't really work with supershell before (https://github.com/sbt/sbt/issues/5246):

Now the cursor correctly moves back to the right place:

I also made it so that if there are fewer than 10 main classes that the user doesn't have to enter a newline to select the class.
I also made some improvements to ConsoleChannel and the CommandExchange that makes the output more sane when a client connects and runs commands. With 1.3.5, the output could get mixed with the shell prompt and the prompt wasn't restored after the command completed:

Now the output is saner and sbt logs what command is being invoked by the network client:

Miscellaneous fixes:

Clear the current line and the screen to the bottom in the shell prompt. This prevents stray supershell lines from ending up stuck at the bottom.
We no longer poll System.in in ConsoleChannel and block instead. As a result the cpu utilization of sbt when idle drops from about 2% to 0% on my computer. Input latency should also be slightly lower.
In continuous, we stop trying to read from System.in if we read a -1 byte that indicates the channel has closed.

Fixes #5246
I think should fix #5162

@eed3si9n I rebased this branch, made a number of the requested changes and added commentary about some wording issues where I have some different opinions.

@eatkins Thanks for the update. I'll take a closer look.


feat: add paseto extension

What type of PR is this?
feat

What this PR does / why we need it (en: English/zh: Chinese):

en: add paseto middleware
zh: 添加 paseto 中间件
Which issue(s) this PR fixes:
Fixes https://github.com/cloudwego/hertz/issues/618

PATL @zstone12 @L2ncE

Also I'm curious if there are any practical use scenarios? verify that your middleware is usable in complex scenarios.

Also I'm curious if there are any practical use scenarios? verify that your middleware is usable in complex scenarios.

soon will be used in FreeCar

LGTM


i don't can remove this permission

command
/luckperms group svip permission unset essentials.speed

not removes this permission for group
/luckperms group svip permission set
essentials.speed true

i no have any error
and i have the latest build luckperms

how to?
I copied badly this permissions
http://wiki.ess3.net/wiki/Command_Reference
and paste to console

Those commands need to be on the same line in the exported file.

i know that
you can fix this bug?

this is not export bug

i don't can remove this "essentials.speed" permission for my databas

Show me the output when you run
/lp group svip permission unset essentials.speed, as well as any errors you get in console.

no, command /lp group svip permission unset essentials.speed normaly work
i use discordsrv plugin console to paste text copy for that site "http://wiki.ess3.net/wiki/Command_Reference"
and now i can remove this data
essentials.speed true```

Sorry, I have no clue what you're trying to ask.
You just pasted the same exact command 3 times.

you can help me?

ok. relax
steps to reproduc

copy ```

essentials.speed
http://wiki.ess3.net/wiki/Command_Reference

2. paste to discord console
![2017-09-12 20_04_00- s2-survival-console - discord](https://user-images.githubusercontent.com/5884000/30341243-9cd583ca-97f5-11e7-8f26-a2debecdb0cc.png)

3. generate input file
![2017-09-12 20_09_51-c__users_y7_appdata_local_temp_fz3temp-2_inpot yml - notepad](https://user-images.githubusercontent.com/5884000/30341488-708f4250-97f6-11e7-904a-376477af012a.png)

4. test  ( not work )
![2017-09-12_20 10 52](https://user-images.githubusercontent.com/5884000/30341517-92bbaf3a-97f6-11e7-903c-39643e6602c7.png)

5. add true permissions in game
![2017-09-12_20 13 08](https://user-images.githubusercontent.com/5884000/30341593-e27cc180-97f6-11e7-934d-ead41007b065.png)

6. input
![2017-09-12 20_17_18-movies tv](https://user-images.githubusercontent.com/5884000/30341750-74799c48-97f7-11e7-8d45-9a6fad3e173f.png)

7. test ( work )
![2017-09-12_20 18 19](https://user-images.githubusercontent.com/5884000/30341776-8cf09a74-97f7-11e7-812b-85dab9c80e25.png)

...

how do i can remove this from my database?
![2017-09-12 20_17_18-movies tv](https://user-images.githubusercontent.com/5884000/30341821-c858cf0a-97f7-11e7-8722-71e878c4e2d5.png)

https://git.io/v5DnJ

Seems like the Discord console appends a newline to the start of the permission. This is not a LuckPerms bug.
You'll probably have to remove the permission manually, or run the unset command via Discord.

yep, of course
anyway, thx for help me : P

@lucko you can block give this badly permissions?

Yes.

do it ^^


Provide a workspace ability

Enhancement Request:
Provide a workspace concept. This doesn’t necessarily have to be called workspace. The FBI often refers to them as cases. The goal is to allow the analyst to save his investigation work/progress. This could include the searches performed. The graph generated. The email and attachments that are of most interest to the analyst. An input text form that explains the goals, purpose, steps taken, status, and notes for his investigation. Etc. etc. One major requirement is that what is saved today is available tomorrow.

Workspaces is being handled by Mosaic user interface.


added a new location SLR

i have validated the json format from the site you have mentioned.
it gave me a green ticket.

Correct the info and re-open the PR.

that was correct sorry for inconvenience i will send the pull request again with some more added location
this time i will make sure it is correct

i will fix and resend you this request again


Validator on field defined with Union of arbitrary type and with default value raises ConfigError

Question
Hi,
I wonder if the behavior below results from a bug or me missing a point.
A clarification would be appreciated.
Thanks
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
pydantic version: 1.3
pydantic compiled: False
install path: ~/miniconda3/lib/python3.8/site-packages/pydantic
python version: 3.8.0 | packaged by conda-forge | (default, Nov 22 2019, 19:11:19)  [Clang 9.0.0 (tags/RELEASE_900/final)]
platform: macOS-10.13.6-x86_64-i386-64bit
optional deps. installed: []

Customize pydantic BaseConfig and define arbitrary class Test:
from pydantic import BaseModel, validator, Field, BaseConfig
from typing import Union, Type

BaseConfig.arbitrary_types_allowed = True

class Test:
	pass

Defining class A without validator works:
class A(BaseModel):
	view: Union[Type[Test], str] = Test

Defining class A with a validator fails:
class A(BaseModel):

	view: Union[Type[Test], str] = Test

	@validator("view")
	def validate(cls, v):
		return v

With the following error:
---------------------------------------------------------------------------
ConfigError                               Traceback (most recent call last)
<ipython-input-2-72c9cde31275> in <module>
----> 1 class A(BaseModel):
      2
      3         view: Union[Type[Test], str] = Test
      4
      5         @validator("view")

~/miniconda3/lib/python3.8/site-packages/pydantic/main.py in __new__(mcs, name, bases, namespace, **kwargs)
    221         if _custom_root_type:
    222             validate_custom_root_type(fields)
--> 223         vg.check_for_unused()
    224         if config.json_encoders:
    225             json_encoder = partial(custom_pydantic_encoder, config.json_encoders)

~/miniconda3/lib/python3.8/site-packages/pydantic/class_validators.py in check_for_unused(self)
    171         if unused_validators:
    172             fn = ', '.join(unused_validators)
--> 173             raise ConfigError(
    174                 f"Validators defined with incorrect fields: {fn} "  # noqa: Q000
    175                 f"(use check_fields=False if you're inheriting from the model and intended this)"

ConfigError: Validators defined with incorrect fields: validate (use check_fields=False if you're inheriting from the model and intended this)

Setting the default value to Field(Test) avoids the ConfigError:
class A(BaseModel):

	view: Union[Type[Test], str] = Field(Test)

	@validator("view")
	def validate(cls, v):
		return v

I agree it's unclear, the problem is
https://github.com/samuelcolvin/pydantic/blob/dbc044e3570170e30b1046474ac7040b1b100b07/pydantic/main.py#L141
Attributes of these types are ignored, thus the view field doesn't exist and the validator is correctly complaining that it's being asked to reference a field that doesn't exist.
This check is because in the following example, we wouldn't expect Foo to be a field of the model:
class MyModel(BaseModel):
    a: int
    b: str

    class Foo:
        pass

Obviously thing = Foo looks very different to class Foo:..., but they both come through as types aka isintance(Foo, type) is True.
Options to resolve this:

use Field as you have above which works fine
skip the UNTOUCHED_TYPES check on annotations but not attributes. That would fix your case, but would mean thing: Type[Foo] = Foo worked, but thing = Foo would silently fail, which might be even more confusing
only exclude subtypes of type if the name of the attribute matches the name of the type. This should exclude sub-classes while probably not effecting assignments like yours.
Keep the logic same but allow some escape hatch, e.g. a function on Config which replaces isinstance(value, untouched_types)

I'm inclined to to lend towards 3 and 4. Thoughts?

Thanks for the clarification.
For the sake of completeness, please let me add that this also fails:
class A(BaseModel):

    view = Test

    @validator("view")
    def validate(cls, v):
        return v

which 3. will not be able to fix, correct ?
As for a potential 5.: requiring inner classes to feature a custom decorator (say @innerclass) is not something you would consider, would you ?

No, 3. should still fix that because the attribute is called "view" not "Test"

Closed by #2214

Closed by #2214


Traversal in WHERE stops working between connected vertices

Hi,
I am using OrientDB 2.1.0 GA I am not sure what to do here and how to debug this!
I have an edge between two existing vertices and a query similar to the following (I cannot post original, sorry):
select
from MyClass
where outE('egde_class').inV()['attr'] = 'some_value'

This query used to give me results, but not anymore.  The funny thing is that if I do:
select expand(outE('egde_class').inV())
from MyClass

then it works fine (results are returned). Also the following will work correctly:
select
from MyClass
where outE('egde_class').inV().length() > 0

To verify the problem I created two test classes, some vertices and edges between them.(see the following GIST). The query for test classes did give me results (gist, line 14)! So, logic works.

Has anyone else encountered similar problem?
How can I debug this?
How can I fix this at the moment?

Moved to 2.1.1GA. Still the same.

Tried with 2.1.3-SNAPSHOT and seems to work:

@lvca it works in general, yes. Like I said, when I created test classes it did work, but it would not work in one particular case. I think there might have been some data corruption or smth. I dont know.
This happened to me twice already. The first time I counted it as my own fault, because I did not know OrientDB well enough. This second time, however, I was quite sure that something went wrong with the db.
Unfortunately, I had to remove my development data and start again. So now everything works well and I cannot help with replicating the problem. It's ok to close the issue, i guess, but if I encounter that problem again I will let you know and we can re-open it. Cheers

@andreyvk, please reopen the issue if happens again. Thanks.

@lvca will do, thanks!

Use the IN, not equals, because when when you traverse with out/in/both you ends with having collections, so the IN always works.

@lvca Been struggling to understand why my approach didn't work all this time! Obviously your tip is what i was looking for. Checked it and it works! Thanks a lot again!


Enforce hoisting of some packages to fix giant disk size with Yarn

Yarn has known issues deduplicating dependencies, sometimes failing to hoist dependencies shared between a ton of packages (such as babel-runtime) and producing installs 5 times bigger than npm: https://github.com/yarnpkg/yarn/issues/2306.
While we could disable Yarn integration (https://github.com/facebookincubator/create-react-app/issues/1390), I'd like to explore an alternative approach. We should be able to determine which package causes most duplication, and then hoist it manually by making it an explicit dependency of react-scripts. This would ensure Yarn deduplicates it. This is a temporary workaround until Yarn actually fixes the algorithm (tracked in https://github.com/yarnpkg/yarn/issues/2306).
Please don't hesitate to help with this. You would need to:

Reproduce the issue with giant (~1GB) install locally with Yarn.
Figure out which dependencies get duplicated. (I don't know the easiest way but you could probably look at Babel packages and see which dependencies end up in node_modules of each of them.)
Add the duplicated dependencies to react-scripts itself to hoist them.
Verify that the issue is resolved.

I've been looking to get more involved in open source work. I'd be glad to look into this and see if I can fix it.

Yeah me too. I have created a list of duplicate modules installed by create-react-app, but am not sure what to do next.
Can I run create-react-app locally from a cloned copy?
Would it be just a matter of adding the duplicates to the Package.json in react-scripts?

Can I run create-react-app locally from a cloned copy?

Yes, see instructions here.

Would it be just a matter of adding the duplicates to the Package.json in react-scripts?

Should be.

Great I'll give it a crack thanks

According to pkgcount --du, after a fresh create-react-app, there's 57 copies of<EMAIL_ADDRESS>coming in at 394.1 MB.
<EMAIL_ADDRESS>                                           28  458.75 kB
<EMAIL_ADDRESS>                                          56  55.28 MB
<EMAIL_ADDRESS>                                                 57  394.1 MB

It appears deduping core-js should be the only priority.

FYI this is the same issue as npm 2 (why we didn't suggest using npm 2 for Babel 6).
Luckily after dropping Node < 4 we should be able to remove babel-runtime (core-js/regenerator) from babel itself after some work in https://github.com/babel/babel/issues/5118. Otherwise workaround is to make babel-runtime an explicit dependency for now.
(This is because almost all the babel plugin packages have babel-runtime (since we use https://babeljs.io/docs/plugins/transform-runtime/) to support Symbol, Promise, Map/Set, etc which are in Node 4.)

I don't mean to imply that I'm somehow more qualified to prioritize tasks than anyone else, especially @gaearon, but I fail to see why this issue is a big deal.
At the end of the day, disk space is cheap, this has no impact on build size or, if I understand correctly, build time. Its just feels to me like complaints about the size of node_modules are almost like saying "back in my day we used to walk uphill both ways to hand pick minified jQuery plugins and lovingly lay them down in the vendor directory."
Certainly this duplication bug is a problem that yarn should strive to fix, but I don't really see how this matters enough to warrant temporary workarounds in CRA.

Disk space may be cheaper but it's not universal truth especially for older machines running out of space. If you work on many projects it's annoyingly each each of them takes a whole gig.
It also impacts install times negatively because the linking step takes a long time.

I was just tracking down where the<EMAIL_ADDRESS>module comes from. It looks like<EMAIL_ADDRESS>referenced by<EMAIL_ADDRESS>is using that. All other package are using<EMAIL_ADDRESS>I have no idea how feasable it would be to update react to use the next version of fbjs (which is in 0.9.0-alpha.1)

I can't categorically disagree with anything you've said, but I just don't think that annoyance is enough of a reason to "dirty" react-scripts' package.json with incorrect dependencies, let alone disable yarn integration entirely.

@mmgm The link time alone is a bit of a killer for me, taking up to 2 minutes for an app that really isn't so big. I'm on a not so powerful laptop but it's still a little crazy.

@FWeinb oh interesting. Sounds like a similar issue with when Babel 6 was still being compiled with Babel 5 causing core-js@1 and core-js@2 deps and fixed with https://github.com/babel/babel/pull/3438 (to have the same core-js/babel-runtime version)
Also FYI: https://github.com/babel/babel/pull/3340 for core-js@1 to core-js@2 update in Babel a while ago.

@hzoo
Yes. When having the same babel-runtime should be the second goal. core-js@2 is responsible for a major chunk of memory. So trying to remove that should benefit create-react-app greatly.

So I've done some investigation into deduping some of the dependencies:
Baseline:
--- /create-react-app/test-1 ---------------------------------------------------------------------------------------
  543.5 MiB [##########] /node_modules
  188.0 KiB [          ]  yarn.lock
   60.0 KiB [          ]  README.md
   32.0 KiB [          ] /public
   24.0 KiB [          ] /src
    4.0 KiB [          ]  package.json
    4.0 KiB [          ]  .gitignore

Adding core-js as a top level dependency reduces the size significantly:
--- a/packages/react-scripts/package.json
+++ b/packages/react-scripts/package.json
@@ -28,6 +28,7 @@
     "babel-eslint": "7.1.1",
     "babel-jest": "18.0.0",
     "babel-loader": "6.2.10",
+    "core-js": "2.4.1",
     "babel-preset-react-app": "^2.0.1",
     "case-sensitive-paths-webpack-plugin": "1.1.4",
     "chalk": "1.1.3",

--- /create-react-app/test-2 ---------------------------------------------------------------------------------------
  194.0 MiB [##########] /node_modules
  188.0 KiB [          ]  yarn.lock
   60.0 KiB [          ]  README.md
   32.0 KiB [          ] /public
   24.0 KiB [          ] /src
    4.0 KiB [          ]  package.json
    4.0 KiB [          ]  .gitignore

Also adding babel-runtime reduces the size even further:
--- a/packages/react-scripts/package.json
+++ b/packages/react-scripts/package.json
@@ -28,6 +28,8 @@
     "babel-eslint": "7.1.1",
     "babel-jest": "18.0.0",
     "babel-loader": "6.2.10",
+    "babel-runtime": "6.20.0",
+    "core-js": "2.4.1",
     "babel-preset-react-app": "^2.0.1",
     "case-sensitive-paths-webpack-plugin": "1.1.4",
     "chalk": "1.1.3",

--- /create-react-app/test-3 ---------------------------------------------------------------------------------------
  142.3 MiB [##########] /node_modules
  188.0 KiB [          ]  yarn.lock
   60.0 KiB [          ]  README.md
   32.0 KiB [          ] /public
   24.0 KiB [          ] /src
    4.0 KiB [          ]  package.json
    4.0 KiB [          ]  .gitignore

From this is looks like the quickest solution would be add both babel-runtime and core-js to react-scripts. What do you think @gaearon ?
Node version: v6.9.0
Yarn version: v0.18.1
Operating system: Mac OS X 10.11.6

<EMAIL_ADDRESS>contains<EMAIL_ADDRESS>so dono if there are other babel-runtime versions as well? regenerator-runtime may help as well but probably not as much (also the other dep in babel-runtime).
Either way we will try to fix this soon for Babel 7.

From this is looks like the quickest solution would be add just babel-runtime to react-scripts. What do you think @gaearon ?

Sounds awesome, send a PR please.

@gaearon https://github.com/facebookincubator/create-react-app/pull/1441

Fixed by #1441, will be out in 0.9.0.


Weather not working with Orion 2.3.0

I'm following the tutorial, but I'm unable to obtain weather live data (OpenWeatherMap api) related to a store. The live weather health check works fine.
I've registered the context-provider with the URL http://context-provider:3000/weather/weatherConditions, but when I obtain the store information (http://{{orion}}/v2/entities/urn:ngsi-ld:Store:001), the weather information is not showing, only the data in database appears. When retrieving live data from http://{{context-provider}}/weather/weatherConditions/op/query with the payload from the tutorial, I get a bad request with the message "bad query", and the URL to the OpenWeatherMap show q=undefined. The logs also show "tutorial:proxy Making a OpenWeatherMap API request: undefined".
If I change the Orion to version 2.2.0 and register the context provider with legacyForwarding equal to true, all works fine.
How can I make the tutorial work with Orion 2.3.0?
Thanks

There was a bug in the NGSI-v2 proxy code for the Weather API - a fix has been applied here: ef9c08b
If you run ./services create, you should be able to get the latest tutorial proxy which offers NGSI-v1, NGSI-v2 and NGSI-LD endpoints.
curl -iX POST \
  'http://localhost:3000/weather/weatherConditions/op/query' \
  -H 'Content-Type: application/json' \
  -d '{
    "entities": [
        {
            "type": "Store",
            "isPattern": "false",
            "id": "urn:ngsi-ld:Store:001"
        }
    ],
    "attrs": [
        "temperature",
        "relativeHumidity"
    ]
}'

Should now return:
[
   {
      "id":"urn:ngsi-ld:Store:001",
      "type":"Store",
      "temperature":{"type":"Number","value":284.02},
      "relativeHumidity":{"type":"Number","value":50}
   }
]

Bug fixed in NGSI-v2 proxy code. Closing


GitHub repo link fixed

There was an issue where the github repo link would end with .git depending on how it is cloned


[tls] Remove use of SSL_CTX_set_client_CA_list for TLS server credentials.

This PR does the following: for the TLS server credentials, stops calling SSL_CTX_set_client_CA_list by default in ssl_transport_security.cc, and gives users a knob to re-enable calling this API.
What does the SSL_CTX_set_client_CA_list API do?
When this API is called, a gRPC TLS server sends the following data in the ServerHello: for each certificate in the server's trust bundle, the CA name in the certificate.
This API does not change the set of certificates trusted by the server in any way. Rather, it is just providing a hint to the client about what client certificate should be sent to the server.
Why are we removing the use of SSL_CTX_set_client_CA_list by default for the TLS server credentials?
Removing the use of this API by default has 2 benefits:

Calling this API makes gRPC TLS unusable for servers with a sufficiently large trust bundle. Indeed, if the server trust bundle is too large, then the server will always fail to build the ServerHello.
Calling this API is introducing a huge amount of overhead (1000s of bytes) to each ServerHello, so removing this feature will improve connection establishment latency for all users of the TLS server credentials.

Thanks for the reviews @markdroth and @erm-g, I think the PR is ready for another pass.

Thanks for the review Mark!


Feature/azure whisper

Here is my first attempt at creating an AzureOpenAIWhisperParser:

Modelled after the OpenAIWhisperParser.

No tests yet.

Not quite sure how to make check 3.8 work as it says it ran out of disk space?

I've tested to see how the class works when no env variables are set and it works as expected. It checks using os.environ and will throw appropriate errors for each value if it can't find them.
I added a few comments in the class to clarify a bit what needs to be set/what env variables are expected.

Finally got 3 unit tests working for the parser! woooo

Will now quickly work on making a doc for the Parser to fit contribution guidelines


darwin: handle EINTR in /dev/tty workaround

On OS X, special files like /dev/null and /dev/tty don't work with
kqueue.  Libuv falls back to select() in that case but the initial
probe didn't handle EINTR.
Introduced in August 2012 in commit 731adaca ("unix: use select()
for specific fds on OS X"), this bug was only ten days away from
celebrating its fourth birthday.
CI: https://ci.nodejs.org/job/libuv-test-commit/88/

LGTM, hood catch!

LGTM


Fix preprocess.py path

There is no preprocess.py in the root directory, so I assume the correct path must be fairseq_cli/preprocess.py.

Thanks @cifkao for pointing this out. The actual way to solve this is just to call fairseq-preprocess (see https://github.com/ofirpress/attention_with_linear_biases/commit/7c11dc97ef26f3570ffae96afb18aea2daa9cb7b )

@ofirpress OK, thanks. I assume that does the same thing? (I already generated my data by calling fairseq_cli/preprocess.py and it seemed to work fine.)

Yup, it's the same.


Use DB

use database for storing pokemon data instead of json file

Please assign this to me.

Can you clarify which database would you prefer to use?
I found an API that can be called to access pokemon data, will that work, or do you want to have a remote database with all the data that can be accessed via nodejs?


custom rule derived from FileName rule can no longer be disabled in 1.63

If you have a custom FileName rule defined, say Acme/FileName that is derived from the FileName rule, you are now unable to disable it with # rubocop:disable Acme/FileName at the top of the file in version 1.63+. Rubocop is insisting that the violation must be fixed.

Expected behavior
# rubocop:disable Acme/FileName should continue to disable the custom rule on a per file basis as it did in 1.62 and prior.
Actual behavior
It does not disable the rule and insists we fix the violation.
Steps to reproduce the problem
Create a custom rule based on FileName:
module Acme
  class FileName < RuboCop::Cop::Naming::FileName
  # it's enough just to define the class and not actually override or add any behaviour
  end
end

require the rule in your rubocop.yml, then create a file that violates the naming rule and try adding:
# rubocop:disable Acme/FileName
# code here
# rubocop enable Acme/FileName

run: bundle exec rubocop --only Acme/FileName <filename>
Notice the disable does not work.
RuboCop version
1.63.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.2, running on ruby 3.3.0) +server [arm64-darwin23]
  - rubocop-factory_bot 2.25.1
  - rubocop-minitest 0.35.0
  - rubocop-performance 1.21.0
  - rubocop-rake 0.6.0
  - rubocop-sequel 0.3.4

See https://github.com/rubocop/rubocop/pull/12802, which I'm guessing resulted in this change for you. I think global offenses can be disabled only in the cop configuration (you can exclude the file from being checked by the cop), but I may be mistaken about this.

There are quite a few places that do disable Naming/FileName via comment on the first line, see this GH code search: https://github.com/search?q="rubocop%3Adisable+Naming%2FFileName"&type=code
I didn't anticipate this behaviour change, I will open a PR to continue supporting these cases, though I think the config would be a more appropriate place for this.


API key not found, although using custom tiles (OSM)

Is this a bug report?
No (I guess)
Have you read the Installation Instructions?
Yes
Environment
react-native: 0.55.4
react: 16.3.1
react-native-maps: 16.3.1
Android 28 (Emulator)
Steps to Reproduce
Installing with the steps in the instructions with react-native-link.
Expected Behavior
Using custom tiles (OSM) and seeing the map instead of an error for missing Google API Key.
EDIT: Copy+pasted following lines without an actual API Key. No Error messages seems to appear. I have a gray map with google's label at the left bottom corner. It tries to get the tiles from google I guess (hmm).
Actual Behavior
Error:
API key not found.  Check that  is in the  element of AndroidManifest.xml
EDIT: ON iOS I'm getting following error message:
Invariant Violation: Native component for "AIRMapUrlTile" does not exist
Reproducible Demo
      ...
       this.state= {
            urlTemplate: 'https://c.tile.openstreetmap.org/{z}/{x}/{y}.png'
        }
       ....
     <MapView
                region={{latitude: 48.117949,longitude: 7.853377,latitudeDelta: 0.0922,longitudeDelta: 0.0421,}}
                provider={null}
                rotateEnabled={false}
                style={{flex: 1}}
                mapType={"none"}
                showsUserLocation >
                    <MapView.UrlTile urlTemplate={this.state.urlTemplate} />
               </MapView>

@seco35  Custom tiles will still load google lib in the background so you need an api key to work.

@seco35 Custom tiles will still load google lib in the background so you need an api key to work.
Will add an API Key later, thank you. What is the problem with iOS though? Getting an error message for a missing native component of "AIRMapUrlTile"?


Incompatible with typing-extensions 4.6.x

Hi! First of all, thank you for building and maintaining all of these IPLD libraries. They're awesome!
You may have already noticed that typing-extensions v4.6.0 made a change that broke this lib. Specifically, this is what I get when I import bases with it:
>>> import bases
Traceback (most recent call last):
  File ".../bases/__init__.py", line 40, in <module>
    from . import encoding as encoding
  File ".../bases/encoding/__init__.py", line 293, in <module>
    base8 = FixcharBaseEncoding(alphabet.base8, pad_char="=", padding="include")
  File ".../bases/encoding/fixchar.py", line 106, in __init__
    validate(char_nbits, Union[int, Literal["auto"]])
  File ".../typing_validation/validation.py", line 635, in validate
    _validate_union(val, t)
  File ".../typing_validation/validation.py", line 515, in _validate_union
    validate(val, member_t)
  File ".../typing_validation/validation.py", line 691, in validate
    raise unsupported_type_error
ValueError: Unsupported validation for type typing_extensions.Literal['auto'].

Not an emergency. Thanks in advance for looking!

is not compatible with pydantic v2. any updates?

bump @sg495

Blocking us too. Anyone found any workarounds for running Pydantic 2.x on Python <3.11 with this (or multiformats more generally)?

@declension i just write my own and migrated https://github.com/MarshalX/python-libipld

@declension i just wrote my own and migrated https://github.com/MarshalX/python-libipld

Thanks - different use case here I think but maybe I'll start working around too then

@declension i just wrote my own and migrated https://github.com/MarshalX/python-libipld

Thanks - different use case here I think but maybe I'll start working around too then

feel free to contribute more functions that should be bond :)

@sg495 identified the root cause here months ago in https://github.com/hashberg-io/multiformats/issues/10#issuecomment-1581693117 . any chance you all could get a fix for that in sometime soon? thank you in advance!

For what its worth, I've been using the code from pull request https://github.com/hashberg-io/typing-validation/pull/2 for the development version of the application I'm working on (Python 3.11, Pydantic >=2.0), and haven't seen any issues so far.
Having a published version that has this fix would be great though.

Apologies for the delay in this: personal issues have prevented me from keeping up with this over the past few months. I have now merged the pull request: I will run local tests and publish an updated version shortly.

This has now been released on PyPI as v1.1.0.


Problems rendering score on Android.

I have also successfully compiled the lomse library for Android using cocos2d-x.
( It was very hard work :) )
However, it does not render properly as the attached image does.

Did you have a similar case?
There is no problem with iOS.

This problem is solved.
I specified the font path incorrectly.
It works fine


[Bug] Requesting changes to 'Get a Role' (get-role)

Describe the bug
approvalSchemes's revocationRequestConfig for roles is the same as for access profile whereas approvalSchemes's accessRequestConfig is different.
cf. https://developer.sailpoint.com/idn/api/v3/get-role :

Expected behavior
approvalSchemes's revocationRequestConfig for roles should not allow APP_OWNER or  SOURCE_OWNER.
As for approvalSchemes's accessRequestConfig, only OWNER, MANAGER, GOVERNANCE_GROUP should be allowed.

Closing as these updates have been made


Fix ImageStreamListener error due to 1.7.8+hotfix.2 stable version's new API.

I also upgraded 1.7.8 today，I had the same problem

@chavesgu Then you can follow this to update.


@click.native doesn't work for non-custom-component  dynamic component

Version
2.6.10
Reproduction link
https://jsfiddle.net/heaven_xz/h5836dnt/
Steps to reproduce
Reproduce: Just click the content after the timer fire.
What is expected?
The click listener will be bound to the actual component.
What is actually happening?
No listener be bound

你传入的就不是组件类型，谈何“非自定义组件”。你传 div / span or 任何符合约定的 name 结果都一样。

Unfortunately this is because of what code is generated to handle native events listeners: https://template-explorer.vuejs.org/#<component v-bind%3Ais%3D"currentComponent" %40click.native%3D"handleClick">
    inner text node

    <div>
      inner div
    <%2Fdiv>
  <%2Fcomponent>.
Since the variable passed to is could be a component or a native tag, we cannot guess which one to use and putting both would be problematic.
You could make your custom components emit click events by reattaching event listeners to the root with v-on="$listeners"

因为当你传入 div/span这样合法的 tagName 会视为模版元素，而并非是一个组件类型。所以 绑定的 click 被视为自定义事件 。此时你改成这样就可以了：
 <component v-bind:is="currentComponent" @click="handleClick">
   ...
  </component>

如果传入的是组件类型， .native 才会监听到绑定的原生事件。

@posva Thanks for your response

Since the variable passed to is could be a component or a native tag, we cannot guess which one to use and putting both would be problematic.

I don't understand fully, add listener is a runtime process, when I use @click.native, if the is is a component, Vue can add listener to the real DOM of that component renders, if the is is a HTML tag like div, Vue can just add the listener to the 'div', I'm a little confuse about why need to guess(this is not in the compiler phase)

You could make your custom components emit click events by reattaching event listeners to the root with v-on="$listeners"

For the custom component case, @click.native works fine, the issue case I described is for the HTML tag case.
After read the @xingleizhao's  comment, I find that if I add both @click and @click.native like <component v-bind:is="currentComponent" @click.native="handleClick" @click="handleClick">...</component>, this will works for custom component and HTML tag, but I'm not sure if it's a correct/legal way to use or has some potential bugs


New Update

When the new update with new deadzone will came out ?

It's already out. Just download it.

DavidRieman said that he will improve the original deadzone but the source code is not yet updated

Well the code is available at https://github.com/DavidRieman/XboxKeyboardMouse (master branch).  But I am hoping the others will respond soon WRT releasing via the ClickOnce installer.

Ok, the new version is available!
Per the git homepage it is available at: https://cdn.rawgit.com/fqlx/XboxKeyboardMouse/master/Installer/setup.exe

Let us know if you have any issues with the new installer / deadzone aiming.  Closing issue.


util/rhsm: Check if repositories is None before iterating

When get_fallback_rhsm_secrets was used, Subscriptions.repositories
was None, and get_secrets never returned the fallback secrets.
So check if repositories is None before
iterating over it, otherwise return the fallback secrets.

Thanks for this! We should add tests too, maybe open an issue?
@msehnout maybe something to back port to 8.4/8.5/9-beta?

Note that this was on a rhel8.3 machine, I have no idea if this issue exists on rhel 8.4/8.5

I just checked on the cloud rhel-8.4 image we use for the service and now it's obvious why it even gets to the fallback:
[ec2-user@ ~]$ ls /etc/yum.repos.d/
composer.repo  osbuild.repo  redhat-rhui-beta.repo.disabled  redhat-rhui-client-config.repo  redhat-rhui.repo

So this is an official rhel 8.4 cloud access image from may.


fix: small fix lint

Proposal

Fix lint
FIx error when env not found

Codecov Report

Merging #32 (5e61775) into main (119c599) will increase coverage by 0.08%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   77.95%   78.04%   +0.08%
==========================================
  Files          45       45
  Lines        1143     1143
==========================================
+ Hits          891      892       +1
+ Misses        252      251       -1

Impacted Files
Coverage Δ

src/config/kafka.rs
0.00% <0.00%> (ø)

src/config/env_var.rs
84.00% <100.00%> (ø)

src/core/usecase/charge/charge_create_usecase.rs
95.00% <0.00%> (+2.50%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Error: Failed to resolve entry for package "csstype". The package may have incorrect main/module/exports specified in its package.json.

Describe the bug
After adding Hope-UI v1.0.0-next.7 to a  SolidJS 1.6.0 / solid-start 0.2.0 like described in the tutorial, running npm run dev results in the following error:
error when starting dev server:
Error: Failed to resolve entry for package "csstype". The package may have incorrect main/module/exports specified in its package.json.
    at packageEntryFailure (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:34096:11)
    at resolvePackageEntry (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:34093:5)
    at tryNodeResolve (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:33834:20)
    at Context.resolveId (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:33595:28)
    at Object.resolveId (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:40152:46)
    at async file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:62476:21
    at async addManuallyIncludedOptimizeDeps (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:41601:31)
    at async createDepsOptimizer (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:40793:9)
    at async initDepsOptimizer (file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:40707:9)
    at async file:///gringofe/coding/test-solid-start123/node_modules/vite/dist/node/chunks/dep-51c4f80a.js:61877:17

To Reproduce
Steps to reproduce the behavior:

Clone my  demo repo
Run npm run dev in the directory

Expected behavior
No error

Also getting the exact same issue!

Fixed in 1.0.0-next.8


Add better support for debugging python code

It's incredibly helpful to be able to set the log level to info or debug and watch the log file as you run commands.

Thanks! Any idea why the checks are failing? I haven't had a chance to look yet.

Yeah, looks like I just needed to mock the g:OmniSharp_loglevel variable in the tests

Great, this is going to come in really handy!


Implement Iterator for Poll

A typical use case of Poll is:
let mut poll = Poll::new()?;
let mut events = Events::with_capacity(128);

// ...

loop {
    poll.poll(&mut events, timeout)?;

    for event in events.iter() {
        match event.token() {
            token1 => { ... }
            token2 => { ... }
        }
    }
}

It would be better to implement Iterator<item=io::Result<&Event>> and Iterator<item=io::Result<&Events>> for Poll.
The implementation may require GAT, which is expected to be stabilized soon.

I don't think it's a good idea to hide a system call behind an Iterator implementation. Furthermore how would you pass the timeout? Also once you get away from the most basic example the Iterator implementation wouldn't be useful any more...

Yeah, I don't really think that this is a good idea, largely for the reasons Thomas mentioned above.

@cjwcommuny I'm going to close this for now, if you still feel strongly about this we can reopen.


EEP: Milestones to Embulk v1.0, and versioning strategies to follow

The Embulk core has completed the huge architectural transition through the Embulk v0.10 "development" series.
The stable production versions, v1.0, have come into our view, but we have not been very clear about when to declare v1.0.
The goal of this EEP is to clarify the milestones for Embulk v1.0, and to suggest future versioning strategies to follow v1.0.

Rendered version: https://github.com/embulk/embulk/blob/eep-milestones-to-v1.0/docs/eeps/draft-milestones-to-v1.0-and-versioning-strategies.md

Jfyi, it's draft yet. More updates would come, but nice to hear that it looked good. :)

Oh Draft.., I was half asleep.

This would be a big milestone. I'm wondering if I can get as many feedbacks as possible from @embulk/core-team members. Looking forward to your comments!

We observed no concerns. Finalizing this as EEP-8 to be Accepted. (Not "Final" because we have not reached to v1.0.)


How to get Firebasestore file extension

How to get Firebasestore uploaded file extension. Like link I added product with image when I get the data that time How can  I show Product details with image

Hi @polashrp
This platform is not met for assistance on personal code.,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with FlutterFire itself,
if you disagree please write in the comments and I will reopen it
Thank you


server/api: check the legality of the label

What have you changed? (required)

check the legality of the label. some careless user use pdctl  like:
pdctl >> config set location-labels host;

and it will bet set successfully.
What are the type of the changes (required)?

Improvement (non-breaking change which is an improvement to an existing feature)\

How has this PR been tested (required)?

unit test

LGTM.

/rebuild

LGTM


Alternate Keys aren't supported in the Target of a Pre-Validation/Operation Plugin

It should probably called out that attempting to set a lookup attribute to an EntityReference for the target in the Pre-Operation or Pre-Validation of a plugin is not supported.  It will error in the Pre-Validation, and create a reference to 00000000-0000-0000-0000-000000000000 in the Pre-Operation

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: cd844db9-ff8f-0573-1596-bb31d6d4609a
Version Independent ID: cd844db9-ff8f-0573-1596-bb31d6d4609a
Content: Use an alternate key to reference a record (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/use-alternate-key-reference-record.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @divka78
Microsoft Alias: dikamath

Thanks for your comment, @daryllabar. This issue has been assigned to the doc author for review.

@daryllabar
This seems like issue that needs to be examined by the dev team.
I see you have opened a support case. Let's see how that gets resolved and whether we need to add a note to it in the docs.
Closing this.


swap_file resource doesn't set update status

Hi Seth,
it seems swap_file resource doesn't  set resource status whenever it gets updated during Chef run. I've briefly scrolled through the provider, but didn't find any references to updated_by_last_action or converge_by there.
Recipe:
swap_file '/swapfile' do
  size  2048
  persist true

  action :create
end

Chef run output (debug enabled):
 * swap_file[/swapfile] action create[2016-02-26T11:36:15+00:00] INFO: Processing swap_file[/swapfile] action create (cognifide-base::swap line 19)
[2016-02-26T11:36:15+00:00] DEBUG: Providers for generic swap_file resource enabled on node include: []
[2016-02-26T11:36:15+00:00] DEBUG: No providers responded true to `supports?` for action create on resource swap_file[/swapfile], falling back to enabled handlers so we can return something anyway.
[2016-02-26T11:36:15+00:00] DEBUG: Dynamic provider resolver FAILED to resolve a provider for action create on resource swap_file[/swapfile]
[2016-02-26T11:36:15+00:00] WARN: Class Chef::Provider::SwapFile does not declare 'provides :swap_file'.
[2016-02-26T11:36:15+00:00] WARN: This will no longer work in Chef 13: you must use 'provides' to use the resource's DSL.

[2016-02-26T11:36:15+00:00] DEBUG: swap_file[/swapfile] filesystem listing is '
'
[2016-02-26T11:36:15+00:00] DEBUG: swap_file[/swapfile] dd command is 'dd if=/dev/zero of=/swapfile bs=1048576 count=2048'
[2016-02-26T11:36:15+00:00] DEBUG: swap_file[/swapfile] swap creation command is 'dd if=/dev/zero of=/swapfile bs=1048576 count=2048'
[2016-02-26T11:36:15+00:00] DEBUG: swap_file[/swapfile] dd command is 'dd if=/dev/zero of=/swapfile bs=1048576 count=2048'
[2016-02-26T11:36:15+00:00] DEBUG: swap_file[/swapfile] fallback swap creation command is 'dd if=/dev/zero of=/swapfile bs=1048576 count=2048'
Filename                Type        Size    Used    Priority
/dev/dm-1                               partition   950268  0   -1
2048+0 records in
2048+0 records out
2147483648 bytes (2.1 GB) copied, 3.8416 s, 559 MB/s
[2016-02-26T11:36:19+00:00] INFO: swap_file[/swapfile] Creating empty file at /swapfile
[2016-02-26T11:36:19+00:00] DEBUG: swap_file[/swapfile] Empty file at /swapfile created using command 'dd if=/dev/zero of=/swapfile bs=1048576 count=2048'
[2016-02-26T11:36:19+00:00] INFO: swap_file[/swapfile] Set permissions on /swapfile to 600
Setting up swapspace version 1, size = 2097148 KiB
no label, UUID=9d76a8bf-f5e6-4e92-8b9b-94e7e2c2ffc1
[2016-02-26T11:36:19+00:00] INFO: swap_file[/swapfile] /swapfile made swappable
[2016-02-26T11:36:19+00:00] INFO: swap_file[/swapfile] Swap enabled for /swapfile
[2016-02-26T11:36:19+00:00] INFO: swap_file[/swapfile] adding entry to /etc/fstab for /swapfile
(up to date)
...
Chef Client finished, 0/1 resources updated in 10 seconds

Cheers,
Jakub

can you test this with 2.0 release and let us know.

closing, reopen if this is an issue.


🐛 disableFrameProcessors always set to TRUE

What were you trying to do?
i wanted to use frameprocessor but in my gradle.properties config its always set to TRUE
# Project-wide Gradle settings.

# REACT NATIVE VISION CAMERA
disableFrameProcessors=false

is this the problem while build?
https://github.com/mrousavy/react-native-vision-camera/blob/c046440e0e62b6e2416610ab216583bc47544c4c/android/build.gradle#L115
log output say this
https://github.com/mrousavy/react-native-vision-camera/blob/c046440e0e62b6e2416610ab216583bc47544c4c/android/build.gradle#L122
Reproduceable Code
comment this config

# disableFrameProcessors=true

What happened instead?
https://github.com/mrousavy/react-native-vision-camera/blob/c046440e0e62b6e2416610ab216583bc47544c4c/android/build.gradle#L122
Relevant log output
https://github.com/mrousavy/react-native-vision-camera/blob/c046440e0e62b6e2416610ab216583bc47544c4c/android/build.gradle#L122

Device
Android Emulator
VisionCamera Version
2.15.2
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

https://github.com/mrousavy/react-native-vision-camera/issues/1283#issuecomment-1354796863


Error for undefined method after do, end points to the wrong line

def method
  yield
end

method do
end.undefined_method

gives this error with the green pointer at the wrong line:
Error in spec/spec_helper.cr:5: undefined method 'undefined_method' for Nil

method do
    ^~~~~~~~~~~~~~~~

Rerun with --error-trace to show a complete error trace.

but a one-line bracket block
def method
  yield
end

method {}.undefined_method

gives a correct error:
Error in spec/spec_helper.cr:5: undefined method 'undefined_method' for Nil

method {}.undefined_method
          ^~~~~~~~~~~~~~~~

Rerun with --error-trace to show a complete error trace.

In general this happens when the call name is on a separate line than the receiver:
1
  .bar

Output:
Error in foo.cr:1: undefined method 'bar' for Int32

1
   ^~~


Add a default font

Adds a default font under a new "bevy_default_assets" feature flag. Used FiraSans-Bold since it's used in most of the examples as the default font. Resolves #1017

I'm not yet completely sold on this implementation. This creates a situation where all Bevy apps come packaged with a default font, unless they specifically opt out. I expect that "cargo features" generally aren't something most people will think about when publishing their game, so this will likely result in some high percentage of Bevy apps including extra font bytes (90 percent? 99 percent?). Imo we should be as lean as possible by default.
Some alternatives to discuss:

Bundle this into the (yet to be implemented) Bevy Editor instead, with some gui tool that makes it easy to add "default" fonts to a project's asset folder
Make this a separate crate (not provided by the default bevy distribution) and provide a plugin that can be easily dropped in to apps. The plugin could set the "default" font asset, which should make Bundles with font assets work by default.
Choose some hyper-minimalist font that takes up no space.

Load font from the system the same way most programs do.

Load font from the system the same way most programs do.

font-kit would let us do that, but currently has many dependencies/parts we don't need
see https://github.com/servo/font-kit/issues/180

Closing this out as I think the other options (1-4) are all preferable.