chore: release 0.141.0

:robot: I have created a release *beep* *boop*
0.141.0
google/cloud-asset 1.3.0
Features

bigquery destination, output result, v1beta1 formatting (#3398) (5d5460c)

google/cloud-bigtable 1.6.1
Bug Fixes

adds quota project doc to language, debugger and bigtable (#3435) (ef88c44)

google/cloud-debugger 1.3.1
Bug Fixes

adds quota project doc to language, debugger and bigtable (#3435) (ef88c44)

google/cloud-firestore 1.17.0
Features

add parent() method for CollectionReference (#3269) (d3460b8)
add support for query limit-to-last (#3448) (b3452e9)

google/cloud-language 0.24.1
Bug Fixes

adds quota project doc to language, debugger and bigtable (#3435) (ef88c44)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/google-cloud-php/releases/tag/v0.141.0


Replace all instances of sed with strings.Replace(r) function

Problem: As described here OSX version of sed behaves a little different than the Linux variant which can cause certain abnomilities in the code. Therefore we need to replace usage sed with a golang native solution.
Example solutions:
https://gist.github.com/tdegrunt/045f6b3377f3f7ffa408
https://gist.github.com/dallarosa/b58b0e3425761e0a7cf6
https://socketloop.com/tutorials/golang-read-a-text-file-and-replace-certain-words

Would it not be better to properly parse the YML files as a data structure rather than sed or String replace methods which are more error prone

@frezbo Since the cli is parsing multiple type of files therefore we will have to use the String.Replace(r) function

@MuellerMH This is not needed anymore as we are using templates now. This can be closed.


[Feature]: Deleting steps up to a given point

Description
In HPC environments we often have job allocations (e.g. 10 days). It is possible that a save is triggered on 9 days and 20 hours (e.g. at step 100). After saving, logging just continues until the 10 days are over (e.g. until step 105). Then, we continue with a new job from that latest saved checkpoint at step 100. But that leads to a discrepancy: wandb logging is at step 105 already, but we restart from step 100 - so the graph will be messed up a bit. It would therefore be incredibly useful if some tooling exists to remove data points in a given range of steps.
Suggested Solution
A CLI interface that, given a project name and run as well as a steps-range, allows us to remove all data points within that range. From the outside looking in, this seems straightforward but I'm sure there are technical reasons why this is more difficult to do.

Hi @BramVanroy! Thank you for writing in! I will go ahead and submit the feature request for you to our engineering team.

Hi @BramVanroy, please find more discussion on the feature backlog here: https://github.com/wandb/wandb/pull/7078#issuecomment-2030175571
In short, this is on our radar as a high-value feature. We already have one part complete, but have some remaining work implement.

@sephmard That's looking great! Any chance that this will also be controllable by environment variables? That would make life easier in HPC environments where passing around env vars might be easier than setting vars in scripts. Perhaps WANDB_RESUME can be update to align with this behavior, or a new WANDB_RESUME_RUN and WANDB_RESUME_STEP can be added?


EdgeX controller and EdgeX components work in different namespace

From the demo video, I have seen that EdgeX controller is deployed in the edgex-system namespace, but EdgeX components are deployed in default namespace. Would you agree with  that deploy EdgeX components in edgex-system namespace is more reasonable?

@lwmqwer

The edgeX is namespace Custom resource, in this example I do not set the namespace of EdgeX so it is defualt, but used can set the EdgeX CR in some namespace and its related service and unitedeployment in the same name space. This is same as the kube-controller-manager in the kube-system namespace but pod can in any namespace.


fix(github.go): supply token via authz header instead of query param

Updates api usage to supply the access token in the form of an Authorization header as opposed to a query parameter, as informed by GitHub:
On February 12th, 2020 at 16:28 (UTC) your personal access token (general curl token) using Go-http-client/1.1 was used as part of a query parameter to access an endpoint through the GitHub API:

Please use the Authorization HTTP header instead, as using the `access_token` query parameter is deprecated and will be removed July 1st, 2020.

Tested via building the cli (make) and issuing an info command, e.g. ./github-release info -u org -r repo.  (Command works and no email response from GH received.)

FWIW, fixed this on my fork at github.com/meterup/github-release. Feel free to try it out.

See discussion in #97

Thanks for the PR, this should be fixed now on master!


fix fix session var memtable on sink for bulk load

Proposed changes
The added line had been there, but was deleted somehow.

run buildall

run p0

run p0

run buildall

run external

run buildall


Update kennethreitz.zsh-theme

:sparkles: :cake: :sparkles:

?

Hi @kennethreitz ! sorry for the delay. happy to pull this through, hope all is well!

✨ 🍰 ✨


gym ignores Dir.chdir

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
I change the working directory with Dir.chdir but gym can't find the workspace file :-(
After Dir.chdir I added sh("pwd") and it prints the right directory path like:
[11:46:54]: $ pwd
[11:46:54]: ▸ /Users/ralphbergmann/development/flutter/foo/store

But gym fails with:
[11:47:38]: Workspace file not found at path '/Users/ralphbergmann/development/flutter/foo/ios/Runner.xcworkspace'

Environment
✅ fastlane environment ✅
Stack

Key
Value

OS
10.13.4

Ruby
2.3.3

Bundler?
false

Git
git version 2.15.1 (Apple Git-101)

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.13.4 (17E202)

Ruby Lib Dir
/System/Library/Frameworks/Ruby.framework/Versions/2.3/usr/lib

OpenSSL Version
LibreSSL 2.2.7

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.4

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE

fastlane files:
`./fastlane/Fastfile`
fastlane_version "2.33.0"

platform :flutter do

    lane :alphaBuild do

        # Return the number of commits in current git branch
        build_number = number_of_commits()

        Dir.chdir "../core" do
            sh("pwd")
            sh("flutter", "packages", "get")
            sh("flutter", "clean")
        end

        Dir.chdir "../store" do

            sh("pwd")
            sh("flutter", "packages", "get")
            sh("flutter", "clean")
            sh("flutter", "build", "ios", "--release", "--no-codesign", "--build-number=#{build_number}")

            # Easily sync your certificates and profiles across your team using git
            match(
                type: "enterprise",
                team_id: "foobar",
                git_branch: "foobar",
                app_identifier: "foobar"
            )

            # Easily build and sign your app using gym
            gym(
                workspace: "ios/Runner.xcworkspace",
                scheme: "Jenkins",
                silent: true,
                clean: true,
                configuration: "Release",
                export_method: 'enterprise'
            )

        end

    end

end

`./fastlane/Appfile`
apple_id<EMAIL_ADDRESS># Your Apple email address

fastlane gems

Gem
Version
Update-Status

fastlane
2.97.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

did_you_mean
1.0.0

slack-notifier
2.3.2

atomos
0.1.2

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.4

xcodeproj
1.5.7

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

plist
3.4.0

addressable
2.5.2

multipart-post
2.0.0

word_wrap
1.0.0

public_suffix
2.0.5

tty-screen
0.6.4

tty-cursor
0.5.0

tty-spinner
0.8.0

babosa
1.0.2

colored
1.2

highline
1.7.10

commander-fastlane
4.4.6

faraday
0.14.0

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.1

mini_magick
4.5.1

multi_json
1.13.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
1.0.0

bundler
1.16.1

faraday_middleware
0.12.2

naturally
2.1.0

simctl
1.6.3

uber
0.1.0

declarative
0.0.10

declarative-option
0.1.0

representable
3.0.4

retriable
3.1.1

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

logging
2.2.2

jwt
2.1.0

memoist
0.16.0

os
0.9.6

signet
0.8.1

googleauth
0.6.2

httpclient
2.8.3

google-api-client
0.21.2

emoji_regex
0.1.1

io-console
0.4.6

excon
0.62.0

CFPropertyList
3.0.0

libxml-ruby
2.9.0

json
2.1.0

generated on: 2018-06-07

It looks the apk option of the hockey command ignores Dir.chdir too.
hockey fails with:
[12:04:27]: Couldn't find apk file at path 'build/app/outputs/apk/app.apk'

https://docs.fastlane.tools/advanced/#directory-behavior


Use CI to ensure that features list stays up to date

This list is very helpful, but seems easy to forget to update when we add new feature flags.
We should use CI to ensure that they stay in sync.

Rust-analyzer handles this style of problem by having some code which runs in cargo test. This would also be a reasonable solution.


Private members overwriting each other (prototype version)

With the "prototype" version, the following program prints 2 although it should print 1:
class Type1() {
    Integer i = 1;
    shared void test() { print(i); }
}
class Type2() extends Type1() {
    Integer i = 2;
}
void run() {
    Type2 tst = Type2();
    tst.test();
}

That's because both members i get mapped to a variable this.i of the same JS object.

Member names for non-shared members now have "$TypeName" appended to them in the JS code. But this solution probably needs to be revised later because it likely doesn't work if the derived class has the same name as the base class, only in a different package.

Yes, great, that's the right solution. Thanks.


modules and configs in pwd should be ignorable by the Ceylon CLI

By analogy, with bash:

Problem: you might have a $PATH with . (bad idea), and an ls command in pwd. Scripts or commands might use the wrong ls!
Workaround: don't have . in $PATH, or qualify ls, as in /bin/ls

with ceylon:

Problem: You may wish to run ceylon commands from within a directory that is used for development of Ceylon code that is unrelated to the command you are running. This directory might have .ceylon/config and/or ./modules.
Workaround: ???

In both cases, this is a also a potential security issue, since you might run things from /tmp, etc.
Demo
From an empty directory, assuming you have Ceylon formatter installed:
$ ceylon format --help # works
$ mkdir .ceylon
$ echo -e "[defaults]\noverrides=overrides.xml" > .ceylon/config
$ ceylon format --help
ceylon help: Could not load module 'ceylon.formatter/1.3.0' because: java.lang.IllegalArgumentException: No such overrides file: /private/tmp/er/overrides.xml

If I understand correctly, the problem involves and is limited to:

.ceylon/config settings
./modules default repo

It would be nice to have a ceylon option like --ignore-pwd-configs. It would also be nice to have a better approach to trusting pwd, but I'm not sure what that would be.

@quintesse I'm associating this with 1.3.0 just in case a quick fix is possible. This is impacting a project I'm working on, and I don't have a great solution yet, other than perhaps creating a new temp directory to serve as pwd.

I understand your issue but I don't think this is something we should think about hours before making a release. Especially because this would affect all commands in all kinds of ways. And quickly adding an option to perhaps want to remove it in the next version doesn't seem like a good idea.
Somehow we need to figure out when and how to apply configurations. Perhaps a way for Ceylon commands to specify that they don't care about local projects (and would therefore be unaffected by the local config and repositories). But I think it should be a responsibility of the command, not of the user.

Ok, makes sense. Thanks for taking the quick look.

Btw, perhaps as a work-around you can trick ceylon by using a combination of --cwd=dummy and absolute paths for any path arguments you want to pass to the command

Cool, I'll try.


Wrap nlohmann::json in our namespace

As per https://github.com/alpaca-core/ac-local/issues/177#issuecomment-2454378703

using our fork here: https://github.com/alpaca-core/nl-json

our fork can't reasonably deal with the include directory issue... moving to full splicing


Master

Fix for issue #245: Lagarto mishandles naked ampersands in attributes and text

Just started reviewing :):rocket:


TSServer: Rename errors on non-existent property

From https://github.com/Microsoft/vscode/issues/59426

TypeScript Version:  3.2.0-dev.20180926

Search Terms:

rename
tsserver

Code
In a TS file:
class Foo  {
    constructor() {
        this.bar = 1;
    }
}

Trigger a rename on bar
bug
The rename request fails with:
[Trace  - 11:46:15 AM] Sending request: rename (247). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/san/a.ts",
    "line": 3,
    "offset": 15,
    "findInStrings": false,
    "findInComments": false
}
[Trace  - 11:46:15 AM] Response received: rename (247). Request took 2 ms. Success: false . Message: Error processing request. Cannot read property 'start' of undefined
TypeError: Cannot read property 'start' of undefined
    at IOSession.Session.toLocationTextSpan (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122830:65)
    at IOSession.Session.mapRenameInfo (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122428:242)
    at IOSession.Session.getRenameLocations (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122423:39)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:121655:61)
    at /Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123133:88
    at IOSession.Session.executeWithRequestId (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123124:28)
    at IOSession.Session.executeCommand (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123133:33)
    at IOSession.Session.onMessage (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123153:35)
    at Interface.<anonymous> (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:124417:27)
    at emitOne (events.js:116:13)
    at Interface.emit (events.js:211:7)
    at Interface._onLine (readline.js:282:10)
    at Interface._normalWrite (readline.js:424:12)
    at Socket.ondata (readline.js:141:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)
[Trace  - 11:46:15 AM] Sending request: getCodeFixes (248). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/san/a.ts",
    "startLine": 3,
    "startOffset": 14,
    "endLine": 3,
    "endOffset": 17,
    "errorCodes": [
        2339
    ]
}
[Error  - 11:46:15 AM] 'rename' request failed with error.
Error processing request. Cannot read property 'start' of undefined
TypeError: Cannot read property 'start' of undefined
    at IOSession.Session.toLocationTextSpan (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122830:65)
    at IOSession.Session.mapRenameInfo (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122428:242)
    at IOSession.Session.getRenameLocations (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:122423:39)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:121655:61)
    at /Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123133:88
    at IOSession.Session.executeWithRequestId (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123124:28)
    at IOSession.Session.executeCommand (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123133:33)
    at IOSession.Session.onMessage (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:123153:35)
    at Interface.<anonymous> (/Users/matb/projects/san/node_modules/typescript/lib/tsserver.js:124417:27)
    at emitOne (events.js:116:13)
    at Interface.emit (events.js:211:7)
    at Interface._onLine (readline.js:282:10)
    at Interface._normalWrite (readline.js:424:12)
    at Socket.ondata (readline.js:141:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at Pipe.onread (net.js:594:20)

@RyanCavanaugh We hit this one pretty often in my testing. I think a fix should go into 3.1.5

3.1 fix is in cd1803fdf8fe7061fbe97a154014d0238ccf5551


python3.pkgs.mailman is not a Python module

python3.pkgs.mailman is not a Python module and python310.pkgs.mailman actually uses Python 3.9!???
It can also not be added to python3Packages with the correct Python version cause it overrides sqlalchemy.
Thus mailman and the packages depending on it need to be removed from python-packages.nix.
cc @mweinelt @jonringer

agreed

cc @lheckemann


[cPP CHANGE] Table 2 filled out

What is the change request for the cPP? Please describe.
The Security Objectives Rationale table is blank at this point. This should be filled in.

As there are no Security Objectives for the TOE, I have removed the table entirely. Will defer to the iTC in next call for inclusion.

Accepting Brandon's resolution.


fix(types): utility type removed from RN 0.71

Since react-native v0.71 bundles typescript types, some private types are not exported: https://github.com/facebook/react-native/blob/v0.71.1/types/private/Utilities.d.ts
Also, NativeMethodsMixin has been deprecated, replaced by NativeMethods.
resolve: #188

This is blocking my app's RN upgrade - can we get it merged?

any update on when this will be merged? :)


Benchmarks

Hey really cool project!
I find this repo after searching about FDTD from this topic. I am searching for a (fast) FDTD engine to play a bit. I was wondering if you had any benchmark between the solver commonly available (aka MEEP or fdtdz). Especially considering the scale multi-CPUs or multi-GPUs.
Thanks and really nice project! :)

Thanks for reaching out! I haven't done benchmarks yet. We simply use Julia's diff and broadcasted array arithmetic with GPU calls (if used) compiled by CUDA.jl. So it's probably comparable to Meep but slower than fdtdz.
What system scale are you thinking? For inverse design of typical components less than 10 x 10 um even a single machine is sufficient. From there one would extract the component S parameters used for system simulation. So I don't really see a big need for systems level FDTD which fdtdz caters to except in some 3d metamaterials cases - I could be wrong though.
Our focus is on ease of use and flexibility wrt inverse design. We're bare bones AD compatible with Zygote.jl w/ no custom adjoints. There's some broken code now - we're fixing our examples and planning a first stable release next week. Afterwards I'm happy to do a quick presentation (live meeting or recorded) for your team if desired.

In my academic work, I mainly play with photonic crystals. Depending on the size, I think you can assimilate them as a meta surface. Or if you try to integrate them, you can end up with large devices...
Right now, I only investigate the possibilities. But I will keep an eye on the next release!
Thanks for your complete response!

Got it that makes sense. There's also the commercial package Tidy3D which can also handle massive scale but doing AD with backend hidden in the cloud is maybe not easy.. I pushed a new release and documented the tutorials on project site. It's still in alpha and likely buggy - I'm happy to help write a test script if you wanna try something out. Using Float16 on gpu should help

I thought Meep had a GPU variant but officially no. Assuming Meep stays CPU only, our GPU implementation should be faster, though not as fast as a bespoke GPU implementation like fdtdz which is more feature restrictive.


define a empty enum in RP: adhybridhealthservice

I find some definitions like:
          {
            "name": "nextPartitionKey",
            "in": "query",
            "description": "The next partition key to query for.",
            "required": true,
            "type": "string",
            "enum": [
              " "
            ],
            "x-ms-enum": {
              "name": "nextPartitionKey",
              "modelAsString": true
            }
          },

It is not a valid swagger schema if enum is empty.

@chenjianfei2017 Which service?

rp: "adhybridhealthservice " in title


⚠ File access (beta) has degraded performance

In 0e5cce1, File access (beta) (https://beta.fyle.uk/file/fish) experienced degraded performance:

HTTP code: 200
Response time: 3036 ms

Resolved: File access (beta) performance has improved in f9b0772.


[9.x] Enable batch jobs delay for database queue

[before]

batch jobs was ignoring delay time when inserting into database.

[after]

batch jobs delay time will be considered and stored into database.

use App\Jobs\ImportCsv;
use Illuminate\Bus\Batch;
use Illuminate\Support\Facades\Bus;

$batch = Bus::batch([
    (new ImportCsv(1, 100))->delay($delay),
    (new ImportCsv(101, 200))->delay($delay)
])->dispatch();

@civilcoder55 thanks for this update. Does this work with setting the delay in the job constructor as well? I've no been able to bump to Laravel 9 yet, but this will be a nice improvement.

Hi, @georanma
sorry for the delay in replying!
if you mean delay in the job constructor like this, yes you can do it
<?php

namespace App\Jobs;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;

class SampleJob implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    private $data;

    /**
     * Create a new job instance.
     *
     * @return void
     */
    public function __construct(array $data)
    {
        $this->data = $data;
        $this->delay(now()->addMinute(1));
    }

    /**
     * Execute the job.
     *
     * @return void
     */
    public function handle()
    {
        //
    }
}

@civilcoder55 awesome. Thank you again.


ENH: fixed failing test with macOS Rosetta emulation by increasing bu…

…ffer size
Increased buffer size from 1 MiB to 256 MiB.
Before this, on an arm64 Mac, this test would usually fail when run under Rosetta emulation. No idea why. Probably some caching effect, or other effect of emulation.  Making the buffer bigger seems to allow it to pass always.

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@dzenanz @thewtex this is the only failng test under Rosetta BTW!

Yeah, basically :)


Automate Project Account (Unity-Sips-Test) SPS Load Balancers Deployment

The necessary load balancers that connect project-level (i.e. Unity-Sips-Test) API Gateway/VPC-link pathways to SPS resources residing in the private subnet need to be created. The SPS team will manage the creation of these, as part of their terraform code in the SPS github repo(s).  This terraform will automatically get run by the U-CS deployment infrastructure, but the details of the VPC links, and load balancers will be managed by the SPS developers.
NOTE: When creating VPC links and/or load balancers, we may want to store some info about endpoints in SSM.  (see also:  https://github.com/unity-sds/unity-cs/issues/202)
Acceptance Criteria:

[ ] [https://github.com/https://github.com/unity-sds/unity-cs/issues/200] is completed
[ ] [https://github.com/https://github.com/unity-sds/unity-cs/issues/205] is completed
[ ] Inspection into both the Github Action logs, and the AWS console prove that the project deployment led to the automatic creation of the load balancers

Update to this after conversation:
The OIDC provider issue is a known MCP constraint we've ran in to before. A solution is being worked on with MCP, but it's sort of up in the air right now. I'm going to wait to look into more solutions until we have more information on the timeline for resolving this with MCP. The service teams have a working, if not ideal, solution in the form of Classic Load Balancers provisioned through the default eks controller for now, so this shouldn't block any body from getting their work done.


Some findings from go vet

# github.com/Fraunhofer-AISEC/cmc/attestationreport
attestationreport/validationreport.go:97:2: struct field tag `json:appName` not compatible with reflect.StructTag.Get: bad syntax for struct tag value
attestationreport/validationreport.go:98:2: struct field tag `json:verificationName` not compatible with reflect.StructTag.Get: bad syntax for struct tag value
# github.com/Fraunhofer-AISEC/cmc/provclient
provclient/provclient.go:66:6: github.com/sirupsen/logrus.Warn call has possible formatting directive %v
# github.com/Fraunhofer-AISEC/cmc/tpmdriver
vet: tpmdriver/tpmdriver_test.go:69:14: undeclared name: GetTpmInfo
# github.com/Fraunhofer-AISEC/cmc/attestedtls
attestedtls/tpmkey.go:146:3: github.com/sirupsen/logrus.Error arg err.Error is a func value, not called

It probably makes sense to include go vet ./... in the CI build as well.

Fixed findings, added go vet ./..., go fmt ./... and go test -v ./... to CI build


feat: Update poddisruptionbudget name

Fixes #
Description
Update the name of the pod disruption budget to be same as other resources.
How was this change tested?
This PR doesn't have any major changes. Testing done in locally to make sure there is no errors.
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

Release Note

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This has the potential to break current users if they were relying on the configurable PDB name for some reason.  We could make this change later, but not until we do a breaking release. There are some upcoming helm changes that we were planning on making for specifying images and tags. We could revisit later.


If the Version string is empty, the version section should not appear in usage output

If the Version string is empty, then the version section should not be displayed at all in the usage output. Currently if the version string is empty, the usage output prints "VERSION:" followed by an empty line.
It would be up to the user to set HideVersion=true to remove the now-superfluous option when they define Version=""
(I was originally going to suggest that the default be an empty string as well, but that could have unwanted side-effects on downstream projects. Instead, one would simply empty the Version string to cause it to be hidden from the usage output, which should not have side-effects).

I've submitted pull request #250 to address this issue (note that this pull request contains the branch for pull request #249 as well)

Closed via #251; thank you!

How come when I use this package it still shows the version?

Are you seeing the VERSION: info in the help output or elsewhere?

Yes, in the help output. On second look, I am assuming that PR only affects it not in the help output?

After getting the pointer from NewApp (e.g., "app"), did you define app.Version as ""?

In addition, if you don't want the option named "version" working nor displayed in the help, you should also set app.HideVersion = true (as noted in the PR above). Then -v will throw an error and -h will not show a version number nor a version global option.


Error: Benchmark configuration file path is not set

when execute   :    caliper launch master \

--caliper-workspace ~/caliper-benchmarks \
--caliper-benchconfig benchmarks/scenario/simple/config.yaml \
--caliper-networkconfig networks/fabric/fabric-v1.4.1/2org1peergoleveldb/fabric-go.yaml

then wrong as title

@dretrtg
Can we have more information please?

version of caliper used
platform run on
actual error returned

hi,
i have the same  issue how can i resolve it?

hi,
i have the same  issue how can i resolve it?

am facing the same issue, using caliper 0.3.2 on ubuntu 22.04
has anyone solved it yet?


added the ability to rotate the swipe buttons 180° to accommodate for…

… another package that flips the table view 180 to load from the bottom
https://github.com/marty-suzuki/ReverseExtension

This solution isn’t complete, nor something I would accept as it’s architected. If you want to do this you can do it via the SwipeAnimatorDelegate where you can access the UIButton where you can transform there.

Aha! I see how to do that now! Thank you.


simp error "unknown metavariable"

@[simp] theorem liftOn_mk (a : α) (f : α → γ) (h : ∀ a₁ a₂, r a₁ a₂ → f a₁ = f a₂) :
    Quot.liftOn (Quot.mk r a) f h = f a := rfl

theorem eq_iff_true_of_subsingleton [Subsingleton α] (x y : α) : x = y ↔ True :=
  iff_true _ ▸ Subsingleton.elim ..

section attribute [simp] eq_iff_true_of_subsingleton end

@[simp] theorem PUnit.default_eq_unit : (default : PUnit) = PUnit.unit := rfl

example : (default : PUnit) = x := by simp
-- unknown metavariable '?_uniq.827'

Reported on Zulip.

Is liftOn_mk an actual [simp] theorem in Mathlib?
It is tried everywhere since we index things modulo reducibility.

It is, and was just ported to mathlib4, in https://github.com/leanprover-community/mathlib4/pull/559. I'll go have a look at where it is actually used in mathlib3.

I haven't finished recompiling, but it looks like it's an easy fix to remove the @[simp] in mathlib. We'll need to write a new linter to catch these, however!

Sorry, actually, I don't understand why this lemma would be a problem. The LHS still has head Quot.lift.

Sorry, actually, I don't understand why this lemma would be a problem. The LHS still has head Quot.lift.

@semorrison Lean 4 is computing the whnf modulo reducible constants. The term reduces to f a.
It seems this is going to be a big issue for the Mathlib port. We should probably change Lean 4, and only reduce nested terms when creating the index.
cc @digama0

Unfolding reducibles is indeed a change from lean 3, but I'm not yet convinced we can't just adapt to it as long as the new behavior is strictly better. One area where it is problematic is that simp is very unwilling to unfold reducible constants when asked to, because it sees right through them and hence doesn't see anything that needs unfolding. For the most part, if the LHS and RHS are equal up to reducible unfolding then I think we can just safely remove the @[simp] attribute on them. It would be nice to get a warning about this but I think a mathlib linter can do the job, but it does seem to be a regression that
@[simp] theorem liftOn_mk (a : α) (f : α → γ) : f a = f a := rfl

(which is how lean sees this lemma) is not an error, like it would be in lean 3 because the lemma trivially loops and also starts with a metavariable. (EDIT: actually this isn't an error in lean 3 either. It probably should be...)

I'm not yet convinced we can't just adapt to it as long as the new behavior is strictly better.

Not sure it is better. It seems to make it harder for users to control which [simp] theorems are applied. For example, suppose we have a reducible constant f ... := g ..., any simp theorem we add for f will also be applied to g applications. It seems we should disable reduction at least for the head symbol.
I will tag this issue with dev meeting.

Not sure it is better. It seems to make it harder for users to control which [simp] theorems are applied. For example, suppose we have a reducible constant f ... := g ..., any simp theorem we add for f will also be applied to g applications.

At least as far as near term advice for mathlib porting, I will be telling people not to tag simp theorems which have reducible definitions in them, and checking this in the simpNF linter. It just doesn't have the desired behavior - either firing on the wrong things or not firing on the right things - and this is because reducible definitions are supposed to be transparent to tactics like simp and rw, or at least that's what I tell people.


fixed a typo

Describe your PR, what does it fix/add?
it fixes a typo
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
I fixed a typo
Is it ready for merging, or does it need work?
it is ready

based

based on a typo

not a typo

the other ones are capitalized


Add "text snippets" to stats

= Sentences recorded.
Can it be performant?

This will burden the DB - since requires join against all segments.
To accomplish this properly - we need to create a post-process job to decorate an "uploaded" session with segment count.
This should also support the "discarded text segment" feature.
Deprioritizing.


Assertion from Http2Stream::main_event_handler into HttpSM

New assertion:
#0  0x00002b4d9310a337 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:55
#1  0x00002b4d9310ba28 in __GI_abort () at abort.c:90
#2  0x00002b4d90b167d1 in ink_abort (message_format=0x2b4d90b840e8 "%s:%d: failed assertion `%s`") at ink_error.cc:99
#3  0x00002b4d90b1380e in _ink_assert (expression=0x970b1d "pending_action == nullptr", file=0x9701d6 "HttpSM.cc", line=6800) at ink_assert.cc:37
#4  0x00000000006ed834 in HttpSM::kill_this (this=0x2b4db0778000) at HttpSM.cc:6800
#5  0x00000000006dc586 in HttpSM::main_handler (this=0x2b4db0778000, event=104, data=0x2b4d9929c490) at HttpSM.cc:2608
#6  0x000000000064a6b3 in Continuation::handleEvent (this=0x2b4db0778000, event=104, data=0x2b4d9929c490) at /usr/local/src/trafficserver/iocore/eventsystem/I_Continuation.h:190
#7  0x00000000007a0c79 in Http2Stream::main_event_handler (this=0x2b4d9929c200, event=104, edata=0x2b4d9900eb60) at Http2Stream.cc:160
#8  0x000000000064a6b3 in Continuation::handleEvent (this=0x2b4d9929c200, event=104, data=0x2b4d9900eb60) at /usr/local/src/trafficserver/iocore/eventsystem/I_Continuation.h:190
#9  0x0000000000946b23 in EThread::process_event (this=0x2b4d95806000, e=0x2b4d9900eb60, calling_code=104) at UnixEThread.cc:143
#10 0x0000000000946d75 in EThread::process_queue (this=0x2b4d95806000, NegativeQueue=0x2b4d989ffda0, ev_count=0x2b4d989ffd98, nq_count=0x2b4d989ffd9c) at UnixEThread.cc:182
#11 0x0000000000947015 in EThread::execute_regular (this=0x2b4d95806000) at UnixEThread.cc:242
#12 0x0000000000947457 in EThread::execute (this=0x2b4d95806000) at UnixEThread.cc:351
#13 0x0000000000945d9d in spawn_thread_internal (a=0x2b4d946c5d40) at Thread.cc:92
#14 0x00002b4d9249ce65 in start_thread (arg=0x2b4d98a00700) at pthread_create.c:307
#15 0x00002b4d931d288d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

(gdb) bt full
#0  0x00002b4d9310a337 in __GI_raise (sig=sig@entry=6) at ../nptl/sysdeps/unix/sysv/linux/raise.c:55
        resultvar = 0
        pid = 2693
        selftid = 2704
#1  0x00002b4d9310ba28 in __GI_abort () at abort.c:90
        save_stage = 2
        act = {
          __sigaction_handler = {
            sa_handler = 0x0,
            sa_sigaction = 0x0
          },
          sa_mask = {
            __val = {0 <repeats 16 times>}
          },
          sa_flags = 0,
          sa_restorer = 0x2b4d95806000
        }
        sigs = {
          __val = {32, 0 <repeats 15 times>}
        }
#2  0x00002b4d90b167d1 in ink_abort (message_format=0x2b4d90b840e8 "%s:%d: failed assertion `%s`") at ink_error.cc:99
        ap = {{
            gp_offset = 32,
            fp_offset = 48,
            overflow_arg_area = 0x2b4d989ff800,
            reg_save_area = 0x2b4d989ff740
          }}
#3  0x00002b4d90b1380e in _ink_assert (expression=0x970b1d "pending_action == nullptr", file=0x9701d6 "HttpSM.cc", line=6800) at ink_assert.cc:37
No locals.
#4  0x00000000006ed834 in HttpSM::kill_this (this=0x2b4db0778000) at HttpSM.cc:6800
        __FUNCTION__ = "kill_this"
#5  0x00000000006dc586 in HttpSM::main_handler (this=0x2b4db0778000, event=104, data=0x2b4d9929c490) at HttpSM.cc:2608
        jump_point = (int (HttpSM::*)(HttpSM * const, int, void *)) 0x6d4b7a <HttpSM::state_watch_for_client_abort(int, void*)>
        __FUNCTION__ = "main_handler"
        vc_entry = 0x2b4db07793c8
#6  0x000000000064a6b3 in Continuation::handleEvent (this=0x2b4db0778000, event=104, data=0x2b4d9929c490) at /usr/local/src/trafficserver/iocore/eventsystem/I_Continuation.h:190
No locals.
#7  0x00000000007a0c79 in Http2Stream::main_event_handler (this=0x2b4d9929c200, event=104, edata=0x2b4d9900eb60) at Http2Stream.cc:160
        lock = {
          m = {
            m_ptr = 0x2b4d990b39b0
          },
          locked_p = true
        }
        lock = {
          m = {
            m_ptr = 0x2b4d990b39b0
          },
          locked_p = true
        }
        __FUNCTION__ = "main_event_handler"
        e = 0x2b4d9900eb60
#8  0x000000000064a6b3 in Continuation::handleEvent (this=0x2b4d9929c200, event=104, data=0x2b4d9900eb60) at /usr/local/src/trafficserver/iocore/eventsystem/I_Continuation.h:190
No locals.
#9  0x0000000000946b23 in EThread::process_event (this=0x2b4d95806000, e=0x2b4d9900eb60, calling_code=104) at UnixEThread.cc:143
        c_temp = 0x2b4d9929c200
        lock = {
          m = {
            m_ptr = 0x2b4d990b39b0
          },
          lock_acquired = true
        }
        __FUNCTION__ = "process_event"
#10 0x0000000000946d75 in EThread::process_queue (this=0x2b4d95806000, NegativeQueue=0x2b4d989ffda0, ev_count=0x2b4d989ffd98, nq_count=0x2b4d989ffd9c) at UnixEThread.cc:182
        e = 0x2b4d9900eb60
#11 0x0000000000947015 in EThread::execute_regular (this=0x2b4d95806000) at UnixEThread.cc:242
        done_one = false
        sleep_time = 60000000
        e = 0x0
        NegativeQueue = {
          <DLL<Event, Event::Link_link>> = {
            head = 0x0
          },
          members of Queue<Event, Event::Link_link>:
          tail = 0x0
        }
        next_time =<PHONE_NUMBER>438563178
        delta = 17885706
        loop_start_time =<PHONE_NUMBER>136450775
        loop_finish_time =<PHONE_NUMBER>136448884
        prev_metric = 0x2b4d9590e410
        nq_count = 2
        ev_count = 3
        METRIC_INIT = {
          _loop_time = {
            _start = 0,
            _min =<PHONE_NUMBER>854775807,
            _max = 0
          },
          _events = {
            _min =<PHONE_NUMBER>,
            _max = 0,
            _total = 0
          },
          _count = 0,
          _wait = 0
        }
#12 0x0000000000947457 in EThread::execute (this=0x2b4d95806000) at UnixEThread.cc:351
        __FUNCTION__ = "execute"
#13 0x0000000000945d9d in spawn_thread_internal (a=0x2b4d946c5d40) at Thread.cc:92
        p = 0x2b4d946c5d40
#14 0x00002b4d9249ce65 in start_thread (arg=0x2b4d98a00700) at pthread_create.c:307
        __res = <optimized out>
        pd = 0x2b4d98a00700
        now = <optimized out>
        unwind_buf = {
          cancel_jmp_buf = {{
              jmp_buf = {47612273100544,<PHONE_NUMBER>177197686, 0, 2101248, 0,<PHONE_NUMBER>0544,<PHONE_NUMBER>771416694,<PHONE_NUMBER>436867702},
              mask_was_saved = 0
            }},
          priv = {
            pad = {0x0, 0x0, 0x0, 0x0},
            data = {
              prev = 0x0,
              cleanup = 0x0,
              canceltype = 0
            }
          }
        }
        not_first_call = <optimized out>
        pagesize_m1 = <optimized out>
        sp = <optimized out>
        freesize = <optimized out>
#15 0x00002b4d931d288d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111
No locals.

Oh, this is a duplicate of  #4119. Closing.


Warning when enabling Sourcemaps

Hi,
when enabling source maps (build.lib.sourcemaps) on the config file, vite is showing the following message:
(vite-plugin-lib-assets plugin) Sourcemap is likely to be incorrect: a plugin (vite-plugin-lib-assets) was used to transform files, but didn't generate a sourcemap for the transformation. Consult the plugin documentation for help

Tested on the playground and the message also shows there.
Is it possible to generate the sourcemaps ?

@marcelobotega Thanks for your feedback, I'll confirm the issue and fix it soon.

@marcelobotega I just fixed the issue and released v0.5.18.

As mentioned on the other issue, the fix work.

Why is this plugin transforming files in the first place? The transform hook could simply return undefined.

@AriPerkkio Thanks for your reminder, I wasn't aware that this could be done in this way


fix: fix migration dialog only appear after refresh

Summary

fix migration dialog only appear after refresh
fix migration cannot success

Does this close any open issues?
Closes #3255
Closes #3109

This ESLint warning should be resolved as well (missing dependency)
/home/runner/work/incubator-devlake/incubator-devlake/config-ui/src/hooks/useDatabaseMigrations.jsx
[16](https://github.com/apache/incubator-devlake/actions/runs/3151698589/jobs/5125983010#step:5:17)
  86:5  warning  React Hook useCallback has a missing dependency: 'migrationWarning'. Either include it or remove the dependency array  react-hooks/exhaustive-deps

@likyh The reason for a refresh being needed is so the local storage key is picked up. If the user navigates away or reloads the page the warning will happen, for a more instant warning we could just try to use a history.push to send the user to /dashboard or some alt route so the migration warning local storage key is detected.

But now it can not appear when first open. So I use a new way to get if migration needed.
Now it show appear when first request sent. (Usually is /versions)

I like your proposed refactored solution, however I think if we tried to add a history push and re-route the user to another location when we receive the migration warning my original design of using local storage key will work as expected.

You means add history push in https://github.com/apache/incubator-devlake/blob/main/config-ui/src/utils/request.js#L43 ?
https://github.com/apache/incubator-devlake/c732e7bddeb1f498873c6cc807e66772deb205e9/main/config-ui/src/utils/request.js#L43

@likyh The reason for a refresh being needed is so the local storage key is picked up. If the user navigates away or reloads the page the warning will happen, for a more instant warning we could just try to use a history.push to send the user to /dashboard or some alt route so the migration warning local storage key is detected.

But now it can not appear when first open. So I use a new way to get if migration needed.
Now it show appear when first request sent. (Usually is /versions)

I like your proposed refactored solution, however I think if we tried to add a history push and re-route the user to another location when we receive the migration warning my original design of using local storage key will work as expected.

I like your proposed refactored solution, however I think if we tried to add a history push and re-route the user to another location when we receive the migration warning my original design of using local storage key will work as expected.

Do you mean that add history push in https://github.com/apache/incubator-devlake/blob/main/config-ui/src/utils/request.js#L43 ?
https://github.com/apache/incubator-devlake/blob/c732e7bddeb1f498873c6cc807e66772deb205e9/config-ui/src/utils/request.js#L40-L50

We can't use a Hook call at this layer as it breaks core hook rules regarding usage, (Additionally, I left this utility Pure and it is Not a React component) what we need to do is find out where we can check the storage key first and send the appropriate signal.

@likyh If this new solution is working for now, I'll approve and we can try to fix the localStorage method later. It would be ideal to restore the timeout and fix the eslint dependency warning however.

@likyh Also remember to "npm run lint" locally to fix formatting and indentation etc..


Add clean-typescript

This adds clean-typescript config to the list. The details can be seen in the link :)

Thank you!


NPM Release

Hey, thanks for the library.
I noticed there are some changes in changelog that are yet to be released on NPM. Community is pretty eager to get updates-  at least autolinking: https://github.com/davodesign84/react-native-mixpanel/issues/188
Would it be possible to make an npm release?

1.1.10 released :)


Added ability to install non-standard primitives

This PR refactors how primitives are installed within classes in both interpreters to allow installing primitives for both undeclared methods and user-defined methods instead of just those marked with primitive.
This change should then allow to implement the Symbol>>#= primitive (which is not declared in the core lib), and should allow the symbol equality tests to finally pass.

This code in this PR has worked well from the testing I did this last couple of days.
This should be good to merge.


What should happen with trailing slash plus query?

Right now it looks like a normalization bug, unless these are in fact 2 different URLs:
> normalizeUrl('/index/?a=b')
'/index/?a=b'
> normalizeUrl('/index?a=b')
'/index?a=b'

Not sure. Removing trailing slash with a query string is not always a safe change.
For example, https://sindresorhus.com/feedback/?foo works, but https://sindresorhus.com/feedback?foo does not.

They aren't the same thing. If I'm not mistaken, some REST APIs tend to use /resource/ to get a list of the resources and /resource to get one resource.

This might be a case-by-case thing. Based on @SamVerschueren's observation, the existing functionality might be wrong too:
> normalizeUrl('example.com/index/')
'http://example.com/index'

If the server is...

a static file store

/dir-or-file/ is the same as /dir-or-file and can't co-exist (and the trailing slash normalization seems correct)

dynamic

/dir-or-file/ could be different than /dir-or-file and you can't normalize a trailing slash (because you don't know if it's different or not)

Removing a trailing slash regardless of a query is not always safe, as pointed out by @jotto. This library is already opinionated in regards to what is safe, though, so you may not always want the default settings.


Vague Error When Doing SFrame.save(...)

When trying to save an SFrame to a directory that does not exist, this exception gets thrown. It would be nice if there were more detail to make the specific problem more obvious to the user.
Traceback (most recent call last):
  File "/Users/user/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/data_structures/sframe.py", line 3036, in save
    self.__proxy__.save(url)
  File "cy_sframe.pyx", line 87, in turicreate._cython.cy_sframe.UnitySFrameProxy.save
  File "cy_sframe.pyx", line 90, in turicreate._cython.cy_sframe.UnitySFrameProxy.save
OSError: No such file or directory
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/Users/user/cvml-data-team/collection_projects/gurn-lb/create_split.py", line 10, in <module>
    split_uploader.build_trove_object()
  File "/Users/user/cvml-data-team/tools/trove/trove_uploader.py", line 685, in build_trove_object
    self.save_sframes_locally(sframes)
  File "/Users/user/cvml-data-team/tools/trove/trove_uploader.py", line 668, in save_sframes_locally
    sframe.save(save_path)
  File "/Users/user/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/data_structures/sframe.py", line 3043, in save
    raise ValueError("Unsupported format: {}".format(format))
  File "/Users/user/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/_cython/context.py", line 50, in __exit__
    raise exc_type(exc_value)
OSError: No such file or directory

I'm not able to reproduce this issue. The following code works:
import turicreate as tc
sf = tc.SFrame({'a': range(10)})
sf.save('/tmp/this/folder/does/not/exist')

It creates the directory structure and save the SFrame there.
@nseidl - please provide some simple, standalone code to reproduce the issue.

This fails, as expected, because /a does not exist. But, the error could be slightly more descriptive, such as "the directory where you'd like to save the sframe does not exist, or you don't have permission to create it"
import turicreate as tc
sf = tc.SFrame({'a': range(10)})
sf.save('/a/b/c')
Traceback (most recent call last):
File "/usr/local/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/data_structures/sframe.py", line 3062, in save
self.proxy.save(url)
File "cy_sframe.pyx", line 87, in turicreate._cython.cy_sframe.UnitySFrameProxy.save
File "cy_sframe.pyx", line 90, in turicreate._cython.cy_sframe.UnitySFrameProxy.save
OSError: No such file or directory
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/data_structures/sframe.py", line 3069, in save
raise ValueError("Unsupported format: {}".format(format))
File "/usr/local/anaconda3/envs/py36/lib/python3.6/site-packages/turicreate/_cython/context.py", line 50, in exit
raise exc_type(exc_value)
OSError: No such file or directory

Surprisingly, the error message is not actually generated by TuriCreate. It's generate by std::strerror. There might be some more going on here. We probably don't want to just blindly change the error message.


Support global options configuration

Added new global options:
GLOBAL OPTIONS
    -d, --versionFileDirectory=arg - Directory where to search for version file. (default: .)
    -t, --tagNamePattern=arg       - (default: v%s)

Example:
incr -d ./subprojects/web/ -t 'MyCustomTagPrefix/%s' npm patch
This will:

search for package.json and package-lock.json file inside subprojects/web/
the tage name will be something like MyCustomTagPrefix/2.3.4
the commit message will be MyCustomTagPrefix/2.3.4

Great, thanks for your PR!
I’d rather use the snake_case naming convention instead of camelCase, to avoid clashing with the rest of the variable names in the project.


Make header optional

Maybe it's just a personal preference but I prefer to use modules without their headers. I know which modules I use and where I placed them, therefore a header such as "RKI Daten" doesn't provide any value to me.
Could we add the option to disable it? This could be done for instance in the configuration by setting 'header: ""' in the config. If this is set no header is shown. Optionally, we could still show the header if the RKI numbers are not from today, which would only require to modify the if-clause in the getHeader function a bit.

Hello @sthuber90 ,
can you please varify, that the new possabilities in the configuration fit to your request?
I suggest for your config:
header: "",
config:{
showUpdateTimestampInHeader: false
}
Best

Thank you for the quick response and fix. I think this'll do. I was more thinking of something in the direction of:
getHeader: function () {
        // show date in header only if date exists and is not today's date
        if (this.dataRKI.lastUpdate
            && !((new Date(this.dataRKI.lastUpdate)).setHours(0, 0, 0, 0) === (new Date()).setHours(0, 0, 0, 0))) {
            return "RKI-Daten von " + this.dataRKI.lastUpdate;
        }
        // header: '' will hide the header
        return typeof this.data.header !== 'undefined' ? this.data.header : "RKI-Daten";
},

So with a configuration such as:
{
     module: "MMM-RKI-Covid19",
     header: '',
}

... you won't see the header, unless the date of the data is not today's date.
But to be clear: I think your solution is just fine as is


Add search bar

https://tomordonez.com/jekyll-search-bar/

if there is no pagination, we are able to search with CMD-F on the post list ... only titles but that is sufficient for now


How do I specify the include files location in portfile.cmake?

I'm trying to write a portfile for a library, let's call it the_library which is built with cmake. The include hierarchy for using this  library is:

a
b
the_library
c
d

With the standard portfile.cmake approach the ${VCPKG_ROOT_DIR}/installed/${TARGET_TRIPLET}/include directory after installation looks like this (listing only the new directories, not those of already previously installed libraries):

a
b
the_library
c
d

This is not nice, especially because there can be name conflicts with other installed libraries (actually one of the subdirectories is named boost which conflicts with the boost library). All the headers should go to ${VCPKG_ROOT_DIR}/installed/${TARGET_TRIPLET}/include/the_library which itself should then look like this:

a
b
the_library
c
d

A user would then have to #include <the_library/the_library/header.h> (or if needed also #include <the_library/a/header.h>.
Does anybody know how can I acheive this in a way that using the library with cmake will find the library and report the right include directory?

In general, all header files need to be archived in include. In order to resolve the conflict with the header files of other ports, we want to install the directory as include/$, and create sub-folders according to the directory structure in source.

That's exactly the reason I'm asking. If a library project (not yet supported by vcpkg) with its CMakeLists.txt doesn't respect this, how can I fix it in portfile.cmake?

A user would then have to #include <the_library/the_library/header.h> (or if needed also #include <the_library/a/header.h>.
Or the user could add include/the_library to their include path as well as include and then write #include <the_library/header.h> and #include <a/header.h>?

Yes, that's how I do it now, i.e. in a non CMake Visual Studio project I just set the vcpkg/include directory plus the vcpkg/include/nmos-cpp directory as additional include paths and the same for the lib directories. Then one can use nmos-cpp as if the project would be part of the nmos-cpp project.
With nmos-cpp and the vcpkg files from "my" nmos-cpp fork the outer CmakeLists.txt "adds" the nmos-cpp include directory and below then there is nmos, bst, detail, etc.
I looked for a way to do this without the need to change the nmos-cpp project. With the generic changes in the .cmake files it's working.
Still it would be interesting if vcpkg could do something similar.

nmos-cpp project is effectively a package of several smaller co-developed libraries or modules like nmos, mdns and so on (plus some extensions to existing libs like boost and cpprestsdk). Adding an extra project directory might solve very basic filename conflicts with other 'ports', but any such conflict is likely to indicate greater conflicts in C++ symbol names, which simple file-system reorganisation cannot hope to solve.

I see two issues here:

nmos-cpp doesn't export only one include directory, but more than one
some of the sub-include directories use names of other common libraries like boost or cpprestsdk

I don't see conflicts in C++ symbol names, because if they would be existing, these errors would already pop-up when building nmos-cpp in the classical way, because boost, cpprestsdk and so on are all already there, regardless where the include files reside.
IMHO the only real issue (for adding it to a system like vcpkg) is that nmos-cpp is "exporting" more than one include directory. One vcpkg library should either export one include file ore one include directory.
I think for the time being the chosen approach with the outer CMakeLists.txt is right for working with nmos-cpp (as a vcpkg library) and make some experience. We will soon find out if there will problems like C++ symbol name clashes will pop up.
If not, we can again rethink the directory setup and afterwards try to get nmos-cpp upstream to be an official/supported vcpkg library, can't we?

You need to confirm if the port uses dependent ports in vcpkg. I think that if the port's built-in dependencies are used, their header files will be exported.
You can also delete duplicate header files in portfile.cmake, such as:
file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/include/boost)

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


chore: upgrade charts

updated chart jx3/jx-build-controller from 0.0.28 to 0.0.31
updated chart jx3/jx-pipelines-visualizer from 1.1.6 to 1.1.8

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign macox
You can assign the PR to them by writing /assign @macox in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test vault

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

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test vault

tls
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test tls

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

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test vault

tls
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test tls

gsm
dff2610dbe5e3c54ea02fd83485a2e66888e5b09
link
/test gsm

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


AdoptOpenJDK is now Eclipse Temurin

Documentation should be updated to direct users to the Eclipse Temurin Java releases.
The AdoptOpenJDK project has moved to the Eclipse Foundation and rebranded as Adoptium.
Users of AdoptOpenJDK should now be directed to use Eclipse Adoptium (ref) to stay up to date with the latest security and functional fixes.
This is a supported Java runtime in Mule core, and is a drop-in replacement for the old AdoptOpenJDK.

Thanks @tellison . We will add the information in our upcoming sprint.

@tellison We have addressed the issue: https://github.com/mulesoft/docs-mule-runtime/pull/2434
Thanks again for the heads up! :)


PDF is blank after attempting to fill in fields

EDIT: I am attempting this on wsl2 - in case that makes a difference
I am trying to wrap my head around how a simple form can be filled but I can't seem to make it work.
I tried the examples - ie. cargo test in formfill but the filled.pdf has no information entered in the fields?
I have put together a small example using the same template - attempting to fill just one field - but I have the same result, no data entered.
Am I doing something wrong here?
PS: not really a go person but I really need to get some PDF forms filled in automatically
package main

import (
        "fmt"
        "github.com/benoitkugler/pdf/formfill"
        "github.com/benoitkugler/pdf/reader"
        "os" // Import the os package
)

var data = []formfill.FDFField{
        {T: "z2",
        Values: formfill.Values{
            V: formfill.FDFText("testing_here"),
        },

    },
}

func main() {
        const path = "original_sample.pdf"
        doc, _, err := reader.ParsePDFFile(path, reader.Options{})
        if err != nil {
                fmt.Fprintf(os.Stderr, "Error: %s\n", err)
                os.Exit(1) // Exit the program if there is an error
        }

        err = formfill.FillForm(&doc, formfill.FDFDict{Fields: data}, true)

        if err != nil {
                fmt.Fprintf(os.Stderr, "Error: %s\n", err)
                os.Exit(1) // Exit the program if there is an error
        }
        out, err := os.Create("filled.pdf")
        if err != nil {
                fmt.Fprintf(os.Stderr, "Error: %s\n", err)
                os.Exit(1) // Exit the program if there is an error
        }

        if err = doc.Write(out, nil); err != nil {
                fmt.Fprintf(os.Stderr, "Error: %s\n", err)
                os.Exit(1) // Exit the program if there is an error
        }

}

Hi !
To reproduce the issue and help you, I would need the PDF you use as input : "original_sample.pdf". Could you post it here  (or send me by email) ?

Thanks for your response. It was one of the sample files from the repo:
https://github.com/benoitkugler/pdf/blob/main/formfill/test/ModeleRecuFiscalEditable.pdf
I just wanted to ensure that I am using the package the right way before I go digging into other things.

Huh, this is quite strange.
I've just run your sample code (as a test) and I see filled.pdf being correctly filled (only the field "Nom ou dénomination" of course).
Could you send me the output file ("filled.pdf") you get  ?

I just tried it again on a different machine and I think I understand what the issue is.
In the original setup - on wsl2 and on my Ubuntu installation, I would default to firefox filled.pdf to view the filled-in form.
On this other machine, due to having a different environment to run through VcXsrv, I can't do the same thing if the Xsrv is not started so I defaulted to opening the explorer and then opening the pdf in the default reader.
And the field appears to be filled in. I originally thought it may be the fact that I'm on a different system but then realized that I am using different viewers.
filled.pdf
To further test my theory I opened the document in both Chrome and Firefox side by side and it appears that Firefox does not like the way the pdf is presented and fails to render the text - snap attached (Firefox on the left, Chrome on the right).
I am also attaching my filled-in file.

Very useful, thank you. I'll try to find why Firefox doesn't like the generated PDF

PDF.js (the library used by Firefox) seems to demand Annotation widget and Form fields to be merged, which is only suggested by the spec. This is not implemented so far, but it seems worth to support it.
Will work on it as soon as possible.

Alright, commit f9b3220 (tagged as v0.0.8) should fix the issue with Firefox. Could you try and confirm it works as expected ? Thank you.

All works fine. Thank you for the prompt responses and action.


Improve punctuation tokenization and word split

Description
The tokenizer previously ignored most (but not all) punctuation characters. Now all ASCII printable characters are included in tokens.
These characters are often very salient features when detecting the programming language. In some cases, even more than identifiers.
A sequence of consecutive non-alphanumeric characters will be recognized as a single token (e.g. == != >= <==>) with the following exceptions:

Comment tokenization behavior is preserved, so /* or <!-- are still ignored as part of comments. As a side-effect, some edge cases are now fixed (e.g. <!---->, <!--Comment-->).

SGML tokens are mow more restrictive, so <a>, <?xml>, <!DOCTYPE> are still SGML recognized, but <!-- or <-- are not. This also simplifies the corresponding lexer action.

Parenthesis (()), square brackets ([]) and curly brackets ({}) are tokenized independently.

Word splitting is also altered by this change:

[.@#/*] are not considered part of an identifier anymore. But a single character from [.@#] is recognized as part of a word if it's the first one. So .globl in Assembly is still a single token, but console.log is now split as console and .log. In Objective-C, @interface will still be a single token, but<EMAIL_ADDRESS>will be 3 tokens. This change leads to less unique tokens in the vocabulary, which helps classification too.

Checklist:
Checklist does not apply.

Don't pay much attention to the generated lexer files. Note that they currently include changes from #5006 but I'll rebase when that's merged.
Doing some simple cross-validation, this changes from 58 errors in master to 45 errors after this change. It has a particularly positive effect in some languages, including C and C++.
diff report
--- master.list	2020-10-22 21:59:07.650376934 +0200
+++ tokenizer-symbols3.list	2020-10-22 21:58:34.013900257 +0200
@@ -6 +6 @@
-AGS Script/KeyboardMovement_102.asc GOOD
+AGS Script/KeyboardMovement_102.asc BAD(Public Key)
@@ -16,2 +16,2 @@
-AsciiDoc/list.asc BAD(Public Key)
-Assembly/3D_PRG.I GOOD
+AsciiDoc/list.asc GOOD
+Assembly/3D_PRG.I BAD(Motorola 68K Assembly)
@@ -21,2 +21,2 @@
-Assembly/fp_sqr32_160_comba.inc BAD(C++)
-Assembly/lib.inc BAD(Motorola 68K Assembly)
+Assembly/fp_sqr32_160_comba.inc GOOD
+Assembly/lib.inc GOOD
@@ -24,2 +24,2 @@
-BitBake/gstreamer-libav.bb BAD(BlitzBasic)
-BitBake/qtbase-native.bb BAD(BlitzBasic)
+BitBake/gstreamer-libav.bb GOOD
+BitBake/qtbase-native.bb GOOD
@@ -34 +34 @@
-C++/16F88.h GOOD
+C++/16F88.h BAD(C)
@@ -36 +36 @@
-C/array.h BAD(Objective-C)
+C/array.h GOOD
@@ -44 +44 @@
-C/bootstrap.h BAD(Objective-C)
+C/bootstrap.h GOOD
@@ -48 +48 @@
-C++/ClasspathVMSystemProperties.inc BAD(Pawn)
+C++/ClasspathVMSystemProperties.inc GOOD
@@ -75 +75 @@
-C++/libcanister.h GOOD
+C++/libcanister.h BAD(C)
@@ -77 +77 @@
-C++/metrics.h BAD(C)
+C++/metrics.h GOOD
@@ -80,2 +80,2 @@
-C/Nightmare.h GOOD
-C/ntru_encrypt.h BAD(C++)
+C/Nightmare.h BAD(C++)
+C/ntru_encrypt.h GOOD
@@ -114 +114 @@
-C++/rpc.h BAD(C)
+C++/rpc.h GOOD
@@ -118 +118 @@
-C/syscalldefs.h BAD(Objective-C)
+C/syscalldefs.h GOOD
@@ -166 +166 @@
-Formatted/long_seq.for BAD(Forth)
+Formatted/long_seq.for GOOD
@@ -172 +172 @@
-Fortran/bug-185631.f GOOD
+Fortran/bug-185631.f BAD(Forth)
@@ -197 +197 @@
-GLSL/recurse1.fs GOOD
+GLSL/recurse1.fs BAD(Filterscript)
@@ -253 +253 @@
-INI/defaults.properties BAD(Java Properties)
+INI/defaults.properties GOOD
@@ -266,2 +266,2 @@
-Java Properties/libraries.properties GOOD
-Java Properties/sounds.properties GOOD
+Java Properties/libraries.properties BAD(INI)
+Java Properties/sounds.properties BAD(INI)
@@ -294,2 +294,2 @@
-Mathematica/HeyexImport.m BAD(Objective-C)
-Mathematica/Init.m GOOD
+Mathematica/HeyexImport.m BAD(MATLAB)
+Mathematica/Init.m BAD(Objective-C)
@@ -298 +298 @@
-Mathematica/PacletInfo.m BAD(Mercury)
+Mathematica/PacletInfo.m GOOD
@@ -385 +385 @@
-M/ZDIOUT1.m BAD(Mercury)
+M/ZDIOUT1.m GOOD
@@ -408 +408 @@
-NewLisp/log-to-database.lisp BAD(Common Lisp)
+NewLisp/log-to-database.lisp GOOD
@@ -429 +429 @@
-Objective-C/Siesta.h BAD(C)
+Objective-C/Siesta.h BAD(C++)
@@ -459 +459 @@
-Pawn/Check.inc BAD(PHP)
+Pawn/Check.inc GOOD
@@ -466 +466 @@
-Perl/exception_handler.pl BAD(Raku)
+Perl/exception_handler.pl GOOD
@@ -505 +505 @@
-Proguard/proguard-rules2.pro GOOD
+Proguard/proguard-rules2.pro BAD(IDL)
@@ -532 +532 @@
-Raku/01-dash-uppercase-i.t BAD(Perl)
+Raku/01-dash-uppercase-i.t GOOD
@@ -536 +536 @@
-Raku/A.pm BAD(Perl)
+Raku/A.pm GOOD
@@ -539,2 +539,2 @@
-Raku/calendar.t BAD(Perl)
-Raku/ContainsUnicode.pm BAD(Perl)
+Raku/calendar.t GOOD
+Raku/ContainsUnicode.pm GOOD
@@ -558 +558 @@
-Rebol/booters.r GOOD
+Rebol/booters.r BAD(R)
@@ -561 +561 @@
-R/hello-r.R BAD(Rebol)
+R/hello-r.R GOOD
@@ -579 +579 @@
-SaltStack/eval.sls BAD(Scheme)
+SaltStack/eval.sls GOOD
@@ -599 +599 @@
-Smalltalk/scriptWithPragma.st BAD(HTML)
+Smalltalk/scriptWithPragma.st GOOD
@@ -623 +623 @@
-Text/LIDARLite.ncl GOOD
+Text/LIDARLite.ncl BAD(NCL)

The last error (LIDARLite.ncl) is not relevant. See #5054.

@smola it looks like the changes here cause a change in detection of one of the samples causing the only failing test blocking this PR being merged:
  1) Failure:
TestClassifier#test_classify_ambiguous_languages [/home/runner/work/linguist/linguist/test/test_classifier.rb:56]:
/home/runner/work/linguist/linguist/samples/AGS Script/KeyboardMovement_102.asc
[["Public Key", -7.575176404971187], ["AsciiDoc", -10.933410495077375], ["AGS Script", -13.908242416753797]].
Expected: "AGS Script"
  Actual: "Public Key"

Could you please look into why this is. 🙇

@smola it looks like the changes here cause a change in detection of one of the samples causing the only failing test blocking this PR being merged:
  1) Failure:
TestClassifier#test_classify_ambiguous_languages [/home/runner/work/linguist/linguist/test/test_classifier.rb:56]:
/home/runner/work/linguist/linguist/samples/AGS Script/KeyboardMovement_102.asc
[["Public Key", -7.575176404971187], ["AsciiDoc", -10.933410495077375], ["AGS Script", -13.908242416753797]].
Expected: "AGS Script"
  Actual: "Public Key"

Could you please look into why this is. 🙇

@lildude There were a couple of problems that caused incorrect eating of some files (bad C comment detection). Those are now fixed. Remaining errors were on things like SQL, which quite reasonable. I added the test change to skip files with extensions that have a catch-all rule, like .sql. I was going to introduce that in #5061 anyway.
So I think this is ready.

@lildude There were a couple of problems that caused incorrect eating of some files (bad C comment detection). Those are now fixed. Remaining errors were on things like SQL, which quite reasonable. I added the test change to skip files with extensions that have a catch-all rule, like .sql. I was going to introduce that in #5061 anyway.
So I think this is ready.


请问BPO支持上下文吗？

Thank you so much. Our company is dedicated to developing voice interaction products, and we're quite intrigued by BPO . We believe it has the potential to greatly enhance the user experience of our products.
With that in mind, I would like to ask for your advice on how we can integrate it into our products more efficiently, based on your deep understanding of BPO, to optimize and improve the interaction quality. Could you provide some specific suggestions or best practices?
I look forward to your valuable insights and would like to thank you once again for your support and cooperation.

I don't know a lot about speech generation, but I believe voice interaction and text are essentially the same. So, perhaps a similar process can be adopted. Firstly, collect some data with human preferences, then use BPO to obtain an optimization model. After that, maybe consider integrating this optimization model directly into the generation pipeline (depends on your application), perform some reprocessing of the user's input, before inputting it into the actual generator. Furthermore, implementing multi-turn BPO is also fairly easy, we just need to use multi-turn human feedback data and include the conversation history during the optimization. Hope this helps you.

Thank you for your response. As you mentioned, indeed, we have completed the bidirectional conversion between voice and text through ASR and TTS . Therefore, it is unequivocal that what we are actually invoking with the LLM  is text. However, a characteristic of voice interaction is that users' language tends to be very brief, and much of the necessary information must be extracted from the context (in multi-turn conversations).
Therefore,I would like to konw whether we can directly utilize our existing BPO for this purpose. For example, if we introduce the context information in the format of [CLS] I am a student [SEP] I like studying [SEP], can we obtain satisfactory BPO results? Or must we retrain a model based on multi-turn dialogue data?

you can try using this single-turn model, but I think maybe retrain a model is necessary not only because of muti-round but also our model focuses on instruction-following tasks instead of chatting.

Got it, thanks!


[FG]   parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/lib/type_check.pyi [fs read 0ms] (0ms)
[FG]   binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/lib/type_check.pyi (0ms)
[FG]   parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/lib/ufunclike.pyi [fs read 1ms] (1ms)
[FG]   binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/lib/ufunclike.pyi (0ms)
[FG] completion at /Users/lky/code_repos/detect_by_text/train.py:4:3 [found 583 items] (133ms)
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
[FG] parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/__init__.py [fs read 0ms] (2ms)
[FG] binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/__init__.py (0ms)
[FG] parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/_ufunc.pyi [fs read 1ms] (4ms)
[FG] binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/_ufunc.pyi (1ms)
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: getSemanticTokens delta
[BG(1)] getSemanticTokens delta previousResultId:1629771581108 at /Users/lky/code_repos/detect_by_text/train.py ...
[BG(1)]   parsing: /Users/lky/code_repos/detect_by_text/train.py (3ms)
[BG(1)]   binding: /Users/lky/code_repos/detect_by_text/train.py (1ms)
[BG(1)] getSemanticTokens delta previousResultId:1629771581108 at /Users/lky/code_repos/detect_by_text/train.py (27ms)
Background analysis message: analyze
[BG(1)] analyzing: /Users/lky/code_repos/detect_by_text/train.py ...
[BG(1)]   checking: /Users/lky/code_repos/detect_by_text/train.py (5ms)
[BG(1)] analyzing: /Users/lky/code_repos/detect_by_text/train.py (5ms)
Background analysis message: resumeAnalysis
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
Background analysis message: setFileOpened
Background analysis message: markFilesDirty
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange
[FG] parsing: /Users/lky/code_repos/detect_by_text/train.py (1ms)
[FG] binding: /Users/lky/code_repos/detect_by_text/train.py (1ms)
Background analysis message: getSemanticTokens delta
[BG(1)] getSemanticTokens delta previousResultId:1629771581936 at /Users/lky/code_repos/detect_by_text/train.py ...
[BG(1)]   parsing: /Users/lky/code_repos/detect_by_text/train.py (5ms)
[BG(1)]   binding: /Users/lky/code_repos/detect_by_text/train.py (1ms)
[BG(1)]   parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/__init__.py [fs read 1ms] (21ms)
[BG(1)]   binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/__init__.py (2ms)
[BG(1)]   parsing: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/_ufunc.pyi [fs read 0ms] (17ms)
[BG(1)]   binding: /Users/lky/miniforge3/lib/python3.9/site-packages/numpy/typing/_ufunc.pyi (2ms)
[BG(1)] getSemanticTokens delta previousResultId:1629771581936 at /Users/lky/code_repos/detect_by_text/train.py (74ms)
Background analysis message: analyze
[BG(1)] analyzing: /Users/lky/code_repos/detect_by_text/train.py ...
[BG(1)]   checking: /Users/lky/code_repos/detect_by_text/train.py (6ms)
[BG(1)] analyzing: /Users/lky/code_repos/detect_by_text/train.py (6ms)
Background analysis message: resumeAnalysis
Background analysis message: getDiagnosticsForRange
Background analysis message: getDiagnosticsForRange

It appears you are using a conda environment.. I'm guess your built opencv is in a different python3.9 environment. can you try selecting a global python3.9 environment?

This is the record I wrote, install python 3.9 and build OpenCV for Apple Silicon.
https://zhuanlan.zhihu.com/p/394141304
I have been used conda-forge/miniforge Python 3.9 since I buy it the computer, and ONLY created a virtual environment named "base", no global Python 3.9 on my computer.
(base) ➜  ~ conda env list
# conda environments:
#
base                  *  /Users/lky/miniforge3

Does the trace above include you trying to do cv2. to check for a completion?
The screenshot doesn't show that cv2 is unresolved, only unused. We won't actually do any work to analyze an imported library until you try and use it, and just having it in the file doesn't count as using it.
Can you try showing in the trace what happens if you do a completion or a hover with cv2?

@jakebailey, @heejaechang - I think this is a (recent?) regression. Pylance isn't properly resolving any bundled stubs that are in directories that end in -stubs. There appears to be a bad interaction between the import resolver and the virtual file system. When the import resolver enumerates the directories within the bundled/stubs directory, the readdirEntriesSync method in pyrightFileSystem.ts is filtering out all of the directories that end in -stubs.

The code that is ignoring those stubs folders has been around for about 5 months, it seems; I'm not sure what would have changed to ignore those.
Does this happen in pyright too, or is this just a Pylance thing and something we've done recently broke it?

Pyright doesn't know anything about the bundled directory (since that's specific to pylance), and it doesn't implement the resolveImportEx method. The bug is in that code path.

If I rename the bundled/stubs/cv2-stubs to bundled/stubs/cv2, it works fine.

Aha, just that folder. The way we do it right now is weird as it is, we should really just be sticking our bundled stubs in as another search path with the other third party libs. I'll take a look.

Renaming it kills the partial-ness, as you have to have it be a -stubs for it to be a partial stub, though. How are you managing to reproduce this? We thought we could reproduce it and fixed it (hence, closing this once), but apparently the setup still differs.

I installed the latest version of pylance, then typed:
import cv2
cv2.imread

Pylance reported that it couldn't resolve the import even though the bundled stub was present.
I just realized that I don't have cv2 installed in my virtual environment. (My venv is Python 3.9, and apparently cv2 isn't compatible with it.) My expectation was that pylance would use the bundled cv2 partial stub even though I don't have the cv2 library installed, but perhaps that's a bad assumption? Stub libraries (partial or otherwise) shouldn't require the associated library to be installed.

Our cv2 stub is a partial stub, and can only apply if the library is actually installed, as it's only stubbing out a subset of the modules (via a FS overlay). This is probably the only partial stub that's of the top level module, though, and not some deeply nested compiled module, so I can see why it would seem like it would resolve, but that's not how this particular stub is designed to function.
I think the user above is in a situation where the library is installed (via some local build), but it's still not working. @bschnurr tried to emulate this buy doing local builds, package installs in Ubuntu, etc, which both found the module naming mismatch plus the dist-packages thing, neither of which seemed to matter (even though the former should have really worked).
I tried to go to https://zhuanlan.zhihu.com/p/394141304 to see if there are build instructions there, but that site doesn't load for me.
The only thing I can think of at this point is that the module is cv2.cpython-39-darwin.so, but the library we stubbed out is cv2-stubs/__init__.py, and the mismatch is causing problems. Previously we had cv2.cv2 as a native stub, as from our experience OpenCV as shipped on PyPI actually just reexports everything in an cv2/__init__.py from cv2.cv2 (with a few extra members); it doesn't actually ship to end users as a single sofile. That to me seems like the problem, and I'm not sure how to resolve it (or if this is even a valid way to build the library, if this __init__.py is important.

I tried to go to https://zhuanlan.zhihu.com/p/394141304 to see if there are build instructions there, but that site doesn't load for me.

The same article in Github.io
https://mosdeo.github.io/2021/07/25/如何在Apple Silicon上安裝OpenCV與TensorFlow給Python用？/

when I was trying to repo i didn't use a venv with a symlink to cv2 in usr/local/lib/python3.9...
I was just using usr/bin/python3/ or /bin/python3/ and they were both python3.8

i'll try again with a venv and symlink

@mosdeo is this still an issue with latest pylance release?

@mosdeo is this still an issue with latest pylance release?

Pylance（v2022.3.2）working better than before, but not all.
It works in the OpenCV static method：

But not working in the OpenCV dynamic object menber method：

The same dynamic object member method under Jedi：

Auto suggestions still not works in the OpenCV dynamic object menber method：

The same dynamic object member method under Jedi：

@mosdeo pylance is a static analyzer, none of dynamic behavior will be supported. for those, Jupyter is needed (assuming you ran the cell)


Setting Toast elevation and absolute positioning not working; still blocking elements below it.

I have a Bottom Tab Navigator, but putting a Toast like this at the bottom:
    Toast.show({
        text: "Done!",
        duration: 4000,
        position: 'bottom',
        style: { marginBottom: 50, elevation: -1 }
    });

Will block interaction with the tab navigator. Setting elevation was suggested before, but did not work; the Toast is still absorbing the clicks that is meant for the tab navigator.
Is there a way to set the toast 50 from the bottom and still have interaction with the elements below? I have also tried position: absolute, bottom: 50 in the styles, but this didn't even show the Toast at all.

@newbiemate checked this. It is working fine for me. Can you check whether you have placed <Root> over the topmost component. Attaching gif.

Code
import React from 'react';
import { View } from 'react-native';
import { createBottomTabNavigator } from 'react-navigation';
import { Root, Container, Header, Content, Toast, Button, Text } from 'native-base';

class HomeScreen extends React.Component {

  constructor(props) {
    super(props);
    this.state = {
      showToast: false
    }
  }

  render() {
    return (
      <Container>
        <Header />
        <Content padder>
          <Button onPress={()=> Toast.show({
              text: 'Wrong password!',
              buttonText: 'Okay',
              style:{backgroundColor:"blue"}
            })}>
            <Text>Toast</Text>
          </Button>
        </Content>
      </Container>
    );
  }
}

class SettingsScreen extends React.Component {
  render() {
    return (
      <View style={{ flex: 1, justifyContent: 'center', alignItems: 'center' }}>
        <Text>Settings!</Text>
      </View>
    );
  }
}

const App = createBottomTabNavigator({
  Home: HomeScreen,
  Settings: SettingsScreen,
});

export default ()=> <Root><App/></Root>

@newbiemate Toast currently does not allow interaction with the views below it.

That shouldn't happen actually as per design guidelines

@newbiemate Is this what you are looking for?

Closing this today in case of no response

@SupriyaKalghatgi this not help me. Toast still blocking elements below it.


Consistent use of DTOs in Jira/Bitbucket/BambooService and improved test coverage in BitbucketService

Description

This add test cases for the BitbucketService and handles edge cases by catching Exceptions.
The methods deleteWebHook() and deleteExistingWebHooks() were unused and therefore deleted.
Also this PR gets rid of the Map Casting in the restTemplate and its mock functions by replacing them with DTOs.
Steps for Testing

As now all rest calls for Bamboo/Bitbucket/Jira are called with DTOs, all relevant rest calls should be tested.

Login using a TUM account over JIRA
Create a Programming Exercise and verify that all necessary steps are working as expected.
Participate in it and also verify that all works as expected.
Delete the exercise and make sure everything works
Use an admin account to change the groups of an existing user in the user management view

Test Coverage

BitbucketService.java: 51% -> 76%

I'm not able to access any of the admin menu items. I compared this to the develop branch where I am able to, so I think it's something related to this pr. Programming tests worked fine!
Steps to reproduce (test server 1)

As a TUM account user and an admin user, log in (I tested with my own user/TUM identifier)
Try to access any of the admin menu items, such as user management.
The menus are not accessible. Expected: should be able to access the admin menus

logs:
bootstrap:128 GET https://artemistest.ase.in.tum.de/app/13.20dfd72fed667222a82d.chunk.js net::ERR_ABORTED 404
r.e @ bootstrap:128
loadChildren @ app-routing.module.ts:15
loadModuleFactory @ router.js:3493
load @ router.js:3479
Show 52 more frames
instrument.js:109 ERROR Error: Uncaught (in promise): ChunkLoadError: Loading chunk 13 failed.
(error: https://artemistest.ase.in.tum.de/app/13.20dfd72fed667222a82d.chunk.js)
ChunkLoadError: Loading chunk 13 failed.
(error: https://artemistest.ase.in.tum.de/app/13.20dfd72fed667222a82d.chunk.js)
at Function.r.e (bootstrap:105)
at loadChildren (app-routing.module.ts:15)
at te.loadModuleFactory (router.js:3493)
at te.load (router.js:3479)
at l.project (router.js:2441)
at l._tryNext (mergeMap.js:44)
(anonymous) @ instrument.js:109
Wo @ core.js:4352
handleError @ core.js:4400
handleError @ sentry.error-handler.ts:51
next @ core.js:28039
o @ core.js:24853

I've tested it on testserver1 with my tum credentials and I don't have this problem. I have access to all menu entries

@fde312 and I found an issue with regards to test runs, sometimes you cannot switch between programming exercises. We are not sure if this is related to the changes introduced here or to the big TypeScript PR we did a while ago, however we could not reproduce this on other branches.


Update list.dart

This PR fixes #37262.

Review in https://dart-review.googlesource.com/c/sdk/+/108180

@MetalDent thanks for the contribution! Please see feedback in the above review link

Review in https://dart-review.googlesource.com/c/sdk/+/108180

Okay, so you don't want me to make a separate class?

The extra List class won't work, there is already one List class declared in that library. The only issue is that the existing List class inherits the operator== method from Object, including the documentation.
By adding an abstract operator== method to the existing List class, we can add a new documentation for the method, without changing the inherited implementation.

Looks good!
Thanks.


Add new algos

I want to add algorithms like

Karastuba's Fast Multiplication
Fast Power
GCD
Is Prime

Where should I add it. Or shouldn't I ?

Sure you can!
Add them to the NumberTheory dir.


Java native object peers must not allow double free

ZstdDictDecompress, ZstdInputStream, etc. must not allow double free of native data structures, even if the classes are used incorrectly, for example close() is called from multiple concurrent threads. (Apart from that close() should be idempotent.)
It means that in addition to isClosed flags, the close() methods should be synchronized. Methods that access the native pointers should be synchronized too, to prohibit use-after-free.

The fact that native zstd is not thread-safe makes it doubly important to make Java objects thread-safe and not expose Java users to segmentation faults and native memory corruption, isn't it?

Makes sense, I will look at it.

I think the last commit addresses the issue

I left comments to the commit: https://github.com/luben/zstd-jni/commit/6149ee4473b1cec8ad3865c8d437600611cbe9c5

Thanks a lot for the comments and suggestions. I did not only understands the concerns about the aligned access. I think zstd is align safe itself.

I'm not 100% sure about the alignment, could you please check https://github.com/facebook/zstd/issues/1210?

Yes, it looks like it was not alignment issue the problem there. How do you think alignment can start being a problem in multi-threaded usage, while it was not a problem in the single-threaded usage?
I think we should continue as-is and if it turns to be a problem we can always check the alignment on the C side.

I have released it as 1.3.7-3. It should be available shortly on Maven-central


Birth Sex ValueSet only allows M and F

This may be more of a UDS question than UDS+, but requiring a birth sex be only M or F seems problematic, as many states allow other values on birth certificates.  US Core handles this with NullFlavors, but UDS+ does not.
This means that intersex or other non-binary sex values must get folded into either M or F codes, which means that by definition the birth sex values you get from UDS+ reporting will not be reliable.  When you get an "M" as the birth sex, you won't be able to tell whether the patient's birth sex was actually male, or if a non-male sex was "folded in" to the M code by the submitter.

It also means that a submitter cannot rely on a US Core compliant API to gather UDS+ data from a certified EHR.  This is a broader problem, but even when a health center is using a non-EHR data submitter, the EHR will still need to do UDS+ specific development to support the movement of data from the data source to the data submitter.

Cooper, currently the UDS+ profiles are aligned with the UDS Manual and hence values other than M/F are not allowed. In the future it can be considered for alignment between UDS and US Core.

I'm less concerned with alignment between UDS and US Core, but more concerned about the UDS Manual having this issue.  Is it well understood that the UDS birth sex data and any data derived from it is, by definition, not reliable?

(note, I'm using the term "non-binary" here as a term for patient's whose birth sex is not M or F.  I do NOT mean it as a gender identity in this situation).
Given some discussion we had about this being a UDS requirement we can't change as part of UDS+, could we at least have some guidance in UDS+ on what systems should do if a patient has a birth sex that is not M or F?  I think the options are:

Don't include the patient in the UDS+ report at all.  The downside here is that it will look like the FQHC is seeing fewer patients than they actually are, since HRSA will never know about the non-binary patients.
Always send "M" (or "F") for non-binary patients.  This means HRSA will have an accurate  count of the patients being seen, the birth sex will just be known to be incorrect for some unknown number of patients.


The questions about processing datasets.

Hi, authors.
Utilizing the processed datasets can reproduce the results reported in your paper.
However, when I rebuild needed datasets based on the raw data (e.g, reviews_Sports_and_Outdoors_5) by released codes (i.e., build_data.py), the performance is reduced severely.
Therefore, can you provide the remaining files like 'user-item-dict.json', 'user_list.txt', and 'item_list.txt' of the three datasets?

Hello!
I have uploaded relevant files to Google Drive. I guess the problem may be the wrong version of the dataset?  We used the old version of Amazon review dataset .


Some web pages can not save content with Capture Page Snapshot.

Some web pages can not save content with Capture Page Snapshot, such as this issue page (github.com), v1.3.5.

I'm afraid we're leveraging https://github.com/mozilla/readability for identifying and capturing page content, @ajmz -- feel free to post your concerns about particular pages to that project, but unfortunately it's out of this project's hands.
Good luck!


Computer restarts

OS version: Win 10 Pro N
Editor version : 1.53.2
Programming language: HTML,js
TabNine extension version: 3.2.7

Issue Details:
Every time when I open VSC with tabnine extension enabled after approximately 1 minute my computer just reboots. Nothing more really happens. That's the only thing and it's underwhelming. Can you tell me what should I do?

Thanks for reporting this.
Can you please share the content of the developer tools - console tab before the PC reboots? (Help => Toggle Developer Tools => Console)

The first three errors are not related to the case as I just moved the files to another director.

Thank you @MroseQ!
Sorry, I will need more information in order to be able to help.
Please set Tabnine logs to write to your disk, and after your PC restarts share the log file here or send it directly to me.
This is how you do it:

Go to vs-code settings
Type 'tabnine' in the search box
You should see the option: 'Tabnine: Log File Path'
Enter a path you would like the log file to be saved in (including the log file name)

Thanks.

I've managed to insert the log path, but now the computer restarts after 5 seconds from starting VSC. How will the log file be named?

I opened the VSC again, yet no log file is visible. Maybe the reboots are too fast for the program to make a log file :/

It could be the case.
The log file should be named by the file name that was entered on your path.
For example, if my path is:
/Users/some-user/logs/tabnine
So my log file name will be: tabnine

Oh... so if that's the case... the files should be named Desktop because I assumed that it would finish the path on its own.
wd_log.txt
And I found it, but there are two files. The txt above and the "Desktop-11972" file with no extension, which I can't upload. I hope you can manage something with the txt though.

I will need the other file.
Try sending it to<EMAIL_ADDRESS>Thanks.

i've sent it. Good luck <3


chore(logging): add logs to pre and post reconcile hooks

Closes #5851
Release Notes
The operator now outputs logs when interrupting the reconciliation loop when requested by a plugin or encountering an error while invoking the plugins
Note for reviewers
I don't think we should backport this, given that it isn't a bug fix

/ok-to-merge


Update UserGuide.md

Potential fix

Codecov Report

Merging #192 (2504422) into master (f9c1e18) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #192   +/-   ##
=========================================
  Coverage     71.32%   71.32%
  Complexity      511      511
=========================================
  Files            81       81
  Lines          1566     1566
  Branches        217      217
=========================================
  Hits           1117     1117
  Misses          363      363
  Partials         86       86

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 371b7c0...2504422. Read the comment docs.