docs: update installation option with openupm badge

Hi @QFSW,
Thanks for creating the amazing project. I'm the creator of OpenUPM, a public upm registry with automatic build pipelines that publish upm packages based on valid git tags. You can read the announcement to get the basic idea, so far it hosts 100+ packages as of Jan 2020.
I have included your package to the platform https://openupm.com/packages/com.qfsw.mop2/. The process is completely automatic that all your future GitHub releases will get published.
I prepared this PR to update docs and badge.
If you like the idea, please consider adding other your upm packages to the platform. We provided a form to submit new packages: https://openupm.com/packages/add/
Cheers,
Favo

Looks like a good initiative, I've merged your PR to master. I'll take a look at adding more packages of mine as I make them :)

@QFSW Thanks for the efforts. Feel free to use this thread to ask me anything if you have one.


Electron: On linux ELIFECYCLE error while running npm run build.desktop.linux

IMPORTANT: This repository's issues are reserved for feature requests and bug reports. Do not submit support requests here.
Steps to reproduce and a minimal demo of the problem
just run npm run build.desktop.linux using npm 3.10.3 and node 6.3.0
TypeError: Request path contains unescaped characters
at new ClientRequest (_http_client.js:53:11)
at TunnelingAgent.exports.request (http.js:31:10)
at TunnelingAgent.createSocket (/home/ykoehler/angular2-seed-advanced-master/node_modules/github-releases/node_modules/tunnel-agent/index.js:117:25)
at TunnelingAgent.createSecureSocket [as createSocket] (/home/ykoehler/angular2-seed-advanced-master/node_modules/github-releases/node_modules/tunnel-agent/index.js:184:41)
at TunnelingAgent.addRequest (/home/ykoehler/angular2-seed-advanced-master/node_modules/github-releases/node_modules/tunnel-agent/index.js:80:8)
at new ClientRequest (_http_client.js:158:16)
at Object.exports.request (http.js:31:10)
at Object.exports.request (https.js:199:15)
at Request.start (/home/ykoehler/angular2-seed-advanced-master/node_modules/github-releases/node_modules/request/request.js:584:30)
at Request.end (/home/ykoehler/angular2-seed-advanced-master/node_modules/github-releases/node_modules/request/request.js:1212:28)
npm ERR! Linux 3.10.0-327.22.2.el7.x86_64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "run" "build.desktop.linux"
npm ERR! node v6.3.0
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build.desktop.linux: rimraf desktop/linux/* && gulp desktop.linux
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build.desktop.linux script 'rimraf desktop/linux/* && gulp desktop.linux'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular2-seed-advanced package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     rimraf desktop/linux/* && gulp desktop.linux
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular2-seed-advanced
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular2-seed-advanced
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/ykoehler/angular2-seed-advanced-master/npm-debug.log

npm-debug.zip

Try to npm run postinstall. This task haven't worked automatically with npm install for me on linux.

@ykoehler did running npm run postinstall work for you here?

closing due to no response.


[stable/mercure] Fix secrets definition

What this PR does / why we need it:

Fix publisherJwtKey and subscriberJwtKey config
Allows to update secrets and configmaps

Which issue this PR fixes

fixes #14455
fixes #14531

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm


Fix dependency workflow failures

Summary

Three fixes to the dependency workflow:

Fix the path to the upload-to-s3 action, which was incorrect
Add a length check for number of new versions found, to prevent GHA issues with empty matrices.

length-related changes have been tested here: https://github.com/paketo-buildpacks/nginx/actions/runs/3245137061 (the workflow was previously failing)

Add logic to only run compilation/compilation-related steps on the matrix of metadata entries that need compilation (missing checksum and URI). This protects against failures we saw in https://github.com/paketo-buildpacks/cpython/actions/runs/3223469727 in which one stack-variant of cpython didn't need compilation and caused workflow failures

the changes have been tested in https://github.com/paketo-buildpacks/cpython/actions/runs/3266553148

Use Cases

Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[ ] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

length-related changes have been tested here: https://github.com/paketo-buildpacks/nginx/actions/runs/3245137061
this resolves failures we saw in the Nginx GHAs, due to no new versions being found the compilation step was erroring due to  retrieval metadata matrix being empty.

Workflow changes tested:
https://github.com/paketo-buildpacks/nginx/actions/runs/3266794598/jobs/5371055424
https://github.com/paketo-buildpacks/cpython/actions/runs/3266797067

bump @paketo-buildpacks/tooling-maintainers I'd love to get this reviewed ASAP so we can unblock workflow failures.


Ncp: Implement the get handler for IPV6_LL_ADDR spinel property

This commit implements a new API otIp6GetLinkLocalAddress() to  get
the link-local address assigned to the Thread interface.  It then uses
the new API to implement the get handler for spinel property
IPV6_LL_ADDR in NcpBase class.
Should address issue https://github.com/openthread/openthread/issues/1794.

Plan of action on this change: We treat the (single) link-local IPv6 address which is derived from the MAC address (as the IID) as the main/default IPv6 address which is returned by the public API and then the same address passed up as IPV6_LL_ADDR spinel property and used within wpantund. Other link-local IPv6 address(es) are treated as any unicast address.


How does this compare to Firefox's inbuilt crypto miner blocker?

Mozilla has implemented a crypto miner blocker in Firefox in version 67.0 - https://blog.mozilla.org/futurereleases/2019/04/09/protections-against-fingerprinting-and-cryptocurrency-mining-available-in-firefox-nightly-and-beta/
Other than being newer and probably not having as complete as a blocklist as MinerBlock, how does this feature compare to MinerBlock?

Minerblock not only uses domain blacklisting, but it also scans the HTML DOM for known cryptomining function calls and kills them. That is the difference.


Would be nice to have an option for Spinsys for SIMPSON

Expanding on issue raised by @AKM2022

Agreed. Being able to choose a set of atoms and write out the NMR parameters ready for a simulation is one of the key selling points. It fits naturally into a graphical programme since you you might well need to choose the spins interactively - indeed writing out whole unit cell would generally be a Very Bad Idea and should generate a warning.
Use cases:
Selecting an individual C atom + all the H atoms (including within adjacent molecules) within a certain radius, including the dipolar couplings.
Selecting atom X sites and writing them out. With multiple sites, you often would not want to include couplings and would generally construct the spectrum as a sum of single spin simulations.


[Feature Request] Shift Right Click

Would it be possible to add the option to be able to shift right click in addition to right clicking when using the use() function?

I need think about how to properly implement it via API ...


ExprGraph: major refacto and tests

This is a work in progress for the exprgraph package.
The API has changed a bit. The Graph is now a Weighted directed graph so it can track the order of the children of a node.
The most important part is that all the examples are still working with very little modification.
There are some tests, and more will come eventually (the goal is 100% code coverage).


Error -1010 with Exoplayer

Bug
I try to play dynamically generated fragmented MP4 file. For some files, I get a -1010 error, which means MEDIA_ERROR_UNSUPPORTED. Which is weird because random apps from google play (like "ExoPlayer") can play the video without any issues (Those random apps are based on ExoPlayer).
I'm testing on a real device (ExoPlayer apps and my code). That device is Mi9 Se with Android 10.
Platform

Android only
MediaPlayer and ExoPlayer have issues with the same files, but only ExoPlayer calls the onError callback.

Environment info
❯ npx react-native info
info Fetching system and libraries information...
System:
    OS: Linux 5.8 Ubuntu 20.10 (Groovy Gorilla)
    CPU: (8) x64 Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz
    Memory: 1.36 GB / 15.20 GB
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 12.18.2 - /usr/bin/node
    Yarn: 1.22.5 - /usr/bin/yarn
    npm: 6.14.8 - /usr/bin/npm
    Watchman: Not Found
  SDKs:
    Android SDK:
      API Levels: 23, 25, 28, 29, 30
      Build Tools: 25.0.2, 28.0.3, 29.0.2, 30.0.0
      System Images: android-29 | Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom
      Android NDK: Not Found
  IDEs:
    Android Studio: Not Found
  Languages:
    Java: Not Found
    Python: 3.8.6 - /home/pearman/.virtualenvs/defaultenv/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: 0.63.0 => 0.63.0
  npmGlobalPackages:
    *react-native*: Not Found

Library version: 5.0.1, 5.1.0-alpha8
Steps To Reproduce
Try to play this video: https://files.kaifer.cz/aNzKIvsd/public-broken-video.mp4
Expected behavior
Just play the video correctly.
Reproducible sample code
<Video source={{uri: "https://files.kaifer.cz/aNzKIvsd/public-broken-video.mp4"}} />

Video sample
https://files.kaifer.cz/aNzKIvsd/public-broken-video.mp4

Are you able to reproduce this?


migrate to GitHub Actions

Let's plan to migrate to the wonderful GitHub Actions
🙌🏼

I believe this has been done in https://github.com/changesets/changesets/pull/1085


Added A Guess the number game

Issue no: #6

@omk-coder you are using plain html, css and js . This wont work in Reactjs . You have to write in jsx .


I dont know about this.

I feel like as your friend, I should know every detail about your life. Please inform me.
TIA

Many issues have been assigned


Change auto-dithering to DitherType.ORDERED and ERROR_DIFFUSION where applicable

Courtesy of @wiwaz. From our PMs:
@wiwaz:
i did tests
at high bitdepths the difference between ordered and error diffusion spatially is unnoticeable
but ordered is static
using error diffusion all the time is hurting our compressability
and maybe even making quality worse
since its adding random temporal noise that we really dont even need
again, the difference is unnoticeable spatially at high depths
ordered only looks ugly when dithered down to 8
so basically when we are doing 32 to 16 or 16 to 10 whatever we should really be considering ordered instead
and opting for error diffusion only for the final dither down

With this in mind, this PR is more of a call to action for additional testers to see how well this works out in real-life application. Please test! @NSQY @Setsugennoao

Should be based on input and output bitdepth, not just blindly use ordered (which i don't like tbh...)

Courtesy of @NSQY (PLEASE POST HERE >:((( )
<html>
<body>
<!--StartFragment-->

1 | >>> from vstools import core, vs, clip_async_render
-- | --
2 | >>> core.num_threads = 1
3 | >>> clip = core.std.BlankClip(None, 3840, 2160, format=vs.YUV444PS, length=500, keep=True
4 | ... )
5 | >>> clip_async_render(clip.resize.Point(format=vs.YUV444P16, dither_type='none'), None, 'Test')
6 | Test ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 500/500 100.00% 257.48 fps 0:00:00
7 | >>> clip_async_render(clip.resize.Point(format=vs.YUV444P16, dither_type='ordered'), None, 'Test')
8 | Test ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 500/500 100.00% 259.63 fps 0:00:00
9 | >>> clip_async_render(clip.resize.Point(format=vs.YUV444P16, dither_type='error_diffusion'), None, 'Test')
10 | Test ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 500/500 100.00% 53.58 fps 0:00:00

<!--EndFragment-->
</body>
</html>

Should be based on input and output bitdepth, not just blindly use ordered (which i don't like tbh...)

Ordered specifically is terrible for dithering down to 8 bits so yes it'd have to be dependant on in/out bits unless a more robust dithering algorithm (void?) is used.

Sorted in #85.


Not able to setup aarch64 based native images

Due to the platform architecture being hardcoded in the setup instructions, I'm not able to build native images on Mac OS running on M1 (aarch64). Snippet attached below:
https://github.com/micronaut-projects/micronaut-gradle-plugin/blob/96f4a5fc28e2f7b443a922b0888de3c9b5811a1a/docker-plugin/src/main/java/io/micronaut/gradle/docker/NativeImageDockerfile.java#L360
Just like other params being configurable, the platform should also be user-configurable and a fallback should be used when not provided.

I've got the example project for r2dbc almost building...  it gets to the step where the docker build wants to install glibc on the alpine container
Produced artifacts:
 /home/app/application (executable)
 /home/app/application.build_artifacts.txt
========================================================================================================================
Finished generating 'application' in 2m 8s.
Removing intermediate container 33510fcaf6ab
 ---> 8e4137567893
Step 10/13 : FROM frolvlad/alpine-glibc:alpine-3.12
 ---> 15b73cac1129
Step 11/13 : RUN apk update && apk add libstdc++
 ---> [Warning] The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested
 ---> Running in 94084aa51d69
standard_init_linux.go:228: exec user process caused: exec format error

Not sure where to go from there.
I believe the root problem is the same one discussed in #365


[Spanner]: Rename Postgre to Postgres

[x] Add new sample files with the correct naming.
[x] Rename all test files so that they have the correct names and test the new samples with correct names.
[x] Create a PR for deleting all postgre files except for the 6 samples that are being used
[x] Internally, update docs to point to the new sample files for the 6 samples being used.
[x] Delete the old sample files.

Closed by #2152 . Thanks @anuragsrivstv


how to use on windows

can u tell  me axactly how to use on windows i dont have mac devices
and i dont too much about javascript

You'll need to install NodeJS and NPM first - this article looks like it provides instructions to do so: https://phoenixnap.com/kb/install-node-js-npm-on-windows


about how to train

hello,i want 2 train my own model through your pretrained model,how to put the video and the label?

Generate annotations needed for dataloader ("<path_to_video> <video_class>" in annotations) The annotation usually includes train.txt and val.txt. The format of *.txt file is like:dataset_root/video_1.mp4  label_1
dataset_root/video_2.mp4  label_2
dataset_root/video_3.mp4  label_3
...
dataset_root/video_N.mp4  label_N

Add the information to ops/dataset_configs.py.
Modify this function and define the filename_categories = #num_of_class.


Patch for /contrib/externs/w3c_audio.js

This issue was imported from Closure Compiler's previous home at http://closure-compiler.googlecode.com
The link to the original issue is:
https://blickly.github.io/closure-compiler-issues/#1316

@retromodular Unfortunately the patch is lost with Google Code... Do you want to submit this as a PR?

@blickly This PR is significantly out of date and no longer actionable as the original patch is gone so this can probably be closed


Every item in Inbox has a next_action

I'm new to programming and have used what I think is the latest revision by Timharton ("Behaviour with parallel taske now fixed as well." 224 lines of code).  Thank you very much for this.  The intended functionality works perfectly.
The issue I have is that every item in my Inbox also received a next_action label.  Please can someone advise me how to resolve this?  Thank you

@kajetanj has a fork that adds in a new option to disable inbox processing. I'll bring it into my version once I have time to do the next release.


Plz make a wiki

Head over to the wiki tab and you know what to do... :P

What should I put in the wiki? Should it be like a Wikia for the game or a general development wiki?

Well, you could have several sections. :D I'm willing to make the wiki. ;)

@Tymewalk: something like Wikia, but reserve some pages for development wiki :smile:

@DrKat123 I think if it's going to be a Wikia I could just put it on Wikia itself (since that has a lot more features), and development wiki could go here.
@bjskistad When this gets sorted out I'll give you a link to wherever it is :smile:

:roll_out_tongue:

Whoops :P

IT'S DRIVING ME NUTSO HAVING NO WIKI!! It's just sitting there. D:

W00T

@DrKat123 @bjskistad I honestly think a wiki for the game would be best on Wikia, and I could link it in README.md. I'll turn off the wiki here for now until I actually use it for development.

Anyway gonna fork this repo and do some tweakd

@DrKat123 Sounds neat! Once you're done make a pull request, I might add it in (and credit you, of course.)

There is now a Tymeventure Wikia that anyone can contribute to.
To recap:

Tymeventure Wikia is being used as a Wiki for the gameplay
The Dev Wiki is not needed
If the Dev Wiki is ever needed, it will be a GitHub Wiki


[AI] unable to locate nearby waypoints

@Sythivo
So, first of all; I added prints to the "Roam" module

The Map has [23] total RoamPoints. (It's very closed in some parts)
[The map isn't too small or too big, it's just a mildly closed map]
And yes. I tried modifying the PivotOffset a lot, didn't help. Well, almost didn't. It would make the AI go in a loop of going to the same 2 RoamPoints and getting Stuck for 4s. Instead of not moving at all.
I did add more RoamPoints (close to the AI) but that made it worse.
Modified the AgentParams, barely helped.
Before you say "It's a issue with the map", then yes. I'm aware. I can't do much about that except edit the code, and I have to get this done soon elsewise, the game is 100% getting delayed.
And I'm not even the Founder of it, making it worse for me.
(saying this so you know I need urgent help)
And finally, the stairs are designed in a way in which you can't add wedges to most which bugs the AI out a lot too. So, I made the "CanJump" true but it didn't work because you didn't implement that feature. I would implement it myself but I have a hard time with others code.

What are you trying to do exactly?
From what I've read, you want the AI to wander near 2 nearby RoamPoints.
Why don't you use table.sort on the copy of the RoamPoints list and index it?
table.sort(CopyOfRoamPoints, function(a: BasePart, b: BasePart)
    -- logic
end);

local first = CopyOfRoamPoints[1];
loca second = CopyOfRoamPoints[2];

OR
You could sort the main RoamPoints table.

listen, that's not it. the AI has a lot of trouble with roaming, i'm not trying to make it roam to specific points or anything. i've tried my best (to my knowledge at least) but failed.

like I said before, changing the pivotoffset and agentparams barely helped. it's the map that's causing and there's almost nothing i can do about that except fix the AI

if I put the exact same AI with the same AIEngine module stuff, on a baseplate it works just about right.

If I put the exact same AI with the same AIEngine module stuff, on a baseplate it works just about right.

Did you modify the source?
If so, what modifications?
Do you have custom mechanisms or are you sticking to the examples?

Yeah I modified but it I manually reverted it to normal, I will go have a look at the code in some time and no, no custom mechanisms at the moment.

I'm talking about before I inserted it to a Baseplate

Example modules aren't really the best, but they should give you a head start on refining it. CanJump can be modified within the agent params created, I suggest using that for maps with high gaps, and you could copy the CyclicLoopAsync code to support jumping if you need, but best option might be to have high walking height to have no jumping but still allow walking though high gaps seemlessly.
Another note, this framework likely would've gotten updates while developing The Mimic but in recent times, we currently aren't using this module. It doesn't mean it's bad, it's just we developed a new system that would not be compatible with this framework.

There aren't really any obstacles in the map, it's just the stairs that the AI gets kind of stuck on so I'll have to implement jumping to the loop but for now, I'll keep the issue open.

The AI also has high speed and the corners/gaps aren't that high.

Check the Navigation Map.
File > Studio Settings > Studio > Visualization > Show Navigation Mesh
Check the area, and the stairs should have a purple plane everywhere, if not there might be a problem with the Map's navigation.

The AI also has high speed and the corners/gaps aren't that high.

Strange. In my AI test, I jokingly set the AI's speed to 100 studs/s and it worked fine with stairs and all. Are you sure that the model is rigged properly? Rigging issues may cause pathfinding issues as well.

It's rigged and the rigging of the bodyparts is the default Rig that Roblox has.

Check the Navigation Map.
File > Studio Settings > Studio > Visualization > Show Navigation Mesh
Check the area, and the stairs should have a purple plane everywhere, if not there might be a problem with the Map's navigation.

Okay, I'll go see. But there's something I wanna say, The AI for some reason, on the third RoamPoint tried going downstairs instead (there wasn't a RoamPoint there). It was completely off. Had to modify the AgentParams for that one.

AIEngine won't be able to provide any automatic AgentParams yet. AgentParams are manual, you would have to make the values.

I know about that. Also I checked the Map's navigation. The stairs had purple plane but it was all messed up.

Also, how are automatic AgentParams supposed to even work?

Also, how are automatic AgentParams supposed to even work? (since you said "yet")
basically something like this:

local AgentParameters = {
	AgentHeight = Humanoid.HipHeight;
	AgentRadius = HumanoidRootPart.Size.X;
	--// ...
}

To provide dynamic agent parameters.

Also, how are automatic AgentParams supposed to even work? (since you said "yet")

basically something like this:
local AgentParameters = {
	AgentHeight = Humanoid.HipHeight;
	AgentRadius = HumanoidRootPart.Size.X;
	--// ...
}

To provide dynamic agent parameters.

That's the default one, seen that everywhere.

That doesn't work well sometimes


Idea: Add asset selector

Actually, if you want to change the asset, you need to edit a .js file (correct me if I'm wrong)
It will be interesting to add a selector with currently installed asset, like for thumbnail generation (https://github.com/joaorb64/TournamentStreamHelper/issues/48) but globally, so no need to edit the .js file

Or move the one used by thumbnail generation to the right of game selection (or elsewhere) and edit its behavior (when the PR is merged)

It depends, depending on your scene you might want to have icons on one OBS scene and artwork on another so both your layouts wouldn't be pointing to the same pack anyways despite the rest of the code being identical
As such it might be worth looking into OBS plugins for that ^^

Oh I see, yeah maybe not a good idea than 😅


update noRdata to limit acceptable empty RDATA cases

The current implementation of noRdata in msg_helpers.go just checks whether the RDLENGTH is 0, and its callers skip any further validation.  But it allows invalid empty RDATA for most types of RRs (A, AAAA...) in wire format to be successfully "unpacked".  According to the code comment, the intent seems to allow empty RDATA in a dynamic update request.  If so, we can limit the case to some specific RR classes since it's only possible for "delete" variants of update requests.
So I propose including this check in noRdata.  The actual diff also covers some special types, mainly for allowing empty OPT RR (which is indeed valid, but its unpack rejects).
I personally think it would be even better if UnpackRR is more aware of the unpack context and also checks OPCODE so it won't accidentally accept broken wire form data from the net, but that will be a much bigger change, so I chose to suggest this small adjustment first.
I'd also note that the unpack implementation of some RR types will have to be updated to reject empty RDATA when it's not allowed.  Just from a quick look, at least TXT.unpack should be updated (empty TXT RDATA isn't possible per RFC1035).  There are probably some more.  See the comment in revised TestNoRdataPack.

Codecov Report

Merging #1080 into master will increase coverage by 0.66%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1080      +/-   ##
==========================================
+ Coverage   55.28%   55.94%   +0.66%
==========================================
  Files          41       41
  Lines       10085    10085
==========================================
+ Hits         5575     5642      +67
- Misses       3482     3540      +58
+ Partials     1028      903     -125

Impacted Files
Coverage Δ

msg_helpers.go
64.4% <100%> (+2.07%)
:arrow_up:

scan.go
83.12% <0%> (+0.28%)
:arrow_up:

msg.go
77.02% <0%> (+0.33%)
:arrow_up:

server.go
67.95% <0%> (+0.55%)
:arrow_up:

zmsg.go
42.34% <0%> (+2.69%)
:arrow_up:

Continue to review full report at Codecov.

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

this stuff is super under specified and the opcode is not tied to the RR, so I don't think you can just claim that this is invalid binary data. It can exist on the wire so we should be able to parse it.

It can exist on the wire

In which case the does following RR validly (i.e. confirming the protocol standard) exist on the wire?
Type = TXT
Class = IN
RDLEN = 0
RDATA: empty
As far as I understand the RFC, this is not possible either per RFC1035 or RFC2136.  So I can only imagine a broken DNS message if we see it on the wire.

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] update noRdata to l..." ]

It can exist on the wire

In which case the does following RR validly (i.e. confirming the protocol standard) exist on the wire?
Type = TXT
Class = IN
RDLEN = 0
RDATA: empty
As far as I understand the RFC, this is not possible either per RFC1035 or RFC2136.  So I can only imagine a broken DNS message if we see it on the wire.

Why not? So you can't use this in dynamic updates? That would be odd and would go against
the intent of dyn updates.
If you're right you should bring it up on dnsops, i see no reason to decide here that this
cannot be done, because clearly a rdlen=0 is saying that there will not be rdata

Why not? So you can't use this in dynamic updates?

In the case of CLASS=IN, it means an addition of RR to a class IN zone.  From Section <IP_ADDRESS> of RFC2136, for example:
   CLASS    TYPE     RDATA    Meaning
   ---------------------------------------------------------
   ANY      ANY      empty    Delete all RRsets from a name
   ANY      rrset    empty    Delete an RRset
   NONE     rrset    rr       Delete an RR from an RRset
   zone     rrset    rr       Add to an RRset

(In this case 'zone' = IN).  So the RDATA is 'rr', and must be able to be added to a zone, just like loadbale to a zone from a zone file.  It may be able to be empty type-specific basis (although the only type I know of that allows it is NULL), and will have to be validated per type basis, e.g., in AAAA.unpack.
But the current code allows, for example, the following in an update request:
   CLASS    TYPE     RDATA    Meaning
   zone=IN  AAAA     empty    (what to add to the zone?)

This PR just tries to prevent that, deferring the validation to the type specific unpack code.  Do you still think this is a protocol specification issue and I should bring it to dnsop?

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] update noRdata to l..." ]

Why not? So you can't use this in dynamic updates?

This PR just tries to prevent that, deferring the validation to the type specific unpack code.  Do you still think this is a protocol specification issue and I should bring it to dnsop?

This would mean that parsing rdata is class dependent? That's slightly less worse than
opcode dependent, but still sounds like a new requirement to me. Can an RR have different
RDATA semantics based on class? is essentially the question.

Can an RR have different RDATA semantics based on class? is essentially the question.

No, except for some very class-specific types such as AAAA or A.  And that's actually why we can't simply allow empty RDATA in UnpackRRWithHeader like this:
	if noRdata(h) {
		return rr, off, nil
	}

and we should rather call corresponding unpack for the RR to see if the empty RDATA is allowed for that type:
	off, err = rr.unpack(msg, off)
	if err != nil {
		return nil, end, err
	}

The dynamic update delete cases are very special exceptions to these.  RFC2136 allows the RDATA can be empty when (and only when) the RRCLASS is ANY so that the record can represent the semantics of "delete all RRs of this name (and type)".  As far as I know, no other RFCs allow an outright acceptance of an empty RDATA.  This PR tries to tighten the validation while still allowing the only known exception made in RFC2136.
So, I believe the code in this PR still allows any standard-compliant RRs to be accepted, while rejecting some of the invalid cases (like an empty AAAA or empty TXT for class IN) that were previously accepted.
(By the way, through the discussion I've realized the code in this PR is still too lenient.  Since class NONE in a dynamic update request doesn't use empty RDATA, it should have excluded class NONE, too).

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] update noRdata to l..." ]

The dynamic update delete cases are very special exceptions to these.  RFC2136 allows the RDATA can be empty when (and only when) the RRCLASS is ANY so that the record can represent the semantics of "delete all RRs of this name (and type)".  As far as I know, no other RFCs allow an outright acceptance of an empty RDATA.  This PR tries to tighten the validation while still allowing the only known exception made in RFC2136.

Ok, but does this have 2119 language? I.e. from what text from 2136 do you get to this
conclusion [that is normative] ? This also implies ANY is a very, very special class that
can alter RRs; though IMO be explicitly be called it out.
I get your point; I just don't see the evidence yet that is (clearly) be called out in the
RFC.
/Miek
--
Miek Gieben

We should probably start an I-D on fixing this
On Mon, 2 Mar 2020, 21:41 Miek Gieben<EMAIL_ADDRESS>wrote:

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] update noRdata
to l..." ]

The dynamic update delete cases are very special exceptions to these.
RFC2136 allows the RDATA can be empty when (and only when) the RRCLASS is
ANY so that the record can represent the semantics of "delete all RRs of
this name (and type)".  As far as I know, no other RFCs allow an outright
acceptance of an empty RDATA.  This PR tries to tighten the validation
while still allowing the only known exception made in RFC2136.

Ok, but does this have 2119 language? I.e. from what text from 2136 do you
get to this
conclusion [that is normative] ? This also implies ANY is a very, very
special class that
can alter RRs; though IMO be explicitly be called it out.
I get your point; I just don't see the evidence yet that is (clearly) be
called out in the
RFC.
/Miek
--
Miek Gieben

The definition of each RRTYPE includes the corresponding RDATA format, and like @jinmeiib, I am only aware of one type for which zero-length RDATA would be valid—NULL (which isn't allowed in master files and doesn't even have a defined presentation format). But in the special context of an UPDATE message (OPCODE=5), there are entries with the syntax of an RR that permit RDLENGTH=0 for an RRTYPE (but only under special CLASS-dependent circumstances): https://tools.ietf.org/html/rfc2136#page-7

2.4.1 - RRset Exists (Value Independent): NAME=<nonempty>, TYPE=<nonempty>, CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.4.3 - RRset Does Not Exist: NAME=<nonempty>, TYPE=<nonempty>, CLASS=254 (NONE), TTL=0, RDLENGTH=0
2.4.4 - Name Is In Use: NAME=<nonempty>, TYPE=255 (ANY), CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.4.5 - Name Is Not In Use: NAME=<nonempty>, TYPE=255 (ANY), CLASS=254 (NONE), TTL=0, RDLENGTH=0
2.5.2 - Delete An RRset: NAME=<nonempty>, TYPE=<nonempty>, CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.5.3 - Delete All RRsets From A Name: NAME=<nonempty>, TYPE=255 (ANY), CLASS=255 (ANY), TTL=0, RDLENGTH=0

There's even specific text precluding the behavior currently exhibited by this library.
https://tools.ietf.org/html/rfc2136#page-7

RDLENGTH is zero and RDATA is therefore empty.  CLASS must be specified as ANY to differentiate this condition from that of an actual RR whose RDLENGTH is naturally zero (0) (e.g., NULL).

https://tools.ietf.org/html/rfc2136#page-8

The RDLENGTH of this record is zero (0), and RDATA field is therefore empty.  CLASS must be specified as NONE in order to distinguish this condition from a valid RR whose RDLENGTH is naturally zero (0) (for example, the NULL RR).

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] update noRdata to l..." ]

The definition of each RRTYPE includes the corresponding RDATA format, and like @jinmeiib, I am only aware of one type for which zero-length RDATA would be valid—NULL (which isn't allowed in master files and doesn't even have a defined presentation format). But in the special context of an UPDATE message (OPCODE=5), there are entries with the syntax of an RR that permit RDLENGTH=0 for an RRTYPE (but only under special CLASS-dependent circumstances): https://tools.ietf.org/html/rfc2136#page-7

2.4.1 - RRset Exists (Value Independent): NAME=<nonempty>, TYPE=<nonempty>, CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.4.3 - RRset Does Not Exist: NAME=<nonempty>, TYPE=<nonempty>, CLASS=254 (NONE), TTL=0, RDLENGTH=0
2.4.4 - Name Is In Use: NAME=<nonempty>, TYPE=255 (ANY), CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.4.5 - Name Is Not In Use: NAME=<nonempty>, TYPE=255 (ANY), CLASS=254 (NONE), TTL=0, RDLENGTH=0
2.5.2 - Delete An RRset: NAME=<nonempty>, TYPE=<nonempty>, CLASS=255 (ANY), TTL=0, RDLENGTH=0
2.5.3 - Delete All RRsets From A Name: NAME=<nonempty>, TYPE=255 (ANY), CLASS=255 (ANY), TTL=0, RDLENGTH=0

There's even specific text precluding the behavior currently exhibited by this library.
https://tools.ietf.org/html/rfc2136#page-7

RDLENGTH is zero and RDATA is therefore empty.  CLASS must be specified as ANY to differentiate this condition from that of an actual RR whose RDLENGTH is naturally zero (0) (e.g., NULL).

https://tools.ietf.org/html/rfc2136#page-8

The RDLENGTH of this record is zero (0), and RDATA field is therefore empty.  CLASS must be specified as NONE in order to distinguish this condition from a valid RR whose RDLENGTH is naturally zero (0) (for example, the NULL RR).

so this is not 2119 language. And it defines it specicially for a dyn update.
So from the above I need to infer that class={255,254} allow empty rdata for all RR
types? This make it even worse, not only is rdata class dependent, the meta-classes
override this and make it legal for all RRs to have no rdata?
So need to reason to complicate packing code to take the class into account, rdlength=0
looks legal. What does ClassANY even mean, does that include 'IN' ?

so this is not 2119 language. And it defines it specicially for a dyn update.

I don't know what language you're looking for; the RFC I linked to uses lowercase "must" but obviously does not constrain other RFCs—it is specific to UPDATE messages because that's where empty RDATA is allowed. In other cases, it is not allowed unless it is valid for the RRTYPE (which it is generally not).

What does ClassANY even mean, does that include 'IN' ?

In the QCLASS field of a QUERY message, 255 (ANY) indicates a query for RRs regardless of CLASS (although it's basically pointless given how things actually developed). In the Prerequisite and Update sections of an UPDATE message, it indicates either a broad assertion or a bulk deletion as described in RFC 2136 and summarized above.

So from the above I need to infer that class={255,254} allow empty rdata for all RR
types? This make it even worse, not only is rdata class dependent, the meta-classes
override this and make it legal for all RRs to have no rdata?

Inside UPDATE messages (OPCODE=5), empty RDATA is allowed for any RRTYPE value, but only if the CLASS value is 254 or 255 to form an assertion or a bulk deletion. Inside QUERY messages (OPCODE=0, and note that this includes responses), empty RDATA is allowed only as permitted by the RRTYPE, so only for NULL records (which are class-independent). And as an aside, RDATA is only class-dependent for for RRTYPEs that are class-dependent.

So need to reason to complicate packing code to take the class into account, rdlength=0
looks legal.

I would recommend leaving the packing code alone—the input is under author control, so it seems acceptable to accurately represent their possibly-invalid data (or it might even be valid, if they are constructing an UPDATE message). But the deserialization code can reject empty RDATA for types that are known to not support it unless the CLASS is either 254 or 255, which is almost all of them. I think this PR implements that for unpacking wire-format data, but in my opinion it's actually even more important for parsing presentation-format data, which is often written by humans who need to be informed about their mistakes rather than having the system silently accept them.

Thanks @gibson042 for the detailed explanations.  I basically have nothing to add to it, but want to clarify one thing: this PR shouldn't affect packing.  noRdata is called only from UnpackRRWithHeader and ToRFC3597.  The former is obviously for unpacking.  I'm not sure how ToRFC3597 is supposed to be used (within the dns package it's only called from tests), but in any case the patch shouldn't change the behavior.  With this PR, empty RDATA will be passed to RFC3597.unpack, which seems to allow empty data.
If we still think something should be clarified in the protocol specificatin, I don't mind bringing it to the IETF dnsop WG, but I'm actually not sure what should be clarified.  We could propose, for example

RDATA MUST be formatted as specified in the corresponding resource record specification (for example, RFC8659 for type CAA) unless other standard document makes an explicit exception.  As of this writing the only known such exception is RFC2136, which allows empty RDATA regardless of its type when it is used in a dynamic update request message with class ANY.  Any empty RDATA MUST be considered invalid unless its resource record specification allows it.

but I suspect it's mostly obvious even if there's no RFC that says exactly this with RFC2119 keywords, so I doubt the WG will adopt it as something worth publishing.

Thanks
I will not make this unpacking more complex and fragile, but focussing on non-standard
things. empty rdata is legal in my book. All I've seen is interpretation of how things
should work 2136 doesn't use 2119 langauge so falls in the same league as 1035 and
friends; hardly useful when actually implementing things
/close

empty rdata is legal in my book

What book are you reading from that would permit successful decoding of e.g. 0x00    0x00 0x10    0x00 0x01    0x00 0x00 0x00 0x01    0x00 as a . 1 IN TXT RR in a DNS message? It's certainly not RFC 1035, which defines TXT RDATA as "One or more s", nor should it be RFC 2136, which explicitly allows empty RDATA in UPDATE messages when paired with CLASS=254 or CLASS=255 but is otherwise mute on the subject.


Add Support for Pony

Fixes #1790
Changes proposed:

[X] Add
[ ] Delete
[ ] Fix
[ ] Prepare

The icon file is way too large (203KB). An icon should not exceed 2~3KB unless it's a filename that doesn't repeat often in a project.
Suggestion: Strip the circular background from the icon and keep only the word PONY and the 'plate'.

The icon file is way too large (203KB). An icon should not exceed 2~3KB unless it's a filename that doesn't repeat often in a project.
Suggestion: Strip the circular background from the icon and keep only the word PONY and the 'plate'.

Done mate, now it is just 3KB. Hope it is okay 👍


Enhance token.Owner with a Type field

The Type field can be by driver to understand which kind of identity the Owner contains. This would avoid drivers to introduce they specialisation as currently done by the fabflow driver with RawOwner.

deprecated


timeouts does not apply all the time

The default 1800 sec timeout insanely long for the most part, so I added lower values like this:
...
		TimeoutClone:      pulumi.Int(180),
		TimeoutCreate:     pulumi.Int(180),
		TimeoutMigrate:    pulumi.Int(180),
		TimeoutMoveDisk:   pulumi.Int(180),
		TimeoutReboot:     pulumi.Int(240),
		TimeoutShutdownVm: pulumi.Int(180),
		TimeoutStartVm:    pulumi.Int(180),
		TimeoutStopVm:     pulumi.Int(180),
	}
...

to simulate a possible issue I just stopped the qemu-agent on of our our vm-s, so the API cannot issue a reboot command.
At first run, the 240sec TimeoutReboot applies nicely, 4 min and some change and we have a proper timeout with error massage:

    error: update failed

  proxmoxve:VM:VirtualMachine (103):
    error: 1 error occurred:
        * updating urn:pulumi:nkcloud::proxmox-vm::proxmoxve:VM/virtualMachine:VirtualMachine::103: 1 error occurred:
        * error waiting for VM reboot: task "UPID:test-devops-proxmox01:0023DCB8:030A542D:65E5BDD9:qmreboot:103:root@pam:" failed to complete with exit code:                                                   VM quit/powerdown failed

Outputs:
  - vmname: "pooltest-vm"

Resources:
    4 unchanged

Duration: 4m7s

If I immediately rerun the same code not fixing he "issue" first, the 240sec does not apply anymore:

    error: update failed

Outputs:
  - vmname: "pooltest-vm"

Resources:
    4 unchanged

Duration: 9m30s

error: update cancelled

9:30 when I finally hit ctrl-c
I'm not sure if its Pulumi, Terraform or Proxmox API level issue, but lets start here.
ps.:
I tried https://www.pulumi.com/docs/concepts/options/customtimeouts/
but its just simply does not work, no matter if I apply the provider level values or not.
...
	if vmMachine, err = vm.NewVirtualMachine(
		ctx,
		vmId,
		machineArgs,
		opts,
		pulumi.DeleteBeforeReplace(true),
		pulumi.IgnoreChanges(ignoreString),
		pulumi.Timeouts(&pulumi.CustomTimeouts{
			Create: "4m",
			Update: "4m",
			Delete: "4m"}),
	); err == nil
...

would you mind testing this in the terraform provider itself? this would clarify whether it's a pulumi or terraform issue.
we are working with 2 different timeouts here, i guess: the terraform provider ones and the pulumi ones. maybe there are side-effects between those but it's necessary to know if terraform is applying them correctly or not.

Turns out you are right. I have created a very simple Terraform file to reproduce the issue and indeed its there.
(some of the) TF file:
resource "proxmox_virtual_environment_vm" "timeouttester9000" {
  bios          = "seabios"
  vm_id         = "1301"
  name          = "timeouttester9000"
  scsi_hardware = "virtio-scsi-pci"
  node_name     = "test-devops-proxmox02"
  timeout_clone = 180
  timeout_create = 180
  timeout_move_disk = 180
  timeout_migrate = 180
  timeout_reboot = 180
  timeout_shutdown_vm = 180
  timeout_start_vm = 180
  timeout_stop_vm = 180

  agent {
    timeout = "3m"
  }
  cpu {
    cores   = 3
    sockets = 1
    type    = "qemu64"
  }
...

First run, nice timeout at 3 mins mark:
Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

Please wait for Terraform to exit or data loss may occur.
Gracefully shutting down...

Stopping operation...
╷
│ Error: error waiting for VM shutdown: error retrieving task status: failed to perform HTTP GET request (path: nodes/test-devops-proxmox02/tasks/UPID:test-devops-proxmox02:000425B4:063B204F:66017E41:qmshutdown:1301:root@pam:/status) - Reason: Get "https://<IP_ADDRESS>:8006/api2/json/nodes/test-devops-proxmox02/tasks/UPID:test-devops-proxmox02:000425B4:063B204F:66017E41:qmshutdown:1301:root@pam:/status": context canceled
│
│
╵
╷
│ Error: execution halted
│
│
╵
╷
│ Error: execution halted
│

9 min when eventually I get bored and cancel it.
I'll open a ticket there.
Thank you for checking!


Matching service

Implemented backend of matching service.

Pls note that this is not using queuing technologies like RabbitMQ yet. However, this version is ready for demo.

Thanks for the work fahim, could you use the structure that we are using for questions-service and users-service so that we have proper separation of concerns between the functional layers of the backend (i.e. entry point, domain and data-access), and setup eslint? The eslint configurations can be taken from users-service.


block users from sending empty playlist ids

this breaks the app, also block operations from the backend too

this have been done with commit https://github.com/Playlistic/Playlistic/commit/03f1dfa77ad1020bc50c9fbbdcf2c3f50b9bbe40


Required minimum Node.js bump for Azure SDK

Minimum Node.js version may be incorrect This issue was created because the following file has been flagged with an out-of-date version number for Node.js: {filename and path} Required Fix The Azure SDK now supports a minimum Node.js version of LTS (long term support). Please fix this file to indicate LTS support and remove the specific version number it currently references found on line {line number}. If you find your code sample no longer works when moving up to the LTS version, please log an issue against the specific Azure SDK at https://github.com/azure/azure-sdk-for-js. Required Exception If you can't change this file to support the minimum LTS of Node.js, please comment on this issue with the reason and details so we can refine our file search.

Document Details

Content: Create a Terraform base template in Azure using Yeoman
Content Source: articles/terraform/create-base-template-using-yeoman.md
Service: ****
GitHub Login: @
Microsoft Alias: barbkess

@bobbytreed Closing issue as this article doesn't tell the customer which node version to install. (Also, per guidelines, please open issues in our private repo.)


Use the AP mode

Now this lib is running at STA mode, can it use at the AP mode ?
That means  I can use the ESP32 as the broker without another WLAN. thx.

Yes, you can.


Link to weights is broken

Hi,
Thank you for sharing the weights. With this being said, the link is broken :(
Can you fix it?

Sorry, we had to delete the weights as we ran out of free storage.


Remove text formatting from tile objects

Fixes #76
Hopefully this works. Let me know if there's any issues you see!

By "hopefully this works", did you test it out yourself? Or did you mean something else? :P

It seems to work as far as I can tell.


Matterbridge as a linux systemd service

Add scripts to have matterbridge run as a systemd daemon in Linux.

Hi, thanks for the PR!
It seems like a mix between a sysv initscript and systemd though?

Could be. I cobbled it together based on the mattermost init.d script, I'm no expert. We needed it because when a connection drops, de bridge seemed to stop working. I decided that having it as a service would be nice, so we can do an easy restart, while also having the benefit of automatic starting at boot.
If you have tips or pointers on how to improve the script I'd be happy to improve this pull request.

An example for a true systemd service unit can be found here
https://github.com/42wim/matterbridge/issues/116#issuecomment-299642840
Normally you put those files in /etc/systemd/system
The daemon-reload, enable and start commands remain the same.

I have this:
[Unit]
Description=matterbridge
After=network.target

[Service]
ExecStart=/usr/bin/matterbridge -conf /etc/matterbridge/bridge.toml
User=matterbridge
Group=matterbridge

[Install]
WantedBy=multi-user.target

Should I make a pull request? It's helpful when projects have some standard service files. :)

@heinrich5991 looks good, sure, go ahead :)

FYI, this is something related, which you might consider directing folks toward instead of including init scripts :)
https://github.com/jordansissel/pleaserun

So this would mean:
sudo mkdir -p /opt/matterbridge/bin
sudo cp matterbridge-$ARCH /opt/matterbridge/bin
gem install pleaserun
pleaserun --install  /opt/matterbridge/bin/matterbridge

Puts it in the users hands to install ruby, but then it works on any platform pleaserun supports :)

@patcon feel free to edit stuff about this on the wiki, I'm going to close this PR now as @heinrich5991 #291 is merged.


Adds init scripts

Simplifies starting up different druid notes with convenience bash scripts.
Example:  ./bin/historical start

@waltonseymour just got your corporate CLA, thanks

:+1:

👍
@waltonseymour might be good to have another PR that updates the tutorials to use these new scripts


Update cloudflare starter to use Pages correctly

Fixed by https://github.com/BuilderIO/qwik/commit/ab9c1eb651812be03bc7a052fe8124659565c41e


Fresh install. node-balancer fails to start.

haibu-server works just fine but when i try to run haibu-balancer I get the following error.
/usr/lib/node_modules/haibu/node_modules/http-proxy/lib/node-http-proxy/http-proxy.js:52     throw new Error('Both `options` and `options.target` are required.');           ^ Error: Both `options` and `options.target` are required.     at new <anonymous> (/usr/lib/node_modules/haibu/node_modules/http-proxy/lib/node-http-proxy/http-proxy.js:52:11)     at new <anonymous> (/usr/lib/node_modules/haibu/lib/haibu/balancer/balancer.js:58:20)     at startServer (/usr/lib/node_modules/haibu/lib/haibu/balancer/index.js:52:20)     at initialized (/usr/lib/node_modules/haibu/lib/haibu.js:48:7)     at /usr/lib/node_modules/haibu/lib/haibu/utils/index.js:366:5     at /usr/lib/node_modules/haibu/node_modules/async/lib/async.js:126:25     at /usr/lib/node_modules/haibu/lib/haibu/utils/index.js:361:7     at /usr/lib/node_modules/haibu/node_modules/mkdirp/index.js:10:21     at path.js:292:19

I'm having the same issue.
Any chance haibu-balancer will get back on its feet?  If it's no longer under active development, do you have any suggestions of what to use for dynamically deploying multiple drones routed via proxy/subdomains?  Looking into rolling my own with http-proxy, although I'm not sure how to best handle the integration with haibu and dynamically altering routes...

I'd try to jump into the binary for haibu-balancer and try to do a pull request to get it working again. I don't think it's going to be very hard, it's not much code.

"Perhaps there's a simple way to dynamically load/unload proxies via node-http-proxy?"
This is exactly what I tried to do in the end:
https://github.com/bradleyg/nserv/blob/master/www.js
and it works... kind of, it's fiddly. i finally managed to get a nodejitsu account so left the project redundant.

haibu-balancer is no longer supported.


Bug 2061616: [release-4.10] Pre-compute  bundle-plugins.txt

Pre-compute  bundle-plugins.txt

/close


[Docs] Fix info about Input events

Document Input's "input" event, similar to HTML's input element
Fix mistake in Input's "change" event description

@dodas update Chinese and Spanish docs.


Update to Product Updates

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

CC @schoudhury @KMarquart89

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


[1799] Fix  in IE9

Context
mailto links look a bit wonky in IE.
Changes proposed in this pull request
The tag does not seem to work in some versions of IE. This adds some CSS to patch it up. Copy and pasting still works as expected.
Guidance to review
Repro of the fix is deployed at https://ie9-wbr-fix.tvararu.now.sh.

Before
After

 Basic details IE error


Ignore public keys that are not in ssh format

It seems that some ~/.ssh/*.pub files are public SSL keys, so add some minimal validation before using them as authorized_keys.
Ref https://github.com/lima-vm/lima/issues/104#issuecomment-914030614

still getting this issue

@asdfghjkl403622 We would need to see the log files to check this further, but I wonder if this isn't related to your all-numeric username, so maybe test again once #243 has been merged?


Tooltip - missing isVisible/controlled tooltip functionality from ap-components-react

Tooltip - missing isVisible/controlled tooltip functionality from ap-components-react
Detailed description
We currently using ap-components-react, and we need to migrate to carbon-components-react.
We have around 600 inputs across 230 files, which are wrapped by ap-components-react Tooltip, with isVisible, nowrap and hideOnHover properties.
In case of error in input field, error/validation message is displayed as tooltip, and hidden when value is correct, or displayed always during input modification. The tooltip is used, since input fields are in general to narrow to display any message inside or under the input. The hideOnHover sets pointer-events: none.

Currently we are relay on controlled Tooltips from ap-components-react.
Without this isVisible/controlled tooltip functionality, it will be difficult to perform migration.

Hi @tomkdgun! 👋 Am including our designers to weigh in on this 😄
cc @IBM/carbon-designers is this something that we should support?

We have a similar tooltip pattern in out Data tables for edit mode. I think the tooltip is a fair responsive solution for small fields however on larger fields if there is room to show the message below the field I think its is better ux practice to show it that to hide it in a tooltip.


Broken links in institutional header

The links to the RDM policy and RDM info don't resolve anywhere. They shouldn't show if nothing has been added.

These are just placeholders for now. I'll add in the code to make them functional (duplicate of #729 )


cannot correctly adjust positions of axis titles

Hello,  @marcomusy
I am trying to visualize .vtu files from a specific camera view such that it feels that a 2-D perspective mesh only exist. The vtu files contain all 3 coordinates by with y=0. I could ajust the camera view for my purpose, however, the axes' titles and labels are "out of mesh plane" and appear to be missing, as shown in the pictures here captured from a few different camera angles. The example code is attached. The source code of Axes object seems to suggest many parameters (e.g., xTitleOffset, yrange, etc ) are interacting with each other. Any solution to this ? Thanks.

** 2-D perspective view, zTitle appears to be missing **

** 2-D perspective view, zTitle and its labels are out of x-z plane **
Minimum-work-example of code:
#!/usr/bin/env python3

import numpy as np
## see https://vedo.embl.es/ for vedo module
from vedo import UGrid, show, screenshot, settings, Picture, Text2D, Text3D, \
    Arrow, Plotter
#from wand import image
settings.defaultFont = 'Theemim'
settings.useParallelProjection = True # avoid perspective parallax
settings.multiSamples=8

vtu_file = "true_model_1"
Is_full_mesh = 0
mesh_file = vtu_file + ".vtu"

# unstructured grid object
ug = UGrid(mesh_file)

picture_file = vtu_file + ".png"
print( "writing file: " + picture_file)
# tomesh: Build a polygonal Mesh from the current Grid object.
if Is_full_mesh > 0:
    msh = ug.tomesh().lineWidth(1).lineColor([7, 15, 190])
else:
    msh = ug.tomesh().lineWidth(4).lineColor([7, 15, 190])

xlabels = [(-40000, '-40000'), (-20000, '-20000'), (0, '0'), (20000, '20000'), (40000, '40000')]
zlabels = [(-50000, '-50000'), (-25000, '-25000'), (0, '0'), (15000, '15000')]
# make a dictionary of axes options
axes_opt = dict(
    xyGrid=False,
    yzGrid=False,
    zxGrid=False,
    zxGridTransparent=True,
    showTicks=True,
    xtitle='Easting (m)',
    xTitlePosition=0.6,  # distance along axis
    xTitleOffset=[0.0, 1.0, 0.0],   # distances from axes
    #xTitleRotation=[0, -90, 0],
    #xLabelRotation=[0, 90, 0],
    xValuesAndLabels=xlabels,
    xLabelSize=0.018,
#    xrange=msh.xbounds(),
    xrange=[-40000, 40000+2000],  # range of values of x-axis
    xAxisRotation=90,  # rotate X axis elements (ticks and labels) around X axis.
#    ytitle='',
#    yTitlePosition=0.3,  # y is not used
#    yTitleOffset=0.2,
    yrange=[0, 5000],   # random values here; needed to ajust xTitleOffset; see source code for proper value
    ztitle='Northing (m)',
    zTitlePosition=0.65,
    zValuesAndLabels=zlabels,
#    zLabelRotation=90,
    zLabelSize=0.018,
#    zrange=msh.zbounds(),
    zrange=[-50000, 15000 + 2000],
    axesLineWidth=2,

    xUseBounds=False,
    yUseBounds=False,
    zUseBounds=False,
)

# add texts
# arrow = Arrow(startPoint=(504000, 6413500, 512),
#               endPoint=(504800, 6414150, 512),
#               s=3, c='w', res=200)

plt = Plotter()
plt.camera.SetFocalPoint( [0,0,0] )
plt.camera.SetPosition( [0,-5000,0] )
plt.camera.SetViewUp( [0,1,0] )
size = [3940, 2160]
plt.show(msh, axes=axes_opt, size=size, interactive=True, zoom=1.2)

screenshot(picture_file)

true_model_MWE.zip

sorry i don't understand.. you don't have a z depth in the file how should be the z-axis visible..?
and if it's 2D in the xy plane the z is irrelevant anyway..

sorry i don't understand.. you don't have a z depth in the file how should be the z-axis visible..?
and if it's 2D in the xy plane the z is irrelevant anyway..

The coordinates are 3D and have a z depth there. It's just that the y coordinates are all 0. As such, the mesh is essentially a 2D mesh.

I think this is probably a bug... but in any case you can always modify everithing in the axes manually by unpacking:
axes_opt = dict(
    xyGrid=False,
    yzGrid=False,
    zxGrid=False,
    zxGridTransparent=True,
    showTicks=True,
    xtitle='x - Easting (m)',
    xTitlePosition=0.6,  # distance along axis
    xTitleOffset=[0.0, 1.0, 0.0],   # distances from axes
    #xTitleRotation=[0, -90, 0],
    #xLabelRotation=[0, 90, 0],
    xValuesAndLabels=xlabels,
    xLabelSize=0.018,
#    xrange=msh.xbounds(),
    xrange=[-40000, 40000+2000],  # range of values of x-axis
    xAxisRotation=90,  # rotate X axis elements (ticks and labels) around X axis.
    ytitle='',
#    yTitlePosition=0.3,  # y is not used
#    yTitleOffset=0.2,
    yrange=[0, 5000],   # random values here; needed to ajust xTitleOffset; see source code for proper value
    zValuesAndLabels=zlabels,
    ztitle='z - Northing (m)',
    zTitlePosition=0.65,
    zTitleOffset=0,
    # zLabelRotation=90,
    zLabelSize=0.018,
    zLabelOffset=[0.9,0,0],
    zLabelRotation=0,
    zAxisRotation=-5,
#    zrange=msh.zbounds(),
    zrange=[-50000, 15000 + 2000],
    axesLineWidth=2,
    # xUseBounds=False,
    # yUseBounds=False,
    # zUseBounds=False,
)
axes = Axes(**axes_opt) #############

for a in axes.unpack():
    #print(a.name)
    if 'ztitle' in a.name:
        a.shift(-2000, 5500, 0)

plt = Plotter()
plt.show(msh, axes)

Thanks for the temporary solution above.
I can guess why the y-shift is 5500 here:
for a in axes.unpack():
    #print(a.name)
    if 'ztitle' in a.name:
        a.shift(-2000, 5500, 0)

What about x- and z-shifts here ? Why these numbers ?

sorry for the late reply.. the numbers are the shifts in world units... but it's basic trial & error until you get the desired output..


Section called "Use Microsoft Graph API to make this a root domain" is outdated and suggests using a deprecated method

Section called "Use Microsoft Graph API to make this a root domain" is incorrect. That's an Azure Active Directory Graph API method, not a Microsoft Graph API method. Unfortunately Azure Active Directory Graph API is deprecated, and new App Registrations in AzureAD can't use it this:
POST https://graph.windows.net/{tenant_id}/domains/child.mydomain.com/promote?api-version=1.6
This documentation needs to be updated with the correct Microsoft Graph API endpoint to complete this task. I tried this, but it didn't work:
POST https://graph.microsoft.com/v1.0/domains/child.mydomain.com/promote

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

ID: 02c07bd2-01c5-77f4-11e0-6d942c5c0885
Version Independent ID: f61490c2-0f18-4609-29f1-99ccb4215ad2
Content: Change subdomain authentication type using PowerShell and Graph - Azure Active Directory
Content Source: articles/active-directory/enterprise-users/domains-verify-custom-subdomain.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@firth
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#reassign:@barclayn

@firth thank you for your feedback. I have created a work item to track this internally and address your concerns.

#please-close


fixed errors is undefined

in MongoMQ.js line 64, you'd get errors is undefined

Thanks for the PR, merged.  Will version and publish.


Store solution post parameters as separate fields apart from description

Problem
Currently, the solution post type adds only name and description fields to the post snapshot structure.
This creates issues, when it is impossible to edit and atomically show certain parameters of the solution post, like requested amount of funds, currency, or requested sponsor.
User story
As a DevHub developer, I need to be able to get and store solution post metadata separately from the description, so that I can implement features that rely on solution post metadata fields.
Acceptance criteria

The solution post data structure ( Submission ) has the following properties:

amount: Option<Balance>
sponsorship_token: Option<SponsorshipToken>
requested_sponsor: Option<AccountId>

Regular solution posts and solution posts posted as replies have the same structure.

It's possible to atomically edit any Solution post properties.

The contract data integrity is preserved after the corresponding migration.

For all the preexisting solution posts, the missing fields should be populated with blank values.

All blocked ticket display features implemented in https://github.com/near/neardevhub-widgets/issues/250 should be available on the front end.

@ori-near I encountered some time-consuming difficulties with updating the post editing code because of its implementation details. I would like to add 3 more days to the estimation because I believe it's the amount of time needed to finish and test everything

Hi @carina-akaia – can you please sidebar with @ailisp today to align on this ticket, your progress so far, and some of the difficulties you encounters? Just want to see if he can give any guidance to help unblock and reduce the complexity.

@ori-near okay!


shuffle-related random failures

Tests are randomly failing with some frequency (maybe 1 in 10 times):
FAILED tests/benchmarks/test_join.py::test_set_index[1-p2p-True] - RuntimeError: P2P shuffling 1ae921940dd916ee7eeb387609c98fe2 failed during transfer phase
https://github.com/coiled/benchmarks/actions/runs/6262315429/job/17004221888
FAILED tests/benchmarks/test_join.py::test_set_index_on_uber_lyft[p2p-client0] - RuntimeError: shuffle_barrier failed during shuffle 9c5bfc79791693bcedf99f51bea8c3d8
https://github.com/coiled/benchmarks/actions/runs/6262315429/job/17004219737
CC @hendrikmakait

Two more examples:
https://github.com/coiled/benchmarks/actions/runs/6262315429/job/17012993085
https://github.com/coiled/benchmarks/actions/runs/6262315429/job/17012990170

This could be because of spot interruption. On my phone so can’t search for the cluster

I was under the assumption that our benchmarks were not using spot, apparently false

at least one of those failures is https://cloud.coiled.io/clusters/276019/information?account=dask-benchmarks which was due to a connection failure, not spot interruption

It looks like these not popping up anymore.


every_param bug

POST /foo/bar/baz.html?foo=13&foo=bar&foo=baz
in controller:
$c->req->every_param('foo') is [13, bar, baz]
$c->every_param('foo') is [baz]

Sorry, issue work only with param name 'permissions';

Fucking stash :(

POST /foo/bar/baz.html?foo=13&foo=bar&foo=baz
$c->req->every_param('foo') is [13, bar, baz]
$c->every_param('foo') is [13, bar, baz]
$c->stash('foo' => 'boo')
$c->req->every_param('foo') is [13, bar, baz]
$c->every_param('foo') is [boo]


Wizja.tv: crash if channel has no fanart.

Going into Wizja.tv menu has no effect - nothing happens after short loading. xbmc.log does not show anything useful. I've investigated the issue and looks like py.tv:wizja_list() handles incorrectly a case, when there's no fanart:
                try:
                    fanart = control.addonFart()
                except:
                    fanart = '0'
                    pass
                fanart = fanart.encode('utf-8')

addonFart() returns None without hitting exception. Then, .encode() is called on None, what obviously generates AttributeError.
Here's the simple fix:
diff --git a/plugin.video.mrknowtv/resources/lib/indexers/tv.py b/plugin.video.mrknowtv/resources/lib/indexers/tv.py
index cf44057..9e9d72a 100644
--- a/plugin.video.mrknowtv/resources/lib/indexers/tv.py
+++ b/plugin.video.mrknowtv/resources/lib/indexers/tv.py
@@ -209,10 +209,10 @@ class tv:

                 try:
                     fanart = control.addonFanart()
+                    fanart = fanart.encode('utf-8')
                 except:
                     fanart = '0'
                     pass
-                fanart = fanart.encode('utf-8')

                 plot = '0'
                 plot = client.replaceHTMLCodes(plot)
--

Dzięki. Poprawiłem. Obecnie powinno działać poprawnie


cmd/go: add go mod download to download modules

What version of Go are you using (go version)?
go1.11beta2
Does this issue reproduce with the latest release?
Yes (1.11beta2)
What operating system and processor architecture are you using (go env)?
amd64 - linux
What did you do?
Copying just go.mod into an empty directory should allow go mod -vendor (or maybe even go mod -sync) to download the dependencies regardless of having any other files within the directory. Without this, you cannot use something like Docker to cache the dependencies independent of the codebase itself.
$ docker run -it --rm golang:1.11beta2-alpine /bin/sh
/go # apk add -u git
/go # mkdir /foo
/go # cd /foo

/foo # echo "module github.com/sgnn7/test
>
> require (
>         cloud.google.com/go v0.25.0 // indirect
> )
> " > go.mod

/foo # cat go.mod
module github.com/sgnn7/test

require (
        cloud.google.com/go v0.25.0 // indirect
)

/foo # go mod -vendor
go: finding cloud.google.com/go v0.25.0
go: no dependencies to vendor

/foo # go mod -sync
/foo # # Nothing

What did you expect to see?
go mod should have downloaded go.mod dependencies based on go.mod content regardless if there's code in the repo or not.
What did you see instead?
Neither go mod -vendor nor go mod -sync downloaded anything.

Without this, you cannot use something like Docker to cache the dependencies independent of the codebase itself.

Can you include some more detail about your use-case? (Why is that a thing you want to do?)

@bcmills Sure.
Let's say that you have your code build a Docker image on every git change (most pipelines do this and is probably the biggest use case).

With dep you could create a Docker layer of cached dependencies by just copying Gopkg.* and fetching the dependencies after which you then copy the full repo and build the project (see here for the original issue for dep on this). This meant that unless you changed the Gopkg files, no dependency re-download was ever needed, making builds with non-dependency changes quick and efficient.
With current go mod, you cannot just copy go.mod and go.sum and then run go mod -vendor to get this feature parity since it just doesn't download anything without Go files in the repo, forcing the Docker layer to be cache-busted on the smallest of changes in the repo (e.g. README), causing re-download of all dependencies every time which is both slow and extremely inefficient.

@rsc Hmm.. I'll give that a try. Is there a reason why this isn't a base go mod command?

@rsc: Your cache warming suggestion doesn't work out of the box. The fixed version is below:
go list -e $(go list -m all 2>/dev/null)

IMO, If this seems like something that should be used often, it probably needs to get into the main CLI. As a sidenote to this hack, go list has no distinction in docs as to why it is the only one that can download modules vs -sync and -vendor.

All the commands downloads modules as needed.
sync and vendor are defined to chase down only the package-granularity dependencies of the packages in the main module.
If there are no packages in the main module, they chase down nothing.
List on the other hand can be asked directly about dependency packages,
and answering requires downloading them to look at the code.
It's certainly not a direct answer, but it should work.
That said, it seems to me that go get -d -m asks for exactly this
operation and fails to deliver it (the -d is ignored with -m).
We should fix that.

I got pretty close with https://github.com/golang/go/issues/26610#issuecomment-408172520. To make this work for now, I am using the following Dockerfile:
FROM golang as builder
RUN go get -u golang.org/x/vgo
WORKDIR /go/src/github.com/$ORG/$REPO

# Populate the module cache based on the go.{mod,sum} files.
COPY go.mod .
COPY go.sum .
RUN vgo list -e $(vgo list -f '{{.Path}}' -m all)

# Build all binaries, with -getmode=local, this will _not_ fetch packages over the network.
COPY . .
RUN vgo install -getmode=local ./...

# Runtime image
FROM alpine AS base
COPY --from=builder /go/bin/$PROG /bin/$PROG
ENTRYPOINT ["/bin/$PROG"]

@moretea This takes care of caching downloads but if I add -v to the go install/build the dependencies are being built each pass still so build/install times can still be very slow if you have a lot of dependencies (firebase, grpc, kubernetes etc).

Small update for those that are using the go list to fetch the packages: https://github.com/golang/go/pull/26631 PR by @marwan-at-work should remove the need to pipe stderr to /dev/null once it gets into the releases.

@sgnn7 the CL was closed by @myitcv, not sure if it needs to be re-opened. But my own use case for having that CL is resolved by just using stdout.
Thanks

@marwan-at-work I don't know why that issue would be abandoned 😢. Maybe it's because the v1.11 GA release date is tomorrow?
As for using stdout, that's what the interim fix uses but it would be nice not to need 2>/dev/null in it to separate stdout from stderr.

@marwan-at-work I closed the CL based on our discussion, yes. Did I prematurely close it? Was there something I missed?
@sgnn7 as things stand, stdout is defined to contain the JSON output; stderr informational messages, with the exit code ultimately used to then determine success or otherwise (with the caveat that -e can alter the handling of erroneous packages, per go help list) of the list operation.
Does that help to clarify things?

@myitcv CL being closed is good on my side :) Just wanted to ping you about @sgnn7's use case.
Thanks

@myitcv Makes sense though I wouldn't say that it increases simplicity/usability cases (compared to most other *nix tooling).  Either way, this issue is not for that discussion and thank you for the clarification. 👍

I started looking at go get -d -m but it's a weird combination to redefine.
Instead I am going to add a 'go mod download' that does what you want.
So you could run
go mod download

to warm a Docker image cache.

@rsc will this new subcommand take care of this issue? https://github.com/golang/go/issues/26577
Most notably, go mod download would not download dependencies except just the targeted import path.
Would love to help in any way.
Thanks!

There is a caveat here, you must have a main package in the directory with go.mod for go mod download.

There is a caveat here, you must have a main package in the directory with go.mod for go mod download.

@zangbuild no longer the case with go1.12, there's already a beta out if you want to give it a shot.

Nice. I will try when I can.

Does go mod download also build (and install) the packages?

@mbana go mod download just downloads content for a module, it does not build and install the package nor does it fetch its dependencies.

Hi, related question: is there a go mod download-from-vendor for those already having everything in their vendor directory? Thanks!

@mcluseau, the vendor directory stores individual packages, not complete modules. It is not a suitable input to the module cache.

So just to follow up, the best way I've found considering that is to use a Go proxy (athens) and the following Dockerfile:
from golang:1.12.1-alpine3.9 as build
run apk add --update git

env CGO_ENABLED=0
arg GOPROXY

workdir /src
add go.mod go.sum ./
run go mod download

add . ./
run go test ./...
run go install . ./cmd/...

from alpine:3.9
#entrypoint ...
copy --from=build /go/bin/ /bin/

is go mod download designed to pre-download all go.mod defined dependencies? I found a case might be a bug of it, because I have a similar case want to pre-download as cache and build go code later,
while, in my case, I found go mod download indeed download many dependencies but not all of them,
the later go build ... command still try downloading some last dependencies,
$ go mod download
....
$ go build ...
go: finding github.com/alecthomas/template latest
go: downloading github.com/alecthomas/template v0.0.0-20160405071501-a0175ee3bccc
go: finding github.com/alecthomas/units latest
go: downloading github.com/alecthomas/units v0.0.0-20151022065526-2efee857e7cf
go: extracting github.com/alecthomas/units v0.0.0-20151022065526-2efee857e7cf
go: extracting github.com/alecthomas/template v0.0.0-20160405071501-a0175ee3bccc

I'm not sure does this mean go mod download failed to parse a complete list of dependencies? leaked a few to go build command,  should be filed as a go mod bug?   here is using FROM golang:1.12-alpine AS builder

@c0b, if all dependencies are declared (that is, if go mod tidy is a no-op), then go mod download is intended to download all needed dependencies.
If you find that is not the case, please open a new issue with steps to reproduce (ideally using go1.13beta1 or newer).

So yeah go mod download needs a verbose option, please
https://github.com/golang/go/issues/38756
go mod download --verbose

or
go mod -v download

whatever, o/w I am just watching the command hang there forever, hoping that nothing is wrong. some feedback would be nice, like with go install -v


On ios with open chat show chat icon

Hi. When you open chat to screen add an icon with chat. How can I delete or hide it? only on ios.

[[[ZDCChat instance] overlay] setEnabled:NO];


Release 0.2.0

TODOs

[x] Review the changes in the release commit
[x] Update the Release notes
[x] Review the Draft Release
[x] Review and approve this PR
[x] Trigger the Perform Release Workflow
[ ] Once the  workflow is through, head over to the Central Sonatype and log in with the credentials in the Team Password Safe. There should be a published release.

Requires #168


Permission / Missing system directories (/etc/xdg/pytaskmanager/)

(from @jvsoest) on linux, the folder (and subfolders) in the system-directory are not setup correctly when using pp[...] new.
Work around is to create these manually:
sudo mkdir /etc/xdg/pytaskmanager
sudo chown $(whoami): /etc/xdg/pytaskmanager

sudo mkdir /usr/local/share/pytaskmanager
sudo chown $(whoami): /usr/local/share/pytaskmanager

On second thought, I think this is expected behavior. If you want to use system folders the user you are using to run the command should have permission to read/write in these folders. You can either execute vnode/vserver  using sudo, create these folders manually (not preferred) or use the user-folder.
Please let me know your thoughts on this.


[cleanup] Teach the precompiler and program visitor where ffi callback functions are

We should teach the precompiler and program visitor where ffi callback functions are.
Context: https://dart-review.googlesource.com/c/sdk/+/158399/3/runtime/vm/program_visitor.cc
cc @sstrickl

In https://dart-review.googlesource.com/c/sdk/+/274387/10/runtime/vm/compiler/aot/precompiler.cc we've added some logic.


Mat/Pat - Improve partner info UX

changing the email but not hitting get info - unclear what would happen, and it can cause some issues

The page where we get partner info - email id and employee number!
If someone doesn't hit "Get info", the UX for that page needs to be worked on.


[Numpy] Support N_D(N>=3) batch_dot

Description
Now we support N-th dim batch_dot. It solves https://github.com/apache/incubator-mxnet/issues/10386
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at https://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] batch_dot (ndim >=3), tests

@eric-haibin-lin Can you take another look?


Add a Bounds model component

We want to easily determine whether something is "in a region" in Creator projects. One way to do that is to add Bounds as a model component you can create. Then there will be an easy way in the interface to write functions that control other components when a position is within those bounds.

Documenting steps to add a bounds component here. There is quite a lot, maybe we can look for ways to abstract these steps.

Create a model class for the component that extends NamedProperty. Needs to implement save, load, getStateSchema.
Add an observable array for instances of the component to the ProgramModelContainer. Add functions to add/remove the component from the container. Add the observable array to save/load for the container. Add the observable array to dispose.
Make sure that projects will load after this schema change. Either be graceful in the load code or modify save data for existing projects.

I updated the schema at the database, but realized that would not be enough. People will be saving data with old versions of paper-playground checked out, sending the old schema. Next time they pull main those projects will fail to load. So we need these load functions to be graceful.

Add a form in React to create the component. There are many steps here, but in general follow the pattern established by the other components. Add lots of data to ComponentForm.js. Then create a new tab and form for the specific data you need. This is probably where a lot of refactoring could be done.

Noticed while testing this new component.
I think it is important for the position controllers to set positions in normalized coordinates. Currently they automatically convert to display coordinates. That is convenient in the view code because you can just go like this:
setCenterX( positionComponent.x );
setCenterY( positionComponent.y );

But it means working in model coordinates is not really possible. For this issue we wanted to easily determine if a point is within a set of bounds. That requires both bounds and point values to be in the same normalized coordinate frame. After this change view code might look like this:
setCenterX( unitPositionToDisplayPosition( positionComponent ).x );
setCenterY( unitPositionToDisplayPosition( positionComponent ).y );

OR maybe there is a checkbox that would automatically do that conversion for you (default checked?)  This change could disrupt a lot of existing projects and I am hesitant to go ahead just before a codesign. I think it is required to get the Bounds component working though.

Lets add a checkbox here that would automatically convert to display coordinates in the generated code.

If that works, current projects will behave the same. And I think that will be useful for most cases because in view code where you are updating positions you are generally linking directly to a model component.

OK, this change is merged into dev branch. This has been working pretty well, we will test a bit more before merging into main.

Merge complete.


[BUG]: IntegrationError loaderUi Element didn't mount normally.

What happened?
While using stripe checkout, we found a unexpected error on:
    "@stripe/react-stripe-js": "^1.6.0",
    "@stripe/stripe-js": "^1.19.1",

Here is the public details for the error:
https://sentry.io/share/issue/c98c0faf67194c2d8c7ddcd0a483e684/
Environment
Chrome 108.0.5359.124
Reproduction
https://letsumai.com/widget/zen-by-mel

Hi @m-nathani, I'm unable to reproduce this error at all from your link. If you've got anything else we can look at or more information about how often this is happening, in what browsers, etc. then we can take another look. You might also try reaching out for help on the Stripe Developers Discord, where plenty of engineers are available to help debug issues like this.


Request: Date Picker fontFamily

is possible to add fontFamily for DatePicker

@SateJoen Yep, it will be supported in next version soon.

This is supported in 2.3.8.


initial photometry cuts code commit

Good ideas! I'll implement them.

The newest code in https://github.com/Scylla-HST/scylla-data/pull/1/commits/286063eb150ab50d839a41b1b0bd5df4ec9c5b4f has been corrected and updated per discussions regarding the utility of recording a catalog file - still named 'vgst' - which includes only good sources. We settled on removing all sources with a zero flux in any band, and all sources identified by Dolphot with flags other than '0' or '2'. (For BEAST fitting purposes a non-zero flux is needed in all bands for a given source.) The original quality cuts of crowding, sharpness squared, and roundness are preserved. The catalogs may contain sources with Vega magnitudes of 99.999 in a band as long as they have a non-zero flux in the same band, hence the potential difference in the source counts between the spatial and CMD plots. A sample new plot is attached for Scylla's SMC-1.


Missing release

Can u provide a compiled version of the plugin please? :)

I have no idea how to install this :) there is an empty release section.

Some compiled files are available here https://github.com/kus/cs2-modded-server/tree/master/game/csgo/addons/counterstrikesharp/plugins/disabled/CS2Multi1v1


Fixed issue #1488

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Thanks @Raghav17 for this PR, but this change is already merged by this PR https://github.com/openebs/openebs/pull/1489


[BUG] Some builtin algorithms in GAE is the single-thread version and not every performant in multi-core environment

Describe the bug
Many builtin algorithms (e.g., pagerank) in GAE (see https://github.com/alibaba/GraphScope/blob/main/coordinator/gscoordinator/builtin/app/.gs_conf.yaml) doesn't use the parallel version and is not the fastest implementation variant, and doesn't scale to many cores in a multi-core environment, as we current encourage 1 fragment per worker, especially on Kubernetes.
Additional context
Add any other context about the problem here.

@lidongze0629 could you please take some time to sync the builtin algorithm variant with the best implementation reported in libgrape-lite?
Thanks!


fix: add messageId property to message definition

issue https://github.com/asyncapi/asyncapi/issues/458

As @derberg said earlier, there is no clear versioning / semver for Schema itself. We have been fixing stuff in 2.0.0, without version bump.

We have been merging fixes to the schema, which is understandable. This introduces a new feature and that's totally different.

Ok. Is there some roadmap/plan for 2.1.0? Are there any other new features on hold?

There are no plans yet. In the upcoming weeks, we'll be announcing a few things that should allow us to move faster :)
Regarding things on hold: https://github.com/asyncapi/asyncapi/compare/2.0.0...master. There's at least the Mercure binding waiting.

And also we have this issue from @lbroudoux to improve message examples.
We had those grouped under https://github.com/asyncapi/asyncapi/issues/353 (you need to have zenhub installed to see related linked issues to this epic) but that will need review as it is few months old

Ok. I'll be waiting for announcements ;)
You guys would be working on a lot of things but having a roadmap would help both sides. (users and core contributors)
I'll try to hop-on on the next weekly meeting to have some more details about this.

We do have a roadmap (and you're part of the organization on Zehub) but it's impossible for us to follow. That's why I decided not to create a new one yet. It would be utter bullshit from our side to plan things for a whole year when things are moving so fast. We'll follow a different approach soon, just give us time to reorganize :)

@fmvilas I see it. Yup, that is what I was looking for. 👍
I understand (and see) there's a bunch of stuff ongoing. Hopefully from next year, things will be in better shape.

@WaleedAshraf I suggest we close it and when you are ready to work on it you will open it up properly from a fork and point to a release branch. Agreed?


Support native build on MacOS

This allow to build and running all system native on macos. :)
Signed-off-by: Federico Guerinoni<EMAIL_ADDRESS>

Thank you!
Now that we have a macos compatible makefile, it would be nice to have a platform-agnostic makefile that works on linux too. So we can ditch the old build script.
I'm working on it


obsidian: use electron-24 which is not EOL

Description of changes
Obsidian is hard-coded to use electron 21, which is now considered "unsafe" and gives the user an error.
I've upgraded to 24 and quickly started an instance. Seemed to work fine.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build obsidian

Any reason not to bump to electron_24?

Any reason not to bump to electron_24?

It's a good point. Tried it and it works just as well.

Any reason not to bump to electron_24?

It's a good point. Tried it and it works just as well.

I'm not sure about stability guarantees. So, there might be a good reason not to. But, yeah, it works for me, too. I'd just like to delay this happening, again, for as much as possible.

Also, if we go this way it might be good to change the title/issue body (in case people search for this in the future).

Also, if we go this way it might be good to change the title/issue body (in case people search for this in the future).

Also an excellent point, I only adapted the commit message, but have now updated the rest as well.

@atila @conradmearns @zaninime @qbit
Can we get some eyes on this to get it merged?

I am torn. I feel like we should align as close as possible to what upstream is doing (so we don't introduce nixos-specific issues).

I am torn. I feel like we should align as close as possible to what upstream is doing (so we don't introduce nixos-specific issues).

Me, too. I wasn't able to easily figure out what version of electron is used upstream, though.

Result of nixpkgs-review pr 226591 run on x86_64-linux 1

1 package built:

obsidian

Me, too. I wasn't able to easily figure out what version of electron is used upstream, though.

In Obsidian v1.1.16 on macOS process.versions.electron in dev tools reports 18.3.5. It's also EOL.

In Obsidian v1.1.16 on macOS process.versions.electron in dev tools reports 18.3.5. It's also EOL.

Did a fresh install, and now it's 21.4.1. Seems their autoupdater fails to update electron.

@krav I think the ship may have sailed on this one. We can redress if the nixpkgs obsidian behaves/fails relative to the official version. I'll try to ping the Obsidian devs to bump their electron version.

According to joethei#4474 (Discord) they're working on a v24 release and it should be available soon. Better just to wait, I think.


#419 fix

Serialize the string order type even though scoring doesn't require it so that it doesn't get lost when reloading mleap bundles in spark 2.3.0.
Fixes #419.

@clocklear You can see the discussion under #419, for reference.

Alternatively, we could just make the stringOrderType optional in StringIndexerOpV23.scala, if the reloaded Spark pipeline will just be used for transforming only.

We should avoid serializing information that is only used at training time. If StringOrderType is only for training, let's avoid serializing it all together.

Makes sense, I've removed all references to string_order_type now, there's no need for a v23 op for string indexer.


Migrate remaining files to super parameters

Fixes https://github.com/flutter/flutter/issues/101455.
Part of https://github.com/flutter/flutter/issues/100575.

Verified that google3 is no longer breaking with this migration applied.