Get large messages sent via the ZC API to be received directly using a GET from the new home. (rather than a GET from the old home and forwarding the large message)

Original issue: https://charm.cs.illinois.edu/redmine/issues/69

On systems that communicate large messages via RDMA GET operations, it takes the latency of one fewer packet across the network (4 vs 5) to convey a message to a migrated object if the sender passes a GET handle to the home PE, which then informs the recipient PE directly about where to get the message in addition to telling the sender where that object lives for future sends. Where available, pass a GET handle along with the location request to enable this.

Original date: 2016-09-23 16:07:11

Nitin, Vipul: Your recent work on RDMA support probably makes this and #68 easy to implement as well.

Original date: 2019-03-20 21:41:26

I think Phil is trying to basically describe a “zero-copy” based transfer scheme for a large message (whose target is a migrated object) from the original object location (PE) to the new object location (PE). Is that right?
I'm guessing that we currently use a copy-based scheme for forwarding the message from the original PE to the new PE?

Original date: 2019-04-03 20:05:10

Juan and I had a discussion about this feature on slack.
We finally understood this feature to some extent.
Let's say a chare array element 'c' migrates from PE 'y' to PE 'z'. Currently, if another chare array element 'b' on PE 'x' wants to send a large message to 'c', it sends the message to PE 'y' and then PE 'y' forwards it to PE 'z'. With this feature, chare array element 'b' on PE 'x' will not send the entire large message to PE 'y', it will only send the buffer information (GET handle) to PE 'y' and then PE 'y' will forward the buffer information (GET handle) to PE 'z'. Following that, PE 'z' will perform a GET operation to get the data directly from PE 'x'. This ends up saving the latency of one hop for the large message in addition to avoiding the memory allocation on PE 'y'.

Original date: 2019-04-03 20:49:40

This feature is only relevant for use cases of the Zero copy Entry Method API and not the regular API.
This API basically impacts use cases where a sender (PE 'x') sends a GET handle to the original home (PE 'y') and instead of the original home rgetting the entire message onto PE 'y' and then forwarding the large message to PE 'z' (the new home of the migrated chare), with this feature, PE 'y' just forwards the GET handle and PE 'z' performs the Rget.

Original date: 2019-04-10 19:51:41

Since this impacts a very small use case (where ZC API is used in conjunction with load balancing) and the impact is potentially small (only the iteration after load balancing), I am not including this in 6.10.


Add the Thai Speech Emotion Dataset

Add the Thai Speech Emotion Dataset
Dataset: https://github.com/vistec-AI/dataset-releases/releases/tag/v1
Paper: --

    "THAI SER": {
        "Year": 2021,
        "Content": "The recordings are 41 hours, 36 minutes long (27,854 utterances), and were performed by 200 professional actors (112 female, 88 male).",
        "Emotions": "5 main emotions assigned to actors: Neutral, Anger, Happiness, Sadness, and Frustration.",
        "Format": "Audio",
        "Size": "12 GB",
        "Language": "Thai",
        "Paper": "--",
        "Access": "Open",
        "License": "CC BY-SA 4.0",
        "Dataset-link": "https://github.com/vistec-AI/dataset-releases/releases/tag/v1",
        "Paper-link": "--",
        "License-link": "https://creativecommons.org/licenses/by/4.0"
    }


Unable to instal node modules after cloning repo

Hi,
I cloned the repo and followed the installation instractructions from the README.

git clone git@github.com:openshift-quickstart/openshift-ghost-quickstart.git
cd openshift-ghost-quickstart.git
npm install (I have also tried npm install --production)

I get the following unmet dependency errors:
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN package.json<EMAIL_ADDRESS>No description
npm WARN unmet dependency /Users/davekingsnorth/Desktop/playground/ghost-sandbox/openshift-ghost-quickstart/node_modules/busboy/node_modules/dicer requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/davekingsnorth/Desktop/playground/ghost-sandbox/openshift-ghost-quickstart/node_modules/busboy/node_modules/readable-stream,
npm WARN unmet dependency which is version 1.1.13-1

Any help would really be appreciated. I have successfully installed installed the ghost quickstart on OpenShift but I want a dev sandbox to test my theme changes rather than deploying blind every time I commit on my main repo.
Thanks!

Hi @Hanked,
This is just a warning, it does not impact your installation.

I'm having the same issue, following the same steps as OP, except my installation just fails silently and nothing gets installed. I'm running npm install --production, and I'm using Node v.0.10.40 on OS X. The app runs fine on the OpenShift server, so I know the install is working remotely.
I also added the -v flag hoping to gain some more insight into what was happening, but the only thing I got with that was a single line that said: 1.4.28
Any help here would be greatly appreciated!


HL-812 | E2E tests for submitting applications manually from handler UI

Description :sparkles:
Rebased from develop -> nginx-quality-of-life -> next-serve-https. I'll merge these before applying HL-812!
NOTE:

I've run tests with TestCafe 3.0.1 (1.x.x won't work with MacOS Ventura)

Be sure to include --experimental-proxyless with TestCafe if using newer version

I've run tests by using compose.benefit-backend.yml (backend container only)
I've run tests by using yarn bf-hdlr:start (build version of the app) on port 3100 with HTTPS on

You can get the certs from local-proxy container at /etc/nginx/localhost.key and /etc/nginx/localhost.crt

Tests are now filling all the mandatory fields. All of the other combinations have been left unfilled (all features behind the checkboxes). The test fails because build version crashes on form submit (HL-864).

Codecov Report

Merging #2149 (53744b8) into develop (0438c10) will increase coverage by 14.39%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           develop    #2149       +/-   ##
============================================
+ Coverage    30.63%   45.02%   +14.39%
============================================
  Files          815       32      -783
  Lines        16767      462    -16305
  Branches      3643      155     -3488
============================================
- Hits          5136      208     -4928
+ Misses       11029      244    -10785
+ Partials       602       10      -592

see 783 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

EMPLOYER branch is deployed to platta: https://kesaseteli-pr2149.dev.hel.ninja :rocket::rocket::rocket:

API branch is deployed to platta: https://kesaseteli-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

YOUTH branch is deployed to platta: https://youth-pr2149.dev.hel.ninja :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://handler-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://handler-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://kesaseteli-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://youth-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://kesaseteli-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-pr2149.dev.hel.ninja :rocket::rocket::rocket:

YOUTH branch is deployed to platta: https://youth-pr2149.dev.hel.ninja :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://handler-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://handler-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://kesaseteli-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://youth-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://kesaseteli-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

HANDLER branch is deployed to platta: https://handler-pr2149.dev.hel.ninja :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-pr2149.dev.hel.ninja :rocket::rocket::rocket:

YOUTH branch is deployed to platta: https://youth-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://handler-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://kesaseteli-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://youth-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://kesaseteli-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

YOUTH branch is deployed to platta: https://youth-pr2149.dev.hel.ninja :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://handler-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://helsinkilisa-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

TestCafe result is success for https://kesaseteli-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

HANDLER branch is deployed to platta: https://helsinkilisa-ui-handler-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://youth-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

API branch is deployed to platta: https://kesaseteli-pr2149.api.dev.hel.ninja/healthz :rocket::rocket::rocket:

YOUTH branch is deployed to platta: https://youth-pr2149.dev.hel.ninja :rocket::rocket::rocket:

EMPLOYER branch is deployed to platta: https://kesaseteli-pr2149.dev.hel.ninja :rocket::rocket::rocket:

APPLICANT branch is deployed to platta: https://helsinkilisa-ui-pr2149.dev.hel.ninja :rocket::rocket::rocket:

TestCafe result is success for https://helsinkilisa-ui-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://kesaseteli-pr2149.dev.hel.ninja :laughing::tada::tada::tada:

TestCafe result is success for https://youth-pr2149.dev.hel.ninja :laughing::tada::tada::tada:


Prefabs not working correctly

When adding a decal to a prefab the Editor Tool does not recognize static meshes, if you deselect All static meshes and apply it to a mesh manually it will render correctly at first, but after re-opening the prefab or placing it in a scene the decal will disappear.
Steps to reproduce:

Create an empty game object as a prefab
Add a mesh object
Add a Driven Decal Asset
Project the mesh
Close the prefab tab
Re-open the prefab Tab
The decal is gone.

Thank you for the report. I've been able to reproduce the issue, and will work on fixing it. It's definitely something that needs correcting!

Hi,
I've got the same problem with prefabs not updating and decals disapearing.

Can confirm it doesn't work with prefabs & that prefabs instantiated it also breaks with. This might prevent me from using it on our project unfortunately ;-;

This is probably because the generated decal mesh is not properly serialized and saved by Unity when prefab is closed.
The workaround to similar issues seems to be to use some sort of simple serialization to store the mesh data in the prefab. Example: https://forum.unity.com/threads/solution-save-a-prefab-with-a-generated-mesh-es-without-creating-assets.463496/
I'll test it and report back.

this work, but displayed with wrong rotation of mesh (so don't work correctly)


Revisiting the request to create NetCDF "classic" files with large dims and vars count

This is reopening the discussion that was in #148 (request increase in NC_MAX_DIMS and NC_MAX_VARS).
I would like to revist the discussion in #148 with hopefully a new proposal that will not break any existing applications and give exodus users the options to use the "classic" formats and in particular the option to use the PNetCDF output format if desired for parallel I/O performance.
Background
As you are probably aware, Sandia's exodus database uses netcdf, but for exodus, we need to increase the values of NC_MAX_DIMS and NC_MAX_VARS to store reasonably-sized models. In the past, this "locally-modified" netcdf library was OK; however, recently, the use of exodus has spread to more projects and people have started to provide binary packages for Ubuntu and other systems.
Although the netcdf-4 non-classic format solves a portion of this issue, we would still like to be able to use the classic format and also use the PNetCDF library for parallel I/O.  Although HDF5 is very capable and performant, there are still areas where we want to use PNetCDF.
Proposal
I had previously proposed increasing the values of NC_MAX_DIMS and NC_MAX_VARS, but that proposal has backwards compatibility issues in that it can cause memory issues in existing applications and therefore cause an application to break when linked to the modified library.
My current proposal is as follows:

Provide an additional flag, 'NC_IGNORE_MAX_VARS_DIMSthat would be passed through thecmodeargument toex_createandex_open` (read/write). Open to better name.
For writing, as the application is creating the file, the library would behave normally until either the NC_MAX_DIMS or NC_MAX_VARS values were exceeded by a nc_def_dim or nc_def_var call.
IF the calling code had not passed the NC_IGNORE_MAX_VARS_DIMS flag, then the library would error out as normal; no difference in behavior compared to current library.
IF the calling code did pass the NC_IGNORE_MAX_VARS_DIMS flag, then the library would modify the file type (the x in the CDFx file signature).  One option would be to change the 1,2,3 to A,B,C or some other convention.  Basically, any file in which the number of dims or vars defined in the file exceeded the default NC_MAX_DIMS or NC_MAX_VARS values would no longer advertise itself as a valid NetCDF classic file.
For reading, if the application is linked to a library that existed prior to this change, the open of the file would fail with NC_ENOTNC "not a netcdf file" error.  An entry on the NetCDF FAQ page could be added which would tell users something about the issue so they could google the problem and find a discussion of the issue.
For reading if the application is linked to a library that contains this change, a better error message could be output which would tell the user that this is an "enhanced" or "nonstandard" or some other good message and tell the user what needs to be done to enable the reading of this file.  This is probably similar or better than the current behavior when a user presents a netcdf-4 NetCDF file to an application that was compiled without netcdf-4 support.  Or, in other words, the current library has times when an application can not read a "valid" NetCDF file that was created with a differently configured NetCDF library.
For reading if the application is linked to a library the contains this change and the application has passed the NC_IGNORE_MAX_VARS_DIMS flag to the nc_open call, then the library correctly opens the file and converts the x of the CDFx back to the correct value (1,2,3,...)  which the file would have been had it not exceeded the NC_MAX_DIMS or NC_MAX_VARS values.

** Summary **
Hopefully this proposal addresses all or at least most of the objections to the previous proposal.  Most importantly, I think that this proposal does not violate the strict backwards compatibility guarantee.
It also does not create files that an "unknowing" application would try to read and then break due to the dims or vars count exceeding the standard values.  The only way that the application would be able to open the file successfully would be to pass the NC_IGNORE_MAX_VARS_DIMS flag to the nc_open call -- basically advertising that they are willing and able to handle a file with unlimited dims and vars.  This is similar to reading a non-classic netcdf4 file which also has no limits on the dims and vars count.
If you think that this is worth pursuing, or would like to see a more concrete implementation, I will be willing to create a pull-request so we can discuss concrete details instead of vapor-ware.
@nmhamster, @wkliao

In thinking about this I propose instead the following:

We add a new netcdf API function wit signature something like this:
int_nc_inq_all(int ncid, int *ndimsp, int *nvarsp, int unlimdimidp, int ngroups)
which will return the total numer of dimensions, vars, unlimiteddimensions, and groups
defined anywhere in the file whose root id is ncid. [?should we also allow asking
for total # of attributes as well?]. Any *n arguments that are null are ignored.
This function would allow users to allocate fixed space conforming to the specific file.
we eventually eliminate NC_MAX_VARS and NC_MAX_DIMS -- this is AFAIK ok because
we no longer use these internally in the library.

For the new API, nc_inq_all, proposed by Dennis, isn't this already covered by nc_inq and its family APIs?
I am glad to hear that Dennis is OK to eliminate NC_MAX_VARS and NC_MAX_DIMS.
Here are my 2 observations.
First, NC_MAX_DIMS/NC_MAX_VARS/NC_MAX_ATTRS are only defined in NetCDF library. The specifications of CDF file formats do not impose such limits. Strictly speaking, files containing more dimensions, variables, and attributes than these limits are still valid CDF files.
Secondly, regarding to backward compatibility, I do not see lifting these limits (or even eliminating them) should create a problem. By definition, the fact a software of newer version can still work properly on the data created by older versions is backward compatible. On the other hand, a file created by a newer version is not guaranteed to be readable by the older versions.
From the discussion in #148, I can understand the issue concerned by Ward and Dennis. As a software developer, sometimes all I can do is to encourage our users to use the latest release.
My personal preference is also to eliminate these limits (same as Ed and Dennis). I am glad to hear from Dennis that NetCDF no longer uses them and OK with their elimination. I also have revised PnetCDF to remove their usage internally. (An exception is NC_MAX_NAME which requires more work, as there are still many statically allocated arrays of that size.)
CC to people also involved in discussion.
@roblatham00
@edhartnett
@jedwards4b

BTW although I advocate for the elimination of NC_MAX_VARS and NC_MAX_DIM, I use NC_MAX_NAME everywhere and if you took that away, I would just have to define it myself.
I think NC_MAX_NAME is not a problem for users, so I hope we will not be eliminating that.

Both NC_MAX_NAME and NC_MAX_VAR_DIMS would remain.  They are still used throughout the code and are not an issue for the exodus uses of NetCDF.   I don't need for NC_MAX_ATTRS to be unlimited, but for consistency it would probably be good to treat it the same as NC_MAX_DIMS and NC_MAX_VARS

I think that the nc_inq_ndims() and similar return the number of dims per group instead of per file as Dennis proposed.  Since the NC_MAX_VARS and NC_MAX_DIMS changes only affect "classic" files which only have one group, the nc_inq_ndims() function should provide the needed information, but it may be useful to also have the nc_inq_all function for other uses.

Agreed, NC_MAX_NAME and NC_MAX_VAR_DIMS must remain. I am only considering the
per-file global counts NC_MAX_VARS and NC_MAX_DIMS.

I would like to point out that the current value set for NC_MAX_NAME is 256. I will not be surprised in the future, if not soon, some application users challenge this limit. If these limits must be enforced, I suggest we come up with a mechanism to leave a room for future extension, something similar to what Greg proposed. Note that the current implementation of NetCDF and PnetCDF treat values beyond the limits as an error and terminate the function call.
For PnetCDF, I plan to start minimizing the use of these constants if not possible to eliminate them. Of course, my top priority is the consistency to NetCDF.

Eliminating NC_MAX_NAME is probably not possible. There is no way for the API to tell you in advance what the length of an object's name is. So when I do an nc_inq_varname(), how do I know how much space to allocate to hold the name?
It would be fine if it is increased, but difficult to do without.

In retrospect, this is a flaw in the API. When asking for a name, it should have been possible
to specify the size of the name buffer and/or get the size of the name. There are probably
other, similar, flaws elsewhere.

One could imagine a new function which got the maximum name size in a file...

That is what is done in the exodus library.  We write an attribute to the file when it is closed which is the size of the maximum name that was stored in the file.  The reader of the file can then query that value and then allocate the right amount of memory for all reads of name data.   We currently have a hard limit of NC_MAX_NAME.

[continued]  The user can also specify a maximum name length on read and the library will truncate any names that are larger than that length.   This is done for backward compatibility since we used to have a maximum name length of 32 characters in exodus library.  The default name length is set to 32 characters, so any client that does not change the maximum name length after opening the file will get the old maximum of 32 characters -- this way we don't exceed any buffers that the client may be using to store the names.  Newer clients query the library for the maximum name length in the file and then set the maximum length for returned strings to that value so no names are truncated.

The consensus seems to be that we will work toward eliminating NC_MAX_VARS, NC_MAX_DIMS and NC_MAX_ATTRS' .  The NC_MAX_NAME` limit will be retained.
There will be a function int nc_inq_all(int ncid, int *ndimsp, int *nvarsp, int *nunlimdimidp, int *ngroups) which will return the total number of dimensions, variables, unlimiteddimensions, and groups anywhere in the file whose root id is ncid if the corresponding argument is non-null. (For example, if *nvarsp is NULL, then nvarsp will not be set).
The counts are cumulative counts over all groups.  For example, if the file contains 5 groups and each group has 3 dimensions and 4 variables, then *ndimsp=15, *nvarsp=20, and *ngroups=5.
If a client is only interested in per-group counts, then they can use the existing int nc_inq(int ncid, int *ndimsp, int *nvarsp, int *nattsp, int *unlimdimidp); function or they can use the nc_inq_dimids and nc_inq_varids to query the dimension and variable counts individually.

My main question at this point is what is the path to that end?  Is it permissible to just eliminate the checks for exceeding NC_MAX_VARS and NC_MAX_DIMS or does there need to be an intermediate step in which the client can specify that they want the "new" behavior (i.e. no limits) both on write and on read?
I am fine developing a patch for the above, but would like to have a consensus on what the intermediate step(s) need to look like.

I believe the checks against NC_MAX_DIMS and NC_MAX_VARS should simply be removed.
I don't think this violates the promise of netcdf backward compatibility.


Illustrations of usefulness for each function

The explainer is incomplete.
Each function’s section needs at least one code-block example illustrating its usefulness (for example, identity as the default value of an optional parameter). Ideally, this would be taken from real-world code (and preferably from real-world JavaScript code).

@theqp: Thanks for the comment.
When I included that React example in the pipe proposal’s explainer, I did it with the intent of demonstrating a deeply nested expression that actually wasn’t amenable to a pipe function. s => chalk.dim($ ${s}, "node", args.join(" ")) is still kind of difficult to read—I would split it up further into two arrow functions (s => $ ${s}ands => chalk.dim(s, …)`). But several Committee representatives have had concerns about encouraging copious inline arrow functions in pipes (see tc39/proposal-pipeline-operator#221).
So, with this proposal, I will try to emphasize to the Committee that flow can usefully compose cached unary functions, including unary arrow functions, in a single function allocation, which can then be reused.
For this reason, I am less certain about the Committee accepting pipe in addition to flow. I would prefer real-world examples that would not involve allocations of many new inline functions, although I have been having difficulty finding such examples.


Volume controls don't seem to work

On my Sonos system, the app doesn't seem to be able to control the volume reliably. Sometimes clicking the mute button does nothing, sometimes it mutes but won't unmute. The slider doesn't do anything when dragged. (It doesn't even move.) Clicking on either side doesn't do anything either.
I checked the console output for the background page and the app page and there's not much there except some complaint from react like this:
ui.js:37899 Warning: Any use of a keyed object should be wrapped in React.addons.createFragment(object) before being passed as a child.

One odd thing is that the <body> element of the app page seems to have the react-draggable-dragging class assigned multiple times:
<body class=" react-draggable-dragging react-draggable-dragging react-draggable-dragging react-draggable-dragging">

I have app version 0.11 on a Pixel v1:

ChromeOS Version 45.0.2454.98 (64-bit)
Platform 7262.57.0 (Official Build) stable-channel link
Firmware Google_Link.2695.1.156

My Sonos is running:

Version: 5.5 (build 29693090)
Hardware Version: <IP_ADDRESS>-2

The additional react-draggable-dragging classes are added every time I try to drag the volume control. I just verified that by dragging a few more times and watching the classes get added.

Hey, thanks for this very comprehensive bug report. I cannot replicate this in my ubuntu dev setup, looking into ways to test this without buying a chromebook :/

I just read about CloudReady (http://www.neverware.com/) which allows you to install ChromiumOS on older laptops. You may be able to get that to work for you. (I don't know if you can install it on an emulator, but I would guess so.)
I'd like to say that I'd be willing to try to do a bit of debugging, but I have no time. :-(

Thanks for that, will give it a whirl ... in the meantime, I have updated to react 0.14.0 and bumped the draggable component as well. Can you let me know how that performs for you?

OK, the update to version 0.12 makes the volume control work partially.
In my kitchen, I have a pair of Play:1s in a stereo (L + R) configuration. When I use the stereo slider, only the R speaker's volume is increased. Likewise clicking on the mute button only mutes the R speaker.
Also, the warnings above have changed to:
ui.js:36126 Warning: Each child in an array or iterator should have a unique "key" prop. Check the render method of `BrowserList`.  Check the render method of `ZoneGroupList`.  Check the render method of `ZoneGroup`.  Check the render method of `QueueList`.  Could you send me your topology info.:
http://{sonos player ip}:1400/status/topology

Zone Players

Zone Name
Coordinator
Group
Location
UUID
Version
MinCompatVer
Compat
WiMode
HasSSID
WiFreq
WiEna
BeEx

Kitchen
false
RINCON_B8E937B79BEA01400:1
http://<IP_ADDRESS>:1400/xml/device_description.xml
RINCON_B8E937B797EC01400
29.6-93090
28.0-00000
1
1
2412
1
0

Kitchen
true
RINCON_B8E937B79BEA01400:1
http://<IP_ADDRESS>:1400/xml/device_description.xml
RINCON_B8E937B79BEA01400
29.6-93090
28.0-00000
1
1
2412
1
0

Hope that helps.

BTW, I verified that this is still happening and that it wasn't a one-off. However I did have to start the app twice because the first time the volume didn't respond at all.

I just added some debug info while the app was running in Chrome. In particular, I had it print the URL that the sonos.request() method is using. Turns out, it's using:
http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control

when updating the volume but
http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control

when querying state. This seems very odd. I'll try to get stack traces for these.

Here's one while updating the position:
uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetPositionInfo" Error: for stack trace
    at Sonos.request (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:40986:93)
    at Sonos.currentTrack (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:41173:16)
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:40508:10
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:41890:12
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:41004:13
    at Object.parseString (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:42533:5)
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:40999:45
    at XMLHttpRequest.xhr.onreadystatechange (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:42476:5)

Here's one while setting the volume:
uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace
    at Sonos.request (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:40986:93)
    at Sonos.setVolume (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:41760:9)
    at Object.setPlayerVolume (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:37116:9)
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:39367:46
    at Array.forEach (native)
    at VolumeControls._changeGroupVolume (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:39348:9)
    at chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:39637:17
    at debounced (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:8541:27)
    at VolumeSlider._onDrag (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:39639:4)
    at module.exports.React.createClass.handleDrag (chrome-extension://cojfokmeikpnickdpoopoockilamcmoc/js/ui.js:17040:16)

and even though I see in the stack trace that there is a forEach, I only see <IP_ADDRESS> being updated. Here's a log with the stack traces collapsed:
uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetPositionInfo" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetTransportInfo" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetTransportInfo" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetPositionInfo" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/RenderingControl/Control
action = "urn:schemas-upnp-org:service:RenderingControl:1#SetVolume" Error: for stack trace(…)
ui.js:40986 uri = http://<IP_ADDRESS>:1400/MediaRenderer/AVTransport/Control
action = "urn:schemas-upnp-org:service:AVTransport:1#GetPositionInfo" Error: for stack trace(…)

Oh, this is interesting. I put a debug statement in _changeGroupVolume to dump the state and this is what is there:
_dragEndTimer: 435
_hideTimer: 436
_reactInternalInstance: ReactCompositeComponentWrapper
context: Object
getDOMNode: (...)
isMounted: (...)
props: Object
refs: Object
replaceProps: (...)
replaceState: (...)
setProps: (...)
state: Object
  dragging: false
  isExpanded: false
  players: Object
    <IP_ADDRESS>: Object
      muted: false
      name: "Kitchen (L + R)"
      volume: 17
      __proto__: Object
    __proto__: Object
  __proto__: Object
updater: Object
__proto__: VolumeControls

Note that there is only one entry in the state.players object.

Indeed ... which is not the coordinator ...

Maybe have a look at https://github.com/pascalopitz/sonos-controller-for-chrome/blob/master/src/ui/stores/VolumeControlStore.js#L35 and see what's coming through there.
If things are already out of whack there, this is the place to look:
https://github.com/pascalopitz/sonos-controller-for-chrome/blob/master/src/ui/services/SonosService.js#L471

It's inconsistent at VolumeControlStore. Sometimes it has the right one (and then the volume works as expected) and sometimes it doesn't. Here's an example where it doesn't:
behindwifiext: "0"
bootseq: "17"
channelfreq: "2412"
coordinator: "false"
group: "RINCON_B8E937B79BEA01400:1"
hasconfiguredssid: "1"
host: "<IP_ADDRESS>"
legacycompatibleversion: "24.0-0000"
location: "http://<IP_ADDRESS>:1400/xml/device_description.xml"
mincompatibleversion: "28.0-00000"
name: "Kitchen (L + R)"
uuid: "RINCON_B8E937B797EC01400"
version: "29.6-93090"
wifienabled: "1"
wirelessmode: "1"

Looking at SonosService.excludeStereoPairs:
	excludeStereoPairs: function excludeStereoPairs(zones) {

		// TODO: find a better place for this
		zones.forEach(function (z) {
			var matches = REG.exec(z.location);
			z.host = matches[1];
		});

		return (0, _lodash2['default'])(zones).groupBy('name').map(function (g) {
			// TODO: what happens when a sub is added?
			if (g.length === 2) {
				g[0].name = g[0].name + ' (L + R)';
			}
			return _lodash2['default'].findWhere(g, 'coordinator', 'true') || g[0]; // <== PROBLEM
		}).value();
		return zones;
	}

The problem line, as pointed to above is:
_lodash2['default'].findWhere(g, 'coordinator', 'true') || g[0]

because it turns out that findWhere always returns undefined, so g[0] is taken.
From the console, this gives the correct results:
_lodash2['default'].findWhere(g, {'coordinator': 'true'})

Makes me think that the lookup {'coordinator': 'true'} might be sometimes not a string, or captialized or something. Can you just dump out whatever comes through as zones?

It's just that the call is wrong--you forgot to wrap the 'coordinator', 'true' in curly braces.

Doh! Yes, you're right. Fixed and pushed to chrome store etc ...

Just got the update to version 0.14. Thanks for the shout-out in the contributors section. :-)
So far, it seems to work great. I'm going to close this and I'll file a new bug if something else comes up.

Thanks for helping out!


Does not work on Leaflet v1.0.0-rc1

The following Javascript error occurs...
 L.Path.prototype.initialize is undefined

A Solution is on the way https://github.com/lethexa/leaflet-tracksymbol/pull/5

Some Grunt-tests fail with leaflet 0.7:
test/test_with_invalid_heading.html
test/test_with_all_valid.html
Please check ...

I don't think an updated version for a release-candidate is necessary.

I will close the issue because there is a working version for leaflet v1.0 now.


Fix newline breaks in markdown

Fixes https://github.com/NuGet/NuGetGallery/issues/8642

Hey @CyberAndrii, thanks for taking the time to investigate this and provide a fix. @lyndaidaii will review this PR and support you in any PR comments that come up.

Looks great to me. Already verified changes. Overall, it improved how existing readme display, made prettier. However, It might impact few existing readme. I think it's great contribution and encourage users to adapt best practice of markdown readme.
Let me get another reviewer before we merge. @CyberAndrii  thanks for great work!


std::search called in search() when compiling with C++14

I'm trying to compile one of the examples of my cuda-api-wrappers library, which uses string-view-lite, using C++14 instead of C++11 like I was compiling it so far. I get:
... snip ...
/home/eyalroz/src/mine/cuda-api-wrappers/examples/other/jitify/string_view.hpp:572:23: error: call to non-‘constexpr’ function ‘_FIter1 std::search(_FIter1, _FIter1, _FIter2, _FIter2) [with _FIter1 = const char*; _FIter2 = const char*]’
  572 |     return std::search( haystack.begin(), haystack.end(), needle.begin(), needle.end() );
      |            ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
... snip...

and, indead, std::search is not constexpr before C++20.

Oh, actually, maybe it's a dupe of my other bug.

You can try defining __OPTIMIZE__ before including string_view.hpp or build with optimizations enabled.
https://github.com/martinmoene/string-view-lite/blob/5b1d95fe2c0ee18e654876487898b9a423a954db/include/nonstd/string_view.hpp#L554-L576
https://stackoverflow.com/questions/14618249/is-there-a-builtin-macro-defined-when-optimisation-is-enabled-in-clang

@eyalroz, @oliverlee
The unconditional constexpr in the non-optimize case of constexpr const CharT* search() should perhaps be replaced with something like nssv_constexpr20_lib.
I'd like nssv_HAVE_CONSTEXPR_20_LIB to be not overly restrictive as it may be as show below. Suggestions are quite welcome.
// Presence of C++20 library features:

#define nssv_HAVE_CONSTEXPR_20_LIB  nssv_CPP20_000

// ...

#if  nssv_HAVE_CONSTEXPR_20_LIB
# define nssv_constexpr20_lib  constexpr
#else
# define nssv_constexpr20_lib  /*constexpr*/
#endif

// ...

#else // OPTIMIZE

// non-recursive:

template< class CharT, class Traits = std::char_traits<CharT> >
nssv_constexpr20_lib const CharT* search( basic_string_view<CharT, Traits> haystack, basic_string_view<CharT, Traits> needle )
{
    return std::search( haystack.begin(), haystack.end(), needle.begin(), needle.end() );
}

#endif // OPTIMIZE

I think it makes sense to remove the branch on _OPTIMIZE_. By removing the call to std::search, this would allow use in constant expressions with debug builds.

However, a possibly recursive search, possibly exhausting the stack doesn't look like the right thing to do at default.

Ah right. In that case, what about replacing the second implementation with something like:
template< class CharT, class Traits = std::char_traits<CharT> >
nssv_constexpr14 const CharT* search( basic_string_view<CharT, Traits> haystack, basic_string_view<CharT, Traits> needle )
{
    while (needle.size() <= haystack.size()) {
        if (haystack.starts_with(needle)) {
            return haystack.cbegin();
        }

        haystack = basic_string_view<CharT, Traits>{ haystack.begin() + 1, haystack.size() - 1U };
    }

    return haystack.cend();
}

Although I suppose it would make sense to allow a user to select std::search via a preprocessor macro (or vice-versa).

@oliverlee Thanks for your suggestions!

Thanks for sharing your implementation of string view!


analysis process is killed without any error message

The analysis process is being killed without an explicit message. I have 8GB memory on the container that I am running the analysis on.
Do you have any ideas?
...
Unknown instructions:
Ldelema: 1
Ldelem_U2: 1
Mul_Ovf: 1
Conv_Ovf_U2: 1
Conv_Ovf_U4: 1
Conv_Ovf_U8: 1
Ldelem_U4: 2
Conv_Ovf_U1: 2
Initobj: 4
Stobj: 4
Localloc: 4
Ldelem_U1: 4
Constrained: 15
Ldind_U1: 23
Leave: 25
Ldobj: 32
Ldvirtftn: 37
Unbox_Any: 75
Switch: 114
Ldlen: 115
Leave_S: 437
Throw: 577
Ldftn: 1226
Castclass: 4958

Coverage Statistics:

Method successfully translated: 24774 (76%)
Method partially translated: 7674 (24%)
Instructions translated: 1081979 (68%)
Instructions skipped: 511962 (32%)
======================================

-e You may see 'Unable to parse instruction xxx' above. This is expected as we have not yet translated all the CIL instructions, which follow a long tail distribution. We are continuing to expand our .NET translation coverage.

-e Translation completed. Analyzing...

Logs in /app/infer-out/logs
Killed

Hi Hasan, according to the coverage stats, your code base seems quite big. The failure was probably due to the sheer size. A few things you can try:

Only analyze your code binaries. You may have included some third-party libraries which contain both dll and pdb files. Infer# will also analyze those but there is no value to your code.
Increase the Docker memory size. We have not seen any failures from our own testing on fairly large code bases with 12GB of memory. But each repo is different, it may not be a guarantee.
Try our 1.1 beta release. While are still working on it, a lot of improvements have been made. Either use WSL2 (it has better performance than Docker in general based on our internal testing), or v1.1 Docker beta image:

docker pull mcr.microsoft.com/infersharp:v1.1

@aayc I was using sh to run infersharp.sh using the following command
docker run -it --name infer --hostname infer -v myproject:/myproject mcr.microsoft.com/infersharp:v1.1 sh run_infersharp.sh /myproject

I've changed to bash after your comment, and substitution error is no more. 👍🏻

Ah that makes sense, thank you for letting me know 😁


Why we chose Java to build Airbyte's platform

A question that the community sometimes ask is why we chose Java, over Python, to build the platform.
Please note again that you can build your connector in whatever language you want.

because we were better at writing java than python 😉

Hi, this is more of an internal topic. But I'm curious to know what you think is the answer :D.

Hi there, I'm a Java Developer by profession and I would like to work on this!


GPL-nnn As Traction developers we would like an api v2 endpoint to allow us to retrieve plates, wells, samples and tags

User story
As Traction developers we would like an api v2 endpoint to allow us to retrieve plates, wells, samples and tags because for heron they will be tagging samples with 96 PCR barcodes and we need to know thay they exist and what they are
Who the primary contacts are for this story
Stephen Inglis / Harriet Craven
Acceptance criteria
To be considered successful the solution must allow:

I should be able to make a request to an endpoint on the api/v2 using a list of plate barcodes
The response should include the plate along with the wells (position), samples (name, uuid) and tags (information to be decided).

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

This should already be available, with possibly just needing to extend the tag information. I'll pull out an example query shortly.

http://.../api/v2/plates?filter[barcode= DN550065I,DN550071G&include=wells.aliquots.sample
Picked plates in training and production


Fixing query to fetch latest TuningJobExecutionParamSet

DESCRIPTION
In this PR changes are made to the query to fetch TuningJobExecutionParamSet in the AbstractFitnessManager method. Earlier it was assumed that there will be unique row for each JobSuggestedParamSet in TuningJobExecutionParamSet, it is fixed in this PR.
HOW IT IS TESTED
Testing is done on EI.

LGTM.


Please change the namespace of the Configuration extensions methods

As a convention of .NET Core / ASP.NET Core, the extension method namespace is generalized.
Azure App Configuration (2.0.0-preview-009470001-1371)

Example: Key Vault

I'd like you to change the namespace to match other Configuration libraries.

This is a breaking change. We can still take it at this point but we will need to weigh out the pros and cons on this.

@shibayan Thanks for the feedback. Since this is a breaking change, it might take a while, but we will include it in our next major release for the configuration provider.

@abhilasharora I think that Breaking change is allowed while previewing. You actually have deleted the Watch / WatchAndReloadAll method.
It was quite a breaking change for me

@shibayan I believe we moved from 1.x.x to 2.x.x with that change though? We certainly can make the change when we go to 3.x.x

@jimmyca15 Umm. In general, don't change the major version in preview. It is strange that the SDK version is v2.x.x or v3.x.x in the first release of the service 😕
I think it is important for usability to follow general rules.

@shibayan We just released version 2.1.0-preview of the packages with the namespace change.


Impossible to get Nested Field

Hello,
How can we access on the nested field on angucomplete ? The "dot" are not working.
There is an example of the code :
The JSON data on the $scope.countries : -------
[{
    "id": 1,
    "name": "Senegal",
    "continentName": "Africa",
    "cities": [{
        "id": 1816,
        "name": "Dakar",
        "countriesId": 1
    }, {
        "id": 1820,
        "name": "Saint-Louis",
        "countriesId": 1
    }]
}, {
    "id": 2,
    "name": "Mali",
    "continentName": "Africa",
    "cities": [{
        "id": 1286,
        "name": "Bamako",
        "countriesId": 2
    }]
}]

And the angular view : ---------
<div angucomplete-alt id="activities" placeholder="Where are you going ?" pause="100" selected-object="activitiesSelected" local-data="countries" search-fields="name,cities.name" description-field="name" title-field="cities.name" minlength="1" input-class="form-control2" match-class="highlight">
</div>

Any idea of what wrong ?

cities.name surely won't work because cities is an array.
There is no such thing as cities.name in your object.
And, you can't do cities[0].name as we don't support array notations.

i have a profile.first_name and it dont working :(


Feat/aut722 Rework for a more modular GUI / Settings

This is not ready to merge right now but comments are welcome
Rework for a more modular GUI / Settings.

Removed a lot of redundant code (Session handler, GUI).
Created some dedicated widgets from the widget we were creating from within GUI.
Reduced the amount of code to write to add a new widget to the GUI:

More modular. a list of top, bottom, and left windows are created, the GUI is responsible for calling the draw methods on it.
Adding a widget is to declare it on the global list of widgets that will be used, set added on of the layout lists (top, bottom, left), and add a menu to activate it.
Added visible flag to widget class to avoid creating variable for each created widget
Added title (not same as id..) to the definition of a widget

Very nice WIP!
Are there any particular areas you wanted feedback on? It is a lot of changes so if there is something specific you wanted an opinion on, please let me know and I can provide more specific feedback.
Also, I think it would be a good idea to have a meeting where you can go over the changes that you did once ready. You are using a lot of very nice design concepts and techniques that would be informative for myself to understand more your rational behind and I think educational to myself and all in general.

Thanks for the feedback! indeed the expected feedback was if you feel it's going to the right direction. I will present the modification at the end of the demo meeting next week :)


occupations plot edits

get rid of "Other"
group decision on how to group the other occs

https://docs.google.com/spreadsheets/d/1R1bkAZRz1gOx65D0qxf2moRNWVN5liBRfI4JXv-H5P0/edit#gid=1261644232

Pulling this out also from cath:

Do you think we should change 'number of job adverts' to 'number of job adverts for this occupation' just to make it super clear.
'mid similarity'  doesn't quite sound right. Perhaps 'somewhat similar'
'Select an occupation'
How can we make the 'Any' option clearer?
'Percentage of job adverts that mention this skill at least once' (if that's right!)
Could we make the blue bar area smaller so there's more space for the skill names?

I've addressed most of these in https://github.com/nestauk/ojd_daps_skills/pull/171/commits/ac75e7c3aa30926b1fa9bf9ce05d123d5564abb2 apart from :

How can we make the 'Any' option clearer?

On "Could we make the blue bar area smaller so there's more space for the skill names?" <- I've kind of done this, but found it a bit tricky . where the y line sits is dynamic to how much space the y labels take up (e.g. if they are all short skills then the y axis will be over to the left), but I have increased its maximum spacing, so it does seem to fit in the full skill label now.


Toast message  -as fall back during own node

Only during send and Refresh
Toast - We could not connect to your own node.
Refreshed/Sent using the BitHyve node....
If the fall back was not selected.
We could not connect to your  node.
Try connecting to the BitHyve node-  Go to settings ....

-Verified the second point i.e
[If the fall back was not selected.
We could not connect to your node.
Try connecting to the BitHyve node- Go to settings ....]

-Verified this issue on Dev app v1.4.4(226)


WIP: Ignore EPIPE when sending NewSessionTickets in TLSv1.3

If a client sends data to a server and then immediately closes without
waiting to read the NewSessionTickets then the server can receive EPIPE
when trying to write the tickets and never gets the opportunity to read
the data that was sent. Therefore we ignore EPIPE when writing out the
tickets in TLSv1.3
Fixes #6904

It should probably also handle at least ECONNRESET.
I'm not at all convinced this is a good idea.

I think we should fix the problem that this is trying to fix, I'm just not sure that this approach is the correct one.

I think we should fix the problem that this is trying to fix, I'm just not sure that this approach is the correct one.

I'm not married to this approach if there are alternative suggestions?

I'm not married to this approach if there are alternative suggestions?

Just in case, a better solution might be to issue new tickets right after the server's Finished message, as suggested by RFC 8446:
   Note: Although the resumption master secret depends on the client's
   second flight, a server which does not request client authentication
   MAY compute the remainder of the transcript independently and then
   send a NewSessionTicket immediately upon sending its Finished rather
   than waiting for the client Finished.  This might be appropriate in
   cases where the client is expected to open multiple TLS connections
   in parallel and would benefit from the reduced overhead of a
   resumption handshake, for example.

This is how BoringSSL handles it, and it seems to produce less clutter from application point of view.

Small correction: we currently only do that in 0-RTT. It wasn't for this issue but because otherwise the server sees a read-triggered write and applications are often confused by "unusual" I/O patterns. (Read-triggered write is less avoidable on the client, but 0-RTT is sufficiently complex on the client that asking them to handle I/O in general is minor by comparison.)
I don't think we'd thought about this particular issue much. Half-RTT tickets might paper over it a bit but you could still be sad depending on send buffer sizes and the like.

Just in case, a better solution might be to issue new tickets right after the server's Finished message, as suggested by RFC 8446:

I'm not sure that really fixes the problem (it just hides it). I'm also worried about creating sessions before the client has been authenticated. So I'm not keen on this approach.

It should probably also handle at least ECONNRESET.

I'm not clear on when a write will return ECONNRESET and when it will return EPIPE. A quick search on google didn't particularly help. Are you able to explain?

I took this out of WIP, and updated the use of errno to get_last_sys_error(). I've not changed it to check for ECONNRESET yet (awaiting feedback as per my question above).

See https://stackoverflow.com/questions/2974021/what-does-econnreset-mean-in-the-context-of-an-af-local-socket

I'm not clear on when a write will return ECONNRESET and when it will return EPIPE. A quick search on google didn't particularly help. Are you able to explain?

Thanks for the link. So reading the answer in that link it seems to me that the behaviour is quite platform and socket type dependent. But, at least on Linux for TCP sockets, we could get ECONNRESET in a scenario where we are writing two tickets. The first ticket writes successfully as far as OpenSSL is concerned, but the data has not been written out by the time we get a close from the client. When we then attempt to write the second ticket we would get ECONNRESET.
So, I think you are right - we do need to handle ECONNRESET too.

Fixup commit pushed adding a check for ECONNRESET.

I pushed a fixup commit that pulls the error code checking out into a separate function so that we can easily add new values. I also added a call to clear_sys_error() to make sure we don't pick up any stale state.

Pushed. Thanks.

Using errno values won't work when the BIO is a memory stream, right?

Using errno values won't work when the BIO is a memory stream, right?

Well - yes - but then I don't think there is a problem if the BIO is a memory stream. We shouldn't get these errors writing to a mem BIO because there is no concept of the peer having closed the connection to the mem BIO. The application may then encounter EPIPE if it later attempts to copy the contents of the mem BIO into a socket fd. But that's really within the application's remit to handle. I don't think we can do much about that.


fix(svelte-scoped): dynamically import presetUno

fixes https://github.com/unocss/unocss/issues/4116
Allows using @unocss/svelte-scoped without having to install @unocss/preset-uno

Please see https://github.com/unocss/unocss/issues/4116#issuecomment-2330565846


Update documentation on some details

The package updated a lot, still some documentation seem a little outdated or refering removed sections.
Need to do a little review !

Also have to update the documentation on next release for the server_index.html automatic generation functionnality. Have to update migration documentation on this content too.


docs: fix syntax error in readme

Changed "written on" Go to "written in go"

Thanks @buildcomplete , please follow the Details link in DCO check to pass it firstly.

@buildcomplete Could you please take 5 minutes to sign the commit? Feel free to contact us if you have difficulties to do that. Thanks!

hi @ngjaying, I am not sure how to do it as I am not used to working with GIT, can I do it from github?

ah, unfortunately, github do not have the capability. Would you like to learn a little bit about git. https://github.com/lf-edge/ekuiper/pull/2421/checks?check_run_id=18754132161

Thanks for completing the request, was having difficulty with coorporate network blocking git🪓 will remember the signoff for future.

Thanks for completing the request, was having difficulty with coorporate network blocking git🪓 will remember the signoff for future.

It is not easy to set it up for new commers. You may want to share the tutorial if you can get it work. Appreciate your contribution to the project. Thank you.


Updated cpacs_gen

generated code now provides const correct GetParent() methods
added missing const on CCPACSFuslage::GetLoft()
deprecated CCPACSWingCells::GetParentElement and replaced occurrences in code (breaks const correctness)

Thank you very much. Looks good!


When I type "[", a "" appears before "[" in the markdown source code

Initial checklist

[x] I agree to follow the code of conduct
[x] I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected packages and versions
7.3.3
Link to runnable example
No response
Steps to reproduce
当我输入[的时候，[的前面会出现一个多余的\。
导致无法使用 Link和 Img，因为Link和Img是[开头。在官方的Playgroud中也出现了这个现象
eg:
当我输入
[百度](http://www.baidu.com/)

在md源码中会被解析成
\[百度]\(http\://www\.baidu.com/)

最终导致Link无法显示
Expected behavior
[百度](http://www.baidu.com/)

Actual behavior
\[百度]\(http\://www\.baidu.com/)

Runtime
Electron
OS
macOS
Build and bundle tools
Vite

Meeting the same problem.

Same here.
@Saul-Mirone do you have a plan/idea how to fix this?

Same issue with using Shift+Enter and then typing anything.

Been debugging the last couple of hours and found the issue is with remark-stringify

I'm seeing this as well. If I include a [ in the initial value, it is parsed correctly and links work. But if a [ is typed into the editor, it is escaped with . This means links are impossible to create in the editor, including in the Playground.

I've tested that this issue was introduced in 7.3.3 and still exists in 7.3.6. Reverting to 7.3.2 resolves the problem.

I've tested that this issue was introduced in 7.3.3 and still exists in 7.3.6. Reverting to 7.3.2 resolves the problem.

That part was extracted as @milkdown/plugin-automd. You can install the plugin and enable it.

Is the escaping of square brackets intentional, or is it a bug that will be resolved at some point? I'm writing a plugin for wiki links, which uses square brackets. Is the expectation that I need to write an InputRule that removes escaped brackets?
Is plugin-automd now required for commonmark support? It doesn't appear to be documented, but links no longer work with the commonmark preset alone.

Is the escaping of square brackets intentional, or is it a bug that will be resolved at some point? I'm writing a plugin for wiki links, which uses square brackets. Is the expectation that I need to write an InputRule that removes escaped brackets?
Is plugin-automd now required for commonmark support? It doesn't appear to be documented, but links no longer work with the commonmark preset alone.

The escaped brackets is introduced by remark, it will add escape mark to squre bracket to make sure the brackets won't be parsed as links.

If you really want to inseart a link instead of bracket, there're a lot of methods to do that. For example, an inputrule should be created to convert patterns like [\w+](\w+) into links. You can also use a shortcut to add link to the selected text.

For plugin-automd. It tries to reparse everything in user input and it will introduce other issues. So I split it out from preset-commonmark to let users to decide if they need to use it. I plan to add another inputrule for links but I just didn't get time to do that. I'm working on the @milkdown/components and @milkdown/crepe libraries.

@Saul-Mirone
Thank you! I took your advice and added @milkdown/plugin-automd, then enabled it, which fixed the problem.


File "pipes.pyx", line 1465, in spacy.pipeline.pipes.Sentencizer.from_nlp TypeError: type() takes 1 or 3 arguments

Running the re-ranking example throws the following exception:
Traceback (most recent call last):
  File "/project_scratch/t5_ranking.py", line 2, in <module>
    from pygaggle.rerank.transformer import MonoBERT
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/rerank/transformer.py", line 13, in <module>
    from .similarity import SimilarityMatrixProvider
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/rerank/similarity.py", line 5, in <module>
    from pygaggle.model.encode import SingleEncoderOutput
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/model/__init__.py", line 6, in <module>
    from .encode import *
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/model/encode.py", line 8, in <module>
    from .tokenize import BatchTokenizer
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/model/tokenize.py", line 172, in <module>
    class SpacySenticizer:
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/pygaggle/model/tokenize.py", line 174, in SpacySenticizer
    nlp.add_pipe(nlp.create_pipe('sentencizer'))
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/spacy/language.py", line 302, in create_pipe
    return factory(self, **config)
  File "/project_scratch/users/amir/projects/RAG/dialdoc-task/env/lib/python3.9/site-packages/spacy/language.py", line 1045, in factory
    return obj.from_nlp(nlp, **cfg)
  File "pipes.pyx", line 1465, in spacy.pipeline.pipes.Sentencizer.from_nlp
TypeError: type() takes 1 or 3 arguments

Do you know what's the problem and how can I fix it?
spacy version: 2.2.4
tokenizers version: 0.10.3
transformers version: 4.15.0
Python 3.9.7
Thanks

Install from source? The spacy version is not 2.2.4 in the requirements file

Thanks for your quick reply.
I installed the library by running pip install -r requirements. The issue is due to my python version, the problem was fixed when I switched to python 3.7 :)

Hi @hadifar - Can you try using Python 3.8 and see what happens? cf. https://github.com/castorini/pyserini/issues/916


[WikiJS] Running as wrong user

WikiJS is running as the wrong user, and currently runs as user abc as shown the below printscreen:
https://i.imgur.com/0AKAbFh.png
This then results in the below error:
https://i.imgur.com/oDVpfOF.png

I believe this may be a problem with the linuxserver.io image. Perhaps the issue needs to made over there? https://github.com/linuxserver/docker-wikijs/issues

Perhaps you just need to change the PUID and GPID values. https://docs.linuxserver.io/general/understanding-puid-and-pgid


FORCE option unrecognized

In the docs you have the FORCE option to skip vercels build cache. But when used throws the following warning:

Hey @wottpal it's actually me who made a PR for the force options, you need to specify the excat version of the action and the force options is available from the version 1.9.0 !
I have something like
    steps:
      - uses: actions/checkout@v3
      - uses: actions/setup-node@v3
        with:
          node-version: 16

      - name: 'Deploy'
        uses<EMAIL_ADDRESS>
Hope it helps

Good to know, thanks for the follow-up :)
But may i ask why there is this limitation to specify the version specifically? Just out of curiosity.

Hey @wottpal
Not gonna lie, didn't take the time to look on the why, it was driving me crazy :D
But i should investigate on the why 👁

It seems that no release since the v1.7.1 have made it to the Github Marketplace:

@BetaHuhn Did you make sure to correctly deploy packaged versions to Github? I don't know the exact publishing process myself tbh.

⚠ This issue has not seen any activity in the past 2 months so I'm marking it as stale. I'll close it if it doesn't see any activity in the coming week.

Just preventing the bot to close this

⚠ This issue has not seen any activity in the past 2 months so I'm marking it as stale. I'll close it if it doesn't see any activity in the coming week.

push

⚠ This issue has not seen any activity in the past 2 months so I'm marking it as stale. I'll close it if it doesn't see any activity in the coming week.

push

Hey everyone, sorry for the very long delay!
The latest release should now be available on the Marketplace.

Did you make sure to correctly deploy packaged versions to Github?

The problem is that GitHub doesn't automatically publish the latest release to its Marketplace, you have to manually edit the release (without changing anything) for it to show up.


[Bug Report] Incorrect Copy Result When Using fp32_dest_acc with dataflow cb

Describe the bug
I discovered the bug while testing because I wanted to try using the dataflow CB instead of intermed.
There are no issues when using bfloat16 tensors.
However, when using the FP32 CB and enabling fp32_des_acc_en, the issue occurs.
I modified the existing moreh_softmax to create a simple test code for testing purposes.
The test code copies the input of size 1 tile to the temporary CB and then copies it again to the output CB.
In this case, using intermed with the temporary CB works without issues, but using the dataflow CB results in incorrect output values.
To Reproduce
Steps to reproduce the behavior:

Go to https://github.com/tenstorrent/tt-metal/tree/hyungsuk/bug_dataflow
run cmd pytest tests/ttnn/unit_tests/operations/bug.py
See Result

input tensor([[0., 3., 1., 0.],
        [3., 3., 3., 3.],
        [1., 3., 1., 2.]], dtype=torch.bfloat16)
output tensor([[  0.,   0.,   0.,   0.],
        [103., 102.,  inf,  inf],
        [ inf,  inf,  inf,  inf]], dtype=torch.bfloat16)

Expected behavior
The kernel's operation involves copying the input tensor through a temporary tensor to the output.
Therefore, the values of the input and output should be the same.
Screenshots
If applicable, add screenshots to help explain your problem.
Please complete the following environment information:

OS: [e.g. Ubuntu 20.04]
Version of software: cb49c1b4e97b365482edd7bc2647a7136c776cd8
arch: wormhole_b0

Additional context
Add any other context about the problem here.

@hschoi4448,
could you please clarify the difference between intermed and dataflow CB?

@hschoi4448, could you please clarify the difference between intermed and dataflow CB?

dataflow CB

tt::CB::dataflow0
return correct result

intermed CB

tt::CB::c_intermed0
return incorrect result

Dataflow CBs and intermediate CBs are defined in here by the Metal developers.
https://github.com/tenstorrent/tt-metal/blob/e5f887e5c54164e93836b5a6d2bd5747fe81f27f/tt_metal/hostdevcommon/kernel_structs.h#L25
https://github.com/tenstorrent/tt-metal/blob/e5f887e5c54164e93836b5a6d2bd5747fe81f27f/tt_metal/hostdevcommon/kernel_structs.h#L45
I think he (Hyungsuk) is trying to use dataflow CBs because he ran out of intermediate CBs.

Could you also explain the reason for adding this restriction to the dataflow CB in JIT builds?

What are the pros and cons of applying this restriction?
@rdjogoTT

The restrictions on CB types are no longer needed since we moved to tt-metal, they just haven't been removed yet. There is work that has been already started that will update how CBs work and fix this issue as well. I don't yet have a timeline for when that will be finished, but if you need this to work with a very high priority then I can look to add a workaround for now.

@rdjogoTT
If the workaround is prepared, does it have a chance to be merged into the main branch?
Or do we have to maintain it at a local branch?

@razorback3 Good news, the CB work I spoke about above has been given a high priority and is nearly ready. I am helping push it through, and we should have this solved as well soon.

Hey this should be resolved with https://github.com/tenstorrent/tt-metal/pull/14971
Please try to repro again on main


Can support ubuntu with npu?

I try run this project with ubuntu,but some dependence cant find(bigdl-core-npu,pipeline.so);
How can we run it with ubuntu?

Hi @dockerg,
IPEX-LLM currently only supports Intel NPU on Windows. You could refer to IPEX-LLM NPU QuickStart for more information :)

Is there any plan for support on linux?

Hi @dockerg,
IPEX-LLM currently only supports Intel NPU on Windows. You could refer to IPEX-LLM NPU QuickStart for more information :)

Is there any plan for support on linux?

Hi @dockerg,
IPEX-LLM currently only supports Intel NPU on Windows. You could refer to IPEX-LLM NPU QuickStart for more information :)

Is there any plan for support on linux? @Oscilloscope98

Hi @dockerg,
We currently do not have plans to support Linux on Intel NPU. We will share here for any updates :)


Autolink [Question]

Does autolinking also require to follow the installation steps inside documents?

we support autolinking

@LichKing-2234 So, do we need also to add and the framework ?
Im using<EMAIL_ADDRESS>+<EMAIL_ADDRESS>post_install do |installer|
  installer.pods_project.targets.each do |target|

    # The following is needed to ensure the "archive" step works in XCode.
    # It removes React & Yoga from the Pods project, as it is already included in the main project.
    # Without this, you'd see errors when you archive like:
    # "Multiple commands produce ... libReact.a"
    # "Multiple commands produce ... libyoga.a"

    targets_to_ignore = %w(React yoga)

    if targets_to_ignore.include? target.name
      target.remove_from_project
    end

  end
end

@MrHazimAli we use pod. don't need to add the framework.


Missing subclass responsibility for #containerClass in ASN1UnmappedType and ASN1UnstructuredType

#containerClass is only implemented in subclasses of ASN1UnmappedType but called already in ASN1UnmappedType>>#decodeValueIndefiniteLengthWithDERStream: and other
so I guess a subclass responsibility is missing

same for ASN1UnstructuredType


Markers count inside cluster doesn't sum properly

There is an other issue posted about a problem with markers count, but i think this one is a bit different.
Also changing the DefaultClusterRenderer like this:
descriptor = BitmapDescriptorFactory.fromBitmap(mIconGenerator.makeIcon(cluster.getSize()));
It works fine, but when zooming out, when clustering for the last time, it doesn't sum correctly. In my case it has to sum a red bucket with a green one, lets say 300 + 100, but when clustered it stays at 300.
PD: I have noticed the same with  the rounded numbers ('300+' + '100+' == '300+')

I've met this issue when I changed DefaultClusterRenderer.MIN_CLUSTER_SIZE to 1. After that single markers are marked as clusters sometimes and items count inside cluster becomes  wrong.


Where to find Terraria's textures?

Ok, so where do I find the textures for Terraria, since the screen is greyed out and I need to find the files/folder

It depends on where terraria is installed.
It defaults to c:\Program Files(x86)\Steam\steamapps\common\Terraria\Content\Images
and on Linux it's  ~/.local/share/Steam/SteamApps/common/Terraria/Content/Images


Fix backticks in doc comments

Hello!
Courtesy of clippy::doc_markdown, here is a collection of what I hope are fairly uncontroversial tweaks to the docs: adding/removing backticks for visual consistency. 😅
I left all mentions of SysEx alone because I see that they are consistently unticked and it's more of a concept than an explicit keyword (currently just an example plugin's name), but I could add those too if that's desired?

Thanks!


Update README.adoc

fixed the TBTC channel name

The linting jobs aren't relevant in this case, so I'm going to just push this on in.


localstorage: Skip localstorage warning for spectators.

These are more likely than not not even caused by real human browsers, and even if so, are much less important to preserve the state of.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Browsers have supported localStorage for 14 years, and even for browsers that don’t, I would expect window.localStorage to simply evaluate to undefined rather than throwing an exception. So I’d be curious to see the what the real exception looks like before we squish it.

Thanks for prodding to look deeper -- this looks to be caused by spectators failing their register call (#25683), and then attempting to access message_lists.home which is undefined.
Given that, I'll close this PR.  I don't know that we can mark the Sentry error as ignored, since I don't know if it will take into account the "caused by" secondary stacktrace.


Android native crash-- Attempt to invoke virtual method 'int android.graphics.Bitmap.getWidth()' on a null object reference

We're seeing this on Android devices on production. One example is a Samsung SM-G930V, Android 7.0 / API 24. It happens when ProcessingManager.getPreviewForSecond() is called with a value very close to the end of the video. We suspect rounding errors or something else is causing it to try to extract a frame past the end.
Solving this bug would be nice, but even before it's solved, it should be properly trapped and raised as a JS-error, not a native Java error that crashes the whole app. Right now the crash is making this really tough for us.
We've already tried subtracting an epsilon value, and we're actually subtracting a full second and still going past the end, so this is not easy to work around.

Sorry for late reply @GeoffreyPlitt .
Can you please provide more details.
Can you please show me the Callstack ?

@zachrnolan

Hi @shahen94, I work with @GeoffreyPlitt.
Here's the callstack, let me know if you need anything else.
    Trimmer.java:348 com.shahenlibrary.Trimmer.Trimmer.getPreviewImageAtPosition
    TrimmerManager.java:83 com.shahenlibrary.Trimmer.TrimmerManager.getPreviewImageAtPosition
    Method.java:-2 java.lang.reflect.Method.invoke
    JavaMethodWrapper.java:372 com.facebook.react.cxxbridge.JavaMethodWrapper.invoke
    JavaModuleWrapper.java:162 com.facebook.react.cxxbridge.JavaModuleWrapper.invoke
    NativeRunnable.java:-2 com.facebook.react.bridge.queue.NativeRunnable.run
    Handler.java:751 android.os.Handler.handleCallback
    Handler.java:95 android.os.Handler.dispatchMessage
    MessageQueueThreadHandler.java:31 com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage
    Looper.java:154 android.os.Looper.loop
    MessageQueueThreadImpl.java:199 com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run
    Thread.java:761 java.lang.Thread.run```

@zachrnolan @GeoffreyPlitt Thanks guys!
And how much is the length of your video file ?

@shahen94 it seems to be happening at any length.

+1

@shahen94 hey, just wanted to check in and see if you were able to find anything. Thanks!

Hi @zachrnolan .
Unfortunately no - @gevorg94 can you please take a look at this issue ? Might you can help

Hey @shahen94 @gevorg94 friendly ping to see if you guys were able to look at this yet. Thanks!

I'm having the same problem here. Have anyone find a solution?

Facing the same issue here on some Android devices (just happened on Google Pixel running Android version 8.1.0).

I also  stuck here please help.


Add error handling to compiler when linker is unavailable

Fixes #12839
The first two commits are simply refactoring to drop the process_wrapper helper method which is unnecessary and only complicates control flow.
The third commit introduces a short name in linker_command which refers to the command itself without the additional autogenerated arguments (which can be quite a lot).
Alternatively, we could also use parse_arguments to extract the name from the final command. But the original value is already available anyway, and it's easy to just forward it.
The fourth commit then implements the actual error handling. Due to https://github.com/crystal-lang/crystal/issues/12873 it covers errors indicated by exceptions as well as exit status.

Rebased on top of https://github.com/crystal-lang/crystal/tree/release/1.7 for merging into 1.7.2. That required a force push.


"No valves to update"

I'm trying to upload any of examples in the repo to the pipelines settings tab. But when I try to save I get the error: No valves to update.
Can somebody point to what might be going on? Thank you.
Allow me show my code for context:
from typing import List, Union, Generator, Iterator
import os
from pydantic import BaseModel
import requests

class Pipeline:
    class Valves(BaseModel):
        AGENT_HOST: str
        AGENT_PORT: int
        AGENT_ENDPOINT:str
        pipelines: List[str] = []
        priority: int = 0

    # Update valves/ environment variables based on your selected database
    def __init__(self):
        self.name = "Xerket Chat"

        # Initialize
        self.valves = self.Valves(
            **{
                # "pipelines": ["*"],                                                           # Connect to all pipelines
                "AGENT_HOST": os.getenv("AGENT_HOST", "<IP_ADDRESS>"),
                "AGENT_PORT": os.getenv("AGENT_PORT", 5006),
                "AGENT_ENDPOINT": os.getenv("AGENT_ENDPOINT", "/message"),
            }
        )

    async def on_startup(self):
        # # This function is called when the server is started.
        pass

    async def on_shutdown(self):
        pass

    def pipe(
        self, user_message: str, model_id: str, messages: List[dict], body: dict
    ) -> Union[str, Generator, Iterator]:

        AGENT_ENDPOINT = self.valves.AGENT_ENDPOINT
        AGENT_PORT = self.valves.AGENT_PORT
        AGENT_HOST = self.valves.AGENT_HOST

        url = f"http://{AGENT_HOST}:{AGENT_PORT}{AGENT_ENDPOINT}"
        data = {"content": user_message}

        response = requests.post(url, json=data)

        agent_response = ""

        if response.status_code == 200:
            agent_response = response.json['response']
        else:
            agent_response =  "Agent not available"

        return agent_response

My guess is you need to have this function defined?
async def on_valves_updated(self):
        # This function is called when the valves are updated.
        pass

I have (kinda) the same problem. The tool is still in the workbench but there are no valves for either the tool itself or any user. I temporarily fixed this by editing the tool and saving as it is. Hoping for a fix soon

I am also facing the same problem. I tried to directly add it from github url.
https://github.com/open-webui/pipelines/blob/main/examples/pipelines/integrations/python_code_pipeline.py

Tried during one hour and the solution is very simple !! You need to click on the button to validate your script :)

Yes, the upload button helps to validate the pipeline file, but the save button still throws the same error

Not sure what the save button does but @Paulhb7 answer works


Daniel Widdis: Watch thetopsites.net

Daniel Widdis requests the watch of the watch_keyword thetopsites\.net. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

thetopsites\.net has been seen in 1 true positive, 4 false positives, and 3 NAAs.

This website scrapes/plagiarizes Stack Overflow content

Approved by Mast in Charcoal HQ


Is it possible to negate a conditition (i.e. {#!valueChecked})?

Hello,
I can't figure out how negate a condition. I tried this:
{#!value}
{#value == false}
{#value != true}

But it doesn't work.
An If/else block style would be great too.
Can you help me?
Thank you

Hi, by default negation is not supported, but you can check out this  advanced syntax which may provide what you are looking for.

Thank You!


cubepp: Fix presentKHR assert

cherry pick: be8467a6c64b0ddd31af09619b2dc09d8cfc860b
Currently causes crash for Ubuntu packages

CI Vulkan-Tools build queued with queue ID 17685.


DW-4490 - Add tf init step to workspace create

I believe this is required, as TF doesn't actually create the WorkSpace properly with the process as it stands - it says it does, but when you try and run Concourse against them, it hasn't.
I intercepted a Concourse container and did the additional step of a tf init, and it seems to work OK in Concourse now.
Signed-off-by: Benjamin Sherwood<EMAIL_ADDRESS>

Leaving this here for now, it'd be good to take some time to understand why it doesn't work currently.


'Goto Definition' command finds nothing if the function-call is declared over more than one line

Now fixed in Release 1.3.0


Add Spanish (Spain) translation for Philips Hue Play HDMI Syncbox integration

Add Spanish (Spain) translation for Philips Hue Play HDMI Syncbox integration
Description
This pull request introduces the Spanish (Spain) translation (es-ES) for the Philips Hue Play HDMI Syncbox integration. The translation ensures that Spanish-speaking users can interact with the integration in their native language.
Changes Made

Added a new es-ES.json file containing translations for all strings used in the integration.

Motivation
Providing localization for Spanish (Spain) helps expand the usability of the Philips Hue Play HDMI Syncbox integration for non-English speakers and enhances the inclusivity of the integration.
Notes
If any additional changes or adjustments are needed, feel free to suggest them!

Summary by CodeRabbit

New Features

Introduced Spanish translations for configuration and operational messages for the Philips Hue Play HDMI Sync Box, improving accessibility for Spanish-speaking users.

Enhancements

Added translations for error messages, user instructions, and entity definitions, ensuring a comprehensive user experience in Spanish.

Thanks for the translation


Vertical line overflows into actor boxes

The vertical lines overflows into the actor boxes.

                        .messageLine0 {
                            stroke-width:1.5;
                            stroke-dasharray:"2 2";
                            marker-end:"url(#arrowhead)";
                            stroke:black; }
                        .messageLine1 {
                            stroke-width:1.5;
                            stroke-dasharray:"2 2";
                            stroke:black; }
                        .actor-line { stroke:black; }
                        .loopLine {
                            stroke-width:1.5;
                            stroke-dasharray:"2 2";
                            marker-end:"url(#arrowhead)";
                            stroke:black; }
                        .messageText {
                            fill:black;
                            stroke:none;
                            font-family:'trebuchet ms', verdana, arial;
                            font-size:14px;}
                        text.actor {
                            fill:black;
                            stroke:none;
                            font-family: Helvetica;}
                        .labelBox {
                            stroke: none;
                            fill: black;}
                        .note {
                            stroke: black;
                            fill: none;}
                        .actor {
                            stroke: black;
                            fill: none;}
                        #arrowhead {
                            fill:black;}

This is the css that I used

Could you make the background of the actor boxes white?

That does help. Thanks.


openstack-ardana: ansible role for amphora image setup

We were previously setting up the amphora image on Ardana
deployments only during tempest testing and only if either
the lbaas or heat filters were enabled.
This change moves that logic to its own role and calls it
during regular cloud deployment as well as after a cloud
update, in case the amphora image was updated.

Testing with: https://ci.suse.de/blue/organizations/jenkins/cloud-ardana9-job-entry-scale-kvm-gerrit-x86_64/detail/cloud-ardana9-job-entry-scale-kvm-gerrit-x86_64/2/pipeline


Request total_solar: sensor.lux_solar_output_live has the option to view the actual sensor graph

Hi
Thanks for such a great dashboard that I've configured for use with my Lux Hybrid inverter using @guybw LuxPython_DEV integration.
Looking at your dashboard options, using the total_solar: sensor.lux_solar_output_live, instead of the default sum of PV1+PV2 etc, is there any chance that this item on the dashboard could link to the related sensor graph?
i.e. Pv1 and PV2 are links to the individual solar input sensors and when clicked load the related sensor graphs, using a sensor with total_solar, shows the total solar input value and can link to the associated graph but in your dashboard doesn't offer that option.
In short - if I use a sensor as apposed to adding values together, could we also have the availability of opening the sensor graph please?
Phil

From reviewing some of your examples, the last one showing full configuration details: -
pv_total: sensor.sunsynk_totalsolar
but if I use this as follows: -
pv_toatl: sensor.lux_solar_output_live
it still doesn't give access to the functionality of being able to click and open the senor graph?

Hi. It should already do that. Are you sure that sensor.lux_solar_output_live is setup correctly in HA. It needs to have an associated unit of measurement for the historical graph to work.

Thank you @slipx06, after reading/watching your reply, on checking my own configuration I'd realised that I simply hand't saved the configuration change - so it was still just doing the automatic sum.
sorry, me bad it is working as demonstrated above.... and there's so many questions I could ask around various elements on that dashboard in your reply, but I better do a bit more 'proper' settings testing first.
Really appreciated the quick reply though!

Hi @slipx06 my friend, sorry I'm back with the same issue but on a different element of the card - this time, it's: -
essential_load1: sensor.value_tapo_total
essential_load2: sensor.phoenix_plug_06_current_power
Load1 I expect to not be clickable to view a graph as it's my own combination sensor that just sums up 5x other graph capable sensors ..... no issues there as I didn't expect it to work.
Load2 though can be programmed with any graphable sensor like the tapo plug as above or replaved with pv1 solar output sensor etc BUT it's still not allowing the option to display the related sensor graph, the same initial scenario I started with above.
I must be doing something wrong, or editing incorrectly but for the life of me I can't get these small essential boxes to allow for the sensor graph to be accessed?

In essence are
essential_load1:
essential_load2:
in your card design also setup to be used with graphable sensors?

Much appreciated my friend :o)

Added to v2.0.0

Thank you, those load graphs are working great !


A few UI alignment fixes

Description

Areas changed

npm packages changed:

[ ] responsibleai/causality
[ ] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[ ] responsibleai/localization
[ ] responsibleai/mlchartlib
[ ] responsibleai/model-assessment

Python packages changed:

[ ] erroranalysis
[ ] raiutils
[ ] raiwidgets
[ ] rai_core_flask
[ ] responsibleai

Tests

[ ] No new tests required.
[ ] New tests for the added feature are part of this PR.
[ ] I validated the changes manually.

Screenshots (if appropriate):
Documentation:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #1359 (d6d94e6) into main (2527f0a) will decrease coverage by 3.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1359      +/-   ##
==========================================
- Coverage   83.87%   80.43%   -3.45%
==========================================
  Files          54       18      -36
  Lines        2698      322    -2376
==========================================
- Hits         2263      259    -2004
+ Misses        435       63     -372

Flag
Coverage Δ

unittests
80.43% <ø> (-3.45%)
:arrow_down:

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

Impacted Files
Coverage Δ

erroranalysis/erroranalysis/__init__.py

...ranalysis/erroranalysis/analyzer/error_analyzer.py

raiwidgets/raiwidgets/__version__.py

raiwidgets/raiwidgets/dashboard.py

...nalysis/erroranalysis/_internal/version_checker.py

...idgets/raiwidgets/responsibleai_dashboard_input.py

erroranalysis/erroranalysis/analyzer/__init__.py

erroranalysis/erroranalysis/_internal/constants.py

raiwidgets/raiwidgets/cohort.py

raiwidgets/raiwidgets/error_analysis_constants.py

... and 26 more

Continue to review full report at Codecov.

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


Error in setwd(mainDir) : cannot change working directory

Having issues setting the output folder for images generated by the RPlotHist stored procedure. I've created C:\temp\plots on my computer. Does the error have something to do with isolation level changes in SQL Server 2019? (https://docs.microsoft.com/en-us/sql/advanced-analytics/install/sql-server-machine-learning-services-2019?view=sql-server-ver15)

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

ID: ddb4497c-66d7-b890-bb98-fb4bdb9e32d6
Version Independent ID: 3794acda-9e96-834e-9931-c8e115014db9
Content: R + T-SQL tutorial: Explore data - SQL Server Machine Learning Services
Content Source: docs/advanced-analytics/tutorials/sqldev-explore-and-visualize-the-data.md
Product: sql
Technology: machine-learning
GitHub Login: @dphansen
Microsoft Alias: davidph

@awilbert Thank you for opening a GitHub issue. Would you be able to provide us with the error/output message that you get? Thanks.

Error returned in messages window:
Msg 39004, Level 16, State 20, Line 1
A 'R' script error occurred during execution of 'sp_execute_external_script' with HRESULT 0x80004004.
Msg 39019, Level 16, State 2, Line 1
An external script error occurred:
Error in setwd(mainDir) : cannot change working directory
Calls: source -> withVisible -> eval -> eval -> setwd
Error in execution.  Check the output for more information.
Error in eval(ei, envir) :
Error in execution.  Check the output for more information.
Calls: runScriptFile -> source -> withVisible -> eval -> eval -> .Call
Execution halted
------
SQL Server log just says:
SQL Server received abort message and abort execution for major error : 18 and minor error : 42

@awilbert Thanks. We will investigate and get back to you.

@awilbert You are correct. This is due to the isolation level changes in SQL Server 2019.
The issue can be resolved by granting Read & execute permissions to ALL APPLICATION PACKGES to the folder you are trying to change their working directory to:

In File Explorer, right click on the folder you want to use as working directory, and select Properties.
Select Security and click Edit... to change permissions.
Click Add...
Make sure the From this location is the local computer name.
Enter ALL APPLICATION PACKAGES in Enter the object names to select and click Check Names. Click OK.
Select Read & execute under the Allow column. Click OK and OK.

I will update the docs to point out this change. Thank you for bringing it to our attention.

The documentation has been updated. Please see the File permissions section in SQL Server 2019 on Windows: Isolation changes for Machine Learning Services.


Want to Help !!!

@gautamsi after #MSBuild I was hoping to put together a SharePoint Framework / Office Add-In generator for Aurelia. I know a ton of work has been done with React already, but Aurelia would be a fantastic addition; of course we would want Office Fabric UI as a core component of this.
I looked at the list of implemented vs non implemented components, but I am having a hard time getting started with how to actually get this all working.
Could you write up a quick tutorial and include in the wiki? Include how to build a Aurelia / office fabric UI component, and how to write the unit test / integration tests for a component?  I think that would be of great help. I will keep trying to figure the component stuff out on my own but any help would be fantastic.
Also once I get started, would it make sense for me to create an issue for each component I am working on so we can start tracking them individually?
And thanks for this project. 👍

Let me write something over weekend. Opening issues for each component is also good idea.

sorry for the delay, havent had much time to work on this.
quick and dirty summary, this is how it is organized.
src/components contains sub folder for each  fabric-js components
I create a html view based on sample hbs file from original repo + combination of missing html lines from various examples from https://dev.office.com/fabric-js
if the component is also using a js method or corresponding js file, copy that with name prefixed with "ms". for example, CommandButton component has CommandButton.html as view, CommandButton.ts as viewmodel, and msCommandButton.ts as original ts file from fabric-js repo. there is also small change needed for msCommandButton.ts file, remove the namespace line and add export keyword to class. you have to initialize the original component class in attached method. you can keep reference to container div in template with ref=name attributes.
you have to configure global resources in the index.ts file.
checkout the CommandButton component in dev branch for more details.
once it is ready for consumption, you can create corresponding demo page in documentation/app/components.
if you are looking at specific cmponent, I can share the information on a screen sharing call and work with you to explain the repo,
I am also renaming the Class to the ts file name to match the component name, custom-element decorator should add fabric-*-* for example fabric-command-button (aurelia way of normalizing based on PascalCasing)


DDPRateLimiter

Hi there, just a question. Is this working properly with DDPRateLimiter?

Don't know anything about DDPRateLimiter. Since publishComposite uses Meteor's native publish under the hood, I don't see why it wouldn't work. Closing, but feel free to reopen if you find that it isn't working.


[improvement] improve async submit streamload

Proposed changes
Issue Number: close #xxx
Problem Summary:
Now when writing to multiple tables, when the checkpoint arrives, an HTTP request will be submitted. If the number of tables is large, the subsequent tables will continue to wait. Here, the submission will be turned into an asynchronous request to reduce the waiting on the connector side.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

Considering that stopload should theoretically be very fast because it only submits http requests, so if stopload is particularly slow, it means there may be a problem on the Doris side. At this time, becoming asynchronous will only cause more congestion and create a larger backlog.
So draft first


tls_close_notify should send a "warning" TLS alert instead of "critical"

If a "critical" TLS alert is sent, then openssl, libressl show this message at each connection close:
4151646920:error:140943E8:SSL routines:SSL3_READ_BYTES:reason(1000):s3_pkt.c:1293:SSL alert number 0. Their shell exit statuses are 1.
A change is trivial: tls_build_packet inside tls_close_notify should pass critical parameter as 0 instead of 1. With this, both clients show successful closed message, with shell exit status 0.

Fixed, thanks!


Taxon view: fixed query for proteins tab

Closing for history to be cleaned up and a new PR opened.


Webhook delivery test unavailable for organisations

Gitea version (or commit ref):
1.1.2
Can you reproduce the bug at https://try.gitea.io:

[x] No (It's currently down)

Description
"Test delivery" button is missing in the organisation's webhook settings. Not sure if a bug or a feature is missing.

It's a missing feature I think.

this would be good

Stumbled across this one, because the "Test Delivery" button is mentioned in documentation although it is not available to organisations:
https://docs.gitea.io/en-us/webhooks/

Ok the test delivery button is on:
https://github.com/go-gitea/gitea/blob/5fb1ad70113d3272232c266b4ff58e9f7f646594/templates/repo/settings/webhook/history.tmpl#L6
It requires:
https://github.com/go-gitea/gitea/blob/5fb1ad70113d3272232c266b4ff58e9f7f646594/templates/repo/settings/webhook/history.tmpl#L1
I'm not immediately certain I can see a route for this to be ever set on an organisation setting page. Certainly https://github.com/go-gitea/gitea/blob/master/routers/org/setting.go does not appear to set it.

I don't think this is so easy as test delivery currently works only to trigger push event