[1.12-rc2] docker should not allow start and restart on swarm-managed containers

test case 1 (no ports published)
docker service create --name foobar --replicas 5  nginx:alpine

# (wait tasks to be up)

docker stop $(docker ps --filter label=com.docker.swarm.service.name=foobar -q)

# (wait for swarm to recover)

docker start $(docker ps --filter label=com.docker.swarm.service.name=foobar -aq)

After this, there's 10 containers running, docker doesn't kill/remove the excessive containers
I'd expect docker to kill/remove those containers
test case 2
docker service create --name foobar --replicas 5 -p 8008:80  nginx:alpine

# (wait tasks to be up)

docker stop $(docker ps --filter label=com.docker.swarm.service.name=foobar -q)

# (wait for swarm to recover - note in my case it didn't fully recover, but hung at 3/5)

docker start $(docker ps --filter label=com.docker.swarm.service.name=foobar -aq)

results in:
Error response from daemon: Address already in use
Error response from daemon: Address already in use
Error response from daemon: Address already in use
Error response from daemon: Address already in use
Error response from daemon: Address already in use
Error response from daemon: Address already in use
0467a6877366
2ab26cbf3e20
Error response from daemon: Address already in use
fd07976c407f
Error response from daemon: Address already in use
Error response from daemon: Address already in use
79d552f268aa
Error response from daemon: Address already in use
0d659b1bafa1
Error: failed to start containers: 7f4936520a2a, 26a3b00257e1, 616064cf7938, 3c3fcb883565, 9e15f45321e6, b2fcb6182fe7, 8583b7c2dabf, f887d00e4811, 72e0596b161a, 249800a970a0

Tested on Docker for Mac, but similar results on other platforms
Client:
 Version:      1.12.0-rc2
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   906eacd
 Built:        Fri Jun 17 20:35:33 2016
 OS/Arch:      darwin/amd64
 Experimental: true

Server:
 Version:      1.12.0-rc2
 API version:  1.24
 Go version:   go1.6.2
 Git commit:   a7119de
 Built:        Fri Jun 17 22:09:20 2016
 OS/Arch:      linux/amd64
 Experimental: true

#dibs

After this, there's 10 containers running, docker doesn't kill/remove the excessive containers

I'm not sure what you mean by "doesn't kill" as you stop the containers yourself. The command you run includes -a so I guess you only mean remove.
It may not be a good idea to remove these containers, at least as long as they continue to appear in docker service tasks -a. They may be crucial to debugging the issues that caused them to crash. They are cleaned up though when you do service rm. cc @stevvooe
The other part of the report Address already in use seems to be a networking issue. This error appears on container create, I assume because the port is already used. Maybe makes sense to track this in a different issue. cc @mrjana

Could network partition cause the same problem?

ping @aaronlehmann

@mgoelzer: I'm not sure I understand the question correctly. A network partition could lead to a situation where more containers than the desired number are running across all nodes, but this is by design because the alternative is to have situations where no instances of the container are running because the two sides of the network partition assume that the containers are being run on the other side.
The issue here seems to be more about how the agent logic should handle manually stopping and starting containers associated with a service.

@thaJeztah @tonistiigi Address already in use is expected because when you stopped the containers which were exposing ports swarm recreated those containers with probably the same IPs so when you restarted the same container it is going to complain about IP address conflicts. It is actually good that it is failing to start the container because it detected an IP address conflict rather than starting them but using an IP of another container. That will be worse.
At the end of the day I think we should have a generic way to deal with the users trying to manage the containers OOB. May be we should not allow stop/starts to happen for managed containers. If the container crashes they can be kept around for debug purposes. But IMO we should block users from stop/starting managed containers

@mrjana I thought I got this error already on stop when it was doing the replacing but couldn't reproduce anymore. Maybe it was fixed recently.

May be we should not allow stop/starts to happen for managed containers

This seems correct thing to do.

A start would be to allow filtering on "managed" / "unmanaged" containers

On 28 Jun 2016, at 17:45, Jana Radhakrishnan<EMAIL_ADDRESS>wrote:
@thaJeztah @tonistiigi Address already in use is expected because when you stopped the containers which were exposing ports swarm recreated those containers with probably the same IPs so when you restarted the same container it is going to complain about IP address conflicts. It is actually good that it is failing to start the container because it detected an IP address conflict rather than starting them but using an IP of another container. That will be worse.
At the end of the day I think we should have a generic way to deal with the users trying to manage the containers OOB. May be we should not allow stop/starts to happen for managed containers. If the container crashes they can be kept around for debug purposes. But IMO we should block users from stop/starting managed containers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

May be we should not allow stop/starts to happen for managed containers

I don't think blanket changing fundamental docker behaviour (start/stop) is the answer.
Users should 100% be able to stop any arbitrary container that they want to. As long as 'docker exec' works on managed containers, there is the possibility to need to 'docker stop' a particular managed container. Just like with a crashed container, you may want to stop and investigate, not just remove it.

Containers are managed by a task that must maintain an accurate state of the container. Tasks can only proceed in one direction, towards failure or success. As tasks proceed through their lifecycle, they report state that monotonically increases. If you startup containers that are managed as tasks, this guarantee is broken and the task will no longer manage container.
The lifecycle of containers managed by a service is set by the manager (via the dispatcher). The manager passes down the list of assigned tasks to a worker and the worker will hold on to those as long as they are there. This allows the manager to have some control over the lifecycle of the containers in the cluster to support investigation after a crash.
For the most, its okay to stop containers. They will be restarted by the orchestrator. If a container is restarted by another mechanism, the task controller won't try to shut it down again. This is a property of the system to handle cases like this. The issue here is that they won't be shutdown automatically, since the user has effectively taken control of them. If we didn't do this, we would shut it down immediately after you started it.
The current behavior is that these will go away when the service is deleted. For a long running service, this may be problematic but they can be pruned from the assignment set to keep it manageable.
TL; DR If you take over the management of an individual container, swarm mode will no longer manage its lifecycle. It is up to the user to delete the extra container. In general, don't start containers managed by a service but there isn't a good reason to prevent it.
Also, there is a networking problem preventing the startup of the new containers, which I suspect is the issue with 3/5 replicas.

@stevvooe I don't think we can allow containers to get decoupled from the manager lifecycle. Especially for networking reasons. Because they hold resources allocated by the manager. This is what manager-centric resource allocation results in. So either that container needs to get managed all the time even if they get manually restarted using an OOB mechanism or we should disallow those OOB mechanisms to interfere with swarm container lifecycle management.

@mrjana I agree that it should discouraged, but I am not sure if we should prevent it. Can we not cover this with documentation?
Ultimately, these containers will still be cleaned up when the assignments are removed. I've also been toying with the idea of a task reaper to really make sure these things are destroyed.

@stevvooe @mrjana what should we do for 1.12? Should we add a filter to easily identify / filter managed/unmanaged containers and tell people "don't touch those managed ones", and open an issue for a "task reaper" in 1.13?

@thaJeztah i changed the title on this issue to make it clear.

Hey, are there any decisions regarding this? Or some discussion around it somewhere?
Thx!


test: Add Negative unit tests

Description of your changes
Add negative unit tests to TestReconcilerUpdateBindings function.

Fixes #
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

There are more negative UTs to be added, there are three branches where,

The first branch with case fleetv1beta1.BindingStateBound has an Update you can mock to return error
The second branch with case fleetv1beta1.BindingStateScheduled has another update you can mock to return error
The third branch with case fleetv1beta1.BindingStateUnscheduled has a Delete which can possibly return a not found or some other error as well


Tweak unverifiable signature

Rather than stating that a signature is invalid, it's more correct to state it could not be verified. It may be a perfectly valid signature, but not meant for this application/issuer.
Please note that this includes PR #33.

:+1:


Google My Business API Support

The API library is here: https://developers.google.com/my-business/samples/
I am still having to autoload this separately from the rest of the API services, is this a planned integration?
Thanks

@jsiebach im also having this trouble, but I have not been able to autoload it properly, can you tell me how you did it?

I am using a Laravel app and created a subfolder outside of my autoload directory which I added to my composer.json:
"autoload": {
        "classmap": [
            "database",
            "resources/unnamespaced/MyBusiness.php"
        ],
        "psr-4": {
            "App\\": "app/"
        }
    },

@jsiebach thanks, that worked

To answer your original question, that plan is up to the My Business API team. As soon they publish to the root discovery document that library will be automatically available here.

I don't want to create noise with a new issue as I'm currently not _yet affected, but currently that library is defined as
class Google_Service_MyBusiness extends Google_Service

In 2.*, Google_Service was replaced with an alias to Google\Service which will be removed eventually.
Since it seems they're still not yet added to the root discovery, I hope someone has it on their agenda to provide compatibility here? 🤔


Allow finer control over UnifiedPush in the android app

Right now, if an app wants to use UnifiedPush, it can simply register itself. This causes ntfy to use instant delivery, which makes sense given the usecase of UnifiedPush, but is a bit frustrating if you only wanted to use ntfy for other pushes and don't want the battery drain of instant delivery.
Ideas for possible controls:

An option to disable UnifiedPush entirely
An option to block a specific app from enabling UnifiedPush
An option to disable instant delivery for a UnifiedPush topic

I don't know how the project feels about any of these, but I imagine that #1 is perhaps the "easiest" (said without much context).

(I am assuming that you are talking about the Android app)
I used to have a toggle in the Settings that allowed disabling UnifiedPush entirely, but I removed that because I didn't want toggle-hell. I haven't heard this argument before, to be honest. Usually you can choose the notification method in the app itself and unselect "ntfy" there. What app is misbehaving and registering itself without your consent?

/cc @S1m @karmanyaahm

Yeah, I am.
The offending app is Tusky. As far as I can tell there's no option for it. See this issue https://github.com/tuskyapp/Tusky/issues/2692

I'd be ok with an off switch for UP if somebody wants to implement it (cough, @wunter8, maybe?)

@wunter8 Did this already.


Pass meta/setup script to binder

Pass meta/setup script to binder to be able to load variables and stuff before the student evaluates the code.

Don't know how this can be solved, maybe put it on the list for the fiverr guy?

Can be somehow done by setting a negative top_margin


CORTX-28899: Ability to create multiple LoadBalancer IO services

The solution.yaml and platform Helm chart have been updated to support the ability to create multiple CORTX IO Services. This service provides access to the RGW servers for S3 IO. By default, only one service is created. Multiple services can be created for redundancy in a cloud environment.

Multiple service instances are only supported for the LoadBalancer service type. Defining multiple ClusterIP or NodePort services doesn't provide any benefit, so those are fixed to a single instance.
If custom nodePorts are configured, the first instance of the service will take the specified nodePort value, and the port number will be incremented for each additional service instance. The port numbers must not conflict else the deployment will fail.
The service names are assigned based on the Chart value and a sequential incrementing number, e.g. cortx-io-svc-0, cortx-io-svc-1, ..., cortx-io-svc-N.

Possible breaking changes

Solution configuration field solution.common.external_services.s3.count is required. Set to 1 for previous behavior.
If you previously had a dependency on the service name, then you will need to update to use the generated number-based names instead, e.g. cortx-io-svc-0 instead of cortx-io-svc.

TODO
The configmap (config.yaml) makes use of the service name. For now this is fixed to the singular cortx-io-svc-0. We need to understand what this value is being used for. For example, the CSM agent endpoint uses this name, which doesn't make much sense.
Signed-off-by: Keith Pine<EMAIL_ADDRESS>

There are several Codacy warnings worth fixing.
Have you considered using a standard test framework, like pytest, instead of rolling your own?


[3] As a front-end developer, I should be able to do an API call to add/remove a friend.

Will be defined as the following to use:
add_friend(user_id: str)
remove_friend(user_id: str)

Tests TODO (Pending database implementation in next sprint)

TODO: Check that the destination user has the pending friend request in the list (pending database).
TODO: Make sure the user does not have that friend in its profile (pending database).
TODO: Make sure the user add request is no longer in the user's profile (pending database).
TODO: Make sure the added user is in both of the user's profile (pending database).
TODO: Make sure the added user is not in the pending friend request list (pending database).
TODO: Make sure both of the profiles no longer have each other under friends.

TODO
Replace MOCK database calls with the real ones next sprint.


fix: don't include port in the url when 443 or 80

When browsing to a host that has 80 or 443 as port number we are getting http://url80

sorry my bad


output_partial_domain for post-processing

Description
Adds an option to output only part of the domain during post-processing.
Type of change

[x] New feature (non-breaking change which adds functionality)

Scope

[x] This PR comprises a set of related changes with a common goal

How Has This Been Tested?

[x] Works as expected when post-processed using Matlab binary reader.
[x] Passes the original tests with post-processing enabled (with -a)

Checklist

[x] I have added comments for the new code
[x] I added Doxygen docstrings to the new code
[x] I have made corresponding changes to the documentation (docs/)
[ ] I have added regression tests to the test suite so that people can verify in the future that the feature is behaving as expected
[ ] I have added example cases in examples/ that demonstrate my new feature performing as expected.
They run to completion and demonstrate "interesting physics"
[x] I ran ./mfc.sh format before committing my code
[x] New and existing tests pass locally with my changes, including with GPU capability enabled (both NVIDIA hardware with NVHPC compilers and AMD hardware with CRAY compilers) and disabled
[x] This PR does not introduce any repeated code (it follows the DRY principle)
[x] I cannot think of a way to condense this code and reduce any introduced additional line count

@ChrisZYJ thanks this appears very useful! I'm curious if this works for a couple different scenarios (have you verified):

Serial vs serial output
Format == 1 and 2
Stretched grids

It works with stretched grids and both serial and parallel outputs. However, it only works for format == 2 (binary) and not format == 1 (silo). The silo format requires more careful data handling, so perhaps that could be completed in another PR.
I've documented the scenarios where it doesn't work yet in the documentation and added prohibitions in the checkers.

@ChrisZYJ thanks this makes sense. May I ask how you are handling your binary output files? Matlab/Python or into Visit/Paraview? I think this would be useful to either document or perhaps even put a helper script in the misc/ directory for others.

I've been doing it in Matlab using the f_binary_reader.m function, which is already in the /misc directory. I could document it and add my wrapper function to /misc later if needed.

Thanks @ChrisZYJ. I find using the binary files rather annoying. If we had a more broad way of doing it (and could suggest to the user when and how to do it) that would be useful.
I'm glad to merge the PR but we have two big PRs that are very near merging that I don't want to disrupt. Hopefully can merge this in a couple days.

Yeah, I also find binary files quite cumbersome to work with. While I'm not sure what a broader way to handle them would be, I can add some documentation and helper scripts in another PR if you find those helpful.
I'm in no hurry to merge. This PR should be quite independent from everything else, and I could easily merge it into them later if needed.

Yeah, I also find binary files quite cumbersome to work with. While I'm not sure what a broader way to handle them would be, I can add some documentation and helper scripts in another PR if you find those helpful.
I'm in no hurry to merge. This PR should be quite independent from everything else, and I could easily merge it into them later if needed.

That would be very useful indeed. I hope to merge this soon but don't want to risk messing up these larger long-standing PRs right now.


Public Intersect Town Hall Planning Workgroup session 2 - July 11th 2024

Intersect Town Hall Workgroup Meeting Overview

When: 11.07.2024 - 16:00 UTC
Duration: ~90 min
Zoom: https://bit.ly/3W4tb8q
Proposal: https://bit.ly/3Rct33S
Miro Board: https://bit.ly/4c4Htv0
GitHub Problem Board - https://github.com/orgs/Catalyst-Swarm/projects/2
Context: Workgroup helps to Co-Create and Operate Intersect Town Halls
Parts of Sessions are recorded and uploaded to the Swarm YouTube channel.
Miro Board is used for activities like meeting notes and information collection.

Suggested Workshop Agendas

5min - Welcome and Agenda
5min - (Optional) Introductions, Onboarding and Open Questions
20min - Review and Update Community Town Hall Problem Board
20min - Update Town Hall Agenda
15min - Coordinate After Town Hall Topics
10min - Decide on Intersect Town Hall date and time
10min - Open Topics
5min - Session Overview (to be recorded)

Previous planning meeting

https://github.com/Catalyst-Swarm/Intersect-Town-Hall/issues/40

This is the task issue for “Public Intersect Town Hall Planning Workgroup session 2“ which was held on 11th July 2024 at 1600 UTC.
Last TH Slide deck:
https://docs.google.com/presentation/d/1c-UQtW_Jdm5TPjgVScJFzQdEnRrbQn7TAucGm9LzBOw/edit#slide=id.g2e81879d9ac_0_21
After Town Hall sign up form

https://forms.gle/jhomzjzXx9GQUWnx5

This task is intended to
Proposal context
Deliverable

https://catalyst-swarm.gitbook.io/intersect-town-hall-playbook/the-proposal/intersect-town-hall-by-swarm

Acceptance Criteria

. [not yet decided]

Estimated Duration

Based on last month results it took about 5 hours to prepare
This month it took 4 hours for this one and prolly for the next one
The short-term goal is to get it down to 1-2 hours per person
Meeting Summary usually takes under 2 hours, but this one was heavy one and don't know yet the after affects of using Git Hub as meeting summary drafting
and the final component is facilitation time we do about 90min sessions

Cost

[not yet decided]

Meeting notes:
https://miro.com/app/board/uXjVKG5h5BA=/?moveToWidget=3458764593265587959&cot=10
Meeting Summary
People Present:
after meeting will be updated

Suggested agenda for 23rd July:
https://docs.google.com/document/d/1h2H6VJ9eAsZNsZE4pBnAePx-NeSDNOOnEn-fz_7M-mI/edit#heading=h.t8wrmopmsksn


chore: Begin mocking out some util unit tests

Initial work stubbing out unit tests
Resolve #890

@spmcbride1201 LGTM


Core: normal pauses stdDev is counter-intuitive

Current value is a double, but it's actually interpreted as a millisecond value.
We should provide 2 versions: one that takes a duration, and one that takes a percentage of the value.

Introduced normalPausesWithPercentageDuration and normalPausesWithStdDevDuration.


Pre-compilation instructions need to be removed.

https://github.com/KhronosGroup/OpenXR-Tutorials/blob/main/tutorial/1-introduction.rst?plain=1#L258
still talks about pre-compiling the SDK, that should no longer be needed.

This has been resolved by commit b7d4b230.


IAP connector + demo

@drebes wrote the module, I just cleaned up the README and pushed it to the repo.

Wow! Let's add Roberto as reviewer, I'll also start reading through this today. :)

Yes, I discussed this with Andras, I can review this.

@lgandras can you make sure all files have the standard copyright boilerplate?

What is the rationale for copying the content from https://github.com/GoogleCloudPlatform/iap-connector/tree/master/terraform-and-helm into this repo? It seems like a major addition of unnecessary maintenance burden.

Hi Morgante, thanks for chiming in. It is indeed, we were just discussing
this today. I'm removing it and referencing the other one.
Morgante Pell<EMAIL_ADDRESS>schrieb am Mo., 21. Sept. 2020,
20:09:

What is the rationale for copying the content from
https://github.com/GoogleCloudPlatform/iap-connector/tree/master/terraform-and-helm
into this repo? It seems like a major addition of unnecessary maintenance
burden.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terraform-google-modules/cloud-foundation-fabric/pull/136#issuecomment-696280748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFWJNPYHUHUSUVFMJLWNY3SG6JGZANCNFSM4RM2B2LA
.

Rationale is always for fabric to give a consistent checkout with no
external deps.
I'm also a bit skeptical about the latest round of changes in the cft
modules we contributed, who could simply refer to our versions, who went
through several rounds of improvements and fixes.
Let's accept once and for all we have different goals...
On Mon, Sep 21, 2020, 20:09 Morgante Pell<EMAIL_ADDRESS>wrote:

What is the rationale for copying the content from
https://github.com/GoogleCloudPlatform/iap-connector/tree/master/terraform-and-helm
into this repo? It seems like a major addition of unnecessary maintenance
burden.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/terraform-google-modules/cloud-foundation-fabric/pull/136#issuecomment-696280748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQDVYXRYC67BYGVWKOELDSG6JGZANCNFSM4RM2B2LA
.

@ludoo That's not even a CFT module, it's the official Helm chart published by the IAP connector team. If CFT Fabric is maintaining a copy of all source code, including Helm charts, where do you draw the line? Are you also doing custom machine image builds?

that will teach me to answer PR comments from my phone :) thanks for
pointing that out!
On Mon, Sep 21, 2020 at 8:17 PM Morgante Pell<EMAIL_ADDRESS>wrote:

@ludoo https://github.com/ludoo That's not even a CFT module, it's the
official Helm chart published by the IAP connector team. If CFT Fabric is
maintaining a copy of all source code, including Helm charts, where do you
draw the line? Are you also doing custom machine image builds?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/terraform-google-modules/cloud-foundation-fabric/pull/136#issuecomment-696284646,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQDV6UHE6OZBXITPAMMDTSG6KCVANCNFSM4RM2B2LA
.

--

•  *Ludovico Magnocavallo
•  *Strategic Cloud Engineer
•  *Google Italy
•<EMAIL_ADDRESS>•  +39<PHONE_NUMBER>

This email may be confidential and privileged. If you received this
communication by mistake, please don't forward it to anyone else, please
erase all copies and attachments, and please let me know that it has gone
to the wrong person.
The above terms reflect a potential business arrangement, are provided
solely as a basis for further discussion, and are not intended to be and do
not constitute a legally binding obligation. No legally binding obligations
will be created, implied, or inferred until an agreement in final form is
executed in writing by all parties involved.

Note that it is a fork of the iap-connector module, so if we want to use that it's better to contribute the changes to a (fork?) of the original repo.

@ludoo / @drebes: Sorry for the hicup in the latency ;) I already implemented the requested changes. I have it up and running in case anybody wants to test it out. The only question that remains is what do we do with the fork of the IAP connector. I see three options:

Keep a parallel branch in the original repo
Adapt this repo to use the original iap-connector (if possible)
Branch out, i.e. keep a clone of the IAP connector in this repo

I like the second one the most, but I don't know why @drebes took the decision of forking in the first place. If there are good reasons for it, then only options 1 and 3 remain as possible.

Wow thanks a lot for driving this. I'm also for option 2, but will defer to you and Roberto on what's possible and what to choose, of course.

@lgandras can we close this, or do you still plan on merging?

This is obsolete. Thanks for the effort!


Resubmissions Required for Guam

The following files need to be resubmitted to Guam following the SSHJ connection issue. These files do not require any corrections. They just need to be resubmitted.
2a28dd54-d8f8-4fe2-909b-814261a3c0bf	2022-06-02T21:16:34.691459-04:00
2ca6c64e-fe41-4ffa-b8c5-6534f0a3e3fd	2022-06-02T23:16:25.584839-04:00
17aba261-3bbb-4d0a-91e6-ff14eec26df9	2022-06-03T01:17:17.744109-04:00
e42fc506-5592-4a06-8d56-8d91ced3d1f1	2022-06-03T03:17:05.996528-04:00
0ebda417-6ca6-458b-84e5-e43bd2891738	2022-06-03T05:17:33.249271-04:00
de9b586f-31f2-42ff-8aa5-aeb30d02a0d9	2022-06-03T07:17:52.892512-04:00
4c0dcf53-aa8d-4b8c-a6c2-73a798e0fd4a	2022-06-03T09:15:46.065762-04:00

Starting resubmits.

Resubmits are done. @anshulkumar-usds @sarahnw @Adrian-Brewster


Support custom config file

It's great that you've updated/built upon GregYankovoy's image but it does not look like the goaccess.conf file is user-overwritable which means anyone NOT using NPM is unable to use this.
I would suggest:
Move geoip db
COPY the db into /var so its unlikely to be overwritten by mounting a volume (and update the default config to reflect the location change)
Copy default config on container start instead of in build
Instead of COPY the NPM config into /goaccess-config during build, COPY it somewhere else as a backup and then during startup check for the file and if it does not exist then copy it to the required location.
This will make it so:

users can mount a host directory to /goaccess-config that contains a custom config
if no config exists the default one is created in the host's directory which allows them to edit it themselves

Since the config in the current image cannot be modified by the user you should be able to make these changes without any compatibility issues for current users.

I could fork and do it myself...was just hoping to keep efforts consolidated, especially for the unraid community apps (not wanting to add another goaccess template)

@FoxxMD let me take a look and see what I can do.

did you want the geoip db mountable as well?

I'm actually doing a bunch of changes to the code to support another instance of goaccess running in the container so let me do it. The other instance will parse the redirection logs from npm.

@FoxxMD a custom config is now supported in the develop branch and image.
docker pull xavierh/goaccess-for-nginxproxymanager:develop
Feel free to test. Please review the readme file for more information. You'll need to set an environment flag LOG_TYPE of custom and pass in a new volume that will contain the config file.

Thanks for the hard work!
I am not able to get it read my config file...
GOAN v1.1.1

NGINX SETUP...

NGINX BASIC AUTH
-------------------------------
None

CUSTOM
-------------------------------
goaccess.conf file found
goaccess.conf not readable

I've verified it should be at the very least readable by anyone

I've tried to use PUID and PGID correctly (setting to nobody/users) as well as trying with root with no change. I have DEBUG=True but do not get any other output.
I've also tried using the default conf from your repo but had no success (same output).
How can I tell if this is a permission issue or a problem with my config file (line endings or something?)

@FoxxMD it doesn't sound like a line-ending issue. It found the file. The bash script is checking if the file has read permissions.
if [[ -r ${goan_config} ]]; then
echo -e "goaccess.conf readable"
else
echo -e "goaccess.conf not readable"
exit
fi

@FoxxMD  I just updated the develop image and code. Please pull the latest develop image.
docker pull xavierh/goaccess-for-nginxproxymanager:develop

You should see something like this.
CUSTOM
goaccess.conf file found
BEGIN DEBUG*********
/opt/custom/goaccess.conf file permissions
644 root root
/opt/custom/goaccess.conf content
######################################
Time Format Options (required)
######################################
#.....

@xavier-hernandez thanks for the quick and changes!
When using the correct user/group for my filesystem (nobody=99, users=100) PUID=99 PGID=100:
GOAN v1.1.2
NGINX SETUP...
NGINX BASIC AUTH
-------------------------------
None
CUSTOM
-------------------------------
goaccess.conf file found
**************BEGIN DEBUG***********************
/opt/custom/goaccess.conf file permissions
777 users UNKNOWN
/opt/custom/goaccess.conf content
######################################
# Time Format Options (required)
######################################
#
....

When using PUID=0 PGID=0 (root/root) or I do not set them at all:
GOAN v1.1.2
NGINX SETUP...
NGINX BASIC AUTH
-------------------------------
None
CUSTOM
-------------------------------
goaccess.conf file found
**************BEGIN DEBUG***********************
/opt/custom/goaccess.conf file permissions
777 users UNKNOWN
/opt/custom/goaccess.conf content
######################################
# Time Format Options (required)
######################################
#
...

Are the file contents still listed?

Yes they are.
I may be missing it but I also don't see anywhere where you are using PUID/PGID in the dockerfile or startup scripts. Changing those is not something built into the alpine base image.
You could easily migrate to the linuxserver.io base image which would take care of that for you, though. I use it for the base of one of my projects to handle puid/pgid.

@FoxxMD - this "777 users UNKNOWN" is the permission for the file not for the PUID or PGUID. You are correct in that I'm setting the PUID and PGUID correctly but since I'm not writing anything to a mount this hasn't come up as an issue. Maybe the problem here is that bash script is running as a different user within docker. I'll look into it.
But for now if I remove the exit command from the container it should work for you since it is "readable".

Getting much closer now with the new commit, thanks!
...(config omitted for brevity)
**************END DEBUG***********************
goaccess.conf not readable
DEBUG
-------------------------------
ON
RUN CUSTOM GOACCESS
Daemonized GoAccess: 24

Web server is outputting Custom instance processing: (might take some time...) and nothing else. Been running for about 5 minutes now. CPU is idle. Nothing else in the docker logs.

@FoxxMD anything show up? How many logs file did you add to the config?

I went back and used the default config from the repo with only a few changes for my log format. The web server is still outputting custom instance processing... but the logs now have:
goaccess.conf not readable

DEBUG
-------------------------------
OFF

RUN CUSTOM GOACCESS
<!DOCTYPE html><html lang='en'><head><meta charset='UTF-8'><meta name='referrer' content='no-referrer'><meta http-equiv='X-UA-Compatible' content='IE=edge'><meta name='google' content='notranslate'><meta name='viewport' content='width=device-width, initial-scale=1'><meta name='robots' content='noindex, nofollow'><link rel='icon' href='data:image/x-.......

It looks like goaccess report is getting output to buffer?

@FoxxMD would it be possible for you to share your goaccess.conf file?


list_expenses and list_deposits - date inclusivity

the list_expenses and list_deposits dates are not inclusive. this adds extra work for the user to actually know when something was inputted in as an expense / deposit. consider making it inclusive?

Duplicate


How to disable the update notification

What feature or improvement do you think would benefit Files?
When i open the app, an update notification appears, can I turn it off?
Files Version
<IP_ADDRESS>
Windows Version
Windows 11 (22H2): 22621
Comments
No response

There isn't a way to do so as its intended for people to be on the latest version.

@shead0shead is there a reason why you're still using v2.3.12?

It's more convenient for me and in the version I use there are fewer functions that I don't need.

This version is no longer supported but we'd like to hear more about your use case  to see if we can improve the latest version.


[FEA]  Audit ShuffledHashJoinExec

Is your feature request related to a problem? Please describe.

Ensure Spark and Rapids plugin version of the exec match functionality.

Verify Config specific to the operator match.

Verify API is consistent and fully translated.

Port relevant tests.

Ensure Spark and Rapids plugin version of the exec match functionality.

Yes, functionality matches

Verify Config specific to the operator match.

Verified

Verify API is consistent and fully translated.

Translation required. Expression-> GpuExpression. Created PR https://github.com/NVIDIA/spark-rapids/pull/351

Port relevant tests.

No new tests needs to be ported. All cases are covered.


Move compute storage

Move compute storage commands to storage service
The subcommands from the compute storage task has been moved
to the storage service. The following new commands exists

storage container
list, show, create, delete, sas containers

storage share
list, create, delete file shares

storage disk
upload, delete, blob data, preferably vhd disks

References:
https://trello.com/c/vQ8nMkW3/138-azurectl-move-compute-storage

This is mostly moving code around and changing name spaces. I'm afraid file movement and renaming is always hard to review in patch like results.

LGTM


Question: Bicep - Is there a plan or approach to support US Gov Cloud and Microsoft China

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
Given the frequent variations in API versions across different cloud environments within Azure, I was wondering if there have been discussions about incorporating separate modules that are deployable in each cloud environment.

@oZakari, I believe this topic has come up a couple of times since we started this program. The short answer is that we don't have any near-term plans for adding support for the above named "variants" of Azure. As this question is fairly complex to elaborate on, I'd recommend you raise this in one of our upcoming internal Office hours calls. Thanks!


New package: SPPL v0.1.0

Registering package: SPPL
Repository: https://github.com/femtomc/SPPL.jl
Created by: @femtomc
Version: v0.1.0
Commit: a8fd2e6a8566aacb90cd0ac6dab6c2d28a2ccc3c
Reviewed by: @femtomc
Reference: https://github.com/femtomc/SPPL.jl/issues/1

@DilumAluthge This is a wrapper library for sppl - despite failing the guidelines, is it okay to merge?


Implement a platform abstraction layer

This is a general tracking/discussion issue for the idea of adding an abstraction layer over the top of some (all?) of the backend libraries we use. This would (hypothetically):

...make it easier to switch out libraries later.
...allow users to disable bits of the context they're not using (e.g. if you don't have gamepad support, why waste processing time handling those events?).
...open up a vague pathway to getting Tetra running on WASM.

This probably won't be a publicly exposed API to begin with, as I don't trust myself to get it right first time - but if we could have e.g. all of the SDL code in one place, it'd probably clean up the codebase quite a bit.

I am for everything that brings Tetra closer to a WASM build.

Fair warning, that might be a long way off 😅 But I'm definitely looking into it.

Phase one of this work is now done! 🎉
All of the SDL code (barring a few re-exported types which will need replacing in Tetra 0.3) is now located in the platform module. I'm sure there's still lots of bits of the code that rely on the assumption that SDL will always be there, but this is a good first step towards figuring out what those bits are :)

This was merged as part of 0.2.20 :) Still need to do some work to extract out the audio backend, but I think that probably deserves a seperate issue.


Switch to com.rallyhealth organization and publish to bintray

@usufruct99 @sklei2  @rallyhealth/engineers
This will publish the artifact to bintray. I want to re-release this as 2.0.0 on bintray under com.rallyhealth

retest this please

Coverage remained the same at 64.085% when pulling f288161e1426768b3135a1ea8a0df62f8e2ec259 on move-to-rally into 370ff91259698267ac0874eb99d22ca773935bb6 on master.


Add an "ob shell" command

I happened to build this off of Dan Bornside's asset gathering strictness change, because that's what we had at hand, hopefully that's fine.

@cgibbard https://github.com/obsidiansystems/obelisk/pull/393 looks to be already merged so why is that commit still showing up here? Either way can you rebase to not include it?

@kmicklas I don't know why that was included here, either - maybe a github glitch? - but in any case I'm just going go merge, since it's passed CI.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1735 ms

Resolved: Bofrost.be performance has improved in 90121c9 after 6 minutes.


2.0.0 everywhere basically not working at all for complicated data structure

I tried to whittle my trait/case class domain model down to something minimal to show you but it basically didn't work at all when I had the model in a different file to the definition and application of the everything. I'd have to start again to show.
Is 2.1 going to have better support for this?
As a workaround, we're going to use runtime recursive Product traversal. If we can upgrade to scala-2.11 I'll try running shapeless 2.1 vs our runtime solution and let you know if anything breaks. If it does,  we should talk privately as I really can't share this domain model publicly and talking you through it might give you some ideas for a test case.

I've worked around this with runtime lookup. Maybe worth using something like this in your tests:
object ScalaVisitor {
  def everything[T <: Any](t: T)(f: Any => Unit): Unit = {
    f(t)
    t match {
      case t: AnyRef if isArray(t) =>
        t.asInstanceOf[Array[_]].foreach(everything(_)(f))
      case t: Traversable[_] =>
        t.foreach(everything(_)(f))
      case t: Product =>
        t.productIterator.foreach(everything(_)(f))
      case t =>
    }
  }

  def allTheThings(t: Any): List[Any] = {
    var seen = Queue.empty[Any]
    everything(t) { el => seen :+= el }
    seen.toList
  }

  def someOfTheThings[T](t: Any)(f: PartialFunction[Any, T]): List[T] =
    allTheThings(t).collect(f)
}

I would guess that your original example involves (possibly hidden) recursion in a way which isn't supported in 2.0.0 ... can you boil you example down to something which you can compare between that and 2.1.0?
Other than that, I'm afraid that without something reproducible there's not a lot I can do apart from saying that what you're seeing isn't completely unexpected in 2.0.0 and is most likely fixed in 2.1.0.

I tried to boil it down, but it kept failing (because the classes were defined in a different file?) and I went too far, it is quite a big domain model so it would take me a few hours to try again. 2.1 sounds great, I'll just try and get us onto scala 2.11 and then I can report back with more meaningful results. No point trying to reproduce a bug on a branch that is known to be broken. On the plus side, this will actually be a reasonably good stress test of everything. Enjoy your holiday :smile:


Java added to install page

Description
This PR adds Java to the install page:
Preview
Linux CPU
Linux GPU
Mac CPU

Restarted CI due to failure on ARM test. https://github.com/apache/incubator-mxnet/issues/13410

@mxnet-label-bot add [pr-awaiting-review]
Thanks for your contribution @aaronmarkham

@andrewfayres

git's merge of the clojure install is horrific... I'm going to have to manually deal with clojure+java... stay tuned...

I tried to add additional logic so that the Java (and Scala and Clojure) install option shows a different message for older versions, but it doesn't work: https://github.com/apache/incubator-mxnet/issues/13430


Li metal spm

Description
Add li metal (half cell) models for SPMe and SPM
Also fixes #1720
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ flake8
[x] All tests pass: $ python run-tests.py --unit
[x] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

@brosaplanella now ready to review

@tinosulzer you didn't add a line to CHANGELOG for this, but you can introduce it through #1782


[Inference] Add validated models for Gaudi

Model list:
bloom-7b1 single card without template
Falcon-7b single card without template
Falcon-40b multiple cards without template
Gemma-2b single card without template
Llama3-7b single card unknow
Llama3-70b multiple cards unknow
Mistral-7b single card without template
Mixtral-8x7B-Instruct-v0.1 single card with template
llama-2-7b single card unknow
llama-2-70b multiple cards unknow
CodeLlama single card unknow
GPT2 single card without template
GPT-J single card without template
MPT-7b single card without template
Qwen1.5-110B single card with template

All CI passed. Gentle ping @carsonwang for review, thanks!

@kira-lin is helping to review this. For qwen, can you please update to use Qwen/Qwen2-7B-Instruct?

@kira-lin is helping to review this. For qwen, can you please update to use Qwen/Qwen2-7B-Instruct?

Added Qwen1.5-7B-Chat and Qwen2-7B-Instruct.

For falcon and qwen, @KepingYan can look into this.

https://github.com/intel/llm-on-ray/blob/cdce225cb2285baf9c151fecc7b6af853412e030/llm_on_ray/inference/predictors/hpu_predictor.py#L340
Let's modify this line according to: https://github.com/huggingface/optimum-habana/blob/595cc3e4ec219b1ce469b323cf94e994c5c5d8f3/examples/text-generation/utils.py#L311-L312

Updated, thanks for comment. Btw, will it still be any place to be changed since I found some other places specially handled through model_type == llama?


Use cfx_getLogs to accelerate catch-up sync

Batch epochs for synchronization per worker to improve efficiency
Use cfx_getLogs to retrieve event logs, then assemble minimal epoch data for persistence

This change is

This PR is closed and merged into #256


Update _config.yml

Closes #
📑 Description

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[ ] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[ ] No

ℹ Additional Information

Summary by Sourcery
Build:

Change the Jekyll theme from 'dinky' to 'cayman' in the _config.yml file.

Summary by CodeRabbit

New Features

Updated the website theme for a refreshed look and feel, enhancing user experience with the new Cayman theme.

@gstraccini create labels


Add UA Dp DSAU University

@bloodrain777 First stage of verification passed, official domain for Дніпровський державний аграрно-економічний університет is student.dsau.dp.ua as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@bloodrain777 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


remove material-ui-search-bar dependency

If it's possible, let's implement it by ourselves, to avoid having an extra lib.
Also, wrap and move it to safe-react-components.

This component is used only in the sidebar:

It's basically just an input with a magnifying glass icon. It should be trivial to replace.

@katspaugh we might (95%) introduce search field in the safe apps tab


Aggregate PR

Aggregate of the following PRs:
https://github.com/wildfly/wildfly/pull/14927
https://github.com/wildfly/wildfly/pull/14922
https://github.com/wildfly/wildfly/pull/14920

All PRs merged.


PHPStorm-Debugging

Description:
Broken link address
Expected Behavior:
I`m reading the documentation and this link to Remote debug is broken
Context information:
https://laradock.io/guides/#PHPStorm-Debugging

Yes, in the documentation https://laradock.io/documentation/#phpstorm-debugging-guide the link to the PHPstorm guide is not working.


Jodit and Tributejs

Jodit Version:  3.1.71
Jodit angular: 1.0.29
Tributejs: 3.2.0
Code
Attaching zurb/tribute  to Jodit works fine, here's the essential:
this.tribute.attach(document.getElementsByClassName('jodit_wysiwyg));

Everything works, except selecting item with keypress enter, which Jodit captures.
Expected behavior:
Should be able to configure Jodit to disable capturing keypress when Tribute (or other) is open.
See this Quill with Tribute comment
Actual behavior:
Jodit captures keypress enter and inserts the configured enter type

Hi try this https://github.com/jodit/jodit-tributejs

Thanks @xdan!
I've also been looking at the events listing and is it possible to use the configuration option events to accomplish the same?
Something along:
config = {
events: (event) => {
   if (event === 'beforeEnter') {
      ...
   }
}

Maybe you could give an example of using the events option?

var editor = new Jodit('#editor', {
   events: {
       beforeEnter: function (event) {
           if (tribute.isActive) {
               return true; // prevent enter plugin but not prevent default behavior
           }
       }
   }
});
// or
editor.events.on('beforeEnter', function (event) {
         if (tribute.isActive) {
               return true; // prevent enter plugin but not prevent default behavior
           }
});

Thanks again @xdan!
However, the only event I'm able to make work is afterInit. Haven't tried all events, but a fair amount.
I'm using jodit-angular, have not tried pure ts.

Unfortunately i don't know how return value from EventEmitter in Angular
<jodit-editor #editor (onBeforeEnter)="handleEvent($event)"></jodit-editor>

and
export class AppComponent {
  title = 'app';
  handleEvent() {
    console.log('triggered');
    return false;
  }
}

It is triggered but not return false

For that approach, events via jodit-angular, I think you could solve it by passing a callback from Jodit which then is called in handleEvent with the return value.
But I'm a bit puzzled why the afterInit works but not the other events. Anyway, I updated jodit-angular and tried the last approach but with the same results as in your example.
I then tested the options approach again (with the latest version of jodit-angular) - and now it works! I have not tried replicating by doing a reinstall of the previous jodit-angular version, but with the following configuration works for afterInit and beforeEnter with<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Template:
<jodit-editor  #editor id="jodit-id" [config]="joditConfig" [defaultValue]="content.body" ></jodit-editor>

Component:
 this.joditConfig = {
      buttons: 'paragraph, bold,strikethrough,underline,italic,|, \
                superscript,subscript,|, \
                ul,ol,|, \
                outdent,indent,|, \
                brush,|, \
                image,video,table,link,|, \
                align,undo,redo,\n,cut,hr,eraser,copyformat,|, \
                symbol,selectall,source, fullsize',
      imageDefaultWidth: 400,
      height: 600,
      events: {
        beforeEnter: (event) => {
          if (this.tribute.isActive) {
            return true; // prevent enter plugin but not prevent default behavior
          }
        },
        afterInit: (event, jodit) => {
          this.tribute.attach(document.getElementsByClassName('jodit_wysiwyg'));
          // this.jodit = jodit; // get editor instance
          return true;
        },
      }
    };

Ok, replicated and can confirm that<EMAIL_ADDRESS>do not work while<EMAIL_ADDRESS>do!
Ref: jodit/jodit-angular#5

Also, (onChange)="handleChange($event)" seems to only fire once in<EMAIL_ADDRESS>when having events defined in options.
Removing (onChange)="handleChange($event)" from the template and defining change event in options works but angular throws an error on init because (I guess) change fires on the [defaultValue]="content.body".
The easiest way around this is to make sure you do not assign the new value on the first change being fired.


Generalized Cullen Number in Java

Have you read the Contributing Guidelines on Pull Requests?
Yes, I have read the Contributing Guidelines on Pull Requests
Description
To generate the first n cullen numbers with base b.
Checklist

[yes] I've read the contribution guidelines.
[yes] I've checked the issue list before deciding what to submit.
[yes] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Fixes issue #6749

I am unable to solve the issues mentioned in the DeepSource report, as it does not seem to be my code. Please help.


Issue with "Fast5File.py"

I'm trying to execute 'yield_plot' and 'occupancy', yet get hit with the same error message, listed below, about half way through the fast5 files
The other poretools functions seem to work fine. Any ideas? I'm starting to wonder if my data is corrupted.
poretools times pass > pass_times.txt
Traceback (most recent call last):
  File "/usr/local/bin/poretools", line 9, in <module>
    load_entry_point('poretools==0.5.1', 'console_scripts', 'poretools')()
  File "/usr/local/lib/python2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/poretools_main.py", line 435, in main
    args.func(parser, args)
  File "/usr/local/lib/python2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/poretools_main.py", line 49, in run_subtool
    submodule.run(parser, args)
  File "/usr/local/lib/python2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/times.py", line 20, in run     start_time = fast5.get_start_time()
  File "/usr/local/lib/python2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/Fast5File.py", line 470, in get_start_time
    node = self.find_event_timing_block()
  File "/usr/local/lib/python2.7/site-packages/poretools-0.5.1-py2.7.egg/poretools/Fast5File.py", line 440, in find_event_timing_block
    path = fastq_paths[self.version]['template'] % (self.group)
KeyError: 'template'

Seems like it was just a corrupt file.


Something wrong when trying to authenticating. Reason:  The method or property WebBrowser.openBrowserAsync is not available on android

Prerequisites

[X] I have searched the repository’s issues and Kinde community to ensure my issue isn’t a duplicate
[X] I have checked the latest version of the library to replicate my issue
[X] I have read the contributing guidelines
[X] I agree to the terms within the code of conduct

Describe the issue
I'm using @kinde-oss/react-native-sdk-0-7x in an expo managed project and login is working fine on iOS but I'm getting this error on Android:
Something wrong when trying to authenticating. Reason:  The method or property WebBrowser.openBrowserAsync is not available on android, are you sure you've linked all the native dependencies properly?
Library URL
https://github.com/kinde-oss/kinde-react-native-sdk-0-7x
Library version
@kinde-oss/react-native-sdk-0-7x": "^1.3.0"
Operating system(s)
macOS
Operating system version(s)
macOS Sonoma 14.3
Further environment details
No response
Reproducible test case URL
No response
Additional information
No response

The same issue was encountered, any update?

If you npm install @kinde-oss/react-native-sdk-0-7x@next, that version works ok. (See #38)


Confliction with Refinery

I am using refinery cms as an engine fo some modules with my rails application. also using rails admin. but rails admin is not working properly with refinery in the stack.
when I loading admin page any next route is overriden by  refinery page and all requests ia handled by refinery controller instead of RailsAdmin::MainCOntroller. as a result I can not load any model data. always returning 400 not found from refinery page controller.

Outdated.


Error executing lua vim.lua:0: attempt to call method 'byte' (a nil value)

Hi,
I'm using neovim version v0.7.0-dev, along with packer and this plugin, but I can't use the s command an the Scommand cause an exception is thrown. The f and the F command is working properly.
Mi packer config is just:
 use { "ggandor/lightspeed.nvim" }

Here is the error:
E5108: Error executing lua vim.lua:0: attempt to call method 'byte' (a nil value)
stack traceback:
        vim.lua: in function 'region'
        /usr/share/nvim/runtime/lua/vim/highlight.lua:37: in function 'grey_out_search_area'
        .../site/pack/packer/opt/lightspeed.nvim/lua/lightspeed.lua:2139: in function 'go'
        [string ":lua"]:1: in main chunk

Thanks for your time!

(Fortunately) I cannot reproduce with the latest nightly. But I have no clue about the problem.
In Lightspeed, grey-out-search-area calls highlight-range-compat, with "v" passed in as regtype:
https://github.com/ggandor/lightspeed.nvim/blob/3a8759e10c315d4fe36bfb15a7e243fdee5f33e7/fnl/lightspeed.fnl#L343-L349
highlight-range-compat calls vim.highlight.range (that even defaults to "v" if the regtype argument is missing, so it actually shouldn't matter at all what we are doing in the plugin's code)
https://github.com/neovim/neovim/blob/e383543342db8e9dcf40f7d5237edeab29638880/runtime/lua/vim/highlight.lua#L38-L49
that in turn calls vim.region (passing on regtype to it), which throws the exception, when trying to call the byte method on "v".
https://github.com/neovim/neovim/blob/e383543342db8e9dcf40f7d5237edeab29638880/src/nvim/lua/vim.lua#L319-L332.

Ok, I think I have found it. #126, again :D It's insane how could both me and the author overlook so many errors, in such a small PR (and not test it...). Fixing this right now. (Now I'm actually wondering, why does it work for me, and apparently, for others, without errors?)

Hi, unfortunately this issue persists for me.

As the original author i use an unconfigured lightspeed.

I'm using latest neovim nightly and (surely) this plugin after fix commit

Okay, then first test it with stable Nvim, and then try older commits of Lightspeed...

The regtype argument apparently exists (not nil), because otherwise we'd have a message of "attempt to index... a nil value".
As a desperate sanity check, try lua local a = "v"; print(a:byte()) on the Nvim command line first.

Sorry for late response.
Latest version works on nvim v0.6.1.
Commit 005320ff9e128de8689c6e675fa64ed5963e2d1c seems to be the last working with 0.7.1 for me.
Commit 8fd73360e89a219b5701339c08a469f5703cf163 doesn't make any keymaps.
Commit e53a14b2b279e92fbca9646ec14188ab76dd009a shows error described above.

Could you give me the exact version of the nightly you are using?

NVIM v0.7.0-dev+995-g255935903
Build type: Release
LuaJIT 2.1.0-beta3```

Just realized two neovims were installed on the system, and older one was chosen. It works on the true latest version.
I apologize and thanks.

Okay, great, no problem!

This occurs for me with (an older) Neovim nightly NVIM v0.7.0-dev+1032-g01e58fb14 making the plugin unusable, so does this mean that the minimum required Neovim version has changed? The readme still mentions 0.5+.

This still happens for me on the absolute latest master branch.
NVIM v0.7.0-dev+e23af71d3
Build type: Debug
LuaJIT 2.1.0-beta3

@mattdemichele-pearson I cannot reproduce. At least on
NVIM v0.7.0-dev+1182-ge23af71d3
Build type: RelWithDebInfo

Maybe it has something to do with the Debug build(?)

I'm having this problem as well since today, I build from source, did it just now.

@yngwi And you're pretty sure that you're using the latest Lightspeed too?

@yngwi And you're pretty sure that you're using the latest Lightspeed too?

I thought I was as I am using a docker container that automatically updates everything on command, but apparently I was mistaken. I tried again today (updating my environment) and the error is gone now. Thanks and sorry for the confusion.

I removed and reinstalled it and still get this error.
E5108: Error executing lua vim.lua:0: attempt to call method 'byte' (a nil value)
stack traceback:
        vim.lua: in function 'region'
        /usr/local/share/nvim/runtime/lua/vim/highlight.lua:37: in function 'grey_out_search_area'
        ...ite/pack/packer/start/lightspeed.nvim/lua/lightspeed.lua:2145: in function 'go'
        [string ":lua"]:1: in main chunk

@h3ndry This should work now, double-check that it's the latest Nvim nightly and the latest Lightspeed you're using.

Same error here and updated from v0.7.0-dev+1077-g2c4efc034 to v0.7.0-dev+1237-g76769b8a1 and now it works like a charm :)

I reverted to the neovim stable version https://launchpad.net/~neovim-ppa/+archive/ubuntu/stable and that seemed to fix it too

@h3ndry This should work now, double-check that it's the latest Nvim nightly and the latest Lightspeed you're using.

Thank you. I updated neovim to  v0.7.0-dev+1245-gf291a0339. and that seemed to fix it.

I have this issue in NVIM v0.7.0-dev+1280-gf2e5f509d.
Is anyone able to repro?

@vidjul Update Lightspeed too.

I was using neovim version v0.7.0-dev+882-g9304ee387. Uninstalled then installed stable (0.6.1) and this now works like a charm. Thanks!

Same here, Ubuntu with latest neovim from ppa, lightspeed updated.

Reverting to 005320ff9e128de8689c6e675fa64ed5963e2d1c fixes the problem

Issue remains on latest release.
:version
NVIM v0.7.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by nixbld

switching to Commit 005320ff9e128de8689c6e675fa64ed5963e2d1c  fixed it for me
`
NVIM v0.7.0
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by runner@fv-az316-460
Features: +acl +iconv +tui
See ":help feature-compile"
system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/share/nvim"
Run :checkhealth for more info
`


BoxInterleaved model does not display properly

I tried the model below and it will not display correctly.
https://github.com/KhronosGroup/glTF-Sample-Models/tree/master/2.0/BoxInterleaved
Grimoire.js + BoxInterleaved.gltf result:

I confirmed that it is fixed in the latest version library.
Grimoire.js + BoxInterleaved.gltf result:


Proposed changes

Overview
This is a large pull request that represents a fairly significant divergence from the IARPA Janus phase 2 API. The main reason for this divergence is that the original API was designed to exclusively support the multi-image per template paradigm, which is not the traditional paradigm for face recognition. The Janice version of the API needs to support both the traditional and the new paradigm. The following sections will outline the various paths through the API.
Template Generation

janus_initialize
janus_allocate_template
Load image
janus_detect
janus_augment
janus_finalize_template
janus_finalize

Steps 3-5 can be called multiple times in a loop to enroll multiple images into a template. The maximum number of images that can be enrolled into a single template is given by janus_max_images_per_template(). For implementations that only support the traditional FR paradigm this function would return 1. For implementations that have no limit on the number of images this function would return -1
Template I/O
Template I/O is supported with janus_write_template and janus_read_template. Templates are stored as files on disk. Templates must be finalized before they are written.
Verification

janus_initialize
janus_allocate_template (template A)
janus_read_template (template A)
janus_allocate_template (template B)
janus_read_template (template B)
janus_verify
janus_free_template (template A)
janus_free_template (template B)
janus_finalize

Gallery Generation

janus_initialize
Template generation using steps described above
janus_enroll
janus_finalize

Steps 2 and 3 can be called multiple times in a loop. Each iteration enrolls a single image.
Gallery I/O
Gallery are stored as directories on disk. Gallery I/O is handled with the janus_write_gallery and janus_read_gallery functions.
Adding and removing Templates from a Gallery
Templates can be added with janus_enroll and removed with janus_remove_template
Search

janus_initialize
janus_allocate_template
janus_read_template (Probe)
janus_allocate_gallery
janus_read_gallery (Gallery)
janus_search
janus_free_template
janus_free_gallery
janus_finalize

Clustering

janus_initialize
Create a gallery using the steps described above
janus_cluster
janus_finalize

Clustering works on an enrolled gallery of templates. The traditional use case of finding identities in unlabelled galleries is supported if each template has a single image.

This was supposed to be committed to a fork not here.


error in makemigrations

Hi charettes,
when i use

python manage.py makemigrations

it get an error about

TypeError: Couldn't reconstruct field content_type on mutant.FieldDefinition: init() got multiple values for keyword argument 'polymorphic_type'

How do I deal with it?

Try upgrading to the latest django-polymodels and django-mutant alpha version, this should be fixed.

which one is the alpha version? @charettes

im on the master, i dont see any branch or tags name alpha.

thanks


[breaking] Special properties

This converts special sensor methods to properties for the sake of uniformity throughout the library. See discussion in #228.
This will break existing code!
Instead of
print( cs.ambient() )   # function call syntax
cs.mode = 'COL-AMBIENT'
while True:
    log(cs.ambient(do_set_mode=False)) # do not set mode when polling the sensor value

one has to
print( cs.ambient )   # property syntax
cs.mode = 'COL-AMBIENT'
cs.auto_mode = False  # do not set mode when polling the sensor value
while True:
    log(cs.ambient)

I've updated my part of the demo scripts. @dwalton76, if we decide to merge this, could you take care of yours?

I never use do_set_mode in openroberta so fine for me.

@ensonic, I was more concerned about function-call vs get-property syntax (cs.ambient() vs cs.ambient). I think openroberta was implemented before we introduced special sensor properties though, so I guess you use cs.value() instead? That should still work.

This looks good to me by the way, although I'd like to hear back from @ensonic on whether this will be an issue to get some user feedback.

@ddemidov , yes I use value(), see
https://github.com/OpenRoberta/robertalab-ev3dev/blob/develop/roberta/ev3.py#L452
FYI, all the code that interfaces with hardware is in there.


Added name and profile link

As per the rules of contribution,added name and github profile link for connecting to other enthusiasts.

@subrah107 thanks for contributing, now you can have a look and work on existing issues.


Fix SQLite support

SQLite DB was not writable because it was always located in /usr/src/shynet/, which is owned by root and not writable by appuser. SQLite needs the parent directory containing the DB file to be writable by the running user.
The applied fix is to place the DB file in /var/local/shynet/db and to create that directory in the Docker image with the right permissions.
SQLite setup is now documented in README as an alternative to Postgres.
Fixes #208.

Thanks!


[RESTEASY-2411] resteasy-jackson2-provider: default white-list based PolymorphicTypeValidator extension

Issue:
https://issues.jboss.org/browse/RESTEASY-2411
https://issues.jboss.org/browse/RESTEASY-2422
Upstream PR:
https://github.com/resteasy/Resteasy/pull/2223
https://github.com/resteasy/Resteasy/commit/51ab2fbc18e227e8aa8773c0cdd12b7c5836f334
https://github.com/resteasy/Resteasy/commit/cba57e29869b34450912da23dd3d243bd1a1a303

retest this please

I believe we're not willing to merge this anymore, hence closing.


Added config option to disable stack trace in browser window

On a production server it's not desirable to show users a stack trace in case of an error. Depending on the audience this might be for user friendliness or security reasons. I feel the option should default to False, but set to True for now to maintain current behaviour.

Can you expand on why this is useful?
This applies to the reporting of server errors when loading a page. IMHO I don't see how hiding these would help. The end-user then knows that something is wrong instead of getting only a status code, and can even copy-paste the error message when reporting the problem, which should help the dev team to fix the problem.

In my experience even fairly technical users like engineers don't appreciate a stack trace. If there is an error, they'll just report this fact. It even takes considerable effort to get them to post the actual error message in an issue report. The stack trace itself can be found in the server log, so it shouldn't be necessary for developers. It's obviously handy during development, so hence a config option.
Personally, I don't want the stack trace to be displayed for security reasons. Revealing code details and file locations to users may make it easier to attack the code (even if security through obscurity is no security at all)

Ok, I see your point. Thanks.
Can you please update the description of the new config option (now 'Show stack traces in browser window'), to indicate that its about server-side stack traces being shown in the browser?

Not really sure about the travis error. TEST_DOCS. looks unrelated.

Yeah that CI build is rather iffy lately :/  this looks good to go. Thanks!


Conditionally show addChildAction based on maxDepth limit

This got merged in feat/relationships and will be available on the next release, Thanks!


Implement user DM flow for sending new messages.

How do we DM users now then and respond to DMs?
make two API calls per message we want to send to ensure the channel is open?

Yep sounds about right, right now this cache is gateway only and seeing as the events were removed from V8 there's no point in keeping any of this around as it'll always be empty
Originally posted by @FasterSpeeding in https://github.com/nekokatt/hikari/pull/234#issuecomment-699677059

So can you explain what the issue actually is? What do we want to do?

we want to send messages to users.
This may require one API call if you know the channel ID, or two if you dont.

how come we don't know the channel id? Won't the user specify the ID when they want to send a message? and if we don't have the channel ID, how do we get it?

@ahnaf-zamil how come we don't know the channel id?
Because discord don't tell us anymore, v8 removed the event for it.

Won't the user specify the ID when they want to send a message?

Sure, they can. Where are they going to find an arbitrarily generated snowflake from Discord's internal datastore though? :)

and if we don't have the channel ID, how do we get it?

call the create_dm_channel endpoint first. Then call the create_message endpoint with that result for the channel.

So if we make this whole thing as a single function, is that it for this issue?

We wanted to implement it in a nicer implementation. This should allow sending several messages to the same dm without having to execute both calls all the time. Something along the lines of this:
async with user.dm() as dm_channel:
    await dm_channel.send("hello")
    await dm_channel.send("hello again")

I'd probably implement dm() in the same way I did trigger_typing(), where it works in an async with or as an await, possibly.

Yep, that was my idea


Login page redesign

This page should follow the design requirements as showing in this design file.
The main functionalities are:

once the email is submitted, the user should receive a verification email which allows to log in
page restyling following the design guidelines
add header and footer

Can I work on this issue.


Update script.cc

I added:
/**
     * @description Added this to find a workout around for {@link https://github.com/CondorLang/Condor/issues/8}
     * @author Berkmann18
     * @since 29/10/16 19:45 BST
     */
    void Script::Free(Isolate* isolate){
        script->Free(isolate);
    }

In order to do the second and third solutions from the issue #8

@Berkmann18. The code is incomplete, that's why it's not compiling. The api.cc function dynamically casts the script object to the Condor::Internal::Script object. Try doing the following:

Pass isolate through as a parameter in shell.cc
Add a void Free(Isolate* isolate) to the script.h
Free the memory in the [implementation](https://github.com/CondorLang/Condor/blob/master/src/condor/types/script/script.cc] of that function. The body should look like this:

isolate->FreeMemory(this, sizeof(Script));

removed shell.cc and created main.cc to handle the shell and file input.


[Bug] [dolphinscheduler-api] The UT coverage of AlertPluginInstanceController is zero

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The UT coverage of AlertPluginInstanceController is zero
What you expected to happen
The UT coverage of AlertPluginInstanceController is 100%
How to reproduce
Run test with coverage
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Fixed by https://github.com/apache/dolphinscheduler/pull/7952


Increases row data results to 100 for the filter analysis

Closes #8529

CR: @xavijam
CC: @saleiva

Wait for our frontend tests

It was @javisantana :)

github does not say that


Add example integration with structlog

Would you be interested in having an example integration with structlog? Or perhaps add it to extensions?

Yes that would be great 👍 A PR is very welcome

✅  PR ready for review!


Error pushing "payload logs" to ElasticSearch

Hi
I have 2 different matches to push tu AWS ElasticSearch, one for audit logs and the other for payloads.
When I configure audit logs, it works fine, but when i configure payloads, I get the following result repeatedly :
2016-10-20 18:26:18 +0000 [info]: reading config file path="/fluentd/etc/custom.conf"
2016-10-20 18:26:18 +0000 [info]: starting fluentd-0.12.28
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-mixin-config-placeholders' version '0.4.0'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-aws-elasticsearch-service' version '0.1.6'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-concat' version '0.6.2'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-elasticsearch' version '1.6.0'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-kinesis' version '1.1.1'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-parser' version '0.6.1'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-record-reformer' version '0.8.2'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-s3' version '0.7.1'
2016-10-20 18:26:18 +0000 [info]: gem 'fluent-plugin-systemd' version '0.0.4'
2016-10-20 18:26:18 +0000 [info]: gem 'fluentd' version '0.12.28'
2016-10-20 18:26:18 +0000 [info]: adding filter pattern="*.**" type="record_transformer"
2016-10-20 18:26:18 +0000 [info]: adding match pattern="service_audit" type="copy"
2016-10-20 18:26:18 +0000 [info]: adding match pattern="service_payload" type="copy"
2016-10-20 18:26:18 +0000 [info]: adding match pattern="fluent.**" type="copy"
2016-10-20 18:26:18 +0000 [info]: adding source type="forward"
2016-10-20 18:26:18 +0000 [info]: using configuration file: <ROOT>
  <filter *.**>
    @type record_transformer
    <record>
      product_name service
      environment_tag intqa102
      region us-east-1
      hostname ip-10-185-73-159.nonprod.xyz-aws.io
    </record>
  </filter>
  <source>
    @type forward
    port 24224
  </source>
  <match service_audit>
    type copy
    <store>
      @type s3
      s3_bucket prd00000280.open_platform_logs.nonprod1.us-east-1.xyz-aws
      path service/intqa102/audit/
      buffer_path /fluentd/log/s3_buffer_audit
      s3_object_key_format %{path}application_log_ip-10-185-73-159.nonprod.xyz-aws.io_%{time_slice}_%{index}.%{file_extension}
      s3_region us-east-1
      storage_class REDUCED_REDUNDANCY
      time_slice_format %Y%m%d%H
      utc
      format json
      include_time_key true
      flush_at_shutdown true
    </store>
  </match>
  <match service_payload>
    type copy
    <store>
      @log_level debug
      @type aws-elasticsearch-service
      type_name payload_log
      logstash_format true
      logstash_prefix intqa102
      include_tag_key true
      tag_key @log_name
      time_key_format %FT%T.%L%z
      flush_interval 5s
      <endpoint>
        url http://logs.open.platform.nonprod1.nonprod-xyz-aws.io:80
        region us-east-1
      </endpoint>
    </store>
    <store>
      @type s3
      s3_bucket prd00000280.open_platform_logs.nonprod1.us-east-1.xyz-aws
      path service/intqa102/payload/
      buffer_path /fluentd/log/s3_buffer_payload
      s3_object_key_format %{path}payload_log_ip-10-185-73-159.nonprod.xyz-aws.io_%{time_slice}_%{index}.%{file_extension}
      s3_region us-east-1
      storage_class REDUCED_REDUNDANCY
      time_slice_format %Y%m%d%H
      utc
      format json
      include_time_key true
      flush_at_shutdown true
    </store>
  </match>
  <match fluent.**>
    type copy
    <store>
      @type stdout
    </store>
    <store>
      @type s3
      s3_bucket prd00000280.open_platform_logs.nonprod1.us-east-1.xyz-aws
      path service/intqa102/fluent/
      buffer_path /fluentd/log/s3_buffer_fluent
      s3_object_key_format %{path}fluent_log_ip-10-185-73-159.nonprod.xyz-aws.io_%{time_slice}_%{index}.%{file_extension}
      s3_region us-east-1
      storage_class REDUCED_REDUNDANCY
      time_slice_format %Y%m%d%H
      utc
      format json
      include_time_key true
      flush_at_shutdown true
    </store>
  </match>
</ROOT>

2016-10-20 18:26:19 +0000 [info]: listening fluent socket on <IP_ADDRESS>:24224
2016-10-20 18:28:19 +0000 [info]: Connection opened to Elasticsearch cluster => {:host=>"logs.open.platform.nonprod1.nonprod-xyz-aws.io", :port=>80, :scheme=>"http", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id="BSIEG3OXZUHFJVASPPEB">, :region=>"us-east-1"}}
2016-10-20 18:28:19 +0000 fluent.info: {"message":"Connection opened to Elasticsearch cluster => {:host=>\"logs.open.platform.nonprod1.nonprod-xyz-aws.io\", :port=>80, :scheme=>\"http\", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id=\"BSIEG3OXZUHFJVASPPEB\">, :region=>\"us-east-1\"}}","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:20 +0000 [warn]: Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)
2016-10-20 18:28:20 +0000 fluent.warn: {"message":"Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:22 +0000 [info]: Connection opened to Elasticsearch cluster => {:host=>"logs.open.platform.nonprod1.nonprod-xyz-aws.io", :port=>80, :scheme=>"http", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id="BSIEG3OXZUHFJVASPPEB">, :region=>"us-east-1"}}
2016-10-20 18:28:22 +0000 fluent.info: {"message":"Connection opened to Elasticsearch cluster => {:host=>\"logs.open.platform.nonprod1.nonprod-xyz-aws.io\", :port=>80, :scheme=>\"http\", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id=\"BSIEG3OXZUHFJVASPPEB\">, :region=>\"us-east-1\"}}","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:22 +0000 [warn]: Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)
2016-10-20 18:28:22 +0000 fluent.warn: {"message":"Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:26 +0000 [info]: Connection opened to Elasticsearch cluster => {:host=>"logs.open.platform.nonprod1.nonprod-xyz-aws.io", :port=>80, :scheme=>"http", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id="BSIEG3OXZUHFJVASPPEB">, :region=>"us-east-1"}}
2016-10-20 18:28:26 +0000 fluent.info: {"message":"Connection opened to Elasticsearch cluster => {:host=>\"logs.open.platform.nonprod1.nonprod-xyz-aws.io\", :port=>80, :scheme=>\"http\", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id=\"BSIEG3OXZUHFJVASPPEB\">, :region=>\"us-east-1\"}}","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:26 +0000 [warn]: temporarily failed to flush the buffer. next_retry=2016-10-20 18:28:20 +0000 error_class="Fluent::ElasticsearchOutput::ConnectionFailure" error="Could not push logs to Elasticsearch after 2 retries. Broken pipe (Errno::EPIPE)" plugin_id="object:d37f0c"
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluent-plugin-elasticsearch-1.6.0/lib/fluent/plugin/out_elasticsearch.rb:333:in `rescue in send'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluent-plugin-elasticsearch-1.6.0/lib/fluent/plugin/out_elasticsearch.rb:323:in `send'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluent-plugin-elasticsearch-1.6.0/lib/fluent/plugin/out_elasticsearch.rb:308:in `write'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluentd-0.12.28/lib/fluent/buffer.rb:354:in `write_chunk'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluentd-0.12.28/lib/fluent/buffer.rb:333:in `pop'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluentd-0.12.28/lib/fluent/output.rb:338:in `try_flush'
  2016-10-20 18:28:26 +0000 [warn]: /var/lib/gems/2.1.0/gems/fluentd-0.12.28/lib/fluent/output.rb:149:in `run'
2016-10-20 18:28:26 +0000 fluent.warn: {"next_retry":"2016-10-20 18:28:20 +0000","error_class":"Fluent::ElasticsearchOutput::ConnectionFailure","error":"Could not push logs to Elasticsearch after 2 retries. Broken pipe (Errno::EPIPE)","plugin_id":"object:d37f0c","message":"temporarily failed to flush the buffer. next_retry=2016-10-20 18:28:20 +0000 error_class=\"Fluent::ElasticsearchOutput::ConnectionFailure\" error=\"Could not push logs to Elasticsearch after 2 retries. Broken pipe (Errno::EPIPE)\" plugin_id=\"object:d37f0c\"","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:26 +0000 [warn]: Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)
2016-10-20 18:28:26 +0000 fluent.warn: {"message":"Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:28 +0000 [info]: Connection opened to Elasticsearch cluster => {:host=>"logs.open.platform.nonprod1.nonprod-xyz-aws.io", :port=>80, :scheme=>"http", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id="BSIEG3OXZUHFJVASPPEB">, :region=>"us-east-1"}}
2016-10-20 18:28:28 +0000 fluent.info: {"message":"Connection opened to Elasticsearch cluster => {:host=>\"logs.open.platform.nonprod1.nonprod-xyz-aws.io\", :port=>80, :scheme=>\"http\", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id=\"BSIEG3OXZUHFJVASPPEB\">, :region=>\"us-east-1\"}}","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:28 +0000 [warn]: Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)
2016-10-20 18:28:28 +0000 fluent.warn: {"message":"Could not push logs to Elasticsearch, resetting connection and trying again. Broken pipe (Errno::EPIPE)","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}
2016-10-20 18:28:32 +0000 [info]: Connection opened to Elasticsearch cluster => {:host=>"logs.open.platform.nonprod1.nonprod-xyz-aws.io", :port=>80, :scheme=>"http", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id="BSIEG3OXZUHFJVASPPEB">, :region=>"us-east-1"}}
2016-10-20 18:28:32 +0000 fluent.info: {"message":"Connection opened to Elasticsearch cluster => {:host=>\"logs.open.platform.nonprod1.nonprod-xyz-aws.io\", :port=>80, :scheme=>\"http\", :aws_elasticsearch_service=>{:credentials=>#<Aws::Credentials access_key_id=\"BSIEG3OXZUHFJVASPPEB\">, :region=>\"us-east-1\"}}","product_name":"service","environment_tag":"intqa102","region":"us-east-1","hostname":"ip-10-185-73-159.nonprod.xyz-aws.io"}

Can you please help withh this issue ?
Thanks
Guillaume

reload_connections false fixed it for me.


Remove TODO about required httpHeader-bound collection shapes

We do support required httpHeader-bound collection shapes, as
evidenced by the test operation this commit adds, which is generated by
the server, that does not use Option for required member shapes.
This is because even though the deserialization always returns an
optional collection, we return None if no headers are found (as
opposed to an empty collection), so the call that sets the member field
on the operation structure does nothing; when build() gets called, a
BuildError is raised because the member is required.
Testing
See added test.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Is the set to be removed here (because removed in smithy 2)?


Change hardcoded value for GetMetricData from 20/10/2023 to "yesterday"

Change L47 to be more dynamically allocated to 24 hours prior the querying instead of a fixed time in the past.

Solved in 3f0689e.


Blades in the Dark v3.0

@wing-it Something weird is happening here that I cannot explain. The very first rule in my CSS (.charsheet{/STUFF/}) is not being applied on live Roll20. But I cannot see why; when I copy the exact same code that is on here into a custom sheet, it works!
The difference is easy to see, since all the fonts become sans-serif fonts when that first CSS rule is missing.
Apart from that, everything seems to work, so I really don't know what is going on. Any ideas?

interesting. I'm not sure what would cause that. I'll investigate a bit when I get a chance and see if I can narrow down the problem.

Thanks! It's not so catastrophic, since it just looks a bit worse, but it would still be great if it could be fixed once you get a chance.

If you think it will help, I can just submit a non-minified version of the CSS for next week for now. Don't know if that makes a difference, but since it is impossible for me to test anyway, maybe that's the best choice.


[Format] Doc has some format or type errors.

Complex-read3  has a format error in params.

Please remove the special symbol.
Complex-read8 word ratio's font is different from other words.

Complex-read9

Result could't match query result description.

miss 'ratio' property in 'invest' edge.


Home on takeoff

@DonLakeFlyer This is what you were suggesting.

@DonLakeFlyer It now also sets the takeoff yaw angle for RTL, so it should come back in the exact same orientation as it took off.

@tumbili Could you test this in sim?

It should RTL in direction of HOME, then turn into the direction it took off with once it descends.

@tumbili Please merge once tested in SITL. This shouldn't require an outdoor flight since its just logic changes. Please RTL from different heights over ground / home ( e.g. 1 m and 30 m) and ensure it turns back to the correct orientation both times.

Works fine, merging!


Fixed 2 typos

wrong directory command for MacOS and misspelled Eleanor

Thanks for your interest in palantir/alerting-detection-strategy-framework, @clhefton! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.


Running QNN C++ example with other models

I am trying to run the QNN C++ example on a Windows Dev Kit 2023 with models other than mobilenetv2-12 such as resnet50-v1-12, inception-v2-9, and shufflenet-7 but I am running into some issues. I can get the example to work with mobilenet, but when running the other models, I keep getting errors such as this:
2024-01-30 12:08:51.2430493 [W:onnxruntime:, qnn_execution_provider.cc:231 onnxruntime::QNNExecutionProvider::IsNodeSupported] BatchNormalization node BatchNormalization_token_2658 is not supported: qnn_utils.cc:454 onnxruntime::qnn::utils::GetQminQmax Qnn Data Type: %d not supported yet.562
Node 'BatchNormalization' OpType:BatchNormalization with domain:com.ms.internal.nhwc was inserted using the NHWC format as requested by QNNExecutionProvider, but was not selected by that EP. This means the graph is now invalid as there will not be an EP able to run the node. This could be a bug in layout transformer, or in the GetCapability implementation of the EP.
Is there any guidance that can help me solve this issue?
Thanks!

Are you using models from Onnx model zoo? Will take a look.

The root cause is op_types_to_quantize is not the full list. It doesn't cover the BatchNorm op which used by Resnet50 model. I created a PR to fix it.
https://github.com/microsoft/onnxruntime-inference-examples/pull/380

Got it thanks! I will try out again.

Have you tried? Can I close the issue?

I don't think QNN supports GPU on windows. For Linux/Android, you can change the QnnHtp.dll to libQnnGpu.so to give it a try.
https://github.com/microsoft/onnxruntime-inference-examples/blob/33c9f0fab885c0ebbb220e3ef04b3dc4dd49402c/c_cxx/QNN_EP/mobilenetv2_classification/main.cpp#L278


matter server doesn't properly handle SIGTERM when server failed to start

I had an upgrade issue which caused the server to be in a 'non-running' state, but the process hadn't actually exited:
(ignore the specific error here, this is just to show that the service was still running)
● matterserver.service - matterserver
     Loaded: loaded (/etc/systemd/system/matterserver.service; enabled; preset: enabled)
     Active: active (running) since Tue 2024-07-23 18:56:01 CDT; 14s ago
   Main PID: 817 (python)
      Tasks: 4 (limit: 1093)
     Memory: 80.5M
        CPU: 1.668s
     CGroup: /system.slice/matterserver.service
             └─817 /srv/homeassistant/bin/python -m matter_server.server

Jul 23 18:56:03 homeassistant python[817]: 2024-07-23 18:56:03.065 (MainThread) INFO [matter_server.server.stack] CHIP Controller Stack initialized.
Jul 23 18:56:03 homeassistant python[817]: 2024-07-23 18:56:03.065 (MainThread) INFO [matter_server.server.server] Starting the Matter Server...
Jul 23 18:56:03 homeassistant python[817]: 2024-07-23 18:56:03.069 (MainThread) ERROR [asyncio] Task exception was never retrieved
Jul 23 18:56:03 homeassistant python[817]: future: <Task finished name='Task-1' coro=<run.<locals>.new_coro() done, defined at /srv/homeassistant/lib/python3.12/si>
Jul 23 18:56:03 homeassistant python[817]: Traceback (most recent call last):
Jul 23 18:56:03 homeassistant python[817]:   File "/srv/homeassistant/lib/python3.12/site-packages/aiorun.py", line 219, in new_coro
Jul 23 18:56:03 homeassistant python[817]:     await coro
Jul 23 18:56:03 homeassistant python[817]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/server/server.py", line 160, in start
Jul 23 18:56:03 homeassistant python[817]:     raise VersionMismatch(
Jul 23 18:56:03 homeassistant python[817]: matter_server.common.errors.VersionMismatch: CHIP Core version does not match CHIP Clusters version.

however, when attempting to stop it with a systemctl stop matterserver, the service becomes hung, failing to properly handle SIGTERM
● matterserver.service - matterserver
     Loaded: loaded (/etc/systemd/system/matterserver.service; enabled; preset: enabled)
     Active: deactivating (stop-sigterm) since Tue 2024-07-23 18:56:25 CDT; 1min 4s ago
   Main PID: 817 (python)
      Tasks: 2 (limit: 1093)
     Memory: 82.5M
        CPU: 1.781s
     CGroup: /system.slice/matterserver.service
             └─817 /srv/homeassistant/bin/python -m matter_server.server

Jul 23 18:56:25 homeassistant python[817]:   File "/srv/homeassistant/lib/python3.12/site-packages/aiorun.py", line 270, in run
Jul 23 18:56:25 homeassistant python[817]:     loop.run_until_complete(shutdown_callback(loop))
Jul 23 18:56:25 homeassistant python[817]:   File "/usr/local/lib/python3.12/asyncio/base_events.py", line 687, in run_until_complete
Jul 23 18:56:25 homeassistant python[817]:     return future.result()
Jul 23 18:56:25 homeassistant python[817]:            ^^^^^^^^^^^^^^^
Jul 23 18:56:25 homeassistant python[817]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/server/__main__.py", line 230, in handle_stop
Jul 23 18:56:25 homeassistant python[817]:     await server.stop()
Jul 23 18:56:25 homeassistant python[817]:   File "/srv/homeassistant/lib/python3.12/site-packages/matter_server/server/server.py", line 216, in stop
Jul 23 18:56:25 homeassistant python[817]:     raise RuntimeError("Server not started.")
Jul 23 18:56:25 homeassistant python[817]: RuntimeError: Server not started.

home-assistant-chip-clusters-2024.7.0
python-matter-server-6.3.0

Running a production setup with just installing the python packages is NOT supported.
Please read the documentation.

marcel, why the defensive attitude?


表格列头添加内置排序功能

Is your feature request related to a problem? Please describe.
在表格列头中添加内置的排序能力
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

https://drip-table.jd.com/drip-table/schema/columns/sorter


[Vue] HTML draggable attribute not set correctly when using draggable callback

In my use case, I have a list of items that should only be draggable when a specific button is clicked. To achieve this, I used the draggable callback to control whether each item can be dragged.
The callback works as expected: items are not draggable when the mode is inactive. However, the HTML element does not have its draggable attribute properly set to false, which should prevent the user from dragging the item.
Expected:
When the draggable callback returns false, the corresponding HTML element should have its draggable attribute set to false, preventing the user from dragging the item.
Actual:
Although the draggable callback prevents dragging in the non-draggable mode, the HTML element's draggable attribute is not being set to false. As a result, users are still able to drag the item around despite the mode being inactive.
Code Example:
I am currently working around this issue by manually setting the draggable attribute within the callback, but it would be ideal if the library handled this natively.
 dragAndDrop({
  parent: parent,
  values: items,
  plugins: [animations()],
  draggable: (el) => {
    // Workaround to set the attribute manually
    el.setAttribute('draggable', isReordering.value.toString())
    return isReordering.value
  },
})

Versions:

Vue: ^3.4.34
Formkit/drag-and-drop: ^0.1.6

Browser:

Chrome on MacOS: 15.0

Hello @sashamilenkovic,
Something similar happens to me, when I close the element modal the reactivity in the Drag And Drop elements stops working.
Do you know what this could be due to?
Code

<template>
<Dialog>

        Listado de categorías

                   {{ category }}

      <button
          class="btn btn-primary mt-5"
          @click="handleSortedCategory"
           >
            Ordenar Categorías
      </button>

</Dialog>
</template>

Bug
When I open the modal in the first render of the component, Drag And Drop works, but when I close the modal, the sortable elements stop working.
Preview
https://github.com/user-attachments/assets/1692b657-6a6b-42f0-b0a9-2e9d45603f1f

I experienced the same issue.
Thank you for your workaround, it saved me a lot of time! ❤

Hey @vBenTec! I experienced the same issue. Thank you for your workaround, but it causes an error in console.log.
@formkit_drag-and-drop_vue.js?v=603f3c1b:1134 Uncaught TypeError: Cannot read properties of undefined (reading 'el')

It’s possible that there’s an issue in your component setup, particularly with how refs are being handled. I recommend verifying that both your refs are properly initialised when the component mounts and adding a guard clause (?) for an early return if necessary. Additionally, please ensure you’re using the same version as I did to rule out version inconsistencies.

Hey @vBenTec! I experienced the same issue. Thank you for your workaround, but it causes an error in console.log.
@formkit_drag-and-drop_vue.js?v=603f3c1b:1134 Uncaught TypeError: Cannot read properties of undefined (reading 'el')

It’s possible that there’s an issue in your component setup, particularly with how refs are being handled. I recommend verifying that both your refs are properly initialised when the component mounts and adding a guard clause (?) for an early return if necessary. Additionally, please ensure you’re using the same version as I did to rule out version inconsistencies.

Thanks for your help. I tested your version, and it worked without errors. The workaround is to update the draggable attribute using the DOM and JS instead.


Fix parser options

Fixes issue #351, adding support for parser to be a function, as mentioned in the existing documentation.
Type

ℹ  What types of changes does your code introduce?

[x] Fix
[ ] Perf
[x] Docs
[ ] Test
[ ] Chore
[x] Feature
[ ] Refactor

SemVer

ℹ  What changes to the current semver range does your PR introduce?

[x] Bug (:label: Patch)
[x] Feature (:label: Minor)
[ ] Breaking Change (:label: Major)

Issues

ℹ  What issue (if any) are closed by your PR?

Fixes #351

Checklist

ℹ  You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. This is a reminder of what we are going to look for before merging your code.

[x] Lint and unit tests pass with my changes
[x] I have added tests that prove my fix is effective/works
[x] I have added necessary documentation (if appropriate)
[x] Any dependent changes are merged and published in downstream modules

@Cybolic tests failed, can you fix it?

The test fail on comparing the snapshot, not on the actual code. I'm afraid I'm not familiar enough with the test setup to know what needs to be done to fix the error.

Closing due to inactivity for now, feel free to reopen :)


Add temporary path credentials api

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes

Added temporary path credentials openapi spec and implementation

are we going to add unit tests for it or we plan to fully rely on the spark integration project to test it?

@cloud-fan, I added some tests for CredentialOperations, however it is mostly testing the test flows with mocks, and that requests reach the cloud clients.


Ability to store encrypted files or messages

Maybe we could just add an "Add file"-button to the "Notes" textbox. But I wouldn't overuse it.

This could be useful when having recovery codes in a file

I also think it's useful and we should implement it. But why do you need recovery codes stored in your password manager when you are storing your credentials in the same password manager? If it's broken, they're both gone. But nevertheless it's useful for other things.

LOL, didn't think about that 😂

Not easy to implement, and would result in long loading times for the database. So closing this.