make run fail in database crd test

Search before asking

[X] I had searched in the issues and found no similar issues.

Environment
Linux (self-host)
CPU Architecture
amd64
laf version
1.0.x
What happened
I had set sealos cluster by multipass yesterday , then I do the following steps by the readme content that is in the laf/controllers/database,I type make run ,and run it ,but error occur.
How to reproduce

set up sealos cluster
exec make run command in the laf/controllers/database/ directory
error occur

Debug logs
/home/alice/Desktop/demos/laf/controllers/database/bin/controller-gen rbac:roleName=manager-role crd webhook paths="./..." output:crd:artifacts:config=config/crd/bases /home/alice/Desktop/demos/laf/controllers/database/bin/controller-gen object:headerFile="hack/boilerplate.go.txt" paths="./..." /usr/local/go/src/net/cgo_linux.go:12:8: no such package located Error: not all generators ran successfully run controller-gen object:headerFile=hack/boilerplate.go.txt paths=./... -wto see all available markers, orcontroller-gen object:headerFile=hack/boilerplate.go.txt paths=./... -h for usage make: *** [Makefile:48: generate] Error 1
Are you willing to submit PR?

[x] Yes I am willing to submit a PR!

uh,it's need gcc...after install gcc in ubuntu,error disappeared


Conversion to CSC matrices is slow

Currently the Python API requires P and A to be in CSC format.
Problem is: on small instances, it can take much more time to convert to CSC than to solve the QP itself. In this  small example where P and A are 3x3, CSC conversions takes ~200 µs on my machine, versus ~20 µs taken to solve the QP itself.
In [17]: P.shape
Out[17]: (3, 3)

In [18]: %timeit csc_matrix(P)
10000 loops, best of 3: 116 µs per loop

Could the API allow for dense numpy.array arguments for P and A?

Thank you for adding OSQP to your unified QP API! OSQP internally supports sparse matrices. Thus, the dense-to-sparse conversion has to happen somewhere.
Not that most solvers, including commercial ones, support sparse matrices internally. In most practical applications, problems have a very specific sparse structure.
In my opinion, I would use the time returned by the individual solvers to make the comparison fair. The Python timer includes some overhead that could change the outcome.
OSQP returns the internally computed run time in the results structure as results.info.run_time.  GUROBI, for example, returns the computation time in model.Runtime.
I suggest you to use those values instead of the timeit reported time.

The fact that it is slow is related to Scipy.

Yes, that's why I was wondering if there would be a way to circumvent Scipy, for instance if you subsequently convert from Scipy to your own internal representation?

I suggest you to use those values instead of the timeit reported time.

Well, at the end of the day, the price paid is total_time = build_time + run_time. (Feel free to let me know if there is a better way than %timeit to measure this total computation time.)
For example, the MPC I'm working on runs with a loop cycle of 5 ms (where P and A are state-dependent so the QP is rebuilt at each cycle). I think Scipy's sparse matrices are too slow for such settings (e.g., it takes ~50 µs to update one entry of these matrices).

The OSQP representation is Compressed Sparse Column (CSC) format which corresponds exactly to the Scipy representation. No conversion from scipy.sparse.csc_matrix is performed.
Python is mainly used as a prototyping language. It is useful to check how much time it takes if the solver is called from a lower level language (C/C++).
If you are dealing with MPC at 5ms loop cycles, I suggest you to check the timings as I said before, and then to implement the loop in a faster language (C/C++ or also Julia/Matlab).
Note that there are two possible MPC formulations: dense (suitable for qpOASES) where the only variables are the inputs and sparse (suitable for sparse solvers like OSQP) where the variables are both inputs and states.
If you have a question related to your MPC formulation as QP, I am happy to discuss it in the OSQP mailing list.

OK. Thanks for your feedback :)


rules/sdk: iterate over maps should suppress protobuf related code and auto-generated code in pb.go and pb.*.go files

There is a bunch of code that is flagged but it is generated from the protobuf compiler. We should suppress such code from pb.*.go files
/cc @kirbyquerby

Better yet, if we want to avoid all generated code, we can even search for the standard regex pattern for generated files:
^// Code generated .* DO NOT EDIT\.$

Also see:

https://github.com/golang/go/issues/13560

Great idea, and yes that shall be our primary filter! Let's also check for the extension :-)

This would be huge!


Readme update

updated name of script, forgot to do this in the last PR;
mentioned some important info about how scripts are deployed to Jenkins.

The changes we decided to keep from this branch are now in https://github.com/alphagov/digitalmarketplace-scripts/pull/179


Feature: Allow this program to install for multiple users.

I use this program to help me keep track of computer use time for my children and I have it send me alerts when the log in and 30, 15, 5, minutes till the end of their hour and then one for the completion of that hours.
This all works well if remember to sign into the computer first and then let them use their own user accounts. But it does not work at all if I let the kids use the computer.
A solution would be to install this program for all users in the Program Files directory. and have a system wide and user settings. This would allow users to create their own settings and it would allow the admin to crate settings that will affect everyone
Right now i tried having it launch using Task scheduler using my admin account and works if i soft reboot but it does not work if its a cold boot. Doesnt make sense

Hmm, let me propose two things - that'll show if I understand the problem correctly or I'm missing something :D

HASS.Agent (the agent itself, not satellite service) can be installed on "per user" basis - you could install it for each of the lil one's pc's and then have separate sensors for each of them

Satellite Service - it's installed to Program Files and runs with the system rather than with the user session - LoggedUser/LoggedUsers sensors could tell you who is using the PC and then create a counter for example in HA or nodered


docs(hugo): Modify updated line in footer.

This PR overrides page-meta-lastmod.html provided by the Docsy theme to remove the commit subject line from the "Last Updated" line in the page footer.

@alexfornuto Awesome! LGTM - merge at your leisure.


d.ts result Minimap dose not export

Describe the bug
in diagramMakerMinimap.d.ts
declare class Minimap<NodeType, EdgeType> extends React.Component<MinimapProps<NodeType, EdgeType>> {
render(): JSX.Element | undefined;
private getCanvasSize;
private getMinimapNode;
private renderMinimapNodes;
}
Minimap dose not export
To Reproduce
Steps to reproduce the behavior:

typescript import Minimap from this package

Expected behavior
in d.ts export Minimap
Screenshots

Desktop (please complete the following information):

OS: OSX
Browser chrome
Version 89

Additional context
none

Could you verify after using the latest version? Looks like it should have been fixed now. Feel free to close the ticket if its all good.

Closing this issue because this has been stale over weeks now. If this remains to be an issue, let us know and we will reopen this issue.


Clean up metrics

Describe your changes

Remove clipped metrics
Remove python implementations of metrics (resulted in unwanted NAs / NaNs)
Add clipped_sign_log10_pval layer to the kaggle dataset
Set default layer to clipped_sign_log10_pval

Checklist before requesting a review

[x] I have performed a self-review of my code

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality (new method, new metric, ...)
[ ] Major changes
[x] Minor changes
[x] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[ ] CI Tests succeed and look good!

I would rather make all of the metrics (or have a component that clips the input) clip, to avoid the potential issue of penalizing methods that may still predict values beyond the threshold.

Just to make a note of this:
We decided to clip the input values of the sign_log10_pval by creating a clipped_sign_log10_pval. Clipping the predictions to the same range as that layer would improve some method's rmse and mae score, but in practice the effect is found to be negligible:
   dataset_id        method_id                        rmse   mae pearson spearman cosine   diff_rmse   diff_mae diff_pearson diff_spearman diff_cosine
   <chr>             <chr>                           <dbl> <dbl>   <dbl>    <dbl>  <dbl>       <dbl>      <dbl>        <dbl>         <dbl>       <dbl>
 1 neurips-2023-data ground_truth                    0     0      1        1      1       0           0            0               0         0
 2 neurips-2023-data jn_ap_op2                       0.894 0.649  0.327    0.306  0.329  -0.00105    -0.00111     -0.000228       -4.08e-7  -0.000225
 3 neurips-2023-data lgc_ensemble                    0.794 0.577  0.450    0.423  0.454  -0.00690    -0.00699     -0.000845       -3.87e-5  -0.000850
 4 neurips-2023-data mean_across_celltypes           0.892 0.644  0.297    0.281  0.302   0           0            0               0         0
 5 neurips-2023-data mean_across_compounds           0.943 0.698  0.259    0.243  0.263   0           0            0               0         0
 6 neurips-2023-data mean_outcome                    0.899 0.636  0.220    0.212  0.226   0           0            0               0         0
 7 neurips-2023-data nn_retraining_with_pseudolabels 0.756 0.547  0.490    0.462  0.493   0           0            0               0         0
 8 neurips-2023-data pyboost                         0.795 0.560  0.462    0.440  0.465  -0.00000491 -0.0000147    0.0000132      -1.18e-9   0.0000133
 9 neurips-2023-data sample                          1.37  0.965  0.0492   0.0517 0.0515  0           0            0               0         0
10 neurips-2023-data scape                           0.774 0.571  0.473    0.442  0.476  -0.000392   -0.000637    -0.0000266      -3.78e-7  -0.0000270
11 neurips-2023-data transformer_ensemble            0.897 0.628  0.220    0.216  0.226   0           0            0               0         0
12 neurips-2023-data zeros                           0.918 0.635  0        0      0       0           0            0               0         0


CSRF Token mismatch

Hey there, good work! Unfortunately on L5.1 when you the phpBB extension tries to POST the laravel website, it return CSRF token mismatch. The reason is the middlewere -\App\Http\Middleware\VerifyCsrfToken::class which is applied on every request. So that's why it won't work, unless the middleware is commented out of the app/Http/Kernel.php or if you modify the VerifyCsrfToken.php file.
Can you please confirm that this is the reason and may be add this to the readme?

You are correct. Problem in VerifyCsrfToken.php file. I added to readme this.
Thanks for issue.


[TODO]Generate function correctly

As title, and this feature should be released at v0.0.1.
For example:
fn add(lv: num, rv: num) -> num {
  return lv + rv
}

Should be compile to
define float @add(float, float) {
  ret %0 + %1 // not totally correct syntax, focus on concept
}

Now can generate correct function, just need to fill the hole.


Set up build and deploy for documentation; clean up documentation

The inline documentation via Doxygen needs clean up and the writing and build of documentation via Sphinx and Readthedocs needs done.
Fix the RTD "docs" badge in the README.md to point to the correct project slug.

Merged and documentation link should show up correctly in the README.md now.


ci: use manual changelog for BW Packager

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#48  👈
#47
working-trunk

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @juemrami and the rest of your teammates on  Graphite

Merge activity

May 20, 8:08 PM EDT: @juemrami started a stack merge that includes this pull request via Graphite.


Import in BaseModel.__init__ method instead of at the top of the file

What happened?
The base imports a module:
from importlib import import_module

within the __iinit__ method of the BaseModel class:
https://github.com/nipreps/eddymotion/blob/main/src/eddymotion/model/base.py#L135
If that is done for performance reasons, would lazy loading help so that the import is put at the top of the file (if that is relevant)?
What command did you use?
None.

What version of the software are you running?
main
How are you running this software?
Docker
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
Not applicable.

Additional information / screenshots
No response

This is intended to allow using virtually any DIPY model with a minimal boilerplate. I agree it delays errors if DIPY is not installed or in an incompatible version. That could be tested at the top, certainly.


NIFI-4085 Add exception in log message when (S)FTP listing fails

Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Reviewing...

Ran contrib-check and all tests and everything passes. I ran a simple FTP test and everything worked fine; I did not have a way to simulate an error in listing the directory, but the code change was pretty trivial.
+1, merged.


Heron admin UI does not work in offline environment

React and a few other javascript libraries are being referenced from cloudera cdn which fails in an offline (no internet) environment.
Cloudera reference in HTML template

https://github.com/twitter/heron/pull/2513
Same pull request. Use one of these.

#2513 has passed all checks, please use this pull request. Thanks.
https://github.com/twitter/heron/pull/2513

fixed in #2513


Fix Github TOS link

I'll tell you if the CI still fails. The CI failure is at https://github.com/github/choosealicense.com/runs/6032461847?check_suite_focus=true#step:4:86.

@github Unable to fix, see https://github.com/github/choosealicense.com/runs/6064078415?check_suite_focus=true#step:4:85.


We need a run-time dependency on numpy

This goes in setup.py in the install_requires section.
@razvanvasile

I think we also need scipy.

I added both scipy and numpy to the install_requires component of setup.py.


feat(adaptor): Add BLENDER_EXECUTABLE environment variable

What was the problem/requirement? (What/Why)
The assumption that the blender executable exists on the PATH doesn't always make sense, often it is easier to specify the version of Blender to use by explicit path and using an environment variable is an easy way to do this for customers.
What was the solution? (How)
We have defined a new environment variable BLENDER_EXECUTABLE which follows the naming from
deadline-cloud-for-cinema-4d of <EXE_NAME>_EXECUTABLE
What is the impact of this change?
Additional functionality, non-breaking, adds the environment variable override but falls back to existing functionality
How was this change tested?
Was this change documented?
Yes added in the README.md
Is this a breaking change?
No
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Rebased off of mainline


Fixing mess from merging directly to live

Docs Build status updates of commit c3c892e:
:warning: Validation status: warnings

File
Status
Preview URL
Details

articles/user-guide/machines/resources-estimator.md
:warning:Warning
View
Details

articles/user-guide/libraries/numerics/numerics.md
:white_check_mark:Succeeded
View

articles/user-guide/machines/resources-estimator.md

Line 4, Column 9: [Warning-author-not-found] Invalid value for author: 'anpaz-msft' is not a valid GitHub ID.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


modify code to use pty virtual serial devices

Thanks for the pull request! I'm not familiar with the pty module, so I'm hesitant to accept this right now. I would also need to modify the other examples and update the accompanying blog post to keep everything in sync. If you wanted to make a new example using the pty module and explain what's going on, I would consider accepting that.
Unrelated to the PR, I've also discovered that you can do async serial much easier using the trio library. It provides an async file wrapper (read this), which you can use to wrap the serial.Serial class as it exposes the same API as a file object. Using that method the only difference between the sync and async code is a few awaits here and there.


Update README

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[ ] I have read the contributing guidelines.

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[x] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
Changed up some wording, removed unnecessary hyphens, etc.

Please rebase your changes on docs/development

Gonna open a new PR to target docs/development from GreemDev/JDA:docs/development

Afaik, you can update the target branch later on, can't you? At least I saw Minn changing it sometimes.

Afaik, you can update the target branch later on, can't you? At least I saw Minn changing it sometimes.

He changed the target; not the source branch. I'm pretty sure that was the issue

Oh yeah, just saw it. My bad.


Update Abcissa dev-dependency

Follows @tarcieri's suggestion here: https://github.com/ZcashFoundation/zebra/pull/105#discussion_r299705959

@hdevalence you might try running cargo check and see if it impacts Cargo.lock, as it appears to be checked in

Hmm, running cargo check gives
error: failed to parse manifest at `/home/hdevalence/code/zebra/zebrad/Cargo.toml`

Caused by:
  invalid type: boolean `false`, expected a version string like "0.9.8" or a detailed dependency like { version = "0.9.8" } for key `dev-dependencies.default-features`

SGTM! We may want to update to a git version for a little while (to pick up dev changes) but this is good in the mean time.

@hdevalence I'll try to get 0.2 out by EOW, but would recommend switching to one of the git revisions for now if you'd like the bugfixes immediately


[BUG] .NET error

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[ ] My answer given in the checkbox above is a lie.

What problem did you encounter?
I try to open Bloxtrap but it says I require a newer version of .NET framework, so I downloaded it. But I installed Bloxtrap after me downloading the framework and it still says I need version 6.0 even though I just installed it. No matter what i download it keeps saying that I need to download a newer framework. Thanks for your time.

Open a Command Prompt window, run this command:
dotnet --info
Show a screenshot of what appears

Issue has stale for over a month, and will be reopened if further information is given.


fix(codegen): Fix bug triggered by operation input/output naming

Motivation and Context
During codegeneration, synthetic copies of input & output shapes are created. However, due to a Smithy shape id conflict, these
shapes were overwriting existing shapes in the model instead of creating new shapes.

Description
This lead to a number of bugs exposed by the new s3control model. A minimal example of this model is included as a test in rest-xml-extras.
This commit augments the namespace of synthetic shapes to exist in the synthetic Smithy namespace, removing the conflict. After this bug was fixed, a subsequent bug in the serializer function naming was exposed where two shapes with the same name but in different modules generated a conflicting serializer.

Testing
Tests:

[x] new s3control model compiles
[x] test case added to rest-xml-extras

Checklist

[ ] I have updated CHANGELOG.md
[ ] I have updated aws/SDK_CHANGELOG.md if applicable

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I think this fixes #662. I'm curious how you avoided the instantiator failures. I wonder if it works since it's a nested namespace rather than a completely disjoint namespace?
Should also clean this up: https://github.com/awslabs/smithy-rs/blob/3b8f69c18db227f99ea8b458324f4e2ed3daddf4/codegen/src/main/kotlin/software/amazon/smithy/rust/codegen/util/Smithy.kt#L71-L76

I think that #662 didn't fix it because in the codegen unit tests, we actually load the entire service closure all at once from multiple services and the lack of nested namespacing caused collisions.


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 14:09:29 on 2023.05.24
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 0, errors out: 0
Original problem ID: 22319
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=22319

Problem has been resolved in 3m 0s at 14:12:29 on 2023.05.24
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 22319


Add GitHub discovery tags

Do this after 1.0.0 is released to RubyGems

@JacobEvelyn closing because you just did this -- please feel free to reopen if there's more to do 😄


Firefox: Checkbox in Button

Bug, feature request, or proposal:
Bug
What is the expected behavior?
Toggle checkbox within a button
What is the current behavior?
Doesn't toggle checkbox withing a button
What are the steps to reproduce?
https://plnkr.co/edit/sno7838427zQaEwOM1x5?p=preview
use a mat-checkbox within a button
What is the use-case or motivation for changing an existing behavior?
To use this structure:
<mat-menu>
  <button mat-menu-item>
     <mat-checkbox>Check me!</mat-checkbox>
   </button>
</mat-menu>

Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Firefox 56, Material 2.0.0-beta.12
Is there anything else we should know?
Nope

ok it seems this is the way how firefox handles click events.
https://stackoverflow.com/questions/31092439/click-on-checkbox-inside-a-button-element-firefox-issue


Crash when destroying but the VPC doesn't exist

I have issues
I'm submitting a...

[ ] bug report
[ ] feature request
[ ] support request
[ ] kudos, thank you, warm fuzzy

What is the current behavior?
Terraform destroy crashes if the VPC doesn't exist any more.
If this is a bug, how to reproduce? Please include a code sample if relevant.

Create an EKS cluster.
Add some state that will fail deleting, e.g. a add a TF-managed hosted zone and insert some records to it manually
terraform destroy
Destroy will successfully delete the cluster and fail to delete the zone.
terraform destroy again.
Destroy will fail with the zone error and:

Error: Error in function call

  on .terraform/modules/eks/terraform-aws-modules-terraform-aws-eks-9c3d222/local.tf line 6, in locals:
   6:   cluster_security_group_id = coalesce(
   7:
   8:
   9:
    |----------------
    | aws_security_group.cluster is empty tuple
    | var.cluster_security_group_id is ""

Call to function "coalesce" failed: no non-null, non-empty-string arguments.

(I think that's the way it should be possible to reproduce it)
What's the expected behavior?
It would be nice to accept resources not existing when destroying.
Environment details

Affected module version: 5.0.0
OS: Ubuntu 19.04
Terraform version: v0.12.5

Duplicate of https://github.com/terraform-aws-modules/terraform-aws-eks/issues/402


"URL does not return JSON content"

actually i input the IP (http://<IP_ADDRESS>:7912/), and leave API-Key empty and the error i get is "URL does not return JSON content"
Originally posted by @micro-fx in https://github.com/Disane87/spoolman-homeassistant/discussions/15

When you get the message URL does not return JSON content it seems your spoolman really doesn't return the exppected data. You can try to grab your URL and append api/v1/spool. Like this (only an example: https://spoolman.disane.dev/api/v1/spool)
Now you should get a json with all your spools. In the code I don't do anything else:
https://github.com/Disane87/spoolman-homeassistant/blob/0155d5937b844bb7c4022f5e95a55584ff54a3e2/custom_components/spoolman/coordinator.py#L27
https://github.com/Disane87/spoolman-homeassistant/blob/0155d5937b844bb7c4022f5e95a55584ff54a3e2/custom_components/spoolman/const.py#L9
Hope this helps :)

so i added such as http://<IP_ADDRESS>:7912/api/v1/spool
and i get some json format, and includes spool info.
so what is the path i should use when adding the HA integration?
if i use the appended url when adding the integration, i get "Failed to connect to the URL. Status code: 404"

You only need http://<IP_ADDRESS>:7912 without trailing slash. That should work.

@micro-fx I've implemented a check for missing trailing slashes. Unfortunatly there was a bug. You should install the latest version and it should work.

I can confirm it now works on the new version

Great, thank you for the feedback @micro-fx! Have fun with it! If you have further questions or concerns just hit me up 🆙


Added CSS Peek: Allows peek and goto definition for CSS items

Note: base is current MASTER, not the Rust PR

I will publish the new extension pack version with the latest pull requests #1 and #2 as soon as I have Capgemini VPN access next week, as VS Code Extension Marketplace requires Azure DevOps services and I used my Capgemini account.


ENH: Speed up morph map creation

Speed up morph map generation ~4x using numba. This code:
import time
import mne
subjects_dir = mne.datasets.sample.data_path() + '/subjects'
t0 = time.time()
mne.surface._make_morph_map('sample', 'fsaverage', xhemi=False,
                            subjects_dir=subjects_dir)
print(time.time() - t0)

On master takes ~18 sec
After refactoring to be a tiny bit faster takes 17 sec (what people will get without numba installed)
On first run with numba takes ~6 sec (includes compilation time)
On subsequent runs takes ~2.5 sec (no longer needs to compile)

is the new code slower when numba is not available?

No it's 17s instead of 18s

On my macOS laptop, the same code snippet:

master: 27.9 sec
This PR without numba: 27.1 sec
This PR with numba, first run: 11.2 sec
This PR with numba, subsequent runs (because the compilation is cached): 5.7 sec

... and 2.5 sec on my Linux machine now cut down to 1.76 sec by using cKDTree instead of BallTree in the nearest step.
I think I'll stop optimizing now :)

ok I just tested. It works indeed much faster. 2 things:

numba should be added to mne.sys_info() (note also that vtk + pyvista should be added too cc @GuillaumeFavelier )
check that numba version is actually >= 0.4 on import?
do we have 2 codes (with our without numba) tested in our CIs?

thx @larsoner !

do we have 2 codes (with our without numba) tested in our CIs?

Yes the minimal dependencies version should test it

Okay, next up I'll probably add mne make_morph_maps


mn create-acme-account help and command output misleading about existing keys

The documentation for mn create-acme-account says "This command will either create a new account keypair for you or you can pass the account keypair that you have generated using the mn create-key or via openssl or other means in as a parameter."  (And the command does seem to behave according to the documentation.)
However the command help implies a key is always created and the output when using an existing key seems to imply a new key was created.
Steps to Reproduce

mn create-acme-account --help

or

openssl genrsa -out existing-key.pem 4096
mn create-acme-account<EMAIL_ADDRESS>-k=~/keydir -n=existing-key.pem --lets-encrypt-staging

Expected Behaviour

Help output should indicate an existing key can be used
Actual command output should indicate that the existing key was used.

Actual Behaviour
Usage: mn create-acme-account [-fhvVx] -e=<email> [-k=<keyDir>] -n=<keyName> [-s=<keySize>]
                              (-u=<serverUrl> | --lets-encrypt-prod | --lets-encrypt-staging)
Creates a new account on the given ACME server
  -e, --email=<email>        Email address to create account with.
  -f, --force                Whether to overwrite existing files
  -h, --help                 Show this help message and exit.
  -k, --key-dir=<keyDir>     Custom location on disk to put the key to be used with this
                               account.
                               Default: src/main/resources
  -n, --key-name=<keyName>   Name of the key to be created
  -s, --key-size=<keySize>   Size of the key to be generated
                               Default: 4096
  -v, --verbose              Create verbose output.
  -V, --version              Print version information and exit.
  -x, --stacktrace           Show full stack trace when exceptions occur.
ACME server URL
      --lets-encrypt-prod    Use the Let's Encrypt prod URL.
      --lets-encrypt-staging Use the Let's Encrypt staging URL
  -u, --url=<serverUrl>      URL of ACME server to use

and for the actual command
Key creation complete. It can be found here ~/keydir/existing-key.pem.
Account creation complete. Make sure to store your account pem somewhere safe as it is your only way to access your account.

Environment Information

Operating System: macOS Big Sur
Micronaut Version: 2.5.6
JDK Version: 16

Weird, I just ran mn-create-account another time (this time with --lets-encrypt-prod option) and got the following output:
Key already exists and can be found here : ~/Omni/omniproxy-deploy/env-testnet/account-testnet-btcproxy.consensusj.org.pem. If you want to overwrite it use the -f/--force flag.
Account creation complete. Make sure to store your account pem somewhere safe as it is your only way to access your account.

OK, the create key output was correct -- the problem was I was using ~ in the directory path and it wasn't being substituted.
However, the --help output is still misleading.
I'm going to change the Issue title to reflect the problem is only with --help output and add a note in the initial post.

Since the actual code is in the micronaut-starter project, I opened an issue over there:
https://github.com/micronaut-projects/micronaut-starter/issues/849


Fix UI render failure

Somehow I get the following failure now when starting Trino Gateway and trying to access the login screen.
io.dropwizard.jersey.errors.LoggingExceptionMapper: Error handling a request: 0c1704d55dbb0f4e
! io.dropwizard.views.common.ViewRenderException: Unable to find a renderer for /template/query-history-view.ftl
! at io.dropwizard.views.common.ViewMessageBodyWriter.writeTo(ViewMessageBodyWriter.java:77)
! ... 66 common frames omitted

This might have to do with the recent removal of the dependency
        <dependency>
            <groupId>io.dropwizard</groupId>
            <artifactId>dropwizard-views-freemarker</artifactId>
        </dependency>

but my initial testing of adding it back in did not fix the issue yet. I need to do more testing I guess.

Could we related to https://github.com/trinodb/trino-gateway/pull/190

Or https://github.com/trinodb/trino-gateway/pull/185


analytics sessions time-range

mobile-center analytics sessions returns values for an unspecified time-range.
It doesn't correspond to the last7, last30, last60 or last90 options in the portal.
    0-10s    4
    10-30s   0
    30s-1min 0
    1-30min  1
    30min-1h 0
    >1h      0

Session Statistics
    Total Sessions               5     0%
    Average Sessions Per Day     5     100%
    Average Session Length (sec) 28.38 100%```

![image](https://cloud.githubusercontent.com/assets/6565986/26654100/5db54f6e-460a-11e7-8b1d-e2e3dc129e03.png)

@matt-gibbs It shows the values from 00.00 to "Now". If we maintain it this way, we can add this information in the help functionality indicating what's the time range by default.
The other option would be re implementing to last 7/30/60 days, but I think the customization in the CLI brings more value.


Updating changelog

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[ ] New extensions are written in Swift 3.
[ ] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[ ] I have added tests for new extensions, and they passed.
[ ] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[ ] All extensions are declared as public.

@LucianoPAlmeida lets keep this PR open and add more entries to it as we get them


Library options doesn't seem to be respected

Hi,
I noticed that library options example on this page: https://ankane.github.io/chartkick.js/examples/
is rendered without any respect to the library options hash passed. Is it a known issue?
Thanks!

Hey @ipoval, the library options on that page are specific to Google Charts, but the page currently uses Chart.js. The library option works just fine.


Check for the green ring and fix

Reference: similar to issue #340
Action plan:

take a fresh browser (both chrome and firefox) and install the extension
pick 100 random sites from the sample (in the crawler repo) and go to random.org
manually check if they have the green ring and then make a list of it.
check the google developer tools (fn F12 -> console) for error content message like content security policy prevented (like attached)

site list: https://github.com/privacy-tech-lab/gpc-web-crawler/blob/main/selenium-optmeowt-crawler/full-crawl-set.csv

This checklist is for both Chrome and Firefox:
Notes:

Unless specifically stated, the extension had the same effect on both browsers for all websites.

Some websites could not be reached. I manually searched up the website (eg. incontrol clothing) but there are some websites that could not be reached and I could not find the new updated site name

Only one site that has no green ring (yourdictionary) but this site has a green ring on Firefox

[x] https://kimgarst.com/

[x] https://tarvalon.net

[x] https://uppermichiganssource.com

[x] https://musterlive.aggienetwork.com

[x] https://ama.org

[x] https://nesn.com

[x] https://godtube.com

[x] https://vizio.com

[x] https://cafilm.org

[x] https://incontrolclothing.com/ (changed site name to https://www.incontrolclothing.com/)

[x] https://zenefits.com/ (redirected to https://www.trinet.com/)

[x] https://eloquii.com

[x] https://canon.com/ (redirected to a page that state "site took too long to respond" — I manually went to the search bar and look up ‘canon’ which sent me to https://www.usa.canon.com/ ) (complete green)

[x] https://boston.com

[x] https://ranker.com/ (complete green)

[x] https://socialworkdegreeguide.com

[x] https://caring.com

[x] https://sagaftra.org

[x] https://bodyartforms.com

[x] https://sacred-texts.com

[x] https://submittable.com

[x] https://marketplace.ivanti.com

[x] https://assurancewireless.com

[x] https://gunnar.com

[x] https://neurostar.com

[x] https://highdefdigest.com

[x] https://brand24.com

[x] https://belvederebanquets.com

[x] https://forexfactory.com

[x] https://bestbooks.multcolib.org/ (redirected to https://bestbooks.multcolib.org/#view=covers&year=2023)

[x] https://coreseries-tape-selection-app.averydennison.com

[x] https://msccruisesusa.com

[x] https://elsevierhealth.com/ (site changed name to https://www.us.elsevierhealth.com/)

[x] https://texomashomepage.com

[ ] https://lordandtaylor.com/ (site can’t be reached)

[x] https://charactercounts.org

[x] https://shiekh.com

[x] https://dunkindonuts.com

[x] https://timesunion.com

[x] https://svahausa.com

[x] https://careers.stelizabeth.com/ (takes a while before the green ring is on in chrome — normal on Firefox)

[x] https://espn.com

[x] https://velcro.com

[x] https://sprite.com

[x] https://mongabay.com

[x] https://bamboohr.com

[x] https://swnewsmedia.com

[x] https://rs-stripe.mycbs4.com

[x] https://strathmore.org

[x] https://makershed.com

[x] https://redrocknews.com

[x] https://mediavine.com

[x] https://wonderopolis.org

[x] https://worldlabel.com

[x] https://rpg.net

[x] https://l-r-g.com

[x] https://belk.com

[x] https://genealogybank.com

[x] https://natrol.com

[ ] https://apoc.varian.com/ (resource not found)

[x] https://greencarreports.com

[x] https://childrens.com/ (takes a while to load the green ring on Chrome — normal on Firefox)

[x] https://learning.rcan.org

[x] https://careerarc.com

[x] https://bryant.sidearmstats.com/

[x] https://verilymag.com

[x] https://vendingworld.com

[x] https://airliftcompany.com

[x] https://frostriver.com

[x] https://starwars.com

[x] https://scrapbook.com

[x] https://store.gardengatemagazine.com

[ ] https://lyncdiscover.irmi.com/ (site can’t be reached)

[x] https://sacnas.org

[x] https://guide.farmigo.com/

[x] https://hopecu.org

[x] https://redandblack.com

[x] https://greenvillelibrary.org

[x] https://renewedvision.com

[x] https://overdrive.daytonmetrolibrary.org

[x] https://airforce.com

[x] https://kjct8.com

[x] https://ipm.auhsd.us

[ ] https://yourdictionary.com/ (no green ring on Chrome but there is green ring on Firefox)

[x] https://everestbands.com

[x] https://cameraprive.com

[x] https://flyontario.com

[x] https://news-press.com

[x] https://iphonelife.com

[x] https://sugarandcloth.com

[x] https://etundra.com

[x] https://wboy.com

[x] https://quote.americancollectors.com

[x] https://afw.com

[x] https://safari.katyisd.org

[x] https://overstock.com

[x] https://981thehawk.com

[x] https://shopusahockey.com

[x] https://rexburgstandardjournal.com

[x] https://insideclimatenews.org

Attached is the difference of the developer tools' console for Chrom and Firefox for specifically yourdictionary.com

Thanks, @franciscawijaya.

pick 100 random sites from the sample (in the crawler repo) and go to random.org

Well, random.org was for selecting the 100 random sites. :)

I manually searched up the website (eg. incontrol clothing)

What do you mean by "incontrol clothing"?
Which version of OptMeowt did you use? The problem does not seem to be with the current version in the stores but with @OliverWang13's and @Mattm27's mv3 version (branch 350-1 or branch 350-2; @OliverWang13 and @Mattm27, which branch should be used actually?).

What do you mean by "incontrol clothing"?

Some sites changed their name so I just manually searched the updated name

https://incontrolclothing.com/ (changed site name to https://www.incontrolclothing.com/)

Which version of OptMeowt did you use? The problem does not seem to be with the current version in the stores but with @OliverWang13's and @Mattm27's mv3 version (branch 350-1 or branch 350-2; @OliverWang13 and @Mattm27, which branch should be used actually?).

I loaded unpacked extension for version 4.3.1. I think I used branch 350-1.

URL
Chrome
Firefox

url
✅
blah

(Just a sample to modify)
@franciscawijaya will check again using branch 350-5.

Check if that version would work for a good percentage of sites. If not, contact Oliver & Matt.

@franciscawijaya, to get the most recent versions, you can do:
For Firefox: git checkout issue-350-5.
For Chrome: git checkout 13c0f830011ef9ed72dddf16b16b3e533b7abae7 (the currently latest commit reference).
Then, you can check with git status that you are on those branches and, if so, follow the normal setup procedure.

 Github on Firefox has green ring; only github on Chrome does not.

Great! Feel free to close any issue once it is done, @franciscawijaya.

Reopening. Seems v5.0.0 does not have the green ring and is also not sending the GPC signal properly.

According to the browser developer tools, there currently is an xml parsing error. Potentially concerning the xml syntax in the new updated version?
It is also worth to note that the "cookie-test" being rejected for invalid domain; though, this could maybe come from the underlying xml parsing error.

the xml parsing error seems to be from from Permutive so that actually does not have any correlation with the extension.

As @Mattm27 found, the issue after re-opening was related to the dist manifests being outdated; only the dev manifests were updated after the switch to mv3. v5.0.1 fixes this bug.


Adjust notebooks to run in unattended mode

To enable automatic testing (e.g. updating the conda environment and running the notebooks automatically against the new environment for a basic test of compatibility), the notebooks should be adjusted so that they can be executed from top to bottom without manually specifying input every time.

Thanks @smithara in this new branch I have address this suggestion. You will find a MagGeo_main_py that can be run in unattended mode if users provide the VirES token. I will also later add the parallel mode as part of the parameters, at the moment is sequential.  https://github.com/MagGeo/MagGeo-Annotation-Program/tree/minconda_version

I'd say this issues is being address in the new update. The only part I could not make unattended in the notebooks was the token entry.

A hint in case you want to set up an automated test in the future on GitHub Actions:
You can set the token beforehand with the CLI instruction:
viresclient set_token https://vires.services/ows ${{ secrets.VIRES_TOKEN }}

(having set VIRES_TOKEN as a secret on the repository)
If you then have the notebook without any token entry step, then it will be able to use that preconfigured one in the automated test, but a user will be prompted to enter one if they haven't set one up before. It' probably not worth the hassle to set up the automation now, but something to consider in the future!

Thanks @smithara for the suggestion!! I think I will try that in the next version where I plan to make other changes to the MagGeoFunctions, and the main script to run the parallel mode using the shell and the parameters file.


create-react-app-starter does not work

Describe the bug
I cloned your repo, went into create-react-app-starter directory, did "npm install" and "npm run start", there are 4 errors, app doesn't work
To Reproduce
Steps to reproduce the behavior:

Go to create-react-app-starter directory,
npm install
npm run start
See errors

Expected behavior
No errrors
Screenshots

Desktop (please complete the following information):

OS: Mac OS
Chrome

This is not a bug. It's because you didn't build the rest of the library packages. Please follow the instructions on building from source: https://github.com/solana-labs/wallet-adapter#build-from-source
In the last step, instead of cd packages/starter/react-ui-starter, do cd packages/starter/create-react-app-starter, then yarn start.


expect: Improve report when matcher fails, part 16

Summary
For toEqual and toStrictEqual matcher:

Display matcher name in regular black instead of dim color
Display promise rejects or resolves in matcher hint
Display // deep equality in dim color at end of matcher hint
Omit redundant or distracting information, as described below

When negative assertion fails:

Display not between expected label and value
Display received value only if it has different serialization

When positive assertion fails:

Display either diff (without Difference label) or Expected/Received values, not both
Display Received: has same serialization instead of redundant value

Moved code added in #8281 from toBe matcher to print.ts file and edit message string
@jeysal after we have improved reports for all matchers, let’s discuss whether to move the additional logic in shouldPrintDiff to jest-matcher-utils package
Two positive reports changed from diff to Expected/Received:

strings which are not both multi-line
received array and expected asymmetric matcher do not have same getType value

Residue: stringify (that is, pretty-format with min option)

omits constructor name for array or object, so in edge case of received value constructed from different class than expected value, it should have a different serialization
which is especially confusing for Number(0) and String('abc')

Test plan
Updated snapshots:

file
matcher

4
expect extend
toEqual
positive
hint

2
e2e failures
toEqual
positive
hint including rejects and resolves

1
jest-matcher-utils index
toEqual
positive
hint

5
expect matchers
toBe
positive
display Received: has same serialization

42
expect matchers
toEqual
positive
remove Difference and display Received: has same serialization

48
expect matchers
toEqual
negative
omit redundant Received value

1
expect matchers
toStrictEqual
positive
remove Difference

1
expect matchers
toStrictEqual
negative
omit redundant Received value

See also pictures in following comment with baseline at left and improved at right

For negative toEqual when received value has different serialization:

For negative toEqual when received value has same serialization:

For positive toEqual when received value has different serialization:

For positive toEqual when received value has same serialization:

For positive toBe when received value has same serialization:

For positive toEqual which has diff:

For positive toStrictEqual which has diff:


Adding support for O1

This adds a new mode which should support the o1 model family using the openai client sdk. We'll need to upgrade the supported version to support the new max_completion_tokens parameter though. This allows us to do something like what we see below.
from instructor import from_openai, Mode
from openai import OpenAI
from pydantic import BaseModel

client = from_openai(OpenAI(), mode=Mode.JSON_O1)

class UserDetails(BaseModel):
    name: str
    age: int

resp = client.chat.completions.create(
    model="o1-mini",
    messages=[{"role": "user", "content": "Jason is 10"}],
    response_model=UserDetails,
    max_completion_tokens=300, # This has to have a minimum of 200+ because of reasoning tokens
)

print(resp)

:rocket:
This description was created by Ellipsis for commit f35b04c32fe447855b1af5885bebac732b9deed6

feat: add support for JSON_O1 mode with OpenAI SDK
Summary:
Adds support for JSON_O1 mode using the OpenAI client SDK, requiring an updated openai dependency.
Key points:

Behavior:

Adds support for JSON_O1 mode in from_openai() in client.py.
Updates handle_response_model() in process_response.py to handle JSON_O1 mode, ensuring no system messages are included.
Updates from_response() in function_calls.py to parse JSON_O1 mode using parse_json().

Dependencies:

Updates openai dependency to ^1.45.0 in pyproject.toml to support max_completion_tokens.

Enums:

Adds JSON_O1 to Mode enum in mode.py.

Generated with :heart: by ellipsis.dev

Important here to note that the current o1-preview doesn't support things like tool-calling, streaming and so this is really just supporting a simple json mode support


Limit search space to (2t+1) max distance corridor

This is the simple version that uses O(m) space instead of O(d), but at least it seems to work.
Algorithmic idea: if we have an upper bound d on the Levenshtein distance we are interested in (say, 3), we don't need to evaluate the entire O(nm) matrix. Instead, it suffices to compute a corridor of width 2d+1 around the diagonal of the matrix in time O(nd). The proof is left as an exercise to the reader ;)
Furthermore, we can abort the search if the minimum Levenshtein distance between a prefix of s and a prefix of t exceeds d, we can abort the computation and report that the threshold was exceeded. This is obviously correct as the Levenshtein distance increases monotonically as we add arbitrary suffixes to the strings.
Possible improvements:

[x] if |n-m|>d, we don't need to look at the strings at all, as their Levenshtein distance cannot be ≤ d
[x] if n>m, flip the strings → O(min(n,m)*t). Unclear whether this provides further benefits after the above suggestion has been implemented.
[x] document the code
[x] add MathJax support to GitHub flavoured markdown lose hope

I'll just merge this myself then :P

I actually would like to have a benchmark added for this, but probably I'll just do it myself. That's your flatmate bonus.

:+1: sure, I already have a benchmark lying around on another computer. It's pretty much the same as the pairwise benchmark though


How to add space between bars ( indivuals group)  when  user using group bar chart

Example here:
https://github.com/gitbrent/PptxGenJS/issues/718#issuecomment-633179277


(yuyuyuyuka) investigate potency scaling on auto CDH skills

https://discord.com/channels/277897135515762698/1274591512902238270/1274626404306391050
https://www.akhmorning.com/allagan-studies/stats/dh/#formulae

resolved by #12, hopefully...?


feat: 🔥 add date functions

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/ngneat/falso/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

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

What is the current behavior?
There are no date funtions.

Issue Number: #2
What is the new behavior?
add weekday, month, between, soon, recent, past and future functions
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information
✅ Closes: #2

Great work! Let's wait for @shaharkazaz PR, make the changes, and merge this.

TODO: migrate weekday and month data to a new JSON file (related to #26)

data moved to json files for month and weekday functions


scratch based container does not allow healthchecks

Issue: When container is based on scratch, nothing else can be run inside it - obviously. This sadly also means it includes basic commands like pidof or wget / curl - meaning creating a container healthcheck is impossible. Docker healthchecks are really handy to have - especially combined with something like docker-autoheal that will automatically restart your unhealthy containers.
Off the top of my head, these are solutions I would propose:

Move back to alpine base image. Cons: image size increased by ~5MB; pros: simplest to implement (one liner).
Add wget and its dependencies manually to the image. Cons: finding libraries needed to run and keeping track of the versions in case of build base image updates; pros: keeping the image small.
Keep two images (:latest and :alpine-latest). Cons: twice the Dockerfiles, bloated Actions. Pros: relatively simple to implement, keeps everyone happy.

If you would be inclined to address this issue, I would be happy to prepare PRs if you selected option 1 or 3. :)

I'm leaning towards 1, 5MB is negligible nowadays.


[Idea] Get GPU info

First question, where to get the information from?
I found in the pciutils repository the following:

In runs on the following systems:
Linux		(via /sys/bus/pci, /proc/bus/pci or i386 ports)
FreeBSD		(via /dev/pci)
NetBSD		(via libpci)
OpenBSD		(via /dev/pci)
GNU/kFreeBSD	(via /dev/pci)
Solaris/i386	(direct port access)
Aix		(via /dev/pci and odmget)
GNU Hurd	(direct port access)
Windows		(direct port access, see README.Windows for caveats)
CYGWIN		(direct port access)
BeOS		(via syscalls)
Haiku		(via /dev/misc/poke)
Darwin		(via IOKit)
DOS/DJGPP	(via i386 ports)
SylixOS		(via /proc/pci)

There are many possible places to search, depending on the kernel (or operating system). But in some cases it's a matter of reading a plain text file. In order not to make the issue too long, I recommend the first part of this article. As you can see in the article, it is possible to read the vendor identifier, device identifier and class identifier.
Step two: mapping identifiers to GPUs
I quote again the previous repository, the following:

The database of PCI IDs (the pci.ids file) gets out of date much faster
than I release new versions of this package, so it is maintained separately.
It lives at https://pci-ids.ucw.cz/, where you can browse the database,
download the most recent pci.ids file (e.g., by running the update-ids utility)
and also submit new entries.

If we go through that website, we can corroborate that the GPU information (in addition to other devices that are not relevant) can be obtained from these ID's.
At first I thought that the pci.ids file should be downloaded regularly to keep it updated but, at least in Arch Linux, I found it in /usr/share/hwdata/pci.ids.
I'm not sure this is the best way to solve it, but I think it can work, without relying on system commands or external crates.

Hey @PandaFoss!
If implemented correctly, I believe this is the best shot we have at correctly identifying GPU information no matter the system.
If it turns out to be a lengthy implementation, then we might consider turning this into a separate crate.
My suggestion is we start with Linux, it's the first one to get new features and it's where I do all my work.

I'm a Linux user myself, so I think it's best to start there. I'm not an experienced Rust user but I'd love to contribute as much as I can.

I'm going to start working on this, most likely in a different repository, once I have a good bit of code setup, I'll share it with you and we can commence the real work.

Hey! It's been a while, I've resumed my work on this feature :)
(pci_fetch)[https://github.com/grtcdr/pci_fetch] is the library that handles fetching PCI devices and soon enough it'll also integrate with pci.ids to get the strings associated with the connected PCI devices. So as you can already tell, the library is still a work in progress.
My apologies for the month long delay it took, uni's been hectic :/

Slowly but surely imitating lspci.

@PandaFoss Feel free to take a look around the code if you'd like to contribute, I cleaned it up and made it a lot more library-use friendly.

@PandaFoss Feel free to take a look around the code if you'd like to contribute, I cleaned it up and made it a lot more library-use friendly.
And here's a nice little demo smiley
output.mp4
What do you think @uttarayan21?

Incredible work. I've been pretty busy lately but as soon as I can I'll take a closer look at the code (which from what I can see is great). Thank you!

Incredible work.

Thanks! I wish I had the time to do it sooner, this was a great experience :smile:

I've been pretty busy lately but as soon as I can I'll take a closer look at the code

Great, I'd love to hear your feedback when you're free.

@PandaFoss Feel free to take a look around the code if you'd like to contribute, I cleaned it up and made it a lot more library-use friendly.
And here's a nice little demo smiley
output.mp4
What do you think @uttarayan21?

Looks really cool :+1:
But since you are using GPL-2.0 for pci_fetch, you might want to change that to MIT or Apache or any permissive license since GPL-2.0 requires you to license macchina under GPL-2.0 as well.
see here

But since you are using GPL-2.0 for pci_fetch, you might want to change that to MIT or Apache or any permissive license since GPL-2.0 requires you to license macchina under GPL-2.0 as well.
see here

Oh, good catch, will do!

libmacchina v0.5.0 now integrates with aparato (previously named pci_fetch), so I'll move this issue back over to macchina so we can print the new readouts.


Inf Libs DataTables

I have tried to implement the lib DataTable.min.js
But I've only gotten it to throw an error in the console
localhost/proyect\libs\DataTables/datatables.min.css
localhost/proyect\libs\DataTables/datatables.min.js
Uncaught TypeError: Cannot read properties of undefined (reading 'value')
at datatables.min.js:229:96
at datatables.min.js:64:299
at datatables.min.js:64:326
I don't know, if I'm implementing it wrong, or it should be done in another way

Hi @WillsMasthers,
Thanks for getting in touch with us!!
Can you please check our downloaded package and in the live demo once and let me know how it works with you?
If it's there please share live demo URL for the same.
Awaiting to hear from you :)
Stay Safe!!


feat: add charter page

Describe what changes this pull request brings
This pull request brings the Charter page.
Additional changes

Prevented the horizontal scroll by changing the absolute positioning of the submenu in the header navigation.

Steps to steps

Open the page
Confirm that everything looks and works as expected

LGTM. Let's wait for final review. It matches the correct version of the charter now.

@borkmann any objection to merging?


Selecting first lld and later mold, duplicates flags in rustlags

create new project
create/overwrite any profile with lld linker
save changes
create/overwrite any profile with mold linker
save changes
rustflags now enables two different linkers

https://github.com/Kobzol/cargo-wizard/assets/41945903/4a08717a-7c9b-4d81-bc49-c4c86e272855

Yeah, RUSTFLAGS are quite opaque, so I don't know if I want to go through the route of parsing them. You can input -Clink-args in several different ways, and even the configuration of the linker can be performed using different flags. So it might not be so easy to detect if a flag is duplicated or not.
That being said, we could probably at least detect our own way of specifying the flag.

https://github.com/Kobzol/cargo-wizard/pull/15 should make this better.


Disable hostname based check

Pull request describing use of flag to disable hostname based check.

Has unintended commits.


Option to keep last state instead of becoming unavailable

Is your feature request related to a problem? Please describe.
I have a similar problem to the ones reported in issue #39 and #61.

Sometimes my Deco will timeout and then everything will go unavailable and then recover their status on the next try. This results in dozens of devices becoming home or away in a short period of time, which is bad for some automation or guests detection.
Here's an example:

Describe the solution you'd like
Instead of devices going unavailable I'd like an option for the devices to keep their last know state when there's an error during status retrieval.
Describe alternatives you've considered
I've created a simple template for some device trackers, but it's not a good idea to create dozens of these.
Additional context
I believe this would make the integration work in a more stable manner, even if the fault is on the Deco devices.

Hi I am also facing similar issues as the devices on my deco m5 are not home/not connected to my deco m5 are showing as 'unavailable' rather than 'away' or 'not_home'.
This is messing up my automations and they fail to trigger due to this.
Can this be please looked into ?

Same issue, this also messes up the group property, because if one goes unknown/unavailable it will report the whole group as offline.

This would help resolve some issues for me too! I would love this as an option, especially to check which AP the client was connected to last (before being unavailable). I know this can be achieved with an SQL sensor, but having it as a feature would be great.

This would help resolve some issues for me too! I would love this as an option, especially to check which AP the client was connected to last (before being unavailable). I know this can be achieved with an SQL sensor, but having it as a feature would be great.

Hello, your issue really seems similar! After checking the logs, I noticed sometimes that my Deco M4 takes over 30 seconds to return the device list, sometimes even going as high as 50 seconds.
Try and change line 26 on the file api.py from 30 to 60 or higher.
TIMEOUT = 60
I made this change recently, I'm still testing, but it seems to have "fixed" my issues.

Thank you for taking the time to answer me. I'll give your suggestion a try. I also have Deco M4 units, and a timeout is a likely cause. If this resolves my issue, I will create a PR to increase the default timeout value. Thanks again!

Thank you for taking the time to answer me. I'll give your suggestion a try. I also have Deco M4 units, and a timeout is a likely cause. If this resolves my issue, I will create a PR to increase the default timeout value. Thanks again!

I believe a PR that allows this value to be changed from the UI will be better.
I'm just testing over the weekend before making the PR. Let me know if this fixes it for you as well!

Unfortunately, so far, the results are not good:

I've increased to 90, but so far the results are still disappointing.
Logger: custom_components.tplink_deco
Source: custom_components/tplink_deco/api.py:215
Integration: TP-Link Deco ([documentation](https://github.com/amosyuen/ha-tplink-deco), [issues](https://github.com/amosyuen/ha-tplink-deco/issues))
First occurred: 15:31:18 (120 occurrences)
Last logged: 16:04:32

    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}

Unfortunately, so far, the results are not good:

I've increased to 90, but so far the results are still disappointing.
Logger: custom_components.tplink_deco
Source: custom_components/tplink_deco/api.py:215
Integration: TP-Link Deco ([documentation](https://github.com/amosyuen/ha-tplink-deco), [issues](https://github.com/amosyuen/ha-tplink-deco/issues))
First occurred: 15:31:18 (120 occurrences)
Last logged: 16:04:32

    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}
    List Clients <snip> parse response error='result', data={'success': False, 'errorcode': 'timeout'}

In that case you need to enable debug options and check out what's happening.


Guidance for CSS tokens

Summary
Add guidance for using CSS tokens.
💬 Description
The UI kit offers developers CSS tokens which help to ensure application styles better match those used in the components. There should be guidance on the site to better explain the use of these tokens including listing the tokens as they would be used in code, potentially with examples. A component similar to the 'props table' used for components would be useful to show CSS tokens on relevant pages.
💰 Use value
Better documentation for the CSS tokens and their usage would improve developer experience when employing them.
Additional info

Will review with our theming piece of work


feat(apify): pass through Document metadata generic type

This PR gives users of the ApifyDatasetLoader class the ability to pass through their custom Document metadata generic type. Currently, the datasetMappingFunction of the ApifyDatasetLoader returns a type of Document[] despite the fact that the user is explicitly setting specific properties on the document, from the docs
const loader = new ApifyDatasetLoader("your-dataset-id", {
  datasetMappingFunction: (item) =>
    new Document({
      pageContent: (item.text || "") as string,
      metadata: { source: item.url },
    }),
  clientOptions: {
    token: "your-apify-token", // Or set as process.env.APIFY_API_TOKEN
  },
});

const docs = await loader.load();

One would expect that the type of docs to be Document<{ source: string }>[] but it ends up being Document[] because the types are not plumbed through. This is slightly annoying if you want to then pass the docs to another function that is expecting certain metadata to be present because you need to add an unnecessary type check. With the changes of this PR, you could provide your Metadata type up front and get the type safety

Seems good to me, thank you!

Seems good to me, thank you!

Any time! I realize I should also add a default so it's backwards compatible, e.g.
Metadata extends Record<string, any> = Record<string, any>
Will add a follow-up commit

Ah yes, please do.

Hi @jacoblee93 -- thanks for your patience. I took a look again and realized it doesn't need the default generic because it can infer the type.
Therefore it is safe to merge as is. Thanks again!

Thank you!


Perform database migrations from command line (without visual studio)

We should add a script to run migrations and update databases to the latest version from the command line.
DB deployment could be automated without needing to jump to visual studio to update to the latest migration.

I'll start to work on this on my personal fork...


How to edit the code of a column ?

Hello
How can I edit the code used for a column except going to the history but without knowing which line belongs to a column?
Thanks
Bernard

Please use the mailing list for support request regarding the usage of OpenRefine. Github issues are to report bugs and request new features.


Update Game.java

Abstract class ... but it was missing abstract funcs. And if we updated GameObject , we need to update Game too, right?

Can this be closed?

But is there any reason to do that instead of making it abstract? Every game that people will reasonably be making with the engine will need to initialise something or other; so why not have it abstract? If you are in the very small minority who don't need it, it won't kill you to have a empty definition at the bottom of your game class, and would be useful for the majority of other users.


How to train ?

How to train own data, can you share the training code, thank you

https://www.youtube.com/watch?v=Rgpfk6eYxJA
You can use the trainer example  with Tensorflow.


Pagination

A pagination feature can help make our bot's messages and embeds look very cool and tidy. Page for page baby!!!. Anyway, anyone want to implement this feature, I will assign u? I will be busy tomorrow so I won't do it.

Having a default pagination interface would make menus easier to make :+1:
I could work on this..
Though I haven't worked with pagination a lot, so might have a few issues

Excellent. Personally, I think a menu row of reactions for navigation would be best if we had different versions of it. The buttons would navigate the embed pagination or delete the embed. There are three main types of navigation that I think we should have — the following:
If the embed is 4+ pages:
⏪ ◀ ▶ ⏩ 🗑
If the embed is 2 or 3 pages:
◀ ▶ 🗑
If the embed is 1 page:
🗑
The last one is a simple 🗑 that we could also use to let the user remove a bot embed. I think all three should be different pagination types that can be used separately, with an "dynamic" paginator that could be used if we want the bot to dynamically scale the amount of emojis (this one would simply apply the appropriate paginator).
Any thoughts on this? Would you like to implement something like this? You could get some inspiration and possibly migrate (parts of) the code for the PyDis paginates, if you want to.

Excellent. Personally, I think a menu row of reactions for navigation would be best if we had different versions of it. The buttons would navigate the embed pagination or delete the embed. There are three main types of navigation that I think we should have — the following:
If the embed is 4+ pages:
⏪ ◀ ▶ ⏩ 🗑
If the embed is 2 or 3 pages:
◀ ▶ 🗑
If the embed is 1 page:
🗑
The last one is a simple 🗑 that we could also use to let the user remove a bot embed. I think all three should be different pagination types that can be used separately, with an "dynamic" paginator that could be used if we want the bot to dynamically scale the amount of emojis (this one would simply apply the appropriate paginator).
Any thoughts on this? Would you like to implement something like this? You could get some inspiration and possibly migrate (parts of) the code for the PyDis paginates, if you want to.

I would checkout PyDis' paginator, when I get time :+1:

I would checkout PyDis' paginator, when I get time :+1:

This can be closes, implemented in #114


🐛 Bug Report: Search bar in techdocs reader page doesn't use baseUrl

📜 Description
We have backstage deployed with a URL prefix/sub path, e.g. https://example.com/our-backstage
Setting backend.baseUrl and app.baseUrl is enough to make links work correctly in most places, but the results from the search bar at the top of the techdocs reader page don't include the prefix.
👍 Expected behavior
Links in the techdocs reader page search should go to https://example.com/our-backstage/docs/...
👎 Actual Behavior with Screenshots
They go to https://example.com/docs/...
👟 Reproduction steps
Enable techdocs and search.
Configure baseUrl to include a prefix.
📃 Provide the context for the Bug.
This may be specific to our setup - I had a look through our techdocs setup (collators, config etc.) but couldn't find how the location is constructed. Essentially, I am not sure whether this is something that should work "out of the box" or whether this is something that I need to configure, in which case documentation/guidance would be very useful!
🖥 Your Environment
Backstage 1.10.1
    "@backstage/plugin-search-backend": "^1.2.2",
    "@backstage/plugin-search-backend-module-pg": "^0.5.3",
    "@backstage/plugin-search-backend-node": "^1.1.2",
    "@backstage/plugin-techdocs-backend": "^1.1.1",

👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Can you share a little bit more about your set up? I tried to test this locally with the master branch demo app and a basic app.baseUrl: http://localhost:3000/backstage but not able to reproduce.
The relevant lines are https://github.com/sennyeya/backstage/blob/4e5891524ca57f0cb7a787e91aa7966a67f83782/plugins/techdocs/src/search/components/TechDocsSearch.tsx#L114 and https://github.com/sennyeya/backstage/blob/4e5891524ca57f0cb7a787e91aa7966a67f83782/plugins/techdocs-backend/src/search/DefaultTechDocsCollatorFactory.ts#L199, but those were working as expected in my tests. The navigate hook should handle any resolution of baseUrl as needed, and the values produced by the DefaultTechDocsCollatorFactory or any other collator can just use an absolute /docs/... instead of ${app.baseUrl}/docs/....

Thanks for taking a look!
I will share anything I can find that might be relevant...
We have this route in App.tsx:
    <Route path="/docs" element={<TechDocsIndexPage />} />
    <Route
      path="/docs/:namespace/:kind/:name/*"
      element={<TechDocsReaderPage />}
    >
      <TechDocsAddons>
        <ReportIssue />
      </TechDocsAddons>
    </Route>

We have a packages/backend/src/plugins/techdocs.ts file that matches the one in the docs (https://backstage.io/docs/features/techdocs/getting-started#adding-techdocs-backend-plugin). We also have the import and apiRouter.use calls in packages/backend/src/index.ts
In packages/backend/src/plugins/search.ts:
import { DefaultTechDocsCollatorFactory } from '@backstage/plugin-techdocs-backend';

  indexBuilder.addCollator({
    schedule,
    factory: DefaultTechDocsCollatorFactory.fromConfig(env.config, {
      discovery: env.discovery,
      logger: env.logger,
      tokenManager: env.tokenManager,
    }),
  });

Here are the relevant parts of our config:
app:
  title: <org-name> Backstage
  baseUrl: https://<our-admin-domain>.com/<org-name>-backstage/

organization:
  name: <org-name>

backend:
  baseUrl: https://<our-admin-domain>.com/<org-name>-backstage/
  listen:
    port: 7007
techdocs:
  builder: 'local' # Alternatives - 'external'
  generator:
    runIn: 'local' # Alternatives - 'local'
    mkdocs:
       legacyCopyReadmeMdToIndexMd: true
  publisher:
    type: 'local' # Alternatives - 'googleGcs' or 'awsS3'. Read documentation for using alternatives.
    local:
      # (Optional). Set this to specify where the generated documentation is stored.
      publishDirectory: '/backstage/docs'

That all looks good, what version of react-router are you using in your project? Could be something with the relative path resolution. The recommended version is 6.3, https://backstage.io/docs/tutorials/react-router-stable-migration/.

Two thoughts,

Can you verify that your links in App.tsx are absolute, ie start with /?
Are you using createApp in App.tsx? May not be a difference, but I know that was a recent shift that may effect this.

Thanks for the snippets, I was able to verify that there is an issue. Looks like it's happening here. We're using the navigate function with the full URL, ie /our-backstage/docs/...,  instead of the relative url with the app.baseUrl removed, ie /docs/.... The href on the link element is correct due to this function call, but it isn't being parsed correctly on the other end.
I'm taking a look at this now and should have a PR out soon.

any chance to merge PR #16682 ? It would really help us


Added derives for traits such as Debug

As per title, added derives for traits including Default, Clone, Copy, Debug, PartialEq, Eq.

Done!


carry response header through connect callback

Hi. how can I  get Content-Disposition and Content-type of an url before download ?
Now I use this code:
                URL url = new URL(item.URL);
                HttpURLConnection connection = (HttpURLConnection) url.openConnection();
                connection.setRequestMethod("HEAD");
                connection.connect();
                String contentType = connection.getContentType();
                String contentDisposition = connection.getHeaderField("Content-Disposition");

I know what you want, but the current version we have not yet supported return the response header to the up layer because for the most user they don't need such object and transfer the response header is a little heavy.
But for the current version, you also can get the response header through customizing your own connection component, ref: https://github.com/lingochamp/FileDownloader/wiki/Customizable-Component

BUT I have another point to support this feature very light, on the later version, I will support carry response header through connect callback.

This has been implemented on okdownload.


[FEAT]: Ability to Specify Schema Name Alongside Database Credentials in the SQLAgent custom skill

What would you like to see?
I would like to propose a feature enhancement for the AnythingLLM project. Currently, when configuring database connections, users can specify database credentials, but there is no option to define the schema name directly.
Suggested Enhancement
It would be beneficial to have the ability to specify the schema name in addition to the existing database credentials. This enhancement would provide greater flexibility and usability for users working with databases that contain multiple schemas.
Use Case
For projects that require access to specific schemas within a database, being able to specify the schema name directly in the configuration would streamline the process and reduce the need for additional queries to select the desired schema.
Additional Information
Please let me know if further details are required or if there are existing discussions related to this feature.
Thank you for considering this enhancement!

This can be done with prompt (system or input) guiding again for specify conditions or unique schema fields for a given database! This will perform better than just relying on the SQL:list-schema skill since most times column names are not super descriptive and on a sufficiently large DB - would be very helpful for an agent.
Likewise, if your SQL use case is very unique or just you have a desired way that SQL calls should be done (referring to your DLL upload request) you should consider building out a custom agent skill!.
There are no limits to what you can do with custom skills and as long as it works in the framework for an agent you can run any code you want. This would ensure SQL agent calls work exactly the way you want without relying on us to build out enhancements or features that complicate this built-in skill beyond a typical use case


ImportError: No module named 'sdd_segment'

Hi, here is the error occurred while I'm trying to import torch_blocksparse
>>> import torch
>>> import torch_blocksparse
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch_blocksparse/__init__.py", line 2, in <module>
    from .matmul import MatMul, Linear
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch_blocksparse/matmul.py", line 209, in <module>
    class _sparse_matmul(torch.autograd.Function):
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch_blocksparse/matmul.py", line 400, in _sparse_matmul
    extra_cflags=['-O2', '-fopenmp']).sdd_segment
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 1124, in load_inline
    keep_intermediates=keep_intermediates)
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 1190, in _jit_compile
    return _import_module_from_library(name, build_directory, is_python_module)
  File "/home3/liyz/venv_tbs_2/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 1534, in _import_module_from_library
    file, path, description = imp.find_module(module_name, [path])
  File "/home3/liyz/python36/lib/python3.6/imp.py", line 297, in find_module
    raise ImportError(_ERR_MSG.format(name), name=name)
ImportError: No module named 'sdd_segment'
>>> exit()

Environment info:

OS: Ubuntu 16.04.6 LTS.
python version: virtualenv based on python3.6.9
Since I don't have the root user access for the sudo apt-get install operation, I just download clang+llvm-9.0.1-x86_64-linux-gnu-ubuntu-16.04 from (here)[https://github.com/llvm/llvm-project/releases/tag/llvmorg-9.0.1] and add the bin folder to PATH.
I used pip install torch-blocksparse rather from source.
Tried both gcc of version 5.5 and 7.4.
Tried both cuda of version 10.1 and 10.2
modified ['ninja','-v'] to ['ninja','--version'] in .../site-packages/torch/utils/cpp_extension.py . ( otherwise a ninja error will orcur)

I'm not sure that if I missed something necessary for the installation. Could you help me out of this? Thanks!

Hey!
Thanks for pointing this out. Can you pip uninstall torch-blocksparse and pip install torch-blocksparse ? I have upgraded the version on pip.

Thanks for your reply and update.
I've solve the problem by reinstall the packages in a conda environment.


More tests

Codecov Report

Merging #26 into master will increase coverage by 3.89%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   64.85%   68.74%   +3.89%
==========================================
  Files          19       19
  Lines        1269     1267       -2
==========================================
+ Hits          823      871      +48
+ Misses        428      381      -47
+ Partials       18       15       -3

Impacted Files
Coverage Δ

couchdb.go
100% <100%> (+70%)
:arrow_up:

options.go
100% <100%> (+100%)
:arrow_up:

client.go
100% <100%> (+13.63%)
:arrow_up:

chttp/chttp.go
69.65% <0%> (+2.06%)
:arrow_up:

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 a188a91...02e3f19. Read the comment docs.


wildcard match of interface for dubbo proxy

Title: One line description
Description:
The 'interface' in the current dubbo proxy only support exact matching. That is, if a dubbo service provides multiple interfaces, we need to create multiple route rules.
It would be more flexible if they supported wildcards. For example:
"*"
"com.dubbo.example.*"
"com.envoy.dubbo.*"
"com.dubbo.*"

wip

/no-stale


Fix formatting

Fix formatting error from my suggestion

@hexbabe Hi, sorry, quick formatting fix for you if you have a sec. Thanks!


[Web] HardwareKeyboard logicalKeysPressed returns previously pressed keys

HardwareKeyboard.instance.logicalKeysPressed returns keys that was previously pressed instead of only the currently pressed key. Happens when pressing the meta key in addition to other keys, on Mac OS.
Steps to Reproduce

flutter create --platforms web foo
Put attached code sample in lib/main.dart
flutter run -d chrome -t lib/main.dart
Open page in chrome
Press keys cmd + k and release
Press key arrow left and release
Check console output
See that the last keypress includes the key k when only the arrow left key was pressed

Expected results:
LogicalKeySet#4b191(keys: Meta Left)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#6c398(keys: Arrow Left)

Actual results:
LogicalKeySet#4b191(keys: Meta Left)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#6c398(keys: Arrow Left + Key K)

Code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: GlobalKeyboardListener(
        onKeyEvent: (KeyEvent event) {
          if (event is KeyDownEvent) {
            if (HardwareKeyboard.instance.logicalKeysPressed.isNotEmpty) {
              final pressedKeys = LogicalKeySet.fromSet(
                HardwareKeyboard.instance.logicalKeysPressed,
              );

              // ignore: avoid_print
              print(pressedKeys);
            }
          }
          return false;
        },
        child: const Text("hello"),
      ),
    );
  }
}

class GlobalKeyboardListener extends StatefulWidget {
  final bool Function(KeyEvent keyEvent) onKeyEvent;
  final Widget child;

  const GlobalKeyboardListener({
    required this.onKeyEvent,
    required this.child,
    Key? key,
  }) : super(key: key);

  @override
  State<GlobalKeyboardListener> createState() {
    return _GlobalKeyboardListenerState();
  }
}

class _GlobalKeyboardListenerState<T> extends State<GlobalKeyboardListener> {
  @override
  void initState() {
    startListen();
    super.initState();
  }

  @override
  void dispose() {
    stopListen();
    super.dispose();
  }

  @override
  Widget build(BuildContext context) {
    return widget.child;
  }

  void startListen() {
    HardwareKeyboard.instance.addHandler(widget.onKeyEvent);
  }

  void stopListen() {
    HardwareKeyboard.instance.removeHandler(widget.onKeyEvent);
  }
}

Logs
Analyzing foo...
No issues found! (ran in 1.3s)

$ flutter doctor -v
[✓] Flutter (Channel stable, 2.8.0, on macOS 12.0.1 21A559 darwin-x64, locale en-GB)
    • Flutter version 2.8.0 at /usr/local/Caskroom/flutter/2.2.1/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision cf44000065 (13 days ago), 2021-12-08 14:06:50 -0800
    • Engine revision 40a99c5951
    • Dart version 2.15.0

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[!] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 96.0.4664.110

! Doctor found issues in 3 categories.

Hi @prasmussen, Thanks for filing the issue. I am able to reproduce the issue on stable and the master channel on the web, When cmd+k is pressed and then released (both keys) and when the left arrow is pressed and released the console outputs:  keys: Arrow Left + Key K) instead of (keys: Arrow Left)
This issue is only on the web and works fine on Desktop.

logs (web)
mahesh@Maheshs-MacBook-Air-M1 counter_app % flutter run -d chrome
Launching lib/main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             13.9s
This app is linked to the debug service: ws://<IP_ADDRESS>:56848/J7BTWRCU9Zk=/ws
Debug service listening on ws://<IP_ADDRESS>:56848/J7BTWRCU9Zk=/ws

💪 Running with sound null safety 💪

🔥  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:56848/J7BTWRCU9Zk=
The Flutter DevTools debugger and profiler on Chrome is available at:
http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:56848/J7BTWRCU9Zk=
LogicalKeySet#4b191(keys: Meta Left)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#6c398(keys: Arrow Left + Key K)
LogicalKeySet#4b191(keys: Meta Left)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#6c398(keys: Arrow Left + Key K)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#7a008(keys: Arrow Left)
LogicalKeySet#4b191(keys: Meta Left)
LogicalKeySet#5c7f7(keys: Meta Left + Key K)
LogicalKeySet#6c398(keys: Arrow Left + Key K)

logs (macos)
mahesh@Maheshs-MacBook-Air-M1 counter_app % flutter run -d macos
Launching lib/main.dart on macOS in debug mode...
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:x86_64, id:6A745C96-D83E-5C8D-8EE9-5958E598001B }
{ platform:macOS, name:Any Mac }
Building macOS application...
Syncing files to device macOS...                                   111ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h List all available interactive commands.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).

💪 Running with sound null safety 💪

An Observatory debugger and profiler on macOS is available at: http://<IP_ADDRESS>:56640/9oDhGyz1Joc=/
The Flutter DevTools debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:56640/9oDhGyz1Joc=/
flutter: LogicalKeySet#00106(keys: Meta Left)
flutter: LogicalKeySet#de8aa(keys: Meta Left + Key K)
flutter: LogicalKeySet#00302(keys: Arrow Left)
flutter: LogicalKeySet#00106(keys: Meta Left)
flutter: LogicalKeySet#de8aa(keys: Meta Left + Key K)
flutter: LogicalKeySet#00302(keys: Arrow Left)
flutter: LogicalKeySet#00106(keys: Meta Left)
flutter: LogicalKeySet#de8aa(keys: Meta Left + Key K)
flutter: LogicalKeySet#00302(keys: Arrow Left)
flutter: LogicalKeySet#00106(keys: Meta Left)
flutter: LogicalKeySet#de8aa(keys: Meta Left + Key K)
flutter: LogicalKeySet#00302(keys: Arrow Left)

flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.8.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (6 days ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

[✓] Flutter (Channel master, 2.9.0-1.0.pre.93, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.9.0-1.0.pre.93 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 1be7f1ea56 (7 days ago), 2021-12-14 22:14:07 -0500
    • Engine revision e444009bf4
    • Dart version 2.16.0 (build 2.16.0-109.0.dev)
    • DevTools version 2.9.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.61.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

cc: @dkwingsmt

This looks like a framework issue. Removing the web label for now.

I can reproduce it on windows desktop but on macOS desktop, this indeed seems to be from the framework
flutter: LogicalKeySet#00104(keys: Alt Left)
flutter: LogicalKeySet#720bd(keys: Alt Left + Key K)
flutter: LogicalKeySet#e3f7b(keys: Arrow Left + Key K)
flutter: LogicalKeySet#c8814(keys: Arrow Right + Key K)
flutter: LogicalKeySet#113e1(keys: Arrow Down + Key K)
flutter: LogicalKeySet#6b3a8(keys: Arrow Up + Key K)
flutter: LogicalKeySet#113e1(keys: Arrow Down + Key K)
flutter: LogicalKeySet#e3f7b(keys: Arrow Left + Key K)
flutter: LogicalKeySet#113e1(keys: Arrow Down + Key K)
flutter: LogicalKeySet#6b3a8(keys: Arrow Up + Key K)


wascc-host v0.12.0

Including<EMAIL_ADDRESS>The main change is making the standard Result error type Box<dyn std::error::Error + Send + Sync>.

Need to wait for standard capability providers (like logging) to be upgraded:
error[E0277]: the trait bound `wascc_logging::LoggingProvider: wascc_codec::capabilities::CapabilityProvider` is not satisfied
   --> runtime/src/main.rs:82:45
    |
82  |     let logging_provider_id = capability_id(&logging_provider)?;
    |                                             ^^^^^^^^^^^^^^^^^ the trait `wascc_codec::capabilities::CapabilityProvider` is not implemented for `wascc_logging::LoggingProvider`

Replaced by https://github.com/wascc/aws-lambda-wascc-runtime/issues/58.


Benchmark should include a test of deletion

With regulation such as GDPR in the EU, deletions is a very important feature to support. Benchmark should measure the performance of random deletions as conformance with GDPR cannot rely on expiry of the data.

Great suggestion @hbs . Adding UPDATEs and DELETEs is on our shortlist of functionality to test across DBs, even more so with regulations like GDPR now fully in place.
Feel free to add any thoughts on the implementation/design around deletion benchmarking for any of the featured databases to this issue.


Implementing Milestone 1 Report to Wiki

As the Milestone 1 Report is set with the #65, we need to implement the missing chapters of the report to related wiki page.

Completed Evaluation of Tools and Processes and added it to the report.
Added deliverable titles to be filled with the views of them.


Provide more descriptive error message when pipenv fails locking requirements

Describe the bug
As a user,
If for example, a package which I am trying to install does not exist, I get an error which is very uninformative (see below)
and therefore, I want the output to be descriptive
so that I could understand where the problem came from.
Error:
Couldn't process kernel message SyntaxError: Unexpected token c in JSON at position 0
    at JSON.parse (<anonymous>)
    at Function.Parse (requirements.ts?d871:10)
    at output_callback (thoth.ts?3aef:413)
    at Kernel._handle_output_message (kernel.js:1196)
    at i (jquery.min.js:2)
    at Kernel._handle_iopub_message (kernel.js:1223)
    at Kernel._finish_ws_message (kernel.js:1015)
    at kernel.js:1006

Wed, 23 Oct 2019 11:28:43 GMT [nbrequirements] ERROR: Failed to lock requirements.
 Error: Error: Requirements locked are not defined.
    at eval (notebook.ts?3f4b:148)

To Reproduce
%dep add non-existing-package
%dep lock

/label hacktoberfest

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
/lifecycle stale

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
/close

@sesheta: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
/close

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