HStoreExtension is missin in 0001_initial.py

What happened?
If you use the postgresql specific model field HStoreField() after creating a 0001_initial.py file does not contain the HStoreExtension() for migration.
What should've happened instead?
If you use HStoreField(), the 0001_initial.py has to contain the HStoreExtions() in operations[HStoreExtension(), migration.CreateModel.....]
Steps to reproduce

create a model with postgres field like "param =  django.contrib.prostgres.fields.HStoreField()"
run makemigration
run migrate to create a 0001_initial.py

Ubuntu 18.04
docker client community 19.03.8
docker engine community 19.03.8
docker compose 1.25.5
docker-py 4.1.0
cpython 3.7.5
pycharm prof 2020.1 build 30 April 2020
cookiecutter  Latest commit 606b4c5 May 13, 2020
//: # (* Replay file https://cookiecutter.readthedocs.io/en/latest/advanced/replay.html)
//: # (* Logs)

Terminal output after running migrate:
(venv)$ docker-compose -f local.yml run --rm django python manage.py migrate reporter
Starting postgres ... done
PostgreSQL is available
Operations to perform:
Apply all migrations: reporter
Running migrations:
Applying reporter.0001_initial...Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql)
psycopg2.errors.UndefinedObject: type "hstore" does not exist
LINE 1: ...e" varchar(100) NOT NULL, "environment_variables" hstore NOT...
^
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "manage.py", line 31, in
execute_from_command_line(sys.argv)
File "/usr/local/lib/python3.8/site-packages/django/core/management/init.py", line 401, in execute_from_command_line
utility.execute()
File "/usr/local/lib/python3.8/site-packages/django/core/management/init.py", line 395, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 328, in run_from_argv
self.execute(*args, **cmd_options)
File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 369, in execute
output = self.handle(*args, **options)
File "/usr/local/lib/python3.8/site-packages/django/core/management/base.py", line 83, in wrapped
res = handle_func(*args, **kwargs)
File "/usr/local/lib/python3.8/site-packages/django/core/management/commands/migrate.py", line 231, in handle
post_migrate_state = executor.migrate(
File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 117, in migrate
state = self._migrate_all_forwards(state, plan, full_plan, fake=fake, fake_initial=fake_initial)
File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 147, in _migrate_all_forwards
state = self.apply_migration(state, migration, fake=fake, fake_initial=fake_initial)
File "/usr/local/lib/python3.8/site-packages/django/db/migrations/executor.py", line 245, in apply_migration
state = migration.apply(state, schema_editor)
File "/usr/local/lib/python3.8/site-packages/django/db/migrations/migration.py", line 124, in apply
operation.database_forwards(self.app_label, schema_editor, old_state, project_state)
File "/usr/local/lib/python3.8/site-packages/django/db/migrations/operations/models.py", line 92, in database_forwards
schema_editor.create_model(model)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/schema.py", line 324, in create_model
self.execute(sql, params or None)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/base/schema.py", line 142, in execute
cursor.execute(sql, params)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 100, in execute
return super().execute(sql, params)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 68, in execute
return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 77, in _execute_with_wrappers
return executor(sql, params, many, context)
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 86, in _execute
return self.cursor.execute(sql, params)
File "/usr/local/lib/python3.8/site-packages/django/db/utils.py", line 90, in exit
raise dj_exc_value.with_traceback(traceback) from exc_value
File "/usr/local/lib/python3.8/site-packages/django/db/backends/utils.py", line 84, in _execute
return self.cursor.execute(sql)
django.db.utils.ProgrammingError: type "hstore" does not exist
LINE 1: ...e" varchar(100) NOT NULL, "environment_variables" hstore NOT...

The django docs are clear about [this]:(https://docs.djangoproject.com/en/3.0/ref/contrib/postgres/fields/#django.contrib.postgres.fields.HStoreField):

Add 'django.contrib.postgres' in your INSTALLED_APPS.
Setup the hstore extension in PostgreSQL.

So it is your own responsibility to create the migration containing the HStoreExtension.

I tried the approach like foarsitter and django docs said. It did not work.
It is necessary to add HStoreExtension() to 0001_inital.py
operations = [         HStoreExtension(),         migrations.CreateModel(


fix: introduce Filler

:tada: This PR is included in version 0.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Merge ft/GCP to rel/7.4-beta

@ironman-machine
DEFAULT_BRANCH=rel/7.4
SCALITY_FEDERATION_BRANCH=ft/gcp-to-7.4-beta
SCALITY_INTEGRATION_BRANCH=ft/gcp-to-7.4-beta
try

Hello @alexanderchan-scality
"DEFAULT_BRANCH=rel/7.4": Success
"SCALITY_FEDERATION_BRANCH=ft/gcp-to-7.4-beta": Success
"SCALITY_INTEGRATION_BRANCH=ft/gcp-to-7.4-beta": Success
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/20461' with the following env. args:
{
    "REPO_NAME": "S3",
    "DEFAULT_BRANCH": "rel/7.4",
    "SCALITY_INTEGRATION_BRANCH": "ft/gcp-to-7.4-beta",
    "SCALITY_FEDERATION_BRANCH": "ft/gcp-to-7.4-beta",
    "SCALITY_S3_BRANCH": "ft/gcp-to-7.4-beta"
}

@ironman-machine
DEFAULT_BRANCH=rel/7.4
SCALITY_FEDERATION_BRANCH=ft/gcp-to-7.4-beta
SCALITY_INTEGRATION_BRANCH=ft/gcp-to-7.4-beta
try

Hello @alexanderchan-scality
"DEFAULT_BRANCH=rel/7.4": Success
"SCALITY_FEDERATION_BRANCH=ft/gcp-to-7.4-beta": Success
"SCALITY_INTEGRATION_BRANCH=ft/gcp-to-7.4-beta": Success
"try": Success: Try build successfully launched on 'http://ci.ironmann.io/gh/scality/Integration/20472' with the following env. args:
{
    "REPO_NAME": "S3",
    "DEFAULT_BRANCH": "rel/7.4",
    "SCALITY_INTEGRATION_BRANCH": "ft/gcp-to-7.4-beta",
    "SCALITY_FEDERATION_BRANCH": "ft/gcp-to-7.4-beta",
    "SCALITY_S3_BRANCH": "ft/gcp-to-7.4-beta"
}

:sunny: :+1: circleCI test succeeded!


Add optional icon to GridCellText

PR Checklist

[x] Verify if the change requires a changeset. See CONTRIBUTING.md
[x] Link to the respective task if one exists: SVK-264
[x] Provide screenshots/screencasts if the change contains visual changes

Previously

Now

Now that we support an icon, is GridCellText still the best name? 😁

Maybe GridCellContent or just GridCell?
Not sure if the primary and secondary names are the best names then though 😅

GridCellContent

:+1:

Not sure if the primary and secondary names are the best names then though

yes, not the best - alternative ideas?

yes, not the best - alternative ideas?

primaryText and secondaryText

@jamesricky please rebase


Changed the color of Add-Patient Icon to White

I have changed the color of the Add-Patient button from blue to white.

Do have a look and please tell me if anything else is required.
Thank you.

What's the reason for closing this @ujjwaltyagi355 ?

@FlorianRappl,
After merging and everything I was not sure if the package-lock.json, is reverted back to the original state or not. So I just closed this.
Sorry for the inconvenience.


Let IDE(s) know public symbols

🚀 Feature (or Bug?)
I'm using Pylance on VSCode, and it doesn't autocomplete, for example, Accuracy even if I typed the following:
from torchmetrics import Ac

This is because imported symbols are considered private by default (by at least pylance (pyright) as documented here).
Motivation
For better coding experience (autocomplete + docstring reference) :]
Pitch
As stated in https://github.com/microsoft/pyright/blob/main/docs/typed-libraries.md#library-interface, we have the following options to explicitly make symbols public.
1. Add __all__ to __init__.py files.
# torchmetrics/__init__.py
from torchmetrics.classification import (
    Accuracy,
    ...
)
__all__ = [
    "Accuracy",
    ...
]

2. Use wildcard imports.
# torchmetrics/__init__.py
from torchmetrics.classification import *

# torchmetrics/classification/__init__.py
__all__ = [
    "Accuracy",
]

3. Use redundant symbol aliases.
# torchmetrics/__init__.py
from torchmetrics.classification import (
    Accuracy as Accuracy,
    ...
)

Additional context
https://github.com/microsoft/pylance-release/issues/1032
https://github.com/microsoft/pyright/pull/1877

@akihironitta sounds good to me :]
I would prefer option 1 but have no strong opinions here.


GRPC fails to build on Windows using cmake/Visual Studio

What version of gRPC and what language are you using?
v1.11.0 / c++
What operating system (Linux, Windows, …) and version?
Windows
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio 15 2017 (latest install of Visual Studio 2017 Community w/ C++ tools)
What did you do?
If possible, provide a recipe for reproducing the error. Try being specific and include code snippets if helpful.
I followed the INSTALL instructions:
> md .build
> cd .build
> cmake .. -G "Visual Studio 14 2015" -DCMAKE_BUILD_TYPE=Release
// At this point I open Visual Studio and build grpc/grpc++ as these are the only two pieces I need.

What did you expect to see?
Build successful / no errors
What did you see instead?
Lots of errors like this:
Error C2220 warning treated as error - no 'object' file generated	ssl c:\users\max\documents\grpc\third_party\boringssl\ssl\ssl_lib.cc	3461
It outputs grpc.lib/grpc++.lib, but my own cmake project bails when building grpc as a dependency because it outputs errors.
I would just disable treating warnings as errors, but I imagine there's a good reason that's turned on for boringssl.
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
Anything else we should know about your project / environment?
nope

Try turning off treating warnings as errors. You're building BoringSSL as a dependency, and it doesn't build cleanly without warnings under Windows.

there shouldn't be any errors: we are testing VS2017 build continuously and the tests are passing very consistently:
https://source.cloud.google.com/results/invocations/4b8299f2-06e5-4cbb-989e-81e8c54f24ab/targets
Can't there be a problem in your own cmake file? (whether your cmake file can influence the grpc build also depends on the way you are including grpc as a dependency).
btw, the error msg you provided is not that useful without the actual warning (that's also appears in your log, but is not mentioned here).

After running into the error in my cmake file, I tried building it by hand using the instructions from INSTALL. I'm using the exact instructions I put into the bug report above in vanilla gRPC checkd out at the v1.11.0 tag with all submodules.
Also, I assume the warning is "no 'object' file generated" meaning that there are a bunch of cc files that don't have corresponding .o files which I assume means that on windows those source file does not produce any symbols.
I'll be back at my Windows PC later this week. I'm curious to see why ssl_lib.cc in boringssl wouldn't produce any symbols when building for Windows.

Also, I assume the warning is "no 'object' file generated" meaning that there are a bunch of cc files that don't have corresponding .o files which I assume means that on windows those source file does not produce any symbols.

That's not what this means at all. This is the general message from Visual Studio that says something went wrong compiling this one file.

I'm on a different computer this week. I did this in a fresh grpc repo and it worked fine... Going to debug more when I get back on my other computer.


Filter all data from one Scenario by a set element; add utils.diff()

This PR adds features to get data from every parameter in an ixmp.Scenario that matches some filters, e.g. a single value on a single dimension. In message_ix this makes it possible to get all parameter data related to a single technology, i.e. iiasa/message_ix#267; but the functionality is generalized here so users can find other applications for it.
How to review

Read the new tests, including test_diff and ensure they cover easily imaginable use cases.
Note that CI checks, including new tests, pass.
Run the new CLI command against some existing, “real world” scenarios, e.g. things like:$ message-ix --url="ixmp://my-platform/my-model/my-scenario" export example.xlsx -- technology=coal_ppl,wind_ppl

PR checklist

[x] Add basic functionality to return data across all parameters.
[x] Modify to_excel() to accept filters.
[x] Add 2-scenario "diff" utility per https://github.com/iiasa/message_ix/pull/267#issuecomment-623395698.
[x] Add tests for all of the above.
[x] Add documentation.
[x] Update release notes.
[x] Boost code coverage: test when one param is missing from each scenario passed to diff().

Hi @ClaraLuisa @behnam-zakeri, I'm requesting your review of this code.
I think I've managed to preserve a superset of the features that you (respectively) provided and requested in iiasa/message_ix#267; however, the code is now usable for any dimension(s) of any ixmp Scenario. Please see the "How to review" in the description above.
Based on your feedback or questions, I will then update the documentation so the functionality and how to use it is made clear.

Based on your feedback or questions

Seeing none, I'll merge this. If the feature could be enhanced somehow, please open an issue to describe how.

Thanks a lot @khaeru for this piece of work, which is very useful. Sorry for the late feedback. As mentioned in the meeting, it will be useful to add a line or two to show how the functionality can be used in a script without exporting the data to Excel.

it will be useful to add a line or two to show how the functionality can be used in a script without exporting the data to Excel.

Since this PR is already merged, can you open a new issue for this improvement?


Fix parser N/As

cclib 1.5.3 should alleviate these issues, as soon as this PR is merged.

While we wait, there's an option to hide N/A rows :)


run migration

INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 2e561a42af25, Initial migration
INFO  [alembic.runtime.migration] Running upgrade 2e561a42af25 -> ebe551f9e7e9, Add index to promo_code
Traceback (most recent call last):
  File "D:\PycharmProjects\venv\lib\site-packages\sqlalchemy\engine\base.py", line 1967, in _exec_single_context
    self.dialect.do_execute(
  File "D:\PycharmProjects\venv\lib\site-packages\sqlalchemy\engine\default.py", line 941, in do_execute
    cursor.execute(statement, parameters)
psycopg2.errors.UndefinedTable: relation "bike_ride_3d" does not exist

INFO  [alembic.runtime.migration] Context impl PostgresqlImpl.
INFO  [alembic.runtime.migration] Will assume transactional DDL.
INFO  [alembic.runtime.migration] Running upgrade  -> 2e561a42af25, Initial migration
INFO  [alembic.runtime.migration] Running upgrade 2e561a42af25 -> ebe551f9e7e9, Add index to promo_code
Traceback (most recent call last):
  File "D:\PycharmProjects\venv\lib\site-packages\sqlalchemy\engine\base.py", line 1967, in _exec_single_context
    self.dialect.do_execute(
  File "D:\PycharmProjects\venv\lib\site-packages\sqlalchemy\engine\default.py", line 941, in do_execute
    cursor.execute(statement, parameters)
psycopg2.errors.UndefinedTable: relation "bike_ride_3d" does not exist

Fixed it in the new release ✅


Question about which CLI to use and how to get the binary of the latest version

Hi @chanwit,
I noticed that the gitops CLI has lots of overlapping commands with tfctl. Which one is the recommended CLI?
Also, I use curl -s https://fluxcd.io/install.sh | sudo bash (from here) in combination with flux install --export on a GitHub Workflow that runs on a schedule for periodically checking for Flux upgrades and opening pull requests when a new version is available, i.e. when flux install --export > gotk-components.yaml produces a diff. I want to do the same for tf-controller. Right now we have the v0.15.0-rc.1 release/tag but it doesn't contain the compiled tfctl binary, therefore I'm kinda stuck with the tf-controller Helm chart to get the latest version right now.
It would be really cool if tf-controller integrated well, for example, with this binary manager:
https://github.com/marcosnils/bin
So when I run bin install https://github.com/weaveworks/tf-controller I get the binary of the latest version. But right now this is what happens:
➜  terraform git:(main) bin install https://github.com/weaveworks/tf-controller
   • Getting latest release for weaveworks/tf-controller
   ⨯ command failed            error=repository weaveworks/tf-controller does not have releases
➜  terraform git:(main)

But it does work for the gitops CLI:
➜  terraform git:(main) bin install https://github.com/weaveworks/weave-gitops
   • Getting latest release for weaveworks/weave-gitops
   • Starting download of https://api.github.com/repos/weaveworks/weave-gitops/releases/assets/107631273
36.48 MiB / 36.48 MiB [--------------------------------------------------------------------------------------------------------------------------------------------------------------------] 100.00% 13.53 MiB p/s 3s

Multiple matches found, please select one:

 [1] LICENSE
 [2] README.md
 [3] gitops
 Select an option: 3
   • Copying for<EMAIL_ADDRESS>into /home/pimenta/.local/bin/gitops
   • Done installing gitops v0.23.0
➜  terraform git:(main)

This or something like curl -s https://fluxcd.io/install.sh | sudo bash would be amazing!

Hi @aristapimenta
Sorry for late reply.
tfctl is deprecated and will be served as a development tool only.
We're moving towards integrating all Terraform commands into Weave GitOps (gitops) CLI.
Thing's still a bit messy at the moment, so please stay tuned.

Thanks @chanwit!


Additional chromeos key mappings

(From my comment in #573)
The documentation can be found here: https://support.google.com/chromebook/answer/183101?hl=en
Text editing / Delete the next letter (forward delete): Alt + Backspace
Page & web browser / Page up : Alt + Up arrow
Page & web browser / Page down : Alt + Down arrow
Page & web browser / Go to top of page: Ctrl + Alt + Up arrow
Page & web browser / Go to bottom of page: Ctrl + Alt + Down arrow

Thanks. Merged as 9132184. In the future, please prepend doc: to the commit message for consistency.


Group nav changes

Summary of the changes
Group-navigation has been changed to apply new WCAG features;
.5 second delay when hovering a grouped-nav-item (when mouse is idle) before dropdown.
delay not applied when mouse enters sibling grouped-nav-item.
dropdown can be cancelled with escape, space or enter (without focus).
.5 second delay after mouse leaves grouped-nav-item before dropdown is closed
Related issue
#674
Checklist

[ ] I have added relevant unit and visual regression tests.
[ ] I have manually accessibility tested any changes, if relevant.
[ ] I have ensured any changes match the Figma component library.

getting some console errors in a couple of functions: triggerShowDropdown & handleMouseLeave

Just a little comment on the behaviour. It seems a bit strange to me that the menu pops up after 0.5 seconds if I accidentally mouse over the nav group, or if I move my mouse onto a nav group item (when the menu\subitems are not visible) then quickly off. Just want to check with @DBD324 that is the intended behaviour.

That's not intended behaviour, the mouseIdle function should be catching it and only opening once the mouse has been idle on the element for 0.5, I'll take a look and see what's happening


feat: Blueprint for VPC lattice with kubernetes api controllers

Description
Provisioning VPC Lattice and use the API gateway controller to connect services across EKS clusters

Motivation and Context

Resolves #1638

How was this change tested?

[ ] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Yes, I have updated the docs for this feature
[x] Yes, I ran pre-commit run -a with this PR

Additional Notes

Thanks for updated, is it working on your side ? My targets groups are not able to join the pods on my side

Thanks for updated, is it working on your side ? My targets groups are not able to join the pods on my side

let me test it again

As it turns out the current version of  EKS blueprint addon needs extra permissions for the aws gateway api controller to create httproutes , i have already issued a PR to fix it
could someone please look into it , thanks
tracked issue

@allamand @bryantbiggs I have introduced new workloads for the VPC lattice patterns with the gateway API controllers, Could you please review them :D

@allamand  please let me know if my latest changes are sufficient to merge this PR , thanks

@neelaruban rhere was an issue with VPC lattice controller v1.0.2 which prevented service import to work properly. This is now fixed in latest release v1.0.3. In the meantime we actually have 2 PRs for lattice. We are working to have different use cases in same pattern starting with the other one which is simpler. In my opinion the real value will be to focus on cross cluster flows with TLS encryption and IAM authorizations.
What I suggest would be to adapt your actual pattern to be a use-case dedicated to microservices flows sécurisation. (Merge the code as new use case in the other VPC lattice pattern, and add examples for iam auth). I can help with this


Release docker image for arm64

The following file has been modified:
Modified file .github/workflows/ci.yml to build and push the image for both amd64 and arm64 platforms.
Signed-off-by: odidev<EMAIL_ADDRESS>

LGTM. @mervinkid  please approve running workflows.

@odidev Some checks were not successful

Follow Settings => Secrets => New repository secret , we need create DOCKER_TOKEN as docker password.

@odidev Some checks were not successful

@leonrayang
Publish job is failing as you need to generate token for your own docker repository and store it as secret key in Github Actions. Please refer the following document to generate the token -https://docs.docker.com/ci-cd/github-actions/#:~:text=Add your Docker ID as,Personal Access Token (PAT).
The LTP test which is failing here, is passing on my Github actions. For more details, please check here.
It looks like the test result is not consistent and I think failures are not related to these changes.

@odidev Some checks were not successful

@leonrayang
Publish job is failing as you need to generate token for your own docker repository and store it as secret key in Github Actions. Please refer the following document to generate the token -https://docs.docker.com/ci-cd/github-actions/#:~:text=Add your Docker ID as,Personal Access Token (PAT).
The LTP test which is failing here, is passing on my Github actions. For more details, please check here.
It looks like the test result is not consistent and I think failures are not related to these changes.

I see, the pr will be managed by mervin who familiar with the process

Hi team,
Can you please review this PR and let me know if any changes is required.

Hi team,
Can you please review this PR and let me know if any changes is required.

@mervinkid  please help to review this pr, thank you.

@odidev please resolve the conflicts


⚠ Media Fetcher has degraded performance

In 47eec17, Media Fetcher ($SITES_URL_DOWNLOADS) experienced degraded performance:

HTTP code: 525
Response time: 21681 ms

Resolved: Media Fetcher performance has improved in 501612b after 9 minutes.


Order products list alphabetically

Do it on the service. The caller should receive the data already ordered.


Fatal Error when sending email

PHP Fatal error:  Uncaught Error: Call to undefined method SimpleMail::setentity()

There is no setentity() method on SimpleMail, it must be an issue with your own code. Submit more information and I'll help you debug.

False alarm, closed the bug.


YouTube-dl has received a DMCA takedown from RIAA. Will it affect this repo in the future?

YouTube-dl has received a DMCA takedown from RIAA (github.com/github)

Hi.
It's still not clear how the situation will resolve itself for youtube-dl, but in any case it's concerning.
At the very least, it seems like the screenshots in the readme should be updated to not show Imagine Dragons. :)

Thanks for answering & thanks again for useful tool


Document how to write tests for a custom module

Maybe a standalone page, but also could just be another chapter in the kitties workshop.
@Kianenigma is this still something you want to do? If so, will you have bandwidth in the next 2 weeks to do it?

I can take this one on likely this weekend. Have some initial notes started on this already. Would be good to get it reviewed by Kian and Shawn later though.
It would also be good as another kitties chapter - don't see why not

It doesn't have to be a kitties chapter if you think it would fit some other format better. But I do think you want to start with some "pre-written" runtime module, and might as well start with this one?

@nczhu Like some other attempts that I've had to take stuff that I think I can properly do, it looks like I am a much better asset at helping the doc team with the bit of knowledge that I have rather than doing it myself :D. In case you decide to take this, I would be happy to help/review.

@Kianenigma If Nicole is interested in it, and you have other stuff to do, I think you simply helping and reviewing would be awesome enough :)
Maybe you and her can talk if you want to actually team up and tackle it!

pr out for this here:
https://github.com/shawntabrizi/substrate-collectables-workshop/pull/100
Eventually would still be great to have a quick reference or recipes page on Tests that live as its own page in docusaurus

The PR is merged now, so I'm going to close this issue.

Eventually would still be great to have a quick reference or recipes page on Tests that live as its own page in docusaurus

@nczhu Here's an issue for that. https://github.com/paritytech/substrate-developer-hub/issues/86
Wanna team code sometime next week? I've assigned us, but feel free to unassign yourself and I can take it.

Sure, let’s do it. should be quick -one hour?
On Thu, May 30, 2019 at 8:30 PM Joshy Orndorff<EMAIL_ADDRESS>wrote:

The PR is merged now, so I'm going to close this issue.
Eventually would still be great to have a quick reference or recipes page
on Tests that live as its own page in docusaurus
@nczhu https://github.com/nczhu Here's an issue for that. #86
https://github.com/paritytech/substrate-developer-hub/issues/86
Wanna team code sometime next week? I've assigned us, but feel free to
unassign yourself and I can take it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/paritytech/substrate-developer-hub/issues/81?email_source=notifications&email_token=AFS4DLNQZMA3QEZSZ3IYVALPYAMLLA5CNFSM4HLACXF2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWTEEUI#issuecomment-497435217,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFS4DLIUGFHLZ4WGWI4KO2DPYAMLLANCNFSM4HLACXFQ
.

--
This communication and any attachments are confidential.


Check connection state before give it up

Description of the bug
Currently, once Vaadin receives a offline event, it doesn't trying to reconnect, instead giving up the connection immediately, as in DefaultConnectionStateHandler.java
private void registerConnectionStateEventHandlers() {
      Browser.getWindow().addEventListener("offline", event ->
      // Browser goes offline: CONNECTION_LOST and stop heartbeats
      giveUp());

      Browser.getWindow().addEventListener("online", event -> {
          // Browser goes back online: RECONNECTING while verifying
          // server connection using heartbeat
          resumeHeartbeats();
          handleRecoverableError(Type.HEARTBEAT, null);
      });
}

But this can cause the application to set to a false offline state, because when a developer cuts the wifi connection, the browser sends an offline event, but the developer can still connect to localhost, thus the app should be in online state.
Minimal reproducible example

Go to start.vaadin.com
Choose Flow and V20
Download and unzip
run mvn spring-boot:run
go to localhost:8080 with your browser and verify that the app works.
Cut your internet connection.
Notice that your app is not responding anymore (In offline state)

https://user-images.githubusercontent.com/30408303/118264193-037a4f80-b4c0-11eb-9568-0b9a56a679e1.mov
Expected behavior
The application should stay in online state.
Actual behavior
The application is in offline state.

User story, just to make sure the intent is clear:

As a developer working on an online-only Flow application
I want my internet connection not to affect my local Flow application in any way
so that I can continue developing my Vaadin app uninterrupted even if my internet goes down.

I was trying to work on my Vaadin project in a place where I didn't have an internet connection, and I couldn't access it before I enabled hotspot from my phone.

For Flow/Hybrid applications, when receiving offline event, instead of immediately giveup, should send the retry requests.


bug: which-key hunks prefix does not work

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
v0.10.0
Operating system/version
Linux
Describe the bug
This:
https://github.com/LazyVim/LazyVim/blob/8c94842b4a613995bf4c0c79febcd74c310439d0/lua/lazyvim/plugins/editor.lua#L356
Does not work:

I discovered it while trying to remap gitsigns keys in my config:
  {
    "folke/which-key.nvim",
    opts = {
      defaults = {
        ["<leader>h"] = { name = "+hunks" },
      },
    },
  },

^ this doesn't work either.
I am not 100% sure why, but I suspect it's because Gitsigns loads after which-key. When which-key loads first, it doesn't see any keymaps <leader>gh*, and doesn't register an "empty" prefix.
For refence, I've tested this in the default starter config.
That's all the info I have and I don't know how to fix it, sorry.
Steps To Reproduce

Use starter config
Press <leader>g
h shows as +prefix instead of +hunks

Expected Behavior
Expected to show +hunks instead of +prefix.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

This is a problem with which-key.nvim itself and there's a workaround mentioned in this comment (it also includes the initial comment at which-key repo from which I got the idea).

Ahhh now I see. This makes perfect sense then:
https://github.com/nvim-lua/kickstart.nvim/blob/5aeddfdd5d0308506ec63b0e4f8de33e2a39355f/init.lua#L283-L291
I can create a PR for this.

You would have to create it at which-key's repo if you want. This is not the place for this.

Okay, no problem. I am going to add the placeholder in my config.

@dotfrag I'm just a regular user like you, so the opinion I expressed was only my personal one. Go ahead and do the PR if you like and the maintainer will make the decision for it.


http package still uses sdk source for unittest

unittest is now in pub.dartlang.org
http://code.google.com/p/dart/source/browse/trunk/dart/pkg/http/pubspec.yaml

This issue has been moved to dart-lang/http_server#1.


Add slicing args defined in @listSize directive

Addresses https://github.com/IBM/graphql-query-generator/issues/15
Does not yet document the option, as that relies on exposing the "specification" defining the exact structure and semantics of the @listSize directive.

We need to describe this in the README, no?


Add a table for assessments

The commit adds a new database table for assessments, which are
groupings of results (e.g., scans). This helps organize results based on
assessments of systems or clusters.
This schema was discussed in https://github.com/rhmdnd/compserv/issues/84
Fixes #118

/lgtm


Update maintainer.go

change log level to debug of logs in maintainer that printed a lot

Coverage decreased (-0.08%) to 86.707% when pulling 0672f92c76d2e05a432128a4e0f92444397d6bff on pavelmaliy-patch-1 into 2823f5c993dc10fe8188e47fc0691b26631a6739 on master.


Refactor: restructure workbook, worksheet, style and driver

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

Project coverage is 99.10%. Comparing base (b3bb168) to head (48db495).
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
+ Coverage   97.57%   99.10%   +1.53%
==========================================
  Files          11       10       -1
  Lines        1029      779     -250
==========================================
- Hits         1004      772     -232
+ Misses         16        7       -9
+ Partials        9        0       -9

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


Detect duplicates when uploading an image in a modal chooser

Based on #8284.

[x] Do the tests still pass?
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?

[ ] Please list the exact browser and operating system versions you tested: Chrome
[ ] Please list which assistive technologies you tested: None

[ ] For new features: Has the documentation been updated accordingly?

Merged in 503298556efe0ac47b6ac624712e46467e3a2943


fix(live upgrade): newly live upgraded engine doesn't respect the disableRevCounter setting

We forget to set disableRevCounter for the newly live upgraded engine. As a result, the newly live upgraded engine always has disableRevCounter as false even if the volume has disableRevCounter as true.
This mismatch prevents the volume from successfully rebuilding a new replica.
longhorn/longhorn#9331

@mergify backport v1.7.x v1.6.x v1.5.x


iPad: Quick open matches are illegible

Light theme

🤔 Hmm...temporary bug? Looks syntax highlighting in your image hasn't kicked in yet.

Can't repro today 😕


TransformException: java.util.zip.ZipException: duplicate entry

I met this error:
Execution failed for task ':app:transformClassesWithJarMergingForStagingDebug'. > com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: android/support/v4/graphics/drawable/DrawableCompatBase.class
How to solve it ?Thx

compile ('com.jakewharton:butterknife:8.5.1'){
exclude module: 'support-compat'
}
this maybe be would help you

This is a build system problem, not a Butter Knife problem. See my comment here: https://github.com/JakeWharton/butterknife/issues/903#issuecomment-287264156


Test method from php class in zephir

Oh right, i thought that in phalcon dispatcher there is just somewhere controller class, there is not, let me update.


Mock a change

Fixes #
Description
How was this change tested?

Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[ ] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5272

Relevant Lines:
6282

💛  - Coveralls


Lose customed FancyZones solution but keeping configuration

windows 10 enterprise
Windows build number: 10.0.17763.832
PowerToys version:
v0.15.2
PowerToy module for which you are reporting the bug (if applicable):

Steps to reproduce
normal shut down and start PC ,the settings just lost today ,no warming and exception or error I can catch

Expected behavior
Non

Actual behavior
my customed FancyZones style lost ,including FancyZones and no default solution ,press shift but no reaction

Screenshots

what this from a 0.14.x to 0.15 upgrade?

no ,I set the customed style after upgrading it to 0.15

@yshch2008
is this still a problem with https://github.com/microsoft/PowerToys/releases/download/v0.21.1/PowerToysSetup-0.21.1-x64.exe ?


Weighted Custom Loss Function Different Training Loss

I am experimenting with weighted custom loss functions for a binary classification problem but I am having trouble replicating the exact behaviour when using the default loss function in conjunction with sample weights. If there are no sample weights, then the custom loss function (same as in the advanced example notebook) works as expected and the training/validation loss as well as final predictions are exactly the same.
However, if I weight the samples (doesn't matter what weights - any weights), the training loss in the custom function becomes slightly different to that of the default loss function. The difference is very minor but I still do not understand why there is any difference at all. This looks like it might be a bug to me. It's not clear which is right and which is wrong. Intriguingly, the final predictions however are still exactly the same.
I have looked at the C++ source code and I believe I have followed the same logic. When sample weights are provided, the gradient and the hessian are simply multiplied by the sample weights before being returned - that is the only difference between when sample weights are provided and when they are not. This is exactly what I'm doing.
Please let me know if my loss function is wrong or if this is a bug. Thanks
Environment info
Operating System: Ubuntu 18.04.4 LTS
CPU/GPU model: Intel Core i7-8565U CPU
C++/Python/R version: Python 3.7.6
LightGBM version or commit hash: 2.3.0
Error message
No error message.
Reproducible examples
import lightgbm as lgb
import numpy as np
from sklearn.datasets import make_classification
from sklearn.model_selection import train_test_split

# CREATE DATASET
np.random.seed(42)
X, y = make_classification(n_samples=10000, n_features=10)
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.1)
train = lgb.Dataset(X_train, y_train)
valid = lgb.Dataset(X_test, y_test)

# ASSIGN WEIGHTS
weights = np.concatenate((np.full((4500), 1, dtype=float), np.full((4500), 2, dtype=float)), axis=None)
train.set_weight(weight=weights)

# DEFINE PARAMS
params = {
    'objective': 'binary',
    'n_estimators': 10,
    'boost_from_average': False,
    'random_state': 42
}

# DEFINE CUSTOM LOSS FUNCTION
def logloss(preds, data):
    y_true = data.get_label()
    preds = 1. / (1. + np.exp(-preds))
    weight = data.get_weight() if data.get_weight() is not None else 1
    grad = (preds - y_true) * weight
    hess = preds * (1.0 - preds) * weight
    return grad, hess

# DEFINE CUSTOM EVAL LOSS FUNCTION
def logloss_eval(preds, data):
    y_true = data.get_label()
    preds = 1. / (1. + np.exp(-preds))
    loss = -(y_true * np.log(preds)) - ((1 - y_true) * np.log(1 - preds))
    return "binary_logloss", np.mean(loss), False

# TRAIN MODEL_1
model_1 = lgb.train(params=params, train_set=train, valid_sets=[train, valid], valid_names=['train','valid'])
scores_1 = model_1.predict(X_test)

# TRAIN MODEL_2
model_2 = lgb.train(params=params, train_set=train, valid_sets=[train, valid], valid_names=['train','valid'], fobj=logloss, feval=logloss_eval)
scores_2 = model_2.predict(X_test)

# PRINT SCORES
print(scores_1[:20])
scores_2_trans = 1. / (1. + np.exp(-scores_2))
print(scores_2_trans[:20])

Steps to reproduce

Simply run the above code in python. This will output the training and validation losses as well as the first 20 predictions for the model which uses the default loss function and the model which uses the custom loss function. The printed training loss numbers will be different but the final predictions will nevertheless be the same

However if you now comment out the line which sets the weights (train.set_weight(weight=weights)), the training losses for the two models will now be exactly the same.

In built-in metric, the weight is also used: https://github.com/microsoft/LightGBM/blob/7cfb492e7733f73526fbf8958d850bbb94af83ed/src/metric/binary_metric.hpp

Thanks very much Guolin, I thought since I am not passing weights to the validation set, the logloss_eval function doesn't matter but I now realise it's used to compute the metrics for the training set as well.
In case anyone else comes across this, the following function for training metrics on training and validation sets works:
def logloss_eval(preds, data):
    y_true = data.get_label()
    weight = data.get_weight() if data.get_weight() is not None else np.ones(len(y_true))
    preds = 1. / (1. + np.exp(-preds))
    sum_loss = sum((-(y_true * np.log(preds)) - ((1 - y_true) * np.log(1 - preds))) * weight)

    return 'binary_logloss', sum_loss / sum(weight), False


Provide a way to setup selenium timeout for testing

Could be a website configuration that not only check that selenium is properly installed but also allows you to set settings.

I don't remember the use case. It either works or fails.
I did'nt need to set the timeout.


Licensing

Hello,
I'm a Gentoo Linux developer, and I'm one of the maintainers for Blender. While updating it today, I noted that eltopo is actually bundled in it. I went looking for the original library so that I could package it separately, and since I found that the build system wouldn't be too flexible for distributions, I ended up writing an autotools-based one, that most distributions would be happy to use...
Unfortunately, now I notice that there is no license assigned to any file in this package, which means for us that we have to treat it as an "all rights reserved" proprietary package. Is this intentional, or just an oversight?
Please note that if you want to just have the most open license possible, you still have to declare copyright and a license — I'd strongly suggest going with either the WTFPL or the X11/MIT license (I'm linking to FSF only because they have a note about their actual free/non-free status that most distributions follow ... I'm in no way suggesting you ought to copyleft this as they are, although if you feel that's a good decision, go for it).
Just please handle licensing one way or the other, I'll be glad to contribute back my changes, and the buildsystem, then so that your software can be made available to Gentoo Linux's users, and we'll build Blender against that.
Thanks,
Diego

I'd be curious to find out if you ever picked a license for el topo.

According to http://www.cs.ubc.ca/labs/imager/tr/2009/eltopo/eltopo.html its public domain.
Though it would be good to include a LICENSE file in the source repository.

FWIW another option is the Los Topos library, which we put explicitly under a FreeBSD license and which has some improved merging functionality, along with support for multiple materials. (I hope Tyson will forgive this shameless plug.)
http://www.cs.columbia.edu/cg/multitracker/code/

Yeah, I always intended it to be public domain, however I now realize that this was a mistake, and we should have chosen a license from the start. I'm not sure if legally I can now apply a license to it or not. At one point we were going to meet with the UBC IP office to see what we could do, but I ended up graduating before we got around to it.
Anyways, I wholeheartedly endorse Los Topos if you need code with a license -- it basically does everything El Topo does and more. Even if you don't care about multimaterial tracking, the improved merging operation is really nice.

cool. thanks.
On Thu, Nov 5, 2015 at 12:39 AM, tysonbrochu<EMAIL_ADDRESS>wrote:

Yeah, I always intended it to be public domain, however I now realize that
this was a mistake, and we should have chosen a license from the start. I'm
not sure if legally I can now apply a license to it or not. At one point we
were going to meet with the UBC IP office to see what we could do, but I
ended up graduating before we got around to it.
Anyways, I wholeheartedly endorse Los Topos if you need code with a
license -- it basically does everything El Topo does and more. Even if you
don't care about multimaterial tracking, the improved merging operation is
really nice.
—
Reply to this email directly or view it on GitHub
https://github.com/tysonbrochu/eltopo/issues/6#issuecomment-153766177.


Halin Unable to connect if Default Database has been stopped

With 4.x, if you stop the default database and then try to connect with Halin, you get the standard connection error (see screenshot).

Cause of this bug is that when Halin is first initializing, it has to run certain queries, like getting cluster overview, in order to know how to proceed.  By default, halin grabs a session and runs a query on the default database, which fails if the default database isn't active.   The issue is, if the default database isn't available to answer queries, Halin can't be sure where else to send the query because it doesn't even have available information on version or multi-db capabilities yet.   I'll see what work-around is possible.

I made the same mistake,
The neo4j version is community version 4.2.7, which uses the default library, but the above problems also occur.
My question is, does the community version not support it at all? Or something else?


add masked mse loss

Codecov Report

Merging #61 (eab155e) into main (ec758c9) will decrease coverage by 0.18%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   93.99%   93.81%   -0.19%
==========================================
  Files          45       45
  Lines        1947     1955       +8
==========================================
+ Hits         1830     1834       +4
- Misses        117      121       +4

Impacted Files
Coverage Δ

src/hearth/__init__.py
100.00% <100.00%> (ø)

src/hearth/losses.py
96.69% <100.00%> (+0.23%)
:arrow_up:

src/hearth/vision/ops.py
93.33% <0.00%> (-6.67%)
:arrow_down:

src/hearth/vision/utils.py
93.75% <0.00%> (-6.25%)
:arrow_down:

src/hearth/callbacks/monitors.py
94.59% <0.00%> (-2.71%)
:arrow_down:

src/hearth/callbacks/learning_rate.py
97.67% <0.00%> (-1.17%)
:arrow_down:

Continue to review full report at Codecov.

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


Add structure-modules.md page

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
yes

Fixed issues
#12233

Thank you! It was accidentally removed by @jaaf while working on translation.


Return data and keys of CaseInsensitiveArray as public attributes

What is the expected behavior?
The ability to systematically loop through the Request and Response Headers to evaluate them and make decisions off of them
What is the actual behavior?
These are private attributes, so I am unable to access them as an array. Rather I have to call offsetGet from CaseInsensitiveArray but that requires a key to be included. I need the ability to store the array in the database.
What steps will reproduce the problem?
No need to reproduce. Just attempt to retrieve all the headers as an array.

Hi. You are able to loop through all the headers.
foreach ($curl->responseHeaders as $key => $value) {
    echo $key . ' => ' . $value . "\n";
}

Status-Line => HTTP/1.1 200 OK
Connection => keep-alive
Content-Type => application/json
Access-Control-Allow-Origin => *
Access-Control-Allow-Credentials => true
...


rt-app fails at the end of run

I'm trying to run the following workload. After the workload finishes it fails with:
INFO     Creating output directory.
INFO     Initializing run
INFO     Connecting to target
INFO     Setting up target
INFO     Initializing execution context
INFO     Generating jobs
INFO         Loading job wk1
INFO     Installing instruments
INFO     Installing output processors
INFO     Starting run
INFO     Initializing run
INFO         Initializing job wk1 [1]
WARNING  Event [sched_energy_diff] not available for tracing
WARNING  Event [sched_group_energy] not available for tracing
WARNING  Event [sched_energy_calc_duration] not available for tracing
WARNING  Event [sched_sugov_util] not available for tracing
INFO     Running job wk1
INFO         Configuring augmentations
INFO         Configuring target for job wk1 [1]
INFO         Setting up job wk1 [1]
INFO         Running job wk1 [1]
INFO         Processing output for job wk1 [1]
INFO         Tearing down job wk1 [1]
ERROR          File "/usr/local/lib/python2.7/dist-packages/wa/framework/execution.py", line 420, in run_next_job
ERROR            self.do_run_job(job, context)
ERROR          File "/usr/local/lib/python2.7/dist-packages/wa/framework/execution.py", line 473, in do_run_job
ERROR            job.process_output(context)
ERROR          File "/usr/local/lib/python2.7/dist-packages/wa/framework/job.py", line 119, in process_output
ERROR            self.workload.update_output(context)
ERROR          File "/usr/local/lib/python2.7/dist-packages/wa/workloads/rt_app/init.py", line 180, in update_output
ERROR            self._pull_rt_app_logs(context)
ERROR          File "/usr/local/lib/python2.7/dist-packages/wa/workloads/rt_app/init.py", line 277, in _pull_rt_app_logs
ERROR            self.target.pull_file(target_path, host_path, timeout=120)
ERROR
ERROR        AttributeError('AndroidTarget' object has no attribute 'pull_file')
INFO         Completing job wk1
ERROR    Job wk1 iteration 1 completed with status FAILED. Max retries exceeded.
INFO     Finalizing run
INFO         Processing using "csv"
INFO         Finalizing job wk1 [1]
INFO     Done.
INFO     Run duration: 46 seconds
INFO     Ran a total of 1 iterations: 1 FAILED
INFO     Results can be found in /var/www/wa_output//ASB-2017-11-06_4.9-o-release-2967-g770961336703/camera
WARNING  There were errors during execution.
WARNING  Please see /var/www/wa_output//ASB-2017-11-06_4.9-o-release-2967-g770961336703/camera/run.log
Attached run.log, config file and agenda
run.log
camera.tar.gz

Should be fixed by https://github.com/ARM-software/workload-automation/pull/642


It would be nice if your application would respect XDG_CONFIG_HOME ($HOME/.config)

Is your feature request related to a problem? Please describe.
It would be nice if your application would respect the XDG_CONFIG_HOME, which defaults to $HOME/.config. It's nice to have an uncluttered $HOME!
Describe the solution you'd like
It looks like setup.sh defines some temp- and configfolder settings and battery.sh uses them and start the app (?) but I didn't test it.
I dont know anything about electron apps but i think it would be much cleaner to use a standard library to read the XDG_CONFIG_HOME env. If you want, define your own env BATTERY_CONFIG etc.
Additional context
https://wiki.archlinux.org/title/XDG_Base_Directory

I like your thinking @rafo, an uncluttered home is nice indeed.
How is XDG_RUNTIME_DIR read? I get an empty state when I access it through CLI.
Do you have the bandwidth to PR for this?

I'd also like to see ~/.battery be moved to $XDG_CONFIG_HOME/.
I don't have $XDG_RUNTIME_DIR set neither, but $XDG_CONFIG_HOME would do. And ~/.battery could be a fallback solution.

Hi @actuallymentor, do you foresee any updates on this topic?

Hi @actuallymentor, do you foresee any updates with this topic?

Apologies for the slow responses. The time I can dedicate to battery fluctuates depending on "real" work.
This is extremely low priority for me (has next to to user impact), however I'm happy to merge a PR for it.
Anyone making a PR: please add fallback logic and remember that the paths need to be synced between the cli code (bash) and the GUI code (js).

@actuallymentor It's because the XDG-env var is a linux thing. For mac to use them they have to be explicitly set in the environment. Typically XDG_CONFIG_HOME would be set to $HOME/.config.

@actuallymentor It's because the XDG-env var is a linux thing. For mac to use them they have to be explicitly set in the environment. Typically XDG_CONFIG_HOME would be set to $HOME/.config.

I'm a tad confused then... This is a Mac only app, why is the XDG variable set desirable?

It's just about organizing all sorts of config directories you got into ~/.config instead of having them spread in the home dir. Also it gives you consistent experience if you also use linux.


Roll engine 59b5a0cf7676..f0371e2f8f97 (1 commits)

git@github.com:flutter/engine.git/compare/59b5a0cf7676...f0371e2f8f97
git log 59b5a0cf7676..f0371e2f8f97 --no-merges --oneline
2019-11-16<EMAIL_ADDRESS>Work around Fuchsia a11y / ICU name conflict (flutter/engine#13885)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


'Olive Editor' found an issue with concurrent accesses

Refer to https://github.com/olive-editor/olive/issues/1382 for details.

Closing the issue as the Olive editor code needs to be updated i.e. that's not a bug. By design OpenColorIO forbids any multi-threaded manipulation accesses of an in-memory config.


[change] deprecate getParentElement in favor of findDOMNode.

findDOMNode is available since React v0.14 and it's the minimum
version for react-modal.
To keep things simple, each Modal can be placed on any level
of the react's tree and it will do a proper clean up when it's
time to unmount the component. Also, it make things simple enough
to deal with overlapped modal.
Changes proposed:

Use a div provided by the instance of  to setup things.
Each <Modal /> can perform its own clean up.

Upgrade Path (for changed or removed APIs):

Remove getParentElement.

Acceptance Checklist:

[x] All commits have been squashed to one.
[x] The commit message follows the guidelines in CONTRIBUTING.md.
[x] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[x] If the commit message has [changed] or [removed], there is an upgrade path above.

needs some testing with different reactand react-dom versions.

I believe It's only needed to test with<EMAIL_ADDRESS>Some version of react has a bug on findDOMNode, but it should be better to ask to upgrade for a new version.

This patch need rework.


Error trying to export namespace: module does not exist

Is there an existing issue for this?

[X] I have searched the existing issues

Version of Corteza
2023.3.5
Current Behavior
When trying to export a namespace (Manage namespaces -> Pick from list -> Export namespace), I'm encountering the following response:
{
   "error":{
      "message":"invalid module reference {corteza::compose:module {[347533651178741763] map[347533651178741763:true]} {corteza::compose:namespace {[344241087105269762] map[344241087105269762:true]}} false}: module does not exist"
   }
}

344241087105269762 is a valid namespace ID - it's the built-in CRM namespace that I modified over time. No module exists with the id of<PHONE_NUMBER>78741763, at least not now. I guess it was created and then deleted. However, when I go into modules list in this namespace and hit "Export", the modules list exports without problem. Module with ID<PHONE_NUMBER>78741763 was not in that list.
Expected Behavior
Export tool should ignore non-existing modules during export or have some sort of clean-up after deleting module.
Steps To Reproduce
No response
Environment and versions
No response
Anything else?
When I created a blank test namespace with two modules and deleted one of them, the export succeeded, so that wasn't the case here. There's something wrong with this one particular namespace.

Update: after restoring yesterday's backup of the database, the error message has changed:
{
   "error":{
      "message":"parent reference {corteza::compose:page {[347533949125320707] map[347533949125320707:true]} {corteza::compose:namespace {[344241087105269762] map[344241087105269762:true]}} false} not found"
   }
}

This error appeared exactly once. After that, every error looks like before (referrencing module, not page)
Note that nothing really should have changed in the database contents since yesterday, we don't yet use the application in production.

Ref: https://forum.cortezaproject.org/t/namespace-export-error/1570/2

# Fixed

## What

namespace export erroring out with an invalid reference error whenever we're referring to deleted resources.

## How

by ignoring missing references and exporting/importing regardless of missing references.
This should be expanded and improved upon in a future release.

Thank you for adding the CL

Issue still exists in 2023.9.0:
{"error":{"message":"parent reference {corteza::compose:page {[347533966607179779] map[347533966607179779:true]} {corteza::compose:namespace {[344241087105269762] map[344241087105269762:true]}} false} not found"}}

Hm.. can you try to figure out what resource causes this error? My guess would be a page.
I can take another look if I missed something

I guess it could be some already non-exiatent page or one that got replaced by new one... but shouldn't the application now just ignore this error and export anyway?

I took a look at my PostgreSQL database and identified the affected page(s) in compose_page:
SELECT title FROM compose_page WHERE id = {{corteza::compose:page}};

I had to delete the respective entries in both compose_page and compose_page_layout to successfully export the namespace:
DELETE FROM compose_page WHERE id = {{corteza::compose:page}};

@tjerman since it still doesn't work, should I create a new issue or will this one be reopened?

Ah.. so it turns out the problem is caused by page layouts regarding the deleted page. If you delete those does the problem go away?

All right, fingers crossed the next one does it for good :see_no_evil: 1d5402678 ignores cases for resource translations

Unfortunately, the issue still persists for 2023.9.2-rc.1.

Well then I'll need some help from you. Could you provide a DB dump for me?

Well then I'll need some help from you. Could you provide a DB dump for me?

I can send you the DB backup from my env by email, if that's okay

2023.9.2-rc.1 finally works in my case. However, if I try to export namespace, delete it in Corteza and then import it again, it complains with error: Namespace failed to import: not unique. The problem exists even if I give it a different name when importing, change the id line in the exported yaml or even hard delete the namespace from database. There should be an option to force importing the namespace as new, with new IDs generated for namespace, modules, pages etc for test environments and such.


Add missing passwordConfirmation arg to createUser mutation

Codecov Report

Merging #110 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #110   +/-   ##
========================================
  Coverage    76.74%   76.74%
========================================
  Files            7        7
  Lines           43       43
  Branches         6        6
========================================
  Hits            33       33
  Misses          10       10

Continue to review full report at Codecov.

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


[WIP] Implement virtual selection

This is still in very early PoC state and it is barely usable. I am opening this mainly for feedback on the implementation.
Still lots to be done here.

I was thinking to bind mouseup/mousedown, tracking the start and end position. Then draw 1-3+ rectangles divs under the text but over the background, depending on which text was selected. This would enable things like selecting more text than is in the viewport and scrolling the terminal when the mouse is down and the cursor approaches the top/bottom boundaries.
Some things to consider:

In a browser, copying will probably require copying the text to a temporary textarea
Middle click to paste may be messed up?
I would like to keep the inverted selection color on vscode so maybe more than 3 rectangles would be required, what about adding .selected to span elements when the entire span is selected

Closing because of #293.


Creating an iPhone/android app for this extension.

Hi,
So I've been using this app for a few years and I'm really excited to see it being worked on again. I would love to make an iPhone/Android app that does the same thing as the chrome extension. It would advertise this plugin and pretty much serve a way for users on mobile to use this plugin since chrome apps aren't available to them.
If this mobile project is something you don't want affiliated with this extension, I understand. Please let me know your thoughts one way or another!
Thanks,
Andrew

In my personal opinion, I'd prefer just to keep things the way that they are as trying to keep both in sync with updates would be a challenge, and stuff like that would also fall out of the project's scope. A workaround that does allow users to install the extension onto their Android phones is via an all called Kiwi Browser which allows users to install chrome extensions as they would on desktop. Additionally, a lite version of the extension that runs as a standalone website could potentially happen in the future. The main concern of publishing an app for iOS is Apple being stricter with the use of the music, Google is quite laid back with allowing apps on their stores in comparison. It's up to the others if they want anything else to happen too, this is just my personal opinion as this has been brought up by many others before.

That's understandable. That's why I figured I'd ask first. Thanks for letting me know!


Sponsor layout and BG updates

Sponsor images now lay out two to a row
Move down to one per row on small widths
Background color of sponsor page is white

Also tagging @zacharysang and @zcollins0 for their feedback. Here's a current screenshot of the branch:

I agree. More whitespace would be better. And normalized sizing for logos would also be better.

Changed the image spacing and fixed the large amount of padding at small screen widths

I think this would be good to go if each column was 50% and each image was centered in its column

This requires a fair bit of finessing because there's a max width on the container but the images have a defined height, which can result in stretching. At the moment it's fine, but if there's an image with a higher aspect ratio than Vantiv's, there might be issues.


Improved error message to indicate when project flavour does not support gliffy import

Preflight Checklist

[x ] I agree to follow the Code of Conduct that this project adheres to.
[x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
I'm able to import *.gliffy files on browser based draw.io, but unable to import *.gliffy on windows application version of draw.io. When I try to import or create from existing on windows version of draw.io. I get "Error loading file", but same file was used for browser version of draw.io which works fine.
To Reproduce
Steps to reproduce the behavior:

Import *.gliffy or create from existing and use *.gliffy on windows installed draw.io

Expected behavior
Import should succeed and I should be able to edit the diagram in draw.io.
Screenshots

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version: 14.6.13

Desktop (please complete the following information):

OS: Windows 10
Browser: chrome
Version:  91.0.4472.114

Hi,
Gliffy import on desktop is not supported.
Regards,

I've changed the issue to be improving the error message where Gliffy import isn't supported.

Will be fixed in the next release

In 16.5.1


Include metadata in violation message

Currently when constraint violation is detected tool returns message like this:
Found Violations:
[rego message]
It would be useful to include also policy metadata in this message.
Forseti Config Vaildator includes this data in violation report.

Are you using the Terraform validator?
How would you like the metadata surfaced?

Rego rule schema according to OPA constraint framework is following:
deny[{"msg": msg, "details": {}}] {
  # rule body
}

So I suggest to add new flag --script and return only details json object when this flag is used.
BTW, I opened issue in wrong project.
Now I created another issue in:
https://github.com/GoogleCloudPlatform/terraform-validator/issues/35


follow up for graphql-codegen dependency pump in examples folder

Cont' to https://github.com/Shopify/hydrogen/pull/2094
codegne generated files need to be updated from the graphql-codegen dependency as well.

@frandiox actually, not all of the examples are non-diff!
A few have graphql changes in the example so the x-generated.d.ts files were checked in and had to get change anyway.


Does not appear in search for Firefox Android

Empty:

https://addons.mozilla.org/en-US/android/search/?q=refined

But the Android-specific link works:

https://addons.mozilla.org/en-US/android/addon/refined-github-/

This is what I see in the developer hub on AMO:

Continues from:

https://github.com/refined-github/refined-github/issues/6367

Related:

https://github.com/refined-github/refined-github/pull/7480

Hi, I work on AMO - This is a quirk caused by accessing https://addons.mozilla.org/en-US/android/search/?q=refined with a non Firefox for Android browser. AMO tries to detect your browser and version to only show you relevant add-ons, but if you're forcing /android/ while on desktop or on Android without Firefox, it doesn't know for sure what version to aim for, and so it picks 120.0 at the moment. Your extension is only compatible with 121.0 or higher so the API excludes it from the results.
If you check out that URL from a Firefox for Android >= 121.0 browser (either a real one or overriding your UA with Firefox's Responsive Mode, for instance), it should work fine.

Can confirm that it works on Firefox for Android 128.0

Great to hear! Thanks for the comment


[BUG] When admin delete the problem back to the problem list page, the problem list still present deleted problem

deleted problem:

problem list:

click in deleted problem:

這邊我有個解法，但不一定是 '合理' 的。
Problem List:

Problem Editor:


[get_team_view_fast] Fails for player name with trailing whitespaces (assumption for now)

I actually do not know if the trailing whitespaces are the real issue here, however, the following error popped up in our logs for an otherwise legitimate looking player:
[2022-07-18 20:34:42,907][ERROR] rcon.commands commands.py:wrap:59 | Auto retrying _request ('playerinfo  Showa ',) {'can_fail': False}
Traceback (most recent call last):
  File "/code/rcon/commands.py", line 54, in wrap
    return method(self, *args, **kwargs)
  File "/code/rcon/commands.py", line 126, in _request
    raise HLLServerError(f"Got FAIL for {command}")
rcon.commands.HLLServerError: Got FAIL for playerinfo  Showa
[2022-07-18 20:34:42,948][WARNING] rcon.commands commands.py:_reconnect:95 | reconnecting
[2022-07-18 20:34:45,099][ERROR] rcon.extended_commands extended_commands.py:get_team_view_fast:237 | Failed to get info for {'name': ' Showa ', 'steam_id_64': '765xxxxxxxxxxxxxx', 'steam_bans': {'CommunityBanned': False, 'VACBanned': False, 'NumberOfVACBans': 0, 'DaysSinceLastBan': 0, 'NumberOfGameBans': 0, 'EconomyBan': 'none', 'has_bans': False}, 'country': 'DE'}
Traceback (most recent call last):
  File "/code/rcon/commands.py", line 54, in wrap
    return method(self, *args, **kwargs)
  File "/code/rcon/commands.py", line 126, in _request
    raise HLLServerError(f"Got FAIL for {command}")
rcon.commands.HLLServerError: Got FAIL for playerinfo  Showa

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/code/rcon/extended_commands.py", line 235, in get_team_view_fast
    player_data = future.result()
  File "/usr/local/lib/python3.8/concurrent/futures/_base.py", line 437, in result
    return self.__get_result()
  File "/usr/local/lib/python3.8/concurrent/futures/_base.py", line 389, in __get_result
    raise self._exception
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/code/rcon/cache_utils.py", line 146, in wrapper
    return cached_func(*args, **kwargs)
  File "/code/rcon/cache_utils.py", line 72, in __call__
    val = self.function(*args, **kwargs)
  File "/code/rcon/extended_commands.py", line 362, in get_detailed_player_info
    raw = super().get_player_info(player)
  File "/code/rcon/commands.py", line 236, in get_player_info
    data = self._request(f"playerinfo {player}", can_fail=can_fail)
  File "/code/rcon/commands.py", line 62, in wrap
    return method(self, *args, **kwargs)
  File "/code/rcon/commands.py", line 126, in _request
    raise HLLServerError(f"Got FAIL for {command}")
rcon.commands.HLLServerError: Got FAIL for playerinfo  Showa

I can provide the actual steam ID if that would be of help, however, I'm not comfortable sharing that here publicly :)

It actually isn't that easy to add trailing whitespaces to the your player name anymore. It looks like Steam added some additional validation here and simply strips away the whitespaces. So this player I mentioned probably has some old name set, still.
However, as we had the same issue just yesterday again, but with another name, I was able to reproduce. As some background:
the players name is [NSK] | desp3rado | vi'D, however, as the name in HLL seems to have a different length limit then in Steam, it is currently stripped to be [NSK] | desp3rado |  and it shows up like that in the RCon. However, when executing playerinfo [NSK] | desp3rado |  returns with a FAIL:
Got FAIL for playerinfo [NSK] | desp3rado |

Using the "official" RCon tool provided by the GSP returns FAIL as well :/ So this really looks a bit more like a game server side RCon issue.

Update: I got the response from Team17, that this issue with trailing whitespaces should be fixed with the next update. I'll wait for it to verify that afterwards. I did not get any more info, yet, on how this is fixed (like either the whitespace needs to be there, should be removed (trim), or if the name should be quoted. Let's see 😅)

Good stuff. Thanks for the follow up with T17
On Sat, 27 Aug 2022 at 16:36, Florian @.***> wrote:

Update: I got the response from Team17, that this issue with trailing
whitespaces should be fixed with the next update. I'll wait for it to
verify that afterwards. I did not get any more info, yet, on how this is
fixed (like either the whitespace needs to be there, should be removed
(trim), or if the name should be quoted. Let's see 😅)
—
Reply to this email directly, view it on GitHub
https://github.com/MarechJ/hll_rcon_tool/issues/116#issuecomment-1229204396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABB5JQD3PXNVRANZBLYSPIDV3IRVRANCNFSM5354XN3Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Julien Maréchal
Software Engineer
Amsterdam - The Netherlands
+31 6 41 09 71 06
@.***

fr.linkedin.com/pub/julien-marechal/72/478/1a5/

Just checked this again and.. what should I say: It is still the same behavior as before U13 🤷 I'll get in touch with T17 again to find out what they actually tried to fix and how.

Thanks for following up. it's quite annoying indeed, to have that 1% of failure especially in the automoderation. We could potentially make those guys immune but that may lead to abuses / bypasses.
And for stats gather, well, we're just screwed.

Just for your interest: U14 did not fix this issue 🤷

As we still can't rely on a fix from game developpers, we may try to change CRCON behaviour.
Please give some advice on my suggestion here : https://github.com/MarechJ/hll_rcon_tool/issues/249

CRCON v9.1.0 introduces the option to warn/kick these bad profile players when they join.

I would really like to keep this issue open as a reminder to check this from time to time, if the underlying issue is fixed by T17. Right now, anything that we implemented is an ugly workaround only :)

This is fixed, potentially with U16, as player names seems to not be truncated anymore.


Refactor the Jade

We need to refactor the Jade so it uses e.g. mixins, has a separate lib directory, etc. However this needs to happen on a branch that includes the Express 3 changes, as the merge conflicts will be hell otherwise.
Since we consider non-trivial template changes to be breaking, this needs to go in before 2.0.0.

See also #1457


feat: adds fromPem method for identity-secp256k1

Description
Adds a utility method to initialize a secp256k1 identity from a pem file.
Fixes SDK-1255
Fixes SDKTG-221
How Has This Been Tested?
Tested against a test identity exported from dfx and compatible with dfx and keysmith
Checklist:

[x] My changes follow the guidelines in CONTRIBUTING.md.
[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[x] I have made corresponding changes to the documentation.

@randombit got back to this today - should be more legible now. Thanks for your suggestions!

LGTM


remove spotify docker plugin

I am not sure if this is used or not internally Ryan, but if you say it's not - we should remove it. wdyt?

It is used internally, that is how we push the image to dockerhub


[bug]  build error :: [commonjs--resolver] Missing "./event-41a9edf5" specifier in "@tauri-apps/api" package error during build: Error: Missing "./event-41a9edf5" specifier in "@tauri-apps/api" package

Describe the bug
[commonjs--resolver] Missing "./event-41a9edf5" specifier in "@tauri-apps/api" package
error during build:
Error: Missing "./event-41a9edf5" specifier in "@tauri-apps/api" package
Reproduction
No response
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Windows 10.0.22621 X64
    ✔ WebView2: 114.0.1823.82
    ✔ MSVC:
        - Visual Studio Build Tools 2019
        - Visual Studio Community 2022
    ✔ rustc: 1.72.0-nightly (114fb86ca 2023-06-15)
    ✔ Cargo: 1.72.0-nightly (0c14026aa 2023-06-14)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: nightly-x86_64-pc-windows-msvc (default)
    - node: 19.0.0
    - pnpm: 8.6.7
    - yarn: 1.22.19
    - npm: 8.19.2

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: 1.4.0
    - @tauri-apps/cli [NPM]: 1.4.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../build
    - devPath: http://localhost:5173/
    - framework: Svelte
    - bundler: Vite

Stack trace
No response
Additional context
No response

sorry, it was an auto-import in the code.


Passing scoped_session method dependency to path dependency triggers Invalid args for response field pydantic error

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from typing import Any

from fastapi import APIRouter
from fastapi import Depends
from pydantic import BaseModel
from sqlalchemy import create_engine
from sqlalchemy.orm import scoped_session
from sqlalchemy.orm import sessionmaker

DB_URL = "some_db_url"

class TestSuccess(BaseModel):
    message: str = "Test success"
    code: int = 200
    param_1: str

router = APIRouter(prefix="/test_router")

def get_db_session() -> scoped_session:

    engine = create_engine(DB_URL)
    session_maker = sessionmaker(bind=engine)
    db_session = scoped_session(session_maker)

    return db_session

def query_db(param_1: str, db: scoped_session) -> bool

    # return True if param_1 in some db table else raise HTTPException

@router.post(
    path="/test",
    response_model=TestSuccess,
    dependencies=[Depends(query_db)], # <--------- Removing this dependency solves the error
)
def test_method_1(
    param_1: str,
    db: scoped_session = Depends(get_db_session),
) -> dict[str, Any]:

    # Do stuff with db
    # ...

    response = {"param_1": param_1}

    return response

Description
Hi everyone,
I've been having some issues with overriding a database dependency in my app for testing purposes, which seem to come from a conflict with the use of the response_model path parameter.
As suggested by the docs, I tried to use a dependency returning an SQLAlchemy scoped_session as default method parameter, so that I would then be able to override it in my tests. A simplified version of the code is available above. This method fails with message:
fastapi.exceptions.FastAPIError: Invalid args for response field! Hint: check that <class 'sqlalchemy.orm.scoping.scoped_session'> is a valid pydantic field type

This seems to be due to the fact that the db parameter passed from the method dependency to the path dependency is not recognised by pydantic as valid. This error is suppressed by removing the query_db path dependency.
Can anyone suggest a correct way of achieving this?
Thanks in advance for the help.
Operating System
Linux
Operating System Details
Debian 11
FastAPI Version
0.85.0
Python Version
3.9.2
Additional Context
pydantic==1.10.2

class ResponseModel(BaseModel):
    message: str = "Test success"
    code: int = 200
    param_1: str

Can we assume that wherever you use TestSuccess as response model, you actually mean ResponseModel?
I found the 422 in case two weird but cannot reproduce it. Maybe you don't import Depends somehow? The error suggests that you should pass a query parameter named db_url, so it doesn't solve for your dependency. Dependencies do not show up in the docs, you can try it with the below MRE (will run as is):
from fastapi import Depends, FastAPI

app = FastAPI()

def get_string():
    return "Marc"

@app.get("/")
async def root(string: str = Depends(get_string)):
    return {"oh hi": string}

if __name__ == "__main__":
    import uvicorn
    uvicorn.run(app, host="<IP_ADDRESS>", port=8000)

You'll see that string will not show up in the docs.
Lastly; you don't have to cast to the response model, then to dict(), and then return that dictionary. You should just return {"param_1": param1}, and FastAPI will cast it for you to TestSuccess. That is the point of specifying a response_model.
As for Case 1; you might want to inspect how you set param_1. You say it is coming from the Query Param (or you imply that with your code) but I would bet you are changing that parameter with something from the database, which is not a string.
Either way; the two cases show very different problems that are not really related, would be my guess.

Between Case 1 and Case 2, which would be the most sensible way to go about what I am trying to do, and how can I get it to work?

There is good documentation how to deal with database connections via dependencies in the documentation here. I would recommend to follow that approach.

Hi @JarroVGIT , and thank you for your answer.
I realised after reading your reply that my question was indeed two-pronged and poorly formulated. I have edited my question and the code sample to make the issue both clearer and more correct.
Also, I went overboard when first attempting to simplify the code sample, and omitted the part that seems to be actually triggering the error: the path dependency.
Apologies for the low-quality question. I hope its second version will make it less turbid.
Thanks again for the help, and please do let me know if there is anything I can do to make the issue clearer.

Could you try to change:
def query_db(param_1: str, db: scoped_session) -> bool
into this:
def query_db(param_1: str,  db: scoped_session = Depends(get_db_session)) -> bool
I am not sure if FastAPI will pass a parameter from your path operation function to a dependency that is declared in the path decorator.

Thanks for the tip @JarroVGIT, it worked instantly.
Closing the issue as everything is running fine now.
Thanks again four your help.

Thanks for the help here @JarroVGIT  ! 👏  🙇
Thanks for reporting back and closing the issue @NoahY43619  👍


Implements reading a binary STL's header

This PR is the same as #2, but it has an additional commit which fixes the tests.

Thanks!
Are all binary STL files guaranteed to have this header, or does this code somehow handle the case where a header isn't present?
I did originally have binary STL files that had no header, but it's been a while and I forget which tool generated them. I usually work with OpenSCAD so it might well have been OpenSCAD output.

It is my understanding that all binary STL files have an 80byte header; However, that header is supposed to be all null bytes in binary STLs, and only populated in non-conforming cases. (at least according to Wikipedia)
The code in my patch will happily create a null string if the entire first 80 bytes of the binary STL file are all null.
The actual book excerpt in which the STL file format was(is) defined can be read at http://www.ennex.com/~fabbers/StL.asp
There is reads that for binary STLs, the header is to be 80 bytes of ASCII data that is just a header, and does not impact the actual data of the mesh.

Seems reasonable to me. Thanks!


Multiple pen colors in same page.

A lot of people reached out to me on mails and Twitter to have this feature.
We can keep everything the way it is right now, but we can also allow users to select text and then change the value in drop-down to color only the selected part.

To color the font or the background sir.Will like to work on this too.

https://github.com/saurabhdaware/text-to-handwriting/issues/34#issuecomment-656999003

@saurabhdaware Could you please share the acceptance test criteria so that we can work with better efficiency and understand what do you expect after this issue is resolved.

We can keep everything the way it is right now, but we can also allow users to select text and then change the value in drop-down to color only the selected part.

This ^. So normally when I select ink color, it changes the color of the whole text.
After the PR,

if we select a particular text and then change the ink color in dropdown, it will only color the selected part and leave the other part as it is.
if we don't select any text, then it will change the color of the whole input (the way it works right now)


Add command sysl env

Fixes #550 .
Changes proposed in this pull request:

Add command sysl env to show sysl-related environments

Checklist:

[ ] Made corresponding changes to the documentation

@anz-bank/sysl-developers

In addition, I'm thinking about adding a package cfg in charge of all sysl-related environment variables.


"Too Much Xor" problem in CodeChef (June CookOff 2021).

Aim
I want to add the code for the "Too Much Xor" problem in CodeChef (June CookOff 2021).
Details
Problem name: Too Much Xor
Problem Code: TOOXOR
Problem link: https://www.codechef.com/COOK130B/problems/TOOXOR/
Please assign me this issue
LGM-SOC Participant
Programming language

C++

[ ] C
[x] C++
[ ] Java
[ ] Python

Hello @prudhvirajstark @cmulay @VinayakMohite4040 @shantamsultania @Sumanth2905 sir, I think this is an important question and this should definitely be added in the Daily-Code-DS-Algo-Practice and I will start working on it as soon as I get assigned!! as I am a part of the LGM-SOC'21 participant, so I would like to implement this in c++.

@Amit366 thank you for assigning me this issue


Pattern: Content group horizontal: Grid alignment

Content group horizontal: Grid alignment

Describe in detail the issue you're having.

The pattern is not properly aligning to the grid. Some text is not sitting on columns.

Is this a feature request (new component, new icon), a bug, or a general
issue?

Bug

Is this issue related to a specific component?

Content group horizontal (may be dependent on content item horizontal)

What did you expect to happen? What happened instead? What would you like to
see changed?

As seen in Storybook:
Text and links in the second column at the item level is not aligned with the grid. This can be seen at Medium, Large, XL and Max breakpoints

Expected from visual specs:
Text aligns on grid columns at all breakpoints

What browser are you working in?

Chrome

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

IBM.com Reboot
Steps to reproduce the issue
https://ibmdotcom-react.s3.us-south.cloud-object-storage.appdomain.cloud/deploy-previews/2333/index.html?path=/story/patterns-blocks-contentgrouphorizontal--default

Notes from June 11, Leadership planning meeting.
The grid alignment will be fixed all at once.  Wonil will perform an audit and provide the information to Roberta so that Dev issue can be created.  This issue and (#2673) will be referenced in the new dev issue and closed.


HDFS-16179. Update loglevel for BlockManager#chooseExcessRedundancySt…

JIRA: HDFS-16179
private void chooseExcessRedundancyStriped(BlockCollection bc,
    final Collection<DatanodeStorageInfo> nonExcess,
    BlockInfo storedBlock,
    DatanodeDescriptor delNodeHint) {
  ...
  // cardinality of found indicates the expected number of internal blocks
  final int numOfTarget = found.cardinality();
  final BlockStoragePolicy storagePolicy = storagePolicySuite.getPolicy(
      bc.getStoragePolicyID());
  final List<StorageType> excessTypes = storagePolicy.chooseExcess(
      (short) numOfTarget, DatanodeStorageInfo.toStorageTypes(nonExcess));
  if (excessTypes.isEmpty()) {
    LOG.warn("excess types chosen for block {} among storages {} is empty",
        storedBlock, nonExcess);
    return;
  }
  ...
}

IMO, here is just detecting excess StorageType and setting the log level to debug has no effect.
We have a cluster that uses the EC policy to store data. The current log level is WARN here, and in about 50 minutes, 286,093 logs are printed, which can cause other important logs to drown out.

LGTM

Thanks @ayushtkn for your review.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  4s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
37m 48s

trunk passed

+1 :green_heart:
compile
1m 25s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m 15s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
1m  0s

trunk passed

+1 :green_heart:
mvnsite
1m 22s

trunk passed

+1 :green_heart:
javadoc
0m 56s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 27s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 19s

trunk passed

+1 :green_heart:
shadedclient
19m  7s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 15s

the patch passed

+1 :green_heart:
compile
1m 21s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m 21s

the patch passed

+1 :green_heart:
compile
1m  7s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
1m  7s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 53s

the patch passed

+1 :green_heart:
mvnsite
1m 17s

the patch passed

+1 :green_heart:
javadoc
0m 48s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
1m 23s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
3m 21s

the patch passed

+1 :green_heart:
shadedclient
18m 46s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
354m 15s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

451m 10s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestBlockScanner

hadoop.hdfs.server.namenode.ha.TestEditLogTailer

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3313/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3313

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 0cb16d9f8243 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a121470e5b175de09f0697d510e9b26612b2f6e1

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3313/1/testReport/

Max. process+thread count
2110 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3313/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Hi @jojochuang @ferhui , could you please also review the code? Thanks a lot.

@tomscut Thanks for contribution.
I'm confused here. The log is necessary or not. If it's necessary, many logs as you said drown out other logs. If it's not necessary, I think DEBUG level is OK.

@tomscut Thanks for contribution.
I'm confused here. The log is necessary or not. If it's necessary, many logs as you said drown out other logs. If it's not necessary, I think DEBUG level is OK.

Thanks @ferhui for your review. I think this method is just detecting redundancies and deleting them, but there are no redundancies in most cases.

@tomscut Thanks for comments.

but there are no redundancies in most cases

I see that if shouldProcessExtraRedundancy returns true, It will enter this method.
If no redundancies, it will not enter this method, is it right?
Are there any reasons that it prints lots of logs, maybe there is a hidden reason ?

@tomscut Thanks for comments.

but there are no redundancies in most cases

I see that if shouldProcessExtraRedundancy returns true, It will enter this method.
If no redundancies, it will not enter this method, is it right?
Are there any reasons that it prints lots of logs, maybe there is a hidden reason ?

Thanks @ferhui for your comments and careful consideration. I found this issue HDFS-9876, and we can refer to it.
Hi @Jing9 , could you please take a look at this. Thanks.

There are no so many logs you mentioned in our cluster  of thousands of nodes. So I guess there is a hidden reason behind of it. If we change it to DEBUG level, maybe we neglect some information.

There are no so many logs you mentioned in our cluster of thousands of nodes. So I guess there is a hidden reason behind of it. If we change it to DEBUG level, maybe we neglect some information.

Do you also use EC policy? We had a cold data cluster of 350 nodes, all directories are using EC policy, and we ran into this problem.

Yes, EC policy is used.

Yes, EC policy is used.

Thank you very much for your quick reply. There may be other reasons, we will take a closer look.

@tomscut  Hi, does this WARN log be printed when only writing EC files ? This WARN logs also appeared in our cluster without writting any files, but not as many as you said.
I found that the block in the WARN log belongs to the file written a long time ago. So, I have some guesses:

is there a daemon thread calling this method?
or other conditions trigger this method?

Here is our 3-hour running log.

@tomscut Hi, does this WARN log be printed when only writing EC files ? This WARN logs also appeared in our cluster without writting any files, but not as many as you said.
I found that the block in the WARN log belongs to the file written a long time ago. So, I have some guesses:

is there a daemon thread calling this method?
or other conditions trigger this method?

Here is our 3-hour running log.

Thanks @whbing for your comments. I found those logs were printed after completeFile. Triggered by FSDirWriteFileOp#completeFileInternal().
  private static boolean completeFileInternal(
      FSNamesystem fsn, INodesInPath iip,
      String holder, Block last, long fileId)
      throws IOException {
    (...)
    fsn.finalizeINodeFileUnderConstruction(src, pendingFile,
        Snapshot.CURRENT_STATE_ID, true);
    (...)
    return true;
  }

@tomscut Hi, does this WARN log be printed when only writing EC files ? This WARN logs also appeared in our cluster without writting any files, but not as many as you said.
I found that the block in the WARN log belongs to the file written a long time ago. So, I have some guesses:

is there a daemon thread calling this method?
or other conditions trigger this method?

Here is our 3-hour running log.

Thanks @whbing for your comments. I found those logs were printed after completeFile. Triggered by FSDirWriteFileOp#completeFileInternal().
  private static boolean completeFileInternal(
      FSNamesystem fsn, INodesInPath iip,
      String holder, Block last, long fileId)
      throws IOException {
    (...)
    fsn.finalizeINodeFileUnderConstruction(src, pendingFile,
        Snapshot.CURRENT_STATE_ID, true);
    (...)
    return true;
  }

@tomscut Hi, does this WARN log be printed when only writing EC files ? This WARN logs also appeared in our cluster without writting any files, but not as many as you said.
I found that the block in the WARN log belongs to the file written a long time ago. So, I have some guesses:

is there a daemon thread calling this method?
or other conditions trigger this method?

Here is our 3-hour running log.

Yes, those WARN logs were printed only writing EC files. Because the logs printed in BlockManager#chooseExcessRedundancyStriped().

@ferhui @whbing I found if balancer is started, the log will be printed. If balancer is stopped, the log will not be printed.

@ferhui @whbing I found if balancer is started, the log will be printed. If balancer is stopped, the log will not be printed.

@tomscut Hi, Do you find the root cause?  We encountered the same issue after balancer started.


[1.9.18 FABRIC 1.20.4] Resource reload failed bug

When I try to run Minecraft I meet endless loading game and notification «Resource reload failed»
It happens when I just run Minecraft

I'm unable to reproduce this. 1.20.4 is going to be deprecated anyways. But I'll be happy to help you. Can you send me your latest log file?

I'm unable to reproduce this. 1.20.4 is going to be deprecated anyways. But I'll be happy to help you. Can you send me your latest log file?

Sure
latest.log

Hmm... Which (extra) resource-packs are you using?

Hmm... Which (extra) resource-packs are you using?

No one. I have just deleted any resorce packs and launched game. Still the same problem.

latest.log

Hmm... Which (extra) resource-packs are you using?

No one. I have just deleted any resource packs and launched game. Still the same problem.  latest.log

Nevermind. I have troubles exactly in 1.20.4 Minecraft version. There is nothing wrong in your modpack. Sorry <3


Mention format for --max-ram

Hey @nromashchenko,
the new option --max-ram is a really useful improvement, thanks for implementing this! One short comment though: The readme does not state how to provide this. The dehumanize_ram function tells me that suffixes [BKMG] are supported, but that should be mentioned in the readme ;-)
Cheers
Lucas
PS: As for the "unfortunate naming" of that function, maybe call it parse_cli_ram_value or something like that ;-)

Hi @lczech
Thanks for noting this. Issue fixed by the pull request and this commit
Cheers!