Feature request: Alternative Edge arrow style

Arrow styled edge represents how data goes from source node to sink node. It would be nice to draw an edge with different width on both side. Currently I configure edge.width=3. I expect to configure edge.width.from=5, edge.width.to=1.
A not exactly example is http://www.nytimes.com/interactive/2012/11/02/us/politics/paths-to-the-white-house.html?_r=0

Sorry, posted to wrong project. https://github.com/almende/vis/issues/709


React Pathway added

conflict resolved!

Hello @genialkartik!
As coruses are no longer kept in the latest of react.dev nor maintained in the page you modify in this PR, I will close it as outdated.
Thank you!


Fix select field with statement

Not working with query like this:
query.select("COALESCE(SUM(IF(wallet.`fund_amount` > 0 AND wallet.`action` != 'refund', wallet.`fund_amount`, 0)),0)", "fund_in");
So put ! and ' into resolve field name regex.

I am reviewing this PR. Thanks @khanifikhsanudin


SelectionSort

contributing (HacktoberFest2021)
kindly accept the request :)

We have stopped accepting DSA PRs, please try something new (any kind of cool code snippets or fun projects )


docs(website): Move company logos into tabs categorized by industry

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

One minor comment - What's the diff between E-Commerce and B2C? These categories don't seem mutually exclusive.. Is that intentional?

\

One minor comment - What's the diff between E-Commerce and B2C? These categories don't seem mutually exclusive.. Is that intentional?

I'd prefer just E-commerce. It's more standard. Still doesn't seem quite right.. Since Peloton could be considered ecommerce. Maybe not time to slice these upp..


Update “hack” to catch Duration.Inf.toMillils now throwing IllegalArgumentException.

Using any version of Play 2.6.X the default value of play.http.session.maxAge of null is being converted to Duration.Inf. https://github.com/playframework/playframework/commit/84c883d3865266da10706eddd71026821912000c#diff-4378e6503263d81d7869f6056b31e6b1L1037
private[this] def fail(what: String) = throw new IllegalArgumentException(s"$what not allowed on infinite Durations")
    final def length: Long    = fail("length")
    final def unit: TimeUnit  = fail("unit")
    final def toNanos: Long   = fail("toNanos")
    final def toMicros: Long  = fail("toMicros")
    final def toMillis: Long  = fail("toMillis")
    final def toSeconds: Long = fail("toSeconds")
    final def toMinutes: Long = fail("toMinutes")
    final def toHours: Long   = fail("toHours")
    final def toDays: Long    = fail("toDays")

IllegalArgumentException is now being thrown where ConfigException.Null may have been thrown in the past.
Another route to go instead of using this hack could be to use getOptional[Long] instead of getMilliseconds since it uses getOptional[Duration].map(_.toMillis). This would prevent the use of a Duration in the maxAge field when using this library, but may be less brittle going forward.
I apologize if this PR isn't fully complete. This is my first open source PR, and can definitely make changes to documentation that I may have missed, or make changes to version numbers and resubmit.

Hey, thanks for the work. Not sure about your code, we do not want to swallow every possible IllegalArgumentException that could happen for any other reason. Could you try to comment your code and setup a failing test so I can easily reproduce? Already existing tests are here: https://github.com/pauldijou/jwt-scala/tree/master/play/src/test/scala
When done, try using getMillis here since getMilliseconds is now deprecated : https://github.com/pauldijou/jwt-scala/blob/master/play/src/main/scala/JwtSession.scala#L102.
If not solving it, I will give it a closer look.

Sorry I didn't include a test. They all ran fine until I got: [error] (playLegacy/test:testGrouping) java.lang.NoClassDefFoundError: $5a39c98268d8f521595f$$anonfun$groupPlayTest$1
and never could manage to get beyond that.
Since the Play Framework is actually trying to gather a FiniteDuration, instead of just a Duration (that is returning InfiniteDuration, hence the error), how about just converting to getOptional[Duration]?
https://github.com/playframework/playframework/blob/2.6.x/framework/src/play/src/main/scala/play/api/http/HttpConfiguration.scala#L206
The code would then just become:
key => Play.maybeApplication.flatMap(.configuration.getOptionalFiniteDuration.map(.toMillis))
and the catching IllegalArgumentException could be removed. That's probably much cleaner. I'll resubmit that, and keep attempting to run the tests so I can add checks in that way too.

@pauldijou I think these tests should validate my findings, I was finally able to get them running.

Thanks a lot for failing test. I finally decided to go with checking if the key is present or not. Is that ok with you? https://github.com/pauldijou/jwt-scala/commit/1d466abc6e903a0d724465a6259e75892f2f4b04

No problem! Sure, that will definitely prevent any errors from being thrown if the value is null. Thanks for taking the time to review and merge my PR, and thanks for your libraries!

Cool. Should be released in 0.14.1


added error-handling

There was no error handling for wget command and so, regardless of its exit status, the rest of the script would still run. Some lines were added to make sure the script exits when the download fails.

nice script bro for automation. Keep up!

super late merge but done and dusted at least


Expressions can't be concatenated

Expressions in /defs cannot be concatenated.
/def /foo x:i
    /post bla {$x + 1}

/def /bar x:i
    /post bla{$x + 1}

/def /zee x:i
    /post {$x}{$x + 1}

/foo 0  # >>> bla 1
/bar 0  # >>> 1 - should be bla1
/zee 0 # >>> 1 - should be 01

This happens because in https://github.com/loopier/animatron/blob/d0ad48d9c7c84c29fc12d90b0b183695e410b391/Main.gd#L243 the arg[i] is replaced by the expression result. We probably need to create a new RegEx to get all the expressions in the current argument (arg[i]) and iterate over the expressions to get their results; then replace the original expressions with the result of each one.
This would be very useful in /def files, because we would be able to store algorithmic defs.

Do you have an example where this would be needed (or helpful) that doesn't use /post? ...just to have a sense of how/what this would be needed? What about if quotes were required to indicated the whole thing corresponded to a single OSC argument?
The issue isn't really the line you mention above,  but the design would have to be rethought with the knowledge that you could have multiple expressions per argument (this wasn't a requirement before). Maybe ocl._getExpression() should optionally return a list of expressions, or...?
When I designed the expression stuff, I was thinking of it more in terms of "producing a value" more than "producing a string"...it's a different thing. We can definitely figure something out, if this is important.

Here's a possible example. If it was concatenatable we could move even circles, leaving odd circles in place:
/for i 8 /create b$i circle
/size b* 0.1
/for i 8 /y b$i {$i * 1080 / 9 + 100}
/for i 8 /move/x b{$i % 2} 100

You are rignt, it wasn't a specification, and it's not the line but the desing. I just pinpointed the exact lne as a reminder to myself, because that's where the substitution happens.
If you have an idea on how to achieve this kind of things without having to refactor, I'd be totally fine with it. I just can't see it. Any ideas?

sounds great! Should I do it or will you?

I changed the title because this is a general issue, not specific to /def.


How to compute GridFunction and Coefficient at arbitrary point?

Hi all,
I want to compute a GridFunction and a Coefficient at an arbitrary point. I have checked this and this. But my situation is a little different.
For any given edge (3D mesh), I get the midpoint of the edge and I want to compute the values of the given GridFunction and Coefficient.
Through reading the source code, it seems like I should use ElementTransformation and IntegrationRule to compute the values. Is this correct? Or are there some simple ways?

Hi @fanronghong,
If you know the position of the point you need, you can create an IntegrationPoint (by setting the reference space coordinates). Then you can use GridFunction::GetValue() to evaluate the GridFunction or ElementTransformation::SetIntPoint() to call Coefficient::Eval(). You don't need an IntegrationRule to do the evaluation.

Hi @vladotomov ,
Thanks for your reply. I tried your suggestion but something is not right. How do I fix the below MWC.
#include <iostream>
#include "mfem.hpp"
using namespace mfem;
using namespace std;

double func(Vector& x)
{
    return x[0] + x[1];
}

int main()
{
    Mesh mesh(10, 10, Element::TRIANGLE, true, 1.0, 1.0);

    int p_order = 1;
    H1_FECollection h1_fec(p_order, mesh.Dimension());
    FiniteElementSpace h1_space(&mesh, &h1_fec);

    GridFunction gf(&h1_space);
    FunctionCoefficient coeff(func);
    gf.ProjectCoefficient(coeff);

    IntegrationPoint ip;
    ip.Set2w(0.3, 0.7, 1.0);

    ElementTransformation* tran;
    tran->SetIntPoint(&ip);

    // Below 2 ways throw segmentation errors.
    cout << gf.GetValue(1, ip) << endl; // arbitrary element?
    cout << coeff.Eval(*tran, ip) << endl;
}

I think you must initialize the ElementTransformation before using it, see Mesh::GetElementTransformation().

And keep in mind that the coordinates of the IntegrationPoint are assumed to be in reference space.

Hi @vladotomov ,
Yes, I know that way: find the Element which the physical point belongs to, then find the IntegrationPoint in reference space corresponding to the physical point, finally use Eval() to compute the value.
But here, I don't know which Element the physical point belongs to. I don't know the IntegrationPoint, and I only know the physical point.

Hi @vladotomov ,
I miss some details in reference links. Now the code works well. I put the complete code here.
#include <iostream>
#include "mfem.hpp"
using namespace mfem;
using namespace std;

double func(Vector& x)
{
    return x[0] + x[1];
}

int main()
{
    Mesh mesh(10, 10, Element::TRIANGLE, true, 1.0, 1.0);

    int p_order = 1;
    H1_FECollection h1_fec(p_order, mesh.Dimension());
    FiniteElementSpace h1_space(&mesh, &h1_fec);

    GridFunction gf(&h1_space);
    FunctionCoefficient coeff(func);
    gf.ProjectCoefficient(coeff);

    Vector phy_point(2);
    phy_point(0) = 0.15;
    phy_point(1) = 0.25;
    phy_point.Print(cout << "physical point: ");

    IntegrationPoint ip;
    int elem_idx;
    ElementTransformation* tran;
    for (int i=0; i<mesh.GetNE(); ++i)
    {
        tran = mesh.GetElementTransformation(i);
        InverseElementTransformation invtran(tran);
        int ret = invtran.Transform(phy_point, ip);
        if (ret == 0)
        {
            elem_idx = i;
            break;
        }
    }

    cout << elem_idx << "-th element\n"
         << "reference point: " << ip.x << ", " << ip.y << endl;

    DenseMatrix phy_point_mat(2, 1);
    phy_point_mat(0, 0) = 0.15;
    phy_point_mat(1, 0) = 0.25;
    Array<int> elem_ids;
    Array<IntegrationPoint> ips;
    mesh.FindPoints(phy_point_mat, elem_ids, ips);

    elem_ids.Print(cout << "element ids: ");

    for (int i=0; i<ips.Size(); ++i)
    {
        cout << i << "-th integration point: " << ips[i].x << ", " << ips[i].y << endl;
    }

    return 0;
}


Spyder 3.0.1 failed to start, Could not find Qt platform plugin "xcb"

Description of your problem
After a fresh installation of miniconda, and the installation of Spyder, Spyder 3.0.1 fails to start from the console with the error message

This application failed to start because it could not find or load the Qt platform plugin "xcb"
in .
Available platform plugins are: xcb (from /home/beberhardt/miniconda3/pkgs/qt-5.6.0-1/plugins/platforms), minimal, offscreen, xcb.
Reinstalling the application may fix this problem.
[1]    9578 abort (core dumped)  spyder

Please provide any additional information below
Searching for "xcb" I found two closed issues https://github.com/spyder-ide/spyder/search?q=xcb&state=closed&type=Issues&utf8=✓
It was suggested to start Spyder by specifying the path

QT_API=pyqt5 QT_QPA_PLATFORM_PLUGIN_PATH=${CONDA_ENV_PATH}/lib/qt5/plugins/platforms spyder

However, this does not work.
Versions and main components

Spyder Version: 3.0.1
Python Version: Python 3.5.2
Operating system: Linux manjaro 4.4.31-1-MANJARO
pyqt                      5.6.0                    py35_0
qt                        5.6.0                         1
qtawesome                 0.3.3                    py35_0
qtconsole                 4.2.1                    py35_1
qtpy                      1.1.2                    py35_0

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
Inside Spyder? It does not start.

Are you using Anaconda or the packages provided by your distribution?

Everything was installed with miniconda/conda, i.e. without anaconda. Then I installed anaconda, however, the same error occurs. Spyder 2.3.9 does work, only the versions 3.x do no at the moment.

I am experiencing this same issue.

This could be caused by the lack of libselinux in your systems, or because you have LD_LIBRARY_PATH set (which messes with Anaconda).

I somehow resolved this issue but not I'm getting the error in issue: https://github.com/spyder-ide/spyder/issues/3545
>spyder
Traceback (most recent call last):
  File "/usr/local/anaconda/bin/spyder", line 6, in <module>
    sys.exit(spyder.app.start.main())
  File "/usr/local/anaconda/lib/python2.7/site-packages/spyder/app/start.py", line 103, in main
    from spyder.app import mainwindow
  File "/usr/local/anaconda/lib/python2.7/site-packages/spyder/app/mainwindow.py", line 49, in <module>
    requirements.check_qt()
  File "/usr/local/anaconda/lib/python2.7/site-packages/spyder/requirements.py", line 39, in check_qt
    import qtpy
  File "/usr/local/anaconda/lib/python2.7/site-packages/qtpy/__init__.py", line 125, in <module>
    raise PythonQtError('No Qt bindings could be found')
qtpy.PythonQtError: No Qt bindings could be found

I don't have an LD_LIBRARY_PATH set and SELinux is set to Permissive.

I have solved the issue by pip uninstalling PyQt5/QtPy, then reinstalling. There seems to be some conflict between the pip/Arch linux/ conda installed libraries that causes this issue (I could also install these packages via the system package manager but using pip seems to work better in this case).
libselinux was installed on my system.

@vroomanj, what happens if you uninstall Anaconda and install it in your home directory?

In my case, I installed Anaconda 4.2.0 as the root user, but then only root could start Spyder.  Everybody else got the error about xcb.  After troubleshooting, I found that a lot of files were installed without world read or execute permission.  Correcting the permissions solved the problem and other users could work with Spyder.
Relevant commands:
find . ! -perm /o=r -ls
find . ! -perm /o=r -exec chmod o+rx {} ;

Removal of the LD_LIBRARY_PATH variable works for me. Thanks.

Closing as it seems solved by trying different things.


web page titele replaced

can you please only push the changes that you did. For example push only Index.html only. remaining all files should not be committed e.g. package.json, package.lock.json, yarn.lock.json


Analyse Rules

This feature was disabled in #798, with the discussion as to why on #751

As Carrot now deals with scale better in other areas... the analyse function can't deal with that. The existing button the React app is actually now a quick way for any user to bring the app to a halt (unbeknownst to them), which is why it might appear to not be working at first.

But we really need this feature to work.
As the analysis is expensive to run, we would like to move this work to an Azure function.
User journey

Request the analysis by clicking a button "Request Analysis" (?)
Get a loading status,
The user is notified when the analysis is available (they can resume using the app in the meantime).
If it's a small SR, then it'll complete quite quickly.

Technical Implementation
Move the work to Azure functions, we have existing patterns for this.
The uncertainty for me is actually where this is stored, we could write to a new table, or a file.

This could be scheduled otherwise - either 24 hours or hourly.

Do users want to select specific Scan Reports to compare against?


Filling in missing babbage unit tests

[x] I have enabled babbage era tests in core unit tests
[x] I have enabled babbage era tests in shelley unit tests
[x] I have unpended byron witnesses tests for any era
[x] I have investigated/fixed byron wits tests for Alonzo and Babbage
[x] I have postponed checking byron wits tests for Alonzo and Babbage to future PR when ledger's https://github.com/input-output-hk/cardano-ledger/pull/2863 is included in node bump

Comments
I have the following error:
  calculateBinary - ShelleyBasedEraShelley
    Byron witnesses - mainnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
    Byron witnesses - testnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
  calculateBinary - ShelleyBasedEraAllegra
    Byron witnesses - mainnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
    Byron witnesses - testnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
  calculateBinary - ShelleyBasedEraMary
    Byron witnesses - mainnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
    Byron witnesses - testnet
      1 input, 2 outputs [✔]
      2 inputs, 3 outputs [✔]
  calculateBinary - ShelleyBasedEraAlonzo
    Byron witnesses - mainnet
      1 input, 2 outputs [✘]
      2 inputs, 3 outputs [✘]
    Byron witnesses - testnet
      1 input, 2 outputs [✘]
      2 inputs, 3 outputs [✘]
  calculateBinary - ShelleyBasedEraBabbage
    Byron witnesses - mainnet
      1 input, 2 outputs [✘]
      2 inputs, 3 outputs [✘]
    Byron witnesses - testnet
      1 input, 2 outputs [✘]
      2 inputs, 3 outputs [✘]

and one example (Alonzo)
  test/unit/Cardano/Wallet/Shelley/TransactionSpec.hs:1436:13:
  1) Cardano.Wallet.Shelley.Transaction, calculateBinary - ShelleyBasedEraAlonzo, Byron witnesses - mainnet, 1 input, 2 outputs
       expected: "83a4008182582000000000000000000000000000000000000000000000000000000000000000000001828258390101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101011a001e84808258390102020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202021a0078175c021a0001faa403191e46a1028184582001000000000000000000000000000000000000000000000000000000000000005840d7af60ae33d2af351411c1445c79590526990bfa73cbb3732b54ef322daa142e6884023410f8be3c16e9bd52076f2bb36bf38dfe034a9f04658e9f56197ab80f5820000000000000000000000000000000000000000000000000000000000000000041a0f6"
        but got: "84a600818258200000000000000000000000000000000000000000000000000000000000000000000d8001828258390101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101011a001e84808258390102020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202021a0078175c021a0001faa403191e460e80a10281845820010000000000000000000000000000000000000000000000000000000000000058408d3a42ab5219a082c7993ec39f205bf1a6c39e611ec855f6c64eca6116891567b0fcdc09e86258f6720dfb48ce6bf9777211d306811e08128f2460e2520f1f0d5820000000000000000000000000000000000000000000000000000000000000000041a0f5f6"

and corresponding example for Babbage:
  test/unit/Cardano/Wallet/Shelley/TransactionSpec.hs:1436:13:
  5) Cardano.Wallet.Shelley.Transaction, calculateBinary - ShelleyBasedEraBabbage, Byron witnesses - mainnet, 1 input, 2 outputs
       expected: "83a4008182582000000000000000000000000000000000000000000000000000000000000000000001828258390101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101011a001e84808258390102020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202021a0078175c021a0001faa403191e46a1028184582001000000000000000000000000000000000000000000000000000000000000005840d7af60ae33d2af351411c1445c79590526990bfa73cbb3732b54ef322daa142e6884023410f8be3c16e9bd52076f2bb36bf38dfe034a9f04658e9f56197ab80f5820000000000000000000000000000000000000000000000000000000000000000041a0f6"
        but got: "84a700818258200000000000000000000000000000000000000000000000000000000000000000000d8012800182a2005839010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101011a001e8480a2005839010202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202011a0078175c021a0001faa403191e460e80a10281845820010000000000000000000000000000000000000000000000000000000000000058400ff32401589cbffd01f4ee20a716493e1502bff1d45e084d1122f9d54bd49742c7ab1ed3c2205d8de2dffacef77109ad0ea298001293f99c7c93090288a4410c5820000000000000000000000000000000000000000000000000000000000000000041a0f5f6"

After analyzing cbor we see:
shelley,mary,allegra
[{0: [[h'0000000000000000000000000000000000000000000000000000000000000000', 0]], 1: [[h'010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101', 2000000],   [h'010202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202', 7870300]], 2: 129700, 3: 7750}
, {2: [[h'0100000000000000000000000000000000000000000000000000000000000000', h'D7AF60AE33D2AF351411C1445C79590526990BFA73CBB3732B54EF322DAA142E6884023410F8BE3C16E9BD52076F2BB36BF38DFE034A9F04658E9F56197AB80F', h'0000000000000000000000000000000000000000000000000000000000000000', h'A0']]}
, null]

alonzo
[{0: [[h'0000000000000000000000000000000000000000000000000000000000000000', 0]], 13: [], 1: [[h'010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101', 2000000], [h'010202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202', 7870300]], 2: 129700, 3: 7750, 14: []}
, {2: [[h'0100000000000000000000000000000000000000000000000000000000000000', h'8D3A42AB5219A082C7993EC39F205BF1A6C39E611EC855F6C64ECA6116891567B0FCDC09E86258F6720DFB48CE6BF9777211D306811E08128F2460E2520F1F0D', h'0000000000000000000000000000000000000000000000000000000000000000', h'A0']]}
, true
, null]

babbage
[{0: [[h'0000000000000000000000000000000000000000000000000000000000000000', 0]], 13: [], 18: [], 1: [{0: h'010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101010101', 1: 2000000}, {0: h'010202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202020202', 1: 7870300}], 2: 129700, 3: 7750, 14: []}
, {2: [[h'0100000000000000000000000000000000000000000000000000000000000000', h'0FF32401589CBFFD01F4EE20A716493E1502BFF1D45E084D1122F9D54BD49742C7AB1ED3C2205D8DE2DFFACEF77109AD0EA298001293F99C7C93090288A4410C', h'0000000000000000000000000000000000000000000000000000000000000000', h'A0']]}
, true
, null]

Issue Number
ADP-1885

bors r+

bors r+

bors r+

bors r+


Conversation flow builder for Rasa

What problem are you trying to solve?
Provide ability for non devs to setup conversation flow (story based, rule based etc.)
What's your suggested solution?
Have a UX something like https://github.com/alyssaxuu/flowy . This is in line with what Microsoft Power Virtual Agent (PVA) / similar products provide users to help setup bots easily. This can enable RASA to become a SaaS solution for bots.
Examples (if relevant)

Microsoft PVA
Chatbot.com
Kairon

Is anything blocking this from being implemented? (if relevant)
Too much dev bandwidth being spent on setting up bot conversations
Definition of Done
Non devs should be able to create, test & deploy RASA bots

Wanted to understand from the team on challenges you anticipate if we go down the path of building a Tree based UX to enable non devs to setup RASA flows. Few we can see right away:

Forms (validation, activating & deactivating)
Custom actions (for now we are hoping to have ability to list all custom actions from which users can choose from)

Thanks for submitting this feature request 🚀 @aeshky will get back to you about it soon! ✨

@aeshky let me know if you need more info for this one


doc: improve readme, add Makefile

What type of PR is this?

/kind documentation
What this PR does / why we need it:
Adds a basic Makefile to build the gwctl binary and improves the documentation in the readme.
Which issue(s) this PR fixes: none

Does this PR introduce a user-facing change?:

NONE

Hi, I have a question. gwctl should be in a separate repository similar to https://github.com/kubernetes-sigs/ingress2gateway, right? Is there any particular reason why it is kept here? Having a separate repository could make it easier to manage.
I can help improve gwctl, as mentioned by @robscott in this comment: https://github.com/kubernetes-sigs/gateway-api/pull/2428#issuecomment-1751530167.
But I don't see any open issues. Do you think I should create them? Some of the problems I think we need to fix initially are:

Failing gracefully: Currently, gwctl throws a lot of errors when an expected resource or something else is missing. We can add checks to prevent this and fail gracefully while showing better messages to the user.
Move subcommands to proper packages: The way the code is organized can be a bit confusing. Refactoring it to look like other Go/Cobra CLI projects would be better.
Add tests.
Set up a workflow to publish binaries instead of installing them from the source.
Change the CLI name to kubectl-gw so that it is a kubectl plugin.

I think these are reasonable first steps, and after this, we can proceed as mentioned in the comment by sketching out a desired UX to be a drop-in replacement for kubectl get can be achieved.
Let me know what you think and I can fix these!

Another thing here is that if we are running go mod vendor,  shouldn't those files under the vendor folder be committed? But I don't think we need it and we can remove the command from the build process.

Thanks @pottekkat!
/assign @gauravkghildiyal

/ok-to-test

Thanks @pottekkat! This LGTM.
/lgtm
Yes I have an upcoming PR (should be out this week) to discuss and document the next steps of gwctl, which will be followed by the creation of the relevant issues. This should help facilitate contributions towards a common goal from everyone community :)

Thanks @pottekkat!
/approve

@gauravkghildiyal That sounds great! I would like to get involved. I was also working on a PR to organize the code better and fail gracefully with better error messages. I can also open that PR.

@pottekkat
That's great! I can only hope you don't run into issues with the changes from https://github.com/kubernetes-sigs/gateway-api/pull/2708 I'm trying to get this in sooner so it unblocks other development. Otherwise someone will have to deal with lots of merge conflicts :)

Hmmm I don't think the pull-gateway-api-verify is supposed to take so long.
/retest


Using GitHub Bots?

GitHub has lots of really handy bots that integrate with pushed commits and PR actions, as well as GitHub Actions CI. I have used GitHub’s bots in my own projects and found them very secure, helpful and easy to integrate.
I have raised #133 re: [ImgBot] but I am open to your feedback on more general Bot integrations as well.
Thanks.

We're just sticking with Swift CI for now. Thanks, @LemonSpike!


[Feature requests]: Messages: Configuration of buzz strength & do not power on LCD on message-auto-close

Messages:

I would like to have a configuration for the buzz strength.
It would be nice if the LCD would not turn on when an unread message is being auto-closed after the delay.

I can work on these myself but i am busy right now.
Leaving this here so i don't forget it.

I would like to have a configuration for the buzz strength.

I believe the current strength is already at its maximum. The doc says

strength - Power of vibration from 0 to 1 (Default 1)

The messages app call the buzz with default values. What we can do is increase the duration (default is 200ms but we can allow user to customize this value).
I can work on it later today.

The messages app call the buzz with default values.

How about we use the buzz library and then extend it, so you can have not just . and - but something else that's a lower power buzz. Then in all things (like alarms too) you could have a mix of powerful and less powerful buzzes.

The current strength is already at its maximum.

I do not want it to be stronger but rather weaker :)

That would be good if it wasn't for existing apps that use it, but I guess it's not the end of the world to have them buzzing a bit weaker until they reinstall or update the settings

Great! But you do realise that's gone from 14 lines up to 54? And if you don't use one of :.-= it just fails to resolve the promise now?
I yours is more readable now, but it's going to use more memory and probably be slower. Couldn't we just do it by adding 2 lines to the if...else chain?

Done!

It would be nice if the LCD would not turn on when an unread message is being auto-closed after the delay.

@myxor Does the fixed version work for you?
This is the fix I pushed last week: https://github.com/espruino/BangleApps/pull/1890/files#diff-97acef7b0efa75daea98d9fec30c299f193c5daaa88c7928c569915eb0a1e2f7I tried the change before I opened the PR and  I am 100% (99.999999%) sure it worked but I just noticed that now the screen turn on again. I reinstalled the latest version (from dev app loader) but the issue is still here - Does it only happen to me?

@alessandrococco no, it does not work for me. The screen still turns on on auto closing messages.

@alessandrococco no, it does not work for me. The screen still turns on on auto closing messages.

I don't understand the problem then :-/
There is another Bangle.setLCDPower(1); call but it has nothing to do with the end of the unread timeout 🤔
@gfwilliams Sorry to bother you - any idea what's going on here?

Is there a Bangle.unlock or similar? I guess one thing to do might be to add some boot code that caused Bangle.setLCDPower/unlock to write something to the console, then you could maybe see what was happening?

Is there a Bangle.unlock or similar?

This is the only setLocked/LCDPower in the messages app:

I guess one thing to do might be to add some boot code that caused Bangle.setLCDPower/unlock to write something to the console, then you could maybe see what was happening?

Sorry, I don't understand what you mean 😅

The root of the problem seems more trivial than expected: the load() function!
I just tried to call the load() from the Web IDE and the screen of my Bangle turned on!

I am not allowed to reopen this issue. @gfwilliams has to do it.

So what's the issue here? It's actually that the call to load() causes the Bangle to unlock?

Nope, the issue is that load() turns on the screen.
The goal here is to avoid to turn on the screen after the unread timeout expires.

Bangle 2.
What's your firmware? I just tried on 2.14: if I call load() from web IDE the screen always turns on 🤔

Well that's very odd. What are your 'wake on ...' settings set to?
Also what apps are installed? I wonder if some app/boot code it turning it on for you?

Well that's very odd. What are your 'wake on ...' settings set to?

Button only.

Also what apps are installed? I wonder if some app/boot code it turning it on for you?

widlock, widbt, launch, boot, swiperclocklaunch, messages, sched, setting, alarm, patclock, android, widChargingStatus (patclock is a custom clock I'm working on, ClockFace + Layout)

I just tried it on my Bangle.js 2 with 2v14.125, and load() from the IDE turns on the screen for me as well.
>require('Storage').readJSON('setting.json')
={ ble: true, blerepl: true, log: false, timeout: 15,
  vibrate: true,
  beep: "vib",
  timezone: 2, HID: false, clock: null, "12hour": false, brightness: 1,
  options: { wakeOnBTN1: true, wakeOnBTN2: true, wakeOnBTN3: true, wakeOnFaceUp: false,
    wakeOnTouch: true, wakeOnTwist: false, twistThreshold: 819.2, twistMaxY: -800, twistTimeout: 1000 },
  theme: { fg: 65535, bg: 0, fg2: 2047, bg2: 31,
    fgH: 65535, bgH: 63488, dark: true },
  quiet: 1, firstDayOfWeek: 1, batFullVoltage: 0.31744384765 }

@rigrig just tried it on a Bangle.js 2 here with your settings, and still nothing here. Please can you upload the contents of your .boot0 file and let me know which clock you have? It might give a hint as to what's turning the screen on

Now it doesn't happen anymore....
Which pointed me to qmsched: it applies Quiet Mode-specific settings in boot, but only while active: turns out Bangle.setLCDBrightness(1) briefly turns on the screen.
And it stays on until all code finishes running, so putting this in the IDE:
Bangle.setLCDBrightness(1);for(i=0;i<10000;i++){}
turns on the screen for several seconds.
I guess that's not the whole story though, because after load() it even stays on for a while after the clock is started, even though boot0 should be finished by then.
Just in case: I'm using barclock / boot0.txt

Ok, great - that makes sense then. If I recall I had to allow setLCDBrightness to turn the backlight on because there's this disconnect between whether the screen is unlocked, the LCD on, or the LCD backlight on...
I guess for most people, just not calling Bangle.setLCDBrightness if the value is 1 might fix it?

for most people, just not calling Bangle.setLCDBrightness if the value is 1 might fix it?

I think so, but

it will break setting brightness back to 1 when disabling Quiet Mode. ("broken" as in "won't change until the next reboot instead of immediately", so that's probably acceptable) too bad there isn't a getLCDBrightness...
Actually, I guess we could just add some code to make the qmsched library distinguish between boot-time and other changes.
it will remain broken for people who set a lower brightness (either always, or only during Quiet Mode).

Still, it's probably better than nothing.
Maybe we could also wrap the setLCDBrightness in a timeout (both in qmsched and settings), so at least it only flashes very briefly?
@alessandrococco: Did you have your LCD brightness set below 1? Otherwise this wouldn't explain why the screen turns on for you, as you don't have qmsched installed.

It seems that the LCD is still turned on when load() is called. Do we have an idea if this can be changed?

Did you see the comments above? What if LCD brightness is left at 1?
If so it should be easy enough to fix I hope

This was recently brought up on the Bangle.js subreddit: https://www.reddit.com/r/Banglejs/comments/17qtpgd/how_to_turn_off_backlight/

Did you see the comments above? What if LCD brightness is left at 1?

Just confirming that a load() call doesn't light up the screen if LCD brightness is left at 1. But it does if it has another value set (fw 2v21.148).

Ahh, interesting - thanks! Just checked and it's Bangle.setLCDBrightness that turns the LCD backlight on because it's checking if the LCD is on and not the LCD backlight (on Bangle.js 1 they used to be the same).
Looking at how it's used I'm a little bit concerned that some apps are using it as a way to turn the backlight on, so I wonder what the best fix is? any thoughts?

Just fix  Bangle.setLCDBrightness and change any apps that seem to have problems with it
Add a second argument dontTurnOn and only fix  Bangle.setLCDBrightness  when that is true (and then we set that in the bootloader code that calls it)

Made up my mind. #1 is far simpler - and putting a fix in now.
Pretty sure this is fixed now as buzz strength can be configured by the buzz pattern


do we plan for 1.8.1 release ?

There are some bugs in 1.8.0 release, will be nice if you plan to have 1.8.1 release.
Though it is deprecated, will be appreciated to keep this track for a while
thx

Which version is most stable just now? Thx

As far as docker releases, the current is 1.11.2.

This project (the boot2docker Windows Installer) is officially deprecated in favor of the new Docker Toolbox.

https://github.com/boot2docker/windows-installer#deprecated


tests for the publish and unpublish commands

Maybe we could set up a test S3 bucket with encrypted credentials?

Check out https://github.com/spulec/moto
I'm working on this for a specific feature, hoping to send a PR soon.  If all goes well, I'd be happy to implement this for more general tests of the publish command.


[Android][Controls -Slider] Missing shadow for slider indicator

Steps to Reproduce:

Launch app
From Burger menu select Controls > Slider
Notice shadow for slider indicator

Current behaviour

Expected behaviour

Repro rate
100%
Affected platform(s):
Android
Version & devices
1.0.0 (675)
Samsung Galaxy A50 - 9
iPhone 6S - 14.0.1

Verified on build 1.0.0 (714)


Update dependencies

Description
CVE-2022-41721 is introduced by golang.org/x/net. Update it to the latest.
upgraded golang.org/x/net => v0.5.0
upgraded golang.org/x/sys => v0.4.0
upgraded golang.org/x/term => v0.4.0
upgraded golang.org/x/text => v0.6.0
GitHub Issues
List the GitHub issues impacted by this PR:

GitHub Issue #

https://github.com/dell/csm/issues/583

Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[x] I have maintained at least 90% code coverage
[ ] I have inspected the Grafana dashboards to verify the data is displayed properly
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have maintained backward compatibility

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

[x] make check test

➜ make check test
./scripts/check.sh ./cmd/... ./opentelemetry/... ./internal/...
=== Checking format...
=== Finished
=== Vetting...
=== Finished
=== Linting...
=== Finished
=== Running gosec...
=== Finished
go test -count=1 -cover -race -timeout 30s -short ./...
?       github.com/dell/csm-metrics-powerstore/cmd/metrics-powerstore   [no test files]
ok      github.com/dell/csm-metrics-powerstore/internal/common  0.170s  coverage: 91.7% of statements
ok      github.com/dell/csm-metrics-powerstore/internal/entrypoint      0.677s coverage: 91.4% of statements
ok      github.com/dell/csm-metrics-powerstore/internal/k8s     0.078s  coverage: 91.8% of statements
?       github.com/dell/csm-metrics-powerstore/internal/k8s/mocks       [no test files]
ok      github.com/dell/csm-metrics-powerstore/internal/service 0.086s  coverage: 90.3% of statements
?       github.com/dell/csm-metrics-powerstore/internal/service/mocks   [no test files]
?       github.com/dell/csm-metrics-powerstore/opentelemetry/exporters  [no test files]
?       github.com/dell/csm-metrics-powerstore/opentelemetry/exporters/mocks   [no test files]
?       github.com/dell/csm-metrics-powerstore/opentelemetry/tracers    [no test files]

Manual inspection of the GUI
I have verified that the dashboards show the data properly while generating I/O and storage resources

[ ] Yes
[x] No

run e2e test

run e2e test


Customer question in nuget.exe CLI reference (dependencies)

https://docs.microsoft.com/en-us/nuget/tools/nuget-exe-cli-reference#comments-container
simon.vuong Aug 10, 2017
coming from a node background and working on .net core.
would "nuget add <package" be the .net core equivalent of "npm install ?" does it automatically satisfy version requirements between other dependencies? where does "nuget add" place the dependencies when source isn't specified? is there a "node_modules" equivalent?

responded to the customer.


Remove REDUCER_KEY

fixes #221

You can tag the issue in the OP to auto close it when the PR is merged :)  I did it for you on this one

You can tag the issue in the OP to auto close it when the PR is merged

I don't understand what you mean.

If your message starts with fixes and an issue, it'll associate them

This is somehow broken. I don't know why.


Using Chrome extensions while running RPA similar to Selenium

I'm wondering if there's a way to use the .crx file of a Chrome extension to be able to use it while running a script. This is how it's done in Selenium:
from selenium import webdriver
from selenium.webdriver.chrome.options import Options
options = Options()
options.add_argument("--disable-notifications")
options.add_argument("--headless")
options.add_extension(r"C:\Py\instant data scrapper.crx")
driver = webdriver.Chrome(options=options)
Thanks!

@damebash, I don't think this is doable, and not the goal of the project to use API to activate extensions since users will be manually activating from the UI. What you can try is simply do a wait(300) on first run, then download and setup and activate that extension for your user profile. The next time it runs will have that extension running because there is a specific Chrome user profile saved for the Python rpa package.

Selenium's goal is for testing websites so it has a wide and sophisticated API to do all kinds of things.
rpa package merely aims to let user automate repetitive process they do manually with a simple API.

I use extensions, I installed them by running the command to open some url and then the wait command to install it manually in chrome

Closing issue for now but do continue the discussion and let us know if that works for you!

Thanks for the quick replies!


vscode-python 2021.9.1246542782 doesn't install as it depends on ms-toolsai.jupyter-renderers, which isn't available.

Environment data

VS Code version: 1.60.2
Extension version (available under the Extensions sidebar): 2021.9.1246542782
OS and version: Linux
Python version (& distribution if applicable, e.g. Anaconda): python 3.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: ms-toolsai.jupyter-renderers
Value of the python.languageServer setting:

Expected behaviour
Install vscode-python
It works
Actual behaviour
It fails with a message that 'ms-toolsai.jupyter-renderers' as it is not compatible with VS Code '1.60.2'
Steps to reproduce:
Try to install vscode-python on a ssh-remote computer.
Work around
Downloading older versions of vscode-python seems to have fixed the issue for me.
Logs

[2021-10-07 14:33:53.101] [renderer6] [info] Downloading the 'ms-python.python' extension locally and install
[2021-10-07 14:33:53.320] [renderer6] [info] Downloading the workspace dependencies and packed extensions of 'ms-python.python' locally and install
[2021-10-07 14:33:53.427] [renderer6] [info] Downloaded extension: ms-toolsai.jupyter /c:/Users/GEORGE~1/AppData/Local/Temp/a40fccd5-8f1c-4fce-9f09-327216630650
[2021-10-07 14:34:35.361] [renderer6] [info] Successfully installed 'ms-toolsai.jupyter' extension
[2021-10-07 14:34:35.362] [renderer6] [info] Downloaded extension: ms-python.vscode-pylance /c:/Users/GEORGE~1/AppData/Local/Temp/cd7e84f9-b5bd-410a-9d2d-30a647b2ba40
[2021-10-07 14:35:23.284] [renderer6] [info] Successfully installed 'ms-python.vscode-pylance' extension
[2021-10-07 14:35:23.509] [renderer6] [error] Error: Unable to install extension 'ms-toolsai.jupyter-renderers' as it is not compatible with VS Code '1.60.2'.

Try following the instructions here to remove the Jupyter dependency #17491. If you need Jupyter please file a bug on https://github.com/microsoft/vscode-jupyter

Please try this again after updating VS Code to version 1.61.0


Revert "Translate the registry_spi.md"

Reverts apache/dolphinscheduler-website#399

sorry for merging this PR


How to adjust learning rate when the number of workers incresing?

For example, I set original learning rate 0.001 when standalone mode, and 0.001 / sqrt(10) performs well when 10 workers running. But when 20 workers running, 0.001 / sqrt(20) performs very bad. So, is there any suggestion to adjust when the number of workers incresing?

Invalid, closed.


Fix wrong Objective-C use example

Hi,
It cost me quite some time to find out that the category on UIImage specifies instance methods, not class methods as depicted in the current example. No wonder Xcode could not find this category method at first.
This PR updates the example to fix this.

Oh, this was quite embarrassing, thanks for the PR!


ddev-router does not work on VM with proxy after update from 1.16.* to 1.17.*

Full Gist Diagnostic
After upgrading from v1.16.7 to v1.17.7, the ddev-router does not work anymore. DDev is running on a VirtualBox Debian 10 buster behind a LAN proxy.
ddev-router failed to become ready; debug with 'docker logs ddev-router'; logOutput=, err=health check timed out: labels map[com.docker.compose.service:ddev-router] timed out without becoming healthy, status=

Install a VirtualBox VM with Debian 10 Buster
Configure proxy.
Create DDev config
Start DDev

It's not possible to downgrade, as ddev-homebrew is not versioned.
Expected behavior
The DDev containers start up and work just fine.
Version and configuration information (please complete the following information):

Windows 10 with VirtualBox and Debian 10 Buster
1.17.7
php and mysql project config
no custom config
LAN proxy to route through (I cannot get rid of that one, so don't "fix" it by suggesting it)

Additional context
It would have been more helpful for users if the Homebrew receipt would copy the versioning from the main project so that brew versioning could be utilized properly to work around untested new versions.

Please provide a gist with the output of docker logs ddev-router as the error message says.
Could you say a bit about why you're using Virtualbox instead of WSL2, which has a far more likely future?
Your gist shows a completely different problem (duplicate mount for /var/lib/mysql) which is often seen on older versions of docker-compose like Debian 10 ships with. You can fix that issue by upgrading docker-compose with either pip3 install docker-compose or by downloading the binary from docker's site. IIRC this is an intermittent problem.
So let's get to where it starts up and get to the ddev-router problem, and see what the logs from ddev-router say.

Please provide a gist with the output of docker logs ddev-router as the error message says.

Here is the gist you requested. To me, it was not telling at all, which is why I left it out in the first place.

Could you say a bit about why you're using Virtualbox instead of WSL2, which has a far more likely future?

I am in a migration team helping a company to migrate their software production away from VirtualBox towards WSL2. The latter just recently cleared the company's bar for a feasible process during ongoing production, and that triggered the internal preparations to follow that process. DDev enables us to quickly draw a containerized environment around our products and cave them out the former VirtualBox environment.

Your gist shows a completely different problem (duplicate mount for /var/lib/mysql) which is often seen on older versions of docker-compose like Debian 10 ships with. You can fix that issue by upgrading docker-compose with either pip3 install docker-compose or by downloading the binary from docker's site. IIRC this is an intermittent problem.

I use the docker-compose package from Docker's APT sources, not from the maintainers. The duplicate mount can easily be solved if I were just removing the ddev-generated images. You ignored the earlier error in the same gist, which I outlined above:
ddev-router failed to become ready; debug with 'docker logs ddev-router'; logOutput=, err=health check timed out: labels map[com.docker.compose.service:ddev-router] timed out without becoming healthy, status=

Here's what I need for the ddev-router timeout:
docker inspect --format "{{json .State.Health }}" ddev-router
Please make sure it's not just slow. Do a ddev list a minute or two after the start.

I'm pretty sure you're using apt-get install docker-compose, which is just the Debian packaging (very, very old) of docker-compose. We can confirm that with docker-compose --version. Docker does not package docker-compose with their repositories at this time, never have to date, although they plan to.

I did a PR to bump the minimum required, https://github.com/drud/ddev/pull/3099, but in the process noted that the ddev Debian installation instructions do already tell you to install docker and docker-compose from Docker using their technique, not to use the built-in docker and docker-compose on Debian 10. Obviously I should have updated the minimum version when I updated that doc.

I uninstalled the faulty docker-compose version from the distro maintainers to reinstall it from GitHub. I re-run the test script and created a new gist with their output. (Unfortunately, it's the same error, just without any report of double mounts.)
I also ran inspection and created the gist output here.

Now it looks like your web container is failing, and maybe that's the whole thing. Make sure you do a ddev poweroff before testing things, as you have quite a number of things going now.
The last thing I see from the healthcheck in your web container is
{"Start":"2021-07-12T18:45:52.624928183+02:00","End":"2021-07-12T18:45:52.72261712+02:00","ExitCode":1,"Output":"phpstatus: FAILED /var/www/html: OK mailhog: FAILED "
So it looks like the healthcheck script is failing. I think that container should still be running in this case, so you should be able to ddev ssh <projectname> or if it won't let you, docker exec -it ddev-<projectname>-web bash and you can then run the healthcheck script manually with bash -x /healthcheck.sh and see what it's saying.
You haven't mentioned the web container timing out in your previous descriptions of this problem. Is this the same as what you're seeing on your project?
I'm happy to do a call with you on this so we can look at it together. I'm on US Mountain time, UTC-7. Looks like you're on CET so it's already getting late in your day, but if you want to take a look still today we can.

Looking back at your original gist, I see that you installed docker with homebrew; that may be OK, but it's not the recommended technique. Did you reinstall both Docker and docker-compose using the Docker.com instructions?

Now it looks like your web container is failing, and maybe that's the whole thing. Make sure you do a ddev poweroff before testing things, as you have quite a number of things going now.

All I really have going on DDev in this machine was run and generated by the test script recommended for diagnosis. Everything else was off and removed and DDev only resides for diagnosis on the system now. The project I transform now will use docker-compose directly, as there is no problem in running that and the problem I described in this issue did not exist in 1.16 of DDev.
After the company behind DDev closed down, the company I work for had to be convinced to wait for any further development of this project before making the decision to exclude and replace DDev as part of the process. The advantage of DDev really was its easy rollout and manageability by developers with little to no pre-existing knowledge, but if we weren't able to quickly call off an update of DDev to continue using an older version, just because any force-updated current version would brick the setup, this would be another big no for DDev and we would probably need to give up DDev for the transformation process then.

I'm happy to do a call with you on this so we can look at it together. I'm on US Mountain time, UTC-7. Looks like you're on CET so it's already getting late in your day, but if you want to take a look still today we can.

Thank you for this kind offer but for strict NDA reasons we would not be able to make that call. We would really just be happy to be able to go back to 1.16 of DDev until 1.17 has been properly tested.

You haven't mentioned the web container timing out in your previous descriptions of this problem. Is this the same as what you're seeing on your project?

If the container listing says the containers are "done", I do not assume that there is a problem. Also, it does not make sense to run in this very same problem no matter where I create a new ddev configuration on the machine. It also happens on other VMs.

Looking back at your original gist, I see that you installed docker with homebrew; that may be OK, but it's not the recommended technique. Did you reinstall both Docker and docker-compose using the Docker.com instructions?

That I did. If docker was also installed by Homebrew, it must have been part of any installation process in Homebrew, as a dependency for example.

ddev v1.17 has been out for many months now (since April) so I expect that this is a configuration error or perhaps some weird thing about Virtualbox (but I doubt it). Perhaps you're mounting on a non-native filesystem. Anyway, if you can't accept direct help, you can do the things suggested in https://github.com/drud/ddev/issues/3098#issuecomment-878469916 - When you get into the web container, you should be able to run that healthcheck script and see what's going on.
But nothing you've demoed for me at this point is related to the original complaint, as far as I can tell.
You can also just ddev poweroff && mkdir junk && cd junk && ddev config --auto && ddev start and see what happens. If you get this same issue, it probably means php-fpm just isn't working, so that could actually be a problem with permissions on ports in the containers, so could possibly be something about virtualbox, but again, I doubt it.

Perhaps you're mounting on a non-native filesystem.
As a matter of fact, this error appears on when DDev config resides on both a Linux native file system and vboxsf, a VirtualBox emulated filesystem for external mount, as well.

When you get into the web container, you should be able to run that healthcheck script and see what's going on.
I will run this healthcheck in office hours tomorrow (CEST), but as all the current outputs on our table here tell you as much as they tell me, I doubt that the healthcheck would output any significant hint to the problem.

In general, all errors regarding double database mounts or a Docker installation coming from Homebrew instead of APT would bare no reason why the router (which is somewhat of a withstanding component) would not start.

Right, you haven't yet showed me a situation where the router won't start. I'm hoping you can demonstrate that at some point. The earlier demo you did was a db container failure, the most recent was a web container failure.

Since you're on the way to WSL2 (which is fully supported) from Virtualbox (which is way not supported) you might be able to stay with v1.16 until you're on WSL2. But I have no clue why there would be any difference in behavior of the router between 1.16 and 1.17. But like I say... you haven't yet demonstrated the error you originally reported.

The router can't come up until your db and web containers come up.

you haven't yet demonstrated the error you originally reported.

Except that I did. The error appears in both demos, way before any other error, for example in line 40 in this one.

The router can't come up until your db and web containers come up.

Again, then the DDev diagnosis script does not create proper db and web containers.

I'm trying to help you. You'll have to play along.

I'm trying to help you. You'll have to play along.

And you have to believe me when I give you feedback after following your instructions. Here is the gist of the "junk" test you gave me earlier: "Junk" test
It also ends with a router fail.
The health check states "phpstatus" and "mailhog" failed.

The router can't come up because the web container has failed. Ddev waits for the "all OK" from the web container.
Now please do what I suggested and ddev ssh or docker exec -it ddev-<project>-web bash and try to figure out why the healthcheck script fails. Run bash -x /healcheck.sh and give it a shot.

Now please do what I suggested and ddev ssh or docker exec -it ddev--web bash and try to figure out why the healthcheck script fails. Run bash -x /healcheck.sh and give it a shot.

I did that already, it's in the three last lines of that last gist I gave you.

(I see you did a bash +x, you want to do a bash -x.

And you'll need to use your debugging skills inside the container to figure out why those are failing. You'll want to look at php-fpm's status and the nginx and php-fpm configuration. Obviously I would help you with this.

(I see you did a bash +x, you want to do a bash -x.

Nice, did not know that. I updated the last gist.

And you'll need to use your debugging skills inside the container to figure out why those are failing. You'll want to look at php-fpm's status and the nginx and php-fpm configuration. Obviously I would help you with this.

Alright, no problem. The services of the junk project report back just fine:
root@junk-web:~# service --status-all
 [ - ]  apache-htcacheclean
 [ - ]  apache2
 [ - ]  blackfire-agent
 [ ? ]  hwclock.sh
 [ + ]  nginx
 [ + ]  nginx-debug
 [ - ]  php5.6-fpm
 [ - ]  php7.0-fpm
 [ - ]  php7.1-fpm
 [ - ]  php7.2-fpm
 [ - ]  php7.3-fpm
 [ + ]  php7.4-fpm
 [ - ]  php8.0-fpm
 [ - ]  procps
 [ - ]  rsync
 [ - ]  sudo
 [ + ]  supervisor
 [ - ]  x11-common
root@junk-web:~# service php7.4-fpm status
[ ok ] php-fpm7.4 is running.
root@junk-web:~# service nginx status
[ ok ] nginx is running.
root@junk-web:~#

Could it be that the services report back their success just too slow? Is the sleeptime of 59 maybe too short?

Your next task is to figure out why curl --fail -s <IP_ADDRESS>/phpstatus fails. Just ddev ssh into the web container and debug that.

I did that and I found out that if I replace the IP <IP_ADDRESS> with localhost, I can get mailhog and phpstatus.

So do you have a VPN or something that is rerouting traffic to <IP_ADDRESS>?
What does ping -c 1 localhost show you?

I have no VPN in my VM nor the host system there. I mentioned the LAN proxy in my initial report.

What does ping -c 1 localhost show you? (inside the container)

guhl@junk-web:/var/www/html$ ping -c 1 localhost
PING localhost (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from localhost (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=0.033 ms

--- localhost ping statistics ---
1 packets transmitted, 1 received, 0% packet loss, time 0ms
rtt min/avg/max/mdev = 0.033/0.033/0.033/0.000 ms
guhl@junk-web:/var/www/html$

Could we have some bootstrapping problem here? The router is not there so the curl calls won't be dispatched, and since the curl calls won't be dispatched, the router won't come up?

Curl here is entirely within the web container, the router is not involved in any way. If you can curl <IP_ADDRESS> and you can't curl localhost... something is wrong with your networking/docker.
Why don't you show the difference between curl -I localhost/phpstatus and curl -I <IP_ADDRESS>/phpstatus

If you can curl <IP_ADDRESS> and you can't curl localhost... something is wrong with your networking/docker.

It's the other way round. Here's the difference between the curls:
guhl@junk-web:/var/www/html$ curl -I localhost/phpstatus
HTTP/1.1 200 OK
Server: nginx
Date: Mon, 12 Jul 2021 19:57:00 GMT
Content-Type: text/html; charset=UTF-8
Connection: close
Vary: Accept-Encoding
Expires: Thu, 01 Jan 1970 00:00:00 GMT
Cache-Control: no-cache, no-store, must-revalidate, max-age=0

guhl@junk-web:/var/www/html$ curl -I <IP_ADDRESS>/phpstatus
HTTP/1.1 503 Service Unavailable
Server: squid/3.5.23
Mime-Version: 1.0
Date: Mon, 12 Jul 2021 19:57:08 GMT
Content-Type: text/html;charset=utf-8
Content-Length: 3625
X-Squid-Error: ERR_CONNECT_FAIL 111
Vary: Accept-Language
Content-Language: en
X-Cache: MISS from firewall2017
X-Cache-Lookup: MISS from firewall2017:3128
Via: 1.1 firewall2017 (squid/3.5.23)
Connection: keep-alive

guhl@junk-web:/var/www/html$

I repeated the IP calls and it is in fact the case that the LAN proxy informs me that it could not retrieve "http://<IP_ADDRESS>:8025/", means that the IP call
However, of course I have set localhost as well as <IP_ADDRESS> in the the conventional $no_proxy variable of the linux host's shell profile, but the DDev container does not take the no_proxy list from its host but creates its own:
guhl@junk-web:/var/www/html$ echo $no_proxy
<IP_ADDRESS>/8,localhost

IP class range notation is an invalid notation for the no_proxy list and it explains why curls on localhost work but curls on the IP do not.

There's the problem, "Server: squid/3.5.23"
You have <IP_ADDRESS> taken over by a squid proxy. There is no usage of squid anywhere in ddev. So your network/docker is routing <IP_ADDRESS> to somewhere it shouldn't.
Please exclude <IP_ADDRESS> from your proxy settings in the docker configuration.

https://github.com/drud/ddev/commit/98e02d692a802ba48b0ea0851da55b46bace13bb

It's excluded since the beginning.
guhl@marseille:~/junk$ cat ~/.docker/config.json
{
 "proxies":
 {
   "default":
   {
     "httpProxy": "http://<IP_ADDRESS>:3128",
     "httpsProxy": "http://<IP_ADDRESS>:3128",
     "noProxy": "<IP_ADDRESS>,localhost"
   }
 }
}
guhl@marseille:~/junk$

noProxy was running with the IP range notation which was documented to work for Docker, but the container directly took that over for its own no_proxy configuration, so I changed the IP to simply .1
I rerun the whole "junk" rollout, and this gist is very weird: weird output
Here is what's weird: I deleted the "junk" test and repeated a new instance for the "junk" test and it did not work until I also repeated the health check manually in the container - I would have expected that when I run a new configuration, the health check would result differently, not to what has been there before.

The ddev-router image in docker, as well as the container, would have to be manually removed and recreated to make the health check work.

ddev poweroff is your friend. That kills off all containers. It's very unusual to have to delete images, but I've seen it happen. Images in a normal world are just static files downloaded by dockerhub, and of course are unrelated to any proxy configuration.
But it sounds like you have it working now?
I saw this once before, but it was the opposite - they were proxying localhost by accident, and only <IP_ADDRESS> would work. At that time we changed the healthcheck to use the IP address :)

ddev poweroff is your friend. That kills off all containers.

So you would think. But it did neither stop nor remove the ddev-router.

Images in a normal world are just static files downloaded by dockerhub, and of course are unrelated to any proxy configuration.

Unless these downloads are derived to alter configurations, say, from the host's docker configuration. I am sure it's not optimal, either only a few distributions or Docker itself would support an IP range notation like /8, as they document it as a working configuration, but ddev-router would take over that range notation without ensuring it would even work with its base.

But it sounds like you have it working now?

Digest: sha256:30bf3f056210d258dadf64ba155f06513c3e46410b0444fd681bb656c575bc28
Status: Downloaded newer image for drud/ddev-router:v1.17.6
docker.io/drud/ddev-router:v1.17.6
Building ddev-ssh-agent
Creating ddev-ssh-agent ...
Creating ddev-ssh-agent ... done
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
Pushed mkcert rootca certs to ddev-global-cache/mkcert
Running   Command=ip address show dev docker0
Building db
Building web
Creating ddev-junk-db ...
Creating ddev-junk-db ... done
Creating ddev-junk-dba ...
Creating ddev-junk-web ...
Creating ddev-junk-dba ... done
Creating ddev-junk-web ... done
Creating ddev-router ...
Creating ddev-router ... done
Project type has no settings paths configured, so not creating settings file.
Project type has no settings paths configured, so not creating settings file.
Successfully started junk
Project can be reached at https://junk.ddev.site https://<IP_ADDRESS>:49167
guhl@marseille:~/junk$ nano index.htm
guhl@marseille:~/junk$ curl https://<IP_ADDRESS>:49167

Issue 3098 resolved!

guhl@marseille:~/junk$

they were proxying localhost by accident, and only <IP_ADDRESS> would work. At that time we changed the healthcheck to use the IP address :)

I would not recommend to change it back or change it in any other way, as the current configuration is basically working. I only recommend to check if the underlying system would support as no_proxy what Docker allows for a noProxy configuration, which would be an IP address like <IP_ADDRESS>/8 - in the system we were testing here, this was just not supported.
And maybe it is possible to include a basic loopback check on <IP_ADDRESS> and localhost if a proxy is present.

The resolution here for anybody who might stumble upon it: Docker was misconfigured with inappropriate proxy addresses, and that cause the ddev-webserver healthcheck to be sent off to a squid proxy instead of staying inside the container. https://github.com/drud/ddev/commit/98e02d692a802ba48b0ea0851da55b46bace13bb adds to the docs to add turning off any proxy configuration in the troubleshooting phase.

Actually, Docker allows for configurations which would not be allowed in the container. The word "misconfiguration" is misleading. For troubleshooting, I did not turn off any proxy configuration but changed it into a configuration which was accepted by the ddev-router.

ddev-router wasn't involved in this at all. All of your docker containers were broken by this, as they all have healthchecks. The problem was due to proxy misconfiguration. I'm glad you got it sorted out. If you think there's a bug in what docker allows, please open an issue there.

ddev-router wasn't involved in this at all. All of your docker containers were broken by this, as they all have healthchecks.
ddev-router receives bad healthchecks even if the configuration was changed. They either were to be manually repeated on each of the containers or the containers were to be completely rebuilt with ddev poweroff && ddev delete images && ddev start.

I also opened this issue with the specific note that some users cannot get rid of their LAN proxy, for example in a company environment or other networks they lack control of. The solution would not be to remove the proxy configuration for Docker completely. It's completely the opposite direction of what I did.

Again, this was the result of docker proxy misconfiguration. Again, if docker allowed you to misconfigure it, please open an issue with docker. Please be thankful for all the support you got with your misconfiguration problem and let this die.

Well thank you for helping me working around a ddev compatibility problem with legitimate Docker environment configurations. I didn't have it with any other container, so I don't see the point to report an issue there.

For those who might want to configure for a web proxy in the future, there are instructions here: https://github.com/drud/ddev-contrib/tree/master/recipes/proxy


[FLINK-27673][tests] Migrate flink-table-api-scala to JUnit5

What is the purpose of the change
Update the flink-table-api-scala module to AssertJ and JUnit 5 following the JUnit 5 Migration Guide
Brief change log
Use JUnit5 and AssertJ in tests instead of JUnit4 and Hamcrest
Verifying this change
This change is a code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): ( no)
The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: ( no)

Documentation

Does this pull request introduce a new feature? ( no)
If yes, how is the feature documented? (not applicable)

@flinkbot run azure


Improve plural keys

Is your feature request related to a problem? Please describe.
Not sure if this is only me but I would assume the following rules would apply for the plural keys.
zero count = 0
one count = 1
two count = 2
other count = 3,4,5,6...
But if you have en-US as locale you will get the following (look code snippet).
[...new Array(30).fill(0)].forEach((_, count) => {
    console.log(`Count = ${count}`, new Intl.PluralRules('en-US').select(count))
})
Count = 0 other
Count = 1 one
Count = 2 other
Count = 3 other
Count = 4 other
Count = 5 other
Count = 6 other
...
...

Describe the solution you'd like
This PR solves my problem https://github.com/QuiiBz/next-international/compare/main...lindesvard:next-international:improve-plurals?expand=1
Additional context
For me coming from other translations libraries I feel like using Intl.PluralRules is somewhat wrong. Especially when it has quite different behavior between different locales.
For instance this is how i18n-next is doing https://www.i18next.com/translation-function/plurals

I think Intl.PluralRules is not the best way of handling plurals for this kind of lib. Best case scenario would be if Intl.MessageFormat would be a thing. There is also some good polyfills for this but I understand if you do not want to bloat the bundle with this.
The use case why I want this is because zero only works with some locales and I think its quite confusing reading the documentation. Since it implies that zero will match count: 0. But that depends if your locale support it.
t('selected', { count: 0 }) // No options selected
t('selected', { count: 1 }) // One option selected
t('selected', { count: 2 }) // 2 options selected
t('selected', { count: 5 }) // 5 options selected

This is the behavior I want for all locales and with the current documentation it looks like I can get that.
So either I think you should make it clear that zero and other plural words might not work for your locale. And link to this page for more information.
Or we just add the zero use case to be default when count is 0 for all locales. This should not be a breaking change either since we will fallback on #other if #zero is not defined.


In the setup docs, mention sudo groups

For easier sudo configuration with the sudo isolation plugin, we really want the vespene user to be in sudoers.
This is not something that the setup automation currently sets up.
We should at least add a note in the docs for this

Running the setup scripts as root triggers the warning that the script needs to run as a sudoer.
I however, made a user called vespene and then realized that the setup script tries to create a vespene user as well.

Hi @chopraaa,
Thanks for the heads up that this may be unclear, I should make it more obvious that this will create the vespene user in the docs.
It's intended that you run the setup scripts from a user like "chopraaa" that has sudo access.


Add types package

You mentioned in the description that there is a matching types package, however it doesn't exits in the registry. Could you please add the types package?

@erzki Was this an oversight or are you missing the types package and it needs to be built?

@erzki Was this an oversight or are you missing the types package and it needs to be built?

I missed to add the types. If you can, please create a PR with the types
included.
Den fre 8 jan. 2021 01:38Greg Veres<EMAIL_ADDRESS>skrev:

@erzki https://github.com/erzki Was this an oversight or are you
missing the types package and it needs to be built?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/erzki/signalr-asp-net/issues/1#issuecomment-756473329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMIMEKDVOAQXIBK56NEANTSYZHZ7ANCNFSM4QBUQYFQ
.

I missed to add the types. If you can, please create a PR with the types
included.
Den fre 8 jan. 2021 01:38Greg Veres<EMAIL_ADDRESS>skrev:

@erzki https://github.com/erzki Was this an oversight or are you
missing the types package and it needs to be built?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/erzki/signalr-asp-net/issues/1#issuecomment-756473329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMIMEKDVOAQXIBK56NEANTSYZHZ7ANCNFSM4QBUQYFQ
.

Fixed in #4
It's also published as a new version to npm (1.0.3)


Triggering JSViz JSON update after Data Function

This isn't an issue with JSViz examples, but I have an issue that JSViz does not update when the underlying data is updated via a Data Function.  I am using a Python script that takes a document property into account to generate and replace an existing table.  When I point this table in JSViz component, and when I trigger the data function to update the table, it does not update the JSViz even though the data table content is updated.  Have you come across this issue?  Do you know any work around? Or, am I missing something???

It should call the render function again with the new data. If it’s Not using my JSViz components then your best To raise it on TIBCO’s community as they own JSViz itself.


Filewatch fails to build after latest pull-request

Because of the new line add_subdirectory(example) in the main CMakeLists.txt, filewatch fails to build as one of my dependencies. The example's CMakeLists.txt has a target_include_directories(main PRIVATE ${CMAKE_SOURCE_DIR}) that sets some include directory, but when filewatch is included as a library/dependency, this does not add the right directory, and hence main.cpp has a fatal  error C1083: Cannot open include file: 'FileWatch.hpp' for me.
Easy fix would be to remove the add_subdirectory(example), or use something else than ${CMAKE_SOURCE_DIR} (maybe ${CMAKE_CURRENT_SOURCE_DIR}?).

ahhh damn, yes when I originally built the project I intended it to be just be a copy and paste job of the filewatch.hpp file, but yes really the cmake needs to be set up in a digestible way.
I'll quickly remove the add_subdirectory stuff

Yes exactly, with FetchContent. I've managed to get it working locally by using target_include_directories(main PRIVATE ${PROJECT_SOURCE_DIR}) in the examples/CMakeLists.txt.

Thanks for the quick help. Working locally with your fix now, re-started the Github Actions build now

No worries, I think this will prompt me to to looking at getting this digestible from cmake more easily while still having private tests.
At the moment it seems like I also set tons of CMAKE_CXX_FLAGS flags, however I don't know if they are scoped to the projects, or can leak out.
I think I can at least expose it as something like
add_library(FileWatch INTERFACE FileWatch.hpp)


Kubernetes Installing with regular manifests section is missing challenge resources

In the documentation on https://cert-manager.io/docs/installation/kubernetes/#installing-with-regular-manifests is stated:
Installing with regular manifests
All resources (the CustomResourceDefinitions, cert-manager, namespace, and the webhook component) are included in a single YAML manifest file:
However, the yaml file appears to be missing the CRD's related to challenges, causing the cert-manager to log numerous messages like:
E0410 08:47:15.780099       1 reflector.go:153] external/io_k8s_client_go/tools/cache/reflector.go:105: Failed to list *v1alpha2.Challenge: the server could not find the requested resource (get challenges.acme.cert-manager.io)
This is noticed when trying to issue a letsencrypt-staging certificate.
If I rebuild the environment and apply the listed crds file (https://github.com/jetstack/cert-manager/releases/download/v0.14.1/cert-manager.crds.yaml) in the Helm section beforehand, everything works perfectly.
Can either the documentation on regular manifests installtion or the yaml file used in that section be updated so it works after following the listed steps?

I just ran the static manifests and it does seem to deploy the CRDs correct
Can you post the output you see when applying the static manifest?
kubectl apply --validate=false -f https://github.com/jetstack/cert-manager/releases/download/v0.14.2/cert-manager.yaml
customresourcedefinition.apiextensions.k8s.io/certificaterequests.cert-manager.io configured
customresourcedefinition.apiextensions.k8s.io/certificates.cert-manager.io configured
customresourcedefinition.apiextensions.k8s.io/challenges.acme.cert-manager.io configured
customresourcedefinition.apiextensions.k8s.io/clusterissuers.cert-manager.io configured
customresourcedefinition.apiextensions.k8s.io/issuers.cert-manager.io configured
customresourcedefinition.apiextensions.k8s.io/orders.acme.cert-manager.io configured
[...]

I tried recreating the issue myself but don't seem to be able to. I guess we can ignore it and focus on other things.