ActionView::Template::Error (no implicit conversion of Symbol into Integer): for Buttons with Blocks

Hi, I'm also now getting this same error as question #5 and probably #10. I didn't want to comment on issue #10 as I'm not sure if he's having the same issue nor did I want to comment on a closed issue.
The error is:
ActionView::Template::Error (no implicit conversion of Symbol into Integer):
Using the Code:
<%= f.button(type: "submit", class: "btn btn-primary btn-outline p-sm text-center w-150") do %>

Legal Pending

<% end %>

According to #5 it was resolved but I am still getting this error using Rails 5.0.2, jQuery 3.1.1, and jQuery_ujs. I've tried putting the calls in to sweet alert js files within application.js before jquery and jquery_ujs but no luck. I've confirmed that if I remove this gem then the button block works fine.
The usage for the button doesn't need to be f.button do but it does need to be a button with a block, so if there is something else I can change it to that I'm unaware of but known to work with your gem then I'm okay with that too.
Thanks!

I've since replaced the button block above with button_to to produce the desired results. So I'm fine on that front, however, the gem doesn't seem to work with button_to .. data: { confirm: .. }, only link_to .. data: { confirm: ..}.

same problem here!


WFCORE-2722: Add OTP support for identity set-password operation using filesystem-realm

Added the possibility to set OTP configurations for an identity using the file system realm
Jira Issues:
https://issues.jboss.org/browse/WFCORE-2722
https://issues.jboss.org/browse/JBEAP-10532

@bstansberry commit was updated with changes requested, I think now cover your points, param names were discussed on Elytron room.

retest this please


Needs update for Swift 4.2

Does not compile with Swift 4.2.

I'll fix it by next week. Thanks @adamsantos67

@adamsantos67 0.7.1 is released.
If there are some errors, let me know.
Thanks @adamsantos67


Can't get WebRTC Test Page running on my EC2 server?

Seems to npm install just fine. npm run develop launches the server. But I can not connect via my elastic IP port 3001? Security group is set to allow? I know it works because I can use the WebRTC test page.

I am not sure that I fully understand what you mean when you say "I can use the WebRTC test page." Does that mean you can access it from within the EC2 instance, just not over the EIP?
I am guessing that you need to set the host to <IP_ADDRESS> in the devServer config (per webpack docs) in webpack.dev.config.js

Yes. Added the host to <IP_ADDRESS> in the devServer section of webpack.dev.config.js. The webRTC test page comes up but it does not work like my local host version. I'm using the same parameters (key, secret key, and channel name) that functions correctly in my local version.

Interesting. When you say "it does not work like my local host version", what do you mean? Are there any error logs? If you look in the developer console of the browser are there any clues?

Thank you. I got it working. My bad, did not understand my chrome browser needed to ignore the certificate request. Needed to set HTTPS:true in the devServer config (per webpack docs) in webpack.dev.config.js


Incorrect simulation of always block with commercial simulators

The following always block has incorrect simulation behavior in some commercial simulators.
https://github.com/cliffordwolf/picorv32/blob/881f928e05dca9f1e750ff68793e5cc7653d7ccd/testbench.v#L83-L87
The issue is related to irq being undefined till reset, since no signal in the sensitivity list toggles till then. The resulting x-propagation causes downstream issues with some firmware including the test firmware in the firmware directory.
A simple fix is to rewrite the always block as follows:
wire [31:0] irq = {26'h0, &count_cycle[15:0], &count_cycle[12:0], 4'h0};

I think 3f9b504 should fix the issue. Please verify.


✨ 💫  sent char_span through with spaCy & regex

🐛Old Approach
TLDR - Sentences(string) were right ✅ Sentence char spans were wrong ❌
Earlier pysbd v0.2.3 depended on creating Doc object through regex search - SENTENCE_BOUNDARY_REGEX - on a preprocessed text. The preprocessing is an essential part of pysbd rules which adds Unicode symbols to decide when to/not to break the sentence. Due to the addition of Unicode symbols, char_spans found by regex were not proper in contrast to original text. Another caveat was - spaCy doesn't provide Doc object if there is leading/trailing whitespace and returns None which used to lead to wrong char spans.
✨New Approach
Use Token.is_sent_start attribute over getting an Doc object to get correct char_spans
Rely on pysbd to give proper sentences (strings) out of the original text -> use regex to find those sentence within original text & keep start char indices of sentence matches > iterate over each Token of Doc object and set Token.is_sent_start True (if in sent_start_token_idx) else False.
In preliminary testing, new approach seems to be resolving issues raised by developers.

[x] Add tests for following issues - #49, #53, #55 , #59, #58
[x] 🔐Fix security vulnerability bleach - #62
[x] Use Github Actions for CI
[ ] Refactoring & cleanups

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f7c640f). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #63   +/-   ##
=========================================
  Coverage          ?   97.15%
=========================================
  Files             ?       18
  Lines             ?      703
  Branches          ?        0
=========================================
  Hits              ?      683
  Misses            ?       20
  Partials          ?        0

Flag
Coverage Δ

#unittests
97.15% <0.00%> (?)

Continue to review full report at Codecov.

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


Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works]

Details
This is a generic password spray detection that can be used across all CIM compliant authentication sources.
Works based on a failure to success ratios common with malicious password guessing and can be run at longer durations to catch subtle password guessing attempts. (low/slow)

Sample data provided in https://github.com/splunk/attack_data/pull/851 as an internal test case, but also works well against external facing authentication data sources (VPNs, webportals, etc)
Checklist

[ ] Validate name matches <platform>_<mitre att&ck technique>_<short description> nomenclature
[ ] CI/CD jobs passed ✔
[ ] Validated SPL logic.
[ ] Validated tags, description, and how to implement.
[ ] Verified references match analytic.

Hello @nterl0k : It looks like we already have a similar detection for detections/endpoint/detect_password_spray_attack_behavior_from_source.yml,. Can you perhaps remove this detection from this PR and then we can work towards getting this this detection Detect Password Spray Attack On User tested and merged!
https://research.splunk.com/application/086ab581-8877-42b3-9aee-4a7ecb0923af/
https://research.splunk.com/application/b1a82fc8-8a9f-4344-9ec2-bde5c5331b57/

Also, we recently updated our Github CI to run build and unit-testing on all PRs. When I look at the build failure, it looks like the yaml is incorrect. :

Not sure what that screenshot is telling me, worked fine in local execution
The detection I wrote was a bit different, but I'll remove it for argument sake.
Regards,
Steven.
-------- Original message --------
From: Bhavin Patel @.>
Date: 7/23/24 1:55 PM (GMT-05:00)
To: splunk/security_content @.>
Cc: Steven Dick @.>, Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
Also, we recently updated our Github CI to run build and unit-testing on all PRs. When I look at the build failure, it looks like the yaml is incorrect. :
image.png (view on web)https://github.com/user-attachments/assets/f72acaab-180d-4eb7-8476-83d5848552be
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2245886116, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7SHWA45YZICJXHYBHDZN2KIZAVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVHA4DMMJRGY.
You are receiving this because you were mentioned.Message ID: @.***>

I think the detections are different enough from the others we already have that keeping them is probably worth it.
...from_source is currently blocked on the same issue in contentctl that another one of your PRs is blocked on: https://github.com/splunk/contentctl/pull/204
Essentially, some of the fields are not always present. In the sample dataset for that one, user is an empty field. There's only one result though so the fix proposed in contentctl won't actually fix that for this one. Easiest way to fix that is ensure there's a successful logon in that dataset that will be aggregated into the conditions.
...on_user has an issue with the link to the attack_data for it. I've tested it with the dataset for ...from_source in case that was the intended dataset, but that dataset doesn't actually have the right conditions to pass the math in your | where statement.

Interesting... Maybe I forgot to update the dataset for _user
I thought I did but it's been a busy few months.
Lemme look at it and test local again.
Regards,
Steven.
-------- Original message --------
From: Lou Stella @.>
Date: 7/29/24 12:29 PM (GMT-05:00)
To: splunk/security_content @.>
Cc: Steven Dick @.>, Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
I think the detections are different enough from the others we already have that keeping them is probably worth it.
...from_source is currently blocked on the same issue in contentctl that another one of your PRs is blocked on: splunk/contentctl#204https://github.com/splunk/contentctl/pull/204
Essentially, some of the fields are not always present. In the sample dataset for that one, user is an empty field. There's only one result though so the fix proposed in contentctl won't actually fix that for this one. Easiest way to fix that is ensure there's a successful logon in that dataset that will be aggregated into the conditions.
...on_user has an issue with the link to the attack_data for it. I've tested it with the dataset for ...from_source in case that was the intended dataset, but that dataset doesn't actually have the right conditions to pass the math in your | where statement.
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2256384889, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7VHYBEA5F4BKUHHG2TZOZUU5AVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJWGM4DIOBYHE.
You are receiving this because you were mentioned.Message ID: @.***>

Either way, looks like I need to tweak the dataset to keep _source from failing contentctl

From: Steven D @.>
Sent: Monday, July 29, 2024 12:55 PM
To: splunk/security_content @.>; splunk/security_content @.>
Cc: Mention @.>
Subject: RE: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
Interesting... Maybe I forgot to update the dataset for _user
I thought I did but it's been a busy few months.
Lemme look at it and test local again.
Regards,
Steven.
-------- Original message --------
From: Lou Stella @.>
Date: 7/29/24 12:29 PM (GMT-05:00)
To: splunk/security_content @.>
Cc: Steven Dick @.>, Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
I think the detections are different enough from the others we already have that keeping them is probably worth it.
...from_source is currently blocked on the same issue in contentctl that another one of your PRs is blocked on: splunk/contentctl#204https://github.com/splunk/contentctl/pull/204
Essentially, some of the fields are not always present. In the sample dataset for that one, user is an empty field. There's only one result though so the fix proposed in contentctl won't actually fix that for this one. Easiest way to fix that is ensure there's a successful logon in that dataset that will be aggregated into the conditions.
...on_user has an issue with the link to the attack_data for it. I've tested it with the dataset for ...from_source in case that was the intended dataset, but that dataset doesn't actually have the right conditions to pass the math in your | where statement.
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2256384889, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7VHYBEA5F4BKUHHG2TZOZUU5AVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJWGM4DIOBYHE.
You are receiving this because you were mentioned.Message ID: @.***>

@ljstella I added a log to the data source that should help clear the errors.
https://github.com/splunk/attack_data/pull/904
For _source

For _user

@nterl0k weird question- in the environment that you developed this in, did you have a non-stock CIM config or something?
One of the detections is passing now, but detect_password_spray_attack_behavior_on_user.yml is not. And its an odd reason that it is not. Splunk authentications are being mapped to the Authentication datamodel, and in the test environment, we use the account admin which is also the account name in your attack data that meets the conditions for the detection. However, the extra logins from the admin user in Splunk as part of the testing workflow throws the math off, so no results end up showing up. There's a few options here to account for this:

we can move Authentication.app to the by clause, and adjust the rest of the search as necessary, accounting for spraying against the same app instead of all apps (probably okay since we're really targetting devices, even if the logic is portable to other services?)
we can specifically remove it with Authentication.app!='splunk'
we can keep it the same and try to change the default username in our testing infrastructre. (cc: @pyth0n1c for this bit, haven't looked to see how impossible that would be)

Lol.... I could also drop new data that uses a different user besides admin?
I can Search/Replace like a champ.
Regards,
Steven.
-------- Original message --------
From: Lou Stella @.>
Date: 8/7/24 11:00 AM (GMT-05:00)
To: splunk/security_content @.>
Cc: Steven Dick @.>, Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
@nterl0khttps://github.com/nterl0k weird question- in the environment that you developed this in, did you have a non-stock CIM config or something?
One of the detections is passing now, but detect_password_spray_attack_behavior_on_user.yml is not. And its an odd reason that it is not. Splunk authentications are being mapped to the Authentication datamodel, and in the test environment, we use the account admin which is also the account name in your attack data that meets the conditions for the detection. However, the extra logins from the admin user in Splunk as part of the testing workflow throws the math off, so no results end up showing up. There's a few options here to account for this:

we can move Authentication.app to the by clause, and adjust the rest of the search as necessary, accounting for spraying against the same app instead of all apps (probably okay since we're really targetting devices, even if the logic is portable to other services?)
we can specifically remove it with Authentication.app!='splunk'
we can keep it the same and try to change the default username in our testing infrastructre. (cc: @pyth0n1chttps://github.com/pyth0n1c for this bit, haven't looked to see how impossible that would be)

—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2273683499, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7U3QAM75QQF2VYAJM3ZQIZALAVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZTGY4DGNBZHE.
You are receiving this because you were mentioned.Message ID: @.***>

If you want to, go right ahead and I'll merge it as soon as it comes in. I know this particular set of detections has already been a handful when it comes to data, but it has also shown us some of the... odd assumptions we've made, and where they break things.

Can do, gimme a little bit to update the dataset and retest local.

From: Lou Stella @.>
Sent: Wednesday, August 7, 2024 11:20 AM
To: splunk/security_content @.>
Cc: Steven Dick @.>; Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
If you want to, go right ahead and I'll merge it as soon as it comes in. I know this particular set of detections has already been a handful when it comes to data, but it has also shown us some of the... odd assumptions we've made, and where they break things.
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2273726483, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7TS2QPKX7LDZQZQYQLZQI3KRAVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZTG4ZDMNBYGM.
You are receiving this because you were mentioned.Message ID: @.***>

Did the thing https://github.com/splunk/attack_data/pull/905

Wooo! All tests passed, putting this in the queue for release! Thanks again @nterl0k

Woohooo

From: Lou Stella @.>
Sent: Wednesday, August 7, 2024 4:06 PM
To: splunk/security_content @.>
Cc: Steven Dick @.>; Mention @.>
Subject: Re: [splunk/security_content] Nterl0k - T1110.003 [Spray and Pray] or [Boring but Works] (PR #2915)
Wooo! All tests passed, putting this in the queue for release! Thanks again @nterl0khttps://github.com/nterl0k
—
Reply to this email directly, view it on GitHubhttps://github.com/splunk/security_content/pull/2915#issuecomment-2274258097, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJIYP7W7HQLF2VR3OZNTV7LZQJ44FAVCNFSM6AAAAABJOCDRM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZUGI2TQMBZG4.
You are receiving this because you were mentioned.Message ID: @.***>


Output not shown on Windows7

I have a 32 bit julia 0.3-pre on Windows 7 64bit + 32 bit python-2.7. IJulia installs and starts without problems, but there are no output cells in the notebook (using Firefox 24.4.0). It appears to execute what I put into the input cells, but the output is just not shown.
There are no error messages in the console.
Julia Version:
Version 0.3.0-prerelease+2444 (2014-04-03 00:04 UTC)
Commit 9a00d2e* (0 days old master)
i686-w64-mingw32

Closing this unless it can be reproduced with the latest version.


Restore source material for footer asset

It's important to version the source material for assets, not just the end product. Otherwise, something as simple as changing the colour of the footer would require reproducing the PDF from scratch.

I agree, but this implementation introduces a makefile and a dependency to inkscape. Just save the file as .eps, and \includegraphics will handle it natively. Anyone who wants to change the image can open it in inkscape themselves.
If you made this in inkscape, newer versions use 96 dpi vs 90 dpi. I got a notice of that when I opened the svg file. Also, if you zoom in on an old copy of the basic rules, the footer scroll has a slight bit of transparency.
footerscroll.zip

I agree, but this implementation introduces a makefile and a dependency to inkscape. Just save the file as .eps, and \includegraphics will handle it natively. Anyone who wants to change the image can open it in inkscape themselves.

I don't really follow. Inkscape converts EPS files to PDF for importing, so that's no different than working with the PDF. Either format is potentially lossy compared to Inkscape's native SVG.
Just sharing the PDF or PS would be like sharing minified JavaScript source. It works, but you wouldn't want to make changes to it.

Also, if you zoom in on an old copy of the basic rules, the footer scroll has a slight bit of transparency.

Good catch. Around 70% seems to match Basic Rules 0.2 (to my eyes, anyway).

I don't really follow. Inkscape converts EPS files to PDF for importing, so that's no different than working with the PDF. Either format is potentially lossy compared to Inkscape's native SVG.
Just sharing the PDF or PS would be like sharing minified JavaScript source. It works, but you wouldn't want to make changes to it.

Thanks for pointing that out. I have not done as much with vector graphics and thought inkscape handled .eps better.
Okay, you convinced me. Let's roll back the .svg and .pdf so there is no transparency and use tikz built in transparency. That way someone can have fully opaque scroll work if they want it. You pass a opaqueness key into the square brackets of the two tikz nodes that include the scroll image to alter the opaqueness. One of the following values should work:

ultra nearly opaque
very nearly opaque
nearly opaque

After you roll back the images to opaque and commit the change in dndheader.sty, I will merge it in.

Good suggestion. TikZ is definitely the right place to handle the transparency. Less surprising if you make changes to the SVG.
I've adjusted the asset and header style.


Blogpost: Announce the EOS Spend Explorer tool

We need to write a blogpost explaining the EOS Spend explorer and how users can utilize it. Let's enhance the importance.
Include:

[x] How to create a memo for a token transfer?
[x]  How to use the spend tool?
[x]  How to categorize past TXs?
[x]  How to view the data?

II Part: Create the spanish version and set the integration for : https://edenia.com/blog


Lat/Lon incorrect in polar stereographic grid

Getting around to doing more testing with GribApi.  The latest version still does not read Lat/Lon correctly from grib files. (comparing to wgrib2)
The bottom left of the grid has the correct lat/lon, but after that it appears they are incorrect

wgrib2 shows the bottom right
wgrib2.exe cloud_003 -s -ijlat 935 1
1:0:d=2017110700:TCDC:surface:3 hour fcst::(935,1),lon=282.212082,lat=17.342726,val=22
GribApi shows the bottom right of the grid to be at
lat=19.4115478973818, lon=278.646091411221
The gribs I'm using come from the Canadian Meteorological Center.  Examples can be found here - http://dd.weather.gc.ca/model_gem_regional/10km/grib2/00/003/.  Doesn't matter which you chose in that folder since they are all on the same grid.
Not sure how much control you have over the lat/lon locations coming out of the native portion of this solution, but figured I'd let you know it doesn't work with polar stereographic

This may have been resolved in ecCodes 2.5.0
https://software.ecmwf.int/wiki/display/ECC/ecCodes+version+2.5.0+released
GribApi.NET 1.0.0-beta4 uses ecCodes 2.4.1

Looks like the following bug got fixed in ecCodes 2.5.0
[ECC-529] - grib_get_data for polar stereographic gets wrong lat and long
I don't have permissions to read more about the bug, but potentially this is the fix.
Is it possible for me to patch?

This issue was moved to GribApiDotNet/GribApi.NET#3


test: add a delay after generation to let romans test settle

Our end-to-end test for the aila chat with fixtures is regularly failing. It got worse after removing an arbitrary delay before clicking the continue button
Problem:

The e2e tests do some things quicker than a typical user, for example the moderation step is a lot quicker. This could cause race conditions which we don't normally see
This doesn't mean that there isn't a bug, it could just be a rare one
We want tests to be snappy, but also not spend too much time optimising our code for them

Solution:

Use a small delay before clicking the continue button
If we know that the test run is a retry, assume that the error could have been this race condition and increase the delay before clicking continue

I think we could also look for a different element to indicate that the response is still streaming. I think that the "enqueue while moderating" will be active in these cases, but wouldn't be if we used a different element to determine a streaming response

Should we add a data-test property that explicitly states the chat state so that we can test for it?

Should we add a data-test property that explicitly states the chat streaming state so that we can test for it?

I like that. We could expose the streaming state value so we can hook into whatever state we need "natively" rather than using UI as an analog

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


Uncaught Error: The specified module could not be found./?~\AppData\Local\atom\app-1.45.0\res...

[Enter steps to reproduce:]

...
...

Atom: 1.45.0 x64
Electron: 4.2.7
OS: Microsoft Windows 7 Ultimate
Thrown From: Atom Core
Stack Trace
Uncaught Error: The specified module could not be found.
\?\C:\Users\USER\AppData\Local\atom\app-1.45.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
At ELECTRON_ASAR.js:160

Error: The specified module could not be found.
\\?\C:\Users\USER\AppData\Local\atom\app-1.45.0\resources\app.asar.unpacked\node_modules\keyboard-layout\build\Release\keyboard-layout-manager.node
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:1:721712)
    at get_KeyboardLayoutManager (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:14:2784203)
    at get_manager (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:14:2784305)
    at Object.getCurrentKeyboardLayout (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:14:2784431)
    at e.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:14:1083896)
    at KeymapManager.keystrokeForKeyboardEvent (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:11:1251844)
    at KeymapManager.handleKeyboardEvent (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:11:1249829)
    at WindowEventHandler.handleDocumentKeyEvent (~/AppData/Local/atom/app-1.45.0/resources/app/static/<embedded>:11:284496)

Commands

Non-Core Packages
atom-beautify 0.33.4

Why open #20504 and #20505 This is spamming the bug tracker.

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/21285. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


[QUESTION] How can I prevent passing float and bool to integer fields

Integer fields convert bool and float values to integer (1.5, true -> 1, 0) without error raising.

It is a pydantic question imho
Take a look at https://github.com/samuelcolvin/pydantic/issues/360
Or 284
Le mer. 21 août 2019 à 4:00 PM, Vadim Suharnikov<EMAIL_ADDRESS>a écrit :

Integer fields convert bool and float values to integer (1.5, true -> 1,
0) without error raising.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/fastapi/issues/453?email_source=notifications&email_token=AAINSPUDZNDHHTPD2E5USJTQFVDA3A5CNFSM4IOHLQE2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HGQW2IA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAINSPSZIRQ7SWXCBO77UDDQFVDA3ANCNFSM4IOHLQEQ
.

@deterok Look at the pydantic docs on validator. You'd want to add a validator (with pre=True to prevent the cast) that performs the check you want.
It's not clear whether you want to check if the input is of type int, or if it is allowed to be a float as long as when rounded to an int it is equal to itself. Either way, it can be done inside a validator, just raise a ValueError or TypeError if you want to fail validation, and return the original value (or the int-casted version of it) if you want it to pass.

@deterok,
Pydantic already has StrictBool and StrictStr that require passed value to be of str or bool type respectively. You can complement it with StrictInt as follows:
class StrictInt(int):
    @classmethod
    def __get_validators__(cls) -> Generator[Callable[[Any], int], None, None]:
        # yield int_validator
        yield cls.validate

    @classmethod
    def validate(cls, v: Any) -> int:
        if not isinstance(v, int):
            raise IntegerError()
        return v

@samuelcolvin - wouldn't it be nice to have StrictInt as core part of Pydantic (similar to existing StrictBool / StrictStr)? Will you accept PR for that?

@haizaar to that point, I think it probably makes sense to just have a generic way to achieve StrictX. Something like:
T = TypeVar("T")
def strict_type(type_: Type[T]) -> Type[T]:
    class StrictType(type_):
        def __get_validators__(cls) -> Generator[Callable[[Any], int], None, None]:
            # yield int_validator
            yield cls.validate

        @classmethod
        def validate(cls, v: Any) -> int:
            if not isinstance(v, type_):
                raise TypeError(f"{v} was not a {type_}")
            return v
    return StrictType

StrictInt = strict_type(int)

This probably needs a little more massaging to get it to play nice with mypy, etc; maybe it should be a generic type (though this might be more awkward for consumers):
T = TypeVar("T")
class StrictType(Generic[T]):
    type_: ClassVar[Type[T]]
    def __get_validators__(cls) -> Generator[Callable[[Any], int], None, None]:
        # yield int_validator
        yield cls.validate

    @classmethod
    def validate(cls, v: Any) -> int:
        if not isinstance(v, cls.type_):
            raise TypeError(f"Received {type(v)}; required {cls.type_}")
        return v

class StrictInt(StrictType[int]):
    type_ = int

But a generic approach might be better than a proliferation of StrictX classes.

@dmontagu I think your idea is brilliant.
I prefer creator function approach - anyone can create classes for strict instance checking to their liking. I took your example one step forward. Still mypy complains :( Any mypy gurus here to help?
Here is the complete code:
from typing import Any, Callable, Generator, Type, TypeVar

from pydantic import BaseModel

T = TypeVar("T")

def strict_type(type_: Type[T]) -> Type[T]:
    class StrictType(type_):
        @classmethod
        def __get_validators__(cls) -> Generator[Callable[[Any], T], None, None]:
            yield cls.validate

        @classmethod
        def validate(cls, v: Any) -> T:
            if not isinstance(v, type_):
                raise TypeError(f"Expected instance of {type_}, got {type(v)} instead")
            return v
    return StrictType

StrictInt = strict_type(int)

class MyModel(BaseModel):
    foo: StrictInt

print(MyModel(foo=10).json(indent=2))

Any mypy complains about:
/tmp/foo.py:10: error: Variable "type_" is not valid as a type
/tmp/foo.py:10: error: Invalid base class "type_"
/tmp/foo.py:27: error: Variable "union2.StrictInt" is not valid as a type

Yeah, I don’t think mypy lets you do this — it really doesn’t like dynamically generated types (for good reason, I suppose). They can be handled via mypy plugin, but that’s also kind of awkward.

Let's create an issue on pydantic to discuss. I agree a general implementation would be the best solution, we might then create StrictInt and StrictFloat etc. for easy use.
However note the following:
Pydantic is not a validation library, it's parsing library. It makes guarantees about the form of data you get out, rather than making a guarantee about the form of the data you put in. That's how pydantic started, it's decision I'm happy with and it is not going to change. If you use pydantic hoping it will do strict validation you will be disappointed.
Please see https://github.com/samuelcolvin/pydantic/issues/578.

Created https://github.com/samuelcolvin/pydantic/issues/780
@samuelcolvin The difference between validation vs parsing library never occurred to me until now. All I tried is to define a union of str, bool, and int that manages to keep the original data type coming from JSON.
I looks to me that one should be able to define a model in such way that de/ser cycle, i.e. JSON->pydantic model -> JSON is idempotent. This looks like a parsing issue to me. Do you see different?
(I don't want to start validation vs parsing argument, just would like to understand your opinion and approach on this specific matter.)

@deterok yeah, that's how standard Python works. And Pydantic is quite Pythonic, so it uses the same behavior, it tries first to make it work before complaining. You can still use Pydantic's strict versions if you need them.
But for example, have in mind that if you declare something like a path parameter /users/{users_id}/ and declare user_id: int, the URL spec (and HTTP) only define URLs as strings. So that user_id will come as a string, not as an int. There's no way to "validate" it is exactly an int without trying to convert it to an int.
That automatic conversion is what allows declaring more specific data types for Path, Query, Header, and Cookie parameters, that otherwise would be just strings.

I've done StrictInt. Thank you for the help.

Another option might be dacite, if there's a good way to plug it into FastAPI.
https://github.com/konradhalas/dacite


regression: empty pathprefix causes "401 Unauthorized"

What happened:
Starting with ocm-cli 0.16.0 the .ocmconfig files having an empty identity.pathprefix element have stopped working. It started to lead to the error: "failed to authorize: failed to fetch anonymous token: unexpected status from GET request" and "401 Unauthorized".
The error is also seen with the current ocm-cli 0.17.0. The workaround is to remove the empty pathprefix elements from .ocmconfig.
p.s. I've seen this problem with hostname referring to an instance of JFrog Artifactory, but I suppose that it is not Artifactory-specific.
What you expected to happen:
I'd expect the behavior not to change between the versions. An empty pathprefix is superfluous but not invalid.
How to reproduce it (as minimally and precisely as possible):

Create an .ocmconfig with an empty pathprefix containing credentials for a private repository, e.g.:

type: generic.config.ocm.software/v1
configurations:
  - type: credentials.config.ocm.software
    consumers:
      - identity:
          type: OCIRegistry
          hostname: <PRIVATE_REGISTRY_URL>
          pathprefix:
        credentials:
          - type: Credentials
            properties:
              username: <USER_NAME>
              password: <USER_TOKEN>

Supply the created .ocmconfig to an ocm command requiring access to the mentioned repository.
See the error message.
Remove the empty pathprefix and re-execute the command.
Note that now it has succeeded.

Fixed with #1057


add documentation to all the locator annotation and inializable_dependency

add documentation to all the locator annotation and inializable_dependency

fixes https://github.com/Stacked-Org/stacked/issues/984

@ferrarafer Please review

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

v1.4.1
GitHub release

Your semantic-release bot :package::rocket:


tpmtest: fix one more place needing initializing TPM2B size

I found we have new macro to do the TPM2B structure initialization. I would like to make this fix in first, then I will do the macro replacement later.

Jimmy, seems like you ought to fix all of these issues, like I did in my PR #183 and use the new macro.  Just port those changes to tpmtest.

Sure, I will do that soon.

close it and create a new PR.


enhancement: deploy script specific version deployed

Description
currently latest release is deployed.
It is required to deploy specific version

[x] app
[x] service

closing as done.


data request

Hi!
I  can't find 'outdoor_temperatures_noaa.csv' from “Pecan Street Inc.” Does it is from the other web?
I would appreciate it if someone could teach me how to find this data or could send me a copy of it.
My education email is<EMAIL_ADDRESS>Thank you!

I hope you are doing well. I am currently working with the repository on GitHub titled 'MARL-iDR Multi-Agent Reinforcement Learning for Incentive-based Residential Demand Response.' I am in need of the 'outdoor_temperatures_noaa.csv' dataset to replicate this work.
Thank you for your assistance.

Hi!
I can't find 'outdoor_temperatures_noaa.csv' from “Pecan Street Inc.” Does it is from the other web?
I would appreciate it if someone could teach me how to find this data or could send me a copy of it.
My education email<EMAIL_ADDRESS>Thank you!

Hi!
I can't find 'outdoor_temperatures_noaa.csv' from “Pecan Street Inc.” Does it is from the other web?
I would appreciate it if someone could teach me how to find this data or could send me a copy of it.
My education email is<EMAIL_ADDRESS>Thank you!


Sorting by menuPosition query is longer than expected

This is probably a simple answer but I'm curious if sorting by field contentType___node___menuPosition is a query that normally takes a long time?
Here's the query I run:
{
  allWpPost(sort: {order: ASC, fields: contentType___node___menuPosition}) {
    edges {
      node {
        id
        title
      }
    }
  }
}

If I remove the sort by menu position filter the results are instant but with the sort filter added it takes ~30s. While I'm running that query this also appears in the terminal window.
warn  gatsby-source-wordpress  cG9zdF90eXBlOnBvc3Q= contentType was updated, but no data
warn  gatsby-source-wordpress  cG9zdF90eXBlOnBvc3Q= contentType was updated, but no data
warn  gatsby-source-wordpress  cG9zdF90eXBlOnBvc3Q= contentType was updated, but no data
...

The query does return normal but it takes an unusually long time, that's all.

That's really odd 🤔 when I run it it resolves immediately. Are you able to share the /graphql endpoint of the site that has this issue? That'll help me understand what's going on.
Thanks!

I think so. I'm working on another issue now, once that's resolved I'm back on site development. I'll test again and if it's the same I'll send it your way. Thanks!

Hey @emaildano did you get a chance to take a peek again?

@TylerBarnes I just ran this query on the same site I had an issue with and it returned without delay. I'm going to say this is resolved. If I do run into this issue again, I'll send over my log files for review.


Roll Flutter Engine from 63238fe979de to da1493b7e501 (2 revisions)

https://github.com/flutter/engine/compare/63238fe979de...da1493b7e501
2023-03-17<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from 3NfZ5JD61eNz4jJqB... to z32cF6YFs6CvZbY3g... (flutter/engine#40374)
2023-03-17<EMAIL_ADDRESS>Manual roll Dart SDK from 2cd913e06b8c to 05265d5fec65 (2 revisions) (flutter/engine#40369)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from 3NfZ5JD61eNz to z32cF6YFs6Cv
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC aaclarke@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Enzyme Selectors do not recognize SVG attributes

I'm creating SVG components with React as follows:
function IconWrapper({ fillColor, width, height }, children) {
  return (
    <svg fill={fillColor} height={height} viewBox="0 0 24 24" width={width}>
      {children}
    </svg>
  );
}

IconWrapper.propTypes = {
  fillColor: PropTypes.string,
  width: PropTypes.number.isRequired,
  height: PropTypes.number.isRequired,
};

IconWrapper.defaultProps = {
  fillColor: '#222',
};

const ArrowDown = props =>
  IconWrapper(props, [
    <path key="a" d="M7.41 7.84L12 12.42l4.59-4.58L18 9.25l-6 6-6-6z" />,
    <path key="b" d="M0-.75h24v24H0z" fill="none" />,
  ]);

But when I try to test that the svg returned by IconWrapper has certain attributes that are set via props, Enzyme seems to not find them using the attribute selector syntax:
describe('IconWrapper', () => {
  const props = {
    fillColor: '#222',
    width: 40,
    height: 40,
  };
  const children = [<path key="a" />, <path key="b" />];
  const svg = Icons.IconWrapper(props, children);
  const wrapper = mount(<svg />);
  const rendered = wrapper.render();

  // this test passes
  it('renders an svg element', () => {
    expect(wrapper.type()).toEqual('svg');
  });

  // this test fails
  it('assigns a viewBox attribute to the svg element', () => {
    expect(wrapper.find('svg[viewBox="0 0 24 24"]')).toHaveLength(1);
  });

  // this test also fails
  it('sets the svg element fill, width, and height attributes', () => {
    expect(wrapper.find(`svg[fill="${props.fillColor}"]`)).toHaveLength(1);
    expect(wrapper.find(`svg[width="${props.width}"]`)).toHaveLength(1);
    expect(wrapper.find(`svg[bla="${props.height}"]`)).toHaveLength(1);
  });
}

Current behavior
wrapper.find('svg[fill="${props.fillColor}"]') or rendered.find('svg[fill="${props.fillColor}"]')
returns ReactWrapper { length: 0 }
shallow(<svg />).find('svg[viewBox="0 0 24 24"]');
returns ShallowWrapper { length: 0 }
Expected behavior
wrapper.find('svg[fill="${props.fillColor}"]') or rendered.find('svg[fill="${props.fillColor}"]')
returns ReactWrapper { length: 1 }
shallow(<svg />).find('svg[viewBox="0 0 24 24"]');
returns ShallowWrapper { length: 1 }
Your environment
MacOS 10.12.6
NodeJS 6.11.2
API

[x] shallow
[x] mount
[x] render

Version

library
version

Enzyme
3.3.0

React
16.2.0

Jest
22.1.4

Adapter

[x] enzyme-adapter-react-16
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[ ] others ( )

If you .find('svg'), do they show up? If you use .findWhere instead of string selectors, are you able to find them?

@ljharb thanks for responding

If you .find('svg'), do they show up?

Not sure I follow what you mean. If I do wrapper.find('svg') I get ReactWrapper { length: 1 }. Do you mean chain another .find() on, like wrapper.find('svg').find('[fill="#222"]')?
That returns ReactWrapper { length: 0 }

If you use .findWhere instead of string selectors, are you able to find them?

If I do the following using .filterWhere the test passes:
  it('sets the svg element fill, width, and height attributes', () => {
    expect(wrapper.findWhere(n => n.find('[fill="#222"]'))).toHaveLength(1);
    expect(wrapper.findWhere(n => n.find('[width=40]'))).toHaveLength(1);
    expect(wrapper.findWhere(n => n.find('[height=40]'))).toHaveLength(1);
  });

however it still passes if I alter the values of the attributes to be different than what props set them to, e.g.
  it('sets the svg element fill, width, and height attributes', () => {
    expect(wrapper.findWhere(n => n.find('[fill=999]'))).toHaveLength(1);
    expect(wrapper.findWhere(n => n.find('[width="abc"]'))).toHaveLength(1);
    expect(wrapper.findWhere(n => n.find('[height="xyz"]'))).toHaveLength(1);
  });

Does this also happen with shallow?

Yes, if I do: shallow(<svg />).find('svg[viewBox="0 0 24 24"]'); I get ShallowWrapper { length: 0 }
I think my problem relates to https://github.com/airbnb/enzyme/issues/634 in that I'm trying to test for the attributes on the rendered HTML?  It seems that there are varying opinions on whether or not HTML attributes should be tested for and that perhaps unit tests with React should stick to testing for props?

Okay I think I solved it. The problem I was having was that I should have mounted my component as follows:
const wrapper = mount(Icons.IconWrapper(props, children));

instead of doing:
const svg = Icons.IconWrapper(props, children);
const wrapper = mount(<svg />);

Now the following tests pass:
  it('assigns a viewBox attribute to the svg element', () => {
    expect(wrapper.find('[viewBox="0 0 24 24"]')).toHaveLength(1);
  });

  it('sets the svg element fill, width, and height attributes', () => {
    expect(wrapper.find('[fill="#222"]')).toHaveLength(1);
    expect(wrapper.find('[width=40]')).toHaveLength(1);
    expect(wrapper.find('[height=40]')).toHaveLength(1);
  });

Feel free to close as this seems to be my fault implementing mount with usage of a React HOC.


docs: Getting xsticks rotated

In the function _plot, within the section where x-axis labels are configured (# SET X-LABELS), the modification ensures that x-axis labels are rotated by 45 degrees and aligned to the right for better readability. This change enhances the appearance of the plots by providing a more visually appealing and informative representation of the x-axis.
ax.set_xticklabels(ax.get_xticklabels(), rotation=45, ha='right')
This modification affects the formatting of x-axis labels in each subplot, making them more readable and improving the overall presentation of the plots.

Dear Jose,
sorry for not answering that long. I have seen your pull request and checked it. Hard coding that part will break up plot designs for some groups using horizontal labels. It is certainly possible to include that as an option however.
As pumping up the rather old and pieced together MagPy code with such options gets more and more complicated I decided to give it a complete refurbishment. Thus I will not include that in MagPy1.x but will make label
I am currently working on MagPy 2.0.0 which is actually already on the way and which will include rotated labels within its plotting method.  You can checkout the current progress in branch magpy2. All packages of MagPy will be cleaned and updated (some of them are already) and combined with runtime and unittest features for continuous integration. My main focus currently lays on all backend methods, testing/verification and speed issues. The graphical interface XMagPy is last on the list. I am also updating the README frequently in that branch so that you can follow changes easily. I try to incorporate all suggestions into this new version. It will need a while however until that part reaches a publication stage.
Best wishes,
Roman


Pulumi generates output on stderr / OnStandardError callback with update warning

What happened?
I'm using the pulumi dotnet automation API to deploy infrastructure on demand. I'm using the callbacks OnStandardOutput and OnStandardError on the UpAsync method:
await stack.UpAsync(new UpOptions
{
	OnStandardError = error => Logger.Warning("Pulumi up error on VirtualMachineId {VirtualMachineId} Error: {Error}", vm.Id, error),
	OnStandardOutput = msg => Logger.Debug("Pulumi up output on VirtualMachineId {VirtualMachineId} Message: {Message}", vm.Id, msg)
}, token);

I'm getting output on the OnStandardError when pulumi is outdated:
[10:11:39 WRN] Pulumi up error on VirtualMachineId 6f5cecf3-40e5-464e-92f0-5befd240f584 Error: warning: A new version of Pulumi is available. To upgrade from version '3.60.0' to '3.61.0', run  => Application.CloudRendering.CloudProvider
[10:11:39 WRN] Pulumi up error on VirtualMachineId 6f5cecf3-40e5-464e-92f0-5befd240f584 Error:    $ brew update && brew upgrade pulumi => Application.CloudRendering.CloudProvider
[10:11:39 WRN] Pulumi up error on VirtualMachineId 6f5cecf3-40e5-464e-92f0-5befd240f584 Error: or visit https://pulumi.com/docs/reference/install/ for manual instructions and release notes. => Application.CloudRendering.CloudProvider

Expected Behavior
Add option to suppress update warnings or send them on stdout instead of stderr as it is not an error.
Steps to reproduce
Use the snipped above, add a stack and adjust logging.
Output of pulumi about
CLI
Version      3.60.0
Go Version   go1.20.2
Go Compiler  gc
Host
OS       darwin
Version  13.3
Arch     arm64
Backend
Name           pulumi.com
URL            xxx
User           xxx
Organizations  xxx
Pulumi locates its logs in xxx by default
warning: Failed to read project: no Pulumi.yaml project file found (searching upwards from xxx). If you have not created a project yet, use pulumi new to do so: no project file found
warning: Failed to get information about the current stack: no Pulumi.yaml project file found (searching upwards from xxx). If you have not created a project yet, use pulumi new to do so: no project file found
warning: A new version of Pulumi is available. To upgrade from version '3.60.0' to '3.61.0', run
$ brew update && brew upgrade pulumi
or visit https://pulumi.com/docs/reference/install/ for manual instructions and release notes.
Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hey again. Thanks for filing. There's an environment variable you can set via export PULUMI_SKIP_UPDATE_CHECK=1 which should disable the warning.

Thanks for the very fast answer. This indeed seems to work.
Can I access this somehow through the C# SDK?

Can I access this somehow through the C# SDK?

Not currently. You can set it as the configuration to UpAsync.
We should consider setting this by default for Automation API to keep the stderr output clean. I've turned this into an kind/enhancement and updated the title to reflect that.


MI250X (gfx90a) not available as cuda device on pytorch

I have MI250X system

with rocm5.4 installed

I installed pytorch using following command
pip3 install --pre torch torchvision torchaudio --extra-index-url https://download.pytorch.org/whl/nightly/rocm**5.4**/
notice 5.4 at the end. The installation completed successfully.
But when I use torch.cuda.is_available(), it return False

what am I missing?

I have the same problem on my 7900 XTX as well.

I have MI250X system  with rocm5.4 installed  I installed pytorch using following command pip3 install --pre torch torchvision torchaudio --extra-index-url https://download.pytorch.org/whl/nightly/rocm**5.4**/ notice 5.4 at the end. The installation completed successfully.
But when I use torch.cuda.is_available(), it return False
what am I missing?

hello, bro!
I got a AMD MI250X GPU card recently. Obviously it was a part from HPE cray EX235a system.
But I got problem in using it now in ubuntu.
So, do you have drivers or BIOS that can make my card alive? It is writtern D65201-OB Rev13 on GPU.

@nishshah0 Do you still need assistance with this issue? Thanks!

Hi @nishshah0, if you are still experiencing this issue after following the steps here, Installing PyTorch on ROCm, please re-open this ticket. Thanks!

Yes this problem still exists. I first noticed this with openwebui, i have two different AMD cards on two different Linux systems that are running two different version of PyTorch that would fine with diffusion. it seems to be local to Whisper.
 torch.cuda.is_available() is expected with ROCm

always try sudo usermod -a -G render,video $USER:$GROUP for each user using the audio


[Feature][Spark] Reading speed limit of source connector

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Reading speed limit of source connector
references
https://github.com/apache/seatunnel/pull/5502
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

You can assign it to me. Thank you


Add "client" , "server and client" tags on modrinth

Please add "client" , "server and client" tags on modrinth . When this mod is added to mrpacks for download on a small community server, prism launcher automatically treats this as a server only mod and wont download it on client.
This causes sihywtcamd-extensions to throw an error saying sihywtcamd v1.11 is required
PS : Love this mod as it adds a challenge for long time players

Hi, thanks for the report! I added the "server and client" tag on Modrinth, it should be enough I guess?
But the real issue is that the extension mod shouldn't require the base mod to avoid this kind of issue. I will keep the issue open as long as I don't have updated the extension mod to not require it

I updated the mod and the extension. The extension mod doesn't require the base mod anymore. The base mod is only required on the server, if you don't have it on the client, everything will still work as expected :)


Out of memory when playing video and performing

https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=41783.

/cc @a-siva @rmacnak-google

This is being worked on by a combination of Fuchsia people and co-ordinated on the Flutter side by @cbracken.

Just spoke to @cbracken, this is still being worked on.

@cbracken any updates?

No longer seeing this issue (also no longer assigned to Flutter).


Error 500 - SHH KEY integration fork

I am using FORK as GUIO client of GITEA. the option REQUIRE_SIGNIN_VIEW IS TRUE. we generate de SSH KEY with fork. We wi add the key en gitea we have a error cod '500'.
Can you help me. How can I integrate FORK with gitea when gitea requiere password to see any page.

Gitea version (or commit ref):
Git version:  last
Operating system: Windows 10
Database (use [x]):

[ ] PostgreSQL
[ x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ x] No
[ ] Not relevant

Log gist:

Description
code error 500
...
Screenshots

Check gitea.log for errors


POB doesnt count more projectiles added to molten strike on the DPS count.

  Is this a bug? Why doesnt it show damage from adding more projectiles?
Thanks :)

Damage is per ball. So you gotta do DMG multiplied by the number of balls.
BUT keep in mind that not all will ever hit. So best you can do is damage * balls / 2
This will be roughly a viable number.

Alright, thanks. Ive also noticed that multistrike doesnt show a dps increase. Are all these things possible to implement or will they not be implemented in the future? Just thought I should report it in for u guys :)

I don't think he will implement this as it is impossible to create a real DPS number as it matters on how many balls hit. If you're surrounded by masses of mobs it MIGHT be possible that all balls hit once. If you're unlucky on a boss you might hit only with one single ball.
What a nice thing would be is the following:
Count the projectiles (no matter what skill) and offer either the settings to set a % of balls hitting, so we can tell POB "50% of the projectiles hit" for example or offer just 4 premade infos of 100%, 75%, 50%, 25%

Yeah, sounds like a good option. And the multistrike needs to be fixed as
well, it doesnt add dps.
On Sat, May 5, 2018 at 12:48 AM, Exportforce<EMAIL_ADDRESS>wrote:

I don't think he will implement this as it is impossible to create a real
DPS number as it matters on how many balls hit. If you're surrounded by
masses of mobs it MIGHT be possible that all balls hit once. If you're
unlucky on a boss you might hit only with one single ball.
What a nice thing would be is the following:
Count the projectiles (no matter what skill) and offer either the settings
to set a % of balls hitting, so we can tell POB "50% of the projectiles
hit" for example or offer just 4 premade infos of 100%, 75%, 50%, 25%
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Openarl/PathOfBuilding/issues/981#issuecomment-386754008,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab0tmBL6FGUVT-g5xepn4Hr4HQX9o-Xyks5tvNq0gaJpZM4Ty9nv
.

That's because multistrike is only increasing the number of balls again.  Multistrike works for the melee portion of the attack.

There should be an average estimation on the total damage of it, with the
calculations of the average numbers of balls hitting depending on if its
white mobs / standard boss / shaper.
On Thu, May 10, 2018 at 2:07 PM, Trevor Lund<EMAIL_ADDRESS>wrote:

That's because multistrike is only increasing the number of balls again.
Multistrike works for the melee portion of the attack.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Openarl/PathOfBuilding/issues/981#issuecomment-388034819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab0tmBd4TH5OxYkyo1hyJwGTS0FYUaFkks5txC1sgaJpZM4Ty9nv
.

https://github.com/Openarl/PathOfBuilding/issues/241
This is a limitation that applies to a lot of skills that have multiple parts; in this case, the Magma Balls part is a projectile skill, so its attack speed must be affected by projectile attack modifiers, just as its damage is. The attack/cast rate for secondary skill parts should generally be ignored anyway, or at least taken with a grain of salt, as the skill mechanics often render it inaccurate; it's only shown at all because it is sometimes correct.

Thanks.
On Thu, May 31, 2018 at 2:15 PM, bdg0<EMAIL_ADDRESS>wrote:

#241 https://github.com/Openarl/PathOfBuilding/issues/241
This is a limitation that applies to a lot of skills that have multiple
parts; in this case, the Magma Balls part is a projectile skill, so its
attack speed must be affected by projectile attack modifiers, just as its
damage is. The attack/cast rate for secondary skill parts should generally
be ignored anyway, or at least taken with a grain of salt, as the skill
mechanics often render it inaccurate; it's only shown at all because it is
sometimes correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Openarl/PathOfBuilding/issues/981#issuecomment-393511044,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ab0tmOYjLTpZY9TVs_1DTUhTLfZ1gKftks5t397sgaJpZM4Ty9nv
.


Welcome Microsoft to MicroProfile community

MicroProfile community should create a blog to welcome Microsoft to the community and link to this blog
See an example of this in Jakarta EE.
The blog can then be tweeted.

This would be great! Thanks @Emily-Jiang!

MicroProfile community should create a blog to welcome Microsoft to the community and link to this blog
See an example of this in Jakarta EE.
The blog can then be tweeted.

hola @Emily-Jiang,
I have untagged @rstjames from this ticket.

The member can follow AJUG and GSJUG's blog https://microprofile.io/category/community/mpwg-onboarding-members/
--
Why?
B/c MPWG doesn't have a hired writer nor copy/editor and cannot speak for its members on why joining MP was right for them.
-- I hope this helps with current member and future advice during onboarding.
The guidelines for onboarding are here

Unfortunately, it would not really be appropriate for me to add anything on behalf of Microsoft on top of Julia's announcement and it would not be appropriate to syndicate Julia's announcement. This can be closed if needed. However, perhaps a suitable solution is to simply tweet Julia's announcement from the official Twitter handle?

https://azure.microsoft.com/en-us/blog/microsoft-joins-jakarta-ee-and-microprofile-working-groups-at-eclipse-foundation/
@rstjames, thank you in advance for helping close this ticket with the below action.

MP Social media push of MSFT's formal letter.

recommended Text or similar -  @m-reza-rahman, you are most welcome to improve the message.
_@Microsoft joins the @JakartaEE and @MicroProfileIO Working Groups. "We believe our experience running @java workloads in the cloud will be valuable to the working groups,..." by Julia Liuson.
Read more [Link]_

I think this is perfect. Much appreciated!

I think this can be closed now, @rstjames?

https://twitter.com/MicroProfileIO/status/1591098778253074432?s=20&t=Fkt9kcCZw3wIEXyQkQLSfg


Telos testnet not syncing

I was busy syncing the Telos testnet from block 104880000, but then it stopped halfway through with the following error.
eosio-contract-api-filler      | 2022-05-20T04:51:20.930Z [PID:39] [info] : Launching deserialization worker...
eosio-contract-api-filler      | 2022-05-20T04:51:21.242Z [PID:39] [warn] : Could not find ABI for atomicmarket in cache, so requesting it...
eosio-contract-api-filler      | 2022-05-20T04:51:22.459Z [PID:39] [warn] : Could not find ABI for atomicassets in cache, so requesting it...
eosio-contract-api-postgres    | 2022-05-20 04:51:23.852 UTC [35] ERROR:  insert or update on table "atomicmarket_auctions" violates foreign key constraint "atomicmarket_auctions_maker_marketplace_fkey"
eosio-contract-api-postgres    | 2022-05-20 04:51:23.852 UTC [35] DETAIL:  Key (market_contract, maker_marketplace)=(atomicmarket, ) is not present in table "atomicmarket_marketplaces".
eosio-contract-api-postgres    | 2022-05-20 04:51:23.852 UTC [35] STATEMENT:  COMMIT
eosio-contract-api-filler      | 2022-05-20T04:51:23.853Z [PID:39] [error] : Failed to execute SQL query  {"queryText":"COMMIT","values":[],"error":{"length":386,"name":"error","severity":"ERROR","code":"23503","detail":"Key (market_contract, maker_marketplace)=(atomicmarket, ) is not present in table \"atomicmarket_marketplaces\".","schema":"public","table":"atomicmarket_auctions","constraint":"atomicmarket_auctions_maker_marketplace_fkey","file":"ri_triggers.c","line":"2465","routine":"ri_ReportViolation"}}
eosio-contract-api-filler      | 2022-05-20T04:51:23.853Z [PID:39] [error] : Error occurred while executing block range from #109316981 to 109316990
eosio-contract-api-postgres    | 2022-05-20 04:51:23.854 UTC [35] WARNING:  there is no transaction in progress
eosio-contract-api-filler      | 2022-05-20T04:51:23.855Z [PID:39] [error] : Consumer queue stopped due to an error at #109316990 Release called on client which has already been released to the pool. {"stack":"Error: Release called on client which has already been released to the pool.\n    at throwOnDoubleRelease (/home/application/app/node_modules/pg-pool/index.js:27:9)\n    at Client.release (/home/application/app/node_modules/pg-pool/index.js:296:9)\n    at ContractDBTransaction.abort (/home/application/app/build/filler/database.js:485:25)\n    at processTicksAndRejections (node:internal/process/task_queues:96:5)\n    at async StateReceiver.process (/home/application/app/build/filler/receiver.js:175:17)\n    at async /home/application/app/build/filler/receiver.js:90:17\n    at async run (/home/application/app/node_modules/p-queue/dist/index.js:163:29)"}
eosio-contract-api-filler      | 2022-05-20T04:51:30.769Z [PID:39] [warn] : Reader atomic-1 - No blocks processed [DS:0|SH:40|JQ:0] - Stopping in 3590 seconds

Any ideas?

Yes need to start syncing at the block where atomicassets was deployed


Unable to build

Description:

Unable to build using HEMTT and provided bat, produces error in picture

Resolved, didn't know it needed to be built from repo folder


Server::createConnection breaks SRP

Shouldn't a ConnectionFactory be passed into Server::__construct instead of having a hardcoded classname in createConnection?
According to the single responsibility principle usage of an object should be separate from its construction right?

I think this makes sense :+1: Out of curiosity, do you have any practical use case for this?

@clue Yes I do, since it is hard to get traction for changing the ConnectionInterface (since not all connections have a port), it makes sense to have different connection interfaces and different underlying implementations. This would allow you to use such a different connection implementation without changing the server.

Ping @SamMousa, what do you think, is this and #47 still relevant now that #65 is in? :+1:

I believe this has been addressed via #65. Closing this due to a lack of feedback. If you feel this problem persists, please comment on this and we can reopen this :+1:


Initial review and release to NPM of FlowForge theme for Node-RED

Current Behavior
Theme is now ready for review along side the following 2 PRs...

https://github.com/flowforge/flowforge/pull/613
https://github.com/flowforge/flowforge-nr-launcher/pull/37

TO DO...

SRC code review
Test run
Publish

Expected Behavior
No response
Steps To Reproduce
No response
Environment

FlowForge version: 0.7 (target)

Having used the theme for a couple days, the only tweak I'd suggest is the menu item separator:

I'd suggest sticking to the original colour that's a bit more muted against the dark/light menu backgrounds.

Just trying to tie up issues here, is this the delivery of flowforge/flowforge#214

Related - in that there can be no theme choice without a theme :).

I'd suggest sticking to the original colour that's a bit more muted against the dark/light menu backgrounds.

@knolleary
Addressed in #10 - please review & merge if happy.

Looks great.
I think we're good to publish.


New micro:bit firmware

Hi @venkateshpotluri,
We've turned off some of the services exposed by the micro:bit. Hopefully this will put less strain on the software stack!
Let us know!
microbit-jacdac-microbit.zip

@jamesadevine  I tested the sound level with this firmware and I think the data stops streaming after a few seconds. I tested the same code with the version of the firmware I was using earlier and this does not happen. I tried looking at the logs but there are too many log statements for me to identify a specific cause. Do you have any suggestions for me to send debugging info that would help?

CC @pelikhan

How do I repro this?

James can you add the makecode project somewhere in the repo?

@pelikhan https://github.com/make4all/sonification/commit/11f117b1606d304171d10aaf90b0030e3adbf0ce

@pelikhan  repro steps:

start the development server with yarn start
go to localhost:3000/dashboard.
connect Micro:Bit with the linked firmware installed, and connect.
add a note handler to the sound level.
keep making some sounds for a few seconds.
You will notice that the value stops changing after some time (after about 30-60 seconds).

I've added instructions to rebuild the firmware at https://github.com/make4all/sonification/blob/main/makecode-microbit/README.md

@venkateshpotluri try again with the new firmware and build of jacdac-ts. I'll leave my device to run overnight to see

@pelikhan  I tried the following and the issue still persists.
trial 1

installed the makecode project by running npm install -g makecode
navigated to the makecode-microbit folder.
executed mkc -d

trial 2.
steps 1-3 above, and then copied in the firmware linked in this issue.

The firmware linked is outdated. I have removed it from the comment to avoid any kind of confusion.

Make sure to run npm install -g -u makecode to update any previously installed version of the command line tool (-u forces the update)

I will try to repro again. Are you connecting any Jacdac devices to your micro:bit?

I followed the steps you suggested and still run into the same issue. I do not have any jacdac devices connected: I just have the Jacdapter connected to the Micro:bit.

I tried again with the jacdapter removed and the issue persists.

Venkatesh, incase the instructions were unclear, please load the following hex file onto your micro:bit:
mbcodal-binary.zip

Peli and I had a debugging meeting, and my understanding is that we were able to resolve this with the latest firmware that Peli shared with me. This issue appears to be resolved, @pelikhan  we can close this if you feel that this is fixed as well.


added Attention Free Transformer, and update README.md

What
Added the paper An Attention Free Transformer.
Compared to the original transformer, it is lightweight, fast, and as accurate or more accurate, and beginners who understand the structure of attention with attention is all you need will find it easy to understand and useful.
I complied with the specified color code.

Hey thanks for the contribution, it looks good!!! will look forward to more of these


Component: Press Release Page Block

Initial spec based on Figma design & notes

Still some more work to do on this.


Bug 1881222: Annotate CSVs with the properties used during dependency resolution.

This ensures that property information is not lost after an operator
is installed, and that the properties of an operator are the same
before and after installation. Preserving properties also allows
installed operators to satisfy dependencies on properties that cannot
be inferred from a ClusterServiceVersion spec, and is a step toward
unifying installed and available operators from the perspective of
resolution.
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

/lgtm

/bugzilla refresh

@benluddy: The following test failed, say /retest to rerun all failed tests:
Test name 	Commit 	Details 	Rerun command
ci/prow/e2e-aws-olm 	bcf0ace link /test e2e-aws-olm
Full PR test history. Your PR dashboard.

/hold
This looks like a problem with the index image fixture. It was last pushed July 22, and it's exhibiting a bug in its ListBundles response that was fixed a month later in https://github.com/operator-framework/operator-registry/pull/417.

/hold cancel


Payload with process array fails if no id specified

If no id is specified in the input payload, then we run https://github.com/cirrus-geo/cirrus-lib/blob/main/src/cirrus/lib/process_payload.py#L154. However, if we are also using a process array for workflow chaining, the line assigning the id fails with the following traceback:
[ERROR] TypeError: list indices must be integers or slices, not str
Traceback (most recent call last):
  File "/var/task/lambda_function.py", line 60, in lambda_handler
    payloads.append(ProcessPayload(payload, update=True))
  File "/var/task/cirrus/lib/process_payload.py", line 59, in __init__
    self.update()
  File "/var/task/cirrus/lib/process_payload.py", line 155, in update
    self['id'] = f"{collections_str}/workflow-{self.process['workflow']}/{items_str}"

It should be pretty easy to pop the id off of the test payload for one of the workflow chaining tests to duplicate this issue and facilitate debugging.

The error here was caused by an invalid input payload that had an array of process definitions in the first index of the process array. While this could be considered user error with no required changes, I do think having a more descriptive errors would be helpful in two main cases:

in this case we should have a specific error message stating that branching is not supported in the first step of a workflow chain
in the general case of an invalid payload we should try to have an invalid payload exception (probably using the from keyword to show the specific exception text.

I don't think we can necessarily enforce the latter throughout all the code, but it could be useful in the class constructor in particular.


[DR-1591] Create update cdn script

Background
Create script to upload build resources to akamai cdn
Changes done
<Enumerate changes done, that are not obvious from the contents of the code. Explain only general approaches to the changes. Explain design decisions if any.>
Pending to be done
Integration with Jenkins
Notes
It is possible that we need to do some updates on the script when we integrate DR-1588 and DR-1592

Hi @cristiangiagante, nice script! I have added some comments.
Regarding the PR description, it is not clear to me, I want to confirm if we are in the same page or not.
After this PR you will create another (or more than one) with Docker image definition, CI, Changes in README file indicating how to use it, etc, right?

I forgot to update the README, I will do in the next pr update


Use geoJSON?

My location data is generally saved in geoJSON format. I believe I can convert this to GPX relatively easily, but before I do so I wondered whether you had any plan to allow the use of a linestring extracted from geoJSON.
Thanks.

I'm converting from geoJSON myself, so closing, but I have other issues :)


Guest user available for e2e test in production RabbitMQ Clusters envs

Changes

🧹 Fixed vhost test case when the available rabbitmq cluster is in producction mode

/kind cleanup

/cherry-pick release-1.6 release-1.7

/cherry-pick release-1.6


Minor versions shouldn't make incompatible API changes

Hello, Thanks for the great package.
In the readme file, you mentioned moving from a minor version to another cause changing traits used which we can say are incompatible API changes that shouldn't be done in a minor version changing please check semantic versioning.

@ScSherifTarek What changes are you referring to?

@joveice Switching from 1.1.x to 1.2.x requires replacing some traits with another.

@ScSherifTarek You have a point there.

@joveice Just to know for future releases :smile:

Back then, very few people installed it and i made the change way before people were able to use it in production. But yes, breaking changes require new major. 😅

Now no need for this issue


Create Deno.gitignore

Reasons for making this change:

I started a new project in Deno, and not found its entry here; so I'm proposing a first version
Deno is not much dependent on any folder structure, this makes its .gitignore for start project very slim.
TODO
Links to documentation supporting these rule changes:
Documentation Link
If this is a new template:

Link to application or project’s homepage:  Deno.Land

⚠ Duplicate of #4101

So closing in decorrence of duplication.


4k BD Rip - Failing with Error "The Volume Key is Unknown for this Disc - video can't be decrypted"

As per the title, having issues with 4k discs that I was able to decrypt with my drive previously while connected directly to PC - moved it to external enclosure and been having great success with regular BR discs - However for the few 4k Discs that I have, it is failing.
Happy to share more info as needed.

This is not really an issue, but this is how MakeMKV works: it needs a key to decrypt the disc.  It seems that no key is available yet for your disc.
More details on the MakeMKV forum:
https://forum.makemkv.com/forum/viewtopic.php?t=24711
https://forum.makemkv.com/forum/viewtopic.php?f=12&t=16883
If the container has access to the internet, you did not disabled internet access in MakeMKV settings and you don't see other errors, you probably need to send the dump.

As I mentioned, i was able to decrypt those previously with regular
makemkv, so keys are def. available.
I'm checking the internet connection from the container next, setting is
enabled in makemkv but did not check if container has proper connectivity
yet.
On Tue, Feb 14, 2023, 18:30 Jocelyn Le Sage @.***>
wrote:

This is not really an issue, but this is how MakeMKV works: it needs a key
to decrypt the disc. It seems that no key is available yet for your disc.
More details on the MakeMKV forum:
https://forum.makemkv.com/forum/viewtopic.php?t=24711
https://forum.makemkv.com/forum/viewtopic.php?f=12&t=16883
If the container has access to the internet, you did not disabled internet
access in MakeMKV settings and you don't see other errors, you probably
need to send the dump.
—
Reply to this email directly, view it on GitHub
https://github.com/jlesage/docker-makemkv/issues/168#issuecomment-1430119530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG6BYK2MPQAAMA7FMHLZKHLWXO6MVANCNFSM6AAAAAAU3YQP4A
.
You are receiving this because you authored the thread.Message ID:
@.***>

How is the external enclosure connected? USB?

MakeMKV should clearly tells you if keys failed to be downloaded.

It is an ICY BOX IB-550STU3S connected to a QNAP TS464 via USB.
With this config:
DVD Rip: OK
BR: OK
4k BR: Fail With the error mentioned.
Moved Enclosure to my PC, Was able to open disc w/o Issues:
[image: image.png]
With container:
[image: image.png]
I reviewed the output and indeed I don't see any mention of keys failing;
Will further TS on this later and provide more as I move along. If there
are any specifics or if someone else could confirm 4K BR rips should be
fine, would be great :)
On Tue, Feb 14, 2023 at 6:44 PM Adam White @.***> wrote:

How is the external enclosure connected? USB?
—
Reply to this email directly, view it on GitHub
https://github.com/jlesage/docker-makemkv/issues/168#issuecomment-1430136494,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG6BYK5AGIPHLJBMZY4PES3WXO77ZANCNFSM6AAAAAAU3YQP4A
.
You are receiving this because you authored the thread.Message ID:
@.***>

@alexandrenobrega1, the images didn't attach because of the email reply. Try putting them directly on GitHub.

Apologies - images as follows:

the error code means there is no decryption key for the disc. do you have connect to internet disabled?

also upgrade the container to the latest version

from your screenshot above, it shows MakeMKV Version 1.17.2 and the current container is Version 1.17.3

ok, not sure why you are posting screenshots from 2 different machines together. post the screenshot from the container with the failing to decrypt message. the full screenshot

@C4Wiz posting both screenshots to rule out the "this is a new disc and we don't have the keys to decrypt it yet" theory. As you can see, outside of the container, the disc is decrypted just fine.
Screenshot from Container's logs where I tried to open the same disc twice:

yea, there is no key for the disc, it clearly states it right below the hyperlink.
submit the dump and check again in a few days.
if you have sucesfully opened this EXACT PHYSICAL DISC on the other machine, then your docker container is not reaching the server to download the keys

just a fyi 2 of the same disc could need a different key

connect the usb enclosure to your pc and open the disc with makemkv on your pc, if it opens it's your container not reaching the server, if it doesnt then there is no key for it yet, submit the dump and wait

Ok, well im telling you the docker IS NOT reaching the server to download keys.
Copy your private_data.tar and sdf.bin from your pc to your docker and see if it works. If it does it proves my point.

Im running the same docker on my unraid server and it works fine. So it’s not the container

@C4Wiz, did you remove the downloaded data and confirmed that it was re-downloaded ?

i do not have a encrypted disc to test with at the moment

I'll confirm this issue.
Using identical 4k bluray that i've previously ripped on the exact host machine receiving identical behavior to alexandre in the opening upon trying to read the disk with the docker image.
As suggested above verified that internet access is reported as enabled, container can ping both <IP_ADDRESS> and google.com resolves as host name.
Attempting to open the 4k bluray in the container results in the following output to the information window:

Happy to provide more information or make attempts at diagnosis if it will assist.

Can you try (the older) version 1.22.2 of the Docker image ?

sure. updated my docker-compose image to pull that tag, restarted container and.....

Success.
I notice that this info screen mentions downloading latest SDF on startup where the prior version did not, and on opening the disk and loading the content hash table it mentions downloading the HK so I suspect something in the new image is preventing that?
Let me know if you would like further diagnosis steps to be taken, or beta images to be tried to test fixes.
-Accalia

Thanks for the info.
Could you try the Docker image tag issue168 ?

@jlesage thank you so much for actually looking into the issue here and @AccaliaDeElementia for also actually double-checking :)
I just ran tests with container tagged as issue168 and it was a great success:

I'll also confirm. tag issue168 is also able to read and decrypt 4k blurays.
I cleared config and storage volumes to ensure no false positives from
already having the keys downloaded and it worked. Thanks for looking into
this and for getting a fix out so quickly.
On Wed, Feb 15, 2023 at 2:24 AM alexandrenobrega1 @.***>
wrote:

@jlesage https://github.com/jlesage thank you so much for actually
looking into the issue here and @AccaliaDeElementia
https://github.com/AccaliaDeElementia for also actually double-checking
:)
I just ran tests with container tagged as issue168 and it was a great
success:
[image: image]
https://user-images.githubusercontent.com/29105195/218959748-593e18e8-ab5c-4ac1-a788-34fb7034eeb8.png
—
Reply to this email directly, view it on GitHub
https://github.com/jlesage/docker-makemkv/issues/168#issuecomment-1430870413,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGKQJEH73OJJCXEPGIY62DWXSABFANCNFSM6AAAAAAU3YQP4A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for confirming.  I will prepare and publish an official image with the fix.

Official image now available!

This issue seems to have popped up for me in the latest version. My logs appear as below:
001011:0000 Using LibreDrive mode (v06.3 id=F046CEB2F3AB)
003007:0000 Using direct disc access mode
005085:0000 Loaded content hash table, will verify integrity of M2TS files.
DEBUG: Code<PHONE_NUMBER> at v,`=g{&=n3FORp$%:121265499
001003:0020 DEBUG: Code 0 at Dba&0i#r_n[Z|</K]|d_@P:29393631
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:29399956
003332:0000 Saved AACS dump file as file:///config/data/MKB20_v77_MARVEL_STUDIOS'_LOKI_SEASON_1_DISC_2_2A52.tgz
003303:0000 The volume key is unknown for this disc - video can't be decrypted
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213132721
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213131341
005010:0000 Failed to open disc

The network setting is set and I'm fairly positive that the server is reachable as I can use MakeMKV on my own machine. I don't know if this might cause this, but this is running on a Raspberry Pi CM4.

This issue seems to have popped up for me in the latest version. My logs appear as below:
001011:0000 Using LibreDrive mode (v06.3 id=F046CEB2F3AB)
003007:0000 Using direct disc access mode
005085:0000 Loaded content hash table, will verify integrity of M2TS files.
DEBUG: Code<PHONE_NUMBER> at v,`=g{&=n3FORp$%:121265499
001003:0020 DEBUG: Code 0 at Dba&0i#r_n[Z|</K]|d_@P:29393631
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:29399956
003332:0000 Saved AACS dump file as file:///config/data/MKB20_v77_MARVEL_STUDIOS'_LOKI_SEASON_1_DISC_2_2A52.tgz
003303:0000 The volume key is unknown for this disc - video can't be decrypted
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213132721
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213131341
005010:0000 Failed to open disc

The network setting is set and I'm fairly positive that the server is reachable as I can use MakeMKV on my own machine. I don't know if this might cause this, but this is running on a Raspberry Pi CM4.

It means makemkv does not have a decryption key for the disc, submit the dump and wait. A quick search on the makemkv forum would have answered this question

I’m well aware of what the error means. I posted in this issue because I am having the exact same issue as the original issue.
I am able to rip this UHD disc on my main laptop if I take the drive to that machine as the hash keys are available for download. However the docker version is not downloading the already available hash keys. If you would like manual proof I can provide logs but I believe the error causing this that was fixed in February appears to be present again, at least in the ARM version of the docker container.

The following log was from the Docker machine. I tried loading the disc and it failed. Then I SSHed in the HK file from my computer where I had successfully loaded the blu ray 5 minutes earlier and the docker container was successfully able to load the disc.
Debug log started at Wed Oct 11 15:29:03 2023 , written by MakeMKV v1.17.5 linux(arm64-release)
001005:0000 MakeMKV v1.17.5 linux(arm64-release) started
001004:0000 Debug logging enabled, log will be saved as file:///config/MakeMKV_log.txt
Using 262272KB for read cache.
001003:0020 DEBUG: Code 0 at Dba&0i#r_n[Z|</K]|d_@P:29393631
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:213144750
SDF  v097: BUFFALO_Optical_Drive_BU10_211705301728_MOAKACA4523
SDF  v097: BUFFALO_Optical_Drive_BU10_211705301728_MOAKACA4523
001003:0020 DEBUG: Code 0 at Dba&0i#r_n[Z|</K]|d_@P:29393631
SDF  v097: BUFFALO_Optical_Drive_BU10_211705301728_MOAKACA4523
001011:0000 Using LibreDrive mode (v06.3 id=F046CEB2F3AB)
003007:0000 Using direct disc access mode
005085:0000 Loaded content hash table, will verify integrity of M2TS files.
DEBUG: Code<PHONE_NUMBER> at v,`=g{&=n3FORp$%:121265499
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:29399956
003332:0000 Saved AACS dump file as file:///config/data/MKB20_v77_MARVEL_STUDIOS'_LOKI_SEASON_1_DISC_2_2A52.tgz
003303:0000 The volume key is unknown for this disc - video can't be decrypted
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213132721
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213131341
005010:0000 Failed to open disc
001011:0000 Using LibreDrive mode (v06.3 id=F046CEB2F3AB)
003007:0000 Using direct disc access mode
005085:0000 Loaded content hash table, will verify integrity of M2TS files.
DEBUG: Code<PHONE_NUMBER> at v,`=g{&=n3FORp$%:121265499
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:29399956
Found java version=<IP_ADDRESS>2 path=/usr/lib/jvm/java-1.8-openjdk/bin/java
Found java version=<IP_ADDRESS>2 path=/usr/bin/java
003344:0000 Using Java runtime from /usr/lib/jvm/java-1.8-openjdk/bin/java
DISCID=91AB806257021684C68017D7F65F9489CC8FFFFE
003025:0000 Title #00152.mpls has length of 5 seconds which is less than minimum title length of 120 seconds and w...

Then, I renamed the new HK file to force a new download, and the docker container did not download and therefore failed.
SDF  v097: BUFFALO_Optical_Drive_BU10_211705301728_MOAKACA4523
001011:0000 Using LibreDrive mode (v06.3 id=F046CEB2F3AB)
003007:0000 Using direct disc access mode
005085:0000 Loaded content hash table, will verify integrity of M2TS files.
DEBUG: Code<PHONE_NUMBER> at v,`=g{&=n3FORp$%:121265499
001003:0020 DEBUG: Code 0 at v,`=g{&=n3FORp$%:29399956
003332:0000 Saved AACS dump file as file:///config/data/MKB20_v77_MARVEL_STUDIOS'_LOKI_SEASON_1_DISC_2_2A52.tgz
003303:0000 The volume key is unknown for this disc - video can't be decrypted
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213132721
001003:0020 DEBUG: Code 0 at ,pDYX3_ZfOf\9%`k:213131341
005010:0000 Failed to open disc

Finally, here is the lookup for the HK download location, showing that the server can be reached from the docker machine.
Non-authoritative answer:
Name:	hkdata.fairuse.org
Address: <IP_ADDRESS>

So, again, this appears to be a very similar issue as the original issue that appears to be popping up again in the new version.

Is it working if you use the previous version ?

I actually cannot as this is on an arm64 system and the prior versions of makemkv bundled with the docker version do not work.

I just want to state that this problem is still occurring. Every so often I have to move my drive to a regular PC, open a disc (usually a 4k disc) to download the new keys, and then manually transfer the key file to the docker container on my ripping computer. And then move the drive back to the ripping / docker computer. Newer versions of the docker container installed in the last month have not fixed this issue either.

I just noticed the same issue with a disc I've ripped before:

Strangely, there's no error here like I'd typically see on my desktop.
I just set up a Raspberry Pi with this container today at version 1.17.7.
I was benchmarking the rip speed of discs on a Raspberry Pi and noticed that this UHD Blu-ray couldn't be read.
Troubleshooting
My container has Internet access, but it can't access that URL because something blocks it. Either my ISP or my UniFi router. I get the IP address through DNS strangely enough.
My Desktop's Fix
The way I fixed this on my desktop was to create a persistent route that went through my phone's mobile hotspot. I'd connect to that over Wi-Fi from my desktop:
Persistent Routes:
  Network Address          Netmask  Gateway Address  Metric
    <IP_ADDRESS>  <IP_ADDRESS>    <IP_ADDRESS>       1

I don't have that luxury here because there are 3 Raspberry Pis (one for each drive).
Ideas for a fix on these Raspberry Pis
Wi-Fi connection to mobile hotspot
I guess, I could get them to connect to my phone's mobile hotspot and also have a presistent route to prefer it for that one IP. That's probably the best option; although, I don't like enabling Wi-Fi on more devices than need it.
Set the gateway address of packets for this site
First, I have my phone's mobile hotspot going through another Raspberry Pi as WAN2. If I could somehow send a request to use a different gateway address, that's one potential fix.
The gateway for my phone to my router is on a VLAN and isn't accessible to these Raspberry Pis though, so I'm not sure how I'd target that unless I can specify the IP of a gateway hop.
VLAN tagging
Since there's a VLAN available with access to the phone's access, I could tag the ports these Raspberry Pis connect to on that VLAN, so they'd have 2 IPs and access to two gateways. At that point, all they'd need to do is add a route for this one URL through that IP address. I don't know enough to know if that's possible.

I tried everything and was only successful with the mobile hotspot + IP route solution:

But for some reason, I'm having the exact same issue as before!
Both the Raspberry Pi and the Docker container have access to that IP. Not sure why only MakeMKV can't do anything with it.

why are you running docker? raspberry pi runs linux, just install the linux version of MakeMKV

The container adds a web interface and runs in a container with easy to configure settings. Why wouldn't I use it?

Any Firewalls in place that would prevent your containers to reach that compared to when it is connected to your Mobile Hotspot?

The container adds a web interface and runs in a container with easy to configure settings. Why wouldn't I use it?

why not just run the full desktop enviorment on the pi and install makemkv and use it's native ui, instead of dealing with docker and the apparent networking challenge's you are facing?

Any Firewalls in place that would prevent your containers to reach that compared to when it is connected to your Mobile Hotspot?

Not that I know of. I literally just installed Raspberry Pi OS Lite 64-bit and installed this and Samba. No other changes.

The container adds a web interface and runs in a container with easy to configure settings. Why wouldn't I use it?

why not just run the full desktop enviorment on the pi and install makemkv and use it's native ui, instead of dealing with docker and the apparent networking challenge's you are facing?

It's not easy to use. I buy and rip, discs all the time, so it's easier for me to open a web browser. I can monitor from any machine. I need 1 RPi per drive, so it doesn't scale if I have to install TigerVNC on multiple machines just to manage it and then connect to 3 sessions at once (I'm on Windows too).
I'd rather use the Docker container because it's simple to use remotely.

I just thought of another solution. I could use proxies. When it tries to contact that domain, I could proxy that to another server that makes the call instead. But if it has access to the domain and still can't do it, I think setting up a proxy won't work either.

obviously the raspberry pi is your issue, there are so many other ways you can do this, you can install make mkv on a windows machine and run 1 instance for each drive, install vnc on your windows machine and get the same effect.
hotspots, proxies....all un needed, are you ripping to a nas? why not run docker on the nas.

i run this docker on unraid with 0 issues.


fix for 'run project' when current editor scene is not saved to a file

After opening new scene tab, assigning some root node and playing the project (f5) at the moment when it was not saved to a file yet, wrong scene is being run.
closes #35947
This work has been kindly sponsored by IMVU.

Superseded by #40635.


cargo check --tests fails to compile macros in examples

Running cargo check --tests on https://github.com/nickel-org/nickel.rs unexpectedly can't find macros. However, cargo test --all compiles fine.

warning: unused #[macro_use] import
--> examples/https.rs:1:1
|
1 | #[macro_use] extern crate nickel;
| ^^^^^^^^^^^^
|
= note: #[warn(unused_imports)] on by default

error: cannot find macro middleware! in this scope
--> examples/integration_testing.rs:90:22
|
90 |     server.post("/", middleware! { |req, res|
|                      ^^^^^^^^^^

error: cannot find macro lazy_static! in this scope
--> examples/integration_testing.rs:279:9
|
279 |         lazy_static! {
|         ^^^^^^^^^^^

cargo check --tests fails to compile examples
cargo check --examples does compile examples, but with warnings
cargo build --examples compiles examples without warnings

cc @ehuss it looks like something that would be fixed by #5384

I investigated this some more.  nickel.rs has a special set of integration tests for their examples.  In particular, the one that errors out (integration_testing) uses include! to pull in the example into their testing framework.
I think this can be closed.  With #5464 things should work better.  cargo check --tests no longer checks examples directly, so it will no longer generate an error.  The warning is legitimate (it looks like they removed SSL support since it was dropped from hyper, but didn't remove the https example which requires the ssl feature which no longer exists!).

Thank you for checking this.


Creation date

Currently the creation date stored in the database is the date the post is saved. The date does not update itself once a post is published. This is unexpected behavior.
It might be necessary to add new fields to Post structure for the initial creation date and for the latest update.

This is probably a bug which will require me to introduce new DB fields. Once fixed I'll mark a new release.

post.Date is now separated into post.Created and post.Updated


HibernateQuery does not utilize the hibernate session properly

Observed vs. expected behavior
I want the query to call SessionFactoryOptionsBuilder#LiteralHandlingMode.interpret method from Hibernate session, whenever the query is fetched. But it is not called and  so hibernate.criteria.literal_handling_mode=inline property becomes ineffective.
It is called when directly creating JPQL without querydsl but not with querydsl's HibernateQuery.
Steps to reproduce
I initialize HibernateQuery as below:
HibernateQuery query = new HibernateQuery<>(reportEntityManager.unwrap(Session.class));
Any call with this should be ok for reproducing issue.
Environment
SpringBoot: 2.6.2
Hibernate: 5.6.3
Querydsl version: 5.0.0
Querydsl module:
Database: Oracle
JDK: 17
Additional details
Root cause:
Hibernate fails when creating groupBy statement with literals in it. For example, below statement returns ORA-00979 error even though the expression in groupBy is also in select statement. I think Hibernate creates new bind parameter for every literal (40, in this example). Because of this issue with Hibernate, I have decided to use inline parameters for literals.
(ttvehdeflo4_.LOCX-mod(ttvehdeflo4_.LOCX,
40))/40 as col_1_0_
I asked a question about this on stackoverflow Link
Hibernate bug link

@eocak, would you mind closing this issue, since this bug is not related to querydsl?


Translate tweet to English&fix max length

Everything else on the page is in English so probably this should be too.
Also truncates the session name if exceeds the twitter limit.
So below

becomes

Ahh, perfekt. Tack! :)