Unable to use in a nextjs application

See https://github.com/0xFlicker/airstack-next-bug-repro for a reproducible case
yarn dev is fine, but yarn build will throw an error with a lot of text output, mainly because it includes some minified source code.
the example above is the quick start nextjs example with this library included
the build error can be fixed by commenting out the init function from this library

Hi @0xFlicker thanks
will check further later on
but briefly looking, looks like you're using it in a client environment instead of server-side, like under /app/api for nextjs, which is not something intended for the SDK as the SDK is purposely built for the Frames.
If you're looking for a client solution for using Airstack, recommend that you check out Airstack web SDK

it's in the app router, which runs server side and the same thing will happen in api route, e.g. https://github.com/0xFlicker/airstack-next-bug-repro/blob/main/src/app/frame/route.ts produces the same error


iOS 9 crash + Old version of ProMotion

Some of our users are reporting consistent crashes on iOS 9 (and these crashes are at different screens). Our application heavily uses ProMotion (PM::Screen PM::FormotionScreen PM::TableScreen PM::GroupedTableScreen) but we are on a old version - 1.2.1 as you can see below. Do you know if that could be a problem on iOS 9 and perhaps the reason for these intermittent crashes?
gem "afmotion", "2.5"
gem "bubble-wrap", "1.9.2", require: false
gem "cocoapods", "0.37.2" # BE CAREFUL CHANGING VERSION
gem "formotion", "1.8"
gem "motion-cocoapods", "1.7.1"
gem "motion-devices", "0.0.2"
gem "motion-dryclean", "0.0.2"
gem "motion-kit", "0.18.0"
gem "motion_model", "0.6.2"
gem "motion-support", "0.2.6", require: false
gem "ProMotion", "1.2.1"
gem "ProMotion-formotion", "0.1.6"
gem "rake", "10.4.2"
gem "RackMotion", "0.3"
gem "RedAlert", "1.3"
gem "ruby_motion_query", "1.6.1"
gem "sugarcube", "3.3.7", require: false

I do know that ProMotion, motion-support, and Formotion don't play all that well together. I've done quite a bit of work in the past to bring them in alignment but eventually gave up on motion-support and Formotion. If you have any crash backtraces or something it would certainly help!

The interesting thing is is this was all working before iOS 9, no issues at all. Ever since iOS 9, its been crashing.
Here's the crash report we got from one of our users, unfortunately the .dsym file was not saved for the build in the App store thats crashing. So not sure if this crash report will be of much help. https://gist.github.com/kapso/5e38ad73c32ac1b2340d
Thanks.

Also the crashes are only happening for some of the iOS 9 users, not all.

@jamonholmgren do you think upgrading to the latest ProMotion will fix these issues. Also would Infinite Red be willing to help with a project like this? Please let me know<EMAIL_ADDRESS>if this is something that you guys would be interested in.

@kapso Sure, I'll email you. It does sound like a separate issue. Just for future reference, what version of RubyMotion are you using?

@jamonholmgren great.
Also got some traces of the crash and theres a reference to ProMotion.

So 1.2.1 does not work on iOS 9 (or does not work on all iOS 9 devices). Upgraded to 2.4.2 and app stopped crashing.

For some reason I thought you'd already tried that. Glad it works now!

@kaspo was your issue fixed? I'm trying to help figure out which issues can be closed so let me know (and cc jamon) if so :+1:

Yea updating to the latest ProMotion version fixed the issue. Thanks for checking.


Overlay problem on rtl page

Hi.
Overlay not fixed on the map when the page direction is rtl.
To Reproduce
1- go to https://codepen.io/HRohani/pen/zYqXGrd
2- Set dir="rtl" for HTML tag.
3- Zoom in and out to see the problem.

I found simple solution.
set map container direction to ltr!
`

`


kaizen: Cleanup DFA/NFA

I am starting to implement the "suffix" pattern which obviously requires an NFA to implement (as does "shellstyle" and anything slightly regexp-ish). At the moment, Quamina reduces all NFAs to DFAs, which is well-known to be prone to producing O(2**N) explosions of the kind we're seeing in some tests (although it's not obvious this is the source of that problem).  Anyhow, the DFA/NFA confusion is adding lots of complexity. We should throw away DFAs and do everything in NFAs.  This would have a few benefits, including:

lose a lot of code
simplify the addPattern code path
de-genericize the SmallTable structure - this doesn't seem to be a good use of generics, lots of things are made harder to read and write.

Fixed in 8833629


Improve handling of caveats for API installs

Verification

[X] This issue's title and/or description do not reference a single formula e.g. brew install wget. If they do, open an issue at https://github.com/Homebrew/homebrew-core/issues/new/choose instead.

Provide a detailed description of the proposed feature
Currently, you can see different caveats depending on whether HOMEBREW_NO_INSTALL_FROM_API is set or not. For example, as I mentioned in #14925,
❯ diff -u <(HOMEBREW_NO_INSTALL_FROM_API= brew info socket_vmnet) <(HOMEBREW_NO_INSTALL_FROM_API=1 brew info socket_vmnet)
--- /dev/fd/13  2023-03-17 15:39:33.183107616 +0800
+++ /dev/fd/14  2023-03-17 15:39:33.183970272 +0800
@@ -16,6 +16,11 @@
 socket_vmnet is keg-only, which means it was not symlinked into /usr/local,
 because /usr/local/bin is often writable by a non-admin user.

+
+To restart socket_vmnet after an upgrade:
+  sudo brew services restart socket_vmnet
+Or, if you don't want/need a background service you can just run:
+  /usr/local/opt/socket_vmnet/bin/socket_vmnet --vmnet-gateway=<IP_ADDRESS> /usr/local/var/run/socket_vmnet
 ==> Analytics
 install: 116 (30 days), 664 (90 days), 773 (365 days)
 install-on-request: 116 (30 days), 662 (90 days), 771 (365 days)

What is the motivation for the feature?
Users should see caveats as intended regardless of whether they install using the API or from a Git clone.
How will the feature be relevant to at least 90% of Homebrew users?
90% of Homebrew users are installing, or will eventually install, from the API. They should see the same caveats as non-API installs.
What alternatives to the feature have been considered?
Removing caveats? Doing nothing?

Quoting @apainintheneck:

It looks like caveats are hardcoded when loaded from the API right now and that's what's probably causing the problem. Probably worth making a separate issue about that.
https://github.com/Homebrew/brew/blob/5550aa215e172f492cbcc137d64e79c57cb784c9/Library/Homebrew/formulary.rb#L266-L270

It looks like there is also at least one formula that uses Dir.home in caveats as well.

redshift.rb

Did this get fixed by adding the service block to the API? It seems like in this specific case it wasn't showing because it didn't know it was a service but now it should work fine.

This would be a one line change but do we want people to use Dir.home in caveats?

Seems reasonable to do so.

Did this get fixed by adding the service block to the API?

No idea, maybe!

I just double-checked and this works fine now because the service block was added to the API. Will close this out once the one line Dir.home substitution mentioned above is added for caveats.

want people to use Dir.home in caveats?

Why would caveats not just use ~?

Why would caveats not just use ~?

I think most of them do. It should be equivalent so I don't know why it can't be used instead.

@reitermarkus and I just found out that the caveats with on_* blocks and branching logic aren't getting serialized correctly with the API. This affects 57 different formulae.

Script: Prints out all caveats functions with branching logic or on_* blocks.
Results: All offending caveats functions
Affected Formulae List

@carlocab @apainintheneck is this now done?

on_* should certainly be working correctly, and it looks like it does... sometimes? e.g. gawk has a correct JSON but apcupsd does not.
Will take a look along with a few other API issues tomorrow.

I count ~70 affected formulae, but my script is a lot simpler so maybe there are a bunch of false positives.
#!/usr/bin/env brew ruby

require "formula"
require "method_source"

CoreTap.instance.formula_names.each do |name|
  formula = Formula[name]
  caveats_source = formula.method(:caveats).source

  first_method_line = caveats_source.lines.second.strip
  next if first_method_line == "nil"
  next if first_method_line.start_with?("<<~")

  puts name
end

Formulae
apcupsd
cdpr
cntlm
coreutils
csound
ed
emscripten
findutils
firefoxpwa
fontforge
freerdp
gawk
gdb
gnu-indent
gnu-sed
gnu-tar
gnu-time
gnu-units
gnu-which
golo
grep
haskell-language-server
inetutils
liboping
libtool
llnode
llvm
llvm@13
llvm@14
llvm@15
make
mercurial
mmseqs2
moc
mono
mysql
<EMAIL_ADDRESS>neko
neovim
nu
ocl-icd
open-tyrian
opencl-icd-loader
openconnect
openj9
openjdk
openjdk@11
openjdk@17
openjdk@8
percona-server
podman
postgresql@12
postgresql@13
postgresql@14
pymol
python-typing-extensions
pyyaml
ruby
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>samba
six
swift
synergy-core
util-linux
uutils-coreutils
vis
ykdl
you-get

Formulae with if branches etc will never be supported and a homebrew-core issue should be opened for those.
I do have a fix for on_os blocks not working when the only on_os block in the file is in caveats. Formulae which already use on_os blocks elsewhere should already be working for caveats.

Formulae with if branches or of other dynamic nature will never be supported and a homebrew-core issue should be opened for those.

Yeh, and we should probably have an audit etc. too.


CSV manipulation utiities

@ybasket If you can take a minute or two to review my generic implementation, it would be nice.


No valid trust anchor

Hello everyone, I am having a problem with the configuration of Routinator.
I installed it on a machine in my virtual network and gave the following command:
routinator init --rir-tals --accept-arin-rpa
The response is:
Created local repository directory /root/.rpki-cache/repository
Installed 5 TALs in /root/.rpki-cache/tals
When I try to get the ROAs with the command routinator -v vrps I get the problem:
No valid trust anchor for TAL afrinic
No valid trust anchor for TAL apnic
No valid trust anchor for TAL arin
No valid trust anchor for TAL lacnic
No valid trust anchor for TAL ripe
ASN,IP Prefix,Max Length,Trust Anchor
What could be the problem?
Thanks.

Can you add another -v to your command to get more output? (I.e., routinator -vv vrps). My guess would be that you have networking issues, which the extra -v should expose.

It was a problem of DNS, when properly setted routinator works in the correct way.
Thank you.


Add code formatting instruction for catkin

Coverage remained the same at 79.305% when pulling 198f0fe1f979cc75a1ed9f8d6a4dce3648406a57 on doc/style into 84659d1f253895bb62916f78c5aaaa2a4020b188 on master.


chore(deps): update minor dependencies (main)

Proposed change

Related issues

:bug: Fixes #(issue)
:rocket: Feature #(issue)

Duplicate of https://github.com/AmadeusITGroup/otter/pull/1275


Diff Editor - Collapsed Unmodified Ranges: Don't uncollapse on revert?

When reverting, the collapsed range gets uncollapsed:

/duplicate #187844


Playerlist Bug

There is one bug in the playerlist. There is "me" (localplayer) in the playerlist.

are you serious?
do you think there's a bot with your name when you open the scoreboard or after every kill you get too?

every cheat i've used with a player list has shown the local player in it.


iphone 6s plus unable to jailbreak right after restarting

Describe the bug
I'm using unc0ver 4.3.1. My 6s plus after restarting, was unable to jailbreak right away. The phone will just go through another restart. Doesn't matter how many times I tried restarting or hard restart, it is still impossible. The only solution is to unlock the device after restarting and let the device idling for around 10mins, then the jailbreak can be successful.
To Reproduce
Steps to reproduce the behavior:

Shutdown the device
Turn on the device
Open Unc0ver right away
Press Jailbreak
See the device restart again

Screenshots
The device automatically restarts, so it's impossible to get screenshot
Device (please complete the following information):

iOS Version: 13.1.3
iOS Device: Iphone 6s plus
unc0ver Version: 4.3.1

Place an "x" between the brackets if true:

[x] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[ ] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs

This is an issue with the exploit itself, not an issue with unc0ver. It is normal to have to idle for a short period of time before the exploit actually can work (based off my experience).

@SooOverpowered how long is it at the apple logo after you jailbreak? Does it go black instantly and show the logo? Or is the a spinning wheel first?

It goes black instantly and show the logo

@SooOverpowered can you go into dfu mode, then come out of dfu, then try jailbreaking again?

Duplicate of #1844


stack trace implementation for MACH-O

Currently if you cause a safety violation in an ELF binary, zig gives you a nice stack trace. Not so for MACH-O binaries.
It appears to still be DWARF information, so that code could be reused, but it's not stored directly in the binary - there's some kind of mapping from binary to .o files where the actual debug info is stored.

oops, this should not have been closed. text reads "partially fixes" not "fixes".

#1365


working on fixing unknown

Trying to add in an "unknown que" that gets checked every time a new unknown person might be added to the screen so that way the "unknown" category doesn't get flooded with images when running the web app

can you rename this to queue?

ryan ... queue


CorsHandler not responding with Content-Length

Expected behavior

CorsHandler responds with Content-Length: 0 when origin is allowed
CorsHandler responds with Content-Length: 0 when origin is not allowed

Actual behavior

CorsHandler responds with Content-Length: 0 when origin is allowed
CorsHandler  does not respond with a Content-Length header when origin is not allowed

Steps to reproduce

Run an NioEventGroupLoop with a CorsHandler in the pipeline. (see code)
Make a request from an allowed origin, notice Content-Length: 0 in the response. This is expected. (see code)
Make a request from an origin that is not allowed, notice no Content-Length header in the response. This is unexpected. (see code)

Minimal yet complete reproducer code (or URL to code)
Server side:
import io.netty.bootstrap.ServerBootstrap;
import io.netty.channel.*;
import io.netty.channel.nio.NioEventLoopGroup;
import io.netty.channel.socket.nio.NioServerSocketChannel;
import io.netty.handler.codec.http.HttpMethod;
import io.netty.handler.codec.http.HttpObjectAggregator;
import io.netty.handler.codec.http.HttpServerCodec;
import io.netty.handler.codec.http.cors.CorsConfigBuilder;
import io.netty.handler.codec.http.cors.CorsHandler;
import io.netty.handler.logging.LogLevel;
import io.netty.handler.logging.LoggingHandler;

/**
 * Runs netty with a CORS handler on 8080
 */
public class NettyCorsBugApp {
    private static final int PORT = 8080;
    private static final String ALLOWED_ORIGIN = "http://example.com";

    public static void main(String[] args) throws Exception {
        EventLoopGroup eventLoopGroup = new NioEventLoopGroup();
        try {
            ServerBootstrap bootstrap = new ServerBootstrap()
                    .group(eventLoopGroup)
                    .handler(new LoggingHandler(LogLevel.INFO))
                    .childHandler(new ChannelInitializer<Channel>() {
                        @Override
                        protected void initChannel(Channel ch) throws Exception {
                            ChannelPipeline pipeline = ch.pipeline();
                            pipeline.addLast(new HttpServerCodec());
                            pipeline.addLast(new HttpObjectAggregator(1024 * 1024)); // 1MB
                            pipeline.addLast(new CorsHandler(
                                    CorsConfigBuilder.forOrigin(ALLOWED_ORIGIN)
                                            .allowedRequestMethods(HttpMethod.POST)
                                            .build())
                            );
                        }
                    })
                    .channel(NioServerSocketChannel.class);
            Channel channel = bootstrap.bind(PORT).sync().channel();
            channel.closeFuture().sync();
        } finally {
            eventLoopGroup.shutdownGracefully();
        }
    }
}

Client side from an allowed origin. Notice how curl exits.
$ curl -sv -X OPTIONS -H 'Origin: http://example.com' -H 'Access-Control-Request-Method: POST' http://localhost:8080
* Rebuilt URL to: http://localhost:8080/
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
> OPTIONS / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.54.0
> Accept: */*
> Origin: http://example.com
> Access-Control-Request-Method: POST
>
< HTTP/1.1 200 OK
< access-control-allow-origin: http://example.com
< vary: origin
< access-control-allow-methods: POST
< access-control-allow-headers:
< access-control-max-age: 0
< date: "Tue, 26 Sep 2017 20:03:53 GMT"
< content-length: 0
<
* Connection #0 to host localhost left intact

Client side from an origin that is not allowed. Notice how curl does not exit.
$ curl -sv -X OPTIONS -H 'Origin: http://invalid.com' -H 'Access-Control-Request-Method: POST' http://localhost:8080
* Rebuilt URL to: http://localhost:8080/
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
> OPTIONS / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.54.0
> Accept: */*
> Origin: http://invalid.com
> Access-Control-Request-Method: POST
>
< HTTP/1.1 200 OK
* no chunk, no close, no size. Assume close to signal end
<

Netty version
Tested 4.1.6-Final and 4.1.16.Final, same behavior.
JVM version (e.g. java -version)
java version "1.8.0_131"
Java(TM) SE Runtime Environment (build 1.8.0_131-b11)
Java HotSpot(TM) 64-Bit Server VM (build 25.131-b11, mixed mode)

OS version (e.g. uname -a)
Darwin WF14616 16.7.0 Darwin Kernel Version 16.7.0: Thu Jun 15 17:36:27 PDT 2017; root:xnu-3789.70.16~2/RELEASE_X86_64 x86_64

Nevermind, I read more about http 1.0 vs 1.1 and keep-alive vs close connections and see this behavior is intended. The reason the connection was closing for the valid origin was because it adds Content-Length: 0.
https://github.com/netty/netty/commit/ecd6e5ce6d82756475e88ae69d2bfef1082edb68#diff-9cc075ed08ebc635ad5afcdb82bd449b
Here is the proper way to test:
$ curl -sv -X OPTIONS -H 'Origin: http://invalid.com' -H 'Access-Control-Request-Method: POST' -H 'Connection: close' http://localhost:8080
* Rebuilt URL to: http://localhost:8080/
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
> OPTIONS / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.54.0
> Accept: */*
> Origin: http://invalid.com
> Access-Control-Request-Method: POST
> Connection: close
>
< HTTP/1.1 200 OK
< connection: close
<
* Closing connection 0

$ curl -sv -X OPTIONS -H 'Origin: http://example.com' -H 'Access-Control-Request-Method: POST' -H 'Connection: close' http://localhost:8080
* Rebuilt URL to: http://localhost:8080/
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 8080 (#0)
> OPTIONS / HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.54.0
> Accept: */*
> Origin: http://example.com
> Access-Control-Request-Method: POST
> Connection: close
>
< HTTP/1.1 200 OK
< access-control-allow-origin: http://example.com
< vary: origin
< access-control-allow-methods: POST
< access-control-allow-headers: Authorization,Ot-Tracer-Spanid,Accept,Content-Transfer-Encoding,Ot-Tracer-Traceid,Content-Type,Ot-Tracer-Sampled
< access-control-allow-credentials: true
< access-control-max-age: 0
< cache-control: no-store
< pragma: no-cache
< connection: close
<
* Closing connection 0

@normanmaurer This looks related to https://github.com/netty/netty/commit/ecd6e5ce6d82756475e88ae69d2bfef1082edb68#diff-9cc075ed08ebc635ad5afcdb82bd449b I think we just need to add the Content-Length header for all cases, including when consumers use preflightResponseHeader() too.

Pull request for the fix, including unit tests: https://github.com/netty/netty/pull/7261

Fixed by https://github.com/netty/netty/pull/7261


Incorrect work of pact-mock_server

@mefellows @uglyog
I'm using pact-consumer-js-dsl to generate pacts. But found interesting behaviour.
Currently I have 4 pairs of consumer-providers. I run gulp task to run the test cases with pact-dsl to generate them and they all go to the same instance of running pact_mock server. I start it before the tests run.
As a result I have 4 jsons with correct names,. The first json lists correctly the pair C1-P1 and C1P1 interactions but the second json lists C2-P2 and interaction C1P1 + C2P2. Where C = consumer, P = provider. And so on. Last json has all interactions in it but with C4P4.
How can this be overcome? I need json files with interactions specific for specific C-P pair only.

Hi @ichyr, it sounds like your problem is that you are running the mock service between each C-P pair. I'm no expert on the Mock Service, however as I understand it you should start the service for each combination of C-P. Have you tried this approach?

@mefellows But this is more or less not very efficient. I'm developing Angular.js front-end application that consumed data from microservice infrastructure ( thus one consumer with multiple providers ). I have moved all my pact generation jasmine specs into seperate guild job, but they still run in one batch.
How should this be resolved?
In my opinion this is the way it is supposed to be - run all pact generation script at once.
But current implementation implies that the pact_mock-server should be fired before and shut down after each batch of test with same C-P pair. This is a mess.

@mefellows  @uglyog
maybe you already have some patterns to solve problems like mine?

We are looking at ways of fixing this, probably by creating a standard mock server for all language implementations. Unfortunately, this won't be something that can be fixed in the short term.


[feature] moon generate compute destination

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Often when generating code, I would like moon to force the destination of the code, to keep the repo in harmony. Let's say I have a template "rustapp", it would be nice to have an option for the template to provide a computed destination (like apps/rust/<project-id> for example).
This would be configurable from the template definition, as any other variables.
Describe alternatives you've considered

No alternative for now
Additional context

Will land in v1.19 https://github.com/moonrepo/moon/pull/1252


Support for structured logging (JSON)

Problem:

At the moment only the standard env_logger based logging is supported. In some scenarios it would be beneficial, if Oura would support structured logging as well, in an easy to parse JSON format.

For a more detailed description about the structured logging format, please see the Google Cloud Documentation:

Google Cloud Guides - Structured logging

Suggestion:

Add a feature flag for the configuration of the logging method. Depending if the json_logs feature is set, Oura could be using json_env_logger2 or env_logger.

I have created a PR with the proposed changes:

https://github.com/txpipe/oura/pull/438


Error updating Nuget Package Explorer

Hi! Thanks for reporting this feature/bug/question!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
Type (choose one):

Bug
Feature request
Question

NPE version: (e.g. 3.12) (see Help -> about)
Installed from: ClickOnce / chocolatey.org / Zip on CodePlex
In case of a BUG:

What is the current result?
What is the expected result?
Please post full exception details in case of an Exception (message, stacktrace, inner exceptions)
Are there any work arrounds? yes/no
Is there a version in which it did worked?
Is there a package on Nuget.org which shows the bug?

in case of a FEATURE REQUEST:

Why do we need it?
Please provide an use case.

Try uninstall and install fresh


No shadow for navbar with react navigation

When using react-navigation with this component, the navbar doesn't have a shadow:

Hi @RWOverdijk!
Sorry for the late response. I think it depends by the composition model you use. Can you provide some code, please?


@/charterafrica/Search

Implement Search input component per designs.

Please refer to @/codeforafrica/Search component as reference. If reusable, move @/codeforafrica/Search to @commons-ui/core/Search and reuse it in both apps.


[Runtime Bug]: Falcon 4: Allied Forces - RTX hooks, but does nothing

Describe the bug
The game can be nicely converted and playable to DX9 with dxwrapper. Next step is to inject RTX-Remix.
RTX UI comes, but no raytracing happens, also FPS drops to 2-4. Also, "Capture scene" button does nothing. No error message no nothing.
How do you reproduce the bug?
Basically just running the game.
What is the expected behavior?
It should allow me to capture the scene so I can start working on F16 cockpit models and retexturing the whole game. Would be really awesome.
Version
x.x.x
Logs
d3d9.log
FalconAF_d3d9.log
Crash dumps
No response
Media

Thanks for opening a ticket! We've created REMIX-3697 to investigate the issues internally. In the meantime, we suggest reaching out to the Discord community to see if anyone has had success with this title - there may be engine-side tweaks that help.

Hello!
Many thanks - I have already tried discord, but nobody seems to be working on this game other than myself :) Also created an entry in the projects sub channel, to record my activities.
In case you need access to this game it can be downloaded from here:
https://www.myabandonware.com/game/falcon-4-0-allied-force-e53
Also the latest patch is needed:
https://www.patches-scrolls.de/patch/1592/7/43294/download


swagger.json generated by the example has errors

Swagger generation doesn't seem to work correctly. The generated swagger.json contains may errors from the example project.
In order to debug this problem, I eliminated all the resources except for the people.js resource. In addition, I got rid of references to the other resources in the people.js file.
I then used the swagger editor to fix the problems found in the resulting swagger.json
I am providing the links to my people.js file : https://drive.google.com/file/d/0BzXg1kzQ457OeHpUMVhtVDk2MEk/view?usp=drivesdk
Here is the swagger.json file that was generated: https://drive.google.com/file/d/0BzXg1kzQ457ONDlVZUZZOElZdUE/view?usp=drivesdk
Here is the swagger.json file that I fixed using the Swagger editor: https://drive.google.com/file/d/0BzXg1kzQ457ORVdpdGlQRU90bVk/view?usp=drivesdk

Thanks for reporting this, I'll take a look and sort it out :+1:

Here is one fix, but there are more:
lib/swagger/path.js line 119
required: [ "jsonapi", "meta, links" ],
Should be
required: [ "jsonapi", "meta", "links" ],

I'm going to investigate adding another step to the CI to validate the swagger output on each build, just in case we introduce any regressions in the future.

If you do a diff on these two files, you will see where all the changes that I made to fix my swagger.json.
In my swagger.json, there are 4 error types and 1 warning which are repeated over and over. So even though I had to make a lot of changes, there are really only 4 bugs and 1 warning.
I am adding these files
swagger_json_broken.txt
swagger_json_fixed.txt
because there is additional errors that you would not see in my first report.

Thanks @jbockerstette - I think I've fixed all of the bugs, and cleaned up the required attributes. There's also a CI build task that will validate the generated swagger docs, which should help prevent this from happening again.

You're awesome. Great job. Is there some way to configure the required
attributes so they only show up if you want them?
On May 21, 2016 3:52 AM, "Oliver Rumbelow"<EMAIL_ADDRESS>wrote:

Thanks @jbockerstette https://github.com/jbockerstette - I think I've
fixed all of the bugs, and cleaned up the required attributes. There's
also a CI build task that will validate the generated swagger docs, which
should help prevent this from happening again.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/holidayextras/jsonapi-server/issues/129#issuecomment-220769131

Thanks :)
The required attributes come from the Joi schema in the resource definition, for example in https://github.com/holidayextras/jsonapi-server/blob/master/example/resources/articles.js we have:
attributes: {
    title: jsonApi.Joi.string().required()
      .description("The articles title, should be between 8 and 15 words")
      .example("Learning how to use JSON:API"),
}

Notice the .required() - the generated Swagger document will require those attributes marked as required. You can tag attributes however you like, and all models passing through to your handlers are validated against the schema.

Ok. I was playing with that but no matter what I tried, it always put the
required flag in my swagger. I will try again now that you have fixed it.
Thanks
On May 22, 2016 11:01 AM, "Oliver Rumbelow"<EMAIL_ADDRESS>wrote:
Thanks :)
The required attributes come from the Joi schema in the resource
definition, for example in
https://github.com/holidayextras/jsonapi-server/blob/master/example/resources/articles.js
we have:
attributes: {
title: jsonApi.Joi.string().required()
.description("The articles title, should be between 8 and 15 words")
.example("Learning how to use JSON:API"),
}
Notice the .required() - the generated Swagger document will require those
attributes marked as required. You can tag attributes however you like, and
all models passing through to your handlers are validated against the
schema.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/holidayextras/jsonapi-server/issues/129#issuecomment-220843377

The changes have been merged and pushed to NPM as v1.7.0 :+1:


Update PDFRenderer.java to fix scaling for image

fix issue #97 scaling to image did not work

Merged in commit https://github.com/TomRoush/PdfBox-Android/commit/c375908be6ef3c20600f1ac89bd630578c7844b2. Thank you for the fix. Closes #97


NoClassDefFoundError API 10

From Android Arsenal the minimumSDK is indicated 9.
http://android-arsenal.com/details/1/1240
For higher versions it works, but for api 10 i get this error.
java.lang.NoClassDefFoundError: it.sephiroth.android.library.tooltip.Tooltip$TooltipViewImpl$1

Minimum supported SDK is 14. I changed the requirement a while ago.


Increase consistency, and style guide compliance

not using object after API objects per https://kubernetes.io/docs/home/contribute/style-guide/#use-camel-case-for-api-objects

This change is

/assign @steveperry-53

@lrakai Thanks for doing this.
/approve

Still need an LGTM for technical accuracy. In particular, we need to resolve the question raised by @tengqm.

Fwiw, a different commit removed the line that I believe was in question.  I merged it in with my changes.

@lrakai Sounds good to me. Just to follow procedure, I'll let @tengqm apply the LGTM.

/lgtm


[BACKEND] Support bf16 global atomic add on Hopper and Ampere

This patch adds support for bf16 global atomic adds on Hopper. If the check for Hopper (sm_90) fails, then a atomic compare and swap pattern is produced (as is the case with the cuda C library's atomicAdd().
For the <sm_90 case, lowering code obtained from the ROCm fork of Triton is used to produce the necessary pattern (which is actually handled by the NVPTX LLVM backend for LLVM IR atomicrmw fadd ptr addrspace(1) %ptr, bfloat %val). The ROCm matchAndRewrite method works in this case because it is generating a generic LLVM IR instruction.
The patterns produced for each Target are as follows:
Hopper:
@%p1 atom.global.gpu.acq_rel.add.noftz.bf16 %rs1, [ %rd1 + 0 ], %rs2;
Ampere:
  $BB:
  shr.u32               %r1, %r2, %r3;
  cvt.u16.u32           %rs1, %r1;
  add.bf16              %rs2, %rs1, %rs1;
  cvt.u32.u16           %r4, %rs2;
  shl.b32               %r5, %r4, %r3;
  and.b32               %r6, %r2, %r9;
  or.b32                %r7, %r6, %r5;
  atom.global.cas.b32   %r8, [%rd1], %r2, %r7;
  setp.ne.s32           %p1, %r8, %r2;
  mov.u32               %r2, %r8;
  @%p1 bra              $BB;

Still experimenting with some things here, will add test cases soon.
@htyu

Hi, thank you for your patch!
Can you clarify, is this ready to be reviewed (other than tests), or are you still working on it?
In any case, I left a few preliminary comments.

Thanks for the fast comments!
It is ready for review but please note that the fallback path is based on code cherry-picked from https://github.com/ROCmSoftwarePlatform/triton. I'd like to leave that bit of the code alone so that there's as little difference as possible, and I also would like to experiment with possibly moving atomics off of inline ptx assembly in the future too (this would require better much better lowering on the llvm backend side, I suspect in isel).

Got it, thanks for the info!
I don't think I'm comfortable copy-pasting code from the ROCm fork without cleaning it up to "make sense" in the context of mainline.  It's no different than copying code from a random other project.

for hardware that doesn't support it natively, the codegen seems unnecessarily specific. Maybe we could talk about atomics that take arbitrary binary functions (although I'd need to see a compelling use case), but baking that in the compiler for standard binary operators seems like a bad idea.

Does also this mean that it generally it isn't  a good idea to replace inline ptx with llvm irgen or that in this case it just doesn't make sense?

I think in this case it is too specific. I think we could potentially allow AtomicRMW to take in an arbitrary associative operator instead of a hardcoded op flag, although I'd have to see a better use case to justify the added complexity

I think in this case it is too specific. I think we could potentially allow AtomicRMW to take in an arbitrary associative operator instead of a hardcoded op flag, although I'd have to see a better use case to justify the added complexity

Actually we had to work it around in Pytorch (https://github.com/pytorch/pytorch/pull/113204/files#r1387160822) and it'd be nice if it can be supported in Triton.

fp16 atomics are barely useful already, and bf16 has even fewer mantissa bits

The main place where TorchInductor will generate atomics is the backwards of an indirect load (e.g. embeddings and graph neural networks).  Embeddings seem pretty common, so it would be nice fp16/bf16 embeddings to be supported.

fp16 atomics are barely useful already, and bf16 has even fewer mantissa bits

The main place where TorchInductor will generate atomics is the backwards of an indirect load (e.g. embeddings and graph neural networks). Embeddings seem pretty common, so it would be nice fp16/bf16 embeddings to be supported.

Agree, it's not only graph neural networks, some kernels like SplitK GEMM also require atomic accumulation, it's generally good to see the BF16 atomic add support for wider user case


OpenAI Node v4 improperly polyfills fetch

Describe the bug
Using this library<EMAIL_ADDRESS>(and confirmed on a few other beta versions) improperly polyfills fetch with node-fetch, which has different semantics than the fetch in Node v18 and above.
This is one of the causes of this issue:

https://github.com/vercel-labs/ai/issues/199

The type error looks like this:
TypeError: responseBodyStream.pipeThrough is not a function
    at AIStream (file:///app/node_modules/ai/dist/index.mjs:135:29)
    at Module.OpenAIStream (file:///app/node_modules/ai/dist/index.mjs:172:18)
    at renderCompletion (file:///app/index.js:19:34)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async file:///app/index.js:33:3

And it's due to node-fetch not behaving idiomatically.
To Reproduce
Proof of concept repo
Clone https://github.com/AaronFriel/openai-vercel-ai-bug
Run node ./index.js, you should see:
$ node index.js
Catching 💥
TypeError: responseBodyStream.pipeThrough is not a function
    at AIStream (file:///app/node_modules/ai/dist/index.mjs:135:29)
    at Module.OpenAIStream (file:///app/node_modules/ai/dist/index.mjs:172:18)
    at renderCompletion (file:///app/index.js:19:34)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async file:///app/index.js:33:3
💥 caught!
Should the OpenAI library polyfill fetch, or should it use the global fetch on Node v18 and above?

The OpenAI library should use the global fetch on Node v18 and above, rather than polyfilling fetch.

On Next.js
1. Configure a next.js app
Create an app or copy the app dir here: https://github.com/vercel-labs/ai/blob/main/examples/next-openai
2. Create a server route handler for OpenAI
In a route handler, instantiate an OpenAI client and return a stream. Do not set runtime = 'edge' - we want to see how this behaves on Node.js on the server.
(Modified from https://github.com/vercel-labs/ai/blob/main/examples/next-openai/app/api/chat/route.ts)
// ./app/api/chat/route.ts
import { Configuration, OpenAIApi } from 'openai-edge'
import { OpenAIStream, StreamingTextResponse } from 'ai'

// Create an OpenAI API client (that's edge friendly!)
const config = new Configuration({
  apiKey: process.env.OPENAI_API_KEY
})
const openai = new OpenAIApi(config)

// IMPORTANT! DO NOT SET THE RUNTIME
// export const runtime = 'edge'

export async function POST(req: Request) {
  // Extract the `prompt` from the body of the request
  const { messages } = await req.json()

  // Ask OpenAI for a streaming chat completion given the prompt
  const response = await openai.createChatCompletion({
    model: 'gpt-3.5-turbo',
    stream: true,
    messages: messages.map((message: any) => ({
      content: message.content,
      role: message.role
    }))
  })

  // Convert the response into a friendly text-stream
  const stream = OpenAIStream(response)
  // Respond with the stream
  return new StreamingTextResponse(stream)
}

3. Observe a TypeError on using the route
TypeError: responseBodyStream.pipeThrough is not a function
    at AIStream (file:///app/node_modules/ai/dist/index.mjs:135:29)
    at Module.OpenAIStream (file:///app/node_modules/ai/dist/index.mjs:172:18)
    at renderCompletion (file:///app/index.js:19:34)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async file:///app/index.js:33:3

Code snippets
import * as ai from 'ai';
import { OpenAI } from 'openai';

import * as fs from 'fs/promises';

async function renderCompletion(client, content) {
  const res = await client.chat.completions.create({
    model: 'gpt-3.5-turbo',
    stream: true,
    messages: [
      {
        role: 'system',
        content:
          'You are an expert software engineer, with deep Node.js and web development experience. You are succinct, brief, and to the point.',
      },
      {
        role: 'user',
        content,
      },
    ],
  });

  for await (const message of ai.OpenAIStream(res.response)) {
    process.stdout.write(message);
  }
  process.stdout.write('\n');
}

try {
  // This API client uses the node-fetch polyfill:
  const brokenClient = new OpenAI({
    apiKey: process.env.OPENAI_API_KEY,
  });

  console.log('Catching 💥');
  await renderCompletion(brokenClient, 'PING');
} catch (error) {
  console.log(error);
  console.log(`💥 caught!`);

  // This API client uses the built-in fetch support in Node v18:
  const workingClient = new OpenAI({
    apiKey: process.env.OPENAI_API_KEY,
    fetch: globalThis.fetch,
  });

  const currentSourceCode = await fs.readFile('./index.js', 'utf-8');
  const question = `Should the OpenAI library polyfill fetch, or should it use the global fetch on Node v18 and above?`;
  const prompt = `
\`\`\`javascript
${currentSourceCode}
\`\`\`

That code threw this error, as a result of the OpenAI API client named \`brokenClient\`.

The \`workingClient\` however works.

\`\`\`
${error}
\`\`\`

${question}
`;
  console.log(`${question}\n\n`);
  await renderCompletion(workingClient, prompt);
}

OS
Linux
Node version
v18.16.1
Library version
openai v4.0.0-beta.5

For an opinionated perspective:
https://www.builder.io/blog/stop-polyfilling-fetch-in-your-npm-package

Thanks for opening this!
We intentionally use node-fetch on Node instead of the built-in fetch, as it's still in Experimental stability, and (I believe) does not yet support connection pooling.
As I'm sure you noticed, the critical line new ai.OpenAIStream(res.response) is where the problem arises, and this library actually deprecates res.response; it'll be removed before this package leaves beta, partly due to these sorts of incompatibilities.
Vercel plans to add support for you to simply pass res directly here, so it'll look like this: new ai.OpenAIStream(res) – and they'll read from our stream's async iterator directly, rather than from the raw Response, which should address your use-case. I'll bump this issue with their team.
For posterity (if others come here with similar issues), as you noted in the issue, as a workaround you can pass the global fetch function like so:
const client = new OpenAI({
  fetch: globalThis.fetch,
});


@conda-forge-admin, please add user @rxm7706

Additional comment:
No response

Hi @rxm7706 , I'm wondering if you could help with the https://github.com/conda-forge/opentelemetry-instrumentation-flask-feedstock/pull/22 ? Not sure whether you have permissions to modify this yet due to https://github.com/conda-forge/opentelemetry-sdk-feedstock/issues/34 . Thank you!

Hi @sy-bee - I don't have permissions yet - I have pinged for help.

@sy-bee versions 0.39b0, 0.40b0, .0.41b0, 0.42b0, 0.43b0 are available now. You should be gtg.
https://anaconda.org/conda-forge/opentelemetry-instrumentation-flask/files
Thank you @hmaarrfk !

Thank you so much @rxm7706 and @hmaarrfk !


no H.264 format video track:

Hey,
Im currently getting this error when testing out convert-video. Not sure why. Any suggestions?
Just to confirm the video track is H264

@dgibbs64 You're getting that error because the video your'e trying to convert doesn't contain any H.264 format video track. Very likely you're trying to use convert-video on a DVD rip with MPEG-2 format video or on a Blu-ray rip with VC-1 format video. Or you're simply trying to use convert-video on a non-video file.
What does mediainfo tell you about the contents of the file you're trying to convert?
You can get the mediainfo tool via Homebrew with a brew install media-info command.

@dgibbs64 Did you ever figure out what's going on here? Did the media file actually contain H.264 video?

yeah the media did contain H.264. I will need to investigate again as I have been busy and not been able to test this in a while. I will look at this again once I have settled in to my new house in a few weeks :)

@dgibbs64 Any update on this?

Will be doing a fresh install once I get my VMware server installed :D. I will test then

Just setting up my new VM I can confirm I still have the issue with an mkv file that does contain a H.264 track.

here is a sample output from mediainfo
Format                                   : Matroska
Format version                           : Version 4 / Version 2
File size                                : 22.7 MiB
Duration                                 : 1mn 1s
Overall bit rate                         : 3 075 Kbps
Encoded date                             : UTC 2014-09-06 03:22:13
Writing application                      : mkvmerge v7.0.0 ('Where We Going') 64bit built on Jun 22 2014 08:19:39
Writing library                          : libebml v1.3.0 + libmatroska v1.4.1
Video
ID                                       : 1
Format                                   : AVC
Format/Info                              : Advanced Video Codec
Format profile<EMAIL_ADDRESS>Format settings, CABAC                   : Yes
Format settings, ReFrames                : 5 frames
Codec ID                                 : V_MPEG4/ISO/AVC
Duration                                 : 1mn 1s
Bit rate                                 : 2 630 Kbps
Width                                    : 1 280 pixels
Height                                   : 720 pixels
Display aspect ratio                     : 16:9
Frame rate mode                          : Constant
Frame rate                               : 23.976 (24000/1001) fps
Color space                              : YUV
Chroma subsampling                       : 4:2:0
Bit depth                                : 8 bits
Scan type                                : Progressive
Bits/(Pixel*Frame)                       : 0.119
Stream size                              : 19.4 MiB (86%)
Writing library                          : x264 core 142 r2479 dd79a61
Encoding settings                        : cabac=1 / ref=5 / deblock=1:0:0 / analyse=0x3:0x113 / me=umh / subme=8 / psy=1 / psy_rd=1.00:0.00 / mixed_ref=1 / me_range=16 / chroma_me=1 / trellis=1 / 8x8dct=1 / cqm=0 / deadzone=21,11 / fast_pskip=1 / chroma_qp_offset=-2 / threads=18 / lookahead_threads=3 / sliced_threads=0 / nr=0 / decimate=1 / interlaced=0 / bluray_compat=0 / constrained_intra=0 / bframes=3 / b_pyramid=2 / b_adapt=2 / b_bias=0 / direct=3 / weightb=1 / open_gop=0 / weightp=2 / keyint=250 / keyint_min=23 / scenecut=40 / intra_refresh=0 / rc_lookahead=50 / rc=crf / mbtree=1 / crf=18.0 / qcomp=0.60 / qpmin=0 / qpmax=69 / qpstep=4 / ip_ratio=1.40 / aq=1:1.00
Language                                 : English
Default                                  : Yes
Forced                                   : No
Color range                              : Limited
Matrix coefficients                      : BT.709

Audio
ID                                       : 2
Format                                   : AC-3
Format/Info                              : Audio Coding 3
Mode extension                           : CM (complete main)
Format settings, Endianness              : Big
Codec ID                                 : A_AC3
Duration                                 : 1mn 1s
Bit rate mode                            : Constant
Bit rate                                 : 384 Kbps
Channel(s)                               : 2 channels
Channel positions                        : Front: L R
Sampling rate                            : 48.0 KHz
Frame rate                               : 31.250 fps (1536 spf)
Compression mode                         : Lossy
Stream size                              : 2.83 MiB (12%)
Default                                  : Yes
Forced                                   : No

@dgibbs64 Thanks. Is it possible to upload this file somewhere so I could download and test it?

@dgibbs64 If uploading is not possible, can you run this command on the input and paste the output here:
HandBrakeCLI --title 0 --scan --previews 2:0 --input "/path/to/Movie.mkv"

@dgibbs64 BTW, the reason why I need the output of a HandBrake scan is that's what the convert-video tool (or any of my tools) uses to detect a H.264 video stream.

@dgibbs64 Any chance to try that HandBrakeCLI command yet?

[19:45:40] CPU: Intel(R) Core(TM) i3-3220 CPU @ 3.30GHz
[19:45:40]  - Intel microarchitecture Ivy Bridge
[19:45:40]  - logical processor count: 4
[19:45:40] OpenCL: library not available
[19:45:40] hb_scan: path=/home/dgibbs/nas-Downloads/convert-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv, title_index=0
udfread ERROR: ECMA 167 Volume Recognition failed
disc.c:269: failed opening UDF image /home/dgibbs/nas-Downloads/convert-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv
disc.c:350: error opening file BDMV/index.bdmv
disc.c:350: error opening file BDMV/BACKUP/index.bdmv
[19:45:48] bd: not a bd - trying as a stream/file instead
libdvdnav: Using dvdnav version 5.0.3
libdvdread: Encrypted DVD support unavailable.

**  No css library available. See             **
**  /usr/share/doc/libdvdread4/README.css     **
**  for more information.                     **

libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.IFO failed
libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.BUP failed
libdvdread: Can't open file VIDEO_TS.IFO.
libdvdnav: vm: failed to read VIDEO_TS.IFO
[19:45:50] dvd: not a dvd - trying as a stream/file instead
Input #0, matroska,webm, from '/home/dgibbs/nas-Downloads/convert-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv':
Metadata:
encoder         : libebml v1.3.0 + libmatroska v1.4.1
creation_time   : 2014-08-23 03:01:46
Duration: 00:01:08.42, start: 0.000000, bitrate: 2188 kb/s
Stream #0:0(eng): Video: h264 (High), yuv420p(tv, bt709/unknown/unknown), 1280x720, SAR 1:1 DAR 16:9, 23.98 fps, 23.98 tbr, 1k tbn, 47.95 tbc (default)
Stream #0:1: Audio: ac3, 48000 Hz, 5.1(side), fltp, 384 kb/s (default)
[19:45:54] scan: decoding previews for title 1
Scanning title 1 of 1, preview 1, 0.00 %[19:45:54] scan: audio 0x1: ac3, rate=48000Hz, bitrate=384000 Unknown (AC3) (5.1 ch)
Scanning title 1 of 1, preview 2, 100.00 %[19:45:57] scan: 2 previews, 1280x720, 23.976 fps, autocrop = 0/0/0/0, aspect 16:9, PAR 1:1
Scanning title 1 of 1, preview 2, 100.00 %[19:45:58] libhb: scan thread found 1 valid title(s)

title 1:

stream: /home/dgibbs/nas-Downloads/convert-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv
duration: 00:01:08
size: 1280x720, pixel aspect: 1/1, display aspect: 1.78, 23.976 fps
autocrop: 0/0/0/0
support opencl: no
support hwd: not built-in
chapters:

1: cells 0->0, 0 blocks, duration 00:01:08

audio tracks:

1, Unknown (AC3) (5.1 ch) (iso639-2: und), 48000Hz, 384000bps

subtitle tracks:

HandBrake has exited.
Hope this helps

It may be worth seeing if you can setup an Ubuntu VM and replicate that way. If I remember correctly your using Mac? It could be some slight differences in the software that are causing the issue? This is retested on just released Ubuntu 16.04 LTS and still the same issue.

@dgibbs64 Can you include the entire scan? You're missing the first few lines which contain the HandBrake version number.
Thanks.

@dgibbs64 Are you using a development/nightly build of HandBrakeCLI? I ask that because the scan says you're using libdvdnav version 5.0.3. And that's not included with HandBrake version 0.10.5, the current official release.

Im just using whatever the default is for Ubuntu 16.04 Installed:
Which appears to be
dgibbs@ubuntu:~$ apt-cache policy handbrake-cli
handbrake-cli:
Installed: 0.10.2+ds1-2build1
Candidate: 0.10.2+ds1-2build1
Version table:
*** 0.10.2+ds1-2build1 500
500 http://gb.archive.ubuntu.com/ubuntu xenial/universe amd64 Packages
100 /var/lib/dpkg/status

@dgibbs64 Maybe the version is 0.10.2 but it's hard to tell with all that noise from apt-cache so give me the output of:
HandBrakeCLI --preset-list

Specifically, the first line of that output. That's the crucial version information.

@dgibbs64 BTW, getting back to the scan output. This is the crucial line I use to detect whether an input file contains a valid H.264 video stream:
    Stream #0:0(eng): Video: h264 (High), yuv420p(tv, bt709/unknown/unknown), 1280x720, SAR 1:1 DAR 16:9, 23.98 fps, 23.98 tbr, 1k tbn, 47.95 tbc (default)

And there's nothing wrong with that line. My parser should see that. So, I have no idea why it's failing.
This means I need to go back to your original attempt to use convert-video and get FULL diagnostic information from the failure. Here's how you do that:
convert-video -vv "/path/to/video.mkv"

See that -vv there? That's the same as using the --verbose option twice. It's just less to type. :)
Please paste in ALL of the output from that command -- from the first line to the last line.
Thanks.

Output of HandBrakeCLI --preset-list
dgibbs@ubuntu:~$ HandBrakeCLI --preset-list
HandBrake 0.10.2<PHONE_NUMBER>) - Linux x86_64 - https://handbrake.fr
< Devices
  + Universal:  -e x264  -q 20.0 -r 30 --pfr  -a 1,1 -E ffaac,copy:ac3 -B 160,160 -6 dpl2,none -R Auto,Auto -D 0.0,0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 720 -Y 576 --loose-anamorphic --modulus 2 -m --x264-preset fast --h264-profile baseline --h264-level 3.0

  + iPod:  -e x264  -q 22.0 -r 30 --pfr  -a 1 -E ffaac -B 160 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -I -X 320 -Y 240 --modulus 2 -m --x264-preset medium --h264-profile baseline --h264-level 1.3

  + iPhone & iPod touch:  -e x264  -q 22.0 -r 30 --pfr  -a 1 -E ffaac -B 160 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 960 -Y 640 --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 3.1

  + iPad:  -e x264  -q 20.0 -r 30 --pfr  -a 1 -E ffaac -B 160 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 1280 -Y 720 --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 3.1

  + AppleTV:  -e x264  -q 20.0 -r 30 --pfr  -a 1,1 -E ffaac,copy:ac3 -B 160,160 -6 dpl2,none -R Auto,Auto -D 0.0,0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 960 -Y 720 --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 3.1 -x qpmin=4:cabac=0:ref=2:b-pyramid=none:weightb=0:weightp=0:vbv-maxrate=9500:vbv-bufsize=9500

  + AppleTV 2:  -e x264  -q 20.0 -r 30 --pfr  -a 1,1 -E ffaac,copy:ac3 -B 160,160 -6 dpl2,none -R Auto,Auto -D 0.0,0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 1280 -Y 720 --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 3.1

  + AppleTV 3:  -e x264  -q 20.0 -r 30 --pfr  -a 1,1 -E ffaac,copy:ac3 -B 160,160 -6 dpl2,none -R Auto,Auto -D 0.0,0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 1920 -Y 1080 --decomb=fast --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 4.0

  + Android:  -e x264  -q 22.0 -r 30 --pfr  -a 1 -E ffaac -B 128 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 720 -Y 576 --loose-anamorphic --modulus 2 --x264-preset medium --h264-profile main --h264-level 3.0

  + Android Tablet:  -e x264  -q 22.0 -r 30 --pfr  -a 1 -E ffaac -B 128 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 1280 -Y 720 --loose-anamorphic --modulus 2 --x264-preset medium --h264-profile main --h264-level 3.1

  + Windows Phone 8:  -e x264  -q 22.0 -r 30 --pfr  -a 1 -E ffaac -B 128 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 -X 1280 -Y 720 --modulus 2 --x264-preset medium --h264-profile main --h264-level 3.1

< Regular
  + Normal:  -e x264  -q 20.0 -a 1 -E ffaac -B 160 -6 dpl2 -R Auto -D 0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 --loose-anamorphic --modulus 2 -m --x264-preset veryfast --h264-profile main --h264-level 4.0

  + High Profile:  -e x264  -q 20.0 -a 1,1 -E ffaac,copy:ac3 -B 160,160 -6 dpl2,none -R Auto,Auto -D 0.0,0.0 --audio-copy-mask aac,ac3,dtshd,dts,mp3 --audio-fallback ffac3 -f mp4 --decomb --loose-anamorphic --modulus 2 -m --x264-preset medium --h264-profile high --h264-level 4.1

Output of convert-video -vv "/path/to/video.mkv"
dgibbs@ubuntu:~$ convert-video -vv nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv
convert-video 0.7.0
Copyright (c) 2013-2016 Don Melton
HandBrake 0.10.2 found...
Processing: nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv...
Scanning media title 1 with HandBrakeCLI...
[23:09:12] hb_init: starting libhb thread
HandBrake 0.10.2<PHONE_NUMBER>) - Linux x86_64 - https://handbrake.fr
4 CPUs detected
Opening nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv...
[23:09:12] CPU: Intel(R) Core(TM) i3-3220 CPU @ 3.30GHz
[23:09:12]  - Intel microarchitecture Ivy Bridge
[23:09:12]  - logical processor count: 4
[23:09:12] OpenCL: library not available
[23:09:12] hb_scan: path=nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv, title_index=1
udfread ERROR: ECMA 167 Volume Recognition failed
disc.c:269: failed opening UDF image nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv
disc.c:350: error opening file BDMV/index.bdmv
disc.c:350: error opening file BDMV/BACKUP/index.bdmv
[23:09:13] bd: not a bd - trying as a stream/file instead
libdvdnav: Using dvdnav version 5.0.3
libdvdread: Encrypted DVD support unavailable.
************************************************
**                                            **
**  No css library available. See             **
**  /usr/share/doc/libdvdread4/README.css     **
**  for more information.                     **
**                                            **
************************************************
libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.IFO failed
libdvdread:DVDOpenFileUDF:UDFFindFile /VIDEO_TS/VIDEO_TS.BUP failed
libdvdread: Can't open file VIDEO_TS.IFO.
libdvdnav: vm: failed to read VIDEO_TS.IFO
[23:09:13] dvd: not a dvd - trying as a stream/file instead
Input #0, matroska,webm, from 'nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv':
Metadata:
encoder         : libebml v1.3.0 + libmatroska v1.4.1
creation_time   : 2014-08-23 03:01:46
Duration: 00:01:08.42, start: 0.000000, bitrate: 2188 kb/s
Stream #0:0(eng): Video: h264 (High), yuv420p(tv, bt709/unknown/unknown), 1280x720, SAR 1:1 DAR 16:9, 23.98 fps, 23.98 tbr, 1k tbn, 47.95 tbc (default)
Stream #0:1: Audio: ac3, 48000 Hz, 5.1(side), fltp, 384 kb/s (default)
[23:09:13] scan: decoding previews for title 1
[23:09:13] scan: audio 0x1: ac3, rate=48000Hz, bitrate=384000 Unknown (AC3) (5.1 ch)
Scanning title 1 of 1, preview 2, 100.00 %[23:09:13] scan: 2 previews, 1280x720, 23.976 fps, autocrop = 0/0/0/0, aspect 16:9, PAR 1:1
[23:09:13] libhb: scan thread found 1 valid title(s)
+ title 1:
+ stream: nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv
+ duration: 00:01:08
+ size: 1280x720, pixel aspect: 1/1, display aspect: 1.78, 23.976 fps
+ autocrop: 0/0/0/0
+ support opencl: no
+ support hwd: not built-in
+ chapters:
+ 1: cells 0->0, 0 blocks, duration 00:01:08
+ audio tracks:
+ 1, Unknown (AC3) (5.1 ch) (iso639-2: und), 48000Hz, 384000bps
+ subtitle tracks:
HandBrake has exited.
  {:title=>1, :size=>18718680, :directory=>false, :duration=>68, :width=>1280, :height=>720, :fps=>23.976, :autocrop=>nil, :audio=>{1=>{:format=>"AC3", :channels=>5.1, :language=>"und", :bps=>384000}}, :subtitle=>{}, :h264=>false, :mpeg2=>false, :bd=>false, :dvd=>false, :mkv=>true, :mp4=>false}
  /usr/local/bin/convert-video: no H.264 format video track: nas-Downloads/transcode-video/output/original/sample-the.knick.s01e03.720p.hdtv.x264-killers.mkv

@dgibbs64 Thanks! But most of that is still formatted as regular text. Can you can re-edit that comment to format it as code?

@dgibbs64 Nevermind. I realized I could edit it myself. :)

@dgibbs64 OK, I see the problem now. Sorry that I missed it the first time but it's very subtle.
For some strange reason, your particular version of HandBrakeCLI is using a colon (":") instead of a period ".") to separate the two numerical components of a stream identifier. For example, your scan shows Stream #0:0 instead of Stream #0.0. And that's screwing up my parser.
I've never seen that output from any other version of HandBrakeCLI. Now I need to do some research and testing...

That's great news you have spotted the issue. Hopefully you can figure out a work around. Good Luck!

@dgibbs64 Looks like this has to do with the version of libavcodec to which HandBrake is linked. Because libavcodec contains the code which generates the steam information in the scan output. And different versions of libavcodec use different punctuation. Go figure.
Anyway, (I think) I can fix this in my parser by being a little more lenient about about the stream identifier. So, I'll put this on the list of things to fix for 0.8.0. Probably due out next week.
Stay tuned! And thanks for going through all these hoops with me. :)

@dgibbs64 OK, I have a patch ready for this and #30 that I'm evaluating now. Unfortunately, I don't have your setup so I'm taking it a bit on faith that it will work. But I'll figure out someway to test it. Stay tuned.

@dgibbs64 OK, the fix has been released in version 0.8.0. Just update to get it. Please test on your configuration and let me know if it actually works.
Thanks again!


Support Split And Retry for GpuProjectAstExec

This is to support split & retry for GpuProjectAstExec. It's tracked in 8311.
Tested by newly added UTs locally.

build

build


Unexpected server response (206) while retrieving PDF...

Link to PDF file (or attach file here): http://api.idocv.com/view/KeDWwzf
Configuration:

Web browser and its version: 微信WeChat 6.3.27
Operating system and its version: IOS 10.0.3
PDF.js version: PDF.js v1.5.337
Is an extension:

Steps to reproduce the problem:

Open 微信WeChat
Wiew http://api.idocv.com/view/KeDWwzf

What is the expected behavior? (add screenshot)
What went wrong? (add screenshot)
Unexpected server response (206) while retrieving PDF...

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

the same problem

This may be related to https://github.com/mozilla/pdf.js/issues/5405#issuecomment-252546051, try setting PDFJS.disableRange = true; and see if that helps.

Seems like setting PDFJS.disableRange = true;  it's helps in iOS 10.0.1, but  in iOS 10.1.0 can not load PDF file ,hasn't loading bar and exception tips , just black page.

Closing as fixed by PR #7800.

@Snuffleupagus The problem still exist.
As I mentioned before, I tested in WeChat App in iPhone, NOT safari in iPhone.
Safari in iPhone is OK, but app in iPhone does NOT work.

@Snuffleupagus I have the same problem of @Lyife but in iOS 10.1.1.
I tried to set PDFJS.disableRange = true and changecheckRangeRequests in compatibility.js but I have a black page.

我的解决办法是，打开错误信息内容，复制PDF文档路径，使用迅雷下载。


virt-controller, vm: Set domain firmware UUID based on namespace & name

What this PR does
The UUID generated needs to depend on the uniqueness of the VM name and its namespace (same name may appear in two different namespaces).
fix: https://github.com/kubevirt/kubevirt/issues/11166

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

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

/cherrypick release-1.2


Peptides: cache tooltips for barchart

Cache barchart tooltips so that data is not recalculated if the mouse points at the same bar

How expensive is the recalculation on a dataset of 1M peptides? Just give a rough estimate please - if it's too much, then likely the calculation itself should be improved

@skalkin what I meant is caching the result that is currently used. Right now the data is recalculated for a single bar for any slightest mouse movement, event if the mouse didn't leave the bar it was pointing at. It results in recalculating the same data over and over when nothing is actually changed. What I want to do is caching result for a single bar that the user is pointing at.


Remove duplicated jcenter block

Seems like, in #22640, we accidentally added two jcenter blocks.
One should be enough.

@bot-gradle test BD

OK, I've already triggered the following builds for you:

BuildDistributions build

@bot-gradle test and merge please

OK, I've already triggered a build for you.


 config is very lenient

It seems that the plugin (dev.aspectj:aspectj-maven-plugin: 1.13.1) doesn't care at all what the elements inside of <aspectLibraries> are called.
This
<aspectLibraries>
    <aspectLibrary>
        <groupId>org.agroup</groupId>
        <artifactId>to-weave</artifactId>
    </aspectLibrary>
</aspectLibraries>

works just the same as this
<aspectLibraries>
    <toast>
        <groupId>org.agroup</groupId>
        <artifactId>to-weave</artifactId>
    </toast>
</aspectLibraries>

That's not a "problem" per se, but it doesn't feel right and it can lead to inconsistencies with other tools or in the future.
The Eclipse m2e-ajdt connector, for example, silently skips anything but <aspectLibrary> nodes.
This can lead to problems that are difficult to pin down because they depend on which tool did the last compilation. It took me quite a while to understand where my problems were coming from.
Enforcing the elements to be called <aspectLibrary> would be nice, but may be too strict for existing projects. But maybe it's possible to emit a warning if other names are detected.

Well, that is not an effect specific to AJ Maven Plugin. In general, sub-element naming for array parameters is not enforced in the Maven plugin API. It would be nice to have an additional @Parameter parameter to indicate the desired sub-element name, but there is none. You can try the same thing e.g. in Maven Surefire:
<plugin>
<artifactId>maven-surefire-plugin</artifactId>
<configuration>
  <excludes>
    <exclude>**/MyFirstTest.java</exclude>
    <toast>**/MySecondTest.java</toast>
  </excludes>
</configuration>
</plugin>

The plugin will not complain and just use the toast exclude normally.


APFS Dirty volumes fail to process

If an APFS volume has not been cleanly unmounted, the resulting disk image cannot be processed by mac_apt. The following error will be seen:
MAIN-ERROR-Unknown error while trying to determine partition
MAIN-ERROR-Exception
Traceback (most recent call last):
  File "/Users/yogesh/Downloads/mac_apt_binaries_10.10/mac_apt.app/Contents/Resources/mac_apt.py", line 390, in <module>
  File "/Users/yogesh/Downloads/mac_apt_binaries_10.10/mac_apt.app/Contents/Resources/mac_apt.py", line 239, in FindOsxPartition
  File "/Users/yogesh/Downloads/mac_apt_binaries_10.10/mac_apt.app/Contents/Resources/mac_apt.py", line 177, in FindOsxPartitionInApfsContainer
  File "plugins/helpers/apfs_reader.pyc", line 669, in __init__
AttributeError: 'Node' object has no attribute 'root'
MAIN-WARNING-:( Could not find a partition having an OSX installation on it

Need to figure out the right way to recover from a dirty volume. Not trivial, and will take time :(

Fixed in the py3 branch, will move to master soon.


[Work in progress] Tcp http handshake timeout

PR for story Add official support to kill on slow handshake for the TCP and HTTP layers

@dpwspoon I created a separate PR which only contains the changes for the tcp and http layers, since I split the initial story into several smaller stories.
I addressed most of your code review comments from my previous PR, except for the one about setting the taskExecutor as a session key. You mentioned it should be a key for the HttpSession.class, but in the HttpHandshakeFilter, the HttpSession is not yet created, I only have access to an IoSession. Since now I only have one filter for the HttpSession and the taskExecutor is not being passed around in different filters, is keeping this as a key in the session still needed ?
I also have a few questions regarding this:

Since you mentioned that having the protocol.handshake.timeout option also as a connect option is needed for the reverse conectivity feature, does this mean that these should only be added as connect options in the enterprise.gateway .xsd and not in the gateway one?
In the requirements doc, it is mentioned that there should be conflict resolution for each layer. What exactly does this imply for the TCP layer?
I check if the EarlyAccessFeature is enabled in the DefaultAcceptOptionsContext class. This check does not apply for tcp tests where a TcpAcceptorRule is used, instead of a GatewayConfigurationRule (meaning the TCP filter for killing slow handshakes is always added). Is this the desired behavior or should the check be done in the NioAbstractAcceptor class?

This is still a WIP I believe so I've moved it to blocked...

retest this please

retest this please

SonarQube analysis reported 245 issues

 3 blocker
 10 critical
 180 major
 44 minor
 8 info

Watch the comments in this conversation to review them.
Top 10 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 GatewayContextResolver.java#L387: Change this condition so that it does not always evaluate to "true"
 HttpAcceptor.java#L458: NullPointerException might be thrown as 'localAddress' is nullable here
 AbstractNioAcceptor.java#L397: Catch Exception instead of Throwable.
 ServiceConfigurationBeanImpl.java#L223: Either log or rethrow this exception.
 ServiceConfigurationBeanImpl.java#L360: Either log or rethrow this exception.
 ServiceConfigurationBeanImpl.java#L404: Either log or rethrow this exception.
 ServiceDefaultsConfigurationBeanImpl.java#L153: Either log or rethrow this exception.
 ServiceDefaultsConfigurationBeanImpl.java#L230: Either log or rethrow this exception.
 ServiceDefaultsConfigurationBeanImpl.java#L250: Either log or rethrow this exception.
 TcpResourceAddressFactorySpi.java#L276: Either log or rethrow this exception.


HDDS-4210. ResolveBucket during checkAcls fails.

What changes were proposed in this pull request?
When bucket links are levels, When ACLS are enabled, during checkAcls It will fail with NPE.
Finally in HA, this causes OM to be terminated.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4210
How was this patch tested?
Added UT.
The same test will fail without the fix.

@xiaoyuyao Do you have any more comments?

Review comments are addressed, and no further comments, proceeding with commit.
Thank You @adoroszlai and @xiaoyuyao  for the review.


np.equal is not a known member of module Pylance

Environment data

Language Server version: 2020.10.1
OS and version: OSX 10.15.7
Python version (& distribution if applicable, e.g. Anaconda): Python 3.7

Expected behaviour
no error highlight when typing:
import numpy as np
np.equal(1, np.ones(2))

Actual behaviour
an error is indicated:
"message": "\"equal\" is not a known member of module",
"source": "Pylance (reportGeneralTypeIssues)",

Logs

N/A

Code Snippet / Additional information

import numpy as np
np.equal(1, np.ones(2))

Duplicate of #150; this should be improved once numpy releases with type stubs.
Numpy has some dynamic behaviors that prevent us from knowing if certain values exist, or should be re-exported from modules. Their new stub is much more clear.

Thanks for the explanations; Am I correct in deducing that Pylance and Numpy will only play well together with future versions of Numpy, whenever they support type stubs?

Some things work, but others don't. At some point we may add back compiled code scraping, but that is difficult to do correctly and doesn't fit nicely into the type system/analysis without some more thought.


Re-implement Motorola S-record output

The first implementation sucked, and was removed before committed.
Go back and do this properly, akin to how the (new) Ihex module is done.
Not necessary until we activate the MC68xx targets.

Fixed in #087ce28.


WCT documentation update v8.1.240328-rc

This PR updates the Windows Community Toolkit documentation to align with the 8.1.240328-rc prerelease.
Documentation and ToC was generated by our ./tooling/Build-Toolkit-Docs.ps1 build script, detailed here.
The index.md and getting-started.md, which are not present in the generated docs, were manually re-added to the ToC, as well as the links at the bottom.

@microsoft-github-policy-service agree

Thanks Arlo, not sure why we're not seeing the doc build run to give us a preview...
We also need the map of the moved docs, so we can get redirects in place.

@michael-hawker click here and then you can see the build previews:

In the build log there are View links for each changed file.
For example:  https://review.learn.microsoft.com/en-us/dotnet/communitytoolkit/windows/colorpicker/index?branch=pr-en-us-393

Thanks @JasonWHowell for the info, I see that now. I wasn't part of the discussion when the flag to bubble it up with the bot was turned off here: https://github.com/MicrosoftDocs/CommunityToolkit/commit/51d7ca459ae49a13c2bbad870229ee3a6d806032 - so I'm following up with our co-owners of the repo.
The bot post certainly made it clearer when there were warnings and things we should be fixing before committing. FYI @jfversluis, started a thread on teams with the team. 😊

Example of redirects can be found here

Voila, careful what you wish for... A lot of warnings to fix 😜

I've added the redirects and followed the existing formatting, not sure why the warnings are still showing there.

Issue persists

Awesome, thanks @arlodotexe! I believe everything in main goes live pretty quick, so we should wait to merge this until we're ready to release, right?


chore: Cake 0.31.0 -> 0.32.1

:tada: This PR is included in version 2.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore: support for phpdocumentor/reflection 4 in snippets

phpunit/phpunit:8 is required in order to run tests on PHP 8.0+
phpdocumentor/reflection:3 conflicts with phpunit/phpunit:8
phpdocumentor/reflection:4 does not run on < PHP 7.2

I've resolved this by adding support for both reflection v3 and v4 when running snippets tests.

Thanks for the review @dwsupplee! It's ready for another look!


One way collisions doesnt work

Describe the bug
One way collisions don't work.
To Reproduce
Have an AnimateableBody2D with one way checkbox enabled on it's collision shape. Make this body go up and down with PathFollow2D. Put Rigidbody2D object under it. Make sure collision masks and layers can see each other.
Expected behavior
Object should've passed through the under of the moving platform since one way collision is on, but it doesn't. Correct behaviour can be seen when switched to default engine.
Environment:

OS: Windows 10 22H2  (19045)
Rapier version 0.7.26, Cross platform deterministic.
Godot Version 4.2.2 stable
Type Collisions..? (Don't know what this means)

Example project(zip)
Here is the project zip with no addons folder.
rapier-oneway-bug.zip

One way collisions for rigidbodies should be fixed now. For character body still no. Re-checking if this issue is fixed(for rigidbodies)


tone v0.1.7 reports version 0.1.6

➜  tone-0.1.7-linux-x64 ./tone --version
0.1.6

As you can see, it reports the wrong version number.

Ah crap, forgot to update the version string. Should be fixed now (please ensure to re-download an uncached version of the file).


Add proper, tested support for Overcast.fm

I use Overcast daily and really wanted Keysocket support. I use this daily, so I can confirm that it works.

Also adds support for https://developer.apple.com/videos where WWDC sessions can be viewed.


Add platform_interaction_test_swift to devicelab

Description
Add new devicelab test to launch example platform_channel_swift app.
Related Issues
Fixes https://github.com/flutter/flutter/issues/35774.
Tests
I added the following tests:

platform_interaction_test_swift

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read [Handling breaking changes]). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

This is blocked by https://github.com/flutter/flutter/issues/35773.  Will remove WIP label when that is fixed.


Add MM_Graph_API missing tests

file_Get, file_List, file_Save

This is not needed anymore since the missing MM_Graph_API are covered by the controllers tests (via the expectGET calls)


Fixes #685

The link doesn't need to be changed to absolute, it works already.
You seem to be missing the * from the beginning of the two indented lines now.
The leading \ before the character code is gone too.

@lewisgoddard ha yeah good review, I missed the missing *s.

Closing in favor of #703, thank you for your contribution.


Need to stop suggested videos as a default

I tried two ways that do not work:
yt_html = yt_html + "";               Doesn't work?
}
The other was appending ?rel=0 to each video name, which works, but video icon picture is lost

it put a picture over the section above   "

It will not show that I placed "?rel=0" before + ""

got it to work by added to all ? places that MichaelOuellette had suggested. Still needs to to added permanently as you would never want youtube to send you out of an embeded video


fix(CI): change cbl file extension to lower case

Signed-off-by: Sven Pfennig<EMAIL_ADDRESS>

Wait, we're going with lowercase .cbl or .cob?

@KTSnowy, like this?

@KTSnowy, like this?

Yep, looks good to me. Should we merge this so that we can merge the Leap exercise as well?


Reopening Enums

Is there any particular reason for prohibiting reopening enums?
https://carc.in/#/r/6nq9:
enum Foo
  Bar
end

enum Foo
  Bar
  Baz
end

Error in line 5: can't reopen enum and add more constants to it

Usecase
In Worcr, I have multiple "editions" (OSS, Pro), and I have a raw protocol which has a number of commands, which are enums and being transimtted as single bytes over a TCP socket. Pro version should have more commands available but still support OSS commands. And my intention was to re-use the commands enum from OSS part and extend it with Pro commands:
# worcr-oss/proto.cr
module Worcr
  enum Proto
    Ping
    SendJob
  end
end

# worcr-pro/proto.cr
require "worcr-oss/proto"

module Worcr
  enum Proto
    Ping
    SendJob
    UpdateCheckpoint
    UpdateProgress
  end
end

Yeah, I see a problem. Reopening should either "extend" existing enums or "copy-and-add". Both approaches have their flaws.
Extending:
enum Foo
  Bar
end

enum Foo
  Baz
end

Expected Foo to have both Bar and Baz. But what if require order changes? It will be Baz and Bar then. Absolutely non-transparent and would silently break the code.
Copy-and-add:
enum Foo
  Bar
end

enum Foo
  Bar
  Baz
end

This is more realistic to me. If Foo already exists, all reopening must have previous enums declared, raising in compilation time otherwise:
enum Foo
  Bar
end

enum Foo
  Baz
end

# Error: Foo must have Bar with value 0, as declared previously

Extension:
That's another idea. Allowing to extend enums. Should fit to my use-case.
enum Foo
  Bar
end

enum Qux < Foo
  Baz
end

# Result:

enum Qux
  Bar
  Baz
end

The idea of enums is that you have a limited set of values to handle. You can switch over them and be sure to handle them all (eventually we could make the compiler check this). If enum is extensible then this property is lost.

If you want flexible I don't think enum fits. If a shard needs to use that enum it needs to target to either:
a) target to one of the versions only (not very friendly)
b) be prepared to handle both, which is equivalent to assume a superset of values (ie: defining them)
Using non-fixed value types like symbols or strings seems more appropriate.
You could theoretically layout the files to locate dependable parts of the code to allow easier switching, but that will be pushed to the consumers of the shards.

Ary, I don't see how extended Enums and

be sure to handle them all

contradict each other. You say that compiler would eventually check if I've covered all of an enum's values. If I have a case and am iterating through all the values, it would raise if I have extended an enum occasionally. For example, a worcr-oss/server.cr code:
enum OSS
  Ping
  SendJob
end

case command
when .ping?
  handle_ping
when .send_job?
  handle_send_job
end

And then in worcr-pro/server:
enum Pro < OSS
  UpdateCheckpoint
end

case command
when .ping?
  handle_ping # Reuse code
when .send_job?
  handle_send_job # Reuse code
when .update_checkpoint?
  handle_update_checkpoint # New code
end

I see no issues here. Could you provide a better devastating point?
In response to Brian — there will be no shards depending on Worcr, as it's being a standalone application. I have to use enums because I want to encode commands in a single byte. I do not expect it to be extendable (i.e. middleware with custom commands).

@vladfaust I provide this library:
enum Color
  Red
  Green
end

def handle_color(color : Color)
  case color
  when .red?
    # ...
  when .green?
    # ...
  else
    raise "unexpected color!"
  end
end

Now someone uses my lib but repones the enum:
enum Color
  Blue
end

handle_color is now broken, it doesn't handle that case.
If you have pro and non-pro code you can define separate enums for each, and compile them separately... if that's what you really want. Alternatively it's probably better to list the options for pro and non-pro as runtime values, as @bcardiff suggests.

@asterite, I understand. First two cases from https://github.com/crystal-lang/crystal/issues/7629#issuecomment-480173091 don't work, therefore this issue can be closed. See #7632 instead.
Opposing your example with inheritance:
enum Color
  Red
  Green
end

def handle_color(color : Color)
  case color
  when .red?
    # ...
  when .green?
    # ...
  else
    raise "unexpected color!"
  end
end

# In another lib...
#

enum CustomColor < Color
  Blue
end

def custom_handle_color(color : CustomColor)
  case color
  when .red?
    # ...
  when .green?
    # ...
  when .blue?
    # ...
  else
    raise "unexpected color!"
  end
end

I could also think of some kind of downcasting if needed:
def custom_handle_color(color : CustomColor)
  case color
  when .red?, .green?
    handle_color(color.as(Color)) # Compiler could ensure it
  when .blue?
    # ...
  else
    raise "unexpected color!"
  end
end


Issue #15 is applicable for auto-tags

The Issue #15 exists when applying multiple auto-tags and might affect other configurations when applying in multiple as well.. Please help with quick fix for the same

Same issue when applying multiple dashboards

@Dynatrace-Reinhard-Pilz : Could you please help us on this as well.. Only this is blocking us from using this provider

As a matter of fact you're in luck. Because of your input tomorrow's v1.0.8 will introduce a re-implementation of AutoTags just like for Management Zones. We're still performing a few manual tests, but the release tomorrow is pretty certain.

Thanks a lot @Dynatrace-Reinhard-Pilz for your great contributions..
multi Dashboards migration support ASAP, we will start using this provider in prod

v1.0.8 takes care of this issue for Auto Tags.
I will keep this issue open until Dashboards have undergone the necessary changes - you should expect that being done in about a week with v1.0.9.

in v1.0.10, multiple Dashboard apply is fixed?

Hi @Karthikeyanraman94,
v1.0.11 takes care of Dashboards. Please be aware of the fact, that the HCL Schema for dynatrace_dashboard has changed in a couple of significant places. That was unfortunately unavoidable. You will have to recreate the .tf files again, either using the download functionality of the provider or by importing them via terraform cli.


Phonon V 1.4.2 - Dependence of Awesome

Scenario
Using Phonon V 1.4.2 I get error on console "The autocomplete component requires Awesomplete dependency.", just by run Phonon (phonon.min.js).
Project no using autocomplete.

Hi again @John-Henrique,
Thank you for opening this issue, it has been fixed in the v1.4.3 ..


Pefomance issue over 1179 items

Hi,
I am having huge performance issue when there is a large number of items in my input model(1179 items to be exact). I had sliced it to only 30 items and it works fine. The page has 3 other multi select as well and they are not impacted(all 3 has less than 20 items each). All 3 of them works fine as well.
Also, the select with large input model, shows only few data when I open it. I assume this is for performance and some kind of pagination. But how do I see the rest as I don't see an auto scroll kind of pagination or any other button for that matter. Of course I can see the rest of the items if I start typing them.
Performance is hit badly that in IE it takes more than 6 seconds for the select to be shown. Also if we start typing something in that select, its getting stuck.
Please let me know if something can be done.
Thanks,
Sumesh

@sumeshk ,
Unfortunatley, as for now, there's nothing much I can do. This is a well documented issue, with some users in the past also experiencing slight delay on big data. I also mentioned this on the manual; http://isteven.github.io/angular-multi-select/#/faq
Some suggestions:

Somehow cut or group the data into smaller chunks
Try to use earlier version; such as v1.2.0 or v2.x.x
Use another multi-select directive. Perhaps the default HTML's multi-select element will work for you?


modified:   Number Guess Game JS Extension/src/style.css

Issue- #345
Description
In Number Guess Game Js Extension when we shrink window size the word Check goes outside the box so to make responsive and it look well even in small screen i modified css.

Hi @Sulagna-Dutta-Roy ,
I have resolved the issue #345 assigned to me.
Thanks


IRGen: Don't encode conditional requirements to Copyable as normal conformance requirements.

For new runtimes, this is redundant with the invertible requirement encoding, and for old runtimes, this breaks dynamic conformance checking because Copyable and Escapable aren't real protocols on those older runtimes. Fixes rdar://129857284.

@swift-ci Please test

@swift-ci Please smoke test


Feature Request: "Distinct colours for different labels"

Branch:
I am using the latest develop branch of cvat
Problem:
I am annotating three different classes. I have set appearance to "Label" so it is easy to see when I have made a mistake and selected the wrong class for labelling. But the colours used for different classes are three subtly different shades of pinkish purple.

Suggestion:
It would be much easier to spot my mistakes if distinct colours were used instead.
This website http://phrogz.net/css/distinct-colors.html  generates lists of visually distinct colours.
Maybe the default colours used could be taken one by one from a list like:
#bf8f8f, #7f1100, #ff4400, #ffa280, #4c3026, #a65800, #d9a66c, #ffaa00, #664d1a, #807560, #b2982d, #ffee00, #e6e2ac, #90d900, #4c661a, #2f3326, #688060, #6cbf60, #00330e, #00ff66, #008066, #40ffd9, #8fbfb6, #10403d, #40d9ff, #406a80, #003666, #40a6ff, #acc3e6, #4059ff, #606cbf, #14004d, #3d0099, #4e3366, #b56cd9, #cc00ff, #b68fbf, #332630, #4d0033, #b32d86, #99003d, #ff80b3, #4c000a, #ff4059

@LukeAI
Thank you for the information, it is really useful.
We will look into that.

Oh I just saw that I can set the label colour in the Labels tab. So this is not super-important. Better defaults would be nicer though. It would also be really nice if the same label had the same colour across all tasks. One thing I've been a little confused about is that every time I create a new Task and upload the same label list, every label from every task seems to have a unique ID? It doesn't really matter if the annotations I download don't have differen IDs, but why is that?

@LukeAI , thanks for the issue and the link. Agree that it is necessary to fix default colors.


Backporting forms

This PR brings in various forms from past projects into one - lots of refactoring and customization in order to make the forms consistent and compatible with the new versions of the packages like React Hook Form and the Yup validation, Flowbite etc..
The forms and functionalities that have been added in this PR are:

A generic form having the common fields (required and optional)
Form with conditional fields
Two step form
Newsletter
and the generic form being loaded in a modal (navigation button)

I have either commented out the form submission codes from each form, or have commented them out since the form submission requirement could be specific for a real project and can be implemented out of Starter template's scope.
Preview link

@seemaion8 I feel like the description of this PR misses a golden opportunity to use screenshots and other markdown features for polish. Big PRs should always have an extra nice description.
When you're listing the different components and things you've brought in, I think using ### headers and visual aids would have looked really good. It makes it a lot easier for the reviewer to understand exactly what these different components do.
GitHub's support for screenshots is extremely smart, so if you use some sort of snipping tool (Cmd + Shift + 5 in your OS, I think) to take a capture of part of your screen (like, just the component), copy it to the clipboard, then you can just paste it straight into the description box and GitHub will upload and embed the image automatically. Like, you literally just CTRL V it into the text, as if it was text. This also works with GIFs.
Whenever you see an opportunity to add visuals to the description of a PR - especially when the PR is very big and has frontend components - you should take that opportunity. It makes it look much more professional.
If the description becomes a bit long, you can break it into sections by using # headers, this will create an h1.
Formatting helps, too!

By using ** before and after some text, you make it bold.
By using _ before and after some text, you make it italic.

You can even add footnotes[^1] ! And there can be text after the footnote. This is done using [^1] up here, and then [^1]: footnote text at the end.
Example of h1 header. Components added!
Ninja component (h3, using ### before text)

Tiny text, like below an image, can be done with the html tag <sub> before and after the text.
Duck component

If you want the text even smaller, just stack multiple <sub>s!
You can make the text REALLY SMALL by doing this.
[^1]: They're pretty cool, right?


ScrollView with DialogButton

I have some problem with ScrollView into PopupDialog.
This is my code:
<View style={styles.container}>
        <Button
          text="Show Dialog"
          onPress={() => {
            this.popupDialog.show();
          }}
        />
        <PopupDialog
          ref={(popupDialog) => { this.popupDialog = popupDialog; }}
          dialogAnimation = { new SlideAnimation({ slideFrom: 'bottom' }) }
          dialogTitle={<DialogTitle title="DialogTitle "/>}
          width={300}
          actions={[
            <Text>
              <ScrollView>
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-1"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-2"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-3"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-4"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-5"
                />,
              </ScrollView>
            </Text>
          ]}
        />
      </View>

Because I have many buttons into my dialog, I want to scroll down to see all buttons that I have into dialog. How I can do that? I try with ScrollView but it's not working.

Try moving them outside of the actions list. That seems like it's only supposed to be used to render a button or two at the bottom of the dialog. Something like this:
<PopupDialog
          ref={(popupDialog) => { this.popupDialog = popupDialog; }}
          dialogAnimation = { new SlideAnimation({ slideFrom: 'bottom' }) }
          dialogTitle={<DialogTitle title="DialogTitle "/>}
          width={300}
          >
            <Text>
              <ScrollView>
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-1"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-2"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-3"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-4"
                />,
                <DialogButton
                  text="ACCEPT"
                  onPress={() => {}}
                  key="button-5"
                />,
              </ScrollView>
            </Text>
          </PopupDialog>

Notice that I'm not defining an actions attribute, but adding your Text block with all the buttons to inside of the element's body.


Fix GSS codes for some councils in existing Business Support editions

Following on from frontend PR 948, we had to edit existing Business
Support Editions so that they would refer to the correct GSS codes
from the new data in MapIt, not the old ones that no longer worked.
This migration does just that, replacing the old GSS codes in the
edition's area_gss_codes array with the new ones. The councils
edited are: Northumberland, Gateshead, East Hertfordshire and
Stevenage.
We had to skip callbacks that checked for archived Artefacts in
Panopticon, and skip validations that said that already published
editions couldn't be edited.

Looks good


Trino Gateway Health check fails when trino cluster has oauth2 enabled

Whenever Trino gateway runs a health check with the trino backend clusters, it returns an 'Unauthorized to fetch cluster stats' error and sets the status of the cluster to 'false'. When changing trino authentication to 'form' from 'oauth2' the health check works.

Trino only currently supports the authorization code flow, so you'll need to set up a separate authn mechanism for the gateway service user.
For example, you can set http-server-authorization.types = OAUTH2, PASSWORD and set up the password database with a service user for the Gateway. The gateway only needs access to the system.runtime schema for health checks so you can restrict its access to everything else.

if the auth oauth2 then useApi has to be set to false, since oauth2 doesn't support /ui/api/stats

TLS is not required for JDBC connections

Please reopen if things still doesn't work for you.


python312Packages.lnkparse3: 1.4.0 -> 1.5.0

Diff: https://github.com/Matmaus/LnkParse3/compare/refs/tags/v1.4.0...v1.5.0
Changelog: https://github.com/Matmaus/LnkParse3/blob/1.5.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 313387 run on x86_64-linux 1

6 packages built:

donpapi
donpapi.dist
python311Packages.lnkparse3
python311Packages.lnkparse3.dist
python312Packages.lnkparse3
python312Packages.lnkparse3.dist