WIP: Make transmitting boundaries calmer

This PR consists of one fix and one... hack, for lack of a better term.
The fix: transmitting boundaries were not packing the velocities along with the density and internal energy, for some reason.  This meant that the ghost cells always had zero velocity, meaning that the pole exerted more drag than it needed to, since it lowered all the velocities around it.  I'll be checking the other uses of that particular flag pack; probably every instance should just be FillGhost+Cell, now that we explicitly mark anything we don't want to sync as StartupOnly.
However, the fix doesn't address the instability at the pole -- if anything, it made the problems worse (or at least more frequent).  So I've also written a band-aid for that instability which works frighteningly well.
The problem: in high-resolution simulations with KHARMA so far this year (and indeed, ever) there have been explosions at the pole.  The energy for these seems to be provided by the toroidal magnetic field next to the pole: B3 in the last circumpolar zones.  Checkerboarding in the magnetic field then causes a positive feedback loop of some sort, where the field strength dramatically increases, and the resulting pressure blows material and field lines into the disk and ruins the simulation (or in worse cases, the field grows so strong that the divergence check triggers and ends the whole simulation).
Note that physically, though, high B3 next to the poles represents tiny loops of field, one little loop encircling the pole itself at every radial zone.  If flux were allowed through the pole, the components opposite one another, pointing in opposite directions, would simply advect together and reconnect.  But since the pole does not allow flux (even though it does allow an EMF along it), the little loops just build and build throughout a simulation, powering all manner of unpleasantness.
So, my hackish solution is just to reconnect them manually, as it were.  Or at least, I think that's the net effect of what I'm doing.  I take B3 in each zone next to the pole, and just average the values together (we always assume axisymmetric geometry, so no need for face area weighting), and subtract the average from the value at each face.  All the face-to-face differences stay the same, so the divergence is preserved -- just, with the symmetric portion removed as if the loop had been advected together and disappeared.  Would it have done so without help?  Maybe!
This operation certainly shouldn't be applied to B1, as the component directions do not cancel around the pole.  There's a reasonable argument that it's allowable for B2 if it is for B3, but in practice B2 seems to reconnect just fine without help and disappears nicely at the poles.
TODO:

[x] Make sure none of the current tests suddenly blow up
[x] Option to turn this thing off, for the scrupulous and the adventurous

Some things I've learned:

This solution is quite magical and makes any simulation in spherical coordinates cleaner and quieter.  Energy is conserved, and I think the argument is strong that this is a globally neutral operation, tantamount to loss of resolution or perhaps excision.  Therefore I've enabled it by default for transmitting polar conditions.  YMMV applying it to reflecting conditions, but maybe it helps there too.

Subtracting just the minimum B3 by absolute value is insufficient to provide similar (any?) stability benefits.  Only centering B3 about 0 seems to calm it, not just lowering it until the smallest value is 0.

Applying similar averaging to U3 (under the same argument above) even further reduces polar effects -- though the pole still slows material moving across it, it no longer kills nearly the velocity it did without averaging.  HOWEVER, my implementation of U3 averaging does not conserve angular momentum, so it's hidden behind the option cancel_U3_, implemented per-boundary, e.g. cancel_U3_inner_x2.  YMMV, I am not responsible for lost T^0_3.

(4. This PR works as intended, so I'll be putting it into dev when the tests pass)

Merging since polar B3 reconnection works fine, but I'd urge caution using U3 cancelling.
It can cause material to get stuck next to the pole, or can preserve vacuum where material should be flowing.  That is, it minimizes disruption to momentum, at the cost of disrupting the local density, which seems like a bad trade-off -- especially as the density disruptions build over time.  There's almost certainly a way to write the averaging so that it doesn't produce these artifacts, but being primarily interested in aligned simulations, I am not here to find it.


⚠ Bofrost.be has degraded performance

In 5292cdd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 965 ms

Resolved: Bofrost.be performance has improved in 552c59c after 6 minutes.


Add Animation Import(s) to NLA Track List

I wish github had an ignore white space changes as part of the change view. =/
Anyway, this is a WIP patch to enhance importing multiple animations in 2.82+. I noticed that selecting multiple animation files and importing still only showed one animation as brought in the Track list. This patch additionally pushes the animation(s) to the NLA Tracks whereas before it was just getting setup in:
bpy.data.actions.new
Besides just bulk importing of animations, my use case is for exporting to glTF and importing to Godot. Adding these to the NLA Tracks list fixes glTF exports such that Godot sees and can play the given animations automagically.

When I say WIP Patch I mean that I'm curious if there are any concerns/issues that y'all would know that need ironing out. As far as I can discern, this works for me without issue. But, maybe my approach is bad or y'all know something about the track lists that I don't.

It looks good to me but I didn't write any of the animation stuff and don't know much about how Blender works in this regard. If @WHSnyder is able to quickly look at this and sign off on it that'd be great.

I'll admit I'm a bit confused in this regard. The code pushes it to the animations data list and I see imported animations there. But, I'm surprised/confused on why it doesn't add to the NLA Tracks list too. This doesn't touch export or anything. It still adds to the other list too. The thing is is you can manually push those to the Track List too but it's tedious. So, really I'm just making the process user friendly in that regard. =P
Based on my research of the Blender API you can actually pull the animation data from the NLA Tracks List. Maybe it might be worth moving the code to use it from there at some point. Again, still getting my feet wet so I might be off.

Ah yea when I wrote this I just never bothered to even figure out what NLA tracks were for since I wrote it for reverse engineering purposes.  But yeah this seems super useful so I'll have a closer look this week.  Thanks so much for your interest in the project!

I look forward to your review and I want to thank the both of you for your work! At one point I was working on trying to convert the XSIZETools to Python 3 and then wrap around that the Blender APIs and only got up to the point of meshes importing without textures when at some point someone on the Gametoast forums mentioned your project.
Honestly, I'm stoked by this project given it's easier to follow logic wise. And, I love Blender and having msh support is wonderful. I'm hoping to pick this project further but am certain if y'all gave up maintaining it that I could still work with it/patch it as Bender updates.

Removed arma.animation_data.action assignment b/c after checking we can still play animations and they still added to the arma.animation_data.actions list. Tested by exporting imported animations and just importing back the exported animation.

Nvm, forgot Active animation export still needs it. 🫣

Any progress on confirming this PR?

I'm happy for it to be merged if @WHSnyder is.


Rebase by default / Disable Merge

Description
With the release of https://github.blog/changelog/2022-02-03-more-ways-to-keep-your-pull-request-branch-up-to-date/
If there was a way to default to "Rebase Branch" -> "Update with Rebase"?
That would save a lot of accidental "Update Branch" -> "Update with merge commit".
This mode often puts a merge commit into the branch, which is not preferred.
As a workaround, we've written a linter to check for merges but that still wastes time for developers.
If defaulting is less preferred than outright removing it (or giving more options), disabling/removing  "Update Branch" -> "Update with merge commit" would be fantastic.
Screenshot

Proposal / Request

Example URL
https://github.com/refined-github/refined-github

This mode often puts a merge commit into the branch, which is not preferred.

That's just a preference. We can't change this default because everyone has a different way of doing things. For me the merge commits in the PRs are easier and in the end every PR is squashed anyway, so the presence of merge commits in it isn't a problem.
Again, it's a preference, so we can't change it as a default in Refined GitHub

I'll concede the preference, with some of the customization that refined-github offers, my intent was to see if it could be a flag that could enable this functionality. Feel free to close if it's not in scope!

That's understandable, but we only have enabled-by-default features
https://github.com/refined-github/refined-github/issues/2960#issuecomment-606765382


deps: Upgrade 'node-gyp' to avoid python3.12 issue

Since python3.12 became the default, the module has not been building properly due to a problem where node-gyp was calling upon a python module no longer included by default. This has been taken care of in node-gyp >= 10.

Built and tested on m2 mac. Working as expected. Great find @Alphrag !


Face Detection on Web: Accessing Facial Point Detections

Hello!
I am trying to use mediapipe's face detection capabilities on the browser but I am unable to access the facial landmarks detected by the application.
The demo I reference is this one:
https://viz.mediapipe.dev/runner/demos/face_detection/face_detection.html
So far I was able to download the html file and various other scripts, json and graph definitions by the use of chrome devtools' network interface, and make the app work locally on my machine by an impromptu node server.
Upon making the app work locally, I tried printing the value of the "detections" object referenced in the 42nd line (*) of the source code above, hoping it would contain the facial landmark points I am looking for, but it does not.
Since that lead me nowhere, I inspected the graph definition (cpu_oss_facedetect.pbtxt) to find a textual reference to a "Tensorflow Lite Model" used by the Mediapipe system on line 60 (**).
I could not find this model through google or through the catalogue of pre-trained tensorflow lite models on the tensorflow website
My goal is to gain access to and use this specific model the mediapipe demo above uses to make clientside inference.
Thus my question is two-fold:

Does the mediapipe API expose the facial landmark position information?
What is  the "Tensorflow Lite model" that the above demo uses, and how can I gain access to it?

I tried greping through all the files that the above demo uses for a reference to a model to no avail.
Thank you.
*
    videoProcessor.setOutputProcessor({
      process: (detections) => {
        let detectionString = 'No Detections.';
        if (detections && detections.current_mspf > 0.0) {
          // console.log(detections);
          detectionString = (1000.0 / detections.current_mspf).toFixed(2) + ' fps';
        }
        document.getElementById('message').textContent = detectionString;
      }
    });

**
# Runs a TensorFlow Lite model on GPU that takes an image tensor and outputs a
# vector of tensors representing, for instance, detection boxes/keypoints and
# scores.

https://blog.tensorflow.org/2020/03/face-and-hand-tracking-in-browser-with-mediapipe-and-tensorflowjs.html

Thank you very much for your help.

hey @tomatopuree could you please shed some light on exactly how you were able to " to download the html file and various other scripts, json and graph definitions by the use of chrome devtools' network interface, and make the app work locally on my machine by an impromptu node server" ?
I'm trying to do the same with this example https://viz.mediapipe.dev/demo/hand_detection

Hey Giovanni
If you can find a working link for the demo, you can press ctrl+shift+i on chrome, go to networks tab, refresh the page.
This will send all the uncached files that the server sends to you when you run the program.
I don't recall which of these files are essential for the demo to work. You'll have to use your best judgement to select those files and download them. I think I identify some of these files in the original post.
Then you can google "how to create a node server" and "how to serve html pages with node" and create one on your local machine to serve the main page of the files you've downloaded. I'm sure you can figure out how to serve a page with a node server.
Normally after downloading a html file, you can double click it and bypass having to make a server to serve it to your browser. In this case I think some additional files need to be served alongside the demo html.
You can identify which files you need to serve, and at which URIs, by using the networks tab of the ctrl+shift+i interface. In that tab you will see a list of things, these are a complete list of the requests and deliveries that the server is making. What you need to do is to request and deliver the things that the server is requesting and delivering (emulate).
This is a hacky way to go about it, but since mediapipe is a bit of a blackbox in the context of this demo, it is a way that allowed me to run the script in a couple of 10 minutes.

thanks a ton! @tomatopuree


Failed to Install VS Code Update

VSCode Version: 1.22.1
OS Version:  Windows 8.1

Steps to Reproduce:

Install the update when prompted by Visual Code application once launching Visual Code
Wait for it to finish downloading in the background
VS Code error appears describing to file an issue on Github and attach the log

Does this issue occur when all extensions are disabled?: Yes
vscode-inno-updater.log

This should be #47815


Vite Branch template changes not triggering Tailwind JIT recompile

Describe the bug
On the branch craft-vite Tailwind JIT is not recompiling the css when template changes occur in template files.
To reproduce
Steps to reproduce the behaviour:

composer create-project nystudio107/craft:dev-craft-vite vitecraft --no-install --remove-vcs
make dev
Load http://localhost:8000 in browser
Editing: src/templates/index.twig
Add a paragraph to the template (confirms that auto reload is working and detecting changes to template files)
Add a class to the paragraph eg: <p class="bg-red-900">
Auto re-load will occur again, but the background will not update.
Reload the docker containers ctrl-c then make dev
Reloading http://localhost:8000 after the Docker Project finishes restarting confirms a valid Tailwind class and that Tailwind will see and use the template file to rebuild the CSS.

Alternative option starting at step 4:

Editing: src/vue/App.vue
Add a paragraph to the vue template.
Module will successfully re-load adding the paragraph.
Add the class already added in (step 6 above) to the paragraph (background is added successfully).
Change to a new class class="bg-blue-900"
Changes will not show up until the docker project is re-loaded.

Expected behavior
A red background to appear at step 6. Or a blue background at step 5
Discussion
At first I thought this could be an issue with the Docker file system, but that doesn't follow with Vite successfully detecting template changes to cause the page re-load. And also properly reloading the Vue templates.
Then I thought it could be because ../src/templates/**/*.{twig,html} was pointing to a different source directory from ./src/vue/**/*.{vue,html}, but changing to ./src/templates/**/*.{twig,html} did not resolve the issue. Which led me to also test the Vue side of this.
Then I attempted simplifying the tailwind config:
// module exports
module.exports = {
  mode: 'jit',
  purge: [
      './src/templates/**/*.{twig,html}',
      './src/vue/**/*.{vue,html}',
    ],
  theme: {
  },
  corePlugins: {},
  plugins: [],
};

Which also didn't resolve the issue (though still correctly resolves things on initial load).
So I think it is a path problem of some kind, but I can't work out how to correct it, because basically everything appears to be running on defaults.

Some additional trouble shooting after reading through #69. This appears to be another Vite 2.6 regression, pinning the Vite version to 2.5.10 resolved the issue. However, moving the source files into the folder and removing the symlink did not provide a work around as in #69. Which led me to finally finding Vite #5408. So known issue.

yep, known issue, unfortunately. Track it here too: https://github.com/tailwindlabs/tailwindcss/issues/5877

Do you happen to know of a better work around instead of re-loading the containers?

Roll back to 2.5.x is the only solution I'm aware of right now.

Looks like they've fixed it (no release cut yet tho): https://github.com/vitejs/vite/pull/5571

I meant to make a note to that effect. I've been debating if it is time to pull the plug on a legacy build though. It is used by more than just IE, but not by a lot. And I wouldn't call my support of those browsers particularly robust beyond, "here is a bundle that might work on your machine, but also good luck with the some CSS stuff where your browser is also a bit buggy"


Amend swiftlint to work with older installations

Noticed an issue when trying to build the examples project that I couldn't because of the Swiftlint build phase (see the image above)
As it turns out in between the version I have installed (0.16.1) and the latest (0.19.0) they've made it so you can drop lint from the command (which is the default task).
I see two options really:

We add lint to the command which at least lets older versions of Swiftlint compile the project without making changes to run phases. It's worth noting that this means that certain (newer) rules won't be detected.
We add a way of detecting if they have an older version (likely by comparing the output of $(swiftlint version) to the version we know it works on. And display a more appropriate message and not fail the build.

Happy to discuss this
@heshamsalman @jessesquires @rnystrom

I don't take issue with adding lint to the command -- I think it's definitely the simplest solution.
The only issue that may arise is a discrepancy between Danger output re: linting, and local output. Not sure if it's immediately obvious to the end user that the discrepancy is bc of outdated dependencies.

@heshamsalman I think we can probably deal with the discrepancy within the PR if Danger raises something - better than someone raising an issue saying they can't build the project!

Agreed, sounds good to me! Thanks @Sherlouk !


Jobs authorization

Description
This PR implement the jobs authorization model which includes a mix of group authorization like all the other endpoints and individual job type authorization
Motivation
After collecting some use cases from multiple facility, we realized that the job authorization needed to be slightly more flexible
Changes:

jobs controller
jobs services
casl factory

Tests included/Docs Updated?

[ ] Included for each change/fix?
[ ] Passing? (Merge will not be approved unless this is checked)
[X ] Docs updated?
[ ] New packages used/requires npm install?
[ ] Toggle added for new features?

Can this be rebased without the dependabot changes? It's hard to check what's new in this PR

Type issues has been fixed by the latest PR to this branch

I have tested this PR locally and everything seems working fine for me.

Since we have agreed to move it elsewhere, I removed statusHistory from the job schema because the code was compiling and running correctly without it.
Instead, I kept configuration in the job schema and the code as is.
Just needed a couple of more fixes to resolve compilation errors and now it seems to be working fine. I was able to create and update a job via swagger.
Will create separate issues for the rest of the topics that we discussed (see comments), so that we can address them in separate PRs and merge this one.


Update carbon dependency

The version of nesbot/carbon that is currently used is not compatible with the latest Symfony version.
This PR updates the package to the latest 3.8 version.
Note: the minimum PHP version of this new package version is 8.1.
I don't think this should be a problem since 8.0 and older is not supported anymore: https://www.php.net/supported-versions.php

Hi @runelaenen,
Thanks for the PR! We will review this and get back to you as soon as possible. Your support is much appreciated!

@runelaenen Hi, we are intending to remove nesbot/carbon from the SDK entirely and instead use PHP's native DateTime. Presumably this will work just as well for you?

Hi @sta1r
Yes, that would be a great solution. Thank you!

@runelaenen nesbot/carbon is removed in v2.5.0. I'm aware the Shopware plugin still requires v2.1.0 - can you work around this for now or would an updated release over here help you out?


Hyperion cores selectively during quit

Hyperion.NG sometimes ends with a Segmentation fault and cores.
[hyperiond HYPERION] <INFO> Hyperion instance 'First LED Hardware instance' has been started
^C
[hyperiond MAIN] <INFO> Application closed with code 0
[hyperiond FLATBUFSERVER] <INFO> Stopped
Segmentation fault (core dumped)

The related code is during quiting the various Qt Treads.

...
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Core was generated by `./build-x86x64/bin/hyperiond'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x00007f53027f4e98 in QObject::thread() const () from /usr/lib/x86_64-linux-gnu/libQt5Core.so.5
[Current thread is 1 (Thread 0x7f52ff808800 (LWP 25146))]
(gdb) bt
#0  0x00007f53027f4e98 in QObject::thread() const () at /usr/lib/x86_64-linux-gnu/libQt5Core.so.5
#1  0x0000562e7e8db408 in HyperionDaemon::freeObjects() (this=0x562e7ff31100) at /home/thomas/hyperion.ng/src/hyperiond/hyperiond.cpp:166
#2  0x0000562e7e8db261 in HyperionDaemon::~HyperionDaemon() (this=0x562e7ff31100, __in_chrg=<optimised out>) at /home/thomas/hyperion.ng/src/hyperiond/hyperiond.cpp:142
#3  0x0000562e7e8db2f2 in HyperionDaemon::~HyperionDaemon() (this=0x562e7ff31100, __in_chrg=<optimised out>) at /home/thomas/hyperion.ng/src/hyperiond/hyperiond.cpp:145
#4  0x0000562e7e8ebf52 in main(int, char**) (argc=1, argv=0x7ffe00f498a8) at /home/thomas/hyperion.ng/src/hyperiond/main.cpp:379

void HyperionDaemon::freeObjects()
{
	// destroy network first as a client might want to access hyperion
	delete _jsonServer;
	_flatBufferServer->thread()->quit();
	_flatBufferServer->thread()->wait(1000);
	_protoServer->thread()->quit();
->	_protoServer->thread()->wait(1000);
	//ssdp before webserver
	_ssdp->thread()->quit();
	_ssdp->thread()->wait(1000);
	_webserver->thread()->quit();
	_webserver->thread()->wait(1000);
	_sslWebserver->thread()->quit();
	_sslWebserver->thread()->wait(1000);

Looking at the implementation of the different network related classes, it seems that the Qt Tread affinity is not considered properly (similar as in LedDeviceWrapper & LedDevice).
I will have a look after the PR654 has been merged with the master branch to ring-fence any updates.

Stand der Dinge? lassen?

@tpmodding Issue is still open and requires further investigation.


IE11: sap.ui.unified.FileUploader fails after first upload

OpenUI5 version: 1.34.7
Browser/version (+device/version): IE11 on Windows 7
Any other tested browsers/devices(OK/FAIL): Chrome: OK
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

Upload a file with a sap.m.UploadCollection with instantUpload=true
Upload a second file with a sap.m.UploadCollection with instantUpload=true

What is the expected result?
Both files are uploaded
What happens instead?
The second and following uploads doesn't work and in IE devtools console a InvalidStateError exception is thrown when this line of code is executed in FileUploader.prototype.sendFiles
oXhr.xhr.setRequestHeader("Content-Type", sContentType);
Any other information? (attach screenshot if possible)
Does this possibly relate to issue #371 ?

Hi and thanks for your message.
I have informed the responsible colleague, we'll keep you posted on the progress here.
Kind regards,
Christiane

Hi @christianekurz ,
this is a high priority issue in our application since users have to reload the entire application if they want to upload more than one document.
Is there yet any information on when this will be fixed or if any previous UI5 versions did not have this behavior?
Thank you in advance for a short update.
Frank

Sorry, closed this accidentally...

Just to be sure i openend another internal issue to track it (#1670187666). Expert @Stanley-78 for this control contacted so that he can get back to you and analyze the issue in more detail.
Kind Regards,
Michael

Please notice that I have also opened an incident directly at SAP for this issue:

Regards,
Frank

Hi,
the issue is fixed with UI5 version 1.34.10.
Change: https://github.com/SAP/openui5/commit/bb2ab42f44b37519723916604e924fdf16ced09d
Cheers
Stanley

Hello  Stanley
I am facing exactly the same problem. I am using UI5 1.32 and IE11. With Google Chrome, upload collection works fine but with IE 11, after uploading one file, it throws error InvalidStateError. Could you pls suggest how can I resolve this issue with these versions


docs: reflect ami works to docs

What does this PR do? / Related Issues / Jira
Updates the README with the followings:

fix docker image URL
golden ami
update egress list to the latest

/label tide/merge-method-squash
@mjlshen thanks! I've applied the review items, could you re-check?

/lgtm


[Discussion]: Expanding FireANTs Support to MPS and License Clarification

Hi @rohitrango,
Sorry about the spam, I am just excited 🥳. I originally wanted to post this under Discussions but I think it hasn't been enabled in the repo. Anyway - FireANTs is providing a significant speed-up due to the Torch+CUDA implementation, which is fantastic. Since PyTorch now supports MPS (Metal Performance Shaders) on macOS, I was wondering if there are any plans to extend FireANTs' support to MPS as well? This could be particularly beneficial for users on Apple Silicon.
Additionally, I noticed that the package doesn’t currently have a specified license. Would you be kind enough to clarify what license FireANTs is distributed under?
Cheers,
Lalith

No problem haha. I've enabled the Discussions section.
MPS support: this is slightly low-priority as of now because I'm still trying to get mostly power users onboarded first. But if you can work on MPS support and give some Perf metrics, that would be excellent! Nothing better than a registration algorithm that also runs fast on the newer macs!
I will update the license soon.

updated the license

Happy to do so. We just figured out MPS for one of the packages. It's pretty good. I am happy to look into this and keep you posted.

An update on MPS support: it looks like the MPS backend does not support crucial ops like F.conv3d yet

this leads to the method simply falling back to CPU. This is a bummer but we have to wait for PyTorch devs to implement these ops (or do it ourselves - and I dont have the bandwidth to do it).

Hi @rohitrango,
You are right. But we have a fork for that. I haven't forgotten about this. Conv3d and transposeconv3d is available on this fork:
git+https://github.com/LalithShiyam/pytorch-mps.git
I am in transition, so busy sorting out things. But this is definitely on my priority list!

sounds exciting! In that case, I'll put you in charge of MPS support for FireANTs (mostly just the interpolation and conv operations)
Good luck with your move!

🫡🫡


Support getting connection string from environment specific appsetting.json

support asp.net core environment specific appsetting.json

When #73 is released to nuget, then you can do this (With help from this):
<extensions>
    <add assembly="NLog.Targets.ElasticSearch"/>
    <add assembly="NLog.Appsettings.Standard" />
</extensions>

<target xsi:type="ElasticSearch" url="${appsettings:name=ConnectionStrings.MyUrlName}" />

Then there is no need to explicit load the appsettings.json in this target. But it will probably be a breaking change to remove the explicit loading of appsettings.json completely (Btw. notice that #76 disables the automatic file monitoring as it is not supported on all NetCore platforms).

With NLog.Extension.Logging ver. 1.4.0 then you can now use ${configsetting}
See also: https://github.com/NLog/NLog/wiki/ConfigSetting-Layout-Renderer


Using sort: false with multiple

Describe the bug
If you have 2 sortable areas with the multiple plugin and the first has sort: false, it still sorts the first if you drag from the first box, and back but dont select the first item
To Reproduce
Steps to reproduce the behavior:

Select item 1 and 2 in the example
Drag from item 2 into the second list
Drag back to the original list
The items are now swapped

Expected behavior
I would expect the first list to keep the old the order
Information
sortablejs = 1.15.2
Additional context
Add any other context about the problem here.
Reproduction
https://jsbin.com/nomeyixera/edit?js,output
Edit: Fun thing is that is seems to work on second drag. If I deselect everything and try again, it works as expected..

not sure if I am going crazy but using your jsbin example I couldn't reproduce the issue. Can you confirm what browser?

I couldn't reproduce either.

Funny thing is, when I started swapping really fast, I end up breaking it every time to where you can move the selected items anymore. You can select other items and move them, but the items that were selected when it originally breaks are immovable.

Hm weird. I am certain I checked it multiple times and was able to recreate it. I will close this for now and reopen if I find out how I was able to break it


ROX-17895: Don't write the inner channels if the component is stopped

Description
This is a known issue when we stop sensor. There are some race conditions when we stop some of the components and consequently a component that is still running might try to write in a channel that was already closed in another component. We have plans to organize the start up and stopping of components to address this but in the mean time we make sure to not write in the inner channels in the outputQueue and the Resolver components.
Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed

[x] CI

/test gke-sensor-integration-tests

/test gke-sensor-integration-tests


No subtitles matching the filter were found. Download failed.

i dont know why its not working for me...
any help?
the error
https://i.imgur.com/x5T1nGl.png
my settings
https://i.imgur.com/3MZNh0x.png

Fixed on v1.0.4


HIVE-24470 - Separate HiveMetastore Thrift and Driver logic

What changes were proposed in this pull request?

Refactor HiveMetastore.HMSHandler into its own class

Why are the changes needed?
This will pave the way for cleaner changes since now we don't have the driver class nested with 10,000 line HMSHandler file so there is a clearer separation of duties.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing unit tests, building/running manually
Not additional tests were added since this was a pure refactoring

@nrg4878 and others, PTAL, thanks.

@miklosgergely can you run the tests again?

retest

recheck

@mwalenia please push your changes again to your remote branch, with a new commit having the same content.

@mwalenia Go ahead and just close the PR manually for 30s and then re-open.  That should trigger the tests again.

Also, please create a feature branch (HIVE-24470) on your local repository and PR from there.
git checkout -b HIVE-24470

since I'm not the PR creator, we'll need to wait with this for @Noremac201. Thanks for the tips!

Superseded by #1777


Update README

Update README to point to docker image as a default source of ALIGN.

@kkunal1408 Missed editing the README files in the previous PR. This PR adds the docker links to the README.


Mark Widget::destroy as unsafe

Part of #957.
cc @sdroege @EPashkin

@GuillaumeGomez Thanks.
@sdroege How we implement closing application now?
Just changing to unsafe{window.destroy();}?
   Compiling gtk-rs-examples v0.0.1 (C:\projects\gtk\_examples)
error[E0133]: call to unsafe function is unsafe and requires unsafe function or block
  --> src\bin\menu_bar_system.rs:86:9
   |
86 |         window.destroy();
   |         ^^^^^^^^^^^^^^^^ call to unsafe function
   |
   = note: consult the function's documentation for information on how to avoid undefined behavior
error: aborting due to previous error

@sdroege How we implement closing application now?
Just changing to unsafe{window.destroy();}?

Just window.close() :)
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.

Looks good to me apart from https://github.com/gtk-rs/gtk/pull/958#discussion_r381716230
And we should first update the examples so that everything continues to compile. Every destroy() call should be replaced with close().

Updated and waiting for https://github.com/gtk-rs/examples/pull/296 now.


Link Docs > Installing to Docker

That's it!

I've reviewed these changes, and double checked the link.  Clicking "Merge pull request".


fix: mispellings

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

v1.9.1
v1.9.1
GitHub release

Your semantic-release bot :package::rocket:


Create new automated tests for v1.0.1

Create new e2e tests for feature in v1.0.1. We will expand this with the feature list as well.

Not relevant now.


How to use whole new tree inside context menu

Support Requests should not be opened as issues and should be handled in the following ways:
On StackOverflow using the angular-tree-component tag
In our slack support channel at https://angular-tree-component.herokuapp.com/

The link gives the following message
Application error
An error occurred in the application and your page could not be served. If you are the application owner, check your logs for details. You can do this from the Heroku CLI with the command


fix data-type mismatch for mlperf_inference accuracy test

fix data-type mismatch for mlperf_inference accuracy test

https://github.com/huggingface/optimum-habana/pull/1146


Problem with integration full calendar plugin

Hi team, I am having problem integrating with the fullcalendar plugin
This is the code of my component.ts
import { element } from 'protractor';
import { Component, Input, OnInit, AfterViewInit, ElementRef, ViewChild } from '@angular/core';

import * as $ from 'jquery';
import 'fullcalendar';
import { Options } from 'fullcalendar';
import { ModalComponent } from 'ng2-bs3-modal/ng2-bs3-modal';

@Component({
  selector: 'app-calendario',
  templateUrl: './calendario.component.html',
  styleUrls: ['./calendario.component.css']
})
export class CalendarioComponent implements OnInit, AfterViewInit {

  @ViewChild('modal') modal: ModalComponent;

  @Input() options: Options = {
    locale: 'pt-br',
    header: {
      left: 'prev,next today',
      center: 'title',
      right: 'month,agendaWeek,agendaDay'
    },
    aspectRatio: 2,
    events: [
      {
        title: 'Alisson',
        start: '2017-06-27 10:00',
        end: '2017-06-27 12:00',
        className: 'bg01 b01'
      }
    ],
    buttonText: {
      today: 'Hoje',
      month: 'Mês',
      week: 'Semana',
      day: 'Dia',
    },
    dayClick: function(date, event, modoVisualizacao){
      console.log('clicou no dia');
    },
    eventClick: function(event) {
      console.log('clicou no evento');
      this.modal.open();
    },
    allDayText: 'Dia Todo',
    timeFormat: 'H',
    slotLabelFormat: 'HH:mm',
  };

  text: string;
  calendarInitiated: boolean;

  constructor(private element: ElementRef) { }

  ngOnInit() {

  }

  ngAfterViewInit() {
    $('app-calendario').fullCalendar(this.options);
  }

  open() {
    this.modal.open();
  }

  updateEvent(event) {
    return $(this.element.nativeElement).fullCalendar('updateEvent', event);
  }

  clientEvents(idOrFilter) {
    return $(this.element.nativeElement).fullCalendar('clientEvents', idOrFilter);
  }

}

If I call the ng2bs3 on component.html like this

<button (click)="modal.open()">Teste

works fine, but I need call the modal.open() on "eventClick" and "dayClick", this error happens today:

ERROR TypeError: Cannot read property 'open' of undefined

I thinks its because something related to life circle hooks, right?
What I am doing wrong?

@alissonschneiderr what does your template look like?

Oh, it may be because this is not scoped to your component. Consider using arrow functions for eventClick and dayClick or bind to this.

Hi, my component.html has this

<button (click)="modal.open()">Teste</button>

<modal #modal>
  <modal-header>

I'm a modal!

  </modal-header>
  <modal-body>
      Hello World!
  </modal-body>
  <modal-footer>
      <button type="button" class="btn btn-default" data-dismiss="modal" (click)="modal.dismiss()">Cancel</button>
      <button type="button" class="btn btn-primary" (click)="modal.close()">Ok</button>
  </modal-footer>
</modal>

Can you give me an example of what you said?
I'm new at angular

@alissonschneiderr - sure:
  @Input() options: Options = {
    ...
    dayClick: (date, event, modoVisualizacao) => {
      console.log('clicou no dia');
    },
    eventClick: (event) => {
      console.log('clicou no evento');
      this.modal.open();
    },
    ...
  };

oooh man, what is the difference between arrow function and normal function?
I thought it was just the syntax, so I didn't understand.
Sorry.

@alissonschneiderr no there's a little more too it than just syntax. See https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Functions/Arrow_functions.
Functions in javascript have their own this bound to them. Arrow functions do not bind to their own this, so when this is used inside an arrow function it remains intact. To mimic this behavior in a regular javascript function you would need to bind the function's this to the this in the outer scope.
So using your same example:
  @Input() options: Options = {
    ...
    dayClick: function (date, event, modoVisualizacao) {
      console.log('clicou no dia');
    }.bind(this),
    eventClick: function (event) {
      console.log('clicou no evento');
      this.modal.open();
    }.bind(this),
    ...
  };


Eloquent ORM: using substr in hasOne

I have two tables user and pno
user:
ticket_no as (tno.file_no) eg 499385
and pno:
tno eg. 99385
file_no, eg. 4

every user has one pno.
How can i use?
return $this->hasOne('Userpno','tno', substr('ticket_no',1);
to relate these two table in one to one.
Please help.
Thanks

Please direct questions to the fourms.


Draw card event occurs on next players turn

Currently, the drawRandomCard function does not execute, until a player's turn has already ended. This results in modal display issues with the online version, effectively not showing cards being drawn.
This event should occur after a player moves, but prior to their turn ending.

Should be addressed, will require testing.
Commit: https://github.com/arevi/Kill-Doctor-Lucky-Web-Port/commit/84958b3b3e63371d6ee9bbdb9c6624a1a50140fd


Deduplication is broken in 1.1.2

I'm on swr-react-native 1.1.2 and swr 1.1.2. It looks like request deduplication is broken-- when I use useSWRNative and switch between react navigation screens, requests don't seem to get deduped.  For example, a request to one url happens >30 times within a very brief period (just one render) when I switch focus or go to a new screen.
This doesn't happen when I call useSWR. Of course there is no revalidation on focus change, but even when new screens get mounted useSWR dedups the requests while useSWRNative doesn't.

For example, a request to one url happens >30 times within a very brief period (just one render) when I switch focus or go to a new screen.

Well that’s not good. I think I can guess for why this is happening. I assume that they are deduped relative to a single hook, but not across multiple hooks, since the deduping is done inside of the hook itself.
As a test case, if you just call the hook once on a screen, I would assume it dedupes based on interval, but not based on concurrent requests?
I think the reason is that we’re triggering revalidations by calling mutate(), and SWR may not dedupe on its own when you call this manually. I wonder if their latest version allows you to dedupe when you call mutate manually?

i’m looking at this: https://twitter.com/shuding_/status/1476608107623108611?s=21
I would want to know if there’s a way to do mutate(undefined, { dedupe: true })

i asked about this here https://twitter.com/fernandotherojo/status/1481761936459055105?s=21

Ah yep, looking at the code this appears to be the issue. It'd be great if their revalidate flag was more of an option, like always|never|dedup. Hopefully they add that 🤞

hmm something feels off: https://twitter.com/shuding_/status/1481772476078215170?s=21

see my comment here: https://twitter.com/spakhm/status/1481799938342375426. I think their test covers a subtly different case where you pass a value to mutate.

Ah yep, looking at the code this appears to be the issue. It'd be great if their revalidate flag was more of an option, like always|never|dedup. Hopefully they add that 🤞

I can’t quickly think of an example of not deduping, but feel free to open a feature request in vercel/swr so that can be properly discussed! :)

see my comment here: https://twitter.com/spakhm/status/1481799938342375426. I think their test covers a subtly different case where you pass a value to mutate.

I’ll open an issue for this and take a look, thanks a lot!

thank you @shuding!

closing this in favor of https://github.com/vercel/swr/issues/1786 on SWR's repo.


Compatibility with Froggit DP1500?

Any Info if it is compatible with https://www.froggit.de/product_info.php?info=p410_dp1500-wi-fi-wetterserver-usb-dongle.html
As Froggit DP1500 supports soil moisture through WS View I'm curious if someone tried this yet.

I have not tried it as I do not own this type of station. But on this thread it is mentioned that the DP1500 works with a shell skript using the same technology like my adapter. So it should basically work.
If you add additional sensors like soil moisture I would most likely need to add some config to read/store the data coming from that sensor. But that is a simple exercise once you have it. I just need the debug log to see what the station is delivering.

...good to know that the output looks the same like from the Stations. I ordered sensors and receiver and will provide you with data for integration asap! Thanks for your work!

Got the Froggit DP1500 today and did a debug log for you. There can be a total of 8 Soil Moisture Sensors (called DP1000) in the system, according to the docs. Would be very nice if you add them to your adapter.
sainlogic-debug-GW1000A.txt

The interessting part ist this one:
2021-03-22 17:01:16.047 - debug: sainlogic.1 (10353) Listener body is PASSKEY=5166D3F509F4FD30F5EF49B0BF29D5EE&stationtype=GW1000A_V1.6.6&dateutc=2021-03-22+17:01:16&tempinf=71.2&humidityin=49&baromrelin=29.935&baromabsin=29.935&tempf=66.0&humidity=56&winddir=229&windspeedmph=4.70&windgustmph=9.17&maxdailygust=13.65&solarradiation=132.24&uv=1&rainratein=0.000&eventrainin=0.000&hourlyrainin=0.000&dailyrainin=0.000&weeklyrainin=0.000&monthlyrainin=0.000&yearlyrainin=0.000&totalrainin=0.000&soilmoisture1=35&soilmoisture2=31&wh65batt=0&soilbatt1=1.6&soilbatt2=1.5&freq=868M&model=GW1000_Pro
In your case the parameters for soil are these:
soilmoisture1=35
soilmoisture2=31
Do you know what unit these are? I can easily add them to the config.

Version 0.7.0 should support now these devices. Check it out and let me know.

Thanks! Will try when I'm home next week

I tested GW1000 and WH51 (Same from Ecowitt) today, and it works great.

Version 0.7.0 should support now these devices. Check it out and let me know.

Back from an injury I tested and it worked for my 5 sensors. Only caveat is the battery voltage is only reported for the first sensor. Will provide log later!

Awesome, i take the GW100 and WH51 into the supported devices list...

can confirm its pulling soilbatt aka Batterylevel for all sensors now!


Forwared endpoint to /owners/{address}/safes

Closes #529
Cache both response and request to backend service for 1 minute as agreed

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 5 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 83.383%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/models/commons.rs
0
2
0.0%

src/config/mod.rs
0
3
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
6749

Relevant Lines:
8094

💛  - Coveralls


Drop Python 2 Travis CI configuration

Part of #8005

Looks good to merge?

Jenkins, test this please

just testing it as customary, shouldnt be any issues at all

Jenkins CI test (for commit bc399ae5fd47db93d04c016915442fd86311d00f, target branch master) failed with status FAILURE.

CI Unrelated


assignment expression

Hi Tom,
Thank you for the great library !!!
I have an issue. Is there a way to achieve below scenario with the current library ?
const context = {
age: 36
}
const res = await jexl.eval('age = 32', context)
Thanks,
Wasim

Thanks for your kind words @wasimshariff !
Jexl does not currently support assignment within expressions. Jexl expressions are more focused on creating a result value, rather than offering a Turing-complete programming capability. With that said, I'm always interested to hear others' use cases and will be happy to reconsider!
It's worth noting, though, that if Jexl ever does support assignment operators, I can saw with confidence that it will not support changes to the context persisting after the expression completes. Jexl is built with absolute safety in mind, so it cannot allow a expression to modify anything in the parent javascript environment. That's what Javascript's native eval() does.

Thanks for the reply Tom.
We are migrating our App from JSF to Angular. In JSF world, we were using Apache JEXL library for evaluating our business rules.
We have tons of business rules that is defined by the business in a file.
Eg : app is an List in jexl context.

app[appIndex].distributionChannel eq "TELESALES"
app[appIndex].applicant.age = fn:calcAge(app[appIndex].applicant.dateOfBirth)

We use assignment pretty extensively. Let me know your thoughts if this can be implemented in the library or if its not the use case of this library.
Thanks again for such quick reply :)

That is a super cool use case. I love it.
On the surface that seems like it would require writing back to the context, which at this point I'm not sure I like the idea of... though I'll consider safe options.
However! There's another way to approach this: What if the result of your Jexl expression were an object that you could merge onto your context after every expression, perhaps with lodash? Consider this:
const _ = require('lodash')
const context = {
  applicant: {
    dateOfBirth: '1984-03-09',
    name: 'John Doe'
  },
  distributionChannel: 'TELESALES'
}
const expressions = [
  "{ distributionChannel: 'DIRECT' }",
  "{ applicant: { age: applicant.dateOfBirth|toAge } }"
]

expressions.forEach(expr => {
  const result = jexl.evalSync(expr, context)
  _.merge(context, result)
})

// The context is now:
// {
//   applicant: {
//     name: 'John Doe',
//     dateOfBirth: '1984-03-09',
//     age: 36
//   },
//   distributionChannel: 'DIRECT'
// }

What do you think?

Another thought! I'm about to cut a new release that adds top-level functions to Jexl (in addition to the transform functions it already has). If the above will work for you, you could define a function that will take two arguments: the "address" of the field to modify, and the new value. Then it would split the address on the dots and produce the object to be merged. Like this:
const result = jexl.evalSync("write('applicant.age', applicant.dateOfBirth|toAge)")
// result is now { applicant: { age: 36 } }

Same concept, but that syntax might be easier on the eyes.

Thanks Tom. One constraint we have is , i cant make lot of modifications to business rule files. They have to be backward compatible.
I will look into your suggestions.  I am closing this question for now. Thanks for the help !!!

Oo, I’m afraid that even with assignment operators in Jexl, the syntax is just too different from Apache’s Jexl that all the business logic expressions would still have to be rewritten. You might be better served by a small Java lambda function behind API Gateway (assuming AWS, but choose your provider’s equivalent) specifically to run expressions and pass back the result.

Sorry for adding to a closed issue, but thought I'd ask if you've seen the FEEL (Friendly Enough Expression Language) used by a number of BPMN/DMN products (notably Camunda) and their handling of "context" expressions, which are essentially expressions where the expression is a JSON object - much like in your example "{ applicant: { age: applicant.dateOfBirth|toAge } }".
A couple of things they allow is for items in a context expression to rely on other properties in the expression, so long as it is defined before it, for example:
{ a: 1, b: a + 1 } is valid, and would return { "a": 1, "b": 2 }.
Given each expression in a context is essentially assigning to a property, then you get the assignment capability (something you already have), but also the ability to calculate things based on the context.
Of course, the same could be achieved right now in JEXL by merging the result back into the context (as per your example). That however brings me onto another FEEL feature that might be interesting and that makes a good case for it being handled "natively" - you can define functions within the context expression, for example:
{ a: 1, b: a + 1, sum(x,y) x + y, result: sum(a, 3) } would return { "a": 1, "b": 2, "result": 5}
Functions can only contain expressions that are valid against the grammar (i.e. they are not "javascript functions", just reusable functions for convenience).
I'm not suggesting for one moment that you'd want to be attempting to create a FEEL like grammar, but in the interest of these assignment based use cases, I thought it might be interesting to show examples of not dissimilar use cases I have seen elsewhere.


allow extended SWA config

Fixes #1.
I hate the name of ExtendStaticWebAppConfig but I can't think of anything better at the moment.
Also not 100% certain on the validation logic. I figure it's better to try stop people getting into situations that will leave their app completely broken, but I'm not sure how far down that rabbit hole to go, so I settled for checking if they try override the catch all route or if they specify navigationFallback (even though the type says you navigationFallback is not valid, since I used a destructure on the extended config, it could still technically happen, maybe there is a better way to do this).
I could see an issue with someone trying to specify auth on the catch all route that this would then reject?
The docs specify an auth property (https://docs.microsoft.com/en-us/azure/static-web-apps/configuration#authentication) as well as forwardingGateway and networking on the config, but these aren't present in the SWA CLI types: https://github.com/Azure/static-web-apps-cli/blob/f2ec90afb5a4d3d4caeb1ffe4f56776a937cfac1/src/swa.d.ts#L130 so I decided not to include them.
I figure I should probably update the readme with a quick example too.

I put a note in the docs that's basically "use at your own risk." I want to eventually play around with SWA auth + this adapter myself, so I might have some ideas then. But I think that's good enough for now until someone opens an issue.

Thanks for the PR!


Allow the user to clear a date time picker

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Please make sure you are posting an issue pertaining to the Ionic Framework. If you are having an issue with the Ionic Pro services (Ionic View, Ionic Deploy, etc.) please consult the Ionic Pro support portal (http://support.ionicjs.com)
Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
Please do not submit support requests or "How to" questions here. Instead, please use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When a user accidentally picks a date he is unable to undo/remove the date via the date picker.
Expected behavior:
A clear cross icon or something which also allows the user to remove the current set value
Steps to reproduce:
Create a date picker component and set a date
Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
cli packages:

    @ionic/cli-utils  : 1.6.0 (ionic-app/node_modules/@ionic/cli-utils)
    ionic (Ionic CLI) : 3.6.0 (ionic-app/node_modules/ionic)

global packages:

    Cordova CLI : 6.5.0

local packages:

    @ionic/app-scripts              : 2.1.4
    @ionic/cli-plugin-cordova       : 1.4.1
    @ionic/cli-plugin-ionic-angular : 1.3.2
    Cordova Platforms               : android 6.1.2 ios 4.3.1
    Ionic Framework                 : ionic-angular 3.6.0

System:

    Node       : v8.2.1
    OS         : macOS Sierra
    Xcode      : Xcode 8.3.3 Build version 8E3004b
    ios-deploy : 1.9.1
    ios-sim    : 5.1.0
    npm        : 5.4.2

Hello! Thank you for opening an issue with us! I am closing this issue because it is a duplicate of Issue #11961, which itself has been moved to the feature request list as feature request number 248.
Thank you for using Ionic


chore: upgrade swappers to structurally support multi-hop trades

Description
Upgrades swappers to support multi-hop trades structurally, without actually implementing multi-hop trades in the application. Also removes redundant sendMax and maximumCryptoHuman from swappers.
Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[x] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
NA
Risk
This is a fairly high risk change.

Risk of trades generally broken
Risk of broken max trades

Testing
This change should introduce zero change from production.

check that quotes are being created correctly including fees and receive amounts
check that trades execute correctly for all swappers
check that max trades execute correctly for all swappers

Engineering
See above
Operations
This change should introduce zero change from production.
Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4625   👈

PR #4630

This comment was auto-generated by Graphite.


feat: add link functionality to views

Views can have whatever documents have, and documents can have links. For our needs we needed views to have links too, so here we are!

Hi @omivore ,
Thank you for the PR, this feature is reasonable.
Could you please add respective tests?

@roman-right I spoke with @omivore and he gave me the go ahead to finish up the work for this PR, which I put in https://github.com/roman-right/beanie/pull/472
We can probably go ahead and close this one.


Configuration extension does not merge ban entries

Bug Report

TSLint version: 4.1.0
TypeScript version: 2.1.4
Running TSLint via: CLI

TypeScript code being linted
Foo.bar("test");

with tslint.json configuration:
{
  "extends": "./base_tslint.json",
  "rules": {
    "ban": [true, ["Foo", "baz"]]
  }
}

content of base_tslint.json:
{
  "rules": {
    "ban": [true, ["Foo", "bar"]]
  }
}

Actual behavior
Lint pass.
Expected behavior
Warn about usage of Foo.bar.

Subset of #3474. It's an interesting area!


Bug: regression in 3.1.13 causes accessKeyId is a required option

after upgrading to 3.1.13 causes this error when running vitest which has a dependency on Resource.
If I go back to 3.1.12 it works
TypeError: accessKeyId is a required option
       ❯ new AwsClient ../node_modules/.pnpm/aws4fetch@1.0.20/node_modules/aws4fetch/dist/aws4fetch.esm.mjs:31:36

Hmm that's odd. What's the resource trying to access? Which component?

@leon can u try 3.1.17 or later?

It seems to be fixed now, so I'm closing 👍


Add option to dump AST to files

This adds two things:

Calling swiftc -dump-ast foo.swift [...] -o foo.ast will dump the AST to the file foo.ast, instead of dumping it to stderr as usual.
Calling swiftc -dump-ast -output-file-map=outputFileMap.json *.swift [...], given an outputFileMap.json file that contains entries in the form "ast-dump": "foo.ast", will dump the ASTs of the input files to their respective output files in the file map.

This should serve as a valid workaround to a bug mentioned in the forums where the AST dump functionality crashes when called with too many input files. A few implementation details were also discussed in the same forum post.
As an aside, this also fixes a comment in include/swift/Basic/PrimarySpecificPaths.h that was incorrect.

@swift-ci Please smoke test

Hm.
@swift-ci Please smoke test

Uh, hm. Looks like a bunch of tests were still expecting the AST dump output on stderr.

Oh thanks, I couldn't find out why...
I think I'm gonna change the tests to expect stdout (rather than change the AST dump to stderr), what do you think?

@swift-ci Please test


Make print, eval, parser and lexer work together

We should now be able to call
print(eval (parser(lexer "(+ 137 349)"))) to get  486  in the terminal.

This has been working in testing


Add a User Profiles page 🌟🌟🌟

👤✨ I want to create a normal user profile page and a profile icon in the home page navbar that redirects to the profile page...
PA please assign me for this task...

@Akshat111111

please assign me this profile creating task @Akshat111111

please assign me this profile creating task @Akshat111111

No need, as soon as he merge my pull request #78, I will start this work...


Nested component dot path completion should be replaced with slash (/) path completion

For nested component templates, intellij-emberjs currently offers .  completion.
Example:

However, with Ember 2.10 the . syntax is no longer supported and everyone should switch to / syntax.

good point, thanks!

Upvote.
(Thanks again for this plugin)

@Ramblurr @nadnoslen this should be a relatively easy bugfix, feel free to open a pull request for this :)

I poked into the source seeing if I could add this relatively easily, but wasn't succesful. Here's what I tried. Keep in mind this is my first time around this part of the IntelliJ SDK.
The com.emberjs.hbs.HbsReferenceContributor registers two element patterns with the HbsComponentReference.

HbsPatterns.BLOCK_MUSTACHE_NAME
HbsPatterns.SIMPLE_MUSTACHE_NAME

I figured these patterns needed to be updated to support / as well as . triggers. There doesn't seem to be any explicit reference to . (DOT) as a separator, so I am assuming it is the default. The handlebars plugin itself uses slash and dot as separator types in its grammar  (ref1, ref2). So I figured these patterns needed to be updated to use HbTokenTypes.SEP, however I can't find any place to add this that works.
The obvious place was the two patterns registered with HbsComponentReference, but adding
            .afterLeaf(psiElement(HbTokenTypes.SEP)) to SIMPLE_MUSTACHE_NAME and BLOCK_MUSTACHE_NAME doesn't seem to work.
@Turbo87  any tips or pointers?

@Ramblurr actually we don't seem to be caring about the separator that much in the HBS AST at all. I guess the magic is in https://github.com/Turbo87/intellij-emberjs/blob/master/src/main/kotlin/com/emberjs/lookup/EmberLookupElementBuilder.kt#L11, but I'm not sure right now where else that class is used, so be careful about just removing that line.

I have a couple of quite large Ember applications that have a hundred or more components, some nested in directories, some not.  I've leveraged the auto-complete of my IDE using this plugin and have dug a deep hole where it concerns making my app Glimmer2-ready.
I took a little time yesterday to write a Ruby script that combs through my EmberJS application and replaces MOST (not all) dot-notation paths with slash-notation.  It may be helpful to others: https://github.com/cybertoothca/emberjs-component-dot-to-slash-paths
Cheers.

@dwickern in addition to #119, do you think it would make sense to write a quickfix that converts from dots to slashes automatically?

@Turbo87 ember-watson might be a better fit for that sort of thing

Hey Gang,
Thanks for the fix!  However, I'm now noticing that route auto-completion is not following dot-notation.  In the Ember Guide (https://guides.emberjs.com/v2.10.0/routing/defining-your-routes/#toc_nested-routes):

A nested route's names includes the names of its ancestors. If you want to transition to a route (either via transitionTo or {{#link-to}}), make sure to use the full route name (posts.new, not new).

I'm pretty sure these should stay with the dot-notation.  Any thoughts?
(PS. thanks for the note about ember-watson)

@nadnoslen looks like we missed that part 😞
could you open a separate issue for that and crosslink this issue here?


Feedback & Suggestion

Nice Work Buddy !
Suggestions : -

i think there is no need of log in & sign up .
you can also add category of to do work.
you can add  direct download link of your APP

yes, I will definitely do that thank you for your suggestions and feedback.


Insert space when text contains line break

I use markdown to create  elements like so:
Foo
  : Bar

Which outputs:

Foo

Bar

In the search result excerpt, this shows up as "FooBar".
It looks like this also occurs between text nodes in any block level element. For instance, if a paragraph ends with "Foo" and the next one starts with "Bar", the excerpt is "FooBar".
Excerpts should respect line breaks/carriage returns and insert a space between elements.

👋 Hey @seezee
Thanks for bringing this up! Pagefind should indeed handle these elements better than it is now.

Excerpts should respect line breaks/carriage returns and insert a space between elements.

There's actually no semantic line break here for Pagefind to spot — the whitespace between HTML elements doesn't carry any meaning and is lost when parsing, so it's equivalent to
Foo

Bar
.
Instead, Pagefind categorizes the elements that should break text into sentences, and makes sure to separate them with spaces and periods if required. dt and dl haven't been categorized correctly, so are being treated as inline elements.
In this case, I'll add handling so that when indexing, the dt is followed by a : if not present, and the dd is followed by .. So indexing:

Morgawr

A sea serpent

Owlman

A giant owl-like creature

Will come through as Morgawr: A sea serpent. Owlman: A giant owl-like creature.

Awesome!


darwin.adv_cmds: darwin.adv_cmds

When building adv_cmds (as a dependency of darwin.cctools) in the Github CI system, it fails:
…
6 warnings generated.
/private/tmp/nix-build-adv_cmds-osx-10.5.8.drv-0/adv_cmds-119
/private/tmp/nix-build-adv_cmds-osx-10.5.8.drv-0/adv_cmds-119/mklocale /private/tmp/nix-build-adv_cmds-osx-10.5.8.drv-0/adv_cmds-119
/private/tmp/nix-build-adv_cmds-osx-10.5.8.drv-0/adv_cmds-119
bsdmake: setrlimit: Invalid argument
builder for '/nix/store/x2zn339gffbyklr7zvqh21zpmbqqhyjf-adv_cmds-osx-10.5.8.drv' failed with exit code 2

This is the log file: https://github.com/entropia/tip-toi-reveng/runs/1371182797 (sorry for not providing a one-line reproducer, but I don’t actually have a OSX machine besides these CI runners…)
Google didn’t point to much, although others had that problem before, judging from this gist: https://gist.github.com/mitchty/6c9ac57a1e44b2ce9fb8061404fd4803
@mitchy, do you remember that issue, and could you fix it somehow?

I guess darwin.adv_cmds fails already for other reasons (reported at https://github.com/NixOS/nixpkgs/issues/82640)…

Started getting something similar to this in Home Manager CI as well:
bsdmake: setrlimit: Invalid argument
builder for '/nix/store/kmby1sxg1jjqf57frkp9kl19g4463flh-adv_cmds-119.drv' failed with exit code 2

See, e.g., https://github.com/nix-community/home-manager/runs/3843511030

BTW, is there some reason why we can't build adv_cmds on Hydra? It seems the package itself is missing the meta.license.
While the license from the package itself is unclear (there is many declarations there, one for each program), Gentoo at least seems to consider it to use BSD: https://gitweb.gentoo.org/repo/gentoo.git/tree/sys-apps/darwin-miscutils/darwin-miscutils-12.ebuild?id=56d30f30691c40e606d848ffbafac8029750bb99#n25

FWIW. It seems like this issue has resolved itself, just noticed a successful run in Home Manager: https://github.com/nix-community/home-manager/runs/3968786521

Maybe safe to close this?

I think so.

I just hit this while running a nixpkgs-review:
6 warnings generated.
/private/tmp/nix-build-adv_cmds-119.drv-0/adv_cmds-119
/private/tmp/nix-build-adv_cmds-119.drv-0/adv_cmds-119/mklocale /private/tmp/nix-build-adv_cmds-119.drv-0/adv_cmds-119
/private/tmp/nix-build-adv_cmds-119.drv-0/adv_cmds-119
bsdmake: setrlimit: Invalid argument
error: builder for '/nix/store/s3lf3ixxsr9cw7zqbimdngrax8papccp-adv_cmds-119.drv' failed with exit code 2

As discussed in https://github.com/Homebrew/legacy-homebrew/issues/14847, this should only affect systems for which the launchctl limit maxfiles setting is changed. With a regression https://github.com/NixOS/nix/issues/4046 in the never versions of Nix, more users should need to change that setting.


Django date format settings ignored

Setting Django's date format to %d.%m.%Y (or any other with possible strings) renders the % character, as in the list of the offers' restriction in the Dashboard.

The USE_L10N = True setting is being ignored, showing the time in 12-hour format. When USE_L10N is True, the locale-dictated format has higher precedence and should be applied instead.
I'm in Austria (CET, language code de-at).

It's not the browser language issue, as it's the same for an en-us and de-de versions of newest Firefox.
Technical details

Python version: 3.7.3
Django version: 2.2.18
Oscar version: 3.0

I think you are confusing two different things. Django's DATE_FORMAT setting does not accept formatting that uses the % sign - the link you provided shows all the available formatting options.
You've mixed it up with DATE_INPUT_FORMATS which uses a different format entirely.


fix(material/dialog): content directives picking up wrong ref for stacked mixed type dialogs

The dialog content directives try to resolve their closest dialog using DI, and if it fails (e.g. inside a template dialog), they fall back to resolving it through the DOM. This becomes problematic if the ng-template was declared inside another dialog component, because it'll pick up the declaration dialog, not the one that the button exists in.
These changes remove the resolution using DI and switch to only going through the DOM.
Fixes #21554.

I'll mark this as blocked for now. There should be a better way of doing it with injectors now.


Reformat success page to match URx spec

success.html is still left as it was in the template from stripe. It should be modified to match the design principles established in index.html.

Resolved.


Clear Clipboard

Possible to add Clear Clipboard to the list after activating with c:?
Have it at the bottom of the stack and/or an option to invoke it with a PowerToys Run activation command like cc:.

Clipboard history is easily clearable via the Windows history (Windows key + C) then selecting clear all but I could potentially add a shortcut here too if you still think it would be useful, something like c: -c/--clear could work?

Yeah a hands-on shortcut would help greatly. Unless text input is foreground and active, it'll pop-up on the bottom right of the screen. The pause in workflow for those who spend 75% of their time with two hands on keyboard is enough to wonder why there isn't a shortcut for it in the first place, or why it's not drawn by the cursor.
I think the solution you proposed is good. It looks like c:- is ambiguous that it'll invoke it with those few keystrokes.

Hey @able16,
I have implemented this and its available in the latest release here
Thanks for the suggestion!


URL containing two slashes returns empty response

I created the following json file.
{
    "foo": { "name": "xxx" },
    "foo/bar": { "name": "yyy" },
    "foo/bar/baz": { "name": "zzz" }
}

I started JSON Server, and got strange behavior like this.
First, http://localhost:3000/foo returns
{
  "name": "xxx"
}

Next, http://localhost:3000/foo/bar returns
{
  "name": "yyy"
}

Then, http://localhost:3000/foo/bar/baz returns
{}

I don't know why I got empty response only when a URL has two slashes. This is a bug or wrong usage?

Hi @rockwillj,
/ in db.json are not supported.
Without knowing what foo, bar and baz it's hard to say what would be the good structrue for db.json.
Bu you can try this for yourdb.json:
{
  "things": [ { "id": 1, "name": "xxx" }, { "id": 2, "name": "yyy" }, ... ]
}

with the following routes.json:
{
  "foo": "/things/1",
  "foo/bar": "/things/2",
  "foo/bar/baz": "/things/3"
}

Or alternatively you can do:
{
  "foo": {},
  "foo-bar": {},
  "foo-bar-baz": {}
}

With the following routes.json
{
  "/foo": "/foo",
  "/foo/bar": "/foo-bar",
  "/foo/bar/baz": "/foo-bar-baz"
}

See https://github.com/typicode/json-server#add-routes
If there's a relationship between foo, bar and baz, you can use the example provided here for inspiration:
https://github.com/typicode/json-server#example
as it shows basic relationships between some posts and comments.

Hi @typicode,
Thank you for the quick response.
OK, I have understand / does not supported in db.json.
Workarounds you show me above are great, they works perfectly.
Thank you from the bottom of my heart!
P.S. I know the URL structure is not good but I have to use it. I'm not a server side engineer...


I'm wondering about snippet of code you have used

I'm wondering about your reasons of using this mechanism instead of the regular typeof, instanceof .
Just I want to learn from you, I've tried to search about differences my self but I got no thing.

I got the same question. Though I've already read about your thinking about one-line module, I really did not understand why an npm module is required for this thing. IMHO, it's more simple to just put it as typeof fn === 'function' or something similar?

@RoyLING I'm not wondering about the one-line node module.
The question here why:
toString.call(x) === '[object Function]'
not:
typeof x === 'function'
nor:
x instanceof Function

ok, got your point.

This is what Lodash, Underscore and jQuery does. In modern environments typeof is good enough, but in older browsers (even Safari 8) and PhantomJS there are bugs and calling the internal [[Class]] directly Object.prototype.toString.call() works around that. I guess this is not useful in Node.js though, so I added a note https://github.com/sindresorhus/is-fn/commit/3d25982b0d57afa454eb238f749b9779735375c5.
You should not use instanceof though as it won't work if the function comes from another realm, e.g. another frame or window, in the browser.
Read more here: http://javascript.info/tutorial/type-detection

Interesting, thanks for clearing it up.

@sindresorhus Thanks, for clarifying the differences.


[Security] Bump ini from 1.3.5 to 1.3.8

Bumps ini from 1.3.5 to 1.3.8. This update includes a security fix.

Release notes
Commits

Signed-off-by: dependabot-preview[bot]<EMAIL_ADDRESS>


Add old components to new website

We want to simply copy the old GUI features (Road Measurements, Road Conditions, Altitude) to the new GUI, so that we can then start rebuilding them in MUI.

OBS: det skal ikke omskrives, komponenterne skal bare smides over fra den gamle version, så de i fremtiden kan omskrives


Errors (19 Items) undefined reference SDL...

Hello, I try to build pc_simulator project, but I get follow 19 errors, what do I wrong??:
Description	Resource	Path	Location	Type
undefined reference to SDL_CreateRenderer'	monitor.c	/pc_simulator/lv_drivers/display	line 211	C/C++ Problem undefined reference to SDL_CreateTexture'	monitor.c	/pc_simulator/lv_drivers/display	line 212	C/C++ Problem
undefined reference to SDL_CreateThread'	main.c	/pc_simulator	line 105	C/C++ Problem undefined reference to SDL_CreateThread'	monitor.c	/pc_simulator/lv_drivers/display	line 70	C/C++ Problem
undefined reference to SDL_CreateWindow'	monitor.c	/pc_simulator/lv_drivers/display	line 207	C/C++ Problem undefined reference to SDL_Delay'	main.c	/pc_simulator	line 117	C/C++ Problem
undefined reference to SDL_Delay'	monitor.c	/pc_simulator/lv_drivers/display	line 245	C/C++ Problem undefined reference to SDL_DestroyRenderer'	monitor.c	/pc_simulator/lv_drivers/display	line 249	C/C++ Problem
undefined reference to SDL_DestroyTexture'	monitor.c	/pc_simulator/lv_drivers/display	line 248	C/C++ Problem undefined reference to SDL_DestroyWindow'	monitor.c	/pc_simulator/lv_drivers/display	line 250	C/C++ Problem
undefined reference to SDL_Init'	monitor.c	/pc_simulator/lv_drivers/display	line 203	C/C++ Problem undefined reference to SDL_PollEvent'	monitor.c	/pc_simulator/lv_drivers/display	line 234	C/C++ Problem
undefined reference to SDL_Quit'	monitor.c	/pc_simulator/lv_drivers/display	line 251	C/C++ Problem undefined reference to SDL_RenderClear'	monitor.c	/pc_simulator/lv_drivers/display	line 228	C/C++ Problem
undefined reference to SDL_RenderCopy'	monitor.c	/pc_simulator/lv_drivers/display	line 229	C/C++ Problem undefined reference to SDL_RenderPresent'	monitor.c	/pc_simulator/lv_drivers/display	line 230	C/C++ Problem
undefined reference to SDL_SetEventFilter'	monitor.c	/pc_simulator/lv_drivers/display	line 205	C/C++ Problem undefined reference to SDL_UpdateTexture'	monitor.c	/pc_simulator/lv_drivers/display	line 217	C/C++ Problem
undefined reference to `SDL_UpdateTexture'	monitor.c	/pc_simulator/lv_drivers/display	line 227	C/C++ Problem

windows 10, and yes I have followed the steps, but I have not the folder "/x86_64-w64-mingw32" so I have include the folder/file in C:\MinGW\include and lib additional in C:\MinGW\mingw32\include and lib

Maybe you copied the files into not the good folder. Please have look at this: https://github.com/littlevgl/lvgl/issues/123#issuecomment-371322117
It seems somehow the directory structure of mingw is not same for everyone.

should I create the follow two folder? ->
/.../x86_64-w64-mingw32/
is "..." really a folder or does it stand for something else?

No you shouldn't.
"..." means some folder which might be different.
What if you search for "x86_64-w64-mingw32" folder? Is there any result?

What if you search for "x86_64-w64-mingw32" folder? Is there any result?

no result, I also tried to create that folder with required content "C:\MinGW\x86_64-w64-mingw32\include and lib - but without success.

If you have a 64-bit Windows you should choose 64 bit MinGW with 64 bit SDL libraries.

If you have a 64-bit Windows you should choose 64 bit MinGW with 64 bit SDL libraries.
I am a little bit confused about that, because there is no choice on the homepage, respectively there is only one download, one file (no 32 and 64bit version)....
https://sourceforge.net/projects/mingw/files/

and probably I should try another mingw like below link, but I think that is another and not same mingw as required for littlevgl pc simulator...:
http://mingw-w64.org/doku.php/download

I suggest to try the latter (http://mingw-w64.org/doku.php/download)
LittlevGL doesn't depend too much on the compiler (I mean it doesn't use compiler specific features) so it should work with the other MinGW too.
If you will confirm that the MinGW from the link above works well I will suggest it in on LittlevGL's website!

I suggest to try the latter (http://mingw-w64.org/doku.php/download)

I did it now (install x86 architectur) and now I can find the mentioned folder as follow:
C:\MiniGW\mingw64\x86_64-w64-mingw32\
but at the moment I have another problem, respectively two errors with gcc and g++? I think I must setup something in the environment variables... but I don't know how??
Description	Resource	Path	Location	Type
Program "g++" not found in PATH	pc_simulator		Project Properties, C++ Preprocessor Include.../Providers, [CDT Cross GCC Built-in Compiler Settings] options	C/C++ Scanner Discovery Problem
Program "gcc" not found in PATH	pc_simulator		Project Properties, C++ Preprocessor Include.../Providers, [CDT Cross GCC Built-in Compiler Settings] options	C/C++ Scanner Discovery Problem

Have you tried to restart your PC?

If the restart doesn't work please try this: https://stackoverflow.com/questions/11703432/eclipse-c-program-g-not-found-in-path
And be sure the path to "gcc" is really in your PATH variable.

ok. I don't have the two errors (gcc, g++)  now.
I have add under windows in - system - advanced system setting - environment variable  => in "Path" follow path: C:\MiniGW\mingw64\bin, additional the same in eclipse.
In toolchain I choose MinGW GCC.
Than I delete the project and import it new.
So now, I have 7 warnings + 5 Infos.
In the console I still see an error as follow:
..........collect2.exe: error: ld returned 1 exit status
anyway, I still get the info "unable to launch" if I want run the project.
Is there something more I should do?
thankyou

Super! Well done! :)
Just to be sure: finally you used this link, right? http://mingw-w64.org/doku.php/download

Just to be sure: finally you used this link, right? http://mingw-w64.org/doku.php/download

YES. Thanks

I updated https://littlevgl.com/pc-simulator
Thank you!


[Docenhancement][BotSDKmethods][3846]

Added table for Bot framework SDK methods. Please review.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 706490b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@v-rodriguezd, please resolve conflicts.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit 0f16c8f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@v-rodriguezd, please resolve conflicts.

Done. Thanks.

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
96
6
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit e66a89a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
:bulb:Suggestion
View
Details

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md

Line 63, Column 130: [Suggestion-disallowed-html-tag] HTML tag 'title' isn't allowed. Disallowed HTML poses a security risk and must be replaced with approved Docs Markdown syntax.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We do not currently enforce a minimum score for content merged to the default branch. This will change in the future.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
97
4
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Docs Build status updates of commit c83b33e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/create-apps-for-teams-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


keyword white list support

Sometimes the configuration is needed for the negation of the keyword like a white list.
Old Style
// Assume that blog address would be "https://blog.example.com"

.pipe(anchorRewriter({
    keyword: "twitter.com",
    rel: "nofollow",
    target: "_new"
}))
.pipe(anchorRewriter({
    keyword: "instagram.com",
    rel: "nofollow",
    target: "_new"
}))
.pipe(anchorRewriter({
    keyword: "medium.com",
    rel: "nofollow",
    target: "_new"
}))
.pipe(anchorRewriter({
    keyword: "github.io",
    rel: "nofollow",
    target: "_new"
}))

With Suggested Feature Style
.pipe(anchorRewriter({
    keyword: "https://blog.example.com",
    rel: "nofollow",
    target: "_new",
    whiteList: true
}))

The white list may support multiple item such as
.pipe(anchorRewriter({
    keyword: ["https://blog.example.com","https://image.example.com"],
    rel: "nofollow",
    target: "_new",
    whiteList: true
}))
```


Add menu item to report a bug

An item for reporting bugs should be added to the 'Help' menu.

Task completed in c6759b8 - closing.


License

Do you have a license for this code?  BSD?  LGPL?  Other?
May I request that you add one?
thanks

@WendellDuncan Hi! Sorry, it took some time for me to reply, but I added the MIT license now.


Use the secret.Query in the AuthToken reconciler

Description
DAQ-1456
Not much has changed, as this secret is rather unique in the codebase, so only small adjustments made sense,
How can this be tested?
no big noticeable (biggest one would be that the Auth-token secret now has an ownerReference)

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 91.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 64.42%. Comparing base (fc298fb) to head (16ec222).
Report is 2 commits behind head on main.

Files with missing lines
Patch %
Lines

...nakube/activegate/internal/authtoken/reconciler.go
84.61%
0 Missing and 2 partials :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3941   +/-   ##
=======================================
  Coverage   64.42%   64.42%
=======================================
  Files         381      381
  Lines       20616    20620    +4
=======================================
+ Hits        13281    13285    +4
  Misses       6044     6044
  Partials     1291     1291

Flag
Coverage Δ

unittests
64.42% <91.66%> (+<0.01%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Wildcards strip the first character from req.url

Describe the bug
Wildcards (*) don't correctly populate the URL on /-prefixed paths: they strip the first character.
To Reproduce
import { App } from '@tinyhttp/app'

const app = new App()

app.get('/foo/*',(req,res) => {
  console.log(req.url);
  // on GET /foo/bar
  // should show @/bar
  //   will show @/ar
  res.send(`@${req.url}`);
});

app.listen(3001);

Expected behavior
Should show @/bar
Versions

node: 16
@tinyhttp/app: 2.2.0

Thanks for reporting this issue. The bug lies due to the way the req.url is property is set when a * is present
https://github.com/tinyhttp/tinyhttp/blob/ed51d60c760a34e0b944603e6b81f6295b129fd3/packages/app/src/app.ts#L368
A PR is welcome

I'll look at making a PR but I'm not sure about 2 things:

When the path is /foo/:pat and the url is /foo/bar is it expected that the url becomes /foo (I would have expected the URL to remain unchanged)
When the path is /foo/* and the url is /foo/bar should the url become /foo or /foo/bar?

When the path is /foo/:pat and the url is /foo/bar is it expected that req.url becomes / (I would have expected the URL to remain unchanged)?

As per express, req.url is /foo/bar when the path is /foo/:param

When the path is /foo/* and the url is /foo/bar should req.url become /foo or /foo/bar or /bar?

Since the aim is to offer similar APIs to express, req.url should output /foo/bar.

Here's the express code
import express from 'express'

const app = express()

app.get('/foo/:path',(req,res) => {
  console.log(req.url, req.params);
  res.send(`@${req.url}`);
});

app.get('/hello/*',(req,res) => {
  console.log(req.url, req.params);
  res.send(`@${req.url}`);
});

app.listen(3000);

Thanks for reporting - seems like a very simple bugfix - PR is welcome as said above

@aarontravass What you descibe as express's spec is not what I observe with tinyhttp:
// shows @/ on /x/bar
app.get('/x/:y',(req,res) => {
  res.send(`@${req.url}`);
});

This is probably a different issue from the one I mentioned above, which I'm reproducing here:
// shows @/ar on /foo/bar
app.get('/foo/*',(req,res) => {
  res.send(`@${req.url}`);
});

Finally I'm seeing a similar but maybe unrelated behavior:
// shows @/bar @/ar @/ on /z/bar
app.use('/z',router,router,router,(req,res) => {
  res.send(`@${urls.join(' @')}`);
  urls = [];
});

I'm guessing all of the above needs to be fixed but I'm waiting to get your input on that.

Looks like the code for req.url when a param is present is broken as well. The buggy line is
https://github.com/tinyhttp/tinyhttp/blob/ed51d60c760a34e0b944603e6b81f6295b129fd3/packages/app/src/app.ts#L365

Finally I'm seeing a similar but maybe unrelated behavior:

This issue is cause by modifying the original request object. Essentially, Line 368, as mentioned by your original post, keeps stripping the first character and it does so for all middle wares. Good catch!

I don't understand how handle, as it is today, differentiates between a route (app.get, etc) and a middleware (app.use).

There is not much different between a route and a middleware. In fact, if you look at packages/router/src/index.ts, you'll find that routes are pretty much similar to middlware
https://github.com/tinyhttp/tinyhttp/blob/585633f025434bea3a12d9dbfdf07fb59712c8a8/packages/router/src/index.ts#L202
The only difference is that routes are 'mounted' on the app itself and hence, do not need an extra path variable (also called fullPath in the code) while middlwares do. This is due to the fact that you can mount middlewares such as subapps onto a specific path.

Replacing the current logic with the logic I describe above is quite a big change so I'd like to check with you that it makes sense:

Yes, your idea is correct.

Replacing the current logic with the logic I describe above is quite a big change so I'd like to check with you that it makes sense:

I believe the fix should be pretty simple since all you need to do is differentiate correctly between a route and a middlware.


Update request: openmpi 4.1.6 → 5.0.1

Package name: openmpi
Latest released version: 5.0.1
Current version on the unstable channel: 4.1.6
Current version on the stable/release channel: 4.1.6
[X] Checked the nixpkgs pull requests

Notify maintainers
@markuskowa
Changelog: https://docs.open-mpi.org/en/v5.0.x/news/news-v5.0.x.html

Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

This is work in progress. I have been working on it for a while now.
However, it was not ready since it broke mpi4py and thus all Python applications that depend on openmpi.
I will open a PR soon.

See https://github.com/NixOS/nixpkgs/pull/275796

This package is on the latest 5.0.5 version.


[VectorExt] Add indices and incoming mask to vector.predicate

This patch extends the vector.predicate op to:

take in a list of indices to map each dimension of the
vector mask to a vector dimension of the iteration space.
take in an incoming mask that is combined with the predicate mask
to generate the "true" mask (need a better name) influencing the region.
generate the "true" mask as a region argument.

Kind ping to the reviewers :)

Thanks, Tobias!
Good point! The problem is that for some cases the "incoming" mask could come from "the inside". For example, an if operation may produce a mask that is used as "incoming" mask outside the if operation. IIRC, this is called BlockInMask in LLVM, where "In" stands for incoming or input (?). We can definitely revisit the name when/if we move this to core. More suggestions are welcome, though!

Thanks for the explanation. If there is a LLVM counterpart it definitely makes sense to stick to it.


Method locale_to_sap1 does not exist on SAP_ABA SAPKA74003

abapGit was already installed as single file, today I tried uploading the new version and activation failed because of this method.

thanks, added 11 days ago,
https://github.com/abapGit/abapGit/blame/main/src/objects/aff/zcl_abapgit_json_handler.clas.abap

@AndreaBorgia-Abo can you help testing #5727 ?


Like Media by Location giving error

Traceback (most recent call last):
  File "like_medias_by_location.py", line 89, in <module>
    print(u'{0}. {1}'.format(n, location['title']))
  File "C:\Users\miranthaj\AppData\Roaming\Python\Python35\site-packages\colorama\ansitowin32.py", line 40, in write
    self.__convertor.write(text)
  File "C:\Users\miranthaj\AppData\Roaming\Python\Python35\site-packages\colorama\ansitowin32.py", line 141, in write
    self.write_and_convert(text)
  File "C:\Users\miranthaj\AppData\Roaming\Python\Python35\site-packages\colorama\ansitowin32.py", line 169, in write_and_convert
    self.write_plain_text(text, cursor, len(text))
  File "C:\Users\miranthaj\AppData\Roaming\Python\Python35\site-packages\colorama\ansitowin32.py", line 174, in write_plain_text
    self.wrapped.write(text[start:end])
  File "C:\Users\miranthaj\AppData\Local\Programs\Python\Python35\lib\encodings\cp437.py", line 19, in encode
    return codecs.charmap_encode(input,self.errors,encoding_map)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 4-12: character maps to <undefined>

Describeing issue
When I run like like_medias_by_location.py the above error pops up. Seems like something to do with encoding as I understand. Would appreciate help to clarify this.
Thank you.

Ok I figured out the problem. Apparently this has something to do with windows.
execute this before running the script
chcp 65001
It worked! Thank you for this great project! Keep up the good work!

how to run that script
chcp 65001