🤖 Add authors and contributors to Practice Exercises

If you got tagged in this PR, it is because you are being credited for work you've done in the past on Exercism here, and we want to ensure you don't miss out on the recognition you deserve 🙂

With v3, we've introduced the idea of exercise authors and contributors. This information is stored in the exercises' .meta/config.json files (see the docs).
Concept Exercises, which are all new, already contain authors and contributors. Practice Exercises though are missing this information. In this PR, we attempt to populate the authors and contributors of Practice Exercises based on their commit history.
Maintainer TODO list
These are the things you, the maintainers, should do:

[ ] Check the authors and contributors, adding/removing users where the data is missing or incorrect
[ ] Double-check any renamed Practice Exercises
[ ] Check for Practice Exercises with missing authors, which are most likely due to the author not being on GitHub anymore
[ ] Add additional authors if you know that a Practice Exercise has actually been created by more than one author

For clarity, authors should be the people who originally created the exercise on this track. Contributors should be people who have substantially contributed to that exercise. PRs for typos or multiple-exercise PRs do not automatically mean someone should be considered a contributor to that exercise. Those non-exercise-specific contributions will get recognition through Exercism's reputation system in v3.
If you find something needs updating, just push a new commit to this PR.
Automatic Merging
We will automatically merge this PR before we launch v3, but will give the maximum time we can to maintainers to review it first.

The rest of this issue explains how this PR has been generated. You do not need to read it.
Algorithm
We start out by looking at the current Practice Exercises. For each Practice Exercise, we'll look at the commit history of its files to see which commits touched that Practice Exercise. How renames are handled is discussed in the "Renames" section later in this document.
Any commit that we can associate with a Practice Exercise is used to determine authorship/contributorship. Here is how we assign authorship/contributorship:
Authors

Find the Git commit author of the oldest commit linked to that Practice Exercise.
Find the GitHub username for the Git commit author of that commit
Add the GitHub username of the Git commit author to the authors key in the .meta/config.json file

Contributors

Find the Git commit authors and any co-authors of all but the oldest commit linked to that Practice Exercise. If there is only one commit, there won't be any contributors.
Exclude the Git commit author and any co-authors of the oldest commit from the list of Git commit authors (an author is not also a contributor)
Find the GitHub usernames for the Git commit authors and any co-authors of those commits
Add the GitHub usernames of the Git commit authors and any co-authors to the contributor key in the .meta/config.json file

We're using the GitHub GraphQL API to find the username of a commit author and any co-authors. In some cases though, a username cannot be found for a commit (e.g. due to the user account no longer existing), in which case we'll skip the commit.
Renames
There are a small number of Practice Exercises that might have been renamed at some point. You can ask Git to "follow" a file over its renames using git log --follow <file>, which will also return commits made before renames. Unfortunately, Git does not store renames, it just stores the contents of the renamed files and tries to guess if a file was renamed by looking at its contents. This can (and will) lead to false positives, where Git will think a file has been renamed whereas it hasn't. As we don't want to have incorrect authors/contributors for exercises, we're ignoring renames. The only exception to this are known exercise renames:

bracket-push was renamed to matching-brackets
retree was renamed to satellite
resistor-colors was renamed to resistor-color-duo
kindergarden-garden was renamed to kindergarten-garden

If you know of a rename of a Practice Exercise, please check to see if its authors and contributors are correctly set.
Exclusions
There are some commits that we skip over, which thus don't influence the authors/contributors list:

Commits authored by dependabot[bot], dependabot-preview[bot] or github-actions[bot]
Bulk update PRs made by ErikSchierboom or kytrinx to update the track

Tracking
https://github.com/exercism/v3-launch/issues/24

cc @amscotti, @camuthig, @Corentin-Leffy, @devkabiir, @ErikSchierboom, @fwip, @jackhughesweb, @jerold, @johnngugi, @jvarness, @kabiir, @kytrinyx, @rebelwarrior, @sisminnmaw, @Stargator, @SuperPaintman, @Zureka as you are referenced in this PR

@devkabiir You'll get to take your time to review this:

We will automatically merge this PR before we launch v3, but will give the maximum time we can to maintainers to review it first.

@devkabiir @Stargator Once you both agree on the changes to be made, feel free to apply the suggestions yourselves. That makes it easier for me :)

@devkabiir @Stargator Once you both agree on the changes to be made, feel free to apply the suggestions yourselves. That makes it easier for me :)

Sure thing!
I believe the initial changes in this PR made by your script were intended to be more about bootstraping rather than final decision.
I was very hesitant prior to submitting my review with the suggested changes, as it is not my intention to discourage any authors/contributors or offend them.
We mostly need your help in deciding some open questions as I think you have the most experience with how other tracks are handling this PR
From OP

Those non-exercise-specific contributions will get recognition through Exercism's reputation system in v3.

Could you point us to any available documentation for this, as it'll help in adding/removing authors/contributors. if the docs don't exist, what are the widely used interpretations of this among other tracks?
Quoting @amscotti

Do we list users as contributors for renaming? bracket-push was renamed to matching-brackets with this PR, #188

Because this change would have to be made in all tracks and It's not changing anything in the exercise itself, I removed their username from contributors as I interpreted it as non-exercise-specific. One could argue that this kind of change is explicitly exercise specific, but I had to choose 1 interpretation to get things going.
From my review

* I've removed the PR reviewers from the contributors section, should that be kept or not?

I don't believe merging/committing a PR is a substantial exercise-specific contribution, however reviewing a PR is definitely. But IDK whether these contributions count as exercise-specific or get recognized through Exercism's reputation system in v3, I've interpreted such changes as non-exercise-specific, resulting in removal of PR reviewers' usernames as well as username of the committers. I think v3 reputation system will most likely recognize contributions by track maintainers as PR committers.
Also from my review

* I've removed bulk PR authors from the contributors section, should that be kept or not?

Also interpreted such changes as non-exercise-specific.

@devkabiir If you read https://github.com/exercism/docs/blob/main/product/reputation.md#acquisition-of-reputation, hopefully the reputation system will become clearer and answer your questions.

@ErikSchierboom I haven't heard from @devkabiir. I suppose the would not be any harm in merging the suggestions I have in lieu of theirs because we could always go back and update these config files. Does that sound good to you?

@Stargator Yes, that sounds like a good plan!


Distribution proposal: support reverse-lookup from a manifest to its referrers (such as an index)

With regards to the manifest referrer metadata store discussed in https://github.com/notaryproject/nv2/pull/14, two options come to mind for implementing the reverse lookup:

Let each manifest have a reverse lookup to all indexes that reference it. Retrieving referrer metadata for a particular manifest would mean first retrieving the indexes, and then following their references to the config objects to get the requested metadata references.

Let each manifest have a reverse lookup to all config objects referenced by indexes that reference the manifest. This approach short circuits the indexes from the reverse lookup, providing direct access to the referrer metadata.

Closing as this is being tracked through the proposed reference type working group, with the current design tracked through oci.artifact.manifest PR #29


Consolidate import statement

First easy PR

Why did you close?

@MoOx You use this kind of import everywhere. So I think I should open another PR which fix all of them?
Or is this your recommend ?

Haha, maybe a new PR then ?
Thanks for this.


Review how to contribute

We have to do a big review on how to contribute in order to fit our new dev model.

@beraldoleal this was for the previous milestone. Should it be rescheduled? Or it was done?

I think that @cemsbr already did. But maybe we should take a second look.

@beraldoleal seems it can be closed, the docs page has everything working and is up to date.


OSError: [Errno 63] File name too long

Got this error for a SQL query that returned a large number of columns. Here's the culprit:
https://github.com/simonw/django-sql-dashboard/blob/d8a1f7dc177bbbb114c6a64673012f00fe257fa9/django_sql_dashboard/views.py#L119

I'm going to set a maximum length for this. It looks like 255 characters is the right value on macOS and Windows, I'll use that.


add moderator's comment

This PR is for task requested here https://utopian.io/utopian-io/@amosbastian/add-moderator-s-review-comment-to-utopian-info-database

Add comments asynchronously. It uses threads instead of processes for better performance
I have changed from map to list comprehension for threads. Somehow executor.map(x, r.json()['results']) gives TypeError once all futures are complete. It is not clear to me why, but I hope this alternative is fine 😄
I set workers 100 futures.ThreadPoolExecutor(100), this might need to be adjusted based on target machine. Works fine on my local.

concurrent.futures docs https://docs.python.org/3/library/concurrent.futures.html

Looks like using executor.map is bad idea, see here (including comments) https://stackoverflow.com/questions/49466794/

Did you find workaround or don't plan to add comments anymore?

@chhantyal couldn't get it to work, so not sure if I will try and add it in the future. Especially since the moderator's comments are becoming much less important. Thanks for the help though, even though I couldn't get it to work on my machine!


refactor

More descriptive commit messages in the future would be appreciated. :)
You know refactor doesn't say much about the change itself.


[CP] Fix flutter upgrade failing with "Unknown Flutter tag" on beta 3.14

Issue Link
https://github.com/flutter/flutter/issues/133441
Commit Hash
https://github.com/flutter/flutter/commit/33210218ba4968f6a0c118239abdcd933049bb59
Target
beta
PR Link
https://github.com/flutter/flutter/pull/133821
Changelog Description
This fixes a regression to flutter upgrade that would result in an error stating "Unknown Flutter tag" unless either the user's current working directory or they run with the --force flag.
Impacted Users
All Flutter developers upgrading their SDK via flutter upgrade.
Impact Description
Users issuing flutter upgrade on any affected release will get an error.
Workaround
Use flutter upgrade --force or change your working directory to your Flutter SDK checkout before running flutter upgrade.
 This should succeed if the fix works.

@zanderso one more please

cp lgtm


OKTA-77233

Fix an issue where internal Okta dependencies leaked into the public POM file, breaking the build.
@jmaldonado-okta @senthilkumaran-okta @lboyette-okta @federations-okta

:shipit: I agree with this patch. We should revisit the original implementation for CI to avoid these dependencies and provide a robust solution.

LGTM. Ship it!

@senthilkumaran-okta thanks, I've made the additional changes you suggested.

Thanks, the pom file changes looks good to go. #shipit :+1:


Translation of the README into French

Type of changes

[ ] Bug fix
[ ] New feature
[x] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[ ] I've run the latest black with default args on new code.
[x] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Translation of the README into French

This is great. Merci!
Would you mind updating README.md to link to the French version?

Also if you wouldn't mind could you edit codespell.yml to exclude the French readme?

That's great. Thanks again!

With pleasure, glad to have been able to help the community!


Update Delay way too big if set to something other than 0

[x] I confirm that this is an issue rather than a question.

Bug report
Actual update delay is a couple of seconds when set to something reasonable like 70ms
Steps to reproduce

Open the web UI -> Configuration -> Image Processing
Under Smoothing click activate
Set Update Delay to something other than 0 (e.g. 1)

What is expected?
Delay the update of the LEDs by set amount
What is actually happening?
The update of the LEDs gets delayed by 5-10s even when set to 1ms
System

Hyperion Server:

Build:           (HEAD detached at 83338dc3) (Paulchen Panther-ca50487e/83338dc3-1605733041)
Build time:      Nov 18 2020 21:34:12
Git Remote:      https://github.com/hyperion-project/hyperion.ng
Version:         2.0.0-alpha.9
UI Lang:         en (BrowserLang: de)
UI Access:       default
Avail Capt:      dispmanx,v4l2,framebuffer,qt
Database:        read/write

Hyperion Server OS:

Distribution:   LibreELEC (official): 9.2.6
Architecture:   arm
CPU Model:      ARMv7 Processor rev 3 (v7l)
CPU Type:       Raspberry Pi 4 Model B Rev 1.4
CPU Revision:   b03114
CPU Hardware:   BCM2835
Kernel:         linux (4.19.127 (WS: 32))
Qt Version:     5.7.1
Python Version: 3.5.3
Browser:        Mozilla/5.0 (X11; Linux x86_64; rv:85.0) Gecko/20100101 Firefox/85.0

Are there any news about this? I also have the need to delay a little the LEDs since they change before the actual image on the screen and it is pretty annoying.

Please note the update delay is in frames not in ms.
I think this was wrongly presented in the UI before and was corrected.
I suggest you retest with the latest version.
Depending on your update frequency configured, you would need to define the number of frames to be delayed.

Please note the update delay is in frames not in ms. I think this was wrongly presented in the UI before and was corrected. I suggest you retest with the latest version.
Depending on your update frequency configured, you would need to define the number of frames to be delayed.

After some tinkering I figured that for my case i solved the problem by setting duration to 25ms, frequency 25hz and delay 5 frames.

Thanks for the feedback.


Cluster fails to provision with k8s 1.16.0-beta.1-rancher1-1

rancher/rancher:master-head 70918c63b

Deploy Rancher Server
Create an ec2 cluster using k8s 1.16.0-beta.1-rancher1-1
Nodes create but the cluster fails to finish provisioning

Log statements are continuous:
2019/08/26 21:19:11 [DEBUG] REST LIST apis/apps/v1beta2//deployments
2019/08/26 21:19:11 [DEBUG] REST LIST apis/apps/v1beta2//daemonsets
2019/08/26 21:19:11 [DEBUG] REST LIST apis/extensions/v1beta1//podsecuritypolicies
2019/08/26 21:19:11 [DEBUG] REST LIST apis/apps/v1beta2//replicasets
E0826 21:19:11.907565       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.Deployment: the server could not find the requested resource
E0826 21:19:11.910706       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.DaemonSet: the server could not find the requested resource
E0826 21:19:11.923157       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.ReplicaSet: the server could not find the requested resource
E0826 21:19:11.923307       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta1.PodSecurityPolicy: the server could not find the requested resource
2019/08/26 21:19:12 [DEBUG] REST LIST apis/apps/v1beta2//statefulsets
E0826 21:19:12.863528       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.StatefulSet: the server could not find the requested resource
2019/08/26 21:19:12 [DEBUG] REST LIST apis/apps/v1beta2//deployments
2019/08/26 21:19:12 [DEBUG] REST LIST apis/apps/v1beta2//daemonsets
2019/08/26 21:19:12 [DEBUG] REST LIST apis/apps/v1beta2//replicasets
2019/08/26 21:19:12 [DEBUG] REST LIST apis/extensions/v1beta1//podsecuritypolicies
E0826 21:19:12.931742       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.Deployment: the server could not find the requested resource
E0826 21:19:12.935225       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.DaemonSet: the server could not find the requested resource
E0826 21:19:12.950053       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta2.ReplicaSet: the server could not find the requested resource
E0826 21:19:12.950392       5 reflector.go:126] github.com/rancher/norman/controller/generic_controller.go:175: Failed to list *v1beta1.PodSecurityPolicy: the server could not find the requested resource

rancher/rancher:master-head 57508ebba

Created a 1.16 cluster
Cluster created and was able to deploy some workloads.

the affected apis are here:
https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
for us this meant:

Apps
DaemonSet
Deployment
ReplicaSet
StatefulSet
PSP

Functional places touched in the code:

alerts
monitoring
pipelines
our workload api (which is an almagamation of daeomo/deploy/stateful/replica)
PSP stuff


Add support for Ambari 2.0, other versions.

Hey guys!
Ambari 2.0 has been out for awhile now.  There are some pretty significant changes, so I think it's probably a good idea to support multiple versions.
I have a forked version supporting 2.0 I've been running for about a week now.  I'll put together a pull request if there is interest in that.
Regards,
Joe Reid

Closed by https://github.com/jp/ambari/pull/5.
Cheers.


VideoCapture.Retrieve(...) or VideoCapture.Read(...) don't work for UWP

Summary of your issue
In UWP app, VideoCapture.Retrieve(...) does not work. It keeps giving access violation error.
Environment
UWP, VisualStudio 2022
What did you do when you faced the problem?
Get stuck
Example code:

paste your core code
```            VideoCapture capture = new VideoCapture(0);
            using (Mat image = new Mat()) // Frame image buffer
            {
                //Cv2.WaitKey(30);
                while (openCamera)
                {
                    bool b = capture.Grab();
                    capture.Retrieve(image);
                }
            }

## Output:

<!-- ↓do not remove ``` please!!! -->

paste your output
  HResult=0x80004003
  Message=Attempted to read or write protected memory. This is often an indication that other memory is corrupt.
  Source=<Cannot evaluate the exception source>
  StackTrace:
<Cannot evaluate the exception stack trace>

## What did you intend to be?
Read video frame captured

I have the same problem.
I uwp I can't retrieve nothing, not udp, https, rtsp or local file..


dune_2: 2.3.0 -> 2.3.1

Motivation for this change
Changelog: https://github.com/ocaml/dune/releases/tag/2.3.1
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg build ocamlPackages.dune-configurator


traffic light (apollo 3.5)

**, Linux Ubuntu 14.04
**source
**Apollo version (3.5):

Hello, Apollo :
Excuse me, Does apollo 3.5 support traffic light detection? If not, which version of the previous version is supported? I read the news that apollo 2.0 seems to be supporting. I found that the traffic light detection module of apollo3.5 has changed a lot compared with the previous one.

@breakisx we do support traffic light detection in Apollo 3.5. It is an essential part of Urban Autonomous Driving. Let me know if you have additional questions. Thanks!

In 3.0 and before, used to have modules/perception/traffic_light. In 3.5 it's gone. May I know where it is moved to? Thanks

From Apollo 3.5, traffic light detection algorithms are in modules/perception/camera/app/ and modules/perception/camera/lib/traffic_light.

Closing this issue as it appears to be resolved. Feel free to reopen if you have additional questions. Thanks!


crypt_and_hash.c bug with GCM

The following test is a buggy in that it fails when used with GCM ciphers (e.g. AES-128-GCM)
mbedtls/programs/aes/crypt_and_hash.c

    if( ( ( filesize - mbedtls_md_get_size( md_info ) ) %
            mbedtls_cipher_get_block_size( &cipher_ctx ) ) != 0 )
    {
        mbedtls_fprintf( stderr, "File content not a multiple of the block size (%d).\n",
                 mbedtls_cipher_get_block_size( &cipher_ctx ));
        goto exit;
    }

My understanding is that GCM does not require padding.

ARM Internal Ref: IOTSSL-678

Fixed by PR#546, so the issue can now be closed.


Feature: Support for external ATS for parachutes

Describe problem solved by this pull request
Support for external automatic trigger systems for parachutes (ATS) as required by ASTM F3322-18. This serves to terminate the flight while the ATS handles deploying of the chute.
i.e: SATS-MINI
Describe your solution
Added uORB publishing to the pwm_input module. Added parameters to enable the ATS and configure the PWM trigger threshold . Added another status check in Commander::FailureDetector to monitor the PWM input on AUX5.
Describe possible alternatives
We could not support this, but I think it is useful. Perhaps @eengelgau can chime in here.
Test data / coverage
Tested on the bench successfully.
FYI
@eengelgau
@dagar

Dang, flash overflow. Looks like I need to take care of CDev usage right meow.

Completely removed CDev usage from pwm_input and ll40ls, moved to just uORB. Cleaned up dead code, fixed the pwmin_info command, and refactored out lidar lite specific code to the ll40ls driver.
Tested successfully on the bench, if anyone has a LidarLite it would be good to verify the PWM interface still works as expected.

Hi Everyone, this is Gene Engelgau with Fruity Chutes.  We manufacture parachute recovery systems for drones. Last year (2018) I was involved with the ASTM standards committee that created the specification for what it means to have a certified parachute recovery system for a drone for the purposes of flight over people, or for other high-risk use.  The standard in the F.38 section is ASTM F3322-18 and is available here for purchase:
https://webstore.ansi.org/Standards/ASTM/ASTMF332218
The FAA (and other similar government agencies in other countries) are now referring people to this standard who want waivers for flight over people. In that specification there are some mandatory requirements for several key items that also affect the autopilot.  First, the standard says that there shall be a Autonomous Trigger System (ATS) that will detect the drone failure to trigger the parachute.  We make this device here that satisfies this requirement:
https://fruitychutes.com/uav_rpv_drone_recovery_parachutes/sentinel-automatic-trigger-system.htm
Below is the F3322-18 clause that defines this:
3.1.4 autonomous triggering system, ATS, n—device or components independent from any ﬂight critical system of the sUA that will detect and initiate parachute deployment upon detection of a critical failure of the sUA in ﬂight.
The ATS has it's own IMU and other sensors that can detect the drone falling or flipping (out of control motion that should otherwise never happen).  In this case the ATS will trigger the parachute.
The ATS also will generate another signal that is sent to the Autopilot to stop the rotors from spinning. This is called the flight termination system.  The best and most reliable way to do this is to have the autopilot accept a PWM pulse width (or other ways can be done as well like logic level) that when asserted will cause all the rotors to immediately stop. Here is the paragraph from the standard that refers to the flight termination (stop the rotors)
3.1.16 ﬂight termination system, FTS, n—device or components that will disable the propulsion system of the sUA.
Below are several paragraphs that makes the assertions regarding these:
5.8.4 To prevent propellers from injuring people on the ground after a deployment the PRS shall also include a Flight Termination System (FTS).
<IP_ADDRESS> The FTS shall activate at the time of or before parachute deployment is initiated.

Having this feature added to the Autopilot is the best way to do this.
Contact me if people have questions - Gene Engelgau, Fruity Chutes Inc.
PS - A company in California is already flight testing this feature using our ATS system (SATS-MINI).

@bkueng I wasn't sure how to make ModuleBase work with the timer ISR. Since the ISR needs to access functions from PWMIN, and currently there is no method to share the ModuleBase class pointer to functions outside of the class.

Another thing worth mentioning. Daniel suggested using the px4::work_queue, however I unsure if interrupts + wq scheduling is going to cause issues.
i.e: interrupt fires, sets member data "status, period, pulse_width", and calls ScheduleNow. If another interrupt fires during the execution of Run there could be a data race. Also requires member variable access from a static member function, which is annoying.

Another thing worth mentioning. Daniel suggested using the px4::work_queue for the uORB publication, however it seems like interrupts + wq scheduling might cause issues.

Either way is fine here, but longer term I want to see if we can get the majority of work (anything involving floating point or ORB) out of ISRs.

Blocked by v2 flash limit

FYI This will require (at least) an update to http://docs.px4.io/master/en/peripherals/parachute.html. From what I can tell this is just a signal on a particular port (MAIN5/AUX5) that tells the system that parachute triggering has happened? (or is it "must happen" - and PX4 does the triggering?).
How does this interact with the existing flight termination behaviour?
@bkueng Triggering off fixed ports has always been a bit of a pain for camera - ie we're always running into particular frames that use the ports already etc. Is there no way to make these things more configurable?

From what I can tell this is just a signal on a particular port (MAIN5/AUX5) that tells the system that parachute triggering has happened?

Yes that is correct. This PR simply engages flight termination based on that signal.

Yes that is correct. This PR simply engages flight termination based on that signal.

Thank you! Is everything else respected? i.e. the circuit breaker etc?

Thank you! Is everything else respected? i.e. the circuit breaker etc?
Yes, this behaves in exactly the same way as exceeding the roll/pitch limits

@dagar Are we OK to merge?


NEP-5 Amendment

This amendment of NEP-5 standardizes the implementation details of each method.

It's perfect for me

There are several issues related to the completeness of the NEP-5 specification from an implementation and testing/quality assurance perspective.  For example, http://docs.neo.org/en-us/exchange/v2.7.3.html#dealing-with-nep-5-assets-transactions is a more complete, more definitive, more authoritative specification for NEP-5 than than the NEP-5 specification itself.   This content needs to be either (a) moved/copied into NEP-5, or (b) there needs to be a new NEP-5 that defines the basic architecture of a NEO smart contract (e.g. the Main method, if-then-else dispatching, individual callable methods).
You can find a more detailed discussion and motivation for the above here:
https://github.com/neo-project/proposals/issues/33
and here
https://github.com/neo-project/proposals/issues/32
The specification needs to be corrected/updated. It's not strong enough to be implemented reliably on the NEO platform as it is written. It needs to be a specification ...not a high-level guide.

The different formats and lengths of an NEO Account address are applicable across the entire NEO architecture and should be contained in their own NEP specification - and referenced by NEP-5.  They shouldn't be defined in NEP-5. The new NEP should also define specific named constants for these lengths.

@mwherman2000 NEP is Neo enhancement proposal, not the yellow paper.
NEO yellow paper is on the way.

@erikzhang What yellow paper? Do you have a link to a draft?
These are highly relevant topics as far as the NEP-5 discussion is concerned and the discussion can't assume anything about a document that no one has seen.  Right?

@mwherman2000 I mean, NEO's basic design will be defined in the upcoming yellow paper, not in the NEO Enhancement Proposals. NEP is for new proposals.

@erikzhang when can we expect to see it? Do you have a draft version?

I don't have a draft yet. Maybe I should create a new repository for it and anyone can contribute to it.

The function SHOULD check whether the from address equals the caller contract hash. If so, the transfer SHOULD be processed; If not, the function SHOULD use the SYSCALL Neo.Runtime.CheckWitness to verify the transfer.

Is this implying that sending a transfer(address1,address1,amount) should succeed?

If the to address is a deployed contract, the function SHOULD check the payable flag of this contract to decide whether it should transfer the tokens to this contract.

How does one check the payable flag?

The function SHOULD check whether the from address equals the caller contract hash. If so, the transfer SHOULD be processed; If not, the function SHOULD use the SYSCALL Neo.Runtime.CheckWitness to verify the transfer.

Is this implying that sending a transfer(address1,address1,amount) should succeed?

It means that if contract A calls the transfer method of contract B, and parameter from is the scripthash of contract A, then the transfer should be processed.

How does one check the payable flag?

Use the API Neo.Contract.IsPayable
https://github.com/neo-project/neo/blob/b0b0646fb86b350e57f9010ca88ca373144d1353/neo/SmartContract/StateReader.cs#L127

Let's vote?

NEP-5 now talks about methods everywhere ...and not "operationX".
Does this imply that only an NEP-5 SC's detailed methods are the only way to invoke their functionality?  ...i.e. has calling Main been deprecated?  When should an app call Main and when should it call one of the detailed methods?

The parameter amount MUST be greater than or equal to 0
to
The parameter amount MUST be greater than 0
?
Why we need a TX with 0 amount?

Why we need a TX with 0 amount?

The transfer method may be invoked by another contract, and the number may be calculated in that contract, possibly equal to zero. To ensure the success of the entire contract, we should allow a transfer of 0 amount.

The changes are good but I'm just wondering how it's possible to set Neo.Contract.IsPayable on existing contracts.

The changes are good but I'm just wondering how it's possible to set Neo.Contract.IsPayable on existing contracts.

I will create new APIs for doing that.

lgtm - I particularly like the clarity around transfer events that create and burn tokens.

@erikzhang Can you answer these questions? https://github.com/neo-project/proposals/pull/44#issuecomment-395970612

@mwherman2000 I'm sorry I didn't understand your questions.

@erikzhang  There are 2 primary ways that the functionality in a "detail method" in an SC (i.e. any public method other than Main()) can be invoked:

As an operation string + an array of arguments passed in an invocation of Main(), or
By executing an invokefunction to invoke one of the detail methods directly - bypassing Main() (reference:  http://docs.neo.org/en-us/exchange/v2.7.4.html#dealing-with-nep-5-assets-transactions)

The new wording in the NEP-5 Amendment removes any references to approach #1 ...and now only refers to method calls (approach #2).
Is this what was intended? ...or does the wording need to be expanded to cover both approach #1 and approach #2?

I think you have misunderstood. It is not possible to bypass the main method to invoke the other methods in a contract directly.

The NEO documentation "Dealing with NEP-5 Assets Transactions" says you can bypass Main(), doesn't it?
Reference: http://docs.neo.org/en-us/exchange/v2.7.4.html#dealing-with-nep-5-assets-transactions

The code you linked to shows what? ...that all calls to non-Main() SC methods are always invoked by calling Main() with the operation string + array of objects as arguments?  Please be patient with me.

The code you linked to shows what? ...that all calls to non-Main() SC methods are always invoked by calling Main() with the operation string + array of objects as arguments? Please be patient with me.

Yes, that's it.

RE: Erik: "It is not possible to bypass the main method to invoke the other methods in a contract directly."
If the only way to invoke the non-Main() methods is by calling Main() with the operation string + array of objects as arguments, then I've got the following questions about the wording in the NEP-5 Amendment:

Why does the Amendment use method signatures to document the NEP-5 operations? ...this is very confusing when in reality it doesn't matter what the non-Main() method signatures are as long as Main() does the right thing when it receives a valid operation and array of object arguments, correct?

Why does the Amendment declare the non-Main() methods to be public when they should be private ...if Main() is the only method that calls them?  ...why are these methods needed at all?

If "it is not possible to bypass the main method to invoke the other methods in a contract directly"., then an NEP-5 compliant SC can be implemented entirely within the Main() method with no non-Main() methods, correct?  That is, there is no requirement to have any non-Main() methods at all in an NEP-5 compliant SC, correct?

The simplest specification could be a table, n'est pas?

operation
args[0]
args[1]
args[2]
args[3]
return type

"totalSupply"

BigInteger

"name"

string

"symbol"

string

"decimals"

byte

"balanceOf"
byte[] account

BigInteger

"transfer"
byte[] from
byte[] to
BigInteger amount

bool

Actions  will still need to be defined using "method" notation.  i.e. I don't think they fit into the tabular format very well.

If "it is not possible to bypass the main method to invoke the other methods in a contract directly"., then an NEP-5 compliant SC can be implemented entirely within the Main() method with no non-Main() methods, correct? That is, there is no requirement to have any non-Main() methods at all in an NEP-5 compliant SC, correct?

Yes. Theoretically, we can light the bulb by simply controlling the electrons. But we're still going to use the switch, right? Although using the switch, we still control the electrons at the physical level, but this is two different levels of things.

@erikzhang , can you enlighten me? :-)  To implement an NEP-5 compliant SC, do the non-Main() methods need to be implemented or not?  If they do, what is the non-electron justification?

To implement an NEP-5 compliant SC, do the non-Main() methods need to be implemented or not?

I don't want to be overly entangled in words. A method of a contract is not specifically a method in some programming languages, but something abstract. You can just think of it as an operation.

NEP-5 needs to be usable and useful as an implementation, testing and an off-chain/on-chain interoperability specification. @erikzhang, it can't be vague. It can't leave Interface details open to speculation or subjective interpretation.

@erikzhang  Would you like me to re-edit NEP-5 based on the above?

You only have one entry point. Is main. Then you can do whatever you want

@shargon Agreed. But this needs to be made absolutely clear in the NEP-5 specification and the Amendment as well as the current specification are misleading.  They both use C# syntax to describe the agruments to Main().  This makes no sense, is confusing and misleading.  Why does the Amendment specifiy an NEP-5 SC use C# syntax to specify the SC has a method
public static bool transfer(byte[] from, byte[] to, BigInteger amount)
when it is more simply and unambiguously stated that one of the allowed sets of arguments that can be passed to public static object Main(string operation, params object[] args) is:

operation
args[0]
args[1]
args[2]
args[3]
return type

"transfer"
byte[] from
byte[] to
BigInteger amount

bool

Let's break the pattern of creating a House of Cards.

At this time there needs to be an allocation of resources to explicitly document the neo smart contract platform. It may be a bounty, stake holder driven, or a person without neo council.
From the outside looking in Solidity is much more straightforward. I am not a c# dev so I may have some bias, as well as the fact ethereum is very well documented.
Even with well documented code there are still room for mistakes, the latest being the icx contract. The more room left for interpretation is the more rope you have given to the developer.

Would you like me to re-edit NEP-5 based on the above?

If we decide to change the wording, then all the NEP will have to be modified so that their style is consistent. But I prefer to use the existing wording.

From the outside looking in Solidity is much more straightforward. I am not a c# dev so I may have some bias, as well as the fact ethereum is very well documented.

Neo and NeoContract are not designed for a particular language. Their design goal is to use any programming language for smart contract development. At the same time Neo and NeoVM also have a variety of different language implementations.
So their documents must be an abstract expression. But I agree that we should use as precise a language as possible to describe all the standards.

FWIW, ethereum also uses solidity to describe a large part of their specs, even though solidity is not the only language that compiles for the evm.
See the recently finalized NFT standard for example: https://github.com/ethereum/EIPs/blob/master/EIPS/eip-721.md

@RavenXce, the ETH examples are largely irrelevant because Solidity/ETH SCs don't have the same SC architecture/structure that NEO smart contracts have where a NEO SC invocation is a call to one and only one SC entrypoint method Main().  The sole essence of the NEO smart contract procotol is to pass operation and args parameters to Main() and Main() alone.  NEP specifications need to be clear and compatible  and totally unambiguous about this.
Any other functions you define in your C# contract can't be called and aren't call.  So list these methods in the NEP-5 (or any NEP) specification is misleading.

Sounds good to me, adding SHOULD recommendations for burning and minting tokens on the transfer event is the only request I would make (since it doesn't change the standard practically I don't thing that is a big issue.
Is it possible to just use null ? So if is minting from is null and if burning to is null in the event and observers can just treat as such ?

null is not a valid/usable/reliable construct in C#.NEO ...if that is the specific null concept you're referring to @canesin.
Reference: https://youtu.be/Nj4-m2o94VE?t=33m32s ...listen to the next 2 slides ...until timecode 35:39  ...2 minutes.

RE: Given that, @mwherman2000, what you're suggesting is that the spec should be written in terms of the state of the stack at the beginning of the smart contract script execution.
@dicarlo2 To be perfectly clear, I've never stated, suggested or endorsed defining NEP-5's invocation pararmeters in terms of the NEO VM stack, correct?  never.
However, if you reread this thread, we have had two experts, @erikzhang and @shargon, attest that, in fact, there is and needs to be the Main() method in a C# smart contract and all invocations of a NEO C# smart contract call the Main() method.  It's the only prescribed/supported way to invoke a SC's internal functionality.  Here's the evidence:

Erik: https://github.com/neo-project/proposals/pull/44#issuecomment-399314648
Erik: https://github.com/neo-project/proposals/pull/44#issuecomment-399319297
Shargon:  https://github.com/neo-project/proposals/pull/44#issuecomment-399434394

@dicarlo2 Where/who is the source of your information?  Do you have to unwind/rewind the discussion thread?  Please clarify your sources of your information?  If have to revisit these first principles, let start again.
@dicarlo2 Perhaps you can double check you information offline with @erikzhang  and @shargon.  We need a 3-way confirmation/agreement here.

@Ejhfast Exactly because of these situations.  I know the answers and I'm only asking the questions to lead the discussion and get written answers that we can reference in discussions like the one we're having now.
@Ejhfast and @dicarlo2: Are @erikzhang  and @shargon correct or not correct?  See the 3 links quoted above.
[Note: I've edited my previous comment several times to make it as clear and precise as possible.]

Yes: Erik, Shargon, and Alex are all correct. There is no inconsistency in
what they are saying. The current C# compiler requires a Main() method.
This does not mean that future compilers or the VM require that. Alex even
linked to a compiler that does not require a Main() method right now
On Sun, Jun 24, 2018 at 10:33 AM, Michael Herman (Toronto) <
<EMAIL_ADDRESS>wrote:

@Ejhfast https://github.com/Ejhfast Exactly because of these
situations. I know the answers and I'm only asking the questions to lead
the discussion and get written answers that we can reference in discussions
like the one we're having now.
@Ejhfast https://github.com/Ejhfast and @dicarlo2
https://github.com/dicarlo2: Are @erikzhang
https://github.com/erikzhang and @shargon https://github.com/shargon
correct or not correct? See the 3 links quoted above.
[Note: I've edited my previous comment several times to make it as clear
and precise as possible.]
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neo-project/proposals/pull/44#issuecomment-399773305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFeEmtnHeDu_ONlJ0olWvM8zpMaK6Lkks5t_810gaJpZM4UC78S
.

@Ejhfast and @dicarlo2: Maybe we're reshaping this discussion topic to: How to represent multiple language bindings in an NEP.
I need on the C# Main() binding as the authoritative approach because C# is the reference implementation language for the NEO platform.  If a non-C# language has a different binding, that's a different situation. Perhaps we need to discuss that separately.
However, the core issue is the current convention of using C# method syntax is misleading:
public static bool transfer(byte[] from, byte[] to, BigInteger amount)
...and is contributing to the fragility and failures of the NEO platform.

You are trying to create a lot of discussion around something that is not a
big deal. At NEX we are building very sophisticated infrastructure on this
chain, and the good and bad parts of NEO become much, much clearer when
doing that. I can think of many important things that should be improved in
NEO right now (e.g., the current fee system), and this discussion is
distracting from that.
It would be much better if the energy put into discussion here was instead
directed at documentation efforts or other forms of ecosystem building. You
seem to be worried about people being confused by standards. There is
general agreement here that the standard is fine. But I think we'd all
agree that more needs to be done to onboard new developers through
tutorials and documentation and new resources -- the community needs that
very much!
On Sun, Jun 24, 2018 at 10:44 AM, Michael Herman (Toronto) <
<EMAIL_ADDRESS>wrote:

@Ejhfast https://github.com/Ejhfast and @dicarlo2
https://github.com/dicarlo2: Maybe we're reshaping this discussion
topic to: How to represent multiple language bindings in an NEP.
I need on the C# Main() binding as the authoritative approach because C#
is the reference implementation language for the NEO platform. If a non-C#
language has a different binding, that's a different situation. Perhaps we
need to discuss that separately.
However, the core issue is the current convention of using C# method
syntax is misleading:
public static bool transfer(byte[] from, byte[] to, BigInteger amount)
...and is contributing to the fragility and failures of the NEO platform.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neo-project/proposals/pull/44#issuecomment-399774020,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFeEiF9QZ9DMr43KATaWj0e7pmMriADks5t_8_lgaJpZM4UC78S
.

@Ejhfast I wrote NEP-10. My goal is to promote a develop a trusted ecosystem of hundreds ...maybe thousands of NEPs (and NCPs).  NEP-5 is incredibly important in this regard.  It's like the Block 0 NEP.
To allow the current situation of vague and misleading C# method syntax continue is untenable to the point of ludicrous.
The NEO platform is in serious trouble because of its fragile software architecture (https://github.com/neo-project/neo/issues/272#issuecomment-398721473).  More or better developer tutorials isn't going to fix the problem.

The proposed changes are straight forward and further enhance the clarity of the standard: APPROVED
As @Ejhfast has indicated, the extent of this conversation is a poor use of resources for people who could be focusing on critical platform development (including some that @mwherman2000 has directly requested).  @mwherman2000 I would also recommend auditing your method of seeding discussion.  I am receiving many messages on the discord about the aggressive (and sometimes offensive) nature of these conversations.

#RIP


https://developer.mozilla.org/ru/docs/Web/JavaScript/JavaScript_technologies_overview

[ ] Correction or update

Details
The page (https://developer.mozilla.org/ru/docs/Web/JavaScript/JavaScript_technologies_overview) at the end refers to the common YUI library that died in 2015

YUI link removed.


Update relays.yaml

Adding nostr.ncsa.illinois.edu

Pretty sure this is the first .edu relay I've seen


[exp] Lazyload for multimodal inputs

What does this PR do?
Fixes #5308
Fixes #5331
In this PR, we moved the image processor's process step from the pre-processing phase to the training phase, thereby saving disk space for cached datasets. However, it could affect training throughput if the CPU is poor.
Moreover, we introduced a preprocessing_batch_size argument to control the batch size during the pre-processing phase, to prevent process hangs.
To use dataset streaming for multimodal datasets, the best practice is:
dataset: your_dataset
buffer_size: 128
preprocessing_batch_size: 128
streaming: true
accelerator_config:
  dispatch_batches: false

dispatch_batches: false is necessary for multimodal datasets since the batch size of images may differ from that of input ids.
https://huggingface.co/docs/transformers/main_classes/trainer#transformers.TrainingArguments.accelerator_config
Before submitting

[x] Did you read the contributor guideline?
[x] Did you write any new necessary tests?

Is there anyway to restore this behaviour? Like an option somewhere?

streaming=true mode has a hug impact on throughput
If your cpu-memory is enough, but you dont want to wait a long tokenizing time before training, you can use LazyLLMDatset instead.

If I start with DeepSpeed, do I still need to set this parameter?
accelerator_config:
dispatch_batches: false

yes


[Question] Why using a custom controller and not an initializer?

I haven't had a close look at the code, but I would say that it is worth specifying why you implemented a custom controller instead of an custom initializer. What you describe in your README matches the description here: https://kubernetes.io/docs/admin/extensible-admission-controllers/#initializers:

When to use initializers?
Initializers are useful for admins to force policies (e.g., the AlwaysPullImages admission controller), or to inject defaults (e.g., the DefaultStorageClass admission controller), etc.

My understanding of initializers  is that they are used for mutating a single resource when it's created/updated.
In the case of the pdb-controller it's looking at some resources; deployments and statefulsets, and creating a third resource, PodDisruptionBudget, based on that. So it doesn't need to mutate anything, and thus I don't think an initializer would be the right thing.

👍


"Rasa" is listed in the drop-down menu

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

Step 1
Type "Rasa" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Rasa, CĂLĂRAȘI (Grădiștea)".


Unable to load model from different directory

Hi there,
I'm trying to load a trained language model from another project. Unfortunately, I'm not able to load it because it requires the definition of the model module. As far as I know, this is a known problem for PyTorch model saved using torch.save (as discussed here: https://github.com/pytorch/pytorch/issues/3678).  How do you deal with this problem?
Thank you in advance!
Alessandro

Hey,
have you found a solution, because this is currently my problem


Max recv buffer size reached, dropping for user Plumble_User seq: 11484

I ran the example scene and everything worked as expected.
The sound quality feels good!
But after a while when receiving there is a warning on the screen and the sequence number increase:
...
Max recv buffer size reached, dropping for user Plumble_User seq: 1708
Max recv buffer size reached, dropping for user Plumble_User seq: 1710
Max recv buffer size reached, dropping for user Plumble_User seq: 1740
Max recv buffer size reached, dropping for user Plumble_User seq: 1742
...
This happens in the AudioDecodingBuffer but the sound still work and the sound is clear.
There is a red line in the editor graph that is correlated with the warning message, close to be continuous after the first warning.
The audio is transmitted from a plumble (android client) continuously. Whenever the transmission is stopped, the counter reset.
Should the data dropped affect the output audio?
Is that warning important?
I am currently in unity 2018.2 and using
mumur 1.2.19
Client version 1.2.5 (Android: Plumble Free 3.2.0)

This also happens to me, and to be honest I'm not certain what the cause is.
I know that this happens whenever the audio isn't being pulled fast enough, like during frame drops when the CPU or GPU choke, but it also seems like it may occur in periods when the application is hitting the target framerate.

The audio is transmitted from a plumble (android client) continuously. Whenever the transmission is stopped, the counter reset.
That number correlates to the audio packet sent from the android client, and is expected to increase monotonically.

Should the data dropped affect the output audio?
Yes, it should, but I haven't observed audio corruption occurring at the same time as this log message.

I can't take a deeper look right now, but if you are able to do any deeper research into this problem I'd be happy to assist in any way I can

Tested again at home, and I am unable to reproduce. Maybe rebooting my pc or unity "solved" the error.
I did get a few "Crypt: 4"  error but that seems unrelated
Crypt: 4 UnityEngine.Debug:LogError(Object) Mumble.CryptState:Decrypt(Byte[], Int32) (at Assets/Mumble-Unity/Scripts/CryptState.cs:274)
I suggest closing the issue, and I will post update if I'm able to reproduce at some point.
Thank you,
Simon

I'm going to leave this open for now, in case anyone experiences this problem again


Automatically scale pool maximum connections

I see that right now, DatabaseConnectionPoolConfig.maximumConnections is hardcoded to 10 by default.
It would be nice if this number could dynamically scale at startup based on something like:
database maximum available connections / number of workers (cores?)

My database knowledge is rusty, but they all support some kind of connection reporting, right?, ie for Postgres:
select max_conn,used,res_for_super,max_conn-used-res_for_super res_for_normal
from
  (select count(*) used from pg_stat_activity) t1,
  (select setting::int res_for_super from pg_settings where name=$$superuser_reserved_connections$$) t2,
  (select setting::int max_conn from pg_settings where name=$$max_connections$$) t3

Produces:

max_conn
used
res_for_super
res_for_normal

300
175
3
122

can someone please answer this.  I have the same question.

One issue I can see with querying the number of connections available/used is that it might cause problems if your app scales horizontally. E.g. if you have 300 connections and one app instance, we might choose
max_connections - some_buffer / number of workers

So let's say 200. However, if we then scale up to 5 instances we either end up in the situation where each app is trying to use 200 connections (so 1000 attempted connections for 300 max connections) or one instance has 200 and the other 4 have to share the remaining 100, meaning that they're less performant


Spevacus: Watch firmware-file.in

Spevacus requests the watch of the watch_keyword firmware-file\.in. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

firmware-file\.in has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Mast in Charcoal HQ


memory usage increasing

Yes, memory will increase with more keyframes placed in the environment.

Hello !
First thanks for the package, I have great results and the localization is really robust!
I have the same “problem” as yueyihua, with the memory increasing as the mission goes on. I'm on Orin Nx, Ubuntu 20.04, Ros Noetic.
I was wondering if it was possible to set up some kind of buffer that could save and reset keyframes as they go (like a snapshot) and that would allow the memory used to be reduced regularly?
It would make the code even more lightweight!
Thanks again !


The label-studio-ml package is shipped with an unnecessary scikit-learn dependency

My understanding of the Python ecosystem is quite limited, please excuse my shortcomings related to this issue.
Description
The label-studio-ml package is shipped with scikit-learn==0.24.1.
This means that projects using scikit-learn>=0.24.1 cannot be used in conjunction with label-studio-ml==1.0.7 as it will uninstall the newer version of scikit-learn.
More precisely, my model uses the method get_feature_names_out on a CountVectorizer object, added in scikit-learn==1.0.0 (please note that the method get_feature_names available in scikit-learn==0.24.1 has been deprecated in scikit-learn==1.0.0 as well).
bag_of_words = CountVectorizer(
    stop_words="english", max_features=max_features, ngram_range=(1, ngrams)
)

bag_of_words.fit(train_words)
train_words_binary_matrix = bag_of_words.transform(train_words)
feature_names = bag_of_words.get_feature_names_out()
tfidf = TfidfTransformer(smooth_idf=False)
tfidf.fit(train_words_binary_matrix)
train_words_tfidf_matrix = tfidf.transform(train_words_binary_matrix)

Steps to reproduce
# Generate the virtual environment
$ python3 -m venv .venv

# Activate the virtual environment
$ source .venv/bin/activate

# Display the current requirements
$ cat requirements.txt
scikit-learn==1.1.1

# Install the requirements
$ pip install --requirement requirements.txt
Collecting scikit-learn==1.1.1
  Using cached scikit_learn-1.1.1-cp39-cp39-macosx_10_13_x86_64.whl (8.6 MB)
Collecting joblib>=1.0.0
  Using cached joblib-1.1.0-py2.py3-none-any.whl (306 kB)
Collecting threadpoolctl>=2.0.0
  Using cached threadpoolctl-3.1.0-py3-none-any.whl (14 kB)
Collecting numpy>=1.17.3
  Using cached numpy-1.22.4-cp39-cp39-macosx_10_15_x86_64.whl (17.7 MB)
Collecting scipy>=1.3.2
  Using cached scipy-1.8.1-cp39-cp39-macosx_12_0_universal2.macosx_10_9_x86_64.whl (55.6 MB)
Installing collected packages: threadpoolctl, numpy, joblib, scipy, scikit-learn
Successfully installed joblib-1.1.0 numpy-1.22.4 scikit-learn-1.1.1 scipy-1.8.1 threadpoolctl-3.1.0

# Install label-studio-ml package
$ pip install label-studio-ml
Collecting label-studio-ml
  Using cached label_studio_ml-1.0.7-py3-none-any.whl (26 kB)
Collecting label-studio-tools==0.0.0.dev11
  Using cached label_studio_tools-0.0.0.dev11-py3-none-any.whl (9.8 kB)
Collecting Flask==1.1.2
  Using cached Flask-1.1.2-py2.py3-none-any.whl (94 kB)
Collecting requests<3,>=2.22.0
  Using cached requests-2.28.0-py3-none-any.whl (62 kB)
Collecting colorama>=0.4.4
  Using cached colorama-0.4.5-py2.py3-none-any.whl (16 kB)
Collecting itsdangerous==2.0.1
  Using cached itsdangerous-2.0.1-py3-none-any.whl (18 kB)
Collecting lxml>=4.2.5
  Using cached lxml-4.9.0-cp39-cp39-macosx_10_15_x86_64.whl (4.6 MB)
Collecting attrs>=19.2.0
  Using cached attrs-21.4.0-py2.py3-none-any.whl (60 kB)
Collecting scikit-learn==0.24.1
  Using cached scikit_learn-0.24.1-cp39-cp39-macosx_10_13_x86_64.whl (7.3 MB)
Collecting Pillow
  Using cached Pillow-9.1.1-cp39-cp39-macosx_10_10_x86_64.whl (3.1 MB)
Collecting attr==0.3.1
  Using cached attr-0.3.1.tar.gz (1.7 kB)
  Preparing metadata (setup.py) ... done
Collecting werkzeug==2.0.2
  Using cached Werkzeug-2.0.2-py3-none-any.whl (288 kB)
Collecting appdirs>=1.4.3
  Using cached appdirs-1.4.4-py2.py3-none-any.whl (9.6 kB)
Collecting Jinja2==3.0.3
  Using cached Jinja2-3.0.3-py3-none-any.whl (133 kB)
Collecting click>=5.1
  Using cached click-8.1.3-py3-none-any.whl (96 kB)
Collecting MarkupSafe>=2.0
  Using cached MarkupSafe-2.1.1-cp39-cp39-macosx_10_9_x86_64.whl (13 kB)
Requirement already satisfied: threadpoolctl>=2.0.0 in ./.venv/lib/python3.9/site-packages (from scikit-learn==0.24.1->label-studio-ml) (3.1.0)
Requirement already satisfied: joblib>=0.11 in ./.venv/lib/python3.9/site-packages (from scikit-learn==0.24.1->label-studio-ml) (1.1.0)
Requirement already satisfied: scipy>=0.19.1 in ./.venv/lib/python3.9/site-packages (from scikit-learn==0.24.1->label-studio-ml) (1.8.1)
Requirement already satisfied: numpy>=1.13.3 in ./.venv/lib/python3.9/site-packages (from scikit-learn==0.24.1->label-studio-ml) (1.22.4)
Collecting charset-normalizer~=2.0.0
  Using cached charset_normalizer-2.0.12-py3-none-any.whl (39 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.9-py2.py3-none-any.whl (138 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2022.6.15-py3-none-any.whl (160 kB)
Collecting idna<4,>=2.5
  Using cached idna-3.3-py3-none-any.whl (61 kB)
Using legacy 'setup.py install' for attr, since package 'wheel' is not installed.
Installing collected packages: attr, appdirs, werkzeug, urllib3, Pillow, MarkupSafe, lxml, itsdangerous, idna, colorama, click, charset-normalizer, certifi, attrs, scikit-learn, requests, label-studio-tools, Jinja2, Flask, label-studio-ml
  Running setup.py install for attr ... done
  Attempting uninstall: scikit-learn
    Found existing installation: scikit-learn 1.1.1
    Uninstalling scikit-learn-1.1.1:
      Successfully uninstalled scikit-learn-1.1.1
Successfully installed Flask-1.1.2 Jinja2-3.0.3 MarkupSafe-2.1.1 Pillow-9.1.1 appdirs-1.4.4 attr-0.3.1 attrs-21.4.0 certifi-2022.6.15 charset-normalizer-2.0.12 click-8.1.3 colorama-0.4.5 idna-3.3 itsdangerous-2.0.1 label-studio-ml-1.0.7 label-studio-tools-0.0.0.dev11 lxml-4.9.0 requests-2.28.0 scikit-learn-0.24.1 urllib3-1.26.9 werkzeug-2.0.2

What is my understanding
I downloaded and extracted the label-studio-ml-1.0.7.tar.gz file and searched for the usage of scikit-learn or sklearn and no results were found, except a mention in the label_studio_ml.egg-info/requires.txt file, where we can find the scikit-learn==0.24.1 dependency. It seems to indicate that the scikit-learn is not necessary to the label-studio-ml dependency.
Related issues/pull requests
I have found the following issues/pull requests related to this issue:

#107

How can I contribute
Please let me know how can I contribute to this repository and fix this issue; I would happily do it.

Hi @ludelafo
I have released ML backend version 1.0.8 without scikit-learn dependency, you can use for your task.


slack: deploy bot to generate invite links

What were you trying to do?
What did you expect to see?
What did you see?
How can we reproduce the problem?

There are a few projects which might help us here.
The most promising is this blog and linked project.

https://intoli.com/blog/make-a-public-slack-community/
https://github.com/outsideris/slack-invite-automation

Otherwise, I think the Gophers slack operates a service which handles invite requests at https://invite.slack.golangbridge.org/ and their slackbot is hosted on GitHub:

https://github.com/gobridge/gopher

Can we leverage micro's invite binary from master and host a redirect for ourselves?
Wade also found a link that works for 2k invites, so we can use that for a bit.

We're going to deploy https://github.com/rauchg/slackin

https://slack.moov.io/ is up and running now.


Fix natp implementation. Add pairs.

This fixes #1, and allows you to try out the examples in chapters 9 and 10.

@holguinj any chance you want to merge this into master?

Sorry about the delay! Thanks a lot for fixing this, @sritchie.


[63] Issue #63 - March 30, 2017

Slava voiced some concerns about Swift Evolution, and I wrote a blog post about it: https://basthomas.github.io/swift-evolution-without-swift-evolution


CI: Use matrix for metal3 images building (2nd try)

2nd trial for https://github.com/Nordix/metal3-dev-tools/pull/599

/hold

Nice work! This looks great (and I learned some groovy sweat_smile )! /lgtm

Thanks Lennart. I learned a lot as well xD


Add a negative lookahead assertion for log check

Description
The HDF5 retry errors should not generate an alert email.
Interface impacts

None
Testing

Unit tests

[x] No unit tests (for task-schedule code)

Functional tests

The watch_cron_logs3.pl documentation shows an example which indicates this regex and syntax should be valid:
<check>
 	<error>
              #    File           Expression
              #  ----------      ---------------------------
 		*		Use of uninitialized value
 		*		(?<!Program caused arithmetic )Error
 		*		Warning
                 *               fatal
 	</error>
</check>

Ran the following Python script to verify the regex's are good. The unwanted lines with error or warning are not matched, while other error or warning lines are matched.
The text is adapted from an alert email today, with a couple of extra lines added.
import re

text = """
warning: could not open file!
WARNING: open_file(/proj/sot/ska3/flight/data/eng_archive/data/acis2eng/5min/1DEAMZT.h5, mode=a, filters=Filters(complevel=5, complib='zlib', shuffle=True, bitshuffle=False, fletcher32=False, least_significant_digit=None)) exception: HDF5 error back trace

  File "H5F.c", line 620, in H5Fopen
    unable to open file
  File "H5VLcallback.c", line 3501, in H5VL_file_open
    failed to iterate over available VOL connector plugins
  File "H5PLpath.c", line 578, in H5PL__path_table_iterate
    can't iterate over plugins in plugin path '(null)'
  File "H5PLpath.c", line 620, in H5PL__path_table_iterate_process_path
    can't open directory: /proj/sot/ska3/flight/lib/hdf5/plugin
  File "H5VLcallback.c", line 3351, in H5VL__file_open
    open failed
  File "H5VLnative_file.c", line 97, in H5VL__native_file_open
    unable to open file
  File "H5Fint.c", line 1898, in H5F_open
    unable to lock the file
  File "H5FD.c", line 1625, in H5FD_lock
    driver lock request failed
  File "H5FDsec2.c", line 1002, in H5FD__sec2_lock
    unable to lock file, errno = 11, error message = 'Resource temporarily unavailable'
Error Message:
  unable to lock file
End of HDF5 error back trace
"""

lines = text.split('\n')

regex = re.compile("error(?! (back trace|message =))", re.IGNORECASE)
print(regex)
for line in lines:
    if regex.search(line):
        print(line)

print()

regex = re.compile("warning(?!: open_file)", re.IGNORECASE)
print(regex)
for line in lines:
    if regex.search(line):
        print(line)

Output:
$ python check_error_regex.py
re.compile('error(?! (back trace|message =))', re.IGNORECASE)
Error Message:

re.compile('warning(?!: open_file)', re.IGNORECASE)
warning: could not open file!

They are benign because they eventually succeed. If all the tries fail then an exception is raised and the program execution halts giving a non-zero exit status.

OK, is that exception caught with the current checks?

I believe task_schedule will send an alert email if the job fails outright from an uncaught exception. This is no longer the responsibility of watch_cron_job:
https://github.com/sot/task_schedule/blob/128ffa073ce44bc6cd84bf5db144cb2218de0a60/task_schedule3.pl#L265

A recent example is in email with subject "ACA weekly report: ALERT".


TypeError: init() got an unexpected keyword argument 'get_action'

In the docs[1] it says to create authorized_sessionmaker like so
>>> AuthorizedSession = authorized_sessionmaker(bind=engine,
...                                             get_oso=lambda: oso,
...                                             get_user=lambda: user,
...                                             get_action=lambda: "read")

However, in the API docs[2] get_action is not defined, and specifying it throws the exception in the title of this issue.
Wondering what is the correct syntax. If get_action is no longer required, what is the default behavior?
[1] https://docs.osohq.com/reference/frameworks/data_filtering/sqlalchemy.html
[2] https://docs.osohq.com/python/reference/api/sqlalchemy.html

Hey @radiantone, that doc has unfortunately fallen out of date — the get_action parameter has been replaced with get_checked_permissions. I've created a ticket for us to get that updated ASAP; sorry for the hassle!
For now, you should be able to pass a lambda that returns a dictionary instead of a single string, like the following test case:
https://github.com/osohq/oso/blob/434ffbe22c5fc5e8011b1f22d8c07170697f01c2/languages/python/sqlalchemy-oso/tests/test_sqlalchemy.py#L150
If you have any other questions or run into issues getting started with Oso or the sqlalchemy-oso package in particular, please don't hesitate to reach out here or on Slack.

Gotcha! Thank you. This framework is pure brilliance btw!


doc: improve godoc

A small set of GoDoc fixes that adds some instructions on how to import the package, run the examples, as well as a small fix to one of the examples.

Nice one, GitHub! While the PR was auto closed (I never closed actually it), it still got merged (which was intended but GitHub decides not to show that). Ref.: https://github.com/axiomhq/axiom-go/commit/2c9f5b03c524a073b8d280a126c7f4772feb8ab2.


LUN-141: Remove exercise results and sessions from async storage

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword LUN.

What will happen with the storage data that was already created?

Good question, right now it will just be kept. I think that is fine as it shouldn't be a huge amount of data and does not hurt. Other opinions?

And furthermore the color of the label of the Eingabe Überprüfen button seems off

Is this something new? There is already an issue for this (if I understood you correctly): https://issues.tuerantuer.org/browse/LUN-199

@Taggotty if you think this should be handled differently/be removed, please open an issue :)


Application view for subscription application monitoring

on top of https://github.com/red-hat-storage/odf-console/pull/1114

Can we remove the "ApplicationSet" in parenthesis from everywhere? It's not needed I suppose since we are telling users the application type in the view @GowthamShanmugam @TimothyAsirJeyasing

Can we remove the "ApplicationSet" in parenthesis from everywhere? It's not needed I suppose since we are telling users the application type in the view @GowthamShanmugam @TimothyAsirJeyasing

@TimothyAsirJeyasing other PR is removing these

LGTM

/approve
/lgtm

retest /test odf-console-e2e-aws

/test odf-console-e2e-aws

/cherry-pick release-4.15


Home-Manager not working as anticipated

First of all: Great work! Your starter-config helped me to understand nix-flake configuration a little more and your nix-config checked all the boxes for me: encrypted btrfs, impermanence, multiple users and host-machines ans everything in one flake-file.
Thanks to your nix-config I have a running NixOS although I am struggling with the home-manager (not for the first time). I can build and switch with home-manager switch --flake $user@$host but home-manager packages gives me nothing. I remember that I had problems with home-manager before (with a much simpler config) getting sway to work. Do you have any clues, what might be the problem? I basically edited your configuration to my needs.
Thank you in advance
Lutz

Thank you for your kind words! I'm really glad I could help out fellow nixers with my configs :D
Hmm, maybe you might have some kind of issue with the new nix profile? home-manager isn't very compatible with it.
I think you can switch back to nix-env by using rm /nix/var/nix/profiles/per-user/$USER/profile.
Let me know if it works!

Thank you for the fast reply.
Removing of the per-user-profile had the effect, that the home-manager-configuration was build completely (my hopes where getting up). But after completion there is still nothing to be found with home-manager packages. I would really like to use home-manger especially for sway and the cli-config.
If it helps, my configuration is located at nix-config. Feel free to comment on my humble copy of your configuration (at the time I am using Gnome in the host-config as I have to get some things working).
Best Regards
Lutz

Hey!
I'm hmm... Kinda out of ideas here. You're running something like (of course, with your own user and host) home-manager switch --flake .#misterio@atlas? The . is very important, as it tells home-manager to use the repository you're in, not the default path (which is .config/nixpkgs IIRC)

Thank you for your patience. The home-manager switch -b backup --flake .#lgo@flores was build as expected (I have to use -b backup because of the existing firefox-profile. After the build I got:
Starting Home Manager activation
Activating checkFilesChanged
Activating checkLinkTargets
Activating unmountPersistentStoragePaths
Activating createAndMountPersistentStoragePaths
fuse: mountpoint is not empty
fuse: if you are sure this is safe, use the 'nonempty' mount option

The same goes for the nix build-approach.

Yeah, this is an error from impermanence, it's kinda of a pain they don't tell which mountpoint is conflicted haha.
Okay hm you can try commenting out every home.persistence block (or set persistent = false on your flake), check if it works, and then uncomment half of then and so on, until you know what is conflicting.

Hey, that was a partial success. home-manager is working with persitance completly disabled. As soon as I enable it (wether or not something else is commented out or not) I get the described error.
Thank you so far. My system is usable now :)

Glad you got it working :)
Impermanence sure is a tough nut to crack. Enabling one at a time should help!

Thank you for your patience. The home-manager switch -b backup --flake .#lgo@flores was build as expected (I have to use -b backup because of the existing firefox-profile. After the build I got:
Starting Home Manager activation
Activating checkFilesChanged
Activating checkLinkTargets
Activating unmountPersistentStoragePaths
Activating createAndMountPersistentStoragePaths
fuse: mountpoint is not empty
fuse: if you are sure this is safe, use the 'nonempty' mount option

The same goes for the nix build-approach.

Indeed, it's a bit annoying, there is no debugging option in the home-manager mode.
I think that during the home-manager switch phase, if an application writes to one of the impermanence mount folders, that impermanence (fuse) cannot mount the whole impermanence.
It's a shame, because the idea was good, but too restrictive, I deactivated it


New logging view for SplitPDFLog

It should work along with #3074. Feedback would be appreciated

The big comment is that this doesn't show enough information. All fields should be shown for each log object.

Also please do a pull from master.


added remove_chars to util/init.py

I was trying to run this code as in the "Quickstart" section of the docs:
./bmodel model -n 100 -d 20 -g included_data/graphs/AFG_graph_20201014.p
I was encoutering an error that was solved by adding remove_chars to util/__init__.py

Thanks for the PR. We've cherry-picked this fix into master, I'm going to close this PR though b/c we don't want to pull in all the changes from the ocha fork into this repo.

Thanks for the PR. We've cherry-picked this fix into master, I'm going to close this PR though b/c we don't want to pull in all the changes from the ocha fork into this repo.


 doesn't work after c+hop

I figured out that <BS> doesn't work after c+<M-t>, where <M-t> is the map with nvim_buf_set_mark:
vim.keymap.set("o", "<M-t>", function()
  vim.api.nvim_buf_set_mark(0, "<", 16, 21, {})
  vim.api.nvim_buf_set_mark(0, ">", 16, 10, {})

  vim.cmd("normal! gv")
  if vim.fn.visualmode() ~= "v" and vim.fn.visualmode() ~= "" then
    vim.cmd("normal! v")
  end
end, {})

It's been fixed in the neovim https://github.com/neovim/neovim/pull/26648/commits/a07a651ac0066e6f755decb8f48f3f04dd76cfca


I have a question about "Local anchor transfers in Unity"

[여기에 피드백 입력]
The Web page below states that when I export anchor data, you run the "TransferDataToClient" function.
https://docs.microsoft.com/ko-kr/windows/mixed-reality/out-of-scope/local-anchor-transfers-in-unity
Do I need to implement a function(TransferDataToClient) that sends the data to other devices over the network?
Or can I just send the data to another device with the content on that webpage?
문서 세부 정보
⚠ 이 섹션을 편집하지 마세요. docs.microsoft.com ➟ GitHub 문제 연결에 필요합니다.

ID: b01e03ae-bf91-fa69-3085-3a565e88c5ee
Version Independent ID: 42518045-dd41-6405-5a0b-a7e6b5fedb96
Content: Unity의 로컬 앵커 전송 - Mixed Reality
Content Source: mixed-reality-docs/mr-dev-docs/out-of-scope/local-anchor-transfers-in-unity.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @fieldsJacksonG
Microsoft Alias: jacksonf

TransferDataToClient is a function you'd need to implement yourself. That page describes obtaining the data to send but not the exact method of sending the data.

Nice to meet you, Mr/Ms. MaxWang.
Thank you for your kind reply.
Your advise was very helpful to me.
May I ask you a few questions?
Is there a way to see the anchor data file in the hololens on the PC after
connecting the hololens and the PC?
I want to see the capacity of the anchor data file.
And I want to copy the anchor data file to my PC and input it into another
hololens.
Have a nice day.
Thank you.
Best regards,
Thomas.
2021년 4월 15일 (목) 오전 1:31, Harrison Ferrone @.***>님이 작성:

Closed #254 https://github.com/MicrosoftDocs/mixed-reality/issues/254.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/mixed-reality/issues/254#event-4594680973,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATJDRBIYZPNZJLYJVPBKTPLTIW7OHANCNFSM42BVGGGQ
.

Nice to meet you, Mr/Ms. Harrison Ferrone.
Thank you for your kind reply.
Your advise was very helpful to me.
May I ask you a few questions?
Is there a way to see the anchor data file in the hololens on the PC after
connecting the hololens and the PC?
I want to see the capacity of the anchor data file.
And I want to copy the anchor data file to my PC and input it into another
hololens.
Have a nice day.
Thank you.
Best regards,
Thomas.
2021년 4월 15일 (목) 오전 1:31, Harrison Ferrone @.***>님이 작성:

Closed #254 https://github.com/MicrosoftDocs/mixed-reality/issues/254.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/mixed-reality/issues/254#event-4594680973,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATJDRBIYZPNZJLYJVPBKTPLTIW7OHANCNFSM42BVGGGQ
.

Nice to meet you, Mr/Ms. Kurtis.
Thank you for your kind reply.
Your advise was very helpful to me.
May I ask you a few questions?
Is there a way to see the anchor data file in the hololens on the PC after
connecting the hololens and the PC?
I want to see the capacity of the anchor data file.
And I want to copy the anchor data file to my PC and input it into another
hololens.
Have a nice day.
Thank you.
Best regards,
Thomas.
2021년 4월 13일 (화) 오전 3:17, Kurtis @.***>님이 작성:

TransferDataToClient is a function you'd need to implement yourself. That
page describes obtaining the data to send but not the exact method of
sending the data.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/mixed-reality/issues/254#issuecomment-818024099,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATJDRBPWYMJNZ5PSLZL4S7TTIM2MBANCNFSM42BVGGGQ
.


append col and append row for scalars

Summary:
Fix issue https://github.com/stan-dev/stan/issues/1347
Intended Effect:
Add signatures:
 append_row(vector, scalar)
 append_row(scalar, vector)
 append_col(row_vector, scalar)
 append_col(scalar, row_vector)

And unit tests for them. Added special rev-diff test functions to make sure the vectors were being correctly promoted.
How to Verify:
Verify code and test functions.
Side Effects:
None.
Documentation:
To be included in stan/stan-dev
Reviewer Suggestions:

Jenkins, retest this please.

I'm still working through some of the upstream problems... not sure why it's behaving the way it's behaving now.

Jenkins, retest this please.

Jenkins, retest this please.

Jenkins, retest this please.

@randomm @syclik  This one has a conflict and still requires doc for the manual.  I don't know if that's from the new includes or what, but I'd love to have this feature in, as I've needed it a few time in coding models.

[ ] requires LaTeX doc in manual

still requires doc for the manual.

But shouldn't this be done in another pull request in stan-dev/stan?

I don't know if that's from the new includes or what

Exactly! Fixed. But it seems Jenkins has a problem with the doxygen build.

On Mar 22, 2016, at 12:48 PM, Marco Inacio<EMAIL_ADDRESS>wrote:

still requires doc for the manual.

But shouldn't this be done in another pull request in stan-dev/stan?

Yes, you're right.

I don't know if that's from the new includes or what

Exactly! Fixed. But it seems Jenkins has a problem with the doxygen build.

Does the doxygen build work on your machine?

Bob

Does the doxygen build work on your machine?

Yes, it also build on Jankins machine too, but is giving some odd git
related errors:
http://d1m1s1b1.stat.columbia.edu:8080/job/Math Pull Request - Doc - Doxygen/245/warnings14Result/source.555/#1
http://d1m1s1b1.stat.columbia.edu:8080/job/Math Pull Request - Doc - Doxygen/245/warnings14Result/source.554/#1

/usr/bin/git rev-list 3ef75d9b54fcc1c36d96ca57df3f86367943351c # timeout=10
warning: inexact rename detection was skipped due to too many files.
warning: you may want to set your diff.renameLimit variable to at least 10631 and retry the command.

Looks good.


RBAC - add oauthbearer auth mechanism

what
Enable support for OAuth Bearer authentication using JwtLoginService.
why
C3 needs this for RBAC. Rather than only doing this inside C3, why not put it inside rest-utils too?
how
authentication.method=OAUTHBEARER
authentication.oauthbearer.issuer=Confluent
authentication.oauthbearer.public.key.path=/tmp/keys/public.pem
authentication.oauthbearer.roles.claim=clusters

@confluentinc It looks like @brianbushree just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

closing since we don't need this at the moment.

reopening since actually we do...
either we need https://github.com/confluentinc/rest-utils/pull/139 OR this.

@brianbushree I am not sure, how to utilize these changes in open source rest projects like Kafka-rest, ksql etc. We can not add proprietary implementations to open source projects.
https://github.com/confluentinc/kafka-rest/blob/master/kafka-rest/src/main/java/io/confluent/kafkarest/KafkaRestApplication.java#L47

@omkreddy You're right. This is mainly for use with C3 and any other proprietary projects using rest-utils.

@brianbushree if we're not going to provide an implementation for this particular method, how about we allow arbitrary authentication method string? i.e. instead of hardcoding this particular string without an implementation, let an implementation decide whether a given auth method string is valid and provide hooks to construct the necessary auth bits for a given string.

spoke offline with @xvrl . As suggested, I think a good follow up to this PR would be to let users add arbitrary auth mechanisms to work with pluggable authenticators/login-services


Add pugixml

Hi,
pugixml is light-weight, simple and fast XML parser for C++ with XPath support
https://github.com/zeux/pugixml
CMake:
https://github.com/zeux/pugixml/blob/master/CMakeLists.txt
Regards

I created pull request
https://github.com/ingenue/hunter/pull/95

I created pull request

@rusdevops Please reopen it in https://github.com/ruslo/hunter

@ruslo, I reopened pull request:
https://github.com/ruslo/hunter/pull/902

@ruslo, I reopened pull request:
#903


neon-dashboard problems

I'm seeing some dashboard related problems:

[x] I'm seeing TLS connection issues when showing the dashboard immediately after cluster setup.
I'm seeing this in the browser and also in Postman, but after waiting a few minutes, the dashboard displays a blank page.  Might this be due to cluster setup completing before cert-manager was able to do its thing with LetsEncrypt?

[x] When the dashboard does display, it's entirely blank, the UX for selecting dashboards is not displayed at all in the browser, so it's not possible to select any dashboards.  Perhaps @codelastnight has disabled this?

[x] This is a reminder that we're going to make the Kubernetes Dashboard the default for the time being.

We need these resolved very soon.  I won't be able to do the demo video without at least the second one is fixed and I'm not sure we want to release this to anybody before the first and third ones are fixed.  Having the dashboard display a TLS error in the browser after cluster setup won't be a create first impression.
Related: #1459

Closing: It looks like @marcusbooyah has already made the last required changed


Try to return headers during fast sync

What is wrong?
During fast sync we respond to all GetBlockHeader messages with an empty array of headers.  We however, may have the requested headers in our database.
How can it be fixed
We should try to return the headers that were requested.

See https://github.com/ethereum/py-evm/issues/587#issue-317154937 for details on exactly how geth handles this

I think the fix in #847 will not be enough -- geth will still drop us if we always reply with an empty list

It at least takes them a little longer to drop us based on my experiments.  In general though, yes, we should probably prioritize implementing this as it will improve sync speed by reducing our peer churn.


[CodeCompletion] Dont mark type mismatching items 'not recommended'

func foo() {}
let a: Int = #^HERE^#

Previously, we marked foo() as NotRecommented because Void doesn't have any member hence it cannot be Int. But it was confusing with deprecated.
Now that we output 'typerelation' which is 'invalid' in this case. So clients can de-prioritize them, or even filter them out.
rdar://problem/57726512

@swift-ci Please smoke test

@swift-ci Please smoke test

Have you considered exposing the NotRecommendedReason instead and letting the client decide?

Yes, but I don't think the reason should have TypeMismatch even if we expose it. We already exposed the type relation which is "invalid" in this case, so "Not recommended for type mismatch" would be a duplicated information for clients.

Okay, I don't have a strong preference and I'm okay with this approach.  I would prefer we cleanup the internal API as well though.  I'm not clear why you think that would be a big change?  CodeCompletionOrganizer already uses the type relations, so I don't see why we couldn't handle "invalid" specifically in the places where we currently handle "not recommended".

@swift-ci Please smoke test

Removed TypeMismatch from NotRecommendedReason.
@benlangmuir Could you review this again?

@swift-ci Please smoke test


bump deps for kube 0.60 + fix minor issues

bump deps and, make work with latest + fix crd issue
document the otlp port
keeping it in a branch for a while because i suddenly am failing to get trace ids out of my system (the original method seems to be gone in telemetry.rs)

revamped docs and added some things to speed up local dev (.dockerignore, Tiltfile, smarter make targets).
stuff seems to be working with trace ids again.


CB-2187. Add CCM tunnel registration to cloud-init.

With these changes, all the plumbing has been done for communicating
CCM parameters down through the freemarker templates to the cloud-init
user data script.
Specific changes include:

New model objects for CCM parameters, including server parameters
that represent information required to contact CCM, instance
parameters required to identify the instance to CCM, and tunnel
parameters which represent the tunnels to be registered with CCM.
The ability to pass the CCM parameters from the stack service
through the image service to the user data builder. Note that
with this commit, the CCM parameters are always absent. They
will be populated from communication with the mina management
service in a subsequent commit.
Updates to the freemarker templates for the cloud-init user data
script for both CB and FreeIPA, to do all the work required to
configure and invoke the CCM client scripts.
Logic to add the CCM parameters to the template models for
freemarker template processing. Note that with this commit,
the absent CCM parameters means the ccmEnabled template model
parameter will be absent, and none of the CCM part will be
included in the generated user data. I have tested locally with
populated template model parameters based on hard-coded CCM
parameters, and observed not only correct user data generation,
but actual successful tunneling.
Updates to the freemarker template for the Cloud Formation stack,
and corresponding template model updates in the AWS stack request
helper, to support user data up to the fully allowed 16k, passed
in up to four chunks, each of size up to 4k. The previous mechanism
used only a single parameter, which was limited to 4k in size.
Although the current commit does not increase the user data size,
manual testing revealed that the new keys for CCM pushed us past
the 4k limit, but nowhere near the 16k limit.

@jadair-cloudera Could you please rebase to latest master? The integration-test failure you ran into has since been fixed in #6164 (a couple of commits after the parent yours is currently based on).


QnAMaker-App-Insights Sample App

Sample app to show QnA Maker with Telemetry integrated

@sgellock

At the very least the folder name for this sample needs to conform to the other folder names in the samples/csharp_dotnet folder.  the folder name should be all lower case, and use "-" as word separators.  We don't use "_" underscores or capital litters in the folder names.

I fixed the folder name.


Can you provide lora tutorial for mistral 7b instruction model on custom dataset?

Can you provide lora tutorial for mistral 7b instruction model on custom dataset?

Looking forward to hearing about this soon


Example on calling an optional parameter

Could I kindly ask someone to please share an example of calling one of the APIs with an example optional value using ES6 notation? I'm having a total mind block trying - for example with the histoDay API and limiting the results to 3? Thank you in advance!

Sorry for the false alarm - found an example in one of the test scripts :)

Sorry for the false alarm - found an example in one of the test scripts :)


Trivia: version trivia data

Versioning trivia data is entirely irrelevant now, but it's necessary for the rewrite to upgrade old trivia data automatically after making changes to it that would break the plugin.

This is not necessary...

if (typeof triviaData.version !== 'number')  can be done whenever you actually add an incompatible version (and will need to be done anyway because it's not like you don't need to support versions before the version field existed.

True. I could just do something like this instead:
let version = triviaData.version;
if (!version) {
    // reformat questions, submissions, leaderboard, delete ladder
    version = 1;
}

// verify the rest of triviaData is correct

return {


Column ReportReferenceLink Description Change

Hello,
It seems that the column 'ReportReferenceLink' under the Malicious IP (https://docs.microsoft.com/en-us/azure/azure-monitor/vm/vminsights-log-search#malicious-ip) section states 'Linked to reports related to a given observable.'
Upon further investigation the URL that populates this field points to an MS internal site that can be provided by the customer to MS Support in case the customer wants to get more details about the Malicious IP. Can we update this documentation to reflect that for this field as it can be misleading if the URL shows up in the customers table but they are unable to reach it?

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

ID: 0a10be9a-2be1-9db1-5603-777ab82ce850
Version Independent ID: 9ab994c5-1b39-d9db-c2c7-1dc4df31b25c
Content: How to Query Logs from VM insights - Azure Monitor
Content Source: articles/azure-monitor/vm/vminsights-log-search.md
Service: azure-monitor
Sub-service: virtual-machines
GitHub Login: @bwren
Microsoft Alias: bwren

@bdegs
I have assigned the issue to the content author for further review.

Thanks for this and sorry for late reply. Updating article now.
#please-close


(fixes #168) Delete appstores files

I think that were all the instances left 🤞

Added a couple more leftovers related to App Stores (but not strictly to these docs).


TumblingWindowTrigger struct is missing the Annotations field

Ported from https://github.com/Azure/azure-sdk-for-go/issues/4147
See type defined here:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/entityTypes/Trigger.json#L333

The annotations have been added to Trigger base class:
https://github.com/Azure/azure-rest-api-specs/blob/master/specification/datafactory/resource-manager/Microsoft.DataFactory/stable/2018-06-01/entityTypes/Trigger.json#L27


Can't find the files ‘gee_county_lc.py’ and ‘gee_landcover.py’

Hi，
I‘m tring to run your code. However, I couldn't find the files ‘gee_county_lc.py’ and ‘gee_landcover.py’ to prepare the data. Could you provide those two files? Or, could you provide some example data?

same question.I have been repeating this work recently. Can you give me some advice


add tests

supported/unsupported chart types
with convertToExcalidrawElements()

@dwelle @ad1992  I miss these tests, especially with the fixes. Do you have an some example of how this could work?

@igorwessel let's set up vitest first and then we can start adding tests

I have added vitest and some tests so going forward we will be adding tests


  Is it ok to submit them as a single pull request?

This is looking good. However, could you remove the split definition? @nshahpazov made a more comprehensive version which is merged just now.
I would love to see some tests too.

@ccapndave can you recommit your contribution? If you don't have time, I can do it for you too, although I'm not sure if git remain seeing you as the contributor.

So sorry, had a deadline for the end of the month and now am on my way to a conference :) I'll resubmit shortly (I won't have time to write any tests unfortunately)

Great, I'll push some tests afterwards.

There you go!  I've taken out the split commit and added two extra ones.  Let me know if its ok!


The image should run with a non root user ?

Today I requested my company's security team to allow me to use the slim-bullseye image for a project to be run on Openshift.
They denied the request because Twistlock scan have detected an issue:
Compliancy:   failed:  the image should run with a non root user
Well, I was not able to argue against it.

I agree wholeheartedly that all users of this image should be running it with a non-root user!  However, we cannot set a non-root user by default because it makes it much harder for users to build images based on this one, and also brings up the question of which user we'd set it to -- 1000?  999?  65536?
In this case, the image itself isn't really all that useful to run by itself, and you'll almost always have to include some amount of your own application source code inside it (because it's just the raw interpreter / package manager for Python by default), so you'll likely write yourself a short Dockerfile for your application which installs any necessary dependencies, copies in your application, and that would then be the appropriate place for you to choose/set a specific user to run your derived image as.
For further help/advice, I would suggest trying a dedicated support forum, such as the Docker Community Slack, Server Fault, Unix & Linux, or Stack Overflow.


Make trigger importable

Hi @itowlson,
@leonardpahlke added the SQS trigger to the spin containerd shim as an example of how to add a trigger: https://github.com/deislabs/containerd-wasm-shims/pull/192. To import the SqsTrigger we had to create a lib.rs and move all the functionality except the main function there.
Is this something you can see in the scope of this crate?

@0xE282B0 Totally in scope.  I had been meaning to do this to the timer sample as well so that I could do a post on multiple trigger types in Lepton; we should establish this as the pattern for custom triggers going forward.  Thanks for tackling this!

Hey @itowlson, could we merge this PR & cut a new release so we can update the dependency in the containerd shim PR https://github.com/deislabs/containerd-wasm-shims/pull/192 🙌

@leonardpahlke Sorry about that!  I'll get a release under way.

@leonardpahlke Released v0.5.0 https://github.com/fermyon/spin-trigger-sqs/releases/tag/v0.5.0

Awesome! I will update the shim PR accordingly


Fix clang incompatible function type error.

Resolves https://github.com/Azure/azure-uamqp-python/issues/386:
uamqp/c_uamqp.c:87603:146: error: incompatible function pointer types passing 'void (void *, enum MESSAGE_RECEIVER_STATE_TAG, enum MESSAGE_RECEIVER_STATE_TAG)' to parameter of type 'ON_MESSAGE_RECEIVER_STATE_CHANGED' (aka 'void (*)(const void *, enum MESSAGE_RECEIVER_STATE_TAG, enum MESSAGE_RECEIVER_STATE_TAG)') [-Wincompatible-function-pointer-types]
  __pyx_t_1 = ((struct __pyx_vtabstruct_5uamqp_7c_uamqp_cMessageReceiver *)__pyx_v_receiver->__pyx_vtab)->create(__pyx_v_receiver, __pyx_v_link, __pyx_f_5uamqp_7c_uamqp_on_message_receiver_state_changed, ((void *)__pyx_v_callback_context)); if (unlikely(!__pyx_t_1)) __PYX_ERR(7, 23, __pyx_L1_error)

The issue seems to be that ON_MESSAGE_RECEIVER_STATE_CHANGED is defined as:
typedef void(*ON_MESSAGE_RECEIVER_STATE_CHANGED)(const void* context, MESSAGE_RECEIVER_STATE new_state, MESSAGE_RECEIVER_STATE previous_state);

where context is const void* and not void*. I'm not sure whether context really needs to be const since even in upstream code it's cast to something else and its state modified, but we already cast in this function so we can just adopt the same signature.

Hi @scbedd and @kashifkhan,
I understand that this package is deprecated, but is there possibility to merge this and release another patch version until this is formally archived? From the linked issue, it looks like it can resolve some pain.