Dinosaur Game

Description
Make an application for the dinosaur game exactly similar to the game present in chrome.
Technologies

Python
PyGame

@bislara Can I work on this?

Sure. Go for it. 👍

Any update? Feel free to reach out to me if you need any help.

@yash-agarwal17  Any progress?

@yash-agarwal17  has been unassigned from the issue due to the absence of response since the last 24 hours, making this issue up for grabs for anyone interested in it.

Hi @bislara, I've created a PR for the dinosaur game. Please review and let me know if any changes are required. Thanks


Fix folder copying to sibling folder with almost the same name

To reproduce this bug create folder with name "Some Folder" and another sibling folder with name "Some Folder 1".  Try to copy or move first folder in second.

@tsvietOK Hi thanks for this PR.
The original issue that this block of code tried to solve was users copying a folder into itself. Does this change still prevent all instances of a destination folder being a subfolder of the source folder?

@duke7553 hmmm, your right, i didn't test this case. I have to find another workaround.

@tsvietOK I'll mark this as a draft for now then.

Taken from https://stackoverflow.com/questions/5617320/given-full-path-check-if-path-is-subdirectory-of-some-other-path-or-otherwise
@duke7553  what do you think about it?

@tsvietOK That looks pretty good to me

@duke7553 do you have troubles with folder copying?

@tsvietOK Do you want me to test this now? I can mark this as ready for review.

@duke7553 If your copy function working fine, of course.

@duke7553 did you tested it?

@tsvietOK Can you merge with the latest from develop?

Before merging, this should be tested to make sure that the changes work in release mode.

@yaichenbaum Part with subfolder detection working fine for me. Another thing, we currently don't  supporting Windows's path length limit(as i understand 260 characters)

@tsvietOK I would think that is a different issue, right?

@yaichenbaum Yes, your right


Support passing slices to MySQL queries

While MySQL supports the ANY operator, it does not support arrays. This means the currently documented way to pass slices to a query will not work.

Here's some related discussion from the go-sql-driver repository:

https://github.com/go-sql-driver/mysql/issues/107
https://github.com/go-sql-driver/mysql/issues/176

The MEMBER OF syntax may offer a solution. The problem is that you need to encode the slice as JSON before passing it into the query.
CREATE TABLE pkg (
  id text NOT NULL,
  vic VARCHAR(50) NOT NULL,
  kim text NOT NULL,
  order INTEGER NOT NULL,
  PRIMARY KEY (id)
);

/* name: GetPkgID :many */
SELECT id, vic, kim FROM pkg
WHERE vic MEMBER OF (?) AND shop_id = ?;

Okay, here's my current idea. We add a second type parameter to sqlc.arg. That parameter, if passed, will compile to a cast statement. We can support the ARRAY type syntax in this type parameter and generate the correct code.
 AS JSON));
`

func (q *Queries) FilterUsers(ctx context.Context, ids []int64) ([]User, error) {
	// ...
}

A few issues I can think of:

The array type must convert to a scalar type that can be represented inside a JSON array
We'd need to generate a helper method that converts slices to JSON
We'd probably also need to add a method that unmarshals JSON as well

I'm not sure if it's bad practice, but maybe the generated code could produce a dynamic query, generating N placeholders for each array value. So, something like:
func filterUsers(vals int[])
{
return fmt.Sprintf(`-- name: FilterUsers :many
SELECT id FROM users
WHERE id MEMBER OF(?%v);
`, strings.Repeat(",?", len(inValues)-1));
}

Just throwing an idea (or hack) regarding IN statement in SQL.
Current behaviour:
const listAuthors = `-- name: ListAuthors :many
SELECT id, name FROM authors WHERE id IN($1) AND name = $2;
`

type ListAuthorsParams struct {
	ID   int
	Name string
}

func (q *Queries) ListAuthors(ctx context.Context, arg ListAuthorsParams) ([]Author, error) {
	rows, err := q.db.QueryContext(ctx, listAuthors, arg.ID, arg.Name)

My idea is to change the parameter into slice and expanding the query based on the size of the slice.
What I meant by expanding is to replace id IN ($1) into id IN($1, $2, $3, ...).
If there's 3 ids, name = $2 will become name = $4.
Using the same query, the generated code will now look like this:
type ListAuthorsParams struct {
	ID   []int
	Name string
}

func (q *Queries) ListAuthors(ctx context.Context, arg ListAuthorsParams) ([]Author, error) {
	query, args := q.expand(listAuthors, expandArg{arg.ID}, arg.Name)
	rows, err := q.db.QueryContext(ctx, query, args...)

The expandArg type is used to differentiate normal slice with slice from IN statement.
The q.expand() function will return modified query and args.
For example, given that arg = ListAuthorsParams{[]int{9, 8, 6, 7}, "Joe"}, q.expand() will return:
query = "SELECT id, name FROM authors WHERE id IN($1, $2, $3, $4) AND name = $5;"
args = []interface{}{9, 8, 6, 7, "Joe"}

The code for q.expand() can be seen here. I haven't managed to find how to change arg.ID into []int.
Not forget to mention, this solution will not work for prepared statement.

How about for MySQL only, ( ? ) will be interpreted as accepting a list of items of that type, rather than a single one?
Pros

It's still valid SQL
It would not be confusing to users. As in previous issues like https://github.com/kyleconroy/sqlc/issues/706, https://github.com/go-sql-driver/mysql/issues/107, https://github.com/go-sql-driver/mysql/issues/176, people naturally assume you can pass in a slice. Any strongly-typed language will prevent you from passing in a single value.

Cons

The type of the binding differs from what you'd have to pass in to the driver, but sqlc already abstracts the driver away so that's fine.
What would you do about (?, ?, ?) etc...? Imo sqlc could just error in the parse stage and say that you have to write your query as a single (?)

MEMBER

Syntax error with keyword MEMBER OF ? I use sql 1.6.0.

I have an idea,
using FIND_IN_SET
SELECT id FROM users
WHERE FIND_IN_SET(id, ?)

But name of variable after run sqlc generate: FINDINSET and FINDINSET_2, ...
@kyleconroy, How to fix or rename variable in above?

How can I achieve this in postgres?

How can I achieve this in postgres?

Just throwing an idea (or hack) regarding IN statement in SQL.
Current behaviour:
const listAuthors = `-- name: ListAuthors :many
SELECT id, name FROM authors WHERE id IN($1) AND name = $2;
`

type ListAuthorsParams struct {
	ID   int
	Name string
}

func (q *Queries) ListAuthors(ctx context.Context, arg ListAuthorsParams) ([]Author, error) {
	rows, err := q.db.QueryContext(ctx, listAuthors, arg.ID, arg.Name)

My idea is to change the parameter into slice and expanding the query based on the size of the slice.
What I meant by expanding is to replace id IN ($1) into id IN($1, $2, $3, ...).
If there's 3 ids, name = $2 will become name = $4.
Using the same query, the generated code will now look like this:
type ListAuthorsParams struct {
	ID   []int
	Name string
}

func (q *Queries) ListAuthors(ctx context.Context, arg ListAuthorsParams) ([]Author, error) {
	query, args := q.expand(listAuthors, expandArg{arg.ID}, arg.Name)
	rows, err := q.db.QueryContext(ctx, query, args...)

The expandArg type is used to differentiate normal slice with slice from IN statement.
The q.expand() function will return modified query and args.
For example, given that arg = ListAuthorsParams{[]int{9, 8, 6, 7}, "Joe"}, q.expand() will return:
query = "SELECT id, name FROM authors WHERE id IN($1, $2, $3, $4) AND name = $5;"
args = []interface{}{9, 8, 6, 7, "Joe"}

The code for q.expand() can be seen here. I haven't managed to find how to change arg.ID into []int.
Not forget to mention, this solution will not work for prepared statement.

i fufill  a version that solve this problem https://github.com/xiazemin/sqlc

Is there a workaround for the time? Listing by IDs seems to be a rather crucial thing to do after all

Is there a workaround for the time? Listing by IDs seems to be a rather crucial thing to do after all, e.g. for pre-loading associations

The only work around I ended up doing was creating a separate file that shares the same package name and writing the golang code manually :/

Is there a workaround for the time? Listing by IDs seems to be a rather crucial thing to do after all, e.g. for pre-loading associations

The only work around I ended up doing was creating a separate file that shares the same package name and writing the golang code manually :/

my fork solved this issue https://github.com/xiazemin/sqlc

and the example of https://github.com/xiazemin/sqlc is here https://github.com/xiazemin/sqlc_study

hey @xiazemin this is great! Just read your previous messages also, maybe you can make a proposal or pull request to this repo?

@kyleconroy Hi, any plan to support slice?

A good solution to this is crucial.  There is an acceptable solution for the postgres driver using pg.Array() but in the comment thread above there was nothing that really works for mysql.  In the meantime, I'll fall back to using a query generator like squirrel but that's not my first choice.

I've been using a forked version of sqlc where they implemented sqlc.slice() and all it does is a string replace in the generated code. Not ideal but works really good and would prefer that we had this then not having the feature at all.

my fork also support insert into xxx values xxx， where values is a slice,you can try it!

I haven't tried it yet, but would something like that work for a temporary solution?
SELECT id FROM authors
WHERE id IN (@list = ?);

I don't know, wouldn't it make sense to always treat a value in IN() as a slice?

https://github.com/xiazemin/sqlc

Xiazemin, your sqlc repo renames everything and is not a true fork, your code can’t be migrated back into the main repository.

On May 31, 2022, at 8:49 AM, xiazemin @.***> wrote:
https://github.com/xiazemin/sqlc https://github.com/xiazemin/sqlc
—
Reply to this email directly, view it on GitHub https://github.com/kyleconroy/sqlc/issues/695#issuecomment-1142310125, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAKW6GUNJEE2FNROJSWWBLVMYYITANCNFSM4QPX4D2Q.
You are receiving this because you commented.

When will this feature be updated?

SELECT * FROM a_table WHERE (a, b) in (?);

How to implement this case in sqlc?

Hello, this seems like a pretty important feature to have. Anybody have a plan to implement this? If somebody can point me to the correct place I can try submitting a PR.
Thanks

It works in Postgresql
-- name: BatchUpdateUsageRecordGroupStatus :exec
UPDATE metering.usage_record_group
SET status = $2, last_update_time = CURRENT_TIMESTAMP
WHERE organization_id = $1 AND id = ANY(@ids::text[]);

Thanks @chengjun-suger I tried with
-- name: SelectWhereInLastNames :many
SELECT * FROM users WHERE last_name = ANY(@last_name::text[]);

and it generates
import (
	"context"
	"database/sql"
	"encoding/json"

	"github.com/lib/pq"
)

const selectWhereInLastNames = `-- name: SelectWhereInLastNames :many
SELECT id, first_name, middle_name, last_name, email, password, favourite_colour FROM users WHERE last_name = ANY($1::text[])
`

func (q *Queries) SelectWhereInLastNames(ctx context.Context, lastName []string) ([]User, error) {
	rows, err := q.db.QueryContext(ctx, selectWhereInLastNames, pq.Array(lastName))
	if err != nil {
		return nil, err
	}
	defer rows.Close()
	var items []User
	for rows.Next() {
		var i User
		if err := rows.Scan(
			&i.ID,
			&i.FirstName,
			&i.MiddleName,
			&i.LastName,
			&i.Email,
			&i.Password,
			&i.FavouriteColour,
		); err != nil {
			return nil, err
		}
		items = append(items, i)
	}
	if err := rows.Close(); err != nil {
		return nil, err
	}
	if err := rows.Err(); err != nil {
		return nil, err
	}
	return items, nil
}

I see this issue discussed for nearly two years, is there a solution or a plan in the near future, which will determine whether my project can choose to use sqlc instead of others, such as gorm.
Thanks.

Really want this feature as well and started to think about an alternative approach to the problem.
Not sure if this is possible in the way sqlc works, I'm just putting the idea out there to start a discussion.
If we can somehow identify where in the query the problem is and we can modify the query accordingly. One approach to this would be to add some kind of "variadric" configuration based on index
For example, take the following query:
-- name: FindThings :many :variadric(0)
SELECT *
FROM a_table
WHERE status IN ( ? )
AND created_at > ?

This would tell sqlc that the first ? is actually a slice of values and would change the generated function to instead expecting T actually expect []T and when calling to do the query it uses the same approach as mentioned here to replace the first ? with the number of items in the slice passed in.
Would this be possible with the current approach? And if so, does anyone know where to start?

@LIQRGV Interesting, going to have a deep dive in the code on Monday to see what you've done.
The syntax with ??? is a much better idea imo, only drawback is that it's not valid SQL anymore(?).
Currently looking to doing something similar to what you've done but as a plugin, but haven't decided anything yet.

only drawback is that it's not valid SQL anymore(?)

Yep, @karatekaneen technically it's not a valid SQL anymore since it'll parse the query into another placeholder depend on the number args passed.
So if you have 2 params, it will become
WHERE x_column IN (?,?)
Anyway, do you know how to use IN keyword on normal SQL ? I mean using it on prepared statement.
Maybe I can make it to comply with SQL standard.

FIND_IN_SET does the job

This waits for #1816 to be reviewed and merged.

https://docs.sqlc.dev/en/stable/reference/macros.html#sqlc-slice
Adding a link to the docs for future folks

@kyleconroy https://github.com/sqlc-dev/sqlc/issues/2002 was marked as a duplicate of this issue.
However, it seems it isn't.
#2002 concerns multiple inserted values. sqlc.slice can't be used for that purpose as it replaces name by multuplie escaped params entries of type string.
We have this other issue https://github.com/sqlc-dev/sqlc/issues/3305 that is open that explains it better IMHO.
https://play.sqlc.dev/p/7bfa28e99e0207fb0c7d2b457ccc452674daa60acb5bccea336d7c3de293b220


fix(material/datepicker): add aria descriptions for comparison range

Add aria descriptions to the calendar for the start and end of comparison range.
Fixes #23464

I thought about this and decided I don't like the API of it.
Developer should be able to configure the aria descriptions for the start and end of the comparison range, so that they can make sense in the context that the date-range picker is used. (This is similar problem we ran into with #25457


Type definitions are not properly aligned

I'm probably opening too many issues, but hey I hope these help.
The type definitions are not properly aligned as shown here with extra multiple spaces.

I'm guessing this is because of the lsp attribute not being able to hold line break information present in the type definition because it usually doesn't happen in some long strings. I'm not so sure but I think it could be fixed by replacing the \n with
 before putting them in the attribute.

Agree, probably just a text transform somewhere


python310Packages.cachelib: 0.8.0 -> 0.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.cachelib is: "Collection of cache libraries in the same API interface"
meta.homepage for python310Packages.cachelib is: "https://github.com/pallets/cachelib"
meta.changelog for python310Packages.cachelib is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.9.0 with grep in /nix/store/39c2r4gniycc6q2rqb343cjfn00p8n4p-python3.10-cachelib-0.9.0
found 0.9.0 in filename of file in /nix/store/39c2r4gniycc6q2rqb343cjfn00p8n4p-python3.10-cachelib-0.9.0

Rebuild report (if merged into master) (click to expand)

39 total rebuild path(s)

20 package rebuild(s)

20 x86_64-linux rebuild(s)
19 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
apache-airflow
isso
octoprint
powerdns-admin
python310Packages.apache-airflow
python310Packages.cachelib
python310Packages.flask-caching
python310Packages.flask-common
python310Packages.flask-session
python310Packages.graphite_api
python310Packages.howdoi
python310Packages.influxgraph
python39Packages.apache-airflow
python39Packages.cachelib
python39Packages.flask-caching
python39Packages.flask-common
python39Packages.flask-session
python39Packages.graphite_api
python39Packages.howdoi
python39Packages.influxgraph

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/39c2r4gniycc6q2rqb343cjfn00p8n4p-python3.10-cachelib-0.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.cachelib https://github.com/r-ryantm/nixpkgs/archive/ca5ee125844048125663f96c18bb685b775ea697.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/39c2r4gniycc6q2rqb343cjfn00p8n4p-python3.10-cachelib-0.9.0
ls -la /nix/store/39c2r4gniycc6q2rqb343cjfn00p8n4p-python3.10-cachelib-0.9.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages marked as broken and skipped:

apache-airflow
python310Packages.apache-airflow
python39Packages.apache-airflow

5 packages failed to build:

powerdns-admin
python310Packages.graphite_api
python310Packages.influxgraph
python39Packages.graphite_api
python39Packages.influxgraph

12 packages built:

isso
octoprint
python310Packages.cachelib
python310Packages.flask-caching
python310Packages.flask-common
python310Packages.flask-session
python310Packages.howdoi
python39Packages.cachelib
python39Packages.flask-caching
python39Packages.flask-common
python39Packages.flask-session
python39Packages.howdoi

Maintainer pings
cc @gebner for testing.

Result of nixpkgs-review pr 179280 run on x86_64-darwin 1

9 packages marked as broken and skipped:

apache-airflow
python310Packages.apache-airflow
python310Packages.graphite_api
python310Packages.howdoi
python310Packages.influxgraph
python39Packages.apache-airflow
python39Packages.graphite_api
python39Packages.howdoi
python39Packages.influxgraph

2 packages failed to build:

python39Packages.flask-caching
python39Packages.flask-common

8 packages built:

isso
octoprint
python310Packages.cachelib
python310Packages.flask-caching
python310Packages.flask-common
python310Packages.flask-session
python39Packages.cachelib
python39Packages.flask-session

The failures seem to be actually related to the cache:
    def test_cached_view(app, cache):
        @app.route("/")
        @cache.cached(2)
        def cached_view():
            return str(time.time())

        tc = app.test_client()

        rv = tc.get("/")
        the_time = rv.data.decode("utf-8")

        time.sleep(1)

        rv = tc.get("/")

>       assert the_time == rv.data.decode("utf-8")
E       AssertionError: assert '1656312705.942673' == '1656312707.015083'
E         -<PHONE_NUMBER>.015083
E         +<PHONE_NUMBER>.942673

also, looks like the test sleep for a second, and then test that roughly a second went by. On overloaded systems, this will obviously cause delays longer than a second.

@jonringer the expectation is the opposite. It's testing caching, so the test sleeps, but then expects to see the same cached time value in response. I'll be able to run this properly and check the failure in a couple hours.

I'm not sure why i didn't see those failures then :/


Add color property to speckleobject

byObject, byLayer?

Will be just a generic "colour" property:

If not null, set colour from that
If null or not present, set it from object's layer

what about byMaterial xD?
Also, .TextColor for textdot would be nice (.Color being the background one)

Yep, and from memory:

All speckle objects will have a f¢ colour property, which can stay null
SpeckleAnnotation will extend that with, say backgroundColor - because we need to keep things readable.

Any thoughts on Linear dimensions and other stuff like this? I loathe them

For Linear dimensions, if you want I can start to create a SpeckleLinearDimension (or however you want to call it) based on this: http://developer.rhino3d.com/api/RhinoCommon/html/M_Rhino_Geometry_LinearDimension__ctor_1.htm
But I have no idea how it would be constructed in other platforms (Revit, etc.), no not sure how to make something properly generic.
Other objects to look at in Rhino are also Hashes and Lights


Search feature is broken

The search feature broke on this commit: d6b5783639454b8d05b85fad3387bc790926df9d
I had originally suspected we we're runnint into a timeout error for the search do to the increasing content. I now think this is unlikely and that the issue has to do with the text in the carriage return text.

Update, my  suspicion is confirmed, adding "\n" within any of the posts will break the search functionality.


Stop using YouTube for notifications

As YouTube doesn't send notifications on stream starts and Hololivers don't always give the correct start time.
It might be an idea to use the official Hololive schedule website. This will probably need to be done by scraping.

Options:

TeepaBlue/holo-schedule-CLI
suisei-cn/pystargazer
cst0601/holo_schedule_api
YunzheZJU/non-stop-story Has a hosted version available
holofans/holoapi Also has a hosted version available
Choooks22/vt-api

Options:

TeepaBlue/holo-schedule-CLI
suisei-cn/pystargazer
cst0601/holo_schedule_api
YunzheZJU/non-stop-story Has a hosted version available
holofans/holoapi Also has a hosted version available
Choooks22/vt-api


Login popup doesn't appear

I'm not using the built in interface but instead building one from scratch and I am having a lot of trouble getting the authentication flow to work. It gets caught in this state where I am not authenticated and the popup doesn't appear. It detects authentication if I am logged into a connect instance in another tab. If I am not logged in the popup rarely displays, or if I log out of the other tab and refresh it doesn't popup.
When it is in this state, there is a looping of calls happening every few seconds to https://.my.connect.aws/connect/login and https://.my.connect.aws/login

you can use local storage clear, it will help you

Hi, we just ran into this too. The problem seems to be the following mechanism in Streams:
connect.PopupManager.prototype.open = function (url, name, options) {
  var then = this._getLastOpenedTimestamp(name);
  var now = new Date().getTime();
  var win = null;
  if (now - then > ONE_DAY_MILLIS) {

https://github.com/amazon-connect/amazon-connect-streams/blob/master/src/util.js#L517
If the user has triggered the login popup in the last 24 hours, and the user hasn't set a loginUrl, then the Streams lib will not open the login popup no matter what you do. The only workaround is to wait 24 hours, or login using the CCP-V2 directly, or clear your localStorage.
// clear out last opened timestamp for SAML authentication when there is ACK_TIMEOUT
if (params.loginUrl) {
  connect.core.getPopupManager().clear(connect.MasterTopics.LOGIN_POPUP);
}

https://github.com/amazon-connect/amazon-connect-streams/blob/master/src/core.js#L878-L881
This seems to be a bug in Streams SDK since it's pretty easy to get into this state.
There is an additional clearing of the timestamp in localStorage which happens here on ACKNOWLEDGE from the shared worker: https://github.com/amazon-connect/amazon-connect-streams/blob/master/src/core.js#L893-L902, but this is subject to race conditions and timing errors - if the JavaScript execution is killed with bad timing, the browser can get into a situation where it will "lock out" the agent from getting back.

Why is there a 24 hour lockout period for the popup window?
Could this entire system be reworked to be more resilient to failures?
If not, perhaps there are other circumstances where clearing the timestamp in localStorage would be good. For example, it would be a good idea to clear it when refreshing the browser. Even choosing sessionStorage might be better.


"Missing google_app_id. Firebase Analytics disabled." when running installed app of Analytics sample.

Hi,
I'm facing the problem as I wrote in the title, with the latest version of Firebase.
The instant app works fine and I can find myself immediately being tracked on StreamView. However when running the installed one, the error came out and of course nothing appears on StreamView.

03-26 10:00:29.308 13498-13514/com.example.android.instant.analytics E/FA: GoogleService failed to initialize, status: 10, Missing google app id value from from string resources with name google_app_id.
03-26 10:00:29.318 13498-13514/com.example.android.instant.analytics E/FA: Missing google_app_id. Firebase Analytics disabled. See https://goo.gl/NAOOOI
03-26 10:00:29.324 13498-13514/com.example.android.instant.analytics E/FA: Uploading is not possible. App measurement disabled

Anyone has ideas?

The root cause seems to be that Firebase hasn't supported Instant Apps yet. So for now, a workaround is to manually add the google_app_id into the feature module as a string resource.
Thanks for the support from Firebase team.

@scm573 could you be more specific? you mean in the manifest? In the build.gradle file of the instant app?
TIA


Revert "Development/trace buffers"

Reverts rdkcentral/ThunderNanoServices#52

https://jenkins.code.rdkcentral.com/job/github-triggered/782/ started


Fix null inner exception when retrying on WebException

When using connection pooling (and retrying), WebExceptions fall out of the normal exception handling in RequestHandler and weren't being added to SeenExceptions, ultimately causing MaxRetryException.InnerException to be null.
Closes #1176

LGTM :+1:


Testen: Aktion Stromcheck Edit mode off dann 404

Could you please post in english? I'm going to close this. Feel free to reopen with english text


The signing key's size is 208 bits which is not secure enough

This is what I get in the GUI after trying the Provision certificates for your data nodes step:
THE SIGNING KEY'S SIZE IS 208 BITS WHICH IS NOT SECURE ENOUGH FOR THE HS256 ALGORITHM. THE JWT JWA SPECIFICATION (RFC 7518, SECTION 3.2) STATES THAT KEYS USED WITH HS256 MUST HAVE A SIZE >= 256 BITS (THE KEY SIZE MUST BE GREATER THAN OR EQUAL TO THE HASH OUTPUT SIZE). CONSIDER USING THE IO.JSONWEBTOKEN.SECURITY.KEYS CLASS'S 'SECRETKEYFOR(SIGNATUREALGORITHM.HS256)' METHOD TO CREATE A KEY GUARANTEED TO BE SECURE ENOUGH FOR HS256. SEE HTTPS://TOOLS.IETF.ORG/HTML/RFC7518#SECTION-3.2 FOR MORE INFORMATION.
in console:
datanode  | Caused by: io.jsonwebtoken.security.WeakKeyException: The signing key's size is 208 bits which is not secure enough for the HS256 algorithm.  The JWT JWA Specification (RFC 7518, Section 3.2) states that keys used with HS256 MUST have a size >= 256 bits (the key size must be greater than or equal to the hash output size).  Consider using the io.jsonwebtoken.security.Keys class's 'secretKeyFor(SignatureAlgorithm.HS256)' method to create a key guaranteed to be secure enough for HS256.  See https://tools.ietf.org/html/rfc7518#section-3.2 for more information.

@8666 Hi, do you use the .env file for your settings?  I used my own password generator.

Tried 5.2 then 5.2.3
The console error was from datanode. The problem is that I did not read the whole .env file .. or the instructions are too long.
The setup should be doable without looking at the console long for the very first password
Also it is not clear what password should be used after you finish the setup. I created also a long 2nd password for GRAYLOG_ROOT_PASSWORD_SHA2
so I have a very long web admin password :)


feat(chrome-extension): Poll for new active sessions

Description
Polls for active session changes allowing for synchronizing newly authenticated users after the extension has been opened.

Fixes SDK-1843
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[ ] (If applicable) Documentation has been updated

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

LGTM

I wonder if we have checked that the opposite flow works. User logins to chrome extension and session is synced to website.

It's not possible in this fashion. We could, in theory, set the cookies to the permissioned domains from the extension.


Namespace needed

The lack of a namespace makes it difficult to reuse this code in complex applications.

Been meaning to implement it, thanks for giving me an excuse to do so ^^

https://github.com/DanielEverland/ScriptableObject-Architecture/releases/tag/v1.5.4

Nice. Thank you.


Update grammar

as title.

Note that use the newest SNAPSHOT by ./gradlew build --refresh-dependencies (note for myself)

Great thanks!! I was doing the same thing by deleting all files under ~/.gradle before!!!

bors merge


rename in bower.json

The module is registered to Bower as "angular-recaptcha", but the {{bower.json}} had name of {{vc-angular-recaptcha". This fixes that. If you prefer to register to Bower as "vc-angular-recaptcha" instead or as well, that is also an option.

Thanks @adjohnson916 !


PS Move Controller Not Detecting

The PSMoveService launches and the camera works, but the software (or maybe PC) does not properly detect the move controller. Don't know what to do.
PSMoveService Text
[2016-07-22 15:45:24.797530]: main - Starting PSMoveService
Config file name: "C:\Users\Ryan\AppData\Roaming\PSMoveService\ControllerManagerConfig.json"
[2016-07-22 15:45:24.813158]: bluetooth_get_host_address - Found a bluetooth radio
[2016-07-22 15:45:24.813158]: bluetooth_get_host_address - Retrieved radio info
Config file name: "C:\Users\Ryan\AppData\Roaming\PSMoveService\TrackerManagerConfig.json"
[2016-07-22 15:45:24.954215]: PS3EyeTracker::open - Opening PS3EyeTracker(USB\VID_1415&PID_2000\6\0, camera_index=0)
ps3eye::PS3EYECam::getDevices() found 1 devices.
Config file name: "C:\Users\Ryan\AppData\Roaming\PSMoveService\PS3EyeTrackerConfig_ps3eye_b2_p3.json"
Config file name: "C:\Users\Ryan\AppData\Roaming\PSMoveService\PS3EyeTrackerConfig_ps3eye_b2_p3.json"
Config file name: "C:\Users\Ryan\AppData\Roaming\PSMoveService\PS3EyeTrackerConfig_ps3eye_b2_p3.json"
[2016-07-22 15:45:25.129548]: SharedMemory::initialize() - Allocating shared memory: tracker_view_0
[2016-07-22 15:45:25.145196]: DeviceTypeManager::update_connected_devices - Device device_id 0 (PSEYE) opened
[2016-07-22 15:45:28.153011]: ClientConnection::start - Starting client connection id 0
[2016-07-22 15:45:28.155013]: ClientConnection::send_connection_info - Sending connection id to client 0
PSMoveConfigTool
ERROR [OpenVRContext::startup] Failed to initialize OpenVR: Hmd Not Found Presence Failed (126)!
INFO [App::init] Failed to initialize OpenVR!
INFO [Renderer::init()] Initializing Renderer Context
[INFO] ClientNetworkManager::start_tcp_connect - Connecting to: <IP_ADDRESS>:9512...
[INFO] ClientPSMoveAPI - Successfully initialized ClientPSMoveAPI
[INFO] ClientNetworkManager::handle_tcp_connect - Connected to <IP_ADDRESS>:9512
[INFO] ClientNetworkManager::handle_tcp_read_response_header - Received Message Header
[INFO] ClientNetworkManager::handle_tcp_read_response_body - Received Response Body
[INFO] ClientNetworkManager::handle_tcp_response_received - Received notification type 1
[INFO] ClientNetworkManager::handle_tcp_connection_info_notification - Got connection_id: 0
[INFO] ClientNetworkManager::send_udp_connection_id - Sending connection id to server over UDP: 0
[INFO] ClientNetworkManager::handle_udp_write_connection_id - Successfully sent UDP connection id
[INFO] ClientNetworkManager::handle_udp_read_connection_result - UDP Connect Success!
[INFO] handle_server_connection_opened - Connected to service
[INFO] ClientNetworkManager::handle_tcp_read_response_header - Received Message Header
[INFO] ClientNetworkManager::handle_tcp_read_response_body - Received Response Body
[INFO] ClientNetworkManager::handle_tcp_response_received - Received response type 2

What happens when you try to pair the controller?

It won't even show it in the service. Or bluetooth. The pc detects the
controller as HID compliant keyboard and mouse. As I looked into it some of
the drivers it auto installer were motioninjoy but I removed that.
On Jul 22, 2016 3:51 PM, "Chadwick Boulay"<EMAIL_ADDRESS>wrote:

What happens when you try to pair the controller
https://github.com/cboulay/PSMoveService/wiki/PSMove-Bluetooth-Pairing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cboulay/PSMoveService/issues/129#issuecomment-234639829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATph9hQRdrb9Wkx4AFUSQQLaYO7-cevuks5qYR9fgaJpZM4JTEsB
.

On Fri, Jul 22, 2016 at 3:53 PM, Ryan Silva<EMAIL_ADDRESS>
wrote:

It won't even show it in the service. Or bluetooth. The pc detects the
controller as HID compliant keyboard and mouse. As I looked into it some of
the drivers it auto installer were motioninjoy but I removed that.
On Jul 22, 2016 3:51 PM, "Chadwick Boulay"<EMAIL_ADDRESS>wrote:

What happens when you try to pair the controller
https://github.com/cboulay/PSMoveService/wiki/PSMove-Bluetooth-Pairing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cboulay/PSMoveService/issues/129#issuecomment-234639829,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATph9hQRdrb9Wkx4AFUSQQLaYO7-cevuks5qYR9fgaJpZM4JTEsB
.

Do you mean that when you plug it in via USB cable, Windows installs it as a KB+M, and PSMoveService can't detect it?
When I plug my PSMove into my Windows 10 PC via USB windows installs it as a "Motion Controller".
My guess is that you didn't fully uninstall MotionInJoy. You probably have some .inf's lying around that are being used by Windows whenever you plug in the device. I think MotionInJoy is difficult to uninstall fully.

Ah okay thank you  I'll try. I'll get back to you.
On Jul 22, 2016 4:11 PM, "Chadwick Boulay"<EMAIL_ADDRESS>wrote:

Do you mean that when you plug it in via USB cable, Windows installs it as
a KB+M, and PSMoveService can't detect it?
When I plug my PSMove into my Windows 10 PC via USB windows installs it as
a "Motion Controller".
My guess is that you didn't fully uninstall MotionInJoy. You probably have
some .inf's lying around that are being used by Windows whenever you plug
in the device. I think MotionInJoy is difficult to uninstall fully.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/cboulay/PSMoveService/issues/129#issuecomment-234644059,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATph9oap5n8Eo8MBe-PJuBavBrQfd7dCks5qYSPrgaJpZM4JTEsB
.

Forgot to get back, but uninstalling the drivers worked. Thank you!

Closing since it sounds like this is fixed for you. But feel free to continuing commenting in the issue if need be.

As far as I understand, the problem was that the PC was installed as a mothion controller device? If so, how do I fix it? Uninstalling the driver caused it to reinstall the device with the same name


vllm: handle max_length better and substitute Collator

Updated the max_length estimation to more closely mirror HF and handle case where the tokenizer has an implausibly large max_length. This allows the inputs to be truncated to the proper length: fixes #1234 and similar bugs where vllm will not provide log_probs if the inputs exceed its internally calculated model max length.
Also substituted Collator for Reorderer. Also fixed a lingering batch_size bug from #1229.

Have confirmed the outputs are the same on gsm8k and arc_easy!


Do you have any plan to make dense_multimap container ?

Hello!
entt has only dense_map, dense_set.
Do you have any plan to make dense_multimap container ?

I don't but it might be an option actually. Should be easy to adapt the dense map. Do you want to give it a try?

I am trying to make hash grid map for collision, so I need to use dense multi-map for the performance.

Should be easy to adapt the dense map.

I tried to implement this and it basically turned out that the underlying data structure (technically, a glorified sparse set) isn't a good fit to support multiple elements in a bucket based dense map.
This is true unless one is willing to make a lot of compromises in terms of performance and functionalities. This would make it a niche implementation in turn and I don't want to maintain something that is pretty much useless.
Therefore, the answer is no. A corresponding dense multi-map will not be implemented. I'm sorry.
I'll close the issue soon if there are no comments, contribution proposals or suggestions on alternative approaches.

@gamephysics take a look at https://github.com/greg7mdp/gtl 's flat_hash_map or parallel_flat_hash_map if those fit your needs.


Unexpected window swallowing

Regression?
Yes
System Info and Version
 System/Version info
Hyprland, built from branch  at commit 918d8340afd652b011b937d29d5eea0be08467f5  (flake.lock: update).
Date: Tue Jun 25 12:06:02 2024
Tag: v0.41.2, commits: 4886

flags: (if any)

System Information:
System name: Linux
Node name: bael
Release: 6.9.6-zen1-1-zen
Version: #1 ZEN SMP PREEMPT_DYNAMIC Fri, 21 Jun 2024 19:48:59 +0000

GPU information:
04:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Raven Ridge [Radeon Vega Series / Radeon Vega Mobile Series] [1002:15dd] (rev c5) (prog-if 00 [VGA controller])

os-release: NAME="EndeavourOS"
PRETTY_NAME="EndeavourOS"
ID="endeavouros"
ID_LIKE="arch"
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://endeavouros.com"
DOCUMENTATION_URL="https://discovery.endeavouros.com"
SUPPORT_URL="https://forum.endeavouros.com"
BUG_REPORT_URL="https://forum.endeavouros.com/c/general-system/endeavouros-installation"
PRIVACY_POLICY_URL="https://endeavouros.com/privacy-policy-2"
LOGO="endeavouros"

plugins:

Description
This is my window swallow configuration
misc {
  enable_swallow = true
  swallow_regex = ^(Alacritty|[Nn]eovide)$
}

In first instance, it's an expected behaviour of neovide swallowing the terminal, but when I try launching the markdown preview the neovide also disappears only leaving the markdown preview
how would I exclude that?
How to reproduce

Install neovide and alacritty
add the options above to your config
Install peek.nvim in neovim
Open neovide via terminal (the window will get swallowed)
Open peek.nvim to preview markdown in terminal

[!NOTE]
Same behavior is noticed when trying to open peek.nvim via neovim in terminal

Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/58597122/10cdc157-7145-4518-af06-26bf33c98de7

Previously alacritty didn't get swallowed by peek.nvim when opening via neovim in alacritty


add an option to bypass the document readability check

A check is done to test if a content can be cached but this test may disqualify a readable document so an option is needed to allow users to bypass this test and force bookmarks processing.
Try to fix https://github.com/go-shiori/go-readability/issues/26

Fix 1e407e22418de7caed98b697e1b0b6618e899cab


Notebook Instance Lifecycle Config timeout

The lifecycle config in the template referenced by another template in the identity graph solution here
Gave me the following error:
2020-05-13 15:35:37 UTC-0700	NeptuneNotebookInstance	CREATE_FAILED	Notebook Instance Lifecycle Config 'arn:aws:sagemaker:us-west-2:XXX:notebook-instance-lifecycle-config/neptunenotebookinstancelifecycleconfig-7fpg6wxfzxq4' for Notebook Instance 'arn:aws:sagemaker:us-west-2:XXXX:notebook-instance/neptunenotebookinstance-kuhtgb3nwlns' took longer than 5 minutes. Please check your CloudWatch logs for more details if your Notebook Instance has Internet access.
In the template could we just cat the stuff commands into a script, chmod +x it and then nohup that execution? or create a cloudformation wait condition somewhere?

@triggan Do you mind taking a look at this one?

@ranman - Yep, that's actually a good idea (re: build those items into a shell script).  Just for others that may find this, the referencing template that you mention is part of a Neptune-SageMaker stack that we've created as a foundational stack for multiple use cases.  The original template is located here.
We've run into some sporadic issues in the past with downloads of both RDF4j and the TinkerPop Gremlin Console using slow mirrors and holding up the notebook install.  While neither of those libraries are directly used for this sample workload, they are useful for troubleshooting.  The RDF4j package is only downloaded, so we previously nohup'd the wget command to fetch that.  The Gremlin Console, however, is actually unzipped and pre-configured for use against the Neptune cluster deployed in this stack.
To your point, I think we should be able to get around some of these slow mirror issues by putting all of those shell commands into a shell script and nohup'ing the entire script.  I'll work back with the team to do that and fix the upstream version of the template.


Ho wot get output string in Codesearch

Hi,
I am an researched on code retrieval and I want to reproduce your work for research purpose.
I fine-tuned your codesearch model and I run the run_classifier.py to get the test result.
But All I can see are two numbers, where is the generated code? How can I get the code sequence instead of two numbers?

Thanks!

Please refer https://github.com/microsoft/CodeBERT/blob/master/GraphCodeBERT/codesearch/README.md#model-and-demo

Thanks for the Clarification and Demo link!. It can be a good idea to add a little explanation on the read me file so that other people will not asking this question any more.


Background low-power / significant location change Geolocation updates

An application we are building has a need for Foursquare style persistent low power / low accuracy location updates.
In our all native app, we are using CLLocation::startMonitoringSignificantLocationChanges to do this.
I'd like to propose that we add this to Geolocation:

Add a "background" option that translates to asking for requestAlwaysAuthorization on iOS.

then either:

Redefine the "enableHighAccuracy" option to use startMonitoringSignificantLocationChanges instead of startUpdates when this is false. This seems to be the intent of what the W3C spec was looking for here in terms of providing a bool for this.

-or-

Add a "networkBasedUpdates" boolean option that when true uses startMonitoringSignificantLocationChanges on iOS and LocationManager.NETWORK_PROVIDER on Android.

I think I understand the code well enough to provide a PR for this.  Let me know your thoughts.

+1 for this feature

Redefining enableHighAccuracy makes sense, and then potentially exposing seperately an accuracy option.
Because we are following the html geolocation api, this seems to be about the best mapping possible. If someone needs a more ios specific solution, I imagine that could live as a seperate module.
I implemented specifically the background(mode) option in https://github.com/facebook/react-native/pull/1506

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out it's new home: https://productpains.com/post/react-native/geolocation-background-low-power-significant-location-change-geolocation-updates

+1

Background geolocation update function is crucial to my app too. A feature like this will be awesome. Avoids all the Java native code to run a geolocation update service. Are there any updates on this?

+1
I need a background geolocation feature too, but I want avoid writing any Java code. Some progress on that?


Zoma/move

Add the integrate work item into chrimc/move branch

Delete file

Refers to: experimental/rush.json:16 in 6e6e47d. [](commit_id = 6e6e47d92cbd4c7b552b6a8bca233d721e7716ca, deletion_comment = False)

dependencies:

Delete file

Refers to: experimental/common/config/rush/pnpm-lock.yaml:1 in 6e6e47d. [](commit_id = 6e6e47d92cbd4c7b552b6a8bca233d721e7716ca, deletion_comment = False)

Once you get the test to pass, you can check-in.


Added an option to allow rouletting while the streamer is on/offline

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable
[x] Documentation in docs/ or install-docs/ was updated, if applicable

Added in a dropdown menu to clean module settings up

Thank you very much!

Thank you very much!


Toggle Tab Key Moves Focus Stuck

Type: Bug
On two different computers I have accidentally seemingly pressed the shortcut for "Toggle Tab Key Moves Focus" and it is impossible to turn off. If I press the shortcut again, I can see that it "turns off" because the accessibility window in the bottom right where it says "Tab Moves Focus" disappears but the effects remain, and any time I press tab in editor it switches to a different window instead of inserting a tab.
VS Code version: Code 1.80.0 (660393deaaa6d1996740ff4880f1bad43768c814, 2023-07-04T15:06:02.407Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 2700X Eight-Core Processor          (16 x 3693)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
31.91GB (23.47GB free)

Process Argv
--crash-reporter-id 1e4235de-a239-4eb2-839e-73fc669626a5

Screen Reader
no

VM
0%

Extensions (27)

Extension
Author (truncated)
Version

vscode-javascript-repl
ach
0.7.4

jsoncrack-vscode
Ayk
1.0.8

speech
bie
0.1.0

compare-view
che
0.13.0

markdown-julia
col
0.0.1

vscode-markdownlint
Dav
0.51.0

vscode-dimmer
hoo
2.1.0

latex-workshop
Jam
9.12.3

language-julia
jul
1.47.2

octaveexecution
Luc
0.7.3

vscode-latex
mat
1.2.0

zotero
mbl
0.1.10

jupyter
ms-
2023.6.1001861915

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

remote-wsl
ms-
0.80.2

wordcount
ms-
0.1.0

quarto
qua
1.89.0

r
REd
2.8.1

LiveServer
rit
5.7.9

vscode-julia-formatter
sin
0.6.3

rmarkdown
Tia
0.0.14

octave
toa
0.0.3

JavaScriptSnippets
xab
1.8.0

markdown-all-in-one
yzh
3.5.1

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
a9j8j154:30646983
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
hideindicator:30785051
pythongtdpath:30769146
bgfeh915:30780428
pythonnosmt12:30779714
pythonidxpt:30784022
pythonnocebcf:30776496
e537b577:30786199

Reloading the window doesn't help? have you tried disabling all extensions?

I tried restarting but it didn't fix anything. I disabled all the extensions that I was actively using at the time (some may have remained enabled in the background, I'm not sure).
On the second computer that this problem appeared on, I happened to have a markdown tab open and pressing tab seemed to still work within a Markdown script. So repeatedly pressing tab would cycle through open windows until it found one that was in Markdown, then stayed there and made tabs like expected. Markdown All in One plugin doesn't seem to have any settings related to the tab key, though.
I uninstalled VSCode and switched to the VSCode Insiders and have deactivated the keyboard shortcut. The problems have not resurfaced so far but I have a lot on my plate at the moment so I'm not willing to spend time deliberately breaking my VSCode to recreate the issue, sorry.
Will update with more info if I accidentally do it again !

I've had the same problem!
It occurred after upgrade vscode to  1.80.0 (Universal) on my macOS 13.4
Meanwhile, the store and debug button on left bar  always shows the clock icon  connecting to server

and you are encountering this with all extensions disabled? if so, can you run the toggle keyboard shortcuts troubleshooting command, use the keyboard shortcut to toggle tab key moves focus, and share the output?

I once again pressed the hotkey accidentally but because I'm now using VSCode Insider, the problem isn't persisting, so I guess this means that in future releases this won't be an issue?


display DataLoader as materializer in viz

The DataLoader doesn't match the DataSaver display

How hard would it be to add a raw mode? E.G> one that just displays the graph in a plain way?

By "raw", you mean "no formatting"? It think this should already be feasible through the exposed graphviz_kwargs and render_kwargs, but would have to double check
The PR for this change was merged though.

How hard would it be to add a raw mode? E.G> one that just displays the graph in a plain way?

that's a different issue than this one. So closing this as completed.


Draft Kata visibility

Why are "Draft" Kata descriptions visible to anyone but the author prior to Kata publication?
I had been using this draft mechanism to jot down ideas for future Kata as they occur to me... but my draft thoughts are not meant for public exposure until I publish them...

If you're talking about "Wise drunk man" kata, that was published earlier. The author unpublished all of his/her authored kata. I can access them from "Completed" tab, but it's not publicly listed.
I don't think "Draft" kata are visible to anyone but the author prior to Kata publication.

Actually, it might be possible for non-author to view a "Draft" kata that's never been published if you know the slug or id of the kata somehow. For example, maybe a friend authored a kata and sent you a link asking for your opinion.
If you're suggesting to completely restrict the access whenever the status is "Draft", I'm not sure if that's a good idea. I've helped authors to resolve issues in "Draft". It's useful to prevent unnecessary down votes while fixing the issue.

If you're talking about "Wise drunk man" kata, that was published earlier.

Not specifically - that was the one that prompted the question though. When  I looked at https://www.codewars.com/users/AwelEshetu/authored I could see quite a few "Draft" Kata from that author. I was surprised that I could see them all. Are you saying I could see them all because they had once all been published? Anyway - now there is nothing there at all. Looks like they removed the whole lot (???)
I also have a number of "Draft" Kata which might turn into real Kata one day or maybe not. Are you able to see them or not?
https://www.codewars.com/users/dinglemouse/authored

@dinglemouse2250 I am currently not logged in to Codewars but I just clicked on your link and I can't see a single "Draft" Kata.  Hope that addresses your concern :smile:

Are you saying I could see them all because they had once all been published?

No, I just meant that they were published earlier today and unpublished not too long ago. You probably saw them because it was still being unpublished.

Are you able to see them or not?

I don't see any "Draft" and the total published kata is 27.

I got muddled looking at all those other Draft ones that apparently were being being changed at the same time I was looking at them. I am still confused exactly what happened...

I guess I could see the Draft ones only because they had previously been published?
I guess I can't see anything now because the have been deleted?

Anyway and long as draft/unpublished is not visible there is no issue here. Closing this. Sorry for the confusion

@dinglemouse2250

I guess I could see the Draft ones only because they had previously been published?

Like I said, I think they were still being unpublished. Maybe it took longer to be fully unpublished because of the background job queue issue. I don't know the details of how jobs are handled, but the stats aren't refreshing for the past few days. The publishing job has higher priority according to #731.
Also, people can access the Draft if they know the slug or id of the kata. This is what my second comment was about. So I wouldn't assume secrecy if it was once published and was seen by others. Google seems to know about it too.

If Google search shows your Draft that's never published, I'd open a new issue.

I guess I can't see anything now because they have been deleted?

They're not deleted. I can still access them from my "Completed" tab. "Wise drunk man" is back to Beta.


[Feature Request] Snapshot of the full screen as a fallback

Firstly, just wanted to say what a fantastic package this is. I've thoroughly enjoyed working with it the past couple of evenings.
I've struggled to take a snapshot when trying trying out a visual automation. I've tried coordinates, passing over an image with a transparent background and it often stalls. As a backup, it would be great to be able to pass in 'None' to the first parameter and take a screenshot of the whole desktop.

Thanks @olimorris! use snap('page.png') might be mentioned in docs but only briefly


add missing include statement

@addaleax @gribnoysup @mcasimir
This issue prevents building compass on a mac with an arm cpu running linux.
Is there something we can do to help getting this merged?

thanks for the ping! published in 1.2.1 👍


Update user

Noticed a bug where updating user info but leaving the password field empty actually changed the password to empty string for that particular user. This PR will fix that by looking at the length of the input values before adding them to the PUT request.
Also, refactored the use of ref attributes to use state and be more compliant with react recommendations.

Hey! @mikaelgson
First of all, love your handleChange :heart_eyes:
I'm however unable to reproduce the issue. There is a check on the backend that checks if the password is empty. If it is, then it just doesn't update that.

Hey! @mikaelgson
First of all, love your handleChange :heart_eyes:
I'm however unable to reproduce the issue. There is a check on the backend that checks if the password is empty. If it is, then it just doesn't update that. Can you which version of Matterwiki you're on?
Also, could you try and pull from the refactor branch and target your refs and other fixes to that.
A lot has changed.

@nshntarora Thanks :) I was running the latest master branch. Fetched the latest from refactor branch and  i can reproduce the bug there as well. Seems to me the backend problem is here: https://github.com/Matterwiki/Matterwiki/blob/refactor/api/users.js#L102
When i debug, the password is received as an empty string "" and will pass as truthy in req.body.password != null check. Checking for type and length should fix that. Something like: typeof req.body.password === 'string' && req.body.password.length > 0.
Should i make a new PR to refactor with both those changes? Seems like some code duplication could be avoided there as well.

@mikaelgson Yes please. It'd be great. Thanks :smile:

Aren't we merging this? I'd hate to have passwords becoming empty when people update their profile.


chore: release main

:robot: I have created a release beep boop
app-info: 1.0.0
1.0.0 (2022-08-22)
Bug Fixes

refactor to centralise environment variables (d8837f5)

log-error: 1.3.4
1.3.4 (2022-08-22)
Bug Fixes

refactor to centralise environment variables (d8837f5)

Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/app-info bumped from ^0.0.0 to ^1.0.0

@dotcom-reliability-kit/middleware-log-errors: 1.2.5
Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/log-error bumped from ^1.3.3 to ^1.3.4

middleware-render-error-info: 1.1.3
1.1.3 (2022-08-22)
Bug Fixes

refactor to centralise environment variables (d8837f5)

Dependencies

The following workspace dependencies were updated

dependencies

@dotcom-reliability-kit/app-info bumped from ^0.0.0 to ^1.0.0
@dotcom-reliability-kit/log-error bumped from ^1.3.3 to ^1.3.4

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/app-info-v1.0.0 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/log-error-v1.3.4 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/middleware-log-errors-v1.2.5 :sunflower:

:robot: Release is at https://github.com/Financial-Times/dotcom-reliability-kit/releases/tag/middleware-render-error-info-v1.1.3 :sunflower:


update fallible-iterator to version 0.2.0

This updates the fallible-iterator dependency to version 0.2.0 :slightly_smiling_face: Was running into some weird problems when resolving https://github.com/rustwasm/twiggy/pull/263, and afaict it seems to be because of some changes that happened regarding adaptors' closures in the new fallible-iterator version, noted in the changelog here.

Coverage remained the same at 86.68% when pulling 4cd10840951c8c2636ca0940217b791538364618 on data-pup:update-fallible-iterator into f1f641c73eaa3caa38ef673e89bffd08469512dd on gimli-rs:master.


Feature/societree

Society Tree Visualizer: The tool should represent a caste family tree with 72 nodes, where each node represents a primary family branch. Upon clicking on a node, it should expand to show the surnames associated with that branch, along with their respective deity and gotra information.
This JSON file contains information about different branches of Indian families, their Gotras (ancestral lineage), associated deities, and common surnames. Each entry includes:
Branch Name: The name of the family branch.
Gotra: The ancestral lineage associated with the branch.
Deity: The deity that the branch worships.
Surnames: A list of common surnames associated with the branch.

resolve this conflicts and is that new PR?

resolve the conflicts


fix: react native macos build

Fixes: https://github.com/lottie-react-native/lottie-react-native/issues/1018

/cc @Saadnajmi

/cc @Saadnajmi

Do we have a way to test macOS? What we've done in other repos is move the test app to use this (https://github.com/microsoft/react-native-test-app). That way, it is easy to add out of tree platforms like macOS and windows without needing to worry about maintaining extra boilerplate code.

/cc @Saadnajmi

Do we have a way to test macOS? What we've done in other repos is move the test app to use this (https://github.com/microsoft/react-native-test-app). That way, it is easy to add out of tree platforms like macOS and windows without needing to worry about maintaining extra boilerplate code.

Yeah this is what I was trying to go for but we somehow decided to not go with it that time.
Maybe it's a good time to move to RNTA.

Awesome! :D


its not much but its honest work - typos

A few quick typos that I could find. Looking forward to help more.

Thanks for your contribution, much appreciated 💯


Add ability to disable minifier

What
Add an option that can disable this plugin
Why
In some cases, a project may not want to enable the minifier. While it's possible to use things like Webpack Blocks or WebpackConfig to conditionally include plugins or to have multiple config files, for simple projects it is often easier to have a bit of config in the webpack.config itself. A prime example of this is the Webpack define plugin which is commonly used with "process.env.NODE_ENV": JSON.stringify(process.env.NODE_ENV). This PR allows for an override to simply not register the plugin.
How
Rather than disable the babili options, this PR prevents the Webpack plugin from registering in the first place if the minify: false override is set.

Is this a common pattern now? I still see a lot of
if (PROD) {
  module.exports.plugins.push(new ProdOnlyPlugin());
}

patterns. Can you link to other plugins that take an option to disable itself ?

If you don't look as specifically at Webpack plugins, the most prominent examples are Babel, Eslint and Webpack itself although I'm sure there are others. I found that tools are increasingly taking the config out of their core and letting 3rd parties define what it most useful. They each have their own way of having a set of recommended defaults but the important thing is that a particular set of config definitions is not part of the tool itself.
I've seen Webpack config modified in 100 ways and none of them is particularly graceful. The top three ways I've seen are:

One webpack config per env.
webpack.config.production.js webpack.config.development.js
Modifying a root config.import config from 'webpack.config.base.js';
if (production) {
  addMinifiers(config)
}

Composition using Webpack Blocks or WebpackConfig

I'm not trying to solve Webpack config management.  Hard problem. This PR is mostly meant to follow what I've found to be a very productive and useful trend that can help alleviate config management pain no matter how you choose to do it.
What I'd really love is to make this tool a pass-through like webpack-plugin-babel or webpack-plugin-eslint but I thought that changing the default behavior here would be too dramatic so I opted to add a flag. Making the default behavior better could be the next step if you're interested.
Even so, I see your point that not everybody is doing this (though not everybody is doing any one thing these days). This was just something that would make my life easier so I thought I'd share. Either way, thanks so much for your hard work and contributions!

Closing this after the reviews by others.


Adjust Makefile for actual code compilation

add possibility to configure gcc compiler depending on OS
add run build target in Makefile to run test application
remove device_canfd.c and test_cp_core.c from compilation because of
conflicting types for CpCore functions
additionally line 20 in RunAllTests() in test_cp_main.c have to be
removed to compile code without errors

Makefile needs to be modified in order to create different targets for functions and macros.


Fixed typo (atribute)

@hubalazs : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Responsive Navigation bar and new color palette, addition of bootstrap framework

KWOC22

KWOC22

Hey can you give me a preview of the changes you've done by attaching some pictures/video to the pr
thankyou!

KWOC22

Hey can you give me a preview of the changes you've done by attaching some pictures/video to the pr thankyou!

https://user-images.githubusercontent.com/99898464/207397464-ff089cca-0652-4345-aadf-73bd72f2f19a.mp4

do you need to label this pr as kwoc22 to make it count?


Add newtab page. Fixes #533. r=vp

Coverage remained the same at 42.018% when pulling cc420a994ec25ddeb64744aabd302a99795b2ae1 on bgrins:newtab into 49a8765b7d7b79db8bbb5358b6a5aa4b1450ac54 on mozilla:master.

OK, closing this for now.


[6.4 RC] Menu flex wrap

I've been testing out 6.4rc2 on a project and noticed 2 potential issues with the new menu styles.
I think we have the horizontal and vertical flex-wrap properties mixed up.
See https://github.com/zurb/foundation-sites/blob/develop/scss/components/_menu.scss#L163
I think horizontal should wrap, so that menu items nicely stack when there isn't enough space, and vertical should be set to nowrap.
Wanted to check with others before I made the change.

A CodePen example of the issue: https://codepen.io/brettsmason/pen/OgNMdB along with the fixed versions. I'll put a PR together for this tomorrow.


Adding Get/Set-CsTenantBlockedCallingNumbers.md

Adding
Get-CsTenantBlockedCallingNumbers.md
Set-CsTenantBlockedCallingNumbers.md

Docs Build status updates of commit 8aeceef:
:clock10: Full build: the average full build time is 12 min(s) 12 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 8aeceef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 8e10b77:
:clock10: Full build: the average full build time is 12 min(s) 12 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 8e10b77:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 53ad23e:
:clock10: Full build: the average full build time is 12 min(s) 12 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 53ad23e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Get-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

skype/skype-ps/skype/Set-CsTenantBlockedCallingNumbers.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Use port 1 as default port for host connectivity

Looks like an incomplete PR. Considering the latest one.


Using never to return from Area::getRoomById(...)

Folks,
why are we doing typing the return from Area::getRoomById with never?
getRoomById(id: string): Room | never {
	const room = this.rooms.get(id);
	if (!room) {
		throw new Error(`Area did not find Room with id: [${id}]`);
	}
	return room;
}

Never should be used for:

A function never returns (e.g. if the function body has while(true){})
A function always throws (e.g. in function foo(){throw new Error('Not Implemented')} the return type of foo is never)

And in this case we return (room) or we may (but not always) thrown an Error.
Shouldn't this be:
getRoomById(id:string): Room { ... }

I've seen never used to annotate the fact that the method returns an error but if that's an antipattern we can remove it

That would be my recommendation.
Typescript Deep Dive Book describes the use of never in detail.

Makes sense to me -- let's ditch it, then.
On Mon, May 17, 2021 at 2:57 PM Roger Sen @.***> wrote:

That would be my recommendation.
Typescript Deep Dive Book describes the use of never in detail.
https://basarat.gitbook.io/typescript/type-system/never
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Ranvier-TS/core-ts/issues/12#issuecomment-842555637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6GX6IHE3Y4KQTGOW2PEXTTOFRKBANCNFSM447B7QFA
.


Migrate to package:web

Is your feature request related to a problem?
Official Flutter documentation:

If you maintain a public Flutter package that uses dart:html or any of the other Dart SDK web libraries, you should migrate to package:web as soon as possible. package:web is replacing dart:html and other web libraries as Dart's web interop solution long-term. Read the package:web vs dart:html section for more information.

Dart core web libraries, like dart:html and dart:svg, are not supported when compiling to Wasm

https://dart.dev/interop/js-interop/package-web
Describe the solution you'd like
Update lib/wagmi_flutter_web.dart
Only in this file is dart:html used.
Import web
Before
// ignore: avoid_web_libraries_in_flutter
import 'dart:html' as html;

After
import 'dart:js_interop';
import 'package:web/web.dart' as web;

Update _completeOnReadyEvent
Before
void _completeOnReadyEvent(Completer completer) {
  const readyEventName = 'wagmi_flutter_web_ready';

  void readyEventListener(event) {
    html.window.document.removeEventListener(
      readyEventName,
      readyEventListener,
    );
    completer.complete();
  }

  html.window.document.addEventListener(
    readyEventName,
    readyEventListener,
  );
}

After
void _completeOnReadyEvent(Completer completer) {
  const readyEventName = 'wagmi_flutter_web_ready';

  void readyEventListener(web.EventListener event) {

    web.window.document.removeEventListener(
      readyEventName,
      readyEventListener.toJS,
    );
    completer.complete();
  }

  web.window.document.addEventListener(
    readyEventName,
    readyEventListener.toJS,
  );
}

Update _injectJavascriptModule
Before
Future<void> _injectJavascriptModule(String assetPath) async {
  final scriptPath = 'assets/packages/wagmi_flutter_web/$assetPath';

  final scriptNode = html.ScriptElement()
    ..type = 'module'
    ..src = scriptPath;
  html.window.document.getElementsByTagName('html')[0].append(scriptNode);
}

After
Future<void> _injectJavascriptModule(String assetPath) async {
  final scriptPath = 'assets/packages/wagmi_flutter_web/$assetPath';

  final scriptNode = web.HTMLScriptElement()
    ..type = 'module'
    ..src = scriptPath;
  web.window.document.getElementsByTagName('html').item(0)!.append(scriptNode);
}

If item(0) not work, try https://github.com/dart-lang/web/issues/281
Additional context
No response
Epic
No response

Thank you @lukassgramm for this issue and the suggested solutions.
It's true that we want to make this lib WASM-compatible, but we don't always have the skills to go all the way.
Any help is welcome
See: https://github.com/archethic-foundation/wagmi_flutter_web/discussions


Training from scrach

As the paper titled  "Rethinking ImageNet Pre-training" suggests, it is possible to train the generalized R-CNN network from scratch. Does this feature currently support it?

@fmassa
Is FREEZE_CONV_BODY_AT should be -1 to train from scratch instead of 0 as 0 will freeze stem network in resnet.py
def _freeze_backbone(self, freeze_at):         if freeze_at < 0:             return         for stage_index in range(freeze_at):             if stage_index == 0:                 m = self.stem  # stage 0 is the stem             else:                 m = getattr(self, "layer" + str(stage_index))             for p in m.parameters():                 p.requires_grad = False


Add LTF-inspired drift injection functions

Closes #125
Added

functions for injecting drift according to Dirichlet, tweak-one, and minority shift methods
accompanying tests/documentation

Details

excludes other drift injection techniques that may be implemented elsewhere in OS code

Checklist

[ ] tests added and passing
[x] docs added and building
[x] pandas support added

General comment on class join, I know you wrote this prior but took a second look at it.
Rereading the original DeSouza description of this method, the way they set this method up was to essentially introduce a new class that is not present in the training set. This type of drift injection may cause issues for users hoping to apply our concept drift detection algorithms, since ours are not capable of detecting a new class. Options:
1. We could only use this drift injection method for detecting data drift and allow a user to specify a new class label for these joined classes. Then, a data drift detector could be used to detect this new change in the distribution of the class feature. My main concern is this could negate the purpose of this method - a user would ideally use a class joining drift injection technique with a classifier and a concept drift detector.
2. We could just mention in the documentation for this algorithm that a user should only assign a label for the newly joined classes that is already present in the training data. Then, it would merge two classes together and it could be used with a learner / concept drift detector.
Thoughts?
tagging @tms-bananaquit for comments too

I think the approach in (2) is reasonable, and straightforward to implement with the code as it is now. We could default the new_class argument to the same value as class_1 (or class_2, whichever) if it's not specified, and tweak the docstrings to reflect that.
Right now, menelaus only has concept drift detectors that expect binary outcomes. I think the code might actually run for the multinomial case, but (usually) it'd be evaluating overall accuracy (predicted_class == true_class), which isn't necessarily desirable in the multinomial case. Inserting a new class label would mean accuracy would immediately tank on the modified classes, though, since the classifier can't spit out a label that's not in the training data. In that sense, the detection would probably fire, assuming that the novel class is prevalent enough in new data.
"Renaming" two classes to the same thing doesn't actually make sense with a binary classifier, unless there's already an additional preprocessing step, when training the classifier, that buckets all the classes into 0/1. I'm not sure we need to worry too hard about that one, though that might be a candidate for later experiments.

Oh hm everything is working apparently. I'd say this is ready for a new review @indialindsay, whenever you have time


Opta Force-Unlock

take care of the lint issue, add tests for the aws/gcp/azure get_terraform_lock_id to get over the 70% code coverage and looks great!

@juandiegopalomino Lint issues don't seem to be popping up on local.
As for the test cases, I am yet to add test cases for getting terraform lock id for various clouds.


Allow Users to Edit Extension Requests Before Approval

Issue Description

Currently, once a user raises an extension request, they are unable to edit it, regardless of its status. This issue creates problems when users need to update or correct information before it is  approved.

Expected Behavior

Users should be able to edit extension requests if they have not yet been approved.
Once an extension request is approved, it should become locked for editing.

Current Behavior

Users are unable to modify their request details, even if the request has not yet been approved.

Screenshots

Reproducibility

[x] This issue is reproducible
[ ] This issue is not reproducible

Steps to Reproduce

Raise an extension request as a user.
Try to edit the request before it is approved.
Observe that editing is allowed on UI but once save is hit the content is still not updated even though the request is still pending approval.

Severity/Priority

[ ] Critical
[ ] High
[ ] Medium
[x] Low

Additional Information

Checklist

[x] I have read and followed the project's code of conduct.
[x] I have searched for similar issues before creating this one.
[x] I have provided all the necessary information to understand and reproduce the issue.
[x] I am willing to contribute to the resolution of this issue.

if extension requests are made by the logged in user, for the user can we store the extension requests by the user as pending extension requests , and user can edit the pending extension requests and once approved we will update the extension requests.
But I didn't got what are the extension request you are referring to , I assumed this is any feature of the website(actually new, so didn't saw the actual feature), and just proposed a solution,


Migrate hellaswag benchmark to Inspect Evals

Reminder to pickup moved evals in examples.yaml as well

inspect_evals is a proper Python module (so easier to have dependencies for than standalone source files). Further, the evals can now be run by just referencing their name rather than their filesystem path:
inspect eval inspect_evals/drop

It's also easier for other Python code to import and use the tasks when they are in a proper module.
The plan is that everything goes into inspect_evals (there will be no more evals)


Further prevent degenerative joins in dpp_test [databricks]

Fixes #10182 and follow-on from #10168.
This adds one more fix required to prevent the join tests from being degenerative in dpp_test, which is to prevent a filter_val that only matches value <= 0 (since the UNION filters the fact_table with value > 0).

build


How did you generate depth maps and precomputed camera coordinate files?

The instruction says to download generated depth maps and precomputed camera coordinate files.
Did you generated those from the 7scenes sequence images?
If yes, how did you compute/generated?
Is there any code for that?

There is no code available for generating the depth maps, we used a custom renderer for that. We extracted a scene mesh from the TSDF volumes that come with the 7Scenes dataset (using ParaView) and rendered these using the ground truth poses.
Alternatively, we can also use the original depth maps of 7Scenes and register them to the RGB images, see eg. https://projet.liris.cnrs.fr/voir/activities-dataset/kinect-calibration.html
Generating camera coordinates from the depth maps is easy. Refer to the generation of scene coordinates in dataset.py and omit the transformation with the camera pose.
Best,
Eric

Can we get calibrated test depth images for 7scenes dataset. There are only calibrated depth maps for the training sequences in the given link.
Thanks for sharing your work.


class not found BeeEntity$PollinateGoal

MC version: 1.15.2
Fabric version: 0.7.6+build.1.7.9 (also tested 0.7.8+1.8.4)
Fabric API version: 0.4.32+build.292
Mods:

Fabric API
BeeProductive 1.0.2+1.15.1 (listed as 1.15.2 compatible on curseforge)

receiving the following error during game launch.
[00:41:51] [main/WARN]: Error loading class: net/minecraft/entity/passive/BeeEntity$PollinateGoal (java.lang.ClassNotFoundException: net/minecraft/entity/passive/BeeEntity$PollinateGoal)
[00:41:51] [main/WARN]: @Mixin target net.minecraft.entity.passive.BeeEntity$PollinateGoal was not found mixins.beeproductive.json:MixinBeeEntity$MixinPollinateGoal

maybe i am wrong but...
@Inject(method = "Lnet/minecraft/entity/passive/BeeEntity$PollinateGoal;stop()V", at = @At("HEAD")) this says to inject into the stop method inside the PollinateGoal class, that is contained within the BeeEntity class?
if so, i dont see a "stop" method in yarn, but i do see a "cancel" method. Am i missing something or misunderstanding?

Ling's PR seems to fix it


Fix "Error: Negative numbers should be created in combination with createPrefixUnaryExpression factory."

  [  Error: Negative numbers should be created in combination with createPrefixUnaryExpression

    - typescript.js:21097 Object.createNumericLiteral
      [app]/[parser]/[typescript]/lib/typescript.js:21097:13

    - factory.ts:461
      [app]/[@kubb]/parser/src/factory.ts:461:45

    - Array.map

    - factory.ts:457 Module.createEnumDeclaration
      [app]/[@kubb]/parser/src/factory.ts:457:20

    - typeParser.ts:312
      [app]/[@kubb]/swagger-ts/src/typeParser.ts:312:20

    - Array.forEach

    - typeParser.ts:310 typeParser
      [app]/[@kubb]/swagger-ts/src/typeParser.ts:310:17

    - SchemaGenerator.tsx:48 SchemaGenerator2.getSource
      [app]/[@kubb]/swagger-ts/src/SchemaGenerator.tsx:48:24

    - Schema.tsx:146 Component
      [app]/[@kubb]/swagger/src/components/Schema.tsx:146:28

    - react-reconciler.development.js:7478 renderWithHooks
      [app]/[react-reconciler@0.29.0_react@18.2.0]/[react-reconciler]/cjs/react-reconciler.development.js:7478:18

  ]
]
✖   Error: Debug Failure. False expression: Negative numbers should be created in combination with createPrefixUnaryExpression

Having negative enumerated values causes this error during generation.
Here are the lines that caused this error:
          application/json:
            schema:
              type: object
              properties:
                rating:
                  type: integer
                  enum:
                    - -1
                    - 0
                    - 1

This is a known issue. You can also take a look at this issue for OpenAPI-TS: https://github.com/hey-api/openapi-ts/issues/466

@dmitry-blackwave Thanks for the change, I was not aware that negative numbers were not working. Before merging can you run pnpm run changeset and create a changeset. This will bump the package and create a CHANGELOG.md when a new version is released.

@all-contributors please add @dmitry-blackwave for code


Support keyword-only args in UnpackArgs (i.e. for Built-Ins)

It would be nice if builtins could easily have keyword-only args. It would be nice of UnpackArgs also supported this.
Ways that come to mind:

UnpackArgs("f", args, kwargs, "a", &a, "*", "b", &b)
Pros: much like the current
Cons: "Pairs" is no longer just pairs
UnpackArgsWithKwonly("f", args, kwargs, 1, "a", &a, "b", &b) // new param is # of non-kwonly args
Pros: pairs are still pairs
Cons: easy to forget to update, requires new function, etc.

Hi Dan, sorry for the delay. Yes, this would be a nice feature, as most application built-ins seem to want most of their parameters to be keyword-only.
Currently we encode one extra bit into the name string: "x" means a mandatory parameter and "x?" means an optional one. Perhaps we could use "x=" to mean a mandatory keyword-only parameter and "x=?" to mean optional keyword only.  This is only the start of a design though; we need to think through the interactions carefully. Ignoring the *args and **kwargs residuals for now, the general schema for a Python3 function consists of three tuples divided by / and *:
positional-only /  positional-or-named * keyword-only
with each tuple having zero or more ident (mandatory) followed by zero or more ident=expr (optional) parameters. I wonder how close we can get to expressing the full schema by encoding extra bits in each parameter name, in the style of "x?", without excessive complexity, and ideally while remaining compatible with the existing UnpackArgs function.

Ah, yeah, I like adding the = tag to mean keyword-only, and then giving you the full set.
I didn't realize python3 added positional-only args.


Type check oddity

Its not high priority, and possibly not even an issue of your code, but I installed rc-3 and get an eslint error:
ERROR in c:/some/path/node_modules/@vueuse/core/dist/index.d.ts(351,17):
351:17 Cannot find name 'Coordinates'.
    349 | declare function useGeolocation(options?: GeolocationOptions): {
    350 |     isSupported: boolean | undefined;
  > 351 |     coords: Ref<Coordinates>;
        |                 ^
    352 |     locatedAt: Ref<number | null>;
    353 |     error: Ref<{
    354 |         readonly code: number;
Version: typescript 4.1.2

Do you have "lib": ["DOM"], in your tsconfig.json? Can you share a minimal reproduction? Thanks

I had the same problem. Since TypeScript 4.1.2, Coordinates has been removed and replaced with GeolocationCoordinates.

@shinnosuke-ooyama That was useful, thanks! Just updated the deps and types, it should work on rc.4. (and hope we don't break TS<4.1?)

For TS <4.1:  Cannot find name 'GeolocationCoordinates'.

Tried to make a minimal repro, but it took too long. You fixed it already, thanks!

@DarkXanteR If you don't want to bump your TS version, maybe you can set this in tsconfig.json
{
  "compilerOptions": {
    "skipLibCheck": true
  }
}


How about process field data before access?

Every field can define a process function, just like validate, used like this:
name = StringField()

def process_name(self, field):
    return field.data.capitalize()

So, when user input foo, we get Foo.

That's confusing because it does something different from the process_* methods and also basically does what filters already do


View Directories

Currently we just error that this file doesn't exist if we edit a directory, but it would be cool to list all the files in the directory (at the very least)

Done!