K8s runtime

Alright, @josevalim asked me if I wanted to contribute a K8s runtime so here it comes. Took me ages to build a UI with a few forms on it!!! 🤣
A few notes
Connection and Authentication
When working with Kubernetes you usually don't ever see connection and authentication details. Instead, you use the helper of your cloud provider which adds all infos into your user's Kubernetes configuration YAML file - on unix systems that's on ~/.kube/config. I therefore decided to rely on an ENV variable pointing to such a Kubeconfig file instead of prompting for credentials. If users want an ad-hoc way of connecting to a cluster, I can add this in a later PR. But I'd rather let people use it and gather some feedback.
Specs
Kubernetes Pod specs contains soo many fields. I'm only allowing to configure the ones I think are most important especially and also for running on GPU nodes (node selector, tolerations, resources, persistent volume). For FLAME I support a BYO Pod manifest. This might be something to add later so that people can paste their YAML instead of filling the form. We would than only override the image and add required ENV vars.
UI
Yeah... please somebody make that form more beautiful! 😜 No seriously, I think it's ok for now but I'm sure somebody could do a better job.

Ready for Review
I'm still gonna run some tests (in-cluster and error handling for missing permissions and other non-happy cases) and maybe commit some fixes but feel free to review what's there.

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

Thank you @mruoss! The PR looks great, I think the biggest questions are exactly the ones you wrote in the issue. Authentication and specifications.
About authentication, the most likely scenario is that I am using my desktop and I want to connect to a k8s cluster my company runs. Would this also be done via ~/.kube/config?
About the specs, I think replicating it in the UI is just too much hassle. I can think of two options:

Templates? Is it possible to specify templates within k8s? If so, maybe we could simply list all deployments and we use the template in that deployment? Then we can provide some documentation on how to create a default deployment.

Another option is to just have a text area with the yaml . We can include a default yaml and we just use that. Later we can even make it use the codemirror editor. I think this may provide the better experience. We can simplify all of the code, users can copy and paste, and make it as complex and as simple they need it to be.

About authentication, the most likely scenario is that I am using my desktop and I want to connect to a k8s cluster my company runs. Would this also be done via ~/.kube/config?

If the company cluster's API Server is reachable, then it is technically possible, yes. But there are some caveats as I already mentioned earlier. Kubernetes supports several mechanisms of authentication. Some of these mechanism (exec) require a shell for a user that is logged in to the cloud provider (e.g. GCP). These auth mechanisms won't work for let's say livebook dekstop. They only work if you run livebook from the CLI as the user that's logged in to the cloud provider.
There's a workaround for this use case, though. If the user has enough permissions, they can create a K8s Service Account (SA) or use an existing SA and make sure it has all the required permissions (Role and RoleBinding). They can then Create an API Token for the SA and create new cluster, user and context entries in ~/.kube/config or create a new and dedicated kubeconfig file. This is not very difficult but you have to know what you're doing. I could create a mix task that does all of this. But then again, when Livebook is run as Desktop version, there's no mix tasks to the rescue.
To come back to your initial question: I think most/all auth mechanism are supported by Kubereq via Kubeconfig file. But creating UIs for all of them (API Token, Client Cert auth, ...) would be a lot of work. That's the reason I decided to go with the file. It's just the "creating the file" that's the difficult part.
Hope I could make things clear.

About the specs, I think replicating it in the UI is just too much hassle. I can think of two options:

Are you proposing to remove the form completely and only offer a BYO manifest version? Or leave the simple form and offer a more sophisticated alternative?

Templates? Is it possible to specify templates within k8s? If so, maybe we could simply list all deployments and we use the template in that deployment? Then we can provide some documentation on how to create a default deployment.

It is a nice thought. "Deployment" is the wrong term here, because a Deployment actually schedules pods to be run. But I think there is a PodTemplate resource we could use. Users would then have to create a PodTemplate and apply it to the cluster prior to creating the runtime. I'll dig a bit.

Another option is to just have a text area with the yaml . We can include a default yaml and we just use that. Later we can even make it use the codemirror editor. I think this may provide the better experience. We can simplify all of the code, users can copy and paste, and make it as complex and as simple they need it to be.

Yup, that's the thought I had as well. Both options would require users to also care about PVCs. Mixing UI/Form elements with BYO is also going to be too bit of a hassle...
Let me give the options a bit more thought....

To come back to your initial question: I think most/all auth mechanism are supported by Kubereq via Kubeconfig file. But creating UIs for all of them (API Token, Client Cert auth, ...) would be a lot of work. That's the reason I decided to go with the file. It's just the "creating the file" that's the difficult part.

To be clear, I actually like the file approach. The less we have to do, the better. :) I was just wondering how users would get that file in non-conventional cases.

Are you proposing to remove the form completely and only offer a BYO manifest version? Or leave the simple form and offer a more sophisticated alternative?

Correct. I would get rid of the form altogether. Almost everyone using k8s will be familiar with the yaml config. We can support either YAML, PodTemplates, or both. For PVC, we could still manage it, and either dynamically modify the yaml before applying it OR just change the default YAML we will show based on your PVC config.

I would get rid of the form altogether.

You've got to be kidding me! There is blood in this form! 😂 Just kidding. It will be much easier this way to maintain. I'll try to find a good way to support YAMLs.

It will be much easier this way to maintain. I'll try to find a good way to support YAMLs.

Feel free to start with just the textarea and we can do the rest. :)

Okay, it's just a plain monospace textarea now. Using the codemirror editor instead would be amazing of course! But I didn't look into that yet.
I can definitely also add a "load PodTemplate from cluster" functionality. Although I'm not sure this really adds value for anybody.

Okay. I have one last thing on my mind/list: I think I need to make the timeout (how long to wait for the pod to get scheduled and start running) configurable. In some cases, this can take minues (e.g. downloading the docker image already can take a minute).
Also the default probably has to be higher than 15s.

Perhaps we hust make the timeout arbitrarily high and we allow the user to press the stop button if they want to give up?

Or that... And find a way to show event stream (e.g. "downloading image...)

I've pushed a few commits today which make the whole feature more robust (checks for all required permissions, error messages etc). Now I am okay with the code all-in all with the following exception:
I have put the timeout to 5 minutes now but can even make it higher. However, I didn't really figure out a good way to abort the connection. What I have implemented now is that the runtime updates (send_update/3) the runtime LV component and assigns the pod_name. If that variable is non-nil in the runtime component, it renders an "abort connection" button which snipes the pod. See https://github.com/livebook-dev/livebook/pull/2756/commits/951bfd602592e1f5d6951212727e44f65e7e1fed.
I don't like the solution too much because of the messaging back and also because the user gets an error message saying that the Pod was deleted... So if there's a better way to abort the connection, let me know.

It would be great if we could add an integration test (against a local cluster) for the runtime itself, similar to test/livebook/runtime/fly_test.exs. This way we have an easy way to assert all the connection and communication, which is the most important. It would only run when TEST_KUBECONFIG or similar is set. I would appreciate if you could also add a comment with minimal steps necessary for a matching local k8s setup.

Oh yeah, absolutely agree. For flame_k8s_backend I have implemented the setup_all/N callback to check for a local Kind cluster and create one if it is not there etc. It is super handy for me because I just need to run mix test --include integration and the env is setup for me. Not sure though if you want to go that far here?
I can document K8s dev setup using markdown or by implementing a mix task - up to you.

We should have a test for the UI. For this one we want to stub all the requests. Similar test for Fly here.

Sure will do. Just wanted to get to an agreement on the concept / basic implementation first.

All public functions in the codebase should have docs and specs. It may not feel necessary in all cases, but I think sticking to this rule makes the codebase more accessible.

We can definitely share some code with the Fly runtime, but I will do that separately!

👍

For flame_k8s_backend I have implemented the setup_all/N callback to check for a local Kind cluster and create one if it is not there etc.

Sounds great!

@jonatanklosko I've added an integration test now. I've also added it to the CI because I think this really helps to catch regression. However, it does slow down the pipeline quite a bit (40-50s to install Kind and the cluster plus >1min to run the test as in the setup_all I have to build and load the docker image to the cluster. WDYT?

@mruoss honestly I think it's fine to skip it on CI. The tested code is rather encapsulated, so it should be evident when we want to run these tests. The Fly runtime tests are already opt-in, because they require a token. This is definitely a trade-off, but I think slowing the CI feedback by 1min+ is not worth it :)

OK I will remove it from CI again. Can always put it back later. Also possible in a different pipeline that runs under different conditions.

I think it's only UI tests left now. Have to find a good way to mock the kubeconf setup first. Mocking the req should then be straight forward...

@mruoss honestly I think it's fine to skip it on CI.

@jonatanklosko It looks like kind is installed on those GH runners which makes the test run automatically. Shall I add --exclude k8s to mix test in the CI job?

@mruoss I did small adjustments to the form. I grouped the image picker and template under the same Pod section and moved storage to the bottom, since it's more of an addition to the template. Let me know if this makes sense :)

@jonatanklosko @josevalim I've added integration and UI test and feel pretty confident now regarding regression. So I think this is ready for a final review. I guess the changelog entry will be created on the 1.14 branch?
Sorry for taking so long but I didn't have that much time to work on it. That being said, I really really enjoyed working on this and refreshing my LiveView knowledge a little. There's been a lot of progress since the last time I've worked on a productive LV project and oh man... the whole framework feels so nice! Even writing tests, mocking calls etc. It all feels clear and easy and I had so much fun.

@mruoss thank you! I write the changelog when releasing, so no need to worry about that. Great that you enjoyed working on this and the timing was great, so no worries :)


driver for HP 83650A

@unga @jenshnielsen  @WilliamHPNielsen

@giulioungaretti Fixed some of the comments. @brunobuijtendorp Can you look at the other ones (in particular related to renaming functions)?

@peendebak will do tomorrow when in Delft :)

@brunobuijtendorp PR was blocked because the formatting was not right. I did an autopep8 to fix it.
@giulioungaretti PR is ready for inclusion

@peendebak looks like something went wrong with the last commit ? It reverted some changes (e1d9ecea1780d16f0b486a4f68ba3f64f9e7840d)

@giulioungaretti Thanks. It was not in the last commit, but the one before. I reverted the reverted changes.
@brunobuijtendorp Can you test the latest commit and give a green light when it is ready?

@peendebak Yes I will check it out tomorrow at qutech! Looks like in commit 'f9e2dabdab3fedfcdf9fe7cbd7a3e1fd94f99f04' I must have accidentally edited an old commit, thereby reverting the changes. Thanks for fixing that :)

@eendebakpt, @giulioungaretti Changed units to unit in two more lines. Driver is tested and ready now.

@brunobuijtendorp see review! then ready to merge!

@giulioungaretti Requested changes have been committed. :)

@brunobuijtendorp @eendebakpt thanks for the contribution! mergin' 💖


michalhorvat

ADD ISSUE DESCRIPTION HERE
Version: 1.81.1
Commit: 6c3e3dba23e8fadc360aed75ce363ba185c49794
User Agent: Mozilla/5.0 (X11; CrOS x86_64 14541.0.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

### Tasks

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

Michal Horvat  @atumcurrency


[K32W] Endless loop while calling Heap::CAlloc

The OT stack gets stuck in the while loop illustrated in the below image:

Investigated further and put an assert(0) in the heap constructor (Heap::Heap) but that code is not reached. Also, investigated with the debugger and it looks like the memory structures are not initialized, so this double confirms my assumption that the heap constructor is not called. When should the heap constructor be called?

Does your cstartup include C++ runtime initialization?
For example: https://github.com/openthread/openthread/blob/b7751460e04a233589393fd4faccef5bc6e8e88b/examples/platforms/cc2538/startup-gcc.c#L201

Thanks, Jonathan. Indeed, I was not calling C++ runtime initialization.


[VOL-138] Read and save config

https://clusterhq.atlassian.net/browse/VOL-138
This branch adds support for storing the configuration, reading it properly, and being able to print the configuration.
I moved the configuration location into basePath after a discussion with Luke (multi-user support isn't here yet).

Following the discussions from Slack this morning, the dvol builds are failing because golang.org/x/tools/cmd/vet has moved. Following @mdevilliers instructions, I added f1a8a4c to a branch of mine and the builds then proceeded.

Looks good! Please address comments and merge when green.


BEV range 90mx30m

Trying to reproduce the BEV range 90mx30m in the paper, what parts of the code should be changed?

In config, point_cloud_range = [-15.0, -45.0, -2.0, 15.0, 45.0, 2.0],
We choose num_vec to 75, thus the related numbers need to be changed
In this line, change 50 to 75
In this file, related parameters need to set as 75.

Sorry for our poor implementation. We will refine our code later.

The size of the BEV map feature should also be proportionally enlarged.


Calling unknown method: filsh\yii2\oauth2server\Module::handleAuthorizeRequest()

Calling unknown method: filsh\yii2\oauth2server\Module::handleAuthorizeRequest()
In method

public function actionAuthorize()
{
if (Yii::$app->getUser()->getIsGuest())
return $this->redirect('login');
$module = \Yii::$app->getModule('oauth2');
$response = $module->handleAuthorizeRequest(!Yii::$app->getUser()->getIsGuest(), Yii::$app->getUser()->getId());

I had this same problem because I used composer to require 2.0.1, however this repo has a tag 2.0.1 and a branch v2.0.1.
The branch has the handleAuthorizeRequest() method and is more recent than the version tagged as 2.0.1.
I solved this by forking the repo, creating a new 2.0.2 branch in my fork based off the 2.0.1 branch and the modifying composer to use my fork, instead of filsh.
You're welcome to use my fake 2.0.2 branch here: https://github.com/tahpot/yii2-oauth2-server/tree/v2.0.2

I solved this by forking the repo, creating a new 2.0.2 branch in my fork based off the 2.0.1 branch and the modifying composer to use my fork, instead of filsh.

I had created this fork that I wanted to maintain but didn't got attention. So I paused a bit! There is confusion in this module due to mucked branches. They are somehow confusing and sources of all these issues.

Fixed in the v2.1.0. Also the code base between branches was synced.


paraview: Add version 5.10.1

There once had been a bucket for Paraview, but it got removed: e6b9494eba9a88674569601fba3973557e0f5ecc. However, I cannot figure out the reason for this deletion.
This new PR installs correctly. But there are two caveats:

How can we incorporate the ParaView Settings Files into the persist directory? There seem to be two files under %APPDATA%\ParaView, namely ParaView-UserSettings.json and ParaViewX.Y.ini. In this filename, X.Y are kind of dynamically named. Maybe we could somehow use$majorVersion.$minorVersion?

How can we add checkver and autoupdate? A machine-readable list of all versions can be found at https://www.paraview.org/files/listing.txt. But I cannot figure out how use this list in checkver.

[x] I have read the Contributing Guide.

/verify

There once had been a bucket for Paraview, but it got removed: https://github.com/ScoopInstaller/Extras/commit/e6b9494eba9a88674569601fba3973557e0f5ecc. However, I cannot figure out the reason for this deletion.

I think the author accidentally put it in the PR for Coppeliasim, so he deleted it, but he did not send another PR for Paraview.

How can we incorporate the ParaView Settings Files into the persist directory?

The config file is under %appdata%, we usually don't "persist" this in official buckets, unless it is necessary for the package to work properly (for example, if the app's update machanism conflicts with Scoop).
By the way, if you want to persist files like ParaViewX.Y.ini, you can copy it in pre_install and pre_uninstall scripts, such as:
"pre_install": "if (Test-Path \"$dir\\ParaView*.ini\") {",
"    ensure \"$persist_dir\"",
"    Copy-Item \"$dir\\ParaView*.ini\" \"$persist_dir\\\",
"}",
"pre_uninstall": "if (Test-Path \"$persist_dir\\ParaView*.ini\") { Copy-Item \"$persist_dir\\ParaView*.ini\" \"$dir\\\" }",

ensure is a Scoop helper function that creates a folder if it does not exist.

thank you @issaclin32 for your support!

/verify

/verify


Proposed App Architecture

Overview
(1) DataHub: Basically all the data from the exchange servers can be fed into something called a DataHub - where we store all relevant information any strategy would need about currency pairs. Indicators can either be stored here, or calculated upon a strategy's request (might affect efficiency).
(2) Strategies: The data from DataHub can be called by any strategy. Originally I was thinking about using Websockets to emit all the data, then have the strategies open up listeners based on what data they want. @BirchellE helped me realize that we can just call the all exchange objects since Node keeps everything live and updated.
(3) TradingHub: The strategies then send all their orders to our TradingHub, which contains exchange API wrappers/interfaces. The TradingHub then sends those requests out to the relevant exchanges.
Possible issues:

Standardizing exchange data to a universal format that can be used by any strategy.
Inefficiencies calculating trading indicators that aren't being used (might be fixable by having strategies calculate them instead).
Redundancies involved with creating API wrappers in TradingHub, when API framework might already be available in DataHub.

Overall, I think this architecture provides clean boundaries between our distinct components (modularity), and will allow us to add a backtesting feature or plug-in any exchange or strategy (extensibility) we or a user might need.

We need to break this into a client server model doe. A person's computer can only run constantly if on a desktop and even then it's not ideal. We ought to have the
What is the symbol in upper right of DataHub and TradingHub mean?
Don't name an object type "Exchange", too redundant in our naming scheme. Doesn't DataHub == Exchange here anyway?
OrderBook.currencyPair should be two JS enums variables for explicit-ness.
Are orderbooks ordered by time? Is a list the ideal structure?
Lol 30DayAvg let's not have a variable with a number first.

Redundancies involved with creating API wrappers in TradingHub, when API framework might already be available in DataHub.

Let me throw this idea at you. Because API calls have to be made with the least possible code "distance" so as to minimize errors at this level since it's a really common occurrence, we have the TradingHub cover all API needs and be a dependency within DataHub (and rename them differently cuz then they aren't that related horizontally anymore).
Now since Strategies and DataHub require different API calls (e.g. DataHub should never be able to make a "buy" call), the exports of TradingHub with all the API wrappers is a factory function that, given a JS enum (same link as above) exports an object with ONLY THE NEEDED CALLS. Obviously Strategies will have access to a larger portion (perhaps even a super-set) of DataHub's API calls.
Otherwise looks good, I can tell you put a lot of thought into this!

Sweet, thanks for the feedback. Addressing your points:

Which components do you think should be part of the server and client? Similar to what you said, a client should be able to run their strategies whether or not they're at their computer. Therefore, I'd say all the components the strategies need to run should be part of the server -- to me those components look like DataHub, Strategies, and TradingHub.
That's a UML symbol for a generic 'component'.
DataHub should hold all the exchanges. Each exchange will hold their currency pair order books and other information. Where would the redundancy occur?
Got it. Some exchanges return their pairs as "BTC_ETH". So we'd have to split on the '_'.
The Orderbook won't be like the Maps of bids and asks we have now. They will contain additional information.  Exchange --> [Orderbook, Orderbook, ...] --> {pair: "BTC_ETH, bids: [...], asks: [...], indicators, etc..}. But if this is still confusing then maybe we should change it.
Placeholder:) This [repo for technical analysis] (https://github.com/andredumas/techan.js/tree/master/src/indicator) has some examples.

Oo, that sounds like a very smooth solution -  I'll work that into the next iteration!

Okay yeah that makes sense.
Missing for Strategies box was why I asked.
Oh DataHub to Exchange is a 1-to-many. I just now got that.
Ah so each orderbook in an exchange is differentiated by currency pair. Then it seems like they should be stored in a js {} object and use the currency pairs as keys.

So most of my discussion has been with Adam (I'll try to do that in these comments from now on), but I'd like to say that the whole accessing subsets of data is definitely a great idea. That way there doesn't have to be one single data set that everything is accessing. I'm excited to start putting this plan into action!

Here's a second version that includes @Mingling94's suggestion:
I tried to balance showing the class hierarchy with TradingHub <-- DataHub, while making things look pretty

I don't think DataHub should extend TradingHub. Although it will use its methods, it should just hold an instance of TradingHub, whose purpose is completely different than DataHub's. How about we name them Exchanges and Wrappers? Similar to how we named Strategies?


Search field position issue Safari

fixed https://github.com/zurb/building-blocks/commit/56aca632b2367ffffd51caf6aa67a1fa021b5ac5


fix: update url to polyfill source

Yes, please fix this; super annoying that every source link doesn't work.

Bump @JakeChampion


Fix installation failures with special characters in path (fix #1129)

don't allow brackets (they confuse powershell) and remove some jdk issue with carets

Codecov Report

Merging #1268 into master will decrease coverage by 0.45%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1268      +/-   ##
==========================================
- Coverage   93.42%   92.97%   -0.46%
==========================================
  Files          43       43
  Lines        4624     4696      +72
  Branches      511      798     +287
==========================================
+ Hits         4320     4366      +46
- Misses        304      309       +5
- Partials        0       21      +21

Impacted Files
Coverage Δ

browser/model/jdk-install.js
93.22% <100%> (-0.92%)
:arrow_down:

browser/directives/pathValidator.js
100% <100%> (ø)
:arrow_up:

browser/model/devstudio-autoinstall.js
97.29% <0%> (-2.71%)
:arrow_down:

browser/model/helpers/downloader.js
96.13% <0%> (-1.66%)
:arrow_down:

browser/model/che.js
95% <0%> (-1.62%)
:arrow_down:

browser/pages/account/controller.js
98.61% <0%> (-1.39%)
:arrow_down:

browser/model/cdk.js
98.71% <0%> (-1.29%)
:arrow_down:

browser/model/devstudio-p2-zip.js
64.4% <0%> (-1.12%)
:arrow_down:

browser/model/devstudio-p2.js
81.66% <0%> (-1.1%)
:arrow_down:

browser/services/platform.js
97.38% <0%> (-1.05%)
:arrow_down:

... and 22 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 e8f2409...12c7057. Read the comment docs.


Question: PRAYGROUND_DIR represents the prayground library?

Hi @sketchbooks99!
Although your library works on OptiX 7.5, I am trying to do some upgradation of the CMakeLists.txt to see how the latest Optix-IR intermediate code works on it. One answer I would like to know, when you are using the PRAYGROUND_DIR, did you mean it is the prayground library folder, or the whole PRayGround project? I am a bit confused because both the library and project name is same. Here is the permalink of that line:
target_compile_definitions(${pg_target} PRIVATE PRAYGROUND_DIR="${PRAYGROUND_DIR}")
Same goes to here:
https://github.com/sketchbooks99/PRayGround/blob/4bf899d00d78e22ab905de3513c811d61c30e74c/prayground_config.h.in#L31
L31 to L36, the PRAYGROUND_ meant the entire project or the prayground library?
With advance thanks!

@bipul-mohanto
PRAYGROUND_DIR means the path to whole project, so it is PRayGround/. It's defined in PRayGround/CMakeLists.txt, and PRAYGROUND_PTX_DIR is also in it.
https://github.com/sketchbooks99/PRayGround/blob/4bf899d00d78e22ab905de3513c811d61c30e74c/CMakeLists.txt#L252

Thanks.


Preps site for June election

Fixes #270
Preview

Updates election date
Archives November local election page
Indicates there are no statewide questions on the homepage

@rentry Can you please take a look and see why it doesn't like the local.csv file?

I cleared the merge conflict with local.csv. Hopefully that fixes the problem.

@rentry Thank you!


added method to job to get PromiseValue using handle

I will try to submit an update very soon, thanks for the feedback.

Looks good. Please add tests and it should be ready for merge.

I have added unit tests and applied all the comments where possible. Please review.

updated @aozarov

Thank you. Nice work. few minor comments and should be ready for merge.

@aozarov applied comments.

Merged. Thank you!

https://www.youtube.com/watch?v=0wHYHMpHmSg

Hey @billy1380.
Thank you for your quick response and your willingness to help resolving this issue.
Indeed it feels like testUnfilledPromiseFromHandle is covering the second use-case I described (passing a filled-slot that will never trigger SlotFilled and, possibly, blocked Job[s]) but
the reason this test is passing is because pipeline is, shamefully, not trying to optimize on
task-queue calls. Basically when you pass an immediate as param pipeline will treat it
as a filled slot and register a SlotField task. However when you register a FutureValue
it does not register a SlotField task as it expect that task to be triggered when the
slot get fields.
A SlotField task is the task that checks all the Jobs (run barrier) associated with a Slot
and triggers a RunJob task for every Job that all its other Slots are filled.
If you change testUnfilledPromiseFromHandle and remove the extra immediate arguments
you should be able to reproduce this issue (and the test should never complete).
Here is a modified version that you can try:
public void testFilledPromiseFromHandle_noArgs() throws Exception {
PipelineService service = PipelineServiceFactory.newPipelineService();
String pipelineId = service.startNewPipeline(new FilledPromiseFromHandleParentJobNoArgs());
String value = waitForJobToComplete(pipelineId);
assertEquals("13", value);
}
@SuppressWarnings("serial")
private static class FilledPromiseFromHandleParentJobNoArgs extends Job0 {
@Override
public Value<String> run() throws Exception {
  PromisedValue<String> promise = newPromise();
  WaitForSetting wait = waitFor(futureCall(new FillPromiseJob(), immediate("3"),
      immediate(promise.getHandle())));

  return futureCall(new PromiseDelegateJobNoArgs(), immediate(promise.getHandle()), wait);
}

}
@SuppressWarnings("serial")
private static class PromiseDelegateJobNoArgs extends Job1<String, String> {
@Override
public Value run(String handle) {
PromisedValue promise = promise(handle);
return futureCall(new StringAdderJobNoArgs(), promise);
}
}
@SuppressWarnings("serial")
private static class StringAdderJobNoArgs extends Job1<String, String> {
@Override
public Value run(String value) {
return immediate("1" + value);
}
}
We could change PipelineManager to treat PromisedValue similar to immediate if it was field
but then we will face the second issue (that for me is highly reproducible using the w test) where we get a race-condition between reading
a state of a Slot and updating the Slot. This issue could have never happened before
as the PromisedValue Slot binding to a Job was fixed before it could ever be filled.
In my testUnfilledPromiseFromHandle I could clearly see this race conditions (via some printouts)
where StringAdderJob is being registered with a stale copy of the PromisedValue's slot (slot is
marked as yet unfilled) but then before it is saved it gets filled by FillPromiseJob.
BTW, the saving after the JobFilled could 'revert' the filled value (and effect other dependent jobs).
One way to deal with this issue is to treat PromiseValue's slot specially and transactionally verify
it wasn't change before saving it.

Hey @billy1380, I plan to revert this change but it should be easy to re-apply it once you have the fix.
Thank you.

Go for it... I have picked up the latest version before the rollback, ran both tests and can recreate the issue quite consistently.
Will keep you posted.


Clickhouse refused to work with multidimension array from sub select

assume uid is a string column in table page_visit, then:
SELECT arrayMap(uds -> uds,      (             SELECT groupArray(uids)             FROM              (                 SELECT groupArray(uid) AS uids                 FROM                  (                     SELECT *                     FROM page_visit                      LIMIT 10                 )                  GROUP BY date                 ORDER BY date ASC             )      ))
throw: Type inference of multidimensional arrays is not supported.
but this code works perfectly:
select arrayMap(x -> x, [[1,2], [3,4]])
And as I can see from this file:
https://github.com/yandex/ClickHouse/blob/master/dbms/src/DataTypes/FieldToDataType.cpp#L162
The problem, is that CH can't choose datatype for returned multidimensional array.
But why we can't just ensure, that all elements in inner arrays are of string type, and set string type to whole 2d array?
Or may be it could be solved other way.
It would be very useful to have opportunity to operate with multidimensional arrays from subqueries.

Fixed in latest version.


Question about enumerate classes

Snippets in dump.cs like below

try to find this class return null

use classes to filter can find this class

as the above show , the class's name is <>c  , so try to use the snippet as below

so how to correct this snippet ?

That class is a nested class, that's why it doesn't come with class or tryClass. The classes property, instead, shows every class, no matter if it's nested or not. This is not a decision of my own, but this is how the il2cpp internals work!
So, this how to do it:
AssembleCSharp.class("LitJson.JsonMapper").nested("<>c");


StakingRewards.sw

Type of change

New feature

Changes
Implements a Sway port of Synthetix's StakingRewards.sol

Superseded by #168


Additional interaction service types.

Issue number: N/A
Description of change
This has Interaction Service types required for SPI.

Codecov Report

Merging #946 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #946   +/-   ##
========================================
  Coverage    96.66%   96.66%
========================================
  Files          182      182
  Lines         5066     5066
  Branches       479      479
========================================
  Hits          4897     4897
  Misses         112      112
  Partials        57       57

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 6a74333...8f36038. Read the comment docs.

I'm not sure they look like services 🤔
E.g. Investment Enquiry – Transferred to LEP (IST use only), what does it mean?

That means project has been given to someone else - LEP in this case. Each new project will eventually have an interaction with one of those eight outcomes. Two of those count towards SPI.

Given limited time the goal was to reuse existing features to enable measurement of the SPI. I understand this is not ideal.


Migrate konflux tiers from host-operator to infra deployment

Implements: ASC-548

Looks good, but make sure that you use the latest version from the host-operator repo. It looks like that some of the TierTemplates are outdated and that your PR is missing the viewer SpaceRole https://github.com/codeready-toolchain/host-operator/blob/master/deploy/templates/nstemplatetiers/appstudio/spacerole_viewer.yaml

Thanks, good catch

Changing this PR to only deploy to staging so I can verify the new tiers are recognized by the host operator


Set up sign in and sign up

Sign in flow is now:
---
title: "Sign in flow: The happy path"
---

flowchart LR
    A[Sign up page] --> B[Enter email & name] --> C[Open link in email] --> D[Signed in]
    E[Sign in page] --> F[Enter email] --> C

Other changes:

Added nanoid for better non-guessable ids that don't leak internal information such as count of users
Added ability to extract authenticated user information in request handlers to fetch user related data (e.g. a user's surveys)
...and things I probably forgot to mention so please ask if there's any question

Not implemented:

Refresh session to keep users signed in as long as they are active and therefore reduce friction
Passkey support for less sign in steps/friction

I know this is a lot so we can go through it later if you want.

Forgot to mention: I've used Resend for sending emails because it was the easiest way. We could later add SMTP so users can use a private email account they have as an alternative.

Also missing: Sign out button to delete cookie and invalidate session


Feature/remove sheriff.config from lint programatically

Per your comment, 2nd bullet point: https://github.com/softarc-consulting/sheriff/issues/161#issuecomment-2481411348
This PR adds the ability for rules to have a filename inclusions or exclusions list set inside the createRule factory and filters the 'sheriff.config.ts' file from all the existing rules. This allows a future set of rules that can be filename/type-specific.
Of my two PRs this is the most testable, but both are written with consideration to solving https://github.com/softarc-consulting/sheriff/issues/165.

With hindsight, if you like this PR, I'd be happy to write more unit tests to cover all possible include/exclude configurations.


The release of reward model training code?

It's an excellent paper and has a significant contribution to the entire alignment community.
I wanted to inquire if you have any plans to open-source the training code for the reward model.

Thank you for your great supports to us!
Because reward model training involves more methods, this part will be explained in the second part of the technical report, thank you for your support and recognition!

Thank you for your response. May I know if there is a specific timeline for the release of the second part of the technical report?

Probably in August or September of this year, thanks for your interest.


Moment incorrectly validates times

var time = moment("13:22rrr", 'HH:mm','en', true)
time.isValid() //true
var time1 = moment('03:a5jojojo', 'HH:mm',true)
time1.isValid() //true
Moment considers these times as valid ones.

Are you running an old version of Moment? Both of those returns false for me

Yes, it seems I used old version, current one 2.19 seems works correctly.
But here it returns true: http://jsfiddle.net/sjdfhftv/

👍


Add tileserver layers as part of package import

Currently adding the tileserver layer definitions by way of Django fixture. Need to determine how to define them as part of the package

Layers added to map_layers/mapbox_spec_json/ directories. There is currently no way to define the order they are presented but will work to see if there is a small tweak to Arches core to allow for it.


Introduction to the React Challenges - better wording

Describe your problem and - if possible - how to reproduce it
I don't know if this is the place, but the text on the first page of React has some grammatical mistakes.
Also, my English is not perfect, so the text might be improved further.
the original text:
"React, popularized by Facebook, is a open-source JavaScript library for building user interfaces. With JSX, it is used to create components, handle state and props, utilize event listeners and certain life cycle methods to update data as it changes.
React combines HTML with JavaScript functionality to create its own markup language, JSX. This section will introduce you to all of these concepts and how to implement them for use with your own projects."
the improved text: - "an" instead of "a" and removed JSX in the first paragraph (not need as it is better explained in the second one).
"React, popularized by Facebook, is an open-source JavaScript library for building user interfaces. It is used to create components, handle state and props, utilize event listeners and certain life cycle methods to update data as it changes.
React combines HTML with JavaScript functionality to create its own markup language, JSX. This section will introduce you to all of these concepts and how to implement them for use with your own projects."
Add a Link to the page with the problem
https://learn.freecodecamp.org/front-end-libraries/react
Tell us about your browser and operating system

Browser Name: all
Browser Version: all
Operating System: all

If possible, add a screenshot here

Yeah it’s a grammar mistake, thanks for pointing it out. Any beginners are free to work on this in the curriculum repo.

I can make a pull request for this issue?

The explanation for all React lessons are overly complicated to explain simple things, and confusing.
There are a lot of words, but after reading it, I don't understand anything.
I am going to work through the tutorials in the original Documentation for React instead.
Maybe React documentation providers can collab with freeCodeCamp to put up their tutorial in this section.

Hi @Nirajn2311 ,
I'd be happy to help with this. Can you please guide me in the right direction? How do I edit the document in the the curriculum repo?

Hi @Nirajn2311
How do I go on the branch to begin editing the file?
It's my first time trying to do this here.
Thank you.

@KoniKodes have you set up a local copy of learn repo by following the steps given in contributing.md

@Nirajn2311 I cannot set up locally. My machine causes issues...

I just created a PR with the changes suggested by @KoniKodes.

Thank you @randelldawson
Is it possible to submit a pull request without having to yarn to my computer?
I thought that I used to be able to do that...

This issue was fixed before the move and shows up correct in the new curriculum
https://github.com/freeCodeCamp/learn/commit/5faf8f2f4bd66507ee791767bce983d3af2e3633

Close via PR https://github.com/freeCodeCamp/learn/pull/228


Add Cargo.lock to gitignore

Not entirely sure about the dropbox internal setup, but when I cd into this directory, to e.g. run one of the examples, it creates a Cargo.lock file. As divans is a library, such a file is generally not checked into git.


net/haproxy: unable to configure to listen on IPv6 address

When I try to configure a Public Service listening to an IPv6 address the following error is display on save:

Please provide a valid listen address, i.e. <IP_ADDRESS>:8080 or www.example.com:443. Port range as start-end, i.e. <IP_ADDRESS>:1220-1240

I guess this is just a validation error.
I tried both 2a01:....:1:443 as well as [2a01:.....:1]:443
OPNSense verion 18.1_1
os-haproxy plugin version: 2.4
UI URL: https://firewall.hartmaier.priv.at/ui/haproxy/#frontends

Looks like a IPv4-only regex to me: https://github.com/opnsense/plugins/blob/master/net/haproxy/src/opnsense/mvc/app/models/OPNsense/HAProxy/HAProxy.xml#L317

@abraxxa I can't reproduce this on OPNsense 18.1.6 (anymore). Is this still not working for you in the current release?

Closing this issue, because I can't reproduce it. Please report back how to reproduce this issue and provide a screenshot of the error.

Sorry for the delay, marking mails as important in the inbox isn't sufficient enough :(
I've checked it with 18.1.8 and it works now, thanks!

@abraxxa Thanks for the confirmation!

Clearly something is not working even with latest 19.7. I tried to add something like [2a02:8765:d000:123::1]:443 and it refuses to add it, displaying an error message which is useless for those trying to add an IPv6 address: Please provide a valid listen address, i.e. <IP_ADDRESS>:8080 or www.example.com:443. Port range as start-end, i.e. <IP_ADDRESS>:1220-1240.

Maybe your syntax is wrong?
https://forum.opnsense.org/index.php?topic=13695.msg64859;topicseen#msg64859

@ssbarnea Please try this patch:
root@opnsense # opnsense-patch -c plugins ec84327a

@fraenki I applied the patch, restarted all services and I did allow me add the address. Still, I think that binding to ipv6 does not work for the haproxy.
I tried running netstat -ln6 and I am unable to see anything on 80 or 443, even if I configured them as public services. I do see port 53 open and I know that unbound is working on ipv6.

Can you try sockstat -6

@ssbarnea I don't have an IPv6 test box, but binding on local IPV6 seems to work.
root@opnsense-test:~ # sockstat -6 | grep haproxy
www      haproxy    99791 4  tcp6   ::1:8088              *:*
www      haproxy    99791 12 tcp6   ::1:8080              *:*

For me binding haproxy to the Internet IPv6 address assigned via DHCP works as long as the static IPv6 address entered in the haproxy config is available and hasn't changed.

works as long as the static IPv6 address entered in the haproxy config is available and hasn't changed

True. HAProxy will even refuse to start if the configured listen address is unavailable (i.e. not configured on a local interface).

I can confirm that this patches worked. Sorry that it took so long to reply. I mention that for some reason I had to add an extra rule to the firewall but that is clearly unrelated to the patch.
Thanks for fixing that!

@ssbarnea Thanks for testing! The patch will be included in the upcoming release 2.19 of os-haproxy.


quickfilter dropdown list not shown

Hi all,
I have an issue trying to add a simple quickfilter to property.
I guess it cames after an update some weeks ago.
On a new fresh app (startsharp <IP_ADDRESS> installed but I have the same issue with older version), I try to add a simple quickfilter on CompanyName property in CustomerColumns.cs and the result is a simple input (instead of a select2). There is no arrow, no list and if a put a valid company name, the filter works fine.

Anyone allready had that before ? I guess Visual Studio.
Thank's.

My mistake ...
No LookupEditor added to the row property.


Multicast related issue with Fast-RTPS on QNX 7.0 [4096]

Compiled the FastRTPS examples and ran the fastrtps/examples/C++/HelloWorldExample/HelloWorldExample publisher/subscriber example. It works between two systems(one is QNX x86_64 VM and the other a ubuntu system). But it does not work when both publisher and subscriber executed in the same QNX x86_64 VM. The publisher just waits where it cannot find a match. Wrote few multicast test examples and do not see an issue with multicast in the x86_64 VM.
Any recommendations will be helpful.
Thanks

@anup-pem  you can check the close issue at https://github.com/eProsima/Fast-RTPS/issues/272
thanks to a kind man @jilinzhou, he help me solve the multicast communication problem.  You can try the patch to check if it solved your problem

@darbee @jilinzhou Thank you for pointing that out. It helped and it works.

@MiguelCompany Great

Closing this as #350 has been merged.


Add I2S Camera mode driver

Thank you for your contribution!
We appreciate the time and effort you've put into this pull request.
To help us review it efficiently, please ensure you've gone through the following checklist:
Submission Checklist 📝

[x] I have updated existing examples or added new ones (if applicable).
[ ] I have used cargo xtask fmt-packages command to ensure that all changed code is formatted correctly.
[x] My changes were added to the CHANGELOG.md in the proper section.
[ ] My changes are in accordance to the esp-rs API guidelines

Extra:

[x] I have read the CONTRIBUTING.md guide and followed its instructions.

Pull Request Details 📖
Description
Adds a driver for I2S Camera mode on the ESP32 only. (ESP32-S2 to come in a future PR)
Testing
Ran the example on a MaTouch_ESP32 3.2" TFT Touch with Camera.

Part of #1475

Anyone using this driver will almost certainly want to setup a separate clock signal with the LEDC driver. Should this be included in the example in the doc of the driver?
@MabezDev are you able to run the example?

@Dominaezzz Thanks for the PR! I'm trying to run the example on the TTGO-Camera board I have here, but without success so far. I've changed the required pins for this board, and I can communicate with camera, but the read back data is  garbage.
Probe successful!
Found PID of 0x26, and was expecting 0x26
Failed to find JPEG terminator
// followed by a stream of useless bytes

Any suggestions on debugging this?

What are the first 10 bytes that come out? I'm hoping the stream at least starts as a valid JPEG.
If it's truly garbage, is there any consistency at least?

Any suggestions on debugging this?

Sadly I don't have a good answer (besides pulling out a logic analyser).
In my experience with I2S camera mode, it tends to return garbage for the first image (or sometimes it's several images).
I haven't been able to prove whether this is from the camera, the peripheral or the firmware. I do know that esp32-camera have similar issues but they hide it by using framebuffers (if a framebuffer isn't a valid jpeg, it is discarded but this sacrifices latency).
I will just change the example to skip the first 10 images. i.e. Wait for the vsync pin to toggle 10 times.
Might try using the PCNT peripheral for this if the borrow checker lets me.

@MabezDev I've updated the example to skip the first 10 images. You should be able to reliably get an image every time you run the example now.
Also I've use the experimental coroutines feature for the example, I hope this is okay.
Side note: read_dma_circular now works properly after rebasing on the recent DMA pop() fixes!

I don't know why this PR failed. I need to capture raw data via I2S camera mode of ESP32-S2, and now I have to do it in ESP-IDF. However, this code helps me understand how to drive it (ESP32-S2 I2S registers differ from that of ESP32, though), because I'm not using any camera supported by esp32-camera component in IDF.

I don't know why this PR failed.

Was tired of rebasing at the time, I was juggling 4 to 5 PRs at a time and closed this to reduce my workload.
I also wasn't 100% happy with how the API came out tbh. Wasn't a big fan of the "different mode in a separate driver" idea here.
Feel free to resurrect the PR with my commits if you like, I currently don't have the motivation not personal need to drive this to the finish line. The DMA APIs themselves need a bit of improving before this driver can shine imo and that's where I'm at.
Btw I strongly recommend using the ESP32-S3 for cameras. Much better hardware than the S2 and base ESP32.

via I2S camera mode of ESP32-S2

Are you actually using Rust? And what devkit are you playing around with?

I had considered using Rust, but I encountered a problem of not having enough internal SRAM (#1084, #2001). Maybe memory management (including esp-alloc crate) needs to be improved. Now I just use ESP-IDF.


WindowStartupLocation is ignored on Linux

Setting WindowStartupLocation to CenterScreen or CenterOwner has no effect on Linux.

Is it not ignored but the child window returns a wrong width/height when the code to set the position runs. It also reports incorrect height/width on Opened event. You can see this yourself:
XAML
Width="200"
In Code
this.Opened += (s,e) => { Console.WriteLine(this.Width.ToString());
It reports in my case a width of 1200. If you a place a button on your children and re-run the WindowStartupLocation code it gets properly set in the middle of the owner window.
I expect that the draw engine on Linux still renders while on Windows it completes before window position is set.

Duplicates of https://github.com/AvaloniaUI/Avalonia/issues/3454 https://github.com/AvaloniaUI/Avalonia/issues/6159
I will keep one issue with more details


Please update documentation on native extensions

https://www.dartlang.org/articles/native-extensions-for-standalone-dart-vm/
Native Extensions for the Standalone Dart VM
Written by William Hesse
May 2012
This is a very helpful document, but needs to be updated.

Here is the source for the article: https://github.com/dart-lang/www.dartlang.org/blob/master/src/site/articles/native-extensions-for-standalone-dart-vm/index.markdown
We'll gladly take pull requests.
If the article is misleading or gives guidance that is no longer applicable, we should take it down until it's updated.
How relevant or correct is the article?

See also https://github.com/dart-lang/sdk/issues/23024

@sgmitrovic I started yesterday with a cmake base sample - maybe it helps: https://github.com/MikeMitterer/cmake-dart-extension-master but be aware - it's in a very early stage!!!!
@sethladd Do you know it there are plans to to support native extension for "pub global"? It would be even cooler if there would be a support for automatic compilation! cmake based... any plans?

Do you know it there are plans to to support native extension for "pub global"?

That would be so great, but no one is working on it now. I don't know if the pub team would take those patches, but it doesn't hurt to ask!

Closing: native extensions are deprecated are expected to be removed, see https://github.com/dart-lang/sdk/issues/45451.


Simplify Planner code by making it stateful and asynchronous

Is your feature request related to a problem? Please describe.
Currently, the Planner implementation is extremely over-complicated, because it was made to contort around the requirement that it get all of its information in "one shot" from the underlying view service. That choice was originally made with the intention of being able to directly reuse the Planner code in the web extension, or across an FFI boundary. But this turned out to be impractical anyways, and now we are left with the complexity for no benefit.
This complexity also has a cost. To implement fees, we had to do a baroque method of over-estimating gas costs, then refunding them in change notes, in order to try to ensure that no matter what spends were required to satisfy the intent, we would have over-spent by sufficiently much that we could still have enough gas. This is complex and unnecessary. We should instead just add notes, then adjust the gas, then continue iterating, all within the plan method.
Describe the solution you'd like

[ ] Eliminate plan_with_spendable_and_votable_notes
[ ] Fold all relevant code into plan
[ ] Simplify fee computations.

See https://github.com/penumbra-zone/web/pull/663 for reference on a new planning strategy

Superseded by https://github.com/penumbra-zone/penumbra/issues/4081


fix: ignores files with permalink=false

When using permalink=false in front matter, the outputPath is undefined.

That's perfect, thanks! 👍


Minor documentation formatting issue

URL for relevant page?
https://docs.influxdata.com/influxdb/v1.5/introduction/installation/
Description
Look for the following two lines in the page:
Set the environment variable INFLUXDB_CONFIG_PATH to the path of your configuration file and start the process.
For example:
There is a bullet point besides "For example:" which should be instead on the preceding line.

Resolved by 11068613f11b3f323ec689a776c3ebfc411eeeb0


Fix gulp-header with multiple files

Fixes https://github.com/tracker1/gulp-header/issues/18 and https://github.com/tracker1/gulp-header/issues/17 with tests for each.

@jacobboland can you look at merging this with #11.
The main differences are here
-    var template = gutil.template(headerText, extend({file : file}, data));
+    var template = headerText;
+    if (data !== false) {
+      template = gutil.template(headerText, extend({file : file}, data));
+    }
+

This way you are not forced to provide a data Object.
That way I can close my PR

@whitneyit That would be up to @tracker1. This is fixing a change that broke our build.

Valid point. I'll wait :smile:

sorry for breaking this! I believe this also fixes #22 and other issues caused in the prev release @tracker1

No worries @madhums . Would you mind verifying that this branch still works for source maps?

Merged and published... @madhums can you confirm that sourcemaps still works, and across multiple headers per #22 ?
Sorry, I just don't have time to do this right now.

yep! sourcemaps seems to be working fine!

@madhums  Cool, thanks for confirming... It looks like all the open issues are closed/confirmed now.

@whitneyit fyi, your requested change is now in place... passing an explicit false as the data parameter will disable template processing and inject the header as-is.

Awesome mate. Cheers
On Thu, 27 Aug 2015 12:55 Michael J. Ryan<EMAIL_ADDRESS>wrote:

@whitneyit https://github.com/whitneyit fyi, your requested change is
now in place... passing an explicit false as the data parameter will
disable template processing and inject the header as-is.
—
Reply to this email directly or view it on GitHub
https://github.com/tracker1/gulp-header/pull/19#issuecomment-135263804.


API Management Purge ServiceNotFound

Related to https://github.com/terraform-providers/terraform-provider-azurerm/issues/11061
When I try to purge a soft-delete of API Management service, it returns "ServiceNotFound" error.
Details in https://gist.github.com/yupwei68/fd9439e54c1f2d70b3d8318226dac766.

The service team has replied that this will be fixed and released in a couple of weeks.


Complement usages of await operator with invokations of ConfigureAwait(false)

According to MSDN Article Async/Await - Best Practices in Asynchronous Programming, all usages of await should be complemented with usage of ConfigureAwait(false) - everywhere where it's applicable.

Because we call into the user's methods within an IDialog-derived class, we did not want to non-deterministically lose any SynchronizationContext the user was expecting.
Because we don't synchronously wait on any tasks, we should not deadlock.
One remaining risk is excess latency for scheduling task continuations, subject to any throttling inherent in the ambient SynchronizationContext.
Of course, the caller of the library is free to drop the synchronization context before calling into the library if they feel it's a concern.

Thanks for the answer! Would you kindly also explain when ConfigureAwait(false) should not be used in one's bot code? When it may lead into issues? Specifically in context of using Bot Framework.

If one of your bot methods depended on the static HttpContext.Current, then using ConfigureAwait(false) beforehand might prevent you from getting the HttpContext from that static variable.
But yes, overall (e.g. in non-UI code), using ConfigureAwait is a good practice.

Thanks for the info!
Well, I think this is a pretty rare case, so losing performance just because of that is a quite big price.
One more thought about my initial suggestion about using .ConfigureAwait(false): maybe provide some sort of configuration option, which will control this behavior? If the option is enabled, apply .ConfigureAwait(false) . If it is disabled, behave as now. And make this option disabled by default (for safety). I know it's not simple to implement, for now it's just a general idea on how to resolve this issue.

Before calling into the builder library, you can try to drop the SynchronizationContext and that might accomplish the same goal.
I'm going to close this issue, but feel free to open a new issue for further discussion.


Consider giving detail about diff audit suggestions based on imports/wildcards/trusted entries

When adding new crates, cargo vet check (and probably cargo vet suggest) will find a path-of-least-effort to get the new crates certified. This can result in some slightly confusing suggestions since these commands look at all imports as well. It'd be nice if, when a cargo vet diff is suggested which is based on something other than a local audit, the certification reason for the base of the diff is displayed.
For example:
recommended audits for safe-to-deploy:
    Command                                           Publisher    Used By                                   Audit Size
    cargo vet diff webpki-roots 0.23.0 0.23.1         ctz          hyper-rustls                              4 files changed, 81 insertions(+), 5 deletions(-)
      NOTE: webpki-roots 0.23.0 is certified by imported audits from `bytecode-alliance`

While this might result in a bit more noise in the output, as an auditor it feels much better to have the full picture of the chain of audits and why a particular command is suggested. For consistency, it might also be a good idea to include the note for diffs from local audits, something like NOTE: webpki-roots 0.23.0 is certified by audits in the local supply chain.

This seems like a nice feature!


Turn on lint rules and up type-script compiler warnings

In order to ensure that during extension development we do not make simple coding mistakes (such as not waiting for promises, not handing undefined\null, etc.) I have turned on lint rules and typescript compiler options to help catch these issues.
While doing so, I obviously had to touch a lot (a LOT) of code to fix all the warnings. I also had to correct code that simply was not handling undefined\null properly.
I need everyone's help verifying the extension.

Gabe, this change appears to break the viewer for files that don't have file locations (such as our PE checker that only flags problems at the binary level).

I'm following your new code comments. Very useful as I'm ramping up.

/**

This file is really a namespace, not a class...

Refers to: src/CodeFlows.ts:16 in a7cd129. [](commit_id = a7cd129044e9533ce40ccfc3ac929200791603c9, deletion_comment = False)

}

This does not need to be a map, or created dynamically.

Refers to: src/FileConverter.ts:280 in a7cd129. [](commit_id = a7cd129044e9533ce40ccfc3ac929200791603c9, deletion_comment = False)

export class RunInfoFactory {

also a namespace

Refers to: src/RunInfoFactory.ts:13 in a7cd129. [](commit_id = a7cd129044e9533ce40ccfc3ac929200791603c9, deletion_comment = False)


update build name for personal build only

Jenkins throws MissingPropertyException if it cannot find
BUILD_USER_EMAIL. This happens when the build is started by timer.
Update build name for personal build only.
Signed-off-by: lanxia<EMAIL_ADDRESS>

MissingPropertyException case: https://ci.adoptopenjdk.net/view/Test_openjdk/job/openjdk11_hs_openjdktest_ppc64le_linux/79/console


Fix Add Color Uniform of CustomMaterial & PBRCustomMaterial

In CustomMaterial & PBRCustomMaterial,  AddUniform can not pass color value to shader.
https://www.babylonjs-playground.com/#9ZQL2G can reproduce the bug.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14558/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14558/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14558/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14558/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14558/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14558/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14558/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14558/merge#BCU1XR#0


Improve neighbor list behavior and safety with non-cubic unit cells.

This PR adds some extra functionality to neighbor lists and cell lists to make them safer, particularly with noncubic unit cells. For the moment, this code is limited to box matrices that are upper-triangular (which should always be possible to construct via a rotation).
Increased safety is enabled by adding a variety of error codes that get updated in the NeighborList type. These error codes are accommodated by a single uint8 with each of the eight bit fields being used to represent one error type.

Codecov Report

Merging #232 (6585d93) into main (37271cf) will decrease coverage by 0.26%.
The diff coverage is 80.74%.

@@            Coverage Diff             @@
##             main     #232      +/-   ##
==========================================
- Coverage   81.68%   81.41%   -0.27%
==========================================
  Files          22       22
  Lines        4771     4881     +110
==========================================
+ Hits         3897     3974      +77
- Misses        874      907      +33

Impacted Files
Coverage Δ

jax_md/space.py
89.03% <ø> (ø)

jax_md/quantity.py
85.43% <10.00%> (-3.10%)
:arrow_down:

jax_md/partition.py
88.73% <86.40%> (-0.92%)
:arrow_down:

jax_md/test_util.py
81.77% <0.00%> (-4.21%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


extension license issue

Seems the the license was not updated since its initial release and then was only copied.
https://marketplace.visualstudio.com/items/ms-windows-ai-studio.windows-ai-studio/license
currently says:

MICROSOFT LIVE SHARE EXTENSION FOR VISUAL STUDIO FAMILY OF PRODUCTS
These license terms are an agreement between you and Microsoft Corporation (or based on where you live, one of its affiliates). They apply to the software named above.

I therefore have to conclude that there is currently no license for the use of the extension at all :-/
And when you adjust this, please replace the restriction under "INSTALLATION AND USE RIGHTS", which currently reads

You may install and use any number of copies of the software to use solely with Microsoft Visual Studio family of products.

by something like

You may install and use any number of copies of the software to use solely with Microsoft Visual Studio Code or derivatives thereof.

[reasoning: it is already documented to need vscode - not any other Visual Studio products - and there is otherwise no option to run it with self-built versions of vscode]

@GitMensch Thank you for the feedback, and we are working with Licensing and legal team to update it.

@GitMensch Thank you for your feedback, we've been working with our licensing and legal team to update it.

This should have been replaced with updated license, so closing this.


Unable to export AKS nodepool

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Related command
works: az group export --name $RG --resource-ids /subscriptions/$SUB/resourcegroups/$RG/providers/Microsoft.ContainerService/managedClusters/$CLUSTER
does not work: az group export --name $RG --resource-ids /subscriptions/$SUB/resourcegroups/$RG/providers/Microsoft.ContainerService/managedClusters/$CLUSTER/agentPools/$POOL
Describe the bug
az group export works for the AKS cluster but fails for the agent pool. It returns an empty ARM template
{
  "$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
  "contentVersion": "<IP_ADDRESS>",
  "parameters": {},
  "resources": [],
  "variables": {}
}

To Reproduce
See above
Expected behavior
az group export shows the config for a single nodepool.
Environment summary
xplat
Additional context
n/a

@zhoxing-ms for awareness

In fact, export AKS node pool needs the support of REST service resource-groups/export-template, so it needs the help from service team.

Non customer-reported issue. So adding Service team to look into this.
@josephkwchan, @jennyhunter-msft. Could you please look into this once you get a chance ? Thanks in advance.


Env vars?

Is there a way to specify env vars? Something like...
~w(pg_dump -h #{host} -p #{port} -U #{user} #{format} --schema #{schema} #{db})
|> Exile.stream!(env: %{PGSSLMODE: "allow"})

Thanks!

Hi @cjbottaro. Yes, env vars are supported, and it is exactly like you mentioned.
output =
  Exile.stream!(~w(printenv FOO), env: %{"FOO" => "bar"})
  |> Enum.to_list()
  |> IO.iodata_to_binary()

assert output == "bar\n"

All parameters which are accepted by Exile.Process.start_link/2 can be passed. I guess the stream documentation needs to clarify this better.

Closing this since env var is already supported. Let me know if you are facing any issue with that.
Thank you!


Temporary workaround for RMM initial pool size bug

Signed-off-by: Jason Lowe<EMAIL_ADDRESS>This works around the RMM bug in https://github.com/rapidsai/rmm/issues/471 by specifying only 50% of the memory should be initially used.

build

build


Using -zl is broken in 0.4.7dev

The MTF change broke the -zl parameter. In this part of commit 74a431, it can be see why:
before: if (levels_sorted[index] == -1) break;
after: if (comp_mem_level_count[index] == -1) break;
Before the change, the levels were sorted, so those with -1 (the disabled ones that weren't given in the -zl parameter) were at the end of the list and we could break out of the loop because no other levels would follow. After the change, however, this is not true anymore.
To reproduce the bug, any file that gives -zl suggestions can be used. If the suggested parameters is different to "-zl11", using it will skip any streams.
@MarcioPais : I'm not sure about the correct solution, replacing the break with a continue seems to be a good workaround at a first glance, but it also seems like the disabled levels should be removed from the MTF list completely.

@MarcioPais: Fixed this with the solution above (replacing break with continue), it works and I think the overhead of up to 80 continue calls is negligible, so I didn't bother with removing the levels from the MTF list.

I completely forgot about the -zl parameter, sorry. Your solution seems like a good workaround, and is simpler than modifying the zlib class to build a list with just the enabled zlib parameter combinations.


[AutoPR] servicefabricmesh/resource-manager

Created to accumulate context: servicefabricmesh/resource-manager

Codecov Report

Merging #2947 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2947      +/-   ##
==========================================
+ Coverage   56.91%   56.91%   +<.01%
==========================================
  Files        7774     7774
  Lines      159828   159837       +9
==========================================
+ Hits        90960    90970      +10
+ Misses      68868    68867       -1

Impacted Files
Coverage Δ

azure-common/azure/profiles/__init__.py
90% <0%> (+4.63%)
:arrow_up:

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 5081613...43785cc. Read the comment docs.


Can't delete a sign under glosses in Django

There's a sign without an Annotation ID gloss somehow peeped in in the ASL Signbank (shows up at top of search results). If I try to delete it in the Gloss table in the Django admin, I get an error message.

@Woseseltops , if this is related to a migration of the db that @ErwinKomen was already preparing, it would perhaps be a good timing to tackle #139 and #137 now?

Fixed, I can remove users and glosses again. Was indeed related to a database migration by @ErwinKomen that was not applied to the website databases yet.


[DDW-1179] Update LedgerJS to 6.0.0 version

This PR introduces @cardano-foundation/ledgerjs-hw-app-cardano library update to version 6.0.0
Jira Ticket
Useful links:
https://input-output-rnd.slack.com/archives/G010N9UFDNH/p1669826305137549
NOTE: NO NOT MERGE because it is blocked by cardano-node update ( we can't test it )
Thread: https://input-output-rnd.slack.com/archives/CDM8B0AN9/p1676874307251919
Task we are waiting: https://github.com/input-output-hk/cardano-node/pull/4902
Todos

[x] Update LedgerJs to v6.0.0-rc2
[x] Update LedgerJs to official release v6.0.0
[x] Update @ledgerhq/hw-transport-node-hid to the latest stable version

Screenshots
No
Testing Checklist

Slack QA thread

In the scope of this PR we should test voting breaking changes. Also, it would be nice to check regular transactions and delegation. LEDGER ONLY!

[ ] Test with current / official Ledger Cardano App
[ ] Update Ledger Cardano App to version v6.0.2. - This is an experimental app and you can install it using Ledger Live ( with an enabled Experimental flag under provider number 4)

Review Checklist
Basics

[ ] PR assigned to the PR author(s)
[ ] input-output-hk/daedalus-dev and input-output-hk/daedalus-qa assigned as PR reviewers
[ ] If there are UI changes, Alexander Rukin assigned as an additional reviewer
[ ] All visual regression testing has been reviewed (assign run Chromatic label to PR to trigger the run)
[ ] PR has appropriate labels (release-vNext, feature/bug/chore, WIP)
[ ] PR link is added to a Jira ticket, ticket moved to In Review
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes
[ ] PR contains screenshots (in case of UI changes)
[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with typescript types
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Update Slack QA thread by marking it with a green checkmark

Undocumented breaking change with str_to_path util export explained here: cardano-foundation/ledgerjs-hw-app-cardano#36

Thank you soo much @rhyslbw <3


Add method and headers in request fetch

Add method and auth headers in request fetch

Thanks! Will make a little additions before patch.

Thanks! This is great!! I'll be waiting anxiously for the new version!


Java, Nodejs, Python and .NET

Summary
Added Java, Nodejs, Python and .NET observability packs with dashboards and alerts.

Hey!
Sorry for the double commit, all changes was made in the first one "feat(packs)"!

several checks are failing on this PR...

@polfliet I fixed the 2 alerts now. But before I commit, what about the 'Ensure images are valid' job that fails?

@EmilHammarstrand It looks like you are eceeded in the limit of 6 on image files  You can see the failed check here.
Please check the following directories:
{
  folder: '/home/runner/work/newrelic-observability-packs/newrelic-observability-packs/packs/dotnet/dotnet',
  imageCount: 7
}
{
  folder: '/home/runner/work/newrelic-observability-packs/newrelic-observability-packs/packs/java/java',
  imageCount: 7
}
{
  folder: '/home/runner/work/newrelic-observability-packs/newrelic-observability-packs/packs/python/python',
  imageCount: 7
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Error: Process completed with exit code 1.

@jpvajda Yeah. Where do you want me to put the images for it to work?

@aswanson-nr suggested to move the images to the images/ folder instead of the dashboards/ folder, so that looks ok to me @EmilHammarstrand.
@aswanson-nr @jpvajda do we need to update the image check to only look at the dashboards/ folder then and ignore the images/ folder?

Yes, I made that /images folder and put the images in there in a older commit.

There shouldn't be an /images folder for quickstarts. Images for dashboards should go into /dashboards.  icons or logo files should go in the root quickstart directory like /apache @polfliet  @EmilHammarstrand
So just remove the image directories and ensure image files are in the properly location and you haven't exceeded the max limit of 6 dashboard images.
I'll check our documentation to see if this needs to be more clear as well.

There shouldn't be an /images folder for quickstarts. Images for dashboards should go into /dashboards. icons or logo files should go in the root quickstart directory like /apache @polfliet @EmilHammarstrand
So just remove the image directories and ensure image files are in the proper location and you haven't exceeded the max limit of 6 dashboard images.
I'll check our documentation to see if this needs to be more clear as well.

Emil was following @aswanson-nr's recommendations. Alec, can you chime in?

Hi Emil, given that the launch date is getting close, our team is struggling to find time to fix/improve this. Can you please remove 1 image from those dashboards?

I removed one dashboard screenshot. Now it's 6 images in those packs!

Thank you @EmilHammarstrand I just fixed a couple of schema validations that were failing (I think the schema might be too strict). I believe this is good to merge now @aswanson-nr @jpvajda


Use AUTH_USER_MODEL

Waitinglist should work with custom user models
Changed instances of "django.contrib.auth.models import User" to "User = getattr(settings, 'AUTH_USER_MODEL', 'auth.User')"

@paltman Can this be merged?

Thank you @paltman!


[BUG]

Aged Version: 3.1.1

All modded wood types (cypress, maple, aspen, willow, olive, wisteria, sugi, fir, cherry, saxaul, mahogany, larch, cedar, coconut, palo verde, and ghaf) break without the need for a tool and drop full logs, while it seems like all vanilla wood types need a tool in order to get the respective logs to drop.
I don't think this is intentional

Yep, this is an earlystage incompat.
Wont get fixed for 1.20.1 sadly tho.


bump: update dependencies

Bumping up to notation-go v1.0.1.
Note: The build fails due to E2E test. Therefore, this PR depends on https://github.com/notaryproject/notation/pull/810

Looks like we need to update E2E testing?

Looks like we need to update E2E testing?

Yes, as I mentioned, this is fixed in PR https://github.com/notaryproject/notation/pull/810

Codecov Report

Merging #815 (a787e0d) into main (bb2669b) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@           Coverage Diff           @@
##             main     #815   +/-   ##
=======================================
  Coverage   64.05%   64.05%
=======================================
  Files          40       40
  Lines        2323     2323
=======================================
  Hits         1488     1488
  Misses        709      709
  Partials      126      126

:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Unable to copy field names after aggregation

When query like select sum(*) from foo is executed, the field names are having added sum_ prefix by default. So, the resulting fields might be sum_bar & sum_baz. AS is supported, but only allows to change one prefix to another, not to remove the prefix. I.e. sum(*) as summary will name fields summary_bar & summary_baz. The problem is: it does not allow lack of prefix. The best you can do is to sum(*) as "", but it will still produce sum_bar & sum_baz as if AS would not be there.
Why this is a problem? When having a number of continuous queries to gradually degrade the precision, the prefixes do accumulate. So, if you have one CQ to aggregate last hour by 1 minute and another CQ to aggregate last day by 1 hour, you will end up with sum_sum_bar or sum_sum_baz.
That requires your application to be very sensitive throughout its code to the aggregation settings, as introducing new aggregation period will add another instance of the prefix into the fields, potentially breaking unprepared applications. Alternative - you need to specify each field manually, which kind of breaks the schemaless approach.
Expected behavior: being able to get bar and baz plain original names after CQ is executed.
System info
InfluxDB: 1.3.0, from Docker Hub.

Since there is :MEASUREMENT backref possible when operating on numerous measurements, maybe some sum(*) AS :FIELD would do? Or at least AS "" to actually drop the prefix (& the _ join string!) - but this will alter existing semantics, so probably is not advisable.

Anyone? :(

Seems like a reasonable feature request to me. I can't speak to how it compares to our other current priorities.

I think this may be the same as #7332. If I'm wrong, please comment and I'll reopen.


fix(curriculum): Added 22 missing solutions to challenges in the Regular Expressions section

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Added working solutions to the following 22 challenge files in the Regular Expressions section.  Tested all of them locally and passed when running npm run test:curriculum.

curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/extract-matches.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/find-characters-with-lazy-matching.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/find-one-or-more-criminals-in-a-hunt.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/ignore-case-while-matching.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-a-literal-string-with-different-possibilities.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-all-letters-and-numbers.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-all-non-numbers.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-all-numbers.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-anything-with-wildcard-period.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-beginning-string-patterns.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-ending-string-patterns.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-everything-but-letters-and-numbers.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-letters-of-the-alphabet.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-literal-strings.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-non-whitespace-characters.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-numbers-and-letters-of-the-alphabet.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-single-character-with-multiple-possibilities.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/match-single-characters-not-specified.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/specify-only-the-lower-number-of-matches.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/specify-upper-and-lower-number-of-matches.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/use-capture-groups-to-search-and-replace.english.md
curriculum/challenges/english/02-javascript-algorithms-and-data-structures/regular-expressions/using-the-test-method.english.md

@RandellDawson these all pass curriculum and app tests.
Some of the app tests will pass without being strictly the same as the solution, an extra g flag for instance. Is that a bit too pedantic? I have filenames if you want a quick look.

I have filenames if you want a quick look.

@nikrb Let me know if you see any tests which will incorrectly pass or fail based on any of the solutions I have added.

@RandellDawson your solutions work great so this pr looks good to go.

@RandellDawson Thank you for your contribution to the page! :+1:
We're happy to accept these changes, and look forward to future contributions. :memo:


stop charging failed cards

Closes #3531.

I think the fix for this is actually pretty trivial, simply constraining the participants query in payday.sql with AND error = ''. We should have a test, though, and I don't see that we demonstrate the working case first yet.

@rohitpaulk Looks like you accidentally committed gratipay.{js,css}, ya?

Ah, right. Force pushing that off...

I'm good to merge (once Travis is green) if you're okay with 8a6f517088d5df499b87a0a7a26292f2336bf1c9.

Interesting, I'm not seeing that locally.

/me runs a full make test

Hrm ... I now get the error Travis gets, plus a bonus error. O.O

___________________________ TestNewTeams.test_migrate_tips_to_subscriptions ____________________________
Traceback (most recent call last):
  File "/Users/whit537/personal/gratipay/gratipay.com/tests/py/test_teams.py", line 133, in test_migrate_tips_to_subscriptions
    assert subscriptions[0].subscriber == 'alice'
AssertionError: assert 'bob' == 'alice'
  - bob
  + alice

Gah, the Travis error is because my commit was bad, one sec ...


Lib updates [no sqlalchemy upgrade]

Thanks for the interest!
This has many changes on the same PR, it would be better to have one PR per specific change. Some of these things are already covered by other PRs, and some I wouldn't take yet, for example removing support for Python 3.7.
So I'll pass on this one, but thanks for the effort! 🍰
If there are some of these changes that you still consider important, feel free to create new independent small PRs for each change. Thank you!


flutter driver: built-in library 'dart:ui' is not available on the stand-alone VM

$ flutter drive --target=./test_driver/login.dart
Using device Android SDK built for x86 64.
Starting application: ./test_driver/login.dart
Initializing gradle...                                0.6s
Resolving dependencies...                             9.8s
Installing build/app/outputs/apk/app.apk...           4.5s
Running 'gradlew assembleDebug'...                   22.8s
Built build/app/outputs/apk/app-debug.apk (28.1MB).
Installing build/app/outputs/apk/app.apk...           4.3s
I/flutter ( 4881): Diagnostic server listening on http://<IP_ADDRESS>:54279/
I/flutter ( 4924): Diagnostic server listening on http://<IP_ADDRESS>:43723/
I/flutter ( 4924): Observatory listening on http://<IP_ADDRESS>:45147/
The built-in library 'dart:ui' is not available on the stand-alone VM.
'package:flutter_test/src/binding.dart': error: line 7 pos 1: library handler failed
import 'dart:ui' as ui;
^
Stopping application instance.
Driver tests failed: 254

$ flutter doctor
[✓] Flutter (on Linux, locale en_US.UTF-8, channel master)
    • Flutter at /home/reyer/reyer/flutter/master
    • Framework revision 0a85db2958 (13 days ago), 2017-10-06 07:13:27 +0300
    • Engine revision 459f722b86
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.2)
    • Android SDK at /home/reyer/Android/Sdk
    • Platform android-26, build-tools 26.0.2
    • ANDROID_HOME = /home/reyer/Android/Sdk
    • Java binary at: /home/reyer/.local/share/umake/android/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] Android Studio (version 2.3)
    • Android Studio at /home/reyer/.local/share/umake/android/android-studio
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2)
    • Flutter plugin version 18.1
    • Dart plugin version 172.4155.35

[✓] Connected devices
    • Android SDK built for x86 64 • emulator-5554 • android-x64 • Android 6.0 (API 23) (emulator)

package:flutter_test/src/binding.dart': error: line 7 pos 1: library handler failed import 'dart:ui' as ui;

You should enable or reinstall flutter plugin .

I had this problem too. For me, the resolution was to follow the advice in http://cogitas.net/write-integration-test-flutter/ and remove any Key objects from my page objects (or test sreens, or ui proxies, or whatever you call them). So instead of
final _wordArea = find.byValueKey(wordKey); //A Key

I used

final _wordArea = find.byValueKey('word');
This worked, but as the author of the above article comments, it's a poor solution.


Analyze the numerical basis for issue #37 and issue #44

Do a pencil and graph paper and spreadsheet analysis of one minute of the actual ISS orbit versus a single linear step of the current algorithm to understand the nature of the error in the approximation.

One thing that intuitively seems problematic is that the velocity of an orbiting body at the end of each step won't be in the same direction as the tangent to the curve at that point.

Closed by commit 174ae0e0016db769ec3d57d17e419c1a84f51ab9
Analysed a single step of one minute and the approximated orbit is too high and far, too fast, and with the velocity at the wrong angle. See spreadsheet for details.


Almost consumed ALL CPU

On one Intel i9-14900KF CPU, when using the default parameters, all CPU resources are above 50%. Is there any way to run the code on a single CPU with real-time performance?
Algorithm configuration.
sigma: 0.35
rMap: 2
rNew: 0.5
convergenceTol: 1e-3
minSensorRange: 5
Hardware configuration.
maxSensorRange: 120

Hi,
A few notes,

The real-time applicability relies heavily on having the ability to thread the objective function.
The testing in the paper is performed with an Intel Core i7-10700K CPU @ 3.80GHz × 16.
I'm currently using Intel's TBB library to parallelize the objective function. This automatically allocates available threads to perform the task. I have also tested using OpenMP (see the plot below) and achieved real-time performance with a minimum of four threads.
A simple example of using OpenMP for the objective function is provided at main/archive/usingOpenMP for an older version of the code but should be easy to see the main difference I have highlighted below. To build this, you will need to include "omp.h" and link OpenMP in the CMakeLists as shown in the main/archive/usingOpenMP/CMakeLists.txt file.

Current objective function parallelization using TBB (ObjectiveFunction.cpp lines 31 to 56):
  // Calculate the hypothesis reward.
  // Loop through each transformed target scan and find the closest point
  // in the source scan.
  tbb::parallel_for(
  tbb::blocked_range<int>(0, scan_.size()),
  [&](tbb::blocked_range<int> r)
  {
      for (unsigned int i = r.begin(); i < r.end(); i++)
      {
          uint32_t retIndex;
          double outDistSqr;
          double query_pt[3];
          Eigen::Vector4d ptTf = hypothesis*scan_[i];
          query_pt[0] = ptTf(0);
          query_pt[1] = ptTf(1);
          query_pt[2] = ptTf(2);

          // Search for the closest point.
          subMapKdTree_->knnSearch(&query_pt[0], numResults, &retIndex,
                                   &outDistSqr);

          // Score the difference between the current and nearest point,
          // Equation 5.
          scores[i] = exp(-outDistSqr * sigma_);
      }
  });

Using OpenMP you can se the number of threads and the section would be:
  // Calculate the hypothesis reward.
  // Loop through each transformed target scan and find the closest point
  // in the source scan.
  omp_set_num_threads(10); // Manually set the number of threads.
  #pragma omp parallel for
  for (unsigned int i = 0; i < scan_.size(); i++)
  {
          uint32_t retIndex;
          double outDistSqr;
          double query_pt[3];
          Eigen::Vector4d ptTf = hypothesis*scan_[i];
          query_pt[0] = ptTf(0);
          query_pt[1] = ptTf(1);
          query_pt[2] = ptTf(2);

          // Search for the closest point.
          subMapKdTree_->knnSearch(&query_pt[0], numResults, &retIndex,
                                     &outDistSqr);

          // Score the difference between the current and nearest point,
          // Equation 5.
          scores[i] = exp(-outDistSqr * sigma_);
  }

Please let me know if there are any questions.
Thanks,
Vedant


feat(ai-promp-guard) add config option to prompt guard for scanning "system" and "assistant" messages

Summary
This PR also fixes an issue when allow_all_conversation_history
is set to false, the first user request is selected instead of the last
one.
Unit tests are refactored to so that it's cleaner and has wider coverage.
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
AG-58

@jschmid1 could you take another look on this? Thanks!

If you add a test for backwards compat verification, we can move this forward. In the meanwhile, I'll request another reviewer @Kong/gateway-moderators


Documentation: Add blog post on performance evaluation with ebpf

Need to add to skydive.network (or a different public facing location) documentation based on the information that @andrekassis and @safchain created that is currently available here:
https://github.com/skydive-project/skydive.network/blob/8197fe853c6020b33a8b962ad3a14ec5bec64073/documentation/skydive-with-ebpf.md

Just rebased the blog post and merged it


Auto-completing React Native tags

VSCode Version: 1.4.0
OS Version: OS X Yosemite

I've already opened an issue in vscode-react-native repo but I think it's related to vscode itself.
Microsoft/vscode-react-native#247
When I need to insert a component or any tag, I have to write both opening and closing tags. Auto completion on these could be great to have.
Similar behavior is available on React with double TAB.
Example behavior on ; https://atom.io/packages/react
Thanks

The completion framework allows you to do that. @aeschli is this something the html-language should do? be configured to do?

That would be great!
Demo videos atom-react

will be really helpful

Closing as duplicate of #34307


Add a means to configure

Create a configuration dialog and a way to access it. Maybe this is in app settings or maybe it’s a screen in the app.
Enhancements for time zone setting depend on this.

If the configuration is via the app, perhaps access it by poking the center of the dial. This solves the where to put the button problem with no button. It requires reading the docs or accidental discovery however, in order to find it.


7.0.3 doesn't seem to actually be released

It's called out in the Changelog as having been released July 7, but it isn't on Rubygems...

@justin808 @ahangarha

Thanks @swrobel
V7.0.3 just released.


Resolve Sass urls based on original file

Webpack has the resolve-url-loader to make the Sass urls resolve from the original file instead of resolving from the file that imported the original file.
The Sass Plugin currently has macros, that can be helpful to deal with this problem, but it's a FuseBox only solution, and doesn't work with the autocomplete from IDEs.

@ypconstante you can add additions paths to make it work without the importer

I am not sure we can help with your IDE.  Use this option