Test gives correct outcome, but I cannot pass exercise

Challenge Waypoint: Replacing If Else Chains with Switch has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function myTest(val) {
  var answer = "";
  // Only change code below this line

  switch (val) {

    case 'bob':
    answer = "Marley";
    break;

    case '42':
    answer = "The Answer";
    break;

    case '1':
    answer = "There is no #1";
    break;

    case '99':
    answer = "Missed me by this much!";
    break;

    case '7':
    answer = "Ate Nine";
    break;
  }

  // Only change code above this line
  return answer;
}

// Change this value to test
myTest('1');

Thanks for reviewing :)

You're checking for strings when you should be looking for numbers. In the future please visit the Help Chat to get confirmation that the error you're seeing is indeed a site error and not an issue with your code. Thanks and happy coding!

Usually I do check the Help Chat, this time I was pretty sure there was and error... Sorry about that and thanks :)


Breakage when Roll Tables have individual currency items

I think the line here needs to use CurrencyHelper.... instead of this....
https://github.com/jopeek/fvtt-loot-sheet-npc-5e/blob/388bb49d5e4f303a784c3ac9a62a3c7b0f745b8f/src/modules/classes/LootProcessor.js#L342

Thanks for reporting.
This should be fixed by #342


"Edit and Resend" feature is not working in 131.0.2903.63

After testing, it works on the old version (131.0.2903.51)

I'm having exactly the same problem

Thank you for reporting this bug. I'll add this to the dev team's backlog.

It seems that it has been fixed in the latest version. (131.0.2903.70)

This was fixed with a hotfix in .63 and confirmed fixed in .70. Thanks for reporting the issue!


[SPARK-33080][BUILD] Replace fatal warnings snippet

What changes were proposed in this pull request?
Current solution in build file to enable build failure on compilation warnings with exclusion of deprecation ones is not portable after SBT version 1.3.13 (build import fails with compilation error with SBT 1.4) and could be replaced with more robust and maintainable, especially since Scala 2.13.2 with similar built-in functionality.
Additionally, warnings were fixed to pass the build, with as few changes as possible:
warnings in 2.12 compilation fixed in code,
warnings in 2.13 compilation covered by configuration to be addressed separately
Why are the changes needed?
Unblocks upgrade to SBT after 1.3.13.
Enhances build file maintainability.
Allows fine tune of warnings configuration in scope of Scala 2.13 compilation.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
build/sbt's compile and Test/compile for both Scala 2.12 and 2.13 profiles.

Test build #129608 has started for PR 29995 at commit ee5f641.

Test build #129608 has finished for PR 29995 at commit ee5f641.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34211/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34211/

Merged build finished. Test PASSed.

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

Test build #129633 has started for PR 29995 at commit 0bf0faa.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34237/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34237/

Merged build finished. Test PASSed.

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

Test build #129633 has finished for PR 29995 at commit 0bf0faa.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #129642 has started for PR 29995 at commit 2b42a30.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34246/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34246/

Merged build finished. Test PASSed.

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

Test build #129642 has finished for PR 29995 at commit 2b42a30.

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

Merged build finished. Test PASSed.

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

cc @dongjoon-hyun

Test build #129700 has started for PR 29995 at commit 20d4c3e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34305/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34305/

Merged build finished. Test PASSed.

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

Test build #129700 has finished for PR 29995 at commit 20d4c3e.

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

Merged build finished. Test PASSed.

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

Test build #129703 has started for PR 29995 at commit c8d2594.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34309/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34309/

Merged build finished. Test PASSed.

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

Test build #129703 has finished for PR 29995 at commit c8d2594.

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

Merged build finished. Test PASSed.

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

Jenkins test this please

Test build #129758 has started for PR 29995 at commit d4415a3.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34364/

Test build #129758 has finished for PR 29995 at commit d4415a3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34364/

Merged build finished. Test PASSed.

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

Merged build finished. Test FAILed.

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

Test build #129774 has started for PR 29995 at commit a9582e1.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34380/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34380/

Merged build finished. Test FAILed.

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

Jenkins test this please

Test build #129774 has finished for PR 29995 at commit a9582e1.

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

Merged build finished. Test PASSed.

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

Test build #129787 has started for PR 29995 at commit a9582e1.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34394/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34394/

Merged build finished. Test FAILed.

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

Test build #129787 has finished for PR 29995 at commit a9582e1.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Jenkins retest this please

Test build #129840 has started for PR 29995 at commit a9582e1.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34445/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34445/

Merged build finished. Test PASSed.

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

Test build #129840 has finished for PR 29995 at commit a9582e1.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Seems to be unlucky week to run tests that were green previously :)

Yeah it passed once, this is almost certainly spurious. Let me just try again anyway.

Jenkins retest this please

Test build #129846 has started for PR 29995 at commit a9582e1.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34452/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/34452/

Merged build finished. Test PASSed.

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

Test build #129846 has finished for PR 29995 at commit a9582e1.

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

Merged build finished. Test PASSed.

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

Merged to master

@srowen thanks a lot for your efforts to pass this changeset


FIXME: swift/validation-test/stdlib/Set.swift:95

Use real arc4random_uniform() instead of just random() % max

Is truncatingBitPattern proper way to prevent integer overflow?
Does UInt32 available on every platform?

Off topic question:
Why do you use if true {} in test cases? Is it equivalent for do {}?

No, UInt32(truncatingBitPattern:) is not the right answer here. If the max value passed to uniformRandom() is too bit to fit in a UInt32 then truncatingBitPattern: will silently shrink it. Then the random numbers you get are in fact not uniform in the requested range.
UInt32 is available on every platform. There is a platform difference with Int: Int is only 32 bits on 32-bit platforms. That will make your Int() conversion fail when arc4random_uniform's result is too big.
This is only a test and it looks like we only call uniformRandom with small values. In that case it should be fine to use checked conversions to UInt32 and back to Int. They will never fail with the tests as written today, and if something does fail someday then we can write a more complicated fix.

The if true statements probably pre-date the existence of do. IIRC do was added late in the development of Swift 2.0.

@gparker42 thanks for comments. I have update the PR. Is that ok?

Looks good, thanks!


Is CentOS 8 not supported?

I'm surprised to see that CentOS 6 and 7 are supported for self-hosted Linux build agents but CentOs 8 is not! Is that true or is the document perhaps out of date?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 23328b13-f079-ad21-5d7a-6c640f25391c
Version Independent ID: 4a39889d-fa00-ae2b-aa43-f358631b7b1a
Content: Deploy a Azure Pipelines agent on Linux - Azure Pipelines
Content Source: docs/pipelines/agents/v2-linux.md
Product: devops
Technology: devops-cicd-agents
GitHub Login: @steved0x
Microsoft Alias: sdanie

@lurifaxel Thank you for the question.
Currently CentOS 8 is only officially supported by .NET Core with known issues, see https://github.com/dotnet/core/issues/3751.  Azure DevOps users have made recent posts here and here.
Please use this for development purposes only and we will update the documentation once support is officially announced.

Hi,
Is CentOS 8 supported now?

@sriramdasbalaji -- thank you for your question.  Please see this article, which includes this note:

Warning

CentOS 8 reached an early End Of Life (EOL) on December 31st, 2021. For more information, see the official CentOS Linux EOL page. Because of this, .NET 6 won't be supported on CentOS Linux 8.


Delete Agent

I can't delete an agent or pause it

Working on this feature would be live soon!

Reopening this until feature is live. This is currently on by @Arkajit-Datta


Embed youtube-nocookie.com Domain - GDPR

Description
I want to use GDPR conform Youtube Videos(https://www.youtube-nocookie.com) in a project. Is this possible?
Example Output!

Other information

Embedded Assets version: 2.8.6
Craft CMS version: 3.6.12

It doesn't look like it's currently possible, as there doesn't seem to be a way to get YouTube to return the nocookie domain.  We'll have a look at adding a setting for it, though.

Just added a Use Youtube nocookie domain? setting for Embedded Assets 2.6, which will be released later this month.  It's important to note that it won't affect what gets saved in the embedded asset JSON files (i.e. the saved iframes will still use youtube.com) but the setting being enabled will give you embedded asset models in which the iframes use youtube-nocookie.com instead.  This means any existing YouTube embedded assets won't need to be replaced in order to use youtube-nocookie.com, but also means any YouTube embedded assets saved while the setting is enabled will revert to using youtube.com if the setting is disabled later.

This is great! exactly what I was looking for just now :)


Doesn't handle  within

ReverseMarkdown.convert("hello

world")

=>
**hello

world**

Expected:
**hello**

**world**

Related:
[70] pry(main)> ReverseMarkdown.convert '
foo
bar
'

=> "# foo  \nbar\n"


nginx listen address isn't configureable

This should be a configureable option.
The request was originally made against the omnibus-chef-server repo https://github.com/opscode/omnibus-chef-server/issues/94

GOTO https://github.com/chef/chef-server/issues/188


install to gitlab-ci

What happened: can't to install to gitlab-ci
How to reproduce it (as minimally and precisely as possible):
I used next stage fot my task:
horusec:
  stage: check-code
  image: horuszup/horusec-cli:v2.8
  services:
    - docker:19.03.12
  script:
    - docker run -v ${CI_PROJECT_DIR}:/src horuszup/horusec-cli:v2.8 horusec start -p /src -P ${CI_PROJECT_DIR} --output-format json --json-output-file horusec-report.json
  artifacts:
    paths:
      - horusec-report.json
    expire_in: 1 weeks

Anything else we need to know?:
I can't use dind service because runner configured without privileged mode and it cannot be reconfigured.
Environment:

Horusec version (use horusec version):

Version:          v2.8.0
Git commit:     df32c1ce03d2de748cecb76cff383f2851e198c3
Built:               Wed Jun 08 13:57:08 2022
Distribution:    normal

Others:

logs from gitlab-ci:
time="2022-06-10T11:17:25Z" level=warning msg="{HORUSEC_CLI} Config file not found"
time="2022-06-10T11:17:25Z" level=error msg="{HORUSEC_CLI} Error when check if docker requirement it's ok!" error="error during connect: Get \"http://...": dial tcp: lookup docker on IPAddress: no such host"
time="2022-06-10T11:17:25Z" level=info msg="{HORUSEC_CLI} If your docker is not installed check in docs of how to install in:\n\t\thttps://docs.docker.com/get-docker\n\t"
Error: error during connect: Get "http://...": dial tcp: lookup docker on IPAddress: no such host

Please tell me how to configure the application to work without a dind service?

horusec:
  stage: check-code
  image: docker:19.03.12
  script:
    - docker run -v /var/run/docker.sock:/var/run/docker.sock -v ${CI_PROJECT_DIR}:/src/horusec horuszup/horusec-cli:v2.8 horusec start -p /src/horusec -P ${CI_PROJECT_DIR} --output-format=json --json-output-file=./horusec-report.json
  artifacts:
    paths:
      - ./horusec-report.json
    expire_in: 1 weeks

In the above job I get the following logs:
time="2022-06-10T22:23:01Z" level=warning msg="{HORUSEC_CLI} Config file not found"
time="2022-06-10T22:23:01Z" level=warning msg="Horusec will return a timeout after 600 seconds. This time can be customized in the cli settings."
time="2022-06-10T22:23:01Z" level=warning msg="{HORUSEC_CLI} PLEASE DON'T REMOVE \".horusec\" FOLDER BEFORE THE ANALYSIS FINISH! Don’t worry, we’ll remove it after the analysis ends automatically! Project sent to folder in location: [/src/horusec/.horusec/dc9dc862-4c50-4f50-98ca-8f6673d70a41]"
time="2022-06-10T22:23:08Z" level=info msg="{HORUSEC_CLI} Writing output JSON to file in the path: /horusec-report.json"
time="2022-06-10T22:23:08Z" level=warning msg="{HORUSEC_CLI} No authorization token was found, your code it is not going to be sent to horusec. Please enter a token with the -a flag to configure and save your analysis"
time="2022-06-10T22:23:08Z" level=warning msg="YOUR ANALYSIS HAD FINISHED WITHOUT ANY VULNERABILITY!"
time="2022-06-10T22:23:08Z" level=warning msg="{HORUSEC_CLI} Horusec not show info vulnerabilities in this analysis, to see info vulnerabilities add option \"--information-severity=true\". For more details use (horusec start --help) command."
Uploading artifacts for successful job
Uploading artifacts...
WARNING: ./horusec-report.json: no matching files
ERROR: No files to upload
Cleaning up project directory and file based variables
Job succeeded

Please explain how to save artifacts.

Should a report file "horusec-report.json" be generated with such a message - level=warning msg="YOUR ANALYSIS HAD FINISHED WITHOUT ANY VULNERABILITY or not?
Help, please!

hello @johnTux tnks for your contribution
First, it is really necessary to use a privileged environment so that horusec can use docker and its features well.
So the second point is that as we have no vulnerability, no report is generated at the end and you can see the message YOUR ANALYSIS HAD FINISHED WITHOUT ANY VULNERABILITY!

@wiliansilvazup, thanks for your reply


Task 1 - Step 11 is out of date

When I got to step 11 in Task1, what I see in my lab is different.
What this guide shows I should in step 11 of task 1 is this:

However, what I see is the following:

Hi,
Thanks for letting us know about this. We have updated the instructions in the lab.
Thanks


ts intellisense displays wrong message

As you can see, the typescript extension suggests removing the await keyword even though it is useful here (i.e removing it would cause a bug).
Could be related to #60385.
Suggestions to improve this issue are appreciated.

Please text source for a minimal example of this (not just screenshot)

Just made a github repo with the code and the needed files.
my repo
I hope this helps.

Your JSDoc annotations are incorrect. loadData should be needs to return a promise: @returns {Promise<JSON>}

Ok, thanks


Fix has_one polymorphism

Problem: has_one relationships that are polymorphic do not work as expected: instead of getting the actual polymorphic resource, you get instead the base class.
Fix: extend the handling used in the polymorphic belongs_to relationship to cover the has_one case as well. I used the exact same code as the block above.
Tested on Rails 4.2.x.
Let me know if you have any questions or concerns.

@nagirrab Thanks for the fix. Could you add a test that demonstrates the issue and confirms the fix?

@nagirrab Thanks!


Tab stops showing as active between 2.0.12 and 2.1.0

react-native, react and native-base version
rn: 0.42.3, r: 15.4.1, nb: 2.1.0
Expected behaviour
Active tab should display differently from other tabs
Actual behaviour
The first tab is always displayed as active (The underline moves but styling does not).
Steps to reproduce (code snippet or screenshot)

Use the Tabs sample http://docs.nativebase.io/COMPONENTS.html#Tabs.
Click the second tab.
First tab is still displaying as selected.

Is the bug present in both ios and android or in any one of them?
Only tested in Android
Any other additional info which would help us debug the issue quicker.
This code does work in an earlier version 2.1.0

Same for mem on iOS

Same for me on Android (7.1.1)

Same for me on iOS

I face the same problem and I fixed it with a temporary hack.

Eject the theme using.
node node_modules/native-base/ejectTheme.js

Import the variables found in the theme to the component.
import variables from '../../native-base-theme/variables/platform';

Add the textStyle and activeTextStyle to every Tab so that it updates accordingly.

<Tab  heading="Hello Tab"
           textStyle={{ color: variables.topTabBarTextColor }}
           activeTextStyle={{ color: variables.topTabBarActiveTextColor }} >

Hope this helps. It works for both Android and iOS.

Merge the theme of newer version with already ejected theme of previous version.


Sporadic test failure: MiqAeClass waypoint_ids_for_state_machine check ids

https://travis-ci.org/ManageIQ/manageiq/jobs/57347177
  1) MiqAeClass waypoint_ids_for_state_machine check ids
     Failure/Error: MiqAeClass.waypoint_ids_for_state_machines.should match_array(ids)
       expected collection contained:  [1000000000408,<PHONE_NUMBER>409,<PHONE_NUMBER>410,<PHONE_NUMBER>410,<PHONE_NUMBER>411,<PHONE_NUMBER>411,<PHONE_NUMBER>412,<PHONE_NUMBER>413,<PHONE_NUMBER>414,<PHONE_NUMBER>415]
       actual collection contained:    [1000000000410,<PHONE_NUMBER>411,<PHONE_NUMBER>411,<PHONE_NUMBER>412,<PHONE_NUMBER>413,<PHONE_NUMBER>414,<PHONE_NUMBER>415]
       the missing elements were:      [1000000000408,<PHONE_NUMBER>409,<PHONE_NUMBER>410]
     # ./spec/models/miq_ae_class_spec.rb:325:in `block (3 levels) in <top (required)>'

...

Randomized with seed 57137

cc @gmcculloug @mkanoor


[seacas] Build error

Host Environment

OS: Windows
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install seacas:x64-windows
Failure logs

Cut and paste the appropriate build messages from the console output.
Attach every failure log file mentioned in the console output.

Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.
issue_body.md

install-x64-windows-dbg-out.log

might just be a simple include missing. Port was tested with VS2022
e:\code\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\text_mesh\Iotm_TextMeshFuncs.h(66): error C2039: 'isspace': is not a member of 'std'
D:\Program Files\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\include\numeric(14): note: see declaration of 'std'
e:\code\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\text_mesh\Iotm_TextMeshFuncs.h(68): error C2039: 'isspace': is not a member of 'std'
D:\Program Files\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\include\numeric(14): note: see declaration of 'std'
e:\code\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\text_mesh\Iotm_TextMeshFuncs.h(106): error C2039: 'isdigit': is not a member of 'std'
D:\Program Files\Microsoft Visual Studio\2017\Community\VC\Tools\MSVC\14.16.27023\include\numeric(14): note: see declaration of 'std'

So I should modify the file Iotm_TextMeshFuncs.h?

other error appeared with vc143 :
OS:Windows11
complier:msvc vc143
To Reproduce
Steps to reproduce the behavior:
.\vcpkg install seacas:x64-windows
Failure logs
-- Building x64-windows-dbg
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
Command failed: D:/ProgramFiles/CMake/bin/cmake.exe --build . --config Debug --target install -- -v -j13
Working Directory: D:/vcpkg/buildtrees/seacas/x64-windows-dbg
See logs for more information:
D:\vcpkg\buildtrees\seacas\install-x64-windows-dbg-out.log
Call Stack (most recent call first):
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
ports/seacas/portfile.cmake:75 (vcpkg_cmake_install)
scripts/ports.cmake:147 (include)
error: building seacas:x64-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+seacas
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[seacas]+Build+error
Include '[seacas] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
vcpkg-tool version: 2023-01-24-8a88d63f241d391772fbde69af9cab96c3c64c75
vcpkg-scripts version: 0ba60bfef 2023-02-03 (2 days ago)
Additional context
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1910): error C2039: 'MY_Alltoallv': is not a member of 'Ioss'
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus/Ioex_Utils.h(33): note: see declaration of 'Ioss'
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1838): note: while compiling class template member function 'void Ioex::DecompositionData::create_implicit_global_map(const std::vector<int,std::allocator> &,std::vector<int64_t,std::allocator<int64_t>> &,Ioss::Map &,int64_t *,int64_t *)'
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1910): error C3861: 'MY_Alltoallv': identifier not found
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1921): error C2039: 'MY_Alltoallv': is not a member of 'Ioss'
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus/Ioex_Utils.h(33): note: see declaration of 'Ioss'
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1921): error C3861: 'MY_Alltoallv': identifier not found
D:\vcpkg\buildtrees\seacas\src\76a825bc51-907a76ce33.clean\packages\seacas\libraries\ioss\src\exodus\Ioex_DecompositionData.C(1620): warning C4244: 'argument': conversion from 'T' to 'size_t', possible loss of data

@Bzz2745
Thanks very much for your feedback🌻, can you please re-open an issue? We'd better do one process for one problem (or problems caused by the same reason).

@Bzz2745 Thanks very much for your feedback🌻, can you please re-open an issue? We'd better do one process for one problem (or problems caused by the same reason).

OK，I have already open an issue,thanks.

So I should modify the file Iotm_TextMeshFuncs.h?

I can't reproduce your problem locally, can you please modify the ctype.h in Iotm_TextMeshFuncs.h to cctype, and try again?
#include <ctype.h> // for toupper
↓↓↓
#include <cctype> // for toupper, isspace, isdigit
You can use --editable in classic mode to preserve source code changes in buildtree/.

It did work!!! Thanks a lot!!!

Thanks for your verification, a request has been made upstream: https://github.com/sandialabs/seacas/issues/367

Thanks for your verification, a request has been made upstream: https://github.com/sandialabs/seacas/issues/367

Why are we waiting for upstream here ? As far as the maintainer rules are concerned simple include corrections are directly allowed.


Issue with embed/embed 4.4.13

New issue that showed up in CI, did not happen with 4.4.12, started with 4.4.13 - diff
 1) SilverStripe\View\Tests\Embed\EmbedContainerTest::testGetDimensions
TypeError: preg_match(): Argument #2 ($subject) must be of type string, null given

/home/runner/work/recipe-kitchen-sink/recipe-kitchen-sink/vendor/embed/embed/src/Document.php:31

https://github.com/creative-commoners/recipe-kitchen-sink/actions/runs/11963492575/job/33354106153
Happens in CMS 6, haven't tested in CMS 5
PRs

https://github.com/silverstripe/silverstripe-framework/pull/11474

PR merged


Prevent overwriting existing file

- Do you want to request a feature or report a bug?
BUG
- What is the current behavior?
When I add a file with the same title as an existing one and my slug equals title, the existing file is overwritten.
- If the current behavior is a bug, please provide the steps to reproduce.
make sure your slug equals title. add a file, choose a title that is already used by another file, save.
you have overwritten the existing file
- What is the expected behavior?

either:
a new file is created (i.e. with counter to filename)
or:
a warning that the file cannot be saved as it needs a unique title

Less urgent if #1069 were implemented.

- What is the expected behavior?

either:
a new file is created (i.e. with counter to filename)
or:
a warning that the file cannot be saved as it needs a unique title

In my opinion, a warning is the most reasonable option until #445 is done. @erquhart @talves @Benaiah

Can I claim this?

@brianlmacdonald That would be great, thanks!

I agree, this should be a warning until #445 then let the user decide how they want to resolve.

@brianlmacdonald Let me know if you have any questions!

@tech4him1 , yes, can you point me to the files I need to be looking at for this?

@brianlmacdonald The area where the slug is created from the file title is:
https://github.com/netlify/netlify-cms/blob/f948f7a4082bccaed837684ee4915c9038db7d4d/src/backends/backend.js#L242-L251

You could likely just throw an error there, and it would be caught in the action:
https://github.com/netlify/netlify-cms/blob/55f01e6f1da080b59c711cbc45703be28429a34c/src/actions/entries.js#L304-L322
I'm not sure if something else would need to be done when in Editorial Workflow mode, that would be something to test.

@tech4him1 As a starting point, I was thinking about calling this.getEntry(collection, slug) after line 248,  and if the return object comes back with the data key as anything other than an empty object, meaning it it found a preexisting file with the same name, then it throws an error.

@brianlmacdonald That sounds reasonable enough to me. The entry data is probably already stored under entries.entities in Redux, but that is not guaranteed to have all the entries anyway (i.e. pagination). getEntry is probably the best way to do it.

@tech4him1 I got it working in regular mode-- it throws a nice error when you try to save with a duplicate filename. Is it possible to set up editorial workflow mode using the default test repo?

@brianlmacdonald No, you have to use a live repo for editorial workflow currently. (One way to do it is just creating a blank GitHub repo  for testing -- it doesn't need to be connected to a Netlify site.)

Re-opening, #1239 was reverted a while back.

Can I claim this issue?

@siddhant1 for sure!


Propseg fail for t2star image when using -init-mask or -init-centerline

Gives a black image as output ...
Data are in /Volumes/folder_shared/greymattersegmentation/data_testing/original_data/d_sp_pain_pilot/pilot2/t2star

Using propseg with -init-mask on a MT image works fine (same place, pilot8/MTC1) and not for a T2*

The T2* images were 4D (I did only dcm2nii), worked after doing an average multiecho

I close this issue because sct_propseg does not work on 4D data.
sct_propseg detects now if the input image is not 3D and stops if it is the case.


Unable to use excludes as mentioned in Docs

Tried the Android code as given in the Docs.
I am unable to exclude the files.
testOptions {
      unitTests.all {
          if (name == "testDebugUnitTest") {
              kover {
                  disabled = false
                  binaryReportFile.set(file("$buildDir/custom/debug-report.bin"))
                  // includes = ['com.example.*']
                  excludes = [
                          "com.makeappssimple.abhimanyu.financemanager.android.navigation.di.NavigationManagerModule"
                  ]
              }
          }
      }
  }

I expect this code to exclude com.makeappssimple.abhimanyu.financemanager.android.navigation.di.NavigationManagerModule file, but it is not working.
Also tried with wildcard names.
Kover setup,
plugins {
    id "com.android.application"
    id "kotlin-android"
    id "kotlin-kapt"
    id "dagger.hilt.android.plugin"
    id "com.google.gms.google-services"
    id "org.jetbrains.kotlinx.kover" version "0.5.0"
}

// Kover
kover {
    disabled = false                        // true to disable instrumentation of all test tasks in all projects
    coverageEngine.set(kotlinx.kover.api.CoverageEngine.INTELLIJ) // change instrumentation agent and reporter
    intellijEngineVersion.set('1.0.656')    // change version of IntelliJ agent and reporter
    jacocoEngineVersion.set('0.8.7')        // change version of JaCoCo agent and reporter
    generateReportOnCheck = true            // false to do not execute `koverMergedReport` task before `check` task
    disabledProjects = []                   // ["project-name"] or [":project-name"] to disable coverage for project with path `:project-name` (`:` for the root project)
    instrumentAndroidPackage = false        // true to instrument packages `android.*` and `com.android.*`
    runAllTestsForProjectTask = false       // true to run all tests in all projects if `koverHtmlReport`, `koverXmlReport`, `koverReport`, `koverVerify` or `check` tasks executed on some project
}

tasks.koverHtmlReport works. 😀
Ok. I understood it now.
The earlier exclusion list was for tests and this is for coverage.


Bad indent HTML tag

Description
Hi !
I think i have found a problem with HTML indent with the "a" tag
Input
The code looked like this before beautification:

ButtonIm

Expected Output
The code should have looked like this after beautification:

                       ButtonIm

Actual Output
The  code actually looked like this after beautification:

ButtonIm

Steps to Reproduce
Copy, paste, enjoy
Environment
OS: Linux
Settings
Default settings

Is this on http://jsbeautifier.org ?   I'm not get that output.

Hi, this happened when using the js-beautify module not by using http://jsbeautifier.org/

Please see if you can reproduce this using the website. or otherwise include the settings you are using.

Ok,
so i finally did it on the website with the default parameters !
The input is :

      Button 1

And the output is :

      Button 1

I tried many times with different input and i found that everything become well indented expect  ''
I hope you'll have the same output as me !
Thank you for your attention

@Undefinaids Yes, that is that same output I got.  I think the stuff inside the  element is "unformatted", so if you add indents it will stay there.

Fixed in 1.7.6.


Field collection

To correctly implement height limiting in the Apollo router, we should expose the CollectFields() algorithm:
https://spec.graphql.org/October2021/#sec-Field-Collection
This algorithm is part of execution and requires resolving variable values. We don't have an API for providing variable values so perhaps we can start with a partial implementation that ignores @skip and @include.

Does this algorithm need to be implemented within the compiler, as opposed to in the router on top of other compiler APIs?

It's also possible to do in the router, but we will eventually want to support it in apollo-rs anyways.
Sent from Proton Mail mobile
-------- Original Message --------
On 24 Mar 2023, 14:40, Simon Sapin wrote:

Does this algorithm need to be implemented within the compiler, as opposed to in the router on top of other compiler APIs?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.Message ID: @.***>

I happened to throw together a rough implementation of collectFields today on top of the HIR types if this is useful:
https://github.com/zackangelo/phoebus/blob/7f3cde8ccac2e97177997aa3555430b35d9d258f/src/executor/collect_fields.rs

@zackangelo oh cool, you're working building an executor! I think my colleague @goto-bus-stop is working on an implementation that closely ties together with field merging. I am not certain as how different their approach is to yours.

I'd probably prefer to use/contribute to another well-tested executor on top of apollo-rs if one exists! But I'll continue plodding along on this one until that day comes because I need at least a basic one for the project I'm working on. If there's an early effort ongoing somewhere I'd love to take a look.

I'm not aware of anything at the moment; we want to work on execution in or on top of apollo-rs in the (near) future, but the initial implementation for this particular issue will be more static and not support variables and skip/include directives.

After giving this a shot & talking it over with Iryna and the router team, I think it doesn't actually make that much sense to provide a "static" version of this at the moment. We're doing something much simpler for the router use case for now and then we'll see if they have a need for this afterward. If not we'll work on this when we get to execution, and then it would be easier to provide a static variant of the algorithm if there is still a point in doing that

Field collection is done in #758

https://github.com/apollographql/apollo-rs/pull/758 contains a full execution engine (including field collection) with a Resolver trait in order to support introspection, but it’s not exposed in the public API. If you’re interested in using it, let us know about your use cases to inform the design.


The Document URL in README.md file  was invalid

https://github.com/apache/flink-cdc/pull/3106

New documentation website is available at https://nightlies.apache.org/flink/flink-cdc-docs-stable/


Document require rationale

Coming from a non-Ruby background, I first found the require system to be crazy (what, just a C/C++ #include in such a high-level language?!), and I see I was not alone. I later came to realize the actual power of the require way (vs. say, the typical import way of many other languages).
The following three realizations/understandings helped clarify the rationale behind why Crystal has adopted the require system in its existing form (aside from "that’s just the way Ruby does it” or "it's always been done that way, so why change?") and what its implications are:

from https://github.com/crystal-lang/crystal/issues/4439#issuecomment-303343828:
The library has an unique name; you can't have two shards with the same name added to the same project (even deeply nested). Whatever methods and types it provides should be defined under that namespace (obviously named after the shard unique name) and not pollute anything, unless this is expected by its nature (e.g. ActiveSupport like libraries that extend the core/stdlib) or optionally provided as an easy mean (e.g. sinatra vs sinatra/app).

from https://github.com/crystal-lang/crystal/issues/4439#issuecomment-303541688:
Consider a bookstore-vs.-library analogy:
The import camp prefers to have dependencies be like books at a bookstore in that I have my own copy of a book I buy, and any monkey-patching I do to the book stays local to my book, not affecting anyone else's book. On the other hand, the require camp prefers to have dependencies be like books at a library in that there is only one book, and any monkey-patching I do to a book I check out applies to everyone who checks out the book thereafter.

All dependencies (including dependencies of dependencies of dependencies) are necessarily stored as globals that cannot be renamed from what they are in their defining files from the point where they are first required (either directly or from a recursive require) until the end of the program, and this is by design. It should be made clear that this is meant to be a feature (rather than a naming collision waiting to happen as one might initially conclude) - see (1). Perhaps another library analogy would be helpful here, e.g. having a single layout of the entire library in place with requiring a new dependency being like checking out a new book that you keep until the end of your program rather than having distributed, non-global sections as in the import way of doing things.

Any objection to adding a cleaned-up version of the above comments + any other insights people may have to the docs? I feel this will significantly reduce the confusion others coming from a non-Ruby background will likely have.
In the end, what should my picture be for the Crystal way of doing things? As far as I understand, the library analogy in (2) + (3) above seems quite fitting...but please use some sort of roadmap/visual/guide so that newbs are not completely confused about how to reason about a typical Crystal project and its dependencies.
This will also ease the burden of dealing with proposals to amend/complement the current require system, since they then won’t be made in the first place so that useful discussions can ensue rather than get shut down without any explanation whatsoever (especially in cases where they introduce new, undiscussed solutions to issues mentioned by the core team that meet all discussed criteria, but maybe don't quite fit with the standard Crystal schematic/conventional way of doing things in their current form). This will result in time saved for everyone, and everyone's better understanding will keep more non-Rubyists interested in Crystal :-)

Closing, as I have decided not to own this. Someone else can revive this if they desire


Request for nodes to repeal each other

Is there anyway, the nodes can repeal each other so they won't overlay and labels could be read better ?  I've done this on vis.js, they have as physics inside options.
See example here

I think d3-force has real potential when it comes to that kind of implementation, we could explore this sample implementation and fine-tune d3 implementation within rd3g to have this behavior, what do you think @corganfuzz ?

I think that's exactly what I need.


subgroups: update use case group charter

As we move forward, use case group is changing it's goals
and responsibilities and focusing on creating a
channel with the end users and potential deployment partners
Signed-off-by: Elad<EMAIL_ADDRESS>Signed-off-by: Boris Glimcher<EMAIL_ADDRESS>

Looks like the changes were merged between time I started review and clicked submit
I'll open a new PR against the merged changes for the above suggestions.

thanks you @pdp2shirts ! good Collab


Added protoc 3.11.4 for osx-aarch_64.

Built from git@github.com:protocolbuffers/protobuf.git tag v3.11.4
Symbols stripped with "strip"
Built with static linkage:  ./configure --disable-shared
% otool -L bin/3.11.4/protoc-3.11.4-osx-aarch_64.exe
bin/3.11.4/protoc-3.11.4-osx-aarch_64.exe:
	/usr/lib/libz.1.dylib (compatibility version 1.0.0, current version 1.2.11)
	/usr/lib/libc++.1.dylib (compatibility version 1.0.0, current version 904.4.0)
	/usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1292.60.1)

Note the license at: https://github.com/protocolbuffers/protobuf/blob/master/LICENSE

HI guys, do you have some idea when this PR will be merged and published? This will help a lot of M1 users. Thanks.

Hi, we need this change too. Can we help some how? Thank you!!

The one single thing that could unstuck this project would be to find some sponsorship. Maybe put in a word with Google or something..

The one single thing that could unstuck this project would be to find some sponsorship. Maybe put in a word with Google or something..

Define "sponsorship"?  As I need this I am also willing to help on it, but need it for private uses not for my current employer.

https://github.com/sponsors

Hi @os72 . Not sure if it's enough (probably not) but I just did a tiny buymeacoffee contribution for this project. Just hoping to get this PR merged so that we can use this maven plugin out of the box for M1 macbooks.

Hi @os72,
Is there a chance you merge this PR ?

hi @os72,
Do we have any news about this PR? or any workaround with this version of os72?

The one single thing that could unstuck this project would be to find some sponsorship. Maybe put in a word with Google or something..

Hi @os72, you are basically saying, you would rather let this project die if there is no sponsorship to give you the resources to work on this by yourself?
If you don't have the time, kindly let others do what they can. That's part of open source software, isn't it?

Hi @os72! I confirmed that we (Spotify) were able to secure corporate sponsorship for protoc-jar. Can we unblock this PR?

Also Google published, in latest version ready artefacts for this architecture - no longer need for custom builds

There is one more option - fork.

it's better - this repo can be archived! We have official artefacts from Google.
@clairemcginty plese use something like bellow
https://github.com/protocolbuffers/protobuf/issues/9973
      <plugin>
        <groupId>com.github.os72</groupId>
        <artifactId>protoc-jar-maven-plugin</artifactId>
        <version>3.11.4</version>
       <configuration>
         <protocArtifact>com.google.protobuf:protoc:3.21.1</protocArtifact>

If you don't have the time, kindly let others do what they can. That's part of open source software, isn't it?

@daveish - not sure i understand this comment, this is open to anyone to modify (keyword fork)..

Hi @os72! I confirmed that we (Spotify) were able to secure corporate sponsorship for protoc-jar. Can we unblock this PR?

hi @clairemcginty - this is awesome and very appreciated! kudos to spotify for showing some serious OSS leadership
i'm not sure the process has completed, but yes this will help unfreeze the project. hopefully word can spread (beyond just spotify) and help establish a long-term sustainable maintenance level for the project

Would be really great if you could merge this any time soon

By when we expect this to close?

One of workarounds for error wth protoc v3.11.4 and JDK 11.0.6 is maven goal parameter:
mvn install -Dos.arch=x86_64

I tested this PR and it works perfectly.
If the maintainer cannot continue it I suggest someone who can to fork the project and allow others to contribute.

Please checkout the fork by BlackRock: https://github.com/blackrock/protoc-jar-maven-plugin
This forked version uses Google’s Protoc version 3.25.1 by default which supports osx aarch64.

Please checkout the fork by BlackRock: https://github.com/blackrock/protoc-jar-maven-plugin
This forked version uses Google’s Protoc version 3.25.1 by default which supports osx aarch64.

Seems like the README should refer to protoc-maven-plugin? I can only find jars under https://repo1.maven.org/maven2/io/github/blackrock/protoc-maven-plugin/2.0.0/ and not under https://repo1.maven.org/maven2/io/github/blackrock/protoc-jar-maven-plugin/2.0.0/

@cmardini , you are correct.  This is a missed on my part, thank you for pointing this out.  I will get this updated.


Improved titles of merit order charts and Sankey's

Closes #2137 and #2139 . Assigning @dennisschoenmakers , feel free to re-assign.

We say 'use', but shouldn't we rather say 'load'?

Load of flexibility options on hourly basis
Load of flexibility options per hour

We say 'use', but shouldn't we rather say 'load'?
And, what do you, @AlexanderWirtz, think is better?

@dennisschoenmakers
Looking at the Dutch translations, I gather @jorisberkhout means 'use' in the sense of 'deployment' or 'utilization', not 'energy use'.

Looking at the Dutch translations, I gather @jorisberkhout means 'use' in the sense of 'deployment' or 'utilization', not 'energy use'.

Correct, I meant 'use' as the translation of the Dutch 'inzet'. 'usage' would be a possibility, too. What do you think, @dennisschoenmakers ?
I agree with the 'per hour' and will commit that change to this pull request.

@dennisschoenmakers
Looking at the Dutch translations, I gather @jorisberkhout means 'use' in the sense of 'deployment' or 'utilization', not 'energy use'.

Agree!

:+1: Excellent!


Add support for Packer

Added couple of colours for Packer which should make it easier to read change logs.
Before:

After:

Thank you! Sorry, read your comment too late, wouldn't mind changing commit message, of course.

No worries! I just figured I'd do it myself because it was such a minor thing :sweat_smile:


Feature suggestion - Convolver presets.

I was quite amazed how comprehensive the audio web api really is.
Most astonishing feature i have found unexplored by web devs is convolution. Seen some demos but no real usage - like theatrical mode for youtube or binural audio for sound cloud.
Feel like this extension could make use of some small set of convolution preset on top of equalization.
To gave a feel of space for headphones by applying binural HRIR or expand stereo effect for movie with some theatrical space simulation IRs.

Thanks for suggestions. Once I have time again I will look into it.


Borderless windows do not allow to select the first line

With sway 0.11 rc3, you can not select the first line in borderless windows

Select... the first line?

yeah, like the terminal double clicking on a word

Can reproduce.


ClangImporter: pass _WASI_EMULATED_MMAN when triple is WASI

Defining _WASI_EMULATED_MMAN is required when compiling with WASI libc.

Related to SR-9307.
(cc @compnerd)

Is there no intention of ever supporting mmap on WASI?  If so, this is fine, otherwise, I think that we should just say that users are required to pass -Xcc -D_WASI_EMULATED_MMAN as embedding this into the driver makes that migration more difficult.

It doesn't look like they're going to support it in this form without emulation:

And, some features of mmap can lead to userspace triggering signals. Accessing memory beyond the end of the file, including in the case where someone else changes the size of the file, leads to a SIGBUS on POSIX-style systems. Protection modes other than PROT_READ|PROT_WRITE can produce SIGSEGV. While some VMs are prepared to catch such signals transparently, this is a burdensome requirement for others.
Another issue is that while WASI is a synchronous I/O API today, this design may change in the future. mmap can create situations where doing a load can entail blocking I/O, which can make it harder to characterize all the places where blocking I/O may occur.
And lastly, WebAssembly linear memory doesn't support the semantics of mapping and unmapping pages. Most WebAssembly VMs would not easily be able to support freeing the memory of a page in the middle of a linear memory region, for example.
To make things easier for people porting programs that just use mmap to read and write files in a simple way, WASI libc includes a minimal userspace emulation of mmap and munmap.

I would assume that if it's ever supported, WASI would provide a different API for it. I think it would still make sense to use the current emulation API by default.

Defaults here are problematic because WASI is still in flux.  I really would prefer that we document the differences and make users aware of them and intentionally be explicit about the fact that there are differences from standard expectations.

@compnerd since the original commit we've done plenty of changes on the WASI/Wasm side of things, for example we're requiring users to rely on import WASILibc instead of import Glibc to be explicit about the differences in libc behavior. Do you think that's explicit enough?
From that perspective the exact way mmap is implemented is an implementation detail. We could still require users to pass -Xcc -D_WASI_EMULATED_MMAN every time they build for WebAssembly. But given how WASI libc is designed, it looks like we'll have to require users targeting WASI to do it for indefinite time in the future. The vast majority of Swift users won't be concerned about this low-level implementation detail, and for those who care import WASILibc should be explicit know. WDYT?

@swift-ci please smoke test

Is mman emulation the default and standard behaviour? Can the wasm VM not support it?

It isn't a Wasm feature, but a feature of WASI libc, so in that sense the answer is "yes". It's default and "standard" as long as we rely on WASI libc, I'm not aware of other libc implementations for Wasm. If we require our users to pass -fwasm-features +emulated-mman, they'll have to do it every time they target Wasm and WASI.
I don't think there's a way to build Swift stdlib with WASI, but without _WASI_EMULATED_MMAN.

I think that my concern is that the libc itself doesn't define it.  Another option to consider, if we can get it to work - add a header that is implicitly pulled in by the WASILibc that defines the macro.  That should hopefully prevent he WASILibc module from being built, and it would also define the proper macros as necessary.


Can I customise button UI?

The application I'm working on requires a different UI to what's provided here, is it possible to update this package to allow for customised buttons?

That is a great idea @moosefaceee, it would be great to allow different UI, should be easy to implement I think. Any ideas so far


Fix Envoy config to be valid as an extension

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/lgtm

/approve


use return value from onCellEdit as new cell value

fixes #631

@bontibon, thanks your contribution :)


Doesn't work.  It just doesn't work

Android version of ByeDpi does not work.  The video still loads at the speed it was during the slowdown.

You miss. It's not repository of ByeDPIAndroid. Go here.


Fix dependency warnings

This PR:

Fix dependency warnings  (used npm-check to generated new package.json)
Fix linter errors caused by new lib versions
Add package-lock.json to ensure a consistent install and compatible dependencies

Dependency warnings:
$ npm i
npm WARN deprecated<EMAIL_ADDRESS>1.2.0 should have been a major version bump
npm WARN deprecated<EMAIL_ADDRESS>🙌  Thanks for using Babel: we recommend using babel-preset-env now: please read babeljs.io/env to update!
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
npm WARN deprecated<EMAIL_ADDRESS>gulp-util is deprecated - replace it, following the guidelines at https://medium.com/gulpjs/gulp-util-ca3b1f9f9ac5

Linter errors:
~/react-redux-boilerplate/app/containers/RepoListItem/RepoListItem.js
  27:71  error  Using target="_blank" without rel="noopener noreferrer" is a security risk: see https://mathiasbynens.github.io/rel-noopener  react/jsx-no-target-blank
  30:84  error  Using target="_blank" without rel="noopener noreferrer" is a security risk: see https://mathiasbynens.github.io/rel-noopener  react/jsx-no-target-blank

~/react-redux-boilerplate/app/utils/injectReducer.js
  17:5  error  contextTypes should be placed after displayName  react/sort-comp

~/react-redux-boilerplate/app/utils/injectSaga.js
  21:5  error  contextTypes should be placed after displayName  react/sort-comp

Thank you!


system backup failed

Discussed in https://github.com/longhorn/longhorn/discussions/5517

Originally posted by linpanru March  9, 2023

i am a new longhorn user, by longhorn ui ,i create a systembackup, but  it failed finally, please help me

Already been fixed in https://github.com/longhorn/longhorn/issues/5286.
Closing.


Avoid allocation of String during formatting.

The format function is now passed a &mut std::io::Write trait object
and is responsible for outputting the formatted message to it. Format
functions can take care (by using the std::fmt macros) to avoid heap
allocations during formatting/outputting of log messages, and the
default format function takes advantage of this.
Alternatives considered:

Have the format function return a fmt::Arguments (from a call to
format_args!(). Unfortunately I couldn't find a way to make this
work with the lifetime parameter on fmt::Arguments.
Give the format function something like fern's FormatCallback:
https://docs.rs/fern/0.4.2/fern/struct.FormatCallback.html rather
than giving it the &mut Write. This would be more future proof, but
I worry it introduces unnecessary abstraction into env_logger,
which generally tries to keep things simple.

... but I don't think either of those are better.
Some open questions:

Is referring to it as a 'format function' still appropriate? Should
Builder::format() be renamed?
Should we be backwards-compatible with current env_logger by allowing
people to give a Fn(&Record) -> String still? (Should we make
upgrades easier or encourage people to rethink their format functions
and whether they heap allocate?)

Fixes #3.

If it's desirable to add color support (#2) then perhaps we should pass some kind of new Output type to the format functions (rather than std::io::Write). This would allow us to add an Output::supports_ansi_colors() method. Formatters could use this in order to decide whether to use ansi_term to output colored logs on ANSI terminals, or to output non-colored logs for non-ANSI terminals.
We could also build the machinery for outputting colors in a cross-platform way into the Output type, but that seems like massive scope creep. :)

@mjkillough color support is not in the current scope but something we could look at in the future. For now I think this is completely fine.
Thanks!

Hi,
Is there any actual need for this PR? The problem is that unbuffered writes of logs aren't very good if you have multiple threads or even multiple processes. I.e you will get a mess of different pieces of different log messages. Previous implementation added \n afterwards and even that was a problem. So I think the good idea is to make better buffering as added in https://github.com/rust-lang-nursery/log/pull/219 rather than remove buffering.

I think this is a valid point. Thanks for following this up @tailhook! I'll open up an issue shortly to discuss sinks so we can iterate towards an implementation that satisfies all the things we expect from them.


graph-tool not available on M1 via conda

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:
graph-tool is not available on M1.
$ uname -m
arm64
$ wget https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh
$ chmod +x Miniforge3-MacOSX-arm64.sh
$ ./Miniforge3-MacOSX-arm64.sh
$ exit # exit to get a conda base environment
(base) $ conda install graph-tool graph-tool-base
Collecting package metadata (current_repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.
Collecting package metadata (repodata.json): done
Solving environment: failed with initial frozen solve. Retrying with flexible solve.

PackagesNotFoundError: The following packages are not available from current channels:

  - graph-tool-base
  - graph-tool

Current channels:

  - https://conda.anaconda.org/conda-forge/osx-arm64
  - https://conda.anaconda.org/conda-forge/noarch

Note that there seems to be no fundamental obstacle to getting graph-tool on M1, as brew install graph-tool succeeds.

Environment (conda list):

$ conda list
# packages in environment at /Users/nathompson7/miniforge3:
#
# Name                    Version                   Build  Channel
brotlipy                  0.7.0           py39h5161555_1001    conda-forge
ca-certificates           2021.5.30            h4653dfc_0    conda-forge
certifi                   2021.5.30        py39h2804cbe_0    conda-forge
cffi                      1.14.6           py39hda8b47f_0    conda-forge
chardet                   4.0.0            py39h2804cbe_1    conda-forge
charset-normalizer        2.0.0              pyhd8ed1ab_0    conda-forge
colorama                  0.4.4              pyh9f0ad1d_0    conda-forge
conda                     4.10.3           py39h2804cbe_0    conda-forge
conda-package-handling    1.7.3            py39h5161555_0    conda-forge
cryptography              3.4.7            py39h73257c9_0    conda-forge
idna                      3.1                pyhd3deb0d_0    conda-forge
libcxx                    12.0.1               h168391b_0    conda-forge
libffi                    3.3                  h9f76cd9_2    conda-forge
ncurses                   6.2                  h9aa5885_4    conda-forge
openssl                   1.1.1k               h27ca646_0    conda-forge
pip                       21.2.2             pyhd8ed1ab_0    conda-forge
pycosat                   0.6.3           py39h5161555_1006    conda-forge
pycparser                 2.20               pyh9f0ad1d_2    conda-forge
pyopenssl                 20.0.1             pyhd8ed1ab_0    conda-forge
pysocks                   1.7.1            py39h2804cbe_3    conda-forge
python                    3.9.6           h54d631c_1_cpython    conda-forge
python_abi                3.9                      2_cp39    conda-forge
readline                  8.1                  hedafd6a_0    conda-forge
requests                  2.26.0             pyhd8ed1ab_0    conda-forge
ruamel_yaml               0.15.80         py39h5161555_1004    conda-forge
setuptools                49.6.0           py39h2804cbe_3    conda-forge
six                       1.16.0             pyh6c4a22f_0    conda-forge
sqlite                    3.36.0               h72a2b83_0    conda-forge
tk                        8.6.10               hf7e6567_1    conda-forge
tqdm                      4.62.0             pyhd8ed1ab_0    conda-forge
tzdata                    2021a                he74cb21_1    conda-forge
urllib3                   1.26.6             pyhd8ed1ab_0    conda-forge
wheel                     0.36.2             pyhd3deb0d_0    conda-forge
xz                        5.2.5                h642e427_1    conda-forge
yaml                      0.2.5                h642e427_0    conda-forge
zlib                      1.2.11            h31e879b_1009    conda-forge

Details about  conda and system ( conda info ):

$ conda info
(base) ➜  reproduce conda info

     active environment : base
    active env location : /Users/nathompson7/miniforge3
            shell level : 1
       user config file : /Users/nathompson7/.condarc
 populated config files : /Users/nathompson7/miniforge3/.condarc
                          /Users/nathompson7/.condarc
          conda version : 4.10.3
    conda-build version : not installed
         python version : 3.9.6.final.0
       virtual packages : __osx=11.5.1=0
                          __unix=0=0
                          __archspec=1=arm64
       base environment : /Users/nathompson7/miniforge3  (writable)
      conda av data dir : /Users/nathompson7/miniforge3/etc/conda
  conda av metadata url : None
           channel URLs : https://conda.anaconda.org/conda-forge/osx-arm64
                          https://conda.anaconda.org/conda-forge/noarch
          package cache : /Users/nathompson7/miniforge3/pkgs
                          /Users/nathompson7/.conda/pkgs
       envs directories : /Users/nathompson7/miniforge3/envs
                          /Users/nathompson7/.conda/envs
               platform : osx-arm64
             user-agent : conda/4.10.3 requests/2.26.0 CPython/3.9.6 Darwin/20.6.0 OSX/11.5.1
                UID:GID : 501:20
             netrc file : None
           offline mode : False

We can't create an osx-arm64 build of this feedstock until all of its dependencies have been ported first.  Fortunately, it looks like that might not take too long.  I wrote a little script to call conda search --subdir osx-arm64 --json ${dep} on each of graph-tool's dependencies.  As far as I can tell, only cairomm and its dependency sigcpp are currently lacking M1 support.
Just for fun, here's their relationship to graph-tool:

I have no idea when to expect those last two dependencies to become available.
Since cairomm is only used for drawing, I suppose it would be technically feasible to provide graph-tool-base for M1 right now...

FWIW, A description of the conda-forge effort to provide M1 packages can be found in this blog post.  According to those notes, a bot will automatically open a PR for our feedstock once all of the dependencies are available.  I guess we can just wait for that to happen.

@stuarteberg : Thanks so much for your input! The diagram is very helpful.

@NAThompson BTW, conda-depgraph is a handy little tool for creating such diagrams.

Any updates or workarounds for this? I want to have all my packages installed via miniforge, not have graph-tool installed  through brew and then use some hacky linking.

Any updates or workarounds for this?

Not yet.  I think we'll need a resolution to the following question before progress can be made on cairomm and then graph-tool itself.

https://github.com/conda-forge/libsigcpp-feedstock/issues/23

I don't know why the bots haven't automatically pushed M1 support to that feedstock.
It appears that libsigcpp feedstock lacks dedicated maintainers.  (The only listed maintainers are conda-forge core developers, who are probably quite busy with other things.)  If you want to prod things along, you can try asking for advice on the conda-forge gitter channel.

The above issue with libsigcpp is still unresolved, so a drawing-capable M1 port is still not possible.  But now that #28 has been resolved, it should be possible to build graph-tool-base for the M1 package.  But I don't know how many users would actually be interested in that.

Now that the issue above for libsigcpp has been fixed, what is the status on this? Is there anything else blocking cairomm?

I tried to make progress via #73, but I got stuck with an error I didn't understand. Someday when I have time, I might ask for help on gitter, but this week is definitely bad for me.

Any updates or workarounds for this? I want to have all my packages installed via miniforge, not have graph-tool installed through brew and then use some hacky linking.

@chrisvander as this is still not solved, can you please elaborate a bit more on hacky linking? How do you link graph-tool to conda?

@tomaskourim I got a base version of graph-tool working on Apple Silicon by first installing graph-tool via homebrew. I then added the location of the graph-tool site-packages folder to my python path. For me the location was /opt/homebrew/Cellar/graph-tool/2.44_1/lib/python3.9/site-packages  .
I use PyCharm so this was relatively easy to add under the interpreter preferences. However I believe you can also do so in the script with the following.
import sys
sys.path.append('/opt/homebrew/Cellar/graph-tool/2.44_1/lib/python3.9/site-packages')
from graph_tool.all import *

It is worth noting that I found my way here because I still get runtime errors related to cairo draw and will not be able to draw graphs. This makes sense given the dependency issues discussed above.

@tomaskourim I got a base version of graph-tool working on Apple Silicon by first installing graph-tool via homebrew. I then added the location of the graph-tool site-packages folder to my python path. For me the location was /opt/homebrew/Cellar/graph-tool/2.44_1/lib/python3.9/site-packages .
I use PyCharm so this was relatively easy to add under the interpreter preferences. However I believe you can also do so in the script with the following.
import sys
sys.path.append('/opt/homebrew/Cellar/graph-tool/2.44_1/lib/python3.9/site-packages')
from graph_tool.all import *

It is worth noting that I found my way here because I still get runtime errors related to cairo draw and will not be able to draw graphs. This makes sense given the dependency issues discussed above.

Thanks. Adding to path did not work for me (but I can try the sys.path.append as well), what helped is to copy files from homebrew installation directly into the conda site-packages. Super hacky, I would say. Now I have to deal with CBC packages, having the same problems...

@NAThompson @tomaskourim @Minyall
We may have a working build of graph-tool for M1 macs, via #73.  Before we publish it, would one of you be willing to test it?  Here's how to download the (unpublished) build artifacts:

Go to the build artifacts listing for our successful build.

Download the tarball for your platform.  (If you can't find the Download button, check out this little video.)

Unzip the contents (twice since it's a nested zip) and install it to a conda environment:
cd ~/Downloads
unzip conda_artifacts_20220524.8.1_osx_arm64_numpy1.21python3.10.____cpython.zip
cd conda_artifacts_20220524.8.1_osx_arm64_numpy1.21python3.10.____cpython
unzip graph-tool-feedstock_conda_artifacts_20220524.8.1_osx_arm64_numpy1.21python3.10.____cpython.zip
cd graph-tool-feedstock_conda_artifacts_20220524.8.1_osx_arm64_numpy1.21python3.10.____cpython
conda create -n test-gt -c file://$(pwd) graph-tool-base
conda activate test-gt

Try a few functions and report back here!

@NAThompson @tomaskourim @Minyall @chrisvander
Update: We know the artifacts I linked to in the above message are broken.  We're now rebuilding #73, so don't test anything until that is done.


Yices converter and internalizer

Add Yices converter and  internalizer

Please, fix tests.

Yices API to delete unused terms:
https://yices.csl.sri.com/doc/misc-operations.html#garbage-collection
https://github.com/SRI-CSL/yices2_java_bindings/blob/d9858e540425072443830d2638db5ffdc8c92cd1/src/main/java/com/sri/yices/Yices.java#L457


get age of birth date

This is a wonderful tool!
Is it also possible to retrieve the age or birth date of the users somehow?

@heeringa0, I doubt


Removed duplicate properties "list-style"

What do the changes you have made achieve?
Removed duplicate properties "list-style"
Are there any new warning messages?
No
Have you written tests?
No
Have you included relevant documentation
No
Which issues does this resolve?
<DCO 1.1 Signed-off-by<EMAIL_ADDRESS>

Coverage remained the same at 97.532% when pulling c90d2cb82ea744bf2d49ac1af01a38280fdabf06 on glizer:removed-duplicate-prop into 52ccf2e1fa3e4d17d9f8f7bd748f7caf20830f30 on sasstools:develop.


(maint) Add back package:bootstrap and package:implode for compatibility

CLA signed by all contributors.


Ajustando classe com base no sonar

Codecov Report

Merging #5 (1279a58) into main (d16013c) will decrease coverage by 3.37%.
The diff coverage is 64.51%.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
- Coverage   98.57%   95.20%   -3.38%
==========================================
  Files           5        5
  Lines         630      646      +16
==========================================
- Hits          621      615       -6
- Misses          9       31      +22

Flag
Coverage Δ

unittests
95.20% <64.51%> (-3.38%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/UfType.cs
94.69% <46.15%> (-5.31%)
:arrow_down:

src/CpfType.cs
96.00% <61.11%> (-4.00%)
:arrow_down:

src/PisType.cs
95.09% <61.11%> (-4.28%)
:arrow_down:

src/CepType.cs
99.34% <92.30%> (-0.66%)
:arrow_down:

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 d16013c...1279a58. Read the comment docs.


[Feature request] 3rd option for debuggers, to fade parts that are occluded

A option, to set how much is it faded out, would not hurt. Maybe it can be just a color, and plugin developer can decide how occluded parts look?


Fix left-aligned EditorHelp when first opened

Fixes https://github.com/godotengine/godot/issues/69773.
In https://github.com/godotengine/godot/pull/68132, the theme was wrongly updated because there's a "middle" style being generated from to the background's style that increases the margins depending on the window's size. This was not accounted for.

Yeah, that's how it should be. It technically doesn't make much sense to change the margins between focus and unfocus, or does it?

Overall it doesn't look horrible. Maybe its just me, but I don't like all the empty space.

The margin isn't expected to change when you (un)focus text. Regarding the empty spaces on the side, it's done so that each line of text doesn't become too long, as readability is decreased if lines span more than ~100 characters.

Thanks!


I have started using latest dexie and faced DB access error.

I couldn't able to see any error from the Dexie but request to the DB is not responding. I mean the transactions are not processing at all. I'm using version 1.5.1 .  FYI , I've been using very old version of Dexie before upgrading to latest. Can i get any help on this. Thanks.

Hey valishah,
can you provide us with a simple example that demonstrates the issue? That would be great.


Like inet_aton function in MySQL, return the numeric value of an IPv4 address.

For example :
select inet_aton('<IP_ADDRESS>');
3232235944
Firstly, we can access converting ipv4 to numeric value. Then ipv6, and conversion between IP and numerical value.

https://dev.mysql.com/doc/refman/5.7/en/miscellaneous-functions.html#function_inet-aton

@wangsimo0 can you assign this task to me? I would like to try.


Add tracing module/response functionality from openethereum

Motivation
https://openethereum.github.io/JSONRPC-trace-module
This module is apparently necessary for getting Gnosis Safe contracts deployed to non-Ethereum native blockchains. We've been looking into it in the Edgeware community and would like to propose integrating it here (even as a feature, maybe as a grant in the future).
Suggested Solution
Integrate these RPC calls.
Additional Information
Any ideas from anyone in the Frontier community?

Can this be considered as a duplicate of #703 ?
There is a history of discussion there, so keeping track of it from a single place may be better.

I think it it. Close it.


Update spree_payu_integration.gemspec

Each spree_core version should be on a separate branch. Will update it here https://github.com/netguru/spree_payu_integration/pull/12


[iRonCub-Mk3] Add Jetpack FTs configuration files

Adds also the jetpack FTs to the robot launch files.

Nice!

Thanks @davidegorbani
Some notes:
With this PR, the YARP ports /icub/left_arm/FT/measures:o will publish the force/torque and the temperature of both l_arm_ft_sensor and l_jetpack_ft_sensor.
And /icub/right_arm/FT/measures:o will publish the force/torque and the temperature of both r_arm_ft_sensor and r_jetpack_ft_sensor.
Also, the robot can't be launched with this configuration if the jetpack is unmounted.

Thanks guys!


[3.0] Fix bsc#1072242: Map the keyboard file into velum container.

Map the keyboard file from the admin node into the valum container to make
keyboard defined in YaST available.
Signed-off-by: Florian Bergmann<EMAIL_ADDRESS>(cherry picked from commit fda5fa463a34d67b8e7d2686ac8790514e101e4c)

 Bug 1072242 - Keyboard selection not reflected in AutoYast file


bracket-less numbered format

It appears that csl only supports full citations, so I expect (maybe wrongly) that pandoc-citeproc is handling the formatting of [@ref1] vs @ref1 itself? If this is the case I have a question regarding numbered csl styles. Is there any way to add author names to the bracket-less style so that @ref1 becomes "John Doe (1)" rather than just "(1)" in the same way as it becomes "John Doe (2016)" with author-year csl styles...
If this is not something that is handled by pandoc-citeproc, but is indeed defined in the csl I would be extremely happy if someone could point me towards a description of how to specify it...
best
Thomas

As this is to be used in-text, I believe a fairly generic formatting should work. Still can't believe that this isn't part of the csl spec.
Thanks for considering


CORS blocks API requests from frontend to backend app with internal ingress.

Please provide us with the following information:

This issue is a: (mark with an x)

[x] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
I have a frontend and backend container in my containerapps environment, frontend is React-TS built with Vite & served with nginx, and backend API is Python/FastAPI. The app works fine when ingress is set to external on the backend - I have the CORS origin set for the frontend fqdn in ACA CORS and FastAPI CORS Middleware - but when I flip ingress to internal, which should still work according to the ACA examples, I get:
Access to XMLHttpRequest at 'https://backend.{randomname}-{id}.centralus.azurecontainerapps.io/api/' from origin 'https://frontend.{randomname}-{id}.centralus.azurecontainerapps.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource.
I tried it with setting allowed CORS origins to all in ACA and got the same error. I also tried just using the app name for the requests e.g. https://backend and got same error. I certainly could be missing something but I'm not sure what, and this seemed odd enough to submit an issue since everything works just fine with external ingress.
Steps to reproduce

Change ingress to internal on backend containerapp

Expected behavior [Frontend app in the same environment can send requests to backend app with internal ingress without CORS origin error. CORS config works fine when ingress is set to external on backend app]
Actual behavior [CORS origin error. See above]

I figured it out, I think. Misunderstood internal ingress apparently, assuming this is accurate: https://stackoverflow.com/questions/75949766/azure-container-apps-cors-issue.
I guess to do this sort of thing you need a proxy to forward the requests, which seems like is what's going on here with the express server:
https://github.com/Azure-Samples/container-apps-connect-multiple-apps/tree/main/with-fqdn


Add beta warning in UI

To do:

[ ] add beta notification to splash screen

[ ] option to dismiss (for current session only - warning must be shown again on app relaunch)
[ ] link to xlite docs (or URL if no link can be added for security reasons)

[ ] add beta badge next to the logo for all main pages

Ref:
See UI designs attached:

Please find me on https://www.upwork.com/freelancers/wasil
Working on 5th upwork job with amazing reviews on previous ones. Can complete the job in 1-3 hours with 💝

Sure

Sure

you may invite me to the job. I want to work on it. Know a lot about notification libs in react. a js king i am


keep() and remove() does not work for all HTML tags

Some tags such as  and  do not work when passed in the keep() and remove() methods. The tags are still converted to markdown even when it was instructed to either keep the tags or remove the tags.
Here is an example to reproduce this...
Turndown v7.1.3

Because current implementation uses the pre-defined rule before keep-rules and remove-rules. The fix is changing order of rules applied, simply. But I can't say it's backward compatible.
The same issue is happen when user defined the custom rule of strikethrough(del tag) or underline(ins tag).


update(mediatype/video): support for identify mediatype of video disc images

this PR adds a new array for file extensions found on Blu-ray and DVD disc images (excluding single image files - iso/img/etc) and adds a case to the switch in getMediaType for unparsable titles/folders that contain these to be identified as the VIDEO mediaType

Should VIDEO_DISC_EXTENSIONS also be added to parseTitle and nonVideoSizeRatio? Also do we still want to ignore the disc folders in dataFiles now that explicit support is added?

Should VIDEO_DISC_EXTENSIONS also be added to parseTitle and nonVideoSizeRatio? Also do we still want to ignore the disc folders in dataFiles now that explicit support is added?

As far as I'm aware, prefiltering will exclude all the folders that actually contain the files matching the extensions in VIDEO_DISC_EXTENSIONS anyway, It's only real utility is in checking the searchee.files for mediaType.
I'm not sure we should really add these to be considered video files, but I don't particularly see how it could hurt I suppose - I don't download full disc images like that so it doesn't really affect me and I'm not sure the interaction. Seems like it would make sense though given what nonvideo is used for in the majority of cases.
As far as parsing goes, none of those files will ever contain any useful information for parsing.


Update athom-rgbww-light.yaml

Added min_power and max_power to each output with required values. This prevents leds from turning off when below 7% brightness in Home Assistant

It looks like this broke the ability to shut off the bulb completely. It needs to have zero_means_zero: true added. I'll open a PR shortly


Fix Mesh dashboard table ordering issue and correct the wrong table cell references

In Mesh dashboard, the table say for HTTP/GRPC Workloads we have actually without "hidden" property set to be "true". The full table looks like:

we should sort by col: 5 (Requests) by default, not col: 4
Also a fix for wrong referencing the cell number.
Verified the fix, now the default "descending" order by "Requests" column number is correct:

[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@carolynhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3e204b34bb63eb2774b017ea8d157407c15c9c81
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@carolynhu: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3e204b34bb63eb2774b017ea8d157407c15c9c81
link
/test release-notes_istio

integ-pilot-multicluster-tests_istio
3e204b34bb63eb2774b017ea8d157407c15c9c81
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


chore(infra): upgrade project for 2020

closes #2
closes #3

fixed all Buffer deprecated warnings (also in tests).
refactored code to use classes and other es2015 syntax (let/const; arrow functions; destructuring). Requires Node 8+.
Travis now tests on existing node versions (8/10/12/13)
use npm test in Travis (and removed "test-travis" script), as coveralls is not currently integrated (badge removed in previous commits).
replaced deprecated istanbul with nyc. renamed script to "coverage".
ensure published package doesn't include test folder by defining "files" field in package.json (changelog/license/readme are always included)
upgrade all dependencies to latest versions.
adjusted git to ignore lock files and nyc output.

Hey @andrewrk, any chance you can review this? :)

ping @andrewrk
would be awesome to get this in and released...

Fork it

Appreciate the response.
If project is unmaintained, please consider archiving repository and/or deprecating npm package (which has over 5 million weekly downloads).

@AviVahl looks very neat, will use some of that code in my fork 👍
@andrewrk can you deprecate the npm package? 🙏


Attempt to resolve reflection

As an exercise to better understand how it uses clong I tried to resolve some reflective calls.
Note that I haven't figured out how to try this library yet, so if this is useful please run some tests before merging.

Thanks! I'll take a look!
Fyi, I just added a status section to the readme.


make sure the current working dir is in the sys.path

rationale
I maintain multiple load tests, each with their own locustfile.  They each
share helper utilities located at the project root, but locust/main.py loads
the locustfile in a way that does not make importing the helpers easy.  I have
to add the following line to every locustfile to make sure the helper utilities
are importable:
sys.path.append(os.getcwd())

Then I invoke locust from the project root:
locust -f path/to/locustfile

I think it's understandable why locust/main.py inserts the locustfile
directory into the sys.path before loading it.  I also think it would be
harmless to also make sure the current working directory is in the sys.path.
minimal example
directory structure:
$ find
.
./__init__.py
./helpers
./helpers/__init__.py
./loadtests
./loadtests/__init__.py
./loadtests/test1
./loadtests/test1/locustfile.py
./loadtests/test1/__init__.py
./loadtests/test2
./loadtests/test2/locustfile.py
./loadtests/test2/__init__.py

helpers/__init__.py:
print("helpers imported!")

loadtests/test1/__init__.py:
from locustfile import MyLocustClass

loadtests/test1/locustfile.py:
"""
Load test suite 1
"""
from locust import Locust, task, TaskSet

# our custom shared helper modules
import helpers

class MyTaskSet(TaskSet):
    @task(1)
    def some_task(self):
        pass

class MyLocustClass(Locust):
    task_set = MyTaskSet

testing running locust without this patch
Before this change, the common helpers module could not be found.
$ locust -f loadtests/test1
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: Traceback (most recent call last):
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/bin/locust", line 9, in <module>
[2016-09-27 14:00:48,712] carbon/ERROR/stderr:
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: load_entry_point('locustio==0.7.3', 'console_scripts', 'locust')()
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/local/lib/python2.7/site-packages/locustio-0.7.3-py2.7.egg/locust/main.py", line 349, in main
[2016-09-27 14:00:48,712] carbon/ERROR/stderr:
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: docstring, locusts = load_locustfile(locustfile)
[2016-09-27 14:00:48,712] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/local/lib/python2.7/site-packages/locustio-0.7.3-py2.7.egg/locust/main.py", line 321, in load_locustfile
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: imported = __import__(os.path.splitext(locustfile)[0])
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/local/lib/python2.7/site-packages/gevent-1.1.1-py2.7-linux-x86_64.egg/gevent/builtins.py", line 93, in __import__
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: result = _import(*args, **kwargs)
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/example/loadtests/test1/__init__.py", line 1, in <module>
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: from locustfile import MyLocustClass
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/local/lib/python2.7/site-packages/gevent-1.1.1-py2.7-linux-x86_64.egg/gevent/builtins.py", line 93, in __import__
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: result = _import(*args, **kwargs)
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/example/loadtests/test1/locustfile.py", line 10, in <module>
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: import helpers
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: File "/home/tsankey/Documents/perf-357/locust_minimal_example/venv/local/lib/python2.7/site-packages/gevent-1.1.1-py2.7-linux-x86_64.egg/gevent/builtins.py", line 93, in __import__
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: result = _import(*args, **kwargs)
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: ImportError
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: :
[2016-09-27 14:00:48,713] carbon/ERROR/stderr: No module named helpers
[2016-09-27 14:00:48,713] carbon/ERROR/stderr:

testing running locust with this patch
$ locust -f loadtests/test1
[2016-09-27 13:59:24,798] carbon/INFO/stdout: helpers imported!
[2016-09-27 13:59:24,798] carbon/INFO/stdout:
[2016-09-27 13:59:24,798] carbon/INFO/locust.main: Starting web monitor at *:8089
[2016-09-27 13:59:24,799] carbon/INFO/locust.main: Starting Locust 0.7.3

fyi  @cpennington

Your use case is very similar to how I and many others use locust. Unfortunately, I'd be worried about this breaking people's integrations or having unintended side effects depending on where you are running from and if you have modules or packages with the same name across your project.
What I'd like to see before this gets merged is some best practices documentation for structuring a non-flat locust project. Trying to standardize project structures and import paths to some extent will make them easier to support in the future. I made https://github.com/locustio/locust/issues/500 to track that separately, but feel free to take a first stab if you're up for it.

First off, thanks!
Just a bump!
Will this be revisited now that #500 is fulfilled via #501 ?

@cgoldberg what do you think about this one?

Sorry to bump on this one, but is it likely to get merged at all? @justiniso @cgoldberg
(I know it can be done manually in test files, but it'd be nice not to have to).

@heyman What is your opinion on this? I agree that it may be a bit disruptive to existing tests because it changes how locust runs. But I do think people expect to be able to import modules in the same fashion as if they had run python somelocustfile.py instead of locust -f somelocustfile.py

I understand the issue but I'm a little bit hesitant because of potential negative effects it might have (that are currently unknown to me). But I think I'm +0 at the moment.
#1112 that was just merged makes it possible to avoid the issue by starting locust using python -m locust -f locustfiles/myfile.py ... instead where imports from CWD should work.

@heyman merge or reject, I think it is up to you :)
Like I said, I'm +1, but I dont need this change myself, so it's not that important to me...

Okay, let's merge it. It solves a real issue, and if we realize that it creates other issues at some point in the future we can always re-evaluate :).


Look back at user stories to come up with buttons for each page

Commented on each Usert story and assigned general issues to each epic


sbt scripted not working?

There's something fishy going on with:
https://github.com/cquiroz/sbt-locales/blob/master/build.sbt#L50
and
https://github.com/cquiroz/sbt-locales/blob/master/build.sbt#L60
If I remove them, I can do a sbt +publishLocal; scripted

I don't test locally with 0.6 but it is exercised on CI.

@cquiroz the lines are setup to ignore 1.x  though, not 0.6?

Ok, this is fairly old, probably when 0.6 was the default. I have changed the style on other
projects to default to 1.x and ignore publication on 0.6
See
https://github.com/olafurpg/sbt-ci-release#how-do-i-publish-cross-built-scalajs-projects


Attempted to load class "Sms77SyliusPlugin" from namespace "Sms77\SyliusPlugin". Did you forget a "use" statement for another namespace?

i added Sms77\SyliusPlugin\Sms77SyliusPlugin::class => ['all' => true], the i got error:
Attempted to load class "Sms77SyliusPlugin" from namespace "Sms77\SyliusPlugin". Did you forget a "use" statement for another namespace?

Hi,
I think the README is lacking here.
You need to install the package via composer first:
Run composer require https://packagist.org/packages/sms77/sylius-plugin in a terminal and go on with the steps from the README.


Broken on flake8>=3.8.0

See this issue.

Just wanted to follow up here.
I installed the tool from HEAD on master and tested it locally to confirm that it works.
I reached out to @kataev, to see if he'd be willing to bump & polish a newer version or let me contribute if he's open to it.


application crashes when internet is down

Hey, when I am trying to use the download method, while the video is downloading if the internet disconnects the application crashes, also when I use the stream and search methods, if there is no internet the application crashes.
how can I handle it?

Hello, I'm working on several improvements, soon this will be improved. But for now you can set a timeout on InnerTube.java requests.
package javatube;

import javax.net.ssl.HttpsURLConnection;
import java.io.*;
import java.net.HttpURLConnection;
import java.net.URL;
import java.net.URLConnection;

class InnerTube{
    public static String post(String param, String data) throws IOException {
        StringBuilder output = new StringBuilder();
        URL url = new URL(param);

        URLConnection conn = url.openConnection();
        conn.setConnectTimeout(5000);
        conn.setReadTimeout(10000);
        conn.setDoOutput(true);
        conn.setRequestProperty("Content-Type", "application/json");
        conn.setRequestProperty("Content-Length", Integer.toString(data.length()));

        try (DataOutputStream dos = new DataOutputStream(conn.getOutputStream())) {
            dos.writeBytes(data);
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
        try (BufferedReader bf = new BufferedReader(new InputStreamReader(
                conn.getInputStream())))
        {
            boolean keepGoing = true;
            while (keepGoing) {
                String currentLine = bf.readLine();
                if (currentLine == null) {
                    keepGoing = false;
                } else {
                    output.append(currentLine);
                }
            }
        } catch (IOException e) {
            throw new RuntimeException(e);
        }
        return output.toString();
    }

    public static ByteArrayOutputStream get(String videoUrl) throws IOException {

        URL urlObj = new URL(videoUrl);
        HttpURLConnection connection = (HttpURLConnection) urlObj.openConnection();
        connection.setRequestMethod("GET");
        connection.setConnectTimeout(5000);
        connection.setReadTimeout(10000);
        int responseCode = connection.getResponseCode();
        if (responseCode == HttpURLConnection.HTTP_OK) {
            InputStream in = new BufferedInputStream(connection.getInputStream());
            ByteArrayOutputStream out = new ByteArrayOutputStream();
            byte[] buf = new byte[1024];
            int n;
            while ((n = in.read(buf)) != -1) {
                out.write(buf, 0, n);
            }
            out.close();
            in.close();
            connection.disconnect();
            return out;
        } else {
            throw new IOException("Connection fail. Response code: " + responseCode);
        }
    }

    public static String downloadWebPage(String webpage) throws IOException {
        URL url = new URL(webpage);
        HttpsURLConnection con = (HttpsURLConnection) url.openConnection();
        con.setRequestProperty ( "accept-language", "en-US,en" );
        con.setRequestProperty ( "User-Agent", "Mozilla/5.0" );
        con.setConnectTimeout(5000);
        con.setReadTimeout(10000);
        InputStream ins = con.getInputStream();
        InputStreamReader isr = new InputStreamReader(ins);
        BufferedReader in = new BufferedReader(isr);
        String inputLine;
        StringBuilder html = new StringBuilder();

        while ((inputLine = in.readLine()) != null) {
            html.append(inputLine);
        }
        in.close();
        return html.toString();
    }

    public static ByteArrayOutputStream postChunk(String chunk) throws IOException {
        URL url = new URL(chunk);
        InputStream in = new BufferedInputStream(url.openStream());
        ByteArrayOutputStream out = new ByteArrayOutputStream();
        byte[] buf = new byte[1024];
        int n;
        while (-1!=(n=in.read(buf))) {
            out.write(buf, 0, n);
        }
        out.close();
        in.close();

        return out;
    }

}

In java if a connection time is not defined, it is waiting for the server for an indefinite time. I put the correction here, this will generate an exception if the internter disconnects. Thanks for reporting.

thank you!