[DO NOT MERGE; PENDING MAKE POST LINK] Add management command to send api move announcement email

Fixes

Part of #3742 by @sarayourfriend (does not fix because the issue also requires sending the email, i.e., running the command in production).
Description

Review the management command. The message text has already undergone review. The placeholder for the make post URL will be replaced before merging, after the post is available next week.
Testing Instructions

Review the code and confirm tests are adequate and pass.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[x] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

Upgrading this to high priority so that we can send it out early next week along with the make post on Monday. #2037 for context.
@WordPress/openverse-api pinging y'all for expedited review to match the priority.

The issue references the old management command that we sent an email with before: https://github.com/WordPress/openverse/issues/3742
A general one is out of scope, as you've both said, but I want to share further context, as I don't believe we even need one. We've sent two emails in as many years, and the first was extremely specialised in the selection of who received it. It's hard to imagine it would be worth the time to write, test, and maintain a generalised management command when this one can be copy/pasted (and crucially in the meantime deleted, and not kept as unused code for who knows how long) and whatever modifications made as needed.

Agree on both points: generalising is out of scope for this PR and this code can be deleted when unused (at 2 emails in ~3 years this is basically 100% of the time).

Because application emails are not unique, there is a chance of accidentally emailing people twice, if their multiple applications registered to a single email are pulled; that is, there is a chance of this if the distinct() call on the queryset is not filtering distinct on the application email. I need to check this. I'll do so by adding a new unit test first to determine whether this is the case, and if so, fix the distinct call or modify the loop to prevent it. I'll do that before merging this, of course, as well as addressing the editorial remarks from @dhruvkb.
@dhruvkb Just an FYI that I did ask for editorial reviews from everyone in the maintainer group before marking this PR ready for review; ideally these editorial remarks would have been shared there, as some of them overlap with content in the Make post. Just giving context for where the text came from, and the process it's already been through.

The emails are sent as of 2024-05-08T00:59:58.366Z.


help link in readme.md seems outdate?

https://caddyserver.com/docs/json/apps/tls/automation/policies/issuer/acme/
link docs content is empty now
api_token in the example seems auth_token today:
https://caddyserver.com/docs/json/apps/tls/automation/policies/issuers/acme/challenges/dns/provider/dnspod/

Fixed


QAWTM-15: Updating appium-python-client version to 1.3

Updating appium-python-client version to 1.3

python 3.5 support has already been removed, please merge master into your branch to fix error

Coverage remained the same at 90.23% when pulling c80dad594751ab6ca57df4b4b56e074c8607edd3 on lmcalvo:QAWTM-15_appium-python-client_1_3 into b5c37636baf1eab9b34a2f821fbdc767a438ce35 on Telefonica:master.


Add description field to the MergeRequest model

Hey there,
Ran into an issue running Gitlab 7.10 where description was coming back from the API for merge requests, but description was not in the list of properties in Gitlab\Model\MergeRequest. As a result, Gitlab\Model\MergeRequest::fromArray (calling Gitlab\Model\AbstractModel::hydrate) was throwing an exception.
This pull request adds the description key to the list of properties in Gitlab\Model\MergeRequest.
It looks like the current documentation for the Gitlab API lists description as a field, so this shouldn't be a old API version issue, probably will be this way going forward. The list of properties I referenced is available here for review: http://doc.gitlab.com/ee/api/merge_requests.html#get-single-mr
Hope this helps!
Take care,
JB

Thanks!


fix set_current_tab: Indexp_current=-1 out of size

fixes #12997
My first PR 🎉, hopefully I didn't mess things up 😅

Congrats for your first PR :)

Thanks :D


The type of the first element of the [...T[], T] type is T | undefined when noUncheckedIndexedAccess is enabled

Bug Report
🔎 Search Terms
spread array type, first element, undefined, noUncheckedIndexedAccess
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about Array

⏯ Playground Link
Playground link with relevant code
💻 Code
// Note that the noUncheckedIndexedAccess option is enabled

const array: [...number[], number] = [1, 2, 3]
const first: number = array[0]   // Type error! The type of array[0] is number | undefined

🙁 Actual behavior
The type of array[0] is number | undefined.
🙂 Expected behavior
The type of array[0] is number.
In general, the type of the first element of the [...T[], U] type should be T | U.

Hi, with the noUncheckedIndexedAccess option enabled, TypeScript assumes that indexed access can potentially return undefined if the index is out of bounds. However, in this case, since the array is not empty, accessing array[0] should correctly result in the type number, rather than number | undefined.
Two different approachs in the mean time as workaround :)
Ensure that the array is not empty before accessing its first element:
const first: number | undefined = array.length > 0 ? array[0] : undefined;
If you are certain that the array will always have at least one element, you can use a non-null assertion operator (!) to assert that the value will not be undefined:
const first: number = array[0]!;

Ensure that the array is not empty before accessing its first element: const first: number | undefined = array.length > 0 ? array[0] : undefined;

This actually doesn't help - or at least, array.length checks do nothing to remove undefined from the type of array[0]. I wish they would - it would make noUncheckedIndexedAccess a lot more usable. But the situation in this issue should be an exception, since array is typed to contain at least 1 element.


Change word tokenizer

Here I change word tokenizer engine from newmm to nercut and I also change some of the word list to fit the new engine change

Nice very cool I don't have time to review it and I'm free now so. The code was very good and nice looking.


What if the html tag attributes or values change??

We need to manually update the code to make it work if anything related to the html elements changes, right?

Unfortunately this is the case - though i still believe that updating the scraper is easier than trawling thru the job sites as intended.


Sample

There is a problem with the sample.
After cloned, installed and run-android, the packager keeps popping error 500:
"...react-native-draftjs-render/src/getblocks doesn't exist"

Hey.
Have you installed the project with npm install or make setup?

Hey,
I used npm install
npm version: 4.0.5

To improve our development workflow, we've created a script sync-lib that copies lib files to sample.
This is useful because we can edit our lib and see the results automatically (live reload), otherwise we would keep reinstalling the lib everytime...
npm install doesn't run sync-lib but make setup does, that's why we advise to use it :)

Didn't know about that, everything is right now.
Thank you very much for your help.


Migration failing in 5.8

The migration seems to be failing in Laravel 5.8,
Migrating: 2017_11_24_094130_create_stripes_table

   Illuminate\Database\QueryException  : SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint (SQL: alter table `stripes` add constraint `stripes_user_id_foreign` foreign key (`user_id`) references `users` (`id`))

Probably related to the changes to Migrations & bigIncrements

#22 merged! Thank you!


OmniSharp doesn't show build errors anymore

Issue Description
After recent update, Omnisharp no longer shows build errors or warnings.
I have to make edit to a file to see the highlighted problems. Also when build error is fixed, again have to reedit the file to clear the error notifications.
Expected Behavior
OmniSharp used to highlight build errors and warnings before save or build.
Logs
OmniSharp log
Starting OmniSharp server at 3/14/2020, 6:24:37 PM
    Target: /home/georgi/Projects/autohub
OmniSharp server started.
Path: /home/georgi/.vscode/extensions/ms-dotnettools.csharp-1.21.14/.omnisharp/1.34.14/run
PID: 16544
    Starting OmniSharp on fedora 31.0 (x64)

    DotNetPath set to dotnet

    Located 1 MSBuild instance(s)
        1: StandAlone 16.4 - "/home/georgi/.vscode/extensions/ms-dotnettools.csharp-1.21.14/.omnisharp/1.34.14/omnisharp/.msbuild/Current/Bin"

    MSBUILD_EXE_PATH environment variable set to '/home/georgi/.vscode/extensions/ms-dotnettools.csharp-1.21.14/.omnisharp/1.34.14/omnisharp/.msbuild/Current/Bin/MSBuild.dll'</details>

C# log
Installing C# dependencies...
Platform: linux, x86_64, name=fedora, version=31
Downloading package 'OmniSharp for Linux (x64)' (43753 KB).................... Done!
Validating download...
Integrity Check succeeded.
Installing package 'OmniSharp for Linux (x64)'
Environment information
VSCode version: 1.43.0
C# Extension: 1.21.14
Mono Information
        OmniSharp using built-in mono
Dotnet Information
.NET Core SDK (reflecting any global.json):
 Version:   2.1.801
 Commit:    c404ee777a
Runtime Environment:
OS Name:     fedora
OS Version:  31
OS Platform: Linux
RID:         fedora.31-x64
Base Path:   /usr/share/dotnet/sdk/2.1.801/
Host (useful for support):
Version: 2.1.12
Commit:  ccea2e606d
.NET Core SDKs installed:
2.1.801 [/usr/share/dotnet/sdk]
.NET Core runtimes installed:
Microsoft.AspNetCore.All 2.1.12 [/usr/share/dotnet/shared/Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.12 [/usr/share/dotnet/shared/Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.12 [/usr/share/dotnet/shared/Microsoft.NETCore.App]
To install additional .NET Core runtimes or SDKs:
https://aka.ms/dotnet-download

Visual Studio Code Extensions

Extension
Author
Version

auto-close-tag
formulahendry
0.5.6

auto-rename-tag
formulahendry
0.1.1

bracket-pair-colorizer
CoenraadS
1.0.61

code-settings-sync
Shan
3.4.3

code-spell-checker
streetsidesoftware
1.8.0

csharp
ms-dotnettools
1.21.14

csharpextensions
jchannon
1.3.1

debugger-for-chrome
msjsdiag
4.12.6

docomment
k--kato
0.1.8

dotnet-test-explorer
formulahendry
0.7.1

githistory
donjayamanne
0.6.2

gitignore
codezombiech
0.6.0

gitlens
eamodio
10.2.1

npm-intellisense
christian-kohler
1.3.0

path-intellisense
christian-kohler
1.4.2

php-intellisense
felixfbecker
2.3.14

project-manager
alefragnani
10.11.0

remote-wsl
ms-vscode-remote
0.42.3

theme-cobalt2
wesbos
2.1.6

todo-tree
Gruntfuggly
0.0.169

vscode-coverage-gutters
ryanluker
2.4.3

vscode-diff
fabiospampinato
1.4.0

vscode-docker
ms-azuretools
1.0.0

vscode-eslint
dbaeumer
2.1.1

vscode-great-icons
emmanuelbeziat
2.1.47

vscode-language-babel
mgmcdermott
0.0.26

vscode-nuget-package-manager
jmrog
1.1.6

vscode-todo-highlight
wayou
1.0.4

vscode-yaml
redhat
0.7.2

vscodeintellicode
VisualStudioExptTeam
1.2.6

@gmarokov Can you provide the complete OmniSharp and C# logs? If what you provided is complete then, it could be an install error. try reinstalling.

@gmarokov Can you provide the complete OmniSharp and C# logs? If what you provided is complete then, it could be an install error. try reinstalling.

That's the whole log. I just reinstalled it, and the issue persists. There are no logs in C# and in OmniSharp I see that last entry [info]: OmniSharp.Roslyn.CSharp.Services.Diagnostics.CSharpDiagnosticWorkerWithAnalyzers         Solution initialized -> queue all documents for code analysis. Initial document count: 218. but I don't have a .sln file, can that be the issue?
I just deleted one file and as you can see on build I get the errors, but don't see them highlighted in the file explorer.

Thanks for the support guys, but suddenly was fixed with the latest release.


[Dart fix] Support top-level variable replacedBy field in class

Support converting a top-level variable such as standardEasing.
where the class looks like
class Easing {
  static const Curve legacy = ...;
}

this
const Curve curve = standardEasing;

would become
const Curve curve = Easing.legacy;

with the following fix
  - title: "Migrate to 'Easing.legacy'"
    date: 2023-07-04
    element:
      uris: [ 'material.dart' ]
      variable: 'standardEasing'
    changes:
      - kind: 'replacedBy'
        newElement:
          uris: [ 'material.dart' ]
          field: legacy
          inClass: Easing

@keertip

Thanks for opening the issue. Will take a look.

@guidezpl, tested it out and it works for me, both as a test case and from 'dart fix'. Used the transform from above.
Do you have the @deprecated annotation on standardEasing?

Ah that fixed it, I had those commented out to land the change in stages, thanks!


[Bug]: JS client throwing weird errors

What happened?
I feel like I must be missing something obvious. This is so weird to me.
Every method call from my JS client is throwing errors that don't look like errors..
JS code:
        const client = new ChromaClient();

        try {
            await client.heartbeat();
        } catch(e) {
            console.log('error');
            console.log(e);
        }
        try {
            await client.listCollections();
        } catch (e) {
            console.log('error');
            console.log(e);
        }

Browser console

Docker logs

Versions
installed<EMAIL_ADDRESS>in a Meteor.js app
Meteor 2.10.0
npm 6.14.17
Relevant log output
No response

@Russell-Pollari the requests are working but throwing errors? or the requests are failing? and just to understand correctly - it seems that 1.4.0 and 1.5.0 both have this ?

@jeffchuber nvm the behavious is different on 1.4.0
It looks like the requests are working (I see "GET /api/v1/heartbeat HTTP/1.1" 200) on the docker logs.
And it looks like an error is being thrown (caught by try..catch) but when I log the error I just the Response object shown in my browse logs above

I'm baffled and assume I'm making some silly mistake

FYI @jeffchuber - this is only an issue when trying to use a Chroma client on the browser.
Tested on a Node server and everything works

@Russell-Pollari i think this is a CORS issue
if you update your docker file,
this should work
environment:
      - CHROMA_DB_IMPL=clickhouse
      - CLICKHOUSE_HOST=clickhouse
      - CLICKHOUSE_PORT=8123
      - CHROMA_SERVER_CORS_ALLOW_ORIGINS=["http://localhost:3000"]

@Russell-Pollari i think this is a CORS issue

of CORS it is!
Thanks, @jeffchuber the fix worked!

@Russell-Pollari haha
i am adding a note about this to the README for the browser example which i think is currently incorrect.


can't play the map [A Learnable Meta World - Beginner]

Hi, Thanks for the amazing resource!
Novice and Intermediate map works properly with the hint shown after the game
But the beginner map is not working :(

Hi,
Are you running the latest version of the userscript?
I does work on my computer. It could don't work if the tab was created prior to the installation of the userscript. Please try to open the map in a new tab.


ImportError in django test (auto annotation)

When i enable openvino (OPENVINO_TOOLKIT: "yes") for auto annotation and run manage.py test in cvat docker shell
fails with error below:
 File "/home/django/cvat/apps/reid/urls.py", line 6, in <module>
    from . import views
  File "/home/django/cvat/apps/reid/views.py", line 10, in <module>
    from cvat.apps.reid.reid import ReID
  File "/home/django/cvat/apps/reid/reid.py", line 7, in <module>
    import cv2
ImportError: No module named 'cv2'

I disable openvino (OPENVINO_TOOLKIT: "no") and It work well.
Thanks in advance

@KiteYeh
Have you installed OpenVINO with OpenCV?

@KiteYeh
Have you installed OpenVINO with OpenCV?
hi bsekachev,
I referenced document to enable openVINO:

Build docker image
# From project root directory
docker-compose -f docker-compose.yml -f components/openvino/docker-compose.openvino.yml build

Run docker container
# From project root directory
docker-compose -f docker-compose.yml -f components/openvino/docker-compose.openvino.yml up -d

I couldn't sure if openVINO and openCV are installed or not after build openVINO image, Did I missed any step for enabling auto annotation?
Thanks in advance

@KiteYeh
Could you please send us the whole sequence of steps you do to reproduce the issue?

@KiteYeh
I built container with OpenVINO support.
Went to its bash, install all dependencies pip3 install -r cvat/requirements/testing.py
And finally I was be able to run testing: python3 manage.py test cvat/apps/engine/ --settings=cvat.settings.testing
Everything is alright.

@bsekachev
thanks for your reply,
The cvat codebase pull from master branch not developer,
It set DJANGO_CONFIGURATION: "production" in docker-compose yml file so that docker only installed production requirement.
the sequence of steps:

enable openVINO and run the docker container:
Went to its bash
run django test: python3 manage.py test

fails with error below:
 File "/home/django/cvat/apps/reid/urls.py", line 6, in <module>
    from . import views
  File "/home/django/cvat/apps/reid/views.py", line 10, in <module>
    from cvat.apps.reid.reid import ReID
  File "/home/django/cvat/apps/reid/reid.py", line 7, in <module>
    import cv2
ImportError: No module named 'cv2'

is there any mistake in test steps?

@KiteYeh
Master branch has been updated a long time ago.
So, first of all I would recommend you to use newer version of CVAT.
In the middle of September we are going to make a release if you don't want to use code from develop branch.
Also short question. When you go to container's bash, do you get a message "[setupvars.sh] OpenVINO environment initialized"?
And another short question. What OpenVINO version do you use?

@bsekachev
Yes, I got a message "[setupvars.sh] OpenVINO environment initialized"
and I use OpenVINO 2019R2.
the auto annotation work well, but python3 manage.py test has failed.
If no any suggestion, I will upgrade too after you release the new version and test again.
Thanks for your reply.

@KiteYeh
I use OpenVINO 2019R1, maybe it's a reason. But I am not sure. Will check it later.

@KiteYeh
I use OpenVINO 2019R1, maybe it's a reason. But I am not sure. Will check it later.

@bsekachev
Has the new version of master branch been released?
Do you try the OpenVINO 2019R1 and is there any feedback on the test result?
thanks for your reply

@KiteYeh
I use OpenVINO 2019R1, maybe it's a reason. But I am not sure. Will check it later.

@bsekachev
It work well on master branch with released 0.5.0 version.
Thanks for your reply

@KiteYeh
Great news.
So, has the issue been resolved?

@KiteYeh
Great news.
So, has the issue been resolved?

Thanks for your reply,
the sequence of steps:

Went to docker bash
run django test: python3 manage.py test

Another fail with error below
===============================================================
ERROR: utils.voc.tests.test_process_cvat_xml (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: utils.voc.tests.test_process_cvat_xml
Traceback (most recent call last):
  File "/usr/lib/python3.5/unittest/loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib/python3.5/unittest/loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "/home/django/utils/voc/tests/test_process_cvat_xml.py", line 5, in <module>
    from utils.voc.converter import process_cvat_xml
  File "/home/django/utils/voc/converter.py", line 16, in <module>
    import glog as log
ImportError: No module named 'glog'
----------------------------------------------------------------------

could you help to check this issue?
Thanks in advance

@bsekachev
Thanks for your reply, the issue could be closed.


Empty file is created on canceling, while take photo with SaveToAlbum = true

This is a

[x] Bug

Which plugin does this impact:

[x] Media

Bug
Version Number of Plugin: 2.3.0
Device Tested On:
Samsung S4 Droid 5.0.1
Samsung S6 Droid 6.0.1
Simulator Tested On:
Visual Studio emulator Droid 5.1.1
Expected Behavior
No file should be created when you canceling a photo
Actual Behavior
When you take a photo and you choose to cancel, an empty file is created on disk
Steps to reproduce the Behavior
Select cancel when to take a photo

This issue was moved to jamesmontemagno/MediaPlugin#2


DataPackage SetData method should have an example

The only hint about how to use this method and what to pass in as the second object parameter is "RandomAccessStream" which links to a whole article about writing/reading text from files.
It'd be great to have a complete example for how to use the SetData method to binary serialize an instance of a C# object to the proper stream format to pass to this method (and how to deserialize it back to the same type again from GetData).

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

ID: 1cd0e6e0-c1fc-6f5d-9552-1d11dbff8320
Version Independent ID: 4aba3b11-3c28-e6a5-865a-5daea64c590c
Content: DataPackage.SetData(String, Object) Method (Windows.ApplicationModel.DataTransfer) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.ApplicationModel.DataTransfer/DataPackage.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for submitting your feedback and we apologize for the delayed response. I see that updates have not been made since the creation of this Issue, so I went ahead and created an internal work item to work on an example and resolve this doc bug. The timeline for this may vary, so I will go ahead and close this and circle back when the changes are live.
We have moved to a new feedback system, and I want to ensure that we don't lose access to the feedback in our open issues. Though the Issue will be marked as Closed, we will monitor any replies that come afterwards so treat this like an open Issue. Many thanks.


connecting libusb in cmake

I am using cmake. How can I include libusb in the assembly? (without cmake everything works)

Hi @Roman-Koshelev, libusb doesn't export cmake files, so we can only use find_path and find_library to solve this issue. Example:
find_path(LIBUSB_INCLUDES libusb.h PATH_SUFFIXES libusb-1.0)
find_library(libusb libusb-1.0 REQUIRED)

target_include_directories(... PRIVATE ${LIBUSB_INCLUDES})
target_link_libraries(... PRIVATE ${libusb})

thanks

And where is the libusb finder implemented (which is needed for find_library ())?


Web directory

I cannot find the web directory inside the example app

Hello @Dev291199 ,
We plan to add web support for Flutter, but it is not currently on our immediate roadmap. We will keep you updated on the progress of this feature and notify you once it is released.
Thank you for your patience and understanding.
Best regards,
Bhahirathan


[Bug]: Can't initialise OpenEphysRecordingInterface with more than one events folder

What happened?
I tried to use NWB Guide to convert the openephys data from Constantinople lab, but received an error. The data is in OpenEphys binary format and has multiple streams (NIDAQ, AP, LF):

I can parse the stream names with OpenEphysBinaryRecordingInterface:
from neuroconv.datainterfaces import OpenEphysBinaryRecordingInterface

folder_path = "/Volumes/t7-ssd/GCP/Constantinople/J076_2023-12-06_13-24-28/Record Node 117"
print(OpenEphysBinaryRecordingInterface.get_stream_names(folder_path=folder_path))
>>>['Record Node 117#NI-DAQmx-114.PXI-6133', 'Record Node 117#Neuropix-PXI-119.ProbeA-AP', 'Record Node 117#Neuropix-PXI-119.ProbeA-LFP']

However, when I try to initialise the interface, I get an Exception that more than one events folder found (see traceback).
I haven't seen OpenEphys data with multiple streams, so I would appreciate any suggestions where to start fixing this.
@CodyCBakerPhD (maybe @h-mayorquin ) FYI @bendichter
Steps to Reproduce
from neuroconv.datainterfaces import OpenEphysRecordingInterface

folder_path = "/Volumes/t7-ssd/GCP/Constantinople/J076_2023-12-06_13-24-28/Record Node 117"

interface = OpenEphysRecordingInterface(folder_path=folder_path)

Traceback
Traceback (most recent call last):
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/neuroconv/datainterfaces/ecephys/openephys/openephysbinarydatainterface.py", line 94, in __init__
    raise error
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/neuroconv/datainterfaces/ecephys/openephys/openephysbinarydatainterface.py", line 79, in __init__
    _open_with_pyopenephys(folder_path=folder_path)
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/neuroconv/datainterfaces/ecephys/openephys/openephysbinarydatainterface.py", line 18, in _open_with_pyopenephys
    pyopenephys_file = pyopenephys.File(foldername=folder_path)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/pyopenephys/core.py", line 175, in __init__
    self._experiments.append(Experiment(op.join(self._absolute_foldername, rel_path), id, self))
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/pyopenephys/core.py", line 263, in __init__
    Recording(op.join(self._absolute_foldername, rel_path), id, self, verbose=verbose)
  File "/Users/weian/anaconda3/envs/constantinople-lab-to-nwb/lib/python3.11/site-packages/pyopenephys/core.py", line 425, in __init__
    raise Exception("More than one events folder found!")
Exception: More than one events folder found!

Operating System
macOS
Python Executable
Conda
Python Version
3.11
Package Versions
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] Have you ensured this bug was not already reported?

It looks like this might be an issue with pyopenephys, written by @alejoe91 (who is currently on vacation). @weiglszonja, could you please open a sister issue there and link it back to here?

It looks like pyopenephys intentionally does not support binary data with > 1 event:
https://github.com/CINPLA/pyopenephys/blob/b81e29e852fe3badb0ef7beb9f763e80ead34029/pyopenephys/core.py#L416-L425

It looks like pyopenephys intentionally does not support binary data with > 1 events folder:
https://github.com/CINPLA/pyopenephys/blob/b81e29e852fe3badb0ef7beb9f763e80ead34029/pyopenephys/core.py#L416-L425

Yeah right, but it seems like we're using it to get the session_start_time:
https://github.com/catalystneuro/neuroconv/blob/6a3046d187bbe81b07731857804df839ef505060/src/neuroconv/datainterfaces/ecephys/openephys/openephysbinarydatainterface.py#L120
so maybe there could be a workaround to still fetch that; or better if we could retrieve this from OpenEphysBinaryRecordingExtractor

Hmm, well first of all, let's try our best not to access private attributes. Private attributes can be removed without warning, which would break our code.
Second, yes, it does seem like there should be a more robust way to access this information. As a quick fix, let's just put this in a try/except, and raise an issue to handle this more robustly in the future.

Would it be bold to parse the settings.xml file ourselves in NeuroConv for accessing the session_start_time?
We're using OpenEphysBinaryRecordingExtractor from spikeinterface for writing the traces, it looks like in NeuroConv we really just care about accessing the DATE field from the settings.xml file which happens in pyopenephys:
https://github.com/CINPLA/pyopenephys/blob/b81e29e852fe3badb0ef7beb9f763e80ead34029/pyopenephys/core.py#L324

oh, yeah, let's just parse it ourselves. If that's really all we use pyopenephys for, that's overkill just to read and parse a date string


adopt features from snewpy

snewpy is a SNOwGLoBES frontend developed by members of the SNEWS 2.0 collaboration. Like SNOwGLoBES, it’s not a replacement for sntools; but it does have features that would be useful in sntools, too:

support for more flavour transformations
support for more supernova models

Time- and energy-independent flavour transformations should be straightforward to add to sntools, analogous to the current implementation of MSW effect in normal/inverted mass ordering. (However, it will be necessary to change the command line interface to e.g. --transformation=AdiabaticMSW_NMO (instead of --ordering=normal), once additional transformations are available.)
Time- or energy-dependent flavour transformations will require further changes to sntools.
For additional supernova models, I need to check if it’s possible to write an adapter that maps snewpy’s interface to the one expected of an input format in sntools.

First steps are now on the JostMigenda/SNEWPyIntegration branch.

I’ve added the majority of the models and transformations in a PR earlier today.
The following flavor transformations from SNEWPY are not yet supported:

[x] NoTransformation and AdiabaticMSW: identical to the transformations already natively supported by sntools (in fact, the native implementations were renamed for consistency)
[ ] NeutrinoDecay: has energy-dependent transformation propabilities, which requires additional changes to sntools
[ ] AdiabaticMSWes and NonAdiabaticMSWes: requires additional input parameters (sterile mixing angles)

The following supernova models from SNEWPY are not yet supported:

[ ] Fornax_2019 and FornaxModel: alternative implementations of the same model; will wait until SNEWPY has decided how to move forward.
[ ] Janka: inconsistent return type of get_time(), see issue 45 on SNEWPY repo.
[x] Warren_2020: identical to the native warren2020 input format already supported by sntools.

Additional open issues:

[ ] SNEWPY currently needs to be installed separately by the user. Once it is available on PyPI, it should be added as a regular dependency in requirements.txt and setup.py.
[ ] Using a supernova model from SNEWPY is currently much slower than a model that’s natively supported by sntools. E.g. using --format SNEWPY-Warren_2020 is about 3× slower than using the native --format warren2020 on the same input file. See the issue on the SNEWPY repo linked above for details.
[ ] Should the format names in sntools be renamed for consistency with SNEWPY SupernovaModels?

https://github.com/JostMigenda/sntools/commit/5b110082141f5630370ec01b0f40306868500647 improves performance for SNEWPY supernova models, reducing the difference in the Warren_2020 example to under 2×. Still not ideal, but this was probably the lowest-hanging fruit.

Several new models were added to SNEWPY over the last few months:

[ ] Tamborra_2014, Walk_2018, Walk_2019 and Fornax_2021: should be straightforward, to be added in an upcoming PR
[ ] OConnor_2013: needs mass and EOS as arguments (instead of path of input file), which would require more changes to accommodate


How to debug the lombok-intellij-pligins ?

Short description
How to debug the lombok-intellij-pligins ?
I am a freshman and want to know how to add  a new feature to  lombok-intellij-pligins .
There is any methods to debug the lombok-intellij-pligins?

The first thing you need to know about Gradle.
After cloning the project, open it with IDEA, open the Gradle tab, and click on the Sync button (the two encircling arrows).
This process will download the Intellij source code for you, which will take some time, so please try again when the network fluctuation fails.
Next, in the Gradle tab, expand the project Tasks > intellij, right-click on runIde and select Debug to start it.
The rest is no different than debugging a normal Java application.
But please note: You MUST get the compiler to support your new feature as well, otherwise you will see compiler errors.
For the compiler (javac or ecj), see: https://github.com/rzwitserloot/lombok
If you need help, please contact me via QQ( 931920447 ).


[docs]i18n plan

Context
I feel Playwright is super cool for E2E testing tool ever. However, There is no Japanese or some other langage documentation except English.
Ask
Is there plan to translate documentation in Japanese or some other langageges so far at Playwright.dev?
I will be able to contribute as translater in Japanese in the future if there is the plan.

There are currently no plans for translating the docs into other languages. I'll keep the issue open so we can get feedback on it and we can reconsider it in the future.
Thanks already for offering your support!

closing this as I don't see us adding other languages for now especially as we continue to rework and improve the docs therefore keeping languages updated would be quite difficult


Redis operator

This PR adds the initial version of the Redis Operator for K8s. It was based on the mongodb, elasticsearch and this PR.
It was made with the goal to first serve for the Commercial Systems team to deploy a Redis cache without replication. Therefore, at first it does not provide Redis cluster capabilities.
As a newbie on the operator framework, juju and at certain point k8s I still uncertain of parts of this code. I have left a comment on each of them. If anyone knows, I'd appreciate any further clarification.

Thanks @alnvdl and @cmars for the review so far. One common thing you have commented in common was the volume thing I was not sure about. I will let a TODO there so that if we need it in the future we know directly where it should go.
@cmars I have removed the JUJU variables then. It was also something @balbirthomas suggested me to do in a direct message.
@cmars I have added the PING to actually check the communication with Redis as your suggestion. By instantiating the client already establishes the connection, but the PING actually calls it, so that at least has the extra action of using the connection pool and being explicit in the code.

Thanks @alnvdl and @cmars for the review so far. One common thing you have commented in common was the volume thing I was not sure about. I will let a TODO there so that if we need it in the future we know directly where it should go.
@cmars I have removed the JUJU variables then. It was also something @balbirthomas suggested me to do in a direct message.
@cmars I have added the PING to actually check the communication with Redis as your suggestion. By instantiating the client already establishes the connection, but the PING actually calls it, so that at least has the extra action of using the connection pool and being explicit in the code.


Apex support

Hi, I'd like to ask the progress of apex support.
If no one is doing it, I believe I could do something to help.

No one is working on it for now, so if you want to tackle this, feel free to give it a go!


 There seems to be a leak somewhere that leads to a 100% cpu usage by the lua garbage collector after a few minutes. Is this a know issue of lua.vm.js or something wrong on my side ? My lua stack is always empty after each function call and I've no idea what other check I can to do try and figure out where the problem is.

Thanks for your interest.
I've moved on from lua.vm.js and am now only accepting bug fixes.
Please consider using the successor to lua.vm.js, fengari

OK I'll give fengari a try, though I'm a bit concerned about the performance impact of going from asm.js to plain javascript. I'll be able to give you some data about the switch soon. Thanks for your reply.

I actually gained frames per second by switching to fengari! (went from 45 fps to 55 fps)! great job!

@jice-nospam great ! Is your project viewable somewhere ? Any use of fengari is interesting to us.

It's still in early stage. You can test it here: http://roguecentral.org/~jice/viper/


[Derived]RecvAsync is cancelled while running tensorflow RNN tutorial code on GPU

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
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): Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): 2.5.0
Python version: 3.8.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:  CUDA: 11.2.0
GPU model and memory: NVIDIA Titan RTX 24 GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I am running this tutorial https://www.tensorflow.org/text/tutorials/text_classification_rnn and during  training I am getting below error,
Epoch 1/10
67/391 [====>.........................] - ETA: 18s - loss: 0.6928 - accuracy: 0.4977
CancelledError                            Traceback (most recent call last)
 in
----> 1 model.fit(train_dataset, epochs=10,
2                     validation_data=test_dataset,
3                     validation_steps=30)
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\keras\engine\training.py in fit(self, x, y, batch_size, epochs, verbose, callbacks, validation_split, validation_data, shuffle, class_weight, sample_weight, initial_epoch, steps_per_epoch, validation_steps, validation_batch_size, validation_freq, max_queue_size, workers, use_multiprocessing)
1181                 _r=1):
1182               callbacks.on_train_batch_begin(step)
-> 1183               tmp_logs = self.train_function(iterator)
1184               if data_handler.should_sync:
1185                 context.async_wait()
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\def_function.py in call(self, *args, **kwds)
887
888       with OptionalXlaContext(self._jit_compile):
--> 889         result = self._call(*args, **kwds)
890
891       new_tracing_count = self.experimental_get_tracing_count()
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\def_function.py in _call(self, *args, **kwds)
915       # In this case we have created variables on the first call, so we run the
916       # defunned version which is guaranteed to never create variables.
--> 917       return self._stateless_fn(*args, **kwds)  # pylint: disable=not-callable
918     elif self._stateful_fn is not None:
919       # Release the lock early so that multiple threads can perform the call
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\function.py in call(self, *args, **kwargs)
3021       (graph_function,
3022        filtered_flat_args) = self._maybe_define_function(args, kwargs)
-> 3023     return graph_function._call_flat(
3024         filtered_flat_args, captured_inputs=graph_function.captured_inputs)  # pylint: disable=protected-access
3025
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
1958         and executing_eagerly):
1959       # No tape is watching; skip to running the function.
-> 1960       return self._build_call_outputs(self._inference_function.call(
1961           ctx, args, cancellation_manager=cancellation_manager))
1962     forward_backward = self._select_forward_and_backward_functions(
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\function.py in call(self, ctx, args, cancellation_manager)
589       with _InterpolateFunctionError(self):
590         if cancellation_manager is None:
--> 591           outputs = execute.execute(
592               str(self.signature.name),
593               num_outputs=self._num_outputs,
~\AppData\Roaming\Python\Python38\site-packages\tensorflow\python\eager\execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
57   try:
58     ctx.ensure_initialized()
---> 59     tensors = pywrap_tfe.TFE_Py_Execute(ctx._handle, device_name, op_name,
60                                         inputs, attrs, num_outputs)
61   except core._NotOkStatusException as e:
CancelledError:  [Derived]RecvAsync is cancelled.
[[{{node Adam/Adam/update/AssignSubVariableOp/_57}}]]
[[gradient_tape/sequential/embedding/embedding_lookup/Reshape/_54]] [Op:__inference_train_function_22715]
Function call stack:
train_function
Describe the expected behavior
It should work ok without issues. I have not changed any code, it is a tutorial from tensorflow official documentation. If you train on CPU there is no issue so something wrong with training on GPU
Contributing

Do you want to contribute a PR? (yes/no):
Briefly describe your candidate solution(if contributing):

Standalone code to reproduce the issue
It is very easy to reproduce. Just run https://www.tensorflow.org/text/tutorials/text_classification_rnn in a jupyter notebook on a machine that has GPU
This seems to be similar to https://github.com/tensorflow/tensorflow/issues/33721, opening this because that previous issue was closed without a resolution.

@dhavalsays ,
I haven't faced any issue while executing the code in tf v2.5 and v2.4.Please find the gist.
please try to execute code in virtual environment and limiting GPU memory growth using any of the methods listed in this guide and check if it helps.Thanks!

I tried limiting GPU memory growth check here, line # 3 but still getting error at line # 26
@tilakrayal I can see it works ok in Google colab but there is definitely a problem when someone runs this on local environment that has following,
tf 2.5
CUDA 11.2 (Windows 10)
Python 3.8.5
Also many others are facing same problem in https://github.com/tensorflow/tensorflow/issues/33721

Have you tried reducing your batch size to a smaller value in addition to limiting gpu memory growth?


Koi animations and models need to be improved.

Koi fins need to be designed better.
Extra animations need to be added for he koi.

Add flopping animation while out of water.
Fins not rendering correctly after changing the design.


Added italian translations, corrected some grammar error, fixed Travis error(s)

Forked and fixed syntax error from @marchacio PR, I tested and it seems working.
I also corrected some grammar error: "qual'è --> qual è"

Also fixed missing libraries in dev-requirements.txt now in README there's a link to my Travis button, just merge
82b3a33 commit to evoid this.
Greetings from Italy :pizza:.

please revert commit f946f2e which contains a link to your Travis button otherwise this pull request cannot be merged

please revert commit f946f2e which contains a link to your Travis button otherwise this pull request cannot be merged

Done, sorry, I forgot my Travis button.


Create determinant_solver.cpp

This can solve any 3x3 determinant

@arnavbansal2764 Thanks for making the contribution. Please change the location of your file to the correct folder.


DEV 10348.4 set an env INSTALLED_DOCKER_VERSION when admiral boots up

https://github.com/Shippable/pm/issues/10348

this is done, closing


opensmt: 20101017 -> 2.0.1

Motivation for this change
Broken since 2014, googlecode site says it migrated to github but nothing is visible there
Things done

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

[ ] 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
[ ] Fits CONTRIBUTING.md.

https://github.com/usi-verification-and-security/opensmt

Ok so I will look to update the package instead

Working on it, they migrated to cmake build (cool) but it tries to download an external project (googletest) :'(
Any idea how to properly handle this ?

Maybe predownload and feed it inside the build directory?

I don't know where the folder is supposed to appear and to hook into cmake's FetchContent system

@freezeboy friendly ping

I could not find where the folder is supposed to appear for cmake, any inputs are welcome

Bit of a hack but this seems to work for me.
diff --git a/pkgs/applications/science/logic/opensmt/default.nix b/pkgs/applications/science/logic/opensmt/default.nix
index 249d0722699..3641229f7c1 100644
--- a/pkgs/applications/science/logic/opensmt/default.nix
+++ b/pkgs/applications/science/logic/opensmt/default.nix
@@ -1,6 +1,7 @@
 { stdenv, lib, fetchFromGitHub
 , cmake, libedit, gmpxx, bison, flex
 , enableReadline ? false, readline
+, gtest
 }:

 stdenv.mkDerivation rec {
@@ -17,6 +18,15 @@ stdenv.mkDerivation rec {
   nativeBuildInputs = [ cmake bison flex ];
   buildInputs = [ libedit gmpxx ] ++ lib.optional enableReadline readline;

+  preConfigure = ''
+    substituteInPlace test/CMakeLists.txt \
+      --replace 'FetchContent_Populate' '#FetchContent_Populate'
+  '';
+  cmakeFlags = [
+    "-Dgoogletest_SOURCE_DIR=${gtest.src}"
+    "-Dgoogletest_BINARY_DIR=./gtest-build"
+  ];
+
   meta = with lib; {
     description = "A satisfiability modulo theory (SMT) solver";
     maintainers = [ maintainers.raskin ];

In integrated your patch, thank you for it.
I don't know precisely if other people use it in nixpkgs, I have no strong feeling about it.
I let you decide if it is worth integrating this version now it works, or to remove it from nixpkgs

In integrated your patch, thank you for it.
I don't know precisely if other people use it in nixpkgs, I have no strong feeling about it.
I let you decide if it is worth integrating this version now it works, or to remove it from nixpkgs


extend on chain identity with image

vik just discovered that onchain identity supports picture this would be nice we could include on user profile their pictures

https://github.com/paritytech/substrate/blob/15cfd9c5dbbfa1f3ed49623eb55b67354d1645e9/frame/identity/src/types.rs

Looks like image field was added 😌❤

Looks like the image field was added 😌❤
is this working on KodaDot? This would be AWESOME once it will be done.

Continue to

#4751


Add form reportValidity() method

Learned about this API today, but not up to date docs at https://developer.mozilla.org/en-US/docs/Web/API/HTMLFormElement/reportValidity and doesn't seem to be covered by http://caniuse.com/#feat=form-validation
demo here. Appears to work in desktop Chorme and Firefox.

I'd say this should be covered by #1068?

Yup, covered by #1068. I have also updated the title there for clarity.


Add token for current line number

Added file.line which returns string version of line number of active editor's selection (or '0' if there is no valid editor). Useful for jumping to matching position in blame of large files.

Hey Taylor. Thank you for the pull request! Looks very good.
I don't know when the next release will be, but this will be part of it.


Remove dev branch

I would like to remove the dev-branch.
I think it is mainly confusing. We introduced  it to contain examples of a not release versions. At the moment we do have different packages so that each package have to create its own non-visible branch with examples for upcoming releases. We do not have the same release date for all packages.
If anybody knows more about the actual state of the dev branch, feel free to do what every is necessary and remove it. I would just merge it into the master so that nothing gets lost.

I also introduced the labels "windpowerlib", "tespy" and "solph" so that one can label issues and PR according to the package. I will add a label for each package that provides an example here.

Would it make sense to have the examples in the corresponding repositories instead? This would also allow to have new features and the corresponding example in one PR.


Support optionals

Proto 3.15.0 enabled optionals by default, which especially relevant for kotlin due to nullability. Sadly the generator crashes when using optionals. Will support be added?

Greetings @Cydhra!  This repository isn't being actively maintained at this stage.


EPS occasionally soft-faults (status 2)

On large frame vehicles, soft faults were randomly happening. It appears these were associated with the RT rate checks on the panda latching.

Disabling RT rate limit checks has resolved this issue - there is another issue covering the safety code


feat: add base64js and modify some utils、components

Signed-off-by: moweiwei<EMAIL_ADDRESS>
What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

/lgtm
/approve


Code block text from markdown cells gets default hydrogen styling

Create Python file with the following code:

#%%md
# `Shift-Enter`
#Shift-Enter

Run line 2 and line 3 in any way you would like
The results will appear visually the same

Close this for now since this issue is sort of solved in #1670.


Upgrade to jupyter-js-services 0.14

Still checking for errors...

FYI I've upgraded in #259.

Nice, thanks!


Pipeline fails in prep_align_input stages when directories contain spaces

This was not intentional, but at a time, someone created a result folder for the sequencer which included spaces. When running a new analysis, it bombed right in prep_align_inputs:
exceptions from call to method: prep_align_inputs
[Engine Exception]CalledProcessError: Command 'set -o pipefail; gunzip -c /storage/miseq_data/sequencing_storage/OV.225 1_2 DNA/fastq/21506_blood_S3_L001_R1_001.fastq.gz | /opt/bcbio/galaxy/../anaconda/bin/pbgzip -n 8  -c /dev/stdin > /scratch/bcbiotx/a71b7929-3f32-4091-9671-d5c0fb30ba9e/tmp63qHDL/21506_blood_S3_L001_R1_001.fastq.gz
gzip: /storage/miseq_data/sequencing_storage/OV.225.gz: No such file or directory

I already told the relevant people not to use spaces in the directories. ;) But I guess these commands should be shell-escaped to prevent this?

Hi @lbeltrame
I remember we did something to avoid this while ago.
Please, let us know if you still find further issues with this.
cheers


"Provide Feedback" doesn't work

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create#
Operating System: windows
Steps to reproduce:
Go to https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create#
Read article
Run into an issue
Press "I ran into an issue"
(An overlay opens with options "Report an issue" and "Provide feedback")
Press "Provide feedback"
Expected outcome
Any variation of feedback mechanism - texbox opens, default email program opens, windows help feedback app
Actual outcome
The "I ran into an issue" overlay closes.

Testing environment: Microsoft Edge for Business, Version 121.0.2277.112 (Official build) (64-bit)
Extension: uBlock Origin Version 1.55.0
OS: Microsoft Windows 11 Pro, Version	10.0.22631 Build 22631
With uBlock Origin installed and enabled, check I navigate to https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/create# and click the Feedback button in the sidebar, I don't get a Feedback popup at all.
When I test on the Usabilla (GetFeedback) site, https://www.getfeedback.com/sign-in/, I don't even get a "feedback" (Help us improve) button. With uBlock Origin enabled, I do get a button, and the popup works.
My recommendation, in conjunction with the comments on this ticket and its pr: https://github.com/NuGet/NuGetGallery/issues/7784, is that we should not show the "Feedback" button if the script is blocked, e.g. by an adblocker.


Release notes for v1.14.1 including explanation for the failed v1.14.0 release

Preview: https://deploy-preview-1411--cert-manager-website.netlify.app/docs/releases/release-notes/release-notes-1.14
Updating versions for forthcoming patch release  v1.14.1
Part of:

https://github.com/cert-manager/cert-manager/issues/6710

/hold until release is published.

/unhold
This is merging into release-next, which will then be merged to master in #1400. That PR is the one on hold until the release.