testing no fork e2e run

Problem Statement
What is the problem you're trying to solve?
Related Issue
Fixes #...
Proposed Changes
How do you like to solve the issue and why?
Checklist

[ ] I have read the contribution guidelines
[ ] All commits are signed with git commit --signoff
[ ] My changes have reasonable test coverage
[ ] All tests pass with make test
[ ] I ensured my PR is ready for review with make reviewable

/ok-to-test sha=212c784411558fc6175bf13b2fbfa23b44fb1d43

/ok-to-test sha=71629bee815ff191d812b6161b9d1dbb61e4dc09

/ok-to-test sha=4c92aea6799a983aa052eb1d7b5d7f53b2d29b59

/ok-to-test sha=b6367dc3b7563d82c8898b39d716e6c2ddcbcac1

/ok-to-test sha=2073a22656c65fbfd87de611499997d64378cd76

/ok-to-test sha=a07f4c6154870d0dda9ee6b2842b12a9bab16a1f

/ok-to-test sha=44911c3590a7639f374db03e09322c7894f890f8

/ok-to-test sha=f62aa5e5cba18bf0e8192f04f89b84645e0f6b05

/ok-to-test sha=277869d55d213087d13dc5fd3c23dfe5f5dae8b9

/ok-to-test sha=dbd6cbdb22e1c6ab559b6bc674b30a66fc55ae3f

No idea what the linter is smoking
(base) ➜  external-secrets git:(test-fork-e2e) ✗ ct lint --config=.github/ci/ct.yaml
Linting charts...

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 external-secrets => (version: "0.10.3", path: "deploy/charts/external-secrets")
------------------------------------------------------------------------------------------------------------------------

Hang tight while we grab the latest from your chart repositories...
...Unable to get an update from the "weave-gitops-enterprise-charts" chart repository (https://charts.dev.wkp.weave.works/charts-v3):
        Get "https://charts.dev.wkp.weave.works/charts-v3/index.yaml": dial tcp: lookup charts.dev.wkp.weave.works: no such host
...Unable to get an update from the "tf-controller" chart repository (https://weaveworks.github.io/tf-controller/):
        failed to fetch https://weaveworks.github.io/tf-controller/index.yaml : 404 Not Found
...Successfully got an update from the "1password" chart repository
...Successfully got an update from the "autoscaler" chart repository
...Successfully got an update from the "nginx-stable" chart repository
...Successfully got an update from the "karpenter" chart repository
...Successfully got an update from the "cert-manager" chart repository
...Successfully got an update from the "external-secrets" chart repository
...Successfully got an update from the "gitea-charts" chart repository
...Successfully got an update from the "datawire" chart repository
Update Complete. ⎈Happy Helming!⎈
Saving 1 charts
Downloading bitwarden-sdk-server from repo oci://ghcr.io/external-secrets/charts
Deleting outdated charts
Pulled: ghcr.io/external-secrets/charts/bitwarden-sdk-server:v0.3.1
Digest: sha256:d41e67f43140aba5a30e01e34ec9a1007772ba411992d0ad9d165fbb35edc1e0
Linting chart "external-secrets => (version: \"0.10.3\", path: \"deploy/charts/external-secrets\")"
Validating /Users/skarlso/goprojects/kubernetes/external-secrets/deploy/charts/external-secrets/Chart.yaml...
Validation success! 👍
Validating maintainers...

Linting chart with values file "deploy/charts/external-secrets/ci/main-values.yaml"...

==> Linting deploy/charts/external-secrets

1 chart(s) linted, 0 chart(s) failed

------------------------------------------------------------------------------------------------------------------------
 ✔ external-secrets => (version: "0.10.3", path: "deploy/charts/external-secrets")
------------------------------------------------------------------------------------------------------------------------
All charts linted successfully

/ok-to-test sha=6ef95a89953c9be6751d6c73d18c3d2b8df84400

/ok-to-test sha=d11f698c40c18d2ab3db3fe2cf03312fd8158532


Wrong Recipe [Pyanodons]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to settings
Click on base, and click on Pyanodons
Enable the Automation Science Pack Recipe
Use it as the recipe you want to calculate
Recipe is wrong for current version

Expected behavior
The recipe should include a petri dish instead of a mushroom
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome

Additional context
Add any other context about the problem here.

@LonnonjamesD The current data set for Pyanodons does not include the alien life mod. My current plan is to address this by adding Pyanodons + AL as a separate option in the dropdown list. Would this be good enough? I would imagine there are players who prefer to play without alien life.

@LonnonjamesD The current data set for Pyanodons does not include the alien life mod. My current plan is to address this by adding Pyanodons + AL as a separate option in the dropdown list. Would this be good enough? I would imagine there are players who prefer to play without alien life.


[SP-6208] - Backport of PDI-19513 - Editing Field Mapping Doesn't Flag Transformation as Changed (9.3 Suite)

@smmribeiro
@bcostahitachivantara
@renato-s

:white_check_mark: Build finished in 15m 41s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:exclamation: No tests found!
:information_source:  This is an automatic message

:white_check_mark: Build finished in 27m 48s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine,ui

:exclamation: No tests found!
:information_source:  This is an automatic message


Parsing Java sources is slower than it should be

Parsing the set of Java files in Spring Boot can take hours.

We should be seeing a single parser operation for each file and one attribution operation for the entire set of files. But it appears that somehow we're doing type attribution fileCount*2 times.

Can use the utility object ParseJavaProjectOnDisk with a single arg to the root directory of a Java project cloned to disk to execute this.
Run ./grafana.sh and ./prometheus.sh from rewrite-java-11/scripts to see metrics. If you are on Mac be sure to run sudo ifconfig lo0 alias <IP_ADDRESS>/24 once per system restart before launching either of these scripts.


fix: disable the username check

disable the username check

 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.


add Num a => Rope -> Maybe a

A very common scenario is that we want to pass a parameter to a CLI that is to be interpreted as an integer or a float. Currently Core.Text does not easily handle that situation.

I think there are two issues here.

Converting numbers (in various forms) to Rope.
Handling numeric command-line option values.

It's possible a good solution to 1 might make 2 trivial but we also might want a dedicated helper query* function in core-program if it's at all finicky.
Anyway, 1 first. What does a good convertInteger look like?

Would a Read instance help here? You would not have the Num restriction but there is a 'readMaybe' that (mostly) matches the type signature you want: https://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Text-Read.html#v:readMaybe
The downside could be that having a Read instance opens up the possibility for the use of unsafe functions such as the simple 'read'

@carlosdagos you're right, a Read instance could be a good idea, but it's String -> Rope. We've kinda already got that, in Textual. Which perhaps suggests the solution: an instance of Textual for Int, Float etc.
λ> execute (write (intoRope 17))
17
λ>

which would presumably go through Show which is inefficient, but whatever; most numerical conversions to text strings do go via Show -> String. And meanwhile we've got writeS and debugS which do exactly that. Is what it is.
There'd be no particularly valid fromRope implementation; so at best the Textual instance would be partial, and we don't like to do that to people, but perhaps it's ok. Or we return "" on an invalid conversion back.

I've been thinking along the lines of creating another conversion typeclass. Rather than Textual Int (which really doesn't have a valid fromRope) perhaps
class Numeric x where
    parseNumber :: Rope -> Maybe x

Most conversion functions around the ecosystem are Maybe whatever, and most types have one way or another to parse text, so this might be a nice way to wrap those up in one place.

The design I'm heading toward is
    thing <-
        queryOptionValue "thing" >>= \case
            Nothing -> throw Boom
            Just value -> case parseExternal value of
                Nothing -> throw Boom
                Just value' -> pure value'

that's still very cumbersome.

How about this:
    thing2 <-
        queryOptionValue "thing" >>= parseOptionValue

with:
parseOptionValue :: Externalize x => Maybe Rope -> Program z x

Better idea:
    thing3 <-
        queryOptionValue' "thing" >>= \case
            ...

queryOptionValue' :: Externalize x => LongName -> Program z (Maybe x)

this allows us to just assume success and throw an appropriate exception if not, which is actually what goes on elsewhere in the command-line parameter handing subsystem.


Make form validation work with nested form fields

 (1) Reactions: Make form validation work with nested form fields


Add build-templates-e2e namespace for syncing with vault

We updated components/tekton-ci/base/external-secrets/quay-push-secret.yaml in vault and it was synced with the tekton-ci namespace and it works there. But the same task fails in the build-templates-e2e namespace where the e2e tests with wrong credentials error.
I have a feeling that since it was missing from here, it did not sync the new secret.

/retest-required

/lgtm

/approve

/retest-required

/retest-required

/lgtm
/approve


Proxy should be configurable

We don't want to hard-code a particular proxy, so should be configurable...somehow

Maybe make azure search api key and baseurl configurable as well.


Mk stash kata 2

Corrections related to previous pull request comments.

The kata has become slightly longer and more complex now.  In order to retain oversight for the user I introduced a 2nd level in the command list as well as quoted text.  I believe it makes it easier to follow the kata, but it does represent a discontinuity with other katas
In order to illustrate the difference between git stash apply and git stash apply, I introduce a dependency on git reset. This is unfortunate, but the simplest way i could come up with.
I am creating a new pull request, from a new branch.  It is unclear to me whether this idiomatic or undesirable.

Hi Tom!
Again thank you! This has become a great exercise!
I agree the dependency to git reset is unfortunate. I think we can either fix that by linking to the reset kata or by having the user run the setup script again. I'm not sure what is the least confusing solution :)
Pick your poison and I'll happy to merge your pull request.
I think generally the work should be done in the same pull request, rather than opening a new one - but I'm no open source wiz, so I might be off :)

I had considered re-sourcing the script but rejected it on the grounds that I want to provide a learning path internal to the repo. Mistakes are made and usually there'll be no script to bail me out.
I will update the readme with a reference and try to figure out how to update the pull request

All done. It seems the pull request includes the new commit automatically so we're good to go


Docker build fails

The docker build command fails with the following error:
Your bundle is locked to public_suffix (2.0.4), but that version could not be found in any of the sources listed in your Gemfile. If you haven't changed sources, that means the author of public_suffix (2.0.4) has removed it. You'll need to update your bundle to a different version of public_suffix (2.0.4) that hasn't been removed in order to install.

I'm not able to replicate the issue with docker build -t dashing . Would you mind sharing the command you're using?
The container builds just fine for me. Are you fetching gems from a 3rd party registry such as artifactory?

Apologies, the error doesn't happen at build time, it happens when you try to run the docker image, so the command 'docker run -ti -p 3030:3030 -e GITHUB_REPOS=UKHomeOffice/my-project-1,UKHomeOffice/my-project2 -e GITHUB_TOKEN=mypassword dashing' would throw this error

@tamer316 would you mind testing this branch? https://github.com/UKHomeOffice/ho-example-dashboard/pull/14

Yep that works perfectly

Great, thanks


Include buttons in fixed README

Please describe the changes this PR makes and why it should be merged, as well as any and all proof of successful tests:
This PR adds the changes of #13 to the changes in #14.
Semantic versioning classification:

[ ] This PR changes the framework's interface (methods or parameters added)

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)

[x] This PR only includes non-code changes, like changes to README, etc.

👍


restore python 3.7 compatibility

Codecov Report

Merging #26 (22931e2) into main (464d6c5) will decrease coverage by 1.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #26      +/-   ##
==========================================
- Coverage   91.86%   90.60%   -1.27%
==========================================
  Files          14       14
  Lines         381      383       +2
==========================================
- Hits          350      347       -3
- Misses         31       36       +5

Impacted Files
Coverage Δ

src/git_guardrails/validate/__init__.py
87.87% <100.00%> (-3.88%)
:arrow_down:

src/git_guardrails/validate/options.py
90.24% <0.00%> (-2.44%)
:arrow_down:

src/git_guardrails/command_line.py
74.07% <0.00%> (-0.93%)
:arrow_down:

src/git_guardrails/cli/ux.py
100.00% <0.00%> (ø)

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 464d6c5...22931e2. Read the comment docs.


Update Readme rules

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.91%. Comparing base (10f5913) to head (c83b13e).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main     #335   +/-   ##
=========================================
  Coverage     99.91%   99.91%
  Complexity      766      766
=========================================
  Files            66       66
  Lines          2358     2358
=========================================
  Hits           2356     2356
  Misses            2        2

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

Todo, add https://github.com/VincentLanglet/Twig-CS-Fixer/pull/336 too


Question regarding width height

Hi, not an issue more like questions regarding width height.
is width height in relations to the position?
width 1 = position 1x?
Trying to get some kind of bearing to how to scale things up and down.
I don't suppose there is any real world translation on this?
Looking for something like 1 = 1 feet or 1 meter

Hi @jimji1005,
Units on the Viro platform are in meters.
So all measurements are in meters and all positions are in meters from [0,0,0].
Thanks,
Andy

Nice! Thanks! Please update the doc to include this.
Sorry to segway, a different question
Once AR session initializes, it sets position 0,0,0, which starts tracking cam position.
Is there anyway to reset this without restarting the scene?

Hi @jimji1005,
No, there's currently no way to reset the camera position.
Is there a reason/use case for doing this?
Normally in AR applications, you'd want your components to "stay" where they have been placed.

yeah for sure
I am testing the feasibility of tracking the position of the phone as it moves around on all 3 axis
thinking of creating a virtual box that can render where is the phone

Hi @jimji1005,
I'm interpreting your use case in 2 ways:

You would like to render a box around the camera and have it follow the camera around. This is something we don't currently support but is something on our backlog and the idea is to be able to fix things relative to the camera position constantly.

You want to render a box at the current camera position and have it stay there. In this case, you can use the getCameraOrientationAsync() to get the position of the camera at the current time: http://docs.viromedia.com/docs/viroarscene#methods

In either case, you shouldn't have to reset the world to the camera position.
Andy

As for the rendering, i was more thinking about having a server that is tracking the device position.
No worries, i can always use math to track device position in 3axis
I wasn't able to utilize getCameraOrientationAsync for some reason, seems like the method wasn't exposed. i copied the code out and placed it in the demo app to get it to work.

Interesting, let us know if that doesn't work for you.
That's interesting that you couldn't get getCameraOrientationAsync() working. An example of it working is in our ARSampleApp line 172: https://github.com/viromedia/ViroARSampleApp/blob/master/js/ARHitTestSample.js


Is there a way to easily wrap or extend the components/directives in this library?

I like using this library, it works very well and helps achieve a very clean effect.  The problem is, for my job, it's against coding standards to put third-party components/directives directly into html... the justification is that it hurts maintainability as its a lot harder to pull out than it is to put in.
I was hoping there was a way to wrap or extend each item I needed with my own name-spaced element, for re-use throughout a project, so that if we ever want to change something we simply change the implementation of that element to another library or to our own custom code.  But so far I cannot find a way to do this.

Generally we discourage extending the components but don't do anything to strictly prevent it.


MTU changer

[X] I have read and acknowledged the preliminary instructions.
MTU

would like to say could there be an MTU changer please cause roblox sets it to 1396 which slows my connection and i wanted to change it to 1500 this could help me alot with my connection cause normally when my MTU is set at 1500 i get like 20 ping which is very good

No

set your mtu as 5000 for a free robux prize


Add information for apache mod_userdir

Description
Resolution of issue #4471
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[x] User guide updated
[ ] Conforms to style guide

We could just take the top 100 forum posts as a to do list...
In seriousness though, it's a good idea. I'd be interested in drafting Apache on Debian-based Linux and MacOS. My Ngix-fu is marginal, but likely also sufficient. I'm no use for IIS or whatever Azure is though.
Docker, AWS, and Kubernetes could be a fun rabbit hole. Tough to draw a line between server and OS though, especially in VM land.

Agreed. And we don't need to duplicate existing documentation for these services, that is on the service itself. I like how you supplied very specific steps and example code for how to configure. I believe that some other services might actually need some files supplied in another project, but for the docs section this is good.
Would you be willing to start such a section, with this content as the first?

Would you be willing to start such a section, with this content as the first?

Yes.
As I've mentioned, my availability varies greatly due to the nature of teaching, and I'm heading into another time of the year where I will not have time. But by late June and July this should be fun.
Is a branch or simply PR more appropriate for something this size?


METRON-2114: [UI] Moving components to sensor parser module

Contributor Comments
This PR is part of a larger sequence of PRs we are opening to make the reviewing process of the relatively large Parser Aggregation UI feature easier.
To see only the changes that apply to this PR, please review the last commit in the list of commits (the commit to review contains the title of the PR)
If you want to learn more about why we picked this method, please refer to the discussion thread here
Please find the full list of relating PRs below.
About this changeset
In this first PR, I'm moving the components which belong to parser configuration feature to a separated angular module. This makes the code cleaner and gives us the opportunity to initialize the parser config state inside a separated module.
(Please keep in mind this is only a subset of changes belongs to Parser Aggregation UI, therefore, you not gonna be able to test the whole function through this PR alone. Because of the same reason some unit tests might failing. The purpose of this PR is to break down and support the code review process.)
You can find the belonging Jira ticket here.
If you like to learn more about the feature and its context please check the Jira ticket about Parser Aggregation feature on the Metron Alerts UI
Previous and following PRs
Current: PR#1 METRON-2114: [UI] Moving components to sensor parser module
PR#2 METRON-2116: [UI] Removing redundant AppConfigService
PR#3 METRON-2117: [UI] Aligning models to grouping feature
PR#4 METRON-2115: [UI] Aligning UI to the parser aggregation AP
PR#5 METRON-2122: [UI] Fixing early app config access issue
PR#6 METRON-2124: [UI] Move status information and start/stop to the Aggregate level
PR#7 METRON-2125: [UI] Making changes visible in the parser list by marking changed items
PR#8 METRON-2131: Add NgRx and related dependencies
PR#9 METRON-2133: Add NgRx effects to communicate with the server
PR#10 METRON-2134: Add NgRx reducers to perform parser and group changes in the store
PR#11 METRON-2135: Add NgRx actions to trigger state changes
PR#12 METRON-2136: Add parser aggregation sidebar
PR#13 METRON-2137: Implement drag and drop mechanism and wire NgRx
PR#14 METRON-2138: Code clean up
PR#15 METRON-2139: Refactoring sensor-parser-config.component and wire NgRx !!! ONLY THIS LAST PR MENT TO BE MERGED TO THE FEATURE BRANCH!!! It should build fine on Travis and can be tested against full dev vm.
Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

[x] Have you ensured that any documentation diagrams have been updated, along with their source files, using draw.io? See Metron Development Guidelines for instructions.

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

+1, looks pretty straight forward, let's get this ball rolling

Thanks, Otto for reviewing!

Can we get travis passing @tiborm?

Can we get travis passing @tiborm?

@merrimanr, @tiborm laid out their plan in the PR description on this one. i.e.

ONLY THIS LAST PR MENT TO BE MERGED TO THE FEATURE BRANCH!!! It should build fine on Travis and can be tested against full dev vm.

This one looks pretty straight forward to me - I'm +1 with @ottobackwards  as well.

Hey @tiborm, we appreciate the effort on splitting up your work for attribution and review purposes. To be clear, the individual PRs need to be merged and propagated through.

@mmiklavc @ottobackwards @merrimanr Having all the PRs merged and propagated through make some additional work but we can live with that. Thanks for reviewing this!

Closing and updating


Ability to set log levels

Describe the feature
Allow configuring the log level, e.g. from INFO to WARN or ERROR. Could be configurable as a global CLI flag.
Additional context
Some of our SQL occasionally has sensitive information that we wouldn't like to be logged.
Who will this benefit?

People who want less "chatty" logging
People using Spark external tables with credentials.

Thanks for opening this issue @friendofasquid. My instinct is that rather than configure log levels, we'd want to support masking sensitive values in the logs.
I think we can do this by:

Automatically masking sensitive values defined in the active target (passwords, etc)
Adding a context function that lets you register new sensitive values, eg:

select
  '{{ sensitive(env_var('MY_SENSITIVE_VAR')) }}'
...

Here, the sensitive context function would add a value to a list of sensitive values to mask in the log handler. This isn't a spec per se - there are likely other good ways to go about adding masking like this, and I don't have strong instincts about best practices around logging like this.
@beckjake do you have any strong opinions on how something like this could work?

I think it's hard to say without knowing more about what spark external tables with credentials look like - does querying them include the credentials in the select statement, or something?
It seems like this user's needs could potentially be served by adding a flag to only log to the file at INFO level, which seems like a very reasonable flag to me.
I think the sensitive functionality would have to be limited to text-mode logging, as digging through the extras dict for sensitive data sounds tedious and error-prone. I'm not sure on the performance impact (my guess is: more than you'd think!) but you could just do something along the lines of:
for replacement in GLOBAL_REPLACEMENTS:
  msg = msg.replace(replacement, '*' * len(replacement))

or use a huge regex pattern by combining all of the replacements and call re.sub - that might be faster, I'd have to test to be sure.
The '*' * len(replacement) replacement scheme leaks information, but it doesn't screw up the formatting... trade-offs.

Here's what a Spark external table declaration might look like, within a simple dbt template:
{% statement() %}
CREATE TEMPORARY VIEW source
USING jdbc
OPTIONS (
url 'jdbc:postgresql://<IP_ADDRESS>/d?',
user 'scott',
password 'password',
dbtable 't',
driver 'org.postgresql.Driver'
);
{% endstatement %}
SELECT * FROM source

@friendofasquid @beckjake is there any way to hide or mask the password getting logged in to dbt logs?

@rajeshmese — I'm not aware of any enhancements that would help, unfortunately. We still struggle with this.

Hello! Please reopen this issue cause i have trouble with thousands of rows in my dbt log and output from Databricks when having mistake and can not reduce it somehow!

@Gunnnn That sounds like it may be caused by something unrelated. Shot in the dark: Any chance you're using python 3.10? It sounds like folks are seeing tons of extra log lines related to some deprecation warnings in a dependency (https://github.com/dbt-labs/dbt-core/issues/4537), something we plan to fix when we add official support (https://github.com/dbt-labs/dbt-core/issues/4562)

@Gunnnn That sounds like it may be caused by something unrelated. Shot in the dark: Any chance you're using python 3.10? It sounds like folks are seeing tons of extra log lines related to some deprecation warnings in a dependency (#4537), something we plan to fix when we add official support (#4562)

dbt version: 1.0.1
python version: 3.7.9
I see a lot of strings from Databricks stacktrace.


Creating multiple OpenAPIBackend classes with shared default handlers cause data corruption

This is something I hit which took me a while to figure out.
I was creating a server that implemented two different API definitions, so I created two instances of OpenAPIBackend. Each one was given its own definition and its own set of operation handlers.
However, in the constructor I also submitted a set of default handlers for notFound and such. This object was passed to both instances of OpenAPIBackend.
In the constructor, there's this line to copy the options passed:
const optsWithDefaults = Object.assign({ apiRoot: '/', validate: true, strict: false, quick: false, ajvOpts: {}, handlers: {},
    securityHandlers: {} }, opts);

While this creates a new opts object, it does not deep-copy opts, so opts.handlers and opts.securityHandlers are copied by reference. This can result in weird behaviour if a constant is passed around, ending up being a shared mutable property on the OpenAPIBackend objects.
The result of this was that each instance used the same object for the map between operationIds and handlers, meaning a lot of warnings about unknown operation etc, since OpenAPIBackend 1 had a definition with an operation getFoo but operations getFoo and getBar registered in the handlers object, and OpenAPIBackend 2 had a definition with an operation getBar but also both operations in the handlers object.
Simple PoC:
test.ts:
import * as OpenAPI from 'openapi-backend';

const handlers = {
    notFound() {
        throw new Error('not found');
    },
};

async function test() {
    const api1 = new OpenAPI.OpenAPIBackend({
        definition: __dirname + '/foo.yml',
        apiRoot: '/foo',
        handlers
    });

    api1.registerHandler('getFoo', () => {});
    await api1.init();

    console.log('after registering api1');
    console.log('api1:', api1);

    const api2 = new OpenAPI.OpenAPIBackend({
        definition: __dirname + '/bar.yml',
        apiRoot: '/bar',
        handlers
    });

    api2.registerHandler('getBar', () => {});
    await api2.init();

    console.log('after registering api2');

    console.log('api1:', api1);
    console.log('api2:', api2);
}

test();

foo.yml:
openapi: 3.0.2
info:
  title: 'FOO'
  version: 1.0.0
paths:
  /foo:
    get:
      operationId: getFoo
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                type: object
                properties:
                  message:
                    type: string

bar.yml:
openapi: 3.0.2
info:
  title: 'BAR'
  version: 1.0.0
paths:
  /bar:
    get:
      operationId: getBar
      responses:
        '200':
          description: OK
          content:
            application/json:
              schema:
                type: object
                properties:
                  message:
                    type: string

If adding Object.freeze() around my supposedly constant handlers object, I got an error instead, obviously, which puts a light on the problem (and the fact that one should really use Object.freeze more :) ). Another workaround is to pass {...handlers} instead of handlers to the constructor.
However, I'm wondering if you have any thoughts on whether this is really reasonable behaviour, or if the constructor should in fact copy the handlers and securityHandlers objects to prevent things like this? I think it's fairly often a good idea to deep copy options that are passed, but it's not always feasible so I won't say that it's definitely the right thing. But since opts.handlers and opts.securityHandlers are maps which might very well be re-used from constants, I think it wouldn't hurt copying them?

(I have no problem whipping up a PR for this if you agree it would be an improvement and there are no side-effects.)

Excellent investigation work! PR would be very much appreciated 😍

Fixed in release 3.5.4


Custom Button Grids

Is your feature request related to a problem? Please describe.
Button grids exist as a special element but cannot be edited.
Describe the solution you'd like
Button grids as a custom element.
Use code editor to design button grid similar to how the remote layout is designed. Then add that button grid to the remote layout using it's custom action name.
Describe alternatives you've considered
Remap existing button grid buttons.
Additional context
NA

It makes more sense to let users apply styles to rows and columns and to better document how to do so than to make button pads a custom element.


Add textFieldShouldReturn

Hey,
I added textFieldShouldReturn so that when the user pressed return the keyboard will disappear.
Thanks.

Hello @Maryom , thanks for the pull request.
I'll take a look in this changes asap.

@Maryom I don't think this should be a standard behavior.
There are (or there might be) cases where the default behavior of a return button press shouldn't be to dismiss the keyboard. The most common case for this is when the return key press should call becomeFirstResponder on the "next" textField.
@serralvo this should be fixed by either:

Allowing delegate assignment, by creating a local private delegate for you to use
Creating a custom delegate so that people must assign to those delegates (tip: add an assertion on setDelegate so that only your own class can be the delegate of your component)

Have fun :)


Fix unit-test related to webservice resolvers

Changes were made to the code generator and unit-test were not updated.

Done. Commit #8feb5d


<EMAIL_ADDRESS>Download failed

You most likely hit redirect from https to http mirror, which is not valid in powershell.
You have two options

Change URL in manifest and replace https with http
Download the file via browser, move it into cache folder and rename accordingly.

Is it permanent issue with libreoffice fresh and I should do it for every new version?

This is general Apache issue that their mirrors are going down frequently and lots of them are not https compatible.
Just use aria2 for these and you should be fine.

Just use aria2 for these and you should be fine.

Thx! Do you mean to somehow incorporate aria2 into scoop? Or manually download libreoffice?

scoop install aria2; scoop update libreoffice-fresh -f; scoop config aria2-enabled $false will take care of update and disable aria2. You can reenable it next time you need it.

@Ash258 amaizing! Thx!


Misc code cleanup in lab.train endpoint

This is code cleanup (see each commit message in the series for details of what is done and why). It also fixes a NameError issue when data-dir is passed on linux (found the issue during code cleanup).

It conflicted with https://github.com/instructlab/instructlab/pull/1257 so I rebased.

#1267


Add someone to GitHub

Please add the following IBMers to Hyperledger org:
MayRosenbaum
semil
arkadiPiven
HagarMeir

@denyeart which groups?

Invites sent!

@ryjones I think they just need to be Hyperledger org members to be able to get issues assigned to them. I don't think we even have groups for core fabric contributors, do we?

@denyeart sort of - take a look

@ryjones Ok, but is the "fabric contributors" group actually granted privileges anywhere? Not that I'm aware of...

no, but sub-teams are. Anyway, looks gtg


add check_token middleware

Hi,
I think this current example should better give users a hint about how to get token correctly, which is by storing the expired date and check the status of current token every time before calling.
I've add 1 additional global var, which is called expire_date. It's the expire date of current token.
And another function check_token, which will check the token availability before really call the callback function.
Thanks,
Vincent

Hi Vincent,
Thanks for your help. By design client should be unaware of the expire_date, and just get a new token when last one expires. This will simplify processes in many cases.

I see, thanks for your comment


Vim bindings: motion commands (jkhl) perform action once, even when held

When using the vim bindings, when I press and hold a motion key (jkhl) I only move one line. This is not as expected -- default vim goes multiple lines after some delay when held. I can get around this issue by eyeballing 20 lines and pressing 20j, but that's not optimal.
Further information:

tested with online vim bindings demo in all browsers below
tested on Safari 9.0, Chrome 46.0.2490.86, Firefox 42. Issue present in all cases
operation system: OS X 10.10.5
I found this issue while using vim bindings in the Jupyter notebook https://github.com/lambdalisue/jupyter-vim-binding/issues/17

I can't reproduce this on Linux. Will try on OS X later.

It does indeed happen on OS X where, contrary to other platforms, if you preventDefault a keypress event, no repeat events are fired for that key as it is being held down. We do need to call preventDefault, since we don't want pressing J to type a j character.
@mightguava Would it be problematic to have cmKeyToVimKey convert events like J (keydown) to j and Shift-J to J to work around this issue?

A couple of LightTable users have experienced this issue as well - https://github.com/LightTable/Vim/issues/49 . This is caused by an os-level global config that changed in 10.9 or 10.10. You can globally disable that config in shell with defaults write -g ApplePressAndHoldEnabled -bool false

Hm, no luck here. Is a restart is required?

Confirm, restart fixed it.


Update to cp-kafka:7.3.2 in test extention

Update the Docker image used for Kafka in the system tests to the latest version
Reviewer checklist

[ ] Read the contributing guide
[ ] PR should be motivated, i.e. what does it fix, why, and if relevant, how
[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage: 99.349%. Remained the same when pulling b7a3dbadd505eda76b545a9a5757d1a68c9511df on update-kafka-docker into a85a1e067b0b40243ee3ad66979fa0476d48d7e2 on main.


update_node_property/update_node_properties, AKA update_column(s)

This'll close #1285. I agree with @cheerfulstoic's statement that update_column is a little weird in our context, but I think that plain ole update_property is also potentially misleading because we have a very well-defined concept of "properties" within the gem. I made the distinction of node properties because it strikes me as just a tiny bit clearer, but I'm open to something else if someone has a good suggestion.
Pings:
@cheerfulstoic
@ProGM

update_node_property(ies) doesn't make sense because it's in shared and thus on ActiveRel right?  I'm totally down with just update_property(ies)

Ah right, ActiveRel. I think update_property is still potentially
misleading, can we think of something better?
On Tuesday, October 4, 2016, Brian Underwood<EMAIL_ADDRESS>wrote:

update_node_property(ies) doesn't make sense because it's in shared and
thus on ActiveRel right? I'm totally down with just update_property(ies)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neo4jrb/neo4j/pull/1301#issuecomment-251384502, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD6E98iiJBTGThsGeUAoVtqGk3ysEGRCks5qwlGPgaJpZM4KLzoP
.

update_neo4j_property or update_db_property?

Yeah, I was thinking db property. Let's go with that, I can do it tonight
and we can get this in!
On Thursday, October 6, 2016, Brian Underwood<EMAIL_ADDRESS>wrote:

update_neo4j_property or update_db_property?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neo4jrb/neo4j/pull/1301#issuecomment-252039396, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD6E98nWYfVw3bYp23tiPGDwDqlu-prwks5qxTX8gaJpZM4KLzoP
.

Looks great! 👍 Not sure that "db properties" is clear, but every AR user would use update_column(s) without thinking too much about it.

Aaaand... four days later, let's make sure things are still passing. If so, I will merge.

Current coverage is 97.09% (diff: 97.43%)

Merging #1301 into master will increase coverage by 0.02%

@@             master      #1301   diff @@
==========================================
  Files           203        203
  Lines         12252      12510   +258
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          11893      12147   +254
- Misses          359        363     +4
  Partials          0          0

Powered by Codecov. Last update bf06972...2d55a28

👍


recent_topics: Improve behaviour of inplace rerender.

This PR aims to fix both https://github.com/zulip/zulip/issues/23331 and https://github.com/zulip/zulip/issues/23332, along with behaviour described in https://chat.zulip.org/#narrow/stream/9-issues/topic/Recent.20topics.20Enter.
We used to hide and show topic rows in the DOM when topics are updated. This resulted in incorrect calculations in the length of visible topics. As a consequence, focus is sometimes set to hidden topics. Removing hidden topics from DOM helps us keep the calculations correct.
discussion: https://chat.zulip.org/#narrow/stream/9-issues/topic/Recent.20topics.20Enter
@m-e-l-u-h-a-n FYI

Marked it as draft since I noticed this has some side effects which needs to be addressed together with this change.

This should be ready for review! @alya FYI

Cool, thanks!
I thinks this solves #23331, but I'm still seeing the issue #23332:

I don't know how to reproduce the Enter behavior, so I haven't tried to test that part.

https://github.com/zulip/zulip/pull/23385#issuecomment-1297633552

Weird, I am not able to reproduce with this PR. Do you have a reproducer?

I don't know how to reproduce the Enter behavior, so I haven't tried to test that part.

I don't have a reproducer for it, but should be fixed since we removed the underlying cause of it in this PR.

#23332 is about muting a topic with a PM line below it, which is not what I see in your screen capture. Have you tested that behavior?

#23332 is about muting a topic with a PM line below it, which is not what I see in your screen capture. Have you tested that behavior?

Oh! Looks like I fixed a different bug above, then!

Sorry! Should be fixed now!

Playing with this PR, the behavior of muting a topic just above a PM line is different from what was specified in the issue:

Muting a topic (via keyboard) that has a PM row directly below makes the cursor disappear. Instead, we should use the same patten as for unread markers, and go to the topic on the next line if there is no mute icon to go to.

In this PR, the cursor instead goes to the unread marker on the PM line if there is one, rather than the topic name.

I don't have strong feelings on which behavior is better; with the current implementation, the intuitive pattern is "go down, and go left until there's a place for the cursor to go."

Otherwise looks good to me!

Merged, after a bit of work on the comments; I think this is enough of a usability improvement to be worth merging despite not being the cleanest code.
In particular, I think the accesses to offset are functionally correct, but would very much benefit from being abstracted as list_widget methods, like list_widget.remove_rendered_item() replacing the lines that do offset -=1; the current approach is an abstraction violation that is OK for now but will likely lead to pain eventually.
I added some TODO comments to that effect; it'd be great if you can try to follow-up on that.


adding Typescript + defining new components w/styled components

Overview

Initial config for TS components
New components with styled components

Codecov Report
Patch and project coverage have no change.

Comparison is base (19f2eb6) 52.24% compared to head (3c1600f) 52.24%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #702   +/-   ##
========================================
  Coverage    52.24%   52.24%
========================================
  Files          167      167
  Lines         7565     7565
  Branches      1536     1536
========================================
  Hits          3952     3952
  Misses        3613     3613

Flag
Coverage Δ

coverage
52.24% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


add open source license

most likely depends on https://github.com/aboe026/software-update-checker-cli/issues/16

implemented in #44


Problem with netcdftime?

I found #2322 and @djkirkham reported that the issue was fixed here (on April 3; Iris 1.13.0 was released on May 17), but I am seeing the same issue when using a time constraint using a clean Conda installation of Iris (following the instructions here):
% conda create --name iris-test -c conda-forge iris
[...]
    netcdf4:                       1.3.1-py27_0          conda-forge
[...]
% source activate iris-test
(iris-test) % python
Python 2.7.14 | packaged by conda-forge | (default, Oct  5 2017, 14:19:56)
[GCC 4.8.2 20140120 (Red Hat 4.8.2-15)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import iris
>>> print iris.__version__
1.13.0
>>> from iris.time import PartialDateTime
>>> iris.FUTURE.cell_datetime_objects = True
>>> filename = 'aq112a.p51851may.pp'
>>> cube = iris.load_cube(filename, iris.AttributeConstraint(STASH='m01s03i236') & iris.Constraint(time=lambda cell: PartialDateTime(1851, 5) <= cell.point <= PartialDateTime(1851, 6)))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/__init__.py", line 371, in load_cube
    cubes = _load_collection(uris, constraints, callback).merged().cubes()
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/__init__.py", line 311, in _load_collection
    result = iris.cube._CubeFilterCollection.from_cubes(cubes, constraints)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/cube.py", line 145, in from_cubes
    collection.add_cube(cube)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/cube.py", line 158, in add_cube
    pair.add(cube)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/cube.py", line 110, in add
    sub_cube = self.constraint.extract(cube)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/_constraints.py", line 151, in extract
    resultant_CIM = self._CIM_extract(cube)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/_constraints.py", line 214, in _CIM_extract
    self.rhs._CIM_extract(cube))
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/_constraints.py", line 175, in _CIM_extract
    resultant_CIM = resultant_CIM & coord_constraint.extract(cube)
  File "/data/local/ehogan/miniconda/envs/iris-test/lib/python2.7/site-packages/iris/_constraints.py", line 290, in extract
    r = np.array([call_func(cell) for cell in coord.cells()])
  File "<stdin>", line 1, in <lambda>
  File "netcdftime/_netcdftime.pyx", line 1307, in netcdftime._netcdftime.datetime.__richcmp__
TypeError: cannot compare netcdftime._netcdftime.Datetime360Day(1851, 5, 16, 0, 0, 0, 0, -1, 136) and PartialDateTime(year=1851, month=6)

Note that an installation of Iris using netcdf4=1.2.4 works fine:
% module load scitools/default-current
% python
Python 2.7.12 | Met Office - SciTools | (default, Sep 22 2016, 17:01:57)
[GCC 4.4.7 20120313 (Red Hat 4.4.7-3)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import iris
>>> print iris.__version__
1.13.0
>>> from iris.time import PartialDateTime
>>> iris.FUTURE.cell_datetime_objects = True
>>> filename = 'aq112a.p51851may.pp'
>>> cube = iris.load_cube(filename, iris.AttributeConstraint(STASH='m01s03i236') & iris.Constraint(time=lambda cell: PartialDateTime(1851, 5) <= cell.point <= PartialDateTime(1851, 6)))
>>> print cube
air_temperature / (K)               (latitude: 324; longitude: 432)
     Dimension coordinates:
          latitude                           x               -
          longitude                          -               x
     Scalar coordinates:
          forecast_period: 11880.0 hours, bound=(11520.0, 12240.0) hours
          forecast_reference_time: 1850-01-01 00:00:00
          height: 1.5 m
          time: 1851-05-16 00:00:00, bound=(1851-05-01 00:00:00, 1851-06-01 00:00:00)
     Attributes:
          STASH: m01s03i236
          source: Data from Met Office Unified Model
          um_version: 10.7
     Cell methods:
          mean: time (1 hour)

Is this expected?

Thanks for raising this issue.
There's a slight difference between this issue and https://github.com/SciTools/iris/issues/2322. In that issue the comparison was between a Cell and a PartialDateTime, whereas here the comparison is between a netcdftime.datetime (cell.point) and a PartialDateTime. In the former case we could work around the issue in the Cell comparison method, but we can't do that here.
The underlying issue was introduced into the netcdftime library some time after v1.2.4. Unfortunately there's nothing we can do until a fix is put into the library.
A workaround is to make sure PartialDateTime is always on the left hand side of the comparison, e.g.
PartialDateTime(1851, 5) <= cell.point and PartialDateTime(1851, 6) >= cell.point

Awesome, thank you very much! :)
Do you know if / when a fix will be added to the library?

I reported the issue some time ago, and although it's a simple fix nothing's been done. It's probably worth me putting up a fix myself, but we'd still have to wait for a release of the library, I don't know when that might be done.

Great, thank you for the information! :)


feat: Refactored Select

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma, Zeplin or any other documents that might be relevant to the PR

Add a TL;DR when description is extra long (helps content team)

Fixes # (issue)

if no issue exists, please create an issue and ask the maintainers about this first

Depends on # (pr)

every PR in the design-system repository should have a corresponding PR in the appsmith repository to ensure that changes
here don't break existing flows. Link the corresponding PR here to trigger dpulls and prevent accidental merging.
If relevant, link the enterprise PR here as well.

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual on storybook
Manual on main repo
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

Bugs in this PR:
###Select:
Functional:

open - can we make the name more appropriate? Also the default value mentioned is undefined but by default true is selected.
multiple - can we rename this to multiSelect & also change the description to something like: Support multiple selections.
Hover state is maintained even when mouse is not hovering over an option
If we switch off multiple, then selected items still have an active state even though the selection is removed.
Search doe not filter all the results: e.g. enter 1 in search & Option 1 will not show up.
x icon is controllable. can't we just show it by default once any search text is entered?
How to manipulated maxTextLength?
showSearch - can we modify the description to - Whether to enable search input. Why have we specifically mentioned Single mode? Also can we change the name to allowSearch?
When showSearch is enabled, we don't see the appropriate cursor on hovering over the search input.
After entering something in the search input when we click outside, the search text disappears.
filterOption - can we make the description a little simpler? Also change the first sentence to Whether to filter options by input value.
After selecting any option, the cursor appears in the beginning of the option & not the end.
Can we have syntaxes for onSearch, onBlur, onFocus, onPopupScroll, onSelect, onDeselect, onDeselect? And how to use them...
If we set defaultValue & value together & then remove value, the defaultValue does not appear.
What does defaultActiveFirstOption, autoFocus & virtual do? and can we make their desc more simpler to understand.
Need more clarity on autoClearSearchValue.
Can't add multiple values when multiple is enabled.
Can we rename the notContentFound control to something more appropriate?
No Label control.
No Error control.

Design:

Wrong disabled colour.
Hover & Focus state for border in Disabled state.
Wrong Length for small. Expected is 190px.
Wrong size of icons in sm state. Expected is 12px
Wrong gap between icon & text in small state. Expected is 4px.
Wrong left spacing between placeholder & border.
Wrong Placeholder colour.
Wrong gap between dropdown border & hover/active state of the options

Avatar:
Functional:

Not able to change the pic.
Label displays as a tooltip.
Entering any value in svgIconName throws error.
Can't add more members using the default state

Design:

Wrong active state. We don't see the blue focus border in this state
Why is there a white outline around the pics in avatar group.

Adding onto the callouts, some observations:
Avatar:

svgIconName: In the description, to has been mentioned as too

Select:

Text cursor is seen instead of an arrow/hand cursor when dropdown and multiple are set to false
On toggling dropdownMatchSelectWidth to TRUE/FALSE, there doesn’t seem to be any change in the width

open - can we make the name more appropriate? Also the default value mentioned is undefined but by default true is selected. -> We will keep it as open now to maintain consistency between components. Menu also uses open.
multiple - can we rename this to multiSelect & also change the description to something like: Support multiple selections. -> Fixed
Hover state is maintained even when mouse is not hovering over an option -> This is because defaultActiveFirstOption is by default true and it automatically keep first item active state
If we switch off multiple, then selected items still have an active state even though the selection is removed. -> This is because when there is a selected item and when you toggle multiselect it will take the previous select as selected of multiselect.
Search doe not filter all the results: e.g. enter 1 in search & Option 1 will not show up. -> Fixed. This was because value of Option 1 was added as test by me.
x icon is controllable. can't we just show it by default once any search text is entered? -> The x button is not to clear input but the selected value. Hence it is controllable
How to manipulated maxTextLength? -> Like I mentioned in description, it works if content is only text. For testing purpose I have added a simple select story with just text as options.
showSearch - can we modify the description to - Whether to enable search input. Why have we specifically mentioned Single mode? Also can we change the name to allowSearch? -> Changed the description. I guess we should not rename the flag since that was the one exposed from the library. So if any dev try to refer library documentation in any scenario might get confused if we keep different flags
When showSearch is enabled, we don't see the appropriate cursor on hovering over the search input. -> Something we can't control since the underlaying library is doing some calculation to apply the width of the input field. We may can dig deeper later on this.
After entering something in the search input when we click outside, the search text disappears. -> When the focus is out the Select will be reset to it's previous state. Something the library doing
filterOption - can we make the description a little simpler? Also change the first sentence to Whether to filter options by input value. -> Updated
After selecting any option, the cursor appears in the beginning of the option & not the end. -> Libarary is handling this behaviour. Also I guess if we keep the cursor in the end people might get confused by are they typing with the rest of the selected option
Can we have syntaxes for onSearch, onBlur, onFocus, onPopupScroll, onSelect, onDeselect, onDeselect? And how to use them... -> Added syntax as well as command logs
If we set defaultValue & value together & then remove value, the defaultValue does not appear. -> This is how it behaves. defaultValue shows only when the first time component loades.
What does defaultActiveFirstOption, autoFocus & virtual do? and can we make their desc more simpler to understand. -> defaultActiveFirstOption, for some reason it is not working with storybook. autoFocus description will make sense for developers, and this flag is to automatically bring focus to the component when it is loaded/mounted. virtual flag is used to enable virtual scrolling, this enables better performance if the list is too long.
Need more clarity on autoClearSearchValue. -> When you search for something and choose an option from list, it will clear the search automatically
Can't add multiple values when multiple is enabled. -> Can you retest this? I can select multiple
Can we rename the notContentFound control to something more appropriate? -> As mentioned above we should not rename library exposed variables.
19 & 20 we decided not to go with for now.

Issues that are not fixed:

If we switch off multiSelect, then selected items still have an Active state even though the selection is removed. This is still happening. Please have a look at the below Loom for better clarity:
https://www.loom.com/share/9451178cb46f4bbbbbc1c74c1f360662

maxTagTextLength - Can we add Works only when multiSelect is enabled to the description.

filterOption - can we make the description a little simpler? Also change the first sentence to Whether to filter options by input value.

autoClearSearchValue - When it is set to False, showSearch is set to False & multiSelect is enabled, we are able to search.
https://www.loom.com/share/f0133fccee3c40a6ae3f92a818ae5adc

Can't add multiple values in the value control when multiple is enabled.

notFoundContent - Can we change the description to Content to display when search fetches no matching results.

Please change the D of the default to upper case.

Design issues mentioned by @shadabbuchh

Wrong disabled colour. -> This is wrong in design
Hover & Focus state for border in Disabled state. -> Fixed
Wrong Length for small. Expected is 190px. -> Ignore width, it will adapt to the parent container
Wrong size of icons in sm state. Expected is 12px -> Select option content is defined by developers, component don't have an icon property
Wrong gap between icon & text in small state. Expected is 4px. -> Like above comment, option content is upto developers.
Wrong left spacing between placeholder & border. -> Reverify this once again, I see it is right.
Wrong Placeholder colour. -> Inspect the placeholder element and check the color. I see that is correct color.
Wrong gap between dropdown border & hover/active state of the options -> Will follow same styles as menu

@laveena-en
Avatar:

svgIconName: In the description, to has been mentioned as too -> Fixed

Select:

Text cursor is seen instead of an arrow/hand cursor when dropdown and multiple are set to false -> Fixed
On toggling dropdownMatchSelectWidth to TRUE/FALSE, there doesn’t seem to be any change in the width -> **This actually works. I or @shadabbuchh  can explain this. **

@albinAppsmith tested the changes that Vinay had asked for. Looks good to me.


[OSDEV-1084] Claims. Update logic for number of workers.

OSDEV-1084 Claims. Update logic for number of workers.
The implementation of this issue makes it possible to add a range for the number of workers during the claiming process, either after pressing the “I want to claim this production location” link or on the Claimed Facility Details page.
To change the type of the facility_workers_count field in the FacilityClaim table from IntegerField to CharField, there were added migrations with the next sequence:
* 0146_add_facility_workers_count_new_field_to_facilityclaim - adds the facility_workers_count_new field to the FacilityClaim model.
* 0147_copy_facility_workers_count_to_facility_workers_count_new - copies the data from the facility_workers_count field to the facility_workers_count_new field.
* 0148_remove_facility_workers_count_field_from_facilityclaim - removes the facility_workers_count field from the FacilityClaim model.
* 0149_rename_facility_workers_count_new_to_facility_workers_count - renames the facility_workers_count_new field to facility_workers_count.

Please fix merge conflicts

Please, resolve the conflicts and I will approve it

I don't see any test written on this new functionality. could you please cover facilities_view_set and facility_claim_view_set with tests

Can we make this issue as a tech dept?

I don't see any test written on this new functionality. could you please cover facilities_view_set and facility_claim_view_set with tests

Can we make this issue as a tech debt?

Yes, please. Create it, include into next sprint and post link to it here

@killev @VadimKovalenkoSNF Here is Tech Debt issue for adding test cases https://opensupplyhub.atlassian.net/browse/OSDEV-1095


Can pytube be combined with UI to solve captcha when encountered?

Hey, Juan
thanks for the nice project. I had been downloading a lot and what I have seen in every 1K batch only 100 videos get captcha and in every 10K videos 5K get captcha. I wanted to know if you can add features like
where I can choose best streaming server of YouTube (e.g., iOS, Android, Web etc) much like yt-dlp. And especially I wanted to know if it was possible pytubefix can be used in an UI where I can create a captcha solver so that I don't get captcha errors?

do something like this:
from pytubefix import YouTube

url = ''

yt = YouTube(url, client='WEB')
stream = yt.streams.get_highest_resolution()
stream.download()

As for the captcha I don't know

Hello @sleepingcat4, Maybe I can help you, we recently added support for the Proof of Origin Token (PoToken) to pytubefix #209. I don't know of any other captcha method that YouTube uses.
Would you mind providing more information about this captcha?

@felipeucelli I read a few PR of Pytubefix and it said most errors regarding

login to view
age restricted
Failed to connect tunnel detected
not detected

comes from a recaptcha error. I have been downloading at scale for days now using Pytube fix and what I learned this error gets more annoying as I increase the batch size.
For example in every 1K videos 100 videos receive this error but for every 10K videos 5K videos receive this error.
That's why, I was thinking if I could combine an Anti-recaptcha API with Pytubefix, it can automatically solve this verification issues and I can download it using terminal or use UI (but UI looks complicated)

Were you able to detect any parameter, token or something similar, so that we could implement it in the pytubefix API?
How does this Anti-recaptcha API work?

I was looking at this repo and some other paid services. These paid APIs look cheap, if you could have something native in Pytubefix (if paid still fine since those who badly need it can pay for it) then it will be fantastic.
https://github.com/anti-captcha/anticaptcha-python
I wasn't able to detect any token or parameter since the error was generic. But I will try to gather information next time.

I tested your problem for several days and what I noticed is that it seems that YouTube is blocking it when several requests are made in a row. Needing a short break to continue.
When using a fast internet connection, I was blocked after about 5 requests. But when using a slower connection, I had no problem.
This is bad, but for now I don't know a way to get around this block.
Note: my tests were done without using po_token and oauth.


Edit sermon backend

NOTE:
EDIT SERMON WILL NOT WORK UNTIL WE PAY FOR MAKE TO TURN THE API CONNECTOR ON:


Refine fssh: extract hostname from a variable number of arguments

This PR:

Enable fssh to extract the hostname from a variable number of arguments.
Remove entries from known_hosts using ssh-keygen (in place of sed)
Add a man page for fssh

@FineFan for awareness ^


Reduce memory footprint of data i/o: added a SentenceReader that stre…

…ams integer sequences from disk. Removed label arrays from ParallelBucketSentenceIter. Labels are created on the fly from target sequence.

I realized that this is only mxnet12 compatible due to multi-indexing support.
I will only merge this once we have mxnet-12 update in place. But reviewing can start before :)

@tdomhan Rebased onto latest master (mxnet 0.12) and checks now pass here

great change that should reduce the memory footprint considerably, as python lists are quite memory hungry. This should at least reduce the problem before we we move to some sharding/streaming solution.


when i authorise an App addin am i approving the app or just a specific version?

[Enter feedback here]
If I authorise an app to specific group of users - am i authorising the app as a whole or a specific version.
thus does each app version need approval or is the app ever green.

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

ID: 41508035-2045-1706-9917-cd2da1e99cc7
Version Independent ID: ea51ce32-18e2-68ed-ddc5-74f0d3d8a3d1
Content: Publish Office Add-ins using Centralized Deployment via the Microsoft 365 admin center - Office Add-ins
Content Source: docs/publish/centralized-deployment.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

It depends on what you mean by "version". You can change the files hosted in the server side, without changing the manifest. The manifest and Office don't know about that kind of version change, so the original approval still applies. However, if by version change, you mean a changed manifest, then that new manifest would have to be deployed and reapproved. Does this answer the question?


Error: goaway Frame with Invalid Size when trying gRPC Request

Symptoms

When trying to send requests to gRPC endpoints for the evmos blockchain (using https://grpc.bd.evmos.org:9090), I encountered with the following error.
connection error: hyper::Error(Http2, Error { kind: GoAway(b"", FRAME_SIZE_ERROR, Library) })
I wonder what is the root cause of the error and how to deal with the error. thanks.
The endpoint works well on the terminal

export NODE=grpc.bd.evmos.org:9090
grpcurl -d '{"'address'": "'$EVMOS_ADDR_3'"}' $NODE cosmos.bank.v1beta1.Query/AllBalances

{
  "balances": [
    {
      "denom": "aevmos",
      "amount": "19254711178107386863"
    },
    {
      "denom": "ibc/ED07A3391A112B175915CD8FAF43A2DA8E4790EDE12566649D0C2F97716B8518",
      "amount": "63743"
    }
  ],
  "pagination": {
    "total": "2"
  }
}

Error Stack
/Users/sigridjin.eth/Documents/github/ParachuteDrop-LightClient/target/debug/parachutedrop-rust-server
[2022-12-26T03:01:44Z INFO  actix_server::builder] Starting 10 workers
[2022-12-26T03:01:44Z INFO  actix_server::server] Actix runtime found; starting in Actix runtime
[2022-12-26T03:01:51Z DEBUG hyper::client::connect::dns] resolving host="grpc.bd.evmos.org"
[2022-12-26T03:01:51Z DEBUG hyper::client::connect::http] connecting to <IP_ADDRESS>:9090
[2022-12-26T03:01:51Z DEBUG hyper::client::connect::http] connected to <IP_ADDRESS>:9090
[2022-12-26T03:01:51Z DEBUG h2::client] binding client connection
[2022-12-26T03:01:51Z DEBUG h2::client] client connection bound
[2022-12-26T03:01:51Z DEBUG h2::codec::framed_write] send frame=Settings { flags: (0x0), enable_push: 0, initial_window_size: 2097152, max_frame_size: 16384 }
[2022-12-26T03:01:51Z DEBUG h2::proto::connection] Connection; peer=Client
[2022-12-26T03:01:51Z DEBUG h2::codec::framed_write] send frame=WindowUpdate { stream_id: StreamId(0), size_increment: 5177345 }
[2022-12-26T03:01:59Z DEBUG tower::buffer::worker] service.ready=true message=processing request
[2022-12-26T03:01:59Z DEBUG h2::codec::framed_write] send frame=Headers { stream_id: StreamId(1), flags: (0x4: END_HEADERS) }
[2022-12-26T03:01:59Z DEBUG h2::codec::framed_write] send frame=Data { stream_id: StreamId(1) }
[2022-12-26T03:01:59Z DEBUG h2::codec::framed_write] send frame=Data { stream_id: StreamId(1), flags: (0x1: END_STREAM) }
[2022-12-26T03:02:03Z DEBUG h2::proto::connection] Connection::poll; connection error error=GoAway(b"", FRAME_SIZE_ERROR, Library)
[2022-12-26T03:02:03Z DEBUG h2::codec::framed_write] send frame=GoAway { error_code: FRAME_SIZE_ERROR, last_stream_id: StreamId(0) }
[2022-12-26T03:02:03Z DEBUG h2::proto::connection] Connection::poll; connection error error=GoAway(b"", FRAME_SIZE_ERROR, Library)
[2022-12-26T03:02:03Z DEBUG hyper::proto::h2::client] connection error: broken pipe
[2022-12-26T03:02:03Z DEBUG hyper::client::service] connection error: hyper::Error(Http2, Error { kind: GoAway(b"", FRAME_SIZE_ERROR, Library) })
[2022-12-26T03:02:03Z DEBUG hyper::proto::h2::client] client response error: connection error detected: frame with invalid size

Code
#[get("/evmos")]
pub async fn evmos_health() -> Result<HttpResponse, HTTPError> {
    // grpcurl -d '{"'address'": "'$EVMOS_ADDR_3'"}' $NODE cosmos.bank.v1beta1.Query/AllBalances
    let endpoint = tonic::transport::Endpoint::new("https://grpc.bd.evmos.org:9090"
            .parse::<tonic::transport::Uri>().unwrap())
            .unwrap();
    let channel = endpoint.connect().await;
    let mut connected_client = QueryClient::new(channel.unwrap());
    let request = QueryAllBalancesRequest {
        address: "evmos1f57x5pm9wpvlu4qldy864j0hdnq025287r65z2".to_string(),
        pagination: Option::from(PageRequest {
            key: Vec::from("".to_string()),
            offset: 0,
            limit: 0,
            count_total: false,
            reverse: false
        }),
    };
    let response = connected_client.all_balances(request).await;

    // let response = connected_client.all_balances(request).await;
    // response.unwrap();
    Ok(HttpResponse::Ok().json(HealthResponse {
        status: StatusCode::OK.as_u16(),
        message: "OK".to_string(),
        data: None,
    }))
}

Hi there!
This is an error I'm assuming is arising in the generated code of cosmos-sdk-proto? (perhaps cosmos_sdk_proto::cosmos::bank::v1beta1::QueryAllBalancesRequest?)
Whatever the cause is, it's something out of our control, arising in the low-level HTTP/2 client.
Either there's some error with https://grpc.bd.evmos.org:9090 or a bug in Tonic, the gRPC library we use.
I would suggest trying to file this issue against Tonic and see if they might be able to assist: https://github.com/hyperium/tonic

@sigridjineth Did you find a solution for this issue? I am running into the same error with a very similar setup.

Perhaps file an upstream error against Tonic? https://github.com/hyperium/tonic


STANDALONE: faer isn't in the prelude

Part of quick PRs to just adjust some things we have not noticed.

The consequence of this, if one chooses to enable both faer and ndarray is you get this warning:

warning: ambiguous glob re-exports
  --> /Users/elea/Documents/GitHub/extendr-main/extendr-api/src/prelude.rs:73:9
   |
73 | pub use ::ndarray::*;
   |         ^^^^^^^^^^^^ the name `prelude` in the type namespace is first re-exported here
...
79 | pub use ::faer::*;
   |         --------- but the name `prelude` in the type namespace is also re-exported here
   |
   = note: `#[warn(ambiguous_glob_reexports)]` on by default

warning: ambiguous glob re-exports
  --> /Users/elea/Documents/GitHub/extendr-main/extendr-api/src/prelude.rs:73:9
   |
73 | pub use ::ndarray::*;
   |         ^^^^^^^^^^^^ the name `linalg` in the type namespace is first re-exported here
...
79 | pub use ::faer::*;
   |         --------- but the name `linalg` in the type namespace is also re-exported here


warn: Action middleware (POLICY: passport) was bound to a target auth/* that doesn't match any registered actions.

Hi,
Created brand new sails project and sails-auth-new --s and sails lift.
The warning messages are below. What can we do?
Regards
gab
======================================
debug: marlinspike (auth): loading Models...
debug: marlinspike (auth): loading Controllers...
debug: marlinspike (auth): loading Policies...
warn: Action middleware (POLICY: passport) was bound to a target auth/* that doesn't match any registered actions.
warn: Ignored attempt to bind route (/register) to unknown action :: { controller: 'UserController',
action: 'create',
skipAssets: true }
warn: Ignored attempt to bind route (/logout) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'logout' }
warn: Ignored attempt to bind route (/auth/local) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }
warn: Ignored attempt to bind route (/auth/local/:action) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }
warn: Ignored attempt to bind route (/auth/:provider) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }
warn: Ignored attempt to bind route (/auth/:provider) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'provider' }
warn: Ignored attempt to bind route (/auth/:provider/callback) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }
warn: Ignored attempt to bind route (/auth/:provider/:action) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }
warn: Ignored attempt to bind route (/auth/:provider/:action) to unknown action :: { controller: 'AuthController',
skipAssets: true,
action: 'callback' }

Any solution?

You need to generate the controller again.
Just rename your existing controller with another name and register the controller again using below command.
sails generate controller ControllerName
Example
sails generate controller User
it will create UserContoller.js
Thanks and Regards
Arun Kamboj


Test mode for methods

Long running methods (e.g. scvi) take a long time to test. In v1 we allow a test parameter to e.g. limit the number of iterations / training epochs / etc. Is there an equivalent in v2?

Not yet. Since the example dataset is downsampled, all components currently in v2 can get unit tested in less than 15m (example, which includes tools like scvi. Hence I didn't put effort in implementing this functionality.
If this ever changes, I'm happy to add in this functionality. As long as it isn't, I think less is more.

In my experience from v1, some methods take a very long time to test, even with the test dataset (which was 500 cells). I'm reopening and labelling as wontfix for now; if the test runtime changes as we complete the migration we can add back to the milestone.

This issue currently doesn't present itself in the v2. Let's reopen it when there is a component that runs for too long. IMO it doesn't make sense to keep this open if the need for this functionality is not there.


Add support for namespaces

Description
Changes proposed in this pull request:

support for installing gcp-broker as a ServiceBroker in a given namespace.
"remove-gcp-broker" has two additional parameters: SkipDeprecated and SkipGCPIntegration

Related issue(s)
Resolves #218

I signed the CLA


can use number for version????????????

please......

For what? Please take the time to describe in detail your issue, what versions you're using and how to recreate it.

啥意思？你中文说，我帮你问:wink::wink::wink:


opencv.h

Errors     << lego_loam:make /home/ashwani/catkin_ws/logs/lego_loam/build.make.001.log
In file included from /home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/src/mapOptmization.cpp:34:
/home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/include/utility.h:13:10: fatal error: opencv/cv.h: No such file or directory
13 | #include <opencv/cv.h>
|          ^~~~~~~~~~~~~
compilation terminated.
In file included from /home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/src/imageProjection.cpp:35:
/home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/include/utility.h:13:10: fatal error: opencv/cv.h: No such file or directory
13 | #include <opencv/cv.h>
|          ^~~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/imageProjection.dir/build.make:63: CMakeFiles/imageProjection.dir/src/imageProjection.cpp.o] Error 1
make[2]: *** [CMakeFiles/mapOptmization.dir/build.make:63: CMakeFiles/mapOptmization.dir/src/mapOptmization.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:458: CMakeFiles/mapOptmization.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
make[1]: *** [CMakeFiles/Makefile2:323: CMakeFiles/imageProjection.dir/all] Error 2
In file included from /home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/src/featureAssociation.cpp:35:
/home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/include/utility.h:13:10: fatal error: opencv/cv.h: No such file or directory
13 | #include <opencv/cv.h>
|          ^~~~~~~~~~~~~
compilation terminated.
In file included from /home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/src/transformFusion.cpp:33:
/home/ashwani/catkin_ws/src/LeGO-LOAM/LeGO-LOAM/include/utility.h:13:10: fatal error: opencv/cv.h: No such file or directory
13 | #include <opencv/cv.h>
|          ^~~~~~~~~~~~~
compilation terminated.
make[2]: *** [CMakeFiles/featureAssociation.dir/build.make:63: CMakeFiles/featureAssociation.dir/src/featureAssociation.cpp.o] Error 1
make[2]: *** [CMakeFiles/transformFusion.dir/build.make:63: CMakeFiles/transformFusion.dir/src/transformFusion.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1383: CMakeFiles/featureAssociation.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:1160: CMakeFiles/transformFusion.dir/all] Error 2
make: *** [Makefile:141: all] Error 2
I am trying to run this on ros noetic, but it is showing the above error?????
Another thing I want to ask is that will it work on ros noetic or do I have to make some changes????

The problem may be the changes in OpenCV library the complete library was distributed into submodules.

If you detected OpenCV and configured your make scripts using CMake or pkg-config tool, your code will likely build fine without any changes. Otherwise, you will need to modify linker parameters (change the library names) and update the include paths.

It is still possible to use #include <cv.h> etc. but the recommended notation is:

#include "opencv2/imgproc/imgproc.hpp"

If anyone still has this issue, there is a fork for OpenCV4 here; https://github.com/Self-SLAM-Lab/LeGO-LOAM_opencv4


Haproxy Request Not In Inventory

BUG REPORT INFO
Issue:
I am trying to use the VMware configuration and everything goes according to plan until it suddenly asks for a hostname that was not included in the inventory. Is there a missing variable in the ocp-on-vmware.ini for master? And why is it suddenly asking for this name and ignoring the requested domain and wanting master.host.com? Here is my inventory:
Here is what should go into your DNS records
$ORIGIN apps.ose.host.com.

   Make sure this host can be reached over ssh
Anything else do we need to know:
18-03-19 18:48:29,854 p=3700 u=root |  PLAY [Configure ha proxy server files] ****************************************************************************************************************************************
2018-03-19 18:48:29,857 p=3700 u=root |  PLAY RECAP ********************************************************************************************************************************************************************
2018-03-19 18:48:29,857 p=3700 u=root |  localhost                  : ok=9    changed=8    unreachable=0    failed=0
2018-03-19 18:48:29,857 p=3700 u=root |  master                     : ok=0    changed=0    unreachable=1    failed=0
2018-03-19 18:48:29,857 p=3700 u=root |  nodehaproxy-0              : ok=21   changed=11   unreachable=0    failed=1
2018-03-19 18:48:29,857 p=3700 u=root |  nodeinfra-0                : ok=2    changed=0    unreachable=0    failed=0
2018-03-19 18:48:29,858 p=3700 u=root |  nodeinfra-1                : ok=2    changed=0    unreachable=0    failed=0
2018-03-19 18:48:29,858 p=3700 u=root |  nodeinfra-2                : ok=2    changed=0    unreachable=0    failed=0

ocp-on-vmware.ini 2.zip


Function parameter info attempts to show return type

plugin version:
V0.68
Description
function parameter info (cmd +p) on mac tries to show return type and show the incorrect type. (At least that's what I think it's trying to show).
eg below the parameter info tries to show a polymorphic variable as the return type (where it's unit).
(it should really just show parameter info)

eg here's a python example

or swift

return type removed in 0.72


Fix crash in - [SDWebImageDownloaderOperation connectionDidFinishLoading:], line 376

Version: 3.0.2 (2<IP_ADDRESS>) | net.artsy.artsy
Stacktrace
SDWebImageDownloaderOperation;connectionDidFinishLoading:;SDWebImageDownloaderOperation.m;376
SDWebImageDownloaderOperation;start;SDWebImageDownloaderOperation.m;117
Link to HockeyApp

https://rink.hockeyapp.net/manage/apps/37029/crash_reasons/146712659

Possibly related #2046


[Bug]: Cannot connect to MongoDB Atlas (cloud offering) using mongodb client

Connector Name
module/mongodb (Mongo DB)
What happened?
I'm trying to connect to a mongodb atlas database using the mongodb client. First I tried with sslEnabled: false as per the examples. Then I got  a connection prematurely closed error when client is initialized. This is probably because the connection is not secure.
error: Timed out after 30000 ms while waiting to connect. Client view of cluster state is {type=UNKNOWN, servers=[{address=cluster0-shard-00-01.1zwpo.mongodb.net:27017, type=UNKNOWN, state=CONNECTING, exception={com.mongodb.MongoSocketReadException: Prematurely reached end of stream}}] {"mongoDBExceptionType":"MongoTimeoutException"}
        at ballerinax.mongodb.3.Client:find(client.bal:131)
           imesha.backend.0.$anonType$_0:$get$customers(main.bal:58)

Next, tried with sslEnabled: true but then the secureSocket is required to be provided. Both trust store and key store are mandatory there. Also the protocol field is not documented properly.
Relevant log output
See above section

This is fixed with MongoDB 5.0.0


oasis-align:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: Cleanly place content side by side with equal heights using automatic content sizing.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

[x] ensured that my package is licensed such that users can use and distribute the contents of its template directory without restriction, after modifying them through normal use.

Thanks!


Phone Auth with Twilio

So I've been able to setup phone auth with OTP using twilio but I've found out that signin needs to be hit twice before the sms is sent. Once the user is created under Authentication->Users on the first hit there is no OTP sms sent but the second followup hit then sends the sms.
I've tried phone signin with Flutter's supabase library and also using the http request, both require two hits to send the OTP.
Can be reproduced using this tutorial and then creating an http request:
https://supabase.com/docs/guides/auth/auth-twilio#using-otp-as-a-passwordless-sign-in-mechanism

hey @solobits-nelson, are you still facing this issue? I'm unable to reproduce it currently but if it's still affecting you, feel free to email<EMAIL_ADDRESS>with your project reference so i can take a look.


Fix argument buffer padding on shader conversion.

Fixes issues #2216 and #2180.

For the record, the SPIRV-Cross change looks good, too.

Dunno why you asked for my review here, and not on the SPIRV-Cross change, since all this does is pull in an updated version of SPIRV-Cross... but, whatever.

Normally I would.
With the SDK release coming up, it was a bit of a rush to get everything sorted out. So to expedite, I thought I'd get the SPRIV-Cross MoltenVK branch set up, and populated here, so they could be reviewed together. If it didn't work out, I could then just delete the SPIRV-Cross branch.
All part of my master plan! 😂

I've updated this to include the fix for the atomic issue in #2216, and to use the new commit hash from the main branch in SPIRV-Cross.


[JOSS Review] Fix typos

While going through my JOSS Review in https://github.com/openjournals/joss-reviews/issues/5670, I found several typos that I thought would be easiest to put up in a PR.

Thanks for this, I really appreciate the effort. The typos are my kryptonite. I am really terrible, thanks!
I agree with the low-caps also.


Slowdown problem when recording for long periods of time

Hello
After running ez-sound-capture, I got this problem after recording for 24 hours.
I played a YouTube video for 24 hours, but the recording time was only 6 hours.
I also checked the memory through the task manager and found that it was taking up a significant percentage.
I ran the source code as is to directly observe this phenomenon.
How can I solve this problem so that I can record for a long time without any problems?

Hello, and thank you for reporting this issue!
It seems that the recording process gradually slows down because all the recorded audio data is continuously stored in memory. To achieve stable recording, I think it will be necessary to improve memory handling, such as writing data to disk in fixed-sized chunks while recording.
In the meantime, please try the following:
There is a process at L.244 that backs up all audio data stored in memory every SETTINGS.recording.backup_interval (default: 60 secs). Try commenting out this line.
# threading.Thread(target=backup_data_every, args=(SETTINGS.recording.backup_interval, backup_dir), daemon=True).start()

However, please be aware that disabling this backup means the data will be lost if the Python program encounters an issue or is forcibly terminated.
I apologize for the limitations of this simple implementation. Please note that recording for extended durations, such as over an hour, may result in degraded performance or quality.

I tried the method you suggested, but the problem was not solved.
However, when I compared the recorded audio file with the YouTube video, the beginning and the end sounded the same.
I felt it was necessary to find out why this was happening.
For example, the recorded frame(?) was compressed.


Terminal Plugin

A plugin wrapper for siac to bring command line support to the UI . Features include command history, integrated help menu (type 'help' or '?' to toggle it), and password prompts. This plugin is made in a similar style to the new wallet plugin #226. All code is written using the redux-react framework. A siac binary is required, and on first launch the plugin will prompt the user to provide the path to the binary and store this location in the config file for future use. Plugin screenshots are below.

needs updated .eslintignore to enable eslint on this plugin

Commit messages should be in the imperative; Add instead of Adding, Fix instead of Fixed, etc.

Also, prefer const to let, and never use var.  Always use let.  Might add a linting rule for this


CheckForUpdates and Dependabot

Hi @freddydk ,
a quick question about the "CheckForUpdates" step. Is it planned or generally possible to implement/trigger this functionality via Dependabot?
Thank you!

Haven't looked into dependabot.
It is possible to run Update AL-Go System files on a schedule, but I have Dependabit on my list of things to look into

CheckForUpdates can be automated by adding a schedule to .github/AL-Go-Settings.json, like:
  "UpdateGitHubGoSystemFilesSchedule": "0 0 * * *"

This will however use direct COMMIT (which might not be allowed).
I will add an item to the backlog for allowing Update AL-Go System Files to be able to create Pull Requests and then allow dependabot to approve PRs automatically when created from Update AL-Go System Files.

Transferred to project backlog
https://github.com/orgs/microsoft/projects/521/views/1?filterQuery=update+&pane=issue&itemId=44366441


Error when executing sample call

Steps to reproduce
node index.js
error: Error: Unexpected error determining execution environment: Invalid response from metadata service: incorrect Metadata-Flavor header.
at metadataAccessor (/Users/macuser/visionAPI/node_modules/gcp-metadata/build/src/index.js:60:19)
at process._tickCallback (internal/process/next_tick.js:68:7)
Making sure to follow these steps will guarantee the quickest resolution possible.
Thanks!

I am running the example as shown in this video:
https://www.youtube.com/watch?v=S9hiRUOGN4Y&t=458s
following the steps here to authenticate:
https://github.com/googleapis/nodejs-vision
Does not work with Curl either
On Thu, Oct 10, 2019 at 10:10 AM Stephen<EMAIL_ADDRESS>wrote:

Hello! How are you authenticating? And could you show the code you're
running?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googleapis/nodejs-vision/issues/461?email_source=notifications&email_token=ABWIMSNU2M7WKNLBP3EMHZTQN5AVXA5CNFSM4I7I6332YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEA4WPZI#issuecomment-540633061,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWIMSPVPSLMOH37MJWXN53QN5AVXANCNFSM4I7I633Q
.

--
See, I'm not a monster. I'm just ahead of the curve

It would be best if you could copy and paste the code you're using here :) As well as the versions you're running (Node, npm, and @google-cloud/vision). Here is some other info and the template you can use: https://raw.githubusercontent.com/googleapis/nodejs-vision/master/.github/ISSUE_TEMPLATE/bug_report.md

I'm closing this out until we get more info. @inductiveStep - Please feel free to reopen.


fix: Mono.CecilX namespace to work around Unity 2019 Cecil namespace collision

done

You need to rebuild it with he correct name,  just renaming the dlls is not enough, I get this:
Assembly 'Mono.Cecil.Rocks' has non matching file name: 'Mono.CecilX.Rocks.dll'. This can cause build issues on some platforms.
UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)

:tada: This PR is included in version 3.6.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


openshift incompatibility -connection refused

I can can't connect where i can do via oc and kubectl
kubectl get services -n project            -> works
sudo kubefwd services -l -n project    -> dial tcp xxx.xxxx.xxxx: connect: connection refused
Maybe it's also a http proxy issue ?

Any more information on this would be helpful. I don't use Open Shift.

The issue is with your $KUBECONFIG. You are most likely using a custom Kube Config location and not the default $HOME/.kube/config
You have to use the -E option with sudo in order for your environment variables to persist

   -E, --preserve-env
              Indicates to the security policy that the user wishes to preserve their existing environment variables.  The security policy may return an error if the user does not have permission to
              preserve the environment.

Thanks @latchmihay I'm going to close this issue. @cforce if you continue to have trouble connecting please create a new issue with more detail.


Include the documentation on the website on right hand side

include the content in this link : https://rawgit.com/allenhwkim/angularjs-google-maps/master/build/docs/index.html on the right hand side of the website http://ngmap.github.io/, before all the examples. Right now it's included in the index page, but seeing the examples on one site but the docs on another site make the reading is rather difficult.

ngMap.github.io is mobile friendly, but documentation is not.
So if we move the navigation to the left, and  you see documentation in mobile browser, you will see the menu only.
This was intentional and it needs to be improved as a mobile friendly version. Simply moving the navigation is not a good solution, i think.

Closing


Fix metadata reading during test import

Looks like the run dict exposes the "metadata" property instead of data. Fixing that to properly import tests results.
Fixes: https://github.com/ibutsu/ibutsu-server/issues/291

Ibutsu uses pre-commit for enforcing black and other linting rules. Please run pre-commit run --all against your code to fix the failing lint action. Thank you!