Implement Veldrid renderer

Closes #4909

Prereqs:

[ ] https://github.com/ppy/osu-framework/pull/5655

Extracted / improved upon from https://github.com/ppy/osu-framework/pull/5647 with much thanks to @frenzibyte for a large chunk of implementation that I've refactored since.
Abstract
This implements the Veldrid renderer, which is accessible via the argument --renderer veldrid. It's not the default yet, pending testing/fixing on more platforms.
Coordinate spaces
A big change in this PR is the introduction of workarounds for clip-space differences between renderers. The best documentation I've seen on the matter is https://matthewwellings.com/blog/the-new-vulkan-coordinate-system/, and I've chosen to follow the same path of adding a shader-level workaround for it.
I propose we break this in the future and invert our Z-coordinates - right now -1 is into the screen and 1 is the camera, and inverting it would make 0 the camera position and 1 into the screen.
Testing
osu!-side testing can be done with the following changesets:

osu-framework: https://github.com/smoogipoo/osu/tree/veldrid-renderer-2 (#5661)
osu: https://github.com/smoogipoo/osu/tree/veldrid-renderer (#22750)
osu-resources: https://github.com/smoogipoo/osu-resources/tree/opengl-core-renderer (#247)

Renderer/surface combinations can be tested with the following program arguments:
--renderer veldrid/glrenderer
--surface opengl/metal/vulkan/direct3d11

The below tests are only for the Veldrid renderer.

[x] Linux (AMD) (OpenGL)

intro animation is broken

[x] Linux (AMD) (Vulkan)

vsync broken (#5649)
graphics broken (likely driver issue)

[x] Linux + VMWare (OpenGL)
[x] Linux + VMWare (D3D)

effects broken

[x] Windows (OpenGL)
[x] Windows (D3D)

effects broken

[x] Windows (Vulkan)

videos crash

[ ] macOS (M1, OpenGL)
[ ] macOS (M1, Metal)
[ ] macOS (x86, OpenGL)
[ ] macOS (x86, Metal)
[ ] iOS (OpenGL)
[ ] iOS (Metal)
[ ] Android (OpenGL)
[ ] Android (Vulkan)
[x] llvmpipe (OpenGL)

Keep in mind that the renderer is not default yet so testing is optional as we work towards that.

M1 macos
--renderer veldrid --surface metal and --surface opengl
[runtime] 2023-02-27 19:49:05 [error]: An unhandled error has occurred.
[runtime] 2023-02-27 19:49:05 [error]: System.NullReferenceException: Object reference not set to an instance of an object.
[runtime] 2023-02-27 19:49:05 [error]: at Veldrid.OpenGL.OpenGLGraphicsDevice.Init(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height, Boolean loadFunctions)
[runtime] 2023-02-27 19:49:05 [error]: at Veldrid.OpenGL.OpenGLGraphicsDevice..ctor(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height)
[runtime] 2023-02-27 19:49:05 [error]: at Veldrid.GraphicsDevice.CreateOpenGL(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height)
[runtime] 2023-02-27 19:49:05 [error]: at osu.Framework.Graphics.Veldrid.VeldridRenderer.Initialise(IGraphicsSurface graphicsSurface) in /Users/integer/code/osu-framework/osu.Framework/Graphics/Veldrid/VeldridRenderer.cs:line 126
[runtime] 2023-02-27 19:49:05 [error]: at osu.Framework.Graphics.Rendering.Renderer.osu.Framework.Graphics.Rendering.IRenderer.Initialise(IGraphicsSurface graphicsSurface) in /Users/integer/code/osu-framework/osu.Framework/Graphics/Rendering/Renderer.cs:line 172
[runtime] 2023-02-27 19:49:05 [error]: at osu.Framework.Platform.GameHost.Run(Game game) in /Users/integer/code/osu-framework/osu.Framework/Platform/GameHost.cs:line 744
[runtime] 2023-02-27 19:49:05 [error]: at osu.Desktop.Program.Main(String[] args) in /Users/integer/code/osu/osu.Desktop/Program.cs:line 131
Unhandled exception. System.NullReferenceException: Object reference not set to an instance of an object.
   at Veldrid.OpenGL.OpenGLGraphicsDevice.Init(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height, Boolean loadFunctions)
   at Veldrid.OpenGL.OpenGLGraphicsDevice..ctor(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height)
   at Veldrid.GraphicsDevice.CreateOpenGL(GraphicsDeviceOptions options, OpenGLPlatformInfo platformInfo, UInt32 width, UInt32 height)
   at osu.Framework.Graphics.Veldrid.VeldridRenderer.Initialise(IGraphicsSurface graphicsSurface) in /Users/integer/code/osu-framework/osu.Framework/Graphics/Veldrid/VeldridRenderer.cs:line 126
   at osu.Framework.Graphics.Rendering.Renderer.osu.Framework.Graphics.Rendering.IRenderer.Initialise(IGraphicsSurface graphicsSurface) in /Users/integer/code/osu-framework/osu.Framework/Graphics/Rendering/Renderer.cs:line 172
   at osu.Framework.Platform.GameHost.Run(Game game) in /Users/integer/code/osu-framework/osu.Framework/Platform/GameHost.cs:line 744
   at osu.Desktop.Program.Main(String[] args) in /Users/integer/code/osu/osu.Desktop/Program.cs:line 131

--renderer glrenderer --surface opengl
(and --renderer glrenderer --surface metal but i believe the --surface arg here does nothing?)

video broken - same as in #5655

Testing GLRenderer is out-of-scope here, as noted in OP right above the tasks list.

On a gtx 1060 3gb with linux:
OpenGL: Runs fine, intro bugged.
Vulkan: Looks fine visually and high FPS doesn't break it, but the graphics freeze when trying to do nearly anything, while input and audio remain normal. As well, my PC freezes for a short while closing the game.

@Natelytle thanks for the testing, but you haven't provided much we can work with.
Vulkan: unless you can provide logs or any kind of useful system log output, we can't say more than "this is likely a problem with your system or drivers"
Intro being bugged: provide video, screenshots, phone camera footage, description. anything a developer can work with. "intro bugged" is as good as saying nothing at all.

I will provide proper logs tomorrow then, sorry about that

That's probably referring to what I wrote in the PR description that served as a reminder for me to look into it. Basically the intro looks like this:

And doesn't show when exiting the game. I'm already investigating it.

Id like to give a feedback on this. I couldnt spot the difference with all of combinations (maybe I did something wrong) but all I got is the fps reduce. Here are some videos for comparison:
Current master:
https://user-images.githubusercontent.com/86934170/222306219-8f15bac5-58e5-4c3d-a317-43563d5c6799.mp4
https://user-images.githubusercontent.com/86934170/222306453-9be6a24a-484e-44e7-9abf-14fdb82bcc1f.mp4
Proposed branch (without any launch parameters)(any of combinations is kinda the same):
https://user-images.githubusercontent.com/86934170/222306543-963aca38-cd6c-476d-90bc-6993117bc111.mp4
https://user-images.githubusercontent.com/86934170/222306721-8af0c21f-b36f-449d-8247-6c14c913b501.mp4

Metal will not work on windows.
Performance is expected to be lower when using veldrid, we are aware of this.

I've done a pass over this (mostly focused on changes to the existing code, not new veldrid classes). I think it's fine as an initial push, and we can work forward from here.

Probably the most concerning thing is/should be the blur shader, which I’m not entirely sure how to explain and/or make validation for at the moment other than “ensure either all your VS outputs are aligned to vec4s or you have no unused VS outputs in the FS”.
And I’m not even sure if the first of those even works, the second definitely does.

might be an issue on my end but when using --renderer veldrid --surface vulkan on arch, osu fails to find libdl despite having latest glibc and libdl being in both  /usr/lib and /usr/lib32


Add translate function for storage prefixed map.

Add a function to migrate values from one type to one another.
In case we fail to migrate one element we return an error with the key that has failed to decoded.
We could also remove values that fail to decode and finish the migration and at the end returning an error saying there is some value that has been removed.

related comment in regard to which implementation in case of error: https://github.com/paritytech/substrate/pull/4052#issuecomment-552399173

now key that couldn't be decoded are removed and an error is returned with the number of key that failed to decode.

EDIT: actually maybe better not to remove them and just let them untranslated in the storage, anyway this will only happen if the storage is corrupted or the translate function expect wrong type. in both case I don't think we can solve anything from our side.

If you leave them untranslated, runtime code could really get messed up. Better to remove data then to have bad data where the runtime assumes it is good. imo


added a report uri and unsafe-inline to the csp header.

i understand the policy is against unsafe-inline,
but as a matter of fact there are as lot of inline js/css,
and part of the js is adding additional remote js.
the report uri is implemented in a plugin with this purpose.
Improved:
Implemented:
Documented:
Completed:
Reverted:
Fixed:
(OFBIZ-)
Explanation
Thanks:

Hi Alex,
I agree there are still inline js/css even if most are handled correctly. We have related Jiras to eventually use require.js: https://issues.apache.org/jira/browse/OFBIZ-9465 https://issues.apache.org/jira/browse/OFBIZ-9976. Even if they yet got much attention we still hope to work on it.
So you can do that in your own project, we won't in official repo. If you disagree you may start a thread on the dev ML. Else please close.

@alexbodn Can you please close this PR?

@alexbodn Can you please close this PR?

Hi Michael,
It seems Alex is no longer interested in OFBiz, closing...


Add BOILERPLATE_README.fr.md

Most of our projects are documented in french, so we now have BOILERPLATE_README.fr.md. Its content is literal french translation of BOILERPLATE_README.md.

🦀 Requesting reviewers for this pull request:

@Madumo (contributor with 1 commits in the last 90 days and 19 commits overall)


toArray refactor

Just reducing array allocations and adding a test.

FFS... this PR is jacked up... I'll try it again.


Stunner service still pending

Hi,
I try to install stunner.  I finish installation and configuration but stunner pod still pending.
$ kubectl get pod -n stunner
NAME                       READY   STATUS    RESTARTS   AGE
stunner-7ff4875b47-6dtzt   0/2     Pending   0          12m

$ kubectl get service -n stunner
NAME      TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)    AGE
stunner   ClusterIP   <IP_ADDRESS>   <none>        3478/UDP   13m

There is some way how to debug?

Hi @krajcikondra ,
This can happen when your k8ss cluster does not support LoadBalancer services on UDP ports.
First thing to check: Do you use self-managed or managed kubernetes?  If it is managed, does the provider support UDP load balancers? If it is managed, extra annotations might be required, see https://github.com/l7mp/stunner/issues/32
If it is self-managed, you can install a load balancer (e.g., MetalLB or in case of minikube, minikube tunnel). As a last resort you can use stunner via its NodePort.
Hope this helps your troubleshooting.

I have managed kubernetes cluster on digital ocean and UDP load balancers are supported. But for UDP load balancer is needed implementation of health check.
https://docs.digitalocean.com/products/kubernetes/how-to/configure-load-balancers/#health-checks
I cannot say I fully understand what exactly should I do. After add two lines from documentation is load balancer status still Pending.
apiVersion: v1
kind: Service
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"gateway.networking.k8s.io/v1alpha2","kind":"Gateway","metadata":{"annotations":{},"name":"udp-gateway","namespace":"stunner"},"spec":{"gatewayClassName":"stunner-gatewayclass","listeners":[{"name":"udp-listener","port":3478,"protocol":"UDP"}]}}
    stunner.l7mp.io/related-gateway-name: stunner/udp-gateway
    # I added following two lines
    service.beta.kubernetes.io/do-loadbalancer-healthcheck-port: "80"
    service.beta.kubernetes.io/do-loadbalancer-healthcheck-protocol: "http"
  finalizers:
  - service.kubernetes.io/load-balancer-cleanup
  labels:
    app: stunner
    stunner.l7mp.io/owned-by: stunner
  name: udp-gateway
  namespace: stunner
  ownerReferences:
  - apiVersion: gateway.networking.k8s.io/v1alpha2
    kind: Gateway
    name: udp-gateway
    uid: f8481e46-507a-4c6d-8102-3c31f2ed4c45
spec:
  allocateLoadBalancerNodePorts: true
  clusterIP: <IP_ADDRESS>
  clusterIPs:
  - <IP_ADDRESS>
  externalTrafficPolicy: Cluster
  internalTrafficPolicy: Cluster
  ipFamilies:
  - IPv4
  ipFamilyPolicy: SingleStack
  ports:
  - name: udp-gateway-udp
    nodePort: 31780
    port: 3478
    protocol: UDP
    targetPort: 3478
  selector:
    app: stunner
  sessionAffinity: None
  type: LoadBalancer
status:
  loadBalancer: {}

Can somebody help what is next step?

Good news is that STUNner "supports" Digital Ocean.
This is the default config for the built-in STUNner healthcheck service:
service.beta.kubernetes.io/do-loadbalancer-healthcheck-port: "8086"
service.beta.kubernetes.io/do-loadbalancer-healthcheck-protocol: "http"
service.beta.kubernetes.io/do-loadbalancer-healthcheck-path: "/live"

Can you please try to apply these annotations instead of the default DO-recommended lines?

Just to give more context, the annotations come from here: https://github.com/l7mp/stunner/blob/main/docs/GATEWAY.md#gateway

Thanks, now stunner is running :-)


Autocomplete doesn't add the final semicolon for objects

Description:
type Person object {

}

The final semicolon is not getting added in auto complete
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Can you check whether you are using the latest and use the snippet for object? This is working in latest master


Fix paideia app sidebar

Paideia APP Sidebar has some issues since it wasn't build using the MUI components, and it isn't inside the container that the main page is in (uses margin to push main container aside). Another issue is the component has a top margin to avoid the DAO Selection section, so when a scrollbar is generated, it doesn't let you scroll all the way to the bottom.
I propose refactoring the layout so the sidebar component uses MUI's drawer, which will make rendering on mobile and desktop easier, and make sure internal components are rendered as flexbox divs rather than using absolute positions, so they move around one another appropriately and scroll properly.

when menu open in mobile sizes, and page is resized, you get a double menu


net: allow not creating a goroutine for netFD.connect in case of caller passes context which is not going to be cancelled manually

What version of Go are you using (go version)?
go1.17.2
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GO111MODULE="" GOARCH="amd64" GOBIN="" GOCACHE="/Users/mikefaraponov/Library/Caches/go-build" GOENV="/Users/mikefaraponov/Library/Application Support/go/env" GOEXE="" GOFLAGS="" GOHOSTARCH="amd64" GOHOSTOS="darwin" GOINSECURE="" GOMODCACHE="/Users/mikefaraponov/go/pkg/mod" GONOPROXY="" GONOSUMDB="" GOOS="darwin" GOPATH="/Users/mikefaraponov/go" GOPRIVATE="" GOPROXY="https://proxy.golang.org,direct" GOROOT="/usr/local/Cellar/go/1.15.6/libexec" GOSUMDB="sum.golang.org" GOTMPDIR="" GOTOOLDIR="/usr/local/Cellar/go/1.15.6/libexec/pkg/tool/darwin_amd64" GCCGO="gccgo" AR="ar" CC="clang" CXX="clang++" CGO_ENABLED="1" GOMOD="/Users/mikefaraponov/go.mod" CGO_CFLAGS="-g -O2" CGO_CPPFLAGS="" CGO_CXXFLAGS="-g -O2" CGO_FFLAGS="-g -O2" CGO_LDFLAGS="-g -O2" PKG_CONFIG="pkg-config" GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/97/d1hwd7t96mj0_jxmpngf2m5w0000gn/T/go-build319243062=/tmp/go-build -gno-record-gcc-switches -fno-common"
What did you do?
Creating TCPConn connection via net.Dialer
What did you expect to see?
No netFD func2 goroutines to be spawned.
What did you see instead?
Each new connection created created netFD goroutine
Solution
net/fd_unix.go
if deadline, hasDeadline := ctx.Deadline(); hasDeadline {
fd.pfd.SetWriteDeadline(deadline)
defer fd.pfd.SetWriteDeadline(noDeadline)
}

// Start the "interrupter" goroutine, if this context might be canceled.
// (The background context cannot)
//
// The interrupter goroutine waits for the context to be done and
// interrupts the dial (by altering the fd's write deadline, which
// wakes up waitWrite).
if ctx.Done() != nil { // skip creation of goroutine and allow fd.pfd.SetWriteDeadline(deadline) to gracefully close this connection on timeout, this allows to pass custom context.Context implementation with deadline but not cancelable or just check some ctx.Value("bypasscreationofgoroutine") instead of ctx.Done() != nil check may be prefereble
// Wait for the interrupter goroutine to exit before returning
// from connect.
done := make(chan struct{})
interruptRes := make(chan error)
defer func() {
	close(done)
	if ctxErr := <-interruptRes; ctxErr != nil && ret == nil {
		// The interrupter goroutine called SetWriteDeadline,
		// but the connect code below had returned from
		// waitWrite already and did a successful connect (ret
		// == nil). Because we've now poisoned the connection
		// by making it unwritable, don't return a successful
		// dial. This was issue 16523.
		ret = mapErr(ctxErr)
		fd.Close() // prevent a leak
	}
}()
go func() {
	select {
	case <-ctx.Done():
		// Force the runtime's poller to immediately give up
		// waiting for writability, unblocking waitWrite
		// below.
		fd.pfd.SetWriteDeadline(aLongTimeAgo)
		testHookCanceledDial()
		interruptRes <- ctx.Err()
	case <-done:
		interruptRes <- nil
	}
}()
}

Please look https://github.com/golang/go/pull/49024

Or https://github.com/golang/go/compare/master...moredure:patch-2 as an alternative without custom context to be used just value marker to bypass this goroutine creation

@moredure thanks for the issue. (For the future) you probably want to mention which package  the issue is about and where the code snippet is that  you changed. Thanks.


onSubmitEditing (feature request)

Hello,
Is there any way to implement the input onSubmitEditing callback? Or a workaround to achieve this behaviour? So when you press the Enter/Next button, the focus moves to the next input field.
I have a registration form with multiple input fields and would like to apply this behaviour for my phone field, but I'm currently not able to achieve this.
Thanks in advance!

It's possible using textInputProps:
const textInputProps={ onSubmitEditing: () => ref.current?.focus() }

return(
  <PhoneInput
    {...rest}
    textInputProps={textInputProps}
  />
)

doesn't work

@domoretrix did you define and set the ref?
const ref = useRef(null)

return <TextInput ref={ref} />

yes. Only onBlur and onFocus works for textInputProps

@domoretrix can you copy/paste your code here?

const textInputProps={ onSubmitEditing: () => tiRef.current?.focus() }

return(
  <PhoneInput
    {...rest}
    textInputProps={textInputProps}
  />
 <TextInput
   ref={tiRef}
 >
)

Try this:

const tiRef = useRef(null)

const textInputProps={
  onSubmitEditing: () => tiRef.current?.focus()
}

return(
<>
  <PhoneInput
    {...rest}
    textInputProps={textInputProps}
  />
 <TextInput
   ref={tiRef}
 />
</>
)

ya, i mean of course const tiRef = useRef(null). I omitted this line for showcase purposes. Point is, adding any other property than onBlur or onFocus on textInputProps does not work.

@10000multiplier sorry, you are actually right, your solution does work. I think my simulator is lagging a bit and sometimes it does not carry the changes, so that's why it was confusing for me.

Try this:
const tiRef = useRef(null)

const textInputProps={
  onSubmitEditing: () => tiRef.current?.focus()
}

return(
<>
  <PhoneInput
    {...rest}
    textInputProps={textInputProps}
  />
 <TextInput
   ref={tiRef}
 />
</>
)

Where do you import this  ?
Thanks!


Aanish/add attempt

Coverage decreased (-0.1%) to 66.526% when pulling 6bce0d5175a3017dd0b31613ed1dea99e8a1acdd on anish531213:aanish/add-attempt into 808265e91c6b0b7730b4425ab7611d75727f97d7 on uber:master.


Please replace toml dependency with tomli

This package still requires toml for old Python versions. That library is unmaintained (last release in 2020), does not implement TOML correctly and has been removed from Linux distributions, making the dependency a blocker for packaging inline-snapshot. Could you please switch over to tomli instead? It's 100% compatible and interchangeable with the stdlib tomllib, so replacement should be trivial.

Thank you for the info. I will fix it.

fixed in 0.13.4

Thank you.


Handle postgres bools and use params in transact

Currently any params to transact are ignored. This leads to getting an exception thrown when trying to do something like
(db/transact conn
               '[:update orders
                 :set    [orders/retrieved true]
                 :where  [orders/order-id ?oid]]
               {:oid order-id})

Also, postgres bools are not recognized. Any boolean values in a query result are false.
This pull request fixes them both. However I did not test with sqlite, so I might have broken something.

Oh interesting, yes, the boolean handling only worked on sqlite, but this will work with both now.
That transact bug is pretty bad, thanks for taking the time to fix it!


fix green and blue color components assignment in handle_osc

hm, what problem does this fix?

hm, what problem does this fix?

For xterm or any other term that report background via OSC with
      {lead}{code};rgb:{rrrr}/{gggg}/{bbbb}{tail}
or    {lead}{code};rgb:{rr}/{gg}/{bb}{tail}

color components (red, green and blue) are used to detect whether to use light or dark background.
So we have
		    rval = hexhex2nr(tp_r);
		    gval = hexhex2nr(tp_g);
		    bval = hexhex2nr(tp_b);
#endif
		    if (is_bg)
		    {
			char *new_bg_val = (3 * '6' < *tp_r + *tp_g +
					     *tp_b) ? "light" : "dark";

where green and blue components are mixed, most probably by mistake.
It shouldn't matter that much as, I guess, most of xterm backgrounds users set the same red, green and blue for the chosen color (#000000, #d0d0d0, #1c1c1c etc) but there might be ppl using less plain colors, e.g. #3faf5f (imaginary example).

I see thanks!


add Callipepla accelerator

add an open source accelerator named Callipepla to -Designs & Generators - Accelerators

Where is the open source hardware part?

Where is the open source hardware part?

Where is the open source hardware part?

Click the link to the Callipepla repo, https://github.com/linghaosong/Callipepla, the open source hardware is under Callipepla/Callipepla/bitstream, Callipepla/Callipepla/src, Callipepla/SerpensCG/bitstream, Callipepla/SerpensCG/src


repository as template

seeing stuff like #20 and #24, i feel like i should be separating my content and actual site, and turning the repository as a template. this would involve:

renaming the current repository
creating a new "12beesinatrenchcoat.github.io" repository
idk maybe some other stuff

i'll consider doing this after the rest of v6.0. we'll see.

i'll do this once #23 is done.

current idea on how to do this:

new branch, with-content, site is built off this branch
all personal content removed from master
make repository a template

will do tomorrow. i'm tired.


allow lua expressions in templates

allows using lua expressions for string manipulation during replace. Example:
-- ORIGINAL TEXT:
local foo_bar = 3

-- SEARCH:
local $var = $val

-- REPLACE:
local $(snake_to_camel_case(var) .. val) = $val

-- RESULT:
local fooBar3 = 3

things to note:

var and val are exposed as "readonly" global in the lua expression - but existing globals will take priority (lua limitation)
snake_to_camel_case is user defined

this probably counts as a breaking change
I'm curious to see what others think - if you have suggestions let me know

TODO:

figure out error handling
handle nil return
how to handle indent?

nice to have:

extend the standard string lib with common functions such as toCamelCase etc. would love to be able to write $(var:toCamelCase()) ootb
allow swapping out for fennel. I don't use fennel myself but the pipe operator is so good here


Create a conda recipe and package

Note: To be done after v0.1.0 is released

See https://github.com/conda-forge/staged-recipes/pull/20716

Thanks @dalthviz!

Closing since the package is already available on conda-forge: https://anaconda.org/conda-forge/spyder-vim


H.264 / AVC / MPEG-4 AVC / MPEG-4 part 10 play failed

Version
Media3 1.5.0
More version details
No response
Devices that reproduce the issue
Xiaomi 13
Devices that do not reproduce the issue
No response
Reproducible in the demo app?
Yes
Reproduction steps

Add test url to media.exolist.json
Open AndroidManifest.xml and add android:usesCleartextTraffic="true" to application
Run demo

Expected result
Play well as mpvplayer
Actual result
No errors and tracks
But TsExtractor detected H264 and AC-3
Media
http://httplive.slave.bfgd.com.cn:14311/playurl?playtype=live&protocol=http&accesstoken=R5F2408FEU3198804BK78052214IE73560DFP2BF4M340CE68V0Z339CBW1626D4D261E46FEA&playtoken=ABCDEFGH&programid=4200000501
Bug Report

[ ] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

Try DefaultExtractorsFactory with FLAG_ALLOW_NON_IDR_KEYFRAMES
DefaultExtractorsFactory extractorsFactory = new DefaultExtractorsFactory()
  .setTsExtractorFlags(FLAG_DETECT_ACCESS_UNITS)
  .setTsExtractorFlags(FLAG_ALLOW_NON_IDR_KEYFRAMES);


[ion-side-menus] drag-content="false" only works on first tab.

I'm developing an app using ion-side-menus which has ion-tabs under ion-side-menu-content.
I want to disable drag-content functionality, so I set the attribute to false.
<ion-side-menu-content drag-content="false">

Yes it works, but only on the first tab if ion-side-menu-content has an ion-tabs inside it.
Here is a codepen: http://codepen.io/hiroshi-kuwabara/pen/MYeorv
How to reproduce:

In a "Home" tab, drag content left or right, and side menu doesn't open (drag-content="false" is working).
In "About" tab, or "Contact" tab, drag content left or right, then side menu opens.

Will look at the PR tomorrow

hi @mhartington, please let me know if you would like me to raise a separate issue for this, but wanted to give heads up as you mention you will be looking into this fix.
I applied the PR fix above, and now the content respects the 'drag-content="false"' attribute, however when arriving from a separate (non-tab) login state, the attribute edge-drag-threshold=.. on 'ion-side-menu-content' is ignored; you can drag from anywhere in the page.
Codepen forked from above below demonstrates:
http://codepen.io/halfpt/pen/LELbXa

@kuwabarahiroshi #2752 was merged, can you confirm your issue was resolved by testing the nightly builds?
@halfpt please open a new issue for this.

@mhartington Thanks a lot!
I confirmed that the issue is resolved with the nightly builds.
I tested it on Chrome(Mac), iPhone, Android.

I still face this issue, I have disabled caching completely so that my controllers are executed everytime.
In my controller I have $ionicSideMenuDelegate.canDragContent(false);
It does work for the first time, but after switching few states and returning back to the state which invokes that particular controller, but  $ionicSideMenuDelegate.canDragContent(false); doesn't stop it anymore.

@ritvick : Have you tried  - $scope.$on('$ionicView.afterEnter', function(){}), I have used this in the past?

Yes thats what I was thinking, I have  $ionicSideMenuDelegate.canDragContent(false); defined in my controller, but I think this is executed after the view is loaded and then it does not have any effect on that view. Though if I move to other view then it works as expected.

Where I shall define $scope.$on('$ionicView.afterEnter', function(){}) ?
Inside run function ?
Do I need to specify $ionicSideMenuDelegate.canDragContent(false); inside above mentioned inline function ?

The problem is still there in v1.0.0-rc.2-nightly-1177. Only the first tab works ok with edge-drag-threshold="true", the next tabs I click ignore edge-drag-threshold. If I go back to the first tab it still works ok. Tested on Android 5.0.2.

Can you all please provide a codepen or demo and test it against the nightly builds?

I know it's not exactly the same problem but some kind of similar one:
I'm on an app based on the principle of another app which goes around here and does not show the tab bar before "Sign in" is clicked.
Now the combination of this app and the side menu starter app brings back this problem: As long as you are not on the tab page everything is fine but as soon as you open the view with the tab bar, it's possible again to swipe in the side menu (found out because it interferes with your image slider).
Thanks for any feedback!

hi,
while    works great.
but if i use it on the tab where map is there then map event listeners (like drag, dragstart, dragend etc) are not working.
i would like to design in such a way that on that tab when user drags the map i can listen to the map even listeners and at the same time disable the ionic side menu drag content.
how to achieve that?
please help.
regards,


[Event Request] Report 99001025 - Refresh Production Order - OnBeforeModifyRoutingNo

Hi, would it be possible to have an event just before modify the Routing No in the Report Refresh Production Order ?

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Update to use ES Modules

Updates code to use modern import / export syntax, as well as removing 'use-strict' as strict mode is enabled automatically for ES Modules.

why are you do that?

why are you do that?

Because it is more readable.


validations, model specs, and n+1 problems

Add the following validations:

For the User model:

Name must not be blank.
PostsCounter must be an integer greater than or equal to zero.

For the Post model:

Title must not be blank.
Title must not exceed 250 characters.
CommentsCounter must be an integer greater than or equal to zero.
LikesCounter must be an integer greater than or equal to zero.

Add unit specs for all of your models' methods and validations.
Add flash messages in the create actions in all your controllers.
Make sure that the N+1 problem is solved when fetching all posts and their comments for a user.

Dear Reviewer,
Kindly note that implementing methods for update_likes_counter update_posts_counter and update_comments_counter separately won't be possible because they are private methods and regarding creating rails db:create everything is working fine from my end.


degree is not order in splines

The following explanation of parameters said k is the B-spline order, while here it uses degree, but degree is different from order, and they satisfy order = degree + 1. The same issue https://github.com/scipy/scipy/pull/11861 has been corrected.

Reference issue

What does this implement/fix?

Additional information

Thanks


TypeError: newOpts.merge is not a function

I'm using angular2-cookie v1.2.6.  I am now getting the following error (was working in previous versions):

TypeError: newOpts.merge is not a function
at CookieService._mergeOptions (cookies.service.js:194)
at CookieService._buildCookieString (cookies.service.js:167)
at cookies.service.js:152
at CookieService.put (cookies.service.js:83)
at LoginControlComponent.saveToken (login-control.component.ts:105)
at login-control.component.ts:76
at ZoneDelegate.invoke (zone.js:242)
at Object.onInvoke (ng_zone.js:269)
at ZoneDelegate.invoke (zone.js:241)
at Zone.run (zone.js:113)
at zone.js:520
at ZoneDelegate.invokeTask (zone.js:275)
at Object.onInvokeTask (ng_zone.js:260)
at ZoneDelegate.invokeTask (zone.js:274)
at Zone.runTask (zone.js:151)

Here is the code:
`saveToken(token: string, loginEmail: string) {
let dtm = new Date();
let minutes = 115;
dtm.setTime(dtm.getTime() + (minutes * 60 * 1000));
this.cookieService.put('tokenCookie', token, { expires: dtm, path: '/' });
    dtm = new Date();
    dtm = this.nk.dtm.addDays(dtm, 365);
    this.cookieService.put('loginEmailCookie', loginEmail, { expires: dtm, path: '/' });

}`

Is there something wrong with my code?  Or is this a bug?  Thank for any help on this.

Same here on Angular 2.4.1. It works when not passing options as the 3rd argument of .put().

This line causes the error.

@nukuuk I think this is related to https://github.com/salemdar/angular2-cookie/issues/37. I think the culprit here is my temporary fix. Since we provided a blank object for the CookieOptions. It throws the error about .merge() is not a function and the class CookieOptions defined the .merge() function.
What do you think of this @salemdar ?

@exequiel09  I am using the fix as mentioned in #37 thread:
{ provide: CookieOptions, useValue: {} }
But still get this 'newOpts.merge is not a function' error.  However, seems the culprit certainly is CookieOptions.

@nukuuk Checkout the new solution on #37 but that will only work on JIT not on AOT.

All solution Doesn't work for me. when i don't add options it work. when i add options it doesn't work.

@Spawnrad
Works fine for me, with Angular 2.4.8, BUT without a 3rd parameter for the .put method (so, this means not being able to set an expiration date):
In app.module.ts:
import { CookieService, CookieOptions } from 'angular2-cookie/core';

// ...

providers: [
    // ...
    CookieService,
    { provide: CookieOptions, useValue: {} }
]

@MrCroft
Could I ask if there is another way to include expiration dates? Many thanks!

@SyangC Sorry, haven't figured out any, yet... Luckily for me, don't really need it now.

Damn. Thanks anyways!

@MrCroft @SyangC @Spawnrad
I find a way.
use
{ provide: CookieOptions, useValue: false }

Please use the new library guys:
https://github.com/salemdar/ngx-cookie

I get the same with the other library also when using AOT.

the new library introduced a new error and is not working with angular 5


Mark the Coalesce function as asynchronous to account for lazy evaluation

Since we made the Coalesce function lazy, we need to mark it as asynchronous so that it may need to block while evaluating some of its arguments, since they can be asynchronous themselves.

✅ No public API change.


introduce cloud logging

/gcbrun exp -n dgkbbb21 -m vertex_ai_gemini-1-5 -i -b minor-for-ci

/gcbrun exp -n dgkbbb22 -m vertex_ai_gemini-1-5 -i -b minor-for-ci

the logs are looking good now

Thanks for fixing this!


Unable to generate a TensorRT GeneralizedRCNNTransform model with a dynamic batch

🐛 Describe the bug
I'm trying to generate a TensorRT engine of RetinaNet that uses GeneralizedRCNNTransform. By bypassing a couple of layers, it works fine for a static batch size. However, when using a dynamic batch size, I get an issue that I don't know how to fix.
In class GeneralizedRCNNTransform(nn.Module), there is this part in the forward method:
for i in range(len(images)):
    image = images[i]
    target_index = targets[i] if targets is not None else None

    if image.dim() != 3:
        raise ValueError("images is expected to be a list of 3d tensors "
                            "of shape [C, H, W], got {}".format(image.shape))
    image = self.normalize(image) # here are where the sub and div nodes comes from
    image, target_index = self.resize(image, target_index) # I'm bypassing this
    images[i] = image
    if targets is not None and target_index is not None:
        targets[i] = target_index

When converting my model with:
torch.onnx.export(
    model,
    example,
    onnx_model_path.split('.')[0] + '_dynamic.onnx',
    verbose=False,
    opset_version=11,
    input_names=['input'],
    dynamic_axes={
        'input': {0: 'batch_size'}
    }
)

I get the following tree:

Which is prefectly fine for the given example batch size (3 in this case), but this tree will not work for any other batch size once it is converted to a TensorRT engine. I get this kind of error:
[12/10/2021-13:29:40] [TRT] [E] 7: [shapeMachine.cpp::execute::565] Error Code 7: Internal Error (Split_0_0: ISliceLayer has out of bounds access on axis 0
condition '<' violated
Instruction: CHECK_LESS 1 1
)
[12/10/2021-13:29:40] [TRT] [E] 2: [executionContext.cpp::enqueueInternal::366] Error Code 2: Internal Error (Could not resolve slots: )

Is there any way I can bypass this for loop which is creating the split and concat nodes? Do I need to redefine the forward method? I can't really bypass the normalisation, or else I will loose in accuracy. Can I maybe normalize it all at once without that for loop?
Thanks 😄
Versions
Collecting environment information...
PyTorch version: 1.10.0
Is debug build: False
CUDA used to build PyTorch: 10.2
ROCM used to build PyTorch: N/A
OS: Ubuntu 18.04.6 LTS (aarch64)
GCC version: (Ubuntu/Linaro 7.5.0-3ubuntu1~18.04) 7.5.0
Clang version: 6.0.0-1ubuntu2 (tags/RELEASE_600/final)
CMake version: version 3.10.2
Libc version: glibc-2.25
Python version: 3.6.9 (default, Mar 15 2022, 13:55:28)  [GCC 8.4.0] (64-bit runtime)
Python platform: Linux-4.9.253-tegra-aarch64-with-Ubuntu-18.04-bionic
Is CUDA available: True
CUDA runtime version: Could not collect
GPU models and configuration: Could not collect
Nvidia driver version: Could not collect
cuDNN version: Probably one of the following:
/usr/lib/aarch64-linux-gnu/libcudnn.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_adv_infer.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_adv_train.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_cnn_infer.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_cnn_train.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_ops_infer.so.8.2.1
/usr/lib/aarch64-linux-gnu/libcudnn_ops_train.so.8.2.1
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
Versions of relevant libraries:
[pip3] numpy==1.19.5
[pip3] torch==1.10.0
[pip3] torch2trt==0.3.0
[pip3] torchvision==0.11.1
[conda] Could not collect
cc @datumbox @YosuaMichael

@aurelien-m Thanks for reporting.
This is not currently a use-case we support. It's hard to guide you to the best option because I don't have deep expertise on TensorRT and ONNX. We could potentially look into this on the future, but currently it's hard due to our limited resources. Apologies I can't provide better assistance at this point.


MDS-16 Add a java document

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Kouji Ijima seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: [#378] [Bug] InputArray returns a wrong value when requesting by form

Closes https://github.com/goravel/goravel/issues/378
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Codecov Report
Attention: Patch coverage is 85.45455% with 8 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (v1.1.x@29df926). Click here to learn what that means.

Files
Patch %
Lines

context_request.go
85.45%
4 Missing and 4 partials :warning:

Additional details and impacted files
@@            Coverage Diff            @@
##             v1.1.x      #68   +/-   ##
=========================================
  Coverage          ?   78.94%
=========================================
  Files             ?       12
  Lines             ?      907
  Branches          ?        0
=========================================
  Hits              ?      716
  Misses            ?      165
  Partials          ?       26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tests failed


Remove .gitignore from npm package

This allows projects to include their dependencies into their repository
if they choose, and not have it ignore the nested dependencies
Could possibly remove benchmark/ and tool/ as well?

+1
you can read about the motivation here: http://www.mikealrogers.com/posts/nodemodules-in-git.html

isaac has talked about handling this for all modules in npm itself

@glenjamin awesome, can you give me a link to his thoughts?

found it! https://github.com/isaacs/npm/issues/1862


Reconciled all waffle flags for edxapp deployments.

What are the relevant tickets?
#1935
Description (What does it do?)
This will set up the consul keys for managing waffle flags in our edxapp environments. Once these keys are in place, an automated process at the startup of any new web instance of edxapp will run a script that sets the flags as specified in this config.
Screenshots (if appropriate):

[ ] Desktop screenshots
[ ] Mobile width screenshots

How can this be tested?
Should be carefully verified vs existing flags.
Additional Context

CC @pdpinch


[fix] gws schedule some issues 2

Coverage decreased (-28.0%) to 56.066% when pulling b2cd92fcbddf2684da1318695d03cc612ff443da on fix-gws-schedule4 into bc9ff65df933f1aa2243e407dadf8cfffea9c824 on master.


AttributeError: module 'tensorflow' has no attribute 'ConfigProto'

when I run task1a, it reported the following error：
[E] Uncaught exception  (logging.py:221)
Traceback (most recent call last):
File "task1a.py", line 1297, in
sys.exit(main(sys.argv))
File "task1a.py", line 206, in main
processed_items = do_learning(
File "task1a.py", line 680, in do_learning
dcase_util.keras.setup_keras(
File "/root/anaconda3/lib/python3.8/site-packages/dcase_util/decorators/decorators.py", line 14, in call
return self.f(*args, **kwargs)
File "/root/anaconda3/lib/python3.8/site-packages/dcase_util/keras/utils.py", line 302, in setup_keras
config = tf.ConfigProto(
AttributeError: module 'tensorflow' has no attribute 'ConfigProto'
Package version:
dcase_util 0.2.12 / dcase_util 0.2.16
keras 2.4.3
tensorflow 2.3.1
Python 3.7.4
I found that it may caused by the mismatched package version, so I installed tensorflow==1.14.0. And then another error was reported. It is shown as following:
Traceback (most recent call last):
File "task1a.py", line 1297, in
sys.exit(main(sys.argv))
File "task1a.py", line 211, in main
overwrite=overwrite
File "task1a.py", line 860, in do_learning
shuffle=param.get_path('learner.parameters.fit.shuffle')
File "/usr/local/python3/lib/python3.7/site-packages/keras/engine/training.py", line 1178, in fit
validation_freq=validation_freq)
File "/usr/local/python3/lib/python3.7/site-packages/keras/engine/training_arrays.py", line 200, in fit_loop
callbacks._call_batch_hook('train', 'begin', batch_index, batch_logs)
File "/usr/local/python3/lib/python3.7/site-packages/keras/callbacks.py", line 84, in _call_batch_hook
batch_hook = getattr(callback, hook_name)
AttributeError: 'ProgressLoggerCallback' object has no attribute 'on_train_batch_begin'
I also tried other suggestions for these problems, but it can't be solved. I appreciate if you can give me some suggestions.

The baseline system was originally developed for TensorFlow v1.x. You were using v2.3.1, which has a slightly different API. The error is coming from the dcase_util library which currently has utilities only for TensorFlow v1.x. Utilities for TensorFlow v2.x will be finally released in the upcoming version v0.2.17 of this library shortly.
The error coming after you downgraded to TensorFlow v1.14.0 is caused by the mismatched Keras version. The attributes such as on_train_batch_begin were introduced to Keras callbacks in v.2.2 and dcase_util versions <= 0.2.16 work only with Keras <= 2.1.x.

Just a couple of minor updates as for May 2021:
The error above is triggered if someone (oops) installs the requirements.txt from a fresh conda environment. Those aren't meant for that case, right? The conda instructions should be followed instead. Still, there were a couple of changes I needed to make to get it up: Librosa needed the forge channel and libsndfile was missing:
conda install -c conda-forge librosa
conda install -c conda-forge libsndfile

Tested on a fresh 3.6 conda environment, Linux server with Nvidia driver 450 and worked like a charm.
Anyway, it seems that pip installing dcase_util already upgrades the conda librosa, so maybe that one could be dropped altogether.
Cheers,
Andres


[release-4.9] Add missing control plane prometheus rules

What this PR does / why we need it:
Manual cherry-pick of https://github.com/openshift/hypershift/pull/1406
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #1201
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

@csrwng Updated. Amended the changes to the original commit.

/lgtm

/approve


Fix gemspec

When I tried to install the bundler gave the follow error:
Bundler could not find compatible versions for gem "i18n":
In Gemfile:
html-proofer (~> 3.10) was resolved to 3.11.0, which depends on
activesupport (< 6.0, >= 4.2) was resolved to 5.2.3, which depends on
i18n (< 2, >= 0.7)
wax_theme was resolved to 1.0.0, which depends on
  jekyll (~> 3.8) was resolved to 3.8.6, which depends on
    i18n (~> 0.7)

Bundler could not find compatible versions for gem "ruby":
In Gemfile:
ruby
wax_theme was resolved to 1.0.0, which depends on
  wax_tasks (< 1.1, >= 1.0.1) was resolved to 1.0.2, which depends on
    ruby (>= 2.4)

Nevermind. See that my Ruby version is not 2.4. You should really think about adding a vagrant box to this repo so users don't have to install exact versions of the dependencies. It would make this much more usable.


SCSS doesn't appear to "climb the tree", only shows 1 level of selector

Given the following SCSS:
.root {
  .child {
    .grandchild {
      color: #CCCCCC;
    }
  }
}

When the cursor is on the color rule, ctags-status will report [.grandchild], not [.root > .child > .grandchild]. Is this the expected behavior? Is there a trick to make it "climb the tree" to show the full selector ancestry?

Did you select to option "Show outer scope(s)"?

Thanks, but at this point, it just says "[global]" all the time, never changes at all. I'm not sure what I've done wrong, but as I was actually looking for a status-line display of the calculated scss selector at the insertion point, I'm not sure that ctags-status is the tool for me. Thank you for your help, but I think you can close this issue.

Hey @dmland, which OS are you using? And, did the screencap above tell you if ctags-status really fits your use case?

I'm on Mac OS X 10.9.5.
The screencap suggests that it would go part of the way to fitting my use case.
As long as we're discussing blue skies, ideally, a full implementation would be more like this (and I apologize for starting off with a too-simple example):
.root, .trunk {
  .child {
    .grandchild {
      color: #CCCCCC;
    }
  }
}

Would yield [.root .child .grandchild, .trunk .child .grandchild]
Two things are different from the current behavior:

no right-angle indicators of "climbing the hierarchy," because they have a very specific meaning in CSS
.root, .trunk generates two separate selectors. It's a tree with two roots, I guess.

Re 1: Good point. '>' doesn't make sense in CSS/SCSS/etc.
Re 2: Yes, ctags found two node on that line (.root and .trunk).
CSS/SCSS/etc. support is still pretty beta to be honest. I am still looking at if ctags can work better with these languages.


Types missing

Seems like tpyes arent being exported properly or perhaps Im missing something when I install this and try to use it in a typescript project I get the following:
  Try `npm i --save-dev @types/sebastianwessel__quickjs` if it exists or add a new declaration (.d.ts) file containing `declare module '@sebastianwessel/quickjs';`ts(7016)```

There should be no @types/sebastianwessel__quickjs needed.
The package should contain the types already

Yes Im aware, but from installing this package Im being told that no types are found. Thats just the standard error if no types are found in a package so for whatever reason it seems like they arent being exported properly by default

You're right! Thanks for your feedback 🙏
There was an issue in the configuration. I have published a fixed version, 1.1.1, which should resolve the issue. There should be no additional type import necessary now.

Thanks that fixed it


Do not add '-isystem /usr/local' to gcc's CXX_INCLUDES flag

Fixes: #195
Refs: pmem/libpmemobj-cpp#508

And maybe TBB should enable NO_SYSTEM_FROM_IMPORTED? See the fix: pmem/libpmemobj-cpp#514

@ldorau removal of INTERFACE_INCLUDE_DIRECTORIES property from the TBB targets will likely lead to compilation errors due to missed TBB headers.

And maybe TBB should enable NO_SYSTEM_FROM_IMPORTED?

TBB doesn't touch NO_SYSTEM_FROM_IMPORTED property, it relies on CMake default behavior. Moreover, looks like there is no way to change the behavior from TBB side, because it works for consuming targets. From CMake documentation for NO_SYSTEM_FROM_IMPORTED:

Do not treat include directories from the interfaces of consumed Imported Targets as SYSTEM.

Here is a discussion of this topic: https://gitlab.kitware.com/cmake/cmake/issues/17348
Since

This property is initialized by the value of the CMAKE_NO_SYSTEM_FROM_IMPORTED variable if it is set when a target is created.

https://github.com/pmem/libpmemobj-cpp/pull/514 seems to be a proper solution for the issue.

I wonder why only TBB (no any other component used by libpmemobj-cpp) needs CMAKE_NO_SYSTEM_FROM_IMPORTED to be enabled? What does TBB do that it requires that?

I can assume that TBB uses some system headers which lead to the issue, but they are not used by other components. BTW, from the error log in https://github.com/pmem/libpmemobj-cpp/issues/508 I see inclusion of test headers and system headers. Have you checked that it is not an issue in test headers?

If you normalize path before setting it to INTERFACE_INCLUDE_DIRECTORIES then cmake will be able to handle it properly in all cases.
get_filename_component(COMPONENT_INCLUDE_DIRECTORY "${CMAKE_CURRENT_LIST_DIR}/@TBB_INC_REL_PATH@" ABSOLUTE)
set_target_properties(TBB::${_tbb_component} PROPERTIES
                    INTERFACE_INCLUDE_DIRECTORIES "${COMPONENT_INCLUDE_DIRECTORY}")

@Aldrog thank you very much! I was able to reproduce and fix it with your approach. I'll prepare a patch.

This is not a good fix for #195. Please continue discussion in #195.


Accessibility: Add BoundingRects to UiaTextRanges

Summary of the Pull Request
Add the visible rectangle indicators to text ranges. They are now sized appropriately and are located in their correct position. This supports panes, and makes them fairly accessible to use.

References
🚨 Target branch is "Refactor Accessibility Providers" #2414

PR Checklist

[x] Closes #2103
[x] CLA signed.
[x] ~Tests added/passed~
[x] ~Requires documentation to be updated~
[x] I am a core contributor.

Detailed Description of the Pull Request / Additional comments
Thanks to that refactoring, this code is fairly easy to read. There are a few additional refactoring changes throughout this commit.
The core of these changes are in UiaTextRange::_TranslatePointToScreen(). This is where we translate the RECTs to the correct size and position.

Size: we needed the current font size. Acquired through ScreenInfoUiaProvider --> TermControl

🚨 NOTE: using IRenderData's is incorrect because Terminal has a fake one there.

Position:

using ScreenInfoUiaProvider's get_BoundingRectangle to leverage XAML's knowledge of where we are on the screen

XAML already uses this function to create a box around the TerminalControl. Let's just borrow that.

using TermControl's padding value to adjust even further.

Validation Steps Performed

Create various panes
Make one of the panes have default font size, another with larger, and another with smaller
For each pane, use inspect.exe to test as follows:

get document range
do bounding rects align properly with the text ✔
create a selection
get selection range
do bounding rects align properly with the selection ✔

verify that each pane creates bounding rects of different sizes for the text.

Rebased off of the updated base branch.
Heads up, I'll be sure this goes into master as a separate commit from the base branch after the base branch goes into master. Just did this for the better diff :)

I approve of this PR, but I'm not totally sure that GH will let you just change the target branch of a PR. You might need to make another PR once #2414 is in master. Either way, count this as my signoff on this diff

I know if you click the "Edit" button at the top, you can retarget. Hopefully, the approvals transfer over. But this'll be a learning experience for all of us #DayOfLearning haha


Bug with Windows paths and adds CHANGELOG files

This PR resolves #44

Ah yeah! Forgot to mention I tested it on my Windows Bootcamp partition :)


Add jenkins-schema.sh to verify contacts-frontend against the schemas

As noted in https://github.com/alphagov/contacts-frontend/pull/37#issuecomment-163666678. Copied from contacts-admin, with some words changed to refer to contacts-frontend.

Something needs to call this script too

@boffbowsh We've just made the govuk_contacts_frontend_schema_tests job in CI and added it to the triggers on the main content_schemas master and branches jobs.

I've just tested whether the new job gets run on CI, and it doesn't seem to at the moment.

Ah, I am mistaken.

@issyl0 @boffbowsh I think the jenkins script is failing:
https://ci.dev.publishing.service.gov.uk/job/govuk_contacts_frontend_schema_tests/4/console
FWIW, in collections-publisher the jenkins-schema.sh calls the normal jenkins.sh to run the tests:
https://github.com/alphagov/collections-publisher/blob/master/jenkins-schema.sh
Might be useful here too.

Interesting, thanks for bringing this up, @tijmenb. I thought Jenkins would email me about failures. I'll look into it on Monday.


Fix crash when choosing image from downloads on Android using image_picker

Fix for:
https://github.com/flutter/flutter/issues/21863
Changes:

The content uri content://downloads/public_downloads doesn't always work, so need to check other options too.
Added three options public_downloads, my_downloads, all_downloads that will be checked after which the method returns null.
Added a catch for Exception class, because IllegalArgumentException is thrown for this case

Please refer https://stackoverflow.com/questions/30671548/unknown-url-content-downloads-my-downloads as the changes I've done are almost identical to the solution to this answer.
Testing:

Tested on Android emulator 9.0 (was crashing before this change)
OnePlus 5T 8.1.1 (was crashing before this change)

@googlebot I signed it!

Thank you for the PR.
This issue has been fixed by https://github.com/flutter/plugins/pull/1089. I am closing the PR now.
If you still encounter the crash or related issues, please feel free to open an issue or reopen the PR with details.


Replace usages of Atomic* for Atomic*FieldUpdater

Replaced some usages of AtomicInteger, AtomicLong and AtomicReference for their Atomic*FieldUpdater counterpart on classes which get instantiated a lot, this should reduce memory usage for the broker and clients which use lots of topics and consumers.

Change looks good, though there are few tests failing in travis build. eg:
2017-02-08 17:29:48,710 - INFO  - [main:TestListener@53] - @@@@@@@@@@@@@ STARTING TEST - com.yahoo.pulsar.broker.service.ResendRequestTest / testExclusiveCumulativeAckedNormalTopic -- attrs: []
2017-02-08 17:30:19,760 - ERROR - [main:TestListener@41] - ------------ Test Failed -

@merlimat couldn't reproduce this errors on my machine, but I did realize that Travis is running extremely slow, maybe this is producing the failures?

Do you know of any performance impact in switching to this, since it uses reflection?
For the build issue, Can you restart and see if it succeeds? Travis build env is super slow.

@saandrews it shouldn't have any performance impact that I know of, in fact it should benefit performance, since it reduces memory usage, and so gc pauses.
I cannot restart the build in Travis, maybe @merlimat can

Do you know of any performance impact in switching to this, since it uses reflection?

@saandrews  The reflection is only used at static load time of the class. Using the updater is actually faster because is treated as an intrinsic operation by jvm (apart from the spared allocation).

@sschepens

I cannot restart the build in Travis, maybe @merlimat can

After restarting the build: It took around 49 mins and then it has been terminated as it reached max time-limit of build-job: https://api.travis-ci.org/jobs/199668485/log.txt?deansi=true
Do you want to check it or I can trigger another time.

@rdhabalia I will indeed check it

@saandrews @merlimat @rdhabalia the issue seemed to be that I accidentally defaulted OwnedBundles to isActive = false.
Tests should run now.

@merlimat @rdhabalia I'm investigating test failues.
Apparently the issue seems to be that we're creating spies of Producer
This creates a new instance of a class, and so when updating the state it actually updates the internal state of the spy and not of the original producer.
Maybe we can change the test to not spy on the producer, or have all changes to state go through methods so they get proxied to the original instance.
What do you guys think?

@merlimat @rdhabalia I managed to fix the failing tests making spies proxy method calls to original instances, please check the tests I modified


Levenshtien Transformer Performance

Hi I ran the Levenshtein Transformer Model on Both Original and Distilled Dataset.However I GET bleu of 23.87 on DIstilled after 230000k steps and training using fp--16 on single GPU without using --freq-update for GPU Simulation and 23.68 on original dataset after 260000k on 4 GPUs without fp--16.
What will I need to do to achieve the paper performance of 25.20 and 27.3 using Teacher Rescorer.
ALso when I dont give Batch size while evaluation The BLEU4 is 23.87 and 23.68.It takes 250 as Default batch size. However if I mention Batch size 1 the BLEU DROPS to 4.16 and speed to 7.14 sentences/Second.
WHY SO?
@kahne @MultiPath @myleott
Hereby Giving the outputs
Result with Batch size 1
| Translated 3003 sentences (38803 tokens) in 389.2s (7.72 sentences/s, 99.69 tokens/s)
| Generate test with beam=1: BLEU4 = 4.30, 51.4/16.2/8.4/4.6 (BP=0.320, ratio=0.467, syslen=30151, reflen=64504)
Result with Default Batch size of  250
| Translated 3003 sentences (84361 tokens) in 9.3s (321.44 sentences/s, 9030.04 tokens/s)
| Generate test with beam=4: BLEU4 = 23.87, 56.5/29.9/17.7/10.9 (BP=1.000, ratio=1.000, syslen=64536, reflen=64506)

@spprabhu  We recently found there is a new bug introduced by a recent refactor for batch size=1 decoding. Please update the code.

Ah, the "needs triage" label gets added automatically by the new Issue templates

Sounds like 5d7a81099462e9f19715ce5fa37c03816a750e12 should resolve the problem.  Closing...

@spprabhu Have you reproduced the result reported in the paper?

Yes, But I didn't get the same results.
I could translate 3003 sentences in 9.3 sec with test beam size 4 and BLEU4 = 23.87
But I could only train it for 230000 steps and I used 1V100 using --fp16
The training lasted for about 15 hrs

Also if you want to translate just a single sentence at a time the BLEU decreases significantly.

Hi @alphadl did you get the exact results


Should we make angular the default standard convention?

Previous discussion #82 and #53 but please keep this discussion of the topic in this thread.

A bit of background:
Currently if no preset is used, it defaults to no convention so it just list the commits nicely.


Move block_backward_sync from ParallelPlugin to DDPPlugins

What does this PR do?
Parallel plugin should be generic and self-contained. Move reference to DistributedDataParallel from parallel block_backward_sync
Subtask 2 for "remove reference to DistributedDataParallel from parallel plugin"
Subtask 1 is https://github.com/PyTorchLightning/pytorch-lightning/pull/8943
Tests:
python -m pytest -v tests/plugins                     passed
python -m pytest -v tests/accelerators           passed
python -m pytest -v tests/trainer           passed

Fixes #<issue_number>
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@carmocca @awaelchli, how about this one? :otter:

This has been implemented for Lite in https://github.com/Lightning-AI/lightning/pull/14966. It'll eventually trickle into the PL interfaces as we merge implementations. cc @awaelchli


Duplicated auth_seq_id (auth_seq_num)

Several entries have duplicated auth_seq_id:
Here is an example from PDBDEV_00000051:
loop_
_pdbx_poly_seq_scheme.asym_id
_pdbx_poly_seq_scheme.entity_id
_pdbx_poly_seq_scheme.seq_id
_pdbx_poly_seq_scheme.mon_id
_pdbx_poly_seq_scheme.pdb_seq_num
_pdbx_poly_seq_scheme.auth_seq_num
_pdbx_poly_seq_scheme.pdb_mon_id
_pdbx_poly_seq_scheme.auth_mon_id
_pdbx_poly_seq_scheme.pdb_strand_id
<...>
K1 1 908 VAL 1032 1032 VAL VAL K1
<...>
K1 1 1032 GLU 1032 1032 GLU GLU K1

It seems that it doesn't violate a dictionary format, but doesn't it violate a policy? @brindakv

It introduces a lot of problems for the analysis with tools relying on auth_seq_id (e.g., molprobity)

See the examples attached.
PDBDEV_00000053.log
PDBDEV_00000051.log
PDBDEV_00000052.log

@aozalevsky auth_seq_id can be standardized if it is present in atom_site. If it is not, then it is difficult to fix it.
Using it in pdbx_poly_seq_scheme when it is not present in atom_site is misleading. The correction here would be to remove it completely (or replace with a ?). Would molprobity be able to handle that?


[ConstraintSystem] Support solving expression patterns via injecting call to ~= operator

Augment the constraint solver to fallback to implicit ~= application
when member couldn't be found for EnumElement patterns because
case statement should be able to match enum member directly, as well
as through an implicit ~= operator application.

@swift-ci please smoke test

@swift-ci please smoke test macOS platform

Going to re-run macOS before merging just in case anything changed...


[e2e]: generic workflow_dispatch main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7866776140
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Mon Feb 12 03:11:18 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7881138498
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.workflow_dispatch.main.default.slsa3.yml
Trigger: workflow_dispatch
Branch: main
Date: Tue Feb 13 03:11:46 UTC 2024
Tests are passing now. Closing this issue.


[Bazel CI] ERROR: While parsing option --toolchain_resolution_debug --remote_download_toplevel: Failed to build filter: value looks like another flag

CI: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/4170#019255c7-7f1a-4f86-8e9b-e679cfcc93f5
Platform: Ubuntu
Logs:
ERROR: While parsing option --toolchain_resolution_debug --remote_download_toplevel: Failed to build filter: value looks like another flag (--remote_download_toplevel). Either escape the value with "\-\-", or pass an explicit value to the flag.

CC Greenteam @mai93

The failure is because toolchain_resolution_debug requires a value.
However I don't see that flag in this repo
https://github.com/search?q=repo%3Abazel-contrib%2Frules_nodejs toolchain_resolution_debug&type=code
Maybe it's being set in the CI pipeline defined in bazelbuild/continuous-integration?

Hi @alexeagle, Its not set here https://github.com/bazel-contrib/rules_nodejs/blob/main/.bazelci/presubmit.yml. Can you please take a look.


metadata: add Puppet version compatibility

Also:

Remove "description" field (is deprecated)
Declare support for Debian 7
Remove the ".0" part of the version number for Debian & Red Hat

@3flex killin it!


[Loading] Use generic "Loading" message for all statuses

Close #769
Changes
I searched for all instances of "loading", "updating", or "saving" and replaced any detailed status messaging with a plain "Loading"

@meissadia
Did we also fix the exclamation point in the error message that I flagged?

@meissadia Did we also fix the exclamation point in the error message that I flagged?

@natalia-fitzgerald Yes
https://github.com/cfpb/sbl-frontend/pull/774/files#diff-a172a52e6954972ee06208c37fe8fd13c1d62619ac2be3f623cdb3af4ba1f557R184

@billhimmelsbach - Has this gone live on the platform? I'm not sure how to track it down to check it but I see the change here and see that it has been merged. Should that be enough for me to check this item off of my list?


Support youtube

Create a message on youtube link posted and after permission download the video.

would that thing ask you to download it?
if so, then could you lower the resolution? like 640x360?

lgtm


Suggestion: Add Proxy Settings

Ripme version:1.7.3
Java version: 8
Operating system: win 10

Well as the title says its a suggestion not an issue, so right now in order to use proxy we have to change windows setting, how about adding proxy detection to the ripper, even if its manual its still effective enough.
you can detect proxy from Internet Explorer. 👍
tnx anyway

I agree this would be a good setting to have

I think we've previously said that this should be an OS-level setting. Not sure if we can choose to use a proxy server from an app (going around the OS setting for proxy).
Also not sure how to set up a proxy server to test this functionality.
How do you detect a proxy in Internet Explorer?
You only want ripme to detect a proxy (os-level) is in use? Why? What's your use case here that an OS-level proxy doesn't solve?

@metaprime
yeah i only want the rip me to detect my current proxy settings or have some boxes for ports
similar to IDM
in their setting Downloads>Options>Proxy/Socks
We can fill whatever we want the program to use or it can detect from Internet Option itself.

also found this https://stackoverflow.com/questions/4933677/detecting-windows-ie-proxy-setting-using-java
don't know if its helping or not

Oh, does ripme currently ignore the OS proxy settings?

yeah it does,
check this free proxy https://www.psiphon3.com/
it connects and changes the settings in Internet Options but rip me ignores them or maybe uses different ports or diffrent protocols i dont exactly know, look at it urself :)


remove_diacritics for other languages

I am working on a dataset that has text from English, Japanese, Italian, Dutch, Spanish, German and Swedish. I want to know that happens if I run the remove_diacritics on this dataset. Also how does it effect the emojis present in the text.

Hi, remove_diacritics internally transfors a given text to its NFKD normal form and then throws out all characters that are combining. So it won't affect any emojis :ok_hand:

Hi, remove_diacritics internally transfors a given text to its NFKD normal form and then throws out all characters that are combining. So it won't affect any emojis :ok_hand:

What about characters specific to other languages?

What about characters specific to other languages?

which characters do you have in mind? But like @henrifroese mentioned the NFKD normal form includes all normal characters from these languages. For Japanese, it would be helpful if you could specify, which writing system you want to process 🥇
However, if you just try out the special characters with the function, you can test by yourself, how it behaves 😉

which characters do you have in mind? But like @henrifroese mentioned the NFKD normal form includes all normal characters from these languages. For Japanese, it would be helpful if you could specify, which writing system you want to process 🥇
However, if you just try out the special characters with the function, you can test by yourself, how it behaves 😉

Thanks @mk2510 and @henrifroese for your help! @M-I-Dx do you have any updates? Otherwise, I will close the issue.


If a result does not have a unit, you cannot write a blank unit-record in FHIR. So, if there's no unit, the unit (+ system + code)-fields, should not be printed.

Lotte, can you use Observation.valueInteger, or another variant instead of Observation.valueQuantity?

Please take a look at https://ucum.org/ucum.html

Tested with united code {} and this validates.


Simplify rdkit molecule generation for ligand

Description
At the moment, our ligand data retrieval is a bit too hacky.

We need the (a) complex PDB file that is used to generate the dynophore and (b) the ligand name as used in the PDB.
We extract the ligand atoms using MDAnalysis wit (a/b) to get (c) PDB block and (d) PDB residue IDs.

Potential problem: We add MDAnalysis as dependency for (c/d) only and we need to know the ligand name used in the PDB to do this.

We need the (e) ligand SMILES in the dynophore JSON file.
We use (c-e) to generate an rdkit molecule by inferring bonds from SMILES.

Potential problem: This procedure is ambiguous and might not end up to be the same ligand as in the original PBD.

There is a lot of room for crashes :) It would be much better to add

a MDL mol block and
the PDB residue IDs

to our dynophore JSON file. That way we can create the ligand directly from the MDL mol block and update the atom IDs with the PDB residue IDs.
Todos

[x] Update dynophores JSON file:

[x] Add MDL mol block
[x] Add PDB residue IDs

[x] Update the dynophores.core.Ligand class
[x] Add Ligand class as attribute to Dynophore class
[x] Update backend of view2d module
[x] Update unit tests; rerun notebooks

Questions
None.
Status

[ ] Ready to go

Codecov Report

Merging #31 (e142697) into master (33f2307) will decrease coverage by 0.70%.
The diff coverage is 99.06%.


3.1.1 Upgrade 3.2.0 The devops status cannot be updated automatically, and the user information is not correct.

The publishing user is not admin, but all show admin，The devops status cannot be updated automatically

hi @gzdreamfly thanks for reaching out to us. But please only create different issues for each problem. IMO you just reported two problems.
For the status part, we already fixed it. See https://github.com/kubesphere/console/pull/2594
For the users' part, this is a historical issue. We will improve it.
/kind bug

/close


errorCode 19

Describe the bug
After downloading more than 70 episodes of an Anime,it suddenly started giving me this error:
anime: Downloading episode 112 of Fairy Tail (Sub)

07/14 16:57:19 [NOTICE] Downloading 1 item(s)

07/14 16:57:19 [ERROR] CUID#7 - Download aborted. URI=https://www506.playercdn.net/173/0/YQP6zD5DBZMiOYTFngdJVw/1531623439/170427/6iXPJXswijp2FOR.mp4
Exception: [AbstractCommand.cc:351] errorCode=19 URI=https://www506.playercdn.net/173/0/YQP6zD5DBZMiOYTFngdJVw/1531623439/170427/6iXPJXswijp2FOR.mp4
  -> [AbstractCommand.cc:792] errorCode=19 CUID#7 - Name resolution for www506.playercdn.net failed:Domain name not found

07/14 16:57:19 [NOTICE] Download GID#aa6433af9f705a65 not complete: /media/johnpyp/sdb3/tv/Fairy_Tail/Fairy Tail - s01e112.mp4

Download Results:
gid   |stat|avg speed  |path/URI
======+====+===========+=======================================================
aa6433|ERR |       0B/s|/media/johnpyp/sdb3/tv/Fairy_Tail/Fairy Tail - s01e112.mp4

Status Legend:
(ERR):error occurred.

aria2 will resume download if the transfer is restarted.
If there are any errors, then see the log file. See '-l' option in help/man page for details.

And in turn stopping the process.
Now I might be able to download 3-4 episodes before it stops working again.
To reproduce
anime dl -e 1:175 'http://kissanime.ru/Anime/Fairy-Tail' -xd "{aria2}" -q 360p --provider kissanime --download-dir ./Fairy_Tail -ff "Fairy Tail - s01e{ep_no}"

Same issue when searching rather than using a direct url.

LOG
$ anime dl -e 112:175 'http://kissanime.ru/Anime/Fairy-Tail' -xd "{aria2}" -q 360p --provider kissanime --download-dir ./Fairy_Tail -ff "Fairy Tail - s01e{ep_no}" --log-level DEBUG
INFO root: anime-downloader 3.2.0
DEBUG root: Platform: Linux-4.15.0-24-generic-x86_64-with-Ubuntu-18.04-bionic
DEBUG root: Python 3.6.5
INFO root: Extracting episode info from page
DEBUG root: Unfiltered episodes : ['http://kissanime.ru/Anime/Fairy-Tail/Episode-175?id=26397', 'http://kissanime.ru/Anime/Fairy-Tail/Episode-174?id=25364', 'http://kissanime.ru/Anime/Fairy-Tail/Episode-173?id=24123',
// Long list of episodes
'http://kissanime.ru/Anime/Fairy-Tail/Episode-001?id=816']
DEBUG root: Filtered episodes : ['http://kissanime.ru/Anime/Fairy-Tail/Episode-175?',
// Long list of episodes
'http://kissanime.ru/Anime/Fairy-Tail/Episode-001?id=816']
DEBUG root: EPISODE IDS: length: 175, ids: ['http://kissanime.ru/Anime/Fairy-Tail/Episode-001?id=816', 'http://kissanime.ru/Anime/Fairy-Tail/Episode-002?id=817', 'http://kissanime.ru/Anime/Fairy-Tail/Episode-003?id=818', 'http://kissanime.ru/Anime/Fairy-Tail/Episode-004?id=819',
// Long list of episodes
'http://kissanime.ru/Anime/Fairy-Tail/Episode-175?id=26397']
INFO root: Found anime: Fairy Tail (Sub)
INFO root: Downloading to /media/johnpyp/sdb3/tv/Fairy_Tail
DEBUG root: Extracting stream info of id: http://kissanime.ru/Anime/Fairy-Tail/Episode-112?id=734
DEBUG root: Calling url: http://kissanime.ru/Anime/Fairy-Tail/Episode-112?id=734&s=rapidvideo
DEBUG root: Calling Rapid url: https://www.rapidvideo.com/e/FF2TV2UHL7&q=360p
INFO root: Downloading episode 112 of Fairy Tail (Sub)
DEBUG root: cmd: {aria2}
DEBUG root: episode: <anime_downloader.sites.kissanime.KissanimeEpisode object at 0x7f04bfe76e10>
DEBUG root: file format: Fairy Tail - s01e{ep_no}
DEBUG root: formatted cmd: aria2c https://www506.playercdn.net/173/0/304GNCnncWrO-6BohJ82yw/1531623911/170427/6iXPJXswijp2FOR.mp4 -x 12 -s 12 -j 12 -k 10M -o Fairy Tail - s01e112.mp4 --continue=true --dir=/media/johnpyp/sdb3/tv/Fairy_Tail --stream-piece-selector=inorder --min-split-size=5M

07/14 17:05:12 [NOTICE] Downloading 1 item(s)

07/14 17:05:12 [ERROR] CUID#7 - Download aborted. URI=https://www506.playercdn.net/173/0/304GNCnncWrO-6BohJ82yw/1531623911/170427/6iXPJXswijp2FOR.mp4
Exception: [AbstractCommand.cc:351] errorCode=19 URI=https://www506.playercdn.net/173/0/304GNCnncWrO-6BohJ82yw/1531623911/170427/6iXPJXswijp2FOR.mp4
  -> [AbstractCommand.cc:792] errorCode=19 CUID#7 - Name resolution for www506.playercdn.net failed:Domain name not found

07/14 17:05:12 [NOTICE] Download GID#ab7fc31b72737590 not complete: /media/johnpyp/sdb3/tv/Fairy_Tail/Fairy Tail - s01e112.mp4

Download Results:
gid   |stat|avg speed  |path/URI
======+====+===========+=======================================================
ab7fc3|ERR |        n/a|/media/johnpyp/sdb3/tv/Fairy_Tail/Fairy Tail - s01e112.mp4

Status Legend:
(ERR):error occurred.

aria2 will resume download if the transfer is restarted.
If there are any errors, then see the log file. See '-l' option in help/man page for details.

I have never encountered this. If I had to guess, I would say it is rapidvideo blocking your requests. I had this happen with the CI servers and I have not found a workaround yet.

Is there any way to use one of the other servers? I don't want to use 9anime because they use different subs.

You can use masterani if you want. My ISP blocks(?) openload and streamango, so I could not develop stream url extractors for them. In a few weeks, I would be able to do that.

Is there any way to use one of the other servers?

If you are talking about KA servers other than rapidvideo, then no. They use image captcha which is not easy to fool.
Also, did you try the stream url through your browser?

In browser I get the native "This site can’t be reached", not like an nginx 404.
I'll try masterani, thanks.

--provider for masterani isn't working

--provider for masterani isn't working

You have to provide the full link. I have not implemented search for masterani yet. Kinda busy nowadays. Also for your KA problem, maybe a restart might help.

Oh ok, thanks

I'll try kissanime some more, but it seems Masterani isn't working:
johnpyp@johnpyp:/media/johnpyp/sdb3/tv$ anime dl 'https://www.masterani.me/anime/info/257-fairy-tail' -xd "{aria2}"  -ff "Fairy Tail - s01e{ep_no}"
anime: anime-downloader 3.2.0
anime: Extracting episode info from page
anime: Found anime: Fairy Tail
anime: Downloading to /media/johnpyp/sdb3/tv

It just stops at that.

Yeah. I can reproduce it.
Looks like this anime has only streamango and mp4upload sources. Weird.
As I said I cannot access streamango and mp4upload uses js obfuscation (packer, if my memory serves me right). I could reverse mp4upload, but I'm not in the mood rn :sweat_smile:.
So yeah, you're better off with KA. (But 9A would be the most reliable)

No worries, I'm trying 9A right now. Noticed some inconsistencies in the subs, but it probably isn't that big of a deal. Thanks for this, I was in the middle of building a scraper in node js, but I decided to do some more extensive research and I found the first one that works!

I should've used node instead of python. I have to translate uglified js to python because I went the python way. Anyway this works for now.
Happy that you're enjoying it. Closing this issue now.

Oh yeah. Update your installation before trying 9anime. I have released a new version.

Damn went from 500-1000kib to like 10mb download, does that token change have a big effect?

9anime is very scraper unfriendly. They change their tokens every now and then. So I have a fallback mechanism in place (falls back to 9anime.cloud). I don't know how it can affect the speed though.

Gotcha


Build fix for CentOS 7: somehow the default template argument isn't visible

I didn't mean to use the default argument anyway, so this is fine.
rdar://89744467

@swift-ci Please smoke test and merge

@swift-ci Please smoke test macOS

@swift-ci Please smoke test


Does Opensearch operator version 2.4.0 supports the Opensearch version 2.10.0?

I don't see the compatibility table updated with operator version 2.4.0

Duplicate of discussion #630


Exception (504) Reason: "channel/connection is not open"

when I publish message, the code return error that Exception (504) Reason: "channel/connection is not open". what is the error for?

Hi folks,
I have the following code...
// Connect - Connects to RabbitMQ
func (queue *Queue) Connect(args ...interface{}) {

	var uri string

	if args == nil {

		// Get from env vars
		uri = os.Getenv("RABBIT_MQ_URI")

		if uri == "" {
			log.Panic("No uri for queue given")
		}
	} else {
		uri = args[0].(string)
	}

	// Make max 5 connection attempts, with a 1 second timeout
	for i := 0; i < 5; i++ {

		log.Println("Connecting to:", uri)

		// If connection is successful, return new instance
		conn, err := amqp.Dial(uri)

		if err == nil {
			log.Println("Successfully connected to queue!")
			channel, _ := conn.Channel()
			queue.Connection = conn
			queue.Channel = channel
			return
		}

		log.Println("Failed to connect to queue, retrying...", err)

		// Wait 1 second
		time.Sleep(5 * time.Second)
	}
}

// Declare a new queue
func (queue *Queue) Declare(queueName string) (amqp.Queue, error) {
	return queue.Channel.QueueDeclare(
		queueName,
		true,
		false,
		false,
		false,
		nil,
	)
}

// Publish a message
func (queue *Queue) Publish(queueName string, payload []byte) error {
	return queue.Channel.Publish(
		"",
		queueName,
		false,
		false,
		amqp.Publishing{
			DeliveryMode: amqp.Persistent,
			ContentType:  "application/json",
			Body:         payload,
		},
	)
}

// Listen for a new message
func (queue *Queue) Listen(queueName string) (<-chan amqp.Delivery, error) {
	return queue.Channel.Consume(
		queueName,
		"",
		true,
		false,
		false,
		false,
		nil,
	)
}

Then in my main.go
        // Get queue connection
	queue := drivers.NewQueue()
	queue.Connect()

        // Declare new queue for submissions 'stuff'
	_, err = queue.Declare("submissions_queue")

	if err != nil {
		log.Panic(err)
	}

	// We need to declare the responses queue even though we're not actually
	// going to use it on the bot side as an emitter.
	_, err = queue.Declare("submissions_responses")

	// Log out any queue errors
	if err != nil {
		log.Panic(err)
	}

These queue then interact with a legacy PHP application (Laravel). Which runs perfectly for a day or two, then after about a day, sometimes less, sometimes longer, I start getting this Exception (504) Reason: "channel/connection is not open". Any clues? Am I doing something dumb?
Thanks in advance!

See server logs for channel exceptions: they close channels. Also abrupt client TCP connection closures. This client does not provide automatic connection recovery by design, that's your application responsibility.

I've updated my code where the publish to queue error sometimes occurs, to recreate the channel and try again, is that what you meant?
The only output I'm seeing is:
[bot-rmq-1]2016-11-23T10:27:42.499711789Z =INFO REPORT==== 23-Nov-2016::10:27:42 ===
[bot-rmq-1]2016-11-23T10:27:42.499718307Z closing AMQP connection <0.14157.0> (<redacted>:39617 -> <redacted>:5672)
[bot-rmq-1]2016-11-23T10:27:42.705177619Z
[bot-rmq-1]2016-11-23T10:27:42.705206764Z =INFO REPORT==== 23-Nov-2016::10:27:42 ===
[bot-rmq-1]2016-11-23T10:27:42.705212691Z accepting AMQP connection <0.14215.0> (<redacted>:39622 -> <redacted>:5672)
[bot-rmq-1]2016-11-23T10:27:43.046378706Z
[bot-rmq-1]2016-11-23T10:27:43.046413103Z =INFO REPORT==== 23-Nov-2016::10:27:43 ===
[bot-rmq-1]2016-11-23T10:27:43.046419515Z closing AMQP connection <0.14180.0> (<redacted>:51242 -> <redacted>:5672)
[bot-rmq-1]2016-11-23T10:27:43.253073052Z
[bot-rmq-1]2016-11-23T10:27:43.253103233Z =INFO REPORT==== 23-Nov-2016::10:27:43 ===
[bot-rmq-1]2016-11-23T10:27:43.253109057Z accepting AMQP connection <0.14227.0> (<redacted>:51243 -> <redacted>:5672)
[bot-rmq-1]2016-11-23T10:27:45.950046536Z
[bot-rmq-1]2016-11-23T10:27:45.950085091Z =INFO REPORT==== 23-Nov-2016::10:27:45 ===
[bot-rmq-1]2016-11-23T10:27:45.950091135Z closing AMQP connection <0.14215.0> (<redacted>:39622 -> <redacted>:5672)
[bot-rmq-1]2016-11-23T10:27:46.147715611Z

I'm not actually seeing any exceptions anywhere so far

Yes, applications need to open another channel and use that, there is no other way. Automatic connection recovery in other clients allocates it while preserving channel id but as far as RabbitMQ is concerned, it's a completely separate channel.

Your log connections are being closed < 5 seconds after they are accepted. RabbitMQ connections (with any messaging protocol) are meant to be long lived: are you sure your own code doesn't carry a high degree of connection churn? If it's concurrent on top of that, it's not difficult to see how one thing tries to do something while a connection is being closed. So try using long-lived connections, in particular if you have channel recovery in place already.

Awesome, cheers for you help! I'm not intentionally using a short-lived connection, perhaps something, somewhere is prematurely closing the connection. I'll have further dig into my code.

@EwanValentine connection-level errors ("hard errors" in protocol speak) are rare and should be very visible in the broker log. They usually indicate client bugs w.r.t. serialization or concurrency.

maybe your queue name is created.

I had this problem it turns out the rabbitmq server wasn't on. so maybe check that?

Facing Issue: Failed to publish a message: Exception (504) Reason: "channel/connection is not open"
I have checked logs in rabbitMQ
2020-07-30 11:55:57.792 [error] <X.XXXX.86> Channel error on connection <X.XXXX.86> (XX.XXX.XX.X:42801 -> XX.XX.X.XX:4672, vhost: '/', user: 'openapi-gateway'), channel 1: operation basic.publish caused a channel exception not_found: no exchange 'api-log-aggregator' in vhost '/' 2020-07-30 11:55:58.327 [warning] <X.XXXX.86> closing AMQP connection <X.XXXX.86> (XX.XXX.XX.X:42801 -> XX.XX.X.XX:4672, vhost: '/', user: 'openapi-gateway'):
as per above logs  api-log-aggregator has not deployed that why getting error so I have deployed api-log-aggregator  service and redeploy related services .
It worked fine....

I know how to reproduce this problem. We can first close the network connection locally, then reconnect to the network, and then ch.Publish send a message, and this problem occurs. My current solution is to create a new amqp. Connection and replace the old one if amqp. ErrClosed appears.

My current solution is to create a new amqp. Connection and replace the old one if amqp. ErrClosed appears

This is the correct approach, along with reestablishing the queue/exhange topology.


adds capacity getter and reserveCapacity method

Seems like a useful addition that exists on standard library types. Hope the implementation is okay!

Hi @lukaskubanek  - any thoughts on this addition?

Hi @GalCohen, thanks for your pull request and your patience. In the recent weeks, I was busy launching the app this library was primarily developed for. I’ll come back to this pull request in the next days.

Hi @GalCohen, thanks for this useful addition. I went a step further and added OrderedDictionary.init(minimumCapacity:) as well as some basic tests covering this functionality.
Further, I’ve changed the logic of OrderedDictionary.capacity to use the minimum reserved capacity of the underlying types (Array and Dictionary) to prevent false reports. The Dictionary typically allocates larger portions of memory and thus depending on this value might still lead to reallocation of the Array type.
You can find all changes in the merge commit f303e02f635bf9a28eb605077aa73b346849003a.

Thanks for merging in my PR @lukaskubanek ! Your revisions make sense. Best of luck of with new app!


Two providers from LbrLabs already migrated to Pulumiverse

Description
The following providers have been migrated from LbrLabs (Lee Briggs organization) to Pulumiverse:

https://github.com/pulumiverse/pulumi-grafana
https://github.com/pulumiverse/pulumi-scaleway

I'm close to having a first release from the new Github organization, so the version check should correctly point to the new Github org.
Adding a new package?
If this pull request adds a new package:

[x] The package's schema URL is correct.
[x] The package metadata file, if present, contains:

[x] a supported category (one of Cloud, Infrastructure, Network, Database, Monitoring, or Utility).
[x] a description that explains what the package does.
[x] a valid logo URL that points to a PNG whose dimensions conform to the others in this repo (e.g., 100x100).
[x] a version number prefixed with v that corresponds with a valid GitHub release.

[x] The package repo contains an Overview doc (/docs/_index.md) that includes:

[x] a brief explanation of what the package is and what it does.
[x] at least one representative example in all supported languages.
[x] a front-matter property for the layout set to package.

[x] The package repo contains an Installation and Configuration doc (/docs/installation-configuration.md) that includes:

[x] links to SDKs in all supported languages.
[x] a copyable command for installing the resource plugin if necessary.
[x] an example of configuring the provider with pulumi config set.
[x] an example of configuring the provider with environment variables.

[ ] Someone from the @pulumi/ai team has reviewed the PR.
[ ] Someone from the @pulumi/docs team has reviewed all documentation.

Supersedes #3890

Migration docs for Grafana: https://github.com/pulumiverse/pulumi-grafana/commit/d0d06ed37c71ce9c2b4049a551588e3f05c7f8d4


Rework nsubstitute integration approach

The NSubstitute library was recently extended to provide a graceful API for auto-value providers. This was quite a long way, but I'm glad that it finished. Usage of this API allows to extend NSubstitute in very optimized way and avoid huge limitations which we have now.
This rework introduces the breaking changes to the AutoFixture, therefore I did them in the v4 branch, where such kind of changes is allowed. These breaking changes basically extend the level of integration rather that are intended to limit the existing logic. Consider the following set of changes:

Generic methods are now fully supported (#720).
ref and out argument are now configured for the void methods as well.
The PropertyInfo request is used to resolve property values (#653).
It is possible now to configure methods with ref and out parameters: #707 (that is achieved by both NSubsitute and integration changes).
Concurrency support is improved and substitution methods could be safely invoked from the multiple threads, even for the first time. There are chances that the issue #592 will be resolved now.

So far no tests failed except ones where I changed behavior intentionally (generics and void methods).
I'm currently consuming Release Candidate because I want to start the discussion as early as possible :) I've already put some efforts to push changes on NSubstitute side, so hope everything will go smoothly here as well :) After we agree on approach, we'll wait till NSubsitute v2.0 release and push changes after that.
I'd like to invite @mrinaldi, @dtchepak and @alexandrnikitin to review my implementation and advice whether they see some concerns regarding my approach.
P.S. In case you already started to review this PR, I've improved the initial implementation a bit mostly to reduce the AutoFixtureValuesHandler responsibilities.

Sorry @Zvirja, I'm not familiar enough with AutoFixture code to be of help here.

@mrinaldi I know that you were the guy who wrote the initial implementation for the integration. Would you be able to find some time and review the reworked integration to check whether the potential pitfalls exist? 😊


Show new transactions instantly in activity

Currently when the user submits a transaction, there is a lag time between submission and appearing in the activity tab of the home screen. This lag creates uncertainty for the user as to whether the transaction has actually been submitted successfully.
A potential solution via caching of transactions locally is discussed in https://github.com/blockstack/stacks-wallet-web/discussions/1712
This solution would not only provide greater "speediness" but also prevent users from thinking their transactions have failed to go through at all, should the API experience issues with recognizing transactions per https://github.com/hirosystems/devops/issues/797.
Companion issue for the desktop wallet: https://github.com/blockstack/stacks-wallet/issues/921

Note:All the below occurred in one same account in Mainnet
I have noticed the similar behavior and was even thinking of logging the issue but after couple minutes,I noticed the missing transaction in the account
There were two transaction but the first one was not visible in Activity despite tx was successful.
Within few sec,I placed another tx but that received nonce conflict.
I then tried again in few min on another tx and the tx was successful
This time I was able to view the latest tx as pending.
I even went to explorer to check on the history of the account but was not able to track the first tx.
After few minutes once the latest tx was completed,and the balance was deducted.I saw both of the tx (first and last) in the activity and also in explorer.
There were only Failed to load resources error in the console.

New transaction as shown immediately as 'Queued".Moving the ticket to Done


Can't install ebtables (mandatory for kubespray) on Debian 12 (Bookworm)

E: Package 'ebtables' has no installation candidate

I'm trying to install ebtables on Debian 12 (Bookworm) but it seems that the package is not available on all control-plane nodes. I'm using kubespray to deploy a Kubernetes cluster and the installation fails because of this missing package.
TASK [kubernetes/preinstall : Install packages requirements] ******************************************************************
fatal: [control-plane-1]: FAILED! => {"attempts": 4, "changed": false, "msg": "No package matching 'ebtables' is available"}
FAILED - RETRYING: [control-plane-2]: Install packages requirements (1 retries left).
FAILED - RETRYING: [control-plane-3]: Install packages requirements (1 retries left).
fatal: [control-plane-3]: FAILED! => {"attempts": 4, "changed": false, "msg": "No package matching 'ebtables' is available"}
fatal: [control-plane-2]: FAILED! => {"attempts": 4, "changed": false, "msg": "No package matching 'ebtables' is available"}
changed: [worker-2]
changed: [worker-1]

It seems that the control-plane can't access to internet:
debian@worker-1:~$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS>: icmp_seq=2 ttl=57 time=5.46 ms
64 bytes from <IP_ADDRESS>: icmp_seq=3 ttl=57 time=4.82 ms
#--------
debian@control-plane-1:~$ ping <IP_ADDRESS>
PING <IP_ADDRESS> (<IP_ADDRESS>) 56(84) bytes of data.
From <IP_ADDRESS> icmp_seq=1 Destination Host Unreachable
From <IP_ADDRESS> icmp_seq=2 Destination Host Unreachable

My bad.. I forgot to remove the external interface of all workers.
I think that the problem is on the router, i'll check tomorrow

Solution : The router was not linked to the subnet (forgot to add the port)


Jetty specific import in spark.Request

Hi,
SparkJava release 2.7.0 has a Jetty specific import org.eclipse.jetty.util.URIUtil; in spark.Request (line 34)
Now org.eclipse.jetty.util should be in your classpath even if you deploy Spark application on external servlet container (Tomcat, etc.)
Thanks.

Looks like this PR introduced it: https://github.com/perwendel/spark/pull/872/files

This is bad! We'll fix ASAP and release 2.7.1

Fixed and released already.


Add tags to audio commands

This will allow users to categorize and help find commands if the list becomes too large to manage effectively.
Commands will now have the ability to store arbitrary data - this might be useful later. For now this will store just the tags. The data will be stored as JSON in files in the res/ directory that having a matching file name (but for the .json extension) to the command's audio file.
When the user uploads a file, they can add tags to it by adding them as a comma separated list as comments when uploading the file to the bot. Users will also be able to add and remove tags from commands in PM sessions with the bot.
Two new commands will be added so that users can look up popular tags, and then search for audio commands by tag.

Tags are in!


Add Realm backup \ world download endpoints

is this ready

I believe so, @extremeheat any further suggestions?

Not tested personally, but I assume it works so LGTM

@rom1504 I think this is all good to go?

thanks!

You can open a release PR


ULID KSUID Default For Id Fields

Is this feature request related to a new or existing Amplify category?
api
Is this related to another service?
DynamoDb
Describe the feature you'd like to request
It would be great if the autogenerated id's, generated by the .vtls were sortable i.e. using a ULID or KSUID.
The most common access pattern in most of my apps is where the user wishes to sort items by createdAt date.
Currently, this can only be achieved by adding an additional index to the table in which the createdAt property is the sort key.
Having a sortable id would negate the need for creating additional indicies.
Describe the solution you'd like
When an "ID" field is defined in the schema, the vtl generated to automatically create the Id should create a sortable id (ULID) as opposed to a random UUID.
Describe alternatives you've considered
Currently, this can only be achieved by adding an additional index to the table in which the createdAt property is the sort key.
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Looks like autoUlid is now available in .vtl templates:
https://docs.aws.amazon.com/appsync/latest/devguide/resolver-util-reference.html


Black border after uploading and moving a photo all to the left bottom

When uploading and moving a photo all to the left bottom it's showing a black border on the right side after cropping the image. Does anybody know how to fix this?
Demo: http://bcme.hyperconnected.nl
Using
iPhone 6 iOS 9.3.2
Croppie 2.1.1

When I remove the rotation from your wrapping element, and then drag the image to the bottom left the black border goes away.