feat: Bring back LaunchApp to WindowsDriver

List of changes
Bring back the LaunchApp Command to WindowsDriver
Types of changes
What types of changes are you proposing/introducing to the .NET client?
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] New feature (non-breaking change which adds value to the project)
[ ] Breaking change (fix or feature that would cause existing functionality not to work as expected)

Documentation

[ ] Have you proposed a file change/ PR with appium to update documentation?

This can be done by navigating to the documentation section on http://appium.io selecting the appropriate command/endpoint and clicking the 'Edit this doc' link to update the C# example
Integration tests

[ ] Have you provided integration tests to pass against the beta version of appium? (for Bugfix or New feature)

Details
Please provide more details about changes if necessary. You can provide code samples showing how they work and possible use cases if there are new features. Also, you can create gists with pasted C# code samples or put them here using markdown.
About markdown please read Mastering markdown and Writing on GitHub

Just in case it is needed: https://github.com/microsoft/WinAppDriver/blob/master/Docs/SupportedAPIs.md - the list of JWP APIs supported by WAD.


[BUG]: White screen on settings for Vector Database and Embedder

How are you running AnythingLLM?
Docker (remote machine)
What happened?
Brand new instance of AnythingLLM on docker on remote virtual machine. Everything appears to work as expected except when I click on settings >> Vector Database or Embedder.  No more settings
Console log says:

index.js:40  TypeError: Cannot read properties of undefined (reading 'logo')
    at _e (index-b41a81d3.js:1:32581)
    at Mu (index.js:38:16997)
    at pc (index.js:40:3139)
    at S0 (index.js:40:44734)
    at A0 (index.js:40:39724)
    at fw (index.js:40:39652)
    at ts (index.js:40:39506)
    at Lc (index.js:40:35874)
    at w0 (index.js:40:34825)
    at k (index.js:25:1582)
dc @ index.js:40
n.callback @ index.js:40
ip @ index.js:38
Ap @ index.js:40
y0 @ index.js:40
dw @ index.js:40
gw @ index.js:40
hr @ index.js:40
w0 @ index.js:40
k @ index.js:25
Z @ index.js:25
index.js:40  Uncaught TypeError: Cannot read properties of undefined (reading 'logo')
    at _e (index-b41a81d3.js:1:32581)
    at Mu (index.js:38:16997)
    at pc (index.js:40:3139)
    at S0 (index.js:40:44734)
    at A0 (index.js:40:39724)
    at fw (index.js:40:39652)
    at ts (index.js:40:39506)
    at Lc (index.js:40:35874)
    at w0 (index.js:40:34825)
    at k (index.js:25:1582)

Are there known steps to reproduce?
Launch docker container using all defaults.
Click on settings >> Vector Database
or
Settings >> Embedder
Result: White screen. No settings.

Fixed my own issue. Consider taking the quotes out of the example .env file for docker. I removed all the quotes and now everything is working.


@devicescript/gpio does not export some functions

in servers/pins document,
pinMode, digitalWrite, digitalRead, subscribeDigital are exported by @devicescript/gpio

(sorry error message is Japanese, but TS2305 is no exported member error.)
but in my workspace, it doesn't.
which is correct?

Could you make sure the reference to devicescript dependencies in package.json are >= 2.9.13 ?

Sorry for the slow response.
I had v2.9.12 and update it, I got it resolved.
Thank you!


Completion suggestions inside a comment are really bad. It shouldn't attempt to autocomplete to any "word" found anywhere in the docs.

It is vscode words completion, do not provider by emmyluaanalyzer


Failed to start server - getHost NaN

I'm hitting this error and the extension stays in an infinite retry loop:
[Extension Host] [2020-05-29 11:07:14.546] [TRACE] Server - start true
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] Server - isOnline
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [DEBUG] Server - isOnline? false
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] StatusBarItem - onStarting
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] Server - getPort 52698
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] Server - getHost NaN
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] Server - onServerError { inspect: [Function: inspect] }
console.ts:137 [Extension Host] [2020-05-29 11:07:14.547] [TRACE] StatusBarItem - onError

Even though I set "remote.host": "<IP_ADDRESS>" the getHost command seems to return NaN.
If I change the configuration, the setHost is however correctly executed:
[Extension Host] [2020-05-29 11:09:08.060] [TRACE] extension - onConfigurationChange
console.ts:137 [Extension Host] [2020-05-29 11:09:08.060] [TRACE] extension - getConfiguration
console.ts:137 [Extension Host] [2020-05-29 11:09:08.060] [DEBUG] extension - getConfiguration {  onStartup: true,  dontShowPortAlreadyInUseError: true,  port: 52698,  host: '<IP_ADDRESS>'}
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] extension - hasConfigurationChanged
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [DEBUG] extension - hasConfigurationChanged? true
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] extension - initialize
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] extension - getConfiguration
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [DEBUG] extension - getConfiguration {  onStartup: true,  dontShowPortAlreadyInUseError: true,  port: 52698,  host: '<IP_ADDRESS>'}
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] extension - startServer
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] Server - setPort 52698
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] Server - setHost <IP_ADDRESS>
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] Server - setDontShowPortAlreadyInUseError true
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] Server - start false
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [TRACE] Server - isOnline
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [DEBUG] Server - isOnline? false
console.ts:137 [Extension Host] [2020-05-29 11:09:08.061] [INFO] Server - Starting server
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [TRACE] StatusBarItem - onStarting
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [TRACE] Server - getPort 52698
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [TRACE] Server - getHost NaN
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [TRACE] StatusBarItem - setServer
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [DEBUG] StatusBarItem - setServer remove all listeners
console.ts:137 [Extension Host] [2020-05-29 11:09:08.062] [TRACE] StatusBarItem - handleEvents
console.ts:137 [Extension Host] [2020-05-29 11:09:08.067] [TRACE] Server - onServerError { inspect: [Function: inspect] }
console.ts:137 [Extension Host] [2020-05-29 11:09:08.067] [TRACE] StatusBarItem - onError

I was hit by this issue as well, but only after I upgraded to WSL2 on Windows 10. If you are running WSL2, you may have been hit by the issue where HyperV reserves ports above 50000 by default, as per this blog post. I can't find confirmation of this, but another article points to something similar including a possibly botched install of WSL2. In any case, there seems to be two ways to fix this on Windows 10:

Option 1: If you have Administrator privileges on your Windows machine
Run the following commands to force Hyper-V to use a different port range, for example:

netsh int ipv4 set dynamic tcp start=32768 num=16384
netsh int ipv6 set dynamic tcp start=32768 num=16384

The above commands forces Hyper-V to use a max. of 16000 ports, starting from 32768. If you have other apps that possibly conflict with this range, change the start and num parameters as  per your liking. In either case, reboot your machine and check if this extension now starts up correctly.

Option 1: If you do not have Administrator privileges on your Windows machine

is to change the listening port that the extension uses in Extension Settings. I had success with setting it to something like 45200. But ensure this port does not conflict with anything else.
Create ~/.rmate.rc in your remote machine as documented in the rmate repo.
Change your SSH tunnel settings to forward the new port.

Hope that helps!


Add static libicu

@alesapin, I see that the Linux build is working on this branch. Shall we merge this PR?

@slvrtrn I have better PR


[Feature] dumpable cluster and device type ID tables

Feature description
...
It is a pain to cross reference cluster IDs and device type IDs with names because they're sprinkled all over the spec. Now that we have the XML available, we should make a nice table and keep it up to date so it can be referenced from the test plans and SDK and spec.
Not sure QUITE where to land that, but the SDK seems like a reasonable place since it's more open than the spec or test plans repos.
Platform
core
Platform Version(s)
No response
Anything else?
No response

I know the master list contains a lot of additional information, but it kind of sounds like a slimmed down version of that?


fix(tier4_perception_launch): fix missing container argument

The arguments given in perception.launch.xml and accepted in pointcloud_based_occupancy_grid_map.launch.py are incompatible with each other. It is impossible to pass pointcloud_container as a parameter to probabilistic_occupancy_grid_map package.
Relevant parameters :
perception.launch.xml#L70
pointcloud_based_occupancy_grid_map.launch.py#L82
Signed-off-by: Kaan Colak<EMAIL_ADDRESS>Description

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@yukkysaito Hello Saito-san, Could you review this PR?


Permit specifying multiple buildpacks to execute

Similar to https://github.com/ddollar/heroku-buildpack-multi or heroku buildpacks:add

This is already supported. Just pass the URL to the multi buildpack and add a '.buildpacks' file. I can point you to an example if needed (OpenProject)

@crohr That doesn't appear to function correctly; it executes buildpacks that are not included in the definition: https://packager.io/gh/benlangfeld/QueueFlex/build_runs/4
It also appears that option does not alter the detection behaviour, which is to execute only the first buildpack to successfully detect support; this branch permits executing multiple buildpacks sequentially.

I'm not sure I follow here. The build run you mention was using a .pkgr.yml without buildpack properly set. You need to specify the buildpack option in your .pkgr.yml file (e.g. https://github.com/opf/openproject/blob/dev/.pkgr.yml#L41), and then list the buildpacks that need to be used in your .buildpacks file (e.g. https://github.com/opf/openproject/blob/dev/.buildpacks)

@crohr When I tried using heroku-buildpack-multi, the replacement of /app in the downstream buildpacks (those listed in .buildpacks) did not occur: https://packager.io/gh/benlangfeld/QueueFlex/build_runs/2

You spotted an issue indeed, with the /app not being replaced. I'm not sure I want to introduce the multi-buildpack concept into pkgr just yet. We would need to support the same behaviour as the original project, and I would also like to keep only one flag for specifying buildpack(s), i.e.:
Single buildpack:
buildpack: http://path/to/buildpack

Multi buildpack:
buildpack:
  - http://path/to/buildpack1
  - http://path/to/buildpack2

CLI:
pkgr --buildpack http://path/to/buildpack1,http://path/to/buildpack2 ...

What do you think?

I can certainly use the same config key. I'll push a commit for that tomorrow. As for mirroring behaviour, I believe this already is; is there some specific omission that you've noticed?

The above commit re-uses the buildpack option. As far as I know, there is no behaviour change for projects which specify only a single buildpack, and those which require multiple buildpacks may dispense of heroku-buildpack-multi with no difference in behaviour.

Thanks for that. What I meant by keeping the same feature set than the multi buildpack, is that at the end of each buildpack compilation you would need to handle those optional steps: https://github.com/ddollar/heroku-buildpack-multi/blob/master/bin/compile#L58-L65 (i.e. export whatever env variables is left by the previous buildpack, so that the next one can use them). I don't think we need to support the stuff related to the release yet (including https://github.com/ddollar/heroku-buildpack-multi/blob/master/bin/release), since we don't use it anyway.

at the end of each buildpack compilation you would need to handle those optional steps: https://github.com/ddollar/heroku-buildpack-multi/blob/master/bin/compile#L58-L65 (i.e. export whatever env variables is left by the previous buildpack, so that the next one can use them)

I can get that dealt with this week. Thanks @crohr :)

That should do it, but I havn't tested yet, @crohr

This works nicely for me now :)

@crohr, @benlangfeld what is the status of this feature? Will it be merged soon to the master?

As far as I know it's just pending review by @crohr.


Updated default webhook fixtures to use “source” instead of “card”

Based on the 2015-02-18 API changes: https://stripe.com/docs/upgrades#2015-02-18

True. I'd love to have this merged. Can we have tests fixed? @jasdeepgosal
Once it's done, can we have it merged @rebelidealist?

Would love this: +1

Ok, I rebased onto master and the specs pass now, but I didn't have to change anything in the specs so maybe I missed something?
Please let me know if I need to do anything else to get this merged in. (@alexmamonchik ?)

:+1:


Fixes issues with empty objects in toJson

See https://github.com/RelationalAI/rai-ux/issues/1350

@denisgursky What do I need to do to bump the version and deploy?

You have to be in our org on NPM. Then npm login, bump the version and npm publish --access public


Docs Request: improvements to enhance clarity on web/deploy.yml

What is the issue you are encountering with the docs?
here is the existing code
For static app
engine: Syn
For dynamic app
engine: Odin
continuous: false
build_name: app-web
it is pretty straightforward but some improvements can be made to enhance maintanability and flexibiity.
Links to Impacted Docs
https://github.com/base-org/web/blob/master/apps/web/deploy.yml
Describe the solution you'd like to see.
here is a more clearer version of the code
Engine Configuration
engine: Syn # Use "Syn" for static apps; switch to "Odin" for dynamic apps
Execution Mode
continuous: false # Set to "true" for continuous deployment or updates
Build Metadata
build_name: app-web # Name of the build; customize as needed
Additional context
what was improved?
-comments where placed in line with each setting to clarify its purpose directly to the context
-readability was improved by grouping related keys logically and adding clear headers to distinguish sections
-also added comments to indicate when and why certain configurations might change

Hello, thanks for submitting this issue. Closing as it has been opened with a lot of missing information.


FSD50KDataGen

Adds a FSD50K data generator with setup functions to:

download the FSD50K download DCASE sound events
download the DCASE sound events .txt
generate a DCASE filename to FSD50K filepath mapping

Example use case:
    fsd50k = FSD50KDataGen(dataset_name="fsd50k",
                            dataset_home="/scratch/data/FSD50K",
                            download=False)
    fsd50k.load_dataset()
    fsd50k.save_fsd_to_dcase_json() # I don't think this would be needed, may needed for logging
    # retrive dcase filenames to FSD50K filepaths
    filenames = fsd50k.get_filenames('train')
    # example use case:
    dcase_filename = 'waterTap/train/train/371615.wav'
    fsd_filepath = filenames[dcase_filename]

When saving the DCASE to FSD50k data dictionary we get a JSON file like:
{
    "train": {
        "bell/train/train/156121.wav": "FSD50K.dev_audio/156121.wav",
        "bell/train/train/156122.wav": "FSD50K.dev_audio/156122.wav",

...
},
    "test": {
        "bell/train/test/120493.wav": "FSD50K.eval_audio/120493.wav",
        "bell/train/test/120501.wav": "FSD50K.eval_audio/120501.wav",
...

Updated the PR to address the feedback above.

Thank you @adrianSRoman for the update. I'm getting an error:
FileNotFoundError: [Errno 2] No such file or directory: '/home/iran/SELD-data-ge
nerator/dcase_datagen/metadata/fsd50k/fma_metadata/tracks.csv'

Did you maybe forget to push this file? what is it?

Thank you. This is what we need for now. We'll address more changes when they become relevant in #17


Planning fails when module is disabled

Describe the Bug
When the input variable enabled is set to false, terraform fails to plan.
The issue exists since https://github.com/cloudposse/terraform-aws-elasticache-redis/releases/tag/1.3.0 up until the current 1.6.0
The same issue was previously mentioned by https://github.com/cloudposse/terraform-aws-elasticache-redis/issues/236#issuecomment-2214176533 but the particular scenario apparently overlooked when a fix was implemented
Expected Behavior
When the module is disabled, terraform plan succeeds.
Steps to Reproduce
Try invoking terraform plan on a project using this module with its enabled argument set to false:
module "redis" {
  enabled = false
  source  = "cloudposse/elasticache-redis/aws"
  version = "1.6.0"
  ...
}

Screenshots
Planning failed. Terraform encountered an error while generating this plan.

╷
│ Error: Error in function call
│
│   on .terraform/modules/redis/main.tf line 113, in locals:
│  113:   endpoint_address = coalesce(local.endpoint_serverless, local.endpoint_cluster, local.endpoint_instance)
│     ├────────────────
│     │ while calling coalesce(vals...)
│     │ local.endpoint_cluster is null
│     │ local.endpoint_instance is null
│     │ local.endpoint_serverless is null
│
│ Call to function "coalesce" failed: no non-null, non-empty-string
│ arguments.
╵

Environment

Terraform 1.7.5
Module version 1.6.0

Additional Context
No response

For us it happens also when the variable is set to true.
To reproduce:

create with cluster mode enable - the first create works
Then try just plan without any changes - fails with the same error

Version 1.4.4


Fixes #15 along with unit tests

This PR fixes issues #15 and checks every possible sct items with a comment, including different comment separators (single space, double or tab).
Passes unit tests

This also fixes the same issue regarding parsing positions from ESE file, although that method does not use getParts

Thanks a lot!


docs: add section about Node.js log levels

Changes

:book: adding some documentation around how to set logging levels in Node.js

Note that these documents will likely need to move out of this repo.
Fixes: https://github.com/knative-sandbox/kn-plugin-func/issues/328

/approve

This makes me wonder, are we able to unify the logging experience across all templates? So we can then configure logging in config and not via env variable (which is imho more user friendly).

This makes me wonder, are we able to unify the logging experience across all templates? So we can then configure logging in config and not via env variable (which is imho more user friendly).

Agree that configuration in the yaml is nice. But how do you see this working across multiple languages/runtimes that may be idiomatically different?

This makes me wonder, are we able to unify the logging experience across all templates? So we can then configure logging in config and not via env variable (which is imho more user friendly).

Agree that configuration in the yaml is nice. But how do you see this working across multiple languages/runtimes that may be idiomatically different?

Maybe the config could suggest appropriate logging levels for each language/runtime via func config logging? 🤷♂ Maybe I am overthinking this.

Maybe the config could suggest appropriate logging levels for each language/runtime via func config logging? 🤷♂ Maybe I am overthinking this.

This would be preferable from my view as well.  It would be quite nice to have a generalized language for defining logging expectations.

I see, so we have the ability to set https://github.com/knative-sandbox/kn-plugin-func/pull/462/files#diff-b55c094f2e6794329700479dc202c7ea098e14eeec000cfe72ede2ab57bb00f8R222-R228 in func.yaml - and each buildpack can do what they want with that, or ignore it altogether. I like it.

@zroubalik @lkingland updated with logLevel in func.yaml. The schema should be updated obviously, and it would be nice to add this to the configure command.
/hold

/lgtm

/hold cancel


Unnecessary s in bag download message

See the extra s in the display when making a bag to download on beta.

Fixed in updated beta.


fix(native-filters): Setting default value

SUMMARY
The aim of this PR was to improve setting default value for Native Filters.
The new Default Value Select component was created, which enables to choose default value based on dataset and field.
Now it is possible to set default value for a given filter.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:
for native filter without children (including editing)

for native filters child

TEST PLAN
Go to config.py and set  "DASHBOARD_NATIVE_FILTERS": True
Go to dashboards, choose one and create filter or use existing one.
Set default value.
ADDITIONAL INFORMATION

[x] Has associated issue: Fixes one of the issues mentioned in #12148
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

cc @junlincc @villebro @rusackas
@adam-stasiak could you test it?

I have a bug. Scenario:
Go to dashboard and add native filter. Set default value.
Export this dashboard
Import this dashboard
Open imported dashboard
Check default value imported - > should be the same as the original one
Open filter settings and change default filter value
You can see that default filter value will remain the same.
https://user-images.githubusercontent.com/25153919/104360974-23016b80-5512-11eb-9ef2-8f803954163e.mov

I have a bug. Scenario:
Go to dashboard and add native filter. Set default value.
Export this dashboard
Import this dashboard
Open imported dashboard
Check default value imported - > should be the same as the original one
Open filter settings and change default filter value
You can see that default filter value will remain the same.
https://user-images.githubusercontent.com/25153919/104360974-23016b80-5512-11eb-9ef2-8f803954163e.mov

Codecov Report

Merging #12465 (fc469dd) into master (0f731f2) will decrease coverage by 15.37%.
The diff coverage is 55.88%.

@@             Coverage Diff             @@
##           master   #12465       +/-   ##
===========================================
- Coverage   65.97%   50.59%   -15.38%
===========================================
  Files        1015      475      -540
  Lines       49548    16921    -32627
  Branches     5080     4358      -722
===========================================
- Hits        32687     8561    -24126
+ Misses      16714     8360     -8354
+ Partials      147        0      -147

Flag
Coverage Δ

cypress
50.59% <55.88%> (+5.02%)
:arrow_up:

javascript
?

python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...t-frontend/src/dashboard/reducers/nativeFilters.ts
46.15% <0.00%> (-14.72%)
:arrow_down:

...rd/components/nativeFilters/DefaultValueSelect.tsx
60.00% <60.00%> (ø)

...c/dashboard/components/nativeFilters/FilterBar.tsx
50.38% <71.42%> (-3.14%)
:arrow_down:

...oard/components/nativeFilters/FilterConfigForm.tsx
64.10% <100.00%> (-18.05%)
:arrow_down:

...nd/src/dashboard/components/nativeFilters/state.ts
37.50% <100.00%> (-23.37%)
:arrow_down:

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx
11.76% <0.00%> (-88.24%)
:arrow_down:

... and 891 more

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 0f731f2...fc469dd. Read the comment docs.

Codecov Report

Merging #12465 (fc469dd) into master (0f731f2) will decrease coverage by 15.37%.
The diff coverage is 55.88%.

@@             Coverage Diff             @@
##           master   #12465       +/-   ##
===========================================
- Coverage   65.97%   50.59%   -15.38%
===========================================
  Files        1015      475      -540
  Lines       49548    16921    -32627
  Branches     5080     4358      -722
===========================================
- Hits        32687     8561    -24126
+ Misses      16714     8360     -8354
+ Partials      147        0      -147

Flag
Coverage Δ

cypress
50.59% <55.88%> (+5.02%)
:arrow_up:

javascript
?

python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...t-frontend/src/dashboard/reducers/nativeFilters.ts
46.15% <0.00%> (-14.72%)
:arrow_down:

...rd/components/nativeFilters/DefaultValueSelect.tsx
60.00% <60.00%> (ø)

...c/dashboard/components/nativeFilters/FilterBar.tsx
50.38% <71.42%> (-3.14%)
:arrow_down:

...oard/components/nativeFilters/FilterConfigForm.tsx
64.10% <100.00%> (-18.05%)
:arrow_down:

...nd/src/dashboard/components/nativeFilters/state.ts
37.50% <100.00%> (-23.37%)
:arrow_down:

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx
11.76% <0.00%> (-88.24%)
:arrow_down:

... and 891 more

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 0f731f2...fc469dd. Read the comment docs.

I have a bug. Scenario:
Go to dashboard and add native filter. Set default value.
Export this dashboard
Import this dashboard
Open imported dashboard
Check default value imported - > should be the same as the original one
Open filter settings and change default filter value
You can see that default filter value will remain the same.
import.mov

Whoa... that's some impressive testing!

I have a bug. Scenario:
Go to dashboard and add native filter. Set default value.
Export this dashboard
Import this dashboard
Open imported dashboard
Check default value imported - > should be the same as the original one
Open filter settings and change default filter value
You can see that default filter value will remain the same.
import.mov

Whoa... that's some impressive testing!

@adam-stasiak thanks for thorough testing - as always! 🚀
I think there is a bigger bug with imported dashboards - it is not possible to edit them on master as well. I suggest to separate it - I have created issue for it #12495 that should solve the bug you have described.
There is a gif presenting that if problem with editing imported dashboard is fixed, we won't have that bug.

@adam-stasiak thanks for thorough testing - as always! 🚀
I think there is a bigger bug with imported dashboards - it is not possible to edit them on master as well. I suggest to separate it - I have created issue for it #12495 that should solve the bug you have described.
There is a gif presenting that if problem with editing imported dashboard is fixed, we won't have that bug.

To avoid issue similar to #12519, I have added resetting default value select for removed dataset.

To avoid issue similar to #12519, I have added resetting default value select for removed dataset.

Double checked this - is ok 🟢

Double checked this - is ok 🟢

@agatapst could we resolve conflicts? we probably should have merge this before the other ones🤦🏾♀
thanks! 🙏

@agatapst could we resolve conflicts? we probably should have merge this before the other ones🤦🏾♀
thanks! 🙏

Conflicts solved, but there were some significant changes to adjust this PR to current master, so please wait with merging - I'll check it one more time. 🟡

Conflicts solved, but there were some significant changes to adjust this PR to current master, so please wait with merging - I'll check it one more time. 🟡

Hi it looks good work 👍
I have some question about implementation. I see you use Select to choose defaultValue but we have different native filters chart types, for example for now we have already implemented Range native filter, so I think in this case we can't use Select to select appropriate value.
So I think it can be better use SuperChart here to inject appropriate value selection for default value, I had some PR for it but closed it in favour of yours, because you also implemented it in different way... but I still think that it can be good to use SuperChart here to select default value:
https://github.com/apache/superset/pull/12199
cc @villebro thoughts?

Hi it looks good work 👍
I have some question about implementation. I see you use Select to choose defaultValue but we have different native filters chart types, for example for now we have already implemented Range native filter, so I think in this case we can't use Select to select appropriate value.
So I think it can be better use SuperChart here to inject appropriate value selection for default value, I had some PR for it but closed it in favour of yours, because you also implemented it in different way... but I still think that it can be good to use SuperChart here to select default value:
https://github.com/apache/superset/pull/12199
cc @villebro thoughts?

@simchaNielsen Hi, thanks for your suggestions! Yes, you are right - we use Select component for setting default values for all cases. So for range filters it may not work properly. Tbh creating that PR I didn't know that type should be supported, so thanks for checking that.
Your solution looks nice - I don't mind closing this PR in favor of yours. My PR is completely based on DefaultValueSelect component, which in case of using SuperChart, would require to change the whole implementation here.
@villebro @junlincc @rusackas @suddjian I am waiting for your opinion as well.

@simchaNielsen Hi, thanks for your suggestions! Yes, you are right - we use Select component for setting default values for all cases. So for range filters it may not work properly. Tbh creating that PR I didn't know that type should be supported, so thanks for checking that.
Your solution looks nice - I don't mind closing this PR in favor of yours. My PR is completely based on DefaultValueSelect component, which in case of using SuperChart, would require to change the whole implementation here.
@villebro @junlincc @rusackas @suddjian I am waiting for your opinion as well.

I agree with @simchaNielsen - we should decouple the filters as much as possible from the filter config modal, so it would be preferable to use SuperChart there similarly to how it is being used in the filter tab.

I agree with @simchaNielsen - we should decouple the filters as much as possible from the filter config modal, so it would be preferable to use SuperChart there similarly to how it is being used in the filter tab.

@simchaNielsen thanks for the solution, I do agree we should go this route ♥. I wonder why you closed it earlier though?
@agatapst thanks for your great work still:) 🙏

@simchaNielsen thanks for the solution, I do agree we should go this route ♥. I wonder why you closed it earlier though?
@agatapst thanks for your great work still:) 🙏

Agreed that the other approach makes more sense. Closing this for now.


Error upon using prefix to initiate unity-flowlauncher

2023-02-06.txt
paste.txt

Didn't set unity project folder, closed


Export Requests table as CSV

The green/red results are nice, but i'm finding myself having to manually copy/write the data across into a spreadsheet to then pass on to clients.
It would be really awesome if we could export this as a CSV file.

Looking into saving results along with the rest of the state as a CSV in the next release.
Thanks for the suggestion!

Awesome :)

Added support for saving and loading state as a human readable JSON file in v0_6_3.  This now includes an array for run results.  More info in the CHANGES file.
Please let me know if you experience any issues.


all: copyright headers outside of /src have inconsistent spacing

While /src was made consistent in 20111 to use single spaces in copyright headers many go file headers in e.g. /test and /misc remain inconsistent and could also be updated.

I'll send a CL for this.


Update paperArtist.lua

revive #19

I realize I have the wrong version of the file (references a settings value that only exists in my mod)

Restored the one from actual #19. This pull request is now ready to be reviewed


Issue with Sub URLS in Domain Name settings

I have hosted an Kong Gateway application in one of our server which gives us an output url with a public IP  111.92.xx.xx
http://111.92.xx.xx:8000
And i have different sub urls like
http://111.92.xx.xx:8000/ABC
http://111.92.xx.xx:8000/ABC?find={bla,bla,bla}
http://111.92.xx.xx:8000/XYZ/findOne?access_token=ksdgasdgausygdasj     etc
This working fine and accessible everywhere.
Now i want to add a domain name for my ip "111.92.xx.xx:8000"
then may be my urls will be
http://exmaple.com
and
http://exmaple.com/ABC
http://exmaple.com/ABC?find={bla,bla,bla}
http://exmaple.com/XYZ/findOne?access_token=ksdgasdgausygdasj
I have checked with my network admin and added the DNS settings for this but now i can only able to access the main url like this "http://exmaple.com:8000"
And I can't able to to access the sub URLs,  I am getting the errors like {"request_path":"/XYZ/findOne","message":"API not found with these values","request_host":["example.com"]}
If i need to access the Sub URLs i need to pass the Host header like "Host: 111.92.xx.xx".
We have tried different options in DNS settings like URL Redirect, Adding A or CNAME record... but still it is not working
So how to resolve this issue? Is it some thing that we can do in DNS settings or in myserver i have to make some changes?

Have you read the proxy docs?


az group delete : FAILURE.

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az group delete
Errors:
RequiredArgumentMissingError: the following arguments are required: --name/-n/--resource-group/-g

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az group delete {}

Expected Behavior
Environment Summary
Linux-4.15.0-1103-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.17.1
azure-cli-ml 1.19.0

Extensions:
ai-examples 0.2.5
azure-cli-ml 1.19.0

Additional Context

ARM

ARM

@suaknab We sincerely apologies for the delay. Please update to latest az cli version 2.38.0. Also -n or -g should be provided to this command to delete a particular resource group.
Please reopen this issue if you have any further question in this regard


Docker Container is not running on Windows 10

Describe the bug
failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/sh -c dotnet restore
"src/Apps/ProjName.Api/ProjName.Api.csproj"]: exit code: 1
To Reproduce
Steps to reproduce the behavior:

Create the new template using dotnet new --install Matech.Clean.Architecture.Template
Downloading the Docker desktop
Going to the folder and running command docker-compose up

Expected behavior
The Docker images got up and running
Desktop (please complete the following information):

OS: [Windows 10]
Browser: NA
Version: <IP_ADDRESS>

Hi @AhmadElsayed ,
I tried to switch my windows 10 computer Linux Containers to Windows Containers, but the docker collapse and cannot even run docker run hello-world. I cannot recreate the bug. I have tested it on Linux, macOS, and Windows before.
My guess is that the problem may be on the DockerFile in the API project. I'm leaving the topic open, I'll update when I find a windows machine I can test.
Have a nice coding 😊

Hey @iayti
One of the issues in the template generation using the command
[dotnet new cas]
is having the path wrongly defined in the API Docker file
Instead of
COPY ["src/Apps/sample/sample.Api.csproj", "src/Apps/WebApi/"]
It should be
COPY ["src/Apps/sample.API/sample.Api.csproj", "src/Apps/WebApi/"]
But that didn't solve the issue with me though .. will keep trying

Hi @AhmadElsayed,
Sorry for my late answer. I tried on the Windows Container and get the error aspnet:5.0-buster-slim doesn't work on Windows Container and I change it to aspnet:5.0 base version.
After this change Api project docker build and run error fixed, but the docker-compose up didn't run. Network Bridge doesn't support the Windows Container.
If you want to use the Windows Container, you must create NAT network for docker-compose
Usefull link
Have a nice coding 😊


Made animations visible every time it get's into the view, and removed repetition of navbar component among the home page.

Before

After
https://github.com/user-attachments/assets/2ecc6088-8833-495a-bb72-31f351d3718d
issue no #188

good work!

@yashmandi Thank you...


New pretty printer class

@gepparta
Hi Alex.
I have written a new pretty printer class that addresses the issues #9 and #10. It respects escaping of enclosures and supports a maximum line length. I have added unit tests that pass. Please review it. Maybe you can suggest a better class name.
Best regards
Andreas

Hello Andreas,
please check if the new version satisfy your needs.
Also note the new test method in class  zcl_max_line_length_pp ( aka zcl_op_value_pretty_printer_2).
If you are ok with the new version, I would close this pull request, without merging it,
as I had to replace BASE statements to git it working on my 7.4  SP05 release.
Kind regards
Alex

Hi Alex.
That is perfect fine. I wasn't aware that BASE requires 7.4 SP08. I will look at your changes.
Best regards
Andreas


google-cloud-sdk: Fix default component arches

When a google-cloud-sdk component does not specify architectures on components.json we should fallback to all supported arches.
Previously there was a bug where allArches were incorrectly taken from the attrValues arches (nixOS architectures) instead of the GCS ones using attrNames arches.
Also added arm to the list of supported architectures. No changes needed on components.json since it looks urls and hashes were already there.
Co-authored-by: Fabián Heredia Montiel <fabianhjr@>
Description of changes

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Fixes #182856

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

/cc @NixOS/darwin-maintainers a fix for k8s tooling on darwin/aarch64.


Add PHP7 support

See upstream issue in geerlingguy.php Galaxy role: https://github.com/geerlingguy/ansible-role-php/issues/35

I'm going to close this out for now... it's a 'nice-to-have' that I really haven't needed for anything besides vanity/performance comparisons yet, and since it's still a pretty new/shiny version, and this project is less about new/shiny and more about getting people up and running with Drupal on a VM, I'm going to wait until PHP7 gets a little more stable.

Back in action...
After wrangling https://github.com/geerlingguy/ansible-role-php/issues/35 for quite some time, I decided I'm probably going to approach this just by using Zend's PHP7 nightly repo for the time being, since it will preserve quite a bit of CPU time in building from source, and nightlies are almost as close to bleeding-edge as building from up-to-the-second source.
For those daring enough to build from source, they can still do so, it'll just be a less-supported method of getting non-standard or cutting-edge PHP releases on Drupal VM.

'tis all good; see: https://github.com/geerlingguy/drupal-vm/wiki/PHP-7-on-Drupal-VM
For those who wish to do more manual process, you can still use Zend's PHP 7 nightly builds repository as well. I am just not officially supporting that in Drupal VM.


Option to auto-paste code into issue when created with the Bug button

This should improve the quality of the issues as a lot of issues could be solved immediately if the user posted the code when he opened the issue.
Could be implemented as a checkbox (that is checked on default) maybe?

I'm looking into this.

This looks like it will be hard to implement without changing how we create issues. I'm looking into using the GitHub API instead of a POST request.


Terminal Application/AppCursor mode (echoti smkx) not initiated

The terminfo sequences are meant to be used with the terminal in application mode. This is properly initiated with echoti smkx, usually in a zle line-init hook widget.
But it may cause problems, so until now this plugin has left smkx untouched, and let the framework deal with it. But doing nothing is causing wrong key bindings, so it needs to be sorted out.
Extra info

http://invisible-island.net/xterm/xterm.faq.html#xterm_arrows
https://stackoverflow.com/a/29408977/749778

After reviewing Oh My Zsh's bug reports related to smkx / rmkx and the code changes they're proposing (and have been proposing for 4 years!), I think the right choice for now is to keep the hard-coded special key definitions just as we have them now. We will not use Application mode just in order to make terminfo entries match since there are too many inconsistencies in terminal applications for that to work.
There are two kinds of problems that I think could result from this decision:

We hard-code a key combo to a special key here, but the terminal actually defines that key combo in terminfo for something else. If this happens to you, please file a new issue and report your terminal, what key you are trying to use, and what that key sends. We can make an exception for that terminal to avoid the conflict.

There is a key combo defined in terminfo, that differs in smkx mode, that we have not hard-coded here. So the key is not handled as a special case. An example of this problem is issue #16. It should be easy enough to add another definition to fix the problem, once it is identified that the terminal application is configured correctly. I'm a little more clear on the smkx / rmkx issue now, and hopefully can diagnose issues more effectively in the future. So, again, please file a report if you have a special key (Home, End, arrow keys, etc.) that don't function as you'd like!


Typo in Bern?

There seems to be a strange high number in Bern on 2020-10-08

https://github.com/openZH/covid_19/blob/master/fallzahlen_kanton_total_csv_v2/COVID19_Fallzahlen_Kanton_BE_total.csv
Seems to be this commit:
https://github.com/openZH/covid_19/commit/69b0ec147f3a1622834daed4ac3b8a311d5213cf

The be.ch/corona site messed up with the dates 08.10.2020 instead of 08.10.2020 today.. 19'233 reflects the total number to date - I suspect this was then scraped?

I adapted out scraper to fix the typo here: https://github.com/openZH/covid_19/pull/1345
The new data should be there in a minute.

Merci!


chore: use<EMAIL_ADDRESS>

chore: update ipld-explorer-components
chore: use<EMAIL_ADDRESS>

@SgtPooki I seem to consistently have one test failing locally, is it passing for you? If its just me (I have a old slow PC on purpose) maybe we need to increase timeout?

Some tests were failing for me locally as well (remote rpc api usually).. but I got an intermittent failure for that settings page locally as well

Maybe we just were lucky on CI? I've  re-run the same commit just to me sure.

i've got a fix for the remote RPC one that fixes them locally.

The job you kicked off succeeded, and you can see that e2e tests succeeded in the two commits after that, even though CI failed in total because of lint failures:

https://github.com/ipfs/ipfs-webui/actions/runs/10839115621/job/30078810498?pr=2257
https://github.com/ipfs/ipfs-webui/actions/runs/10839167154/job/30079057757?pr=2257

and then the latest had passing e2e tests as well: https://github.com/ipfs/ipfs-webui/actions/runs/10839174283?pr=2257

No problem.

ps. Playwright tooling improved, the --ui mode is really nice, I did npm run test:e2e -- --ui --grep "Settings" to debug this.

absolutely. I've been running npm run test:e2e -- --max-failures=1 --workers=1 --timeout=0 -g 'Blah' quite a bit during this PR..


importing pytest fixtures raises F401

It appears that ruff isn't handling pytest fixtures correctly.
Consider the following files (adapted from a pytest example):
fruit.py
import pytest

class Fruit:
    def __init__(self, name):
        self.name = name

    def __eq__(self, other):
        return self.name == other.name

@pytest.fixture
def my_fruit():
    return Fruit("apple")

@pytest.fixture
def fruit_basket(my_fruit):
    return [Fruit("banana"), my_fruit]

and test_fruit.py:
from fruit import (
    my_fruit,
    fruit_basket
)

def test_my_fruit_in_basket(my_fruit, fruit_basket):
    assert my_fruit in fruit_basket

Running ruff --isolated test_fruit.py I get:
test_fruit.py:2:5: F401 [*] `fruit.my_fruit` imported but unused
test_fruit.py:3:5: F401 [*] `fruit.fruit_basket` imported but unused
test_fruit.py:6:29: F811 Redefinition of unused `my_fruit` from line 2
test_fruit.py:6:39: F811 Redefinition of unused `fruit_basket` from line 3
Found 4 errors.
[*] 2 potentially fixable with the --fix option.

I'm using ruff 0.0.262

I'll look into this!

I think this has been discussed before, but I’m on my phone and it may be hard to dig up the relevant issue.

#3295 pointed me to this. Pytest suggests avoiding importing fixtures in favor of adding them to conftest.py. In that case, I'm personally a bit torn on implementing this - on one hand, it's definitely confusing behavior since the import is in fact used, but it's not recommended.
I'm almost leaning towards fixing the unused import bug, but maybe implementing a separate rule centered on avoiding fixture imports? FWIW, it seems like Pylint has the same issue as the one mentioned here.

Yeah, reading through the discussion, I think I agree with the conclusion that was reached in #3295: since Pytest itself recommends against this pattern, I'd rather not special-case in Ruff. (I'm not even sure how we could special-case it without introducing lots of false negatives. Best-case is that we could avoid autofixing these cases but still flag them.)

I'm going to close, given that conclusion, but always happy to continue the discussion.

Thank you very much for that!  I’m a novice with pytest and this was a case of me using a linter to help me muddle through, and it has 😜

Today, F401 autofixes wiped away my fixture imports a few times.  I can think of several workarounds:

Use conftest.py (which I am avoiding for reasons beyond this issue's scope)
Manually insert # noqa: F401 before invoking ruff
Place F401 in unfixable aspect of config

I went with option 3, marking F401 as unfixable.


[ISSUE] Errors when source the H2 identity script against a remote H2 server with 5.9.0 wum

Environment
IS pack wso2is-5.9.0+1585323544240.full
Steps to Reproduce

Create a H2 remote database
Execute the below H2 scripts against a remote H2 server

/wso2is-5.9.0/dbscripts/identity/h2.sql
Observation
Observing the below errors
Error occurred during SQL script execution
Reason:
SQL Error [42001] [42001]: Syntax error in SQL statement "CREATE TABLE IF NOT EXISTS IDN_PASSWORD_HISTORY_DATA (
  ID INTEGER NOT NULL AUTO_INCREMENT,
  USER_NAME   VARCHAR(255) NOT NULL,
  USER_DOMAIN VARCHAR(127) NOT NULL,
  TENANT_ID   INTEGER DEFAULT -1,
  SALT_VALUE  VARCHAR(255),
  HASH        VARCHAR(255) NOT NULL,
  TIME_CREATED TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP,
  PRIMARY KEY (ID),
  UNIQUE (USER_NAME,USER_DOMAIN,TENANT_ID,SALT_VALUE,HASH),
)[*]"; expected "identifier"; SQL statement:
CREATE TABLE IF NOT EXISTS IDN_PASSWORD_HISTORY_DATA (
  ID INTEGER NOT NULL AUTO_INCREMENT,
  USER_NAME   VARCHAR(255) NOT NULL,
  USER_DOMAIN VARCHAR(127) NOT NULL,
  TENANT_ID   INTEGER DEFAULT -1,
  SALT_VALUE  VARCHAR(255),
...

Error occurred during SQL script execution
Reason:
SQL Error [42001] [42001]: Syntax error in SQL statement "CREATE TABLE IF NOT EXISTS IDN_CLAIM_MAPPING (
  ID INTEGER NOT NULL AUTO_INCREMENT,
  EXT_CLAIM_ID INTEGER NOT NULL,
  MAPPED_LOCAL_CLAIM_ID INTEGER NOT NULL,
  TENANT_ID INTEGER NOT NULL,
  PRIMARY KEY (ID),
  FOREIGN KEY (EXT_CLAIM_ID) REFERENCES IDN_CLAIM(ID) ON DELETE CASCADE,
  FOREIGN KEY (MAPPED_LOCAL_CLAIM_ID) REFERENCES IDN_CLAIM(ID) ON DELETE CASCADE,
  CONSTRAINT EXT_TO_LOC_MAPPING_CONSTRN UNIQUE (EXT_CLAIM_ID, TENANT_ID),
)[*]"; expected "identifier"; SQL statement:
CREATE TABLE IF NOT EXISTS IDN_CLAIM_MAPPING (
  ID INTEGER NOT NULL AUTO_INCREMENT,
  EXT_CLAIM_ID INTEGER NOT NULL,
  MAPPED_LOCAL_CLAIM_ID INTEGER NOT NULL,
  TENANT_ID INTEGER NOT NULL,
  PRIMARY KEY (ID),
  FOREIGN KEY (EXT_CLAIM_ID) REFERENCES IDN_CLAIM(ID) ON DELETE CASCADE,
...

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Can't find "actors" name in class

After downloading the project, the following error pops up
I'm sorry, but an uncaught exception occurred.

While running game code:
  File "game/constitution_form.rpy", line 541, in script
    init 1 python:
  File "game/constitution_form.rpy", line 541, in script
    init 1 python:
  File "game/constitution_form.rpy", line 546, in <module>
    class HouseData:
  File "game/constitution_form.rpy", line 550, in HouseData
    clss = actors.House
NameError: name 'actors' is not defined

-- Full Traceback ------------------------------------------------------------

Full traceback:
  File "C:\renpy-8.0.0-sdk\renpy\bootstrap.py", line 277, in bootstrap
    renpy.main.main()
  File "C:\renpy-8.0.0-sdk\renpy\main.py", line 558, in main
    renpy.game.context().run(node)
  File "game/constitution_form.rpy", line 541, in script
    init 1 python:
  File "/home/tom/ab/renpy-build/tmp/install.linux-x86_64/lib/python3.9/site-packages/future/utils/__init__.py", line 441, in raise_
  File "game/constitution_form.rpy", line 541, in script
    init 1 python:
  File "C:\renpy-8.0.0-sdk\renpy\ast.py", line 1131, in execute
    renpy.python.py_exec_bytecode(self.code.bytecode, self.hide, store=self.store)
  File "C:\renpy-8.0.0-sdk\renpy\python.py", line 1061, in py_exec_bytecode
    exec(bytecode, globals, locals)
  File "game/constitution_form.rpy", line 546, in <module>
    class HouseData:
  File "game/constitution_form.rpy", line 550, in HouseData
    clss = actors.House
NameError: name 'actors' is not defined

Windows-10-10.0.19045 AMD64
Ren'Py <IP_ADDRESS>90809
Ecclesia 1.0
Wed Mar 15 14:34:38 2023

You're using 8.0.3, which I think doesn't recognize _ren.py files, so you will need the nightly too.


Not Working...

Getting this error...
$ ./bash.command-not-found keooepwjeko

./bash.command-not-found: 6: ./bash.command-not-found: Syntax error: "(" unexpected (expecting "}")

Terminal: Termux (Android)
Shell: Zsh with oh-my-zsh

And what is the content of your bash.command-not-found?

Well, it's the same file, your repo has...

However, Here it is...
print_message () {

    local messages
    local message

    messages=(
        "Boooo!"
        "Don't you know anything?"
        "RTFM!"
        "Haha, n00b!"
        "Wow! That was impressively wrong!"
        "Pathetic"
        "The worst one today!"
        "n00b alert!"
        "Your application for reduced salary has been sent!"
        "lol"
        "u suk"
        "lol... plz"
        "plz uninstall"
        "And the Darwin Award goes to.... ${USER}!"
        "ERROR_INCOMPETENT_USER"
        "Incompetence is also a form of competence"
        "Bad."
        "Fake it till you make it!"
        "What is this...? Amateur hour!?"
        "Come on! You can do it!"
        "Nice try."
        "What if... you type an actual command the next time!"
        "What if I told you... it is possible to type valid commands."
        "Y u no speak computer???"
        "This is not Windows"
        "Perhaps you should leave the command line alone..."
        "Please step away from the keyboard!"
        "error code: 1D10T"
        "ACHTUNG! ALLES TURISTEN UND NONTEKNISCHEN LOOKENPEEPERS! DAS KOMPUTERMASCHINE IST NICHT FÜR DER GEFINGERPOKEN UND MITTENGRABEN! ODERWISE IST EASY TO SCHNAPPEN DER SPRINGENWERK, BLOWENFUSEN UND POPPENCORKEN MIT SPITZENSPARKEN. IST NICHT FÜR GEWERKEN BEI DUMMKOPFEN. DER RUBBERNECKEN SIGHTSEEREN KEEPEN DAS COTTONPICKEN HÄNDER IN DAS POCKETS MUSS. ZO RELAXEN UND WATSCHEN DER BLINKENLICHTEN."
        "Pro tip: type a valid command!"
        "Go outside."
        "This is not a search engine."
        "(╯°□°）╯︵ ┻━┻"
        "¯\_(ツ)_/¯"
        "So, I'm just going to go ahead and run rm -rf / for you."

        "Perhaps computers is not for you..."

        "Don't you have anything better to do?!"
        "I am _seriously_ considering 'rm -rf /'-ing myself..."
        "This is why you get to see your children only once a month."
        "This is why nobody likes you."
        "Are you even trying?!"

        "My keyboard is not a touch screen!"
        "Commands, random gibberish, who cares!"
        "Typing incorrect commands, eh?"

        "Dropped on your head as a baby, eh?"
        "Brains aren't everything. In your case they're nothing."

        "You are not as bad as people say, you are much, much worse."
        "Two wrongs don't make a right, take your parents as an example."
        "You must have been born on a highway because that's where most accidents happen."
        "If what you don't know can't hurt you, you're invulnerable."
        "If ignorance is bliss, you must be the happiest person on earth."
        "You're proof that god has a sense of humor."
        "Keep trying, someday you'll do something intelligent!"

    )

    # If CMD_NOT_FOUND_MSGS array is populated use those messages instead of the defaults
    [[ -n ${CMD_NOT_FOUND_MSGS} ]] && messages=( "${CMD_NOT_FOUND_MSGS[@]}" )

    # If CMD_NOT_FOUND_MSGS_APPEND array is populated append those to the existing messages
    [[ -n ${CMD_NOT_FOUND_MSGS_APPEND} ]] && messages+=( "${CMD_NOT_FOUND_MSGS_APPEND[@]}" )

    # Seed RANDOM with an integer of some length
    RANDOM=$(od -vAn -N4 -tu < /dev/urandom)

    # Print a randomly selected message, but only about half the time to annoy the user a
    # little bit less.
    if [[ $((${RANDOM} % 2)) -lt 1 ]]; then
        message=${messages[${RANDOM} % ${#messages[@]}]}
        printf "\n  $(tput bold)$(tput setaf 1)${message}$(tput sgr0)\n\n"
    fi
}

function_exists () {
    # Zsh returns 0 even on non existing functions with -F so use -f
    declare -f $1 > /dev/null
    return $?
}

#
# The idea below is to copy any existing handlers to another function
# name and insert the message in front of the old handler in the
# new handler. By default, neither bash or zsh has has a handler function
# defined, so the default behaviour is replicated.
#
# Also, ensure the handler is only copied once. If we do not ensure this
# the handler would add itself recursively if this file happens to be
# sourced multiple times in the same shell, resulting in a neverending
# stream of messages.
#

#
# Zsh
#
if function_exists command_not_found_handler; then
    if ! function_exists orig_command_not_found_handler; then
        eval "orig_$(declare -f command_not_found_handler)"
    fi
else
    orig_command_not_found_handler () {
        echo "zsh: command not found: $1"
        return 127
    }
fi

command_not_found_handler () {
    print_message
    orig_command_not_found_handler "$@"
}

#
# Bash
#
if function_exists command_not_found_handle; then
    if ! function_exists orig_command_not_found_handle; then
        eval "orig_$(declare -f command_not_found_handle)"
    fi
else
    orig_command_not_found_handle () {
        echo "$0: $1: command not found"
        return 127
    }
fi

command_not_found_handle () {
    print_message
    orig_command_not_found_handle "$@"
}

try installing ncurses-utils  pkg install ncurses-utils, then try bash-insulter again

@Plloi ,... Already Installed...

well nuts that fixed termux for me :(

Not For me...

Do you get an error if you run it from bash?

Didn't get any error in bash, but didn't worked in bash too

no error/ no output on run is good
in bash source the file (. ./bash.command-not-found) the try to run a non-existent command 5-10 times( Being generous because script is random), you should get an insult or an error, if you get an error it may be more useful that zsh's

Here....
shell.command-not-found is the bash-insulter
And
command-not-found is Termux Default
Hope I Made Everything Clear...
...
...

You need to source it, not execute it
source /etc/bash.command-not-found

then try to execute a command

I can't see where you are sourcing the files? You are executing them, not sourcing them.
./command-not-found means execute  the command command-not-found, and it is not the same as . ./command-not-found, which actually is a more compatible way of writing source ./command-not-found.


Use Kubernetes validation instead of writing logic

We currently check for nil NAB backupSpec in code
https://github.com/migtools/oadp-non-admin/blob/4b2f57f3ad905456240e230895148a6d8f2a4be1/internal/common/function/function.go#L96-L98
instead of using Kubernetes API validation. To achieve this, we just need to remove omitempty here
https://github.com/migtools/oadp-non-admin/blob/4b2f57f3ad905456240e230895148a6d8f2a4be1/api/v1alpha1/nonadminbackup_types.go#L40

If omitempty is removed, it means we never really want this to ever be empty.. what's the point of it being a pointer?


Implement 2FA recovery codes

What's the problem this feature will solve?
The new 2FA feature adds a TOTP second factor to the authentication proces. This can be problematic when you lose access to the device you use for the 2FA authentication (e.g. the phone running the TOTP app).
Describe the solution you'd like
Most TOTP using sites I've used have an option of recovery codes that can be used to log in with a recovery code when you've lost access to the TOTP device.
Additional context
Github documentation on account recovery for 2FA

https://github.com/pypa/warehouse/issues/5758 ("Define manual account recovery process") is related.

Per conversation in IRC today -- recovery codes, while a useful feature, are not in-scope for the deliverables we promised in our grant-funded work from the Open Technology Fund (more details). So right now we'll continue with the current workflow, where it falls on PyPI site admins to perform the recovery process on behalf of users, and if we have time in this grant, we'll prioritize doing this, but otherwise we will work on it on a volunteer basis afterward.

To follow up on @nurupo's question in #996:

All 2FA implementations I have seen have a backup method you can use if you lose your TOTP/U2F device. Often times they provide backup codes. Sometimes it's TOTP with SMS as backup, or U2F with TOTP as backup. But I think I have yet to see a 2FA without a backup method.

If a user has both TOTP and WebAuthn enabled but has lost access to one or the other, they'll still be able to use the one they still have access to. That's not a backup scheme per se since we impose no ordering between TOTP and WebAuthn, but it's something we'll support.
Re: SMS and recovery codes: like @brainwane said these are not in scope for the OTF work, but it's my professional opinion that SMS should not be either a second-factor or recovery option in any greenfield 2FA implementation. Recovery codes are a great idea and IIRC are on the PyPI roadmap, but (again IMO) should not be mandatory, as they circumvent the two factor scheme entirely. Users who don't enable recovery codes will then be at the mercy of the recovery process determined by the site admins.

An issue with using both TOTP and WebAuthn is that if you use a single device for both, e.g. a Yubikey can do both TOTP and WebAuthn, and lose it, then you won't have access to both of those. Of course the obvious solution would be "don't store them on the same device". Or you could treat TOTP secret as a backup code and write it somewhere down as you would with backup codes. So I guess either way it somehow works out in the end, so I'm fine with just TOTP and WebAuthn. Having recovery codes would be a huge improvement though.

it's my professional opinion that SMS should not be either a second-factor or recovery option in any greenfield 2FA implementation.

Of course, please do not use SMS, it's considered insecure as it's subject to being intercepted and you can social engineer phone companies to assign you target's phone number, not to mention it would cost money to send messages to users. I mentioned SMS purely as an example of what I have seen in the wild, not necessarily as a model that should be followed.

as they circumvent the two factor scheme entirely.

Not sure how a website giving a user recovery codes "circumvents the two factor scheme entirely" any more than a website giving a user a TOTP secret. Both are shared secrets known to the website. Both can be written down. Both can be used for 2FA.

Users who don't enable recovery codes will then be at the mercy of the recovery process determined by the site admins.

I'm not following, what does this have to do with recovery codes? You could apply this to any 2FA method. For example, users who enable just WebAuthn but lose their key "will then be at the mercy of the recovery process determined by the site admins."

Anyway, thanks for letting know, I'm glad that recovery codes are on the PyPI road-map, even if not as part of the OTF work.

Not sure how a website giving a user recovery codes "circumvents the two factor scheme entirely" any more than a website giving a user a TOTP secret. Both are shared secrets known to the website. Both can be written down. Both can be used for 2FA.

Yeah, this is admittedly a hazy distinction. TOTP preserves the second factor property by requiring a transform on the secret + windowed timestamp that the user (probably) can't do in their head and thus turn into something they know, but a user could always save their TOTP secret somewhere that isn't sequestered and violate the scheme in that way.
OTOH, recovery codes are (usually) trivial for users to memorize. They probably won't memorize them so maybe that's a moot distinction, but it's common enough to see people save their codes on disk instead of printing them out, stash them in their password managers, &c. Combined with the absence of a transform that humans can't easily perform, I'd say this compromises their ability to be a "real" second factor. Admittedly nitpicky.

I'm not following, what does this have to do with recovery codes? You could apply this to any 2FA method. For example, users who enable just WebAuthn but lose their key "will then be at the mercy of the recovery process determined by the site admins."

Ah, sorry. Implicit in that is the assumption that account recovery requests won't be honored if the account in question has 2FA enabled and the user can't prove that they possess that second factor. But I don't know if that's actually been settled on yet.

Note: we already have draft help text for this feature - see https://github.com/pypa/warehouse/issues/5586

Mockups for this feature can be found here: https://github.com/pypa/warehouse/issues/5587

Guys, I currently can't log in to my PyPI account because for some reason, my TOTP codes aren't being accepted. If I had recovery codes, I'd be good, but I'm currently locked out of my account. I maintain a popular open-source project so this is kind of a problem. Besides implementing recovery codes, can anyone help me regain control of my account?

@cool-RR do you have an update on this?

Ernest fixed the problem with my account.

@ewdurbin @cool-RR do I have to contact someone regarding this??

@pidugusundeep please open an issue to track your recovery request.

Thank you @ewdurbin!


Add basic modal component (Issue #56)

Adds a basic version of the modal component. This component will still need additional updates to align with the functionality of the main Flowbite modal component. This first version implements the functions:

show
hide
toggle

Thanks for your PR, can you fix issues, which I found?

Yes! Sorry about the delay – PR is now updated.

Thank you!


Lineage API response is incorrect when only of the queryParam (fw or bw) is passed

Problem:
Hitting http://localhost:9000/v1/lineage/table/LOREM?bw=2 returns TableNotFound
The API returns full lineage if queryParams are not passed.
For Eg  -
http://localhost:8080/api/v1/lineage/table/LOREM- returns full backward and forward lineage
http://localhost:8080/api/v1/lineage/table/LOREM?bw=0 returns full forward Lineage
http://localhost:8080/api/v1/lineage/table/LOREM?fw=0 returns full backward Lineage
But searching for any different depth for a single param results in error  -
http://localhost:8080/api/v1/lineage/table/LOREM?bw=2  returns Table "LOREM" not found
Instead it should return http://localhost:8080/api/v1/lineage/table/LOREM?bw=2 => Backward Lineage at depth 2 and full forward Lineage
Expectations: Hitting http://localhost:8080/api/v1/lineage/table/LOREM?bw=2 => Backward Lineage at depth 2 and full forward Lineage

I would like to work on it as part of GraceHopperOSD.

Problem: Hitting http://localhost:9000/v1/lineage/table/LOREM?bw=2 returns TableNotFound
The API returns full lineage if queryParams are not passed. For Eg - http://localhost:8080/api/v1/lineage/table/LOREM- returns full backward and forward lineage http://localhost:8080/api/v1/lineage/table/LOREM?bw=0 returns full forward Lineage http://localhost:8080/api/v1/lineage/table/LOREM?fw=0 returns full backward Lineage
But searching for any different depth for a single param results in error - http://localhost:8080/api/v1/lineage/table/LOREM?bw=2 returns Table "LOREM" not found
Instead it should return http://localhost:8080/api/v1/lineage/table/LOREM?bw=2 => Backward Lineage at depth 2 and full forward Lineage
Expectations: Hitting http://localhost:8080/api/v1/lineage/table/LOREM?bw=2 => Backward Lineage at depth 2 and full forward Lineage


buySubscription/requestSubscription type mismatch

Version of react-native-iap
3.0.1
Description
The return type for buySubscritpion and requestSubscription do not match. The old, deprecated api buySubscription returns a Promise<SubscriptionPurchase>. The new requestSubscription return a Promise<string>. It looks like they both use the same implementation, what should be the correct type?

It should be Promise<string | SubscriptionPurchase> currently. However, if buySubscription is removed in the future, it will be Promise<string. Should be improved though! Hope you can give us a PR.

what do you think about returning a Promise<void> since we should now rely on purchaseUpdatedListener/purchaseErrorListener ? It might also be better to remove buySubscritpion from 3.x.x instead of deprecating since the old flow doesn't work anymore without the listeners and purchase acknowledgment.

@benoitdion Yes~! Thanks for sharing, I will keep this in mind for future updates.


Request awesome list "admins" to move repos to awesome-lists organization

https://github.com/awesome-lists
this would be great in case some of the repo owners go MIA or else,someone else could pick it up, it would also just be great to put these in a single account
@sindresorhus what do you think about this, agree? or maybe the idea needs improvement?

I set that org up in case an owner doesn't want to maintain/own a list anymore. I'm not going to actively encourage people to move lists there. In general lists are better off with a BDFL.


AWSSDK.Extensions.NETCore.Setup -"Unable to find credentials" happens on Nuget Version >= 3.3.1, works OK on version <IP_ADDRESS>

appsettings.json content:
{
  "AWS": {
    "Profile": "Development",
    "Region": "eu-central-1"
  }
}

Target framework: .NetCore 2.0
program.cs code:
class Program
    {
        static void Main(string[] args)
        {
            var path = Path.GetDirectoryName(typeof(Program).GetTypeInfo().Assembly.Location);

            var configurations = new ConfigurationBuilder()
                .SetBasePath(path)
                .AddJsonFile("appsettings.json", optional: false, reloadOnChange: true)
                .Build();
            var services = new ServiceCollection();

            var awsOptions = configurations.GetAWSOptions();
            services.AddDefaultAWSOptions(awsOptions);
            services.AddAWSService<IAmazonS3>();

            var serviceProvider = services.BuildServiceProvider();

            var s3Client = serviceProvider.GetService<IAmazonS3>();

            Console.ReadLine();
        }
    }

Referencing AWSSDK.Extensions.NETCore.Setup nuget package version 3.3.1 and above.
Expected Behavior
The s3Client should be resolved successfully.
Current Behavior
If update the nuget package to version <IP_ADDRESS>, the above code can run successfully.
If use any versions >= 3.3.1, the following exception is thrown:
Amazon.Runtime.AmazonServiceException
HResult=0x80131500
Message=Unable to find credentials
Exception 1 of 3:
Amazon.Runtime.AmazonClientException: Unable to find the 'default' profile in CredentialProfileStoreChain.
at Amazon.Runtime.FallbackCredentialsFactory.GetAWSCredentials(ICredentialProfileSource source) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 72
at Amazon.Runtime.FallbackCredentialsFactory.GetCredentials(Boolean fallbackToAnonymous) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 115
Exception 2 of 3:
System.InvalidOperationException: The environment variables AWS_ACCESS_KEY_ID/AWS_SECRET_ACCESS_KEY/AWS_SESSION_TOKEN were not set with AWS credentials.
at Amazon.Runtime.EnvironmentVariablesAWSCredentials.FetchCredentials() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials_bcl+coreclr\EnvironmentVariablesAWSCredentials.cs:line 80
at Amazon.Runtime.EnvironmentVariablesAWSCredentials..ctor() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials_bcl+coreclr\EnvironmentVariablesAWSCredentials.cs:line 58
at Amazon.Runtime.FallbackCredentialsFactory.<>c.b__9_1() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 55
at Amazon.Runtime.FallbackCredentialsFactory.GetCredentials(Boolean fallbackToAnonymous) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 115
Exception 3 of 3:
System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.WinHttpException: A connection with the server could not be established
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Threading.Tasks.RendezvousAwaitable1.GetResult()    at System.Net.Http.WinHttpHandler.<StartRequest>d__105.MoveNext()    --- End of inner exception stack trace ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()
at System.Net.Http.HttpClient.d__59.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.ConfiguredTaskAwaitable1.ConfiguredTaskAwaiter.GetResult()    at System.Net.Http.HttpClient.<GetStringAsyncCore>d__27.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Amazon.Runtime.Internal.Util.AsyncHelpers.<>c__DisplayClass1_11.<b__0>d.MoveNext() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Internal\Util_mobile\AsyncHelpers.cs:line 79
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at Amazon.Runtime.Internal.Util.AsyncHelpers.ExclusiveSynchronizationContext.BeginMessageLoop() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Internal\Util_mobile\AsyncHelpers.cs:line 142
at Amazon.Runtime.Internal.Util.AsyncHelpers.RunSync[T](Func`1 task) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Internal\Util_mobile\AsyncHelpers.cs:line 87
at Amazon.Util.AWSSDKUtils.DownloadStringContent(Uri uri, TimeSpan timeout) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Util\AWSSDKUtils.cs:line 895
at Amazon.Runtime.URIBasedRefreshingCredentialHelper.GetContents(Uri uri) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\URIBasedRefreshingCredentialHelper.cs:line 32
at Amazon.Runtime.InstanceProfileAWSCredentials.d__10.MoveNext()
at Amazon.Runtime.InstanceProfileAWSCredentials.GetFirstRole() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\InstanceProfileAWSCredentials.cs:line 204
at Amazon.Runtime.FallbackCredentialsFactory.ECSEC2CredentialsWrapper() in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 94
at Amazon.Runtime.FallbackCredentialsFactory.GetCredentials(Boolean fallbackToAnonymous) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 115
Source=AWSSDK.Core
StackTrace:
at Amazon.Runtime.FallbackCredentialsFactory.GetCredentials(Boolean fallbackToAnonymous) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\sdk\src\Core\Amazon.Runtime\Credentials\FallbackCredentialsFactory.cs:line 150
at Amazon.Extensions.NETCore.Setup.ClientFactory.CreateCredentials(AWSOptions options) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\extensions\src\AWSSDK.Extensions.NETCore.Setup\ClientFactory.cs:line 125
at Amazon.Extensions.NETCore.Setup.ClientFactory.CreateServiceClient(Type serviceInterfaceType, AWSOptions options) in E:\JenkinsWorkspaces\v3-trebuchet-release\AWSDotNetPublic\extensions\src\AWSSDK.Extensions.NETCore.Setup\ClientFactory.cs:line 72
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitFactory(FactoryCallSite factoryCallSite, ServiceProvider provider)
at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor2.VisitCallSite(IServiceCallSite callSite, TArgument argument)    at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitScoped(ScopedCallSite scopedCallSite, ServiceProvider provider)    at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteRuntimeResolver.VisitSingleton(SingletonCallSite singletonCallSite, ServiceProvider provider)    at Microsoft.Extensions.DependencyInjection.ServiceLookup.CallSiteVisitor2.VisitCallSite(IServiceCallSite callSite, TArgument argument)
at Microsoft.Extensions.DependencyInjection.ServiceProvider.<>c__DisplayClass22_0.b__0(ServiceProvider provider)
at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType)
at Microsoft.Extensions.DependencyInjection.ServiceProviderServiceExtensions.GetService[T](IServiceProvider provider)
at ConsoleApp1.Program.Main(String[] args) in c:\users\quju\documents\visual studio 2017\Projects\ConsoleApp_AWSSetupTest\ConsoleApp1\Program.cs:line 28
Possible Solution
Is the current latest package updated with the correct version number?
Steps to Reproduce (for bugs)
Referencing nuget package version >=3.3.1
Your Environment

AWSSDK.Core version used: <IP_ADDRESS>
Visual Studio version: 15.5.2
Targeted .NET platform: .Net Core 2.0

It looks like SDK is failing to resolve the credentials.  How are you exposing it? Do you have a shared credentials file (~/.aws/credentials)?

Closing due to inactivity.

@sstevenkang I am experiencing similar exceptions to 3rd one that @junjiequ listed. I am using a shared credentials file at ~/.aws/credentials. Any idea what could be going wrong here? I have also oppened an issue #1086

I just had to update AWSSDK.S3 to <IP_ADDRESS>


Next Release

Any chance to get an official release with the oidc support in it?
https://github.com/bitly/oauth2_proxy/commit/cb48577ede89837567a4d030570cc7cbdbf245dc
Thanks,
Kevin

need the open id connect provider too :( any updates?

Gently ping


Compreface setup failed

Compreface started logging this error and no predition is done.
compreface:
  face_recognition:
    host: localhost
    port: 8000
    recognition_api_key:
    cameras:
      camera_1:
    labels:
      - person

2023-05-14 11:44:01 [2023-05-14 09:44:01] [WARNING ] [viseron.components.darknet.object_detector.camera_1] - scan_on_motion_only is enabled but no motion detector is configured. Disabling scan_on_motion_only
2023-05-14 11:44:01 [2023-05-14 09:44:01] [ERROR   ] [viseron.components] - Uncaught exception setting up domain face_recognition for component compreface: [Errno 2] No such file or directory: '/config/face_recognition/faces'
2023-05-14 11:44:01 Traceback (most recent call last):
2023-05-14 11:44:01   File "/src/viseron/components/__init__.py", line 407, in setup_domain
2023-05-14 11:44:01     result = domain_module.setup(
2023-05-14 11:44:01   File "/src/viseron/components/compreface/face_recognition.py", line 45, in setup
2023-05-14 11:44:01     FaceRecognition(vis, config, identifier)
2023-05-14 11:44:01   File "/src/viseron/components/compreface/face_recognition.py", line 54, in __init__
2023-05-14 11:44:01     super().__init__(
2023-05-14 11:44:01   File "/src/viseron/domains/face_recognition/__init__.py", line 90, in __init__
2023-05-14 11:44:01     for face_dir in os.listdir(config[CONFIG_FACE_RECOGNITION_PATH]):
2023-05-14 11:44:01 FileNotFoundError: [Errno 2] No such file or directory: '/config/face_recognition/faces'
2023-05-14 11:44:01 [2023-05-14 09:44:01] [INFO    ] [viseron.components] - Setup of domain object_detector for component darknet with identifier camera_1 took 0.0 seconds
2023-05-14 11:44:01 [2023-05-14 09:44:01] [ERROR   ] [viseron.components] - Setup of domain face_recognition for component compreface with identifier camera_1 failed
2023-05-14 11:44:01 [2023-05-14 09:44:01] [INFO    ] [viseron.components.nvr.nvr.camera_1] - Motion detector is disabled
2023-05-14 11:44:01 [2023-05-14 09:44:01] [INFO    ] [viseron.components.nvr.nvr.camera_1] - NVR for camera video_loop initialized
2023-05-14 11:44:01 [2023-05-14 09:44:01] [INFO    ] [viseron.components] - Setup of domain nvr for component nvr with identifier camera_1 took 0.0 seconds
2023-05-14 11:44:01 [2023-05-14 09:44:01] [INFO    ] [viseron.core] - Viseron initialized in 242.0 seconds
2023-05-14 11:44:04 [2023-05-14 09:44:04] [ERROR   ] [viseron.components.compreface.face_recognition.camera_1] - Error calling compreface: 'FaceRecognition' object has no attribute '_recognition'
2023-05-14 11:44:04 Traceback (most recent call last):
2023-05-14 11:44:04   File "/src/viseron/components/compreface/face_recognition.py", line 96, in face_recognition
2023-05-14 11:44:04     detections = self._recognition.recognize(
2023-05-14 11:44:04 AttributeError: 'FaceRecognition' object has no attribute '_recognition'
2023-05-14 11:44:05 [2023-05-14 09:44:05] [ERROR   ] [viseron.components.compreface.face_recognition.camera_1] - Error calling compreface: 'FaceRecognition' object has no attribute '_recognition'
2023-05-14 11:44:05 Traceback (most recent call last):
2023-05-14 11:44:05   File "/src/viseron/components/compreface/face_recognition.py", line 96, in face_recognition
2023-05-14 11:44:05     detections = self._recognition.recognize(
2023-05-14 11:44:05 AttributeError: 'FaceRecognition' object has no attribute '_recognition'
2023-05-14 11:44:06 [2023-05-14 09:44:06] [ERROR   ] [viseron.components.compreface.face_recognition.camera_1] - Error calling compreface: 'FaceRecognition' object has no attribute '_recognition'

Training is disabled by default so I thought that no directory is needed in order to make compreface work

Using face_recognition_path: "." fixed the problem, but in my case no faces are recognized. Can faces be detected if faces are added using the original CompreFace UI?

I saw the code and sensors are created based on the images in face_recognition_path. However we could try to use this endpoint from CompreFace to add such sensors. Maybe after the train task. I could try to do it by myself but I need time to setup the dev env.

I noticed I didn't even disabled training at all, that's why I was getting the error above. Sorry for the trouble. I'm still going to make a issue for the faces already trained in the CompraFace UI.

https://github.com/roflcoopter/viseron/blob/c5da57d15bf1f7adecfe81e03cd127eb42d82610/viseron/components/compreface/__init__.py#L122
This condition seems to not work properly.
File "/src/viseron/domains/face_recognition/__init__.py", line 90, in __init__
for face_dir in os.listdir(config[CONFIG_FACE_RECOGNITION_PATH]):

Error is thrown even if train is set to false.


Binary with name "npx" not found

Hi,
I am trying to build my page using this theme, which requires postcss, with Nix.
I have tried adding all the necessary packages as input dependencies, which did not work.
Then I tried creating a FHS environment and running hugo inside of that. This did not work either. In both cases npx was available in the $PATH, while hugo was complaining it was unable to find it.
The error message is the following:
Start building sites …
hugo v0.120.3+extended linux/amd64 BuildDate=unknown VendorInfo=nixpkgs

ERROR POSTCSS: failed to transform "css/content.css" (text/css). Check your PostCSS installation; install with "npm install postcss-cli". See https://gohugo.io/hugo-pipes/postcss/: binary with name "npx" not found
ERROR POSTCSS: failed to transform "css/index.css" (text/css). Check your PostCSS installation; install with "npm install postcss-cli". See https://gohugo.io/hugo-pipes/postcss/: binary with name "npx" not found
ERROR POSTCSS: failed to transform "css/single.css" (text/css). Check your PostCSS installation; install with "npm install postcss-cli". See https://gohugo.io/hugo-pipes/postcss/: binary with name "npx" not found
ERROR POSTCSS: failed to transform "css/list.css" (text/css). Check your PostCSS installation; install with "npm install postcss-cli". See https://gohugo.io/hugo-pipes/postcss/: binary with name "npx" not found
Total in 8139 ms
Error: error building site: POSTCSS: failed to transform "css/common.css" (text/css). Check your PostCSS installation; install with "npm install postcss-cli". See https://gohugo.io/hugo-pipes/postcss/: binary with name "npx" not found

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.120.3+extended linux/amd64 BuildDate=unknown VendorInfo=nixpkgs

Which theme? You linked to this issue, not the theme.

Oh sorry, forgot to add in the link - fixed it.
It's this one: https://github.com/michaelneuper/hugo-texify3

hugo v0.120.4-f11bca5fec2ebb3a02727fb2a5cfb08da96fd9df+extended linux/amd64 BuildDate=2023-11-08T11:18:07Z VendorInfo=gohugoio

Works for me on Ubuntu:
hugo new site mysite
cd mysite/
git init
git submodule add https://github.com/michaelneuper/hugo-texify3 themes/hugo-texify3
echo "theme = 'hugo-texify3'" >> hugo.toml
cp themes/hugo-texify3/{package*,postcss*} .
npm ci
hugo

Seems to work, although I am still not sure, why I am not able to use this with just the binaries and the npm ci call does not seem to be documented anywhere.

https://docs.npmjs.com/cli/v8/commands/npm-ci#description

No I mean it is not documented that this is necessary for Hugo Pipes to work correctly

You can use either npm ci or npm install, but npm ci is safer.


FEATURE: Would you consider this theme suitable for launching on the Zed extension?

I am a fan of this theme and would like to inquire about any plans to launch it on the Zed editor. Could you please confirm?🙏🙏🙏

Hey, thank you for requesting a new feature. I've noticed someone has already ported it to Zed, although there are some inconsistencies with the original theme. Anyway, I'll look into it in the next few days.


Task::Star in Travis

As per the Perl 6 installation instructions on rakudo.org:
This will install all the modules that are shipped with the Rakudo Star Perl 6 distribution

Scratch that, this is overkill.


Disambiguate compiler-interface module name

It's compiled only in Scala 2, and used via dotty compat in Scala 3. For Scala 2, it depends on Scala 2 pprint. For Scala 3, it depends on Scala 3 pprint.
Without these changes, only the latter module ends up being published, which makes ammonite for 2.13.6 pull pprint for Scala 3 (along with pprint for Scala 2), which is a problem.
Also, the same module (com.lihaoyi:ammonite-compiler-interface_2.13.6) was built and published twice. A few commits ago, these were published in the same batch, so only one (apparently the latter) ended up actually published. Since two commits on master, these are published in separate batches, which makes one of them fail (because it tries to publish a module already published by the other batch).
With the changes here, we now publish com.lihaoyi:ammonite-compiler-interface_2.13.6 (for Scala 2) and com.lihaoyi:ammonite-cross-23-compiler-interface_2.13.6 (for Scala 3), which solves the ambiguity.
Fixes https://github.com/com-lihaoyi/Ammonite/issues/1214.
Fixes https://github.com/com-lihaoyi/Ammonite/issues/1197 (hopefully).

Not sure why I didn't just build compiler-interface with Scala 3, instead of compiling it with Scala 2 and using it from Scala 3 via dotty-compat… Will probably try that in a later PR.


Launching Assassin's Creed Origins on wine

Hello,
Does somebody already launched Assassin's Creed Origins under wine and how ?
In my case, the game does not start at all. Not a single trace... (sorry)
All others AC Games are working pretty well except this one...
Sincerely
Here is my setup :
Software information
Assassin's Creed Origins
System information

GPU: GTX 970
Driver: 410.57
Wine version: 3.18 staging
DXVK version: 0.90

Log files

d3d11.log: Empty
dxgi.log: Empty

Not possible due to DRM issues. This game runs fine with DXVK on Windows.


ToDo before publication

[ ] add SSDraw (https://github.com/ncbi/SSDraw)
[ ] make retrieval of structure more robust and allow alphafold structures
[ ] add preview image of the svg in the output
[ ] add different profiles for cartoon/sphere/spacefill
[ ] add option to get just the outline of the protein similar to these illustrations:  Could potentially also do:

get cutout and color scan, merge them. Sort the colors by lightness and cluster them to give a bit more texture

Add functions to set illustrative or stylized styles using this code
Reference see here: https://github.com/molstar/molstar/blob/6edbae80db340134341631f669eec86543a0f1a8/src/mol-plugin-ui/structure/quick-styles.tsx#L84C11-L84C19
viewerInstance.plugin.canvas3d.setProps({                 postprocessing: {                     outline: {                         name: 'on',                         params: {                             scale: 1,                             color: {r:0,b:0, g:0},                             threshold: 0.25,                             includeTransparent: true,                         }                     },                     occlusion: {                         name: 'on',                         params: {                             multiScale: { name: 'off', params: {} },                             radius: 5,                             bias: 0.8,                             blurKernelSize: 15,                             samples: 32,                             resolutionScale: 1,                             color: {r:0,b:0, g:0},                         }                     },                     shadow: { name: 'off', params: {} },                 }             });


Skill - Death

Instantly skill the enemy if their name matches a specified string.
As SKILL_EFF0 doesn't accept strings, I've had to put the string to check for inside the skill code block itself for now.
Originally this was called 'DownWithArch' but I renamed it to be more generic.

I have to say that this skill doesn't make much sense... users can go to pre-battle to simply insert a slight function to determine the character's index and then all things have done...

I have to say that this skill doesn't make much sense... users can go to pre-battle to simply insert a slight function to determine the character's index and then all things have done...

It's a gimmick skill sure, but I think it's nice to have if only as a working example of using strings to accomplish something.

It is not reasonable to complicate a simple issue.

Fine, I'll close it.
For the record though, I don't agree. It's not complicated, just thinking outside the box.


Update snippet-ARM-existing-resource-group.md

Fix of plan location parameter name in description

Fixing in private repo

Fixing in private repo


docs: add video demo

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

You could do a relative URL I presume but LGTM either way!

I just dragged the .mp4 to the readme using GitHub UI. It embeds the player automatically and generates the URL. Works awesome!


"Run As -> Server" option missing from GAE projects

Projects created with the "Google App Engine Standard Java Project" wizard and the "Maven-based Google App Engine Standard Project" wizard are missing the Server running option under the context menu "Run As".

I think this happens if you don't have the Eclipse JEE version installed.
You can either download that version of Eclipse, or install the Eclipse JEE tooling from the Eclipse update site.


[WFCORE-3897] Add test coverage

Associated JIRA's:
https://issues.jboss.org/browse/WFCORE-3897
This adds test coverage for WFRCORE-3897 - Host starts with server assigned to non-existent server group.

Can one of the admins verify this patch?


Converting between YCbCr and RGB

Converting between YCbCr and RGB
Some may argue, how can we produce all the colors without using the green?
To answer this question we'll walk through a conversion from RGB to YCbCr. We'll use the coefficients from the standard BT.601 that was recommended by the group ITU-R* . The first step is to calculate the luma, we'll use the constants suggested by ITU and replace the RGB values.
Y = 0.299R + 0.587G + 0.114B
Once we had the luma, we can split the colors (chroma blue and red):
Cb = 0.564(B - Y)
Cr = 0.713(R - Y)
And we can also convert it back and even get the green by using YCbCr.
R = Y + 1.402Cr
B = Y + 1.772Cb
G = Y - 0.344Cb - 0.714Cr

is this done for each pixel ?
and i didn't understand how did you use the diagram to get the Y = R + G + B  values.. and where did you get the constants from
can you give an example ?

is this done for each pixel?

... constants ?

These coefficients varies from standard to standard, for instance, we're using BT.601 (ITU-R)

thank you !
are the R' G' B' which are multiplied by the coefficients the same as the 3 points making the triangle that is representing the color in the diagram ?


Add mathkeyboard test using random inputs

Adds 100 random keypresses to a MathKeyboard and gets LaTeX and checks that there is no crash

Why add another test project?
Also, why use nunit instead of xUnit?

So I can use F#. You are welcome to change from nunit to xUnit if you'd like.

System.Exception : Exeption: Object reference not set to an instance of an object. inputs: [A; Sine; LeftRoundBracket; LeftRoundBracket; BothRoundBrackets]

System.Exception : Exeption: Object reference not set to an instance of an object. inputs: [Power; Up; Left; B; B; Power]

Yep, it is: #78. I used subscript for description there but the problem is exactly the same.

It gets the shortest error case and tends to find [Power; Left; Power] now, again the same error.


Upgrade for cassandra-driver>=3.1.0 & bump to 0.2.0+dd.8

Feel free to merge this if you want to btw. I assume you know this but just in case not, you'll want to add a github release (with git tag) for the new version. Then I think you'll want to upgrade the user's setup.py with something like "beaker_extensions[cassandra_cql]==0.2.0+dd.8". If that doesn't work and you want me to sort it out, I'm happy to.

LGTM


Enable double click to set icons directly

Setting an icon from the modal icon requires two actions: 1) selecting the icon and 2) clickin the set button.
We could enable double click to do both actions at once and make the task super quick ⚡

Quick actions! I like it!


Added float support for footer

Adding support to float Footer component similar to the way we float header.

@ericsoderberg @bryanjacquot any extra look here? :+1:


fix(README): Update to compile example against Rust beta

Remove unused import
res doesn't need to be mutable in parameter list
Change .listen() to string format

Thanks!


fix: custom loading indicator in data grid

🐛 Bug Report
Custom loading indicator, larger than the standard one goes outside the data grid area.

💻 Repro or Code Sample
https://github.com/hedge-code/HiddenLoadingIndicatorReproduction
🤔 Expected Behavior
The grid should adjust its size to the loading indicator.
😯 Current Behavior
Part of the indicator hides under the data grid border.
🔦 Context
This is a new regression. In version 4.7.1 it worked fine
🌍 Your Environment
OS & Device: Windows on PC
Browser Google Chrome 125.0.x.x
.NET and Fluent UI Blazor library Version 8.0.200 and 4.7.2

Fixed in PR mentioned


Allow re-ordering of the plugins array via config

This allows users to add plugins to the existing array as normal, but also gives them control over re-ordering existing ones.
Currently object-assign-deep just concats arrays, which means there is no order control.
Say for example I need stylelint to run before autoprefixer:
// defaults
[
    'postcss-import',
    'postcss-custom-properties',
    'postcss-calc',
    'postcss-custom-media',
    'autoprefixer',
    'postcss-reporter'
  ]

// config
{use: ['stylelint', 'autoprefixer', 'postcss-reporter']}

// result
[
    'postcss-import',
    'postcss-custom-properties',
    'postcss-calc',
    'postcss-custom-media',
    'stylelint',
    'autoprefixer',
    'postcss-reporter'
  ]

This could also be useful to re-order the existing plugins if needed.
// config
{use: ['autoprefixer', 'postcss-at2x', 'postcss-calc', 'postcss-reporter']}

// result
[
  'postcss-import',
  'postcss-custom-properties',
  'postcss-custom-media',
  'autoprefixer',
  'postcss-at2x',
  'postcss-calc',
  'postcss-reporter'
]

This need came out of trying to get stylelint to work and having autoprefixer get in the way, but I think it's a nice enhancement anyway.
I removed the check for postcss-reporter being last, because now the user has control over that easily.
Also made the lodash dependency smaller by using the individual packages, but annoyingly I think postcss-reporter uses the full library anyway.
@giuseppeg If this looks good I'll update the README too :)

@simonsmith yeah it looks good! I wonder if deepmerge was doing that already.

Good question! I will investigate first

It does dedupe arrays, but not in the way that would work for this - http://jsbin.com/genuci/edit?js,console

fair enough, I just asked out of curiosity ;)

Would have been handy!