Bug: Not all suggestions are being displayed on insight page

Describe the bug
Just few repos are being showed when a user searches for a repo even tho the query returns many repos.

Steps to reproduce

Go to the page to create a new insight, https://insights.opensauced.pizza/hub/insights/new
Open dev tools and click on the Network tab

Browsers
No response
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

@Olaleye-Blessing @brandonroberts
    if (req.ok) {       const res = await req.json();       const suggestions = res.items.map((item: any) => item.full_name);       if (suggestions.length > 5) suggestions.length = 5;       setSuggestions(suggestions);
According to this only 5 results can be displayed, is it a bug or a feature?

This thus seem like it was intentional and I want to assume it was indeed made that way in the absence of a ScrollArea component in the beta design system at initial implementation.
Here's what happen without (if (suggestions.length > 5) suggestions.length = 5;) limiting suggestions count to 5

I added a ScrollArea component to the design system recently #1283
Would any of you guys want to give implementing it into the Search component a shot?

@babblebey Okay, I will try implementing it.

Okay, I will try implementing it.

Great 🍕 Let us know if you need any help!
You can reference the ScrollArea documentation here -> ScrollArea Story

@babblebey Could you please checkout the changes I have made.

Could you please checkout the changes I have made.

Great stuff @aryanas159, kindly note that your PR #1377 doesn't exactly fix this current issue.
I have created a new issue for what your PR fixes at #1378. Take the new Issue, then edit your PR Description, relate it accordingly.
Either you or @Olaleye-Blessing can proceed to fixing this current issue with another PR.

Hey @aryanas159,
My apologies, I have taken notice of how separating the PRs makes them both ineffective. So let fix this!

Commit your second change to this branch to effect the removal of line below from the InsightPage component

if (suggestions.length > 5) suggestions.length = 5;

Close your PR #1377
I will close the Issue #1378

This way we are be able to see a working demo of the ScrollArea implemented with the full suggestions list.

@babblebey could you check this issue I think this is a feature, not a bug.

IMHO the page will be so huge if we drop the limitation, or we can make overflow only in the suggestions area to have scroll and see only 5 elements once,

could you check this issue I think this is a feature, not a bug.

@NsdHSO
True, the TableHeader component seen in the 'suggested solution' section of the issue #932 indeed implements the same logic in handling the suggestions counts as with the InsightPage component, that's where it must've gotten the idea from.
if (suggestions.length > 5) suggestions.length = 5;

IMHO the page will be so huge if we drop the limitation, or we can make overflow only in the suggestions area to have scroll and see only 5 elements once,

The goal is introduce all the suggestions without the limit and implement the ScrollArea which will surely allow us see only 5 elements at once and the rest will require a scroll as seen below

What do you think?

I think this approach is more feasible.

@babblebey I have made a different PR and closed the other two, this should fix all the issues.


Could not connect to server: bytes remaining on stream

I am getting the following error. How to correct it?
Mar 23 09:48:13 raspberrypi librespot[1389]: [2020-03-23T01:48:13Z INFO librespot] librespot (raspotify v0.14.0) 3672214 (2020-01-30). Built on 2020-02-16. Build ID: 7pZDdYUK
Mar 23 09:48:13 raspberrypi librespot[1389]: [2020-03-23T01:48:13Z INFO librespot_core::session] Connecting to AP "gae2-accesspoint-b-02m1.ap.spotify.com:4070"
Mar 23 09:48:14 raspberrypi librespot[1389]: [2020-03-23T01:48:14Z ERROR librespot] Could not connect to server: bytes remaining on stream

Did a restart and it is working again.
My bad for the noise.


Remove pretext breadcrumbs on hold and publish

Changes in this PR:
Remove extra pretext breadcrumbs in the 'on hold' and 'publish'
Trello card / Rollbar error (etc)
https://trello.com/c/EslWpogp/1197-remove-extra-breadcrumbs-for-put-on-hold-and-publish-document
Screenshots of UI changes:
Before
Judgment View

Press Summary View

After
Judgment View

Press Summary View

[ ] Requires env variable(s) to be updated

 Remove pretext extra breadcrumbs for Put on hold and Publish document

Codecov Report

Merging #1088 (ea09638) into main (5aca2f9) will decrease coverage by 0.09%.
The diff coverage is n/a.

:exclamation: Current head ea09638 differs from pull request most recent head 8bd0b2e. Consider uploading reports for the commit 8bd0b2e to get more accurate results

@@            Coverage Diff             @@
##             main    #1088      +/-   ##
==========================================
- Coverage   86.56%   86.48%   -0.09%
==========================================
  Files          40       40
  Lines        1407     1398       -9
  Branches      139      139
==========================================
- Hits         1218     1209       -9
  Misses        167      167
  Partials       22       22

Files Changed
Coverage Δ

judgments/tests/test_judgments.py
99.59% <ø> (-0.02%)
:arrow_down:

judgments/views/judgment_hold.py
87.50% <ø> (ø)

judgments/views/judgment_publish.py
84.81% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


This would be really cool if it would be based on buzz tracker style

Meaning having the fantastic sequence editor of buzz! There is already buze for wine, but having just another buzz style alternative is never bad. Did not try your tool yet. There is also hackey trackey lua plugin for Reaper DAW. There you have then also great tracker workflow using all advantages of vsti and DAWs, best of both worlds, thanks to saike! Maybe worth a look at that and improving that instead? Just a side input thought. Btw. thanks for godot! Just discovered it recently indirectly via Blender. Will explore it more in future.
buzz tracker, Reaper DAW, sequetron by Phil Tipping:  music side of things
blender, godot 3: gaming side of things
Now I am trying to figure out, how to create a game, at the push of a button, using some prepared resources on harddisk, like 100 .png images, 100 .svg files, 100 lines of text in a .txt file. Goal would be designing various games in such a way, they would just auto-import those resources and the game would immediately start. You want a different variant, just replace those resources, re-generate, it would be a similar game, but not exactly same. One could imagine each such game like a scene maybe? Any tools which can do this already?

Sorry, not intending to do something like Buzz.


Addition of simple date mining from the title and the abstract and te…

…sts for the function. Closes gh-18, might need some more work. For layers that have no reliability, the script seems not to update the date. Not sure also about regex testing to extract dates from strings.

@jmwenda sorry I was late with this. Can you resend it after merging?
thanks a lot


fix grammar conflict in web-features.md

"an entire JavaScript APIs" doesn't work because it simultaneously requires singularity and plurality.
Since the sentence begins with a very small feature consisting of a singular property and a singular technology and works its way up to a vast feature consisting of some of everything, I'm guessing that the API is meant to be singular—giving two more simple examples before jumping to the complex example.

@ddbeck Thank you!


Create Archetypes Values for Controller types

@shanshanhsieh i will create the Input excel datafile and shapefile to refer to. It includes temperatures of operation of emission systems. Then i will pass it to you.

@shanshanhsieh the work is in branch i159

@JIMENOFONSECA there should actually be two control types, ex. e_ctr_c and e_ctr_h, for heating and cooling respectively, instead of one control type for both heating and cooling emission systems.
could you please add another control field? thanks

We will leave this issue on whether implementing separate control systems for cooling and heating emission systems in buildings until more research is done. @martin-mosteiro , would this be something you want to find out as well?

@shanshanhsieh absolutely!

@shanshanhsieh @martin-mosteiro is anybody working on this? is the level of priority adequate?

@martin-mosteiro could you add the estimate, thanks?

@shanshanhsieh @martin-mosteiro is thi sstill an issue? please close if not...


Typecasting?

connexxion seems to pass Swagger integers as unicode strings. At least a note in the docs would be a start.

Depends on what you mean, type casting "partly" works, see e.g. https://github.com/hjacobs/connexion-example
Can you describe your situation / use case where it fails?

To reproduce:
swagger.yml:
swagger: '2.0'
host: localhost
paths:
  '/foo/{id}':
    get:
      operationId: api.get
      parameters:
        - in: path
          name: id
          type: integer
          format: int64

api.py:
#!/usr/bin/env python2.7
import connexion

def get(id):
    return str(type(id))

app = connexion.App(__name__)
app.add_api('swagger.yml')
application = app.app

if __name__ == '__main__':
    app.run(port=8080)

http http://localhost:8080/foo/1

Actual:
HTTP/1.0 200 OK
Content-Length: 18
Content-Type: application/json
Date: Sat, 05 Dec 2015 17:53:49 GMT
Server: Werkzeug/0.11.2 Python/2.7.6

"<type 'unicode'>"

Expected:
HTTP/1.0 200 OK
Content-Length: 18
Content-Type: application/json
Date: Sat, 05 Dec 2015 17:53:49 GMT
Server: Werkzeug/0.11.2 Python/2.7.6

"<type 'int'>"

Sidenote, this also works ("<type 'unicode'>"):
swagger: '2.0'
host: localhost
paths:
  '/foo/{id}':
    get:
      operationId: api.get
      parameters:
        - in: path
          name: id
          type: fantasia

It would be nice if connexion was a little more defensive, this way typos can lead to hard to track bugs.

True, that's a missing feature: Flask route converters should be supported http://flask.pocoo.org/docs/0.10/api/#url-route-registrations

In the mean time, a small note in the README might save new users  some time. I myself was baffled when a pymongo.find_one() operation did not yield any results because the argument was not an integer. Naive, perhaps, but still.

@kleijnweb my personal recommendation (off topic): stay away from MongoDB :smirk: (we replaced a 5 node MongoDB cluster with a single node PostgreSQL :elephant:)

In general we need more documentation and tutorials for Connexion...

It's only a dummy backend for an AngularJS app, so yeah, not worried about that.
Far as docs go, a simple example in the README with a note about how the arguments are passed (including type coercion -- or lack thereof) would go long way.

@kleijnweb you can check my pull request #105 which adds type casting for int and float path parameters using Flask path converters

changes now released in https://github.com/zalando/connexion/releases/tag/1.0.25

@hjacobs What about arrays/dicts? Difficult?

@kleijnweb Flask supports custom "converts" in the path, so I don't think it's difficult --- I'm not sure what the semantics are for other types in "path" parameters (need to check Swagger spec)


climc update: google account update

What this PR does / why we need it:

climc update: google account update

Does this PR need to be backport to the previous release branch?:
If yes, write the release branches name in the below format and submit the related cherry-pick PR:

release/3.8

/area climc
/cc @zexi @swordqiu

/lgtm

/approve


[node-pod] Add State of Node to Meetups

Description

Meetups should add State of Node

@kenslachtajr @dustinsgoodman
Is the State of Node meetup an event that is not yet published?
I don't see it on the meetup pages.
What information should I add to the node entry?

We don't call it out specifically but it's part of our State of the Web series. This was the last one: https://www.youtube.com/watch?v=H4qu-_Hl_DA
I'd use the state of the web as the website.


Add note to push-api for Chrome about non-standard Web Push service

Chrome currently uses GCM instead of standard Web Push.

It don't believe this is the case anymore (unless I misunderstood something?) https://www.chromestatus.com/features/5748876422152192


Is the AEM extension is supported for RHEL8?

[Enter feedback here]
Is the AEM extension is supported for RHEL8?

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

ID: 3ea8dbf7-4305-1342-c073-8c7b20564fd3
Version Independent ID: a3692496-1cf4-854c-94da-419058bd3e60
Content: Azure Virtual Machines deployment for SAP NetWeaver - Azure Virtual Machines
Content Source: articles/virtual-machines/workloads/sap/deployment-guide.md
Service: virtual-machines-sap
GitHub Login: @MSSedusch
Microsoft Alias: sedusch

It looks like this question has already been resolved, so I'm closing this issue. #please-close


More new modules

Some more ideas for noise modules which may be useful but are expensive or impossible to simulate with existing modules. See #88 for details on procedure for adding new modules.

Pixellate: Pixellates a source module by returning the source value at regularly spaced points. Essentially P(x,y,z) = P.S(floor(x), floor(y), floor(z)). Should have frequency and offset parameters.
Mosaic: Similar to Pixellate, but using Voronoi regions rather than cubic regions. Ideally this would be derived from libnoise's Voronoi module, but that module is not well-encapsulated and would have to be refactored (see https://github.com/qknight/libnoise/blob/master/src/module/voronoi.cpp#L37). So, the options are to make an MIT version of the algorithm or to fork libnoise. It would also be nice to have move options for Voronoi-like modules:

Alternative metrics (L_p with p parametrised).
Vector Frequency or Spacing instead of scalar. This would allow 2D Voronoi diagrams to be simulated.

Blur/Convolve?: Evaluates the source module at several offset points and returns a weighted average of them. The user controls the offset and weight of each point, similar to control points in the Curve module.
CornerSelector: Required for #32. Has 6 source module slots (some of which may be empty), and an integer attribute Corner. Returns the value of the source module specified by Corner.
Sine: Outputs the sine of the source module. Currently has to be approximated with Curve, and has a limited input range. Maybe also Cosine, Tangent. Perhaps trigonometric functions should be in one module with an enum and switch?
Modulo: Outputs the remainder when dividing the first source module by the second (fmod).
Floor: Outputs the floor of the source module.
?

It might be worthwhile to move some or all of the Wangscape noise modules into a libnoise fork.

These can be split up, clarified, and into Hacktoberfest issues (except perhaps the Voronoi-related ones). The module adding guide should be expanded and clarified with links and examples.


Fix BF16 CPX code

Tested for following:
cpx BF16  , cpx SP  ,cpx DP  , cpx BF16F32  , clx BF16, clx BF16F32

OK, then after removal of the eltwise stuff, this can be merged

Reverted eltwise changes


Cursor performance

This gives significant improvements to the debug_info benchmark (see numbers in the commits).
I haven't updated any of the examples to use current_ref, because it is quite unergonomic, due to the borrow checker: you can't call next_dfs and next_sibling while you still have a reference to the current entry. See the changes in bench.rs for example. It's also much worse if you want to use while instead of loop.
So we probably need to rethink the EntriesCursor API. I think the best way will be to avoid the user needing to call current_ref, and instead next_dfs and next_sibling should return the entry ref too. This means we need to change them to allow them to return the first entry when they are first called, rather than immediately skipping the current entry. This is more in line with how the next method works for iterators. I'm open to other ideas though, since I don't know what requirements you had when designing the current API.

It's also worth noting that prior to these changes, the check for current.after_attrs.get() in next_dfs never succeeded, because the caller always iterated attributes on a clone.

It's also worth noting that prior to these changes, the check for current.after_attrs.get() in next_dfs never succeeded, because the caller always iterated attributes on a clone.

D'oh... Good catch.

So we probably need to rethink the EntriesCursor API. I think the best way will be to avoid the user needing to call current_ref, and instead next_dfs and next_sibling should return the entry ref too. This means we need to change them to allow them to return the first entry when they are first called, rather than immediately skipping the current entry. This is more in line with how the next method works for iterators. I'm open to other ideas though, since I don't know what requirements you had when designing the current API.

This sounds good to me. I didn't have any requirements other than a vague idea that by breaking things apart we could provide better performance than when they were stuck together, and that we could later provide nicer iterators (eg DfsIter and SiblingsIter) on top of this that maybe sacrificed a little perf for the nice API. If my inkling was way off mark, then lets definitely change things up!

Going to go ahead and merge this and we can do follow ups as needed.


Mensa plugin gives no response when API response is empty

The plugin should report, if nothing was returned by the API.

tried to get in contact with openmensa, i hope they answer soon. it seems like someone use a wrong parser and push wrong informations to openmensa. A workaround would be to push always right informations if wrong informations get online.


[red-knot] condensed display for literal unions

We should display Literal[1, 2] rather than (Literal[1] | Literal[2]).
This should also work if there are other elements in the union, e.g. (Literal[1, 2] | None)

This should also work if there are other elements in the union, e.g. (Literal[1, 2] | None)

(None itself is actually a valid argument to Literal according to PEP 586, so this example could theoretically be flattened further to Literal[1, 2, None]. But people very rarely do that, and we decided somewhere sometime at typeshed that we never wanted to use that. Literal[1, 2] | None seems clearer.)


set DisableNonReplicatableQueries before sandboxing subcluster during online upgrade

mostly based on the work in https://github.com/vertica/vertica-kubernetes/pull/831 plus fixing merge conflicts, etc

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: jizhuoyu:white_check_mark: cchen-vertica:white_check_mark: roypaulin:white_check_mark: chinhtranvan:white_check_mark: fenic-fawkes:x: spilchenspilchen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

You don't need to modify the added test step(36). It looks good.
Once you fixed the conflicts and passed the test, I will approve it.


Firing trigger/moving to state inside OnEntry deadlocks

The OnEntry handler for TestState.C is never called.
        public enum TestState
        {
            A,
            B,
            C
        }

        public enum TestTrigger
        {
            TriggA,
            TriggB,
            TriggC
        }

        [Test]
        public async Task Test()
        {
            QueuedAwaitableStateMachine<TestState, TestTrigger> stateMachine = null;
            AwaitableConfiguration<TestState, TestTrigger> config = StateMachineFactory.CreateAwaitableConfiguration<TestState, TestTrigger>();

            config.ForState(TestState.A)
                .OnEntry(async () =>
                {
                    await stateMachine.FireAsync(TestTrigger.TriggB);
                })
                .Permit(TestTrigger.TriggB, TestState.B);

            config.ForState(TestState.B)
                .OnEntry(async () =>
                {
                    await stateMachine.FireAsync(TestTrigger.TriggC);
                })
                .Permit(TestTrigger.TriggA, TestState.A)
                .Permit(TestTrigger.TriggC, TestState.C);

            config.ForState(TestState.C)
                .OnEntry(async () =>
                {
                })
                .Permit(TestTrigger.TriggB, TestState.B)
                .Permit(TestTrigger.TriggA, TestState.A);

            stateMachine = new QueuedAwaitableStateMachine<TestState, TestTrigger>(TestState.A, config);

            Assert.AreEqual(TestState.A, stateMachine.CurrentState);
            await stateMachine.FireAsync(TestTrigger.TriggB);
            Assert.AreEqual(TestState.B, stateMachine.CurrentState);
        }

Seems to be because the locking/monitors in QueuedAwaitableStateMachineBase does not support this scenario.
StartQueueIfNecessaryAsync() is called on the nested transition/trigger but never actually processes the queued actions.

One idea I had was to track the async context such that when we enter the nested FireAsync() we can actually process the queue.The InterlockedMonitor is entered but we don't know that it is "us" that did it.
I tried some async primitives from AsyncEx but could not get it working.
The I tried AsyncLocal from .net 4.6, to simply keep track of whether the monitor is entered or not, and then checking that in StartQueueIfNecessaryAsync().
This works, but I have no idea if it's correct...
'''
// Author: Prasanna V. Loganathar
// Created: 09:55 16-07-2015
// Project: LiquidState
// License: http://www.apache.org/licenses/LICENSE-2.0
using System;
using System.Collections.Immutable;
using System.Threading;
using System.Threading.Tasks;
using LiquidState.Awaitable.Core;
using LiquidState.Common;
using LiquidState.Core;
namespace LiquidState.Awaitable
{
public abstract class NestedQueuedAwaitableStateMachineBase_<TState, TTrigger> :
RawAwaitableStateMachineBase<TState, TTrigger>
{
private IImmutableQueue<Func> actionsQueue;
private int queueCount;
private InterlockedBlockingMonitor queueMonitor = new InterlockedBlockingMonitor();
private InterlockedMonitor monitor = new InterlockedMonitor();
private AsyncLocal asyncMonitor = new AsyncLocal();
    protected NestedQueuedAwaitableStateMachineBase_(TState initialState, AwaitableConfiguration<TState, TTrigger> config)
        : base(initialState, config)
    {
        actionsQueue = ImmutableQueue.Create<Func<Task>>();
    }

    public override async Task MoveToStateAsync(TState state,
        StateTransitionOption option = StateTransitionOption.Default)
    {
        var flag = true;

        queueMonitor.Enter();
        if (monitor.TryEnter())
        {
            asyncMonitor.Value = true;

            if (queueCount == 0)
            {
                queueMonitor.Exit();
                flag = false;

                try
                {
                    await base.MoveToStateAsync(state, option);
                }
                finally
                {
                    queueMonitor.Enter();
                    if (queueCount > 0)
                    {
                        queueMonitor.Exit();
                        var _ = StartQueueIfNecessaryAsync(true);
                        // Should not exit monitor here. Its handled by the process queue.
                    }
                    else
                    {
                        queueMonitor.Exit();
                        asyncMonitor.Value = false;
                    }
                }
            }
        }

        if (flag)
        {
            var tcs = new TaskCompletionSource<bool>();

            actionsQueue = actionsQueue.Enqueue(async () =>
            {
                try
                {
                    await base.MoveToStateAsync(state, option);
                    tcs.TrySetResult(true);
                }
                catch (Exception ex)
                {
                    tcs.SetException(ex);
                }
            });

            queueCount++;
            queueMonitor.Exit();
            var _ = StartQueueIfNecessaryAsync();
            await tcs.Task;
        }
    }

    public override void Resume()
    {
        base.Resume();
        var _ = StartQueueIfNecessaryAsync();
    }

    public override async Task FireAsync<TArgument>(ParameterizedTrigger<TTrigger, TArgument> parameterizedTrigger,
        TArgument argument)
    {
        if (!IsEnabled)
            return;

        var flag = true;

        queueMonitor.Enter();
        if (monitor.TryEnter())
        {
            asyncMonitor.Value = true;

            // Try to execute inline if the process queue is empty.
            if (queueCount == 0)
            {
                queueMonitor.Exit();
                flag = false;

                try
                {
                    await base.FireAsync(parameterizedTrigger, argument);
                }
                finally
                {
                    queueMonitor.Enter();
                    if (queueCount > 0)
                    {
                        queueMonitor.Exit();
                        var _ = StartQueueIfNecessaryAsync(true);
                        // Should not exit monitor here. Its handled by the process queue.
                    }
                    else
                    {
                        queueMonitor.Exit();
                        monitor.Exit();
                        asyncMonitor.Value = false;
                    }
                }
            }
        }

        if (flag)
        {
            // Fast path was not taken. Queue up the delgates.

            var tcs = new TaskCompletionSource<bool>();
            actionsQueue = actionsQueue.Enqueue(async () =>
            {
                try
                {
                    await base.FireAsync(parameterizedTrigger, argument);
                    tcs.TrySetResult(true);
                }
                catch (Exception ex)
                {
                    tcs.TrySetException(ex);
                }
            });
            queueCount++;
            queueMonitor.Exit();
            var _ = StartQueueIfNecessaryAsync();
            await tcs.Task;
        }
    }

    public override async Task FireAsync(TTrigger trigger)
    {
        if (!IsEnabled)
            return;

        var flag = true;

        queueMonitor.Enter();
        if (monitor.TryEnter())
        {
            asyncMonitor.Value = true;

            // Try to execute inline if the process queue is empty.
            if (queueCount == 0)
            {
                queueMonitor.Exit();
                flag = false;

                try
                {
                    await base.FireAsync(trigger);
                }
                finally
                {
                    queueMonitor.Enter();
                    if (queueCount > 0)
                    {
                        queueMonitor.Exit();
                        var _ = StartQueueIfNecessaryAsync(true);
                        // Should not exit monitor here. Its handled by the process queue.
                    }
                    else
                    {
                        queueMonitor.Exit();
                        monitor.Exit();
                        asyncMonitor.Value = false;
                    }
                }
            }
        }

        if (flag)
        {
            // Fast path was not taken. Queue up the delgates.

            var tcs = new TaskCompletionSource<bool>();
            actionsQueue = actionsQueue.Enqueue(async () =>
            {
                try
                {
                    await base.FireAsync(trigger);
                    tcs.TrySetResult(true);
                }
                catch (Exception ex)
                {
                    tcs.TrySetException(ex);
                }
            });
            queueCount++;
            queueMonitor.Exit();
            var _ = StartQueueIfNecessaryAsync();
            await tcs.Task;
        }
    }

    public void SkipPending()
    {
        queueMonitor.Enter();
        actionsQueue = ImmutableQueue<Func<Task>>.Empty;
        queueCount = 0;
        queueMonitor.Exit();
    }

    private Task StartQueueIfNecessaryAsync(bool lockTaken = false)
    {
        if (lockTaken || asyncMonitor.Value || monitor.TryEnter())
        {
            return ProcessQueueInternal();
        }

        return TaskHelpers.CompletedTask;
    }

    private async Task ProcessQueueInternal()
    {
        // Always yield if the task was queued.
        await Task.Yield();

        queueMonitor.Enter();
        try
        {
            while (queueCount > 0)
            {
                var current = actionsQueue.Peek();
                actionsQueue = actionsQueue.Dequeue();
                queueCount--;
                queueMonitor.Exit();

                try
                {
                    if (current != null)
                        await current();
                }
                finally
                {
                    queueMonitor.Enter();
                }
            }
        }
        finally
        {
            // Exit monitor regardless of this method entering the monitor.
            monitor.Exit();
            queueMonitor.Exit();
        }
    }
}

public sealed class NestedQueuedAwaitableStateMachineBase<TState, TTrigger> :
    NestedQueuedAwaitableStateMachineBase_<TState, TTrigger>
{
    public NestedQueuedAwaitableStateMachineBase(TState initialState,
        AwaitableConfiguration<TState, TTrigger> awaitableConfiguration)
        : base(initialState, awaitableConfiguration)
    {
    }
}

}
'''

Hello @oysteinkrog,
Okay, there's both good news and bad news..
But before that, let me quickly rule out the problem with the above AsyncLocal scenario. It will break the machine's state coherency. In the example you gave above, the end state will not be StateC as you'd expect, but it'll be StateB.
This is because, the machine pre-determines the set of actions to be executed for each state transition. Now, your transition will have to StateB (but now, only if it is valid at the point of validity checks - the current state of the machine is still one state behind. However, let's say I can easily fix this by just changing the order of the execution (though now it breaks the semantics, as OnEntry is supposed be its entering.. But not quite done yet. Machine successfully enters the state when its OnEntry doesn't throw and is successful) So, the question is, should I? Its a deep dark work once you enter the nested state transition world in this manner, without sacrificing on performance).
It was a conscious decision I made to favor performance when I built it. I would however be open to re-design if people do see it being used heavily.
Now to the problem,
Lets approach this problem from a semantic perspective.
So, lets say StateA triggers StateB inside its Entry. Now I end up with many hidden problems that the state machine cannot actually solve on its own. Why? When StateA triggers StateB, the state of the machine is in transition.
Now the machine has already determined a bunch of actions to be done, its trigger functions if any, exit of the past state and so on.
Now, if I do a nested transition. Which exit do I do first? Or complete take a different approach and do I exit twice?
These are all valid options in different scenarios which you may or may not desire. There are also more subtleties when you think about it how it all integrates together.
Now, by no means is it impossible to make these choice above. But the problem is, what right for one will be wrong for the other. There's no way to come up with one optimal solution.
So, this is a necessary limitation to completely avoid doing this. However, I should update the docs on semantics of each machine, and state that this is a deliberate choice. I tried thinking of throwing exception in certain scenarios, but I couldn't come up with a clear way to detect if it is indeed the right scenario. I'll try to update the docs this weekend. I think I it would be fair to call this behavior being undocumented as a bug.
Okay, so that was the bad news.. Now, to the good one,
There's nothing stopping you from doing these. The way I do it is this. I simply do not call await. This way the action gets queued, and everything's stays sane and simple to reason with. I'd suggest that approach of thinking for scenarios where you cannot avoid it.
But there will scenarios where you'll want to entire set to complete and respect the awaitable context. That is completely lacking at this point. I see a potential solution to be to work a context into the machine where these tasks can be attached to,  so that you're await while firing a machine includes these tasks as a single unit of transition.
I've been want to bring contexts into the machine for a while, but unfortunately didn't get the time to do it yet. :(
Or an alternate way to solve the problem would be have a machine execution core that simply attaches creates a TaskCompleteSource for each transition. So, now dispatches of async completion could happen with an internal response to the state change events. But this means, you're going to have to create TaskCompleteSource every time (currently, the machine tries to take a fast path when there is no concurrency issue, and tcs is only created when required if it cannot.
Anyway, I don't see an easy way for you to exactly what you want out of the box. I hope this gives a good insight into the problems.

Thank you so much for the great explanation, I understand now why this is a tricky problem :)
What I was trying to do is have the state machine represent the "interval"/transition state.
E.g. if the state machine represents a connection to a device, I would have both an "opening" and an "open" state.
Now I think I think that maybe the easiest way to do this is to it separate from the statemachine :)


chore: update cSpell globs and fix typos

What kind of change does this PR introduce?

Did you add tests for your changes?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

Thanks for the review @sokra! Looks like the GitHub Actions might still need to get approved to finish the green checkmark

Thanks


Using the Wiki button on your current Slayer task could open entry for that monster

As a newer player, often when I start a slayer task I'm not always sure where to locate a monster, it's weaknesses and other important information required to complete the task. Typically I will just open the wiki and search for that monster, which is pretty easy.
The new wiki button is really helpful for learning more about the game as I'm walking through the world, however it's not so useful for slayer because I can't directly look up a monster until after I've found the monster somewhere in the world.
It would be nice if I could click on the wiki button, then click on the slayer task progress icon, and have the wiki open up to the monster I'm currently assigned.

The wiki button also has a right click option to bring up a search option to the wiki. It's an extra few keystrokes but you could just do that in case what you're describing doesn't get implemented.

Duplicate of #8229


Rocket Animation

Description 📜
Please include a summary of the change and which issue is fixed.
Fixes #1370

Type of change 📝
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Add a new project
[] Modified or enhanced the existing projects
[] Improve the readme of any project

Domain of Contribution 📊
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] Project using only HTML & CSS
[] Project using HTML, CSS & JS
[] Project using ReactJs

Checklist ✅
Please delete options that are not relevant. In order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

Kindly Remove Labels
cc: @Mrjoy832


Fix uploads in dynamic folder mode

Another issue i ran into seems to be related to dynamic folder mode. In my case, this is enabled, and it seems to be the new default and only option for new Cloudinary accounts. I noticed, however, that you apparently dropped support for it in the second Version of this plugin? As soon as I'll be upgrading to Craft 5 this would likely become an issue, so i'd love to hear your thoughts on this. Since i now rely on this plugin i'd obviously be happy to help :)
As for now, i noticed that at least the upload and the rename webhooks do not work with dynamic folder mode, presumably because the parameter for the folder is called asset_folder instead of folder?
I have no way now to verify this with an account that is in 'fixed' folder mode, but this seems to do the trick for me.
Thanks!

Hello Joris,
Thank you for your contributions to this plugin! I did decide to drop support for dynamic folders, unaware that Cloudinary would soon make it the default for all new accounts, both free and paid. Recently, I’ve been giving dynamic folders another try to see if I can make them work smoothly with Craft’s file organization system. I've managed to resolve most issues, but one challenge remains—related to a delay in propagation on the Cloudinary CDN when moving folders with assets inside. I’m not yet sure if I can fully overcome this, but I’m considering releasing an updated version with support for dynamic folders mode.
I’ll need a few more days to assess what’s feasible. After that, I’ll work on incorporating your additions and fixes into either the current or a new version of the plugin.

Hi Thomas,
Thanks for your response. Please let me know once you've decided on how you'd like to proceed or in case you'd like my support with anything related to this. I've just started out debugging the issues i encountered, but think i slowly start to get a better understanding of the plugin and the flysystem integration. So i'd be happy to coordinate any further effort.
In case you decide to go with a new version, i may consider backporting the updates to the Version running with Craft 4. I think a major difference to v5 are the shared filesystems using different subpaths?

Hi Thomas, just spotted this issue and, even if it's not complete could you give us an update on how feasible you think support for DF is based on your latest work?
When we spoke about this some months ago you had reached the conclusion that it would not be possible and I'm in the process of migrating a large cloudinary repository to static folders mode so as to be compatible with your plugin (which involves reinitialising the repo and migrating the assets with their metadata).
If it looks like this might not be necessary, do let me know! DF would be much better for our application since assets are moved around often and the customer facing parts query cloudinary directly but are statically generated (ie. the asset urls are statically generated, but in static folders mode would become invalid if asset is moved). There would be a workaround for this, to query each asset at runtime by public id if the url bounces, but it would have a negative effect on SEO and caching etc).
Any update would be appreciated so we can make the right plan, thanks!

Thanks for the info and good luck, sounds messy!


build(deps-dev): fix security vulnerabilities

https://github.com/kaciakmaciak/react-query-subscription/issues/31

Codecov Report

Merging #32 (ca4ced7) into master (0654576) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #32   +/-   ##
=======================================
  Coverage   98.55%   98.55%
=======================================
  Files           3        3
  Lines          69       69
  Branches       12       12
=======================================
  Hits           68       68
  Misses          1        1

Flag
Coverage Δ

unittests
98.55% <ø> (ø)

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

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 0654576...ca4ced7. Read the comment docs.

:tada: This PR is included in version 1.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Replace viraintel with zdresearch

Checklist

[x] I have followed the Contributor Guidelines.
[x] I have added the relevant documentation.
[x] My branch is up-to-date with the Upstream master branch.

Changes proposed in this pull request
I have replaced most of the viraintel occurences with zdresearch.
Your development environment

OS: Ubuntu
OS Version: 16.04
Python Version: 2.7.12

Hi,
thanks for your contribution, PR Merged.
Best Regards.


Create DaprD docker image instead of one dapr docker image

Describe the proposal
Currently, Dapr docker image includes 4 binaries, such as operator, daprd, injector, and placement. Therefore, each user app includes this big container as a sidecar in the pod. To make it lighter, having the separate image makes sense.

I am interested in this issue, could you explain it future as I am not familiar with dapr codebase

Currently, the container image which is generated contains 4 binaries:

daprd
injector
operator
placement.

The sidecar which runs alongside user code needs only 1 binary daprd. The work item is to change the container generation to have only daprd in one container and rest in other. This would need changes in helm charts and cli as well.
Helm chart changes: Modify charts to run new container.
CLI changes: change "dapr init" command to get new container for placement service.


LiveQuery seems to bypass Record-level Security limit!!

OrientDB Version: 2.2.13 community
Java Version: 1.8.0_101
OS: linux
Expected behavior
When a record created, LiveQuery should catch nothing, if the user is restricted to access this record, even if he can access the class.
Actual behavior
normal select operation returns nothing, but LiveQuery returns the record.
Steps to reproduce
---------step 1--------------------------------------------------------------
create class box
alter class box superclass +ORestricted
//orientjs programm
db.liveQuery('LIVE SELECT FROM box') .on('live-create', function(data){     console.log("new record->",data.content); });
---------step 2--------------------------------------------------------------
insert into box set title=''not-for-z2'
orientjs programm output:
D:\test>node livequery.js
user: z2 is listening...
new record-> { '@type': 'd',
'@class': 'messageBox',
timestamp: Fri Mar 03 2017 18:17:42 GMT+0800 (中国标准时间),
title: ''not-for-z2',
_allow: [ { [String: '#5:0'] cluster: 5, position: 0 } ] }
---------step 3--------------------------------------------------------------
orientdb console output:
Connecting to database [remote:router.zhiguanyun.com/router] with user 'z2'...OK
orientdb {db=router}> select from box where title=''not-for-z2'
0 item(s) found. Query executed in 0.016 sec(s).

hi @zhiguan-hanks
thanks for this issue. I've pushed a fix. It will be available in OrientDB 2.2.18
Thanks


Add instructions for verifying git tags

All the git tags are signed, but we do not provide instructions for verifying this.

https://github.com/coldhakca/coldkernel/commit/663a48d0d86daf123f4b2c77fc54736f81934cba


Audit Log Flow only returns 100 items

I deployed the Admin | Sync Audit Logs flow and it is only returning 100 items from the Audit Logs. We have a very large environment and we go through more than 100 Power Apps Launches a day. How can I increase the return limit?

You would need to change the scheduled Flow so that it doesn't run daily but hourly for example, and modify the start and end time also to get hourly results.

I will try that but with our large environment I could easy get more than 100 app launches in a hour. Would be possible to add the ResultSize parameter to the custom connector, so we could extend the limit past 100? I have done this in PowerShell with Search-UnifiedAuditLog and it has worked well.

I was able to configure the flow to run every hour and adjusted the Start and End dates to include time, so that it was only requesting logs entries for the last hour and I was still hitting the 100 item limit. We have a large user base and a lot of different apps, so we need to be able to grab more than 100 events. If we can get the ResultSize parameter added then we could significantly increase the result size but we would still need to run the job every hour to ensure we are getting all the Power Apps launch events.

I have just looked at this, and was actually not able to replicate the issue - I've looked at a date span of three months in my demo tenant, and the ResultCount was 667.
If you go to the Test tab in the Custom Connector and enter a date there, does it always return just 100?

This solution is in use with some of our larger customers, and they've not reported an issue with the Audit Log return size.

If I used the test tool I get a Result count of "ResultCount": 26361 but when I copy out the body and put into Notepad++ and count the number "Operations": "LaunchPowerApp" there is only 100 events.
What happens when you copy out the body and count the number of items? Are you getting 667 or 100?

I have spoken to a colleague of mine and we had one customer configure this in 10 minute intervals to work for their log size. I have not played around with configuring the ResultSize. I have added it to the backlog, in the meantime you should be able to edit the Custom Connector yourself if you wanted to change the config there.

I have played around OpenAPI file and creating a test verison of Audit Log connector. I have been able to get up to 400 events returned but after that pagination kicks in and it gives me an odata.nextLink to go out and grab the next batch of events.
As I play around with the API I am struggling on what settings are available. I have worked with getting the UnifiedAuditLog through PowerShell but that isn't exactly the same as the API. Do you know where I can find the documentation on the Outlook API "/psws/service.svc/UnifiedAuditLog"? I can some information doing online searches but not the complete documentation.

In the latest release, we've switched to using https://docs.microsoft.com/en-us/office/office-365-management-api/get-started-with-office-365-management-apis
The solution is available here
https://github.com/microsoft/powerapps-tools/tree/master/Administration/CoEStarterKit/Audit Log (MFA)
This is a more reliable and supported service, and also supports oauth (rather than just basic auth). I would recommend using this one if you can.

Hi,
I just ran into paging issue in my customer using latest MFA version <IP_ADDRESS> of custom connector. The limit of events in 1 request seems to be 200. We haven't test to run more frequently than once per day yet but I think we should anyway implement paging support in the custom connector and/or flow. The paging happens when response contains NextPageUri header, so I guess need to add another loop which runs until that header is not returned anymore

Great point, adding @Jenefer-PA to work on this fix!

Hi Also having same Issue, Limited to 100 returns with Basic Authentication method when run within the flow and 200 with Oauth. any update on this to make sure we get all details returned for accurate reporting

Sorry we are unable to make the recommended change in custom connectors due to product limitations. You will need to change the frequencies of the run and change the intervals in the flow in order to paginate as recommended by Maneula above.

Closing out old issues without existing action.
Thanks for using the CoE


Installation issue

Hello sir,
Can i install it on windows7?if yes then what is the procedure and commands?

Try to install golang. Then simply do go get!

Sorry for the late response, actually the public project is discontinued. Please check https://github.com/haccer/subjack. I am not maintaining this at the moment.


[BUG] .net not working

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
i don't know if this is the right place for this, but im out of hope
I'm trying to instal Bloxstraps but i keep getting this error that sayd "you need to instad .net" witch i have, i dowloaded version 5.0.0, 6.0.0 and all the others but it just dosent work!
Please help

Did you install the version that I linked? https://aka.ms/dotnet-core-applaunch?missing_runtime=true&arch=x64&rid=win11-x64&apphost_version=6.0.16&gui=true


Can't build due to environment variable

I have tried to build the demo app but my android build can't succeed as android/build.gradle:43 contains an env variable.

Hi, sorry for the late answer. We'll check this and then get back to you soon.

Hi @JsonLinesCode there is a .run folder which contains configurations you can use in Android Studio to run the project.
These configurations are using simple scripts under example/.scripts folder to run the app and set env variables.

It makes sense ! Good luck on future updates.


Modifying the default implementation for MulAcc.

Modified the default implementation of MulAcc, so that the default implementation applies to scalar types that does not necessary implement Copy and num_traits::MulAdd. In particular, this modification enables BigInt and Rational types applicable.
Follows discussion from #323.

Thanks @taketo1024 !


Missing members from V&V Team

The V&V Team should include a few other individuals. Your TA and Dr. Smith are there to help with verifying documents such as the SRS, V&V, etc, in the form of feedback and rubrics/guidelines, hence making them a part of the V&V Team.  Additionally, your supervisor (if you have one) should be described here as well as they can help with validating and verifying the requirements in the SRS.

@samm82

TA and Dr.Smith are documented as stakeholder. Stakeholder involvement in VnV is documented such as walkthroughs. No supervisor.


Add support for typedoc

Goal: provide an easy way to create API documentation for the TS code
https://www.npmjs.com/package/typedoc
https://www.npmjs.com/package/gulp-typedoc
Should be supported by #13 too

References:

https://github.com/AngularClass/angular2-webpack-starter/blob/master/package.json
https://raw.githubusercontent.com/AngularClass/angular2-webpack-starter/master/typings.json
https://github.com/AngularClass/angular2-webpack-starter/blob/master/.gitignore

References:

https://github.com/AngularClass/angular2-webpack-starter/blob/master/package.json
https://raw.githubusercontent.com/AngularClass/angular2-webpack-starter/master/typings.json
https://github.com/AngularClass/angular2-webpack-starter/blob/master/.gitignore

Blocked by: https://github.com/sebastian-lenz/typedoc/issues/130

Blocked by: https://github.com/sebastian-lenz/typedoc/issues/130

Should be ok now


GA admin status "tracking not installed," but GA Plugin is working

This is a minor issue but thought it might be worth mentioning. After an hour of fiddling around, I finally got the GA plugin to report user data on my GitBook, which uses a custom domain. I tried several different settings, and even manually pasted the tracking code into the README.md file. Not sure what caused it to start working, but it finally did.
Oddly, the GA Admin > Property > Tracking Code status page still says "Tracking Not Installed," even though I am receiving GA data. See screenshot. Is this an issue with the GA plugin -- or not?

Worse here: "tracking not installed", AND nothing shows up in the real-time panel.
I have uninstalled the plugin.


ERROR running force:source:retrieve:  WARNING: Infinite loop detected.

Summary
ERROR running force:source:retrieve:  WARNING: Infinite loop detected.
Unable to retrieve anything from source org.  Tried several different orgs and metadata types.  Received this error for all of them.
Steps To Reproduce:
Without connecting to remote GitHub repo

Create a brand new sfdx project from VS Code
Auth a sandbox org
Before connecting to any remote repo, retrieve source in manifest
All metadata is successfully retrieved as expected

Now with a connection to a remote GitHub repo

Create a brand new sfdx project from VS Code
Auth a sandbox org
Before connecting to any remote repo, retrieve source in manifest
All metadata is successfully retrieved as expected
git remote add origin to get remote repo
'git pull' to get repo branch
Retrieve source in manifest now errors out with ERROR running force:source:retrieve:  WARNING: Infinite loop detected.

Now from Command Prompt (after doing the above)

>sfdx force:auth:web:login for the same sandbox org as above
sfdx force:source:retrieve --json -m SomeMetadataType:SomeName
Returned more WARNINGS than can count

WARNING: Flows must be used with a minimum API version of 44.0. Make sure that the sfdx-project.json file is set to use "sourceApiVersion": "44.0" or later, and that you’ve upgraded the flow files per the instructions in the Metadata API Developer Guide: https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_visual_workflow.htm#md_flow_upgrade.

FYI sfdx-project.json api version is set to 48.0
Expected result
When connected to a remote repo and sandbox org, expect the retrieve source in manifest to work as it used to.
Actual result
Receive error: ERROR running force:source:retrieve:  WARNING: Infinite loop detected.
Additional information
VS Code Version:  1.51.1
SFDX CLI Version:  7.81.0-7b953c80d1 win32-x64 node-v12.18.3
OS and version:  Windows_NT x64 10.0.17763
salesforcedx: 50.6.0 (core)

On a related note this command when manually typed is working:
sfdx force:source:retrieve -m CustomObject:customstuff__c
But the Org Browser's Retrieve (Beta) is failing for the exact same object.

@jvolkov we changed the library for deploy/retrieve (and in beta, [push/pull] (https://github.com/forcedotcom/cli/issues/1258)).
That should fix the problem.  If it doesn't, please respond.


Add menubar

Given the multipage nature of the gallery we'll need a menu bar w/ at least:

login/logout
if logged in: overview over personal & shared notebooks
gallery itself

menu bar now has:

[x] login if not logged in
[x] link to dashboard if logged in

dashboard has:

[x] dashboard buttons for deleting/editing existing notebooks
[x] generally uses buttons instead of plaintext links


install error

hello,
I tried to install this and got this error
(gaming) jcasanova@casanova-master:~/Desktop/git_test/gaming_django/gaming/src$ ./manage.py runserver
Performing system checks...
Unhandled exception in thread started by <function check_errors..wrapper at 0x7fc80a5986a8>
Traceback (most recent call last):
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/utils/autoreload.py", line 226, in wrapper
fn(*args, **kwargs)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/management/commands/runserver.py", line 121, in inner_run
self.check(display_num_errors=True)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/management/base.py", line 374, in check
include_deployment_checks=include_deployment_checks,
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/management/base.py", line 361, in _run_checks
return checks.run_checks(**kwargs)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/checks/registry.py", line 81, in run_checks
new_errors = check(app_configs=app_configs)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/checks/urls.py", line 14, in check_url_config
return check_resolver(resolver)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/core/checks/urls.py", line 24, in check_resolver
for pattern in resolver.url_patterns:
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/utils/functional.py", line 35, in get
res = instance.dict[self.name] = self.func(instance)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/urls/resolvers.py", line 313, in url_patterns
patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/utils/functional.py", line 35, in get
res = instance.dict[self.name] = self.func(instance)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/urls/resolvers.py", line 306, in urlconf_module
return import_module(self.urlconf_name)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/src/mainapp/urls.py", line 22, in
url(r'', include('base.urls')),
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/site-packages/django/conf/urls/init.py", line 50, in include
urlconf_module = import_module(urlconf_module)
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/home/jcasanova/Desktop/git_test/gaming_django/gaming/src/base/urls.py", line 4, in
from views import home
ImportError: No module named 'views'

seems to be an issue only with python3.
(in src/base/urls.py )
anyway.. it should be fixed now


docs: fix typo in bootstrap example

The example to customize the bootstrap config is wrong and leads to an error:
cat eg.yaml| egctl x translate --from gateway-api --to xds  -f -
Error: proto: syntax error (line 1:10): unexpected token [
proto: syntax error (line 1:10): unexpected token [

@akhenakh seeing a conflict, can you rebase and push ?

@akhenakh seeing a conflict, can you rebase and push ?

done

Thanks!


Displaying journal thumbnails on the left

Adding a Configuration Option to display journal thumbnails on the left side of the entry title.

Added this option in the upcoming version 2 release


Can't type the fake this parameter when overloading functions

TypeScript Version:  3.2.0-dev.201xxxxx

Search Terms: this parameter overloading
Code
class Cash {}

interface Cash {
  text (): string;
  text ( text: string ): this;
}

Cash.prototype.text = function ( this: Cash, text?: string ) {
  return text === undefined ? 'foo' : this;
};

Expected behavior:
No errors.
Actual behavior:
There's an error, it says that 'Cash' is not assignable to 'string'. If I remove the this fake parameter it works, am I doing something wrong here?
Playground Link:
Here.
Related Issues: Not that I could find

I am not overly familiar with declaration merging and overloading of functions, but writing this inside the same class produces no errors.
class Cash {
  text(): string;
  text(text: string): this;
  text(text?: string | undefined): string | this {
    return text === undefined ? 'foo' : this;
  }
}

Playgorund link

@csvn you're typing the fake this parameter though.

It wasn't a problem of the language https://stackoverflow.com/questions/52921185/typing-this-parameters-in-overloaded-function-involving-declaration-merging?noredirect=1#comment92751900_52921185


Wait to query CoreNLP API to avoid error messages in nb

Haven't seen this anymore, closing for now


Streamline dedup process as part of the photo scan

As described in the Discord thread message, the dedup via django admin fails for very large photo collections. So, a more robust solution is required:
Option A:

Add another button in the UI that causes a scan like process, that runs over all photos and dedupes faces
Option B:
Have the dedup operation part of the scan (so when scanning a photo, the last step is the dedup)
Optionally, extend the above and add a checkbox that defaults to off - and only if it is checked, the scan will do the dedup as well (kinda opt-in and not the default)

I wrapped the Django admin task in an AsyncTask, which means that it should work now even with large collections. Would be great if you could test it :)

I tried on 2024w14p1 but no change... Is it merged?

Hey @derneuere , testing on 2024w14p1 made no difference. Clicking on "Select all 91465 photos" does nothing

Yes, it was merged, but apparently it does not work yet. Will have to create more tests for it, but do not have the time for that atm :)


Decide docuementation library / format

Done, decided upon https://github.com/documentationjs/documentation#documentation and started implementing


Move to the latest version of elm-css

Right now I'm using a dump of elm-css from master from before things were rewritten. We want to be on the latest version of elm-css, which should be a simple upgrade. It'll look something like this:

Update the elm-css folder to have the latest dump of non-test stuff from here
Ensure it compiles and fix any compile errors

In theory, it should be a trivial upgrade.

What reason why you don't want to use elm-css as a package?

It wasn't published at the time :)

Sorry, I have removed my question by accident.
Using elm-css as a package causes some troubles with module imports, I'll see if I can get to work during next few days.

I've hit the issue with elm-css,  I think it's somehow related to elm-css#87. I'll see if I can fix that.


Websocket error TextMeshPro downgrade DotNet3.x

Please fill in the following fields:
Unity editor version: 2019.1.1f1
Firebase Unity SDK version: 5.7.0
Firebase plugins in use (Auth, Database, etc.): Auth, Database, Storage
Additional SDKs you are using (Facebook, AdMob, etc.):
Platform you are using the Unity editor on (Mac, Windows, or Linux): Mac
Platform you are targeting (iOS, Android, and/or desktop): iOS/Android
Please describe the issue here:
(Please list the full steps to reproduce the issue. Include device logs, Unity logs, and stack traces if available.)
Hello, I am trying to debug database using Unity Editor on 2019.1.1f1. I get a websocket error using DotNet4 runtime - so I switch to 3.x runtime this works... but for it to work I have to delete the TextMeshPro package in Unity. The problem is my project is completely dependent on the TextMeshPro so I cannot downgrade the runtime for editor debug.
What do I do from here?
Please answer the following, if applicable:
Have you been able to reproduce this issue with just the Firebase Unity quickstarts (this GitHub project) YES

What's the websocket error you get? Could you provide a device log?

Hi thanks for responding. The 2 errors:
5/8/2019 11:50:30 PM [Error] WebSocket: ws_0 - could not connect
System.Net.Sockets.SocketException (0x80004005): No route to host
at System.Net.Sockets.Socket.Connect (System.Net.EndPoint remoteEP) [0x000b6] in :0
at System.Net.Sockets.TcpClient.Connect (System.Net.IPEndPoint remoteEP) [0x00033] in :0
at Firebase.Database.Internal.TubeSock.WebSocket.CreateSocket () [0x000ea] in Z:\tmp\tmp.UJsYTuiaFJ\firebase\database\client\mono\Database\Internal\TubeSock\WebSocket.cs:323
UnityEngine.Debug:LogError(Object)
Firebase.Platform.FirebaseLogger:LogMessage(PlatformLogLevel, String) (at Z:/tmp/tmp.lznH2AxwEU/firebase/app/client/unity/src/Unity/FirebaseLogger.cs:19)
Firebase.Unity.UnityLoggingService:LogMessage(PlatformLogLevel, String) (at Z:/tmp/tmp.lznH2AxwEU/firebase/app/client/unity/src/Unity/UnityLoggingService.cs:13)
Firebase.Database.Internal.Logging.DefaultLogger:Error(String, String) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/DefaultLogger.cs:68)
Firebase.Database.Internal.Logging.DefaultLogger:OnLogMessage(Level, String, String, Int64) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/DefaultLogger.cs:32)
Firebase.Database.Internal.Logging.LogWrapper:Error(String, Exception) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/LogWrapper.cs:36)
Firebase.Database.Internal.TubeSock.WebSocket:LogError(String, Exception) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:143)
Firebase.Database.Internal.TubeSock.WebSocket:CreateSocket() (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:325)
Firebase.Database.Internal.TubeSock.WebSocket:RunReader() (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:386)
Firebase.Database.Internal.TubeSock.Runnable101:Run() (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:516)
Google.Sharpen.Thread:InternalRun() (at Z:/tmp/tmp.vANha6D0V6/third_party/dotnet_src/sharpen/Sharpen/Thread.cs:80)
System.Threading.ThreadHelper:ThreadStart()
——————
5/8/2019 11:50:30 PM [Error] WebSocket: ws_0 - WebSocketException during handshake
Firebase.Database.Internal.TubeSock.WebSocketException: unknown host: yellows.firebaseio.com ---> System.Net.Sockets.SocketException: No route to host
at System.Net.Sockets.Socket.Connect (System.Net.EndPoint remoteEP) [0x000b6] in :0
at System.Net.Sockets.TcpClient.Connect (System.Net.IPEndPoint remoteEP) [0x00033] in :0
at Firebase.Database.Internal.TubeSock.WebSocket.CreateSocket () [0x000ea] in Z:\tmp\tmp.UJsYTuiaFJ\firebase\database\client\mono\Database\Internal\TubeSock\WebSocket.cs:323
--- End of inner exception stack trace ---
at Firebase.Database.Internal.TubeSock.WebSocket.CreateSocket () [0x0010c] in Z:\tmp\tmp.UJsYTuiaFJ\firebase\database\client\mono\Database\Internal\TubeSock\WebSocket.cs:327
at Firebase.Database.Internal.TubeSock.WebSocket.RunReader () [0x0001b] in Z:\tmp\tmp.UJsYTuiaFJ\firebase\database\client\mono\Database\Internal\TubeSock\WebSocket.cs:386
UnityEngine.Debug:LogError(Object)
Firebase.Platform.FirebaseLogger:LogMessage(PlatformLogLevel, String) (at Z:/tmp/tmp.lznH2AxwEU/firebase/app/client/unity/src/Unity/FirebaseLogger.cs:19)
Firebase.Unity.UnityLoggingService:LogMessage(PlatformLogLevel, String) (at Z:/tmp/tmp.lznH2AxwEU/firebase/app/client/unity/src/Unity/UnityLoggingService.cs:13)
Firebase.Database.Internal.Logging.DefaultLogger:Error(String, String) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/DefaultLogger.cs:68)
Firebase.Database.Internal.Logging.DefaultLogger:OnLogMessage(Level, String, String, Int64) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/DefaultLogger.cs:32)
Firebase.Database.Internal.Logging.LogWrapper:Error(String, Exception) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/Logging/LogWrapper.cs:36)
Firebase.Database.Internal.TubeSock.WebSocket:LogError(String, Exception) (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:143)
Firebase.Database.Internal.TubeSock.WebSocket:RunReader() (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:468)
Firebase.Database.Internal.TubeSock.Runnable101:Run() (at Z:/tmp/tmp.UJsYTuiaFJ/firebase/database/client/mono/Database/Internal/TubeSock/WebSocket.cs:516)
Google.Sharpen.Thread:InternalRun() (at Z:/tmp/tmp.vANha6D0V6/third_party/dotnet_src/sharpen/Sharpen/Thread.cs:80)
System.Threading.ThreadHelper:ThreadStart()

On May 8, 2019, at 7:36 PM, jonsimantov<EMAIL_ADDRESS>wrote:
What's the websocket error you get? Could you provide a device log?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/firebase/quickstart-unity/issues/360#issuecomment-490689402, or mute the thread https://github.com/notifications/unsubscribe-auth/AE5HNYWWLAB5G4W4F3WUMN3PUNPZVANCNFSM4HLVNOSA.

Keep the 4.X, this has been a common issue, here's the solution/workaround:
To resolve the compilation error, if you're using .NET 4.x:
Remove or disable the following DLLs for all platforms:
Parse/Plugins/Unity.Compat.dll
Parse/Plugins/Unity.Tasks.dll
Enable the following DLLs for all platforms:
Parse/Plugins/dotNet45/Unity.Compat.dll
Parse/Plugins/dotNet45/Unity.Tasks.dll
From: https://firebase.google.com/docs/unity/setup#net_compatibility_when_using_unity_2017x_and_later

Hi, I’ve done that, I still get the same error.
From: TonyaGitHub<EMAIL_ADDRESS>Sent: Thursday, May 9, 2019 6:47 AM
To: firebase/quickstart-unity<EMAIL_ADDRESS>Cc: Alexandre Baumeige<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [firebase/quickstart-unity] Websocket error TextMeshPro downgrade DotNet3.x (#360)
Keep the 4.X, this has been a common issue, here's the solution/workaround:
To resolve the compilation error, if you're using .NET 4.x:
Remove or disable the following DLLs for all platforms:
Parse/Plugins/Unity.Compat.dll
Parse/Plugins/Unity.Tasks.dll
Enable the following DLLs for all platforms:
Parse/Plugins/dotNet45/Unity.Compat.dll
Parse/Plugins/dotNet45/Unity.Tasks.dll
From: https://firebase.google.com/docs/unity/setup#net_compatibility_when_using_unity_2017x_and_later
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/firebase/quickstart-unity/issues/360#issuecomment-490853736 , or mute the thread https://github.com/notifications/unsubscribe-auth/AE5HNYVEOZJ2UQENC275DODPUP6JHANCNFSM4HLVNOSA .

Hi all,
The C# implementation of Firebase Realtime Database on desktop has this issue as well as many others. We are working hard to finish our native C++ desktop implementation, which should fundamentally fix these sorts of issues in Firebase RTDB desktop in Unity as well.
Please bear with us as we finish the implementation (we are getting very close), and will do so when we are sure it is ready for use.

See https://github.com/firebase/quickstart-unity/issues/106 for more information.

OK thanks for your help.
Alex
From: jonsimantov<EMAIL_ADDRESS>Sent: Thursday, May 9, 2019 1:56 PM
To: firebase/quickstart-unity<EMAIL_ADDRESS>Cc: Alexandre Baumeige<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [firebase/quickstart-unity] Websocket error TextMeshPro downgrade DotNet3.x (#360)
Hi all,
The C# implementation of Firebase Realtime Database on desktop has this issue as well as many others. We are working hard to finish our native C++ desktop implementation, which should fundamentally fix these sorts of issues in Firebase RTDB desktop in Unity as well.
Please bear with us as we finish the implementation (we are getting very close), and will do so when we are sure it is ready for use.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/firebase/quickstart-unity/issues/360#issuecomment-491003834 , or mute the thread https://github.com/notifications/unsubscribe-auth/AE5HNYQPV4K4YPY4R7UNEWTPURQTLANCNFSM4HLVNOSA .


Interop testing with js-libp2p

In order to get interop, one of my attempts is to get the basic examples of js-libp2p to work with go-libp2p. The basic example is:

Two libp2p nodes
Both have spdy + identify enabled
One mounts the 'echo' protocol
The other dials on the echo protocol and writes something to be echoed

So far, I haven't been successful achieve this, here is the latest two attempts
go dialer, js listener

go listener, js dialer

I'm not seeing where I can select the protocol on the go-libp2p, however, the thing that concerns me is the fact that it says that the protocol is not supported when dialing from go to node. Is secio turned by default in these examples?

The js code for these examples can be found https://github.com/ipfs/js-libp2p-ipfs/tree/master/examples/echo

@diasdavid yeah, i think those examples use secio by default. Let me look at making that optional

Status update: We've solved the SPDY issue.
Next: finishing the tests.

Right now identify seems to be working, we just need to make the tests more verbose so that we now what is going on :)
Also, go-ipfs likes do blow up SPDY when it doesn't want to talk with the peer anymore. We need to listen for RST frames and just delete it from our pool of muxed conns in Swarm

Was there any progress on this front?


complie issue

error[E0053]: method `broadcast_transaction` has an incompatible type for trait
   --> src/node.rs:355:5
    |
355 |     fn broadcast_transaction(&self, tx: &Transaction) {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `bitcoin::blockdata::transaction::Transaction`, found a different struct `bitcoin::blockdata::transaction::Transaction`
    |
    = note: expected type `fn(&node::Broadcaster, &bitcoin::blockdata::transaction::Transaction)` (struct `bitcoin::blockdata::transaction::Transaction`)
               found type `fn(&node::Broadcaster, &bitcoin::blockdata::transaction::Transaction)` (struct `bitcoin::blockdata::transaction::Transaction`)
note: Perhaps two different versions of crate `bitcoin` are being used?
   --> src/node.rs:355:5
    |
355 |     fn broadcast_transaction(&self, tx: &Transaction) {
    |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0053]: method `install_watch_script` has an incompatible type for trait
  --> src/lighningconnector.rs:65:5
   |
65 |     fn install_watch_script(&self, script_pub_key: Script) {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `bitcoin::blockdata::script::Script`, found a different struct `bitcoin::blockdata::script::Script`
   |
   = note: expected type `fn(&lighningconnector::LightningConnector, bitcoin::blockdata::script::Script)` (struct `bitcoin::blockdata::script::Script`)
              found type `fn(&lighningconnector::LightningConnector, bitcoin::blockdata::script::Script)` (struct `bitcoin::blockdata::script::Script`)
note: Perhaps two different versions of crate `bitcoin` are being used?
  --> src/lighningconnector.rs:65:5
   |
65 |     fn install_watch_script(&self, script_pub_key: Script) {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0053]: method `install_watch_outpoint` has an incompatible type for trait
  --> src/lighningconnector.rs:70:5
   |
70 |     fn install_watch_outpoint(&self, outpoint: (Sha256dHash, u32)) {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ expected struct `bitcoin::util::hash::Sha256dHash`, found a different struct `bitcoin::util::hash::Sha256dHash`
   |
   = note: expected type `fn(&lighningconnector::LightningConnector, (bitcoin::util::hash::Sha256dHash, u32))` (struct `bitcoin::util::hash::Sha256dHash`)
              found type `fn(&lighningconnector::LightningConnector, (bitcoin::util::hash::Sha256dHash, u32))` (struct `bitcoin::util::hash::Sha256dHash`)
note: Perhaps two different versions of crate `bitcoin` are being used?
  --> src/lighningconnector.rs:70:5
   |
70 |     fn install_watch_outpoint(&self, outpoint: (Sha256dHash, u32)) {
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to 3 previous errors

error: Could not compile `bitcoin-spv`.

To learn more, run the command again with --verbose.

why?

This is due to divergence in the rust-bitcoin dependency between rust-lightning and rust-bitcoin-spv. Now that 0.13 is out (and rust-lightning updated to it at https://github.com/rust-bitcoin/rust-lightning/pull/20), just need to update to the new witness changes here and should be resolved.

resolved


Provide API/Example to use to remotedev-server from other applications

If there would be a piece of documentation / an basic example, then other appliactions
and even other programming-languages could use this project. In concrete i try to integrate
my redux-python-port:
https://github.com/peterpeter5/pyredux
with this server. But i think it could be interesting for other people / applications.

PyRedux looks amazing!
I've just published a docs draft, also with Python examples. Feel free to amend it. I used to be a fan of Django, but it was almost 10 years ago and already forgot everything.

The docs look amazing! I will try to do the integration-work in the  next Days. Thank you for your  (well documented) reply on such short notice!

The documentation is quite helpful, but i guess there is something wrong or left out. I am able to connect to the server, to "authenticate" as "master" and to send the INIT-Command (I can see the visualization). When i then send a lifted-STATE (sniffed by another application), then I am able to reproduce the logged state-tree (everything is visualized as expected). However, sending a request with:
data = {
    "type": "ACTION",
    "payload": {
      "todos": "do some"
    },
    "action": {
      "timestamp":<PHONE_NUMBER>506,
      "action": {
        "type": "ADD_TODO",
        "text": "hggg"
      }
    },
    "instanceId": "tAmA7H5fclyWhvizAAAi",
    "name": "LoggerInstance",
    "id": "tAmA7H5fclyWhvizAAAi"
  }
socket.emit("log", data)   # No difference when i use "log-noid"

I could use the lifted-state, but than i would have to provide values like: isLocked and stagedActionIds which I am sure is not the best way to do

I just tried your snippet (on NodeJS) and I'm getting ADD_TODO action on http://remotedev.io/local/ (switching to local server in Settings):
var scClient = require('socketcluster-client');
var socket = scClient.connect({ hostname: 'localhost', port: 8000 });
socket.connect();
socket.emit('login', 'master', function(error, channelName) {
      if (error) { console.log(error); return; }
      socket.subscribe(channelName);

      var data = {
        "type": "ACTION",
        "payload": {
          "todos": "do some"
        },
        "action": {
          "timestamp":<PHONE_NUMBER>506,
          "action": {
            "type": "ADD_TODO",
            "text": "hggg"
          }
        },
        "instanceId": "tAmA7H5fclyWhvizAAAi",
        "name": "LoggerInstance",
        "id": "tAmA7H5fclyWhvizAAAi"
      };
     socket.emit("log", data);
});

Maybe socketcluster-client-python is not stribgifying the object properly. You could try to add json.dumps:
var data = {
    "type": "ACTION",
    "payload": json.dumps({
      "todos": "do some"
    }),
    "action": json.dumps({
      "timestamp":<PHONE_NUMBER>506,
      "action": {
        "type": "ADD_TODO",
        "text": "hggg"
      })
    },
    "instanceId": "tAmA7H5fclyWhvizAAAi",
    "name": "LoggerInstance",
    "id": "tAmA7H5fclyWhvizAAAi"
  };

you were right about the socket-cluster-client   -   stringification, but in another way. When you pass in a string into the socket.emit - function (which i did, unlike my previous post...)  then this string gets json.dumps again. That was my problem. Now it works like charm. Thank you!


Issues with Antrea Agent after September Microsoft Windows OS  Patches.

After last round of patches getting the following error when antrea agent  pod tries to recreate:
Directory: C:\host\k\antrea

Mode                 LastWriteTime         Length Name

d----           9/22/2021  6:14 PM                bin
FATA[2021-09-22T18:15:45-04:00] rpc error: code = Internal desc = could not rename binary: could not backup the existing target file: rename C:\k\antrea\bin\rancher-wins-antrea-agent.exe C:\windows\TEMP\rancher-wins-antrea-agent.exe: Access is denied.

@thomasdbaker Thanks for reporting the issue. Do you mean you hit the patch in main branch? Or would you provide Antrea version?

@thomasdbaker hello, could you tell why to rename (move?) exe file to C:\windows\TEMP\rancher-wins-antrea-agent.exe? When does the operation happen?

This is the copy in the docker config map that copies the exes to the host volume
Seems to be a problem with this patch level
https://support.microsoft.com/help/5005568 DCWIDAVDKR0010 Security Update KB5005568 NT AUTHORITY\SYSTEM 9/21/2021
DCWIDAVDKR0010 Security Update KB5005701 NT AUTHORITY\SYSTEM 9/21/2021

@thomasdbaker hello, could you tell why to rename (move?) exe file to C:\windows\TEMP\rancher-wins-antrea-agent.exe? When does the operation happen?

It is an operation of the Antrea-Agent pod to copy the files at start-up.

@thomasdbaker Thanks for reporting the issue. Do you mean you hit the patch in main branch? Or would you provide Antrea version?

The version was v1.1.0  but meant the  MS OS monthly patches broke something in the file copy process.
https://support.microsoft.com/help/5005568 Security Update KB5005568 NT AUTHORITY\SYSTEM 9/21/2021
Security Update KB5005701 NT AUTHORITY\SYSTEM 9/21/2021

@thomasdbaker Actually, I am a bit doubt it is related with the privileges of service "rancher-wins" (maybe name as "wins"?), which is the actual worker to call antrea-agent.exe on the Window host and the worker to rename C:\windows\TEMP\antrea-agent.exe to C:\windows\TEMP\rancher-wins-antrea-agent.exe. So could you print the "Security Properties" of directory "C:\windows\TEMP", and the service account and permisssions on "rancher-wins".
Another try is to add command "rm /host/k/antrea/bin/" after command "mkdir -force /host/k/antrea/bin" and before command "cp /k/antrea/bin/ /host/k/antrea/bin/" in Run-AntreaAgent.ps1

I feel like this may actually be a problem with wins itself, I think, when the container is terminated, there is a possibility that the process on the host can remain open and not realise that the client has stopped trying to talk to it, therefore the delete/overwrite fails, I suspect this is actually not to do with the Patch and more to do with the way the termination state of wins.  As the file already exists, could we check if it already exists and not try and overwrite or not fail when it fails to copy, as the process on the hosts will be running wins.exe srv app run ..... we may still be able to access the named pipe that it has open without running again? @wenyingd @lzhecheng what do you think?

FATA[2020-06-17T00:29:15Z] rpc error: code = Internal desc = could not rename binary: could not backup the existing target file: rename c:\etc\kubernetes\bin\rancher-wins-kubelet.exe C:\Windows\TEMP\ranc
her-wins-kubelet.exe: Access is denied.
It seems this is something to do with this internals of wins https://github.com/rancher/windows/issues/45, https://github.com/rancher/rancher/issues/26739 and not related to the windows patch

I feel like this may actually be a problem with wins itself, I think, when the container is terminated, there is a possibility that the process on the host can remain open and not realise that the client has stopped trying to talk to it, therefore the delete/overwrite fails, I suspect this is actually not to do with the Patch and more to do with the way the termination state of wins. As the file already exists, could we check if it already exists and not try and overwrite or not fail when it fails to copy, as the process on the hosts will be running wins.exe srv app run ..... we may still be able to access the named pipe that it has open without running again? @wenyingd @lzhecheng what do you think?

Hi Peri, I think it should be related with the Windows patch. In my own setup (not installing the patch), I restarted Windows Antrea Agent many times, but I never hit the issue. Besides, we shall overwrite the binary for upgrading Antrea Agent case.


Unable to read manifest using the same API key that works for deploying the image

Describe the bug
We have a policy configured in values.yaml that matches the image, and a validator for that policy containing the public key corresponding to the private signing key.
Image signed by cosign.
Error:
Error from server: error when creating "image.yaml": admission webhook "connaisseur-svc.connaisseur.svc" denied the request: Unexpected Cosign exception for image "registry.docker.artifactory.xxx.com/repo/image:0.152.2-20220208T143249Z.master": Error: GET https://registry.docker.artifactory.xxx.com/v2/repo/image/manifests/0.152.2-20220208T143249Z.master: UNAUTHORIZED: The client does not have permission for manifest; map[manifest:repo/image/0.152.2-20220208T143249Z.master/manifest.json]
main.go:46: error during command execution: GET https://registry.docker.artifactory.xxx.com/v2/repo/image/manifests/0.152.2-20220208T143249Z.master: UNAUTHORIZED: The client does not have permission for manifest; map[manifest:repo/image/0.152.2-20220208T143249Z.master/manifest.json]
Note that the image deployment imagePullSecret is the same as the secret configured in values.yaml.
The secret for connaisseur was created per https://sse-secure-systems.github.io/connaisseur/v2.1.2/validators/sigstore_cosign/#authentication and is referenced for each validator as:
auth:
secret_name: myregistrykey
Expected behavior
The signed image is pulled successfully.
Optional: To reproduce

Optional: Versions (please complete the following information as relevant):

OS:
Kubernetes Cluster: v1.19
Notary Server: N/A
Container registry: JFrog Artifactory Enterprise Plus 7.19
Connaisseur:  securesystemsengineering/connaisseur:v2.4.1
Other:

Optional: Additional context

@QWERTY92009 that is indeed surprising. I have tested with jfrog artifactory in the past without problems. It should be either a problem with the secret or Cosign itself.
Could you test whether you can verify with local cosign using the creds above?

Hi @xopham. Good tip. I cannot verify with the local cosign.
But at the same time, I can sign the image, telling me cosign has read access (to pull an image) and write access (to write the signature).
I'll contact our artifactory team. Thanks again, this is clearly the heart of the issue.

Hi @xopham. Our CI/CD pipeline apps team tells me the correct URL for the manifest is:
https://artifactory.sativa.com:443/artifactory/registry/repo/image/0.152.2-20220208T143249Z.master/manifest.json
or
registry/repo/image/0.152.2-20220208T143249Z.master/manifest.json
cosign is apparently trying to access it at
https://registry.artifactory.sativa.com/v2/repo/image/manifests/0.152.2-20220208T143249Z.master
Maybe there has been a jfrog API change?

@QWERTY92009 thanks for sharing :pray:
It would seem that cosign is working according to OCI distribution-spec: v2/<name>/manifests/<reference>. However, the cosign docs list JFROG Artifactory under supported registries. So that is a bit surprising.

So, docker v2 api spec agrees with cosign, so I would lean towards your registry running on docker v1 api spec maybe: https://www.jfrog.com/confluence/display/JFROG/Using+Docker+V1 :shrug:

Excellent advice. Thanks @xopham!

Please let me know in case this has been resolved 🙏 Would close this, as it seems unrelated to Connaisseur. Feel free to reopen if that turns out to be falls 😊


Loglevel

Atm we can't set the loglevel

We may set the loglevel on the runtime and only send messages if needed (logging in file/client attached)


Proposal: Update Runge-Kutta step size algorithms for predictable performance

Proposal
Update the adaptive step-size algorithm to capture the stability and conditioning improvements demonstrated in [1].
Feedback would be much appreciated on the technical details of the approach below, as well as on the overall worth-while-ness of this proposal.
Context
In the past couple of decades the adaptive step-size algorithm has been studied and the stability and conditioning can be significantly improved without affecting speed. Here is an example where on the left a step-size algorithm similar to scipy's was used, and on the right was a new step-size algorithm based on control theory:

This was figure 1 from [1]. The right panel demonstrates that using more sophisticated step size algorithms from control theory result in much more predictable behavior in that increasing tolerance promises to both decrease accuracy and decrease runtime. This is in contrast to the left panel where increasing the tolerance will decrease accuracy but may not decrease runtime.
1. Testing
The first step would be to add testing that accurately analyzes the current algorithms. This would be implemented following the analysis in [2]. This would involve

[ ] updating the method signatures to support the use of tolerance definitions described in [2]
[ ] find/port a test suite of IVPs (maybe Test Set for IVP Solvers)
[ ] ❓  create test methods calculate summary stats for different parameters using RK45

parameters

lt = log(tolerance)
lf = log(number of function calls)
la = log(achieved accuracy)
runtime

summary stats

❓ linear bias of lf vs lt (slope and intercept)
❓ remove the linear bias from lf vs lt, then calculate the unbiased sample variance
❓ linear bias of la vs lt (slope and intercept)
❓ remove the linear bias from la vs lt, then calculate the unbiased sample variance
❓ linear bias of lf vs la (slope and intercept)
❓ remove the linear bias from lf vs la, then calculate the unbiased sample variance

[ ] ❓  create test methods that produce graphs similar to figures 3, 4, and 11-14 in [2] from the aforementioned parameters - with the additional runtime parameter to measure code efficiency (apart from the algorithmic efficiency measured by lf)
[ ] ❓ create automated tests that enforce that the aforementioned summary stats are bounded by hard coded constants equal to the output values for the current implementation. Ideally these bounds would be updated as they are improved to avoid regressions.

⚠ I am seeking feedback especially on each part marked with a ❓. Specifically

Are these the appropriate summary statistics to gather?
where would be an appropriate place to put the graphical test code? It probably shouldn't run automatically...
Are automated tests to prevent regressions in those parameters a good idea?

The idea would be to provide metrics that act as gates to prevent regressions upon algorithmic changes. If this were implemented we would need high confidence that the summary stats are indeed good metrics for judging algorithms against.

What is the appropriate method to ensure future developers tighten them as they improve the algorithms?
Is there any advantage to using methods other than RK45 in this test suite - given that it is largely testing the non-RK portions of the algorithm?

2. Update the adaptive step-size algorithm
Update the step size algorithm in the RungeKutta._step_impl method to use the stepsize control algorithm in section 7 of [1].
This would also involve tightening the test bounds created in part 1 to avoid regressions after this was done.
References

Adaptive time-stepping and computational stability, Soderlind & Wang
Evaluating numerical ODE/DAE methods, algorithms and software, Soderlind & Wang

This sounds like a thorough proposal. I will try to study it carefully.

@solter
I have a general idea about this proposal --- make it a Google Summer of Code Project 2019. It could involve:

Implementing a benchmark suite.
Implementing new step selection strategies for RK with more predictable performance.
Finishing #9290 properly
Working on other reported issues for new ODE solver (the exact plan is to be determined).

In this case, would you be interested to help as a mentor (especially with 1 and 2)?
@rgommers What do you think of such idea for GSoC?
I can write a brief plan for it and if a right student comes, it might be viable.

I made a quick test on this problem (non-stiff, suitable for RK) and I didn't observe such erratic behavior of work-vs-tolerance plot. Attaching the plot

It doesn't mean that it can't happen on other problems. But I might guess that such behavior is more pronounced for stiff problems and methods. And in the paper you referenced they use a stiff problem as a main example.
It need more investigation, I haven't finish reading the paper properly.

@rgommers What do you think of such idea for GSoC?

Could work, it's nicely self-contained and about the right amount of effort. However, my impression is that @solter would like to take this on and from his profile it doesn't look like he's a student.

@nmayorov - GSoC is a neat idea, and if the right student came along I think this would fit really well with their academic work. I personally don't have a lot of experience with the scipy community, but would be open to helping mentor a student on the technical aspects (and using it as an opportunity to get more involved on the community side myself).

However, my impression is that @solter would like to take this on and from his profile it doesn't look like he's a student.

@rgommers - thanks for the thought. On the other hand my ambition/interests far exceeds my actual energy/time for non-work projects, so I'd be fine if a student wanted to take this on - I've got other stuff on the back burner that I can tackle instead.

Well in that case, I'd say go for it:) Our ideas page is at https://github.com/scipy/scipy/wiki/GSoC-2019-project-ideas, if you together want to change the content here to a GSoC project outline, that would be great I think.

Yes it would be good. And we'd need to sign up at least two (primary and backup/secondary) mentors for the project with Google and PSF. The existing mentors who are on board for the one current potential project (fftpack interfaces) probably would not fit with this new project. I'm dealing with the GSoC admin for SciPy this year so let me know if you can get this together.

Hey!
I am assuming there was no significant progress here? I implemented a few of the problems from the proposed test-set, and from some other sources here.
This would at least save the work of re-implementing a bunch of the problems, if anyone wants to work on them.

There is also Petsc suite and it is BSD-2 licensed. They are active in GitLab and probably supersede the DASSL. We might want to contact them too.

When there's a PR for this, it would be great to see whether it resolves the problem reported in gh-9899.


Impossible usage of rust-scrypt on Windows OS due to mmap call

mmap call used in Scrypt C library is only Unix specific.
Added conditional dependency for using rust-crypto on Windows
Related to #138

did you remember to put a flag also in cargo.toml?

cargo test --verbose %cargoflags%
    Updating registry `https://github.com/rust-lang/crates.io-index`
error: manifest path `C:\projects\emerald-rs\Cargo.toml` is a virtual manifest, but this command requires running against an actual package in this workspace
Command exited with code 101

I also want to fix .travis.yml to fix it.

.travis.yml or appveyor?
We using rust-crypto for pbkdf2 on both Linux/Mac & Windows
so they required both, no need for Cargo tweak

you right, probably in future we should switch to ring


Improve look & feel of adios dashboard

Move navigation panel to the left side
Make a 2x2 grid of galleris so we can more easily compare variables
Use drag & drop to select variables for viewing
Synchronize playback across variables with a single set of controls

@zackgalbreath I am seeing the following when I try to drag and drop a parameter:

Thanks for helping me reproduce this issue, Chris. It should be fixed now.

These changes have now been deployed to http://adios-dashboard.kitware.com/


Challenges that are tightly coupled with input files

The contributing guidelines don't specify what should be done for challenges that are tightly coupled with input files. Example of one is [Intermediate] no.19 which links to text file needed for the program. Should these type of files just be included in the folder for the challenge?

Personally, I think that the files should just be included in the folder. It could get messy if people started creating new folders for their code & their text file. Plus, it's only one extra file per upload.
Otherwise, do you have any suggestions/ideas for the way files/directories are organized?

I think all of the solutions should share a text file, like this:
.
├── challenge_text.md
├── _solutions
|   ├── textFileNeededForSolution.txt
|   └── solution.xyz
|   └── solution.rb
|   └── solution.c
|   └── solution.py
That seems most efficient and would eliminate any unnecessary clutter as well as standardize the file name for whatever input file is needed.

@jacob-on-github That's essentially the current format, no?

@Hoenn @jacob-on-github The text files are already hyper-linked in the challenge_text.md. Adding them separately to the repo is tedious.

@rafid059 @FreddieV4 currently they use a hyper link, as rafid said, and while it would be tedious, maybe the should be in the folders as .txt files for the sake of consistency. It's Freddie's call, but I think maybe we should start moving them there.

I think we can safely say that adding the text, hyperlinked from other sites, will definitely be better, as people will be easily able to continue working through challenges without switching context (i.e. from their text editor to browser).
Let's try compiling a list of challenges that are in fact tightly coupled with their inputs.

Let's add something to the readme or contributing guidelines about the files and close this issue.


We should define predicates to match the concepts extracted in ricu & YAIB for MIMIC, eICU, and other datasets

See here for the concept maps: https://github.com/eth-mds/ricu/blob/main/inst/extdata/config/concept-dict.json
Tagging @prockenschaub and @rvandewater and @Oufattole and @justin13601 for input

See https://github.com/justin13601/ACES/issues/75 for realizing the capability to cleanly express these predicates.


fix(dev): Add passthrough for env vars

Add passthrough env vars for weave, parquet, and gorilla.

This PR is included in version 4.5.3 :tada: