The command line tool named 'sherpa-onnx-cli' for converting hotwords to tokens was not found.

Thank you to the entire team for your hard work and dedication. Your contributions have made a significant impact on this project. Your work has been immensely beneficial to me.

Did you solve it? I have the same problem.
berry@berry:~/sherpa-onnx/build/bin $ sherpa-onnx-keyword-spotter --help -bash: sherpa-onnx-keyword-spotter: command not found berry@berry:~/sherpa-onnx/build/bin $ sherpa-onnx-cli text2token --help -bash: sherpa-onnx-cli: command not found
these command tools can be found.

It looks like you need to install Python package. https://k2-fsa.github.io/sherpa/onnx/python/install.html#check-your-installation
Extracting sherpa_onnx-1.10.27-py3.11-linux-aarch64.egg to /usr/local/lib/python3.11/dist-packages
Adding sherpa-onnx 1.10.27 to easy-install.pth file
**Installing sherpa-onnx-cli script to /usr/local/bin**

Installed /usr/local/lib/python3.11/dist-packages/sherpa_onnx-1.10.27-py3.11-linux-aarch64.egg
Processing dependencies for sherpa-onnx==1.10.27
Finished processing dependencies for sherpa-onnx==1.10.27
berry@berry:~/sherpa-onnx $ sherpa-onnx-cli
Usage: sherpa-onnx-cli [OPTIONS] COMMAND [ARGS]...

  The shell entry point to sherpa-onnx.

Options:
  --help  Show this message and exit.

pip install sherpa-onnx
should fix the issue.


MEN-2408: Remove the sudo call for Docker related commands

Changelog: None
Signed-off-by: Lluis Campos<EMAIL_ADDRESS>

See https://tracker.mender.io/browse/MEN-2408 and https://github.com/mendersoftware/integration/pull/584
Merging.


How to create the SLN file?.

Hi team,
The document is vague. it is not telling about how to create the SLN file or what all it contains?. As a novice to MSIX via devops, I dont know where I need to give my executable which will be get created to MSIX. Please add a link on how to create SLN or explain the relevance of the variables you are using in the documentation.

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

ID: 117dd71d-8f8c-3615-a844-bbc0c076327b
Version Independent ID: 962ddf35-8f87-fe37-d90c-dcca230f4f28
Content: MSIX Packaging Extension - MSIX
Content Source: msix-src/desktop/msix-packaging-extension.md
Product: msix
GitHub Login: @Huios
Microsoft Alias: tajimha

Hi @amalkabraham001 the .sln file mentioned here is your solution file from your windows app source code. If you are looking to build an MSIX package from your windows app source code and then use CI/CD, you can start here: https://docs.microsoft.com/en-us/windows/msix/desktop/source-code-overview


Update small translation change

Can you please update the translations?
I've made some small changes for this language-pack via crowdin.
Thanks in advance.

Thanks for your contribution,
It will generate automatically daily :)


About the version tag

Hello @lararennie ,
I am the same guy of cdnjs project.
https://github.com/googlei18n/libphonenumber/issues/1325
I think it's better to use x.x.x except libphonenumber-x.x.x as git tag.
It will be easy for us to add auto-update config  to automatically update new version when you publish new release.
So, could you change the new naming scheme for git tag in the future?
Thanks for your contribution.
https://github.com/cdnjs/cdnjs/issues/7805
https://github.com/cdnjs/cdnjs/pull/8944

I see on stackoverflow that one standard is to use vX.Y.Z, would this also work for you?

Yes
So you will change the tag format in the future?

Yes @kennynaoh, this sounds like a reasonable change.
Given other requirements it may take a bit of time, just to manage expectations.

@keghani
Please let me know if you publish new version with new git tag vX.Y.Z
Thanks!

Will do :) We'll update this issue. Thanks for your patience!

Hi!
I can prioritize this in time for the next release.
Do I understand the scope correctly? Will I be fixing the issue completely if I make the next tag (and so forth) here look like v7.7.4?

v7.7.4 is great, thanks!

Fixed in 7.7.4.

Thank you so much


Update boomer.h

write the code for the lowercase r character
Describes your contribution
Referenced Issues
closes #issue_number

added  lowercase r


Feature/add upload

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Iterative blur transformation

Currently, BlurTransformationPlugin allows only radius == 25 and only 1 iteration. But there are many cases when an image should be blurred more than once (be more blurry).

Hey @TemMax, in version 2.1.4, you can set the radius from 0 to Int.MAX_VALUE. Thanks for reporting nice issues!


Performance with closures, and specialising for a function

The code makes heavy use of closures. It would be useful to investigate the performance of these closures, and decide if e.g. FastClosures.jl is needed.
Similarly, perhaps a more general struct for these problems is required, where the functions D, R, and T are typed, e.g:
struct BootstrapGP
    ⋮
    diffusion::D
    reaction::R
    delay::T
    ⋮
end

This might help the compiler know exactly the "type" of these functions. Maybe - I don't know enough about these deeper details of Julia, but it's worth thinking about.

See also https://discourse.julialang.org/t/extra-allocations-time-when-a-function-is-returned-from-another-function-than-defined-directly/81933/5


[SPARK-44882][PYTHON][CONNECT] Remove function uuid/random/chr from PySpark

What changes were proposed in this pull request?
Remove function uuid/random/chr from PySpark
Why are the changes needed?
The three functions are controversial and needs further discussion, we can add them back in the future.

uuid and random are also the names of built-in modules,

chr is the name of a built-in function

This PR is to avoid namespace conflict which may break existing workloads, e.g.
import uuid
from pyspark.sql.functions import *
print(uuid.uuid4())

Does this PR introduce any user-facing change?
yes
How was this patch tested?
updated CI
Was this patch authored or co-authored using generative AI tooling?
NO

cc @gatorsmile @HyukjinKwon @ueshin

Merged to master and branch-3.5.


Rev: avoiding safeParse() in the ez.file() implementation

Since v16.1.0
Partially reverts #1424, based on the performance impact investigation.
 ✓ tests/bench/experiment.bench.ts (2) 10490ms
   ✓ Experiment (2) 10487ms
     name             hz     min      max    mean     p75     p99    p995    p999     rme  samples
   · original  19,856.90  0.0361   6.9001  0.0504  0.0452  0.0936  0.1605  2.3311  ±1.59%    99285
   · featured  54,453.60  0.0107  10.9036  0.0184  0.0146  0.0285  0.0426  2.7442  ±2.67%   272406   fastest

 BENCH  Summary

  featured - tests/bench/experiment.bench.ts > Experiment
    2.74x faster than original

coverage: 100.0%. remained the same
when pulling 896b429859b7e4854e49e3e22333eec7809394e8 on rev-1424-parse-for-upload
into f17c7903a30b576ef289351f873f064a4f512a2c on master.


In logs folder sometimes a Unix Executable File is created called logs

This executable file sometimes creates issues and the current solution is to delete the file until I can figure out what creates the file.

Possibly caused by pulling out the SD Card while the program is on and writing to a file. Further testing required


cool!

this looks so cool, but is there any README with more details?
keep it up, cheers!

In the next update, i'll push a README too.
Can't believe i forgot it ;_;

Great, but can you fix the link to Casptone in README? thanks.

Fixed, sorry for the inconvenient.


Supabase Realtime not working with table name

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
Supabase Realtime doesn't get emit events when provided with table name, but works without table name in localhost
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Went to /database/replication and turned on supabase_realtime

Turned on table for which I want realtime events

In Javascript, If I do something like this, this won't work
supabase
            .client
            .channel('user_list_changes')
            .on(
                'postgres_changes',
                {
                    event: '*',
                    schema: 'public',
                    table: 'users_list',
                },
                (payload: any) => console.log('payload', payload),
            )
            .subscribe();

but this would
supabase
           .client
           .channel('user_list_changes')
           .on(
               'postgres_changes',
               {
                   event: '*',
                   schema: 'public',
               },
               (payload: any) => console.log('payload', payload),
           )
           .subscribe();

Expected behavior
I was expecting to receive events when provided with table_name
Screenshots
If applicable, add screenshots to help explain your problem.
System information

OS: Mac
Browser: Brave
Version of supabase-js:

   "<EMAIL_ADDRESS>   "@supabase/functions-js" "^2.1.0"
    "@supabase/gotrue-js" "^2.46.1"
    "@supabase/postgrest-js" "^1.8.0"
    "@supabase/realtime-js" "^2.7.4"
    "@supabase/storage-js" "^2.5.1"
    ```
- Version of Node.js: 18.3.0

@kamilogorek
Both of these don't log on update when I have enabled RLS policy on table
    React.useEffect(() => {
        console.log('here');
        serviceSubscription.current = supabase
            .getClient()
            .channel(`service-data-${serviceName}`)
            .on(
                'postgres_changes',
                {
                    event: '*',
                    schema: 'public',
                    table: 'services_connected',
                },
                (payload) => console.log('Change', payload),
            )
            .subscribe();

        supabase
            .getClient()
            .channel(`service-data`)
            .on(
                'postgres_changes',
                {
                    event: '*',
                    schema: 'public',
                },
                (payload) => console.log('Global', payload),
            )
            .subscribe();
    }, []);

Here is my RLS policy. (Normal Select queries work as expected).

@kamilogorek Any update on this?

@RentfireFounder it's working locally but not working when using Supabase infrastructure?

@filipecabaco no it's working when RLS is disabled but not working when RLS is enabled.
With RLS enabled, the select/update and normal queries work but realtime doesn't
if this is the problem with my RLS rules, why is normal queries working and not realtime?
Also, more on this comment: https://github.com/supabase/realtime-js/issues/254#issuecomment-1773552145

@RentfireFounder just to confirm you're spinning up the Supabase stack locally correct? This is a locally running version of Realtime that RLS is not working for you. Is that right?

@w3b6x9 Yap!

@w3b6x9 @filipecabaco @kamilogorek were you able to replicate it? any updatea?

Hello, i got the same error...

Exact same thing is happening with me, (working with only prod) with RLS turned on i only get DELETE events. No matter what RLS policy I add I dont see any events coming in. Like the OP said on turning RLS off again postgres changes start flowing again.

@filipecabaco @w3b6x9

hey everyone, sorry we were working on a lot of changes and were not able to dedicate enough time to close this issue.
is this issue persisted? were you able to tackle it?

@filipecabaco nope, Also, just in case, did you see this comment? could it be because I am creating my own jwt and using that>

We do have an issue currently with custom jwts where the errors are not properly shown to the user 😞 I wonder if this is related 🤔


Spelling and grammar tweak for pager docs

As I was reading the pager documentation I noticed a very minor spelling mistake and a couple of missing words. This PR corrects both.

I've just now signed a CLA, so the CLA bot will need a re-run.


Add this project/repo to opensourceforshopify.com

Find instructions on how to here: https://github.com/discolabs/opensourceforshopify.com#adding-your-repository

Added this module to opensourceforshopify.com
https://gyazo.com/9f367f910eeb7a518f44b7dde430666c

http://opensourceforshopify.com seems down and I don't see Nodify-Shopify on http://shopify.github.io/

@iamandrebulatov http://shopify.github.io/ only lists official Shopify projects.
Regarding http://opensourceforshopify.com, we should probably create an issue here https://github.com/discolabs/opensourceforshopify.com.


ci: fix breaking Golang Linter

Fix breaking Golang Linter.

It's working now. Giving the issues in the Code.

You can merge it and fix those issues later by raising a PR!

Hey,
Can you please merge this? And check the comment in #10?

Hey @vigneshs-dev,
Will you merge this PR?

Hey @vigneshs-dev,
Will you merge this PR?

Sorry for late reply. I will merge it with main branch. Thankyou for your contribution. @Pradumnasaraf

Thanks


Make IClickableControl public

What does the pull request do?
Make IClickableControl public
What is the current behavior?
#7500 broke hotkeys for custom controls because the HotKeyManager now requires controls to implement IClickableControl  instead of just ICommandSource
What is the updated/expected behavior with this PR?
Allow custom controls to implement IClickableControl
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: Seishiin sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

You can test this PR using the following package version. 11.0.999-cibuild0024476-beta. (feed url: https://nuget.avaloniaui.net/repository/avalonia-all/index.json) [PRBUILDID]

I think ICommandSource is still supported in HotKeyManager?

I think ICommandSource is still supported in HotKeyManager?

Correct me if I'm wrong, but this line is necessary for hotkeys to work. But this point is never reached because the check was updated:
- if (control is not ICommandSource)
+ if (control is not IClickableControl)

I think ICommandSource is still supported in HotKeyManager?

Correct me if I'm wrong, but this line is necessary for hotkeys to work. But this point is never reached because the check was updated:
- if (control is not ICommandSource)
+ if (control is not IClickableControl)

Ohhh, that's weird, because there are still code about ICommandSource above so I was thinking that it's working.

IClickableControl if needed to allow hotkeys to work even without using Command and fire Click Event.

@Seishiin Have you tried using KeyBinding instead?

@workgroupengineering  Could you link me an example?

@workgroupengineering Is there a specific reason why the interface is internal and not public?

See these fragments:
https://github.com/AvaloniaUI/Avalonia/blob/c70d10c65f488b1dee8fd09596d09fd4a3e9a646/src/Avalonia.Diagnostics/Diagnostics/Views/MainWindow.xaml#L19
https://github.com/AvaloniaUI/Avalonia/blob/fb0da85ad3a998bdf711b42e7b619550e6ff9de3/tests/Avalonia.Base.UnitTests/Input/KeyboardDeviceTests.cs#L95

That was exactly what I needed. Thanks.

It is planned for internal use and may be subject to refactoring in the future.

Should I close this PR then?

See these fragments:
https://github.com/AvaloniaUI/Avalonia/blob/c70d10c65f488b1dee8fd09596d09fd4a3e9a646/src/Avalonia.Diagnostics/Diagnostics/Views/MainWindow.xaml#L19
https://github.com/AvaloniaUI/Avalonia/blob/fb0da85ad3a998bdf711b42e7b619550e6ff9de3/tests/Avalonia.Base.UnitTests/Input/KeyboardDeviceTests.cs#L95

That was exactly what I needed. Thanks.

It is planned for internal use and may be subject to refactoring in the future.

Should I close this PR then?

@maxkatz6 @grokys

@workgroupengineering Is there a specific reason why the interface is internal and not public?

Keeping as much as possible code members internal we can make maintenance of the project easier. Otherwise we would need to guarantee stability of these APIs, and don't change them/remove in minor updates.

Should I close this PR then?

If you can achieve your goal without making this interface public, then I would say so.
I will close it. But please let me know, if you think it still should be made public.

#12347 fixes broken hotkeys for custom controls after #7500, just use Avalonia.Input.ICommandSource with hotkey registration using HotKeyManager


增加gdb的测试案例，增加gini包

增加gdb的测试案例，增加gini包

Codecov Report

Merging #286 into master will increase coverage by 0.02%.
The diff coverage is 22.61%.

@@            Coverage Diff             @@
##           master     #286      +/-   ##
==========================================
+ Coverage   75.94%   75.96%   +0.02%
==========================================
  Files         224      225       +1
  Lines       18922    19217     +295
==========================================
+ Hits        14371    14599     +228
- Misses       3910     3975      +65
- Partials      641      643       +2

Impacted Files
Coverage Δ

database/gdb/gdb_mssql.go
0% <0%> (ø)
:arrow_up:

encoding/gparser/gparser_api_encoding.go
88.88% <0%> (-11.12%)
:arrow_down:

database/gdb/gdb_oracle.go
0% <0%> (ø)
:arrow_up:

encoding/gparser/gparser_api_new_load.go
37.5% <0%> (-5.36%)
:arrow_down:

encoding/gjson/gjson_api_new_load.go
79.23% <42.85%> (-2.08%)
:arrow_down:

encoding/gjson/gjson_api_encoding.go
93.33% <57.14%> (-6.67%)
:arrow_down:

encoding/gini/gini.go
84.21% <84.21%> (ø)

database/gdb/gdb_pgsql.go
75% <88.88%> (+75%)
:arrow_up:

os/gspath/gspath_cache.go
86.36% <0%> (-6.82%)
:arrow_down:

os/gtimer/gtimer_entry.go
85.05% <0%> (-3.45%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


create Front-End Website Clones of Amazon, Spotify with netflix using HTML and CSS purely

I propose creating front-end clones of popular websites like Amazon, Spotify, and Netflix. These clones will focus on replicating the user interfaces using HTML, CSS (including CSS animations). The goal is to closely mimic the design  of these websites while integrating advanced front-end techniques like hover effects, transitions, and animations.
Key aspects:
Recreate core front-end elements and layouts of Amazon, Spotify, and Netflix.
Add CSS animations, transitions, and hover effects for enhanced interactivity.
—these are front-end demonstrations only.
This project will allow  to practice front-end development and improve their skills in working with real-world website designs.

write a particular project which you want work on right now and then go for other


Fix build on macOs Mojave with Xcode 10

This fixes build issue with Xcode where it couldn't find C++ include files:
CXX(target) Release/obj.target/magic/src/binding.o
warning: include path for stdlibc++ headers not found; pass '-std=libc++' on the command line to use
      the libc++ standard library instead [-Wstdlibcxx-not-found]
In file included from ../src/binding.cc:3:
../node_modules/nan/nan.h:55:10: fatal error: 'algorithm' file not found
#include <algorithm>
         ^~~~~~~~~~~
1 warning and 1 error generated.
make: *** [Release/obj.target/magic/src/binding.o] Error 1

What version of node are you using?

v4.3.1

That makes sense then. Node v8.0.0 changed things by explicitly using libc++.

PR updated

Hrmm.... now I'm confused because Travis tests against macos 10.13 with latest node v4.x and it is able to compile and run tests successfully without this PR.

10.13 is High Sierra and it was working fine, it is Mojave (10.14) that broke it

Ah ok, I see now.

Landed in cacf8fbdf4bc74e53a09f2c968898b1f91c43eba. Thanks!


Virtual Listview

Newbe to Framework7, old Jquerymobile user.  Will someone give me an example building a list view in javascript.  I have tried to do it with and without Virtual Listview.  Here is my situation:
I have an html page with an empty list view.  I am unable to initialize the list view using the instructions in the documentation.  My list is items are constructed from data base info.  I have tried initializing in the home page init.  I have tried initializing outside the init.  In both cases I get vl.pageContent[0].offsetHeight undefined in framework7.js.
Thanks!

If your virtual list is not on home page, then you have to use page callbacks (http://www.idangero.us/framework7/docs/page-callbacks.html) to init it, for example:
app.onPageInit('somepage', function(){
  var vl = app.virtualList(...);
});

Hi Vladimir,
I am having a tough time with Virtual Listview.  I have to use the virtual list view because I am building the list view from a database.  The list view as shown works if it is static.  I will keep the question very simple.
I have a list view in a file different from called other.html that has the following data:

My Requests

Photo gallery

                            Easy create your own photo gallery app. List and full screen options available.

On my .js page I have the following relevant code.  I did not include the variables in the template to keep it simple.  I am assuming the result would be a list view with identical items as shown in the template.:
var vl;
$$(document).on('pageInit', function (e) {
myApp.onPageInit('my_requests', function(){
vl  = myApp.virtualList('.list-block',{                                                                                template: 'Photo galleryEasy create your own photo gallery app. List and full screen options available.',
});});});
var format_loans = function() {
for (facebook_i=0; facebook_i < facebook_objData.length; facebook_i++) {
rowTemplate += 'From ' +facebook_objData[facebook_i].loaner_name +' on ' + facebook_objData[facebook_i].create_date +'' +facebook_objData[facebook_i].loan_message +',';
}
vl.appendItem(facebook_objData);
};
I am stuck and can not get list views to work that need refreshing such as the one dipicted above.
Any help is appreciated!
Thanks!
Howard

On Jun 20, 2015, at 4:00 AM, Vladimir Kharlampidi<EMAIL_ADDRESS>wrote:
var vl

Hi Vladimir,
I think I see the light.  Ignore my previous email.
Thanks for your support!!!
Howard

On Jun 20, 2015, at 4:00 AM, Vladimir Kharlampidi<EMAIL_ADDRESS>wrote:
If your virtual list is not on home page, then you have to use page callbacks (http://www.idangero.us/framework7/docs/page-callbacks.html http://www.idangero.us/framework7/docs/page-callbacks.html) to init it, for example:
app.onPageInit('somepage', function(){
var vl = app.virtualList(...);
});
—
Reply to this email directly or view it on GitHub https://github.com/nolimits4web/Framework7/issues/515#issuecomment-113746247.


clarify format is nonstandard

Email from John Levine:
Thanks, although the target format isn't really Maildir.  It's some mutant
maildir arrangement unique to Kmail.

Fixed in 5249b5e


Optional type checking with mypy

adds dependencies to allow type checking if people wish
adds a workflow to CI to run type checking (not necessarily intended to be a required check, seems worth discussing what seems most useful to people)
fixes up our code so that it passes type checking
updates readme with relevant info for running the type checker
some other opportunistic fixups i noticed along the way

starting this off as a draft PR for discussion, since i'm not sure we want to do this?  mypy was useful for finding issues in dlme-airflow and ils-middleware, and i played with this just a bit during the WC, but mostly tried to stay away from it as it didn't seem like critical path work at the time.  but this did seem like a good experiment to finish off during the first day and a half of unscheduled week.

@jermnelson Do you want to take a look?


Mux ServeHTTPC supports context of generic implementation

ServeHTTPC  method will embed a chi.Context inside an existing different implementation of context.Context

@mrcpvn thanks for the PR! just wondering, did you run into this because of a bug..?

@pkieltyka yes, if the input of the (*Mux).ServeHTTPC function was not a chi.Context there was a panic on the chi.(*treeRouter).ServeHTTPC
panic: interface conversion: interface {} is nil, not *chi.Context [recovered]
panic: interface conversion: interface {} is nil, not *chi.Context
goroutine 41 [running]:
panic(0x3adc40, 0xc8201228c0)
/usr/local/go/src/runtime/panic.go:464 +0x3e6
testing.tRunner.func1(0xc820139320)
/usr/local/go/src/testing/testing.go:467 +0x192
panic(0x3adc40, 0xc8201228c0)
/usr/local/go/src/runtime/panic.go:426 +0x4e9
github.com/pressly/chi.treeRouter.ServeHTTPC(0xc820103f80, 0xc820104150, 0x14c00e0, 0xc820103fb0, 0x14c0288, 0xc8200feea0, 0xc82012b5e0)
/Users/marco/Programmazione/go/src/github.com/pressly/chi/mux.go:329 +0x301
github.com/pressly/chi.(*treeRouter).ServeHTTPC(0xc820120ca0, 0x14c00e0, 0xc820103fb0, 0x14c0288, 0xc8200feea0, 0xc82012b5e0)
:18 +0xca
github.com/pressly/chi.(*Mux).ServeHTTPC(0xc820127080, 0x14c00e0, 0xc820103fb0, 0x14c0288, 0xc8200feea0, 0xc82012b5e0)
/Users/marco/Programmazione/go/src/github.com/pressly/chi/mux.go:288 +0x6c
github.com/pressly/chi.TestMuxServeHTTPC(0xc820139320)
/Users/marco/Programmazione/go/src/github.com/pressly/chi/mux_test.go:325 +0x992
testing.tRunner(0xc820139320, 0x6112c8)
/usr/local/go/src/testing/testing.go:473 +0x98
created by testing.RunTests
/usr/local/go/src/testing/testing.go:582 +0x892
exit status 2
FAIL    github.com/pressly/chi  0.052s

this PR is interesting, but I'll probably go with a different approach to solving it.

Thanks for the PR @mrcpvn .. I've closed it with 79fa9188c5d924a6408a8b8ef4f267acec735a84

Hello @pkieltyka. Can you please explain why my implementation was not a good solution for this issue?
I think that an incoming context should be handled by the router.

hey @mrcpvn. Definitely, I agree the incoming context should be handled by the router and it is, but by ServeHTTP instead of ServeHTTPC like you suggested in the PR. The typical connection point of a chi.Mux is as an http.Handler, passed to http.ListenAndServe().
When would you call mux.ServeHTTPC()? I suppose one example could be if you're writing tests and testing specific handlers via httptest.NewRecorder(). I've been writing my http tests with httptest.NewServer(mux). If the reason is for tests, one way I could do it is export the newContext() method in chi so someone could make a new routing context for purposes of tests. Let me know if you have other reasons to call ServeHTTPC directly on a mux object. In order to support both methods, it just makes the code a bit messy and I'd rather avoid it by convention. If there are specific needs for it, please let me know and there might be a different way to solve it.
Second and most importantly, chi v2.0 will be ready for Go 1.7, which no longer needs the ServeHTTPC() method because http.Request now has a Context field of its own :) yeay... so again, this problem goes away. See https://github.com/pressly/chi/blob/v2 -- this version is going to be a major step forward, still very minimal in philosophy but static checking everywhere in the router, and I'm going to add a few more perks now that I can lean on context.Context even further. At that point chi v1 will go into a stable branch where < 1.7 users can still use it until they upgrade. But still, v1 should be stable an very refined.
Third, in relation to your PR specifically, the defer statement does have a cost, and I'd rather not use it per request.

Thank you for the explanation @pkieltyka. I found the issue with mux.ServeHTTPC because I was calling the method from a middleware handler that is context aware and will pass a context.Context to the chi.Router (calling directly the ServeHTTPC method). This is probably an edge case but should not be ignored.
I know that with Go 1.7 this problem will go away but for the moment I need to support a context in the ServeHTTPC method.

Can you show example of middleware / router? I'll get a proper fix for this to make sure v1 is stable

On May 9, 2016, at 10:22 AM, mrcpvn<EMAIL_ADDRESS>wrote:
Thank you for the explanation @pkieltyka. I found the issue with mux.ServeHTTPC because I was calling the method from a middleware handler that is context aware and will pass a context.Context to the chi.Router (calling directly the ServeHTTPC method). This is probably an edge case but should not be ignored.
I know that with Go 1.7 this problem will go away but for the moment I need to support a context in the ServeHTTPC method.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

I pushed the example in this repository:
https://github.com/mrcpvn/chi_middleware

@mrcpvn @VojtechVitek let me know what you guys think of https://github.com/pressly/chi/pull/40
All tests pass and benchmarks remain the same.

@pkieltyka @VojtechVitek the implementation looks good to me.
I have also done some performance tests and found no issues.


"pomerium-cli -v" is not working to figure out installed version

What happened?
Pomerium CLI Version 0.17.1 is not providing proper version via pomerium-cli -v  or pomerium-cli --version
What did you expect to happen?
pomerium-cli -v  or pomerium-cli --version is returning correct version of Pomerium CLI
How'd it happen?

run pomerium-cli -v or pomerium-cli --version
cli shows following

What's your environment like?

Pomerium version (retrieve with pomerium --version): 0.17.1
Server Operating System: Windows 10

Additional comment
Doesn't allow easy Analysis of user reports in case of Issues with Pomerium and Pomerium CLI

Confirmed.

@sherifkayad https://github.com/pomerium/cli/releases/tag/v0.17.2


[Temp] maven release branch -- Don't merge

rebase codebase to release and CI will automatically release maven deps

No longer needed.


Getting exception Failed to generate ID. Underlying exception was: {} when running test case

Single tests run perfectly but when all the tests are run it fails saying. @nathankleyn  Can you shed some light?
18:40:56 - Failed to generate ID. Underlying exception was: {}
redis.clients.jedis.exceptions.JedisConnectionException: Unexpected end of stream.
	at redis.clients.util.RedisInputStream.ensureFill(RedisInputStream.java:198) ~[jedis-2.7.3.jar:?]
	at redis.clients.util.RedisInputStream.readByte(RedisInputStream.java:40) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Protocol.process(Protocol.java:141) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Protocol.read(Protocol.java:205) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Connection.readProtocolWithCheckingBroken(Connection.java:297) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Connection.getOne(Connection.java:279) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Jedis.getEvalResult(Jedis.java:2642) ~[jedis-2.7.3.jar:?]
	at redis.clients.jedis.Jedis.evalsha(Jedis.java:2668) ~[jedis-2.7.3.jar:?]
	at com.intenthq.icicle.JedisIcicle$2.apply(JedisIcicle.java:85) ~[icicle-jedis-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.JedisIcicle$2.apply(JedisIcicle.java:78) ~[icicle-jedis-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.JedisIcicle.withJedis(JedisIcicle.java:123) ~[icicle-jedis-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.JedisIcicle.evalLuaScript(JedisIcicle.java:78) ~[icicle-jedis-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.IcicleIdGenerator.executeLuaScript(IcicleIdGenerator.java:262) ~[icicle-core-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.IcicleIdGenerator.executeOrLoadLuaScript(IcicleIdGenerator.java:237) ~[icicle-core-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.IcicleIdGenerator.generateIdsUsingRedis(IcicleIdGenerator.java:179) ~[icicle-core-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.IcicleIdGenerator.generateIdBatch(IcicleIdGenerator.java:148) [icicle-core-1.1.0.jar:1.1.0]
	at com.intenthq.icicle.IcicleIdGenerator.generateId(IcicleIdGenerator.java:115) [icicle-core-1.1.0.jar:1.1.0]

May be this is some kind of warning message? As the test case in which this trace appears succesfully passes and have unique ids.

But output of my test cases turns out to be too ugly.

@vijayrawatsan What's happened here is that for some reason Redis has closed one of the workers while running the generation commands. I'm not sure why this is exactly. It may be that your Jedis pool isn't big enough. It could be that you're using a more recent version of Jedis (we're 2.4 currently, although there are plans to upgrade) and it's got an incompatibility with one of the responses.
The ID generation will attempt generation up to 5 times by default. You can set that number to whatever you want. When running tests, you probably want to set it to 1. The message you're seeing is just a warning that a retry took place, there mainly because you'll want to work out the root cause for the retries needing to take place at all (probably one of the above things).

@nathankleyn when I turned down the retries to just 1, value abset exception pops in.
But after I set up pool size correctly and it worked like a charm.
There is one more issue with your lib. (I hope again that its my own issue).
I am using your lib in a multi-module maven project.
Now lets say module2 depends on module1
Both modules are living a happy life before you lib kicks in.
Now module1 depends on icicle. Module1 works and builds just fine.
But spring mongo libraries in module2 are now somehow not working. All the spring-data-mongo imports are in red.
For the time being I copy pasted your code and it seems to work fine.

@vijayrawatsan Sounds like dependency version clashes. Can you double check the dependencies that Icicle has and see if any of your modules have conflicting versions? It's probably about time for us to bump some of these versions up to the latest anyway.

@nathankleyn My common dependencies were already excluded.
<dependency>
    <groupId>com.intenthq.icicle</groupId>
    <artifactId>icicle-jedis</artifactId>
    <exclusions>
        <exclusion>
            <groupId>com.google.guava</groupId>
            <artifactId>guava</artifactId>
        </exclusion>
        <exclusion>
            <groupId>redis.clients</groupId>
            <artifactId>jedis</artifactId>
        </exclusion>
        <exclusion>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-simple</artifactId>
        </exclusion>
    </exclusions>
</dependency>

@vijayrawatsan I'm sadly really not sure what the cause could be - I'd expect Icicle to be the one to possibly not work with this setup as the versions that are being used could possibly not be compatible with what Icicle needs. I'm guessing you've checked transient dependencies for clashes as well?

@nathankleyn I tried figuring out the issue but I couldnot. Plus I didn't check for transient dependencies.
Nevermind its working for me now(by copy pasting the entire library code :disappointed: ). I will try to figure out the exact cause when I have time.

@vijayrawatsan :+1:


Clarify that Vault tokens are not refreshed

Whoops 😬
Thanks for that! 🎉
/lgtm
/approve
/ok-to-test

Whoops 😬
Thanks for that! 🎉
/lgtm
/approve
/ok-to-test


Fix benchmark reporting

Description
This Pr fixes benchmark result reporting by adding the step of combining workflow artifacts into a single archive and adding permissions to benchmark_report.yml workflow. It cannot be fully tested without merging. Partial merge was present in, #6875

Maybe add:
  push:
    paths:
      - '.github/workflows/benchmarks**'

to benchmarks.yml?
Then if there is a follow-up it might be easier?

to benchmarks.yml?
Then if there is a follow-up it might be easier?

As the trigger is only to add a label and a single run is 3h of CI work, I'm not sure if it is required.

It works again https://github.com/napari/napari/pull/6727#issuecomment-2085363545


(feat): Distributed list_objects / _path2list()

Is your feature request related to a problem? Please describe.
Ensure _path2list() runs at scale leveraging Ray
Additional context
This is part of the Wrangler @ Scale release
P.S. Please do not attach files as it's considered a security risk. Add code snippets directly in the message body as much as possible.

Improvements to this call seem to negligible or non-existent based on preliminary testing vs. the existing use of paginator = client_s3.get_paginator("list_objects_v2")


Strapi crashes after uninstall

I tried different ways to uninstall this plugin, always resulting in crashed application.
No matter if I do it with npm remove strapi-plugin-navigation or strapi uninstall navigation --delete-files or from the backend.
Log:
strapi     | [2021-07-07T08:23:05.873Z] info Uninstalling navigation...
strapi     | [2021-07-07T08:23:08.276Z] info File changed: /srv/app/package.json
strapi     | [2021-07-07T08:23:27.177Z] info File changed: /srv/app/yarn.lock
strapi     | [2021-07-07T08:24:41.460Z] info File changed: /srv/app/build/*
[...]
strapi     | [2021-07-07T08:24:41.742Z] debug DELETE /admin/plugins/uninstall/navigation (95899 ms) 200
strapi     | [2021-07-07T08:24:41.742Z] info The server is restarting
strapi     |
strapi     | [2021-07-07T08:24:49.043Z] debug ⛔ Server wasn't able to start properly.
strapi     | [2021-07-07T08:24:49.045Z] error TypeError: Cannot read property 'models' of undefined
strapi     |     at DatabaseManager.getModelsByPluginName (/srv/app/node_modules/strapi-database/lib/database-manager.js:158:86)
strapi     |     at Object.getNature (/srv/app/node_modules/strapi-utils/lib/models.js:41:30)
strapi     |     at /srv/app/node_modules/strapi-connector-bookshelf/lib/mount-models.js:185:21
strapi     |     at Array.forEach (<anonymous>)
strapi     |     at updateModel (/srv/app/node_modules/strapi-connector-bookshelf/lib/mount-models.js:178:40)
strapi     |     at async module.exports (/srv/app/node_modules/strapi-connector-bookshelf/lib/mount-models.js:702:28)
strapi     |     at async Promise.all (index 1)
strapi     |     at async mountConnection (/srv/app/node_modules/strapi-connector-bookshelf/lib/index.js:87:31)
strapi     |     at async Promise.all (index 0)
strapi     |     at async Object.initialize (/srv/app/node_modules/strapi-database/lib/connector-registry.js:30:9)
strapi     |     at async DatabaseManager.initialize (/srv/app/node_modules/strapi-database/lib/database-manager.js:43:5)
strapi     |     at async Strapi.load (/srv/app/node_modules/strapi/lib/Strapi.js:354:5)
strapi     |     at async Strapi.start (/srv/app/node_modules/strapi/lib/Strapi.js:196:9)
strapi exited with code 1

Hello @apokalipscke,
What you're doing is correct. Plugin has been removed but I think you forgot to manually amend the changed in your collections you already done by adding attribute relation to navigation item. Am I correct?

I tried it the same minute you wrote this 😂 .
It worked, I just forgot to remove this
"navigation": {
      "model": "navigationitem",
      "plugin": "navigation",
      "via": "related",
      "configurable": false,
      "hidden": true
}

from my model settings.
Thanks!

@cyp3rius
But the tables still exist in database. Don't know if this is an issue ...

That's the strapi stuff :) I'm not expecting the tables are going to be removed automatically, also when you're changing the other models.
We didn't "automate" by our own it to avoid problems with if someone made an extensions to the plugin.


Filter NodePort service target IPs by node label

This PR adds support for a new Service annotation:
external-dns.alpha.kubernetes.io/node-label-filter
When used on a NodePort Service resource, it filters the list of generated endpoints (nodes' IPs) by filtering the k8s nodes list using a label filter.
Use case:
We expose some stateful services using NodePort, and these services are run on specific nodes (e.g. with NodeSelector, Taints, etc). We create persistent connections to these services.
The cluster also contains nodes which are dynamically added/removed because of HPA (horizontal pod autoscaler) and cluster autoscaler. Now, the HPA is quite a dynamic one, so the list of IPs of nodes changes very frequently. This causes the list of IPs of the exposed NodePort service to be too dynamic. Whenever nodes are added/deleted, persistent connections to the stateful services are broken.
By connecting directly ONLY to nodes that contain our stateful service, we can avoid this connection issue.

/assign @justinsb

Sorry for reviewing your PR a bit late.
You might want to check your tests again, currently they're breaking.
could not walk working directory '/home/travis/gopath/src/github.com/kubernetes-incubator/external-dns': error from go test: # github.com/kubernetes-incubator/external-dns/source [github.com/kubernetes-incubator/external-dns/source.test]
./service_test.go:1451:21: cannot use map[string]string literal (type map[string]string) as type bool in field value
./service_test.go:1457:24: cannot use []*endpoint.Endpoint literal (type []*endpoint.Endpoint) as type []string in field value
./service_test.go:1461:4: cannot use false (type bool) as type []*endpoint.Endpoint in field value
./service_test.go:1462:14: cannot use []*"github.com/kubernetes-incubator/external-dns/vendor/k8s.io/api/core/v1".Node literal (type []*"github.com/kubernetes-incubator/external-dns/vendor/k8s.io/api/core/v1".Node) as type bool in field value
./service_test.go:1462:14: too few values in composite literal

Thanks for the reply, should be all right now! :)

@cmaster11 One last request from my side. Do you mind adding some documentation for this?

Added some simple docs :)

Thanks for the effort @cmaster11 – this would simplify a few annoying edge cases in our setup too! Hopefully just a quick rebase away from being merged. :-)

@cmaster11 Ping, let's get this merged! 😄

Would it be reasonable for someone else to take over this PR if @cmaster11 isn't able to?

@cmaster11 to help a little I produced the following patch that can be applied against the master branch.
diff --git a/docs/faq.md b/docs/faq.md
index 1c00adac..25c66a30 100644
--- a/docs/faq.md
+++ b/docs/faq.md
@@ -266,6 +266,20 @@ Yes, give it the correct cross-account/assume-role permissions and use the `--aw

 Separate them by `,`.

+### How do I expose only IPs of certain nodes?
+
+Using the annotation (on ingresses/services) `external-dns.alpha.kubernetes.io/node-label-filter` you can expose
+IPs of only nodes having the desired labels.
+
+```
+external-dns.alpha.kubernetes.io/node-label-filter: workload=testing
+```
+
+One possible use case is when you want to expose NodePort services bound to pods running only in a restricted nodes
+group and you have Cluster Autoscaler enabled. Without filtering the nodes with this annotation, connections will
+pass through other nodes, which may get deleted by the autoscaler. With the annotation, you can choose to expose
+only nodes which are not going to be deleted by the autoscaler, therefore you can prevent connections to such services
+from being abruptly terminated by dying nodes.

 ### Are there official Docker images provided?

@@ -279,4 +293,4 @@ As tags, you can use your version of choice or use `latest` that always resolves

 If you wish to build your own image, you can use the provided [Dockerfile](../Dockerfile) as a starting point.

-We are currently working with the Kubernetes community to provide official images for the project similarly to what is done with the other official Kubernetes projects, but we don't have an ETA on when those images will be available.
\ No newline at end of file
+We are currently working with the Kubernetes community to provide official images for the project similarly to what is done with the other official Kubernetes projects, but we don't have an ETA on when those images will be available.
diff --git a/source/service.go b/source/service.go
index 1fba3d62..b58e6dfe 100644
--- a/source/service.go
+++ b/source/service.go
@@ -339,6 +339,37 @@ func (sc *serviceSource) filterByAnnotations(services []*v1.Service) ([]*v1.Serv
 	return filteredList, nil
 }

+// filterNodesByLabels filters a list of nodes by a given label selector.
+func (sc *serviceSource) filterNodesByLabels(nodeLabelsFilter string, nodes []*v1.Node) ([]*v1.Node, error) {
+	labelSelector, err := metav1.ParseToLabelSelector(nodeLabelsFilter)
+	if err != nil {
+		return nil, err
+	}
+	selector, err := metav1.LabelSelectorAsSelector(labelSelector)
+	if err != nil {
+		return nil, err
+	}
+
+	// empty filter returns original list
+	if selector.Empty() {
+		return nodes, nil
+	}
+
+	var filteredList []*v1.Node
+
+	for _, node := range nodes {
+		// convert the service's annotations to an equivalent label selector
+		nodeLabels := labels.Set(node.Labels)
+
+		// include node if its annotations match the selector
+		if selector.Matches(nodeLabels) {
+			filteredList = append(filteredList, node)
+		}
+	}
+
+	return filteredList, nil
+}
+
 // filterByServiceType filters services according their types
 func (sc *serviceSource) filterByServiceType(services []*v1.Service) []*v1.Service {
 	filteredList := []*v1.Service{}
@@ -488,10 +519,18 @@ func (sc *serviceSource) extractNodePortTargets(svc *v1.Service) (endpoint.Targe
 			}
 		}
 	default:
+		// all nodes
 		nodes, err = sc.nodeInformer.Lister().List(labels.Everything())
 		if err != nil {
 			return nil, err
 		}
+		// filter nodes
+		if nodeLabelsFilter, ok := svc.Annotations[nodeLabelFilterKey]; ok {
+			nodes, err = sc.filterNodesByLabels(nodeLabelsFilter, nodes)
+			if err != nil {
+				return nil, err
+			}
+		}
 	}

 	for _, node := range nodes {
diff --git a/source/service_test.go b/source/service_test.go
index b5265e33..e2252c06 100644
--- a/source/service_test.go
+++ b/source/service_test.go
@@ -1473,6 +1473,66 @@ func TestNodePortServices(t *testing.T) {
 			[]int{},
 			[]v1.PodPhase{},
 		},
+		{
+			"annotated NodePort services return an endpoint with IP addresses of the private cluster's nodes that match node label filter",
+			"",
+			"",
+			"testing",
+			"foo",
+			v1.ServiceTypeNodePort,
+			v1.ServiceExternalTrafficPolicyTypeCluster,
+			"",
+			"",
+			false,
+			map[string]string{},
+			map[string]string{
+				hostnameAnnotationKey: "foo.example.org.",
+				nodeLabelFilterKey:    "workload=testing",
+			},
+			nil,
+			[]*endpoint.Endpoint{
+				{DNSName: "_30192._tcp.foo.example.org", Targets: endpoint.Targets{"0 50 30192 foo.example.org"}, RecordType: endpoint.RecordTypeSRV},
+				{DNSName: "foo.example.org", Targets: endpoint.Targets{"<IP_ADDRESS>", "<IP_ADDRESS>"}, RecordType: endpoint.RecordTypeA},
+			},
+			false,
+			[]*v1.Node{{
+				ObjectMeta: metav1.ObjectMeta{
+					Name: "node1",
+					Labels: map[string]string{
+						"workload": "testing",
+					},
+				},
+				Status: v1.NodeStatus{
+					Addresses: []v1.NodeAddress{
+						{Type: v1.NodeInternalIP, Address: "<IP_ADDRESS>"},
+					},
+				},
+			}, {
+				ObjectMeta: metav1.ObjectMeta{
+					Name: "node2",
+				},
+				Status: v1.NodeStatus{
+					Addresses: []v1.NodeAddress{
+						{Type: v1.NodeInternalIP, Address: "<IP_ADDRESS>"},
+					},
+				},
+			}, {
+				ObjectMeta: metav1.ObjectMeta{
+					Name: "node3",
+					Labels: map[string]string{
+						"workload": "testing",
+					},
+				},
+				Status: v1.NodeStatus{
+					Addresses: []v1.NodeAddress{
+						{Type: v1.NodeInternalIP, Address: "<IP_ADDRESS>"},
+					},
+				},
+			}},
+			[]string{},
+			[]int{},
+			[]v1.PodPhase{},
+		},
 		{
 			"annotated NodePort services with ExternalTrafficPolicy=Local return an endpoint with IP addresses of the cluster's nodes where pods is running only",
 			"",
diff --git a/source/source.go b/source/source.go
index 83fc8469..0738df9a 100644
--- a/source/source.go
+++ b/source/source.go
@@ -39,6 +39,8 @@ const (
 	aliasAnnotationKey = "external-dns.alpha.kubernetes.io/alias"
 	// The value of the controller annotation so that we feel responsible
 	controllerAnnotationValue = "dns-controller"
+	// The annotation used to configure which nodes should be returned for nodeport services
+	nodeLabelFilterKey = "external-dns.alpha.kubernetes.io/node-label-filter"
 )

 // Provider-specific annotations

Will close this PR, this is stale for a while. It also seems that other PR are picking up those changes here.

Any chance we could resurrect this feature, it would be super useful


Casting int to uint in MorganFingerprintHelper leads to unexpected behaviour.

Using python (probably using c++ too), you can pass a negative value for radius and nBits through to the function MorganFingerprintHelper in rdMolDescriptors.cpp . This leads to them being casted to the max value of int minus the passed value. This might be unintended behaviour. I suggest throwing an error at one point of the chain, to let the user know, that he put a negative value there.
Currently, passing a negative value for radius or nBits just leads to the program freezing (or taking extremely long to finish) without any feedback to the user that something might have gone wrong.

nbits=0 also seems to crash without throwing an error/exception.

Hi @greglandrum ,
I would like to take on this issue. Can you please explain what needs to be done here?

Hi Max,
Thanks for volunteering again.
In this case I think there are two things that are worth looking into:

making sure that negative values for either radius or nBits are not passed from Python to the Morgan fingerprinting code. This would be done in $RDBASE/Code/GraphMol/Descriptors/Wrap/rdMolDescriptors.cpp and may be as easy as changing the types of those arguments to MorganFingerprintHelper from int to unsigned int. Otherwise you need to just check the values in the function and throw a value error if they are invalid. To figure out which of these works you'll need to create a test (of course).
making sure that the C++ fingerprinting code does something sensible (throwing a ValueErrorException would be fine) if the number of bits is 0

While making changes in rdMolDescriptors.cpp it's probably worth looking at the other fingerprinting functions there and seeing if any of those also are accepting ints when they really should only be accepting unsigned ints.

@greglandrum How about diff fingerprints where you'd expect negative values? I'd agree that there should be no negative values coming from the FP generation code, but whenever FP type is int, I'd say its fair to expect negative values too. That could be underlying cause for things like #2813


mkdocstrings: Template 'module.html' not found for 'python' handler and theme 'material'.

Describe the bug
mkdocs serve -a <IP_ADDRESS>:9000 prints the following error:
ERROR    -  mkdocstrings: Template 'module.html' not found for 'python' handler and theme 'material'.
ERROR    -  Error reading page 'api.md': module.html
Traceback (most recent call last):
  File "/root/miniconda3/envs/alpha/bin/mkdocs", line 8, in <module>
    sys.exit(cli())
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/__main__.py", line 181, in serve_command
    serve.serve(dev_addr=dev_addr, livereload=livereload, watch=watch, **kwargs)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/commands/serve.py", line 63, in serve
    config = builder()
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/commands/serve.py", line 58, in builder
    build(config, live_server=live_server, dirty=dirty)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/commands/build.py", line 292, in build
    _populate_page(file.page, config, files, dirty)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/commands/build.py", line 174, in _populate_page
    page.render(config, files)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocs/structure/pages.py", line 175, in render
    self.content = md.convert(self.markdown)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/markdown/core.py", line 264, in convert
    root = self.parser.parseDocument(self.lines).getroot()
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/markdown/blockparser.py", line 90, in parseDocument
    self.parseChunk(self.root, '\n'.join(lines))
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/markdown/blockparser.py", line 105, in parseChunk
    self.parseBlocks(parent, text.split('\n\n'))
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/markdown/blockparser.py", line 123, in parseBlocks
    if processor.run(parent, blocks) is not False:
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocstrings/extension.py", line 120, in run
    html, handler, data = self._process_block(identifier, block, heading_level)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocstrings/extension.py", line 196, in _process_block
    rendered = handler.renderer.render(data, rendering)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocstrings/handlers/python/renderer.py", line 70, in render
    template = self.env.get_template(f"{data['category']}.html")
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/jinja2/environment.py", line 997, in get_template
    return self._load_template(name, globals)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/jinja2/environment.py", line 958, in _load_template
    template = self.loader.load(self, name, self.make_globals(globals))
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/jinja2/loaders.py", line 125, in load
    source, filename, uptodate = self.get_source(environment, name)
  File "/root/miniconda3/envs/alpha/lib/python3.8/site-packages/jinja2/loaders.py", line 214, in get_source
    raise TemplateNotFound(template)
jinja2.exceptions.TemplateNotFound: module.html

Information (please complete the following information):

OS: ubuntu 20.04
Browser: firefox
mkdocstrings version: 0.18
mkdocs version: 1.1.2
material: 8.2.15
autorefs: 0.4.1

Hi, can you list the contents of /root/miniconda3/envs/alpha/lib/python3.8/site-packages/mkdocstrings?
You can also try to recreate a fresh conda env.

It recovered magically. I don't realize anything I did. Is it possible caused by my doc strings but the error report is incorrect?

I don't think so. Closing for now, feel free to reopen.


Not detecting blinking in Eye_Tracking_part3

Hi,
I ran Eye_Tracking_part3 -> main.py, the blink count is not working.
Thank you

I changed the ratio from 5.5 to 3.5, it works.


[WIP] Make kind.sh script self-contained

Description
Kind testing script is now self-contained and refactored to easily handle E2E execution
How Has This Been Tested?

[x] E2E testing

Pull Request Test Coverage Report for Build 283771815

0 of 0   changed or added relevant lines in 0 files are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 47.493%

Files with Coverage Reduction
New Missed Lines
%

internal/liqonet/tunnelEndpointCreator-operator.go
5
64.34%

Totals

Change from base Build 282319404:
-0.04%

Covered Lines:
6072

Relevant Lines:
12785

💛  - Coveralls

/rebase


Adding support for a poweroff watcher

Description
Expands the existing reboot watcher to provide support to power off the host system. Executes /sbin/poweroff once /worker/poweroff is written to disk
Why is this needed
Fixes: (https://github.com/tinkerbell/hook/issues/189)
How Has This Been Tested?
I am unable to build hook locally so this is untested.
How are existing users impacted? What migration steps/scripts do we need?
No impact

Hey @jasonyates-thg, thanks for this PR. I am hesitant to add this in because we've talked about adding this functionality as a builtin in the tink worker/agent. Also, there is an alternative way to go about it via waitdaemon

I agree with Jacob. I think we have sufficient workarounds and a good plan to provide builtins that'll solve the problem. Our builtins should follow the v1alpha2 API work that's ongoing.


about UAV_correct_human.py

Hello, @Zhi-Xian-Xie ! When i run the file(UAV_correct_human.py), the following error occurs:
Traceback (most recent call last):
File "UAV_correct_human.py", line 17, in
from utils.RBF import rbf
ImportError: cannot import name 'rbf' from 'utils.RBF' (Safe-MPC-Alignment/utils/RBF.py)
In the file(RBF.py), i don't find the function(rbf).

Sorry for your inconvenience, files named UAV_correct_human.py and all versions of UAV_correct_human_x.py are legacies in development, thus they may not be working. I have deleted them in the code, and you can just run UAV_correct_human_mj_v3.py. Also, line 30 needs to be commented and I have fixed it in the latest commit.

Ok, thank you!


Add Use Cases to DG v1.1

Actually, I think it works if you have committed to your branch and it should have updated the previous PR, not needing you to make another PR!


line-dasharray wrong drawing

line-width: 5;
line-dasharray: (3,9);
line-cap: round;
 line-join: round;

With this i get wrong drawing. I think it might be related to the "complexity" of the line. As i zoom it gets better (2nd image)

if i remove line cap and join it rendering very differently (which is not normal either) but also better as i zoom in

This is a known issue. It is not really visible with thin lines, but becomes ugly with wide lines. It is not easily fixable either, this is something in the TODO list currently.

This is a known issue. It is not really visible with thin lines, but becomes ugly with wide lines. It is not easily fixable either, this is something in the TODO list currently.


Unnecessary dependencies

Hi,
I've been using this lib with great success on AWS lambda.
My main issue is the generated package size which is quite important for a lambda function (faster code uploads + faster js parsing = less money spent)
With a webpack builder and a default config, a simple lambda that only jwt.verify some token comes at a whopping 741.64 Kb

113.93 Kb of Joi
62.78 Kb of isemail (dependency of joi ?)
455.55 Kb of moment (definitely a dependency of Joi)
other small stuff

As I don't care about moment locales, I removed them with new webpack.IgnorePlugin(/^\.\/locale$/, /moment$/).
The final jsonwebtoken package size is 412.01 Kb

113.93 Kb of Joi
62.78 Kb of isemail (dependency of joi ?)
221.42Kb of moment (definitely a dependency of joi)
other small stuff

I do not know much about this project but it doesn't seem to need the full power of joi and moment. Joi is only used in sign.js and you could gain a lot by just getting rid of joi altogether.
That being said, it's just a suggestion because anyone could exclude unwanted dependencies with webpack, but it's a nice default to have.

I have had the same experience with Lambda so 👍 .
Sounds like we should get ride of Joi, use something smaller or maybe just manual validation.

you guys should check the repo made for exactly this purpose in: https://github.com/hapijs/joi/issues/528

@jfromaniello I've raised a PR (#348) which adds the manual validation. Again for me it's lambda performance which is driving this.

Solved in release 8.0.0


Add apply_grants call to materialization macros

resolves #366
Description
Add calls to 'apply_grants' to table, snapshot, and incremental materializations.
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt-spark next" section.

This is not complete. Needs to have create_or_replace_view macro in dbt-core updated. Waiting for adapter zone tests to pull in.

Thanks for opening the draft PR!
We will likely also need post-processing code to standardize the results returned by show grants, since the column names and contents will be specific to SparkSQL.
I believe that the default get_grant_sql + get_revoke_sql macros will work without changes, but the "adapter zone" tests will tell us for sure.

https://github.com/dbt-labs/dbt-spark/pull/389 gets this assertion in the dbt-core tests passing, by bringing seeds into line with expected / default behavior: truncate if not --full-refresh, such that grants are carried forward

changed pointer, to main, and added changelog


Implement support to get dependencies of branch/tags/commits

Fix #29

This would be awesome, because a lot of times I update dependencies in my develop branch and every so often merge into master.

:+1:

@aivus can you rebase this off the newest changes in master?

I'll rebase it ASAP

Fix behavior with default brabch and change caching for use branches.

Thanks @aivus I'll take a look as soon as I can :thumbsup:

LGTM :+1:

ping @alanshaw

Thanks, this is awesome!


Fixing bad_client test.

Also worth mentioning that with this PR, and #3249, we get this:

this is fantastic, thank you


add missing dependency on python3-dev to get python.h

Fixes: https://build.ros2.org/job/Fbin_uF64__maliput_dragway__ubuntu_focal_amd64__binary/1/console
17:54:34 In file included from /usr/include/pybind11/pytypes.h:12,
17:54:34                  from /usr/include/pybind11/cast.h:13,
17:54:34                  from /usr/include/pybind11/attr.h:13,
17:54:34                  from /usr/include/pybind11/pybind11.h:44,
17:54:34                  from /tmp/binarydeb/ros-foxy-maliput-dragway-0.1.0/src/bindings/dragway_py.cc:31:
17:54:34 /usr/include/pybind11/detail/common.h:112:10: fatal error: Python.h: No such file or directory
17:54:34   112 | #include <Python.h>
17:54:34       |          ^~~~~~~~~~
17:54:34 compilation terminated.

I will create a maliput_dragway release once this is merged.

I will create a maliput_dragway release once this is merged.

https://github.com/ros/rosdistro/pull/33643


Storyboard: Add ability to write and run Tests

Purpose
Users should have clear guidance on how to write and run tests. Adding writing and running tests in try.dot.net will encourage users to test and clarify good testing practices in an easily consumable browser experience.
Experience/Scenarios
General flow. (I'm very open to suggestions.)

Intro

User is able to see one test method, edit that test method (Change an Assert to passing), and run the test.
User can then view a product method and a test method covering it in the same window. The user must change the product code so the test passes. (Test is not editable.)
The user is given product code and asked to write a test to check the expected behavior. (Ex: this code should never return a negative number. Write a test to check for that.)
Tests help visualize behavior changes over time. An overloaded method was added to the above example. Ask the user to add another test for the new method.

Test structure

Explain the different types of tests: unit, integration, manual, performance (stress testing), and more.
Outline the basic ideal for unit testing with a 1-1 relationship of test classes to product code classes and test projects to product code projects.

Good test practices

Benefits of TDD. User receives a test and must write code to make it pass.
Code coverage doesn't necessarily mean good code. User must fix a test that passes that shouldn't have been passing. Writing the same logic twice is not good testing it's just more work.

Additional topics

PR flow. If a feature is being added, it is helpful to have a test documenting the expected behavior of that feature. It gives reviewers a good idea of expectations.

Outstanding Questions

Experience aside, should this issue focus on designing an API for allowing 2nd party users to extend try.dot.net to use tests?

Ideas copied from @jonsequitur:

For test samples, allow the run button to mean "run my tests".
Allow tests to be hidden and the implementation editable to teach test-driven problem solving.
Allow the tests to be visible and the implementation hidden to teach test-driven ways of exploring existing behaviors.
We would want a way to display test results that's a little more interesting than the current console output.

Adding @rowanmiller as we build out this scenario

Our scenario is a "Coding challenge". We would show a stubbed out method with some // TODOs to implement or fix the code. It would be great to define a test to verify the output.
We want to be able to give detailed feedback, so either multiple asserts with different messages we could show if they fail, or multiple unit tests.
It would be cool to have the unit test code available in a separate code listing (but not editable).

BTW this would also apply to our current In-Browser Tutorial - currently all the validation is done in JavaScript by string parsing the code and output.

@rowanmiller One approach that would work right now for having the test code available in a separate listing would be a multi-file Gist or a single-file Gist where the editable code is marked by a region directive. (These would be functionally equivalent.) Thoughts?

@jonsequitur sounds promising... you support that now? Gist's wouldn't be ideal, as the Playpen system would be driven by metadata and it would be cumbersome to put everything into Gists... but we could do it as a temporary solution until we can do it without Gists.

The code masking and multi-file support can be used without Gists. Sorry if that was unclear. The two features are usable independently.

Nice! Do you have an example?

On https://try.dot.net/docs take a look at the "Set source code from workspace" example.

I've been getting the t-rex global tool into shape to gather test output for display. Since we haven't had much discussion about custom UI for this scenario, I'll toss this out to generate some.
https://github.com/jonsequitur/t-rex

@jonsequitur once we've used setWorkspace, is there a way to reset the editor? Or send a setWorkspace message that will load up a snippet (without the full console app etc.)? Calling setCode (like we typically do for the snippets) doesn't work because (not surprisingly) it just puts the code in for the current buffer and leaves all the surrounding test harness code in place.
BTW we have the setWorkspace approach working for the test scenario, with a console app that functions as a test-harness and some well-known output that we parse to find the result and pass/fail message. Hacky... but it works 😄.

Hey all! Not sure where this stands but I was in the process of suggesting it when I found this issue.
I think this change in particular could have wide-reaching advantages across the ecosystem.
This idea occurred to me, for example, while browsing so many spots in the .NET docs where I see a call to Console.WriteLine and then // Will print "xyz".
I think this change has the opportunity to allow the the definition of docs examples in terms of unit tests / assertions, potentially even allowing the tests themselves to live in a repo that effectively regression-tests a lot of the surface area of .NET. It would be fantastic to enable a scenario where certain unit tests for functionality that describe the functionality also become real live docs. A great convergence. :)
I just wanted to offer my support for this as a focus for the group on the roadmap.
Thanks for all the work on this cool thing so far!

As an workaround, I think that I can make a console application that executes
dotnet test
However, it will be usefull if dotnet try can execute dotnet  directly

@SeanKilleen The scenario you outlined fits very much into our story for dotnet try. We'd love your thoughts. https://devblogs.microsoft.com/dotnet/creating-interactive-net-documentation/

@jonsequitur sorry it's taken me a while to get back to this. I think I may be missing some understanding of the tool, but after reading the article I'm still a little unclear.
My goal would be to have something along the lines of:
using xunit;
using FluentAssertions;

public class AdditionTests
{
   [Fact]
   public void AddingTwoNumbersTogether_ReturnsSum()
   {
      var sum = 2+2;
      sum.Should().Be(4);
   }
}

...and have try.net run the xunit test, verifying that it passes.
I'm failing to see a path to this in the current state, but if such a path exists, let me know and I'll see if maybe I can contribute some docs or a tutorial for it. Very excited by the potential still!

Reading back through this, my impression is that the desired goal is to enable the author of a tutorial to verify a learner's code and provide feedback based on whether it met some criteria. The use of a test framework seems like a means to an end but not critical to the experience. In fact, the output of a test run might be too limited for the tutorial author to provide a useful response. For example, you might want access to the actual code submission to make sure they solved a problem in an certain way.
@BillWagner


[GHSA-6c25-cxcc-pmc4] Dromara hutool vulnerable to SQL Injection

Updates

Affected products

Comments
Reported to OSV.dev via https://github.com/google/osv.dev/issues/2195

https://github.com/dromara/hutool/releases/tag/5.8.21 says CVE-2023-24163 was fixed in version 5.8.21, but this PR is adding "last_affected": "5.8.21" - possibly "fixed": "5.8.21" was intended?

@ElectricNroff thank you, yes, I was too quick to just edit the existing version number, without paying attention to how it was used. I don't think I have the ability to modify this PR, so hopefully the lovely GHAD folks can fix this up before they merge it.


Store folder conflict resolution

PR for store folder input to handle case where we want to read MVBS data together with score/mask products

Codecov Report
Attention: Patch coverage is 4.34783% with 44 lines in your changes missing coverage. Please review.

Project coverage is 34.79%. Comparing base (40bc57a) to head (0aed8ff).
Report is 36 commits behind head on main.

Files with missing lines
Patch %
Lines

...chodataflow/stages/subflows/initialization_flow.py
2.32%
42 Missing :warning:

echodataflow/utils/file_utils.py
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
+ Coverage   34.20%   34.79%   +0.58%
==========================================
  Files          45       45
  Lines        3613     3549      -64
==========================================
- Hits         1236     1235       -1
+ Misses       2377     2314      -63

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


feat: analyst dashboard backend

Implements # 639 backend

I believe this needs permissions for ccbc_admin as well.

Done! Awaiting another review


docs(website): add third-party tool in continuous integration section

Summary
I have created github actions that work with reviewdog to utilize the pull request commenting and commit suggestion functionality.
https://github.com/marketplace/actions/run-biome-with-reviewdog
https://github.com/mongolyy/reviewdog-action-biome
I believe this action will improve the usability of the combination of biome and github actions.
Test Plan
Check the deployment preview.

I have created a pull request, but if you would like to create an issue or discussion to discuss it, please let me know.

Can you give us more context? What does this action do more than Biome? The ci command already supports annotations on GitHub

Thanks for the response.
With annotation, the pull request is just commented and we have to open the IDE, github.dev, etc. to fix it.
I thought it would be useful to be able to modify the remarks from Biome with a few clicks on the github pull request screen.
See the sample pull request.
https://github.com/mongolyy/reviewdog-action-biome/pull/27
I made it so that commit suggestions are made based on the results of biome check --apply.
Of course, if something can't be safe fixed, it will be commented in the pull request.
I think this will make it easier for developers to fix things that are pointed out by biome.
Sorry if this is not in line with the intention of the biome development team.
In that case, I will close this pull request.

That's actually pretty neat! Amazing tool indeed! I'm happy to integrate it in our docs

Thank you. I am glad to hear you say so.
I am also happy to contribute to Biome.
I will continue to support Biome.


coprocessor: init count for dag test

@siddontang @hanfei1991 @huachaohuang @hicqu PTAL

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


PYTHONPATH needs setting else the app can't import forest

PYTHONPATH needs setting else the app can't import forest

This is now explained in the README.md


Passing validation when starting from scratch

Post merge of the explinter branch, I cannot seem to pass validation when starting from scratch with no instruments and no channels. The issue seems to be related to the physical channel -> AWG mapping, because after creating an AWG (e.g. and APS1) and then populating some physical channels, when I choose the APS1 as the AWG and then highlight another channel and return, the choice of AWG seems to vanish.

Okay, tracked it down.  It's a the classic reference binding vs copy constructor we've run into many times before.  If you add these lines to ChannelView.enaml [here]https://github.com/BBN-Q/PyQLab/blob/a72845e8266032a31c8b53cf24c843d8a045d391/QGL/ChannelsViews.enaml#L92)
            index ::
                setIfNotEmpty(chan.AWG, instrumentLib, selected_item)
                print("id of chan.AWG {}".format(id(chan.AWG)))
                print("id of instrument {}".format(id(instrumentLib[selected_item])))

you'll see they have different pointers.  We solved this before by passing higher level objects and just updating the member.
@bcdonovan this is probably from setIfNotEmpty.  Maybe you can see a clever way to keep the repeated code smell away and not run into this issue.

Fixed by #35.


fix(curriculum): rework Project Euler 74

Reworks challenge to use argument in function call.
Adds more tests
Changes notations using arrow to use MathJax, to make look consistent and keep it evenly spaced between paragraphs.
Related to #40896.
Tested on local fork.

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

I added this solution to the forum guide article


fix: electron build is broken

Description
It fixes the current broken electron build.
Task Item
#minor

Coverage remained the same at 54.819% when pulling ff7a88af7cd2d5c9da4fecd92d9be4924700c802 on sorgh/fix_build into bb3a8efffe5315e167600c68d046b3524ba62662 on main.

Coverage remained the same at 54.819% when pulling ff7a88af7cd2d5c9da4fecd92d9be4924700c802 on sorgh/fix_build into bb3a8efffe5315e167600c68d046b3524ba62662 on main.

@hatpick did you and @tonyanziano connect on his question? I am also curious.

@hatpick did you and @tonyanziano connect on his question? I am also curious.

@a-b-r-o-w-n yeah we chat on Teams, the electron-builder CLI requires --project or --projectDir to point to the directory of the electron project.

@a-b-r-o-w-n yeah we chat on Teams, the electron-builder CLI requires --project or --projectDir to point to the directory of the electron project.


Unable to use to log in

Duo push never shows up in the extension, I keep getting this error message: (I tried resetting and setting it up again and it didn't help)
"Failed to fetch @https://api-c03055b9.duosecurity.com/push/v2/device/transactions:
401 - Unauthorized
{"code":40102,"message":"Invalid pkey in request credentials","stat":"FAIL"}"

Did you already have Duochrome setup, and then approving pushes stopped working? Or are you trying to set it up for the first time and didn't use it before

I haven't used it before, I hasn't worked since I first downloaded it

That's odd. We've had issues in the past where the API has had "downtimes" and then magically decides to work again after a random interval of time. Unsure if this is one of those times and you're just incredibly unlucky, or maybe your organization uses a specific Duo configuration that needs support (this was built to support Ohio State Univ). I'm not able to debug this until later today but I really appreciate you letting me know.

Thanks for helping! I'm at UC Irvine, I found this on my school's reddit so I'm pretty sure it's working for other people at my school

Was able to create a new device just now.
Go ahead and try it again. Weird how it doesn't work for you but for others at the same university. If you're tech-savvy, you could monitor the network traffic and paste the results here so I can get a better look at what might be happening.

It's working now! Thanks

I'm also a UCI student, and I've been having the same issue. When initially setting up the extension, it works fine, but after a random interval - usually 1 to 2 days - the extension device gets removed.

Hi,
I originally thought this could relate to this issue where there was a computer time synchronization problem, but your device... it's just gone. I've never seen that before.
Devices don't just get removed like that. This random 1-2 day interval before the device is removed (not just stops working) could suggest that this is human activity - where perhaps someone from your IT department is sifting through devices with non-traditional model information (it's attached at activation) and removing them from student's accounts. For instance, this extension's model property is "Auto 2Fa". Usually, this reads something like iPhone 13, or Google Pixel, etc. This makes it an easy target.
Can you elaborate on this 1-2 day interval? For instance, does the device always get deleted during business hours, or do you receive an email when this happens? I'd like to try renaming the model property to pretend to be an iPhone if you're willing to test it.

Not sure of the exact time of day the device gets removed - I suspect it to be during business hours - but I don't receive any notification or email.
I do suspect it to be some manual action, as any devices that did use the Auto 2Fa extension were previously untouched; such devices only started to get removed in the aforementioned manner during the last 2 weeks.

Made a quick update to just Firefox, not Chrome yet. It now lies about the device information. When you activate Auto 2FA again, it should claim it's an iOS iPhone 13 instead of admitting it's an extension. Go ahead and manually update it, and please let me know if it gets deleted again or not.

Browsed through the UCI reddit - found a post where a student claimed that their Auto 2FA device was deleted thanks to their IT department.
I'm guessing the only way UCI would know it's not a supported device is by looking at the device information created at activation that says it's an extension (and not an iPhone, for instance). The solution is to randomly pick a device and platform (or just stick to a tablet) so they can't be confident it's an extension anymore. I could be wrong: they might have a better way to ensure it's an extension so still keep me updated, but due to this new information I'll be making a new update that adds this behavior.
Thanks for bringing this to my attention

@jach1337 any update? Does the device still get deleted from your account? Device information after the new patch initializes itself as a random iOS / Android tablet, so if the IT department really does deletes devices, it shouldn't happen anymore.

So far, the device hasn't been removed after a week, so I think it works.

Cool, I'll close the issue then. If it happens again let me know :)


Add support for circular dependencies

currently, I get the following error
Unhandled exception:
NoSuchMethodError: The getter 'reversed' was called on null.
Receiver: null
Tried calling: reversed
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
#1      main (file:///Users/smkhalsa/.pub-cache/hosted/pub.dartlang.org/multipack-0.1.3/bin/multipack.dart:10:69)
<asynchronous suspension>
#2      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#3      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)

when running multipack pubspec override if I have a circular dependency.

@klavs do you have any insight on this?

I do not know how to universally handle circular dependencies in a directed graph. I'd imagine user would have to define how to break the cycle so that we can figure out what's the topological order.
I think lerna handles circular dependencies, there might be hints in their source code.

https://github.com/huang12zheng/multipack

I do not know how to handle InfoCommand.

https://github.com/huang12zheng/multipack
I just had done a test with multipack pubspec override

test workflow

# step1
# file /Users/huangzheng/.pub-cache/bin/multipack
# change to
dart "/Users/huangzheng/frontend/multipack/bin/multipack.dart" "$@"
# step2
run `multipack pubspec override` in ferry

And, I don't know how to deal with InfoCommand


Request sql improvement

File: https://github.com/AmineZouitine/rmt.rs/blob/main/src/data_manager.rs
Some sql queries are not optimized, example :
pub fn find_trash_item_by_id(connection: &Connection, is_test: bool, id: i32) -> TrashItem {
    find_all_trash_items(connection, is_test)
        .into_iter()
        .find(|trash_item| trash_item.id == id)
        .unwrap()
}

While a where could directly give us the item we want, without retrieving all of them
Another example in the file https://github.com/AmineZouitine/rmt.rs/blob/main/src/input_manager.rs
line 29
        data_manager::find_all_trash_items(connection, is_test).len()

While a "get_element_count" function by spanking a select COUNT(*) could be faster

halo,
is this still available ?
i want to work in this issue

Yes it is still available, moreover it would be interesting to manage the errors better than what I currently do


Correct Dead Link

Refers and fixes #16922

Update dead link in documentation to correct link

@vedipen : Thanks for your contribution! The author, @markgalioto, has been notified to review your proposed change.

Merging broken link fix


Add instructions for automated PDF Draft upload

Based on my personal needs and looking at a couple of repos for papers recently published in JOSS, people in the community like to upload the generated PDF Drafts into their repos after they are generated, but they do it manually.
This small PR adds instructions to README on how to do it automatically within the draft-pdf.yml workflow.

Thank you for this, and sorry for the wait.


[WIP] Qiskit Optimization

Summary
Adding Qiskit Optimization with new problem representation as well as new algorithms and interfaces for Aqua core algorithms.
Open points:

[ ] merge optimization problem refactoring
[ ] rename optimization problem to quadratic program
[ ] add more tests on the converters
[ ] update ADMM

Details and comments
Add a new representation for optimization problems as well as converters to different classes and representations as possible.
Add a new interface and implementations for optimization algorithms that can leverage aqua core algorithms, such as VQE, QAOA, Grover, ...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 6 committers have signed the CLA.:white_check_mark: AustinGilliam:white_check_mark: stefan-woerner:white_check_mark: Cryoris:white_check_mark: t-imamichi:x: ClaudioG-drl:x: adekusar-drlYou have signed the CLA already but the status is still pending? Let us recheck it.

Can we add test case for operator_to_quadratic_program. I see in the coverage result that this is not covered by the unit testing.

@t-imamichi Thanks for pointing this out, fixed in 4253c89.

WOOHOOO!!!! 🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉🎉


Piano

-------------TEST PLAN---------------
https://highfidelity.testrail.net/index.php?/cases/view/12742
https://highfidelity.testrail.net/index.php?/cases/view/12745

Up on marketplace, passed marketplace QA


praw.helpers comment and submission streams are slow to start.

I'm using multiple subreddits (19 at the moment) and it takes over 10 minutes for the first comment or submission to be yielded.

What limit are you using?

No limit set, so the default 1000.

Try setting the limit to 1000 (or lower) explicitly. Each subreddit in a multiple listing actually enables another thousand so it'll have to go through 10 requests for each thousand (190 total) it'll take at least 380 seconds to get the initial listing.


Add test to ensure all flows are visualisable

We've gone to quite some effort to ensure that the graph visualisation
continues to work while we've been refactoring Smart Answers. I'm adding this
test to ensure that we don't undo that work in future.
This was highlighted by a proposed refactoring of benefit-cap-calculator (see
PR #2452) that results in the flow not being visualisable.
Assuming this commit, and the proposed benefit-cap-calculator refactoring make
it into master, we'll have to work out how to handle the broken visualisation.

I think it would be better to add a card to the backlog to fix visualise to handle flows with repeating questions rather than adding a test that could delay the progress of the benefit-cap-calculator update that needs to be live on April 22nd and that restrains our ability to write code in succinct and configurable way.

Hi @chrisroos, thank you for work on this PR. I think it is a good idea to detect automatically when the graph visualization has been broken.
But, as a note, I would wait before merging this PR, and this is the reason:
We are discussing with the content team about which flows visualisation are useful for us. Some of them, like the Benefit cap calculator, is not being very helpful because it's too complex.
So what about adding a list of exceptions to the test? What are your thoughts?

I think it would be better to add a card to the backlog to fix visualise to handle flows with repeating questions rather than adding a test that could delay the progress of the benefit-cap-calculator update that needs to be live on April 22nd and that restrains our ability to write code in succinct and configurable way.

I don't think this test prevents the changes in PR #2452 being merged. We can easily update this test to add an explicit exclusion for benefit-cap-calculator so that flow is skipped. The benefit of having the test for all other flows is that it'll alert us if we accidentally break the visualisation of other flows in future.

We are discussing with the content team about which flows visualisation are useful for us. Some of them, like the Benefit cap calculator, is not being very helpful because it's too complex.

This discussion sounds good. I'm sure there are other flows where the visualisation is too complicated for it to be useful.

So what about adding a list of exceptions to the test? What are your thoughts?

Adding exceptions to the test sounds good to me. Although I'd only add them at the point the visualisation is broken (i.e. when this test starts failing). That way we'll be prompted to question whether or not the visualisation was useful on a flow by flow basis.

My vote would be to get this test merged and then to add exceptions if/when they're required.

@chrisroos agree with the exception on the flow by flow basis.
LGTM 👍

Thanks, @pmanrubia!


Changed Icon to Widget

activeIcon and inactiveIcon expects icons of Icon class.
However, I need icons that are images. Therefore, I changed it to expect a widget.
Tested and worked for a square image (128x128) containing a circular icon. (with rounded edges cut off)

Closes as feature has been implemented


docs: 修复官网demo分群下钻表颜色丢失问题

👀 PR includes

🔧 Chore

[ ] Test case
[x] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

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


Move ResurrectionService to the CrashModule in the Features layer

Goal
The PayloadResurrectionService, for now, depends on the NativeCrashService, which exists in the Features Layer, so I'm moving it there too (in the CrashModule, as it is resurrecting payloads generated sometimes by native crashes) so it can have access to it.
Testing
Change module tests to verify that the service is only initialized if the v2 feature flag is on.

#1498
#1497  👈
main

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

Join @bidetofevil and the rest of your teammates on  Graphite

Yeah this felt odd to me too tbh. The session resurrection is definitely a core feature - what is a feature feature is the native crash bit, finding, sending, and pending said data to the the session. That said, the way we treat JVM crashes and their ID (and handle them in the session logic) makes that seem like it's part of core as well.
Let me see what I can do. At worst, I'll most things back into core and allow the service to be provided at runtime when we do the resurrection. Not a huge fan of that but I think it beats this.

Merge activity

Oct 11, 3:55 PM EDT: A user started a stack merge that includes this pull request via Graphite.


Do not unnecessarily execute GetName() method

Because GetName() executes the localization logic, but we may not need it actually.

@dougbu  - thoughts?

We use GetName() to ensure we get the correct (localized) value. Our localization system is a fallback and does not override what users have done with ResourceType.
See also the first sentence in https://docs.microsoft.com/en-ca/dotnet/api/system.componentmodel.dataannotations.displayattribute.name?view=netframework-4.7.1#Remarks

Do not use this property to get the value of the Name property. Use the GetName method instead.

I definitely understand why you use GetName here: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsMetadataProvider.cs#L143
However, checking Name should be enough here: https://github.com/aspnet/Mvc/blob/dev/src/Microsoft.AspNetCore.Mvc.DataAnnotations/Internal/DataAnnotationsMetadataProvider.cs#L133
Because the first one does not use the return value of GetName(), it only compares with null.
If you accept my PR, then it will not unnecessarily call GetName (and it's subcalls) to only check if it's Name property is set.
But, finally it's your decision. Thanks.

@hikalkan what exactly would your PR fix? GetName() builds on caching done in LocalizedString and does not have a significant performance impact even on first run. Further, using Name instead of GetName() in the null-check you mentioned would miss cases where LocalizedString falls back.

OK. Then we can close this. I just saw it while exploring MVC source code to understand how it works.
Thank you for your time.


fixdescription of import error in broker-load-manual

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buidlall


Pycuda install error: command 'gcc' failed with exit status 1

Hi, thank you for making an amazing library.
I am trying to use your library for my research. However, I have been facing errors while I was trying to install pycuda.
I configure like this
python configure.py --cuda-enable-gl --cuda-root=/usr/local/cuda-10.0
and when I try to compile and install by "make install" ,
warning: no files found matching 'doc/source/_static/*.css'
warning: no files found matching 'doc/source/_templates/*.html'
warning: no files found matching '*.cpp' under directory 'bpl-subset/bpl_subset/boost'
warning: no files found matching '*.html' under directory 'bpl-subset/bpl_subset/boost'
warning: no files found matching '*.inl' under directory 'bpl-subset/bpl_subset/boost'
warning: no files found matching '*.txt' under directory 'bpl-subset/bpl_subset/boost'
warning: no files found matching '*.h' under directory 'bpl-subset/bpl_subset/libs'
warning: no files found matching '*.ipp' under directory 'bpl-subset/bpl_subset/libs'
warning: no files found matching '*.pl' under directory 'bpl-subset/bpl_subset/libs'
writing manifest file 'pycuda.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
running build_ext
building '_driver' extension
gcc -pthread -B /home/orthoinsight/anaconda3/envs/ganspace/compiler_compat -Wl,--sysroot=/ -Wsign-compare -fwrapv -Wall -O3 -DNDEBUG -fPIC -DBOOST_ALL_NO_LIB=1 -DBOOST_THREAD_BUILD_DLL=1 -DBOOST_MULTI_INDEX_DISABLE_SERIALIZATION=1 -DBOOST_PYTHON_SOURCE=1 -Dboost=pycudaboost -DBOOST_THREAD_DONT_USE_CHRONO=1 -DPYGPU_PACKAGE=pycuda -DPYGPU_PYCUDA=1 -DHAVE_GL=1 -DHAVE_CURAND=1 -Isrc/cpp -Ibpl-subset/bpl_subset -I/usr/local/include -I/home/orthoinsight/anaconda3/envs/ganspace/lib/python3.7/site-packages/numpy/core/include -I/home/orthoinsight/anaconda3/envs/ganspace/include/python3.7m -c src/cpp/cuda.cpp -o build/temp.linux-x86_64-3.7/src/cpp/cuda.o
gcc: error trying to exec 'cc1plus': execvp: No such file or directory
error: command 'gcc' failed with exit status 1
Makefile:12: recipe for target 'install' failed
make: *** [install] Error 1

I have been facing that kind of error. Can you please help me with that?
Thank you
Best Regards,
Gun Ahn

Yes, I also encountered this problem. Have you solved it now?

Me too! Desperate for a solution...


Is there no way to change the url where images can be reached?

I'm seeing the built in template tags always point to MEDIA_URL.  Can we define a static location to point to like http://www.testing.com/newDirectory/avatar/user/65/image.png?

MEDIA_URL is the folder where static user files are placed. Please reopen if there is a specific reason to do something else.


Unable to add datasource

I just installed version 0.1.5 of the plugin into my grafana (version 8.5.0).
Then I go in and try to add a datasource, the main page comes up fine, I enter a datasource name and the url, but when I select the "auth type" dropdown and select anything other than none, I immediately get this error:
DetailsError: Minified React error #130; visit https://reactjs.org/docs/error-decoder.html?invariant=130&args[]=undefined&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
Are there minimum versions of grafana that need to be used with this plugin?
Thanks

Hi,
Yes the minimum Grafana version required is 9.2.5 due to certain grafana packages we are using in the ui.
This is in our plugin.json but seems to have been missed out from the read me I'll make sure to update.
Please reopen this if you have any other issues!


CocoaPods could not find compatible versions for pod "FBSDKCoreKit" (part#2)

I'm having this issue: https://github.com/thebergamo/react-native-fbsdk-next/issues/20
@mikehardy has provided a possible solution how ever he has not given me enough details to work out how to get things working, so I'm starting a new thread as that one is locked.
I have tried updating the react-native-fbsdk-next.podspec to: https://pastebin.com/8BDx9r84
but I am still getting the following errors: https://pastebin.com/X8yZbpCh
I also tried changing the version on the Podfile.lock to 9.2.0.
Can any one provide more information on how to resolve this issue?

Managing your dependencies is not the responsibility of this module.
You have this module which is depending one version.
If you delete Podfile.lock and run pod install and continue to get the message, you have some other module depending on some other version, and the error message tells you exactly what, it's some thing that requires "FacebookLogin"
You must fix that in your own project.


Fixes timeleft estimate which was returning negative estimates

Problem: The now was assumed to be higher than the executionWindowEnd or reservedWindowEnd when in reality it is lower.
Fix: Switch around calculations.

Pull Request Test Coverage Report for Build 2147

0 of 0   changed or added relevant lines in 0 files are covered.
29 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-2.9%) to 73.524%

Files with Coverage Reduction
New Missed Lines
%

src/TxPool/TxPoolProcessor.ts
1
86.11%

src/Scanner/WatchableBucket.ts
1
87.1%

src/GasEstimation/EthGasStationFetchingService.ts
1
85.71%

src/EconomicStrategy/EconomicStrategyManager.ts
1
67.48%

src/TxPool/TxPool.ts
2
78.85%

src/Util.ts
3
75.21%

src/TimeNode.ts
3
95.16%

src/WsReconnect/index.ts
17
31.34%

Totals

Change from base Build 2145:
-2.9%

Covered Lines:
860

Relevant Lines:
1112

💛  - Coveralls

Pull Request Test Coverage Report for Build 2147

0 of 0   changed or added relevant lines in 0 files are covered.
29 unchanged lines in 8 files lost coverage.
Overall coverage decreased (-2.9%) to 73.524%

Files with Coverage Reduction
New Missed Lines
%

src/TxPool/TxPoolProcessor.ts
1
86.11%

src/Scanner/WatchableBucket.ts
1
87.1%

src/GasEstimation/EthGasStationFetchingService.ts
1
85.71%

src/EconomicStrategy/EconomicStrategyManager.ts
1
67.48%

src/TxPool/TxPool.ts
2
78.85%

src/Util.ts
3
75.21%

src/TimeNode.ts
3
95.16%

src/WsReconnect/index.ts
17
31.34%

Totals

Change from base Build 2145:
-2.9%

Covered Lines:
860

Relevant Lines:
1112

💛  - Coveralls