fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
patch
1.12.0 -> 1.12.2

terraform-ibm-modules/key-protect-all-inclusive/ibm (source)
module
minor
4.2.0 -> 4.3.0

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
minor
7.3.1 -> 7.5.0

terraform-ibm-modules/vpe-module/ibm (source)
module
minor
2.4.0 -> 2.7.2

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.12.2
Compare Source
Bug Fixes

added extra validation to the existing_serviceref_zone and existing_cbr_zone_vpcs variables in the fscloud profile submodule (#303) (f24ab4a)

v1.12.1
Compare Source
Bug Fixes

added a default value for the operations variable to make it backward compatible with recent CBR service updates around api types (#305) (2f89973)

terraform-ibm-modules/terraform-ibm-key-protect-all-inclusive (terraform-ibm-modules/key-protect-all-inclusive/ibm)
v4.3.0
Compare Source
Features

added new variable force_delete_key_ring which defaults to true (#359) (16b4144)

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.5.0
Compare Source
Updates  (#621) (753b89d)
IBM Cloud Terraform provider v1.56.0 has added a new option for the "ibm_is_vpc" resource that will cause the default VPC ACL and Security Group to contain no rules (empty).
This release will retire the usage of existing backend scripts to remove all rules from the VPC default ACL and SG, and instead use this new provider option to accomplish the same feature.
Input variable changes:
The new provider option covers both security group and ACL in one variable, so we will be deprecating the individual "clean_" variables and replace them with a new single boolean to enable the feature.

REMOVED: clean_default_security_group and clean_default_acl
ADDED: clean_default_sg_acl, if set to "true" will trigger new VPC option to have empty default groups (default is "false")
CHANGED: security_group_rules now has a default of empty, instead of a broad default inbound rule that may not be desired
Validation has been added to make sure that clean_default_sg_acl has not been set to "true" while having rules specified in the security_group_rules input, which are in direct conflict with each other

Upgrade Notes:
If you have already deployed module with the "clean_" variables not specified, you should see no difference after upgrade.
If you have already deployed module with "clean_" variables set to "true/false", you will get an error after upgrade due to those variables being removed, and you should set the new clean_default_sg_acl variable instead. During a plan phase after upgrading, you may see the following resources marked for DESTROY, this is expected as these are the retired scripts that handled this feature in the past, and have been removed:
module.slz_vpc.null_resource.clean_default_acl[0] will be destroyed
module.slz_vpc.null_resource.clean_default_security_group[0] will be destroyed

v7.4.2
Compare Source
Bug Fixes

add fix to support passing null for the prefix input variable (#605) (51856ff)

v7.4.1
Compare Source
Bug Fixes

added workaround for known provider issue (#592) (8707703)

v7.4.0
Compare Source
Features

subnet CRN now added to the subnet_detail_list output (#587) (518204d)

v7.3.2
Compare Source
Bug Fixes

Acls created for network cidr are open to Any IP (#569) (2f74af0)

terraform-ibm-modules/terraform-ibm-vpe-module (terraform-ibm-modules/vpe-module/ibm)
v2.7.2
Compare Source
Bug Fixes

update module documentation (#389) (77f786d)

v2.7.1
Compare Source
Bug Fixes

fixed issue where the vpe_ips output may not contain any IPs (#372) (3800ae1)

v2.7.0
Compare Source
Features

add functionality to pass a custom name for the vpe gateway (#362) (3bf013b)

v2.6.0
Compare Source
Features

update to the data structure of the vpe_ips output. Added the gateway name as the key for each list of objects. This will help in differentiating the IPs for each gateways.(#354) (f864fbb).
For example:

In previous versions, the output was in this format:

[
   [
       {
           address       = "<IP_ADDRESS>"
           id            = "xxxx-xxxxx-xxxxxx-xxxxx"
           name          = "sandbank-disports-sleeve-dugout"
           resource_type = "subnet_reserved_ip"
           subnet        = ""
        },
       {
           address       = "<IP_ADDRESS>"
           id            = "xxxx-xxxxx-xxxxxx-xxxxx"
           name          = "bannister-food-quarrel-concept"
           resource_type = "subnet_reserved_ip"
           subnet        = ""
        },
    ],
   [
       {
           address       = "<IP_ADDRESS>"
           id            = "xxxx-xxxxx-xxxxxx-xxxxx"
           name          = "haunt-tank-recital-princess"
           resource_type = "subnet_reserved_ip"
           subnet        = ""
        },
       {
           address       = "<IP_ADDRESS>"
           id            = "xxxx-xxxxx-xxxxxx-xxxxx"
           name          = "vigorous-tarantula-cash-reverse"
           resource_type = "subnet_reserved_ip"
           subnet        = ""
        },
    ],
]

In v2.6.0, the output is now in the following format:

{
    test-vpc-instance-account-management      = [
        {
            address       = "<IP_ADDRESS>"
            id            = "xxxx-xxxxx-xxxxxx-xxxxx"
            name          = "handwoven-poem-retention-mantis"
            resource_type = "subnet_reserved_ip"
            subnet        = ""
         },
        {
            address       = "<IP_ADDRESS>"
            id            = "xxxx-xxxxx-xxxxxx-xxxxx"
            name          = "smog-dastardly-bountiful-calf"
            resource_type = "subnet_reserved_ip"
            subnet        = ""
         },
     ]
    test-vpc-instance-billing                 = [
        {
            address       = "<IP_ADDRESS>"
            id            = "xxxx-xxxxx-xxxxxx-xxxxx"
            name          = "daffy-outfield-scrambled-showy"
            resource_type = "subnet_reserved_ip"
            subnet        = ""
         },
        {
            address       = "<IP_ADDRESS>"
            id            = "xxxx-xxxxx-xxxxxx-xxxxx"
            name          = "frogman-deflector-foe-deem"
            resource_type = "subnet_reserved_ip"
            subnet        = ""
         },
     ],
}

v2.5.0
Compare Source
Features

updated list of supported services (#308) (a77b293)

v2.4.1
Compare Source
Bug Fixes

updated documentation and examples to use hashicorp registry syntax (#329) (6447906)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

The failure in this PR is an issue on the main branch. This should be resolved by #290 and this PR should automatically rebase and re-run.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

This is impacted by both the 1.58.0 key ring failed to delete and changes to ACLs. It should be re-run after force delete key ring changes are made.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

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


Neither Overview or Drive works in Firefox.

It either hangs the browser, or outputs nothing in my Jupyter notebook. Understand firefox is not supported at this moment, but would love to see them working in browsers other than chrome.

If you add
"" as the first statement in the HTML_TEMPLATE, above the  element, then it should work in Firefox

I've updated the notebook examples in the repo to include this code. Closing this bug. Please re-open if it doesn't solve your issue. Thanks! Sorry for the long wait.


"Find All References" in VS 2015 not working

In the context menu on any token in an F# source file and selecting the Find All References tool, I'm currently being told Can't find a symbol under cursor. Please make sure that the code is typechecked. in VS's status bar. The code compiles fine with one warning (incomplete pattern matches on a WIP), so the message itself seems inaccurate, but I'm also finding that many things that should be colored by the syntax highlighter aren't.
Current repro steps for me are to open Visual Studio, open an F# project, right-click any token, and select Find All References. This happens to me even with the default Console Application project by trying to find references to the printfn use on line 6. Odd point of note is that this does not happen in F# script files, which also appear syntax highlighted normally.
This is happening to me with Visual Studio 2015 CTP 6 and whatever F# compiler/core version came with it. I'll attempt to repro in a fresh VM and see if it still happens, and of course the easiest way to avoid this is to do a text-based search or to just not use VS 2015 ;)

are you referring to "Find all references" from http://fsprojects.github.io/VisualFSharpPowerTools/?
AFAIK "Visual F#" (this project here which is shipped directly with VS) doesn't come with "Find all references" support.
/cc @vasily-kirichenko @dungpa

@amazingant I don't know why it happened but the issue went away when I install latest Visual F# Preview on top of VS 2015 CTP 6. Could you give it a try?

Per @forki's comment, Visual F# does not implement Find All References or extended syntax highlighting, those both come from the Visual F# Power Tools extensions. Please log an issue there https://github.com/fsprojects/VisualFSharpPowerTools/issues

@forki Yes, it looks like I am. Sometimes I forget that so many Visual Studio things don't exist for F# without the Power Tools extension.
@dungpa That fixed it for me as well. Rather than opening another issue over in the Power Tools extension repo I'll just consider this a result of using the most recently outdated version of F#.

@dungpa Just finished lunch and installed on my dev machine, looks like that fixed the syntax highlighting too! Double thank-you!

You're welcome. Glad that it worked.


0.1.13/ui updates

#125 Fixed in side menu
#148 Fixed
#137 Screens now have limited width in desktop view, still needs further work.
Major refactoring done with layout. Only one layout left. A lot of small issues fixed.
Also made UI improvements to RepeatingDaysSelector component
Small fix to timepicker (cannot select 60 minutes or 24 hours anymore, now replaces it with 0)

Reiktu rasyti Closed #148, kad matytusi, jog uzdarai ticket'ai


Can not get SBE compiled

Hi All,
I use the git clone https://github.com/real-logic/simple-binary-encoding.git on my Linux machine.
In the simple-binary-encoding folder I type ant, I will get those error messages:

Buildfile: /home/user/zluo/others/simple-binary-encoding/build.xml

clean:

init:
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/main/java/classes
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/test/java/classes
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/dist
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/docs/java
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/docs/cpp
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports/unit
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports/checkstyle
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/main/cpp/obj
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/test/cpp/gtest/obj

checkstyle:
[checkstyle] Running Checkstyle 5.6 on 79 files

build:
    [javac] Compiling 59 source files to /home/user/zluo/others/simple-binary-encoding/target/main/java/classes
    [javac] warning: [options] bootstrap class path not set in conjunction with -source 1.7
    [javac] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/sbe/xml/MessageSchema.java:20: error: package uk.co.real_logic.agrona does not exist
    [javac] import uk.co.real_logic.agrona.Verify;
    [javac]                               ^
    [javac] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/sbe/ir/Ir.java:18: error: package uk.co.real_logic.agrona does not exist
    [javac] import uk.co.real_logic.agrona.Verify;
    [javac]                               ^
    [javac] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/sbe/ir/HeaderStructure.java:19: error: package uk.co.real_logic.agrona does not exist
  ....
    [javac]   location: class VarDataEncoding
    [javac] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/sbe/ir/generated/VarDataEncoding.java:13: error: cannot find symbol
    [javac]     public VarDataEncoding wrap(final MutableDirectBuffer buffer, final int offset, final int actingVersion)
    [javac]                                       ^
    [javac]   symbol:   class MutableDirectBuffer
    [javac]   location: class VarDataEncoding
    [javac] 100 errors
    [javac] 1 warning

BUILD FAILED
/home/user/zluo/others/simple-binary-encoding/build.xml:240: Compile failed; see the compiler error output for details.

I find that it could be the missing of the Agrona packages.
Then I download the Agrona-0.3-sources.jar from http://search.maven.org/#search|ga|1|g%3A"uk.co.real-logic"
And in the simple-binary-encoding/main/java directory I unzip the Agrona-0.3-sources.jar files and now I got agrona  sbe folders in simple-binary-encoding/main/java/uk/co/real_logic. Then in the simple-binary-encoding I type ant again. I will get those errors:
Buildfile: /home/user/zluo/others/simple-binary-encoding/build.xml

clean:
   [delete] Deleting directory /home/user/zluo/others/simple-binary-encoding/target

init:
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/main/java/classes
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/test/java/classes
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/dist
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/docs/java
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/docs/cpp
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports/unit
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/reports/checkstyle
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/main/cpp/obj
    [mkdir] Created dir: /home/user/zluo/others/simple-binary-encoding/target/test/cpp/gtest/obj

checkstyle:
[checkstyle] Running Checkstyle 5.6 on 142 files
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/UnsafeAccess.java:33:19: expecting "class", found 'PrivilegedExceptionAction'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/UnsafeAccess.java:36:27: unexpected token: Field
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/UnsafeAccess.java:36:49: expecting IDENT, found '.'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/BiInt2ObjectMap.java:137:23: expecting RPAREN, found ','
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/IntHashSet.java:223:39: unexpected token: >
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/Long2LongHashMap.java:56:58: expecting RPAREN, found ':'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/Long2ObjectHashMap.java:333:25: expecting RPAREN, found ':'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/concurrent/Agent.java:34:5: expecting RCURLY, found 'default'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/concurrent/AtomicArray.java:224:30: expecting RPAREN, found ':'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/generation/PrimitiveExpander.java:56:19: expecting "class", found 'List'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/UnsafeAccess.java:37:21: Got an exception - expecting EOF, found 'f'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/BiInt2ObjectMap.java:138:5: Got an exception - expecting EOF, found '}'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/IntHashSet.java:224:5: Got an exception - expecting EOF, found '}'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/Long2LongHashMap.java:57:9: Got an exception - expecting EOF, found 'values'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/collections/Long2ObjectHashMap.java:334:5: Got an exception - expecting EOF, found '}'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/concurrent/Agent.java:45:1: Got an exception - expecting EOF, found '}'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/concurrent/AtomicArray.java:225:5: Got an exception - expecting EOF, found '}'
[checkstyle] /home/user/zluo/others/simple-binary-encoding/main/java/uk/co/real_logic/agrona/generation/PrimitiveExpander.java:61:13: Got an exception - expecting EOF, found 'Path'

BUILD FAILED
/home/user/zluo/others/simple-binary-encoding/build.xml:214: Got 8 errors and 0 warnings.

Can anyone help with that ? Or there is other way to do the install?
Thank you very much.

Ant is no longer supported. We are in the process of removing it.
Please use Gradle to build:
$ ./gradlew

[zluo@simple-binary-encoding]$ ./gradlew
:clean UP-TO-DATE
:compileJava
:processResources
:classes
:jar
:javadoc
:javadocJar
:sourcesJar
:signArchives SKIPPED
:assemble
:generateExampleCodecs
:compileGeneratedJava
:processGeneratedResources UP-TO-DATE
:generatedClasses
:compileExamplesJava
:processExamplesResources UP-TO-DATE
:examplesClasses
:checkstyleExamples
:checkstyleGenerated
:checkstyleMain
:compileTestJava
:processTestResources
:testClasses
:checkstyleTest
:test
:check
:build
:shadowJar
:install

BUILD SUCCESSFUL

Total time: 41.079 secs

[zluo@simple-binary-encoding]$ ls
CHANGELOG.md    LICENSE    build                           build.gradle      build.xml  examples  gradlew      main  perf-build.xml  settings.gradle  vs2013
CMakeLists.txt  README.md  build-local.properties.example  build.properties  config     gradle    gradlew.bat  perf  sbe-pom.xml     test

Can I ask where is the target/dist ? I did not see it anywhere.


Introduce multi-node integration tests

- Create a wrapper script to run intergation tests
  so that setups and teardowns happen in more
  optimal manner
- Add traps to cleanup containers on failure or
  user interrupt
- Introduce basic multi-node integration tests
- Removed default network, default driver tests
  as they may not be useful in the near future

Signed-off-by: Jana Radhakrishnan<EMAIL_ADDRESS>

:+1:
LGTM


Additional columns for data. I can see only 8

Hi, first of all, many thanks for this, loving the tool.
I would find very useful to get access to the columns sold price and year sold for properties for sale.
Is there any way I can do this?
Also, The address doesn't return the house number or the full post code, is there a way around this?
Many thanks

@alandinbedia @toby-p
I believe that is easily achievable by changing some regex and data params iin the init file
https://github.com/toby-p/rightmove_webscraper.py/blob/master/rightmove_webscraper/__init__.py
REgex
244       # Extract postcodes to a separate column:
. . .        pat = r"\b([A-Za-z][A-Za-z]?[0-9][0-9]?[A-Za-z]?)\b"
. . .        results["postcode"] = results["address"].astype(str).str.extract(pat, expand=True)

Adiditonal data from xpath
175      # Create data lists from xpaths:
. . .        price_pcm = tree.xpath(xp_prices)
. . .        titles = tree.xpath(xp_titles)
. . .        addresses = tree.xpath(xp_addresses)

Great thanks
I will give it a try

I originally set the regex to only get the first part of the postcode since this was all I could find available in the listings - if the full postcode is available on some listings then it would definitely be better to collect this as well if possible.

@alandinbedia @toby-p update for full postcode PR


I add a header, it does not work?

There is a website need a Content-Type header contains"; charset=UTF-8", if not, it response wrong msg.
But When i user this:
builder.addHeader.put("Content-Type", "application/x-www-form-urlencoded; charset=UTF-8");
it does not work.
The FormBody used the Content-Type:
private static final MediaType CONTENT_TYPE = MediaType.parse("application/x-www-form-urlencoded");
and in
BridgeInterceptor.intercept:
if (contentType != null) {         requestBuilder.header("Content-Type", contentType.toString()); }
this will override the header which i added.

Please post on stackoverflow, and I suggest including actual running sample code.
I suggest sending your request to a service like https://requestb.in/1n2lqzb1?inspect, so that anyone can run it and that site shows the output.
Alternatively make a PR including a failing test using MockWebServer and we can fix any bugs you find.

you can set MediaType with body:
RequestBody.create(type, jsonObject.toJSONString())

@su1216
I know this.

@yschimke
I means, if i added a Content-Type header and used FormBody to build the request, the BridgeInterceptor would override the header which i added.


Absolute path for configs folder

I guess this PR is related to #23. The configs folder is expected in the current directory, which makes such a command impossible:
$ java -Xmx4g -jar ddw-0.4.6/ddw-0.4.6.jar -input corpus -output .

Using the absolute path to configs (which must still be relative to the JAR file) allows you to be anywhere. Or am I missing something?

This is also related to https://github.com/DARIAH-DE/Topics/issues/72. The tests fail because of the missing config files.


Patch dangerous workflows

This adds a git-diff format patch to every script-injection finding.
The git-diff is generated using a dependency not currently used by Scorecard, which isn't great. Even worse, the project was recently archived. But it's the only way I could find to create a usable git-diff.
I'm now working on the logic to run the tests with this format.

Please take a look at the test format I've suggested in my latest commit.
Basically, instead of a foo_fixed.yml workflow, we just store the diff (as a file) and the "finding" information that GeneratePatch receives.
The diff file can be easily generated with git diff --output foo.diff foo.yaml foo_fixed.yaml.
Note that foo_fixed.yaml isn't actually used in the test. We therefore have a few options:

keep both _fixed.yaml and .diff files. The _fixed isn't used by the logic, but is a useful reference to understand what we expect the final result to be.
drop the _fixed.yaml files, only storing the .diff files. This omits "irrelevant" files and avoids the risk of "drift" (where we modify the diff in the future, but don't update the fixed workflow)
only keep the _fixed.yaml file, and then generate the diff (using the dependency, not GeneratePatch, of course! :p) during the test. This allows us to store a much more legible file identifying the expected output, but means generating the diff for every test run.
This feels weird (using the diff generation logic of GeneratePatch to generate the diff), and it means the tests are only validating that the changes are what we expected, not the git diff output itself (since we're using the same logic for the function and the test).
One could argue this is somewhat equivalent to a testing oracle, where we use the diff generated by the dependency for foo.yaml and foo_fixed.yaml as a "known-good" output against which we compare our output (or, rather, the changes we made, not the literal git diff output).
A final alternative would be the same as above, but actually running git diff in a subprocess instead of using the dependency to create the diff. This would give us the best oracle (since it's actually the output we want), but would have its own issues: the output could theoretically change between devices with different versions of git (though I believe the diff format has been pretty stable for a long time), the tests would fail on devices that don't have git in their PATH (though one could argue that's unlikely to happen... anyone running tests probably used git clone to even get the source code), etc.


Ignore filters do not seem to have any effect

Steps to reproduce the problem:

mitmproxy -T --host
^O -> Ignore patterns
Add regex: .*
Navigate to example.com in browser, requests appear in log.

System information
mitmproxy 2.0.0

Thanks for the report. What OS is this on?
mitmdump -T -v --ignore .* works as expected on Windows, I only see client/server (dis)connect messages, but no requests.

Hi, thanks for the quick response.
On osx 10.11.
I think this may be related to adding/removing rules in the GUI rather than the filtering itself. This seems to be reproduceable for me:

mitmdump -T --host -v --ignore '.*' -> (ignore works, nothing is shown)
mitmdump -T --host -v, then add rule via options menu -> (not working, requests still appear)

This is fixed on master. Thanks again! 😃


Prepare release: 1.1.0

Version bump for 1.1.0.

Cherry-picked #5034 and #5038 into this

Not sure why the cherry-picks are showing up as diffs here. I'm going to close the PR and version bump in new PR, then re-cut 1.1.0 to keep branch history closer to main.
Closing in favor of #5047.


Pointed DC/OS master back to Mesos master.

High-level description
Since the 2.0 branch of DC/OS has been established, the master branch should once again point to a commit on the master branch of Mesos. This PR accomplishes this change.
Corresponding DC/OS tickets (required)

DCOS_OSS-5342 Bump Mesos for DC/OS 1.14.0-beta

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[x] Change log from the last version integrated (this should be a link to commits for easy verification and review): Mesos diff
[x] Included a test which will fail if code is reverted but test is not. If there is no test please explain: Mesos integration tests plus the existing DC/OS integration tests are sufficient to test this bump.
[x] Test Results: link to CI job test results for Mesos
[ ] Code Coverage (if available): N/A

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot sync


README

Include an overview of the project and each mini-game, the team members, the team structure and links to the license, wiki and contributing guidelines

Leaving the issue open as installation instructions are missing

Reopening this issue as the README.md is missing installation notes for people who have never used/installed Cypress


wording: "onCleanup" vs "useCleanup", "sideEffect" vs "useEffect", etc

Could it be more semantic to have functions like useCleanup and useEffect named onCleanup and sideeffect (and similar consideration for other functions)?
f.e.
    const interval = setInterval(() => setState('counter', c => c + 1), 1000);
    onCleanup(() => clearInterval(interval))
    sideEffect(() => console.log(state.counter))

I'm starting think you are right. These aren't hooks so the semantics aren't right. I do still  think there is value in familiarity, but use suggests limitations of hooks that are not present here.
createState createEffect createMemo createSignal onCleanup createRoot can be the alternative. Any one else want to weigh in?


Jenkins agent firewall rules

Amending firewall to allow network access for specified services.

@alexmuller and I have discussed.
Due to the nature of the current Epic.  i.e. security patching jenkins and porting it over to govuk-puppet.  I think this approach is appropriate because all parameters have been defined in ci-puppet.  We are just moving stuff over rather than re-engineering which should be left for when we define the new CI pipeline.
The plugins have already been defined in https://github.com/alphagov/govuk-puppet/pull/5061


Add Docker Integration

Add Docker build files
Goals:
Be able to use Docker to compile and run the program for development purposes.
Use the Docker setup to mimic an offline environment of the circleci test machine.
Todo List:

[x]  Docker image names should be automatically unique (instead of having to replace rust-starter).
[ ]  The circle-ci commands actually fails in some circumstances.
[x]  Preferably the .dockerignore file should be in the docker directory.

Goals:

Be able to use Docker to compile and run the program for development purposes.
Use the Docker setup to mimic an offline environment of the circleci test machine.

Will be moving from CircleCi to Travis/AppVeyor


mgr: "copy ceph key(s) if needed" tries to copy keys to the wrong directory

Bug Report
What happened: Follow up to issue #4518. Pull #4591 didn't fix my issues.
I have 3 mgr servers (ceph-mon-0, ceph-mon-1 and ceph-mon-2) and the ceph cluster is named ceph.
Ansible tries to copy the key from ceph-mon-0 to /var/lib/ceph/mgr/ceph-ceph-mon-0 on ceph-mon-1 and ceph-mon-2, which doesn't exist. "msg": "Destination directory /var/lib/ceph/mgr/ceph-ceph-mon-0 does not exist".
-vvv error logs at https://pastebin.com/LcRJaEwh
Share your group_vars files, inventory
[mons]
ceph-mon-0 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo
ceph-mon-1 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo
ceph-mon-2 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo

[mgrs:children]
mons

[rgws:children]
mons

[osds]
ceph-osd-0 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo osd_crush_location="{ 'root': 'default', 'pod': 'pod-01', 'rack': 'rack-01', 'host': 'sdsc-dev-osd-0' }"
ceph-osd-1 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo osd_crush_location="{ 'root': 'default', 'pod': 'pod-01', 'rack': 'rack-01', 'host': 'sdsc-dev-osd-1' }"
ceph-osd-2 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo osd_crush_location="{ 'root': 'default', 'pod': 'pod-01', 'rack': 'rack-01', 'host': 'sdsc-dev-osd-2' }"
ceph-osd-3 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo osd_crush_location="{ 'root': 'default', 'pod': 'pod-01', 'rack': 'rack-01', 'host': 'sdsc-dev-osd-3' }"
ceph-osd-4 ansible_host=<IP_ADDRESS> ansible_port=22 ansible_user=centos ansible_become=yes ansible_become_method=sudo osd_crush_location="{ 'root': 'default', 'pod': 'pod-01', 'rack': 'rack-01', 'host': 'sdsc-dev-osd-4' }"

Environment:

OS (e.g. from /etc/os-release): CentOS 7.7
Kernel (e.g. uname -a): 3.10.0-1062.1.1.el7.x86_64
Docker version if applicable (e.g. docker version): None
Ansible version (e.g. ansible-playbook --version): ansible 2.8.5
ceph-ansible version (e.g. git head or tag or stable branch): origin/stable-4.0 (fd759f9)
Ceph version (e.g. ceph -v): ceph version 14.2.4 (75f4de193b3ea58512f204623e6c5a16e6c1e1ba) nautilus (stable)

Same problem here - in fact, for me #4591 broke the deployment, going back to 9c05470 fixes the issue.

#4604 should fix this issue

It looks like #4604 resolves my issue

Same here, works with #4604.


Installation Failure on npm install (Ubuntu 18.04)

System information
Linux Ubuntu 18.04 (on Virtualbox)
Zenbot 4
Nodejs 8.10.0
Describe the problem
I have a fatal installation problem. I run npm install in the zenbot file as per instructions and the files begin to download, but the download stops and hangs indefinitely when it reaches extract: npm: sill extract<EMAIL_ADDRESS>extracted to /home//zenbot/node_modules/.staging/npm-4e0a87a1(11818ms)
It looks like a bug to me but I might be wrong. I can get no log on this, as the terminal freezes and I have to restart it.
Is it possible to bypass the npm extraction? I am not exactly clear why I need npm again anyway since I already have it...?
I should add that I have npm, nodejs and mongodb downloaded already with no problems
I would appreciate your thoughts as I've tried all sorts of things today and nothing worked!

I actually figured it out - I had to wait much longer (10 min+) for the extract to happen. It was simply very slow and I assumed nothing will happen. So the issue is solved.
Best,
Jo


Issue84 Add note about ./x.py build

Looks good to me :+1:
There isn't really much more to say about ./x.py check seeing as it's quite similar to cargo check, which most people are already familiar with.


[$500] IOU - Error message in Amount page overlaps with amount field when zoom level in 150%

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

Version Number: 1.4.9-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/32460
Action Performed:

Go to staging.new.expensify.com on mweb Safari
Go to any chat > + > Request money
Click Next without entering the amount
Increase the zoom level to 150%

Expected Result:
The error will not overlap with amount input field
Actual Result:
The error overlaps with amount input field
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/01aa3d5f-4bce-42f5-81b9-fe59ec2cfad0
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01485e3068c4913aa8
Upwork Job ID:<PHONE_NUMBER>677228032
Last Price Increase: 2023-12-07

Welp, I found another bug on production:

I was not able to reproduce on my Pixel 6 on Chrome. I'm going to say this is not a blocker

Proposal
Please re-state the problem that we are trying to solve in this issue.
Error message in Amount page overlaps with amount field when zoom level in 150%
What is the root cause of that problem?
This is because we are using absolute positioning for form error message here
https://github.com/Expensify/App/blob/5a0b7fdb08979ea765f8a6c9e3b03e78ce61eb4a/src/pages/iou/steps/MoneyRequestAmountForm.js#L291-L292
What changes do you think we should make in order to solve the problem?

We shouldn't use absolute positioning to position it bottom of the amount text input
We can use different options like to put it in a separate view below amount text input and we can achieve the same thing
What alternative solutions did you explore? (Optional)

Unsure what the expected result would be here as there's nowhere to display the error message once you're zoomed that far in :smile: @FitseTLT Are you able to post a screenshot of your proposal's result?

Unsure what the expected result would be here as there's nowhere to display the error message once you're zoomed that far in 😄 @FitseTLT Are you able to post a screenshot of your proposal's result?

Here is a demo @jjcoffee
https://github.com/Expensify/App/assets/38649957/5c5de6f6-09f5-4460-94c1-f3d76854c6a0

That solution looks good to me, visually at least. @jjcoffee thoughts?

Am I right in thinking that your proposal will make the amount input always appear offset (as in your demo)? I think ideally we'd still want to keep the amount input vertically centralised.

@jjcoffee here is a demo for my alternative solution which achieves the vertical centering by keeping error message absolutely positioned but giving minHeight for the parent view 👍
https://github.com/Expensify/App/assets/38649957/acc71c9e-5d5b-4395-9657-8e8c8596bffd

Thanks, that looks better IMO. Happy to go with your proposal. We can always tweak on the PR if others disagree!
:ribbon::eyes::ribbon: C+ reviewed

@FitseTLT when can we expect your PR? 🙏

Summarizing payouts for this issue:
Contributor: @FitseTLT $500 (paid via Upwork)
Contributor+: @jjcoffee $500 (payable via Upwork BZ checklist is completed)
@jjcoffee could you please complete the BZ checklist so that I can issue your payment?

The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/30287
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/30287/files#r1439298027
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A - hard one to catch
Determine if we should create a regression test for this bug.
If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to any chat > + > Request money
Click Next without entering the amount
Increase the zoom level to 150%
Verify that the error message doesn't overlap with amount input

Do we agree 👍 or 👎

@sakluger Checklist complete, sorry for the delay!

Thanks! I created the regression test issue and completed all payouts.


feat: replaced image shortcodes w/ image render hook

Replaced the image shortcodes w/ render-image.html, respecting the following logic:

When an image width is less than the value set for smallLimit in ./config/_default/params.toml, an <img> element is used, otherwise a figure element.

See also (temporary, until merge ready): https://render-hooks--hyas.netlify.app/quick-start/

Dropped link render hook containing "target="_blank" rel="noopener" logic (G200: Opening new windows and tabs from a link only when necessary)

Dropped link render hook containing email obfuscation (looks like it's not really possible). Kept email shortcode (for now, as an option, but better to avoid using email links)


HTTP 403 Error when trying to login

Is anyone getting a login 403 error?

just encountered same thing on a different python wsimple api - I'll keep watch here and if were stuck I'll look at adding a PR.

@Zechtitus a PR would be amazing as I will be busy for the next few weeks

there is one other guy working on issue here - https://github.com/ahmedsakr/wstrade-api/issues/96
Will see what he churns up. He is right on the cloudflare security

Great NEWS guys I found a way to log in

Great NEWS guys I found a way to log in

that's great!! will you be updating the repo soon or can you share how you managed to do that, please?

there is one other guy working on issue here - ahmedsakr/wstrade-api#96
Will see what he churns up. He is right on the cloudflare security

I just merged my fixes to wstrade-api and it's available in v1.7.0 :)

Thanks! Nice to get the bot back working

Hello, cross-posting on here as well. Just opened a PR in the other Python client library. Swaps out requests.Session for cloudscraper.Cloudscraper (instance of requests.Session). Got things working with that :)

on an aside, could it be worthwhile merging these two Python client libraries, into a consolidated/"unofficially official" Python client library?

Would have no problem merging his project somehow into mine, also working on a new release with a working fix for the login problems and a couple of other goodies like accessing dictionaries with dot notation (like Javascript) and a more organized HTTP request system - currently on branch api_v2

@yusuf8ahmed if it's helpful for the login related issues, see my PR here

also, you guys might see a folder called wsimple-desktop could be a new part of this project ???

Hello guys. First off - big thank you for making this public! I have used the api_v2 branch to successfully log in and send in buy / sell orders, and cancel them entirely from within the framework you guys have built. I did have to modify line 330 of api.py to get it working, as I think you are adding different URL sources (base vs base public etc.)
Most of the WSimple class functions require passing in the tokens, however the tokens are a member of the class itself. Is there a reason for this?
Cheers!

It was an intentional design choice as I was planning on making it work with a web app(which I am still planning on building), also I am currently working on my local branch with a version of Wsimple that manages tokens after a flag on the Wsimple is set True. Still needs some little testing and more development though

So where are we at here?
Is this patch being rolled into the main branch?
The last update to V2 is 12 days and this this thread talks about changes 6 days ago.

Okay hello, guys v2 is finished and just needs a little more testing so you guys can pull it and test in your environment, if you have any problems report them here:
Changelog:

Change login process from using an exception to a callback function:

from wsimple.api import Wsimple

def get_otp():
    return int(input("Enter otpnumber: \n>>>"))

email = str(input("Enter email: \n>>>"))
password = str(input("Enter password: \n>>>"))

ws = Wsimple(email, password, otp_callback=get_otp)

resolved 403 http issue when logging in
tokens are new internally managed but tokens are still a keyword argument.

Update on api_v2, wsimple is officially on v2, the branch api_v2 will be merged with master very soon.
To update:
pip install -U wsimple
To install the latest version:
pip install wsimple

I just started seeing issues again with https://github.com/seansullivan44/Wealthsimple-Trade-Python - are you guys seeing similar?

I just tested https://github.com/ahmedsakr/wstrade-api and it's still working.

@Zechtitus @ahmedsakr  wsimple is still working, no login errors, also access real-time data via websockets should be working fully in the next update.

Hi guys, you can start testing api_v2 by upgrading wsimple.
upgrade:
pip install -U wsimple
install:
pip install wsimple


New version: ChristianHohnstadt.xca version RELEASE.2.5.0

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

InstallerUrl contains Manifest version instead of PR version:
PackageVersion: RELEASE.2.5.0   InstallerUrl: https://github.com/chris2511/xca/releases/download/RELEASE.2.5.0/xca-2.5.0-win64.msi
(Automated message - build 601)


Update free-courses-ml.md

New Update

Duplicate of #6994

Duplicate of #6994

Try to avoid incremental PRs with same content, please!


[Web display] only show a fraction of map

The simulator home only display a fraction of map, the map can not drag or zoom。

You can move around using the keyboard arrows


Video Conference?  What About Audio Conferences?

I really like the idea of a hotkey to disable the webcam and microphone.  However, I don't think it should only be limited to when the webcam is running (if that is the case).  Often we are on Audio only calls and would like to quickly mute our microphones.  That hotkey would be great if it can run in that context as well.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

Did you see this issue: #6374? It was opened shorty before this issue.


Cannot connect to windows subsystem for android from another windows pc

Connecting to android subsystem with adb from the same device work but follow the doc instructions to connect from another device in the network won't work even thought I used the host machine ip and make it visible across the network.

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

ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Version Independent ID: 31f9a801-c5b5-94e7-41b9-1b2b4a1d27de
Content: Windows Subsystem for Android™
Content Source: hub/android/wsa/index.md
Product: dev-environment
Technology: windows-android
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

Hi @MathewLC - Sorry you're having trouble. I'm wondering if it may depend what version of the Android SDK Platform Tools you're running...? https://developer.android.com/studio/releases/platform-tools  The Android dev docs about adb point out some differences based on what version of Android is being run, I'm not quite savvy enough on the inner workings of WSA to know whether this makes a difference in using adb to debug or not. Here's the info in the adb doc: https://developer.android.com/studio/command-line/adb#connect-to-a-device-over-wi-fi-android-11+
and https://developer.android.com/studio/command-line/adb#wireless.
Have you tried the Android Connection Assistant? https://developer.android.com/studio/run/device#assistant
@hamza-usmani - Can you help?

Actually I can confirm now that with the windows update this bug was fixed. Thank you for the assist.


HSOption display Added

Hello, Thank you for the contribution, will merge it and apply some modifications to it to make it work for 2P as well !


Added option to lock cursor to game window -additional request

is it possible to have this for borderless mode as well. i notice it only works for window mode and full screen. i have duel monitor so when im move my mouse to high up it goes to the next monitor. just a thought.  thanks

The cursor is still not locked in borderless? I wasn't able to test it properly on my end since I don't have a second display. There's some calls to ClipCursor() that might be causing this. Would you be up for testing out a build with a potential fix?

Yes i can test it. just let me know what to do. just a heads up. i have no knowledge of coding or anything.

Okay here you go. Just extract this over the files already in the game folder. You'll have to edit FFXVIFix.ini to enable the cursor lock feature.
FFXVIFix_v0.7.9_test.zip
This build stops two calls to ClipCursor(), so we'll see if that fixes it.

nice, that worked. i did have to change it to window mode then back to borderless mode in the game setting to make it work. nothing too big to worry about.

Since we're killing two calls to ClipCursor(), do you mind testing one more build, just to confirm which one is responsible?
FFXVIFix_v0.7.9_test2.zip

Test 2 failed. cursor still went to 2nd monitor in borderless. when doing the same thing as test 1.

Alright perfect thanks for that. Here's (hopefully) a final build for you try.
FFXVIFix_v0.7.9_test3.zip

ok, it works but this time. to active it. you need to tab out of the game then tab back into the game. good thing is. i didnt have to switch back to window mode and then back to borderless mode in game setting.

Okay this build should work. At least it does in my testing, the cursor is locked to the window even in borderless and without having to alt+tab or anything else.
FFXVIFix_v0.7.9_test4.zip

Perfect, this works. thank you so much.

Added in v0.7.9


Is it Azure Bot Service or Bot Framework Service?

We have the docs under 'Azure Bot Service' but in the docs we refer to the 'Bot Framework Service'. Is this something different? The same thing? We call it the Bot Framework Emulator... if it's the Azure Bot Service we should call it the Azure Bot Service Emulator (a la Azure Storage, Compute, etc).

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

ID: 5bf8a24b-7029-247b-3c5c-3902befe0ccf
Version Independent ID: bcd01444-d332-29e6-2b81-5558bb2d576b
Content: How bots work - Bot Service
Content Source: articles/v4sdk/bot-builder-basics.md
Service: bot-service
Product: bot-framework
GitHub Login: @johnataylor
Microsoft Alias: johtaylo

These are the same things. The branding is a bit confusing at this time. These will be updated once that final decisions are made.


Cannot import the re-exported op2 macro invocation

When trying out code similar to the "extensions" example provided with the crate to develop an external Rust extension
use rustyscript::deno_core::{extension, op2, OpState};
use rustyscript::deno_core::error::AnyError;

#[derive(Debug, Clone)]
pub struct WebConnectors{
}

#[op2(fast)]
#[bigint]
fn op_add_example(#[bigint] a: i64, #[bigint] b: i64) -> i64 {
    a + b
}

I receive the following compiler error:
error[E0433]: failed to resolve: use of undeclared crate or module deno_core
--> labiroot/src/scripting/web_connector_ext.rs:19:1
|
19 | #[op2(async)]
| ^^^^^^^^^^^^^ use of undeclared crate or module deno_core
This is very surprising, since all other items in the re-export (AnyError, OpState) etc are recognized and work well and my IDE can also locate  the reexported op2 macro, but the compiler can't. I activated all features of Rustyscript to ensure it's not a feature issue, but this was unfortunately not fixing the problem. Is there a reasonable explanation for this, am I missing something here?
Thanks in advance, great crate!

Add deno_core to your Cargo.toml:
deno_core = "*"

Then import the macro

Add deno_core to your Cargo.toml:
deno_core = "*"

Then import the macro

Correct.
deno_core's macro body specifies ::deno_core, which requires callers to have deno_core as a direct dependency

@utyfua @rscarson thanks for your help, that in fact solved the problem.


Implementing a SQL handler

Hi,
I'm trying to implement an SQL handler.
But I have a problem that I wanted to ask about.
When I'm trying to create a SQL query I want to get a list of all the attributes that I should put in the query. But the attribute list contains things that are not attributes. Such as id, type and meta.
I see in the code that there is some special code that removes the non-attributes from the attribute list when generating the data item.
responseHelper._generateDataItem = function(item, schema) {

  var isSpecialProperty = function(value) {
    if (!(value instanceof Object)) return false;
    if (value._settings) return true;
    return false;
  };
  var linkProperties = Object.keys(schema).filter(function(someProperty) {
    return isSpecialProperty(schema[someProperty]);
  });
  var attributeProperties = Object.keys(schema).filter(function(someProperty) {
    if (someProperty === "id") return false;
    if (someProperty === "type") return false;
    if (someProperty === "meta") return false;
    return !isSpecialProperty(schema[someProperty]);
  });

  var result = {
    type: item.type,
    id: item.id,
    attributes: _.pick(item, attributeProperties),
    links: responseHelper._generateLinks(item, schema, linkProperties),
    relationships: responseHelper._generateRelationships(item, schema, linkProperties),
    meta: item.meta
  };

  return result;
};

It seems strange to me that the attribute list contains the non-attribute entries. And how about if I have an attribute called meta or type? Wouldn't that be a conflict?
The search() function of my handler looks something like this:
		search(request, callback) {
			let attributeKeys = Object.keys(request.resourceConfig.attributes);
			var sql = `
				SELECT ${attributeKeys.join(',')}
				FROM ${request.resourceConfig.resource}
			`;
			db.pool.query(sql, [], (err, res) => {
				let results = !err ? res.rows : [];
				results = _fixResult(request, results);
				return callback(err, results, results.length);
			});
		}

Hi @Ramzeus! Are you aware of https://github.com/holidayextras/jsonapi-store-relationaldb?

Hi @Ramzeus. Within the scope of your handler, request.resourceConfig.attributes maps directly to the resource definitions we've written (such as this: https://github.com/holidayextras/jsonapi-server/blob/master/example/resources/comments.js)
I agree that combining the attributes and the relationships into one object slightly complicates the use case for writing data into a relational database. As such, I've taken the time to provide that handler for the community, as @pmcnr-hx has said above. We're putting the final touches on that module right now and we're looking to ship it to production next week. On the flip side, combining attributes and relationships simplifies the use case for writing an object store handler.
It might be worth while automating the creation 2x new properties on request.resourceConfig to separate out the attributes and the relations, providing the option to chose which use case works best? If you think this'll help, I can make it happen.

Oh, I was not aware about that project! I will take a look at it.
Thanks!


Catch more errors

Describe what problem your feature request solves
Some errors are not caught, and the resulting trace output is not easy to recover from
Describe the solution you'd like
Catch errors so that the browser does not display trace, and the app still runs
Additional context
Here is a login error (I was messing with github client secret env var):
TokenError: The client_id and/or client_secret passed are incorrect.
    at Strategy.OAuth2Strategy.parseErrorResponse (node_modules/passport-oauth2/lib/strategy.js:358:12)
    at Strategy.OAuth2Strategy._createOAuthError (node_modules/passport-oauth2/lib/strategy.js:405:16)
    at node_modules/passport-oauth2/lib/strategy.js:175:45
    at node_modules/passport-github/lib/strategy.js:77:16
    at node_modules/oauth/lib/oauth2.js:209:7
    at passBackControl (node_modules/oauth/lib/oauth2.js:134:9)
    at IncomingMessage.<anonymous> (node_modules/oauth/lib/oauth2.js:157:7)
    at IncomingMessage.emit (node:events:390:22)
    at endReadableNT (node:internal/streams/readable:1307:12)
    at processTicksAndRejections (node:internal/process/task_queues:81:21)

and another one where I was messing with CSRF:
ForbiddenError: invalid csrf token
    at csrf (node_modules/csurf/index.js:112:19)
    at Layer.handle [as handle_request] (node_modules/express/lib/router/layer.js:95:5)
    at next (node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (node_modules/express/lib/router/layer.js:95:5)
    at node_modules/express/lib/router/index.js:281:22
    at Function.process_params (node_modules/express/lib/router/index.js:335:12)
    at next (/node_modules/express/lib/router/index.js:275:10)
    at Function.handle (node_modules/express/lib/router/index.js:174:3)
    at router (node_modules/express/lib/router/index.js:47:12)
    at Layer.handle [as handle_request] (node_modules/express/lib/router/layer.js:95:5)
    at trim_prefix (node_modules/express/lib/router/index.js:317:13)
    at node_modules/express/lib/router/index.js:284:7
    at Function.process_params (node_modules/express/lib/router/index.js:335:12)
    at next (node_modules/express/lib/router/index.js:275:10)
    at cookieParser (node_modules/cookie-parser/index.js:71:5)

I did not mean to close this with the comment on the ES6 refactor PR.  I need to be more careful with what words I use before linking an issue. :face_with_head_bandage:
This will be done in 2.0

Closing as this is complete in v2


fix(storage):  fix memory usage calculation for shared-buffer

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
We did not calculate the size of shared-buffer correctly. It will cause the wrong usage of memory show in grafana. And it may also make us can not limit the memory used by shared-buffer before compute-node OOM
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Consider merging this? https://github.com/risingwavelabs/risingwave/pull/9525, which also counts the overhead of jemalloc

Shall we close this? Or merge this or #9525?

https://github.com/risingwavelabs/risingwave/pull/9525 seems better. +1 for https://github.com/risingwavelabs/risingwave/pull/9525


Power Apps Cards Tutorial

Hello!
This comments are in reference to https://learn.microsoft.com/en-us/power-apps/cards/tutorials/simple-shopping-list
Following the instructions things didn't work for me as expected. If I do Collect(myGroceryList, theNewItem), I get an error that Text is not allowed. And it seems logical for me.
I used this instead: Collect(myGroceryList, {'Item': TheNewItem})
But before that, I have to write the default for the variable myGroceryList table, like this, in order for the Collect recognize the 'Item' column.
Default:
[
{
"Item": ""
}
]
With that, the Collection was updated, but in order to show the list with new items I have to change the Text property with:
Text: ThisItem.Item
Doing all this I did the Card work as expected.
I hope I was doing right.
Thank you.
Diego.

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

ID: 223a0c37-a1f5-6bfe-1730-2b4d2a47a455
Version Independent ID: 223a0c37-a1f5-6bfe-1730-2b4d2a47a455
Content: Create a shopping list card (Preview) - Power Apps
Content Source: powerapps-docs/cards/tutorials/simple-shopping-list.md
Service: powerapps
GitHub Login: @iaanw
Microsoft Alias: iawilt

Thanks for your comment, @Diego1135. This issue has been assigned to the doc author for review.

Thanks for the additional info @Diego1135.
To get the tutorial to work on my end I had to ensure that variable Title was also included. This is different from the variable Name. Code snippet below, see title.
"variables": {
"myGroceryList": {
"type": "array",
"title": "myGroceryList",
"description": "",
"default": [
{
"Item": ""
}
],
"x-ms-cardapp": {
"scope": "session"
}
}
}

We've updated the documentation with this information, thanks all.


Aptly publish drop won't drop corrupted published repository, no force option

In a futile attempt to clean things up, I ran rm on a number of debs [1], and removed them from their respective aptly repos, and ran aptly db cleanup.  This has gotten aptly repo back to the correct number of packages, and packages can be added to these repos directly.
I then tried to publish some new packages, which failed with:
$ aptly -config=aptly.conf -gpg-key="803709B6" -force-overwrite=true publish update precise v1
WARNING: force overwrite mode enabled, aptly might corrupt other published repositories sharing the same package pool.

Loading packages...
Generating metadata files and linking package files...
Finalizing metadata files...
Signing file 'Release' with gpg, please enter your passphrase when prompted:
Clearsigning file 'Release' with gpg, please enter your passphrase when prompted:
Cleaning up prefix "v1" components experimental, main...
ERROR: unable to update: unable to load package with key Pamd64 flapjack 1.0.0~+20140911074259-v1.0.0-trusty-1 2b7caa2968a1f693: key not found

The publish update hasn't taken notice that a number of packages have been removed from the filesystem.  That's understandable enough, so I tried to drop the publish (to later go and create them again with the correct packages).  This failed with:
$ for distro in precise trusty; do aptly -config=aptly.conf publish drop ${distro} 0.9; done
Removing /home/ubuntu/omnibus-flapjack/aptly/public/0.9/dists/precise...
Cleaning up prefix "0.9" components experimental, main...
ERROR: unable to remove: unable to load package with key Pamd64 flapjack 0.9.5~+20141008003938-v0.9.5-trusty-1 11155af4595696b6: key not found
Removing /home/ubuntu/omnibus-flapjack/aptly/public/0.9/dists/trusty...
Cleaning up prefix "0.9" components experimental, main...
ERROR: unable to remove: unable to load package with key Pamd64 flapjack 0.9.3~+20140901123122-v0.9.3-precise-1 a5c00d00975f3f61: key not found

There appears to be no way to force-remove publishes when they're corrupt.  --force does not work here.  In my view, beyond checking for existence, drop shouldn't try to verify the state of the publish before dropping it.
[1] I subsequently learned that this is the wrong way to do things, so won't try this again.

Fixed in master, available as nightly build aptly_0.9.5~dev+12+gf8bca46

@antonio you might be interested in that fix as well.
How to use: if aptly publish drop fails with error message on 'Cleaning up prefix ...phase, you can try running it with-force-drop` flag, it would ignore errors and continue dropping published repository "no matter what".

@antonio you might be interested in that fix as well.
How to use: if aptly publish drop fails with error message on 'Cleaning up prefix ...phase, you can try running it with-force-drop` flag, it would ignore errors and continue dropping published repository "no matter what".

@smira this did the trick! I was able to drop the published repositories, recreate them and get rid of the errors. Thanks a lot.


Add parameter which allows customizing container security context

On OpenShift 4.10 and older, the default container security context provided by upstream isn't compatible with the restricted SCC. This PR introduces a new parameter which allows customizing or removing the container security context from all containers of the Kyverno deployment.
The PR also adds a new test case which verifies that setting containerSecurityContext: null drops the security context as expected.
Checklist

[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.

Side-note: I've verified that clearing the default deployment spec.securityContext: { runAsNonRoot: true } works fine on OCP 4.10.


Excluding the current element from the list in the nav bar is confusing

I selected paper-behavior from the list of "other elements" and was surprised to see it removed from the list.
Changing the list makes it hard to navigate by clicking down the list, because sometimes you don't move you cursor to select the next element and sometimes you do.

I would vote for All Elements with a highlight as well. I tried to click linearly on all the elements, but because they kept disappearing, I ended up clicking on the first item several times until I figured it out. :)

Agreed - I think best behavior would be to keep the list static with the currently selected one highlighted.


Add support for Llama-3 Model

I think 8b will be:
    src_vocab_size=128256,
    emb_dim=4096,
    norm_eps=1e-5,
    nheads=32,
    kvheads=8,
    nlayers=32,
    hidden_grow_factor=3.5,
    multiple_of=1024,
    max_expected_seq_len=8192,

@JRosenkranz  do we have the configs for any other llama3 variants?

I think 8b will be:
    src_vocab_size=128256,
    emb_dim=4096,
    norm_eps=1e-5,
    nheads=32,
    kvheads=8,
    nlayers=32,
    hidden_grow_factor=3.5,
    multiple_of=1024,
    max_expected_seq_len=8192,

@JRosenkranz do we have the configs for any other llama3 variants?

May I know where should I place these configs to run benchmark_inference.py with Llama 3 8b? Also I think the tokenizer of Llama-3 no longer works with SentencePiece. Is there any other changes to the code that needs to be made?

This is the current config I have for 8B:
LLaMAConfig(
    src_vocab_size=128256,
    emb_dim=4096,
    norm_eps=1e-5,
    nheads=32,
    kvheads=8,
    nlayers=32,
    hidden_grow_factor=3.5,
    multiple_of=1024,
    max_expected_seq_len=8192,
)

I believe for 70B it looks like this:
LLaMAConfig(
    src_vocab_size=128256,
    emb_dim=8192,
    norm_eps=1e-5,
    nheads=64,
    kvheads=8,
    nlayers=80,
    multiple_of=4096,
    hidden_grow_factor=3.5,
    max_expected_seq_len=8192,
)


UI: disable toggles while onroad

Just found this issue. Solving it should also fix mine: #21170
Should I close mine and tag onto this one?

No, that issue is different. The buttons are already disabled when you go onroad, but that doesn't trigger if you start onroad.


Crash

crash-2022-06-14_15.13.15-client.txt
version AE2WTLib-11.1.0

how did you get that crash?
I looked at the code, and this should be impossible to happen

i just crafter somthing from IF and bam crash wierd bc i didnt touch anything from this mod yet or open  curios

b8ad7caec85d93c238fb4fb9c04a0b0fbbf024d1 should hopefully fix this, although I couldn't test this


Add option to not display result of ddev exec

Is your feature request related to a problem? Please describe.
I am working on adding [support for DDEV on GrumPHP (https://github.com/phpro/grumphp/pull/972/files)
At the moment ddev exec outputs the command that failed to execute, either in full or in json with ddev exec -j.
Full Output of a failing commit

JSON Output of a failing commit

Describe the solution you'd like
I'd like  a ddev exec option to not output anything, GrumPHP will be red or green anyway and that's all the verbosity that is required.
Something like ddev exec --no-output COMMAND
Describe alternatives you've considered
Run git commands from inside the container, but it is not really an option because one would have to copy over ssh keys, git config (easy) and gpg keys (I don't know how to do this yet).
Additional context
Grum seems to be widely adopted. If we manage to get https://github.com/phpro/grumphp/pull/972 in a good state it would mean out-of-the-box support for DDEV. PR over there is working at the moment, though I'd 'love to improve it to run both inside and outside of ddev containers.

Would ddev exec COMMAND &> /dev/null work for you?

I tried it, it doesn't work and gives bad results. What happens is that:

files get commited regardless of the output of grumphp
grumphp is running after the files are commited
I get full verbosity (as describe is first post) but I do not return on the terminal until I press enter

Without it things work fine, but I just have that extra verbosity.

Can you show your implementation? How do you call ddev exec?

Look here please: https://github.com/phpro/grumphp/pull/972/files#diff-11109f893cd4e3ac1046f8b102680ee842042e472dcddb9cfd80db2f0b699ff1

I did, as I explain in https://github.com/drud/ddev/issues/3518#issuecomment-1011967625
it doesn't work properly.

I don't see &> /dev/null in your code.
And please elaborate "does not work" means. We can not help you like that.

I tried &> /dev/null locally, since it breaks things I did not update my PR.
As I explained in the above comment, when using &> /dev/null:

files get commited regardless of the output of grumphp
grumphp is running after the files are commited
I get full verbosity (as describe is first post) but I do not return on the terminal until I press enter

As it is in my PR the output is the one displayed in the screenshots I provided. It works fine, but it is more verbose than it should be.

It might be easier if you try these commands without grumphp to understand what's going on.
ddev exec does not return with the exit code of the command for example (something we could improve), but just SUCCESS (0) or FAILURE (1). This should be enough for your case though.
$ ddev exec "git ...."
$ echo $?
$ ddev exec "git .... &> /dev/null"
$ echo $?

Also that your .git-hook (pre-commit?) runs after the commit is pretty weird but also something that is not about ddev really.

Adding &> /dev/null in the command completely hides the grum output.

I really don't feel that this is the way to tackle this. A flag for ddev exec makes more sense in this case imho.

Yup, the idea is that the project lives completely inside DDEV, and the host machine has only docker+ddev.
I already know how to make the commands work inside and outside of ddev, but when I am outside (in the host) I get this extra verbose feedback from ddev exec.

Maybe the option should be called --no-error-output or maybe we should get rid of the additional error output entirely @rfay ?

Could you please demonstrate this with a trivial command instead of a fancy setup?
Is it just stderr you want to be rid of? Or stdout + stderr?
For example, if I do this command, how would you want it to behave differently?
$ ddev exec "ls junker99"
ls: cannot access 'junker99': No such file or directory
Failed to execute command ls junker99: exit status 2

The "ls: cannot access 'junker99': No such file or directory" is stderr coming from the "ls" command.
The "Failed to execute" is coming from ddev.
Please say exactly how you'd like it to change.

Please help me clarify:

ls: cannot access 'junker99': No such file or directory is stdout
Failed to execute command ls junker99: exit status 2 is stderr, right?

In any case, I could use an option to not show the Failed to execute command...
Does this make sense?

It's complicated.
Plain login shell ls junker99 outputs its error ("ls: cannot access...") to stderr and exits with exit code 2 (indicating "serious trouble" as per the man page).
docker-compose exec turns stderr into stdout (as far as I know this is by design. docker exec does not do that). It exits with code 2.
A little plain docker-compose test:
cat <<EOF > docker-compose.yaml
services:
  web:
    image: debian
    command: sh -c 'while true; do sleep 1; done'
EOF

docker-compose up -d
echo "docker-compose plain:"
docker-compose exec web ls junker99 # outputs "ls: junker99 not found"
echo "docker-compose without stderr:"
docker-compose exec web ls junker99 2> /dev/null # although the output is stderr, it still outputs "ls: junker99 not found"
echo "Exit code: $?" # outputs "Exit code: 2"
# you can do tests with `docker exec` here. Depending on your directory/docker-compose project name
# something like "docker exec tmp_web_1 ls junker99 2> /dev/null"
docker-compose down

ddev exec junker99 outputs the stderr of the command to stdout, then adds its own output if there has been an error ("Failed to execute : exit status 2") to stderr and exits with code 1.
So long story short: It would be nice to not have ddev output the erroring command line.
I would fancy even turning that around and making it the norm.
In places where this has been helpful debugging information (exec: hooks) it could still be output by the caller (or we could have a --show-command-on-error).
Having the correct exit code is not what the issue starter requested and (probably) another story.

First @bserem

In any case, I could use an option to not show the Failed to execute command...

I wouldn't be opposed to that. Maybe by default. And maybe ddev could actually parse the docker-compose or docker output to determine the error code and return that. It's an ugly hack, but might be worth it.
For now, could you use ddev exec ... | grep -v "Failed to execute command" - would that solve your problem?

It's really annoying that docker-compose doesn't return the actual return value.
There are two basic classes of problem here:

docker-compose exec doesn't return the actual exit code but docker exec does.
Neither docker-compose nor docker clients properly keep stdout and stderr sorted. They return combined stdout and stderr as stdout.

The first is fairly easy to sort - we could change ddev exec to use the direct-docker-exec technique instead of docker-compose exec and I think this would be preferable. The underlying code is already there and used for some other things.
The second is either a problem with the clients or with the docker api, I don't know which.
There are several related issues, most important seems to be https://github.com/docker/compose/issues/8098

I think https://github.com/docker/compose/issues/8098 is because docker-compose exec defaults to using a tty (so is the equivalent to docker exec -it ....
Does it work if it would use docker-compose exec -T ...  ?

Neither docker-compose nor docker clients properly keep stdout and stderr sorted. They return combined stdout and stderr as stdout.

Issue related to that is https://github.com/moby/moby/issues/31706. If I recall correctly, the root cause for that is that on Linux stdout is "buffered", but stderr not. Docker collects data from both, but due to one of them being buffered, output may not appear directly and because of that appear out of order.

Does it work if it would use docker-compose exec -T ...  ?

Yes, that keeps stderr as stderr with my example (tested with docker-compose v1, v2.2.0 and v2.2.3). Thank you for that! I never knew it had something to do with "pseudo-TTY"!
So docker-compose -T does everything right (stderr stays stderr, exit code is correct) if I understand the problem correctly.
# just something to play with. If stderr stays stderr, it should not get output
docker-compose exec -T web \
  sh -c 'echo "to stdout"; echo "to stderr" 1>&2; exit 2' \
  2> /dev/null
echo "exit code: $?"
# outputs: "to stderr" "exit code 2"

ddev exec already uses -T if in terminal context, see https://github.com/drud/ddev/blob/11f2eeec84f8de831d9ee5cb6b4df67ac41ff360/pkg/ddevapp/ddevapp.go#L1240-L1242
Most ddev exec are assumed to be interactive these days, so it's important not to mess with that.
OTOH, it wouldn't surprise me if ddev isn't properly processing stdout/stderr, a PR would be welcome.
It's using dockerutils.ComposeWithStreams() in the normal situation, a quick review doesn't show problems with stdout/stderr.

@bserem not sure you saw my question above:

For now, could you use ddev exec ... | grep -v "Failed to execute command" - would that solve your problem?

I've seen it, I'll test it on Monday. Sorry for the delay

Sadly, no luck with grepping the output:

In the case of the git hook, adding a grep results in a commit, bit this is unrelated to DDEV.
This turns out to be tougher that I anticipated, and it is a niche thing as it seems. I will update the PR on GrumPHP about this discussion here.

That should be
ddev exec 'ls randomdir' 2>&1 | grep -v 'Failed to execute'

(you want to hide what ddev outputs to stderr so we merge it first into stdout)

or currently just as good:
ddev exec 'ls randomdir' 2> /dev/null

Both of the above work for simple commands (like ls random) with simple output.
None of the above work for more the complex commands for the git-hooks in from Grum.

Could you update here and in the title what it is you really want to accomplish? You just want ddev exec not to add the "Failed to execute command"?
https://github.com/drud/ddev/blob/665ce0e0042248b273836e08313ff85491dd5415/cmd/ddev/cmd/exec.go#L52-L54
You could start by just removing that and make and see if it solves your need.

I haven't forgotten about this, I am just buried deep at the moment so not active with this one. I don't feel that confident about compiling ddev right now, as this would tamper with my work-machine.

@bserem I felt so, too. But it is actually pretty easy.  The new ddev binary is then in .gotmp/bin/*/ddev.
That is totally isolated and does not affect your normal installation.
For testing, ddev poweroff your normal installation and <path to ddev-git>/.gotmp/bin/*/ddev start your test project.

@jonaseberle nice to know, thanks for the heads up. I tried it, but the web container won't start.
$ ~/Development/ddev/.gotmp/bin/linux_amd64/ddev logs -s web
[...]
Instrumentation is opted in, but SegmentKey is not available.

You don't have the private SegmentKey for complication, and this is just a warning for release-time problems, makes no difference. You can ddev config global --instrumentation-opt-in=false if it annoys you too much.

Doesn't annoy me at all, I just thought this was the reason ddev didn't start.
I'll try with a new project first thing in the morning

I can't get 2 versions of DDEV to co-exist on my machine without issues, so I still can't test this. Sorry.
At the moment time is also very limited for me to build a new machine just for tests. Maybe someone jumps in and gives some feedback.
Thanks

It's very easy to run multiple versions of ddev. It's just a binary, so you can just put it anywhere in your path. You can also rename it. Probably the easiest thing to do is to just link a version that you build into ~/bin and add ~/bin to your path before where ddev is on your system, probably /usr/local/bin.

Yeah, I got up to that part, but the second version (custom build to not display the result) does not start:
Container ddev-test-dba  Started
Container ddev-test-web  Started
Container ddev-test-db  Started
Container ddev-router  Running
Failed to start test: container failed to become healthy: err=container 'web' exited, please use 'ddev logs -s web' to find out why it failed

$ ~/Development/ddev/.gotmp/bin/linux_amd64/ddev logs -s web
+ set -o errexit nounset pipefail
+ rm -f /tmp/healthy
+ DDEV_PHP_VERSION=7.4
+ DDEV_WEBSERVER_TYPE=nginx-fpm
+ '[' -n 7.4 ']'
+ update-alternatives --set php /usr/bin/php7.4
+ ln -sf /usr/sbin/php-fpm7.4 /usr/sbin/php-fpm
+ export PHP_INI=/etc/php/7.4/fpm/php.ini
+ PHP_INI=/etc/php/7.4/fpm/php.ini
+ '[' '!' -z ']'
+ '[' -d /mnt/ddev_config/php ']'
+ '[' -d /mnt/ddev_config/nginx_full ']'
+ rm -rf /etc/nginx/sites-enabled
+ cp -r /mnt/ddev_config/nginx_full /etc/nginx/sites-enabled/
+ '[' -d /mnt/ddev_config/apache ']'
+ rm -rf /etc/apache2/sites-enabled
+ cp -r /mnt/ddev_config/apache /etc/apache2/sites-enabled
+ '[' php = backdrop ']'
+ '[' php = drupal6 ']'
+ '[' php = drupal7 ']'
++ id -un
++ id -gn
+ printf '\nexport APACHE_RUN_USER=bserem\nexport APACHE_RUN_GROUP=bserem\n'
+ a2enmod access_compat alias auth_basic authn_core authn_file authz_core authz_host authz_user autoindex deflate dir env filter mime mpm_prefork negotiation reqtimeout rewrite setenvif status
Considering dependency authn_core for access_compat:
Module authn_core already enabled
Module access_compat already enabled
Module alias already enabled
Considering dependency authn_core for auth_basic:
Module authn_core already enabled
Module auth_basic already enabled
Module authn_core already enabled
Module authn_file already enabled
Module authz_core already enabled
Considering dependency authz_core for authz_host:
Module authz_core already enabled
Module authz_host already enabled
Considering dependency authz_core for authz_user:
Module authz_core already enabled
Module authz_user already enabled
Module autoindex already enabled
Considering dependency filter for deflate:
Module filter already enabled
Module deflate already enabled
Module dir already enabled
Module env already enabled
Module filter already enabled
Module mime already enabled
Considering conflict mpm_event for mpm_prefork:
Considering conflict mpm_worker for mpm_prefork:
Enabling module mpm_prefork.
Module negotiation already enabled
Module reqtimeout already enabled
Enabling module rewrite.
Module setenvif already enabled
Module status already enabled
To activate the new configuration, you need to run:
  service apache2 restart
+ a2enconf charset localized-error-pages other-vhosts-access-log security serve-cgi-bin
Conf charset already enabled
Conf localized-error-pages already enabled
Conf other-vhosts-access-log already enabled
Conf security already enabled
Conf serve-cgi-bin already enabled
+ '[' nginx-fpm = apache-fpm ']'
+ '[' false = true ']'
+ disable_xdebug
Disabled xdebug
+ phpenmod assert
+ ls /var/www/html
+ sudo mkdir -p /mnt/ddev-global-cache/terminus/cache
+ sudo mkdir -p /mnt/ddev-global-cache/bashhistory/test-web /mnt/ddev-global-cache/mysqlhistory/test-web
++ id -u
++ id -g
+ sudo chown -R 1000:1000 /mnt/ddev-global-cache/
+ '[' -d /mnt/ddev_config/.homeadditions ']'
+ cp -r /mnt/ddev_config/.homeadditions/. /home/bserem/
cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/bserem/./.gitconfig' are the same file
Instrumentation is opted in, but SegmentKey is not available.

You have something very, very wrong going on there, maybe it's an obscure bug or something. But...
cp: '/mnt/ddev_config/.homeadditions/./.gitconfig' and '/home/bserem/./.gitconfig' are the same file
means that perhaps you have used a symlink for your .gitconfig? Or... ???
I'd love to know how you created this. It's trying to copy /mnt/ddev_config/.homeadditions into the /home directory and it's finding they're the same file. That means a link of some type has been used.
If you just remove the .gitconfig that you have placed in either ~/.ddev/homeadditions or <project>/.ddev/homeadditions it will solve this, but I don't know how you can even get this.

Oh! Yes, I did have a symlink:
$ ls -la .gitconfig
.gitconfig -> /home/bserem/.gitconfig

I removed it for our tests.

So, by following your suggestion (https://github.com/drud/ddev/issues/3518#issuecomment-1015468352) I got this to work:

Here is the old result for reference:

Your existing symlink could never work anyway, because it was trying to reach into the host when it's being used in the container. In the future, you'll want to copy your .gitconfig in there.

That doesn't help though if you want to use the return value of a program run through ddev exec.
If you want to run a program within a ddev container and use it like you are used from local commands or docker/ssh/..., you would currently have to wrap ddev exec to get rid of the error output.
So I still think this (and the stderr > stdout conversion) is not so nice currently.
This is the closest you get to native behaviour IMHO:
docker exec "$(ddev describe -j | jq -r '.raw.services.web.full_name')" ls randomdir

=> stderr stays stderr, return value is what the command has set and no additional error output from the "wrapper program" (ddev).

@gilbertsoft apparently @bserem wasn't just bothered by the return code - I think we talked about that at length. It was the actual stderr output that bothered him.

It seems to me like this either got solved or isn't useful in most workflows, but I'm going to close this for now, happy to continue the conversation here or elsewhere.


[SPARK-10564] ThreadingSuite: assertion failures in threads don't fail the test

This commit ensures if an assertion fails within a thread, it will ultimately fail the test. Otherwise we end up potentially masking real bugs by not propagating assertion failures properly.

Merged build triggered.

Merged build started.

Test build #42346 has started for   PR 8723 at commit a64e7c9.

LGTM

Test build #42346 has finished for   PR 8723 at commit a64e7c9.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MultilayerPerceptronClassifier(JavaEstimator, HasFeaturesCol, HasLabelCol, HasPredictionCol,
class MultilayerPerceptronClassificationModel(JavaModel):
class MinMaxScaler(JavaEstimator, HasInputCol, HasOutputCol):
class MinMaxScalerModel(JavaModel):
        ("thresholds", "Thresholds in multi-class classification to adjust the probability of " +
class HasElasticNetParam(Params):
class HasFitIntercept(Params):
class HasStandardization(Params):
class HasThresholds(Params):
    thresholds = Param(Params._dummy(), "thresholds", "Thresholds in multi-class classification to adjust the probability of predicting each class. Array must have length equal to the number of classes, with values >= 0. The class with largest value p/t is predicted, where p is the original probability of that class and t is the class' threshold.")
        self.thresholds = Param(self, "thresholds", "Thresholds in multi-class classification to adjust the probability of predicting each class. Array must have length equal to the number of classes, with values >= 0. The class with largest value p/t is predicted, where p is the original probability of that class and t is the class' threshold.")

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/42346/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #42355 has started for   PR 8723 at commit a64e7c9.

This actually already passed the ThreadingSuite, so I'm merging it into master and 1.5.

Test build #42355 has finished for   PR 8723 at commit a64e7c9.

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/42355/
Test FAILed.


Old Case Studies CMS

Hey, We are raising this ticket about some of the issues we encountered with the old case studies CMS while working on our most recent post.

[x] Infographic 1 : Cards should appear in one line
[x] Infographic 2 : Cards should appear in one line
[x] Newsletter Description 1 : Need to add paragraph breaks
[x] Impact Card : Need formatting options such as font colour and paragraph breaks
[x] Impact Number : Icon files were getting lost and thus the card would stop appearing
[x] Blog title : Remove "Blog" as subheading
[x] Op-ed title : Remove "Op-Ed" as subheading
[x] Webinar title : Remove "Webinar" as subheading

Hey @aayush-malik , for blog, op-ed, webinar title you want to make it editable from cms or want to remove title permanently?

Hey @riya-2206, please do not remove them. The idea is to keep the titles- 'Blog', 'Op-Ed' as an editable option on CMS, and not hard coded as the titles.

Hey @aayush-malik, have done all the mentioned changes and also raised the PR. Please let me know when can we merge the PR.
Also, please don't publish any cms changes using the preview link.
cc: @karntrehan
https://deploy-preview-51--samagra.netlify.app/

Hey @aayush-malik ,
Can you please test it and let me know when can we merge the PR

Hey @riya-2206, I can't seem to access the CMS for this webpage using the link you shared in this thread (https://deploy-preview-51--samagra.netlify.app/). I logged in using netlify too but all I can see is the webpage and not the CMS. Could you please help me through this?

Hey @Vivaan-Samagra , you can use /admin for cms https://deploy-preview-51--samagra.netlify.app/admin, and also you can test the data in cms but don't publish any changes using cms admin page

Hey @riya-2206, I can't seem to format (line break, embolden, etc.) the text on impact cards. Rest looks good.

@Vivaan-Samagra these formatting tags will not work here as it is used in html, let me check and make changes will update you

Hey @Vivaan-Samagra , have made changes for card text, please let me know if any changes required

Hey @Vivaan-Samagra , shall I merge these changes?

This works @riya-2206, you may merge them.


Adapt to new processing config structure

Background
Link to issue
https://biomage.atlassian.net/browse/BIOMAGE-671
Link to staging deployment URL
Coming soon
Links to any Pull Requests related to this
N.A.
Anything else the reviewers should know about the changes here
Changes
Code changes
Definition of DONE
Your changes will be ready for merging after each of the steps below have been completed:
Testing

[ ] Unit tests written
[X] Tested locally with Inframock
[X] Deployed to staging

To set up easy local testing with inframock, follow the instructions here: https://github.com/biomage-ltd/inframock
To deploy to the staging environment, follow the instructions here: https://github.com/biomage-ltd/biomage-utils
Documentation updates
Is all relevant documentation updated to reflect the proposed changes in this PR?

[ ] Relevant Github READMEs updated
[ ] Relevant wiki pages created/updated

Approvers

[ ] Approved by a member of the core engineering team
[ ] (UX changes) Approved by vickymorrison (this is her username, tag her if you need approval)

Just before merging:

[ ] After the PR is approved, the unstage script in here: https://github.com/biomage-ltd/biomage-utils is executed. This script cleans up your deployment to staging

Optional

[ ] Photo of a cute animal attached to this PR

The API part of the ticket is fixed together with https://github.com/biomage-ltd/api/commit/897a7ce3fffec4af46d9b1ddeeab53b7aa9046c9

Could you add a staging?

This ticket involves also adapting the processing config validation on the api side too

Sorry for taking so long to review: now some merge conflicts have appeared (and potentially with other active PRs) :-(
It does not look to me that the auto radio button works properly, but I'll work on that as a part of my ticket.

Right now there is no functionality for the auto radio button, it just updates the redux store.


No. of rows not documented

I was having some trouble to set the initial no. of rows. After looking into the source code I figured out that there is :rows="1" but its not documented anywhere. Can you please add it. Will be helpful for many

There is no any initial no. of rows in our sources. You can just add it. e.g.  <textarea-autosize rows="10"...

See App.vue here


Thread panic sometimes when attempting to fetch CPU info on Thinkpad X1 Nano Gen 1

Describe the bug
I'm getting a thread panic on my new Thinkpad X1 Nano Gen 1 sometimes when attempting to fetch CPU information. I think the problem is a kernel bug which prevents the data from being read however it should be handled properly instead of panicking.
To Reproduce
Steps to reproduce the behavior:

Run command 'acs get cpus'
Received error

Expected behavior
The os error 61 should be handled properly instead of panicking.
**Screenshots

Version
autoclockspeed 0.1.11 installed with cargo install
I will work to fix this issue since it seems to be a hardware issue with my specific laptop and might be difficult to reproduce.

Seems like the issue is with a certain thermal zone not reading properly on my system, I bet it is a known issue with my outdated firmware/bios which I haven't updated in a while. I'll try to fix the panic within auto-clock-speed either way then update my computer.

Update: the thermal zone is now working and auto clock speed is functioning as expected now. We for sure need to improve error handling on this front because acs should not stop working just because one thermal zone cannot be read. I will work on this issue.

Thanks!!


fix License in pkg/ddc/alluxio/load_data.go

Ⅰ. Describe what this PR does
This pr is to fix License in pkg/ddc/alluxio/load_data.go
Ⅱ. Does this pull request fix one issue?
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@neverland-lhy Thanks very much for you contribution!


Add wrapping logic from TFestBand to transfer function codes

This relates to #7 #54 and #61

We have the ability to regress each output channel independently as of the closure of issue #7.  The needed elements now are to add weights:
channel_weights,
segement_weights,
edf_weights
and test that these all run.
Here is a matlab snippet that interprets an array of weights based on the array shape.
                if nargin==4                     [m,n] = size(wt);                     if m==obj.Nseg && n == 1                         obj.WtSeg = wt;                         obj.weights = 'Seg';                     else                         if n~=obj.NchOut || m== obj.Nseg                             obj.WtCh = wt;                             obj.weights = 'Ch';                         else                             error('TTFestBand: WtCh must be same size as Y');                         end


RASS-586 500 error when trying to submit offence without overall sentence lengh

I think this can be fixed by changing CalculateReleaseDatesService.compareOverallSentenceLength
There is a check to see if the overall sentence length is set:
    if (sentences.length && appearance.hasOverallSentenceLength) {

I didn't realise it was a string, not a boolean. It needs to be:
    if (sentences.length && appearance.hasOverallSentenceLength === 'true') {


chore: bump version to beta.4 and update changelog

Updates for beta.4

Verbal LGTM from @jelbourn


updated virtual_env segment to display according to activate file

fixed virtual_env in config file to point to the right file
modified the virtual_env.py file to conform with the ENV/bin/activate file prompt

BTW, the nvm support added recently:
for those who doesn't have the .nvm file in the folder, it breaks the prompt.
you should try except in the nvm.py file

ignore


UX: improve design of kubectl Commands page

we are using a kui table to display the subcommands. proposal: use markdown so that the content can be read as a document, rather than a table.
currently:

Proposal:

also, the Options part isn't always parsed properly: e.g. in the case of "-f, --filename: xxxx", we parsed that out as "-f" with a doc string of "--filename: xxxx"


Incorrect sounds and state ids

I noticed that some objects have wrong sounds attached to them. I managed to fix them myself in wadtool but everytime a user creates a TEN wad then they have to keep fixing them.

Hi
Can you be specific as to which have the wrong sounds ?
Are you using the most update to date version of Tomb Editor? Because they
remap across fine .
On Wed, 16 Nov 2022, 15:07 Jakub, @.***> wrote:

I noticed that some objects have wrong sounds attached to them. I managed
to fix them myself in wadtool but everytime a user creates a TEN wad then
they have to keep fixing them.
—
Reply to this email directly, view it on GitHub
https://github.com/MontyTRC89/TombEngine/issues/851, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUDDIUFSASX7NVBSOXAG2DTWIT2BJANCNFSM6AAAAAASCKWNKM
.
You are receiving this because you were assigned.Message ID:
@.***>

lara when climbing up on a shelf, there should be a metal sound playing but another sound plays LOOT_SHELF frame 123
wrong harpoon guns
no chest opening sfx
wrong valve sfx on frame 129 of VALVE_TURN animation
no cabinet opening sfx LOOT_CABINET
This is just few I noticed but there may be more.

and yes i am using up to date TE

Changed the title of this issue because it is misleading and heavily implied ALL sounds and stateIDs were incorrect which really isn't true.
Any amendments will be completed in due course when reported.

[x]  lara when climbing up on a shelf, there should be a metal sound playing but another sound plays LOOT_SHELF frame 123
[x ]  wrong harpoon guns
[x]  no chest opening sfx
[x]   wrong valve sfx on frame 129 of VALVE_TURN animation
[x]  no cabinet opening sfx LOOT_CABINET

"wrong valve sfx on frame 129 of VALVE_TURN animation" is subjective. It is copied based on the TR5 sinking sub.
"No Cabinet opening" fixed.
"No Chest Opening SFX" added. Not the original sound used but one I think fits better.
Removed Harpoon Gun draw sound effect when underwater.


Update it.js

Partial update of the translation of the Italian language.
I council calls only Italian to translate, because the Italian language is complex enough.

thanks!


eslint with react plugin

This reduces a number of linting "errors" that shouldn't be errors

Thank you. The project still has a few rough edges.

No problem! Really love the project, though. Was great for getting started on a prototype I was working on.


strange condition and debug statement

@AucaCoyan
i'm a bit confused about that part of format_single_file, why do we want the code before and after to be equal? :open_mouth: https://github.com/nushell/nufmt/blob/1765f8e5191b42aad4c7b72b89bef1de05a4eed2/src/lib.rs#L21-L23
and followup question, why don't we have a similar test and debug statement in format_string?

It's for the rare case that the file is formatted, and you run the formatter anyways.
In that case, instead of printing to stdout onlyformatting file
we can print
formatting file
file is formatted correctly

if we check and did no change at all.

oooooh i get it, thanks for the clarification :yum:
maybe file is already formatted correctly would be clearer? :open_mouth:

That sounds great!


[SPARK-41216][CONNECT][PYTHON] Implement DataFrame.{isLocal, isStreaming, printSchema, inputFiles}

What changes were proposed in this pull request?
1, Make AnalyzePlan support specified multiple analysis tasks, that is, we can get isLocal, schema, semanticHash together in single RPC if we want.
2, Implement following APIs

isLocal
isStreaming
printSchema
semanticHash
sameSemantics
inputFiles

Why are the changes needed?
for API coverage
Does this PR introduce any user-facing change?
yes, new APIs
How was this patch tested?
added UTs

cc @HyukjinKwon @cloud-fan @amaliujia @grundprinzip

Merged into master, thank you all!