Build failing on unstable update for PRs from fork

PR #151 is getting HTTP 403 Forbidden.
It looks like this is because the source purnimab:master is not part of reflectometry and so it cannot push the windows unstable exe to the release page. [Link may be broken if github expires build output.]
We still need to build and test from the fork but don't try deploy. I think we can do this by adding the following to unstable.yml:
jobs:
    test_and_build:
        …
    publish:
        if: github.repository_owner == 'reflectometry'
        …
    updateUnstableText:
        if: github.repository_owner == 'reflectometry'
        …

From https://github.com/prisma/prisma/issues/3539

Do we want to tag the unstable with the branch name and/or commit hash?
That might remove some potential confusion if a change to another PR replaces unstable you have had a chance to grab it. In particular, if you send a link to a collaborator in an email you don't want them unexpectedly grabbing the wrong file.
This means we will have to remove previous builds. If we use a consistent naming pattern Refl1D-latest-COMMIT-… then we can remove Refl1D-latest*. (I see that we are using Refl1D-windows-exe-latest.zip; that should change to Refl1D-latest-windows-exe.zip even if we don't stamp the builds.
If we encode the date in the stamp then maybe we can remove everything more than one week old. If files are named for example Refl1D-unstable-YYYYMMDD-COMMIT-* then maybe we can remove anything less than Refl1D-unstable-$STAMP where STAMP=$(date -j -v-7d "+%Y%m%d")

Tagging is a good idea, so that people know what they've downloaded.  Don't know if it's worth keeping previous builds.  It might be just as hard to delete all previous builds as it is to keep some of them, though.
I would still prefer not to build from any pull requests.

I have occasionally wanted such a thing.

Build_Unstable is failing for me as well on merge with master:

could not find upload_url corresponding to release unstable

That's very odd... I re-ran the failing job and it worked the second time.  Momentary github API failure?

Github gaslight: the above link now shows success :-(
Is it possible that the secrets are associated with bmaranville rather than reflectometry?

I don't think the credentials are tied to who pushes the button to re-run the job.

Just tried it. Yes it worked. I'll go with the "momentary failure" hypothesis for now.


Remove built CSS files from repository

Including the built semantic.css and semantic.min.css as we are doing currently will cause merge headaches for us and also will bloat the size of the repository.
I propose we add all the generated files to .gitignore, and add a prepublish step to our package.json, so that whenever we publish a new version to NPM, the built files get generated and included there, so they can then be pulled down from UNPKG.

yes this makes sense. please lets go ahead with your suggestion

Great. I'll try it now.


how to specificly define the class for grounded-segment-anything?

You can refer to our demo and pass the specific classes through the bash scripts


WMG API query endpoint - cell ordering

The query endpoint response must order cell type according to the cell type ordering file, generated by the backend:

Location: s3://wmg-<env>/<snapshot_uuid>/cell_type_orderings.json
Structure: { "<tissue_term_id>": [ "<cell_type_term_id>" ], … }

Draft cell ordering data at s3://madison-tmp/v1/cell_type_ordering.json, can be used for testing.


Create new Metadata table + add a new create method

Create a new Metadata table and a new add_to_sample method for Metadata.

We will start using the new Metadata table and eventually migrate / stop using the old "metadata columns" on Sample.

For now we basically define Metadata as being either a string or a number. The strings can have limited string options that will be checked if present. It seems like the vast majority of metadata can just be treated as strings.

raw_value is for the original inputted value. validated_value is for the value that passed validation.

Tested pretty well locally.

See this doc for reference: https://czi.quip.com/fZtIAkROUlqB/Metadata-in-IDseq

Thanks all for the comments. I reworked it to run as a Rails validation function to fit with that model better. The only trade-off is to be aware that the validation function now has a side effect in setting the *_validated_value column but that's not too bad.
Also changed things to use constants and ActiveRecord "enum".
Re bulk SQL inserts I've made a note of it for later but in my experience the metadata inserts have been pretty fast / have a limited scope to insert into the db.


Add sparse float vector support

Milvus 2.4 has added beta support to sparse float vector to allow efficient storing, indexing and searching of such vectors. The main targeted use case is to support learned sparse models such as SPLADE and statistics based methods such as BM25.
We need to add sparse float vector to node SDK as well. See: https://github.com/milvus-io/milvus/issues/29419#issuecomment-2024887328 for more details.
Thanks a lot for the efforts!

it is in the 2.4 branch,  thanks.

it will be released once milvus 2.4 released.

thanks @shanghaikid! that is great to know!
We are actively making more advanced features available to sparse fields such as range search and grouping search etc. I hope those features will just work once we finished on the milvus side, but let's leave this issue open and verify when those are ready.
We are planning to have those advanced feature support added in the next major release though, not in 2.4.


kube-prometheus change the k8s node internal-ip to public ip

Environment：
kube-prometheus v.0.6.0
kubernetes v1.18.6
before install:
kubectl get node -o wide
NAME             STATUS   ROLES    AGE   VERSION   INTERNAL-IP    EXTERNAL-IP
ali-k8s-master   Ready    master   16d   v1.18.6       <IP_ADDRESS>
ali-k8s-node     Ready       5d    v1.18.6       <IP_ADDRESS>
but when i finish the installation:
kubectl get node -o wide
NAME             STATUS   ROLES    AGE   VERSION   INTERNAL-IP    EXTERNAL-IP
ali-k8s-master   Ready    master   16d   v1.18.6     <IP_ADDRESS>
ali-k8s-node     Ready       5d    v1.18.6       <IP_ADDRESS>
It is a problem that the internal network ip of the node becomes the public network ip, which will cause many functions to malfunction.How to solve it？

Do you mind to elaborate on the actual question? I don't see anything wrong except the -node address being changed. Is that the problem you are referring to?

Do you mind to elaborate on the actual question? I don't see anything wrong except the address being changed. Is that the problem you are referring to?-node

Yes, it is a problem that the internal network ip becomes the public network ip, which will cause many functions to malfunction

So far i don't see where in prometheus-operator manipulate node endpoints. How did you install prometheus-operator?

So far i don't see where in prometheus-operator manipulate node endpoints. How did you install prometheus-operator?

I installed it like this：
cd kube-prometheus-0.6.0/
kubectl create -f manifests/setup
kubectl create -f manifests/

I did a test and only installed altermanager. If all three pods are on the same node, it will happen, and then reduce the number of pods to 1 to restore. But when I start other components, the internal network ip becomes public network ip again.
So I guess it may be that some components cannot run multiple on the same node, or there will be conflicts between components.


cmd,pkg/deploy: Add function to the deploy package that reads the YAML manifests and builds up objects.

/retest

/hold

/lgtm
/approve

/hold cancel
/retest


Move from my personal repo to 47deg

@raulraja @fedefernandez I haven't updated the COPYING file-- some of the code in this repo is from Scala and/or Paul Phillips. Any idea how we should adjust the licensing?

Mention appropriate licenses for all sources and it should be fine.
On Tue, Jun 28, 2016, 9:25 PM Andy<EMAIL_ADDRESS>wrote:

@raulraja https://github.com/raulraja @fedefernandez
https://github.com/fedefernandez I haven't updated the COPYING file--
some of the code in this repo is from Scala and/or Paul Phillips. Any idea
how we should adjust the licensing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/47deg/scala-commas/pull/1#issuecomment-229155224, or mute
the thread
https://github.com/notifications/unsubscribe/AAb4XOZW5Zll_gvqIAJ9U9ChLn1j8q4Nks5qQXURgaJpZM4JAcJF
.

Same with copyrights
On Tue, Jun 28, 2016, 9:25 PM Raul Raja Martinez<EMAIL_ADDRESS>wrote:

Mention appropriate licenses for all sources and it should be fine.
On Tue, Jun 28, 2016, 9:25 PM Andy<EMAIL_ADDRESS>wrote:

@raulraja https://github.com/raulraja @fedefernandez
https://github.com/fedefernandez I haven't updated the COPYING file--
some of the code in this repo is from Scala and/or Paul Phillips. Any idea
how we should adjust the licensing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/47deg/scala-commas/pull/1#issuecomment-229155224,
or mute the thread
https://github.com/notifications/unsubscribe/AAb4XOZW5Zll_gvqIAJ9U9ChLn1j8q4Nks5qQXURgaJpZM4JAcJF
.

I think it should be good. Take a look, and you can have the merge honors.

LGTM!


Issue to run pygcgopt

Dear support team,
Recently, there was an issue on Pyscipopt that could not be reached, but with an update on SCIP, the issue is fixed.
I checked this by Pygcgopt and I found that the issue is still remaining. The following is the output:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
Cell In[6], line 1
----> 1 from pygcgopt import *

File /opt/conda/lib/python3.10/site-packages/pygcgopt/__init__.py:44
     41 from pyscipopt import SCIP_ROWORIGINTYPE
     43 # export user-relevant objects
---> 44 from pygcgopt.gcg import Model, Detector, PricingSolver
     45 from pygcgopt.gcg import PY_GCG_PRICINGSTATUS as GCG_PRICINGSTATUS

ImportError: libscip.so.8.0: cannot open shared object file: No such file or directory

Would you please, check that and say how we can fix it?
All the best
Abbas

I got same error message as well.

Can you please try to pull the new version of pygcgopt from conda and tell me if the problem still occurs?

@jurgen-lentz , I used PyPI to install pygcgopt instead of conda. Do I need to switch to use conda? In addition, I tried to uninstall pygcgopt (0.2.0) and install pygcgopt (0.3.1). But it still not working. Thanks

@jurgen-lentz,
Thank you so much. now, it works fine.
All the best


Data table issue

When we call records from table after records foreach loop end i assign data table to table but jquery code executes first and foreach loop after

@danishshah80 you need to close this issue or give A LOT more detail. There is no way a maintainer can work with this!


Fix tests failures in conditionals_test.py

FAILED ../../../../integration_tests/src/main/python/conditionals_test.py::test_if_else
FAILED ../../../../integration_tests/src/main/python/conditionals_test.py::test_ifnull
FAILED ../../../../integration_tests/src/main/python/conditionals_test.py::test_nullif
FAILED ../../../../integration_tests/src/main/python/conditionals_test.py::test_nvl
FAILED ../../../../integration_tests/src/main/python/conditionals_test.py::test_nvl2

With ANSI mode off only one test fails ../../../../integration_tests/src/main/python/collection_ops_test.py::test_sequence_too_long_sequence with an error message mismatch which should be straight forward to fix


Single run page header area

User story: (4.1) I want to see high-level description of the run so that I can quickly decide if I would like to investigate further
Tasks

[ ] count of # of files (Frames, Tilt-series, Tomograms, Annotations)
[ ] Overview of tilt series, including tilt quality, tilt range, tilt scheme
[ ] Overview of Tomogram, including resolutions available, tomogram processing, annotated objects

Link to design: https://www.figma.com/file/WEmbsjtlBUtRy7pzmuCCjj/CryoET-Data-Portal---Phase-2-Designs?type=design&node-id=1310%3A11321&mode=dev

Screenshot of design:

Following bug was raise while working on this ticket:
Count of # of files titles in Single Run page are not in bold #238
This is not affecting any functionality so Im moving this ticket forward, thank you!


Load Cell: Testing the Part Fully

[x] Prepare instructions on showing how the part works
[x] Describe fully what the part can and cannot do
[x] Include photos and screenshots of the above process
[x] Design and carry out tests for the application
[x] Convert the value to weight
[x] Explain usage of the HX711 library
[x] Compare load cell graph to the resistive band graph


Return the output of executed queries to one CSV file per query.

Return the output of executed queries into one CSV file per query.

Special character in password need to be percent encoded as this document said.
I think, we can add this to documentation for now.

The empty field is still written as kosong or unencoded field still written as kosong.
Please check which is which.

i have test this issue and the merge request already merged.


Pasting line breaks inside title and description blocks inserts line breaks

And this is wrong at least because of the styling of the title block when it has multiple lines of content:

@silviubogan Can you please provide an example content? I cannot reproduce it anymore.

when pasting inside the title block non-space whitespace + duplicated spaces should be stripped. This should be an extension of the Title block, not something that would be treated in the slate editor by default.

This could be handled on a project basis. These are the default plugins for now https://github.com/eea/volto-slate/blob/develop/src/editor/plugins/index.js


Flutter Doc Website Samples are broken

Tested on IconButton page. In chrome, clicking on tab of one sample changes tab of some other sample. In edge, the sample is not rendered.
https://www.youtube.com/watch?v=IdKhHmUbj54&feature=youtu.be
https://www.youtube.com/watch?v=bD7mhvx1lXY&feature=youtu.be

Hi, could you please check if this is still an issue? Thanks to bring this to our attention.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


CLN: Removed SparsePanel

Title is self-explanatory.  Picks up where #11157 left off.

this is in slight conflict with #13776 as I needed to add a line to catch an errant warning about SparsePanel (which you should then remove). But can sort that later I guess.

@jreback : what error catching are you referring to in your PR?

@jreback : what error catching are you referring to in your PR?

[Current coverage][cc-pull] is 85.21% (diff: 75.00%)

No coverage report found for master at 474fd05.

Powered by Codecov. Last update [474fd05...8cb813a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/474fd0519b768efd1c7ec6555930873bf9910e00...8cb813adff06eae10c3751725f45c52484f6cf75?src=pr
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13778?src=pr

lgtm @gfyoung minor doc changes. ping when green.

@gfyoung I think I took out the reference in my other PR (to SparsePanel); I'll rebase after this.

thanks @gfyoung


Fix spelling: interable -> iterable

I saw the "interable" typo in one of the docstring examples. I assume I don't actually need to fill out an RFC for changes on this level...

Thanks @kfindeisen!


Deployment

Deployment:
https://github.com/Randers-Kommune-Digitalisering/kithosting-randers-kommune-apps/tree/test/nexus

Moved repo, needs to be deployed


Text:

For your consideration . . . .
Given the syntax under:
6.2 What Is Debugging?
IF:
In programming, there are as many ways to destroy something as to create something. One stray character is all that’s required. Say you forget a semicolon or use an accented “é” somewhere, but the code is not prepared for such a peculiarity—KABOOM! Or you add two things together, but one is the numeral 4 and the other is “4” as a string, as you might use to say “4 and 20 blackbirds baked in a pie.” To the computer, that “4 and 20” has no numeric significance.
THEN:
This sort of thing really happens, and part of the job is remembering that 4 + 20 is 24 and 4 + “20” is “420”.
SHOULD BE:
This sort of thing really happens, and part of the job is remembering that 4 + 20 is 24 and "4" + “20” is “420”.
[Said simply]
To denote string rather than numeral, the proper character is "4" not 4.
Thank you,
Tia Dobi
P.S. I'm new to GitBit and couldn't figure out how to add this as a Pull request.
P.P.S. One could also say that "4 and 20 blackbirds" denotes 24 blackbirds because here, the word "and" is synonymous with addition. Or, one might consider creating a coder "Proofreading" markup language to close the gap between 4 [and] 20 which would be changed from: 4 20 to 420.

Actually, 4 with no quotation marks might be fine- if you try programming  4+"20" in say a JavaScript console, the console evaluates it to "420", so I think that the article is fine on this. Check this out:


Bug Fix in ramp constraint

Do we need to propagate to v015?

Do we need to propagate to v015?

Yes, I'll include that in my non-spinning reserve PR

Can you add a test to check this please?

@sourabhdalvi we need to study the tests failures before merging. Please a look carefully for this fixes.

Can you add a test to check this please?

So adding the start variable for the 1st period was causing the ramping constraint to be relaxed (some units turn on in the 1st period) and removing that cause it to be infeasible in ED (not enough power in balance constraint) but if I remove the IntegralLimitFF which was enforcing (sum(p_h <= 0.0) in the same period it works


RC version releases are incorrectly formatted

Waterline version: 0.13.0-rc9
Node version: 7.9.0
NPM version: 4.6.1
Operating system: macOS 10.12.4

Hi! I just wanted to point out a slight mistake in how RC releases of this package are being published. According to the npm docs, they should be formatted as 0.13.0-rc.9 (rather than 0.13.0-rc9). The latter isn't valid semver and causes problems with version tooling because npm sorts lexicographically rather than numerically.

(Just replying to keep the bot happy.) This is still an issue, and a new release has been cut since.

(Just replying again to keep the bot happy.)

(Just replying again to keep the bot happy.)


Indexed keys

Adds a third type of key, "indexed", which corresponds to a Redis hash table. Both keys and values go through the encoding machinery, allowing structured Python objects like tuples to be used as keys (but also with a bit of risk of confusion for non-unique serialisations like sets).

I'm also considering whether ImmutableKeyError should be split up. There are really two cases:

Operating on a key of the wrong type. This should really be a TypeError.
Changing a immutable value (of the right type) to a different value.

Backwards compatibility would be an issue. One way would be to invent two new exceptions, both of which inherit from the old so that code that catches the old error continues to work e.g.
class KeyTypeError(ImmutableKeyError, TypeError):
    ...

class ImmutableError(ImmutableKeyError):
    ...

I think it is time for a KeyUpdate object instead of the message tuple just to avoid the endless tuple indexing all over the place.

I agree, good idea.


Add workloadOverrides control for hosttailer (Chart)

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

License
Apache 2.0

What's in this PR?
Add workloadOverrides control for hosttailer in hosttailer-template in logging-operator-logging chart
Why?
With old construction - can't control HostTailer DaemonSet
For example can't add tolerations for running DS pods on master nodes (Its important if u want get logs from master-nodes (k8s audit logs for example))

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


Azure Government: Confidential Client is not supported in Cross Cloud request.

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
For those of us using Azure For US Government, we will encounter the following error in our session data upon redirect:
'__flash_auth:error__': {
[1]     message: '{"error":"invalid_request","error_description":"AADSTS900432: Confidential Client is not supported in Cross Cloud request.

This is because .com is hardcoded in the package. Maybe we should add some flag or something based on the passed in tenant to determine whether to use .us or .com.
Here is the diff that solved my problem:
diff --git a/node_modules/remix-auth-microsoft/build/index.js b/node_modules/remix-auth-microsoft/build/index.js
index f088946..aa41ae8 100644
--- a/node_modules/remix-auth-microsoft/build/index.js
+++ b/node_modules/remix-auth-microsoft/build/index.js
@@ -8,8 +8,8 @@ class MicrosoftStrategy extends remix_auth_oauth2_1.OAuth2Strategy {
             clientID,
             clientSecret,
             callbackURL,
-            authorizationURL: `https://login.microsoftonline.com/${tenant}/oauth2/v2.0/authorize`,
-            tokenURL: `https://login.microsoftonline.com/${tenant}/oauth2/v2.0/token`,
+            authorizationURL: `https://login.microsoftonline.us/${tenant}/oauth2/v2.0/authorize`,
+            tokenURL: `https://login.microsoftonline.us/${tenant}/oauth2/v2.0/token`,
         }, verify);
         this.name = "microsoft";
+       this.userInfoURL = "https://graph.microsoft.us/oidc/userinfo";

This issue body was partially generated by patch-package.

Hi @jacobcritch 👋  Thank you for bringing this issue to my attention. It seems like this could be easily fixed by adding a parameter you can use to define the Azure instance (.com or .us).
I will update this issue when I've come up with a solution.


Extend extensions

Hi,
I want to update firebase-email-send function
I've cloned the source of extension repository and try to update my extenstion with
firebase ext:update
but it always updates from original source - this repository.
I didn't find any details instructions on how to extend and update this extensions.
Hope you can help

Hey @architechcro, if you're not sure of the Firebase CLI commands, you can run this command firebase --help which lists all the commands with a description. To update your extension with your local changes you need to run the following command:
firebase ext:update <extension Id> <path to firestore-send-email directory with your changes> --project=<firebase project Id> .
Every angled bracket <> needs to be replaced with the actual value. Hope this helps.

@architechcro that feature is still in preview. In order to enable it on the CLI you'll need to run firebase --open-sesame extdev before running firebase ext:update as @russellwheatley explained.

Hi, developing / forking extensions is something we're experimenting with in a closed testing program. If you're interested in learning more, please sign up for the Firebase Alpha program.

Are there any updates on whether this capability would be enabled? I'm currently using the "Manage Marketing with Mailchimp" and would like to make some changes in its behavior.


AssumeRoleWithWebIdentity issues cross partition

What happened?
When trying to create a resource in a different partition with AssumeRoleWithWebIdentity the following error occurs:
"error": "failed to create the Instance resource: InvalidIdentityToken: No OpenIDConnect provider found in your account for https://oidc.eks.us-east-1.amazonaws.com/id/${OIDC_ID}"

How can we reproduce it?
There is an EKS cluster in the global partition
EKS cluster in another partition such as aws-cn
Create OIDC IDP in aws-cn which trusts the global clusters OIDC
Try to create any resource with:
spec:
  assumeRoleWithWebIdentity:
    roleARN: "arn:aws:iam::${TARGET_AWS_ACCOUNT_ID}:role/${IAM_ROLE_NAME}"

What environment did it happen in?
Crossplane version:
index.docker.io/crossplanecontrib/provider-aws:v0.37.1

@nabuskey

@wattie-canva
I've pushed a container image with a fix for this here: public.ecr.aws/m8u6z8z4/manabu-test:web-token-partition-v1
Please test it out and see if it fixes the issue for you. Source code is available here: https://github.com/nabuskey/provider-aws/tree/bugfix/web-token-partition
You should be able to specify the image in your controller config.

Hey @nabuskey I updated the image version but the provider doesn't come up healthy.
Do you know what might cause this or how to fix it?
# k describe providers

Events:
  Type     Reason                  Age                    From                                 Message
  ----     ------                  ----                   ----                                 -------
  Warning  InstallPackageRevision  11m (x7 over 12m)      packages/provider.pkg.crossplane.io  current package revision health is unknown
  Warning  InstallPackageRevision  11m (x7 over 12m)      packages/provider.pkg.crossplane.io  cannot apply package revision: cannot create object: the server could not find the requested resource (post providerrevisions.pkg.crossplane.io)
  Warning  InstallPackageRevision  9m8s                   packages/provider.pkg.crossplane.io  cannot apply package revision: cannot patch object: Operation cannot be fulfilled on providerrevisions.pkg.crossplane.io "provider-aws-f619b9b5a5d9": the object has been modified; please apply your changes to the latest version and try again
  Warning  InstallPackageRevision  9m7s (x4 over 9m8s)    packages/provider.pkg.crossplane.io  current package revision health is unknown
  Warning  InstallPackageRevision  8m58s (x2 over 8m58s)  packages/provider.pkg.crossplane.io  current package revision is unhealthy

I'm using crossplane v1.11.1-stable and crossplane-provider-aws public.ecr.aws/m8u6z8z4/manabu-test:web-token-partition-v1
Do you know what  might cause this or how to fix it?

Can you get events from the provider revision object? Also can you post your controller config?

Providerrevision events:
Events:
  Type     Reason             Age                     From                                         Message
  ----     ------             ----                    ----                                         -------
  Normal   ApplyClusterRoles  33m (x4 over 98m)       rbac/providerrevision.pkg.crossplane.io      Applied RBAC ClusterRoles
  Warning  ParsePackage       4m18s (x1029 over 20h)  packages/providerrevision.pkg.crossplane.io  cannot initialize parser backend: failed to open package stream file: EOF

Controllerconfig with some specifics such as IRSA removed:
Happy to provide full dump privately if that helps.
kind: ControllerConfig
metadata:
  annotations:
    eks.amazonaws.com/role-arn: 'arn:aws:iam::<ID>:role/...crossplane-provider-aws'
  creationTimestamp: '2023-06-25T23:35:04Z'
  generation: 1
  labels:
    app.kubernetes.io/managed-by: argocd
    app.kubernetes.io/name: crossplane-provider-aws
    app.kubernetes.io/version: 1.0.0
    argocd.argoproj.io/instance: crossplane-provider-aws.cluster-name
  name: provider-aws-config
spec:
  affinity:
    podAntiAffinity:
      preferredDuringSchedulingIgnoredDuringExecution:
        - podAffinityTerm:
            labelSelector:
              matchLabels:
                pkg.crossplane.io/provider: provider-aws
            topologyKey: topology.kubernetes.io/zone
          weight: 50
        - podAffinityTerm:
            labelSelector:
              matchLabels:
                pkg.crossplane.io/provider: provider-aws
            topologyKey: kubernetes.io/hostname
          weight: 50
  args:
    - '--debug'
  env:
    - name: LEADER_ELECTION
      value: 'true'
  replicas: 2

The image I linked is for the controller only so you need to specify it in the spec.image field. e.g.
kind: ControllerConfig
metadata:
  name: provider-aws-config
spec:
  image: public.ecr.aws/m8u6z8z4/manabu-test:web-token-partition-v1

Got it running with that image now but the pods provider-aws- crashloopbackoff with the logs
exec /usr/local/bin/crossplane-aws-provider: exec format error


PLAT-807: Enable CSLS logs

Proposed changes
Uncommenting the CSLS to enable cloudwatch logs following the below PR merge:

https://github.com/alphagov/centralised-security-logging-service/pull/960

Issue tracking
PLAT-807: Update CSLS to take passport new accounts

Changes covered by PR https://github.com/govuk-one-login/ipv-cri-uk-passport-api/pull/54


Ignore version and filename when compute hash for the pseudo node

A typical UpfData node has the following objects used to generate the hash:
['1.4.0',
 {'md5': 'e96aa7e3e4a958db16958554ea960e2a',
  'element': 'H',
  'filename': 'H.nc.z_1.oncvpsp3.dojo.v0.5.0-std.upf',
  'z_valence': 1},
 '2ad1f9704b3b12e280562806303ce715cbd9f4735c24b047199cbbcd47e168e8',
 None]

I think the filename and the version of the plugin are relevant to the calculation result thus need to add to _hash_ignored_attributes. Your opinion @sphuber ?

The policy with caching has always been to err on the safe side. This is why the plugin version is included. It is unlikely that in general this will ever have an effect (since anyway all attributes and repo contents are included) but we decided to keep it in.
But I think I have to agree, that really all we care about for a UpfData node is the hash of the file. So really we could even think of overriding the _get_objects_to_hash and just return the md5. I am thinking hard whether there are edge cases where this could go wrong, but I am coming up blank. @mbercx do you see a potential problem with this?

I am thinking hard whether there are edge cases where this could go wrong, but I am coming up blank. @mbercx do you see a potential problem with this?

I'm coming up blank as well. And having the plugin version used for the hash is kind of tedious, breaking caching when it shouldn't. So fine for me to change this!

I was thinking of adding to PseudoPotentialData, is that too aggressive?
For safety, just add it to UpfData? If we agree the plugin version is too tedious here, maybe it can added to PseudoPotentialData and ignore the filename for UpfData.
I can make a PR for it.

I think the argument extends to all subclasses of PseudoPotentialData. There is a reason that it subclasses SinglefileData because (in principle) the file content should uniquely determine the pseudo. The metadata that we add in the attributes are just useful shortcuts to facilitate querying and proxy some common properties. They all come from the file, so their content is redundant.
I think that if we change this, we better change it for all to be consistent, and so put the change on PseudoPotentialData. And as mentioned before, instead of overriding _hash_ignored_attributes, I think we should simply override _get_objects_to_hash straight away and just have it return the hash of the file.

Besides the version of the plugin, I find a more worried issue that the calcjob node hash is changed if the aiida-core is updated. I think the version of the calcjob node should be ignored.
Here is a output of _get_objects_to_hash from a pw calculation. The hash is based on the aiida-core version that runs the calculation, which means everytime the aiida-core updated the rehash is needed, which is net expected I assume if look at the operation needed in https://github.com/aiidateam/aiida-core/blob/main/CHANGELOG.md#v240---2023-06-22
['2.4.0',
 {'version': {'core': '2.4.0.post0', 'plugin': '4.4.0'},
  'withmpi': True,
  'resources': {'num_machines': 1, 'num_mpiprocs_per_machine': 36},
  'append_text': '',
  'parser_name': 'quantumespresso.pw',
  'prepend_text': '',
  'input_filename': 'aiida.in',
...

Besides the version of the plugin, I find a more worried issue that the calcjob node hash is changed if the aiida-core is updated. I think the version of the calcjob node should be ignored.

You are right, the first element is the version of the package that defines the module of the node type, e.g., CalcJobNode and WorkChainNode. Since these can not be changed through plugins, this is always the version of aiida-core. It is a bit redundant actually, because it is anyway included in the version attribute. The reason these versions are included goes again back to what I mentioned before: when we implmented caching, we implemented it to be as conservative as possible. It looks like instead of monkeypatching things here in aiida-pseudo we should open a discussion with the aiida-core team to see if we can remove aiida-core and plugins versions from the hashes.

The hash is based on the aiida-core version that runs the calculation, which means everytime the aiida-core updated the rehash is needed, which is not expected I assume, if look at the operation needed in https://github.com/aiidateam/aiida-core/blob/main/CHANGELOG.md#v240---2023-06-22

Us forcing a database migration to drop the hashes in that case was different I would argue. There was a real bug in the calculation of the hash, which we fixed, but this would therefore invalidate the hash. The reason is not so much to make sure identical nodes are still matched, but rather that different nodes are not accidentally matched because the original hashes are incorrect. I guess that probability is really low.


Ability to limit show-errors-check-validity events to a specific form.

For when you have multiple forms and/or nested forms and you either prefer or are unable to give each form its own controller.

Can we get this merged please?

+1

Will someone please merge this?

+1

+1

+1

+1


Use GitHub Actions for linting and testing

We don't run the Complement Buildkite pipelines for external pull requests due to security concerns around passing the Docker socket through to the container.
...there's no such concern on GitHub's public build infrastructure :)

Note how the linting and synapse checks run in less than half the time of the same steps on Buildkite :)

I accidentally merged this to master here while trying to test out CI on a different origin. My apologies ☹ I'll finish reviewing and get this merged properly in just a sec.

Making the executive decision to merge this: it works, and it allows external contributors to see if their patches pass CI, which is really important with putting Complement forward as a potential Outreachy project

Buildkite is failing for unrelated reasons; having connectivity issues to the Docker registery (docker: Error response from daemon: Get https://registry-1.docker.io/v2/: net/http: request canceled (Client.Timeout exceeded while awaiting headers).)

I'm happy with this - but why bother using Buildkite at all at this point then?

There are open questions about the suitability / scalability of GitHub Actions for broad use at Element. A few teams are experimenting with it and we should have a more solid recommendation by the end of the month.
The most critical concern right now is that we're limited to 20 concurrent jobs until we upgrade the matrix-org organization to GitHub's premium tier for non-profit organizations.

Okay, well if we do upgrade let me know and I'll deprecate Buildkite for Complement in particular, given the security hole it needs in order to operate correctly.

Will do! If we decide to stay with Buildkite, we could patch that security hole by running the host container under sysbox, which would allow it to have its own, contained Docker daemon.


[MuchoHentai] Add extractor

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

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

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

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

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
Explanation of your pull request in arbitrary form goes here. Please make sure the description explains the purpose and effect of your pull request and is worded well enough to be understood. Provide as much context and examples as possible.
Pull Request Contents
Add support for the domain https://muchohentai.com/
Brief info:

Site uses jw-player.
Videos are protected by Referrer URL.
Video url is obfuscated in inline-js and reassembled on client side.
Video url points towards m3u8.
m3u8 is VOD type.
Although there seems to be an authorize endpoint, it is apparently not necessary to download the videos.
Sporadically, the site presents direct download that is protected by recaptcha. This feature is not implemented in this extractor.

One issue that was found during testing.
This Video has actually different formats and fails.
I'll add a patch for that, once I found the cause.

From initial impression, it looks like a piracy site. You sure it is not?

Actually it is distribution of Fansub content.
https://en.wikipedia.org/wiki/Fansub

Fansubs are piracy! (not the subs themselves, but they are distributed with pirated video)

Fansubs are piracy! (not the subs themselves, but they are distributed with pirated video)

Not necessarily. It depends on the source of the raw video. If it aired via open channels, it is obviously not pirated.
It is the same issue with all platforms that distribute content.
I maintain a professional attitude in this matter because I was only interested in the technical part of this endeavor.
To conclude the professional matter, the developer instruction/contributing guideline does not include any section about "piracy/illegal" content. I would not have spend the effort to contribute to this fork if that were the case.

the developer instruction/contributing guideline does not include any section about "piracy/illegal" content. I would not have spend the effort to contribute to this fork if that were the case.

We follow the same guidelines with regards to piracy as youtube-dl https://github.com/ytdl-org/youtube-dl#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free. You are right in that I should have linked/copied the same to our contributing guidelines. Will do so now

If it aired via open channels, it is obviously not pirated.

Airing through open channels and having the rights to re-distribute are different things

It is the same issue with all platforms that distribute content.

The major difference is whether the site is "primarily" distributing pirated content. I agree that is not a very strict definition and sometimes has to be interpreted in a case-to-case basis. If the site follows proper DMCA procedure for it's content, I would be willing to at least consider it. If you think this site satisfies that criteria, feel free to argue for it

feel free to argue for it

As I wrote, to me this was a programming exercise for a weekend and I closed the PR.
Please remove that piracy/illegal tag.
I don't want my name mentioned next to that. Thanks.

Airing through open channels and having the rights to re-distribute are different things

That goes for the US.
TL:DR
In Germany you can freely re-distribute stuff that was recorded from TV for example.
There is also a right for private copy which is the reason why content providers are afraid of Germany and have to invent new copy protection mechanisms every time one is broken.
The system is that crooked, that you had to pay a fine with each legal purchased DVD device, because it's potentially used for illegal purposes. Piracy is de facto priced in. Of course nobody told you so, that would have caused bad vibes for the consumer.
It should be obvious who the criminals are in this context.

Please update this extractor.


Add yaml CI validation for metadata.yml (and probably all yaml in the repo)

This would allow some basic protection against invalid YAML from landing in the project.
/kind feature

I can help with that
/assign


Improve styling at login page

Now login and signup pages are working perfectly but have to improve the CSS to this page so it is looking nice and attractive.

Hey @Sujal-Sahu can you assign this task?

Hey @Sujal-Sahu can you assign this task?

yes @srijoy-paul go for it.

Hey @srijoy-paul  are you working on this?

Hey @Sujal-Sahu I am working on it today, was busy with other tasks

Hey! @Sujal-Sahu assign this task

Hey @srijoy-paul due to time constraint and not PR from your side, i assign this task to @nik25s . so @nik25s please work on it and complete it as soon as possible.


Issues when building polybar on Fedora 30

Build Process
I download the .tar file from the releases and extract it.
Then follow the same instructions given in the README, that is:
$ mkdir build
$ cd build
$ cmake ..
$ make -j$(nproc)
$ sudo make install

List exactly where you are building from and which version you are building.
Building from this repository and Version 3.3.1.
Build Log
[nanda@nandapad build]# cmake ..
-- Trying to enable ccache
-- Couldn't locate ccache, disabling ccache...
-- No build type specified; using RelWithDebInfo
-- Using supported compiler GNU-9.1.1
--  Build:
--    Type: RelWithDebInfo
--    CC: /usr/bin/cc  -O2 -g -DNDEBUG
--    CXX: /usr/bin/c++  -Wall -Wextra -Werror -Wno-noexcept-type -pedantic -pedantic-errors -O2 -g -DNDEBUG
--    LD: /usr/bin/ld
--  Targets:
-- [X]   polybar-msg
-- [ ]   testsuite
--  Module support:
-- [X]   alsa (1.1.9)
-- [ ]   curl
-- [ ]   i3
-- [ ]   mpd
-- [ ]   network (wireless-tools)
-- [ ]   pulseaudio
-- [X]   xkeyboard (1.13.1)
--  X extensions:
-- [X]   xcb-randr (1.13.1)
-- [X]   xcb-randr (monitor support) (1.13.1)
-- [ ]   xcb-render
-- [ ]   xcb-damage
-- [ ]   xcb-sync
-- [X]   xcb-composite (1.13.1)
-- [X]   xcb-xkb (1.13.1)
-- [ ]   xcb-xrm
-- [ ]   xcb-cursor
--  Log options:
-- [X]   Trace logging
-- Font not found: fixed:pixelsize=10
-- Font not found: unifont:fontformat=truetype
-- Font not found: siji:pixelsize=10
-- XCB[XCB]: Found component XCB
-- XCB[ICCCM]: Found component ICCCM
-- XCB[EWMH]: Found component EWMH
-- XCB[UTIL]: Found component UTIL
-- XCB[IMAGE]: Found component IMAGE
-- Found XCB_IMAGE: /usr/lib64/libxcb-image.so
-- Found XCB: /usr/lib64/libxcb.so;/usr/lib64/libxcb-icccm.so;/usr/lib64/libxcb-ewmh.so;/usr/lib64/libxcb-util.so;/usr/lib64/libxcb-image.so
-- Searching for xcbgen with python2
-- Searching for xcbgen with python2.7
-- Searching for xcbgen with python3
-- Found xcbgen in /usr/lib/python3.7/site-packages
-- XCB[RANDR]: Found component RANDR
-- Found XCB_RANDR: /usr/lib64/libxcb-randr.so
-- Found XCB: /usr/lib64/libxcb.so;/usr/lib64/libxcb-icccm.so;/usr/lib64/libxcb-ewmh.so;/usr/lib64/libxcb-util.so;/usr/lib64/libxcb-image.so;/usr/lib64/libxcb-randr.so
-- XCB[COMPOSITE]: Found component COMPOSITE
-- Found XCB_COMPOSITE: /usr/lib64/libxcb-composite.so
-- Found XCB: /usr/lib64/libxcb.so;/usr/lib64/libxcb-icccm.so;/usr/lib64/libxcb-ewmh.so;/usr/lib64/libxcb-util.so;/usr/lib64/libxcb-image.so;/usr/lib64/libxcb-randr.so;/usr/lib64/libxcb-composite.so
-- XCB[XKB]: Found component XKB
-- Found XCB_XKB: /usr/lib64/libxcb-xkb.so
-- Found XCB: /usr/lib64/libxcb.so;/usr/lib64/libxcb-icccm.so;/usr/lib64/libxcb-ewmh.so;/usr/lib64/libxcb-util.so;/usr/lib64/libxcb-image.so;/usr/lib64/libxcb-randr.so;/usr/lib64/libxcb-composite.so;/usr/lib64/libxcb-xkb.so
-- xpp: including xcb proto composite.xml
-- xpp: including xcb proto randr.xml
-- xpp: including xcb proto xkb.xml
-- xpp: including xcb proto xproto.xml
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).


Fix Sandcastle error for non-existent labels

If you copied the URL to a Sandcastle demo from a label like "New in 1.70" that link will produce an error when opened when that label doesn't exist (like when the next version comes out). Noticed on the release blog:
https://sandcastle.cesium.com/index.html?src=Globe Translucency.html&label=New in 1.69
This doesn't prevent the example from loading but prints a console error and no examples show up in the gallery.
This PR fixes it by ignoring the label if it doesn't exist.

Thanks for the pull request @OmarShehata!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

👍 works locally


[ca] Extricate ExternalCA from SecurityConfig

This is the first step towards splitting the CA server's signing functionality from SecurityConfig, which serves more for providing creds for mTLS.  Doing so will help unify the root CA update paths in node/node.go.
The only node that ever needs the ExternalCA object is the leader that is running the CA server anyway, so it makes sense for it to go into the CA server.  This also better makes it possible at a future point to provide separate creds for contacting the external CA server if we ever needed to do so.

I've added an interface to ca.Server that will update the external CA's TLS creds when the security config's TLS creds are updated.  PTAL - will also be happy to split this up in to 2 or 3 PRs, since a lot of files have been changed.

Coming back to this after awhile. Is it ready for review?

@aaronlehmann Also, apologies, I spent a while trying to remember what thought I had regarding this PR and I no longer remember it.  :)  So yes, this is ready for review.

Overall I'm good with the refactor 👍


lost metaData when ImageRequestInfo have outputFormat

Describe the bug

When outputFormat is set, the metadata is lost. I set AutoWebPParameter = "Yes", but the metadata of the converted WebP images, especially the orientation, is missing, resulting in the photos not rotating correctly.
To Reproduce

Deploy the stack using the following command:
npx cdk deploy --parameters AutoWebPParameter=Yes
Set outputFormat to a specific value (e.g., WebP).
Upload an image that includes EXIF metadata, such as orientation.
Check the converted WebP image and observe that the metadata (e.g., orientation) is missing.

Expected behavior

The image should retain its metadata, especially orientation, after being converted to WebP format.
Please complete the following information about the solution:

[x] Version: [e.g. v6.3.1]
[x] Region: [e.g. ap-northeast-2]
[ ] Was the solution modified from the version published on this repository?
[ ] If the answer to the previous question was yes, are the changes available on GitHub?
[ ] Have you checked your service quotas for the sevices this solution uses?
[ ] Were there any errors in the CloudWatch Logs?

Screenshots

Additional context

Hi @dongwookko,
Can you please provide an examples of an incorrect image? In many cases, the exif orientation metadata continues to be set correctly, but most major browsers do not support displaying exif metadata attributes for WEBP images. I believe Safari support webp orientation. As a fix, you can include the strip_exif filter to have sharp apply the orientation for you.
Thanks,
Simon

@simonkrol
It seems that there is a problem with webp rotation (not reflecting EXIF orientation) in desktop chrome.
However, the problem with serverless-image-handler is not limited to webp.
Even if there is orientation metadata in jpeg, metadata.orientation cannot be maintained because metadata is lost due to toBuffer().

ref. https://sharp.pixelplumbing.com/api-output#tobuffer
By default all metadata will be removed, which includes EXIF-based orientation. See withMetadata for control over this.

In my opinion, if there is metadata.orientation, the information should be maintained with withMetadata.
Manual rotation of webp seems to be the next problem.

Hi @YH-Seo,
I've done a little bit of testing with your provided image, and I haven't been able to reproduce the issue you're running into. Can you provide some example paths (or json objects if you're using base-64 encoded requests)?
Using a distribution with AutoWebP enabled:
I've seen the orientation come back as expected when running:
/filters:format(jpeg)/IMAGE_NAME.jpeg
Tthe image displays improperly on Chrome if requested like:
/IMAGE_NAME.jpeg though this is an issue with Chrome, and with the image being converted to WebP as mentioned above.  If you download the image, or view it through Safari, it displays properly.
Thanks,
Simon

Hi @simonkrol 😀
Sorry 😢
I realized that my test was wrong. When I deleted AutoWebP Parameter in CloudFormation and deployed it, the image did not rotate, so I thought the problem was not related to webp.
In conclusion, it was my mistake, and I had to explicitly set AutoWebP Parameter to No.
There is no problem in the jpeg case. The reason is that when AutoWebP=No, outputFormat is undefined, so toBuffer() is not performed.
According to the document below (REF#1), we added logic to read metadata.orientation in the outputFormat===WEBP case and perform rotate() if there is an orientation value.

REF#1 https://docs.aws.amazon.com/solutions/latest/serverless-image-handler/rotate-images-manually.html

In conclusion, the code we modified is as follows.

Hi @YH-Seo,
Sounds good, you may have some luck with the filters:strip_exif() or including a rotate: null edit on your requests, this will cause Sharp to remove any orientation value applied to the image, rotating it to match the expe ted output.
Simon

Closing, feel free to re-open if you have any more questions :)


Support multiple files in single action

If I have a single dependency configured in multiple files (i.e. a static site built using Hugo & deployed using Netlify) each need to be specified in separate steps which result in a longer build time. It would be awesome if the file input were optional and an options files list input added, so the process could loop on all files needed.

PR is welcome.

Maybe it's not an ideal solution, but have you tried to use GitHub Action matrix feature?


Add consuming metadata to broker response

Fix #7144
Added numConsumingSegmentsProcessed, numConsumingSegmentsMatched  to broker response metaData
I am assuming,
numConsumingSegmentsProcessed to be a sub set of numSegmentsProcessed
and numConsumingSegmentsMatched to be a sub set of numSegmentsMatched

Codecov Report

Merging #9092 (37579da) into master (989df64) will decrease coverage by 53.56%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master    #9092       +/-   ##
=============================================
- Coverage     68.88%   15.31%   -53.57%
+ Complexity     4734      170     -4564
=============================================
  Files          1835     1787       -48
  Lines         97119    95031     -2088
  Branches      14632    14392      -240
=============================================
- Hits          66897    14557    -52340
- Misses        25532    79445    +53913
+ Partials       4690     1029     -3661

Flag
Coverage Δ

integration1
?

unittests1
?

unittests2
15.31% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

 Take ten seconds to tell us how you rate us.

cc: @snleee

@ravishankar15 I don't see that your code change is related to the failure in the compatibility test (it fails on segment upload). Can you try to rebase the code with the upstream master? Let's see if it fixes the issue.

It looks that the compatibility test will start to pass after we forcefully merge this PR because the existing job would use the old test set.
@ravishankar15 Thanks you for the contribution!

@snleee The error says something like this 2022/07/27 04:02:38.479 ERROR [SegmentOp] [main] Upload segment verification failed, routing table has not been updated after max wait time 60000 ms. It is not related to mismatch I hope can you shed some light here for my understanding. Thanks

@ravishankar15 Yeah I also saw that the compatibility test is failing on the routing table update. I will take a look on this and see if we can resolve it without rolling back this PR.

Created #9119 to revert this commit for now. We can open a new PR to check it in after fixing the compatible issue. IIRC, the compatibility test won't compare the metadata

   * ATTENTION:
   *  - Don't change existing keys.
   *  - Don't remove existing keys.
   *  - Always add new keys to the end.
   *  Otherwise, backward compatibility will be broken.

@ravishankar15 Can you re-file the PR by adding the metadata key to the end?

   * ATTENTION:
   *  - Don't change existing keys.
   *  - Don't remove existing keys.
   *  - Always add new keys to the end.
   *  Otherwise, backward compatibility will be broken.

@ravishankar15 Can you re-file the PR by adding the metadata key to the end?

Yeah sure @snleee I will raise a separate PR with all the changes


Optimize JWT content

Currently the JWT generated during authentication contains the list of the user's groups.
This list can grow quite big depending on the authentication mecanism (gitlab, LDAP, ...)
{
  "sub": "julien.chanaud",
  "nbf":<PHONE_NUMBER>,
  "roles": [
    "isAdmin()"
  ],
  "iss": "ns4kafka",
  "groups": [
    "project/group1",
    "project/group2",
    "project/group3",
    "project/group4"
  ],
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>
}

It should be replaced with the role-bindings directly
{
  "sub": "julien.chanaud",
  "nbf":<PHONE_NUMBER>,
  "roles": [
    "isAdmin()"
  ],
  "iss": "ns4kafka",
  "role-bindings": [
    {
      "namespace": "ns1",
      "verbs": ["GET", "POST", "..."],
      "resources": ["topics", "schemas", "..."]
    },
    {
      "namespace": "ns2",
      "verbs": ["GET", "POST", "..."],
      "resources": ["topics", "schemas", "..."]
    },
  ],
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>
}

We currently have the problem : our gitlab server contains many groups, and returned JWT tokens exceed max allowed size, thus micronaut fails to handle it.
In the other hand, role-bindings may change after an API call to apply new ones, thus token should be invalidated after such API calls.
Another idea may be to add the ability to limit groups to scan to a subgroup list specified in a micronaut.security.gitlab.groups section ?

See #362 to limit groups to parent a one given group.
@twobeeb : does this workaround solve your issue ?

@piif
Did you try to increase the header max size on Micronaut as a quick fix ?
micronaut:
  server:
    netty:
      max-header-size: 16384

https://micronaut-projects.github.io/micronaut-docs-mn2/2.0.3/guide/configurationreference.html#io.micronaut.http.server.netty.configuration.NettyHttpServerConfiguration
A new JWT needs to be generated to take new role bindings/groups in consideration. You can also lower JWT expiration on Micronaut: https://micronaut-projects.github.io/micronaut-security/latest/guide/#jwtGenerator

Hi @loicgreffier ,
With a admin account, I had to grow this limit to 32k to make ns4kafka accept the token !
In the other hand, filtering on a specific group give me a token less than 500 bytes.
Thus, yes, it works, but it's a bit huge, and the filtering option is very efficient in some cases.

@piif I'm going to consider implementing this option which suits us the best as well

@piif JWT optimization has been implemented in v1.11.10


Suggest: Another option for the Kata "Progress" drop-down to find abandoned Kata

I would like to be able to select "Kata I have trained on but not completed"
For example, I have lots of Kata I started training on but for whatever reason abandoned. I'd like to know what they were so I can revisit them.
Unfortunately (but probably typically) I was too negligent to bookmark them and too disorganized to put them into collections

I said this on Gitter, too, but if you go to your profile page and select the Kata tab and then Unfinished, this bills itself as "View all kata that you have started but have not yet completed" -- on one hand I think the list it shows me, at least, should probably be longer than it is, but on the other hand it's not like I've actually been keeping track of the number, so I may just have an inflated view of things.

And my input from gitter:
Here's your URL https://www.codewars.com/users/dinglemouse/unfinished

It would still be more convenient from the Kata page but this does exactly what I wanted, so I am happy for this issue to be closed...


Add the Common Workflow Language standard

Hello!
This PR adds support for the CWL open standards for portable data analysis tool and workflow descriptions
http://www.commonwl.org/
There are over 4,800 CWL files on GitHub.
Thanks :-)

@mr-c There are also a lot of TeX files under .cwl. Do you know what those are?

@pchaigno I'm not a TeX expert, sorry. Some light searching shows they are associated with the TeXstudio IDE?
All Common Workflow Language files contain cwlVersion:  followed by the version of the spec on a line someplace in the file, so it shouldn't be hard to disambiguate.

There are around 1k .cwl TeX files so it probably mandates support. Since the two languages contain very different keywords, I'd expect the Bayesian classifier to do a pretty good job here. @kivikakk What do you think?

Strongly agree re: the Bayesian classifier! If we add a definition fro the TeX .cwls and collect a few examples, it should be fine.


Streamlit integration (local)

🚀 Feature
We need a quick demo on how to use streamlit with mantisshrimp. The first step in to get it running locally.
We should also add helpers functions to make the process easier.

I'm running late because of STOC, will push code asap.

Don't worry!
I created the issue just to make it more organized

Should help #99 and #83
Something to get strated
https://github.com/streamlit/demo-self-driving

Minimalistic (non-inference code) is working fine locally. We need to add model and run it. Repo is here.
You can simply give me a s3 bucket link/dropbox file link. I have made a progress bar loader too for download.

I'll try this tomorrow!!

Let's try to use Google Drive as a first attempt and take it from there: The notebooks + datasets will be hosted at the same place avoiding to deal with 2 separate providers.

I'm thinking the datasets are not going to be hosted by us, instead we should use the original source whenever posssible

Agree! I actually referring more the the weights files.

Almost there with this  repo. We just need to use the predict function of mantisshrimp. And it should display the results to the user. The other NMS and searching via the class needs a bit of work. Please try to run this repo and let me know the changes.
You can even raise a PR if something is not working or you have an improvement and update.

I ran it yesterday and it worked perfectly (apart from what we already discussed) 😄
Please let us know again when you add prediction. For now, don't bother trying to fix the deployment issues, let's first get this going locally and then we figure out

I fixed that small thing. Prediction will be done hopefully this week. Then we can see about the sidebar thingie of how much customization we can give to user.

Almost done. A bit of final touch ups are left. I guess we can add this demo to read me too.

Can you guys please test this repo?
pip install -r requirements.txt
streamlit run src/object_detection_app.py

It works!

Prediction is wrong, but I think that's caused by the low image resolution

It works!

Great. Please provide me some images from Pets dataset itself so that we can show it works well.
I will polish up the app once more and then we can close this issue..

Please provide me some images from Pets dataset itself so that we can show it works well.

It's a better idea just to google some images, I used a random seed and I don't know what was in train/valid

Yep, Pras helped me to fix it up. Now the predictions look better. We can improve the app further and refine for the sidebar UX. But I will leave this as scope for future improvement for now.
I am updating the readme to have link to this repo. We can close the issue with this.

I Guess we can close this.


incompatible issue between ExplicitLac and message compression

Describe the bug
When I produce some compressed messages to Pulsar, and use ExplicitLac, get an error from the presto server.
To Reproduce
Steps to reproduce the behavior:
I tried to  fix the issue #7076, so:

wrote code to produce messages to pulsar.
producer = pulsarClient.newProducer(JSONSchema.of(Record.class))
 .topic(topic)
 .producerName(NAME_CONST.NAME_PRODUCER_ACCOUNT_NORMAL)
.compressionType(CompressionType.ZSTD)
.create();
I changed some config files, one item is:
in conf/presto/catalog/pulsar.properties
pulsar.bookkeeper-use-v2-protocol=false
pulsar.bookkeeper-explicit-interval=10
I got this error in the presto server when I query from the presto client.

you can see:
io.airlift.compress.zstd.ZstdDecompressor.decompress(ZstdDecompressor.java:60)
4. I removed a line code of my message producer.
//.compressionType(CompressionType.ZSTD)
it works well.
Expected behavior
I guess there is an incompatible issue between ExplicitLac and message compression
Screenshots

Desktop (please complete the following information):

OS: macos Bigsur(Pulsar 2.7.0), CentOS (Pulsar 2.7.0)

Additional context
@codelipenghui

Hi @pointearth This issue is fixed by #8990 and it is cherry-picked to branch-2.7. You can build from branch-2.7 or you can use https://github.com/streamnative/pulsar/releases/tag/v<IP_ADDRESS> directly, we have a weekly release for the branch-2.7.
But I found another problem #9666, @gaoran10 is working on this one. Since 2.7.1 will be released soon, so we need to get this fix merged soon.


1.13.1 Compatibility Fixes

Note: This is backwards breaking, you cannot run this on a server < 1.13.1

Moving to Tebex-MCJava repo


Add a license to the repository

I recently came across your project QGSS2024 and I noticed that the repository github.com/webb-c/QGSS2024 currently does not have a license specified.
Why Adding a License is Important?
As GitHub documents

Public repositories on GitHub are often used to share open source software. For your repository to truly be open source, you'll need to license it so that others are free to use, change, and distribute the software.

Would be possible to add a valid license to this repository? Probably something really OpenSource such as the Apache 2.0 license would fit really well.
Thanks!

Thanks


More accumulated app string updates

PR Checklist

Please check all that apply to this PR using "x":

[x] I have checked that this PR is not a duplicate of an existing PR (open, closed or merged)
[x] I have checked that this PR does not introduce a breaking change
[ ] This PR introduces breaking changes and I have provided a detailed explanation below

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation changes
[x] Other - Please describe: New strings

Fixes
Issue Number:
What is the current behavior?
What is the new behavior?
Other information

@eatdostacos Can you please rebase this PR? Then it can be merged.

Rebased Version created


future media image to be added

Image editing:
rotate image
crop image

Currently looking to adapt existing solution to our needs https://github.com/dsantiago89/WakCropImages
I have to add turn 90°left/right, turn 180°..what else?


UNKNOWN STATE error while testing v1.3

While testing v1.3 (upgrade from v1.2 running ok), the setup process stops and sends the following message:
UNKOWN STATE - there is blockain data config, but blockchain service is not running
It seems that something went wrong during sync/download/copy of the blockchain.
Or something with the config is not correct.
Sometimes a reboot helps --> sudo shutdown -r now

Tried rebooting but no luck.
Full message details below.
***************************************************************
* RASPIBLITZ DEBUG LOGS
***************************************************************
blitzversion: 1.3
chainnetwork: bitcoin / main
 17:17:47 up 13 min,  3 users,  load average: 2.50, 1.84, 0.97

*** BLOCKCHAIN SYSTEMD STATUS ***
Unit bitcoind.service could not be found.

*** LAST BLOCKCHAIN ERROR LOGS ***
sudo journalctl -u bitcoind -b --no-pager -n8
-- Logs begin at Thu 2019-08-29 17:04:00 BST, end at Thu 2019-08-29 17:17:47 BST. --
-- No entries --

*** LAST BLOCKCHAIN 20 INFO LOGS ***
sudo tail -n 20 /mnt/hdd/bitcoin/debug.log
2019-08-29T16:16:08Z dnsseed thread start
2019-08-29T16:16:08Z opencon thread start
2019-08-29T16:16:08Z addcon thread start
2019-08-29T16:16:08Z msghand thread start
2019-08-29T16:16:08Z init message: Done loading
2019-08-29T16:16:08Z Cannot create socket for fno4aakpl6sg6y47.onion:8333: unsupported network
2019-08-29T16:16:08Z Cannot create socket for toguvy5upyuctudx.onion:8333: unsupported network
2019-08-29T16:16:09Z Cannot create socket for ndndword5lpb7eex.onion:8333: unsupported network
2019-08-29T16:16:09Z Cannot create socket for 6m2iqgnqjxh7ulyk.onion:8333: unsupported network
2019-08-29T16:16:10Z Cannot create socket for 5tuxetn7tar3q5kp.onion:8333: unsupported network
2019-08-29T16:16:19Z Loading addresses from DNS seeds (could take a while)
2019-08-29T16:16:23Z 280 addresses found from DNS seeds
2019-08-29T16:16:23Z dnsseed thread exit
2019-08-29T16:17:05Z UpdateTip: new best=00000000000000000016810e9c7a0f404a2f4f1df959347c49f6eb472afd93e8 height=591886 version=0x20c00000 log2_work=91.009246 tx=448909992 date='2019-08-26T21:03:06Z' progress=0.998017 cache=1.4MiB(12316txo)
2019-08-29T16:17:10Z Cannot create socket for fno4aakpl6sg6y47.onion:8333: unsupported network
2019-08-29T16:17:11Z Cannot create socket for toguvy5upyuctudx.onion:8333: unsupported network
2019-08-29T16:17:11Z Cannot create socket for ndndword5lpb7eex.onion:8333: unsupported network
2019-08-29T16:17:12Z Cannot create socket for 6m2iqgnqjxh7ulyk.onion:8333: unsupported network
2019-08-29T16:17:12Z Cannot create socket for 5tuxetn7tar3q5kp.onion:8333: unsupported network
2019-08-29T16:17:46Z ThreadRPCServer incorrect password attempt from <IP_ADDRESS>:58622

*** LND SYSTEMD STATUS ***
Unit lnd.service could not be found.

*** LAST LND ERROR LOGS ***
sudo journalctl -u lnd -b --no-pager -n12
-- Logs begin at Thu 2019-08-29 17:04:00 BST, end at Thu 2019-08-29 17:17:48 BST. --
-- No entries --

*** LAST 30 LND INFO LOGS ***
sudo tail -n 30 /mnt/hdd/lnd/logs/bitcoin/mainnet/lnd.log
tail: cannot open '/mnt/hdd/lnd/logs/bitcoin/mainnet/lnd.log' for reading: No such file or directory

*** LAST 20 RTL LOGS ***
-- Logs begin at Thu 2019-08-29 17:04:00 BST, end at Thu 2019-08-29 17:17:48 BST. --
-- No entries --

*** HARDWARE TEST RESULTS ***
UndervoltageReports in Logs: 0
raspberryPi=3
powerFAIL=0
powerWARN=0
powerMIN='1231300 microVolt'
tempFAIL=7
tempWARN=15
tempMAX='7520 centiGrad'
IMPORTANT: There are some hardware issues with your setup.
'Run Hardwaretest' in main menu or: sudo /home/admin/05hardwareTest.sh

*** SYSTEM STATUS (can take some seconds to gather) ***
localIP='<IP_ADDRESS>'
tempCelsius='65.5'
uptime=823
upsStatus='OFF'
startcountBlockchain=0
bitcoinActive=0
bitcoinErrorShort='Error found in Logs'
bitcoinErrorFull='0'
# OK: log 'ERROR: Error found in Logs' written to /home/admin/systemd.blockchain.log
startcountLightning=0
lndActive=0
scriptRuntime=0

*** OPTION: SHARE THIS DEBUG OUTPUT ***
An easy way to share this debug output on GitHub or on a support chat
use the following command and share the resulting link:
/home/admin/XXdebugLogs.sh | nc termbin.com 9999

UNKOWN STATE - there is blockain data config, but blockchain service is not running
It seems that something went wrong during sync/download/copy of the blockchain.
Or something with the config is not correct.
Sometimes a reboot helps --> sudo shutdown -r now

Running  /home/admin/XXdebugLogs.sh | nc termbin.com 9999 returns:
Unit bitcoind.service could not be found.
Unit lnd.service could not be found.
tail: cannot open '/mnt/hdd/lnd/logs/bitcoin/mainnet/lnd.log' for reading: No such file or directory
https://termbin.com/md4j

Hi. Is that a RPi3? As I understand you have used the HDD of a previously wunning setup and just wrote  anew SD.
It is either that the data on the HDD is not complete/corrupted or the restore process did not go through properly.
If you have changed something in your hardware config there could be a power supply issue.
If there is bitcoin and lnd data on the HDD the setup process should restore the previous lnd wallet only making you to change the PASSWORD_A on the first login.
If you think there is no hardware error/disk corruption could try again with a freshly burnt SD card to trigger the restore process again.

It is a RPi3.
Update: I've downloaded v1.2 again and it worked. Then I tried the 1.3 and ended up with the same error as before.
For some reason the upgrade is not completing.
Will try a couple of things tomorrow and report back but for now I've downgraded to 1.2.

I have the same problem

Ultimately, the only thing that worked for me was to restore LND:
./98repairMenu.sh

MAKE SURE to have your seed (passphrase if used) and channel backups saved!

Pls close this. The cleanup must be complete.


Display name conda environment

Is it possible to add the name of the conda environment in use?
Right now, we can select the conda environment, but VSCode displays only the Python version (see below).

I would like to get something like this:

@JackCaster I think the status bar items are contributed by some extension?

Oh right. It should be the VSCode Python extension. I will post there instead.


Adds ability to update tags for SQS Queues

Brings in the the ability to update and delete tags from an SQS Queue
using the code hook points recently added to the sdkUpdate method for
SetAttributes-based APIs like SNS/SQS.
Issue aws-controllers-k8s/community#1541
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I think there may be some non-deterministic behaviour happening in the syncTags function. Investigating...

/test all


First draft of MN Interaction Diagram

bugfix: Marin did not work correctly for constant strain distribution (curvature = 0)
included first implementation for MN interaction domain

TODO:

[x]  implement NMM version
[x] implement MM version
[ ] add corresponding tests

with 3199c992cd926eedc43477386af6b8d4cfd6eb80 the full 3d NMM interaction domain is computed.
still todo:

tests implementation.
add a function for computing MM domain for given N.
Decide if we want to structure somehow the domain results.

Included a new function for computing MM interaction given axial load calculate_mm_interaction_domain.
Implemented a new function for MN interaction domain (calculate_interaction_domain_2) that is significantly faster (by almost a factor 10) respect to the first implementation. The reason of being so faster is because it avoids a lot of integrations.

I should have done everything asked.


Skip null elements while rendering ReactGridLayout's children

Please mark the type of this issue:

[x] Bug
[ ] Feature Request
[ ] Question

Console: Warning: Failed prop type: Cannot read property 'key' of null

@ibrahimduran Any workaround?

Fixed in #578, will be in next release.


GestureRecognizer throws exception

So I was running debugging to watch for error messages, and upon loading a scene I got the following error message.
Exception thrown: 'System.NullReferenceException' in UnityEngine.dll NullReferenceException: Object reference not set to an instance of an object.  at UnityEngine.VR.WSA.Input.GestureRecognizer.InvokeNavigationEvent(GestureEventType eventType, InteractionSourceKind source, Vector3 relativePosition, Ray headRay)  at UnityEngine.VR.WSA.Input.GestureRecognizer.$Invoke30(Int64 instance, Int64* args)  at UnityEngine.Internal.$MethodUtility.InvokeMethod(Int64 instance, Int64* args, IntPtr method)
I am using the camera prefab from the Holotoolkit. None of the code that handles gestures is my own, it is all default code so it is supposed to only register a tap event. The only thing that calls GestureRecognizer is the GestureManager script It seems like it has a missing delegate. Has anyone else noticed this or have this problem?

So I can debug the same thing, were you using some test scene?

I will try to come up with some repro information for you.

Please do feel free to submit a fix if you find it.

@JeffPack were you able to reproduce this?

I have not been able to give it any time as of yet. I remember this happened when clicking a UI button to load a scene. I have a hunch that due to system hang, it thought that I was trying to do a navigation gesture on the button that was then being unloaded, which may have thrown the error. I could be wrong.

Please open if you repro again with latest master.


Update Pam's Fixes.zs

We don't have MorePlanets - thus no food from that mod.

Yeah this a good catch, not sure why this was added to begin with unless lanse was testing htis mod when he build the script.

It was my script :) And I've warned Lanse that I had a different set of mods at that time...

Heh, good to know. Thanks for this though, it should help cut down on world starting chat spam.


The Composer package manager could not be found

When running rocketeer check -vvv the following is output:
| Check (Check if the server is ready to receive the application) [~5.56s]
|-- Check/Php (Checks if the server is ready to receive a PHP application)
|=> Checking presence of git
$ git --version
<EMAIL_ADDRESS>(production) git version <IP_ADDRESS>
|=> Checking presence of Composer
$ [ -e "/home/thasmo/example.com/current/composer.json" ] && echo "true"
|=> Checking PHP version
|=> Checking presence of required extensions
$ /package/host/localhost/php-5.5/bin/php -r="print defined('HHVM_VERSION') ? HHVM_VERSION : PHP_VERSION;"
$ /package/host/localhost/php-5.5/bin/php -m
<EMAIL_ADDRESS>(production) PHP Warning:  PHP Startup: Unable to load dynamic library '/home/thasmo/pecl/apc.so' - /home/thasmo/pecl/apc.so: cannot open shared object file: No such file or directory in Unknown on line 0
PHP Warning:  PHP Startup: Unable to load dynamic library '/home/thasmo/pecl/memcache.so' - /home/thasmo/pecl/memcache.so: cannot open shared object file: No such file or directory in Unknown on line 0
<EMAIL_ADDRESS>(production) [PHP Modules]
<EMAIL_ADDRESS>(production) bcmath
calendar
Core
ctype
curl
<EMAIL_ADDRESS>(production) date
dom
ereg
exif
fileinfo
filter
ftp
gd
gettext
hash
iconv
imap
intl
<EMAIL_ADDRESS>(production) json
ldap
libxml
mbstring
mcrypt
mysql
mysqli
mysqlnd
openssl
pcntl
<EMAIL_ADDRESS>(production) pcre
PDO
pdo_mysql
pdo_pgsql
pdo_sqlite
pgsql
Phar
posix
pspell
Reflection
<EMAIL_ADDRESS>(production) session
shmop
SimpleXML
soap
sockets
SPL
sqlite3
standard
sysvmsg
sysvsem
<EMAIL_ADDRESS>(production) sysvshm
tidy
tokenizer
xml
xmlreader
xmlrpc
xmlwriter
xsl
zip
zlib
<EMAIL_ADDRESS>(production) [Zend Modules]
|=> Checking presence of required drivers
The Composer package manager could not be found
The tasks queue was canceled by task "Check"
Execution time: 4.5499s
Saved logs to /cygdrive/s/Project/web/example.com/.rocketeer/logs/production--20150118.log

When logging in via SSH manually to the server composer is in the path and I can easily run it by typing composer.
Why can't rocketeer find Composer? Any idea?

I started getting this a few days ago after having not got it (and upgrading Rocketeer). Just wanted to +1, can give any info needed (but having the same issue).

If you do which composer on your server what do you get?

which composer returns ~/bin/composer.

Well that's weirder then, and in paths.php you haven't set anything that might throw Composer off?

I've tried this:
<?php

return array(
	'php' => 'php',
	'composer' => 'composer',
	'artisan' => 'artisan',
);

and this:
<?php

return array(
	'php' => '',
	'composer' => '',
	'artisan' => '',
);

Hm ok, do rocketeer tinker and then type $ssh->which('composer') see what Rocketeer's which returns? Is this Rocketeer 2.0 or 2.1-dev?

@Anahkiasen, it returns:
PHP Fatal error:  Class 'Boris\Boris' not found in phar:///usr/local/bin/rocketeer/src/Rocketeer/Console/Commands/TinkerCommand.php on line 26

Fatal error: Class 'Boris\Boris' not found in phar:///usr/local/bin/rocketeer/src/Rocketeer/Console/Commands/TinkerCommand.php on line 26

It's version 2.0.4.
Also I'm on Windows, using Cygwin and I think Boris only runs on Linux/Mac.

Also I'm on Windows, using Cygwin and I think Boris only runs on Linux/Mac.

Boris will still install on windows. Composer will just be retarded and give you and old version because the maintainer forgot to add extension requirements to it.

Sorry, I meant Rocketeer is version 2.0.4.

Boris seems to be missing in the phar file which I am using.

Seems the PHAR didn't get compiled properly by the cron, will manually regenerate them

Any update on this issue? I am getting the same errors re: Composer not found and Boris not found.
I am on Rocketeer v2.0.4

Ok I fixed the issue in the PHAR recompilation, can you try with the develop phar see if the issue still occur then?

@Anahkiasen, this happens:
> rocketeer tinker
Psy Shell v0.3.3 (PHP 5.5.19 — cli) by Justin Hileman
>>> $ssh->which('composer')
=> "composer"

> rocketeer check
If a keyphrase is required, provide it
| Check (Check if the server is ready to receive the application) [~3.84s]
|-- Check/Php (Checks if the server is ready to receive a PHP application)
|=> Checking presence of git
$ git --version
<EMAIL_ADDRESS>(production) git version <IP_ADDRESS>
|=> Checking PHP version
|=> Checking presence of Composer
The Composer package manager could not be found
The tasks queue was canceled by task "Check"
Execution time: 3.4415s
Saved logs to /cygdrive/s/Project/web/thasmo.com/.rocketeer/logs/production--20150126.log

What. In. The. Hell. Ok this is definitely not right, will try to reproduce this and see if I can come up with a fix.

I get similar results:
vagrant@deploy:~/example_development$ rocketeer tinker

Psy Shell v0.3.3 (PHP 5.5.9-1ubuntu4.5 — cli) by Justin Hileman
$ssh->which('composer')
=> "/usr/local/bin/composer"
exit
Exit:  Goodbye.

Check (Check if the server is ready to receive the application) [~0.53s]
-- Check/Php (Checks if the server is ready to receive a PHP application)
=> Checking presence of svn
$ svn --version
<EMAIL_ADDRESS>(production) svn, version 1.8.8 (r1568071)
<EMAIL_ADDRESS>(production) compiled Aug 13 2014, 17:12:39 on x86_64-> pc-linux-gnu
<EMAIL_ADDRESS>(production)
<EMAIL_ADDRESS>(production) Copyright (C) 2013 The Apache Software Foundation.
This software consists of contributions made by many people;
see the NOTICE file for more information.
Subversion is open source software, see http://subversion.apache.org/
<EMAIL_ADDRESS>(production) The following repository access (RA) modules are available:

ra_svn : Module for accessing a repository using the svn network protocol.

with Cyrus SASL authentication
handles 'svn' scheme

ra_local : Module for accessing a repository on local disk.

handles 'file' scheme

ra_serf : Module for accessing a repository via WebDAV protocol using serf.

using serf 1.3.3
handles 'http' scheme
handles 'https' scheme
|=> Checking PHP version
|=> Checking presence of Composer
The Composer package manager could not be found
The tasks queue was canceled by task "Check"
Execution time: 0.444s

There's two reasons a Package Manager would fail the check:

Rocketeer can't find the path to it
Rocketeer didn't find any manifest for it, locally or on the server (composer.json)

Can you check which of the two failed the check by doing this:
$ rocketeer tinker
Psy Shell v0.3.3 (PHP 5.6.4 — cli) by Justin Hileman
>>> $ssh->binary('composer')->hasManifest();
=> true
>>> $ssh->binary('composer')->getBinary();
=> "/usr/local/bin/composer"

Hmmm... It seems that there is no manifest:
Pay Shell v0.3.3 (PHP 5.5.9-1ubuntu4.5 - cli) by Justin Hileman
>>> $ssh->binary('composer')->hasManifest();
=> false

Seems to be the problem. Can you tell me how to fix this or point me in the right direction?

Does you application actually need Composer?

No, I don't think so... How can I make it stop checking for it?

You can just disable the PHP related checks by setting 'Check' => null in 'strategies.php– you wouldn't get much use of the checks anyway since everything Rocketeer knows and checks about your application it infers from what version/dependencies/extensions you require in thecomposer.json` file.
Note that that will only work with 2.1-dev, not on 2.0.4

Thanks...
I am running:
Rocketeer version 2.1.0 (f865b2d6a32af2defaf1511e453d602d981fba8d)

I tried:
'check' => '',

in strategies.php. Here is the output of rocketeer setup:
vagrant@deploy:~/example_development$ rocketeer setup

No agent is set for [production], please provide one:
| Setup (Set up the remote server for deployment)
|-- Check (Check if the server is ready to receive the application)
|===> Checking presence of svn
$ svn --version
<EMAIL_ADDRESS>(production) svn, version 1.8.8 (r1568071)
<EMAIL_ADDRESS>(production) compiled Aug 13 2014, 17:12:39 on x86_64-pc-linux-gnu
<EMAIL_ADDRESS>(production)
<EMAIL_ADDRESS>(production) Copyright (C) 2013 The Apache Software Foundation.
This software consists of contributions made by many people;
see the NOTICE file for more information.
Subversion is open source software, see http://subversion.apache.org/
<EMAIL_ADDRESS>(production) The following repository access (RA) modules are available:

* ra_svn : Module for accessing a repository using the svn network protocol.
  - with Cyrus SASL authentication
  - handles 'svn' scheme
* ra_local : Module for accessing a repository on local disk.
  - handles 'file' scheme
* ra_serf : Module for accessing a repository via WebDAV protocol using serf.
  - using serf 1.3.3
  - handles 'http' scheme
  - handles 'https' scheme
PHP Catchable fatal error:  Argument 1 passed to Rocketeer\Tasks\Check::checkLanguages() must be an instance of Rocketeer\Abstracts\Strategies\AbstractCheckStrategy, null given in phar:///usr/local/bin/rocketeer/src/Rocketeer/Tasks/Check.php on line 120

Is there something else I need to do?

Yeah no sorry, I just pushed that actually so the PHAR wasn't done rebuilding when you tested :p Download the PHAR again from http//rocketeer.autopergamene.eu/versions/rocketeer-develop.phar and it should work

Yes! Thanks so much for your help.

@Anahkiasen, using the new phar file and disabling the check strategy worked for me too!

Aight, closing then

@Anahkiasen  sorry since issue is related to this thread, I am posting it here
will this not be ported back to 2.0.4  ?  since you said that is the latest stable release, honestly I am kind of blocked here with that issue and can't move forward :)  I appreciate your response and patience.

version 3.0.0 and and disabling check in strategies worked for me too! for now I will just use it for my test environment.

We're having this problem as well (in our case with a legacy project with no composer.json at all). The rocketeer-develop PHAR seems far from stable for production use and 2.1.2 doesn't seem to be able to handle an empty value for check.
Is there a hotfix for 2.1.x available or is there anything else we can do?

I had similar behavior and a google search lead me here, but thanks to the $ssh->binary('composer')->hasManifest();, I realized that I needed a composer.json in my app's root directory (duh?).
$ rocketeer --version
Rocketeer version 2.1.2
$ rocketeer check
...
|=> Checking presence of Composer
The Composer package manager could not be found
The tasks queue was canceled by task "Check"

Simple solution, but maybe it will save someone else some time.
$ cd /path/to/app/root/dir
$ composer init
$ cat /path/to/app/root/dir/composer.json
...

$ rocketeer tinker
Psy Shell v0.3.3 (PHP 5.6.4 — cli) by Justin Hileman
>>> $ssh->binary('composer')->hasManifest();
=> true
>>> $ssh->binary('composer')->getBinary();
=> "/usr/local/bin/composer"

Maybe there is a way to add a more descriptive solution when the check task fails with composer?


Cluster Provisioning: Create v2 Cloud Credentials & Node Pools - Azure

Tied to v2 work with @vincent99. This could be hopefully easily combined with work tied to RKE2 node pools though if that's not the case, then we can split that into another ticket.

Test report:
This works correctly and it corresponds to the mockups visible here https://github.com/rancher/dashboard/pull/3151
I was able provision several RKE2 clusters within https://github.com/rancher/dashboard/issues/2166 (with ~ default values)
Close.


Updated LUT for chromatic adaptation

I tested this locally and it appears to now produce the same output as the EASYDCP transform, so this is most excellent work!!!!


add(consensus): Check consensus branch ids in tx verifier

Motivation
Close #9064.
Context
The tx verifier is not checking if the nConsensusBranchId field in V5 txs conforms to the following consensus rule:

[NU5 onward] If effectiveVersion ≥ 5, the nConsensusBranchId field MUST match the consensus branch ID used for SIGHASH transaction hashes, as specified in ZIP-244.

We have this check in the block verifier. However, since the mempool uses only the tx verifier, it can currently accept txs with an incorrect nConsenusBranchId.
Specifications & References

The consensus rules in § 7.1.2 Transaction Consensus Rules.

Solution

Add the check mentioned above to the tx verifier.
Clean-ups:

Move test-only tx methods out of the default impl

Tests

Add unit tests.
Adjust existing unit tests.

PR Author's Checklist

[x] The PR name will make sense to users.
[ ] The PR provides a CHANGELOG summary.
[x] The solution is tested.
[x] The documentation is up to date.
[x] The PR has a priority label.

PR Reviewer's Checklist

[ ] The PR Author's checklist is complete.
[ ] The PR resolves the issue.

Just for reference I think the 4.10 rules are about the branch ID used in SIGHASH. What we seem to have missed is from 7.1.2:

the nConsensusBranchId field MUST match the consensus branch ID used for SIGHASH transaction hashes

which we check for transactions in blocks, but not for transactions in the mempool.

Consensus rules often start with conditions such as "[NU5 onward]". We take those conditions as implicit in the code, but we could implement them by checking what NU is active for the thing being checked. Should we start making those conditions explicit?

I made the NU5 onward condition explicit in c698507.


Add flag for biokepi machine to not be a script, but ready for compilation

This is already there: https://github.com/hammerlab/secotrec/blob/master/src/lib/command_line.ml#L171-L175

Amazing—sorry I missed it!


self keyword is not highlighted

For example, given a function
fn distance(self, other: Vec3f) -> f64 {
    (self - other).length()
}

self does not appear highlighted as other keywords would.

This is not accidental, self is highlighted as a variable and Self as a type.
I remember coloring them as keywords and it looked pretty bad.

Mm, I see. It caused me some confusion as the compiler's error messages do state that "self" is a keyword. I'll close this issue then. Thank you for the clarification.


K8S View does not render correctly

Only when build with aot and a prod build.
Side nav gets hidden.
Top navigation header has incorrect vertical spacing - table renders underneath tabs


Clone hanging when reading asyncio response object body from S3

Today, when I was using bookstore to clone from S3, I found that responses were just hanging indefinitely.
It seems to be occurring when we attempt to read the s3 response object's body (with await obj['Body'].read()).
Importantly, it doesn't hang when using minio's mock s3 server.
Does anyone have any insight as to what could be causing this?
@kylek @willingc @mseal

I figured out that this seems to be related to trying to read the Body outside of the client context manager.
What's odd is that it still works with minio, and so I'm wondering if there is a hidden race condition.
In any case, I opened https://github.com/aio-libs/aiobotocore/issues/710 so we could track the underlying issue there.
In the meantime I'll work on a refactor so we can get 2.3.1 out the door.


Please update the minify-html create to the latest v0.15.0

Old minify-html-common crates download files through https during build which isn't allowed in package builders.

There's already a PR for that: #677 … the original PR had some issues, so I wanted to test it a bit more.

Should be released with 0.18.6.


Should we convert to isomorphic?

My vote is YES, what you think about?
more about:
http://josdejong.com/blog/2015/03/28/a-broader-view-on-isomorphic-javascript/

YES

We need to plan how to initiate the lib in both worlds
Em qui, 22 de jun de 2017 09:26, Marcos Tischer Vallim <
<EMAIL_ADDRESS>escreveu:

YES
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mvallim/javascript-collections/issues/4#issuecomment-310364896,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADtMflqEF3wrx2p1OYyn-oYDEbibTI9nks5sGl1ggaJpZM4OAdYi
.

Do you have any idea, to how?

Leave this one to me.

I used to use/like this UMD pattern

@mvallim @lnfnunes
Update:
Made it work for node and browser with webpack, but it is not working for AMD.
I'll try some more tricks and if no result come up I'll share with you and ask for help!

Which branch ?

@mvallim @lnfnunes
Well...
Finally it is commited.
To check everything i changed, please go to commit 2a2f4b45454610e69c1daee2020a9c7cfa276f57
I ended changing a lot of things to get it done. Also I got some extra tools to help us keep quality.

Editorconfig : I've added editorconfig to help us use a standardized way to write code. Feel free to change or update as you wish.
Eslint: Added, also, eslint to validate our code. Some fixes were made (parameters and variables not being used, and this kind of mistake).

Eslint will check ES6 for Tests, but ES5 for Main code. This is because UMD does not transpile to browser compatible code like Babel would be able to do.
New task added to npm/gulp: npm run lint / gulp lint

UMD: the UMD pattern is included in the code by browserify. This way, we need to program Node-like and browserify makes it compatible with browsers.
Tests: Since the code changed to be Node-Like, I couldn't keep testing with Karma anymore since it runs tests against a browser and a browser does not recognize the "require" function, so...

All tests were converted to Jest/jasmine syntax since Jest is the new tool to run the tests.
To execute the tests the command is the same as before : npm t

Nice job @rmalveis !
I'm closing this issue.


Danger alert does not appear unless I refresh the page

The plugin works when I do router push to another location, however, it does not work if I want alert to show right away.
For example, I am using danger alert to warn users when there is an error during logins, such as incorrect password, or incorrect user. In these situations, the route does not change (no this.$router.push).
How do I flash an alert right away when the route does not change?

Is there an appropriate way to handle this? I'm guessing it has to do with how Vuex updates state?


Add expect-puppeteer and rewrite one of the existing tests using it

https://www.npmjs.com/package/expect-puppeteer
Why do we need it
Writing integration test is very hard, especially when you are testing a Single Page Applications. Data are loaded asynchronously and it is difficult to know exactly when an element will be displayed in the page.
Puppeteer API is great, but it is low level and not designed for integration testing.
This API is designed for integration testing:

It will wait for element before running an action
It adds additional feature like matching an element using text

Assign it to me please.

Sounds good. I don't have any knowledge of testing yet. Can I still take this issue ? I am willing to learn the testing though. This looks good to start with.

@rajat-mehra05 We need to write E2E tests for all the pages across multiple applications.
You can work on them. I am creating some in a minute.

@priyanshk20 Can we quickly have a PR with expect-puppeteer added, and write tests in a separate PR so that others can start working on it ASAP.

Sure, I will be doing this ASAP.


Update "Sample projects" refs in documentation.

[x] Update Readme.md guide on how to run sample apps
[ ] Update Readme_CN.md guide on how to run sample apps
[x] Update broken links in MultiTarget.md and rephrase it (supersedes #1551)
[ ] Update broken links in MultiTarget_CN.md and rephrase it
[x] Update broken links in Plugins.md
[x] Update broken links in Plugins_CN.md

Once this PR is merged, we should add these two docs changes to the CN todo-list.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #1573      +/-   ##
==========================================
+ Coverage   82.05%   87.82%   +5.76%
==========================================
  Files           5        5
  Lines         156      156
==========================================
+ Hits          128      137       +9
+ Misses         28       19       -9

Impacted Files
Coverage Δ

Sources/RxMoya/Observable+Response.swift
70.58% <0%> (+8.82%)
:arrow_up:

Sources/ReactiveMoya/SignalProducer+Response.swift
90.9% <0%> (+9.09%)
:arrow_up:

Sources/RxMoya/Single+Response.swift
100% <0%> (+12.5%)
:arrow_up:

Continue to review full report at Codecov.

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

Looks like you can't tag issues through a PR review. Double tag #1357


Chart not working properly

Describe your bug here.
Hello, I have recently made a few charts for some custom songs. I saved them and put them in their respective folders. Expecting them to load in when I clicked ''Reload JSON'', I just got the default chart (when you load a song) and not the chart I made. What have I missed?
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No.

put in the name into box reload audio then json and make sure to load events sometimes too but this maybe due to some error im not an expert thats why i backup mine

This is likely an error on your part, explain what you mean by "default chart" A different song? Difficulty? Old chart??

Yes, a different, normal song like Bopeebo or Blammed.

put in the name into box reload audio then json and make sure to load events sometimes too but this maybe due to some error im not an expert thats why i backup mine

Doesn't work, sadly! I tried both case sensitive methods, and doing load events first, reload audio second and reload json third.

The game should just crash if the chart file isn't found so this is probably an issue with your chart, post all the difficulties

The game should just crash if the chart file isn't found so this is probably an issue with your chart, post all the difficulties

My song only has ''Hard'' difficulty, I don't know if adding easy and normal should fix it.

you need a normal difficulty for reloading to work.

is the song in your json correct

there's too little information for me to help much

is the song in your json correct  there's too little information for me to help much

Nope, Fresh is replaced by Interrogation. Also i have both normal and hard difficulties, i just checked.

Still needs more info
If the game isn't crashing then it's loading what you told it to load, I'd recommend trying to reload autosave and saving the file in the proper folder again if possible

Make sure the json file itself is also named 'interrogation', if it's named anything that's in assets/data, it will go with that one no matter what for no known reason

The interrogation json is located both in assets/data and mods/data and it still wont work. I'll update you on this after i try some common fixes, thanks!

still waiting for the files to be posted to see what the problem is

My apologies, but json files aren't supported in github. If you need any other files tell me.

you can change extensions

you can change extensions

ok
[interrogation.txt](https://github.com/ShadowMario/FNF
interrogation-hard.txt
-PsychEngine/files/8856116/interrogation.txt)

both charts are just bopeebo?? it's loading bopeebo's chart because the chart is literally bopeebo

Yeah i think that's the issue, but I don't know how to make my own chart, so I just replaced it with bopeebo, thinking that it would overwrite it. Guess not.

So there there is no issue? If you want a blank chart there is a button to delete all notes


Add mesh operator install command

This command would basically kubectl apply the manifest to bootstrap the in-cluster controller.
There should be a corresponding operator delete command to remove the operator from the cluster (this could have a --delete-istio flag to also remove Istio).

This is marked as a 1.4 P0, do we have an owner?

Fixed through https://github.com/istio/operator/pull/348.


(PUP-7042) Mark strings in transaction

This commit marks user-facing error and info level strings in
lib/puppet/transaction.rb and lib/puppet/transaction/* for
translation.

CLA signed by all contributors.

Closing and reopening to fix Travis.

Closing and reopening to fix CI.


Rename Math.h

This is going to cause a lot of build issues on Windows, as Visual Studio is case insensitive with headers.  Perhaps MathHelpers.h or MathLib.h

Good call @abwood.
@ErixenCruz can you make this fix?

cmake --build Build --config Release -j 4
not found Math.h