Execute client synchronously

Hello,
My test involves multiple client execution, is there a way I can set Zopkio to execute next client after the first one is finishes and closes ssh session?
Thanks,
Chuong

:+1:

@arpras do you have any timeline for when this enhancement will be addressed?

I don't fully follow the question, but can we use the test ordering feature of zopkio to help with this and have separate test per client ?
If the above doesn't make sense we probably need to write a wrapper to do this(or create a test recipe). In terms on inherent support for this in Zopkio I am not sure If I will be able to add something in the near time. I will be super happy to merge any pull request though.
Thanks

Hello Arpas, I'm on the same team with Chuong at Pubnub.
let me give you an example of what we are doing/looking for:
Imagine I am testing a login system.
For this test, I need to make GET requests that do the following:
Create and account
Log in a user to that account
Modify some account details
Get those account details.
In that order.  Because you have to create the account first, then log in, then modify, for all the api calls to work.
This flow would be a single test.
Right now, in the framework, my intuition would be to do the following:
def test_modify_user():
client.start("/create")
client.start("/login")
client.start("/modify")
client.start("/get")
I'm told that this wouldn't work, as the clients aren't guaranteed to sequentially run AND finish each request in order. For example, the user needs to exist before the login request happens.  Is the correct?
One way to solve this problem, as you suggested, is to create a separate client for each test.
So I would do the following
def test_modify_user():
client.start("create_login_modify_get_client")
and the client would do something like:
create_login_modify_get_client()
requests("create")
requests("login")
requests("modify")
requests("get")
The problem with this is that lets say I have 20 API calls that I want to test, and I want to have hundreds of tests that test them in different orders.
Would I have to create a client for each api test ordering?
ie client_call_1_2_3_4...20
and a client_2_1_3_4_5 and a client_5_6_2_1, ect.
That doesn't seem optimal.  One way to "solve" this is to send a list of actions to a single client, in some sort of array  IE ["create", "login", "modify, "get"].  But you start to run into problems there, such as what happens when your login command requires a user_id that is returned by the create command?  (or one of the other 20 api endpoints).  What happens when you want to do more complicated logic based on the return values?
It would be much easier if in each test you could guarantee ordering in some way.
IE
def test_modify_user():
client.start("/create")
do stuff
client.start("/login")
do stuff
client.start("/modify")
do stuff
client.start("/get")
Also what do you mean by wrapper, or test recipe?

Please let me know if the below would work for your need ?
When you do client.start() you can pass an optional argument to run things synchronously.
---- Relevant Code in adhoc_deployer.py----
 with get_ssh_client(hostname, username=runtime.get_username(), password=runtime.get_password()) as ssh:       exec_with_env(ssh, command, msg="Failed to start", env=env, sync=configs.get('sync', False))
This will trigger the below function in remote_helper.py
def better_exec_command(ssh, command, msg):
"""Uses paramiko to execute a command but handles failure by raising a ParamikoError if the command fails.
Note that unlike paramiko.SSHClient.exec_command this is not asynchronous because we wait until the exit status is known"""
Its also used in the examples.
client_deployer.start("client1", configs={"args": "localhost 8001 1 2".split(), "sync": True}) client_deployer.start("client1", configs={"args": "localhost 8002 1 2 3".split(), "sync": True})

Also what do you mean by wrapper, or test recipe?
By wrapper I meant you can extend the existing deployer and create a new deployer for your needs which enforces rules you need.

Test Recipes are a concept of reusing the test by changing the parameters. For example you can take a look at recipes.py

Can I close this ?

Hi Arpras,
Thanks for following up. We actually always have "sync"=True in all tests. And the idea of recipes is nice but not applicable for sharing test data between test cases. We will try to resolve this from the client execution side.
Please close this if you see fit.
Kind regards,
Chuong


Error: entry Object values must be an Array.

Operating System: macOS
Node Version: v9.6.1
Yarn Version: v1.3.2
webpack Version: v4.0.1
webpack-serve Version: v0.1.5

This issue is for a:

[x] bug
[ ] feature request
[ ] modification request

Code
CLI Command
  $ webpack-serve webpack.config.js

webpack.config.js
{
	entry: {
		app: '/var/www/html/js/index.js',
	},
	mode: 'development'
}

Expected Behavior
Given the config above, I would expect to be able to launch a webpack-serve server without any issue.
Actual Behavior
However, the server never starts and I get the following error:
TypeError: webpack-hot-client: `entry` Object values must be an Array. Please check your webpack config.
    at validateEntry (/Users/ev.haus/Git/ux/node_modules/webpack-hot-client/lib/util.js:131:19)
    at module.exports (/Users/ev.haus/Git/ux/node_modules/webpack-hot-client/index.js:32:3)
    at fn (/Users/ev.haus/Git/ux/node_modules/koa-webpack/index.js:74:18)
    at middleware.webpack.state.getPort.then (/Users/ev.haus/Git/ux/node_modules/webpack-serve/lib/server.js:32:27)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:160:7)
    at Function.Module.runMain (module.js:705:11)
    at startup (bootstrap_node.js:193:16)
    at bootstrap_node.js:665:3

Note, the error doesn't appear until you kill the process due to https://github.com/webpack-contrib/webpack-serve/issues/25
If you change entry.app from '/var/www/html/js/index.js' to ['/var/www/html/js/index.js'] then the server starts up just fine.
NOTE: This issue was also briefly discussed here: https://github.com/webpack-contrib/webpack-hot-client/issues/11#issuecomment-369387441
How Do We Reproduce?
See config and CLI command and above.

Thanks for the issue 🍺  We'll get a fix out for this quickly.

I would expect to

Haha just a friendly heads up; I appreciate the verbosity (really) but you don't have to preface this section with "I expect to|etc..."


Add Github Action to check for issue numbers in commit messages

This works on pushes as well as pull requests. You can use a "skip issue" label to make the check green.
Caveat 1
Github Actions have a bug: labeled and unlabeled events on pull requests create duplicate checks on pull requests instead of replacing the previous check of the same type. This means we'd have an old failed check and a new passing check in the status, making merging impossible for the pull request. As per @elprans' advice, I attempted a workaround:

don't handle labeled/unlabeled in the main action;
create a separate action to re-run the previously failed check via API and subscribe this one to labeled/unlabeled.

This second action will also have duplicate entries on the pull request if there are many label/unlabel actions applied but since this action is implemented so that it always passes, this does not block the pull request.
Caveat 2
The Checks API required to re-request a run of the check is beta (since May 2018) and for some reason it doesn't work. We send the right request, the API server receives it and is happy with it (HTTP 201). The Check run gets stuck:

On the "Actions" view in the repository you can see that the check is supposed to re-run but it never does.
On the check suite view ("Details" from the pull request) you can see the failed check in the old state, not re-run. However, the status circle of the suite now shows the check pending and the check doesn't have the "Re-run check" button anymore. Instead, there's a "Cancel check suite" button that does nothing.

Note
Updating the pull request's source code fixes the problems with duplication and stuck checks. Under the hood, Github Actions tie check suites with a commit SHA1 so a new commit creates an entirely new suite.
Conclusion
I am rather unhappy with this state of things. We can do the following:

merge this as is and find some friends at Github/Microsoft to help us diagnose and fix either caveat 1 or caveat 2, both of which are bugs in my opinion;
merge this with automatic re-running disabled, then you'd have to either push a change to the pull request to re-run the check or re-run it yourself manually by going to it and clicking its "Re-run checks";
do not merge this, set up a $3.5/month instance on Amazon Lightsail with GidgetHub and do it like CPython does it which we know works pretty well.

The problem with Solution 1 is that it's probably long-term. The Checks API's been around since May 2018. Solution 2 will just be cumbersome. Solution 3 I can only tackle in February.
There's also a potential Solution 4 which would be to poke at the YAML files to figure out a magic combination that makes the Re-run workflow work. @elprans has seen it work in the release pipeline, it's unclear what's different between the two. However, I think this is taking too long and I don't want to sink more time into such a trivial thing.

I'm going to close this because it seems unlikely to move forward.
Also I merge tons of PRs that don't have an issue number associated with them and it doesn't bother me :P


Run lint when building

I use gulp start-server while developing, which depends on build. build doesn't depend on test and on lint though, so I often forget to run lint.

Often when I build I've inserted test code that won't pass lint so I find it a little cumbersome for build to always call lint. Would it be an acceptable solution to have Travis run lint on every push?

@TimAbraldes I've updated the PR to run lint only when you run start-server.

We've discussed this on IRC, and @TimAbraldes was OK with it.


Please check this code!

Please accept 1 at a time. So that there can be two requests..
I hope you will understand.

Make 2 different pull requests.

Adding day3.py first.


Update caret-operator.md

This information should be in an obvious place, such as the operator reference.

OPS Build status updates of commit 18cd63d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Language/Reference/User-Interface-Help/caret-operator.md
:white_check_mark:Succeeded
View

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


How to add another label jsut above the page title

Hi
I want to add label just above to page title can i do this.please see attachment

Hey @sahabe1 , Your issue has been fixed and code has been updated in master branch.
For multiple line in your case ,create tabs as
ViewPagerTab(title: "9\nBuy", image: nil)


Hello, I would like to ask how I can use this source code and generate dll. I am a novice. Thank you.

Hello, I would like to ask how I can use this source code and generate dll. I am a novice. Thank you.

www.google.com

Sorry, I didn't understand

So, you need visual studio 2019 with build tools v143. There are some resources on google to help you figure the tool redirect out.
https://learn.microsoft.com/en-us/answers/questions/769911/visual-studio-2019-build-tools-v143
https://stackoverflow.com/questions/72573273/the-build-tools-for-v143-cannot-be-found-build-tool-installed-using-correct
^^^
Try learning here, then compile


Dependency package @types/eslint has a new version available

We have identified a dependency on version 8.56.11 of @types/eslint. A new version (9.6.0) is available for upgrade.
Following are the steps to upgrade package dependency.

Understand the breaking changes between the version being used and the version you want to upgrade to.

Identify all packages that take a dependency on this package.

Go to the root folder for each such package (/sdk/service-name/package-name) and update package.json to have the new version.

Run rush update to ensure the new version is pulled in.

Make relevant changes to absorb the breaking changes.

Repeat steps 3 to 5 for each of the packages that have a dependency on this package.

upgraded to ^9.9.0


github.com/Sirupsen/logrus package name changed

Hi,
Several days ago package github.com/Sirupsen/logrus changed it's name to github.com/sirupsen/logrus.
Because of that using of vulcand/oxy as library package is breaking dependency tree.
It seems like simply imports update will fix all integration and futher issues

Fixed by https://github.com/vulcand/oxy/pull/84


OpenAI-DotNet 6.4.1

Fixes #64
Added ImageEditRequest overloads for optional mask parameter

Test failure was from models having been removed. I suspect it's a bug on OpenAI's side.


Memcache adapter

Basic implementation of a Memcache adapter.
Perhaps some notes about the preferred usage of the Memcached adapter is required?

Awesome. Thank you! Just fix my minor comment and the code style and I am happy to merge

Have a loog at https://github.com/LeaseWeb/LswMemcacheBundle/blob/master/.travis.yml and https://github.com/LeaseWeb/LswMemcacheBundle/blob/master/.travis-php.ini to see how to configure travis.

Current coverage is 100.00%

Branch #1 has no coverage reports uploaded yet.

No diff could be generated. No reports for master found.

Powered by Codecov. Updated on successful CI builds.

Any idea how to get it to work on HHVM and php7?

I’ll look it up, perhaps the lib hasn’t been ported.
On 7 January 2016 at 23:52:54, Tobias Nyholm<EMAIL_ADDRESS>wrote:
Any idea how to get it to work on HHVM and php7?
—
Reply to this email directly or view it on GitHub.

Awesome. Thank you for this!
Would you mind adding a factory for this in the AdapterBundle? The AdapterFactories are quite light weight. See the MemcachedAdapter and services.yml.

Sure.


Modifying a pointer to a slice from Go doesn't seem to update properly in goja

Modifying a slice value or pointer to a slice from Go doesn't seem to update properly in goja. Consider the following code sample:
package main

import (
    "fmt"

    "github.com/dop251/goja"
)

func main() {
    data := &[]int{}

    vm := goja.New()
    vm.Set("data", data)
    vm.Set("print", fmt.Println)
    vm.Set("append", func(a *[]int, v int) {
        *a = append(*a, v)
    })

    _, err := vm.RunString(`
        // modify with js
        data.push(1);
        print("push:", data.length); // 1

        // modify with go
        append(data, 2)
        print("append:", data.length); // still 1

        // but both elements are there
        print("data.0", data[0]) // 1
        print("data.1", data[1]) // 2
    `)
    if err != nil {
        panic(err)
    }
}

I haven't explored the goja internals yet to understand what actually is happening, but:

Is this some version of the implementation caveats mentioned in the readme?

If it is a caveat, is there a workaround, even if that means sacrificing the performance, eg. overwriting the default array behavior?

This is a case of an overzealous optimisation around the length property, and not maintaining the original value type for non-reflect slices (does not directly apply to your case, but still needs fixing).


[DOCFIX]Add doc for Key

Add doc for UnderFileSystemBlockStore#Key.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15029/
Test PASSed.

Merged build finished. Test PASSed.


Sync from local to Dropbox after initial sync is not working at symlinked directory

Synchronization from local server to Dropbox is not functional after initial sync. Initial sync is ok, but when I add new file or folder in symlinked folder on local side it is not synchronized to Dropbox, from Dropbox side synchronization working. When I stop Maestral and start again, my changes from local are synced to Dropbox. Synchronization from regular folders working fine.
System:

Maestral version: 1.4.6
Python version: 3.6.8
OS: CentOS 7.9.2009
Desktop environment: without gui

fs.inotify.max_queued_events = 16384
fs.inotify.max_user_instances = 1024
fs.inotify.max_user_watches = 524288
Usage:        38.4% of 3.3 TB used
Status:       Up to date
Sync errors:  0

Yes, that is as expected. We currently don't get file system events for symlinked directories so they are only done when running a full scan (at startup or after a pause -> resume). See https://maestral.app/docs/symlinks.
Basically, I am ambivalent about how to treat symlinks, but the situation at the moment is certainly not ideal. In the mid-term, I'd like to do either of the following:

Not follow symlinks at all. Instead, sync the symlink file as-is. This would for example enable relative symlinks to folder inside the same Dropbox without duplicating any files. It could however be problematic when the user also syncs their files to a Windows PC.
Follow symlinks anywhere and explicitly request to receive file system events for (potentially nested) symlinked folders. This would be a bite more work to implement and we would need to handle corner-cases such as recursive symlinks. It would be annoying if the symlink points to another Dropbox folder but convenient otherwise.

Thanks for explanation, I moved data to one big hdd.


Update HadISD station names

Updates the station names in the hadisd_stations.csv to include their ICAO as well as removing the / character in two of the new stations

Looks good to me! Tested with Select and data was properly imported. I think we will eventually need to expand the Select panel to not have the border overlap with station names but that is definitely non-urgent.


Chore: 31 add client Dockerfile

ZH CARD 31

approved as a starting point. Once we have the container working we will address the sonarcloud issues


Order of Pillage and indicator of which character is doing the Pillaging

If possible could you add a prompt when there are multiple Pillage characters in a challenge for the order of their Pillage triggers. This matters when you have characters that trigger off of the cards pillaged and you have some way to stack ("The Bear and the Maiden Fair") or filter (Tywin Lannister (LoCR version)) the opponent's deck. An example of this would be:
You win a challenge with Black Winds Crew (Pillage, but no special abilities), Gregor's Mauraders (Pillage, ability that triggers off pillaging an Event), and Ser Gregor (Pillage, ability that triggers off pillaging a character) with Tywin Lannister (LoCR version) (When a card is discarded from opponent's deck, look at the top 2 and choose which to Pillage) on the board. In this situation it is important to be able to trigger the Black Winds Crew first so that when you look at the top two cards you can decide who to pillage with next in case you see a character you want to kill or an event.
Thanks!

This hasn't been fixed still and it's still as important as it ever was. It can't be difficult to add a prompt/notification to explain whose pillage is being used while reactions/interrupts to it are happening. Ideally, actually let us choose the order.

Please understand that a lot of our priority has been to finish implementing all cards as well as fixing critical bugs. We're closing in on that goal and have begun to spread out to other things now. In allowing Pillage order to be chosen, we've also discussed making keywords like Renown + Insight optional, as well as allowing first player to choose the order of keyword resolution per the rules. These will likely go out at the same time.
As a workaround, please note that Pillage happens in the order you selected attackers currently. So if you were to click Gregor, then click Marauders when selecting attackers, Gregor's pillage will be done first. It isn't ideal, though, hence why this issue exists.


Re-Authenticating

I recently changed my iTunes Connect password and I'm trying to "re-authenticate" with Cupertino but I can't.
You are already authenticated
Username:
[redacted]
Password:
*
security: SecKeychainAddInternetPassword : The specified item already exists in the keychain.
Error saving credentials
I have tried looking for the keychain item, but I have no idea what it is called so I can't find it. I am no Ruby programmer but I tried looking at the source and I couldn't deduce what the keychain item would be either.
We need a an easy way to reauthenticate.

We need a an easy way to reauthenticate.

$ ios logout
$ ios login


stress servers function

it would be nice (if you want and find it usefull) to add stress server function to quasar, methods like udp flood, http get, http post, etc.

DDoS functions will not be added.


glibc: 2.34-115 -> 2.35-30

Description of changes
Announcement: https://sourceware.org/pipermail/libc-alpha/2022-February/136040.html
Note: very early stage, probably needs a Hydra jobset (cc @vcunat @jonringer )

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

let me see if hydra-unstable builds on master...
I can setup a personal jobset..... and it failed :(

So, do you want a jobset on hydra.nixos.org now?  aarch64-linux is relatively free, and that most likely won't change soon.

Let me know, when you drop the static-pie flag and I can test if glibc still builds pie crt files.

They even say the flag was removed:

The --enable-static-pie option is no longer available.  The glibc build
configuration script now automatically detects static-pie support in the
toolchain and architecture and enables it if available.

I assume that this won't aim for NixOS 22.05.

I assume that this won't aim for NixOS 22.05.

Absolutely not! The earliest for this is 22.11 IMHO.

So, do you want a jobset on hydra.nixos.org now?

I assume that this is currently not a good idea considering that we're releasing in the foreseeable future, right?
I don't think I'll be able to do much anyways during the next weeks so I'm fine with waiting until 22.05 was released and then we can create a jobset.
Alternatively, I'd ask @jonringer  (and perhaps help if their Hydra is still broken) because I don't have the resources for that.

At this point rules don't allow inclusion in 22.05 anymore.
Right now Hydra is down, but this week it had some spare x86_64-linux capacity.  We can always prioritize (cancel) in case we accumulate a too long queue.

At this point rules don't allow inclusion in 22.05 anymore.

Well aware of that ;-)

Right now Hydra is down, but this week it had some spare x86_64-linux capacity

if that's possible currently, I'd say let's go for it :)

Please add
disallowedReferences = lib.optional (stdenv.cc.libc != null) stdenv.cc.libc;

this new glibc refers to bootstrap's glibc increasing closure size
it may need
      patchelf --set-interpreter $(readlink -f $bin/bin/ld.so)  $bin/bin/ldconfig
      patchelf --set-interpreter $(readlink -f $bin/bin/ld.so)  $bin/bin/sln

in postInstall to fix (but I tested vanilla 2.35,  2.35-30 might have it fixed; anyway, disallowedReferences would be good to avoid future regression)

also resulted ${glibc.bin}/bin/ldd is a shell script starting with !#/bin/bash
it should be replaced with !#/bin/sh or patchShebangs'ed

Do we still need the C.UTF-8 stuff in https://github.com/NixOS/nixpkgs/blob/master/pkgs/development/libraries/glibc/default.nix ?

Probably not; the support should've been upstreamed, perhaps even earlier than 2.35.


I don't have access to homework submission repository

I'd like to confirm my gold access so that I can submit my homework to the repository. Thank you! :)

@minnie89 please tag admin or DM them so that you can get access soon.

@hsharma2304 who is the current admin to tag?

@nabea1 need to confirm this in discord group please message their in channel.

@nabea1 In Discord the disc-access channel is the place to request status verification & access to the VIP or gold channels.


Update NVAccess.NVDA manifests to all have a ProductCode and X86 architecture

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

#65253 (comment) You need to make separate pull requests for each of the packages.

I assume that would make te pipelines run succesfully, but it should be possible to review this pr manually, right?

Oh i didn't see the comments

I assume that would make te pipelines run succesfully, but it should be possible to review this pr manually, right?

They don't manually merge PRs other than for very special reasons, any normal PRs have to be separate until the pipeline supports multiple package version changes

Ugh, ooh well. I will do separate pull requests then.


OT URNs not resolving or not listed correctly

I was checking proofs for an article and noticed some OT URNs not functioning properly..
urn:cts:copticLit:ot when put in the search box does not resolve.  The general URL  http://data.copticscriptorium.org/urn:cts:copticLit:ot does not resolve either
On the corpus page the URN for sahidica.ruth is incorrectly listed as urn:cts:copticLit:ot . It should be urn:cts:copticLit:ot.ruth.coptot . It looks like the document URN is correct.
I don't know if this is related to an issue we previously closed https://github.com/CopticScriptorium/cts/issues/126
Thanks in advance for your help!

Ruth should have its correct corpus URN listed in the mappings file. The error is coming from the fact that Ruth's automatically-assigned corpus URN (which as we know is unreliable) is identical to the old testament's URN, and our code assumes that no two corpora will have the same URN.

Thanks Luke!
I think I’m not sure I understand. I don’t see Ruth in the linked mapping file. Are you saying that’s the problem?

Oops, that "should" was ambiguous. I intended deontic should, not epistemic :)
Yeah, I can't find the discussion where we decided this atm, but because it's impossible for us to automatically infer from a document URN what the correct corpus URN ought to be, we need to maintain an explicit list of them for every corpus in that file. I think @amir-zeldes's been maintaining it for the most part so far. I just added the line for Ruth and updated it in prod.

Thanks @lgessler - that is all correct, the corpus URN is not automatically deducible from anything, so it has to be mapped manually in this file: https://github.com/CopticScriptorium/cts/blob/master/coptic/gh_ingest/name_mapping.tab
Personally I'd prefer to see it be a corpus metadata, but that would mean we'd need to enter it and remember it for each corpus (it's probably like this because it's almost easier to have them all in this one file, and we must have launched this feature between releases, with corpora simply not having such a metadatum). BTW the same applies to corpus 'human' titles, which are specified in the same tab file.

Got it @lgessler thanks!
It's usually pretty easy to figure out what the corpus URN is manually, so I'm happy for either method. If it can be added to corpus metadata with validation that would be great. But won't we still need to enter it into the mapping file, since the architecture of the software is?

If we decide to go with corpus metadata, the architecture would need to be updated to reflect that (two mechanisms are a recipe for trouble)

I think then for now we should stick with updating the file, unless you think we'll update the architecture . I will add it to the wiki workflow!

Ok I updated the wiki.

Thanks!


Inconsistency between design documentation and generated C definition for dynamic arrays

Reading through the design documentation  of Libcanard, I see that the corresponding C definition of a dynamic array should be an array with the maximum number of elements which is allowed in the dynamic array. E.g Foo[<5] bars in DSDL should become Foo bars[4]; in C.
As of now, dynamic arrays are created as pointers, not as arrays. Is this the desired behavior, or should the pointers be replaced with the definition stated in the design documentation?

I don't have any objections against this deviation. As to the motivation, we should ask @storola.

I asume the reason is to allow application designers to use a 32 byte buffer even if the datatype supports 256.
I agree it's a bit weird that the behavior is different between composite types and arrays and even more between dynamic arrays and static arrays. And I would prefer that this was made more consistent.

This issue is somewhat hanging in an odd position where it's neither a bug nor a discussion. Do we want to change the behavior to what is specified in the design document?
Pros: simpler and less error-prone API; cons: higher memory footprint.
On one hand, Libcanard tends to attract people who are stuck with their memory-impaired microcontrollers, so that change might be seen as a regression. On the other hand, if one can't afford an extra couple of hundred bytes on the stack, they are probably better off serializing stuff manually than asking a computer to do that for them.

On the other hand, if one can't afford an extra couple of hundred bytes on the stack, they are probably better off serializing stuff manually than asking a computer to do that for them.

I agree with this point of view. The "sensible default" should definitely lean towards ease of use.
Perhaps we can give the 8bit guys some extra control by allowing a preprocessor define called DSDLC_DYNAMIC_ARRAYS_AS_POINTERS or similar to opt into the manual version?

Perhaps we can give the 8bit guys some extra control by allowing a preprocessor define called DSDLC_DYNAMIC_ARRAYS_AS_POINTERS or similar to opt into the manual version?

Uh oh, looking at the API I can't imagine how that can be done easily. The external API of serialization functions depends on that quite a bit.

As of now, dynamic arrays are created as pointers, not as arrays. Is this the desired behavior, or should the pointers be replaced with the definition stated in the design documentation

This is desired behavior. Dynamic arrays in C are pointers, I don't see any problem with this approach. I wanted to create library work like this, this allows users to allocate memory as little as needed. This allows a lot of control for the user, it is not strict to any heap allocation like malloc / alloc (but can be used if used in system). Using this approach it also unfortunately gives responsibility to user to reserve enough space for the dynamic arrays.

I agree it's a bit weird that the behavior is different between composite types and arrays and even more between dynamic arrays and static arrays. And I would prefer that this was made more consistent.

In static vs dynamic array, it works a differently because there is no possibility to save RAM in static array definition, but you need to reserve that always, even if not needed.
So overall I don't want to this generated approach to be changed as it gives a lot of flexibility. True, it is not easiest API to use, but you have way bigger libuavcan for that. So if wanted maybe documentation could be updated to make this more clear.

Nobody seems to mind, closing.


Erroneus activation when computer locked/unlocked on Windows 10 Pro

I use 3RVX as a volume control on a Windows 10 Pro 64-bit system. My volume shortcut is set to WINDOWS KEY + mouse wheel Up / Down. This works fine normally.
But:
If I lock my desktop with WINDOWS KEY + L, and then unlock by entering my password, when I next scroll the mouse wheel the volume changes up/down. It should not change because I am not still holding the WINDOWS KEY. It seems that 3RVX has not detected the release of the WINDOWS KEY after I locked my screen.
Workaround:
If I press and release the WINDOWS KEY then 3RVX works properly again, i.e. the volume only changes when the WINDOWS KEY is held and the mouse wheel is scrolled at the same time.
Can you add some code to the hotkey routines to check the current status of the WINDOWS KEY (and other accelerator keys) to fix this?

Good analysis. I have this same problem, associated it with not properly checking Windows Key state on unlock, but didn't consider that the problem may be a residual state from Window Key + (L)ock, which I also use frequently.


Parcelable issue for relationship entity

Parcelable seems not working for relationship entity.
Before I start an activity, the entity fields are good:

Then I check in the received activity, all relationship fields are null:

Did I miss something? Is this related to lazy loading?

Hi parceling relational data is not supported, only the direct fields of the entity. This is because the relational graph of objects can have cycles and refer back to the same object which is not supported by Parcelable. In general it doesn't make sense to do that inside your own process, instead include a unique id for the object in the bundle and retrieve it using findByKey this is more efficient and when combined with entity caching (which is default enabled) preserves the in memory state of the object.

You can find an example of this here

OK, good to know. Thanks a lot!

Sure :)


[release-1.6] Use CA bundle if provided via command args.

Manual cherrypick required.
#23612 failed to apply on top of branch "release-1.6":
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pilot/pkg/bootstrap/sidecarinjector.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/bootstrap/sidecarinjector.go
CONFLICT (content): Merge conflict in pilot/pkg/bootstrap/sidecarinjector.go
Patch failed at 0001 Use CA bundle if provided via command args.

cc @shakti-das
I'm unable to assign this issue to you as you are not part of the istio community. Please manually cherry-pick commit, link to this issue and tag @ramaraochavali on it for approval as well.


feat: rate limiter support auto tune

Search before asking

[X] I had searched in the issues and found no similar issues.

Motivation
https://rocksdb.org/blog/2017/12/18/17-auto-tuned-rate-limiter.html
Solution
https://github.com/apache/incubator-kvrocks/blob/30e9fd788a4acb585566a008f4d5fff65ee06ce3/src/storage/storage.cc#L163
change to this
rocksdb::NewGenericRateLimiter(
        rate_limiter_bandwidth(),
        rate_limiter_refill_period_us(),
        rate_limiter_fairness(),
        rocksdb::RateLimiter::Mode::kWritesOnly,
        rate_limiter_auto_tuned()
      );

Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Hi @wanghenshui
Thanks for your proposal, would you like to take this?

Seems that it can benefits I/O scheduling rather than using kIORateLimitMaxMb as fixed io limits? Should we default using that or just as a config?

Hi @wanghenshui
Thanks for your proposal, would you like to take this?

ok

Seems that it can benefits I/O scheduling rather than using kIORateLimitMaxMb as fixed io limits? Should we default using that or just as a config?

I think it'd be better to make this as default. How do you think?

I guess we can, but we'd better testing it first. @xiaobiaozhao would you like to take a look or do some testing if auto tune rate limiter is supportted?

This parameter looks very good, can promote the whole DB performance college
I can test the parameters of performance improvement this weekend


background mode not working when app's minimized!

I'm using background mode to keep my app running in background. it work if i turn off my screen. but when i minize app, my code in webview stop running after 5 min.
My code inside platform.ready() :
this.backgroundMode.enable();
    this.backgroundMode.overrideBackButton();

    this.backgroundMode.excludeFromTaskList();

    cordova.plugins.backgroundMode.disableBatteryOptimizations();

    this.backgroundMode.disableWebViewOptimizations();

I'm using this plugin https://github.com/boltex/cordova-plugin-powermanagement with backgroundmode to make it working with android 8.0
I don't know if it's enough for android 9.0
P

I'm using websocket as core function of my app. I press home button to make app run in background then i turn screen off. after 5 min later, i open my app and i see socket status label is connecting.do your app working in this case. sr for my bad english

Hi, I have the same problem, I even used an app to keep the screen on, but my app is being minijmized from time to time, and that makes the javascript (ajax calls) stop,
Can you give me and advice please ?


Responsive Images in README.md

Description

Changed image styling in README.md to make them responsive.
Added PNG images in Images/ with transparent backgrounds. (Have not deleted the original JPG Images)

Fixes #16
Type of change
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this :

[] Add a new project

[] Modified or enhanced the existing projects

[x] Improve the readme of any project

Domain of Contribution
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[] Project using only Solidity

[] Project using only Javascript

[] Project using only Python

[] Project Test on the Remix IDE

[] Mention below if project uses other language and ide .

Checklist ✓
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] l have performed a self-review of my own code or work.

[] I have commented on my code, particularly in hard-to-understand areas.

[x] My changes generate no new warnings

[] I'm HSOC'22 contributor

[] I'm SSOC'22 contributor

Screenshots/Gif (Recommended)

Excellent LGTM . Your codes look's great and clean .
I will  merge your PR ASAP . This will not affect your  Hacktoberfest profile means u will see the review timeline for this .
If this not happen let me know ASAP .
@SadashayKanungo


Will it be possible to use the large-v3 model?

Guillaume started a job as Machine Learning Engineer at Apple last month (which he absolutely deserved to get), so I honestly don't think he'll have the time to continue his work on faster-whisper :(

I tried to do this, but I think this can only be done when OpenAI uploads the model to huggingface, maybe.

The weights are open-source so they should it should be possible to upload?
https://openaipublic.azureedge.net/main/whisper/models/e5b1a55b89c1367dacf97e3e19bfd829a01529dbfdeefa8caeb59b3f1b81dadb/large-v3.pt
https://github.com/openai/whisper/pull/1761/files

I think this is not only conversion problem. The new large-v3 model uses 128 Mel frequency bins instead of 80 which is hardcoded in faster whisper now.

Change the feature_size of the FeatureExtractor from 80 to 128.

i kind of got it working by converting the pt with the openai to hf converter script and then running the ct2 converter on that + the tokenizer.json copied from large-v2
[23.60s -> 28.60s]  You changed it to Ron when you bought your first Ron's coffee shop six years ago.
[28.60s -> 32.60s]  Now you got 17 of them with eight more coming next quarter.
[32.60s -> 36.75s]  May I help you with something?
[36.75s -> 40.75s]  I like coming here because your Wi-Fi was fast.
[40.75s -> 44.75s]  I mean, you're one of the few spots that has a fiber connection with gigabit speed.
[44.75s -> 46.75s]  It's good.
[47.45s -> 47.97s] ,
[47.97s -> 48.47s] 제를,
[58.93s -> 73.47s]  and I only, the Martinez, bad, I see, that's, is, and you know, it, it's, that, that, the negative, and I know, . . This, and I, inter, I look, William, the model, and I mean, I know throwing the society kettle, and, I, like, descon, the Mess, and, the, or the Rad, the head, I mean, he, and, but, it. So, you'reтом, I, the VI, the sub, I took, and, called the holes, and Iemente, so I, the cow такую, YouTube don't, and, the, the gear, collaborative,業vez. I welcome, the fortunate, and, و, the threekan, the handy, remote,ばい, degree,nem, Frank, de,б, "Con, �,rent, besoin, نہ, MR's,
[76.77s -> 81.03s]  protocol it's not as anonymous as you think it is whoever's in control of the
[81.03s -> 86.67s]  exit nodes is also in control of this traffic which makes me the one in

then tried copying over the config files from large-v2 (everything except for the model files) & adjusting as necessary ("num_mel_bins": 128, "vocab_size": 51865). didnt change any of the token ids
[14.05s -> 49.17s]  Uh, you're Ron, but your real name's Rohit Mehta. You changed it to Ron when you bought your first Ron's Coffee shop six years ago. Now you got 17 of them with eight more coming next quarter. May I help you with something? I like coming here because your Wi-Fi was fast. I mean, you're one of the few spots that has a fiber connection with gigabit speed. It's good. It's so good it scratched that part of my mind.
[49.17s -> 79.15s]  reconnect the data, but it's real. It's all in one thing. Part that doesn't allow good to exist without condition. So I started intercepting all the traffic on your network. That's when I noticed something strange. That's when I decided to hack you. Hack? I know you run a website called Plato's Boys. Pardon me? You're using tor networking to keep the servers anonymous. You made it really hard for anyone to see it. But I saw it. The onion rooting protocol. It's not as anonymous as you think it is.

.....(eventually broke down)

[205.46s -> 235.44s] ,
[215.74s -> 215.88s] G.
[216.30s -> 216.74s] G,
[216.80s -> 221.62s] ,
[221.62s -> 221.66s] ,
[221.66s -> 221.98s] ,

also did the second method with large-v2.pt and it works perfectly. just gotta wait for the official hf release now but if you really want to get it working now, play around with tokenizer.json and the token ids in config.json

also did the second method with large-v2.pt and it works perfectly. just gotta wait for the official hf release but if you really want to get it working now, play around with tokenizer.json and the token ids in config.json

Thanks!

i kind of got it working by converting the pt with the openai to hf converter script and then running the ct2 converter on that + the tokenizer.json copied from large-v2
[23.60s -> 28.60s]  You changed it to Ron when you bought your first Ron's coffee shop six years ago.
[28.60s -> 32.60s]  Now you got 17 of them with eight more coming next quarter.
[32.60s -> 36.75s]  May I help you with something?
[36.75s -> 40.75s]  I like coming here because your Wi-Fi was fast.
[40.75s -> 44.75s]  I mean, you're one of the few spots that has a fiber connection with gigabit speed.
[44.75s -> 46.75s]  It's good.
[47.45s -> 47.97s] ,
[47.97s -> 48.47s] 제를,
[58.93s -> 73.47s]  and I only, the Martinez, bad, I see, that's, is, and you know, it, it's, that, that, the negative, and I know, . . This, and I, inter, I look, William, the model, and I mean, I know throwing the society kettle, and, I, like, descon, the Mess, and, the, or the Rad, the head, I mean, he, and, but, it. So, you'reтом, I, the VI, the sub, I took, and, called the holes, and Iemente, so I, the cow такую, YouTube don't, and, the, the gear, collaborative,業vez. I welcome, the fortunate, and, و, the threekan, the handy, remote,ばい, degree,nem, Frank, de,б, "Con, �,rent, besoin, نہ, MR's,
[76.77s -> 81.03s]  protocol it's not as anonymous as you think it is whoever's in control of the
[81.03s -> 86.67s]  exit nodes is also in control of this traffic which makes me the one in

then tried copying over the config files from large-v2 (everything except for the model files) & adjusting as necessary ("num_mel_bins": 128, "vocab_size": 51866). didnt change any of the token ids
[14.05s -> 49.17s]  Uh, you're Ron, but your real name's Rohit Mehta. You changed it to Ron when you bought your first Ron's Coffee shop six years ago. Now you got 17 of them with eight more coming next quarter. May I help you with something? I like coming here because your Wi-Fi was fast. I mean, you're one of the few spots that has a fiber connection with gigabit speed. It's good. It's so good it scratched that part of my mind.
[49.17s -> 79.15s]  reconnect the data, but it's real. It's all in one thing. Part that doesn't allow good to exist without condition. So I started intercepting all the traffic on your network. That's when I noticed something strange. That's when I decided to hack you. Hack? I know you run a website called Plato's Boys. Pardon me? You're using tor networking to keep the servers anonymous. You made it really hard for anyone to see it. But I saw it. The onion rooting protocol. It's not as anonymous as you think it is.

.....(eventually broke down)

[205.46s -> 235.44s] ,
[215.74s -> 215.88s] G.
[216.30s -> 216.74s] G,
[216.80s -> 221.62s] ,
[221.62s -> 221.66s] ,
[221.66s -> 221.98s] ,

also did the second method with large-v2.pt and it works perfectly. just gotta wait for the official hf release but if you really want to get it working now, play around with tokenizer.json and the token ids in config.json

Thanks!

Was there any confirmation that OpenAI will upload the model to huggingface?

i kind of got it working by converting the pt with the openai to hf converter script and then running the ct2 converter on that + the tokenizer.json copied from large-v2
[23.60s -> 28.60s]  You changed it to Ron when you bought your first Ron's coffee shop six years ago.
[28.60s -> 32.60s]  Now you got 17 of them with eight more coming next quarter.
[32.60s -> 36.75s]  May I help you with something?
[36.75s -> 40.75s]  I like coming here because your Wi-Fi was fast.
[40.75s -> 44.75s]  I mean, you're one of the few spots that has a fiber connection with gigabit speed.
[44.75s -> 46.75s]  It's good.
[47.45s -> 47.97s] ,
[47.97s -> 48.47s] 제를,
[58.93s -> 73.47s]  and I only, the Martinez, bad, I see, that's, is, and you know, it, it's, that, that, the negative, and I know, . . This, and I, inter, I look, William, the model, and I mean, I know throwing the society kettle, and, I, like, descon, the Mess, and, the, or the Rad, the head, I mean, he, and, but, it. So, you'reтом, I, the VI, the sub, I took, and, called the holes, and Iemente, so I, the cow такую, YouTube don't, and, the, the gear, collaborative,業vez. I welcome, the fortunate, and, و, the threekan, the handy, remote,ばい, degree,nem, Frank, de,б, "Con, �,rent, besoin, نہ, MR's,
[76.77s -> 81.03s]  protocol it's not as anonymous as you think it is whoever's in control of the
[81.03s -> 86.67s]  exit nodes is also in control of this traffic which makes me the one in

then tried copying over the config files from large-v2 (everything except for the model files) & adjusting as necessary ("num_mel_bins": 128, "vocab_size": 51866). didnt change any of the token ids
[14.05s -> 49.17s]  Uh, you're Ron, but your real name's Rohit Mehta. You changed it to Ron when you bought your first Ron's Coffee shop six years ago. Now you got 17 of them with eight more coming next quarter. May I help you with something? I like coming here because your Wi-Fi was fast. I mean, you're one of the few spots that has a fiber connection with gigabit speed. It's good. It's so good it scratched that part of my mind.
[49.17s -> 79.15s]  reconnect the data, but it's real. It's all in one thing. Part that doesn't allow good to exist without condition. So I started intercepting all the traffic on your network. That's when I noticed something strange. That's when I decided to hack you. Hack? I know you run a website called Plato's Boys. Pardon me? You're using tor networking to keep the servers anonymous. You made it really hard for anyone to see it. But I saw it. The onion rooting protocol. It's not as anonymous as you think it is.

.....(eventually broke down)

[205.46s -> 235.44s] ,
[215.74s -> 215.88s] G.
[216.30s -> 216.74s] G,
[216.80s -> 221.62s] ,
[221.62s -> 221.66s] ,
[221.66s -> 221.98s] ,

also did the second method with large-v2.pt and it works perfectly. just gotta wait for the official hf release but if you really want to get it working now, play around with tokenizer.json and the token ids in config.json

can you share the converted v3 model , put it in some net drive , like google drive, and related modified files, so anyone want to use it can just copy it , thanks

Was there any confirmation that OpenAI will upload the model to huggingface?

According to this comment, it is converting now
(https://github.com/openai/whisper/discussions/1762#discussioncomment-7496805)

Was there any confirmation that OpenAI will upload the model to huggingface?

According to this comment, it is converting now (openai/whisper#1762 (reply in thread))

Alright lets go!

Hello. I wrote to Guillaume to see if he is willing to accept help to maintain the project. I have an old Guillaume's email address. If somebody has a recent email that works please send it to me<EMAIL_ADDRESS>

@jordimas Guillaume said ping to nguyendc-systran, there I did, let's see if he shows up.

this PR should work: https://github.com/guillaumekln/faster-whisper/pull/548

it doesn't, i just tested it and the provided ct2 conversion is the same as my method 1 above, also alignment doesn't work
Traceback (most recent call last):
  File "c:\git\faster-whisper-3\test.py", line 22, in <module>
    for segment in segments:
  File "c:\git\faster-whisper-3\faster_whisper\transcribe.py", line 947, in restore_speech_timestamps
    for segment in segments:
  File "c:\git\faster-whisper-3\faster_whisper\transcribe.py", line 563, in generate_segments
    self.add_word_timestamps(
  File "c:\git\faster-whisper-3\faster_whisper\transcribe.py", line 790, in add_word_timestamps
    alignment = self.find_alignment(
  File "c:\git\faster-whisper-3\faster_whisper\transcribe.py", line 900, in find_alignment
    result = self.model.align(
RuntimeError: CUDA failed with error out of memor

just gotta wait for the hf release to do a proper conversion

hmm, you're right, it returned correct results on very short segments I tested but is nonsense on longer segments. weird, I wonder why this is.

think its the tokenizer copied from large-v2, depending where they put in the new Cantonese token a lot of the token ids could be offset
fwiw turning temperature down to 0 has given me reproducible output across all the conversions i have tried so far, previously it was random, frequently non-english text that made me suspect the language switching but its probably (hopefully) just a side effect of the tokens being off
[1.66s -> 23.60s]  You're Ron, but your real name is Rohit Mehta.
[23.60s -> 28.60s]  You changed it to Ron when you bought your first Ron's coffee shop six years ago.
[28.60s -> 32.60s]  Now you got 17 of them with eight more coming next quarter.
[32.60s -> 36.75s]  May I help you with something?
[36.75s -> 40.75s]  I like coming here because your Wi-Fi was fast.
[40.75s -> 44.75s]  I mean, you're one of the few spots that has a fiber connection with gigabit speed.
[44.75s -> 46.75s]  It's good.
[47.51s -> 48.41s] ,
[49.23s -> 58.65s]  I'm I'm I'm I'm I know, I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm a
[90.66s -> 103.54s] , I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I'm I's I'm I's I'm I's I's I's I'm I's I's I's I's I's I's I's I's I's I's like, I's I's like, I's I's I's I's I's I's I's the
[295.81s -> 305.81s] , I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I, I,
[305.81s -> 315.83s] ,

The model is available now at https://huggingface.co/openai/whisper-large-v3 thanks to @sanchit-gandhi !
I tried converting it to CT translate with the following command: ct2-transformers-converter --model openai/whisper-large-v3 --output_dir faster-whisper-large-v3 \     --copy_files tokenizer.json --quantization float16 (same as used for v2)
However, I get this error:
OSError: openai/whisper-large-v3 does not appear to have a file named
pytorch_model.bin, tf_model.h5, model.ckpt or flax_model.msgpack.

The huggingface repo does indeed only have model.safetensors for the model weights.
Anybody have a solution for this? Can we convert the .safetensors to .bin?

@thomasmol there is no tokenizer.json, only the tokenizer_config.json. renaming that didn't work but i wrote a quick script to save the tokenizer and copy the files over
from transformers import AutoTokenizer

tokenizer = AutoTokenizer.from_pretrained("openai/whisper-large-v3")
tokenizer.save_pretrained("./whisper-large-v3")

and it seems to be working, uploading to hf now

bababababooey/faster-whisper-large-v3

Hey, sorry to jump in at the last minute. What do i have to do to use this now? bababababooey/faster-whisper-large-v3

@User1231300 my fork https://github.com/bungerr/faster-whisper-3 should work in the meantime while we work on getting #548 merged
if you want to use english only, pass is_multilingual=False to WhisperModel(). Otherwise it's set to multilingual as default
model = WhisperModel(
    'large-v3', device="cuda", compute_type="float16", is_multilingual=False
)

thanks a lot for the effort, been waiting for this and will try it later.
had some quick tests with the official large-v3 yesterday but the performance was not very satisfactory, with more errors and duplicates when transcribing Japanese.

@thomasmol checkout this repo. It has pytorch_model.bin file
https://huggingface.co/versae/whisper-large-v3/tree/main

thanks all contribution for whisper-v3!
I found some mismatch v2 and v3 at whisper.c in Ctranslate2. So I fixed it.
compile this pacakage if you need to use multiligual mode at faster-whisper 3
https://github.com/circuluspibo/CTranslate2

thanks all contribution for whisper-v3!
I found some mismatch v2 and v3 at whisper.c in Ctranslate2. So I fixed it. compile this pacakage if you need to use multiligual mode at faster-whisper 3
https://github.com/circuluspibo/CTranslate2

can you please give more info how I can do this?

@bungerr
Been experimenting with your fork and can't get language detection to work, it will translate to english if language is not specified.

@circuluspibo want to make a PR to upstream? it feels like that will resolve a lot of issues.

@circuluspibo want to make a PR to upstream? it feels like that will resolve a lot of issues.

There was already https://github.com/OpenNMT/CTranslate2/pull/1530 fixing that issue (among others).

The model is available now at https://huggingface.co/openai/whisper-large-v3 thanks to @sanchit-gandhi ! I tried converting it to CT translate with the following command: ct2-transformers-converter --model openai/whisper-large-v3 --output_dir faster-whisper-large-v3 \ --copy_files tokenizer.json --quantization float16 (same as used for v2) However, I get this error:
OSError: openai/whisper-large-v3 does not appear to have a file named
pytorch_model.bin, tf_model.h5, model.ckpt or flax_model.msgpack.

The huggingface repo does indeed only have model.safetensors for the model weights. Anybody have a solution for this? Can we convert the .safetensors to .bin?

Pursuant to the conversation I STARTED HERE, they graciously uploaded the Float32 version, and I believe that the .bin files are up there now.  However, they need to be combined before trying to convert, is that correct?  Here's an example regarding a different model:
Windows
COPY /B dolphin-2.2-70b.Q8_0.gguf-split-a + dolphin-2.2-70b.Q8_0.gguf-split-b dolphin-2.2-70b.Q8_0.gguf
del dolphin-2.2-70b.Q8_0.gguf-split-a dolphin-2.2-70b.Q8_0.gguf-split-b

Linux/Mac
cat dolphin-2.2-70b.Q8_0.gguf-split-* > dolphin-2.2-70b.Q8_0.gguf && rm dolphin-2.2-70b.Q8_0.gguf-split-*

Assuming that we have the .bin...as far as converting (either the float32/float16)...the Ctranslate2 repository is working on it right now and I think they're close to a solution if not complete.  See HERE.  i'm no expert, but maybe wait to see how the converter is ultimately modified in Ctranslate2 since faster-whisper relies on it?
Interested in helping any way I can.  Thanks!

Standalone Faster-Whisper r160.2 now supports large-v3, only for Windows atm.

Standalone Faster-Whisper r160.3 now supports large-v3, only for Windows atm.

Nice, I'll take a look.  Does it use the Float32 or Float16, both?

Nice, I'll take a look. Does it use the Float32 or Float16, both?

Models? int8_float32 model by default, if you want float16 then type large-v3-fp16.

Cool, I'll check the --help but thanks for the tip.  How did you implement the large-v3 so quickly or is it a trade secret?  I know the people at the Ctranslate2 github have been working on it, maybe they solved it and you implemented it?  I'd like to use the large-v3 in a python script, not CLI, but if you did it a proprietary way I can respect that...

#578 has implemented v3


Progress stepper

https://github.com/mu29/react-stepper

https://material-ui-next.com/demos/steppers/


V0.8.8

All Submissions Basics:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 1972

119 of 124 (95.97%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 95.615%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/test/test_copod_parallel.py
77
78
98.72%

pyod/models/copod.py
42
46
91.3%

Totals

Change from base Build 1889:
0.2%

Covered Lines:
5015

Relevant Lines:
5245

💛  - Coveralls


Update readme example to pin to major version

👋 Hey I'm working on adding in-the-box actions for GHES. I noticed the README doesn't pin to a major version. Pinning to a major version is good for customer stability. New major versions are intended for compat breaking changes. So for example, customers referecing @master may break when behavior changes in a future major version.


ScaleLine's "nautical" suffix should be in uppercase

Describe the bug
The ScaleLine control uses "nm" as suffix for Nautical Miles. However, "nm" is the abbreviation for nanometer.
To Reproduce

https://openlayers.org/en/latest/examples/scale-line.html
Select "nautical mile" and "ScaleLine" from the select elements.

Expected behavior
ScaleLine should use "NM" as suffix for Nautical Miles.

Yes NM should be better :
ol/control/ScaleLine.js
line 292

Fixed with #13554


the estimation result on 3DPW using ResNet50 backbone

Hi, I'm curious about the quantitative performance of METERO with ResNet50 backbone on the 3DPW dataset, since the official repo doesn't provide the pre-trained models with ResNet50. I'd be grateful if any advice was given.

And the joints for estimation are predicted by the network, instead of the regressed ones obtained by using the pre-defined regression matrix. Have I got this right?

Q&A1: Unfortunately we didn't try ResNet50 backbone on 3DPW dataset. In our early experiments, we were using Human3.6M for the ablation study of the use of different backbones. We found HRNet gives better results, so we use HRNet for the rest of the experiments.
Q&A2: Yes. We mainly evaluate the 3D joints which are regressed from the 3D mesh via the pre-defined regression matrix. This is because we want to understand the 3D pose of the estimated 3D mesh. In fact, in our early explorations, we have tried to evaluate the 3D joints which are directly predicted by the network. It actually gives very similar results (about ~0.1 mPJPE).

Just want to add more comments about Q2.
Since the 3D joints we used are computed from 3D mesh, you may wonder what if we use transformer to just predict 3D vertices. In fact, in our early explorations, we have tried to use transformer to predict vertices only. But the training was not converging well.
To make it converging, we found that we need to have both joint and vertex queries, and train our transformer to predict both joints and vertices. We think this is probably because in this approach, we can better leverage self-attention mechanism to directly learn non-local interactions between them, which leads to further improvements.

Thank you for your clear explanation! My confusion has been dispelled :)


Imagenet example error

Environment info
Operating System: Windows with latest docker image (only cpu)
after running: python classify_image.py and loading the model I got the folling error:
Succesfully downloaded inception-2015-12-05.tgz 88931400 bytes.                                                                                [libprotobuf ERROR google/protobuf/src/google/protobuf/io/coded_stream.cc:207] A protocol message was rejected because it was too big (more than 67108864 bytes).  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.                             Traceback (most recent call last):                                                                                File "classify_image.py", line 213, in <module>                                                                                tf.app.run()                                                                                File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/default/_app.py", line 30, in run                                                            sys.exit(main(sys.argv))                                                                                File "classify_image.py", line 209, in main                                                                                run_inference_on_image(image)                                                                                File "classify_image.py", line 159, in run_inference_on_image                                                                                create_graph()                                                                                File "classify_image.py", line 141, in create_graph                                                                                graph_def.ParseFromString(f.read())                                                                                google.protobuf.message.DecodeError: Error parsing message
what can I do ?

This is likely the same issue as issue 582.

For a quick solution type the following before running tensorflow
export PROTOCOL_BUFFERS_PYTHON_IMPLEMENTATION=python

Were you using a precompiled version or building from source?
I used to see the same problem. If you are building from source, updating to the latest code and doing a rebuild should solve the problem (it seems to be fixed in 2f756013f5bcc39ee288eed9da8d27cf4ec16f7e).

@ritsu1228 I'm experiencing this with the official prebuilt 0.7.0 development docker image.

@tmaila Seems like 0.7.0 was released before commit 2f756013f5bcc39ee288eed9da8d27cf4ec16f7e. So I think 0.7.0 should have this problem and your solution works perfect. If you upgrade to 0.7.1, you won't need to do export.

yeah with the latest image I get no errors anymore. Sry for the late response.
Thanks for your help guys!


HV-1304 Add metadata support for container elements

https://hibernate.atlassian.net/browse/HV-1304

This one is on top of https://github.com/hibernate/hibernate-validator/pull/733.

Looks like this one was based on an old version of HV-1313. Opening a new one based on the right version.


Add Generated annotation to kotlin-dsl generated files

JaCoCo ignores classes and methods annotated with any annotation containing Generated in the name ( since 0.8.3, previously at 0.8.2 it was mandatory to be named exactly Generated ) with RetentionPolicy class or runtime.
The kotlin-dsl plugin generates kotlin-dsl-accessors, kotlin-dsl-external-plugin-spec-builders, and kotlin-dsl-plugins folders with lots of files that JaCoCo add to the reports.
Expected Behavior

JaCoCo shouldn't report coverage for any of those generated files.
Current Behavior

The coverage is times lower because there are a lot of generated code.
Context

I'm developing a gradle plugin and the coverage it's really low while the real coverage would be hight.

Thanks for the report Christian, that makes sense.
Here are some pointers about Jacoco and @*Generated*:

https://www.jacoco.org/jacoco/trunk/doc/changes.html, search for "generated"
https://github.com/jacoco/jacoco/commit/2458e8a4e467a5afc376b56a4e487ac54de0158e
https://github.com/jacoco/jacoco/issues/831
https://github.com/jacoco/jacoco/issues/685

It's a shame that @javax.annotation.Generated has source retention 🤷♂

Just in case, I've raised an issue for IntelliJ IDEA: https://youtrack.jetbrains.com/issue/IDEA-224315


fuzzgen: Enable SIMD fuzzing for RISC-V

👋 Hey,
This PR enables fuzzing SIMD instructions for the RISC-V backend. This is the only backend that requires a extension (has_v) for SIMD to work, so we need to check that it has been enabled before allowing SIMD types and instructions.
I've ran this for a while with icache and It's no longer crashing. But I haven't been able to run fuzzgen very well since I don't have a machine with the V extension, and running it with QEMU is quite slow.

Failed MacOS CI with an error in a WASI sockets test? Looks like a flaky test, I'm going to give this another go


Error: 14 UNAVAILABLE: Getting metadata from plugin failed with error: invalid_client

In the project credentials are retrieved from home directory as follows:
const homedir = require('homedir');
const deviceCredentials = require(`${homedir()}/.config/google-oauthlib-tool/credentials.json`);

What I am trying to do is populating deviceCredentials Directory in the code itself:
const deviceCredentials = {
  client_secret: 'xxxxxxxxxxx',
  refresh_token: 'xxxxxxxxxxx',
  scopes: ['https://www.googleapis.com/auth/assistant-sdk-prototype'],
  client_id: 'xxxxxxxxxxxxxxx',
  token_uri: 'https://accounts.google.com/o/oauth2/token',
};

I am getting following error:
Auth error:Error: invalid_client
{ Error: 14 UNAVAILABLE: Getting metadata from plugin failed with error: invalid_client
    at Object.exports.createStatusError (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/common.js:87:15)
    at ClientDuplexStream._emitStatusIfDone (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client.js:235:26)
    at ClientDuplexStream._receiveStatus (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client.js:213:8)
    at Object.onReceiveStatus (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client_interceptors.js:1290:15)
    at InterceptingListener._callNext (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client_interceptors.js:564:42)
    at InterceptingListener.onReceiveStatus (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client_interceptors.js:614:8)
    at /home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/node_modules/grpc/src/client_interceptors.js:1110:18
  code: 14,
  metadata: Metadata { _internal_repr: {} },
  details:
   'Getting metadata from plugin failed with error: invalid_client' }
Error: Error: 14 UNAVAILABLE: Getting metadata from plugin failed with error: invalid_client
    at getAssistantResponse (/home/akshay/Desktop/Workspace/assistant-sdk/remoteAssistant/server.js:34:11)

How are you obtaining the deviceCredentials values? There may be an issue with how you obtained the client_id.

have you solve this issue?

Same issue, I followed the steps register a device model in the README to get my credentials.json
There is not refresh_token value in this file, only a token_uri


for issue #1527

I have update dependency io.swagger.core.v3:swagger-annotations from 2.2.8 to v2.2.9
https://issues.apache.org/jira/browse/SOLR-XXXXX

Description
Please provide a short description of the changes you're making with this pull request.
Solution
Please provide a short description of the approach taken to implement your solution.
Tests
Please describe the tests you've developed or run to confirm this patch implements the feature or solves the problem.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[ ] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

This is a fix for issue 1527


First and Last Aggregator

Reopening https://github.com/druid-io/druid/pull/3226 from @acslk to finish review/merge

Still 👍  from me after we resolve merge conflicts

rebased

@nishantmonu51 can u review?

Also, would be nice to add these aggregators to existing integration tests, we can add them to ITTwitterQueryTest and ITWikipediaQueryTest

@nishantmonu51 Addressed your comments aside from the one about integration tests, haven't gotten around to that yet

@nishantmonu51 @fjy
I added some first/last aggs to ITWikipediaQueryTest
I didn't add them to ITTwitterQueryTest since I saw the queries there weren't testing the full set of aggs, only using doubleSums generally.
I noticed that some other agg types weren't represented in the integration tests (like the max/min or javascript), and it'd be convenient to have a reference to the raw data for twitterstream and wikipedia_editstream, so I'll open issues for those points

👍 after travis

Opened two new issues re: integration tests
https://github.com/druid-io/druid/issues/3783
https://github.com/druid-io/druid/issues/3784


Using '__IOM' for peripheral register definitions

This is pulll request to close #3 if there is specific reason to use "__IO" in the libraries.
IMPORTANT INFORMATION
Contributor License Agreement (CLA)

The Pull Request feature will be considered by STMicroelectronics after the signature of a Contributor License Agreement (CLA) by the submiter.
If you have not signed such agreement, please follow the rules mentioned in the CONTRIBUTING.md file.

This pull request has been refused, the Contribution License Agreement must be signed.


cursor positioning in new messages with signature

RainLoop version, browser, OS: current (1.11.1) / chrome / windows 10
question; when I begin a new message, my signature appears correctly but my cursor is always at the very bottom of the window and i need to move it up above my signature. any way to have the cursor start by default above my signature, but below any quoted text? i have my signature set to be applied after the quoted text, if I'm replying. thanks.

I'm pretty sure this was working fine before. Now, composing a new message and tabbing through the fields, when the cursor reaches the text area, it's after the signature, and not in the first line as expected.
I guess https://github.com/RainLoop/rainloop-webmail/commit/a72b95a86846faa4691626bc0deb2ead0752c32e fixes it but not sure why cursor placing keeps getting broken...

if i have an existing installation, is it safe to update from git via a clone of the repo? what do i have to keep in mind when doing so if possible? Thanks.

I think the GH repo is the source code and needs to be built. The safest way to upgrade is through the admin panel.
https://www.rainloop.net/docs/

understood. thanks. if this issue is fixed then i will close this.


Improve formatting of NoViableAlt exception message.

Uses " ," as delimiters instead of ", "
Spaces after newline
Double space between prefix and suffix of error message.

Now looks like:

Expecting: one of these possible Token sequences:
  1. [X, Y, Z]
  2. [A, B, C]
but found: 'foo'
instead of previously:

Expecting: one of these possible Token sequences:
  <[X,Y,Z] ,[A,B,C]>  but found: 'foo'


Add support for unit for character string types

Add support for optional OCTETS and CHARACTERS after the precision. I haven't seen that syntax for any big dialects that I remember but, since ANSI supports it (1), I wanted to add the possible syntax.
@alamb ready for review :)
[1] : https://jakewheat.github.io/sql-overview/sql-2016-foundation-grammar.html#character-string-type
Related to: https://github.com/sqlparser-rs/sqlparser-rs/issues/661

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

58 of 60 (96.67%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 85.817%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/ast/data_type.rs
19
21
90.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
10510

Relevant Lines:
12247

💛  - Coveralls


Agent Identification feature from advanced profile

Do you see any chance to get the version number automatically set within the build process? Manually having to adjust such things tend to be forgotten.
E.g. have the version string stored in a seperated variable that get's set by Maven (corresponds to Maven library version)?
Apart from that it looks good.

now version is get from manifest.

Looks good to me now! Thanks


Setup Travis

…so we have automated tests

[x] setup travis build
[ ] setup tests so they run against all relevant rails versions
[x] setup tests so they run rubinius and JRuby as well

@oliverbarnes: maybe you can figure out why the JRuby build keeps failing - the needed gem is in the Gemfile, it just never seems to get installed (see e.g. here).

will do


urllib3 error

python3 mastodon_bot/main.py
/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/__init__.py:34: NotOpenSSLWarning: urllib3 v2.0 only supports OpenSSL 1.1.1+, currently the 'ssl' module is compiled with 'LibreSSL 2.8.3'. See: https://github.com/urllib3/urllib3/issues/3020
  warnings.warn(
Traceback (most recent call last):
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connectionpool.py", line 467, in _make_request
    self._validate_conn(conn)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connectionpool.py", line 1092, in _validate_conn
    conn.connect()
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connection.py", line 642, in connect
    sock_and_verified = _ssl_wrap_socket_and_match_hostname(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connection.py", line 783, in _ssl_wrap_socket_and_match_hostname
    ssl_sock = ssl_wrap_socket(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/util/ssl_.py", line 469, in ssl_wrap_socket
    ssl_sock = _ssl_wrap_socket_impl(sock, context, tls_in_tls, server_hostname)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/util/ssl_.py", line 513, in _ssl_wrap_socket_impl
    return ssl_context.wrap_socket(sock, server_hostname=server_hostname)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/ssl.py", line 500, in wrap_socket
    return self.sslsocket_class._create(
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/ssl.py", line 1040, in _create
    self.do_handshake()
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/ssl.py", line 1309, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:1129)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connectionpool.py", line 790, in urlopen
    response = self._make_request(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connectionpool.py", line 491, in _make_request
    raise new_e
urllib3.exceptions.SSLError: [SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:1129)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/adapters.py", line 486, in send
    resp = conn.urlopen(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/connectionpool.py", line 844, in urlopen
    retries = retries.increment(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/urllib3/util/retry.py", line 515, in increment
    raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='3615.computer', port=443): Max retries exceeded with url: /api/v1/apps (Caused by SSLError(SSLError(1, '[SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:1129)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/mastodon/authentication.py", line 73, in create_app
    response = requests.post(f"{api_base_url}/api/v1/apps", data=request_data, headers=headers, timeout=request_timeout)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/api.py", line 115, in post
    return request("post", url, data=data, json=json, **kwargs)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/api.py", line 59, in request
    return session.request(method=method, url=url, **kwargs)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/requests/adapters.py", line 517, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: HTTPSConnectionPool(host='3615.computer', port=443): Max retries exceeded with url: /api/v1/apps (Caused by SSLError(SSLError(1, '[SSL: TLSV1_ALERT_PROTOCOL_VERSION] tlsv1 alert protocol version (_ssl.c:1129)')))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/mastodon_bot/main.py", line 26, in <module>
    main()
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/mastodon_bot/main.py", line 22, in main
    register_app()
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/mastodon_bot/main.py", line 10, in register_app
    Mastodon.create_app(
  File "/Users/mathieu/Code/Python/mastodon-bot-tutorial/lib/python3.9/site-packages/mastodon/authentication.py", line 76, in create_app
    raise MastodonNetworkError(f"Could not complete request: {e}")
mastodon.errors.MastodonNetworkError: Could not complete request: HTTPSConnectionPool(host='3615.computer', port=443): Max retries exceeded with url: /api/v1/apps (Caused by SSLError(SSLError(1, '[SSL: TLSV1_ALE

Can you check your Applications/Python 3.11 directory and see if it has Install Certificates.command in it? If it does try double clicking that to install and see if it resolves the error.

Well I'm afraid I don't even have an Applications/Python 3.9 directory 😕
I do have a ~/Library/Python/3.9 directory but I didn't find any Install Certificates.command in it

Ok, let's see if running this works: pip install --upgrade certifi


[AMP] Fix upload_date and timestamp extraction

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

BleacherReportCMSIE test is also affected by this change.

That video seems to not be available anymore anyway.

BleacherReportCMSIE test is also affected by this change.

That video seems to not be available anymore anyway.

BleacherReportCMSIE test is also affected by this change.

That video seems to not be available anymore anyway.

the extractor does not rely on the webpage.

BleacherReportCMSIE test is also affected by this change.

That video seems to not be available anymore anyway.

the extractor does not rely on the webpage.

the extractor does not rely on the webpage.

You're right, thanks for your diligence.

the extractor does not rely on the webpage.

You're right, thanks for your diligence.


Translate Advanced Guides > Forwarding Refs

Note to Hacktoberfest 🎃 participants:
While we appreciate the enthusiasm, we are experiencing a high volume of drive-by pull requests (one-line changes, README tweaks, etc.). Please remember that hundreds of people are subscribed to this repo and will receive notifications for these PRs. Spam submissions will be closed and won't count towards your Hacktoberfest goals.
Please search for issues tagged good first issue or hacktoberfest to find things to work on.
You can also search all of GitHub.

Progress

[x] 번역 초안 작성 (Draft translation)
[x] 공통 스타일 가이드 확인 (Check the common style guide)
[x] 모범사례 확인 (Check best practices)
[x] 용어 확인 (Check the term)
[x] 맞춤법 검사 (Spelling check)
[x] 리뷰 반영 (Resolve reviews)

@hg-pyun 님, 그런데 한 가지 번역이 되지 않은 곳이 있는데요, 타이틀이요.
제목을 간과했네요. T.T
번역해야 하지 않을까요?
https://ko.reactjs.org/docs/forwarding-refs.html

@hapumee 오.. 그렇군요. 다른 PR로 보내주시면 좋을 것 같습니다.


link to svgjs wrong

it is https://svgjs.dev/ ...

Thank you for reporting this! It will be fixed in the next version.


allow heix ftype brand for heifs

We have a number of heic files that imgproxy cannot process with the error
Can't download source image: Image is not compatible with heic/avif

These heic files seem to be from DSLR cameras, and use the ftyp brand heix in their box metadata:
sandbox/jtomson-imgproxy [allow_heix_ftype] % heif-info -d FILE1.heic
MIME type: image/heic
main brand: heix
compatible brands: mif1, MiHA, miaf, heix
Box: ftyp -----
size: 32   (header size: 8)
major brand: heix
minor version: 0
compatible brands: mif1,MiHA,miaf,heix

sandbox/jtomson-imgproxy [allow_heix_ftype] % heif-info -d FILE2.heic
MIME type: image/heic
main brand: heix
compatible brands: mif1, heix, miaf, MiHA, jpeg, SHIF
Box: ftyp -----
size: 40   (header size: 8)
major brand: heix
minor version: 0
compatible brands: mif1,heix,miaf,MiHA,jpeg,SHIF

libheif has supported this ftyp since 2020 and imgproxy built locally with this patch can process the files without issue
https://github.com/strukturag/libheif/commit/3cb99d787d93f02eca0aec129b1373fec2e15a78#diff-1dede4959a1188fc7739739647e681aa8f8b3f5d68dcee80286f2b6cb90c28d7R108

Hey @jtomson!
You're right, the heix brand should work fine with imgproxy, I just thought that heic should be specified in the compatible brands anyway.
Let's change things a bit. I don't particularly appreciate converting byte slices to strings. I'd add an heixBrand var and use two bytes.Equal in the check.

Thanks for your contribution! 👍

Thanks for your contribution! 👍

cheers appreciate your time! 🤝


error[E0425]: cannot find function collect in module collector

Describe the bug
Build fails:
error[E0425]: cannot find function `collect` in module `collector`
   --> /usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13/src/lib.rs:221:32
    |
221 |         let mut m = collector::collect();
    |                                ^^^^^^^ not found in `collector`
    |
note: found an item that was configured out
   --> /usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13/src/collector.rs:36:16
    |
36  | pub use macos::collect;
    |                ^^^^^^^
    = note: the item is gated behind the `macos` feature
note: found an item that was configured out
   --> /usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13/src/collector.rs:39:16
    |
39  | pub use linux::collect;
    |                ^^^^^^^
    = note: the item is gated behind the `linux` feature
note: found an item that was configured out
   --> /usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13/src/collector.rs:42:14
    |
42  | pub use win::collect;
    |              ^^^^^^^
    = note: the item is gated behind the `windows` feature

For more information about this error, try `rustc --explain E0425`.
error: could not compile `metrics-process` (lib) due to previous error

Caused by:
  process didn't exit successfully: `CARGO=/usr/local/bin/cargo CARGO_CRATE_NAME=metrics_process CARGO_MANIFEST_DIR=/usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13 CARGO_PKG_AUTHORS='Alisue<EMAIL_ADDRESS>CARGO_PKG_DESCRIPTION='Cross-platform Prometheus style process metrics collector of metrics crate' CARGO_PKG_HOMEPAGE='' CARGO_PKG_LICENSE=MIT CARGO_PKG_LICENSE_FILE='' CARGO_PKG_NAME=metrics-process CARGO_PKG_README=README.md CARGO_PKG_REPOSITORY='https://github.com/lambdalisue/rs-metrics-process' CARGO_PKG_RUST_VERSION='' CARGO_PKG_VERSION=1.0.13 CARGO_PKG_VERSION_MAJOR=1 CARGO_PKG_VERSION_MINOR=0 CARGO_PKG_VERSION_PATCH=13 CARGO_PKG_VERSION_PRE='' LD_LIBRARY_PATH='/usr/ports/finance/reth/work/target/release/deps:/usr/local/lib' /usr/local/bin/rustc --crate-name metrics_process --edition=2018 /usr/ports/finance/reth/work/reth-0.1.0-alpha.13/cargo-crates/metrics-process-1.0.13/src/lib.rs --error-format=json --json=diagnostic-rendered-ansi,artifacts,future-incompat --diagnostic-width=189 --crate-type lib --emit=dep-info,metadata,link -C opt-level=2 -C panic=abort -C linker-plugin-lto -C codegen-units=1 -C metadata=d6f274dab7c39cad -C extra-filename=-d6f274dab7c39cad --out-dir /usr/ports/finance/reth/work/target/x86_64-unknown-freebsd/release/deps --target x86_64-unknown-freebsd -C linker=cc -L dependency=/usr/ports/finance/reth/work/target/x86_64-unknown-freebsd/release/deps -L dependency=/usr/ports/finance/reth/work/target/release/deps --extern metrics=/usr/ports/finance/reth/work/target/x86_64-unknown-freebsd/release/deps/libmetrics-15c44dd37dab6168.rmeta --cap-lints warn -C link-arg=-fstack-protector-strong` (exit status: 1)

version: 0.1.0-alpha.13
rust-1.74.1
FreeBSD 14.0
Steps to reproduce
regular build
Node logs
No response
Platform(s)
No response
What version/commit are you on?
No response
What database version are you on?
No response
What type of node are you running?
None
What prune config do you use, if any?
No response
If you've built Reth from source, provide the full command you used
No response
Code of Conduct

[X] I agree to follow the Code of Conduct

hmm, interesting
https://github.com/lambdalisue/rs-metrics-process/blob/dbb1d752e0b497d32853a6c7b3acd77d6f64157d/src/collector.rs#L35-L42
not sure if FreeBSD is mapped to linux?

not sure if FreeBSD is mapped to linux?

No. When I attempted to copy metrics-process-1.0.13/src/collector/linux.rs to metrics-process-1.0.13/src/collector/freebsd.rs - it can't find procfs.


Consistent use of "private" vs "internal" terms for GitHub Apps

GitHub Apps can be either "public" or "private". We previously used the term "internal" and "private" interchangeably, leading to confusion. The UI has been updated to remove all references to "internal".
Why:

Closes https://github.com/github/docs-content/issues/4306
What's being changed:

This branch replaces all references to "internal" Apps with "private".
Check off the following:

[ ] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[ ] For content changes, I have reviewed the localization checklist
[ ] For content changes, I have reviewed the Content style guide for GitHub Docs.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (eg. a "before and after video")

Thanks so much for opening a PR @jamesmartin! I'll triage this for review ⚡

@felicitymay thanks for the review! ✨

My only question is when did you update the UI and which versions of GHES the new UI text is correct for. If GHES 2.21+ use the old terminology, then we'll need to add some versioning for these changes.

The UI was updated 10 days ago in https://github.com/github/github/pull/178328. It would not have been backported to older versions of GHES, so presumably we'll need to preserve the old terminology in those cases. How do we get started on that? 🤔

I've had confirmation that this is correct for GHAE currently. So ready to merge.


Proposal: Create standard methodology in the event that an app cannot be reviewed by a reviewer

Currently, there are some apps that do not qualify to be tested in certain circumstances. E.g. they may need to use another app to be useful, or they may require a certain hardware that is not accessible to everyone. As time goes on and reviewers are added, this will become more and more of an issue.
Open question:

How can we create a method that is fair to the community, while not punishing the app founders when this happens?

Next steps:

Determine said process, add to FAQ.

The current method is that it gets a theta score of -1 (with the idea that -1 is not the worst).

Sounds like this is related to https://github.com/blockstack/app-mining/issues/35 and is already resolved, closing.

I would like to re-open this issue because with the dry run in April shows that it is possible that a half reviewed app can become first place.

I see that there is a discussion about this on #91


typeahead collection changes but it is not reflect to the list

Typeahead shows the results from the previous digest loop, and does not show the current ones:
(Can't use JSON_CALLBACK because of the apple service I am depending on,  so I am triggering typeahead through scope.$apply outside.)
Used within input tag with:
ng-change="search(searchStr)" ng-model="searchStr"
typeahead="suggestion for suggestion in typeAheadResults"

applying outside:
 scope.$apply(function() {
    scope.typeAheadResults = _.values(results);
  });

$scope.$watch shows that the scope is changing:
  $scope.$watch('typeAheadResults', function(neu, old){
    console.error("$watch typeAheadResults changes: " );
    console.error(neu);
  })

Check the console, where it writes the $watch results where it logs that collection changes, but it is not reflected into typeahead:
http://plnkr.co/edit/2MQuP2YUwaIZJjEVjdOz?p=preview

Please provide an updated plunkr and verify this is still an issue in Angular 1.3 with UI Bootstrap 0.13.

Closing due to lack of response & age - if this is still an issue, feel free to open another issue with reproduction based on a current version of UI Bootstrap.


tart: 1.6.0 -> 2.0.0

Description of changes
https://github.com/cirruslabs/tart/releases/tag/2.0.0
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Updated to target 2.0.0


fix: address Pagination prop type warning and resolve eslint errors in www

While upgrading Paragon, it was observed there's a prop type warning when using DataTable's pagination, where alt prop is not passed to the IconButton components. This PR ensures we pass alt prop to IconButton components in Pagination.
It also disables an eslint rule when using the SEO component due to the component name.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


AX-351 update moment

@APshenkin Hey, out of curiosity, any reason why this was closed? Moment > 2.29.2 has some important security patches.

@samuxyz we depend on moment using ^, so there's no reason to update react-dates - just install the latest moment.


Unable to Start DLedger without config.yaml

After supporting multi-raft mode (#150 ), DLedger should be able to start with the DLedgerConfig under the normal mode and DLedgerProxyConfig under the multi-raft mode.
However, DLedger can only boot after providing a config.yaml and --config in the command line at the moment, which means that users are forced to use multi-raft mode to start it. It supposed to support both modes (original one & multi-raft) after this upgrade.

I fix that problem yesterday, please have a try in now master branch~


Alert Generator Compliance Specification 1.0

This PR adds a specification for alert-generator compliance which was open for public review at https://docs.google.com/document/d/1QyGA3c0Eys9rZRMEbSSXcd1C0yuph8wCQ6B3Dc90rk0/edit

@brian-brazil I should have addressed your comments. I have not resolved the comments where I have open questions.


Virtual City looks like a white dome

I tried displaying VC.gltf model with Minimal glTF loader.
However, Virtual City looks like a white dome.
minimal-gltf-loader + VC.gltf result

Three.js +VC.gltf result

Hi @cx20 ! Thanks for your constant support and patience for testing!
I fixed the issue with the latest HEAD at master. Basically by enabling back face culling.

@shrekshao Thank you for fixing. It seems that the problem of culling has been solved.
It may be another problem, but it seems that the floor etc is displayed in white depending on the angle.
minimal-gltf-loader + VC.gltf result

Is the usage of the library incorrect?

@cx20
I used to think you set the shaded area to white on purpose. Now you help me realize it's a bug.
Basically it's blended with the canvas background color. It's because I interpolate the alpha channel in the shader. Get it fixed, great thanks! I wouldn't spot this without your work. This could lead to weird behavior in the future.
You just need to update the shaders for this one (index.html in your gltf-test)

@shrekshao  Thank you for helping! The mystery was solved and it was refreshing.
minimal-gltf-loader + VC.gltf result

Issue is closed as the problem has been resolved.


Add Void Linux to distrobutions with packages

Void Linux has the alacritty package in the main repository's, so compilation from source isn't needed.

Cool, thanks!


Remove show full site address setting pixels

Task/Issue URL: https://app.asana.com/0/1205278999335242/1206864291180522/f
Tech Design URL:
CC:
Description:
Planned pixels removal for Show Full Site address setting toggle.

Steps to test this PR:

Make sure app builds and runs without issues

Internal references:
Software Engineering Expectations
Technical Design Template

Will be done later. See comment for details.