danger-swift local fails

Hi !
I'm trying to setup danger-swift to run with my project and I'm trying to run it locally. I cannot figure out what is wrong with my setup but the result is that danger-swift local fails when I try to run it. Here are my implementation's details :
Package.swift
// swift-tools-version:4.2
import PackageDescription

let package = Package(
    name: "Eigen",
    dependencies: [
      .package(url: "https://github.com/danger/swift.git", from: "1.0.0")
    ],
    targets: [
        // This is just an arbitrary Swift file in our app, that has
        // no dependencies outside of Foundation, the dependencies section
        // ensures that the library for Danger gets build also.
        .target(name: "eigen", dependencies: ["Danger"], path: "ProApp", sources: ["../DangerDummy.swift"]),
    ]
)

Same as the one in getting started. DangerDummy.swift contains one line of code that imports Foundation
Dangerfile.swift
import Danger
let danger = Danger()

let editedFiles = danger.git.modifiedFiles + danger.git.createdFiles
message("These files have changed: \(editedFiles.joined())")

Once again, it's the same as in the getting started
danger-swift local logs
// DEBUG=* danger-swift local
Launching Danger Swift local (v1.6.5)
Finding out where the danger executable is
Running: /usr/local/bin/danger local --process .build/debug/danger-swift --passURLForDSL
  danger:localGetDiff > git diff master...HEAD +0ms
  danger:process_runner Debug mode on for Danger v6.1.13 +0ms
  danger:process_runner Starting sub-process run +0ms
  danger:localGetDiff > git log master...HEAD --pretty=format:{ "sha": "%H", "parents": "%p", "author": {"name": "%an", "email": "%ae", "date": "%ai" }, "committer": {"name": "%cn", "email": "%ce", "date": "%ci" }, "message": "%f"}, +0ms
  danger:process_runner Preparing to run: .build/debug/danger-swift +112ms
  danger:runDangerSubprocess Running sub-process: danger-swift -  +0ms
  danger:runDangerSubprocess Started passing in STDIN via the URL: danger://dsl//var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json +4ms
  danger:runDangerSubprocess Passed DSL in via STDIN +0ms
Ran with: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json
Decoding the DSL into Swift types
ERROR: Failed to parse JSON: valueNotFound(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "git", intValue: nil), CodingKeys(stringValue: "created_files", intValue: nil), _JSONKey(stringValue: "Index 45", intValue: 45)], debugDescription: "Expected String but found null value instead.", underlyingError: nil))

  danger:runDangerSubprocess Got JSON URL from STDOUT, results are at:
  danger:runDangerSubprocess danger-results://var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json +385ms
Got URL for JSON: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json
Created a temporary file for the Dangerfile DSL at: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json
Running Dangerfile at: Dangerfile.swift
Preparing to compile
Running: /usr/bin/swiftc --driver-mode=swift -L /Users/admin/Desktop/REDACTED/proapp/.build/debug -I /Users/admin/Desktop/REDACTED/proapp/.build/debug -lDanger /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json
Completed evaluation
ERROR: Dangerfile eval failed at Dangerfile.swift
Saving and storing the results at /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json

  danger:runDangerSubprocess child process exited with code 1 +5ms
  danger:runDangerSubprocess Handling fail from subprocess +0ms
  danger:jsonToDSL Creating pr DSL from JSON +0ms
[before-after-hook]: "Hook()" repurposing warning, use "Hook.Collection()". Read more: https://git.io/upgrade-before-after-hook-to-1.4
  danger:localGetDiff > git diff master...HEAD +1s
  danger:localGetDiff > git log master...HEAD --pretty=format:{ "sha": "%H", "parents": "%p", "author": {"name": "%an", "email": "%ae", "date": "%ai" }, "committer": {"name": "%cn", "email": "%ce", "date": "%ci" }, "message": "%f"}, +1s
  danger:executor Got results back: {
  warnings: [],
  messages: [
    {
      message: 'These files have changed: CHANGELOG.mdSources/DangerFixtures/DangerFixtures.swiftTests/DangerTests/XCTestManifests.swiftSources/Danger/Plugins/SwiftLint/CurrentPathProvider.swiftSources/Danger/Plugins/SwiftLint/ShellExecutor.swiftSources/Danger/Plugins/SwiftLint/SwiftLint.swiftSources/Danger/Plugins/SwiftLint/SwiftLintViolation.swiftTests/DangerTests/SwiftLint/DangerSwiftLintTests.swiftTests/DangerTests/SwiftLint/FakeCurrentPathProvider.swiftTests/DangerTests/SwiftLint/FakeShellExecutor.swiftTests/DangerTests/SwiftLint/URL+Utils.swiftTests/DangerTests/SwiftLint/ViolationTests.swift'
    }
  ],
  fails: [ { message: 'danger-swift` failed.' } ],
  markdowns: [
    {
      message: '### Log\n' +
        '\n' +
        '\n' +
        '```sh\n' +
        'Ran with: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0mDecoding the DSL into Swift types\n' +
        '\u001b[0;0m\u001b[31mERROR: Failed to parse JSON: valueNotFound(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "git", intValue: nil), CodingKeys(stringValue: "created_files", intValue: nil), _JSONKey(stringValue: "Index 45", intValue: 45)], debugDescription: "Expected String but found null value instead.", underlyingError: nil))\n' +
        '\u001b[0;0mdanger-results://var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\n' +
        'Got URL for JSON: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json\n' +
        '\u001b[0;0mCreated a temporary file for the Dangerfile DSL at: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json\n' +
        '\u001b[0;0mRunning Dangerfile at: Dangerfile.swift\n' +
        '\u001b[0;0mPreparing to compile\n' +
        '\u001b[0;0mRunning: /usr/bin/swiftc --driver-mode=swift -L /Users/admin/Desktop/REDACTED/proapp/.build/debug -I /Users/admin/Desktop/REDACTED/proapp/.build/debug -lDanger /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0mCompleted evaluation\n' +
        '\u001b[0;0m\u001b[31mERROR: Dangerfile eval failed at Dangerfile.swift\n' +
        '\u001b[0;0mSaving and storing the results at /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0m\n' +
        '```\n'
    }
  ]
} +0ms
  danger:executor Evaluator settings {
  stdoutOnly: true,
  verbose: undefined,
  jsonOnly: false,
  dangerID: 'default',
  passURLForDSL: true
} +3ms
  danger:executor Writing to STDOUT: {
  warnings: [],
  messages: [
    {
      message: 'These files have changed: CHANGELOG.mdSources/DangerFixtures/DangerFixtures.swiftTests/DangerTests/XCTestManifests.swiftSources/Danger/Plugins/SwiftLint/CurrentPathProvider.swiftSources/Danger/Plugins/SwiftLint/ShellExecutor.swiftSources/Danger/Plugins/SwiftLint/SwiftLint.swiftSources/Danger/Plugins/SwiftLint/SwiftLintViolation.swiftTests/DangerTests/SwiftLint/DangerSwiftLintTests.swiftTests/DangerTests/SwiftLint/FakeCurrentPathProvider.swiftTests/DangerTests/SwiftLint/FakeShellExecutor.swiftTests/DangerTests/SwiftLint/URL+Utils.swiftTests/DangerTests/SwiftLint/ViolationTests.swift'
    }
  ],
  fails: [ { message: 'danger-swift` failed.' } ],
  markdowns: [
    {
      message: '### Log\n' +
        '\n' +
        '\n' +
        '```sh\n' +
        'Ran with: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0mDecoding the DSL into Swift types\n' +
        '\u001b[0;0m\u001b[31mERROR: Failed to parse JSON: valueNotFound(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "git", intValue: nil), CodingKeys(stringValue: "created_files", intValue: nil), _JSONKey(stringValue: "Index 45", intValue: 45)], debugDescription: "Expected String but found null value instead.", underlyingError: nil))\n' +
        '\u001b[0;0mdanger-results://var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\n' +
        'Got URL for JSON: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json\n' +
        '\u001b[0;0mCreated a temporary file for the Dangerfile DSL at: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json\n' +
        '\u001b[0;0mRunning Dangerfile at: Dangerfile.swift\n' +
        '\u001b[0;0mPreparing to compile\n' +
        '\u001b[0;0mRunning: /usr/bin/swiftc --driver-mode=swift -L /Users/admin/Desktop/REDACTED/proapp/.build/debug -I /Users/admin/Desktop/REDACTED/proapp/.build/debug -lDanger /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0mCompleted evaluation\n' +
        '\u001b[0;0m\u001b[31mERROR: Dangerfile eval failed at Dangerfile.swift\n' +
        '\u001b[0;0mSaving and storing the results at /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json\n' +
        '\u001b[0;0m\n' +
        '```\n'
    }
  ]
} +0ms
Danger: ⅹ Failing the build, there is 1 fail.
## Failures
danger-swift` failed.
## Messages
These files have changed: CHANGELOG.mdSources/DangerFixtures/DangerFixtures.swiftTests/DangerTests/XCTestManifests.swiftSources/Danger/Plugins/SwiftLint/CurrentPathProvider.swiftSources/Danger/Plugins/SwiftLint/ShellExecutor.swiftSources/Danger/Plugins/SwiftLint/SwiftLint.swiftSources/Danger/Plugins/SwiftLint/SwiftLintViolation.swiftTests/DangerTests/SwiftLint/DangerSwiftLintTests.swiftTests/DangerTests/SwiftLint/FakeCurrentPathProvider.swiftTests/DangerTests/SwiftLint/FakeShellExecutor.swiftTests/DangerTests/SwiftLint/URL+Utils.swiftTests/DangerTests/SwiftLint/ViolationTests.swift
## Markdowns
### Log

```sh
Ran with: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json
Decoding the DSL into Swift types
ERROR: Failed to parse JSON: valueNotFound(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "git", intValue: nil), CodingKeys(stringValue: "created_files", intValue: nil), _JSONKey(stringValue: "Index 45", intValue: 45)], debugDescription: "Expected String but found null value instead.", underlyingError: nil))
danger-results://var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json

Got URL for JSON: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json
Created a temporary file for the Dangerfile DSL at: /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json
Running Dangerfile at: Dangerfile.swift
Preparing to compile
Running: /usr/bin/swiftc --driver-mode=swift -L /Users/admin/Desktop/REDACTED/proapp/.build/debug -I /Users/admin/Desktop/REDACTED/proapp/.build/debug -lDanger /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/_tmp_dangerfile.swift /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-dsl.json /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json
Completed evaluation
ERROR: Dangerfile eval failed at Dangerfile.swift
Saving and storing the results at /var/folders/6c/_dj0nh5s7g1b0w29rkn6j34r0000gn/T/danger-response.json

The message I get from Danger is consistent with what's in my Dangerfile but it doesn't looks like it runs on the right project.
I tried to wrap my head around the meaning of ERROR: Failed to parse JSON: valueNotFound(Swift.String, Swift.DecodingError.Context(codingPath: [CodingKeys(stringValue: "danger", intValue: nil), CodingKeys(stringValue: "git", intValue: nil), CodingKeys(stringValue: "created_files", intValue: nil), _JSONKey(stringValue: "Index 45", intValue: 45)], debugDescription: "Expected String but found null value instead.", underlyingError: nil)) from other similar issues but in vain too
Looking forward to your answer and thank you for your time !

This looks a danger-js issue, in the array of the files the item at index 45 is nill, I don't know why, it probably is related to your diff from master (danger-swift local uses your diff from your branch to master)

Hi, thanks for your answer !
At first I didn't have the master branch locally so I took care of that. I will try and merge where I can to reduce the diff output it's been a long time coming anyway :) Maybe this will work out.
Is there a way that I could set the branch against which danger has to work ?

No, but I think it could be a good addition to https://github.com/danger/danger-js/blob/master/source/commands/danger-local.ts

You can actually use --base to set the branch, I think

Ok thanks ! Will try that


fix: Python generation for type casts

Summary of Changes
Previously, the Python generator threw an error, since type casts were not handled. Now, they are handled properly.

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
v0.10.0
GitHub release

Your semantic-release bot :package::rocket:


I want to use a NON-default browser for live preview button...

Currently Live Preview opens code in the default browser, for me this is Safari.
I would like the option to define which browser opens (namely Chrome).
WHY?  Because Chrome is a much better debugger than Safari, but Safari is more energy efficient.  Whenever I'm coding I'm usually plugged in to power so it doesn't matter, when I'm out and about I'm usually not coding, but working on other aspects of my business, therefore the default of Safari is what I need.

use file> enable experimental live preview. then click on live preview button.
After that you can copy the URL to any browsers you want to live preview in.
closing as feature already suported.

I am well aware I can copy that URL from one browser to another.  What I'm asking for is a way to only open the browser I want to use.

I have the same issue. I'm on a MAC, Safari is my default browser and I want to have live preview on Chrome directly from the live preview button on Brackets. I don't want to copy and paste the URL from Safari to Chrome.
Also, weither I'm on Safari or Chrome, I don't see the changes unless I reload the browser which is the main reason I would use Brackets at this point.
Please advise, thank you much!
Best,
Michel

I am also looking for the configuration how to open a page-preview in a non-default browser. My default is Firefox Dev but I want my previews to open in Brave without copy-and-paste.

I am also looking for the configuration how to open a page-preview in a non-default browser. My default is Firefox Dev but I want my previews to open in Brave without copy-and-paste.


Bump minimum supported rust version from 1.31 to 1.32

Signed-off-by: ice1000<EMAIL_ADDRESS>As title. According to https://github.com/pingcap/raft-rs/pull/250#issuecomment-507464872

error[E0658]: `cfg(target_vendor)` is experimental and subject to change (see issue #29718)
   --> C:\Users\travis\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.32\src\lib.rs:147:45
    |
147 | #[cfg(all(windows, feature = "dbghelp", not(target_vendor = "uwp")))]
    |                                             ^^^^^^^^^^^^^^^^^^^^^
error[E0658]: `cfg(target_vendor)` is experimental and subject to change (see issue #29718)
   --> C:\Users\travis\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.32\src\backtrace\mod.rs:140:59
    |
140 |     } else if #[cfg(all(windows, feature = "dbghelp", not(target_vendor = "uwp")))] {
    |                                                           ^^^^^^^^^^^^^^^^^^^^^
error[E0658]: `cfg(target_vendor)` is experimental and subject to change (see issue #29718)
   --> C:\Users\travis\.cargo\registry\src\github.com-1ecc6299db9ec823\backtrace-0.3.32\src\symbolize\mod.rs:462:52
    |
462 |                         any(unix, all(windows, not(target_vendor = "uwp"), target_env = "gnu")),
    |                                                    ^^^^^^^^^^^^^^^^^^^^^
error: aborting due to 3 previous errors

Seems that we have to bump to 1.33?

PTAL @nrc @Hoverbear


Update server.rs to fix recursive inscriptions on iOS/Safari

Recursive inscriptions on ordinals.com are currently broken on iOS and Safari. Moving 'self' further down fixes this so recursive inscriptions can render on iOS and Safari on Ordinals.com.

I'm unsure on this, but the second set of CSPs might need to be adjusted as well (moving each whitelisted endpoint after the unsafe eval and unsafe inline params)

Unfortunately I'm unable to make this work on Safari. How did you test this?

I think it is not working because you still use two CSP headers (insert and then append, which doesn't work for Safari). I've made a fix for this in this pull request: https://github.com/ordinals/ord/pull/2251

Yeah, I can confirm that #2251 fixes it. Great work @devlz303

Ok, I will close this in favor of #2251. Will write there.


Add Scala Native

Better sooner than later 😅

This no longer builds on my machine.  It should be fairly simple if I update the flake.

I had more or less done de38f64 locally, but now the native build crashes on GC overhead.  Do you have some global setting that makes it work?

Thanks!


231 Fix functionality on collab and admin pages

Fixed the functionality on collab and admin pages - ensured that the data is passed seemlessly to both pages.

:)


Unhandled Exception: PlatformException(error, StorageManager failed to initialize, null, java.lang.IllegalStateException: StorageManager failed to initialize

Describe the bug
Unhandled Exception: PlatformException(error, StorageManager failed to initialize, null, java.lang.IllegalStateException: StorageManager failed to initialize
Screenshots

Desktop (please complete the following information):

Android


Configure a Docker Automated build

Hi all.
Let's track the requirements of this feature (now scattered across various PRs and issues).
Background

I opened #591 (Dockerfile and Docker automated builds)
#595 (Move repo to organization) was created but took a bit to complete
In the meantime #591 was closed by #634 (Remove Docker bits) because I (temporarily) moved the GHCG Docker image under the ferrarimarco Docker Hub namespace

Current Implementation
ferrarimarco/github-changelog-generator is backed by the corresponding GitHub repository.
Each commit and Git tag pushed to that repository triggers a build on Docker hub and, if successful, in a new Docker image.
I am currently handling GHCG updates with Dependabot. This service automatically creates PRs to update dependencies by looking at your Gemfile.
This is the reason why I'm using a separate Gemfile even if we could get rid of it and run commands directly in the Dockerfile.
Ideas for a more refined implementation
Instead of just installing the updated Gem when available we could provide two kinds of Docker images:

One tagged with latest: by convention it could point to the latest (successful) build of the master branch. This image is overridden on each commit on master (if the build is ok)
One tag for each GHCG release tagged using the same Git tag (i.e. when you release 1.15.0 there will be a corresponding 1.15.0 docker image). This is needed to make production deployment repeatable

Thoughts?
/cc @olleolleolle @skywinder

Smart use of dependabot!

@skywinder @olleolleolle did you have the chance to think about this?

@olleolleolle @skywinder let's resume this work if you want since there are a couple of issues related to this. :)
What do you think?

Happy to assist as well if needed; one thing I noticed in the old Dockerfile, is that it was fetching the source from inside the Dockerfile; using a COPY instead would make sure that the Dockerfile can be built from any branch, and the code always matching the code in that branch

Sounds like a good use case for a multi-staged build actually. So yes, good idea! I'll open a PR for this, since I don't yet have write access here :(

@olleolleolle @skywinder I created an org in Docker Hub but you should approve the access request so I can eventually set up an automated build

since I don't yet have write access here

It's generally good etiquette to use pull requests, even if you have write access 🤗

@thaJeztah I'm a member of this org :)

(I don't have admin rights on this repository.)
I also can't see a Docker Hub Organization - does it exist somewhere?
Looking around, I was able to find quite a few people who had set up builds: https://hub.docker.com/search?q=github-changelog-generator&type=image

@olleolleolle I created an org here: https://cloud.docker.com/orgs/githubchangeloggenerator
but need approval to create an automated build...

@olleolleolle @skywinder I need admin access to this repo to proceed further on this.

Done in https://github.com/github-changelog-generator/docker-github-changelog-generator


Dutch Essent account with no readings?

Hi team,
Really like your idea for this project! I have a NL Essent account, and I can see my usage online after logging in. Nevertheless, i get a parsing error in home assistant, because and '.find('Installations')' fails. When I try to use this code with spyder I get to the result xml, where the data pull seems to be successful, but it contains no readings.
I get 10 EANs, but each of them seems to have no readings. Below the response, with the EAN hidden. Any suggestion to solve this? I am really looking forwards to get my readings in HA :)
<?xml version="1.0" encoding="utf-8"?> <GetMeterReadingHistory>              <Context>             <ServiceUser>MIJNESSENT</ServiceUser>             <Domain>SelfService</Domain>             <Brand>essent</Brand>         </Context>          <request>         <PartnerID>0150307865</PartnerID>         <ServiceProviders>             <SupplierEAN name="Essent Retail Energie" EAN="xxx"/>         </ServiceProviders>         <Installations>             <Installation>                 <ConnectEAN>xxx</ConnectEAN>             </Installation>         </Installations>         <OnlyLastMeterReading>false</OnlyLastMeterReading>         <Period>             <StartDate>2018-03-26T00:00:00+02:00</StartDate>             <EndDate>2019-05-26T23:59:59+02:00</EndDate>         </Period>     </request>     <response>         <Info/>         <Messages>             <Message>                 <Type>S</Type>                 000                 <Message>Service Call is Successful</Message>             </Message>             <Message>                 <Type>I</Type>                 200                 <Message>No Records Found</Message>             </Message>         </Messages>     </response> </GetMeterReadingHistory>

Oh, interesting. I just grabbed the first EAN when implementing this because that works for me. Does only one EAN return meter info or do multiple do? In the last case, how would I decide which is the correct EAN?

I get 10 EANs, but each of them seems to have no readings.

Oops, misread this. Could you check if the VerbruiksManager is enabled on the Mijn Essent website and if it has anything under "Meterstanden" for you?

Wow, quick reaction!
Yes, i can access that part of the MijnEssent and I see monthly (some months 2 measures) incremental readings for :

Opnamedatum | Stroom dagtelwerk | Stroom nacht/weekend telwerk
Instead, I see missing values for
Stroom teruglevering, dagtelwerk | Stroom teruglevering, nacht/weekend telwerk
"Bron" is "Opgave door slimme meter" for every entry.

Hmm, that sounds about right then. You're sure not a single one of the EANs returns any readings? Because as far as I'm aware at least one of them should return this data.

Yes, I am sure. I printed the content of each EAN directly from within your code. Any idea of what could be different from other cases?

Can you try the code in the README again on the latest master branch?
from pyessent import PyEssent

essent = PyEssent(username=username, password=password)
for ean in essent.get_EANs():
    essent.read_meter(ean)

And see if it outputs anything at all?

Interesting. The code does not beak now, but i also do not get any value out. If I print the response as I did in the previous commit, I now get a simple
<Response [200]>

How odd, I expect output like this:
$ python3
Python 3.7.3 (default, May 11 2019, 00:38:04)
[GCC 9.1.1 20190503 (Red Hat 9.1.1-1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
@>>> from pyessent import PyEssent
@>>> essent = PyEssent(username='username', password='password')
@>>> for ean in essent.get_EANs():
@...     essent.read_meter(ean)
@...
{'type': 'Elektriciteit', 'values': {'TLV': {'N': {'unit': 'KWH', 'records': {'2019-05-01T00:00:00+02:00': '0', '2019-04-01T00:00:00+02:00': '0', '2019-03-01T00:00:00+01:00': '0', '2019-02-01T00:00:00+01:00': '0', '2019-01-01T00:00:00+01:00': '0', '2018-12-31T00:00:00+01:00': '0', '2018-12-12T00:00:00+01:00': '0'}}, 'L': {'unit': 'KWH', 'records': {'2019-05-01T00:00:00+02:00': '0', '2019-04-01T00:00:00+02:00': '0', '2019-03-01T00:00:00+01:00': '0', '2019-02-01T00:00:00+01:00': '0', '2019-01-01T00:00:00+01:00': '0', '2018-12-31T00:00:00+01:00': '0', '2018-12-12T00:00:00+01:00': '0'}}}, 'LVR': {'N': {'unit': 'KWH', 'records': {'2019-05-01T00:00:00+02:00': '100', '2019-04-01T00:00:00+02:00': '100', '2019-03-01T00:00:00+01:00': '100', '2019-02-01T00:00:00+01:00': '100', '2019-01-01T00:00:00+01:00': '100', '2018-12-31T00:00:00+01:00': '100', '2018-12-12T00:00:00+01:00': '100'}}, 'L': {'unit': 'KWH', 'records': {'2019-05-01T00:00:00+02:00': '100', '2019-04-01T00:00:00+02:00': '100', '2019-03-01T00:00:00+01:00': '100', '2019-02-01T00:00:00+01:00': '100', '2019-01-01T00:00:00+01:00': '100', '2018-12-31T00:00:00+01:00': '100', '2018-12-12T00:00:00+01:00': '100'}}}}}
{'type': 'Gas', 'values': {'LVR': {'N': {'unit': 'MTQ', 'records': {'2019-05-01T00:00:00+02:00': '100', '2019-04-01T00:00:00+02:00': '100', '2019-03-19T00:00:00+01:00': '100', '2019-03-01T00:00:00+01:00': '100', '2019-02-01T00:00:00+01:00': '100', '2019-01-01T00:00:00+01:00': '100', '2018-12-31T00:00:00+01:00': '100', '2018-12-12T00:00:00+01:00': '100'}}}}}

Just to make sure, under "Instellingen", both these boxes are checked?

Should be fixed in #6.


Update external snippet

Need to move to the new external attribute for external functions in code snippets

Thank you


devnet build error when  trying to build sunodo from WSL2

🙂 Expected behavior
Finding the EACCESS error I tried to build the cli from the branch: https://github.com/sunodo/sunodo/tree/feature/use-volumes-instead-of-bind-mount
Ideally it would build normally following yarn && yarn build or (cd apps/cli; yarn && yarn build) everything should run properly including the build of the devnet package
🫠 Actual behavior
It seems that after yarn && yarn build the hardhat compile command finds nothing to compile and this leads to a series of errors as shown below:
john@DESKTOP-AGNT5V1:~/sunododev/sunodo/packages/devnet$ yarn build
yarn run v1.22.21
$ run-s compile export deploy:anvil build:docker
$ hardhat compile
Nothing to compile
$ hardhat export-artifacts export/artifacts --hide-sources --including-empty-bytecode --including-no-public-functions
Nothing to compile
$ hardhat --network localhost deploy-anvil --dump-file build/anvil_state.json --silent
deploying to anvil and dumping state to build/anvil_state.json
Error: spawn anvil ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:286:19)
    at onErrorNT (node:internal/child_process:484:16)
    at processTicksAndRejections (node:internal/process/task_queues:82:21)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
ERROR: "deploy:anvil" exited with 1.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

🧪 Minimal test case
For this test I cloned the repo, switched to the branch:
https://github.com/sunodo/sunodo/tree/feature/use-volumes-instead-of-bind-mount
And updated the Anvil version as follows, switching from "3d6bfddf1e9335c623ba47b590da532f8d785ef4" to "6fc74638b797b8e109452d3df8e26758f86f31fe":
 # download pre-compiled binaries
-ARG ANVIL_VERSION=3d6bfddf1e9335c623ba47b590da532f8d785ef4
+ARG ANVIL_VERSION=6fc74638b797b8e109452d3df8e26758f86f31fe
 RUN curl -sSL https://github.com/foundry-rs/foundry/releases/download/nightly-${ANVIL_VERSION}/foundry_nightly_linux_$(dpkg --print-architecture).tar.gz | \
     tar -zx -C /usr/local/bin

🌎 Environment
Windows using WSL 2 and Docker Desktop for Windows
WSL: Ubuntu: 22.04.3 LTS
Node version: v20.10.0
Yarn version: 1.22.21
Docker version: 24.0.6, build ed223bc

You need to install anvil.
This should be somewhere in the README, but I guess it's not.

With anvil installed it works, thanks!


[Bug]ResourceManager: Concurrency problem during resource release

Describe the bug
In the case of large concurrency, ResourceManager(RM) ignores the consideration of resource increase and decrease when the engine is released

How many concurrency, this problem occurs

How many concurrency, this problem occurs

At the same time, it will be triggered when 15,000 jobs are submitted


Linux: Permission denied: '/dev/ttyACM0'

I installed on Ubuntu 22.04.1 and after launching, terminal reported this permission denied error. Follow instructions from this StackExchange post (https://askubuntu.com/questions/133235/how-do-i-allow-non-root-access-to-ttyusb0) to verify the ttyACM0 Arduino device is the "dailout" user group and add your $USER to the group. Suggest adding to Linux install instructions

Thanks for the suggestion. Install instructions have been added to the README file.


Set proper mime type for HL7 messages received from senders

Need to set the proper mime type(s) for HL7 messages received from senders.
Acceptance criteria:

Determine the proper mime type(s) to accept for HL7 messages.
The report service checks for the proper mime type(s) to accept an HL7 message

First, there is no HL7 mime type specified in the IANA list.  There is an old 2008 proposal by HL7 on what mime types to use, but it mentions registering them with IANA and that has not happened the best source I can find is the HAPI library, which is already using some similar mime types as documented here.  Based on that information and HAPI's implementation, I will recommend using application/hl7-v2 per their implementation in https://hapifhir.github.io/hapi-hl7v2/hapi-hl7overhttp/xref/ca/uhn/hl7v2/hoh/encoder/EncodingStyle.html#16


Public Cloud: Delete text about MoU from modal windows of Requests that have eMOU mechanism

Delete text about MoU from modal windows of Requests that have eMOU mechanism:
The modal window after Admin Review Approval (here delete the text to user as the modal will be seen by Reviewing Admin):

The modal window after Submitting Create Request:

The text for 'All Set' modal is updated:
In case if Account Coding is new:

odal is updated:
In case the Account Coding is new:

In case if Account Coding is known:

The Admin's 'Approved' modal is updated:


Still maintained/active?

I see a lot of open issues without activity (many of which could be closed) and no commits since Sept 14, 2014. Is this project still active?

Sadly I think its dead. IMO

Sorry, didn't get this or you didn't aware that a 1.8 pre-release has been
released?
krisdb2009<EMAIL_ADDRESS>於 2015年2月4日星期三 寫道：

Sadly I think its dead. IMO
—
Reply to this email directly or view it on GitHub
https://github.com/alecgorge/jsonapi/issues/425#issuecomment-72782836.

--
Should you require further information, please do not hesitate to contact
the undersigned.
Thank you very much
Yours sincerely,
Ivan<EMAIL_ADDRESS>Phone: (852) 9806-3212

Try switching to the 1.8 branch, last commit was a month ago...
There is a working 1.8 build, can't say it's 100% working; more a slow rolling build, you have to remember developers usually have months of working with unstable bukkit builds before it turns stable enough for people to actually start switching over, this update was a lot different in that it just popped out 'stable' one day with no release builds, so normal time constraints of life still exist into such a shorter time period.

It is alive but I'm not sure it is active for now.
I don't play Minecraft anymore and I haven't for years. With the implosion of CraftBukkit, I have absolutely no idea what API to write this plugin against because I can't figure out what people are actually using. Is it Spigot? Is it Sponge? Is it Forge?
I want to write JSONAPI v6 that supports this full platform natively and remove pretty much all of the legacy support for the old API, old authentication strategies and old platform support but I don't feel comfortable putting in the substantial effort necessary to keep up with things because it feels like any of them can disappear at any time.
I would also be open to adding any number contributors and pull requests are always accepted :)

I would got with Spigot for 1.8. That is the "new" Bukkit.

That sounds cool. We are using Spigot and are glad when the new version come out.

New builds on Jenkins should be compatible with Spigot 1.8 only: https://ci.alecgorge.com/job/JSONAPI/
It probably isn't perfect, but it is a lot better than it was before.

Please see #426 if you would like to discuss new features and open an issue for new bugs


Issue getting simultaneous Deepdream working alongside Neural-Style

I am trying to get Fast-Neural-Style's DeepDreamLoss feature working in Neural-Style.
I currently have this modified version of Neural-Style setup: https://gist.github.com/ProGamerGov/9e7b109458828a1f9e7cf995a4d2fd4b, but the Deepdream loss remains 0 no matter what I do. Here is the command line output when using the Deepdream related parameters: https://gist.github.com/ProGamerGov/fa20f22cdd5a89c2ef998b2f6b716dd1
Currently in my modified version of neural_style.lua, there are two parameters: -deepdream_layers, and -deepdream_weights.
I tried to replicate the style/content loss functions with the Deepdream loss function as well as remaining faithful to the Fast-Neural-Style code. But I am not sure where exactly the Deepdream code should connect with the loss functions as the Fast-Neural-Style code is setup differently than Neural-Style's code.

@ProGamerGov In DeepDreamLoss:updateOutput you are not updating self.loss, so it will always be 0. Computing the loss doesn't actually matter for creating images; it's just a convenient way for you to track progress. What actually matters is that you compute the gradient with respect to the loss, which your code seems to do correctly.
Therefore even though the loss remains 0, I think that your code should be able to produce DeepDream images if you get the hyperparameters right.

@jcjohnson Thanks for the reply. It seems to work now that I fixed the name of the deepdream_weights parameter on line 155.

Though as seen in the image above, and this image, there seems to be a lot of blue and green lines. I think this may be related to the max_grad input, but I am not sure what value it is supposed to be. I am currently assigning the the normalize_gradients value to the max_grad input.
Also, is implementing the other Deepdream parameters like -num_octaves and -octave_scale possible?

@jcjohnson Thanks for the reply. It turns out that I misspelled the -deepdream_weights parameter I was passing to nn.DeepDreamLoss , and that's why the code was not working. After some experimentation, I believe it now works like slow_neural_style.lua's DeepDream, but with all the features of the regular neural_style.lua.
I cleaned up the script by removing the unneeded parts that I originally added for DeepDream: https://gist.github.com/ProGamerGov/ff2715386d738e8245335b1a95617a1b

In DeepDreamLoss:updateOutput you are not updating self.loss, so it will always be 0. Computing the loss doesn't actually matter for creating images; it's just a convenient way for you to track progress.

The style and content loss functions update self.loss with the criterion. Is it possible to update the Deepdream self.loss without the criterion, and is the information obtained from doing so as useful as it is for the content and style loss functions?

I am attempting to get octaves working, but I image.scale does not appear to work inside DeepDreamLoss:updateGradInput using the code below.
local octaves = {}
     octaves[self.octave_n] = self.gradInput:add(self.clipped, -self.max_grad):div(self.max_grad)
     for i=self.octave_n-1,1,-1 do
        local _,h,w = unpack(self.base_img:size():totable())
        octaves[i] = image.scale(octaves[i+1], math.ceil((1/self.octave_scale)*w), math.ceil((1/self.octave_scale)*h),'simple')
     end

From DeepDreamLoss:__init:
  self.octave_n = octave_n or 4
  self.octave_scale = octave_scale or 1.4

  self.base_img = base_img

base_img is the resized input image:
-- DeepDream contrast fix
  if params.deepdream_contrast ~= '' then
    for i = 1, #deepdream_losses do
      deepdream_losses[i].mode = 'contrast'

      local content_image_size = image.load(params.content_image, 3)
      content_image_size = image.scale(content_image_size, params.image_size, 'bilinear')
      deepdream_losses[i].base_img = content_image_size
    end
  end

The full modified neural_style.lua can be found here: https://gist.github.com/ProGamerGov/ad143adf1b7d4eb2bd2d177153abc00e

@ProGamerGov
at this line you effectively make gradInput a copy of gradOutput, which is probably why you avoid the errors, but you also lose everything you have done with gradInput so far.


Clicking "Add your review" from a PR, will break some CSS

Command attempted:
Clicking "Add your Review" from a PR
What happened:
Next page shows full width, some CSS (or markup) seems to be missing/broken. It's also missing all the repo nav.
More info:
PR:

After clicking "Add your Review":

After hard refresh:

Hi, I believe this is a question about GitHub.com, the website. This issue tracker is for a command-line utility called hub, not the website itself. For reporting bugs and requesting features for GitHub the website, please go to https://github.com/contact or visit https://github.community. Thank you!


update dependencies

Pull Request Test Coverage Report for Build 270

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

Totals

Change from base Build 266:
0.0%

Covered Lines:
223

Relevant Lines:
223

💛  - Coveralls


Taking out the singularity or upgrade renders the teminal unusable until you come back in range

Don't close the terminal when out of range (not really feasible)
alternatively allow inserting the singularity without opening the GUI
same applies to the upgrade

leaving it open is basically impossible without rewriting half of the ae2 code (which I don't really want to do)

fixed in ae2


Testing/guble #79 improve code coverage for store package

Changes Unknown when pulling<PHONE_NUMBER>716f9fc6df4e356543e2b093fa9f9a on testing/guble-#79-improve-code-coverage-for-store-package into ** on master**.

Coverage increased (+1.8%) to 80.744% when pulling 0b7a97adddb3ba53ceb4f7a5180fcdd5a7109298 on testing/guble-#79-improve-code-coverage-for-store-package into 5b83f624ec8607814a0327dc5b5b2e5cb980f77b on master.


[IOTDB-856] Clear up the IoTDBSessionIT

It looks better 👍 One more step, we could move some tests that are independent to another IT file
There will be two IT files:
IoTDBSessionComplexIT
IoTDBSessionSimpleIT

OK, thanks for your suggestion. I will improve it

Hi, please fix the conflict~

Hi, please fix the conflict~

Copy that


Reverse Geocoder option for Intersection/Blocks

I would like to request an enhancement where the reverse geocoder accepts a parameter specifying the output format.  Currently it looks like I can only receive the closest address. It would also be helpful if the closest intersection could be returned, or closest block face.

I did a little digging into what the API supports and how things work currently.  It looks like this would be fairly straightforward to do today even with the following:
var reverseGeocoder = MapManager.geocodeService.reverse();
reverseGeocoder.params.returnIntersection = true;
reverseGeocoder.latlng([Lat, Long]).run(...)
However it looks like it returns some weird results for the intersection.  Use the example point [45.3679962, -75.77500239999999] it returns:
Address: "RR-38 & HWY-17B"
Without this it returns an address on Carling Ave.  It looks like the intersection functionality is not using the common names for the roads?

i'm seeing all kinds of bogus results in Ottawa.  best i can tell we are storing both french and english streetnames and incorrectly calling the two an intersection.

i'll notify the World Geocoding team and let you know what i hear back.

good news!  they're not only already aware of the problem, its already been patched.
look for more appropriate behavior after the next release of the World Geocoding Service (planned for December).

Great, thanks for the update.  Any thoughts of exposing returnIntersection directly in esri-leaflet-geocoder, or should it be used as I suggested?

Also in your example it's at least picking the correct road - just getting the french/english versions as the intersection.
In my example I would have expected it to pick Carling Ave. as one of the points as it does with closest address.  However it was giving regional road names - I'm not sure if this is the same problem as the English/French issue you saw can you confirm?

nothing wrong with the approach you're using now, but I agree it needs to actually be 'surfaced'.
I had hopes initially that the geocoder would behave after I used the same technique to apply langCode, but no joy.

since the repro case you mentioned displays a tack in a location that lacks an actual intersection, i think its safe to say that its a result of the same problem (the geocoding engine's failure to throw out potential intersections composed of streets with identical geometry).
as to how alternate streetnames are prioritized in the matching process for valid intersections, i don't know...

feel free to take a look at #106 and provide more feedback if my fix doesn't seem sufficient.  either way, thanks for the feedback!

Took a look at #106 looks good, thanks for the quick response!


CLPF partial super blocks

Apply CLPF to any partial super blocks along the right and/or bottom of
frame.

This will cause CLPF to be applied outside the frame dimensions, and reconstructed pictures are not padded unless/until they become references, so this breaks.


Moving ram (4) resources to v1beta1 version

Moving 4 resources in the ram group:

[ ] aws_ram_principal_association
[ ] aws_ram_resource_association
[x] aws_ram_resource_share
[ ] aws_ram_resource_share_accepter

Hi team,
Is there any progress with that ticket? Because I have not been able to find a way to send invitations to multiple accounts within the same organization, nor have I found a way to accept invitations using Crossplane. While there is one CRD available for creating a resource share, it does not provide an option to specify which accounts to invite.


Implement foreign-toplevel-info-v1

wayland-protocols MR: https://gitlab.freedesktop.org/wayland/wayland-protocols/-/merge_requests/59
I'm not sure how best to namespace these within wlroots such that things are consistent and concise but there is no conflict with the current wlr_foreign_toplevel APIs. I've gone with a wlr_ext prefix for now but am hoping for suggestions.
Implementation of both new protocols in river on this branch.

wlroots has migrated to gitlab.freedesktop.org. This pull request has been moved to:
https://gitlab.freedesktop.org/wlroots/wlroots/-/merge_requests/2588


Improve @fuel-ts packages references inside fuels-ts

When clicking on from "@fuel-ts/<package>". The link opens the node_modules folder. But it should open the package folder.
PR #238 used the typescript directly, which was good, but pnpm link was broken.
PR #246 fixed the issue with pnpm link, but the correct track of the package was lost opening the node_modules.
We should have both

Good linking
pnpm link working

This is another issue I was having since #238. Both linking and clicking refs worked fine before.
I'm actually going to write a script that switches this repo from PNPM+tsup to Yarn+tsc so I can have all the functionality lost with #238 back.

I think we can revert #246. The issue was not being able to link to a Yarn project, but since SwaySwap is also PNPM now maybe that's not an issue?
I'm trying this locally now and it seems fine: https://github.com/FuelLabs/fuels-ts/pull/345
(Didn't try linking with NPM though, and I might be breaking other stuff so not sure.)


Update contracts to use MIT license

Codecov Report

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

@@            Coverage Diff            @@
##             develop     #83   +/-   ##
=========================================
  Coverage           ?   0.00%
=========================================
  Files              ?      19
  Lines              ?    1419
  Branches           ?     234
=========================================
  Hits               ?       0
  Misses             ?    1419
  Partials           ?       0

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 94d3482...f5bf758. Read the comment docs.


How do I handle multiple web applications?

This is great for a single web application but how do I handle binding multiple web applications?

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

ID: f079fa08-284c-3f14-be4a-1f32d5b0e7b9
Version Independent ID: 707cbeee-d311-d430-6119-e9ae8b8aeae5
Content: Using Azure AD for SharePoint Server Authentication
Content Source: Enterprise/using-azure-ad-for-sharepoint-server-authentication.md
Service: o365-solutions
GitHub Login: @MSFTTracyP
Microsoft Alias: tracyp

@kaevans I don't think I quite understand what you mean.  Say per your example, with the reply URL, https://portal.contoso.local/_trust/default.aspx, how does that apply to a MySites Web Application that should have a reply URL of https://mysites.contoso.local/_trust/default.aspx?  I cannot seem to apply multiple reply URLs or wildcards through the GUI or through Set-AzureADServicePrincipal.  When I try to add additional Reply URLs via Set-AzureADServicePrincipal I get the error

Set-AzureADServicePrincipal : Error occurred while executing SetServicePrincipal Code: Request_BadRequest Message: One or more properties on the service principal does not match the application object.

Add multiple provider realms to the SPTrustedIdentityTokenIssuer. See Steve Peschka's blog on multiple claims auth web apps

Could you please clarify, do I need multiple sptrustedidentitytokenissuers or I need one sptrustedidentitytokenissuers with multiple realms? I think it will be helpful if scenario with multiple web apps will be described in the article.
Thanks.

now it works :)
$t=Set-SPTrustedIdentityTokenIssuer "TRUSTNAME"
$t.UseWReplyParameter=$true
$t.Update()

Hi Joker-it, how did you add more than one reply url to the azure ad enterprise app? I see the possibility to add only one reply url.
I've created 2 azure ad apps and two sptrustedidentitytokenissuers with both realms in each. It works for me, but i guess it's not a good solution.

It looks like this issue is closed per kaevans' response. Please reopen if you are still having trouble. Thank you very much for the feedback!

Hello SashaSidorenko, you can edit the reply urls in the app registration. its possible to add more than one replyurl... you can also do this with powershell..
i need also more then one identifier.. in the app registration you can edit the manifest directly... this way you can add more than one identifier and replyurl..
i think two trustedidentitytokenissuers are not a good solution because the permissionmanagement would be complicated...
i hope i could help 👍

Added pull request https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/pull/52


Modify training_on_test_set task to also check for git commit hashes

When commits are viewed on github, they show the commit hash ... but they show only excerpts of files being modified. Even if modified files had a canary string, the canary string might not be present on the web page showing the commit. Additionally, many files have earlier commits where they did not yet have their BIG-bench canary string. Naive filtering of the BIG-bench canary string may still include these webpages (or mirrors of these webpages) in training data.
In order to better help identify when web scrapes hold BIG-bench task data, this PR expands the set of canary strings tested for in the training_on_test_set task to include github hash strings, which are shown on most github webpages looking at repository history.
See also discussion in https://github.com/google/BIG-bench/issues/574

LGTM!


Some indication that search has wrapped around to the beggining of the file and/or beginning of the search

I often find myself searching for a string through very large text files.
When the search hits the end of the document, it wraps around to the beginning.
When there are no more results, the search returns to the first find and keeps going.
There's no super-obvious indication that either of these has happened - so I find myself having to keep an eye on the scrollbar and memorize the approximate location where the search started so I can know when I've probably passed that point again.
Visual Studio adds a message to status bar when wrapping to the start of the file.
By default it also has pretty good behaviour when the search is complete:
It plays the generic attention bong sound and changes the status bar to "Find reached the starting point of the search" and leaves the selection on the last found string. Thus it's quite obvious on that press of F3 when the selection doesn't change that the search is complete.
Pressing F3 again will move to the original selection and start searching again.
A lot of (most?) editors do something similar.

Sublime text has the option to turn wrap off. So it stays at the last result if you keep pressing enter. Something like this would be awesome.


price-limit should also be enforced on state layer

price-limit should also be enforced on state layer
Description
On our mainnet, it appears that despite enforcing a price-limit of 1 gwei on our RPC and most of our validator nodes, validators that do not have price-limit set are still writing blocks with underpriced txs - and all validators successfully confirm these txs rather than failing on them.
We confirmed that consensus_backend.InsertBlock() is still being called which calls executor.ProcessBlock(), ignoring the --price-limit value. As a result, all receipts for any underpriced txs are seen as successful.
Your environment

OS and version Ubuntu 20
version of the Polygon Edge 0.5.1

Steps to reproduce

Spin up 5 validator testnet
Enforce price-limit of<PHONE_NUMBER> (1gwei) on validators 1-4. Validator 5 should still allow for any tx price.
Send underpriced txs to validator 5.
Observe that although validator logs 1-4 show transaction underpriced errors while handling gossiped txs (see logs below), the txpool of validator 5 adds it, and as a result the txs are written to chain by validator 5.
Validators 1-4 confirm and write these blocks with underpriced txs to chain rather than erroring out on them.

Expected behaviour
I would expect validators to avoid writing txs that are priced under the price-limit value, even if written by another validator that does not have price-limit set.
Actual behaviour
It looks like underpriced txs can still be written to chain despite having the price-limit set on the majority of validators.
Logs
2022-10-24T15:55:23.888Z [DEBUG] polygon.txpool: add tx: origin=gossip hash=0x3f41e998f83aba9c9c0aa129f69d3beccb8c21f8b74914aeeaab06316add8235
2022-10-24T15:55:23.889Z [ERROR] polygon.txpool: failed to add broadcast tx: err="transaction underpriced" hash=0x3f41e998f83aba9c9c0aa129f69d3beccb8c21f8b74914aeeaab06316add8235
2022-10-24T15:55:23.891Z [DEBUG] polygon.ibft: validator message received: type=COMMIT height=8977808 round=0 addr=0x8fB53330b1AEa01f6d34faff90e0B7c2797FC3aD
2022-10-24T15:55:23.891Z [DEBUG] polygon.txpool: add tx: origin=gossip hash=0x0c389a01683ffd929e74e1a6aa0c38e7adf634afe41ce0277b4633235f45151b
2022-10-24T15:55:23.892Z [ERROR] polygon.txpool: failed to add broadcast tx: err="transaction underpriced" hash=0x0c389a01683ffd929e74e1a6aa0c38e7adf634afe41ce0277b4633235f45151b

Proposed solution
Improve executor.ProcessBlock() logic so that txs that are underpriced also fail to be applied (receipt = 0).

Hi @dankostiuk ,
How did you send the transactions? Via Metamask or some script?

@laviniat1996
Via a script, however the behaviour should be the same for simple Metamask transfers if targeting the validator RPC url that does not have price-limit set.

@laviniat1996 This is not issue. But as @dankostiuk said, price limit configuration is not applied in consensus layer currently. So the validator that doesn't set price-limit can propose the block including transaction with higher gas price and other validators can accept the block because price-limit is not used in consensus layer.

@Kourin1996 Thank you. I thought this was expected behaviour, but wasn't 100% sure.

@dankostiuk As this is expected behaviour, I will close this issue for now. If you have any suggestions on improvements regarding this, please feel free to open a new issue. Thank you 🙏


Clean error message for check_out() and refactor to ..._solution

This PR migrates the error message when checking out a Scenario with a solution from Java to Python, to get the correct function names: remove_solution() instead of the underlying Java function removeSol().
In addition, the PR adds a function has_solution() and renames remove_sol() to remove_solution(). The kwarg of clone() is changed from keep_sol to keep_solution, but keep_sol continues to work (with a deprecation warning).
Closes #71

CircleCI passed with tests using deprecated clone(..., keep_sol=False) formulation. Now updated the tests formulation to keep_solution=False to avoid deprecated-warning when running continuous integration.

Ok, just going to merge, not worth changing at this point. Thanks @danielhuppmann!

hey @danielhuppmann, I forgot to think about the release notes. probably worth adding a note that there is a new has_solution() function..


Stuck at install.js

At this stage of installing node-sass via NPM:
><EMAIL_ADDRESS>install C:\temp1\node_modules\node-sass
> node scripts/install.js

With the log at this stage:
npm sill doParallel update-linked 0
npm sill doSerial install 1
npm sill install<EMAIL_ADDRESS>C:\temp1\node_modules\.staging\node-sass-265c51ce
npm info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>
Nothing happens. It remains as it is, forever...
Full logging details can be found attached.
error.txt
info.txt
log.txt

Can you check if it is maybe downloading the binding and it takes a very long time? (for example netstat output might give you info about the open connections).

@saper bugs like this and https://github.com/sass/node-sass/issues/1662 are why I think the progress bar is a good idea.

@xzyfer I understand but still not convinced. For those cases a simple message informing we are about to start the download would do.

@saper It was a proxy issue. Company firewall was blocking all HTTPS traffic, which install.js uses. I agree with @xzyfer a progress bar or at least some more information in the install.js would be useful


[help] How can to get USDT balance from margin account?

I can get my balance as follows:
balances = client.get_account()

But I won't able to get margin balances especially USDT. Is there any way to get USDT balance from margin account?

Hello,
It might not be the most elegant way to do it, but you can try:
def BalanceUSDT():     try:         b=client.futures_account_balance()         b = pd.DataFrame.from_dict(b)         b = b.loc[b['asset']=='USDT']         balance_USDT = float(b['balance'].values)     except:         balance_USDT=0     return balance_USDT

@TheSeacfr Working code:

def get_balance_margin_USDT():
    balance_USDT = 0
    try:
        b = client.futures_account_balance()
        b = pd.DataFrame.from_dict(b)
        b = b.loc[b['asset']['USDT']]
        balance_USDT = float(b["balance"].values)
    except:
        pass
    return balance_USDT

But still it returns 0 for the balance in margin.

I am sorry but I don't use margin so I cannot really test on my account.
However, based on what I have sent before, something like that should work:

def get_balance_margin_USDT():
try:
b=client.get_margin_account()[1:]
b = pd.DataFrame.from_dict(b)
b = b.loc[b['asset']=='USDT']
balance_USDT = float(b['free'].values)
except:
balance_USDT=0
return balance_USDT

Maybe you'll need to modify it slightly since, as I said, I could not test it.

b=client.get_margin_account()[1:] returns *** TypeError: unhashable type: 'slice'

I verify that following code piece works:
def get_balance_margin_USDT():
    try:
	_len = len(client.get_margin_account()["userAssets"])
	for x in range(_len):
            if client.get_margin_account()["userAssets"][x]["asset"] == "USDT":
		return client.get_margin_account()["userAssets"][x]["free"]
    except:
        pass
    return 0

works for me:
def get_balance_margin_asset(asset):
balance_asset = 0
try:
b = client.futures_account_balance()
b = pd.DataFrame.from_dict(b)
b = b.loc[b.asset==asset, 'balance'].values[0]
balance_asset = float(b)
except:
pass
return balance_asset
to run e.g.:
get_balance_margin_asset('USDT')


Enable frozen_string_literal magic comment in pubsub

When memory profiling an app using this gem, I noticed that a large number of "@grpc" and "ASCII-8BIT" strings allocated. This occurs for each received message.
This patch addresses the issue and future issues by enforcing a frozen string literals for all files in this library, and adding a rubocop rule to enforce this pattern in new files.

For a reference, after receiving 3000 messages, here is the string allocation report we get from memory_profiler
     99059  ""
     45000  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/google-cloud-pubsub-0.30.2/lib/google/cloud/pubsub/message.rb:67
     30000  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/google-cloud-pubsub-0.30.2/lib/google/cloud/pubsub/message.rb:68
     21000  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/grpc-1.12.0-x86_64-linux/src/ruby/lib/grpc/generic/rpc_desc.rb:47
      3000  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/google-cloud-pubsub-0.30.2/lib/google/cloud/pubsub/received_message.rb:57
        42  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/google-cloud-pubsub-0.30.2/lib/google/cloud/pubsub/subscriber/async_pusher.rb:164
        16  /home/ianks/.rbenv/versions/2.5.1/lib/ruby/gems/2.5.0/gems/google-cloud-pubsub-0.30.2/lib/google/cloud/pubsub/subscriber/stream.rb:314

Great catch! This was an oversight and should be fixed.
My understanding is the frozen_string_literal: true comment works on Ruby 2.3 and newer. Since we are supporting Ruby 2.0 and newer I have created #2123 to address the strings reported in this PR on all supported Ruby versions. If there are more strings that should not be created please report them there.
That said, I can see an argument for also adding the frozen_string_literal: true comment to google-cloud-* gems in addition to not creating more strings than is necessary. At a minimum, we should know where we are mutating strings so we can avoid those mutations in preparation of Ruby 3.0.
In the past I've run the following, only to have it error in Bundler:
RUBYOPT="--enable-frozen-string-literal" bundle exec rake test

@ianks Can you please rebase your branch on master in order to get this commit which should fix the broken CI build for Ruby 2.0 for this PR? Thanks!

@quartzmo Done!

@quartzmo I'm considering putting this PR on hold until #2130 is unblocked and merged. Pairing the frozen string comments with dropping unsupported rubies, and updating rubocop. Thoughts

@blowmage Agreed. I'll create a new issue for adding frozen string comments to all packages.
@ianks Are you interested in expanding this PR to add the comment to all packages, or would you rather close it and leave that to us? In either case, we really appreciate your push to make this happen.


Week 1 Hackathon Entry

My week 1 World Builder

You’ve done a fantastic job on this hackathon challenge. Well done on handling the max recursion error you were getting and I like how you label the continents differently on the visited map.
You’ve already identified one way to level up your solution and that’s to ask the user what size world they would like to create. Another way would be to take into account if multiple continents had the largest land sizes. For example, when I ran your program, the largest continent had 420 land tiles, and the next largest continent size was 4. But there were 2 continents that each had 4 land tiles. Perhaps you could implement a check for this type of situation.
You could also tell the user which label to look for on the map to see the largest continents, e.g. if the largest continent is marked with a 2 and the next largest is 3 then you can tell them that the 2 continent is the largest and the 3 continent is the second largest for them to play on.
Your code is very clearly written and you’ve named your variables well so your program will be easy to maintain in the future. The TODOs that you’ve added are very helpful as well.
I love the ‘Welcome to water world!’ message, it’s a great way of dealing with a potential error and giving the user some fun as well.
Congratulations on solving the hackathon challenge and good luck with the rest of the summer!


Unleash the pandas

PR Description
Currently using the banner art from https://github.com/ConsenSys/teku/pull/5781/
Prints the panda when the first block with a non-default payload is successfully imported. Will only ever print once and records the fact that it printed on disk to be completely sure of that.
Fixed Issue(s)
fixes #5231
Documentation

[x] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[x] I thought about adding a changelog entry, and added one if I deemed necessary.

We may want to consider only printing the pandas if the merge block is imported within some reasonable period of when the block actually happened (ie not when you sync a historic chain).

This is very late, but I'm very happy :heart: :panda_face:

You can't waste pandas on these lesser testnets you know... :)


undefined: netFD while trying to run a simple http server

This is the code
package main

import (
	"fmt"
	"log"
	"net/http"
)

func main() {
	http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		fmt.Fprintf(w, "Hello, world")
	})

	log.Fatal(http.ListenAndServe(":8080", nil))
}

Trying to compile this gives -
GOOS=js GOARCH=wasm ./bin/go build -o net.wasm wasm_net.go
# net
src/net/net.go:164:6: undefined: netFD

If you can point me out to the area where it needs to be worked, I will be happy to look into it. Really looking forward to Go integration for wasm and will be glad to help out in any capacity.

Ah my bad. I see that test_all.sh doesn't support the net package yet. Can you give me some pointers to where I can start to support the net package ?

Looked into it. The build tag js is missing for platform specific files under src/net. Not sure how you want to do this. Do you just want to add the js tag on all files ? But then if there are different implementations for unix and windows, how do we handle that ?

HTTP servers are unlikely to be supported soon, because they would never work in web browsers.

Oops, didn't intend to close right away.

I see. Understood.
I have a general question though - Since you are using build tags to control the build output, what if we have a Go file which has different implementation for unix and windows ? How are we handling that case ?
Thanks for your work.

Those packages will need to target js/wasm as well, since it would probably need a different implementation anyways.

shouldn't we also plan to support say linux/wasm, darwin/wasm, ... too ?
so, e.g., one could instantiate a wasm module with definitions for println, runtime.nanotime, ... that call into a "host-module" instead of the js importObject that you define in the nodejs bootstrap code.
this would allow to directly execute the wasm module w/o a javascript VM.

How would you do that? Which WebAssembly compiler/runtime would you use?

Also: Why would you want to run WebAssembly like that? Isn't it easier to simply compile Go to that target in the usual way? WebAssembly is only really useful in the web browser.

wasm is a nice bytecode standard (one that is bound to be widespread, thanks to its web support)
why wouldn't one want to run it directly, w/o a browser ?
wagon/cmd/wasm-run can already interpret and run a fair set of the official wasm testsuite. (imports aren't supported right now, though)
one could dream of wagon being, eventually, a complete Go interpreter, written in Go, consuming wasm bytecode that has been compiled with go :)

What are the practical advantages compared to compiling Go directly to the target architecture?

you get a REPL :)

A REPL? How?

with a wasm module, you get something that is more amenable to being interpreted by an interpreter.
once you have an interpreter that can ingest wasm bytecode, you can build an interactive interpreter on top of that: you have your REPL.
you could of course directly interpret the Go source code, either using directly the AST (such as here, or there or there) or using SSA (such as there).
but having a VM that can interpret bytecode is a more efficient way to get at a REPL. (that's what CPython, PyPy, Ruby, ... all do, in the end.)

once you have an interpreter that can ingest wasm bytecode, you can build an interactive interpreter on top of that: you have your REPL.

It doesn't work that way. The Go compiler always emits a fully linked application.

Those packages will need to target js/wasm as well, since it would probably need a different implementation anyways.

 But how can a single wasm implementation of a package support both linux and windows, when they have different implementations in Go for linux and windows ? Is there some wasm specific code which works at a level above the OS layer ?

Here's the layer: https://github.com/neelance/go/blob/wasm-wip/misc/wasm/wasm_exec.js#L130

Ahh thanks. I think I get it now. Thanks for your patience. Closing this now as my questions are answered.


feat(runtime): add Python 3.13 support

Issue number: #5411
Summary
Changes
Lambda already published the base image  for Python 3.13 (preview mode) and then we can start adding support for this new runtime in Powertools.
User experience
WIP
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Constructor running with Python 3.13 - https://github.com/leandrodamascena/test-powertools-action/actions/runs/11828109022/job/32958160741

CI is failing with
OSError: [Errno 28] No space left on device

😰

e2e test is green
https://github.com/aws-powertools/powertools-lambda-python/actions/runs/11828379716/job/32959447655

CI is failing with
OSError: [Errno 28] No space left on device

😰

To be honest, this was my fault. I was running the CDK layer tests with an matrix of all Python versions, but I don't need that because I don't want to test if it's being built on a specific Python version, we know that build, I want to know if given a Python Runtime I can build the dockefile and synthesize CloudFormation.
Lets wait the CI run again.

Ok thanks- can we also address the SonarCloud findings?

3 New issues

I'd rather not change them. These warnings are related to the Dockerfile in the sense that you're deploying a Docker image and it can create extra layers, add unnecessary Docker layer space, and so on. But that's not our case, the only reason we use a Dockerfile is to build using the actual Lambda environment (base images) and address things like libraries that need to be installed for a specific architecture or OS, and architecture. We don't use this Dockerfile to serve Docker images to customers, we just "build" and extract the resulting files from that build.
I'll mark all of them as "accepted"

@dreamorosi ready for review.


move peer sync to sync module

to be done after sync worker is implemented #97

We ended up moving sync to peer module.


Docs logo and style update

Added the scPortrait logo to the docs
Added logo versions for light and dark mode
Updated the Docs theme

@nik-as as soon as the checks complete and don't fail feel free to merge


bash-completion: fix build on darwin (#107768)

Motivation for this change
As discussed in https://github.com/NixOS/nixpkgs/issues/107768, macOS APFS and HFS+ seem to have issues extracting some of the test fixtures because of filesystem issues (case sensitivity, binary representation of characters, ...). This means that people currently won't be able to build things from source on macOS.
I'm a bit confused as to how the binary caches with this version got created.
By switching to fetchurl, we will still validate the tarball checksum itself while being able to compile on mac and linux.
It's not the cleanest solution, but it does work.
I tried to look into things like cleanSource / cleanSourceWith, but I'm not 100% sure that would solve the issue at hand.
Comparison:
Grab the new files:
% nix-build default.nix -A bash-completion
/nix/store/br1vvv70z4icia58fzlcw1nxid28ww62-bash-completion-2.11
% pushd /nix/store/br1vvv70z4icia58fzlcw1nxid28ww62-bash-completion-2.11; find . > ~/new.txt; popd

Grab the current binary results:
% nix-env -i bash-completion
warning: there are multiple derivations named 'bash-completion-2.11'; using the first one
installing 'bash-completion-2.11'
these paths will be fetched (0.14 MiB download, 0.97 MiB unpacked):
  /nix/store/jyniimfqps4b1dgm2ya0hqyilc7g4y8v-bash-completion-2.11
copying path '/nix/store/jyniimfqps4b1dgm2ya0hqyilc7g4y8v-bash-completion-2.11' from 'https://cache.nixos.org'...
building '/nix/store/xbfs4rdn10ir4bjbnqahkzbrc0s6fs5a-user-environment.drv'...
created 257 symlinks in user environment
% pushd /nix/store/jyniimfqps4b1dgm2ya0hqyilc7g4y8v-bash-completion-2.11; find . > ~/binary.txt; popd
/nix/store/jyniimfqps4b1dgm2ya0hqyilc7g4y8v-bash-completion-2.11 ~/workspace/nixpkgs
~/workspace/nixpkgs

Compare the filenames:
% diff -qs ~/binary.txt ~/new.txt
Files /Users/mseeger/binary.txt and /Users/mseeger/new.txt are identical

Sanity check:
% head -n 5 ~/binary.txt
.
./etc
./etc/profile.d
./etc/profile.d/bash_completion.sh
./share
% head -n 5 ~/new.txt
.
./etc
./etc/profile.d
./etc/profile.d/bash_completion.sh
./share

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[X] macOS
[X] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@SuperSandro2000  noticed that this derivation caused issues on the last merge: https://github.com/NixOS/nixpkgs/pull/101028
I'm trying to run nixpkgs-review, but that'll take a while...
macOS:
[0/1699 built, 1854/3460 copied (10789.7/25415.3 MiB), 2430.0/7720.6 MiB DL]

Linux:
[0/14753 built, 1/156/14619 copied (1424.5/161995.0 MiB), 1253.8/69348.2 MiB DL]

Result of nixpkgs-review pr 107813 run on x86_64-darwin 1

1 package built:

bash-completion

Result of nixpkgs-review pr 107813 run on x86_64-linux 1

1 package built:

bash-completion

Please target staging branch.

/rebase-staging

Almost there, just need a second to properly rebase that on staging. The mercurial all day at $DAYJOB is showing :)

the files which caused issues were part of the testing framework.. do we still have tests?

why does this cause so many rebuilds...

the files which caused issues were part of the testing framework.. do we still have tests?
They are there, probably mangled on OSX. I don't think the current derivation executes any of the tests.

why does this cause so many rebuilds...

Because all these packages depend on bash-completions, presumably because they ship their own. That's why it's targeted at staging rather than master

we actually never had tests ;(, so this is just an improvement for mac users

Because all these packages depend on bash-completions, presumably because they ship their own. That's why it's targeted at staging rather than master

no, it's because dconf "links" against it for some reason, thus gets used as a build dependencies in a lot of glib / gnome related projects.


Update yacc.go

The generated $$Tokname() function doesn't work.  It seems to assume the tokens start at 1.  Changed to use $$Private instead.  Tested against a trivial xxx.y file, and added several fmt.Printf() calls to the main procedure to print out the names of defined tokens.

Test file and results:
xxx.y.txt
[dswartz@buildbox ytest]$ cat BAD
ABLE => "tok-57346"
BAKER => "tok-57347"
CHARLIE => "tok-57348"
[dswartz@buildbox ytest]$ cat GOOD
ABLE => "ABLE"
BAKER => "BAKER"
CHARLIE => "CHARLIE"

Fixes issue #44556.

This turned out to be a little more complicated than I thought.  The original comment is that this can't be called with a user-defined token, and that is in fact true.  Unfortunately, the generated GO code internally will subtract the starting information, such that if you set yyDebug=3,  you get things like 'lex tok-30 (57376)', because the function yylex1 will subtract $$Private if the input token is >= to that.  This still makes it infeasible for user code to call $$Tokname() unless it does the same math.  It's cleaner, I think, to have $$Tokname() restore the original code, but subtract $$Private if the token is >= to that.  This works for both the internal client (yylex1), and any user calls.

On reconsideration, this seems less than essential.  Might revisit it at some later time.


Hide note preview list

I am missing the ability to hide the note list (aka search results). You can pull up the divider so that it is gone but as soon as you want to switch to another note it pops open again (which to be honest makes sense since there are usually multiple search results).
This way private notes (at least their title and preview) are displayed in the upper half of the window every time they match the search word, which can be unpleasant with people looking over your shoulder.
I would love it if I could hide the entire list and access notes just by typing in their title until it is nonambiguous. An alternative would be an option to hide the note preview. Any chance either solution gets in the next update?

It worked correctly in original notational velocity, when you collapsed notes list it stays collapsed even when searching or after closing application. I miss this so much, search bar is all I need for navigation.
I will donate 20$ for this to get fixed.

Contributing 20$ as well. Still missing this.


Windows Workflow with E2E tests that are run against a podman machine with a --user-mode-networking flag set

After encountering a https://github.com/containers/podman/issues/20921 we could make use of a workflow, that would prevent a regression.
Idea could be as follows:
A host machine installs podman, then 3 options are provided as a parameter in order to prepare a podman machine for e2e tests to run against.

Rootless machine is initialized and started
Rootfull machine is initialized and started
A machine with (--user-mode-networking) flag enabled with the init command. There is also alternative to first init the machine and then call set --user-mode-networking=true afterwards.

Alternative approach was not handled in this issue. We create a machine with the --user-mode-networking flag set.

The PR brings also running e2e tests against rootfull and rootless machines besides the rootful with --user-mode-networking.


"$(fzf_history)unknown option: --tiebreak=index" when running C-r

I think there is a mismatch between the supporting scripts and the binary.
I just installed on a fresh box and it seems the binary does not have the required option.
Is there a simple way to fix?

Sorry my bad, I had a old copy in the path!


SCM: Timeline refresh calls trigger other refresh calls

Move Timeline view to under the SCM view
Refresh the SCM view
Notice the SCM view refreshes

🐛 The Timeline view refreshes as well
🐛 Then, the SCM view refreshes again

VS Code Will execute the  git.refresh command when clicking the refresh button.
https://github.com/microsoft/vscode/blob/9c76d66e695381ac392144bc3ec5c7e21edbb472/extensions/git/src/commands.ts#L380 and https://github.com/microsoft/vscode/blob/9c76d66e695381ac392144bc3ec5c7e21edbb472/extensions/git/src/repository.ts#L1163
Then, it will go into here which causes it refresh twice. After updateModelState method is executed, GitTimeline will be notified to update. This means that the run method will be executed again.
https://github.com/microsoft/vscode/blob/9c76d66e695381ac392144bc3ec5c7e21edbb472/extensions/git/src/repository.ts#L1879
https://github.com/microsoft/vscode/blob/9c76d66e695381ac392144bc3ec5c7e21edbb472/extensions/git/src/repository.ts#L1867

Here is the sequence of events:

Refreshing the SCM viewlet will call status - show progress in SCM viewlet
GitTimelineProvider listens to status operations and emit onDidChange event
GitTimelineProvider provides the timeline items - show progress in Timeline view

While providing timeline items it calls logFile() - show progress on the SCM viewlet

I have made a tactical change so that no progress is being shown while running the LogFile operation. By doing so, we will not be showing progress in the "Source Control" view while the Timeline is refreshing. This however does not fix the fact that when the Timeline is shown, it refreshes every time git status is being executed which is excessive.
@joaomoreno, one approach that would be valuable in multiple scenarios would be for the onDidRunGitStatus()  event  to pass the name of the "Operation" that was executed. This would enable listeners of the event to decide whether they want to do something or not based on the operation that was just executed. what do you think?

@joaomoreno, one approach that would be valuable in multiple scenarios would be for the onDidRunGitStatus() event to pass the name of the "Operation" that was executed. This would enable listeners of the event to decide whether they want to do something or not based on the operation that was just executed. what do you think?

Love it. 👌


Problem with sound in iOS passes

Hi,
I am testing the ApnsPHP_Push object as indicated in the example documents.
The APNS notification reaches the device, but I can't manage to hear a sound, no mater how I configure  the message with setSound();
I tried:
-not calling setSound();
-calling setSound();
-calling setSound('default');
-calling setSound('chimes');
I also worked through the settings in 2 different iPhones, enabling and disabling the alternatives in the notifications settings, with no success.
Is it possible the pass needs some configuration that could be missing to enable sound notifications or, for instance, in this case, a coupon pass can not receive sound notifications?
I am using latest version of iOS 6.0.1 with an iPhone 3GS.
Thank you!

@chefren Its very old issue, But could you please paste your code by which we can send Pass push notification to IOS device.

@chefren Its very old issue, But could you please paste your code by which we can send Pass push notification to IOS device.

Apologies @dharmesh-verve , it was for a project in a place I used to work, so I don't have it anymore. Did you generate the ref docs with doxygen? I guess I only took the example code from there


Avoid no-op export map updates

This is a follow-up to #44402.  Internal teams with large projects have seen significant perf wins from that change (~20% reduction in build time) but I've stumbled onto some pathological cases where the change actually increases build time by ~50%.
The builder maintains two copies of the exported modules map - changes are accumulated in a copy so that they can be discarded if something invalidated the update.  When the build does complete successfully, one ManyToManyPathMap is updated from another, which requires traversing both.  In local measurements, each such update was taking 2-4 seconds (this is very project-specific) and there were 2000 in the course of building a single project.  However, it turned out that the copy was never actually modified after its initialization, so all subsequent updates of the "real" map were no-ops.  This change introduces a cheap way to determine that the cache hasn't changed.
I think we could probably use a more sophisticated method to handle these updates (e.g. accumulating only a delta), but I deliberately made the change very simple because we're so late in the 4.4 cycle.

For my local repro
4.2: 631.23s
4.4: 962.68s
PR: 279.44s

@DanielRosenwasser @RyanCavanaugh I think it's important to get this into 4.4 because I don't have a good sense of whether/which real world scenarios will see the large build time regression I noticed in my (contrived) local experiments.

@typescript-bot perf test this

As far as I know, the perf suite doesn't cover build mode, so I don't expect any changes.


Error activating ISamplesAggregator

I'm trying to use SoundFingerprinting to fingerprint some audio streams, and am getting this error. I'm not messing around with the Ninject stuff at all. This is the only block of code I have dealing with it:
var hashes = await new FingerprintCommandBuilder()
            .BuildFingerprintCommand()
            .From(await GetSamples(request.Actor, request.SourceId, request.Index))
            .WithFingerprintConfig(config => config.Stride = new IncrementalRandomStride(512, 1024, 128 * 64, 0))
            .UsingServices(new NAudioService())
            .Hash();

Matching bindings:

  1) binding from ISamplesAggregator to SamplesAggregator

  2) binding from ISamplesAggregator to SamplesAggregator

Figured it out. Don't create multiple NAudioServices.


Support for different plural forms (feature request)

We have a custom resource with a non-s plural form (Lets say a Bison CRD) - the plural of Bison is Bison.
The API server etc all seem to handle this fine, but I can't see a tag to control this in code-gen at the moment. Right now I'm handling that by post-processing the generated code (sed -e /bisons/bison/) but this is suboptimal.

/kind feature

/remove-lifecycle stale
On Wed, 13 Mar 2019, 07:11 fejta-bot<EMAIL_ADDRESS>wrote:

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually
close.
If this issue is safe to close now please do so with /close.
Send feedback to sig-testing, kubernetes/test-infra and/or fejta
https://github.com/fejta.
/lifecycle stale
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/code-generator/issues/53#issuecomment-472119523,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAef3nJ_gpoZ_woQmLDBrB4PJAolt37aks5vV-3fgaJpZM4V6WD-
.

/reopen


Linter(?) recognises spaces around colon as error "(ERROR (symbol))"

Please forgive me as I am just a novice, I sincerely hope this isn't a waste of time due to my ignorance.
Basic informations

OS: [Windows 10]
VSCode version: 1.84.0
Extension version: v6.2.0

Further Information
I get a red line and error indication under variable declarations which have a space before the colon. The code still compiles without error, but the linter seems to not like it (not sure if I'm using 'linter' correctly here).
This is a minor issue because when I use autoformat in VS Code (Shift + Alt + F), the formatter adds a space around the colon in variable declarations. I get distracted by these red lines and have to keep deleting the spaces manually.
e.g.
new Handle:g_hFile = INVALID_HANDLE;

which looks good in VS Code, ('Handle' is in green, not blue) becomes
new Handle : g_hFile = INVALID_HANDLE;

Now with an error indicated and 'Handle' coloured in blue.
To Reproduce

Create a sp file in VS Code
Declare a variable with a space before the colon, e.g., new Handle:g_hFile = INVALID_HANDLE;
Observe red line and error

Expected behaviour
I presume there should be no error indicated, and the variable type should still be coloured green
Error messages
(ERROR (symbol))
Additional
I would like to know if I can make changes to the formatter settings JSON file to prevent spaces around colons in these variable declarations.

Thanks for the quick response. I just realised that I was wrong fundamentally - this does not compile. I must have been not saving the file before running the compiler... always something simple! So the bug is not 'displaying an error when there is none' - there is rightfully an error indicated (but I'm sure you spotted this).
I appreciate your workaround suggestion, think I will go with that so I don't have to delete spaces manually. The other workaround would be to not use auto formatter of course 😜


[Testing] [Go-gen] Fix illegal shadowing of ID in test

For some reason id was being shadowed as an int, which was leading to some illegal SQL statements being generated.

@jaylees14 I thought that would have been the case, but I was updating it for structs and thought I may as well update it here too. Tests are often where people look to understand how our stuff works

@jaylees14 I thought that would have been the case, but I was updating it for structs and thought I may as well update it here too. Tests are often where people look to understand how our stuff works

Absolutely - wasn't a dig at all, just thought I'd give some context - thanks for fixing 😄


Database modeling - experiment with loading GKS variation data into parquet and DuckDB

Submitter Name
Kyle Ferriter
Submitter Affiliation
Broad Institute
Project Details
We would like to load GKS data into a relational model for rich querying. We currently heavily use BigQuery in backend processes but would like something free and portable that can be distributed to users. Anyvar uses Postgresql with most data stored in JSON columns. We would like something easier to configure and run than Postgres, such as embedded, single file/directory database like SQLite, Rocksdb, Duckdb. We have looked a little bit at DuckDB, which can import/export Parquet and NDJSON files and can provide a SQL query interface over them with basically no configuration needed. A difficulty with NDJSON import is that the automatic schema inference fails when the data contains heterogeneous rows, which is the case for GKS data.
We are aiming for these deliverables:

Convert Variation NDJSON files to Parquet files using DuckDB. Determine whether there needs to be multiple sets of parquet files with different schemas or whether a big schema of union types is possible. (*not by adding a set of columns for each different type)
Load the parquet datasets into a DuckDB database file (based on the docs it sounds like the internal format is similar to parquet, but preloading it into a native database file may improve performance compared to loading parquet directories).
Provide notebooks demonstrating queries we anticipate to be common.

Stretch:

Look at table indexing. What storage overhead does adding indexes add? To what degree does it speed up common queries.
Write an Anyvar storage backend to use this.
Investigate the write performance. If this serves as a backend to a writeable API like Anyvar, how quickly can it write new records? Parquet is not known for write performance, as it is designed for analytics/warehousing. Is it good enough? Is writing rows a bottleneck?

Required Skills

Python
SQL
Experience defining relational data schemas

Semi-related, this was just posted today:

Pg_parquet: An extension to connect Postgres and parquet

https://news.ycombinator.com/item?id=41871068

pg_parquet is a PostgreSQL extension that allows you to read and write Parquet files, which are located in S3 or file system, from PostgreSQL via COPY TO/FROM commands

https://www.crunchydata.com/blog/pg_parquet-an-extension-to-connect-postgres-and-parquet


Remove Amethyst

The project ceased in 2022 and the domain has been taken over by a placeholder.

https://github.com/amethyst/amethyst


restoring without site backup / only with database backup

I am trying to restore CM Primary Site. I do not have a CM server and site backup and that's why  I do not have CD.latest folder. Is it possible to restore server if you have just DB?

Thanks @jamelia1303 for taking the time to share this with the Configuration Manager documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this:

Post on Configuration Manager forums. These are moderated by our community MVPs who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Premier team, such as a Premier Field Engineer or Technical Account Manager.

I'm closing this issue as there's nothing actionable for docs at this time.


Add OpenAI Priority Load Balancer for Azure OpenAI

This PR is only intended to showcase how the OpenAI Prioritized Load Balancer integrates.
This PR should NOT be merged.

Hi @pamelafox & @kristapratico,
This is how the OpenAI Priority Load Balancer integrates. Nevermind the hard-coded backend and the location of the backends list in this PR. I don't intend to ask for a merge, but this was the best way to give you an idea of the setup.
If you have two AOAI instances with the same model, you can plug them both in and should see load-balancing.

Cool! I made a few changes to the PR to make it a little easier to test out, by actually making the additional backend deployment, mind if I push them to the branch?
I think we should mention this option in the Productionizing guide, and if there are multiple customers wanting to use this approach, we could consider integrating it into main as an option.

Here are what my usage graphs look like during a load test btw:

Cool! I made a few changes to the PR to make it a little easier to test out, by actually making the additional backend deployment, mind if I push them to the branch?
I think we should mention this option in the Productionizing guide, and if there are multiple customers wanting to use this approach, we could consider integrating it into main as an option.

Hi Pamela, please do push! I very much welcome your expertise and improvements. If there are aspects of the 1.0.9 package itself that should/need to be improved, I'm all ears there, too, of course.
Thank you so much! I know this is extraordinary time spent.

Here are what my usage graphs look like during a load test btw:

Help me understand your test results, please. Are you hitting different backends or just different models?

@simonkurtz-MSFT Those graphs were for two different OpenAI instances in the same region.

@simonkurtz-MSFT Could you send a separate PR adding a mention of this approach to https://github.com/Azure-Samples/azure-search-openai-demo/blob/main/docs/productionizing.md#openai-capacity with a link to this PR? You could contrast when someone might opt for this over ACA/APIM (presumably cost/complexity).

Hi @pamelafox, could I trouble you for another review of this PR, please? Thank you very much for all your help!


Fix misplacement of unused comment

I actually placed it intentionally at the bottom, because I thought that's were it is most easily seen. But okay, thanks!


Add Weak and Unowned types as storage types of a Wrapped instance

The Problem with Hashable would be to differentiate between a data-based hash and a reference based hash.
We could make any Weak, Unowned hashable where the Wrapped type conforms to Hashable or we could add Hashable with the ObjectIdentifier like ObjectIdentifier(nonNilValue).hashValue.
That is a problem, you could expect to hash by the memory address because they are references or you could expect to hash by the data represented in the instance itself.
I'm not sure if we should add a default conformance for Hashable.

I was only referring to making Weak and Unowned Hashable in case the Wrapped is Hashable, nothing more. I don't think anything speaks against this, right?


build: upgrade dependencies

Node 14.x
npm audit upgrades
simplified docs website for now (due to breaking changes and low priority to fix)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 13 (100.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 99.634%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
831

Relevant Lines:
833

💛  - Coveralls


#2468 fetch identity on mounted()

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #2468
[x] Added fetch identity on mounted

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Before:
https://user-images.githubusercontent.com/734428/158045442-ac0a4467-df84-4a2e-bee1-88ca93663072.mov
After:
https://user-images.githubusercontent.com/734428/158045847-4938d5f1-2fe3-45df-9bb2-6cb12a77993d.mov

identityOf cries in the corner on the line 120.
Your PR makes request on RPC node each time the component mounts :) This feature does not leverage cache at all

updated, to use identityOf instead

Just 🔇 `console.log` and we are good to go
(Tested)

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 125.13 USD/KSM ~ 0.799 $KSM
🧗 DY4SQF2iD456tH89aQtz5wv1EV3BbSW8wKKuMcwbmXaj1pM
🔗 0x3cc0ff8f370dacc6aab97493ee130dfd26436a9daa4363241e340456bfc74bce
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


More Extensions

What do you think about
https://marketplace.visualstudio.com/items?itemName=aaron-bond.better-comments
and
https://marketplace.visualstudio.com/items?itemName=CoenraadS.bracket-pair-colorizer-2

Bracket Pair Colorizer is already there, and better comments was eschewed in favor of TODO tree which provides similar functionality but also a viewer pane to organize the comments as a nice project workflow.
Thank you for suggesting!


Remove / in between host and endpoint

https://github.com/rackerlabs/scantron/blob/2b95f7c7c827eaea20baa47133a376f29eef0d17/scantron_api_client/scantron_api_client.py#L68
Currently the nginx logs look like:
GET //results/41?file_type=json HTTP/1.1"

should be
GET /results/41?file_type=json HTTP/1.1"

https://github.com/rackerlabs/scantron/pull/195


Detect clicked node of  GLTF model

Hi,
I have used GLTF model which shows the human body. I want to detect which part of model is clicked on single tap like hand, leg, back, etc. In short I want to fetch clicked child node name from model.
I have attached a model file and screenshot of model.
OS:Android
man.zip

Let's use the similar discussion topic for follow-up: https://github.com/google/filament/discussions/7745


JSON import and export messes up column ordering

data = tablib.Dataset(('John', 'Doe', '10'), headers=['fname', 'lname', 'age'])
print(data, data.json, tablib.import_set(data.json), sep='\n\n##############\n\n')

fname|lname|age
-----|-----|---
John |Doe  |10

##############

[{"fname": "John", "lname": "Doe", "age": "10"}]

##############

fname|age|lname
-----|---|-----
John |10 |Doe

This was actually detected before in the failing tests in #149 and it was fixed by fixing the tests.  I would expect users to at least be warned about this, if it cannot be fixed with reasonable effort.

In Python 3.7 and up, key insertion order is preserved, maybe this will solve this issue for good?

I'm quite confident that this should not happen on recent Python.