Generic stringify for custom types

Is your feature request related to a problem? Please describe.
I’m created a higher level API to use custom types for an usage of my own. It worked perfectly up to version 1.9.2 but stopped working since 1.10.0. This is because I was using the undocumented ctx.doc.schema.stringify method just like that:
const superStringify = (value, ctx, onComment, onChompKeep) => {
  return (typeof value === 'object' ? '\n' + ctx.indent : '') +
    ctx.doc.schema.stringify(value, ctx, onComment, onChompKeep)
}

const someTag = {
  // ...
  stringify(item, ...args) {
    const data = // get some arbitrary data from item
    return superStringify(data, ...args)
  }
}

Describe the solution you'd like
I'm pretty sure it was a hack, but it still saved my life.
I think it would be awesome if we could get a proper generic stringify method written for this that could take a plain JS object and render it properly in the current context.

Could you give an example of a JS object that you'd like to stringify, and the output that you'd like of it? It should be possible to achieve what you want without a custom stringifier, as the default one is applied automatically.

Yes sure. Here is a relatively simple use case : https://heptapod.lsystems.fr/lsystems/config.io/-/blob/a5b91f20acb571c4bb92e784d49a96c9f1f6d6ca/src/tags/path.mjs
The relevant part is there:
toYAML (data) {
return data.path.length > 1
  ? data.path
  : typeof data.path[0] === 'object' ? { value: data.path[0] } : data.path[0]
}

As you can see, the object has 2 representations and the serializer picks the most appropriate one depending on the context. It returns the correct representation as a plain JS object that is then serialized using the code I provided earlier.

Would it be appropriate to add a toJSON() method to the object that would perform this transform? That would get called automatically when the YAML node is created during the stringification.

You mean it’s already working?
Because I tried to put a toJSON method on my objects and it never got called. If it’s not implemented yet, I'd rather have a toYAML method that would work the same as the toJSON method. It would still be possible to have one that calls the other if we want to.

Perhaps I'm not understanding your usage pattern right?
The way it's intended to work, there are two steps when getting from a pure-JS representation to a YAML string representation of the data:

First, the data is mapped to nodes (doc.createNode(data)). This is the stage at which toJSON methods get called and a replacer function can do additional tuning.
Next, each node is stringified. If the node's tag includes a stringifier, that's called; otherwise stringifyString() is used for scalars, and collections have their toString() methods called.

The YAML.stringify() API is effectively a convenience wrapper that takes both of these steps in one go.
It's unclear to me if your source data is pure JS, or if it's meant to already be YAML nodes. If it's the former, then the node creation should allow for powerful tooling to shape the data as you want it. If it's the latter, then it's too late to apply its transforms, and indeed a custom stringifier ought to be used. For that, a custom tag for a scalar value could make use of the publicly accessible stringifyString() for scalar values. To format the output as a collection, your best bet would be to create a new temporary node, and to call its toString() method: doc.createNode({ value: 'foo' }).toString(ctx, onComment, onChompKeep).
Does that help, or could you describe in more detail how your intended system works?

I don’t use YAML.stringify(). Since the document was parsed from a file I want to retain its comments and such. So I create myself a Document using new and I then call doc.parse(YAML.parseCST(str)). To stringify I then use doc.toString().
Maybe it’s for that reason that I can’t use toJSON()? Wouldn’t it be better if the toJSON method was called later in the process? It’s a little disturbing that depending on how the stringification process is called you get different results with custom types. I think they are confusing right now and that it would be easier if they did have a more consistent behavior. I think I’m a rather correct dev, I read your codebase to get to my previous implementation, and I'm very lost with the internals there.

I just tried to update my stringify function so it reads like this:
if (typeof value !== 'object') {
  return stringifyString({ value }, ctx, onComment, onChompKeep)
}
return ctx.doc.createNode(value).toString(ctx, onComment, onChompKeep)

It throws an Error saying that ctx.doc.createNode is not a function. When logging ctx.doc it’s a proper Document. Any idea why?

Ah, sorry, that's the right function to call in yaml@1. The API changes in v2.

Ok thanks! It works like a charm to write this. Do you think there is something to improve over this use case − I think this bug is still valid and there is a need for some more consistent (& elegant) way to stringify custom types. If so I keep this bug open and I'm willing to discuss this here. If not I'm also okay to close it!

The difficulty here is arising from the way that the library assumes that a node in a document is either a scalar, a map, or a sequence, because those are the base types that YAML itself provides. And the nodes that you're stringifying want to get stringified as any one of those options, depending on their internal value. Therefore, a custom stringifier is indeed the right thing to use here.
I'm open to brainstorming about improving the API. It's tricky, really, because there are four stages that need to be considered:

node composition from CST
transform from node to native JS
node creation from native JS
node stringification

The current structure isn't really optimal for cases where one of these stages can have more than one different output, as you have here.
I'm happy that you're finding a workable solution, though! In the last code sample that you posted, it shouldn't be necessary to include the '\n' + ctx.indent prefix, but presumably the outputs aren't valid without that?

Yes. If the current stringifying process happens in a property of an object I get results like this:
test: !tagname attr1: value1
  attr2: value2

instead of:
test: !tagname
  attr1: value1
  attr2: value2

Back to the main topic, why not keep the JS custom type in the node and move the call to toJSON call in the stringification process, after the node creation? It would be a little more involved but maybe more powerful in the end?

You really made the right choice here. Anything else would be too complicated for the feat I'm afraid.
I think the only thing that might have helped me and would be great is to provide the function I gave you by default in the utils. Something like stringifyJS that would take any input and do the right thing without having to think much about it.

I left this open earlier in case I thought of a way of providing something like an easy-to-use generic node stringifier, but I'm going to close it as wontfix, at least for now. Practically speaking, it's too useful for the library to internally presume that all nodes will inherit from one of scalar, alias, map, or sequence.
It's certainly possible for an external user to implement a ploymorphic node (as is done here), and end up with the desired results, but I don't think it makes sense to add a utility function for that to the library, esp. as it would not be used by any internal tooling.


enable hermes by default

Summary
Hermes landed Proxy and supports most ES6 features, and IIRC, we talked and agreed to enable Hermes by default.
Changelog
[Android] [Changed] - enable Hermes by default
Test Plan
New projects will default to Hermes.

Closing as this has been implemented already


Feature Request for configuring buttons...

... to remove, modify, or add to the 2d6 d66 buttons. I imagine a checkbox, and image button, and a roll formula field.
I love the ability to roll with a single click. Thank you!

The feature is implemented in v0.6


[Fix-8187][UI] Add the function to the module of resource that you can re-upload the file on the page of the file management.

Purpose of the pull request
This PR will close #8187 .

Brief change log

Verify this pull request
This change added tests and can be verified as follows:

Manually verified the change by testing locally.
When the type of resource is file, you can re-upload a file to replace the file.

hi @calvinjiang I see we have a new functional added here. We should better add doc in repo https://github.com/apache/dolphinscheduler-website . The detail page in https://github.com/apache/dolphinscheduler-website/blob/master/docs/en-us/dev/user_doc/guide/resource.md and https://github.com/apache/dolphinscheduler-website/blob/master/docs/zh-cn/dev/user_doc/guide/resource.md Thanks


feat(CDN#guildIcon): implement dynamic logic

Please describe the changes this PR makes and why it should be merged:
Status and versioning classification:

Code changes have been tested against the Discord API, or there are no code changes
I know how to update typings and have done so, or typings don't need updating
This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

This needs a rebase @Fyko


fix race condition in ModelStruct.TableName()

[x] Do only one thing
[x] No API-breaking changes
[x] New code/logic commented & tested

What did this pull request do?
Fix race condition in ModelStruct.TableName(): https://github.com/jinzhu/gorm/issues/1407

sorry, i was too fast


Support HAPI v17

Hi, are there any plans to support HAPI v17? Thanks.

yeah I see no reason not to :) Can you send a pull request to make the relevant changes? I’ll release it as a breaking release

Cool. I try to find some time in the next days. :-)

Honestly, I am not event sure if and why this plugin is needed at all. Since Hapi v17 does have CORS support. Well, Hapi v16 did also have CORS support -- so, what is this plugin doing what is not provided by Hapi directly?

In the Hapi v17 release notes it says:

route.options.cors.origin can be set to 'ignore' which provides a CDN-friendly mode that ignores Origin headers and always responds with 'Access-Control-Allow-Origin' set to '*'.

Shouldn't that do the job?

I quickly tested with route.options.cors.origin and Hapi v17. Workling just fine. So I don't really see the need for this plugin with Hapi v17. Or am I missing something here?

I’m not sure, but if it works for you without, I guess it’s fine :) Do you have access-control-allow-origin configured to specific domains? This plugin sets it dynamically based on the origin header.  access-control-allow-origin must be set if you send any kind of authentication

No, I don't think so. You either say route.options.cors.origin = 'ignore' that tells Hapi to always respond with Access-Control-Allow-Origin set to *. Or you set a list of specific domains, e.g., route.options.cors.origin = ['example.com'], even with wildcards, which is the way to go in authentication use cases. See the docs: https://hapijs.com/api#-routeoptionscors. As far as I understand, you cannot configure Hapi out of the box to simply mirror the origin header into Access-Control-Allow-Origin. Not sure if that's really needed. If yes, we can easily update the plugin to support Hapi v17.

See https://github.com/gr2m/hapi-cors-headers/pull/11. Better make a new branch at your side and continue from there.


Feature: Port forwarding

How could we implement port forwarding?
In case I have multiple servers on 4001, 4002, 4003... and I would like to have them in project.dev:80, api.project.dev:80, api2.project.dev:80?
At the moment I'm using local Apache server with mod_proxy_(http)
How could I achieve this with Hotel?

Hi @blacktea,
You can pass URLs instead of commands:
hotel add --name project
hotel add http://<IP_ADDRESS>:4002 --name api.project
hotel add http://<IP_ADDRESS>:4003 --name api2.project

It should forward project.dev to  http://<IP_ADDRESS>:4001

You can also have your servers started by hotel on some specific ports too:
# if your server takes no port option
# or listens on PORT environment variable
hotel add 'command1' --port 4001

# if your server takes a port option
# and you want it to be set to 4002
hotel add 'command2 -p $PORT' --port 4002

Yes, but in the end I want all of them to be accessible from port 80.
And I cannot start 3 servers on port 80

To access them on port 80, you need to configure your browser or system proxy to use hotel's proxy.pac:
https://github.com/typicode/hotel/blob/master/docs/README.md
After that, you'll be able to access hotel.dev:80, project.dev:80 or simply hotel.dev, project.dev, ...

Yep I did this port configuration the first day, but chrome just kept redirecting me to localhost with specific port

Strange, what's your OS?

win 10 64 bit

thanks, will try to reproduce :)

but chrome just kept redirecting me to localhost with specific port

I was facing the same problem (exact same system, browser), and it's just Chrome HTTP answers caching. Clearing the cache solves the issue.


closes #661 and #737

@DrewAns please make sure the frontend portion of this ticket is completed as well


add CheckTabs.jl

Coverage remained the same at 100.0% when pulling 616ca17333fae069c2934e8fac47172496bb6ba2 on CheckTabs into 03735df117eee252da94cd12d9ac5d0c037ea4f1 on master.

Coverage remained the same at 100.0% when pulling 616ca17333fae069c2934e8fac47172496bb6ba2 on CheckTabs into 03735df117eee252da94cd12d9ac5d0c037ea4f1 on master.


Fixes #1 '$' Sign Jumping

This Fixes #1, consider this as a temporary fix, it is NOT the best solution.
The problem of the '$' jumping is caused by pointers interfering with each other,  the bug only appeared with the first element of the foods array, his x position was constantly changing. I just added a filter that sets a constant value to that bugged element. Every change is marked by a comment, so you can easily check.

@OhFrancy , Thanks for Contributing and the explanation.


Updated repositories.txt with new URL

The URL for the library has been intentionally changed since the repository has been transfered to a new maintainer.

Hello @8059blank. I have confirmed your request.
Unlike submissions, this type of request is still handled manually. The Library Manager index maintainer occasionally comes through to clear all the outstanding requests and closes those issues/PRs when that is done. You will get a notification from GitHub at that time.
After that, you can monitor the indexing of the library via its dedicated logs web page.
Information about that here:
https://github.com/arduino/library-registry/blob/main/FAQ.md#can-i-check-on-library-releases-being-added-to-library-manager

Backend work done :tada:
2022/03/04 15:08:37 Loaded 4676 libraries from DB
Changing URL of library Cytron_PikaBot from https://github.com/dbsqwerty123/CytronPikaBot.git to https://github.com/8059blank/CytronPikaBot.git
Success!


Source Of Truth

As we explore new hosts for the pod, it's going to be important to truly understand where each piece of data is living as the source of truth. Right now we have Libsyn and MD as duplicating the source of truth.
I've done some reflection on this and I'm not sure that .md being THE source of truth for the new site is the right way to go.
Specifically, having .md will make it hard to query by meta data. AKA, find me all episodes with a specific guest.
The big question becomes how do we maintain one location for all information, how do we keep that in sync with the pod host (if it ends up being Transistor) and how do we connect that to the website in a meaningful way to query and explore off of.

Keeping markdown as the source of truth is important. It makes it easy to update, and anyone can submit edits.
I picture up ingesting the data into a database on startup or build. When something changes, it gets pushed tot he db, and to the podcast host


backend dependency not available

Couldn't find the candb dependency mentioned in the package-set.dhall in the backend directory , This is the repo https://github.com/canscale/candb which is isn't available..

@RahulRupani8 Thanks for opening this issue!
The CanDB repositories do exist, but they are currently private, which is why you are currently unable to fully build the Supernova application.
This is because CanDB is currently in a closed alpha, through which I'm collaborating with over 15 projects to gather feedback in order to fix bugs and further refine the API before releasing CanDB publicly (hopefully late Q4 this year).
If you're interested in becoming an alpha participant, I recommend first watching the Supernova demo video https://www.youtube.com/watch?v=QSwwMRoPF0M and then reaching out to me via one of the avenues listed in the video if you think your project is a good fit.

Thanks for your response, I'll surely get back to you as soon as possible.
On Thu, 29 Sep, 2022, 01:27 ByronBecker, @.***> wrote:

Closed #1 https://github.com/canscale/supernova-candb-demo/issues/1 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/canscale/supernova-candb-demo/issues/1#event-7481055418,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2YUFFI4VOJ4KGGM4YM5GKDWASPKNANCNFSM6AAAAAAQXN3NB4
.
You are receiving this because you were mentioned.Message ID:
@.***>


Review

this review contains a couple of optimisations

These are all great additions!

Getting rid of manual .env hacking, great!
Feeding process.env to Vite instead of trying to get dfx auto generated code to play nice with import.meta, great!

Merging.


Circular dependency detected while autoloading constant Api

I'm using this gem together with the module ng-token-auth in my Ionic mobile app (which bases on angular).
After logging in, if I reload the page, a request auth/validate_token will be sent. I get a 'Circular dependency detected while autoloading constant Api' on that.
I'm also using the gem grape for building the API, which could also be the source of the problem.
This is my routes.rb file (devise is also included because I'm using it in my Rails app which contains the API):
Rails.application.routes.draw do
  devise_for :users, controllers: { omniauth_callbacks: 'omniauth_callbacks'}
  namespace :api do
    mount_devise_token_auth_for 'User', at: '/v1/auth'
    mount API => '/'
    mount GrapeSwaggerRails::Engine => '/doc'
  end
  (...)
end

There is no useful info in the server logs. How can I solve this problem?
Thank you!

Can you try upgrading to version 0.1.31.beta8? This looks like it may be related to an issue with namespace that was just fixed in that version.

that worked! thank you so much!


Re: 'How to use kubectl to connect to the cluster'

Just reporting a difference betwixt the screenshot (/images/github-authorize-moj.png) in the document, and that which I actually found.
I found this:

Thanks @MalcolmVonMoJ. Have raised a PR #561 to update the image.

Thanks @MalcolmVonMoJ. Have raised a PR #561 to update the image.

PR Merged

PR Merged


[Misc]: Does vllm support scenarios where multiple rounds of inferring the same model require loading the weights of that model only once?

Anything you want to discuss about vllm.
As the title write:
Sometimes i need to run the same model multiple times, but reloading the weights for that model each time is too time consuming. So i want to know if vllm can just load weight one time?
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

For offline inference, as long as the LLM instance remains in memory, you can keep using the same model weights in your code.


replace redundant commands in docker-compose files with generic one

Motivation: keep it DRY
Modifications: move redundant commands from 18-04-5.0 and 16.04-5.1 to the generic task definitions

LGTM, will let @weissi review as well.


feat: added TableOfContents components for About pages

Overview

Adds tables of contents to main About pages in Storybook to summarize the components in each section.

Includes a basic unit test that verifies those tables match up with
PR Checklist

~[ ] Related to designs:~
[ ] Related to JIRA ticket: GM-133
[x] I have run this code to verify it works
[x] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://5feb488c9463208ccb7284b0--gamut-preview.netlify.appDeploy Logs


nimble/mesh: Fix provisioning errors handling

Provisioning errors won't cause device to close link.
Upon error, device will send Provisioning Failed PDU, and any further
PDU's will be considered as unexpected. (Mesh Pofile section 5.4.4)
Also a timer starts every time device sends a correct PDU.

@jhedberg FYI

Is this causing some qualification test case failures?

@jhedberg yes
@szymon-czapracki  please update commit message with affected TS name

Looks good to me as well, I'll proceed with backporting these to Zephyr.


BUG: *syrk functions ignore overwrite option

There seems to be an overwrite option that doesn't work for these functions. Example below
>>> import numpy as np
>>> from scipy.linalg import blas
>>> a = np.random.random((100,100))
>>> c = np.zeros((100,100))
>>> blas.dsyrk(1.0, a, c=c, overwrite_c=True)
array([[ 33.19339651,  24.12943429,  26.71785963, ...,  24.73044741,
         22.97901315,  22.99444606],
       [  0.        ,  32.83715648,  24.07729365, ...,  24.73987735,
         23.78197391,  21.39989852],
       [  0.        ,   0.        ,  37.33024914, ...,  26.96535023,
         24.00661787,  22.97043113],
       ...,
       [  0.        ,   0.        ,   0.        , ...,  35.66241571,
         23.48535436,  22.59825865],
       [  0.        ,   0.        ,   0.        , ...,   0.        ,
         30.79795771,  21.66025167],
       [  0.        ,   0.        ,   0.        , ...,   0.        ,
          0.        ,  28.62412903]])
>>> c
array([[ 0.,  0.,  0., ...,  0.,  0.,  0.],
       [ 0.,  0.,  0., ...,  0.,  0.,  0.],
       [ 0.,  0.,  0., ...,  0.,  0.,  0.],
       ...,
       [ 0.,  0.,  0., ...,  0.,  0.,  0.],
       [ 0.,  0.,  0., ...,  0.,  0.,  0.],
       [ 0.,  0.,  0., ...,  0.,  0.,  0.]])

Generally, the overwrite options mean that the input array may be overwritten, not that they are guaranteed to be.
The functions always return the result arrays.

FWIW, the overwrite option requires that c be fortran ordered.  So
In [150]: a = np.random.random((5,5))

In [151]: c = np.zeros((5,5))

In [152]: linalg.blas.dsyrk(1.0, a, c=c.T, overwrite_c=True)
Out[152]:
array([[ 1.584339  ,  0.93694834,  0.77466281,  1.29040001,  2.10473892],
       [ 0.        ,  1.01464741,  1.00492064,  1.03302596,  1.74079086],
       [ 0.        ,  0.        ,  1.07764925,  0.90634314,  1.72147965],
       [ 0.        ,  0.        ,  0.        ,  1.32449458,  1.94092007],
       [ 0.        ,  0.        ,  0.        ,  0.        ,  3.57297992]])

In [153]: c
Out[153]:
array([[ 1.584339  ,  0.        ,  0.        ,  0.        ,  0.        ],
       [ 0.93694834,  1.01464741,  0.        ,  0.        ,  0.        ],
       [ 0.77466281,  1.00492064,  1.07764925,  0.        ,  0.        ],
       [ 1.29040001,  1.03302596,  0.90634314,  1.32449458,  0.        ],
       [ 2.10473892,  1.74079086,  1.72147965,  1.94092007,  3.57297992]])

Works just fine.  This is true for essentially anything that uses f2py to wrap fortran code.

Ok, so then I probably did something wrong. However, it would help if this could be documented. ( http://docs.scipy.org/doc/scipy/reference/generated/scipy.linalg.blas.dsyrk.html#scipy.linalg.blas.dsyrk )

Also, a warning could be generated if the order is wrong. I'd just like to get a clue as to what I am doing wrong.

Those are wishlist items for f2py (resides in numpy).
The documentation issue can maybe be resolved by adding general comments on this page:
https://docs.scipy.org/doc/scipy/reference/linalg.blas.html

Could you please list the related issues?

Added doc issue ( https://github.com/scipy/scipy/issues/5739 ).

I'm not sure whether there are existing issues for f2py for these.


Support Help Template overrides for Subcommands

What type of PR is this?

bug

What this PR does / why we need it:
When using the Subcommands, it is currently no possible to override the Subcommand Template that is used due to the Command being overwritten when initializing Context inside of the RunAsSubcommand function. I outline everything in #1303. The comment in the code details how I came to the conclusion I came to. Ideally, the Command would not be overridden but set to the Parent Command that houses the Subcommands that are being executed.
Which issue(s) this PR fixes:
Fixes #1303
Special notes for your reviewer:
This is a bandaid at best. I do not expect this PR to be merged. I might update it later with an actual fix, but I just wanted to test my hypothesis, I did that, this works, but ideally, you give Subcommand Context to the NewContext Command should the c.Command is initialized with the parent command instead of a empty *cli.Command
Release Notes
Adds Support for overriding cli.SubcommandHelpTemplate

@ddouglas Sorry for the big delay 😭 What do you think about adding a new field to App such as CustomSubcommandHelpTemplate instead of what you have here? This would mean bumping from a "bug" to a "feature" given the increased API surface area, but I think it'd result in fewer surprises in the future.

@ddouglas Can you add a test case as well ?

@ddouglas I'm going to close this and push another PR with the proper fix/tests.


Arcade Prize Additions

About the PR
Added several items to the possible rewards list and rearranged list to be alphabetized to the first letter.
Why / Balance
Its arcade prizes so I don't think it will affect balance much, if at all. I don't often see people play Space Villain but I get tired of receiving 3 griffin figures in a row, hopefully increasing the number of prizes will help with variety.
Technical details
just a yml edit
Media
NA
Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
none
Changelog

:cl:

add: Added more prizes to the Space Villain arcade machine

Pretty sure the singulo toy causes server lag and that's why it's reserved as a admeme item.

Pretty sure the singulo toy causes server lag and that's why it's reserved as a admeme item.

i'd believe that, it was behaving a little odd during testing. I'll remove it

This should probably be reworked to use entitytables

Mmm... plushie throngler should only be loto... I like it being super rare

Mmm... plushie throngler should only be loto... I like it being super rare

I'll remove that one too

Pretty sure the singulo toy causes server lag and that's why it's reserved as a admeme item.

i'd believe that, it was behaving a little odd during testing. I'll remove it

thought you said tesla for some reason.
I'd be fine removing the singulotoy and the telsa but I like to keep them if it doesnt cause problems.

also it might be good to split it up so that there's a 50% chance to get a plushie and 50% chance to get a "toy" or something similar

also it might be good to split it up so that there's a 50% chance to get a plushie and 50% chance to get a "toy" or something similar

I had considered switching the plushie spawns to a new "random plushie" item or something, same with figures (maybe only have it spawn the mystery box instead of single figures, but its was a quick PR so reworking it with entity tables and weighted spawning so could happen in the future.

Mmm... plushie throngler should only be loto... I like it being super rare

Winning that broken ass arcade machine is super rare :trollface:

You resolved the pull request wrong: You removed the PlushieRainbowLizard again, which was added as an arcade prize in #32564

whoops, its my first time having a merge conflict so I just went with the section it highlighted, didnt notice the rainbow lizard. If I add it back in is everything good to go?

Yeah, if you add it back I can merge this PR. The addition of the rainbow lizard was what caused the merge conflict: Two Pull requests modifying the same part of the code in different ways.

whoops, its my first time having a merge conflict so I just went with the section it highlighted, didnt notice the rainbow lizard. If I add it back in is everything good to go?

Yeah, if you add it back I can merge this PR. The addition of the rainbow lizard was what caused the merge conflict: Two Pull requests modifying the same part of the code in different ways.

Oh I see now, ok cool.
Do I just add it back in with a new commit? or is there a way to roll it back then resolve the conflict properly?

Just adding it back in a new commit is fine. When we merge a PR we squash it first, which means all your commits will be turned into a single one for better overview.

Looks good to me. Thank you for your contribution!


AddComment used too many memory ( more than 10GB for only 100 comments)

we added about 100 comments in 5 batch,  but it use too many memory, like more than 10GB.
code and files are in the attachment
test1.zip

Thanks for your issue. I have test with your attachment and get runtime memory usage by printBenchmarkInfo function, and it just cost about 11MB, please make sure your code and real memory usage.

@xuri 你这个函数可能不太准，开个 top  看一下，处理100条就很慢，而且内存明显飙升
我的操作系统是 centos 7.9 ， Go 版本 1.6 / 1.8/ 1.9 都试过，都有问题

The Getrusage uses the low-level operating system primitives, which get the same resident memory usage as the top command. Version 2.6.0 of this library requires the Go language to be 1.15 later. If you're using 1.6, 1.8, or 1.9, you couldn't build successfully. Please check your code or service, and you can also diagnose logic and performance problems by profiling tools.

I closed this issue. If you have any questions, please let me know to reopen this anytime.

@xuri  是   1.16 / 1.18/ 1.19 , 上面打错了

@xuri I think i've found what's wrong.
https://github.com/qax-os/excelize/blob/master/comment.go#L236
It's ok when you create a new file, add some comments, and save it. But when you reopen that file, and append some comments, it loads the existing shapes from xl/drawings/vmlDrawing1.vml  everytime, and append it to the current vml.
so, the ploblem is : The existing shapes should only load once !

I've made a fix for this, tested locally and works fine

Thanks for your feedback, I've accepted pull request #1311, and this patch will be released in the next version.


[Autocomplete] onHighlightChange does not trigger when options changes but not the length

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Changing options asynchronously will with autoHighlight have weird behaviour with onHighlightChange when options are changed but coincidentally the length doesn't change. The onHighlightChange don't trigger, or the first item does not get highlighted properly.
The behaviour is normal if options length changes.
Expected behavior 🤔
With autoHighlight it should properly highlight first item on any options change including options.length , as well as triggering onHighlightChange.
Steps to reproduce 🕹
Steps:

Asynchronously change options but does not change the length of the array with a slight delay
autoHighlight enabled
onHighlightChange set up to see what's being highlighted
Change input to see the first item does not get highlighted after a change
Change input again to see the previous first item became the highlighted item
https://codesandbox.io/s/loving-cerf-d212pw

Context 🔦
I'm trying to use this to fetch options async on every input change, like google search. Using onHighlightedChange to track currently highlighted item and direct to it onKeyDown on the input component. I couldn't use onChange because I wanted to track other keys pressed with enter to open in a new tab or a new window.
Looking at the source code, this line and comment is suspicious:
https://github.com/mui/material-ui/blob/master/packages/mui-base/src/AutocompleteUnstyled/useAutocomplete.js#L503
Your environment 🌎

npx @mui/envinfo
  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Seeing the same thing in my project too. would love to see this fixed soon!


new --project error

Installed correctly, but had this error in red, in the middle. Windows/Ubuntu install:
ERROR: Command errored out with exit status 1:
command: /home/maru/MyProject/venv/bin/python3 -u -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-7lt72ssz/fprime-fpp/setup.py'"'"'; file='"'"'/tmp/pip-install-7lt72ssz/fprime-fpp/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' bdist_wheel -d /tmp/pip-wheel-_cy5aoqc
cwd: /tmp/pip-install-7lt72ssz/fprime-fpp/
Complete output (33 lines):
WARNING: The wheel package is not available.
WARNING: The wheel package is not available.
WARNING: The wheel package is not available.
WARNING: The wheel package is not available.
/tmp/pip-install-7lt72ssz/fprime-fpp/.eggs/setuptools_scm-7.1.0-py3.8.egg/setuptools_scm/integration.py:28: RuntimeWarning:
ERROR: setuptools==44.0.0 is used in combination with setuptools_scm>=6.x
Your build configuration is incomplete and previously worked by accident!
setuptools_scm requires setuptools>=45
This happens as setuptools is unable to replace itself when a activated build dependency
requires a more recent setuptools version
(it does not respect "setuptools>X" in setup_requires).
setuptools>=31 is required for setup.cfg metadata support
setuptools>=42 is required for pyproject.toml configuration support
Suggested workarounds if applicable:

preinstalling build dependencies like setuptools_scm before running setup.py
installing setuptools_scm using the system package manager to ensure consistency
migrating from the deprecated setup_requires mechanism to pep517/518
and using a pyproject.toml to declare build dependencies
which are reliably pre-installed before running the build tools

warnings.warn(

usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
or: setup.py --help [cmd1 cmd2 ...]
or: setup.py --help-commands
or: setup.py cmd --help
error: invalid command 'bdist_wheel'
ERROR: Failed building wheel for fprime-fpp

Tracking in F´as it is more than just this tutorial.


feat: support (at least) dual themes

Hello 👋,
This PR is a follow-up to https://github.com/spatie/commonmark-shiki-highlighter/pull/13 in order to add support for dual themes.
I add think that having a key named themes makes sens when we want to support multiple themes.

Hey @riasvdv 👋, this is the latest PR to support dual themes in Laravel!

🥳 thanks for all the work on this!

🥳 thanks for all the work on this!

Thank you so much for your speed! It's truly a pleasure to contribute to well-built and tested projects and this will allow me to publish dark theme for my website https://soubiran.dev 🙌


Getting Started with DBSCAN Clustering in Python

This closes #4677

The plagiarism score is quite high. Please correct the highlighted areas. Thanks
New Scan 12_30 PM Copyleaks Report.pdf

Hello @WanjaMIKE  would you please do a plagiarism check on this PR?

Plagiarism check done. Please correct the highlighted areas
New Scan 6_47 AM Copyleaks Report.pdf

@Daniel695
cc @jerimkaura

@WanjaMIKE
Plagiarism is now corrected

I can jump on this one @lalith1403

If you have not started 👍
For final review

Sure @hectorkambow. Thanks 😊

👍
Overall nice work gents


Use Java 8 stream API

@martint thank you for the comments, get it updated

@electrum thank you, get comments addressed

@electrum @martint added a few more java8 stream api update

rebased

Merged, thanks!


Update Information Protection SDK Clients

SDK clients of Information Protection were updated according to changes introduced in corresponding swagger. No new clients were added.
This PR is based on https://github.com/Azure/azure-rest-api-specs/pull/8677 swagger spec review PR.

@isra-fel Thanks for the review! You comments were addressed.

@isra-fel Can we please merge, as you comments were addressed?

Hi @bashahee sorry for late response. We were just back from holiday.
The metadata txt should be under /eng/mgmt/mgmtmetadata . Perhaps you need to update your build tools. Or you can simply move it there.

@isra-fel Done, thanks!

/azp run

@isra-fel Can we please merge?


Remove loop argument

... and some other fixes.
Closes #38.

@decaz good work. could you please split this into two different PR's (or at least rebase to create two separate commits)? One with style changes (removing object, changing super calls to syntax without parameters) and another with loop-related changes.

@cybergrind I removed all style changing commits. Also I've turned back loop argument to asyncio.ensure_future and asyncio.open_connection.

@decaz uploaded as 0.18.0


Add swift-filename-matcher

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (swift-filename-matcher) have been updated in this PR. Please review the changes.

CI failure

i think you need https://github.com/bazelbuild/bazel-central-registry/pull/2661#issuecomment-2308323732

Enabled auto merge


Invalid exlude - file not found warning in Xcode 13.0

Description:
I am getting warnings after building the app which contains EasyTipView 2.1.0 via SPM with Xcode 13:
Invalid Exclude '/Users/.../Library/Developer/Xcode/DerivedData/.../SourcePackages/checkouts/EasyTipView/Sources/EasyTipView/Example': File not found.
Invalid Exclude '/Users/.../Library/Developer/Xcode/DerivedData/.../SourcePackages/checkouts/EasyTipView/Sources/EasyTipView/assets': File not found.
Invalid Exclude '/Users/.../Library/Developer/Xcode/DerivedData/.../SourcePackages/checkouts/EasyTipView/Sources/EasyTipView/Source': File not found.

Similar issue in different repo:
https://github.com/tid-kijyun/Kanna/pull/249

Same here!

same here!


FIX #9 MlflowMetricsDataSet implemented

Hello @akruszewski and @kaemo, thank you very much for the detailed implementation (with tests and documentation, you rock :-) ). Apart from the minor comments from above review, I have more general concerns on some design decisions and I don't feel cobfortable to merge this PR this until we agree on them. I will put some detailed explanations in #9 to keep track this more general discussion.

We have PR with better implementation. Closing.


does arcface  used in this repo support multi- batching?

i build a model with
#define BATCH_SIZE 5 // currently, only support BATCH=1

since they have mention 'currently, only support BATCH=1', i am not sure how it works internally inside deepstream. One thing I noticed is that   if i set the batch size property greater than 2 sgie.set_property("batch-size", 3) it doesn't save the cropped and aligned images

@Athuliva batch-size of secondary gened infer engine is not a factor as I test so far. Unlike pgie, whose batch-size is strictly restricted to be the same of input-source number. Input to pgie will be batched in previous Nvstreammux, so they must be the same.
If you check the source code, the sgie's input-batch is pushed once reaches sgie's max batchsize.
If you want to dive into, you can debug the codes I mentioned above and see how it goes.

@Athuliva I add some new features including multi-batch support to gst-nvinfer-custom[https://github.com/zhouyuchong/gst-nvinfer-custom], have a try and inform me if there is any problem. :smile_cat:


fixed commit loss msg

Codecov Report

Merging #67 (d1d1e0d) into main (c2bf6cc) will decrease coverage by 0.16%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   60.66%   60.49%   -0.17%
==========================================
  Files          12       12
  Lines        1177     1177
==========================================
- Hits          714      712       -2
- Misses        372      373       +1
- Partials       91       92       +1

Impacted Files
Coverage Δ

pkg/kafsar/kafka_impl.go
48.46% <75.00%> (-0.48%)
:arrow_down:

Continue to review full report at Codecov.

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


Add Frame

Please review our Contribution Guidelines to ensure your changes abides by it.
Review your changes and complete the checklist below.

[x] Company name
[x] Company description
[x] Company location
[x] GitHub link
[x] How Elixir is used
[x] Sorted alphabetically

Frame's director of engineering (Chris Bell) announced the move on Twitter: https://twitter.com/cjbell_/status/931904584666308610

Thanks @stevedomin!  You're so dang helpful and on top it, would you like be a contributor?

@doomspork what would it entails? (just want to make sure I don't overcommit, I've plenty of stuff on my plate already)

@stevedomin no worries — I can appreciate being overextended myself 😀
I was strictly speaking to giving you merge rights so you could add businesses as you find them and not have to worry about a PR but it's no trouble

@doomspork that sounds good, if that can save you some time I'm all for it. Maybe I'll find the time to merge a PR or two as well.

You've been added!  Thanks for all your help @stevedomin.  I'm open to any suggestions you have as well to make the resource more useful.  😁

Will think about it! I always thought it would be nice to have an auto-updating count somewhere.


Fix yt-dlp invocation for unix systems.

I have no clue how the original worked on windows. It looks like quoting is handled differently there. (Edited version not tested on windows)
Also replaced legacy Runtime#exec calls with ProcessBuilder, which are (to my knowledge) not deprecated. Additionally, there should be less attack surface for injections now (although I might have spotted some elsewhere, not sure though)

The general call to yt-dlp seems to work. However, when trying to get a song from youtube it is unable to retrieve any information about it, other then the song itself. For SoundCloud it works fine. Is this the same behaviour as on linux?

After updating yt-dlp it works fine for me as well. There are still a lot of warning for youtube in the console but as far as functionality goes there are no problems anymore. From my side this PR is approved


Refactored &  fixed NoteRead feature

What type of PR is this? (check all applicable)

[x] Refactor
[x] Feature
[x] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
refactored the code made by the prev contributor and fixed minor bugs
Features Covered in this PR

[x] made a different NoteReadIconButton widget to seperate TTS feat
[x] fixed bug, where the icon changes to indicate play when TTS completion happens

Related Tickets & Documents

Related Issue #60
Closes #60

Screenshots, Recordings
https://github.com/SankethBK/diaryvault/assets/113239561/a4d5aad1-2961-4a49-bff9-ce07c3e5c348
Tested Feature??

[x] In Real Device.
[x] In Emulator

Also @SankethBK, I was planning to implement the bloc arch for state management for this feat. But given the feat's current scope , I went with the simple setState .  But I am ready to implement the bloc arch if you say so.

@vaibhavppandey setState is sufficient for this, no need for bloc as the state can be contained within the widget

Minor comments, rest all looks good! Nice job!!

@vaibhavppandey setState is sufficient for this, no need for bloc as the state can be contained within the widget

@SankethBK I just saw this... please take it back!!😂😂 SetState is never sufficient...

@vaibhavppandey setState is sufficient for this, no need for bloc as the state can be contained within the widget

@SankethBK I just saw this... please take it back!!😂😂 SetState is never sufficient... If you don't agree with me this might help change your mind...

I agree, it is definitely not a good practice to call setState on a widget that has large number of children widgets. But since this widget doesn't has any child widget, i think it should be fine

@vaibhavppandey setState is sufficient for this, no need for bloc as the state can be contained within the widget

@SankethBK I just saw this... please take it back!!😂😂
SetState is never sufficient...
If you don't agree with me this might help change your mind...

Ya I second this but as @SankethBK mentioned, the NoteReadIconButton isn't complex yet so I used setState. However,  based on the feedback , I've transitioned toValueNotifier. Thank you for the review! Got to learn something new

@micedreams I don't see any new linter errors, once we fix the ones in master we'll be able to track it easily.


Add special message for procuration bc of residency

Closed by https://github.com/EnMarche/en-marche.fr/pull/1020 (https://github.com/EnMarche/en-marche.fr/pull/1020/commits/e71d397d453fa88b400dc19c5e74e004f3ab3a0b)


feat(icons): icons for heic and dng filetypes

[x] I have read the CONTRIBUTING.md file.
[x] I have read the CODE_OF_CONDUCT.md file.

Usually, iPhones have dng and heic filetypes for images. This PR adds icon support for those as well.

Thanks for the PR!
I am afraid both 'heic' and 'dng' extensions don't really fit the criteria for making them built-in. In particular, they don't really seem to be popular (at least, this is the first time I am hearing about them) and they seem mostly tied to language/software (this is not present in criteria for extensions, but it is for "file" and "directory"; I'll fix that).
Closing as not planned.


Demo walkthrough doesn't work because Adaptive Card Action no longer exists

The demo instructs the user in instruction item 11 to create a new step with the action labeled Post an Adaptive Card to a Teams user and wait for a response, however, that card does not currently exist.
There is a similar card titled Post adaptive card and wait for a response, but that card has different required parameters than the ones described in instruction item 12. Instead of sending an Adaptive Card directly to a user, you must select who the message will be posted as and where it's posted to, even when setting the recipient to be a user by email.
How would we achieve the same effect as described in the demo?

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

ID: 5d5d64ce-91a7-5caa-89f4-d16acac667bd
Version Independent ID: 07baec70-00d6-e47b-f73a-f52bb10fd92e
Content: Office Scripts sample scenario: Automated task reminders - Office Scripts
Content Source: docs/resources/scenarios/task-reminders.md
Service: excel
Sub-service: scripts
GitHub Login: @o365devx
Microsoft Alias: o365devx

Thanks for reaching out, @jessiewestlake.
@alison-mk, could you please investigate this when you get a chance?

Hi @jessiewestlake, thanks for bringing this to our attention. I'm researching this issue and will report back when I have more information.
Cheers,
Alison

Looks like there's a new option in Power Automate. I'll start reworking the tutorial to use it.

Hi @jessiewestlake,
You're correct that the new "Post adaptive card and wait for a response" action is what you should be using. I've drafted a PR to clarify the changes.
Here's what the action would look like:


Remove the PR template.

I'm currently working on the project primarily from VSCode, which automatically populates PRs from commit descriptions.

Hmm, yes, I'm using that one, and I see I'm also using v0.21.0, which should already include this functionality. But in any case, I've already made a habit of including detailed change descriptions in the commit messages, so I think it should be fine to remove this template for the time being. We can add it back later of course if you or others will need it.


Jobs working-directory is ignored

It seems like pdm-setup is ignoring the working-directory set for a job e.g. like this
jobs:
  lint:
    defaults:
      run:
        working-directory: packages/api

and instead tries to install from the root directory. I managed to at least install the dependencies by adding the cache-dependency-path but it still installs the dependencies in the root directory.
I think it'd be nice if pdm-setup would use the default working-directory and/or provide a parameter to change the directory the pdm.lock is located and install the venv there.

I don't see why it is related to this project, the installation is done by the subsequent steps written by yourself, isn't it? Nothing stops you from changing working directory in the run steps.

Sorry you're right. The run: pdm install works as expected with in the set working-directory. I guess I thought the cache-dependency path would be relative to that as well but its not a big deal. Closing

So this might actually be a bug. We are utilizing a mono repo, and i get the error:
No file in /home/runner/work/MY_REPO/MY_REPO matched to [pdm.lock], make sure you have checked out the target repository

Where MY_REPO is naturally the name of my repo.
I've set:
defaults:
  run:
    working-directory: api

(the api is the subfolder that has the python project)
Which works in the other actions in the pipeline, but not this one.
Can we re-open it?

Still an issue, please reopen.
Run pdm-project/setup-pdm@v4
  with:
    cache: true
    allow-python-prereleases: false
    token: ***
    prerelease: false
    enable-pep58[2](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:2): false
    cache-dependency-path: pdm.lock
    cache-restore-exact-match: false
    update-python: true
  env:
    pythonLocation: /opt/hostedtoolcache/Python/[3](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:3).10.15/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.10.15/x6[4](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:4)/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.1[5](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:5)/x64
    Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.15/x[6](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:6)4
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.10.15/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.10.15/x64/lib
Warning: Neither 'python-version' nor 'python-version-file' inputs were supplied. Attempting to find 'pyproject.toml' file.
Warning: pyproject.toml doesn't exist.
Successfully set up CPython (3.12.[8](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:8))
/opt/hostedtoolcache/Python/3.12.8/x64/bin/python3 - -o install-output.json
Usage: pdm [-h] [-V] [-c CONFIG] [-v | -q] [--no-cache] [-I]
 [--pep582 [SHELL]] [-n]
 ...

    ____  ____  __  ___
   / __ \/ __ \/  |/  /
  / /_/ / / / / /|_/ /
 / ____/ /_/ / /  / /
/_/   /_____/_/  /_/

Options:
  -h, --help            Show this help message and exit.
  -V, --version         Show the version and exit
  -c CONFIG, --config CONFIG
                        Specify another config file path [env var:
                        PDM_CONFIG_FILE]
  -v, --verbose         Use `-v` for detailed output and `-vv` for more
                        detailed
  -q, --quiet           Suppress output
  --no-cache            Disable the cache for the current command. [env var:
                        PDM_NO_CACHE]
  -I, --ignore-python   Ignore the Python path saved in .pdm-python. [env var:
                        PDM_IGNORE_SAVED_PYTHON]
  --pep582 [SHELL]      Print the command line to be eval'd by the shell for
                        PEP 582
  -n, --non-interactive
                        Don't show interactive prompts but use defaults. [env
                        var: PDM_NON_INTERACTIVE]

Commands:
  add                   Add package(s) to pyproject.toml and install them
  build                 Build artifacts for distribution
  cache                 Control the caches of PDM
  completion            Generate completion scripts for the given shell
  config                Display the current configuration
  export                Export the locked packages set to other formats
  fix                   Fix the project problems according to the latest
                        version of PDM
  import                Import project metadata from other formats
  info                  Show the project information
  init                  Initialize a pyproject.toml for PDM. Built-in
                        templates: - default: `pdm init`, A simple template
                        with a basic structure. - minimal: `pdm init minimal`,
                        A minimal template with only `pyproject.toml`.
  install               Install dependencies from lock file
  list                  List packages installed in the current working set
  lock                  Resolve and lock dependencies
  outdated              Check for outdated packages and list the latest
                        versions on indexes.
  publish               Build and publish the project to PyPI
  python (py)           Manage installed Python interpreters
  remove                Remove packages from pyproject.toml
  run                   Run commands or scripts with local packages loaded
  search                Search for PyPI packages
  self (plugin)         Manage the PDM program itself (previously known as
                        plugin)
  show                  Show the package information
  sync                  Synchronize the current working set with lock file
  update                Update package(s) in pyproject.toml
  use                   Use the given python version or path as base
                        interpreter. If not found, PDM will try to install
                        one.
  venv                  Virtualenv management
Installing PDM (latest): Creating virtual environment
Installing PDM (latest): Installing PDM and dependencies
Installing PDM (latest): Making binary at /home/runner/.local/bin

Successfully installed: PDM (2.22.1) at /home/runner/.local/bin/pdm
Writing output to install-output.json
Successfully setup 2.22.1 with Python 3.[12](https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3#step:4:12).8
Error: No file in /home/runner/work/imec-workshop/imec-workshop matched to [pdm.lock], make sure you have checked out the target repository

workrun here: https://github.com/ChipFlow/imec-workshop/actions/runs/12412602592/job/34652737498?pr=3

In case this never gets fixed - work around is to set cache-dependency-path: to your target lock file

I understood @frostming like thats the intended solution/behaviour


Constraints patch

@StuartLittlefair , @bmorris3 . This is in relation to the results I got from profiling the code. I have made some changes regarding that.
Adjusted the code slightly so that memory usage is minimized, the properties _all_constraints and constraints were being used repetitively .
Also separately added the part which can append AltitudeConstraints in b.constraints if initially the constraints was not empty and there was no instance of AltitudeConstraint (default 0 deg) in it.

This was supposed to be the patch I planned to create for GSoC-17 . However, due to some unavoidable reasons, I could not submit it before deadline.


Add Tox and flake8 support

Ha! Didn't know Travis would pick up branches. That's nice! 👍

Hahaha OMG so many flake8 things to fix :+1:
Actually there are a bunch of them all reporting the same line, so they'll drop like flies

And I should say well done on having this set up nicely already.  A quick:
pyenv install 3.6.0     # install a 3.6.0 interpreter on my machine
pyenv virtualenv 3.6.0 OpenGHX    # create a virtual env inside my new 3.6.0 interpreter called OpenGHX
pyenv activate OpenGHX   # activate this new virtual env so I can install packages into it
pip install -r requirements.txt    # install your dependencies
python setup.py test    # run your tests

worked just fine :+1:

Well, thanks for the help. We'll have to discuss the pyenv a little more next time. It would be good practice to learn more about it and use it.

I turned off flake8 support just by commenting one line.  That way this could get merged at your leisure and then you can turn that line back on and fix those issues whenever you want later.

I ran autopep8 on everything. I wonder what it would look like now... Might turn it on and see.

Sounds like a tomorrow task :sleepy:

Fo sho. Thanks!


merge from template-sync/template-arlac77-github

.github/pr_labeler.yml

chore(labels): merge from template .github/pr_labeler.yml

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 22.976%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
102

Relevant Lines:
453

💛  - Coveralls


fix assignment timestamp in block

Fixes a bug where the assignment timestamp ended up in the block. Related to #198

Current coverage is 93.30% (diff: 100%)

Merging #627 into master will increase coverage by <.01%

@@             master       #627   diff @@
==========================================
  Files            23         23
  Lines          1254       1255     +1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           1170       1171     +1
  Misses           84         84
  Partials          0          0

Powered by Codecov. Last update 92981e0...9a19add

:+1:


Add code lint for Go plugin code

Why do you need it?
With an increasing number of developers working on Go WASM plugins, it is advisable to introduce Go code checking tools to ensure the code quality of these plugins.
How could it be?
Add golangci-lint check for the Go wasm plugin code

I want to solve it.Could you assign to me？


[Bug] CUDA out of memory on A10 24G with device_map auto

Describe the bug
I can load the model into VRAM. However, when I call the chat methods, an exception occurs:
CUDA out of memory. Tried to allocate 1.57 GiB (GPU 0; 22.05 GiB total capacity; 19.76 GiB already allocated; 1.32 GiB free; 19.77 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

Is it an expected behavior? How much VRAM do I need to run the model?
Environment
Python 3.8
Cuda 12.1
CPU: 8 cores, RAM: 32GB
GPU: A10, VRAM: 24GB

Other information
I set the device_map to auto since I need to load the model into GPU. Otherwise, it says
RuntimeError: [enforce fail at alloc_cpu.cpp:75] err == 0. DefaultCPUAllocator: can't allocate memory: you tried to allocate 845152256 bytes. Error code 12 (Cannot allocate memory)

Hi,
You can try using float16.
https://github.com/InternLM/InternLM/pull/80


Weird behaviour with Source Han Sans SC

Description
My template (slightly modified from web app):
// The project function defines how your document looks.
// It takes your content and some metadata and formats it.
// Go ahead and customize it to your liking!
#let project(title: "", authors: (), date: none, body) = {
  // Set the document's basic properties.
  set document(author: authors, title: title)
  set page(numbering: "1", number-align: center,
  paper: "a5",
  margin: (x: 1.5cm, y: 0.5cm)
  )

  // Save heading and body font families in variables.
  let body-font = "Source Han Serif SC"
  let sans-font = "Source Han Sans SC"

  // Set body font family.
  show heading: set text(font: sans-font)
  set text(font: body-font, lang: "zh")
  show raw: set text(font: "Sarasa Mono SC")
  set heading(numbering: "I – a - 1")

  // Title row.
  align(center)[
    #block(text(font: sans-font, weight: "bold", 1.75em ,lang: "zh")[#title])
    #v(1em, weak: true)
    #date
  ]

  // Author information.
  pad(
    top: 0.5em,
    bottom: 0.5em,
    x: 2em,
    grid(
      columns: (1fr,) * calc.min(3, authors.len()),
      gutter: 1em,
      ..authors.map(author => align(center, strong(author))),
    ),
  )

  // Main body.
  set par(justify: true)

  body
}

my main file:
#import "Template.typ": *

// // Take a look at the file `template.typ` in the file panel
// // to customize this template and discover how it works.
#show: project.with(
  title: "研究报告",
  authors: (
    "KyNSyZsgV0o8A-fSirq",
  ),
  date: "July 21, 2023",
)

Main Text 主文本

What i Get:

The title is clearly using a serif font
Both fonts are installed on my system (win10, typst 0.6.0).
Any idea what is causing the problem?
(These fonts can be download from Adobe's SourceHan repo)
Reproduction URL
No response
Operating system
Windows
Typst version

[X] I am using the latest version of Typst

This happened because the compiler didn't found the specified sans font. In this case, the compiler uses a default font as a fallback, which is a serif font.
My guess is that the name from the specified font is slightly different than the installed font. To check this, run .\typst.exe fonts. Search for your font and make sure the spelling is exactly the same as in the template file. For example, the "Source Han Sans" font from this repo is named "Source Han Sans SC VF". You need to add " VF" and it should work.

I see, I will check it out ASAPThx!Sent from my iPhoneOn Jul 22, 2023, at 03:29, Julian Waßmann @.***> wrote:
This happened because the compiler didn't found the specified sans font. In this case, the compiler uses a default font as a fallback, which is a serif font.
My guess is that the name from the specified font is slightly different than the installed font. To check this, run .\typst.exe fonts. Search for your font and make sure the spelling is exactly the same as in the template file. For example, the "Source Han Sans" font from this repo is named "Source Han Sans SC VF". You need to add " VF" and it should work.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

This happened because the compiler didn't found the specified sans font. In this case, the compiler uses a default font as a fallback, which is a serif font. My guess is that the name from the specified font is slightly different than the installed font. To check this, run .\typst.exe fonts. Search for your font and make sure the spelling is exactly the same as in the template file. For example, the "Source Han Sans" font from this repo is named "Source Han Sans SC VF". You need to add " VF" and it should work.

Did a fresh font installation with https://github.com/adobe-fonts/source-han-sans/releases/tag/2.004R
Checked that terminal output with typst fonts contains:
Sitka Text
Snap ITC
Source Han Sans
Source Han Sans SC
Source Han Serif
Source Han Serif SC
STCaiyun

Tried other fonts, and all work fine.
The above serif font is not the fallback font, the fallback font looks like this:

(Created by typing a random string in sans-font)
It seems that the problem only occurs with Source Han Sans SC

After assigning both fonts to Source Han Sans SC like:
  // Save heading and body font families in variables.
  let body-font = "Source Han Sans SC"
  // let body-font = "No Font Test"
  // let body-font = "Adobe Caslon Pro"
  // let body-font = "LXGW Wenkai"
  // let body-font = "FZNewShusong-Z10s"
  // let sans-font = "Sarasa Gothic CL"
  let sans-font = "Source Han Sans SC"

I get the proper output:

It seems like there is something wrong with the config file (or how the compiler deal with it), the compiler obviously can find the font.

Sorry, I was a bit confused which exact font you downloaded. Did you used the release page and downloaded "Simplified Chinese" under "Language-specific OTFs", then installed the regular and bold OTFs?
I installed the Simplified Chinese font myself (regular and bold, both Sans and Serif). In Typst, I get the expected output, wich looks like this:

Using:
let body-font = "Source Han Serif SC"
let sans-font = "Source Han Sans SC"

Title and headings are using the Sans font (bold). Date, author and the main text are using the Serif font. The output is equivalent to what you have specified in the template.
Does your output look the same with the same settings? If this is the case, can you please explain, what you consider as a bug?

Thx for the testing.
I am currently unable to conduct further examination on this problem. I will come back to this thread and check it out ASAP

Sorry, I was a bit confused which exact font you downloaded. Did you used the release page and downloaded "Simplified Chinese" under "Language-specific OTFs", then installed the regular and bold OTFs?
I installed the Simplified Chinese font myself (regular and bold, both Sans and Serif). In Typst, I get the expected output, wich looks like this:  Using:
let body-font = "Source Han Serif SC"
let sans-font = "Source Han Sans SC"

Title and headings are using the Sans font (bold). Date, author and the main text are using the Serif font. The output is equivalent to what you have specified in the template.
Does your output look the same with the same settings? If this is the case, can you please explain, what you consider as a bug?
Weird.
I am using the exact config and fonts, but I get this instead:

let body-font = "Source Han Serif SC"
let sans-font = "Source Han Sans SC"
  // Title row.
  align(center)[
    #block(text(font: sans-font, weight: "bold", 1.75em ,lang: "zh")[#title])
    #v(1em, weak: true)
    #date
  ]

I just confirmed that this is not a typst problem, this is a VS CODE problem.
With the same file, the screenshot from VSCode shows the weird serif result, but any other reader (firefox, edge) displays the right result.


Audio playback issue at receiver end during audio live streaming after upgrading from 0.11.2 to latest version 0.13.6

Describe the bug
Here is the link for the audio live streaming issue which is already closed. After your comment in that issue, I have made changes accordingly but audio playback stopped working at receiver end.
https://github.com/shogo4405/HaishinKit.kt/issues/266
We have implemented the changes based on your comments. However, following this update, we are encountering an audio playback issue on the receiver's end during live audio streaming. I do not see any obvious errors on the streaming side.Attached the logs for the streaming side as well for your review here, the receiver is experiencing a SocketTimeoutException while listening.
Is there any other change made in the latest version apart from the encoder? Is there any change related to the streaming side?
Any suggestions to fix this issue? Please let me know.
We have checked again with the older version v0.11.2, in which audio streaming and playback was working fine.
Scenario: I am starting live audio streaming from android device and playback it from another device, but the playback stalls every time, resulting in a SocketTimeoutException. We have also tried playback with iPhone/iPad, and it was not working on those devices either.
Here I am attaching log for streaming end and playback end.
live audio playback log.txt
live streaming log.txt
To Reproduce
Scenario: I am starting live audio streaming from android device and playback it from another device, but the playback stalls every time, resulting in a SocketTimeoutException. We have also tried playback with iPhone/iPad, and it was not working on those devices either.
Expected behavior
Audio playback should be work fine at the receiver end.
Version
0.13.6
Smartphone info.
No response
Additional context
No response
Screenshots
No response
Relevant log output
No response

Posting the same content in Issues or Discussions won't solve the problem and is a nuisance. Please stop doing it.


Move BlockChainName to configuration file to support dev on regtest / testnet

BlockChainName is currently a const. Can it be in config or --flag?
// global/global.go
14 | const BlockChainName = "lbrycrd_main"

Yes, I will add it as a config.

solved with 84f875b3f68266c7a1d6ad95cdf0d3dfd68b5ce5


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99571

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99750

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99752

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99794

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/99866

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/101308


Destination not empty after 80% of data migration

I was trying to migrate Parse data to mlabs  and after around 80% of data transfer, it failed with error that

The destination database was not empty

Also the transfer rate keeps on decreasing, (even to 60 rows per second).
Please suggest what should be done to avoid this.

For issues with the migration process, please contact our support through parse.com/help


Raw_input in pointop_key raises exception

I'm using gazebo to simulate a TurtleBot3. So far I've been using the Burger. When trying to issue navigation instructions to the robot following this guide the node kept quitting with the message "shutdown program.". I snooped around in the source code and found that it was probably raising an exception when waiting for user input.
The issue was solved when I changed the referenced line to use input() instead of raw_input(). Function raw_input() is deprecated in python3 and renamed to input().
https://github.com/ROBOTIS-GIT/turtlebot3/blob/5744eab1c3962c64eb754203180c95ae6858c043/turtlebot3_example/nodes/turtlebot3_pointop_key#L127
If this project does not target python3, I guess this issue can be closed.

Side discussion
The following is not inherently a part of the issue.
This also raises (pun intended) a good point about how try-catch blocks can hide away problems. While the program fails silently (with a single uninformative error message) with a try:...except: block, removing that block lets the program gracefully blow on your face with a pretty informative error:
Traceback (most recent call last):
  File "/home/user/catkin_wc/devel/lib/turtlebot3_example/turtlebot3_pointop_key", line 15, in <module>
    exec(compile(fh.read(), python_script, 'exec'), context)
  File "/home/user/catkin_wc/src/turtlebot3/turtlebot3_example/nodes/turtlebot3_pointop_key", line 165, in <module>
    GotoPoint()
  File "/home/user/catkin_wc/src/turtlebot3/turtlebot3_example/nodes/turtlebot3_pointop_key", line 70, in __init__
    (goal_x, goal_y, goal_z) = self.getkey() # This excepts
  File "/home/user/catkin_wc/src/turtlebot3/turtlebot3_example/nodes/turtlebot3_pointop_key", line 135, in getkey
    x, y, z = raw_input("| x | y | z |\n").split()
NameError: name 'raw_input' is not defined

@KomaGR
Thank you for your input. Since current TurtleBot3 pointop_key instruction is written based on ROS 1 Kinetic and to avoid possible confusion for someone who followed ROS 1 instruction, we'll handle this deprecated function in ROS 2 section.
Have a great day!

Apologies for the delayed response. We have assembled a TF team to address TurtleBot3 issues, but as this issue has been open for a long time, we will close it for now. Currently, our team is supporting Noetic and Humble versions. Please try again with the latest version, and if you encounter the issue again, feel free to reopen it.


added --verbose option to show piece info bars

closes #296

Awesome - thanks!

Released as 0.49.2.


feat: create a backoff sender level in case no allocation was

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

26 of 26 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 72.433%

Files with Coverage Reduction
New Missed Lines
%

tap-agent/src/agent/sender_account.rs
1
93.07%

Totals

Change from base Build<PHONE_NUMBER>6:
-0.003%

Covered Lines:
4945

Relevant Lines:
6827

💛  - Coveralls


Bot Control from Youtube

Hi @swolekat,
first of All: Thanks again for all the adjustments you have made. I used this in 24/7 Scenrio with OBS on Twitch for showing older Arma MilSim Missions etc. and it was well received by the viewers. (so with including Video)
But on Last Friday I had to stop this Stream.
Instead of Playing the Playlist there is just showing up a message from Youtube to this Browesersource: Text is in german but means: Registrate / Log In that we can see you are no Bot and safe the community. (see attached picture)
I also tried with the direct Playlist Link (not routing by your Website / Webservice) in annother Browsersource and result is the same.
But on this way I can login on my Account what is not possible with the rerouting.
I deeply know that this is no problem on your side and your project probaly just Youtube changed something on their end.
But maybe you have an idea to solve this. Your solution with getting the Videoname and customize this with CSS is so clear and easy.

Kindly thanks.

I fixed it by myself. I logged in at Browsersource which not points to your Website and as I finished this the other "nromal" Playlists worked also. A bit confusing but okay, now.
Thanks.


core: emit auto-auth errors

:warning: Opinions needed on this one.
We have been ignoring the error that google-auth-library returns when it can't find the environment's default credentials.
As explained in the comment from the code:

google-auth-library returns a "Could not load..." error if it can't get an access token. However, it's possible an API request doesn't need to be authenticated, e.g. when downloading a file from a public bucket. We consider this error a warning, and allow the request to go through without authentication, relying on the upstream API to return an error the user would find more helpful, should one occur.

We have a bug currently because of this: https://github.com/GoogleCloudPlatform/nodejs-getting-started/issues/25 - I'll explain this in a comment on the offending line of code.
Considering this instantiation:
var gcloud = require('gcloud');

// No credentials provided:
var dataset = gcloud.datastore().dataset({
  projectId: 'grape-spaceship-123'
});

Here is what will happen if we merge this PR:
dataset.runQuery(/*...*/, function(err) {
  // The error comes from google-auth-library:
  // err.message = 'Could not load the default credentials. Browse to https://developers.google.com/accounts/docs/application-default-credentials for more information.'
});

Pretending we didn't merge this PR, and just fixed the bug mentioned earlier, here's what would happen:
dataset.runQuery(/*...*/, function(err) {
  // The error comes from the API:
  // err.message = 'Unauthorized'
});

To get the better error (the first example), we have to not allow unauthenticated requests. I'm not sure if/what/how much of our API allows unauthenticated requests, or if we even want to support that in the first place?

Let's just go with it.


npm run dev在linux环境下的报错修复

https://github.com/bailicangdu/react-pxq/blob/master/webpack.config.dev.js#L9
linux下文件名区分大小写,建议改为
var APP_FILE = path.resolve(APP_PATH, 'App');
就不会报错

thanks,it is very helpful.


Renaming files change other filenames aswell

Description
The build in renaming functionality doesn't work for me correctly. I got several .pdf-files in a single Folder. Renaming a file will change the name of every other file which was renamed before.
Expected
Renaming a file should just have an effect of the selected files and not on every file that was renamed with a previous action.
Actual
When I rename the first document it works as expected. But renaming another file to a different name will also alter the name of the previously renamed file. Same for the next file wich will also rename all previously renamed files. It's as if the other files are still selected, which they are not. Even when going out of the folder and in again this behaviour stays.
Info:

OS: Unraid 6.8.3
Browser: Firefox
Database: SQLite
Papermerge Version: 1.5.0

The build in renaming functionality

What exactly you mean with "The build  in renaming" ?

Maybe I missed a hyphen there. I mean the build-in renaming functionality (anklicken des Dokuments und Umbenennen über das Aktionsmenü)
I will test it tomorrow on your demo instance if I don't forget.

Can you please confirm that same behaviour is on demo?

Alright. I just tested and confirmed the same behaviour on your test instance. I kept the 6 pdf-files uploaded so you can confirm it yourself.
Steps to reproduce:

Browse to /F_Test/P_Test/K_Test
Select the file 1.pdf (doesn't matter if with grid or list view)
Click on Action -> Rename
Name it test1.pdf
Select the file 2.pdf (the renamed file should be visually deselected)
Click on Action -> Rename
Name it test2.pdf
-> Both file are named test2.pdf now

I noticed that if you close the session and login again the issue is gone. At least for the first file. If you rename a 2nd file it's the same behaviour.

@Greenlander92, thank you for detailed steps!
I was able to reproduce it :partying_face:
I will fix it and make fix available as part of next 1.5.1 version

I was able to reproduce this for folders as well. With the exact steps @Greenlander92 was mentioning:
Steps to reproduce:

Browse to /Folder1/
Select the folder "Folder 1.1" (doesn't matter if with grid or list view)
Click on Action -> Rename
Name it "Folder 1.4"
Select the folder "Folder 1.2" (the renamed file should be visually deselected)
Click on Action -> Rename
Name it "Folder 9.9"
-> Both folder are named "Folder 9.9" now

I experienced the bug in the current version 2.0.1. Any plans on when it gets fixed?

The thing is I split the application in two parts - backend and frontend. This bug described here is on the javascript side. However frontend/javascript side was completely rewritten and buggy behavior is not there any more  - at very least is not there in development version of the frontend.
My plan is to help you to migrate to newer version of the application (featuring new frontend) as soon as it will be out.
The development is very slow, because, as I mentioned, I rewrote the entire frontend and it took me some time to figure out how to do it correctly.

I am closing this issue as it is not relevant for 3.0.3.


set selinux to permissive mode when glusterfs is included

glusterfs will only work when selinux is in permissive mode: http://www.gluster.org/pipermail/gluster-users/2015-November/024476.html
also updates glusterfs repo checksum

Fixes #867

I've merged this into feature/glusterfs-fixes (#880)

this will be included in #880


Feature request: ignore potentially erroneous overly permissive software configurations

Is your feature request related to a problem? Please describe.
As many people may be aware, https://nvd.nist.gov/vuln/detail/CVE-2022-31569 (part of https://github.com/github/securitylab/issues/669) was opened a couple of days ago which opens a CVE related to Flask path traversal (Python lib). The CVE itself has been marked with an overly permissive configuration pattern of cpe:2.3:a:projects_project:projects:*:*:*:*:*:*:*:*. This is clearly not intended, as it is very unlikely that a CVE would be so wide in scope that it should be able to match every version of every library that exists, across all programming languages.
For example, the following dependencies get flagged because of this pattern:
[ERROR] jakarta.annotation-api-1.3.5.jar: CVE-2022-31569(9.3)
[ERROR] spring-cloud-commons-3.1.2.jar: CVE-2022-31569(9.3)
[ERROR] spring-kafka-2.8.6.jar: CVE-2022-31569(9.3)
[ERROR] spring-security-rsa-1.0.10.RELEASE.jar: CVE-2022-31569(9.3)

Describe the solution you'd like
I am wondering if it would be possible to consider implementing a feature that allows ignoring patterns that are match-all without any consideration as to the thing being matched. A starting point would probably be ignoring anything marked as purely wildcards to prevent this.
While this might sound like a poor solution, this would act as a failsafe to prevent countless production-ready systems being unable to deploy due to build failures when a mistake such as this occurs. The benefit of suppressing clearly erroneous mistakes such as this would prevent developers from considering temporarily disabling OSA scanning, which would be putting them at greater risk.
This feature could be toggled using a configuration flag in the Maven plugin <configuration /> block.
Describe alternatives you've considered

Manually suppress these issues - this is okay to do if you have only a few repositories. For
organisations with hundreds or even thousands of repositories, this creates a large amount of
work for having to suppress irrelevant CVEs.
Vulnerability database should probably be preventing full wildcard CVEs - this is out of scope
for this project as it is managed by a different organisation.
Skipping OSA builds where possible until this issue is resolved - there is no time frame on
when the overly permissive configuration will be amended, as this is down to the author or
another entity to consider. Disabling OSA is also highly undesirable because it may hide other
potentially important and relevant vulnerabilities in the process.

As I understand it, this pertains specifically to the Flask vulnerability as used in this repository. The problem here is that this is just some random person's personal projects repository. It's not a proper product, but a CVE was nevertheless requested.
IIUC, the name "project" or "projects" got matched with anything that has "project" in its name. In the case for a project at my day job, it was a dependency being developed by the "Android Open Source Projects" team (which is the "developer name" or "vendor"). i.e.:
<evidence type="vendor" confidence="MEDIUM">
  <source>pom</source>
  <name>developer name</name>
  <value>The Android Open Source Projects</value>
</evidence>

Is this even considered good practice, to issue separate CVEs for every project using a subcomponent, which already had a CVE assigned to it?

Such a pattern seems not very exotic. For example cpe:2.3:a:apache:log4j:*:*:*:*:*:*:*:* in https://nvd.nist.gov/vuln/detail/CVE-2021-44228
The problem is that projects has no significance. The risk that there are valid CVEs are ignored is high.

CVE  has been rejected. All is well!


If I keep hold the up and down array then hours and minutes are not changing it just change by 1 only when I leave click

Hi,
I was using this ngx-mat-timepicker-field, I found one unusual problem. If I keep hold the arrow svg then hours and minutes are not changing fast they are increasing only once when I leave the svg click. Please check below image for reference.

Expected - It should increase/decrease immediately till when I'm holding keys up/down svg. It should behave same as up and down keyboard arrow keys.
Kinda like this (Try to check inline timepicker and when you hover on hour and minute then arrow keys will come and try to hold arrow keys.
Thank you.

@ErVishalM absolutely not mate it's not unusual. It's how click events work in browser.
Cheers.


Redesign of DCH to separate interpolator and hull algorithm

discussed with @victorprincipe a possible redesign of GCH in some future to have additional functionalities and more flexibility. At the moment it is just used scorer. It has hard to make a estimator out of the GCH, because to be consistent with the score function we would need to predict X_HD, but in the current fit function X_HD is part of the X input, so it would be an inconsistent in fit and predict function if we predict X_HD.
So we thought about separating the current DCH directional convex hull and the interpolator part into two separated classes and bringing them then together
class DCH(TransformerMixin, BaseEstimator) # maybe ClusterMixin
  def __init__(self, low_dim_idx)
  def fit(self, X_LD, y)
  def predict(self, X_LD) # -> y (linear interpolation on directional convex hull)
  def transform(self, X_LD) # -> X[self.selected_idx_] (directional convex hull vertices)

class Interpolator(BaseEstimator):
  def __init__(self, interpolator_type)
  def fit(self, X_LD, X_HD)
  def predict(self, X_LD) # -> X_HD

# notebook example
dch = DCH().fit(X_LD, y)
interp = Interpolator().fit(dch.transform(X_LD), X_HD)
interp.score(X_LD, X_HD)

I feel like the last part cannot be abstracted and has to be just a recipe in an example notebook because it is such a specific use case.

I think this is an abstraction that is not important for people who are interested the DCH. If we need such an abstraction we can revive this issues, but I don't see the necessity. This issue was just to keep track of some ideas in the conception phase of the DCH.


No easy way to select a server after hidding the sidebar

If you hide the server switch sidebar, on the settings page, there's no obvious way to get back to seeing your single Zulip server's UI.  It turns out you can do it with ctrl-1, but that's really not obvious.
Some ideas for fixing:

The "Servers" section of the settings page should let you jump to that server, maybe be clicking on the icon or something.
We should list the servers you have setup, with hotkeys in a "servers" menu next to the file menu?
We could list the servers in the taskbar menu.

Doing something here is a priority, since this is basically the first  thing that anyone is going to try to do after toggling the setting to hide the sidebar.

Click server name to show webview. Done in #262.
Will add servers to the view menu in the next step. Like this:

@geeeeeeeeek in window menu :)

@akashnimare The sub menu is done. But seems that window submenu is only displayed on macOS. Do we need to do the same on Linux and Windows?

Yeah, we should add the window menu on Linux and Windows as well.

On windows -

Looks great!


pinctrl-0 issue in device tree building

Describe the bug
During the west build process, it says:
devicetree error: 'pinctrl-0' is marked as required in 'properties:' in /home/zhu/zephyrproject/zephyr/dts/bindings/serial/st,stm32-usart.yaml, but does not appear in <Node /soc/serial@40004400 in '/home/zhu/zephyrproject/zephyr/misc/empty_file.c'>
Please also mention any information which could help others to understand
the problem you're facing:

What target platform are you using?
Arm
What have you tried to diagnose or workaround this issue?
Changed usart1 part in dts file, form:
&usart1 {
current-speed = <115200>;
status = "okay";
};
To:
&usart1 {
current-speed = <115200>;
pinctrl-names = "default";
pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
status = "okay";
};
got same result.

To Reproduce
Steps to reproduce the behavior:

the dts file works in zephyr-2.3.
but not on zephyr-2.7 or zephyr3.0-rc3
west build would bring this error
See error

Expected behavior
A clear and concise description of what you expected to happen.
Pass the building.
Impact
What impact does this issue have on your progress (e.g., annoyance, showstopper)
Don't work.
Logs and console output
If applicable, add console logs or other types of debug information
e.g Wireshark capture or Logic analyzer capture (upload in zip archive).
copy-and-paste text and put a code fence (```) before and after, to help
explain the issue. (if unable to obtain text log, add a screenshot)
devicetree error: 'pinctrl-0' is marked as required in 'properties:' in /home/zhu/zephyrproject/zephyr/dts/bindings/serial/st,stm32-usart.yaml, but does not appear in <Node /soc/serial@40004400 in '/home/zhu/zephyrproject/zephyr/misc/empty_file.c'>
Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows)   Linux
Toolchain (e.g Zephyr SDK, ...)   Zephyr SDK
Commit SHA or Version used   zephyr-2.7, zephyr-3.0-rc3

Additional context
Add any other context that could be relevant to your issue, such as pin setting,
target configuration, ...

Are you including the file where &usart1_tx_pa9 &usart1_rx_pa10 are defined?

@zzzh  Depending on the board and the soc of that board there is a  (ex of nucleo_l476rg)
#include <st/l4/stm32l476r(c-e-g)tx-pinctrl.dtsi>
This dtsi refers to the ../modules/hal/stm32/dts/st/l4/stm32l476r(c-e-g)tx-pinctrl.dtsi
that contains the pin definition usart1_tx_pa9 and usart1_rx_pa10
the pins are not defined.
Second they must be free (not conflicting with another pin function) on the board. For example, in the same board config, pa10 is also the one  for  tim1_ch3_pa10

@zzzh Depending on the board and the soc of that board there is a (ex of nucleo_l476rg) #include <st/l4/stm32l476r(c-e-g)tx-pinctrl.dtsi> This dtsi refers to the ../modules/hal/stm32/dts/st/l4/stm32l476r(c-e-g)tx-pinctrl.dtsi that contains the pin definition usart1_tx_pa9 and usart1_rx_pa10 the pins are not defined. Second they must be free (not conflicting with another pin function) on the board. For example, in the same board config, pa10 is also the one for tim1_ch3_pa10

@FRASTM For me, it's "#include <st/f0/stm32f030c8tx-pinctrl.dtsi>", and I'm sure they are free (not conflicting with another pin function) on the board.

@gmarull @FRASTM :
Can it because they are already defined in pinmux.c?  maybe pinmux.c way had been obsoleted?

The pinctrl had changes since the zephyr 2.3
In the current zephyr version (Zephyr version: 3.0.0-rc3) the nucleo_f030r8.dts already includes the pins for usart1 :
&usart1 {
	pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
	pinctrl-names = "default";
	current-speed = <115200>;
	status = "okay";
};

That's the way to define the node.

The pinctrl had changes since the zephyr 2.3 In the current zephyr version (Zephyr version: 3.0.0-rc3) the nucleo_f030r8.dts already includes the pins for usart1 :
&usart1 {
	pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
	pinctrl-names = "default";
	current-speed = <115200>;
	status = "okay";
};

That's the way to define the node.

@FRASTM  Thank you.
So, is pinmux.c still useful?  what about CONFIG_PINMUX?

The pinctrl had changes since the zephyr 2.3 In the current zephyr version (Zephyr version: 3.0.0-rc3) the nucleo_f030r8.dts already includes the pins for usart1 :
&usart1 {
	pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
	pinctrl-names = "default";
	current-speed = <115200>;
	status = "okay";
};

That's the way to define the node.

Is there any document about the change from using pinmux.c to pinctrl way?

The pinctrl had changes since the zephyr 2.3 In the current zephyr version (Zephyr version: 3.0.0-rc3) the nucleo_f030r8.dts already includes the pins for usart1 :
&usart1 {
	pinctrl-0 = <&usart1_tx_pa9 &usart1_rx_pa10>;
	pinctrl-names = "default";
	current-speed = <115200>;
	status = "okay";
};

That's the way to define the node.

Is there any document about the change from using pinmux.c to pinctrl way?

https://docs.zephyrproject.org/latest/guides/pinctrl/index.html


Appstudio update test-component-pac-ehjf

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-ehjf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
34 seconds
build-container

✅ Succeeded
19 seconds
sast-snyk-check

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
13 seconds
inspect-image

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
37 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-ehjf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
19 seconds
sast-snyk-check

✅ Succeeded
34 seconds
build-container

✅ Succeeded
13 seconds
inspect-image

✅ Succeeded
10 seconds
deprecated-base-image-check

✅ Succeeded
9 seconds
clair-scan

✅ Succeeded
37 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
7 seconds
show-summary

Pipelines as Code CI/test-component-pac-ehjf-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
15 seconds
clone-repository

✅ Succeeded
56 seconds
build-container

✅ Succeeded
20 seconds
sast-snyk-check

✅ Succeeded
20 seconds
deprecated-base-image-check

✅ Succeeded
21 seconds
inspect-image

✅ Succeeded
12 seconds
clair-scan

✅ Succeeded
32 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
6 seconds
show-summary


atom npm package isn't working on windows

Have released 0.7.3 which invokes java with classpath directly instead of the atom shell and bat files.

0.7.5 fixes the problem completely. Thank you.


Package request: LibreTranslate

Project description
Free and Open Source Machine Translation API, entirely self-hosted. Unlike other APIs, it doesn't rely on proprietary providers such as Google or Azure to perform translations. Instead, its translation engine is powered by the open source Argos Translate library.
Metadata

homepage URL:  https://libretranslate.com
source URL: https://github.com/LibreTranslate/LibreTranslate
license: GNU Affero General Public License v3
platforms: Python

Already packaged: https://github.com/NixOS/nixpkgs/pull/228563

Oh, sorry, I didn't check unstable

Or I checked, but in options
And it seems there's no options

@misuzu sorry, but how do I add librentranslate as a service? 🥹

https://github.com/NixOS/nixpkgs/pull/243050
In the meantime something like this should work too:
{
  systemd.services.libretranslate = {
    description = "LibreTranslate service";
    after = [ "network.target" ];
    wantedBy = [ "multi-user.target" ];
    environment = {
      HOME = "/var/lib/libretranslate";
    };
    serviceConfig = {
      ExecStart = "${cfg.package}/bin/libretranslate";
      DynamicUser = true;
      WorkingDirectory = "/var/lib/libretranslate";
      StateDirectory = "libretranslate";
    };
  };
}


Alguns Comandos

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank your for you contribution to the FreeCodeCamp Portuguese Translation, this way we will improve our translation to help millions of users to learn how to program in their native language.
Hope you continue to help us with more of your contributions.


Provide prebuilt library for s390x

See https://github.com/matrix-org/matrix-rust-sdk/issues/1503

@AndrewFerr with the above PR now merged, it would be great if we could have a new release... any plans for that anytime soon?
thanks!

Closing the issue, but I'll let @AndrewFerr do a new release.


Wrong calculation of sample variance

The calculation of the sample variance is currently wrong.
Instead of (correct):

(sum_sq - sum^2 / n) / (n - 1)

or, equivalently (also correct)

(sum_sq - n * mean^2) / (n - 1)

it is now computed as (wrong)

(sum_sq - mean^2) / (n - 1)

This was solved in v0.0.5


Dockerfile update

Updating generated dockerfiles from 2.2 ones to the ones from 2.11.

Summary
(Summarize the problem solved by this PR, and how to verify it manually)
Please check the relevant options

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update

Checklist:

[ ] Example scenarios has been updated / added
[ ] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

Oops, closing as this duplicates #537


Finish the wallet setup and home page flow

This task tracks the release of the first milestone of the wallet, which accomplish;

[x] https://github.com/FuelLabs/fuels-wallet/pull/84

[x] https://github.com/FuelLabs/fuels-wallet/issues/94
[x] https://github.com/FuelLabs/fuels-wallet/issues/93
[x] https://github.com/FuelLabs/fuels-wallet/issues/92

[x] https://github.com/FuelLabs/fuels-wallet/pull/76
[x] #105
[x] #103
[x] https://github.com/FuelLabs/fuels-wallet/issues/95

https://github.com/FuelLabs/fuels-wallet/releases/tag/v0.0.1


Feature Request: 0CC-FT Fixed Tempo mode

This mode entirely ignores tempo calculation, instead directly using the Speed (or resulting speed from grooves) value for row ticking.
This is implemented in .asm export by writing 0 to the Tempo value.

...what? How does this differ from tempo 150?

I think it would mostly differ in PAL mode, where the base tempo for row-based ticking is actually 125. This should be a quick check while initializing a song: if the tempo is 0, then load either the NTSC or the PAL base tempo, after which the engine will tick at a compatible speed.


feat(media): add music projects + rm Music category

Thanks for fixing it!

yeah LOL, that was embarassing. :D