401_CNN.py

I run it on WIN10.I have gotten the mnist.But when I run it,it says "RuntimeError: Dataset not found. You can use download=True to download it"

first, make sure that your conda is setup properly with the right environment
for that, check that which conda, which pip and which python points to the
right path. From a clean conda env, this is what you need to do
conda create --name maskrcnn_benchmark
source activate maskrcnn_benchmark
this installs the right pip and dependencies for the fresh python
conda install ipython
maskrcnn_benchmark and coco api dependencies
pip install ninja yacs cython matplotlib
follow PyTorch installation in https://pytorch.org/get-started/locally/
we give the instructions for CUDA 9.0
conda install pytorch-nightly -c pytorch
install torchvision
cd ~/github
git clone https://github.com/pytorch/vision.git
cd vision
python setup.py install
install pycocotools
cd ~/github
git clone https://github.com/cocodataset/cocoapi.git
cd cocoapi/PythonAPI
python setup.py build_ext install
install PyTorch Detection
cd ~/github
git clone https://github.com/facebookresearch/maskrcnn-benchmark.git
cd maskrcnn-benchmark
the following will install the lib with
symbolic links, so that you can modify
the files if you want and won't need to
re-build it
python setup.py build develop
or if you are on macOS
MACOSX_DEPLOYMENT_TARGET=10.9 CC=clang CXX=clang++ python setup.py build develop

您好，      您的邮件已收到。陈通宝广东工业大学

你好，你的邮件已收到，稍后将会查看，尽快给你回复。


I cant install mediapipe

OS Platform and Distribution
windows 10
Compiler version
No response
Programming Language and version
Python 3.11.3
Installed using virtualenv? pip? Conda?(if python)
pip 23.1.2
MediaPipe version
<IP_ADDRESS>
Bazel version
Not installed
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
None
OpenCV version (if running on desktop)
4.7.0
Describe the problem
C:\Users\USUARIO\Downloads>pip install mediapipe
ERROR: Could not find a version that satisfies the requirement mediapipe (from versions: none)
ERROR: No matching distribution found for mediapipe

Complete Logs
When i try to install mediapipe on windows 10, this gives me an error, specifically after using the

'pip install mediapipe'

command, the error is this:

'ERROR: Could not find a version that satisfies the requirement mediapipe (from versions: none )
ERROR: No matching distribution found for mediapipe',

when trying to install other pip packages this error doesn't happen, only when I try to install mediapipe

Hello @Davidd-u,
We see that you are using Python 3.11.3, Which is not supported by MediaPipe. We only support Python >=3.8 and <=3.10. Python 3.11 is only supported on MacOS and Linux. Please upgrade to the supported Python version.


'self' not defined in DataFrameView.update_section_height()

@andfoy I think you may need to add self here: https://github.com/spyder-ide/spyder/blob/master/spyder/widgets/variableexplorer/dataframeeditor.py#L486
It would be good to test the resize functionality afterwards to make sure it works :)

@andfoy, please work on @jitseniesen's suggestions :-)

Such a simple mistake! Sorry! Now it should be corrected in PR #3852, also I've tested it and it works like a charm :)


Model bootTraits executes same method multiple times if two traits have the same base name

Laravel Version: 5.6
PHP Version: 7.1
Database Driver & Version: MySQL

It's independent of versions so far I know.
Description:
The Illuminate\Database\Eloquent\Model::bootTraits() method executes the same bootXyzTrait method multiple times if the model uses two (or more) traits with the same base name.
This is simply resolvable by renaming the second trait but isn't cool if you want to extend a package trait for example.
Steps To Reproduce:
https://github.com/spatie/laravel-activitylog/issues/392
Create a model (App\User) with a trait (App\Traits\HasEmail) which uses a trait (Developer\Package\Traits\HasEmail) with a trait boot method (bootHasEmail).
class App\User {
  use App\Traits\HasEmail;
}

trait App\Traits\HasEmail {
  use Developer\Package\Traits\HasEmail;
}

trait Developer\Package\Traits\HasEmail {
  protected static function bootHasEmail() {
    var_dump('booting:HasEmail');
  }
}

// will output:
// booting:HasEmail
// booting:HasEmail

If the trait adds oberserver or other logic that should run just once it will run these things two times.

Can you submit a PR to resolve this?

PR with a fix submitted to https://github.com/laravel/framework/pull/24556, thanks for sharing.

Thanks for this fix. Will this get a backport to older versions?
And thanks for the fast fix - wrote this just as a copy of the linked issue on the road.


There is an error in the auth configuration description

File: /docs/zh_CN/access-control/http.md

password value is %P not %p

Thanks for your reminder. I can fix this by my own, But... since you share the credit, could you fix it via a PR?


🏷 Added a release of the manual on tags

We require a tag because creating a github release requires a tag.
The pipeline changed the file name to be able to easily pull the manual from https://github.com/InsaLan/backend-insalan.fr/releases/latest/download/manuel.tar.gz

would it be possible to include the short hash of the commit, or at least the tag name, in the manual archive name? That way we won't mistake one version for another when we inevitably have to download multiple versions

I could, but CD would definitely be harder since the filename would not be fixed.

Can releases have a name? If so, could we also try and distinguish them?  I arbitrarilly took the pattern "manuel*" as tag name, but anything could do, we just need tags. Shall I pick something looking like semver?


[Proposal] A new PacketSampling CRD

Describe what you are trying to solve

Traceflow works well for network flow diagnose, but sometimes users may want to take a look into the raw packet in the flow. Currently, antrea lacks the ability to capture raw packet in live traffic.
Describe the solution you have in mind

Add a new CRD to support the packet sampling future. This future will allow user to capture raw packet data for further diagnose.
apiVersion: crd.antrea.io/v1alpha1
kind: PacketSampling
metadata:
  name: tf-test
spec:
  timeout: 60             # a hard limit for all sampling session
  type: FirstNSampling    # support one type first
  parameters:
    number: 15            # the number of packets to be captured
  source:                 # same selector as Traceflow
    namespace: default
    pod: tcp-sts-0
  destination:
    namespace: default
    pod: tcp-sts-2
  packet:
    ipHeader:
      protocol: 6
    transportHeader:
      tcp:
        srcPort: 10000
        dstPort: 80
  fileServer:
    url: sftp://youtestdomain.com:22/root/test
  authentication:
    authType: “BasicAuthenticaion“
    authSecret:
      name: support-bundle-secret
      namespace: default

Describe how your solution impacts user flows

The default (first supported) capture method will have great peformance impact on user flows, see https://github.com/antrea-io/antrea/issues/5345#issuecomment-1665222167
Describe the main design/architecture of your solution

design doc
Alternative solutions that you considered

extend the current Traceflow CRD
Test plan

Additional context

@hangyan please help to estimate the implementation efforts for the new design, and help to confirm if this can be included in Antrea 1.14 or not. Thanks.

@shi0rik0 Please take a look if you are interested.


bug when setting a select value to emty (null or "") while changing number of attrs

When changing the number of attributes  on a select node, the node gets recreated.
But when you set the value to empty at te same time, this is'nt enforced on the select node, resulting in an inconsistent ui.
I made a fiddle to illustrate the problem : https://jsfiddle.net/cnrdz0n9/3/
(Just toggle a few times between "two" and empty)
(this worked until version 0.1.22)

Fixed in components branch


Add more fallbacks relations

How can I add more fallback relations e.g. I want en-US and en-GB to fallback to en, but I want ca to fallback to es.

@PolGuixe That's not really fallbacks if it's for a different language.
You have to initialize the lib differently, do a manuel test. Check the main file: https://github.com/AlexanderZaytsev/react-native-i18n/blob/master/index.js


[FEATURE REQUEST] ... "All Budget Accounts"

The "All Accounts" page is very useful, especially for looking at all upcoming scheduled transactions at once, but I have a ton of tracking accounts, and I would love a similar page that just showed all my budget accounts and not the tracking accounts.
Perhaps this could be accomplished with a toggle/filter within the all-accounts page.  This would also provide for the ability to see All-Tracking-Accounts as well.

+feature

We use Trello to track Feature Requests so that users can vote on what they'd like to see next. This is crucial for us since there are so many requests and so little time amongst those of us who work on the Toolkit.
Your feature request has been added here and will need to be manually sorted.
This issue can now be closed.

My guess is that this would/should just be implemented as a filter.
So just adding "Only Budget Accounts" or "Only Tracking Accounts" to the filter dropdown.

I'm not very good at this :(
This was actually a duplicate of: https://trello.com/c/TJe15xHB so I've closed the generated trello card in favor of that one.


feat: run ui and lexical through react compiler

This noticeably improves performance in the admin panel, for example when there are multiple richtext editors on one page (& likely performance in other areas too, though I mainly tested rich text).
The babel plugin currently only optimizes files with a 'use client' directive at the top - thus we have to make sure to add use client wherever possible, even if it's imported by a parent client component.
There's one single component that broke when it was compiled using the React compiler (it stopped being reactive and failed one of our admin e2e tests): https://github.com/payloadcms/payload/pull/7688/commits/150808f60808c91ef607b6fe550c9432a4d1de29 opting out of it completely fixed that issue
Fixes https://github.com/payloadcms/payload/issues/7366. Canary: 3.0.0-canary.79e92af

For context we've been using the compiler in prod on many Meta apps for several months now without any major issues. The issues we have seen usually involve some sketchy React pattern (that the compiler can't always detect statically) that results in unexpected behavior. However in those cases once the offending component/hook(s) are found it's trivial to disable compilation for just the offending functions without touching anything else. So in my opinion shipping the compiler to prod should be pretty safe.
If you have the ability to, running an A/B test can help get more perf data and changes to engagement, as well as be a proxy to check if anything might be broken (ie some metric goes down after shipping the compiler to prod)

For context we've been using the compiler in prod on many Meta apps for several months now without any major issues. The issues we have seen usually involve some sketchy React pattern (that the compiler can't always detect statically) that results in unexpected behavior. However in those cases once the offending component/hook(s) are found it's trivial to disable compilation for just the offending functions without touching anything else. So in my opinion shipping the compiler to prod should be pretty safe.

Awesome, that's reassuring enough, thank you!

If you have the ability to, running an A/B test can help get more perf data and changes to engagement, as well as be a proxy to check if anything might be broken (ie some metric goes down after shipping the compiler to prod)

We're planning to do performance testing in the future. Will definitely include tests with / without react compiler


test: DRAFT verify tests

What I did
Pointed
at_client to replace_encrypt_key_with_atchops
at_onboarding_cli - uptake_atchops_major_version_atonboarding_cli
at_chops - 2.0.0
at_lookup - 3.0.45

How I did it

How to verify it
Tests should pass

Closing the PR, as this PR was only to make sure all the tests are passing.


DBZ-7819 Allow log.mining.flush.table.name to optionally include schema

https://issues.redhat.com/browse/DBZ-7819

/packit test --labels oracle

/packit test --labels oracle

/packit test --labels oracle

The Oracle 21 XStream test is unrelated.


Query issue:  no function matches "ymonth(date)" name and argument types

I upgraded to latest fava-dashboards version. I'm seeing this error on my dashboard now:
error processing panel "Essentials MoM 🍔": failed to execute query SELECT YMONTH(date) AS date, CONVERT(SUM(position), 'USD') AS value WHERE account ~ '^Expenses:Food:Groceries': no function matches "ymonth(date)" name and argument types
I'm using beancount 2.3.6. If I run the exact same query on the query editor in fava I get a result. Any idea how to resolve this?
Not sure which package is the issue coming from.

I recently switched from beancount.query to beanquery, which is actively maintained and the successor of the BQL engine in beancount. See https://groups.google.com/g/beancount/c/iTdRuvZnE4E/m/o9V91WfgFQAJ for the deprecation notice.
beanquery is almost a drop-in replacement, but the function ymonth() you're using in your query got renamed to yearmonth(). Can you confirm that with this change the query works again?

That works OK now.

But another query is failing but works OK on editor:

SELECT year, CONVERT(SUM(position), 'USD', LAST(date)) AS value WHERE account ~ "^Assets:US:Fidelity:401K:Cash" and 'XXX Payroll' in narration GROUP BY year: operator "in(str, str)" not supported

I've seen the IN operator only used to check if value X is in a set Y.
afaics the correct query in this case is
SELECT year, CONVERT(SUM(position), 'USD', LAST(date)) AS value WHERE account ~ "^Assets:US:Fidelity:401K:Cash" and narration ~ 'XXX Payroll' GROUP BY year

Thanks. I have fixed the query on your suggestion.

Now everything is working normally. I will close this ticket.


add get cgroupdriver from RuntimeConfig to integration

Fixes: https://github.com/containerd/containerd/issues/9475

/retest

/ok-to-test

@akhilerm @thaJeztah PTAL,

f6dbb3ab6f17630e7e156110d040d2f6a25675d7

https://api.developer.coinbase.com/waas/blockchain/v1/{assetName

https://api.developer.coinbase.com/waas/blockchain/v1/{assetName

@Marymigmariell What's the meaning?

because this pr https://github.com/containerd/containerd/pull/9470 depend on current this pr , so anybody help me review it.
https://github.com/containerd/containerd/actions/runs/7162167958/job/19498815261?pr=9470

@fuweid PTAL


Drop php 5.4 and 5.5 support

No can do, at least not yet. We provide support for versions prior to 5.6 to accommodate the symfony ecosystem (as v2 introduced the PSR-7 bridge, and the last LTS still supports 5.4). We can look at this next year.

I'm thinking what we'll do is one last minor version supporting pre-5.6 versions, and then mark that an LTS version that will only receive security and/or critical bugfixes. We can then do a 2.0 version sooner rather than later, and move any new features to that.
I'll come back to this in a few weeks so we can proceed.

@weierophinney a few weeks go away, what about this?

Currently on vacation, @snapshotpl; will review next week.

I've changed my mind on this, and am scheduling this for 1.5.0.

@snapshotpl I've pushed the changes I've requested to your branch at this time, and will rebase to incorporate them into your own commits once I can verify the changes pass continuous integration.

Thanks, @snapshotpl


Add 'tesla' tag to internal dashboards for consistency

Unless there was a reason for these being left out?

These two dashboards are called by the other dashboards and offer nothing valuable on their own. I believe the reason they were not tagged was due to the number of issues being raised by people asking why the dashboards did not work (as they were clicking on them directly).

Thanks @ngardiner, makes sense, I will close this PR.


[Focus Mode] Network reference article

Simplified image names, corrected instructions, renewed screenshots.

Closing. Replaced by https://github.com/MicrosoftDocs/edge-developer/pull/2704.


Pass Provider

New Provider Request
Details

Name: Pass
URL: https://www.passwordstore.org/
Modes: read+write
GitHub projects: N.A.
API Documentation: https://www.passwordstore.org/

An example of a project that does have Pass integration is https://github.com/99designs/aws-vault
You can find their Pass implementation here.
https://github.com/99designs/keyring

Hey @marcofranssen this for provider suggestion
Do you want to create a pull request with the new provider? I can assist you if you need to

If no one else picks it up, over time if I find some spare moments I can try to give it a shot.

Cool, so the first volunteer will write a message here to prevent duplicate work.

Hi @kaplanelad, I would like to work on this. Can you please assign this to me and help me get started? Thanks!

Hey @marcofranssen,
You can follow this guide to initialize a new provider in Teller.
If you have any follow-up questions feel free to write them here, or we can jump to a short live session if you need to

@kaplanelad
Hi, if nobody started yet, I'd like to give it a shot

After having a look, Teller already supports gopass (https://github.com/tellerops/teller/blob/master/pkg/providers/gopass.go) which states to be 100% API-compatible to pass so I don't think we need a dedicated Pass implementation


None MSBuild items added for .fs files when compiling

If you compile a F# console project you see None items added for each .fs file in the project.
To reproduce:
.NET SDK 1.0.1 installed.

dotnet new console -lang F#
dotnet restore
dotnet build /v:diag > build.log

If you look in the build.log you see a None and a Compile item for Program.fs:
                   Compile
                       Program.fs
                   DebugSymbolsProjectOutputGroupOutput
                      ~/Projects/Tests/dotnet/bar/obj/Debug/netcoreapp1.1/bar.pdb
                           FinalOutputPath = ~/Tests/dotnet/bar/bin/Debug/netcoreapp1.1/bar.pdb
                           TargetPath = bar.pdb
                   DeployManifest
                       obj/Debug/netcoreapp1.1/bar.application
                           TargetPath = bar.application
                   IntermediateAssembly
                       obj/Debug/netcoreapp1.1/bar.dll
                   None
                       Program.fs

If you edit the .fsproj file and add  <EnableDefaultItems>false</EnableDefaultItems> the None items are not added.
The FSharp.NET.Sdk 1.0.1 NuGet package has the following in its FSharp.NET.Sdk.props:
<EnableDefaultCompileItems>false</EnableDefaultCompileItems>

So maybe this should also have EnableDefaultItems set to false. Not sure if EnableDefaultEmbeddedResourceItems and EnableDefaultNoneItems need to be set to false. It looks like just setting EnableDefaultItems to false would fix the problem.
In general this does not seem to break anything. The build still works. I only noticed it with recent VS for Mac builds since VS for Mac is now using the SDK imports to determine the files that should be displayed in the Solution window and I was seeing duplicate files. The duplicate files in VS for Mac was not caused by this, since due to another bug it is using resolved MSBuild items ignoring the conditions, but I whilst debugging I ran across this and initially thought it was the cause.

@mrward i checked and i can repro.
But i prefer try to not disable defaults (EnableDefaultItems) if possibile, because it mean duplicate that logic (like for Resource) and that mean more maintenance.
Also is not f# specific but more a corner case in sdk default items.
I think i fixed that directly with https://github.com/dotnet/sdk/pull/1163 , removing from None all the Compile items (in that special scenario).


fix release-notes kind

Please provide a description of this PR:
wrong kind broken gen-release-notes

In response to a cherrypick label: new pull request created: #50909


[Excel] (Table) Add Table.resize article section

Docs Build status updates of commit d603734:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-tables.md
:white_check_mark:Succeeded
View

docs/images/excel-tables-resize.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 354a2ca:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-tables.md
:white_check_mark:Succeeded
View

docs/images/excel-tables-resize.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit c360fd8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-tables.md
:white_check_mark:Succeeded
View

docs/images/excel-tables-resize.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fe60eb6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/excel-add-ins-tables.md
:white_check_mark:Succeeded
View

docs/images/excel-tables-resize.png
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


how to use tabbar view with same controller using controller.refresh() method

Hi ,
do you have any example shows how to use same controller with different items from api that should showing in every tab bar view ?
so when i hit on first tabbar it should get controller items from api that have category = 1 so i call  controller.refresh() to show this new data from api

No, I don't have any examples.
What you're doing doesn't seem correct. You should either use other widget to filter your categories or use one controller per tab.


delete minor unreachable code caused by t.Fatal

Signed-off-by: Abirdcfly<EMAIL_ADDRESS>Please take a look.

Thank you. I don't think this is the correct fix; the continue should be executed and an error raised. I'll send that.


Loading materialize libraries with jQuery breaks the Waves on buttons

The example i have written below, will load materialize.js with jQuery.
<!DOCTYPE html>

    <meta charset="utf-8">
    <title>title</title>
    <link rel="stylesheet" href="materialize.css">

Hello world

      Button

Doing so, makes the Waves on buttons broken. It simply wont work. However, Modals and other things still work if i do M.AutoInit();
I am in a special situation where i must load it with jQuery. So i cant just put the script tag inside the html like you normally would.

Try just calling  Waves.displayEffect(); after Materialize has been loaded


Updating .NET uninstall tool max version

Depends on https://github.com/dotnet/cli-lab/pull/170, I will convert this from a draft once we create a release with this change.
Fixes #22290

Thanks @tdykstra, the new version is released so feel free to merge this whenever you're ready!

Thanks @tdykstra, the new version is released so feel free to merge this whenever you're ready!


Desktop file produced by script/grunt mkdev isn't working

Following the installation guide for Linux in the README I ran into some problems using the .desktop file produced (See #1969 ). The atom icon shows up, but nothing happens when I hit it.
Running Debian Sid 3.14 with GNOME Shell 3.8.4.
A little summary:
htor@ultra:~/.local/src/atom$ script/grunt mkdeb
Running "mkdeb" task
Done, without errors.
htor@ultra:~/.local/src/atom$ cd /tmp/
htor@ultra:/tmp$ file atom-build
atom-build: Debian binary package (format 2.0)
htor@ultra:/tmp$ sudo dpkg -i atom-build
(Reading database ... 194042 files and directories currently installed.)
Preparing to unpack atom-build ...
Unpacking atom (0.95.0) over (0.95.0) ...
Setting up atom (0.95.0) ...
Processing triggers for desktop-file-utils (0.22-1) ...
Processing triggers for gnome-menus (3.8.0-2) ...
Processing triggers for mime-support (3.54) ...
htor@ultra:/tmp$ sudo dpkg -L atom
/.
/usr
/usr/share
/usr/share/applications
/usr/share/applications/Atom.desktop
/usr/share/pixmaps
/usr/share/pixmaps/atom.png
/usr/bin
/usr/bin/atom
htor@ultra:/usr/share/applications$ cat Atom.desktop
[Desktop Entry]
Name=Atom
Comment=A hackable text editor for the 21st Century.
Exec=/usr/share/atom/atom %U
Icon=atom
Type=Application
StartupNotify=true
Categories=GNOME;GTK;Utility;TextEditor;
htor@ultra:/usr/share/applications$

Hmm, I wonder what the /usr/share/atom/atom is..
htor@ultra:/usr/share/applications$ file /usr/share/atom/atom
/usr/share/atom/atom: cannot open `/usr/share/atom/atom' (No such file or directory)

Apparently nothing. Let's change that to where atom was actually installed:
Exec=/usr/local/bin/atom

Now it works and I can launch atom normally.

Assuming fixed.  If this is still reproducible on the latest Atom version please comment here!


[Feat]: Workflow for Closing Old Issues

This feature aims to automate the process of managing old issues, ensuring that the repository remains organized and focused on relevant tasks. By automatically closing old issues, it reduces manual overhead for repository maintainers and helps keep the issue tracker tidy and up-to-date.

@BamaCharanChhandogi please assign me this issue under gssoc


feat(gateway): resolve DNS in the background

When receiving a connection request for a DNS resource, the gateway will first resolve the domain name and only then respond with the ICE credentials needed by the client to establish the connection. This was necessary because prior to #4994, the response sent to the client included the resolved IPs for the domain name.
With #4994, the gateway only needs to resolve the domain name in order to correctly map the client's proxy IPs upon incoming traffic. As a result, we can remove the DNS resolution from the hot-path of connection setup and always directly accept a client's connection request.
Currently, if the DNS resolution fails, we never respond to the client, thus forcing it into a timeout to try and establish a new connection. By always accepting the connection, this case simply doesn't occur.
This change makes a particular edge-case more obvious although not necessarily worse in my opinion: If DNS resolution fails, the client is currently left hanging without a response and will eventually timeout the connection. With this change, the client has a connection but due to the failed DNS query, packets for this DNS resource will be dropped as unallowed packets. Packets to other DNS resources will send an allow-access request and resolved accordingly.
If DNS resolution on the gateway doesn't work reliably, almost nothing of firezone works. The only difference with this PR is that a failed connection (i.e. behaviour on main) is self-healing after ~20s whereas with this change the user would have to sign-out and in again in order to clear all connection state.

If DNS resolution on the gateway doesn't work reliably, almost nothing of firezone works. The only difference with this PR is that a failed connection (i.e. behaviour on main) is self-healing after ~20s whereas with this change the user would have to sign-out and in again in order to clear all connection state.

We can improve on this with some refactoring: The gateway already has functionality to refresh a DNS query if it sees packets for a proxy IP but no traffic for the translated IP. We can adopt this to also refresh the DNS query if we see traffic for a proxy IP and don't even have a real IP for this domain.

If DNS resolution on the gateway doesn't work reliably, almost nothing of firezone works. The only difference with this PR is that a failed connection (i.e. behaviour on main) is self-healing after ~20s whereas with this change the user would have to sign-out and in again in order to clear all connection state.

We can improve on this with some refactoring: The gateway already has functionality to refresh a DNS query if it sees packets for a proxy IP but no traffic for the translated IP. We can adopt this to also refresh the DNS query if we see traffic for a proxy IP and don't even have a real IP for this domain.

I just attempted to implement this, I think it is actually reasonably clean.

If we wait with this PR until the new clients are released, we can make it substantially simpler by not worrying about backwards-compatibility!

This is blocked on being able to remove the backwards-compatibility layer for clients < 1.1. We could theoretically ship it earlier but it is a lot simpler to just delay this until we are happy to no longer support < 1.1 clients on newer gateways.

Will essentially be replaced by https://github.com/firezone/firezone/pull/6732.


-Werror -Weverything may be too fragile

We're adding new warnings in minor releases as well, as those are not considered breaking changes. Therefore, rgbasm -Werror -Weverything may break in the next release, especially considering that def-less symbol definitions should be deprecated soon. (Either v0..6.1 or v0.7.0, though the latter is more likely atm.)
I'd suggest either doing without -Werror, or being more selective than -Weverything; -Wall -Wextra may be indicated. (Take the latter with a grain of salt, as -Wobsolete is in the default flag set anyway.)

OK, so the truth is I like when it breaks. :)
I watch releases of RGBDS and I plan on always upgrading to the latest one asap, so that the samples are always following the best practice from RGBDS. It happened recently with the old macro syntax deprecation. I put the version of RGBDS I tested the sample against in the README.
Are there any rgbasm warnings that might generate false positives? GCC and clang have quite a lot of those, so -Weverything is not usable there. But for rgbasm, my intuition was that there were no risk of false positives.

The warnings that have several levels can currently present false positives, but they're really fringe.

OK, I think there is a good compromise.
I will keep -Werror -Weverything for the samples, but I will add a note in the book mentioning that -Wall -Wextra, while not covering as many warnings, is less prone to false positives.
I will also be proactive in checking the toolchain/sample compatibility.

Thanks!


[crunchyroll] This video is DRM protected

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm reporting that yt-dlp is broken on a supported site
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Germany
Provide a description that is worded well enough to be understood
Since 2024.06.18 16:55 i get the message
ERROR: [crunchyroll] <Partnumber>: This video is DRM protected
I tried it with other links, but i get only this message.
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--user-agent', 'Crunchyroll/1.8.0 Nintendo Switch/<IP_ADDRESS> UE4/4.27', '--username', 'PRIVATE', '--password', 'PRIVATE', '--sub-langs', 'de-DE', '--write-description', '--write-thumbnail', '--write-subs', '--progress', '--extractor-args', 'crunchyrollbeta:hardsub=de-DE', '-vU', 'https://www.crunchyroll.com/de/watch/GK9U3EV5Q/castle-of-sand']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [12b248ce6] (zip)
[debug] Python 3.11.2 (CPython x86_64 64bit) - Linux-6.1.0-21-amd64-x86_64-with-glibc2.36 (OpenSSL 3.0.11 19 Sep 2023, glibc 2.36)
[debug] exe versions: ffmpeg 5.1.5-0 (setts), ffprobe 5.1.5-0, phantomjs 2.1.1
[debug] Optional libraries: Cryptodome-3.11.0, brotli-1.0.9, certifi-2024.06.02, curl_cffi-0.5.10, mutagen-1.46.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.2.1, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1820 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[debug] Loading<EMAIL_ADDRESS>from cache
[crunchyroll] Extracting URL: https://www.crunchyroll.com/de/watch/GK9U3EV5Q/castle-of-sand
[crunchyroll] Requesting token
[crunchyroll] GK9U3EV5Q: Downloading object info JSON (cms)
[crunchyroll] GK9U3EV5Q: Downloading stream info
[crunchyroll] Available hardsub languages: en-US, de-DE, es-419, es-ES, fr-FR, it-IT, pt-BR, ru-RU, ar-SA
[crunchyroll] To extract formats of a hardsub language, use "--extractor-args crunchyrollbeta:hardsub=<language_code or all>". See https://github.com/yt-dlp/yt-dlp#crunchyrollbeta-crunchyroll for more info
[crunchyroll] GK9U3EV5Q: Downloading hardsub-de-DE MPD manifest
[crunchyroll] GK9U3EV5Q: Invalidating stream token
[crunchyroll] GK9U3EV5Q: Downloading chapter info
[info] GK9U3EV5Q: Downloading subtitles: de-DE
ERROR: [crunchyroll] GK9U3EV5Q: This video is DRM protected
Traceback (most recent call last):
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1606, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1762, in __extract_info
    return self.process_ie_result(ie_result, download, extra_info)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1821, in process_ie_result
    ie_result = self.process_video_result(ie_result, download=download)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 2825, in process_video_result
    self.raise_no_formats(info_dict)
  File "/usr/local/bin/yt-dlp/yt_dlp/YoutubeDL.py", line 1104, in raise_no_formats
    raise ExtractorError(msg, video_id=info['id'], ie=info['extractor'],
yt_dlp.utils.ExtractorError: [crunchyroll] GK9U3EV5Q: This video is DRM protected

I tried it with crunch-cl 3.6.6 and it works fine. I can download audio and video:
:: ✔ Logging in
:: ✔ Parsed url 1
:: ✔ Loaded series information for url 1
:: Unnamed Memory Season 1 (Unnamed Memory)
:: 	1. Ein Schloss aus Sand » S01E11
:: Downloading Ein Schloss aus Sand to 'Unnamed Memory 11 – Ein Schloss aus Sand.mp4'
:: 	Episode: S01E11
:: 	Audio: ja-JP
:: 	Subtitles: de-DE
:: 	Resolution: 1920x1080
:: 	FPS: 23.97
:: Downloading ja-JP audio  22.68 MiB   8.50 MiB/s [#####] 100%
:: Downloading video #1      1.32 GiB  13.42 MiB/s [#####] 100%:: Downloading subtitles    de-DE                                                                                :: Generating output file   [####] 100%
It seems like it's not really DRM.

https://github.com/crunchy-labs/crunchy-cli/issues/438

is this the end for cr in yt-dlp or is there a chance it will get fixed to work with the new drm?

is this the end for cr in yt-dlp or is there a chance it will get fixed to work with the new drm?

so yt-dlp doesn't work with DRM'd content, and never has. CR has a non-drm stream which is what yt-dlp picks up and downloads. At the moment CR has 3 streams, 2 that are DRM and one that is not. Seems yt-dlp is having trouble grabbing that stream.

This could be the end of the Crunchyroll extractor for yt-dlp, unless someone is able to find a new Non-DRM protected endpoints other than the Switch. I really hope it will not result in the removal of the extractor, as downloading subtitles and thumbnails still works with --ignore-no-formats-error thanks to bashonly's fix in https://github.com/yt-dlp/yt-dlp/pull/10005.

This could be the end of the Crunchyroll extractor for yt-dlp, unless someone is able to find a new Non-DRM protected endpoints other than the Switch. I really hope it will not result in the removal of the extractor, as downloading subtitles and thumbnails still works with --ignore-no-formats-error thanks to bashonly's fix in #10005.

Circumventing DRM is not in the goals of yt-dlp, so it could be necessary to go with another project. The problem is: there is no other projects (that is not built upon yt-dlp, at least)… :/

Circumventing DRM is not in the goals of yt-dlp, so it could be necessary to go with another project. The problem is: there is no other projects (that is not built upon yt-dlp, at least)… :/
https://github.com/stratuma/Crunchyroll-Downloader-v4.0

Personally speaking, I deactivated the crunchyroll subscription renewal last time (although the subscription will remain active until November so I'm still using it until that date). When they asked why I decided to cancel my subscription I actually decided to say the truth that I did not like their DRM policy, probably they don't care but I wanted to be clear about it, I pay for stuff that I can backup.
I don't like DRMs at all, if I can't backup my favorite series (which I had no interest in sharing online anyway and which it seems to me can still be found with all these DRMs) I prefer not to be a subscriber and dedicate myself to other hobbies ( in particular video games on Gog where they are DRM-free and manga/books on Amazon and Kobo (where DRM is easy to be eliminated for now).
A few years ago I stopped buying movies because it had become too complex to remove the DRM from films purchased from iTunes and similar so for me it's not the first time I decided to quit paying for stuff that use a drm that is too strong (even if it seems clear that those who want to remove it at all costs always find a way (those who will share it online)... but I get bored if the method is too complex, for example I took a look at crunchyroll downloader 4.0... and make an android virtual machine just to get the L3 widevine keys (that maybe in the future will only work for low quality videos like on netflix) is not something that I want to do... too much trouble, wasted space on my pc, just for something that I should be able to do by default... crunchyroll don't even sell anime in digital download drm-free so I cannot even decide to pay for that instead of subscriptions).
One thing I've learned over the years is that in these cases it's better not to pirate at all... otherwise companies will take it as an excuse for decreasing revenues and will increase DRMs instead of doing the opposite. For my part, I prefer to give a clear signal "your product is not fundamental, if you use DRM that is too invasive I prefer to ignore the product".
In the meantime, I still want to thank yt-dlp team who for many years have always provided constant updates to this splendid tool. I built a small interface for my personal convenience but the "dirty work" has always been done by yt-dlp (except for a period when crunchyroll was in beta, there I had created a small tool for beta crunchyroll which however I was unable to keep updated).
I am still happy to have the interface to be able to continue using yt-dlp on YouTube and other similar sites where it is still the best tool.
I know that, if there is any not drm stream still on crunchyroll you will find it.
If there is not I suggest to keep the subtitles download part like Tama47 suggested... Even if it's easy to find it by using the developer tools I think using yt-dlp is still more convenient since it will rename it the correct way.
I know that messages like this are slightly offtopic but I hope an exception is made given the particular case. If you want you can delete this message (or ask me to do so).

This could be the end of the Crunchyroll extractor for yt-dlp, unless someone is able to find a new Non-DRM protected endpoints other than the Switch. I really hope it will not result in the removal of the extractor, as downloading subtitles and thumbnails still works with --ignore-no-formats-error thanks to bashonly's fix in #10005.

Circumventing DRM is not in the goals of yt-dlp, so it could be necessary to go with another project. The problem is: there is no other projects (that is not built upon yt-dlp, at least)… :/

I don't think anyone is asking for that (as much as I'm sure many of us secretly want that) but if the metadata and subtitles are still not drm protected it would be nice to at least grab those while we can.

@federicorosso1993

I am still happy to have the interface to be able to continue using yt-dlp on YouTube and other similar sites where it is still the best tool.

You can still download anime on YouTube for free from channels like Muse Asia and AniOne Asia. Although the former offers low-quality hardsubbed 720p, the latter does offer 1080p softsubs. BiliBili also works, even their 4K upscaled videos if you prefer. An even better option is to download from ABEMA TV, which offers high-quality DRM-free 1080p downloads. While not as high quality as Crunchyroll (1.5 GB vs ~750 MB), it is the next best thing in terms of quality. Plus, the subscription price is cheaper due to the conversion rate. But you will need a VPN, if you don’t live in Japan.

If there is not I suggest to keep the subtitles download part like Tama47 suggested... Even if it's easy to find it by using the developer tools I think using yt-dlp is still more convenient since it will rename it the correct way.

I agree. Even though you can easily find the thumbnails and subtitles through the developer tools, yt-dlp provides a much more convenient interface. With that said… you can now easily pair CR subtitles with other releases like ABEMA and so on.

README:
This is not the place to air your grievances with DRM or Crunchyroll, and this is definitely NOT the place to discuss DRM circumvention or link to DRM circumvention resources. Anyone who does the latter will be blocked from this repo without further warning.
The only reason this issue isn't being locked is in case anyone has any info/leads on the availability of non-DRM formats.


Error message for removal of last org admin

Issue #5463, messaging per @anangaur
Scenario is current user (org admin) as only admin deleting their membership or changing self to collaborator... which we block.

@scottbommarito Tests existed, but only verified exception type... added verification of the message.


Volume Should Ignore Ring/Silent Switch

Right now, any video will be silent if the device has its ring/silent switch set to silent.
In most other apps, having the switch in either mode doesn't affect video playback.

Looks like playsInSilentModeIOS is the answer: https://docs.expo.io/versions/latest/sdk/audio/#arguments-1


how to remove anonymous user and go to login page

I don't know if this is the right place to ask this but when you start the app, It goes directly to the anonymous user, how to remove this behavior and go to the login page instead, thanks.

A quick fix would be to add the following in App.js
componentWillMount() {
    const { user } = this.props;
    const { router } = this.context;

    // If this page is restricted, go to loginPage first.
    // (But pass on this page's path in order to redirect back upon login)
    if (!user) {
      const path = this.props.location.pathname;
      router.push(`/login?redirect=${path}`);
    }
  }

But this may not be bullet-proof, but should get you going.  I will check in a better solution when I have time.

It does the trick, thanks and keep up the good work!

Hello, with the new react router 4, this code to redirect no longer works, any hint on how to do this , thanks for your work.


Retire software?

Should adblock be retired, and suggest switching to https://github.com/StevenBlack/hosts? It seems to have almost all of the features of this piece of software, and more.
What is missing:

Updates autodetection. Ok, I can live with that.
Ability to switch off the blocks in one line. That's annoying.

I'm not sure if the merge is really pertinent. If we take my favorite host (google-analytics), the file once merged contains multiple entries. The ones commented out from http://someonewhocares.org/hosts/zero/ but also entries still active from other sources.
As you mentioned earlier, given the number of sites broken when you block analytics (so sad…) you don't want to multi-edit the hosts file manually each time you want to access that content.
This is something I should discuss upstream but it's just my initial gut feeling with the project.

Alright, I just read the doc and my case is made possible via white listing, nice!

The adblock piece of software only manages downloading, updating and reverting a hosts file. It trusts an external source (currently Dan Pollock's hosts file) for listing the domains.
The referenced https://github.com/StevenBlack/hosts today only merges and consolidates several sources into a single hosts file.
So really, they are complementary rather than competitive 🙂
One could suggest to switch to the consolidated ads + malware source hosts file from https://github.com/StevenBlack/hosts, or that some combinatories are suggested for installation by adblock, since https://github.com/StevenBlack/hosts offers options beyond ads and malware: fakenews, gambling, porn, social. This is not a need I have at the moment: Dan Pollock's file fits my needs.


[BUG] cython.final extension type with fused type method

Describe the bug
When using cython.final decorator for a cdef class that has a method with an argument of  fused type, the generated C code lacks some prefix __pyx_fuse_0 and __pyx_fuse_1 in a few places.
The code compiles without error. But when loading/importing one gets an error:
ImportError: dlopen(XXX.cpython-311-darwin.so, 0x0002): symbol not found in flat namespace '___pyx_f_54_cython_magic_570ee92356009cb8197cfd5d5ebfbcc01ad73c1d_6AClass_c_sum'

Code to reproduce the behaviour:
import cython
%load_ext cython

# %%
%%cython

import cython
import numpy as np

ctypedef unsigned char uint8_t
ctypedef unsigned short uint16_t
ctypedef fused uint_fused:
    uint8_t
    uint16_t

@cython.final
cdef class AClass:
    def compute_sum(self, a):
        if a.dtype == np.float32:
            return self.c_sum[uint8_t](a)
        else:
            return self.c_sum[uint16_t](a)

    cdef c_sum(self, uint_fused[:] a):
        cdef:
            int i
            uint16_t s = 0
        for i in range(a.shape[0]):
            s += a[i]
        return s

# %%
aclass = AClass()
aclass.compute_sum(np.arange(10).astype(np.uint16))

Note that without the @cython.final, everything works as expected without error.
Expected behaviour
No error, result 45.
OS
macOS
Python version
3.11.7
Cython version
3.0.8
Additional context
No response

@da-woods You seem active right now and this is a road blocker for one of my projects.

I'll try to have a look in the next week or so. I should warn: this fused stuff is usually quite heavy going to debug and fix, so I'm really not going to promise anything quick.

Thanks for having a look.
A striking argument for Cython is allocating numpy arrays in Python and then use memoryviews in Cython, in particular in a prange. Then one quickly wants c++ templates, the next best thing being fused types, last resort tempita.
So the combination of c++ with memoryviews would be super helpful, but the C(++) structure of memoryviews is quite hidden.
TLDR: If Cython works, it‘s very cool. If it doesn’t, try n error can drive you crazy 😜

@da-woods @matusvalo Thanks for the fix, really appreciated 🥳


general weirdness with combined conduits?

Is there an existing issue for this?

[x] I did not find any existing issues.

Current Behavior
rover sent me on that one,
i placed a me conduit next to an already existing me conduit line and it did that
it seems that placing me conduits inside already existing power conduits or the opposite causes general issues when making new contacts at placement time.
i also got crashes from it in the past https://discord.com/channels/914926812948234260/1295835279285813388/1295835279285813388, but the same crash happened to deepa without making a contact on placement.
Expected Behavior
No response
How to Test
No response
Local Environment

Operating System:
Java version:
Minecraft Version:
Version:
Other Mods: Applied Energistics 2, Mekanism...

Any Additional Information?
No response

From what I've heard and seen in the Monifactory discord server, this looks like an issue with Effortless Building


dotbot: fix build

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@ofborg build dotbot

@ofborg build dotbot

Thanks for the PR @Bot-wxt1221, I've been meaning to get into this but I don't have the time.
If you update the package does it still use the wrong version of pytest? I raised an issue on their repo and they told me that the reason the tests weren't running is that they should actually run with tox instead of pytest directly, can you try that?

@ludat Actually. We expect AssertionError here. However, nix just report it.

@ludat I try to fix the problem.


Nameable

Project requirements
I did the below tasks:

[ ] Created nameable class with methods
[ ] Created base decorator class with methods
[ ] created capitalize class with methods
[ ] Created  trimmer class with methods

Hello @samiullahbahadur ,
Your linter checks are not showing because you've not named your files well.

It should be .github/workflows. Could you correct that now so that I can proceed with the review?


Help Login

I just did follow the steps and made it possible to run but no matter what password I give or input I can enter the backend.
Is there anything I should be doing

try make sure your first_superuser is email or just don't set it.
also on other note: i can't login to front end either after rebuild the project from scratch. i can enter postgres db tho with it

try make sure your first_superuser is email or just don't set it.
also on other note: i can't login to front end either after rebuild the project from scratch. i can enter postgres db tho with it

I experienced the same issue on my first try. Turns out I missed this step during the setup as it is tucked about half of the way down in the documentation.
Frontend development

Enter the frontend directory, install the NPM packages and start the live server using the npm scripts:

cd frontend
npm install
npm run serve

After that everything ran as expected

I experienced the same issue on my first try. Turns out I missed this step during the setup as it is tucked about half of the way down in the documentation.
Frontend development

Enter the frontend directory, install the NPM packages and start the live server using the npm scripts:

cd frontend
npm install
npm run serve

After that everything ran as expected

ok i did found the problem, if you regenerating the project or restarting the project make sure you drop all the previous database table from postgres data in the docker. it's really hidden in the docker.  here's what i did solve the issue using this stackoverflow method

@pramadito that was the correct answer you have to make docker-compose to delete everything cus it saves the state. I guess is more I'm a newbie in docker

Thanks for the help here everyone! 👏  🙇
Thanks for reporting back and closing the issue @subspider  👍

Sorry for the long delay! 🙈  I wanted to personally address each issue/PR and they piled up through time, but now I'm checking each one in order.


python3Packages.ha-philipsjs: init at 2.7.4

Motivation for this change
Python library to interact with Philips TVs with jointSPACE API
https://github.com/danielperna84/ha-philipsjs
This is a Home Assistant dependency.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123181 at 32aaf7c6 run on aarch64-linux 1

3 packages built successfully:

home-assistant
python38Packages.ha-philipsjs
python39Packages.ha-philipsjs


pre-commit-config: Upgrade ruff

This is the half of #1867 that passes.

#1866

Description
Please include a summary of the change and which issue is fixed. Please also
include relevant motivation and context.
Fixes # (issue)
Checklist:

[ ] I have added the relevant tests for this change.
[ ] I have added an item to the Pending section of docs/changes.rst.

@matthiask

Thanks!


Performance fix for comparison in FlatString

As reported in #1838, a lot of unnecessary unboxings were done when comparing bytes in FlatString, this should be fixed by short-cutting the access to the bytes of a FlatString via _items.
Also while I was at it, the operation is now optimized when working on any FlatText

Reroll: Fixed remaining test and offset issues in less than method

sav not uptodate?

K reroll'd


Show button images in iOS8

When using iOS8 project, browser's bottom-left button images are now shown. I fixed in this pull request.
Thanks for good library!
Current

Fixed

Thank you for this pull request. KINWebBrowser 1.3.0 features this functionality.


[cleanup] replace wrap_string_with_quotes with dbt.string_literal

Closes #418

😬

@b-per i was really hoping this would "just work" :((((((

We can't just replace our macro by dbt.string_literal because we handle nulls/empty strings differently.
I think that we should:

handle nulls differently still
but call dbt.string_literal when the value is not null

There is some benefit in knowing when a value is null or when it is the empty string (which is actually different). I will create a new PR for it.


The detail about the generation of tokens?

Hello, guys. Thanks for the great work – it's been super helpful.
I'm trying to replicate this method you've shared. Actually, I'm curious about how you generate tokens based on corresponding item features?
More specifically, how you deal with the side information of item (e.g., cate/brand)? I'd love to know that whether you treat sideinfo as extra tokens (e.g., <item_id><cate_id><brand_id>), or input all features of the item into MLP and take its output as a token, or some method else?
Really appreciate your help on this!

Hi, the most straightforward way to encode category information and brand information would be to use them (together with id information) to derive the content token representation $\Phi_i$s. An MLP should work as an initial baseline?

Thanks for your response.
In light of that, I'm curious about what side information you have utilized to derive the content token representation?
Additionally, do you derive this representation by simply concatenate item id & sideinfo together in practice? Or use an MLP after concatenation?
Looking forward to your reply!

Hi, the most straightforward way to encode category information and brand information would be to use them (together with id information) to derive the content token representation $\Phi_i$s. An MLP should work as an initial baseline?

Thanks for your response.
In light of that, I'm curious about what side information you have utilized to derive the content token representation?
Additionally, do you derive this representation by simply concatenate item id & sideinfo together in practice? Or use an MLP after concatenation?
Looking forward to your reply!

Hi - we cannot disclose the specific features we used in our internal settings, but GRs used a strict subset of features used by our DLRM baselines. We'll think about how to make this particular aspect easier to test.


Add vpc name

Addition of Name Output And CircleCI Fixes

Added vpc_name as an output
Modified circleci checks:

Install missing curl binary for tflint checks
Add default aws region required by aws module
Modified init and validate steps to traverse directories
Removed invalid (in 0.12.x) -check-variables flag

Thanks, Ben!
v2.9.0 has been just released.


Basic Usage with TS

Hello, is there any example of this library using TypeScript?

There is a wrapper for using it in Typescript but I wouldn't recommend it. Either way, here it is. If you get it working, it'd be nice if you let me know.
https://github.com/sdkcarlos/artyom.js/issues/64#issuecomment-403797114


[SPARK-25460][BRANCH-2.4][SS] DataSourceV2: SS sources do not respect SessionConfigSupport

What changes were proposed in this pull request?
This PR proposes to backport SPARK-25460 to branch-2.4:
This PR proposes to respect SessionConfigSupport in SS datasources as well. Currently these are only respected in batch sources:
https://github.com/apache/spark/blob/e06da95cd9423f55cdb154a2778b0bddf7be984c/sql/core/src/main/scala/org/apache/spark/sql/DataFrameReader.scala#L198-L203
https://github.com/apache/spark/blob/e06da95cd9423f55cdb154a2778b0bddf7be984c/sql/core/src/main/scala/org/apache/spark/sql/DataFrameWriter.scala#L244-L249
If a developer makes a datasource V2 that supports both structured streaming and batch jobs, batch jobs respect a specific configuration, let's say, URL to connect and fetch data (which end users might not be aware of); however, structured streaming ends up with not supporting this (and should explicitly be set into options).
How was this patch tested?
Unit tests were added.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3383/
Test PASSed.

Test build #96480 has started for PR 22529 at commit b6f8880.

Test build #96480 has finished for PR 22529 at commit b6f8880.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96480/
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3386/
Test PASSed.

Merged build finished. Test PASSed.

Test build #96484 has started for PR 22529 at commit b080b0d.

Test build #96484 has finished for PR 22529 at commit b080b0d.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96484/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3387/
Test PASSed.

Test build #96486 has started for PR 22529 at commit b080b0d.

Test build #96486 has finished for PR 22529 at commit b080b0d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96486/
Test PASSed.

cc @cloud-fan and @gatorsmile

LGTM

Merged to branch-2.4.

Thank you, @HyukjinKwon and @cloud-fan .

Thanks, @dongjoon-hyun and @cloud-fan.


Auto Updater

Implement inline Auto Updates via Electron update service.

[ ] Get code signing and notarizing working on Windows and Mac
[x] Implement application dialogs
[x] Tie up Github releases

Since only dmg, AppImage and nsis targets support auto-updates through electron-updater, we might want to create a custom solution that just informs the user about a new version being available, downloads it, but requires the user to run the installer manually.


New Spotify web player has a new interface and no internal Last.fm scrobbling option anymore

This seems to be the new web player interface that Spotify is rolling out to all users at the moment:

There are no settings anymore, so you cannot add your Last.fm username and password anywhere. And the playbar is at the bottom now like in their desktop app, so your extension does not recognise the tracks. Users who already have this new interface and rely on it due to restrictions not allowing the installation of the desktop app cannot scrobble to Last.fm anymore.

We added support for new Spotify web player in v1.61.0.
I tested the connector yesterday, and it worked fine.

Oops, I just noticed I had disabled the extension in Opera. It's scrobbling now like it should, great and sorry for that! ;-) Is it supposed to still work with the old interface as well, as I have read several reports recently that users suddenly have problems with scrobbling?

Is it supposed to still work with the old interface as well, as I have read several reports recently that users suddenly have problems with scrobbling?

Yes, I haven't remove the old connector, and the extension should work with old web player as well (I don't know if it still accessible, though).

I have just checked the old player URL in Opera and Chrome, and I still get the old interface there:
https://play.spotify.com/
The screenshot above was made while accessing https://open.spotify.com/, but I don't know if this works for everyone in every country with these browsers.

I have just checked the old player URL in Opera and Chrome, and I still get the old interface there:
https://play.spotify.com/

It redirects me to new website every time I click on URL. Looks like Spotify enable new version of their website eventually.
Anyway, I have no intention to remove the old connector, and I'll keep support for old website for some time. So I guess, the issue is resolved, right?

I have checked that URL again with MS Edge, and it also shows me the old interface, so I guess Spotify has not established the redirect to open.spotify.com for all users yet. Anyhow, keeping support for it is a good idea, and the issue is resolved.


Refactor cirq.has_unitary into a list-of-strategies implementation

Also move it into its own file

Automerge cancelled: A status check is failing.


Query [Bug]

Can't create content.
Error in: EfContentQueryStore line 34 Method: Query
An exception of type 'System.NullReferenceException' occurred in EntityFramework.Core.dll but was not handled in user code
Additional information: Object reference not set to an instance of an object.
InnerExceptionMessag: Message = "An attempt was made to use the model while it was being created. A DbContext instance cannot be used inside OnModelCreating in any way that makes use of the model that is being created."

Yeah I am getting the same, I am hesitant to touch this code as it is due to be replaced by the work @sebastienros is doing

oh it will be good :) on Brochard migrated on Yessql?


Simplify customize

Description:
Customize has gotten very complex. Too many different features got packed into a single config entry. This prevented us from doing proper config validation or making a config UI.
This PR breaks it up into three parts: exact match, domain match, glob match.
Example entry for configuration.yaml (if applicable):
homeassistant:
  customize:
    light.kitchen:
      friendly_name: Beer
  customize_domain:
    light:
      icon: mdi:cow
  customize_glob:
    'light.*_kitchen':
      entity_picture: /local/wut.png

Checklist:
If the code does not interact with devices:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

Is #5909 solve with that?

No, that's a bug in the frontend.

Thanks for this feature @balloob. I saw @andrey-git has a similar pr for zwave to bring it back to its former glory.
Nice job guys :) !

If I understood this correctly, then you would need three separate includes if you were using split YAML files - assuming you were using all three methods?

@rpitera
You have 2 options to avoid split files

Always use customize_glob :)
Put domains in glob form first, i.e. light.*

Use packages

configuration.yaml
homeassistant:
  packages:
    p1:
      homeassistant: !include customize.yaml

customize.yaml
customize:
   light.bed:
    ...
customize_domain:
   light:
    ...
customize_glob:
   light.b*:
    ...

Thanks!  And in the case that you're simply using per entity customization like I am doing now, I assume nothing would change, right?   I ask only because balloob added the breaking change tag.

Does this invalidate the list-style customize?

@rpitera
Yes, if you are not using globs then the old format is working as before.
@happyleavesaoc
The list is gone. Now its a trinity of customize (old format), customize_domain, customize_glob.

Thanks!


Struggling to make this work via NPM with purgcss

Describe your question
Are EasyMDE classes namespaced?
What do I (can I) whitelist with PurgeCSS to get it to overlook EasyMDE? As far as I can tell, this isn't a simple task - but perhaps I'm missing something.

EasyMDE creates the HTML elements when the page loads, so that will probably confuse PurgeCSS.
You could try adding all the classes in https://github.com/Ionaru/easy-markdown-editor/blob/master/src/css/easymde.css to the whitelist, but this can be tedious and isn't guaranteed to work with future updates.

Hi Ionaru.
Other libraries work the same way (creates elements when instantiated) but they only have one or two namespaces or patterns to put in the passlist. (e.g. Pikaday, which starts all classes with "pika-").

One way around it would be to add the following at the beginning and end of the EasyMDE css file:
/*! purgecss start ignore */
/*! purgecss end ignore */


improve pkg/k8sutil/pod.go ut coverage

What type of PR is this?
/kind test
ref https://github.com/Project-HAMi/HAMi/issues/569
What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Currently the test coverage of this file has reached 100%


GEODE-3618: Move GfshRule into geode-junit

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

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

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

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

[x ] Does gradlew build run cleanly?

[ x] Have you written or updated unit tests to verify your changes?

[ x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Hi @jaredjstewart I don't actually see the change to the gradle file that is going to include this in geode-junit. Also, it seems that the $GEODE_HOME/lib dir does not contain geode-junit. Most of the users are going to configure maven/gradle for getting the geode-junit jar, so this probably is not a huge issue.

@sbawaska The change that includes this in geode-junit was to move the source files from geode-assembly/src/test/ into geode-junit/src/main/.  Geode-junit doesn't end up in $GEODE_HOME/lib, so I think that behavior is expected.
The way I envision this rule to be used is one of the following:

For use inside of Geode (i.e. tests inside of geode-assembly), the Gradle build sets $GEODE_HOME to point at geode-assembly/build/install/apache-geode.
For external users, get the geode-junit jar through gradle/maven, and have $GEODE_HOME set to point at an existing Geode installation.
For external users, get the geode-junit jar through gradle/maven in addition to the full apache-geode distribution zip file from our snapshot repo.  I'm planning to write a Gradle task (doesn't exist yet) that will extract that distribution zip and set $GEODE_HOME to the expanded location (similar to what happens in 1 for internal usage).

I should note that 2 & 3 are certainly experimental at this time.


tf.fill fails when dims has type tf.int64

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.8.0 (CPU version)
Python version: 3.6.5

Describe the problem
tf.fill([n], 42.0) fails when n is of type tf.int64, according to the docs:

dims: A Tensor. Must be one of the following types: int32, int64. 1-D. Represents the shape of the output tensor.

Source code / logs
The following code describes the problem:
import tensorflow as tf

n1 = tf.constant(3, tf.int32)
n2 = tf.constant(3, tf.int64)

x1 = tf.fill([n1], 42.0)  # OK
x2 = tf.fill([n2], 42.0)  # not OK

output:
>>> import tensorflow as tf
>>> n1 = tf.constant(3, tf.int32)
>>> n2 = tf.constant(3, tf.int64)
>>> x1 = tf.fill([n1], 42.0)  # OK
>>> x2 = tf.fill([n2], 42.0)  # not OK
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_array_ops.py", line 2717, in fill
    "Fill", dims=dims, value=value, name=name)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 513, in _apply_op_helper
    raise err
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 510, in _apply_op_helper
    preferred_dtype=default_dtype)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1104, in internal_convert_to_tensor
    ret = conversion_func(value, dtype=dtype, name=name, as_ref=as_ref)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/constant_op.py", line 235, in _constant_tensor_conversion_function
    return constant(v, dtype=dtype, name=name)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/constant_op.py", line 214, in constant
    value, dtype=dtype, shape=shape, verify_shape=verify_shape))
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/tensor_util.py", line 432, in make_tensor_proto
    _AssertCompatible(values, dtype)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/tensor_util.py", line 340, in _AssertCompatible
    raise TypeError("List of Tensors when single Tensor expected")
TypeError: List of Tensors when single Tensor expected

Thanks!

Hi, List is not Tensor. I think you should pass a tf.constant([3], tf.int64) directly for dims.

Hi, thanks for the reply, @facaiy. Indeed it seems like it is best to pass what you said and it works for tf.int64 and tf.int32.
Any idea why this works for tf.int32 and not for tf.int64? Is it a bug?
import tensorflow as tf

n = tf.constant(3, tf.int32)
x = tf.fill([n], 42.0)

with tf.Session() as sess:
    print(sess.run(x))

I think it's not bug. tf.int32 is the default dtype for tf.fill, so tensorflow always try to cast your list to tf.int32 if possible.


RGB or BGR

When I load the trained models in the torrent files, do they work with RGB or BGR images? In 'convert.py', the image is converted from RGB to BGR (with 'preprocess()' function) after being loaded by skimage.io.imread(). However, this routine is not found in 'forward.py' (only loading routine is there).

Actually, see the sess.run, the input is img but not img_p.

To me it looks like those commands are baked into the network loaded in forward. It does bgr to rgb, scaling, and mean subtraction.


Issue when using coc-basedpyright with framework

Link to my .vimrc configuration
Steps to repeat

Install VIM (vim editor)
Use my .vimrc file config
Then run :PluginInstall inside vim and then exit
After this you would have to run npm install inside ~/.vim/bundle/coc.nvim folder
Now install miniconda https://docs.anaconda.com/miniconda/
Created new environment with python 3.11 and masonite 4 framework (pip install masonite==4.20.0)
Switch to this new environment
Create a new project using command project start blog
Then go into the blog folder
Open vi and run inside it :CocInstall coc-basedpyright
Now open blog/app/controllers/WelcomeController.py and you would see

You would see stub error on all masonite imports, below as one e.g.
Stub file not found for "masonite.request" (basedpyright from  reportMissingTypeStubs)
And also coc error in the status line of vim like below
[coc.nvim] Request workspace/configuration failed with message: Cannot read properties of undefined (reading 'stubPath')

I managed to configure in pyproject.toml
[tool.basedpyright]
reportMissingTypeStubs = false
reportUnknownMemberType = false

to resolve the stub file not found error
But how to resolve the coc.nvim error for workspace/configuration failed error ?

Fixed in v1.17.0, now you can disable in coc-settings.json
  "basedpyright.analysis.diagnosticSeverityOverrides": {
    "reportMissingTypeStubs": false
  },


Can't create git branch for local repository

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I can't create a branch for a local repository
Steps to Reproduce

Install atom new
Create in git tab a repo
Try to create a branch

Expected behavior:
A branch should be created
Actual behavior:
Nothing happens
Reproduces how often:
The whole time
Versions
1.41.0

Has something happened?


Cannot assign non-sendable fields in actor literals

It may be desired to include non-sendable fields in actor literals, but there doesn't seem to be any way to include them.
pony version:
0.24.4 [release]
compiled with: llvm 3.9.1 -- cc (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609
Defaults: pic=false ssl=openssl_0.9.0

source (run in playground):
actor Main
  new create(env: Env) =>

    // Desire to create and actor literal with fields NOT captured from
    // the lexical scope. There does not seem to be a way to include
    // non-captured / non-sendable fields in actor literals.

    //let a': Array[String] iso = recover Array[String] end // [3]

    // Create an actor literal, note the `be` at the bottom
    let o = object

      // Because `o` is an actor literal,
      // Error: this parameter must be sendable (iso, val or tag)
      let a: Array[String] ref = Array[String] // [1]

      // This works:
      //let a: Array[String] iso = recover Array[String] end // [2]
      // Much like consuming some array from the lexical scope:
      //let a: Array[String] iso = consume a' // [3]

      /*
      I suppose this is due to the new Array[1] being considered part of the
      lexical scope the actor literal is defined in. I think [2] works because
      the new Array is iso^ and basically consumed to the literal's field such
      as [3].

      Is there (or should there be) a way to assign to fields in an actor literal
      such that they don't capture from the lexical scope, and can then be a
      non-sendable reference capability like ref? Something that would result in
      an anonymous actor with the structure:

        actor Anonymous
          let a: Array[String] ref = Array[String] // `a` is ref rather than iso
          be other() => None

      This would make dealing with `a` inside the actor much easier.
      */

      be other() => None // Lift `o` to an actor
    end

    env.out.print(".")

I think we should check the right hand side of the assignments for sendability, rather than the field type. This would allow let a: Array[String] ref = recover Array[String] end to work, which is IMO a good enough work-around.
In order to allow let a: Array[String] ref = Array[String], we could require that all variables used in the right hand side be sendable, which would essentially be the equivalent of wrapping the expression in an implicit recover block.
This would however break let a: Array[String] iso = foo(x), where foo has type foo(Bar ref): Array[String] iso^, even though that is safe and works today.
Maybe a combination of the two, ie. either is the right hand side is already sendable, or we can recover it to something sendable.

@plietar :+1:
Also as promised on IRC, here some bigger explanation, an actor-object-literal is rewritten into a definition of the type with some unique cryptic name (like $1$0) or something, which has all the methods, fields and provides list as defined in the object literal. Then at the place of the actual object literal, there is a constructor call to that type inserted.
During that process all fields are added as constructor arguments and the initializers e.g. let x: U8 = U8(0) are transformed into constructor params.
Small example:
let o = object is Foo
  let f1: U8 = U8(0)

  fun f(): String => "bla"
end

will  be rewritten to something like:
$1$0.create(U8(0))

class $0$1
  let f1: U8
  new create($blerch: U8) =>
    f1 = $blerch
  fun f(): String => "bla"

Maybe it is worth putting some effort into discovering whether a field initializer references anything from the outer scope, if not, it can remain an initializer on the object-literal field and may not be transformed into a constructor argument. Although i have to admit that this also might lead to some behaviour that might be considered inconsistent.


Touch events not working in Chrome (desktop in device mode) and in Android webview

Hi,
The color-picker (tested with v2.4.5, v2.4.6 and v2.4.7) does not respond when in Google Chrome in device mode (Windows 10 and Mac OS X).  The color picker works fine when not in device mode.
The color picker also works fine on iOS in webview (Ionic app), but it also breaks in Android webview (same Ionic app).
You can replicate this behaviour with the demo page at: https://ruhley.github.io/angular-color-picker/
Just open the page in Chrome, open the Dev Tools and toggle the device toolbar.

Fixed in commit 16eff08 and released in v2.4.8

Hi ruhley,
I can confirm that the bug is fixed. It now works in the browser (Chrome) in device mode. It also works in my webview on Android.
Thank You so much for addressing this issue so quickly, it really helps me out to meet a deadline!


validate Krun / KM with podman

The work to fix this was done with the following change:
947175f Changes to support make validate-*-image targets using podman (#1322)
https://github.com/kontainapp/km/pull/1322


Attackflow Added

Thanks for your contribution @sust4in. :octocat:
I've moved the tool into the "web services" section and adjusted the wording a bit. Also added the copyright labels. Cheers!


add ctr pull image verbose parameter

add ctr pull image verbose parameter https://github.com/containerd/containerd/issues/9664

Just to make sure I understand — currently, unless the debug flag is set to true, when pulling an image, ctr displays pull data to stdout, correct?
If so, this seems like perfectly fine default behavior. But, it might be worth having an option to not direct it to stdout. Maybe a quiet flag instead of a verbose flag? That would be useful IMO.

Just to make sure I understand — currently, unless the debug flag is set to true, when pulling an image, ctr displays pull data to stdout, correct?
If so, this seems like perfectly fine default behavior. But, it might be worth having an option to not direct it to stdout. Maybe a quiet flag instead of a verbose flag? That would be useful IMO.

yes ,it means that ,you can see it from this https://github.com/containerd/containerd/issues/9664 , but set IO maybe clearly and easily ,quiet flag may be a good idea not just for 'ctr pull'.

is this pr still open ?


Missing wifiConfApp

I just installed on Raspberry Python 3.9. Install looks good, but it is missing wifiConfApp
pi@raspberrypi:~ $ sudo pip3 install wifiConfig
   Looking in indexes: https://pypi.org/simple, https://www.piwheels.org/simple
   Collecting wifiConfig
   Using cached https://www.piwheels.org/simple/wificonfig/wifiConfig-0.1.14-py3-none-any.whl (9.6 kB)
   Requirement already satisfied: PyAccessPoint in /usr/local/lib/python3.9/dist-packages (from wifiConfig) (0.2.5)
   Requirement already satisfied: flask in /usr/lib/python3/dist-packages (from wifiConfig) (1.1.2)
   Requirement already satisfied: netifaces in /usr/local/lib/python3.9/dist-packages (from PyAccessPoint->wifiConfig) (0.11.0)
   Requirement already satisfied: psutil in /usr/lib/python3/dist-packages (from PyAccessPoint->wifiConfig) (5.8.0)
   Requirement already satisfied: wireless in /usr/local/lib/python3.9/dist-packages (from PyAccessPoint->wifiConfig) (0.3.3)
   Installing collected packages: wifiConfig
   Successfully installed wifiConfig-0.1.14
   pi@raspberrypi:~ $ sudo python3 wifiSetup.py
python3: can't open file '/home/pi/wifiSetup.py': [Errno 2] No such file or directory
   pi@raspberrypi:~ $ cd rfid
   pi@raspberrypi:~/rfid $ sudo python3 wifiSetup.py
   Traceback (most recent call last):
   File "/home/pi/rfid/wifiSetup.py", line 1, in <module>
   from wifiConfig import WifiConfApp
ImportError: cannot import name 'WifiConfApp' from 'wifiConfig' (/usr/local/lib/python3.9/dist-packages/wifiConfig/__init__.py)

Doesn't that assume you will be using the command line version? If I want to include your code in my own app, I would have to run something like what I was trying to do, which is the example you give.

Sorry, but I lacked context of what you wanted to do, and I gave you a quick solution, just like your question.
If what you want to implement in your code, first of all, have you tested if the example provided works?
import wifiConfig
access_point_config = {"wlan":'wlan0', "inet":None, "ip":'<IP_ADDRESS>', "netmask":'<IP_ADDRESS>', "ssid":'MyAccessPoint', "password":'1234567890'}
flask_app_config = {"host":"<IP_ADDRESS>", "port":"8080"}
myWifiConfig = wifiConfig.WifiConfApp(access_point_config, flask_app_config)
myWifiConf.start()

That is exactly the file I ran. I gave me all the errors. There is a problem in how you have the install set up as changing
myWifiConfig = wifiConfig.WifiConfApp(access_point_config, flask_app_config)
to
myWifiConfig = wifiConfig.main.WifiConfApp(access_point_config, flask_app_config)
[from my memory as I deleted the file]
works. Also,
myWifiConf.start()
should be
myWifiConfig.start()
You must not have run your example otherwise you would see the problems I mentioned.


Add possibility to get transaction's block hash when querying it

Motivation
In the Iroha Explorer, we fetch transactions with FindAllTransactions query, and we want to know to what block each of them belongs to.
Possible solutions

(the most preferable) In FindAllTransactions query return not Vec<TransactionValue> but some other struct like:
struct TransactionExt {
  tx: TransactionValue,
  block_hash: Hash
}

Extend Transaction or RejectedTransaction  structures with block_hash field

Extend general TransactionValue with block_hash field (i.e. make it struct instead of enum)

https://github.com/omkar-mohanty showed interest

Hi I want to work on this!

The two latter approaches to me seem untenable.

Extend Transaction or RejectedTransaction structures with a block_hash field

Would open Iroha to an OOM attack.

Extend general TransactionValue with block_hash field (i.e. make it struct instead of enum

Which is slightly better, but to me still problematic in terms of memory.
Since the block hash is implicit, we don't need to store it, so my suggestion would be to compose the block hash into the TX value only in the query.


contentInsets always overwrites scrollIndicatorInsets of ScrollView/ListView

Description
I set contentInsets and scrollIndicatorInsets of a ScrollView, actually  ListView, to different values. But seems the value of contentInsets always overwrites the scrollIndicatorInsets one.
Reproduction
https://sketch.expo.io/Byi2M7psg
In this demo, scrollIndicatorInsets.top is 20, it's expected to be 100. After deleting contentInset={{top: 20, left: 0, right: 0, bottom: 0}}, the scrollIndicatorInsets is correct.
Solution
On line 262 of https://github.com/facebook/react-native/blob/master/React/Views/RCTView.m#L262 scrollIndicatorInsets is set to baseInset which is contentInset most of the time. This method is called when setContentInset for RCTScrollView.
On line 372 of https://github.com/facebook/react-native/blob/v0.42.3/React/Views/RCTComponentData.m#L372 when enumerate props, the order should not be predictable, but actually contentInset always comes after scrollIndicatorInsets during my debugging. So contentInset always overwrites scrollIndicatorInsets unless it's Zero (returned on if (UIEdgeInsetsEqualToEdgeInsets(contentInset, _contentInset)) return).
Additional Information

React Native version: 0.42.3
Platform: iOS
Operating System: MacOS
Dev tools: iOS 10.2.1

Thanks for reporting this. Would you consider sending a PR? It seems like you've already done some of the work here.

I would like to give it a shot if this still exists

👋 hey all, thanks for reporting this issue.
There was a commit some time ago (https://github.com/timotew/react-native/commit/d1b9d329314b18497a310a443ea195d88eb1e06f) that was meant to have fixed this issue, because of this I'm going to go ahead and close this issue for now.
Should someone be able to provide a reproduction on the latest version of React Native I'd be happy to re-open this, please let me know.
Thank you

@Salakar This bug is back, the code from the PR seems to have disappeared. Can you open the issues again?


Error running 'bash package.sh' on macOS High Sierra

$ bash packages.sh
packages.sh: line 20: ${DISTRO^^}: bad substitution
Unknown distribution 'darwin'
Platform: Darwin Dean-Mac.wv.cc.cmu.edu 17.5.0 Darwin Kernel Version 17.5.0: Mon Mar  5 22:24:32 PST 2018; root:xnu-4570.51.1~1/RELEASE_X86_64 x86_64

Odd, seems to work for me on 10.13.3. Can you also try ./packages.sh ?

The error message is pretty much the same. I copied out the macOS part to another script and it works. Thanks!

Just as a sanity check I just spun up a clean 10.13.4 image and couldn't reproduce the issue.

And our macOS build run with this script. Closing for now.


"event.send_temporary_message" is to be  ArgumentError

Summary

I tried to execute the following method.
bot.command :test do |event|
  event.send_temporary_message("String", 10.0)
end

I think the minimum required arguments are two, but The result at this time was ArgumentError.
I tried analysis and found a place that I thought was the cause.
path is "discordrb-3.4.0\lib\discordrb\bot.rb" line 388
and "discordrb-3.4.0\lib\discordrb\data\channel.rb" line 360.
Each has a different number of arguments.
p.s. I'm Japanese and not good at English, so I may have mistake in English grammar.

Environment

Ruby version:

ruby 2.7.2p137 (2020-10-01 revision 5445e04352) [x64-mingw32]
Discordrb version:

discordrb (3.4.0)
discordrb-webhooks (3.3.0)

This was fixed via shardlab/discordrb#17 in the new repo and released in v3.4.1 and can be closed. Thank you for the report.


Facing error when loading the checkpoints after training cpc/train.py

I have trained the cpc/train.py model. When I was evaluating the model using pc/eval/linear_separability.py using checkpoints saved by training cpc/train.py I got an error "FileNotFoundError: [Errno 2] No such file or directory: '/content/drive/My Drive/CPC_audio/cpc/checkpoints/checkpoint_args.json' ". After checking the file I found that no checkpoint_args.json file was saved. Code is also not saving "checkpoint_args.json".
`def run(trainDataset,
valDataset,
batchSize,
samplingMode,
cpcModel,
cpcCriterion,
nEpoch,
pathCheckpoint,
optimizer,
scheduler,
logs):
print(f"Running {nEpoch} epochs")
startEpoch = len(logs["epoch"])
bestAcc = 0
bestStateDict = None
start_time = time.time()

for epoch in range(startEpoch, nEpoch):

    print(f"Starting epoch {epoch}")
    utils.cpu_stats()

    trainLoader = trainDataset.getDataLoader(batchSize, samplingMode,
                                             True, numWorkers=0)

    valLoader = valDataset.getDataLoader(batchSize, 'sequential', False,
                                         numWorkers=0)

    print("Training dataset %d batches, Validation dataset %d batches, batch size %d" %
          (len(trainLoader), len(valLoader), batchSize))

    locLogsTrain = trainStep(trainLoader, cpcModel, cpcCriterion,
                             optimizer, scheduler, logs["logging_step"])

    locLogsVal = valStep(valLoader, cpcModel, cpcCriterion)

    print(f'Ran {epoch + 1} epochs '
          f'in {time.time() - start_time:.2f} seconds')

    torch.cuda.empty_cache()

    currentAccuracy = float(locLogsVal["locAcc_val"].mean())
    if currentAccuracy > bestAcc:
        bestStateDict = fl.get_module(cpcModel).state_dict()

    for key, value in dict(locLogsTrain, **locLogsVal).items():
        if key not in logs:
            logs[key] = [None for x in range(epoch)]
        if isinstance(value, np.ndarray):
            value = value.tolist()
        logs[key].append(value)

    logs["epoch"].append(epoch)

    if pathCheckpoint is not None \
            and (epoch % logs["saveStep"] == 0 or epoch == nEpoch-1):

        modelStateDict = fl.get_module(cpcModel).state_dict()
        criterionStateDict = fl.get_module(cpcCriterion).state_dict()

        fl.save_checkpoint(modelStateDict, criterionStateDict,
                           optimizer.state_dict(), bestStateDict,
                           f"{pathCheckpoint}_{epoch}.pt")
        utils.save_logs(logs, pathCheckpoint + "_logs.json")`

with open(args.pathCheckpoint + "_logs.json", 'w') as file:
  json.dump(vars(args), file,indent=2)

It should be added in definition of run fuction.

Maybe we can directly insert the codes
with open(args.pathCheckpoint + "_args.json", 'w') as file:
            json.dump(vars(args), file, indent=2)

just after the 349 lines of train.py (the main function - the checkpoint part) because we need not save the json file for many times, considering args will not change afterwards.

Respected sir,
I thank you for your time. I understand that I arised very silly question
and I apologize for it. I just wanted to let you know it should be
mentioned because it shows error when using saved checkpoints.
With regards
Arti Kumari
IIT (ISM) Dhanbad
On Fri, 3 Jul 2020 at 19:06, Chen<EMAIL_ADDRESS>wrote:

Maybe we can directly insert the codes
with open(args.pathCheckpoint + "_args.json", 'w') as file:
json.dump(vars(args), file, indent=2)
just after the 349 lines of train.py (the main function - the checkpoint
part) because we need not save the json file for many times, considering
args will not change afterwards.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/CPC_audio/issues/6#issuecomment-653552295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMO436WD7LR3K2IVKZWQDPLRZXNE3ANCNFSM4MX7Y2HQ
.

I'm sorry for the apply before. I'm also a user and I found this question too. I just wanted to assure whether I'm done correctly. Very thank you for arising this question!


Cell menu redesign

Will you change the appearance of the message context menu like in iOS 13?
Are there any plans to add it?

Hey @Rouix, thanks for bringing this up.
I think it'd be a great addition to Chatto.
Is that something you might be interested working on?


Add legend rv-histogram - docs

Reference issue
Closes #16892 .
What does this implement/fix?
Added legend to rv-histogram docs.

Thank you for the PR @liortct. I believe though that @jo-mueller wanted to work on this. I would prefer to consider his PR first.
In general we don't want people to compete to make PRs. There is more work to do than there are contributors so we don't want to engage in such things.
@jo-mueller can you confirm that you would want to do a PR?

Can confirm. Just wanted to take my time looking through the contribution guidelines before submitting as I hadn't done it before here. Thanks for notifying @tupui

Thank you for confirming and opening #16899. I will close this PR then.


[ENH, MRG] First youtube video in docs!

Fixes https://github.com/mne-tools/mne-python/issues/10771.

https://output.circle-artifacts.com/output/job/750429f3-ad54-4612-a80a-57d7f3b41ea3/artifacts/0/dev/auto_tutorials/forward/80_fix_bem_in_blender.html#editing-in-blender
Super nice!!!!!

Yay this looks awesome and was very easy! https://output.circle-artifacts.com/output/job/750429f3-ad54-4612-a80a-57d7f3b41ea3/artifacts/0/dev/auto_tutorials/forward/80_fix_bem_in_blender.html?highlight=blender

Thanks @alexrockhill!


Error: SQLITE_BUSY: database is locked

I'm seeing Error: SQLITE_BUSY: database is locked errors occurring when requesting vtiles via mapbox-gl-js. In my case the source is a slow to respond postgres databases of osm coastline data. I'm creating this issue to track two issues: 1) Why is this error happening and can we prevent it? and 2) why is the error silent and not obvious without manual print statements to debug (see below)?
Originally these failures were resulting in failed tile responses but no obvious errors from studio. After putting print statements around I see that:

mbtiles.startWriting is returning an error
which hits here: https://github.com/mapbox/mapbox-studio/blob/mb-pages/lib/cache.js#L39
which then hits here: https://github.com/mapbox/mapbox-studio/blob/mb-pages/lib/middleware.js#L186
the error is never printed to the logs or console

refs https://github.com/mapbox/node-sqlite3/search?q=SQLITE_BUSY&type=Issues&utf8=✓

Consistently seeing this: it appears to happen for individual vector tile requests that are large. I presume what is happening is that the sqlite db is taking a long time to write the file and cannot service the request. I'm finding that if I open up the failed request url it always works the second time (once the data is cached)


demos\interactive list issues

not working
13:03:51.032 panic: runtime error: invalid memory address or nil pointer dereference
13:03:51.034 [signal 0xb code=0x5c7268 addr=0x17130029 pc=0x0]
13:03:51.035 <empty string>
13:03:51.035 goroutine 6 [running]:
13:03:51.035 github.com/hexops/vecty.(*HTML).reconcileProperties.func1(0x730b8, 0x662460, 0x662440, 0x1, 0x1, 0x18420, 0x4e078)
13:03:51.035 	D:/Users/Vince/Documents/Go/pkg/mod/github.com/hexops/vecty@v0.6.0/dom.go:246 +0x11
13:03:51.036 github.com/hexops/vecty.funcOf.func1(0x7ff8000100001328, 0x660210, 0x662430, 0x1, 0x1, 0x660218, 0x27a00)
13:03:51.036 	D:/Users/Vince/Documents/Go/pkg/mod/github.com/hexops/vecty@v0.6.0/dom_js.go:60 +0x15
13:03:51.036 syscall/js.handleEvent()
13:03:51.036 	C:/Program Files/Go/src/syscall/js/func.go:96 +0x24
13:03:51.037 exit code: 2

sorry; I seem to have missed this.