Jaeger task not having expected results

Related: #32132 and #32026 and https://github.com/istio/istio.io/issues/9668
I attempted to follow the instructions to Install Jaeger.  I modified them slightly.  I did
istioctl-1.10.0-rc.0 install \
  --set meshConfig.accessLogFile="/dev/stdout" \
  --set meshConfig.defaultConfig.tracing.zipkin.address=jaeger-collector.istio-system.svc.cluster.local:9411 \
  --set meshConfig.defaultConfig.tracing.sampling=100 \
   && say installed || say failed

I followed all of the other steps including for i in $(seq 1 100); do curl -s -o /dev/null "http://$GATEWAY_URL/productpage"; done
In the Jaeger dashboard, I see two services: istio-ingressgateway and jaeger-query.  I do not see the productpage.default promised by https://preliminary.istio.io/latest/docs/tasks/observability/distributed-tracing/jaeger/
Traffic is working ... bookinfo works for me.  I see traces from istio-ingressgateway to productpage.default.svc.cluster.local:9080/productpage.  I don't see calls to details, ratings, or reviews.
I see only 3 traces.  I expected because I set sampling=100 that I would see the query default, 20, traces.

Was bookinfo injected with sidecar?

Especially see https://github.com/istio/istio.io/issues/9669
Yes, pods have sidecars.
$ kubectl get pods
NAME                              READY   STATUS    RESTARTS   AGE
details-v1-79f774bdb9-mv8x9       2/2     Running   0          50m
httpbin-95db65f-6rtx5             2/2     Running   0          48m
productpage-v1-6b746f74dc-xckdk   2/2     Running   0          50m
ratings-v1-b6994bb9-bw8hm         2/2     Running   0          50m
reviews-v1-545db77b95-58mmd       2/2     Running   0          50m
reviews-v2-7bf8c9648f-z9vlh       2/2     Running   0          50m
reviews-v3-84779c7bbc-jhd74       2/2     Running   0          50m

And bookinfo proxy is using 1.10.0-rc.0?

$ kubectl get pod productpage-v1-6b746f74dc-xckdk -o jsonpath="{.spec.containers[1].image}"

docker.io/istio/proxyv2:1.10.0-rc.0

Works fine for me with 1.10.0-rc.0.
Looks like you configured jaeger collector service with wrong port. Should be 14268.

Still not working for me.
I had no problem redeploying with 14268 and restarting pods.
There are two things I am confused about.  The first thing I want to point out is that the ingressgateway has no trouble sending traces.  It's configuration Envoy bootstrap has nothing about tracing.  It's <IP_ADDRESS>_8080 listener, filter chain has
            "tracing": {
             "client_sampling": {
              "value": 100
             },
             "random_sampling": {
              "value": 1
             },
             "overall_sampling": {
              "value": 100
             },

The productpage-v1 sidecar, whose distributed traces are not being caught by Jaeger, has an Envoy bootstrap with
       "tracing": {
        "zipkin": {
         "address": "jaeger-collector.istio-system.svc.cluster.local:14268"
        },
        "sampling": 100
       },

It also has a listener with
            "tracing": {
             "client_sampling": {
              "value": 100
             },
             "random_sampling": {
              "value": 100
             },
             "overall_sampling": {
              "value": 100
             },

... but this listener moves around.  I have seen it on listener "<IP_ADDRESS>_15014" and on listener "<IP_ADDRESS>_8081".  I would expect it to be listening on all of the HTTP ports, or all of the bookinfo ports, not on just a single port...

Then what is the tracer address used by ingress? Have you restarte istio-ingressgateway pod to get new trace sampling rate?
I don't quite understand what do you mean by listener moves around.

Adding networking because Istio 1.9.2 generates Envoy config with
            "tracing": {
             "client_sampling": {
              "value": 100
             },
             "random_sampling": {
              "value": 100
             },
             "overall_sampling": {
              "value": 100
             },

on every HTTP port, but 1.10.0-rc.0 only generates that for a single random port.

@bianpengyuan I had thought istioctl install was restarting the ingressgateways when the meshconfig changes, but it isn't.  So I can't trust my experiments.
I am not restarting both Istiod and the ingressgateway after each change of meshconfig.  Now I am not getting any traces to appear in Jaeger, from the ingressgateway or my working pods.
What do you see when you do kubectl -n istio-system exec deployment/istio-ingressgateway -- curl localhost:15000/config_dump | jq ".configs[0].bootstrap.node.metadata.PROXY_CONFIG.tracing"?
What other settings should I look at?
I am installing Jaeger 1.20 using kubectl apply -f https://raw.githubusercontent.com/istio/istio/release-1.10/samples/addons/jaeger.yaml as recommended by the Istio docs.

Could you remove the trace address, just use the default zipkin tracing service address and try again?
I did not use jaeger collector directly and I am not familiar with it at all. I am not sure if trace span emitted by Envoy is compatible with the format expected by the jaeger collector)

I will close this because the Jaeger task does work if Jaeger is installed correctly.
For an explanation as to why my config was wrong, see https://github.com/istio/istio.io/issues/9668#issuecomment-831613196
I am leaving open the doc issues to improve the doc and the automated test.

Still not working in #32044


Method not allowed / Payload too large errors

Reverb Version
1.0.0-beta14
Laravel Version
11.13.0
PHP Version
8.3.3
Description
Deployed Reverb via Forge to production, and I get the following error:
[2024-06-29 21:25:36] production.ERROR: Pusher error: Method not allowed.. {"userId":1,"exception":"[object] (Illuminate\\Broadcasting\\BroadcastException(code: 0): Pusher error: Method not allowed.. at /home/forge/.../vendor/laravel/framework/src/Illuminate/Broadcasting/Broadcasters/PusherBroadcaster.php:164)
[stacktrace]
#0 /home/forge/.../vendor/laravel/framework/src/Illuminate/Broadcasting/BroadcastEvent.php(92): Illuminate\\Broadcasting\\Broadcasters\\PusherBroadcaster->broadcast()
#1 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(36): Illuminate\\Broadcasting\\BroadcastEvent->handle()
#2 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/Util.php(41): Illuminate\\Container\\BoundMethod::Illuminate\\Container\\{closure}()
#3 /home/forge/.../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(93): Illuminate\\Container\\Util::unwrapIfClosure()

Locally this error doesn't happen.
Another error I do get locally:
[2024-06-29 21:49:41] local.ERROR: Pusher error: Payload too large.. {"userId":11,"exception":"[object] (Illuminate\\Broadcasting\\BroadcastException(code: 0): Pusher error: Payload too large.. at .../vendor/laravel/framework/src/Illuminate/Broadcasting/Broadcasters/PusherBroadcaster.php:164)
[stacktrace]
#0 .../vendor/laravel/framework/src/Illuminate/Broadcasting/BroadcastEvent.php(92): Illuminate\\Broadcasting\\Broadcasters\\PusherBroadcaster->broadcast(Object(Illuminate\\Support\\Collection), 'App\\\\Events\\\\Host...', Array)
#1 .../vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php(36): Illuminate\\Broadcasting\\BroadcastEvent->handle(Object(Illuminate\\Broadcasting\\BroadcastManager))

Steps To Reproduce
My Event class:
namespace App\Events;

use App\Models\Event;
use App\Data\EventStatData;
use Illuminate\Broadcasting\Channel;
use Illuminate\Queue\SerializesModels;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Broadcasting\InteractsWithSockets;
use Illuminate\Contracts\Broadcasting\ShouldBroadcastNow;

class EventStatsUpdated implements ShouldBroadcastNow
{
    use Dispatchable, InteractsWithSockets, SerializesModels;

    public function __construct(
        public Event $event
        ) {
    }

    public function broadcastWith(): array
    {
        return EventStatData::from($this->event->stats)->toArray();
    }

    public function broadcastOn(): array
    {
        return [
            new Channel("Events.{$this->event->id}.stats"),
        ];
    }
}

Usage:
broadcast(new EventStatsUpdated($this));

local .env:
REVERB_APP_ID=812456
REVERB_APP_KEY=1cd7uv8gp2fv6y1tlxyi
REVERB_APP_SECRET=pacnwjlkxjgfkeums21o
REVERB_HOST="localhost"
REVERB_PORT=8080
REVERB_SCHEME=http

VITE_REVERB_APP_KEY="${REVERB_APP_KEY}"
VITE_REVERB_HOST="${REVERB_HOST}"
VITE_REVERB_PORT="${REVERB_PORT}"
VITE_REVERB_SCHEME="${REVERB_SCHEME}"

production .env
REVERB_APP_ID=396379
REVERB_APP_KEY=h95uycobejaarqfzftpq
REVERB_APP_SECRET=3lpzwncd1yvflc5pnubm
REVERB_HOST=ws.my.app # my domain
REVERB_PORT=443
REVERB_SCHEME="https"

VITE_REVERB_APP_KEY="${REVERB_APP_KEY}"
VITE_REVERB_HOST="${REVERB_HOST}"
VITE_REVERB_PORT="${REVERB_PORT}"
VITE_REVERB_SCHEME="${REVERB_SCHEME}"

Have you tried increasing the size of 'max_message_size' in config/reverb.php ?

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord or InertiaJS Discord for Inertia questions
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

@Yiddishe-Kop hello! I had the same error Pusher error: Method not allowed... In my situation, the reason was a discrepancy in settings:
I had
REVERB_PORT=80
REVERB_SCHEME="http"

But it should be:
REVERB_PORT=443
REVERB_SCHEME="https"

The Forge re-recorded my initial settings.


Higher-order unit conversion with GWP context

Describe the bug
Unit conversion fails when using a GWP context for units that have higher-order dimensionality. For example, one cannot convert kg CO2/EJ to kg CH4/EJ, but kg CO2/yr => kg CH4/yr works.
The above example fails with the following message:
Cannot convert from 'CO2 * metric_ton / exajoule' ([carbon] * [time] ** 2 / [length] ** 2) to 'CH4 * metric_ton / exajoule' ([methane] * [time] ** 2 / [length] ** 2)

Maybe I'm doing something wrong, but the dimensions match (except for carbon=>CH4 which should be handled by the GWP context).

        Parameters
        ----------
        value :
            value
        src : UnitsContainer
            source units.
        dst : UnitsContainer
            destination units.
        inplace :
             (Default value = False)
        check_dimensionality :
             (Default value = True)

        Returns
        -------
        type
            converted value

        """

        if check_dimensionality:

            src_dim = self._get_dimensionality(src)
            dst_dim = self._get_dimensionality(dst)

            # If the source and destination dimensionality are different,
            # then the conversion cannot be performed.
            if src_dim != dst_dim:
>               raise DimensionalityError(src, dst, src_dim, dst_dim)
E               pint.errors.DimensionalityError: Cannot convert from 'CO2 * metric_ton / exajoule' ([carbon] * [time] ** 2 / [length] ** 2) to 'CH4 * metric_ton / exajoule' ([methane] * [time] ** 2 / [length] ** 2)

../../venv/lib/python3.9/site-packages/pint/registry.py:1077: DimensionalityError

Expected behavior
Units are able to be converted
System (please complete the following information):
Ubuntu
Python 3.9
commit f97fac6c1ac00dd507e23d55949d8d45b07bf9a8
@znicholls @rgieseke

I think related:

https://github.com/hgrecco/pint/issues/854
https://github.com/hgrecco/pint/issues/855

Good detective work.
Option a would be the simplest given that there is already functionality to build the unit graph. The downside is that we can't capture all permutations that could be useful and the error message isn't exactly helpful. But adding a few extras shouldn't hurt?
Do you see an easy way to detect this situation and provide a better error message/raise a different exception?

But adding a few extras shouldn't hurt?

Ye definitely fine until we work out a longer term solution

Do you see an easy way to detect this situation and provide a better error message/raise a different exception?

It's probably possible to do something with base units (pint has some functionality to show the difference) and then if they're both units we've defined say something like 'check compound units' but it would take some fiddling I suspect


Vault kubernetes authentication doc update

Fixed a couple of typos, updated some commands and fixed some failing commands and missing steps.

Thank you!
On Sat, Mar 24, 2018 at 3:21 PM, Oleksandr Kushchenko <
<EMAIL_ADDRESS>wrote:

Awesome, thanks a lot!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kelseyhightower/confd/pull/690#issuecomment-375897476,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEdWnjOQj-6j3ZjCfJqP5-V9Zn9Vow_eks5thmSVgaJpZM4S5ktN
.

--
Abubakr-Sadik Nii Nai Davis
http://niinai.me
@ttousai


Checkpoint creation error

Hi,
I get the following error when trying to make a checkpoint for my algorithm:
AttributeError: Can't pickle local object 'ElementwiseDuplicateElimination.__init__.<locals>.<lambda>
My script contains the following statements:
res = minimize(problem=problems[2],
                   algorithm=algorithms[2],
                   termination=my_termination,
                   seed=1,
                   verbose=True,
                   copy_algorithm=True,
                   copy_termination=True,
                   save_history=False,
                   )
np.save('checkpoint', res.algorithm)

I quickly had a look and the ElementwiseDuplicateElimination uses a lambda function that can not be pickled.
What algorithm are you executing?

I am executing an instance of NSGA-III.

Okay. You are passing your own Duplicate Elimination which has a lambda function. Eliminate the lambda function and copy it out of the class and it shall work.

Okay. You are passing your own Duplicate Elimination which has a lambda function. Eliminate the lambda function and copy it out of the class and it shall work.

Ok! Thanks. I will put a feedback here.


Intermittent issue || Not able see Requester's request when logged as Approver.

Describe the bug
We have implemented this for one of the customer 1 year back. It was working fine till end Feb.
Since March users (who have access as Approvers) were reporting issue : they not able to see request when someone elevated access has been requested.
To Reproduce
Since this is intermittent issue, not able to replicate easily
Request for Elevated Access --> Check if Approver able to see request.
Expected behavior
Approver Should able see request.
Screenshots

Analysis.
I checked Cloudtrail logs and StepFunctions : I see that user request for Elevated access  written in DynamoDB, also I can see one of the steps in Stepfunction in pending/waiting state as Approver needs to approve it.
But since Approver not able to see request, he is not able to Approve it.
So need help in debugging this issue.

We are having the same issue started to happen since yesterday. requests dynamodb table has request items with empty email field which is probably the source of the problem about not seeing current requests.

@flexelem and @banagevikas  what version are you on ?

@tawoyinfa we are using 1.1 The issue went away when I removed all data from dynamo tables. But couldn't figure it out why emails were not getting persisted.

@tawoyinfa For us issue is still there. Regarding the version, any specific version you are looking for? If I check the version under path amplify > config > project-config.json, TEAMIDCAPP version is 3.1.

facing this issue too... approver cannot see the requests.... I installed TEAM 1.1
what is the workaround for this @tawoyinfa ?
thank you.

@flexelem is the issue resolved now ?

nope :(

For my case, this is not an issue anymore. I understand that a requestor cannot be the same person as the approver. Hence requestor cannot view his own requests for approval. This is by design.
I have done further testing including different users making requests and as the approver, I can see their requests for approval when I log in.

In my case, requestor and approver are different. Any workaround guys ?


Travis deploy

Super quick changes to deploy to S3 and update the version of node to LTS to avoid the binary-building issues we saw in CLUE.
https://travis-ci.com/concord-consortium/connected-bio-spaces
http://connected-bio-spaces.concord.org/branch/travis-deploy/

Sounds good to me, but I'm not a Travis expert.


add base templates

Fixes #87
Changes you made
-non

Please mention someone for code review
add base templates linked to the pages of the current homepage to users and managers

Is this the same as #90?

please add a meaningful description

@avraham1992 can we close this one? it looks like #99 contains this PR


Update packages

Description
Syncs the version at 0.0.1 and adds repos to each field
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.

Codecov Report

Merging #7 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #7   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          41     41
  Lines         407    407
  Branches       33     33
=====================================
  Hits          407    407

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 8092001...a87fb57. Read the comment docs.


Support for environments other than dev and prod

Is your feature request related to a problem? Please describe.
We have six environments. One is named dev and one is named prod but there are four others. It's not ideal to be limited to dev and prod only. Two of our environments are "production" and we'd like to have them distinguished within elementary.
Describe the solution you'd like
Ideally we would be able to provide the name of our environment as a string, so that we are not limited to the two provided environment names.
Describe alternatives you've considered
Additional context
Would you be willing to contribute this feature?
Yes, happy to make an attempt. Might need some direction.

We would also love this feature, as @menzenski organization, we also have a bunch of environments in different regions we would like to differentiate in the dashboard. We also would be willing to help make it happen.

Hey @menzenski !
Would love to help you get started on it
Our final goal is to get the env name from the env mentioned in the invocations table (We assume we have the env and project names already saved in our tables, so to make it more efficient we want to automatically receive it from there)
BUT as we understand this is a bit more complicated and this feature is required by a lot of users we can start from a very simple "step 1" solution - which would just be removing the Choice from the CLI (if you look at the file containing all the flags you'll see env is only getting a choice between dev and prod, we can turn it into a string for free text)
Is that clear? Do you need me to give you more details about which files we're talking about and what are the steps for this change?

For us the second option ("step 1"), would be more than enough, to just pass our desired string to the cli option.

I've opened a PR for this here: https://github.com/elementary-data/elementary/pull/1373


delete files of qemu

delete copied files after docker build

@ktong It's actually important that these stay there I think. Because this image can serve as the base image for other builds. For example in https://github.com/petrosagg/armv7hf-python-dockerhub there's a FROM resin/armv7hf-debian-qemu which allows that Dockerfile to use those files without adding them to the project's root directory.

@spmaniato is right, and since docker saves the layers anyway there is no actual disk space saved. closing


Add base color names option

Adds an option to specify the names of base color texture and factor uniforms when updating glTF with KHR_techniques_webgl extension to PBR.
By default, gltf-pipeline will try to convert materials that use the KHR_techniques_webgl extension into PBR materials. This is not possible generically, but it is possible to make some "educated guesses". Specifically, it is possible to guess which texture is supposed to be the PBR baseColorTexture by looking at the name of the uniform in the techniques definition: When the uniform is called u_diffuse (and refers to a texture), then it is very likely that this texture is supposed to become the baseColorTexture.
The set of names that suggest that a texture may become a base color texture is ... somewhat arbitrary. One can never know which names one might encounter in the wild.
This PR adds an option to specify the base color texture names at the command line and pass them to the update function in the options object. For example, when a GLB contains techniques where the uniform names are u_my_diffuse_tex and u_another_diffuse_tex, then the line
node ./bin/gltf-pipeline.js -i input.glb -o output.glb --baseColorTextureNames u_my_diffuse_tex u_another_diffuse_tex

will cause these names to be treated as indicators for the textures to become baseColorTexture.
(All this applies to the base color factors as well, with baseColorFactors as the command line option)

Things to consider:
The README marks these arguments as 'optional', but explicitly says that the set of "default names" for this is not specified. I'd hesitate to explicitly list things like "u_tex", "u_diffuse", "u_emission", "u_diffuse_tex" there...
The current approach is somewhat "fail silent" in nature. When someone has a GLB where such a name is u_diff_tex, then this texture will just be removed, and it may not be obvious for the user why this happened. Loading the model in a viewer might give the impression that the viewer can not display the texture for some reason (with that 'reason' being the too obvious one: It's not there...).
I considered to bring a few more high-level guesses into that conversion, and/or add some sort of warning or hint when something looks odd. For example:

If the technique values contains only a single texture reference
And if this texture reference does not have one of the known names
And (maybe:) If this texture becomes unused when the technique is omitted
Then print a warning:

Warning: Omitting texture that is used via uniform u_diff_t, because it is not one of the known diffuse texture names. See (link to readme) for details.

I think that could help to avoid some confusion for users, some support requests, and save some debugging time...

This looks good. If this keeps coming up it may be worth doing the logging you described.

If this keeps coming up it may be worth doing the logging

One could expect the set of predefined names to "converge" towards a set that covers "most" assets that are out there (for example, I added u_diffuse_tex and u_diffuse_mat to the defaults in this PR). But I expect some people to try this directly (or indirectly, via the upgrade in the 3d-tiles-tools) and be suprised by the missing textures.


Complete expectMaxResponseTime

Added docs and tests for the max response time assertion.

Coverage increased (+0.7%) to 93.127% when pulling a2e0ace2be813d0f0ff62d408baf48beadd17828 on Fishbowler:finish_max_response_time into 73d59f97b4b00731bdce8d91b749c78e3d7504cc on MarkHerhold:master.

Coverage increased (+0.7%) to 93.127% when pulling 16f5936293ba48103eda806d182c386f74bf79d7 on Fishbowler:finish_max_response_time into 73d59f97b4b00731bdce8d91b749c78e3d7504cc on MarkHerhold:master.


guidelines and resources for communication

Describe the task
Here all users’ suggestions with their followed category like for an example communication, should be available, for creating that view page, you have given down steps-
Expected behavior
Create a view page on hr guideline and resource where it would have tabular
data, with the following headers like,

Category - This will have all guidelines and resource for communication with edit and delete button where you can delete
and edit category as well.
Read by – it would have all users’ avatar whose set mark as read.
Suggestions- it would have particular avatar suggestion, if you click particular avatar in read by column on that time that
user’s suggestion will come on suggestion column.
Wireframes shared for reference

@kanuj365 Here is how I think this can be achieved:

$('.resource-avatar').on('click', (evt) => {
	const resourceClicked = $(evt.target)

	const resourceSuggestionDataUrl = resourceClicked.data('url')

	$.ajax({
		type: 'GET',
		url: resourceSuggestionDataUrl,
		success: (response) => {
			console.log(response)

			const resourceSuggestions = response.data // Expected array

			/*
				[
					'suggestion 1',
					'suggestion 2',
				]

				1. Start with string: '
'
				2. Loop over the resourceSuggestions array:
				3. In each iteration: concatenate '
 suggestion
' to the original string
				4. Finally concatenate: '
'

				Example: const string =

suggestion 1

suggestion 2

			*/

			// Give a class/ID to the place where you want to show the list of suggestions.
			// Grab that section using $
			// Use jquery `html` method to put/place the string generated above inside that section
		},
		error: (err) => {
			console.log(response)
			// show alert
		}
	})
})


Exempt parentIdFieldName from i18n rule

The parentIdFieldName attribute of the sl-table does not require translation, so add it to the exemptions.

FYI @m-akinc released 🎉


Darwin: Fix some more ambiguous Swift symbols.

The following were still ambiguous in Swift:

MTRCommissioningParameters.csrNonce
MTROTAProviderDelegate.handleBDXTransferSessionBegin
MTROTAProviderDelegate.handleBDXQuery

Use MTR_SWIFT_DISFAVORED_OVERLOAD instead of NS_UNAVAILABLE_FOR_SWIFT and introduce MTR_DEPRECATED that makes use of it to make Swift prefer non-deprecated symbols where there is ambiguity due to the old and new ObcJ name being imported as the same name in Swift.
Also avoid some typedefs for single-use completion block types since they add little value but carry a cost in terms of maintaining source compatibility going forward.

Fast-tracking platform-specific change with platform owner review.


Gantt sections are not separated by colors - Fix: set numberSectionStyles to 4 instead of 3

Hi,
First, thank you for this great tool, especially with gitlab!
I found that, the gantt diagram has a bug: some sections has the same background color and cannot see the separation.
It is also visible in your gitbook demonstration and I have the same issue in my gitlab: two following sections have the same color background, hard to recognize sections on the diagram.
Please check the screenshot from your gitbook page:

Did I miss something, or is it a minor bug? Thnx

Hi @pongraczi Thank you for this detailed bug report.
The default theme doesn't have this issue:

So maybe it's a theme issue.

I agree with you that numberSectionStyles should be 4 instead of 3.
I have fixed this issue. Please wait for the upcoming mermaid version 8.0


Fix custom device builds using encoding and decoding library

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
This change is a workaround for NI bug 1723615, which is a build failure in LabVIEW 2021 when using the BNR/BCD Encode/Decode malleable VIs from the encoding/decoding library.
This change prevents error 1004, claiming a VI is not in memory during a PPL build of the Ballard custom devices. This seems to stem from having nested malleable VIs, where the second level of malleables is contained within some structure. Investigation is still pending for the LabVIEW fix, but this change resolves the build failure.
Why should this Pull Request be merged?
Currently, neither the Ballard ARINC 429 nor the Ballard MIL-STD-1553 custom devices will build in LabVIEW 2021 due to the bug mentioned above. We need to build these custom devices in LabVIEW 2021 to support VeriStand 2021.
What testing has been done?
Ran the automated tests for the library in the repo. Built the custom devices in both LabVIEW 2020 and LabVIEW 2021. I have not run any functional tests of the custom devices yet, but did verify the build outputs look correct.

I built the Ballard ARINC 429 custom device using these changes and ran performance comparisons between the 20.6 release of the custom device. To run the tests, a parameters file with a huge number of parameters creating large numbers of inputs and outputs across multiple channels was generated and a loopback was set up to make sure encoding/decoding worked correctly and that PCL timing and CPU usage were within the same general windows.
The maximum PCL rate without jitter was within about 2 Hz and the total CPU usage was about 0.5% less with the changes made in this PR. The test was run on a PXIe-8880 with VeriStand 2020 R6. The hardware and parameters files used for testing are attached to NI Bug 1723615.


Unary operators

Running agda2hs on code like
+++_ : Nat → Nat
+++_ x = 2 * x
{-# COMPILE AGDA2HS +++_ #-}

usage : Nat
usage = +++ 5

reports an error
Invalid name for Haskell function:  +++_

Is there a way to define unary operators in agda2hs?
Note that according to the Haskell Wiki, "Unary operators may also be defined and used by the programmer with similar rules to infix operators."

Currently this is not supported by agda2hs, though a PR adding support for it would be welcome. (I personally didn't even know that Haskell had unary operators.)

What does the Haskell code look like for the unary operator (I also was not aware of user-defined unary operators in Haskell)?

Hi @bwbush , what is the expected code this should compile to?


Only Audio Publishing

Is it possible to publish rtmp only audio not video through your library.Kindly help me..

Can anyone help?

Have you got any solution regarding this issue? means only audio
On Thu, Apr 4, 2019 at 9:17 AM quyduc1412<EMAIL_ADDRESS>wrote:

Can anyone help?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TakuSemba/RtmpPublisher/issues/23#issuecomment-479739475,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AYCWM_8ZRXuKyTB15BXQfrXEAWRdYzlzks5vdXXZgaJpZM4cFDzO
.


refactor(action-block): remove log for html2canvas and node

html2canvas: https://html2canvas.hertzen.com/
node: type Node = designerCabin.Node

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


fix: Import initial implementation

Description
Contains the initial implementation of the Deployable architecture
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Initial implementation.
Contains separate modules for different components, ibm_catalog.json, solution
The solution will fully setup the account to onboard to apptio cloudability. This includes:

Creation of a secure COS bucket that is encrypted with key protect
Enables billing report exports for the account to the cos bucket
Grants access to apptio cloudability service id to read the data from the cos bucket, and retrieve the list of accounts in the case of an enterprise
Adds the account to apptio cloudability

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

Thanks @ocofaigh for the re-review. I have updated the PR based on your additional comments.


[WIP]: Implement new profile page

Description:
WIP version of https://github.com/LN-Zap/zap-desktop/issues/2317
Motivation and Context:

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes:

Checklist:

[ ] My code follows the code style of this project.
[ ] I have reviewed and updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[ ] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.1%) to 19.743% when pulling ea9d98cfa563ce7995aa3515cce54f3923e6c1b2 on mrfelton:feat/profile into bc1dfdf145633806590248026c0e4dc4df050d12 on LN-Zap:next.

Have made those updates @korhaliv and refactored the new info selectors based on  our slack convo


Add schoolmail domain

Please add my school, thank you
Middle East College
mec.txt

Middle East College
mec.txt

@Vector1dev This domain does not belong to a university, nor to a college. I have to close the request without merging. Please ensure that you are familiar with this repository rules located at the bottom of this page before submitting a new request.
@Ethea2 Please ensure you are familiar with this repository rules located at the bottom of this page and submit a new separate request with correct information. Thank you for the understanding!

@philipto It domain of russia schools.


[FEAT] Provide something like interactive(scale:) for custom animators or method like didScroll

It would be nice to have the ability to track all scroll way between Pages. .onDraggingChanged works great but only when the user touches the screen. As fast as the user finishes the pan gesture progress stops. But I need to continue my custom animation (for example change background color)

Hi @AndrewPetrov  let me see if I understand you. So if you start scrolling and it's enough to move forward, onDraggingChanged reports while there's a touch but as soon as you let go and the animation starts then you don't get reported. Is that it?
I don't have any way to get that information but you could do the following. You can calculate a progress based on that offset that's changed and then once onDraggingEnded is called then you finish the animation.


Description
When building a c-archive using the command go build -buildmode=c-archive on Intel-based macOS 14.6 (version 23G80) with Xcode 15.4 (Build version 15F31d), the resulting archive file is invalid.
Reproduction Steps

go build -buildmode=c-archive -o hello.a golang.org/x/mobile/example/basic
A file named hello.a is successfully created
xcodebuild -create-xcframework -library hello.a -output hello.xcframework
xcodebuild fails with error: unable to find any architecture information in the binary at '/Users/issue/hello.a': Unknown header: 0x50000000

What did you see happen?
xcodebuild complains that the hello.a is not a valid binary.

lipo -info hello.a output: Non-fat file: hello.a is architecture: x86_64
file hello.a output: hello.a: current ar archive
hexdump -n80 -C hello.a to show the first 80 bytes:00000000  21 3c 61 72 63 68 3e 0a  2f 20 20 20 20 20 20 20  |!<arch>./       |
00000010  20 20 20 20 20 20 20 20  30 20 20 20 20 20 20 20  |        0       |
00000020  20 20 20 20 30 20 20 20  20 20 30 20 20 20 20 20  |    0     0     |
00000030  30 20 20 20 20 20 20 20  32 34 30 30 20 20 20 20  |0       2400    |
00000040  20 20 60 0a 00 00 00 50  00 00 09 a4 00 00 09 a4  |  `....P........|

What did you expect to see?
The go build command should produce a valid c-archive so xcodebuild -create-xcframework -library hello.a -output hello.xcframework could successfully creates the "hello.xcframework": xcframework successfully written out to: /Users/issue/hello.xcframework.

lipo -info hello.a output: Non-fat file: hello.a is architecture: x86_64
file hello.a output: hello.a: current ar archive random library
hexdump -n80 -C hello.a to show the first 80 bytes (compare these numbers with the ones above):00000000  21 3c 61 72 63 68 3e 0a  23 31 2f 32 30 20 20 20  |!<arch>.#1/20   |
00000010  20 20 20 20 20 20 20 20  31 37 32 32 35 33 38 34  |        17225384|
00000020  36 36 20 20 35 30 31 20  20 20 32 30 20 20 20 20  |66  501   20    |
00000030  31 30 30 36 34 34 20 20  32 37 32 34 20 20 20 20  |100644  2724    |
00000040  20 20 60 0a 5f 5f 2e 53  59 4d 44 45 46 20 53 4f  |  `.__.SYMDEF SO|

This issues appears to be a platform-specific issue related to Intel macOS 14.6, it is not observed on "Intel macOS 13" or "Apple Silicon macOS 14".

Related Issues and Documentation

cmd/cgo: failed to run c-archive library built for iOS #41177
cmd/link: wrong c-archive architecture using GNU binutils ar on  macOS Mojave 10.14.1 #28796
cgo -buildmode=c-archive out ar file not current ar archive random library at macOS #47546 (closed)
CGO seems to force x86 on Apple Silicon #46724 (closed)
x/mobile/cmd/gomobile: gomobile bind fails with Xcode 15 #63141 (closed)
cmd/cgo: does not compile for iOS simulator arm64 #57442
cmd/go: cross-compiling for Linux x86_64 with CGO -race enabled failed on macOS Mojave (Darwin) #30863 (closed)
x/mobile: building for iOS Simulator, but linking in object file built for iOS, for architecture arm64. #45805
cmd/cgo: building c-archive with function pointer works with Go 1.14, but not 1.15 and beyond #44648
cmd/go: Go 1.8.x cannot build executables for darwin_arm and darwin_arm64 on macOS, but Go 1.7.x could #20456 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

Root caused found, I have GNU ar installed (included in binutils), after deleting that, it works.


Show which subpages are going to be deleted

This makes it harder to accidentally delete the wrong page, as it's more obvious whether a page has unexpected descendents

Inspired by the way Django Admin shows affected models during a delete. This implementation should be much faster than Django's, as we know we only need to show descendant pages.
Someone who is more CSS-y, I'm open to improvements. A simple ul definitely felt fine enough.
Please check the following:

[ ] Do the tests still pass?[^1]
[ ] Does the code comply with the style guide?

[ ] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

question: are there any UI and pontentially performance considerations if we're deleting a very large tree? (say a few hundred to a few thousand pages)

I haven't tested this trying to delete a large number of pages, but it should be ok. There shouldn't be any O(n) here, and the actual query / iteration should be pretty fast.
My only slight concern would be with loading that many pages into memory at once, but that could be solved by doing an extra query for .count and then using .iterator, which is probably better and likely to scale much nicer.
I'll do some benchmarking.

@RealOrangeOne there's also #7984 to consider (in an i18n context, with WAGTAILSIMPLETRANSLATION_SYNC_PAGE_TREE = True). I suppose in that context we'd only provide the updated counts messaging and only display the pages from the originating locale

@zerolab rewrote the PR from scratch, taking into account the translation changes. Performance will probably be about as good as we can get it. Showing individual translations could show a lot of pages!
Something I was thinking is whether just listing the subpages, and only showing the count of translated pages, but that could get complex.
Adding the page count to the delete button could also just be enough to prompt the user to be careful?

See also #8703 as an alternative solution to avoiding accidental deletion.

What if we combined #8703 and this? I am thinking the full list of pages available on separate URL, linked from the deletion confirmation message

@RealOrangeOne what are you thinking for this PR now? are you keen to rebase it on main now that #8703 has gone in? or close this one for now?

I think that PR makes it more prominent when lots of pages are going to be deleted - I think it's enough for now.


serverless deploy this bot

Hi, you have been of great help and im almost ready to go live with this bot for an ai app.
but the memory 40-50mb limits having multiple instances.
you have suggested /commands and one instance for all users. thats very good but in matrix client it limits the customization of user icon and user profile and description.
so for some use cases i need to go with multiple bots.
hence exploring the serverless option for deploying these bots.
https://vlad.roam.garden/How-to-create-a-serverless-Matrix-Chat-bot
this implementation shows how to do it for unencrypted rooms
do you have any suggestions here. that work in encrypted rooms also
thanks

I think memory is cheaper nowadays compared to some severless solutions and you have more control.
Some serverless providers support deploy with docker.
https://fly.io/docs/languages-and-frameworks/dockerfile/
https://devcenter.heroku.com/categories/deploying-with-docker
https://docs.railway.app/deploy/dockerfiles


When i use the empirical_attention plug_in in  the resnet_34 of mask_rcnn, it occur a problem.

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
When i use the emirical_attention plug_in in  the resnet_34 of mask_rcnn, it occur a problem. But i had run the resnet_34 of mask_rcnn successfully.

Reproduction

What command or script did you run?
python tools/train.py configs/empiricxal_attention/mask_rcnn_r34_fpn_2x.py --work-dir work_dirs/attention_r34

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
I change the empirical_attention  first line :  base = '../mask_rcnn/mask_rcnn_r34_fpn_2x_coco.py'

What dataset did you use?
The dataset is mine. And the dataset is coco dataset.
Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.

You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]      conda
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

Hi, what code did you change? For example, what code did you change in config file?

Hi. Your teammate had told me that the Resnet_34 is BasicBottle, But empirical_attention plug_in is BottleNeck ,not support in BasicBlock.
Do you have any solution?

You can write a Model to support using bottleneck in Resnet-34. However, I don't think this is a good way to fix this error, because using bottleneck in Resnet-34 may get lower performance. Maybe you can use Resnet-50?


[youtube_dl] Added cancelable metadata / playlist downlod progress hook

I added cancelable hooks to playlist metadata downloads.
Here an example showing how to skip downloads after the 3rd entry of a playlist:
import youtube_dl

url = 'http://www.youtube.com/watch?v=QonQ_8qnAbY&list=PLC3CC1956BA739976&index=8'

ydl = youtube_dl.YoutubeDL({
    'ignoreerrors': True,
    'quiet': True,
    'socket_timeout': 10
})

ydl.add_default_info_extractors()

# dummy hook
def hook(status):
    if status['current'] > 3:
        print "Skipping..."
        return False # returning False from any of the added hooks makes the next request skipped
    else:
        print(status)

# Adding the hook
ydl.add_metadata_hook(hook)

result = ydl.extract_info(url, download = False)

Sorry, without insufficient context my imagination fails me why this would be useful - why not simply pass in --playlist-end 3? What does this option enable that was missing previously?

@phihag<EMAIL_ADDRESS>@disfasia @dstftw  Any news?


Rewrite ImageView based on GtkReactive

Here's the NEWS entry, pasted for convenience (also, you can see the new README here):
New in 0.5
ImageView has been rewritten from scratch. Effort was made to maintain
backward compatibility where possible.
Breaking changes

The return value of imshow has changed; it is now a Dict that
stores Gtk widgets, Reactive signals, etc.

Major features

This package now uses Gtk. Rendering is considerably faster in some
cases, and the package is faster to load due to precompilation.

Navigation and zoom region are controlled by GtkReactive/Reactive
signals, allowing one to more easily extract this information for
reuse elsewhere.  Examples are shown in the README.

A new contrast GUI is independent of any plotting package, leading
to faster loading and faster time-to-first-plot.

One can now display objects that are not subtypes of
AbstractArray. See test/cone.jl for a demonstration.

Deprecations

canvasgrid now returns more arguments; a deprecation warning
encourages transitioning to the new syntax

pixelspacing is deprecated as a keyword (use an AxisArray instead)

The xy keyword has become axes, and it takes dimension integers
or Symbols (if the image is an AxisArray)

<End of NEWS entry>
For me a major point of the rewrite was the ability to interactively "couple" images together, for example to show an original and segmented image side-by-side, and be able to zoom into the same region on both of them without any effort on the part of the user.
There aren't that many watchers of this package, so to give folks a fair chance to complain let me ping some people who have been active developers for the broader JuliaImages ecosystem: @Evizero, @bjarthur, @mronian, @tejus-gupta, @annimesh2809, @SimonDanisch, @ihnorton, @tkelman, @juliohm, @Cody-G, @tknopp, @mdhe1248.

From reading the readme this looks really amazing. I'll be sure to try it out soon

woa, this seems so much more complicated than ImageInTerminal.jl 😄

Nice work!  Can't wait to give this a spin.

woa, this seems so much more complicated than ImageInTerminal.jl :smile:

It's really hard to compete with ImageInTerminal's elegance. 3d/4d is my niche!

This is working well in my testing. I'll merge over the weekend barring objections.

I did some quick testing, and it was very nice. I did run into an issue with the contrast control on macOS. Unfortunately, I don't have time to dig into it at the moment to give you a report-- perhaps in a few weeks.

Thanks for testing. No problem about the report, presumably someone else will complain and hopefully that will be enough to figure out the problem.
I want to come up with an easier API for linking two axes together, so there will be some additional changes later. But I don't think that needs to hold this up.


[IntelliJ Plugin] Add diagram view and editing support

Purpose
This PR introduces diagram view and editing support for ballerina intellij plugin.
Samples

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #15399 into master will decrease coverage by 8.45%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master   #15399      +/-   ##
============================================
- Coverage     26.38%   17.92%   -8.46%
- Complexity      123     1162    +1039
============================================
  Files            92      576     +484
  Lines          2172    18327   +16155
  Branches        338     3237    +2899
============================================
+ Hits            573     3285    +2712
- Misses         1502    13678   +12176
- Partials         97     1364    +1267

Impacted Files
Coverage Δ
Complexity Δ

...a/sdk/BallerinaSdkLibraryPresentationProvider.java
0% <ø> (ø)
0 <0> (?)

...lugins/idea/actions/BallerinaCreateFileAction.java
0% <ø> (ø)
0 <0> (?)

...io/ballerina/plugins/idea/BallerinaModuleType.java
16.66% <ø> (ø)
2 <0> (?)

...a/io/ballerina/plugins/idea/BallerinaFileType.java
42.85% <ø> (ø)
4 <0> (?)

...o/ballerina/plugins/idea/sdk/BallerinaSdkType.java
0% <ø> (ø)
0 <0> (?)

...idea/project/BallerinaProjectLibrariesService.java
0% <ø> (ø)
0 <0> (?)

.../idea/runconfig/BallerinaRunConfigurationBase.java
0% <ø> (ø)
0 <0> (?)

...idea/runconfig/BallerinaRunLineMarkerProvider.java
0% <ø> (ø)
0 <0> (?)

...lerina/plugins/idea/BallerinaProjectGenerator.java
0% <ø> (ø)
0 <0> (?)

...w/diagram/actions/BaseChangeSplitLayoutAction.java
0% <0%> (ø)
0 <0> (?)

... and 528 more

Continue to review full report at Codecov.

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

@Shan1024 Could you please review the changes you requested? :)


add await to the async isValidGitDirectory

Requirements for Contributing a Bug Fix

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only fix an existing bug. To contribute other changes, you must use a different template. You can see all templates at https://github.com/atom/.github/tree/master/.github/PULL_REQUEST_TEMPLATE.
The pull request must update the test suite to demonstrate the changed functionality. For guidance, please see https://flight-manual.atom.io/hacking-atom/sections/writing-specs/.
After you create the pull request, all status checks must be pass before a maintainer reviews your contribution. For more details, please see https://github.com/atom/.github/tree/master/CONTRIBUTING.md#pull-requests.

Identify the Bug
Fixes #22577

Description of the Change
Pretty self explanatory
isValidGitDirectory is an async function, and so actually returns a Promise which is always truthy

Alternate Designs
N/A

Possible Drawbacks
N/A

Verification Process
N/A (Hopefully)

Release Notes
The findGitDirectory function now checks if the directory is valid

There might also be a bug on line 144 with non-awaited promises, but lgtm.com doesn't say anything about it, so maybe it's intentional??

Looks like line 144 is just returning the promise instead of returning the awaited value.
It shouldn't make a difference but I feel like it should await the last call just to be consistent.


Fixed crashing issue

Fixes an issue where for some reason, the error code was missing. Instead of crashing the program, it now returns undefined

I appreciate the PR but in this instance, there shouldn't be any situation where either field provided is undefined/null so I don't really want to go about it this way.
I've just looked into the issue you previously reported and it was caused by an additional 'data' field lookup which shouldn't have been there on: https://github.com/MC-Market-org/js-api-wrapper/blob/3cc509f30a91a60f8119db47418d4fe6c2f566e6/src/Http.js#L65
I'll push a fix for that & a few other changes within the next few days so I'm not going to merge this. Thanks nonetheless.


Cleanup package dependencies

Looks good, would you mind opening the same PR on the official repo? See if they agree and keep differences to a minimum.


⚠ Samsung has degraded performance

In f9b844c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2106 ms

Resolved: Samsung performance has improved in 0d279c0.


Pass Cisco 200-901 Exam

I hope this post will be useful for people who wants to take next step in their career and Pass Cisco 200-901 Exam certification. The range of knowledge as you can see is wide and sometimes you need to know the details, but this Pdfcerts will make you a better specialist and will open more opportunities in front of you. I hope you pass Cisco 200-901 exam questions and find your name under Cisco Certified DevNet Associate Developing Applications and Automating Workflows using Cisco Core Platforms Exam!
Download Full Practice Exam Material Here: https://www.pdfcerts.com/Cisco/200-901-exam-questions.html

Thank you for your PR.  We can only accept content that strictly complies with the Cisco Certification and Confidentiality Agreement, also known as the Cisco Certification Non-Disclosure Agreement (NDA).  We welcome and appreciate any Cisco NDA-compliant contributions from the community.


broadcasting operations in BIDMat?

Any function in BIDMat that can do something similar to broadcasting in python? For example we have a matrix X of size 1 * N, and a matrix t of size 1 * n, we want to get a matrix of size n * N, where every row of the resultant matrix is (X - t(i)).

There's a lot of broadcasting in BIDMach. Example:
scala> val a = rand(3,3)
a: BIDMat.FMat =
   0.62465   0.26990   0.39953
   0.63891  0.050579   0.91040
   0.84862   0.73763   0.49750

scala> val b = ones(3,1)
b: BIDMat.FMat =
   1
   1
   1

scala> a + b
res3: BIDMat.FMat =
  1.6247  1.2699  1.3995
  1.6389  1.0506  1.9104
  1.8486  1.7376  1.4975

Thanks, in the particular case I mentioned above where X is just 1 * N but the result should be n * N dimension, simply X - t does not work. One solution I can think of it's to replicate X in multiple rows and deduced by t.t (t transpose). Any alternatives?

I see. We can use the function kron for this purpose.

Yeah, glad that worked! The kron operator is definitely useful.
I'm closing this question, since it seems like it was resolved.


ClinVar Clinical Significance not being consistently pulled in

Moved from genome/civic-server#275 as it looks like a client issue.
Basically this is another case where  MyVariant.info gives an inconsistent response. In this case its clinvar.rcv. Sometimes rcv is a single object (not in an array), sometimes its multiple wrapped in array. Right now it looks like its only handling the array case: clinvar.rcv[0].clinical_significance

An example of each type of MyVariant.info query:
chr7:g.87160618A>T
chr7:g.140453136A>T


changed background LightImplementation of LightPsiClass

This looks like a fix for #145
The new implementation of LombokLightClassBuilder provide correct package for generated BuilderClass (the same as the containing class). Previous Implementation provided default package for all generated classes. I think this was a root of the problem in #145.
@AlexejK can you have a look and maybe do more manually tests in your environment?

Codecov Report

Merging #407 into master will decrease coverage by 0.08%.
The diff coverage is 67.56%.

@@             Coverage Diff              @@
##             master     #407      +/-   ##
============================================
- Coverage     61.79%   61.71%   -0.09%
+ Complexity     1353     1344       -9
============================================
  Files           168      168
  Lines          5047     5028      -19
  Branches       1158     1158
============================================
- Hits           3119     3103      -16
  Misses         1427     1427
+ Partials        501      498       -3

Impacted Files
Coverage Δ
Complexity Δ

...ij/plugin/action/delombok/BaseDelombokHandler.java
78.26% <100%> (+0.19%)
30 <0> (+1)
:arrow_up:

...ellij/plugin/processor/handler/BuilderHandler.java
81.45% <100%> (-0.07%)
101 <0> (ø)

...v/intellij/plugin/psi/LombokLightClassBuilder.java
65.21% <62.5%> (-4.02%)
17 <15> (-9)

.../plushnikov/intellij/plugin/util/PsiClassUtil.java
56.36% <0%> (-1.82%)
21% <0%> (-1%)

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 cf68968...f04e273. Read the comment docs.

Cool, will test during next week!


Use correct base image for output artifacts

What this PR does / why we need it:
In Dockerfile, the terminal stages that produce our images are using
BUILDPLATFORM to determine the architecture of the base image before
pulling in the static binary. It should be TARGETPLATFORM.

It's blowing up the CI, so there must be something wrong here. I'll look into it


Switch to getLiteUser where applicable on frontend

This PR switches out useUser and useUsers with useLiteUser and useLiteUsers, which returns a much smaller user object. Backend code was already merged for this.
Closes #4936
Web frontend checklist

[ x ] Formatted my code with yarn format
[ x ] There are no warnings from yarn lint --fix
[ x ] There are no console warnings when running the app
[ N/A ] Added any new components to storybook
[ N/A ] Added tests where relevant
[ x ] All tests pass
[ x ] Clicked around my changes running locally and it works
[ N/A ] Checked Desktop, Mobile and Tablet screen sizes

wow such a nice job, looking good!

I'll test to see if everything works good on the page

@aapeliv @bakeiro just checking up on this one. I did click around the app already for this but if anyone wants to take a second look I couldn't check the reference user info as I don't have any references in my test account.
Although we had pretty good tests for most of these components so hopefully it would catch anything.

sorry, totally forget, if the test was good then is a ready to go for me, I´ll check it briefly, if all good I approve it :)

This is exciting to see - the original useUsers hook implementation has been a hack all along to work around the limitation of not being able to get multiple users in one trip! I'd go as far to say a good measure of this new API doing its work is that there should be no useEffects and the weird workaround with the ref to store the list of user IDs once you've fully switched over to the new RPC!

This is exciting to see - the original useUsers hook implementation has been a hack all along to work around the limitation of not being able to get multiple users in one trip! I'd go as far to say a good measure of this new API doing its work is that there should be no useEffects and the weird workaround with the ref to store the list of user IDs once you've fully switched over to the new RPC!

Thanks for this, I was getting hung up on refactoring the existing hook but this made me realize I should just re-write and use my own common sense haha, since the whole api call works differently.
Done now, re-wrote the tests. Could be missing some test cases, but I have most of the old ones covered (except the invalidate param since we don't need to pass that anymore).

@darrenvong Thanks so much for the thorough code review! I did a bit of refactoring as your comments made me realize I was not fully understanding how the useQuery hook should be working. It's ready for re-review now!


add nyumba hole

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 15 (86.67%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 97.545%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/js/core/Board.ts
4
6
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
549

Relevant Lines:
556

💛  - Coveralls


Can I use the events from WebSocket/SignalR (javascript) on blazor?

The Blazor is pretty fast for now.
I almost click on an element and I imediately get an answer from the server.
But this is an hypothethical scenario.
The demos on internet, or some UI (like devexpress components) has some lag.
This is normal, i will always has lag, but thinking in a large system using Blazor this lag can be more intensive.
Given this scenario, can I 'intercept' some events from WebSockets/SignalR (in client-side) and call something like the blockui (http://malsup.com/jquery/block/#page)?
Something like blocking page/component before send an message to the blazor server and unblock when receive it.

Thanks for contacting us, @jeanklann.
You should do whatever you can to reduce the latency. If however the final outcome is not satisfactory, you should consider switching to Blazor WASM.
SignalR Hub is an implementation detail for Blazor and we do not expose it for consumption directly.
We will, however, expose certain configuration options/events to enable certain scenarios. For this particular case, we haven't heard many asks to enable this. If we hear community asking for this a lot, we will consider doing something here.


Improved existing phone number verification jquery hack

#191 , #196

@vrthra Requesting testing

Tested on the desktop.

@vrthra sure . On it

@vrthra Please test

@Gotham13121997
Ok, here is what I did; I tried adding 11 clicked submit. It told me that there is an error. Then I continued with 22 33 44 55 at each point it told me to provide a valid phone number. The error stayed even after I reached all ten digits (55), and went away only after a few seconds. This can cause confusion. Is there a better way?

ok I will check

@Gotham13121997 didn't seem to stop me from submitting "123" as a phone number on iOS Safari

@shajith thats weird. Were you able to submit the form?? This is why backend verification is necessary.

@shajith it seems oninvalid is not supported in safari

@Gotham13121997 caniuse indicates that iOS Safari supports input field patterns, and it works on http://html5pattern.com/. The other validations kick in, just the pattern thing - it let me submit with just "123". It did prevent submission if I left the field blank though.

@shajith can you try this commit and check. If not working then the default message will have to be left as it is

sorry for the lag, but no luck - same behavior on latest.

We will need to rely on server side verification for safari as it seems pattern is not supported. @shajith  @biswaz @vrthra @salmanulfarzy #196

closing this as #484  is better option


Inventory cleanup

I'm going through testing all of @dbernstein's PRs for our roles, and I'm noticing lots of small things doing all of this on droplets instead of locally through vagrant.  I'm making this ticket as a bucket for cleanup PRs to our roles/inventory.
My goal here is to shrink the number of variables you have to edit when provisioning a remote server.  There's big chunks I can eliminate by pushing things into variables.

@dannylamb
Perhaps, related issue: https://github.com/Islandora/documentation/issues/1434

Resolved by https://github.com/Islandora-Devops/ansible-role-alpaca/commit/4896d8507b386ac7c394032ea46416a61d287309


Particle functionalities may generate false figure if memory is almost full

Particle Functionalities May Generate False Figure if Memory is Almost Full
Related Issue

#32

Description
Initially, particle plots may generate false figure, if the MPI size is too large. After testing on different machine, this doesn't seems to be an issue in libyt. This is more or less related to memory space of one machine has. But we still cannot find where actually is this issue.
(I haven't reproduce the issue.)

I cannot reproduce the issue... :sweat:
I guess it is because I move .save() outside of if yt.is_root(), so that each rank is creating the same file which caused the problem.
Or I simply implemented an ill-functioned libyt that time.
Anyway, I'm closing this issue since after running lots of examples, I did not encounter this issue at all, not even once.


Handler combined data on Year/Month detail report calculation

Resolved on:
https://github.com/chaineco/bisonexplorer/commit/d722550c092045f5d5a1dca788a19bbec56d96d1


option to disable language autodetect

My use case required this option, so I added it. Test included.

I'm using this patch; seems to work well.
Do you know if there's a way to have the "hljs" class still applied even if to type was specified and auto detection is off?


Add helmet to improve default security

@pnw

It's considered a best practice as listed on the express website. We could
provide a way to turn it off in congress I guess. That's probably a good
idea.
On Tue, 29 Nov 2016 at 11:47, Patrick<EMAIL_ADDRESS>wrote:

@pnw commented on this pull request.
In src/classes/application.js
https://github.com/ash-framework/ash/pull/6#pullrequestreview-10509530:

@@ -160,6 +161,8 @@ module.exports = class Application extends Base {
const app = express()
_app.set(this, app)

app.use(helmet())

Hmm is it presumptuous to assume everyone is going to want helmet? Should
it be something you can turn off in the config? Id say that ceding control
to the end user is good - helmet is on by default, but they can turn it off
with ENV.helmet = false or ENV.middleware.helmet = false or something.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ash-framework/ash/pull/6#pullrequestreview-10509530,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABH2CrFni8fxVdBBL8AKfDw1aTh4azcqks5rDALagaJpZM4K9vVE
.

Did this work on master


chore: nicer error message for windows users

From https://phpc.social/deck/<EMAIL_ADDRESS>
Hi, curious library. Users are complaining on socials about using [python because of] it, and I'm not sure it's fair on either side.
I Can see that sysops folks might not want to use python in the same way I want to. (I would open a damn sub-process)
This new error lets folks know that only unix style python is supported, and points them at Docker, which I see you make use of in this repo.
The core use case is:
As a user, who has been able to successfully pip install sh
I would like to receive meaningful errors, that signpost me
So that I do not rage on socials, about python itself, or try to install non-packages such as fnctl
I do also intend upon raising this with python. Maybe this would be a good stub behaviour for fnctl so that users at least know "hey we deliberately chose not to include this" rather than "Module not found"

fnctl = { version = "^1.0", markers = "sys_platform != 'linux' and sys_platform != 'darwin'" }

Might also work now I've published a package to pypi called fnctl it might be prettier than this.

bugger, got the name wrong in that python package. No wonder it didn't complain 😂

see https://github.com/python/cpython/issues/125568 and https://github.com/pypi/support/issues/4929 as well...

btw tested the following monstrosity and confirmed still working with sh
poetry run pip install https://github.com/Lewiscowles1986/python-fcntl/releases/download/1.0.0/fcntl-1.0.0-py3-none-any.whl

then
import sh
from sh import bash

bash(['-c'], 'echo hi')

under windows it obviously shows the error discussed / proposed here and exits the code.

I think we just need to code this error up earlier in the file:
https://github.com/amoffat/sh/blob/develop/sh.py#L73-L76
If you agree, would you mind making this change?

Hi Andrew, I don't necesarrily disagree, but if you look at the imports needed and import ordering, it might conflict with a few linting rules.
Thank you for considering this. I've also approached python and pypi, as it's sad that the library needs to solve for what feels like a runtime defficiency in standard libraries.
Also are you principally against asking folks to run in a docker container? The reason I mention that is that for a Windows user, it's not that windows is not supported per-se, as much as they have a hobbled core utils layer (WSL, Git Bash or Docker).
The thought was also that the language is that this is a windows problem, but it's not. It's a non Linux or OSx problem.

I think it's fair to keep the wording in our error message that Windows isn't supported. If you run Docker on Windows, or any other virtualization platform for that matter, then you're effectively not running Windows anymore.
I like the solution of printing the error message before attempting to import packages not available on Windows. We can just add noqa specifiers to silence linters.

Apparently no linting errors anyway.
Closing this in favor of #740


Prevent ModalBottomSheet from rebuilding its child

Description
This PR uses the child property of the AnimatedBuilder (as documentation explains) in order to prevent ModalBottomSheet's child widget of being rebuilt needlessly.
Checklist

[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.
[ ] Yes, this is a breaking change.

The change looks reasonable. In order to accept this PR you will have to sign the CLA (see message above) and add a test to ensure that we never regress this again. Can you please do those two things? Thanks!

@googlebot I fixed it

@googlebot I fixed it.

Looks like there is still a problem with the CLA. From what I can tell at my end it looks like the email address associated with the git commit doesn't match the email address you used when signing the CLA. Maybe update the email address of the commit to match? See here for instructions: https://stackoverflow.com/questions/3042437/how-to-change-the-commit-author-for-one-specific-commit

@googlebot I fixed it.

Some commits in this PR still use an incorrect email address. Can you fix those as well? (Or alternatively, squash everything into one commit with the correct address.)

@goderbauer mmmm I have updated the author of all commits but it seems that new commits were generated instead of overwriting the previous ones. I'm trying to squash all them into one single. Thanks ;)

@goderbauer I was not able to edit email's author of existing commits or stash all into single one. So I have finally created a new pull request with a single commit  with a valid email and I'm closing this one. Here is the new PR https://github.com/flutter/flutter/pull/55832 Sorry for the mess.


[GHI #125] Enhanced Color Support

Task
#125
#16
What Changed

[x] Add documentation for colors
[X] Update Light mode colors
[X] Add alt colors to Light mode scales
[x] Update Dark mode colors
[x] Add alt colors to Dark mode scales

Sanity Check

[X] Have you updated any usage of changed tokens?
[X] Have you updated the docs with any component changes?

[X] Have you tried building the css?
[X] Have you tried building storybook?
[X] Do you need to update the package version?

Screenshots

 Light Theme Revisions

 Dark Theme Revisions


EnzymeRules

We need an easier way to register custom adjoints that are compatible with Enzyme. The tricky bit here is coming up with a good ABI.
The goal would be to somewhat mimic Diffractor,jl (cc: @simeonschaub), meaning that the rule selection would run before activity analysis and during Julia inference. Alternatively we could attempt to have a callback function that can create additional code after activity analysis, but I rather would like that kind of unlimited compiler recursion.
So taking Base.sin as an example. During type-inference we have Tuple{typeof(Base.sin), Float64} (and maybe the return type) and we somehow need to map that onto:
function tfunc_rrrule(::typeof(Base.sin), ::Type{Active}, ::Type{Active{T}}) where T
    function ∇sin(ret_grad, x::Active{T}, tape) where T
        return cos(x) * ret_grad
    end
end

function tfunc_rrrule(::typeof(Base.sin), ::Type{Const}, ::Type{Active{T}}) where T
    function ∇sin(ret_grad, x::Active{T}, tape) where T
        return zero(T)
    end
end

or:
function rrrule(::typeof(Base.sin), ret_grad::Active{T}, x::Active{T}, tape) where T
    return cos(x) * ret_grad
end

function rrrule(::typeof(Base.sin), ret_grad::Const{T}, x::Active{T}, tape) where T
    return zero(T)
end

cf https://github.com/JuliaDiff/ChainRulesCore.jl/issues/452
and in particular the gist: https://gist.github.com/oxinabox/c6ad25c468b3108f8a799bda66c147f8/
I suggest that this should be done in a way that depends on ChainRules and ChainRulesCore as a prototype.
then once we get it nice and stable we move it into those packages.
Since similar things are wanted needed by operator overloading ADs like Nabla.

I am a bit wary of depending on ChainRules immediately since most of the rules are written using a closure passing/capture style.
And I am not sure that we can make that work well with Enzyme. That's why in the example above the rule is explicitly receiving the input arguments.
But maybe @wsmoses has some clever idea for squaring the circle.

I think the ChainRules and proposed EnzymeRules interface solve two orthogonal problems.
The need for the design proposed here is making an interface for Enzyme to take any Julia custom derivatives (besides ones we've internally hard-coded). This immediately presents questions as to how we resolve such custom rules (wrt type of argument, activity, etc, from the Enzyme internal conventions).
I think the chain rules dependency solves a related, but yet technically distinct problem of importing existing definitions for custom rules (as opposed to the former making an interface).
I'm inclined to continue on both fronts (and they should eventually meet once here the custom derivative registration interface has been created, and ChainRules supports activity annotations)

module EnzymeRules
    """
        augmented_primal(::typeof(f), args...)

    Return the primal computation value and a tape
    """
    function augmented_primal end

    """
    Takes gradient of derivative, activity annotation, and tape
    """
    function reverse end
end

using Test
using EnzymeRules
using Enzyme

f(x) = x^2
EnzymeRules.augmented_primal(::typeof(f), x::Active) = (f(x.val), nothing)
EnzymeRules.reverse(::typeof(f), ::Tuple{Active}, dret, tape) = (2*dret,)

EnzymeRules.reverse(::typeof(f), ::Tuple{Const}, dret, tape) = (zero(dret),)
# or is this
EnzymeRules.reverse(::typeof(f), ::Tuple{Const}, dret, tape) = (NoTangent(),)

g(x, y) = x * y
EnzymeRules.augmented_primal(::typeof(g), x::Active, y::Active) = (g(x, y), (x, y))
EnzymeRules.augmented_primal(::typeof(g), x::Const, y::Active) = (g(x, y), x)
EnzymeRules.augmented_primal(::typeof(g), x::Active, y::Const) = (g(x, y), y)

EnzymeRules.reverse(::typeof(f), ::Tuple{Active, Active}, dret, (x, y)) = (y*dret, x*dret)

EnzymeRules.reverse(::typeof(f), ::Tuple{Const, Active}, dret, x) = (NoTangent(), x*dret)
EnzymeRules.reverse(::typeof(f), ::Tuple{Active, Const}, dret, y) = (y*dret, NoTangent())

# TODO:
# - Duplicated?
# - NoTangent() vs nothing
# - Combinatorial explosion
# - activity dispatch doesn't quite work yet

Combinatorial explosion

Yeah.

Per an earlier offline discussion.
Active vs const is easy since a const rule can always fallback to active and not use the derivative result.
Active and duplicated are incompatible so one value can only be one (Eg it is impossible for a float to be duplicated in reverse mode)
Duplicated vs const should have correct available rules. Hopefully the design enables taking say a union of either and being able to do the duplicated update conditionally on a type check.
Nevertheless this is something meriting design iteration

EnzymeRules has landed on main.


Azure.ResourceManager.DataFactory

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

/azp run prepare-pipelines

/azp run net - datafactory - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.DataFactory


Median Function in expression language

Add support for the median function in the expression language.

We should really talk (again) about Library function.
That wil solve these problems.
Willing to chat on discord - we can have a video call.

It would not solve the problem that we are talking about above. The request is to allow non-technical users to extract the median value within the decision table. Similarly to how we have:
avg([1, 1, 4]) -> 2
We want to expand to
median([1, 1, 4]) -> 1

For this case, i suggest an expression language. Something like Excel has
sum(), avg(), ...
There is this package for javascript: https://github.com/AjaniBilby/BNF-parser/blob/master/test/uv/uniview.bnf
You can basically create a parser (like in compiler crafting). The result is something that can be executed in Javascript.
You can create very complex and Excel like thigns eg.g if .. else ... excel-ish things business analysts love to use :)
Willing to help here.

Hi @egandro we already have expression language, here are references:
https://github.com/gorules/zen/blob/master/core/expression/src/compiler.rs
and implementation:
https://github.com/gorules/zen/blob/master/core/expression/src/vm.rs
We will just extend our expression language and add median and mean. I'm using ticket to track it

@ivanmiletic  I was thinking about this the whole day :)
I have a suggestion. Maybe you guys can wirte down a nice Unit Test for a future iteration of this.
Have all of your internal functions for your expression language tested against the equivalend functions within Excel.
From experience, Excel is a can of worms, when it comes to do "math". It would be super cool (since you have Business Analysts working with gorules) to be clear where Excel does math in a different way.
You might not need to implement this "now" but at some point the bug hunting will be much simpler.
I can imagine discussions like this :) "Excel gives me 14 contracts while >> your << tool gives me 15"

@egandro we can do that, however, the syntax is completely different between both tools.
We have pretty good and sane coverage with end-to-end tests that are currently in place, we will look into increasing it in the future.
For reference, the engine has about ~80% coverage and the expressions crate has ~78%. We'll likely look into improving it closer to ~85-90% but never aim for 100%.


ZK client times out against zk 3.3 and 3.4

My friend @dallasmarlow told me that he has been seeing client timeouts against zk 3.3 and 3.4 using this zk client, as the session timeout is not being properly respected.  This is causing his ephemeral znodes to constantly be invalidated.  We need to:

reproduce the timeouts
fix the zk dep or set up a background pinger that does something simple like perform a get on the root znode every few seconds (maybe every 10s with a small amount of splaying) to prevent the session from timing out due to failing to respect the heartbeat interval.

@spacejam https://github.com/samuel/go-zookeeper/issues/82
it looks like this doesn't cause an issue for the mesos-go client in most cases due to it's rewatch interval being so low (statically configured to 200ms), which in turn is acting like a heartbeat and keeping the session alive.

@spacejam: Ping.


TypeError on DataArray.stack() if any of the dimensions to be stacked has a MultiIndex

If I call .stack() on a DataArray, and any of the dimensions to be stacked already has a MultiIndex on it, I get an error:
TypeError: Must pass list-like as `names`.
Maybe it doesn't make sense for .stack() to work on an existing MultiIndex, but that should probably be documented, and perhaps throw a more informative error.
Here's the full trace:
Traceback (most recent call last)
<ipython-input-6-ce81ac38d71e> in <module>()
----> 1 xr_one_flat_stacked = xr_one_flat.stack(alphanumeric=("letters", "numbers"))
      2 xr_one_flat_stacked

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/xarray/core/dataarray.pyc in stack(self, **dimensions)
   1027         DataArray.unstack
   1028         """
-> 1029         ds = self._to_temp_dataset().stack(**dimensions)
   1030         return self._from_temp_dataset(ds)
   1031

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/xarray/core/dataset.pyc in stack(self, **dimensions)
   1832         result = self
   1833         for new_dim, dims in dimensions.items():
-> 1834             result = result._stack_once(dims, new_dim)
   1835         return result
   1836

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/xarray/core/dataset.pyc in _stack_once(self, dims, new_dim)
   1801                       else level
   1802                       for level in levels]
-> 1803         idx = utils.multiindex_from_product_levels(levels, names=dims)
   1804         variables[new_dim] = IndexVariable(new_dim, idx)
   1805

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/xarray/core/utils.pyc in multiindex_from_product_levels(levels, names)
     79                               indexing='ij')
     80     labels = [x.ravel() for x in labels_mesh]
---> 81     return pd.MultiIndex(levels, labels, sortorder=0, names=names)
     82
     83

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/pandas/indexes/multi.pyc in __new__(cls, levels, labels, sortorder, names, copy, verify_integrity, _set_identity, name, **kwargs)
    103         if names is not None:
    104             # handles name validation
--> 105             result._set_names(names)
    106
    107         if sortorder is not None:

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/pandas/indexes/multi.pyc in _set_names(self, names, level, validate)
    505         # set the name
    506         for l, name in zip(level, names):
--> 507             self.levels[l].rename(name, inplace=True)
    508
    509     names = property(fset=_set_names, fget=_get_names,

/Users/jjpr/anaconda/envs/dldata/lib/python2.7/site-packages/pandas/indexes/base.pyc in set_names(self, names, level, inplace)
    997
    998         if not is_list_like(names) and level is None and self.nlevels > 1:
--> 999             raise TypeError("Must pass list-like as `names`.")
   1000
   1001         if not is_list_like(names):

TypeError: Must pass list-like as `names`.

Thanks for the report.
I agree that a better error message would be helpful here, but I do think that stack can still be meaningfully if one of the components is a MultiIndex (e.g., by combining the existing levels with the new levels to be stacked)

I have been playing around with the MultiIndexes as part of #1317, so I could take a stab at implementing some sort of stack which combines the levels etc.

Any updates on this by any chance? (now that there seems to be progress on #1603)


Either StripeProvider or InjectStripe is stripping child component props

Summary

Hi folks! I am attempting to pass props to a child node of <StripeProvider />. It seems like as part of transforming the child object, props are not being re-applied to the child as it's rendered. I am following the structure shown in the readme. Example:
In parent component:
        <StripeProvider apiKey="pk_123">
            <CheckoutForm foo="baz"/>
        </StripeProvider>

In <CheckoutForm />, I'm exporting the component using injectStripe.
export default injectStripe(CheckoutForm);

Inside CheckoutForm, this.props.foo is always undefined, and the only prop that ever makes it is this.props.stripe. What am I missing? I
Thanks in advance!

@fholgado Yikes, that's no good. I have a failing test for this and it's definitely injectStripe that is stripping the props. I'll work on a fix and hope to have it released before too long.
In the interim, I hope you found a solution by juggling the way props get passed down in your application.

Thanks so much! I have a workaround I can use in the meantime, but it involves saving state to the ‘window’ object, so it’s not great 😂
––
Fed
On May 6, 2019, 11:06 PM -0400, Adam Solove<EMAIL_ADDRESS>wrote:

@fholgado Yikes, that's no good. I have a failing test for this and it's definitely injectStripe that is stripping the props. I'll work on a fix and hope to have it released before too long.
In the interim, I hope you found a solution by juggling the way props get passed down in your application.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Hey @fholgado could you share some of the code or a minimal reproduction? (Privately by email is fine if it's sensitive.) My supposed reproduction was actually just a poorly-written test. It appears that this currently works fine:
const Child = (props) => <div>{props.text}</div>;
const InjectedChild = injectStripe(Child);
const Parent = () => <InjectedChild text="Hello, world!"/>

The text props gets passed down and render by the Child. I'm curious if you're doing something different from this or what specifically you are seeing.

Ok. Let's see:
I've got an App/index.jsx file that contains my router and app wrapper. One of the routes to my checkout form looks like this:
    CheckoutFormRoute = ({match}) => {
        return (
            <StripeProvider apiKey="blah">
                <CheckoutForm checkoutState={this.state.checkoutState} />
            </StripeProvider>
        );
    }

The `CheckoutForm` component is nothing special, and uses the `InjectStripe` HOC to wrap the return:

export default injectStripe(CheckoutForm);

I wonder if that combination of `StripeProvider` and `InjectStripe` is what's causing the issue?

Ah, interesting! I had tried testing a provider with a child with a prop, and another parent passing a prop to the injected child. But this bug appears to happen only when it is exactly this setup: the provider with the injected component as its immediate child. I've now got a failing test and will look into a fix.

@fholgado When you try to run your code in the current state, do you see an error in the console that says something like:
Please be sure the component that calls createSource or createToken is within an <Elements> component.```

I think you'll want the overall hierarchy to be StripeProvider -> Elements -> injectStripe(CheckoutForm). Injecting the stripe object without being inside an `Elements` component is going to produce some unexpected behavior.

Now, we definitely have some more work to do to make this clearer in the docs. And if your app isn't getting that console warning, then we should fix that too.

Hey Adam. You were totally right. I missed the <Element> HOC when following the getting started guide. I didn't notice anything too weird in that scenario other than the props missing, and there was nothing in the console that was throwing a warning. Thanks so much for your help, and sorry for it being a dumb mistake on my part.

Heey @asolove-stripe , I am still having this issue. I followed the comments and the solution that has been mentioned with zero success...
Long story short I tried the way @fholgado mentioned
 <StripeProvider apiKey='....'>
   <Elements>
      <StripePayment />
   </Elements>
</StripeProvider>

I checked of there were any props in the , but the prop object is empty. Thus, the strip() etc. are not working.
Also, I was wondering how  I am suppose to send the api request on submission? I've been going through the documents quite a few times for a couple of days now. But there is no example how to make my own custom submission.
I did like to have an alternative option in case I want something a more customised, since I am using WP, WP-Woocommerce, Gatsby, Apollo. And for that reason I don't want my products to be stored in stripe with the products unique sku from stripe (seems a bit odd to have my products duplicated at two services), but only have the payment confirmation that response a success/fail.


Add IntelliJ project files

TestNG ships with Eclipse project files like .classpath. Since IntelliJ is better than Eclipse, why not include IntelliJ project files as well? I have imported TestNG as a Gradle project, since that's evidently what TestNG is doing from now on, and I've added specific IntelliJ project files to source control in accordance with JetBrains's official guidance on this subject.

:-1: I think the repository shouldn't contain ide files because they could be generated by build tools like maven or Gradle.

I think removing Eclipse files instead will be a better improvement.
The problem is Eclipse users will have to reimport project after the pull.

Agree that the IDE files shouldn't be included but I put these in before I learned that :)
I usually put IDE files in an ide/ directory and I let users decide how they want to handle their own importing.
In this case, @juherr is right that removing the Eclipse files would force users to reimport the project after mv ide/.classpath ., which I'm not super excited about.
Let's start by putting the IDEA files under ide/.

For my information, what is the value-add to have an ide folder when Gradle and maven are able to generate ide files?
And ide should be able to understand pom.xml or build.gradle too (at least with appropriate plugins).

AFAIK IDEA should work nicely without any specific files in version control, it'll import information from gradle/maven.
I'd suggest add .idea and testng.iml into .gitignore (or just mark that files as ignored inside IDEA, it will not commit them via IDEA ui)
+1 to remove all IDE specific files from repository.


Example customElements.define() usage in readme does not work

The readme shows the following usage:
import reactify from 'skatejs-react-integration';

// Create your constructor.
const MyComponent = class MyComponent extends HTMLElement {};

// Define your custom element.
const CustomElement = window.customElements.define('my-component', MyComponent);

// Reactify it!
export default reactify(CustomElement);

However, when I try this, customElement.define() returns undefined. Even the unit tests don't do this, using document.registerElement() instead. I've found no online documentation or examples that indicate customElement.define() has a return value. Are you sure this example is correct?
What exactly is should I pass to reactify()?

This appears to work:
import reactify from 'skatejs-react-integration';

// Create your constructor.
const MyComponent = class MyComponent extends HTMLElement {};

// Define your custom element.
const CustomElement = window.customElements.define('my-component', MyComponent);

// Reactify it!
export default reactify(window.customElements.get('my-component'))

I'll update the docs. The right example would be:
import reactify from 'skatejs-react-integration';

// Create your constructor.
class MyComponent extends HTMLElement {};

// Define your custom element.
window.customElements.define('my-component', MyComponent);

// Reactify it!
export default reactify(MyComponent);

Closing now. If this is still an issue, feel free to reopne.


simplified: syntax

This pull request replaces <App></App> with <App /> in the main.tsx file.
The <App></App> syntax is redundant and can be simplified to <App />. This change improves code readability and consistency.

This pull request replaces  with  in the main.tsx file.
The  syntax is redundant and can be simplified to . This change improves code readability and consistency.

Thanks a lot. We will apply a eslint rule self-closing-comp and fix this issue in the whole project.


request help: add auth for etcd

https://github.com/api7/lua-resty-etcd/commit/7f95abff051308a75d251a98c4a50f8b095c0175 support etcd auth, do we need upgrade to etcd v3 first? @nic-chen

support etcd v3 is a big event. I hope we can start this step asap.

I can start working on this. But considering that V2-> V3 migration requires a data migration as well we might want to maintain these changes in a seperate branch. Any thoughts?

We are using etcd v2 now.
If you want to do this job, you can only support auth for etcd v2.
support etcd v3, we discuss it here: https://github.com/apache/incubator-apisix/issues/1767

I think the hardest part of this work should be how to add test cases.
test::nginx is not very suitable for this case. @shenal

Ok thanks for the heads up


MWAA::Environment fails to create: "#/SourceBucketArn: failed validation constraint for keyword [pattern]"

Hi,
I'm trying to create an MWAA Environment resource via. CloudFormation.   When attempting to create the stack I receive an error:

Error: ROLLBACK_COMPLETE: ["The following resource(s) failed to create: [AirflowMWAA]. Rollback requested by user." "Properties validation failed for resource AirflowMWAA with message:\n#/SourceBucketArn: failed validation constraint for keyword [pattern]"]

Here's a snippet of my CF stack:
Description: airflow-head
Resources:
  AirflowMWAA:
    Type: AWS::MWAA::Environment
    Properties:
      ...
      SourceBucketArn: arn:aws:s3:::my-bucket-name
      ...
    DeletionPolicy: Delete

The bucket ARN is valid and is accepted if I create the env via. the MWAA wizard in the AWS console.
How can I fix this?   The error is too terse to be useful.
Thanks,
Nigel.

Those messages are definitely currently difficult to interpret without looking at the resource schema itself:
      "minLength": 1,
      "maxLength": 1224,
      "pattern": "^arn:aws(-[a-z]+)?:s3:::airflow-[a-z0-9.\\-]+$"

but it looks like the bucket name needs to start with airflow

Those messages are definitely currently difficult to interpret without looking at the resource schema itself:
      "minLength": 1,
      "maxLength": 1224,
      "pattern": "^arn:aws(-[a-z]+)?:s3:::airflow-[a-z0-9.\\-]+$"

but it looks like the bucket name needs to start with airflow

@PatMyron thank you!   I'll give that a try.

@PatMyron thank you!   I'll give that a try.


BUG: python exits if no df is empty

I've ran into an issue where df_to_azure makes the python code exit when the dataframe is empty.
This makes the python script exit, which is undesired. A better solution in my opinion would be to simply return None. i have written a test that shows the problem (and fails). In this suggested solution the validation for checking if the dataframe is empty is moved up.

Looks good.