Remove references to TestGrandpaTestSuite

This e2e suite was removed at some point during sdk 0.52 upgrade. It is still referenced in our workflows (and is one causing one of em to consistently fail).
Should be possible to remove references to it.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged/assigned
[ ] Estimate provided

We have #6835, as well, but we are def at place where we can just remove it entirely.

yea, I think for time being this is really just a clean-up of workflow files.


configure classifiers and transitions from extensions

When connecting to an extension server it would awesome if we could send the bot our own classifiers and transitions. This would allow us to programmatically generate questions and answers.
An example where this would be used is the company I work at. We have an internal StackOverflow site. We could write an extension that gets all the questions and answers from that site and generate classifiers and transitions and register them with the bot.
It would be super powerful for automatically training the bot to respond to questions with a set of predefined answers from other tools.

I've been thinking about this issue as well as https://github.com/jaimeteb/chatto/issues/24 and https://github.com/jaimeteb/chatto/issues/18. I'm thinking about a solution that can solve all three.
Change extension servers to use events for receiving a command by using WebSockets
Add a feature to the Bot that will produce extension events over WebSockets instead of calling an extension REST/RPC endpoint.
The extension server would then no longer need to implement a web service and the complexities. Like firewall access from the bot, authorization, and setting up endpoints.
The extension server would then become an event consumer. We could use a tool similar to https://www.asyncapi.com/ to generate consumer clients in multiple languages. This would address https://github.com/jaimeteb/chatto/issues/24 as generated clients and consumers are easier to than generated servers.
Generate client SDKs
We would then generate and publish client SDKs for the Bot REST API and Extension WebSocket Event API in multiple languages. Extension servers would import these clients.
Classifier and FSM instance per extension
When an extension server comes online it can register its extensions with the Bot along with Clasifiers and FSM transitions. The Bot would have the endpoints /bot/classifiers/<extension_name> and /bot/fsm/<extension_name>. Which provide the mechanism to register. In the Clasifier and FSM there would be multiple instances, the default instance which is loaded by the Bot at start and an instance per extension. Having an instance per extension allows configuring without worrying about overwriting each other.

I like the idea of using the Bot as the centerpiece of the system. If I understand correctly, what you're proposing is: if an extension is to be run, the Bot produces an event, then the extension server consumes it, and finally, the extension makes a request to a Bot endpoint to register its result.
Some remarks:

I wouldn't want to get rid of the REST extension server, because of how easy it is to integrate. For example, I'm developing a serverless Chatto bot, which involves a serverless extension server as well, and the easiest way to trigger it is via REST/HTTP.
I'm all for the WebSocket idea, as long as it is flexible and versatile. I'll do some research, perhaps I'm too tied to REST and HTTP.

I'm not sure I'm convinced by that last part, I feel like multiple instances of FSM/Classifier are essentially multiple bots, so I don't know if hosting them in the same bot server is the best solution. Or maybe I'm misunderstanding. I thought you were referring to an "auto-training" feature, where an extension would be able to retrain the bot by sending new FSM/Classifier data, and maybe we could have versioning of the files so that original or previous data isn't lost.
Another issue I see with the multiple instances is that with #41, we'll now be able to write/read Classifier models to/from files, and if you're using Word Vectors, these files become very large. If there are multiple instances, there would be many of these huge files.

Still thinking about this, not discarding anything because these are great ideas, but let me give it some thought.

No problem I understand this kind of changes the model a bit.
Remark 1 response
I would prefer to stick with REST as well. The hard part about REST is requiring extension servers to implement concrete endpoints correctly. Writing service endpoints is harder to get right and easy to get wrong. So turning the model into an event driven one makes it a little easier. Where developers only need to worry about consuming events as a client and posting responses to a REST endpoint.
The AWS provider for serverless seems to support websockets https://www.serverless.com/framework/docs/providers/aws/events/websocket/.
Remark 2 response
Yeah this one is a little more difficult. The trick is being to update the configuration/train the bot without wiping the other configurations/trainings and preventing race conditions.
Separating by extension solves that although like you mentioned will become a huge issue for Classifiers. I think it might be okay to have a unique FSM implementation per extension. I'm not 100% until we do some testing. Then have a global Classifier because the issue won't be as big of a deal there.
Saving the training seems unnecessary if we can get the extensions to re-register themselves. Maybe through a WebSocket event to tell extensions to re-register.
I can put an example PR together to flesh it out more and you can 👍  or 👎  it.

1
You're right about that, REST can get very messy. Besides, the event-driven approach makes things a bit more modern I think. What I like about REST is that it is platform/language agnostic, but WebSockets are too, right?
2
For sure, maybe with something more tangible we would see the possibilities. Again, I'm very influenced by Rasa, and I was imagining something like Rasa X, a tool that can update the bot's model, but it doesn't overwrite the previous one, instead, it saves versions of the bot.

I think we could try the WebSockets first, and after that, the second feature. I'll keep thinking about it, I'm sure we're onto something cool here.

BTW, take a look at the new classifier features, maybe they'll come into play for this.

The new classier features look awesome. Will have to understand them a bit more. As someone who knows nothing about ML I'm not sure I would choose to use some of these features. I will google them and see if I can also add some info as to why I would use them.
Good call about WebSockets first. And yes they are ubiquitous in all languages, even web browsers and generally pretty easy to implement.


Fixed the state byte in the route input none scripts (made it 00).

I checked the other "none" scripts,, they are okay.

@jfallows Here's the spec fix. I don't have permission to assign this ticket to you.


Create accountlogon.txt

Matt we are now unifying this git into https://github.com/microsoft/MTP-AHQ Microsoft threat protection GitHub.
The tables have new names and there are better ways to track user logon events, by using both DeviceLogonEvents and IdentityLogonEvents.
I suggest to update the query and insert to it more content.

Perfect, thanks!
Br Mattias
22 apr. 2020 kl. 12:23 skrev tali-ash<EMAIL_ADDRESS>
Matt we are now unifying this git into https://github.com/microsoft/MTP-AHQ Microsoft threat protection GitHub.
The tables have new names and there are better ways to track user logon events, by using both DeviceLogonEvents and IdentityLogonEvents.
I suggest to update the query and insert to it more content.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/WindowsDefenderATP-Hunting-Queries/pull/103#issuecomment-617690517, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFZK5CKKZA6QSBW7QHYEOELRN3ARLANCNFSM4J3GONMA.


[W5.8b][W15-A2]Ho Bing Xuan

Add 10 NFRs

Your understanding of NFR is quite good; ensure you do a good job in the project documentation as well


Property inheritance or modifier or mixes

I'm adapting BEM to create more maintainable CSS and I like it a lot. Some cases are still bit fuzzy. Sorry about long post, but these things are so closely related.
I have an icon and I'm using it in a text and in a button, like:

    Text and icon should be black

<button class="button">
    Text and icon should be white
</button>

In the text block the icon should be black and in the button it should be white. In both cases the required color is same as the container's text color.
First question, should icon inherit color OR should I add modifiers like "icon--in-text" and "icon--in-button" OR should I use mixes?
icon {
    color: inherit;
}

/* OR */

icon.icon--in-button {
    color: white;
}
icon.icon--in-text {
    color: black;
}

/* OR */

button__icon {
    color: white;
}

Then I may come up with a need to add disabled-state to the button. Text color of disabled button and the icon should be gray.
If I choose to use the "icon--in-button" modifier, then I need to add a "icon--disabled" modifier too, I guess?
<button class="button button--disabled">
    Text and icon should be gray
</button>

icon.icon--disabled {
    color: gray;
}

In this simple case inheriting seems valid option, but things get more complicated when we have an icon stack, e.g. background and foreground icons, which means two colors. The color of the background icon should be same than the text color of the container and the color of the foreground icon should be same than the background color of the container.
<button class="button">

</button>

<button class="button button--disabled">

</button>

How would you handle icon stack situation?

I think you should declare block and element first:
.button {
  &__icon {
    color: white;
  }
  &--disabled__icon {
    color: gray;
  }
}
.text {
  &__icon {
    color: black;
  }
}

icon can be a Block but should have modifiers representing its parent


Feature strict dns cluster

Issues associated with this PR
Your PR should present related issues you want to solve.
Solutions
Support resolve address in hosts to IP list, and apply load balance strategy based on IP list.
Support update according to TTL, fixed duration.
Support configure dns resolve file instead of default /etc/resolv.conf. Also support configure dns resolve port.
configure example:
     {
        "Name": "testCluster",
        "type": "STRICT_DNS",
        "respect_dns_ttl": true,
        "lb_type": "LB_ROUNDROBIN",
        "max_request_per_conn": 1024,
        "conn_buffer_limit_bytes": 32768,
        "hosts": [
          {
            "address": "www.test.com:80"
          }
        ]
      }

respect_dns_ttl (bool)
use TTL in dns result as refresh rate
dns_refresh_rate (time, duration)
use fixed duration to update dns result
dns_resolver_file (path, string)
use specified resolv.conf file instead of /etc/resolv.conf
dns_resolver_port (port, string)
use specified port instead of default port 53
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #1149 into master will increase coverage by 0.14%.
The diff coverage is 72.09%.

@@            Coverage Diff             @@
##           master    #1149      +/-   ##
==========================================
+ Coverage   56.19%   56.34%   +0.14%
==========================================
  Files         268      270       +2
  Lines       25415    25793     +378
==========================================
+ Hits        14281    14532     +251
- Misses       9772     9868      +96
- Partials     1362     1393      +31

Impacted Files
Coverage Δ

pkg/config/v2/upstream.go
70.83% <ø> (ø)

pkg/network/dns.go
69.04% <69.04%> (ø)

pkg/upstream/cluster/strict_dns_cluster.go
72.12% <72.12%> (ø)

pkg/upstream/cluster/cluster.go
82.30% <100.00%> (+2.10%)
:arrow_up:

pkg/admin/store/effectiveconfig.go
76.54% <0.00%> (-16.00%)
:arrow_down:

pkg/upstream/healthcheck/factory.go
71.42% <0.00%> (-1.91%)
:arrow_down:

pkg/configmanager/configmanager.go
95.92% <0.00%> (-1.77%)
:arrow_down:

pkg/module/http2/transport.go
78.55% <0.00%> (-0.37%)
:arrow_down:

pkg/module/http2/frame.go
84.17% <0.00%> (-0.31%)
:arrow_down:

pkg/module/http2/server.go
81.85% <0.00%> (-0.08%)
:arrow_down:

... and 8 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 c9218dc...a82f0ad. Read the comment docs.


Feature/move text

Summary
You wanted the ability to move text around the transcript. I give you the ability to move text around the transcript. While I'm not stocked with how it is implemented

Why is this change needed?
It is a key feature of our final product.

What did you change?
I added a few key bindings (x, c, v) for cut, copy, and paste. I made an operation for pasting text for the undo stack. This was the only op required because cutting is just a combination of deleting text which already exists as an op and pasting.
I also created a clipboard state in react which tracks the indexes of the currently copied block of text. This may not be ideal but it gets the job done and I'm lazy.

Explain things which the reviewers are likely to need explaining. Remember that they didn’t write the code.
No tests because it will take too long to figure out how to do it by myself. Might be good to have a masterclass.

Testing details if applicable
N/A

Any sections in particular you want reviewed/discussed
The getIndexSelectedWords() function is a little bit janky. If anyone is inspired to write a better one, the user experience will be greatly increased.

OS

[ ] Linux
[ ] MacOS
[x] Windows

It's up to you whether you want to fix these now or later, as we're close to the deadline. But there are some bugs in your code:

If you cut and paste a sequence of words, some of the spaces between words disappear.

Before:

After:

If you delete words, then cut those words and paste them elsewhere, the deleted words return.

Before:

After delete:

After paste:

If you delete words then cut them, then undo the cut but don't undo the delete, the deleted words return. If you redo the delete and then the cut, the state adjusts as expected. This indicates there might be a problem with your undo but not your do part of the cut op.

This one is a little harder to show in screenshots but if you follow the instructions you should be able to replicate it.

Copying words all over the place breaks undo. This might be linked to your known issue of indexes not working - but in theory this should be deterministic even with that bug. Below is the result of copying a bunch of times, undoing and then redoing, then undoing until there is no more left to undo. As you can see this doesn't return us to the original transcript, which should always be the result of undoing to the top of the stack. In fact in the state shown below, if I redo a bunch of times and then undo some more, even more copies of "financial business" appear.

It's up to you whether you want to fix these now or later, as we're close to the deadline. But there are some bugs in your code:

If you cut and paste a sequence of words, some of the spaces between words disappear.

Before:

After:

If you delete words, then cut the words surrounding them and paste them elsewhere, the deleted words return.

Before:

After delete:

After paste:

If you delete words then cut them, then undo the cut but don't undo the delete, the deleted words return. If you redo the delete and then the cut, the state adjusts as expected. This indicates there might be a problem with your undo but not your do part of the cut op.

This one is a little harder to show in screenshots but if you follow the instructions you should be able to replicate it.

Copying words all over the place breaks undo. This might be linked to your known issue of indexes not working - but in theory this should be deterministic even with that bug. Below is the result of copying a bunch of times, undoing and then redoing, then undoing until there is no more left to undo. As you can see this doesn't return us to the original transcript, which should always be the result of undoing to the top of the stack. In fact in the state shown below, if I redo a bunch of times and then undo some more, even more copies of "financial business" appear.

Good finds! Thankyou for taking the time to investigate and document those. I reckon those will be something fun to work on over the break and second semester. I don't intend to put any more time into the coding until some of my other assignments are finished.
Since it shouldn't break anyone else's code I'm still inclined to merge with develop and save those bugs for a future PR.
Cheers


Which PCL version should I use?

An error occurred when I compiled your package: /usr/include/pcl-1.8/pcl/kdtree/kdtree_flann.h:230:29: error: field ‘param_k_’ has incomplete type ‘flann::SearchParams’ ::flann::SearchParams param_k_;

It turns out to be a conflict between OpenCV and PCL. Was able to solve the issue by moving "cv_bridge" down in the CMakeLists.txt.

It turns out to be a conflict between OpenCV and PCL. Was able to solve the issue by moving "cv_bridge" down in the CMakeLists.txt.


Bug: 'retain_line_breaks' on folded bleeds '#magic___^_^___line'

Version: latest
To reproduce:
printf "a: >-\n  b\n\n# c\n" | yamlfmt -formatter "retain_line_breaks=true" -
Actual Input:
a: >-
  b

# c

Actual Output:
a: >-
  b #magic___^_^___line
# c

Expected Output:
a: >-
  b

# c

Thanks for making an issue! This looks similar to #86 so I'll keep tracking the work there. Keep an eye out there for updates. I'll close this issue for now.

Gotcha. Thanks for your work on yamlfmt! It's very useful!


Improving wasm loading logic

This improves the loading logic for the web and no-modules targets in the following ways:

It's now possible to pass a Promise<WebAssembly.Module>, Promise<ArrayBuffer>, or Promise<Response> to the wasm_bindgen function.
That also means that it's now possible to do wasm_bindgen(fetch("foo")) which previously wasn't possible.
This makes it easier to use wasm-bindgen when serving the .wasm from cache (e.g. in service workers).

Some code duplication has been removed.

The logic is now much clearer and easier to understand.

The old logic
function init(module) {
    if (typeof module === 'undefined') {
        let src;
        if (self.document === undefined) {
            src = self.location.href;
        } else {
            src = self.document.currentScript.src;
        }
        module = src.replace(/\.js$/, '_bg.wasm');
    }
    let result;
    const imports = {};
    imports.wbg = {};
    imports.wbg.__wbg_alert_f5f3a7ed098bf3ca = function(arg0, arg1) {
        alert(getStringFromWasm0(arg0, arg1));
    };

    if ((typeof URL === 'function' && module instanceof URL) || typeof module === 'string' || (typeof Request === 'function' && module instanceof Request)) {

        const response = fetch(module);
        if (typeof WebAssembly.instantiateStreaming === 'function') {
            result = WebAssembly.instantiateStreaming(response, imports)
            .catch(e => {
                return response
                .then(r => {
                    if (r.headers.get('Content-Type') != 'application/wasm') {
                        console.warn("`WebAssembly.instantiateStreaming` failed because your server does not serve wasm with `application/wasm` MIME type. Falling back to `WebAssembly.instantiate` which is slower. Original error:\n", e);
                        return r.arrayBuffer();
                    } else {
                        throw e;
                    }
                })
                .then(bytes => WebAssembly.instantiate(bytes, imports));
            });
        } else {
            result = response
            .then(r => r.arrayBuffer())
            .then(bytes => WebAssembly.instantiate(bytes, imports));
        }
    } else {

        result = WebAssembly.instantiate(module, imports)
        .then(result => {
            if (result instanceof WebAssembly.Instance) {
                return { instance: result, module };
            } else {
                return result;
            }
        });
    }
    return result.then(({instance, module}) => {
        wasm = instance.exports;
        init.__wbindgen_wasm_module = module;

        return wasm;
    });
}

The new logic
async function load(module, imports) {
    if (typeof Response === 'function' && module instanceof Response) {

        if (typeof WebAssembly.instantiateStreaming === 'function') {
            try {
                return await WebAssembly.instantiateStreaming(module, imports);

            } catch (e) {
                if (module.headers.get('Content-Type') != 'application/wasm') {
                    console.warn("`WebAssembly.instantiateStreaming` failed because your server does not serve wasm with `application/wasm` MIME type. Falling back to `WebAssembly.instantiate` which is slower. Original error:\n", e);

                } else {
                    throw e;
                }
            }
        }

        const bytes = await module.arrayBuffer();
        return await WebAssembly.instantiate(bytes, imports);

    } else {

        const instance = await WebAssembly.instantiate(module, imports);

        if (instance instanceof WebAssembly.Instance) {
            return { instance, module };

        } else {
            return instance;
        }
    }
}

async function init(input) {
    if (typeof input === 'undefined') {
        let src;
        if (self.document === undefined) {
            src = self.location.href;
        } else {
            src = self.document.currentScript.src;
        }
        input = src.replace(/\.js$/, '_bg.wasm');
    }
    const imports = {};
    imports.wbg = {};
    imports.wbg.__wbg_alert_f5f3a7ed098bf3ca = function(arg0, arg1) {
        alert(getStringFromWasm0(arg0, arg1));
    };

    if (typeof input === 'string' || (typeof Request === 'function' && input instanceof Request) || (typeof URL === 'function' && input instanceof URL)) {
        input = fetch(input);
    }

    const { instance, module } = await load(await input, imports);

    wasm = instance.exports;
    init.__wbindgen_wasm_module = module;

    return wasm;
}

Nice!
Mind adding some documentation in the book as well indicating the new types that can be passed into the initialization function?

@alexcrichton Done.

👎

Am I missing something, or does this not negate the benefits of instantiateStreaming? As far as I can tell, this awaits for the fetch to be finished before passing it to instantiateStreaming.

Am I missing something, or does this not negate the benefits of instantiateStreaming? As far as I can tell, this awaits for the fetch to be finished before passing it to instantiateStreaming.

@TethysSvensson No, because fetch only returns a Response (which just contains the headers, not the body). So the body is still streamed as it should be.
And even if you pass in a fetch Promise directly, the spec will simply wait for it to return a Response before continuing, so it's exactly the same as using await:
https://webassembly.org/docs/web/#process-a-potential-webassembly-response

Upon fulfillment of sourceAsPromise with value unwrappedSource:

@TethysSvensson No, because fetch only returns a Response (which just contains the headers, not the body). So the body is still streamed as it should be.
And even if you pass in a fetch Promise directly, the spec will simply wait for it to return a Response before continuing, so it's exactly the same as using await:
https://webassembly.org/docs/web/#process-a-potential-webassembly-response

Upon fulfillment of sourceAsPromise with value unwrappedSource:


Horizontal scaling via sharding

What is missing?
Horizontal Scaling via Sharding
Why do we need it?
Right now we are collecting approx. 2 Million series with 2 hour blocks and using thanos for long term storage and deduping at query time. Despite these small block sizes, the ram usage is around 11 gigs. We run in HA mode so multiple prometheus pods all have about the same ram usage.
As we scale out more metrics, it will become unfeasible to keep scaling vertically (currently they are on 32 gig boxes so we do have room).
Environment
Kubernetes 1.15.7
Thanos 0.10.1

Prometheus Operator version: quay.io/coreos/prometheus-operator:v0.37.0

Anything else we need to know?:
I believe I know a way to fix implement this simply enough. We can add an option to the operator like so:
sharding:
  enabled: true
  numShards: 2
  hashKey: __address__

In turn, this would add something like this to each of the jobs in the prometheus config:

On prometheus-replica-0:

- job_name: "foo"
  metrics_path: "/metrics"
  kubernetes_sd_config:
  - role: endpoints
  relabel_configs:
    - source_labels: [__address__]
      modulus:       2
      target_label:  __tmp_hash
      action:        hashmod
    - source_labels: [__tmp_hash]
      regex:         ^0$
      action:        keep

On prometheus-replica-1:

- job_name: "foo"
  metrics_path: "/metrics"
  kubernetes_sd_config:
  - role: endpoints
  relabel_configs:
    - source_labels: [__address__]
      modulus:       2
      target_label:  __tmp_hash
      action:        hashmod
    - source_labels: [__tmp_hash]
      regex:         ^1$
      action:        keep

HA would still be available by having more replicas than shards. In the above example that shards using modulus 2, prometheus-replica-2 config would match prometheus-replica-0 config.
Deduping would be done by thanos and/or alertmanager.
To merge results, federation can be enabled and then the master can scrape them and consolidate OR use a third party tool like thanos to do this for you.
I would be happy to implement this feature if this proposal is accepted.

We use this approach and it works like a charm.

I always imagined sharding to be another layer on top of the Prometheus object, but I think something in between could actually be achieved here as well, as the approach is quite elegant I think. How about we essentially calculate the replicas of the statefulset by multiplying shards * replicas replicas being the amount of times to replicate data, and shards in how many to split it. If we can consistently generate a scrape config without having another layer of abstraction then I would actually like this even better. It implies specifying the ordinal and total replicas, but we already have mechanisms for that, so I actually like this a lot.
Going even further, that means we could make shards the field specified for the /scale subresource, which would even allow us to autoscale shards using the HPA.
I would even say that for a start we wouldn't even make the hashed label configurable, I would want to see a real scenario before making it configurable. And typically we don't do the enabled thing in our APIs, so just if shards is unspecified or smaller than 1 we default to 1, and if larger then we enable sharding.

@brancz I like the idea of making shards part of /scale however I would say this isn't required for MVP.
I also like the idea of simply specifying shards at the beginning so essentially the config would have this by default:
shards: 1

I don't think we can have just one stateful set though. Since each each pod in the statefulset mounts the config volume the only way to modify it after mount would be some sort of hacky script on the pod that used the pods index to set the shard.
I propose one statefulset per shard using the following naming schematic:
values.yaml:
replicas: 2
shards: 2

Generated pods:
#shard 0
foo-shard-0-0
foo-shard-0-1
#shard 1
foo-shard-1-0
foo-shard-1-1
...

To be clear, pods foo-shard-0-0 and foo-shard-0-1 would be duplicates that keep parity with the current setup. Let me know what you think!

Since each each pod in the statefulset mounts the config volume the only way to modify it after mount would be some sort of hacky script on the pod that used the pods index to set the shard.

We already have this for external labels to label each replica uniquely, and it’s not so hacky I think, as it’s intentionally built into our reload sidecar already.

Well then that sounds like exactly what we want! What exactly is the process to get this implemented? As in is there more discussion that needs to happen or should I start working on a PR?

You’ve largely already outlined the implementation details in this issue so from my side I would say you can get right to implementing it :)

Looks like it took a bit more than I anticipated. I took a stab at it in #3241.


Cthub 150 - Build eligibility page

Dang... package-lock.json conflicts are the worst. Do you think we should just add it to the .gitignore file? Maybe that's frowned upon???

it was in the gitignore before but i think it was removed, so i'm guessing it was intentional?


Intent Recognition with Microsoft Audio Stack

Describe the bug
I'm unable to use MAS with an IntentRecognizer with StartContinuousRecognitionAsync. I am able to use my application otherwise perfectly fine (without trying to specify audioProcessingOptions).
To Reproduce
Steps to reproduce the behavior:
var speechConfig = SpeechConfig.FromSubscription(_options.LuisKey, _options.LuisRegion);
var audioProcessingOptions = AudioProcessingOptions.Create(AudioProcessingConstants.AUDIO_INPUT_PROCESSING_ENABLE_DEFAULT);
var audioConfig = AudioConfig.FromStreamInput(_audioStream, audioProcessingOptions);
var recognizer = new IntentRecognizer(speechConfig, audioConfig);

Expected behavior
Intent recognizer works with MAS.
Version of the Cognitive Services Speech SDK
Microsoft.Cognitive.Speech 1.22.0
Platform, Operating System, and Programming Language

OS: Linux (using FROM mcr.microsoft.com/dotnet/sdk:6.0)
Programming language: C#

Additional context

Caught exception processing conversation stream Exception with an error code: 0x5 (SPXERR_INVALID_ARG)
System.ApplicationException: Exception with an error code: 0x5 (SPXERR_INVALID_ARG)
at Microsoft.CognitiveServices.Speech.Internal.SpxExceptionThrower.ThrowIfFail(IntPtr hr)
at Microsoft.CognitiveServices.Speech.Recognizer.FromConfig(GetRecognizerFromConfigDelegate fromConfig, SpeechConfig speechConfig, AudioConfig audioConfig)
at Microsoft.CognitiveServices.Speech.Intent.IntentRecognizer..ctor(SpeechConfig speechConfig, AudioConfig audioConfig)

Might be related to this issue https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1717

Internal work ref: 5084195

AudioProcessingOptions.Create(int audioProcessingFlags) can only be used when the input is from a microphone and the microphone is a microphone array. This method tries to retrieve microphone array geometry from the driver.
When you use this method with stream or file input, MAS cannot figure out the microphone array geometry required for audio processing. That's why you are getting the exception.
You will have to use either
AudioProcessingOptions.Create(int audioProcessingFlags, PresetMicrophoneArrayGeometry microphoneArrayGeometry, SpeakerReferenceChannel speakerReferenceChannel)
or
AudioProcessingOptions.Create(int audioProcessingFlags, MicrophoneArrayGeometry microphoneArrayGeometry, SpeakerReferenceChannel speakerReferenceChannel)
to provide the microphone array geometry of the microphone that was used to record the audio that you are passing as stream input.

Closed as answered; the API documentation has been updated with corresponding notes and will be published with the Speech SDK 1.29.0 release.


Update a restriction for another test suite

Follow up for: https://github.com/apache/beam/pull/30863 https://github.com/apache/beam/issues/30852

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Run Python PostCommit Dependency

ran gradlew :sdks:python:test-suites:tox:py38:testPy38TensorflowHubEmbeddings-014  manually and checked that tox suite goes beyond the point of failure that happens on master.

thank you!


chore(config): update devnet-amplifier with new chain deployments

we can remove the outdated deployments after a week or so once services/integrators migrate to it


Typescript error for partial-element-with-additional-attributes.d.ts

I believe the error stems from the syntax [key: data${string}]: string; being invalid because TypeScript does not support template literals as keys in index signatures. There seems to be an issue with the TypeScript definitions provided by this library. Is there a workaround for this?

Hi @dmdinh22 , thanks for reporting!
The issue you mention can be easily solved by upgrading the Typescript version you're using to >=4.4 (see this documentation page, which describes when allowing template literals as index types was introduced to Typescript).
What's clear though is that the library is not clear about which versions of Typescript are supported, and we'll address that.

:tada: This issue has been resolved in version @typeform/embed-v4.4.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Wait for #1381] [capi] Implement dataset_set_property_for_usage

[Pending commit: #1381]

[capi] Implement dataset_set_property_for_usage

This patch implements `ml_train_dataset_set_property_for_usage` and it's
test

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1382. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.


Migrate existing Azure AI Face service to TypeSpec

Data Plane API - Pull Request
Face API is an existing service & restful API for 5+ years as documented in What is the Azure AI Face service?. We continue to extend with new API and the latest addition was Detect liveness in faces that was released in Ignite in Nov 2023 as a Public Preview. At that time, we wanted to release a REST SDK for detect liveness too. However, we were told we cannot use OpenAPI spec that we used to create SDK for old APIs like Detect/Verify/Identify, etc. anymore and TypeSpec is the only option to release new SDK. Our current objective is to release an All-In-One SDK for all the APIs by using TypeSpec, including the existing Face Detect, Verify, Identify, FindSimilar, and the latest detect liveness.
API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[x] a public preview
[ ] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/cognitiveservices/data-plane/Face/preview/v1.0-preview/Face.json
Updated paths: https://github.com/Han-msft/azure-rest-api-specs/blob/hachiang/face-typespec/specification/ai/data-plane/Face/preview/v1.1-preview.1/Face.json

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Checks stuck in `queued` state?
If the PR CI checks appear to be stuck in `queued` state, please add a comment with contents `/azp run`.
This should result in a new comment denoting a `PR validation pipeline` has started and the checks should be updated after few minutes.

@AzureRestAPISpecReview
We got Breaking Change check failure after we copied our previous version swagger by #28424.
I briefly go through them and believe most of them are annotation difference caused by migrating to TypeSpec.
I also added some suppressions for Swagger LintDiff.
Please help take a look.
We'd like to merge it if there is no concern.

/azp run

/azp run


New version: QMPlay2.QMPlay2 version 23.02.05

Result: Installation Failed

Manifest
Add/Remove Programs

Name
QMPlay2

Version
23.02.05

Publisher
QMPlay2

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 5789
Microsoft Reviewers: Open in CodeFlow

/AzurePipelines run


Does mosby handle situation when View is de-attached on screen rotation?

What if some async work in Presenter will be finished in the moment when View is de-attached?
For example:

User click 'do work' button
Presenter does request to server(and doesn't stop work in de-attach method)
Screen rotation in progress and View de-attached
Presenter receives response from the server and wants to notify View about it. But there is no attached View.
View is attached(in another orientation) and restore itself from ViewState. Response is executed, but View is not changed.

Is it real situation? Or de-attaching and attaching View  is happening in one moment?
There is my solutaion:
public abstract class BaseMvpPresenter<V extends MvpView> extends MvpBasePresenter<V> {

    private Queue<Runnable> executeOnAttachQueue = new LinkedList<>();

    public void executeOnAttachView(Runnable runnable) {
        executeOnAttachQueue.add(runnable);
    }

    public void executeGranted(Runnable runnable) {
        if(isViewAttached()) {
            runnable.run();
        } else {
            executeOnAttachView(runnable);
        }
    }

    @Override
    public void attachView(V view) {
        super.attachView(view);
        while (!executeOnAttachQueue.isEmpty()) {
            Runnable nextElement = executeOnAttachQueue.remove();
            nextElement.run();
        }
    }
}

But do I really need it?

See github pages FAQ:
http://hannesdorfmann.com/mosby/summary/

Can the Presenter and its view be out of sync during a screen orientation change?
Excellent question. Mosby assumes that all interaction from Presenter with the View happens on android’s main UI thread. Hence the answer is no that cannot happen since screen orientation changes are executed on the main UI thread as well. So either is a screen orientation executed completely (view reattached) or the presenter invokes the views method after view is reattached since both run on main UI thread or the presenter invokes the views methods before starting screen orientation change.

So as long as your Presenter invokes View methods on main UI thread your solution is not needed.

I got it. If I use Retrofit&RxJava, then I don't need things like isViewAttached() or my solution.
        api.getProjects(accountManager.getUserId())
                .subscribeOn(ioScheduler)
                .observeOn(AndroidSchedulers.mainThread())

As my code always will run on Android UI Thread. But, your example calls isViewAttached() method anyway. Is it mistake or there is any reason for it?
I read below:

Why do I need isViewAttached() check in MvpBasePresenter
In a perfect world you wouldn’t need this check. As already said, in Mosby all interaction from Presenter to View happens on android’s main UI thread. Since screen orientation changes are also executed on the main UI thread it is not possible to run into the scenario where view is detached while presenter still wants to update the UI, right? However, we are not living in a perfect world. Let’s say you use an AsyncTask to make an http request and the Presenter gets the result back and updates the View. If you forget to cancel this AsyncTask after the View has been destroyed (i.e. your apps user press the back button, presenter.detachView(false) will be called) then you can run into the scenario that the View is detached from Presenter and then the Presenter gets the result back from AsyncTask and wants to update the View which is null (because detached). So the isViewAttached() check is basically some kind of safety net. In a perfect implementation you wouldn’t need it (presenter - view interaction on main thread, cancel all async background task when view gets destroyed etc.), but in general I would say it wouldn’t hurt to drive with a safety net.

But still don't get it.
Is there any forum about Mosby to discuss questions like this?

If you are sure that you are unsubscribing your RxJava subscription properly in presenter.detachView(false) then you shouldn't need isViewAttached()
I did so in the examples (and also in my production code) because

but in general I would say it wouldn’t hurt to drive with a safety net.

There is no Mosby forum. I would like to see those kind of question asked on Stackoverflow because it is better searchable through google so it is easier to discover those question and answers for others.
I have seen (right now) that you have already asked this question on stackoverflow. Usually I check stackoverflow regularly for new question about mosby, but didn't yesterday ...

I understood. Well, thanks again!


Frontend goroutine leak

Describe the bug
On oCIS experimental we saw a goroutine leak in the frontend service on uploads:
goroutine profile: total 4505
4479 @ 0x43e376 0x436b17 0x46a209 0x4aabd2 0x4abf3a 0x4abf28 0x528869 0x53d245 0x7a0811 0x722f5f 0x723b4f 0x723da7 0x727dd9 0x79bb99 0x79bb9a 0x7a1daa 0x7a62e5 0x4702e1
#	0x46a208	internal/poll.runtime_pollWait+0x88		runtime/netpoll.go:305
#	0x4aabd1	internal/poll.(*pollDesc).wait+0x31		internal/poll/fd_poll_runtime.go:84
#	0x4abf39	internal/poll.(*pollDesc).waitRead+0x259	internal/poll/fd_poll_runtime.go:89
#	0x4abf27	internal/poll.(*FD).Read+0x247			internal/poll/fd_unix.go:167
#	0x528868	net.(*netFD).Read+0x28				net/fd_posix.go:55
#	0x53d244	net.(*conn).Read+0x44				net/net.go:183
#	0x7a0810	net/http.(*connReader).Read+0x170		net/http/server.go:786
#	0x722f5e	bufio.(*Reader).fill+0xfe			bufio/bufio.go:106
#	0x723b4e	bufio.(*Reader).ReadSlice+0x2e			bufio/bufio.go:372
#	0x723da6	bufio.(*Reader).ReadLine+0x26			bufio/bufio.go:401
#	0x727dd8	net/textproto.(*Reader).readLineSlice+0x98	net/textproto/reader.go:56
#	0x79bb98	net/textproto.(*Reader).ReadLine+0x78		net/textproto/reader.go:37
#	0x79bb99	net/http.readRequest+0x79			net/http/request.go:1036
#	0x7a1da9	net/http.(*conn).readRequest+0x249		net/http/server.go:994
#	0x7a62e4	net/http.(*conn).serve+0x344			net/http/server.go:1916

Steps to reproduce
TODO: verify if it also happens on master
Expected behavior
Goroutines are closed after work is done / timeout it exceeded
Actual behavior
Uploads lead to a increase goroutines in the frontend service

Will try to reproduce on master

OK, removing from GA blockers list.


Fix doc links to point to new DTC

There are links in the README that refer to dev.twitter.com - need to update for the new website structure.

Also, apps.twitter.com is going away in future, fix for new dev portal process.

... and fix the dev forum link in the README too....

Underway in enhancements branch


3.0.0-ALPHA3

This change is not substantial enough to merge. In future, if you want to make change only to the plugin.yml to add support for a new API, you must provide some evidence that you have tested your work.


Ditto. Also ive heard from others that you have to toggle damage off every time you get into a new car.

This is correct, when we have the time it will be fixed.

Thanks, i am trying to find as much bugs as possible so that mello trainer can become the best trainer for fivem. :D


v1.7 backports 2020-10-21

#13683 -- endpoint: Fix goroutine leak when EP is deleted (@christarazi)

Once this PR is merged, you can update the PR labels via:
$ for pr in 13683; do contrib/backporting/set-labels.py $pr done 1.7; done

test-backport-1.7

test-backport-1.7


Code Release data

Hi Andy,
very interesting work! Would you have an expected code release data?
Thanks!

Thanks for your patience; we have released the code.


Benchmark theta

I believe we need to keep the $\theta$ parameter at its benchmark value, and not update with each solve.

As a start, I am working on a conditional within the $\theta$ function
if [variable name combination of theta, pf, and input] doesn't have a value
   generate variable and assign value
   otherwise just return the variable

and then it will be evaluated in the other functions as it is now.
But I think there might be better ways to do it...
maybe set it as a parameter, evaluated before any solve. That would fix the problem of having to make sure to run a benchmark solve.
or set a benchmark value for the Inputs and Outputs, separate from the quantity
or...

I worked on calculating and storing the share parameters before the model goes to JuMP, and that worked!
Somehow I've broken other things in the process, so... work is ongoing.

Looks like 2 issues (at least).

Sharaparams aren't set up for indexing so we're generating non-unique variable names.
Something has gone wrong with the evaluation of other JuMP parameters in production equations.

As far as I can tell so far, it works. I'm not sure it's as elegant, efficient, or future-proof as it could be.
I reckon I could add more tests and test a few more things, but I'm a bit stoked it's working, and maybe it will be ok to move forward, add tests later on a different branch, and try this update on the transformation elasticities and taxes branches.

I believe that this work is already merged into the transf_elas branch which is ready for PR.
Assuming that is true, this brach/PR is redundant.

[ ] Wait for final review and merge of transf_elas
[ ] double-check redundancy of this branch
[ ] delete this PR draft and the branch

This was merged into transf_elas, and that branch was finished and merged. This is a dead end.


tf_record issue while following tutorial

ValueError: Shape must be rank 0 but is rank 1 for '{{node DecodeJpeg}} = DecodeJpegacceptable_fraction=1, channels=3, dct_method="", fancy_upscaling=true, ratio=1, try_recover_truncated=false' with input shapes: [?].
using this tutorial: https://keras.io/examples/keras_recipes/tfrecord/

@abhishekbalu,
On running the Colab Notebook, I am facing a different error due to the invalid GCS_PATH. Please find the gist of it here.
Could you please provide the Python script/notebook you are running along with the dataset, so that we can reproduce the issue on our end. Thanks!

the following is my gist. The link to DATASET. I am using the nuscene DATASET

Hi any updates on this i am kind of stuck here for a week. Please help me out.

@abhishekbalu,
Sorry for the delayed response. I do not have access to the drive link, could you please grant the required permissions to view the files. Thanks!

sorry for the delay. dataset and gist  these are sharable link

Was able to reproduce the issue with TF v2.3 and TF-nightly. Please find the gist of it here. Thanks!

@abhishekbalu,
Please check this comment from a similar StackOverflow query and let us know if it helps. Thanks!

@abhishekbalu,
Can you please respond to the @amahendrakar's comment. Thanks!

I am passing a tensor and not a scalar but still getting this error

@abhishekbalu Can you please share simple dataset. Current dataset is too big (7 GB). Thanks!


Should install scripts get their own directory?

I'm not sure. They'd have to all begin with changing to the parent directory, then...

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/24854099-should-install-scripts-get-their-own-directory?utm_campaign=plugin&utm_content=tracker%2F7646105&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F7646105&utm_medium=issues&utm_source=github).

If you decide to move them, you can use git rev-parse --show-toplevel to get the top-level directory of the project.  So for example, install-ubuntu-python2.sh has this line:
./install-base-python2.sh

If you moved install-ubuntu-python2.sh into a sub-directory (or deeper) then you could change that line to this:
BASE_INSTALL_SCRIPT="$(git rev-parse --show-toplevel)/install-base-python2.sh
exec "$BASE_INSTALL_SCRIPT"

That assumes install-base-python2.sh is still in the same top-level location it is now.  If you were to move it as well then you'd add the appropriate sub-directories after git rev-parse --show-toplevel.

Great notes, @ejmr! I will adhere to your advice if I decide to indeed put install scripts in their own directory.

I'm questioning if the install scripts are strictly necessary. They could easily be replaced with a little documentation, which would be better than creating install scripts for every platform under the sun...
My 2c anyway.

@alxcenz I agree that the installation scripts should be supplemented by documentation, explaining dependencies to install and things like that.  But it's always been my impression that the goal is to provide install scripts for commonly-used platforms, not all of them, since that would be overkill (which I felt like you were alluding to).

Yeah, I mean honestly the biggest benefit is just... I have a lot of friends that are using Unix-y platforms and they're not Python developers and they wanna checkout my work, so I'm just like "Well just run the install script." :) I know I could be creating binaries, distributions, packages, etc., but that honestly takes a whole lot more of my time compared to maintaining the install scripts. In the future, however, @alxcenz has the right idea--Hypatia will be large enough to have actual dedicate platform-specific ports being developed separately and the install scripts will be completely redundant, save the ability to easily install nightlies, but who cares about that (I don't see any reason a non-developer should be playing with nightlies).

Closed with #69.


Tweak Windows CLI docs

Dropped the windows terminal (developer cmd prompt seems to be there by default now) and arm toolchain troubleshooting (fixed) parts.


Update env-vars.md

Add missing env. vars and rename host and port variables.

Pull Request Test Coverage Report for Build 543039717

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 20.766%

Totals

Change from base Build 542943065:
0.0%

Covered Lines:
1996

Relevant Lines:
9852

💛  - Coveralls

I have documented them in my other pr #535


Use "(untitled)" as default name and require Save As first time

Currently the system uses gridnxn or puzzlenxn as the initial name when a new grid or puzzle is created.  Better to switch to (untitled).  Then do not allow file to be saved when untitled, only "Saved As" a real name.

Fixed.


Update Typescript Definitions and reorganized main script.

This closes #135.  Using the fix provided in that issue as a base and enhanced it.
Slightly optimized the definitions file
Both files have been reorganized a bit, having the functions sorted by alpha order, within its own IPv4 area, IPv6 area, and common convenience/utility functions.  This makes it slightly easier to read by hand.
Seeing that you're planning on pushing out 2.0.0 it may as well be reorganized.  I do think one should try converting the main js file into typescript, but that's up to you.
Signed-off-by: Kenneth Tran<EMAIL_ADDRESS>

Seeing that you're planning on pushing out 2.0.0 it may as well be reorganized.

I'm actually not sure what you're referring to here?

My bad, I mean 'reorganized' as in the organizing the function names alphabetically.  So what I have done is grouped the general utility ones at the bottom of the file, and grouped the rest of the functions into their IPv4 and IPv6 sections.  It still works the same in the user perspective, but if one was to read the source code, I think some organization would help.
This was the same with the TypeScript Definitions.

There's no functional change here at all, right?

There shouldn't be any functional changes, with the exception of the TypeScript definitions.  Other Minor changes would be a small clarification/format on some error message but that is it.


How to set a list of SSL Cipher Suites

What is the functional equivalent of:
CURL *curl = curl_easy_init();
if(curl) {
  curl_easy_setopt(curl, CURLOPT_URL, "https://example.com/");
  curl_easy_setopt(curl, CURLOPT_SSL_CIPHER_LIST, "TLSv1");
  ret = curl_easy_perform(curl);
  curl_easy_cleanup(curl);
}

using RestClient-cpp?
See: CURLOPT_SSL_CIPHER_LIST

There isn't a way to do this yet, but I agree this would be useful to have.

@badfd it would be pretty easy to add a method to the Connection for power users that wraps curl_easy_setopt, without giving the curl handle.  Would that give you enough of what you need?  Feel free to make a PR for that change so it can be reviewed.


Timestamp improvement

I have linting set to auto-lint on save.
As a habit, I always lint files, even if I did not modify. The issue is if there was no linting issue (the file was not changed), the modified date is updated regardless, even if nothing was actually modified.
It gets worse when I lint all files, because all files will have their modified timestamps changed. This is an issue esp when using version control, because this will modify all files all the time.
NB: I don't think anyone would want to change the timestamps of an unchanged file, so maybe it would be better to just add this improvement without a settings checkbox. Should be a minor version change.

I am curious as to whether it actually updates the date modified on the file. If that is the case, the update of the date modified is accurate. However if it does not match the date modified on the file, I can see what your saying being the case.
Are you able to verify whether the date modified matches the file metadata, @usernotnull ?

From what I can tell, this is a bug assuming it is happening. I just need to reproduce the scenario and then I can see about fixing this.

Ok, so I usually lint by pressing CTRL + S to save & lint. I assumed the issue may be from obsidian since it might be rewriting the file even if there are no changes.
But I also tried with the command Lint current file, and the same thing is happening, the YAML modified date always changes to the current date too even if there are no changes.
This also applies to lint all file in vault command.
The modified date YAML on lint is the current date of command execution. It will coincide with the file modified date since the file is actually modified.
I hope I understood and answered your question.

This does seem like a bug to me, but it will take some time to determine how to properly do this given the order rules run in is not defined.

Were you able to reproduce the issue?

I was not able to yet, but I did confirm that just running lint file/ctrl+s does not update the date updated unless a change is present in the file.

I'm running the latest obsidian on Windows 11.
I created a new vault, added only the linter plugin.
In the settings of linter, I have enabled to keep track of the date the file was last edited in the YAML front matter.
Now, every time I run Lint the current file, Linter updates the date modified to the current date.
Now, this could be your intended use of that rule, to actually always rewrite the modified date to the time when Linter runs.
But I'm suggesting to see if the modified date needs to be updated if nothing was actually linted in the file (as in, if no other rule was actually applied and the original file not changed by linter, the modified rule date should not be applied).

I am close yo a fix on this, but I am trying to account for the situation where the date modified key is present and blank. I just need to work out a couple of kinks, and I will see about moving the fix forward.

I have what I believe to be a fix to this issue pretty much ready to merge. I am just checking with the owner of the repo to see if I can remove what I believe to be some old logic. Once I have that answer, I will merge this change.


[feature] Multiple Identity Center / SSO Session support

Is your feature request related to a problem? Please describe.
I have a situation where I need to manage separate AWS Identity Center connections independently (and potentially simultaneously). Currently go-aws-sso cannot hanle more than one login at a time, as the Identity Center/SSO tokens get stored in a single file with a hard-coded path.
Describe the solution you'd like
I would like the tool to be able to manage separate access token files for separate SSO sessions.
Describe alternatives you've considered
I have a proof-of-concept hack which provides an optional CLI flag to override the default access-token.js path. This solves for my basic use case, but aligning token file storage with credentials profiles manually. At the minimum, the profile probably needs to be written with the flag/path when the tool writes it to the file. This is probably the quickest way to add the functionality, but may be a bit awkward to explain, document, and use (and it requires semi-manual management of the token file names).
I started looking at the awscli [sso-session] configurations typically stored in .aws/config. I wonder if it would make sense to leverage those, rather than directly having the user decide on access token file path names. The CLI flag could then be based on the session name, rather than the cache file location. Conceivably, using the sso-session config could supplement, override, or replace the go-aws-sso/config.yml file.
This leads to a second thought about researching whether we could use/share the .aws/sso/cache files that are generated by aws sso login. At a glance it just looks like there are minor marshalling differences and the file name is based on a hash of the session name and start url, but there may have been Good Reasons why this tool was designed to ignore those files?
Additional context
Is it worthwhile to honor the AWS_CONFIG_FILE and AWS_SHARED_CREDENTIALS_FILE environment variables? This isn't directly relevant to solving my issue, but I could imagine scenarios where overriding the location for these files would allow the tool to be used in multiple contexts.
I'm willing to put some time into implementation, and I have some further thoughts and discussion on implementation details, but I want to make sure the scope of this functionality is desirable in the tool before getting too far into the weeds.

Hi there @joshrivers. Thanks for opening this issue, and deep sorry for the late reply.
Your feature request makes totally sense to me.

but there may have been Good Reasons why this tool was designed to ignore those files?

I wanted to have go-aws-sso to be independent of any 3rd party application (especially the aws cli with its Python dependencies)

Is it worthwhile to honor the AWS_CONFIG_FILE and AWS_SHARED_CREDENTIALS_FILE environment variables

It is!

whether we could use/share the .aws/sso/cache files that are generated by aws sso login

That would be fine for me, so go-aws-sso could act as a drop-in replacement and can leverage existing config.
Happy to hear your thoughts on that!

I wanted to have go-aws-sso to be independent of any 3rd party application (especially the aws cli with its Python dependencies)

Exactly why I love it, too.

That would be fine for me, so go-aws-sso could act as a drop-in replacement and can leverage existing config.
Happy to hear your thoughts on that!

Excellent. I haven't done a thorough analysis yet, because I didn't want to presume that I could take the tool in a different direction without asking and I didn't want to do throwaway work.
What I'm thinking of doing:

Determining a method of having more than one ClientInfo / access-token file. Preferably this will mean using the same file naming and creation scheme used by the official CLI.
a) I will probably need to come up with an answer to "is this a 'breaking'" change or a backwards-compatible one. Perhaps it will be easy to re-use the existing session file while also supporting the new file format and location. Perhaps users will need to do a one-time re-login on upgrade.
b) Presumably the new file locations would be preferred for future use, but it is conceivable that both mechanisms could be maintained side-by-side and usage determined on configuration.
Since you like the idea of supporting the official config/credentials file, I would want to implement alternative SSO endpoints and configs by reading sso-session sections from the config files. I think these would be selected by a new flag on calls to the assume command.
a) To be determined: which config source has precedence if both exist for the default SSO Session.
b) Is it desirable to have first-run for go-aws-sso just use the SSO start-urls from .aws/config?
c) I think there would be support for writing to .aws/config as well as .aws/credentials. The goal would be to have this non-conflicting with the current AWS CLI usage of those files.
In the process of implementing the above, I would like to do some refactoring in internal/aws.go and pkg/sso/ to make setup some boundaries between the components, use struct-and-method patterns and some interfaces to inject dependencies, and make things a bit more testable. This will probably address #163 and I may clean up #12. It might be a breaking change on #65, (but I'd hope it makes the library API cleaner)

I think I have some additional small features and/or bugfixes in my notes. I guess we should decide if a bigger refactor/feature MR is a good thing, or if I should do some more work to layer the above into multiple MRs so that too much doesn't change at once. I expect that I'll have solid testing, so I don't expect breakage, but I can put in the extra effort if you'd rather have several phases of review rather than a single bigger MR.

First: Thanks for sticking around and investing so much time!

A breaking change is not a problem for me. People can choose to not upgrade - they won't miss any new core functionality. And folks can handle manual one-time re-logins, I am pretty sure 🤞🏻

2.a) see b)
2.b) Mhh. I tend to no. I would not expect to have it watching for 3rd party config files - but it would be okay if we say we are going to cannibalize this file and make it go-aws-sso exclusive, meaning we use it, we write it, we override it.
2.c) Totally agree

Go ahead! I am looking forward to it!

PR size: You decide. I can handle both. A bigger one may take a bit longer (with discussions etc.) - but would be easier for you.
Thanks!

Hey @theurichde, I wanted to give you an update. I had gotten a day or two into my refactor and was looking up some API details when I tripped over aws-sso-cli. I was excited for a few minutes, thinking I'd be able to grab some implementation details and hints from their implementation...then I was sad, because it looks like that project has implemented just about every feature and enhancement that I was looking to add to go-aws-sso. I was enjoying doing the code work, but for now I think I'm going to use the synfinatic CLI to solve my SSO needs rather than work up a redundant implementation.
Thanks for the invitation to contribute.


letsencrypt giving me error "certificate public key must be different than account key"

When running cert-manager I keep getting an invalid order from letsencrypt with an error with a reason:
Error finalizing order :: certificate public key must be different than account key
I'm not using helm but here is the script i'm running:
kubectl create namespace cert-manager
kubectl label namespace cert-manager certmanager.k8s.io/disable-validation=true

# Install the CustomResourceDefinition resources
kubectl apply -f https://raw.githubusercontent.com/jetstack/cert-manager/release-0.6/deploy/manifests/00-crds.yaml

kubectl apply -f ./letsencrypt-Issuer.yml

kubectl apply -f https://raw.githubusercontent.com/jetstack/cert-manager/release-0.6/deploy/manifests/cert-manager.yaml --validate=false

My issuer looks like:
apiVersion: certmanager.k8s.io/v1alpha1
kind: Issuer
metadata:
  name: letsencrypt-issuer
spec:
  acme:
    server: https://acme-staging-v02.api.letsencrypt.org/directory

    # Email address used for ACME registration
    email: [myemail]
    # Name of a secret used to store the ACME account private key
    privateKeySecretRef:
      name: vs-portal-tls-secret
    # Enable the HTTP-01 challenge provider
    http01: {}

I have this set on my ingress config:
    kubernetes.io/ingress.class: "nginx"
    certmanager.k8s.io/acme-challenge-type: http01
    certmanager.k8s.io/issuer: letsencrypt-issuer
spec:
  tls:
    - hosts:
        - [mydomain]
      secretName: vs-portal-tls-secret

I'm using Microsoft AKS with Kubernetes v1.9.11 and cert-manager 0.6.

As the error says, you shouldn't be using the same secretName for both the Let's Encrypt account private key (the one referenced by issuer.spec.acme.privateKeySecretRef) as well as the Certificate's actual certificate (ingress.spec.tls[].secretName).
Usually, you'd name your account private key secret something more like letsencrypt-staging-private-key and name your ingresses certificate something similar to what you've got now (e.g. vs-portal-tls-secret).
The Issuer private key is used to identify your ACME account with the ACME server, and it has no relation to the private key that is used to serve with.
Hope that makes sense!
If you're still struggling, feel free to drop into the #cert-manager channel on the Kubernetes Slack and we can give you a hand!
/triage support
/close


Update 1.0.x branch jetty to 9.3.20.v20170531

Update Jetty version from 9.3.9.v20160517 to 9.3.20.v20170531 to address CVE-2017-9735 per #2113

@msymons I think I merged this PR too soon.
There seem to be some problems with the Gzip filter: https://travis-ci.org/dropwizard/dropwizard/builds/259260838#L7023-L7026

I fixed it with 092432cc49dc0240fac9ea38a3a73b58fadb0e48


Not working with rootless Docker

I'm trying this out in a virtual machine with rootless Docker. The container won't start:

dsm  | ❯ Starting Virtual DSM for Docker v7.00...
dsm  | ❯ For support visit https://github.com/vdsm/virtual-dsm
dsm  |
dsm  | mknod: /dev/net/tun: Operation not permitted
dsm  | ❯ ERROR: TUN device missing. Please add the following setting to your container: --cap-add NET_ADMIN
dsm exited with code 0

My docker-compose.yml:
services:
  dsm:
    container_name: dsm
    image: vdsm/virtual-dsm
    environment:
      DISK_SIZE: "16G"
      DISK_FMT: "qcow2"
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 5000:5000
    volumes:
      - /home/me/docker/virtual-dsm/dsm:/storage
    restart: unless-stopped
    stop_grace_period: 2m

The error suggests I should add cap_add, which is already present in my compose file. Is there anything else I can try?

Oh my that actually works, thank you :-)


302 Error when trying to install OracleJDK

Looks like there is a change in the URL for the OracleJDK:
scoop install oraclejdk
Updating Scoop...
Updating 'extras' bucket...
Scoop was updated successfully!
Installing 'oraclejdk' (8u151-b12).
The remote server returned an error: (302) Moved Temporarily.
URL http://download.oracle.com/otn-pub/java/jdk/8u151-b12/e758a0de34e24606bca991d704f6dcbf/jdk-8u151-windows-x64.exe is not valid

Thanks

I'm not sure why the URL is returning a 302 because it looks like it may be valid:
http://download.oracle.com/otn-pub/java/jdk/8u151-b12/e758a0de34e24606bca991d704f6dcbf/jdk-8u151-windows-x64.exe
I also notice that there is a u152 build at:
http://download.oracle.com/otn-pub/java/jdk/8u152-b16/aa0333dd3019491ca4f6ddbe78cdb6d0/jdk-8u152-windows-x64.exe

We’re on 162 now.


No module named 'roi_align_api'

Make sure your device is CUDA enabled. Build and install source code of roi_align_api and rod_align_api.
how to do this step? can you show the file of yours?

make_all.sh

确保您的设备启用了 CUDA。构建并安装 roi_align_api 和 rod_align_api 的源代码。
这一步该怎么做呢？你能展示一下你的文件吗？

How did you do this step to build and install the source code of roi_align_api and rod_align_api? I kept getting errors and the path to the THC/THC.h file was missing.

The "make_all.sh" script depends on two files in "rod_align/make.sh" and "roi_align/make.sh". You need to adjust the -arch parameter for your GPU. For example, a 2080ti GPU, with a Compute Capability of 7.5 (found on https://developer.nvidia.com/cuda-gpus), requires setting -arch=sm_75. For a 4090 GPU, it should be -arch=sm_89. Also, update the CUDA_HOME path to match your CUDA installation directory.


Multiple breaking changes

This PR brings several significant changes to Flexpart.jl:

The FlexpartDir object has been refactored to FlexpartSim
The NetCDFF version has been forced to 4.5.3 (solves issue #9)
Ensemble runs works better
FlexExtract has been moved to a separate repo, since both modules can work independently, and very little code was shared between both.

Codecov Report
Patch coverage: 65.34% and project coverage change: +0.64 :tada:

Comparison is base (d803176) 62.50% compared to head (e545fe5) 63.14%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #10      +/-   ##
==========================================
+ Coverage   62.50%   63.14%   +0.64%
==========================================
  Files          10        9       -1
  Lines         680      540     -140
==========================================
- Hits          425      341      -84
+ Misses        255      199      -56

Impacted Files
Coverage Δ

src/outputs.jl
36.00% <20.00%> (-1.50%)
:arrow_down:

src/run.jl
38.46% <33.96%> (-13.01%)
:arrow_down:

src/flexpartsim.jl
67.27% <67.27%> (ø)

src/abstracts.jl
64.28% <75.00%> (-4.95%)
:arrow_down:

src/inputs.jl
84.61% <84.61%> (ø)

src/FlexpartOptions.jl
81.30% <100.00%> (ø)

src/miscellaneous.jl
86.79% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


SPARK-48380: SerDeUtil.javaToPython to support batchSize parameter

What changes were proposed in this pull request?
This simple PR adds a parameter batchSize to the underlying javaToPython function.  This parameter is already available on pairRDDToPython.
Why are the changes needed?
With this, pyspark program can decide to use a different batch size when the default AutoBatchPickler ends up with 2GB exceeded error as in the JIRA ticket: https://issues.apache.org/jira/browse/SPARK-48380
Does this PR introduce any user-facing change?
This introduces a new way to access python RDD from python DataFrame:
# python
# The existing way to convert dataframe to RDD with auto batching (NO CHANGES)
df.rdd
# The new way to convert dataframe to RDD with batchSize of 1 (New API)
df._jdf.javaToPython(1)

How was this patch tested?
mvn test
(still testing)
Was this patch authored or co-authored using generative AI tooling?
No.

Let's:

Create a PR against master branch
Add a test case
Fix the PR title (see also https://spark.apache.org/contributing.html)

Will move to https://github.com/apache/spark/pull/46697


Org runner/auth integrations

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

Additional context

we have had org runners for some time now


Weird transition between tabs

Currently, when you scroll to the bottom of a tab, and then switch to a tab that has less content on it, it will display this weird scrolling animation. We should fix this.

I don't know how but apparently this is fixed now?? I mean I'm not complaining I guess lol


there is no transfer.pth file error

as mentioned on readme i tried to ran the steps but getting this error
please help me out as i am trying to understand this project.

Please download transfer.pth from the website below and place it to lpmc/music_captioning/exp/transfer/lp_music_caps/ folder.
https://huggingface.co/seungheondoh/lp-music-caps/blob/main/transfer.pth


os_profile metadata is missing

When using the Azure Python SDK and parsing through servers to find the computer_name inside of os_profile.  We found that some servers are missing the os_profile in the metadata.  See attached error, code and metadata.
ICM 185158839 with more information on the issue
Failed
Traceback (most recent call last):  File 'C:\Temp\i3escdt1.rfw\f4ce65b0-9c16-459e-b0b3-814ede83f69b', line 117, in    linux_vm_names=getVmsByOs('Linux',rg,key)  File 'C:\Temp\i3escdt1.rfw\f4ce65b0-9c16-459e-b0b3-814ede83f69b', line 76, in getVmsByOs    print(vm_dict['os_profile']['computer_name'])KeyError: 'os_profile'
def getVmsByOs(os,rg,key):
vm_names=[]
for vm in compute_client.virtual_machines.list(rg):
vm_dict=vm.as_dict()
if vm_dict['storage_profile']['os_disk']['os_type'] == os:
for i in vm_dict['tags']:
if vm_dict['tags'][i] == key:
vm_names.append(vm_dict['os_profile']['computer_name'])
return vm_names

I cannot created a vm without os_profile. And if I can get the os_profile metadata from the created VM by compute_client.virtual_machines.list.
Ask service team to check why there are compute vms who have no os_profile, or check if the service REST CALL return the os_profile correctly.

The OS profile is only required when a VM is deployed from a generalized image. If the VM is deployed from a specialized OS disk then an OS Profile is not required. This is because the primary use for the OSProfile is for configuration of the VM when it is deployed from a generalized image.
Note: The compute name in the OSProfile is only the initial computer name. It is possible that the computer name (also known as host name within the guest OS) has changed since it was originally deployed. It is possible to get the current computer name for any running VM using the Instance View: https://docs.microsoft.com/en-us/rest/api/compute/virtualmachines/instanceview. However, if the VM is not running, then the computename property of the instance view will not be provided.


Broken init arguments table in Missing Imputation Selector

Describe the bug
Broken init arguments table in Missing Imputation Selector
https://ing-bank.github.io/probatus/api/imputation_selector.html
One needs to check the docstring of init, and make sure running mkdocs serve provides correct API table

@anilkumarpanda Can you have a look at this one? it is probably something very small

Seems like,will have a look.


⚠ Bofrost.at has degraded performance

In 9b61830, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1007 ms

Resolved: Bofrost.at performance has improved in 2a06371.


Update allcourses.json

added https://portswigger.net/web-security for web security training

only acceptin 2 pr in same repo


Istio integration tests

introducing basic integration tests for Istio reconciler
Related issue:
https://github.com/kyma-incubator/reconciler/issues/798

LGTM


Bump monoskope to v0.4.13

Coverage remained the same at 37.011% when pulling 8f0ffc3990ecc0410065fc07a6015bbfbf247ba3 on chore/bump-monoskope into 1df52348ad827b4b3a84538833f99472c68b8b2f on main.


feat(#1): JS linear search

Please add your country. Also add the functionality to take input from user


[mlx-ui] Use proper namespace in iframe

Receive message from iframe when iframe is loaded
and send namespace info back to the iframe in order
to display the correct content.
fix: #77
Signed-off-by: Yihong Wang<EMAIL_ADDRESS>

in the change, it sends fixed namespace: mlx to iframe. we need a mechanism to query user's namespace and use user's namespace instead once we support multi profiles.

Thanks Yi-hong, since we only have anonymous can't-run-anything users, and, admin can-do-everything users, using the mlx namespace for all runs seems reasonable.
@drewbutlerbb4 @Tomcli ?

/lgtm
Right now we only have two role, admin and anonymous. We agreed to use mlx namespace as admin for the moment. This may need to change when we are able to introduce new user role in the UI and API.


What is the magic string at KeymaestroHybridEncrypter?

def encryptedPrivateKey = new KeymaestroHybridEncrypter("eb10fe8f7c7c9df715022017b00c6471f8ba8170b13049a11e6c09ffe3056a104a3bbe4ac5a955f4ba4fe93fc8cef27558a3eb9d2a529a2092761fb833b656cd48b9de6a".decodeHex())

What is the magic string "eb10fe8f7c7c9df715022017b00c6471f8ba8170b13049a11e6c09ffe3056a104a3bbe4ac5a955f4ba4fe93fc8cef27558a3eb9d2a529a2092761fb833b656cd48b9de6a"?

@Kolyall It's the public Google encryption key for app signing -- Google actually provides this information when trying to change your app signing key via the Play Console, but it's kind of hidden

java -jar pepk.0.0.1.jar --keystore sign-keystore.jks --alias sign --output=sign.zip --encryptionkey=eb10fe8f7c7c9df715022017b00c6471f8ba8170b13049a11e6c09ffe3056a104a3bbe4ac5a955f4ba4fe93fc8cef27558a3eb9d2a529a2092761fb833b656cd48b9de6a --include-cert


list of acceptable resolutions

Hi - me again.
After the previous issue where the default capture resolution is set to 360 x 180 which doesn't work for my TV - is there a way to determine acceptable capture resolutions?

Hey,
actually not really. It also seems to be different from application to application. With libdile_vt we may will have a independet solution for that. But actually is earlier than our other early stuff.
Maybe one solution will be to enable logging, start capture and get the supported resolution from /var/log/messages. As far as we know its stated there after a start

180 x 90 x 30fps/60fps/0fps
360 x 180 x ……
640 x 360 x ……
just try it…

maybe we can get the information for available resolutions from the backends and display them in a drop down-menu in future.

Over time we have collected some resolutions. They are now available in a drop down menu in settings page.


Project#classifiers_count was outdated for project 2025. Was 129, should've been 667.

I think this is an issue for Project 1506, as well. The project's classifiers_count was 124 before launch, and after launch the count remains 124.

Hi Marten,
The MWP (Project 2025) classifiers_count is still stuck at 667 and this has been the case for 2-3 days. I am assuming that we have more users participating than being shown in the front page!

Might be fixed by #1982

Counter is behaving normally these days. Closing this issue.


CryptoCompare API

Hey,
This is more a suggestion than anything, but have you considered implementing the CryptoCompare API, found here?
It provides a good amount of data on the cryptomarket; including daily, hourly and minutely price data. Beyond that there's good 'coin snapshots' which also provide good stats on various products. The PriceHistorical endpoint could also allow for a good currency conversion implementation.
Thanks for taking the time to read.

Hi! Thanks for your proposal! We will study their docs and implement their API, why not )

Hi!
An update on this: after a thorough investigation, we came to a conclusion, that CryptoCompare has pretty much the same list of supported exchanges as ccxt does. But we don't see a reason to link to exchanges through an intermediary, whereas with ccxt you can connect to same exchanges and trade directly (read: faster, lower round-trip times and latencies, etc...).
So, for that particular aggregator, we don't think we're going to implement it in ccxt. Instead, we will be adding new exchanges (the ones we miss), so that you would not need CryptoCompare in the first place. Sorry if that is not what you waited for! This is not our final decision, we may return to it later. If you know a reason for why users would want indirect connections through CryptoCompare, we will be happy to know. Thx for your feedback anyway!

@kroitor CryptoCompare has great aggregation index for comparison.
Please add CryptoCompare. We have already CoinMarketCap. ;-)

@JosefJezek we will reconsider this, thx!

I agree in general with what you've said @kroitor, but for some particular exchanges, like cryptopia, there is no support in the API to get charts.
Although ccxt does a nice workaround of building ohlcv data from trades, this usually renders very short-term data. This could possibly be a good use for cryptocompare.

I was having a look at cryptocompare ohlcv data, and I found out that for any ohlcv data below 1h period, not more than 48h of data is given. Which is similar to what can be obtained currently with the ccxt-generated ohlcv data.
For 1h candles there is a lot of depth given, something like 2-3 months. In any case, i couldn't find this specified anywhere, it is just an observation from a quick test of mine.

@payala Cryptocompare historic prices (OHLCV) maxes out at 7 days worth of data for resolutions below 60 minutes


Is it possible to reorder bars in a bar graph?

Hi guys !
Here is the thing : I created a horizontal bar graph whose data is to change according to various inputs. Is it possible to make it so that the bars "rearrange" themselves in ascending or descending order ?
For the moment I'm using a quicksort algorithm to reorder the data and associated labels, but as a result the labels just "change", I would prefer to have an animation showing the bars "reordering".
https://jsbin.com/yocisatawe/edit?js
Note : This is not a support request, I just want to know if this is possible (I could not find anything in the documentation), if it is then how, and if it is not if it could be implemented.
Thank you very much !
Tom.

@TomFevrier I don't think this is currently possible. The reason is that when updating each dataset controller, here items are not shuffled during updates, ie the 3rd bar object remains the 3rd bar during the update.
To be able to get the animation you desire, the buildOrUpdateElements method probably needs to be extended to do the same sorting. The key though is going to be to make sure that the shuffle happens exactly as required. Then, when update runs on the controller, the x coordinate of the bars will be updated and the animation should work.

@etimberg Thank you very much, so if I understand right I need to edit the buildOrUpdateElements method which is in the source code of Chart.js ? I have absolutely no idea on how to implement this...

@TomFevrier yeah, you'd need to edit the method on the internal controller for each dataset.
I think with a prebuilt version, this is Chart.DatasetController.prototype.buildOrUpdateElements. You could patch that to add some kind of hook to use to trigger your changes.
Another option is to use the beforeUpdate plugin call to do the shuffle for you. That would make it a lot easier.
Chart.plugins.register({
  beforeUpdate: function(chart) {
    var meta = chart.getDatasetMeta(0); // parameter is the index of the dataset
    // meta.data is the array of elements (bar objects) for the dataset
   // shuffle the meta.dataset array here to be the same order as your new data.
  }
});

Here's an example of what I was thinking (I am surprised it works, I am still in need of a ☕)
https://jsfiddle.net/fx0a6o6q/ I cannot guarantee nothing was broken by this or that this is the most efficient way to solve this problem, it's just the first solution I thought of.

@etimberg Thank you very much, that is exactly what I had in mind, though I don't understand everything you did. The only issue is that the labels do not rearrange themselves, do you know how to do this ? And could you explain how this works, I am afraid I am a newbie in JavaScript :stuck_out_tongue:
Have a good :coffee: and thanks for you time !

Actually I tried stuff drawing on what you did, and I implemented this in beforeUpdate :
(...)
let labels = chart.data.labels.slice();
(...)
labels.sort((a,b) => b - a); // Note : I changed it to descending order
(...)
chart.data.labels = labels;
It does work as the country names change, but they do not animate... Is this possible ?

@TomFevrier unfortunately the axis labels are not able to animate without a lot of changes to the library internals.
Here's an updated version that moves the colours and adds some comments https://jsfiddle.net/fx0a6o6q/3/

@etimberg Thank you very much, I understand now. It is great even without animated labels, and this will be very useful for my project.

Closing as resolved

@etimberg  i used your code to create sorted bar graph. My problem is i have 33 data points and i just want to show 10 of them in the bar graph. Does anybody know how can i do this?
Thanks
Serap

This works perfect on initial data load, but how can this be achieved when data changes?

What are the alternatives to sort() method in chartjs with text and integers together?


[FEATURE] Blueprint for serverless Fargate cluster

Is your feature request related to a problem? Please describe

A blueprint that demonstrates how to create a serverless based EKS cluster using Fargate profiles. This solution will rely on showing users how they need to manage patching CoreDNS to avoid having to run an EC2 based node for essential addons

Describe the solution you'd like

Blueprint of EKS cluster with Fargate profiles for data plane

Describe alternatives you've considered

None

Additional context
Add any other context or screenshots about the feature request here.

We use this
resource "null_resource" "patch_coredns" {
  count = var.fargate_create ? 1 : 0

  triggers = {
    cmd_patch = <<-EOT
      ## TODO : replace with token
      aws eks --region "${var.aws_region}" update-kubeconfig --name "${data.aws_eks_cluster.this.id}"
      kubectl patch deployment coredns \
        -n kube-system \
        --type json \
        -p='[{"op": "remove", "path": "/spec/template/metadata/annotations/eks.amazonaws.com~1compute-type"}]'
      kubectl rollout restart -n kube-system deployment coredns
    EOT
  }

  provisioner "local-exec" {
    interpreter = ["/bin/bash", "-c"]
    command     = self.triggers.cmd_patch
  }

  depends_on = [
    module.fargate_profiles,
  ]
}

after that, we instal coredns addon


FEAT: Support launch model with specific model uid

Is your feature request related to a problem? Please describe
Support launch model with specific model uid when using command line to launch model.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Fixed by #358


Document transitioning from 1.x to 2.0 client libraries

InfluxDB 1.8 ships with a 2.0 compatible /write endpoint that lets users use the new 2.0 client libraries to write to InfluxDB 1.8 and Cloud 1. We need to write a guide or guides for transitioning from the 1.x to the 2.0 client libraries.
Notes & questions

These may need to be language/client specific
What are the gotchas of the new clients?
Are there new concepts that 1.x users need to understand to successfully use the new clients.

close per https://github.com/influxdata/docs.influxdata.com/pull/2757


I am facing a problem  createDriver() .

ErrorException in Manager.php line 77:
Missing argument 1 for Illuminate\Support\Manager::createDriver(), called in C:\Users\MD.MOSTAFA KAMAL\ela-search\vendor\laravel\framework\src\Illuminate\Support\Manager.php on line 87 and defined

What version of the driver/scout/laravel are you using?

laravel 5.4

+1


Need I ROOT?

Need I root to use this?

No, but you need to ensure ADB works fine between your PC and your Android device.

do you have other way to replay without ROOT

Could work if the developer released an android app to replay the recorded data using accessibility service. No need for root for that on Android 10+ devices. There are many auto clicker apps that use accessibility to simulate clicks but most of them offer recording with an overlay on top which is mostly fine for static clicks but there is a huge lag with games where you are constantly swiping.

Could work if the developer released an android app to replay the recorded data using accessibility service. No need for root for that on Android 10+ devices. There are many auto clicker apps that use accessibility to simulate clicks but most of them offer recording with an overlay on top which is mostly fine for static clicks but there is a huge lag with games where you are constantly swiping.

Do you know how the APP get the click from accessibility service? is it the same as ADB inputs, this one is detected easy.
the reason I am using this repo method is that it really do a click event from HW, I am not sure for other like accessibility, did you test ?


Changes to KivyCatalog LabelContainer demo

Add some extra features showing effects of padding, bold lines, and
other options.

Thanks @merriam. :cake:


Dev Tools Menu should open in a dedicated window

As per issue #43...  Currently, if the main window size is too small, the '...' menu for dev tools wraps so that the menu items become obscured.  This happens because the toolbar is being rendered in a window that is truncated by the browser view for rendering the tab content.  The current menu for devtools is rendered in the DOM of the toolbar window - so if there isn't enough space - it just gets cutoff.
The solution is to use the same method for rendering this menu as is used for the color picker menu, search results, etc, which is to render as a dedicated window that can overlay the content of the main window and is not bound by it's DOM.

Actually, taking another look, this should be resolvable with the Electron menu apis https://www.electronjs.org/docs/latest/api/menu

addressed with PR #45


Format JSON

Make JSON pretty *
* As pretty as JSON in Bash can possibly look

With Prettier?

A great thought, except that everything is quoted and escaped for Bash, which I don't believe Prettier supports yet.
I would like to eventually add some formatters like that. Obviously Rubocop for the (very few, but should be more) Ruby portions, probably ShellCheck for the Bash code, and maybe Prettier for the one Node.js script (used by Lambda).


Create Folder Structure for FileCabinet if doesn't exist

If importing a file from FileCabinet and the directory doesn't exist, create it.

Resolved in #dd92cfd7.


Wrong link in README.md

Description
'Visit Falko' in README is pointing to a deprecated domain. Should point to http://falko.surge.sh/#/
Expected behavior
Should point to http://falko.surge.sh/#/
Current behavior
Points to falko.solutions
Steps to reproduce the behavior
Click the 'Visit Falko' link in README
Checklist

[x] This issue has a significant name.
[x] This issue has labels.
[ ] This issue has screenshots (if necessary).

Issue resolved. #374


Methods in services appear under the module.

Methods using "@ngdoc method" and name them as they are in the angular documentation in services such as $http (e.g. "@name $http#get") are getting shown in both the service (e.g. $http) as well as the module (e.g. "ng").  This happens for multiple services, where the module has all of the methods of  all of its services.

This is as-designed.

May I ask why?  This is distinctly different from angular's documentation, as in: https://docs.angularjs.org/api/ng/service.  In docular, this would have every single method of every single subcomponent in the "services" tab, which gets pretty crowded, instead of only in $http.


feat: tag resources by friendly attribute

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-oclif3.1 :tada:
The release is available on:

GitHub release
npm package (@beta dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-rc.1 :tada:
The release is available on:

npm package (@prerelease dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


iOS 9 - bug

[cell.imageView setImageWithURL:[NSURL URLWithString:currentItem.img_url] placeholderImage:placeholder];
it's not showing the image on iOS 9.
Same line i have no problem with lower version of IOS

if i use:
[cell.imageView setImageWithURLRequest:request placeholderImage:placeholder success:^(NSURLRequest *request, NSHTTPURLResponse *response, UIImage *image) {
[cell.imageView setImage:image];
} failure:^(NSURLRequest *request, NSHTTPURLResponse *response, NSError *error) {
NSLog(@"Request failed with error: %@", error);
}];
i got this error:
Error Domain=NSURLErrorDomain Code=-1200 "An SSL error has occurred and a secure connection to the server cannot be made."

That is an ATS error probably, google iOS 9 ATS

Hi, I have the same issue. I tested this part of AFNetworking
`BlockerWrapper *wrapper = [[BlockerWrapper alloc] init];
[wrapper setSuccessBlock:^(UIImage *img) {
    [weakSelf setImage:img];
    [weakSelf setNeedsDisplay];

    if(fadeIn) {

        [UIView beginAnimations:nil context:nil];
        [UIView setAnimationDuration:0.6];

        weakSelf.alpha = 1.0;

        [UIView commitAnimations];
    }
}];`

So it never return success block

for people who are facing this problem:
http://stackoverflow.com/questions/30739473/nsurlsession-nsurlconnection-http-load-failed-on-ios-9


OOP school library: set up associations

In this branch, I have added the following;

[ ] Created a class Classroom with a @Label instance variable,
[ ] Created a has-many/belongs-to relationship between Classroom and Student.,
[ ] Created the has-many side (a classroom has many students),
[ ] Create the belongs-to side (a student belongs to a classroom),
[ ] Set the classroom for the student, when adding a student to a classroom and vice versa,
[ ] Create a class Book with @title and https://github.com/author instance variables,
[ ] Created a class Rental with the following @Date instance variable,
[ ] Created the many-to-many (also has-many-through) relationship between Person and Book using the intermediate class Rental,

Created the has-many side of Book and Rental (a book has many rentals),
Created the belongs-to side of Rental and Book (a rental belongs to a book),
Create the has-many side of Person and Rental (a person has many rentals),
Created the belongs-to side of Rental and Person (a rental belongs to a person),
Modified the constructor of Rental to include Book and Person variables.

Hi @zainsadaqat, Thank you! :hugs: