Add support for .tar.xz format

Zephyr SDK version 0.16.0 (and newer) uses .tar.xz : https://github.com/zephyrproject-rtos/sdk-ng/releases
Would be great if this format is supported.

Please test if it works with tag test/xz.

I got this:
2023-04-27T05:46:48.4377310Z Requested labels: ubuntu-latest 2023-04-27T05:46:48.4377615Z Job defined at: GCX-EMBEDDED/gcx-modbus-com-system-firmware/.github/workflows/essentials.yml@refs/heads/update-ci-dependencies 2023-04-27T05:46:48.4377757Z Waiting for a runner to pick up this job... 2023-04-27T05:46:48.7456009Z Job is waiting for a hosted runner to come online. 2023-04-27T05:46:51.8074537Z Job is about to start running on the hosted runner: GitHub Actions 2 (hosted) 2023-04-27T05:46:55.0737694Z Current runner version: '2.303.0' 2023-04-27T05:46:55.0769606Z ##[group]Operating System 2023-04-27T05:46:55.0770302Z Ubuntu 2023-04-27T05:46:55.0771111Z 22.04.2 2023-04-27T05:46:55.0771598Z LTS 2023-04-27T05:46:55.0772020Z ##[endgroup] 2023-04-27T05:46:55.0772437Z ##[group]Runner Image 2023-04-27T05:46:55.0772919Z Image: ubuntu-22.04 2023-04-27T05:46:55.0773501Z Version: 20230426.1 2023-04-27T05:46:55.0774158Z Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230426.1/images/linux/Ubuntu2204-Readme.md 2023-04-27T05:46:55.0774969Z Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230426.1 2023-04-27T05:46:55.0775542Z ##[endgroup] 2023-04-27T05:46:55.0776100Z ##[group]Runner Image Provisioner 2023-04-27T05:46:55.0776555Z <IP_ADDRESS> 2023-04-27T05:46:55.0776946Z ##[endgroup] 2023-04-27T05:46:55.0778257Z ##[group]GITHUB_TOKEN Permissions 2023-04-27T05:46:55.0779098Z Actions: write 2023-04-27T05:46:55.0779508Z Checks: write 2023-04-27T05:46:55.0780239Z Contents: write 2023-04-27T05:46:55.0780840Z Deployments: write 2023-04-27T05:46:55.0781374Z Discussions: write 2023-04-27T05:46:55.0781787Z Issues: write 2023-04-27T05:46:55.0782188Z Metadata: read 2023-04-27T05:46:55.0782608Z Packages: write 2023-04-27T05:46:55.0783107Z Pages: write 2023-04-27T05:46:55.0783573Z PullRequests: write 2023-04-27T05:46:55.0784044Z RepositoryProjects: write 2023-04-27T05:46:55.0784555Z SecurityEvents: write 2023-04-27T05:46:55.0784984Z Statuses: write 2023-04-27T05:46:55.0785459Z ##[endgroup] 2023-04-27T05:46:55.0789828Z Secret source: Actions 2023-04-27T05:46:55.0790592Z Prepare workflow directory 2023-04-27T05:46:55.1854011Z Prepare all required actions 2023-04-27T05:46:55.2090852Z Getting action download info 2023-04-27T05:46:55.5465899Z Download action repository 'grandcentrix/actions-ssh2https@v1' (SHA:6fec85fda82c9971ed8976a646c40f35b76b4f13) 2023-04-27T05:46:56.0700261Z Download action repository 'grandcentrix/actions-zephyr-sdk@test/xz' (SHA:035feeb1e2c5c68e29dec6097025e0dc5168ec7b) 2023-04-27T05:46:56.5786100Z Download action repository 'actions/cache@v3' (SHA:88522ab9f39a2ea568f7027eddc7d8d8bc9d59c8) 2023-04-27T05:46:57.2027372Z Download action repository 'actions/upload-artifact@v3' (SHA:0b7f8abb1508181956e8e162db84b466c27e18ce) 2023-04-27T05:46:57.6529005Z Download action repository<EMAIL_ADDRESS>(SHA:4209421db54c8764d8932070ffd0f81715a629bf) 2023-04-27T05:46:58.2544922Z ##[error]System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter ''using: node18' is not supported, use 'docker', 'node12' or 'node16' instead.')    at GitHub.Runner.Worker.ActionManifestManager.ConvertRuns(IExecutionContext executionContext, TemplateContext templateContext, TemplateToken inputsToken, String fileRelativePath, MappingToken outputs)    at GitHub.Runner.Worker.ActionManifestManager.Load(IExecutionContext executionContext, String manifestFile) 2023-04-27T05:46:58.2581098Z ##[error]Fail to load grandcentrix/actions-zephyr-sdk/test/xz/action.yml

I got this:
2023-04-27T05:46:48.4377310Z Requested labels: ubuntu-latest
2023-04-27T05:46:48.4377615Z Job defined at: GCX-EMBEDDED/gcx-modbus-com-system-firmware/.github/workflows/essentials.yml@refs/heads/update-ci-dependencies
2023-04-27T05:46:48.4377757Z Waiting for a runner to pick up this job...
2023-04-27T05:46:48.7456009Z Job is waiting for a hosted runner to come online.
2023-04-27T05:46:51.8074537Z Job is about to start running on the hosted runner: GitHub Actions 2 (hosted)
2023-04-27T05:46:55.0737694Z Current runner version: '2.303.0'
2023-04-27T05:46:55.0769606Z ##[group]Operating System
2023-04-27T05:46:55.0770302Z Ubuntu
2023-04-27T05:46:55.0771111Z 22.04.2
2023-04-27T05:46:55.0771598Z LTS
2023-04-27T05:46:55.0772020Z ##[endgroup]
2023-04-27T05:46:55.0772437Z ##[group]Runner Image
2023-04-27T05:46:55.0772919Z Image: ubuntu-22.04
2023-04-27T05:46:55.0773501Z Version: 20230426.1
2023-04-27T05:46:55.0774158Z Included Software: https://github.com/actions/runner-images/blob/ubuntu22/20230426.1/images/linux/Ubuntu2204-Readme.md
2023-04-27T05:46:55.0774969Z Image Release: https://github.com/actions/runner-images/releases/tag/ubuntu22%2F20230426.1
2023-04-27T05:46:55.0775542Z ##[endgroup]
2023-04-27T05:46:55.0776100Z ##[group]Runner Image Provisioner
2023-04-27T05:46:55.0776555Z <IP_ADDRESS>
2023-04-27T05:46:55.0776946Z ##[endgroup]
2023-04-27T05:46:55.0778257Z ##[group]GITHUB_TOKEN Permissions
2023-04-27T05:46:55.0779098Z Actions: write
2023-04-27T05:46:55.0779508Z Checks: write
2023-04-27T05:46:55.0780239Z Contents: write
2023-04-27T05:46:55.0780840Z Deployments: write
2023-04-27T05:46:55.0781374Z Discussions: write
2023-04-27T05:46:55.0781787Z Issues: write
2023-04-27T05:46:55.0782188Z Metadata: read
2023-04-27T05:46:55.0782608Z Packages: write
2023-04-27T05:46:55.0783107Z Pages: write
2023-04-27T05:46:55.0783573Z PullRequests: write
2023-04-27T05:46:55.0784044Z RepositoryProjects: write
2023-04-27T05:46:55.0784555Z SecurityEvents: write
2023-04-27T05:46:55.0784984Z Statuses: write
2023-04-27T05:46:55.0785459Z ##[endgroup]
2023-04-27T05:46:55.0789828Z Secret source: Actions
2023-04-27T05:46:55.0790592Z Prepare workflow directory
2023-04-27T05:46:55.1854011Z Prepare all required actions
2023-04-27T05:46:55.2090852Z Getting action download info
2023-04-27T05:46:55.5465899Z Download action repository 'grandcentrix/actions-ssh2https@v1' (SHA:6fec85fda82c9971ed8976a646c40f35b76b4f13)
2023-04-27T05:46:56.0700261Z Download action repository 'grandcentrix/actions-zephyr-sdk@test/xz' (SHA:035feeb1e2c5c68e29dec6097025e0dc5168ec7b)
2023-04-27T05:46:56.5786100Z Download action repository 'actions/cache@v3' (SHA:88522ab9f39a2ea568f7027eddc7d8d8bc9d59c8)
2023-04-27T05:46:57.2027372Z Download action repository 'actions/upload-artifact@v3' (SHA:0b7f8abb1508181956e8e162db84b466c27e18ce)
2023-04-27T05:46:57.6529005Z Download action repository<EMAIL_ADDRESS>(SHA:4209421db54c8764d8932070ffd0f81715a629bf)
2023-04-27T05:46:58.2544922Z ##[error]System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter ''using: node18' is not supported, use 'docker', 'node12' or 'node16' instead.')
   at GitHub.Runner.Worker.ActionManifestManager.ConvertRuns(IExecutionContext executionContext, TemplateContext templateContext, TemplateToken inputsToken, String fileRelativePath, MappingToken outputs)
   at GitHub.Runner.Worker.ActionManifestManager.Load(IExecutionContext executionContext, String manifestFile)
2023-04-27T05:46:58.2581098Z ##[error]Fail to load grandcentrix/actions-zephyr-sdk/test/xz/action.yml

I downgraded to node16, please try again

Works! (:

The fix was released to v1, v1.3 and latest.


Not working

I tried most of those examples and none of them worked.
class ApexAPI {
    static {
        Apex.init(new Apex.ApexBuilder()
                .addParser("en", new EnglishParser())
                .build());
    }

    Event getEvent(String sentence) {
        return Apex.nlp("en", sentence);
    }
}

But when I ask for a simply parse I get only time parsed correctly
for example
ApexAPI apexAPI = new ApexAPI();
Event event = apexAPI.getEvent("Piano lessons Tuesdays and Thursdays at 5-6pm from 1/21 to 2/23");

I get:
Event{title='', startDateTime=2017-05-25T17:00:35.780, endDateTime=2017-05-25T18:00:35.780, isAllDay=false, recurrence=null}

Am I holding it wrong or what?
Thanks!

Did it work? If yes, let me know what you did to make it work?

No, I gave up :(

Title extraction does NOT work at all. The documentation is misleading. Sample inputs DO NOT produce outputs as advertised in the documentation. I understand its a relatively new project and i appreciate the work put in by the authors but documentation should not mislead.


[patch] Support for res.status(XXX).ok and res.status(XXX).serverError

Updates responses so that they support using res.status(xxx).ok|serverError . Also show deprecation warnings
For some reason I could not yet figure out, the standard 500 error sets statusCode to 200 so, unit tests fails, I tried to track it couldn't find the reason yet. I'll continue later but I want you guys to check this PR so far.

@luislobo looking really nice!  Esp. appreciate the comment updates and bringing over of the deprecation warnings.  (Not sure what's going on w/ the 200 vs. 500 though either...)
I'm in there w/ blueprints right now, so I'll go ahead and change master to have it do the return res.status(409).badRequest(err) thing for the uniqueness errors.  I just merged master into your branch and didn't see any weirdness or conflicts (so far so good!)

Hey @luislobo, all of the updates to comments and deprecation notices look great!  As far as the customizable status codes, now that we have a response generator (sails generate response) I'd really rather leave res.ok() and res.serverError() with their hard-coded 200 and 500 codes.  I understand the intent of making them configurable, but since res.status() is sticky, it can lead to situations where you get a status code in the response and it's not clear where it's getting set.  This is one of the problems that the built-in responses are supposed to solve.
@mikermcneil re: the uniqueness errors in blueprints, I think it'd be better to leave it calling res.badRequest() and document how to check for uniqueness errors in a custom badRequest response (i.e. checking data.name and data.code).
I'd definitely like to merge this in, if the status codes for the ok and serverError responses were set back to their hard-coded values!

@sgress454 the issues I see about ok, and serverError affixing the status to a specific one, is that it is not following Express 4 way of handling responses, like in .send and .json, where any response you send can override the default status using status().
I understand that you can override the responses, and that's what we have done from day 1, but, having to code your own response only for sending the status code seems like something very simple to overcome by just setting the default to what you want, 200 and 500 but also respecting the user decision of setting the statusCode.
I also understand that ok and serverError, as well as the other responses are Sails API definitions and that does not need to follow Express rules, but, in the end, Sails is overloading Express functionality adding better features and methods.
The other thing is that you cannot say that Sails is a RESTfull API "good player" from the ground up, you have to customize it. Again, that might be something that Sails is not looking forward to, but it would be a plus to just be able to build a REST API with just doing sails new my-api --no-frontend and have everything you need to start building one that respects what everyone is doing and expecting to have when building a REST API.
Again, this are all my points of view and I respect your decisions on this one as you define the API interfaces here. I'm just trying to think about it in the future, where Sails can still be competitive and usable in use cases like microservices (where REST is very used) and not having to explain much on how you can still use it, and why there is no need to go and use featherjs or any other framework to build a simple application/service.
Having said that, until I fix the issue about serverError having a 200 "preset", this PR cannot be merged.
I'll create one with just the other fixes, and if based on these arguments you still feel like it is not worthy, I can just continue doing what I have been doing, making my own responses.


Error Modal "Ok" Button Behavior For Dismissible Errors

Currently all the error modals have an "ok" button that by default will redirect users to another page (chaiseConfig.dataBrowser or homepage). This makes sense in most cases because we don't know the last stable page that users was in, and it's safer to redirect them to the top level page.
But in case of dismissible errors, the "ok" button can behave the same way as the "x" button. We should change the "ok" button in this case to just dismiss the modal. I think the only type of dismissible error that we currently have is the error that we're showing after deleting a record.

@RFSH @jrchudy Have we done this? Is this still an open issue?

We have not done this yet.

This has been done as part of React migration.


Wildcard Multi Word/Sentence Entities

In my NLU training data, I want to be able to have certain similar (but NOT always identical) phrases which are followed by multi-word or sentence entities and for all of the text following the similar phrases. For example,
Text my friend [I want to go to the movies tomorrow, but I have something else going on. Sorry!]
Make a text to my friend saying [had so much fun!]
As you can see the entities are not similar in content and could be anything.
ALSO, I am aware that this topic has been discussed before (https://stackoverflow.com/questions/49113915/is-there-wildcard-entity-in-rasa-nlu) and https://github.com/RasaHQ/rasa/issues/797. However, there was no clearly-defined answer, so I am creating another issue.
Thanks!

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

Hi @developer543, is this inherently different than https://github.com/RasaHQ/rasa/issues/6140? If so please describe how; if not, let's keep the discussion to your other issue.

Closing this due to inactivity

Closing this due to inactivity


IISHttpServer does not drain pending requests

When the server stops, all pending requests should be drained. Currently there is just a TODO there, not using the cancellation token or doing any actions.

Will be done by #576

This is done.


[SPARK-34911][SQL] Fix code not close issue in monitoring.md

What changes were proposed in this pull request?
Fix code not close issue in monitoring.md
Why are the changes needed?
Fix doc issue
Does this PR introduce any user-facing change?
No
How was this patch tested?
Not need

ping @srowen

Test build #136743 has started for PR 32008 at commit 8740821.

Test build #136743 has finished for PR 32008 at commit 8740821.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136743/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41325/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41325/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41325/


typeahead updates for search redesign

This is a collection of prep commits for #24345, the first of which is in a separate PR #26077

I reviewed these changes previously when they were a part of #24345, so I'm marking for integration review here.

...or I will once @evykassirer addresses or comments on the failing tests here.

Ah, I need to remove search.js from the 100% code coverage list, which I was already doing in a future commit of the redesign -- this file is UI heavy and it doesn't make sense to cover it all by our unit tests

Finally figured out the puppeteer test issue(s) and now it's finally passing. Adding integration review!

@timabbott do you need anything from me for this?

Closing since it seems like Tim would prefer to merge all together.


acessing *_seg.npy file

Hello, I installed cellpose using the terminal on my macbook. I was able to run one of my own images and get masks, and it was saved as .npy file in the same folder as the image. However, I am having trouble accessing this file. Is there something specific I can type in the terminal to load it (similar to what's described on the docs: https://cellpose.readthedocs.io/en/latest/outputs.html)?
I'm not using a notebook when I run the GUI, I just launch it from the terminal. I tried typing 'import numpy as np' when I'm in the same folder as the .npy file but get an error that the command is not found.
I would appreciate any insights into this, thank you!

if you want to load npy files in python you will need to run python first from the terminal by first typing python. then you can use import numpy as np and load accordingly. going to close the issue but let me know if you still have problems

Hello, I'm still getting the error (see below) in my terminal when I try to load the image with the red outlines. Is there a way to get this in the GUI instead? Thank you!
"Traceback (most recent call last).
File "", line 1, in
AttributeError: module 'cellpose.plot' has no attribute 'outlines_list'"


OSError: /lib64/libm.so.6: version `GLIBC_2.27' not found

Describe the bug
When using the Application Version, an error might occur with FS-GNN
OSError: /lib64/libm.so.6: version `GLIBC_2.27' not found

It is caused by torch-spline-conv.
Just uninstall it by:
pip uninstall torch-spline-conv

Thank you @rayrayraykk ! We hope this issue can help users who meet the same error.


Latching topics and the 3dPanel

Hello,
FYI @dbking77. We're having a problem with the way the 3dPanel keeps data between scrubbing on the timeline and latched topics. The problem is that when we seek forward and back on the timeline, the messages disappear
The problem comes down to the way the seekPlayback function is implemented here.  For us this is particularly frustrating as our maps are latched.
There are 2 ways we were thinking of to solve this issue:

We look for latched connections when we run getMessages https://github.com/cruise-automation/webviz/blob/2e7db3aafffec39b541728668c97ce7d83eee007/packages/webviz-core/src/players/RandomAccessPlayer.js#L437 and then just pick up the last message from them regardless of time
We implement something that goes through all the topics and gets the last message before this time for that topic.

Ah very interesting. We don't use latched topics as such, but we do have similar issues with our own bags. I'm currently actually working on something that should enable (2), but it's a pretty major refactor of our data storage APIs, so it might take a while. But it's my current priority internally as well, so that works out nicely. 😄

(2) would be the ideal solution for this I think so that works out. Let me know if you want any help on the smaller chunks. If you make them issues on this project I can help tackle some

+1 on a solution for this...

@cjds @bigredfrog update: still working on this, and @jasonimercer also mentioned this to me, so there's definitely a lot of interest. I had to pause working on it for a little bit though as we've had quite a few bugs that I've had to look into instead.
Btw @cjds and @bigredfrog, I'm trying to get to know our open source community better, and would love to learn more about what you use Webviz for. Could you perhaps shoot me an email at<EMAIL_ADDRESS>

Sure will do

Hey @janpaul123 any update on this?

Not yet!

Any updates on this?

A workaround for you could be to republish the latched topics at a regular interval. Not a great solution but maybe that can help for now?

That's actually interesting. So make a "node" that listens to those topics?

Btw we also have a Slack workspace now if you want to talk to us more easily:  https://github.com/cruise-automation/webviz/issues/461

Was trying to mess with this today and this is where I got till
In the below example, I was trying to use the global variable and a local variable in the node to try to republish the message.
The issues I ran into were:

The global variables do not seem editable from inside a node. Once changed they would get reset on the next loop
Local variables to the function seem to be reset when the scrubber on the bottom is moved.

As such I think I'm either doing the repub wrong or it isn't possible with this feature
import { Input, Messages } from "ros";

type Output = {};
type GlobalVariables = {
  id: number;
  localization: Messages.nav_msgs__OccupancyGrid;
};

let x: Messages.nav_msgs__OccupancyGrid;
export const inputs = [
  "/atlas/localization",
  "/controller_costmap_updater_clock"
];
export const output = "/webviz_node/my_new_map";

// Populate 'Input' with a parameter to properly type your inputs, e.g. 'Input<"/your_input_topic">'
const publisher = (
  message:
    | Input<"/localization">
    | Input<"/controller_costmap_updater_clock">,
  globalVars: GlobalVariables
): Messages.nav_msgs__OccupancyGrid | undefined => {
  if (message.topic == "/atlas/localization") {
    log("Resetting message");
    x = message.message;
  }

  if (x !== undefined) {
    log(x.info.resolution);
    return x;
  }
  log("Variable is undefined");
};

export default publisher;

Hey guys
Is there any update regarding the latching topics?

You should check out https://github.com/foxglove/studio, they might have added it!


multiple dB connections is not supporting

I have tried to connect with multiple databases using mysql_connector, it is working fine for some time. later it is showing errors like
mysql.connector.errors.OperationalError: MySQL Connection not available.

Ah yeah it's not currently designed to support usage of two separate MySQL objects. It's using the same key "mysql_db" in the app context stack which may be causing issues. I think for that to work it will need to allow you to pass an arg to optionally define your own key so the two objects don't fight over the same key. I can work on a fix this week.

I've updated the package so this should be resolved. please use pip install flask-mysql-connector==1.1.0. In order to use multiple MySQL db objects you'll need to set a custom app context stack key like so:
mysql = MySQL(app, ctx_key="num1")
mysql2 = MySQL(app, ctx_key="num2")


Having Compile Errors

Getting some compile error when uploading the fw to the minim osd
I added the both libs to arduino:  MAX7475 and SerialPort and still getting errors.
here is the output of the build:
http://pastebin.com/VdhEZzPp

Looks like the libraries are now double. You should not have to include the libs to arduino for it to compile correctly. Not sure what you did, but try undoing it.
Also: Are you using Arduino 1.0.5-r2?


[WIP]Convert iterator to mx.img API

The change bring me 3x performance improvement in single GPU.
@zhreshold Please check if this is what you suggested.
TODO:

[x] Fix crop when xmin,ymin < 0. Directly convert from np code doesn't work with ndarray.
[x] Figure out why SmoothL1=nan when multiGPUs is using.

I confirmed the new code converged in single GPU. however multi GPU will get L1Loss to nan quickly (less than 100 iterations). I need help. @zhreshold @piiswrong

Could you try use small lr for the first epoch, and then load the model from epoch 1 and use a normal lr, to see if it converges.
For compatibility, batchnorm is not used, so the gradients are not stable especially in the very beginning phase.

@zhreshold  your suggestion works. let me update README a bit to put this instruction and resubmit the patch.

Do you have numbers on io speed up?

I don't have number on IO. SSD with one single GPU in my box (GTX1070), the speed is increasing from 18samples/sec to 38samples/sec.
I used the profiler to check. the major of time is spending in flip function in data iterator.
However this still doesn't solve two GPUs scale problem. when I added one more GPU (GTX970), the speed is around 36 sampels/sec. I am still profiling it.

attached is a profiler log when training with two GPUs. there are 0.5s space when every second. I don't know why it happens.
twogpu.json.gz

try wrapping ssd's data iter with mx.io.PrefetchingIter

I did wrapping with prefetchingIter, I guess we need to optimize further the cpu operation performance

The profile looks strange. I wonder what it's doing between the bursts.
Maybe try only running IO or only running gpu (with dummy data) to see if the gap is still there?

Can u run the same thing in some standard machine and collect profiler result? I am wondering if my machine is not correct setup for 2 GPUs. As my motherboard is not a great one.

Can we merge this? And leave multi GPUs problem solve separately?
Also the author of SSD updated his paper and proposed a new network structure (use more conv layers) and some additional data augments. Shall I update this SSD sample? the challenge is how to maintain back compatibility?

A random thought, since data loading is so critial to performance, shall we also move data iterator under execute engine?

nnvm is now merged every layer can be used as a operator
Howard0Su<EMAIL_ADDRESS>周四上午5:03写道：

@howard0su commented on this pull request.

In example/ssd/dataset/iterator.py
https://github.com/dmlc/mxnet/pull/4375:

             else:

                 # padding mode

                 new_width = xmax - xmin

                 new_height = ymax - ymin

                 offset_x = 0 - xmin

                 offset_y = 0 - ymin

               data_bak = data

               #TODO need ndarray to support this

               data_bak = data.asnumpy()

               data = np.full((new_height, new_width, 3), 128.)

               data[offset_y:offset_y+height, offset_x:offset_x + width, :] = data_bak

there is a pad layer. but not an operator.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/pull/4375, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiudNklSYL9sg0NK3sCiSnMbr0-XyXOks5rM6-qgaJpZM4LVljm
.


fix: Fix for java_gapic_assembly_gradle_pkg rule.

More details at https://github.com/googleapis/gapic-generator/issues/3249.
The Bazel's native pkg_tar rule behaves weirdly when package_dir parameter is specified (at least on some Linux machines it does not put all the files under the package_dir).
As a workaround for that bug we tis fix creates the custom implementation of the pkg_tar rule.

@aohren FYI


Add move semantics to sf::Context

Description
This PR has two commits that show a bit of the history of development but they'll be squashed eventually.
The idea behind supporting move semantics is that I took the code that used to run in sf::Context::~Context and moved that to a custom deleter for m_context so that this class's data automatically cleans up after itself. This should allow us to use default, compiler-generated copy and move operations. In this case copy operations are automatically deleted and move semantics is as simple as moving this pointer with its custom deleter.

@kimci86 What do you think of this?

@ChrisThrasher Code looks mostly ok assuming moving between threads is not supported.
However, I am just not sure about the behavior of getActiveContext and underlying currentContext.
Consider the following code:
sf::Context c1;
CHECK(sf::Context::getActiveContext() == &c1);

sf::Context c2 = std::move(c1);
CHECK(sf::Context::getActiveContext() == nullptr); // <- This doesn't look right

@ChrisThrasher Code looks mostly ok assuming moving between threads is not supported.
However, I am just not sure about the behavior of getActiveContext and underlying currentContext. Consider the following code:
sf::Context c1;
CHECK(sf::Context::getActiveContext() == &c1);

sf::Context c2 = std::move(c1);
CHECK(sf::Context::getActiveContext() == nullptr); // <- This doesn't look right

That's a good point. I don't see how this unique_ptr deleter-based solution could ensure the newly move-constructed sf::Context sets the current active context. I might have to roll this back in favor of a more conventional solution.
The fact that I keep forgetting to write delete ptr; and the fact that we have to write delete at all has me calling into question whether this is the right solution. I feel like I'm abusing what deleters are meant for.

Another way to question this situation is whether tracking currentContext globally (for each thread) is of any use. It is well known that global states are bad and getCurrentContext is not used by SFML itself. Maybe we could consider removing this method.

Another way to question this situation is whether tracking currentContext globally (for each thread) is of any use. It is well known that global states are bad and getCurrentContext is not used by SFML itself. Maybe we could consider removing this method.

You might be onto something there. I'll let @binary1248 weight in on the value on maintaining this API.

We have to be careful not to assume too much. The difference between getActiveContext and getActiveContextId is that getActiveContext either returns a pointer to an actual constructed sf::Context object or nullptr whereas getActiveContextId can also return IDs to internal contexts that SFML constructs for housekeeping purposes.
If someone tried to make use of a weird "context pool" in their own code and kept track of which context was active using only getActiveContextId they would realize that it sometimes returns IDs of contexts they didn't even create. AFAIK SFML doesn't construct many if any sf::Context objects internally (we make use of TransientContextLock instead), so it is not far fetched to be able to assume that all contexts returned by getActiveContext were ones the user created themselves.
I think everybody who has been writing C++ code for a while can agree that any global state is something that should be avoided. However, we are working with an almost 30-year-old system API at this point. The damage was done long before SFML even existed. All this ugliness is just our feeble attempt at hiding this ugliness from the user under a nice C++ API. If here and there OpenGL manages to peek its head out of the library then it is something we will have to live with until we come up with a new rendering backend.

@binary1248 It is true that we cannot avoid global states with OpenGL, but it is not clear to me what is the purpose of getActiveContext in particular and the corresponding currentContext pointer. This method is not used in SFML nor its examples. What are the arguments for not removing this method?

I would have less of an issue getting rid of getActiveContext if sf::Context got a isActive and/or getId method. Checking for whether a sf::Context you create is currently active shouldn't require you to activate them right after construction just to save its ID. Right now the sf::Context API seems half-baked.
You are right that many of its currently public functions were introduced to be used by SFML at some point in time and end up being very technical in their usage, but we mustn't forget that our goal should also be to improve usability of the public API in user code as well.
While we have always actively recommended against using multiple OpenGL contexts, there are some legitimate very advanced use cases where using multiple contexts can improve performance, and having a semi-usable sf::Context API makes already complex use cases more complicated to implement than they need to be.

I rebased on #2816 and added more tests that should fail thus demonstrating some issues Kimci brought up.


Marks the interaction events as being of the type PIXI.interaction.InteractionData

For the docs

@themoonrat out of interest, why is both @memberof and @type needed?

Because it's an @event I believe
Without @memberof the entry will not show in the docs
I tried just without @memberof, and I tried without @memberof but changing @type to @member, and neither worked.
I don't know why, and I don't tend to question how jsdoc works anymore, I just let out a little cheer inside when it outputs what I want.


Modernize the script

Hello @a-m-s. I'm posting tiliado/nuvolaruntime#424 for you as well because there may still be a few tiny changes to make (probably the first three tasks from the check list). Could you check that `nuvolasdk convert-project' works as advertised? Thanks :-)

There have been a lot of changes lately. I need you to review and modernize your script (tiliado/nuvolaruntime#424) so that it keeps providing the best user experience possible. Eventually, I'd like to push some of them to Ubuntu software store as snap packages (tiliado/nuvolaruntime#273). If it increases the revenue of the project, I'd love to share it with the script maintainers (tiliado/nuvolaruntime#430).
I also need to know whether you still use this script and are willing to maintain it further. If you don't, I'll remove it from the stable repository until a new maintainer is found. No hard feelings though, you created & maintained it voluntarily and I'm glad for that.
Checklist
Use it to mark finished items.

[x] .gitignore is up-to-date
[x] README.md corresponds to the latest template
[x] metadata.in.json uses two spaces for indentation and no trailing whitespace
[x] integrate.js uses Standard JavaScript coding style
[x] The Chromium-based backend is required and Flash is disabled unless it is really required.
[x] The integration script works as expected with Nuvola API 4.11.
[x] Custom actions were added if applicable.
[x] Volume management is integrated if applicable.
[x] A track progress bar is integrated if applicable.
[x] Track rating is integrated if applicable.
[x] The web view snapshot (src/webview.png) was added.

Getting started
Since Nuvola migrated from DEB/RPM packages to cross-distribution Flatpak packages, the development also occurs inside Flatpak sandbox. All you need is to install Flatpak and Nuvola App Developer Kit (ADK), then you can start hacking. You don't have to worry about installing/building of dependencies, all you need is at your disposal. Learn more about Nuvola ADK.
The way the web app scripts are launched has changed as well. All you need is to launch Nuvola ADK, cd to your project directory and run nuvolaruntime -D.
Semi-automatic changes

I tried hard to make some changes automatic. Install Nuvola ADK and run nuvolasdk convert-project. If there are any issues, please report them.
Update .gitignore: nuvolasdk convert-project should have fixed that.
Update README.md: nuvolasdk convert-project will provide you with the new template (template--README.md) and a difference between it and your README.md (template--README.md.diff). Update your README.md to match the template - I will probably copy most of the template content and keep only the Copyright section from your current README.md.
Convert metadata.in.json: The metadata.in.json now uses two spaces for indentation and no trailing whitespace. nuvolasdk convert-project should have fixed that.
Convert integrate.js: Nuvola adopted Standard JavaScript coding style. This allows us to automatically check the scripts for coding errors and mistakes. nuvolasdk convert-project runs standard --fix, which tries to convert your integrate.js.

If you see errors regarding tab characters, replace tabs with spaces in a text editor and run standard --fix again.
If there is something ugly, please fix that manually. It sometimes creates long if-else single lines.
If there are other errors, please fix them and run standard --fix again and again...

Commit the changes before moving to the next section.

Chromium-based backend and HTML5 playback without Flash plugin
A long-time effort to get rid of Flash plugin may finally succeed. Nuvola had to switch a web rendering engine from WebKitGTK+ to Chromium Embedded Framework. Many scripts have already been ported, it's time to port the rest.

Add/set "requirements": "Chromium[65] Codec[MP3] Feature[MSE]" to metadata.in.json, run ./configure, and then launch your script with nuvolaruntime. Remember that you need to run ./configure whenever you changes metadata.in.json.
If there are any JavaScript errors or the website fails to load, let me know to help you.
Note that some synchronous API calls are not available in the Chromium-based backend for technical reasons. Use async variant instead - see Nuvola 4.8 API changelog.
Check whether audio playback works right away.
If your web app complains about missing Flash plugin, use "requirements": "Chromium[65] Codec[MP3] Feature[MSE] Feature[Flash]" and test again.

Update integration script

Set '"api_major": "4", "api_minor": "11" in metadata.in.json.
Look at NuvolaKit JavaScript API changelog - a few useful functions were added.
Make sure that your script and basic integration features (player metadata) work as expected.

Add new features
Several new features have been added to Nuvola. Look at them whether they apply to your web app.

Custom actions - these are not new, but aren't widely used.
Volume management - since Nuvola 4.5
Track progress bar  - since Nuvola 4.5
Track rating - since Nuvola 3.1

Add web view snapshot

Provide a representative snapshot of web view of your script to be used for screenshots. See tiliado/nuvolasdk#5 for the reasoning behind that.
First, navigate to a proper page.
Enable Developer tools in Preferences and open Developer sidebar.
Try to remove annoying elements (ads, nag banners):

Open Web View sidebar, set dimensions to 1080×600 and click Resize web view. If is necessary, e.g. because of the breakpoints of a responsive layout of the page, you can choose narrower dimensions.
Then take a snapshot of the web view and save it as src/webview.png.
Run ./configure; make all and look at the resulting screenshots in the screenshots directory.
The base screenshot still says "Deezer" but that will change to "Nuvola Player" or something like that in the final base screenshot. Also, the Unity base screenshot has an extra scrollbar, that will be fixed as well.

Done.


Please also add your adapter to the stable repository once ready

Your adapter is already in ioBroker latest repository available to be used by testwilling users. In order to make your adapter also available for the wide user base which uses the stable repository you should also add your adapter to the stable repository. Please see instructions at https://github.com/ioBroker/ioBroker.repositories#requirements-for-adapter-to-get-added-to-the-stable-repository
Thank you for your support.

https://github.com/ioBroker/ioBroker.repositories/pull/721


Feature/BE-40: Recommendation System

Added popularity field and updatePopularity function for ArtItem.
Created exhibition history.
Added number_of_views and popularity fields to Exhibition, created updatePopularity function.
Added popularity field and updatePopularity function for User.
Created UserInterest model to keep track of user interests (category based), created updateFirstThree and updateInterest functions.
Called these functions in necessary places to update popularities and interests.
Created signals and connected in necessary places to initiate UserInterest creation at user creation.
Updated history model and related receiver to overcome traversing GenericForeignKey problem.
Created 6 API endpoints for recommendation:

Tested everything through admin site and Postman.
Created detailed swagger documentation for APIs.

Algorithm Details and Parameters

ArtItem popularity is calculated taking into account; number_of_views of the artitem, number of likes and its creation date. The date parameter is added to age the popularity as time goes by, so that user always get recommended the freshest content.

User popularity is calculated taking into account; number of follwers of the user, number of art items that the user has and the number of exhibitions they have.

Exhibition popularity is calculated taking into account the number of views the exhibition has and its creation date. The date parameter is added to age the popularity as time goes by, so that user always get recommended the freshest content.

User Interest is calculated in a category based manner. Everytime the user interacts with content from that category, the users interest score in that category is updated. This interaction can be in various forms such as liking, viewing or uploading an art item in that category. These all affect in varying degrees, and  among these interests top 3 are selected for recommendation.

Recommendation Functions

Recommendation functions (APIs) make use of these parameters.

For instance art item recommendation function recommends to user art items in  user's top 3 favourite categories, that the user has never seen before. And inside these categories it ranks the items according to popularity and recommends the most popular ones. Popularity of items are everchanging as more users interact with them and as time goes by (aging), so that with enough activity on the website users will get fresh recommendations. We can also consider introducing more randomness to the system in the future.

Exhibition and User recommendation functions operate in similar ways. (Using not following condition for user reco instead of being never seen). For more detail please refer to the swagger documentation.

Anonymous versions of functions mostly make use of the popularity and don't specifically cater to user interests. Again refer to swagger for more detail.

Thank you for this excellent work, much appreciated! @BElifb
While reviewing the code, I've come up with the following question:

Why do we separate routes of anonymous user and registered user cases? We can simple check it within an if-else statement and act accordingly. By doing so, we'll also make the work of frontend easier as they won't have to check if the user is anonymous or not at the front side.

Thank  you for your review @KarahanS
I combined anonymous endpoints as recommended, but I think frontend still needs to check, because if they send an erroneous token instead of not sending one at all, it throws an error.
While testing I fixed a small mistake that led to items appearing twice if there wasn't enough in the database.

Updated urls to be:

recommendations/artitems/
recommendations/exhibitions/
recommendations/users/

Also updated description for 401 as Invalid Token rather than Authentication is required since anonymous users are also able to use them.
One possible bug I spotted: When I call /api/v1/exhibitions/online/1, without any token, I get the following error:

Cannot assign "<django.contrib.auth.models.AnonymousUser object at 0x00000218EC8F1D00>": "History.user" must be a "User" instance.
fyi @BElifb

I left the urls that way to differentiate, if we had art item recommendations in the future. But that doesn't really matter now, so I'm leaving them as is.
Good catch with the error. Although I don't understant why the same thing doesn't throw errors in the artitem and user profile cases, I added authentication checks for all of them. So it should be good now. @KarahanS

Tested with Postman, everything seems to be working properly. Approved!


WGL not implemented

The WGL bindings are missing the GetApi/IsExtensionPresent/TryGetExtension implementations. These need to be added.

whoops, round 2 aint it? how did this happen again? I'll get a PR open to add GetApi, although im not sure what IsExtensionPresent would do here? The WGL API does not mention extensions, should it just NOP?


Update guides for Swift 4

https://github.com/IBM-Swift/Kitura/issues/1156#issuecomment-333896228

@youming-lin please can you close this now, as the Getting Started has been rewritten and no longer contains this information.  Swift 4 updates have been made across the kitura.io site now and the Package.swift here http://www.kitura.io/en/starter/generator/package_swift.html has been updated accordingly.


Include plugin repo's version.json in releases

Description of Changes
Please check that the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Add a changelog entry (for non-trivial bug fixes / features)
[ ] CI passes

Log Files
Logs / Second
CPU Average (%)
Memory Average (MB)

1
1000
2.7931685
27.923222

1
5000
6.327763
36.07422

1
10000
10.9658165
44.643456

1
50000
40.15605
115.5528

1
100000
73.12471
189.30173

10
100
4.0862947
28.456896

10
500
8.603698
35.352776

10
1000
13.207252
46.302803

10
5000
45.20799
90.575294

10
10000
80.002075
198.40976


VS Code Support

Hi, I finally got around to finishing things and making this PR. You may have already seen some of these changes since I saw that you starred my fork a few days ago, but here's a breakdown of what I've done and some notes about them.
Extension

The extension now expects harper-ls to be bundled with it.
Users can now manually restart harper-ls if they want.
When a user changes any setting contributed by the extension, harper-ls is now automatically restarted so those changes are reflected right away.
Added icon for the Marketplace.

Tests
I've added some integration tests, mostly to serve as a smoke test to ensure that the extension works with the current state of harper-ls. In the process, I removed unused scripts and dependencies in package.json and cleaned up the files in .vscode a bit.
justfile

Added test-vscode recipe.
Updated package-vscode recipe, mostly so it can be used by the workflow discussed below.
Opted to use test-vscode instead of package-vscode in both precommit, because it's a better gauge if the extension is working properly, and setup, because testing downloads extra resources that packaging doesn't.

Docs

Added an "Unreleased" section in CHANGELOG.md for the extension changes I've done.
Updated README.md for the Marketplace. I intentionally left the "Installation" section empty for when you're ready to release or have released the extension as people might be misled when they get an error trying to install elijah-potter.harper and get an error.
Added development-guide.md to make it easier for those who want to contribute.

Publishing Workflow
I created a workflow to assist in publishing the extension, basing on build_harper_ls.yml, it compiles harper-ls for every platform currently supported, packages it with the extension, then publishes the extension for each platform. It can be triggered by pushing a tag prefixed with harper-vscode or manually in the GitHub Actions UI. For the workflow to work, you need to create the elijah-potter publisher, get a Personal Access Token (PAT), and create a secret named VSCE_PAT set to the value of the PAT you previously obtained.
My idea with this is that the extension would have the same version as harper-ls so it's easy for users (and us if they file an issue) to know that extension version X is also harper-ls version X. So when publishing with this workflow, the main manual work that needs to be done before running it is to update the version in package.json and maybe change the "Unreleased" section in CHANGELOG.md, if there is any, to the new version. That said, I'm open to other ideas, maybe you prefer a different way of publishing the extension or you want to reduce the manual steps or maybe you want to test the extension first before it's packaged to be sure it works.
Note, you may notice that I added "endOfLine": "auto" to .prettierrc and that's because Prettier throws a fit in Windows without this. Here's an example of a run where everything is okay with macOS and Linux but not Windows.
Closes #116

I would like to test this out on a couple systems other than my own. @lukasmwerner can you confirm that this works on ARM Mac? Make sure you remove your existing harper-ls before testing.

Confirmed working on M1 running Sonoma 14.0! Loads the bundled harper-ls build perfectly. No issues, even after adding my local install of harper-ls back, very impressive!

I've done some testing and it looks like our current CI setup for Windows is broken. See #208. I'd like to get this fixed before merging.

Sorry this has takes so long to get through. I'm going to merge it and cut a release as soon as possible. Thank you so much for all you work.


[iOS] [google_maps_flutter] plugin stopped working

Hi,
I'm trying to launch the iOS version of my app, and I'm running into a roadblock. Google Maps integration has seemingly stopped working from 2019-12-05 (when my last build was released) to 2019-12-09, without having changed a single thing in the build or the config. In fact, even the December 05 version, installed from TestFlight, has stopped working after I tried reverting there.
When a Maps widget is loaded on screen, I'm getting a deluge of BoringSSL errors, all along the lines of TIC Read Status [21:0x0]: 1:57 (obviously, connection numbers change, but somehow the connection pointers always remain 0x0. From time to time, I manage to catch a more verbose error in the debugger:
2019-12-09 18:51:35.972806+0100 Runner[19478:1766346] [BoringSSL] nw_protocol_boringssl_get_output_frames(1301) [C21.1:2][0x113efaf60] get output frames failed, state 8196
2019-12-09 18:51:35.979757+0100 Runner[19478:1766346] TIC Read Status [21:0x0]: 1:57
2019-12-09 18:51:35.980041+0100 Runner[19478:1766346] TIC Read Status [21:0x0]: 1:57
2019-12-09 18:51:45.731088+0100 Runner[19478:1765378] Google Maps SDK for iOS cannot connect or validate APIKey: Error Domain=com.google.HTTPStatus Code=400 "(null)" UserInfo={data=<3c48544d 4c3e0a3c 48454144 3e0a3c54 49544c45 3e426164 20526571 75657374 3c2f5449 544c453e 0a3c2f48 4541443e 0a3c424f 44592042 47434f4c 4f523d22 23464646 46464622 20544558 543d2223 30303030 3030223e 0a3c4831 3e426164 20526571 75657374 3c2f4831 3e0a3c48 323e4572 726f7220 3430303c 2f48323e 0a3c2f42 4f44593e 0a3c2f48 544d4c3e 0a>}
2019-12-09 18:51:45.731345+0100 Runner[19478:1765378] Your key may be invalid for your bundle ID: hu.skawa.dive-site-overwatch-ios

Despite what the error message says, I've checked and re-checked the key, its permissions, its associated bundle IDs, and everything should be working. Yet, without having changed anything, it is not.
So far, I've tried upgrading to google_maps_flutter: 0.5.21+13 (from build +7), as well as upgrading Flutter itself, along with the requisite cleans, to no avail. Not that it would explain why a four days old build re-installed from TestFlight would also produce the error (unless this is a Schroedinbug).
The issue presents itself in the Simulator, as well as a physical iPhone.

Output of Flutter Doctor:
[✓] Flutter (Channel beta, v1.12.13+hotfix.3, on Mac OS X 10.14.6 18G1012, locale en-GB)
    • Flutter version 1.12.13+hotfix.3 at /Users/zmeggyesi/tools/flutter
    • Framework revision 57f2df76d7 (4 days ago), 2019-12-05 21:23:21 -0800
    • Engine revision ac9391978e
    • Dart version 2.7.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/zmeggyesi/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.2.1, Build version 11B500
    • CocoaPods version 1.8.4

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 41.1.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[!] VS Code
    • VS Code at /Applications/Visual Studio Code.app/Contents
    ✗ Flutter extension not installed; install from
      https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (1 available)
    • iPhone (2) • 9cc55aacffcf500003c8f83bd7cbcc2aeb67ae37 • ios • iOS 12.4.3

! Doctor found issues in 1 category.

Well, this is somewhat perplexing: I never had Maps SDK for iOS enabled in my GCP project.
Which raises the question as to how Maps could have worked for the last four days...?

Hi @zmeggyesi
can you please provide your flutter run --verbose?
I see that there are newer version of the plugin 0.5.21+15,
there are being released quite often,
which is understandable given that it's a developer preview;
could you try if it works with the latest?
Thank you

Hi @iapicca
Like I said, it's a bit perplexing, as it turns out the root cause was that the iOS SDK API was never enabled on the GCP project.
On its own this wouldn't mean much, but the question then becomes "How has this ever worked?", especially in the last four days or during development during which the API was still not enabled, and maps were still being rendered.
To be fair to you, if you don't notice anything particularly alarming about this report, I would rather you focused Flutter efforts towards more severe/rewarding issues.

This should be reported against the Google Maps iOS SDK https://developers.google.com/maps/documentation/ios-sdk/support


Please Add Azure Percept links to this page

[Enter feedback here]
Percept is missing in this page
Thanks
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3485aba5-b234-dcb7-0e9f-a473c3335582
Version Independent ID: 55b2938c-41c1-4dec-a1df-925988b91035
Content: Azure IoT support and help options
Content Source: articles/iot-fundamentals/iot-support-help.md
Service: iot-fundamentals
GitHub Login: @philmea
Microsoft Alias: phil

Hello @ozhernan thanks for the feedback.
Our SME will review your feedback.

any updates about this?

@ozhernan We are looking into this.
Cc: @asergaz to help with this page update.


Please remove getPlayerBalance API

Is your feature request related to a problem? Please describe.
getPlayerBalance is constantly abused for incorrect code where they check for whether the player has enough balance through getPlayerBalance and call subtractFromPlayerBalance without actually checking whether the latter is successful.
Describe the solution you'd like
Remove the method. It is good for nothing other than abuse.
People who want to display the balance should use an API dedicated for displaying, such as InfoAPI. getPlayerBalance is good for nothing.

Here is a quick grep of released plugins where getPlayerBalance is used.

@DaDevGuy /EconomyRename: incorrect use as described in OP
ZiteDesigns/EnchantShop: just a var_dump
BeeAZ-pm-pl/SeasonShop: they don't know how to do async
FlxiBoy1313/Perks: they don't know how to do async
@Shock95x /AuctionHouse: incorrect use as described in OP, but still handles purchase error as unknown error
muqsit/ChestShop: for display use only
NovaStark1234/SellStick: they don't know how to do async
udachin26/BedrockEconomy-udachin26: used for display only
@MulqiGaming64 /EconomyEnchant: incorrect use as described in OP
RevandDev/BuyHealFeed: just a var_dump
AlexPads/ShopUI-FormAPI-Working: FINALLY someone who uses it correctly! Used for display only

hm

Currently it is used for the balance and the global cache in v4.0.0. I've marked it as internal with comments that describe its usage.


Fix event handlers for InputWithSelect.vue

Summary
This allows the InputWithSelect component to properly function by explicitly declaring update:value in the emits property of the component and moving the associated event handler from the wrapper div to the actual components that emit events.
Fixes #11950

Occurred changes and/or fixed issues

Add emits: ['update:value'] to InputWithSelect.vue
Remove @update:value listener from wrapper div and add appropriate listeners to components

Technical notes summary

Adding an explicit emits helped to identify the core issue by disabling any other events that might've been attached to the component.
Areas or cases that should be tested

Details in issue

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

After rebasing, I see that #11964 has already merged. I'm leaving my notes the same as they are relevant to how the problem was resolved.


Upgrade to 4.0, I get a message when the app runs in an installer can't find Microsoft.Data.SqlClient 3.0.0

Describe the bug
I updaded a project and Nuget package to .NET6, updgraded from System.Data.SqlClient to Microsoft.Data.SqlClient 4.0.0, and the installer runs. When I try and connect to a database, I get the error can't find Microsoft.Data.SqlClient version 3.0.0.
With packages references I knew how to add a binding redirect, I don't see how to that in a Project file.
Just a comment, your breaking change of 'Encrypt = true' now by default. I went the other way. I started off with encryption by default in 2010 and decided in 2019 it was time to make it Encrypted = false. You broke my project DB Compare, because people paste in connection strings, and they won't work now. I wish this was an option, not a dictate by you. Not everyone is as paranoid about security as you are.
Can someone explain to me how to force the entire app to use 4.0.0 version?
If you are seeing an exception, include the full exceptions details (message and stack trace).
************** Exception Text **************
System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.Data.SqlClient, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=23ec7fc2d6eaa4a5'. The system cannot find the file specified.

Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.
Console.WriteLine("Hello World!");

Expected behavior
A clear and concise description of what you expected to happen.
Further technical details
Microsoft.Data.SqlClient version: (found on the nuget or Microsoft.Data.SqlClient.dll)
.NET target: (e.g. Framework 4.7.1, Core 2.2.2)
SQL Server version: (e.g. SQL Server 2017)
Operating system: (e.g. Windows 2019, Ubuntu 18.04, macOS 10.13, Docker container)
Additional context
Add any other context about the problem here.

Please share the .csproj of your main executable project
You must have a 3.0 reference somewhere...

My project file is attached. Thank you for the fast response.
My project file is attached. I looked all through the projects that are mine, I can't find anything.
Is there a way to force it to use 4.0, like a binding redirect? Switching from packages.config to project references is another thing I wouldn't have voted on.
I understand the benefit to updating if things work. I don't get paid to fix all these changes like salaried people do, but thanks none the less for trying to help.

WinExe
net6.0-windows
true
6.0.0
DataJuggler.DBCompare
DataJuggler
DataJuggler
DataJuggler.DBCompare
DB Compare compares two instances of a SQL Server database. This project has been on .Net Framework
for about 10 or 12 years or more. DB Compare has been updated to .NET6.0, and should run faster.
C# Developers - Now you can create a project in a manner of minutes and this way you can stay updated with the latest version.
This product is also a standalone application. Using it as a Nuget package is just a nice way to always have
the latest version.
License.txt
https://github.com/DataJuggler/DBCompare
Database Compare Transparent.png
https://github.com/DataJuggler.git
Git / Git Hub
C#, SQL, Database, Compare Schema, Compare Databases, Compare SQL
Version 5.0.9 - I created a Generate Scripts Button that shows after a Comparison. For now it only creates the Fields that are missing. I will work on this over time. All fields are created as a nullable. If you need to change that, you can change the SQL that is copied to your clipboard.
Version 5.0.8 - I added a link to a video I just made for this project in the ReadMe.txt that opens.
Version 5.0.7 - The text was out of date. I need to convert the text to pull from the binary version number, or remember to enter the text when I update.
Version 5.0.6 - The YouTube Button showed up on the Form, but the background image didn't make it. Now this is fixed. I should have checked for this when I fixed the Swap Button. Oops.
Version 5.0.5 - The Swap Button showed up on the Form, but the background image didn't make it. Now this is fixed.
Version 5.0.4 - Testing including a ReadMe that opens with the project. (Worked)
Version 5.0.3 - It turns out the Label text is set in the Init method of the Main Form.
It is fixed now.
Version 5.0.2 - I left the version # as 2 something on the Main Form. Now it is up to date.
I now know this works! New video coming soon.
Version 5.0.1  - I had an image path that was in my Graphics folder, not in the project folder.
This is the 1st version for .Net 5. This project has been ported from .Net Framework, and this is the 1st
attempt as a Nuget package.
Database Compare Transparent.ico
true

True

True

True
True
Resources.resx

ResXFileCodeGenerator
Resources.Designer.cs

I found it. One of my Nuget packages had a reference to 3.0.0.
It seems kind of hard to go down the chain and see this.
Sorry to bother you, and thank you.

@DataJuggler I tried to clone your application from your GitHub repo , correct me if I have gone to a wrong repo. By just changing System.Data.SqlClient to Microsoft.Data.SfqlClient the application worked fine as a project. I did not try the installer or using the created nuget though. There might be referencing to other projects that have dependencies to version 3 of MDS?
Regarding the breaking change, we tried to avoid it for several years, but the security concern was going high. you can read more at this discussion here.

The latest version is not checked in until I can fix this.
Not sure what TargetFramework the check in version is. It was just upgraded to 6.0.0, and Microsoft.Data.SqlClient 4.0.0
I can put a zip file on my server or somewhere else if you prefer.
The problem occurs when a method called Test Database Connection is called, if you don’t add encrypt=false. Short solution is I can just add it, but then I don’t see any benefit to your security upgrade.
Forcing SSL of SQL on people should have been an option, especially for non servers. If anyone is sniffing between my ATT Fiber Modem and my machine, I have bigger problems than SQL.
From: Javad @.>
Sent: Friday, December 31, 2021 11:39 AM
To: dotnet/SqlClient @.>
Cc: Corby Nichols @.>; Mention @.>
Subject: Re: [dotnet/SqlClient] Upgrade to 4.0, I get a message when the app runs in an installer can't find Microsoft.Data.SqlClient 3.0.0 (Issue #1448)
@DataJugglerhttps://github.com/DataJuggler I tried to clone your application from your GitHub repo https://github.com/DataJuggler/DBCompare , correct me if I have gone to a wrong repo. By just changing System.Data.SqlClient to Microsoft.Data.SfqlClient the application worked fine as a project. I did not try the installer or using the created nuget though. There might be referencing to other projects that have dependencies to version 3 of MDS?
Regarding the breaking change, we tried to avoid it for several years, but the security concern was going high. you can read more at this discussion herehttps://github.com/dotnet/SqlClient/pull/1210#issuecomment-898658021.
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/SqlClient/issues/1448#issuecomment-1003424881, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGI4P7I65JGUGWR3ADEVIY3UTXTFBANCNFSM5LBM5ZTQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>


Limit the serching range of static table by the first letter of header name

Before:
finished in 14.74s, 6784.85 req/s, 53.19MB/s
requests: 100000 total, 100000 started, 100000 done, 100000 succeeded, 0 failed, 0 errored, 0 timeout
status codes: 100000 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 783.92MB (822000249) total, 976.78KB (1000219) headers (space savings 96.45%), 781.25MB (819200000) data
                     min         max         mean         sd        +/- sd
time for request:      860us      3.60ms      1.45ms       179us    89.26%
time for connect:     4.08ms      4.08ms      4.08ms         0us   100.00%
time to 1st byte:     7.58ms      7.58ms      7.58ms         0us   100.00%
req/s           :    6784.90     6784.90     6784.90        0.00   100.00%

After:
finished in 13.09s, 7642.25 req/s, 59.91MB/s
requests: 100000 total, 100000 started, 100000 done, 100000 succeeded, 0 failed, 0 errored, 0 timeout
status codes: 100000 2xx, 0 3xx, 0 4xx, 0 5xx
traffic: 783.92MB (822000227) total, 976.75KB (1000197) headers (space savings 96.44%), 781.25MB (819200000) data
                     min         max         mean         sd        +/- sd
time for request:      766us      3.28ms      1.28ms       163us    89.01%
time for connect:     3.78ms      3.78ms      3.78ms         0us   100.00%
time to 1st byte:     6.61ms      6.61ms      6.61ms         0us   100.00%
req/s           :    7642.30     7642.30     7642.30        0.00   100.00%

Before:

After:

Since entries in HPACK static table is sorted, we can narrow the search range by checking the first character of a header name.

I added the comment and also a temporal variable to store the first character.


Documentation about Profile Settings page

Summary
Document about different options available in the Profile Settings page and how users can make use of them.
Why do we need this ?
To let users be aware of how they can change their passwords, Email addresses, etc.
What is already there? What do you see now?

...
What is missing? What do you want to see?
Documentation of different options that are available in the Profile Settings page.
How do you propose to document this?
To add a new section in the Console section in Getting started.
Can you do this yourself and submit a Pull Request?
No

@nejraselimovic can you pick this one up


cannot start

github.com/dubbo/go-for-apache-dubbo/dubbo/client.go:237:21: undefined: hessian.Heartbeat

github.com/dubbo/go-for-apache-dubbo/dubbo/client.go:237:21: undefined: hessian.Heartbeat

Is your version of Hessian 2 different from version in go.mod?
Try to run go mod tidy

@MuFe Are u interested in joining the dubbogo develop team?

@MuFe Are u interested in joining the dubbogo develop team? If so, please send a email to me ( alexstofcks at foxmail dot com ) to contact with me.

@MuFe Hi，Does your issue has been resolved?

@MuFe If there is no problem, I will close the issue. If there is still have  problem, you can open it at any time.


sort instance_colors when it is given

fix #861

pfnCI, test this please

pfnCI, test this please

pfnCI, test this please


chore: bdd-nh-1583236807 to 0.0.1

chore: Promote bdd-nh-1583236807 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Doctest failing on Github actions

Run check/doctest -q
cirq/contrib/routing/device.py
1 failed, 4 passed, 5 total
**********************************************************************
File "cirq/contrib/routing/device.py", line 64, in cirq_doctest_module.nx_qubit_layout
Failed example:
    nx.draw_networkx(g, pos=pos)
Exception raised:
    Traceback (most recent call last):
      File "/opt/hostedtoolcache/Python/3.6.10/x64/lib/python3.6/doctest.py", line 1330, in __run
        compileflags, 1), test.globs)
      File "<doctest cirq_doctest_module.nx_qubit_layout[4]>", line 1, in <module>
        nx.draw_networkx(g, pos=pos)
      File "/opt/hostedtoolcache/Python/3.6.10/x64/lib/python3.6/site-packages/networkx/drawing/nx_pylab.py", line 277, in draw_networkx
        draw_networkx_labels(G, pos, **kwds)
      File "/opt/hostedtoolcache/Python/3.6.10/x64/lib/python3.6/site-packages/networkx/drawing/nx_pylab.py", line 814, in draw_networkx_labels
        clip_on=True,
    TypeError: text() missing 1 required positional argument: 's'

Failed: 1 failed, 88 passed, 89 total
##[error]Process completed with exit code 1.

Related to #2925.

Fixed by https://github.com/quantumlib/Cirq/pull/2987


Bug: Can't reach classes in exported functions on compiled code using the module resolution "system" with "es6"

TypeScript Version: 1.8.10
Code
This is the file in the libary
// File ../../lib/test.ts
/**
 * Created by ThomasP on 29.04.2016.
 */

export function test(): number {
    return Model.getNum();
}

export class Model {

    static getNum():  number {
        return 1;
    }
}

And this is the test spec for the libary file
// File: test.Spec.ts
import {test} from "../../lib/test";
/**
 * Created by ThomasP on 29.04.2016.
 */

describe( 'test', function () {

    it( 'should be able to get Model', function () {
        expect( test() ).toBe( 1 );
    });

});

The actual compile target is es6 and the module system is system, see below
{
  "compilerOptions": {
    "target": "es6",
    "module": "system",
    "moduleResolution": "node",
    "sourceMap": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "removeComments": false,
    "noImplicitAny": false
  },
  "exclude": [
    "node_modules"
  ]
}

Expected behavior:
The test should work and the method test should be able to call the static method on the class Model
Actual behavior:
The class Model is not reachable for the method test. The compiled target looks like this. Model will be defined but is undefined. Creating the class Model in the local execute function will not set Model to the variable defined in the context.
/**
 * Created by ThomasP on 29.04.2016.
 */
System.register([], function(exports_1, context_1) {
    "use strict";
    var __moduleName = context_1 && context_1.id;
    var Model;
    function test() {
        return Model.getNum();
    }
    exports_1("test", test);
    return {
        setters:[],
        execute: function() {
            class Model {
                static getNum() {
                    return 1;
                }
            }
            exports_1("Model", Model);
        }
    }
});
//# sourceMappingURL=test.js.map

Solution:
I think all defined classes must be set to the variables, which are defined in the upper context. Than the module will work as expected. I changed he output like above and now it will work fine. Another possible solution is to define a class not in the execute function, should be also work as expected.
/**
 * Created by ThomasP on 29.04.2016.
 */
System.register([], function(exports_1, context_1) {
    "use strict";
    var __moduleName = context_1 && context_1.id;
    var Model;
    function test() {
        return Model.getNum();
    }
    exports_1("test", test);
    return {
        setters:[],
        execute: function() {
            Model = class Model {
                static getNum() {
                    return 1;
                }
            }
            exports_1("Model", Model);
        }
    }
});
//# sourceMappingURL=test.js.map

I forgot to mention that if you export vars instead of functions you could prevent the problem. But this could only a workaround for the problem.
/**
 * Created by ThomasP on 29.04.2016.
 */

export var test = function test(): number {
    return Model.getNum();
};

export var Model = class Model {

    static getNum():  number {
        return 1;
    }
};

will export to
/**
 * Created by ThomasP on 29.04.2016.
 */
System.register([], function(exports_1, context_1) {
    "use strict";
    var __moduleName = context_1 && context_1.id;
    var test, Model;
    return {
        setters:[],
        execute: function() {
            exports_1("test", test = function test() {
                return Model.getNum();
            });
            exports_1("Model", Model = class Model {
                static getNum() {
                    return 1;
                }
            }
            );
        }
    }
});
//# sourceMappingURL=test.js.map

looks fixed in typescript@next. the original sample produces:
System.register([], function(exports_1, context_1) {
    "use strict";
    var __moduleName = context_1 && context_1.id;
    var Model;
    function test() {
        return Model.getNum();
    }
    exports_1("test", test);
    return {
        setters:[],
        execute: function() {
            Model = class Model {
                static getNum() {
                    return 1;
                }
            };
            exports_1("Model", Model);
        }
    }
});


Side channel issues

According to nullc on reddit (http://www.reddit.com/r/Bitcoin/comments/2rrxq7/on_why_010s_release_notes_say_we_have_reason_to/cnip3wx):

That code may be too slow for the non-static parts of our test to be very effective, as seems to be doing all operations with affine coordinates. Though it appears to be focused on signing and signing is less critical because it can be validated and (in the case of Bitcoin applications) is validated before transmission (and I suppose the host not the device is doing most of the pubkey generation). Independent of the testing, the code you linked to appears to be have a very large timing and power sidechannel-- e.g. timing alone should directly read out the hamming weight of the DSA nonce, even worse than OpenSSL. (and seems to have fairly little testing the repository?). I wouldn't advise using it in any place where that potentially mattered.

I'll respond in the reddit thread as no patch is attached anyway ...


[HUDI-604] Update docker page

What is the purpose of the pull request

Change one line command to multi line
Modify  sub title

Verify this pull request
Compare changes

https://hudi.apache.org/docs/docker_demo.html
https://lamber-ken.github.io/docs/docker_demo.html

Committer checklist

[X] Has a corresponding JIRA in PR title & commit

[X] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

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

Will merge this with other doc changes later next week after cutting 0.5.1 doc version


Install the agreed-upon sims version at SLAC for Run3 data generation and analysis

@heather999 @danielsf @rbiswas4 @jbkalmbach  Can you guys sort out which version to install?

I'll keep an eye on the discussion here.  @jchiang87 would you want this installed at SLAC in:
/nfs/farm/g/lsst/u1/software/redhat6-x86_64-64bit-gcc44/DMstack  ?
I'm assuming we're not using NERSC due to the Cori outage.

I think that should be fine.


Suggesting edit to r-tutorial

Hi,
Somewhere in your r-tutorial green DETAILS box under section 3.2.2 Slicing, you mentioned that

Second, DT[1:2] will slice the first two columns instead of the first two rows, thanks to the fact that data frames (and data.tables) are lists and the no-comma usage of [ triggers list slicing.

Should it be slice first two rows instead of first two columns?

Thanks! I have now changed it to say:

Second, DF[1:2] will slice the first two columns instead of the first two rows, thanks to the fact that no-comma usage of [ on a data.frame triggers list slicing.

The way I wrote it was not a typo, but I can see how it could be unclear. This might not be up on the web version soon since I am not very adept with Github yet.


Add --no-install-recommends to apt-get in Dockerfile

Prevent to install unnecessary recommended packages, make it lighter.

Merged, thanks


background-opacity does not function properly

Minimal reproducible config:
theme = catppuccin-latte
background-opacity = 0.2

It's especially obvious with light themes:

Dark themes perform better, but the color is still not accurate:

The bug is related to color blending, or more specifically, the incorrect usage of premultiplied alpha.

I guess it might be related to premultiplied alpha, although I haven't read the code.

Confirmed. Premultiplying alpha resolves this issue.

as seen in #3449, this is not just a macos issue, it is based on theme alone.

@SimonYde I think #3347 can resolve the problem on Linux as well. If it's convenient for you, could you help me test this fix on Linux?

I just brew installed ghostty. I set the theme in an included config file. in the base config file I put background-opacity = 95, and it only affects the title bar of the window. The actual window contents stay 100% opacity

@sethgrid Have you tried restart (of Ghostty)? For me the reload config does not work as reliably when it comes to opacity. Had the title bar thing as well, but I think restart instead of reload helped.

background-opacity = 95

@sethgrid Do you literally have background-opacity = 95 in your config? You probably want 0.95 instead.

@injust I believe that changing the background-opacity from 1 (opaque/default) to any value lower than 1 requires a restart, at least on macOS.

@injust I believe that changing the background-opacity from 1 (opaque/default) to any value lower than 1 requires a restart, at least on macOS.

turns out it is the same on Linux, so restarting fixed it for my theme. However, I can reproduce the bug with catppuccin-mocha as my theme

@lixiaoyan Thanks for tracking this down. I was just noticing that setting an alpha could make my Ghostty window brighter than either the background theme or my wallpaper, so I was thinking it was a blend mode bug.
Reviewing your PR, I'm wondering if you should be premultiplying by (1-alpha) for the background:
alpha * fg + (1-alpha) * bg = resulting color, no?

@raylee Nope, that's not how "premultiplied alpha" works. We only need to premultiply alpha for the foreground (the background of terminal), and the shader will handle the rest.
See: https://en.wikipedia.org/wiki/Alpha_compositing#Straight_versus_premultiplied

I just brew installed ghostty. I set the theme in an included config file. in the base config file I put background-opacity = 95, and it only affects the title bar of the window. The actual window contents stay 100% opacity

per other's recommendations, restarting worked. Opacity working as expected. And, yeah, typo: that is 0.95; updated to 0.6

It works if you do a hard restart of Ghostty like @rplnt said.

I'm running @lixiaoyan 's fork on my machine now, works very well. merging #3347 seems like a win to me

in my case the following config didn't work even after a hard restart:
background-opacity = 0.7
background = #fafafa


System.InvalidOperationException When Using IAsyncDisposable

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes
Occurs When
When using MassTransit DI with a scoped consumer which has a dependency which only implements IAsyncDisposable.
It appears MassTransit should be calling DisposeAsync()
Stacktrace
MassTransit.ReceiveTransport: Error: R-FAULT rabbitmq://mq/report_queue 000a0000-ac16-0242-5f5f-08d794932d75 <redacted>.IMyCommand <redacted>.MyConsumer(00:00:06.8635823)

System.InvalidOperationException: '<redacted type name>' type only implements IAsyncDisposable. Use DisposeAsync to dispose the container.
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ServiceProviderEngineScope.Dispose()
   at MassTransit.Scoping.ConsumerContexts.CreatedConsumerScopeContext`3.Dispose()
   at MassTransit.Scoping.ScopeConsumerFactory`1.Send[TMessage](ConsumeContext`1 context, IPipe`1 next)
   at MassTransit.Pipeline.Filters.ConsumerMessageFilter`2.GreenPipes.IFilter<MassTransit.ConsumeContext<TMessage>>.Send(ConsumeContext`1 context, IPipe`1 next)

Environment
Dotnet version: .NET Core 3.0.0
Package: MassTransit (6.0.1)
Package: MassTransit.Extensions.DependencyInjection (6.0.1)

Service Configuration
services.AddScoped<MyConsumer>();

// Add MassTransit
services.AddMassTransit(x =>
{
    // Add Consumers
    x.AddConsumer<MyConsumer>();

    x.AddBus(provider => Bus.Factory.CreateUsingRabbitMq(cfg =>
    {
        var host = cfg.Host(new Uri(RabbitMqConstants.BASE_ADDRESS + Environment.GetEnvironmentVariable("RABBITMQ_SERVER")), h =>
        {
            h.Username(Environment.GetEnvironmentVariable("RABBITMQ_DEFAULT_USER"));
            h.Password(Environment.GetEnvironmentVariable("RABBITMQ_DEFAULT_PASS"));
        });

        cfg.ReceiveEndpoint(RabbitMqConstants.MY_QUEUE, ep =>
        {
            ep.PrefetchCount = 0;

            // Map Messages To Queue
            EndpointConvention.Map<IMyCommand>(ep.InputAddress);

            // Configure Consumers
            ep.Consumer<MyConsumer>(provider);
        });
    }));
});

services.AddSingleton<IPublishEndpoint>(provider => provider.GetRequiredService<IBusControl>());
services.AddSingleton<ISendEndpointProvider>(provider => provider.GetRequiredService<IBusControl>());
services.AddSingleton<IBus>(provider => provider.GetRequiredService<IBusControl>());
services.AddSingleton<IHostedService, BusService>();

MyConsumer
public class MyConsumer : IConsumer<IMyCommand>
{
     private readonly ISomeAsyncDisposable _disposable;

     public MyConsumer(ISomeAsyncDisposable disposable)
     {
          _disposable = disposable;
     }

     public async Task Consume(ConsumeContext<IMyCommand> context)
     {
          // Removed for brevity
     }
}

Based on this 426, it seems like IAsyncEnumerable was never added to the interface, but instead added to the implementation. So they're expecting a cast to IAsyncEnumerable then call DisposeAsync()

MassTransit will not manage the lifecycle of your dependencies if you're using a container.
In this case, a consumer with an IAsyncDisposable dependency, and I'm guessing you're using the .NET version of the interface, which MassTransit doesn't use or support. Unfortunately, they have the same name.
Seems like you figured out the issue, though.

MassTransit will not manage the lifecycle of your dependencies if you're using a container.

@phatboyg I don't quite follow this. Isn't MassTransit creating a scope and disposing it?

This one: https://github.com/MassTransit/GreenPipes/blob/develop/src/GreenPipes/IAsyncDisposable.cs

MassTransit is calling dispose on the container scope, the container is responsible for calling any disposable methods on any dependent objects.

I see. That's correct, I'm using the .NET version of IAsyncDisposable.  Are there any plans to support it?
As of .NET Core 3.0, it is now a standard interface. The default Microsoft.Extensions.DependencyInjection container supports it. So Masstransit doesn't currently seem to support the full capabilities of the default container. Basically the container is expecting that if it resolves any dependencies implementing IAsyncDisposable (but not IDisposable), the scope is disposed via DisposeAsync(). I would imagine, moving forward, it will become more and more relevant.
My current workaround is to inject an IServiceProvider into my consumer, and then create a child scope to resolve my dependencies from. I later call DisposeAsync() on that scope before finishing the IConsumer<T>.Consume(..) method. Any suggestions on a better workaround?

Your workaround seems to be enough, given what you've stated.
It will take a while before I support the netstandard2.1 features, since they force developers to move to the latest and not everyone is there yet.

Fair enough. Props on a great library!

Thanks!

@phatboyg I was thinking about this again. Would this be doable using multi-targeting to avoid breaking people?
Something like:
#if NETSTANDARD2_1
await using var asyncScope = scope as IAsyncDisposable;
# endif

Why? v7 and beyond of MassTransit uses https://www.nuget.org/packages/Microsoft.Bcl.AsyncInterfaces/ so this shouldn't be an issue.

That doesn't solve this issue. The issue is calling IServiceScope.Dispose(). Instead of casting to IAsyncDisposable, which is what AspNetCore does. The original error here reproduces in MassTransist 7.2.1.
Not wanting to force everyone into Standard 2.1 is perfectly logical. But if multi-targeting solves that issue, does it make sense to add this? This essentially prevents anyone from using consumers properly in DI if they have even a single async disposable dependency (forces you to use service location & manage the scope in every consumer). Admittedly, I don't know if it would add a bunch of maintenance burden to MT though.

I don't plan to multi-target, it's too difficult to deal with honestly.

Converting all the scope providers to use IAsyncDisposable is fairly extensive, but might be doable.

The latest develop NuGet packages should have this properly implemented now.

Thanks! All works for me in my test project now w/ those new changes.

Great, will be in the next release.


fix(tokens): automated token update

Created by the update-tokens GitHub Actions workflow

@brian-patrick-3, the new blur token value is under a Misc. section of the Foundation stories that doesn't currently exist. i think we need new Misc.mdx and Misc.stories.js, but going to wait for you to weigh in on that. maybe we can find another place for it. merging this round for now
cc: @kumarivijeta

:tada: This PR is included in version 2.0.0-next.20 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


End Time not visible for events ending on next day

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/keen-mopsa-wb63ky?file=/src/App.js:788-930
Bug description
When events start on Day 1 and end on Day 2, with showMultiDayTimes enabled:

The day 1 event doesn't show the end time
The day 2 event doesn't show the start time for the event

Expected Behavior
The events should show the correct time even when it is spanning across days, or at least a prop where we can set what needs to be displayed.
Actual Behavior
The day 1 event doesn't show the end time, The day 2 event doesn't show the start time for the event.
react-big-calendar version
1.5.0
React version
18.1.0
Platform/Target and Browser Versions
MacOs
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a react-big-calendar issue and not an implementation issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

The reproduction link you provided doesn't seem to be related to the problem? Which localizer are you using?
The format can be specified with
https://jquense.github.io/react-big-calendar/examples/index.html?path=/docs/props--formats-event-time-range-end-format
and
https://jquense.github.io/react-big-calendar/examples/index.html?path=/docs/props--formats-event-time-range-start-format
I don't see a bug here.


[Feature]: Add more printing option

Is the feature relevant to the Firefox PDF Viewer?
Yes
Feature description
Currently, pages per sheet supports 1, 2, 4, 6, 9 and 16. You're out of luck if you want to make 8 or 10 pages per sheet, etc. Add option to custom page per sheet that will solve the problem
Also, it lacks an option to change page orientation from portrait to landscape.
Other PDF viewers
pdfmake library supports changing page orientation.

This sort of fine-grained control over printing output isn't really possible from JavaScript.

pdfmake library supports changing page orientation.

It's somewhat difficult to understand how a PDF creation library is relevant to this issue.


first preprocess dataset or conceptnet?

Hi YuanFuDao team, thanks for sharing this wonderful work!
I have one question about preprocessing. It seems that:
1- when preprocess dataset, it needs to compute the relation (via compute_feature) from a preprocessed conceptnet
2- when preprocess conceptnet, it needs the vocab (via build_vocab) built from a preprocessed dataset.
This becomes the "egg comes first or chicken comes first" problem... Is there anything I'm missing here? It would be great if you could point out my mistake or give a more detailed description about how to preprocess.
Thanks!

Oops!
You are right, I precomputed the vocabulary at the very early stage of this project and then just assume they already exist.
Sorry for the trouble, I'll fix this issue soon.

Thanks for the quick response!
BTW, it only has an assertion in utils.py for the rel_vocab; however, there should be some code to build the rel_vocab.
# Load conceptnet relation vocabulary
    assert os.path.exists('./data/rel_vocab')

Hope you can fix these soon. Meanwhile I will first try a model without conceptnet part. Thanks!

Awesome! Thanks for the super quick response.
Two last questions. To process RACE dataset. Just need to run again run a script including "preprocess_race_dataset('./data/RACE/')" after
python3 src/preprocess.py conceptnet
python3 src/preprocess.py

right?
For preprocessing RACE, it's using the vocab built from the SemEval dataset, which is kind of small. Is there any logic behind this?
Thank you!

Yes, I used preprocess_race_dataset function for preprocessing RACE dataset.
I also merged two vocabularies based on RACE(only frequent words) and SemEval dataset into one vocabulary, which is ./data/vocab in this repository.  I kind of lost track of that part of the code, hopefully you can implement by yourself.
Thank you.

Hi,
The vocab file you given contains the  vocabularies from both RACE and SEMEVAL datasets, right?
Since I can only generate 12717 vocabs from SEMEVAL dataset.


MongoDB docs 21 to 25 added

Related Issue
Fixed : #906
Description
i have added following MongoDB docs under MongoDB tutorials

Using

complex diagram using mermaid

Java Code example

PHP code for example

4 . Table according to topic

5 . Note

Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

hey @Ajay-Dhangar , updated as per your suggestion you can review it.


Toggle skinnable map layers

Story

As a website admin

I Want to add layers to the map (containing, e.g migration arrows)

So That contextual information beyond just location can be provided

Acceptance Criteria
NB, this is just the proof of concept -> adding these images and determining exactly how they get displayed/not displayed is for a followup

Given there exists an image with arrows that is in exactly the same projection as the map

And the map does not currently show the image
When I click the "show image with arrows" toggle
Then the layer is shown

Given the layer is currently shown

When I click the "show image with arrows" toggle
Then the layer is hidden

Implementation Notes

Checklist:

[ ] I have defined the user story
[ ] I have defined some acceptance criteria
[ ] I have included any implementation notes
[ ] I have included all relevant links

We need a map and migration arrows in order to execute this.

I'll get on that

@munozirianni1988 - let's discuss this on Wednesday this week and see how we're going. It's currently in the backlog but I see that it's important to the artist and the PO


fix: unhandled error thrown by busboy

As per, https://fetch.spec.whatwg.org/#dom-body-formdata, TypeErrors are expected to be thrown. Busboy errors were being unhandled.

@mrbbot FYI, from Nodejs v16.19.0 onwards, the undici is included with the multipart/form-data parser.


Added metric and test for NER but encountering error while executing tests for NER

Reference Issues/PR
Encountering issues when executing tests for NER. Getting the error - "No module named deepeval" despite having deepeval installed on the system
What does it implement/fix?
The pull request serves as a feature enhancement to the current library by adding a metric and test cases for Named Entity Recognition which is a popular NLP task.
Any other comments?
None
I would like to apologize for the issues., this is my first time contributing to an open-source project. If anyone is interested in collaborating with me on this task, feel free to drop me a message.

@RigvedRocks hey!
First of all, thanks for the PR and no worries at all! You can come join our discord, we have a channel for contributors were we help and learn from each other: https://discord.com/invite/a3K9c8GRGt
Here are some questions and suggestions for you, but would prefer if we can talk on discord:
Are you encountering the error when running test_ner.py?
A thing that you can try is to set up poetry, which manages dependencies. You are facing this problem likely because your machine is pointing to the wrong deepeval on your local computer, and poetry can fix this.

Head over to your root directory in deepeval
Run poetry shell
Run poetry install

(you will need poetry installed)
Also, I noticed that you're running the wrong metric in test_ner.py.
Last thing, in the NERMetric implementation, change:
self.success = score > self.minimum_score

to
self.success = score >= self.minimum_score

Thank you for your response, Mr. Ip. I am looking forward to your
suggestions and will fix them as soon as possible. Thank you for sending me
the invite link to the Discord server.
Thank You
Regards
Rigved Alankar
On Sat, Dec 16, 2023 at 12:26 AM Jeffrey Ip @.***>
wrote:

@RigvedRocks https://github.com/RigvedRocks hey!
First of all, thanks for the PR and no worries at all! You can come join
our discord, we have a channel for contributors were we help and learn from
each other: https://discord.com/invite/a3K9c8GRGt
Here are some questions and suggestions for you, but would prefer if we
can talk on discord:
Are you encountering the error when running test_ner.py?
A thing that you can try is to set up poetry, which manages dependencies.
You are facing this problem likely because your machine is pointing to the
wrong deepeval on your local computer, and poetry can fix this.

Head over to your root directory in deepeval
Run poetry shell
Run poetry install

(you will need poetry installed)
Also, I noticed that you're running the wrong metric in test_ner.py.
Last thing, in the NERMetric implementation, change:
self.success = score > self.minimum_score
to
self.success = score >= self.minimum_score
—
Reply to this email directly, view it on GitHub
https://github.com/confident-ai/deepeval/pull/367#issuecomment-1858748762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKNNYVSC2DY63SWZPHEAMSLYJVELVAVCNFSM6AAAAABAXMUQQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJYG42DQNZWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>