Plan output not properly formatted

Describe the bug
Capturing the terragrunt plan output to use it into a comment is not properly formatted.
The plan output is on a single line.

To Reproduce
Steps to reproduce the behavior, code snippets and examples which can be used to reproduce the issue.
    - name: Check terragrunt HCL
      uses: gruntwork-io/terragrunt-action@v1
      id: fmt
      with:
        tf_version: ${{ inputs.tf_version }}
        tg_version: ${{ inputs.tg_version }}
        tg_dir: ${{ inputs.workdir }}
        tg_command: 'hclfmt --terragrunt-check --terragrunt-diff'

    - name: Plan
      id: plan
      uses: gruntwork-io/terragrunt-action@v1
      with:
        tg_comment: 1
        tf_version: ${{ inputs.tf_version }}
        tg_version: ${{ inputs.tg_version }}
        tg_dir: ${{ inputs.workdir }}
        tg_command: 'plan'

    - uses: actions/github-script@v6
      if: github.event_name == 'pull_request'
      env:
        PLAN: "terraform\n${{ steps.plan.outputs.tg_action_output }}"
      with:
        github-token: ${{ inputs.GITHUB_TOKEN }}
        script: |
          // 1. Retrieve existing bot comments for the PR
          const { data: comments } = await github.rest.issues.listComments({
            owner: context.repo.owner,
            repo: context.repo.repo,
            issue_number: context.issue.number,
          })
          const botComment = comments.find(comment => {
            return comment.user.type === 'Bot' && comment.body.includes('Terragrunt Format and Style')
          })

          // 2. Prepare format of the comment
          const output = `#### Terragrunt Format and Style 🖌\`${{ steps.fmt.outcome }}\`

          #### Terragrunt Plan 📖\`${{ steps.plan.outcome }}\`

          Show Plan

          \`\`\`\n
          ${{ steps.plan.outputs.tg_action_output }}
          \`\`\`

          *Pusher: @${{ github.actor }}, Action: \`${{ github.event_name }}\`, Workflow: \`${{ github.workflow }}\`*`;

          // 3. If we have a comment, update it, otherwise create a new one
          if (botComment) {
            github.rest.issues.updateComment({
              owner: context.repo.owner,
              repo: context.repo.repo,
              comment_id: botComment.id,
              body: output
            })
          } else {
            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: output
            })
          }

Expected behavior
Should output the plan formatted as shown in the action log:

Nice to have

[X] Terminal output
[X] Screenshots

Versions

Terragrunt Action version: v1
Environment details (Terragrunt version, Terraform version, etc.):

tg version: 0.46.3
tf version: 1.5.6

Additional context
I've tried a few things to output in the proper format but no luck so far.

@Mmasson-01 @denis256 Was having similar problem with terraform, and it was solved but reformatting the plan output. I think it should work here too
      # Sed is taking all lines that begin with one or more spaces followed by a `+` or `-`.
      # It stores the amount of spaces in `\1` and the +/- in `\2`.
      # Then replace that portion of the line with `\2\1` (+/- followed by the number of matched spaces).
      # cat is used instead of echo to avoid issues with quotes.
      # plan is limited initially to 65300 characters as its the GitHub env variable limit
      - name: Reformat Plan
        if: steps.plan.outcome == 'success'
        run: |
          plan=$(cat <<'EOF'
          ${{ format('{0}{1}', steps.plan.outputs.stdout, steps.plan.outputs.stderr) }}
          EOF
          )
          echo "PLAN<<EOF" >> $GITHUB_ENV
          echo "${plan:0:65300}" | sed -E 's/^([[:space:]]+)([-+])/\2\1/g' | grep -v 'Refreshing state' >> $GITHUB_ENV
          echo "EOF" >> $GITHUB_ENV

@vorotech I attempted to use the code you pasted and am getting the following error:
Error: Process completed with exit code 1.
Error: Unable to process file command 'env' successfully.
Error: Invalid value. Matching delimiter not found 'EOF'

The code I am using is the following:
      - name: Reformat Plan
        if: steps.plan.outcome == 'success'
        run: |
          plan=$(cat <<'EOF'
          ${{ format('{0}', steps.plan.outputs.tg_action_output) }}
          EOF
          )
          echo "PLAN<<EOF" >> $GITHUB_ENV
          echo "${plan:0:65300}" | sed -E 's/^([[:space:]]+)([-+])/\2\1/g' | grep -v 'Refreshing state' >> $GITHUB_ENV
          echo "EOF" >> $GITHUB_ENV

      - uses: actions/github-script@v6
        with:
          script: |
            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: "${{ env.PLAN }}"
            })

Unless I am doing something glaringly stupid (possible since I am sleep deprived) it looks like it will have to be modified a bit. I have been looking for an excuse to learn sed so I can take it on when I have time. Just an FYI.

@aiell0 I haven't tested myself. I mentioned that this is a code I'm using with terraform, not terrugrunt yet. And to be precise my previous step with plan looks like. Maybe the show command does slightly different output than plan.
      - name: Plan Terraform
        id: plan
        continue-on-error: true
        run: |
          cd ${{ inputs.TF_PATH }}
          terraform plan -input=false -no-color -out=tfplan \
          && terraform show -no-color tfplan

I will give it a try and update here.

@vorotech I actually think a better way to do this would be to just change the module. For example, we could include another option to colorize using that clean_colors function. Right now, we are basically fighting a sed statement with another sed statement, seems brittle to me.

Yo @vorotech, in case you still have the same issue, here's my current workaround for this:
      - name: Terraform Plan
        uses: gruntwork-io/terragrunt-action@v1
        id: plan
        with:
          tg_command: 'plan'
          tf_version: ${{ env.TF_VERSION }}
          tg_version: ${{ env.TG_VERSION }}
          tg_dir:     ${{ matrix.tg_dirs }}

      - name: Update result icon
        if: always()
        run: |
          if [[ ${{ steps.plan.outcome }} == "success" ]] ; then echo "RESULT_ICON=$(echo ✅)" >> $GITHUB_ENV  ; else echo "RESULT_ICON=$(echo ❌)" >> $GITHUB_ENV ; fi

      - name: Output Cleaning
        run: |
          TG_OUT=$(echo "${{ steps.plan.outputs.tg_action_output }}" | sed 's|%0A|\n|g')
          echo "TG_PLAN_OUTPUT<<EOF" >> $GITHUB_ENV
          echo "$TG_OUT" >> $GITHUB_ENV
          echo "EOF" >> $GITHUB_ENV

      - name: Comment PR
        uses: actions/github-script@v6
        if: github.event_name == 'pull_request'
        with:
          script: |
            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: `### *${{ github.workflow }}* Action Summary 🚀
              #### Terragrunt Format and Style 🖌 ${{ steps.fmt.outcome }} ${{ env.RESULT_ICON }}
              #### Terraform Plan Results: **${{ steps.plan.outcome }}** ${{ env.RESULT_ICON }}

              Show Plan

              \`\`\`\n
              ${{ env.TG_PLAN_OUTPUT }}
              \`\`\`

              Job 🔗: [Job Number ${{ github.run_number }}](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }})
              Pusher: *@${{ github.actor }}*, Action: *${{ github.event_name }}*`
            })

      - name: Generate Summary
        if: always()
        run: |
          cat > $GITHUB_STEP_SUMMARY << EOL
          ## *${{ github.workflow }}* Action Summary 🚀
          Terragrunt Format and Style 🖌 ${{ steps.fmt.outcome }} ${{ env.RESULT_ICON }}
          Terraform Plan 📖 ${{ steps.plan.outcome }} ${{ env.RESULT_ICON }}

          Show Plan

          \`\`\`\n
          ${{ env.TG_PLAN_OUTPUT }}
          \`\`\`

          Pusher: *@${{ github.actor }}*, Action: *${{ github.event_name }}*
          EOL

Yo! in case you still have the same issue, here's my current workaround for this:
      - name: Terraform Plan
        uses: gruntwork-io/terragrunt-action@v1
        id: plan
        with:
          tg_command: 'plan'
          tf_version: ${{ env.TF_VERSION }}
          tg_version: ${{ env.TG_VERSION }}
          tg_dir:     ${{ matrix.tg_dirs }}

      - name: Update result icon
        if: always()
        run: |
          if [[ ${{ steps.plan.outcome }} == "success" ]] ; then echo "RESULT_ICON=$(echo ✅)" >> $GITHUB_ENV  ; else echo "RESULT_ICON=$(echo ❌)" >> $GITHUB_ENV ; fi

      - name: Output Cleaning
        run: |
          TG_OUT=$(echo "${{ steps.plan.outputs.tg_action_output }}" | sed 's|%0A|\n|g')
          echo "TG_PLAN_OUTPUT<<EOF" >> $GITHUB_ENV
          echo "$TG_OUT" >> $GITHUB_ENV
          echo "EOF" >> $GITHUB_ENV

      - name: Comment PR
        uses: actions/github-script@v6
        if: github.event_name == 'pull_request'
        with:
          script: |
            github.rest.issues.createComment({
              issue_number: context.issue.number,
              owner: context.repo.owner,
              repo: context.repo.repo,
              body: `### *${{ github.workflow }}* Action Summary 🚀
              #### Terragrunt Format and Style 🖌 ${{ steps.fmt.outcome }} ${{ env.RESULT_ICON }}
              #### Terraform Plan Results: **${{ steps.plan.outcome }}** ${{ env.RESULT_ICON }}

              Show Plan

              \`\`\`\n
              ${{ env.TG_PLAN_OUTPUT }}
              \`\`\`

              Job 🔗: [Job Number ${{ github.run_number }}](${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }})
              Pusher: *@${{ github.actor }}*, Action: *${{ github.event_name }}*`
            })

      - name: Generate Summary
        if: always()
        run: |
          cat > $GITHUB_STEP_SUMMARY << EOL
          ## *${{ github.workflow }}* Action Summary 🚀
          Terragrunt Format and Style 🖌 ${{ steps.fmt.outcome }} ${{ env.RESULT_ICON }}
          Terraform Plan 📖 ${{ steps.plan.outcome }} ${{ env.RESULT_ICON }}

          Show Plan

          \`\`\`\n
          ${{ env.TG_PLAN_OUTPUT }}
          \`\`\`

          Pusher: *@${{ github.actor }}*, Action: *${{ github.event_name }}*
          EOL

@escanoru I ended up needing to clarify the statement a bit but got this working:
      - name: Output Cleaning
        id: clean
        run: |
          TG_OUT=$(echo '${{ steps.plan.outputs.tg_action_output }}' | sed 's|%0A|\n|g ; s|%3C|<|g')
          echo "TG_PLAN_OUTPUT<<EOF" >> $GITHUB_ENV
          echo "${TG_OUT:0:65300}" >> $GITHUB_ENV
          echo "EOF" >> $GITHUB_ENV

Thanks so much to everyone in here!

This is still fix problem on the surface.
The real solution should fix within Terragrunt action script.

Found a bug here....if the plan fails, it doesn't show up. Will look into a fix.

@escanoru I would suggest add ${TG_OUT:0:65300} on your output cleaning step to avoid argument too long error from GitHub script.

Thanks for the tip @kamontat, I didn't know about the Github action character limit.

Found a bug here....if the plan fails, it doesn't show up. Will look into a fix.

I came across this today, the reason is happening is because I forgot to add if: always() condition on the 'Output Cleaning' step, after adding this, I caused an error on purpose and was able to get the fail plan. Hope this helps you.
Cheers!

If you look in the action code, it escapes the plan so it can be passed as the action output.
You can find how it is done here: clean_multiline_text.
To make it multiline again you can unescape it in your script:
`
....
const multiline_plan = unescape('${{ inputs.terragrunt_plan_output }}');
....
Show Plan
```\n
${multiline_plan}
```
....
`


feat(graphql-elasticsearch-transformer): add 'from' query parameter

Issue #, if available: closes #5086
Description of changes:
adds offset pagination for searchable queries. Uses the elasticsearch parameter 'from'.
adds test for new feature to verify offset.
updates snapshots for searchable template.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@RossWilliams LGTM, but looks like another snapshot needs to be updated in
FAIL src/__tests__/SearchableAuthTransformer.test.ts
  ● test auth logic is enabled for iam/apiKey auth rules in response es resolver


[gRPC] Design support for controlling integral type encodings

Protobuf supports many integer types, and most types control the encoding of the integer as part of the binary-packed protobuf message.

int32 / int64 : variable-length 32/64 bit signed integers, inefficient for negative numbers
uint32 / uint64 : variable-length 32/64 bit unsigned integers
sint32 / sint64 variable-length 32/64 bit signed integers, more efficient for negative numbers
fixed32/fixed64 fixed-length 4/8 byte unsigned integer, more efficient than uint32/uint64 if values are often greater than 2^{28} / 2^{56}
sfixed32/sfixed64 fixed-length 32/64 bit signed integers, always 4/8 bytes

In the prototype, we are mapping Cadl int32/int64/uint32/uint64 to the same-named types in Protobuf, but we should expose these encoding knobs to developers, since they can have a big impact on the performance characteristics of a service.

gRPC Priority: 1
Points Estimate: 3
Needs Design - possibly just default to variable-length int32/int64 and uint32/uint64 and use decorators to designate sint/fixed/sfixed.


[ISSUE #1770]Add a query message trace command in mqadmin.

What is the purpose of the change
Add a query message trace command in mqadmin.
Brief changelog
Add a query message trace command in mqadmin.

Coverage decreased (-0.09%) to 50.829% when pulling 4fca91fa6f3c91ba0b6c8f13f4fdfcfffb2401d7 on zhangjidi2016:add_query_trace_command into 3974677f04815609951c17059d85d3795eb51247 on apache:develop.

The command result in console，@zongtanghu @duhenglucky ，please help to review it，thanks！


chore: bump version to 1.2.0 Beta 1

Update changelog and Cargo.toml

coverage: 95.796%. remained the same
when pulling e1e9bcf992db8197dc734744e1c9f052c0de346b on version-120-beta1
into 44537d53e218746196c5853bb10a8a2f0f85a58c on master.


Dealing with multiple databases

currently for you to leverage the tbl function you have to specify upfront the database.
Its normal in projects that you need to use tables from multiple databases.
So would be great if we can find a way to as part of the tbl statement add the database name prior to table name.
tbl(sc, "databasename.tablename")

perhaps another feature that could be added is src_databases(sc) similar to src_tbls(sc)
This will smooth the joins of tables from multiple databases and avoid multiple dbGetQuery(sc, "use database").
Thanks

I found it depends on dplyr behavior. dplyr assumes the second argument of tbl() is only for the table name and it doesn't handle database name.
https://github.com/hadley/dplyr/blob/master/R/sql-escape.r#L241-L248

Great stuff thanks


Ondex GenericOBO Parser

Fix problem with latest TO version
Add support for Disease Ontology (requested by Kumar)

New generic parsers for OBO and OWL are available in ondex-knetbuilder


Fixed an issue where empty SVP reports were causing errors.

Coverage increased (+0.004%) to 99.051% when pulling 00e1dab9ba14269b715ef5d963a98c14b55c296d on zmoore-usgs:master into beaa1763b73f0c663a01ca0bf6aed00430fba016 on USGS-R:master.

Coverage increased (+0.004%) to 99.051% when pulling 00e1dab9ba14269b715ef5d963a98c14b55c296d on zmoore-usgs:master into beaa1763b73f0c663a01ca0bf6aed00430fba016 on USGS-R:master.


Failed to patch

FailtoPatch.zip

Hey,
Thanks for reporting!
Seems like you are using .NET Framework.
This tool isn't tested with that and I suggest you to migrate your project to .NET (Core) as .NET Framework is out of support anyways.
If not, you can change the date locally to 10 years in the future and obfuscate. This should also bypass the trial.

EDIT: I've tested with the latest version of Eaz.NET with .NET Framework 4.8 and it works just fine. Can you provide more info?

EDIT: I've tested with the latest version of Eaz.NET with .NET Framework 4.8 and it works just fine. Can you provide more info?

i use Wice lib, maybe it is not support ?
https://github.com/aelyo-softworks/Wice
you can try sample Wice.Samples.Gallery.exe


chore: release 5.1.1

:robot: I have created a release *beep* *boop*
5.1.1 (2019-06-14)
Bug Fixes

deps: update dependency @opencensus/propagation-stackdriver to v0.0.14 (#509) (3cba4dc)
deps: update dependency snakecase-keys to v3 (#510) (eb2193e)
docs: move to new client docs URL (#514) (9043cfa)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-logging/releases/tag/v5.1.1


Update index.js

Fix angular / webpack / babel not transpiling this correcly when using single chunk compilation

This module mainly targets Node.js, not the browser. It's up to you to transpile it with Babel if you want to use it in the browser. You can find a more detailed explanation here: https://github.com/sindresorhus/ama/issues/446
If you use Webpack, check out babel-engine-plugin, which transpiles only the dependencies that needs to be transpiled.
If you use Create React App, upgrade to Create React App v2. It supports automatic transpilation of dependencies, which will make this package just work.

If you use Webpack, check out babel-engine-plugin, which transpiles only the dependencies that needs to be transpiled.

This did the trick. Thank you!


Celestial Bodies Database TUTORIAL.MD instructions are not logically organized

Describe the Issue
When looking at the steps presented to users of the Celestial Bodies Database course in coderoad, I noticed that the list of requirements was not ordered in a way that would be considered logical. A person would need to read the whole list in order to re-organize it by the required chronology.
For eg. The list currently shows the following at the start
You should create a database named universe
Be sure to connect to your database with \c universe. Then, you should add tables named galaxy, star, planet, and moon

Then some steps later it says
Your database should have at least five tables

This instruction should be listed immediately after the first two as it informs us of the number of tables required.
Another example around the 10th line of instruction shows these 3 lines:
Each "star" should have a foreign key that references one of the rows in galaxy
Each "planet" should have a foreign key that references one of the rows in star
Each "moon" should have a foreign key that references one of the rows in planet

Then some number of lines later the following is mentioned:
Each primary key column should follow the naming convention table_name_id. For example, the moon table should have a primary key column named moon_id
Each foreign key column should have the same name as the column it is referencing

This makes many people get confused as they begin working on the list from top-to-bottom and they wonder why their primary key and foreign key links aren't being accepted (oh it's because you didn't read all the way down the list and set them up with the expected names)
We have had some people hit this issue, enough to make me investigate why. I do believe the reason is that the list has not been organized in a logical way (or that it was not made clear that the entire list should be read and considered before attempting to work on the project)
One more reason to re-order the list is related to how the checks are done. In order for this step for eg to be marked as done
Each "star" should have a foreign key that references one of the rows in galaxy, the user must first complete a step listed 3 rows  below this one which says Each table should have at least three rows
(people complete the foreign key setup then wonder why there is no checkmark shown next to it, and it turns out its because they haven't added the rows yet - an instruction they didn't quite get to yet)
Affected Page
https://github.com/freeCodeCamp/learn-celestial-bodies-database/blob/main/TUTORIAL.md
Your code

Expected behavior
I would expect some instruction to warn people that the list is not ordered and must be read in full in order for the steps to be clear. Or I would re-organize the list so it works from top to bottom, and a person can read one requirement at a time without having to be fully aware of the details of the remaining steps.
One possible re-organization could be:
SUBTASKS

You should create a database named universe
Be sure to connect to your database with \c universe. Then, you should add tables named galaxy, star, planet, and moon
Your database should have at least five tables
Each table should have a primary key
Each primary key should automatically increment
Each table should have a name column
All columns named name should be of type VARCHAR
Each table should have at least three columns
The galaxy, star, planet, and moon tables should each have at least five columns
At least two columns per table should not accept NULL values
At least one column from each table should be required to be UNIQUE
You should use the INT data type for at least two columns that are not a primary or foreign key
You should use the NUMERIC data type at least once
You should use the TEXT data type at least once
You should use the BOOLEAN data type on at least two columns
Each table should have at least three rows
Each "star" should have a foreign key that references one of the rows in galaxy
Each "planet" should have a foreign key that references one of the rows in star
Each "moon" should have a foreign key that references one of the rows in planet
Each foreign key column should have the same name as the column it is referencing
The galaxy and star tables should each have at least six rows
The planet table should have at least 12 rows
The moon table should have at least 20 rows

Screenshots
No response
System
N/A
Additional context
Some forum questions that relate to the confusion caused by the order include:
https://forum.freecodecamp.org/t/celestial-bodies-database-foreign-key/569351
https://forum.freecodecamp.org/t/celestial-bodies-database/559342
https://forum.freecodecamp.org/t/celestial-bodies-database/565496
https://forum.freecodecamp.org/t/tasks-in-celestial-database/568823
https://forum.freecodecamp.org/t/celestial-body-database-cant-make-some-test-pass/557609
and more.

@moT01 what do you think about the above? I like the proposed changes.

Thanks for the detailed issue and for all the help on the forum @hbar1st 🎉 I haven't been on there much lately - I'm glad to see people are getting the help they need 👍
I would prefer to leave the user stories/tests in the order they are in - that order is intentional. The goal with that was to try and make campers use more of the things they learned. e.g. the Each primary key column should follow the naming convention... test is near the bottom. If it's at the top, they are more likely to name their keys correctly right away - with it at the bottom, they may have to use some RENAME commands. Similar thoughts with the few other tests that are "out of order". I'm quite sure that all of the tests are passable without reading further down. I think the minimal frustration this may cause is worth it to make the project a little more difficult. That frustration is not the... crappy type  - e.g. broken tests or unclear instructions - it's just... campers having to do more work. Once campers get to those tests, they should be able to complete the task and pass it. I only glanced at the forum posts you linked, but I don't think the order is the key thing causing any of their problems - they mostly just aren't completing the task asked. Some of which, may be a little unclear...
Many of the tests on this and the other RDB projects check that at least one row exists. So for the Each "star" should have a foreign key that references one of the rows in galaxy (and similar) tests - it checks all the "star" rows have a foreign key referencing a galaxy. But if no rows exists, it fails. This was intentional, as well, since the test was passing if the user didn't have any rows - so if you started the project, didn't do anything but run the tests, those would pass. I think this is a little unclear and could be improved. Maybe Each "star" should have a foreign key that references one of the rows in galaxy. You need at least one row in the `star` table to pass this test.
A few of the other user stories mentioned...
Each primary key column should follow the naming convention table_name_id. For example, the moon table should have a primary key column named moon_id
and
Each foreign key column should have the same name as the column it is referencing
...seem pretty clear to me. Those campers just weren't doing what was asked. If you have any suggestions for grammer improvements on those, please share. Maybe Each foreign key column should have the same name as the column it is referencing. For example, a `moon_id` foreign key should be referencing a column named `moon_id`.

hi Tom, thanks for your kind and detailed response as well.
Tbh, I wasn't expecting your answer that you were hoping to make things a little harder via the organization of these steps. I'm not sure how I feel about this because on the one hand, that's cool, but on the other, it makes the tests look disordered and I don't feel that's fair to you/the exercise either.
One initial thought that comes to mind is that if we want the student to practice things like renaming, why don't we just tell them to do that? I have a tendency to read all instructions when I am unsure of something (eg. cooking recipes) and so if I had done this to this list, I would have immediately noticed the requirements aren't listed in order and evaded any extra work... My point is, I think, that it seems like a not very balanced/even way of treating everyone. Those more inclined to read first will have an easier time? (also I never suggested renaming the columns to anyone on the forum with this issue, so for all we know they were dropping them and re-adding them when they were made aware of the instructions) I think it is good to think about if we want different people to go thru the project in different ways or not. For eg. maybe since it is a project it doesn't matter much how they get it done but if it ever did matter then perhaps finding a more consistent method to ensure everyone practices the same skills is needed. (I have no formal experience creating course content so I do not know what the standards are for this type of thing. My prior experience as a sofware dev working for a large corp suggests that consistency is paramount so that's the instinct I'm fighting here)
Moving on though from this, yes I do think if you rewrite the instructions for the foreign keys to mention that they need to add at least one row would be helpful.

Each foreign key column should have the same name as the column it is referencing
...seem pretty clear to me.

Finally, I agree the instruction's wording is clear, but the order is the cause of the confusion (if the camper hasn't read far enough down, they end up asking for help and then get told that they missed this requirement - which I feel the need to say is listed later on in the list - and then yes they end up renaming the column) But if that was the intention anyway, I'm not sure what else there is to say about other than the previous thoughts I expressed.

hi Tom,
It is true that I can find more examples to show you of people struggling with these specific steps, but I think we have covered already that the order is meant to be the way it is so I will not dredge up more.
I would be happy to have the additional mention of adding rows to the checks for the foreign keys as that will save me and others on the forum having to state that every time it comes up.
So yes, I will  close this one so that you can open the other PR for that update and that will be perfectly fine with me.
thanks again.


Fixes E303

I'm on a roll today!

Automated message from Dropbox CLA bot
@grave-w-grave, it looks like you've already signed the Dropbox CLA. Thanks!


Update yarn.lock file

Codecov Report

Merging #16 into master will increase coverage by 1.2%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master      #16     +/-   ##
=========================================
+ Coverage   97.72%   98.93%   +1.2%
=========================================
  Files           8        8
  Lines          44       94     +50
  Branches       24       37     +13
=========================================
+ Hits           43       93     +50
  Misses          1        1

Impacted Files
Coverage Δ

src/storage.js
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


webview headers no effect !

[ y] I have reviewed the documentation
[ y] I have searched existing issues
[y ] I am using the latest React Native version

Environment

Environment:
  OS: macOS Sierra 10.12.6
  Node: 9.11.1
  Yarn: 1.5.1
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.2 Build version 9C40b
  Android Studio: 3.0 AI-171.4443003

Packages: (wanted => installed)
  react: ^16.3.0-alpha.1 => 16.3.1
  react-native: ^0.55.4 => 0.55.4

Steps to Reproduce
 <WebView
              ref = {(webview)=>{this.web = webview}}
              style={{width:'100%',height:'100%',justifyContent:'center', alignItems: 'center'}}
              source={{uri:resultUrl,headers:{'mykey':'testing'}}}//  here no work ! ! ! !
          />

Expected Behavior
can work.
Actual Behavior
android&&ios  no work !
please help me !

@zhongwuzw  你好 我想使用webview 发送get请求同时添加header信息 这样理解不对嘛？
求解答


feat: create alias for a snap

Issue
We would like to be able to create snap aliases using the charm library
Solution
Add alias method to create aliases.

I'll merge this tomorrow either way.

Thanks @dragomirp - looks like you've addressed all of Ben's feedback so I've merged this! :)


Linux: TypeError for any URL above 104 characters

Any URL above 104 characters makes pageres choke immediately (before even sending any web request it seems):
$ pageres "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

events.js:74
        throw TypeError('Uncaught, unspecified "error" event.');
              ^
TypeError: Uncaught, unspecified "error" event.
    at TypeError (<anonymous>)
    at EventEmitter.emit (events.js:74:15)
    at cleanupSync (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:46:42)
    at WriteStream.emit (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:53:12)
    at WriteStream.<anonymous> (evalmachine.<anonymous>:1660:12)
    at OpenReq.Req.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:141:5)
    at OpenReq.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:61:22)
    at Object.oncomplete (evalmachine.<anonymous>:107:15)

pageres 1.1.0
Ubuntu 2014.04
nodejs v0.10.25
http_parser: '1.0'
node: '0.10.25'
v8: '<IP_ADDRESS>'
ares: '1.10.0'
uv: '0.10.23'
zlib: '1.2.8'
modules: '11'
openssl: '1.0.1f'
npm: '1.3.10'
Difference between URLs of respectively 105 and 104 characters:
$ pageres "http://123456789012345678901234567890123456789012345678901234567890123456789012345678901234567890123.com" 1680x1080

events.js:74
        throw TypeError('Uncaught, unspecified "error" event.');
              ^
TypeError: Uncaught, unspecified "error" event.
    at TypeError (<anonymous>)
    at EventEmitter.emit (events.js:74:15)
    at cleanupSync (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:46:42)
    at WriteStream.emit (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:53:12)
    at WriteStream.<anonymous> (evalmachine.<anonymous>:1660:12)
    at OpenReq.Req.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:141:5)
    at OpenReq.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:61:22)
    at Object.oncomplete (evalmachine.<anonymous>:107:15)

$ pageres "http://12345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012.com" 1680x1080
Couldn't load url: http://12345678901234567890123456789012345678901234567890123456789012345678901234567890123456789012.com

Seems to work for me on OS X:
❯ pageres "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

✔ Generated 1 screenshot from 1 url and 1 resolution

Just we need to catch errors from the write stream too https://github.com/sindresorhus/pageres/blob/master/lib/util.js#L109? Thought that was handled by listening for errors on pipe.

@kevva You gotta learn. In programming nothing is handled for you :p

At least not when it comes to node streams.

Works fine for me too on Ubuntu.
$ pageres "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

✔ Generated 1 screenshot from 1 url and 1 resolution

@kevva What Ubuntu version?
I am using the latest LTS version, which is 2014.04

14.10.

For others who might have the same problem, a low-tech workaround is to first generate a frameset.html page like this:
<frameset rows="100%">
  <frameset cols="100%">
    <frame src="https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" frameborder="0" scrolling="no">
  </frameset>
</frameset>

and then take a screenshot like this:
pageres file:///path/to/frameset.html

Sorry I was wrong, the workaround does not work, same error :-(

If there is anything I can do to debug the problem, I would be glad to help.
I installed pageres exactly as specified in the README.

@nicolas-raoul We just need to handle the error and then have you run it to see what it says. We'll let you know.
Quite weird though. Seems like atom write throws when the filename get's very long, but only on @nicolas-raoul's system.

By any chance, are you generating a server!very!long!path.png filename and then renaming it to the value of --filename?
As you can see in the five first lines below, my Ubuntu limits file names to 144 characters in $HOME and 255 characters in /tmp. The difference is due to Ubuntu's home encryption layer.
Anyway, I run pageres in /tmp so home encryption is not a valid reason to crash.
nico@nico:~$ >`printf 'a%.0s' {1..143}`
nico@nico:~$ >`printf 'a%.0s' {1..144}`
bash: `printf 'a%.0s' {1..144}`: File name too long
nico@nico:~$ cd /tmp
nico@nico:/tmp$ >`printf 'a%.0s' {1..255}`
nico@nico:/tmp$ >`printf 'a%.0s' {1..256}`
bash: `printf 'a%.0s' {1..256}`: File name too long
nico@nico:/tmp$ pageres --filename /tmp/out.png "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

events.js:74
        throw TypeError('Uncaught, unspecified "error" event.');
              ^
TypeError: Uncaught, unspecified "error" event.
    at TypeError (<anonymous>)
    at EventEmitter.emit (events.js:74:15)
    at cleanupSync (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:46:42)
    at WriteStream.emit (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:53:12)
    at WriteStream.<anonymous> (evalmachine.<anonymous>:1660:12)
    at OpenReq.Req.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:141:5)
    at OpenReq.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:61:22)
    at Object.oncomplete (evalmachine.<anonymous>:107:15)

I humbly suggest you use short temporary names in /tmp for all temporary filenames (with mktemp or similar).

@nicolas-raoul, it's fs-write-stream-atomic that's adding a bunch of stuff to the filename.

Haven't been able to reproduce this but I've added error handling for the atomic write-stream in https://github.com/sindresorhus/pageres/commit/71e5979be5f0f0f607be04696331fefaafa3c8d5. Going to close.

Bug still present in pageres 1.3.0, please reopen:
$ pageres --version
1.3.0
$ pageres "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

events.js:74
        throw TypeError('Uncaught, unspecified "error" event.');
              ^
TypeError: Uncaught, unspecified "error" event.
    at TypeError (<anonymous>)
    at EventEmitter.emit (events.js:74:15)
    at cleanupSync (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:46:42)
    at WriteStream.emit (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/index.js:53:12)
    at WriteStream.<anonymous> (evalmachine.<anonymous>:1660:12)
    at OpenReq.Req.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:141:5)
    at OpenReq.done (/usr/local/lib/node_modules/pageres/node_modules/fs-write-stream-atomic/node_modules/graceful-fs/graceful-fs.js:61:22)
    at Object.oncomplete (evalmachine.<anonymous>:107:15)

@kevva Looks like we're still not catching all the stream errors.

@sindresorhus, I think it's fs-write-stream-atomic that's not catching all the errors.

I updated to Ubuntu 2014.10/pageres 1.4.0 and now I get a different error:
$ pageres "https://tools.wmflabs.org/wikivoyage/w/poimap2.php?lat=42.333333&lon=12.283333&zoom=13&layer=O&lang=en&name=Fabrica_di_Roma" 1680x1080

/usr/local/lib/node_modules/pageres/cli.js:98
				throw err;
				      ^
Error: ENAMETOOLONG, open '/home/nico/tools.wmflabs.org!wikivoyage!w!poimap2.php!lang=en&lat=42.333333&layer=O&lon=12.283333&name=Fabrica_-1680x1080.png.3f6f08b6d459421985fc3619a713980f'

Same error as immediately above on Ubuntu 2015.04
I have found a workaround: Do NOT use a encrypted home, and that will work.
Anyone wanting to using pageres while staying secure must create another user dedicated to pagers execution.

Fixed with Pageres 3.0.2 and NodeJS 5.2.0 on Ubuntu 2015.04 encrypted home, thanks!


Handle to grid scrollbar

I want to modify the column width.  I tried the following:
w2ui['draw'].columns[{ field: 'draw', caption: 'Draw', size: width }];
w2ui['draw'].refresh();
This should have displayed the horizontal scrollbar, since, the width is larger than the panel, but, I cannot see the scroll bar.  Is there a way to display the horizontal scroll bar always?
How to trigger when grid’s  vertical scroll bar is added, removed or scrolled?   How to retrieve scroll position?  How to scroll to a particular position?

Please post your questions to w2ui.com comments section or stackoverflow. The issue section is only for bugs.


The file list can be scrolled under the navigation bar area

This is happening without the new card style and super broken.

I think I fixed this in fe36b59b, are you using 3.0.1? Can you try using master?

Cool let me try bump my version. :)

Fixed?


drawBox with XPath doesn't work

In the following example (and many similar ones) the XPath selector as I used it in Pupeteer doesn't seem to work. I get the following error:
[MalformedLocatorException] CSS or XPath locator is malformed: .t3js-tabmenu-item:contains("Includes")
**{
  "suites": {
    "Introduction": {
      "screenshots": {
        "_default": [
          {"action": "resizeWindow", "width": 1024,"height": 768},
          {"action": "setScreenshotsDocumentationPath", "path": "Documentation"},
          {"action": "setScreenshotsImagePath","path": "Images/AutomaticScreenshots"},
          {"action": "setScreenshotsRstPath","path": "Images/AutomaticScreenshots"},
          {"action": "setCodeSnippetsSourcePath","path": "typo3conf/ext/styleguide"},
          {"action": "setCodeSnippetsTargetPath", "path": "CodeSnippets"}
        ],
        "templatesRecords": [
          {"include": "_default"},
          {"action": "setNavigationDefaultTable", "table": "sys_template"},
          {"action": "setNavigationDefaultUid", "uid": 1},
          {"action": "goToRecord"},
          {"action": "click", "link": "Includes"},
          {"action": "drawBox", "selector": ".t3js-tabmenu-item:contains(\"Includes\")"},
          {
            "action": "makeScreenshotOfElement",
            "fileName": "TemplatesStaticIncludes2",
            "selector": "t3js-module-body",
            "captionText": "Screenshot of table sys_template",
            "altText": "Screenshot of table sys_template"
          }
        ]
      }
    },
**

.t3js-tabmenu-item is a CSS selector instead of a XPath selector. The pseudo class :contains seems to have been supported temporarily by CSS3 but did not make it in the current / final release, see e.g. https://www.w3.org/TR/selectors-3/#content-selectors.


Update server_os_packages.rb

This is my idea on how to deal with the imagick php package. I tested it for Centos 6.5. What do you think?
Take a look on this site for the topic: http://tecadmin.net/install-imagemagick-on-linux/

:+1:


rofi: add options to for location, xoffset & yoffset

Just added the options location, xoffset and yoffset to be able to position rofi at the desired location

I had some spare time so I squashed and rebased this into master 6833f96c147779fe197657bbcc4bd323324cd6c3. Many thanks for the improvements!


Split out functionality and folders into core and app

Based on what @martonvago has investigated in #479. For instance, views, templates, and static go into the app.

See this comment in particular.


class DecoderModel(ModelBase) using padding token for prediction

Hi There,
In the t0/model.py's  DecoderModel class, I noticed that there are padding tokens between the input_ids and labels. It seems padding tokens are used when predicting the labels. Is this an issue?

fixed with #32 thanks to @thomasw21


Add CamTech University domain

University website: https://camtech.edu.kh/

@srengzee First stage of verification passed, official domain for CamTech University is camtech.edu.kh as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@srengzee Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[Question] virt-operator for ARM64?

Is this a BUG REPORT or FEATURE REQUEST?:
What happened:
Does kubevirt/virt-operator currently only available for X86? If yes, where is the Dockerfile and source for the image?  or the gaps to build the image for ARM64?
What you expected to happen:
kubevirt should be able to use on ARM64 as well.
How to reproduce it (as minimally and precisely as possible):
kubectl create -f https://github.com/kubevirt/kubevirt/releases/download/${KUBEVIRT_VERSION}/kubevirt-operator.yaml
and the container crashed,
kubectl get pods -n kubevirt
NAME                            READY   STATUS             RESTARTS   AGE
virt-operator-bd7d6c76b-4hkl5   0/1     CrashLoopBackOff   16         59m
virt-operator-bd7d6c76b-p9c5f   0/1     CrashLoopBackOff   16         59m

# kubectl logs virt-operator-bd7d6c76b-p9c5f -n kubevirt
standard_init_linux.go:211: exec user process caused "exec format error"

Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version): latest
Kubernetes version (use kubectl version): latest
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release): all
Kernel (e.g. uname -a):
Install tools:
Others:

Hey,
kubevirt uses bazel as build system, including container images, see e.g. https://github.com/kubevirt/kubevirt/blob/master/cmd/virt-operator/BUILD.bazel#L34. So there are no Dockerfiles.
For ARM64, seems there were some people busy with adding support for it, see https://github.com/kubevirt/kubevirt/issues/1376.

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale

/cc @zhlhahaha  is that image already supported on ARM?

/cc @zhlhahaha  is that image already supported on ARM?

/cc @zhlhahaha is that image already supported on ARM?

Not yet. Please refer to the patch https://github.com/kubevirt/kubevirt/pull/3415

/cc @zhlhahaha is that image already supported on ARM?

Not yet. Please refer to the patch https://github.com/kubevirt/kubevirt/pull/3415

/remove-lifecycle stale

/remove-lifecycle rotten

/close
We made quite some progress lately.
Let's continue tracking the remaining work in #3558.


DRAFT: Remove Pulpcore 3.28 pipelines

Remove Pulpcore pipelines related to Koji

Needs https://github.com/theforeman/pulpcore-packaging/pull/902

You can convert it to a draft with a button under Reviewers, which I've done now.

We now have one green 3.28 that published the last bits, we should be good to merge this one.


_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+0 :ok:
shelldocs
0m  0s
Shelldocs was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
0m 31s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m  8s
Maven dependency ordering for patch

+1 :green_heart:
hadolint
0m  2s
There were no new hadolint issues.

+1 :green_heart:
shellcheck
0m  0s
There were no new shellcheck issues.

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

2m 59s

Subsystem
Report/Notes

Docker
Client=19.03.8 Server=19.03.8 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1579/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1579

Optional Tests
dupname asflicense hadolint shellcheck shelldocs

uname
Linux 9556aed11d29 4.15.0-74-generic #84-Ubuntu SMP Thu Dec 19 08:06:28 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 023c6fe56b

Max. process+thread count
45 (vs. ulimit of 12500)

modules
C:  U:

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1579/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f) shellcheck=0.4.6 hadolint=1.17.5-0-g443423c

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Fix: ~= doesn't match tokens that are surrounded by spaces

See test-case for details.

Ah, you're right. Can't believe I overlooked this.


Can't access request info

example
Maybe my usage is wrong.

key code
https://github.com/Lucifier129/farrow/blob/9ecb212953d0083b94b4c146689eaa6f9062e804/packages/http/src/router.ts#L150

Type Infer in TS will not work if we add type annotation manually

const note: HttpMiddleware = (req, next) => {
  // typeof req is RequestInfo, it will never change
}

api
  .match({
    pathname: "/note"
  })
  .use(note);

api
  .match({
    pathname: "/note"
  })
  .use((req, next) => {
     // typeof req is { pathname: string }
    // because req has no type annotation,  ts can infer it
  });

Response supports the chainable style that improving the readable
Response.merge(
      Response.status(403),
      Response.json({
        code: 403,
        message: `You need pass the 'name' to me.`,
        data: null
      })
   )

// is equivalent to

Response.status(403).json({
    code: 403,
    message: `You need pass the 'name' to me.`,
    data: null
  })

The suggested way to split code is using Router, like below
// note.ts
import { Response,  Router  } from 'farrow-http'

export const router = Router()

router
  .match({
    pathname: "/note"
  })
  .use((req, next) => {
    // do something with req
   return Response.json({})
  });

// index.ts
import { Http, Router } from "farrow-http";
import { router as note } from "./note";

const http = Http();

// access /note
http.use(note)

// access /api/note
http.route('/api').use(note)

// we can add api-router to combine all

const api = Router()

api.use(note)
api.use(notes)
api.use(user)
api.use(users)

api.route('/product').use(product)

// and just attach api-router to http
http.route('/api').use(api)


Fix #405 - Added tests for chmod and fchmod using Promises

Added test cases for chmod and fchmod using Promises
Fix issue #405
Tests:

[x] Whether they are functions
[x] Whether they return a Promise object
[x] Whether they change the mode of a file

Note
I separated the chmod and fchmod tests to pinpoint any possible future problems. If we use the non-promise test model, it will only report on fchmod being broken - even if both functions break (since we test for fchmod before chmod). We should consider splitting up the non-promises tests too. Maybe put fchmod into its own file?
Also, should the mkdir test case be moved to the mkdir test file?
Even though the main point of the test is to check whether the correct mode is being set by the function, it makes more sense to me to place a mkdir test in a mkdir test file.

@coreyjjames
Thanks for the heads-up. The files have been removed from the PR.


More assertions

I built this tiny program a long time ago, I think you can add its content into lighthouse:
https://github.com/ohaddahan/solana-sandwich/tree/master
It can be used by protocols to ensure no one is trying to act malicious, for example reopening an account that the program closed, can be enforced by marking a instruction as nothing is allowed after, thus making it the end of the transaction and removing this potential issue.

A bit more context @Jac0xb , solana-sandwich is more oriented to be used from inside a solana-program to ensure it's not being abused and it's valid only within one transaction.
Hence it's a slightly different approach and issues vs. the existing ones in lightouse which (I think) fits more for cross-transaction issues, such as mev.

I appreciate this issue, I do however feel since instructions can be inspected before the transaction is signed and sent that this probably wouldn't be used by anyone.


RRBot Demo 5b broken, update code fixed

I posted issue #210 before I saw this PR.  To me, using the full name seems correct and most general, and the issue is remapping of topic in demo launch.


359 - One postgres instance for both keycloak and general database

Solves #359

Added a script to create a keycloak database on the current postgres instance.
Changed the keycloak to now connect to the newly created database

Made the changes that @imilchev proposed in an old issue https://github.com/podkrepi-bg/infrastructure/issues/17 in the infrastructure repo.
I don't know if it is still relevant though or if he is talking specifically about the infrastructure as code that is being deployed . In any case if it is needed in the infrastructure repo the solution should be similar
Notes:
There is a CREATE SCHEMA "public"; line in the init.sql file which I am unsure if is relevant in any way, but couldn't test it.
This is how the file structure in the pg-db container looks like:

Is that right considering we have 2 databases now @imilchev?

nice and quick - thanks for helping Nizamov!
I was just thinking how we are going to do the migration now, because if we merge this it will redirect to the new empty database. So we need a migration script of some kind, don't we? - @imilchev what do you think?

I was just thinking how we are going to do the migration now, because if we merge this it will redirect to the new empty database. So we need a migration script of some kind, don't we? - @imilchev what do you think?

@igoychev I think this setup is only local via docker-compose, so the data exports will be for the testing data only, is that correct?

I was just thinking how we are going to do the migration now, because if we merge this it will redirect to the new empty database. So we need a migration script of some kind, don't we? - @imilchev what do you think?

@igoychev I think this setup is only local via docker-compose, so the data exports will be for the testing data only, is that correct?

I think this is the case for the code in this PR.
From what I understand from @imilchev his intention was to also do that on production though so that we only have 1 postgres instance to manage, however I think that will require help from him and everyone responsible for production!


RHOAIENG-9473: Optionally enable Trivy for pull requests

https://issues.redhat.com/browse/RHOAIENG-9473
Description

This PR enables Trivy scan for PRs. It won't run by default for all PRs because it adds extra minutes to the jobs and the report won't be needed for all PRs. In order to execute the scan, the label trivy-scan must be added to the PR before its creation. The workflow does not react to label changes, so a new commit must be pushed to run the scan if the trivy-scan label is added after the PR is opened.
Also on this PR:

The --severity option includes all of them when the trigger is PR, while the daily report only includes HIGH and CRITICAL ones. This is because low and medium issues are numerous and could be more interesting to be listed in a PR, rather than the daily report.
Added the --ignore-unfixed option to reduce the size of the report table. An issue that does not have a fixed version is not very useful to be listed.
Added concurrency for the PR workflow that builds notebooks to reduce redundant builds.
Removed scan for secrets as it takes more time and it's not bringing value to the reports.
For reviewers: I didn't touch the pull_request: make ${{ inputs.target }} step; I only moved it up.

How Has This Been Tested?

I've executed the tests on my fork.
Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

@atheo89 can  you please create a trivy-scan label in this project, and give it some nice color?


Impossible to set pause

Hardware
ESP32
Firmware version
2.4.3
Application version
2.4.3
What happened? What did you expect to happen?
When it's been a while since I've used the virtual remote controls, I decide to close a shutter and then I decide to pause the descent but it continues. To pause it, I need to bring it back up and after i can pause it.
How to reproduce it (step by step)
1. Wait few time (I don't have the specific time)
2. Close your shutters
3. Try to pause it
4.  And now you can pause it

Logs
No response

@TapsHTS any more information regarding this?

I am going to close this for now as I assume you have figured out the issue.  If not feel free to reopen it.


resolve 0 558 :failed to update node address:4072

@jmartinez-silabs
Problem
failed to update node address:4072
platform:
ESP32+MAC
1.version
zx@ubuntu:~/connectedhomeip$ git log
commit a8ef4197acd4e879141077820abcf5d9bc76535b (HEAD, tag: test_event_3_2021_06_10, origin/test_event_3.1)
Author: doru91<EMAIL_ADDRESS>Date:   Wed Jun 9 22:55:16 2021 +0300

    [K32W] Increase IM timeout (#7490)

    * [K32W] Increase IM timeout

    On K32W, chip-device-ctrl times out while waiting for the OpCSR.
    Double the IM timeout for the moment.

    See also:
    - https://github.com/project-chip/connectedhomeip/issues/7489

    Signed-off-by: Doru Gucea<EMAIL_ADDRESS>
    * Restyled by clang-format

    Co-authored-by: Restyled.io<EMAIL_ADDRESS>
commit 7b59b8fbcc5a0e9fe1694f6c35cd64ee44d28123
Author: Damian Królik<EMAIL_ADDRESS>

2.ESP32 log:
rst:0x1 (POWERON_RESET),boot:0x12 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:7072
load:0x40078000,len:13212
load:0x40080400,len:4568
0x40080400: _init at ??:?

entry 0x400806f4
I (27) boot: ESP-IDF v4.2.1-141-g1e3638390 2nd stage bootloader
I (27) boot: compile time 00:46:20
I (27) boot: chip revision: 3
I (31) boot_comm: chip revision: 3, min. bootloader chip revision: 0
I (48) boot.esp32: SPI Speed      : 40MHz
I (48) boot.esp32: SPI Mode       : DIO
I (48) boot.esp32: SPI Flash Size : 2MB
I (52) boot: Enabling RNG early entropy source...
I (58) boot: Partition Table:
I (61) boot: ## Label            Usage          Type ST Offset   Length
I (68) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (76) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (83) boot:  2 factory          factory app      00 00 00010000 001e6400
I (91) boot: End of partition table
I (95) boot_comm: chip revision: 3, min. application chip revision: 0
I (102) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x2c26c (180844) map
I (180) esp_image: segment 1: paddr=0x0003c294 vaddr=0x3ffbdb60 size=0x03d84 ( 15748) load
I (187) esp_image: segment 2: paddr=0x00040020 vaddr=0x400d0020 size=0xc867c (820860) map
0x400d0020: _stext at ??:?

I (500) esp_image: segment 3: paddr=0x001086a4 vaddr=0x3ffc18e4 size=0x00324 (   804) load
I (500) esp_image: segment 4: paddr=0x001089d0 vaddr=0x40080000 size=0x1d614 (120340) load
0x40080000: _WindowOverflow4 at /home/zx/esp/esp-idf/components/freertos/xtensa/xtensa_vectors.S:1730

I (575) boot: Loaded app from partition at offset 0x10000
I (575) boot: Disabling RNG early entropy source...
I (576) cpu_start: Pro cpu up.
I (579) cpu_start: Application information:
I (584) cpu_start: Project name:     chip-all-clusters-app
I (590) cpu_start: App version:      test_event_3_2021_06_10
I (596) cpu_start: Compile time:     Jul 16 2021 01:24:47
I (602) cpu_start: ELF file SHA256:  54234eeaee428d3c...
I (608) cpu_start: ESP-IDF:          v4.2.1-141-g1e3638390
I (615) cpu_start: Starting app cpu, entry point is 0x400818fc
0x400818fc: call_start_cpu1 at /home/zx/esp/esp-idf/components/esp32/cpu_start.c:287

I (0) cpu_start: App cpu up.
I (625) heap_init: Initializing. RAM available for dynamic allocation:
I (632) heap_init: At 3FFAFF10 len 000000F0 (0 KiB): DRAM
I (638) heap_init: At 3FFB6388 len 00001C78 (7 KiB): DRAM
I (644) heap_init: At 3FFB9A20 len 00004108 (16 KiB): DRAM
I (650) heap_init: At 3FFD0510 len 0000FAF0 (62 KiB): DRAM
I (656) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (663) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (669) heap_init: At 4009D614 len 000029EC (10 KiB): IRAM
I (675) cpu_start: Pro cpu start user code
I (695) spi_flash: detected chip: generic
I (695) spi_flash: flash io: dio
W (696) spi_flash: Detected size(4096k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (706) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (717) all-clusters-app: All Clusters Demo!
I (727) all-clusters-app: This is ESP32 chip with 2 CPU cores, WiFi/BT/BLE,
I (727) all-clusters-app: silicon revision 3,
I (737) all-clusters-app: 2MB external flash

I (787) wifi:wifi driver task: 3ffd4f5c, prio:23, stack:6656, core=0
I (787) system_api: Base MAC address is not set
I (787) system_api: read default base MAC address from EFUSE
I (807) wifi:wifi firmware version: bb6888c
I (807) wifi:wifi certification version: v7.0
I (807) wifi:config NVS flash: enabled
I (807) wifi:config nano formating: disabled
I (807) wifi:Init data frame dynamic rx buffer num: 32
I (817) wifi:Init management frame dynamic rx buffer num: 32
I (817) wifi:Init management short buffer num: 32
I (827) wifi:Init dynamic tx buffer num: 32
I (827) wifi:Init static rx buffer size: 1600
I (837) wifi:Init static rx buffer num: 10
I (837) wifi:Init dynamic rx buffer num: 32
I (837) wifi_init: rx ba win: 6
I (847) wifi_init: tcpip mbox: 32
I (847) wifi_init: udp mbox: 6
I (857) wifi_init: tcp mbox: 6
I (857) wifi_init: tcp tx win: 5744
I (857) wifi_init: tcp rx win: 5744
I (867) wifi_init: tcp mss: 1440
I (867) wifi_init: WiFi IRAM OP enabled
I (877) wifi_init: WiFi RX IRAM OP enabled
V (877) chip[DL]: CHIP task running
I (877) chip[ZCL]: Using ZAP configuration...
I (887) BTDM_INIT: BT controller compile version [ba56601]
I (897) phy_init: phy_version 4660,0162888,Dec 23 2020
I (907) chip[ZCL]: deactivate report event
I (917) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0101', EndPoint ID: '0x01', Attribute ID: '0x0002'
I (927) app-devicecallbacks: Unhandled cluster ID: 257
I (937) app-devicecallbacks: Current free heap: 139284

I (937) chip[ZCL]: 0x3f406e5dep 1 clus 101 attr  3 not supported
I (947) chip[ZCL]: Failed to write DoorState attribute: 0x86
I (957) chip[ZCL]: 0x3f406e5dep 1 clus 101 attr 21 not supported
I (957) chip[ZCL]: Failed to write Language attribute: 0x86
I (967) chip[ZCL]: OpCreds: Initiating OpCreds cluster by writing fabrics list from admin pairing table.
V (977) chip[DIS]: Set the admin pairing table delegate
I (977) chip[ZCL]: OpCreds: Call to writeAdminsIntoFabricsListAttribute
I (987) chip[ZCL]: OpCreds: Stored 0 admins in fabrics list attribute.
I (997) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0000'
I (1007) app-devicecallbacks: Unhandled cluster ID: 40
I (1017) app-devicecallbacks: Current free heap: 139284

I (1267) app-devicecallbacks: Unhandled cluster ID: 40ack - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0001'
I (1267) app-devicecallbacks: Current free heap: 125608

I (1277) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0002'
I (1287) app-devicecallbacks: Unhandled cluster ID: 40
I (1287) app-devicecallbacks: Current free heap: 125608

I (1297) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0003'
I (1297) CHIP[DL]: BLE host-controller synced
I (1317) app-devicecallbacks: Unhandled cluster ID: 40
I (1317) app-devicecallbacks: Current free heap: 125564

I (1327) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0004'
I (1337) app-devicecallbacks: Unhandled cluster ID: 40
I (1337) app-devicecallbacks: Current free heap: 125564

I (1347) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0008'
I (1357) app-devicecallbacks: Unhandled cluster ID: 40
I (1367) app-devicecallbacks: Current free heap: 125564

I (1377) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0007'
I (1387) app-devicecallbacks: Unhandled cluster ID: 40
I (1387) app-devicecallbacks: Current free heap: 125564

I (1397) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x000a'
I (1407) app-devicecallbacks: Unhandled cluster ID: 40
I (1417) app-devicecallbacks: Current free heap: 125564

I (1417) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0028', EndPoint ID: '0x00', Attribute ID: '0x0009'
I (1427) app-devicecallbacks: Unhandled cluster ID: 40
I (1437) app-devicecallbacks: Current free heap: 125564

I (1447) chip[ZCL]: Start identify callback on endpoint 1 time 0
I (1447) chip[ZCL]: Stop identify callback on endpoint 1
I (1457) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0004', EndPoint ID: '0x01', Attribute ID: '0x0000'
I (1467) app-devicecallbacks: Unhandled cluster ID: 4
I (1477) app-devicecallbacks: Current free heap: 125564

I (1477) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0005', EndPoint ID: '0x01', Attribute ID: '0x0000'
I (1487) app-devicecallbacks: Unhandled cluster ID: 5
I (1497) app-devicecallbacks: Current free heap: 125564

I (1507) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0300', EndPoint ID: '0x01', Attribute ID: '0x0007'
I (1517) app-devicecallbacks: Unhandled cluster ID: 768
I (1517) app-devicecallbacks: Current free heap: 125564

I (1527) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0300', EndPoint ID: '0x01', Attribute ID: '0x0008'
I (1527) chip[DL]: Configuring CHIPoBLE advertising (interval 25 ms, connectable, device name CHIP-3840)
I (1547) app-devicecallbacks: Unhandled cluster ID: 768
I (1557) app-devicecallbacks: Current free heap: 125520

I (1557) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0300', EndPoint ID: '0x01', Attribute ID: '0x4001'
I (1577) app-devicecallbacks: Unhandled cluster ID: 768
GAP procedure initiated: advertise; disc_mode=2 adv_channel_map=0 own_addr_type=1 adv_filter_policy=0 adv_itvl_min=40 adv_itvl_max=40I (1587) app-devicecallbacks: Current free heap: 125520

I (1597) chip[ZCL]: WARNING: ATTRIBUTES ARE NOT BEING STORED IN FLASH!
I (1607) chip[ZCL]: DEVICE WILL NOT FUNCTION PROPERLY AFTER REBOOTING!!
I (1617) chip[DL]: CHIPoBLE advertising started
I (1617) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0500', EndPoint ID: '0x01', Attribute ID: '0x0001'
E (1627) chip[DL]: Long dispatch time: 739 ms
I (1637) app-devicecallbacks: Unhandled cluster ID: 1280
I (1637) chip[DL]: Starting ESP WiFi layer
I (1647) app-devicecallbacks: Current free heap: 125520

I (1647) phy_init: phy_version 4660,0162888,Dec 23 2020
I (1667) wifi:mode : sta (7c:9e:bd:cd:ab:28)
W (1667) wifi:Haven't to connect to a suitable AP now!
I (1667) chip[DL]: Attempting to connect WiFi station interface
I (1677) chip[DL]: WiFi station state change: NotConnected -> Connecting
I (1677) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0500', EndPoint ID: '0x01', Attribute ID: '0x0002'
I (1677) chip[DL]: Done driving station state, nothing else to do...
I (1697) app-devicecallbacks: Unhandled cluster ID: 1280
I (1707) app-devicecallbacks: Current free heap: 122552

I (1717) app-devicecallbacks: Current free heap: 122552

I (1717) chip[DL]: WIFI_EVENT_STA_START
W (1727) wifi:Haven't to connect to a suitable AP now!
I (1727) chip[DL]: Attempting to connect WiFi station interface
E (1737) wifi:sta is connecting, return error
I (1727) chip[ZCL]: Failed to send IAS Zone update. Err 0x70
E (1737) chip[DL]: esp_wifi_connect() failed: Error 12295 (0x00003007)
I (1747) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0406', EndPoint ID: '0x02', Attribute ID: '0x0001'
I (1757) chip[DL]: Done driving station state, nothing else to do...
I (1767) app-devicecallbacks: Unhandled cluster ID: 1030
I (1777) app-devicecallbacks: Current free heap: 122548

I (1777) app-devicecallbacks: Current free heap: 122548

I (1787) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0406', EndPoint ID: '0x02', Attribute ID: '0x0002'
I (1797) app-devicecallbacks: Unhandled cluster ID: 1030
I (1807) app-devicecallbacks: Current free heap: 122548

V (1817) chip[DIS]: Init admin pairing table with server storage
V (1817) chip[IN]: TransportMgr initialized
I (1827) chip[DL]: NVS set: chip-counters/GlobalMCTR = 2000 (0x7D0)
I (1827) chip[IN]: local node id is 0x0000000000BC5C01
I (1837) chip[SVR]: Network already provisioned. Disabling BLE advertisement
GAP procedure initiated: stop advertising.
I (1847) chip[DL]: CHIPoBLE advertising stopped
I (1857) chip[SVR]: Next available admin ID is 1
I (1857) app-devicecallbacks: Current free heap: 122368

I (1867) chip[SVR]: Retrieved value from server storage.
I (1877) chip[IN]: Loading certs from KVS
I (1877) chip[DIS]: Admin (0) loaded from storage. Calling OnAdminRetrievedFromStorage
I (1887) chip[ZCL]: OpCreds: Admin 0x0 was retrieved from storage. FabricId 0x000000000001B669, NodeId 0x000000000000022E, VendorId 0x0000
I (1897) chip[ZCL]: OpCreds: Call to writeAdminsIntoFabricsListAttribute
I (1907) chip[ZCL]: OpCreds: Skipping over unitialized admin with fabricId 0x000000000001B669, nodeId 0x000000000000022E vendorId 0x0000
I (1917) chip[ZCL]: OpCreds: Stored 0 admins in fabrics list attribute.
I (1927) chip[SVR]: Found admin pairing for 0, node ID 0x000000000000022E
I (1937) chip[SVR]: Restored all admin pairings from KVS.
I (1937) chip[SVR]: Found 1 stored connections
I (1947) chip[SVR]: Retrieved value from server storage.
I (1947) chip[SVR]: Fetched the session information: from 0x0000000000000000
V (1957) chip[IN]: New pairing for device 0x0000000000000000, key 0!!
I (1967) chip[SVR]: Server Listening...
0000'
I (1977) app-devicecallbacks: Unhandled cluster ID: 1026
I (1987) app-devicecallbacks: Current free heap: 111804

I (1997) app-devicecallbacks: PostAttributeChangeCallback - Cluster ID: '0x0101', EndPoint ID: '0x01', Attribute ID: '0x0000'
I (2007) app-devicecallbacks: Unhandled cluster ID: 257
I (2017) app-devicecallbacks: Current free heap: 111672

I (2027) chip[ZCL]: Failed to read AutoRelockTime attribute: 0x86
                                                                                I (2027) all-clusters-app: Short Manual(decimal) setup code:<PHONE_NUMBER>2
I (2047) all-clusters-app: Long Manual(decimal) setup code:<PHONE_NUMBER>09050177298
I (2047) all-clusters-app: QR CODE Text: 'CH:C8XA062600A50324800'
I (2057) all-clusters-app: Copy/paste the below URL in a browser to see the QR CODE:
	https://dhrishi.github.io/connectedhomeip/qrcode.html?data=CH%3AC8XA062600A50324800
I (3377) wifi:state: init -> auth (b0)ap:<255,255>, sta:<1,1>, prof:1
I (3407) wifi:state: auth -> assoc (0)
I (3437) wifi:state: assoc -> run (10)
I (3607) wifi:connected with TEST-ZX, aid = 11, channel 1, 40U, bssid = 58:41:20:d9:1a:c5
I (3607) wifi:security: WPA2-PSK, phy: bgn, rssi: -30
I (3617) wifi:pm start, type: 1

I (3617) chip[DL]: WIFI_EVENT_STA_CONNECTED
I (3617) chip[DL]: WiFi station state change: Connecting -> Connecting_Succeeded
I (3637) chip[DL]: WiFi station state change: Connecting_Succeeded -> Connected
I (3647) chip[DL]: WiFi station interface connected
I (3647) chip[DL]: Done driving station state, nothing else to do...
I (3657) app-devicecallbacks: Current free heap: 109864

I (3657) chip[DL]: Updating advertising data
I (3657) app-devicecallbacks: Current free heap: 109864

I (3667) wifi:AP's beacon interval = 102400 us, DTIM period = 1
I (4767) esp_netif_handlers: sta ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (4767) chip[DL]: IP_EVENT_GOT_IP6
I (4767) chip[DL]: IPv6 addr available. Ready on WIFI_STA_DEF interface: fe80:0000:0000:0000:7e9e:bdff:fecd:ab28
I (4777) chip[DL]: IPv4 Internet connectivity ESTABLISHED
I (4797) app-devicecallbacks: Current free heap: 109456

I (4797) chip[DL]: IP_EVENT_STA_GOT_IP
I (4807) chip[DL]: IPv4 address changed on WiFi station interface: <IP_ADDRESS>/<IP_ADDRESS> gateway <IP_ADDRESS>
I (4807) app-devicecallbacks: Current free heap: 109816

I (4817) app-devicecallbacks: Server ready at: <IP_ADDRESS>:11097
I (4817) chip[DIS]: Start dns-sd server
I (4837) chip[DIS]: CHIP minimal mDNS started advertising.
I (4837) chip[DIS]: Replying to DNS-SD service listing request
I (4847) chip[DIS]: Replying to DNS-SD service listing request
I (4847) chip[DIS]: Found admin pairing for admin 0, node 0x000000000000022E
V (4857) chip[DIS]: Using wifi MAC for hostname
I (4857) chip[DIS]: Advertise operational node<PHONE_NUMBER>000000-000000000000022E
I (4877) chip[DIS]: CHIP minimal mDNS configured as 'Operational device'.
I (4877) app-devicecallbacks: Current free heap: 109276

I (4887) chip[DIS]: Start dns-sd server
I (4887) chip[DIS]: CHIP minimal mDNS started advertising.
I (4887) chip[DIS]: Replying to DNS-SD service listing request
I (4907) chip[DIS]: Broadcasting mDns reply
I (4907) chip[DIS]: Replying to DNS-SD service listing request
I (4907) chip[DIS]: Broadcasting mDns reply
I (4917) chip[DIS]: Found admin pairing for admin 0, node 0x000000000000022E
V (4927) chip[DIS]: Using wifi MAC for hostname
I (4927) chip[DIS]: Advertise operational node<PHONE_NUMBER>000000-000000000000022E
I (4937) chip[DIS]: CHIP minimal mDNS configured as 'Operational device'.
I (4947) app-devicecallbacks: Current free heap: 108880

I (4947) chip[DIS]: Start dns-sd server
I (4947) chip[DIS]: CHIP minimal mDNS started advertising.
I (4967) chip[DIS]: Replying to DNS-SD service listing request
I (4967) chip[DIS]: Broadcasting mDns reply
I (4977) chip[DIS]: Replying to DNS-SD service listing request
I (4977) chip[DIS]: Broadcasting mDns reply
I (4987) chip[DIS]: Found admin pairing for admin 0, node 0x000000000000022E
V (4997) chip[DIS]: Using wifi MAC for hostname
I (4997) chip[DIS]: Advertise operational node<PHONE_NUMBER>000000-000000000000022E
I (5007) chip[DIS]: CHIP minimal mDNS configured as 'Operational device'.
I (5007) app-devicecallbacks: Current free heap: 106672

I (9197) chip[DIS]: Broadcasting mDns reply
I (12207) chip[DIS]: Broadcasting mDns reply
I (16157) chip[DIS]: Broadcasting mDns reply
I (24257) chip[DIS]: Broadcasting mDns reply
I (31217) chip[DIS]: Replying to DNS-SD service listing request
I (31227) chip[DIS]: Broadcasting mDns reply
I (37977) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (37977) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (39007) chip[DIS]: Broadcasting mDns reply
I (39547) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (39547) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (39837) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (39837) chip[DIS]: Directly sending mDns reply to peer on port 5353
I (40297) chip[DIS]: Broadcasting mDns reply
I (40537) chip[DIS]: Broadcasting mDns reply
I (40547) chip[DIS]: Broadcasting mDns reply
I (72197) chip[DIS]: Broadcasting mDns `reply`
I (112317) chip[DIS]: Replying to DNS-SD service listing request
I (112317) chip[DIS]: Broadcasting mDns reply
I (136167) chip[DIS]: Broadcasting mDns reply`

3.Linux
zx@ubuntu:~/connectedhomeip/examples/all-clusters-app/esp32$ avahi-browse -rt _chip._tcp
+  ens33 IPv4<PHONE_NUMBER>000000-000000000000022E             _chip._tcp           local
=  ens33 IPv4<PHONE_NUMBER>000000-000000000000022E             _chip._tcp           local
   hostname = [7C9EBDCDAB28.local]
   address = [<IP_ADDRESS>]
   port = [11097]
   txt = ["="]
zx@ubuntu:~/connectedhomeip/examples/all-clusters-app/esp32$ avahi-browse -rt _chip._udp
zx@ubuntu:~/connectedhomeip/examples/all-clusters-app/esp32$ ifconfig
ens33: flags=4163<UP,BROADCAST,RUNNING,MULTICAST>  mtu 1500
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>  broadcast <IP_ADDRESS>
        inet6 fe80::1037:7f7f:c16:4b9a  prefixlen 64  scopeid 0x20<link>
        ether 00:0c:29:2d:d4:72  txqueuelen 1000  (以太网)
        RX packets 1383052  bytes<PHONE_NUMBER> (1.7 GB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 512057  bytes 35193289 (35.1 MB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

lo: flags=73<UP,LOOPBACK,RUNNING>  mtu 65536
        inet <IP_ADDRESS>  netmask <IP_ADDRESS>
        inet6 ::1  prefixlen 128  scopeid 0x10<host>
        loop  txqueuelen 1000  (本地环回)
        RX packets 7921  bytes 767400 (767.4 KB)
        RX errors 0  dropped 0  overruns 0  frame 0
        TX packets 7921  bytes 767400 (767.4 KB)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

4.MAC:
CHIP: [DIS] Node ID resolved for 0x00000000000022E
Node address has been updated
current address: fe80::7e9e:bdff:fecd:ab28:11097`

5.photo
https://static.dingtalk.com/media/lADPBFf_-nawCj_NA0_NBJ4_1182_847.jpg_720x720q90g.jpg?bizType=im
https://static.dingtalk.com/media/lADPBE1XfohY2xvNA43NBJM_1171_909.jpg_720x720q90g.jpg?bizType=im
https://static.dingtalk.com/media/lADPBFuNeRtz6hrNAvXNA4g_904_757.jpg_720x720q90g.jpg?bizType=im

I am not familiar with the ESP platform. But I suggest using the same commit hash for both the app and the controller

我对ESP平台不熟悉。但我建议对应用程序和控制器使用相同的提交哈希
Mac and ESP32 are the same versions。
Have you encountered this problem before?  I don't know how to solve it, or anyone knows about ESP32.

@bzbarsky-apple

@736676938 It might be helpful if you could post the terminal log of chip-device-ctrl where you invoke the resolve 0 558

Yes, it's not clear to me what exactly the problem here is.  The "MAC" part of the log shows a successful address resolution...

We're in progress of reviewing older issues. Is this still an issue given how old the SDK this was tested against? Please re-open if so. Thanks!


Error on note import: 'charmap' codec can't decode byte 0x81 in position 5455: character maps to

Hi,
i want to import md file with notes to google keep or anywhere I can open them.
When I trying to do this i got this error: Error on note import: 'charmap' codec can't decode byte 0x81 in position 5455: character maps to
You knows how to solve this?
i exported my notes from xiaomi cloud using https://github.com/nogiszd/xiaomi-note-exporter

Yes! This has been fixed in the next version 0.5.1 in the devel branch. Choose that branch here and just download the single file kim.py to overwrite the current kim file in your version. @NumberOne925

wow, it's working, thank you very much!!
I have another question, it is possible to separate notes by separation mark? e.g. ****

Glad it's working!! I'm not sure what you mean by the separation of notes. Can you explain more?

I have several notes in one note, it looks like:

some text

some text

some text

some text

It would be difficult to add since determining where to place the dividers would be hard to calculate. Your notes might be highly structured making it easier whereas others may not be.


Adding telemetry publish for unhandled exceptions to developer exception page

For #180 cc @Eilon @Tratcher

Hi @JunTaoLuo, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@avanderhoorn @rynowak

Include the bug number in the commit message.

@Tratcher I will add the bug number once I squash all commits

When is this event triggered as apposed to "Microsoft.AspNet.Hosting.UnhandledException" here.

@avanderhoorn When the developer exception page successfully displays. Since the exception will be suppressed, we want to publish it to telemetry source as diagnostic unhandled exception. If for other reasons displaying the exception fails, the exception will be bubbled up to hosting and will trigger the telemetry publish there.

Great, so in the case that this event happens, we wont get the hosting one.

@avanderhoorn right.

Makes sense. Thanks! :shipit:

We also need this here: https://github.com/aspnet/Diagnostics/blob/dev/src/Microsoft.AspNet.Diagnostics/ExceptionHandler/ExceptionHandlerMiddleware.cs#L66

@Tratcher I was asking for that in my previous comment in the issue #180 . That error is technically handled and I wasn't sure if that should also publish to telemetry.

For our part we would like to know about that exception too.

@avanderhoorn cool. I added the telemetry publish for exception handler as well. It's going to publish to Microsoft.AspNet.Diagnostics.HandledException just to differentiate from the unhandled exception from developer exception page, is that good?

Sounds great!
On Friday, October 2, 2015, John Luo<EMAIL_ADDRESS>wrote:

@avanderhoorn https://github.com/avanderhoorn cool. I added the
telemetry publish for exception handler as well. It's going to publish to
Microsoft.AspNet.Diagnostics.HandledException just to differentiate from
the unhandled exception from developer exception page, is that good?
—
Reply to this email directly or view it on GitHub
https://github.com/aspnet/Diagnostics/pull/185#issuecomment-145190816.

I'm going to take that as a :shipit: All tests passed so I'll merge to dev.

Thanks!


webhooks: Add ReviewBoard integration.

@timabbott: Could you please review this one? :) Thanks!

Just updated this! :)

Looks great, merged, thanks @eeshangarg!


Update help for task command group

Waiting on @SteveLasker input.

Closing this since we have it itemized in https://github.com/AzureCR/azure-cli/issues/202


Added the new blog on how to contribute to CNCF project

Summary
Added the new blog on how to contribute to CNCF project for those who is new to world of opensource.
Basic example
Include a basic example, screenshots, or links.
Motivation
Why are we doing this? What use cases does it support? What is the expected outcome?
Checks

[x] Read Create a Pull Request
[x] Supports all screen sizes (if relevant)
[x] Supports both light and dark mode (if relevant)
[x] Passes npm run test

looks fine in my machine

@satrox28 Please have a look.

@satrox28 Thank you for your review. I thinks now it's looks fine .

LGTM


add dynamic wrapper for nexus-repository-manager plugin

Description
Adds dynamic wrapper for nexus-repository-manager plugin after its migration to the community-plugins repo.
Which issue(s) does this PR fix

Fixes RHIDP-3803

Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

/retest

/lgtm

/lgtm

The e2e-tests seem to be failing 🤔


Feature Request: Create Gemfile and Docker File

Good morning ! I was trying to use this tool for the RedTeamCapstoneChallenge and found that installing the right Ruby and Gem dependencies to be quiet a hassle. I've started writing a Gemfile to track and maintain a consistency of the Ruby version used by the script and Gem dependencies. Could you provide me with the version of Ruby and Gems used in your original development environment? Try: ruby -v or gem -v httparty.
Additionally I've started creating a Dockerfile to package the app and make it easier to run without having to install Ruby and a bunch of other gems.

Excuse me, I had no time to maintain this project. Today I have reworked the whole tool so a lot of things have changed. Anyway here are the versions you asked for:


Can't import TimeMixer model from neuralforecast

What happened + What you expected to happen
after installing neuralforecast I run this code from neuralforecast.models import TimeMixer to import the model but I keep getting  importerror :   ImportError: cannot import name timemixer from neuralforecast.models (/usr/local/lib/python3.10/dist-packages/neuralforecast/models/init.py)
Versions / Dependencies
kaggle notebook and google colab
Reproduction script
from neuralforecast.models import TimeMixer
Issue Severity
High: It blocks me from completing my task.

Hi,
Our docs have already been updated but TimeMixer is not available yet; we expect to make a release next week which will make TimeMixer available to use.
Sorry for the inconvenience; I'll drop a note as soon as we make the release. To use TimeMixer already, you can git clone the repo and install from source.

Le sam. 27 juil. 2024 à 20:53, Olivier Sprangers @.***>
a écrit :

Hi,
Our docs have already been updated but TimeMixer is not available yet; we
expect to make a release next week which will make TimeMixer available to
use.
Sorry for the inconvenience; I'll drop a note as soon as we make the
release. To use TimeMixer already, you can git clone the repo and install
from source.
—
Reply to this email directly, view it on GitHub
https://github.com/Nixtla/neuralforecast/issues/1087#issuecomment-2254239035,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BC6CSOVCV55GZOLZWHH3P23ZOP3CTAVCNFSM6AAAAABLSDH2NGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGIZTSMBTGU
.
You are receiving this because you authored the thread.Message ID:
@.***>

@redaelkate We released 1.7.4 today, which contains TimeMixer. Hence, if you update neuralforecast (pip install neuralforecast --upgrade), you should be able to use TimeMixer.
Let me know if this works for you.

@elephaint I'm confused about the TimeMixer model, are you not able to use future exogenous variables with it?

We encountered a bug when trying to support future exogenous variables. For now, TimeMixer does not support them, but we are working on including them.
Still, the model is multivariate, so it will considered any inter-dependence between multiple series, and also note that future exogenous variables were not used in the original paper.


API diff between .NET 9 Preview 4 and .NET 9 Preview 5

Repo area owners:

ASP.NET - @dotnet/aspnet-api-review
WinForms - @merriemcgaw @dreddy-work @JeremyKuhne
WPF - @singhashish-wpf and @pchaurasia14
Libraries - @artl93 @jeffschwMSFT @jeffhandley @karelz @ericstj @stephentoub

Libraries area owners:

System @dotnet/area-system-runtime
@dotnet/area-system-collections
@dotnet/area-system-componentmodel
@dotnet/area-system-diagnostics
@dotnet/area-system-diagnostics-metric
System.IO.Pipelines @adityamandaleeka @davidfowl @halter73
@dotnet/area-system-numerics
@dotnet/area-system-reflection
@dotnet/area-system-reflection-metadata
@dotnet/area-system-runtime-compilerservices
System.Runtime.InteropServices @dotnet/interop-contrib
@dotnet/area-system-runtime-intrinsics
@dotnet/area-system-text-json

Known api-diff tool issues:
If yo usee any of these, please provide a GitHub suggestion in this PR to correct it:

The AsmDiff tool sometimes skips adding { or {} arcade#10981(https://github.com/dotnet/arcade/issues/13814)
RequiresPreviewAttribute not getting auto added arcade#14498
Changes to protected internal visibility should stop showing. https://github.com/dotnet/arcade/issues/14579
The AsmDiff tool shows APIs with decorating attribute changes as removed and re-added, instead of showing only the API. Please ignore this, it might be confusing but technically it's not incorrect: [arcade#13814]

System.Diagnostics LGTM.
I left a comment regarding System.Diagnostics.Metrics.

@mairaw is there a known issue with the CI validation in this repo? The "Checking for ability to merge automatically" message is stuck.


Non-MAM links don't work

Issue is because it was assumed all would be when I first created this.

Fixed in 0.5.8, but also added in handler to use URL redirect on external links.  My understanding is that the reason for URL redirect is to hide referrer from linked site and we don't want to risk exposing MAM on certain sites.


Hydrotel hooks

Pull Request Checklist:

[x] This PR addresses an already opened issue (for bug fixes / features)

No.

[x] (If applicable) Documentation has been added / updated (for bug fixes / features).
[x] (If applicable) Tests have been added.
[x] HISTORY.rst has been updated (with summary of main changes).

[x] Link to issue (:issue:number) and pull request (:pull:number) has been added.

What kind of change does this PR introduce?

Code to prepare, then run a Hydrotel project.
Partial standardisation of the outputs. The 'Atlas-style' formatting is performed elsewhere.

Does this PR introduce a breaking change?

No

Other information:

@Zeitsperre No need for you to review everything, but I'd like you to take a look at the testing setup (with tmpdir) to make sure that it's done correctly. I also create and subsequently delete a folder during the new notebook. For now, I don't think that we need something as fancy as what we do in xscen.
@EdouardMailhot Si tu veux, on peut organiser une réunion pour tranquillement passer à travers les différents processus Github, incluant la révision de PRs. Pas besoin d'approuver cette PR à court terme.

@EdouardMailhot @JPelletierDumont Ça devrait être prêt pour un review !
@richardarsenault Idem pour toi. Tu iras voir la documentation; j'ai implémenté un exemple complet pour la modélisation hydrologique, incluant la calibration.


GCS bucket not being created

terragrunt version v0.18.7 and terraform version v0.11.14 (can't upgrade to 0.12 until the Vault provider supports it)
config files:
at env/terraform.tfvars
terragrunt = {
  remote_state {
    backend = "gcs"
    config {
      project = "<redacted>"
      bucket = "<redacted>"
      prefix = "${path_relative_to_include()}"
    }
  }
}

at env/region/terraform.tfvars
terragrunt {
  terraform {
    source = "${get_tfvars_dir()}/../../../../templates"
  }
  # include gcs bucket state storage configuration
  include {
    path = "../terraform.tfvars"
  }
}

I'm getting the following unexpected error:
Initializing the backend...

Successfully configured the backend "gcs"! Terraform will automatically
use this backend unless the backend configuration changes.
Failed to get existing workspaces: querying Cloud Storage failed: storage: bucket doesn't exist

This error is unexpected as terragrunt documentation indicates that the gcs bucket should get automatically created on the first run.
Any ideas? Bug? Misconfiguration?

Hi, the GCS bucket auto creation was introduced in v0.19.6. If you look at the README for the version you are using, the GCS bucket creation isn't mentioned there.

Ah, makes sense, thanks for the quick response. I'll add a quick .tf file & use the terraform cli to handle bucket creation beforehand then.


Bosh uses vcap user for everything

Bosh and bosh releases by default have all services running as user 'vcap'.  This is in direct opposition to Linux best practices, in which there would be a specific user for each job / service provided.  For example, instead of the current mysql exploit applying to the entire vcap tree, this should be restricted to only the mysql-job user and group.

[ ] Bosh jobs should not have read / write / execute to other jobs by default
[ ] Bosh jobs should not have full vcap.vcap access by default
[ ] The bosh jobs should include 'job.user' property and the bosh-agent should invoke that job by de-escalating privileges from root to 'job.user'

In it's current state it means that any access to vcap is granting all access to vcap, which is every service bosh deploys under /var/vcap/jobs, /var/vcap/packages and /var/vcap/data along with hard coded device ownerships such as /dev/log /dev/shm which grant full system access regardless of intended permissions.  This means that deploying MySQL by bosh currently gives full root access to the system functionally.

@sharms Yup. That's exactly the plan to have bosh-agent provision individual users for each job installed on the system. We are planning to do that as we fulfilling each one of the CIS/STIG benchmarks.

What we also intend to do is to run the different processes with specific capabilities provided by linux at process and file levels, so as to restrict the potential of having more privileges then needed

What's the timeline for this feature? For me, this sounds quite important.

Some of this is mitigated by using https://github.com/cloudfoundry/bpm-release which most releases should use.


Design walk cache

I have the beginnings of a walk cache design from some early thinking when I was planning the zar layout changes. It's more complicated than the small file caching, since files/directories can be added and removed.  I'll collect my thoughts & put it up for review.

No longer relevant as lake model supersedes this...


Refactor the project and add generics and getter options

I used basic eslint and prettier installs without any configs to format the files.
There were also function refactors for better maintainability and JS standards as well as variable renames.
I also added generics to collections as well as a optional getter options object which has a reconstructor and default value property.

Thank you @petarvujovic98 !

I used basic eslint and prettier installs without any configs to format the files.
There were also function refactors for better maintainability and JS standards as well as variable renames.

These parts are great! Function refactor and variable renames also look good.

I also added generics to collections as well as a optional getter options object which has a reconstructor and default value property.

This seems already in #203 , would you mind wait that PR merge and doing a rebase? I believe except #203, there's no other PR that should merge before this one to avoid conflictions.

Also, just a small suggestion: it'll be great to separate the refactor commit with the reformatting next time, to make code review easier

@ailisp Of course, I'll wait for the generics merge and rebase this PR.

@volovyks CI was not triggering for this PR, do you know why

@ailisp that is because @petarvujovic98 is not a part of our GitHub org. I have started a discussion about it in Slack.

@ailisp I just saw this comment now, I could separate the reformat and refactor after I rebase the code. This was just me going fast

@petarvujovic98 Sorry looks like a lot of conflictions are in! It may be faster to have your function refactor changes first. Then we can re-apply your formatting to new develop branch in another PR.

@ailisp Sounds good. Will do the refactor on Monday

@ailisp @volovyks I just did a refactor of this PR. Please review and add any comments on the changes.

@petarvujovic98 usually when linting is available - we add it a separate job in GitHub actions. Do you think we should do that?

@volovyks I think we should. I would first like to properly configure ESLint for this project as it seems that the JavaScript files in the tests and examples are getting false errors about not having experimentalDecorators turned on, when they definitely do.

@volovyks Let me add the CI for linting real quick and let's merge it.

@volovyks @ailisp Should be ready to merge now.

@volovyks Do we want to add formatting checks into the Lint workflow as well?

@volovyks Do we want to add formatting checks into the Lint workflow as well?

@petarvujovic98 We do not have it in other projects, but let's try it. We can always turn it off.

@volovyks Done!

Merging, great work! @petarvujovic98

Great job! Thanks @petarvujovic98 !


Hello World

Method
In this section, we describe our unsupervised framework for monocular depth estimation. We first review the self-supervised training pipeline for monocular depth estimation, and then introduce the co-attention module and pose graph consistency loss function.
Supervision from Image Reconstruction
Following the formulation in \cite{zhou_unsupervised_2017}, the whole framework includes a DispNet and a PoseNet, the DispNet produces depth map and the PoseNet produces the relative pose between two RGB frames.
Given a sequence of consecutive frames $X_{t-1}, X_t$ and $X_{t+1}$，we estimate the depth for each frame, and the relative pose for every two adjacent frames, then we get depth map $D_{t-1}, D_t, D_{t+1}$ and translation matrix $T_{t-1\rightarrow t}, T_{t\rightarrow t+1}$.
Consider the adjacent frame pair $I_t$ and $I_{t+1}$, once the estimated depth $D_t$ and translation matrix $T_{t\rightarrow t+1}$ are available, we can project the source image $I_t$ to the next moment
$$
p(\hat{I}{t+1}) = KT{t\rightarrow t+1}D_tK^{-1}p(I_t)
$$
the function $p(.)$ denotes sampling from the homogeneous coordinates of image and $K$ denotes the camera insrinsic matrix, $\hat{I}_{t+1}$ can be reconstucted using the differentiable sampling mechanism proposed in \cite{jaderberg_spatial_2015}.
Hence the problem is formulated to the minimization of a phtometric reprojection error $L_p$
$$
L_p = \alpha \left|I_{t+1} - \hat{I}{t+1}\right|1 + (1 - \alpha)SSIM(I{t+1}, \hat{I}{t+1})
$$
$SSIM(.)$ is the structural similarity\cite{wang_image_2004} loss for evaluating the quality of image predictions, and to regularize the depth, we use a disparity image smoothness constraint as widely used in previous work\cite{mahjourian_unsupervised_2018,zhou_unsupervised_2017,garg_unsupervised_2016}
$$
L_{\mathrm{s}}=\sum_{x, y}\left|\partial_{x} D_{t}\right| e^{-\left|\partial_{x} I_{t}\right|}+\left|\partial_{y} D_{t}\right| e^{-\left|\partial_{y} I_{t}\right|}
$$
List
Here is a list:

Xue Bai, Jue Wang, David Simons, and Guillermo Sapiro.Video SnapCut: robust video object cutout using localized classifiers. TOG, 28(3):70, 2009.
Linchao Bao, Baoyuan Wu, and Wei Liu. CNN in MRF: Video object segmentation via inference in a CNN-based higher-order spatio-temporal MRF.


Zero copy

This PR adds options to accept hardware memory frame support for directx rendering.

Moved over to #112 because of signing issue in this one.


[Feature Request]: config to disable rainbow armor when at 30 points

Is there an existing issue for this?

[X] I have searched the existing issues

Description
i might have missed this but i didnt see any mention of this feature. i had to go through a lot of mods till i narrowed it down to this one, and it should probably be mentioned in the mod description. can we get a config to disable this feature as some people might not want it.
Solution
add a config to remove the rainbow armor effect when at 30 armor points
Alternatives
No response
Additional Context
No response

Yea I could do that