[WIP] Added direct fill

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9218/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1163/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/954/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9243/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/977/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1190/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9244/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/978/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1191/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9259/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/993/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9261/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/995/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1208/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1030/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9295/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1242/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1277/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9329/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1066/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1280/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9332/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1070/

Build Success with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9336/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/1284/

LGTM


Application suddenly quits when uploading source

Mapbox Studio quits any time I hit the “Upload to Mapbox” button on a local Postgres-backed source (not every source, however). The application doesn’t crash; this is the only thing printed to the console:
2015-07-06 12:16:50,336 com.apple.xpc.launchd[1]: (com.mapbox.mapbox-studio.41212[30955]) Service exited with abnormal code: 8

Mapbox Studio 0.2.7 on OS X 10.10.4.

This is a known bug in 0.2.7, was fixed in https://github.com/mapbox/mapbox-studio/pull/1199, and will be working in 0.2.8

specifically the bug was that for complicated reasons we cannot know the extent of postgis local source layers, and now this crash is avoided by defaulting to the global extent at https://github.com/mapbox/mapbox-studio/blob/mb-pages/lib/source.js#L333


Job interval

Hi there,
I use this package via meter-mailer. Only a small question: it's necessary to do this job every 250ms?
https://github.com/VeliovGroup/Mail-Time/blob/eb326f6710570637c771c0b568cf7bed86acd13b/index.js#L170
or it's safe to update this db job to a greater value?
{
   "_id" : ObjectId("5a2adfa557d79d32b2c55af3"),
   "uid" : "mailTimeQueuesetInterval",
   "delay" : 256,
   "executeAt" : ISODate("2018-01-31T14:59:46.717Z"),
   "isInterval" : true,
   "inProgress" : false
}

Any other way to do it?
TIA,

Thanks for the tip (and for sharing you work, I use some other packages of your group here and there)...

@dr-dimitru, but you will need to do it on every app launch inside Meteor.startup(() => {/.../}) callback, should be mentioned, right?


When the batch size is set to 2, how much gpu memory is required at least.

My 3090 graphics card has 24G of memory but it’s still not enough. So how much gpu memory is required at least, when the batch size is set to 2.

Hello! We suggest using A100, the application need at least 60Gb of VRAM due to the size of original SAM. But you can try use ViT-L with a 24Gb memory card.


Path change

Changed path of the ZIP download.

Tnx & sorry for the wait.

No worries! Thank you for making the puppet module in the first place! :)

@gorandev i've also finally uploaded it to forge as well, so if you prefer you can fetch it from there as well.

What's this "forge" you're talking about? Sorry for asking but I have no clue!


When to release the full code？

Excellent work! When to release the full code, can't wait to try.

Hi, thank you for your interest. We will release the code probably this week.

Excellent work! When to release the full code, can't wait to try.

Hi, thank you for your interest. We will release the code probably this week.

Thanks for your reply. Waiting for your code.

Hi, wanted to remind you that there is a pytorch implementation here: https://github.com/zipengxuc/PPE-Pytorch


Building Influx, how to get service mode?

This is more of a question than a real issue. The front page says

Starting InfluxDB• service influxdb start  if you have installed InfluxDB using an official Debian or RPM package.
• $GOPATH/bin/influxd  if you have built InfluxDB from source.

I am building my own on a Raspberry Pi, but would like to run it as a service (so it can run even after I close the SSH connection).
How do I get the service option on my Raspberry with a custom build from scratch?

@chaithrar This is neither a feature request nor a bug. Please ask questions like this on the mailing list rather than opening a GitHub Issue.

You need to build a package using package.sh, then run dpkg -i , sudo service influxdb start


Added Winter events

Changes
Updated event descriptions.
Fixes #424
Type of change
Please delete options that are not relevant.

[X] Content Update (non-breaking change which updates the content of the website)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Note:
Still missing assets for HOTH, location for HOTH TBA.
Abbreviated / shortened some of the Stackschool workshop names to prevent text overlapping the Details button of the card component-- I'm assuming it's intentional that the card component dimensions are hard-coded so height doesn't increase.

This is what I get for using copilot


Set member firstname/lastname

I'm trying to set the firstname/lastname of members I am adding to a mailchimp list. Is that possible using this API? I can't see how it can be done.
MailChimpList mailChimpSubscriptionList = con.getList(mailChimpList.getId());
List mailChimpMembers = mailChimpSubscriptionList.getMembers(0,0);
mailChimpSubscriptionList.addMember(MemberStatus.SUBSCRIBED, emailAddres);

Hi @alan-parry 👋 ,
thanks for you question.
In fact there are several overloaded methods to create a member for a list.

addMember(MemberStatus status, String emailAddress): adds a member just with an email address
addMember(MemberStatus status, String emailAddress, HashMap<String, Object> merge_fields_values): adds a member with additional information in form of a hashMap
createUpdateMember(Member member) - takes a Member object and either updates the member if already existent, or creates a new one

In your case only the second option is valid, because the the first does not provide the necessary parameters and the third one only works if you already have an Member object which can only be retrieved from the API. I see that there needs to be some better naming and documentation for the methods. I will file an issue for that. But for now we stick to the 2. option and you can do the following.
With the 2. option you can declare values for the  so called MergeFields which you can find in your MailChimp List settings under "Settings" -> "List fields and |MERGE| tags". Here you need to choose the tags in the Put this tag in your content: column as keys into a HashMap in Java and the value is the value to which you want to set the merge field. For example
MailChimpList mailChimpSubscriptionList = con.getList(mailChimpList.getId());
List mailChimpMembers = mailChimpSubscriptionList.getMembers(0,0);

HashMap<String, Object> mergeFields = new HashMap<>();
mergeFields.put("FNAME", "John"); // merge field for the first name of a member
mergeFields.put("LNAME", "Smith"); // merge field for the first name of a member

list.addMember(MemberStatus.SUBSCRIBED , emailAddress, mergeFields);

With that you can add members with a first and last name. Let me know if this helped you :)

Thanks, this really helped! I'd seen the merge fields but didnt understand what they did. Thanks to this I'be got it working so yes having this documented would be a big help for others I'm sure. Cheers :)

Your are welcome, I am glad that my description helped you.
I try to document this as soon as possible.


Rewriting URL's for Laravel

Given the following Caddyfile and a fresh Laravel setup with the following routes:
root ./public
fastcgi / /var/run/php5-fpm.sock php
errors error.log

<?php

Route::get('/', function () {
    return view('welcome');
});

Route::get('/foo', function () {
	return 'bar';
});

The index route works perfectly, but the /foo route can only be access at /index.php/foo.

The recommended NGINX configuration for Laravel is:
location / {
    try_files $uri $uri/ /index.php?$query_string;
}

Don't know if that helps?

This should work
rewrite {
    r .*
    ext /
    to /index.php?{query}
}

Perfect :ok_hand:
You're my hero @abiosoft :)


http.request type is not found.

Good day!
Currently, I am trying to run tutorial with the newest version of the event-gateway using minikube. This tutorial doesn't work for me when I try to create a subscription:
curl -k --request POST \
  --url https://minikube_ip:4001/config/v1/spaces/default/subscriptions \
  --header 'content-type: application/json' \
  --data '{ "type": "sync",
            "eventType": "http.request",
            "functionId": "hello-serverless-kubeless",
            "method": "POST",
            "path": "/hello-serverless-kubeless"}'

I get an error:
{"errors":[{"message":"Event Type \"http.request\" not found."}]}
here is yaml file to deploy event-gateway:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: serverless-event-gateway
  labels:
    app: serverless-event-gateway
spec:
  selector:
    matchLabels:
      app: serverless-event-gateway
  replicas: 1
  template:
    metadata:
      labels:
        app: serverless-event-gateway
    spec:
      containers:
        - name: serverless-event-gateway
          image: serverless/event-gateway:0.9.1
          args:
            - "-db-hosts=etcd-event-gateway:2379"
            - "-log-format=text"
            - "-log-level=debug"
          livenessProbe:
            httpGet:
              path: "/v1/status"
              port: 4001
          readinessProbe:
            httpGet:
              path: "/v1/status"
              port: 4001

apiVersion: v1
kind: Service
metadata:
  name: serverless-event-gateway
spec:
  type: ClusterIP
  ports:
    - name: events
      protocol: TCP
      port: 4000
    - name: config
      protocol: TCP
      port: 4001
  selector:
    app: serverless-event-gateway

I would be super thankful if you have an idea why the build in type is not recognizable in my case.
best,
Oleksii

Thank you @sebito91  for a quick answer.
Yes, you are right. The event type wasn't registered before.
I thought that "http.request" is a built-in type and it shouldn't be registered.
Now it seems working. Thank you for a help.


Develop

Merge branch 'master' into develop
NSError as a result of validation/schema adding

@samskiter this patch is going to break backward compatibility. For the existing library users it may be like #define true false .
Still, I agree that NSError should be returned by the affected method.
// AS IS
-(BOOL)validateJSONData:(NSData*)jsonData withSchemaData:(NSData*)schemaData;

// from the patch
-(NSError*)validateJSONData:(NSData*)jsonData withSchemaData:(NSData*)schemaData;

// My proposal
// follows Cocoa conventions
-(BOOL)validateJSONData:(NSData*)jsonData
                withSchemaData:(NSData*)schemaData
                                    error:(NSError**)outError;

@dodikk that's a great catch! thank you. My head is in Java mode at the minute and I forgot about the auto cast. also, this is actually the way i prefer to return errors if any - more consistent with apple's APIs and other libraries
@nut-code-monkey sorry for another delay on this PR.

@samskiter , I think, these days I'll finally get a chance (in terms of time) to add your validator to my project. Eventually, I'll adopt these changes and remaster the API in the Cocoa way so that you'll get a patch from me.
P.S. @samskiter , I guess, this one is no longer relevant and can be closed. https://github.com/samskiter/KiteJSONValidator/pull/9

@samskiter spent some of my leisure time trying to make the API follow the cocoa style https://github.com/dodikk/KiteJSONValidator/commit/7db8daee50fc3488d80c1e157ebfe57c5a235cb1
I can't afford spending so much time copy-pasting the if (NULL != outError) thing and structuring  "return" statements. So, I'm giving up for now. Sorry.

It would be a lot easier if I could use NSAssert(NULL != outError) rather than if statements. It would makes the changes so much more simple.
Such policy makes sense since it will be a lot harder to ignore validation errors. @samskiter , what do you think?

Hi @dodikk thanks very much for your efforts. this is on my TODO list to sort out properly. It does look a little smelly that we have to have a lot of checks agains NULL (this should probably be nil by the way). so I will have to take a closer look.

It does look a little smelly that we have to have a lot of checks agains NULL (this should probably be nil by the way).

That's how out parameters are done in "plain old C". And yes, apple allows NULL errors in their own API.

this should probably be nil by the way
No, it should not. NSError* (one star) and NSError** (two stars) are two completely different types. http://nshipster.com/nil/


Test Coverage doesn't work when code runs in a container

Current tests make calls to REST API in a running container. So, test coverage doesn't work well and gives wrong statistics. To get better results, tests must run inside the container, following the pattern to call the app directly, as used in ordinary FastAPI tests.
As codebox doesn't have external dependencies, we might use something like:
docker run --rm -it --privileged --init -e ENV=TESTING --ipc=none \
      -v $(pwd)/tests:/codebox/tests \
      --name codebox codebox pytest -svx

The pieces of code used in tests must run inside a container due to security reasons. Current tests call a running container to do that but it prevents test coverage to gather correct results.
Another option is to run the tests from inside the container. conftest.py would use an ordinary FastAPI test strategy to create a client fixture:
@fixture(scope='session')
async def app() -> AsyncIterable[FastAPI]:
    """
    Create a FastAPI instance.
    """
    async with LifespanManager(_app):
        yield _app

@fixture
async def client(app: FastAPI) -> AsyncIterable[AsyncClient]:
    async with AsyncClient(
        app=app,
        base_url='http://testserver',
        headers={'Content-Type': 'application/json'},
    ) as client:
        yield client


Add new csproj format and GitHub workflow

Copies the binary to a bin folder in the root such that everest.yaml does not refer to the full $(OutputPath).
GitHub workflow creates a zip artifact that can be dropped into Everest.
Note: This PR obsoletes #4

seems to work


Support default/json output formats

This is an initial start in order to eventually support https://github.com/invertase/github-action-dart-analyzer/issues/199
It works pretty much, most of the output code comes from the Dart SDK. Wonder if there is a way to use it directly?
Fixes https://github.com/invertase/dart_custom_lint/issues/22
Fixes #101

Wonder if there is. a way to use it directly?

I don't know honestly.

Problem is currently the print overrides which don't work well together with cli_util package and the output expectations in the tests.

What's the issue exactly?

What's the issue exactly?

The cli_util package which the Dart analyzer also uses, has a Logger which uses print internally: https://github.com/dart-lang/cli_util/blame/master/lib/cli_logging.dart#L140
But there is a Zone override to write the log file, maybe the log file should only be generated for the server and not for the client?

No it is crutial that logs in the client are reported. That's probably the most important bit.
But I do not understand why you need to disable log reporting in the client for this

Maybe you're looking at modifying the CommandCustomLintDelegate?

Well cli_util uses print and in the current setup print goes into the log file and not to stdout as far as I can tell :)
Can we use some dedicated logging for the log file instead of the zone override?

I think this is good now, I added a test for the json format and extended the sorting test to take the severity into account.

I'm looking forward to this being merged. :)

Are the 2 output examples in the screenshots something that we generally agree on?

If you don't mind, I'll wait until I can fix the funky platform crash before merging this.

I am gonna split this up in smaller parts.


fixed parsing UTF-16 encoded inputs which do not end with a surrogate…

This PR fixed a bug in parsing UTF-16 encoded inputs reported in #233.
Also, the test suite has been modified to ensure the change works as expected.

Pull Request Checklist
Read the CONTRIBUTING.md file for detailed information.

[x] Changes are described in the pull request or in a referenced issue.
[x] The test suite compiles and runs without any error.
[x] The code coverage on your branch is 100%.
[x] The documentation is updated if you added/changed a feature.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Please refrain from proposing changes that would break YAML specifications. If you propose a conformant extension of YAML to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

coverage: 100.0%. remained the same
when pulling cd8f3e51460cf708442b8fabd1d899b480675bbd on bugfix/233_fix_parsing_utf16_inputs
into e02af6b1d90d2bef4c042c8b9382b500beb762c9 on develop.


How to config Conversation Transcription to recognize Chinese?

Hi,
I've set CTSRegion="eastasia" in conversation class but recognized text is still in english though i speaks chinese. It seems conversation does not derive languageRecognition i set as zh-cn in MainActivity.
Tried to set participant's en-us to zh-cn but it doesn't work either.
Anywhere else config i missed?

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

ID: 26b5a45a-6fa7-f898-fcf5-781210fbee9e
Version Independent ID: 90ff521b-37f5-38a1-ace0-6ad5989b6dab
Content: Quickstart: Run the Speech Devices SDK on Android - Speech Services - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/speech-devices-sdk-android-quickstart.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @erhopf
Microsoft Alias: erhopf

@wukun2015 Thanks for the feedback. We are investigating the issue and will update you shortly.

Any update?

Hi wukun2015!
Sorry for interfering with your question but I have two questions for you:
Did you get the Roobo microphone array up and running using this quickstart alone? Was there a version of Google Play Services installed on your device when you got it? Cause it wasn't installed on mine. Thanks for helping me out.
Cheers!

After speaking with engineering, the recommendation is to call SetSpeechRecognitionLanguage(“zh-cn”) on their SpeechConfig object.

thanks erhopf. @timtre, i‘m not sure. Don't see Play icon on desktop but there are lot of google advertisement jump.

#please-close
@wukun2015 let me know if you run into any other issues. I'm going to close this issue, but feel free to reopen it at any point you need help ;).


Create and export GraphQLSchemaToken

The schema token can be used for using LocalLink on the server.

Done in https://github.com/fusionjs/fusion-apollo/pull/76


Migrate to Zig build system

using zig build with either zig c++ or a wrapper through a main.zig file


Store frame time, not frame rate

When used, we're taking the inverse every frame (and converting it to an f32).
Instead, store an f32 frame time in the FrameRateLimit resource (see #10) and use that directly to save on wasted work and reduce complexity.
Provider helper methods to allow users to specify the settings with either frame rates or frame times.

Yeah, I waffled on this a bit, I think you're right that this is a better approach.


Metric to monitor the Happy Eyeballs behavior ?

Wondering if there is a way to monitor the Happy Eyeballs behavior... would this be useful information?
For example :
// <IP_ADDRESS>:443 BLOCKED on FW //
// initial request //
dns_cache_impl.cc:109] cache miss for host 'dir.bg:443', posting to main thread
dns_cache_impl.cc:307] main thread resolve complete for host 'dir.bg:443': [<IP_ADDRESS>:0, <IP_ADDRESS>:0, <IP_ADDRESS>:0]
dns_cache_impl.cc:374] host 'dir.bg:443' address has changed from <empty> to <IP_ADDRESS>:443

multi_connection_base_impl.cc:14] [C7] connections=3
happy_eyeballs_connection_impl.cc:25] C[7] address=0x4525bf03ef18
connection_impl.cc:924] [C8] connecting to <IP_ADDRESS>:443
connection_impl.cc:943] [C8] connection in progress
// 300ms later //
connection_impl.cc:924] [C9] connecting to <IP_ADDRESS>:443
connection_impl.cc:943] [C9] connection in progress
connection_impl.cc:683] [C9] connected

// 2nd request //
dns_cache_impl.cc:102] cache hit for host 'dir.bg:443'
multi_connection_base_impl.cc:14] [C30] connections=3
connection_impl.cc:924] [C31] connecting to <IP_ADDRESS>:443
connection_impl.cc:943] [C31] connection in progress
// 300ms later //
connection_impl.cc:924] [C32] connecting to <IP_ADDRESS>:443
connection_impl.cc:943] [C32] connection in progress
connection_impl.cc:683] [C32] connected

This results in envoy_upstream_cx_total counter increasing by 2 when in fact 4 connections are attempted.
Also /clusters shows only the cached host <IP_ADDRESS> while in fact all connections are established with <IP_ADDRESS>.
Obviously I can log and parse the admin log but this is noisy and requires a lot or resources.
Prometheus / Envoy AccessLog is preferable I guess.
BR,
Stoyan

@RyanTheOptimist @alyssawilk

Hm, I agree it'd be useful to have a stat for happy eyeballs connection attempts but I'm not sure if it should be cx_total - the multiplexed connection code will kick off a number of connections but as soon as one is confirmed connected the others are torn down so only one is ever established. That said for non-happy eyeballs connections we do increment that counter for connections which never connect so I could go either way re separate stat or reused stat.  cc @mattklein123 @jmarantz for preference.
If we opt to use the same stat we'd just have to wire cx_total to the MultiConnectionBaseImpl, and increment where we schedule another attempt.


Database Interface

A database interface must provide method signatures which all database libraries must provide in order to be used in a SlaxWeb Model.
Interface should define the following signatures:

runQuery
getLastQuery
getLastError
getRowCount
TBD

Part of slaxweb/framework#91

Resolved. With slightly different signatures.


Delete button for treatment optimization constraint term doesn't work

After adding a constraint term, the delete button does not get rid of the cost term.

@RobSalati I updated the jar file with a fix, please download and install again and update the issue accordingly, thanks for reporting

Fixed, thank you!


Multiple Placeholders example?

First of all, Thank you for making such an awesome WYSIWYG editor.
Can anyone give me an example where we can use multiple placeholders, like for example placeholder text for a list item saying 'Enter what to do', something like an input?
Thanks in advance.
Something like the below example:

Hi @upendra-web,
Did you find the solution? Thanks.

Nope, dude. I couldn't find a solution.

Any solution for this?


Some examples of implementing Avalonia controls for displaying video (using OpenGL and Software rendering).

LibMpv.Client - a complete libmpv API wrapper and some examples of implementing Avalonia controls for displaying video (using OpenGL and Software rendering).
Works on Linux and Windows (maybe MacOS)

Awesome thanks!

For Desktop, there is OpenGlVideoView and NativeVideoView, what's the difference between both? NativeVideoView is not used.
And on Android, there is no OpenGlVideoView. Can't that renderer  be used on Andrroid?

Refactoring is done. Now got some pretty clean code.
https://github.com/mysteryx93/LibMpv-OpenGL
For some reason, your Android version runs under "AndroidSample" folder; but I can't make my version to run. Can't figure out what I'm missing.
Also, although your Android version runs, it crashes if we try to switch windows. There's still more work to do on the Android implementation; and then iOS.

For Desktop, there is OpenGlVideoView and NativeVideoView, what's the difference between both? NativeVideoView is not used.

You will have to read the libmpv documentation to understand the difference between these player embedding technologies
https://github.com/mpv-player/mpv-examples/blob/master/libmpv/README.md (Methods of embedding the video window)

Also, although your Android version runs, it crashes if we try to switch windows. There's still more work to do on the Android
implementation; and then iOS.

Yes you are right. There is a problem when switching windows. It's probably trivial, but I don't have time to fix this bug at the moment. My interest was just to try to run on android. Perhaps in the future I will return to this problem.

Notes about iOS. There is no easy way to get compiled libmpv library for iOS. Unfortunately, I don't have a macOS computer to try it out. You will most likely have to compile it yourself for that platform.

Notes about embedding the player. It seems to be better to use native embedding under Windows to achieve better video playback results.
In my example, the implementation of naive embedding is not complete.
How to implement better, you can look at the example of the implementation of LibVLC
https://github.com/radiolondra/libvlcsharp/tree/3.x/src/LibVLCSharp.Avalonia.Unofficial

I started implementing the full API of commands and properties but that's a lot of work. Properties work for basic data types, but strangely, GetAsync doesn't work on string values but works for other data types? Complex data types aren't yet implemented. Still a lot of work to do.
I'll leave it in the current state and add it to MediaPlayerUI.
Here's a TODO list.
https://github.com/mysteryx93/LibMpv-OpenGL#contributions-wanted
It may sit in the current state for a while unless someone picks up the work.

I started implementing the full API of commands and properties but that's a lot of work. Properties work for basic data types, but strangely, GetAsync doesn't work on string values but works for other data types? Complex data types aren't yet implemented. Still a lot of work to do.

Precisely because there are so many properties, I did not do this in my implementation. In addition to simple properties, there are many more properties for setting audio and video filters. Of course, the absence of mapping will not meet the classical requirements of MVVM, but this is the reality. It is not always appropriate.

I had already done the full API mapping for controlling the player via IPC controller, so I copy-pasted that; but there are quite a lot of differences and I did not finish fixing the changes.
MVVM does not apply to the library. You can create your own derived controls that implement the properties you need as MVVM, or it passes the MpvContext to the ViewModel, from there the ViewModel has full control. But the library itself doesn't care what UI or design pattern you use.

With the full properties mapping, added auto-listening to property changes which is pretty cool. Binding to the property Changed event and it just works by magic.
It's now working with MediaPlayerUI so I'll just publish the packages as-is. They can be fixed and improved later, but I think a decent video player for Avalonia is just overdue.

It's released!
https://www.nuget.org/packages/MediaPlayer.Avalonia.Mpv/2.0.0-rc1


Implement different Pool strategies

[x] Round robin (implemented)
[ ] Random
[x] Factor-based selection (minimal value)

@dmitryfokin not related to this issue but idle1 and idle2 are commutative values from thread start obtained at a different times

https://github.com/dmitryfokin/node/discussions/1


Control + Backspace

Using Control + Backspace deletes the whole word as expected, but it wont delete the space, so u cant use it to quickly delete a couple of words or the whole thing. It not that i can use PgUp and PgDn, its just that it bugs me cuz i use Ctrl.

Thank you for reporting that, but I'm sorry to say that we wont fix that because its a lot of work for such a little feature. I encourage you to try to propose a fix and PR it!


Openapi integration is failing due to dependency error

While trying to configure openapi connector with trino 402, but it is failing with below error
2023-10-31T03:27:54.089Z        INFO    main    io.trino.connector.StaticCatalogManager -- Loading catalog openapi -- 2023-10-31T03:27:54.333Z        ERROR   main    io.trino.server.Server  io/opentelemetry/api/OpenTelemetry java.lang.NoClassDefFoundError: io/opentelemetry/api/OpenTelemetry         at io.airlift.http.client.HttpClientModule$HttpClientProvider.<init>(HttpClientModule.java:101)         at io.airlift.http.client.HttpClientModule.configure(HttpClientModule.java:77)         at com.google.inject.spi.Elements$RecordingBinder.install(Elements.java:426)         at io.airlift.http.client.HttpClientBinder.bindHttpClient(HttpClientBinder.java:48)         at pl.net.was.OpenApiModule.setup(OpenApiModule.java:67)         at io.airlift.configuration.AbstractConfigurationAwareModule.configure(AbstractConfigurationAwareModule.java:47)         at com.google.inject.spi.Elements$RecordingBinder.install(Elements.java:426)         at com.google.inject.spi.Elements.getElements(Elements.java:113)         at com.google.inject.spi.Elements.getElements(Elements.java:106)         at io.airlift.configuration.ConfigurationFactory.registerConfigurationClasses(ConfigurationFactory.java:179)         at io.airlift.bootstrap.Bootstrap.initialize(Bootstrap.java:216)         at pl.net.was.OpenApiConnectorFactory.create(OpenApiConnectorFactory.java:52)         at io.trino.connector.DefaultCatalogFactory.createConnector(DefaultCatalogFactory.java:201)         at io.trino.connector.DefaultCatalogFactory.createCatalog(DefaultCatalogFactory.java:123)         at io.trino.connector.LazyCatalogFactory.createCatalog(LazyCatalogFactory.java:44)         at io.trino.connector.StaticCatalogManager.loadInitialCatalogs(StaticCatalogManager.java:139)         at io.trino.server.Server.doStart(Server.java:139)         at io.trino.server.Server.lambda$start$0(Server.java:88)         at io.trino.$gen.Trino_402_e_14____20231031_032736_1.run(Unknown Source)         at io.trino.server.Server.start(Server.java:88)         at com.starburstdata.presto.StarburstTrinoServer.main(StarburstTrinoServer.java:46) Caused by: java.lang.ClassNotFoundException: io.opentelemetry.api.OpenTelemetry         at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)         at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:592)         at io.trino.server.PluginClassLoader.loadClass(PluginClassLoader.java:126)         at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:525)         ... 21 more
Reference link:- https://github.com/nineinchnick/trino-openapi/tree/v1.3
Downloaded the plugin data from  https://github.com/nineinchnick/trino-openapi/releases/tag/v1.3
Could you the right dependency in the repo to resolve this issue?

updated

Why have you chosen the version 1.3? It's one of the initial versions of this connector that probably is very limited in functionality and has lots of issues. It's also built using Trino 418.
The error you're getting means the OpenTelemetry libraries are missing. This is expected, because they were added in Trino 414, and you're running 402. Things you can try:

Upgrade Trino
Add the OpenTelemetry jars to this plugin's dir. The plugin might still not work because of incompatibilities in the SPI.
Fork this repository and build the latest version of this plugin with an older version of Trino. This will require code changes, as there were many SPI changes since 402, but it's the most robust solution.

I'll close this since there are no updated on it. It doesn't sound like an issue with this connector.


Include  for system_clock

Notes for Reviewer
I am a member of Microsoft vcpkg, due to there are new changes merged by microsoft/STL#5105, which revealed a conformance issue in iceoryx. It must add include <chrono> to fix this error.
Compiler error with this STL change:
D:\b\iceoryx\src\v2.0.6-c8abea20ae.clean\iceoryx_platform\win\source\time.cpp(42): error C2039: 'system_clock': is not a member of 'std::chrono'

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[ ] Assign PR to reviewer

Checklist for the PR Reviewer

[ ] Consider a second reviewer for complex new features or larger refactorings
[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #2377

@LilyWangLL thanks for the pull request. Since iceoryx is a project for safety-critical systems as part of the eclipse foundation we have to follow some processes.

Could you please sign the eclipse contributor agreement with the e-mail you have used to push your commits. See: https://www.eclipse.org/legal/eca/
You have to always use the pull request template. I have adjusted it for you.
Every pull request must be linked to an issue. I created the issue Fix windows performance issue #2377 for you.
Every commit must be linked to the issue by using the prefix iox-#2377

As soon as the ECA is signed and you have added the prefix iox-#2377 to all commit messages we can merge this pull request.

Thanks for your reminder, I submitted a new PR which modified the branch name, please review the new PR https://github.com/eclipse-iceoryx/iceoryx/pull/2378.


Add support for Vue plugins

Now all the components within retejs vue-render-plugin are rendered separately as their own vue instances. This disables the option to have a global Vuex store for example, maybe for now it can be imported per component, I don't know.
It would be useful in general to be able to globally define the Vue plugins in the options for example to use them over all components created throughout rete.

v0.5.0
editor.use(VueRenderPlugin, {
    options: {
        // router, plugin, etc.
    }
});

It is working great now, thank you!

can you please point me out to a working example. I'm new to Vue and would like to use Retejs for a project.

@Relatora do you want to integrate this into an existing project or create a new one?

I'm creating a new project

@Relatora For me its working now and I am using it like this:
Define the store in e.g. ./store/index.ts.
import Vue from 'vue'
import Vuex from 'vuex'

Vue.use(Vuex)

export default new Vuex.Store({
    plugins: [],
    modules:{
    },
    state:{
    },
    mutations: {
    },
    actions: {
    },
    strict: false
})

And where ever you define the vuerenderplugin import it into Rete.js like this:
import store from '../store'

this.editor.use(VueRenderPlugin, {
        options: {
            store
        }
      });

Hope this is of some help to you.


[Error] how to fix about std::bad_alloc

my output
terminate called after throwing an instance of 'std::bad_alloc'

Do you have items.dat in same folder as decoder? If so, can you please share your items.dat?

dm my discord bengt#5300
Linux method is working all is fine so I can close this issues


update urls from git to https

Change the use of git@ to https for more convenient cloning

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


CommandStore as a Repository

This PR implements CommandStore as a Repository.
Breaking changes:

The Repository now does not have BoundedContext constructor argument. AggregatePartRepository, ProcessManagerRepository, and ProjectionRepository still have such a constructor argument.
CommandStore creates its storage with the help of StorageFactory for creating CommandStore as a repository.
Because of the above, StorageFactory does not produce CommandStorage any more.
Because of the above, InMemoryCommandStorage class was also removed.

@armiol, PTAL.


Is it desireable and possible to decrease the reference count of GtkListStore?

Hello,
Sorry if this is stupid question, but I am not so skilled to figure it out myself.
I wanted to write simple program using GtkTreeView with GtkListStore corresponding respectively to gtk::TreeView and gtk::ListStore.
In tutorial/guide for GTK+:
https://en.wikibooks.org/wiki/GTK%2B_By_Example/Tree_View/Columns_and_Renderers
there is an example C program using GtkTreeView and GtkListStore.
There is a line of code:
g_object_unref(model);
I would like to know if there is a function/method in gtk-rs bindings to be used by programmer using bindings? Or is it automatically done under the hood by bindings?

Done under the hood. To put it simply: when an object is destroyed, we call this function.

"When an object is destroyed", does it mean that variable is going out of scope?
Like in this example with variable model_list_of_data?
    let box_container = Box::new(Orientation::Vertical,0);
    let view_list = TreeView::new();
    {
        let model_list_of_data = ListStore::new(types_inside_columns);
        view_list.set_model(Some(&model_list_of_data));
    }
    box_container.pack_start(&view_list,false,false,0);

If yes, I know everything I want. I think issue can be closed. Thanks for help and for your work on bindings.

Absolutely. Like in your example with model_list_of_data.


"Average price per hour" in calculator misleading

The problem
The "Average price per hour" field at https://steamdb.info/calculator uses the formula:
sum_over_all_games(price_of_game_X/hours_in_game_X)/count_of_games
instead of the formula that (in my opinion) it should be using:
total_price/total_hours
Why this is a problem
There have been several issues over the years for the "Average price per hour" field (see #362, #403, and #703). In these issues, it has been made clear that the current behavior is intended. However, I think it's clear that this behavior, in its current implementation, is misleading - if the mentioned issues and my own experience aren't enough, a quick google search also brings up:

This confused post (with the next one agreeing "no idea").
This reddit thread with multiple comments listing data that shows that the "overall cost per hour" from gaugepowered.com (now dead) used the simpler formula. No one is confused. Note also that it appears that gaugepowered.com was the inspiration for this feature, so the current behavior may actually have originally been a mistake.
This reddit thread with this comment giving an explicit (wrong for steamdb! the user uses the second formula, not the first) explanation of what "Average price per hour" means, this comment correctly mentioning that the calculator is not accurate for them, this comment saying that "their $8 per hour is kind of sad" (first formula), then comparing the number to a manual calculation using the second formula (they come out with 36 cents, but don't notice the discrepancy) if they bought everything at lowest price, and at least like three more comments in the same vein. I didn't see anyone mentioning the formula that's actually used.

I'm sure I could find more evidence, but that seems like enough.
I also argue that the current behavior isn't very useful, even if you do understand it. I can kind of see how it means... something, but it's really hard to wrap my brain around what it actually represents, even knowing the formula. On the other hand, a simple overall average is easy to think about and draw conclusions from: "Whenever I sat down to play games from my Steam library, I was effectively paying X dollars for every hour of overall entertainment."
Solutions
Based on the evidence, I think it's clear that the formula should be changed from the first formula I mentioned to the second one. (Of course, I'm certainly open to the possibility that I missed a reason why the second formula is better.)
Regardless of whether the formula is changed, though, I think the description/tooltip should be changed as well. If the formula is changed, it should be made clear to users that it's been changed, and that there used to be a different formula in the same spot. And if the formula is not changed, it should be made explicitly clear what the actual formula is, to at least stop the confusion.
(Note: I'm not sure this should be classified as a bug, but being an existing problem rather than something new, I would say it's closer to a bug than a feature request. 🤷♂)

Used this as an opportunity to calculate more stats. Does this match what you would expect now?

Thanks for the quick response! Yeah, I think that takes care of the big issue. I like the extra new statistics!
Following up, here are my thoughts on a few other things that could potentially be improved:

It looks like the average price formulas use the lowest prices for your games, whereas the old formula used the current prices. This could be made more clear (maybe in a tooltip?).
In a similar vein, it might be useful to be able to see the average(s) using today's prices as well. For example, it could say something like "All games: $1.23 - $4.56". (This might not be useful at all, I haven't really thought it through - more just throwing the idea out there.)
I still think it would be good to be clear that the formula was changed, since there's now a completely different number under the same label. This could be something as little as adding another line to the tooltip saying something like "This formula was changed on June 23, 2023." People who check the page semi-regularly might easily get confused by why their average price per hour suddenly changed, but this extra line would clear up the confusion, at least for most of them.

But these are both more arguable and less important than the main issue was.

I don't really want to use today's prices because they fluctuate, and people then ask why it's lower during big sales.


Javascript text translation

Hello,
What do you suggest  to translate text in javascript like "Share", "Download image" without change the javascript file ?
Great package btw :)

This is a very good point. I'll extend the package with this function

Help wanted:

#3

#4

Here you go:

Version 0.2.0
Version 1.3.0


Add unit tests with bUnit

Egil Hansen has an extensive playlist of videos on YouTube explaining how to use bUnit to add unit tests to a fork of the Blazing Pizza application.
It's time to start adding tests to Foto Storio.

This is now in progress. I've added a test project (FotoStorio.Tests) and some simple tests of the client project components.
This is going to be an ongoing process.
I say this because there are 12 videos that Egil has added to the YouTube playlist (linked above). All of them were originally streamed live on Twitch. None of them are less than two hours long.
It's kind of like watching Bob Ross for people who are into Blazor and TDD. Just with fewer trees. 👍


prefix all route IDs with /

closes #7332
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[x] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

I think @Conduitry is right, it will be annoying to have that warning appear once everyone has already made the change


Correctly test for implementing IBonded in runtime-defined schemas ut…

…ilizing Bond.Bonded<>

Hi @mserd, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

Associated issue https://github.com/Microsoft/bond/issues/188

This wasn't the cause of my issue.


Reverse fix to temporary work around in finalize assembly rule.

          Note this is a temporary workaround and should be changed back to `end_repair` once the end repair module is working. Maybe we should make a TODO so we don't forget?

Originally posted by @jmtsuji in https://github.com/jmtsuji/rotary/pull/46#discussion_r1363347469

@jmtsuji I think this can be repaired in the end repair branch as you work on it.

OK, sounds good.

Addressed in 97a766b and 0b32b6b


code reviewed

In this commit
This PR adds flake8 and black to the project to ensure code quality and consistent formatting for Python.
Demo
https://github.com/user-attachments/assets/1a63e141-da16-4eab-8956-bcf6b24d02d6
After merging this PR, use flake8 and black locally before submitting new PRs.

All looks good, not sure why you made all the quotes single instead of double (my programming courses use doubles and standard and nest singles inside of doubles and doubles inside of triples ) happy to move to single quotes if its easier for you, the muscle memory will take me a bit though

You will not do it, it will fix automatically ...


pull request for e2e test at 11/13/2016 19:15:08

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


How to create upload annotation of polygon points to a job to create polygon tracks for multiple frames?

Hi,
I used this api to create polygon shape for the masks I stored in the format of polygon points. And it works. but I have to create the polygon shape for each frame. and the code is like:

with ApiClient(configuration) as api_client:
    action = "create"  # str |
    id = 24  # int | A unique integer value identifying this job.
    patched_labeled_data_request = PatchedLabeledDataRequest(
        version=0,
        shapes=[
            LabeledShapeRequest(
                outside=False,
                z_order=0,
                rotation=0.0,
                frame=0,
                label_id=33,
                group=0,
                source="manual",
                attributes=[],
                points=[
                    1067, 181,
                    1052, 330,
                    906, 340,
                    ......
                ],
                type=ShapeType("polygon"),
                occluded=False,
            ),
        ],
    )  # PatchedLabeledDataRequest |  (optional)
    try:
        # (data, response) = api_client.jobs_api.partial_update_annotations(
        (data, response) = api_client.tasks_api.partial_update_annotations(
            action,
            id,
            patched_labeled_data_request=patched_labeled_data_request,
        )
    except exceptions.ApiException as e:
        print("Exception when calling JobsApi.partial_update_annotations(): %s\n" % e)

I just want to know how to create the mask as polygon track from frame 0-n? so that I do not have to create mask for every frame
https://docs.cvat.ai/docs/api_sdk/sdk/reference/apis/jobs-api/#partial_update_annotations
I used LabeledTrackRequest which does create track, but it is a track from the frame I specifiy to the last frame of the dataset, and there is no interporlation. I want to the track only created for frames from 1-10.
Basically, I am looking for API to do this https://github.com/cvat-ai/cvat/issues/5401#issuecomment-1333601084
There is an object moving among frame 1 to 10. So I need interpolation between those frames.
Thanks

Hi. If you want to upload a track, just put in in the tracks field, instead of shapes. Example.


Declare specified OTel package versions as a peer dependency to prevent problems caused by different versions

Description
Using different OTel versions can cause problems if Faro would not used the version as specified.
Proposed solution
Declare respective packages as peer-dependencies

Hey @codecapitano, do you have any updated on this?
I am having an issues with different versions of @opentelemetry packages in my build. When it was 1.2.0 version I had to add in package.json resolution the following:
{
  "resolutions": {
    "@opentelemetry/core": "1.15.2",
    "@opentelemetry/resources": "1.15.2",
    "@opentelemetry/semantic-conventions": "1.15.2",
    "@opentelemetry/sdk-trace-base": "1.15.2",
    "@opentelemetry/sdk-trace-web": "1.15.2"
  }

Now I switched to 1.2.8 faro and got one more complaint:
ERROR in @opentelemetry/instrumentation
  Multiple versions of @opentelemetry/instrumentation found:
    0.41.2 /builds/product/web-app/~/@opentelemetry/instrumentation from /builds/product/web-app/~/libjs-pdobservability/traces/traces.js
    0.44.0 /builds/product/web-app/~/@grafana/faro-web-tracing/~/@opentelemetry/instrumentation from<EMAIL_ADDRESS>

And one more issue with dependency as I can see is:

I would like to install "@grafana/faro-web-sdk": "1.2.4" and "@grafana/faro-web-tracing": "1.2.4"
When I look into my node_modules I can see, that @grafana/faro-web-tracing installs it's own<EMAIL_ADDRESS>
That again trigger multiple versions of package error

Hi @Memfisrain thanks for bumping this,
Will update update the otel deps in the web-tracing  package and clean up our own resolutions (seems they cause problems with recent otel version and we don't need them anymore).
Also I'll declare the otel packages we use as peer deps.
You can track it here.
Unfortunately we will release the next Faro version after Black Friday or maybe even Black Week.
Cheers,
Marco

Not actually closed

Not actually fixed

@SimenB can you give me a brief description.
Is it still the problem with different package versions in a build?

Title of issue is "use peer deps" - in that regard I don't think it should be considered completed. Dependency mismatch has been fixed, tho 👍

Title of issue is "use peer deps" - in that regard I don't think it should be considered completed. Dependency mismatch has been fixed, tho 👍

Thank you Simen.
Yes good catch, it got auto closed because of the linked PR.
Will reopen the issue.

Quick heads up @Memfisrain and @SimenB.
We released Faro v1.2.9 which contains the updated otel-deps in the  web-tracing package.


Search box linking problem

The search box on the website seems to only ever link to #top no matter where in the page the search result is located. It used to link directly to the headings you clicked on in the search results. This is a poor user experience because many of our documentation pages are quite long.
I've contacted support to see if we can get it resolved.

It turns out this is caused by us. For some reason, we are not generating IDs for our headings and so the search is not picking them up. I suspect this happened when we migrated from Jekyll to Eleventy.

@nzakas By IDs do you mean id attributes on the heading elements?

We're adding id attributes to these headings on DOMContentLoaded, and the search seems to be working fine with them (for example, type "suggestions" in the search).

Working on this.


HSEARCH-2746 Translate parameters of WordDelimiterFilterFactory from 0/1 to booleans on Elasticsearch

https://hibernate.atlassian.net//browse/HSEARCH-2746

merged!
From a performance perspective I'm not particularly fond of us creating a Map of parameters to then later convert them all and create a new Map of parameters.
But that's having sub-zero priority right now so let's not spend time on it, we'll revisit if we see it's actually a problem during perf tests.
Thanks!

merged!

Thanks!

But that's having sub-zero priority right now so let's not spend time on it, we'll revisit if we see it's actually a problem during perf tests.

It's especially low priority given that we're talking about bootstrapping code executed at most once per analyzer definition... Granted, bootstrapping code shouldn't be inefficient, but copying a dozen maps is a drop in the ocean.

It's especially low priority given that we're talking about bootstrapping code executed at most once per analyzer definition... Granted, bootstrapping code shouldn't be inefficient, but copying a dozen maps is a drop in the ocean.

Ooops, ok I missed it was in the context of bootstrap. Good thing I merged then ;)


Merge pumpPaintFrame and pumpFrame in WidgetTester

We can make all the test pass by removing more of the mocks and using the real
code.

Less is more.  LGTM


Bugfix: show the default validation message when there is a required error

Might related to #276.
However, showError() is still buggy.

OK, I manage to fix showError() by this commit .

@christianalfoni Works for me.

Thanks @wuct, just published v0.19.1 with this in it https://www.npmjs.com/package/formsy-react

Thanks for merging


explain status codes for 'fail' and 'redirect' functions in documentation

Describe the problem
you have set typed status codes for both but I cannot figure out what they mean or why they are what they are, explaining in docs would be useful
Describe the proposed solution
see above
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

Maybe we could have a link to https://developer.mozilla.org/en-US/docs/Web/HTTP/Status somewhere, but I don't think we ought to be doing more than that. These status codes have been around a very long time and are very well-documented online.


Fix the dependency code blocks' language

Github only highlights it if it's java :)

Thank you. How about gradle language instead of java?

@castorflex Woops! Gradle files use groovy, my bad. There is no gradle language? afaik.

You're right but if think github handles it. Anyway, Groovy's perfect


remove redundant z input to CSR sparse matrix ops

Currently, the CSR code is using w, v, u, z input and I want to change it so that it only uses w, v, and u.  This will bring it in line with the standard CSR implementation:
https://en.wikipedia.org/wiki/Sparse_matrix

[ ] add u_to_z() function to compute z based on u

[ ] unit tests
[ ] update doxygen doc

[ ] remove zargument from csr_to_dense_matrix()

[ ] update tests to match
[ ] update doxygen doc

[ ] remove z argument from csr_matrix_times_vector()

[ ] update tests to match
[ ] update doxygen doc

[ ] remove csr_extract_z() function

[ ] update tests to match

Also, I want to rename these functions:

[ ] csr_extract_w to csr_values

[ ] rename in tests
[ ] update doxygen doc

[ ] csr_extract_v to csr_col_indexes

[ ] rename in tests
[ ] update doxygen doc

[ ] csr_extract_u to csr_row_starts

[ ] rename in tests
[ ] update doxygen doc

Krzyzstzof pointed out that the redundancy was intention for reasons having to do with "garbage" values in Eigen that I don't understand.  So I'm going to close this and open a new issue to document the behavior of "garbage" values in the code and tests.

I back-pedalled on this, you should re-open. Let me know if you want me to get some pieces of it done.


Rename apis/core package to apis/kueue

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This is necessary to play well with kubebuilder
Which issue(s) this PR fixes:
Part of #171
Special notes for your reviewer:

/lgtm


Router stops working after using it on first page with nothing to go back to

Reported in forum.
Fuse 1.7.3 Preview
Android
When using the back button with nothing to go back to it stops working when navigating to other pages. Happens both on Android phone and in preview mode on local PC.
Steps to reproduce the problem:

In Preview Viewport -> Go back
Click Page1, you get to goto_page
In Preview Viewport -> Go back (Not going back)

Code to reproduce:
<App>
	<JavaScript>
    function goToPage() {
        console.log("Go page");
        router.push("goto_page");
    }

    module.exports = {
        goToPage : goToPage
    };
	</JavaScript>
	<Router ux:Name="router" />

	<ClientPanel>
	    <Navigator DefaultPath="home">
	        <Page ux:Template="home">
	            <PageControl ux:Name="containerControl">
	                <Page>
	                    <Panel Tapped="{goToPage}">
	                    	<Text Value="Page 1" />
	                    </Panel>
	                </Page>
	            </PageControl>
	        </Page>
	        <Page ux:Template="goto_page">
	            <PageControl ux:Name="containerControl">
	                <Page>
	                    <Text Value="Page 2" />
	                </Page>
	            </PageControl>
	        </Page>
	    </Navigator>
	</ClientPanel>
</App>

Can confirm a defect in the "GoUp" logic... would likely be fixed by https://github.com/fusetools/fuselibs-public/issues/1068 where GoUp is just discarded.

This will be resolved by https://github.com/fusetools/fuselibs-public/pull/1107
The default behaviour will not include the broken GoUp
It also adds an option to intercept this action if there is no back.


Different singleton registrations resolve to same instance

Singleton uses same instance when registered with different service types
Scoped instances are same within a scope but are different across scopes
Singleton instances are disposed exactly once

https://github.com/aspnet/DependencyInjection/issues/360

@yufeih,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

This is a breaking change. If we decide to support this, I'm not sure we can just break the existing behavior.

I see, like services.AddSingleton<IBar, IBaz, Bar>();

Closing per https://github.com/aspnet/DependencyInjection/issues/360#issuecomment-289603154


Is there a way to make the ships damageable by explosions?

Currently explosions ignore them.

What sort of damage do you mean? Like the ship falls out of the sky? Some of the blocks get removed as though it were a physical stationary block? This would require a lot of new code to find the blocks that were potentially affected by explosions and then treat them like Minecraft does. Easiest would be to just /kill the ship in an explosion (and I bet you could do this using KubeJS), but I assume you don't want that.

Thanks for the response:
So I actually discovered after posting this that vanilla explosions DO damage the ship, it's just that specifically explosions from Crayfish's Gun Mod don't (the bazooka in that mod can normally destroy blocks)
If this is something that's not possible I understand and I appreciate the consideration.

I find Prelaw's Blocky Siege to be great for putting cannons onto the ships, and they will damage other ships.


fido-authenticator: authenticatorGetAssertion does not work with hmac-secret extension

Trying to use the authenticatorGetAssertion command with the hmac-secret extension causes a CTAP2_ERR_INVALID_CBOR error.
For example using the hmac-secret.py example from python-fido2:
$ python3 hmac_secret.py
no pin

Touch your authenticator device now...

New credential created, with the HmacSecret extension.
Authenticate with salt: b'9b17deb11321cdb10b56d8b75c85125a0e5627440892ef067fdd9f052c1ffd22'

Touch your authenticator device now...

Traceback (most recent call last):
 File "/home/robin/reps/python-fido2/fido2/client.py", line 690, in get_assertion
   assertions, used_extensions = self._do_get_assertion(
 File "/home/robin/reps/python-fido2/fido2/client.py", line 754, in _ctap2_get_assertion
   assertions = self.ctap2.get_assertions(
 File "/home/robin/reps/python-fido2/fido2/ctap2/base.py", line 857, in get_assertions
   first = self.get_assertion(*args, **kwargs)
 File "/home/robin/reps/python-fido2/fido2/ctap2/base.py", line 829, in get_assertion
   return self.send_cbor(
 File "/home/robin/reps/python-fido2/fido2/ctap2/base.py", line 675, in send_cbor
   raise CtapError(status)
fido2.ctap.CtapError: CTAP error: 0x12 - INVALID_CBOR

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
 File "/home/robin/reps/python-fido2/examples/hmac_secret.py", line 113, in <module>
   result = client.get_assertion(
 File "/home/robin/reps/python-fido2/fido2/client.py", line 704, in get_assertion
   raise _ctap2client_err(e)
fido2.client.ClientError: (<ERR.BAD_REQUEST: 2>, CtapError('CTAP error: 0x12 - INVALID_CBOR'))

The problem is that ctap-types does not yet support the new pinUvAuthProtocol(0x04) field in the extension data, see https://github.com/solokeys/ctap-types/issues/1.


Amplify CLI adds Admin Queries API with wrong auth type

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v13.7.0
Amplify CLI Version
4.45.2
What operating system are you using?
Mac
Amplify Categories
auth, api
Amplify Commands
add, pull
Describe the bug
When I add the admin queries API it adds it wiht the wrong auth type, as you can see below it adds it with AWS_IAM and not AMAZON_COGNITO_USER_POOLS like it should be
    "api": {
        "plugins": {
            "awsAPIPlugin": {
                "amplifyDatasource": {
                    "endpointType": "GraphQL",
                    "endpoint": "https://z4idgkzgevagjmr4vwfh5ozcsq.appsync-api.us-west-2.amazonaws.com/graphql",
                    "region": "us-west-2",
                    "authorizationType": "AMAZON_COGNITO_USER_POOLS"
                },
                "AdminQueries": {
                    "endpointType": "REST",
                    "endpoint": "https://oall6h52m6.execute-api.us-west-2.amazonaws.com/dev",
                    "region": "us-west-2",
                    "authorizationType": "AWS_IAM"
                }
            }
        }
    },

Expected behavior
I expect the cli to add the admin queries api with the cognito user pool auth.
Reproduction steps
amplify update auth
walk through steps to add admin queries api give cli the group name "Admins"
GraphQL schema(s)

# Put schemas below this line

Log output

# Put your logs below this line

@kneekey23 what file is shown above? I don't recognize the format.

@edwardfoyle this is from the amplifyconfiguration.json on iOS:
{
    "UserAgent": "aws-amplify-cli/2.0",
    "Version": "1.0",
    "api": {
        "plugins": {
            "awsAPIPlugin": {
                "amplifyDatasource": {
                    "endpointType": "GraphQL",
                    "endpoint": "https://XXX.appsync-api.us-west-2.amazonaws.com/graphql",
                    "region": "us-west-2",
                    "authorizationType": "AMAZON_COGNITO_USER_POOLS"
                },
                "AdminQueries": {
                    "endpointType": "REST",
                    "endpoint": "https://XXXX.execute-api.us-west-2.amazonaws.com/dev",
                    "region": "us-west-2",
                    "authorizationType": "AWS_IAM"
                }
            }
        }
    },
    "auth": {
        "plugins": {
            "awsCognitoAuthPlugin": {
                "UserAgent": "aws-amplify/cli",
                "Version": "0.1.0",
                "IdentityManager": {
                    "Default": {}
                },
                "AppSync": {
                    "Default": {
                        "ApiUrl": "https://XXXX.appsync-api.us-west-2.amazonaws.com/graphql",
                        "Region": "us-west-2",
                        "AuthMode": "AMAZON_COGNITO_USER_POOLS",
                        "ClientDatabasePrefix": "amplifyDatasource_AMAZON_COGNITO_USER_POOLS"
                    }
                },
                "CredentialsProvider": {
                    "CognitoIdentity": {
                        "Default": {
                            "PoolId": "us-west-2:XXXXXXXXXXXXXXXX",
                            "Region": "us-west-2"
                        }
                    }
                },
                "CognitoUserPool": {
                    "Default": {
                        "PoolId": "us-west-2_XXXXXXXXX",
                        "AppClientId": "XXXXXXXXXXXXXXX",
                        "Region": "us-west-2",
                        "AppClientSecret": "XXXXXXXXXXXXXXXXXXXXXX"
                    }
                },
                "Auth": {
                    "Default": {
                        "authenticationFlowType": "USER_SRP_AUTH"
                    }
                }
            }
        }
    }
}

👋 Hi, this issue was referenced in the v4.50.2 release!
Check out the release notes here https://github.com/aws-amplify/amplify-cli/releases/tag/v4.50.2.


build: add detect-secrets stage to build

Looks good but the semantic release step needs a fix

Fixed

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


chore: add latest tag action

SUMMARY
Action that adds a "latest" tag to release. It also overwrites/removes the last "latest" tag.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Cool automation! maybe drop a comment that this will automatically run on https://github.com/apache/incubator-superset/blob/master/RELEASING/README.md and a different one superset docs/instalation

good idea. I had an earlier PR with docs/instructions to check out the latest before this tag that was blocked, so I just included it in here.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d95b7c2). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #11148   +/-   ##
=========================================
  Coverage          ?   55.20%
=========================================
  Files             ?      403
  Lines             ?    13370
  Branches          ?     3424
=========================================
  Hits              ?     7381
  Misses            ?     5797
  Partials          ?      192

Flag
Coverage Δ

#cypress
55.20% <ø> (?)

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

Continue to review full report at Codecov.

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

LGTM. My only worry is the scenario where we might do a fix to an old version (say 0.37.3) after we've released a more recent major/minor 0.38.0. In that case I believe this would point latest to 0.37.3. I think this won't be a problem right now, but once we hit 1.0, we'll probably start having this problem when we start releasing new majors while maintaining minors for stability.

That's a good point. Let me see if there's a check I can do in this action to make this more accurate.

@villebro I updated the logic in the action to see if the new release is tagged with a later version, if you want to take another look. thanks!

Thanks for the feedback @villebro. I'll make some updates in the next few days.

@villebro this should be ready for another look. I pulled the script out of the github action, so it can be run separate, with or without a --dry-run flag. You can also use a tool like nektos/act to run the github action directly. I could potentially add info into the readme on that, or we can leave as is. Either way, lmk what you think, especially if you get a chance to test this out!

Found one more bug I want to test...

I tested a few more times, and it's looking stable so far. Maybe we should consider some bash automated testing systems?

@villebro I caught another small bug during demo. I'll put up a fix today.

Ok, just one more small change https://github.com/apache/incubator-superset/pull/11148/commits/ba3189821437f22d3baf8ee3f7f8109fc536d35d It shouldn't affect the rest of the functionality, it's just one more check to be sure that the tag passed in exists. It won't be an issue when using the action itself, but could happen when someone uses the script directly.

@villebro @dpgaspar I think this is ready on my end. If everything looks good to you, you can merge when ready. Thanks for all your help!

When we land this, we should create a latest tag on 0.38.0.


Try astro@next--build

Hello there! We're experimenting with some ways to free up memory in https://github.com/snowpackjs/astro/pull/1979.
This points your astro-next branch to astro@next--build to see if this fixes the memory issues you're having!

Thanks @natemoo-re — unfortunately it looks like the (Netlify) build is still breaking with the JavaScript heap out of memory error

Thanks @trevormunoz. Bummer! We'll keep investigating this!


Failed to add a agent node to HA cluster, read: connection reset by peer

Hello,
Im trying to add a agent node to my HA cluster on k3s but im getting an error when trying that, here the info about my cluster.
Environmental Info:
K3s Version:
k3s version v1.21.14+k3s1 (982252d7) -> I NEED TO USE THAT VERSION BECAUSE THE APPLICATION IM GONNA RUN ON K3S ONLY WORKS ON KUBERNETES 1.21.X
 go version go1.16.10
Node(s) CPU architecture, OS, and Version:

Linux tardis10.icarotech.com 4.18.0-408.el8.x86_64 #1 SMP Mon Jul 18 17:42:52 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:

2 Masters , 3 agents
Installed k3s-agent with this options:
export K3S_KUBECONFIG_MODE="644"
export K3S_URL="https://<IP_ADDRESS>:6443"
export K3S_TOKEN="K1066b89416c06b863e9d3dea3a997de9298e6f1412045eb485bc3f346c679c6ebb::server:29707218bcc959f6dc6ec0565f392357"
export INSTALL_K3S_SKIP_SELINUX_RPM=true
curl -sfL https://get.k3s.io | sh -
Problem:

The cluster is up and i already have 2 master nodes running succesfully on it, but when i tried to add the first worker node, my k3s-agent process returned this:
Sep 19 16:06:12 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:12.170956529-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:46202-><IP_ADDRESS>:6444: read: connection reset by peer"
Sep 19 16:06:14 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:14.174570104-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:46212-><IP_ADDRESS>:6444: read: connection reset by peer"
Sep 19 16:06:16 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:16.178372378-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:46226-><IP_ADDRESS>:6444: read: connection reset by peer"
Sep 19 16:06:18 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:18.182254294-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:46242-><IP_ADDRESS>:6444: read: connection reset by peer"
Sep 19 16:06:20 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:20.185373192-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:48138-><IP_ADDRESS>:6444: read: connection reset by peer"
Sep 19 16:06:22 tardis10.icarotech.com k3s[3476]: time="2022-09-19T16:06:22.188957226-03:00" level=error msg="failed to get CA certs: Get \"https://<IP_ADDRESS>:6444/cacerts\": read tcp <IP_ADDRESS>:48164-><IP_ADDRESS>:6444: read: connection reset by peer"

Can you successfully curl -ks https://<IP_ADDRESS>:6443/cacerts from the new agent?

Hello @brandond,
0 No, tried now and i get this:
root@tardis10:~#  curl -vs https://<IP_ADDRESS>:6443/cacerts
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* connect to <IP_ADDRESS> port 6443 failed: No route to host
* Failed to connect to <IP_ADDRESS> port 6443: No route to host
* Closing connection 0

Tried again with another port and its succeed:
root@tardis10:~#  curl -vs https://<IP_ADDRESS>:22/cacerts
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 22 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/pki/tls/certs/ca-bundle.crt
  CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* error:1408F10B:SSL routines:ssl3_get_record:wrong version number
* Closing connection 0

Its about the 6443 i guess, any tips to solve this?

This sounds like the exact same thing you were trying to get working in https://github.com/k3s-io/k3s/issues/6133 - why did you move to a new issue?

Following on https://github.com/k3s-io/k3s/issues/6133.


Windows GPU gradient always 0 but CPU works

Hello all,
I was working on installing and testing the new windows gpu support and it does not seem to be working right for me. To start off, I was following this and I have all of the same software as their windows 10 known good configuration, except I have Microsoft Visual Studio Community 2015 with Visual C++ 2015.
My github head is: 2cbb9b529a6723ebbe07663156b18c04fbaf4531
After I few times of trying to get it to install, I managed to get it to install by changing to cmake 3.6 from 3.7. It installed fine but it does not seem to work properly.
I downloaded some known good code, file here, and when I run it specifiying that tensorflow should use cpu, I get good output that looks like this. If I allow it to run on gpu, I get bad output that looks like this. I am not sure why. I feel like it is probably an issue with my install even though it had no errors. Any help would be greatly appreciated.

Sorry, it doesn't work on 3.7. We're amending the documentation PR #5103 . I think it's possible to make it work, but not out of the box.

I know that it doesn't work on 3.7. I got it to install by changing from cmake 3.7 to cmake 3.6, but even though it installed, it does not run code properly on gpu for me. As shown by the above outputs, the code fails on gpu but works fine on cpu.

We're still working on the windows GPU, which is an outstanding community contribution, so not everything works quite yet. We're hoping to get more fixes in the next few days.
@guschmue FYI

Thanks for the information. I will make sure to stay in tune. I will also mess around and see if I can get it to work.

This sounds a lot like https://github.com/tensorflow/tensorflow/pull/5207.
I ran your sample code on my box and it works ok.

@Bren077s Please try @guschmue 's PR or wait until we push.

I tried installing tensorflow, by using the commands you told me and I got new errors. I also tried downloading directly from @guschmue github fork of tensorflow and got the same errors. Here are the errors that I get: Errors

This was fixed with: https://github.com/tensorflow/tensorflow/pull/5157 and merged into the master yesterday. #5207 was merged today into the master.
I have a jenkins for windows gpu following the master and its happy.

Nice!


Solve the following issues please

Can anyone help Solving the following errors:
Launching lib\main.dart on SM A346E in debug mode...
Warning: SDK processing. This version only understands SDK XML versions up to 3 but an SDK XML file of version 4 was encountered. This can happen if you use versions of Android Studio and the command-line tools that were released at different times.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':flutter_plugin_android_lifecycle:compileDebugJavaWithJavac'.

Could not resolve all files for configuration ':flutter_plugin_android_lifecycle:androidJdkImage'.
Failed to transform core-for-system-modules.jar to match attributes {artifactType=_internal_android_jdk_image, org.gradle.libraryelements=jar, org.gradle.usage=java-runtime}.
Execution failed for JdkImageTransform: C:\Users\deepa\AppData\Local\Android\sdk\platforms\android-34\core-for-system-modules.jar.
Error while executing process C:\Program Files\Android\Android Studio\jbr\bin\jlink.exe with arguments {--module-path C:\Users\deepa.gradle\caches\8.10.2\transforms\fa2e1a7f88e7e3f306017f7b8208dcc2-8e16f793-d8f9-44d6-95c7-6d4991536c59\transformed\output\temp\jmod --add-modules java.base --output C:\Users\deepa.gradle\caches\8.10.2\transforms\fa2e1a7f88e7e3f306017f7b8208dcc2-8e16f793-d8f9-44d6-95c7-6d4991536c59\transformed\output\jdkImage --disable-plugin system-modules}

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org.

BUILD FAILED in 37m 23s
Error: Gradle task assembleDebug failed with exit code 1
Help me solving the issues while assemble debugging anyone help please

This was initially reported in https://github.com/flutter/flutter/issues/156304 and has since been fixed. The fix is available on the master channel, however, there's a solution you can use in https://github.com/flutter/flutter/issues/156304#issuecomment-2397707812.
Closing as duplicate.


Use client accessor inside consumer

When a consumer is started, a new Kafka JS client instance is created https://github.com/jlandersen/vscode-kafka/blob/b72c127c4b443608fc89ea505685d443d7f3e258/src/client/consumer.ts#L102 which means:

a new Kafka instance is created on each consumer start
this Kafka instance is never disposed.

To fix this problem, we should use clientAccessor (which is a pooling of Kafka client) but it requires some refactoring like adding consumer method for Client API like we have for producer https://github.com/jlandersen/vscode-kafka/blob/b72c127c4b443608fc89ea505685d443d7f3e258/src/client/client.ts#L85.
Adding this consumer method to client will be consistent with producer method.
But a matter that we must clarify is about Client API structure. @jlandersen if I understand the Client API goal is to have an API which doesn't depend on kafkajs. But if I take https://github.com/jlandersen/vscode-kafka/blob/master/src/client/client.ts#L85 the method return a Producer structure which is a kafkajs structure.
The question is how to fix that:

keep like this?
redefine Producer structure?

Any feedback are welcome!

Unfortunately the coupling between Client and the KafkaJS already exists (since the switch to KafkaJS). We'll probably need non-trivial refactoring to decouple them.
I think we choose the pragmatic solution and merge this PR anyways, and deal with a potential refactoring after the next release.

Unfortunately the coupling between Client and the KafkaJS already exists (since the switch to KafkaJS). We'll probably need non-trivial refactoring to decouple them.

I created the issue https://github.com/jlandersen/vscode-kafka/issues/166


SPIKE: Add Cloud.gov sandbox banner to all dev/test/staging UIs

Research task to figure out how we might be able to accomplish story #508
@jwhitehead77 and @maheese will discuss Swagger UI implementation

Plan design around documentation site once API gateway is figured out.


Let active components highlighted

Hi i would like to have components of GTK Apps that are active highlighted with this theme.
example: Xournal++ active modes. Currently you only have a visual identifier of a component being active when hovering over it:
non hovered:

pen active, pen hovered:

eraser active, pen hovered:

this does not only apply to Xournal++ but to some other programs as well, they dont have visual identifiers for active components.

sure :D gtk is totally under rewrite it may take some time


Intellisense stops working

Issue Type: Bug
Intellisense stops working, I have to frequently restart  VS code to make it work
VS Code version: Code 1.33.0 (0dd516dd412d42323fc3464531b1c715d51c4c1a, 2019-04-04T15:14:28.026Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2400)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.90GB (5.91GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (24)

Extension
Author (truncated)
Version

vscode-svgviewer
css
2.0.0

dart-code
Dar
2.25.0

EditorConfig
Edi
0.13.0

vscode-npm-script
eg2
0.3.5

easy-icons
jam
0.3.1

vscode-styled-sort
joh
0.0.7

vscode-styled-components
jpo
0.0.26

code-beautifier
mic
2.2.1

csharp
ms-
1.18.0

debugger-for-chrome
msj
4.11.3

vscode-docker
Pet
0.6.1

material-icon-theme
PKi
3.7.0

java
red
0.42.1

Align
ste
0.2.0

language-stylus
sys
1.11.0

shell-launcher
Tyr
0.3.0

vscodeintellicode
Vis
1.1.5

vscode-java-debug
vsc
0.17.0

vscode-java-dependency
vsc
0.4.0

vscode-java-pack
vsc
0.6.0

vscode-java-test
vsc
0.15.1

vscode-maven
vsc
0.16.0

vscode-icons
vsc
8.4.0

glean
wix
4.10.1

(1 theme extensions excluded)

Please describe how to reproduce the issue
/needsMoreInfo

@mjbvz Intellisense and go to definition both aren't working since after the update.

I have the same problem today when VSCode update to version 1.33. I have been downgrading to version 1.32.3 and stop the option automatic update checking. It works fine while we are waiting for the hot fix from the VSCode team.

Same problem after update

Same problem, however it also reproduces on 1.32 if I select the project Typescript version (3.4) instead of internal VSCode version (3.3).

I was looking through the extension host log and saw and error that reads:
[error] Error: TypeScript Server Error (3.4.1)
Debug Failure. Expected 1 === 2. Registered script kind should match new script kind. <path to file>

And a bunch of:
[error] Error: TypeScript Server Error (3.4.1)
Debug Failure. Expected <# greater than 190> <= 190

Not sure if this helps

Duplicate of #71751
See https://github.com/Microsoft/vscode/issues/71751#issuecomment-480126916 for possible workarounds
If that doesn't solve the issue for you, please open a new issue


Add Kubernetes controller app service

Codecov Report

Merging #61 (4e49ad5) into main (611faf1) will decrease coverage by 1.32%.
The diff coverage is 8.33%.

@@            Coverage Diff             @@
##             main      #61      +/-   ##
==========================================
- Coverage   83.43%   82.10%   -1.33%
==========================================
  Files          14       14
  Lines         682      693      +11
==========================================
  Hits          569      569
- Misses         77       88      +11
  Partials       36       36

Impacted Files
Coverage Δ

internal/k8sprometheus/kubernetes.go
0.00% <0.00%> (ø)

internal/app/generate/prometheus.go
82.08% <100.00%> (ø)

Continue to review full report at Codecov.

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


Sort challenges by Latest (start date), Oldest

Filter inspired by Kaggle.

[ ] Update OpenAPI description (@tschaffter )
[ ] Implement backend (@tschaffter )
[ ] Implement frontend (@rrchai )

Overlap with Closing soon, starting soon


Add UI tests

Add helper class for testing, adds UI tests with snapshots:

"Notebook Jobs" card is visible in JupyterLab launcher and leads
"Create a notebook job" button is visible in notebook toolbar and leads to "Create a Job" page
"Create Notebook Job" item is visible when right clicking a notebook in File Browser and leads to "Create a Job" page
Create a job and see it in the list of jobs

Things not covered in this PR for follow-up PRs:

Edit a job
Details view (labeled value component needs to be modified to accept test-ids so that dynamic data like timestamps could be targeted for masking)
Sorting in list view (createJob helper function needs to be created)
Job Definition tests
Add additional helper functions like createJob and deleteJob, use them to isolate tests from each other
Figure out why when attempting to delete a job on CI, I get "unexpected error occurred while deleting a job":

Fixes #381

@andrii-i
You seem to have conflict with the current changes on main. Can you give me access to your fork, so I can rebase and merge.

@3coins invited you as a collaborator. Based on a brief glance status is green / "This branch has no conflicts with the base branch"

Hey @andrii-i, I've just pushed two commits:

Adds ui-tests/yarn.lock file to fix local Playwright installation
Runs prettier to lint the test files

Just realized we're not running lint in CI; I've filed an issue to track this: https://github.com/jupyter-server/jupyter-scheduler/issues/399

I noticed that when I ran my tests locally, most failed because snapshots generated on different platforms (MacOS v.s. Linux) use different snapshot filenames, which fails the test b/c the snapshot file doesn't exist. Do you think you could run the tests locally on MacOS, push the snapshots, and then add an additional job under build.yml that's the same as the integration-tests job but it runs on MacOS? I think it's good to run these E2E tests on both MacOS and Linux platforms, since it's basically free confidence for us.
While most tests failed solely because I was missing the snapshot files, this assertion in particular fails for me locally:
  test('"Create Notebook Job" item is visible when right clicking a notebook in File Browser and leads to "Create a Job" page', async ({
    page
  }) => {
    await page.sidebar.openTab('filebrowser');
    expect(await page.sidebar.isTabOpen('filebrowser')).toBeTruthy();

Watching the video replay, it looks like the filebrowser is already initially open on MacOS, but the statement await page.sidebar.openTab('filebrowser'); closes the filebrowser again? 🤔 Not sure why this is only happening on MacOS (since CI is green), so if it proves too difficult to debug, you can just skip the test for the MacOS platform and open an issue to track it.

I rolled the PR back to before macos / darwin tests, created #400 to track it. Other than that all comments were addressed, it’s ready to go

@dlqqq Thank you for the review


Make it easier to create a VM with existing qcow2

Use case:  I inadvertently deleted a Windows 11 VM from UTM. The .utm package with the qcow2 drive was stored in an external SSD drive so I still had it. It was not straightforward creating another container which included the qcow2. (Copied the qcow2 to separate file and imported it into an existing VM - could not create a new VM and add it. One of the issues is that the import file dialog will not let you open a .utm package and access the package contents - you have to copy/move the qcow2 out of the package.)

I don’t get it, why couldn’t you just double click the .utm?

Looks like I overthought it.  Didn't even think of that. I thought UTM stored its plists in user's library and it would be more complicated than just double-clicking the .utm.
Cheers.


GetSafePathsToAllow creates rules for only last path provided as many times as the number of paths listed

When adding paths to CustomizationInputs/GetSafePathsToAllow.ps1 such as:
### Windows Defender put their binaries in ProgramData for a while. Comment this back out when they move it back.
 There should be three: one path rule each for EXE, DLL, and Script collections.

I'm experiencing the same issue. I added multiple lines at the end of the GetSafePathsToAllow.ps1 file. The end of the file looks like this:
### Windows Defender put their binaries in ProgramData for a while. Comment this back out when they move it back.
"%OSDRIVE%\PROGRAMDATA\MICROSOFT\WINDOWS DEFENDER\*" "%OSDRIVE%\PROGRAMDATA\MICROSOFT\WINDOWS\PASSWORDEXPIRENOTIFICATION.VBS" "%OSDRIVE%\PROGRAMDATA\SEAGULL\DRIVERS\*"
If I run the script the output looks good
PS C:\AaronLocker\CustomizationInputs> .\GetSafePathsToAllow.ps1
\\MYDOMAIN.LOCAL\netlogon\*
\\MYDOMAIN.LOCAL\sysvol\*
%OSDRIVE%\PROGRAMDATA\MICROSOFT\WINDOWS DEFENDER\*
%OSDRIVE%\PROGRAMDATA\MICROSOFT\WINDOWS\PASSWORDEXPIRENOTIFICATION.VBS
%OSDRIVE%\PROGRAMDATA\SEAGULL\DRIVERS\*
But when you run Create-Policies.ps1  path rules are created using only the last line of  GetSafePathsToAllow.ps1. In this case only EXE, DLL, and Script rules for %OSDRIVE%\PROGRAMDATA\SEAGULL\DRIVERS\*"  are created and any previous entries are omitted.


Allow in TaxonomyPicker tree select entire node with its child elements

Category
[x ] Enhancement
[ ] Bug
[ ] Question
Version
Please specify what version of the library you are using: [ 1.9.0 ]
Expected / Desired Behavior / Question
At the moment  if I want select a node in taxonomy tree with large numbers of child items I need select each element of this node.... It is very uncomfortable.
Please add ability to select entire node and choose the way how show selected items in box.
For example:
Default: just show child nodes.
SHOW ALL: show all checked treeNodes (include parent treeNode).
SHOW_PARENT: show checked treeNodes (just show parent treeNode).
Thanks!

Hi @bva999,
Did you have the chance to use latest version of TaxonomyPicker? I think there're a bunch of improvments since version 1.9.0.
There's also another control called ModernTaxonomyPicker, which provides another approach of using term sets.


Add code to autogenerate new problems with LLM

I was able to generate code for problems with chat got. But would be nice to have sth to do it with simple command instead of copy pasting.

kind of working with groq. may need few fixes tho.


Add support for UTF-8 when downloading CSV

This pull should add UTF-8 support for special / accented characters when downloading CSV files.

Coverage remained the same at 76.339% when pulling 9bea535f41713d28134f85e69c1ce58c4b0ddbe5 on rinorbytyci:master into a929a3f173e5bbcd2d3c65ba4b793546f5cbda16 on gregnb:master.

This can also be done with the onDownload method:
onDownload: (buildHead, buildBody, columns, data) => {
    return "\uFEFF" + buildHead(columns) + buildBody(data);
}

The problem with always adding the BOM in is that it's Excel specific, and the actual character will show up in some spreadsheet programs (ex: https://github.com/react-csv/react-csv/issues/89).

This library has intentionally steered away from providing specific types of exports as this can become a source of bloat. Specific implementations are left up to the devs and external libraries.
Perhaps a better use of this PR would be to add something to the documentation or to an example that points out how to work with downloads intended for Excel (or other programs).

Excel specific, so closing for now. I'll add an example in the documentation though.

// option
downloadOptions: {
filename: 'excel-format.csv',
separator: ';',
filterOptions: {
useDisplayedColumnsOnly: true,
useDisplayedRowsOnly: true,
}
},
//function
onDownload: (buildHead, buildBody, columns, data) => {
component.pdfReportAutoTable(
columns,
data
);
return false;
},


Wikidata integration

EPIC ticket with some suggestions for better wikidata integration.  This should be discussed on a future CL call where we agree which items in this epic should be turned into separate tickets.  This issue can be closed once all items have been discussed and we've decided which to implement.
Currently mapped terms (as known to wikidata:  https://w.wiki/6WGX)

[ ] #1944
[ ] #1945
[ ] #1946

Some Wikidata constraint checks in CL mappings:

one CL term to many WD entities: https://w.wiki/6eaQ
one WD entity to many CL terms: https://w.wiki/6eaT

I’ll investigate the possibility of using SSSOM for that. If we can somehow get a SSSOM mapping set (either directly from Wikidata or possibly through mapping-commons), we could easily derive a component ready to be imported into CL.

@gouttegd that sounds very doable; I'll work on deriving this SSSOM mapping set,

@gouttegd  here is the SPARQL query for the minimal SSSOM set: https://w.wiki/7x$C
something like:

subject_id
predicate_id
object_id
match_type

wd:Q116343126
skos:closeMatch
http://purl.obolibrary.org/obo/CL_0000735
Complex

wd:Q116446765
skos:closeMatch
http://purl.obolibrary.org/obo/CL_0000744
Complex

(...)
And here is a Wikipedia-only version of the set: https://w.wiki/7x$S

I've just found this by @dosumis from ~8 years ago: https://github.com/OBOFoundry/obo_wikidata_mapping
Might be relevant.

cc @andrawaag FYI

Comming back here, did we have any updates on this?  The (basic) SSSOM mapping to add to CL was done, but I am not sure if we have a pipeline in place to generate ontology changes from that. @gouttegd do you have any thoughts?

I had no time to work on that until now, but I should be able to find it now that the whole overhaul of Uberon’s composite-metazoan is done. And the upcoming ODK 1.5 will come with the SSSOM plugin built-in, so deriving a component from the SSSSOM set should be easy.


Cmake support

Are there any plans to provide support for CMake as meta build system?

Hi, thanks for reaching out, and apologies for the late response.
We'll shortly release the next version of the ATP Engine, but it will not include CMake support. There is no plan to add it in future releases as of now. If this is of high relevance for you, please let me know and we can look into adding it. Feel free to reach to my e-mail address if that's more suitable.
Of course, we're also happy to accept any contribution.

Hi, thanks for reaching out, and apologies for the late response.
We'll shortly release the next version of the ATP Engine, but it will not include CMake support. There is no plan to add it in future releases as of now. If this is of high relevance for you, please let me know and we can look into adding it. Feel free to reach to my e-mail address if that's more suitable.
Of course, we're also happy to accept any contribution.


docs: Update doc about CSV upload

SUMMARY

Docs about CSV upload is stale. I update it to match current UI.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

NOTE: not full difference.
Before:

After:

TESTING INSTRUCTIONS

N/A. Just updating doc.
ADDITIONAL INFORMATION

[x] Has associated issue: #14143
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thank you for your review.
Our organization has not determined to use Superset yet. I will add it when I will contribute Superset again after our org stating use it. Thanks.