HDDS-6666. Make Hugo markdown image syntax add responsive image class

What changes were proposed in this pull request?
This should solve the image size issue (some are too large, like in PrefixFSO.md) when using the Markdown image syntax (not the shortcode).
Also see comment: https://github.com/apache/ozone/pull/2582/files#r861196807
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-6666
How was this patch tested?

Check the generate doc html manually, images should now fit the page text width

cd hadoop-hdds/docs
hugo serve &
open http://localhost:1313/feature/prefixfso.html

@JyotinderSingh can you please take a look, too?

Hi @smengcl thank you for the patch, just to make sure I understand the functionality correctly - the change you have added automatically substitutes every Markdown img tag with the custom code <img src="{{ .Destination | safeURL }}" alt='{{ .Text }}' class="img-responsive"/>? I believe it's called a render hook, could you please confirm if I am understanding it properly?
Adding to what @adoroszlai said, currently, the only reason the website does not have the image sizing fix added is that we are skipping the logic added in #3122 in the deployment process because our Jenkins server does not have Maven installed. For the image sizing logic to work correctly, we should use the mvn clean install command to generate the website.
Looking at the logic that you have added, I think it completely resolves the issue in my approach (having to generate site using mvn clean install) and is a more elegant and understandable solution. I think you should be able to safely remove the the changes linked below in hadoop-hdds/docs/dev-support/bin/generate-site.sh I made as a part of #3122.
https://github.com/apache/ozone/pull/3122/files#diff-d9443550225fa81eaea96777ca3c5f615fff136ac1d2fd58dc789e2a458107fcR34-R46

Thanks @JyotinderSingh for the comment. Yes it is a render hook.
I can push a commit to remove the make_images_responsive.py.

I can push a commit to remove the make_images_responsive.py related changes.

@smengcl Yes, please, let's do that.

Thanks @smengcl for the patch, @JyotinderSingh for the review.

Thanks @smengcl @JyotinderSingh for the review!


[SILGen] Fix handling of dynamic Self in key paths (SR-12428)

emitKeyPath* did not handle key paths involving covariant Self types (where either the root or a computed property is of type Self); we would just emit key path instructions involving @dynamic_self types. This led to various crashes at both compile-time and at runtime, since the rest of the compiler and standard library do not expect the dynamic Self type to show up in key paths. The type system does not make it possible to declare a key path whose leaf type is a dynamic Self, so it seems sensible to just get rid of dynamic self types in SILGen.
Handling key paths where the root is a dynamic Self is easy: if the root type is @dynamic_self Foo, then the key path can only access properties on Foo -- the dynamic Self attribute has no effect and can simply be removed (in visitKeyPathExpr).
Key paths with a computed property returning Self is only slightly more complicated. If we have a dynamic Self property on class Foo, and we access that property through a key path on an object of class Bar: Foo, then we need to downcast the getter result to Bar before we can remove the dynamic Self attribute. I've done that by detecting that case in emitKeyPathComponentForDecl and passing an extra parameter to getOrCreateKeyPathGetter informing it that we want the key path getter to downcast the result before returning it. getOrCreateKeyPathGetter was already using emitSubstToOrigValue to reabstract the result; changing the call to the more general emitTransformedValue allows us to easily downcast the result as well.
I have also added a test case and SILVerifier checks that catch all the crashes I am aware of.

Resolves SR-12428.

Sorry for the late response. I'm closing this PR for now, as I don't currently understand the Swift compiler internals well enough to implement this properly. Maybe I'll circle back to it at some point in the future, but I don't expect to have enough spare time (between work, school, and COVID) to dive in anytime soon.
Thank you for your help, and sorry for taking up your time.

No need to be sorry. Thanks for taking a look into it!


tabiew: 0.3.5 -> 0.4.1

meta.homepage for tabiew is: https://github.com/shshemi/tabiew
meta.changelog for tabiew is: https://github.com/shshemi/tabiew/releases/tag/v0.4.1
To inspect upstream changes
 * [Release on GitHub](https://github.com/shshemi/tabiew/releases/tag/v0.4.1)

 * [Compare changes on GitHub](https://github.com/shshemi/tabiew/compare/v0.3.5...v0.4.1)

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4163

please don't delete the PR template in the future


Wear OS not loading

Hi,
I installed Home Slide on my OnePlus 6, and my Fossil Gen 5 but when I open the app on Wear OS, it goes into an infinite loading screen.

Hi, thanks for the feedback!
I'm trying to fix this issue, you can check issue #338 for a workaround that might work for you.


Fix issue #1312

This little patch fixes the TypeError: Cannot read properties of null (reading 'previousElementSibling') in core.removeNode().

Thank you for your contribution!
I'll notify you after version update


Create Tab UI

Still some ways to go; Here's how it looks so far:

What's left (updated):

background of the container
🎛 filter by "day" (eg. Day 1, Day 2, etc.) & 🔨  filter by Category

Closes #20

Update: https://styled-components.com/docs/advanced
Looks like it can take in props and use them directly within the template string. Let's see how this goes. I might just end up putting the colors in the theme file for global use.

Here's what I've got so far. I can change it to light grey if we want but I experimented with light blue. Thoughts? Removed the "Getting Started" section since I was getting some duplication errors. Otherwise, I just need to change the purple color but idk what blue is in the photo. Should I just use Variable blue values?

Here's the updated version with the light grey. Personally, I think the light blue looks better though.


Cannot configure input devices through hyprctl

Hyprland Version
 System/Version info
Hyprland, built from branch HEAD at commit c5e28ebcfe00a510922779b2c568cfa52a317445  (props: bump ver 0.37.1).
Date: Sat Mar 16 14:51:49 2024
Tag: v0.37.1

flags: (if any)

System Information:
System name: Linux
Node name: joe
Release: 6.8.1-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Sat, 16 Mar 2024 17:15:35 +0000

GPU information:
0000:00:02.0 VGA compatible controller [0300]: Intel Corporation Alder Lake-P GT1 [UHD Graphics] [8086:46a3] (rev 0c) (prog-if 00 [VGA controller])
0000:01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GA106M [GeForce RTX 3060 Mobile / Max-Q] [10de:2520] (rev a1) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Bug or Regression?
Regression
Description
There seems to be no way to configure individual input device settings via hyprctl after 1.37. Before the update, running the command hyprctl keyword device:<device name>:enabled 0 would disable the device. Now, it does not. This is because there is no way to specify the device name anymore.
How to reproduce
Run the command hyprctl keyword device:<device name>:enabled 0, with the device name taken from the output of hyprctl devices. On my machine, I would run hyprctl keyword device:asup1205:00-093a:2003-touchpad:enabled 0 and I would get the error config option <device:asup1205:00-093a:2003-touchpad:enabled> does not exist.
Crash reports, logs, images, videos

Same here, 1.37 breaks screen rotation which is handled by iio-hyprland.  iio-hyprland uses the syntax above to apply device transformations of input devices (touchpad, touchscreen).

#4856

tl;dr you can't device:name:prop anymore.

in v0.37, I can use hyprctl keyword device:name=<NAME>:enabled false or getoption, but after updated to 0.37.1 I cannot do it anymore

even when I try to create a variable in config file $touchpadEnabled, I cannot flip it using hyprctl keyword either or getoption.
config & output

tl;dr you can't device:name:prop anymore.

So what DO you run instead?

@vaxerski is there a new way to configure per-device settings through hyprctl?

$amongus = true
device {
    name = sus
   enabled = $amongus
}

hyprctl keyword '$amongus' false

(note the single quote so that bash doesnt parse it)
option 2 is putting the above snippet with your desired values in a .conf file and sourceing it

This does not work. I have this section in my config file:
$touchpad_enabled = 1
device {
    name = asup1205:00-093a:2003-touchpad
    enabled = $touchpad_enabled
}

When I run hyprctl keyword '$touchpad_enabled' 0, it changes nothing. I have tried both integers and boolean types.
I do not want to enable automatic config reloading and use naive file sourcing, because this feature should ideally exist. Could you please add back a way to configure device settings, @vaxerski?

what's your hyprlang version? Strings with : were broken before like... 0.4.1 ish

 ~  pacman -Qi hyprlang
Name            : hyprlang
Version         : 0.5.0-1

0.5.0 from arch repos

It seems like arch got a bunch of hypr.* updates pushed out all at once, so that's why I'm bringing all this up now.

at any rate, changes to keyword to accept keys of categories is something to request over at hyprlang, not here

Hmm this sucks a bit. I also used this logic to toggle my touchpad:
#toggle touchpad trackpad
bind=SUPER,T,exec,hyprctl keyword "device:bcm5974:enabled" false
bind=SUPERSHIFT,T,exec,hyprctl keyword "device:bcm5974:enabled" true

Let's see what https://github.com/hyprwm/hyprlang/issues/35 brings us.


Use GZipMiddleware instead of using zipfile

@romifz Is there a reason package-releases creates a ZIP file? If it's just to make the download size smaller, we can use GZipMiddleware.

Sounds good, we can do that.

Although in the other hand, plain json/csv files can get quite large and there is no good reason to save them that way...

True, plus we need to ZIP the output if there are multiple files in the output. I figure it's better to be consistent, and some recent OSes automatically UNZIP files anyway. So, I'll close this.


[Sprig App] Sprong

Author name
Author: Licnex
About your game
What is your game about?
Its classic pong where you have to protect your side from the goal. There are two game modes, single player and multiplayer.
How do you play your game?
press w and s for player 1 (the one on the left side) and i and k for the right hand side

This looks great! The only bug I notice is that if I hit A or L after the game is running, I get an error. If you fix that I'll be happy to approve!

Thanks, Fixed it, do you know when the hackclub guys will reach out to me for the sprig shipment details?


Make documentation consistent with other plugins

Example: https://github.com/jenkinsci/google-kubernetes-engine-plugin/blob/develop/docs/Home.md#iam-credentials
Also good to reference: https://cloud.google.com/iam/docs/creating-managing-service-accounts

On that note, the documentation says 'for how to consume this plugin go look at our cloud storage plugin'
https://github.com/jenkinsci/google-oauth-plugin/blob/develop/docs/home.md
But there's not example there of how they consume it. Unless maybe you are familiar enough with the plugin directory structure to know where they're actually accessing the credentials.

Thanks for pointing that out, we'll definitely be updating this part of the docs as well.
As for what it's referring to there,  the place it is consumed is in UploadModule. The two types of GoogleRobotCredentials that this plugin provides (GoogleRobotPrivateKeyCredentials and GoogleRobotMetadataCredentials) are adapted to work with the Jenkins credentials store, but underlying them is a Credential which is used by most Google client libraries. In this example, it's creating a Storage client which can be used for authenticated API requests.
Soon we'll be changing how this is used by incorporating those parts of the google-storage-plugin into the gcp-client module of gcp-plugin-core-java. All of our clients (modified for convenient usage with our plugins) will be created here using the Credential as a HttpRequestInitializer. For examples of that usage you can see our GKE plugin here.


Error with requirements file.

This error happend when install requirements file
pip install -r requirements.txt
Error: pip install -r requirements.txt
Collecting logging
Downloading logging-<IP_ADDRESS>.tar.gz (96 kB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━ 96.0/96.0 kB 2.1 MB/s eta 0:00:00
Preparing metadata (setup.py) ... error
error: subprocess-exited-with-error
× python setup.py egg_info did not run successfully.
│ exit code: 1
╰─> [24 lines of output]
Traceback (most recent call last):
File "", line 2, in
File "", line 14, in
File "/usr/lib/python3/dist-packages/setuptools/init.py", line 8, in
import _distutils_hack.override  # noqa: F401
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/_distutils_hack/override.py", line 1, in
import('_distutils_hack').do_override()
File "/usr/lib/python3/dist-packages/_distutils_hack/init.py", line 77, in do_override
ensure_local_distutils()
File "/usr/lib/python3/dist-packages/_distutils_hack/init.py", line 63, in ensure_local_distutils
core = importlib.import_module('distutils.core')
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3/dist-packages/setuptools/_distutils/core.py", line 22, in
from .dist import Distribution
File "/usr/lib/python3/dist-packages/setuptools/_distutils/dist.py", line 12, in
import logging
File "/tmp/pip-install-m23ylqv5/logging_d7225d883f274ec38a8f6aa1fc3ed180/logging/init.py", line 618
raise NotImplementedError, 'emit must be implemented '
^
SyntaxError: invalid syntax
[end of output]
note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed
× Encountered error while generating package metadata.
╰─> See above for output.
note: This is an issue with the package mentioned above, not pip.
hint: See above for details.
I think this error because setup.py  file not included in tool directory,And you put too files for tool egy.py & egyscan.py,this tool need some improvement.

Thank You For Your Kind interest into improving Egyscan
First of all Lets Try To Fix this issue
have you tried
pip install logging
?
if yes and still no progress i suggest you start downloading the logging library manually and install it on your machine
https://pypi.org/project/logging/
Steps
download
extract contents into folder
open cmd or terminal what ever os you are using then
run
python setup.py install
then
restart the application
python egyscan.py
python3 egyscan.py
depending on your os system
And Keep me updated via email<EMAIL_ADDRESS>


Test CLA Commit

This is a dummy Pull request to test the workflow of the CLA.

Pull Request Functional Test Report for #2127 / c1f0f7a1b02b7384b31f226169984846642f0ed6
No tests identified for validation.
Help


changing prognostic variables to rho_dry and T

PULL REQUEST
Purpose and Content
We want to compare our simulations with PySDM simulations. In PySDM simulations prognostic variables are density of the dry air, rho_dry, and temperature T. This pull request makes these variables prognostic. The liquid ice potential temperature,  theta_liq_ice, and density of moist air, rho, which were previously prognostic variables are now derived.
Benefits and Risks
With the new change of the code, temperature profiles remain constant and below what they were in previous simulations. Consequently, we get more liquid and rain. The new results compare better with PySDM simulations. As a result calibration of microphysics parameters with PySDM data can be done meaningfully.
Linked Issues
None
PR Checklist

This PR has a corresponding issue OR is linked to an SDI.
I have followed CliMA's codebase contribution and style guidelines OR N/A.
I have followed CliMA's documentation policy.
I have checked all issues and PRs and I certify that this PR does not duplicate an open PR.
I linted my code on my local machine prior to submission.
Unit tests are included.
Code used in an integration test.
All tests ran successfully on my local machine.
All classes, modules, and function contain docstrings.
[N/A] Documentation has been added/updated.

Looks great! I'm thinking there is probably a better way to refactor our equation choices in the tendency file. But we can think about it later. Maybe before adding the 2-moment microphysics functionality.
Could you also add the new options to the buildkite pipeline, so that the simulations actually get run on the cluster CI? After that, lets squash and merge.

bors r+


Add Splash Screen

Closes #48
What changes are introduced to solve this?

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ]  Bugfix
[x]  Feature
[ ]  Code style update
[ ]  Refactoring
[ ]  Documentation content changes
[ ]  Other… Please describe:

How to Test the Code?

Describe:

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other Information :

Your previous changes are also included in this PR. Please remove these changes. This doesn't happen if you make a separate branch for each issue you took. Well, take care next time!  Keep Going you are doing nice!!

@harchani-ritik Status of this PR ?


Unable to get Controller, Scheduler and Metrics dashboards populated

Note: If you have a feature request, you should contact support so the request can be properly tracked.
Output of the info page
I am testing this integration with within minikube this will be configuration will be migrated to our staging infrastructure on AWS in an EKS cluster once it becomes available. What is happening is the following:
➜ kubectl exec datadog-znr26 agent status
...
    kube_metrics_server (1.0.0)
    ---------------------------
      Instance ID: kube_metrics_server:973042caf095d7a8 [ERROR]
      Total Runs: 34
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 1, Total: 34
      Average Execution Time : 19ms
      Error: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f986018d410>: Failed to establish a new connection: [Errno 111] Connection refused',))
      Traceback (most recent call last):
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/base.py", line 503, in run
          self.check(instance)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/kube_metrics_server/kube_metrics_server.py", line 104, in check
          self.process(scraper_config)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/openmetrics/mixins.py", line 330, in process
          for metric in self.scrape_metrics(scraper_config):
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/openmetrics/mixins.py", line 297, in scrape_metrics
          response = self.poll(scraper_config)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/openmetrics/mixins.py", line 479, in poll
          response = self.send_request(endpoint, scraper_config, headers)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/datadog_checks/base/checks/openmetrics/mixins.py", line 542, in send_request
          auth=auth,
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 75, in get
          return request('get', url, params=params, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/api.py", line 60, in request
          return session.request(method=method, url=url, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 533, in request
          resp = self.send(prep, **send_kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/sessions.py", line 646, in send
          r = adapter.send(request, **kwargs)
        File "/opt/datadog-agent/embedded/lib/python2.7/site-packages/requests/adapters.py", line 516, in send
          raise ConnectionError(e, request=request)
      ConnectionError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443): Max retries exceeded with url: /metrics (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7f986018d410>: Failed to establish a new connection: [Errno 111] Connection refused',))
...

Additional environment details (Operating System, Cloud provider, etc):
➜ minikube status
host: Running
kubelet: Running
apiserver: Running
kubectl: Correctly Configured: pointing to minikube-vm at <IP_ADDRESS>
...
datadog-values.yaml
## Default values for Datadog Agent
## See Datadog helm documentation to learn more:
## https://docs.datadoghq.com/agent/kubernetes/helm/
image:
  repository: datadog/agent
  tag: 6.13.0
  pullPolicy: Always

datadog:
  apiKeyExistingSecret: datadog-secret
  appKeyExistingSecret: datadog-app-secret

  logLevel: DEBUG

  tags:
    - name: dev.webapp
    - env: dev
    - purpose: webapp

  apmEnabled: true
  nonLocalTraffic: true

  collectEvents: true
  logsEnabled: true

  logsConfigContainerCollectAll: true

  confd:
    kubernetes_state.yaml: |-
      cluster_check: true
      ad_identifiers:
        - kube-state-metrics
      init_config:
      instances:
        - kube_state_url: http://%%host%%:8080/metrics
    kube_controller_manager.yaml: |-
      init_config:
      instances:
        - prometheus_url: http://<IP_ADDRESS>:10252/metrics
    kube_scheduler.yaml: |-
      init_config:
      instances:
        - prometheus_url: http://<IP_ADDRESS>:10251/metrics
    kube_metrics_server.yaml: |-
      init_config:
      instances:
        - prometheus_url: https://<IP_ADDRESS>:443/metrics
  env:
    - name: DD_LOGS_INJECTION
      value: "true"
    - name: DD_USE_DOGSTATSD
      value: "true"
    - name: DD_LOG_FORMAT_JSON
      value: "true"
    - name: DD_APP_KEY
      value: "===REDACTED==="
    - name: DD_APM_ENABLED
      value: "true"
    - name: DD_EXTRA_CONFIG_PROVIDERS
      value: "clusterchecks"

clusterAgent:
  enabled: true
  token: "===REDACTED==="

  metricsProvider:
    enabled: true

  clusterChecks:
    enabled: true

daemonset:
  useHostPort: true

Steps to reproduce the issue:

Install helm in minikube
Check IP for etcd, scheduler, or apiserver for k8s: kubectl get po -o wide --all-namespaces
Replace IP in datadog-values.yaml where IP <IP_ADDRESS> is with that of etcd pod
Install the datadog chart: helm install -f datadog-values.yaml --name datadog stable/datadog
Run agent test once pods are up & running: `kubectl exec  agent status

Describe the results you received:
Describe the results you expected:
A successful response similiar to:
    kube_scheduler (1.1.0)
    ----------------------
      Instance ID: kube_scheduler:287d9c0cb137ebcf [OK]
      Total Runs: 7
      Metric Samples: Last Run: 338, Total: 2,366
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 2, Total: 14
      Average Execution Time : 211ms

Additional information you deem important (e.g. issue happens only occasionally):

@taneipoo thanks for filing an issue. It looks like we'll need additional information to assist. Would you mind opening a support ticket so we can continue? Thanks.
https://www.datadoghq.com/support/

thank @jeremy-lq I will do so


Update http-accept dependency to 2.1

Changelog between http-accept v1.7 and latest v2.1 https://github.com/socketry/http-accept/compare/v1.7.0...v2.1.0

Thanks for the PR. While it would be good to support the latest version, it looks like http-accept 2.1.0 implicitly requires ruby >= 2.3.0, so will have to think about how we want to handle that.
https://github.com/socketry/http-accept/commit/d002833f96a65900f4bc5d0782e00d8bdc88283d#r34811158

hm maybe restrict Ruby to >= 2.4 and release some major version of gem like 2.2.0 or 3.0.0? Version 2.3 is already EOL since 2019-03-31
This also requires some small refactor i guess, like here:
https://github.com/rest-client/rest-client/blob/f450a0f086f1cd1049abbef2a2c66166a1a9ba71/lib/restclient/utils.rb#L29

Please raise it to the latest version, or even better, make it less restrictive (see https://github.com/rest-client/rest-client/pull/724).

@pvalena updated to < 2.2

Thanks!

Any updates on this? This actually does not appear to break the testsuite, the failure is caused by a renamed rubocop rule.

@dcermak there is PR with fix for this rubocop issue https://github.com/rest-client/rest-client/pull/728 . If this fix gets merged i will update my PR so testsuite will pass.

Hi there! Any news?

Hi there! Any news?

Hi there! Any news?

ping @ab

Hi there! Any news?

i've bumped and locked down rubocop version so CI is passing again. However there are some issues with rubygems on jruby-<IP_ADDRESS>

tempus fugit

ping @mattmanning  @L2G

ping @mattmanning  @L2G

ping @ioquatix @deivid-rodriguez

That's very rude @n-rodriguez, are you just pinging random people?

That's very rude @n-rodriguez, are you just pinging random people?

 I don't know if you as a person can do something, but I'm sure that us as a community can do something.
rest-client is a well known and used Ruby gem (177 millions of downloads) and IMHO is lacking a lot of :heart: from the community.

Ok @n-rodriguez, no problem.

Please mind your language, there's humans with lives of their own on the
other side. Yes it would be nice to get an update, but we don't know the
full story here. So please try to be more considerate next time.

Please mind your language, there's humans with lives of their own on the other side.

lol really?

Yes it would be nice to get an update, but we don't know the full story here. So please try to be more considerate next time.

I've been considerating for more than a year so yeah it's time to have news

Is there any news on this? What needs to happen to get this merged and released? Thanks in advance

Is there any news on this? What needs to happen to get this merged and released? Thanks in advance

I think it is pretty obvious that the maintainers of this gem have moved on and are no longer maintaining it. So the answer is probably: someone needs to take over maintenance. Before that, nothing will happen.

So the answer is probably: someone needs to take over maintenance. Before that, nothing will happen.

https://github.com/jbox-web/rest-client


using Stipple creates a session folder in the current directory

Currently Stipple creates a folder sessions in the current working directory when it is used or imported.
This behaviour is probably unexpected by users.
I realised this when I accidentially started Stipple for testing in C:\Windows\System32 where I didn't have write access.
Not sure how this can effectively be handled, maybe trigger folder creation upon first client access?

I reported this to Adrian as well

This has been solved with the creation of sessions as a temp folder by default.


does it work with alfred3

i tried to use it with alfred3 but it does not show the smart folders

Added in version 2.
In future, please do not spam me with email and the Alfred forum with threads. A GitHub issue is quite sufficient.


Pretrained model does not work for me (ddrnet23 slim)

@ydhongHIT I downloaded pretrained model for ddrnet23(cityscapes). I want to try this model on two images. As you see in below code, I resized images to (1024,2048,3) like mentioned in paper. The get_seg_model defined in somewhere else but I did not copy it to here.
import matplotlib.pyplot as plt
import cv2
import torch
import numpy as np
img = cv2.imread('E:/deneme_alani_iha/uavid_data/npy_deneme/ddrNet_city/ss.png')
img2 = cv2.imread('E:/deneme_alani_iha/uavid_data/npy_deneme/ddrNet_city/ss2.png')
img = cv2.resize(img,(2048,1024))
img2 = cv2.resize(img2,(2048,1024))
img = img.reshape(3,2048,1024)
img2 = img2.reshape(3,2048,1024)
data = [img,img2]
data = np.array(data)
data = torch.Tensor(data)

#%%
model = get_seg_model(1)
with torch.no_grad():
    output = model(data)
#%%
plt.imshow(output[0].cpu().detach().numpy().reshape(128,256,19)[:,:,0])
#%%
def eight2twoconverter(labels, size):
    colors = [  (150,120,90), (153,153,153), (153,153,153), (250,170,30), (220,220,0), (107,142,35), (152,251,152), ( 70,130,180), (220,20,60), (255,0,0), (  0,0,142), (  0,0,70), (  0,60,100), (  0,0,90), (  0,0,110), (  0,80,100), (  0,0,230), (119,11,32), (  0,0,142)]

    b = np.zeros(( size, 256,3))
    for i in range(b.shape[0]):
        for j in range(b.shape[1]):
            indexx = np.argmax(labels[i,j,:])
            b[i,j]  = colors[indexx]
    return b
#%%
out = eight2twoconverter(output[0].cpu().detach().numpy().reshape(128,256,19), 128)
plt.imshow(out)

I gave correct path and num_classes parameter to this function.
def DualResNet_imagenet(pretrained=True):
    model = DualResNet(BasicBlock, [2, 2, 2, 2], num_classes=19, planes=32, spp_planes=128, head_planes=64, augment=False)
    if pretrained:
        checkpoint = torch.load("E:/deneme_alani_iha/uavid_data/npy_deneme/best_val_smaller.pth", map_location='cpu')

        new_state_dict = OrderedDict()
        for k, v in checkpoint['state_dict'].items():
            name = k[7:]
            new_state_dict[name] = v
        model_dict.update(new_state_dict)
        model.load_state_dict(model_dict)

        model.load_state_dict(new_state_dict, strict = False)
    return model

Raw Image

Output
As you see above output of the network is some thing like a random picture. How can I solve it?

@ydhongHIT I downloaded pretrained model for ddrnet23(cityscapes). I want to try this model on two images. As you see in below code, I resized images to (1024,2048,3) like mentioned in paper. The get_seg_model defined in somewhere else but I did not copy it to here.
import matplotlib.pyplot as plt
import cv2
import torch
import numpy as np
img = cv2.imread('E:/deneme_alani_iha/uavid_data/npy_deneme/ddrNet_city/ss.png')
img2 = cv2.imread('E:/deneme_alani_iha/uavid_data/npy_deneme/ddrNet_city/ss2.png')
img = cv2.resize(img,(2048,1024))
img2 = cv2.resize(img2,(2048,1024))
img = img.reshape(3,2048,1024)
img2 = img2.reshape(3,2048,1024)
data = [img,img2]
data = np.array(data)
data = torch.Tensor(data)

#%%
model = get_seg_model(1)
with torch.no_grad():
    output = model(data)
#%%
plt.imshow(output[0].cpu().detach().numpy().reshape(128,256,19)[:,:,0])
#%%
def eight2twoconverter(labels, size):
    colors = [  (150,120,90), (153,153,153), (153,153,153), (250,170,30), (220,220,0), (107,142,35), (152,251,152), ( 70,130,180), (220,20,60), (255,0,0), (  0,0,142), (  0,0,70), (  0,60,100), (  0,0,90), (  0,0,110), (  0,80,100), (  0,0,230), (119,11,32), (  0,0,142)]

    b = np.zeros(( size, 256,3))
    for i in range(b.shape[0]):
        for j in range(b.shape[1]):
            indexx = np.argmax(labels[i,j,:])
            b[i,j]  = colors[indexx]
    return b
#%%
out = eight2twoconverter(output[0].cpu().detach().numpy().reshape(128,256,19), 128)
plt.imshow(out)

I gave correct path and num_classes parameter to this function.
def DualResNet_imagenet(pretrained=True):
    model = DualResNet(BasicBlock, [2, 2, 2, 2], num_classes=19, planes=32, spp_planes=128, head_planes=64, augment=False)
    if pretrained:
        checkpoint = torch.load("E:/deneme_alani_iha/uavid_data/npy_deneme/best_val_smaller.pth", map_location='cpu')

        new_state_dict = OrderedDict()
        for k, v in checkpoint['state_dict'].items():
            name = k[7:]
            new_state_dict[name] = v
        model_dict.update(new_state_dict)
        model.load_state_dict(model_dict)

        model.load_state_dict(new_state_dict, strict = False)
    return model

Raw Image

Output
As you see above output of the network is some thing like a random picture. How can I solve it?

Are you get the correct val iou? You can use  'HRNet-Semantic-Segmentation-pytorch-v1.1' to evaluate and visualize the results.

I'm pretty sure you're looking at the network activations instead of the actual segmentation map

I'm pretty sure you're looking at the network activations instead of the actual segmentation map

Yes, the problem was inside my converter function.

@dasmehdix  Could you provide the corrected converter function!! Thanks!!

@dasmehdix Could you provide the corrected converter function!! Thanks!!

I was using "numpy.reshape" method to change axis on images. I realized that this method change some values inside array. So, I used "numpy.moveaxis" method. My problem solved.
I generally use this line for visualize outputs(with matplotlib):
plt.imshow(outputs[0].argmax(0).cpu())

@dasmehdix Could you share the code as you did above, I am not able to get the required output

Excuse me, have you solved this problem now? I also want to do visualization. Can you share your visualization code


[Android] selection works incorrect and crashes in TextInput component

React Native version:
System:
    OS: Windows 10
    CPU: (8) x64 Intel(R) Core(TM) i5-8350U CPU @ 1.70GHz
    Memory: 6.59 GB / 15.38 GB
  Binaries:
    Node: 8.11.1 - C:\Program Files\nodejs\node.EXE
    npm: 5.6.0 - C:\Program Files\nodejs\npm.CMD
    Watchman: 4.9.0 - C:\Users\Ilya_Staver\Desktop\AMIClient\watchman.EXE
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-183.6<IP_ADDRESS>92245

      react: 16.8.6
      react-native: 0.60.4

Steps To Reproduce

Create TextInput component on Android with onSelectionChange handler and selection prop
Try to type some letters.

Describe what you expected to happen:
TextInput should correct update selection
Snack, code example, screenshot, or link to a repository:
   onSelectionChange = ({ nativeEvent: { selection } }) => this.setState({ selection });

   <TextInput
                    ref={ref => this.inputRef = ref}
                    style={styles.messageInput}
                    maxLength={maxMessageLength}
                    multiline
                    blurOnSubmit={false}
                    onSelectionChange={this.onSelectionChange}
                    selection={this.state.selection}
                    selectionColor={blueBase}
                    onChangeText={this.onChangeText}
                    value={this.state.message}
                    autoCapitalize="none"
                    underlineColorAndroid={transparent}
                    placeholder={this.getPlaceholderMessage()}
                    placeholderTextColor={grayDarkTransparent(0.35)}
                    editable={!this.state.isVoiceRecording}
                />

The same bug happens in 0.60.5

any update?

Issue still reproduced in React Native 0.61.2

Can anyone from the maintainers investigate this?@react-native-bot

same issue in 0.61.2

Still reproduced in 0.61.5


[Settings > Screen ruler] Fix disabled state of infobar

Summary of the Pull Request
The old implementation doesn't hide if the Toy is disabled. (We discussed this in https://github.com/microsoft/PowerToys/pull/20151#discussion_r958050037!) And the tabstop property isn't set correctly.
Before change:

After change:

PR Checklist

[x] Closes: https://github.com/microsoft/PowerToys/pull/20151#discussion_r958050037
[x] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed
Build local debug build and try behavior.

If you are okay with this change feel free to merge. 🚀


fix: avoid repeatedly closing the std handle

To avoid repeatedly closing the std handle, which could result in unexpected behavior for instances that have already obtained the handle.
在引入其他框架时，为避免重复关闭std handle，需要增加开关。

Please use Worker:: $daimonize. If you still cannot meet the requirements, you can try extending Worker with code similar to the following.
请使用Worker::$daemonize，如果仍然无法满足需求，可以尝试扩展Worker类似如下的代码.
class MyWorker extends Worker {

}


Suicidal dragons

Minecraft 1.10.2
Forge <IP_ADDRESS>.54
DragonMounts r46
When moving about and flying to a higher ledge the dragons will commonly hit the side of the ledge, grind against it for a few seconds, and then give up and float to the ground. Normally this wouldn't be as much an issue, but I'm playing on a skyblock map, so this means they commonly commit suicide.

This may be the same problem as Issue #35.


Feature request: add JoinEagerAlgorithm equivalent for loadRelated

A common use case I keep stumbling upon is having allot of 1-1 relations in "large" quantities loaded eagerly. For example, query an account and all its users, and some user meta table.
Account <== User <-- Meta
Instinctively I'd like to avoid throwing a 3-4k whereInComposite query at Postgres by doing:
const account = await Account
  .query()
  .where(...)
await account
  .$loadRelated('[users.meta]')
  .eagerAlgorithm(Account.JoinEagerAlgorithm)

There's simple workaround today by splitting the request e.g.
// pseudo, untested
const account = await Account
  .query()
  .where(...)
account.users = await User
  .query()
  .where('account_id', account.id)
  .eager('meta')
  .eagerAlgorithm(User.JoinEagerAlgorithm)

Disclaimer; I understand the example "table design" is stupid

Good idea. I can't promise anything before I see if there was a reason I didn't do this in the first place.
Regarding your example, why don't you just do this:
const accounts = await Account
  .query()
  .eager('users.meta')
  .eagerAlgorithm(Account.JoinEagerAlgorithm)

?

@koskimas Micro optimisation was the initial reason, really. The example is not real-world.
Consider having a larger amount of users rows, I'd love to avoid duplication thrown from account table for each user, it's the same for each.

I keep stumbling on the following use case:
const user = await User.forge().where(...)
...
if (someCondition) {
  user.$loadRelated(...) // Here's where I'd like to manipulate eager algorithm.
}

You wouldn't believe how difficult this would be to implement 😓 It involves creating a hybrid of WhereInEagerAlgorithm and JoinEagerAlgorithm because the first level is already fetched. I tried to implement such an algorithm because it would be really great from performance perspective also: it could fetch one-to-one relations using joins recursively and all other relations using WHERE INs. It would be the best from both worlds.
Maybe I'll attack that at some point, but not for a while.

Understood.
In my specific case, I found that QueryBuilder.joinRelated (and friends) in combination with a little forethought did the trick. E.g. I preferred to JOIN an extra or two tables instead of conditioning it down the road. I think my real problem is the weird schema design.


onSubmit/validate fatal errors are hidden from console / observability tooling

If I provide an action like this to the HeadlessForm onError argument.
@action async onSubmit() {
  throw new Error("this doesn't bubble up")
}

I would expect the failure to bubble out of the ember app to my observability/logging tools.
However, this isn't the case.
I'm not sure if this is by design.
Ember headless forms wraps the validate and onSubmit callbacks in TrackedAsyncData from ember-async-data.
This approach makes sense for being able to present expected errors in templates.
But what about unexpected failures / bugs that you want to be seen in devTools / observability tools?
ember-async-data readme comes with this statement:

Handling errors is slightly more complicated: TrackedAsyncData “re-throws” the promises it works with when they have errors, to avoid silently swallowing them in a way that prevents you from using them with logging infrastructure or otherwise dealing with them in your app’s infrastructure. However, this means you must also account for them in your testing

This test verifies the submission failure state works:
https://github.com/CrowdStrike/ember-headless-form/blob/main/test-app/tests/integration/components/headless-form-async-test.gts#L222
...but not any bubbling out.
I'm guessing we want to have these errors bubble out.

First, this behavior is not by design. As you can see in the place that you linked to where we call onSubmit, we are not having any catch in place that would hide the rejected promise.
You are quoting ember-async-data that it “re-throws the promises", but I actually don't see that in their code. It does catch an error here, but without rethrowing. So I am not really sure what they refer to here. I think if they would rethrow, that would also change that behavior, as again we don't do any catch on our side.
What would be your expectation for the rejected errors to not be hidden? That you see them triggering a unhandledrejection event on window?

Yes, that was my expectation when developing with ember-headless-form. I was struggling to debug an issue, because I had no feedback.
My use case is to perform an action beyond the form, so I hadn't wired up the isRejected check to show in the UI.
I suppose there could be valid reasons to reject the promise and present this in the form, but equally there might be unexpected bugs where that isn't suitable.
Would it be worth having a specific type of Error to throw in each circumstance? Catch all instances of FormError for presenting in ui, but throw anything else?
Probably a breaking change tough.


only add annotation to rcg if the channel index exists in the header

@merky yeah I think its a numpy array or something


When reference server is in HTTP 1.1 mode, don't allow negotiating HTTP/2 via TLS

Some clients may not have the direct ability to disable negotiating HTTP/2 if the server advertises its support during ALPN step of TLS handshake. So to prevent such clients from using HTTP/2, if HTTP 1.1. is the intent, we disable HTTP/2 in the reference server. (Sadly, we can't also disable HTTP 1.1 when HTTP/2 is the intent since Go's net/http doesn't provide a way to do that 🤷.)

cc @srikrsna-buf


Show .zip archive only for Windows

If a release archive is available in both .tar.gz and .zip format, only
show the .zip archive in the download list.

:+1:
This looks okay, but I'm not a ruby person.

Thanks, I tested it locally and it works fine. The last missing part for Windows zip releases is a new release of promu https://github.com/prometheus/promu/pull/196 and updating all Makefiles.


huggingfacehub model from local folder?

Hi,
I would like to run a HF model ( https://huggingface.co/chavinlo/gpt4-x-alpaca/ ) without the need to download it, but just pointing
a local_dir param as in the diffusers for example. Is this possible? someting like this ...
from langchain import PromptTemplate, HuggingFaceHub, LLMChain, OpenAI

template = """Question: {question}

Answer: Let's think step by step."""
prompt = PromptTemplate(template=template, input_variables=["question"])
print(prompt)
llm_chain = LLMChain(prompt=prompt, llm=HuggingFaceHub(repo_id="/mymodels/chavinlo/gpt4-x-alpaca", model_kwargs={"temperature":0, "max_length":64}))
question = "What NFL team won the Super Bowl in the year Justin Beiber was born?"
print(llm_chain.run(question))

I've found it much easier to use the transformers pipeline and then pass the pipeline object to langchain.
So like this:
`from langchain import PromptTemplate, HuggingFaceHub, LLMChain
from langchain.llms import HuggingFacePipeline
from transformers import AutoModelForCausalLM, AutoTokenizer, pipeline, T5Tokenizer, T5ForConditionalGeneration, GPT2TokenizerFast
template = """Question: {question}
Answer: Let's think step by step."""
prompt = PromptTemplate(template=template, input_variables=["question"])
print(prompt)
model_path = "/mymodels/chavinlo/gpt4-x-alpaca"
model = AutoModelForCausalLM.from_pretrained(model_path)
#max_length has typically been deprecated for max_new_tokens
pipe = pipeline(
"text-generation", model=model, max_new_tokens=64, model_kwargs={"temperature":0}
)
hf = HuggingFacePipeline(pipeline=pipe)
llm_chain = LLMChain(prompt=prompt, llm=hf)
question = "What NFL team won the Super Bowl in the year Justin Beiber was born?"
print(llm_chain.run(question))`

Thanks you for an example. I finally got it working
from langchain import PromptTemplate, HuggingFaceHub, LLMChain
from langchain.llms import HuggingFacePipeline
from transformers import AutoModelForCausalLM, AutoTokenizer, pipeline, T5Tokenizer, T5ForConditionalGeneration, GPT2TokenizerFast
from transformers import LlamaForCausalLM, LlamaTokenizer

template = """Question: {question}

Answer: Let's think step by step."""
prompt = PromptTemplate(template=template, input_variables=["question"])
print(prompt)

model_path = "/sd_models/llama/chavinlo/gpt4-x-alpaca"
model = LlamaForCausalLM.from_pretrained(model_path, device_map="auto", load_in_8bit=True)
tokenizer = AutoTokenizer.from_pretrained(model_path)

print('making a pipeline...')
#max_length has typically been deprecated for max_new_tokens
pipe = pipeline(
    "text-generation", model=model, tokenizer=tokenizer, max_new_tokens=64, model_kwargs={"temperature":0}
)
hf = HuggingFacePipeline(pipeline=pipe)

llm_chain = LLMChain(prompt=prompt, llm=hf)
question = "What NFL team won the Super Bowl in the year Justin Beiber was born?"
print(llm_chain.run(question))

As a sidenote, you need to have accelerate and bitsandbytes to run it on GPU.

I would like to do something similar to this, but for an embedding model as opposed to a local LLM. Specifically, I would like langchain to load the InstructorEmbeddings model from local files rather than reaching out to download it. Any examples of this in practice? @batrlatom @Jflick58

@dcruiz01  I was just looking at the code for the HuggingFaceEmbeddings class and it doesn't look like it currently supports passing your own loaded model from sentence_transfomers. I think it would be relatively easy to implement, I'll see if I have time.

Actually, I was wrong. That code just initializes a SentenceTransformer class, which per this documentation takes in a model name OR path. I think the only change would be to update the documentation on the LC side to indicate this.
As far as code, this should work:
            from langchain.embeddings import HuggingFaceEmbeddings
            model_path = "~/myfolder/all-mpnet-base-v2.whatevertheextensionis"
            hf = HuggingFaceEmbeddings(model_name=model_path)

Interesting, thank you for looking! Will have to check and see if it works specifically for the InstructorEmbedding class.

@dcruiz01  Looking at the codefor that, it looks like the InstructorEmbedding implements the "model name or path" logic as well.

 How would I go about loading that with this system?
Thank you in advance for your response.

Interesting, thank you for looking! Will have to check and see if it works specifically for the InstructorEmbedding class. @Jflick58

Maybe I am a bit off topic, but could you clarify the difference between HuggingFaceEmbeddings and HuggingFaceInstructEmbeddings because I see that if we use the same embedding model and the same input the two output vectors are different:
`embeddings_model = HuggingFaceEmbeddings(model_name=embedding_model_path)
embeddings_model_ = HuggingFaceInstructEmbeddings(model_name=embedding_model_path,
model_kwargs={"device": "cpu"})
embedding_example = embeddings_model.embed_documents([my_chunk])
embedding_example_ = embeddings_model_.embed_documents([my_chunk])`

Does anyone have a fully working example for HuggingFaceEmbeddings? Trying to get this to work for the intfloat/multilingual-e5-large model, running completely locally wihtout having to download from huggingface

@lucamcwood have you find the difference between HuggingFaceEmbeddings and HuggingFaceInstructEmbeddings?


Option to emit mangled names for public symbols into the .swiftinterface

When the frontend option -abi-comments-in-module-interface is provided during interface printing, the printed interface will contain additional comments that provide the mangled names for public symbols.
This is an experiment in seeing how much information we can meaningfully extract from a printed Swift interface for the purpose of bridging with other languages.

@swift-ci please smoke test

@swift-ci please build toolchain


Is there any deevlopment to make swaggo compliant with OpenApi v3?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I'm afraid not.
Work to live.

Same question here #386.


LUA error since 10.0.0 patch - nil 'entry' at 302

I just picked up this addon, I have so many addons that are still out of date and slowly being updated that I wanted more control over the loadout so was hoping this would help with my troubleshooting.  However, when I log in I'm getting the error below.  I tried protecting against nils in the function call (that and the other function that took an 'entry' parameter) which allowed the addon to load without error but I'm getting a new error when I try to save to a new layout, I'll report that separately.  I'll poke around a bit more to see if I can make it functional but I'm guessing there's more wrong after the patch than I'll be able to fix.
I also have no idea why there'd be a nil result in the addon list, could be an in-game bug.
1x ...rface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:302: attempt to index local 'entry' (a nil value)
[string "@Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua"]:302: in function ShowMemoryUsage' [string "@Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua"]:295: in function <...rface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:282> [string "=[C]"]: in function AddonList_Update'
[string "@Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua"]:227: in function RefreshDropDownAndList' [string "@Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua"]:220: in function InitUI'
[string "@Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua"]:116: in function <...rface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:112>
[string "=(tail call)"]: ?
Locals:
self =  {
GetAllAddons =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:560
OnAddonTooltipUpdate =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:238
OnAddonListShow =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:231
OnConfigurationSelected =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:167
ResetAll =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:127
IsConfigurationGlobal =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:541
PLAYER_FLAGS_CHANGED =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:206
OnAddonListUpdate =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:282
OnInputDialogConfirm =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:414
CheckConfigurationCondition =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:393
CheckDistinct =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:536
ShowError =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:24
ShowConfigurationSwitchPromptDialog =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:468
DeleteConfiguration =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:385
ResetConfiguration =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:361
ExportAddonListWithBBSCode =  defined @Interface/AddOns/ImprovedAddonList/Export.lua:14
GetPlayerInfos =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:224
LastUpdateMemoryUsageTime = 66563.265000
OnDropDownMenuInitialize =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:158
GetExportAddonData =  defined @Interface/AddOns/ImprovedAddonList/Export.lua:147
CreateExportDialog =  defined @Interface/AddOns/ImprovedAddonList/Export.lua:98
ExportAddonListWithMarkDown =  defined @Interface/AddOns/ImprovedAddonList/Export.lua:61
Frame = Frame {
}
ExportAddonList =  defined @Interface/AddOns/ImprovedAddonList/Export.lua:4
ShowOrHideInputDialog =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:394
ShowRemark =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:319
IsCurrentConfiguration =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:570
ADDON_LOADED =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:112
OnTipsButtonClick =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:339
GetEnabledAddons =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:547
GetConfiguration =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:594
IsConfigurationMeetCondition =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:376
ShowMemoryUsage =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:301
SaveConfiguration =  defined @Interface/AddOns/ImprovedAddonList/ImprovedAddonList.lua:344
PLAYER_LOGIN =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:174
GetConfigurationWithConditions =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:346
MustbeSaveToGlobal =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:287
GetClassAndSpecInfos =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:237
InitConditionContent =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:211
PLAYER_SPECIALIZATION_CHANGED =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:199
PLAYER_ENTERING_WORLD =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:195
ResetConditions =  defined @Interface/AddOns/ImprovedAddonList/Condition.lua:362
GetConfigurations = <f

Blz change the scrollview construct method  which is addonlist used,  I won't fix it, Instead, I think a whole new addon panel which no taint, more features will better to use. This issue will be closed when a new version arrives.

What will the new addon be called

What will the new addon be called

The same name, not a new addon. For now, just disable this addon. I'm sorry.

For what it's worth I've been using the addon with the workarounds I mentioned without any issue so far, it's working fine for what I need it for.

Well, I temporarily disabled remark and export features, and it shouble be work fine in 1.9.11

fix by v2.0.0, which is a completely reworked version


Smoketest remote: Multiroot test fails with no search results

Ubuntu 18.04 LTS

`yarn smoketest --remote --build /home/aeschli/vscode/1.38.0-insider-3db7e09f3b/VSCode-linux-x64/

Multi root test fails, no search result found

Same when running against source or on Windows.
Typing a space in the quick input box shows the result, so it looks like a timing issue.

Is there a way for the test to wait until the window is connected before triggering the search?

@aeschli I am now looking at the text of the status bar item to tell whether the window is actually connected. Let me know if that's not ok.

We could also add a class/id to that item.


How to understand VCPKG_DEFAULT_TRIPLET and VCPKG_DEFAULT_HOST_TRIPLET?

For cross compile, there are 3 concepts:

build
host
target

VCPKG_DEFAULT_HOST_TRIPLET maps to host, right?
What does VCPKG_DEFAULT_TRIPLET map to?
For the 3 concepts in cross compile above, there should be 3 corresponding things in vcpkg but only 2 now (i.e. VCPKG_DEFAULT_HOST_TRIPLET and VCPKG_DEFAULT_TRIPLET), how can it be?

These three concepts are valid only when building a toolchain. For everything else there are only two concepts: host and target.
Host means the machine you are building on, and target means the machine you are building for.
VCPKG_DEFAULT_TRIPLET is for target triplet.

These three concepts are valid only when building a toolchain.

In particular, this is a particular terminology from autotools, and often a source for confusion.
OTOH host is a term also used by CMake, and much closer to vcpkg.


Should we just use Github for everything?

Project management
Wiki
Issue tracking
Roadmap
And then integrate it with slack..that's it.

@TheRemoteLab/lets-devops

To begin with I don't see any issues with this, as it grows may need to branch out but MVP and all that ;)

Alright..using github for now. @TheRemoteLab/lets-devops


s3.Object.copy_from is missing dict type for CopySource parameter

Describe the bug
In boto3 docs the s3.Object.copy_from supports a union of string and dict:
    CopySource='string' or {'Bucket': 'string', 'Key': 'string', 'VersionId': 'string'},

However in the latest type stub (mypy-boto3-s3 1.34.138) there's only str:
        CopySource: str,

To Reproduce
Steps to reproduce the behavior:

Install boto3-stubs[essential]
Run mypy/pyright on the following code sample

import boto3

boto3.resource('s3').Object('foo', 'bar').copy_from(CopySource={'Bucket': 'bar', 'Key': 'foo'})

Actual output
test.py
  test.py:3:64 - error: Argument of type "dict[str, str]" cannot be assigned to parameter "CopySource" of type "str" in function "copy_from" (reportGeneralTypeIssues)
1 error, 0 warnings, 0 informations

Expected output
0 errors, 0 warnings, 0 informations

Additional context
Very similar issue as https://github.com/youtype/mypy_boto3_builder/issues/169

Hello! Thank you for the report.
Looks like CopySource argumet type annotation is incorrect for Object.copy_from and ObjectSummary.copy_from.

Fixed in the latest mypy_boto3_builder 7.15.3 verison. Also, I released mypy-boto3-s3 1.34.158 with the fix included. Please update and let me know if it works as it should.

works! this was the change? https://github.com/youtype/mypy_boto3_builder/commit/8ea21503e200b11dd6ac3d524b91794328c3af5f
and thanks!

Yes. As you see, I was overriding copy/copy_from for multiple resources already, but Object and ObjectSummary were not covered.


Remove conditional statements

Pull Request (PR) description
This PR removes the conditional statements around two environment variables so that they are always set. In doing so it allows the auto crl renewal process to succeed if either of the two options are missing in the openvpn::server declaration.
This Pull Request (PR) fixes the following issues
Fixes #318

Hi @jameskirsop i don't thing the fix proposed is the good one. All acceptance tests on nodes running EasyRSA v3 are failing.
IMO, the issue is that the command used in resource openvpn::server does not make the difference if the node use EasyRSA v2 or v3.
With EasyRSA v3, i tested the following command  and it works like a charme:
. ./vars && EASYRSA_REQ_CN='' EASYRSA_REQ_OU='' openssl ca -gfncrl -out /etc/openvpn/test_openvpn_server/crl.pem -config /etc/openvpn/test_openvpn_server/easy-rsa/openssl.cnf
So the fix should be, with custom fact easyrsa, to take care of the EasyRSA version installed and then trigger the corresponding command.
Can you modify the PR in this way ?

If you need help fill free to ask. You can also join #voxpupuli on IRC and slack.

The issue is now closed by PR #321


[BUG] openCypher and IAM doesn't refresh token

Describe the bug
When creating a jdbc connection via: jdbc:neptune:opencypher://bolt://xxxxx.cluster-xxxxxus-east-1.neptune.amazonaws.com:8182;authScheme=IAMSigV4;serviceRegion=us-east-1
If you

[x] Have you validated that this is not currently reported in
SQL Support and Limitations?

Steps to Reproduce

[x] Is the JDBC Driver being used in a BI tool?

What vesion of the JDBC Driver are you using? 3.0.0
What BI Tool are you using? DataGrip
What version of the BI tool are you using? 2024.01
What Java version are you using? 17
Please provide steps to reproduce bug from initial launch of BI tool --- Leave connection open for > 5 minutes
If bug occurred during query execution attach the SQL -- no
Attach debug logs if applicable/possible (please ensure it doesn't contain any sensitive information) -- n/a

Expected behavior
For use in a connection pool, I would expect to be able and re-use a connection for more than 5 minutes.

Hi @er1c, no token refreshing used to be a limitation of the bolt driver which is used under the hood. This limitation has since been addressed in v5.14, however neo4j-java-driver requires min java 17, where we are still supporting java 8.

This issue has been fixed in #259.


Use QThread where applicable

This seems like the better option for preventing a frozen GUI.
Read up on QThread here

Since I think I've fixed this with https://github.com/imagej/napari-imagej/pull/130 (and presumably in other places as well), I'm going to close this issue.


EventGrid CLI Extension: 2020-04-01-preview bug fixes

Add support for event subscription msi identity (for destination and deadletter) for both create and update operations.
Add support for update operation for private endpoint connection (approval/rejection)
topic/domain update operation fails if sku is not specified which was wrong.
Fix help texts.

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[X] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

add to S168

@qwordy @Juliehzl   Can you please help review/merge these fixes. thx

@Juliehzl All comments addressed. thanks.

@Juliehzl @qwordy Can you please help merge this change and publish the corresponding wheel ASAP. thanks.


use docker link

I noticed the project has to be build for now to use it. It will be great if you can do docker compose of prebuild images. I think the project will be very popular that way. I have just started looking into it but don't seem to have access to info about environment variables or may it is; will be goings through docs and sources and will follow with with query if required.
Thx 4 awesome project did i mension :)

Hi @jas99, thanks for the interest. This is a topic in which we haven't made a decision yet. We'll keep it in mind.

@mssola thx for reply.
Best of luck with ur independence struggle :+1:

Superseded by #1027. Feel free to re-open this issue if you think that there is still things to discuss here and that they are not suitable to be discussed in #1027.

@mssola
Cool!!
I have been using gitlab for my docker image storage lately; just for feedback.
Thx


feat(lidar_centerpoint_tvm): remove package

Description
Removes the lidar_centerpoint_tvm package.
Related links
Parent issue:

https://github.com/autowarefoundation/autoware.universe/issues/7701

Related PR:

https://github.com/autowarefoundation/autoware/pull/4919

How was this PR tested?
See CI
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@xmfcx @mitsudome-r Hi, would you review this PR?

We'll need to remove these too (from autoware repo):

https://github.com/autowarefoundation/autoware/blob/ada7b8caf76f1d19a38b6da0644798c74c791702/ansible/roles/artifacts/tasks/main.yaml#L432-L475

You can import #7737 to fix this.


[Feature]: Graceful port suggestion if 3000 is taken

What is the new or updated feature that you are suggesting?
Suggest a new port if 3000 is being used
Why should this feature be included?
Currently it fails and leaks a fairly comment error to the end user

s

There are two pr to this question 😁
auto change dev server port
Check if the specified port is in use

Fixed in https://github.com/remix-run/remix/pull/871


Shimmed make is silencing warnings even with --verbose

Please follow the general troubleshooting steps first:

[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed as many issues as possible and retried your prior step?
[x] If you're seeing permission errors tried running sudo chown -R $(whoami) $(brew --prefix)?

Bug reports:
I was running into an issue where what should have been outputting a ton of warnings gave off no warnings at all - example build log: https://travis-ci.org/fontforge/fontforge/jobs/161231014
And I was very perplexed. Some further digging suggests that the shimmed make which makes use of xcrun which is also shimmed is piping the output to /dev/null? This is totally unexpected behaviour, as I should be able to see all warnings when building with --verbose.
If I just run /usr/bin/xcrun /usr/bin/make (from within the interactive shell), warnings appear as they should, but whatever the xcrun shim is doing is dumping all warnings into the void.

Just tried - --env=std doesn't help.

Thanks for the bug report. Just to let you know that this write-up is good and it's on my list of things to work on.

This should probably be reserved for debug mode.

This is no longer on my list of things to work on but I'll happily help someone else who wishes to do so.

Are there any (easy) examples to trigger this? AFAIK everything is rerouted to the log:
https://github.com/Homebrew/brew/blob/780e86904beff771e9f7e90fa9e53dd9dc4aeeaf/Library/Homebrew/formula.rb#L1863
and printed only in case of failure:
https://github.com/Homebrew/brew/blob/780e86904beff771e9f7e90fa9e53dd9dc4aeeaf/Library/Homebrew/formula.rb#L1776-L1796

It's been a long time since I bothered looking at Homebrew, so not sure of an easy example. But easy enough is what does the output from a current build of FontForge (from Homebrew) look like?
Compare against our Travis build which runs without: https://travis-ci.org/fontforge/fontforge/jobs/310179410
There are a ton of compiler warnings, which should show up similarly when built under Homebrew.

@jtanx Thanks for the tip.
Redirecting stderr to the log file prevents the warnings from showing up. Perhaps we should copy $stderr output to both screen and log when using --verbose? E.g. like this stackoverflow question .


Recommend final classes for architecture tests

Non-final classes are open for inheritance when it wouldn't make sense to inherit them.
Also, inheriting them would likely duplicate tests or break the run of phpat.

Hello. Thanks for the PR but I don't think this is a good idea or at least really needed.
First of all, I'm trying to avoid tiny PRs unless they are really meaningful (read CONTRIBUTING.md). The reason is that the tool is in an early stage and there are many small details that could be improved with time, and I want to give space to proper recognition to contributions with big changes.
Regarding the proposed change itself:
It may sound weird but some user might want to define which classes to test or not based on some custom method, in that and many other potential cases, it would make sense to extend an abstract test.
If you check other test suites, for instance, this phpunit test or this phpspec one you will see they don't declare them as final. In fact phpat's unit tests are working with a custom abstract test.
I don't see the benefit of declaring the tests final. If you have any thoughts or arguments, please keep commenting here :)

@carlosas Sure! That's your preference. I'd just like to point out that when a class is specifically meant to be extended, it would of course, not be made final and sometimes it would be abstract. What I meant here is that by default these classes are not meant to be extended and so they should be final. Not that all such classes MUST be final.
Some more thoughts on the topic:

https://ocramius.github.io/blog/when-to-declare-classes-final/
https://verraes.net/2014/05/final-classes-in-php/


Compatibility with Terrain ruler Toggle

I have the Terrain layer  and the Terrain ruler modules, each player can toggle if difficult terrain is ignored (IE flying). the Toggle works with the standard measurement however Drag Ruler always adds difficult terrain if present regardless if toggled on/off.

The intended way to solve this is to integrate Drag Ruler into the game system you're using by utilizing Drag Ruler's API. That Integration could either be done by a separate module or by the game system itself. Thr integration module could then detect if a creature is flying and could make Drag Ruler ignore difficult terrain for that creature.
Nevertheless, I'll think about adding a manual override for ignoring difficult terrain.

I'd also like to see this enhancement implemented. I'm playing using 5e, so the type of movement ignores difficult terrain accordingly, but there are other cases where it would be useful, such as class features and spells that allow ignoring difficult terrain or a particular case I just found in the adventure I'm planning:
A room has some lava channels on it that make those squares difficult terrain for the player characters. However, some of the enemies there are fire elementals that don't really have any issue when it comes to stepping on lava. Being that players can also cause other types of difficult terrain that can affect the elementals, it would be useful to have way to toggle difficult movement on and off while also using Drag Ruler. There's always the workaround of using ctrl+drag when ignoring difficult terrain, of course, but that's more of a drag (pun intended).

Some of this could be achieved using libRuler. Settings in Terrain Ruler could be picked up by Drag Ruler so you get consistent results regardless of whether you are using the Foundry ruler or Drag Ruler. Detecting if a creature is flying is still hard, of course, and probably should be done either in Terrain Ruler or maybe a separate module altogether.


Drop php-5.x supports

As title, I think it's time to drop the php-5.x version supports.
The php-5.x versions are too old to maintain and they're not maintained by official team anymore.

@kevinlebrun, do you have any suggestion about this?

We could drop the 5.x versions support and release a major version of the library with the last few changes. I will try to do this in the next few days.


typo

Coverage remained the same at 71.087% when pulling 6ee3a61089c5c2fdf1a79c1545e7ae56cd5e109b on vitor-alves:master into 900cbb1cf31be6587a49792a8c5a36364dce246b on dvajs:master.


[2.x]Properly handle :undefine-attribute in domain sync-model

for master it is part of #1608

Full integration - Windows Build 42 outcome was UNKNOWN using a merge of a61356ad2c97039543db834095e0dfc1c3f77429
Summary: Canceled (Tests passed: 1549, ignored: 127; exit code 1 (new)) Build time: 00:16:31

Linux Build 43 outcome was UNKNOWN using a merge of a61356ad2c97039543db834095e0dfc1c3f77429
Summary: Canceled (Tests passed: 3310, ignored: 61; exit code 137 (new)) Build time: 00:16:33

Core - Full Integration Build 44 outcome was UNKNOWN using a merge of a61356ad2c97039543db834095e0dfc1c3f77429
Summary: Canceled (Tests passed: 1996, ignored: 134; exit code 137 (new)) Build time: 00:16:33

Windows Build 39 outcome was UNKNOWN using a merge of a61356ad2c97039543db834095e0dfc1c3f77429
Summary: Canceled (Tests passed: 3068, ignored: 50; exit code 1 (new)) Build time: 00:16:35

@ctomc can you file a JIRA for this and resolve it against 2.2.0.CRwhatever and 3.0.0.Alphawhatever please? And link the PRs. This seems like a bug that people are going to hit in the real world so it should be findable in JIRA.

@bstansberry done https://issues.jboss.org/browse/WFCORE-1630


Playapiupdates

This change is

Reviewed 23 of 23 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

player-app/src/main/java/org/gameon/player/AllPlayersResource.java, line 86 [r1] (raw file):

        else {
            List<PlayerResponse> prs = results.stream()
                    .map(record -> {PlayerResponse pr = new PlayerResponse(record); pr.setCredentials(null); return pr;})

This is still modifying/removing the items an item at a time, right? (as they are added?)

player-app/src/main/java/org/gameon/player/entity/PlayerCredentials.java, line 1 [r1] (raw file):

package org.gameon.player.entity;

copyright

player-app/src/test/java/org/gameon/player/AllPlayersResourceTest.java, line 160 [r1] (raw file):

            List<PlayerResponse> resultList = (entity.getEntity());
            Assert.assertEquals("Players list is not the expected size", players.size(), resultList.size());
            //TODO: verify

.... :8ball:

Comments from Reviewable

Review status: 19 of 23 files reviewed at latest revision, 2 unresolved discussions.

player-app/src/main/java/org/gameon/player/AllPlayersResource.java, line 86 [r1] (raw file):
Previously, ebullient (Erin Schnabel) wrote…
> This is still modifying/removing the items an item at a time, right? (as they are added?)

correct.. it's just an iteration over the list that maps them into player response objects and nulls their credentials.. nothing special.. curious to know if we can use a custom couch view to popular player response objects directly without having issues with apikey / location being differently structured..

Comments from Reviewable

This change is probably the easier of all of them, it's not dependent on the new auth stuff, and can be totally delivered at any time once you're happy with it.. it is obviously a prereq for anything that needs the new player api

Reviewed 4 of 4 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Upgrade scripts fails due to E2BIG error

Describe the bug: (a clear and concise description of what the bug is)
I happens that the upgrade scripts yield about too long parameters provided to a shell.
Unfortunately, I am not sure what exactly is causing that issue.
The scripts are encoded in the upgrades.yaml file and executed by in the upgrade() function from the src/cmd/upgrade.ts file.
To Reproduce
Steps to reproduce the behavior:

Trigger Otomi Drone pipeline:
See error

Upgrade records detected for version dev
2023-03-06T12:57:01.070Z otomi:global:error Error: spawn E2BIG
    at ChildProcess.spawn (node:internal/child_process:413:11)
    at Object.spawn (node:child_process:700:9)
    at Promise.promise._run (/home/app/stack/node_modules/zx/dist/bundle.cjs:15252:30)
    at Promise.then (/home/app/stack/node_modules/zx/dist/bundle.cjs:15382:25)

Expected behavior: (a clear and concise description of what you expected to happen)
Screenshots: (if applicable, add screenshots to help explain your problem)

Core Version v0.18.0, v0.18.1
cloud: azure
cluster:  1.23

E2BIG comes error from the gucci function, which is called in the upgrade.ts file.
It turns out that this function convert all values to command line arguments, which causes the error.


Accessibility

I am working with a bunch of presentr's to show various controllers throughout my app.  I love it.
I am asked to do an Accessibility sweep in prep for use by government.  They test using VoiceOver.  When I use Voice Over, I am unable to dismiss the presented controller, by tapping outside.  Also when tapping and dragging on the blurred background behind, Voice Over is reading my underlying controller interface items.
I have tried using accessibilityViewIsModal with no luck.  tapToDismiss is on as well.  Without Voice Over on , all works as designed.
Has anyone had any experience with this?  We have to give the sight-mpaired a way to close the presented controller when voice over is on.

I was able to get  accessibilityViewIsModal to work properly.  Now Voice Over ignores UI elements in underlying view (below the blurred background in my case)
    presenter.blurBackground = true
    definitionsVC.view.accessibilityViewIsModal = true
    customPresentViewController(presenter, viewController: definitionsVC, animated: true, completion: nil)

I am still not having luck dismissing the controller when VoiceOver is on.  I may have to resort to adding a close button in the corner.  Hoping someone has another solution

Hey @DannyJi !
Thanks for using Presentr, I'm glad you love it.
I don't have much experience with Accessibilty, I just assumed that everything would work out of the box. But this is something really important, and I'd be glad to look into it.
So, the first problem was that Voice Over was reading the views behind the blurred background, but you managed to fix that using the accesibilityViewIsModal property on the presented VC, correct?
Your other problem is that tapping on the background, does NOT dismiss the view controller? Do you have any sample code I can try out? Or can you tell me the steps necessary to recreate the problem myself? so I can try and see if there is a fix for that. For some reason the background view is NOT receiving touch I assume.
For now you could add a button do dismiss the VC, but anyways we should fix this.
👍

Hey @danlozano,
Thanks for your response.

Yes.  Adding accesibilityViewIsModal to the presented VC's view solved the problem where, without that VoiceOver would be "speaking" items behind the view.

Yes.  I cannot tap outside the presented view to dismiss the VC as normal when VoiceOver is active.  Without it, it works as expected.  I do not have sample code to try at this point (getting an app ready to send for testing tonight), but any app with a presented VC using Presentr and VoiceOver enable should demonstrate the problem.  You will need to try this on a real device, enable VoiceOver from the Settings > General > Accessibility.  Note that VoiceOver will change your tap pattern, for instance most elements you need to double tap where you would normally tap.

I think your assessment is correct.. the background not receiving the touch.
I will try to work on this in a few hours myself after some deployment meetings.
Thanks again.
Danny

Hey @danlozano,
For the second issue (tapping the background does not dismiss the presented view controller), I first started as we discussed - adding a close button begrudgingly.  While I got this to work, I was not happy with the way it looked and the code I needed to copy around.
I did some more research and found a great article Adding Accessible Behavior
There are two "global gestures" or concepts that can be used to dismiss a modal window.  One is a magic tap, but the other is called the escape gesture.  That is the one I wanted.  The user can simply draw a "Z" with 2 fingers on the touch surface.  It is an escape functionality that takes you back a step when VoiceOver is activated.  I just needed to implement the accessibilityPerformEscape() method on the ViewControllers that would be presented in the app by Presentr.
 override func accessibilityPerformEscape() -> Bool {
    self.dismiss(animated: true, completion: nil)
    return true
  }

This worked, was a lot less coding and helped pass an accessibility review.
It would be nice if these could be incorporated into Presentr as an option somehow - maybe in its configuration.
Possible Future Enhancement
.
.
presenter.blurBackground = true
presenter.accessibilityPresentedViewIsModal = true
presenter.accessibilityEnableEscapeToDismiss = true
.

I just wanted to share back.

Hey @DannyJi !
Thanks for the help, I've been really busy.
Let me see if there is a way to integrate that functionality inside Presentr, it would be great!
Thanks for doing the research :) I'lll get back to you soon.

Hi @DannyJi !
Thanks for sharing the info. I've been really busy at work lately so I have not had time to update or look into Presentr's issues. Finally have some days off to work solely on my open source projects so I'm looking into everything.
So basically for voiceover to ignore the elements under the modal you have to set
view.accesibilityViewIsModal on the view controller.
And if you want to be able to dismiss the view controller you have to override the
accesibilityPerformEscape on the view controller.
After giving it some thought I realized that this is something outside the responsibilities of Presentr. These are "customizations" you are doing on your own view controller to make it behave "differently" or in this case, work properly.
Besides, there is no easy/non-hacky way to automatically insert the overridden method into the presented view controller by Presentr. I think that is better to be left to the developer.
What I could do to make use of your research in finding this "bug" is add a section in the Readme under accesibilty, and outline the things someone might want to do on their own view controller if they are in a similar case.
Sound good?
👍


[Bug Report] cannot set the default checked value while using multiple and lazy option

Element UI version
2.11.1
OS/Browsers version
windows10 chrome 76.0.3809.100
Vue version
2.6.10
Reproduction Link
https://codepen.io/vivianus/pen/LwGMgj
Steps to reproduce
use the cascader with multiple and lazy option to submit a form,next time open the form to edit, and you'll find you have no ways to initialize that cascader again,so  how to initial the cascader in this case?
What is Expected?
has the ability to set default checked while using muliple and lazy option,
What is actually happening?
if i use that cascade with muliple and lazy option in a form,i cannot initialize the cascader in  that form when i enter in 'edit' mode

+1 meet the same problem

+1

+1

+1

Are there other solutions?same problem here.


Show correct debug tooltips for existing variables

Fixes #49939
I'm not entirely sure how this regression happened, I just applied a fix that restores the old behavior.
I didn't find any occurrence of prop_values being indexed using the type and name of a variable, so I think it's fine. nvm, it's used in the Debugger dock.

This fixes the issue but breaks the variable inspector:

This fixes the issue but breaks the variable inspector:

I knew it wasn't this simple! :smile:

Ok, I found out how it worked in 3.x. Here is the old code I used:
https://github.com/godotengine/godot/blob/7e03bd1671050d940c62f758e7495fa3dd9ee662/editor/script_editor_debugger.cpp#L92-L101

Seems working correctly, I wonder why this is not merged a long ago.

I think it broke the inspector, is that resolved in the latest build?

Hmm, I don't see any error:

Hmm, I don't see any error:

B shouldn't be null in the Locals section.

I think it's not related to this PR, b is null and without this PR in that case.

B shouldn't be null in the Locals section.
I recall seeing that bug reported long ago, but I can't find the issue where I first saw it. Either way that bug still happens without this PR, so it's unrelated.


many tests are xfailed for polars-lazy

need to work around them and see how much is salvageable

significantly less than before, getting there


Preferences missing from CartBuyerIdentityInput type

What is the location of your example repository?
No response
Which package or tool is having this issue?
Hydrogen
What version of that package or tool are you using?
2024.4.0
What version of Remix are you using?
2.8.0
Steps to Reproduce
Attempt to define preferences for the buyerIdentity object param of the Hydrogen Cart updateBuyerIdentity method in IDE (VS Code). The reported typescript error should be shown.
Expected Behavior
preferences exists on the Storefront 2024-04 CartBuyerIdentityInput input object.
https://shopify.dev/docs/api/storefront/2024-04/input-objects/CartBuyerIdentityInput
I should be able to set the value of preferences on buyer identity via the Hydrogen Cart updateBuyerIdentity method without getting typescript errors that preferences does not exist on the type.
Actual Behavior
I get the following error when trying to define preferences:

Object literal may only specify known properties, and 'preferences' does not exist in type 'CartBuyerIdentityInput'.ts (2353)

Checking the storefront-api-types.d.ts file there is no preferences on CartBuyerIdentityInput.

I am not sure why the doc for 2024-04 is using preferences where the generated schema is listing walletPreferences
but this will be fix with 2024-07 when it is release next week. Would that be sufficient?


[no ticket] Replace our local @ember/test-helpers types with @types/ember__test-helpers

Purpose
@dfreeman has polished our typedefs for @ember/test-helpers and pushed them to DefinitelyTyped, so we should use 'em!
Summary of Changes

git rm types/ember-test-helpers/index.d.ts
yarn add --dev @types/ember__test-helpers

Side Effects / Testing Notes
If it builds, we're gtg!
Ticket
N/A
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s) (typedef swap only)
[ ] changes described in CHANGELOG.md (typedef swap only)

Pull Request Test Coverage Report for Build 1394

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 61.574%

Totals

Change from base Build 1392:
0.0%

Covered Lines:
2230

Relevant Lines:
3432

💛  - Coveralls


is the pasuse and play meant to go to liked songs?

Consent

[X] I have searched the existing issues and verified that there is no open issue for the same subject.
[X] I understand that the YTMDesktop Elgato Plugin has NO affiliation with Google, YouTube or the YTMDesktop application.

Question
every time i press play or pause it always takes me to liked songs and starts playing that so i just wonder if it was a feature or a bug since it never use to do this?

Hey!
Which version of YouTube Music Desktop are you using?

This is the plugin version. I meant the version of the application YTMDesktop itself.

1.13

v1.13 isn't supported and is full of bugs. Please use the recommended version 1.14.2 (Night Release May 2022)

Small update. This issue seems also to exists in the Nightly releases. YTMDesktop already has an open issue for this (sadly I can't fix this). See ytmdesktop/ytmdesktop#1211

Update: #1210 fixes this issue.

I'm gonna /close this issue as resolved, as the mentioned workaround in my comment above resolved this.

hello again i just wanted to point out that i tried the 1210 but i seem to not work eaither and im on 1.14 and ive copied the same as the dude on the post but still nothing
https://github.com/XeroxDev/YTMD-StreamDeck/assets/153271025/698e0976-61c9-4197-8e4e-7f4b9bcc1191
that is a video of me showing the problem

Like stated in the mentioned ticket above, you've to disable Play/Pause x2 in Settings > Shortcuts > Show/Hide Main Window (last entry

i have tried doing the same as the 1210 dude and it is still doing

and just real quick how do you do that

To contact them, you can either go to their repository (via the link above) or go on their discord (discord link should be on their website, app and/or repository)


docs: Update broken links to cli docs

Signed-off-by: TheMonolithX64<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Update broken links to CLI docs from Upgrade page and backend README
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Still needs review. Let me know if there's anything else I need to do @freben 🙂


Revert "Update Terraform azuread to v2.23.0"

Reverts hmcts/em-annotation-api#789
Seems we have issue with deploying to AAT Staging.

Not required. As build is resolved now. Seems was Jenkins issue


Unable to download packages from offline machine with configured proxy

Checks

[X] I have checked that this issue has not already been reported.

[X] I have confirmed this bug exists on the latest version of pixi, using pixi --version.

Reproducible example
PS C:\Users\user\Desktop\pixi_test> pixi add mypy
 WARN Ignoring 'repodata_options' in at C:\Users\user\Desktop\pixi_test\.pixi\config.toml
 WARN TLS verification is disabled. This is insecure and should only be used for testing or internal networks.
Added mypy >=1.11.1,<2

PS C:\Users\user\Desktop\pixi_test> pixi add pytorch
 WARN Ignoring 'repodata_options' in at C:\Users\user\Desktop\pixi_test\.pixi\config.toml
 WARN TLS verification is disabled. This is insecure and should only be used for testing or internal networks.
  x failed to solve the conda requirements of 'default' 'win-64'
  `-> Cannot solve the request because of: No candidates were found for pytorch *.

PS C:\Users\user\Desktop\pixi_test> pixi add --pypi torch
 WARN Ignoring 'repodata_options' in at C:\Users\user\Desktop\pixi_test\.pixi\config.toml
 WARN TLS verification is disabled. This is insecure and should only be used for testing or internal networks.
  x failed to download pypi name mapping
  |-> Middleware error: Request error: error sending request for url (https://conda-mapping.prefix.dev/hash-
  |   v0/0fce54f8ec3e59f5ef3bb7641863be4e1bf1279623e5af3d3fa726e8f7628ddb)
  |-> Request error: error sending request for url (https://conda-mapping.prefix.dev/hash-
  |   v0/0fce54f8ec3e59f5ef3bb7641863be4e1bf1279623e5af3d3fa726e8f7628ddb)
  |-> error sending request for url (https://conda-mapping.prefix.dev/hash-
  |   v0/0fce54f8ec3e59f5ef3bb7641863be4e1bf1279623e5af3d3fa726e8f7628ddb)
  |-> client error (Connect)
  |-> dns error: No such host is known. (os error 11001)
  `-> No such host is known. (os error 11001)
PS C:\Users\user\Desktop\pixi_test> pixi config list
 WARN Ignoring 'repodata_options' in at C:\Users\user\Desktop\pixi_test\.pixi\config.toml
tls-no-verify = true
detached-environments = false
[mirrors]                                                                                "https://conda.anaconda.org/conda-forge" = [                                                                                "http://MY_MIRROR/repository/conda-forge-proxy",                                                                                "http://MY_MIRROR/repository/anaconda-pytorch",                                                                                "http://MY_MIRROR/repository/anaconda-nvidia",
]
[pypi-config]
index-url = "http://MY_MIRROR/repository/pypi-all/simple"
keyring-provider = "subprocess"

Issue description
So I have several local machines, that cutted out from internet, but have access to already cached packages with Sonatype Nexus.
When I configure my pdm/poerty/pip/conda configs with exact cacher urls that in example above - everything works right with every tool, but pixi rejects to work properly.
I followed step-by-step with this guide https://pixi.sh/v0.18.0/advanced/global_configuration/
BTW, WARN Ignoring 'repodata_options' is very strange message, because you have that in your example.
For now, the only thing that working is installing packages from the first specified mirror for conda-forge (ex. conda-forge-proxy), like it worked for installing mypy in the above example
Expected behavior
I expected that pixi will:

fetch all specified mirrors in order, not only the first one
fetch pypi packages even without compressed_mappings (btw, pip have index and index-url in config)

The error is caused by the requirement to find our custom conda-pypi-mapping This makes sure pixi won't override conda packages with pypi packages when the same packages come from the solve.
You can selfhost or hardcode the mapping in your project. Documentation here: https://pixi.sh/dev/reference/project_configuration/#conda-pypi-map-optional
The mapping is located here: https://github.com/prefix-dev/parselmouth/blob/main/files/compressed_mapping.json
The WARN Ignoring was a typo in the documentation it should have been repodata_options or repodata-options in the newer version. This is fixed in the latest documentation.

I don't really get it. How to config properly? I've downloaded mapping, then tried to add conda-pypi-mapping = {"conda-forge"} and even with replacing "conda-forge" with full link to "https://conda.anaconda.org/conda-forge", but still have this warn:
WARN Defined custom mapping channel https://conda.anaconda.org/conda-forge/ is missing from project channels

pixi_config_list_cat_piped.txt
pixi.txt

Can we just have config flag like disable-conda = true for environments, when we don't use conda at all? Maybe then you don't even will be needed to check mappings... 👉👈

I've changed config to
conda-pypi-map = { }

and now I can install packages only from conda (btw, looks like not only from the first one mirror in list, but even from nvidia conda channel). Also pixi now panicking when I do add/remove some packages - but package itself is installed/uninstalled after this error as normal anyways.
But --pypi packages now failing... all info in .txt below
pixi.txt

Any info about how to make pypi work properly?

We'll have to dive a little deeper. I'm sorry for the inconvenience. You could manually make a pip install task to to the pypi work untill we figure this out properly.

pixi is still failing with --pypi flag like in https://github.com/prefix-dev/pixi/issues/1764#issuecomment-2291460927
the only thing I've changed is upgrading to pixi v28.1


allow symfony/console 3.0+

fixes #41

:+1: Thx! I still embarrassingly don't really know & understand the various syntaxes for versions in composer.


Remove twitter share button

We don't really need to have it on every page, and send data to Twitter.

Can we not leave it on the homepage?

Definitely not in favor of this change. I've actually seen a few people tweeting from the website, and it makes for a consistent footer, so meh.
Also, I expect the design to fully change once we have a pro logo in a few weeks. That most likely means colors at least will change.


Yes

Code of Conduct

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

What article on docs.github.com is affected?

What part(s) of the article would you like to see updated?

Additional information

⁸

Yas


Combine real and imaginary components of FatBands into a single field

Since Julia has the Complex{Float64} type I figured it might be a good idea to combine these two fields. However, @xamberl I'm giving you final say on whether this gets merged since it'll probably affect the notebooks you've written.
Feel free to pull this branch locally and make your own commits to it as well.

Noted. I made the ability to read complex bands but haven't implemented them into notebooks yet, so we can go ahead and merge.


Improve API of componentsAtPoint()

What could be improved
It's been 4 days since componentsAtPoint was introduced, but I think there's a way to make it better. Well, we already talked about the API not being ideal, but couldn't come up with a better solution.
So, here's a proposed solution:
Iterable<Component> componentsAtPoint(Vector2 point, [NestedLocation? nestedLocation]);

Here the nestedLocation argument, if given, will be used to store the local coordinates of point for the component being returned. In fact, it will also store the entire stack of points for every ancestor of the component being returned. For example, if you want to know not only the local coordinates of the point, but also the corresponding coordinates in the parent coordinate space, or in the world coordinate space, then that information will be available.
Any risks?
Technically this is a breaking behavior, but since the method was just introduced, and there hasn't been a release yet, the risk to users is very low.
More information
Are you interested in working on a PR for this? Yes.

Isn't it nicer to have this bundled with the return instead of as an argument, or is that not possible? Although nice that it returns an iterable of components instead now.


pgwire: Handle empty queries

@matriv please review

Wouldn't be more elegant to have a new NoOpStatement class instead of using this select query?

ack

Igtm