djvu2pdf: use resholvePackage

@jwiegley opening this as a draft to start a conversation (I realize you aren't the listed maintainer--feel free to dodge. Just guessing at who'd care, and I see you added the expression.)
This PR repackages djvu2pdf with resholve, a tool I develop to improve Shell packaging in Nixpkgs. resholve identifies unspecified dependencies, and rewrites external invocations to absolute paths so that end-users of Shell projects don't need to manually add dependencies to their system/user packages.
I'm hoping to get to a simple yes/no on whether you're okay with the change (and confirmation that it isn't breaking any of your own uses, if you have some?), so I'm happy to field any questions you have to help you decide.
(If we move forward, I'm happy to pick at any style/format/organization complaints you have :)
For a little additional context, I posted a gist of the ~built script

Motivation for this change
Convert djvu2pdf to use resholvePackage.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

I've force-pushed to track a rename in the resholve API (resholvePackage -> resholve.mkDerivation) that got merged last night.


release GraphQL

Coverage increased (+0.2%) to 88.814% when pulling 1cd815d27b3937c29fa3ed61021cf3eb2fa0c22b on release-graphql into 7a8675ee2975d0421f528383a9bc01dafc9fe10d on preprod.


Don't run facet suggstion calculations unnecessarily

Right now any time we use TableView() we trigger facet suggestion code which we're not actually using.
https://github.com/simonw/datasette-graphql/blob/6fc8fbba55c8049e5be7aa6a2744891219586ce1/datasette_graphql/utils.py#L215-L219
The easiest way to turn this off is to ensure that the Datasette suggest_facets config option is considered off for the duration of those calls.

https://wrapt.readthedocs.io/en/latest/wrappers.html has an ObjectProxy class that could help here. We want ds.config("suggest_facets") to return False but only within our call to TableView.data().


send login failed back to client?

after each failed attempt I want to write back to the client
"Login incorrect"
where do I put this?

Please take a look at: https://thinkincredible.intraway.com/2016/02/05/implementing-python-telnet-server-testing-development/
There you'll see:
def authCallback(self, username, password):
    if username != “admin” or password != “admin”: #Security comes first
        self.writeresponse(“Invalid username/password”)

My guess is that you should put something similar into the class HoneyTelnetHandler(TelnetHandler): inside hontel.py

ok I set it too

    def authCallback(self, username, password):
        if username is not None and password is not None:
            self._log("AUTH", "{}:{}" .format(username, password))

        if not(username == AUTH_USERNAME and password == AUTH_PASSWORD):
            self.writeline("login incorrect")
            raise Exception("[x] wrong credentials ('{}':'{}')".format(username, password))

seems to get the job done just fine

I have updated your hontel into python 3 and os independant so it runs on windows as well as posix will fork soon


Add Opus ETH Staking App by Chorus One to Safe Store

Description

name: List Opus ETH Staking App request
about: Add a Safe app
Type

New addition

Revision checks

[x] Used smart contracts were audited.
[x]  Your Safe App include a manifest.json file at the root with the required data.
[x] The app can be loaded as a custom Safe App in the Apps section of https://app.safe.global/.
[x]  The app auto-connects to the Safe.
[x] You are able to trigger and execute one transaction with a Safe.
[x] RPC requests are optimized (not triggering many requests in a very short time period).

Name/Description
Name: ETH Staking by Chorus One
Description: Stake ETH with Chorus One's MEV Max Pool and earn maximum MEV rewards with enterprise-grade security. Get 100% slashing protection with full custody of your assets.
Compatible Networks

Mainnet

Code for review

Code is closed-sourced. We can grant access to devs for review if needed

Audit document
We're using the following smart contracts Stakewise Smart Contracts under the hood which have been audited.

Contracts
Contract Code URL
Audit Report

Stakewise smart contracts
https://github.com/stakewise/v3-core/tree/main/contracts
https://github.com/stakewise/v3-core/tree/main/audits

Eigenlayer smart contracts
https://github.com/Layr-Labs/eigenlayer-contracts
https://docs.eigenlayer.xyz/eigenlayer/security/audits

App URL

Link to app: https://opus.chorus.one/pool/stake
Link to company website: https://chorus.one/

Team information

Project logo: https://opus.chorus.one/portal/_next/static/media/opus_logo.73ea0090.svg
Project landing page: https://opus.chorus.one/pool/stake
Team or discord contact: https://discord.gg/bQ6JWVa
Twitter link: https://twitter.com/ChorusOne
Github link: https://github.com/ChorusOne

Hey, this is not the repo for the apps submission.
Please follow this guide for the process


Don't assume headers.host is set, fixes #16

Returns forbidden, since this is most likely a malicious attempt.

Ping

Thanks @knownasilya


drivers: spi: spi_sam0: fix spi_sam0_fast_txrx()

The optimisation in spi_sam0_fast_txrx() is broken, loading two bytes into the DATA register in rapid succession will lose one byte.
This can be observed by running tests/drivers/spi/spi_loopback.
The test will get stuck in spi_sam0_fast_txrx() forever waiting for the final byte.
Undo this small optimisation and only load the next byte into the DATA register after the response has been received.
This fixes tests/drivers/spi/spi_loopback for atsame54_xpro. (On this board connect PB27 and PB29 for the loopback test)

I will test this tomorrow on SAM D21 and E54.


Make Checkout page responsive

Hello Contributors, Can anyone make the checkout page responsive? I am using bootstrap for designing.

Tip : remove container class

Assign me I will work on it

Sure @swapn652, I have assigned you the issue, you may start working on it

I want to work on this issue..Do assign me.


Add contribution documents

Description
Adds contribution guidelines and code of conduct.

[x] CODE_OF_CONDUCT.md
[x] CONTRIBUTING.md

Checklist

[ ] I have manually tested my changes and I am happy with the result.
[ ] The documentation reflects the changes.
[ ] The PR branch is set up to merge into dev instead of master.
[ ] I haven't introduced breaking changes that would disrupt existing jobs, i.e. backwards compatibility is maintained.
[ ] In case I changed the dependencies in any requirements.in I have run pip-compile to update the corresponding requirements.txt.
[ ] In case I changed one of the services' models.py I have performed the appropriate database migrations (refer to the DB migration docs).
[ ] In case I changed code in the orchest-sdk I followed its release checklist.
[ ] In case I changed code in the orchest-cli I followed its release checklist.

Turns out we already have a contributing document inside docs/source/development, so for now I just converted it to .md, hoping that GitHub picks it up. If it doesn't, I'll add another one on the top level. We have to see this merged in master though.


Bower and Browser support

Coverage remained the same at 100.0% when pulling 994e49ffc5d70318acc5e647bec3e0f02090efdc on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling 994e49ffc5d70318acc5e647bec3e0f02090efdc on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling e5a1e7ddc622ce9be7e4bddc0027b304b9db1211 on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling 3444eee776ec4be6998793e28f39c6c317a4abcb on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling 3444eee776ec4be6998793e28f39c6c317a4abcb on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling 3444eee776ec4be6998793e28f39c6c317a4abcb on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

Coverage remained the same at 100.0% when pulling 3444eee776ec4be6998793e28f39c6c317a4abcb on feature/Bower_Support into 4baf83916656e72950806a54654ec59d81d81b9a on master.

No more relevant as Bower has been deprecated


Formally deprecate *_METADATA access log command operators

Title: Formally deprecate *_METADATA access log command operators
Description:
The documentation for the command operators for access log formatting DYNAMIC_METADATA and CLUSTER_METADATA has indicated that these will be deprecated in favor of the METADATA formatter since the latter was implemented. We should actually deprecate these and make plans for their removal.
Relevant Links:
https://github.com/envoyproxy/envoy/pull/17457/files#diff-a5eb61dd3a95c605846ebaba934b7334bc18f96fbf9ebf7593a90948c92f19c5

@cpakulski is this actively in progress or should we tag it help-wanted

@jbohanon Thanks for reminding. It will be done for the next release.


Crash OutOfBounds Mekanism Energy Cube

theoneprobe-1.12-1.4.20
https://gist.github.com/Mohron/65edb153226a151aa2d02224c94fa343
Looked at a Basic Mekanism Energy Cube to cause the crash.

Will be fixed in the next release. In the meantime, you can work around this by editing theoneprobe.cfg changing rfFormat to any of the choices except for 1 (compact).


Update jawn-fs2 to 0.15.0

Updates org.http4s:jawn-fs2 from 0.14.2 to 0.15.0.
Release Notes/Changelog
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "jawn-fs2" } ]

labels: semver-minor

addressed in #4470


Download books and extras into separate directories

Short description
If path.separate is set to true in the configuration file, downloads books and extras into separate directories.
Examples
If path.separate is set to true, path.ebooks=ebooks and path.extras=extras, ebooks and extras are downloaded to the following locations:
ebooks/TITLE - AUTHOR/TITLE.pdf
ebooks/TITLE - AUTHOR/extras/TITLE.png
Note that if you use separate directories, path.extras behaves differently:
If path.separate is set to false, path.ebooks=ebooks and path.extras=ebooks/extras, ebooks and extras are downloaded to the following locations:
ebooks/TITLE.pdf
ebooks/extras/TITLE.png
Disclaimer
The code is only a suggestion (preview) of new functionality, but works properly with local downloads. Other features to check (possible complications with Google Drive upload). Instead of setting this in the configuration file, separate or not directory can be changed by additional option from the command line.

Ok, I will try to do it in your my time, but can not promise when.

Ok, I will try to do this in my free time, but I can not promise when.

Ok, I will try to do this in my free time, but I can not promise when exactly.

No problem. Let me know if you have any other idea or suggestion.
Thanks

Thanks for your contribution!


Can not create L2 guest network through UI

ISSUE TYPE

Bug Report

COMPONENT NAME
UI / Guest Networks

CLOUDSTACK VERSION
master / 4.12

CONFIGURATION
N/A
OS / ENVIRONMENT

Advanced Networking
Ubuntu 18.04

SUMMARY
When trying to create a Layer 2 Guest Network through the UI you get a message that it the zoneId is not properly set.
STEPS TO REPRODUCE

Go to Guest Networks
Click on 'Create L2 Guest Network'
This fails

EXPECTED RESULTS

ACTUAL RESULTS
2018-12-05 17:31:36,776 DEBUG [c.c.a.ApiServlet] (qtp1096283470-16:ctx-fb8e8231) (logid:efaf623e) ===START===  2a00:XXXX:XXXX:0:0:f57a:c43b:22 -- GET  command=listNetworkOfferings&response=json&zoneid=&guestiptype=L2&state=Enabled&forVpc=false&_=1544027496695
2018-12-05 17:31:36,787 DEBUG [c.c.a.d.ParamProcessWorker] (qtp1096283470-16:ctx-fb8e8231 ctx-50704778) (logid:efaf623e) Object entity uuid =  does not exist in the database.
2018-12-05 17:31:36,787 INFO  [c.c.a.ApiServer] (qtp1096283470-16:ctx-fb8e8231 ctx-50704778) (logid:efaf623e) Unable to execute API command listnetworkofferings due to invalid value. Invalid parameter zoneid value= due to incorrect long value format, or entity does not exist or due to incorrect parameter annotation for the field in api cmd class.
2018-12-05 17:31:36,788 DEBUG [c.c.a.ApiServlet] (qtp1096283470-16:ctx-fb8e8231 ctx-50704778) (logid:efaf623e) ===END===  2a00:XXXX:XXXX:0:0:f57a:c43b:22 -- GET  command=listNetworkOfferings&response=json&zoneid=&guestiptype=L2&state=Enabled&forVpc=false&_=1544027496695

As you can see, the zoneId parameter is not set.

                                var zones = $.grep(json.listzonesresponse.zone, function(zone) {
                                    return (zone.networktype == 'Advanced' && zone.securitygroupsenabled != true); //Isolated networks can only be created in Advanced SG-disabled zone (but not in Basic zone nor$
                                });

In sharedFunctions.js
This filters away zones with Security Grouping enabled and thus creates this error.
Why can't we create a L2 network in a zone where SG is enabled on other networks?

likely a bug - @nvazquez can you comment?


multiple invokings in the same session makes response dirty

I unset response to prevent contamination on multpile invokings in the same session. Not sure if this is the best way to do it but it is working for my case.

Not sure it's actually in issue or one worth solving. I changed my API so I didn't need multiple calls. Was probably doing something weird.


No /dev/ttyUSB0 in HassOS for deCONZ FTDI usb dongle

I tried to setup HassOS in Proxmox (KVM). Almost everything works as expected, HassOS boots and Home Assistant is running.
However, /dev/ttyUSB0 is not found in HassOS.
I think the drivers for the deCONZ USB dongle are not loaded. The device is found in the VM:
dmesg | grep FTDI
[    3.214149] usb 3-1: Manufacturer: FTDI

lsusb
...
Bus 003 Device 002: ID 0403:6015

In my original HassIO generic install on Ubuntu Server the folowing drivers where loaded for this device:
usbserial
ftdi_sio

dmesg | grep FTDI
[    2.418960] usb 1-6: Manufacturer: FTDI
[    3.904772] usbserial: USB Serial support registered for FTDI USB Serial Device
[    3.904858] ftdi_sio 1-6:1.0: FTDI USB Serial Device converter detected
[    3.908125] usb 1-6: FTDI USB Serial Device converter now attached to ttyUSB0

lsusb
...
Bus 001 Device 004: ID 0403:6015 Future Technology Devices International, Ltd Bridge(I2C/SPI/UART/FIFO)

How do I get this working in HassOS?

Make a PR with needed kernel configs: https://github.com/home-assistant/hassos/blob/dev/buildroot-external/board/ova/kernel.config


Renaming Event.fee_paid to Event.invoiced.

It isn't Amy's job to track whether money has arrived - that's the job of NumFOCUS's accounting system.  Instead, we should track whether an invoice has been sent, i.e., have we handed responsibility off to NumFOCUS.  This change of name makes those semantics clearer.

We'll have a migration conflict in #303, but that's not a big deal.
From what I can see, this PR looks good.
My suggestion is to split future PRs into more commits.


codeunit 5632 "FA Jnl.-Post Line"

You forgot to replace the "Get" with the function "GetPostingGroup".

GitHub issue #6254

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Memory leak found w/ AudioStreamPlayer, but not sure why / how it's happening

Godot version:
3.0, December 27th compiled. Windows 7 64-bit, GTX 950
Spent all day trying to isolate why the memory kept increasing by around 2MB.
Project:
Juan-Godot-AudioStreamBugger.zip
Things to check out:

While scene is running, check the debugger's Monitor tab and the memory usage will increase by the filesize of the .ogg file, every 2 seconds.

Open fakelevel.tscn, and clear the BeginnerForest.ogg from the AudioStreamPlayer, no memory is leaked.

Some .ogg files will cause the leakage, and some won't, try different ones

The solution found was removing the yield in ChangeLevel. However, you would then need to keep track of the node name in a variable since the new scene name generated will contain the unique @ symbols. Unfortunately still extremely baffled on what is causing the leakage
Is it possible to yield for a finished queue_free() for a node? Am officially lost O_O lol :baby:

Is it okay if I re-post this issue?  I isolated the bug better and this is a confusing report, sorry about that.


Blog app - Views

Blog app - Views
In this branch, I:

Hello @noelfoka could you check your slack?


Change title and fix errors

Fixes #27
Also fixes two errors:

air quality data was making call to HTTP link, but needs to be HTTPS. Call to OpenWeatherMap was failing because of this.
5-day forecast stopped working because of a bug trying to access 5 consecutive days in a row. We were incrementing the day by 1, but came across an error when we hit the end of the month. For example, when accessing the data on April 28th, we have to account for the sequence of days to go from 28 – 29 – 30 – 1 – 2, rather than 28 - 29 - 30 - 31 - 32.

LGTM!


Windr integration phase 1

[x]  #6
[x] #7
[x] #9
[x] #10. (for now using GHCR)
[x] #13
[x] #12
[x] #11

Analyser work & configuration:

[x] #14

Great work, thanks!
I have tested it with a gpx file (not extensively though) and it looks good. I also reviewed the code, (I'd be interested in github workflow explanations ;) ) and it sounds good. Good to see that the yaml config file was clear enough you could make change to it. However I would not recommend to g=have vmin = 0 for the jibe analysis, I had difficulties and 12knots is  kind of the min speed it can support if you want reliable results.
Also I will write a windr json method in utils to clean up the dash_server code.
PR accepted thanks :)


increase test coverage of runtime package

Release note:

NONE

This change is

I0914 18:25:57.803] ok  	istio.io/mixer/pkg/runtime	1.681s	coverage: 94.5% of statements

From codecov tests

/lgtm


”Rusca Montană” is listed in the drop-down menu

Description:
"Rusca Montană" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Rusca Montană" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Rusca Montană, CARAȘ-SEVERIN".


PEP8ize and PEP20ize

Number of stylistic and few code changes required to conform to PEP8. While we are at it, here is PEP20:

Beautiful is better than ugly.
Explicit is better than implicit.
Simple is better than complex.
Complex is better than complicated.
Flat is better than nested.
Sparse is better than dense.
Readability counts.
Special cases aren't special enough to break the rules.
Although practicality beats purity.
Errors should never pass silently.
Unless explicitly silenced.
In the face of ambiguity, refuse the temptation to guess.
There should be one-- and preferably only one --obvious way to do it.
Although that way may not be obvious at first unless you're Dutch.
Now is better than never.
Although never is often better than right now.
If the implementation is hard to explain, it's a bad idea.
If the implementation is easy to explain, it may be a good idea.
Namespaces are one honking great idea -- let's do more of those!

Detailed review to follow...

Assigning to myself

The above list is very good. How do you suggest we best implement this? I guess one would be to treat it as a checklist and actually review the code against it, make sure we pass with everything and then after that have some kind of a "release form" associated with push where certain boxes need to be checked before the push can be made. Or we could pick the ones we can to some extent automate, and just put those in to our commit test?

Adding the pep20 to contribution guidelines. Code should be now 100% pep8 compliant, and we're keeping it that way!. I recommend using atom with linter for this purpose, make the whole thing far less painful than one would think. Thanks and closing :)


[test] Modernize AllocSpeedTest

Codecov Report

Merging #2156 (67ff9ff) into master (6461497) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 67ff9ff differs from pull request most recent head e6b23c0. Consider uploading reports for the commit e6b23c0 to get more accurate results

@@           Coverage Diff           @@
##           master    #2156   +/-   ##
=======================================
  Coverage   86.29%   86.29%
=======================================
  Files          53       53
  Lines        9571     9571
=======================================
  Hits         8259     8259
  Misses       1312     1312

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 6461497...e6b23c0. Read the comment docs.


Fix broken headings in Markdown files

GitHub changed the way Markdown headings are parsed, so this change fixes it.
See bryant1410/readmesfix for more information.
Tackles bryant1410/readmesfix#1

Coverage remained the same at 100.0% when pulling 50921d11d09c3ead9552962d920e6e91e834eff4 on bryant1410:master into d6f6e2e64394ba32becb3b307170f97be70863ae on badges:master.


feat:support time slice and add test

Signed-off-by: MrSumeng<EMAIL_ADDRESS>
Ⅰ. Describe what this PR does
Support time slice. Users can set time slices to allow rollout to be executed , if the step not in slices , it will be run at near time when it allowed.
Ⅱ. Does this pull request fix one issue?

fixes #74
Ⅲ. Special notes for reviews
User can customize the time zone. The local time zone will be used , if it is not set .

Codecov Report

Merging #80 (98b9e4d) into master (e1ba1b0) will increase coverage by 0.85%.
The diff coverage is 60.29%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   16.11%   16.96%   +0.85%
==========================================
  Files          30       31       +1
  Lines        3482     3549      +67
==========================================
+ Hits          561      602      +41
- Misses       2821     2843      +22
- Partials      100      104       +4

Flag
Coverage Δ

unittests
16.96% <60.29%> (+0.85%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/controller/rollout/canary.go
0.00% <0.00%> (ø)

pkg/controller/rollout/context.go
0.00% <0.00%> (ø)

pkg/util/time_utils.go
72.22% <72.22%> (ø)

...ollout/validating/rollout_create_update_handler.go
62.59% <93.75%> (+4.33%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


cache-first-then-network link on Step 7 leads to cache-network-race

When I click on the link I'm directed to https://jakearchibald.com/2014/offline-cookbook/#cache-network-race
Was the intent to lead to #cache-then-network ?

Hi!
Thanks for the feedback. I've rewritten the codelab from scratch and believe I have fixed this issue. The code has been updated, and the new guide is now live.
If you find any issues with the new codelab, please let us know!
Thanks!


Managing Custom domain page > "Configuring a subdomain" > second example undifferentiated from first example

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/pages/configuring-a-custom-domain-for-your-github-pages-site/managing-a-custom-domain-for-your-github-pages-site
What part(s) of the article would you like to see updated?
Current version of the "Configuring a subdomain" section has two different www subdomain examples, but I think the intention was for second subdomain example to be a non-www case. Looks like it was supposed to be another used as the subdomain in the second example, but instead the domain name was changed instead (www.example.com becoming www.anotherexample.com).
Additional information
I've got a PR to fix this that will come hot on the heels of this issue (since the preference is to file an issue for every PR).

@vincent-czi Thanks for opening and issue and submitting a PR! ✨


Add fixes to the test suits according to jerad's changes

Changes done according to the https://github.com/wso2/carbon-dashboards/pull/342


Fuzzy search is not supported in run

Microsoft PowerToys version
0.72.0
Installation method
Scoop
Running as admin
Yes
Area(s) with issue?
PowerToys Run
Steps to reproduce
input "vsc"
✔ Expected Behavior
visual studio code is in the list like Wox
❌ Actual Behavior
only shows the web search
Other Software
No response

even searching for things where there is a word in between doesn't yield any results.
for example, when I have links like
"ER Prod 1"
"ER Prod 2",
.....
typing "ER 2" shows no results.
Visual studio code has a perfect example of searching by partial match. You could potentially use that approach.

/dup #20672


Ask for keyed-for loop plan

Hi, thank you for the library
I want to ask is there any plan for support key in "each" directive ?
Or even have a separate directive like ":each-key", depends on purpose (rendering sorting table)
Thanks!

The rendering algorithm of sprae doesn't require key, since it maps array ops to dom directly. Do you have an example where it fails to render properly?

The point is sprae doesn't use any dom diffing/swapping algo, because it directly maps list to DOM. In other words, if you remove or insert any elements to list - they're immediately reflected in DOM as is, in other words elements in array are tightly coupled with DOM elements. So there's not supposed to be situation when, say, we need to pass new list and diff it against existing list (matching current DOM), as react, uhtml etc do - whatever way you update the list, it immediately reflects in DOM.
If you find a case where sprae fails with that regard - please report. I am not aware of edge cases for now.

Oh I see, I just ask for my curious. Thank you, I will test then report the edge cases. Have a nice week ! :)


Add ARM64 worker image

We can't simply run "chroot /mnt apt-get install -y /tmp/google-guest-agent_20210927.00-g1_arm64.deb" from a amd64 machine since it's cross-arch.
We need to run it from an arm64 machine. So we support new arg "machine-type" and "wotker-image".
Meanwhile, we need debian-11-worker-arm64 to play this worker role. A regular debian11-arm64 image doesn't have necessary packages.

/hold

/hold

/lgtm

/lgtm

/unhold


Multicast f_cnt out of order

I'm encountering MIC fails on data fragment messages sent via multicast from the Chirpstack FUOTA server. Looking at LoRaWAN frames for my Gateway on Chirpstack Web UI, I see that the f_cnt on messages jumps around and any time a smaller fcnt than another previous message is sent, it causes a MIC fail on STM32.
It appears to me the fcnt is set by Chirpstack FUOTA server https://github.com/chirpstack/chirpstack-fuota-server/blob/28a40c2a13af08f47d68964055236264cbc8153e/internal/fuota/fuota.go#L1323 before being enqueued and sent to Chirpstack.
Chirpstack appears to take in these queue items and asynchronous schedule all of them: https://github.com/chirpstack/chirpstack/blob/8cde64c4b70a4a78aff9f01d5dc8a1bf54b439b5/chirpstack/src/downlink/scheduler.rs#L81 I suspect it maybe is due to this, but really not that sure and have failed to be able to compile Chirpstack myself yet to verify.
Example of LoRaWAN frames with f_cnt of 6 coming after f_cnt of 21 at end.

I am using a STM32-WL55JC1 as the end client and Multitech MTCAP-868-001A as gateway.

ChirpStack calculates a scheduler_run_after_ts on each enqueue, to add time margin between each downlink. This margin is based on the multicast_class_c_margin config value.
Did you change the multicast_class_c_margin configuration value?

I have similar problem. Did you manage to solve it?
The only workaround I know that helped in my case is disabling the duty cycle on the gateway.

Not managed to solve it yet, still investigating. Interesting that disabling duty cycle fixes it. Further raised question whether issue is within chirpstack or somewhere else?
Are you using LoRa Basic Station or Semtech UDP?

Please note that ChirpStack does add delays between the multicast downlinks. However, if your gateway(s) reject the downlinks because of duty-cycle restrictions, then there is a risk that multiple downlinks end up in the same batch and are handled at the same time, in which case I think you might end up with what you describe.
You might want to increase this setting to make sure the gateway will not exceeds its duty-cycle:
    # Multicast Class-C margin.
    #
    # This defines the minimum margin between scheduling multiple multicast downlinks
    # (within the same multicast-group). This value must be equal or greater than the
    # scheduler interval.
    multicast_class_c_margin="5s"

Thank you so much, ran a load more runs and setting multicast_class_c_margin above the calculated duty cycle fixes it!
I previously must have tested with a value still below the duty cycle. I still get occasional odd runs with lots of losses across all devices, however these are very rare and maybe something external in my setup so not going to worry about it.
Sorry about how long it took to test this full and get back to you


influxctl authentication note

This just adds a helpful pointer to eliminate an annoying and unnecessary warning message that may trip up parsing JSON output from the utility.  The note is Linux specific.  It may be Ubuntu specific as well, but this is one of the most popular distros.
Reference: https://stackoverflow.com/questions/75406844/not-loading-module-atk-bridge-the-functionality-is-provided-by-gtk-natively

[x] Signed the InfluxData CLA
(if necessary)
[x] Rebased/mergeable

While I like a technical solution, I went this route because I doubted working around an OS issue likely be fixed externally is the best prioritization of our limited resources. I would not risk regressing functionality in older distros to fix a cosmetic annoyance. Finding a better way is fine with me but that implies some research/time. Any objection to documenting this and then removing this if and when we have a technical solution later?
My intent is just to help the customer and I don't think the approaches need to be exclusive.

I don't think that recommending turning off accessibility system-wide is the way to go as this could affect how they use their computer outside of influxctl. I don't think influxctl redirecting stderr would be a big lift, but if you are going to document a workaround, I might suggest:
"
... (your other text)
To silence the warning, when invoking influxctl unset the GTK_MODULES environment variable (or remove gail:atk-bridge from it's value). Eg:
$ GTK_MODULES= influxctl ...

Users can then choose to create a shell alias, updated GTK_MODULES system-wide, etc if they want to make this permanent, but the info we provide won't break their system.


:sparkles: make zoomable box dismissible in any direction with rotation

Inspired by @cyriana 's original PR on this issue
The implementation now doesnt use the external dependancy, its fully custom, we listen to each pointer event using the Listener and calculate the velocity, direction etc of the swipe and then animate accordingly.

@uiboy  Just look at the draggable animation. Looks awesome.  :-/

Haha! thanks @cyriana , yep i was thinking of open sourcing this. Many can benefit from it :)


Release v0.5.4

Build

Bump node version from 16.17 to 16.18
Bump packages

Codecov Report

Merging #17 (86fba4f) into main (4de77c8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   92.56%   92.56%
=======================================
  Files           5        5
  Lines         148      148
  Branches       11       11
=======================================
  Hits          137      137
  Misses          9        9
  Partials        2        2

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


Configure PersonList Column Width

With Virtualization enabled, column widths change while scrolling, which looks strange.

Fixed


Integration Test - Release - 2019/04/25 01:22:30

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@PRMergerTest2 : Thanks for your contribution! The author, @v-caxian, has been notified to review your proposed change.


Mint & Claim issues - No data loaded - Soonaverse doesn't work

Describe the bug
The content of the soonaverse is not loaded. Neither on the profile page nor on NFT detail pages.
If the content/data loads it takes ages.
Purchase or claim process can't be executed since something will fail.
To Reproduce
Steps to reproduce the behavior:

Go to Soonaverse.com
Click on "My Profile"
Go to the tab "Activity" and try to claim your weekly rewards
See error - Nothing is lodaing and if the rewards are loaded they can't be claimed. Same is for purchasing NFTs as i understand.

Expected behavior
I should be able to claim my reward and mint NFTs.
Screenshots

Desktop (please complete the following information):

OS: Windows 11
Browser Chrome
Version 120.0.6099.217

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Additionally one can't edit spaces. Just wanted to edit one of my old spaces and there was a message "Failed". Somehow it created a proposal and i'm the only guardian in that space but it failed. Also i'm logged in (can see the VIP level and profile image), it also lists 1 Guardian for the space but it's just blank instead of showing my account as guardian.

what user and space is this? I've edited a space without an issue and also loaded member.

I'm able to load the page. Can you open your developer console and see if there are any issues?

Is this still happening to you? I'm unable to replicate this.


phpdoc for SQLFilter::setParameter

Running phpstan (https://github.com/phpstan/phpstan) with the following command php bin/phpstan analyse -l 5 src tests  i came up to this error message:
 ------ -------------------------------------------------------------------------------------------------------------------
  Line   src/Filter/Configurator.php
 ------ -------------------------------------------------------------------------------------------------------------------
  45     Parameter #2 $value of method Doctrine\ORM\Query\Filter\SQLFilter::setParameter() expects string, int|null given.

The second parameter ($value) is described as string. But it is pretty obvious, due to the code that follows, that not only a string can be passed to this method.
I have not worked / participated in open source that much yet, so here are my questions:

Is this a bug?
Is this covered by a bugfix of somebody else?
Could i create a bugfix for this on my own and contribute it to the project?

chucky

Is this a bug?

Most likely

Is this covered by a bugfix of somebody else?

Not aware of it

Could i create a bugfix for this on my own and contribute it to the project?

Absolutely! Start by forking this repository, then clone your fork locally, create a branch, commit a fix, push it, and then open a pull request (github will suggest it)


Add DS record (test)

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description

Link to Website

We don't support DS records.


Introduce a new output level

There are things like lazy loading components in nuxt or caching requests that we can not really detect. (this is an exampl)
But we can add a new suggestion option to the --level flag level where we can output this suggestions (applicable for nuxt or vue or js/ts)
suggestion is just an example name, if needed it can be replaced.

Would this be added as a different option of the level flag?

yes

Would this only work for Nuxt? or in the future we plan to add more suggestions about vue, js/ts?

Would this only work for Nuxt? or in the future we plan to add more suggestions about vue, js/ts?

No, not only for nuxt, that was just an example.

No, not only for nuxt, that was just an example.

I just updated the description for this issue so the contributors have all the information without having to scroll...


device and app initialization not addressed yet

defaults for parameters, communicate state to app, set radio buttons correctly etc.

Need status byte so the app correctly reflects what the wearable is doing. Might need to expand the control characteristics to 16 bit int.


Pylance unable to detect venv modules

The "Python" extension could detect the venv, but Pylance produces a reportMissingImports error, even though numpy is installed in the venv.
If I remember correctly, this issue was said to be solved some years ago, but apparently, it still exists.
 Environment data

Language Server version: 2022.9.40
OS and version: linux x64
Python version (and distribution if applicable, e.g. Anaconda): 3.9.2
python.analysis.indexing: false
python.analysis.typeCheckingMode: basic

 Code Snippet

 import numpy as np

 Expected behavior
Import the module
 Error
Import "numpy" could not be resolved Pylance(reportMissingImports)
 Image

Thanks for the report. Would you please check if the right virtual environment is being used?
The way to do that would be to:

If you have not selected a virtual environment, the status bar will show the global python interpreter.
Click on version number and choose the right environment from the command palette
If there is no environment offered, you will need to create an environment and then select that as the default environment
When you select the right virtual environment Pylance find the modules installed in that environment.

Here is a link to virtual environments: https://code.visualstudio.com/docs/python/environments
Please reopen this issue if this does not solve the problem.

I just ran across this today, and this worked for me too. Here are the detailed steps for anyone else arriving here:

Click on the VSCode status bar on the bottom-right where it says { } Python and a version number.
At the top of VSCode, click on "Enter interpreter path..."
Browse to your project's .venv folder and select \Your\Project\Dir\.venv\Scripts\python.exe.

Now my status bar says { } Python 3.10.11 ('.venv': venv) and the squiggles are gone.

Im on the same spot, but that didnt resolve my issue :(

+1 hitting the same issue with flatbuffers


Scaling slider needs also input field

Scaling slider limits to 100 pods - in GKE UI user can enter any number. Add a number field?

In addition, a + and - button would be awesome to scale deployments very easily!

@SeriousM see #1191


AutoHiC seems to be making changes, but results look same.

Hi, I have tested AutoHiC at least 5 times in the "onehic.py" mode, and every single time it seems to be making adjustments (As indicated by the infer_results folder) - but as soon as I use 3d-dna post review script to generate .hic map on the adjusted assembly, it looks exactly the same as what was supplied to AutoHiC. Am I missing something?
Before AutoHiC :

After AutoHiC:

{"Raw error number": {"translocation": 3, "inversion": 0, "debris": 573}, "Score filtered error number": {"translocation": 3, "inversion": 0, "debris": 561}, "Length filtered error number": {"translocation": 1, "inversion": 0, "debris": 559}, "Length removed error number": {"translocation": 2, "inversion": 0, "debris": 2}, "Overlap filtered error number": {"debris": 365, "translocation": 1}, "Chromosome real length filtered error number": {"translocation": {"normal": 0, "abnormal": 1}, "inversion": {"normal": 0, "abnormal": 0}, "debris": {"normal": 45, "abnormal": 320}}}
This is the error summary.
Can you please help me out here?
Thanks

autohic.e97171355.txt
Here is the log, and the .assembly files before and after
Before:
base.ctg_info.assembly.txt
After:
adjusted.assembly.txt

These are normal operating conditions. There are many debris errors in the genome. These errors are small and usually require many iterations to remove. My suggestion is to stop running onehic once the translocation and inversion errors have been removed from the genome.


docs: replace comma with colon in dict object

Description
The change involves correcting the syntax of the extra_info dictionary in the Document constructor and the subsequent assignment to document.extra_info. The issue fixed is the incorrect syntax of the extra_info dictionary in the old documentation.
So mainly, I replaced , with :
Fixes # (issue)
Type of Change
Please delete options that are not relevant.

[x] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] No tests needed

Suggested Checklist:

[x] I have made corresponding changes to the documentation

Thanks!


API configuration is poorly documented : ClientResponseException

It seems that even if support for v3 OS APi was added via https://github.com/jenkinsci/openstack-cloud-plugin/pull/133 users will continue to strugle trying to configure the plugin (2.17) due to lack of documentation and less that useful error messages.
I do know that this instance is working with both v2 and v3 API but still I was not able to find any way to configure the plugin so it will past the testing phase.
set|grep -a OS_
OS_AUTH_URL=http://<IP_ADDRESS>:5000/v3/
OS_IDENTITY_API_VERSION=3
OS_PASSWORD=secret123
OS_PROJECT_DOMAIN_NAME=Default
OS_PROJECT_NAME=johnd
OS_USERNAME=johnd
OS_USER_DOMAIN_NAME=Default

As you can see the help section gives example only for the v2 syntax and does this using the obsolete naming.

I had two approaches, both failing:

v3: Identify: johnd:johnd:Default and the endpoint ending in /v3/
v2: Identify: johnd:johnd and the endpoint ending in /v2/

On both I got:
Cannot connect to specified cloud, please check the identity and credentials: Not Found
ClientResponseException{message=Not Found, status=404, status-code=NOT_FOUND}
	at org.openstack4j.core.transport.HttpExceptionHandler.mapException(HttpExceptionHandler.java:38)

If you're using plugin version 2.17 then you are not running a version that contains https://github.com/jenkinsci/openstack-cloud-plugin/pull/133
While that pull-request (adding v3 support) has been merged into the master branch, and that will (in the near future) be used to build version 2.18, there is currently no official release that contains support for v3-auth.
i.e. If you want support for v3 auth then you will need to EITHER build your own version of the plugin, or download one from https://jenkins.ci.cloudbees.com/job/plugins/job/openstack-cloud-plugin/ (just beware that the cloudbees site is building various peoples' pull-requests, and the binaries from PR#133 are now long gone - you may find that https://jenkins.ci.cloudbees.com/job/plugins/job/openstack-cloud-plugin/312/org.jenkins-ci.plugins$openstack-cloud/artifact/org.jenkins-ci.plugins/openstack-cloud/2.18-SNAPSHOT/openstack-cloud-2.18-SNAPSHOT.hpi works for you though).
If you get really stuck, let me know an email address and I can send you a copy of the one I'm using.
Regarding the 404, I'm pretty sure that a 404 isn't directly related to the plugin code.  I suspect that this is more likely because you've not given the plugin the "right" endpoint URL.
Finding out exactly what the "right" URL is can be non-trivial.  In theory, "your local OpenStack deployment administrator can tell you this" ... but you are your own OpenStack admin then you need to find this out somehow :-/
This isn't the sort of thing that the plugin can simply "discover" through software alone - it very much relies on the user telling it where to start.  If the user doesn't know where they deployed the Identity service to then the plugin software can't tell them.
This issue is further compounded by there only being limited support for v3-auth right now.  e.g. Where I work, we run "Horizon" as a web front end for interactive use (point browser at http://somehostname/, login, get pretty WebUI showing an overview of OpenStack deployment) and, if I use that to look at "Access & Security -> API Access -> Identity", it correctly lists the correct endpoint for our v2-auth cloud deployment (it says that the Identify service is at "http://somehostname:5000/v2.0").  However, if I do the same on our v3-auth cloud then it gives the wrong answer (Horizon tells me my Identity service is at "https://otherhostname:5000/v2.0" whereas I need to tell Jenkins to use "https://otherhostname:5000/v3"); I don't know if this is a bug in Horizon or an error in my deployment.
Regarding the OS_AUTH_URL environment variables etc, I suspect that those aren't coming from the plugin code - I did a string search for a few of them and they're not in the plugin source code.  I think that it's rather more likely that its your own cloud-init code (that's being run within the VM on OpenStack when the VM boots up) that's setting those.
As for documentation, you will (I hope) be pleased to hear that the help text in the next version of the plugin does tell you to use PROJECT_NAME:USER_NAME:DOMAIN_NAME syntax for v3 auth: https://github.com/jenkinsci/openstack-cloud-plugin/blob/master/src/main/resources/jenkins/plugins/openstack/compute/JCloudsCloud/help-identity.html
If, when you look at the plugin help, you only see the "old" syntax that you mentioned then the plugin you have installed in your Jenkins doesn't support v3 auth and you need to get a newer version (i.e. newer than 2.17).
Note: If you can provide better help text for the endpoint URL field (which is currently just https://github.com/jenkinsci/openstack-cloud-plugin/blob/master/src/main/resources/jenkins/plugins/openstack/compute/JCloudsCloud/help-endPointUrl.html ) that can make it easier for others to work out what their URL should be then please do contribute and it can be incorporated into future versions of the plugin (I'm strongly in favor of help-text that really helps)

@pjdarton Thank you. In fact after compiling the main branch myself I was able to make it authenticate. The problem was that I was trying to use the last release which didn't include your changes.
I had to postpone my test of this plugin due to other priorities but I will be back.

I am closing this because this was fixed in trunk.

The feature is released in 2.18. Closing this.


Critical Bug Report - Application Crashes on iPhone 14 Pro with iOS 17 SDK in Release Mode

I wish to communicate my disappointment and dissatisfaction regarding the current state of the plugin. Although the initial integration into my application was executed with swiftness and efficiency, a significant issue has surfaced during the application's execution in release mode on an iPhone 14 Pro with iOS 17 SDK.
Upon the initiation of a meeting, the application fails to run on the first attempt, presenting an error message stating, "Unfortunately, something went wrong." Subsequent efforts to utilize the "Rejoin" option prove ineffective, leading to application unresponsiveness and necessitating a system reboot. It is only on the second attempt that I am able to successfully initiate a meeting.
In an effort to provide clarity, I have attached a video that visually illustrates the problem on the device. I trust that this representation will offer valuable insights for addressing and resolving the identified issue. Your prompt attention to this matter is highly appreciated.
[
Uploading 291005442-fcf50326-ca00-4cb1-95d7-c3f2362dbec9.mov…
](url)

Please provide a reproducible example & fix the video link.


fix clippy and gossipnet test in CI

Note that this still has to pass be activated in CI by changing the last step in the test job. Right now it's only testing -p astria-conductor -p astria-sequencer-relayer, so astria-gossipnet is not running.

closing, will open another PR


What if I forgot recovery code of MFA?

Like the title, how can I reset MFA by CLI?

@Tristan0430 Casdoor has no CLI. For your case, you can either:

Use built-in/admin admin account to change Casdoor setting
Use app's clientId/secret to call Casdoor HTTP API to change setting
Manually change Casdoor DB to change setting


cdc source creates containers that consume all the memory in the Yarn cluster

Search before asking

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

What happened
Simply put, when submitting from cdcsource mysql to print with yarn session, it continuously creates containers that consume all the memory in the Yarn cluster, and the two operators remain in the created state for a long time without progressing to the running state; however, when submitted as a yarn application, it operates normally.
What you expected to happen
Simply put, when submitting from cdcsource mysql to print with yarn session, it continuously creates containers that consume all the memory in the Yarn cluster, and the two operators remain in the created state for a long time without progressing to the running state; however, when submitted as a yarn application, it operates normally.
How to reproduce
Simply put, when submitting from cdcsource mysql to print with yarn session, it continuously creates containers that consume all the memory in the Yarn cluster, and the two operators remain in the created state for a long time without progressing to the running state; however, when submitted as a yarn application, it operates normally.
Anything else
No response
Version
1.0.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@scottxing 请规范你的 issue title , 否则将视为无效 issue

Check the JobManager logs to find the reason, it may be due to unreasonable resource allocation.


[Catalog] Dynamically load related cards inside the modal

Signed-off-by: asubedy<EMAIL_ADDRESS>Description
This PR fixes #705
Notes for Reviewers
I did not change the json data, instead manipulated on the frontend, if this is not something we wan, then I can revert it and actually change the json data.
Signed commits

[x] Yes, I signed my commits.

@asubedy can we limit it to 1 or 2 related patterns? and not have the coming-soon ones.

Merge conflicts...


Camera: Square aspect ratio in camera preview.

camera : aspect ratio is currently set to as default aspect ratio given by camera and if we try to set another aspect ratio then it stretches the camera preview.

Closing this as duplicate of https://github.com/flutter/flutter/issues/80513

Also, similar issues are: https://github.com/flutter/flutter/issues/15953, https://github.com/flutter/flutter/issues/80393


Do not log secrets

When debug logging is enabled we log secrets.
This should not happen. At least not by default.

See also discussion here: https://github.com/gopasspw/gopass/pull/1865#discussion_r602135817

I have a plan on how to fix this:

Propagate the fact that a string contains secrets (either via context or type switch) to the log method
Avoid logging these, possibly we could have another env var to log these as well, if it's really necessary for debugging e.g. secrets parser and such


System.Text.Json serializer doesn't work with Guid and Guid?

Describe the bug
System.Text.Json serializer doesn't work with Guid and Guid?. The behavior is as following:

For Guid: the serialized data is always {} regardless of the Guid value.
For Guid?: get exception ArgumentException: Cannot bind to the target method because its signature is not compatible with that of the delegate type.

To Reproduce
Steps to reproduce the behavior:

Using ASP.NET Core 3.0 Preview 5
Create a default API template and remove AddNewtonsoftJson()
Change IEnumerable<int> in ValueController to IEnumerable<Guid> or IEnumerable<Guid?> and the returned data accordingly.
Test the API

Expected behavior
Guid and Guid? should serialized into strings or null.

@Eilon could you transfer this to https://github.com/dotnet/corefx?

This issue was moved to dotnet/corefx#37496


Update docker-compose.yml

Remove IP address from Chronograf port to make Chronograf accessible from outside of Docker container.

What are your thoughts @jonbloom  ?

Theoretically that's fine, but to be pedantic, the <IP_ADDRESS> doesn't refer to the container, but the host the container is running on. I figured with the low level access to Influx it gives you, it'd be better to restrict it.

In my case the whole stack runs on a Raspberry Pi and I cannot acces Chronograf from, for example, my laptop with this setting.

I see it both ways, but I think I'd rather leave the more "secure" path as the default, and end users can change it in their compose files if they have other requirements.
Having said that, should we also limit InfuxDB to only listen on the docker host? Could use an overlay network to only expose the bits that are required, like Grafana's web UI.

I can’t follow your logic. I guess typical users run this stack on some kind of server and access it from a web browser on another machine. Why would you expose Grafana’s web interface, but not Chronograf’s? I can’t even access Chronograf from the host even if I wanted to as I have no GUI running.
Anyway, it’s of course up to you. Perhaps make it somehow optional to open Chronograf up outside of the host.
Thanks

@L2v2P I thought about this more last night. I think you're right. Since Chronograf needs auth to actually do anything, there's not a big risk in exposing it. I'll merge this in.
When accessing interfaces that are bound to localhost (<IP_ADDRESS>), I usually spin up an SSH tunnel to that device and forward the port, or similar.


Help understanding what the workflow looks like to use this.

Hi Michele!
I just watched your "How to test like it's 2017" and really enjoyed it. react-fix-it looks really exciting! I'm having a little trouble understanding what the workflow of debugging looks like by looking at the readme. Could you possibly do a demo that shows set up and the process, or explain it a little more in-depth in the readme? Right now the demo works on my machine, but I'm not getting any tests in the console or screen on my app on my local. Do I need to be calling fixit(myModule) on every module?
Thanks so much,
Alex

Thank you very much for your comment, @awhit012.
I'm glad you liked the talk and the library.

Could you possibly do a demo that shows set up and the process, or explain it a little more in-depth in the readme?

I'm sorry the README is not clear, but the "set up" only consists in wrapping a component with fixIt.

Do I need to be calling fixit(myModule) on every module?

Yes, all the modules must be wrapped and the easiest way is to use babel-plugin-react-fix-it.
I hope this helps.


Run tests for Python code in extension

Set up testing for the extension using pytest for Python code.

No python code yet, hence closing this.


[Bug] 1.8.0b4-dev30 GUI continues to report invalid metadata URI for NFT after valid one added

What happened?
I bought an NFT which turned out to have an invalid metadata URI. I found a correct URI and verified using curl and sha256sum that it hashes to the correct value. I added the new URI to the NFT. The GUI continues to report "Invalid metadata url" (on top of the NFT image) and "Title Not Available." (I'm not certain this is a bug, but that's what pyl in the Keybase support channel seemed to think, and I do think it would make sense to use the most recently added URI to ascertain validity and extract metadata to display, since there's always a risk of data at an old URI getting corrupted.)
I presume this can be reproduced by minting an NFT, changing the data at the original metadata URI, and adding a new URI that points to a good copy of the original metadata. But I haven't tried that yet.
Version
1.8.0b4-dev30
What platform are you using?
Linux
What ui mode are you using?
GUI
Relevant log output
No response

Can you try with the latest 1.7.1 RC build?
https://github.com/Chia-Network/chia-blockchain/releases/tag/1.7.1-rc2

This is a bit strange. The specific original problem seems to be gone in 1.7.1-rc2, but in the mean time, I had bought another NFT for which Chia also says "invalid metadata url" even though the link seems to generate the correct hash, and in this case RC2 is still complaining, even after clearing the cache.

if you hit the refresh metadata option, does it clear the "invalid metadata url" warning?

I don't see a refresh metadata option

I saw this issue with one of the NFT in 1.7.0, and refreshing the NFT data few times can solve it

Ok I think 1.7.1rc2 fix this issue. I have been using 1.8.0b3 these few days buying nft like maybe 10, and all have this issue on 1.8.0b3. But just now I bought 1 using 1.7.1rc2 and without this issue

fixed via https://github.com/Chia-Network/chia-blockchain-gui/pull/1730 it will be part of 1.8.1


:robot: run install test under earthly

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

I see now what @jimmykarily meant, this is slow as heck.

But running it outside earthly doesn't make it automatically faster. We need to enable KVM the here are lost of hoops to jump through to achieve that.

But running it outside earthly doesn't make it automatically faster. We need to enable KVM the here are lost of hoops to jump through to achieve that.

kind of does lol. Im guessing the extra layer of dockering

I managed to get tests to run with KVM here: https://github.com/kairos-io/provider-kairos/pull/306 (self-hosted runners). When that gets merged we can plan how to do that for this repo too. We may need more resources for that.


[QT-Wallet] In Receive Coins page, all the fields should be aligned same.

Description:
In Receive Coins page, "Account to receive incoming MWC coins" field is left aligned but all other Address (MWCMQS, TOR, payment proof address) fields are center aligned.
Please refer screenshot for more details.

Expected Result:
In a page, all the fields should be aligned as same.

Seems like it is the only page that center aligned. At least I don't see other pages.
I think it make sense to make uniform. So I will align everything to the left

Verified, hence closing this ticket.


Sidebar to display label data

Currently, when hovering over the data, the labels are display up top, this should be more visible, perhaps in a sidebar to the right.
This is subject to change due to the site design ticket.

Put the trail text in the donut hole instead.


Question: which script and where in main.cpp should be changed to control Microphone from command line?

Hi,
I had asked a question earlier which was closed and dhpp told me that for the controlling of Microphone means turning it on and off from command line I had to change script and some changes in main.cpp.
I asked in thay issue but there was no reply so I am asking here: In which script and where & how should I do changes and in main.cpp also to control Microphone turning it on and off from command line in raspberry pi.
As I have just learnt python and I am not familiar with C or C++  so I don't have any clue. I hope you all understand and help me regarding it..
Thanks.

hi @rajdpandey
previous issue in question: https://github.com/alexa/avs-device-sdk/issues/1499
You will need to update SampleApp/main.cpp to take in new args for enable/disable microphone. You will also need to update SampleApplication.cpp create() function to create without a microphone.
This implementation is outside of the scope of this forum and I will close the issue here. Feel free to re-open if you have more questions.


Add an API for splitting a terminal

Here are two options:
export interface TerminalOptions {
  splitFrom?: Terminal
}

or
export interface Terminal {
  split(options: TerminalOptions): Terminal;
}

/cc @fabiospampinato

They both look pretty good to me.
Maybe I'm leaning a bit toward the latter, especially if split direction and shell/args/env inheriting were to be implemented in the future, I think this:
export interface TerminalSplitOptions {
  direction: string,
  inherit: boolean
};

export interface Terminal {
  split(options: TerminalOptions, splitOptions?: TerminalSplitOptions): Terminal;
}

Looks a little nicer than this:
export interface TerminalOptions {
  splitFrom?: Terminal
  splitDirection?: string,
  splitInherit?: boolean
}

Has there been any movement on this?

Bump, looks like there hasn't been anything for quite a while. Is this still being worked? Thanks!

Note to self: we should probably align to how editor splitting works in the API, also if terminal tabs could be pulled into the editor area how does that affect things?

Adding to October to discuss, it won't happen in October though.

Discussed this in the API meeting today.
Creation vs placement
One big thing that has blocked this has been wanting to wait to see how the API ends up looking once positioning is more dynamic within the editor area and when terminal tabs (https://github.com/microsoft/vscode/issues/10546) has arrived. I don't think we are actually blocked on this for the following reasons:

We should not try to combine creation and placement APIs. Take the editor for example, showTextDocument doesn't talk about creation it's primarily an API centered around placing a text document somewhere. I can imagine a showTerminal or moveTerminal API in the future which achieves this not just for moving terminal groups but also 'unsplitting' terminals, we can explore that more in the future though.
Splits are not tabs. When we add tabs, splits will remain within a single tab so they will be treated as a whole (ie. a 'terminal group', similar to editor groups).

Split direction
Terminals only split in a single direction, this was initially done for simplicity reasons and because the panel is typically quite narrow. We should probably stick to this philosophy when we allow terminals within tabs, for an elaborate layout it might be better to go with single terminal instances using the editor group's grid system.
Complexity around default split direction within the editor and the ability to toggle the split direction could be provided by commands exposed in the command palette.
Approach
We discussed the following two high-level approaches. Specifying how to split via options:
export interface TerminalOptions {
  splitFrom?: Terminal
}

Or via a method on Terminal:
export interface Terminal {
  split(options: TerminalOptions): Terminal;
}

We preferred the former, the wording needs some refinement but using "beside" to align with ViewColumn.Beside was suggested:
export interface TerminalOptions {
  splitBeside?: Terminal
}

This ties into a parallel discussion we're having in https://github.com/microsoft/vscode/issues/63052#issuecomment-541786538 which would see this terminal exposed on Terminal.creationOptions. An interesting note here is that if the parent that a terminal was split from is disposed then this will keep hold of the disposed terminal reference (just as if you held onto Terminal in some other way). This is how editors work and shouldn't be a big deal.
Restoring terminal layout
If an extension were to restore terminal layout exactly as it was between reloads then it could iterate over window.terminals and identify the layout based on Terminal.splitBeside, unless a terminal is disposed or unless (when we support it) you can unsplit terminals. At that point we would need a dynamic property that specifies what terminal is next to the current terminal, for example:
export interface Terminal {
	// splitBeside as well?
	adjacentTerminal: Terminal | undefined;
}

This is because while TerminalOptions.splitBeside will likely be accessible in the future via Terminal.creationOptions, that one must be static.

Can we add horizontal splitting support into this as well? It would be nice to split code and terminals panes horizontally..

@awarmfastbear I don't think we want to add 2 dimensional splitting to the terminal as that makes it a lot more complicated, however eventually terminal "tabs" will be able to be dragged into the editor space where you will be able to lay them out however you want. When you can do this you will still have single-dimensional splits inside individual tabs.

@Tyriar Thanks for the info, is there an issue # we can track for this feature?

eventually terminal "tabs" will be able to be dragged into the editor space

@awarmfastbear https://github.com/microsoft/vscode/issues/10546

As a temporally solution until an official API lands you can do something like this:
  async createNewSplitTerminal(): Promise<vscode.Terminal> {
    return new Promise(async (resolve, reject) => {
      await vscode.commands.executeCommand("workbench.action.terminal.split");

      vscode.window.onDidChangeActiveTerminal((terminal) => {
        if (terminal) {
          resolve(terminal);
        }
      });
    });
  }

Any updates?

need it...

Any updates on this?
It would be really nice to be able to launch multiple tasks at the same time and have them all show up in the same terminal view.

Unfortunately the temporary solution from @auchenberg-stripe doesn't seem to work anymore:

As a temporally solution until an official API lands you can do something like this:
  async createNewSplitTerminal(): Promise<vscode.Terminal> {
    return new Promise(async (resolve, reject) => {
      await vscode.commands.executeCommand("workbench.action.terminal.split");

      vscode.window.onDidChangeActiveTerminal((terminal) => {
        if (terminal) {
          resolve(terminal);
        }
      });
    });
  }

A hacky solution that does work at time of writing this is
async createNewSplitTerminal(): Promise<vscode.Terminal | undefined> {
  await vscode.commands.executeCommand<vscode.Terminal>('workbench.action.terminal.split');
  return vscode.window.activeTerminal;
}

However without official API support, please use with caution as there's no guarantee this hack will continue to work.

Can anyone provide a working temporary fix? @vcheung-stripe 's answer doesn't work for me, and my extension, https://github.com/EthanSK/restore-terminals-vscode, is useless without this working!

This VSCode extensions seems to be able to do this according to the screenshot: ariassd.terminal-loader https://marketplace.visualstudio.com/items?itemName=ariassd.terminal-loader

Note that now we have terminal editors this should include how to create a terminal in a view column.

@Tyriar Is it opened to external contributions at least like proposed API?
I'm interested though only in placing and splitting in the panel area not editor for now. Could be an option to design API in 2 steps as proposed and then merge eventually when both are clear?

@akosyakov this was actually already planned for August already so we'll probably make some progress here this month.

Cool!

Proposal below to get discussion started in API sync today, I think this covers all the cases:
	//#region Terminal target https://github.com/microsoft/vscode/issues/45407

	// Goals:
	// - Create terminal splits (aka. terminal groups)
	// - Create terminal editors
	//
	// For text documents there is:
	//
	// export interface TextDocumentShowOptions {
	//	viewColumn?: ViewColumn;
	//	preserveFocus?: boolean;
	//	preview?: boolean;
	//	selection?: Range;
	// }

	export interface TerminalOptions {
		location?: TerminalLocation | TerminalEditorLocationOptions | TerminalSplitLocationOptions;
	}

	// TODO: Should we expose this on the Terminal interface?
	export enum TerminalLocation {
		Default = 0,
		Panel = 1,
		Editor = 2,
	}

	export interface TerminalEditorLocationOptions {
		/**
		 * A view column in which the {@link Terminal terminal} should be shown in the editor area.
		 * Use {@link ViewColumn.Active active} to open in the active editor group, other values are
		 * adjusted to be `Min(column, columnCount + 1)`, the
		 * {@link ViewColumn.Active active}-column is not adjusted. Use
		 * {@linkcode ViewColumn.Beside} to open the editor to the side of the currently active one.
		 */
		viewColumn: ViewColumn;
	}

	export interface TerminalSplitLocationOptions {
		/**
		 * The parent terminal to split this terminal beside. This works whether the parent terminal
		 * is in the panel or the editor area.
		 */
		parentTerminal: Terminal;
	}

	//#endregion

In the terminal panel would it be also possible to specify insertion position? for instance If I want to insert new terminal before all other terminals, not at the end as usual?

@akosyakov that's not currently possible outside of dragging after creation, that sort of detail (TerminalSplitLocationOptions.side?) might be overcomplicating?
We could always add it later if there are some compelling use cases.

that's not currently possible outside of dragging after creation

To clarify, if you have a terminal group with 2 terminals and set the left one as the parentTerminal, the new terminal should be created in the middle. You cannot create a terminal in the left most position without dragging.

Will need the test plan and then move this to Sept for finalization. We might want to wait a bit longer for this to make sure extensions pick it up but we can figure that out next month.

@Tyriar Sorry I haven't thought about this in a while. The API proposal seems good to me, the only thing I sort of don't like is that the interface is a bit different depending on where the terminal instance is located, for example I can split a particular terminal if I'm in the panel, but it seems that I can only split the current terminal/editor if I'm in the editor area. Also it might be interesting to allow to set a split direction too, in order to be able to make complex 2d layouts, but it's unclear to me if there's an actual need for that, better not to overcomplicated the API for now until the situation is clearer IMO. Overall it seems good to me 👍


Channel order scrambled when sorting

Any time separate clusters in a channel are grouped or marked as multi/good unit or noise, the program scatters the order of the channels so they are arranged by id # instead of channel #. Is there a way to fix this?Also, occasionally when sorting Kilosort stops automatically changing the channel you are viewing in WaveformView so that it just stays on the same channel, showing noise, as you click through others in ClusterView.

Yes indeed the order updating is very distracting and slows down progress

I have had a similar problem as well. I like to look through clusters pre-categorized as noise and every time I move one out of the noise category I have to go back and find where I was.
Is there any way to change this?

should be fixed in the dev branch


ProBook 6470b | Issue while installing Monterey

After installing from USB it boots to normal installer and then this pops out

I don't have eth cable but I might try Big Sur again


Feature/hdfs Parquet: support multiple compression algorithms

Reuse conf format.class.compression for Parquet too.

thx @pdaures LGTM


Persistent SMBus write/read problems ... Please run my test and report back

Because my UPS control suffers from persistent errors concerning the power control/timer registers no matter how I try to add exception handling, I made the short script below to get a better picture.
I run it with no other scripts else concerning the UPS, but I still get wrong results from writing and then reading back the same register even with time.sleep(0.5) before and after.
The register I test is one that it should be possible to write to again and again acoording to GeeekPi.
I am at my wits end with these errors, really. If you can, stop upsPlus.py in cron and run this script on your UPS/Pi and report your results.
This is a sample output from the script below run in Thonny:

%Run CatchExceptions3.py
OK             0  -  180
OK             0  -  0
OK             1  -  180
OK             1  -  0
OK             2  -  180
OK             2  -  0
read != write  3  -  0 NOT in (180,179)
OK             3  -  0
OK             4  -  180
read != write  4  -  179 != 0
OK             5  -  179
OK             5  -  0
OK             6  -  179
OK             6  -  0

The script:
#!/usr/bin/python3
# -*- coding: utf-8 -*-

# ar - 18-05-2021

# import os
import time
# import smbus2
from smbus2 import SMBus

# Define I2C bus
DEVICE_BUS = 1

# Define device I2C slave address.
DEVICE_ADDR = 0x17

RA = 0x18

def settle():
    time.sleep(0.5)

i = 0
while True:
    try:
        settle()
        with SMBus(DEVICE_BUS) as pbus:
            pbus.write_byte_data(DEVICE_ADDR, RA, 180)
        settle()
        with SMBus(DEVICE_BUS) as gbus:
            byte = gbus.read_byte_data(DEVICE_ADDR, RA)
        settle()
        if (byte in (180,179)):
            pass
            print('OK            ', i, ' - ', byte)
        else:
            print('read != write ', i, ' - ', byte, "NOT in (180,179)")
        with SMBus(DEVICE_BUS) as pbus:
            pbus.write_byte_data(DEVICE_ADDR, RA, 0)
        settle()
        with SMBus(DEVICE_BUS) as gbus:
            byte = gbus.read_byte_data(DEVICE_ADDR, RA)
        settle()
        if (byte == 0):
            pass
            print('OK            ', i, ' - ', byte)
        else:
            print('read != write ', i, ' - ', byte, "!= 0")
        time.sleep(1)
        i += 1
    except TimeoutError as e:
        print(i, ' - ', byte, ' - ', e)
        settle()
    except KeyboardInterrupt:
        break
    except:
        pass

# EOF

Sorry for the formatting. GitHub did that, and I forgot/couldn't find the way to prevent it...

friend, select all code text you want  and click on <> button.

In this script I read the value back after it was written. If the result is different from what was supposed to be written, it is written again until OK. Like driving a nail with a hammer ;-)
Time.sleep here and there doesn't do much.
Please if you can, run this and see for yourself. Every once in a while the write operation fails and the previous value remains, which is when you have tp write it again.
I can only guess what causes this. One suggestion would be timing errors on the SMBus. Is that caused by the hardware? I don't know, but there seems to be a way around, write until i2c gets it right ;-)
`#!/usr/bin/python3
-- coding: utf-8 --
ar - 18-05-2021
import os
import time
import smbus2
from smbus2 import SMBus
Define I2C bus
DEVICE_BUS = 1
Define device I2C slave address.
DEVICE_ADDR = 0x17
def settle():
time.sleep(0)
Write byte to specified I2C register address
def putByte(RA, wbyte):
while True:
try:
with SMBus(DEVICE_BUS) as pbus:
pbus.write_byte_data(DEVICE_ADDR, RA, wbyte)
with SMBus(DEVICE_BUS) as gbus:
rbyte = gbus.read_byte_data(DEVICE_ADDR, RA)
if (wbyte) <= rbyte <= (wbyte):
print("OK ", wbyte, rbyte)
break
else:
if rbyte <  max((wbyte - 2),0):
            raise ValueError
    except ValueError:
        print("Write:", wbyte, "!= Read:", rbyte, " Trying again")

while True:
putByte(0x18, 180)
settle()
putByte(0x18,   0)
time.sleep(1)

EOF`

A failing write on a timer register makes for unwelcome surprises, so I now have all writes to SMBus in my scripts changed into 'write, read back, compare, and if needed write again until it sticks'. I use a version of the above putByte() function. This way I can be sure that registers always get the value they are supposed to have.
I had really persistent problems caused by failing writes to timer registers.

When I had started to use frtz13's UPS script fanShutDownUPS.py I learned to check error messages with journalctl -f. I had regular i2c errors which are now all handled by frtz13's script which one can see in the log from journalctl.
So far so good. At that time I was running MotionEye and of course had a camera attached to the Pi4B with the UPS.
For testing Home Assistant and MQTT and Docker I changed to clean installation of Raspberry Pi OS without MotionEye. I didn't get anything out of HA and MQTT so I removed those. The camera is still connected to the Pi, and enabled, but not working.
In this situation I have not seen any  i2c errors in the log. Not unhandled - which would have crashed the script - nor handled errors either, of which the log would show which part of the script they occurred in.
So, it looks like MotionEye has a hand in the frequent i2c errors that I saw earlier

I don't have two cron tasks related to UPS Plus running at the same time. Which tasks do you mean?
I have only upsPlus.py every minute with a file lock (flock ....) and when I want to run another program, like a test script for instance, I stop upsPlus.py.
On my PowerCycle.py I have the same lock, so that will wait and not start when upsPlus is executing.
I don't run upsPlus_iot.py on a schedule at all, since it sits and waits for up to a minute.
My tests don't last so long that the batteries would run out, and also the charger is connected all of the time, so I don't need to run upsPlus.py at all, since I am in control.
I can read the register contents with my UPS_report.py and see at least what the values of the countdown timers are at any moment, so that I don't experience a sudden shutdown three minutes after I stop the test script, which may last have written 180 to a countdown timer register.
But running CatchExceptions4.py I could find out that I need to read back what I attempt to write to a countdown timer and see if the write has been succesful. If not write again ...
The following may also work: write 0 first to the countdown register when after one minute it is going down at120 and only then write the 180 again (you suggested 120) for the watchdog function. That may work reliably, too. I never tried this on its own. I always combined that with write, read back and if different write again ...

please show me this command's output :　crontab -l


3.12 "type" soft keyword (PEP 0695)

(Per pep0695)
Python 3.12 (Upcoming) introduced a new soft keyword called type which allows you to do something like this:
# A non-generic type alias
type IntOrStr = int | str
# A generic type alias
type ListOrSet[T] = list[T] | set[T]

(It also introduces generics syntax for things like functions and classes but thats probably better left to another issue)

Added on master


Khkh flexapp dep config

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.

@khkh-ms Could you please resolve these CI issues?

@khkh-ms Any update?


Add missing dep state_machines_macros in vstd

I want to use a customized rustc to call verus for crates that needs verification in my project. I will use vstd as a dependency and built by cargo, since I will need to change build target.
Everything works except for the missing dependencies state_machines_macros.

I don't believe state_machine_macros is used in vstd (I think we have CI that checks that vstd builds successfully). If you're using state_machine_macros, can you add it as a dependency of those crates?
I may be missing something about what you're trying to do, though.

state_machines_macros is used in vstd/shared.rs

Nevermind! I wonder why CI isn't catching that.

I may be missing something about what you're trying to do, thou

It does not capture it because state_machine_macros crate is hard linked in verus default mode.


Customize FasterRCNN

Hi,
I've been trying, unsuccessfully to customize a bit the implementation of FasterRCNN proposed by torchvision. For example, one thing I would like to do, would be to write a customized postprocess_detections  function that return confidence for all labels and not only the one with highest confidence.
In the past I've managed to successfully overwrite the loss function by doing something like
model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(pretrained=True)
torchvision.models.detection.roi_heads.fastrcnn_loss = custom_loss

But the postprocess_detections function is within the RoIHeads class. If I try to replace the RoIHead class before defining my model I get this error:
torchvision.models.detection.roi_heads.RoIHeads = RoIHeadsCustom
model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
    pretrained=True
)

Traceback (most recent call last):
  File "test2.py", line 80, in <module>
    model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 470, in fasterrcnn_mobilenet_v3_large_fpn
    return _fasterrcnn_mobilenet_v3_large_fpn(weights_name, pretrained=pretrained, progress=progress,
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 393, in _fasterrcnn_mobilenet_v3_large_fpn
    model = FasterRCNN(backbone, num_classes, rpn_anchor_generator=AnchorGenerator(anchor_sizes, aspect_ratios),
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 222, in __init__
    roi_heads = RoIHeads(
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/roi_heads.py", line 512, in __init__
    super(RoIHeads, self).__init__()
TypeError: super(type, obj): obj must be an instance or subtype of type

But if I define it afterwards, the object is already created and the custom class is not taken into account
model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
    pretrained=True
)
torchvision.models.detection.roi_heads.RoIHeads = RoIHeadsCustom

If anyone has some ideas on how to easily customize torchvision models that would be a great help. The only solution I'm seeing is creating a fork of torchvision, which I'd rather avoid.
Thanks.
cc @datumbox @YosuaMichael

@paullixo Thanks for the question. @YosuaMichael is spot on, this is how you could currently extend the existing detection models. We understand that potentially this is not the most convenient way to swap out losses and postprocessing. We plan on the near future to invest time revamping the Object Detection API. I think we can leave the issue open to keep an eye on the kind of work that the users of the library want to be able to do seamlessly. Let us know if you have any further questions.

Thank you for your answer, however I do get the error mentioned above when doing so.
Here is the full code:
from typing import Optional, List, Dict, Tuple
import torch
import torchvision
class RoIHeadsCustom(torchvision.models.detection.roi_heads.RoIHeads):
    def postprocess_detections(
        self,
        class_logits,  # type: Tensor
        box_regression,  # type: Tensor
        proposals,  # type: List[Tensor]
        image_shapes,  # type: List[Tuple[int, int]]
    ):
        # dummy function to test the customization
        print("custom postprocess function")
        return

torchvision.models.detection.roi_heads.RoIHeads = RoIHeadsCustom
model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
    pretrained=True
)

Which results to the error:
Traceback (most recent call last):
  File "test2.py", line 21, in <module>
    model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 470, in fasterrcnn_mobilenet_v3_large_fpn
    return _fasterrcnn_mobilenet_v3_large_fpn(weights_name, pretrained=pretrained, progress=progress,
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 393, in _fasterrcnn_mobilenet_v3_large_fpn
    model = FasterRCNN(backbone, num_classes, rpn_anchor_generator=AnchorGenerator(anchor_sizes, aspect_ratios),
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/faster_rcnn.py", line 222, in __init__
    roi_heads = RoIHeads(
  File "/home/paul/.local/lib/python3.8/site-packages/torchvision/models/detection/roi_heads.py", line 512, in __init__
    super(RoIHeads, self).__init__()
TypeError: super(type, obj): obj must be an instance or subtype of type

Hi @paullixo , strangely enough your code work on my laptop. However I think @datumbox suggestion to initiate the roi_heads module is a better way to do it. Here is the full code using @datumbox suggestion:
from typing import Optional, List, Dict, Tuple

import torch
import torchvision

class RoIHeadsCustom(torchvision.models.detection.roi_heads.RoIHeads):
    def postprocess_detections(
        self,
        class_logits,  # type: Tensor
        box_regression,  # type: Tensor
        proposals,  # type: List[Tensor]
        image_shapes,  # type: List[Tuple[int, int]]
    ):
        # dummy function to test the customization
        print("custom postprocess function")
        return

model = torchvision.models.detection.fasterrcnn_mobilenet_v3_large_fpn(
    pretrained=True
)
# assign to r just for quick access of its param
r = model.roi_heads
# Initiate the custom roi_heads (see https://github.com/pytorch/vision/blob/main/torchvision/models/detection/roi_heads.py#L492)
new_roi_heads = RoIHeadsCustom(r.box_roi_pool, r.box_head, r.box_predictor,
              r.proposal_matcher.high_threshold, r.proposal_matcher.low_threshold,
              r.fg_bg_sampler.batch_size_per_image, r.fg_bg_sampler.positive_fraction,
              r.box_coder.weights, r.score_thresh, r.nms_thresh, r.detections_per_img,
              r.mask_roi_pool, r.mask_head, r.mask_predictor,
              r.keypoint_roi_pool, r.keypoint_head, r.keypoint_predictor)

model.roi_heads = new_roi_heads

basically need to copy the parameter from original roi_heads and pass it to the RoiHeadsCustom constructor. Hope this will work for you!

Thank you, it does work!


How to form nodes (in GptSimpleVectorIndex form) based on paragraphs not based on fixed token size?

hey,
I want to form node (in GptSimpleVectorIndex form) but based on paragraphs not based on fixed token size (and definitely not containing the full document). so that every node has a fixed context in them so that the minimum no. of nodes is selected for prompt formation.
I currently want to do this on Google Docs. How to achieve this.

Hi, we're currently adding the ability to add your own text splitters to an index, hopefully this should help resolve your issue

cd @siddhant01 try adding a langchain text_splitter field when building an index.
alternatively, "pre-split" the text into separate Document objects, and load that into gpt index during index construction

Going to close this issue for now.
You can write your own custom node parser if needed pretty easily, or pre-split your documents as jerry suggested.
Base Class:
https://gpt-index.readthedocs.io/en/latest/api_reference/service_context/node_parser.html
Docs:
https://gpt-index.readthedocs.io/en/latest/core_modules/data_modules/node_parsers/usage_pattern.html


Add exhaustive list of error codes and description of metric flag

Thank you for helping make the Dapr documentation better!
Please follow this checklist before submitting:

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[ ] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[ ] New file and folder names are globally unique
[ ] Page references use shortcodes instead of markdown or URL links
[x] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

In addition, please fill out the following to help reviewers understand this pull request:
Description
Added the exhaustive set of error codes dapr emits as well as how to enable metric recording for these error codes based on the associated PR below which implements this.
Issue reference
https://github.com/dapr/dapr/pull/8256

@jake-engelberg - Also let's make the Error codes concept more visible by updating this article on observability https://docs.dapr.io/concepts/observability-concept/ and have a sub heading called Error codes describing what they are and also how you can get metrics for them. This can point to the error codes list and the metrics configuration. @hhunter-ms can help as needed


React Error when scrolling down mitmweb

When scrolling through a capture on mitmweb it crashes with the following:
Error: Minified React error #185; visit https://reactjs.org/docs/error-decoder.html?invariant=185 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
at Jg (http://<IP_ADDRESS>:8081/static/app.js:7:8533)
at Object.enqueueSetState (http://<IP_ADDRESS>:8081/static/app.js:5:44513)
at Oc.setState (http://<IP_ADDRESS>:8081/static/app.js:1:3945)
at t.onViewportUpdate (http://<IP_ADDRESS>:8081/static/app.js:58:48851)
at t.componentDidUpdate (http://<IP_ADDRESS>:8081/static/app.js:58:48148)
at t.componentDidUpdate (http://<IP_ADDRESS>:8081/static/app.js:58:46203)
at Yi (http://<IP_ADDRESS>:8081/static/app.js:7:1828)
at dk (http://<IP_ADDRESS>:8081/static/app.js:9:5215)
at Dt.unstable_runWithPriority (http://<IP_ADDRESS>:8081/static/app.js:1:12961)
at gg (http://<IP_ADDRESS>:8081/static/app.js:5:40220)
Component Stack:
at AutoScrollWrapper (http://<IP_ADDRESS>:8081/static/app.js:58:45836)
at ConnectFunction (http://<IP_ADDRESS>:8081/static/app.js:51:11352)
at div
at MainView (http://<IP_ADDRESS>:8081/static/app.js:58:49804)
at div
at ProxyAppMain (http://<IP_ADDRESS>:8081/static/app.js:62:3048)
at ConnectFunction (http://<IP_ADDRESS>:8081/static/app.js:51:11352)
at Provider (http://<IP_ADDRESS>:8081/static/app.js:51:8664)
It seems to do it only when I use the scroll wheel on the mouse to scroll through the flows.
Mitmproxy: 8.1.0 binary
Python:    3.10.4
OpenSSL:   OpenSSL 3.0.3 3 May 2022
Platform:  Windows-10-10.0.22000-SP0

https://github.com/mitmproxy/mitmproxy/search?q=Minified+React+error+%23185&type=issues
https://github.com/mitmproxy/mitmproxy/issues/5198
#5507


Volume limiting during daytime

Currently, the volume of the music in the North canteen can be quite loud during the daytime. Not everyone can control the volume. It would be nice if the volume is reset to say 60% every day such that people are not bothered by loud music. This limit could be released after 4 pm.

I personally would call this feature creep as it makes the system very complex and thus harder to maintain, and it can easily be solved by just adjusting the max volume of the amplifier or instructing people with volume permissions to not go to the max volume.
If you want to go louder during an event, people should just change the amplifier in my opinion.
If there's an additional use case I'm missing please let me know.
Note that I am planning on changing the permission model for canteen reservations itself, meaning that during a reservation, there will be a QR code to scan / URL to visit that will give people access to thaliedje during the reservation

Note that I am planning on changing the permission model for canteen reservations itself, meaning that during a reservation, there will be a QR code to scan / URL to visit that will give people access to thaliedje during the reservation

This has just been released


change distribution management url

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
Please review http://spark.apache.org/contributing.html before opening a pull request.

Can one of the admins verify this patch?

retest this please

retest this please

retest this please


fainted

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 5
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[f]

/trunk merge


[request] Allow BSicon replacement without a config

Could the current English Wikipedia settings be used as default if there isn't a config found? It would let me focus more on translating the edit summary instead of copying and pasting the same list of templates.

This is doable. I'll likely implement a "global" config, but I think it would be better to host it on Commons.

@jc86035 You can create the global config at commons:User:JJMC89 bot/config/BSiconsReplacer/global. Wikis using the same templates can then just have a summary_prefix in the local config.

Thanks, that should work for most wikis.