http-types-0.10

http-types-0.10 is out of bounds for:

[ ] IPv6DB-0.2.2 (>=0.9.1 && < 0.10). Michel Boucey<EMAIL_ADDRESS>@MichelBoucey. @MichelBoucey. Used by: library, executable, test-suite
[ ] bloodhound-<IP_ADDRESS> (>=0.8 && < 0.10). Chris Allen<EMAIL_ADDRESS>@bitemyapp. @bitemyapp. Used by: library, test-suite
[ ] chatwork-<IP_ADDRESS> (>=0.9.1 && < 0.10). Nobutada Matsubara<EMAIL_ADDRESS>@matsubara0507. @matsubara0507. Used by: library, executable, test-suite
[ ] github-0.17.0 (>=0.8.6 && < 0.10). Oleg Grenrus<EMAIL_ADDRESS>@phadej. @phadej. Used by: library
[ ] github-release-1.0.6 (==0.9.*). Taylor Fausak<EMAIL_ADDRESS>@tfausak. @tfausak. Used by: library
[ ] hackernews-<IP_ADDRESS> (==0.9.*). David Johnson<EMAIL_ADDRESS>@dmjio. @dmjio. Used by: library
[ ] hailgun-<IP_ADDRESS> (>=0.8 && < 0.10). Dennis Gosnell<EMAIL_ADDRESS>@cdepillabout.. Used by: library
[ ] hjsonschema-1.7.0 (>=0.8 && < 0.10). Ian Grant Jeffries<EMAIL_ADDRESS>@seagreen. @seagreen. Used by: library
[ ] hoauth2-1.4.0 (==0.9.*). Haisheng Wu<EMAIL_ADDRESS>@freizl. @freizl. Used by: library, executable
[ ] jsaddle-<IP_ADDRESS> (>=0.8.6 && < 0.10). Jens Petersen<EMAIL_ADDRESS>@juhp. @ghcjs. Used by: library
[ ] katip-elasticsearch-<IP_ADDRESS> (>=0.8 && < 0.10). Ozgun Ataman<EMAIL_ADDRESS>@ozataman.. Used by: library, test-suite
[ ] monad-http-<IP_ADDRESS> (>=0.8.6 && < 0.10). Oleg Grenrus<EMAIL_ADDRESS>@phadej. @futurice. Used by: library
[ ] pinboard-<IP_ADDRESS> (>=0.8 && < 0.10). Jon Schoning<EMAIL_ADDRESS>@jonschoning. @jonschoning. Used by: library
[ ] rakuten-<IP_ADDRESS> (>=0.9.1 && < 0.10). Nobutada Matsubara<EMAIL_ADDRESS>@matsubara0507. @matsubara0507. Used by: library, test-suite
[ ] ratel-0.3.5 (==0.9.*). Taylor Fausak<EMAIL_ADDRESS>@tfausak. @tfausak. Used by: library
[ ] rest-client-<IP_ADDRESS> (>=0.7 && < 0.10). Silk<EMAIL_ADDRESS>@bergmark @hesselink. Used by: library
[ ] rest-wai-<IP_ADDRESS> (>=0.8 && < 0.10). Silk<EMAIL_ADDRESS>@bergmark @hesselink. Used by: library
[ ] scotty-0.11.0 (>=0.8.2 && < 0.10). Andrew Farmer<EMAIL_ADDRESS>@xich. @RyanGlScott @xich. Used by: library, test-suite
[ ] sensu-run-0.4.0 (>=0.9.1 && < 0.10). Mitsutoshi Aoe<EMAIL_ADDRESS>@maoe. @maoe. Used by: executable
[ ] servant-0.11 (>=0.8 && < 0.10). Haskell Servant<EMAIL_ADDRESS>@SoenkeHahn @jkarni @phadej. Used by: library
[ ] servant-auth-cookie-<IP_ADDRESS> (==0.9.*). Al Zohali<EMAIL_ADDRESS>@zohl. @zohl. Used by: library
[ ] servant-client-0.11 (>=0.8.6 && < 0.10). Haskell Servant<EMAIL_ADDRESS>@SoenkeHahn @jkarni @phadej. Used by: library, test-suite
[ ] servant-mock-0.8.2 (>=0.8 && < 0.10). Haskell Servant<EMAIL_ADDRESS>@SoenkeHahn @jkarni @phadej. Used by: library
[ ] servant-server-0.11 (>=0.8 && < 0.10). Haskell Servant<EMAIL_ADDRESS>@SoenkeHahn @jkarni @phadej. Used by: library, test-suite
[ ] servius-<IP_ADDRESS> (< 0.10). Michael Snoyman<EMAIL_ADDRESS>@snoyberg. @snoyberg. Used by: executable
[ ] strive-4.0.0 (==0.9.*). Taylor Fausak<EMAIL_ADDRESS>@tfausak. @tfausak. Used by: library
[ ] wai-middleware-static-0.8.1 (>=0.8.2 && < 0.10). Andrew Farmer<EMAIL_ADDRESS>@xich. @RyanGlScott @xich. Used by: library
[ ] web-routes-0.27.12 (>=0.6 && < 0.10). Jeremy Shaw<EMAIL_ADDRESS>@stepcut. @stepcut. Used by: library
[ ] web-routes-wai-0.24.3 (>=0.6 && < 0.10). Jeremy Shaw<EMAIL_ADDRESS>@stepcut. @stepcut. Used by: library

I'm a little frustrated that a package that so many people depend on released a major version to fix the capitalization of a joke status code. 😞 https://github.com/aristidb/http-types/issues/68 https://github.com/aristidb/http-types/compare/0.9.1...f5b4b24

hailgun has been updated for http-types-0.10 in version <IP_ADDRESS>:
https://bitbucket.org/robertmassaioli/hailgun/pull-requests/23/remove-upper-version-bound-on-http-types/diff

hackernews has been updated to accomodate the teapot fix.
On Sun, Oct 22, 2017 at 8:43 AM, Dennis Gosnell<EMAIL_ADDRESS>wrote:

hailgun has been updated for http-types-0.10 in version <IP_ADDRESS>:
https://bitbucket.org/robertmassaioli/hailgun/pull-
requests/23/remove-upper-version-bound-on-http-types/diff
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fpco/stackage/issues/2976#issuecomment-338487064, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA1bPBT9Q35ra_-R6BGwTAYCCgdr8-PAks5su2KmgaJpZM4QByRj
.

--
Cell: 1.630.740.8204

Fixed in IPv6DB-0.2.3

Fixed chatwork and rakuten

servant, servant-server, servant-client, monad-http and github updated

servant-auth-cookie has been updated in version <IP_ADDRESS>.

Fixed in https://hackage.haskell.org/package/sensu-run-<IP_ADDRESS>.

hoauth-1.5.0

Aforementioned servant-auth-cookie's version <IP_ADDRESS> was deprecated.
Updated versions are <IP_ADDRESS> and 0.6.0.

New entry:

influxdb-1.2.2 (>=0.8.6 && < 0.10). Mitsutoshi Aoe<EMAIL_ADDRESS>@maoe. @maoe. Used by: library

https://hackage.haskell.org/package/influxdb-<IP_ADDRESS>

FYI, there is http-types-0.11

As you upgrade to http-types-0.11, I recommend maintaining compatibility with http-types-0.10 if possible, since that is probably the version that will make it into the next LTS series.

The only real difference is the elimination of the blaze-builder dependency
which has been a redirect to bytestring's Builder for a long time so having
a package work with both 0.10 and 0.11 should not be too difficult.
Dan Burton<EMAIL_ADDRESS>schrieb am So., 10. Dez. 2017, 21:58:

As you upgrade to http-types-0.11, I recommend maintaining compatibility
with http-types-0.10 if possible, since that is probably the version that
will make it into the next LTS series.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fpco/stackage/issues/2976#issuecomment-350581060, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAB3-EgdnfIEfFJpga9tVaXkouf14aGqks5s_EXggaJpZM4QByRj
.

I've updated the issue's description to reflect new upper bound issues with http-types 0.11.


Prices are all 0

Recently TCG Player have changed the way they are presenting prices, and it seem that this has affected your API, as all prices are showing as 0.
https://api.deckbrew.com/mtg/cards/tarmogoyf
Is showing
"price": {
"low": 0,
"median": 0,
"high": 0
},

Sadly, TCGPlayer cut off my access to their prices, hence the zeros. I'd love to add the prices back, but I need another source.


Creating global context before window contexts

Just a tiny one, this fixes the issue with the error thrown when a window that was left open during the previous session is closed after re-opening the editor. We should make a pull request back to the original unreal-imgui integration with these changes.

@tia-ylinen if you want to put together a short explanation then yeah we should just get this PR'd to the upstream project.


[FloatingActionButtonLocation] Add diagrams to documentation of FloatingActionButtonLocation

Description
This PR adds diagrams showing the usage of FloatingActionButtonLocation.
The linked diagrams in the docs will be available once https://github.com/flutter/assets-for-api-docs/pull/123 is merged.
Related Issues
Fixes #24732.
Tests
No tests, since it is just some additions to the documentation.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

@RoyARG02 can you update your branch with the latest from master? The failing checks here are not caused by your PR.

@Piinks I've updated the branch.

Hey @Piinks sorry to bug you, but the docs aren't showing as expected in the website:


Refer to other class members in a class

When you do something like the following:
class Example
  field1: 1
  field2: Example.field1 + 1

You get a nil pointer error on the third line since Example is nil.

This is an expected behavior since you can't do the same with Lua table.
local Example = {
  field1 = 1,
  field2 = Example.field1 + 1 -- Example is nil here
}

But with class from Yuescript you can do it in the class constructor to initialize the fields
class Example
  new: =>
    @field1 = 1
    @field2 = @field1 + 1


cannot  setup.py install

/home/jianrong/miniconda3/envs/py3.8/bin/python setup.py install
WARNING:root:In order to use the mouse gastrulation seqFISH datsets, please install squidpy (see https://github.com/scverse/squidpy).
/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/scvi_tools-1.0.3-py3.8.egg/scvi/_settings.py:63: UserWarning: Since v1.0.0, scvi-tools no longer uses a random seed by default. Run scvi.settings.seed = 0 to reproduce results from previous versions.
self.seed = seed
/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/scvi_tools-1.0.3-py3.8.egg/scvi/_settings.py:70: UserWarning: Setting dl_pin_memory_gpu_training is deprecated in v1.0 and will be removed in v1.1. Please pass in pin_memory to the data loaders instead.
self.dl_pin_memory_gpu_training = (
Traceback (most recent call last):
File "setup.py", line 9, in
from scarches import author, email, version
File "/home/jianrong/biosoft/scarches/scarches/init.py", line 1, in
from . import dataset, metrics, trainers, models, zenodo, plotting, utils, classifiers
File "/home/jianrong/biosoft/scarches/scarches/models/init.py", line 1, in
from .trvae.trvae import trVAE
File "/home/jianrong/biosoft/scarches/scarches/models/trvae/trvae.py", line 9, in
from .losses import mse, mmd, zinb, nb
File "/home/jianrong/biosoft/scarches/scarches/models/trvae/losses.py", line 1, in
from scvi.distributions import NegativeBinomial
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/scvi_tools-1.0.3-py3.8.egg/scvi/init.py", line 11, in
from . import autotune, data, model, external, utils, criticism
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/scvi_tools-1.0.3-py3.8.egg/scvi/autotune/init.py", line 1, in
from ._manager import TuneAnalysis, TunerManager
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/scvi_tools-1.0.3-py3.8.egg/scvi/autotune/_manager.py", line 11, in
from chex import dataclass
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/chex-0.1.8-py3.8.egg/chex/init.py", line 17, in
from chex._src.asserts import assert_axis_dimension
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/chex-0.1.8-py3.8.egg/chex/_src/asserts.py", line 26, in
from chex._src import asserts_internal as _ai
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/chex-0.1.8-py3.8.egg/chex/_src/asserts_internal.py", line 34, in
from chex._src import pytypes
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/chex-0.1.8-py3.8.egg/chex/_src/pytypes.py", line 19, in
import jax
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/jax-0.4.14-py3.8.egg/jax/init.py", line 35, in
from jax import config as _config_module
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/jax-0.4.14-py3.8.egg/jax/config.py", line 17, in
from jax._src.config import config  # noqa: F401
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/jax-0.4.14-py3.8.egg/jax/_src/config.py", line 27, in
from jax._src import lib
File "/home/jianrong/miniconda3/envs/py3.8/lib/python3.8/site-packages/jax-0.4.14-py3.8.egg/jax/_src/lib/init.py", line 44, in
minimum_jaxlib_version: str) -> tuple[int, ...]:
TypeError: 'type' object is not subscriptable

Hi, @jianrong0520 jax probably dropped support of python 3.8, you will need to try with 3.9 at least.

Thanks, I would like to try it.

Hi, @jianrong0520 jax probably dropped support of python 3.8, you will need to try with 3.9 at least.

the official docs suggest using 3.7 or 3.8 though... confusion

any updates？I have the same problem

@Koncopd  @chelseabright96 can you plz help

Hi @gaoyifei1122, it seems the docs have not been updated. Thanks for brining this up! scArches works fine with python 3.9 and up. Please write if you come across any issues.


Set Panel State to Hidden not working when panel contains ListView with one item

I use SlidingUpPanel with ListView inside it. Sometimes this ListView contains only one single item.
The workflow is:

the initial state is HIDDEN
change ListView content
set state COLLAPSED
user can drag panel up
set state HIDDEN when needed (here is the problem)

If ListView contains two or more items, SlidingUpPanel hides correctly with animation. But if there is only one item then:

setting state COLLAPSED causes state ANCHORED (but no anchor point was setted)
setting state HIDDEN not working, panel stays in ANCHORED state, method onPanelHidden not called

+1
I've been having the same issue for a while too using a RecyclerView even with multiple items.

I encountered this problem form version 3.3.0 to 3.3.1.
I want to hide the panel when no data is present , but it's still impossible.........


Downstream skip SSL/certificate validation per configuration

Expected Behavior / New Feature
Using ocelot in a development/test environment without valid certificates / self signed certificates failes with:
System.Net.Http.CurlException: Peer certificate cannot be authenticated with given CA certificates
Actual Behavior / Motivation for New Feautre
It would be nice to turn off SSL/certificate validation eather per route or globally.
  "ReRoutes": [
    {
      "DownstreamPathTemplate": "/api/people",
      "DownstreamScheme": "http",
      "DownstreamSkipSslValidation": true,
      "DownstreamHostAndPorts": [
        {
          "Host": "localhost",
          "Port": 8238
        }
      ],
      "UpstreamPathTemplate": "/api/people",
      "UpstreamHttpMethod": [ "Post" ]
    }
  ],
  "GlobalConfiguration": {
    "BaseUrl": "http://localhost:18970",
    "SkipSslValidation": true
  }
}

Which would lead to the setup of the HttpClientHandler in Ocelot.Requester.HttpClientBuilder.
if (request.DownstreamReRoute.HttpHandlerOptions.SkipSslValidation)
{
    _httpclientHandler.ServerCertificateCustomValidationCallback = (message, cert, chain, errors) => true;
}

Steps to Reproduce the Problem

use a downstream https route that uses a invalid certificate.

Specifications

Version: latest

@Burnaster Thanks for your interest in the project. I think you can just do this in your startup.cs / program.cs
ServicePointManager.ServerCertificateValidationCallback += (sender, certificate, chain, sslPolicyErrors) => true;

let me know if that doesn't help!

@TomPallister Could it be that the behaviour on linux is different?
Error making http request, exception: System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.CurlException: SSL connect error
at System.Net.Http.CurlHandler.ThrowIfCURLEError(CURLcode error)
...

@Burnaster it looks like it is supported here. Does it not work for you?

@TomPallister unfortunately it did not. But also the validation callback on the HttpClientHandler did not work. Im still a bit confused with the use of dotnet core on linux and cURL to handle HttpClient requests.
To not be blocked, i use http on the ocelot api gateway. But i will update this issue if i find out something worth shareing.

@Burnaster try this article https://www.humankode.com/asp-net-core/develop-locally-with-https-self-signed-certificates-and-asp-net-core
Self signed should work, I guess you need to get your computer to trust the certificate still.

我也想问，怎么加载 https

Maybe the new 2.1 handler will fix this issue.

@Burnaster btw did you ever test to see if this works when calling your service without Ocelot? E.g. just using a console app using HttpClient for example?

@TomPallister no i did not test that. as mentioned earlier i plan to do additional tests on multiple platforms/frameworks.

@Burnaster cool, might be worth test to see if this is an Ocelot problem, I will have a try myself when I get a chance!

@TomPallister I did some early tests. I executed my test program (see below) on a windows 10 64bit and on a cloud foundry app (cflinuxfs2: see https://github.com/cloudfoundry/cflinuxfs2) to call a untrusted HTTPS with a GET request. One time using the ServerCertificateCustomValidationCallback on the  ServicePointManager callback and one time using it on a HttpClientHandler.

System
Method
Outcome

Win64 .net core 2.0.6
Handler
200=OK

Win64 .net core 2.0.6
SPM
Error: System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.WinHttpException: Es ist ein Sicherheitsfehler aufgetreten

cflinuxfs2 .net core 2.0.6
Handler
200 = OK

cflinuxfs2 .net core 2.0.6
SPM
System.Net.Http.HttpRequestException: An error occurred while sending the request. ---> System.Net.Http.CurlException: Peer certificate cannot be authenticated with given CA certificates

Next would to test the same on dotnet core 2.1, the release should be close so i think i wait and do the tests with the release instead of a preview. What would also be interessting is to repeat on a different linux. As far as i understood the behavior is based on the cURL build (with or without openssl, etc.) used in the particular distro.
Can you reproduce my results?
using System;
using System.Net;
using System.Net.Http;
using System.Threading.Tasks;

namespace HttpClientSkipSsl
{
    class Program
    {
        private const string RequestUri = "https://untrustedurl";

        static void Main()
        {
            RunAsync().GetAwaiter().GetResult();
        }

        private static async Task<HttpStatusCode> GetWithHttpHandler()
        {
            using (var httpHandler = new HttpClientHandler())
            {
                httpHandler.ServerCertificateCustomValidationCallback = (mst, crt, chn, plc) => true;
                var http = new HttpClient(httpHandler);
                var response = await http.GetAsync(RequestUri);

                return response.StatusCode;
            }
        }

        private static async Task<HttpStatusCode> GetWithSpm()
        {
            ServicePointManager.ServerCertificateValidationCallback += (sender, certificate, chain, sslPolicyErrors) => true;

            var http = new HttpClient();
            var response = await http.GetAsync(RequestUri);

            return response.StatusCode;
        }

        private static async Task RunAsync()
        {
            try
            {
                var statusWithHandler = await GetWithHttpHandler();

                Console.WriteLine($"Get invalid HTTPS url with HttpHandler status code: {statusWithHandler}");
            }
            catch (Exception exception)
            {
                Console.WriteLine($"Get invalid HTTPS url with HttpHandler error: {exception.Message}");
            }

            try
            {
                var statusWithHandler = await GetWithSpm();

                Console.WriteLine($"Get invalid HTTPS url with SPM status code: {statusWithHandler}");
            }
            catch (Exception exception)
            {
                Console.WriteLine($"Get invalid HTTPS url with SPM error: {exception}");
            }
        }
    }
}

@Burnaster thanks for this testing. This is also what I see. I need to expose the handler in some way from Ocelot for it to work with I trusted SSL. Though I’m not sure if giving that option is a good idea. What do you think? Is it not possible to get your certificate trusted so SSL works normally?

@Burnaster I have done a quick change for ignoring SSL warnings, you can see it here if you want to make sure it works. I have tested and am confident it does ;) There are even some acceptance tests for it now!

@Burnaster I have released this change in version 5.5.5 so you should be able to ignore ssl errors now. Let me know if you still have problems.


Add Windows runners compatibility

This PR does 2 things to make this awesome action work on Windows runners:

Changes old way of getting action path to new way (https://docs.github.com/en/actions/learn-github-actions/variables)
Forces UTF-8 mode in bash on Windows.

All tests are passing on my Windows + macOS self-hosted runners.

Thanks for your contribution @b0bben !   My apologies it has taken over a month to review & merge


Update aiidalab tools package version to 22.11.0

Is there a url with a site build from this PR? Or would I have to build it locally to test it out?

Sadly, there is no demo site built from this PR. But I think the site itself didn't show many clues if the build is correct. If the schema is all good, the ci test is pass.


Copy docs to master: prep for changing default branch

First part of https://github.com/dotnet/source-build/issues/1919. Copying latest docs to master for maintenance, and include a note about release branches because master will be trimmed down to provide (only) a few dependencies for arcade-powered source-build.

You may want to turn off CI for the Documentation folder in master as well.

It looks like it's actually turned off everywhere, but README.md isn't. I'll add that.

You may want to turn off CI for the Documentation folder in master as well.

It looks like it's actually turned off everywhere, but README.md isn't. I'll add that.

Reverted the changes to make README.md about arcade-powered source-build: should be good to merge now, leaving master as 5.0 for now but still moving the docs over.

Reverted the changes to make README.md about arcade-powered source-build: should be good to merge now, leaving master as 5.0 for now but still moving the docs over.


[flake8-pyi] Implement autofix for redundant-numeric-union (PYI041)

Summary
This PR adds autofix for redundant-numeric-union (PYI041)
There are some comments below to explain the reasoning behind some choices that might help review.

Resolves part of https://github.com/astral-sh/ruff/issues/14185.
Test Plan

Oh, and did you look through the ecosystem changes? If so, could you update the test plan. It's helpful to know if you did because I then don't need to carefully review all of them and can instead just click through some of them.

Oh, and did you look through the ecosystem changes? If so, could you update the test plan. It's helpful to know if you did because I then don't need to carefully review all of them and can instead just click through some of them.

The ecosystem results look good.
From this example I'm wondering if we shouldn't include return types; it appears to skip that one now.
Typeshed shows negatives in the ecosystem results, but I expect that to not be related to the PR. Maybe @AlexWaygood can help out with that.

From this example I'm wondering if we shouldn't include return types; it appears to skip that one now.

Could you expand what you mean by it appears to skip that now. Is it not flagging this case or do you think this case shouldn't be flagged?

Typeshed shows negative in the ecosystem results, but I expect that to not be related to the PR.
~Maybe @AlexWaygood can help out with that.~
It's probably because the example is Py3.13+

The ecosystem reports for typeshed have often not been reproducible locally and nobody's yet had a chance to look into why... if you can't repro the typeshed hits locally, I'd just ignore them!

Could you expand what you mean by it appears to skip that now. Is it not flagging this case or do you think this case shouldn't be flagged?

After investigating it turned out that this rule hooked into statement.rs instead of expression.rs like all the other rules related to unions do. This caused false negatives on the return types and type aliases. We had no test for this. Fixed it, and added a test.


Editor component MVP follow-up

[ ] Typing ">" + space should create a new collapsible element or turn an existing block into a collapsible element.
[ ] Let's remove the full 2-pane layout for now and just allow PDFs to open into a separate read-only pane. We can revisit more complex layouts later when we're ready for more testing. Alternatively, we could consider adding a PDF reader as a 3rd element on the left sidebar, though that might be more complicated for now...
[ ] Bullet and numbered list don't work right now. Apparently it's just a CSS thing but it would be good to polish the editor experience a bit now.
[ ] The 6-gray dot block handle should only appear on hover.

New issue I encountered today: if I select all on the text in an editor window and then paste text over it, the editor window appears blank, but the text content is there, as it can be highlighted and shows up in the AI interactions window as selected text. Video below.
https://github.com/refstudio/refstudio/assets/15233/b7f0dc0d-845b-4265-aa12-c423e6e7ac32

Let's remove the full 2-pane layout for now and just allow PDFs to open into a separate read-only pane. We can revisit more complex layouts later when we're ready for more testing. Alternatively, we could consider adding a PDF reader as a 3rd element on the left sidebar, though that might be more complicated for now...

#105 should make side-by-side editors work properly, except when the same file is open twice (technically both editors work but the content is not synchronised). Maybe what we could do for now is just preventing opening the same file in two panes, I think it would be less work rather than having to redesign the internal state to have a write-only pane and a read-only pane.

👏🏽


Improve Mono portability

Remove workarounds that are no longer needed on Mono master and make the remaining compile-time check a runtime check so the assembly is portable.

@mhutch,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Looks good. You want me to merge it in?

Yes please!

Squashed and merged.


fix: update opentelemetry dependencies to patch security vulnerability

Update dependencies to address security vulnerability opentelemetry-instrumentation Denial of Service vulnerability due to unbound cardinality metrics referred to in Issue #528.

Do not merge before:
https://github.com/lumigo-io/tracing-ingestion/pull/1529
https://github.com/lumigo-io/integration-tests/pull/1544


3.7.1

@ManishDait, can you also add new language to README.md? And format README.md with default rules in Android Studio.


Add support/checks for symlinks

Fixes #205
This should fix symlink issues especially on linux and probably other platforms, as they are working fine in windows.

agreed to not merged, because this is most likely a user problem


fix mouse button handling

fixes crash caused by releasing a main mouse button while having a now main button pressed.
Clicks and double clicks are now handled for all the buttons and not only for the main ones.
Double click will be fired for any combination of 2 fast mouse button clicks.

Is this acceptable? the result is that in the current lazer client all clickables can be activated with any mouseclick since there are never checks on which mouse button it comes from, but it looked weird that mouseclick would only be handled for main buttons, the mousedown event is already handled for any button this means that in the current lazer client hitcircles are clickable with any button

https://puu.sh/sVB0y/9853de3e7d.png should I trigger a different event in a Drawable for left(or main?-> in stable everything works with both left and right as far as I can tell) mouse like this. Or already trigger a separate event in the inputmanager. Or let as is and always check in the client which mouse button triggered it.

I'm going to close this one but implement an equal solution which will fix your issue. I believe only


Fix timestream endpoint configuration

Feature or Bugfix

Bugfix

Detail

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: b12165bd0a6411db27abd78fbde52ed6400dbad7
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: b12165bd0a6411db27abd78fbde52ed6400dbad7
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Submission: Sample Project

Project: Hugging Face Chat UI
Demo: https://huggingface.co/chat/
Brief Description: Full featured chat interface
Architecture: SvelteKit with MongoDB
Conversation Context Carry: 🟢
Conversation History: 🟢
Authentication: OpenID
Models support: Hugging Face Inference API, local, Amazon SageMaker
Rich text support: 🟢
Image support: 🔴
Multi-step prompting (ReAct, CoT, etc): 🔴
(RAG) Data source(s): 🟢  Google search
(RAG) Show sources: 🟢
Quick deploy: 🟢 Hugging Face Spaces
Other features: Theme configuration

Full Description: Blah blah blah

Closing since sample issue replaced with templated Issue (thanks @zbloss)


Leaf icon

Hi,
Amazing job, so simple but nice.
I would like to draw an icon on some nodes, roots, nodes or leafs. Is it possible to do it ? Should I simply use the slots before the text ?

i really want this feature

I think this solution can help for now

please read documentation before ask questions
https://n00ts.github.io/vue3-treeview/guide/properties.html#icons


Allow for scrollable terra form input panel

Feature Request
Description

Right now terra form input allows for a variable height dropdown which scrolls. However when the panel height is attempted to be modified with styling, no scroll appears in the panel to show all the selected contents. Would like a way to change the display of the panel from hidden with props in order to allow scrolling in the panel.
Additional Context / Screenshots

Use case:
My UX designer would like to limit the input panel AND the dropdown to 4 lines each. when more than 4 things are selected it is desired a scorll bar appears inside the input panel and selected elements can be scrolled
@ Mentions

Huh? Completely confused.

This screenshot will hopefully explain it better, My UX designer would like the ability to lock the input field height to something like a maximum of 4 lines long. Once that limit is reached, they would like a scrollbar to appear inside the panel.

We'd recommend having your designer reach out to OCS to see if they have designs for this. They may have a pattern to handle this use case.


Fix syntax highlighting for markdown code blocks on issue description/comment save

Looks like I missed something in #2538. Markdown code block syntax highlighting did still not work when updating an issue description or comment. Is fixed now.
Before:

After:

Thanks!


commit

edited index.html file

This seems to have some merge conflicts because we merged earlier pull request. If you have time to
solve the conflict we happily merge it :)


[Stepper] Allow custom step connector.

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Allows the use of a custom step connector. See #5740.

Thanks.


Add Name or Labels to table of Account ID and Region

Feature name
Sector Names
Is your feature request related to a problem? Please describe.
There is no field to provide a custom name or label to each combination of Account ID  and Region. So if/when a Perspective user Imports multiple Regions across multiple Accounts, they must memorize or maintain a separate cross-reference of each Account ID. Additionally, the class of sets of accounts and regions does not have a name. Therefore it is difficult to specify a unique name to refer to each instance/combination by.
Describe the feature you'd like to see implemented
Each item in a table of accounts and regions should be referred to as a Sector, and each Sector should have a unique name. A sector name could start as basic as {account-id}-{region}, but ideally the Perspective user could provide an arbitrary name like TestAccount-OR, or ServiceName-Prod1.
Describe the value this feature will add to AWS Perspective
Providing a Name or Label field for each sector will greatly improve the user experience for Perspective users with multiple Accounts and/or regions, by rasing the cognitive burden of correlating which Sector in the list contains which business functions and/or resources.
Describe alternatives you've considered
The most important part of this request is the ability to add a custom Name or Label to each entry in the table of "Accounts & Regions". It is less important to me what this solution "Feature" is named. Instead of "Sector", each set of account ID and region could also be called a "set", or compartment.

This is a great suggestion.

This feature was added in v1.1.0.


[Schemas] Utility functions to create Avro schemas

Feature description
Part of https://github.com/ConduitIO/conduit/issues/1560.
Creating an Avro schema can be verbose. We should provide utility functions that make it easier to build an Avro schema.

Closing in favor of https://github.com/ConduitIO/conduit-commons/issues/52.


README.md: Added note about how to post JSON

Credits go to Jason Worley who wrote about this on his blog
(I cleaned up some string quote characters at the same time.)

By the way, I prefer you to create new branch for your repository rather than rack-test/rack-test repository, if you do not mind it.
Because we have to mind cleaning unused branches.

Sure, that's fine. The problem is that the GitHub web editor creates a branch in this repo if you edit the file online, since all of us have write access to the repo. I don't know if that setting can be changed, but it is what caused this to happen.
Regarding deleting the branch: that's fine, I will press the Delete button now right after merging it. 😄


Feat/rn update

Description

Upgrade to RN 72

Breaking Changes

TBC

I have tested my changes on:
ReNative project directly:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

New project:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

Existing Project created with previous version of renative:

[ ] ios simulator
[ ] ios device
[ ] android simulator
[ ] android device
[ ] web browser
[ ] tvos simulator
[ ] tvos device
[ ] androidtv simulator
[ ] androidtv device
[ ] androidwear simulator
[ ] androidwear device
[ ] tizen simulator
[ ] tizen device
[ ] tizenmobile simulator
[ ] tizenwatch device
[ ] webos simulator
[ ] webos device
[ ] macos
[ ] windows
[ ] chromecast device

now part of https://github.com/flexn-io/renative/pull/1113


Update Fantasy.css

<IP_ADDRESS> update to the coding and have Online and offline server groups different colors and added the second for the instances

fantasy amp theme is nice, but what this?


suggestion: different names for the config and opml files

config.yaml isn't a particularly friendly name for the config file.  How about placing it in $XDG_CONFIG_DIRS/matcha/config.yaml by default and allow override on the command like.  Also speficy the opml file name in config and also allow override from command line.

Yes, overriding from the command line of the opml and yaml files is definitely on the roadmap and a great suggestion!
The reason why its located next to the app (current dir) is that since the app doesn't have a GUI (to modify these configs) it has to be in a very visible place and beginner friendly. If we add GUI for the configuration then it will definitely be "somewhere hidden" like XDG_CONFIG_HOME

Added -c file_path will overwrite default config.yaml configuration file
Add -o option to append more feeds through an opml file
Thank you for your suggestions!


Fatal error when generating README

So this happens if the changelog has a line which begins with a backtick
Example:
v0.10.0:
  date: 2014-02-07
  changes:
    - `sourceMappingURL` calculated correctly.

Running "build-contrib" task
Fatal error: Unable to parse "CHANGELOG" file (undefined).

OK, it's due to grunt.file.readYAML. A backtick cannot start the line as per the YAML specs.
Shouldn't we use markdown instead of yaml?

As a workaround I found that it works if I wrap the whole thing into double quotes, like
v0.10.0:
  date: 2014-02-07
  changes:
    - "`sourceMappingURL` calculated correctly."

Please use yaml for now

All right.


Accessing the UI

Hi,
Sorry I know this is going to be an incredibly stupid question but I am much more marketer than I am developer so please stay with me here :)
I've gone ahead and run the appropriate install commands on a brand new Ubuntu 16.04 x64 VPS (with Google Cloud if that is of any relevance).
Everything seems to be working well so far and running curl -i http://localhost:8080/api/v1/ping in my SSH terminal works as expected. What I can't figure out however is how do I access the web based UI.
Going to http://$IP_OF_MY_VPS_BOX:8080 simply times out.
Assuming this is some kind of firewall issue? Any thoughts? What am I missing?

Just wanted to post a quick update / feedback.
Updating the firewall rules to allow connections to port 8080 did indeed help. However it gets me this:

Still no UI unfortunately. Also just in terms of a possible update to the bootstrap script. Would it make sense to check if the appropriate ports are open and if not open them so that the web interface is available?

I think you are on the right track, that it is access to the machine from outside the machine that is the problem. A couple things to try would be to curl against http://localhost:8080 from your ssh terminal, just to check if the UI is in the right place and being served. If that seems ok (you get back some HTML), I would try the same ping URL from the browser, replacing localhost with the ip you referenced earlier. If that is equally unsuccessful, maybe try a curl -i from a terminal window off the machine, like where you're running your browser. That might give you more info about the access problem.

@longdogz Thanks for the reply. Can confirm that using curl  http://localhost:8080 shows the same HTML as shown in the picture above (i.e. a directory containing swagger) but can't seem to find the actual UI at all unfortunately.

Ok, I'll have to do some investigation to see why you might be getting that. What commands did you use for the install and were there any errors?

Full extend of commands on a brand new Ubuntu 16.04 x64 VPS are as follows:
% sudo apt-get install git
% git clone https://github.com/intuit/wasabi.git
% cd wasabi
% ./bin/wasabi.sh bootstrap
% sudo reboot
% ./bin/wasabi.sh build start

Followed by testing that it was generally working using:
% curl -i http://localhost:8080/api/v1/ping
Opened up the 8080 port on the machine and attempted to visit the URL in the browser on port 8080 when I just got the swagger directory
Once more attempted to reboot the VPS and then rerun
% ./bin/wasabi.sh start
Same results unfortunately. No error messages shown but running commands such as
% ./bin/wasabi.sh resource:ui
do produce an error as shown below:

So, you did the build on the Ubuntu machine.  To build the UI, that would need to, at some point, do a command like this:
cd ./modules/ui && npm install && bower install && grunt build
Which is really four separate commands.  So you cd to the modules/ui directory within your clone of the wasabi repo.  In there you run the command npm install.  That should use the package.json file (in that directory) to pull down a bunch of dependent JavaScript libraries, mainly for use in building the UI.  The next command, bower install, pulls in some JavaScript that is used at runtime by the UI application and will be packaged up and served with the wasabi UI code.  Finally, the grunt build will do that packaging (and a bunch of other stuff) and will produce a dist directory beneath the ui directory.
If you run into errors with any of that (it looks like you are getting npm errors from your screen shot), you would need to determine what the problems are and fix them.  Unfortunately, I am not sure what the problem is, but perhaps someone in the community might be?  @jwtodd ?  😄 It is possible, since it is complaining that you don't have "make", that you are missing some basic tools for software development (or that they are not on your "path") which maybe our scripts assume you would have?  While npm mainly downloads JavaScript libraries, sometimes those are just wrappers around programs (like C programs) and if those aren't existing, it will try to build them.  That might be what is failing.  If you google the failures, you might get a clue as to what to do to fix your system so the build will work.
Sorry, but I don't have your system available and am not sure what might be going wrong with the npm install.
If you can get that fixed, though, you should be able to try running the other commands.  When they complete, you will have several files and directories in the modules/ui/dist directory.  Those basically get copied to the /usr/local/wasabi-main-1.0.20170113205431-SNAPSHOT-development/content/ui/dist directory on your server system (which is then served by the same Jetty server that serves the REST API, hence the :8080).  If you can successfully build the UI dist directory and copy those files, the UI should run.  I think if you can fix the problems with your UI build, you could also just run one of those wasabi.sh commands, too, like re-do ./bin/wasabi.sh build start ?

Maybe this helps: sudo apt-get install build-essential. I found a stackoverflow question that at least looked kind of similar to your error and is also dealing with an ubuntu system: npm failed

Just as another update. I made sure to run the following first
sudo apt-get update
sudo apt-get upgrade
sudo apt-get install build-essential

This does get me a working login screen at http://$IP_OF_MY_VPS_BOX:8080/dist
However, attempting to sign in gets me this:

Despite having a functional server as far as I can tell:

OK, I actually probably know this one.  😄
The UI is a JavaScript, single-page app written in Angular JS.  It uses REST APIs in the Wasabi server to implement the UI, e.g., to retrieve a list of experiments, create experiments, etc.  In order to do that, it needs to know the URL of the REST server.  It determines that from a configuration file.  That file is at (in the deployment) at /usr/local//content/ui/dist/scripts/config.js .  It contains some JavaScript that sets a constant named "apiHostBaseUrlValue" to a value like "http://localhost:8080/api/v1".  If you change that value to something like "http://$IP_OF_MY_VPS_BOX/api/v1", then the UI should work fine.
BTW, in order to see why the UI wasn't working, you could load it into Chrome (you shouldn't need the /dist, just http://$IP_OF_MY_VPS_BOX) and then right click on the window and choose "Inspect" to get the debugger.  In the console, you should see errors about being unable to find the server http://localhost:8080.

FYI, one of the engineers here reproduced part of your problem, that is, the /dist thing.  Basically, you should get the UI when you go to http://localhost:8080 and not at http://localhost:8080/dist/  .  We are investigating why that is happening.  There would still need to be a change in the config.js file for the UI to work, but it still won't work even if you make that change because of the /dist thing.  We'll let you know what we find out.

Ran into the same issue, also using ubuntu 16.04 on google cloud, but was able to fix it with one file change and a rebuild.
You'll need to change the 'apiHostBaseUrlValue' in /modules/ui/constants.json to your server ip, since it's set as default to "http://localhost:8080/api/v1" which causes the angular app to send all requests to localhost rather that the public server ip. Then you can rebuild with the ./bin/wasabi.sh script with stop, remove, build, and start. This should probably be put into part of a dev/prod workflow, not sure if this is already being considered.

Thank you for describing so clearly how to change the backend API setting
for the UI, which is necessary when you are not accessing the server vial
localhost:8080 , the default.
The main reason we use the constants.json and a build to configure the
backend API is so that you have the flexibility to not be serving the UI
from exactly the same location as the REST APIs.  For example, you might
want to hit the UI on port 80 and the API on port 8080.  Or you might even
want to serve the UI from a totally different server, e.g., during
development, I use the "grunt serve" command, which serves the UI on
localhost:9000 , but I want to hit the REST API server in docker at
localhost:8080 .
It is for these reasons that we don't have a default, for example, of using
the window.location to determine the URL from which the UI code was served
and then assume that is the URL of the REST API.  That would definitely
solve this problem (running the Wasabi servers in a VM and then needing to
load the UI and hit the API at some IP address URL), but we wouldn't want
it to be the ONLY way to configure the backend API URL.
Maybe we could figure out a way to use the URL from which the UI code was
served as the REST API URL, but then have a way to override that when
necessary?  I'll think about that, but would also accept any suggestions.
BTW, just for complete disclosure, changing the constants.json file and
then doing a UI build is one way to change the backend URL used by the UI.
What the grunt build does with that file is to actually produce a file at
modules/ui/app/scripts/config.js that has that information in it and that
is picked up when the UI loads in the browser.  An alternative way to
change the configuration, if, for example, you don't want to do the
re-build to do it, is after you have deployed the UI, just find the
config.js file and change it to have the correct value.
On Mon, Jan 23, 2017 at 2:41 PM Anders Elmgren<EMAIL_ADDRESS>wrote:

Ran into the same issue, also using ubuntu 16.04 on google cloud, but was
able to fix it with one file change and a rebuild.
You'll need to change the 'apiHostBaseUrlValue' in
/modules/ui/constants.json to your server ip, since it's set as default to "
http://localhost:8080/api/v1" which causes the angular app to send all
requests to localhost rather that the public server ip. Then you can
rebuild with the ./bin/wasabi.sh script with stop, remove, build, and
start. This should probably be put into part of a dev/prod workflow, not
sure if this is already being considered.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/intuit/wasabi/issues/163#issuecomment-274641272, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHXiAzw5KEFUnLx6fS0oVycwthETpxniks5rVSyAgaJpZM4Log2_
.

Ran into the issue where UI was appearing in http://localhost:8080/dist instead of http://localhost:8080.
following change will fix it.(https://github.com/intuit/wasabi/pull/194)
"wasabi.sh build" must be run once after making this change.

I ran into this problem on my Mac and when doing the

cd ./modules/ui && npm install && bower install && grunt build

command recommended by @longdogz
It would keep failing on grunt build due to some old libraries and the version of node used by brew (if you keep it updated) being incompatible.
I fixed this by going through every package in package.json and typing "npm install "
It updated each package to the latest and then when I ran grunt build it worked.
Finally from the wasabi directory:
./bin/wasabi.sh build start
worked for me and the UI loaded. Hopefully that helps.

I'm not sure if I mentioned this earlier, but the "npm install" and "bower install" install things locally that are not reinstalled when you do a grunt build, or even if you do another npm install or bower install (which are done by the wasabi.sh at times).  There are possible problems, over time, where there are inconsistencies between what you installed before and new things added to package.json.  If you are having problems with the installs or the grunt build, one thing to try is to do:
cd modules/ui rm -rf node_modules rm -rf app/bower_components
And then to try the grunt build again.
The grunt clean doesn't delete those directories, partly because grunt needs what is in node_modules and partly because usually you don't need/want to re-do them every time you build.

I've pinpointed what the problem seems to be, the bower install is not actually creating the app/bower_components directory and dependencies. I think it's related to https://github.com/bower/bower/issues/50
I'm trying to run through the bower dependencies and find which one is causing the the entire bower_components not to be created. Because there are no bower_components created, when you run grunt build the task fails and doesn't actually create the dist.

@ctrlaltdylan @longdogz I'm having the same issue. Any known work arounds yet?

Adding to @wukaiyuan's response, similar problem for me was related to bower.
bower ESUDO         Cannot be run with sudo
Additional error details:
Since bower is a user command, there is no need to execute it with superuser permissions.
If you're having permission errors when using bower without sudo, please spend a few minutes learning more about how your system should work and make any necessary repairs.

http://www.joyent.com/blog/installing-node-and-npm
https://gist.github.com/isaacs/579814

You can however run a command with sudo using "--allow-root" option

Running the following commands in sequence solved the issue
npm install
bower install --allow-root
grunt build
./bin/wasabi.sh build stop
./bin/wasabi.sh start

Hello,
I had the initial "only swagger" problem. Now I got it to working. Thank you. However, when packing, it doesn't pack the ui. I get "only swagger" again. How do I make it package the ui as well?
Thank you

grunt build
ERROR
Running "concurrent:dist" (concurrent) task
Warning: (node:21734) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead. Use --force to continue.
Aborted due to warnings.
Worked fine with
grunt build --force
not sure why?
Thanks


[Fix-#4866][common] Fix loading hdfs-site.xml

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

What is the purpose of the pull request
This closes #4866

Codecov Report

Merging #4867 (33003aa) into dev (40bb92a) will decrease coverage by 0.11%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                dev    #4867      +/-   ##
============================================
- Coverage     45.66%   45.55%   -0.12%
+ Complexity     3449     3446       -3
============================================
  Files           539      539
  Lines         23218    23218
  Branches       2707     2707
============================================
- Hits          10602    10576      -26
- Misses        11664    11690      +26
  Partials        952      952

Impacted Files
Coverage Δ
Complexity Δ

...che/dolphinscheduler/common/utils/HadoopUtils.java
34.29% <100.00%> (ø)
21.00 <1.00> (ø)

...olphinscheduler/server/registry/HeartBeatTask.java
37.50% <0.00%> (-53.13%)
2.00% <0.00%> (-1.00%)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
3.00% <0.00%> (-1.00%)

...eduler/server/worker/runner/TaskExecuteThread.java
58.53% <0.00%> (-4.88%)
13.00% <0.00%> (-1.00%)

...r/server/worker/processor/TaskCallbackService.java
38.09% <0.00%> (-3.18%)
8.00% <0.00%> (ø%)

Continue to review full report at Codecov.

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


Problems warning status bar item should have a different name than the normal "problems" item

Testing #199247

Remember to put labels and milestones on issues when you fix them so they can be tracked and get verified @justschen


Azure cloud shell getting stack

Say no subscriptions when there is one

Hi @Entreri86 Sorry that you're running into issues.  Could you share some more details about your account/subscription?
Are you using a specific type of Azure account?  Is it Pay-As-You-Go?

Hi! When she was in MS Learn with the azure terminal on one of the path
(AZ-202) she would get hung up. I had to exit the module, enter to activate
another sandbox (learn XXXX-guid) starting the module again to get the
terminal to work again.
El lun, 12 jul 2021 a las 19:10, Nathan @.***>)
escribió:

Hi @Entreri86 https://github.com/Entreri86 Sorry that you're running
into issues. Could you share some more details about your
account/subscription?
Are you using a specific type of Azure account? Is it Pay-As-You-Go?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azcode/issues/28#issuecomment-878447506,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXWFDBRXOUI5I5X5GBOS2DTXMOXDANCNFSM5AFEITFA
.


Textdraws customization

It might be a good idea to add a third option in /style that is "custom" which takes the user to a textdraw editor (similar to the scripts on SAMP forums) which should give them full control over most of textdraws so they can make their very unique style. It'd be much better if implemented per player of course.

May be useful but I don't think many people will use. I wouldn't since the GM style is pretty now (for me).


Fix creating text plugin in wizard

Description
Implement missing part regarding adding text content as a plugin if use_placeholder setting is enabled
References
Fix #607
Checklist

[x] I have read the contribution guide
[x] Code lint checked via inv lint
[x] changes file included (see docs)
[x] Usage documentation added in case of new features
[x] Tests added

Coverage decreased (-0.09%) to 96.32% when pulling 6b676b4cb69b6b1bf8934b9d103310bab995ba60 on bugfix/issue-607-wizard-text into 1433dcd5e8bcf196765f2ca9a3d9118100bc9796 on develop.


Add ability to format Typst file (source code)

Description
I have seen a few Typst formatters being developed, and I don't know which one to choose, probably should wait more for them to mature. But in the future a typst fmt/typst format subcommand should be added just like cargo fmt. In the meantime, it would be nice to at least be able to remove trailing whitespaces. The https://github.com/mhartington/formatter.nvim has a default all-filetype formatting that removes trailing whitespaces which is super useful.
The same goes for https://typst.app.
Use Case
This will make code more readable, consistent with how things are declared/used, and also potentially decrease file size for free (if there are any trailing whitespaces).

Duplicate of #1772


WIP: API: Add support for PEP 519 GH13823

[ ] closes #13823
[ ] tests added / passed
[ ] passes git diff upstream/master | flake8 --diff
[ ] whatsnew entry

needs tests

@AnkurDedania Do you have time to update this and add tests?

@AnkurDedania Do you have time to update this and add tests?

can you rebase / update

TIL how to rebase, if done correctly

It seems the rebase worked!
Can you add some tests?

also need to add a skip_if_no_fspath in pandas.util.testing (again alongside pathlib, py.path ones)

@AnkurDedania ca you rebase / update

@AnkurDedania something went wrong.
try
git rebase -i origin/master

then
git push yourremote yourbrach -f

closing, you can cherry-pick your changes on top of master. ps open a new PR.


TypeScript build task fails when path to tsconfig.json file has spaces

VSCode Version: Code 1.17.2 (b813d12980308015bcd2b3a2f6efa5c810c33ba5, 2017-10-16T13:59:46.104Z)
OS Version: Windows_NT x64 10.0.15063
Extensions: none

Steps to Reproduce:

Create a new folder with an space somewhere in the full path, lets say "D:\Temp\Space here"
Add a dummy.ts file, it could be empty
Add a tsconfig.json file, it could be just a pair of curly braces:
{	}
Press Ctrl+Shift+P and type/select "Task: Configure Default Build Task" then select "tsc: build - tsconfig.json", the following "taks.json" is generated:

	{
			// See https://go.microsoft.com/fwlink/?LinkId=733558
			// for the documentation about the tasks.json format
			"version": "2.0.0",
			"tasks": [
					{
							"type": "typescript",
							"tsconfig": "tsconfig.json",
							"problemMatcher": [
									"$tsc"
							],
							"group": {
									"kind": "build",
									"isDefault": true
							}
					}
			]
	}

Execute the build command with Ctrl+Shift+B shortcut
The following error message is displayed:

	> Executing task: tsc -p "d:\Temp\Space here\tsconfig.json" <

	error TS5042: Option 'project' cannot be mixed with source files on a command line.
	The terminal process terminated with exit code: 1

	Terminal will be reused by tasks, press any key to close it.

Now close VS
Rename the folder to remove the white space
Execute the build command from VS again, now it works:

	> Executing task: tsc -p "d:\Temp\Space_NOT_here\tsconfig.json" <

	Terminal will be reused by tasks, press any key to close it.

Additional information:
Although the displayed command looks fine and will actually work if you Copy&Paste it into a Terminal window:    tsc -p "d:\Temp\Space here\tsconfig.json"
the error message is consistent with what you'd get if you remove the quotes, which suggests they weren't there when the command was actually executed as a task by VS
EDIT: fix markdown issue

This is a little bit a chicken egg problem which we were discussing how to best solve. The problem is that the creator of the task doesn't know  which shell is used to execute hence doesn't really no how to quote / escape these things and the task executing engine doesn't know the structure of the command. The only possible way I can imagine is that the creator of the tasks give hint which parts should be escaped.

Same problem on macOS. As a workaround I'm launching tsc as type shell command:
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "label": "typescript",
    "tasks": [{
        "taskName": "typescript",
        "type": "shell",
        "command": "tsc",
        "problemMatcher": "$tsc",
        "args": ["-p", "\"${workspaceFolder}/tsconfig.json\""],
        "group": {
            "kind": "build",
            "isDefault": true
        }
    }]
}

Fixed in latest.

To verify:

create workspace folder with space in its name
create a tsconfig.json and TS file
execute the tsc build task (no need to create on. It is the one provided by the TS extension)

@dbaeumer I stumbled over this issue aswell and was wondering if that is in insider yet cause even with insider it breaks for me but with a different error:
VS 1.21.1 had:
> Executing task: c:\Dev\chromajs sample\node_modules\.bin\tsc.cmd --watch -p "c:\Dev\chromajs sample\tsconfig.json" <

c:\Dev\chromajs : The term 'c:\Dev\chromajs' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is
correct and try again.
At line:1 char:1
+ c:\Dev\chromajs sample\node_modules\.bin\tsc.cmd --watch -p c:\Dev\ch ...
+ ~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (c:\Dev\chromajs:String) [], CommandNotFoundException
    + FullyQualifiedErrorId : CommandNotFoundException

VS 1.22.0-insider with commit hash that is 19 hours ago (a716373aaaf50cbce3618688a645b324721b35dd):
> Executing task: 'c:\Dev\chromajs sample\node_modules\.bin\tsc.cmd' --watch -p 'c:\Dev\chromajs sample\tsconfig.json' <

At line:1 char:54
+ 'c:\Dev\chromajs sample\node_modules\.bin\tsc.cmd' --watch -p 'c:\Dev ...
+                                                      ~~~~~
Unexpected token 'watch' in expression or statement.
At line:1 char:1
+ 'c:\Dev\chromajs sample\node_modules\.bin\tsc.cmd' --watch -p 'c:\Dev ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
The '--' operator works only on variables or on properties.
    + CategoryInfo          : ParserError: (:) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : UnexpectedToken

Without spaces everything works fine....

I will reopen the issue.

OK. I only tested it with a global installed tsc. Need to dig why --watch now is interpreted as an expression in PowerShell.

OK. If the command is quoted I need to put a & in front :-)

@WolfspiritM thanks for bringing that to my attention


Link to documentation is broken

There is no documentation in the link provided in the README.

Thanks for letting us know!


fix: search panel styles

Types

[x] 💄 Style Changes

Background or solution
before

after

Changelog

improve search panel style

Codecov Report

Merging #640 (5a6d088) into main (6e57e9d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #640   +/-   ##
=======================================
  Coverage   58.98%   58.98%
=======================================
  Files        1209     1209
  Lines       74645    74645
  Branches    15464    15464
=======================================
  Hits        44028    44028
  Misses      27915    27915
  Partials     2702     2702

Continue to review full report at Codecov.

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

LGTM


Fix BROOKLYN-271: Avoiding object hashcode in entities' toString.

Fixing BROOKLYN-271
Avoiding object references in catalog ConfigKey cards, for example:

Last picture shows that default value of softwareProcess.lifecycleTasks config key does not contain the object hashcode @XXXXX. Then, objects toString methods are modified to avoid the hashcode.

@aledsage I think it should be better than https://github.com/apache/brooklyn-docs/pull/68

Thanks @kiuby88 - agreed this will improve the auto-generated entity docs (as per BROOKLYN-271).
In general, I like the toString to give more specifics about the object where possible. That is possible for some of these, and not for others. I'll comment on individual methods where I think we could include more.

LGTM, thanks @kiuby88 - merging. I incorrectly marked BROOKLYN-271 as resolved before this PR (and https://github.com/apache/brooklyn-library/pull/38) were merged - sorry about that, and thanks for spotting!


outside collaborator: facing Concurrency is only available in iOS 15.0.0 or newer issue after cloning

Hi guys
I was trying to clone and compile it as per documentation but it says
Concurrency is only available in iOS 15.0.0 or newer
I have done the steps required in docs

Hey @NaaeemMalik! This is likely related to your Xcode version, you need to be on Xcode 13.2 or later to compile code which uses Swift Concurrency on versions older than iOS 15.
Are you able to check that you're using Xcode 13.2, and if not, update and try again? Let me know if you're still having issues after that. 🙂 Thanks!

Thankyou sooo much for this instant help, I updated the xcode and it fixed the issues


Can we create long term retention backup for sql managed instances using terraform

@TechyMatt @spikewang
AN SQL instance has a default backup of PITR(7 days), but can we create a LTR using terraform.

@mahesh2543 Apologies for the late reply. Thanks for reaching out to us and reporting this issue. Could you please let us know if you the below article serves your requirement ? Awaiting your reply.
Microsoft.Sql managedInstances/databases/backupLongTermRetentionPolicies using Terraform

@mahesh2543 The action is currently pending on you to check if the above suggestion helps. We will now proceed with closure of this GitHub issue. If you need any further assistance on this issue in future, please feel free to reopen this thread. We would be happy to help.


Credentials for external tools are stored unhashed in database

Please use the following template when submitting issues: (If this is a security issue, then review the disclosure policy and submit via our security reporting program.
Issue Template
Issue Type

[ ] Bug
[ x ] Enhancement

Description
When adding a configuration for an external tool, e.g. JIRA, you can store sensitive information such as password, SSH private key, API key in DefectDojo. However, when you take a look in the database afterwards (e.g. via SELECT * FROM dojo_tool_configuration;) you can see that these credentials are stored in clear text, without hashing.
These credentials should instead be hashed with a salt, as is done for the application user's passwords in the table auth_user.
If the issue is a bug, please also include the following:
Operating System
Please include the version number (ex: Ubuntu 16.04)
Install

[ ] setup.bash
[ ] docker

DefectDojo Version
Steps to Reproduce
Sample Scan File (if applicable)
Screenshots (optional)
Console Logs (optional)

@aaronweaver do you know what fields need to be encrypted / have more info so I can give this to the interns?

pycrypto has been replaced.


Add new section on Style

Here we can show examples of using

[ ] Single File Components
[ ] Less, SASS, SCSS, etc.

Also explaining id's, classes and how they can be combined.

Maybe we should link to the last Imba Changelog episode and mark it as experimental.

Since there might be more changes related to this in Imba 2, it's good to wait before adding it.


add more optional info when register new interact-url

Would be great if in interact we could add more info when we ask for a url.
"randomtoken.interact.sh":{
  "additional_info":{
          "registered_url":"https://vulnerabletoxss.com/form",
          "input_name": "vulnerable-form",
          "request_used": "POST /form HTTP/1.1\r\nHOST: vulnerabletoxss.com\r\n\r\nvulerable-form=<script src=//randomtoken.interact.sh>"
          }

Originally posted by @brenocss in https://github.com/projectdiscovery/interactsh/issues/442#issuecomment-1401455627

Would be great to detect which form is vulnerable, which host, etc


Drop some old functions in the update script of the extension

The following functions should be deleted in the extension upgrade script:
DROP FUNCTION cdb_disable_ddl_hooks;
DROP FUNCTION cdb_enable_ddl_hooks;
DROP FUNCTION cdb_handle_add_column;
DROP FUNCTION cdb_handle_alter_column;
DROP FUNCTION cdb_handle_create_table;
DROP FUNCTION cdb_handle_drop_column;
DROP FUNCTION cdb_handle_drop_table;

Duplicates https://github.com/CartoDB/cartodb-postgresql/issues/341


reindent and add e-diaeresis to a word

Hey @KostaTB, PR that contains some minor changes to format the file and correct a word. Ready to merge?

Fair point, I'd say let's drop it altogether given that it is part of the dictionary. I'll create another PR just for that since it is beyond the scope of this one.


Issue for uploading images for wiki

There is no way to upload images directly into wiki content.
I can still make images part of git report, but that mean unnecessary repo size growing.
There is a popular way of using GitHub Issues/discussions image upload for images in Wiki or Readme.
This issue is for those images.

Issue can be closed and still works.

Issue can be closed and still works.


Can't load new pre-populated database after app update

I'm able to open a database correctly using:
let db = SQLite.openDatabase({ name: database_name, createFromLocation: 1 });
The file is located in android>app>src>main>assets>www folder.
However when I modify the prepopulated database, and replace the database in the same folder with the same filename, it will not use the updated database after an app update. This is expected behaviour since files in the assets folder are cached and don't refresh without a reinstall or manually clearing data: https://stackoverflow.com/a/33469764/1575151
Additionally, it isn't possible to clear assets folder at runtime: https://stackoverflow.com/a/14000813/1575151
My solution thus far has been to update the filename each time there is an app update, however this is not ideal because it is causing the app file size to continually expand since each older version of the DB is still cached on the user device. They would have to manually clear data or reinstall to fix the file size issue.
Are there any solutions for this? If it were possible to read a pre-populated db from the res>raw folder instead, that might fix the issue.
Thanks

You should delete the previous database..
SQLite.deleteDatabase({
  name: "db",
  location: "default"
}, function (res) {
  if(res == "database removed") {
    //openDatabase call here
  }
}

Yes, @brystfire08 is correct. Delete the existing database in order to have it re-created with the pre-populated db.

How is it possible to change a pre-populated database without new installation of the App? I tried with the following code:
SQLite.deleteDatabase({   name: "db",           location: "default"   }, function (res) {   if(res == "DB deleted") {     openDatabase({ name:db, createFromLocation : 1, location: 'Library'})   } }
deleteDatabase correctly deletes the database. But openDatabase does reopen the old database again and not the new one in the iOS/AppName/www folder on iOS or the android/app/src/main/assets/www folder.
How can I achieve that the new database is opened from the www folder?


Disable overflow scrolling on Mac OS X

Hello @mikaa123,
it would be nice if the overflow scrolling of a ruler is disabled.
(Pull request incoming ;))
Best regards,
Steffen

Yes, you're completely right. Looking forward to the PR :)

Apparently, npm's network is a bit flaky. It hangs while running npm install.

Yeah that tends to happen sometimes. Have you tried npm cache clean?

Have you tried npm cache clean?

That didn’t do the trick.
Mind if I send a pull request via GitHub’s UI?

Go ahead, no problem :)

@mikaa123 Done. ;) (Untested, though.)


Add @InjectedObject property for ObservableObject compatibility

Hello, I've used Resolver for a while and just switched over to Factory. Both libraries help to circumvent common issues with EnvironmentObject, but both libraries also run into the same issue wherein you lose the ability to trigger SwiftUI views to update when a property on the object changes by default. This usually requires a bunch of boilerplate code and a required ViewModel, so there's easy way to make lightweight views the way one might with EnvironmentObject, which creates an inverse problem to the one presented by SwiftUI's EnvironmentObject.
Curious what the expectation is for working around this, or if there are plans for something like an @InjectedObject (as in Resolver) property to help account for this? I saw issue #1, but it doesn't really go into detail, and the provided snippet doesn't actually support re-rendering views (see example below), so it's not really a valid workaround for achieving the same behavior by itself.
This library seems like an extremely easy-to-use and easy-to-onboard dependency injection tool, but I would argue it's not really SwiftUI-compatible (or at least not SwiftUI-optimized) without functionality like this in place, and/or some comprehensive documentation in the README for what the expectations/workarounds are for dealing with this.
Problem
Say we want to pull AppSettings into a view, which is just a struct that has an environment property. Ideally, this would be all it takes to achieve that:
struct MyView: View {
    @Injected(Container.appSettings) var appSettings

    var body: some View {
        Text(self.appSettings.environment)
    }
}

However, when appSettings.environment is updated, the view does not reload -- which is a known limitation of this library.
If I take the snippet provided in issue #1, it doesn't resolve the problem -- the view still doesn't reload.
struct MyView: View {
    @StateObject var viewModel = MyViewModel()

    var body: some View {
        Text(self.viewModel.appSettings.environment)
    }
}

final class MyViewModel: ObservableObject {
    @Injected(Container.appSettings) var appSettings
}

Instead, I have to do something like this:
final class MyViewModel: ObservableObject {
    @Injected(Container.appSettings) var appSettings

    private var subscriptions = Set<AnyCancellable>()

    init() {
        // without this block, MyView will not trigger a state refresh.
        self.appSettings.objectWillChange.sink { _ in
            self.objectWillChange.send()
        }
        .store(in: &self.subscriptions)
    }
}

or this
final class MyViewModel: ObservableObject {
    @Injected(Container.appSettings) var appSettings
    @Published var environment: Environment

    private var subscriptions = Set<AnyCancellable>()

    init() {
        // no different than the example above unless I optimize further
        self.appSettings.$environment
            .assign(to: &self.$environment)
    }
}

Proposed Solution
A reimplementation of Resolver.InjectedObject to be included in this library:
@propertyWrapper public struct InjectedObject<T>: DynamicProperty where T: ObservableObject {
    @ObservedObject private var dependency: T

    public init(_ factory: Factory<T>) {
        self.dependency = factory()
    }

    public var wrappedValue: T {
        get { return dependency }
        mutating set { dependency = newValue }
    }

    public var projectedValue: ObservedObject<T>.Wrapper {
        return self.$dependency
    }
}

I don't think we need to provide an implementation for StateObject explicitly since we're not worried about a view being the owner of the StateObject -- similar to how there is no EnvironmentStateObject and EnvironmentObservedObject, for example.
That said, if we needed to (or it would help optimize or anything), we could also add InjectedStateObject for explicit conformance.

You don't show AppSettings but indicated right off the bat that it's a struct, so I'm not clear on how it's supposed to trigger an update since a struct can't be an observable object.
At any rate, and regarding InjectedObject, I've resisted doing this, simply because all we're doing is wrapping Apple's implementation in yet another level of indirection, and because it's simply too easy to do something like example 1 or 2...
struct StateObjectView1: View {
    @StateObject var vm = Container.vm()
    var body: some View {
        Text(vm.text)
    }
}

struct StateObjectView2: View {
    @ObservedObject var vm = Container.scopedVM()
    var body: some View {
        Text(vm.text)
    }
}

Which uses the standard Apple property wrappers and as such makes it clear exactly what behavior is occurring.
Resolver's InjectedObject also hid the fact that we were using ObservedObject and as such from my perspective it didn't make it clear that your registration was required to be scoped. Failure to do so would cause a new instance to be created each and every time the view tree was evaluated prior to a redraw or screen refresh.
Happy to continue the discussion.

I kind of mentioned this, but the InjectedObject wrapper as implemented above will call the Factory to get an instance of the object every single time the view is evaluated. This obviously means the factory must be scoped but StateObject will do a much better job of determining when one instance should go out of scope and a new one created.
Bottom line is that using the injection system for ViewModels in SwiftUI is a bad idea. It's really designed to provide the VM and other services with the dependencies that they need. Especially since those services have no access to the environment.

@hmlongco thank you for the explanation -- and I misspoke when talking about AppSettings being a struct, I meant to imply it was an ObservableObject but was rewriting the snippet a few times so must have missed this.
I think I understand everything you're saying, but can you elaborate on this part?

as implemented above will call the Factory to get an instance of the object every single time the view is evaluated.

I've barely messed with PropertyWrappers still, but my understanding is that it resolves the dependency immediately upon initialization. Is it that it gets re-initialized multiple times? If so, can you elaborate on why this is occurring?
Your examples here are exactly what I'm referring to as missing from your documentation:
@StateObject var vm = Container.scopedVM()
@ObservedObject var vm = Container.scopedVM()

This seems simple, but honestly it never occurred to me to use Factory in this way. I definitely, definitely think you should put this example along with a brief explanation in your README.

I had a version of it. See the example just before Mocking and Testing.
"Finally, note that it's possible to bypass the property wrapper and talk to the factory yourself in a Service Locator pattern."
class ContentViewModel: ObservableObject {
    // dependencies
    private let myService = Container.myService()
    private let eventLogger = Container.eventLogger()
    ...
}

I'll look at adding that it's needed if you want to use Factory with SwiftUI's wrappers.
As to elaborating, please take a look at my article, Deep Inside Views, State and Performance in SwiftUI.
The 60 second version goes like this: In SwiftUI Views are not views, they're lightweight definitions of views. When state changes the view structs are constantly being recreated, evaluated, and compared in order to determine if the portion of the UI they define needs to be updated and redrawn.
Each time a view is recreated its property wrappers are recreated, and then in our case the initialization function would call the factory to get the required dependency.
Again, I highly suggest you read the above article if you're working with SwiftUI, as understanding the mechanisms involved are critical to using it successfully.

Right, I saw that example, I'm talking about examples for use with SwiftUI explicitly. I understand it's the same functionality, but that's why I think there's a benefit to explicitly discussing it within your README.
Will check out the article, thanks for sharing! Keep up the good work. 👍🏻

Updated README with section on SwiftUI.


Import study from disk via DIMSE to Orthanc server fails for certain DICOM files

The two attached DICOM files are part of the same study and series. When uploading them separately using the "Import study" button at the top right corner of the study list, they are uploaded properly via DIMSE to my Orthanc server (running in docker). However, when uploading together it seems that the files are corrupted somehow. The same error occurs when trying to upload the entire study (around 500 files). It successfully uploads 249 files but then fails. I narrowed it down to failling with just these two files (failling together, but working separately).
See the following output when uploading the two files together (placing them in the same directory).
Meteor server log:
I20171205-14:21:31.780(1)? Services.DIMSE.Studies
I20171205-14:21:31.781(1)? Associating...
I20171205-14:21:31.783(1)? Starting Connection...
I20171205-14:21:31.791(1)? Connect
I20171205-14:21:31.791(1)? Connection established
I20171205-14:21:31.803(1)? ==Associated
I20171205-14:21:31.805(1)? Sending command C-FIND-RQ
I20171205-14:21:31.858(1)? Data received
I20171205-14:21:31.864(1)? Connection closed
I20171205-14:21:35.294(1)? Dicom file /tmp/dicomDir/11706606 found
I20171205-14:21:35.294(1)? Dicom file /tmp/dicomDir/11707156 found
I20171205-14:21:35.295(1)? Starting Connection...
I20171205-14:21:35.296(1)? Connect
I20171205-14:21:35.297(1)? Connection established
I20171205-14:21:35.300(1)? Sending file /tmp/dicomDir/11706606
I20171205-14:21:35.301(1)? Sending command C-STORE-RQ
W20171205-14:22:05.407(1)? (STDERR) Association aborted with reason 0
I20171205-14:22:05.408(1)? Connection closed

Orthanc log (shortened - full output is attached):
W: DcmItem: Length of element (5343,000a) is odd
W: DcmItem: Non-standard VR ' �' (03\a3) encountered while parsing element (a303,a503), assuming 2 byte length field
W: DcmItem: Length of element (a303,a503) is odd
W: DcmItem: Non-standard VR '� ' (ea\03) encountered while parsing element (03e5,03e6), assuming 2 byte length field
W: DcmItem: Length of element (03e5,03e6) is odd
W: DcmItem: Dataset not in ascending tag order, at element (03e5,03e6)
W: DcmItem: Non-standard VR ' �' (03\d5) encountered while parsing element (d203,d603), assuming 2 byte length field
W: DcmItem: Length of element (d203,d603) is odd
W: DcmItem: Non-standard VR '� ' (f5\03) encountered while parsing element (0401,03fb), assuming 2 byte length field
W: DcmItem: Length of element (0401,03fb) is odd
W: DcmItem: Dataset not in ascending tag order, at element (0401,03fb)
W: DcmItem: Non-standard VR '  ' (04\13) encountered while parsing element (0303,0e04), assuming 2 byte length field
W: DcmItem: Dataset not in ascending tag order, at element (0303,0e04)
W: DcmItem: Non-standard VR ' �' (01\bc) encountered while parsing element (ab01,bb01), assuming 2 byte length field
W: DcmItem: Length of element (ab01,bb01) is odd
W: DcmItem: Dataset not in ascending tag order, at element (ab01,bb01)
W: DcmItem: Non-standard VR ' �' (00\a9) encountered while parsing element (9500,a100), assuming 2 byte length field
...
W: DIMSE Warning: (OHIFDCM,ORTHANC): DIMSE receiveDataSetInMemory: dset->read() Failed (Invalid stream)
E1205 13:21:49.984349 StoreScp.cpp:294] Store SCP Failed: DIMSE Failed to receive message

full output: orthanc output.txt
To check if they are corrupt, I tried to run DCMTK's dcmdump on all files (both originals and the ones copies to /tmp/dicomDir by meteor):
mydicom/11706606
mydicom/11707156
tmp/dicomDir/11706606
tmp/dicomDir/11707156

but all of them look fine (dcmdump just outputs all the tags and their values as it is supposed to).
I tried uploading the files together directly to Orthanc using the admin interface at localhost:8042, and it works fine. Thus I suspect this to be an OHIF Viewer bug rather than Orthanc.
I tried opening the files in ITK-SNAP, and they load and display just fine.
The two dicom files can be found here: dicom.zip
Extra info: I'm running the OHIFViewer using:
cross-env METEOR_PACKAGE_DIRS=\"../Packages\" meteor --settings ../config/orthancDIMSE.json

I'm running orthanc using a custom configuration file, where the only thing I changed was adding
"ohif": [ "OHIFDCM", "localhost", 3000 ]

to DicomModalities.

We are moving away from support for version 1 of the Viewer. The goal is to make Version 2 extensible and configurable enough that you can customize it for your workflow and purposes; however, DIMSE support is not yet planned.
This issue hasn't seen activity in some time, so I'm going to close it. If you, or anyone else, has new info to add on this issue, please don't hesitate to comment and continue the discussion.


Custom locale configuration in start process and runtime

Proposed Changes

added new builder to simplify start parameters (including new locale configuration)
added new custom locale parameter for the start process
added new start method to support the new parameter builder
generic start supports custom locale (null value for locale has the previous behaviour)
add setter for the locale and propagate the changes to the server (via START call)
added unit test for the builder and the new locale parameter
added unit test for the locale setter

Can one of the admins verify this patch?

why closing the pull request? Do you have a reason to don't support custom locales?


Upgrade tasks to use Powershell.

The current command line / batch task runner is obsolete and should be replaced with the Powershell runner. This is much more powerful and flexible.

I think this can be closed or modified. Using PowerShell doesn't make any sense. I think we either use typescript or fable to make them work cross-platform


Error detected while processing C:\Users\Hp\AppData\Local\nvim\init.lua

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Not a user config issue

[X] Confirm

Neovim version
NVIM v0.9.4 Build type: Release LuaJIT 2.1.1700008891     system vimrc file: "$VIM\sysinit.vim"   fall-back for $VIM: "/ucrt64/share/nvim"  Run :checkhealth for more info
Operating system/version
Windows 11 23H2
Terminal name/version
Windows Terminal 1.17.11461.0
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
在执行一键安装脚本后，用nvim命令启动nvim
Expected behavior
正常工作
Actual behavior

Additional information
安装时遇到错误

Type in the following when u're in cmd mode:
:=string.format("DOS: %s\n$HOME: %s", vim.loop.os_uname().sysname == "Windows_NT", os.getenv("USERPROFILE"))

What do u see?

close due to inactivity.


onMonthSelect and onYearSelect doesn't work if custom renderMonthElementuses React portal

react-dates version
<EMAIL_ADDRESS>Source code (including props configuration)
Codesandbox
Steps to reproduce the behavior:

Open calendar by focusing the input
Try to pick month/year from selects at the top of the calendar

Actual result:
Calendar closes without calling onMonthSelect/onYearSelect
Expected result:
Should pick month/year without closing the calendar
(Works as expected if comment line 18 in month-year-picker.js which is responsible for using portal)
Desktop (please complete the following information):

OS: macOS Mojave (10.14.6)
Browser: Google Chrome 78.0.3904.97 (Official Build) (64-bit)
Version: 21.3.2

@Kenjirou couldn't open provided link

@mmarkelov sorry. I've updated the link.

@Kenjirou I'm not sure that it is react-dates issue, cause it seems like react-select does not fire onChange event:
        onChange={({ value }) => {
          console.log('!!!!') // Does not work
          onMonthSelect(date, value);
        }}


Error in POST calls with 2 or more parameters

Hi there, I can't find a solution for this issue.
All my POSTS works fine, but when I use 2 or more parameters in $postfields, like this:
$postfields = array( 	'status' => $statustext, 	'in_reply_to_status_id' => $TweetID );
and then perform a POST:
$twitter = new TwitterAPIExchange($settings); $twitter = $twitter->buildOauth($url, $requestMethod) ->setPostfields($postfields) ->performRequest(); $return_json= json_decode($twitter, true);
I always get {"errors":[{"message":"Could not authenticate you","code":32}]}
Is there a fix for this?
Thanks!

Could you post the exact code you're using, including the url and tweet ID basically. Thanks.

Ok, thank you for your response :)
You can test it with any tweet ID where you are mentioned. I put the user's screen_name in reply of course.
$TweetID = "715854796750721024";
$statustext= "@R5Belleza Blablabla...";
$url = "https://api.twitter.com/1.1/statuses/update.json";
$requestMethod = 'POST';
$postfields = array(
'status' => $statustext,
'in_reply_to_status_id' => $TweetID
);
$twitter = new TwitterAPIExchange($settings);
$twitter = $twitter->buildOauth($url, $requestMethod)
->setPostfields($postfields)
->performRequest();
$return_json= json_decode($twitter, true);

Hi,
Just ran into this same issue, managed to get around it by reducing the post parameters to 2. Is there anything I can supply you with to help fix this issue in a more structural way?
@TomCrush thanks for posting it with the title the way you did, this pointed me to the workaround I currently have.

Hmm, could you by any chance write a failing test for this? If you can maybe we can work together to figure out what the issue is?

I could give part of my code, but it looks very similar to TomCrush's. Is that not giving the error described when you test is? If not, I'll have a look and anonymize my code.

Well we know from this passing test that it isn't two or more parameters that's causing the issue. The only thing the test doesn't use is in_reply_to_status_id.
Can you try removing that and putting some other parameters so we can narrow it down to the exact issue? Also, I haven't checked but make sure the person's account you're replying to isn't protected perhaps.

Hi guys, I've updated the test to prove in_reply_to_status_id definitely works here.
The code seems to be passing, so it must be something else.
@TomCrush Can you please prove that the library works for you just for posting a status, not with replying to one?

The issue with me what with sending a DM, I'll look up the use case.

Hmm, just tried recreating the issue sending a DM with the screen_name, user_id and text, but it works. Perhaps during my earlier use there was something violating in the text. If I manage to reproduce it in the future I will let you know, sorry. Thanks for providing this code btw, saved me a lot of work :)

No problem, glad we managed to figure it out. I'll close for now then, @TomCrush if you still have a problem please feel free to re-open.


add support for Clear Linux OS

This PR adds basic support for Intel's Clear Linux OS.  This includes:

service management via systemd
managing packages (bundles, in Clear Linux terminology)

The bundles/packages are not versioned, but potentially updated as the entire OS is updated to a new release.
I tested this by building mitamae against my branch:
conf.gem github: 'grothja/mruby-specinfra',  checksum_hash: '4ea379075dcfffe893832af5e1c3361e4e37cb5a'

Going to the directory where mitamae was built and starting up a full privs test container to verify service management:
docker run --rm -d \
  --name clearlinuxos \
  -e container=docker \
  --tmpfs /run \
  --tmpfs /tmp \
  -v /sys/fs/cgroup:/sys/fs/cgroup:ro \
  -v $PWD:/root \
  --cap-add SYS_ADMIN \
  -p 8080:80 \
  clearlinux /sbin/init

and then running:
docker exec -it clearlinuxos /root/mitamae local /root/test.rb

Test I used was this (with various removed, stop, etc. combinations):
package "httpd" do
  action :install
end

service "httpd" do
  action :start
end

And curl http://localhost:8080 should return results.

This looks good, but this repository is meant to be a port of https://github.com/mizzy/specinfra. Could you file the same PR to the repository first? Once it's merged, I'll merge this PR too.

Thanks @k0kubun; per your suggestion, this has made its way into mizzy/specinfra.  A couple modifications were made to the name and how OS detection works, but those changes are reflected here now.

Thank you!
I'll include this in a new mitamae release a few days later because I'm going to a vacation and not carrying my laptop.

This will be released as mitamae v1.7.8 in this Travis build.


make number of Pools configurable

currently hardcoded at 4. https://github.com/steveklabnik/simple-server/blob/4f6050ad6b45caea93ff3caf31eaabfcd75174c3/src/lib.rs#L84
not urgent but nice to have

Might it be beneficial to use the num_cpus crate? If so I can send a PR.


Add nodeSelector to helm charts

Is your feature request related to a problem? Please describe.
We need to provide user a way to control deployment of feast to a specific node pool using node selector
Describe the solution you'd like
Add nodeSelector option in helm charts

@pradithya is this still relevant? Can you please reopen this if necessary.


retinitis pigmentosa

Mondo term (ID Label)
retinitis pigmentosa
Bug/Typo/Error description
retinitis pigmentosa is not a subclass of glycolysis disorder
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

@pnrobinson this was added based on the classification in the recent paper you shared (see https://github.com/monarch-initiative/mondo/issues/4985)
I can exclude this superclass. Please reopen this ticket if you would like to add it back.


[HW]  Pavan Kumar

Name: PAVAN KUMAR
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/pavan-kumar-pulamathi-5bab3429/
CRUD Screenshots.docx
CRUD Screenshots.docx

Attach the homework screenshots below for both step II and step III:

Hey PAVAN KUMAR, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/jZezRVdBT16ubwKMGrjF4Q

Thanks Aleks.


enhancement: move/up down history if on first/last line of prompt

Just a little enhancement:
I replaced zle down-line/up-line with the more elborated functions down-line-or-history/up-line-or-history. Thus, you move the zsh history automatically up/down in hnor mode if you're on the first/last line of the current prompt and press k/j.
That behavior is copied from vicmd mode and I think its very intuitive. Merge it if you like :wink:

The behavior of that key is unfinished at the moment. I think using only bare zle widget most likely wouldn't work since there are other custom variables ZHM defined and used to do helix-styled selection which can break further key presses.


Tag Analyzer - Change formatting for  value

Summary
The <tag not defined> row of values for a selected tag should be changed to remove the background color and make coloring conditional on the selected tag's Enforcement level.
Desired Behaviour
When a tag is selected in Tag Analyzer, the always-present top row of <tag not defined> should not have any background color.
The <tag not defined> text should be bold, and change font color depending on the Enforcement level of the selected tag:

Required = Red
Optional = Dark Yellow
Non-policy = Black

Released in v1.1.2


misplaced user quota position

I'm not sure where is the exact place, but it seems to be odd at this position

should we maybe add the quota in the user menu? or maybe to be sticky top right?


Menu item to show the entire dataset

We need some way to zoom in / out the current view so that all of the data is visible.
This is useful when the current view gets positioned somewhere outside of the range of the current data and it's hard to find our way back.

Added zoom_to_data


Expand env-vars in background image path

Summary of the Pull Request
Expands any environment variables in background image file paths so that users can specify location of background images relative to, for example, %USERPROFILE%
References

Terminal takes an unexpected dirt nap when backgroundImage contains environment variables #2922

PR Checklist

[X] Closes #2922
[X] CLA signed. If not, go over here and sign the CLA
[X] Tests added/passed
[ ] Requires documentation to be updated
[X] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Validation Steps Performed
Thoroughly debugged with previous failing case, now working case, empty strings, etc. Also added basic regression tests that will fail if icon or backgroundImage paths contain env-vars and the path expansion mechanisms are ever removed/subverted.

You should force push this PR instead of deleting it and making a new one! You can even just revert your readme changes in this branch.


Revert "add a new user | ci-with-toad-edge-plugin"

Reverts jenkins-infra/repository-permissions-updater#3687

Hey @mkumar-quest,
due to https://status.jenkins.io/issues/2023-12-18-jira-outage/ I'm going ahead and reverting your PR to restore a functional RPU again. The user specified must log in to https://issues.jenkins.io/ again. Once you have done that, you are welcome to revert my reversion PR adding the user back to the release permission list.
Thanks for understanding.

Superseded by https://github.com/jenkins-infra/repository-permissions-updater/pull/3704


Added background commands

The syntax looks ugly, but this is the optimal way to do it. It doesn't have any extraneous output due to the parenthesis surrounding the command, and also sends error output to /dev/null.

Make the change in the README instead of the manuscript. The manuscript is generated by a script from the README file.
Thanks for your contribution. 👍

O

Cheers!


Modularized three main charts

This isn't to pull quite yet, but spent some time splitting the charts up into different files. I've used events to communicate changes in view and when to render. Thoughts?

#70

Resolves #115 #116 and #117 plus sets up an events system that should make some other modularizations easier

Would appreciate another look-over of this :)

'viewChange' -> 'viewChanged'? Other than that, looks good.

Definitely moving in the right direction. Seeing it split up certainly makes things easier to follow! I'm wondering if rather than each chart creating a new 'viewScale' and using events to communicate the domain change, they could instead share one instance of an xScale? It's also surprising to see 3 d3.behavior.zoom components, but we can work on that later in #121.

viewScale is only created in navChart, and I think that requires the range being navChart's width, as opposed to the width of any other chart. The rest just use the domain passed to them. It's possible for all of them to share the same scale, but I thought we agreed to do it this way?

@CBarrett2 Ok we can think about if shared scales make any sense later.


Update index.html

Hello All, Signing the Manifest as well.

Hello All, Signing the Manifest as well.

Added in https://github.com/opentofu/manifesto/pull/1137