Backreferences in rewrites?

I'm trying to rewrite everything under /admin, e.g.

/admin => /
/admin/index.html => /index.html
/admin/static/style.css => /static/style.css
/admin/static/main.js => /static/main.js

...and so on.
Can this be accomplished using rewrites in serve.json without specificying every possible path?
I tried the following but no luck:
{
  "rewrites": [{
    "source": "/admin/(.*)",
    "destination": "/$1"
  }]
}

Thanks, I had seen that, but I assumed it would work as in Express routes in that it matches only one path component (i.e. would match /foo/:bar matches /foo/index.js but not /foo/static/index.js) and indeed it looks like that's the case:
arrow:/tmp/test$ ls -R public/
public/:
index.js  static

public/static:
style.css
arrow:/tmp/test$ cat serve.json
{
  "public": "public",
  "rewrites": [
    {
      "source": "/admin/:rest", "destination": "/:rest"
    }
  ]
}

arrow:/tmp/test$ serve &
[1] 17823
INFO: Discovered configuration in `serve.json`
INFO: Accepting connections at http://localhost:3000

arrow:/tmp/test$ curl http://localhost:3000/index.js
HTTP/1.1 200 OK
[...]
// this is /index.js

arrow:/tmp/test$ curl http://localhost:3000/admin/index.js
HTTP/1.1 200 OK
[...]
// this is /index.js

arrow:/tmp/test$ curl http://localhost:3000/static/style.css
HTTP/1.1 200 OK
[...]
/* this is /static/style.css */

arrow:/tmp/test$ curl http://localhost:3000/admin/static/style.css
HTTP/1.1 404 Not Found
[...]
<!DOCTYPE html><head> <meta name="viewport" content="width=device-width, [...]

Yes, we implemented it a little differently because it was important for us to allow rewriting the entire ending of a path to a different one. If you want to restrict it, please create more rewrites afterwards that point the other paths to the right one.
For every request, all rewrites that match the source are applied - so that should be easy. 👍

this does not work if you are running serve -s as for some reason before the rewrites can happen serve-handler would actually end up serving index.html instead
user:~/environment/serve $ ls -lR
.:
total 8
drwxrwxr-x 3 ec2-user ec2-user 4096 Sep  6 15:52 public
-rw-rw-r-- 1 ec2-user ec2-user  114 Sep  6 18:23 serve.json

./public:
total 12
-rw-rw-r-- 1 ec2-user ec2-user  182 Sep  6 16:57 index.html
-rw-rw-r-- 1 ec2-user ec2-user   23 Sep  6 15:44 index.js
drwxrwxr-x 2 ec2-user ec2-user 4096 Sep  6 15:44 static

./public/static:
total 4
-rw-rw-r-- 1 ec2-user ec2-user 24 Sep  6 18:20 style.css

serve -s
curl localhost:8080/admin/index.js
# Content of index.html instead

path-to-regexp supports repeated segments
const toPathRepeated = compile("/:segment+");

toPathRepeated({ segment: "foo" }); //=> "/foo"
toPathRepeated({ segment: ["a", "b", "c"] }); //=> "/a/b/c"

I think that should be the solution 🤔

@AndyOGo's solution looks like it wasn't tested, because with it I still get the same issue.
Here is my abridged serve.json:
{
  ...
  "rewrites": [
    {
      "source": "/some/path/:path+",
      "destination": "/:path"
    }
  ]
}

And here is my abridged curl output after running npx serve:
user@Machine project-root % curl -I http://localhost:3000/favicon.png
HTTP/1.1 200 OK
...

user@Machine project-root % curl -I http://localhost:3000/some/path/favicon.png
HTTP/1.1 200 OK
...

user@Machine project-root % curl -I http://localhost:3000/assets/vendor.822f96e6.js
HTTP/1.1 200 OK
...

user@Machine project-root % curl -I http://localhost:3000/some/path/assets/vendor.822f96e6.js
HTTP/1.1 404 Not Found
...

So is there no way to do this?


Error if running makeimage.sh

I have the lastest Git-revision: 96dda7e of FFmpeg-Builds.
I am using the bash within Windows11, but I have the same error within
an ubuntu-Virtual-Box environment.
If I use the command:
./makeimage.sh win64 gpl-shared
or
./makeimage.sh win64 gpl-shared 6.1
the folling errors occur:
=> ERROR [layer-50-vulkan  6/13] RUN --mount=src=scripts.d/50-vulkan/50-shaderc.sh,dst=/stage.sh run_stage /stag  4.0s
=> CANCELED [layer-50-librist 6/6] RUN --mount=src=scripts.d/50-librist/50-librist.sh,dst=/stage.sh run_stage /s  4.1s
=> CANCELED [layer-50-aribb24 6/6] RUN --mount=src=scripts.d/50-aribb24/50-libaribb24.sh,dst=/stage.sh --mount=s  4.1s
=> [layer-50-libjxl 4/9] ADD .cache/downloads/45-lcms2_7aaa094067389721abf0034db974d1d3db2dbcf8af5d124889b19465c  1.9s
=> CANCELED [layer-50-x265 3/3] RUN --mount=src=scripts.d/50-x265.sh,dst=/stage.sh run_stage /stage.sh            0.8s
I use FFMpeg in my Homepage with free Software for everyone:
https://www.somby.de
best regards
Reiner Sombrowsky

That's not the actual error. Please scroll up and find what's actually failing.

If I scroll up I found:
=> CANCELED [layer-50-x265 3/3] RUN --mount=src=scripts.d/50-x265.sh,dst=/stage.sh run_stage /stage.sh           14.7s

Yes, all other stages got cancelled because layer-50-vulkan 6/13 had an error.
Please find the actual error.

Sorry, but I am using only your scripts to
make the actual versions of ffmpeg for my own
software developments for my homepage.
I cannot looking for the actual error.

Then I'm unfortunately not able to help you.
The build works fine for me, and all you shared so far boils down to "There was an error".
Really, the actual error is in the log, all you need to do is scroll up a bit until you get to it.

I have checked all with 32bit with
makeimage.sh and build.sh Script and I
cannot build it

So is it win64 or win32 that's failing?
win64 builds fine for me, I didn't test win32 in a bit.
But really, without the actual error, there is nothing I can do to help you.

Sorry if I disturb again. Today I tried to run the script again and I found the following error lines at the end:
.....
3.088 -- Configuring incomplete, errors occurred!
3.207 CMake Error at third_party/CMakeLists.txt:80 (message):
3.207   SPIRV-Tools was not found - required for compilation
.....
ERROR: failed to solve: process "/bin/sh -c run_stage /stage.sh" did not complete successfully: exit code: 1

rm -f /tmp/tmp.MyCWkVYXfH.toml

After I have installed SPIRV-Tools the same error occurs.


Difficulty in getting any results in any experiment, even with the provided sample run

I have recently downloaded and starting using alphapept v 0.3.29 and have been having difficulty trying to get it to work. I set it up on my personal Macbook (OSX 10.14 Mojave and 2.5 GHz Intel Core i5) using the Python set up instructions from the README.MD file. All the experiments that I start do complete, but they never have any results to display. So, I decided to use the sample run from the Start page (using the irtfusion.fasta and 20200911_EXPL0_IgPa_6min_DDA_2.raw), but this leads to the same result, where no results can be found. Is there some step that I am missing in order to get this program to work? I have looked through the logs of the different tests I have conducted with the sample run, and I think the problem may lie with the Python module clr, since I have identified two errors that mention it: File conversion of file 20200911_EXPL0_IgPa_6min_DDA_2.raw failed. Exception No module named 'clr'. I have tried to download it using python -m pip install clr, but this error instead became: File conversion of file 20200911_EXPL0_IgPa_6min_DDA_2.raw failed. Exception module 'clr' has no attribute 'AddReference'.
Moreover, after the clr related errors, I have found the error in all logs: Feature finding of file 20200911_EXPL0_IgPa_6min_DDA_2.raw failed. Exception "Group Raw/MS1_scans does not exist in <HDF_File 20200911_EXPL0_IgPa_6min_DDA_2.ms_data.hdf>. Error 'Unable to open object (component not found)'". I suspect this is a result of the clr module not working, but I'm uncertain. Any help would be most appreciated.
To Reproduce
Steps to reproduce the behavior:

Run alphapept gui in Terminal.
Get the .fasta and .raw file from the Start tab of GUI, place in a folder of choice.
Go to New Experiment tab and set up New Experiment. Load folder of choice with the sample run, and then run the experiment with default workflow and settings (i.e. don't modify either part).
After a few seconds, the experiment completes, and no results are found.

Screenshots

Hi @fierytortilla , it is because the pythonnet package is not correctly installed. Please follow https://github.com/MannLabs/alphapept#mac to install pythonnet. If it does not work, there may be two reasons.

mono is not correctly installed, we don't know how to fix it then.
Type where mono (you may get mono path /Library/Frameworks/Mono.framework/Versions/Current/Commands/mono), then follow item 3 in https://github.com/MannLabs/alphapept#mac to config mono pkgconfig path (mono pkgconfig path should be /Library/Frameworks/Mono.framework/Versions/Current/lib/pkgconfig), and then 'pip install pythonnet'


Show a heart icon in the menu bar

I found your menu bar controller for Spotify, because the more famous alternatives all rely on the Apple Script API, which is limited and broken. I'm talking about the ability to read and set if the current track is liked or not.
In particular, reading and seeing the like status of the current played track is what I am after. The Apple Script API call is broken for 4(!) years without any noticeable improvement: https://community.spotify.com/t5/Desktop-Mac/Applescript-quot-starred-quot-property-returns-Error/td-p/4853526
So, your project deserves more attention, because it actually is a menu bar agent that leverages the much richer Spotify API that requires authentication.
The only missing piece for me is that I can't see the status (liked or not) right in the menu bar. It would be nice to have this feature.
I think it would be cute to replace the note within the cycle with a little heart when the current track is played. Something like this:

I love this idea!

@stefan786 Do you like to take over #6? All the necessary logic is there to swap the icon already, I just never bothered to find a proper icon and place it in there, accordingly. I think my PR also needs to updated for the recent version. My local build is ancient and outdated.


Reimburse expenses - The milage rate automatically changes to the initial value

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Issue found when executing PR https://github.com/Expensify/App/pull/10692
Action Performed:

Go to URL https://staging.new.expensify.com/
and login
Create a New Workspace
Go to Settings -> Workspace -> Reimburse expenses
Edit the milage rate

Expected Result:
The mileage rate should change after the changes
Actual Result:
The milage rate automatically changes to the initial value
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/188492766-8226fbec-b909-481a-8e66-77811dfd9385.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Ah poop, the mistake is here where we forgot to think about migration to support older data. cc @amyevans.
We could push up a fix to support the older data but I think maybe it's better to push forward and CP these two,
https://github.com/Expensify/App/pull/10692, and
https://github.com/Expensify/App/pull/10557 (for this one, the related web PRs are already live and the PR is in final review so I think it should be simple enough to CP). cc @neil-marcellini

cc @jasperhuangg as well since looks like there's more conflicts when trying to merge main into your App branch in https://github.com/Expensify/App/pull/10557

@amyevans do you wanna take this blocker over from Chirag, since you seem to know how to fix it?

Bumping @amyevans. But I can also take over this as I'll be working on deployers until we release today.

We'll need to cherry pick https://github.com/Expensify/App/pull/10557, but it needs to be updated with main first and then I can review again before we ship it. Does that plan sound okay? Going to comment in the PR

but it needs to be updated with main first

Just wanted to check that this is necessary. As if we mergemain wouldn't your PR include a bunch of unwanted commits when we CP?

We'll get back to this tomorrow once the provisioning build has been confirmed to be working.

Now that the CP workflow is fixed, I retried the CP and it worked 🎉

Testing should begin shortly after the build is made available for Applause.

The original blocker is resolved. We might want to make further improvements as the extra decimal places are being added quite quickly, but I'm marking this off:
https://user-images.githubusercontent.com/10736861/188961057-60be5fd8-c346-48be-a972-220aa6d6c370.mov


Views

Set up views with handlebars.js
Set up views for homepage with top rated characters and upvote/save icons
Set up views for dashboard
Set up views for individual character sheet

Addition:
Set up views for generating new character


Fix omniorb dependency for cpptango on macOS

Checklist

[X] Used a static YAML file for the patch if possible (instructions).
[X] Only wrote code directly into generate_patch_json.py if absolutely necessary.
[X] Ran pre-commit run -a and ensured all files pass the linting checks.
[X] Ran python show_diff.py and posted the output as part of the PR.
[X] Modifications won't affect packages built in the future.

On macOS, cpptango 9.5.0 is explicitely linked against omniorb 4.3.1. Releasing 4.3.2 currently broke 9.5 installation as run_export is set to x.x. It must be tighten to x.x.x.
$ otool -L libtango.9.5.0.dylib
libtango.9.5.0.dylib:
	@rpath/libtango.95.dylib (compatibility version 95.0.0, current version 9.5.0)
	@rpath/libomniORB4.3.1.dylib (compatibility version 0.0.0, current version 0.0.0)

This isn't the case on Linux (and Windows) where 4.3 is used:
$ ldd libtango.so.9.5.0
	linux-vdso.so.1 =>  (0x00007ffe189ae000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007f9beb7e6000)
	libomniORB4.so.3 => /opt/conda/envs/tango/lib/././libomniORB4.so.3 (0x00007f9bec169000)

Not sure why yet.
I will see if it can be fixed in cpptango. Otherwise I might have to change the run_export on omniorb.
For now, current cpptango macOS packages have to be fixed:
$ python show_diff.py --subdirs linux-64 osx-64 osx-arm64 --use-cache
================================================================================
================================================================================
osx-arm64
osx-arm64::cpptango-9.5.0-hf3ed413_0.conda
osx-arm64::cpptango-9.5.0-hace4bee_1.conda
-    "omniorb-libs >=4.3.1,<4.4.0a0",
+    "omniorb-libs >=4.3.1,<<IP_ADDRESS>a0",
================================================================================
================================================================================
osx-64
osx-64::cpptango-9.5.0-he842357_0.conda
osx-64::cpptango-9.5.0-haad327e_1.conda
-    "omniorb-libs >=4.3.1,<4.4.0a0",
+    "omniorb-libs >=4.3.1,<<IP_ADDRESS>a0",
================================================================================
================================================================================
linux-64

ok thanks for the analysis. keep us posted on the resoluton.


dropdown menu styling in more section

Is your feature request related to a problem? Please describe.
The problem is that current  dropdown menu in more section is very simple and not with matching with website theme .
Describe the solution you'd like
-We can add animations and hover effect in  buttons or text which changes when clicked and also matched with UI of website

I have made changes in my local machine .Please assign it to me so that I can proceed and make a PR .

@Muskan-creator can you post some screenshots to show what kind of effects you added. The page already became slow so we don't want any kind of unnecessary load which might increate the weight of the page.

https://user-images.githubusercontent.com/59757988/112718479-518bc180-8f19-11eb-80b6-07a651740596.mp4

Can i work on this @harikesh409 ?

Can i work on this @harikesh409 ?

Yes go ahead.


Regarding the SCHEDULED keyword

Is it in your plans to regard the SCHEDULED keyword? At the moment it shows up in the paragraph section where if I understand the library correctly only plain text is expected.
    ▿ Paragraph(text: SCHEDULED: <2016-12-16 Fri .+1d>)
      ▿ lines : 1 element
        - 0 : "SCHEDULED: <2016-12-16 Fri .+1d>"

Yes, it will be implemented soon. Stay tuned.

@mgmart I just implemented Planning syntax. Please try it out. Open new issues if you find any issues. Closing this one.

Planning looks good. Will take some time before I could work further with SwiftOrg.


Custom Backend interface

As a developer, I would like to develop custom backends that are not tied to a Git system.
Is your feature request related to a problem? Please describe.

There are many ways to store data. For netlify cms, the best option so far has been to use git.
The advantage of netlify cms is that the CMS is integrated directly on the site. This makes it easy for content editors.
Unfortunately, the git based cms does not allow you to use dynamic server side rendering.
To make this possible, a backend would always have to be available and return content on request. If the content is in git, this is not so easy. But it would be possible to use your own backend, such as a backend as a service like firebase or appwrite.
Describe the solution you'd like

In order to connect such a custom backend, a plugin system is to be created to which it is possible to implement their own backend.
In addition, an interface is to be created to manage the auth, media and content yourself.
Developers themselves would be responsible for everything else.
Describe alternatives you've considered

Alternatives would be to merge these custom backends into netlify-cms. However, these would not be relevant for most users. Therefore, this is not the best option

@adrian-goe it already exists.
import CMS from 'netlify-cms-app'
CMS.registerBackend('my-backend', MyCustomBackend)

I've successfully written other backends (private) with this solution.
Although the API's aren't documented (think contribution), you can follow the api's of the official backend functions.


RiveAnimationView does not get re-mounted on Android

Description
I've just upgraded to v4 of this library and noticed that the animations we have setup on Android stopped displaying. Upon closer inspection in the example app you have I was able to re-create the issue (see repro link below) by adding the simple animation on the home screen, navigating to one of the examples and then navigating back. It appears as though when this happens onDetachFromWindow() is called unmounting the RiveAnimationView, however, as isAttached never gets set back to false when we navigate back and onAttachToWindow() is called it therefore does not actually end up calling the addSubview(riveAnimationView) and hence the animation is not displayed.
I've tried adding the isAttached = false when it becomes detached, but this leads to a JNI crash somehow which I'm having trouble getting to the bottom of (see screenshot below) - I've left a comment in the code where I've added this. I think that should fix it as it is the only thing preventing it from adding the view back.
Provide a Repro
https://github.com/thomas-coldwell/rive-react-native
Expected behavior
After navigating back to the home screen the animation that was there should be shown and start playing as is the case when you first open the example app (on my fork).
Screenshots
This shows the issue pretty well - the animation should appear in the view after navigating back but it is not visible:
https://user-images.githubusercontent.com/31568400/225994108-e643854c-afda-458c-9d4e-c578892082a9.mov
This is the error I get when I add isAttached = false to the onDetachFromWindow() method:

Device & Versions

Tested on a Pixel 4 emulator and OnePlus 6 physical device
API 30
NPM 8.17.0

Looks like the crash I have when setting isAttached = false is related to this issue - https://github.com/rive-app/rive-android/issues/277

Ah looks like you're already looking into the issue here https://github.com/rive-app/rive-react-native/pull/149#discussion_r1139425404 🙌

Hi @thomas-coldwell ! Sorry for the delay here - there were some recent patches to the rive-android native dependency that clean up some of the render lifecycle/cleanup, and this should be addressed in 4.0.1! can check out this:
https://github.com/rive-app/rive-react-native/pull/169


The arrow icon in the dropdown component is a little bit off in v1

Describe the bug
The arrow icon is not centered relative to the text of the dropdown button/item:

How to reproduce
It's noticeable in the website of version 1

Looks good in version 2 though

Versions
App

Angular: 7.2.*
Node: 10.*
Clarity: 1.* (latest)

Device:

OS: Windows 10 Enterprise
Browser Edge Chromium
Version latest

Additional notes
Add any other notes about the problem here.

It seems that is some kind of regression after this commit 9cd61b266460613af175271d7a8a95f6e20bf313
This change:
// from
$clr-dropdown-caret-xOffset: 0.333rem;

// to
$clr-dropdown-caret-margin-top: 0.375rem;

It seems that 0.3rem looks good ?!

@bdryanovski you can patch this for v1, it doesn't appear to affect v2+.

Sure will do

This was fixed in v1, closing.


Fix changing categories migrations

@CarlosQ96 Some project has categories so when we set another category we get duplicate error, si I thought maybe it's better to use a for..loop and set them one by one see if one of them got error , we are still ok


[Bug]: partition weighting not working when ParmSelect > Nfleets

Describe the bug
I tried to test the partition weighting in the attached model (based on the U.S. west coast 2019 petrale update) where there are 7 fleets and I tried to use D-M parameters to separately weight discard and retained for the first 4 fleets, resulting in 8 parameters.
This setup triggers this message: Warning 2 Fatal Error! length D-M index for fleet: 4 is: 8 but must be an integer <=Nfleet
which is triggered by these here:
https://github.com/nmfs-stock-synthesis/stock-synthesis/blob/2257bceea176ed5c648ac65907370a7c451bd5f1/SS_readdata_330.tpl#L1457-L1458
I tried just commenting out those two lines but then get an error
matrix bound exceeded -- row index too high in imatrix::operator()(int, int)
which I tracked down to this line:
https://github.com/nmfs-stock-synthesis/stock-synthesis/blob/2257bceea176ed5c648ac65907370a7c451bd5f1/SS_readcontrol_330.tpl#L4018
which I think is because Comp_Err_Parmloc is the wrong dimensions due to Comp_Err_ParmCount being too small which goes tracks back to the lines that I commented out.
At that point I was just confused, so am handing this off to @Rick-Methot-NOAA.
2019_petrale_with_partition_weights.zip
To Reproduce
Compile current "main" branch or download the artifact from the most recent call-build-ss3 and run with the files in the attached 2019_petrale_with_partition_weights.zip.
Expected behavior
I expected the model to run with estimates of 8 values for ln(DM_theta) for each combination of discards and retained for the 4 fishing fleets.
Screenshots
No response
Which OS are you seeing the problem on?
No response
Which version of SS3 are you seeing the problem on?
current "main" branch
Additional Context
No response

readdata correctly computes that:
number of D-M definitions needed for length comp data: 8
Then in readcontrol,
imatrix Comp_Err_parmloc(1,Comp_Err_ParmCount,1,2);  //  for each comp_err_index, locate starting parameter in parcount (2) and in Selparm (1).
So, I get correct model performance by simply removing the test against Nfleet.
I also removed the comparable test against 2*Nfleet for age and for sz as they could produce misleading error


Install Jest

As a developer, I'd like to run unit tests using Jest because it has easier mocking, among other things (cc @aliceriot)
Acceptance Criteria:

[ ] Any existing unit tests are run using the Jest test runner instead of Mocha

Looks like this is closed by #42


Remove .btn-block for utilities

Alternate fix for #30376, which closes #30223.

I don't think we need the .d-block class, only .w-100 should be ok. We can even ditch the mb-2, since the inline blocks already have some spacing.

Might not go this route after all actually... requires more utility classes to undo the responsive blocks (assuming block on mobile, not block on say medium). 🤔

Closing per my last comment.


AzCopy enters dead loop when copying a PremiumV2 (Page) blob container to a Gen2 blob container

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.23.0
Which platform are you using? (ex: Windows, Mac, Linux)
Mac
What command did you run?
copy https://premiumv2.blob.core.windows.net/atset/?sasToken https://jasonadls2gen2.blob.core.windows.net/atset/?sasToken --output-type=json --cancel-from-stdin --overwrite=prompt --cap-mbps 300 --s2s-preserve-access-tier=true --check-length=true --include-directory-stub=false --s2s-preserve-blob-tags=false --recursive --log-level=DEBUG
What problem was encountered?
Let's assume the destination container already exists. AzCopy starts the job and seems to enter a dead loop. It will keep emitting the same message in the log file without making any progress or reporting any error. It will keep writing to the log file and result in huge log files if the user doesn't cancel the job manually.
AzCopy should fail the job within finite time otherwise the log file can infinitely accumulate.
This piece of message is repeated in the log file
2024/02/20 17:43:28 ==> REQUEST/RESPONSE (Try=1/8.583µs, OpTime=55.417µs) -- REQUEST ERROR
   GET https://premiumv2.blob.core.windows.net/atset/4g.vhd?comp=pagelist&sasToken
   Accept: application/xml
   User-Agent: Microsoft/Azure/Storage/ azsdk-go-service.Client/v1.2.0 (go1.19.6; darwin)
   X-Ms-Client-Request-Id: 8abc4e72-6377-496b-608f-000a758d7880
   x-ms-version: 2023-08-03
   --------------------------------------------------------------------------------
   ERROR:
Get "https://premiumv2.blob.core.windows.net/atset/4g.vhd?comp=pagelist&sasToken: context canceled

How can we reproduce the problem in the simplest way?
Create a PremiumV2 page blob storage account, upload a page blob and then copy it to a Gn2 storage account.
Have you found a mitigation/solution?
Manually cancel the job.
More context
This is at least a regression from 10.20.1. With 10.20.1, AzCopy uses a different approach and correctly fails the job.

   Content-Length: [335]
   Content-Type: [application/xml]
   Date: [Tue, 20 Feb 2024 17:59:17 GMT]
   Server: [Windows-Azure-Blob/1.0 Microsoft-HTTPAPI/2.0]
   X-Ms-Client-Request-Id: [3e57c370-bfe3-4f62-45c8-9fd03e25643f]
   X-Ms-Error-Code: [FeatureNotYetSupportedForHierarchicalNamespaceAccounts]
   X-Ms-Request-Id: [b2bded35-301e-0051-6d26-64fa07000000]
   X-Ms-Version: [2020-10-02]

2024/02/20 17:59:17 ERR: [P#0-T#0] COPYFAILED: https://premiumv2.blob.core.windows.net/atset/4g.vhd?sasToken : 409 : 409 Specified feature is not yet supported for hierarchical namespace accounts.. When Creating blob. X-Ms-Request-Id: b2bded35-301e-0051-6d26-64fa07000000
``

This issue should be resolved by our next release. (PR fix here)

Closing as this was fixed in dev. This will be released as part of 10.24.0


Month title change not working after date selection

In the Month view, after I select a different day, the swipe to the previous or next month looks like it works, but it stays on the same month title. Before you select a different date, you scan swipe and it changes the title as expected. Is this a known issue already?

@abouttimeruss It's working for me. May I know how do you set your viewTitle when swiping to the previous/next view? I guess it's likely you are setting the viewTitle on a different "scope".

What you said lead me to the problem. Thanks!


question: Lock authentication to specific IP addresses

I am using SAML auth with Entra ID/Azure AD, but I want to prevent anyone not on a specific IP (or multiple IPs/CIDRs) from trying to authenticate or access the webserver.
How can I do that?
Defense in depth.
I used to have something like
@blocked not remote_ip <ip1> <ip2> <ip3>
respond @blocked "Nope" 403

But then I added caddy-security and it stopped working. I can get exact config on Monday.

@Gunni , not sure whether I understand the use case and how it is related to this plugin.

Basically:

check if user in in access list
check saml/redirect user
forward request to reverse proxy

In that order. Again if i need to post config, i can do it on Monday.

In that order. Again if i need to post config, i can do it on Monday.

@Gunni , let's see your config.

Here it is: https://gist.github.com/Gunni/c00b0eab5115eed846e04b66dfa85662


after installing react-native-vector-icons, react-native-linkedin is not working.

Unable to resolve module.
'react-native-linkedin' module does not exist.
Anyone had the same problem?

npm start clear cache fixed the issue


Add geopolitical zone to az account list-locations

Please add the geopolitical zone as an additional field in the output json.
This would allow JMESPATH filtering on all regions in, say, europe.
E.g.
az account list-locations --query "[?geo == 'europe']"

Thank you for raising this feature request. We will look into it.

This feature request has been implemented in https://github.com/Azure/azure-cli/pull/14229
Now you may use metadata.geographyGroup to filter on regions.
>az account list-locations --query "[?metadata.geographyGroup == 'Europe']"
[
  {
    "displayName": "North Europe",
    "id": "/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/locations/northeurope",
    "metadata": {
>>    "geographyGroup": "Europe",
      "latitude": "53.3478",
      "longitude": "-6.2597",
      "pairedRegion": [
        {
          "id": "/subscriptions/0b1f6471-1bf0-4dda-aec3-cb9272f09590/locations/westeurope",
          "name": "westeurope",
          "subscriptionId": null
        }
      ],
      "physicalLocation": "Ireland",
      "regionCategory": "Recommended",
      "regionType": "Physical"
    },
    "name": "northeurope",
    "regionalDisplayName": "(Europe) North Europe",
    "subscriptionId": null
  },


[SPARK-30894][SQL] Make Size's nullable independent from SQL config changes

What changes were proposed in this pull request?
In the PR, I propose to add the legacySizeOfNull  parameter to the Size expression, and pass the value of spark.sql.legacy.sizeOfNull if legacySizeOfNull is not provided on creation of Size.
Why are the changes needed?
This allows to avoid the issue when the configuration change between different phases of planning, and this can silently break a query plan which can lead to crashes or data corruption.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By CollectionExpressionsSuite.

Test build #118734 has started for PR 27658 at commit f768958.

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/23485/
Test PASSed.

See https://github.com/apache/spark/pull/27628#discussion_r381246285

Test build #118734 has finished for PR 27658 at commit f768958.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Size(child: Expression, legacySizeOfNull: Boolean)

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/118734/
Test PASSed.

thanks, merging to master/3.0!

I created a backport PR for this change to branch-2.4
https://github.com/apache/spark/pull/30058
CC: @MaxGekk @dongjoon-hyun

Thank you, @anuragmantri !


[QUESTION] : Is caching enabled by default?

Description
How can I enable caching if not enabled by default when making API requests?

Can you describe more specifically what sort of caching you want to accomplish? (At the very least, do you mean client side or server side?)

Server side specifically thanks where can I find this information thanks

There is no built-in caching mechanism in fastapi -- you are responsible for implementing your own caching.
(There is a dependency injection "cache", which prevents dependency injection functions from being executed multiple times per request, but I don't think that is what you are looking for.)
If you describe your caching use case you may get some recommendations for approaches.

Just a simple GET request, how to cache it thanks

functools.lru_cache is a built in that might be easiest.

^ that's definitely the simplest solution for an in memory cache.
Hope that helps.

Speaking only about the initial request, if by any chance you are hitting the /openapi.json endpoint, that is created and reused (cached) on the first request to it.
Apart from that, there's no other caching.


Improve code quality by separating intention and implementation

Coverage decreased (-1.7%) to 79.92% when pulling 961f69c8e57580f5961a28d1bb2a51bd483ab155 on code-quality into bdc2e7557f88ea38577a7d112a1e1890f9280b33 on master.

Coverage decreased (-1.3%) to 80.241% when pulling 78e5610acbeb64882634a29ca3345cb2d458f31e on code-quality into bdc2e7557f88ea38577a7d112a1e1890f9280b33 on master.

Coverage decreased (-0.6%) to 80.964% when pulling 73b6d2dbe38911ae23af1977ad3590c28218ef9c on code-quality into bdc2e7557f88ea38577a7d112a1e1890f9280b33 on master.

Coverage increased (+0.02%) to 81.606% when pulling 80c2d44774ec18067e6a06bcd7104ca16a7ba982 on code-quality into bdc2e7557f88ea38577a7d112a1e1890f9280b33 on master.

Coverage increased (+1.2%) to 82.745% when pulling 27dd51177f716e442c35b37a6c9b150f8ae2ccab on code-quality into bdc2e7557f88ea38577a7d112a1e1890f9280b33 on master.


Fix display condition of cookie setting button

About pull request
If color_scheme_switch_side_nav is false, side-nav-bottom-buttons-container was not displayed.
Therefore even though cookie_settings_button_enabled is true, the cookie setting button was also not displayed.
What's changed to accomplish the problem described in the issue
Add OR condition cookie_settings_button_enabled == true for enable side-nav-bottom-buttons-container.
The coding rule check is done [yes]

To get more insight, please check the related issue
#104.

Thank you for letting me contribute.
I really like the template design you created.
I would like to help you with this project as much as possible.

Thank you. ありがとうございます。よろしくお願いします。

@joshbuker Sorry to ping you like this, I thought you may wanna need this.

@joshbuker Sorry to ping you like this, I thought you may wanna need this fix for your page.
If it's irrelevant please ignore.

What does this fix affect? Reading through it and struggling to understand what the original bug was from context.

The bug is, If side nav color scheme button visibility is false and cookie settings button visibility is true
side nav bottom buttons will not displayed.
I checked your web site and you are using color scheme button and  cookie settings button, so this is irrelevant for you.
btw, it seems like you are not using cookies, you can disable side nav cookie settings button because it is non functional on your site.
open main.yml an change enable to false.
cookie_consent:
enable: false


User Input - Mouse

Controls should map easily to mobile.
Using keyboard controls, allow the player to make a mixed drink from a combination of
Alcohol - Whiskey, Rum or Vodka
Mixer - Soda, Coke or Vermouth
Garnish - Lime, Cherry or Olive
Controls should also choose one of four bar lanes.
Choose either a method for making drinks - #4 will use a continuous method like so:
Drinks can be any % combination of alcohol and mixer (ie. 0.5 Whiskey + 0.5 Soda or 0.2 Rum + 0.8 Coke) plus one of the garnishes.  Drinks should also have a "chosen" bar lane.
A more discrete option could be:
Drinks are 2 parts Whiskey and 1 Parts Soda or 1 part Rum and 2 parts Coke, plus one of the garnishes.  Drinks should also have a "chosen" bar lane.
Make something that feels good to you.
In the end, produce a game object that contains, somehow, final drink values of alcohol, mixer, garnish and bar lane.

Is this done?

Although the feature/5 merge does not produce a drink gameobject, I'm closing this out.  The gameobject functionality can be completed via issue #47


spring-configuration-metadata.json  add  spring.cloud.stream.rocketmq.bindings.

version   2.1.0.RELEASE   spring-cloud-starter-stream-rocketmq
now  in  spring-configuration-metadata.json  can't find
spring.cloud.stream.rocketmq.bindings..consumer
spring.cloud.stream.rocketmq.bindings..producer
Cannot resolve configuration property  in application.properties/bootstrap.properties  in IDEA

hi, it exists in my local. Are you sure there is no spring-configuration-metadata.json?

idea can't show like others

The other configurations like spring.cloud.stream.bindings.* is defined by project Spring Cloud Stream.

ok


[firebase_crahlytics] Do not report dart errors

Steps to Reproduce

use firebase_crashlytics 0.0.2
Execute the following code.
Report is not displayed because this does not catch dart exceptions.

void main() {
  FlutterError.onError = (FlutterErrorDetails details) {
    Crashlytics.instance.onError(details);
  };
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    throw Exception('error test.');
  }
}

Because only Flutter Error is supported, it is also necessary to handle dart errors.
https://flutter.dev/docs/cookbook/maintenance/error-reporting#5-catch-and-report-dart-errors
There is no interface to call when a dart error occurs.
runZoned<Future<void>>(() async {
  runApp(CrashyApp());
}, onError: (error, stackTrace) {
  // There is no interface to call here
});

flutter doctor -v
[✓] Flutter (Channel dev, v1.4.7, on Mac OS X 10.14.3 18D109, locale ja-JP)
    • Flutter version 1.4.7 at /Users/ko2ic/Sources/dart/flutter
    • Framework revision 1bfa2f2311 (5 days ago), 2019-03-29 10:05:18 -0700
    • Engine revision c4d14a0071
    • Dart version 2.2.1 (build 2.2.1-dev.2.0 None)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/ko2ic/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = /Users/ko2ic/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version openjdk version "1.8.0_152-release"
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.2, Build version 10E125
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 34.0.1
    • Dart plugin version 182.5215
    • Java version openjdk version "1.8.0_152-release"

@ko2ic
The issue at https://github.com/flutter/flutter/issues/30477 has been closed and moved here. Future collaboration on this issue will be done here.

@ko2ic is it still the same with flutter_crashlytics 1.0.0?

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of 'flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Added simple example at start

Yeah, I have a PR to fix that in FIRRTL, but it will be a while until its released as a snapshot. However, eventually it should work better.


patch: Resolve link checker issues for HTML files

The link checker wasn't working correctly for HTML files so dove deep in Lychee and resolved the issues
Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci I self-certify!


how do i a enroll a new user with a digital?

im try to use "38．Enroll user FP online CMD：0x5C" but i not receive any message or sound at the device.
i receive only 11 bytes, against in the comunication protocol says 12 on return message.

@fdutra22 can you write your own code?


SAA-1208 adding a new transaction handler around attendance changes to guarantee attendances are updated before events are emitting attendance amended events.

A problem has shown itself whereby (bulk) attendance amendment events are being processed by the sync service before the changes had fully committed in our service DB.  This resulted in the activities service and NOMIS being out of sync with each.
The crux of the changes here involve wrapping the attendance changes in a completely independent new transaction which has to complete after which we publish the events in the service/component and not use the attendance entity listener.
Hopefully this is a short term solution with regards to raising attendance amended events until we have time to revisit and address  a more complete solution towards the raising of events in general.  Ideally we should be looking to move away from entity listeners e.g. use of TransactionalEventListener which can determine when a transaction has committed so could replace the use of entity listeners perhaps.

Codecov Report
Patch coverage is 98.80% of modified lines.

Files Changed
Coverage

...esmanagementapi/entity/AttendanceEntityListener.kt
ø

...ce/events/handlers/OffenderReceivedEventHandler.kt
92.30%

...ctivitiesmanagementapi/entity/ScheduledInstance.kt
100.00%

...ivitiesmanagementapi/service/AttendancesService.kt
100.00%

...smanagementapi/service/ScheduledInstanceService.kt
100.00%

...ivitiesmanagementapi/service/TransactionHandler.kt
100.00%

...e/events/handlers/ActivitiesChangedEventHandler.kt
100.00%

:loudspeaker: Thoughts on this report? Let us know!.


Type-safe way to construct/deconstruct Datum/Redeemer

Currently:

The recommended way for Datum/Redeemer construction is using evalParam - which is quite verbose, especially for enum types.
There's no straightforward way for deconstruction - which is useful for chain indexing.

This problem can be split into 2 smaller ones:

Way to construct/deconstruct in a type-safe way, given the schema is available.
Synchronize the schema and Helios type definitions.

For the (1) problem, Lucid had some recent successes with typebox
https://github.com/spacebudz/lucid/blob/main/src/plutus/data.ts
https://github.com/spacebudz/lucid/blob/main/tests/data.test.ts
I personally gave it a try, it works quite well. The only problem is it's not compatible with how Helios is encoding structs (data-lists).

For construction I was thinking the following:
program.types contain javascript equivalents of all the types that are accessible in the top-scope of the main script.
Eg. program.types.MyStruct
Then instantiating would be like a regular javascript class: const {MyStruct} = new program.types; const my_instance = new MyStruct(...)
The fields of MyStruct can either be something primitive (which is automatically converted to the correct DataType), or another Helios-to-javascript-equivalent instance.
For enums the constructable class could be one level deeper: const {MyRedeemer} = program.types.MyRedeemer; const my_instance = new MyRedeemer.MyVariant(...)

Deconstruction could be similar to Helios: automatically defining a static fromData method for each helios-javascript-equivalent type.

Your proposed solution sounds great to me. Just one extra question: Will the TS-type interfaces be auto-generated too? I'm afraid it might not be feasible with dynamic runtime types - so maybe we need some codegen here?

What do you mean by codegen? Like a build step that creates type-definitions?
How would you like to run the codegen? Using the helios cli and a file-listener?

What do you mean by codegen? Like a build step that creates type-definitions?

Yes.

How would you like to run the codegen? Using the helios cli and a file-listener?

I'm fine with having to run helios-cli every time the types are changed. During development, having file listened would be good - but it can be done externally by the devs, we can just leave some docs.

This feature is now available (v0.10.6).
To get an object containing all the user-defined types:
program.types
To construct a struct:
new program.types.MyStruct(field1, field2, ...)
To construct an enum:
new program.types.MyEnum.MyVariant(field1, ...)
To use a constructed value as script parameter:
program.parameters = {MY_PARAM: myValue}
program.parameters returns an object containing all the current parameter values as instances similar to these user-type instances.
Only the following builtin types are supported: Int, Time, Duration, Bool, ByteArray, HeliosString,  List, Option, HeliosMap, Address, TxId, TxOutputId, Value, StakingValidatorHash, PubKeyHash, ValidatorHash, StakeKeyHash, DatumHash


How to separate sentences when there is no punctuation?

for example:
how are you fine thank you and you

I hope to be able to divide the above words into 3 sentences.

how are you
fine thank you
and you

Probably need some sort of machine learning module to do that kind of sentence breaking.  pySBD uses heuristics based on punctuation.

Yes pySBD is rules based and heuristics module so wont work on use-cases where there not all any punctuation.


Help needed

When you want to help with the development of this library, there are plenty of things were you are more than welcome:
Blocked Issues

In #3106, the missing possibility to switch on JSON_DIAGNOSTICS via CMake when installing the library via Homebrew. If you are familiar with Homebrew, you could fix the formula accordingly.

In #2907, an issue in the pkgconfig integration is reported. Apparently, executing tests change the install prefix.

See here for a list of all issues and pull requests that need help.
Reviews
See here for a list of all pull requests that need to be reviewed.
Evergreens

The documentation can always be improved - be it with more examples or just by fixing typos.

Hey guys, i am a new contributor but this platform is very confusing for me. I saw this good first issue tagged and when i opened, i don't know how to solve the issue and also what the issue is! I m a begineer. Can somebody please help??

Did you find a way I am a new started any suggestions or let do it together

Hi nice to meet you Did you find a way I am a new started any suggestions or let do it together

Hey @Viswa70, are you interested in us working together?

Really i enjoy to do it

Really i enjoy to do it

Yeah absolutely. Did you take a look into the task tho?


Stop using 64 bit numbers for all numeric event fields

Right now when we parse the Go event defs, we always generate Rust code with the 64bit version of the type. Even if Go says a field is an i32, we generate the corresponding Rust field with i64. While this works, because memory use is important on lambda we should match the Go size exactly.
That being said, it looks like pretty much all numeric types are 64 bit in the Go typedefs.

Actually, it looks like the events are always using 64 bit numbers so while this isn't "correct" there is no real need to do anything.


GLBC: Each ingress sync updates resources for all ingresses

For every ingress, the controller runs resource synchronization for all ingresses, thus resulting in unnecessary work.
Furthermore, if a GCP error occurs when handling one of the ingress resources, an event will be raised for every ingress resource.

What event?

A K8s Event record noting an error/warning message.
Take a look at this code: https://github.com/kubernetes/ingress/blob/7b38e5a36ed7e097e1cd98aca0f37c504059ab4c/controllers/gce/controller/controller.go#L311-L320
Checkpoint (https://github.com/kubernetes/ingress/blob/7b38e5a36ed7e097e1cd98aca0f37c504059ab4c/controllers/gce/controller/cluster_manager.go#L141-L149) creates/manages the resources for all ingresses. If there's an error, it's returned and an event is recorded for the ingress which caused the sync, but not necessarily the ingress which caused the error.

blech!
On Thu, Aug 3, 2017 at 10:10 AM, Nick Sardo<EMAIL_ADDRESS>wrote:

A K8s Event record noting an error/warning message.
Take a look at this code: https://github.com/kubernetes/ingress/blob/
7b38e5a36ed7e097e1cd98aca0f37c504059ab4c/controllers/gce/
controller/controller.go#L311-L320
Checkpoint (https://github.com/kubernetes/ingress/blob/
7b38e5a36ed7e097e1cd98aca0f37c504059ab4c/controllers/gce/
controller/cluster_manager.go#L141-L149) creates/manages the resources
for all ingresses. If there's an error, it's returned and an event is
recorded for the ingress which caused the sync, but not necessarily the
ingress which caused the error.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/ingress/issues/950#issuecomment-320032018,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFVgVMFWnIrMtJ0fE_9BSeJJIEw2IWOTks5sUf74gaJpZM4OU9aH
.

This issue was moved to kubernetes/ingress-gce#44


Several servers with different IP addresses

Several servers with different IP addresses that you have, for example, a main server and a dev server with other addresses
how can i set this so where can i find this set

Read the docs on convars. This is already possible.


[1.19] Mob Imprisonment Tool does not retain Gossip tag upon releasing Zombie Villagers

Industrial Foregoing Version: <IP_ADDRESS>
Titanium Version: 3.7.0
Using Integrated dynamics I found out that when transporting zombie villagers with the Mob Imprisonment Tool (MIT), the entity data about gossips makes it into the MIT, but once the Zombie Villager is released it does not have the Gossips NBT tag whatsoever. Pictures attached in order are Uncured Villager, Zombified Villager, MIT with Zombie Villager, and then Released Zombie Villager.

Gossips lies between FallFlying and PortalCooldown

This might be some minecraft special thing, I don't remove anything from it.


[BUG] Failed consecutive blip check with 0.5.3 and v2_0_0.wav

Using v0.5.3 pulled shortly before the time of this writing, I am getting an error when using an input/output WAV file set that worked with a previous version of NAM, which was pulled several days ago. The output file was recorded using v2_0_0.wav as the input.
Now I get the following error message when using this input/output file pair that worked previously. I am also getting the error for a new set of input/output files recorded using v2_0_0.wav as the input. However, I am able to train using a different output which was recorded using v1_1_1.wav as the input. I did not try other sets based on v1_1_1.wav to see if they lead to the error message though.
This is the error message:
Delay wasn't provided; attempting to calibrate automatically...
Delays:
5
5
After aplying safety factor, final delay is 1
Plotting the delay for manual inspection...
V2 checks...
Replicate ESR is 0.00099014.
Failed consecutive blip check at start of training signal. The target tone doesn't seem to be replicable over short timespans. Is there a noise gate or a time-based effect in the signal chain?
Failed checks; exit training
Steps to reproduce the behavior:

git pull
conda update conda
conda env update --file environment_gpu.yml --name nam
nam
Initiate a training from the GUI using the following files as input and output.
v2_0_0.wav.zip
output.wav.zip

Expected behavior:
This set of files did not lead to a blip test failure for a past version of NAM.
Desktop:

OS: Windows 11
Running NAM locally
NAM v0.5.3 pulled around 0400 UTC on May 8, 2023.

Replicate ESR is 0.00099014

That's pretty noisy--the data has an ESR of almost 0.001 with itself.  Like the error says-- Is there a noise gate or a time-based effect in the signal chain? or is your recording setup just super-noisy?
I'm still tuning this, so if you wanted to shoot me your reamp & tell me about it I'd love to have a look and keep tweaking things.

Thank you, and yes the noise was indeed high. I hadn't examined the output file's noise, given that it worked up until now. I was able to improve the noise issue through ground lift settings on the DIs involved and a EBTech hum eliminator, and it looks much cleaner. For some frequencies the noise is now almost 40dB lower.
There were no noise gates or time delays in the signal path., and there are none in these new recordings.
It worked for one new recording, but not for another. They are attached here. The only difference between them is the load box that was used for the amp. The DIs all remain the same. I used outboard DIs and did not use those in the load boxes, so only the load is different. And the loads have quite similar impedance curves. Load box B boosts the highs slightly more than Load box A. They are very similar for low to mid frequencies. Looking at the output files, the noise is only a few dB higher for the output that doesn't work with v0.5.3.
These are the files:
output2-OK.wav.zip
Same signal path as output3, except that output2 uses load box A. This file works with v0.5.3.
output3-NG.wav.zip
Same signal path as output2, except that output3 uses load box B. This file gives the following error with v0.5.3.
Delay wasn't provided; attempting to calibrate automatically...
Delays:
5
5
After aplying safety factor, final delay is 1
Plotting the delay for manual inspection...
V2 checks...
Replicate ESR is 0.00008851.
Failed start-to-end blip check for blip replicate 1. The target tone doesn't seem to be same at the end of the reamp as it was at the start. Did some setting change during reamping?
Failed checks; exit training

Welp, at least it checks out--the ESR between the blip at the start and the end is 0.00107 -- 0.00007 higher than the threshold for the error, 0.001!
I guess I can just back off the threshold?. If you make this higher, it won't complain.
I'll probably want to make some option to "ignore checks" so that if you really want to insist on going ahead, you can (otherwise I'll probably never hear the end of it from people and plenty of people trying to share how to hack the code to get around it)...
But the intent is to warn that your data are bad, so your model will probably be bad as well (garbage in, garbage out).
I'll keep tweaking it.

That's indeed close to the threshold! Is that comparing the blip at the start and the end? If so, the main area of difference between the start and end blips in the problem output file looks to exist below 60Hz. There is a slightly longer section of "ringing" below 60Hz for the first blip than the last blip in that file. Otherwise they at least appear similar in a spectrogram for other frequency regions. Perhaps a highpass filter on the blips before that analysis is done? Given that at least for guitar the region doesn't matter much.
Or back off the threshold. This is a high gain capture. Maybe set a different threshold if the user chooses High Gain in the metadata section? I'm assuming this blip difference wouldn't be so pronounced using a cleaner amp setting.

Just pushed some updates.
The typical ESRs associated w/ a blip are much larger (ESR is not an absolute! The "0.01" is specific to the test signal that's part of the standard reamp signals I have people use! 😉), so I took some time to calibrate it a bit better.
I also added the ability to ignore it, so that'll be an option (hopefully not needed anymore, but juts in case! 😬)
Should solve your problem, so closing this 🙂


RSpec does not reset ActiveSupport::CurrentAttributes around each example

What Ruby, Rails and RSpec versions are you using?
Ruby version: ruby 3.0.1p64 (2021-04-05 revision 0fb782ee38) [x86_64-darwin20]
Rails version: <IP_ADDRESS>
RSpec version: RSpec 3.10

rspec-core 3.10.1
rspec-expectations 3.10.1
rspec-mocks 3.10.2
rspec-rails 5.0.1
rspec-support 3.10.2

Observed behaviour
rspec-rails does not reset CurrentAttributes. That means if one test sets a value using CurrentAttributes, it can very easily pollute other tests.
Expected behaviour
Rails ActiveSupport::TestCase automatically resets CurrentAttributes between each test (see active_support/current_attributes/test_helper.rb). I would expect rspec-rails to do the same.
Can you provide an example app?
Here is a trivial example app using CurrentAttributes and an ActiveSupport::TestCase that passes:
require "rails"
require "rails/test_help"

class MyApp < Rails::Application
  config.eager_load = false
end

MyApp.initialize!

class Current < ActiveSupport::CurrentAttributes
  attribute :user_id
end

class CurrentTest < ActiveSupport::TestCase
  test "allows setting user_id" do
    Current.user_id = 123
    assert_equal(123, Current.user_id)
  end

  test "has a nil user_id by default" do
    assert_nil(Current.user_id)
  end
end

Test output:
$ bundle exec ruby testcase.rb
Run options: --seed 52779

# Running:

..

Finished in 0.001012s, 1976.2846 runs/s, 1976.2846 assertions/s.
2 runs, 2 assertions, 0 failures, 0 errors, 0 skips

However the same app, when tested via rspec-rails, fails:
require "rails"
require "action_controller/railtie"
require "action_view/railtie"
require "rspec/rails"

class MyApp < Rails::Application
  config.eager_load = false
end

MyApp.initialize!

class Current < ActiveSupport::CurrentAttributes
  attribute :user_id
end

describe Current do
  it "allows setting user_id" do
    Current.user_id = 123
    expect(Current.user_id).to eq(123)
  end

  it "has a nil user_id by default" do
    expect(Current.user_id).to be_nil
  end
end

Spec output:
$ bundle exec rspec rspec.rb
.F

Failures:

  1) Current has a nil user_id by default
     Failure/Error: expect(Current.user_id).to be_nil

       expected: nil
            got: 123
     # ./rspec.rb:23:in `block (2 levels) in <top (required)>'

Finished in 0.01094 seconds (files took 0.66551 seconds to load)
2 examples, 1 failure

Failed examples:

rspec ./rspec.rb:22 # Current has a nil user_id by default

Here is the Gemfile I used:
source "https://rubygems.org"

gem "rails", "~> 6.1"
gem "rspec", "~> 3.10"
gem "rspec-rails", "~> 5.0"

As per Rails doc:

Abstract super class that provides a thread-isolated attributes singleton, which resets automatically before and after each request. This allows you to keep all the per-request attributes easily available to the whole system.

The reset code is injected by Rails here into ActiveSupport::TestCase.
It was introduced here, but I can't say I follow the "we were missing plain tests, so this is that" rationale behind this change.
Do you have an idea?
In rspec-rails, we don't inherit from ActiveSupport::TestCase, and this reset code is never called.
On the one hand, it makes total sense to reset the current attributes. On the other - it is reset before and after each request, and why would we explicitly reset it in test support code once again?
      app.executor.to_run              { ActiveSupport::CurrentAttributes.reset_all }
      app.executor.to_complete         { ActiveSupport::CurrentAttributes.reset_all }

Can you please check if the issue still reproduces for controller/feature/request specs?

Thanks for the quick response!
Yes, I think your assessment is correct. I just confirmed that request specs do not have this problem. As you said, it is only an issue for "plain tests".
My understanding is that request specs, since they simulate actual web requests, are inherently using the app "executor". That executor takes care of resetting the current attributes. In this case the test environment is very similar to the actual production environment, and so the behavior of current attributes is similar.
For a "plain test" like a model spec, there is no request/response or active job execution being simulated. Hence the app executor is not involved, and so current attributes will not be reset via the executor. That is why https://github.com/rails/rails/pull/39493 introduced the explicit reset in ActiveSupport::TestCase, so that current attributes would get reset for these "plain tests" as well.
I think this is reasonable behavior and expected for a framework feature like ActiveSupport::CurrentAttributes. Unit tests are supposed to be independent of one another, and thread-local storage breaks that interdependence. Since CurrentAttributes is a standard part of the Rails framework I would expect a built-in solution for this test interdependence issue.

Agree. I see your point and the reasoning behind https://github.com/rails/rails/pull/39493 now as well.
However, if the subject under test relies on some specific current attribute, and this value may affect the results of the tests, shouldn't it be explicitly set in the test setup?
What I mean by this is e.g. Time.zone. It's set somewhere in rack in the app, but, obviously, this isn't happening in model specs.
What I could quickly find in the wild:
# chatwoot/app/models/user.rb
class User < ApplicationRecord
  def current_account_user
    account_users.find_by(account_id: Current.account.id) if Current.account
  end
...
  def assigned_inboxes
    inboxes.where(account_id: Current.account.id)
  end

So, apparently, an example group with no explicit setup could rely on the fact that Current.account is nil, but it can't, since CurrentAttributes leak state.
Personally, I feel a smell when Current. is directly used in models. And when it's reset four times for a single test. It's subjective, and I don't account for that in my reasoning.
I have no objections to add the reset code to all Rails-specific example groups (RSpec::Rails::RailsExampleGroup) by including ActiveSupport::CurrentAttributes::TestHelper  to it. I did not test this approach, though. To make it work for everyone, I suppose a check for defined?(ActiveSupport::CurrentAttributes::TestHelper) needs to be made.
Would you like to submit a pull request?

If you add type: :model the problem should similarly go away, we don't include rails helpers except when running a rails test, so currently you'd need to pick one of the rails types. We could add a generic "type: :rails" for the base class, or help guide towards defaulting towards including the helpers, but its not something we can do into bare RSpec tests, because they are for bare Ruby, not bare Rails, and there are people who mix these (e.g. write proper plain old ruby with specs, even in Rails app)

If you add type: :model the problem should similarly go away

Ah, you're right!
I had originally encountered this problem with current attributes in a rails app where I have app/commands and app/services classes. Apparently when I write specs for these, rspec-rails does not have any built-in behavior via infer_spec_type_from_file_location! for "commands" and "services". Thus my specs are treated as "plain ruby".
I see this is a mistake on my part; I need to configure rspec to recognize those directories. Something like this, I guess?
RSpec.configure do |config|
  config.include(
    RSpec::Rails::RailsExampleGroup,
    file_path: %r{spec/commands|spec/services}
  )
end

In any case my original issue does not seem to be a bug, so I will close.

To help others finding this issue, I was confused for awhile as to why this was closed. I realize it has to do with new Rails versions since. This was for Rails 6.1 so likely using type: :model on a plain spec fixed the issue, but once you are on Rails 7.0 and loading 7.0 defaults (so executor_around_test_case is true) this not the case and CurrentAttributes will leak from both plain tests and some rails example group tests.
Since rspec-rails already includes the execution context test helper, I've created #2752 with a temporary fix to include the current attributes test helper to match what Rails 7 does. That will still require a rails based spec (e.g. type: :model) but makes sense based on rspec-rails setup. The ideal fix is in #2753 as that should account for both cases, where executor_around_test_case is true and false, with the same rails based spec type requirement.


Health check related bugs fixes for k8s

https://github.com/rancher/rancher/issues/4481
https://github.com/rancher/rancher/issues/4354
@ibuildthecloud

The PR:

enabled missing health check on kubelet/proxy/kubectld/kubernetes services
excludes instance in unhealthy state from the list of labels providers

LGTM


Run conformance periodic jobs on specific branches for master

What this PR does / why we need it:
It is not necessary to run both master and release 1.10 branch
cloud-provider-openstack to test against k8s master, v1.10 and v1.11
in OpenLab periodic jobs, that allocate lots of testing vm resources.
Add branches option into job define in order to limiting specific job
just run on specific branch.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

recheck cloud-provider-openstack-acceptance-test-flexvolume-cinder

recheck cloud-provider-openstack-acceptance-test-standalone-cinder

/test cloud-provider-openstack-acceptance-test-flexvolume-cinder

/test cloud-provider-openstack-acceptance-test-standalone-cinder

/approve
/lgtm

/ok-to-test
/approve


VMware: volume statistics show wrong values

ISSUE TYPE

Bug Report

COMPONENT NAME

GetVolumeStatsCommand

CLOUDSTACK VERSION

4.11+ (tested 4.11.2 and Master)

CONFIGURATION

NA
OS / ENVIRONMENT

NA
SUMMARY

physical volume size reported is wrong, and doesn't change after you write a lot of data to the VM's root disk (be it linked clone or full clone)
STEPS TO REPRODUCE
configure "volume.stats.interval" to 60000 (these are milliseconds, not seconds as stated atm - I raised a separate PR for this).

Grep management-server.log for "GetVolumeStatsCommand" and observe it's running each minute

Inside the VM, write i.e. 50MB to disk  -  "dd=/dev/urandom of=ddfile bs=1M count=50; sync"

Wait for next run of GetVolumeStatsCommand from the mgmt logs - check the size reported in the text output and also on the GUI

EXPECTED RESULTS

Correct physical size is reported
ACTUAL RESULTS

0.53KB reported instead of the real size (linked clone - from image bellow is ~ 220MB)

Initial support seems implemented in 4.11 https://github.com/apache/cloudstack/commit/4627fb2cd7556173bd7e58bf1bec22c93a78f31d
But 4.11.2 is not working - so not sure if it was ever working properly.
Works fine for KVM.

Will investigate this tomorrow @andrijapanic cc @PaulAngus @borisstoyanov

Reopened due to this case where flat files are not part of the chain/info and are ignored in stats calculations: https://github.com/apache/cloudstack/pull/3473#issuecomment-512731291

Fixed recently https://github.com/apache/cloudstack/pull/4225


WIP NO-ISSUE: Bump golangci-lint to 1.45.2

Maybe use what assisted-service is using?
https://github.com/openshift/assisted-service/blob/master/hack/setup_env.sh#L73-L74
Or is there an issue with that version too we think?

I was thinking we could update both, starting with this repo.
However, the issue is that we did go import which broke down - and apparently not needed for tests?

However, the issue is that we did go import which broke down - and apparently not needed for tests?

Do you mean removing go get -u<EMAIL_ADDRESS><EMAIL_ADDRESS>is what fixed it?
I think we need that for make format unless we're not running that in the CI?

If this fixes the issue and we don't need to use these commands in CI then I'm okay with removing them I guess :shrug:

Do you mean removing go get -u<EMAIL_ADDRESS><EMAIL_ADDRESS>is what fixed it?

Right, I'll check if make format works without these and split this change in a different PR

Will be reworked after #76

This bump is useless since golangci-lint version is defined in the image :/


FullNode.shutdown() should stop network discovery and catchup

Those tasks are started independently, and they keep on living in their own bubble, but can cause random crashes. Trying to upgrade LocalRest to include https://github.com/Geod24/localrest/commit/814377a268953a06c2f1237fc16d3195cb83f80f which stopped ignoring put messages to closed message box revealed the issue:
When a test shut down (with the scope (exit)), there can be a race condition where a getBlockFrom is fired from one node to another node that has already been shut down (in this loop).
This obviously is not the intended behavior.

After careful consideration, this is not needed. The code should handle disconnect / lost peers gracefully. The LocalRest code has been fixed to do so.


runtests: Fix filename manipulation

Previously a file named test.ol.py would shadow a file named test.py, potentially causing the test to incorrectly pass.

Excellent!


Unable to import SmartExplainer. Running into ModuleNotFoundError: No module named 'tkinter' error

Hi Team,
I am trying to import SmartExplainer on Windows OS but getting ModuleNotFoundError: No module named 'tkinter' error. So tried installing tkinter but that also did not work. Please find teh below screenshot and please suggest something.

tkinter should already be a part of your python distribution
Are you running Python 2.7? In python 2.7 tkinter is called as Tkinter.
Try rerunning your code in Python 3.
If you can call python -m Tkinter then you are in Python 2.7
if you can call python -m tkinter you're in Python 3+

Just a heads up that this is shap and different than shapish which you are importing from.
Looks like this is neither a shap or shapish issue though, but an environment issue. Specifically this looks like a Linux environment of some sort running python 3.8. After looking into this, it looks like you need to use whichever distros package manager to install tkinter rather than pip.
If it's Ubuntu then something like this should resolve your issue:
'''
sudo apt-get install python3-tk
'''


Clarify error between MO2 failing to launch and Skyrim crashing

To do this, we should check for the Skyrim process to be launched. Once it is available we can consider the MO2 command a success (rather than failing if the Skyrim process exits).
Then, have a separate process running in the background that checks if the Skyrim process is still running to keep the launcher locked.
This also applies to launching and then killing MO2.

This is no longer an issue. The error message has been clarified and Skyrim crashing does not cause an error message, the launcher will just unlock.


Imported Ring of Protection not applying bonus AC

Seems to have happened after the AC changes to the 5e system in Foundry.
Character:
https://www.dndbeyond.com/profile/Blue_Heron/characters/50710374
I made sure to re-import items using the muncher, and even with the ring of protection equipped, his AC is 20, not 21.

Ah! I'll try and get a fix in for this, but with the new AC stuff, you almost certainly want to install Dynamic Active Effects to get AC bonuses from things like Rings of protection -  you don't need to use any of the other features, just have DAE installed and active.

There is a temporary work around in v 2.3.9 that will create an effect that works without DAE, but not sure if I will keep that or move back to setting a flat AC if DAE is not installed.

Have enhanced this in the latest version, closing this issue.


allow burning of multiple SD cards at once.

It would be an awesome feature of device init if one of the options was to select all available SD drives, and write the same image to them. This could be useful when needing to provision a large number of devices.

This feature is implemented in the form of a plugin, given that it's not a very common use case for the average user: https://github.com/resin-io/resin-plugin-multiburn.


Opening heist through beardlib editor menu doesn't load into editor

When I open a heist in the beardlib menu, it just doesn't load it into the editor, I get no grey loading box like this https://image.prntscr.com/image/yiB2He7fRRm5ougaH5-onw.png, either. F10 does nothing, even on OVERKILL heists (e.g. nightclub)
Here's a conversation I had with Quackertree about this issue:

F10 only works in editor mode, are you sure you're loading the heists through the BeardLib-Editor menu - levels?
What version are you on?

I'm on the newest version of beardlib and beardlib-editor, I have been loading heist through BeardLib-Editor menu - levels

Give me your log file

If you mean a crash log, I don't have one as my game didn't crash at all

mods/logs.

logs.zip

07:41:39 AM FATAL ERROR:  (..\src\InitiateState.cpp:388) cannot open EnvironmentManager.lua: No such file or directory
There's something wrong with your BeardLib I assume, delete it completely and install it from here https://modworkshop.net/mydownloads.php?action=view_down&did=14924

Still have the same issue, here's my current logs now
logs.zip

Also after checking my beardlib and beardlib-editor hooks folder, enviromentmanager.lua is in both of them

02:07:48 PM FATAL ERROR:  (..\src\InitiateState.cpp:408) mods/BeardLib/Hooks/MenuComponentManager.lua:42: attempt to index global 'BLTNotificationsGui' (a nil value)
Please update your BLT.

In anyway what's wrong is not the editor it's most likely the version of BLT you have which is outdated.


Protractor 2.0.0 with directConnect doesn't work on Firefox 38

Firefox 38 is out; trying to run Protractor 2.0.0 on it results in the following output:
[firefox #2] PID: 81496
[firefox #2] Using FirefoxDriver directly...
[firefox #2] ERROR - Unable to start a WebDriver session.
[firefox #2]
[firefox #2] /Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1877
[firefox #2]       var result = fn();
[firefox #2]                    ^
[firefox #2] Error: Failed to install profile; firefox terminated with Result(code=null, signal=SIGSEGV)
[firefox #2]     at Error (native)
[firefox #2]     at /Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/firefox/binary.js:239:67
[firefox #2]     at [object Object].promise.ControlFlow.runInFrame_ (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:1877:20)
[firefox #2]     at [object Object].promise.Callback_.goog.defineClass.notify (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:2464:25)
[firefox #2]     at [object Object].promise.Promise.notify_ (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:563:12)
[firefox #2]     at Array.forEach (native)
[firefox #2]     at Object.goog.array.forEach (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/array/array.js:203:43)
[firefox #2]     at [object Object].promise.Promise.notifyAll_ (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/promise.js:552:16)
[firefox #2]     at goog.async.run.processWorkQueue (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/goog/async/run.js:125:21)
[firefox #2]     at process._tickCallback (node.js:364:9)
[firefox #2] From: Task: WebDriver.createSession()
[firefox #2]     at Function.webdriver.WebDriver.acquireSession_ (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:155:22)
[firefox #2]     at Function.webdriver.WebDriver.createSession (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/lib/webdriver/webdriver.js:129:30)
[firefox #2]     at new Driver (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/selenium-webdriver/firefox/index.js:272:36)
[firefox #2]     at [object Object].DirectDriverProvider.getNewDriver (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/lib/driverProviders/direct.js:75:16)
[firefox #2]     at [object Object].Runner.createBrowser (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/lib/runner.js:180:37)
[firefox #2]     at /Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/lib/runner.js:257:21
[firefox #2]     at _fulfilled (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/q/q.js:797:54)
[firefox #2]     at self.promiseDispatch.done (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/q/q.js:826:30)
[firefox #2]     at Promise.promise.promiseDispatch (/Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/q/q.js:759:13)
[firefox #2]     at /Users/mgol/Documents/projects/bn/cbn/repo/polona-gui/node_modules/protractor/node_modules/q/q.js:573:44

[launcher] Runner process exited unexpectedly with error code: 1

Can confirm this for Ubuntu 12.04. On Windows 8.1 everything works fine.
There was a similar issue on Selenium: https://github.com/SeleniumHQ/selenium/issues/437
But the solution there doesn't work for me. Firefox 38 doesn't start even if I installed it in a separate directory and set the firefoxPath parameter. All versions below 38 start normally.

A coworker found that downgrading to protractor 1.8 was a workaround.

I'm seeing the same issue without Selenium while using the directConnect: true option.

Indeed, this is because of directConnect: true! I've disabled it now and
it works fine for me.

I'm also seeing this with directConnect: true with FF 38.0.1 on OSX 0.10.3.

Actually, I'm seeing the same problem with protractor 1.8.0, if I have directConnect: true, it fails, with false it works.

It looks to me like this could now be addressed, since https://github.com/SeleniumHQ/selenium/issues/437 (which is presumably the cause) was fixed back in May.

This is a known issue (https://github.com/SeleniumHQ/selenium/issues/437). It is fixed by selenium and tagged for released at 2.47.1. https://github.com/SeleniumHQ/selenium/commit/144f5ee7eb7d705bad90346bb2b494ff8212fb4c

@hankduan awesome. Thanks for the update!

This is fixed with 2.4.0


auto split ':' in context

like default context key is server(really useful for me)
it is a pain to copy it again and again
or u can add auto split like if i paste "server:xxx"
it auto set to
key value
server xxx

Do you mean auto split?

yes
sry for my poor english...

We also have to consider that there are people who want to use : in context, EssentialsX does this for example

We also have to consider that there are people who want to use : in context, EssentialsX does this for example

Very good point! It seems that this will be a standard for other plugins supplying context that the contexts are name-spaced like plugin:context. Will close this for now. Remember you can add multiple permissions at once with the same context, also you will be able to bulk update contexts when #285 is merged.


Color contrast issue with the currently playing song

Describe the bug
It's almost impossible to read the currently playing song in playlists, due to the colors chosen.
To Reproduce
Steps to reproduce the behavior:

Visit a playlist
Play a song
See error

Expected behavior
The color choice should be at least AA compliant with accessibility standards. It should be easy for everyone to read.
Screenshots

Here's a crop of the problem area:

General information:

Distribution: Fedora 34 beta, Silverblue
Desktop: GNOME
Installation method: Flatpak, Flathub
Version: 0.1.10
Device used: Desktop

Additional context
If the same color would be applied to the background, it'd be a lot more readable. Perhaps something like this quick mockup was the intent?

Thanks for the report! Yup, that's less than ideal :/ The blue background as per your suggestion is already in use for the selection mode (which you can enter by clicking the check mark button)
I thought about replacing the track number by something else, such as a "play" icon, but that might be too subtle when browsing a long list?

Oh, right. Selection. Good point!
It's probably worth looking at what the official Spotify client, GNOME Music, and Lollypop all do.

^ doing this for now! Not 100% satisfied as it's not visually striking enough to see it when scrolling long content, but that's better still

Fixed, will be in next version!


Improved documentation and Q2 summary visualizations; centralizing parameters

An attempt to summarize new stuff:

A lot of changes to the README. I'll summarize a few of the important things here, but it's probably best if you just read over the new README.

Reorganized things: add quick examples of running Songbird standalone + in Q2 to the top of the README, both examples using the Red Sea dataset (and using parameters that I've previously ensured make the model fit reasonably well). Both examples stress the importance of checking model fit.
Added explicit sections on "Interpreting model fitting information", "Adjusting parameters to get reasonable fitting", and "Specifying a formula". A lot of these sections is cobbled together from stuff that was previously buried in the FAQs.
Added screenshots of tensorboard and the q2 summarize-single visualizations for the same dataset, with the same parameters. (These URLs will be un-broken when this PR is merged in—for now, the screenshots can be viewed here (tensorboard) and here (q2).)
Please double check to make sure that these plots look good to use as reference screenshots for "good" model fitting. I'm pretty sure they're good, but I didn't write Songbird :) If they need changing, I can change up the parameters or whatevs.
Restructured the FAQs into separate sections: added more detailed explanations of the Q2 outputs, cleaned up some of the existing text, etc.
Added a few sentences shilling Qurro ;) (mostly where the README is explaining what you can do with the differentials)

Modified the q2 summary visualizations to be a lot easier to interpret:

Switched around the cv and loss plots to match tensorboard (CV on top, loss on bottom)
Renamed "loglikehood" to loss for the now-bottom plot, to be consistent with tensorboard
Added in-HTML text explaining:

the need to decrease --p-summary-interval if you don't see anything in the plots
how to interpret the plots (this links to the new "Interpreting..." section of the README)
how to adjust parameters to get models to fit reasonably (this links to the new "Adjusting..." section of the README)

Tidied up formatting of the pseudo Q2 value for paired summaries (the plots aren't uncomfortably shifted to the right now :P)
These changes should close #72.

Removed the --i-feature-table parameter for qiime songbird summarize-single, since the number of samples is only used to compute  Q2 values in the paired summary. (Closes #76.)

This necessitated making n an optional parameter for _summarize(). As you can see in the diff, I added some code to check for the never-should-happen case where n is None but baseline is not None (i.e. we're making a paired summary but don't have access to n for computing  Q2 values). _summarize() throws an error in this case.
I also added a test to make sure that error is triggered properly.

All descriptions and defaults for all parameters (of multinomial, at least) are now stored in songbird/parameter_info.py. This centralized information is used by both the standalone and Q2 versions of Songbird, so now if you want to change a parameter you only need to change one thing.

(This is the same way we solved this sort of problem in DEICODE and Qurro.)
Making all of these changes messed up the styling of scripts/songbird, so instead of spending a bunch of time manually formatting it I just ran the file through black -l 79. (That's why that file looks so different in the diff.)
Following up on that, I lowered the default summary interval from 60 seconds to 10 seconds. Since most people's first experiences with Songbird will likely be running a small model that stops quickly, using a smaller default summary interval should help them see something. (Closes #77.)

OK, I think that's the bulk of it. Please let me know if you'd like any clarification or for me to change anything. In sum, I think these changes will make using Songbird a lot more pleasant (and understandable) for many people. Thanks!

Also, it'd probably be a good idea for multiple people to review this since this is a lot of changes, and it's important that they're correct. @lisa55asil if you'll be around tomorrow, would you mind sitting down and going over this with me? A lot of the documentation changes here involved your FAQs, so you'd probably be the most qualified person to comment on that :)

@fedarko , this is awesome!  Will take a close look at it today / tomorrow.

Thanks! @lisa55asil is looking over it right now, so I think we can merge this in once we incorporate her feedback.
One quick question—there's this part of the FAQ:

Q. What's up with the --training-column argument?
A. That is used for cross-validation if you have a specific reproducibility question that you are interested in answering. If this is specified, only samples labeled "Train" under this column will be used for building the model and samples labeled "Test" will be used for cross validation. In other words the model will attempt to predict the microbe abundances for the "Test" samples. The resulting prediction accuracy is used to evaluate the generalizability of the model in order to determine if the model is overfitting or not. If this argument is not specified, then 10 random samples will be chosen for the test dataset. If you want to specify more random samples to allocate for cross-validation, the --num-random-test-examples argument can be specified.

This says "10 random samples", but the default for --num-random-test-examples is 5. Should we change that?

Ok, Lisa's changes have been integrated in. We've both made some changes (I fixed the --num-random-test-examples thing, as well as tidied up the discussions of # of iterations. Also tried to clean up the "baseline model" description in the summarize-paired help here.
@mortonjt, pending your final approval I think this should be good to merge in.

...made a few last-minute commits showing off using qiime metadata tabulate on a differentials file, but i think that's it from me for right now :)

That's awesome! If its easy and you already have it a link to the output of
tabulating the differentials.qza from the Red Sea data would be helpful I
think.
On Mon, Sep 23, 2019 at 3:31 PM Marcus Fedarko<EMAIL_ADDRESS>wrote:

...made a few last-minute commits showing off using qiime metadata
tabulate on a differentials file, but i think that's it from me for right
now :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/pull/80?email_source=notifications&email_token=AE6EV4WU2TFHC6UYXCRBDWDQLE7SLA5CNFSM4IYSYWZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7MO2TA#issuecomment-534310220,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE6EV4UX7Y5V7ROPYBL4BI3QLE7SLANCNFSM4IYSYWZA
.

Ok! I'm not sure adding a link to a QZV is the best idea, since I guess that'd have to be periodically updated if we also update Songbird... but I added a screenshot of the QZV output. I think that's a good advertisement for that functionality :) (here's a link to the screenshot)

Noting qiime metadata tabulate is a great idea!

@fedarko are you finished with edits?

Going to make one more pass over everything today. I'll try to have it done
by 5pm EST.
Thanks!
Marcus
On Tue, Sep 24, 2019, 9:00 AM Jamie Morton<EMAIL_ADDRESS>wrote:

@fedarko https://github.com/fedarko are you finished with edits?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/pull/80?email_source=notifications&email_token=AA736PYLQZ5MMMHB364XZY3QLI2TPA5CNFSM4IYSYWZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7O4EAA#issuecomment-534626816,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA736P4Y5QAKAM273AMFUITQLI2TPANCNFSM4IYSYWZA
.

@mortonjt I'm done! Added a small subsection clearly describing setting the reference for categorical formulas, since this has confused a lot of people (me included) in the past.
Should be ready to merge in—thanks @lisa55asil + @mortonjt for your help with this :)

This is looking dope Marcus! I really like the clarification about the
reference for categorical formulas. I added a short sentence that I think
helps make this description more tangible.
On Tue, Sep 24, 2019 at 2:01 PM Marcus Fedarko<EMAIL_ADDRESS>wrote:

@mortonjt https://github.com/mortonjt I'm done! Added a small
subsection clearly describing setting the reference for categorical
formulas, since this has confused a lot of people (me included) in the past.
Should be ready to merge in—thanks @lisa55asil
https://github.com/lisa55asil + @mortonjt https://github.com/mortonjt
for your help with this :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biocore/songbird/pull/80?email_source=notifications&email_token=AE6EV4XZ6EMQNRAACDYSSLDQLJ527A5CNFSM4IYSYWZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7PZK6A#issuecomment-534746488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE6EV4WIQYV5KM4JVLNRLYLQLJ527ANCNFSM4IYSYWZA
.

@mortonjt I'm done with editing, so provided you're cool with it I think this is ready to go. Thanks!

Awesome.  Thanks @fedarko !


stream via Bluetooth fearture

can you add a Bluetooth feature so we can stream via Bluetooth car

Please use AirPlay.

but in this case i need a airplay device to plug it in the car right ?

Yes


Address breaking changes in MLJBase 0.22

This PR needs:

[x] https://github.com/JuliaAI/MLJBase.jl/pull/909 merged, and MLJBase 0.22.0 released.

In this PR we:

Address breaking changes caused by migration of measure MLJBase.jl -> StatisticalMeasures.jl
Make the string representations of metrics on the MLflow service more informative. The value of the first parameter of the metric is now included in the representation.

To do:

[x] Bump [compat] for MLJBase

This fail appears to be orthogonal to the PR. See #27 .
Tests pass locally if I manually launch the mlflow service.


Support creation of local socket listener

In cases where you want to bypass the creation of local log files, you can use Unix sockets. That is supported in rsyslog with the $AddUnixListenSocket parameter.
http://www.rsyslog.com/doc/master/configuration/modules/imuxsock.html
Currently this module does not expose a way to declare socket listeners.

Please feel free to send in a PR. Thanks


Revert "Add policy check to verify if the digest was scanned"

This reverts commit 901d9e1eab0dbdd3d8842870dfd4ba1b18cea76a.
Revert this temporarily until ec-cli PR #1486 has been merged.

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

Project coverage is 100.00%. Comparing base (80ae1a4) to head (7ca7280).
Report is 6 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main      #964   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          103       103
  Lines         4884      4859   -25
=========================================
- Hits          4884      4859   -25

Files
Coverage Δ

policy/release/lib/attestations.rego
100.00% <ø> (ø)

policy/release/test.rego
100.00% <ø> (ø)

policy/release/test_test.rego
100.00% <ø> (ø)


More relaxed licensing

Hi,
First thank you for this awesome library! We are stunned by the awesome graphics :)
The only problem we have is that this library is licensed as GPLv3, thus incompatible with our library, which is in Apache 2. Is it possible to license this in some more relaxed copyleft license like MPL?
Best regards,
Fan

Hello Fan,
Thanks a lot for your comment. The credit for graphics actually goes to the creators of the JavaScript counterpart Plotly.js. This project is something like a sewer pipe that channels the data to the browser 😉
Now regarding your question - can you, please, explain in more detail what do you mean by "our library" and in what sense the license isn't compatible with Apache 2?
The reason why I've chosen GPL is to encourage contribution back to the library. The idea is roughly this: you get the source code and in exchange the library gets back the improvements if you make any. This sort of sounds like a good deal to me.
I understand that GPLv3 is particularly problematic for a company. I don't have anything against people building products and making money with it but I'm not sure how to achieve the contribution back to the project.
Oner particular "relaxation" I was considering as another option is GPLv2. Would that help in your situation?
Cheers,
Vojta

Sorry. I've just realized what the issue is. I'm no a lawyer (obviously) and it takes me a while to recall consequences of each open source lisense. Using GPLv3 forces you to adopt the same license in your project, right?

Yeah, GPLv3 is a super strict copyleft license that basically says if you link (either statically or dynamically) to GPLv3 code, your code has to be distributed as GPLv3 as well... Also, it is not compatible with other copyleft or non-copyleft licenses like MPL, MIT, Apache, etc.

Just seen you previous comment. We are academic, so the only thing we are afraid of is that anyone with access to our colabs will have full privilege to redistribute it with anyone :)
However, I am pretty sure that this would be a bigger issue for people who work in companies like Google that wants to contribute (when I worked with them they require any code they touch to be licensed as Apache v2 🤣)
From what I understood in your desired outcome (boosting contribution back to the library), MPL 2.0 is also suitable as it is more like "file-level copyleft". I know people who uses LGPL but I don't know if that would play well with Swift's static linking.
In a more commercial view, this piece from Google is a great read.
Thank you again for the work!

I checked the licenses of Plotly.py, Plotly.r and Plotly.jl. These projects are very similar in terms of functionality but serve different languages (Python, R and Julia respectively). And they have the same MIT license as Plotly.js.
However, they the copyright holder in all of those cases is the "mother" company - a Montréal based startup called Plotly, Inc that created the original JavaScript version. They offer commercial support and hosting for graph dashboards. So MIT license makes a lot of sense for them and their customers. Just to be clear - I'm not in any way affiliated with them. They just have a great product and I honestly keep my fingers crossed for them to succeed.
It seems like the other "ports" have some degree of support from them and therefore don't rely as much on community contributions which is the missing piece in my case.
Sorry for the long comment. I just wanted to make my thinking and motivation behind the license clear.

Anyway, if the license prevents even people in the academia from using the library in the first place, there's not much of a chance to get any improvements back...
I'll switch the license to MIT (That's the only one a layman like myself can actually read and understand.)

Hopefully, CEO of Plotly Inc. will grant me some stock options or at least a bottle of champagne when they get their first Swift customer 😄.

@vojtamolda That's REALLY great news! 😄
P.S.1 If you are open to donations, you can try GitHub sponsors, they offer a 1:1 match in amount users donate to your project
P.S.2 If you happen to come by the Atlanta area I'll buy you a beer!

v0.3.1 is out now and licensed under the MIT license.
PS1: I had no idea this existed. I'll check it out. Maybe I can distribute the money between the contributors somehow.
PS2: Thanks. That's very kind of you! I'll let you know when I have a longer layover there 🍻