[Snyk] Security upgrade @angular-devkit/build-angular from 13.3.11 to 14.0.0

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 1 vulnerabilities in the npm dependencies of this project.
Snyk changed the following file(s):

frontend/package.json

Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Server-Side Request Forgery (SSRF) SNYK-JS-IP-7148531
  718

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Server-Side Request Forgery (SSRF)

Checkmarx One – Scan Summary & Details – 87e92591-df71-4318-a522-d39774298170
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2022-25883
Npm-semver-7.3.5
Vulnerable Package

CVE-2023-26136
Npm-tough-cookie-2.5.0
Vulnerable Package

CVE-2023-37466
Npm-vm2-3.9.11
Vulnerable Package

CVE-2023-37903
Npm-vm2-3.9.11
Vulnerable Package

CVE-2023-46233
Npm-crypto-js-3.3.0
Vulnerable Package

CVE-2024-29180
Npm-webpack-dev-middleware-5.3.3
Vulnerable Package

CVE-2024-4068
Npm-braces-2.3.2
Vulnerable Package

Insecure_Storage_of_Sensitive_Data
/routes/likeProductReviews.ts: [17](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/likeProductReviews.ts# L17)
Attack Vector

Insecure_Storage_of_Sensitive_Data
/routes/order.ts: [26](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/order.ts# L26)
Attack Vector

Insecure_Storage_of_Sensitive_Data
/routes/order.ts: [138](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/order.ts# L138)
Attack Vector

Insecure_Storage_of_Sensitive_Data
/routes/order.ts: [138](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/order.ts# L138)
Attack Vector

Insecure_Storage_of_Sensitive_Data
/models/address.ts: [24](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//models/address.ts# L24)
Attack Vector

Insecure_Storage_of_Sensitive_Data
/models/address.ts: [10](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//models/address.ts# L10)
Attack Vector

JWT_No_Signature_Verification
/routes/verify.ts: [101](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/verify.ts# L101)
Attack Vector

JWT_No_Signature_Verification
/routes/verify.ts: [101](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/verify.ts# L101)
Attack Vector

JWT_No_Signature_Verification
/lib/insecurity.js: [84](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//lib/insecurity.js# L84)
Attack Vector

Absolute_Path_Traversal
/routes/vulnCodeFixes.ts: [72](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/vulnCodeFixes.ts# L72)
Attack Vector

Absolute_Path_Traversal
/routes/vulnCodeFixes.ts: [72](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//routes/vulnCodeFixes.ts# L72)
Attack Vector

CVE-2023-44270
Npm-postcss-7.0.39
Vulnerable Package

CVE-2024-21501
Npm-sanitize-html-1.4.2
Vulnerable Package

CVE-2024-28863
Npm-tar-4.4.19
Vulnerable Package

CVE-2024-4067
Npm-micromatch-3.1.10
Vulnerable Package

CVE-2024-4067
Npm-micromatch-4.0.7
Vulnerable Package

Unpinned Actions Full Length Commit SHA
/release.yml: [72](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/release.yml# L72)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/lint-fixer.yml: [24](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/lint-fixer.yml# L24)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: [23](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/codeql-analysis.yml# L23)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: [30](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/codeql-analysis.yml# L30)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/zap_scan.yml: [17](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/zap_scan.yml# L17)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/codeql-analysis.yml: [28](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/codeql-analysis.yml# L28)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/release.yml: [109](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/release.yml# L109)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/update-news-www.yml: [20](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/update-news-www.yml# L20)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/release.yml: [50](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/release.yml# L50)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/update-challenges-www.yml: [25](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/update-challenges-www.yml# L25)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/ci.yml: [338](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/ci.yml# L338)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Unpinned Actions Full Length Commit SHA
/rebase.yml: [17](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//.github/workflows/rebase.yml# L17)
Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps...

Angular_Usage_of_Unsafe_DOM_Sanitizer
/frontend/src/app/score-board/score-board.component.ts: [182](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//frontend/src/app/score-board/score-board.component.ts# L182)
Attack Vector

Angular_Usage_of_Unsafe_DOM_Sanitizer
/data/static/codefixes/restfulXssChallenge_4.ts: [59](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/codefixes/restfulXssChallenge_4.ts# L59)
Attack Vector

Angular_Usage_of_Unsafe_DOM_Sanitizer
/data/static/codefixes/restfulXssChallenge_3.ts: [45](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/codefixes/restfulXssChallenge_3.ts# L45)
Attack Vector

Fixed Issues

Severity
Issue
Source File / Package

CVE-2022-38900
Npm-decode-uri-component-0.2.2

Cxc7705965-e0f0
<EMAIL_ADDRESS>

Passwords And Secrets - Generic Password
/users.yml: [176](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L176)

Passwords And Secrets - Generic Password
/users.yml: [236](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L236)

Passwords And Secrets - Generic Password
/users.yml: [115](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L115)

Passwords And Secrets - Generic Password
/users.yml: [140](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L140)

Passwords And Secrets - Generic Password
/users.yml: [149](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L149)

Passwords And Secrets - Generic Password
/users.yml: [63](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L63)

Passwords And Secrets - Generic Password
/users.yml: [248](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L248)

Passwords And Secrets - Generic Password
/users.yml: [203](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L203)

Passwords And Secrets - Generic Password
/users.yml: [166](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L166)

Passwords And Secrets - Generic Password
/users.yml: [242](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L242)

Passwords And Secrets - Generic Password
/users.yml: [88](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L88)

Passwords And Secrets - Generic Password
/users.yml: [107](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L107)

Passwords And Secrets - Generic Password
/users.yml: [212](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L212)

Passwords And Secrets - Generic Password
/users.yml: [158](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L158)

Passwords And Secrets - Generic Password
/users.yml: [123](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L123)

Passwords And Secrets - Generic Password
/users.yml: [31](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L31)

Passwords And Secrets - Generic Password
/users.yml: [3](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L3)

Passwords And Secrets - Generic Secret
/users.yml: [150](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//data/static/users.yml# L150)

Passwords And Secrets - Google OAuth
/default.yml: [57](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//config/default.yml# L57)

CVE-2023-26115
Npm-word-wrap-1.2.3

Cx24228ad1-81fd
Npm-sanitize-html-1.4.2

Cx65603961-769c
Npm-debug-2.6.9

Host Namespace is Shared
/docker-compose.test.yml: [3](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//docker-compose.test.yml# L3)

Host Namespace is Shared
/docker-compose.test.yml: [7](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//docker-compose.test.yml# L7)

Networks Not Set
/docker-compose.test.yml: [7](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//docker-compose.test.yml# L7)

Networks Not Set
/docker-compose.test.yml: [3](https://github.com/weinrich15/juice-shop/blob/snyk-fix-0fcaa983fa97314ef4075401aa67d48d//docker-compose.test.yml# L3)

CVE-2023-2251
Npm-yaml-1.10.2


Don't wrap non-error instance uncaught errors

The current implementation of automatically wrapping non-error instance uncaught errors can cause issues if your application is expecting said errors, such as in the case of having custom error classes defined.
I think it would make more sense for the developer to have more control over the thrown error and let them check the instance if they wish to do so. The current implementation is problematic as it will produce an error such as ERR_CAUGHT_NON_ERROR_TYPE: [object Object].

Hi, the problem with making this change is that it would be breaking for the Server.set_error_handler method which guarantees that the error parameter is an Error type. Also, I believe you should be able to either wrap or use the Error.cause property to properly communicate your errors upwards in the hierarchy.

Your proposed solution is an option but in my use-case not feasible without a lot of work, as I imagine could be the same case for others using this library.
Would you consider implementing this with a feature flag as to not break backwards compatibility? It would be nice to be able to handle errors myself without the library interfering.

I'm not sure If that would be wise this seems to be a very specific edge case. My recommendation would be the following:
Create a middleware which overrides the Response.throw method with your own custom method that acts as a middleman and wraps any value you provide into it using an Extended Error class or simply a normal Error with your provided value as the cause. Then in the error handler, you can simply reverse the wrapping process and pipe the values to any of your desired source.


Cleansing rules for logged SQL queries

(you don't have to strictly follow this form)
Use case
A clear and concise description of what is the intended usage scenario is.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Erroneous entry caused by Github problem.


Assigning null to a BeanShell variable in a JSR223 sampler does not work as expected

Bernhard (Bug 56816):
vars.get() returns void in a java script if the variable is not existing instead of null (as described in it's documentation http://jmeter.apache.org/api/org/apache/jmeter/threads/JMeterVariables.html#get(java.lang.String) )
This will lead to confusion and bugs if the user doesn't know this.
Severity: major
OS: All

Just ran into that same issue in the current JMeter 5.5.
It is definitely a bug in the included BeanShell library bsh-2.0b6.jar.
Fortunately it is already fixed in the current BeanShell library bsh-2.1.1.jar. I successfully tested this by locally replacing the library in JMeter and rerunnning the reproduction tests (see below).
As this old bug ticket got a bit convoluted in the comments, I added a concise problem description and easy reproducible example below.
A little clear statement first (as some IMHO strange ideas have been thrown around earlier in the ticket):
If you add a JSR223 sampler and select "language" as "java (<engine_names_and_versions>)", you expect (and can expect) that it interprets the text written in the section "Script" below as being written in Java.
That's the whole purpose of that language entry and that kind of handling Scripts is the whole purpose of JSR223 (see: JSR223 Spec: https://jcp.org/aboutJava/communityprocess/final/jsr223/index.html) and JMeter's JSR223 sampler (it's named that way for a reason).
So yes: It's a bug and there's no doubt about it, it's in an external library, and it's already fixed there.
Issue:
The Java implementation behind "java (BeanShell 2.0b6 / BeanShell Engine 1.0)" is catastrophically wrong concerning the handling of "null" and "void" (with void being new in Java 11 bsh mixed something up, about its usage).
Simplest reproduction (a comparison between openjdk 11 and bsh-2.0b6 in JMeter) (with the same openjdk 11 in the background):
Code and result from Java:
public class VoidTest {
public static void main(String[] args) {
System.out.println("Java TEST java.version: " + System.getProperty("java.version")); // 11
String varX3 = null;
System.out.println("Java TEST varX3: " + varX3); // null (as it should be)
System.out.println("Java TEST varX3 is null: " + (varX3 == null)); // true (as it should be)
// System.out.println("Java TEST varX3 is void: " + (varX3 == void)); // compile fail: illegal start of expression (as it should be)
}
}
(Put it in a file "VoidTest.java", run "javac VoidTest.java", run "java VoidTest".)
--> That result is fully compliant with the Java Language Specification, i.e. by definition openjdk11's implementation of that snippet is correct.
Code and result from "java (BeanShell 2.0b6 / BeanShell Engine 1.0)" in JMeter 5.5:
log.warn("BeanShell TEST java.version: " + System.getProperty("java.version")); // 11
String varX2 = null;
log.warn("BeanShell TEST varX2: " + varX2); // void (WRONG!)
log.warn("BeanShell TEST varX2 is null: " + (varX2 == null)); // false (WRONG!)
log.warn("BeanShell TEST varX2 is void: " + (varX2 == void)); // true (WRONG!)
--> That result fails to comply with the Java Language Specification, i.e. bsh-2.0b6 is wrong. And that bug is very prolematic as it breaks a lot of correct Java code.
Fix:
Fortunately this has already been fixed in the current version of the BeanShell library bsh-2.1.1.jar (see https://github.com/beanshell/beanshell/releases/tag/2.1.1), which I tested (same test as above, but with lib/bsh-2.0b6.jar replaced with lib/bsh-2.1.1.jar in JMeter5.5 gives the correct result, so no JMeter code needs to be changed, just an external library updated.
While trying to get a PR together, I found that unfortunately bsh-2.1.1.jar is not already published to Maven Central (https://mvnrepository.com/artifact/org.apache-extras.beanshell/bsh), so integrating it in JMeter without that would need some more effort as JMeter seems to only use dependencies from mvnrepository so far.
I did open a ticket in the bsh repository for publishing it in mvnrepository (https://github.com/beanshell/beanshell/issues/739). Probably give this a thumbs-up, if you can, just in case it speeds up things.
Options:

We wait for beanshell to publish bsh-2.1.1.jar on Maven Central (mvnrepository.com) and someone does a simple library replacement PR (I would have done it, but see above).
A JMeter developer adds a way to retrieve the library directly from https://github.com/beanshell/beanshell .

Workaround:
Replace lib/bsh-2.0b6.jar with lib/bsh-2.1.1.jar in your installation of JMeter5.5.


Cow: Watch oncologistsurgeonindia.com

Cow requests the watch of the watch_keyword oncologistsurgeonindia\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

oncologistsurgeonindia\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Laughing Therapy Page

I will create the complete page of laughing therapy with all the relevant information, how to perform, benefits ...
So can you Assign it to me?
SSOC'22

@MohitNamdev22 Go Ahead

I'm closing this issue because it has been assigned to someone else earlier, Sorry for the inconvenience


Error "Use of undeclared identifier" in Xcode

I'm sorry I am new to Xcode and not familiar with OC. I encountered an error after the following operation:

In terminal

git clone https://github.com/tanersener/ffmpeg-kit-test
cd ffmpeg-kit-test
git checkout tags/ios.v4.5.1

Open the ios/test-app-local-dependency project with Xcode ( Version 13.2.1 (13C100) )
Change the Development Team options
Change the Product Bundle Identifier options
Drag folders in the the folder ffmpeg-kit/prebuilt/apple-ios-arm64/ffmpeg/framework into the framework directory of project and delete the original files with the same names
Rename ffmpeg-kit/prebuilt/apple-ios-arm64/ffmpeg-kit/include to ffmpeg-kit/prebuilt/apple-ios-arm64/ffmpeg-kit/ffmpegkit, and drag folder ffmpeg-kit/prebuilt/apple-ios-arm64/ffmpeg-kit into option System Header Search Path
Try to build on my iPhone 6s ( iOS 15.2 )
It pops up a bunch of errors:

/Users/shine/Documents/git/ffmpeg-kit-test/ios/test-app-local-dependency/FFmpegKitIOS/FFmpegKitTest.m:152:46: ⚠This old-style function definition is not preceded by a prototype
...
/Users/shine/Documents/git/ffmpeg-kit-test/ios/test-app-local-dependency/FFmpegKitIOS/FFmpegKitTest.m:197:44: 🔴Use of undeclared identifier 'MediaInformationSession'
/Users/shine/Documents/git/ffmpeg-kit-test/ios/test-app-local-dependency/FFmpegKitIOS/FFmpegKitTest.m:197:30: 🔴Use of undeclared identifier 'sessions3'
...

Why do errors occur?

You're copying the wrong files. You must copy the bundles. See the following scripts. We use them to for test-app-local-dependency.
https://github.com/tanersener/ffmpeg-kit-test/blob/main/scripts/update_ios_local_dependency_xcframeworks.sh#L8
https://github.com/tanersener/ffmpeg-kit-test/blob/main/scripts/update_ios_local_dependency_frameworks.sh#L8
https://github.com/tanersener/ffmpeg-kit-test/blob/main/scripts/update_ios_local_dependency_frameworks_lts.sh#L8

Because I got an error using command ffmpeg-kit/apple.sh, I had to use command ffmpeg-kit/ios.sh. This results in me having no XCFramework folder and only framework folder, but I don't think is a big problem. I manually deleted the invalid xcframework folders in the project, and copied the *.framework folders from ffmpeg-kit/prebuilt/bundle-apple-framework-ios into the framework folder of the project. The above question disappears. I sincerely appreciate your help.
I'm now running into another problem, compiler prompts:
Command PhaseScriptExecution failed with a nonzero exit code

Then here is the detail of the error:
/Users/shine/Library/Developer/Xcode/DerivedData/FFmpegKitIOS-bdwxpktjllcjsxcyijmotdpavnri/Build/Intermediates.noindex/FFmpegKitIOS.build/Debug-iphoneos/FFmpegKitIOS.build/Script-3448266727328FD400ECED5A.sh:
line 3: /Users/shine/Library/Developer/Xcode/DerivedData/FFmpegKitIOS-bdwxpktjllcjsxcyijmotdpavnri/Build/Products/Debug-iphoneos/FFmpegKitIOS.app/Frameworks/ffmpegkit.framework/Resources/strip-frameworks.sh: No such file or directory
I tried copying the file ffmpeg-kit/tools/release/apple/strip-frameworks.sh to the appropriate location. Well, the build succeeded, but the installation failed. The installation failed on both my iPad mini5 and iPhone6s.

See the instructions given in Importing iOS Frameworks wiki page. I'll update it with a step for strip-frameworks.sh.

The App is working now, and it is awesome! Thank you so much for your help!


self should be VariableName, not keyword

In Python, the variable name self is used only by convention and does not actually carry any special meaning. As such, it should be treated as a variable name.
The following program is legal, but would require any program using leze-python to handle VariableName | self type.
self = True

Current: Script(AssignStatement(self,AssignOp,Boolean))
Expected: Script(AssignStatement(VariableName,AssignOp,Boolean))
Example of how this currently has to be handled (for all params, lambdas, etc.)
switch (cursor.node.type.name) {
  case "VariableName":
  case "self":
    // handle variable
  ...
}

On the topic of self in python, here's an oldy-but-goody on python's use of explicit self (and by extension why it's not a reserved word).

Great to have someone who's more familiar with the language look over the grammar. Merged!

Awesome, marking as closed (since GitHub didn't auto detect?)

Ah, indeed, I rebased over your other PR so it didn't recognize the commit hash.


Robustify dev install instructions in README

Use http protocol rather than git protocol (which was failing for some internal users)
Move commit hash down to script so we can copy-and-paste install instructions
Provide easy suggestion for installing PyTorch which often works fine (w/o GPU)

No need to let tests pass, since this is README only

Yep I'll cut a PR I wasn't sure if it was just my machine if you tested it
On Mar 15, 2018 7:12 PM, "Fritz Obermeyer"<EMAIL_ADDRESS>wrote:

@fritzo commented on this pull request.
In README.md https://github.com/uber/pyro/pull/891#discussion_r174982582
:

Then build PyTorch following instructions in the PyTorch
README.
+Depending on your system, this may be as simple as
+```sh
+pip install ninja
+python setup.py install
@jpchen https://github.com/jpchen can you point me to your PR that
fixes this by removing this line?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uber/pyro/pull/891#discussion_r174982582, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AByHSRW3O0wpeXHrGDwH7rGw0N62e3_iks5tex-NgaJpZM4SrUQ2
.


Borg tweaks

About the PR

Bunch of small fixes:

fixed an issue where the law menu would get duped entries
added player feedback for when a chassis had too many modules to install another
removes spam from setting borg name multiple times
adds an admin command for getting a borg's laws (> ent <entity uid> laws:get)
ui now closes when the panel is shut
laws now correctly default when off-station

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: You can no longer keep a cyborg's UI open after closing the panel.
fix: Cyborgs now retain their laws when off-station.

Honestly I'd suggest not making it so obvious when a borg is emagged, it can already be pretty obvious by the person forgetting to make them not state the law or by how they're acting.


Kourami stalls upon Bubble Processing and Path Assembly for DPB1

Hi!
I've encountered a problem using  Kourami v0.9.6 with IMGTHLA v3.42.0 (preprocessed as per instructions): a single human 30x WGS 100 bp read-length BAM file causes stalling of Kourami at Bubble Processing and Path Assembly for DPB1. Kourami manages to successfully process all previous loci (A, B, C, DQ, DR, etc.). While stalling, Kourami still uses computational resources and its processes are active. It has been stalled for 12 hours already and still does not proceed to the next step.
The observed issue does not happen with any other similar BMA files, which are successfully processed extremely fast within 5 min.
I can provide the on_KouramiPanel.bam file if necessary.
@ckingsford Could you please tell me whether Kourami is still supported? It seems like all of the latest issues do not have any impact. Thank you.

Hi,
Would you be able to share the bam file? Thank you. It could be due to the updated alleles in IMGTHLA.

Hi @heewookl
Unfortunately, I'm unable to reproduce this error now. I'm closing the issue for now.


Audio update

Description
This pull further integrates anchor links into the Why It Matters and Methods themes.
Risks
Similar risks associated with previous anchor link pulls.


Revert accidental package upgrades

@git001 I'm not sure what you ran to do 0ef1ef476710ace5024dc5cfa6abee6a88c30dd8 but I don't think you ran go mod tidy, which is what you should've run. Whatever you ran upgraded a bunch of unrelated dependencies, which is why things broke.

@francislavoie thank you for the PR. I approved it now and will merge it.


Create a Context Script

I am posting a bounty for publishing a Context Script and reporting any issues encountered during the process. The script will need to be added to this directory via pull request and it should follow the format of the other scripts in there. You can test it out using an account at contextscript.com. What the script does is completely open-ended. I've listed some ideas below, but you can bring your own idea if you run it by me first and I approve. (As long as it isn't something trivial like "hello world" I will probably approve).
Here are my suggested ideas for Context Scripts:

query: has this been posted on reddit/hacker news?

response: Search for the current webpage on a link aggregator and optionally provide a way to post it.

query: Find text like "space ship"

response: Highlight words similar to the given phrase (e.g. "space shuttle", "rocket") using NLP techniques.

query: regex search {{regex}}

response: Find and highlight text on the webpage that matches the regular expression.

query: List the people mentioned on this page

response: Use Dbpedia spotlight or another entity recognition system to list all the people mentioned on a page. Could use other types of entities as well.

query: screenshot this page

response: Screenshot the page with http://html2canvas.hertzen.com/ and display the result.

query: How has this page changed recently?

response: Use the wayback machine to report things like when the page last changed, and if possible, what changed about it.

query: make this script into a bookmarklet

response: create a dedicated bookmarklet that invokes the previous script

query: word frequency histogram

Plot a word frequency histogram using the page's textual content.

query on evernote or google keep: find related notes

Show the top N most similar notes. One approach would be to look for correlations in their bag-of-words tf-idf vectors.

query on cloud9 ide: filter nulls out of the list

Inject code at the location of the cursor that filters nulls out of the nearest list.

query on cloud9 ide: js to coffee

Convert the highlighted text to coffeescript.

Hello! I've tried context script for the first time and I thought it was great! Here is my full writeup:

signing in with persona/gmail was easy!
I tried out the tables example and saved them as csv, worked fine.
try the hello world example, couldn't find a new script button so I typed "new script" and it has 2 results! chose create script
when I pressed save I got a a json error:
{"readyState":4,"responseText":... is not of a type(s) object, "status":400,"statusText":"Bad Request"}
tried again and it worked, figured out the error came from deleting the "q: " in the trigger. [maybe the JSON error could be turned into a human readable error message explaining that?]
tried out the word count script and checked out its source code. it was a bit complex so I checked out the 'get page text' source code script next.
I saw a lot of pages that extract information from a page but none that edit the page itself, so I looked through the source for what features cxsAPI has.
I didn't find a reference for what cxsAPI can do. but I did notice this dead link: https://github.com/contextscript/contextscript/blob/master/jspm.io when reading https://github.com/contextscript/contextscript/blob/master/publishing.md
Decided to make a script to hilight all occurences of a word in article by basing it on getText and using the nodeMappings. When I tried this: It displayed "undefined to undefined" so I wonder if there is a bug in that!
I wrote a script that hilights in yellow every occurence of the word quantum, and tells you how many it finds. You are welcome to include this as an example script if you like! Currently the word it  finds is hard coded: It would be nice to take the word to find as a parameter but I didn't see any examples of that.
It was very useful to be able to edit my script on the context script webpage itself then switch back to test it on the wikipedia article.
Here is the script itself:

https://contextscript.com/contextscripts/oZJlxxhgT_ixJM5NsrfzOA-1435419324138

Thanks a ton for the feedback. It has been enlightening. I'll work on the issues you mentioned. Documentation for cxsAPI is my next priority.
To take the word as a parameter, put the part of the trigger phrase you want to be a parameter in brackets like this: q: "find word {{word}}" Then you can access the parameter in the cxsAPI args property: cxsAPI.args.word Here's an example: "https://contextscript.com/contextscripts/1kKhoUfQS0yJ9vn-xJZvKQ-1422683873975
I can't find a way to reproduce the "undefined to undefined" bug  you reported in the getText nodeMappings. It would help a lot if you could provide instructions on how to make it happen. I noticed a bug in your script that causes dom nodes to be removed which might be the root of the problem. On this line if the node has any sibling elements, setting the innerHTML of its parent will cause them to be deleted. The "plot location" script here contains some code for doing highlighting based on character offsets which you might find useful.
Before publishing your word search script, I would like to see it do something a "ctrl+f" search can't do. What you've created so far would be an excellent basis for making a "highlight quantum mechanics keywords" script that highlights a variety of keywords related to the subject. Another possibility would be to make it into a "find regular expression {{regex}}" script.
When the script is ready, create a pull request adding it to this directory and I will award the bounty to you.

Thank you for the feedback!
I've used that trick to have the script accept a regex parameter - and looked into how plot location avoids deleting child nodes, this improved the script a lot! Now it searches for all instances of a user supplied regex and highlights them.
regarding "undefined to undefined", you were right: It was my mistake in forgetting .node so there is no issue there.
https://github.com/contextscript/contextscript/pull/2

Nice work! I'll take a closer look at your PR on Friday before merging it. I'll award the bounty now though.


File missing sharp.node'

Module not found: Error: Can't resolve '../build/Release/sharp.node' in the following files:
utility.js,
output.js,
input.js ,
constructor.js

Hello, are all of the prerequisites  installed? Are any errors reported when npm install sharp is run?

Yes I did not install all the prerequisites with the new version; after installing it worked.
I think the reason why I had the files initially is because a older version does not depend on the dependacies.
Thanks for the help :-)


OData JSON load

Hello,
It's my first time using jsGrid, I'm a lot happy with the design and the implementation. I have read the documentation and saw the odata service demo, but I don't understand how the data is loaded. How can I build my JSON.
My project is to read a xml file that has the list of users and be able to load and edit that data in jsGrid.
Can you help me please?
Best regards,
Tiago Filipe

Please, checkout backend examples http://js-grid.com/samples/. Amount others there is an example with ASP.NET WebAPI.

Thank you very much!! :) I didn't see that tab 'BackEnds'


Icons.keyboard_double_arrow_right_rounded is not visible on mobile web

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Use Icons.keyboard_double_arrow_right_outlined in your code

Other icons I tried worked on both mobile and desktop.
see example here: https://github.com/tomasbaran/dayagenda/blob/main/lib/widgets/task_card.dart
Expected results
Open it on desktop web: you can see it:

Actual results
Open it on mobile web: safari/chrome: you cannot see it:

Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-arm64, locale en-MX)
    • Flutter version 3.13.9 on channel stable at /Users/tomas/code/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (5 weeks ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.14.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.76.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

This could be an implementation issue as the screen is wider on desktop. In order to verify the issue properly, please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code. Thanks!

You are right — it seems only an implementation issue since the minimal repro code is not showing it. Thanks @huycozy!


Expose 'Component' component to plugins

While investigating https://github.com/NickTikhonov/hyperterm-hyperline/issues/37 for @davegomez I came across the following:
This class handles styling of components using css / inline styling from what I can grasp. Right now it doesn't get exposed to plugins. Is this by design / for a particular reason? If not should we include it for plugins to use.
Including it could be done here

Fixed in 8.2


--progress=json CLI output

What are you trying to do?
As mentioned in a private discord channel, it would be good to be able to output JSON structured logs from dagger call/dagger core/etc.
Why is this important to you?
This would be useful for ingesting dagger logs into other tooling, and allow easy post-build filtering, etc.
How are you currently working around this?
User doesn't seem to be right now - I'd recommend using --progress=plain, since it's the closest to something structured.

This should actually be a pretty easy feature to pick up for anyone interested: we already have the Frontend interface which is implemented today by frontendPlain and frontendPretty.
Each Frontend ingests OTEL data by implementing various exporters, and renders the output. A future frontendJSON would ingest OTEL data in a very similar way, and probably render the output on a ticker similar to frontendPlain. We can easily use something like go's log/slog for this.
A few implementation notes:

Dagger logs are hierarchical - we would probable express this using a parent field.
Each log entry has a start/stop time - we would probably render each line twice, once when it starts, once when it stops.
Stdout/Stderr handling is a bit tricky - we would probably want to buffer all of this, and print it on completion of an event. Alternatively, we could slowly buffer it out, but this is much harder to filter later.


Clarifies when abbreviated words are used

As per a suggesting here https://github.com/ethereum/staking-deposit-cli/pull/252#discussion_r837424813 , this PR clarifies the use of abbreviated mnemonic words in code.

I am very confused on what happened here. I don't know what happened to the commit scope. I rebased on the updated dev and force-pushed to https://github.com/ethereum/staking-deposit-cli/commit/fce407cdd30ae7a82c48bd94a0999ee2022830d3, but apparently there is now no difference with dev


vm.createFork() return always 0

Component
Other (please describe)
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (c486fca 2024-05-06T00:20:07.911403000Z)
What command(s) is the bug in?
cheatcode vm.createFork()
Operating System
macOS (Apple Silicon)
Describe the bug
Description
Foundry vm.createFork() returns zero instead of the chainID of sepolia.
// SPDX-License-Identifier: UNLICENSED
pragma solidity ^0.8.25; // up to

import {Test, console} from "forge-std/Test.sol";

contract Demo is Test {

    function setUp() public {

    }

    function testCcipDummy() public {
        string memory ETHEREUM_SEPOLIA_RPC_URL = vm.envString("ETHEREUM_SEPOLIA_RPC_URL");
        uint256 forkId = vm.createFork(ETHEREUM_SEPOLIA_RPC_URL); // my url from alchemy

        console.log("sepoliaFork %", forkId); // return always 0
    }
}

Ok so the identifiers start from 0 but the fork is created correctly then? Correct?

Ok so the identifiers start from 0 but the fork is created correctly then? Correct?

Yes


Add issuer support as url

Description
Add issuer support as url, support also projectID alone for backward.
Must

[x] Tests
[ ] Documentation (if applicable)

@shilgapira / @itaihanski - we need to add this also for swift-sdk


UI: ACL "reason" --> "description"    , label improvement

Adding a new ACL on the ACL list
If others agree, can we rename "reason" which sounds interrogative (and wrong) to "description"?

cc @utchoang can you check if it's a translation/key issue - and if anything we can do.

@andrijapanicsb @rhtyd It's set as reason in the API cmd as well https://github.com/apache/cloudstack/blob/master/api/src/main/java/org/apache/cloudstack/api/command/user/network/CreateNetworkACLCmd.java#L90
I'm a +1 for updating in both backend and frontend for consistency, not just the UI

well, we cannot update the backend API key (breaks compatibility) - let's fix the translation key instead. (check if label.reason is used elsewhere)

label.reason is used only in ui/src/views/network/AclListRulesTab.vue

Will raise a PR to change it then

Fixed in https://github.com/apache/cloudstack/pull/4980


Add Swedish translation

Sorry for the (very) late response, thanks for all the help :)

Not a problem, glad you're getting to it :)


help -ShowWindow parameters only show first section of text.

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
In a *dll-Help.Xml file if a parameter has 2 or more <maml:para> blocks, only the 1st is shown in "ShowWindow".
For example, in the module SqlServer version 21.1.18245 Microsoft.SqlServer.Assessment.Cmdlets.dll)
In the help xml file the description for the Checkparameter: (21.1.18245\en\Microsoft.SqlServer.Assessment.Cmdlets.dll-Help)
        <command:parameter required="false" variableLength="true" globbing="false" pipelineInput="False" position="named" aliases="none">
          <maml:name>Check</maml:name>
          <maml:description>
            <maml:para>One or more checks, check IDs, or tags.</maml:para>
            <maml:para>For every check object, Get-SqlAssessmentItem returns that check if it supports the input object.</maml:para>
            <maml:para>For every check ID, Get-SqlAssessmentItem returns the corresponding check if it supports the input object.</maml:para>
            <maml:para>For tags, Get-SqlAssessmentItem returns checks with any of those tags.</maml:para>
          </maml:description>

In ShowWindow, only the 1st <maml:para> block of text is displayed.
Compare:
help Get-SqlAssessmentItem -ShowWindow

to
help Get-SqlAssessmentItem -Parameter check

multiple <maml:para> blocks work fine in cmdlet description.
Expected behavior
All of the help text for each parameter should be displayed.
Platyps will create help XML files with multiple <maml:para> blocks.

Actual behavior
Only the first block is displayed.

Error details
There is no error given.
Silent failure.

Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0
PSEdition                      Core
GitCommitId                    7.2.0
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0ΓÇª}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

Thank you @joepuresdk -- The Interactive WG has determined that there is a bug in Get-Help -ShowWindow preventing the markup from being displayed.

@theJasonHelmick Can I try fixing this?

I think I might have fixed it.

@paziz001 -- Thats wonderful!!  If you would like to share your solution here, that would be very welcome.  We are thinking about how to address this along with other considerations - so you knowledge is welcome.

This was a long time ago but I have referenced the PR with the possible solution which has been partially approved.


Added text-decoration challenge

Pre-Submission Checklist

[X] Your pull request targets the staging branch of freeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[X] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[X] Tested changes locally.
[X] Closes currently open issue (replace XXXX with an issue no): Closes #13468

Description

Added text-decoration as a seperate challenge from the hover state challenge. Based challenge style on the "Adjust the Color of an Anchor Tag" challenge. Challenge is placed before the hover state challenge as code is to be added to the a selector instead of the a:hover selector. Also updated hover state challenge seed to transition smoothly from the text-decoration challenge.

@systimotic Maybe i could tweak the challenge such that the link has text-decoration:none by default, but has an underline on hover? This design is much more prevalent in web pages and still sticks to introducing the concept of text-decoration.
Regarding the "releasedOn" field, i forgot to ask whether we need to change it before submitting the PR 😅 . Will have to check with other moderators on gitter.

@amberbranson3 Hi! I'm sorry, I'm afraid I don't understand what you're asking. Can you elaborate? If you were looking for coding help, check out our chatrooms.
@hay-dee Sorry for the delay! I've been really busy, and will be really busy for the coming months.
Text Decoration for the title sounds good to me 👍 It might not make as much sense of you don't know about the property yet, but it's probably the best option overall.
The date from the flexbox challenges is the same as the date of this challenge, so that's fine as-is.
I'm still not sure about where to use the property. Showing the underline on hover only is definitely a more common pattern, this is what GitHub does too. However, I think it would make this challenge more complicated than it should be. Maybe we should change it to a double underline?
Maybe the description should touch a bit more on the possible values?

@systimotic No problems with the delay! Thanks for taking time out to guide me on this PR. 😄
I realised that the current text-decoration challenge is immediately before the hover state challenge. Would swapping the order of the text-decoration and hover state challenge reduce the complexity of my proposed text-decoration challenge? Campers will then only be required to add in a line text-decoration: none within the already present hover state selector.
I could definitely add in a one or two-liner in the description to explain the text-decoration property and address the other possible values.

@hay-dee how are we coming along on this one?

Hi @raisedadead , @systimotic and i are still amidst discussion regarding the content of the challenge. Once we have come to a consensus, i can make the necessary amendments and amend this pull request :smile: .

Closing as stale. @hay-dee please re-open when ready. Thanks for your hard work and contribution.

@hay-dee I'm so sorry for disappearing! As I mentioned, I was and would be busy. That turned into basically not being on GitHub from mid-March until last week. I did pass my exams, and I'm very happy I finally have time to write code again!
The proposal in your last comment sounds great to me! Do you want to pick this back up again?

Stop sending the emails!!!

Hi @MarcDearden!
It looks like you may have accidentally subscribed to all notifications for the freeCodeCamp repository. To stop receiving notifications please visit the subscriptions page or click on the image below and select "Not watching".

We are sorry for your inconvenience!

Hi @systimotic,
I have to admit that i have not been keeping up with work in this repo. Apologies that i may not be able to carry out the changes that i have mentioned before. Maybe we can release this change to first timers? :smile:


Highlight in the examples the placeholders and their status

To make it more apparent which elements on the pages are placeholders, and whether they have failed or succeeded.

Mmm, beautiful!


Equipment Tab - Deed Roll Formatting Cleanup

The "Deed Roll" on the Equipment tab needs some formatting assistance.
Vertical align label and field, left-align it on top of the equipment sheet.
Add ring around value box.

I wish either Foundry or Font-Awesome provided a black d4 icon!
Foundry only has a grey one. Font-Awesome's d4 is pro only (and a bit weird).


[Linux Config] Clarify which filesystem is being referenced

Replaces PR #577
Signed-off-by: Rob Dolin<EMAIL_ADDRESS>

I think the problem is in my sign-off name.  I'll try another PR.


Remove duplicate object method ?

How to remove duplicate object method in RLMResult with @property? (@property may be two to judge)

@wgywgy are you asking for a way to do distinct queries? If so, we have an open issue that's tracking adding that feature at https://github.com/realm/realm-cocoa/issues/1103.

it slove the problem of read, but I want save the distinct result.

You're likely looking to use -[RLMRealm deleteObjects:] along with some logic to determine what constitutes a duplicate object.
For example, if you considered a model to be unique based on some value pair int1/int2, you could remove duplicates with the following code:
Model
@interface MyModel : RLMObject
@property NSInteger int1;
@property NSInteger int2;
@end
@implementation MyModel
@end

Usage
RLMRealm *realm = [RLMRealm defaultRealm]; // or some other Realm
NSMutableSet *seenIntegerPairs = [NSMutableSet set];
NSMutableArray *duplicateObjects = [NSMutableArray array];
for (MyModel *object in [MyModel allObjectsInRealm:realm]) {
  NSDictionary *integerPair = @{@"int1": @(object.int1), @"int2": @(object.int2)};
  if ([seenIntegerPairs containsObject:integerPair]) {
    [duplicateObjects addObject:object];
  } else {
    [seenIntegerPairs addObject:integerPair];
  }
}
[realm writeTransactionWithBlock:^{
  [realm deleteObjects:duplicateObjects];
}];

// All duplicate `MyModel` objects with the same `int1`/`int2` pair have been deleted

Please reopen this issue if you have any further questions, @wgywgy


Unable to watch nested data

Vue.js version
2.0.2
Reproduction Link
http://jsfiddle.net/2vqkLkjL/1/
Steps to reproduce

Create a component with a nested data property: Object, of an object containing an array.
Set up watcher for this property, specifying the deep flag.

What is Expected?
When the value of an array changes, the watcher runs
What is actually happening?
Watcher isn't getting triggered.
The code seemed to work fine in Vue 1.x.
Not 100% nested objects/arrays is the correct way for managing a view with lots of inputs.

the same problem with #3958   I am resolving this problem
@allistera  You could use object as a workaround now.

Ah I should have done a better job searching. Thanks deffc awesome work!

closing as it duplicates #3958


[FRAAS-463] Near comprehensive rewrite

Update, July 1. Everything works! I've asked @markcraig to go through this from a Doc PoV (and he's more up to speed on OAuth2/OIDC thank I :))
To review this PR, I'd take the following steps: (My plan -- to set up this procedure in FRAAS-18)

Install Postman
Set up tenant, and log in (staging.forgeblocks.com)
1a) You'll need the tenant name, and the tenant access token from the dev console of your browser (it's associated with the session endpoint)
Download

'ForgeRock Cloud APIs.postman_collection.json' (when you import it, you'll see it with a (June 2019) label
'ForgeRock Environment Vars 0.4.postman_environment.json'

Import files into Postman
Go into the environment (gear icon, upper right), and add

tenant name
access token

Compare text online, between two browsers:

In your Postman app, find the arrow next to the collection, select "View in Web". That'll open up the web view in your default browser. (Or you can view it here)
In a second browser, navigate to developer-api.forgerock.com

Major changes:

I've made the calls more sequential. Example, under Management APIs / Apps / Web, you can run the commands sequentially, to demo create, update, regenerate secret, delete.
I've moved the Management APIs first, as some of those are prereqs to Authentication API calls.

NOTE: While I can continue work on this with the Docs73 tenant, I'm functionally blocked by FRAAS-683 (on the Docs74 tenant)

Importing the collection and the environment works fine for me in the Postman standalone app on Linux. So does logging in to the tenant whose credentials you shared with me for testing.
I still cannot access the REST APIs due to some sort of DNS issue:
$ curl -v https://api-ui-docs79.forgeblocks.com/
* Could not resolve host: api-ui-docs79.forgeblocks.com
* Closing connection 0
curl: (6) Could not resolve host: api-ui-docs79.forgeblocks.com

It would be really nice if the tenant admin UI offered a way to get at the admin access token.
Getting it out of the web developer tools in the browser is maybe okay for beta but seems like an "ease of use" blocker for GA.

Hi Mark, now I get it --
You were trying to connect to api-ui-docs79.forgeblocks.com

curl -v https://api-ui-docs79.forgeblocks.com/

This works for me (and matches what's shown in the collection)

curl -v https://api-docs79.forgeblocks.com/

Ack... I clicked the wrong button, and accidentally "re-requested" a review from @markcraig . My bad :)

Comments:
* Double-check the links. I got some 404s like https://developer-cloud.forgerock.com/quickstarts/password-policies/.

I've checked every link. If we continue to use Postman for our docs, I've added a quest for automation to FRAAS-609.

* Too bad you cannot run the API calls directly in the docs, even if you upload your environment. It seems like a limitation of Postman documents,

That's one reason for me to pursue https://forgerock-demo.redoc.ly/ (forgerock/demo)

* `{{userName}}` defaults to<EMAIL_ADDRESS>Should `{{user_id}}` default to `bjensen`?

I've checked -- not in the original version of the 0.4 environment. Only after you create a user does {{userName}} get set to the new user, in this case<EMAIL_ADDRESS>
* Might want to review the environment variables for consistency. Sometimes you use underscores_in_names, other times camelCase.

Absolutely. A post-beta task for me. I think also I have a few excess variables.

Thanks, Mike. LGTM.

Hi Mike, I think this is a big improvement and like better where the collection is going. I was attempting to get through the PKCE Auth code flows. I noticed a couple of things:
Pre-req is that you need to create a user and in doing that they need to get an m2m token

Did we want to apply PKCE flows to Service( m2m) apps? That would be new.

which then sets a var called appAccessToken. However the var being used in the collection is just accessToken.

I realize I have some "cleanup" to do w/r/t variables. I've hard coded too many things. I set that task aside to work on other pre-beta doc tasks.

It would be slightly more convenient if that was setting appAccessToken and then denoting it in the console like Acquired session is output in the console on the authenticate endpoint. Maybe something like "Setting accessToken to " in the test script. I did run into a roadblock on the authorization code flow getting error: {
description: "The redirection URI provided does not match a pre-registered value.",
message: "redirect_uri_mismatch"

I'm guessing this error -- at least w/r/t Native/SPA apps, is based on the difference between http://localhost:9080/* and https://example.com/homeLogin as login URIs (based on Gemma's advice). When I use example.com, it makes for better screenshots -- (try example.com in Chrome, and I think you'll see what I mean).

Despite having set a redirect url to http://localhost:9080. in the spa app. Not sure if this is due to the upgrade to 7.0 or not since this is a new tenant.
I noticed the vars contain a var for spaAppClientSecret because spa apps don't ever have secrets I think it should be removed from the vars

Perhaps that's still a bug -- I get a client secret when I run the REST call to regenerate client secrets for SPA apps.

Also, I noticed spaclient was used for the PKCE collection. I would assume this could possibly be for native apps as well and perhaps even web apps using a PKCE flow. I don't know if we could use a more generic term for clientid perhaps just clientid and denote this can be used for either spa/native or web flows.

As discussed with Jared, I have two different flows for Native/SPA PKCE and Web App PKCE, based on James P's guidance in FRAAS-458. So I want different client IDs for different types of apps

James P should be able to confirm if this can be used for web flows too difference being web flows would have a secret.

As described by James P in FRAAS-458, and discussed with Jared. (I'm in the middle of drafting the flow for the main docs -- but I need a working tenant to confirm the process.)

Anyhow, I can't say I can get the token in the end right now perhaps we can review together

I know I was having problems this morning with PKCE flows that were working all last week.

Merging after discussion with @janelleallen


use empty string for TestObjectCreateBadMd5Empty

this test case has been passing, but not in the way in we thought
the use of " " here was actually causing 403 SignatureDoesNotMatch; apparently because java is signing the header value of " ", where boto and radosgw are stripping that whitespace from header values when calculating the signature
on 403 errors, the java client retries this request. because of a separate bug https://tracker.ceph.com/issues/58286 in radosgw's http parser, the retried request failed with 500 Bad Request
500 errors aren't retried, so this is the error code caught by the test case
this test case was copied from test_object_create_bad_md5_empty() in s3-tests, where it instead expects 400 InvalidDigest

the fix merged in https://github.com/ceph/ceph/pull/49642, so i've cherry-picked this to ceph-master


Converters return false for optional boolean values that aren't set.

NOTE: Before filing a ticket, please see the following URL:
https://github.com/jamesagnew/hapi-fhir/wiki/Getting-Help
Describe the bug
Converters return false for optional boolean values that aren't set.
To Reproduce
When I convert a Patient from DSTU2 to STU3 or R4, active is set to false, when active wasn't set in the DSTU2(hl7) instance.
Expected behavior
optional booleans that weren't set in the source instance shouldn't be set in the target instance
Use a check like if (src.hasActive()) tgt.setActive(src.getActive()); in converter code, rather than
tgt.setActive(src.getActive());
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

HAPI FHIR Version: 3.6.0
OS: [e.g. iOS] Windows
Browser N/A

Additional context
Add any other context about the problem here.

Problem also appears in other primitive types.  (e.g., Attachment.size)

and time types are respecting precision.
It seems that primitive conversions need to take the form:
tgt.setElement(convert(src.getElement());
So that they respect primitives better.

Attached are two modified version converters (from DSTU2 to STU3, and DSTU2 to R4) with corrections for the above problems.
PatchedVersionConvertors.zip


Mouse disabled + Middle bar not detected

My mouse got disabled while using the bot, so I basically cannot use the bot properly in game. Plus, the middle bar is not detected by the bot (eventually the A-D buttons are not working).

Are you using the old v1.0 version?
The new 2.0 version doesn't touch or alter the mouse at all.

Nope I'm currently using the latest version since I have just downloaded it yesterday after using the 1.0 for a while.
But the main issue is not about the mouse, it's about the A-D output doesn't work at all.

record a video showing the game plus the bot in action.

https://github.com/mdnpascual/ToF-Fishing-Bot/assets/135721849/3de884a4-1fed-46e5-a7d5-172b8b123afb
Here is the video

bot works best when the game is on fullscreen or ran in a higher resolution. I see the game's FPS already chugging so I can see why you are reducing resolution.
If running the game at fullscreen makes the game's fps just really bad, then here's my based on your video:

change lower right settings to target Y:172 or even Y:171
fish stamina to target X: 2179 Y: 193

If the middle bad is still isn't detected after that, then that resolution is just way too small for the bot to work with.
One more thing, if you are on Windows 11, turn off auto-HDR

Ah i see, i'll check my game and the bot again. Thanks a lot


Update PPP and documentation

Pull Request Test Coverage Report for Build 445

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.486%

Totals

Change from base Build 444:
0.0%

Covered Lines:
2370

Relevant Lines:
2709

💛  - Coveralls


documentation improvements

[x] ensure it is clear that the coordinator must contribute before the ceremony is closed otherwise it will error out when trying to finalize
[x] aws ec2 instance profile id vs arn

Doco revised


Fixes neo4j-contrib/neo4j-streams#313 Add Timestamp, Header and Key

Fixes #313
Add support to handle data from a message's timestamp, headers and key both in Neo4J Kafka Plugin and Kafka Connect Neo4J
Proposed Changes (Mandatory)
Right now both Kafka Connect Neo4J and Neo4J Consumer Kafka Plugin only takes into account a message's value, but there is lots of useful information on the other Kafka message's parts as stated in the related issue.
Because of that, this pull request adds the message's timestamp, headers and key to the event object, so all that information is accessible from whoever needs it.
To do so, three fields have been added to the event object:

__timestamp__: Message's timestamp (Long).
__headers__: Message's headers as a Map<String, String>. The type of the value can be anything as a Kafka Header is a Map<String, Array<Byte>>, but the value is usually a String or some sort of data with an easy String representation like booleans, integers, longs, doubles and so on. If a message's header is null, the __headers__ field will be set as an empty object to avoid null references.
__key__: Message's key. It can be accessed as event.__key__ if the key is a simple object, like a String, or as event.__key__.field if the key has some fields after extracting its schema with the deserializer. If the key is null, the __key__ field will be set as an empty object to avoid null references.

As these changes could cause a naming collision between pre existing field names inside the event object, this change also provides a way to customize how these new fields are named.

Neo4J Kafka Plugin

streams.event.timestamp,
streams.event.headers,
streams.event.key,

Kafka Connect Neo4J

neo4j.event.timestamp
neo4j.event.headers
neo4j.event.key

Hi @Dionakra thanks a lot for the effort that you put on this, I think that we should not merge it, there are multiple ways to achieve, outside the Streams Plugin, what you're trying to put here.
See Single Message Transformations in Kafka Connect or KSQL, that can allow the developer to pre-process the stream obtaining the same result.

Hi @conker84 ,
Thank you, I didn't think about SMTs, you are completely right! Sorry for making you waste your time!
Anyway, that only applies to the Kafka Connect Sink, not the Neo4J Kafka plugin. If I wanted to use the Neo4J Kafka Plugin, what would be the recommended approach to access a message's headers and key?
Thank you.

You can use the Kafka Connect framework as middleware, that manipulates your stream and prepare it with the necessary information that you need for the ingestion into Neo4j with te Streams Plugin.
This approach works for both Streams and Neo4j Kafka Connect Sink.


RepositoryExplorer to support URL routing

As discussed with @bogdan-sava it should be possible to support app-route URI routing to Repository Explorer, so that a user can click-through from another page - e.g. asset-catalog - with server name and URL root and entity GUID to load.
The desired outcome is to load the repository explorer page with auto-connection to the server, setting of the enterprise option (since the server may not host the home repository of the entity) and auto-retrieval of the entity whose guid is passed.
The design of this is that the URI is extended with a 'tail' (see app-route in my_app.js) that contains the following fields:

serverName - plain ASCII
serverURLRoot - base64 encoded (to encode the slash delimiters in the serverURLRoot)
guid - plain ASCII

These are added to the tail in the order specified above, separated by slashes.
For example:
https://localhost:8443/#/repository-explorer/Schema_Server/aHR0cDovL2xvY2FsaG9zdDo4MDgy/cd4d5e42-3137-45b6-a270-834287841557
where,
http://localhost:8082 has been base64 encoded to aHR0cDovL2xvY2FsaG9zdDo4MDgy
This capability is intended for auto-loading of entities. To support relationship instances would require that the category is also included in the tail. That's possible but not (yet) seen as a requirement.

The result with the above example URI tail is that Repository Explorer loads directly to the following state:


level_select_lock does not work

Describe the bug
The level_select_lock prelude option does not lock future levels. The level names are hidden, but can still be selected/played.
Reproduction Steps

Load an example game with level_select enabled (e.g. Wriggle)
Add "level_select_lock" to the prelude
Save/run the game, go to the level select screen and try to select a locked level
The level loads and runs

What did you expect would happen?
The level should not load (see PS+ behaviour)
OS & Browser version, desktop/mobile
Desktop/MAC/Chrome (and same on Firefox)
Screenshots, example projects, other relevant issues, etc.
Problem exhibits on both stable and dev release
There are no errors in browser console
Adding "level_select_unlocked_ahead" makes no difference (although it does hide the correct number of level names)

Example game "Gridlocked" will also demonstrate this issue without any source changes required because it already uses level_select_lock.

Fixed. Couple of lines got left out during a merge.
Note there were differences between keyboard and mouse.


Restart gunicorn workers to avoid memory leaks

After raising the request timeouts gunicorn workers start consuming
more memory under sustained high load, eventually resulting in OOM
killing a random worker and interrupting a request.
It's unclear whether this is caused by python VM not managing to
free up memory in time or a memory leak somewhere in our code or
one of the compiled libraries we use to process PDF and PNG files.
It's worth digging into this a bit more at some point, but it's
currently complicated by the fact that we are using different tools
for similar tasks, any of which could be causing the problem.
As a quick fix setting max_requests makes gunicorn restart the
worker process after it has handled Nth request. This slows down
the next request (by around 300ms, which in case of template
preview is not a big difference) but keeps memory usage from
reaching the instance limit.

@sakisv I don't know the exact number. 10 requests limit keeps the instance around 1GB on Prometheus metrics, but that wouldn't show any spikes that could otherwise trigger the OOM.


ConfigurationFile fixes

General housekeeping -- comments and such
WFCORE-513 -- use the correct boot file in reload in the special cases
WFCORE-514 -- use the correct file for taking snapshots when we don't persist changes to the main file

Linux Build 1080 is now running using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120

Windows Build 786 is now running using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120

Core - Full Integration Build 360 is now running using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120

Windows Build 786 outcome was SUCCESS using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120
Summary: Tests passed: 2932, ignored: 95 Build time: 0:15:51

Linux Build 1080 outcome was SUCCESS using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120
Summary: Tests passed: 2932, ignored: 95 Build time: 0:19:52

Core - Full Integration Build 360 outcome was SUCCESS using a merge of 2f2a26baaf694fe39fdb64e2738f9d6bc65af120
Summary: Tests passed: 2868, ignored: 377 Build time: 0:41:32


fix: only get releases if they do not exist

Allows caching the Releases to a releases.json so that 3rd parties do not unnecessarily re-download them every time. We could maybe add some "svm update-releases" command which would force-redownload the file, to update it if it changes.
The motivation is that https://github.com/gakonst/ethers-rs/pull/614 made it so that a releases request is made every time, and that can slows down Foundry's startup time

Closing, the bottleneck was not that. My mistake.


Screens are blank for a few frames when navigating to them

Not sure if this is expected behavior or it's something to do with my configuration, but here's a video slowed down to 25% speed
My navigator setup:
StackNavigator: {
    DrawerNavigator: {
        BottomTabNavigator: {
            StackNavigator: {
                Screen
            }
            StackNavigator: {
                Screen
            }
            ...SimilarStackNavigators
        }
    }
}

Happy to answer any other questions about my implementation. My app works significantly smoother now, very excited about this project.

can you provide an example that reproduces this?

@brentvatne hey have been pretty swamped but will be getting to reproducing as soon as I can. It does happen in both development and production.

@kmagiera forgot to update with this info - I've upgraded to Expo SDK 31 and there is no gray flash anymore. React-Navigation version is the same.

FYI we have found one more instance of this issue that may have cause a flash with stack navigator on Android. It's been addressed in #50 and also in react-navigation repo here https://github.com/react-navigation/react-navigation-stack/pull/73
Both react-nav changes and screens update will be released soon

I'm seeing this behaviour with react native router flux on android - it is using 2.x branch of react navigation as base.
I'm using latest screens and latest 2.x of react navigation - still seeing this flicker. Even if i backport the listed PR, removing opacity completly or switching to ios transitions. It is also not the root view that shines through - i'm wondering if there is any idea or fix that could help

Hey @K-Leon – unfortunately I have no idea how router flux is using react navigation under the hood but as far as I remember the change that fixed it in react navigation has been merged to 3.X not 2.X (might be wrong tho).
What helped me debug this in the past was to try record transision using screen recording and then playing in slow motion (managed to notice it was actually the transparency that's changed). If you have time to prepare such video and post it here in slow motion that might help.
I'm actually going to close this issue as it has been addressed a while ago. Can you please open a new one to hold the discussion about router flux?


Add support for PTX Mesh Multifunction Wireless Switch

}, {
    # https://home.miot-spec.com/spec/chuangmi.switch.meshb01
    10371: ["PTX", "Mesh Multifunction Wireless Switch", "090615.remote.mlsw0a"],
    "spec": [
        MiBeacon, BLEAction, Button,
        EventConv("action", mi="2.e.1", value="click"),
        EventConv("action", mi="3.e.2", value="hold"),
    ]

Enviroment:
Device Xiaomi Multimode Gateway CN v1.5.4_0090
XiaomiGateway3: v3.2.1
Homeassistant: 2023.5.4

by the way, how do I add translation for this integration? I notice HA provided a new translate way.

Thanks!

https://github.com/AlexxIT/XiaomiGateway3/releases/tag/v3.3.0


Bugfix: desktop auto-scrolling wouldn't stop after starting (Resolves #441)

Bugfix: desktop auto-scrolling wouldn't stop after starting (Resolves #441)
There was an adjustment in the calculation that I was applying on every scroll frame. I'm not entirely sure why it was working originally, and then started failing. I might be missing some use-cases, but I removed the adjustment that I was applying to the user's drag position and it seemed to fix the problem.

@roughike I wanted to make sure you saw this PR, too


It'd be nice to have a changelog

something similar to this:
https://github.com/webdriverio/webdriverio/blob/master/CHANGELOG.md

Will consider it, thanks


Cannot install ddcci-plasmoid-backend without pip and --break-system-packages argument

I tried every other method recommended. pipx doesn't exist and I'm unable to find it in the repositories to install, pip warns externally-managed-environment and tells you to use pacman. Neither pacman nor pamac are able to find the python-ddcci-plasmoid-backend or ddcci-plasmoid-backend packages.
$ pip install --user ddcci-plasmoid-backend
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try 'pacman -S
python-xyz', where xyz is the package you are trying to
install.

If you wish to install a non-Arch-packaged Python package,
create a virtual environment using 'python -m venv path/to/venv'.
Then use path/to/venv/bin/python and path/to/venv/bin/pip.

If you wish to install a non-Arch packaged Python application,
it may be easiest to use 'pipx install xyz', which will manage a
virtual environment for you. Make sure you have python-pipx
installed via pacman.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

$ pipx
bash: pipx: command not found
[sollace@Bismuth ~]$ pamac install pipx
Error: target not found: pipx

$ pamac install python-ddcci-plasmoid-backend
Error: target not found: python-ddcci-plasmoid-backend

$ sudo pacman -S python-ddcci-plasmoid-backend
doas (sollace@Bismuth) password:
error: target not found: python-ddcci-plasmoid-backend

$ pip install --user ddcci-plasmoid-backend --break-system-packages
Collecting ddcci-plasmoid-backend
Downloading ddcci_plasmoid_backend-0.1.10-py3-none-any.whl.metadata (2.1 kB)
Collecting fasteners<0.19,>=0.18 (from ddcci-plasmoid-backend)
Downloading fasteners-0.18-py3-none-any.whl.metadata (4.9 kB)
Downloading ddcci_plasmoid_backend-0.1.10-py3-none-any.whl (9.5 kB)
Downloading fasteners-0.18-py3-none-any.whl (18 kB)
Installing collected packages: fasteners, ddcci-plasmoid-backend
Successfully installed ddcci-plasmoid-backend-0.1.10 fasteners-0.18

The Arch Linux package you're looking for is python-pipx.

can confirm that pipx works fine. You just have tell the widget to use ~/.local/bin/ddcci_plasmoid_backend


bolt-theme uses final item in the map, irrespective of theme.

As I understand it, the following should result in yellow background on dark/xdark, white on light/xlight:
packages/micro-journeys/src/nav-shared.scss
    background-color: bolt-theme(
      (
        // Should be yellow
        xdark: primary,
        dark: primary,
        // Should be white
        light: secondary,
        xlight: secondary,
      )
    );

Results in this:

This is present in Chrome and FF.
This can be seen here: https://boltdesignsystem-mar7a5s5m.now.sh/pattern-lab/?p=experiments-micro-journeys

Also, I spoke for a while with @mikemai2awesome about this issue.

@glassdimly looking into this — I'll keep you posted.

@WessWillis @glassdimly just a quick update on this one:
From what I can tell, I think half the issue here likely has to do with the PostCSS plugin never supporting bolt-theme inside of media queries.
I'm working on a fix for at least that piece however we'll still need to address how best to handle theming UI that's fully encapsulated in the Shadow DOM since the default t-bolt-THEME contextual selectors wouldn't apply in those situations.


Add draft submission files for host-guest systems

This updates the repo to have information about template files for submission of host-guest predictions; since there were no new suggestions, I basically just duplicated the files used in SAMPL5 and updated for this challenge.

@andrrizzi , interested in doing super quick review?


Extend the perfect-entanglers example to show a standard gate optimization

Closes #24

Codecov Report

Merging #26 (2d6dc19) into master (588d9df) will increase coverage by 1.38%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
+ Coverage   81.44%   82.82%   +1.38%
==========================================
  Files           6        6
  Lines         361      361
==========================================
+ Hits          294      299       +5
+ Misses         67       62       -5

Impacted Files
Coverage Δ

src/optimize.jl
89.77% <0.00%> (+4.54%)
:arrow_up:

src/GRAPE.jl
100.00% <0.00%> (+50.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


describe paravirt and fullvirt

Coverage remained the same at 38.029% when pulling f6f61548e921a226d94b5fc276a625f904f4e124 on describe-paravirt-fullvirt into 8ff6448850d94db33585dafdf4bd71988a0e5c09 on develop-transitional.

I would argue that we should match the existing controls that follow a similar pattern (on the same page):

Coverage remained the same at 38.104% when pulling c26b9c331224f1bc15113e32bc0c468a259294cb on describe-paravirt-fullvirt into 4883bfe9865e4c817a0d3a380b2dedee9b500062 on develop.

Coverage remained the same at 38.056% when pulling 720d643d5a1cf0ca3a3e2affb426bc3b8ce176f2 on describe-paravirt-fullvirt into 5b6fc4a588fa4d72edde0ddc67f877b447a873ec on develop.

@displague updated to use bootstrap styles instead of custom styles

Coverage remained the same at 38.056% when pulling a7956c9a5c191ea73c36b258250018889df905f9 on describe-paravirt-fullvirt into 5b6fc4a588fa4d72edde0ddc67f877b447a873ec on develop.

I'm merging this as-is, but while trying to make this section more responsive (like the checkboxes below) I realized these text labels don't describe the options they are next to.  These new text labels are actually a single description of this fieldset.

I also would have preferred these text additions to be <small className="text-muted">, but since I later realized these aren't actually element descriptions, I didn't hold the task up.


Use build as default target

To my knowledge it is common practice to invoke a compile/build target when calling make without arguments.

Thank your for submitting this! Merged