Build failure: eventlet

Steps To Reproduce
Steps to reproduce the behavior:

build eventlet

I used python311Full with enableOptimizations = true;.

FAILED tests/greenio_test.py::TestGreenSocket::test_nonblocking_accept_mark_as_reopened - AssertionError: Readers: Listener FdListener('read', 17, <built-in method s...
FAILED tests/greenio_test.py::TestGreenSocket::test_raised_multiple_readers - AssertionError: Readers: Listener FdListener('read', 17, <built-in method s...
FAILED tests/greenio_test.py::TestGreenSocket::test_recv_into_timeout - AssertionError: Readers: Listener FdListener('read', 17, <built-in method s...
ERROR tests/greenio_test.py::TestGreenSocket::test_full_duplex - AssertionError: Readers: Listener FdListener('read', 17, <built-in method s...
= 5 failed, 562 passed, 120 skipped, 18 deselected, 1163 warnings, 1 error in 48.02s =

Additional context
The tests pass if one of these groups of tests are ignored:

test_full_duplex

or

test_invalid_connection
test_nonblocking_accept_mark_as_reopened
test_raised_multiple_readers
test_recv_into_timeout

Notify maintainers

@SuperSandro2000

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.4, NixOS, 24.05 (Uakari), 24.05.20231204.d08f638`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.19.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/store/g95dxc3a9qslvkrmjbh8dqvbw8gwin1j-85zhg3l0s4cnw3n0a6p31gpkyljw8bv4-source`

Add a :+1: reaction to issues you find important.

Does this only happen when building with optimizations?

Grep for python310 = and you will find that it includes bluez and X11 support. Generally er don't test packaged with full and it is frankly quite unused.
Are you constructing the overlay correct? If so you should rebuild everything with python310Full from probably setuptools or so. If not the overwrite might be incomplete and you could be using different interpreters.
Anyway I am inclined to close this as won't fix and my advise is to not use python3Full in such a case.

I have been running into this intermittently, looks like a flaky test. See also the comment at https://github.com/NixOS/nixpkgs/pull/281181#pullrequestreview-1822208759
Should be fixed upstream in https://github.com/eventlet/eventlet/pull/905

Also running into this on Arch linux. Can we get the nix package updated to upstream?

It would be pretty easy to apply the patch for now, but it might be better to wait for eventlet 0.35.2 (which should presumably release soon?) and update the package? I'm not sure if any other packages in nixpkgs will break when moving from 0.33. If anyone is running into this, you just need:
From c127b02b60a8a374d36e98f98e452b37cf59b864 Mon Sep 17 00:00:00 2001
From: Genevieve<EMAIL_ADDRESS>Date: Sun, 4 Feb 2024 19:21:03 -0500
Subject: [PATCH 1/1] patch eventlet for kernel 6.6

---
 .../python-modules/eventlet/default.nix            |  3 +++
 .../python-modules/eventlet/greenio-test.patch     | 14 ++++++++++++++
 2 files changed, 17 insertions(+)
 create mode 100644 pkgs/development/python-modules/eventlet/greenio-test.patch

diff --git a/pkgs/development/python-modules/eventlet/default.nix b/pkgs/development/python-modules/eventlet/default.nix
index 07f19e3493d9..64a7b95ac5dd 100644
--- a/pkgs/development/python-modules/eventlet/default.nix
+++ b/pkgs/development/python-modules/eventlet/default.nix
@@ -34,6 +34,9 @@ buildPythonPackage rec {
   };

   patches = [
+    # shrink TCP buffer window before connecting, so test passes on kernel>=6.6
+    # from https://github.com/eventlet/eventlet/pull/905
+    ./greenio-test.patch
     # Python 3.12 fixes:
     # - remove usage of distutils
     # - replace ssl.wrap_socket usage
diff --git a/pkgs/development/python-modules/eventlet/greenio-test.patch b/pkgs/development/python-modules/eventlet/greenio-test.patch
new file mode 100644
index<PHONE_NUMBER>00..b6f78b5bc813
--- /dev/null
+++ b/pkgs/development/python-modules/eventlet/greenio-test.patch
@@ -0,0 +1,14 @@
+diff --git a/tests/greenio_test.py b/tests/greenio_test.py
+index ce87cf5..b4bce02 100644
+--- a/tests/greenio_test.py
++++ b/tests/greenio_test.py
+@@ -417,8 +417,8 @@ class TestGreenSocket(tests.LimitedTestCase):
+
+         server_evt = eventlet.spawn(server)
+         client = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
+-        client.connect(('<IP_ADDRESS>', listener.getsockname()[1]))
+         bufsized(client)
++        client.connect(('<IP_ADDRESS>', listener.getsockname()[1]))
+         large_evt = eventlet.spawn(read_large, client)
+         eventlet.sleep(0)
+         client.sendall(b'hello world')
--
2.43.0

Awesome. Thank you @genevieve-me

#288357


(fix) fixed tests

Description
fixed tests setupOptions
Release required?

[X] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[x] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

@vbontempi To get pipeline to pass, you need to bump to landing zone v6. There is a renovate PR for this (#78) but it failed as it needs the fix your adding in this PR.
Can you maybe update to landing zone v6 in this PR? Then tests should pass

done

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

GitHub release
v1.2.6

Your semantic-release bot :package::rocket:


No shifting

Thanks a lot for your script! I'm not that keen with JS and wonder how I need to adjust the code to only have one column with the current date and floor price? This could be overwritten each time. Cheers, Felix

Hey @felixdorner did you figure this out? I think if you just change shouldAddColumn to always return false it should work ( have not tested).
function shouldAddColumn(date){
     return false;
}

Hey @printloop, that helps. My hack was just to delete the column... Thanks a lot.


Failure in creating tasks

Hi,
I'm having a strange issue wherein the tasks I attempt to create isn't getting created and listed in the Scheduled Tasks console. The error that I get is as follows.
Access is denied. (Exception from HRESULT: 0x80070005 (E_ACCESSDENIED))   at Microsoft.Win32.TaskScheduler.V2Interop.ITaskFolder.RegisterTaskDefinition(String Path, ITaskDefinition pDefinition, Int32 flags, Object UserId, Object password, TaskLogonType LogonType, Object sddl)   at Microsoft.Win32.TaskScheduler.TaskFolder.RegisterTaskDefinition(String Path, TaskDefinition definition, TaskCreation createType, String UserId, String password, TaskLogonType LogonType, String sddl)
The following is the code I'm using to create the task.
using ( TaskService service = new TaskService( ) )
{
ExecAction action = new ExecAction( @"c:\windows\notepad.exe" );
                TimeTrigger trigger = new TimeTrigger( );
                trigger.StartBoundary = DateTime.Now.AddMinutes( 2 );
                trigger.EndBoundary = trigger.StartBoundary.AddHours( 12 );

                TaskDefinition definition = service.NewTask( );
                definition.RegistrationInfo.Description = txtTaskName.Text;
                definition.Actions.Add( action );
                definition.Settings.AllowHardTerminate = true;
                definition.Principal.RunLevel = TaskRunLevel.Highest;
                definition.Triggers.Add( trigger );

                try
                {
                    service.RootFolder.RegisterTaskDefinition( txtTaskName.Text, definition, TaskCreation.CreateOrUpdate, txtUserName.Text, txtPassword.Text, TaskLogonType.Password );
                    txtResponse.Text = string.Format( "Task: {0} registered successfully.", txtTaskName.Text );
                }
                catch ( Exception ex )
                {

                    txtResponse.Text = ex.Message + Environment.NewLine + ex.StackTrace;
                }

            }

I'm able to manually create the task using the console but not programmatically.
I'm using Windows Server 2008 R2 Standard, .NET Framework 4.0 and v.1.6.4186.14621 of Microsoft.Win32.TaskScheduler.dll
Can anyone please give me a clue as to what must be going wrong here? Is it something to do with CAS settings in the machine or anything else?
Thanks & Regards,
Sanjay.
Originally posted: 2011-09-21T08:03:30

My guess is that it has to do with the privileges of the account you are using to run the program. That account has to have the rights to create tasks. You have two ways to specify an account. When you create a TaskService instance with no parameters, the account running the executable is used. If you are launching this from a user session, make sure you are running as Administrator so UAC doesn’t choke the privileges. You can also instantiate TaskService specifying the username, password and domain. This way ensures that you are running with known privileges.
Originally posted: 2011-09-21T11:34:07

Hi Dahall,
Thank you so much for your guidance. The code was as per your suggestion. But what I could find is that it's some sort a permissions issue in this particular server. I also tried impersonating the task creation
code in an impersonation context and calling the default ctor of TaskService using an account that's a member of the Administrators group. It still fails. But if the application is set to "Run as Administrator"
or set to run in XP SP3 compatibility mode, then it works fine. I suppose it's something to do with User Access Control. UAC seems to be disabled on the server though. What are your picks on this and what
do you think must be happening?
Thank you once again for all the great suggestions.
Regards,
Sanjay.
Originally posted: 2011-10-12T03:58:43

You don't need to use impersonation when connecting. Use the additional parameters on the TaskService constructor to specify the username, domain, and password of the account that has the correct privileges. Then you should be able to register the task without error.
Originally posted: 2011-10-12T13:46:03

Hi,
Am having the same issue, and understand that we require admin rights to create the task, if I run my program as a normal user, i get the access is denied exception as above, if I run as adminstrator the task is created fine. (I am using TaskService with no paramaters)
My question is though why can I successfully create a task as the normal user, through the windows task console, using the same settings as I am using programmatically? I would expect it to have the same behaviour as my program and not be allowed. Does the Windows task console run with escalated priveleges without me knowing it? (I'm on Win 7 so would expect the UAC pop up if it was doing this).
Thanks in advance if anyone can advise.
Andy
Originally posted: 2011-10-13T05:10:54

From what I have read about UAC, yes, all MMC actions run with elevated privileges. I found an article here that shows how to request elevated privileges for your .NET application. Let me know if this helps.
Originally posted: 2011-10-13T14:58:56


Youtube Downloader & a small fix in README.md

Added: Youtube Downloader
Added Youtube Downloader, that can be used to download youtube videos & playlists.
Changes Made in: README.md

Now LinkedIn Badge is clickable and redirects to your profile.
Added info about my own contribution in SCRIPTS table.

To Take Care of:

You may need to change SOURCE in README.md

why did you close it ?

aah i thought that you went inactive

yes

done. #22


Translation Fix

There were some minor errors in the translation you made but it was ok overall

Thank you!


What's the plan for Parameter Files?

Is Bicep going to include Parameter Files?  Or will we be keeping the template in Bicep and the Parameter file in the ARM format?

We have #399 tracking this. We're not closed on the plan yet, but expect to have some "bicep-style" way of declaring parameter values. Feel free to share any thoughts you have on that thread!
Going to close this in the meantime


r11s-driver: Disable default loader caching in DocumentStorageService

After enabling "Network Error" retries in #5800, we would enter a strange retry loop where RetriableDocumentStorageService would output an error on retry, but it wouldn't make any new network calls. Disabling PrefetchDocumentStorageService resolves the issue, and I don't think we need it in r11s anyways.
The policy is still over-writable so that LocalDriver is not impacted by this change.

This change will not remove RetriableDocumentStorageService, it will only remove PrefetchDocumentStorageService. How does it help, if (based on description) the issue is related to RetriableDocumentStorageService?
If I have to guess, the bug is in PrefetchDocumentStorageService. Is this related to https://github.com/microsoft/FluidFramework/issues/4401?
Note that removal of PrefetchDocumentStorageService will have perf impact. It would be great to measure it before removing it.

This change will not remove RetriableDocumentStorageService, it will only remove PrefetchDocumentStorageService. How does it help, if (based on description) the issue is related to RetriableDocumentStorageService?
If I have to guess, the bug is in PrefetchDocumentStorageService. Is this related to #4401?

Yes, I believe the bug is in PrefetchDocumentStorageService, which is why I'm removing it here. RetriableDocumentStorageServicw works great without PrefetchDocumentStorageService between it and R11sDocumentStorageService.
Looking at it with fresh eyes today, I believe I've found the bug in PrefetchDocumentStorageService. On cachedRead, it's caching the Promise, so any future attempted read of a rejected promise will continue to be rejected, thus breaking RetriableDocumentStorageService.
https://github.com/microsoft/FluidFramework/blob/845d073af76e7ec29719e05338d024aef68fae69/packages/loader/container-loader/src/prefetchDocumentStorageService.ts#L52-L58
I believe there are 2 options to fix this behavior:

Swap RetriableDocumentStorageService and PrefetchDocumentStorageService in DeltaManager.connectToStorage, such that PrefetchDocumentStorageService calls RetriableDocumentStorageService.
Remove the cached promise in cachedRead immediately on rejection, such that RetriableDocumentStorageService retries will actually make a new request on each retry.

Note that removal of PrefetchDocumentStorageService will have perf impact. It would be great to measure it before removing it.

Agreed. I might have been too hasty to remove PrefetchDocumentStorageService in the name of "patch bug" rather than "maintain performance". I'm happy to keep PrefetchDocumentStorageService if we can fix the bug. @vladsud let me know which of the 2 above approaches would be preferred (or a 3rd approach that I'm not thinking of), and I'll open a PR to do it. Thanks!

Swapping the order of them sounds like reasonable step to me. It will ensure that only non-retryable errors would get cached, which is what we want.

Closed in favor of fix introduced in #5974. Measured initial doc-load performance was significantly faster with prefetching.


Native review for FileReader

From @piotrcyr on May 18, 2014 1:18
FileReader  module, in short, makes it possible to read in a file passed by user.
The module introduces:

abstraction of Files and Blobs
html  input control
a way to enhance any Element with a capability to open file selection dialog on click
ability to get files from user's drag and drop action
abstraction of a FileReader
reading (d'uh!) and slicing files

If you have any questions about implementation details you can ask them here or mail the creator directly
Copied from original issue: elm-lang/elm-package#32

From @evancz on May 24, 2014 19:40
I just opened some issues on the repo about the API, but I'm excited for this :)

From @Apanatshka on July 7, 2015 17:28
The elm-package tool says to open these reviews on the package.elm-lang.org issues list. Can you move it there?

The native review pipeline is stalled. That's probably not news to you, but I wanted to leave a note on all the open reviews. I cannot speak for everyone, but as one of the reviewers I just want to speak up and say that I will not be doing more reviews. The warning about publishing native packages has changed, and Evan has mentioned multiple times in different places that one of his priorities for 0.17 is improving this process. So I'm going to wait for that. I'm sorry we cannot move forward with publishing your library, but my experience is that even if we review and approve it, it will become stuck waiting for Evan to whitelist it.

Sorry to keep this in limbo for so long. I am moving towards the plan outlined here and discussed a bit further in the 0.17 announcement. The right thing to for now is use ports or get advice on the mailing list or on slack.


Any launch or yaml files work for LVI-SAM?

I'm trying to do some experiments of LVI-SAM using NTU VIRAL dataset.
Any available launch or yaml files work for LVI-SAM?
Thanks.

I tryed https://github.com/brytsknguyen/LVI-SAM.
The Image Feature seems right, but the Image Depth seems wrong.
Is this just a display problem？

Hi,
Yes, I have been testing this method with NTU VIRAL dataset and the visual fusion works normally. It's just that the visua features points appear to flip to the back of the camera. I think it's a visualization issue on the LVI's implementation side where the author assumed some fixed transform between the camera and the lidar.

If you can figure out how to make the visualization consistent I'd love to know too.

I tried. But as you said, there are some fixed transform in LVI-SAM, and the coordinate transformation is too complex.
I haven't solved it yet.
I'll let you know if it's solved.
Thanks.

emmm.......
I think I solved this problem.
You need to modify a lot of code.
the extrinsic parameters from lidar to IMU, camera to IMU
in /src/visual_odometry/visual_estimator/utility/utility.h and /src/visual_odometry/visual_estimator/utility/visualization.cpp
and the lidar to camera frame
in /home/hk/ws_pl-lvi-sam/src/PL-LVI-SAM/src/visual_odometry/visual_feature/feature_tracker.h

Hello, could you share the modified LVI-SAM code? It will be very helpful to me. Thank you in advance


Add workflow to run vgrid tests

vgrid client tests should run in the repo workflow as requested by @xmonader

Update: PR waiting for review


PEAR-1676: Upgrade Next JS to 14.

Description
Upgraded next: 14.1.0
Upgraded eslint-config-next: 14.1.0
Minimum Required Node Version: 20.11.0
Minimum Required NPM Version: 10.2.4
Performed codemod for next/link and next/image
Testing Note: Make sure the application pages works as it is!
Checklist

[ ] Added proper unit tests
[ ] Left proper TODO messages for any remaining tasks
[ ] Scanned for web accessibility with aXe, and mitigated or documented
flagged issues

Screenshots/Screen Recordings (if Appropriate)

Can you update the README with the new version of node and npm?

Done!

Verified all pages loaded.
But: headers of summary pages and "section headings" changed in style and became very tall. See pictures.

This should be fixed before merging.

Edit. the issue with fat headers/sections I saw is in prod too and only manifests after clicking through a particular series of items. this PR should be fine. I'll file a separate ticket.
👍

I do not see tall summary headers.

Could you verify that our existing app directory does not cause issues with the new nextjs app router?

Could you verify that our existing app directory does not cause issues with the new nextjs app router?

Are you referring to this directory?

Could you verify that our existing app directory does not cause issues with the new nextjs app router?

Are you referring to this directory? If so then I didn't notice that it's being used in our app.

Yes. The new app router looks for an app directory at the same location. It was unclear to me if the conflict would cause issues.  Since you've been able to run the portal in both dev and production modes, it sounds like it's not an issue.

holy many comments!
verified all the pages load as expected, including the 404. 👍 👍 👍 👍


Refactor templates within stack repo for ease in user customization use cases

Instead of using templates as src and running our build.sh scripts to generate the stack devfile, stack image Dockerfile, etc.... why don't we move our templates to a template branch and have our main branch be a downstream?
Doc with:
https://github.com/OpenLiberty/devfile-stack/issues/29

This is essentially done (in another way) in https://github.com/OpenLiberty/devfile-stack/commit/52d6ce58c1d2d14543178447aa165d5fa610c239


Basic authentication mechanism is implemented (e.g., JWT)

https://medium.com/@tericcabrel/implement-jwt-authentication-in-a-spring-boot-3-application-5839e4fd8fac


CLI should install boot jar to project root

Instead of under bin directory.  Also mdw-run, etc should look for it in the root.  Folks get the impression that they should cd into the bin directory to execute mdw run.  More importantly, in Cloud Foundry we typically cf push from the boot jar in the root dir.

We should provide more flexibility than my original comment.  Instead, let's add a new value in gradle.properties:
bootJarLoc=./

Then in CLI we'll look for the properly versioned mdw boot executable jar in that place.  Default will be project root.


Typescript: incompatible "this" in component when adding emits

Version
3.0.11
Reproduction link
https://github.com/danielvy/vue-emits.git
Steps to reproduce
run npm run build
See App.vue
What is expected?
Typescript project should compile.
What is actually happening?
Typescript reports an error:
TS2684: The 'this' context of type 'ComponentPublicInstance<{}, {}, {}, {}, { test(): void; }, "test"[], {}, {}, false, ComponentOptionsBase<{}, {}, void, {}, { test(): void; }, ComponentOptionsMixin, ComponentOptionsMixin, "test"[], string, {}>>' is not assignable to method's 'this' of type 'void | ComponentPublicInstance<{}, {}, {}, {}, {}, {}, {}, {}, false, ComponentOptionsBase<any, any, any, any, any, any, any, any, any, {}>>'.
  Type 'ComponentPublicInstance<{}, {}, {}, {}, { test(): void; }, "test"[], {}, {}, false, ComponentOptionsBase<{}, {}, void, {}, { test(): void; }, ComponentOptionsMixin, ComponentOptionsMixin, "test"[], string, {}>>' is not assignable to type 'ComponentPublicInstance<{}, {}, {}, {}, {}, {}, {}, {}, false, ComponentOptionsBase<any, any, any, any, any, any, any, any, any, {}>>'.
    Type 'ComponentPublicInstance<{}, {}, {}, {}, { test(): void; }, "test"[], {}, {}, false, ComponentOptionsBase<{}, {}, void, {}, { test(): void; }, ComponentOptionsMixin, ComponentOptionsMixin, "test"[], string, {}>>' is not assignable to type '{ $: ComponentInternalInstance; $data: {}; $props: {}; $attrs: Record<string, unknown>; $refs: Record<string, unknown>; $slots: Readonly<InternalSlots>; ... 7 more ...; $watch(source: string | Function, cb: Function, options?: WatchOptions<...> | undefined): WatchStopHandle; }'.
      Types of property '$emit' are incompatible.
        Type '(event: "test", ...args: any[]) => void' is not assignable to type '(event: string, ...args: any[]) => void'.
          Types of parameters 'event' and 'event' are incompatible.
            Type 'string' is not assignable to type '"test"'.
    17 |
    18 |        mounted() {
  > 19 |                this.$nextTick(() => {
       |                ^^^^
    20 |
    21 |                });
    22 |        },

The problem is here:
https://github.com/vuejs/vue-next/blob/master/packages/runtime-core/src/componentEmits.ts#L35


[BUG] Fix flaky test org.opensearch.index.ShardIndexingPressureConcurrentExecutionTests.testReplicaThreadedUpdateToShardLimitsAndRejections

Describe the bug
New flaky test org.opensearch.index.ShardIndexingPressureConcurrentExecutionTests.testReplicaThreadedUpdateToShardLimitsAndRejections ,  spotted in [1]
java.lang.AssertionError: expected null, but was:<org.opensearch.index.stats.IndexingPressurePerShardStats@5eae0090>
	at __randomizedtesting.SeedInfo.seed([9EFE9529CF42C133:9166A1D8EFC3EFA6]:0)
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotNull(Assert.java:756)
	at org.junit.Assert.assertNull(Assert.java:738)
	at org.junit.Assert.assertNull(Assert.java:748)
	at org.opensearch.index.ShardIndexingPressureConcurrentE

[1] https://build.ci.opensearch.org/job/gradle-check/1711/testReport/junit/org.opensearch.index/ShardIndexingPressureConcurrentExecutionTests/testReplicaThreadedUpdateToShardLimitsAndRejections/
To Reproduce
Steps to reproduce the behavior:
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.index.ShardIndexingPressureConcurrentExecutionTests.testReplicaThreadedUpdateToShardLimitsAndRejections" -Dtests.seed=9EFE9529CF42C133 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=fr-LU -Dtests.timezone=America/Goose_Bay -Druntime.java=17

Expected behavior
Test should pass
Plugins
Standard
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: CI
Version main

Additional context
Add any other context about the problem here.

https://github.com/opensearch-project/OpenSearch/pull/3633

https://github.com/opensearch-project/OpenSearch/pull/5283#issuecomment-1319863171

https://github.com/opensearch-project/OpenSearch/pull/5069#issuecomment-1320706981

This was added in https://github.com/opensearch-project/OpenSearch/pull/1336

Looked into this test a bit. The test fails only when rejection count if full i.e rejection count is equal to NUM_THREADS. Two possible for workaround/Questions:
-> Can rejection count be allowed to be equal to NUM_THREADS? is this something we should happening in test? or
-> Should we update logic of asserting to null when all the releasable's are null.
@getsaurabh02 can you please give your thoughts on it flaky test.

https://github.com/opensearch-project/OpenSearch/pull/4587#issuecomment-1332837125

Thanks for sharing the info @Rishikesh1159. I agree with the adding the optional null check as above, since the shardStoreStats objects move from hot to cold store in case they are not updated in a while.
Here, if rejection count equals to the NUM_THREADS which means rejections happened until the last request, the  shardIndexingPressure.shardStats().getIndexingPressureShardStats(shardId1) will return the shardStoreStats which was updated on the last request. In other cases, the shardStoreStats simply moves to the cold store and is returned null.

Thanks @getsaurabh02 for your detail explanation. What you said makes sense. I have put out a small PR to fix this behavior of test by adding the optional null check as we discussed. Can you please review the PR


Make ant build location agnostic

This change makes the location of the build in ant the actual ${basedir} rather than using a directory name (that doesn't match the actual repo name)


Update field values for every preset

This fixes #4200
cc @pnorman

Thanks @JamesKingdom, I appreciate all your work on adding these 💪


Multi-part videos from Bilibili?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.03.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2019.03.18

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[x] Question
[ ] Other

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.bilibili.com/video/av40918356/?p=2']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.03.18
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: ffmpeg 3.2, ffprobe 3.2
[debug] Proxy map: {}
[BiliBili] 40918356: Downloading webpage
[BiliBili] 40918356: Downloading video info page
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'http://upos-hz-mirrorwcsu.acgvideo.com/upgcxcode/35/23/71872335/71872335-1-80.flv?e=ig8euxZM2rNcNbRVhWNBhwdlhWd1hbUVhoNvNC8BqJIzNbfqXBvEqxTEto8BTrNvN0GvT90W5JZMkX_YN0MvXg8gNEVEuxTEto8i8o859r1qXg8xNEVE5XREto8GuFGv2U7SuxI72X6fTr859r1qXg8gNEVE5XREto8z5JZC2X2gkX5L5F1eTX1jkXlsTXHeux_f2o859IB_&deadline=1553058111&gen=playurl&nbs=1&oi=2928336272&os=wcsu&platform=pc&trid=f450572bc0ff4ebdb89ed2116cea33ff&uipk=5&upsig=0265ee3c249101c78d0f479e52302230'
[download] 【Live】B-Project Summer live2018  Eternal Pacific-40918356.flv has already been downloaded
[download] 100% of 1.42GiB

Description of your issue, suggested solution and other information
I am wondering if it is possible to have support for videos from Bilibili that are in multiple parts.  One example is this: https://www.bilibili.com/video/av40918356/?p=2
When I enter this into youtube-dl, it only downloads the first part of the video, ignoring the /?p=2 on the end (part 1 has /?p=1 and part 3 has /?p=3 etc.).  I've seen some people talk about potential solutions around this website (like here: #12654), however I have no idea how to implement them myself.  Thanks so much if anyone can help explain this to me, I'm still a novice with things like this.

There are multiple partial solutions for this, none of which have been merged with the main branch. I currently use #18827 with 2 lines changed (appkey broke about 1 month ago, my bilibili.py is here: https://pastebin.com/6AQ6BV4s). It downloads paged videos, but not as one would hope. It does not treat it as a playlist, so you must create a list of URLs. This is not such a big deal, if it weren't for the fact that it sometimes ignores the page title ("part2" in your example), making filename not unique resulting in attempts to download same file repeatedly. In these cases it requires babysitting, moving the files elsewhere or renaming before beginning another URL. I can't see under what conditions it will use the page title as part of the filename.
Anyway, you need to modify bilibili.py, as the main branch's has no logic for paged videos. So you can just replace your bilibili.py with one of the ones found on pull request (this one has just been posted a few days ago: #20393 , maybe it is better than what I use). Once you've done that just rebuild youtube-dl, or just build fresh from the commit which contains the desired bilibili.py.

Same as #20555
Please visit: http://www.cr-soft.net/archives/how-to-download-bilibili-video.html

Maybe, this solution can be implemented in Bilibili.py
https://github.com/iawia002/annie#download-playlist
seems to be working just fine.

same issue

Exactly the same issue


Consider renaming --format to --report

With our recent introduction of the --mode option in https://github.com/lycheeverse/lychee/pull/1398, some CLI calls can get confusing:
lychee --no-progress --mode emoji --format detailed https://example.com

The format parameter is simply for the report format, so I propose to rename it to --report to make its usage clear:
lychee --no-progress --mode emoji --report detailed https://example.com

This is a breaking change, but we haven't published a stable release yet, so it's okay according to semver.
It will still cause some disruption for downstream users, but I'd say it's better to change it sooner rather than later.
I'm open for suggestions, though. Maybe it's just a personal issue.

Hmm, not quite sure if this is much clearer. The parameter is used to configure the output format. So why not name it --output-format? Or maybe there are other tools with a comparable option, maybe they have a better name?
  -f, --format <FORMAT>
          Output format of final status report (compact, detailed, json, markdown) [default: compact]

I like it, but technically, it's just about the output format of the final status report, right? 🤔 I wanted to clarify that part and shift the focus to the actual data format (Markdown, JSON) instead.
I'm suggesting --report-format as well.
Alternatively, we can keep --format if it's not too confusing.

Well yes I also like --report-format. But I actually think that format is not too confusing and I like its simplicity. IMO we can keep it.
Also thought about naming it --output but then I realised that this option already exists. And now I'm wondering why that option even exits. I think that this option has become useless since #984 was released. Because redirecting stdout should be equivalent to the --output option. Or am I missing something?

Yeah, probably --format is equally good/bad compared to the other options. In light of that, let's keep it. The discussion was still fruitful, because we can refer to it the next time this question comes up.
--output is redundant, as you said. I'd still like to keep it, because people might not have a clear expectation about what redirecting stdout would entail. We know that we redirect errors to stderr, but not all projects do so, and that might be a source of confusion.
That said, --output is a misleading, too, because it doesn't imply that it's about writing the output to a file. Perhaps --output-file might be less
ambiguous. Then again, --output is shorter and more concise. Naming things is hard.
I propose to close this issue as "won't fix". We can always open a new issue for the --output option if we find that it's causing confusion.

I totally agree :+1:


ARROW-10203: [Doc] Give guidance on big-endian support in the contributors docs

@kiszk @jacques-n @wesm @pitrou @BryanCutler @nealrichardson  this capture my understanding of the mailing list conversation on endianness.  Please let me know if I've mischaracterized anything (I'll do a proof reading/compiling round once as long as the general points are agreed upon).

I'm having trouble getting the source build environment setup on my machine to verify correct layout, I'll try again tomorrow.  Let me know if there are other comments otherwise I think we can merge this and go ahead and start incorporating the other PRs for endianness.

+1, will merge.


Assess the feasability of using NiceGUI to automatically generate simple forms

For simple forms that only require stdlib types, NiceGUI seems to be a good solution for generating UI on the fly. This could potentially be made easier by the partial models that will likely make it to the 2.10 pydantic version (https://github.com/pydantic/pydantic/pull/10748).
This should replace the current experimental feature that relies on user input:
https://github.com/AllenNeuralDynamics/Aind.Behavior.ExperimentLauncher/blob/b3100b10fdafa18b79845b8b5a108ee56419a1c2/src/aind_behavior_experiment_launcher/ui_helper.py#L127

https://github.com/pyapp-kit/magicgui may be promising


Fix column.sorter bug

column.sorter could be 'ascend' | 'descend'  | undefined;
eg: descend -> undefined, request should to invoke.
close #680

Codecov Report

Merging #681 into master will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #681      +/-   ##
==========================================
+ Coverage   17.22%   17.27%   +0.04%
==========================================
  Files          21       21
  Lines        1155     1152       -3
  Branches      378      398      +20
==========================================
  Hits          199      199
+ Misses        956      953       -3

Impacted Files
Coverage Δ

src/Table.tsx
20.20% <0.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

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

https://procomponents.ant.design/
Can you mention pr here？

https://procomponents.ant.design/
Can you mention pr here？

这里已经提了 合过啦


WSL symbolic linked folders are owned by root

Windows Version:
Microsoft Windows [Version 10.0.18362.449]

What you're doing and what's happening:

ln -s /mnt/c/Projects
ls -al ~/Projects/test

What's wrong / what should be happening instead:
Everything is owned by root:root and doing sudo chown -R pmonty:pmonty . does nothing.

I installed WSL and set it up maybe a week ago and done everything exactly the same way (order and everything). But after a fresh install of WSL now I find that this issue has appeared. Don't recall if it wasn't root:root previously but don't see why it should be.
Main reason i discovered this is because i created the symlink then went into a project to run npm install formik and it was complaining about permissions. Also some issues with installing globally too.
EDIT/UPDATE:
If i ls -al /mnt/c/Test then that is owned by pmonty:pmonty but not the link for some reason. Really strange

Not sure what the fix was. I did add some config to a /etc/wsl.conf that didn't exist. Maybe restarted terminal a few times. But whatever it is working now after some re cloning and remaking of links. I have another system I plan to enable WSL on so might see if anything happens on that.

@pmonty
Can I know what you added in the file /etc/wsl.conf?

I ran into this rebuilding my wsl environment and found this issue trying to fix it. In case it helps anyone else, here's what I did:
set up /etc/wsl.conf
open up /etc/wsl.conf/ in an editor with this contents:

[automount]
enabled = true
options = "metadata,umask=22,fmask=11"

restart wsl
in powershell run wsl --shutdown
when you start wsl back up, the permissions should be right

This doesn't do anything in WSL 2. I even changed the permissions on /etc/fstab to use my uid/gid.


[feature] [null support # 10] Add null support in all transform function and pass the bitmap to upstream

Pass the null bitmap up from transform function to transform blockValueSet.
We only do the null related calculation when enableNullHandling is set to true due to performance reason.
Null support issue:
https://github.com/apache/pinot/issues/10252
Design doc:
https://docs.google.com/document/d/1w2eBuZrS73xE2JIw8MYlySy5pNaX9iE8kZr_YAL20CM/edit

Codecov Report

Merging #10613 (f84571b) into master (0cb456a) will decrease coverage by 45.99%.
The diff coverage is 9.79%.

@@              Coverage Diff              @@
##             master   #10613       +/-   ##
=============================================
- Coverage     70.37%   24.38%   -45.99%
+ Complexity     6495       49     -6446
=============================================
  Files          2106     2090       -16
  Lines        113004   113222      +218
  Branches      17026    17181      +155
=============================================
- Hits          79521    27609    -51912
- Misses        27917    82695    +54778
+ Partials       5566     2918     -2648

Flag
Coverage Δ

integration1
24.38% <9.79%> (-0.16%)
:arrow_down:

integration2
?

unittests1
?

unittests2
?

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

Impacted Files
Coverage Δ

.../transform/function/CoalesceTransformFunction.java
0.00% <0.00%> (-87.31%)
:arrow_down:

.../transform/function/DateTimeTransformFunction.java
0.00% <0.00%> (-90.22%)
:arrow_down:

...transform/function/DateTruncTransformFunction.java
0.00% <0.00%> (-86.67%)
:arrow_down:

...r/transform/function/ExtractTransformFunction.java
0.00% <0.00%> (-84.22%)
:arrow_down:

.../transform/function/GreatestTransformFunction.java
0.00% <0.00%> (-92.99%)
:arrow_down:

...erator/transform/function/InTransformFunction.java
0.00% <0.00%> (-56.55%)
:arrow_down:

...transform/function/IsNotNullTransformFunction.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...or/transform/function/IsNullTransformFunction.java
0.00% <0.00%> (-82.76%)
:arrow_down:

...form/function/JsonExtractKeyTransformFunction.java
0.00% <0.00%> (-81.82%)
:arrow_down:

...tor/transform/function/LeastTransformFunction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 15 more

... and 1560 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Seems like another rebase needs to be done to resolve conflicts from previous PR

@61yao will not continue working on this PR; I will patch this PR and finish it.


Format examples and add $schema field

Closing since the schema itself is not happy with the $schema field :)


NGiИX reverse proxy support

I failed to find a simple explanation on how to access the UI remotely through NGiИX. Here is what I tried
server {
    server_name sub.example.tld;

    auth_basic 'PROVE IDENTITY';
    auth_basic_user_file /etc/nginx/sub.example.tld/.htpasswd;

    listen [::]:443 ssl; # managed by Certbot
    listen 443 ssl; # managed by Certbot
    ssl_certificate /etc/letsencrypt/live/sub.example.tld/fullchain.pem; # managed by Certbot
    ssl_certificate_key /etc/letsencrypt/live/sub.example.tld/privkey.pem; # managed by Certbot
    include /etc/letsencrypt/options-ssl-nginx.conf; # managed by Certbot
    ssl_dhparam /etc/letsencrypt/ssl-dhparams.pem; # managed by Certbot

    rewrite ^/$ /webui/ last;

    location / {
        proxy_pass http://localhost:8080;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
    }

    location ~ ^/(webui|api)/ {
        proxy_pass http://localhost:5001;
        proxy_set_header Host $host;
        proxy_set_header X-Real-IP $remote_addr;
        proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
        proxy_set_header X-Forwarded-Proto $scheme;
    }
}

server {
    if ($host = sub.example.tld) {
        return 301 https://$host$request_uri;
    } # managed by Certbot

    listen 80;
    listen [::]:80;
    server_name sub.example.tld;
    return 404; # managed by Certbot
}

TLS and Basic Auth work. I get redirected to https://sub.example.tld/ipfs/<some blurb>/#/welcome which displays the UI skeleton. But then nothing else loads. The only thing that triggers a change in the UI is clicking the Settings tab. In devtools (firefox) I see a bunch of 404 GET to URLs looking like https://sub.example.tld/ipfs/<some blurb>/locales/en-US/app.json and also 403 POST https://sub.example.tld/api/v0/stats/bw and http://<IP_ADDRESS>:5001/api/v0/stats/bw.

Hi @aureooms -- could you please ask this question in the forums at https://discuss.ipfs.io? Thank you!


DRILL-6190 - Fix handling of packets longer than legally allowed.

@parthchandra or @arina-ielchiieva  can you review this?

+1

@tdunning travis fails with  Failed tests:    TestPcapRecordReader.testDistinctQuery:51->runSQLVerifyCount:56->printResultAndVerifyRowCount:68 expected:<1> but was:<2>

Fixed the test regression. Deferring investigation into why the data field doesn't look unique to Drill because we will probably need to revamp how raw data is returned anyway.


Editorial changes for the tutorial

About this change - What it does
Editorial changes to get the tutorial ready for use on the Aiven Developer Center.

Top title is level 1, rest should be level 2, and we don't normally use "Step N" in the subtitles
Various changes in use of english
Since the recent change in exactly how the Aiven console works, one goes directly to a project and then can click on "Create service"
I believe the language in the second paragraph in "Use the application" is probably more correct
Other minor changes

I've changed
the `Service Overview` page

to be
the service's **Overview** page

to be consistent with use of bold for page/tab names elsewhere, and also because the tab is "Overview".

Article now published at https://aiven.io/developer/deploy-netlify-app-aiven-pg-redis


Extended Bluetooth Features

Hey all, so i’ve been testing out extending continuity consistency with bluetooth modifications. Please try to test the Airplay features of the Mac! I’ve recently been experimenting with success getting monterey’s airplay (using matebook as a speaker) and as an apple tv device to cast media to. This requires ZERO input from the hackintosh itself. all controlled thru whatever is casting to it. including the macs volume, video playback, etc

Please try sharing your experience with continuity features such as Apple Watch Unlock, etc.
With a tool called BLEUnlock, progress is wild!

Thank you for your hard work!
For me continuity, handoff, airplay are working!
Universal control, sidecar, airdrop, Apple Watch unlock not working.
It would be super cool if we could have universal control working at some point :)

Thank you for your hard work!
For me continuity, handoff, airplay are working!
Universal control, sidecar, airdrop, Apple Watch unlock not working.
It would be super cool if we could have universal control working at some point :)

Really, I know the airdrop does not work proplely but sidecar / universal control seemed to work under my testing. can you share your experience. @francescodesu I worked on it a while ago! I agree, needed feature!

BLEUnlock works flowlessly!
sidecar gives the error "the device timed out"... Any idea on how to solve it?

So first thing to check is whether your SMBIOS is changed (ROM, MLB, etc).
I would then reccomend signing in and out of icloud on all devices and
restarting. Also disable home view on the Ipad from what I read. We can
also explore different SMBIOS models to change and attempt (Macbook Pro
13,4 13,1 etc)
On Sat, Apr 16, 2022 at 06:46 francescodesu @.***>
wrote:

BLEUnlock works flowlessly!
sidecar gives the error "the device timed out"... Any idea on how to solve
it?
—
Reply to this email directly, view it on GitHub
https://github.com/tlefko/Matebook-X-Pro-2018-Monterey/issues/8#issuecomment-1100635569,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKHES3CXLFAM5ZIVAC3MILTVFKK6ZANCNFSM5TMAOW5A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks, it seems that sidecar works with the thunderbolt cable but not through Bluetooth.
I tried restarting the device, re-logging to iCloud, and I have tried Macbook Pro 14,2 and 15,2 SMBIOS but it does not seem to work... Which SMBIOS are you using?

i would try 13,1 or 13,3. Make sure the extendbtfeatureflag is enabled in
the config
On Sat, Apr 16, 2022 at 23:28 francescodesu @.***>
wrote:

Thanks, it seems that sidecar works with the thunderbolt cable but not
through Bluetooth.
I tried restarting the device, re-logging to iCloud, and I have tried
Macbook Pro 14,2 and 15,2 SMBIOS but it does not seem to work... Which
SMBIOS are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/tlefko/Matebook-X-Pro-2018-Monterey/issues/8#issuecomment-1100795415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKHES3DCOS7TUHW65ZC5AXDVFOANNANCNFSM5TMAOW5A
.
You are receiving this because you authored the thread.Message ID:
@.***>


Path escape on MS-Windows

Should add shellescape() to wrap the s:vtm_python_host to prevent path error on MS-Windows(the space in g:python3_host_prog, i.e. 'C:\Program Files\Python36\python.exe)
https://github.com/voldikss/vim-translate-me/blob/e32b2a7ddb9de56dd415f5b7f449d86efe939e02/autoload/vtm.vim#L10
But this seems to cause job error on VIM8(e32b2a7). So there must be a way to solve both problems...

4db614b did a tmp fix by switching the order to choose the value of s:vtm_python_host.


Doesn't sync files in /search folder

Search site in 365 is /search (not under a managed path).
"remoteFolders": ["/_catalogs/masterpage/display templates/**/*.html"]
All sub folders under /display templates/ are populated except files from the /display templates/Search folder are copied to the parent folder "display templates"

@imorrish - I've just posted SPGo 1.3.5 to the VSCode Marketplace which will resolve this issue. Please reopen this issue if you are still experiencing issues.


On data event no being emitted on the DataConnection object

Hi there,
Following along from the referenced peerJS documentation link it seems that you should be able to listen for an on data event on the DataConnection object given back to you when an on connection event is triggered. I don't seem to be able to get that to work though. When sending the message I seem to only get on data event when listening on the peer object.
I need it on the DataConnection so I can send a reply to the correct remote connection. Is this an issue or am I doing this incorrectly?
Thanks in advance!
  var peer = Peer();

  peer.once("open").asStream().listen((event) {
    connected = true;
    print("Connection established");
  });

  peer.on<DataConnection>("connection").listen((connection) {
    //Connection of remote peer
    connection.on("open").listen((event) {});

    connection.on("data").listen((event) {
      //This never runs
      print("inner loop");
    });

    // On peer closed.
    connection.on("close").listen((event) {});
  });

  peer.on("data").listen((data) {
    //This always runs
    print("Outer reader worked");
  });
}

Are you using the beta version?

Was using the 4.0 beta, will try 5.0 now, thanks!

That fixed it thanks!


Tree shaking and ESM Support not useful

I've been trying to get tree shaking to work with heroicons in our app using webpack.
When importing I would expect unused icons to be tree shaken, but in order to do so the esm version has to be explicitly imported.
import { MenuIcon } from '@heroicons/react/solid'; // no tree shaking
import { MenuIcon } from '@heroicons/react/solid/esm'; // tree shaking works

This is not expected as other libraries manage to make this work. This is a real issue in that the runtime/bundler can't decide for itself which loader to use. In our app we have a browser bundle, a SSR bundle and we run our code via jest, all these environments need to dynamically choose which loader to use and they can't do that if we have to hard code one in the code.
After some research I believe the package.json needs a main and module key for the bundler to be able to select which build to use https://github.com/rollup/rollup/wiki/pkg.module
I don't think it's that easy though as you can't specify multiple entrypoints and heroicons has 2, solid/esm/index.js and outline/esm/index.js
In the meantime we have to import specific icons to avoid loading the entire library. This isn't ideal as it's hard to enforce and IDEs will often "help" by collapsing them.
import MenuIcon from '@heroicons/react/solid/MenuIcon';

Also experimenting this, here's my ANALYZED bundle from Next.js (https://www.npmjs.com/package/@next/bundle-analyzer).

On second thought, while the Next bundle analyzer shows heroicons is using a lot of space. When running next build there's no difference between the two versions (with or without full import path).
🤷♀

Any thoughts on this @adamwathan?

Unfortunately just not a priority right now, happy to accept a PR otherwise will get to it when we get to it. Sorry 😞

I'm glad I found this issue! I have a blog based on https://github.com/timlrx/tailwind-nextjs-starter-blog - turns out the hero icons were not getting tree shaken correctly in MDX bundling, and I was at 500 KB data size - adding /esm to all imports of hero icons in *.mdx files reduced that to ~50KB, factor of 10 decrease.

This worked for me!
import { MenuIcon } from '@heroicons/react/solid/esm'; // tree shaking didn't work
import QuestionMarkCircleIcon from "@heroicons/react/solid/esm/QuestionMarkCircleIcon"; // tree shaking works

is this related package.json file in esm folders?
the contents of the package.json files are like this
{
  "type": "module",
  "sideEffects": false
}

but for tree-shaking to work, I guess, there should be an entry called "module": "<path-to-entry-file>"
if this is the problem, I would like to contribute 😊.

Hey @rhysemmerson. Sorry that you aren't seeing tree-shaking in your project. I'm going to close this issue because tree-shaking is working correctly with webpack in my tests, and without a reproduction it will be impossible to debug. You can see an example project with tree-shaking working fine here: https://github.com/bradlc/heroicons-webpack
If you're still having issues please open a new issue with a reproduction repository, where you already have everything all set up so we can clone it and immediately see the issue without configuring everything by hand.


Feature: Create an ISSUE_TEMPLATE.md

This way you are provided with the information you need for issues :)

@lindydonna @christopheranderson

Will do, we can use the same format as the other repos. Thanks for the suggestion @johnpapa!

feel free to look at this one i made ... its pretty basic, but could be a starter. https://github.com/johnpapa/vscode-azure-functions-tools/blob/master/ISSUE_TEMPLATE.md


Improve log when Kafka is not up

When kafka is not up, we got
TypeError: 'NoneType' object is not iterable

Can it be more accurate?

You're right, I will look into this very soon.


KAFKA-12995 [WIP] Allow old Broker compatibility for Metadata calls

Old Brokers prior to Metadata V4 don't support the allowAutoTopicCreation field.
The implementation of #3098 introduced a check for old Brokers compatibility falling back to true in MetadataRequest's allowAutoTopicCreation field for some operations like describeTopics.
Then, after that backward-compatibility changes, more methods were added to AdminClient like listOffsets in #7296 which unfortunately don't have compatibility with old Brokers into account. One might argue that, given that new methods were not present previously, it's not a compatibility break. While this is strictly true, I believe that one of Kafka Clients' most appreciated feature is the ability to use old Brokers with new Clients.
I've prepared this small PR to fix for this, following the same approach used in #3098.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the PR. I am tempted to specify list offsets to support 0.11.0 brokers and newer. 0.10.x is extremely old now. It was different when we implemented #3098.

Thanks for the PR. I am tempted to specify list offsets methods to support 0.11.0 brokers and newer. 0.10.x is extremely old now. It was different when we implemented #3098 4+ years ago.

Well, from my POV I'd say that listOffsets looks like a feature supported in 0.10.x, it's just the metadata headers sent from AdminClient that prevent it from working (if I'm not wrong).
I understand that 0.10.x is an old version, but it's the one that our company uses in Production 🤷🏽 I don't know how many companies are out there like mine, but I'd bet they will be happy to know that if they want to use listOffsets for whatever new thing they develop, they are not forced to upgrade the Kafka Cluster version. We all know that nowadays Kafka is a key component in most companies architecture, and performing a major upgrade is a big thing that needs a huge amount of work and synchronization between teams (at least compared with a client update).
That being said, whatever decision you make, will be fine (at least I've tried 🙂 ).

From what I can gather very few companies still use 0.10.x. It has serious bugs that can result in data loss. KIP-101 (introduced in 0.11.0) is a must have if you care about your data. Subsequent versions include other very important fixes.
Kafka cluster upgrades are generally easier than exhaustive client upgrades as clusters are typically managed by fewer teams.


0.3.4 SyntaxError Unexpected token U in JSON at position 45 at JSON.parse

Version 0.3.4 - I can't get a debugging session to work. I have ports forwarded from my debug server (port 9001) at Digital Ocean to local port 9002. This has worked flawlessly in the past, but after upgrading to 0.3.x it hasn't. Attached is a screenshot of the console when a debugging session is requested. All software versions are the latest: PHP 7.2.5, xdebug 2.6.0
The session seems to connect, but hangs, seemingly because of this error.

For what it's worth, this appears to be a Windows issue. The same setup works on macOS High Sierra.

Can you clear your php-debug config from config.cson and try again?

Indeed, happy to. Tried again, and the position changed to 42 from 45. That doesn't appear to be helpful. I've updated the screenshot for you.

The error you're receiving is because the package is failing to parse your pathmaps. Can you provide your config here?

"php-debug":
showWelcome: false
xdebug:
pathMaps: "[{"remotePath":"/var/www","localPath":"C:\Users\msita\OneDrive\GitHub"}]"
welcome:
showOnStartup: false

Hmm, I'm not sure how it would have been saved like that but it should look like
"[{\"remotePath\":\"/var/www\",\"localPath\":\"C:\\Users\\msita\\OneDrive\\GitHub\"}]"

or
"[{\"remotePath\":\"/var/www\",\"localPath\":\"C:/Users/msita/OneDrive/GitHub\"}]"

I noticed after you posted that the escaped back slashes were indeed escaped (they were there in the config file). When I pasted in the first option of yours, nothing changed (they were escaped). The second option didn't work, but I was prompted for a path setup for a file that runs in the apache config:
php_value auto_prepend_file "/var/www/bootstrap.php"
I told php_debug not to use that file/path, and it was added to the path config as expected:
"[{"remotePath":"/var/www/ig","localPath":"C:/Users/msita/OneDrive/GitHub/ig"},{"remotePath":"/var/www/bootstrap.php","localPath":"!"}]"
Now the sessions are being rejected all together. Can I just ignore it, not reject it?

Solved. The issue was that I was only using single backslashes in the php_debug settings interface (resulting in double backslashes in the config.cson file). Once I added the second backslash in the interface (resulting in \\ in the config.cson) it worked!


Unable to make ed25519 while installing homebridge

I tried different versions auf gcc,g++ and nodeJS.
admin@smarthome:~/homebridge$ uname -a
Linux smarthome.local 4.1.15-v7+ #830 SMP Tue Dec 15 17:02:45 GMT 2015 armv7l GNU/Linux
admin@smarthome:~/homebridge$ gcc --version gcc (Raspbian 4.8.4-1) 4.8.4 Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
Package.json
{
"name": "homebridge",
"description": "HomeKit support for the impatient$"version": "0.2.15",
"scripts": {
"dev": "DEBUG=* ./bin/homebridge -D -P example-$},
"author": {
"name": "Nick Farina"
},
"repository": {
"type": "git",
"url": "git://github.com/nfarina/homebridge.git"},
"bugs": {
"url": "http://github.com/nfarina/homebridge/is$},
"licenses": [
{
"type": "ISC",
"url": "http://github.com/nfarina/homebridge/$}
],
"bin": {
"homebridge": "bin/homebridge"
},
"engines": {
"node": ">=0.12.0"
},
"preferGlobal": true,
"dependencies": {
"chalk": "^1.1.1",
"commander": "2.8.1",
"hap-nodejs": "0.0.7",
"semver": "5.0.3"
}
}
make: Entering directory /home/admin/homebridge/node_modules/hap-nodejs/node_modules/ed25519/build'   CC(target) Release/obj.target/ed25519/src/ed25519/keypair.o make: cc: Command not found make: *** [Release/obj.target/ed25519/src/ed25519/keypair.o] Error 127 make: Leaving directory /home/admin/homebridge/node_modules/hap-nodejs/node_modules/ed25519/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:270:23)
gyp ERR! stack     at emitTwo (events.js:87:13)
gyp ERR! stack     at ChildProcess.emit (events.js:172:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:200:12)
gyp ERR! System Linux 4.1.15-v7+
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/admin/homebridge/node_modules/hap-nodejs/node_modules/ed25519
gyp ERR! node -v v4.1.1
gyp ERR! node-gyp -v v3.0.3
gyp ERR! not ok
npm ERR! Linux 4.1.15-v7+
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! node v4.1.1
npm ERR! npm  v2.14.4
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild npm ERR! Exit status 1 npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the ed25519 package, npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls ed25519
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/admin/homebridge/npm-debug.log

How are you installing the compile?
Have you read the wiki on Raspberry https://github.com/nfarina/homebridge/wiki/Running-HomeBridge-on-a-Raspberry-Pi#install-c14-raspbian-wheezy-only?
Which OS have you got on your Raspberry? Wheezy or Jessie?
cat /etc/*-release
Nothing will work till cc is found.

Please install build-essential

How are you installing the compile?

I tried "sudo npm install -g homebridge" first
Later I tried to clone, edit package.json und "nmp install"

Have you read the wiki on Raspberry https://github.com/nfarina/homebridge/wiki/Running-HomeBridge-on-a-Raspberry-Pi#install-c14-raspbian-wheezy-only?
Sure, strictly followed all instructions

Which OS have you got on your Raspberry? Wheezy or Jessie?

Wheezy, but followed instructions under "https://solarianprogrammer.com/2015/01/13/raspberry-pi-raspbian-install-gcc-compile-cpp-14-programs/" to install 4.9 compiler. 4.9 did not work, therefore I aslo tried 4.8
admin@smarthome:/usr/smarthome$ cat /etc/*-release
PRETTY_NAME="Raspbian GNU/Linux 7 (wheezy)"
NAME="Raspbian GNU/Linux"
VERSION_ID="7"
VERSION="7 (wheezy)"
ID=raspbian
ID_LIKE=debian
ANSI_COLOR="1;31"
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"

Please install build-essential

Tried it, did not help.

What does which cc return?
What versions of node -v and npm -v?

What does which cc return?

/usr/bin/cc

What versions of node -v and npm -v?

admin@smarthome:/usr/smarthome$ node -v
v4.1.1
admin@smarthome:/usr/smarthome$ npm -v
2.14.4
I also tried node 4.0.0 and latest Version

You've tried sudo too I assume? sudo npm install -g ...

Sure.

Then it's beyond my pay grade sorry @velocer .
Looks like you've installed into /home/admin - maybe a priivilidge issue?
if cd cd /home/admin/homebridge/node_modules/hap-nodejs/node_modules/ed25519/build and then make what happens

Directory doesn't exist. Maybe really an access problem. Found those warnings:
gyp WARN EACCES user "root" does not have permission to access the dev dir "/root/.node-gyp/4.1.1"
gyp WARN EACCES attempting to reinstall using temporary dev dir "/usr/local/lib/node_modules/homebridge/node_modules/hap-nodejs/node_modules/curve25519/.node-gyp"

I read about the EACCESS warning some time ago in https://github.com/nodejs/node-gyp/issues/454#issuecomment-134140378
It's related to installing globally and using root as user.

Thanks Raoul. Installing homebridge with "--unsafe-perm" while using gcc/g++ 4.8 finally did it.


Add documentation on Camera

#18 should be done first

Done in commit https://github.com/Sygmei/ObEngine/commit/1a897b5826a4d397a8a3dd6a411bf51e8c017906


Fix metrics NaN issue

Not the prettiest, but given the way our metrics serialization is working, it is a simple solution without extra allocation.
It may send empty metrics sets if ALL have NaN values but I decided it doesn't make sense to compromise performance or code simplicity for this unexpected extreme (should't cause an error, just send empty data event)

Codecov Report

Merging #501 into master will decrease coverage by 1.31%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #501      +/-   ##
============================================
- Coverage     65.66%   64.35%   -1.32%
  Complexity       68       68
============================================
  Files           170      166       -4
  Lines          7036     6292     -744
  Branches        889      730     -159
============================================
- Hits           4620     4049     -571
+ Misses         2152     1991     -161
+ Partials        264      252      -12

Impacted Files
Coverage Δ
Complexity Δ

...ent/report/serialize/MetricRegistrySerializer.java
86% <100%> (+4.6%)
0 <0> (ø)
:arrow_down:

.../elastic/apm/agent/bci/bytebuddy/MatcherTimer.java
62.5% <0%> (-31.25%)
0% <0%> (ø)

...c/apm/agent/bci/MatcherTimerLifecycleListener.java
75% <0%> (-18.75%)
0% <0%> (ø)

...lastic/apm/agent/impl/context/AbstractContext.java
75% <0%> (-8.34%)
0% <0%> (ø)

...ic/apm/agent/servlet/ServletTransactionHelper.java
71.75% <0%> (-7.25%)
0% <0%> (ø)

.../co/elastic/apm/agent/impl/error/ErrorCapture.java
77.61% <0%> (-5.41%)
0% <0%> (ø)

.../apm/agent/report/serialize/DslJsonSerializer.java
82.41% <0%> (-3.87%)
0% <0%> (ø)

...ent/jaxrs/JaxRsTransactionNameInstrumentation.java
79.16% <0%> (-3.45%)
0% <0%> (ø)

...java/co/elastic/apm/agent/bci/ElasticApmAgent.java
73.58% <0%> (-1.89%)
0% <0%> (ø)

...va/co/elastic/apm/agent/impl/ElasticApmTracer.java
78.4% <0%> (-0.33%)
0% <0%> (ø)

... and 13 more

Continue to review full report at Codecov.

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

@kuisathaverat is there a way to trigger a build in the CI through GH comment?

jenkins run the tests

Is this fix implemented? I used APM agent in version 1.18.0 and I still got this error
`{ "message": "failed to validate metricset: error validating JSON: I[#] S[#] doesn't validate with "metricset#"\n  I[#] S[#/allOf/5] allOf failed\n    I[#/samples/example.met.processing/value] S[#/allOf/5/properties/samples/patternProperties/%5E%5B%5E%2A%22%5D%2A$/properties/value/type] expected number, but got string",
"document": "{"metricset":{"timestamp":1601377271199000,"tags":{"command":"ExamleCommand","status":"*"},"samples":{"example.met.processing":{"value":"NaN"}}}}"}`

@mierzej91 this was merged long ago.
Can https://github.com/elastic/apm-agent-java/pull/1419 be related to what you are seeing? If so, please try to download and test the related bugfix and see if the problem is resolved.
cc @SylvainJuge


multiple match filters

I have log lines like this
May  3 00:41:07 <IP_ADDRESS> /var/log/pald.log[821] MALONE UPS 1: Output Current = 5.00
May  3 00:41:09 <IP_ADDRESS> /var/log/pald.log[821] MALONE UPS 1: Output Current = 4.00
May  3 00:41:17 <IP_ADDRESS> /var/log/pald.log[839] MALONE UPS 2: Battery Minutes Remaining = 32
May  3 00:41:24 <IP_ADDRESS> /var/log/pald.log[821] MALONE UPS 1: Input Voltage = 120.1
May  3 00:41:25 <IP_ADDRESS> /var/log/pald.log[839] MALONE UPS 2: Input Voltage = 119.0
May  3 00:41:27 <IP_ADDRESS> /var/log/pald.log[821] MALONE UPS 1: Battery Voltage = 54.3
May  3 00:41:36 <IP_ADDRESS> /var/log/pald.log[839] MALONE UPS 2: Battery Minutes Remaining = 33

so I matched them up with this filter
match: '%{MONTH:month}%{SPACE}%{MONTHDAY:day}%{SPACE}%{TIME:time}%{SPACE}%{USER}%{SPACE}%{PATH}%{NOTSPACE}%{NUMBER}%{NOTSPACE}%{SPACE}%{NOTSPACE:location}%{SPACE}%{NOTSPACE:asset}%{SPACE}%{INT:id}%{NOTSPACE}%{SPACE}(?<measurement>%{WORD}%{SPACE}%{WORD})%{SPACE}%{NOTSPACE}%{SPACE}%{NUMBER:val}'

This works great, except it fails to catch the message that has a third word in the value label.  So I made another one that catches it:
match: '%{MONTH:month}%{SPACE}%{MONTHDAY:day}%{SPACE}%{TIME:time}%{SPACE}%{USER}%{SPACE}%{PATH}%{NOTSPACE}%{NUMBER}%{NOTSPACE}%{SPACE}%{NOTSPACE:location}%{SPACE}%{NOTSPACE:asset}%{SPACE}%{INT:id}%{NOTSPACE}%{SPACE}(?<measurement>%{WORD}%{SPACE}%{WORD}%{SPACE}%{WORD})%{SPACE}%{NOTSPACE}%{SPACE}%{NUMBER:val}'

but I couldn't figure out how to run them both at the same time.
So, I just doubled up on the docker containers instead, and that way I end up with my exact desired result:

But there must be a way to get both of those into the same line?

You are pretty close. A pattern to match an arbitrary number of words looks like this:
(%{WORD}%{SPACE}?)+

That means: A word, followed by an optional space, and the whole thing might repeat multiple times.
Instead of copying this directly into your match, you could define it as a pattern named WORDS and use it in your match like this:
grok_patterns:
- 'WORDS (%{WORD}%{SPACE}?)+'
metrics:
- type: gauge
  name: gauge_1
  help: example gauge 1
  match: '%{MONTH:month}%{SPACE}%{MONTHDAY:day}%{SPACE}%{TIME:time}%{SPACE}%{USER}%{SPACE}%{PATH}%{NOTSPACE}%{NUMBER}%{NOTSPACE}%{SPACE}%{NOTSPACE:location}%{SPACE}%{NOTSPACE:asset}%{SPACE}%{INT:id}%{NOTSPACE}%{SPACE}%{WORDS:measurement}%{SPACE}%{NOTSPACE}%{SPACE}%{NUMBER:val}'
  labels:
    measurement: '{{.measurement}}'
  value: '{{.val}}'

However, your match is a bit long, and you might consider simplifying it. This depends on your loglines and it's hard to tell without knowing what else is in your log, but to give you an example: If you know the measurement starts after a :, and the separater between the measurement and the value is a =, and you don't care about the rest, you could write the match like this:
- type: gauge
  name: gauge_2
  help: example guage 2
  match: ': %{WORDS:measurement} = %{NUMBER:val}'
  labels:
    measurement: '{{.measurement}}'
  value: '{{.val}}'

Ah that's a great clue, thanks for taking the time to answer.  Is there a resource that covers the use of the ? And the + in these filters?
In your example, you start with a colon, does it just skip everything until it finds one?
If one Grok filter is more elegant than the other but gives the same result, is there an efficiency difference?
I didn't know how to "get rid" of the number contained in square brackets. I'm not particularly interested in it, so I just did %{NOTSPACE}%{NUMBER}%{NOTSPACE}.  I bet there's an easier way...

The matches are just regular expressions, so if you want to learn more about ? and + you can read any resource about regular expressions.

? means zero or one occurrence
+ means one or more occurrences

As regular expressions tend to be hard to read, we introduced Grok patterns like %{WORD}, which are placeholders for regular expression snippets. For example, WORD is the same as \b\w+\b as defined here, and %{SPACE} is the same as \s* as defined here. As we defined %{WORDS} as (%{WORD}%{SPACE}?)+ it is the same as the regular expression ((?:\b\w+\b)(?:\s*)?)+. As you can see, the regular expression is much harder to read, so it makes sense to define little snippets like %{WORD} or %{SPACE} to improve readability.

you start with a colon, does it just skip everything until it finds one?

yes

is there an efficiency difference?

Yes, you can easily define regular expressions that take very long to evaluate, but the length of the regular expression is not an indicator for that. You can have long regular expressions that are efficient, and you can have short ones that are inefficient. If you experience performance problems, check grok_exporter's built-in metric grok_exporter_lines_processing_time_microseconds_total to learn if this is caused by an inefficient regular expression, as described here.

how to "get rid" of the number contained in square brackets

The easiest way to just "get rid" of something is to use .*, which matches an arbitrary long sequence of arbitrary characters. If you specifically want to match a [ or ], you need to escape it as \[ or \] because [ and ] are special characters in regular expressions. So you could write \[%{NUMBER}\].

Closing this one. Feel free to open a new issue if there are more questions.


Chinese Simplified translation

Hello!
The first version of Chinese translation has been completed. Please merge the translation in order to let me do some further language localization.
BTW: Great app!

Thanks! I'll merge it in soon, but I'm also working on an update so I'll be adding more phrases on OneSky soon.

Updated OneSky with new strings.

Just published a new alpha version (v3.0.0-alpha.6) with the translations included.
If you're not already in the alpha/beta testing program, you can join here.


Forced change from normal mode to insert mode after sending code to repl

Is it possible to maintain mode after sending text to repl? For example after sending code from normal mode to repl I am moved automatically to insert mode.

Hi,
In theory it should be possible, but I will have to double-check since the current way of handling all this is kind of messy... I'm currently trying to figure out a way of simplifying all the logic so hopefully it will be fixed soon.

I've merged v3.0 now and I believe it might've solved some of the issues. Unfortunately, I'm "forced" to break away from visual mode so I can properly extract the values, but I think normal mode is now being kept. Please let me know if the issue is fixed in this new version.

I've merged v3.0 now and I believe it might've solved some of the issues. Unfortunately, I'm "forced" to break away from visual mode so I can properly extract the values, but I think normal mode is now being kept. Please let me know if the issue is fixed in this new version.

For me, whenever I start or restart the REPL I end up in insert mode.

For me, whenever I start or restart the REPL I end up in insert mode.

I searched for this issue because I had this problem, but it turned out I just had an autocommand on TermOpen that switched to insert mode. That's nice when you open the terminal yourself, but since Iron creates a terminal without switching to it, I ended up in insert mode in the code window instead. Maybe that's your situation too? :)


Mobile Wrapper Padding Override?

I'm trying to reduce the wrapper padding on mobile. At the moment I have it set to 34px left and right, but on mobile i would like this to be 10px.
So I set up a custom css class and added that like so:
<mj-wrapper background-color="#fff" padding="0px 34px" css-class="mobile-gutters"/>
And then in my Mj-style I have this:
@media screen and (max-width: 599px) {
      .mobile-gutters > td {
        padding-left: 14px !important;
        padding-right: 14px !important;
      } ...

But it does nothing. I know that the whole wrapper is a DIV so the class is applied to that. The only way I could get it to see this was too loose the > like this .mobile-gutters td { but then it did it to EVERY TD in that  section!
Be great to be able to override this via an attribute.

Actually I had my CSS selectors wrong.
updated to:
@media screen and (max-width: 599px) {
      .mobile-gutters > table > tbody > tr > td {
        padding-left: 14px !important;
        padding-right: 14px !important;
      } ...

And it works.


Add support to release linux aarch64 wheels

Problem
On aarch64, pip install nnabla is giving the below the error-
ERROR: Could not find a version that satisfies the requirement nnabla (from versions: none)

ERROR: No matching distribution found for nnabla

Resolution
On aarch64, pip install nnabla should download the wheels from PyPI.
@TomonobuTsujikawa and Team Please let me know your interest in releasing aarch64 wheels. To start with can you please suggest the steps/CI you are using for building the wheel.

Thank you for trying it!
Yes, as you pointed out, aarch64 is not released now, so anyone cannot install by pip.
But, I think you may be able to build from our source code.
Here is the build instructions from code:

For nnabla (cpu only): https://github.com/sony/nnabla/blob/master/doc/build/build.md
For nnabla-ext-cuda (+cuda): https://github.com/sony/nnabla-ext-cuda/blob/master/doc/build/build.md

Of course, you may need to install additional packages and/or change codes.

I have tried to build the code from the source using the steps mentioned here and the wheel is created successfully for aarch64 but it is taking a lot of time rather than downloading from PyPI and unpacking it. Also, installation from the source code requires additional dependencies to be installed.
It will be a great help if you can release nnabla Linux aarch64 wheels on PyPI.

Thank you! It's good news!
We need environment preparation and have to pass tests to support aarch64 officially.
But please let us investigate if we can support it.

We took some time, but we have released aarch64 wheel package in v1.27.0.
https://pypi.org/project/nnabla/#files
https://github.com/sony/nnabla/releases/tag/v1.27.0
I think the installation environment is limited because glibc2.31 is required, so we will improve it in future release.

@ TomonobuTsujikawa I have tried to install the wheel for aarch64 but it is giving the below issue :
# pip3 install nnabla
ERROR: Could not find a version that satisfies the requirement nnabla (from versions: none)
ERROR: No matching distribution found for nnabla

Python version output -
# python3 --version
Python 3.9.13

# pip3 --version
pip 20.0.2 from /usr/lib/python3/dist-packages/pip (python 3.9)

It looks that the issue is in the name of the wheel created and uploaded on PyPI. I have tried to run auditwheel but getting the below issue:
auditwheel repair nnabla-1.28.0-cp39-cp39-manylinux_2_31_aarch64.whl
INFO:auditwheel.main_repair:Repairing nnabla-1.28.0-cp39-cp39-manylinux_2_31_aarch64.whl
usage: auditwheel [-h] [-V] [-v] command ...
auditwheel: error: cannot repair "nnabla-1.28.0-cp39-cp39-manylinux_2_31_aarch64.whl" to "manylinux_2_17_aarch64" ABI because of the presence of too-recent versioned symbols. You'll need to compile the wheel on an older toolchain.

Can you please look into the issue and help me here?

Thank @odidev for detail description, it's very helpful for us!
I think you need to update pip, because pip supports PEP600 (manylinux_x_yy) from v20.3.
So, please try to update pip by: pip3 install -U pip.

@TomonobuTsujikawa Thanks for the help. I have updated the pip version and now the wheel is getting installed but when I am importing it. It is giving the below error:
>>> import nnabla
2022-06-02 17:09:36,449 [nnabla][INFO]: Initializing CPU extension...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/local/lib/python3.9/dist-packages/nnabla/__init__.py", line 34, in <module>
    from .parameter import (
  File "/usr/local/lib/python3.9/dist-packages/nnabla/parameter.py", line 29, in <module>
    import nnabla.utils.nnabla_pb2 as nnabla_pb2
  File "/usr/local/lib/python3.9/dist-packages/nnabla/utils/nnabla_pb2.py", line 36, in <module>
    _descriptor.FieldDescriptor(
  File "/usr/local/lib/python3.9/dist-packages/google/protobuf/descriptor.py", line 560, in __new__
    _message.Message._CheckCalledFromGeneratedFile()
TypeError: Descriptors cannot not be created directly.
If this call came from a _pb2.py file, your generated code is out of date and must be regenerated with protoc >= 3.19.0.
If you cannot immediately regenerate your protos, some other possible workarounds are:
1. Downgrade the protobuf package to 3.20.x or lower.
2. Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower).
More information: https://developers.google.com/protocol-buffers/docs/news/2022-05-06#python-updates

If I downgrade the protobuf version to 3.20.1 then import is working fine. I have tried the second solution also Set PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python (but this will use pure-Python parsing and will be much slower) but the error is same. Can you suggest how we can import it without doing anything other than installing nnabla from PyPI?

@odidev
This is caused by upgrading protobuf.
I think this PR is not permanent solution, but this will be used for next release: https://github.com/sony/nnabla/pull/1075
Please install protobuf by pip install "protobuf<=3.19.4" at this time.


Fix a few keyvalue test cases

Relates to https://github.com/nosqlbench/nosqlbench/pull/349
While https://github.com/nosqlbench/nosqlbench/pull/349 fixes the docs-api-keyvalue test, this does a similar set of fixes for http-rest-keyvalue, http-graphql-cql-keyvalue, and http-graphql-keyvalue.
The tabular/timeseries tests have their own special way to match keys with the cycle number, and upon review it looks like they are behaving properly.
Also note that the graphql-related tests do NOT accept 4XX as a valid response, this is because graphql is not ever meant to return 4XX for valid requests.

Yep let's merge this one and the #349

@jshook @EricBorczuk I just noticed that the changes in the PR missed to update the driver type in the schema phase for the default scenario to http, as we are having HTTP calls and not the CQL ones. Eric, can you please revisit and fix this in separate PR and can you actually make sure that the default scenario is running fine. I guess that the current one would fail due to this error, maybe we missed something else in addition:
nb ./http-rest-keyvalue.yaml auth_token=.. rampup-cycles=.. main-cycles=..


Avoid other players

bot should not run into other players

anyone has an idea how we could do this?
maybe a new moveto(x,y) function called safeMoveToXY which always avoids other players
so maybe something like this:
*check if someone is between me and the target
*check if it has more mass than me
*if both is true make a new path 1° different - recalculate
@Apostolique how did you solved this ?

@RealDebugMonkey bot is doing much better than others.
Could you save our day?

Depends on how you prioritize what your bot's goals. In the old version of the bot I made it so it would calculate a desirability value for each food pellet we could see. It would then calculate path to each pellet and see if there was someone in the way, if there was it would eliminate it as a candidate.
Of course, that led to issues when there was no food on the screen. It would also stupidly go towards a pellet that was near a player as long as the player was not directly in the path of the food. It's a hard problem to solve.

hmm clever approach, I like it!

added in b057c9856394c066a006f8e26b5aa9e73920292f
currently avoids viruses if smart move is active


Update pyth-sdk-solana to 0.7.0 for compatibility with Anchor 0.26.0

Seahorse actually uses whichever version of Anchor you have installed/active, so no need to explicitly bump it
But there was a dependency clash between pyth-sdk-solana 0.6.1 and Anchor 0.26.0 (on solana-program-library) causing a compile error on seahorse build
Bumping to 0.7.0 works with both Anchor 0.25.0 and Anchor 0.26.0
Also tested with all examples on Anchor 0.26.0

Closes #80

nice!


Add ast tests from CPython 3.8.7

Some say this run is still running to this day.

Yeah, looks like there's a lot of stuff in compiler that panics instead of returning an error on ill-formed input, since until recently the only input it would get was from the parser.


Add scoring logic inspired by npms.io work

CC @isoos

@isoos PTAL

This got mixed commits with the "Use dart files to store the expected output". I'm actually fine with that too, so merging...

In the future, let me do the merge after you approve. I like clean history.
:-)
On Thu, Jun 1, 2017 at 9:42 AM, István Soós<EMAIL_ADDRESS>wrote:

Merged #6 https://github.com/kevmoo/pana.dart/pull/6.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kevmoo/pana.dart/pull/6#event-1106410016, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABCihEG1HOTqPXHcpiYcXHIPr3Qdx5xks5r_uncgaJpZM4NsAts
.


how can i close/dispose of a db connection?`

it would be great to see database helper style example that shows how to close the connection.
is there any way to detect when the instance is destroyed?

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

ID: cb8d85a7-1ca3-71f4-85fb-7fb02850b60e
Version Independent ID: 3d7daee4-fd6d-0e96-299a-90b3637e6150
Content: Manage connections in Azure Functions
Content Source: articles/azure-functions/manage-connections.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@MattFinlay
Thanks for your feedback! We will investigate and update as appropriate.

@FinVamp1 do you have any advice for this connection-related question?

It will depend on the SQL Server\Database type and the library that is used. As the article talks about using SQLClient for .NET applications that uses a connection pool and will manage the assignment and removal of connections from the pool. https://learn.microsoft.com/en-us/dotnet/framework/data/adonet/sql-server-connection-pooling#removing-connections .
"We strongly recommend that you always close the connection when you are finished using it so that the connection will be returned to the pool. You can do this using either the Close or Dispose methods of the Connection object, or by opening all connections inside a using statement in C#, or a Using statement in Visual Basic. Connections that are not explicitly closed might not be added or returned to the pool."
For other SQL databases and other libraries there may be other guidance.  It may not be possible to keep this article updated with specific guidance for each type of SQL database and associated library. The majority of libraries should use connection pooling and async threading models to help with this.


remove logger setup from modules

Reference Issue
Fixes #144
What does this pull request implement/fix? Explain your changes.

removes logging from the module
log error if condor_dag_submit returns something via stderror

Any other comments?
I'm ready to update unit tests or contribute more towards a new version if the proposed changes are approved!

@jodergrosse, if this change is still useful, please rebase against the updated upstream/master branch to pick up the new CI.

Will do, later today!


Internal compiler error. Aborting!

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): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: -
TensorFlow installed from (source or binary): Docker image latest-gpu-py3
TensorFlow version (use command below): 1.15.0
Python version: 3.6
Bazel version (if compiling from source): -
GCC/Compiler version (if compiling from source): -
CUDA/cuDNN version: 10.2
GPU model and memory: TITAN Xp / 12 GB

I have created a fully-quantized tf lite model from a saved model. But trying to compile it with the edgetpu_compiler, I get an error:
(venv) jiatian@desktop:~/tmp$ edgetpu_compiler saved_model_quant.tflite
Edge TPU Compiler version 2.0.267685300

Internal compiler error. Aborting!

Also tried the latest compiler, got same error:
Edge TPU Compiler version 2.0.291256449

Internal compiler error. Aborting!

Below is the how I did quantize the tflite file from saved model:
    def generate_datasets(self):
        num_calibration_steps = 10
        for _ in range(num_calibration_steps):
            input = np.random.random_sample((1, self.img_height, self.img_width, 3))
            yield [np.array(input, dtype='float32')]

    def convert_savedModel_quant_tflite(self, savedModel_dir):
        converter = tf.lite.TFLiteConverter.from_saved_model(savedModel_dir)
        converter.optimizations = [tf.lite.Optimize.DEFAULT]
        converter.representative_dataset = self.generate_datasets
        converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS_INT8]
        converter.inference_input_type = tf.float32
        converter.inference_output_type = tf.float32
        tflite_quant_model = converter.convert()
        save_tflite_path = savedModel_dir + '/saved_model_quant.tflite'
        open(save_tflite_path, "wb").write(tflite_quant_model)

The tflite file is linked (14.5MB):
saved_model_quant.zip

Hi @JiatianWu
It seems the model is hitting a HW limit on the TPU's memory. Could you try syncing the model?
I've forwarded your model to our team for review.

@Namburger How to sync the model? Thanks.

I'm sorry, that was a typo, shrink is the word I meant to say.

Hello Jaitian,
It's is not possible to reproduce the issue with the data you have provided. Could you please provide additional details about your model ? Also did you apply visulaize.py to see if there are any unsupported ops https://github.com/tensorflow/tensorflow/blob/master/tensorflow/lite/tools/visualize.py.
Thanks

I shrink the model size and it can compile now. Gonna close this issue and thanks for the help.

@JiatianWu
As it turned out, we found a bug in our compiler that was causing this issue, you're original model should works fine. We've since fixed it and your model is compiling fine. Unfortunately, we can't give out details on the next compiler release, but please stay tuned to our news page for the next release!

Thanks for sharing this info. Excited to try the next version compiler.


225 SIM-Improve error feedback on contract code changes

Fixed in https://github.com/yeagerai/genlayer-simulator/pull/382


Delete from multiple tables issue

When delete from multiple tables is done, I found $qb_where is not reset.
I'm not sure this is a issue.
test code is
$tables = array('table_a', 'table_b');
$this->db->where('id', 1);
$this->db->delete($tables);

in DB_query_builder.php line 2142:
elseif (is_array($table))
{
	empty($where) && $reset_data = FALSE;

	foreach ($table as $single_table)
	{
		$this->delete($single_table, $where, $limit, $reset_data);
	}

	return;
}

I try overwrite this code to check if $tables is not last element, then set FALSE with $this->delete() variable $reset_data.
elseif (is_array($table))
{
	//empty($where) && $reset_data = FALSE;

	foreach (array_values($table) as $table_key => $single_table)
	{
		$this->delete($single_table, $where, $limit, (($table_key == (count($table)-1))? $reset_data: FALSE));
	}

	return;
}

That's because you're not supposed to use it like that.
If you want to pass multiple tables to delete(), you must also pass the where conditions using the second parameter.

Document tell us can do that. Maybe need to modify delete example with multiple tables.
http://www.codeigniter.com/user_guide/database/query_builder.html#deleting-data

And the document is not incorrect - the example works perfectly fine.
Also relevant: #4047
It is just your expectations that are contradicting - you cannot reasonably expect that a previous where() call would be magically preserved for multiple tables, but then reset afterwards.
qb_where is either reset or not.

ok, I understand that code isn't a good solution.
My point is that when I finish delete with multiple tables, then execute other query, the second query has return a wrong result.
example1 - the second query return current query:
$tables = array('table_a', 'table_b');
$this->db->delete($tables, array('id' =>  1));

$this->db->get_compiled_select('table_a');

result:
SELECT * FROM `table_a`

example2 - the second query return wrong query:
$tables = array('table_a', 'table_b');
$this->db->where('id', 1);
$this->db->delete($tables);

$this->db->get_compiled_select('table_a');

result:
SELECT * FROM `table_a` WHERE `id` = 1

Yes, I know ... and that's why you shouldn't use it that way.

Thank you for your reply. :)


Islate also DB file of etcd per run

see problem:
https://github.com/java-operator-sdk/jenvtest/actions/runs/4605715581/jobs/8138104137?pr=63

done.


How many can the Pico (W) run in total?

I'm looking to control 5 peristaltic pumps via the Pico W. I think there are enough PWM pins, but I figured it would be worth verifying with someone.

https://github.com/CytronTechnologies/MAKER-PI-PICO/issues/13
https://github.com/sparks-baird/self-driving-lab-demo/discussions/89

This board is designed to operate 4 DC motors at up to 10V. It will also operate up to 8 servos (at the same time as the motors) although it is usual to run it at 5-6V for that as servos generally do not want 10V.
You could operate up to 8 pumps using this board and 8 RC speed controllers to drive the pump motor - something like this: https://hobbyking.com/en_us/turnigy-30a-brushed-esc.html
The motor speed ( and hence flow) is controlled via the 'servo position'.
Depending on the speed controller it can be 0% power at 0 degrees to 100% power at 180, or 0% power at 90 degrees, with +100% at 180 and -100% at 0 degrees (motor can go forwards and reverse.)
Using RC speed controllers you could also use the 'Simply Servos' board, driver software for that is here: https://github.com/KitronikLtd/Kitronik-Pico-Simply-Servos-MicroPython
Hope that helps, closing this as it seems to be complete.
If you require further assistance then contact us here
Dave

Hi Dave, thank you for clarifying! I'll keep the suggestion about the servo power controllers in mind.


Add the UAE Azure regions that were opened

LGTM, confirmed with Azure CLI

Thanks for the verification @mikhailshilkov

The build failure was not due to this PR - it was due to a flaky test - will merge and see on master before we make a release


Test fixtures aren't being checked in tests

As discovered in #137

suspected
confirmation

I ran the python3 ../spago-test.py on my local machine. I noticed that there was no text-output.txt file in the spago-test directory after a test is run. Contrast that with these files appearing:

spago-install-failure.dhall
spago-success-failure.dhall
etc.

Should the test-output.txt file be copied over as well?

@JordanMartinez no, the fixture in this case is checked against the process output, code here:
https://github.com/spacchetti/spago/blob/4471eb5d3002a95bf53f5021108c17e1fa9cf326/test/utils.py#L28-L42

I don't know python, so I used your model to add some debug prints:
# added to `call()` function
        print("Command: ")
        print(command)
        print("Failure message: ")
        print(failure_msg)
        print("Expected output fixture: ")
        print(expected_output_fixture)
        return subprocess.check_output(command, stderr=subprocess.STDOUT).decode('utf-8')
# rest of python code

This was the output for test:
Expecting success from: "spago test"
Command:
['spago', 'test']
Failure message:
Spago should test successfullytest-output.txt
Expected output fixture:
None

It looks like you forgot to add a comma at the end of this line to separate the function's arguments. Otherwise, the fixture was being appended to the failure message.
Adding that comma to my code now produces this debug output:
Expecting success from: "spago test"
Command:
['spago', 'test']
Failure message:
Spago should test successfully
Expected output fixture:
test-output.txt

@JordanMartinez thanks for debugging that! It looks like there's a logic bug in the call function: the fixture is only checked if the program doesn't exit successfully 😅

Changing call()'s usage of check_output to this:
        result = subprocess.check_output(command, stderr=subprocess.STDOUT).decode('utf-8')
        print("No exception called...")
        return result

produces this (newlines added to break up tests better:
Expecting success from: "spago init"
No exception called...

Expecting failure from: "spago init"

Expecting success from: "rm spago.dhall packages.dhall"
No exception called...

Expecting success from: "spago init"
No exception called...

Expecting success from: "rm src/Main.purs"
No exception called...

Expecting success from: "rmdir src"
No exception called...

Expecting success from: "spago init -f"
No exception called...

Expecting success from: "spago init -f"
No exception called...
Successfully verified fixture for "spago-psc-success.dhall"

Expecting success from: "spago init -f"
No exception called...
Successfully verified fixture for "spago-psc-failure.dhall"

Going to run this for 0.5s: "spago install -j 3"
Expecting success from: "spago install"
No exception called...

Expecting success from: "spago install simple-json foreign"
No exception called...
Successfully verified fixture for "spago-install-success.dhall"

Expecting failure from: "spago install foobar"
No exception called...
Successfully verified fixture for "spago-install-failure.dhall"

Expecting success from: "spago build"
No exception called...

Expecting success from: "spago build -- -o myOutput"
No exception called...

Expecting success from: "spago build --path another_source_path/*.purs"
No exception called...

Expecting success from: "spago test"
No exception called...

Expecting success from: "spago bundle --to bundle.js"
No exception called...
Successfully verified fixture for "bundle.js"

Expecting success from: "spago make-module --to module.js"
No exception called...
Successfully verified fixture for "module.js"

Expecting success from: "rm -rf .spago src test packages.dhall spago.dhall bundle.js module.js output myOutput another_source_path psc-package.json spago-psc-success.dhall spago-psc-failure.dhall spago-install-success.dhall spago-install-failure.dhall"
No exception called...

I'm going to change the issue to test fixtures aren't checked

On second thought, it appears the other fixtures are using a different mechanism. So, it really is just the test command's fixtures.

This was fixed in #141