meshgrid and open grid functionality?

First of all - this library is a dream come true for a grad student. I've been using Taichi for doing things on the GPU up to this point (which works well for a lot of things - it's a fantastic library) but having vectorized/tensor computations optimized for apple hardware is going to be a game-changer.
Now, for my question: I see that the mlx.core.array constructor can take a numpy.ndarray, which allows us to do lots of things in numpy first and then convert over to mlx arrays. That said - especially for huge grids with fine granularity - it would be nice to have meshgrid, ogrid, etc. natively implemented. Are there currently plans to implement this (and other) array/tensor conveniences from the numpy and torch apis?
Thanks again!

We definitely are working towards including more things in MLX natively - especially common numpy ops. Things like linspace, meshgrid etc.  We currently have our hands pretty full, but I will include this as enhancement and hopefully we can get to it soon. Also, PRs for any of those are more than welcome!


Ability to update configMap through initContainers

We need the ability for the Tezos client to dynamically generate keys which can then be used in the configMap and throughout the rest of the k8s. Once the keys are generated, we need to be able to update the configMap so that the Tezos node can start with the right config.json and parameters.json for both genesis and baker keys.

Currently using kubectl patch cmd to update the config map once the baker and genesis keys are created by an initContainer.

Closing due to https://github.com/tqtezos/tezos-k8s/pull/29


How to uninstall in Linux?

Hi there. Again.
I successfully installed the program on Linux but I want to check replicability of my installation process on Ubuntu 14.04. In order to do that I need to uninstall all Atom files.  However, I don't know how to wipe them all out.
Any suggestions?

I'm going to close this.  In the future, these types of issues might be better suited for Discuss.


Calling useId() in template throws an error

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.10.1
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-sauxej?file=app.vue
<template>
  <div>
    {{ useId() }}
  </div>
</template>

Describe the bug
When calling useId() directly inside a template, Nuxt throws an error when rendering the page: [nuxt] [useId] key must be a string.
Additional context
No response
Logs
No response

useId should not be called in template but in the setup() function.

Oh sh*t, that's embarrassing. Indeed, my bad. 😅
That said, the fact that useId is so small and pretty much what we could expect from a regular getRandomId method makes me think that this situation will happen again in the future.
So, may I still suggest improving the error message? As of today, the error message makes it look like a bug in the function itself rather than a misuse from the consumer. What if we had something along those lines instead?
[nuxt] [useId] must be used in setup functions

That said, thanks for the prompt answer, and sorry for the noise. 🙇

Excellent point.


linux input

load_input: Unknown keycode name 'evdev_aliases(qwerty)'!
linux system

What is your system?32 or 64?

virtual machine?

I get this error as well
$ uname -a
Linux transwhale 4.8.0-26-generic #28-Ubuntu SMP Tue Oct 18 14:39:52 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux

$ lsb_release  -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.10
Release:	16.10
Codename:	yakkety

Thk You,I will test it.

Maybe it is a serious bug,I will fix it.

I fix it,but have a IO error.Please wait.

archlinux 64bit

This bug is fixed, please update.


Add Report Vulnerability info

Ⅰ. Describe what this PR does
Add Report Vulnerability info
Ⅱ. Does this pull request fix one issue?

fixes #2746
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
NA.
Ⅳ. Describe how to verify it
NA.
Ⅴ. Special notes for reviews

/lgtm
/approve


Sort files lexicographically

Thank you to @HerrRobo for the code 🙇
This looks like it will work perfectly. I wanted to create the pull request to test it out 🤝
If it works I'll merge it 🙇
Closes #463

Works beautifully 🎉
Thank you @HerrRobo for the implementation 🤝

Turns out that natural-orderby is slow-ish (with 10,000 videos it takes as long as a second), so I made the code added in this PR an additional sort option, rather than replacing the current sort option #564 with this commit: https://github.com/whyboris/Video-Hub-App/pull/564/commits/375cb6805b6630360d00282824be70f7d70d332e
Thank you @HerrRobo again for the work 😁


feat(TextBox): export TextContent TS definition

Description
This exports the TextContent TypeScript definition from the TextBox component so that consuming applications have access to it.

References
LUI-976 (The component in this ticket currently just copies the definition. This change will allow importing it from here).

Testing
n/a

Automation
n/a

Checklist

[x] all commented code has been removed
[x] any new console issues have been resolved
[x] code linter and formatter has been run
[x] test coverage meets repo requirements
[x] PR name matches the expected semantic-commit syntax

Test Execution Passed.

:tada: This PR is included in version @lightningjs/ui-components-v2.14.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix failing build

Travis CI is not able to load .envfile thus throwing an error and causing the build to fail.
...
/home/travis/virtualenv/python2.7.9/lib/python2.7/site-packages/dotenv/main.py:14: UserWarning: Not loading /home/travis/build/SolubleCode/tubuy/.env - it doesn't exist.
  warnings.warn("Not loading %s - it doesn't exist." % dotenv_path)
...

Build status on 0f0a5ceb58811d74b4a2b1039aefb7688e5c5bf5 is green, haven't encountered error about missing .env.


Inbox cell action icons consistency

The inbox cell has 4 action icons/items. 3 of them (Check/read, speaker/Notification and 3 dots/more) use a filled-in design, 1 of them (Message bubble) uses an outlined design.

I guess the check and more icons can more or less be viewed as outlined as well so really its just the message bubble and the speaker.

Unless there’s strong opposition and some examples of how it should look, I think we should leave them as is. They are using GitHubs official icons. I want to keep the UI somewhat familiar.
Sent with GitHawk

@rnystrom That makes sense. Im not opposed, didn't realize they were all official icons.


Plugin not found my account after youtube granted access

Hi,
I granted access to plugin (youtube say: 'now can close tab') and plugin not recognize me.
Can i pass access_token manually?
I use win7, chrome(50), aimp4.
Thanks.

Hi, It looks like it is the problem with the new AIMP version. It works in v3.60 and in v4.10 alpha (available only for developers). I already notified the AIMP author and we'll have to wait for his response


Good Golf

Your checklist for this pull request
Author name: Alakh Malik
 The Golfer
This game is about a golf game where you have to put the ball in the hole and dodge obstacles
**What is your game about?**This game is about a golf game where you have to put the ball in the hole and dodge obstacle
How do you play your game? you use WASD to Move and the main goal is to pass as many levels you can before the timer

Thanks for your PR!

This was an accident, I am extremely sorry. This was meant to be a side project and is almost a direct copy of another game in the gallery, my actual submission can be found here https://github.com/hackclub/sprig/pull/2088
Once again, I apologise for wasting your time

This was an accident, I am extremely sorry. This was meant to be a side project and is almost a direct copy of another game in the gallery, my actual submission can be found here #2088 Once again, I apologise for wasting your time

Closing this PR as a result of this comment, as it seems this PR was opened in error.


Github can not add connection

Describe the bug
Github can not add connection
Screenshots

please pull the newest version, just checked, it's alright

It's normal now. May it be caused by cache?


bind mounts not working without --privileged flag

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
When using podman on Fedora 31 as a replacement for docker, I've had issues getting bind mounts to continue working correctly.
Unless I pass the --privileged flag, I seem to also get a permission denied error when trying to access any of the files mounted inside the container.
All files are owned by the user that is running the container, and I've tried matching the user/group ids of the user inside the container to the outer host as well.
Steps to reproduce the issue:

Clone https://github.com/mnahkies/podman-bind-mount-bug

Run ./outter-script.sh

Observe that the inner-script.sh that was run within the container was not able to access the file that was mounted into the container.

Describe the results you received:
cat: can't open '/opt/bug/data/my-data.txt': Permission denied
Describe the results you expected:
if you see this, then it worked!
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
podman version 1.7.0

Output of podman info --debug:
debug:
  compiler: gc
  git commit: ""
  go version: go1.13.5
  podman version: 1.7.0
host:
  BuildahVersion: 1.12.0
  CgroupVersion: v2
  Conmon:
    package: conmon-2.0.10-2.fc31.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.10, commit: 6b526d9888abb86b9e7de7dfdeec0da98ad32ee0'
  Distribution:
    distribution: fedora
    version: "31"
  IDMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  MemFree:<PHONE_NUMBER>
  MemTotal:<PHONE_NUMBER>4
  OCIRuntime:
    name: crun
    package: crun-0.10.6-1.fc31.x86_64
    path: /usr/bin/crun
    version: |-
      crun version 0.10.6
      spec: 1.0.0
      +SYSTEMD +SELINUX +APPARMOR +CAP +SECCOMP +EBPF +YAJL
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 8
  eventlogger: journald
  hostname: XXXX
  kernel: 5.4.13-201.fc31.x86_64
  os: linux
  rootless: true
  slirp4netns:
    Executable: /usr/bin/slirp4netns
    Package: slirp4netns-0.4.0-20.1.dev.gitbbd6f25.fc31.x86_64
    Version: |-
      slirp4netns version 0.4.0-beta.3+dev
      commit: bbd6f25c70d5db2a1cd3bfb0416a8db99a75ed7e
  uptime: 2h 53m 5.59s (Approximately 0.08 days)
registries:
  search:
  - docker.io
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - quay.io
store:
  ConfigFile: /home/mike/.config/containers/storage.conf
  ContainerStore:
    number: 0
  GraphDriverName: overlay
  GraphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-0.7.5-2.fc31.x86_64
      Version: |-
        fusermount3 version: 3.6.2
        fuse-overlayfs: version 0.7.5
        FUSE library version 3.6.2
        using FUSE kernel interface version 7.29
  GraphRoot: /home/mike/.local/share/containers/storage
  GraphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  ImageStore:
    number: 8
  RunRoot: /run/user/1000/containers
  VolumePath: /home/mike/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.7.0-2.fc31.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
Physical machine, running a fully updated Fedora 31 installation (which was a fresh install rather than upgraded from previous release)

Are y ou running as root, or rootless

Running as my own user, which has the default uid/gid of 1000:1000
I've tried passing -u 1000:1000 to the run command as well to no avail.

SELinux pops up as the most obvious cause to me. Does appending :z to the volume mount to relabel it (or running the container with --security-opt label=disable) resolve the issue?

Please show the command that does not work, and what errors you are seeing.
Most likely SELinux or User Namespace related.

The linked repository contains a minimal reproduction of the issue I was experiencing.
Appending :z does indeed work, thanks. Unfortunately my knowledge of SELinux is pretty limited. Why do the files need to be re-labelled?
I see that on the docker documentation there is a warning that this could have unintended consequences due to the relabelling, and I notice that the relabelling removes the unconfined_u / user_home_t label and adds system_u / container_file_t
(ref: https://docs.docker.com/storage/bind-mounts/#configure-the-selinux-label)
Why did this work without relabelling using docker? Was it due to docker using root permissions?

Also tested using the --security-opt label=disable parameter, and can confirm that also worked (and did not modify the existing labels)
I'll close this, but if you don't mind explaining the reason why this becomes necessary when migrating to podman from docker I'd appreciate it - this has been my only real pain point so far and I'd like to better understand it

This should be identical between Docker and Podman - you may have been running Docker without SELinux support?

Definitely possible, I've been using fedora with docker since fedora 28 or so and don't believe I made any customisations to disable selinux, but I can spin up a VM and test that theory

Depending on the version of docker, native or upstream, it might not have had the --selinux-enabled flag passed into the daemon.  Without this flag SELinux separation for containers would be disabled.


correct pathspec for build.sh

Correctly points path to android dir in build.sh

test this please

Hm, not sure if this actually helps. build.sh is the standalone build script, means any customer should be able to run the build without lpm, please refer to: https://github.com/Leanplum/Leanplum-Android-SDK/blob/master/README.md#build-the-sdk
Build fails too....

@sayaansaha  could we close this one?

I'm going to test this and update this PR, I still think there is something going on here.

I'm closing this PR for now, please open a new one, when you have done with testing. Thank you!


JsonSchemaValidator fails to load relative schemas even if provided an URI

From<EMAIL_ADDRESS>on August 17, 2014 00:48:40
Line 215 of JsonSchemaValidator has:
JsonSchema jsonSchema = instanceSettings.jsonSchemaFactory().getJsonSchema(schema);

where schema is a pre-loaded JsonNode.
This prevent resolving relative $ref in the parent schema because the validator does not know what is the base URI of the parent schema.
It would be nice if matchesJsonSchema(URL url) and matchesJsonSchema(URI uri) could carry this URI down to the matchesSafely method instead of using JsonLoader.fromURL (thus losing the base URI).
I've confirmed the suggested approach works with the following test case:
String json = expect().statusCode(200)
        .contentType(JSON)
        .when()
        .get("/nodes")
        .extract()
        .asString();

jsonSchemaFactory.getJsonSchema(schemaURI).validate(JsonLoader.fromString(json));

Original issue: http://code.google.com/p/rest-assured/issues/detail?id=346

From<EMAIL_ADDRESS>on August 28, 2014 23:39:01
Is this something that you always want to do when you use a URI or URL? Are there cases when you want to disable it? (i.e. should we make it configurable?)
Status: Accepted

From<EMAIL_ADDRESS>on August 29, 2014 02:08:48
I've made it configurable from the JsonSchemaValidatorSettings but URI's and URL's are parsed according to your suggestions by default now.
Status: Fixed

From<EMAIL_ADDRESS>on August 29, 2014 07:52:46
Thanks much!
Also I can't think of any reason why I would not want this behaviour when providing a URL/URI but that's just me :) So it's good you've made this configurable!


add tolerance to tools._golden_sect_Dataframe

[x] Closes #1087
[x] I am familiar with the contributing guidelines
[x] Tests added
[ ] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Sticking with tools._golden_sect_DataFrame because it vectorizes single-variable optimization, which looks clumsy (if possible at all) with existing scipy tools.  Absolute tolerance was hardwired at 0.01V. Setting a default absolute tolerance of 1e-8V takes care of the reported instability.

Sticking with tools._golden_sect_DataFrame because it vectorizes single-variable optimization, which looks clumsy (if possible at all) with existing scipy tools.

If I recall correctly, @mikofski contributed a vectorized newton optimizer to scipy in large part so that we could remove this function.

He did, and it's in scipy now, thanks for that reminder. I'll have to think a bit about ways to get an initial estimate of Vmp that minimizes risk of divergence. That's why the bisection algorithm was originally in PVLib for Matlab, which led to the golden section function.
It also appears that at least some of the previous "known" solutions for single diode equations using the 'lambertw' method were not converged.

I added 2 SciPy methods, the vectorized newton method which uses NumPy, and a cythonized api to the bounded search methods, with examples to call them with cythonize prange and nogil so they're two different things. Altho easier, the newton method would be useful here, instead I'd recommend the cythonized BrentQ

cython_optimize_brentq_benchmark.ipynb

Toms 748 is another bounded search algorithm but idk if it's vectorized

This is ready for review.
I recalculated the IV curves used for several tests to add precision. I believe the old expected results were done by me, in Matlab, before we improved convergence in the PV_Lib Matlab code.

before we improved convergence in the PV_Lib Matlab code.

I'm a little confused... the linked PR adds a warning to the user that iteration limit was exceeded, but you're removing it in this PR. Maybe you meant to point to a different PR? Still wondering if removing that exception is a good idea though. Perhaps a warning would be better than an exception and maybe more consistent with other solvers.

I'm a little confused

Understandable. That work in PV_LIB for Matlab wasn't done via PRs, now that I look again at the one I linked.
Theoretically, the exit criterion (err < atol) will be satisfied before interations reaches the computed iterlimit. I suppose some very pathological func could be engineered to make this fail, but I couldn't find one so I remove the exception to satisfy codecov. I'm OK putting it back, either as an exception or warning, but that line won't be hit by a test.

but that line won't be hit by a test.

You can add # pragma: no cover to prevent the coverage tools from complaining.
I'm ok moving forward without the check if others are too.

For the golden section function, the old default tolerance was 1e-2. With the new default of 1e-8, convergence takes 3x longer when applied to the test function. That's probably typical of what will be seen when solving single diode equations.
from pvlib.tools import _golden_sect_DataFrame
import timeit

def _obj_test_golden_sect(params, loc):
    return params[loc] * (1. - params['c'] * params[loc]**params['n'])

def test__golden_sect_DataFrame_atol(atol):
    params = {'c': 0.2, 'n': 0.3}
    expected = 89.14332727531685
    v, x = _golden_sect_DataFrame(
        params, 0., 100., _obj_test_golden_sect, atol=atol)

t8 = timeit.timeit('test__golden_sect_DataFrame_atol(1e-8)', number=10000, globals=globals())
t2 = timeit.timeit('test__golden_sect_DataFrame_atol(1e-2)', number=10000, globals=globals())

t8 = 6.4s, and t2=2.5s.


Updated page layout new version information

@kengaderdus

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@garrodonnell
PRMerger will accept sign-off from the listed author only. However, if you are a Microsoft employee authorized to update this content, you can send a request to<EMAIL_ADDRESS>for this PR to be merged. Include your role and an explanation of why the PR should be merged without approval from the listed author.


Update functorch to the latest version ( if possible ) - Windows support

Hi, there is no functorch==0.1.1 for Windows
see: https://pypi.org/project/functorch/0.1.1/

Sure, I've removed the dependency since we don't use it in this repo.


the Tutorial For Absolute Beginners

I forgot about this until just now. We have a broken link at the start of the docs about this.
Clembs has said they can get working on this, and last i've heard this may include video tutorials as well as text based ones.
I just want to throw out there that Docusaurus supports react for pages, so that may be useful maybe not idk.

Working on them! These video tutorials will be divided into short ones for each hook, that will probably be released by the time 2.0 is stable. I'll keep you all updated on this issue.


chore(test): improve test coverage

Codecov Report
Patch coverage: 100.00% and project coverage change: +12.64 :tada:

Comparison is base (451b1f8) 66.35% compared to head (9f11134) 79.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #37       +/-   ##
===========================================
+ Coverage   66.35%   79.00%   +12.64%
===========================================
  Files           8        8
  Lines         428      400       -28
===========================================
+ Hits          284      316       +32
+ Misses        115       70       -45
+ Partials       29       14       -15

Impacted Files
Coverage Δ

internal/plugin/path_config.go
97.16% <100.00%> (+26.09%)
:arrow_up:

internal/plugin/path_token.go
100.00% <100.00%> (+15.21%)
:arrow_up:

internal/plugin/revoke.go
100.00% <100.00%> (+51.85%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


The example 'Music' app should make it clear that users should press the menu button

Perhaps we can show a little arrow pointing to where they should tap?

Fixed in b670d34652967dd1d5bada471073806de26251d4.


Use official NoSleep release when navigator bug is fixed

In #3509, I changed our copy of NoSleep.js so that it checks if navigator is defined.
I submitted a PR to NoSleep here: https://github.com/richtr/NoSleep.js/pull/13
If/When that gets merged, we should switch back to use the official release

Is it merged already?
I can take this up then . :)

Nope, no movement on this one yet.

The latest version should be fixed now, https://github.com/richtr/NoSleep.js/pull/13


Round boosters

This manages roundBooster selection during setup and exchange during after the pass move.
Only missing point is to disable roundBoosters during the init phase in Engine:260

I think it’s an auto completion on a typo :)

Il giorno 23 giu 2018, alle ore 17:28, coyotte508<EMAIL_ADDRESS>ha scritto:
@coyotte508 commented on this pull request.
In src/engine.ts:

import Reward from './reward';
+import boosts from './tiles/boosters';
+import { SSL_OP_ALLOW_UNSAFE_LEGACY_RENEGOTIATION } from 'constants';
?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


⚠ Bofrost.at has degraded performance

In 733b808, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1313 ms

Resolved: Bofrost.at performance has improved in a8b5d33 after 13 minutes.


Fix OnTeamsMessagingExtensionBotMessagePreviewSendAsync to match js

Fixes https://github.com/microsoft/botbuilder-dotnet/issues/2666
Fix OnTeamsMessagingExtensionBotMessagePreviewSendAsync to send the card to:

Teams "General" channel (Roster call to retrieve team).
Channel the messaging extension was started on.
Compose box for the user.
Cleanup send activities for other handlers.

Handles if user is in a group chat or personal scope..

Pull Request Test Coverage Report for Build 82000

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.3%) to 77.203%

Totals

Change from base Build 81947:
0.3%

Covered Lines:
5879

Relevant Lines:
7615

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1


NTR: human traveler role

For new term requests, please provide the following information:
Preferred term label
human traveler role
Synonyms
human traveller role
Textual definition
A role in a human social process that is realized when the bearer travels between an origin and a destination.
Suggested parent term
role in human social processes
http://purl.obolibrary.org/obo/OMRSE_00002072
Attribution
https://orcid.org/0000-0002-9578-0788

We currently have human travel imported into OMRSE from APOLLO SV. We can change the definition to reference that class. Alternatively, you could also represent this as a participates in some human travel axiom instead of creating a human traveler role class. Although this depends on your use case.

Apologies for the late reply. The use case is for the creation of a term that describes "Contact with Person who Recently Travelled" / "exposure to human recent traveler" where a "receptor" was exposed to a "exposure stimulus" that is a human who travelled. We pursued "human traveler role" to try to avoid indicating that the "receptor" in some way "participates in some human travel".

@cmrn-rhi would you or anyone from your group like to join our August 7 call to provide clarification on your use-case for this request and to discuss whether this or another approach might better fit it?

A role in human social processes that if realized, is realized by the bearer participating in human travel.
The human travel class is in Apollo-SV, so we would import that class into OMRSE (http://purl.obolibrary.org/obo/APOLLO_SV_00000532)
That class eventually needs a new ancestor since it is currently a descendant of GO:behavior. We can log a ticket on the Apollo-SV tracker, but resolution of that issue need not impede progress here.
'human travel' =def 'A geographical migration of an organism belonging to the species Homo sapiens.'

@hoganwr Thank you that sounds great.
@dillerm Apologies for not following up to your inquiry! I was away on maternity leave.


fzf.bash kills scp remote completion

Category

[ ] fzf binary
[ ] fzf-tmux script
[ ] Key bindings
[ x] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[ ] Linux
[ x] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[x ] bash
[ ] zsh
[ ] fish

fzf.bash kills scp remote completion

I experienced the same. For me it broke autocompletion on git, scp, ssh, who know what else. I determined that it's related to the order in which things are loaded. I have a lot of single-purpose initialization scripts that my shell runs through upon initialization, and fzf is initialized somewhere down the middle. Moving it entirely to the end (executed last) fixed things for me. I'd say... for what it's worth.


While mocking, check for a running process with an open port before invoking a new process

I would like the ability to debug my functions locally while mocking my appsync api.
I use VSCode as my editor and Go is my preferred language.
This solution is in context to the Go language, but this implementation is agnostic to the language.
In the language runtime providers specifically where invoke is implemented, before we invoke a new process check if there is a process running. More specifically test for a debug process running. In the example of GO getting invoked, we can check for /src/__debug_bin. Since VSCode uses Delve and when running you main.go through Delve it will compile and execute a debug bin we can test this bin is running. The next step would be to use something netstat or any tool that can get the open port so we can send the mock event.
I did try invoking the main.go with a Delve wrapper in localinvoke.ts with the remote flag then try run go remote debugger, but I had issues with synchronizing the process, attachment of VSCode debugger and the invoke call.
I have a working fork that I am using to debug my code, but only for GO and running in linux.

I have added this to our backlog for prioritization, once this is prioritized we will work on this. Feel free to 👍 so this gets more visibility. Also you can open a PR for your use case and discuss with the team.


add support for ECDSA keys (type 3)

I added support for generating records for ECDSA keys also (type 3 in DNS). For example Debian in standard configuration uses RSA, DSA and ECDSA keys for sshd. Omitting ECDSA SSHFP record cause that OpenSSH don't match key and starts warning about man in the middle attack.

Merged, thanks.


Update error/warning levels and error messages

Updated text for clarity/education, and addition of "best_practice" level messages
Melissa is compiling updates here; she will comment below when finished/ready to be updated.
https://docs.google.com/spreadsheets/d/1-YW51qiX2TwVfLSot4waMrQWebA4i1bjxn94vpO6hnk/edit?usp=sharing

Update, error/warn/best-practice messages are written up now!


Allow setting JMS Properties when sending messages via REST.

Allows setting JMS properties on a message posted via REST interface. The property is transferred as a HTTP header having name prefixed with "jms.property.". On successful arrival on topic/queue resource the prefix is stripped and property set as JMS property. This allows filtering on custom fields and using features requiring JMS properties (i.e JMSXGroupID).

The broker is agnostic of JMS. We specifically stripping JMS specifics this is so the broker can be protocol agnostic. Maybe implement this in a way to set anyheader

@michaelandrepearce Ok I get the "agnostic" argument, so assuming I drop or replace the word "jms" (for example instead "jms.property." I use "message property.") are we fine? Or should I rewrite the whole feature? You already use http headers to pass the content-type and I'm reluctant to change that, BUT I need to somehow put custom values into message properties to use them in selectors (also I need to be able to set JMSXGroupID in order to use artemis grouping feature).


Remove unused TileGrid method

It is no longer used since 01702e09af859904dac845245b5d5c90fff97e7f.
It was used to not load tiles that do not intersect a rotated viewport. Maybe this was changed unintentionally?

Thanks for spotting this, @MoonE. This function should indeed be used again, to avoid loading/rendering tiles that would not be visible anyway.


Manage channel bookmarks page needs animated GIFs for mobile app workflows

Context
To help visual learners learn how to perform tasks using the Mattermost mobile app, add visuals and short animated GIFs on the Manage channel bookmarks page.
Parent Issue: #6657
Docs Request

Add/update appropriate visuals/GIFs for all the steps in the Add a bookmark section under the mobile app tab.
Add appropriate visuals/GIFs for all the steps in the Edit bookmarks section under the mobile app tab.
Add appropriate visuals/GIFs for all the steps in the Share bookmarks section under the mobile app tab.
Add appropriate visuals/GIFs for all the steps in the Copy bookmark links section under the mobile app tab.
Add appropriate visuals/GIFs for all the steps in the Delete bookmarks section under the mobile app tab.

Additional Notes

Use the latest Mattermost mobile app and the standard Mattermost Denim theme to capture all workflows.
Avoid drawing lines, arrows, boxes, or callouts on images.
Don't forget to include ALT Text descriptions for all new image assets.
Use existing docs pages with images as a starting point for code guidance. The Docs team can help provide RST syntax support for publication.
Save static image files in PNG format within the docs repository's /images directory.

QA Validation Criteria
In a generated docs preview:

Visiting the docs page results in appropriate new mobile-specific screenshots in each subsection from the Mobile tab.

@cwarnermm Please assign this issue to me. Thanks.


Not registering Slash Commands

Since the only other person to have this issue was rude, lemme open an actual one.
When starting the bot, it logs that the slash commands were loaded, but in the actual server nothing appears
log:
[2023-01-08 19:19:31]: LOG Loading Slash command: leave. 👌  [2023-01-08 19:19:31]: LOG Loading Slash command: ping. 👌  [2023-01-08 19:19:31]: LOG Loading Slash command: stats. 👌
Slash Commands in the server:

You need to manually deploy the commands. You should be able to do run @bot deploy to deploy the commands, and then they'll deploy. There is a pr open, that among other changes, documents this in the readme.


Publish 0.15.8

0.15.8 version allow dart emum into select, but that version hasn't been published yet. Would be nice to have it published
Thanks !

I think this issue can be closed(https://pub.dartlang.org/packages/intl/versions/0.15.8)


[web] Add coverage for js/urlState.js

It's a little bit tricky to complete the tests for this, so I want to explain a bit here.
The reducers(reduceFlows...) are require here to create a store object which our target function will operate on, so we could not mock the whole js file as we usually do. However, we don't care about the implementation of certain actions (setFilter...) and want to mock them up.
In this case, jest.mock() does not work. After looking up the documentation, I found a solution which uses jest.spyOn(). We could import all the reducers and actions first, and mock the action functions before calling the target function. Then we could use the handy toBeCalledWith() function to test if we are passing the right arguments to the action functions.

Now that we have covered almost all of the .js files, it's time to move on to the .jsx file. :rocket:
According to the official guide of Jest, we could either use snapshot test or traditional dom test on this task.
@mhils @cle1000 Which way do you prefer?

Hmm - this looks like a prime candidate for http://arnaudbenard.com/redux-mock-store/ instead - what do you think? I think we could just use a mocked store and then check if .getActions() equals what we expect?
For the DOM part, I think snapshot testing is the more efficient approach in most cases, so let's start with that. 😃

Now that we mock all the actions, updateStoreFromUrl could not affect our store object any more. So it's better to separate the test up and test updateUrlFromStore independently.
And I found a minor mistake in initialize() function of urlState.js, we should subscribe to updateUrlFromStore before updateStoreFromUrl


File Read Faults With Null Reference

The attached file was written by a 3rd party app and causes the reader to fail with a null ref exception.
If I copy the data to a native workbook, it is read OK.
Flingor Skus.zip

Please use latest develop build: no problem opening binary file in TestApp :+1:


Increase contrast of text in disabled text boxes

Before:

After (matches disabled color in QMenuBar):

Fixes #33
One-line change but it applies to a lot of widgets; did my best to browse around and look for problems but would like someone else to test as well in case I missed anything.

Currently OOO so can't test, but screenshot looks good!

Sorry for taking so long.
I think the styling on the specific example has changed with the 0.21 release, as the example box is not a "disabled" one anymore it seems:

Still, the commit makes perfect sense to include.


[TC-130] Service Configuration Streamlining and Sequencing

Hi,
In order to further improve the simplicity and robustness of the control path for provisioning infrastructure and delivery services, we are currently considering ways to streamline management and operations.
Currently, when applying changes in traffic-control that require the synchronization between the traffic-router and traffic-servers, the user should be conscious to do so in a certain order. Otherwise, "black holes" may be created. Furthermore, in some of the scenarios the user have to wait and verify that the configuration reached all traffic server before he may apply it to the traffic-router.
The main use-cases we would like to address at this point are:

Assign servers to a Delivery-Service.
For this operation, the configuration must first be applied to the added traffic servers, propagate, and only then applied to the traffic-router.
Remove servers assignment to a Delivery-Service.
For this operation, the configuration must first be applied to the traffic-router, and only then to the traffic-servers.
Add a new delivery service.
This is practically a private case of servers assignment to a delivery-service.
Delete a delivery service.
This is practically a private case of servers assignment removal from a delivery-service.
Update settings that must be applied together on the traffic servers and the router.

We would like to simplify the procedure, allowing the deployment of new configuration in a single operation, instead of doing it step by step.
One solution can be based on the insight that deploying such configuration changes may be done by initially updating the traffic server with added functionality (e.g remap-rule), then updating the router, and lastly, removing old functionality from the traffic servers. Such a solution can be orchestrated by traffic-ops, probably without complicating other components.
Other solutions may provide more flexibility, but would probably involve adding complexity to other components such as traffic-router.
An example to such a solution, already under discussion in a mail tread (10x Eric may be based on the concept of a delivery-service configuration "generation" which would be an ordinal identifier for the a delivery service configuration. A "generation" changes whenever the remap rule changes or the consistent hash mapping of content to server changes (e.g. due to additional server assignment).
In such a solution, each traffic-server may hold a single generation for each delivery service configuration, while traffic-router may hold a history of generations and know which server holds which configuration generation.
This approach introduces a considerable flexibility. It allows configurations to be set one after the other with no need to wait between them.
On the other hand, complicated algorithms for solving the issue may impose more risk to the network when applied, comparing to a simple "traffic-ops" orchestrated solution.
Both approaches fit well with TC-129: Create TO API endpoint to queue all servers for a delivery service
I'm not sure what is preferable from an operator point of view. I'm also not familiar with TC 3.0 "Config State Machine" solution to validate he different approaches against.
Nir
Author: Nir Sopher
JIRA Link: https://issues.apache.org/jira/browse/TC-130

@nir-sopher - this could warrant a blueprint / mailing list discussion. Want to start that?

This sounds a lot like https://cwiki.apache.org/confluence/display/TC/Self+Service+Change+Integrity.


Refactor List/ListDir

WIP: DO NOT MERGE. Tests are expected to fail.
Concept proposal for #316
This shows the concept. All the heavy lifting is done in the "fs" package, making file system implementations a bit simpler.
/fs shows interfaces, and should compile.
/amazonclouddrive: Changed to new interface.
/b2: Changed to new interface.
/local: Changed to new interface.
The Fs no longer needs to create a separate goroutine when List/ListDir is called. Instead it can block as long as it likes. This simplifies implementations a bit (at least it doesn't become more complex).
Implementation is hidden in "fs" package, and only exposes the minimally needed interface, which of course can be extended.

The only thing I don't particularly like is that func (o *listOpts) Get() (Object, error) returns nil, nil to indicate that there are no more objects. However, func (o *listOpts) Get() (Object, bool, error) seems clunky.
Furthermore Get() is only used internally in "fs", so it isn't a "public" issue.

It looks like a good idea - well done!
I'm not particularly worried about the use of the nil Object in Get as a sentinel.
There are clearly some bits which need reworking a bit after this (eg the fatal errors should be returned up the layers).
Looking at the duplicated code between listdir and list it makes me think that combining the two might be the thing to do.  There are issues like #213 and #100 which would be improved if rclone knew a bit more about directories.  This would mean more options in listOpts to only list directories or files or both, and a way of passing the directories back - maybe as a different sort of Object.
This would be easy for fses like ACD - a little harder for ones like s3/swift but a with a helper function it would be straight forward.

Here is another issue which could be helped by a bit better directory / file integration #268

Great idea to merge the interfaces. Makes it much cleaner, and avoids the duplication.
This change means that List both returns directories and objects, but the receiver can chose to ignore one or the other.
The only complication I found was directory size calculation (see this), which becomes a bit tricky. Since filepath.Walk has no way easy way of keeping track of which directories a file belongs to.
Right now, it should return all (sub)-directories, not just the root.

Originally rclone only supported swift, and swift can tell you how many files and directories are in a bucket which is where the feature came from originally. There is sporadic support amongst the the other remotes for parts of it.  So dropping that from the local fs is no big deal! rclone size works very well for finding out stuff like that now too.
I rebased the branch (which was a little tricky!) and pushed it to the list-refactor branch which I propose we keep as a long running branch.
A couple of things I noticed while doing that

newListOpts will need to be public as fs.List is a public interface
Having a ListOpts.GetObject and .GetDir would simplify the code a bit - these would just call .Get until they got the right kind of object.
We'll have to add a depth parameter to ListOpts so lsd can work as it does now - this will be good for other things too.

I'll take a stab at adding support to some of the other remotes and see what that looks like.

Sounds like a good plan.

Having a ListOpts.GetObject and .GetDir would simplify the code a bit - these would just call .Get until they got the right kind of object.

That could be tricky, since a full channel could block further processing. The intention behind the current design was that you would only need one consumer goroutine. If you separate Get into two functions, you would either need two goroutines, or it should return without blocking, which is also suboptimal.

I meant something like this (untested) which I think satisfies the requirements.
// GetObject gets an object from the listing, skipping over
// directories.  Returns nil, nil at the end of the listing.
func (o *listOpts) GetObject() (Object, error) {
	for {
		o, dir, err := opts.Get()
		if err != nil {
			return nil, err
		}
		// Check if we are finished
		if dir == nil && o == nil {
			return nil, nil
		}
		// Ignore directories
		if dir == nil {
			return o, nil
		}
	}
}

// GetDir gets a Dir from the listing, skipping over Objects.  Returns
// nil, nil at the end of the listing.
func (o *listOpts) GetDir() (*Dir, error) {
	for {
		o, dir, err := opts.Get()
		if err != nil {
			return nil, err
		}
		// Check if we are finished
		if dir == nil && o == nil {
			return nil, nil
		}
		// Ignore objects
		if o == nil {
			return dir, nil
		}
	}
}

Ah - I see. I didn't get the "skip the other type" part. Of course. I see how that makes implementations easier!
If there is more I can do here, just let me know, or open a PR with the new branch, if you would like feedback before merge.

I'm closing this one in favour of #446


Use retention policy for form and create separate reset retention

Follow up to previous PR, this PR completes the decoupling of compliance and retention by using retention policy as source of data for purge policy form instead of complianceInfo. We also create a separate reset that can be called to reset the retention policy and modify the POST request to retention to use retentionPolicy instead of compliance. This should completely solve the issue where, in cases where a dataset does not have compliance, a user was not able to update the purge policy.
ember test results:
1..500
# tests 500
# pass  493
# skip  7
# fail  0

# ok

Coverage remained the same at 23.002% when pulling 93b088fcae698b5dcb3598cead9d1dbedef5638f on cptran777:reset-retention-on-save into d209c93a47570b2714c9de6227399e10abd45571 on linkedin:master.

Codecov Report

Merging #1413 into master will decrease coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1413      +/-   ##
==========================================
- Coverage   60.35%   60.32%   -0.03%
==========================================
  Files         183      183
  Lines        4000     4003       +3
  Branches      918      918
==========================================
+ Hits         2414     2415       +1
- Misses       1586     1588       +2

Impacted Files
Coverage Δ

wherehows-web/app/utils/datasets/retention.ts
100% <ø> (+60%)
:arrow_up:

...mponents/datasets/containers/dataset-compliance.ts
36.13% <0%> (-1.14%)
:arrow_down:

wherehows-web/app/components/dataset-compliance.ts
25.28% <0%> (+0.14%)
:arrow_up:

Continue to review full report at Codecov.

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

Since this is also vital, merging now and will address comments retroactively.


Set-AuthenticodeSignature Changes Script

Steps to reproduce
run Set-AuthenticodeSignature -Certificate $cert -FilePath $AttachedScript
Expected behavior
Only the signature should change
Actual behavior
$Global:LogRoot = "\cyprus\share\ITScriptOutput"
to
$Global:LogRoot = "\cyprus\share\ITScriptOutput
The last character was removed
settings.txt
Environment data

> $PSVersionTable

This issue is reproducible for:
Name                           Value
----                           -----
PSVersion                      5.1.14393.82
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.82
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

@v-mimcbr is this still an issue with the latest version of PowerShell?

I can not duplicate this in the latest versions of PowerShell.
Can we close this issue?

/cc @jborean93 Thoughts?

@StevenBucher98 can this be closed as the issue can not be duplicated?


Default value for ariaLabelledBy should be undefined

@JakeChampion When you questioned this line before, I got the response wrong. undefined || '' in this context always results in an empty string. An empty string results in aria-labelledby="", which is invalid. undefined is the correct default, which does not render the attribute.

Thanks @sussexrick, could you run the "npm run build" command and commit the built files to this pull-request? After that is done we should be able to merge and release this

@sussexrick Thanks! This is released in v2.2.1 👍


What is the difference between this code and the original in details ? Help me please.

I want to repeat the results same as the original paper. But, I get a BLEU-1 score about 0.66. What can I do next to achieve the same results? I have been frustrated for a long time. Please.

Did you run this yunjey's code without any errors?
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
Traceback (most recent call last):
File "train.py", line 25, in
main()
File "train.py", line 22, in main
solver.train()
File "/home/yujunhyung/show-attend-and-tell-tensorflow/core/solver.py", line 77, in train
loss = self.model.build_model()
File "/home/yujunhyung/show-attend-and-tell-tensorflow/core/model.py", line 168, in build_model
_, (c, h) = lstm_cell(1, inputs=tf.concat(1, [x[:,t,:], context]), state=[c, h])
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
I get this error when I run train.py code.
Is there any solution?
i used tensorflow 0.11.0 version & python2.7
What version did you use?

@JunHyungYu As far as I know, there exists some incompatibilities among different tensorflow versions. A good choice is to copy the  error information and paste it in Google for resolution. I ever encountered a revised version of this repository that is compatible with the most recent tensorflow. Another choice is to search that revised version in github, after that there should exist no errors. Good luck!


Recruit more Yetinauts to work on issues and PR

I see many issues and PR without response. I don't think it's a good thing for a so big and active community. You should recruit more Yetinauts to work on it.

Hi @colin-marshall
Since this issue I started to contribute a little to Foundation. I can't always resolve issues and purpose PR, but I can add tags, or close abandoned issues. It could help you by tracing the more important things.
Can the community give me the rights for this ? Thx

@ncoden sorry for the slow response. Thank you very much for your contributions to Foundation! It's great to see fresh faces helping out.
The Yetinaut program is brand new. I believe if you keep contributing you will eventually be added on. I don't think that will take too long at the rate you are contributing. Your contributions won't go unnoticed by the people at ZURB, just give it a little time.
Keep up the great work!

@colin-marshall Thanks for jumping in! You've been doing great things!
@ncoden Great PR's!  There is a great community around Foundation and there are lot's of people like Colin and yourself that contribute. We just launched the Yetinauts into space and are working to build this up more each week. Part of what we're doing moving forward will help people contribute in meaningful ways.
Look for more planning and road-mapping so that people know where to contribute and how to add the most value. We're really excited about the working with our awesome community to push Foundation forward even faster (if that's possible).
We've gotten to know our Yetinauts over time and learned what their into doing. We'd also like to get to know you more , Nicolas! The Help Wanted would be a great place to start or answering questions on the forum

Thanks @colin-marshall @rafibomb . So I'll continue to contribute when I have the time for ;)
I'm currently working on (flex-)grid, and the modifiers mixins you provide for your components, to make them usable in a semantic scss.
BTW, is it possible to discuss about major changes on syntax ? There is lot of modifications we could do to facilitate the Foundation use and maintenance, like BEM/SMACSS mix for naming, layout/module/section recommandation, sass-lint (npm) or scss-lint (gem)...
I don't know how you work at Zurb, but I use this for my projects. There is still a lot work to do, but you can pick some ideas.
Thx :)

@ncoden major changes on syntax are probably not going to happen for v6, but if there are compelling cases for alternatives we can definitely consider them when it comes time to move towards a v7 for foundation.
That being said, I know there was a lot of thought put into the class naming conventions that exist, though I'm not the best person to talk on that subject (maybe @gakimball or @zurbrandon can chime in), perhaps you're running into some situations where there's a bug/inconsistency with the conventions that should get cleared up?

:+1: I was thinking of v7 yes.
 But I think it could be better, more modular, easier to use with sass.


fix(js_semantic): add TS_QUALIFIED type for reference

Summary
Fix discussion: https://github.com/biomejs/biome/discussions/2059
Test Plan

I am not sure if this failed for my pr..

Thanks!


Dotnet Component Metadata not resolved

When im using cdxgen on a folder with a .csproject file, the Bom doesnt contain any component meta information.
This leads to errors using some other tools (eg. Proget).
Currently im able to pass a projectname and version using like this:
--project-name name --project-version 0.21.2
The problem is, that the other metadate couldn't be set in this way (we have libraries and get application as a type)

I think the problem here is, that some of the sourcefiles (package.lock.json) etc dont have any information of the library that you are building. Therefore i think the most generic way would be, to give the cmd the option to overwrite the information or to inject a json file for the component.

Looks like its going in the same direction and would provide a convinient and generic way. Thanks.


DockerService network

Codecov Report

Merging #5 (8048ff0) into master (a5f5ccc) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #5      +/-   ##
==========================================
+ Coverage   97.47%   97.48%   +0.01%
==========================================
  Files          61       61
  Lines        1306     1312       +6
==========================================
+ Hits         1273     1279       +6
  Misses         33       33

Impacted Files
Coverage Δ

Docker.js
100.00% <100.00%> (ø)

DockerService.js
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


notify readiness when registered plugins are ready

Fix: #8468
Added RegisterReadiness func to InitContext to allow plugins to register themselves as required plugins which will be waited for the readiness before systemd notification is sent. This PR only registers CRI plugin to address #8468.

/ok-to-test

Thanks @samuelkarp and @estesp for the review.
Can we backport this patch to 1.6 and 1.7? If so I'm happy to open the PRs too.

Yes, I'd like to see this backported to both 1.6 and 1.7.

+100 to backport


Quantile Loss Tutorial update spelling and links

Quantile Loss Tutorial

Update spelling of Gneiting in two places
Update reference URL to DOI in two places (DOI link should be more stable)

Along with pull request #271, this merge request fixes #233

Checked in jupyter notebook. Spelling correction is good. Moving to use using DOIs is probably reasonable as it should be a more reliable link in general, although I have no issue with using the original link either. Confirmed both links resolve to the article correctly.


Docs section: BCDR- Technical content

Page section: intro

Provide link back to ESLZ BCDR guidance alongside link to WVD DR docs

Page section: Design Consideration - Host Pool Compute Strategy

The content here is great, however does it clash with the WVD docs DR article that is linked in the intro? Set up Windows Virtual Desktop disaster recovery plan - Azure | Microsoft Docs

Page section: Design Consideration

Convert this bullet point to an numbered/ordered list:
"The recommended options for container storage types are (in order): Azure Files Premium, Azure NetApp Files Standard, and Azure NetApp Files Premium. The recommended storage type depends on the resources and latency required by the specific workload."

Fix:

Provide a single source of truth for this information to avoid article drift over time
Update docs

Migrated to ADO


Why are base weights on HF LoftQ models in 16-bit?

The script quantize_save_load.py generates a quantized model with LoRA adapters.
The base model is then saved and uploaded to LoftQ repos such as this one.
I'm puzzled why the base model weights are 16-bits there because that implies that the base model is somehow upcasted (dequantized) in the quantize_save_load.py script, but I don't see that anywhere.
My baseline expectation is that either:
a) The backbone would be stored in nf4, and then loaded with the 16 bit adapters on top, or
b) The backbone would be upcasted to 16-bit, and then quantized in nf4 upon loading with the 16-bit adapters on top. [But then there should be some upcasting code in quantize_save_load.py].
Could someone clarify? Thanks.

Hi, @RonanKMcGovern . In old version of bitsandbytes, it is not allowed to save nf4 format weight. We void this issue by saving it in 16 bits in the disk and transforming it into 4 bits when loading it into GPU. However, as the bitsandbytes get updated recently, it is possible to store it in 4 bit weight. We will update the code soon.

Ok, but are you even running the nf4 quantization then?
Or are you just directly saving the bf16 weights? If you're doing that, there is going to be error when reloading the model because the saved bf16 should be the dequantized weights, not the original...


[CI] The PR Checks of TenrsorFlow are broken.

It is strange. It seems that the PR Checks are suddenly broken. Anyone why this issue is still not fixed?
As you can see, Many PRs have been not reviewed by reviewers because the Tensorflow CI facilities are still a running status. BTW, Why this issue is generated? What is a possible reasons? Who is in charge of the CI facilities of the TensorFlow github repository?
* Ubuntu CC Expected — Waiting for status to be reported  Required
* Ubuntu Sanity Expected — Waiting for status to be reported  Required
* import/copybara Expected — Waiting for status to be reported  Required

Screenshot:

The behavior you see here is expected. It says that those tests are required but are waiting to be initialized. Once we trigger the build tests their status will be updated accordingly.

Ubuntu CC Expected — Waiting for status to be reported
It says that those tests are required but are waiting to be initialized.

@ymodak It seems that the tests are too slow when PRs are submitted.  Are the slow tests generated from the too many pull requests (PRs)?


Bug 1867819 - Rewrite installer_controler status report to include Re…

…visionReason
@sttts @mfojtik

/lgtm
/ok-to-test

/retest

rebased

All of the requests have been addressed

/reopen

Created: https://github.com/openshift/library-go/pull/1063


Remove unmaintained translations

Description
removing translation under 70% and/or unmaintained (original stays on transifex)
Checklist

[x] I have opened this pull request against develop
[x] I have updated the CHANGELOG.rst
[x] I have added or modified the tests when changing logic
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

(just resolved the conflict)

@vinitkumar can we merge this one too? (that's a good 30min download on the release process ;) )

(I can also remove the changelog modification if need be, following talks on slack).


Adding GitHub Actions workflow

My first attempt at adding a GitHub Actions workflow.
This puts two grib files from HRRR in the repo using git-lfs. The tests are our standard set -- linting and pytest suite. I also updated the environment.yml to match our pygraf environment instead of the older verison of adb_grpaphics.
I have no idea if this will work!

There's a good chance that I need to merge this PR before the actions can be tested.... :(

I echo Craig's sentiments.


Dash Core GUI crash on MN "start alias" on unlocked wallet

Dash Core v<IP_ADDRESS> for Windows x64.
Steps:

Unlock wallet (either via GUI : "Settings->Unlock wallet") or via CLI:

walletpassphrase "passphrase" timeout

"My Masternodes" Tab
"Start alias" on several MNs.

Actual results:

GUI becomes unresponsive. Only way is to kill "Dash Qt"

-Technologov, 02.06.2017.

Maybe same as https://github.com/dashpay/dash/issues/646 ?

Duplicate


(Performance idea) Code generation

Hello, Jake!
You've stated that one of this project goals is to Be fast.
I believe that using code generation is faster than using reflection, and (correct me if I'm wrong) current implementation of Binder is using the reflection. Are there any plans to use the code generation?

@VadymLevkovskyi Thanks for your interest!
There are no plans to support code generation.. but I'm not opposed to it. Some of my initial thoughts:

This thing uses a LOT of reflection. I'd want to incorporate some sort of middle layer (or another library?) so it's plug and play as much as possible.
The generic types and methods will most likely be problematic. That may be ok though, perhaps we could use reflection in those situations.

Are you open to working on this?


[core] add proper parameter type for TrustedTypePolicy createHTML callback

Packages impacted by this PR
@azure/core-xml, @azure/core-http
Issues associated with this PR
Fixes issue #24872.
Describe the problem that is addressed by this PR
Upgrading to @trusted-types v2.0.3 caused compilation error. While it is breaking, it surfaces an issue that we can address by adding propert parameter type.
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Pinning to v2.0.2 which isn't desirable as we want to stay on latest dependency versions.

API change check
API changes are not detected in this pull request.


add default thumbnails

This adds default thumbnails for sources.  If a thumbnail is not present for a source, a default image is shown instead.  This will be especially helpful for audio and video sources - admins generally do not make custom thumbnails for these, and can rely on defaults.
This addresses ticket #8064.

That looks good, and I can merge it when it's rebased.

@markbreedlove rebased - should be ready to merge when tests pass


Added setNativeProps for performance

Hi, Thank you!

No worries, although I realised that to properly utilize the setNativeProps, we should probably not be setting the state on move, but only on drag/pinch end and only update the native props between start and end.

Hi. I'm wondering, can I use this function to set the start styles of a component on drag/pinch/rotate?


ko ignores packages that contain their own go.mod

Given the following project structure:
.
├── cmd
│  ├── awscodecommitsource
│  │   ├── go.mod
│  │   ├── go.sum
│  │   └── main.go
│  └── controller
│      └── main.go
├── go.mod
└── go.sum

and the following manifest (simplified):
apiVersion: apps/v1
kind: Deployment
metadata:
  name: aws-sources-controller
    spec:
      containers:
      - name: controller
        image: ko://github.com/triggermesh/aws-event-sources/cmd/controller
        env:
        - name: AWSCODECOMMITSOURCE_IMAGE
          value: ko://github.com/triggermesh/aws-event-sources/cmd/awscodecommitsource

ko builds the image for cmd/controller but ignores cmd/awscodecommitsource, and keeps the literal value.
After removing go.mod and go.sum from cmd/awscodecommitsource, ko builds both images and replaces the value accordingly.

/remove-lifecycle stale

This might be addressed by https://github.com/google/ko/issues/305


break nject into five repos

Codecov Report

Merging #24 (89954fe) into main (441bef9) will increase coverage by 4.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   73.32%   77.61%   +4.29%
==========================================
  Files          26       13      -13
  Lines        3483     2310    -1173
==========================================
- Hits         2554     1793     -761
+ Misses        700      362     -338
+ Partials      229      155      -74

Impacted Files
Coverage Δ

api.go
93.82% <ø> (ø)

bind.go
85.21% <ø> (ø)

cache.go
55.62% <ø> (ø)

characterize.go
86.75% <ø> (ø)

debug.go
83.40% <ø> (ø)

error.go
85.71% <ø> (ø)

filler.go
62.53% <ø> (ø)

filler_api.go
83.33% <ø> (ø)

generate.go
62.24% <ø> (ø)

include.go
92.35% <ø> (ø)

... and 4 more

Continue to review full report at Codecov.

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


Minor cosmetic changes to API reference: reorder gradients methods and add backtick.

This change is

Review status: 0 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


The identifier fileName is missing for a query of Jb\Bundle\FileUploaderBundle\Entity\FileHistory

at ORMException ::missingIdentifierField ('Jb\Bundle\FileUploaderBundle\Entity\FileHistory', 'fileName')
in vendor/doctrine/orm/lib/Doctrine/ORM/EntityManager.php at line 403   +
at EntityManager ->find ('Jb\Bundle\FileUploaderBundle\Entity\FileHistory', null, null, null)
in vendor/doctrine/orm/lib/Doctrine/ORM/EntityRepository.php at line 154   +
at EntityRepository ->find (null)
in vendor/jbouzekri/file-uploader-bundle/Jb/Bundle/FileUploaderBundle/Service/Validator/Constraints/FileOwnerValidator.php at line 55  +

Hello I was facing to this issue that is come from to your validate method of your validator \Jb\Bundle\FileUploaderBundle\Service\Validator\Constraints\FileOwnerValidator::validate.
at this line :  $fileHistory = $this->em->getRepository('JbFileUploaderBundle:FileHistory')->find($value);
Issue is due to the default null value of my field $image passed as parameter to find method.
/**
* @ORM\Column(type="string", length=100, nullable=true)
* @JbAssert\FileOwner
*/
protected $image;
Here the nullable is set to true and.
I fixed it by adding this verification before the call of find method:
if(!$value){
return;
}

Thank you for pointing that out. Sorry I want be able to look into this before Monday.
The fix you are proposing seems good. Can you make a pull request ?

Done in latest release.


Renamed STM32G471RE to STM32G491RE?

Could it be that the chip "STM32G471RE" should be named "STM32G491RE"?
According to the overview, there is no 471.
Also, the product selector doesn't list a 471.

According to https://www.keil.arm.com/packs/stm32g4xx_dfp-keil/devices/ and an errata I found https://www.st.com/resource/en/errata_sheet/es0430-stm32g471xx473xx474xx483xx484xx-device-errata-stmicroelectronics.pdf 471 is correct. I would not trust any ST resource. I have never seen correct ST resources, so yeah :/


step 6

In Step 6, most of the steps needs to be changed and the screen shot updated as it is outdated.

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

ID: 03a0dc36-e3d5-3a28-c9f9-9e514150d7a4
Version Independent ID: 5b19f443-599f-e367-3566-f43be87dcf78
Content: Tutorial: Add SQL Managed Instance to a failover group - Azure SQL Managed Instance
Content Source: articles/azure-sql/managed-instance/failover-group-add-instance-tutorial.md
Service: sql-managed-instance
Sub-service: high-availability
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

@jamserra Thank you for bringing this to our notice and sharing your feedback. I have assigned this issue to our document author @MashaMSFT to further evaluate the feedback and update the document accordingly.

Hi @jamserra , thanks for bringing this to our attention!I have created an internal work item (1744245) to track this work and hope to have this fixed for you this month.
I'm going to close out this Git Issue for the time being but please feel free to comment should you need anything further.
Thanks again and hope you have a wonderful rest of your day!
Masha from the SQL Docs team
#please-close

Hi @jamserra, thanks again for helping improve our documentation! I have processed a change to address this and it will be live later this week :)
Thanks!
Masha from the SQL Docs team


🗑 Deprecation promotion of toNotEmptyList function in ResultContext to hidden level

📝 Description
We would like to promote the deprecation of toNotEmptyList function in ResultContext to a hidden level.
🔗 Dependencies
This issue is blocked by the following ones:

[x] #508

✅ Checklist

[ ] Promote the deprecation of this declaration, update the public API binaries and the unreleased changelog.
[ ] Close this issue as completed and update tracking ones if present.

We are not ready to implement this change.


Add scRNA-seq and scATAC-seq integration methods

Add Cobolt, LIGER, FigR, MultiVI (reference) as well as scDART.
Update the information of cisTopic, SCENIC, and bindSC.
Add MetaMarkers.

Thanks, @carushi, and welcome to the contributor club!


Cannot singup and can't login

Subject: Cannot singup and can't login
Description:
I have one email only in life.
I deleted the account but now want to undelete it.
Apparently I can't, but I also can't sign up with that email anymore.
What do I do?
Results:
See above.
Backend version: develop-7c33e18d
Frontend version:
User Agent: Mozilla/5.0 (X11; CrOS x86_64 14909.124.0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Page: https://couchers.org/login
User:

I wonder if it's worth adding clarification that deletion is permanent and non-reversible etc, similar to the kinda warning you get on here if you try to delete a repo (and most other popular sites)

I responded to this user.


Timer expiring does not always visibly end the game.

This appears to not visibly display to the player who timed out


OpenGL: Screen tearing on some Android devices

Godot version
4.0beta8 - 4.0RC2
System information
Android 9, OpenGL 3, PowerVR GE8300
Issue description
On certain Android devices, there is a persistent, obvious screen tearing issue. This tearing is unaffected by v-sync or scaling settings. The top third of the screen lags several frames behind the bottom two thirds.
At present, I've only observed the issue on a Retroid Pocket 2+ handheld (Android 9, PowerVR GE8300 GPU). I've also tested the same projects on a Huawei P10 Lite phone (Android 8, Mali-T830 GPU) but observed no issues there.

https://user-images.githubusercontent.com/6191653/207448582-a6df595f-2104-4275-804c-9b63ca51af4b.mp4
I will also note that I had this issue with 3.x, as well. In that case, I observed the screen tearing on both the Pocket 2+ and the P10 Lite, and I eventually resolved it by switching the graphics driver from GLES3 to GLES2.
Steps to reproduce

Create any project with moving graphics using gl_compatibility renderer
Export Android APK with debug
Install and run on affected device

Note that the issue only affects certain Android devices, and may require testing on multiple devices in order to narrow down the cause.
Minimal reproduction project
Any project exported using OpenGL 3 will do, but here's the one I made to test it.
Android Test 4.0b8.zip

Just tested export from Godot 4.0beta9 - issue persists

I just tested an export from Godot 4.0 RC 3, and as of this version, the screen tearing is no longer occurring on the Retroid Pocket 2+!
I re-tested on the Huawei P10 Lite, too, and the screen tearing is still no longer occurring on that, either.
This exhausts the Android devices I'm able to test on. Hopefully nobody else is still having the issue on their devices, either.

I'm still having this exact issue with the Retroid Pocket 2+ on v4.0.stable.official [92bee43ad]
I tried downloading and installing the zipped project above and I'm getting the same results as the video above as well.
I'm really new to this so it could very likely be a configuration setting somewhere for the project/android export I'm missing.
Any notes/links/ideas would be great!
Last thing I'm checking tonight is to try to update the Retroid Pocket 2+ to it's latest OTA update.

I'm still having this exact issue with the Retroid Pocket 2+ on v4.0.stable.official [92bee43]

In Godot, which renderer are you using? For exporting for older Android devices, like the RP2+, you should be using Compatibility (gl_compatibility) for best results.

Yeah I was using the Compatibility version, ironically I found that switching to Mobile somehow helped and isn't showing the glitching anymore 🤷♀ . I'm ok with using that for now but I can also keep testing the other mode if needed.

Yeah I was using the Compatibility version, ironically I found that switching to Mobile somehow helped and isn't showing the glitching anymore 🤷♀ . I'm ok with using that for now but I can also keep testing the other mode if needed.

How odd! I didn't think the Mobile renderer was compatible with the RP2+. I'll have to test it myself at some point.

I checked online, and supposedly the GPU in the RP2+ should support vulkan 1 which I think is why it's working, but again I'm very new to this

RetroDodo Review/Specs
PowerVR GE8300 Specs

There's still screen tearing on the RP2+ as of 4.2.2 rc2 unfortunately. I'd reopen a ticket about this, but I don't know if it's worth it for such a niche device.


Rename optimization procedures

What
Changing the names of classes and modules for optimization to the more understandable:

C1 -> OptimalControl
C2 -> Calibration
C3 -> ModelLearning
and correspondingly
c3.optimizers.c1 -> c3.optimizers.optimalcontrol
c3.optimizers.c2 -> c3.optimizers.calibration
c3.optimizers.c3 -> c3.optimizers.modellearning

Are we gonna leave Sensitivity SET alone? Else, I can also update that in my WIP branch for resurrecting Sensitivity.

I plan to do SET as well and change the robust to OptimalControlRobust to be consistent. I'll push some more changes before removing the "draft".


method method

https://docs.raku.org/routine/append
has a 'method method push', which needs to remove the first 'method'. It wasn't caught because we explicitly allow method to be a duplicate in the xt/

Actually here. On it.


Validate node address and port when used

There's a TODO in the code. Let's validate these before connecting, like we do with the socket.

This was completed with #64


Exception cannot be caught when Request-VCFToken cmdlet is supplied with incorrect credentials

Code of Conduct

[X] I have read and agree to the project's Code of Conduct.

PowerVCF
2.2.0
VMware Cloud Foundation
4.4
PowerShell
5.1
Guest Operating System
Windows Server 2012 R2
Environment Details
No response
Description
https://github.com/vmware/powershell-module-for-vmware-cloud-foundation/issues/148 - This was the bug that I had raised on the same issue. Today the issue still persists.
Expected Behavior
Should be able to catch the exception and based on the exception should be able to display custom message instead the exception getting displayed as shown in the attachment.
Actual Behavior
Exception gets displayed as is without being caught.
Steps to Reproduce
Supply Request-VCFToken cmdlet with incorrect credentials and execute.
Log Fragments and Files
No response
Screenshots

Additional Context
No response

This can be resolved with the following:
    Catch {
        if ($_.ErrorDetails.Message) {
            Write-Error $_.ErrorDetails.Message
        } else {
            Write-Error $_
        }
    }

The result is that it displays the specific error message:
Example: HTTP 401
PS F:\VMware.CloudFoundation.Reporting> Request-VCFToken -fqdn sfo-vcf01.sfo.rainpole.io -username<EMAIL_ADDRESS>-password ThisIsNotTheCorrectPassword
Request-VCFToken : The remote server returned an error: (401) Unauthorized.
At line:1 char:1
+ Request-VCFToken -fqdn sfo-vcf01.sfo.rainpole.io -username administra ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Request-VCFToken

Better yet, update Request-VCFToken to use the module's esponseException function:
    Catch {
        ResponseException -object $_
    }

Example: HTTP 401
Script File:        Line: 68
Relevant Command:  $response = Invoke-RestMethod -Method POST -Uri $uri -Headers $headers -body $body
Target Uri:         https://sfo-vcf01.sfo.rainpole.io/v1/tokens
Exception Message: The remote server returned an error: (401) Unauthorized.


Incorrect .NET code for quickstart Azure Open AI

var chatCompletionsOptions = new ChatCompletionsOptions()
{
    DeploymentName = "gpt-35-turbo", //This must match the custom deployment name you chose for your model
    Messages =
    {
        new ChatMessage(ChatRole.System, "You are a helpful assistant."),
        new ChatMessage(ChatRole.User, "Does Azure OpenAI support customer managed keys?"),
        new ChatMessage(ChatRole.Assistant, "Yes, customer managed keys are supported by Azure OpenAI."),
        new ChatMessage(ChatRole.User, "Do other Azure AI services support this too?"),
    },
    MaxTokens = 100
};

Response<ChatCompletions> response = client.GetChatCompletions(chatCompletionsOptions);

Code above doesn't compile due to:

DeploymentName no longer exist on options and have move to the constructor

Correct code for version #r "nuget: Azure.AI.OpenAI, 1.0.0-beta.8" is therefore:
var chatCompletionsOptions = new ChatCompletionsOptions()
{
    Messages =
    {
        new ChatMessage(ChatRole.System, "You are a helpful assistant."),
        new ChatMessage(ChatRole.User, "Does Azure OpenAI support customer managed keys?"),
        new ChatMessage(ChatRole.Assistant, "Yes, customer managed keys are supported by Azure OpenAI."),
        new ChatMessage(ChatRole.User, "Do other Azure AI services support this too?"),
    },
    MaxTokens = 100
};

Response<ChatCompletions> response = client.GetChatCompletions("gpt-35-turbo", chatCompletionsOptions);

Also please call out used library version more clearly as there seems to be syntax changes between versions. Thank you.

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

ID: 4833a089-660d-364f-82a4-1fc26522b6ec
Version Independent ID: 2c415b16-f888-a44b-64d7-a9d1732dacb6
Content: Quickstart - Get started using GPT-35-Turbo and GPT-4 with Azure OpenAI Service - Azure OpenAI Service
Content Source: articles/ai-services/openai/chatgpt-quickstart.md
Service: azure-ai-openai
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@softchris
Thanks for your feedback! We will investigate and update as appropriate.

@softchris
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/259399
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


TP 1 Question 1

Hello,
I think I found an error in the solution of part b of the first question of TP1. When I run the code, it returns 1 and 2 instead of 3 and 2 as explained in the solution. Is it possible that the lowest value is pushed first instead of the highest? Thank you in advance!

Hi,
yes, it depends on your implementation of the priority queue.
If its a min-queue it would pop 1 and 2, if its a max-queue it would pop 3 and 2, so both answers are correct given that the question does not say whether the queue should be max or min.
In python, if you used the heapq implementation, it is indeed a min-queue.

Thank you!


Add an AnimInstance subclass with Gameplay Tag Bindings

Lyra has a custom anim instance class that has a Gameplay Tag Binding map (docs) (Lyra Anim Instance source), which enables the ABP to thread-safely update bool fields to match the presence/absence of mapped GameplayTags on the owning pawn's ASC.
Again, I will need this in my game, and I'll contribute this when I'm done making it (no ETA at all)
Cheers !

I already did something akin to this -- but more focused -- in my own project.
That said, I think the better way to do this would be to literally just create an equivalent to GAS's tag bindings, as having a way to have properties set automatically (and in a thread-safe way) based on tags could potentially be useful even outside of animation blueprints.
(Plus, it's about the same amount of work either way, and one way is useful globally as opposed to just in animation blueprints.)

Totally agree !

There's now a pull request against the dev branch which contains the requested feature. :)


remove canX.yaml files and integrate into component specific yaml files

currently can0.yaml is used on the base pc for the base and on the torso pc for the arms/sensorring/torso. If we keep a common can0.yaml we cannot have different CAN configurations for e.g. base and torso.
@ipa-mdl had some suggestion how to restructure

move all params into the driver yaml
but set bus/device from the launch file

solved in #435


Lb tweak search results summary

Related PR: https://github.com/alphagov/digitalmarketplace-buyer-frontend/pull/680
Originally I planned to just add a border to the summary results for the opportunities page but then realised that the search results summary needed more consideration.
This commit:
• adds a border to the bottom of all search summaries on search results pages
• removes a border-top from the save search
• increases the font-size for search results summary and suggestion
• increases font sizes for mobile
Before:

After:

Is 16 big enough?
Maybe something a bit larger to give it some sort of presence.
16:

23:

I suspect at least some of our users have come to know and love the massive numbering. Certainly I remember labs in the past where G-Cloud buyers have talked about their main objective when searching being to get this number low enough. But having things all the same size does look neater. I'd agree with Dilwoar that it should probably be more prominent on the page though - the headline number is definitely the main focus for at least some of our users.
Would be nice for the commit message to have a bit more detail of what has changed here maybe ("improve" is a very subjective term!).


fix: Fix/chart widget use undefined series name if empty

Fixes #25920

/ok-to-test sha=f6b5d3e

/build-deploy-preview env=release

Tested this PR and LGTM


Revise the plugin with the events and handlers actually used in seekBar

For example I think this is missing:

//disable
player.controlBar.progressControl.seekBar.off("touchstart");
player.controlBar.progressControl.seekBar.off("touchend");

//enable
player.controlBar.progressControl.seekBar.on("touchstart", player.controlBar.progressControl.seekBar.handleMouseDown);
player.controlBar.progressControl.seekBar.on("touchend", player.controlBar.progressControl.seekBar.handleMouseUp);

And this handleClick function is not exists in the seekBar:
player.controlBar.progressControl.seekBar.on("click", player.controlBar.progressControl.seekBar.handleClick);

And maybe click event is not even used? Maybe there's more.
5.x and 6.x has no difference in events and handlers in seekBar.

sorry so late on this but if you fixed it, you can submit a PR thanks!


Invalid regular expression: /dtae**090960/

I got this error message from our app in production that has a zxcvbn driven password meter:
Uncaught SyntaxError: Invalid regular expression: /dtae**090960/: Nothing to repeat

We don't have a log of what the user typed to reproduce this error (security seems to frown on logging passwords). What's in the regular expression is not their password (at least we can't repro with that or any portion of it). The final password is 14 characters but we're not sure at what point during typing the error occurred, it might have been after 8 or so.
We don't have a full stack trace, but we're not using regular expressions ourselves, only within zxcvbn, but we realize it seems very odd that this exception would be coming from this library.
Any thoughts?


feat/mutableData: add missing decrypt binding function

Coverage increased (+0.01%) to 85.626% when pulling 52cc6212c3d7335bd52e9aa7df818e567fde0c72 on bochaco:add-decrypt-binding into 2d4814eeae5b86b500e16e542da396f4c60e2c3b on maidsafe:master.

Coverage increased (+0.01%) to 85.626% when pulling 52cc6212c3d7335bd52e9aa7df818e567fde0c72 on bochaco:add-decrypt-binding into 2d4814eeae5b86b500e16e542da396f4c60e2c3b on maidsafe:master.


[Enhancement] Optimize the performance of bitmap_to_array

Why I'm doing:
Use BitmapValue's native toUint64Array function to optimize the performance of bitmap_to_array. It will reduce the number of function dispatches.
What I'm doing:
Optimize the performance of bitmap_to_array through using the native toUint64Array function of BitmapValue.
Before Optimization:

ExprComputeTime: 806.844ms

After Optimization:

ExprComputeTime: 481.050ms

Performance test case:
CREATE TABLE `t1` (
  `c1` int(11) NULL COMMENT "",
  `c2` bitmap BITMAP_UNION NULL COMMENT ""
) ENGINE=OLAP
AGGREGATE KEY(`c1`)
DISTRIBUTED BY HASH(`c1`)
PROPERTIES (
"replication_num" = "1",
"in_memory" = "false",
"enable_persistent_index" = "false",
"replicated_storage" = "true",
"fast_schema_evolution" = "true",
"compression" = "LZ4"
);

insert into t1 select lo_quantity, bitmap_agg(lo_orderkey) from lineorder group by lo_quantity;

select c1, array_length(bitmap_to_array(c2)) from t1

What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-3.2  branch-3.1  branch-3.0  branch-2.5


Adaptive cards not rendered properly on webchat channel. works on directline though..

The above is an example of scuba bot number input.
The same behavior is observed across all adaptive cards of scuba card...
Used the webchat iframe.

      </div>

Tested on Chrome and IE. same behavior.
Using nodejs code.
Also read about the npm microsoft-adaptivecards module. How is that to be used? any documentation anywhere?

@ubreddy can you elaborate on what's not working? I'm not sure I get it based on the screenshot. Also, what is the payload for the card that doesn't render properly?

This is the payload I am sending through bot
session.send(payload)
{
"type": "message",
"inputHint": "expectingInput",
"speak": "How many people will be joining you?  There is a minimum of 3 and a maximum of 6.",
"attachments": [
{
"contentType": "application/vnd.microsoft.card.adaptive",
"content": {
"type": "AdaptiveCard",
"version": "1.0",
"body": [
{
"type": "TextBlock",
"text": "{{destination}}, nice choice!"
},
{
"type": "ColumnSet",
"columns": [
{
"type": "Column",
"items": [
{
"type": "TextBlock",
"text": "How many people will be joining you?"
}
]
}
]
},
{
"type": "Input.Number",
"id": "numberOfPeople",
"placeholder": "Enter a number",
"min": 1,
"max": 10,
"value": 3
}
],
"actions": [
{
"type": "Action.Submit",
"title": "Next",
"data": {
"people": ""
}
}
]
}
}
]
}
The above is just an example. All cards from scuba bot are behaving the same way. Being rendered as images and options below the image.
And for this I am getting the image as shown in the webchat channel (instantiated from iframe) . But it works properly if instantiated from Botchat.js...
Is webchat iframe not supportive of adaptive cards.

Even the same is not working from Skype channel too...
Error: Request to 'https://smba.trafficmanager.net/apis/v3/conversations/29%3A13
TA28m5CkdUwY3QPXUEkfr2qXLipnH09EJ9ZI3xYzpE/activities' failed: [400] Bad Request
at Request._callback (C:\scuba\node_modu

les\botbuilder\lib\bots\ChatConnector.js:523:46)

@ubreddy can you tell me your bot ID?

AS_Skype

Hi @ubreddy - you need to check the "Enable Preview" checkbox in your Web Chat configuration page for that embed code.

did that.
Still doest seem the serve the adaptive cards as cards... they are being shown as images with the actions showing up as options ( non clickable)

Well, when I created a new site it worked. Existing default after enabling preview didnt though. Strange!


Add some precompiles

Together with a couple of changes to LoopVectorization,
this shaves about one second off the initial mygemmavx! demo.
There may be more methods that could be added, but this is a start.
Overall, VectorizationBase is the only substantive source of
inference time in that demo.

Codecov Report

Merging #29 (67105ae) into master (739e614) will decrease coverage by 3.39%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   56.17%   52.77%   -3.40%
==========================================
  Files          28       28
  Lines        2672     2628      -44
==========================================
- Hits         1501     1387     -114
- Misses       1171     1241      +70

Impacted Files
Coverage Δ

src/precompile.jl
92.85% <92.30%> (+42.85%)
:arrow_up:

src/alignment.jl
20.00% <0.00%> (-33.34%)
:arrow_down:

src/llvm_intrin/binary_ops.jl
32.20% <0.00%> (-22.80%)
:arrow_down:

src/llvm_types.jl
70.68% <0.00%> (-12.39%)
:arrow_down:

src/llvm_intrin/conversion.jl
60.34% <0.00%> (-11.33%)
:arrow_down:

src/llvm_intrin/intrin_funcs.jl
59.80% <0.00%> (-4.79%)
:arrow_down:

src/promotion.jl
28.57% <0.00%> (-4.77%)
:arrow_down:

src/llvm_intrin/masks.jl
62.21% <0.00%> (-4.70%)
:arrow_down:

src/llvm_intrin/memory_addr.jl
51.78% <0.00%> (-4.11%)
:arrow_down:

src/vector_width.jl
47.31% <0.00%> (-2.69%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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

For your amusement, here's the compiler-flamegraph that can be produced using the master branch of SnoopCompile:

(The mouse pointer seems to have been mis-captured, it was actually in the middle of the wide blue bar.)
"empty" spaces correspond to times when it's doing something other than inference. The thick stack on the right is what's left of inference needed for the final stage of codegen, mostly in VectorizationBase. As you pointed out on discourse, for LV inference time isn't dominant, mostly because you've done a good job of adding precompiles to LoopVectorization itself.

xref https://github.com/chriselrod/LoopVectorization.jl/pull/180

The remaining inference time, excluding things like
LoopVectorization._avx_!(::Val{(0, 0, 0, 4)}, ::Type{Tuple{:numericconstant, ...

which will never be practical to precompile, is around 1.9s. That's out of a total of ~10.5s on my machine for the entire benchmark, so there's still hope for more improvement though we're definitely into diminishing returns from inference itself.

The "skinny" flames are mostly due to JuliaLang/julia#38983, but in fact they don't add up to much.

Maybe I should run to see what they're specializing on.
LoopVectorization and VectorizationBase make heavy use of ArrayInterface.StaticInt, but this should be creating separate methods for each constant value.
I'm curious which constant-values are seeing lots of specializations. If they're integers (and they must be?), maybe using StaticInts there would at least give them separate entries in the method table to avoid repeated inference on the same values?

I am curious if a different design of internals, like with having type-asserted muladd wrapping _muladd could improve the fairly heavy inference times we're seeing now, but that'd be a bigger change for another PR.

I just tried turning on the vfmadd statement:
W = pick_vector_width(T) # the size we want to compile is type and CPU-dependent
precompile(vfmadd, (Vec{W, T}, Vec{W, T}, Vec{W, T}))  # doesn't "take" (too bad, this is expensive)

and got
julia> using MethodAnalysis, AbstractTrees

julia> using VectorizationBase
[ Info: Precompiling VectorizationBase [3d5dd08c-fd9d-11e8-17fa-ed2836048c2f]

julia> mi = methodinstance(VectorizationBase.vfmadd, (Vec{8,Float64}, Vec{8,Float64}, Vec{8,Float64}))
MethodInstance for VectorizationBase.vfmadd(::Vec{8, Float64}, ::Vec{8, Float64}, ::Vec{8, Float64})

So it seems to have taken?

I just tried turning on the vfmadd statement:
W = pick_vector_width(T) # the size we want to compile is type and CPU-dependent
precompile(vfmadd, (Vec{W, T}, Vec{W, T}, Vec{W, T}))  # doesn't "take" (too bad, this is expensive)

and got
julia> using MethodAnalysis, AbstractTrees

julia> using VectorizationBase
[ Info: Precompiling VectorizationBase [3d5dd08c-fd9d-11e8-17fa-ed2836048c2f]

julia> mi = methodinstance(VectorizationBase.vfmadd, (Vec{8,Float64}, Vec{8,Float64}, Vec{8,Float64}))
MethodInstance for VectorizationBase.vfmadd(::Vec{8, Float64}, ::Vec{8, Float64}, ::Vec{8, Float64})

So it seems to have taken?

Nice! Also check whether it shows up in the flamegraph:
using SnoopCompile
tinf = @snoopi_deep whatever...
fg = flamegraph(tinf)
using ProfileView
ProfileView.view(fg)

and then hover over various stacks, working your way in the call chain to the point where it might appear.

Nice! Also check whether it shows up in the flamegraph:
using SnoopCompile
tinf = @snoopi_deep whatever...
fg = flamegraph(tinf)
using ProfileView
ProfileView.view(fg)

and then hover over various stacks, working your way in the call chain to the point where it might appear.

VectorizationBase's test suite doesn't seem to be a good choice, as it's dominated by broadcasts. I should probably try replacing them with masks just for the sake of speeding up the test suite.
The more important change for users, however, is that llvmcall functions shouldn't overload methods from other libraries (e.g. Base), and then define these overloads as calls to the VectorizationBase functions. That should increase the amount of inference results getting saved.

VectorizationBase's test suite doesn't seem to be a good choice, as it's dominated by broadcasts. I should probably try replacing them with masks just for the sake of speeding up the test suite.
The more important change for users, however, is that llvmcall functions shouldn't overload methods from other libraries (e.g. Base), and then define these overloads as calls to the VectorizationBase functions. That should increase the amount of inference results getting saved.

Yeah, I often find it's nice to start with a "single thing" (e.g., a single @avxed function) and explore that. Test suites are a bit messy for this kind of analysis because they have their own costs. That said, once you get the hang of this, they can be quite informative.

Yeah, I often find it's nice to start with a "single thing" (e.g., a single @avxed function) and explore that. Test suites are a bit messy for this kind of analysis because they have their own costs. That said, once you get the hang of this, they can be quite informative.