pr-upload: fix for non-homebrew-core taps

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

Before:
$ brew pr-upload --bintray-org=seeker --root-url=https://dl.bintray.com/seeker/bottles-languages
Error: No available formula with the name "seekingmeaning/languages/qbe"
In formula file: /usr/local/Homebrew/Library/Taps/seekingmeaning/homebrew-languages/Formula/qbe.rb
Expected to find class Seekingmeaning/languages/qbe, but only found: Qbe.

After:
$ brew pr-upload --bintray-org=seeker --root-url=https://dl.bintray.com/seeker/bottles-languages
==> seekingmeaning/languages/qbe
  bottle do
    root_url "https://dl.bintray.com/seeker/bottles-languages"
    cellar :any_skip_relocation
    sha256 "988f3c35fd0815a37069f9acc300dd2db302a49df9635c0c9a9e9383e2eb3779" => :catalina
    sha256 "63da44d545ab2ad20cd50412558842830de3bdb33f7d723ca8016c049df012c0" => :x86_64_linux
  end
[develop d4ebc2f] qbe: add 2019-11-25 bottle.
 1 file changed, 7 insertions(+)
curl: (22) The requested URL returned error: 404 Not Found
######################################################################## 100.0%
{"message":"success"}curl: (22) The requested URL returned error: 404 Not Found
######################################################################## 100.0%
######################################################################## 100.0%
{"files":2}

Nice fix!


Detect unconditional redirects and ignore any rules defined afterwards

(this is my first PR for GoLang code and for Kubernetes-related projects, so bear with me if I make mistakes or faux pas)
ALBs only support 100 rules and we have hit the limit in our project -  we are looking for a way to reduce rules and have identified that the aws alb controller is creating some unneeded ALB rules.
We use a annotation to redirect port 80 requests to port 443. This is the annotation documentation: https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/master/docs/guide/tasks/ssl_redirect.md
If a unconditional redirect rule is present in the configuration then all following rules within the same listener are not needed. This PR changes things so they are skipped thereby conserving rules.
For the biggest benefit, it is best to define the redirect rule first in the config. Because only subsequent rules are skipped. This is in line with the ssl-redirect annotation which says:

the ssl-redirect action must be be first rule(which will be evaluated first by ALB)

I have tested on our project, with success. But please let me know if I can make any improvements.
If this is not something that fits with the direction of the project, no problem, please close this PR. I only created it in case it's useful - I'm 100% happy to close if not the case.

Relvant GitHub issue https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/853#issuecomment-578547468

/assign @bigkraig

We have been running a ALB ingress controller that uses the code in this PR at https://peopleperhour.com in production successfully. It freed up > 40 ALB rules for us. The ALB gets about 110 requests per second:

/ok-to-test

would you help fix the lint..i'll clean up some vpcs in our test account to make e2e pass :D

would you help fix the lint..

Yes, I'll fix this. If I have understood it, I should have used gofmt -w yourcode.go. Leason learnt.

I pushed a fix which I hope will pass the linter :crossed_fingers: . I also added more unit tests to gain more confidence the code works as expected and refactored given the suggestion in https://github.com/kubernetes-sigs/aws-alb-ingress-controller/pull/1162#discussion_r379190396

thanks 🤣
/approve
/lgtm

Thanks so much for this @tomfotherby and for merging it @M00nF1sh. We're really keen to use this and just wondering if there is a target date for the next release containing it? Thanks!

Hi @tomfotherby @M00nF1sh
I just tried v1.1.6 with this fix. Is this intended to work across multiple hosts? For example:
spec:
  rules:
    - http:
        paths:
          - path: /*
            backend:
              serviceName: ssl-redirect
              servicePort: use-annotation
    - host: hostA
      http:
        paths:
          - path: /pathA/*
            backend:
              serviceName: serviceA
              servicePort: 80
    - host: hostB
      http:
        paths:
          - path: /*
            backend:
              serviceName: serviceB
              servicePort: 80

I believe that if the redirect rule does not have host specified, it should be considered as unconditional redirect as well. So in this case hostA and hostB rules should be skipped.

@AndrewBoklashko - Good point. I agree with you. Alas, currently the controller processes each rule in turn and the short circuit mechanism that avoids unneeded rules only operates in the scope of the individual rule.
In my understanding, but please correct me if I'm wrong, the bug you found isn't causing active harm, i.e. it's not causing needed rules to go missing, but it's not operating as well as it could (i.e. it's not doing what it says on the tin). Is that correct?

I appologise for this mistake. I hadn't considered "hostless rules" even though I see the example does not have a host specified. Thanks for bringing it up - I will work on a patch because we would also benefit from this. I can't promise speed though, I think it will be a level-up in complexity from the current solution and there's rather a lot going on presently...

In my understanding, but please correct me if I'm wrong, the bug you found isn't causing active harm, i.e. it's not causing needed rules to go missing, but it's not operating as well as it could (i.e. it's not doing what it says on the tin). Is that correct?

That is correct.

I appologise for this mistake. I hadn't considered "hostless rules" even though I see the example does not have a host specified. Thanks for bringing it up - I will work on a patch because we would also benefit from this. I can't promise speed though, I think it will be a level-up in complexity from the current solution and there's rather a lot going on presently...

All good, thanks for the follow up. Will be looking to try a patch, but no rush - we can live with how it works now too.

hi @tomfotherby, @AndrewBoklashko, I just thought I'd follow up and see if either you got any further on the above ^^
Is this being tracked under #853? If not, would you like me to create a new issue for it?

@tdmalone - At my company we don't currently use "hostless rules" in our ALB ingress spec. So to set expectations, I'm not going to be working on an enhancement for the foreseeable future. I would find it hard to test things without a real use-case in my aws account. My PR unfortunately had a bug (raised in https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/1274 and fixed in https://github.com/kubernetes-sigs/aws-alb-ingress-controller/pull/1286) which revealed to me the sharp edges and how important it would be to test well. With any luck, other folk will get involved.

Is this being tracked under #853? If not, would you like me to create a new issue for it?

I would say #853 tracks it.

No worries @tomfotherby, thanks for the reply. I might see if I can refactor to hostless rules and/or try to dig into the code here to see if I can make something work (not a Go programmer, but I could take a stab!)

resolved to version 1.16 https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/1274


show dialog when update available

name: pull request
about: submit changes to the project
title: "[pr] "
labels: ''
assignees: ''

Closes #463
/claim #463
type of change

[ ] bug fix (non-breaking change which fixes an issue)
[ ] new feature (non-breaking change which adds functionality)
[ ] breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] this change requires a documentation update

how to test
describe how to test your changes. include steps like:

checkout this branch
run npm install (or equivalent)
start the app with npm start
navigate to '...'
click on '...'
observe '...'

screenshots (if applicable)
add screenshots to help explain your changes.
checklist

[ ] i read rust & programming best practices
[ ] my code follows the style guidelines of this project
[ ] i have performed a self-review of my own code
[ ] i have made corresponding changes to the documentation
[ ] my changes generate no new warnings
[ ] i have added tests that prove my fix is effective or that my feature works
[ ] new and existing unit tests pass locally with my changes
[ ] any dependent changes have been merged and published in downstream modules
[ ] other operating systems are not broken

additional notes
any other information that you want to share that is relevant to the pr. this could include benchmarks, areas that you want reviewers to pay extra attention to, or notes about the implementation.

hey had done this already on rust side few hours ago sorry forgot to close


Test merge

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.7 out of 9 committers have signed the CLA.:white_check_mark: adamdehaven:white_check_mark: slauzinho:white_check_mark: jefrailey:white_check_mark: bikingbadger:white_check_mark: davestaab:white_check_mark: luminisward:white_check_mark: ivos:x: charshin:x: solufaYou have signed the CLA already but the status is still pending? Let us recheck it.

Looking at merge conflicts


Import for COCIRS_0xxx and COCIRS_1xxx

This pull request (base on https://github.com/SETI/pds-opus/pull/1174) is to track the progress of adding COCIRS_0xxx and COCIRS_1xxx to OPUS: (need to work with https://github.com/SETI/pds-webtools/pull/40)
Items done:
Add import steps for COCIRS_0xxx and COCIRS_1xxx
To do items:

In pdsfile, add rules for COCIRS_0xxx and COCIRS_1xxx
In OPUS, update the ring geometry values
In OPUS, add api results test for COCIRS_0xxx and COCIRS_1xxx

Fixes #1183 , (based on https://github.com/SETI/pds-opus/pull/1174)
Were any Django, import pipeline, table_schema, or dictionary files modified? Y/N

If YES:

Database used: Need to import new test database
All Django tests pass: Y
FLAKE8 run on modified code: Y

Were any JavaScript or CSS files modified? N

If YES:

JSHINT run on all affected files: NA
Tested on Chrome, Firefox, Safari, Edge, and iOS: NA
(Remember to test all browser sizes)
Tested for race conditions (with delayed API calls if applicable): NA

Description of changes:

Update import steps to import COCIRS_0xxx/1xxx
Need to work with updated COCIRS rules https://github.com/SETI/pds-webtools/pull/40

Updated VG_28xx import data based on Mark's spreadsheet

Update data quality score to 'GOOD' for VG_28xx
Update right_asc and declination values for the following stars in STAR_RA_DEC:
'BET_PER':              ( 47.042218542,  40.955646667)
'IO_HER':<PHONE_NUMBER>33,  46.006331945)
Change the spec size to None for VGUVS (since it's TBD)
Add data for instrument constraints and image constraints for VGISS (prof)
Note: for ring geometry like north-based incidence angle/emission angle, the populate functions still return the results of mathematic calculations instead of None (they are TBD in the spread sheet)


For Instance segmentation

Can this image segmentation tool generate data sets suitable for instance segmentation?

Yes, certainly.
If you check the notebook (Image labeller notebook.ipynb) you can see the kind of output it can generate from labels created in the tool.


refactor: version-based config service

Signed-off-by: WuShaoling<EMAIL_ADDRESS>
What is the purpose of the change

Closes #306

Brief change log

The configuration update strategy of the previous version uses a timing mechanism, which is cumbersome to implement and difficult to troubleshoot when a problem occurs. At present, this version uses the mechanism of updating immediately after receiving the event, and each update has a version number (based on timestamp). By comparing the version number, the old configuration is prevented from overwriting the new configuration.

Known issues

When the remote configuration is updated, we will call the getFileContent() to fetch the remote configuration. If the getFileContent() fails, it will return undefined, and the loadConfigFromRemote() function will return {}. Another way is to return undefined and terminate the remote update operation.

loadLuaScriptFromRemote() has the same issue, we need to discuss to decide which mechanism to use.

Checklist

Are tests written for added code?

[x] Yes
[ ] No because:

Did you introduce any new dependencies?

[x] No
[ ] Yes

Dependency:
Reason:

/approve


fix(nick): fix nickelodeon.be downloading

Description of your pull request and other information
Removing this test case fixes video downloading for nickelodeon.be and nickelodeon.nl.
Fixes https://github.com/yt-dlp/yt-dlp/issues/169#issuecomment-1202360313

Template
Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[x] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Removing test case does not fix extractor!

Removing test case does not fix extractor!

Weird, since I was finally able to download a nickelodeon video now.
It doesn't work if I keep the test case.

😕 Show complete verbose logs of both cases


fix(app): fixed demo app image publishing

Signed-off-by: Anton Biriukov<EMAIL_ADDRESS>

Codecov Report
Patch and project coverage have no change.

Comparison is base (2ceb99e) 90.07% compared to head (497c63b) 90.07%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #550   +/-   ##
=======================================
  Coverage   90.07%   90.07%
=======================================
  Files          98       98
  Lines        3797     3797
=======================================
  Hits         3420     3420
  Misses        250      250
  Partials      127      127

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


No specific version for fibers cause make error

I got error like below when "npm install -g fibers &&"

(node) child_process: options.customFds option is deprecated. Use options.stdio instead.
make: Entering directory /root/node_modules/fibers/build' CXX(target) Release/obj.target/fibers/src/fibers.o In file included from /root/.node-gyp/4.2.3/src/node.h:42, from ../src/coroutine.h:1, from ../src/fibers.cc:1: /root/.node-gyp/4.2.3/deps/v8/include/v8.h:336: error: expected unqualified-id before 'using' /root/.node-gyp/4.2.3/deps/v8/include/v8.h: In constructor

The make error got gone when I specified the version like

npm install -g<EMAIL_ADDRESS>&&

I wish this can help others.

Using<EMAIL_ADDRESS>is already documented at:
https://github.com/wekan/wekan/wiki/Install-and-Update#install-and-build-wekan
and
https://github.com/wekan/wekan/blob/devel/Dockerfile
Is there some wiki page or elsewhere that fibers version number is not mentioned?

I cloned https://github.com/wekan/wekan-dev.git and there is no specific version for fibers in Dockerfile.
What's the difference between wekan/wekan and wekan/wekan-dev?

@jameswangcnln78
wekan-dev is docker development environment that is not up-to-date.
I develop wekan on wekan/wekan installed from source.

Thanks for your relply!@xet7


Update OTEL doc mappings with date type that now supports nanoseconds

[x] Update OTEL logs index config
[x] Update OTEL traces index config

Closed via #3362.


Cannot change content src startup html

It seems like no matter what I put in the  line, my cordova app always starts at the www/index.html page.
If the src is not in the format of "http://localhost:[port]/", I would get this error:
WARNING: CordovaLocalWebServer:  tag src is not http://localhost[:port] (is index2.html).
But even if I put it in the expected format, it still only goes to index.html even though I specified index2.html.
I am testing this on iOS.
Joseph

I have the same problem, @joehui could you solve the problem?
The steps for replicate the problem in a new project are:
cordova create testContent
cd testContent/
cordova platform add ios
cordova plugin add cordova-plugin-ionic-webview@latest

change the name of file index.html to index-app.html
change the config.xml preference to
Build and execute the app, you will get a blank page
cordova build ios && xed platforms/ios
If I remove the plugin everything works as expected
cordova plugin rm cordova-plugin-ionic-webview@latest
I have the problem only in IOS.


Make state work after SSO login

This pull request rebases and replaces https://github.com/auth0/wp-auth0/pull/303.
I'm using this to redirect to a specific page after logging in via an Auth0 SSO link:
https://uuid.auth0.com/authorize/?connection_id=google-oauth2&client_id=xxx&redirect_uri=https://mywordpresssite.com/index.php?auth0=1&response_type=code&state=https%3A%2F%2Fmywordpresssite.com%2Fwp-admin%2Fedit.php%3Fpost_type%3Dpost

@jlyon - Checking back in here ... the reason I asked about your setup is that state is not guaranteed to be a URL, in fact it's not set as a URL anywhere in the plugin (we're using an object). How we're handling that needs to improve (see WIP #398) but I don't think this is the way.
More about state here:
https://auth0.com/docs/protocols/oauth2/oauth-state
I'm going to close this one out but check back in if you can help us understand how or why this works for you.


Attach the parent node to the scope

So when building instruction sequences for a scope we can reference items from the parent. This is useful for for loops, where the local variable tables from the parent scope will need to be referenced.

"parent" sounds weird since ScopeNode is not a node part of the AST so it has no "parent".
Maybe it should just be called "node" or "ast_node" or so?

"parent" sounds weird since ScopeNode is not a node part of the AST so it has no "parent". Maybe it should just be called "node" or "ast_node" or so?

Thanks for the feedback. I've used ast_node as I prefer the specificity.


Jenkins no covergage

[Current coverage][cc-pull] is 91.61%

Merging [#143][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #143   diff @@
==========================================
  Files            67         67
  Lines          4281       4281
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           3922       3922
  Misses          359        359
  Partials          0          0

Powered by Codecov. Last updated by [242af68...8d60b24][cc-compare]
[cc-base-branch]: https://codecov.io/gh/ORNL-CEES/Cap/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/ORNL-CEES/Cap/compare/242af6863b40c01f3f7e9bd1427cf016c9cff7d8...8d60b24067527b2027c6ed6574aa4ae29088a8b8
[cc-pull]: https://codecov.io/gh/ORNL-CEES/Cap/pull/143?src=pr

Merging now because all tests on Jenkins CI will fail until we do


404 for png images in nextConfig output: 'export' mode

Fantastic template, but when I had to use it outside vercel with the output: 'export' configuration in next.config.js all the png images are broken, svg files instead are rendered correctly.

Hi @sp-cfx, thanks for choosing our theme. this is expected behavior actually. please follow the instructions to get your result https://nextjs.org/docs/app/building-your-application/deploying/static-exports

Thank you, I'm new to Nextjs so I'll check the docs better then


CODE RUB: InvalidFineTuneException Class Update v2.10.1

Updated InvalidFineTuneException class by removing constructors in which the error message was hard coded in. Updated all relevant service files to include message.

Closing Pull Request. Resolved in PR #554.


[blackduck] Java projects seem to have an entry namespace: "unknown" id: "null". It crashes Spdx-builder

Command executed
blackduck -o fact-service-github.spdx forest fact-service-github

Output
Building tree of components An internal error occurred: java.lang.NullPointerException
java.lang.NullPointerException
	at com.philips.research.spdxbuilder.persistence.blackduck.PackageIdentifier.fromEnd(PackageIdentifier.java:135)
	at com.philips.research.spdxbuilder.persistence.blackduck.PackageIdentifier.namespace(PackageIdentifier.java:81)
	at com.philips.research.spdxbuilder.persistence.blackduck.PackageIdentifier.getPurl(PackageIdentifier.java:28)

Possilble fixes
Report and continue
Make sure spdx-builder reports this unknown package and continue.
Report and fail
Make sure spdx-builder reports this unknown package and stop the tool.
Try to figure out why the unknown component is in blackduck.
A lot of java applications seem to have this "unknown" package included.. maybe a misconfiguration in blackduck configuration.

Is resolved in #63


Ollama Unexpected end of JSON input No. 2

Before submitting your bug report

[X] I believe this is a bug. I'll try to join the Continue Discord for questions
[X] I'm not able to find an open issue that reports the same bug
[X] I've seen the troubleshooting guide on the Continue Docs

Relevant environment info
- OS: Win11
- Continue: v0.8.21
- IDE: vscode
- Model: Ollama

Description
Continue error: Error parsing Ollama response: SyntaxError: Unterminated string in JSON at position 75 (line 1 column 76) {"model":"xxx","created_at":"xxx:16:07.5
c @ notificationsAlerts.ts:42

I found nearly exakt same error this: https://github.com/continuedev/continue/issues/751
and the corresponding bugfix: https://github.com/continuedev/continue/pull/752
I applied to the other function and created a pull request. Works for me, but I haven't researched further.
To reproduce

ollama local
highlight text in vsocde
in continue: /edit rewrite in python
Error pops up:

Continue error: Error parsing Ollama response: SyntaxError: Unterminated string in JSON at position 75 (line 1 column 76) {"model":"xxx","created_at":"xxx:16:07.5
c @ notificationsAlerts.ts:42

no output; chatbox keeps animated

Log output
Continue error: Error parsing Ollama response: SyntaxError: Unterminated string in JSON at position 75 (line 1 column 76) {"model":"xxx","created_at":"xxx:16:07.5
c @ notificationsAlerts.ts:42

https://github.com/continuedev/continue/pull/1015


Create Hack

Yes

Why?

hi!

luv ur hacks helps a lot but not work today :(

Can you provide an error message?


build the landing page

Create the landing page with a CTA to start the process + a dropdown menu to choose the algorithm

done


Responsive members grid and member holo card

This is a down payment / progress toward making member list grids responsive and styling the individual member pages. It introduces the MemberHoloCard component too...which is sort of a visual representation of the NFT that we'll be able to use elsewhere in the application.
Even though work is not complete on the individual member pages, I wanted to go ahead and PR this so other devs have the option to base on it if desired.

Images are a bit heavy. Will address that in subsequent PR.


Replace directory-relative links with file-relative links throughout docs

Closes: #DNE
What's Changed
By design mkdocs checks the validity of file-relative internal links (../installation/index.md, models/extras/status.md) but not directory-relative internal links (../installation/, models/extras/status/). This PR changes every instance I found of the latter with a couple of quick greps to follow the former, and fixes a few broken links that mkdocs reported as a result.
Note that mkdocs does not appear to validate that linked anchors actually exist within the linked file (../installation/index.md#no-such-anchor does not report an error, so long as ../installation/index.md is a valid file path), but this at least gets us closer to the right direction.
I've also opened #2273 as a housekeeping issue to add a linting rule to explicitly disallow directory-relative links within the docs.
TODO

[x] Explanation of Change(s)
n/a Attached Screenshots, Payload Example
[x] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
n/a Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

Included as part of #2303.


IIP

Considering the possibility of having proper IIP to configure components

"IIPs are data chunks associated with a port in the network definition which become "normal" IPs when a "receive" is issued for the relevant port."
http://www.jpaulmorrison.com/fbp/examples.html

Closed in 24a8b8c

As I supected, repeating the IIP continuosuly is the best option:
https://groups.google.com/forum/m/#!msg/flow-based-programming/PyH-KKHfE1k/-aD2yywNnx4J
Thus they can be used as fixed parameters, e.g in the shell.


Show the button to toggle the list of accounts on the dashboard too

Show, on the dashboard, the accounts (and contacts) list button when there is at least 1 account or 1 contact, or the Ledger is connected.
Previously it was necessary to had selected an account. Since account are unlock when they're imported, that didn't provide any special security measure.
That previous behaviour, forced users to select an account to see their contacts (which are global, not per account), which doesn't make any sense.
In fact, this change permits a more secure way to watch the balance of an account:

I could add my addresses as contacts, without unlocking them, and see their balances easily (other wallets have a similar feature, like "Watch addresses"). It would be like a read-only access.

Apart from this change, it modifies the HTML structure, so the sidebar would be displayed on wide screens.

@luciorubeens I've fixed all the conflicts

@j-a-m-l I want to create a new dashboard with the contacts, the latest transactions, price chart, network/peer status, income/expend, notifications.. We will ask our design to do something, so for now I'm closing this.


CORE-11167 Pre-install checks for Corda Cluster

Description
This PR adds a new plugin for the Corda CLI, called preinstall, which performs pre-install checks for the Corda platform. The plugin provides several subcommands, such as check-limits, check-postgres, check-kafka, and run-all.
Motivation
The pre-install plugin is designed to help in the deployment of Corda to ensure that the Corda environment is properly configured and set up before installing the platform. By running the preinstall checks, users can more easily identify and fix potential issues with resource limitations, Postgres connectivity / DB issues, and Kafka cluster connectivity / broker issues.
Usage
The easiest way to test the preinstall plugin is using the CLI-Plugin-Host. To do this, first compile the plugin using Gradle:
./gradlew clean :tools:plugins:preinstall:build
Then, copy the build files from corda-runtime-os to corda-cli-plugin-host:
cp ./corda-runtime-os/tools/plugins/preinstall/build/libs/preinstall-cli-plugin-*.jar ./corda-cli-plugin-host/build/plugins/
Finally, run the plugin from the CLI plugin host with:
./gradlew run --args="preinstall <subcommand> [options] <path>"
Replace <subcommand> with the desired subcommand, <path> with the path to the YAML file containing the relevant configuration settings, and [options] with any optional flags supported by the subcommand. See the README.md for more info.
For example, to run all pre-install checks, use the run-all subcommand:
./gradlew run --args="preinstall run-all -n corda /path/to/config.yaml"

Just a friendly reminder that we generally squash all our PRs and prefer not to have authors like "root root@LDNW-95ZJMN3".


[Web] addEventListener$2 method not found

Example fails in profile/release mode while works in debug mode
Repo: https://github.com/woodemi/web_hid.dart
Branch: repoduce/addEventLisenter
NoSuchMethodError: method not found: 'addEventListener$2' (J.getInterceptor$x(...).addEventListener$2 is not a function)

Steps to Reproduce

Execute flutter run on the code sample
Press button hid.subscribeConnect or hid.subscribeDisconnect

Expected results:
print('hid.subscribeConnect success');
...
print('hid.subscribeDisconnect success');

Actual results:
NoSuchMethodError: method not found: 'addEventListener$2' (J.getInterceptor$x(...).addEventListener$2 is not a function)
js_primitives.dart:47     at Object.addEventListener$2$x (http://localhost:65055/main.dart.js:28209:43)
js_primitives.dart:47     at MyHomePage_build_closure0.call$0 (http://localhost:65055/main.dart.js:92205:9)
js_primitives.dart:47     at _InkResponseState._handleTap$0 (http://localhost:65055/main.dart.js:65650:27)
js_primitives.dart:47     at tear_off.<anonymous> (http://localhost:65055/main.dart.js:7277:45)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$1$3$debugReport (http://localhost:65055/main.dart.js:62520:27)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$2 (http://localhost:65055/main.dart.js:62530:19)
js_primitives.dart:47     at TapGestureRecognizer.handleTapUp$2$down$up (http://localhost:65055/main.dart.js:62849:18)
js_primitives.dart:47     at TapGestureRecognizer._checkUp$0 (http://localhost:65055/main.dart.js:62791:13)
js_primitives.dart:47     at TapGestureRecognizer.handlePrimaryPointer$1 (http://localhost:65055/main.dart.js:62720:15)
js_primitives.dart:47     at TapGestureRecognizer.handleEvent$1 (http://localhost:65055/main.dart.js:62641:17)

Code sample
Example in the repo

Logs
NoSuchMethodError: method not found: 'addEventListener$2' (J.getInterceptor$x(...).addEventListener$2 is not a function)
js_primitives.dart:47     at Object.addEventListener$2$x (http://localhost:65055/main.dart.js:28209:43)
js_primitives.dart:47     at MyHomePage_build_closure0.call$0 (http://localhost:65055/main.dart.js:92205:9)
js_primitives.dart:47     at _InkResponseState._handleTap$0 (http://localhost:65055/main.dart.js:65650:27)
js_primitives.dart:47     at tear_off.<anonymous> (http://localhost:65055/main.dart.js:7277:45)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$1$3$debugReport (http://localhost:65055/main.dart.js:62520:27)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$2 (http://localhost:65055/main.dart.js:62530:19)
js_primitives.dart:47     at TapGestureRecognizer.handleTapUp$2$down$up (http://localhost:65055/main.dart.js:62849:18)
js_primitives.dart:47     at TapGestureRecognizer._checkUp$0 (http://localhost:65055/main.dart.js:62791:13)
js_primitives.dart:47     at TapGestureRecognizer.handlePrimaryPointer$1 (http://localhost:65055/main.dart.js:62720:15)
js_primitives.dart:47     at TapGestureRecognizer.handleEvent$1 (http://localhost:65055/main.dart.js:62641:17)

flutter analyze

NONE

flutter doctor -v on Windows

[√] Flutter (Channel stable, 2.8.1, on Microsoft Windows [Version 10.0.19043.1466], locale en-US)
    • Flutter version 2.8.1 at C:\Users\sunbr\google\flutter-2.8.x
    • Upstream repository https://github.com/flutter/flutter
    • Framework revision 77d935af4d (6 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[X] Android toolchain - develop for Android devices
    X Unable to locate Android SDK.
      Install Android Studio from: https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK components.
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[√] VS Code (version 1.63.2)
    • VS Code at C:\Users\sunbr\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.32.0

[√] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 97.0.4692.99
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 97.0.1072.69

! Doctor found issues in 2 categories.

Hi @Sunbreak, Thanks for filing the issue. I am able to reproduce the issue on both stable and the master channel. The error comes only in profile and release mode and works fine in debug mode.

chrome console logs(profile mode)
Installed new service worker.
js_primitives.dart:47 NoSuchMethodError: method not found: 'addEventListener$2' (J.getInterceptor$x(...).addEventListener$2 is not a function)
js_primitives.dart:47     at Object.addEventListener$2$x (http://localhost:54367/main.dart.js:28209:43)
js_primitives.dart:47     at MyHomePage_build_closure0.call$0 (http://localhost:54367/main.dart.js:92205:9)
js_primitives.dart:47     at _InkResponseState._handleTap$0 (http://localhost:54367/main.dart.js:65650:27)
js_primitives.dart:47     at tear_off.<anonymous> (http://localhost:54367/main.dart.js:7277:45)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$1$3$debugReport (http://localhost:54367/main.dart.js:62520:27)
js_primitives.dart:47     at TapGestureRecognizer.invokeCallback$2 (http://localhost:54367/main.dart.js:62530:19)
js_primitives.dart:47     at TapGestureRecognizer.handleTapUp$2$down$up (http://localhost:54367/main.dart.js:62849:18)
js_primitives.dart:47     at TapGestureRecognizer._checkUp$0 (http://localhost:54367/main.dart.js:62791:13)
js_primitives.dart:47     at TapGestureRecognizer.handlePrimaryPointer$1 (http://localhost:54367/main.dart.js:62720:15)
js_primitives.dart:47     at TapGestureRecognizer.handleEvent$1 (http://localhost:54367/main.dart.js:62641:17)
2js_primitives.dart:47 Another exception was thrown: Instance of 'ErrorSummary'

chrome console logs (release)
Installed new service worker.
js_primitives.dart:47 NoSuchMethodError: method not found: 'eD' (J.j(...).eD is not a function)
js_primitives.dart:47     at Object.o_ (http://localhost:56473/main.dart.js:15329:25)
js_primitives.dart:47     at NJ.$0 (http://localhost:56473/main.dart.js:46859:8)
js_primitives.dart:47     at tq.wO (http://localhost:56473/main.dart.js:33719:15)
js_primitives.dart:47     at tear_off.<anonymous> (http://localhost:56473/main.dart.js:3831:79)
js_primitives.dart:47     at eM.AC (http://localhost:56473/main.dart.js:31935:9)
js_primitives.dart:47     at eM.fm (http://localhost:56473/main.dart.js:31939:21)
js_primitives.dart:47     at eM.Ah (http://localhost:56473/main.dart.js:32091:17)
js_primitives.dart:47     at eM.vN (http://localhost:56473/main.dart.js:32064:3)
js_primitives.dart:47     at eM.Af (http://localhost:56473/main.dart.js:32027:3)
js_primitives.dart:47     at eM.j1 (http://localhost:56473/main.dart.js:31992:16)
js_primitives.dart:47 Another exception was thrown: Instance of 'minified:em<void>'

debug logs
mahesh@Maheshs-MacBook-Air-M1 example % flutter run -d chrome
Launching lib/main.dart on Chrome in debug mode...
Waiting for connection from debug service on Chrome...             20.2s
This app is linked to the debug service: ws://<IP_ADDRESS>:54138/eXLVZ5oY8cE=/ws
Debug service listening on ws://<IP_ADDRESS>:54138/eXLVZ5oY8cE=/ws

💪 Running with sound null safety 💪

🔥  To hot restart changes while running, press "r" or "R".
For a more detailed help message, press "h". To quit, press "q".

An Observatory debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:54138/eXLVZ5oY8cE=
The Flutter DevTools debugger and profiler on Chrome is available at: http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:54138/eXLVZ5oY8cE=
hid.subscribeConnect success
hid.subscribeDisconnect success

flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.8.1 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (3 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.32.0

[✓] Connected device (4 available)
    • Redmi K20 Pro (mobile) • <IP_ADDRESS>:5555                     • android-arm64  • Android 11 (API 30)
    • iPhone 13 Pro (mobile) • 0FA2FB93-2A89-4FDB-8293-DF66188F1E54 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0
      (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 96.0.4664.110

• No issues found!

[✓] Flutter (Channel master, 2.10.0-1.0.pre.307, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.0-1.0.pre.307 at /Users/mahesh/Documents/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b7424c619c (7 hours ago), 2022-01-26 17:25:45 -0800
    • Engine revision c248c6e38e
    • Dart version 2.17.0 (build 2.17.0-63.0.dev)
    • DevTools version 2.10.0-dev.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.32.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 12.1 21C52 darwin-arm
    • Chrome (web)    • chrome • web-javascript • Google Chrome 97.0.4692.99

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@maheshmnj hello. have you a temporary solution to generate and deploy a build in debug mode please ?

I suspect this has to do with the JS-interop code; maybe it's attempting to extend a @Native type or hitting some of the current limitations of dart:js?
I'm going to invoke @srujzs on this issue to take a quick look, he's seen way more JS-interop code than I have, and will probably be able to spot any weirdness.

[dart rotation'] Looks similar to https://github.com/dart-lang/sdk/issues/48240

This is the relevant call.
I think this is because there isn't a @Native definition for the type HID. Even though it's an EventTarget, a class binding for the type isn't available in dart:html. What this then means on the dart2js side is that the interceptor is UnknownJavaScriptObject, the same as any other JS type that isn't bound in dart:html. Since a package:js class was never written with addEventListener$2 (the number is the number of parameters), this symbol doesn't exist on that interceptor, and therefore the call fails.
On the DDC side, it's a little more interesting. DDC installs symbols e.g. dartx.addEventListener onto the prototype of native types e.g. EventTarget. When delegate.addEventListener is called, we get delegate[$addEventListener] which is translated to delegate[dartx.addEventListener]. Here, delegate is a HID type, which is a native subtype of EventTarget. Because we install symbols onto the prototype of EventTarget, this accurately calls HID.addEventListener, leading to the successful behavior. Note that even though DDC succeeds in this manner, it isn't (imo) really intended behavior.
The correct way to handle this is to use interop to define a package:js interface (preferably @staticInterop if you want to update to version 0.64) for the native type HID and disentangle it from EventTarget. It's a bit unfortunate though, since you lose out on the polymorphism, meaning you can't use EventTarget as the type of delegate if you want to support HIDs, as the class you define is a package:js class, and EventTarget is a @Native class. @sigmundch @rileyporter , do you have a better way of potentially handling this?

@srujzs Thanks for your explaination
@staticInterop seems to be disabled according to https://github.com/dart-lang/sdk/commit/7990144badcb4b32fd1a99b0964d50e9011264a8
Any other workaround?

I think this is because there isn't a @Native definition for the type HID. Even though it's an EventTarget, a class binding for the type isn't available in dart:html.

Bluetooth has a similiar problem: https://github.com/woodemi/web_blue.dart/blob/bfc826f7064ce6566b6c84a185befd6e32f54b88/lib/src/web_blue_base.dart#L12-L19
There is a @Native definition for the type Bluetooth
@Native("Bluetooth")
abstract class _Bluetooth extends Interceptor {
}

subscribeAvailabilitychanged still fails in profile/release mode while works in debug mode
  // FIXME allowInterop
  void subscribeAvailabilitychanged(EventListener listener) {
    delegate.addEventListener('availabilitychanged', listener);
  }

It should be enabled again. That was a temporary CL until we had all the plumbing in place. It will however require updating to 2.16 and package:js version 0.64:

https://pub.dev/packages/js/versions/0.6.4#interop-with-native-types-using-staticinterop
https://github.com/dart-lang/sdk/blob/main/sdk/lib/html/doc/WORKAROUNDS.md

If that isn't possible yet, a @JS class can be used for HID, e.g.
@JS('HID')
class Hid {
  external void addEventListener(...);
}

Bluetooth has a slightly different but very similar problem. Bluetooth does not extend from EventTarget (which we should fix or remove altogether, but that doesn't entirely resolve your polymorphic case) even though a dart:html class for it exists, so addEventListener isn't visible on that class and therefore its interceptor.
For Bluetooth, however, the above alternative (using @JS() only without @staticInterop) doesn't work. This is because @Native classes reserve the native type, unfortunately. @staticInterop will resolve your issue with Bluetooth as well e.g.
@JS()
@staticInterop
class JSEventTarget {}

extension JSEventTargetExtension on JSEventTarget {
  external void addEventListener(...);
}

Using JSEventTarget as the type of delegate will get you the polymorphism you desire here. If @staticInterop isn't possible, the best alternative might be dart:js_util here. You would simply replace addEventListener calls on delegate with js_util.callMethod(delegate, 'addEventListener', ...).

@JS('HID')
class Hid {
  external void addEventListener(...);
}

https://github.com/woodemi/web_hid.dart/commit/d41ec950981a5c2f24442f0b239b21b03b975db7
It works. Thx

https://github.com/woodemi/web_hid.dart/commit/d41ec950981a5c2f24442f0b239b21b03b975db7
It works. Thx

Closing as fixed.


feat: add list options API

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

Close https://github.com/kcl-lang/kcl-go/issues/254
feat: add list options API

2. What is the scope of this PR (e.g. component or file name):

kclvm/api/src/service/capi.rs
kclvm/api/src/service/jsonrpc.rs
kclvm/api/src/service/service_impl.rs
kclvm/loader/src/option.rs

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

kclvm/api/src/testdata/option/main.k
kclvm/loader/src/snapshots/kclvm_loader__tests__builtin_call_0.snap
kclvm/loader/src/snapshots/kclvm_loader__tests__builtin_call_1.snap
kclvm/loader/src/snapshots/kclvm_loader__tests__builtin_call_2.snap
kclvm/loader/src/snapshots/kclvm_loader__tests__list_options_0.snap
kclvm/loader/src/snapshots/kclvm_loader__tests__list_options_1.snap
kclvm/loader/src/snapshots/kclvm_loader__tests__list_options_2.snap
kclvm/loader/src/tests.rs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

123 of 212 (58.02%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.145%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

kclvm/loader/src/util.rs
56
57
98.25%

kclvm/api/src/service/capi.rs
1
4
25.0%

kclvm/runtime/src/stdlib/builtin_api.rs
0
3
0.0%

kclvm/loader/src/option.rs
35
39
89.74%

kclvm/api/src/service/jsonrpc.rs
0
7
0.0%

kclvm/api/src/service/service_impl.rs
0
29
0.0%

kclvm/sema/src/builtin/option.rs
0
42
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
44914

Relevant Lines:
62255

💛  - Coveralls


unixPB: Remove gcc-7 package install as we use our self-built one

Not only do we use our self-built one, but GCC-7 is not available in the Ubuntu 22.04 repositories, so the playbook fails. We could bypass this on 22.04 and later, but since we do not use it for building it is simpler to remove it. We install the gcc package elsewhere so we will always have a system installed compiler even with this change.
Signed-off-by: Stewart X Addison<EMAIL_ADDRESS>
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[x] playbook changes run through VPC or QPC (if you have access)  (1492 failed due to https://github.com/adoptium/infrastructure/issues/2692, 1493 is testing with JDK11)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Failing MacOS and Solaris checks are nothing to do with this PR ...

So Debian 10 and Ubuntu 16.04 both fail in VPC with this as they no longer have /usr/bin/gcc-7. Ubuntu 16.04 is already out of support, and Debian 10 also goes out of support this month, so I'm not sure I'm too concerned about those failures. Reviewer thoughts welcome.

Adjusted conditionals so the gcc_7 role is activated on Ubuntu16 and Debian10 across all architectures, not just armv7l (arm32)

Reinstated gcc7 on Ubuntu16 and Debian10 in latest commit due to VPC failures

Final VPC Excluding Ubuntu 21.10 which is broken as per another issue: https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1503/

Final VPC Excluding Ubuntu 21.10 which is broken as per another issue: https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1503/

Two failures to download prerequisite products (cmake and expat) - running again at https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1509


Getting Cannot read property 'forEach' of undefined

I am getting error from this piece of code.
`    _handleMessage (json) {
let entries = json.entry
entries.forEach((entry) => {
  let events = entry.messaging

  events.forEach((event) => {
    // handle inbound messages and echos
    if (event.message) {
      if (event.message.is_echo) {

`
events is undefined.
What could be the issue here?

I am getting the same error and it happens when you receive postback-type of message from Facebook.


Switch alertmanagerconfig to use dispatch with { root:true }

The utility was broken in prod due to the function names changing. Instead of trying to differentiate the dispatch method being used we just set the { root:true } config so we can always use the 'cluster/action'.
https://github.com/rancher/dashboard/issues/1238#issuecomment-719730626

https://github.com/rancher/dashboard/issues/1239


ENH: Better error messages

Closes https://github.com/TomAugspurger/engarde/issues/7
This is a first attempt at some better error handling.
I don't really consider the exact error message to be
part of the public API, so don't rely on them.
Still need to do is_monotonic.
Need to protect against printing out massive arrays.

cc @ianozsvald
Pretty rudimentary for now, but better than nothing.
In [1]: from engarde.checks import none_missing

In [2]: df = pd.DataFrame({'A': [0, np.nan, -1, -2], 'B': [0, 2, -3, np.nan]})

In [3]: none_missing(df)
---------------------------------------------------------------------------
AssertionError                            Traceback (most recent call last)
/Users/tom.augspurger/sandbox/engarde/engarde/checks.py in none_missing(df, columns)
     22     try:
---> 23         assert not df[columns].isnull().any().any()
     24     except AssertionError:

AssertionError:

During handling of the above exception, another exception occurred:

AssertionError                            Traceback (most recent call last)
<ipython-input-3-563ad7dea610> in <module>()
----> 1 none_missing(df)

/Users/tom.augspurger/sandbox/engarde/engarde/checks.py in none_missing(df, columns)
     27         bad = pd.Series(list(all_locs))[np.asarray(missing).ravel(1)]
     28         msg = bad.values
---> 29         raise AssertionError(msg)
     30     return df
     31

AssertionError: [(1, 'A') (3, 'B')]

I'm planning to cleanup the assertion reraise so there aren't nested AssertionErrors and the traceback will be cleaner.

Looks like
except AssertionError as e:
    e.args += (info,)
    raise

Should work. Will update later today.

Cool, will give the improvements a go, cheers!

One comment - if there's a lot of failures (e.g. 200 rows in a dataframe with a None), you get a lot of debug output. Maybe truncate to a fixed character length in the output? Regardless this is an improvement, cheers!


licensing

Everyone's favorite topic!
I was just poking through the code, and noticed what seem to be some ambiguity in licensing, comparing the top-level license to other bits of the code which seem to be GPL (eg, in parsers/*.py).
Maybe it's worth thinking about picking a single license to use across the board?
GPL is probably not a great idea here, given that it's a library that could be embedded in all kinds of things.  I would recommend a BSD-style license, having just gone through all this for librosa.  See also: http://choosealicense.com/ .
Maybe put it to a vote?  Just opening up the discussion here.

So.. is everyone cool with BSD2/ISC?

:+1:


"collection of useful apps"?

I only see 1,  schedy.  and that hasn't been updated for a couple of years.   Whats the deal?  have you given up on making a collection?

Welcome and thanks for your kind inquiry.
Schedy is currently the only app in hass-apps package, you recognized this perfectly.
Back when I started working on it, the plan was to create a design which allows to add more apps in the future, but all in all I'm not so happy with the single-package approach anymore.
Other than that, this is a duplicate of #66.


Complete code coverage

Only critical parts of the library has test now. Need to test all the things.

achieved!


fix a handful of backend bugs

Replace backend.IsHealthy with backend.Health, because it returns an
enum of 3 states (unknown, healthy, unhealthy) and not a bool.
When populating the backend, the runtime will return FastlyStatusNone
if a value isn't set.  This isn't an error, so we should ignore it.
Similarly, the Viceroy environment will return FastlyStatusUnsupported
for some unsupported fields and we should ignore them too.
The exported names for getting SSL min/max versions were copypasta'd and
returned the wrong values.
Fixes #65.

I found most of these with a new integration test I'll be adding to #63 once I've rebased it on this branch.


why pkeyutl: Unknown option: -sig ? openssl-3.0.0-alpha6

Hi,
I am trying to use openssl-3.0.0-alpha6 for signing using Ed25519.
I downloaded the archive and built it from sources for linux-x86_64
after building I installed it in the /home/user/openssl directory.
after that I exported LD_LIBRARY_PATH: export LD_LIBRARY_PATH=/home/user/openssl/lib.

to gen secret key: /home/user/openssl/bin/openssl genpkey -algorithm Ed25519 -out priv.pem - OK
to gen public key: /home/user/openssl/bin/openssl pkey -in priv.pem -pubout -out pub.pem - OK
to sign: /home/user/openssl/bin/openssl pkeyutl -sign -inkey priv.pem -out sig.dat -rawin -in /bin/ls - OK
to verify: /home/user/openssl/bin/openssl pkeyutl -verify -pubin -inkey pub.pem -rawin -in /bin/ls -sig sig.dat - ERROR

pkeyutl: Unknown option: -sig

what am I doing wrong?
thanks!

Try -sigfile instead

yes, it works, thank you @beldmit !


:+1: DX | Making sure storybook can be started encapsulated from the docs.

Summary
Hey :wave: ,
Cool stuff we have here. I wanted to contribute slightly to the DX. Currently we have two apps who live inside of this monorepo:
docs
storybook
They're (to my understanding) supposed to be able to run and live in parallel. I noticed that when starting the storybook in preview mode, that was not the case, as the build would fail, because of a built resource from Gatsby would be missing, and the -o argument inside of the storybook build script would fail.
This fixes that, and allows devs to just start storybook, avoiding a race condition. Vice versa, it does not mess with the docs built progress.
List of notable

updated: The build-storybook command not looks for if a directory $STATIC_DIR exists, if not, it creates it. Thanks to the -p command, we don't have to create a side effect to create it.
updated: The documentation inside of  apps/docs/README.md as it references a command which does not exist.

What should reviewers focus on?
Ive tested pretty thoroughly that the built is not encapsulated in such a way that we can start storybook without messing with the Gatsby build, for quick and easy previewing.
Steps to test:

npm i (in root)
npm run build:storybook:preview (in root)
npm run start:storybook (in root)

On main:
ERR! Failed to load static files, no such directory: [outside of module]/apps/docs/public/static
On branch:
:trumpet: We get a preview of storybook!
Supporting resources (related issues, external links, etc):
Contributor checklist:

[x] All new and existing CI checks pass
[x] Tests prove that the feature works and covers both happy and unhappy paths
[x] Any drop in coverage, breaking changes or regressions have been documented above
[x] New visual snapshots have been generated / updated for any UI changes
[x] All developer debugging and non-functional logging has been removed
[x] Related issues have been referenced in the PR description

Reviewer checklist:

[ ] Check that pull request and proposed changes adhere to our contribution guidelines and code of conduct
[ ] Check that tests prove the feature works and covers both happy and unhappy paths
[ ] Check that there aren't other open Pull Requests for the same update/change

Screenshots:

Please try to provide before and after screenshots or videos

Before:
sad.webm
After:
happy.webm

Sorry @rezrah, I just updated the branch, mind triggering the workflows again, so I can see whats wrong and fix it?

I'd love to fix those failing actions, but I do not think I can. It looks like they are asking for secrets.
Ui test, Visual test, & Deploy all seem to use the action<EMAIL_ADDRESS>and probably relies upon a secret.
But the secret should be tied to the repository, right? Hopefully not the PAT?


UI: Ability to create private clusters in GKE

UI for https://github.com/rancher/rancher/issues/13980

Version: Master 2.2 (2/14/19)
Tested UI. Validated this section (Advanced options) under Cluster Options.

Validated API/HTTP.


Registrator does not work when using Docker with a tcp port

I am trying to get up to speed with Swarm and Docker. The documentation asks that Docker be run and expose a TCP port, using the following option: '-H tcp://<IP_ADDRESS>:2375
From what I can see, this means that the socket file in /var/run/docker.sock does not get created.
Every piece of documentation I've seen on Registrator says to start it with a command like this:
docker run -d --name registrator -h <HOST_NAME> -v /var/run/docker.sock:/tmp/docker.sock gliderlabs/registrator consul://<HOST_NAME:8500

This works fine when using the default docker settings, but as mentioned above, does not work if using Docker with a TCP port. Am I totally missing something, or does this in fact mean I cannot use registrator together with swarm? Thank you.

I was wondering the same myself. Any ideas?

The bigger issue (as I see it) is that registrator can only be setup to run on each host and only registers containers on the local docker daemon with the local consul agent. The issue here is that if for some reason the docker daemon gets restarted, or registrator + other containers crash, we wind up with a situation where any containers that went down while registrator was down never get cleaned up. This has caused us a number of headaches in production where consul is still serving dns for records that no longer exist. I realize this can be cleaned up with healthchecks but I think it masks the underlying issue.
The way I'd like to set this up is to have Registrator running on each of my swarm managers and pointing them at the swarm manager API instead of just the local docker via /var/run/docker.sock. That way if anyone of them went down we would never wind up in a bad state because the other instances would still be keeping track of what is running and what is not.

Yeah. I'm trying to accept that people will be running single instance Registrator deployments. It was not designed for it, but is not practical to run as an agent per host in situations where you don't necessarily have access to the host (Joyent, Swarm users). Sigh.

@progrium Are there any issues running it as I've suggested and pointing it at the swarm? Currently we are running it on each host but we have continually run into issues where docker has become unstable, it restarts, and consul keeps it's data but registrator forgets everything and therefore never removes containers that are no longer running on the host.

I'd rather solve that issue than recommend running it as a single point of failure that will have trouble scaling to large clusters. If you can do it, the agent pattern is really ideal to me. It's easier to set up hosts to always run it as opposed to running one and deciding where it should run and dealing with failures. And you only have to worry about scaling to the events of a single host. That said, I know you can't always run it that way, so I'm more open to alterations to support single instance deployments.

As for being HA I think having the ability to run it anywhere on any server and run multiple instances would solve the problem as long as they point to a global docker state and not just the state of the local docker instance. This is how I'm successfully running mesos-consul. In that case mesos-consul gets all of its state from a single endpoint and dumps it into consul. I just run a single instance on each of the masters (5 node cluster). They all read the same data and send it to consul. If one goes down we never lose anything and there is no harm in trying to register duplicate services because consul just ignores/updates it.
It would be great if we could run registrator the same way. The problem I ran into just now testing it out though is that registrator doesn't know which IP to assign to a service so it just assigns the one that registrator is running on which in this case would be wrong.
When running inspect on a container via the swarm-manager API it returns information about the node that it's running on:
    "Node": {
        "ID": "O3SB:L4HJ:6A7E:2FO5:3OJV:VVOC:7RKH:ABPI:RHLP:MHHM:BARO:WVBG",
        "IP": "<IP_ADDRESS>",
        "Addr": "<IP_ADDRESS>:2375",
        "Name": "ip-192-168-0-15",
        "Cpus": 16,
        "Memory":<PHONE_NUMBER>40,
        "Labels": {
            "executiondriver": "native-0.2",
            "kernelversion": "3.10.0-229.20.1.el7.x86_64",
            "operatingsystem": "CentOS Linux 7 (Core)",
            "storagedriver": "devicemapper"
        }
    },

It would be great if registrator would attempt to use that information if it was available. This way we could pass in the -e DOCKER_HOST=tcp://swarm-manager:3375 to the container, and no matter where it runs as long is it can reach the swarm-manager it can register the right IP for all of the containers running.

@progrium Should I open a separate ticket for this? Do you have any feeling as to the best way to support this scenario? Thanks!

As far as I know there are other open issues about it but I haven't had time to investigate. Maybe you can look into issues and see which ones are relevant. If there aren't, then make an issue.

I prefer to have the registrator on every Swarm "node" running.  Thereby you have scalability by design.

Swarm manager is meant to be run on multiple nodes as well so if you had install it on every manager node you have it by design that way as well. Ultimately the issue that we have consistently run into is registrator losing track of what it has registered and we wind up with lots of orphaned services in Consul.

@micahlmartin :
Setting the -e DOCKER_HOST=tcp://<IP_ADDRESS>:2375 does help but I've got the same issue about "orphaned services" in Consul.    :(

Hello everyone.
I'm trying to connect the registrator to swarm master but it fails to connect with following error:
Get https://<IP_ADDRESS>:3376/containers/json: x509: certificate signed by unknown authority
Starting the registrator container with following command:
docker run -it --net=host -v /var/lib/boot2docker/ca.pem:/certs/ca.pem -v /var/lib/boot2docker/server.pem:/certs/cert.pem -v /var/lib/boot2docker/server-key.pem:/certs/key.pem --nam
e=registrator -e DOCKER_TLS_VERIFY="1" -e DOCKER_HOST="https://<IP_ADDRESS>:3376/" -e DOCKER_CERT_PATH="/certs/"  gliderlabs/registrator:latest consul://<IP_ADDRESS>:8500

Hi all,
I've been revisiting all this given Docker's 1.10 release and the new version of Swarm. I wanted to note the solution to my issue here since I did end up resolving it. The key is that you need to expose the Docker port BOTH as a TCP port as well as a socket. For instance, like this:
-H tcp://<IP_ADDRESS>:2375 -H unix:///var/run/docker.sock

Docker will happily expose its port both ways, and Registrator runs without a hitch. Given that this is not really an issue, I'm going to close this now. Thank you.

@micahlmartin @progrium What is the status of this proposal?
1 doubt: "-e DOCKER_HOST=tcp://<IP_ADDRESS>:2375", what does the <IP_ADDRESS> exactly do? since registrator is running inside container, will <IP_ADDRESS> make it connect to the docker engine which is running out of the container?
If the above doubt do work, then one more thoughts: since the swarm manager can be always changing, if i do something like this:
--mode global \
--constraint 'node.role == manager' \

when all the initial managers be changed to worker, how swarm and registrator will react? i will test a bit then come back to update here.

conclusion:
1. -e DOCKER_HOST=tcp://<IP_ADDRESS>:2375, the **<IP_ADDRESS>** has nothing magic, totally nonsense here.

2. currently in swarm mode, -e DOCKER_HOST=tcp://<docker engine host ip>:2375 totally doesn't work, will receive no events on the overlay network.

3. in swarm mode when a node is changed from worker to manager,  then the --constraint 'node.role == manager' service will be automatically extended to this node, this is good.

So just wait registrator can catch the events up from the swarm manager.


GstreamerMediaPlayer: Fix callback function use-after-free

Allocate a copy of the callback function and promise, and keep it until it
was executed. This fixes a problem where the callback function could
be freed as soon as the callback function sets the promise value and
the thread waiting on the promise exits the function immediately, freeing
the closure data while it is still executing.
We're trying to use the SDK in an x86-32 environment (not using the sample app), but are running into issues that appear to be timing sensitive bugs. This environment is considerably faster than the preferred arm-based Raspi-based setup, which is likely a key factor in reproducing this problem. It would randomly cause Alexa to stop responding and be stuck in "thinking" state, no longer accepting any requests. Usually after making no more than 5-15 requests, which can also easily be reproduced in an automated fashion.

Hi @tbluc4
Thank you for the pull request. Our process is to make changes internally and run all test before pushing the change to github in the form of a new release. I have added this to our internal tracking system to be assessed and prioritized.

Hi @tbluc4
Thank you for the pull request. Our process is to make changes internally and run all test before pushing the change to github in the form of a new release. I have added this to our internal tracking system to be assessed and prioritized.


Also push MAJOR and MAJOR.MINOR tags when releaseing

Summary

As decided in RFC 39
Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[x] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Hi @phil9909! This looks really great!
The changeset you have here will cover what we call the "implementation" buildpacks. This is stuff like node-engine, go-build, or passenger. Each one of these buildpacks does something specific in the build process. We also have "language-family" buildpacks like nodejs or dotnet-core that group these implementations into logical orders to build whole applications. These language-family buildpacks also have their own buildpackage that gets released and a workflow that is similar to the one you modified (https://github.com/paketo-buildpacks/github-config/blob/main/language-family/.github/workflows/push-buildpackage.yml).
Would it be possible to get the same set of changes included in this other workflow as well?

Would it be possible to get the same set of changes included in this other workflow as well?

Done


package-lock.json is out of sync

On freshly installing the npm packages, the package-lock.json get out of sync and deviated from git history.

I did a fresh install and did not get this issue. Please share how to duplicate.

Could it be a windows only issue?

Hey @interactivellama, I think the file is synced again in the commit that was added after the opening of this issue. So, I am closing this issue.

Great! Thanks for following up.


Sass function bug

original:
@function svg($name){
  @return url($svgPath + $name + '.svg');
}

after csscomb:
@function svg($name) {
    @return url($svgPath ) $name + '.svg';;
}

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/39222113-sass-function-bug?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).

Hi @sergey-kozlov!
This seems to be fixed in the latest release 🎉


where is example folder?

sorry, I do not found example folder

This is very much in the early stages of development. There is alot of work to do on iOS side while android is almost complete. So there is no example folder currently. It will be up in a few weeks along with proper documentation.


[SWIG] Add support for int64_t ChunkedArray

Refer to https://github.com/microsoft/LightGBM/pull/3997#issuecomment-801416336.
https://github.com/microsoft/LightGBM/blob/2e54d5f518a4c7db721483347bec6676a8d20c89/swig/ChunkedArray_API_extensions.i#L16-L21
cc @AlbertoEAF

Closed in favor of being in #2302. We decided to keep all feature requests in one place.
Welcome to contribute this feature! Please re-open this issue (or post a comment if you are not a topic starter) if you are actively working on implementing this feature.


Limitations of Device Filtering

Kia Ora, Just want to understand how the the requests are processed for Conditional Access with Device Filtering.
Looking at utilizing Hybrid Azure AD joined Servers (created via Azure AD Connect) as secure access workstations. I have got the Extension attribute populated via Graph as per #77551 (as Transforming AD Attributes via AADC seems to be nigh on impossible), but this doesn't seem to have any impact on controlling access.
I am using a Block Access control with Exclude Filtered Devices from Policy and a Positive operator of ExtensionAttributeX Contains "String", this meets the "Registered device not managed by Intune" scenario.
Reviewing the sign-in Logs doesn't seem to have a way that we are able to deep dive into how the request is formed and interpreted by the device filter policy for granular troubleshooting. A comment in the documentation about how to analyze authentication requests would be of assistance.

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

ID: fcc79933-58e1-66d1-81ae-f43802d41223
Version Independent ID: 5be1fefc-356e-d1eb-ab97-8cbe5654ff43
Content: Filters for devices as a condition in Conditional Access policy - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-condition-filters-for-devices.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@mrhannah88 , thank you for your feedback. We'll review this and get back to you shortly!

Reviewing signin logs in my Log Analytics Workspace
SigninLogs
| where UserDisplayName contains "USERNAME"
| where ResultType startswith "53003"
Doesn't seem to contain any detail about the relevant filters for the Device Filtering policy.
DeviceDetail | {"deviceId":"","operatingSystem":"Windows 10","browser":"Edge 91.0.864"}
There is nothing here regarding ExtensionAttrtibute, DisplayName or any of the other Supported device attributes from the table

@mrhannah88 This is assigned to the author to review and update.

@MicrosoftGuyJFlo Any thoughts?

Asked the PG for feedback

This is a known gap for the feature preview that is in the Product Team's backlog to address. #please-close


How the system_message in get_chat_template works?

I add the parameter like this.

but the final text is

Is the system_message useful? How can I see the final system_message that is thrown into the model? Thank you much!

Sorry on the delay! Yep that's Llama 3.1's default system instruction!
But interesting on why dfffff did not show up - let me fix this - sorry on the issue!

@danielhanchen Hi, did you fix this error? I face same error, can't add my custom system_message when calling get_chat_template
Thanks!

@Erland366 would you be able to look into this - thanks :)

will check them

@Erland366 After reviewing and running code with your new file chat_templates.py it show error. I use qwen2.5 model.

@tadinhkien99 Oh sorry for the type hinting. now it's removed
Can I see what's your input? because I tested mine and it works .-.

@Erland366 I just tried again and it works on my end. It seems there might be some residual spaces at the end (|im_end|). Could you please check and let me know if that resolves the issue?

Thanks!

Not happening in my case tho .-. this is weird
I suspect maybe if user call get_chat_template twice using the same tokenizer then these weird behavior appear .-.
Will check it later tonight


added separate QUEUED and RUNNING states to task status

Summary
Added the suggested code to the specified file.
Details and comments
Issue: https://github.com/qiskit-community/qiskit-braket-provider/issues/46

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Hey @colegibbs!
Thank you for contribution!
PR looks good! We can merge it once you sign CLA :)

@colegibbs hey! Any updates on signing contributor agreement? :)


System Error and then project was deleted

Describe the bug
Was in project (sb1-ra3pxd) and then hit a system error in the middle of executing a normal prompt. Bolt went to a screen that had only 5 lines of red text. Did not copy, but when returning to Bolt, project was gone. Was resuming a project from a few weeks ago. When opening that project on Bolt, it opens, but there is no code.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-ra3pxd
Steps to reproduce
Not sure how to reproduce
Expected behavior
Expected Bolt to edit code like normal when given a prompt
Screen Recording / Screenshot
No response
Platform

OS: macOs
Browser: Chrome

Additional context
No response

thankfully, this resolved itself after a few minutes


bug: user has to sign in again after the app is closed on iOS

Describe the bug
If a signed in user completely closes the app and then re-opens it on iOS, the authStateChange listener returns a null user record and the user is forced to sign in again. It seems like the user is not being persisted to the phone.
To Reproduce
Create a basic iOS capacitor app and try it - If it works I have no idea how.
Expected behavior
Once registered, the authStateChange listener should return a user after the app is re-opened. It only ever returns null.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

N/A

Smartphone (please complete the following information):

Device: iPhone 15 Pro Max (simulator)
OS: 17.4
Browser: whatever capacitor uses by default in the webview
Version: N/A

Additional context
skipNativeAuth: false in the config
"node_modules/@capacitor-firebase/authentication": {
      "version": "6.1.0",
      "resolved"<EMAIL_ADDRESS>      "integrity": "sha512-mJbbCJFhKi/aKNuoDKAgC98EgRSfg0LxoM5lDoGqQrMwp75WDvOSxsjMUMtuuKFKJ9ASofnVYvJ3aigCpLyssw==",
      "funding": [
        {
          "type": "github",
          "url": "https://github.com/sponsors/capawesome-team/"
        },
        {
          "type": "opencollective",
          "url": "https://opencollective.com/capawesome"
        }
      ],
      "license": "Apache-2.0",
      "peerDependencies": {
        "@capacitor/core": "^6.0.0",
        "firebase": "^10.9.0"
      },
      "peerDependenciesMeta": {
        "firebase": {
          "optional": true
        }
      }
    },

We have confirmed that everything is set up fine - we can log in with email/password however the user doesn't stay logged in.
We also have the firebase js sdk installed which works fine on the web. We are going to isolate out just the capacitor firebase package and see if the problem is conflicting firebase sdks, but considering the documentation states they can be used in tandem, I'm not so sure this is the problem.
Note: I just tested on android and the code I have works fine. This seems to be an iOS issue

Turns out we had both version 11 and version 10.20 installed in the same directory (unsure how) so that was the issue.


'ascii' codec can't decode byte 0xef in position 0: ordinal not in range(128)

Traceback (most recent call last):
File "/home/byang/gnrt/tranX/exp.py", line 569, in
train(args)
File "/home/byang/gnrt/tranX/exp.py", line 47, in train
train_set = Dataset.from_bin_file(args.train_file)
File "/home/byang/gnrt/tranX/components/dataset.py", line 33, in from_bin_file
examples = pickle.load(open(file_path, 'rb'))
UnicodeDecodeError: 'ascii' codec can't decode byte 0xef in position 0: ordinal not in range(128)
Have any suggestions for this problem?
the datasets downloading from http://www.cs.cmu.edu/~pengchey/tranx.0.2.0.zip.
and the tranx.0.2.1.zip.
the above dataset both have the Error ."ascii' codec can't decode byte 0xef in position 0: ordinal not in range(128)"

This error appear on all datasets except conala


Included the missing dolphin.cpp and hpp files.

This PR is connected to: https://github.com/h4570/tyra/issues/12


Build VeloView problem in Window 7

Hello:
I try to build VeloView in Window 7 32bit, Visual Studio C++ 2008 Environment.
but I get some error like as below.
Can anyone know how to building correctly.
Nelson
-----command-----
git clone git://public.kitware.com/VeloView.git
mkdir build
cd build
cmake -DENABLE_veloview:BOOL=ON ../VeloView/Superbuild
 Building program using Visual Studio C++ 2008 (debug mode)
-----Environment-----
Git version 2.7.2.window.1
CMake 2.8.8
----- error message-----
// -----------------------
CMake Error at python-stamp/download-python.cmake:6 (file):
file DOWNLOAD MD5 mismatch
for file: [C:/build_5/downloads/Python-2.7.3.tgz]
expected MD5 sum: [2cf641732ac23b18d139be077bd906cd]
actual MD5 sum: [d41d8cd98f00b204e9800998ecf8427e]
Project : error PRJ0019: 工具傳回錯誤碼"Performing download step (download, verify and extract) for 'python'"
//-----------------------
Project : error PRJ0019: 工具傳回錯誤碼"Performing configure step for 'qt'"
//------------------------
libs\regex\build\has_icu_test.cpp(12) : fatal error C1083: 無法開啟包含檔案: 'unicode/uversion.h': No such file or directory
//------------------------
error: Qt4 was not found on your system.  You probably need to set the
QT_QMAKE_EXECUTABLE variable
//------------------------

Could NOT find Qt4 (missing:  QT_QMAKE_EXECUTABLE QT_MOC_EXECUTABLE QT_RCC_EXECUTABLE QT_UIC_EXECUTABLE QT_INCLUDE_DIR QT_LIBRARY_DIR QT_QTCORE_LIBRARY)
CMake Error at CMakeLists.txt:70 (message):
error: Qt4 was not found on your system.  You probably need to set the
QT_QMAKE_EXECUTABLE variable
-- Configuring incomplete, errors occurred!
Project : error PRJ0019: 工具傳回錯誤碼"Performing configure step for 'pythonqt'"

//------------------------
-- Could NOT find Qt4 (missing:  QT_QMAKE_EXECUTABLE QT_MOC_EXECUTABLE QT_RCC_EXECUTABLE QT_UIC_EXECUTABLE QT_INCLUDE_DIR QT_LIBRARY_DIR QT_QTCORE_LIBRARY) (Required is at least version "4.7.0")
CMake Error at CMakeLists.txt:441 (message):
Qt 4.7.0 or greater not found.  Please set the QT_QMAKE_EXECUTABLE
//------------------------
CMake Error at VTK/CMake/FindPythonLibs.cmake:120 (FILE):
file STRINGS file "C:/build_5/python/src/python/Include/patchlevel.h"
cannot be read.
//------------------------
CMake Error at C:/Program Files/CMake 2.8/share/cmake-2.8/Modules/FindPackageHandleStandardArgs.cmake:97 (MESSAGE):
Could NOT find PythonInterp: Found unsuitable version "1.4", but required
is at least "2" (found C:/build_5/python/src/python/PCbuild/python.exe)
//------------------------
CMake Error at CMakeLists.txt:21 (find_package):
By not providing "FindParaView.cmake" in CMAKE_MODULE_PATH this project has
asked CMake to find a package configuration file provided by "ParaView",
but CMake did not find one.

Hi Nelson, here are the steps I use to build on Windows:
For Windows, you must use Visual Studio 2008. Looks like you're already using that. You should NOT use any USE_SYSTEM options.  The build will take a long time to complete because it has to build both Qt and ParaView from scratch. On my machine it can easily take half a day.
The steps:

From the Start Menu, find the Visual Studio 2008 x64 Cross Tools Command Prompt and start it.
Run cmake-gui
Select the ninja generator
Set the CMAKE_BUILD_TYPE to Release
Set ENABLE_veloview to ON
Set the NMAKE_PATH variable to C:\jom\jom.exe

Generate, change to build directory, then run ninja.
Can you give that a try and let me know what happens?

hi timthirion:
Thank you for your prompt reply.
I have try your method and only modify NMAKE_PATH variable  to
C:/Program Files/Microsoft Visual Studio 9.0/VC/bin/nmake.exe
The ninja build about one ~ two hour and show some error messages in the end.
Nelson
----- error message-----
NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio 9.0\VC\BIN
\cl.EXE"' : 傳回碼 '0x2'
Stop.
NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio 9.0\VC\bin
\nmake.exe"' : 傳回碼 '0x2'
Stop.
NMAKE : fatal error U1077: 'cd' : 傳回碼 '0x2'
Stop.
NMAKE : fatal error U1077: 'cd' : 傳回碼 '0x2'
Stop.
ninja: build stopped: subcommand failed.

That error message means NMake cannot find your compiler. Make sure you are using the Visual Studio 2008 x64 Cross Tools Command Prompt, instead of the usual Windows command prompt. There should be a shortcut in your Start menu under the Visual Studio folder.

hi timthirion:
I have builded success in Win 64 bit.
I change OS to Win7 64bit then the build error resolved. (NMAKE : fatal error U1077: '"C:\Program Files\Microsoft Visual Studio 9.0\VC\BIN \cl.EXE"' : 傳回碼 '0x2').
And when I build using ninja in WIn 7 64 bit .
the error happened (platform\DefaultLocalizationStrategy.cpp(327) : error C2001: 常數中包含新行字元).
The solved method is "return WEB_UI_STRING("Look Up “”", "Look Up context menu item with selected word").replace("", truncatedStringForLookupMenuItem(selectedString));
==> """
and the veloview build success.
Do you build veloview success in Linux ?
Thank you very much.
Nelson

I have not built VeloView on Linux but will attempt it soon. I'll leave this issue open for now and update when I know how feasible the Linux build is.

Hi @ctwu0314, here are the updated instructions for building on Linux: https://github.com/Kitware/VeloView/blob/master/Documentation/VeloView_Developer_Guide.rst#linux-build-instructions
Since this issue concerns building on Windows, I will close it.

hi timthirion,
I have builded success in Win10 64 bit and Nmake  generate.But when I open the Veloview.exe of ...install/bin,It shows the lack of python27.dll.
Then I copy the dll to ...install/bin,but Veloview.exe doesn‘t start and show nothing.
Is it a system problem? Or other?

hi timthirion,
I have builded success in Win10 64 bit and Nmake generate.But when I open the Veloview.exe of ...install/bin,It shows the lack of python27.dll.
Then I copy the dll to ...install/bin,but Veloview.exe doesn‘t start and shows nothing.
Is it a system problem? Or other?

Hello:
I have builded success in Win10 64 bit and Nmake generate.But when I open the Veloview.exe of ...install/bin,It shows the lack of python27.dll.
Then I copy the dll to ...install/bin,but Veloview.exe doesn‘t start and shows nothing.
Is it a system problem? Or other?


Improve guide example

My thought is to improve the guide usability with a very simple method, you already have the configuration examples which are great and you have a demo which is also great but I feel like a crucial component there is missing.
You should input a code block into the guide and/or demo site that references the exact configuration utilized in the demo site.
As an example, I'm looking into the same sort of segmentation of sidebars as on the demo site, with the "CSS" "Javascript" and "Markdown index" sections being in bold with rulers above them separating the sidebars but can't easily figure out whether that's performed through the markdown syntax in titles or by utilizing multiple sidebar configuration displayed on the same site.
Having the code snippet used on the demo site would immediately show me how it's performed.

Hello, thank you for your valuable feedback.
As of 2024, the documentation page has undergone a major overhaul, removing ambiguous descriptions and making it easier to find options.
I don't yet have code examples for each option, but this will be added gradually as needed.
I'm closing this issue because it's more or less up to date and i can't link to an actual commit.
However, i will continue to improve the documentation in the future.
Regards,


transfer market crash

READ BEFORE YOU ADD AN ISSUE: Do not request autobuyer features, because they are considered as cheating. Issues regarding autobuyers will be closed immediately.
Expected behavior:
transfer market search should work
Current behavior:
transfer market search dosn't work
Metadata:

**Script version:1.4.1
**Browser:chrome
**OS:windows 10

Please provide console or error messages if there are any. It's working as expected on my end.


Can't save passwords on M110 zip version

Nothing gets saved...
It also wiped the cookies of the previous installs I had, I assume something changed with the local encryption.

Yea I removed the Disable Encryption one and I can save passwords again and as well as my cookies survive closing the browser now (its something I noticed after that all my logins were gone). Disable Machine Id is still enabled, I dunno if it has any effect by itself (as in I'd like to be able to transfer the browser between different computers).

Ok I tested a fresh install and it seems everything works properly. Seems like this new version just didn't like my old profile, probably due to the mix of encrypted and non-encrypted stuff. I'll try to restart my profile from scratch and see if I notice any other problems, until then I'll close this.

Alright I can confirm the browser is perfectly portable now after restarting my profile from scratch. I reinstalled all my extensions and logged back into everything I cared about and everything survives, including passwords and cookies which are otherwise lost.
Might be worth adding a warning in the release text that you can't reuse your old profile and trying to open it with the new default arguments will make the passwords and cookies be wiped instantly. It'll at least help people prepare and make a backup or their previous install so they don't get caught offguard like me can move things over at their own pace.


Unified packaging process

Hey!
It seems to me that xdg-ninja is packaged quite poorly in some cases, like putting a symlink to the programs/ directory directly in the bin/ folder of the system or modifying the script in place to patch the location of the programs/ directory.
This mostly is a consequence of the way the script currently fetches the programs/ directory:
https://github.com/b3nj5m1n/xdg-ninja/blob/4644d409aaf00b7d3b6864840f23e4c05c830322/xdg-ninja.sh#L221
i.e., directly in the current working directory of the xdg-ninja script.
This works fine when working within the git repository but does poorly when trying to package the program as there needs to implement hacks to get this working.
On a useless note, I wouldn't have hacked it this way on my side, I would've just packaged xdg-ninja in the same directory as programs in, for instance, /usr/share/xdg-ninja/ and would create a wrapper script that would call /usr/share/xdg-ninja/xdg-ninja.sh.
I am unsure whether or not this is a goal of your program to be packaged, but if so, I have implemented an simple way to unify the packaging process.

My main goal, when modifying the script, was to get it backwards compatible with how it was previously used, i.e. when working in the git repository, it does fetch the current programs/ dir rather than somewhere else.
So I have just set the script to check in the current working directory and replace ending instances of /bin to /share/xdg-ninja (where the programs/ dir is located when packaged).
If all cases it fails, the XN_PROGRAMS_DIR can be used to override that value. I have preferred to use this instead of a --programs-dir= argument as this isn't something meant for common usage either.
I've also allowed people to override where the script is supposed to be located with the PREFIX variable in the Makefile, I believe package managers should package apps to /usr instead of /usr/local so at least it can be changed. This make variable and DESTDIR and both common naming standards.

Hi! Thank you very much for this, it seems like a much-needed change.
The only concern I have is with the Install section in the README. I'm not sure that it makes sense to present this to users as the best/only option.
There is another big issue with packaging, I think perhaps I should finally think of something to fix that, update the packages, (I'm aware of arch, homebrew and nix) then write a proper installation section which recommends installing through a package manager first. The README needs a bit of an overhaul anyway tbh.
If you're okay with it, I'd say remove the Install section for now.
And again, thanks for putting time into this!

Btw, this breaks the nix flake, I tried to update it but it seems I don't have write permissions for this PR, I'll just leave the patch here, if you want to apply it that'd be great, otherwise I'll do it after merging.

From 640977d61030c576da3b47edfa09fd651b75a6ee Mon Sep 17 00:00:00 2001
From: b3nj5m1n<EMAIL_ADDRESS>Date: Sun, 23 Apr 2023 08:00:13 +0200
Subject: [PATCH] Update flake

---
 flake.nix | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/flake.nix b/flake.nix
index 91adf01..8f3a5bf 100644
--- a/flake.nix
+++ b/flake.nix
@@ -26,7 +26,25 @@
       in rec {
         packages = flake-utils.lib.flattenTree {
           # The shell script and configurations, uses derivation from offical nixpkgs
-          xdg-ninja = pkgs.xdg-ninja;
+          xdg-ninja = pkgs.stdenv.mkDerivation rec {
+            pname = "xdg-ninja";
+            version = "0.1.0";
+
+            src = ./.;
+
+            nativeBuildInputs = with pkgs; [ makeWrapper ];
+
+            installPhase = ''
+              runHook preInstall
+
+              DESTDIR="$out" PREFIX="/usr" make install
+
+              wrapProgram "$out/usr/bin/xdg-ninja" \
+                --prefix PATH : "${pkgs.lib.makeBinPath [ pkgs.glow pkgs.jq ]}"
+
+              runHook postInstall
+            '';
+          };
           # Pre-built binary of xdgnj tool downloaded from github
           xdgnj-bin = pkgs.stdenvNoCC.mkDerivation {
             name = "xdgnj-bin";
@@ -45,7 +63,7 @@
         };
         defaultPackage = packages.xdg-ninja;
         apps = {
-          xdg-ninja = flake-utils.lib.mkApp { drv = packages.xdg-ninja; };
+          xdg-ninja = flake-utils.lib.mkApp { drv = packages.xdg-ninja; exePath = "/usr/bin/xdg-ninja"; };
           xdgnj-bin = flake-utils.lib.mkApp { drv = packages.xdgnj-bin; exePath = "/bin/xdgnj"; };
         };
         defaultApp = apps.xdg-ninja;
--
2.39.2

I haven't taken the time to take a look at nix as I am unfamiliar with this distro, so that's a good thing you were able to test that on your side. I have tested this on Ubuntu and Arch.
I have added your patch on my branch and removed the Install section of the README as discussed.

Great, thank you very much again!

No problem, I can see that the AUR package for xdg-ninja-git has already been updated too!
From there, I can only recommend you to do a new GitHub release to push an update to the xdg-ninja AUR package which relies on tags/releases rather than the git repository version itself.

I can only recommend you to do a new GitHub release

Yeah, I'm still thinking about how to handle releases, I could either use them exclusively for xdg-ninja, (the script & new json files) or have a prefix for xdg-ninja/xdgnj to distinguish between the two.

I can see that the AUR package for xdg-ninja-git has already been updated too!

Wow that was quick!


TestMeshNetworking unit test is Flaky

https://prow.istio.io/view/gs/istio-prow/pr-logs/pull/istio_istio/26224/unit-tests_istio/18457

cc: @stevenctl

Looks like a real race during kube controller's first Run that calls reload, then the handler on meshNetworks watcher. Fix here: https://github.com/istio/istio/pull/26290


Adding SMI Conformance Operation

fixes #3
Currently getting this error on build:
go: finding github.com/layer5io/learn-layer5/smi-conformance latest
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>invalid version: unknown revision<PHONE_NUMBER>00

@leecalcote  We'll need to add replace directive in the go.mod file. Let me check this.

@leecalcote  This is done now.