Add a unit test for challenges sync

In #4808 more functionality was added to runScheduler function and I think we should add a unit test for it.
This function is run periodically to add 'Processing' status to unprocessed challenges. This mechanism allows us to limit the number of challenges being processed concurrently.
We could probably mock the scheduler by creating a custom fake for it, similarly to how it was done for acme client and fake what challenges are returned from scheduler.
The existing test controller building mechanism could be used like we do in other tests i.e in issuing controller to test what actions and events are applied.
The test cases could be:

A finalizer gets added and status gets set to processing in for a challenge that doesn't have any finalizers yet, an event gets thrown
A finalizer gets appended and status gets set to processing gor a challenge that has a random finalizer, an event gets thrown
Status gets set to processing, but no finalizer if there already is the ACME finalizer, an event gets thrown

I would like to try it.
/assign

Thanks for interest in contributing to cert-manager @lonelyCZ !
Do reach out if you have any questions.

Hi, @irbekrm , do we need to create two new main file to complete it?

pkg/controller/acmechallenges/scheduler/fake.go
pkg/controller/acmechallenges/controller_test.go

Hi @lonelyCZ I think that will make sense yes, pkg/controller/acmechallenges/scheduler/fake.go for the fake scheduler and pkg/controller/acmechallenges/controller_test.go for the actual test 👍🏼


Add images: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-add-images?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-add-images#2
In which task and step of the codelab can this issue be found?
[Add images to your Android app]>[Add an Image composable]
Describe the problem
import androidx.compose.ui.res.painterResource
is missing.
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @0nafiz0
Thank you for writing in and bringing this issue to our attention.
The lesson has been updated to address it and the revised lesson will be republished again in the near future.
Hope you are enjoying the course!
Regards.


A.19.IR22.ds.conformity

This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
The reference should be 2.8 and not 2.8.1
Proposed Change
Check and change.

Proposed Resolution
Reference to both 2.8 and 2.8.1 is needed.
Requirement is stated in 2.8 but context is provided in 2.8.1


Archicad mesh element receive

Solves this: https://speckle.community/t/cant-receive-in-archicad/4906

works as expected


XP cant be got in player battles sense last restart.

please tell me this is a bug im about to cry.

Unfortunately this is due to gym battle implementation. We do not want gyms leveling up people who forgot to lock their Pokemon levels. This is also in agreement with the Pokemon online trainer battling.
I also do not think abusing pvp for exp for your Pokemon is something we should have on the server, especially with healing over time, players could kill their opponents full team of high levels have their opponent walk a bit, get the team up to 1hp each, and then rebattle for quick exp.


As a developer i would like the CSVDatabase to create a new csv file if none is present.

User Story
As a developer i would like the CSVDatabase to create a new csv-file if none is present.
Description of the solution you'd like
The CSVDatabase should create a new csv file if none is given with the path argument. This should combat the problem with the single file executables not working with csv file.
Definition of Done

[ ] The program should run even if no path is found or given. This made possible by the csvdatabase creating new files.

Giver den et kig


UI for disabling / editing scans

@cablej Can you go into this a little more?

This will be functionality to allow a global admin to edit scans rather than deleting / recreating at present. Since it’s just on the global admin side I don’t think it’s too high priority for the time being.


fixing registering Doctrine MongoDB ODM Annotations

Made this small modification to solve the issue #4 I opened.
It's fixing the problem for me but I don't know if it should have been done another way.

Apart from that I guess this is good, @kryysler ?

Ok, I just changed that then @Jalle19

Working like a charm with my lumen project

Thanks for this @Gog0.


Live caption don't work

Description
Live caption don't download nor work even after enabling the setting.
Steps to reproduce the behavior:

Go to chrome://flags and enable #enable-accessibility-live-caption
Relaunch the browser
Go to settings -> advanced settings -> accessibility and enable the live captions
Nothing happens, chromium doesn't download the subtitles and they don't show up.

Expected behavior
Chromium should download the subtitles and allow live captions to work.
Environment (please complete the following information):

OS/Platform and version: Windows 10
ungoogled-chromium version: 89.0.4389.114-1

Additional context
It's worth noting that live captions aren't a privacy threat at all, because they work fully offline.

Probably because of what mentioned in brave/brave-browser/issues/15864 and domain substitution. If so then not likely to work unless the download link is whitelisted in domain substitution.


The app crashes upon opening the subtitle settings

Crash Logs

After looking through my fonts folder (1,245 fonts), I found that these two fonts are causing the app to crash.
Fonts


setup email client

[ ] need to upgrade Heroku to include credit card information

@BreakPointMinistries send message over slack to setup a screensharing session

If you would like me to set it up with my cc info, just shoot me a message on slack to tell me where to go.


It doesn't recognize two line license plate

Hello,
I have been using Javaanpr libraries in my final project but i can find a way  to customize it so as it can  recognize two lined number plates,
please help

Can you please be more specific? Do you have some problems with Webcam Capture API or rather with JavaANPR itself?

Most of my Country's license plates are two lined(2 rows) so the javaANPR
fails to recognize the plate, need assistance on how to modify the matrices
so as it can recognize these plates
On May 3, 2016 12:12 AM, "Bartosz Firyn"<EMAIL_ADDRESS>wrote:

Can you please be more specific? Do you have some problems with Webcam
Capture API or rather with JavaANPR itself?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/sarxos/webcam-capture/issues/457#issuecomment-216365774

JavaANPR itself
On May 3, 2016 12:50 AM, "uforo Henry"<EMAIL_ADDRESS>wrote:

Most of my Country's license plates are two lined(2 rows) so the javaANPR
fails to recognize the plate, need assistance on how to modify the matrices
so as it can recognize these plates
On May 3, 2016 12:12 AM, "Bartosz Firyn"<EMAIL_ADDRESS>wrote:

Can you please be more specific? Do you have some problems with Webcam
Capture API or rather with JavaANPR itself?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/sarxos/webcam-capture/issues/457#issuecomment-216365774

In that case I suggest you to open new ticket in JavaANPR project (https://github.com/oskopek/javaanpr) since this one here, where we are discussing, is Webcam Capture API which is not related to licence plates at all.
Actually this is the first time I heard about JavaANPR :)

Thanks very much, i will keep in touch i if i will enconter some
difficulties.
Well i found it from sourceforge site.
On May 3, 2016 12:56 AM, "Bartosz Firyn"<EMAIL_ADDRESS>wrote:

In that case I suggest you to open new ticket in JavaANPR project (
https://github.com/oskopek/javaanpr) since this one here, where we are
discussing, is Webcam Capture API which is not related to licence plates at
all.
Actually this is the first time I heard about JavaANPR :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/sarxos/webcam-capture/issues/457#issuecomment-216377052

Ok, take care!


openfl.net.FileReference error

Hi. I have some error, when I try to open file dialog.
Haxe: 3.2.1
Neko 2.1.0
Haxelib:
   actuate: 1.8.6 [1.8.7]
   hxcpp: 3.2.205 [3.3.49]
   openfl-html5: [1.0.5]
   openfl-native: [1.4.0]
   openfl-nativedialog: [1.2.0]
   openfl-ouya: [1.0.2]
   openfl-samples: [4.0.0] 3.3.1
   openfl: 4.0.3 [4.1.0] 3.6.1
   lime-samples: [2.6.0]
   lime-tools: [1.5.7]
   lime: 2.9.1 3.0.3 [3.1.0]

OS: Linux Mint 18 x64
Main.hx:
package;

import openfl.events.*;
import openfl.display.*;
import openfl.net.FileFilter;
import openfl.net.FileReference;

class Main extends Sprite {
    public function new () {
        super();
        this.onLoadProjBtnClick();
    }

    function onLoadProjBtnClick () : Void {
        var loadRef:FileReference = new FileReference();
        loadRef.browse([new FileFilter('XML', '*.xml')]);
        loadRef.addEventListener(Event.SELECT, function(e:Event) cast(e.target, FileReference).load());
        loadRef.addEventListener(Event.COMPLETE, function(e:Event) trace('selected: ${e.target.data}'));
    }
}

If target is neko, then I have error after launching compiled program:
An exception occured in a neko Thread :
./lime@lime_file_dialog_open_file

If target is cpp, then nothing happens: have no errors, but and the dialog doesn't displays.
Thanks.

In flash, the file reference object has to be a class-level variable. If it's a local variable like yours, the variable loses scope once the function ends, and the request is not followed through.
Maybe the behavior is the same on the other targets.

Ok, I changed example (if I correctly understand):
package;

import openfl.events.*;
import openfl.display.*;

import openfl.net.FileFilter;
import openfl.net.FileReference;

class Main extends Sprite {
	var loadRef:FileReference = new FileReference();

	public function new () {
		super();

		loadRef.browse([new FileFilter('Stablexui XML', '*.xml')]);
		loadRef.addEventListener(Event.SELECT, function(e:Event) cast(e.target, FileReference).load());
		loadRef.addEventListener(Event.COMPLETE, function(evt:Event) trace('selected: ${evt.target.data}'));
	}
}

But this changes have no effect. I have error
An exception occured in a neko Thread :
./lime@lime_file_dialog_open_file

Ok that's an actual error then it seems.

I forgot --
Native file dialog is disabled on Linux right now, we have only GTK support in the sub-library we use for dialog boxes, but the method of including GTK references to build it seemed bad for running on a system that might not have GTK... or a different version of GTK. I do not want to have a problem with Lime projects not running at all on systems that don't match this suitably.

Which version needed for this functions? Mate (DE of Linux Mint) based on Gtk2

We just replaced our FileDialog code in Lime, it worked for me in Linux now :smile:

many thanks!


docs: BYOB documentation

Part of https://github.com/slsa-framework/slsa-github-generator/issues/1574.
There are several TODOs left in the doc. We will address them in follow-up PRs

Thanks @ianlewis for reading thru the entire text!


cmd/internal/obj/x86: TestOclass/386 broken on solaris-amd64-smartosbuildlet

The test fails consistently. An example run (https://build.golang.org/log/5a4d651e6c0f6c765aadbfaed8ea393096f71e63):
--- FAIL: TestOclass (0.00s)
    --- FAIL: TestOclass/386 (0.00s)
        asm_test.go:205: oclass("$(SB)"):
            have: 11
            want: 9
FAIL
FAIL	cmd/internal/obj/x86	2.286s

This test was added in CL 175357 and has been failing ever since (CC @quasilyte @cherrymui @TocarIP).

I'll fix it in a few mins.


🐛 [ISSUE]: Can't see my profile

Description
Recently my PR got merged but I'm unable to see my profile on links.dev, My username is parthsharma
Screenshots
No response
Additional information
No response

It's now visible


Insertion Sort

Enter your question -
The concept behind Insertion Sort divides the range into subarrays that are sorted and unsorted. The classified portion is at the start of duration 1 and matches the first (left side) component in the array. We move through the array and expand the classified part of the array by one component during every iteration.
Enter link to the question(if question belongs to any online platform) -
N/A
Tags for the question(eg - Array, Basic, Stack, etc.) -
array,string

@neelshah2409,
Kindly add your solution to "Sorting" folder
Deadline - 03/10/2021

"Kindly assign this issue to me"

@RAshid602,
You may create a new issue "Add a question" and claim it.
Till the deadline it remains with the assigned.

@RAshid602,
Kindly create a new issue "Add a question" and claim it.
Till the deadline it remains with the assigned.


Translation quality gate

Motivation
Translations can be forgotten or "fuzzy" (bad quality).
Proposed Solution
Assure that all translations are provided by adding a respective job to our CircleCI pipeline

Along the way, exclude translation binary files from version control and create 2 jobs for CI/CD: build-translation and compile-translation.

duplicate see #264


node: update flaming fir chain spec

Updates the flaming fir chain spec so that we can restart the testnet using latest master. The authority ids for Babe were updated since they now refer to sr25519 keys (previously using ed25519 in Aura).
Merge only after #3246 and #3251.

cc @ddorgan

We could also use this opportunity to change the network keys from secp256k1 to ed25519.

@tomaka Apparently we are already using ed25519 network keys. Probably did it last time because --node-key-type default changed.

Don't merge this afterwards, I want to rebase this over master to update chain spec (to make sure code storage entry is actually for the latest runtime that will end up in master at this commit).


Cypress: Adds ability to run and open integration test suite locally

Ticket #1066
Description
This PR does a few things:

Installs Cypress package
Allows user to yarn run cypress:run and yarn run cypress:open
Scaffolding to write new cypress specs

What the PR does not do (yet):

Integrate Cypress tests into a github workflow when PRs are opened.
Documentation on how to use Cypress

Screenshots / Demo Video
Testing
Automated and Unit Tests

[ ] Added Unit tests

Manual tests for Reviewer

[ ] Added steps to test feature/functionality manually

Checklist

[ ] Provided ticket and description
[ ] Provided screenshots/demo
[ ] Provided testing information
[ ] Provided adequate test coverage for all new code
[ ] Added PR reviewers

@swatisqa1 I'll add some documentation here tomorrow. But I wanted to bring this to your attention as the cypress package is working well in this branch and I'm able to run a sample test.


Validation not implemented for HashMaps

Other core and std types have validation implemented, but HashMaps need it too. Validation should include claiming the control and data, then iterating over the items and validating each one.
It shouldn't be too hard to do this using RawIter. The control bytes need to be checked to make sure the number of set bits matches the number of items in the HashMap. This should be possible to do while iterating over the elements of the HashMap.

Fixed by 9c5f2c713f24520f01e367df4eff91a3b773fa55


Index all in roughly the last updated order

Fixes #729

Looks pretty good except for the first few entries: https://custom-elements-staging.appspot.com/elements
Not sure if they don't have the first few don't have updated fields on staging and hence are out of order. vaadin down looks good


Data Export settings not saved between executions.

Prior versions (< 6.2.0) would save the last values used in data export dialogs. 6.2.1 appears to forget them and supply the default values only. I have looked to see if config changes had been done and if there was a way to set the defaults, but I have not found any.
Version : <IP_ADDRESS>909221630
Database: PostgreSQL

System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

The export feature in the previous version 6.2.0 was still ok.
I reinstalled 6.2.0 and it seems that 6.2.1 does indeed save the CSV export settings (I can see the changes from 6.2.1 in 6.2.0), but it doesn't use them when reopening the export form.
Another bug in 6.2.1 is that the CSV Export always uses quotes, even when "Quote always" isn't checked. So there is no way to only quote when required.

Duplicate of #6909 (it was created later than this ticket but I missed this one).
Fix will be dded in 6.2.2


Documentation on how to install plugins on K8s deployment

Is your feature request related to a problem? Please describe.
I cannot find instructions on how to install plugins when deployed on k8s (Helm), how to do the restart?
Describe the solution you'd like
A page in the documentation website showing the steps of "installing a plugin" and restarting opensearch pods.

Found the settings in the helm values.yaml
.Values.plugins.installList


User display issue

Hello,
Thanks for a great module!
I have integrated Hijack into my app and for the most part it works great the only issue I have is that it doesn't display the user, we only get the  "You are currently working on behalf of". and the release button.
We are using Django 1.11, any ideas?
Thanks
Errol

Ok I was able to solve it:
django.template.context_processors.request was missing


header--seemed ?

In the material.css there is
    .mdl-layout__header--seamed {
         box-shadow: none;
    }

It seems to me in the docs the mdl-layout__header--transparent has been written twice instead of mdl-layout__header-seemed for no shadow ? Maybe Add it I am also mentioning this issue on mdl page

Seems they updated the transparent for seamed in this PR: https://github.com/google/material-design-lite/pull/1703

:D


Allowing https://github.com/Microsoft/Docker.DotNet to connect to a swarm cluster

Hi guys
I am experience abit with using https://github.com/Microsoft/Docker.DotNet to run some workloads in my ACS swarm cluster.
I discovered that by default this is not possible: http://stackoverflow.com/questions/42170378/what-is-the-proper-endpoint-for-connecting-to-azure-container-service-swarm-clus/42170719#42170719
Just to see that i could connect, i edited sudo vim /etc/init/docker.conf and forwarded the ports in the load balancer.   Added DOCKER_OPTS='-H tcp://<IP_ADDRESS>:4243 -H unix:///var/run/docker.sock'
Now this works, but of cause is not secured. Also, this exposes the docker master for which swarm is running on, and not the swarm master. What would be needed to expose the remote api for the actually swarm cluster instead?
Since I am out of my core domain here, i was wondering if someone here has the expertise to verify that this would be a good starting point: http://tech.paulcz.net/2016/01/secure-docker-with-tls/
Are any of the certificates that was used to SSL setup somewhere on the machine and can be reused in some of the steps?
Lets say we figure out how to set it up properly, would there be some files from the acs-engine deployment that could be updated on the master to allow it to call the modified swarm-manager command that is needed with tlsverify when the machiens reboot, such one do not have to log in and do this every time.
docker run -d --name swarm-manager \
    -v /etc/docker/ssl:/etc/docker/ssl \
    --net=host swarm:latest manage \
    --tlsverify \
    --tlscacert=/etc/docker/ssl/ca.pem \
    --tlscert=/etc/docker/ssl/cert.pem \
    --tlskey=/etc/docker/ssl/key.pem \
    etcd://<IP_ADDRESS>:2379

Is this in regards to Swarm or Swarm Mode?
Would you be able to get by with unix or TCP socket forwarding over SSH? You could avoid all of this complexity if so?
Today, we don't generate PKI assets for Swarm or Swarm Mode clusters... so, this will likely be a fairly large modification as you'll need to add PKI assets to the apimodel, and support generating them the same way  we do with Kubernetes, and then get them into the VM via CustomScriptExtension.

Now this works, but of cause is not secured. Also, this exposes the docker master for which swarm is running on, and not the swarm master. What would be needed to expose the remote api for the actually swarm cluster instead?

I don't really follow. It should be the same as issuing the docker swarm commands locally when you attach to the remote socket. I don't understand the distinction being made here or what you are blocked from doing that you expect to be able to do.

cc: @amanohar

Closing since the workaround is to get by with unix or TCP socket forwarding over SSH


[PT Run] Improve logs

Summary of the Pull Request
Logs for PowerToys Run are hard to read. This PR aims to improve their readability. See #9347
What is included in the PR:
Changes to log formatting and how logging functions are called.
How does someone test / validate:
Delete the log for the current day, start PowerToys and use PowerToys Run for a while, then check the log.
Quality Checklist

[x] Linked issue: #9347
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

Good start, but we can clean the log even more, there is data that is not very useful and makes reading the log a pain.
[2021-02-05 17:27:04.0660] [INFO] [Wox.Infrastructure.Stopwatch.Normal] [D:\git\ivan\PowerToys\src\modules\launcher\Wox.Infrastructure\Stopwatch.cs::50]
|ImageLoader.Initialize|Preload images cost <13ms>

[2021-02-05 17:27:04.0732] [INFO] [Wox.Infrastructure.Image.ImageLoader+<>c.Initialize] [D:\git\ivan\PowerToys\src\modules\launcher\Wox.Infrastructure\Image\ImageLoader.cs::74]
Number of preload images is <0>, Images Number: 4, Unique Items 4

[2021-02-05 17:27:04.1697] [INFO] [Wox.Infrastructure.Storage.WoxJsonStorage`1[[Wox.Infrastructure.UserSettings.PowerToysRunSettings, Wox.Infrastructure, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null]].Save] [D:\git\ivan\PowerToys\src\modules\launcher\Wox.Infrastructure\Storage\JsonStorage`1.cs::133]
Saving cached data at <C:\Users\Enrico\AppData\Local\Microsoft\PowerToys\PowerToys Run\Settings\PowerToysRunSettings.json>

The [] after the info (in the first log entry it's [Wox.Infrastructure.Stopwatch.Normal]) is not providing anything we may need, the source file with the line number is more than enough
We can also remove the | at the beginning of the line and as a separator (replace it with a space or -)
|ImageLoader.Initialize|Preload images cost <13ms>


Enum value names (avoiding collisions).

As discovered in #7, the initial schema I had in mind for the enums will not work, because of name collision between enum values of different types:
// IssueState represents the possible states of an issue.
type IssueState string

// The possible states of an issue.
const (
	Open   IssueState = "OPEN"   // An issue that is still open.
	Closed IssueState = "CLOSED" // An issue that has been closed.
)

// PullRequestState represents the possible states of a pull request.
type PullRequestState string

// The possible states of a pull request.
const (
	Open   PullRequestState = "OPEN"   // A pull request that is still open.
	Closed PullRequestState = "CLOSED" // A pull request that has been closed without being merged.
	Merged PullRequestState = "MERGED" // A pull request that has been closed by being merged.
)

// ProjectState represents state of the project; either 'open' or 'closed'.
type ProjectState string

// State of the project; either 'open' or 'closed'.
const (
	Open   ProjectState = "OPEN"   // The project is open.
	Closed ProjectState = "CLOSED" // The project is closed.
)

...

# github.com/shurcooL/githubql
./enum.go:71: CreatedAt redeclared in this block
	previous declaration at ./enum.go:17
./enum.go:111: Open redeclared in this block
	previous declaration at ./enum.go:8
./enum.go:112: Closed redeclared in this block
	previous declaration at ./enum.go:9
./enum.go:120: CreatedAt redeclared in this block
	previous declaration at ./enum.go:71
./enum.go:121: UpdatedAt redeclared in this block
	previous declaration at ./enum.go:18
./enum.go:149: CreatedAt redeclared in this block
	previous declaration at ./enum.go:120
./enum.go:150: UpdatedAt redeclared in this block
	previous declaration at ./enum.go:121
./enum.go:152: Name redeclared in this block
	previous declaration at ./enum.go:19
./enum.go:170: Open redeclared in this block
	previous declaration at ./enum.go:111
./enum.go:171: Closed redeclared in this block
	previous declaration at ./enum.go:112
./enum.go:171: too many errors

Solutions
These are the solutions that I've got so far and are up for consideration.
Solution 1
Prepend the type name in front of the enum value name, to ensure each identifier is unique and collisions are not possible:
 package githubql

 // ReactionContent represents emojis that can be attached to Issues, Pull Requests and Comments.
 type ReactionContent string

 // Emojis that can be attached to Issues, Pull Requests and Comments.
 const (
-	ThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
-	ThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
-	Laugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
-	Hooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
-	Confused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
-	Heart      ReactionContent = "HEART"       // Represents the ❤ emoji.
+	ReactionContentThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
+	ReactionContentThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
+	ReactionContentLaugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
+	ReactionContentHooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
+	ReactionContentConfused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
+	ReactionContentHeart      ReactionContent = "HEART"       // Represents the ❤ emoji.
 )

Usage becomes:
input := githubql.AddReactionInput{
	SubjectID: q.Repository.Issue.ID,
	Content:   githubql.ReactionContentThumbsUp,
}

This is very simple, guaranteed to not have collisions. But the enum value identifiers can become quite verbose, and less readable.
Solution 2
This is a minor variation of solution 1. The idea is to make the enum values slightly more readable by separating the type name and enum value by a middot-like character ۰ (U+06F0).
 package githubql

 // ReactionContent represents emojis that can be attached to Issues, Pull Requests and Comments.
 type ReactionContent string

 // Emojis that can be attached to Issues, Pull Requests and Comments.
 const (
-	ThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
-	ThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
-	Laugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
-	Hooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
-	Confused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
-	Heart      ReactionContent = "HEART"       // Represents the ❤ emoji.
+	ReactionContent۰ThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
+	ReactionContent۰ThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
+	ReactionContent۰Laugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
+	ReactionContent۰Hooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
+	ReactionContent۰Confused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
+	ReactionContent۰Heart      ReactionContent = "HEART"       // Represents the ❤ emoji.
 )

Usage becomes:
input := githubql.AddReactionInput{
	SubjectID: q.Repository.Issue.ID,
	Content:   githubql.ReactionContent۰ThumbsUp,
}

The unicode character is not easy to type, but easy to achieve via autocompletion:

Solution 3
This is also a variation of solution 1. The idea, suggested to me by @scottmansfield (thanks!), is to use an initialism of the type name rather than the full name. This makes the identifier names shorter, but still has a risk of there being name collisions if two different types with same initialism have same enum values.
 package githubql

 // ReactionContent represents emojis that can be attached to Issues, Pull Requests and Comments.
 type ReactionContent string

 // Emojis that can be attached to Issues, Pull Requests and Comments.
 const (
-	ThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
-	ThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
-	Laugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
-	Hooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
-	Confused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
-	Heart      ReactionContent = "HEART"       // Represents the ❤ emoji.
+	RCThumbsUp   ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
+	RCThumbsDown ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
+	RCLaugh      ReactionContent = "LAUGH"       // Represents the 😄 emoji.
+	RCHooray     ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
+	RCConfused   ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
+	RCHeart      ReactionContent = "HEART"       // Represents the ❤ emoji.
 )

Usage becomes:
input := githubql.AddReactionInput{
	SubjectID: q.Repository.Issue.ID,
	Content:   githubql.RCThumbsUp,
}

Unfortunately, this approach led to a collision with the existing enums in GitHub GraphQL API:
const ROFCreatedAt ReactionOrderField = "CREATED_AT" // Allows ordering a list of reactions by when they were created.

const ROFCreatedAt RepositoryOrderField = "CREATED_AT" // Order repositories by creation time.

./enum.go:149: ROFCreatedAt redeclared in this block
	previous declaration at ./enum.go:71

It's possible to detect when a collision would occur, and use something more specific that doesn't collide, instead of the initialism. But that introduces complexity, and inconsistency/unpredictability in the naming.
Solution 4
This idea is somewhat related to solution 2, but instead of a hard-to-type character, it becomes a normal dot selector. The idea is to create a separate package for each enum type, and define its values in that package. The enum types are still in main githubql package.
package githubql

// ReactionContent represents emojis that can be attached to Issues, Pull Requests and Comments.
type ReactionContent string

// Package reactioncontent contains enum values of githubql.ReactionContent type.
package reactioncontent

import "github.com/shurcooL/githubql"

// Emojis that can be attached to Issues, Pull Requests and Comments.
const (
	ThumbsUp   githubql.ReactionContent = "THUMBS_UP"   // Represents the 👍 emoji.
	ThumbsDown githubql.ReactionContent = "THUMBS_DOWN" // Represents the 👎 emoji.
	Laugh      githubql.ReactionContent = "LAUGH"       // Represents the 😄 emoji.
	Hooray     githubql.ReactionContent = "HOORAY"      // Represents the 🎉 emoji.
	Confused   githubql.ReactionContent = "CONFUSED"    // Represents the 😕 emoji.
	Heart      githubql.ReactionContent = "HEART"       // Represents the ❤ emoji.
)

Usage becomes:
input := githubql.AddReactionInput{
	SubjectID: q.Repository.Issue.ID,
	Content:   reactioncontent.ThumbsUp,
}

The dot character is easy to type, and the code is very readable once written. But this will require importing many new small packages when using enum values. A tool like goimports will make that significantly easier, but it may still be problematic. Also, the documentation will be split up into multiple small packages, which may be harder to read.
Conclusion
All solutions considered so far seem to have certain upsides and downsides. I'm not seeing one solution that is clearly superior to all others. I'm considering going with solution 1 or 2 to begin with, and be open to revisit this decision.

Thanks for the detailed descriptions.  Here are my $0.02:
I like Solution 4 the best, as it seems like the cleanest one to me.
My second-favorite is Solution 3, with the downside of inconsistent initialisms.
My 3rd-favorite is Solution 1, with the downside of Java-like (long) names.
My least favorite is Solution 2, as getting those unicode dots typed can be a pain and frustrating.

I'm not a fan of Solution 4 for two reasons:

Introduces a risk of import cycles, even though githubql may never import reactioncontent, this pattern may be used for other types where import cycles could be a problem.

Although the design shouldn't be determined by the limitations of editors/IDEs, many (but not all: goglang) can only autocomplete for imported packages.

Although Solution 4 is similar to how stripe-go structures itself, so I don't think it's an unusual solution.
I'm not a fan of Solution 3, mostly because the initialisms may have a high chance of collisions, is PStateOpen Project Open or does P mean something else? I believe my reservations on this are similar to @gmlewis's, or I'd also agree with him on the inconsistencies.
Solution 2 (unicode dot) I don't think really brings too much more value over Solution 1, and also it could be difficult to type (although, one could argue it could be a different symbol).
Therefore I think Solution 1 is my favourite, autocomplete works (as the package is probably being imported already), there's no import cycle risk, it's easily discoverable reading docs (godoc.org separates constant blocks well) and just searching works for "reaction" or "thumbs".

I'd go for 3 if I know the different types of enums (and their possible values) upfront, which I guess should be the case here to a large extent. Next (or rather a safer approach though verbose) would be 1. Last option would be 4.
I wouldn't go anywhere near 2; that road is best avoided.

Thanks for the interesting talk at the Berlin meetup this evening!
This is another option for these constants. I would likely not choose it, but I still want to mention it.
package main

var Reactions = struct {
    ThumbsUp, ThumbsDown, Laugh string
}{
    ThumbsUp:   "THUMBS_UP",
    ThumbsDown: "THUMBS_DOWN",
    Laugh:      "LAUGH",
}

func main() {
    print(Reactions.Laugh)
}

If you want to add an extra layer of nesting you can't use the anonymous structs anymore:
package main

type Reaction struct {
    ThumbsUp, ThumbsDown, Laugh string
}
type Colors struct {
    Red, Blue, Green string
}
type ConstsT struct {
    Reaction Reaction
    Colors   Colors
}

var Consts = ConstsT{
    Reaction: Reaction{
        ThumbsUp:   "THUMBS_UP",
        ThumbsDown: "THUMBS_DOWN",
        Laugh:      "LAUGH",
    },
    Colors: Colors{
        Red:   "RED",
        Blue:  "BLUE",
        Green: "GREEN",
    },
}

func main() {
    print(Consts.Reaction.Laugh)
}

@alicebob, thanks for that suggestion. It's an interesting alternative that I haven't thought of, so it's good to be aware of it.
It has some benefits, and feels most similar to option 2, except without the need for hard-to-type unicode middots. However, it has some flaws, like the fact the enum values are no longer consts, and the name collision between the containing struct name and the enum type.

I've been thinking about this issue for a while, and I've decided to go with Solution 1 as a resolution for now, but be open to revisit this issue later. I need to resolve the issue, because the library is not usable in its current state, so I can keep working on implementing next missing features.
Solution 1 is the simplest, most predictable, and least controversial option. Its only downfall is the verbose enum value identifiers, but it suffers from least other problems.
Since the API of the library is not frozen yet, I think it's a good step to make, and if we can find an even better next step to make, we still can. In fact, I won't close the issue for now, we can keep it open and keep thinking.

Some time has passed, and I want to follow up on this.
I've been using our current solution (Solution 1) and continuing to really like it. It's very simple, and feels good to use. The absolutely only downside of slightly higher than expected verbosity pales in comparison to the downsides of the other more involved alternative solutions we've considered above.
One property of the current naming I appreciate is that typing the const name makes it easy to see what are the possible values via code completion:

It'd be helpful if others who have had a chance to use it posted their thoughts too.
However, at this point, I suspect the chance of another solution being discovered which would beat our current simple solution is extremely unlikely. (Of course, if it happens, we can still consider implementing it.) I'll wait some more, and then close this issue.

I'll wait some more, and then close this issue.

I've waited, nothing has come up. Closing now because this issue is considered resolved. (Can always be reopened if something new comes up.)


Add a welcome comment to a new contribution PRs

Status
In Progress
Related Issues
Related: link to the issue
Description
Add a welcome comment to a new contribution PRs.

we have a welcome bot, do we need this?


Cannot read property '$isError' of null

Issue Id: 82d5a163-11cf-c65f-1fb3-88f3c541adbbVersions - 1.18.1Stack TypeError: Cannot read property '$isError' of null/vs/workbench/api/electron-browser/mainThreadErrors.ts#L19:10 ($isError)/vs/workbench/services/thread/node/abstractThreadService.ts#L41:16 (apply)/vs/workbench/services/extensions/node/rpcProtocol.ts#L109:39 (invoke)/vs/workbench/services/extensions/node/rpcProtocol.ts#L96:36 (_invokeHandler)/vs/workbench/services/extensions/node/rpcProtocol.ts#L32:65 (_receiveOneMessage)/vs/workbench/services/extensions/node/rpcProtocol.ts#L157:4 (onMessage)/vs/base/common/callbackList.ts#L36:22 (apply)/vs/base/common/event.ts#L123:26 (call)/vs/base/parts/ipc/node/ipc.net.ts#L82:22 (fire)    at emitOne (events.js:96:13)

Duplicate of #43052


Added region support

@bverbeken I won't send a PR for every client - they work more or less the same as the Java one.

👍


adds repos with names and links

Like it says, this adds dynamic rendering to each repository. I stored it in state, we can easily convert it over when we have data coming in from github.

Looks good


ENH: Make the printed message more readable and fix the type

Reference issue
What does this implement/fix?
Enhance the error/log message
Additional information

Thanks for the contribution @islem-esi. Unfortunately this is a backwards incompatible change so I don't think this can be accepted.


Update index.js

Just included a nicer notation

Hey @santimirandarp, ES6 syntax is not allowed in express source code


Signed-off-by: Mayur Gupta<EMAIL_ADDRESS>

Added Newsletter links as per issue #657

View rendered Community_Roadmap.md

Please change 'LINK' to 'PDF' for links to pdf files.  Also, please remove the checkmarks for the first two newsletters since we don't have them.

Please change 'LINK' to 'PDF' for links to pdf files.  Also, please remove the checkmarks for the first two newsletters since we don't have them.

Hey Mayur,
This PR is good but the title of it should be descriptive of the PR.  Right now, you have put your DCO into the title of the PR.  That should go in the body of the PR and the title should be something like, "Improving links in community roadmap."


Properly correct batch upload/download skip detection documentation

I got this wrong in #1082, see:
https://github.com/github/git-lfs/pull/1082#issuecomment-198308275
I misread the prePushCheckForMissingObjects() logic and, in my local test case, I had the server send back download instead of upload and saw it skip, but that isn't an accurate test because either rule (presence of download, absence of upload) passes it.
To verify this behavior properly, I had the server send back a made up action (quack) instead of upload, and observed that the client skipped the transfer:
$ GIT_TRACE=1 git lfs push origin master
...
trace git-lfs: HTTP: POST http://local.phacility.com/diffusion/POEMS/poems.git/info/lfs/objects/batch
...
trace git-lfs: HTTP: {... "actions":{"quack":{ ... } }}
...
Git LFS: (0 of 1 files, 1 skipped) 0 B / 87.12 KB, 87.12 KB skipped

Thanks!

Doh, thanks for catching that, @sinbad :)


Adds mouse movement to camera

@Brett-Constantinoff  Also the updated Dev branch should be merged into this branch.


Bugfix/lr finder

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure to update the docs?
[x] Did you write any new necessary tests?
[x] If you made a notable change (that affects users), did you update the CHANGELOG?

What does this PR do?
Fixes https://github.com/PyTorchLightning/pytorch-lightning/issues/1648 where the learning rate finder could not be used by users that prefer to pass val_dataloaders directly to the different methods.
Fixed https://github.com/PyTorchLightning/pytorch-lightning/issues/1650 where the learning rate finder could not be used, when a EarlyStopping callback is passed to trainer.
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

The idea behind dumping and restoring trainer args is to take advantage of fit during the learning rate search (alternative would be to write a new training loop). I agree that this is a fragile feature, however for me the clear advantages of calling fit internally outweigh these disadvantages

@SkafteNicki it makes sense to use fit, that's very good :) we just have to keep it in mind when we add new trainer args :) 👍


Add Happo master test

Reason for change
Happo was erring because it was often lacking a master branch screenshot comparison. This builds Happo on every merge to master to fix PR builds
Testing

Checklist

[ ] CHANGELOG: The Unreleased section of CHANGELOG was updated
[ ] Prop changes: docs were updated
[ ] Prop changes: E2E tests were updated, testing from the highest level possible
[ ] Platform testing: If this change should be tested against a platform, has it been?

I’m not completely sure this will fix the problem, but it’s worth a shot! Will merge as an experiment (cuz we can’t know until it merges to master)


Assign mesh holes max depth for Depth Sensor

🚀 Feature

If there is a hole in the mesh during Depth rendering, assign max Depth for Depth Sensor output.
Motivation
Currently, mesh holes are assigned Depth equal to 0 for the Depth Sensor. On the pictures provided, there is a wall close to the camera. There are also the holes around cabinets that should have infinity depth, but their depth values are equal to 0 and are close to the wall in absolute value:

First, that makes a function of Depth to the distance from the camera nonmonotonic. That may affect models training or require post-processing on the Habitat Lab side.
Second, when noise models have applied on top of the frame, it is impossible to distinguish holes from close objects.

Alternatives
Apply noise on Habitat Lab side with Observations Transform.
Additional context
That may affect trained baselines before, but that should be ok to do the change with version switch.

If there is a hole in the mesh during Depth rendering, assign max Depth for Depth Sensor output.

The decision to assign a hole to zero was/is intentional.  That is what real depth sensors do for undefined values as it's impossible for things to actually be at a depth of zero.

I see, we going with emulating real Depth sensors here. @erikwijmans do you know if our current noise model have special handling of such undefined values? Thank you!

The depth noise model doesn't handle zero values differently than any other value, i.e. there is not a special case that guarantees that zero values in the ground truth depth are also zeros in the noisy depth.  The code itself is fine with zeros however.

Thank you for the clarification. Possibly, that can be a configurable feature of the depth sensor. Otherwise the same can be done with the observation transformation on Habitat Lab side with at batch scale.


Add support for aggregations.

This adds support for setting an aggregation. This aggregation is used
for all metrics collection by the exporter. It adjust the resulting
metric names into prometheus recording rule style to allow for
different exporters to collect different aggregations.
We add support for passing the fields, reducer, aligner and align
duration as per other options.
resulting metrics look as follows
'''target_proxy_name_response_code:stackdriver_https_lb_rule_loadbalancing_googleapis_com_https_backend_latencies:sum_bucket{...}```

This is just work in progress to see if you would be interested in this, and if so, to discuss the overall design. Issues with this implementaiton:

The aggregation is applied to the entire exporter, not per metrics prefix (it would make more sense, but the resulting UI could be a big gnarly)
The metrics names aren't quite right, this could be fixed by manuall producing the const histsogram metrics.
Not fixed up the tests as yet.

This is an interesting idea. It looks like the change needs to be rebaesd.
One comment, exporters should not use : in metric names, since this is reserved for recording rules.

@SuperQ so, my thinking here was that the metric is already aggregated so that the names attempt to match what you'd get if you created a recording rule for the same purpose. Other than replacing with an underscore, do you have any other ideas regarding naming?

FWIW, we never used this code, because I didn't really ant to maintain yet another fork in house, but I'd be happy to polish this PR , since it does seem like useful functionality. May not have time in the next two weeks though.

I think we can just close this. I think aggregating in the exporter might be a bit more than we want to support.

After having a discussion on Slack, and looking at some of the use of Stackdriver data we have, this is more useful than I previously thought.

Any interest in reviving this?

I experimented with this a bit, I think there needs to be some changes.
In order for this to be useful generally, we need to supply per-metrics-prefix aggregation requests. Otherwise it will try and apply aggregations to prefixes that don't make sense.
I'm thinking something along the lines of this:
networking.googleapis.com/vm_flow/egress_bytes_count|fields=metric.label.remote_subnetwork;aligner=ALIGN_DELTA;reducer=REDUCE_SUM;period=1m

Part of the problem here, is we still accept ENV vars for input. This makes command line parsing much more difficult, as we need a lot of string separators.
IMO, I think we need to drop the ENV vars, and change the --monitoring.metrics-type-prefixes to a single input, multi-flag.

Sorry for the late response, I'll need to take some time to get my head around this again.

@tcolgate No worries. I had some more thoughts about this, and I think the best way forward would be to move the metrics prefixes to a config file. I've filed https://github.com/prometheus-community/stackdriver_exporter/issues/106 to track this idea.
Basically, it would be more like the balckbox/snmp exporters. Rather than just a list of "metrics prefixes", there would be scrape modules with different prefixes and aggregations.

Hi, @tcolgate have you find another solution to collect stackdriver_https_lb_rule_loadbalancing_googleapis_com_https_backend_latencies. I search to scrape the same metric with prometheus. Cardinalities of this metric don't help us.
Maybe have you found another solution ?
Thank you for your help.

sorry for digging up an old thread. is this still being worked on?


Upgrade to gradle 8.6

Upgrade to gradle 8.6.
Replace classifier (deprecated) with archiveClassifier.
Remove UnnecessarySubstring codenarc rule (deprecated).
Fix for UnnecessaryGString codenarc rule.
Ensure copyNativeLibs runs before any task that implicitly depends on it (required by gradle).

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.55%. Comparing base (222cdb0) to head (28e4d38).

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #101      +/-   ##
============================================
+ Coverage     58.54%   58.55%   +0.01%
  Complexity      134      134
============================================
  Files            34       34
  Lines          3459     3458       -1
  Branches        781      780       -1
============================================
  Hits           2025     2025
  Misses          900      900
+ Partials        534      533       -1

Flag
Coverage Δ

helper
58.55% <ø> (+0.01%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update BackpackScript.lua

Set HotbarFrame.Active to false
HotbarFrame currently blocks a large portion of screen space at the bottom because it is 'Active', but when not active it functions the same and doesn't block lower elements from being interacted with.

Different solution preferred, as seen here: f54d5cbe2b7be5fc272195855e9d53f19224f6fa


Add screenName to providerData to get Twitter handle

Getting Twitter handle is not currently possible with the library, added the handle to the providerData to make it accessible

Could you show me a documentation where the screenName is referenced? I was just looking at the NodeJS Admin SDK and can't find a field screenName, only displayName.

Merged with 74f663cd29b03a79b89ee426cbd57297510cea1c - thank you for your contribution, and sorry it took me so long (I created a Twitter developer account to test this just today 😅)


SW-2239 Use consistent CRS for planting sites

The default coordinate reference system (CRS) for GeoJSON objects is EPSG:4326,
which uses degrees of latitude and longitude. ArcGIS uses a spherical Mercator
coordinate system (EPSG:3857) by default, and we were returning geometry values
in that system which isn't usually what clients need.
Make two changes to the way we handle coordinates:

Store coordinates in whatever coordinate reference system was used in the
shapefiles we're importing.
At read time, transform to EPSG:4326. (In the future, we might allow the client
to ask for a different CRS.)

Current dependencies on/for this PR:

main

PR #794

PR #795

PR #796

PR #797

PR #798

PR #802

PR #803   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

main

PR #794

PR #795

PR #796

PR #797

PR #798

PR #802

PR #803   👈

This comment was auto-generated by Graphite.


Virtual function EditorNode3DGizmoPlugin::begin_handle_action not registered to ClassDB

Tested versions
v4.2.stable.official, v4.3.dev.custom_build [89cc635c0]
System information
Windows 10.0.22621 - Vulkan (Forward+) - dedicated AMD Radeon RX 6900 XT (Advanced Micro Devices, Inc.; 31.0.22017.5002) - AMD Ryzen 7 5800X 8-Core Processor (16 Threads)
Issue description
I'm working on implementing a gizmo plugin similar to CollisionShape3DGizmoPlugin in Godot using GDExtension (godot-cpp) for my custom node.
However, I'm facing an issue where I cannot override EditorNode3DGizmoPlugin::begin_handle_action because it is not registered to ClassDB with GDVIRTUAL_BIND.
Although I've found a workaround in this proposal, I believe it would be helpful to have this function exposed.
Steps to reproduce
Create a class that inherits from EditorNode3DGizmoPlugin and try to override begin_handle_action.
Minimal reproduction project (MRP)
N/A

Have you tried overriding _begin_handle_action?

This is because the correct override isn't exposed, for some reason, will look into a fix

Have you tried overriding _begin_handle_action?

Yes, it is not exposed in the godot-cpp generated EditorNode3DGizmoPlugin class (editor_node3d_gizmo_plugin.hpp).


WDM-KS support in PortAudio from PyPi install

Hello there!
I'm using sounddevice on Windows 10, having installed it with pip.
If you install the sounddevice module with pip on macOS or Windows, the PortAudio library (with ASIO support on Windows) will be installed automagically.

I'm using a Scarlett 6i6 external sound module and it seems that I can only open the MME devices, not the WDS-KS ones.
Using query_devices(), I get:
1 Microphone (USB Audio 2.0), MME (2 in, 0 out)
4 Speakers (USB Audio 2.0), MME (0 in, 2 out)
8 Microphone (USB Audio 2.0), Windows DirectSound (2 in, 0 out)
14 Speakers (USB Audio 2.0), Windows WASAPI (0 in, 12 out)
18 Microphone (USB Audio 2.0), Windows WASAPI (6 in, 0 out
25 Speakers (Scarlett 6i6 USB), Windows WDM-KS (0 in, 12 out)
26 Microphone (Scarlett 6i6 USB), Windows WDM-KS (6 in, 0 out)

If I set sounddevice.default.device =  (USB Audio 2.0), MME, everything works. But if I set sounddevice.default.device =  scarlett , I get
sounddevice.PortAudioError: Error opening Stream: Invalid device [PaErrorCode -9996]
I understand that this error is coming directly from the PortAudio library. Does the PortAudio library for Windows that comes with sounddevice support WDM-KS?
Thank you.

You should check which DLL is actually used, e.g. by looking at sd._libname.

Does the PortAudio library for Windows that comes with sounddevice support WDM-KS?

Yes, see https://github.com/spatialaudio/portaudio-binaries.
The device list contains WDM-KS entries, so it is supported. That doesn't necessarily mean it also works ...

Hi,
I have exactly the same problem on Windows 10, using sounddevice 0.4.2 installed via pip with the following PortAudio version:
(1246976, 'PortAudio V19.7.0-devel, revision 147dd722548358763a8b649b3e4b41dfffbcfbb6')
C:\Users\Manuel\Anaconda3\envs\looper\lib\site-packages\_sounddevice_data\portaudio-binaries\libportaudio64bit.dll'

This issue also occurred on pip's sounddevice 0.4.1 using PortAudio V19.6.
The PortAudio dll from conda-forge does not have this problem (but lacks ASIO support...)
I hope this information helps. If there is any additional information that could help you, please let me know ....

Thanks @blattm! It's good to know that the DLL from conda-forge doesn't have the same problem.
The way the DLLs are created is different, so it is possible that the behavior is different as well.
The DLL bundled with the sounddevice module uses cross-compilation with https://github.com/mxe/mxe to create the DLLs.
Maybe this should be reported to https://github.com/PortAudio/portaudio?

Hi,
So the work around I currently use is to to take the version of sounddevice from pip if I plan to use ASIO, and the version from anaconda when I plan to use WDM-KS. But this requires two virtual environments. In case of distributing a program as an executable (e.g. via pyinstaller), two different executables are required.
However, a potentially better workaround just came to my mind.
@mgeier, do you know if there is a way to manually choose the dll to use?
It would then be possible (for either sounddevice or the programmer using it) to include both dlls and offer an ASIO and WDM-KS mode.
Of course, solving the underlying problem related to MinGW and PortAudio would be better, but I assume that's not happening soon.

do you know if there is a way to manually choose the dll to use?

There is currently no simple way to do this, see #130.
As a work-around, you could probably monkey-patch find_library(), similar to how it is done here: https://github.com/spatialaudio/python-sounddevice/blob/d353fbf6c751a7a91169c614df8f905f2c55821c/doc/fake__sounddevice.py#L17.


Create foreign key schema item

Within the models we have a specialized type for schema item ID's which are treated as validated strings. It would be really useful to create a new item type that treats ID's as foreign keys. We could expand the serialization so that when we return a model with a foreign key, it actually looks up and includes that model. Likewise, we can also have items removed when foreign keys are required and subsequently deleted. It might be beneficial to separate foreign key types. We could specify different relationships that would do different things based on the request type.
I think there are two basic types of foreign keys.

required-key
optional-key

Example use-cases
required-key

Say you have a project model. Each project can create several asset models, each of which needs the project ID as a reference (currently saved as an ID). If instead we treated the project ID as a foreign key (of type required-key), we could expand the functionality so that if the project is deleted, all database rows that have a required-key matching the project must be deleted as well.

optional-key

Say you have a project model and each has an optional-key 'active-image' that points to an image resource. If the image resource is removed, the project key for  active-image must be nullified. Because the active-image is an optional-key, if the project resource is deleted, the image resource is left in the database.

Serialization
Each foreign key should optionally be retrieved when fetching serialized models. There should be a parameter check for 'include foreign keys', which if true, will append the key model to the serialized object.
Proposed methodology
Each model instance would keep two separate arrays. One for required dependencies and one for optional ones. When the instance is deleted, the delete function initiates a chain of events removing / nullifying each of the items in the respective dependency arrays.
The schema item for foreign keys will specify which collection and identifier the dependency will be added to (i.e. its up to the child item to add the foreign key to the target). Each of the dependencies added to the target would likely have a collection and Id property to identify the resources to target upon deletion.

Foreign keys are now implemented and tested vigorously as part of the comment release, see: aac82ac


AnalysisServer.setPriorityFiles() should remember the given set

The specification states that the path may become an actual priority file later.

Note that this request determines the set of requested priority files. The actual set of priority files is the intersection of the requested set of priority files with the set of files currently subject to analysis. (See analysis.setSubscriptions for a description of files that are subject to analysis.)
If a requested priority file is a directory it is ignored, but remains in the set of requested priority files so that if it later becomes a file it can be included in the set of actual priority files.

I believe that this is now being done. Please re-open if that's not the case.


Enhance SelectionSet annotation to ignore fields in the return type

Add a new use case for SelectionSet annotation where users can still let Mocca resolve the selection set automatically, using the response DTO declared in the method return type, however, ignoring fields specified in a new attribute, called ignore, to be added to SelectionSet.
SelectionSet would then behave like this:

If annotation is present and its value attribute is set, Mocca automatic selection set resolution is turned off, and SelectionSet value is used to define the selection set. In this case if ignore value is also set, then that is not used by Mocca, and a warning is logged.
If annotation is present, its value attribute is NOT set, but ignore is, then Mocca automatic selection set resolution is turned ON, and SelectionSet ignore is used to pick which response DTO fields to ignore from the selection set.
If annotation is present and both value and ignore attributes are NOT set, then a MoccaException is thrown.

Important Note
The fields marked to be ignored must NOT be present in the selection set defined in the request message! This might sound obvious but it is important to emphasize it to make sure the code changes pushed for this issue don't remove those fields only after the server already returned a response with them.
Acceptance Criteria

Use case described above is implemented
End user document is changed accordingly
Javadocs are changed accordingly
Unit and/or functional tests are added
Make sure to add negative tests as well

@fabiocarvalho777 Can you specify how the user will specify ignore for inner values?
For eg: if response type is
{booleanVar complexField {innerBooleanVar innerIntVar innerStringVar} intVar stringVar}
How will I specify ignore for innerBooleanVar and innerStringVar
Also are we supporting to ignore the inner class itself as a whole?
For eg can we specify ignore to be complexField somehow which should ignore all its inner values?
I need clarity on some of these cases.

{
  booleanVar
  complexField {
    innerBooleanVar
    innerIntVar
    innerStringVar
  }
  intVar
  stringVar
}

Given the response type above, to ignore innerBooleanVar and innerStringVar, ignore should be set to {"complexField.innerBooleanVar", "complexField.innerStringVar"}. Please, as I said before, take a look at how ignore works and is documented for Var annotation. The format here is identical to that.

Also are we supporting to ignore the inner class itself as a whole?

Yes, we are. If we want to ignore complexField, all we have to do is set ignore to "complexField".
Let me know if you have other questions. Thanks.

Thanks for clarifying @fabiocarvalho777

3. If annotation is present and both `value` and `ignore` attributes are **NOT** set, then a `MoccaException` is thrown.

Why don't we simply not allow this instead of exposing our users to runtime exceptions? Is a different annotation design (e.g. 2 different annotations) not an option? I don't know that much about 'selection sets', so forgive my ignorance.. Just trying to help with the PR review.

Having two different annotations could be an option. There are some cons in this approach too. For example, what if the user uses both annotations at the same time? Wouldn't we have to check and prevent that in runtime anyways? How would we name both annotations? Could it be confusing for users to have two instead of one? Could it make the logic that process the annotations more complex?
So, when I think about the pros and cons, I think having one annotation sounds better, although not ideal.
What do you think?

I definitely think we can run with this as-is and let users complain.
Of your questions, the first one gives me the most pause.  I am assuming we could come up with good names, but that worries me a bit as well.  Honestly, I was thinking more about types (classes) when I made those comments.  I haven't built many things where I leveraged annotations.  Anyhow, if we ever revisit, here are some options that could be considered:

Single annotation that uses an enumeration (not sure on this, but I think I've seen it).
Compile-time annotation processing (this could solve mutually exclusive annotation/property problem)

Again, I don't think this is required.  I have a mild itch to investigate, but sadly time will likely not permit it.


Feature - refactor cron formatting

Is your feature request related to a problem? Please describe.
The cron job appends the output of  scorecard commandline to an output. This is a problem since, the cron job deals with string output from the scorecard command and tries to get the formatting right.
Describe the solution you'd like
Instead, the cron job should be dealing with in-memory data structure of check results which it can use in any way it likes, including formatting it using pre-existing formatting libraries (like JSON). To do this, we need to refactor such that we have a common library of functions/types which will be used by both the commandline tool as well as the cron job.
@inferno-chromium - this is a fair amount of refactor work, but IMO worth the time/effort. Let me know if this LG to you and I can get started on it.

Instead, the cron job should be dealing with in-memory data structure of check results which it can use in any way it likes, including formatting it using pre-existing formatting libraries (like JSON). To do this, we need to refactor such that we have a common library of functions/types which will be used by both the commandline tool as well as the cron job.

A good idea IMO. But with the in-memory you could run into memory bottlenecks as we scale to 10,000 -100,000 repository.
Will the solution be an issue for such scales?

Depends on how we handle the results. We could keep writing to file/DB every time a result comes back or we can batch it to reduce IO. But yes, we mostly will not hold all results in memory at the same time.

SGTM.

Big +1 for this :) This would clean up a lot of how the current outputting is done.


Fix wrong release filetype

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice catch 🚀
/approve


Transient apiserver errors can cause CSV to show RequirementsNotMet

Errors reaching the discovery API (in isGVKRegistered) can cause required/native API requirements to be marked as NotPresent, and the CSV to be marked Pending/RequirementsNotMet. This kind of error should either cause the CSV to be queued for resync or there needs to be support for partial updates to the requirement status list.

Related to installplan retries

The specific case of discovery check failures should have been resolved by https://github.com/operator-framework/operator-lifecycle-manager/pull/2024.


Feat/add ignore inscriptions UI

Try out Leather build 80e2ee2 — Extension build, Test report, Storybook, Chromatic

I've been putting together the state we need to track discarded utxos. You can test this with the text button attached to each inscription. Currently, I'm adding them to the setting slice, by txid:vout. I realise this should also included the sat offset too. A user might discard an one inscription on a UTXO, but not the second one, on the same UTXO. This complicates things slightly, but not toomuch.
To handle the filtering, without many changes in monorepo, I believe we can create an array of known-safe utxos that do have inscriptions, and add them back to the utxos collection returned by hooks like useCurrentNativeSegwitUtxos.
Next steps is to put together this state, by looking all discarded inscriptions, and looking for matching utxos (that don't have other undiscarded utxos).
There are some other changes in this PR I've been playing around with re a global option as well, tbd if we go that route.

Toggle isn't sitting quite right visually, should it all be in one row?

Thanks for your reviews all, unaddressed feedback I will work on when refactoring this in the new year


Error Occured in Quick Start case

Hello. Thank you for your great product.
I have installed the Memory Pack in Unity and am trying Quick Start, but I get the error  is not registered in this provider.
I thought this might be caused by the fact that Static Constructor is not executed, so I implemented partial void StaticConstructor() to check the execution.
It compiles fine on the IDE, but on Unity, I get a compile error.

Looking at the Problem window on the IDE side, it says Generator 'MonoScriptInofoGenerator' failed to generate source.
But I can't find whether this error caused the matter or not.

It seems that only on the Unity side, the generated file of SourceGenerator is not visible.
How can I fix it?
The Code i used in test:
using System;
using System.IO;
using MemoryPack;
using UnityEngine;

namespace Temporary
{
    public class TempSave : MonoBehaviour
    {
        private void Start()
        {
            var v = new Person { Age = 40, Name = "John" };

            var bin = MemoryPackSerializer.Serialize(v);
            var val = MemoryPackSerializer.Deserialize<Person>(bin);

            Debug.Log(val.Name + " " + val.Age);
        }
    }

    [MemoryPackable]
    public partial class Person
    {
        public int Age { get; set; }
        public string Name { get; set; }
    }
}

my environment
Unity 2022.3.10f1
JetBrains Rider 2024.1.3
Memory Pack 1.21.1
I have Installed MemoryPack via Nuget Importer

And package manager

Repository reproducing the problem
https://github.com/amenonegames/SaveTest

Sorry. I misunderstood the supported Unity versions.
Upgrading to Unity 2023.3.4f1 solved the problem.


Very minor edit, no need to call the cmake-gui

More cross-platform I think.
Thanks for the very nice tool. Just a question, is there a way to import / export point sets ? I'd love to use SamplinSafari as a viewer of the nD pointsets generated by https://utk-team.github.io/utk/

There is currently no support for loading point sets, but that sounds like a useful feature. I'd be happy to accept a pull request. Since the interface is currently designed for interactively adjusting parameters (e.g. sample count), this would require a bit of thought.


Enable EVP_Digest(Sign|Verify)* provider support

This PR implements the plumbing necessary to do EVP_DigestSign and EVP_DigestVerify operations via a provider. We just implement this support for DSA at this stage.

Hopefully a valgrind of evp_extra_test should point out the memory leak..

Rebased to resolve conflicts with master, and new commits added to address memory leak. Hopefully should be ok now.

Fixup commit pushed addressing the error number discrepancy.

Rebased. Ping?

Ping ping?

Fixup commit pushed addressing feedback.

Maybe recheck the doc-nits output -- the Travis failures appear to be these.

Look good to me..

Maybe recheck the doc-nits output -- the Travis failures appear to be these.

A rebase will fix that...

A rebase and merge is fine -- the Travis problems aren't related to this PR.

Pushed. Thanks.


Not Allowed To Export Objects

I am still getting this error:  Error: Plugin/Preset files are not allowed to export objects, only functions.

@dbarks1024 what version of react-native-dotenv do you use?
I had the same issue, but when I upgraded to 0.2.0 as suggested here in #44, I no longer had that problem.

Looks like its working for me now. Reinstalled it. Thanks!


Ability to change the operations order

Hello
I have a controller with 3 actions and each of them. I would like to change the order in which the swagger.json is generated. What is the best way of doing it?
Thanks

Configure SortKeySelector at your Startup.ConfigureServices(IServiceCollection services)
services.AddSwaggerGen(options=>{
   options.SwaggerGeneratorOptions.SortKeySelector = descriptor => descriptor.ActionDescriptor.AttributeRouteInfo?.Order?.ToString() ?? descriptor.ActionDescriptor.DisplayName;
})

Mark route attribute such as HttpGetAttribute on your Controller Action.
[HttpGet(Order = 2)]


Video Poster image resizing bug (flickering effect)

I like this plugin but cannot use it because I see that this bg cover technique makes the poster image resizing when page displays.
I tried on the example included in the repo by adding an image for the poster and I got this resizing/flickering effect on page load.

Well I am sorry in fact its native in VIDEO tag, if the POSTE is not at same size as VIDEO it will make a strange flickering resize. Sorry.


google-api-php-client-2.0.0

Hello @bshaffer
i integrate google analytic dashboard on my admin
Deprecated: google-api-php-client's autoloader was moved to vendor/autoload.php in 2.0.0. This redirect will be removed in 2.1. Please adjust your code to use the new location. in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/autoload.php on line 19
Fatal error: Uncaught exception 'InvalidArgumentException' with message 'file does not exist' in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php:839 Stack trace: #0 /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php(824): Google_Client->setAuthConfig('client_secrets....') #1 /var/www/html/example.com/analiytics/index.php(13): Google_Client->setAuthConfigFile('client_secrets....') #2 {main} thrown in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php on line 839
Above error get

Verify your setAuthConfig file exists. You should always supply the absolute path to your credentials file, i.e:
$credentialsFile = '/full/path/to/myfile.json';
$client->setAuthConfig($credentialsFile);

Hello @bshaffer
I given full path for credentialsfile
$credentialsFile = '/var/www/html/example.com.com/analiytics/client_secrets.json';
$client->setAuthConfig($credentialsFile);
This error genrate
Deprecated: google-api-php-client's autoloader was moved to vendor/autoload.php in 2.0.0. This redirect will be removed in 2.1. Please adjust your code to use the new location. in /var/www/html/4lifesystem.com/analiytics/google-api-php-client-2.0.0/src/Google/autoload.php on line 19
and also every time link redirect to google authentication url after show data
Deprecated: google-api-php-client's autoloader was moved to vendor/autoload.php in 2.0.0. This redirect will be removed in 2.1. Please adjust your code to use the new location. in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/autoload.php on line 19
First view (profile) found: All Web Site Data
Total sessions: 50
Total Pageviews: 124
Total User: 27
Thankyou

The error states the use of the file src/Google/autoload.php is deprecated, and instead you need to use vendor/autoload.php. If you follow what the error states, and update all includes to the new file location, the error will be resolved. i.e:
// what you have now:
require_once '/path/to/google-api-php-client/src/Google/autoload.php`;
// what you should update it to:
require_once '/path/to/google-api-php-client/vendor/autoload.php`;

@bshaffer I ditto the above and even after updating the path to: require_once '/path/to/google-api-php-client/vendor/autoload.php'; while I no longer get

Deprecated: google-api-php-client's autoloader was moved.. >

I still get a set of errors similar to @ashik906 :

Fatal error: Uncaught exception 'InvalidArgumentException' with message 'file does not exist' in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php:839 Stack trace: #0 /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php(824): Google_Client->setAuthConfig('client_secrets....') #1 /var/www/html/example.com/analiytics/index.php(13): Google_Client->setAuthConfigFile('client_secrets....') #2 {main} thrown in /var/www/html/example.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php on line 839

@nmartin4321 You need to either download the package and dependencies found in the releases section, or install via composer

Thanks @bshaffer , apologies I should have set some context.  I followed all instructions here: https://developers.google.com/analytics/devguides/reporting/core/v3/quickstart/web-php
Using Windows 7 (IIS 7+ with IIS Express PHP v 7)
Installed Composer & verified all dependencies are in place.
In 'Client.php' the referenced line suggests that there's something wrong with the information being passed for the Auth config:
public function setAuthConfig($config)   {     if (is_string($config)) {       if (!file_exists($config)) {         throw new InvalidArgumentException('file does not exist');       } ...
Some more googling suggested that I use json_decode() in my index.php and oauth2callback.php fields for the 'client_secrets.json'.  I did this and it seems to move beyond the error I posted earlier but still throws an odd error where it spits out hundreds of lines showing the same thing:
[Mon Jul 25 15:02:33 2016] PHP Deprecated:  google-api-php-client's autoloader was moved to vendor/autoload.php in 2.0.0. This redirect will be removed in 2.1.  Please adjust your code to use the new location. in C:\Users\XXXXXX\vendor\google\apiclient\src\Google\autoload.php on line 20
Now the problem is, I have also confirmed that all my references to the autoload.php in question are updated.  (as "C:\Users\XXXXXX"  is doc root, I've equivalents of $file = '/vendor/autoload.php'; in each location)
So, I'm still not sure the the json_decode() effort actually helped.  Making no other changes, if I revert my json_decode() update, I get back to the original error.  Any thoughts?

Make sure your call to setAuthConfig is an absolute path and that you've renamed the file you download from the Developer Console to client_secrets.json. :
$client->setAuthConfig('/path/to/client_secrets.json');

You do not need to use json_decode, and the deprecated error is being thrown as a result of an error in your local development setup, and not because of this library.

One of the reasons you're having a hard time is you're using the v3 samples, which are older and outdated. Try using the Analytics v4 quickstart instead.

@bshaffer this is perfect, thank you!  applying the path seemed to do the trick (I'd assumed since all the files were in the same directory that it'd pick it up if I just left it with the file name.)
Also, thanks for the pointer on v4.  If I encounter further issues, I'll move there.
Again, thank you!

Hello @bshaffer,
we are using Google Analytics API in PHP project. But when i connect with API it gives error..
Please visit this link
http://www.4lifesystem.com/analiytics/analytic.php
Error:
Catchable fatal error: Argument 3 passed to Google_AuthHandler_Guzzle6AuthHandler::attachCredentials() must be callable, string given, called in /var/www/html/4lifesystem.com/analiytics/google-api-php-client-2.0.0/src/Google/Client.php on line 370 and defined in /var/www/html/4lifesystem.com/analiytics/google-api-php-client-2.0.0/src/Google/AuthHandler/Guzzle6AuthHandler.php on line 30

Here is my code

setApplicationName("lifesystem");
$client->setAuthConfig($KEY_FILE_LOCATION);
$client->setScopes('https://www.googleapis.com/auth/analytics.readonly');
$analytics = new Google_Service_Analytics($client);

return $analytics;
}

function getFirstProfileId($analytics) {
// Get the user's first view (profile) ID.

// Get the list of accounts for the authorized user.
$accounts = $analytics->management_accounts->listManagementAccounts();

if (count($accounts->getItems()) > 0) {
$items = $accounts->getItems();
$firstAccountId = $items[0]->getId();

// Get the list of properties for the authorized user.
$properties = $analytics->management_webproperties
    ->listManagementWebproperties($firstAccountId);

if (count($properties->getItems()) > 0) {
  $items = $properties->getItems();
  $firstPropertyId = $items[0]->getId();

  // Get the list of views (profiles) for the authorized user.
  $profiles = $analytics->management_profiles
      ->listManagementProfiles($firstAccountId, $firstPropertyId);

  if (count($profiles->getItems()) > 0) {
    $items = $profiles->getItems();

    // Return the first view (profile) ID.
    return $items[0]->getId();

  } else {
    throw new Exception('No views (profiles) found for this user.');
  }
} else {
  throw new Exception('No properties found for this user.');
}

} else {
throw new Exception('No accounts found for this user.');
}
}

function getResults($analytics, $profileId) {
// Calls the Core Reporting API and queries for the number of sessions
// for the last seven days.

$profileId=79768194;
return $analytics->data_ga->get(
'ga:' . $profileId,
'7daysAgo',
'today',
'ga:sessions');
}

function printResults($results) {
// Parses the response from the Core Reporting API and prints
// the profile name and total sessions.
if (count($results->getRows()) > 0) {

// Get the profile name.
$profileName = $results->getProfileInfo()->getProfileName();

// Get the entry for the first entry in the first row.
$rows = $results->getRows();
$sessions = $rows[0][0];

// Print the results.
print "First view (profile) found: $profileName\n";
print "Total sessions: $sessions\n";

} else {
print "No results found.\n";
}
}

I've done everything you have suggested "https://github.com/google/google-api-php-client/issues/988" here but still getting this error.
Fatal error: Uncaught InvalidArgumentException: file does not exist in D:\xampp\htdocs\Google\vendor\google\apiclient\src\Google\Client.php:839 Stack trace: #0 D:\xampp\htdocs\Google\HelloAnalytics.php(23): Google_Client->setAuthConfig('D:\xampp\htdocs...') #1 D:\xampp\htdocs\Google\HelloAnalytics.php(6): initializeAnalytics() #2 {main} thrown in D:\xampp\htdocs\Google\vendor\google\apiclient\src\Google\Client.php on line 839
I have the credentials file at the correct location and has set the location correctly.


Refactor

Refactoring the whole project
Making exerevnetes beautiful again!

This is the refactor pull requests.
The goal of the refactor was to re organise the internal infrastructure of the project in order to make the class API more in line with scikit learn, improve readability and the ability for adding extra functionalities


self hosted qr-code generator insted of api.qrserver.com possible ?

Hello,
is it possible to have a self-hosted qr-generator insted of api.qrserver.com ?
So that no external service is need.
So that no external dependencies are needed.
is it possible to use the tool: qrencode

Hi, your proposal is very interesting. I invite you to contribute to the project

Hi,
I have the same idea and start to fork another project called go-qrcode:
https://github.com/tranmh/go-qrcode/blob/master/qrcode-restapi/main.go
go-qrcode has now a restapi and your code at line:
https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/blob/master/src/lib/Qrcode/Qrcode.php#L83
can call it using:
$url = 'http://localhost:6868/api/qrcode?data=https%3A%2F%2Fsvw.info&size=256';
What is your expectation? I assume we need go-qrcode with restapi as git submodule. What else do you need to accept a pull request?
Thx.

Hi, the creation method you propose doesn't have the same customization possibilities (or better) that are currently in use, so it's not good

Sure, wait for the other Pull Request:
https://github.com/skip2/go-qrcode/pull/66
If this PR is going into their repo, I will extend the customization possibilities. Which of those is a must-have for you?

I want the actual customization of my script. Thanks

In PR
https://github.com/skip2/go-qrcode/pull/66
I added another improvement: "add support for ecc aka recovery level, color and background-color". and would look like this:
http://localhost:6868/api/qrcode?data=abc&size=800&color=00FF00&bgcolor=FF00FF&ecc=L

Is it already working?

yes. PR is not accept yet, but I tested on my local machined.

Ok then write to me when it is accepted please

Change would be:
https://github.com/giandonatoinverso/PHP-Dynamic-Qr-code/blob/master/src/lib/Qrcode/Qrcode.php#L83
replace the following line
        $url =
            'https://api.qrserver.com/v1/create-qr-code/?data='.
            $data_to_qrcode.
            '&&size='.$options['size'].'x'.$options['size'].
            '&ecc='.$options['errorCorrectionLevel'].
            '&margin=0&color='.$options['foreground'].
            '&bgcolor='.$options['background'].
            '&qzone=2'.
            '&format='.$data_to_db['format'];

with
        $url =
                'http://localhost:6868/api/qrcode?data='.
                $data_to_qrcode.
                '&size='.$options['size'].
                '&color='.$options['foreground'].
                '&bgcolor='.$options['background'].
                '&ecc='.$options['errorCorrectionLevel'];

I saw that the format is missing. Many of my users need all the formats that my script offers. Does this new generation method have them? For example EPS

https://github.com/skip2/go-qrcode/pull/66 have now the following to formats: png and svg. More will hopefully will come.

Ok let me know when the other formats will be available

Hi,
nice project ... but

is it possible to have a self-hosted qr-generator insted of api.qrserver.com ?

That was also my first thought when I read the information in the documentation that the data is sent to an external service provider. 😮
This is not nice and completely unnecessary, as there are enough offline projects

Hi,
I use this PHP QR Code (chillerlan/php-qrcode) now.


chore: 🤖 update CODEOWNERS

Description

Remome Jere from CODEOWNERS and add myself and Eric in the list

Breaking Changes
NA
JIRA Link
DA-409
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 15.1.1-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 16.0.0-alpha.2 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Websites malfunction upon deployments

I don't think converting the javascript to a website is working. Vercel does not do anything; upon deployment, it just hangs. Netlify gives me an error saying: "yarn run v1.22.5 info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. | warning package.json: No license field error Command "build" not found." and the webpage says "Not Found" and gives me a Request ID. The GitHub Pages is successful; however, when I enter the deployment page, all that is left is the "Green Flag" button. I am trying to use this guide: https://leopardjs.com/help/publish-with-vercel. What is happening?

This is referring to the Publish your project as a website! tutorial on the Leopard website, linked from the homepage. I'm not familiar with Vercel and supporting deployment so I won't be able to help address this issue, sorry!

This is referring to the Publish your project as a website! tutorial on the Leopard website, linked from the homepage. I'm not familiar with Vercel and supporting deployment so I won't be able to help address this issue, sorry!

Vercel was not the main focus but thanks anyway I guess (this is old)

Oh whoops yeah, my fault for managing issues near midnight 😅
If this is something you're still interested in working on, do you know if it's possible to share the exact files which are being deployed to Netlify? It seems a little weird to me for there to be a package.json at all, since Leopard projects should be able to deploy on a completely static website (it's just a normal index.html and static assets/code files).
Actually, it sounds like yarn build or npm run build might just be the Netlify default. I don't know if that's something you can configure directly on CodeSandbox, but you may be able to follow the instructions here and clear the "basic build settings" section (Step 5). Then a second deployment should work correctly.
It sounds like the GitHub Pages build got closer. Did this project work correctly when you first converted it with Leopard, on the codesandbox.com website / online text editor? GH Pages should be basically just a copy of what you see on CodeSandbox. If all you saw was the "green flag" button on CodeSandbox, that means there is a compatibility error in Leopard or sb-edit (it's not 100% compatible with Scratch yet) — but the deployment to GH Pages is probably working too.
On the other hand, if the project does run on CodeSandbox but not GitHub Pages, that's definitely some kind of issue with deployment and would take some further investigation.


is possible auto close after show?

It may be because i'm showing it programmatically?
I tested pressing the child for showing the tooltip and it works fine.

Sorry, can't go fully into this rn. Maybe this weekend. I will point out where I think the issue is though.
In this function in a regular tooltip press it handles the tooltip dismissal.
https://github.com/Nolence/just_the_tooltip/blob/a143964b91642e2a0ac6624b878994a3b138f49f/lib/src/just_the_tooltip.dart#L523
Because a controller _showTooltip never triggers a mouse click event, those _hideTooltip functions are never called which never set the timer for dismissal. I think the correct option would to pass the controller showTooltip a field that defines the behavior of the tooltip after opened.

auto close is not work. When :
               triggerMode: TooltipTriggerMode.tap,


[ot_certs] Add support for custom key usage

This allows certificate templates to specify whether they want a key usage extension at all, and if they do what the key usages are. I only added support for the 3 usages that seem the most

I guess this raises another question: with this PR, I made a breaking where not specifying a key usage will not create a key usage extension at all. The alternative would be to say that not specifying a key usage extension revers to the previous behaviour (just cert_sign set). But what if someone wants to key usage extension at all? (here I am using key usage as an example, but the DICE extension would be another one). Maybe when we make changes like that we would want a tri-state:

if key_usage is not specified, then it defaults to the previous behaviour for backward compatibility,
if key_usage is specified, then it overrides the default
there is a way to say that we don't want a key usage extension at-all, something like key_usage: none.

What do you think?

I guess this raises another question: with this PR, I made a breaking where not specifying a key usage will not create a key usage extension at all. The alternative would be to say that not specifying a key usage extension reverts to the previous behaviour (just cert_sign set). But what if someone wants to key usage extension at all? (here I am using key usage as an example, but the DICE extension would be another one). Maybe when we make changes like that we would want a tri-state:

if key_usage is not specified, then it defaults to the previous behaviour for backward compatibility,
if key_usage is specified, then it overrides the default
there is a way to say that we don't want a key usage extension at-all, something like key_usage: none.

What do you think?

I think it is fine to omit it if the user did not include it in the template.

I don't see how I can vote on this, but FWIW it looks good to me


what's the difference on CRDs between Erie Canal and MCS API?

Please describe what should be documented
It's described that Erie Canal is an implementation of MCS API, but it uses CRDs from a different group.
Could you figure out their difference and where it comes from?
Please suggest where in the repo the document should be located

ServiceExport

ServiceImport

Thanks @reaver-flomesh .


BUG: Missing SVN path for dashboard script initialization

GIT_EXECUTABLE is set in InitCMakeCache.cmake, but Subversion_SVN_EXECUTABLE
was not set.

Is Subversion used anywhere in TubeTK?   Perhaps this dependency should be removed?   I just did a quick search on SVN and found no uses.

Indeed. Closing this PR.


Saving mesh not working

Description
Hi @lukemelas, thanks for open sourcing your great work! Upon reproducing your examples, cat_statue specifically, I noticed that --save_mesh option does not work as expected on testing time. Here's the output:
[INFO] Trainer: df | 2023-04-10_14-25-15 | cuda | fp16 | outputs/default/2023-04-10--13-49-56--seed-101/
[INFO] num parameters: 1_806_983
[INFO] num parameters w/ grad: 1_806_983
[INFO] Loading latest checkpoint ...
[INFO] Latest checkpoint is outputs/default/2023-04-10--13-49-56--seed-101/checkpoints/df.pth
[INFO] loaded model.
[INFO] load at epoch 50, global step 5000
==> Start Test, save results to outputs/default/2023-04-10--13-49-56--seed-101/results
100% 100/100 [00:05<00:00, 18.63it/s]rgb
opacity
depth
/home/tongwang/workspace/realfusion/nerf/trainer.py:590: RuntimeWarning: invalid value encountered in cast
preds_np = (preds_tensor.detach().cpu().numpy() * 255).astype(np.uint8)
normals
textureless
grid
==> Finished Test.
100% 100/100 [00:06<00:00, 15.85it/s]
==> Saving mesh to outputs/default/2023-04-10--13-49-56--seed-101/mesh
==> Finished saving mesh.
Although the log says it is "==>Saving mesh", but it did not actually save the mesh.  Could you please look into this issue? Thanks in advance.
Steps to Reproduce
python main.py --workspace $model_path --O --test --save_mesh
Expected Behavior
save a textured mesh
Environment
ubuntu 20.04, torch1.13+cu116

Nevermind, I figured out how to fix this bug. The reason that it is not saving mesh is because _export() wasn't called. To fix this, add:
 f = torch.from_numpy(triangles).contiguous().int().to(self.aabb_train.device)```

after line 181 in `nerf/renderer.py`, also add

`_export(v, f)` at the end of export_mesh() method  in `nerf/renderer.py`

@tydia I still meet the error
RuntimeError: instance mode - pos must have shape [>0, >0, 4]
how do you deal with it?

Thanks for finding this! I'll create a PR.

@lukemelas thanks! I switch to your PR, but it's still has the error:
RuntimeError: instance mode - pos must have shape [>0, >0, 4]


Fixed Issue #4

Added Illustration

Requested changes made

Requested changes made

Requested changes completed

one minor change just move the image present in the images folder to assets and delete images. SO there will be only import images line.
And In pubspec.yaml keep only the assets folder.


Opencl Compile issue

anyone please Compile it with Opencl i want to run this software on rxt 3090

Agree with you. We all want cuBitCrack with random option. :)


Returning user-friendly error messages for managed services

SDA-6084

@ciaranRoche ptal, I'm probably missing some error messages

@ciaranRoche PTAL and merge it

/retest


Add "Project Description" when publishing to PyPI

If we goto PyPI module page https://pypi.org/project/tensorflow-addons/, we will not see any document about this module: It's empty.
We should add a project description to this page so that the developer could be able to understand what this module is easier.

Thanks @huan! Just created this a few days ago though so duplicate of #114

@seanpmorgan Great! sorry for having not noticed that old one.


Replced iOS with mobile

As everyone can agree that this document is not just for iOS but in general for any mobile platform. I am removing the word iOS.

Just saw you closed this - I remember this coming in and thought I'd merged it. Just did that now - thanks!


feat: allow users to join a drop, assign a random NFT to user at drop…

… time, let user only buy the assigned NFT

And sorry for the late review while still saying I would be available for a review this day

Its all working now!

All working


Bug: OnBeforeCustLedgerEntryFind Event in Codeunit 392 Reminder-Make

Problem
Method call OnBeforeCustLedgerEntryFind(CustLedgEntry, ReminderHeader, Cust); in Codeunit 392 Reminder-Make has the wrong attribute ReminderHeader, it should be ReminderHeaderReq.
Situtation
Variables within the codeunit are set by the method Set.
Within the Set method the global variable ReminderHeaderReq is set.
Also later in the method MakeReminderthe global variable ReminderHeaderReq is used.
The content of ReminderHeader is transfered by assignment to variable ReminderHeaderReq in method Code.
Conclusion
ReminderHeaderReq is the current variable and should be used, not ReminderHeader.
Solution
Drop variable ReminderHeader, include instead variable ReminderHeaderReq in method OnBeforeCustLedgerEntryFind()

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.