Error in "Stack with Nix for system libraries (mixed mode)"

Tried to do stack build following instructions
> nix-shell
> [nix-shell:~/cardano-sl]$ ./scripts/build/cardano-sl.sh
>

I get the error below. I'm guessing stack should be installed on the host system but no documentation or instruction to do so. Is this correct?

> [nix-shell:/opt/cardano-sl]$ ./scripts/build/cardano-sl.sh
> Going to build:  cardano-sl-core cardano-sl-db cardano-sl-lrc cardano-sl-infra cardano-sl-update cardano-sl-ssc cardano-sl-godtossing cardano-sl-txp cardano-sl cardano-sl-auxx cardano-sl-tools cardano-sl-wallet cardano-sl-explorer
> 'wallet' flag: true
> 'explorer' flag: true
> Building cardano-sl-core
>
> stack build --ghc-options="-DGITREV=0d010e21e3af8ff6213d7877d9bb8132764900ec -Wwarn" --test --no-haddock-deps --bench --jobs=4 --flag cardano-sl:with-wallet --no-run-tests --no-run-benchmarks --dependencies-only  cardano-sl-core
>
> No compiler found, expected minor version match with ghc-8.0.2 (x86_64) (based on resolver setting in /opt/cardano-sl/stack.yaml).
> To install the correct GHC into /home/appadmin/.stack/programs/x86_64-linux-nix/, try running "stack setup" or use the "--install-ghc" flag. To use your system GHC installation, run "stack config set system-ghc --global true", or use the "--system-ghc" flag.
>

I then tired the other command suggested and got another error:
nix-shell -p moreutils expect --run "unbuffer ./scripts/build/cardano-sl.sh | ts"
> [nix-shell:/opt/cardano-sl]$ nix-shell -p moreutils expect --run "unbuffer ./scripts/build/cardano-sl.sh | ts"
> Feb 03 14:13:37 Going to build:  cardano-sl-core cardano-sl-db cardano-sl-lrc cardano-sl-infra cardano-sl-update cardano-sl-ssc cardano-sl-godtossing cardano-sl-txp cardano-sl cardano-sl-auxx cardano-sl-tools cardano-sl-wallet cardano-sl-explorer
> Feb 03 14:13:37 'wallet' flag: true
> Feb 03 14:13:37 'explorer' flag: true
> Feb 03 14:13:37 Building cardano-sl-core
> Feb 03 14:13:37
> Feb 03 14:13:37 stack build --ghc-options="-DGITREV=0d010e21e3af8ff6213d7877d9bb8132764900ec -Wwarn" --test --no-haddock-deps --bench --jobs=4 --flag cardano-sl:with-wallet --no-run-tests --no-run-benchmarks --dependencies-only  cardano-sl-core
> Feb 03 14:13:37
> Feb 03 14:13:38 No compiler found, expected minor version match with ghc-8.0.2 (x86_64) (based on resolver setting in /opt/cardano-sl/stack.yaml).
> Feb 03 14:13:38 To install the correct GHC into /home/appadmin/.stack/programs/x86_64-linux-nix/, try running "stack setup" or use the "--install-ghc" flag. To use your system GHC installation, run "stack config set system-ghc --global true", or use the "--system-ghc" flag.
>

Any guidance here please?

Ok I ran

stack setup

from within the nix console that seems to have worked

Ok I spoke to soon. After I ran

[nix-shell:~/cardano-sl]$ stack setup
[nix-shell:~/cardano-sl]$ ./scripts/build/cardano-sl.sh

I got the following (some logs omitted for brevity):

...
...
log-warper-<IP_ADDRESS>: download
log-warper-<IP_ADDRESS>: configure
log-warper-<IP_ADDRESS>: build
statistics-<IP_ADDRESS>: copy/register
Progress: 153/156
--  While building package log-warper-<IP_ADDRESS> using:
/home/appadmin/.stack/setup-exe-cache/x86_64-linux-nix/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file"
Process exited with code: ExitFailure 1
Logs have been written to: /opt/cardano-sl/.stack-work/logs/log-warper-<IP_ADDRESS>.log
Configuring log-warper-<IP_ADDRESS>...
Building log-warper-<IP_ADDRESS>...
Preprocessing library log-warper-<IP_ADDRESS>...
[ 1 of 17] Compiling System.Wlog.Severity ( src/System/Wlog/Severity.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/System/Wlog/Severity.o )
<command line>: can't load .so/.DLL for: /nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so (/nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so: symbol __recv, version GLIBC_PRIVATE not defined in file libc.so.6 with link time reference)

--  While building package optparse-applicative-<IP_ADDRESS> using:
/home/appadmin/.stack/setup-exe-cache/x86_64-linux-nix/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file"
Process exited with code: ExitFailure 1
Logs have been written to: /opt/cardano-sl/.stack-work/logs/optparse-applicative-<IP_ADDRESS>.log
Configuring optparse-applicative-<IP_ADDRESS>...
Building optparse-applicative-<IP_ADDRESS>...
Preprocessing library optparse-applicative-<IP_ADDRESS>...
[ 1 of 15] Compiling Options.Applicative.Help.Pretty ( Options/Applicative/Help/Pretty.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Pretty.o )
[ 2 of 15] Compiling Options.Applicative.Help.Chunk ( Options/Applicative/Help/Chunk.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Chunk.o )
[ 3 of 15] Compiling Options.Applicative.Help.Types ( Options/Applicative/Help/Types.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Types.o )
[ 4 of 15] Compiling Options.Applicative.Types ( Options/Applicative/Types.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Types.o )
[ 5 of 15] Compiling Options.Applicative.Internal ( Options/Applicative/Internal.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Internal.o )
[ 6 of 15] Compiling Options.Applicative.Common ( Options/Applicative/Common.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Common.o )
[ 7 of 15] Compiling Options.Applicative.Help.Core ( Options/Applicative/Help/Core.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Core.o )
<command line>: can't load .so/.DLL for: /nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so (/nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so: symbol __recv, version GLIBC_PRIVATE not defined in file libc.so.6 with link time reference)

any ideas?

tried
https://github.com/commercialhaskell/stack/issues/2130
ran command
nix-shell --command "stack --no-nix build"
also did not work. got this error
> log-warper-<IP_ADDRESS>: configure
> log-warper-<IP_ADDRESS>: build
> Progress: 60/174
> --  While building package log-warper-<IP_ADDRESS> using:
>       /home/appadmin/.stack/setup-exe-cache/x86_64-linux-nix/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.0.2 --builddir=.stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file"
>     Process exited with code: ExitFailure 1
>     Logs have been written to: /opt/cardano-sl/.stack-work/logs/log-warper-<IP_ADDRESS>.log
>
>     Configuring log-warper-<IP_ADDRESS>...
>     Building log-warper-<IP_ADDRESS>...
>     Preprocessing library log-warper-<IP_ADDRESS>...
>     [ 1 of 17] Compiling System.Wlog.Severity ( src/System/Wlog/Severity.hs, .stack-work/dist/x86_64-linux-nix/Cabal-<IP_ADDRESS>/build/System/Wlog/Severity.o )
>     <command line>: can't load .so/.DLL for: /nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so (/nix/store/zpg78y1mf0di6127q6r51kgx2q8cxsvv-glibc-2.25-49/lib/librt.so: symbol __recv, version GLIBC_PRIVATE not defined in file libc.so.6 with link time reference)
>

any ideas?

@denisshevchenko will update docs and then you can try again :)

Did anything ever happen with this? I'm experiencing this right now, except while building optparse-applicative-<IP_ADDRESS>:
stack --no-nix build optparse-applicative-<IP_ADDRESS>
optparse-applicative-<IP_ADDRESS>: configure
optparse-applicative-<IP_ADDRESS>: build

--  While building package optparse-applicative-<IP_ADDRESS> using:
      /home/michael/src/soostone/napkin-container/.stack/setup-exe-cache/x86_64-linux/Cabal-simple_mPHDZzAJ_<IP_ADDRESS>_ghc-8.4.4 --builddir=.stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS> build --ghc-options " -ddump-hi -ddump-to-file -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
    Logs have been written to: /home/michael/src/soostone/napkin-container/.stack-work/logs/optparse-applicative-<IP_ADDRESS>.log

    Configuring optparse-applicative-<IP_ADDRESS>...
    Preprocessing library for optparse-applicative-<IP_ADDRESS>..
    Building library for optparse-applicative-<IP_ADDRESS>..
    [ 1 of 16] Compiling Options.Applicative.Help.Levenshtein ( Options/Applicative/Help/Levenshtein.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Levenshtein.o )
    [ 2 of 16] Compiling Options.Applicative.Help.Pretty ( Options/Applicative/Help/Pretty.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Pretty.o )
    [ 3 of 16] Compiling Options.Applicative.Help.Chunk ( Options/Applicative/Help/Chunk.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Chunk.o )
    [ 4 of 16] Compiling Options.Applicative.Help.Types ( Options/Applicative/Help/Types.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Types.o )
    [ 5 of 16] Compiling Options.Applicative.Types ( Options/Applicative/Types.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Types.o )
    [ 6 of 16] Compiling Options.Applicative.Internal ( Options/Applicative/Internal.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Internal.o )
    [ 7 of 16] Compiling Options.Applicative.Common ( Options/Applicative/Common.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Common.o )
    [ 8 of 16] Compiling Options.Applicative.Help.Core ( Options/Applicative/Help/Core.hs, .stack-work/dist/x86_64-linux/Cabal-<IP_ADDRESS>/build/Options/Applicative/Help/Core.o )
    <command line>: can't load .so/.DLL for: /nix/store/fivq0nbggp4y8mhy3ixprqd7qyn1hy2j-glibc-2.27/lib/librt.so (/nix/store/fivq0nbggp4y8mhy3ixprqd7qyn1hy2j-glibc-2.27/lib/librt.so: symbol __close_nocancel, version GLIBC_PRIVATE not defined in file libc.so.6 with link time reference)

My build context is in a nix shell running stack build optparse-applicative-<IP_ADDRESS> without nix integration.


trustchains api need support pagination query

Problem description
up to v1.2.0 , trustchains api will return all vp documents, when the system running for long time we properly have to many token in 1 api call.
Requirements
Add support pagination params for trustchain api
Definition of done
NA
Acceptance criteria
NA

QA PASSED in feature/schema-topics branch

This ticket is closed on April 15th


TSError: ⨯ Unable to compile TypeScript

I am getting this error while running project
src/modules/graphql/graphql.service.ts (20,9): Type 'GraphQLSchema' is not assignable to type 'GraphQLSchema'. Two different types with this name exist, but they are unrelated.
Types of property 'getQueryType' are incompatible.
Type '() => GraphQLObjectType' is not assignable to type '() => GraphQLObjectType'. Two different types with this name exist, but they are unrelated.
Type 'GraphQLObjectType' is not assignable to type 'GraphQLObjectType'. Two different types with this name exist, but they are unrelated.
Property 'extensionASTNodes' is missing in type 'GraphQLObjectType'. (90010)

I've gotten it to work by matching the @types/graphql version with the one being used in graphql-tools, which was using v0.9.4.
So an npm install<EMAIL_ADDRESS>did the trick for me.


ArgumentError in RailsAdmin::Main#dashboard

I got this error in production env.
rails version:<IP_ADDRESS>
rails_admin version ref: 355dc80
vendor/bundle/ruby/2.2.0/bundler/gems/rails_admin-355dc80f8a20/app/views/rails_admin/main/dashboard.html.haml where line #30

arguments passed to url_for can't be handled. Please require routes or provide your own implementation

Does this solution work?
http://stackoverflow.com/questions/20660604/using-the-correct-url-for-method-in-a-rails-engine-spec

@mshibuya
Yes, It works. Thanks for sharing this solution.
I was including the line  Rails.application.routes.url_helpers in one controller, remove that line it works.
Thanks again !


Some VPNs shows up twice

Some VPNs show up twice with group list and user list, please provide a way to make vpn list uniq.

This issue happened because of migration of VPN.  I have added Uniq condition, we will clean this up with rake task.


Add to "Awesome-SelfHosted" Repository

This is an awesome self-hosted app. I'd recommend getting it added to the long list of Self-Hosted apps people reference here. Maybe BarTender or SaltRim, don't know which.
https://github.com/awesome-selfhosted/awesome-selfhosted

Hi,
I already opened a pull request and it's approved, only waiting for the merge. :D
https://github.com/awesome-selfhosted/awesome-selfhosted/pull/3465
Thanks!


Lambada Method Reference

当检测到指令为Opcodes.INVOKEDYNAMIC时，使用asm动态的将方法引用所引用的方法在当前类中新增为一个方法调用。强制修改Handle的name为新增的方法调用。
修改类:

com.alibaba.testable.agent.handler.SourceClassHandler
com.alibaba.testable.agent.handler.BaseClassHandler

新增单元测试:

com.alibaba.demo.lambda.LambdaDemo
com.alibaba.demo.lambda.LambdaDemoTest

 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.jimcao seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Installing on a cluster with an existing udp-services ConfigMap fails

If you attempt to install k8gb on a cluster with an existing udp-services ConfigMap via the make deploy-gslb-operator target, it will fail as below:
$ make deploy-gslb-operator ...
kubectl apply -f deploy/namespace.yaml
namespace/k8gb unchanged
cd chart/k8gb && helm dependency update
walk.go:74: found symbolic link in path: /Users/donovan/dev/absa/avaf/infra/bolt/k8gb/chart/k8gb/templates/crds/k8gb.absa.oss_gslbs_crd.yaml resolves to /Users/donovan/dev/absa/avaf/infra/bolt/k8gb/deploy/crds/k8gb.absa.oss_gslbs_crd.yaml
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "kuberhealthy" chart repository
...Successfully got an update from the "codecentric" chart repository
...Successfully got an update from the "jetstack" chart repository
...Successfully got an update from the "crossplane-alpha" chart repository
...Successfully got an update from the "elastic" chart repository
...Successfully got an update from the "jfrog" chart repository
...Successfully got an update from the "banzaicloud-stable" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 2 charts
Downloading coredns from repo https://kubernetes-charts.storage.googleapis.com
Downloading etcd-operator from repo https://kubernetes-charts.storage.googleapis.com
Deleting outdated charts
helm -n k8gb upgrade -i k8gb chart/k8gb -f avaf-nonprod-sdc.yml
Release "k8gb" does not exist. Installing it now.
walk.go:74: found symbolic link in path: /Users/donovan/dev/absa/avaf/infra/bolt/k8gb/chart/k8gb/templates/crds/k8gb.absa.oss_gslbs_crd.yaml resolves to /Users/donovan/dev/absa/avaf/infra/bolt/k8gb/deploy/crds/k8gb.absa.oss_gslbs_crd.yaml
manifest_sorter.go:192: info: skipping unknown hook: "crd-install"
Error: rendered manifests contain a resource that already exists. Unable to continue with install: ConfigMap "udp-services" in namespace "ingress-nginx" exists and cannot be imported into the current release: invalid ownership metadata; label validation error: missing key "app.kubernetes.io/managed-by": must be set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name": must be set to "k8gb"; annotation validation error: missing key "meta.helm.sh/release-namespace": must be set to "k8gb"
make: *** [deploy-gslb-operator] Error 1

One workaround is to follow the Helm prompts and annotate and label the existing ConfigMap accordingly:
$ kubectl -n ingress-nginx annotate --overwrite \
  cm/udp-services meta.helm.sh/release-name=k8gb \
  meta.helm.sh/release-namespace=k8gb
configmap/udp-services annotated
$ kubectl -n ingress-nginx label --overwrite cm/udp-services \
  app.kubernetes.io/managed-by=Helm
configmap/udp-services labeled

or alternatively, you could delete the existing ConfigMap, assuming it is empty.

Solved by https://github.com/AbsaOSS/k8gb/pull/354


mWeb/Chrome - Timezone is highlighted when navigating to search field

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

Issue found when executing PR https://github.com/Expensify/App/pull/12201
Action Performed:

Go to staging.new.expensify.com
Log in with any account
Navigate to https://staging.new.expensify.com/settings/profile/timezone
Disable the Automatically determine your location
Tap again on the Timezone selector, which should redirect to another page.

Expected Result:
Timezone field should be empty
Actual Result:
Timezone is highlighted when navigating to search field
Workaround:
Unknown
Platform:

Where is this issue occurring?

Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: new feature
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/205497417-51f813ed-6baa-4c9f-9ba3-eac1ce98cc6c.mp4

Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

proposal
we can change timezoneInputText value to empty string.
https://github.com/Expensify/App/blob/2e8d90736a19de0bf0332fb36afc42bab074434f/src/pages/settings/Profile/TimezoneSelectPage.js#L51
to
-             timezoneInputText: this.currentSelectedTimezone,
+              timezoneInputText: '',

IMO this is a feature and it's intended to be that way.

Confirmed, feature not bug, closing

@applausebot please keep an eye out for the difference here between an actual bug (which we want reported) and a feature like this. Thank you!

@kbecciv ^ not applause bot


Empty inputType and outputType parameters are (very often) added when showing Config form of a Camel Route

Describe the Bug
empty (and so invalid because when added they require the urn parameter to be set) inputType and outputType parameters are (very often) added when showing Config form of a Camel Route
Steps to Reproduce the Bug or Issue

Create a Camel route
Select the route
--> check the source code, invalid route is there with empty inputType and outputType

Screenshots or Videos

Platform
main branch with online versionemptyAndInvalidInputOutTypes

This seems related to https://github.com/KaotoIO/kaoto-next/pull/726, since now updates in the form are made in a field basis, rather than in an entire object fashion. This event happens as soon as the form is open.


should we have a "template" asset type?

@aembler,
sorry to use a bit of your time again, but I just want to ask you.
On the file manager upgrade that I pushed, I fixed the file selector by outputting the file menu context template with the selector every time Aside from the fact that I use a ob_start/ob_get_clean method (which might need some refinement), it means that the template is outputted multiple times if there are multiple elements in the page.
While this is no issue as in the js I shield against the eventuality that there is multiple templates (I take the first one), I feel like this is just throwing useless data at the browser, and I'd rather use a asset for that.
Now, that said, currently we have javascript and css assets, but not "templates" assets. Should I add a new type of asset (call it template), and then move along?
I'd rather think you would want me to do that in a another pull request (or not at all), what do you think?
This is the pull request I am referring to: https://github.com/concrete5/concrete5-5.7.0/pull/1762

I'm not exactly sure I understand what the "bug" here that was "fixed" is. Why was there a need to move the menu out of the javascript and output it multiple times anyway?

The point is we are now using underscore.js, and I see a huge benefit to it. So adding a "template" type of asset that could be pulled as well (and would be separate files) would make sense for a lot of packages.
Moreover, this solves the issue you mentionned in #1762 since the asset would be pulled directly when you get the file selector.
What do you think?

That is intriguing and makes more sense – but isn't the template already included automatically by the page selector asset since it's currently delivered as text within the JavaScript function? It's also only delivered once. Wouldn't the only benefit be a nicer file than a string that's delivered within JavaScript? It's fine if that's the answer – just making sure I understand it.

Yes it is, but that would also be pulled in if you just pull the js and not use the selector. Currently (in that pull request), as you mentionned you have to pull the element to have the menu template in case you are only using the javascript call.
The benefits would be the following as I see it:

Allow to add actual underscore templates (php processed or not, using routes) to assets groups
Allow to differentiate files and have the correct tag outputed (type=text/template)
Encourage addon dev to use underscore and have them place their templates in a specific location (say, /js/templates ?)
I would also through in an improvement to the asset registration to allow the asset to add tag attributes in the <script ...> or <style ...> tag outputed (could be useful if we want to customize them such as having data-xx=yy or name="yy" just to name a few).

I don’t know – part of the core JavaScript is about bundling in the things that need to work together to be together so that a) you always know what you’re going to get and b) file size is reduced. You’re never going to use the file selector without the JavaScript menu, and this way it’s going to be included in the file-manager.js file, minified and everything. I think the Template idea as an asset type sounds intriguing but it ought to be JavaScriptTemplate because otherwise I don’t think it makes a lot of sense immediately.
--
Andrew Embler
Sent with Airmail
On January 7, 2015 at 8:33:06 AM, Florian Delizy<EMAIL_ADDRESS>wrote:
Yes it is, but that would also be pulled in if you just pull the js and not use the selector. Currently (in that pull request), as you mentionned you have to pull the element to have the menu template in case you are only using the javascript call.
The benefits would be the following as I see it:
Allow to add actual underscore templates (php processed or not, using routes) to assets groups
Allow to differentiate files and have the correct tag outputed (type=text/template)
Encourage addon dev to use underscore and have them place their templates in a specific location (say, /js/templates ?)
I would also through in an improvement to the asset registration to allow the asset to add tag attributes in the
—
Reply to this email directly or view it on GitHub.

Well, I agree an that, but the filesize is not a pb here, because a) it is in edit mode, b) it is a string that does not get compressed anyway…
Having js templates would I believe help readability, (and would be useful for packages for instance).

Anyway, can you have a look at the #1785 pull request?

I'm against this, assets are things either external to the page or hierarchy agnostic while a template is part of the DOM hierarchy and is in most cases unique. It costs nothing but a few hundred bytes to send to the client more than once because they aren't rendered anyway, also with gzip if they are the same it shouldn't cost anything sans the compression time to send multiple times. I feel strongly that adding templates outside of the context it's used within is an antipattern that will encourage people to think about templates and the way they write them in the wrong way.
@goutnet If you're fixing the possibility of there being multiple templates by selecting just the first template, you're doing it wrong.

well …
regarding that last comment, in that particular case, I have no parent element to look into… so that is the only solution I found so far. proposition welcomed.

finally, we will go without this, I am closing that issue here.


No tests and documentation about excluding multiple files/paths

When looking at search results for exclude feature, I see it supports multiple arguments, but there's neither test nor documentation to show how to use multiple excludes.
I'd expect test with at least two excludes and documentation with example for that feature.

@yegor256 dispatch this issue please, see par.21

@mkordas what about this: https://github.com/teamed/pdd/blob/7d979eb1f26ed66cdfaa28a5e1346295ecb2901c/features/cli.feature#L54 there are two excludes

@mkordas extra doc: 0a622db

@yegor256 What about multiple paths provided to one exclude? Is the following
pdd --exclude=**/*.java,**/*.groovy

valid or should I write
pdd --exclude=**/*.java, **/*.groovy

or maybe some quotes are needed? From docs it's not clear, I don't see tests for it either.

@mkordas maybe that will work, but I'm not sure. just use multiple --execute and you'll be fine

@yegor256 the thing here is that it works and is used in projects, so the question is if we want to have undocumented feature without tests: https://github.com/teamed/qulice/blob/fe57956fef458ad89afbe379afa217c9863aceb7/.travis.yml#L25


make fails with multiple target patterns on mac

Tried multiple make command on mac all failed with "Makefile.overrides.mk:26: *** multiple target patterns.  Stop." It happened for both container build or local build.

There are two ways to build on a Mac, with BUILD_WITH_CONTAINER=1 and without/BUILD_WITH_CONTAINER=0.  Which are you using?
I am able to make istioctl with both variants.  I may have installed a few GNU utilities on my Mac.  These are some things I had to do at one point to build Istio; now that I have done them I don't know if they are still needed.
brew install shellcheck
brew upgrade bazel
xcode-select --install
brew install yamllint
brew install findutils
ln -s /usr/local/Cellar/findutils/4.7.0/bin/gxargs /usr/local/bin/xargs

I tried both container and local build and they had the same error.
I didn't change my mac environment setup and the make command worked well before. So I am wondering whether it is introduced by some recent change of build system.


automatically setup matchers

eliminates the need for most custom matchers.
this also eliminates the super goofy magical ChefSpec::API inclusion of all defined module constants under that namespace.   just uses ruby to write into ChefSpec::API and ditches the overly fussy 'encapsulation'.

@tas50 here you go, no more maintaining matchers

Thinking this through further, does this work at all? If you have
subject { SoloRunner.converge(whatever }
it { expect(subject).to create_template(...) }

it has to compile the matcher expression before the converge runs. This would only work if we always did aggressive (i.e. let!) expansions I think?

all the examples pass...

  let(:chef_run) { ChefSpec::SoloRunner.new(platform: 'opensuse', version: '42.1').converge(described_recipe) }

  it 'installs a zypper_package with the default action' do
    expect(chef_run).to install_zypper_package('default_action')
    expect(chef_run).to_not install_zypper_package('not_default_action')
  end

e.g. that works fine.
and the problem is that if we're going to take the objectspace-searching approach to finding Chef::Resource things, then it has to be done after libraries have been compiled, so we can't overeagerly do this before we've setup a runner and at least done the compilation phase.
we could do things a bit more dynamically and hook class loading, but that also gets messy.  if we mandated that every Chef::Resource had to declare something like resource_name or had to touch provides or something it might be more doable, but there no 'finalized' or even 'good enough' hook to know when a Chef::Resource subclass has been declared enough to setup matchers for it.  we could add a phase after compiling library/resource classes that would hit matchers, but that is buried down inside chef code.

An easier way to do this would be to put the declaration system in Chef::Resource itself as part of provides, but that might be overly familiar? My specific concern is I have a lot of tests that never call converge and just use chef_runner.run_context or whatever.


Problem with Golden Carrot eating animation

So I have recently found a problem that the Golden Carrot eating animation isn't loaded while the others works fine. I have tried opening the mod file as archive (7-Zip) but the golden_carrot.json file and the "children" are just perfectly fine.
Sidenote: I have once tried to retexture the Enchanted Golden Apple eating texture to sync with the Visual Enchantments texture pack and it worked once, but the second time when I reload the pack, it got the same issue as the current one above.

if you use other resourcepack, you need to animate food in your resourcepack.


Feaure requests: unattended installation, user creation, app creation, sso, ldap integration

Hello,
Here is my proposal to solve:

unattended installation https://github.com/go-gitea/gitea/issues/2324
default admin user creation https://discourse.gitea.io/t/i-cannot-programmatically-setup-users-in-a-docker-container/289/8
default oauth app creation https://github.com/go-gitea/gitea/issues/8764
default sso integration ( not solved here )

https://github.com/go-gitea/gitea/blob/master/docker/root/etc/s6/gitea/setup
if ! gitea migration --test;
then
  printf 'do migration'
  gitea migrate
fi

if [ -z $GITEA_USERNAME && -z $GITEA_PASSWORD && -z $GITEA_EMAIL ];
   printf 'user creation initiated'
   if ! gitea admin get-user $GITEA_USERNAME; then
     printf 'user not found, creating new one'
     gitea admin create-user -u
   else
     printf 'user found, modificating email and password'
     gitea admin edit-user $GITEA_USERNAME -p $GITEA_PASSWORD -m $GITEA_EMAIL
   fi
fi

if [ -z $GITEA_OAUTH2_PROVIDER_ID && $GITEA_OAUTH2_PROVIDER_SECRET ];
then
   printf 'oauth app creation initiated'
   if ! gitea admin get-app $GITEA_OAUTH2_PROVIDER_ID; then
     printf 'app not found, creating new one'
     gitea admin add-app $GITEA_OAUTH2_PROVIDER_ID -p $GITEA_OAUTH2_PROVIDER_SECRET
   else
     printf 'app found , modificating secret'
     gitea admin edit-app $GITEA_OAUTH2_PROVIDER_ID -p $GITEA_OAUTH2_PROVIDER_SECRET
   fi
fi

To do this, its needed to integrate new gitea commands:

gitea migration --test -> test if migration is done ( no need to use postgresql-client, mariadb-client, etc to check db tables are populated/migrated )
gitea admin get-user -> get gitea user ( name, password, email )
gitea admin edit-user -> edit gitea user ( password, email )
gitea admin add-app -> add oauth application ( id secret )
gitea admin edit-app -> edit oauth application ( secret )

P.S. modification commands are not necesary, its just for fully idempotent initialization of new instance. ( new gitea instance with old gitea db ). Even gitea admin auth update-oauth and gitea admin auth update-ldap exists...
Environment variables:

GITEA_USERNAME
GITEA_PASSWORD
GITEA_EMAIL
GITEA_OAUTH2_PROVIDER_ID
GITEA_OAUTH2_PROVIDER_SECRET

Something similar should be used also for oauth and ldap authentication.
There are already commands for it:

gitea admin auth add-oauth
gitea admin auth add-ldap

Would be nice to have fully integrated gitea at deploy time.
Any other ideas @lunny ?
Thanks

It's interesting. We need some new commands.

+1 for this feature, dockerised setup really could use default admin account.

In 1.14.x this is a lot more straightforward, see https://discourse.gitea.io/t/unattended-gitea-installation-from-the-cli/3373/14 for details.

@techknowlogick this works, thanks for the hint.


Pornhubpremium not working

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.01.27. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.01.27

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
C:\Users\death\Desktop>youtube-dl -f best  https://www.pornhubpremium.com/playlist/63054102 --cookies cook.txt
[PornHubPlaylist] 63054102: Downloading webpage
WARNING: unable to extract title; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
[download] Downloading playlist: 63054102
[PornHubPlaylist] playlist 63054102: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: 63054102

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
When replacing the pornhub.com with pornhubpremium.com and using cookie after logging in I can't download the videos and using cookies after logging in.
I replaced the pornhub.py with this:
pornhub.txt
I get unable to extract title error

@858665536 contact me your email at<EMAIL_ADDRESS>

@858665536 contact me your email at<EMAIL_ADDRESS>

@858665536 contact me your email at<EMAIL_ADDRESS>

did this ever get resolved?

Not yet, as far as I know..


Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Name of the service
Spotify
URL of the service
https://open.spotify.com/
GitHub URL to a recipe that has already been created by someone else or you
No response
Features you want implemented
I would like to have a spotify as service on ferdium

I took an initial look at this. It's not as simple as creating a recipe for Spotify requires a CDM such as Widevine CDM which would need to be something that has to be added to ferdium-app. You get the following if you just naively tried to create a recipe for Spotify.

I know, This is the reason why I asket if there is another possibility to
use Spotify on ferdium.
Il giorno dom 30 ott 2022 alle ore 19:17 Brian Chen <
@.***> ha scritto:

I took an initial look at this. It's not as simple as creating a recipe
for Spotify requires a CDM such as Widevine CDM which would need to be
something that has to be added to ferdium-app. You get the following if you
just naively tried to create a recipe for Spotify.
[image: image]
https://user-images.githubusercontent.com/14852675/198894689-9065edca-511c-4795-a331-285efb69a814.png
—
Reply to this email directly, view it on GitHub
https://github.com/ferdium/ferdium-recipes/issues/216#issuecomment-1296319276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN23A2ZPNXPCN7KTWPIRUZ3WF23VHANCNFSM6AAAAAARNZTYT4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update to release versions of Microsoft.Azure.Services.AppAuthentication

Why is this in rc1?

Microsoft.Extensions.Configuration.AzureKeyVault depends on it and ships in 2.1

Ok. Are we shipping that for RC1?

Yes

Talked to @DamianEdwards in person, we'll track it for RTM

This is done.


First of all thanks for the efforts and your work.
Expected Behavior
Should not have any crashes.
Steps to Reproduce (for bugs)

Selected movie "Downsizing" from Trends
Watched it more than 1  hour
Got "Elementum daemon crashed"
Movie continue playing.
In a couple of seconds movie stopped.
Then I started movie again and selected to continue from that time when I had finished.
Got crash, movie stopped again.
I've resumed it again.
While starting I got black screen from my TV that there is no source any more on HDMI.
Then TV got signal and I got some sound but screen was black.
Pressed Stop on remote and restarted movie.
Got just black screen and sound.
Reboot.

Environment

Version used:
v0.0.43
Environment name and version:
17.6 Git:20171114-a9a7a20
Operating System and version:
Originally was KODIbuntu but upgraded to Ubuntu 16.04.4 LTS
Link to Kodi debug log:
https://www.dropbox.com/s/ub8zr6x3fg8b47h/toreport.kodi.log?dl=1

Reported crashes here:
https://github.com/anacrolix/torrent/issues/240

Is https://github.com/anacrolix/go-libutp/commit/89a35c2661e99e6d1c14b1096bed2ed0d59a84a0 included and cgo enabled?

@elgatito could you please answer?

elementum 0.0.43 was built on 14th of February, so has all the packages for that date.
CGO is always enabled.

@anacrolix selected commit was added day later, so we don't have it.
Will make a new version with the latest packages.

@elgatito thank you

@elegatio sorry for disturbing. But could you please publish new version?


Indexer Tech Debt

Summary
List of specific tech debt items that needs minor/major refactoring to cleanup. Probably split this up into separate tickets when it's time to implement.
Main loop refactoring

The main block handler is designed so that for every new block, it will process the current block and any unprocessed block before it. In normal operation all previous blocks have already been processed, but in the case of a sneaky migration or a node restart, this behavior is critical. This cleanup phase should be done directly during an initialization phase (like the genesis import) instead of implicitly while handling a block.

Importer has logic for managing tar files in it, this stuff should get moved to the import.go command.

accounting.addrIsZero can be removed, address.IsZero() is part of the SDK.

Move UpdateAccounting and other related functions from importer/helper.go to importer/importer.go. Rename helper.go to tar_importer.go.

The import subcommand is deprecated. Anything using it needs to be refactored. The main thing here is the new test scripts, they need to be regenerated and updated to use the algod node archive.

~Initialize maps in RoundUpdates::Clear function, remove nil checks in the round update helper functions.~ #330

~updateAccounting could be passed in the block instead of querying the current DB state. Leave it up to the idb layer to decide whether or not to commit the accounting changes.~ #331

~importer.updateAccounting requires a path to the genesis file in some cases. This should be removed and put into a importer.InitializeAccounting function which will be a No-Op if there is nothing to initialize.~ #331

~importer.updateAccounting uses db.GetMetstate. It should use db.GetMaxRound instead.~ #330

~daemon.HandleBlock has logic for updating metastate when the number of transactions in a round is 0. This should be moved down into importer.UpdateAccounting if idb.CommitRoundAccounting could handle empty round updates.~ We just removed it.

~accounting.State.dirty could be removed by making it so db.CommitRoundAccounting can handle rounds with no transactions. This might also resolve the special case in daemon.HandleBlock for rounds with zero transactions.~ #280

~accounting.State.AddTransaction is responsible for calling db.CommitRoundAccounting, this should be moved out of AddTransaction into importer.updateAccounting.~ #280

~accounting.State should not have access to the db object. There are only 3 places where it's used. Two of them are to get the number of transactions in a round, and the last one (above) should be moved to another package.~ #280

IDB Interface

rename dummy.go to interface.go or indexerdb.go
Remove 'YieldTxns- implement it as a helper function that uses theTransactions` interface and the public paging mechanism. This will depend a lot on the performance, maybe having the optimized version is required for the initial sync.
~remove GetMetastate from public IndexerDB interface. It is an implementation detail, treating it as an arbitrary key-value store breaks the abstraction.~ #330

Postgres IndexerDb Implementation

The yieldTxnQuery queries data with round > $1. If we change this to round >= $1 it would remove a lot of strange off-by-one problems where we need to handle the case of passing in -1 to get round 0.

Urgency/Relative Priority
Low priority, these should be addressed as needed.

The remaining tasks are obsolete or not very important.


Fix error when opening REPL from unsupported mode

"Wrong number of arguments: (_mode fn . plist), 0"

Thanks!


Improve exceptions management

Current build lacks clear exception management.
Improve on all exceptions handling, error codes and descriptions.

Linking to issue #7 as example of exception management requirements

Commit 161596105b93035ea7902ae7cf90f072e249d87e
More improvements to be made, so will leave this one open.


Do you have the paper about your work?

I think your work is pretty great. Recently I want to do the work about debluring the turbulent flow, and the NS-equation constriction is a good idea.
Do you have the paper about your work? I want to see the detail in your paper.
Thank a lot. 👍

Hi, thanks. We are working on a paper right now and should put it up on Arxiv in the next few weeks. So stay tuned!

We are working on a paper right now and should put it up on arxiv in the next few weeks. Stay tuned!

That is great! I will expect that.
p.s. Can I ask which dataset do you use?  Thanks

We have finally uploaded the manuscript onto https://arxiv.org/abs/2003.01907

We have finally uploaded the manuscript onto https://arxiv.org/abs/2003.01907 😄

hh，I am reading now. 👍


Command gives incorrectly worded output

Description:
Grammatically incorrect command output.
Steps to reproduce:
Run the following on a bal file:
ballerina openapi gen-contract hello -i demo.bal
It gives the following output:
Note: This is an Experimental tool ship under ballerina hence this will only support limited set of functionality.
Successfully generated the ballerina contract at location
/home/ballerina/demo/src/demo/hello.openapi.yaml

Note the wording of the Note. It is grammatically incorrect and has capitalization issues. It needs to be reviewed and fixed.
Affected Versions:
Ballerin 1.0.0

We updated this sentence in the help texts for the individual OpenAPI commands, but seems like we are specifically printing this for each execution of the commands themselves. https://github.com/ballerina-platform/ballerina-lang/blob/master/misc/openapi-ballerina/modules/openapi-to-ballerina-generator/src/main/java/org/ballerinalang/openapi/cmd/OpenApiGenContractCmd.java#L52
We can update this to what we have in the help text.
"Note: This is an experimental tool, which only supports a limited set of functionality."
We should probably remove that sentence from the help texts when we do this, since it's a duplicate anyway.

@MaryamZi thank you ... will update the message printed through the CLI tools


Plans for SFSymbols2 categories

Is there any plan to include the SFSymbol2 into categories somehow?

I suggest you just fork the repository and take some time to do it yourself. Then you can submit a pull request.

I was more asking if this was a feature that the owner of this repository wanted in the project than for them to do it.

Ah I see. Well I think it would be a good addition.

@aaronpearce Yeah, that could be useful. I am working on a bit of an update since things changed for SFSymbols in WWDC21. This might be the time to play with the idea of adding this in.


Prevent module from disabling non-angular forms

Once this module is included in an application, it will cause all forms to silently to fail to submit.
Normal forms that were not designed for angular would silently fail to submit because of the e.preventDefault()
Angular forms that used ng-submit to perform the action were also not getting called (I think because of the preventImmediatePropogation)
This patch is a bit rough and ready, but I've tested it agains:

A normal non-angular form
An angular form using bootstrap validation provided by this module

And both work as expected.
Changes:
form.on('submit') return true unless it's invalid
form.on('submit') do not call preventDefault or preventPropogation

Okay, let me test this.

Hey @chrisjensen sorry for the late reply on this. What do you mean by Angular forms that used ng-submit to perform the action were also not getting called (I think because of the preventImmediatePropogation)?

Which version of Angular you are using?

Released version v1.0.4.


services

I can't get install-msiproduct to work for applications which create a service - e.g. our anti virus solution
It keeps complaining about insufficient permissions (yes I am admin).
I this not working in general or am I missing something?

Even if you're an admin you have to install from an elevated environment. Right click on your terminal eg powershell.exe and select Run as Administrator.
If you are running elevated, try installing the MSI using msiexec. If that fails it's your MSI. The cmdlets just call the exact same APIs that msiexec does. Unlike msiexec, though, they don't have the ability to elevate on-demand (that functionality is not exposed via the APIs).


forth: Rewrite tests to use hspec with fail-fast.

Rewrite tests to use hspec.

Related to #211.

Suggest that "- Move hint from test suite to HINTS.md." get removed from commit message

Sorry for that, I completely forgot to review the commit message. Fixed! :grimacing:

It may be time for me to write that canonical-data.json soon if I want to get rid of that Ogham space mark...


Args refactor

When working on a integration of sandbox into a polkadot, I've encountered a situation when I needed to use RuntimeArgs::nth in a generic context but was unable to do so since it is impossible to call nth with type T without a TryInto constraint and it is also impossible to specify such constraint since TryInto is not exported.
Furthermore, even though TryInto is just a utility and user can always define conversion operations by himself, it is infeasible to extract [RuntimeValue] from the RuntimeArgs since it's not public.

Can you provide an example of what became possible that was not possible before?

Yeah, sure:
For example, you want to have a method that looks like this:
fn first_arg(args: RuntimeArgs) -> i32 {
  args.nth(0)
}

this will work fine since RuntimeValue has an implementation of TryInto<i32, _>.
But when you introduce generic parameter
fn first_arg<T>(args: RuntimeArgs) → T {
  args.nth(0)
}

it won't compile, since nth::<T> requires that T have a bound RuntimeValue: TryInto<T, _> and you have no way to specify it, since you can't import it.
One way to fix it would be to expose the fact that RuntimeArgs is essentially just a wrapper around a slice. That at least will allow you to define your own mechanism to convert RuntimeValue to some T. This also allows you to use an iterator on RuntimeValue to get args one by one. I found this useful when working in a macro context: instead of having  RuntimeArgs and a position of the current argument, you can just use the iterator for fetching the args.
And exposing the FromRuntimeValue (fka TryInto<T, _>) trait is allows user to preserves the convenience of using the RuntimeArgs::nth in a generic.
So now, it is possible to have a construction like that:
fn first_arg<T: FromRuntimeValue>(args: RuntimeArgs) → T {
  args.nth(0)
}


how to generate detection results for CTRL

hi, I wonder how to generate detection results in the training step 2
https://github.com/tusen-ai/SST/blob/main/docs/CTRL_instructions.md#step-2-use-immortaltracker-to-generate-tracking-results-in-training-split-bin-file-format

Using any detectors and inference on the Waymo dataset, then you can get the detection results. You could use the FSD in this repo if you need it.

Please reopen this issue if you need further discussion.

训练CTRL一定要有跟踪标签吗？
数据集可以是单帧的吗？

@Abyssaledge @yrt1993
您好，请问一下，FSDV2跑waymo数据集，如何存成bin格式的。


Can't run ospMPIDistribTutorial on Windows with Intel oneAPI Tools

I compile ospMPIDistribTutorial.cpp and ospMPIDistribTutorial.c with Intel oneAPI Tools and run them with command mpiexec -n 2 ospMPIDistribTutorial.exe but failed:

Exception thrown at 0x00007FFC812F14E9 (ospray_module_mpi.dll) in ospMPIDistribTutorial.exe: 0xC0000005: Access violation writing location 0x0000009D854963C0.

How can I fix this?

Hmm. Do other MPI tutorials work, like the ospMPIDistribTutorialSpheres?

OspMPIDistribTutorialSpheres and ospMPIDistribTutorialVolume failed too. Only ospMPIDistribTutorialReplicated succeeded.

I think these are probably due to the same issue as #495, but I haven't tested out my fix on Windows yet. I'll take a look there to see if that fixed it, or what else might be going on.


Property not found on intersection type

It is not possible to access certain properties on intersection types if you define a list of keys you want to access.
It works if you add the string[] type to the properties list.
Here is an example:
https://flow.org/try/#0PTAEAEDMBsHsHcBQiAmBTAxtAhgJzaAC4CeADgQELYDOaA8gEYBWmhoAvKAN6KiiSxYALlDVCuAJYA7AOYBuRAF8F6LHgIlyoALLFGLDG048+DPCLGTZCxaABkoKrX2sFiAG55QAWz3MRui6GHNy8-IIiAEQCsJEANGFmuFFJoACMkUpuIKAAYgCCAJIAMgDKiBiwUmKgpLiwpNQAqlKaaCghANrRgvGgkUmRALqIdQ3NrWTtAHTUsN5oABSLpACUHAB8Pn5MnaRDq9lgAOp0AEoA0uWV1WxjjQAqUygW4tIynUNdPbFx-YMje7UJ7kFCzeZLFbrdhbXz6PYHBRAA

Another example


Search not work, (reindex?)

Hi, i change the products in db and now I can't find any product in the search box. I was reading the code of  search bundle and I found a function calls "ReindexAction". Probably I have to execute this function to create a JSON or any other type of file with all my products indexed in it?
When I try to find a product, message "No search results" show.
Please help.
Thank you.

I found how to execute the function from console like this:

app/console wellcommerce:search:reindex

But now im getting this error:

Reindexing locale: en
Total entities: 1143
Batch size: 100
Iterations: 13
Locale: en
Flushing index
100/1143 [==>-------------------------]   8% < 1 sec
[RuntimeException]
Unable to get a property on a non-object.

I'm trying to see the error in dev.log but I can't find the problem.
Any idea?


Add proxy support & fix lock bug

Add Socks5 and Http proxy support.
Fix Viewbot stops on request failure.

sexy lemme merge


tech-review: understand if toxpiR and webR are compatible

WebR requires us to install packages in a different way than RStudio.
A critical dependency of toxpiR has to be installed in an unconventional way. install.packages() and its WebR equivalent do not work out of the box.
Is there a workaround that allows us to install toxpiR in the web?

Surely we close this


docs(README): update link to latest docs (webpack.js.org)

What kind of change does this PR introduce?

[x] Other... Please describe: Updated README

What is the current behavior?
README points to WebPack 1.x docs
What is the new behavior?
README points to latest WebPack docs

@zuzusik Please sign the CLA by closing and reopening the PR to trigger the CLA bot again

@michael-ciniawsky just signed, should be fine now 👍


Fix cabal v2-build

The dependencies in cabal.project were out of sync with stack.yaml.
I haven't measured whether we even need to build with -O2, but it's certainly not needed when -fdevelopment is enabled.

bors r+

Disk space issues on Buildkite, retrying ...
bors r+


Added userAgent prop type in docs for webview

userAgent as a prop type is available in WebView for android but is not documented. This PR fixes it.
TestPlan : Not required as this just adds an entry in the documentation

By analyzing the blame information on this pull request, we identified @nicklockwood and @janicduplessis to be potential reviewers.

Thanks @chiraggshah !
@facebook-github-bot shipit

Thanks for importing. If you are an FB employee go to Phabricator to review.


Money raised on Bountysource

Hi Crystal community! :smile:
This question is mainly for @mgarciaisaia, @waj, @bcardiff, @mverzilli and other core developers.
Why on crystal repo bountysource badge only shows $ 7879 raised?

But, in bountysource Supporters tab, exists people donating since 2015, I did my math and got $ 18,250.
2000 + 500 + 1725 + 975 + ... + 24 + 22 + 14 + 11 + 10 + 9 + 8 + 6 = 18250

I didn't include Anonymous contributor with 12,000 donated, (Who would be the awesome Anon person :smirk:)
I know Bountysouce has a fee, and core developers works hard to become Crystal 1.0 stable. But We want to know who is getting paid or Perhaps a better administration of donations. How are donation used?, I give some ideas:
July month, by example
$   500.00 - Hosting server
$    80.00 - Domain crystal-lang.org
$    62.00 - Github services
$   100.00 - Bountysource fee
$ 1 502.00 - Paid to core developers
--------------------------------------------
$ 2 242.00 - Total

Maybe a montly issue or blog post talking about donations. :moneybag:
Thanks for create Crystal :stuck_out_tongue_winking_eye:

Maybe somebody with financial or administration knowledge can manage this :sweat_smile:

Same: mailing list or email discussion, not bugtrack.

@ysbaddaden my bad,  Sometimes I feel more confortable writing on Github issues :stuck_out_tongue_winking_eye:, I will repost on Google forum :+1:


Bumping source-map?

It looks like source-map-support has a dependency on source-map @ 0.1.32 created in September 2014. The current version is 0.5.6. Is there any reason we need to stay on such an old version. We are seeing a flake from inside of source-map  via this library via karma-source-map-support and it is really hard to track down the error since we are on such an old version.
For reference of the underlying flake we are having trouble tracking down: https://github.com/sinonjs/sinon/issues/1094

If anyone would like to scan thru the changelog and see if there any breaking changes we should be aware of, that would be much appreciated. I'm okay with bumping the dependency if it looks good

source-map was updated in fd4c649209915ab8f4785ceb97bcb9998b22711 to version 0.5.3.
The Mocha tests pass bumping to 0.5.6. I can submit the PR if it's easier for you to confirm @LinusU.

I would love that 👍


Narrow union type with 'in' operator

Like we can narrow down union types with typeof  and instance of, it would be nice if it could  also be narrowed with the javascript in operator.
Code
type Message1 = { name: "foo"};
type Message2 = { name: "personAdded", personId: string, fullName: string};
type Message3 = { name: "personSelected", personId?: string };
type AnyMessage = Message1 | Message2 | Message3;

function handleMessage(message: AnyMessage) {

    // message can be: Message1 | Message2 | Message3
    if ("personId" in message) {

        // narrowed to: Message2 | Message3
        switch (message.name) {
            case "personAdded":    console.log(`Person added: ${message.fullName}`); break;
            case "personSelected": console.log("Person selection changed"); break;
            default: exhaustiveFail(message); break; // error: Message1 is not handled
        }
    }
};

function exhaustiveFail(x: never): never {
    throw new Error(`Unexpected object: ${x}`);
}

When using an exhaustive switch/if, it would allow us to reduce the union type down to only the types that can have the specified property.
Remark
What to do with numbers and symbols?
const a = ["foo", 42];
if (0 in a) {...}
if (Symbol.iterator in a) {...}

Can be marked as duplicate, found a similar issue: #10485

Thanks for digging those up @jovdb.


Francisco<PHONE_NUMBER>

In this pull request (https://github.com/dallaspythondojo/python/pull/164) , I see that you're merging other branches. You shouldn't have to or ned to do this. Just ensure you're master is on the latest tip (equal to class repo, by pulling from it), and you won't have to do any merging of your own).


Google assistant shows thermostat in Fahrenheit but AC only functions in Celsius.

Home Assistant version
Latest docker pulled fresh before this issue.
SmartIR version
Latest, pulled fresh before filing this issue.
Describe the bug
This ac unit only works in Celsius and there's no way to change it. In home assistant the the thermostat correctly shows and works in Celsius. However, in the Google home app the thermostat shows in Fahrenheit and tries to issue calls to temperatures in Fahrenheit which doesn't work. Is there a way to force a temp unit?

Have you tried saying 'Hey Google, I prefer Celsius'


Compilation of code with jbuilder/ppx that uses a newtype in a function fails

Compilation of code with jbuilder/ppx that uses a newtype in a function fails with Error: Unbound type constructor a
Minimal failing test case: https://gist.github.com/reynir/0ca3a1254bd47133dd308d5e2895923e

I don't believe the issue is with jbuilder or the ppx. If you convert the code to OCaml it Just Works™.

Also, the ppx in the example is an empty ppx so nothing is done.

hmm what version of reason are you on?

The latest version of reason isn't on opam yet, that might be the discrepancy. I'll reopen until we figure it out

@jaredly Reason 3.2.0 is on opam now thanks to @pjhenning

It still fails on Reason 3.2.0.

Can't repro anymore. Feel free to reopen if still a bug.


Nearest-neighbor and graphics.beginBitmapFill()

I am a bit new to OpenFL so it might be that I am doing something wrong. Using the latest version (4.0.3) of OpenFL, I am trying to draw an upscaled pixel-art sprite. The code I am using:
graphics.clear();
graphics.beginBitmapFill(bitmapData, null, false);
graphics.drawRect(0, 0, bitmapData.width, bitmapData.height);
graphics.endFill();

The third argument of the beginBitmapFill method turns smoothing on or off. Yet the sprite still looks blurry and not the way it should look if it was using nearest-neighbor interpolation.
I was not able to reproduce this issue on OpenFL version 3.6.1 and Lime 2.9.1.

Same issue on macOS native

Is it also blurry if you compile with openfl test mac --window-hardware=false?

Somehow it doesn't want to let me compile with --window-hardware=false option, compiler says that option doesn't exist, do I have something set wrong? I'm using latest haxelib version.

Yes.
Using Shape & graphics.beginBitmapFill():

Visible boundaries are individual shapes.
Using Bitmap & bitmapData.copyPixels():

which is expected result.

Is it on any platform or you means html5?


Column.cellClassRules type misleading w.r.t. autosize support

Our type for Column.cellClassRules references ColumnCellClassRuleFn which is a function that specs and arg of type AG-provided CellClassParams
https://github.com/xh/hoist-react/blob/30b3222a51fc9746debe35eaebeec20643033cc8/cmp/grid/Types.ts#L239
That includes a number of params - notable column/colDef that are not provided when the same function is called during an autosize operation by ColumnWidthCalculator:
https://github.com/xh/hoist-react/blob/30b3222a51fc9746debe35eaebeec20643033cc8/cmp/grid/impl/ColumnWidthCalculator.ts#L355
Ideally we would support as many of the potentially-useful params here as we can. In any case, we should make it more clear what the limits are.

@jskupsik do you recall what prompted this or how we came across it?


Small fixes in recipe and input file for recipe

BEGINRELEASENOTES

Recipe updated using env variable K4MARLINWRAPPER
clicRec_e4h_input_gun_clue.py file updated w.r.t. to the last version in the wrapper
ENDRELEASENOTES

hi @jmcarcell thanks for the comments. If you are satisfied with the replies, we can merge this PR.


OutlineEffect not work on my gltf model

this is my init() function
  this.scene = new THREE.Scene()

  this.renderer = new THREE.WebGLRenderer({ antialias: true })
  this.renderer.setPixelRatio(window.devicePixelRatio)
  this.renderer.setSize(this.container.clientWidth, this.container.clientHeight)
  this.renderer.shadowMap.enabled = true
  this.container.appendChild(this.renderer.domElement)

  // PostProcess
  this.composer = new EffectComposer(this.renderer)

  this.renderPass = new RenderPass(this.scene, this.camera)

  this.outlineEffect = new OutlineEffect(this.scene, this.camera, {
    edgeStrength: 10,
    pulseSpeed: 0.0,
    visibleEdgeColor: 0xffffff,
    hiddenEdgeColor: 0x22090a,
    blur: false,
    xRay: true
  })

  const outlinePass = new EffectPass(this.camera, this.outlineEffect)
  this.renderPass.renderToScreen = true

  this.composer.addPass(outlinePass)
  this.composer.addPass(this.renderPass)

  let scene = this.scene
  let outlineEffect = this.outlineEffect
  // model
  let loader = new GLTFLoader()
  loader.load('http://localhost/Models/gltf/test.gltf', function (gltf) {
    gltf.scene.traverse(function (child) {
      if (child.isMesh) {
      }
    })
    gltf.scene.position.set(0, 20, 0)
    gltf.scene.rotation.set(-90 * Math.PI / 180, 0, 0)
    gltf.scene.scale.set(5, 5, 5)
    scene.add(gltf.scene)
    console.log(scene.children[3].children)
    //setup selection objects
    outlineEffect.setSelection(scene.children[3].children)
  })

I have set outlineEffect.setSelection while I load my Model
this is my function handle outlineEffect
    addSelectedObject: function (object) {       this.selectedObjects = []       this.selectedObjects.push(object)       this.outlineEffect.selectObject(object)     }
it appears nothing when i click in my scene, I'm not so sure where this issue comes from
here is my model on console

if I remove this.composer.addPass(this.renderPass) ,scene appears tobe like this

Hi,
try changing
  this.renderPass.renderToScreen = true

  this.composer.addPass(outlinePass)
  this.composer.addPass(this.renderPass)

to
  this.outlinePass.renderToScreen = true

  this.composer.addPass(this.renderPass)
  this.composer.addPass(outlinePass)

This allows the RenderPass to render the geometry to an internal frame buffer which can then be modified by the EffectPass.

  this.composer = new EffectComposer(this.renderer)
  this.renderPass = new RenderPass(this.scene, this.camera)

  this.outlineEffect = new OutlineEffect(
    this.scene,
    this.camera,
    {
      edgeStrength: 10,
      pulseSpeed: 0.0,
      visibleEdgeColor: 0xffffff,
      hiddenEdgeColor: 0x22090a,
      blur: false,
      xRay: true
    }
  )
  const outlinePass = new EffectPass(this.camera, this.outlineEffect)
  outlinePass.renderToScreen = true

  this.composer.addPass(this.renderPass)
  this.composer.addPass(outlinePass)

I have change my postprocessing part like u said
emmmm....  seems it does not work 😥

O I have found reason
frame have render twice in my update function
one in composer
another one in this.renderer.render(this.scene, this.camera)
I think it my cover my composer frame😂


Add new configuration for src15

New Prod config
Self check-list:

[x] I have checked the lstchain config, in particular for:

[x] az_tel instead of sin_az_tel if data to be analyzed have been produced with lstchain <= v0.9.7
[x] "increase_nsb" and "increase_psf" are provided in "image_modifier" (if used)

[x] I have checked the environment in the lstmcpipe config and it is the one used to analyse DL>1 data
[x] I have provided the command (in README), or script (in additionnal .py file) used to produce the lstmcpipe config

Prod_ID
Short description of the config
Why this config is needed

Hi @vuillaut
Is there anything else that should be provided for the production?

@nikoliclisa
This configuration cannot work, it starts from a source prod 20240917_v0.10.12_src15_dec_min_2924_tuned that does not exist (it's yours).
You must start from an existing prod. In this case, the most recent base one is 20240131_allsky_v0.10.5_all_dec_base.
Please reproduce the config (you will need to pull the changes I made to lstchain config to be able to push again on that branch).


editing ground truth actions

hello, is it possible to change the ground truth actions

Of course, you can assign gt_actions in eval.py on your own.

Also, one more question, how can i evaluate only one episode that i choose from the demonstration npz files.

Your code is ok, I think. To evaluate on a specific demonstration, you can add some operations in load_data function. For example:
for npz_path in demo_path:
    if file_name not in npz_path:   # add
        continue   # add
    data.append(np.load(npz_path, allow_pickle=True))
    frames.append(npz_path)

Thank you, appreciate the fast response.

Hi, i gave the peract model the customized gt as you said in the eval script, i gave it the position, orientation, and the gripper state.
But the task failed, should the Gt actions be more than these 3 informations?

How do you provide position, orientation, and the gripper state? Peract does not have 100 % success rate . Please refer to the paper.

Why would you overwrite the gt actions? We have provided them in the demonstration file. If you want to overwrite, make sure the customized action is accurate.


Database Query Callback Not Working

Versions

scout-elastic-driver: latest stable
laravel/framwork: 5.8
php: 7.2
elasticsearch: 7.3.0

Description
When trying to add database constraints using the query method, the constraints are not applied to the results.
Steps to reproduce
MySearchableModel::search($query)
    ->query(function (Builder $builder) {
        $builder->where('column', 'value');
    });

+1

The fix was merged and will be released soon

Versions

scout-elastic-driver: latest stable
laravel/framwork: 5.8
php: 7.2
elasticsearch: 7.3.0

Description
When trying to add database constraints using the query method, the constraints are not applied to the results.
Steps to reproduce
MySearchableModel::search($query)
    ->query(function (Builder $builder) {
        $builder->where('column', 'value');
    });

Worked for me


Adds initial implementation for caching middleware

Adds implementation for caching middleware

Don't like ristretto, it looks as overcomplicated library for our current needs,  currently considering https://github.com/jellydator/ttlcache as more strait forward option

Codecov Report
Attention: Patch coverage is 0% with 46 lines in your changes missing coverage. Please review.

Project coverage is 88.02%. Comparing base (48d0786) to head (72a86dc).
Report is 6 commits behind head on main.

Files
Patch %
Lines

middleware/request/cache.go
0.00%
46 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   92.65%   88.02%   -4.64%
==========================================
  Files          23       24       +1
  Lines         926      935       +9
==========================================
- Hits          858      823      -35
- Misses         50       94      +44
  Partials       18       18

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


Max number of clients?

Is there a max number of clients, it seems that after 5 clients I can't connect more.

This is usually defined in firmware.


🚸 Unique contraint on name field of most registries

The Feature and ULabel registries have a unique constraint on the name field:

Feature
ULabel

Let's review all other label registries towards whether they also have it.
Note: It's important to note that versioned entities shouldn't have that unique constraint because multiple versions of the same entity should be able to have the same name; e.g. Transform and Collection.
Internal Slack ref.

Indicating this in the docs:

https://github.com/laminlabs/lnschema-core/commit/589db75aa7c47e52067aadee9652ff68f32b2a49

I looked at wetlab. It has:

Compound: Were using the Drugontology for. Tricky with differentSource`.
Experiment: I think you can have the same experiment with different dates. Not sure whether we should make name unique here.
Well: IDEK whether they need a name in the first place but it should be okay here. If anything, row and column should be (jointly)? unique here.
PerturbationTarget: Guess it's okay to have unique name.
GeneticPerturbation: You can target the same genes with different CRISPR sequences. Users would therefore potentially be forced to encode the sequences in the name which is bad. Not sure.
CompoundPerturbation: Same compound name with different concentrations surely happens in experimental setups. Learning more towards a "no".
EnvironmentalPerturbation: Same argument. You can use different temperatures etc. "no".
CombinationPerturbation: Technically hidden concentrations but w/e.
BioSample: Can have different batches and we might need to encode the batches in the name then. But okay I guess.
TechSample: Different batches but okay.
Donor: No. Names cannot be unique.

My conclusion is that while we could do it, it's always going to be a case by case decision and there's quite some grey area. I fear that this would lead to inconsistencies and unexpected behavior.


[BUG] RWX volume attached in maintenance mode to do a backup, not option to detach.

Describe the bug
RWX volume attached in maintenance mode to do a backup, not option to detach.
To Reproduce
Steps to reproduce the behavior:

Create a RWX volume
Use it
Scale down your deployment/sts so the volume detaches
Go to the longhorn UI, click attach, choose a node, check maintance.
Take a backup (optional).
Try to detach. ( Option is grayed out.)

Expected behavior
Step 6 above should work.
Environment:

Longhorn version: 1.1.0
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Rancher Catalog App
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: RKE

Number of management node in the cluster: 3
Number of worker node in the cluster: 3

Node config

OS type and version: OL7.9
CPU per node: 8
Memory per node: 32
Disk type(e.g. SSD/NVMe): SSD
Network bandwidth between the nodes: n/a

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): VMWare
Number of Longhorn volumes in the cluster: 19

I know RWX are experimental.

@cocampbe This issue is fixed in version v1.1.1

v1.1.1

I had a feeling. I just could not remember. Thanks!

I'm closing this issue for now, feel free to reopen this if you see problem again.
Thank you @cocampbe


Add LogType raw

Feature request

What is the expected behavior?
Would be possible to pipe output from other process into webpack logging flow without changing format.
What is motivation or use case for adding/changing the behavior?
Current when writing simple plugin that spawn typescript in watch mode, and pass output to webpack logger overrides styles and add's a prefix.

How should this be implemented in your opinion?
Are you willing to work on this yourself?
yes

Why do you need this? This reduces the readability of errors, since it is not clear which plugin displays errors/warnings/logs

Why do you need this?

When running webpack --progress and tsc --watch --preserveWatchOutput with concurrently output from webpack progress connects with tsc output at one line, without any space.

since it is not clear which plugin displays errors/warnings/logs

Imho idea in raw is that it could be configurable what user want's to add there. So would be possible to add plugin name [my-webpack-plugin] or other prefixes but as optional.

Do you mean logger.raw, right?

Yes.

Make sense :+1:

I think we have the same here https://github.com/webpack/webpack/issues/10148

But here is more clean idea/solution

I don't think raw is the solution here. It lacks log level info.
I think it's better to detect formated messages with the existing methods and don't mess that up.
The other proposal is also needed to reduce the plugin name spam, but it should always include the plugin name.

I don't think raw is the solution here. It lacks log level info.

We can require logLevel as first argument, like logger.raw(3, '[my-webpack-plugin] ' + data) Then we can have custom formatting off each level.
Current using node-pty to run tsc --watch --preserveWatchOutput to preserve colors and formatting passing the data to logger.info we have

To be honest it doesn't look that bad, but would drop the <i> on each line, maybe include it as part of name in []

Added draft PR  #11576, have a look and if is something that could be merge?
This basic would allow webpack logger to be more flexible, here using the logger.raw for output from tsc

@sokra what was your idea about this

The other proposal is also needed to reduce the plugin name spam, but it should always include the plugin name.

maybe just append name to first message?

Created poc repository with what i'm trying to achieve https://github.com/piecyk/tsc-webpack-plugin-example
As mention above, basic want to log with InfrastructureLogger as not to get this joined lines with webpack status logging.
Having the prefix <i> is not that bad, but would be awesome if we could opt out from colours in output. When there are many errors it's just got too flashy.
This could be alternative to fork-ts-checker-webpack-plugin as imho it's a bit heavy when used only in typechecking mode. As there was many optimisation done to tsc for example incremental, tsBuildInfoFile We can just run the tsc next to webpack and just glue output from them together. Don't seeing a reason why they need to be connect on low level.

We could also pass options object to getInfrastructureLogger that would override default. Current it would be possible to change colorPrefix, colorSuffix WDYT?
const logger = compiler.getInfrastructureLogger("name");
logger.info("with default colors");

const logger = compiler.getInfrastructureLogger("name1", {
  info: null,
});
logger1.info("no colors");

const logger2 = compiler.getInfrastructureLogger("name2", {
  info: {
    colorPrefix: "\u001b[1m\u001b[31m",
    colorSuffix: "\u001b[39m\u001b[22m"
  }
});
logger2.info("red colors");

Something like https://github.com/webpack/webpack/compare/master...piecyk:poc-logger-options 🤔

I think it is good feature

FYI extracted plugin to https://github.com/piecyk/tsc-webpack-plugin

@piecyk still we need this?

@piecyk still we need this?

Yep, let's close it. Thanks @alexander-akait 👍

Yep, let's close it. Thanks @alexander-akait 👍


GEOMESA-1815 Fix Density Coprocessor Empty Scan Ranges and other impr…

…ovements
Squashed commit of the following:
commit bded0ccc14c60d3ac0715927bdd7606f644be3bf
Merge: 9a09366 7676c37
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Mon May 15 14:37:12 2017 -0400
Merge branch 'master' into fcr_hbase_density_filter_jnh_fixes_2

commit 9a093669dd41b57761171a3ae4d7f861d5abf948
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Fri May 12 11:37:35 2017 -0400
Git comment revisions, moved coproList to coprocessor package

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit 78b905cb2b02967a83122fa07b08ba1450eaebb9
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Thu May 11 10:31:25 2017 -0400
Git comment revisions

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit 32ef803a24f3c089d64e5e8ae58108dfc2e82423
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Wed May 10 17:48:58 2017 -0400
Git comment revisions

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit d4e80ad48d01a38463b5cc528d2852a182fc6d55
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Mon May 8 16:49:25 2017 -0400
Small revisions

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit c8234993fbc2407bfebe2a56b4515d1550599b85
Merge: 4ccd618 1f3eb57
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Mon May 8 16:35:11 2017 -0400
Merge remote-tracking branch 'upstream/master' into fcr_hbase_density_filter_jnh_fixes_2

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
Conflicts:
	geomesa-hbase/geomesa-hbase-datastore/src/main/scala/org/locationtech/geomesa/hbase/coprocessor/KryoLazyDensityCoprocessor.scala
	geomesa-hbase/geomesa-hbase-datastore/src/main/scala/org/locationtech/geomesa/hbase/data/HBaseDataStoreFactory.scala
	geomesa-hbase/geomesa-hbase-datastore/src/main/scala/org/locationtech/geomesa/hbase/data/HBaseQueryPlan.scala
	geomesa-hbase/geomesa-hbase-datastore/src/main/scala/org/locationtech/geomesa/hbase/driver/KryoLazyDensityDriver.scala
	geomesa-hbase/geomesa-hbase-datastore/src/main/scala/org/locationtech/geomesa/hbase/index/HBaseFeatureIndex.scala
	geomesa-hbase/geomesa-hbase-datastore/src/test/scala/org/locationtech/geomesa/hbase/data/HBaseDensityFilterTest.scala
	geomesa-hbase/pom.xml

commit 4ccd618907ff78dc70faf31841cb61e5c55679c8
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Mon May 8 15:22:34 2017 -0400
Added coprocessor auto registration and small syntax improvements.

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit 56634b582918c619022a7ebfbc8f94bc3d22e09a
Author: Austin Heyne<EMAIL_ADDRESS>Date:   Wed May 3 13:33:53 2017 -0400
Updated hbase density filter unit tests

Signed-off-by: Austin Heyne<EMAIL_ADDRESS>
commit b6c7e0f4a7fdc127b98268bad8a67ac506f1100e
Author: Jim Hughes<EMAIL_ADDRESS>Date:   Tue May 2 21:22:49 2017 -0400
Cleaning up tests some.

commit 14f606d327a9e73a98c6aadd88b94390a6a08354
Author: Jim Hughes<EMAIL_ADDRESS>Date:   Tue May 2 21:19:21 2017 -0400
Add MRFF support for range handling.

commit 9c517ede3ce2d6ab3c0dd5feb0838107486ee47d
Author: Jim Hughes<EMAIL_ADDRESS>Date:   Tue May 2 20:16:09 2017 -0400
More unit test updates.

commit 12706f4972179662fe8dd050fbd6fcaf76adb340
Author: Jim Hughes<EMAIL_ADDRESS>Date:   Tue May 2 18:51:15 2017 -0400
Refactoring test and DensityDriver.

commit b9083ee473e3659e42db2a77b190c8640ac2dcff
Author: Jim Hughes<EMAIL_ADDRESS>Date:   Tue May 2 13:37:27 2017 -0400
Quick fixes to see a heatmap.


Bug Report: uncaught ECONNREFUSED errors causing runtime errors without bail flag

Continued from discussion in #705
Expected: if server refuses an http connection for an httpyac client request, the http region / test should be reported as a failure in the test report / output but should continue on to execute the next http region / test until all tests have been executed -- unless of course the --bail cli flag is provided in which case, propagating the error and terminating the test suite early is the desired behavior.
Actual: if server refuses an http connection for an httpyac client request, test execution terminates early and the ECONNREFUSED error is propagated regardless of whether the bail flag is supplied or not.

I have now restructured it once so that the exceptions only affect the respective httpRegion. This means that I had to revise the handling for ExitCode and bail. I should have tested everything thoroughly, but this could still lead to unexpected conflicts. But I agree with you that the behavior for tests is not optimal and that your desired behavior should be implemented here. I will release it with the next release


update to servlet API v6

tried to use latest server versions everywhere, tomcat 10.1.2 is buggy though https://bz.apache.org/bugzilla/show_bug.cgi?id=66353 so 10.1.1 it is
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[X] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[X] Ensure the tests and linter pass
[X] Code coverage does not decrease (if any source code was changed)
[X] Appropriate docs were updated (if necessary)

Fixes #283  ☕
relates to #161

@globalworming , thanks again for opening this PR. Based on my conversation with creator of this library, all versions of the supported libraries were chosen simply to cover wider audience of developers which are using some older javax.servlet versions. Any updates to those dependencies should be done only in cases when existing version goes EOL (end of life). If you need to support newer version of the javax.servlet, feel free to fork the code from this repository and perform the change according to your needs.
I am going to close this PR for now - feel free to reactivate or comment If you believe that your change is still needs to be committed


publish docker image

Hi @Issif . Could you please push the 2.10.0 image to docker hub when you have some time?
Thanks.

@palmerabollo I forgot, sorry.
It's pushed : 2.10.0
Thanks.


Including square brackets in arguments does not pass to poe

version: 0.24.4
system: macOS

Issue
When including square brackets in an argument (either positional or non-positional), it seems to break how poe sends along the arguments to the actual task.
Reproduction
# pyproject.toml

[tool.poe.tasks.hello]
cmd = "echo $name"
args = [{ name = "name", positional = true }]

This works as expected:
poe hello "world"
Poe => echo world
world

This does not:
poe hello "world[extra]"
Poe => echo

Use Case
I have a sequence command to install poetry packages (poe add) which runs poetry add $package, and then a second command to sync them with my dev container:
[tool.poe.tasks.add]
sequence = [{ cmd = "poetry add $package" }, { ref = "sync-dev-deps" }]
args = [{ name = "package", positional = true }]

All good if I'm trying to install something like django or numpy, but for something with "extra" packages (ex. django-anymail[mailgun]), I'm not able to utilize this command and need to fall back to poetry add "django-anymail[mailgun]".
Hopefully this isn't documented anywhere - if it is, feel free to point me in that direction and close the issue! Thanks so much for the awesome package, fits my needs exactly.

Hi @stephen-spar, thanks for reporting this issue.
It looks like the square brackets from the argument are getting picked up in the cmd task as glob pattern to be interpreted. This may be revealing of a design flaw at the intersection of how cmd tasks implement shell like features (including parameter expansion and glob expansion) and how they interpolate CLI arguments as env vars. I'll have to think about how to resolve this because I agree it is a surprising result.
Until I work out how to fix it you can work around the issue by adding some additional quotes around the argument value, so that the value includes the double quotes to disable the globbing.
e.g. poe hello '"world[extra]"'

@nat-n ah, thank you for the workaround - works a treat. Appreciate your quick response, and again, thanks for the phenomenal tool, it vastly improves the DX of the project I'm working on. Keep up the great work!!

The relevant behaviour of bash like pattern-matching/quote/escaping is documented here.


Backport fix for "Changing vSphere Credentials causes old nodes to fail on deletion" to v2.7

We need to backport the fixes in rancher-machine and rancher for https://github.com/rancher/rancher/issues/40608.
SURE-7239

Ticket #43574 - Test Results - ✅

Scenario
Test Case
Result

1.
Rotate vSphere credentials without updating rancher credentials
Pass

2.
Rotate vSphere credentials and update rancher credentials
Pass

Verified with (HA Helm or Docker) on Rancher v2.7-head:
Scenario 1: Rotate vSphere credentials without updating rancher credentials

Create a 1 node vSphere cluster
Rotate the vSphere credentials
Delete the cluster node.

Results: The cluster displays an error message as expected

Scenario 2: Rotate vSphere credentials and update rancher credentials

Create a 1 node vSphere cluster
Rotate the vSphere credentials
Update the rancher credentials to match the vSphere credentials
Delete the cluster node.

Results: The node is deleted as expected.
Additional Notes:
If your cluster's node is in an errored state due to credentials, update the credentials to the correct ones and then force delete the node to remove it.


Show similar existing silences when creating a new silence

It would be useful if, when creating a new silence, a list of similar silences was shown. My use case is for avoiding accidental silencing of similar but separate alerts mistakenly. I recently mistakenly silenced an alert that eventually lead to an outage because the alert name was the same, and another label (k8s cluster + namespace) was so similar that I assumed this second alert that I now was creating a silence for should have fallen under the first silence, and that the original silence had been created without an extended duration (default 2h).
If while I was creating that second silence, I could have seen something like "hey, there's already silences for this alertname but with these other label values" this could have been avoided.
cc @pracucci

Also, similar recently expired silences would be good to show to make it easier to recreate silences.


Use contants string instead of string literals as cacerts secrect name

Please provide a description of this PR:
When with external Istiod, we want to beable to load cacerts from a remote cluster, the secrest name is a string literals, we'd better use a constants instread.

Hi @escoffier. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@escoffier: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
06d2b79a5bd52754faeda2296d13ef791b6debda
link
true
/test release-notes

lint_istio
06d2b79a5bd52754faeda2296d13ef791b6debda
link
true
/test lint

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

@escoffier: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
06d2b79a5bd52754faeda2296d13ef791b6debda
link
true
/test release-notes

lint_istio
06d2b79a5bd52754faeda2296d13ef791b6debda
link
true
/test lint

gencheck_istio
06d2b79a5bd52754faeda2296d13ef791b6debda
link
true
/test gencheck

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

/release-note-none

/label release-notes-none

/release-note-none

/retest

/release-note-none

@escoffier: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
eacfa4ed278613631fed8c4df4565f658e2d0017
link
true
/test release-notes

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

/test release-notes


IP2LOCATION-LITE-DB1.IPV6.BIN manually download and use not working ?

Hello,
I have a question because I don't understand something.
Basically the plugin works fine. My config:
In Traefik 2.x docker-compose as command:
- "--experimental.plugins.geoblock.modulename=github.com/nscuro/traefik-plugin-geoblock"
- "--experimental.plugins.geoblock.version=v0.11.0"

Volume - ./traefik-plugins/:/plugins-local/
Traefik dynamic config:
http:
  middlewares:
    geoblock:
      plugin:
        geoblock:
          # Enable this plugin?
          enabled: true
          # Path to ip2location database file
          #databaseFilePath: /plugins-storage/sources/gop-<random-numbers>/src/github.com/nscuro/traefik-plugin-geoblock/IP2LOCATION-LITE-DB1.IPV6.BIN
          databaseFilePath: /plugins-local/IP2LOCATION-LITE-DB1.IPV6.BIN
          # Whitelist of countries to allow (ISO 3166-1 alpha-2)
          allowedCountries: [ "DE" ]
          # Allow requests from private / internal networks?
          allowPrivate: true
          # HTTP status code to return for disallowed requests (default: 403)
          disallowedStatusCode: 403
          # Add CIDR to be whitelisted, even if in a non-allowed country
          allowedIPBlocks: ["x.x.x.x/32"]

For Service (test with whoami) i add in compose file:
- "traefik.http.routers.whoami.middlewares=geoblock@file"
First i use /plugins-storage/sources/gop-<random-numbers>/src/github.com/nscuro/traefik-plugin-geoblock/IP2LOCATION-LITE-DB1.IPV6.BIN as databaseFilePath. But with every Traefik restart the numbers in the path change, so I copied the file to /plugins-local/ and changed the databaseFilePath to this path.
(Path /plugins-local/src/github.com/nscuro/traefik-plugin-geoblock/ from project Readme default not exists.)
This also works, not allowed countries are blocked!
But now I want to use the latest version of IP2LOCATION-LITE-DB1.IPV6.BIN and have downloaded it from the website https://lite.ip2location.com/database/ip-country and overwrite the old file. (The permissions are the same: chmod 755 and root user.)(The bin file is a bit larger.)
But this not working, it does not work even after restarting Traefik and the service. No errors are displayed in Traefik Dashboard.
I have access from allowed country and all others (tested over VPN).
What could be the reason for this? Can I somehow see the logs from Geoblock to see if there is an error when reading the file? (My Traefik log-level is debug.)

Hey @Tob1as, TBH I don't know what is causing the issue you're having.
The dynamic plugin file path is new to me, that didn't use to happen when I originally built the plugin. I personally install the plugin in local mode (see https://plugins.traefik.io/install), so I always have static file paths.
Your latest approach should work, and I don't see a reason why it doesn't. As for the logging, the plugin checks on startup if the database file exists and can be opened successfully. If that doesn't work, it will return an error:
https://github.com/nscuro/traefik-plugin-geoblock/blob/069f3ec4b950b3ee3cf938866ebbfc0b9bc19156/plugin.go#L74-L81
Maybe give this a try:

docker exec into the traefik container, as the traefik user
Try to access the database file (e.g. od IP2LOCATION-LITE-DB1.IPV6.BIN | head)

Hi,
i don't get any of these two errors or other. Access also works.
Now i make update from 0.11.0 to Version 0.13.0 and copy the new database contained in the plugin (path) to my path. Now I have the same problem as when I load the database from the manufacturer's website, my blocks are ignored. If I use the database from 0.11.0 again (copy in my path), the blocking works again.
(Also not working when i use plugin path /plugins-storage/sources/gop-19****51/src/github.com/nscuro/traefik-plugin-geoblock/IP2LOCATION-LITE-DB1.IPV6.BIN. If i use a wrong path, then i get an error.)
I will continue to test it.


[service-bus] recreate Connection object on disconnect

This change updates the ConnectionContext disconnect behavior.
Previous to this change, when a disconnect event was detect and the SDK attempted to recreate its links/sessions/connection, it would do some cleanup of the rhea-promise connection object and re-use it. Normally this works fine, but over time we've seen issues in logs that indicate that the connection object may be getting into an unexpected state in some cases.
This PR updates the disconnect behavior to create a new rhea-promise connection object when the SDK determines that the links/sessions/connection should be recreated.
This relies on #8444 to be merged first.
Testing
To test this change, I create a streaming receiver and run inside a docker container. With logging enabled, I disable the network connection for the docker container until I see a disconnect event fire. Then I re-enable the connection and observe that the receiver begins reading messages again.
/cc @ramya-rao-a

/azp run js - servicebus - tests

Ah, CI is expected to fail until #8444 is merged and released.

/azp run js - servicebus - tests

/azp run js - servicebus - tests

I added some tests for streaming and batched receivers (and the same sender is used before and after disconnects in both tests as well).
One thing that doesn't work is that the session receiver doesn't come back up on disconnect. Looking through the logs, it doesn't even look like the session receiver has detached called on it, and it never attempts to come back online.
I'm not sure if this is intentional or not due to the nature of how sessions require a lock. I noticed I didn't seem to be able to recover when using service bus 1.1.5 with a session receiver either. @ramya-rao-a  do you have any insight on whether session receivers should reconnect?
I can spend some time tomorrow looking at the session receiver code (I'm less familiar with this part) to see why the receiver isn't being recreated.

I reviewed the messageSession code and see that there isn't any logic in place to recover a session receiver when it is closed either due to a disconnect or an amqpClose/sessionClose event is fired.

One thing that doesn't work is that the session receiver doesn't come back up on disconnect. Looking through the logs, it doesn't even look like the session receiver has detached called on it, and it never attempts to come back online.

Yes, that is intentional. Streaming receiver in sessions will not recover. Try batching in sessions, that should work