Feature/issue 1852 streamline nuts criterion

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Tweaks the (generalized) No-U-Turn criterion.  Addresses #1852.
Intended Effect
The (generalized) No-U-Turn criterion is slightly tweaked which not only allows for an implementation that uses less memory but also makes adding Riemannian NUTS much simpler.  The change is essentially an edge case in the trajectory discretization (include the last point or not?) that is ambiguous in the theoretical definition, so it shouldn't make any appreciable difference in sampler performance.
How to Verify
Run unit tests or try running some multivariate models with known expectations.  Everything should fit just as well as before.
Side Effects
The tweak in the criterion definition will slightly change the exact sampler output but should not affect the accuracy of any expectations.  For small step sizes any such effects should be negligible.
Documentation
No user-facing changes.
Reviewer Suggestions
Anyone.
Copyright and Licensing
Copyright: Michael Betancourt
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

I really like that this removes a lot of code duplication.  But I can't tell what the specific tweak is.  Could you give me a hint so that I can try to check if it's doing what it's supposed to be doing?  Are p_sharp_plus and p_sharp_minus the current forward-in-time and backward-in-time endpoints?  If so, what's p_sharp_left?
Also, does there need to be a change to the manual algorithms chapters to match this change?

The theoretical definition is
<p_{N}, rho>
where
rho = \int dt p(t).
The old implementation approximated rho without the final point,
rho_old = \sum_{n = 0}^{n - 1} p_{n},
while the new implementation does use the final point, just to
avoid having to keep the momenta values around for the subtraction,
rho_new = \sum_{n = 0}^{N} p_{n},
This way instead of needing to save the momentum and the
metric to compute M^{-1} * p_{N} in the inner product and
subtract p_{N} from rho we can just store p_sharp = M^{-1} * p.
Abstracting to p_sharp, which is computed by each Hamiltonian
implementation, is what admits the clean up of the criterion
implementations.
minus/plus refer to the forward/backward endpoints where as
left/right refers to the endpoint in the current subtree which may
be flipped with respect to the global definition of minus/plus.  It’s
all reversible so the actual order doesn’t matter, but the naming
should be self-consistent.
The doc doesn’t touch on any of these details so no changes
are necessary.
On Apr 10, 2016, at 11:22 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

I really like that this removes a lot of code duplication. But I can't tell what the specific tweak is. Could you give me a hint so that I can try to check if it's doing what it's supposed to be doing? Are p_sharp_plus and p_sharp_minus the current forward-in-time and backward-in-time endpoints? If so, what's p_sharp_left?
Also, does there need to be a change to the manual algorithms chapters to match this change?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Thanks.  Just what I needed.

This one looks good to go, too.  I love the way you have all this code organized; it makes it very easy to review.


Add support for do/end blocks

It's now possible to express pipes like this, adapted from the Russian Roulette
example:
iex> pipe_matching {:ok, _} do
...>   {:ok, ""}
...>   |> click
...>   |> click
...>   |> bang
...>   |> click
...> end
click...
click...
BANG.
{:error, "bang"}

@batate any progress with this?

I've been thinking about this a while. Wondering what the use case is. My concern is that the meaning becomes unclear. What if you have nested pipes? Should you then expect each one to nest?
And what if you have something besides a pipe in that function?
My preference is that this remains a macro explicitly on piped expressions.

After talking to Eric mj, I've decided not to do this pull request. I think
it's best to make this project a function explicitly on a pipe. Otherwise,
it's unclear what the code should do (e.g. in nesting situations and the
like)/
Ping me if you have any questions.
-bt
On Sun, Oct 11, 2015 at 2:47 AM, Yevhenii Kurtov<EMAIL_ADDRESS>wrote:

@batate https://github.com/batate any progress with this?
—
Reply to this email directly or view it on GitHub
https://github.com/batate/elixir-pipes/pull/10#issuecomment-147167114.

--
Bruce Tate
President, RapidRed, LLC
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
Author of Seven Languages in Seven Weeks, Deploying Rails Applications,
From Java to Ruby, Rails: Up and Running, Beyond Java, 6 others.

To be honest, I was mostly playing around with it to avoid the somewhat weird appearance of the pipe that the macro can enforce.. And, of course, to learn a bit more about metaprogramming myself :-)
In many cases, it might make sense to simply write your own wrapper functions anyway, that expect the pipe argument to be what you need (eg. {:ok, response}) and that have a custom clause for the error case too if needed.

Perfect!  I really liked the way you attacked the problem.
Take care.
-bt
On Sunday, October 11, 2015, Johan Warlander<EMAIL_ADDRESS>wrote:

To be honest, I was mostly playing around with it to avoid the somewhat
weird appearance of the pipe that the macro can enforce.. And, of course,
to learn a bit more about metaprogramming myself :-)
In many cases, it might make sense to simply write your own wrapper
functions anyway, that expect the pipe argument to be what you need (eg.
{:ok, response}) and that have a custom clause for the error case too if
needed.
—
Reply to this email directly or view it on GitHub
https://github.com/batate/elixir-pipes/pull/10#issuecomment-147248683.

--
Bruce Tate
President, RapidRed, LLC
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
Author of Seven Languages in Seven Weeks, Deploying Rails Applications,
From Java to Ruby, Rails: Up and Running, Beyond Java, 6 others.


Update version number on master

Simply update the version number on master. Doesn't matter if it ends up being 2.0.0, this will do for now, the crucial thing is that v1.10.0 already left the building so the version number on master must be increased.

Ping @lbdreyer

(PR to merge changes from v1.10.x into master inbound once this is done.)

I'm just going to let travis finish then I will merge this in (expecting to get the error we are getting in #1738, the fix of which isn't on master yet).

Another unexpected error:
ERROR: test_all (iris.tests.test_analysis.TestRotatedPole)

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

Traceback (most recent call last):

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/Iris-1.11.0.dev0-py2.7-linux-x86_64.egg/iris/tests/test_analysis.py", line 561, in test_all

    self._check_both_conversions(cube)

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/Iris-1.11.0.dev0-py2.7-linux-x86_64.egg/iris/tests/test_analysis.py", line 546, in _check_both_conversions

    self.check_graphic()

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/Iris-1.11.0.dev0-py2.7-linux-x86_64.egg/iris/tests/__init__.py", line 696, in check_graphic

    figure.savefig(result_fname)

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/matplotlib/figure.py", line 1421, in savefig

    self.canvas.print_figure(*args, **kwargs)

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/matplotlib/backend_bases.py", line 2220, in print_figure

    **kwargs)

  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/matplotlib/backends/backend_agg.py", line 517, in print_png

    filename_or_obj, self.figure.dpi)

RuntimeError: Error building image

But I think that should be fixed in a different PR.


Cannot write transaction with contract source

Hey,
I've upgraded ArLocal in our redstone-smartcontracts to the latest version and now during the contract deployment in our intergration tests (eg: https://github.com/redstone-finance/redstone-smartcontracts/blob/main/src/tests/integration/deploy-write-read.test.ts) I'm getting an error while creating a transaction with contract source:
srcTx = await this.arweave.createTransaction({ data: src }, wallet);
The last version that works properly, is 1.1.9

Hey @ppedziwiatr we now force the need to add funds to your wallet before being able to send transactions. This is to have more of an accurate local testnet with the mainnet.

Ok, that makes sense :-)
Tried to fix this by calling the mint endpoint, but I'm still getting an error - can you please suggest what I'm doing wrong? https://github.com/redstone-finance/redstone-smartcontracts/pull/56/files#diff-4c8ef438798c3bfb11da9a1e567605a4ddbce94c7ac1201dc739a65658b353de

Hello @ppedziwiatr , The mint amount is in winston, you minted 1000 winstons, isnt' that too small to create a transaction ?

Oh crap, you're right, not sure why I assumed that it's in ARs...thanks for clarification!


how do i wire this to my Pi

how do i wire this from my onkyo 608 to my Pi?

What do you "wire this to"? There are setup instructions on the main page which should hopefully tell you how to install depending on which operating system you are using with your Pi. If you give me more detail about your specific issue I might be able to help.
https://github.com/Hwulex/hyperion-config-switch/blob/master/README.md

Closing as over a year old and no response from user. Please re-open if still and issue and need support.


feat: eject azion.config.js file

This PR implements the ejection of azion.config.js. Now whenever the build is done, this file will be created in the project directory containing the preset configurations to run on Azion (azion.config.js).
The reason is to make it clear what the rules engine are and how they work.
Bonus: fixes some things that were left behind.

:tada: This PR is included in version 2.7.0-stage.1 :tada:
The release is available on:

npm package (@stage dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


cache-control / max-age on images

Hello!
I am a huge fan of droppy, and I use it on almost all of my servers (raspberry pis and droplets). I have a few personal hobby websites that I embed images from directly from my droppy (using the sharing link), and noticed that the requests are not being cached.

I was wondering if there was something I could set in the config, or the client, to get image or certain file types to have certain headers (for a longer cache).
Let me know if you would like to have this implemented in the project, or if you think it would be a good idea. I could work on a PR for it if maintainers are working on other things.
Thanks!

I opted to not set any cache headers because the content can potentially be quite dynamic, but I could add an option where you specify a cache duration on sharelinks, in seconds. Which value would you set?
Oh, and by the way: From your screenshot, I see that your links are created as downloads. For embedding images in websites, it'd probably be better to uncheck the download option.

Hello!
Nice to hear back, that was super fast! Thanks dude!
That would be super cool, If you would like I can open a PR for it, doesn't sound too bad. I just usually open issues to discuss ideas on peoples projects before trying to throw something in it haha!
For static assests I'm using on my blog and Github repos, they haven't changed in months, and when they do, usually it's a new link, so I'd probably set it for the max, or something reasonably long.
And I'll try unchecking download, thanks!
Also, after doing a bit more research, do you think I could set the headers/caching in nginx? Was looking at my nginx config, and noticed that may be an option.
Thanks again for the quick response!

do you think I could set the headers/caching in nginx?

Yes, I think something like this might work (setting cache duration to 1 month on all sharelinks):
location /$/ {
  add_header "Cache-Control" "public, max-age=2678400";
}

As for a PR, I think I'll decline that one unless you're a expert on browser caching 😉 . It's not that trivial of a change because one has to consider dev mode where I do not like to have any sort of caching and then there's also the ETag caching mechanism (only used for static resources right now). I think I'll investigate applying ETag caching to all resources as it's a mechanism that can work without me having to add another option.

Oh nice, I'll try that out tonight! Thanks!
And I am no means an expert on browser caching haha! And that does sound nice. Yeah development would be a pain if there was any type of caching enabled. Thanks again for the consideration!

Okay, I've added a weak ETag-based cache to all file downloads. It's based on file path and last modification date, which should be sufficient for our use. I'll release v6.6.10 with it in a few minutes.

Oh nice! Thanks for that! Also, the little nginx snippet you passed me works well! I actually recently just converted to Caddy Since it's so much easier to set-up/maintin. But either way, I got it working from there.
Though, I am quite excited to experiment with the Etag as well, I try this out when it's not 2AM where I am haha! So maybe tommorow night?


Path to channels

Hello, how can I provide the channels setting as a path on my checkout, instead of a string in the action?

Untested, but perhaps you can read the file into a variable called CHANNELS in an earlier step (https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-environment-variable) and the use ${env.CHANNELS} in the with: part?


ci: adding and using ci:shipit

Description

adds a new script, ci:shipit , that just executes semantic-release
updates GH Action workflow to use new script

Compatibility

OS
Implemented

iOS
N/A

Android
N/A

Windows
N/A

Checklist

[ ] I have tested this on a device/simulator for each compatible OS
[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (privateTypes.ts, types.ts)
[ ] I added a sample use of the API (example/App.js)

:shipit:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


randomly ticking mob problem with lootbeams.

There seems to be a mod incompatibility with lootbeams.
it seems that some part of lootbeams try to call java.lang.NoSuchMethodError: 'dev.shadowsoffire.placebo.reload.TypeKeyed dev.shadowsoffire.placebo.reload.DynamicHolder.get()' if you wish for me to go put this in placebo or lootbeams feel free to tell me.
files
crash-2024-04-14_07.10.59-server.txt
logs: https://mclo.gs/J7gGRFN.

Looks like loot beams is compiled against an older version of Placebo - they'll need to rebuild against latest.


Swap ppc64le builds to use dynamic docker images

We have been creating a centos7 docker build image for quite some time.
I have managed to get jenkins to dynamically provision machines using the OSUOSL openstack and have run some test builds:
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-linux-ppc64le-hotspot/659/
https://ci.adoptopenjdk.net/job/build-scripts/job/jobs/job/jdk8u/job/jdk8u-linux-ppc64le-openj9/719/
My proposal is to switch jdk15 and jdk16 builds to use these machines for a while and then we can consider switching JDK11 and JDK8 too. My reason for holding off is that we will have to eventually delete the static builds machines to free up some resource and I want to prove this out first.
Any objections @sxa??

From my POV, it makes sense to use stateless containers for building (less infra problems). But before switching over more builds, we should look at some potential issues:

If we need to alter the images as part of a PR to openjdk-build, those changes aren't tested because the builds use an image on Docker Hub.
In my (very limited) experience, container and non-container builds aren't identical (different parts of the build script are executed). So if we switch entire platforms over to container builds, we're very likely going to see bit rot in the build logic.

@aahlenst the docker image gets pulled from the openjdk-infrastructure repo. (not to be confused with the dockerfiles that live in this repo).
These docker images are an identical copy of our build machines today (they have the full ansible-playbook run on them) which means that the build is going to be identical.

You cannot make such "identical" claims regarding docker images as the kernel is not that of a real distribution in most cases
(For that reason I'm very much against against exclusively running testing in docker containers where it's likely more relevant)
Saying that, if this works I'm ok with it going in, but we need to have something documented about the plugins being used and integration points for the docker stuff as I do not believe any of it has been documented yet.

@gdams Sorry, my fault, please ignore.

@sxa we've been successfully building JDK15 and JDK16 using the dynamic docker images for some time now. Would you be happy for me to migrate the JDK8 and JDK11 builds over too? It would require me to delete the two build machines that we have today

WHy does it mandating deleting the existing build machines? Since the builds for JDK8/11/15 don't overlap that shouldn't be a requirement as it doesn't need them all in parallel

@sxa I'm happy to make the change without deleting the build machines. But in the near future we will need to follow up with that task.

Yep that's fine. Happy for this change to be made now
(FYI @andrew-m-leonard)

This needs to be paused until https://github.com/AdoptOpenJDK/openjdk-build/issues/2244 is understood oand resolved as the builds have been been failing repeatedly on JDK15

This needs to be paused until AdoptOpenJDK/openjdk-build#2244 is understood and resolved as the builds have been failing repeatedly on JDK15

Looks like builds are OK again expect for some xlinux builds on PR testers.  So safe to proceed?

This needs to be paused until AdoptOpenJDK/openjdk-build#2244 is understood and resolved as the builds have been failing repeatedly on JDK15
Looks like builds are OK again expect for some xlinux builds on PR testers. So safe to proceed?

Not sure what that's based on ... The ppc64le builds are ok again because I have disabled docker builds for ppc64le as per https://github.com/AdoptOpenJDK/openjdk-build/pull/2246 therefore rollout of that to other platforms needs to be halted until we've resolved why and been able to reinstate it safely.


Simulator Ignores Endless Walk "Moving" Setting

Under the skills/effects tab I adjusted the "Moving" setting for endless walk and re-ran the simulator which resulted in the exact same DPS regardless of if moving was set to 0 or 5 or anywhere in between.
Thanks!

It does, movement in general isn't supported by the simulator atm. I might enable it for just Endless Walk, but there are plenty of other movement-dependent buffs, so it won't be very fair to them.

Fixed.


Remove deprecated metric metadata

What does this PR do?
Removing deprecated metrics on active directory check metadata.csv.
Motivation
Customer request + https://github.com/DataDog/integrations-core/blob/master/active_directory/datadog_checks/active_directory/active_directory.py#L39-L53

@masci done, there was more than one quote after all :p


Update FlatFileDataSourceViewModel.cs

Fix so we can edit rowDelimiter and ColumnDelimiter values

Hey @Gurris,
Thanks for the PR, appreciate the bug fix!
I made some minor changes to the capitalization of variables and pushed it to your branch. Unfortunately, the PR got stuck due to your branch naming not matching our naming standard, I should probably list that in some sort of developer guidelines. I think you'll need to change the branch name to fix/your-branchname-here or feature/your-branchname-here, using nothing but lowercase letters or dashes. Not sure if you'll need to re-create the PR as well.


Update sites.yml

alexeystar.com size updated

This PR is:

[ ] Adding a new domain
[x] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[x] I used the uncompressed size of the site
[x] I have included a link to the Cloudflare report
[x] This site is not an ultra minimal site
[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: alexeystar.com
  url: https://alexeystar.com/
  size: 5.91
  last_checked: 2024-12-14

Cloudflare Report: https://radar.cloudflare.com/scan/120b4ff6-5f37-44fd-8266-d4c8d2f0d0f3/summary

Too minimal, I'm afraid.


Add declarations for attributes and varyings

Hi @oakes, I really like what you've done here. Very nice being able to declaratively build shader programs in lisp syntax. I started using this with regl-project/regl and I had a need for attribute and varying state variables, so I added them to the library.
I was doing this by string-appending at first, but the code for adding the glsl state vars looked pretty simple and I got it working with my test case. Figured this might be useful to someone else, so feel free to pull if you like. I agree to dedicate this as a contribution in the public domain. Thanks for your work!

Is this because you're using an old version of opengl? I think the attribute/varying keywords are deprecated, see here: https://gamedev.stackexchange.com/a/29673

Ah yeah, I might be having that old tutorial code problem. This is what I referenced from the regl project https://github.com/regl-project/regl/blob/master/example/texture.js
Looking at the file history, it looks like that shader was originally added in 2016. I guess they're using older GLSL syntax for wider compatibility over WebGL1/GLES2 platforms? I guess I'll try using the newer GLSL syntax and fall back to this stuff if it's really necessary. Sorry to bug ya

no problem. i'll close for now but let me know if you notice the in/out keywords don't work somewhere, i wouldn't mind adding the older syntax if there is a big incompatibility.
regl looks really cool btw. i basically made play-cljc as a clojure equivalent to it.

play-cljc might actually be a nicer alternative to what I'm doing. I just jumped on regl since it looked like the simplest way to deal with GL canvas, but if play-cljc does all the GL stuff anyway I'll take a closer look sometime. Thanks man!

@oakes Just coming back to update with more info. I guess the decision on regl's part is still somewhat intentional, you check regl-project/regl#378 for more information.
It pretty much boils down to the fact that there wasn't enough of a rally behind WebGL2 from platforms (big boys like Mozilla still mark it as experimental https://developer.mozilla.org/en-US/docs/Web/API/WebGL2RenderingContext ), so they stuck with WebGL1 for wider support and are waiting to see if any competing standards will grow more quickly. For the moment, there's no explicit WebGL2 support in regl, and therefore GLSL 3 and above shaders won't compile. This is true in general for WebGL1 as well.
I understand this library goes well beyond use cases in the web, so I understand why you wouldn't want the world's jankiest platform making decisions for you. This library is still really nice to use either way, so thanks again for the hard work!

@jjsullivan5196 according to this page (in the very last section) the attribute/varying qualifiers were deprecated in GLSL 1.30. I haven't tried it but i assume the in/out qualifiers would work in webgl 1.

oh, i just realized you may have been referring to play-cljc. it requires webgl 2 right now, but honestly that may not even be necessary. i just haven't tried it with webgl 1 yet so i made the spec check for a WebGL2RenderingContext for now. it would be good to support webgl 1 since safari hasn't even implemented 2 yet.

@oakes OpenGL ES specifies it's own version of the GLSL language, and opengl es 2/webgl 1 use the first version which doesn't support the new storage qualifiers
https://www.khronos.org/files/opengles_shading_language.pdf
https://www.khronos.org/registry/webgl/specs/1.0/
I tried using the new storage qualifiers initially with regl, and it would throw when it compiled the shaders. I wasn't sure if this was regl just rejecting the syntax, but I tried it here as well and saw that the exception was thrown by WebGLShader itself *** Error compiling shader '[object WebGLShader]':ERROR: 0:3: 'in' : storage qualifier supported in GLSL ES 3.00 and above only

Ah good to know. I'll merge this then.

Cool, sorry for slipping up on the jargon, it's hard to keep track of all these spec inconsistencies :^)

No problem. I pushed 0.9.2 to clojars in case you want to use the maven version. thanks again!


optimize core module import

Ⅰ. Describe what this PR did
optimize core module import
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #991 into develop will increase coverage by 0.11%.
The diff coverage is 0%.

@@              Coverage Diff              @@
##             develop     #991      +/-   ##
=============================================
+ Coverage      38.34%   38.45%   +0.11%
  Complexity      1046     1046
=============================================
  Files            219      218       -1
  Lines           8714     8687      -27
  Branches        1085     1081       -4
=============================================
  Hits            3341     3341
+ Misses          4951     4924      -27
  Partials         422      422

Impacted Files
Coverage Δ
Complexity Δ

...e/protocol/transaction/GlobalLockQueryRequest.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...otocol/transaction/AbstractTransactionRequest.java
100% <ø> (ø)
1 <0> (ø)
:arrow_down:

...core/protocol/transaction/BranchReportRequest.java
80.68% <ø> (ø)
16 <0> (ø)
:arrow_down:

...core/protocol/transaction/GlobalCommitRequest.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...rc/main/java/io/seata/core/model/GlobalStatus.java
100% <ø> (ø)
7 <0> (ø)
:arrow_down:

...core/protocol/transaction/GlobalStatusRequest.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...tocol/transaction/AbstractTransactionResponse.java
75% <ø> (ø)
5 <0> (ø)
:arrow_down:

.../core/protocol/transaction/GlobalBeginRequest.java
86.48% <ø> (ø)
9 <0> (ø)
:arrow_down:

...re/protocol/transaction/BranchRollbackRequest.java
33.33% <ø> (ø)
1 <0> (ø)
:arrow_down:

...re/protocol/transaction/BranchRegisterRequest.java
73.68% <ø> (ø)
13 <0> (ø)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


3.5mm output works only when booted with display

Hi all,
Happy to be a new member of this community! I have recently acquired a Raspberry Pi 3 Model B with the intention of using it solely as a Spotify Connect device via Raspotify.
After installing Raspotify, I had issues getting Raspotify to output to the 3.5mm jack, even whilst other sources such as the browser had no issue doing so. This was solved by changing the defaults.ctl.card and defaults.pcm card values from 0 to 1 in /usr/share/alsa/alsa.conf
3.5mm output now works with Raspotify, but only when Raspberry Pi is booted with a display connected. If booted without a display, there is no audio output. My goal is to have this Raspberry Pi as a headless playback device so this is less than ideal. I read in another thread that pulseaudio has caused a lot of issues with 3.5mm audio so I uninstalled pulseaudio; nevertheless this issue persists.
Any/all feedback is highly appreciated.
Best,
Alden

You may have to change this back to zero via ssh from another computer again.
To me it was the other way around. Once I connected the display, the audio stopped working.
I tried editing the config.txt as suggested here, but that didn't help.

You may have to change this back to zero via ssh from another computer again.
To me it was the other way around. Once I connected the display, the audio stopped working.
I tried editing the config.txt as suggested here, but that didn't help.

Checkout this pull request I just made (https://github.com/dtcooper/raspotify/pull/386). Had the same problem and made a fix to the docs. Basically for me when you plugged in a display the hw device number changed, so rather than selected a device by number, do it by name

# Available devices can be listed with the following command:
# librespot --name hello --device ? | grep plughw -v | grep hwdep -v | grep hw
# This lists available devices (note b1=HDMI1, b2=HDMI2 etc.).
# If you can’t find your device with that command, try running:
# librespot --name hello --device ?
# This gives a more verbose output, which can help you identify the target
# sound card if it has a weird name
# Say you want Spotify to play out of HDMI1, you run that command, and copy the
# entire line b1 then you add '--device ' with your preferred device to
# the OPTIONS field below. For example, you want to set Spotify to output out of
# HDMI1 you run the command, copy the line 'hw:CARD=b1,DEV=0' from the output and
# add '--device hw:CARD=b1,DEV=0' to options. Another example using a hardware
# DAC would be '--device hw:CARD=IQaudIODAC,DEV=0'
# This method is useful, as the name wont change depending on which devices are
# connected to the Pi, as was with the old way (i.e. --device hw:0,0)
# This might not work if you're using a USB sound card, and you'll be stuck to
# specifying devices like --device hw:0,0


how to launch live preview with a server-side file

I'm trying to launch live preview with a server-side file, I have already installed LAMP and everything is running smoothly, I also changed the permissions of the localhost folder using chown in the terminal. I can run through my local server php files. but inside brackets, when I click the thunder, a pop-up message appears asking for a Live Preview Base URL: and it asks for a url like http://localhost:8000/ but it doesn't seem to work.
what can I do?
Thanks.

I know how to fix the problem:
in a php file with the function phpinfo();
scroll to the hostname:port of the apache2 handler and there will be an address whith the colon and the port number.


can't use np>1

When I run the command: torchpack dist-run -np 4 python tools/train.py
configs/nuscenes/det/transfusion/secfpn/camera+lidar/swint_v0p075/convfuser.yaml
--model.encoders.camera.backbone.init_cfg.checkpoint pretrained/swint-nuimages-pretrained.pth
--load_from pretrained/lidar-only-det.pth
--run-dir output/bev_result/
I get this problem:

And when I want to train the camera-only variant for object detection, I get this:

Does anyone know how to fix it？Thanks！

I think you need to provide more error info

the number of np must match the number of gpus.

That's all the information, only a few lines. @quantumdotsss

I know,  I adhere strictly to this point,  but it always goes wrong. @VanderBieu

same question.

same question.

I started with the environment configuration all over again and it worked.

same question when I run the test.py
error: address already use

 same question when I run the test.py error: address already use

Hello, I have also encountered this problem. Have you resolved it?


[REQ] fix license on GH

Hi there, 1st of all thanks for your awesome work !
Since we've "doxed" it in our HyMPS project (under the VIDEO \ AI-based \ Editing subsection), can you please add the license to it ?
Expliciting the licensing terms is extremely important to let other devs (and not only) better/faster understand how to reuse/adapt/modify your code in other open projects and vice-versa.
Although it may sounds like a minor aspect, license file omission obviously causes an inconsistent generation of the relative badge too:

(badge-generator URL: https://flat.badgen.net/github/license/parthdomadia/VideoEditor/?label=LICENSE)
Anyway you can easily set a standardized one through the GH's license wizard tool.
Last but not least, let us know how - in your opinion - we could improve our categorizations and links to resources in order to favor collaboration between developers (and therefore evolution) of listed projects.
Thanks in advance.

Have added a new license to the repo, thank you for pointing this out and for using this repo!


No global import of testnet constants

At the moment, a number of places are directly importing the testnet constants. We likely need to have an alternative approach than this kind of hardcoding method.

Duplicate to: https://github.com/penumbra-zone/web/issues/900


[Bug] Recursive references in undiscriminated unions

Describe the bug
If I define a recursive type in a undiscriminated union, Fern forgets to surround the recursive type in quotes causing compilation errors.
OpenAPI Schema To Reproduce
# shape.yml
    Json:
      oneOf:
      - type: boolean
      - type: integer
      - type: number
        format: double
      - type: string
        minLength: 1
      - type: array
        items:
          $ref: '#/components/schemas/Json'
      - type: object
        additionalProperties:
          $ref: '#/components/schemas/Json'
      nullable: true

From here
Expected behavior
Json = typing.Union[bool, int, float, str, typing.List[typing.Optional["Json"]], typing.Dict[str, typing.Optional["Json"]]]
Screenshots
If applicable, add screenshots to help explain your problem.
Versions

Fern CLI: "0.15.0-rc74"
Generator: 0.6.5

Additional context
Slack Thread

This should now be fixed!


CompareInfo not available in Blazor WASM

Describe the bug
We are trying to implement filtering items on the web UI by a filter text being typed into a text box. For convenience, we want to ignore casing and diacritics/accents as well.
Ignoring case works fine with specifying StringComparison, but ignoring diacritics doesn't.
One of the most popular solution for this problem is this: https://stackoverflow.com/a/249126 It uses normalization and Unicode categories, but even though it works, in the end it creates a new string which is not cost effective, especially if you want to use it for real-time typing and filtering of data.
So, the optimal way would be to use the built-in CompareInfo class with the IgnoreNonSpace option, that really only compares two strings, without having to create new ones during the search. The IndexOf method gives back no results, even though there are. But there are is exception either, like it is not supported, or culture data is missing to perform that operation.
To Reproduce
@(CultureInfo.InvariantCulture.CompareInfo.IndexOf("Elemzések", "elemze", CompareOptions.IgnoreCase | CompareOptions.IgnoreNonSpace))

Gives -1 instead of 0. If I run the same in a .NET Core Console Application, it works fine.
I've also tried it with CurrentCulture or specifying the culture by its name or id.
Further technical details

ASP.NET Core version: 3.0
Include the output of dotnet --info: 3.1.100-preview2-014569
The IDE (VS / VS Code/ VS4Mac) you're running on, and it's version: Visual Studio 19.4 Preview 4

@Peter-Juhasz thanks for contacting us.
We can see there might be a problem here, but this is likely an issue with mono. We will route the issue to them appropriately.

This issue was moved to mono/mono#17780


Data table subset

Reworked table framework a bit so that only the current rows are being renderered (instead of hiding/showing).
It appears to be much faster on large data sets now, but I broke the sort functionality, so we'll need to clean that up before merging this in.  ;)
cc: @samussiah

@samussiah Alrighty - think this is all done and ready for your review now.


Custom attribute for event-value

Is it possible to add a similar data attribute for value?
data-analytics-value.
Currently it's missing.
Thanks.

@Senseye I just published v2.0.0 which adds data-analytics-value. I'm going to close this issue. If you have any problems, please feel free to open a new issue. Thanks!

@miguelzakharia thanks a lot!


refactor(optimizer): ExportData.imports to ExportData.hasImports

Description
ExportData.imports was only used for import.length > 0.
Rewrote imports into hasImports to remove depending on es-module-lexer's type.
refs #8352
Additional context
ExportData types needs to be exported because

optimizeDeps which is exported from src/node/index.ts returns DepOptimizationMetadata type including ExportData type

the return type of optimizeDeps could be changed to Promise<void> to avoid breaking change. I think the return type of optimizeDeps is too specific if it is considered as an public api.
optimizeDeps is used only from the cli

resolvedConfig.createResolver's param type is InternalResolveOptions and it includes ExportData

getDepsOptimizer could be removed from resolvedConfig.createResolver's param type

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

It look like this PR makes CI super flaky. 🤨

I don't think it is the PR, the CI is super flaky today. Failing even before starting, when it is installing pnpm in mac os for example.
Also, wow... nice move with this change 🙌🏼


Update partial.md

The sentence "An analogous capability is planned for Razor Pages in ASP.NET Core 2.2" is outdated.

Thank you for the PR, @EKanadily!


Frontend rule doesn't work on Docker

Do you want to request a feature or report a bug?
bug
What did you do?
I launch a container with label
docker run --name test --label "traefik.frontend.rule=Host:test.com" --label "traefik.enable=true" emilevauge/whoami

What did you expect to see?
I expect to see my new container in the list of frontend/backend
What did you see instead?
My container is filtered
Output of traefik version: (What version of Traefik are you using?)
Version:      v1.6.0-rc2
Codename:     tetedemoine
Go version:   go1.10
Built:        2018-03-27_03:02:20PM
OS/Arch:      linux/amd64

What is your environment & configuration (arguments, toml, provider, platform, ...)?
docker run -v /var/run/docker.sock:/var/run/docker.sock traefik:1.6.0-rc2 --docker --api --debug

If applicable, please paste the log output at DEBUG level (--logLevel=DEBUG switch)
time="2018-03-28T13:49:24Z" level=debug msg="Filtering container with empty frontend rule /test "

Closed by #3091.


User Properties are not shown with BRIEF message dump

When providing a "formatted" message dump, and using BRIEF dump for the header information, the User Properties are not printed out.
Make/duplicate the output of SdkPerf -md wrt to User Properties.  And maybe double-check User Data as well.

This has been fixed in the latest for some time.  Required implementing custom SDTMap and SDTStream print methods.


Latest launches has more objects than All

Latest Launches does not have a date cap but All does. This leads to a larger number of objects in the Latest table than in all which does not seem right. Likely best to cap Latest to more recently spotted objects than All uses.

Also need to add a cap to priorities so we don't have a flooded priorities list


Update to buildkit v0.10.0

This should help improve build times

Closing in favor of https://github.com/gitpod-io/dazzle/pull/48


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_101cee241p/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_101cee241p/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_101cee241p/clone/recipe/meta.yaml:43: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_101cee241p/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_101cee241p/clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description
ERROR: /tmp/abs_101cee241p/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:21: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:43: missing_description: The recipe is missing a description
ERROR: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:43: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:43: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_7egqtpox32/clone/recipe/meta.yaml:43: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Display menu options from 1 up

Display menu options from 1 and up instead of the current state that the options counting starts from 0

+1

@Tronchoo Cool. Will look into this.

So I have a PR out there but I am trying to decide on the new version of WMenu. v4.0.2 is the latest. If I say v4.1.0 that is saying that code already using this package is compatible with this new version (which not all code will be). Depending on how it was developed it will break (if they use IDs for example). But all method signatures (except for one) has stayed the same. What do you think?

@dixonwille Working great, no more keyboard-distance between the first two options :)

As a heads up, I just got burned by this because after this change, "ID" no longer lines up with the display. So if a user picks "2", I have to math that before using it as an ID in the options list.
This is pretty hard to handle in a backwards compatible way, since depending on the wmenu version I might or might not have to subtract 1 from the user input. Would it make more sense to have IDs be incremented by one so they always line up with what is displayed?

@akerl I may have to create another package... This was one of my first GoLang packages so I was just learning. It will have GoMod support and try and cover all issues on in this package and hopefully in a better backwards compatible manner. It would make sense with what you are saying to have them lined up or use a different ID method?


It's been resolved. Check with the new update.


add contributors

Comment on Issue or Pull Request, asking @all-contributors to add a contributor:
@all-contributors please add @<username> for <contributions>

@all-contributors please add @Forest-L  for code,doc

@all-contributors please add @Forest-L  for code,doc

@all-contributors please add @Forest-L for code, doc

@all-contributors please add @Forest-L for code, doc

@all-contributors please add @zryfish  for  doc

@all-contributors please add @zryfish  for  doc

@all-contributors please add @shaowenchen  for code

@all-contributors please add @shaowenchen  for code

@all-contributors

@all-contributors

@all-contributors please add @shaowenchen for code

@all-contributors please add @shaowenchen for code

@all-contributors please add @pixiake  for doc

@all-contributors please add @pixiake  for doc

@all-contributors please add @pixiake  for doc

@all-contributors please add @pixiake  for doc


Errors on single culture site with more than one language configured

I'm seeing a lot of "GetUrl exception" messages in the log on a site that isn't using the "vary by culture"-system, but has more than one language configured.
It looks to me like Skipper is trying to create an URL for all the languages, not just the one used on the page. This probably relates to this issue: https://github.com/umbraco/Umbraco-CMS/issues/11234

Hi @torerikk
Can you please share your logs?
Thank you!

Hi
Just by opening a page that is a child of a "skipped" page I get this error:
Jun 27, 2022 1:17:57 PM | Error | FW-IDEAPAD3 | GetUrl exception.

System.ArgumentNullException: Value cannot be null. (Parameter 'Base GetUrl for Id 1148.')
at Our.Umbraco.Skipper.SkipperUrlProvider.BuildUrl(IPublishedContent content, Uri current, UrlMode mode, String culture)    at Our.Umbraco.Skipper.SkipperUrlProvider.GetUrl(IPublishedContent content, UrlMode mode, String culture, Uri current)
at Umbraco.Cms.Core.Routing.UrlProvider.<>c__DisplayClass14_0.<GetUrl>b__1(IUrlProvider provider)
at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Func`2 predicate, Boolean& found)
at Umbraco.Cms.Core.Routing.UrlProvider.GetUrl(IPublishedContent content, UrlMode mode, String culture, Uri current)
at Umbraco.Extensions.UrlProviderExtensions.GetContentUrlsByCultureAsync(IContent content, IEnumerable`1 cultures, IPublishedRouter publishedRouter, IUmbracoContext umbracoContext, IContentService contentService, ILocalizedTextService textService, IVariationContextAccessor variationContextAccessor, ILogger logger, UriUtility uriUtility, IPublishedUrlProvider publishedUrlProvider)

I don't have a lot of time today, if you need a more detailed debug log I can share that later this week.

This is also happening to me, to add some more context - it seems to happen whenever you have a published node of a doc type that doesnt have 'Allow vary by culture' set to true.  The error will occur when the page is published (not saved), and any time you navigate to the node.
In my case it happens whenever I forget to set a new doc type to allow vary by culture (which is quite often). It's not a big deal for me to fix this, but there may be some cases where you don't need or want to allow content to vary by content.
Here is my stack trace:
at Our.Umbraco.Skipper.SkipperUrlProvider.BuildUrl(IPublishedContent content, Uri current, UrlMode mode, String culture)
   at Our.Umbraco.Skipper.SkipperUrlProvider.GetUrl(IPublishedContent content, UrlMode mode, String culture, Uri current)
   at Umbraco.Cms.Core.Routing.UrlProvider.<>c__DisplayClass14_0.<GetUrl>b__1(IUrlProvider provider)
   at System.Linq.Enumerable.SelectEnumerableIterator`2.MoveNext()
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Func`2 predicate, Boolean& found)
   at Umbraco.Cms.Core.Routing.UrlProvider.GetUrl(IPublishedContent content, UrlMode mode, String culture, Uri current)
   at Umbraco.Extensions.PublishedContentExtensions.Url(IPublishedContent content, IPublishedUrlProvider publishedUrlProvider, String culture, UrlMode mode)
   at Our.Umbraco.Skipper.SkipperContentFinder.<>c__DisplayClass4_0.<TryFindContent>b__0(IPublishedContent x)
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Func`2 predicate, Boolean& found)
   at Our.Umbraco.Skipper.SkipperContentFinder.TryFindContent(IPublishedRequestBuilder request)
   at Umbraco.Cms.Core.Routing.PublishedRouter.<>c__DisplayClass24_0.<FindPublishedContent>b__0(IContentFinder finder)
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source, Func`2 predicate)
   at Umbraco.Cms.Core.Routing.PublishedRouter.FindPublishedContent(IPublishedRequestBuilder request)
   at Umbraco.Cms.Core.Routing.PublishedRouter.TryRouteRequest(IPublishedRequestBuilder request)
   at Umbraco.Cms.Core.Routing.PublishedRouter.RouteRequestAsync(IPublishedRequestBuilder builder, RouteRequestOptions options)
   at Umbraco.Extensions.UrlProviderExtensions.DetectCollisionAsync(ILogger logger, IContent content, String url, String culture, IUmbracoContext umbracoContext, IPublishedRouter publishedRouter, ILocalizedTextService textService, IVariationContextAccessor variationContextAccessor, UriUtility uriUtility)
   at Umbraco.Extensions.UrlProviderExtensions.GetContentUrlsByCultureAsync(IContent content, IEnumerable`1 cultures, IPublishedRouter publishedRouter, IUmbracoContext umbracoContext, IContentService contentService, ILocalizedTextService textService, IVariationContextAccessor variationContextAccessor, ILogger logger, UriUtility uriUtility, IPublishedUrlProvider publishedUrlProvider)
   at Umbraco.Extensions.UrlProviderExtensions.GetContentUrlsAsync(IContent content, IPublishedRouter publishedRouter, IUmbracoContext umbracoContext, ILocalizationService localizationService, ILocalizedTextService textService, IContentService contentService, IVariationContextAccessor variationContextAccessor, ILogger`1 logger, UriUtility uriUtility, IPublishedUrlProvider publishedUrlProvider)
   at Umbraco.Cms.Web.BackOffice.Mapping.ContentMapDefinition.GetUrls(IContent source)
   at Umbraco.Cms.Web.BackOffice.Mapping.ContentMapDefinition.Map(IContent source, ContentItemDisplay target, MapperContext context)
   at Umbraco.Cms.Core.Mapping.UmbracoMapper.Map[TTarget](Object source, Type sourceType, MapperContext context)
   at Umbraco.Cms.Core.Mapping.UmbracoMapper.Map[TTarget](Object source, MapperContext context)
   at Umbraco.Cms.Web.BackOffice.Controllers.ContentController.MapToDisplay(IContent content, Action`1 contextOptions)
   at Umbraco.Cms.Web.BackOffice.Controllers.ContentController.GetById(Int32 id)
   at lambda_method1090(Closure , Object , Object[] )
   at Microsoft.AspNetCore.Mvc.Infrastructure.ActionMethodExecutor.SyncObjectResultExecutor.Execute(IActionResultTypeMapper mapper, ObjectMethodExecutor executor, Object controller, Object[] arguments)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeActionMethodAsync()
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeNextActionFilterAsync()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Rethrow(ActionExecutedContextSealed context)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)
   at Microsoft.AspNetCore.Mvc.Infrastructure.ControllerActionInvoker.InvokeInnerFilterAsync()
--- End of stack trace from previous location ---
   at Microsoft.AspNetCore.Mvc.Infrastructure.ResourceInvoker.<InvokeNextExceptionFilterAsync>g__Awaited|25_0(ResourceInvoker invoker, Task lastTask, State next, Scope scope, Object state, Boolean isCompleted)

Hi @torerikk
Can you update and retry?
Thank you!

Looks like the problem is fixed now :) Thanks!


fix: change type of slack error

SUMMARY
Small change to log a 400 error if there are slack api errors caught. Also fixes a parsing error in the exception handling where we were trying to parse an exception as a string.
TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

🏷 preset:2023.01


Specify jvm memory limits in gradle.properties

Trying to figure out why there is a OutOfMemoryError on the CI

Do we plan to merge this?

Just increasing the memory does not feel right, but it seems like the only option for the moment as it does not seem to be possible to run the gradle test kit runner without daemon.
Maybe this can bring some live to the discussion in #3778.

Do we plan to merge this?

But to answer your question. Yes, I would merge this for the time beeing.


Thanks for the PR, I feel that the readme is already pretty long and hard to navigate to find the right information so I'm a bit unsure having the history of number of stars would add that much relevant information there.

No worry if we do close this PR, btw i was wondering if it could not be a good idea to move the README to WIKI section ?
To have a light version, what do you think ?


Spaces in TARGETDURATION and MEDIA-SEQUENCE tags

I've observed spaces in generated m3u8 from CDNs for these tags. Note the space after colon. Do we need to handle this as part of library? Spec doesn't say whether space is allowed.
should be #EXT-X-TARGETDURATION:<s>  but seen as #EXT-X-TARGETDURATION: <s>
should be #EXT-X-MEDIA-SEQUENCE:<number> but seen as #EXT-X-MEDIA-SEQUENCE: <number>

For now I've handled with below change in Constants.java.
public static final Pattern EXT_X_TARGETDURATION_PATTERN = Pattern.compile("^#" + EXT_X_TARGETDURATION_TAG + EXT_TAG_END + OPTIONAL_WHITESPACE_REGEX +"(" + INTEGER_REGEX + ")"+OPTIONAL_WHITESPACE_REGEX+"$");
public static final Pattern EXT_X_MEDIA_SEQUENCE_PATTERN = Pattern.compile("^#" + EXT_X_MEDIA_SEQUENCE_TAG + EXT_TAG_END + OPTIONAL_WHITESPACE_REGEX +"(" + INTEGER_REGEX + ")"+OPTIONAL_WHITESPACE_REGEX+"$");

Where regex for optional white space is as below
private static final String OPTIONAL_WHITESPACE_REGEX = "\\s*?";

Let me know if this makes sense. There is no harm if no spaces exist. regex takes care of that case too.

The regexes look good. I would add an allowWhitespaceInTags flag in ParsingMode and set it to true for the LENIENT constant. At this point I'm considering making LENIENT the default.

While  default LENIENT mode seems easy option with allowWhitespaceInTags set to true; Am wondering when would need STRICT parsing mode.
Let me know when can this be added? you need any PR for this?

You would need strict for non-forgiving M3u8 consumers (So far I haven't seen any - usually they are all very forgiving)


Add data.find_keys

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (f0efb13) to head (21bd352).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           29        30    +1
  Lines          512       520    +8
  Branches        43        44    +1
=========================================
+ Hits           512       520    +8

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


Hubot shuts down erroneously (heroku)

Running on the 1.0.2 distribution
After a few minutes of playing with Hubot, it stopped responding and I saw this in the logs (looking at the source it appears it received an "end" response from campfire?
2011-10-25T20:45:54+00:00 heroku[app.1]: State changed from starting to up
2011-10-25T20:45:56+00:00 app[app.1]: Streaming Connection closed. :(
2011-10-25T20:45:56+00:00 heroku[app.1]: Process exited
2011-10-25T20:45:57+00:00 heroku[app.1]: State changed from up to crashed

I'll bug my friends at heroku and get this sorted.

See #25 for fix.

Dupe of #25 (even though this one came first, it did not have code attached. Such are the laws of Issues)


Try macos 14

Github News https://github.blog/changelog/2024-01-30-github-actions-macos-14-sonoma-is-now-available/

A bugfix for this issue is available in https://github.com/pyiron/pympipool/pull/257


update to correct repo for deploy button

nope, bad idea. it seems like https://github.com/kkbankol-ibm/node-red-bluemix-starter does something else, but i wonder if we could roll some of that into this repo?


Add save & share functionality

Users should be able to save their patches to a unique URL for storing and sharing.
Slang is a static site served directly by nginx at the moment, so this will involve adding a super-simple express server (which should still serve index.html from /public as a static file) with:

a POST /save endpoint that accepts a JSON blob containing a text field with the slang code
a GET /:patch route that loads the patch from a mongo database (if it is found) and renders it to a template so that the code can be pre-populated in the editor

Stretch goals:

add an optional title field
add a "saved patches" UI which stores the title and URL to localStorage so your saves hang around

Wow ;D this is cool


"SWS/S&M: Select FX x for selected tracks": use  TrackFX_Show() instead of chunk patching if possible

I've done this some time ago but wanted to wait with submitting until the big SWS rework was finished.

Can you move the whatsnew line to the Miscellanious section below the Quantize one (so that it's alphabetically sorted)? (Unless you put it at the top to wait for the next release cycle?)
Also I'm thinking something along the line of +Improve performance of "SWS/S&M: Select FX x for selected tracks" when the FX chain is opened would have more meaning for those who don't know what API or chunk parsing is.
Additional improvement ideas:

Looks like updated is only written to and used as a boolean (1/0). If so, it should be one (also in the other overload of SelectTrackFX where the return value is used).
pLastSel is only used by ParsePatch and can be moved to its else block.

Unless you put it at the top to wait for the next release cycle?
That was the thought. :) (Thinking it's not a critical bugfix and this release has a lots of other new stuff already.)
Will do the other suggestions, thanks.

updated.

Found a bug (not new): the undo point is only created if the last selected track is updated.
updated = should be updated ||=

Oh so it did turn into a bugfix eventually. :D
I'll add the whatsnew line within v2.11.0 section then, so it could be added to the release.

Thanks.

I did a search for updated = [^tf] and found a few suspicious matches. I'll check (and possibly fix) those too.

Sory, could you rephrase in layman's terms please? :P
Is it about line endings, if there's anything I can enhance on my side let me know. :)

Getting what you're doing but gotta say the line seen in your pic ag 'updated = [^tf]' leaves to totally stumped.
If you're in the modd I'd be happy if you dissect it for me, otherwise just ignore. :)

It is a regular expression: [^tf] is a "character class" that matches any character that is not t nor f. The intent is to filter out updated = true and updated = false from the results.
(That being said a better–but longer to type–regex would have been updated = (?!true|false) where (?!...) is a "negative lookahead". This wouldn't filter out expressions such as updated = toto_bunny just because it begins with a t.)


Make contrast between active, inactive, enabled an disabled filters more clear

Story
Right now, it is hard to easily scan which filters are in use without having to read everything and squint your eyes. This is not accessible for people with visible impairments.

Proposed solution

Make filters with selected options bright blue
Make filters without selected options, with available options light blue
Make filters without selected options, without available options light gray

Success criteria

[ ] Filters that do have one or more filter options available
and does not have any filter options are selected
appear as "default", light blue
[ ] Filters that do have one or more filter options available
and does have filter options selected
appear as "primary", bright blue
[ ] Filters that do not have any fruitful filter options available (clicking options = 0 results)
and does have one or more filter options selected
appear as "primary", bright blue
[ ] Filters that do not have any fruitful filter options available (clicking options = 0 results)
and does not have any  one or more filter options selected
appear as "secondary", light gray

Implementation suggestion
Find the badges layouts here: https://biblio-ugent-prototype-emily-v2.vercel.app/styleguide/b-badges.html
Default: non-active filter + filter options available
badge-default

        Biblio status

Active filter
badge-primary

        Biblio status

Non-active filter + no filter options available
badge-secondary

        Biblio status

More context
Decided not to block anyone from clicking a filter box when there are no filter options available to check. It would require more complex rules and unnecessary blocks.

@mietcls do you have an example of a situation where there is a filter without filter options?  Example:

We need to show the user in that case it would be useless to open that filter to select any filter options in there.

@mietcls for input field like "Created since" and "Updated since", I cannot make the difference between "fruitful" and "not fruitfull"

@nicolasfranck Okay, we can always keep those active then. I'll adapt the issue.


ability to add dashboards via grafana.com url OR id

It would be really nice to be able to add a dashboard by only providing its grafana.com URL or ID in the same way you can when adding them via the Grafana UI

Hi @cuotos, the latest release lets you install dashboards from URLs (implemented in #43). For an example see https://github.com/integr8ly/grafana-operator/blob/master/deploy/examples/dashboards/DashboardFromURL.yaml#L9
In this case the json field is the fallback option if the URL can't be resolved. This version is not yet available on operatorhub.io (but we have a Github release for it). Trying to update the listing on operatorhub as soon as possible.
Is this what you were thinking or do you mean there would be additional value in allowing users to just specify the ID and the operator would then automatically download the dashboard from grafana.com?

The latter is what I was thinking. Another option in the GrafanaDashboard object.
json,
url,
id
as part of spiking this anyway, it would be nice to just drop the id in an object and the operator take care of the rest. currently I have been manually importing its via the ID in the ui. You dont appear to be able to hotlink to the JSON in the Grafana dashboards repo, only down the JSON and host it somewhere myself.

I think you can link to the json in the grafana dashboards repository using this scheme: https://grafana.com/api/dashboards/<ID>/revisions/<REVISION>/download, e.g. https://grafana.com/api/dashboards/10441/revisions/1/download

Hmm, but thats actually a file download, not sure if it would work as a URL in the dashboard CR

Yep - I tried what was suggested in https://github.com/integr8ly/grafana-operator/issues/45#issuecomment-529495895 and it failed to load the dashboard, falling back to my fallback json.

I'll take a look at this

I've not managed to replicate what was observed in terms of URLs of this form: https://grafana.com/api/dashboards/10441/revisions/1/download not woking (possibly the code or the URL output has changed in the past 1+ year.
With this in mind, I think we can implement this now.
Propose adding a new field grafanaCloud that has fields id and revision.

It's invalid to specify both grafanaCloud and url.
If revision is unset, we look up the available revisions and pick the latest one.
As for url, if it can't be resolved (either due to invalid specified id or revision) we fall back to the contents of json, if set.

@DeanBrunt @cuotos this sounds reasonable. Adding a new field to the Dashboard CR (id) is the way to go with the usual fallback to json if the request fails.


getAttribute in Node should add "@"

From<EMAIL_ADDRESS>on February 18, 2011 20:39:14
Add @ if not already defined
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=34

From<EMAIL_ADDRESS>on February 20, 2011 09:56:18
Status: Fixed


Pull in cherry picked stm32f4 fixes from lesshonor

lesshonor has graciously assembled these fixes from newer Zephyr release to address issues with the Zephyr 3.5 USB support for F4. Creating this PR that includes their work to fix F4 for now until we update Zephyr versions.

Tested on F4, where I confirmed the fix works as expected, and tested on my stm32f072 Le Chiffre PCB, and no regressions there. Merging.


Semantic versioning constraint in 'helm search' doesn't work without --versions

It will only ever report the latest version, or nothing at all if that doesn't satisfy the constraint. For example, if my-chart exists at versions 1.0, 1.1, 1.2, and 2.0 then:
> helm search --version "> 1, < 2" my-chart
No results found

This appears to be because the calls to i.AddRepo() in searchCmd.buildIndex() do not set the all flag, so the constraint is only ever applied against the latest version of each matching chart.
Note that it does apply the constraint if --versions is also specified, eg.
> helm search --versions --version "> 1, < 2" my-chart
NAME      VERSION  DESCRIPTION
my-chart   1.2.0    ...
my-chart   1.1.0    ...

but it doesn't seem like this should be necessary, and the help text doesn't imply this.
In summary, specifying --version alone should give the single (latest) matching version.

I have a change (PR coming shortly) that will make it behave like this instead:
> helm search --versions my-chart
NAME      VERSION  DESCRIPTION
my-chart   2.0.0    ...
my-chart   1.2.0    ...
my-chart   1.1.0    ...
my-chart   1.0.0    ...

> helm search --versions --version "> 1, < 2" my-chart
NAME      VERSION  DESCRIPTION
my-chart   1.2.0    ...
my-chart   1.1.0    ...

> helm search --version "> 1, < 2" my-chart
NAME      VERSION  DESCRIPTION
my-chart   1.2.0    ...

> helm search app-comet-explorer
NAME      VERSION  DESCRIPTION
my-chart   2.0.0    ...


Update the link to cloud interface 📝

The link to the cloud interface was for a past release. Therefore I updated the link to point to the latest stable release.
Cheers 🥂  and keep up being awesome 🏆

/assign @steveperry-53

/assign @steveperry-53


Movie list cli doesn't have forget option

Add forget option to fire forget event

Not actually relevant, should only fire forget events for exact titles, and movie list has none of those (just movie titles)


Hypridle Core Dumped - Illegal Hardware Instruction

Hi there, I'm running Hyprland on Freebsd 14.1. Everything has been really good but I can't get Hypridle to work.  exec-once = hypridle leaves a coredump in my home directory. The following is what I get trying to run hypridle from the terminal:
//  hypridle
[LOG] Registered timeout rule for 60s:
on-timeout: hyprctl dispatch dpms off
on-resume: hyprctl dispatch dpms on
[LOG] Registered timeout rule for 120s:
on-timeout: loginctl lock-session
on-resume:
[LOG]   | got iface: wp_tearing_control_manager_v1 v1
[LOG]   | got iface: wp_fractional_scale_manager_v1 v1
[LOG]   | got iface: zxdg_output_manager_v1 v3
[LOG]   | got iface: wp_cursor_shape_manager_v1 v1
[LOG]   | got iface: zwp_idle_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_relative_pointer_manager_v1 v1
[LOG]   | got iface: zxdg_decoration_manager_v1 v1
[LOG]   | got iface: wp_alpha_modifier_v1 v1
[LOG]   | got iface: zwlr_gamma_control_manager_v1 v1
[LOG]   | got iface: ext_foreign_toplevel_list_v1 v1
[LOG]   | got iface: zwp_pointer_gestures_v1 v3
[LOG]   | got iface: zwlr_foreign_toplevel_manager_v1 v3
[LOG]   | got iface: zwp_keyboard_shortcuts_inhibit_manager_v1 v1
[LOG]   | got iface: zwp_text_input_manager_v3 v1
[LOG]   | got iface: zwp_pointer_constraints_v1 v1
[LOG]   | got iface: zwlr_output_power_manager_v1 v1
[LOG]   | got iface: xdg_activation_v1 v1
[LOG]   | got iface: ext_idle_notifier_v1 v1
[LOG]    > Bound to ext_idle_notifier_v1 v1
[LOG]   | got iface: ext_session_lock_manager_v1 v1
[LOG]   | got iface: zwp_input_method_manager_v2 v1
[LOG]   | got iface: zwp_virtual_keyboard_manager_v1 v1
[LOG]   | got iface: zwlr_virtual_pointer_manager_v1 v2
[LOG]   | got iface: zwlr_output_manager_v1 v4
[LOG]   | got iface: hyprland_toplevel_export_manager_v1 v2
[LOG]   | got iface: zwp_text_input_manager_v1 v1
[LOG]   | got iface: hyprland_global_shortcuts_manager_v1 v1
[LOG]   | got iface: zwlr_screencopy_manager_v1 v3
[LOG]   | got iface: wl_shm v1
[LOG]   | got iface: wl_drm v2
[LOG]   | got iface: zwp_linux_dmabuf_v1 v4
[LOG]   | got iface: wl_compositor v6
[LOG]   | got iface: wl_subcompositor v1
[LOG]   | got iface: wl_data_device_manager v3
[LOG]   | got iface: zwlr_export_dmabuf_manager_v1 v1
[LOG]   | got iface: zwlr_data_control_manager_v1 v2
[LOG]   | got iface: zwp_primary_selection_device_manager_v1 v1
[LOG]   | got iface: wp_viewporter v1
[LOG]   | got iface: xdg_wm_base v6
[LOG]   | got iface: wl_seat v9
[LOG]    > Bound to wl_seat v9
[LOG]   | got iface: wp_presentation v1
[LOG]   | got iface: zwlr_layer_shell_v1 v4
[LOG]   | got iface: org_kde_kwin_server_decoration_manager v1
[LOG]   | got iface: wp_drm_lease_device_v1 v1
[LOG]   | got iface: zwp_tablet_manager_v2 v1
[LOG]   | got iface: zxdg_exporter_v1 v1
[LOG]   | got iface: zxdg_importer_v1 v1
[LOG]   | got iface: zxdg_exporter_v2 v1
[LOG]   | got iface: zxdg_importer_v2 v1
[LOG]   | got iface: wp_single_pixel_buffer_manager_v1 v1
[LOG]   | got iface: xwayland_shell_v1 v1
[LOG]   | got iface: wl_output v4
[LOG] found 2 rules
[LOG] wayland done, registering dbus
[LOG] Using dbus path /org/freedesktop/ConsoleKit/Session2
zsh: illegal hardware instruction (core dumped)  hypridle
hypridle.core

please don't post core files, they are useless.
Anyways, illegal hardware instruction errors are impossible to be produced by C++ code, which likely means your hyprlock is compiled wrong. Please pull git and compile manually.

LLDB'ed a debug build and figured out it's the same issue as https://github.com/hyprwm/xdg-desktop-portal-hyprland/issues/207
FreeBSD's libc is catching a calling to mutex::unlock() on a mutex that's not locked.  Apparently it's UDB when trying to do that.  I can confirm that calling mutex::try_lock() before the while loop at src/core/Hypridle.cpp:134 solves the issue.

ok that is stupid, it should abort...


Faild in multi-class classification

In the tensorflow website, You mentioned that "RandomForestModel" support multi-class classification.  In composing ANN with RFM we have problem,  it's not possible to do this.
Nobody could not have use it as multi-class until now. And then please correct your opinion.

Hi,
Tensorflow Decision Forests supports multi-class classification. Our Beginner tutorial shows how to use TF-DF with Random Forests for multi-class classification on the penguins dataset.

Btw, just in case, notice that multi-class != multi-label (wikipedia).

Hi,
Tensorflow Decision Forests supports multi-class classification. Our Beginner tutorial shows how to use TF-DF with Random Forests for multi-class classification on the penguins dataset.

If you consider my comment, I wrote in composing ANN and decision forest. This not possible to do this.

Btw, just in case, notice that multi-class != multi-label (wikipedia).

If you consider my comment, I wrote in composing ANN and decision forest. This not possible to do this.


Nested plucks do not work

data:
[
  {
    "key1": "value1",
    "key2": {
        "key3": "value2"
    }
  }
]

query
pluck("key1", {"key2": "key3"}})

This should return all the same data as displayed above, but it does not

Look into https://glom.readthedocs.io/en/latest/ to potentially reduce the complexity of mocking this


Return value from ParserProxy.visit method

Currently, ParserProxy.visit method doesn't return any value.
  VALUE visit(VisitorProxy* visitor) {
    visitor -> visit(this -> parser -> syntax());
    ...
    return Qnil;
  }

My Ruby visitor returns objects from visit_* methods. How can I propagate them through ParserProxy.visit? I tried to update the method code in various ways and recompile, but haven't succeeded yet. Usually I received RuntimeError: std::bad_cast or ArgumentError: Unable to convert antlrcpp::Any* in runtime.

Closing, I found a solution and I sent a PR #3 instead.

Hey @zakjan thanks for reaching out. Apologies for not responding sooner. Also, thanks for the PRs! I'm looking at them now.


Revise endpoint and channel constructs to facilitate proxyfication

Make the Message construct a public one.
Channels have input/output streams work at the Message level (higher level) as opposed to Payload (lower level). This should facilitate implementation of proxies.
Change the endpoints so that their run methods carry the original InputMessage in addition to the decoded payload. This allows for implementors of endpoints to just forward the Message to some other JsonRPC component (like, a client talking to a proxied server).
Allow for endpoints to be declared with glob patterns.

This is both binary and source breaking, and will require a bump in minor.

This is both binary and source breaking, and will require a bump in minor.

Let the saints of early semver forever bless the ground on which 0.0.6 will grow and break everything.


release: release for 1.5.2

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
release: release for 1.5.2
Ⅱ. Does this pull request fix one issue?

#4762
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #4771 (46d75f3) into master (ede95ef) will decrease coverage by 0.05%.
The diff coverage is 42.72%.

@@             Coverage Diff              @@
##             master    #4771      +/-   ##
============================================
- Coverage     49.07%   49.02%   -0.06%
- Complexity     4057     4082      +25
============================================
  Files           733      736       +3
  Lines         25562    25680     +118
  Branches       3154     3171      +17
============================================
+ Hits          12545    12590      +45
- Misses        11684    11749      +65
- Partials       1333     1341       +8

Impacted Files
Coverage Δ

...c/main/java/io/seata/common/ConfigurationKeys.java
0.00% <ø> (ø)

...n/src/main/java/io/seata/common/DefaultValues.java
0.00% <ø> (ø)

.../src/main/java/io/seata/core/protocol/Version.java
8.33% <ø> (ø)

...ta/core/rpc/netty/AbstractNettyRemotingClient.java
18.53% <0.00%> (-0.10%)
:arrow_down:

...ava/io/seata/core/rpc/netty/NettyServerConfig.java
0.00% <0.00%> (ø)

...pc/processor/client/ClientOnResponseProcessor.java
18.75% <0.00%> (+1.10%)
:arrow_up:

...rpc/processor/server/ServerOnRequestProcessor.java
0.00% <0.00%> (ø)

...b/sql/distributed/lock/BaseDistributedLockSql.java
0.00% <ø> (ø)

...eata/discovery/loadbalance/LoadBalanceFactory.java
66.66% <ø> (ø)

.../autoconfigure/SeataTCCFenceAutoConfiguration.java
0.00% <ø> (ø)

... and 32 more


Re-generate all services via ack-generate by running make build-controller

Description of changes:
Regenerate all services via ack-generate by running make build-controller SERVICE=<service>
This action:

Generate kustomization.yaml files under config/crd using as resource all crds under config/crd/bases
Uncomment crds in all config/default/kustomization.yaml files
Update generated code by ack-generate, some of these changes come from https://github.com/aws/aws-controllers-k8s/pull/412

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Open question: Do you think it might be helpful to follow up with a github action that verifies if the generated code was checked-in?

Open question: Do you think it might be helpful to follow up with a github action that verifies if the generated code was checked-in?

Not at the moment, no. Perhaps later on.

@ricardo-larosa I'd like to regenerate the service controllers Go code in a separate commit and then leave this commit to just the generation of the CRD stuff. Give me a couple hours and then you can rebase.

LOL, never mind, I forgot I already merged the crd template fix :)

Cool Thanks!


M3-850 Linode Disk Resize / Refactor

Purpose
Users were unable to resize disks on their Linode.
During this effort several anti-patterns were realized that begged for a refactoring. Using the newly implemented Context API made this very straightforward but required many line changes.
Context API

add: Updater function to Volumes context object.

Linode Detail

add: Event listener for disk_resize event which get's updated disks. We don't have diskId in the event to get only that disk, so we have to get them all. 👎

VolumesLanding

fixed: An issue where volume actions would never complete.

Disk Drawer

add: Resize a Linode disk.
change: Use specific change handlers, no more generic onChange.
change: Used getDerivedStateFromProps to generate hasErrorFor function.
change: Used our Button abstraction instead of MaterialUI.

LinodeDiskActionMenu

change: Changed action "edit" to "rename".

LinodeConfigsPanel

change: Renamed LinodeAdvancedConfigurationsPanel.
change: Extracted disk operations to LinodeDisk component.
change: Extracted config operations to LinodeConfig component.

LInodeDisk

change: Used context API to update Disks on addition, updating, detachment.

Coverage decreased (-0.3%) to 26.347% when pulling 3dcf6edf992f4d217a27feb4bb3314b479e741f7 on rjstires:M3-850 into 357b61761507899c879bfc9e9917fd6b69414aaf on linode:develop.

After resizing a linode from 20GB to 80GB , the available disk space shows as 81920mb. However, attempting to resize to 81920 fails. It seems that the amount of total disk space is 81920 - 512 (swap).
We should update the amount available to reflect this.

If I'm reading the API docs right, you can't actually include a new size in a request to update; resize is a separate endpoint. If so, we should also disable the size field in the Rename menu. It actually looks like you can change the filesystem in an update, but that might be a documentation error. (Should still be disabled here I think as long as the drawer is called 'rename'.)
Disk resizes also fail for mysterious (to the user) reasons. We could maybe:
a) include this in our requests to the API team for the events system, so we get reasons for errors
b) do client-side checking where possible (a request to resize a disk that is in use will succeed but the resize will fail, is it possible to check this from our end?)
c) send toasts on failure


Generate generic entry point

Could be useful if the plugin allow us to create a generic entry point

Solved in pull request #65


Improve POM files

Bump version to avoid contributing package download number.
Use spring-boot-starter-parent as parent for samples.

@yugnez @xscript @aovechki

@ZhijunZhao,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #113 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #113   +/-   ##
======================================
  Coverage    58.6%   58.6%
======================================
  Files          52      52
  Lines        1104    1104
  Branches      123     123
======================================
  Hits          647     647
  Misses        392     392
  Partials       65      65

Continue to review full report at Codecov.

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


[FAI-7803] Workday Destination

Description
Preparing workday destination converter.
Type of change

[ ] Bug fix
[X] New feature
[ ] Breaking change

The random number generators are in a file that creates a random test input. They aren't a concern because they aren't called during run time, only independently if one were to want to add a test

Link to design spec:
https://docs.google.com/document/d/1765DPVG5nOv-TLBcx2FJud1MVxW6323ji6sNo0p3rPs/edit

Thank you @willmarks for the review. I'll do another pass but it's looking pretty good 🎉
Thank you for the tests @omreego !


Q: Alias for Symfony bundle paths and resolution

Is there a way to have some kind of mapping or resolution strategy in melody for looking up stuff like DefaultBundle:component:home.html.twig or something for include and extend and so on?

Could be done as a Melody plugin or - likely better - as a Webpack plugin/resolver.
Melody, by default, just uses the paths as they are without employing any custom logic in it. Keep in mind that it'd only work for static paths, not for those which would contain variables (DefaultBundle:component:#{fileName}.html.twig)


Using react-to-pdf with typescript

This package doesn't have a types file to use with typ[escript

Any news on this?

Has anyone managed to solve this problem?

I solve this issue. Firstly, Create a folder typings then create file index.d.ts inside the folder. then declare module declare module 'react-to-pdf'. Then add "typeRoots": [         "./typings",         "./node_modules/@types"     ] inside tsconfig.json.
Folder Structure:

typings

index.d.ts
`declare module 'react-to-pdf'

rror-ts7016-could-not-find-a-declaration-file-for-module-xyz-has-an-any-type-ecab588800a8)
[Reference](https://pjausovec.medium.com/how-to-fix-e

I solve this issue. Firstly, Create a folder typings then create file index.d.ts inside the folder. then declare module declare module 'react-to-pdf'. Then add "typeRoots": [ "./typings", "./node_modules/@types" ] inside tsconfig.json.
Folder Structure:

typings

index.d.ts
`declare module 'react-to-pdf'

 rror-ts7016-could-not-find-a-declaration-file-for-module-xyz-has-an-any-type-ecab588800a8)
[Reference](https://pjausovec.medium.com/how-to-fix-e

This doesn't work for me

It works now but there's still error in the code as {toPdf} still has an implicit type of any with red underline

Try declaring a module like this:
`declare module 'react-to-pdf' {
import { RefObject } from 'react';
function ReactToPdf(
    targetRef?: RefObject,
    filename?: string,
    x?: number,
    y?: number,
    options?: object,
    onComplete?: function,
    scale: number
): ReactNode<{ toPdf: () => void; targetRef: MutableRefObject }>;

export = ReactToPdf;

}`

v1 has been just released and it's built with TS, please  see README for more details/examples.


Atom.Object.defineProperty.get is deprecated.

atom.workspaceView is no longer available.
In most cases you will not need the view. See the Workspace docs for
alternatives: https://atom.io/docs/api/latest/Workspace.
If you do need the view, please use atom.views.getView(atom.workspace),
which returns an HTMLElement.
Atom.Object.defineProperty.get (/Applications/Atom.app/Contents/Resources/app/src/atom.js:54:11)
Object.activate (/Users/jreynolds/.atom/packages/pretty-json/index.coffee:32:9)

Are you sure you are running an up-to-date version of atom and this package?

I am now.  I had to uninstall/reinstall the package and it got 0.3.2.  Thank you!

No problem, thanks for checking.


deps: bump sdk to v0.47.2 and cometbft to v0.37.1

Description

closes: #XXXX
Commit Message / Changelog Entry
deps: bump sdk to v0.47.2 and cometbft to v0.37.1

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Any need to backport this to a patch release on v7.0.x, or is v7.1.x the only target?


add support for k8s endpoint slice

This PR adds support for k8s endpoint slice. Since kubernetes does not enable this feature by default, Cilium will detect if the feature is enabled by checking if there is at least a single endpoint slice available in the whole cluster. The user can always fallback to v1.Endpoints by setting the option enable-k8s-endpoint-slice-auto-detection to false which makes Cilium to only watch for v1.Endpoint k8s events.
Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

test-me-please

Coverage decreased (-0.2%) to 45.78% when pulling 17d46be1ed787842633d03e6d657d735d7a67437 on pr/add-endpoint-slice into 27ca26011c1d6674a30df7fb11f9932bb223fcd3 on master.

Docs look fine, is there no way to autodetect whether k8s has enabled this and autoenable it from Cilium?

Unfortunately no because 1) the user needs to enable the feature in k8s as well and 2) if the user disables the feature, after enabling it, the old endpoint slices won't be garbage collected which means that Cilium won't know that the user stopped using the option in kubernetes.

test-me-please

test-me-please

test-me-please


Server-side Logic

There are several issues related to how we're handling sign-up.
Also, the mobile number isn't really required we should remove it from user Model.
Issues
The current setup appears to incorrectly reference files in the local working directory instead of handling uploads directly to Cloudinary. Attached ss for clarity

Please assign this issue to me.

can you also add verification mail feature while sign up.
Why do you think phone number is not required?

Verification mail sounds good, we will need a sender mail for that? I'm not sure how to implement mails might look into it later..
As for the phone no part, I believe the reason we're using the mail is to authenticate and contact the user by sending mails. It's very rare that we receive sms notifs from websites. There are privacy concerns as well.

ok greate then update user schema too,
If you can work on verification part then fine. Otherwise I will do or assign to someone else.

@manjit-hub
I would like to work on verification mail.


why still image detection score is not between [0,1]?

dear @myfavouritekk, Still image detection outputs are bounding box and probability corresponding to different classes, and thus the scores should between [0,1]. Why scores in your example data is not?
Another question
Bounding boxes output from Faster RCNN or other Detecion Program shouldn't be as many as your example data. Did you keep most of the output bounding boxes with a low score threshold? Or the bounding boxes just come from RPN output?
Thank you!

Hi @wwwanghao, the still image detection results were generated by RCNN framework with SVM classifier for each class. SVM scores are not bounded within [0,1]. In addition, we have an ensemble of 5 models, and the scores are the sum of all the models. The details of the classifier are in the tech report.
For the number of boxes, we used the selective search and edge boxes to generate proposals and trained a "weak" classifier to eliminate low-confidence proposals. You can also find some details in the tech report.


Added setup.py

Added setup.py file, and updated readme to reflect new install steps via pip. Addresses issue #6

@Innixma not sure if I handled the install_requires field in the best way, but i've gotten a setup.py file that works (tested clean install on different machine as well)


Fix nightly build of 1.4.x

Description
Nightly build of 1.4.x keeps failing - http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/NightlyTestsForBinaries/detail/v1.4.x/114/pipeline/
In 1.5.0 there is a major rewrite of this tutorial, but since 1.4.0 is already published, I made the smallest change possible to fix the failing build.
I checked the build after this change on my EC2 instance - all good.

Thank you for the fix @Ishitori Could you have a look at the CI failures?
@mxnet-label-bot add [Build, pr-awaiting-review]

@Ishitori The CI still seems to fail. :/
I've noticed it failed on another PR on v1.4.x branch as well : https://github.com/apache/incubator-mxnet/pull/14586

@piyushghai, it does fail, and it seems there is something wrong with TensorRT. We are trying to find someone experienced with TensorRT to figure it out.


create witness if needed

Description
Summary of changes
Create witness if needed when dividing expressions
Checklist

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[ ] I have linked this PR to the issue(s) that it resolves.
[ ] I have reviewed the changes on GitHub, line by line.
[ ] I have ensured all changes are covered in the description.

Can you add more context to this PR and or link it to an issue?

The user did not create an issue and I don't think we need one nor a regression as the fix is a no-brainer

Looks good.
I agree on the not creating an issue/regression test argument. It would be nice if this were included in a test but going forward if we have a dedicated regression test for every issue we will have a test suite that is too large to practically test. Then we'd need to tackle the questions of only testing subsets of the test suite, which subset should be chosen, etc.

I was not suggesting a regression test, my question was about more context.


Refactor time entries report controller

Notion card
https://www.notion.so/saeloun/Refactor-controllers-51ad6bf864c448308db7a9c6f65241ad
Summary
Moved fatty code to a service
Preview

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactor (non-breaking and retains same functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to
not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] I have manually tested all workflows
[x] I have performed a self-review of my own code
[ ] I have annotated changes in the PR that are relevant to the reviewer
[ ] I have added automated tests for my code

@apoorv1316 Please test this on staging

Time entry reports
Download reports


Minor edits to tutorial Launching

edits some typos, call outs and durations

merging this pr, for Training review project v1.1


Starting smart server with -format option can cause standby server reformating the database

Solved by #1488.


Add UX notice that reads "updated xx seconds ago"

Use moment.js.

Issue moved to the rolltime-server repository.


add test case to cover #4890

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] others

Which issues of this PR fixes ：
close #4890

[FE PR Coverage check]
:heart_eyes: pass : 0 / 0 (0%)


Workflow to recreate safekeeper/add a new one to already running compute

In case we have a failure (disk failure or other outage) led to the loss of the safekeeper it is desirable to have a way to recreate safekeeper.

Duplicated of #985


Panicking in spawned Rayon tasks will abort the process by default.

C.f. https://docs.rs/rayon/latest/rayon/struct.ThreadPoolBuilder.html#method.panic_handler, i.e. spawn is fire-and-forget and hence cannot propagate the panics anywhere sensible. This leads to test flakiness like https://github.com/quickwit-oss/tantivy/actions/runs/9203311046/job/25314621091.

@adamreichold thank you for the information! Maybe we should catch panics actually.

@adamreichold thank you for the information! Maybe we should catch panics actually.

We could just install a panic handler which logs and moves on, but that could leave the system in an inconsistent state. Debugging any issues resulting from that downstream might not be fun.
Alternatively, we could wrap all closure in catch_unwind and forward the panic payloads as errors, either as thread::Result<crate::Result<Vec<R>> or by adding a panic payload variant to tantivy::Result.

I was more thinking of catch_unwind

I am merging this. Let me know if you want to work on the catch_unwind stuff.

This wasn't merged


Introducing pe-terminal which replaces relay-term

pe-terminal is written in Golang. Relay-term was the only node.js-based project.
Hence, this eradicates node.js from our Yocto image.

Since the paths are now somewhat cleanly in one file, could we just change them to be /pelion instead of /wigwag @whygoyal ?

Since the paths are now somewhat cleanly in one file, could we just change them to be /pelion instead of /wigwag @whygoyal ?

My personal preference would be to use a more generic name, rather than that of an organisation.

My personal preference would be to use a more generic name, rather than that of an organisation.

Even more correct, yes. It should be just edge.

@JanneKiiskila as of now we do not have a folder /pelion on the box. In the context of pe-terminal, the actual binary is at /wigwag/system/bin/pe-terminal and the config file at /wigwag/system/etc/pe-terminal/config.json. The both locations are configurable. I can change it but not sure what all will break. Let me raise another PR to bring in that change for all the packages.

I have manually tested this and its working as expected. This is ready to be merged.

We need to be mindful of the folders that are OK from OSTree point of view.


[MINOR][DOCS] Pyspark getActiveSession docstring

What changes were proposed in this pull request?
Minor fix so that the documentation of getActiveSession is fixed.
The sample code snippet doesn't come up formatted rightly, added spacing for this to be fixed.
Also added return to docs.
Why are the changes needed?
The sample code is getting mixed up as description in the docs.
Current Doc Link

Does this PR introduce any user-facing change?
Yes, documentation of getActiveSession is fixed.
And added description about return.
How was this patch tested?
Adding a spacing between description and code seems to fix the issue.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

ok to test

Test build #124891 has started for PR 28978 at commit 5f4e7cd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/29497/
Test PASSed.

Test build #124891 has finished for PR 28978 at commit 5f4e7cd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/124891/
Test PASSed.

Merged to master, branch-3.0 and branch-2.4.