New Option to stretch player to full browser width

I have a player set up to span whole browser width. Although player does a good job of stretching it self when I stretch my browser window (fast enough, bit exaggerated!!), but it lags often and thus does not fill the width completely. This problem is also exhibits on mobile on orientation change. In fact parseFloat(parentStyles.width) is not always equals to the window width.
PS: As discussed here https://github.com/mediaelement/mediaelement/pull/2355
Test url http://tinyurl.com/y8mnxgse

Coverage remained the same at 73.832% when pulling e5c1be6538abf84a60021ae51890423828c8c0d0 on tsarma:patch-2 into 0452c27aded564b148b7c3bedf8d13eb956beef7 on mediaelement:master.

Coverage remained the same at 73.832% when pulling 19d47d5c632fa9bbdf0bfcc149b5110d23ff5e9a on tsarma:patch-2 into 0452c27aded564b148b7c3bedf8d13eb956beef7 on mediaelement:master.

Unfortunately I can't merge this because it depends on jQuery for this and the player is framework-agnostic. My proposed solution is to use this:
parentStyles = parent ? getComputedStyle(parent, null) : null;

// Later on the code
parentWidth = parent ? parseFloat(parentStyles.width) : (window.innerWidth || document.documentElement.clientWidth || document.body.clientWidth);

If you can test this and confirm it works I can set up the code in the master branch to avoid this situation

I'll let you know next week as I am not in office till then.  I can not test it before. But by looking at your proposal, I think it will not work because you'll always get the value from parseFloat(parentStyles.width). As I have mentioned parseFloat(parentStyles.width) does not always equals to browser window width, thus we do not get the desired result (Full Width Stretching).

I have checked the current code on master, and using the following markup I don't see any problems when changing orientation or resizing:
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <title>MediaElement.js 3.0 - audio/video unification library</title>
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <link rel="shortcut icon" href="favicon.ico" type="image/x-icon">
    <link rel="icon" href="favicon.ico" type="image/x-icon">

    <link rel="stylesheet" href="https://cdnjs.cloudflare.com/ajax/libs/normalize/5.0.0/normalize.min.css">
    <link rel="stylesheet" href="https://rawgit.com/mediaelement/mediaelement/master/build/mediaelement-and-player.min.js">

            <video preload="none" controls id="player1">
                <source type="video/x-youtube" src="https://www.youtube.com/watch?v=Y_H2i_m5IE0" title="">
            </video>

Let me know if you experience an issue with it

Hmm, I have copied your code and set up a test page here http://bit.ly/2wfy8zw , Since the video is no visible I can not confirm your solution is working. I have checked on chrome desktop and on iPhone, both showing black background while playing the video.

My bad. Change this <link rel="stylesheet" href="https://rawgit.com/mediaelement/mediaelement/master/build/mediaelement-and-player.min.js"> to <link rel="stylesheet" href="https://rawgit.com/mediaelement/mediaelement/master/build/mediaelementplayer.min.css">

Yes it seems working very well. Now I am puzzled! May be I it was because I wrapped the player inside slick slider as here http://bit.ly/2v1KrLE .

It's likely because of that; please note that I always use normalize.css to avoid weird behaviors with CSS and that way have a unified view cross browsers/cross devices. If you are OK with it, we can close this issue

Ok thank you, please close this issue.

Thank you for checking this

Just FYI, I'm not maintaining the project anymore. Just wrapping things for Wordpress at this point, but I'm glad I could help you

Yeah I have read about that (sadly) in other post. I can understand how much time you have donated and it wasn't easy at all to maintain this big project. Any way let me thank you once again and good luck in other projects you are working on.

Thanks @tsarma I wish you the best as well


Capitalization sometimes changes result too much

For example when I search "analyzer", it comes first as expected:

But, when I search "Analyzer", it isn't first and is actually on the second page(possibly due to its low score?):

This happens with other capitalizations as well:

I feel like capitalization shouldn't affect results for exact matches, at least not this much, especially since a lot of mobile keyboards automatically capitalize your first character now.

For most part of search stack we are case-insensitive, but apparently for the package-exact match strings we are not. Should be easy to fix.


devcontainer cabal update && cabal install fails with error code 127

Failed to build prettyprinter-configurable-<IP_ADDRESS> (which is required by
exe:plutus-starter-pab from plutus-starter-<IP_ADDRESS>). The failure occurred
during the configure step. The build process terminated with exit code 127

inputoutput/plutus-starter-devcontainer:latest
using the latest plutus-starter
docker mem = 30 GB

It's hard to tell anything without more logs unfortunately. Maybe it misses cabal update at first or some library is not installed.

The modern version of Haskell VSCode plugin requires ghcup to be installed.
However, it is not installed, and the installation fails (I use the --insecure option just to skip setting up certificates):
[devcontainer]$ curl --insecure --proto '=https' --tlsv1.2 -sSf https://get-ghcup.haskell.org | sh

Welcome to Haskell!

This script will download and install the following binaries:
  * ghcup - The Haskell toolchain installer
  * ghc   - The Glasgow Haskell Compiler
  * cabal - The Cabal build tool for managing Haskell software
  * stack - (optional) A cross-platform program for developing Haskell projects
  * hls   - (optional) A language server for developers to integrate with their editor/IDE

ghcup installs only into the following directory,
which can be removed anytime:
  /home/plutus/.ghcup

Press ENTER to proceed or ctrl-c to abort.
Note that this script can be re-run at any given time.

-------------------------------------------------------------------------------

Detected bash shell on your system...
Do you want ghcup to automatically add the required PATH variable to "/home/plutus/.bashrc"?

[P] Yes, prepend  [A] Yes, append  [N] No  [?] Help (default is "P").

a
-------------------------------------------------------------------------------
Do you want to install haskell-language-server (HLS)?
HLS is a language-server that provides IDE-like functionality
and can integrate with different editors, such as Vim, Emacs, VS Code, Atom, ...
Also see https://github.com/haskell/haskell-language-server/blob/master/README.md

[Y] Yes  [N] No  [?] Help (default is "N").

y
-------------------------------------------------------------------------------
Do you want to install stack?
Stack is a haskell build tool similar to cabal that is used by some projects.
Also see https://docs.haskellstack.org/

[Y] Yes  [N] No  [?] Help (default is "N").

y
main: line 238: getconf: command not found
main: line 240: getconf: command not found
main: line 243: getconf: command not found
Unknown long bit size:
[devcontainer]$


Testing GitHub Actions

to test https://github.com/containerd/go-cni/pull/63

Hmm.. interesting;
go: downloading gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c
flag provided but not defined: -test.root
Usage of /tmp/go-build3941133643/b001/go-cni.test:
  -test.bench regexp
    	run only benchmarks matching regexp
  -test.benchmem
    	print memory allocations for benchmarks

ok, looks like a lot of undocumented things;
Error: exported type `CNI` should have comment or be unexported (golint)
Error: exported type `ConfigResult` should have comment or be unexported (golint)
Error: exported type `ConfNetwork` should have comment or be unexported (golint)
Error: exported var `ErrCNINotInitialized` should have comment or be unexported (golint)
Error: exported type `Network` should have comment or be unexported (golint)
Error: exported method `Network.Attach` should have comment or be unexported (golint)
Error: exported method `Network.Remove` should have comment or be unexported (golint)
....

closing, as https://github.com/containerd/go-cni/pull/63 was merged, so upstream is now running GHA


new_benchmark

Benchmark submissions
Added SchNet for benchmark matbench_mp_e_form. This is a test submission. If the format is correct and the benchmark valid, we would like to add further compatible tasks for SchNet in this folder.
Note that this uses a modified implementation of SchNet in kgcnn. We are not the original authors of this algorithm but are currently maintaining kgcnn. Could you check if the info.json is properly giving the original authors credit? We are unsure about the authors category. The naming "SchNet (kgcnn)" should indicate the modified version of SchNet.
Also answers to issue  #61

Extended to all structure tasks.

Well, first off, this is an excellent submission in regards to submitting the hyperparameters and all the needed data. Bravo!!
As far as the info.json goes, it looks good to me. The one thing I think you might want to add is the version number or some identifier in the submission name of a particular version or hyperparamer set used. Just like some one or two word identifier in the submission name so that if another (let's say improved or modified) Schnet benchmark is submitted, we can differentiate between the versions.
Let me know what you decide for that, and I'll merge in whenever you're ready

@PatReis

Ah sorry, I see, I could have added the kgcnn version. I will do it next time thanks :)


CVE-2020-16156

https://blog.hackeriet.no/cpan-signature-verification-vulnerabilities/

[ ] [CVE-2020-16154] App::cpanminus 1.7044 allows Signature Verification Bypass
[ ] [CVE-2020-16155] CPAN::Checksums 2.12 does not uniquely define signed data.
[ ] [CVE-2020-16156] CPAN 2.28 allows Signature Verification Bypass

Finished in c57649452e4574c6ab381816b8778b3f25823c97


cChocoPackageInstaller should pass on Source parameter

choco install features the alternative sources windowsfeatures and webpi.
cChocoPackageInstaller { Ensure = 'Present'; Source = 'windowsfeatures' } works perfectly but cChocoPackageInstaller { Ensure = 'Absent'; Source = 'windowsfeatures' } does not, because the UninstallPackage function doesn't pass on the source parameter.

If you are using DSC you can use other resources to manage Windows Features. But I agree that we may want to start pushing that through.


Compatibility Issues with PS V 3

Good evening,
Thank you for creating this module. I have been trying to use it with powershell version 3, but there are a lot of compatibility issues. The main function that I am trying to use is converting an excel sheet from .xlsx to .csv. The server instance that I am running it on does not have excel or access installed, which is the reason for me attempting to use the module. What steps could I take in terms of modifying certain ps1 files to have backwards compatibility with powershell version 3?
I was able to run the ConvertFrom-ExcelSheet on my laptop, which worked fine, as I have the newest version of powershell on it. That is the main function that I need, along with Import-Excel, which is called by ConvertFrom-ExcelSheet, to have backwards compatibility with powershell version 3. If such compatibility is not possible, please let me know. Thanks in advance for your help.
Thanks,
David Earnest

Please post the errors you are getting. From there I can see the level of difficulty of making this work with version 3

The main error that I am running into is with this line at the end of the function ConvertFrom-ExcelSheet: Import-Excel $Path -Sheet $($sheet.Name) | Export-Csv @params
The error message is as follows: Export-Csv : Cannot bind argument to parameter 'InputObject' because it is null.

    Import-Excel $Path -Sheet $($sheet.Name) | Export-Csv <<<<  $params

CategoryInfo          : InvalidData: (:) [Export-Csv], ParameterBindingValidationException
FullyQualifiedErrorId : ParameterArgumentValidationErrorNullNotAllowed,Microsoft.PowerShell.Commands.ExportCsvCommand

Thanks in advance for your help.

Can you try two things:
First
Does this throw an error?
function test {
    param($a,$b)

    "`$a: $a"
    "`$b: $b"
}

$params = @{a=1;b=2}
test @params

Second
Can you try to do an Import-Excel on the xlsx and worksheet directly and see if that has a problem.
Thanks
Doug

The function test does not throw an error.
I tried Import-Excel .\temp.xlsx but I do not see anything.
From the example, I should be able to see the data right? I can open the same xlsx from powershell by calling .\temp.xlsx, but if I try to call Import-Excel on it I do not see anything. Perhaps this is an error in compatibility with powershell version 3?
Thanks in advance for your help.

Good, so that rules out splatting as being a problem.
I'm guessing the temp.xlsx has only 1 worksheet and has the name sheet1?
Did you get the module from the PowerShell Gallery or here on GitHub?

Yes that is correct. I got the module from here on GitHub. I used the statement iex (new-object System.Net.WebClient).DownloadString('https://raw.github.com/dfinke/ImportExcel/master/Install.ps1') to download the module.

Hmm. I'm out of ideas.
The answer I have, which won't work, is to upgrade to PowerShell.
 We'll be sure to post it.


Request for Manuall Update PARTS token logo #20512

Hello @keone ,
PR link => https://github.com/solana-labs/token-list/pull/20512
PR => #20512
I am the developer of PARTS token.
I want to change the logo to new one. and logo link update.
Here is PR request #20512
Please Kindly consider this request.
Thank you.

merged


Enable assertions

Codecov Report

Merging #98 (d07bcbe) into master (431d3ed) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #98   +/-   ##
=========================================
  Coverage     69.92%   69.92%
  Complexity      562      562
=========================================
  Files            93       93
  Lines          1895     1895
  Branches        225      225
=========================================
  Hits           1325     1325
  Misses          483      483
  Partials         87       87

Continue to review full report at Codecov.

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

LGTM!


Java Client needs connectors

Connectors join the client host to a downstream device.

[x] Connector framework
[x] Serial connector (TTY - rxtx?)
[x] TCP connector
[ ] UDP connector

Made complex due to the switch to the new client lib model.


Create privacy policy webpage

A privacy policy web page has to made for the gitsetgo project, and has to be linked to the footer via link.
Discussions welcome, please contact @garimasingh128  for the content. #32

I would like to work on this.

Can I work on this?

@garimasingh128 can I work on this issue?


add a Changelog entry for HTTP/3

Codecov Report

Merging #1861 into master will decrease coverage by 0.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1861      +/-   ##
==========================================
- Coverage    86.3%   85.56%   -0.75%
==========================================
  Files          98      106       +8
  Lines        6666     7298     +632
==========================================
+ Hits         5753     6244     +491
- Misses        653      763     +110
- Partials      260      291      +31

Impacted Files
Coverage Δ

session.go
75.96% <0%> (-0.27%)
:arrow_down:

h2quic/client.go
87.58% <0%> (ø)

h2quic/response_body.go
100% <0%> (ø)

h2quic/request_body.go
100% <0%> (ø)

h2quic/request.go
91.11% <0%> (ø)

h2quic/request_writer.go
68.09% <0%> (ø)

h2quic/response_writer.go
83.67% <0%> (ø)

h2quic/server.go
64.29% <0%> (ø)

h2quic/roundtrip.go
96.83% <0%> (ø)

Continue to review full report at Codecov.

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


Hints

Using world building commands should show hints about possible next actions when the commands complete. These hints are enabled by default, and can be disabled with a setting.

This was resolved instead by just adding more hints to the command help strings for world building commands.


Add auto changing quote

Description
Add famous quotes of Steve Jobs on the space between month and footer. And the quote is changing after 10 sec.

I'd like to add this feature , stated working on it.

I'd like to add this feature , started working on it.

Okk 👍

I managed to add the feature. But found some glitches. Looks like there are
some screen size issues to be fixed in code
On Oct 1, 2020 12:40 AM, "Anubhav Singh"<EMAIL_ADDRESS>wrote:

I'd like to add this feature , started working on it.
Okk 👍
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Catalyst-IN/OpenSourceEvents-Frontend/issues/144#issuecomment-701587758,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKVGS6WRBHEGU7KXRNRRFHDSIN7DPANCNFSM4QCFYWMA
.

Hey @anubhavsingh16  ,
I managed to add text transition. But looks like there are some screen issues with screen size specifications.

Ok so add issue in issue list and send PR for same.
On Mon, 12 Oct, 2020, 11:10 PM chanchalvp<EMAIL_ADDRESS>wrote:

Hey @anubhavsingh16 https://github.com/anubhavsingh16 ,
I managed to add text transition. But looks like there are some screen
issues with screen size specifications.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Catalyst-IN/OpenSourceEvents-Frontend/issues/144#issuecomment-707256789,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALMYYFWW5MFVXAOAMXETHVDSKM5P5ANCNFSM4QCFYWMA
.


New serverless snippet - lambda-kafka-glue-schema-registry

Issue #, if available:
Description of changes: This code example shows how an AWS Lambda function can get invoked by Event Source Mapping to a Kafka topic. It downloads the corresponding schema from AWS Glue Schema Registry by looking up the schema version (by which the record has been serialized). The deserializes record is stored in JSON format in Amazon S3.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @awsalialem , will get it merged


Introduce some "Power" class for training (e.g. <500W)

The way the benchmark is set up today, the "leaders" will be dominated by the "Training on 1000 nodes in 15 min" approaches, which may not be useful data for industry. It can also mask algorithmic innovations in favor of pure scaling. Proposed approach here is to have some "single-card" power class for training. This isnt the perfect solution, we are open to suggestions.
Supporting orgs: Intel

No power measurement in 0.5, so postponed to 1.0

Obsolete based on practice.


Cobol Build - DD SYSXMLSD being 45 Characters long exceeding the 44 Characters allowed.

Good day!
We are in the need of assistance related to a problem in a DBB Build for a Cobol compile.
It seems the problem would be related to this DD SYSXMLSD being 45 Characters long exceeding the 44 Characters allowed.
45 Characters
BGZTK0016E An error occurred running BPXWDYN command ‘alloc dd(SYSXMLSD) dsn(PRI.PRIJMO.TESTDBB.LOAD.U8964768.SYSXMLSD.XML) tracks space(200,40) dsorg(PS) blksize(27998) lrecl(16383) recfm(V,B) new delete’.
It seems to be resolved as follow:
${props.hlq}.  ${props.errPrefix}.            SYSXMLSD.XML”)
PRI.PRIJMO     TESTDBB.LOAD.U8964768          SYSXMLSD.XML
We cannot remove the qualifier  “PRI“. USERID” therefore we need to reduce one letter in one of the other two variables so it does not exceed the 44 Characters. But we are not sure if doing this would affect other scripts that would reference this DD. Would someone please help us determine the best option to resolve this problem?
We received the recomendation to remove one letter to the Project Name on IDz Wizard, "TESTDBB"  but customer wants to know if there is any other solution other than restricting Project Name to only 6 letters.
/global/zDevOps/zAppBuild/languages/Cobol.groovy
..// SYSXMLSD.XML suffix is mandatory for IDZ/ZOD to populate remote error list
..compile.dd(new DDStatement().name(“SYSXMLSD”).dsn(“${props.hlq}.${props.errPrefix}.SYSXMLSD.XML”).options(props.cobol_compileErrorFeedbackXmlOptions))
.}
.def errPrefixOptions = props.getFileProperty(‘cobol_compileErrorPrefixParms’, b
if (props.errPrefix)
.parms = “$parms,$errPrefixOptions”
Many thanks in advance!
Best Regards

Did you resolve the reported issue?

Denis good afternoon, it was resolved by changing the Projects name in IDz
to make it shorter, as the full Qualifier names of the Project in IDz was
taken in consideration to build this DDNAME.
Thanks for all your help!
Best Regards
On Thu, Jan 13, 2022 at 5:28 PM Dennis Behm @.***>
wrote:

Closed #139 https://github.com/IBM/dbb-zappbuild/issues/139.
—
Reply to this email directly, view it on GitHub
https://github.com/IBM/dbb-zappbuild/issues/139#event-5890246311, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEE2RAOAM7YDZUCWOZF4U3UV4YVFANCNFSM5CZGT5NQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Error running frodo on older Linux versions

Describe the issue
On CentOS Linux release 7.9.2009 (Core)
$ frodo
/usr/bin/env: invalid option -- 'S'
Try '/usr/bin/env --help' for more information.
$

Will not fix. Older versions of coreutils do not support the '-S' flag with the env command. Expect to run a recent enough version of coreutils for frodo to work.

Adding support for node 19 allows the removal of the -S option.


Sending requirements.txt

In this pull request i did:

Create and update the requirements.txt

This was referring to the requirements.txt file that pip uses for installing packages not in python's standard library, here is a reference to what the issue is referring to: link.

Thank you for submitting a pull request, but issue #4 was resolved in PR #9


Contribution

Can you post information about contribution to the repo? I love what you're doing and would like to see if I can use your work as a baseline for similar support for NX Nastran.

hey @sharkweek , thanks for the interest in contributing! I'll pull together a CONTRIBUTING..md and a CODE_OF_CONDUCT.md in the next day or so. If you're eager to get started, go ahead and fork the project and get started.

Not to add to your workoad, but any chance a Discord channel could be part of it? =)

Give this a try. It's pretty empty for now.

Closed with #20


Digest cycle constantly triggered

I was gonna implement a directive to watch the scroll position and decided to test if this directive would have negative effects on the performances.
At my surprise, I found out that videogular is constantly watching and triggering the digest cycle.
Here is a Codepen to show you the problem: http://codepen.io/PierrickGT/pen/qbJawX
Is there a workaround?

The digest cycle must be triggered by this setInterval.
https://github.com/NamPNQ/bower-videogular-youtube/blob/master/youtube.js#L111
You can try to clean the interval if player is paused, but you still need the interval to update current time value inside Videogular's API.


Adding extra code bit by bit

I added a missing mypy.ini configuration file that was causing mypy to fail on CI. code coverage is going to be low until we return and write unit tests for these modules, so i'll disable code coverage checking for now 👍


Output size is 128 bits in CTR mode

I believe in CTR mode, the output size should be equal to the input size. When I give a 16 byte key and a 32 byte input plain text, the output I receive is 64 bytes.

Are you padding using pkcs#7?

Actually, n/m. That would only result in 48 bytes. Can you provide an example?

I made a mistake.
I used the aesjs.utils.utf8.toBytes() instead of the aesjs.utils.hex.toBytes() and thus misjudged my input size.
I have fixed it now.


Get pinned chats first

Did you notice any performance differences when doing this?

Nothing noticeable for me because I don't have a ton of chats but I wouldn't be surprised if it is slightly slower for those who have a ton of chats. Don't know how else we could accomplish loading pinned chats first though unless we store chats themselves differently or something (like maybe order the table by pinned chats when making a chat pinned)?


Custom fonts!

Probably wouldn't be too terribly hard to accomplish this, but it would be nice to have custom fonts available in the main menu for text.

I'd love to see a reply on this, Since no where in any documentation does it say how (if it's even possible, or if it could be possible) I'm sure there's a ton of people who've wanted to know the answer to this.

You can load in your own font via resource pack. Download a resource pack to see how they do it. If you want to load the font strait in with no resource pack. You can use resource loader. Just put the ascii.png file at oresources/minecraft/textures/font/ascii.png


Error thrown from parsing a partial application of a method application chain

As in, the following throws on both the latest NPM version and the website:
(a.b _ .c)
(a.b _ .c!)
(a.b _ .c d)

Here's the stack trace for each from lsc (same output):
TypeError: Cannot read property 'length' of undefined
    at Chain.exports.Chain.prototype.add (/home/isiah/.npm/.global/lib/node_modules/livescript/lib/ast.js:950:110)
    at Object.anonymous (/home/isiah/.npm/.global/lib/node_modules/livescript/lib/parser.js:96:41)
    at Object.parse (/home/isiah/.npm/.global/lib/node_modules/livescript/lib/parser.js:840:36)
    at Object.exports.compile (/home/isiah/.npm/.global/lib/node_modules/livescript/lib/index.js:45:20)
    at compileScript (/home/isiah/.npm/.global/lib/node_modules/livescript/lib/command.js:197:31)
    at /home/isiah/.npm/.global/lib/node_modules/livescript/lib/command.js:97:9
    at Object.<anonymous> (/home/isiah/.npm/.global/lib/node_modules/livescript/bin/lsc:7:26)
    at Module._compile (module.js:430:26)
    at Object.Module._extensions..js (module.js:448:10)
    at Module.load (module.js:355:32)

By the looks of it, there's no parsing support for this.

@vendethiel Any ideas? Still failing.

So, the crash is easily fixed—all it takes is changing one . to a ?. Unfortunately, while this now lets those expressions compile, they don't have the semantics that I presume you want—the docs make no mention of the meaningfulness of parentheses when compiling partial applications, and so the compiler dutifully treats (a.b _ .c) as equivalent to (a.b _).c, the same way that (a.b x .c) would be equivalent to (a.b x).c. I see from your tests that you were expecting something closer to the way Scala handles underscore placeholders, where the nearest bounding parentheses help determine the extent of the anonymous function created.
If that's indeed what you're proposing, you should probably flesh out the feature request a little more. One potential niggle is what gets captured at the time of partial function creation; right now, everything does:
a = b: (x, y) -> x + 2*y
c = 10
f = a.b _, c
a.b = (x, y) -> 2*x + y
c = 1000
f 1 #=> 21

Contrast with the similar but non-capturing version:
a = b: (x, y) -> x + 2*y
c = 10
f = -> a.b it, c
a.b = (x, y) -> 2*x + y
c = 1000
f 1 #=> 1002

In an expression like (a.b _ .c d .e f), in your proposal, does a.b still get captured? Do d and f? Or do you abandon the capturing semantics to make (any expression including _ as an argument somewhere but not inside any deeper parentheses) semantically equivalent to (-> that same expression with _ replaced by `it`)?

First, I thought I'd clarify with my current stance. In my opinion, it
would be best rejected, since it's not just a simple method call, function
call, or property access. Otherwise, it's nearly impossible to infer
reliably. By that point, you probably should already be using a normal
function, since even if it's trivial to the programmer, it's no longer
trivial to the compiler. To my knowledge, even Haskell's GHC would reject
that kind of thing.
The reason I didn't was because I was uncertain at the time, and hadn't yet
put much thought into it. I hadn't really thought about which was the
better route.
I'll have to convert the tests, but I'm not sure exactly what the message
should be in the rejection.
On Fri, Feb 19, 2016, 00:43 Ryan Hendrickson<EMAIL_ADDRESS>wrote:

So, the crash is easily fixed—all it takes is changing one . to a ?.
Unfortunately, while this now lets those expressions compile, they don't
have the semantics that I presume you want—the docs make no mention of the
meaningfulness of parentheses when compiling partial applications, and so
the compiler dutifully treats (a.b _ .c) as equivalent to (a.b _).c, the
same way that (a.b x .c) would be equivalent to (a.b x).c. I see from
your tests that you were expecting something closer to the way Scala
handles underscore placeholders, where the nearest bounding parentheses
help determine the extent of the anonymous function created.
If that's indeed what you're proposing, you should probably flesh out the
feature request a little more. One potential niggle is what gets captured
at the time of partial function creation; right now, everything does:
a = b: (x, y) -> x + 2yc = 10f = a.b _, ca.b = (x, y) -> 2x + yc = 1000
f 1 #=> 21
Contrast with the similar but non-capturing version:
a = b: (x, y) -> x + 2yc = 10f = -> a.b it, ca.b = (x, y) -> 2x + yc = 1000
f 1 #=> 1002
In an expression like (a.b _ .c d .e f), in your proposal, does a.b still
get captured? Do d and f? Or do you abandon the capturing semantics to
make (any expression including _ as an argument somewhere but not inside
any deeper parentheses) semantically equivalent to (-> that same
expression with _ replaced by it)?
—
Reply to this email directly or view it on GitHub
https://github.com/gkz/LiveScript/issues/746#issuecomment-186065533.


2022-01-11 meeting notes

Should we mention the discussion about ambient loaders?

Should we mention the discussion about ambient loaders?

Sure, the notes are supposed to be more or less a transcript of everything that was discussed, not just notes of decisions/outcomes. So whatever else you can remember to add would be good to include.


Server wont start due to this error with traincars

BkCommonLib version: Latest
TrainCarts version: Latest
Spigot version: 1.8.8

Problem or bug: Server wont start due to this error with traincars
Expected behaviour: won't load?
https://pastebin.com/WHx1EyFR
Steps to reproduce: start up server and wait for traincarts

0:00:43 AM [SEVERE] Ambiguous plugin name BKCommonLib' for files plugins\BKCommonLib-1.12-v4.jar' and plugins\BKCommonLib (1).jar' in plugins'
You need to make sure only one of these is in your plugins folder. It might be loading a very old BKCommonLib plugin, or its corrupting the original installation.


Preference for escape key behavior

When I press "escape", Messenger minimizes into my tray. It's really annoying and also happens if Messenger is the active app while pressing escape during alt+tab or win+tab. Whatsie doesn't behave this way, so I find it very odd that Messenger does.

Operating system: Windows 10
Messenger for Desktop version: Latest version

There's now a preference for this: Window > Close with escape key. (starting from v2)


Can no longer write map values created with new in 0.8.2 (breaking change)

Previously, it was possible to write map values created with new, example:
		function Bar() {
			this.value = 42
		}
		return this.db
			.collection("test")
			.doc("test")
			.set({
				foo: new Bar(),
			})

In 0.8.2 this now fails with:
Error: Cannot encode type ([object Object]) to a Firestore Value
Possibly caused by this change:
https://github.com/googleapis/nodejs-firestore/commit/1d5c2cc0c0d6f0d2d0527a2fa1f53779789d4874#diff-10a61011168244555609992da5f53f1cR1175

This is somewhat intended as we don't only want to support plain objects. I will look into this and see if we can allow this specific use case (and allow for simple objects).

We discussed this internally and would like to not revert back to the previous behavior. Our guarantee is that we can deserialize any data that we accept in out serialization. While it is certainly possible for us to serialize custom JS classes, we don't have a way to deserialize back into their specific types. Any data that we would read back from Firestore would therefore not pass theinstanceof Bar check.
With that said, I'm going to close this issue for now. It is certainly possible that we revisit this decision in the future.


Azure Speech-To-Text service region list miss 3 regions in doc

Hello Speech documentation team,
The speech-to-text service region list doc does not contain below three regions, which users can create service on like below snapshot:

China East2
China North2
China North.

Would you please fix it ? It blocks onboarding customers from these regions.
Thank you.

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

ID: 53d1ff6d-c13e-6e68-a2ba-c8d859d9e573
Version Independent ID: 0290c57c-6a64-b696-d774-2e5e44a613cf
Content: Regions - Speech service - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/regions.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@wukun2015 Thanks for spotting this. I believe this is covered under Azure docs china documentation but chinanorth still seems to be missing.
@eric-urban Do we need to add the above regions as supported in Azure Global  docs?

Hello @RohitMungi-MSFT ,
i'd suggest fix global doc because it is on one of doc journey paths directing customers of those regions. E.g.

Stop1: https://www.azure.cn/ to search speech product in China.
Stop2-A: https://azure.microsoft.com/en-us/services/cognitive-services/speech-services/, this is directed by cognitive service product link in home page of Stop 1.
Stop2-B: https://docs.microsoft.com/zh-cn/azure/cognitive-services/speech-service/regions#speech-to-text-text-to-speech-and-translation or https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/regions#speech-to-text-text-to-speech-and-translation

What's more, such regions shall be visible for global customer as well.
Thank you.

The guidance provided was to keep sovereign clouds in their own page, but will look at making it more discoverable.

why do we closed it w/o doc update ?  @eric-urban


Add checkmark icon in language/region to the current language/region when user opens modal

For better UI/UX

completed


Add ModernAVPlayer

Project URL
https://github.com/noreasonprojects/ModernAVPlayer
Category
Audio
Description
Persistence player to resume playback after bad network connection even in background mode, manage headphone interactions, system interruptions, now playing informations and remote commands.
Why it should be included to awesome-ios (mandatory)
ModernAVPlayer is the simplest way to playback a media from an iOS app.
Everything is managed automatically including system interruption and bad connection..
The library is based on state machine design pattern, so easy to maintain and add features and
I would love people to contribute.
Checklist

[x] Only one project/change is in this pull request
[x] Has unit tests, integration tests or UI tests
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 14 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

206
403
https://www.udemy.com/arkit-beginner-to-professional/?couponCode=CREATORS

407
301
https://github.com/cagricolak/CoreCharts redirects tohttps://github.com/CoreCharts/CoreCharts

547
301
https://developer.couchbase.com/mobile/ redirects tohttps://www.couchbase.com/products/mobile/

930
301
https://github.com/schibsted/layout redirects tohttps://github.com/nicklockwood/layout

1003
301
https://github.com/caffe2/caffe2 redirects tohttps://github.com/facebookarchive/caffe2

1175
301
https://github.com/bakkenbaeck/Viewer redirects tohttps://github.com/3lvis/Viewer

1568
301
https://github.com/quickbirdstudios/RxCoordinator redirects tohttps://github.com/quickbirdstudios/XCoordinator

1683
301
https://github.com/VirgilSecurity/sdk-x redirects tohttps://github.com/VirgilSecurity/virgil-sdk-x

1706
301
https://github.com/VirgilSecurity/crypto-x redirects tohttps://github.com/VirgilSecurity/virgil-crypto-x

2450
301
https://github.com/hyperoslo/Dropdowns redirects tohttps://github.com/onmyway133/Dropdowns

2584
301
https://github.com/vsmithers1087/VSVerticalSlider redirects tohttps://github.com/vsmithers1087/VerticalSlider

2706
301
https://github.com/bakkenbaeck/SectionScrubber redirects tohttps://github.com/3lvis/SectionScrubber

3097
301
https://developer.apple.com/app-store/review/rejections/ redirects tohttps://developer.apple.com/app-store/review/#common-app-rejections

3152
301
https://github.com/hyperoslo/SwiftSnippets redirects tohttps://github.com/onmyway133/SwiftSnippets

  Generated by :no_entry_sign: Danger

@raphrel Thanks for contributing!


Allow to publish to Maven Central in publish command

Another batch of commits from https://github.com/VirtusLab/scala-cli/pull/926.

IMO we need some unit tests to avoid regression in the future.

I wish too (unit or integration tests). But it seems we'd have to "mock" a Sonatype server instance, or use a docker-based one (which needs to be configured and all), and try make it look like the official Sonatype instances.


msconvert yields empty files on windows and linux

On Ubuntu 20.04 running on WSL2:
export LC_ALL=C
msconvert --mgf  1\ V443\ +22,5\ pos_080419_Recal.mzML

The input mzML and output mgf from msconvert are attached:
1 V443 +22,5 pos_080419_Recal.zip
I also ran this command on Windows with the command line version of msconvert, and with the Windows GUI for ProteoWizard, and it yielded the same results.
What am I doing wrong? Thanks so much!

Your mzML file only contains MS1 spectra.
The MGF file format only supports MS2 spectra.
When an MGF file has no spectra in it, the MGF file will be zero bytes long.
-- Nick
On Mon, Oct 18, 2021 at 9:13 PM Jaan Altosaar @.***>
wrote:

On Ubuntu 20.04 running on WSL2:
export LC_ALL=C
/mnt/c/Users/jaan/Downloads/pwiz/msconvert --mgf  1\ V443\ +22,5\ pos_080419_Recal.mzML
The input mzML and output mgf from msconvert are attached:
1 V443 +22,5 pos_080419_Recal.zip
https://github.com/ProteoWizard/pwiz/files/7370087/1.V443.%2B22.5.pos_080419_Recal.zip
I also ran this command on Windows with the command line version of
msconvert, and with the Windows GUI for ProteoWizard, and it yielded the
same results.
What am I doing wrong? Thanks so much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ProteoWizard/pwiz/issues/1804, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKAY66XJOOREHSVJUBWFTUHTV6NANCNFSM5GIER4NA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


feat(kafka): option of setting max number of messages

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
As per the title, it will be used for Kafka performance testing setting as discussed with @huangjw806 .
This is only used internally, so it will not go into the Doc.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

dev=> create source kafkas(v1 bigint) with (                                                          connector = 'kafka',                                                                                kafka.brokers = '<IP_ADDRESS>:29092',                                                                  kafka.topic = 'sink_target',                                                                      --  kafka.bytes.per.second = '20000'                                                                  scan.startup.mode = 'earliest', kafka.max.num.messages = '10'                                     ) row format json;
CREATE_SOURCE
dev=> create materialized view mv1 as select count(*) from kafkas;
CREATE_MATERIALIZED_VIEW
dev=> flushl;
ERROR:  QueryError: sql parser error: Expected an SQL statement, found: flushl
dev=> flush;
FLUSH
dev=> select * from mv1;
 count
-------
    10
(1 row)


feature: L2 notifier

This adds an example of implementation for contracts interested in receiving L2D, including the FIXED_RAND. Also adds some README instructions to give a high level view of how the contracts should behave.
Most of the code here is copied (but reviewed and adapted) from @avoloshko original work.

@dougmrqs let's fix these minor comments and merge. Everything looks good


feat(Storagecontrol): Adding new library sample

Fixes https://github.com/googleapis/google-cloud-php/issues/7243
Storagecontrol Library already created https://github.com/googleapis/google-cloud-php/pull/7245
Tests pass on prod:
➜  storagecontrol git:(feat_storage_control) ✗ ../testing/vendor/bin/phpunit -c phpunit.xml.dist
PHPUnit 9.6.19 by Sebastian Bergmann and contributors.

Warning:       XDEBUG_MODE=coverage or xdebug.mode=coverage has to be set

.                                                                   1 / 1 (100%)

Time: 00:03.340, Memory: 10.00 MB

OK (1 test, 2 assertions)
➜  storagecontrol git:(feat_storage_control) ✗

Should've raised this on main repo, not the forked one.


Release annotations - Image Missing

In the last section of this page, it mentioned that the The following image shows how this marker appears. However, the image is missing. Please add to make it complete

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

ID: b7247c58-0a2c-a9a2-efa3-caf805fe0e18
Version Independent ID: cdd6fbf2-43b6-19c5-1dc6-8d1271bccbeb
Content: How to design your Application Insights deployment - One vs many resources? - Azure Monitor
Content Source: articles/azure-monitor/app/separate-resources.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@prawin2k Thank you very much for bringing this to our attention. I have assigned the issue to the content author to evaluate and update as appropriate.
Meanwhile, please note that annotations should appear as mentioned in this doc.

Thank you so much for pointing this out. The removal of the screenshot was intentional, but the line referencing the screenshot was missed in the removal. I have fixed this internally the change will go-live later today.

#please-close


Adding stats for upgrade test and updating to aetos

What this PR does / why we need it:
Created a stats struct and push data to aetos for PX upgrade
Which issue(s) this PR fixes (optional)
Closes #PTX-14872
Special notes for your reviewer:
http://aetos.pwx.purestorage.com/resultSet/testSetID/67685
sample result from aetos
{
    "_id": {
      "$oid": "63c050a31c575913d7ce0846"
    },
    "name": "px-upgrade-stats",
    "product": "px-enterprise",
    "statsType": "upgrade",
    "version": "0cc6b62",
    "data": {
      "dash-url": "http://aetos.pwx.purestorage.com/resultSet/testSetID/67685",
      "duration": "2 mins",
      "fromVersion": "2.12.2-0cc6b62",
      "status": "PASS",
      "toVersion": "2.12.2-0cc6b62"
    },
    "dateTime": "2023-01-12 18:25:39+00:00"
  }

I am done with my changes, please resolve conflicts and then it should be good


[Feature Request] Syntax highlighting for .svelte

Hi! 👋 This is a very cool and convenient app 👍.  It would be great if there was syntax highlighting for .svelte 🙏. Thank!

ahh, when is this available?


Toggle from disable to enable doesn't work with HeaderAdapter

Here is the source for what I'm working with.
adapter.withOnPreClickListener((v, adapter1, item, position) -> {
	// consume otherwise radio/checkbox will be deselected
	return true;
});
adapter.withItemEvent(new UserColorClickEvent() {
	@Override
	public void onClick(View v, int position, FastAdapter<UserHeaderItem> fastAdapter, UserHeaderItem item) {
		CLog.e("Pre %b %d %d", item.isSelected(), fastAdapter.getItemCount(), fastAdapter.getPreItemCount(position));
		fastAdapter.toggleSelection(position); //TODO toggle isn't working
		CLog.e("Post %b", item.isSelected());
		setColourPrinting(fragment, item.isSelected());
	}
});

The listener is simple and is based off of the checkbox one. If it starts off as unselected, the toggle works.
Pre false 98 0
Post true

However, if it starts off as selected, the item is still selected after the toggle.
Pre true 98 0
Post true

I have tested disable(position) and that works, so that part isn't the issue. Is there something with the list of selected items that isn't updating? I add it using withSetSelected

well if you started "selected" the problem is that the internal "selection" state is not set in the FastAdapter.. For faster access to the number of selected items (without iterating the whole list) the FastAdapter keeps track of this for you.
https://github.com/mikepenz/FastAdapter/blob/develop/library-core/src/main/java/com/mikepenz/fastadapter/FastAdapter.java#L64
If you use the alternative StateManagment this wouldn't be an issue.
https://github.com/mikepenz/FastAdapter/blob/develop/library-core/src/main/java/com/mikepenz/fastadapter/FastAdapter.java#L254
withPositionBasedStateManagement(false)

Ah oops. Wasn't aware that the internal selection wasn't being set with with selected. Thanks


Checkbox (single) field can not be set to be required in application forms

In some forms it’s needed to set single checkbox to required in application forms.

@danblah There is already a PR for this, just want Torchbox to check it out before deploy.
https://github.com/OpenTechFund/opentech.fund/pull/532


Remove deprecations included in 0.10 release

See: #959

Codecov Report

Merging #969 into develop will decrease coverage by <.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop    #969     +/-   ##
=========================================
- Coverage     84.4%   84.4%   -<.1%
=========================================
  Files           58      53      -5
  Lines         4945    5171    +226
=========================================
+ Hits          4176    4365    +189
- Misses         769     806     +37

Impacted Files
Coverage Δ

stellargraph/core/graph.py
98.5% <ø> (-0.1%)
:arrow_down:

stellargraph/data/loader.py
42.1% <ø> (+4.6%)
:arrow_up:

stellargraph/ensemble.py
85.5% <0%> (ø)

stellargraph/calibration.py
79.9% <0%> (ø)

Continue to review full report at Codecov.

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


Add pre-commit-hooks to prepare for  hlint and clean up format checking

removed hacky formatchecking
prepared for running hlint check as soon as we have all the hints applied

How do you format the code?

How is this better than what we have before?

It's more readable
easier to use
will make adding more checks absolutely easy
most importantly: runs automatically so you never have an unformatted commit

I've decided against this unfortunately. While the idea is nice, pre-commit is pretty much just bloatware.
The only useful feature is that it automatically adds the git commit hook when you enter a shell. That's trivial to reimplement. It is not at all worth the complexity of this huge framework. It would take ~20 lines of bash to reimplement.


what does 'acc' mean on the text log?

2019-04-03 20:21:39,389 - INFO - Epoch [12][7000/7330]	lr: 0.00020, eta: 0:03:53, time: 0.709, data_time: 0.012, acc: 93.4033, loss_rpn_reg: 0.0282, loss_rpn_cls: 0.0240, loss_reg: 0.1099, loss_cls: 0.1798, loss: 0.3420
what does 'acc' mean on the text log?

accuracy (for bbox head)

thanks


Validate attr_spec and geom_spec JSON schema in postgres

See pg_jsonschema
https://supabase.com/blog/pg-jsonschema-a-postgres-extension-for-json-validation

Also, consider validating them with an R function and running validation in gaiaShiny


Refactor codeabase

There is a lot of leftover code from the analysis mode of OptMeowt before it was a separate extension. We should refactor the codebase to make it more clear. One way to refactor is to just remove code and see if the extension still works.

I went through and deleted a lot of unnecessary code, but this will need more testing to make sure everything is working as intended.

@OliverWang13 will look into whether we should set up a test framework. @katehausladen will also help out with @sophieeng advising as well.
@OliverWang13 will concretely run the extension with and without refactoring and see if the results are the same (given that both crawls may be a bit different just because). General success rate, total number of DNS links should be about the same. But differences between individual sites may exist.
@katehausladen may provide a testing script.

We want to remove two columns from the database scheme: uspapi_opted_out and usp_cookies_opted_out. This information is already present in the other columns.

Based on some preliminary testing, the extension seems to work normally after my last commit. The new command to create the sql database is
CREATE TABLE entries (id INTEGER PRIMARY KEY AUTO_INCREMENT, site_id INTEGER, domain varchar(255), dns_link BOOLEAN, sent_gpc BOOLEAN, uspapi_before_gpc varchar(255), uspapi_after_gpc varchar(255), usp_cookies_before_gpc varchar(255), usp_cookies_after_gpc varchar(255));

I will update the wiki (for setting up the SQL db) after we merge this issue into main. We will also need to update the architecture after we finish deleting old code.

In the previous commit, I rearranged some code that actually analyzes each site. After I made the edits, I crawled 1001 sites, and the extension performed significantly better.
Success rate: 98.7%, which is 6% higher than the previous crawl (excludes insecure certificate/reached error page sites)
DNSLs: 51.6%, which is ~2% higher than the previous crawl
USPS: 35.5%, which is ~11% higher than the previous crawl
This is comparing the sites from the new crawl to the same sites in the previous crawl.
More details on what I changed:
The functions that start and stop analysis had nested functions inside them that were executing at the beginning of the function where they were defined instead of when they were called. I don’t really understand why this was happening (because I think theoretically it shouldn’t), but when I moved the nested function definitions to immediately before their call, the extension performance was way better.
I realized this was happening because I made a debugging table in the db that essentially functioned as a console log, so I could see the order of execution was wrong. I removed the debugging table before I committed the code, but since there is not console log for the extension during crawls, I think it might be useful to keep and set it up such that it’s optional.

After I made the edits, I crawled 1001 sites, and the extension performed significantly better.

Excellent, @katehausladen!

I removed the debugging table before I committed the code, but since there is not console log for the extension during crawls, I think it might be useful to keep and set it up such that it’s optional.

Just for my information, is there a reason why we are not using a console log during crawls but instead a debugging table?
Yes, an option to have the debugging table created during the crawl would be nice (and let's document it in the readme).

Just to clarify, the debugging table functions as a console log for the analysis extension only. We do have a console log for the crawler script, since it runs locally from a terminal. I haven't found an effective way to use a console log for the analysis extension during crawls. I used the browser console to see a console log for the OptMeowt extension like this, but that isn't really a great option because all the logs disappear when the browser quits. (During crawls, the browser quits every time the crawler restarts as well as automatically when the crawl finishes.)

That sounds good to me. So, I would say, let's do as you say, @katehausladen, and let's include something along the lines of what you comment in the readme.

I have tested since the refactoring and it seems to be working properly. I also tested Kate's data processing process and it works great.

I added the optional debugging table. The option to use the debugging table is selected by running the rest api with
node index.js debug

(to use the table) versus
node index.js

(to not use the table). Obviously, if someone chooses to use the debugging table, they must set up the table in the database, too.
I updated the wiki with this information since the instructions for setting up the database and rest api are there.

As I was watching the crawler, I noticed that sometimes, analysis data would be posted before the site even finished loading. I modified the extension so that it does not start analyzing a site until the site has fully loaded.
I also noticed that the USPAPI is really sensitive to timing (i.e., if we check it too early, it will be undefined, but if we wait long enough, it will have a value). I added longer timeouts to try to accommodate this.
The result is a slower crawler with a slightly lower success rate, as sites that do not fully load will not be analyzed. Depending on the run, the success rate is somewhere between 95 and 98%, but the data we are collecting is a lot more accurate and consistent.

Excellent!

@katehausladen will close this issue once there is nothing more to do here.

I've opened a pull request for this branch with Oliver as a reviewer. I think we've made all the big changes, and any additional changes that relate to testing analysis performance can be done in issue #54.

When I was updating the architecture readme, I found a few more things that needed to be deleted. (keyboard shortcuts out of the manifests as well as the modes.js file)

I'll update the architecture accordingly.


added sensitivity params for manual control

@kd0aij I think this is what you desire, right? Would be nice if you could review and eventually test. I'll also give this a try.

Thanks, this looks like what I was talking about; the ability to specify "low rate" for manual control.
I'll set up my Ninox with lower manual rates for my next flight test.
The root of my perceived problem is that, when I use dual rates at the TX, only the high rate provides full response in auto modes, since those modes assume the TX is on "max rate", but that in an emergency take-over, the pilot must contend with high rates, or else also switch to low rates during the emergency.
These new parameters allow the pilot to apply a lower rate for manual control without the need to flip a switch at the TX when switching to manual in an emergency. The only downside to this is that you have the option to set the manual sensitivity too low to allow recovery from "unusual attitudes", but this should not be a problem if the parameters are used wisely.
For pilots using programmable transmitters, it is probably possible to couple flight mode to control rates to achieve the same (and greater) capability. The question is whether these parameters provide a necessary feature for low-end RC transmitters or for simplifying setup using high-end TXs.

@kd0aij If you test-fly this we can probably get it into the release. This together with expo should make dual rate completely unnecessary.

@kd0aij Any chance of testing this?

not yet, will flash it on my Ninox for the next opportunity at the field.

@kd0aij Hi Mark, could you ping me if you are almost ready to test this one. There is this PR #2424  which you could test together with this one. My plane is in the garage at the moment (fried 2 engines) so I cannot test myself.

this parameter has no effect on elevon deflections, but I verified with warnx printout that it is scaling down the values of _actuators.control[0:1]
Even with the sensitivity params set to 0.0, I get full elevon deflections in roll and pitch
Sorry to hear about the fried motors; hot weather?

@kd0aij @LorenzMeier I think it's because of the manual pass-through. This thing fools me for the 4. time now.

That would be the "failsafe" manual control provided by the IO processor, I guess. Which means the confusing thing is the else block /* manual/direct control */ which does nothing in fw_att_control. If that's right, the block should be replaced by a comment which reminds us that the IO processor handles everything in manual mode.

That's not correct. The IO block is optional in the system architecture, and neither very old nor very new systems have it.

This brings up a question about the new STABILIZE mode in FW: I have it set up on a separate switch for my Ninox, but does this mean that to recover manual control on an FMU hang that I must flip the ACRO/STABILIZE switch back to MANUAL? That will require a different reflex action than my current one.

@kd0aij Not 100% sure but in case of a fmu hang the IO should notice and go to failsafe.

I doubt that the IO processor is smart enough to know when the FMU is confused.


add echarts demo

add echarts demo

Add form components,add lodash,dayjs,jquery.After yarn run lint,some problems have been fixed.


virt-api, webhooks: Decouple VMI create admitter & network validator

What this PR does
The network validator id tangled in various  VMI create including the following components:

VMsAdmitter
VMPoolAdmitter
VMIRSAdmitter
VMIUpdateAdmitter.

The VMICreateAdmitter.Admit() method use the network-validation directly, while other components use it indirectly by calling ValidateVirtualMachineInstanceSpec.
Before this PR:
The VMICreateAdmitter's Admit() method call network-admitter directly, introducing tight coupling on it making it impossible to test.
After this PR:
The network-validator is injected to VMICreateAdmitter making Admit() method decoupled from the network-validator code.
Having unit-test that cover the network validator call as part of the VMI create admitter flow.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Decouple remaining network validator from VMI create admitter code require removing the network-validator call from ValidateVirtualMachineInstanceSpec, require injecting the network-validator to the objects that call it.
This means additional work that require more thinking, it should be done in follow up PR(s).

The decoupling is basicly passing the network validator at the VMI create admitter creation.
It require changing the network validaor facotry to fit the context of the VMI create admitter.
In other words, change the network validator factory to not receive the VMI spec & cluster-config.

Since most of the changes are signature changes causing lot of noise, the PR is structured to deal with most signature changes first, including: changing the network validator methods, the factory.
And finally the actual decoupling.
Hopefully it"ll make reviewing this PR easier.

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/cc @orelmisan

Thank you for the PR @ormergi. The VMI creation admitter instantiates the network admitter twice:
1. Directly in `VMICreateAdmitter.Admit` - which was handled.

2. Indirectly in `ValidateVirtualMachineInstanceSpec` - that still exists following this change, thus  the admitter still depends on `kubevirt.io/kubevirt/pkg/network/admitter`.

To better reflect the current state, please consider updating the relevant commit message and PR description to indicate that a partial decoupling was performed. I think the rest could be done in a follow-up. WDYT?

I thought PR description is clear enough about this change, I will re-rephrase it per your suggestion.
As for ValidateVirtualMachineInstanceSpec, decoupling network admitter from it require additional changes (similar to this PR changes) to the following components:

VMsAdmitter
VMPoolAdmitter
VMIRSAdmitter
VMIUpdateAdmitter.
End result, pkg/virt-api/webhooks/validating-webhook/**admitters** decoupled from pkg/network/admitter
But pkg/virt-api/webhooks/validating-webhook (one level higher in the tree) would still depend on pkg/network/admitter (consuming some interface).
Given we still have a dependency, I wonder if it worth changing the above components.

Changes: resolve https://github.com/kubevirt/kubevirt/pull/13388#pullrequestreview-2478268180

Changes:

Resolve comments https://github.com/kubevirt/kubevirt/pull/13388#discussion_r1871132068 https://github.com/kubevirt/kubevirt/pull/13388#discussion_r1871132911
Move network-validator creation to higher level layer as discussed in this thread https://github.com/kubevirt/kubevirt/pull/13388#discussion_r1870023179

Changes:  resolve https://github.com/kubevirt/kubevirt/pull/13388#discussion_r1871130087

Changes: rebase

Changes:: Decouple VMIUpdateAdmitter and network-validator (it make sense for this PR to address that usage as well).

@orelmisan PR description is updated according to recent changes

Thank you for the changes @ormergi. I think the PR should focus on a single webhook as it did last week.

Sorry for the noise since last week, this PR did change and its socpe cover additional admitter, so that now it treat the validation on create requests.
I also posted a follow up PR to address the remaining network-validator usages, in the non-create request flow.
Please note that only the last commit is added, the rest are mandator. I hope we can keep it and save the overhead of additional PR for that single commit.

Changes: address https://github.com/kubevirt/kubevirt/pull/13388#pullrequestreview-2493029302
@orelmisan PTAL

Thank you for the changes @ormergi. I suggest deferring moving the tests to a follow-up in order to decrease the scope of this PR.

Done, PTAL

/cc @EdDev

@enp0s3 / @iholder101 hey could you please have a look at the files that are out of sig-net approver?
Basically pkg/virt-api

Chnages: Rebase


[Dynamic Regions] Raft-Engine rewrite load results in qps jitter.

Bug Report

In some heavy read-write workload,  Raft-Engine's rewrite load has strong correlation with grpc qps jitter.
Though increasing the raft-engine gc threshold could mitigate it,  we may need a better default value as a short term mitigation.
What version of TiKV are you using?

v7.0
What operating system and CPU are you using?

Steps to reproduce

What did you expect?
What did happened?

/cc @tabokie

Fixed by https://github.com/tikv/tikv/pull/14700


[Shortcut Guide] Move SG out of the runner process

Provide a description of the new feature / enhancement
SG should run in its own process PowerToys.ShortcutGuide.exe
It will stop supporting long press Win key and only support Win + ?.
Work items:

[x] create feature/shortcutguide working branch
[x] create a new project folder for shortcut guide src/modules/ShortcutGuide
[x] move all the SG resources to the new project (including src/runner/svgs/*.svg)
[x] move the src/runner/svgs/icon.ico to src/runner/icon.ico and delete the src/runner/svgs folder
[x] verify if there is any remaining SG-only code in src/common and move it to the new project
[x] drop support for the module interface for settings, settings don't need to be propagated as SG process is started on demand
[x] add new option to rebind the Win + ? hotkey
[x] add logger
[x] add global exception handling
[x] update the BugReportTool to collect the SG log
[x] update installer
[x] update .pipelines/pipeline.user.windows.yml for signing new binaries
[x] update OOBE documentation
[ ] update online documentation (tracked by #11079)
[x] terminate SG process when runner exits
[x] don't run more than one instance of SG
[x] add the icon to the ShortcutGuide process
[x] review telemetry
[x] implement RegisterHotkey interface for ModuleInterface
[x] close SG window as soon as a shortcut from the list was pressed

[ ] when moving the src/runner/svgs/*.svg files to the new location, also move the src/runner/svgs/icon.ico to src/runner/icon.ico and delete the src/runner/svgs folder

Resolved with the release of v0.41.


Question about LSTM and sentiment detection

Sorry I'm new to Machine Learning.
I noticed when i follow the LSTM sentiment detection tutorial, the output is often empty or strange broken words when the input is something that is different from one of the trained inputs.
Why is that the case and is there no way to avoid this issue? Can't the model just return the closest match if it's not sure?

The issue template is there to help us help you.  Here it is in case you missed it.

I noticed when i follow the LSTM sentiment detection tutorial

What tutorial, where?

the output is often empty or strange broken words

Which output?  How do we reproduce?

Why is that the case and is there no way to avoid this issue?

Without knowing what we're debugging, answering this question is next to impossible.


swift version please!

:)

#93

Closed as fixed with https://github.com/Boris-Em/BEMCheckBox/pull/95.


mariadb: 10.3.18 -> 10.4.8

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mariadb-server/versions.
meta.description for mariadb is: '"An enhanced, drop-in replacement for MySQL"'.
meta.homepage for mariadb is: '"https://mariadb.org/"

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 10.4.8 with grep in /nix/store/vdncffr44rmjc898q03b9w1v775mlgcv-mariadb-server-10.4.8

directory tree listing: https://gist.github.com/ea59ca24bc4f1d93efdaa1e7ed362a5a

du listing: https://gist.github.com/53ef2270e81304a35d7bcec1cb183374

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

182 total rebuild path(s)
59 package rebuild(s)
59 x86_64-linux rebuild(s)
57 i686-linux rebuild(s)
9 x86_64-darwin rebuild(s)
57 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
akonadi
akregator
automysqlbackup
diesel-cli
digikam
kaddressbook
kdeApplications.akonadi
kdeApplications.akonadi-calendar
kdeApplications.akonadi-contacts
kdeApplications.akonadi-import-wizard
kdeApplications.akonadi-mime
kdeApplications.akonadi-notes
kdeApplications.akonadi-search
kdeApplications.akonadiconsole
kdeApplications.akregator
kdeApplications.calendarsupport
kdeApplications.eventviews
kdeApplications.incidenceeditor
kdeApplications.kaddressbook
kdeApplications.kalarm
kdeApplications.kalarmcal
kdeApplications.kdepim-addons
kdeApplications.kdepim-apps-libs
kdeApplications.kdepim-runtime
kdeApplications.kgpg
kdeApplications.kmail
kdeApplications.kmail-account-wizard
kdeApplications.kmailtransport
kdeApplications.knotes
kdeApplications.kontact
kdeApplications.korganizer
kdeApplications.libgravatar
kdeApplications.libkdepim
kdeApplications.libksieve
kdeApplications.mailcommon
kdeApplications.mailimporter
kdeApplications.mbox-importer
kdeApplications.messagelib
kdeApplications.pim-data-exporter
kdeApplications.pim-sieve-editor
kdeApplications.pimcommon
kgpg
kmail
kmymoney
kontact
korganizer
lua51Packages.luadbi-mysql
lua52Packages.luadbi-mysql
lua53Packages.luadbi-mysql
luaPackages.luadbi-mysql

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vdncffr44rmjc898q03b9w1v775mlgcv-mariadb-server-10.4.8 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A mariadb https://github.com/r-ryantm/nixpkgs/archive/27dee7ad998c36ffb7ae63935bf288276af32ade.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vdncffr44rmjc898q03b9w1v775mlgcv-mariadb-server-10.4.8
ls -la /nix/store/vdncffr44rmjc898q03b9w1v775mlgcv-mariadb-server-10.4.8/bin

cc @thoughtpolice for testing.

@GrahamcOfBorg test mysql

Replaced by https://github.com/NixOS/nixpkgs/pull/73267.


travis: check code generation for dot parser in graph/formats/dot/internal

This requires that gocc and make be installed on the builder (and possibly other things that may not be present).

For inspiration, see https://github.com/graphism/dot/blob/master/.travis.yml

Please send a PR. It would probably be good if the PR also had a go generate hook in "graph/formats/dot/internal" so we can pretend that make doesn't exist.

Please send a PR. It would probably be good if the PR also had a go generate hook in "graph/formats/dot/internal" so we can pretend that make doesn't exist.

I'll take a look at it during the following week.

I'll take a look at it during the following week.

Time flies, especially when its sunny outdoors :)
Just to let you know, I'll be heading out for a trek and should be back towards early August. Will get back to coding then, if not before.
Have a lovely summer.


Process_KILLED

Hi @kcleal I am trying to run a bam files and after running for couple of minutes it gets KILLED, Could you please suggest some ideas I don't understand what's happening haven't seen dysgu doing this before :(
Thanks in advance.

Hi @poddarharsh15,
It looks like a lot of reads are being analysed - 363 million, is that expected? You could have a look at the bam file generated from the fetch stage in the working directory. If you index it with samtools, you will be able to inspect in in IGV or GW. Perhaps there are lots of adapter sequences, or soft-clipped reads in there? That could cause problems

I hope that's not the problem because adapter has been removed already and after that I removed also the duplicates using samtools. I will check it again last time when I ran Dysgu on the same samples without removing duplicates it didn't give me this problem.

Hi @kcleal I have checked other samples on which I have ran dysgu with similar reads count, please see examples below
bwamem=502504302
dragen=503773618
minimap=502983985

Please let me know if you can provide some advice to make changes.
Thanks in advance.

I am investigating these two flowcells again I will get back to you ASAP. Thanks for the help appreciate it.

Seems like I have found the issue why this was happening because after removal of duplicates the properly mapped read % got extremely lower :( Please the images below for the reference many thanks again.

Ill close this for now


Something is wrong in run.md

May be it's 'xd_test_maple.sh' in 'Now evaluate imageNet model on downstream datasets' ?

Hi @EdmunddzzZ,
Thank you for your message and pointing out the typo.
It has been fixed.
Kind regards.


kona ev elect_ems add

Choose one of the templates below:
Fingerprint
kona ev and kona hev
Car support
This pull requests adds support for <Make - Model - Year - Trim>.
This is an explorer link to a drive with the stock system enabled: ...
This is an explorer link to a drive with openpilot system enabled: ...
Feature
This pull requests adds feature X
Description
Explain what the feature does
Testing
Explain how the feature was tested. Either by the added unit tests, or what tests were performed while driving.

if frame % 5 == 0 and self.car_fingerprint in [CAR.SONATA, CAR.PALISADE, CAR.SONATA_H, CAR.SANTA_FE, CAR.KIA_NIRO_EV]:
Shouldn't we include CAR.KONA_EV since the Niro is built on the Kona Platform, or does it not use 20 Hz?

kona ev lfa....carcontroller not added....cluster no error

thank you very much @crwusiz

I'm assuming with this commit to HKG I should be able to load HKG on my Comma2 and it should work with no edits in my Kona EV, am I right?

I'm assuming with this commit to HKG I should be able to load HKG on my Comma2 and it should work with no edits in my Kona EV, am I right?

yes...kona ev comma two test ok


Missing TIGGE data

Data for some variables is missing for certain months. (Damaged tape).
Solution: Make sure that missing dates are omitted and not filled with NaNs

Fixed in #52


https://unpkg.com/vue-flatpickr-component@7 broken

At the moment: https://unpkg.com/vue-flatpickr-component@7 returns
Cannot find module "dist/vue-flatpickr.min.js" in package<EMAIL_ADDRESS>Might be related to #86

Interestingly
This url works
<EMAIL_ADDRESS>
but
<EMAIL_ADDRESS>
does not list dist folder there
Ref https://github.com/ankurk91/vue-flatpickr-component/issues/91

Sorry, I had wrong url. It should be  https://unpkg.com/vue-flatpickr-component@7, which returns "Cannot find module "dist/vue-flatpickr.min.js" in package<EMAIL_ADDRESS>

Same issue with<EMAIL_ADDRESS>&<EMAIL_ADDRESS>This bug caused several CI failures for Vue.js…

I'm trying a different server configuration on the backend and all the URLs you listed are working for now. But ya, I think this is a dup of #86.


[Bug]:  Requests larger than 75k input tokens cause Input prompt (512 tokens) is too long and exceeds the capacity of block_manager error

Your current environment

The output of `python collect_env.py`
Collecting environment information...
PyTorch version: 2.4.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Amazon Linux 2 (x86_64)
GCC version: (GCC) 7.3.1 20180712 (Red Hat 7.3.1-17)
Clang version: Could not collect
CMake version: version 3.27.7
Libc version: glibc-2.26

Python version: 3.10.9 | packaged by conda-forge | (main, Feb  2 2023, 20:20:04) [GCC 11.3.0] (64-bit runtime)
Python platform: Linux-5.10.220-209.869.amzn2.x86_64-x86_64-with-glibc2.26
Is CUDA available: True
CUDA runtime version: 12.1.105
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA L4
Nvidia driver version: 550.90.07
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:        x86_64
CPU op-mode(s):      32-bit, 64-bit
Byte Order:          Little Endian
CPU(s):              8
On-line CPU(s) list: 0-7
Thread(s) per core:  2
Core(s) per socket:  4
Socket(s):           1
NUMA node(s):        1
Vendor ID:           AuthenticAMD
CPU family:          25
Model:               1
Model name:          AMD EPYC 7R13 Processor
Stepping:            1
CPU MHz:             3364.353
BogoMIPS:            5299.99
Hypervisor vendor:   KVM
Virtualization type: full
L1d cache:           32K
L1i cache:           32K
L2 cache:            512K
L3 cache:            16384K
NUMA node0 CPU(s):   0-7
Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf tsc_known_freq pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch topoext invpcid_single ssbd ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 invpcid rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 clzero xsaveerptr rdpru wbnoinvd arat npt nrip_save vaes vpclmulqdq rdpid

Versions of relevant libraries:
[pip3] mypy==1.9.0
[pip3] mypy-extensions==1.0.0
[pip3] numpy==1.26.2
[pip3] nvidia-nccl-cu11==2.14.3
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] torch==2.4.0
[pip3] torchvision==0.19.0
[pip3] triton==3.0.0
[pip3] vllm-nccl-cu12==<IP_ADDRESS>.4.0
[conda] numpy                     1.26.2                   pypi_0    pypi
[conda] nvidia-nccl-cu11          2.14.3                   pypi_0    pypi
[conda] nvidia-nccl-cu12          2.20.5                   pypi_0    pypi
[conda] torch                     2.4.0                    pypi_0    pypi
[conda] torchvision               0.19.0                   pypi_0    pypi
[conda] triton                    3.0.0                    pypi_0    pypi
[conda] vllm-nccl-cu12            <IP_ADDRESS>.4.0             pypi_0    pypiROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.5
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	0-7	0		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

🐛 Describe the bug
I have a server up and running using this
vllm serve Mistral-Nemo-Instruct-2407/ --port 8006 --gpu-memory-utilization 0.9 --max-model-len 128000 --tensor-parallel-size 1 --pipeline-parallel-size 2 --quantization fp8 --uvicorn-log-level debug

on two separate NVidia GPUs.
However recently I have started noticing this error that I do not recall seeing before.
I am using documents that are up to 125k tokens in size.
Input prompt (512 tokens) is too long and exceeds the capacity of block_manager

i have tried looking around the issues list and going through what I think would be the solutions. I have tried the v2 block manager. setting max_num_batched_tokens to all possible values (outrageously even the context window of the model) but I keep seeing that error (replace number of tokens with the tokens set at max_num_batched_tokens).
I have also tried enabling/disabling chunked prefill and that didn't help either. I am not sure what's more left to do and looking for help around this problem.

The output of vllm once serve command is executed`

INFO 08-26 19:18:49 api_server.py:440] vLLM API server version 0.5.5
INFO 08-26 19:18:49 api_server.py:441] args: Namespace(model_tag='Mistral-Nemo-Instruct-2407/', host='langmodel2', port=8006, uvicorn_log_level='debug', allow_credentials=False, allowed_origins=['*'], allowed_methods=['*'], allowed_headers=['*'], api_key=None, lora_modules=None, prompt_adapters=None, chat_template=None, response_role='assistant', ssl_keyfile=None, ssl_certfile=None, ssl_ca_certs=None, ssl_cert_reqs=0, root_path=None, middleware=[], return_tokens_as_token_ids=False, disable_frontend_multiprocessing=False, model='Mistral-Nemo-Instruct-2407/', tokenizer=None, skip_tokenizer_init=False, revision=None, code_revision=None, tokenizer_revision=None, tokenizer_mode='auto', trust_remote_code=False, download_dir=None, load_format='auto', dtype='auto', kv_cache_dtype='auto', quantization_param_path=None, max_model_len=128000, guided_decoding_backend='outlines', distributed_executor_backend=None, worker_use_ray=False, pipeline_parallel_size=2, tensor_parallel_size=1, max_parallel_loading_workers=None, ray_workers_use_nsight=False, block_size=16, enable_prefix_caching=False, disable_sliding_window=False, use_v2_block_manager=False, num_lookahead_slots=0, seed=0, swap_space=4, cpu_offload_gb=0, gpu_memory_utilization=0.9, num_gpu_blocks_override=None, max_num_batched_tokens=None, max_num_seqs=256, max_logprobs=20, disable_log_stats=False, quantization='fp8', rope_scaling=None, rope_theta=None, enforce_eager=False, max_context_len_to_capture=None, max_seq_len_to_capture=8192, disable_custom_all_reduce=False, tokenizer_pool_size=0, tokenizer_pool_type='ray', tokenizer_pool_extra_config=None, limit_mm_per_prompt=None, enable_lora=False, max_loras=1, max_lora_rank=16, lora_extra_vocab_size=256, lora_dtype='auto', long_lora_scaling_factors=None, max_cpu_loras=None, fully_sharded_loras=False, enable_prompt_adapter=False, max_prompt_adapters=1, max_prompt_adapter_token=0, device='auto', num_scheduler_steps=1, scheduler_delay_factor=0.0, enable_chunked_prefill=None, speculative_model=None, speculative_model_quantization=None, num_speculative_tokens=None, speculative_draft_tensor_parallel_size=None, speculative_max_model_len=None, speculative_disable_by_batch_size=None, ngram_prompt_lookup_max=None, ngram_prompt_lookup_min=None, spec_decoding_acceptance_method='rejection_sampler', typical_acceptance_sampler_posterior_threshold=None, typical_acceptance_sampler_posterior_alpha=None, disable_logprobs_during_spec_decoding=None, model_loader_extra_config=None, ignore_patterns=[], preemption_mode=None, served_model_name=None, qlora_adapter_name_or_path=None, otlp_traces_endpoint=None, collect_detailed_traces=None, engine_use_ray=False, disable_log_requests=False, max_log_len=None, dispatch_function=<function serve at 0x7fe663255ea0>)
INFO 08-26 19:18:49 api_server.py:144] Multiprocessing frontend to use ipc:///tmp/db61d9b2-4054-4ebb-92a8-3a5b1d8a81ed for RPC Path.
INFO 08-26 19:18:49 api_server.py:161] Started engine process with PID 11035
INFO 08-26 19:18:54 config.py:813] Defaulting to use ray for distributed inference
WARNING 08-26 19:18:54 arg_utils.py:839] Chunked prefill is enabled by default for models with max_model_len > 32K. Currently, chunked prefill might not work with some features or models. If you encounter any issues, please disable chunked prefill by setting --enable-chunked-prefill=False.
INFO 08-26 19:18:54 config.py:911] Chunked prefill is enabled with max_num_batched_tokens=512.
2024-08-26 19:18:54,825	INFO worker.py:1603 -- Connecting to existing Ray cluster at address: <IP_ADDRESS>:6379...
2024-08-26 19:18:54,830	INFO worker.py:1779 -- Connected to Ray cluster. View the dashboard at <IP_ADDRESS>:8265
INFO 08-26 19:18:54 llm_engine.py:184] Initializing an LLM engine (v0.5.5) with config: model='Mistral-Nemo-Instruct-2407/', speculative_config=None, tokenizer='Mistral-Nemo-Instruct-2407/', skip_tokenizer_init=False, tokenizer_mode=auto, revision=None, rope_scaling=None, rope_theta=None, tokenizer_revision=None, trust_remote_code=False, dtype=torch.bfloat16, max_seq_len=128000, download_dir=None, load_format=LoadFormat.AUTO, tensor_parallel_size=1, pipeline_parallel_size=2, disable_custom_all_reduce=False, quantization=fp8, enforce_eager=False, kv_cache_dtype=auto, quantization_param_path=None, device_config=cuda, decoding_config=DecodingConfig(guided_decoding_backend='outlines'), observability_config=ObservabilityConfig(otlp_traces_endpoint=None, collect_model_forward_time=False, collect_model_execute_time=False), seed=0, served_model_name=Mistral-Nemo-Instruct-2407/, use_v2_block_manager=False, enable_prefix_caching=False)
INFO 08-26 19:18:55 ray_gpu_executor.py:133] use_ray_spmd_worker: False
INFO 08-26 19:19:04 utils.py:975] Found nccl from library libnccl.so.2
INFO 08-26 19:19:04 pynccl.py:63] vLLM is using nccl==2.20.5
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:04 utils.py:975] Found nccl from library libnccl.so.2
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:04 pynccl.py:63] vLLM is using nccl==2.20.5
INFO 08-26 19:19:04 model_runner.py:879] Starting to load model Mistral-Nemo-Instruct-2407/...
Loading safetensors checkpoint shards:   0% Completed | 0/5 [00:00<?, ?it/s]
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:04 model_runner.py:879] Starting to load model Mistral-Nemo-Instruct-2407/...
Loading safetensors checkpoint shards:  20% Completed | 1/5 [00:00<00:01,  2.15it/s]
Loading safetensors checkpoint shards:  40% Completed | 2/5 [00:01<00:01,  1.91it/s]
Loading safetensors checkpoint shards:  60% Completed | 3/5 [00:01<00:00,  3.00it/s]
Loading safetensors checkpoint shards: 100% Completed | 5/5 [00:01<00:00,  4.69it/s]
Loading safetensors checkpoint shards: 100% Completed | 5/5 [00:01<00:00,  3.62it/s]

INFO 08-26 19:19:06 model_runner.py:890] Loading model weights took 6.5727 GB
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:06 model_runner.py:890] Loading model weights took 6.5727 GB
INFO 08-26 19:19:06 distributed_gpu_executor.py:56] # GPU blocks: 9650, # CPU blocks: 3276
INFO 08-26 19:19:09 model_runner.py:1181] Capturing the model for CUDA graphs. This may lead to unexpected consequences if the model is not static. To run the model in eager mode, set 'enforce_eager=True' or use '--enforce-eager' in the CLI.
INFO 08-26 19:19:09 model_runner.py:1185] CUDA graphs can take additional 1~3 GiB memory per GPU. If you are running out of memory, consider decreasing `gpu_memory_utilization` or enforcing eager mode. You can also reduce the `max_num_seqs` as needed to decrease memory usage.
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:09 model_runner.py:1181] Capturing the model for CUDA graphs. This may lead to unexpected consequences if the model is not static. To run the model in eager mode, set 'enforce_eager=True' or use '--enforce-eager' in the CLI.
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:09 model_runner.py:1185] CUDA graphs can take additional 1~3 GiB memory per GPU. If you are running out of memory, consider decreasing `gpu_memory_utilization` or enforcing eager mode. You can also reduce the `max_num_seqs` as needed to decrease memory usage.
(RayWorkerWrapper pid=2169, ip=<IP_ADDRESS>) INFO 08-26 19:19:33 model_runner.py:1300] Graph capturing finished in 24 secs.
INFO 08-26 19:19:34 model_runner.py:1300] Graph capturing finished in 26 secs.
INFO 08-26 19:19:35 api_server.py:209] vLLM to use /tmp/tmp841d197_ as PROMETHEUS_MULTIPROC_DIR
WARNING 08-26 19:19:35 serving_embedding.py:188] embedding_mode is False. Embedding API will not work.
INFO 08-26 19:19:35 launcher.py:20] Available routes are:
INFO 08-26 19:19:35 launcher.py:28] Route: /openapi.json, Methods: HEAD, GET
INFO 08-26 19:19:35 launcher.py:28] Route: /docs, Methods: HEAD, GET
INFO 08-26 19:19:35 launcher.py:28] Route: /docs/oauth2-redirect, Methods: HEAD, GET
INFO 08-26 19:19:35 launcher.py:28] Route: /redoc, Methods: HEAD, GET
INFO 08-26 19:19:35 launcher.py:28] Route: /health, Methods: GET
INFO 08-26 19:19:35 launcher.py:28] Route: /tokenize, Methods: POST
INFO 08-26 19:19:35 launcher.py:28] Route: /detokenize, Methods: POST
INFO 08-26 19:19:35 launcher.py:28] Route: /v1/models, Methods: GET
INFO 08-26 19:19:35 launcher.py:28] Route: /version, Methods: GET
INFO 08-26 19:19:35 launcher.py:28] Route: /v1/chat/completions, Methods: POST
INFO 08-26 19:19:35 launcher.py:28] Route: /v1/completions, Methods: POST
INFO 08-26 19:19:35 launcher.py:28] Route: /v1/embeddings, Methods: POST
INFO 08-26 19:19:35 launcher.py:33] Launching Uvicorn with --limit_concurrency 32765. To avoid this limit at the expense of performance run with --disable-frontend-multiprocessing
INFO:     Started server process [10979]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://langmodel2:8006 (Press CTRL+C to quit)

Before submitting a new issue...

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

Pretty arbitrary but anything above 75,000 tokens as a request seems to be causing this issue. Given the model context window is ~128k tokens

I'm also having this issue. It seems a silent failure possibly with longer input context lengths. I'm also using the new pipeline parallelism vs. tensor parallelism which likely changes how the kv cache is distributed in a multi-gpu setup.

@servient-ashwin  if you have 2 L4 gpus, I doubt if you can have enough memory to hold 125k tokens 👀

@youkaichao That could be the case, however what would the requirements look like for a model like this? I am even using the fp8 bit quantization mode to load this model across two GPUs with pipeline_parallel as 2 and tensor_parallel as 1.

vLLM will print some logs like # GPU blocks: 790. Multiply the number by 16 (the block size), and you can get roughly the maximum number of tokens that can be served on the current configuration.

Yes indeed, however, from the logs I shared I get this
INFO 08-26 19:19:06 distributed_gpu_executor.py:56] # GPU blocks: 9650, # CPU blocks: 3276

which means 9650*16 = ~154k I should say. but it seems to be half of it as of now.


chore(NumberTheory/WellApproximable): golf proof using tauto

Two independent changes are being proposed here:

Using tauto: this was annoyingly slow in Lean 3 but is not in Lean 4,
Changing p to ((p : ℕ) : ℝ): this is because Lean 4 is unfortunately much worse at coercing from Nat.Primes to Real and will waste about 5 seconds working on this via typeclass inference of CoeT searches without this change.

Some discussion here on Zulip.

bors merge


Refactor "layers" into folders

We already have a somehow implicit layer architecture in the core project. Low level functions are on the top of the project, commands in the middle and public api on the bottom.
I think we should make this more explicit and introduce folders.
What do people think?
Is there a suggested folder structure?
/cc @mexx @theimowski @isaac_abraham

Currently I would not introduce folders, the Paket.Core project with 43 .fs files is small enough for me to find the stuff I'm working with.
Further I would prefer to go more in the direction of Hexagonal Architecture rather than layers.
With it I would group stuff by purpose rather than by level of abstraction.


CASMINST-6790: Modify IMS import/export tools for Cray CLI S3 limitation

Description
When there are a large number of S3 artifacts in a bucket, the Cray CLI may truncate the listing. This problem was documented with CASMPET-6168 and fixed in CSM 1.5, but the problem still exists in prior CSM versions. This limitation also messes up the IMS import/export tools, as they rely on the Cray CLI for listing the contents of S3 buckets.
This PR changes these tools so that they use boto3 to list the bucket contents.
I currently only plan to backport this to CSM 1.3, since (as noted above) this is not a problem in CSM 1.5+, and the IMS tools only exist back to CSM 1.3.
I tested this PR on the UKMet TDSY system, which is where I discovered this issue.
Resolves CASMINST-6790

Checklist

[X] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[X] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[X] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.3


.

Coverage remained the same at 73.832% when pulling 65f177e17f25bbe15d76bfe8c9ec1a8deb21644a on hopeverse:master into 696241dc42ac8fe4eb0853cd1e1a0ef8399f2fbb on mediaelement:master.

Fix done. But I had issues grabbing your repo. So I added it on my side. Thanks


Can't disconnect and reconnect in Firefox

Using the example app with Firefox, if you connect then disconnect and then try to connect again you get errors:

You can connect, disconnect, and reconnect OK in Chromium and Edge.
I believe you used to be able to connect/disconnect/reconnect in Firefox previously. .Could be a code change or a Firefox update that has started this problem.
Note: Reloading the page doesn't work. Your have to close the browser tab and open a new one (or restart the browser, or restart the server).

I confirm that this has been fixed in current master.


Roll engine 8a8610a9a514..63873d9f421f (2 commits)

git@github.com:flutter/engine.git/compare/8a8610a9a514...63873d9f421f
git log 8a8610a9a514..63873d9f421f --no-merges --oneline
2019-09-16<EMAIL_ADDRESS>Fix objects equal to null not being detected as null (flutter/engine#11283)
2019-09-16<EMAIL_ADDRESS>Reset NSNetService delegate to nil，when stop service. (flutter/engine#11270)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


ignore any unconfigured key to reduce typing latency

is it possible to ignore any key that is not marked in Unshaky, or set to 0ms, to reduce a tiny bit of noticeable latency while typing which is added by Unshaky?

Unshaky only has a very minimal operation on keys with 0 delay. Do you have the debug window open? debug window will significantly slow down the Unshaky.

the typing is a little bit less responsive with Unshaky running, it probably won't be an issue for most people but i can definetly notice it.
I have a Macbook Pro 13" 2018 with a double spacing issue. i actually unchecked all other keys except for the spacebar in Unshaky's configuration. I don't have the debug window open...

The CGEventMask does not seem to support masking individual keys... so sorry unfortunately I don't think there is a way to do so...


Update required but it is not possible because it breaks ffmpeg

Describe the bug and how to reproduce it

Environment

@distube/ytdl-core version:
Node.js version:
Operating system:

I am running ytdl-core 4.15.8 and Distube 5.0.4. This is the error that I am getting:
[DEMIDO] -> WARNING: @distube/ytdl-core is out of date! Update with "npm install @distube/ytdl-core@latest".
WARNING: Could not parse n transform function.
Please report this issue with the "1734509067132-base.js" file on https://github.com/distubejs/ytdl-core/issues.

If I update ytdl-core I get this error:
811 |       this.emit("error", err);
812 |     }).on("exit", (code, signal) => {
813 |       this.debug(`[process] exit: code=${code ?? "unknown"} signal=${signal ?? "unknown"}`);
814 |       if (!code || [0, 255].includes(code)) return;
815 |       this.debug(`[process] error: ffmpeg exited with code ${code}`);
816 |       this.emit("error", new DisTubeError("FFMPEG_EXITED", code));
                                   ^
error: ffmpeg exited with code 8
 code: "FFMPEG_EXITED"

      at S:\Development\Demido\shell\node_modules\distube\dist\index.mjs:816:30
      at emit (node:events:48:48)
      at #handleOnExit (node:child_process:449:16)
      at processTicksAndRejections (native:7:39)

Bun v1.1.33 (Windows x64)
error: script "start" exited with code 1

https://github.com/distubejs/ytdl-core/issues/149

Okay, thank you very much!

DisTubeError [FFMPEG_EXITED]: ffmpeg exited with code 1
at ChildProcess. (/home/container/node_modules/distube/src/core/DisTubeStream.ts:134:28)
at ChildProcess.emit (node:events:519:28)
at ChildProcess.emit (node:domain:488:12)
at Process.ChildProcess._handle.onexit (node:internal/child_process:294:12) {
errorCode: 'FFMPEG_EXITED'
}


Add support for ingesting VEX documents in the CSAF format

Marking this as draft to check if I'm on the right path and waiting for a pr to go-vex to be reviewed and merged.
This adds a new parser for CSAF documents that and supporting ingestion of Vex documents. In addition it injects CertifyVulnIngest for known_affected or under_investigation statements as discussed in chat.
I'll fix/add a unit test for the parser as well.

Hey @lulf ! Thanks for opening this PR! in the meantime, if you need help with resolving the DCO or CLA, let us know!

Update: I'm still working on this, and it's almost done. I'm just awaiting a PR to be merged in go-vex and a release of that module. I have added some tests here in the PR as well and will rebase it soon.

Thanks @lulf for the update! We will review this soon!

Ping :)

@lumjjb @pxp928 I rebased the PR and it seems a lot comments are already addressed by https://github.com/guacsec/guac/pull/942. Can you give it another look and let me know if there's anything else we need to do before merging? And let's open issues for the future enhancements.

Great. #1015 and #1016 has beed added.

@lulf a rebase will auto merge this PR