Grok parse fail via kibano logs of dockermonitoringloggingalerting_elastalert

Pretty much all is working, works like a charm. Ony in kibana is see the most entry's coming from dockermonitoringloggingalerting_elastalert. It doesn't say much only it does include the tag "_grokparsefailure" it creates that names because my folder is that btw. Since i see that kibana and elastalert are being build with docker compose (would prefer to  not have that before it though, but no problem)
Is there any way to debug this? To see what the grok is going on :) i did ran that seperate config check of logstash but that says all fine too.
Also any best practices about adding more "hosts" to this setup? Thanks, and good work on all this.

Hey @riemers, the logstash config validation just checks, if the config is legitimate, not if it makes sense. ;)
To debug this, the question really is: what is in the message field of those elastalert logs that are coming in?
`_grokparsefailure is the default tag, that logs get tagged with, if none of the other filters matched. Check the documentation on it here.
Regardless of everything else, if you just want to not get that many logs from elastalert, you can define a filter in the logstash that drops those logs instead of sending them to Elasticsearch (or adjust the loglevel of elastalert itself of course). Just check what the defining characteristics of those unnecessary logs are.
About "more hosts": Not really. But it should be pretty straight forward, throw a filebeat, node-exporter and cadvisor on every host you want to monitor. Configure the Prometheus config, maybe tweak a filter here and there, done. The hard part will be to figure out the networking and security (shutting down ports), but that really isn't specific to this suite. Probably best to use some framework for that (e.g. Docker Swarm, Kubernetes and such). Actually I am currently dabbling with Kubernetes and like it a lot, so I might port the whole thing at some point.

Ah i think i understand the grokparse issues then, if no match then it becomes that. Basicly i need to add more filters.
Made the dashboard more container group aware, it was missing the items on the graps and i use xfs rather then ext on the free and used (shows a bit weird if you have multiple filesystems)
Working on a docker compose for an extra host. If i have an example one could use that to add it to the setup.

Sure thing, I am more than happy to include such an extra docker-compose.yaml! Would you mind adding it via PR, when you feel it's ready?

Sure, its just the stripped version. You still need to add the ip, testing that setup right now. Will update you when its ok.

I have this so far, but i don't see any logs coming in, i assume i have to set the listening on the main config also to something public but since its in a network i don't know if it can reach eachother. I openened the ports for logstash too but have not had a lot of luck as of yet. Also using the latest cadvisor since the last one had a bug. I think it is mainstream now anyways.
`version: '2'
services:
#########################################################
LOGGING
#########################################################
Runs on your node(s) and forwards all logs to Logstash.
filebeat:
container_name: filebeat
image: prima/filebeat
volumes:
- ./filebeat/filebeat.yml:/filebeat.yml
restart: always
labels:
container_group: monitoring
logging:
driver: gelf
options:
gelf-address: udp://realip:12201
labels: container_group
#########################################################
MONITORING
#########################################################
Runs on your node(s) and forwards node(host) metrics to Prometheus.
nodeexporter:
container_name: nodeexporter
image: prom/node-exporter
expose:
- 9100
restart: always
labels:
container_group: monitoring
logging:
driver: gelf
options:
gelf-address: udp://realip:12201
labels: container_group
#########################################################
Runs on your node(s) and forwards container metrics to Prometheus.
cadvisor:
container_name: cadvisor
image: google/cadvisor:v0.24.1
ports:
- 8080:8080
expose:
- 8080
volumes:
- "/:/rootfs:ro"
- "/var/run:/var/run:rw"
- "/sys:/sys:ro"
- "/var/lib/docker/:/var/lib/docker:ro"
restart: always
labels:
container_group: monitoring
logging:
driver: gelf
options:
gelf-address: udp://realip:12201
labels: container_group
#########################################################`

@riemers: Yeah, you would have to foward the ports logstash uses to the host like so:
    ports:
      - "5044:5044"
      - "12201:120201"

leading to a logstash manifest like this:
# Aggregates logs and forwards them to Elasticsearch.
  logstash:
    container_name: logstash
    image: logstash
    expose:
      - 5044
      - 12201/udp
    ports:
      - "5044:5044"
      - "12201:120201"
    volumes:
      - ./logstash/config:/config
      - ./logstash/patterns:/opt/logstash/extra_patterns
    restart: always
    labels:
      container_group: monitoring
    networks:
      default:
        ipv4_address: <IP_ADDRESS>
    logging:
      driver: gelf
      options:
        gelf-address: udp://<IP_ADDRESS>:12201
        labels: container_group
    command: logstash -f /config --auto-reload

Of course then you have to figure out your networking and routing, which may or may not be straight forward, depending on the complexity of your setup. But having done that the other hosts should be able to ship node and container logs to logstash (using the host IP, not the logstash IP to define where to ship the logs).
Vice versa, so Prometheus can scrape the metrics from cAdvisor and nodeexporter, the other hosts  would have to have those ports being forwarded as well. In the manifests you posted above, you're doing so for cAdvisor with port 8080, but nodeexporter is missing it for port 9100.

I'll work it out a bit further and once i have something i'll post it back or make a pull request. A bit slow these days, just became father again. Bit crazy now :)

Alright, I'll just leave this open until then.
And hey, congratulations! All the best... and many calm nights! :)

Just a note, it seems like some images like logstash/elasticsearch went to 5.x it seems. the auto reload function didn't work anymore and the kibana dashboard needed a newer version to work with elasticsearch. Maybe it was just me but i had to change some versions here and there to get it back up and running. Managed to get a host monitored so far with my configuration, adjusted the dashboard for it now trying to get logs from it too.

Thanks for pointing this out, @riemers! I just added tags to fix the images for the ELK stack containers. The idea is to keep things working until ELK 5.0 has been around for a while and the different images have been updated accordingly, then we can upgrade the whole stack at once and iron out little things like dashboards, configuration etc.
Let me open a new issue for this, so we don't forget.
And regarding Logstash? Do you mean you got it to work with multiple hosts?

Another one for the docs, if you use elastalert and you have your own CA for certificates it is not as simple as replacing ./alertmanager/ca-certificates.crt:/etc/ssl/certs/ca-certificates.crt in the docker file. Instead since it runs python, you have to replace the python request pem. Which for me ended up with ./elastalert/certs/ca-certificates.crt:/usr/lib/python2.7/site-packages/requests-2.12.1-py2.7.egg/requests/cacert.pem something to take note if someone has their own CA too.
Still working on the rest, i managed to get the other hosts to attach to the docker setup. Just thinkering on how to make it more "secure" (looked at the FYI issue reported by another guy with ssl support and such but is only for loghost)

Hey @riemers,
that ca-certificate-for-alertmanager-issue seems very specific. How about we don't put it in the README for that reason? I think people who really do have the same issue will be able to find it here as well, or we can refer them here if they don't find it by themselves.
Goof job on making them connect! Did you make any progress on securing them yet? What do you think about #10?
And for your p.s.: You're right, maybe this is a better default behaviour, @pierreozoux seems to have indicated the same. I am going to make some changes to make logging to the screen the default and provide info on how to turn it off.

Fine with me. Just trying to make sure people don't have to do the same
hoop.
Op 13 december 2016 om 20:01:53, Wilhelm Uschtrin<EMAIL_ADDRESS>schreef:

Hey @riemers https://github.com/riemers,
that ca-certificate-for-alertmanager-issue seems very specific. How about
we don't put it in the README for that reason? I think people who really do
have the same issue will be able to find it here as well, or we can refer
them here if they don't find it by themselves.
Goof job on making them connect! Did you make any progress on securing
them yet? What do you think about #10
https://github.com/uschtwill/docker_monitoring_logging_alerting/issues/10
?
And for your p.s.: You're right, maybe this is a better default behaviour,
@pierreozoux https://github.com/pierreozoux seems to have indicated the
same. I am going to make some changes to make logging to the screen the
default and provide info on how to turn it off.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uschtwill/docker_monitoring_logging_alerting/issues/5#issuecomment-266829624,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AApoOWciMIDsAj7goxc80WnwPIfWaGERks5rHuuhgaJpZM4KW8S2
.

Alright then.
Regarding the logging I just got tripped up badly:
With docker-compose 1.8.x it was possible to just log to both logstash and stdout by omitting -d from docker-compose up. But with 1.9.x this doesn't work anymore, every container just prints a single line of WARNING: no logs are available with the 'gelf' log driver with docker-compose not returning but staying in the foreground instead.
The idea was to use that feature of 1.8.x (that people apparently considered a bug) to by default log to stdout as well. Sadly, this won't work anymore.
My next-best idea is to just include a hint in the README on how to selectively "toogle" between logging to logstash and stdout for individual containers.

That is bad indeed, i used that up without -d too. Still works on my side too. Perhaps you could add extra entry's in the docker compose and call them "alertmanager-logs" instead of "alertmanager" so you can still do "docker-compose up alertmanager-logs" don't know if that is something that can be done, just thinking out loud here.

Hmm, I am not sure I understand what you mean. :)

well docker compose consist of multiple containers, perhaps you can make a version with default logging to stdout and another for gelf. That way you can just swap them in. You could even just stop/start individually. Just don't know if 2 docker compose files in the same dir can work or perhaps just rename them when you want to use them (if that makes sense)

Ahhh, now I get it. Yeah, that could work I suppose, something like docker-compose -f docker-compose.kibana.yml and docker-compose -f docker-compose.stdout.yml. On the other hand I feel like this convolutes things a bit, even if wrapped in a setup.sh, so you could use it with sh setup.sh kibana and sh setup.sh stdout ...
So far I've added a paragraph to the readme to clarify things.

Closing this, since the original issue has long been addressed and a new issue has been opened.


remove README-branch.txt

Fixes #1039

OK, this is closer, but the issue1039 branch still branches from develop.  It should branch from 2.5-latest.  I know there is not much difference at this point, but if we are to follow the process correctly, it is best if we do it properly right from the beginning.
Since I already have several v2.5 patches that need to go into a hot fix, should I just do this one as one of those, and not worry about a separate pull request?

It should branch from 2.5-latest

I didnt' see it as a hotfix issue so develop seemed natural to me.

should I just do this one as one of those, and not worry about a separate pull request?

Sounds good.

Since it's wrong in the master branch (and so misleading), and we're making a hotfix anyway, seems like it should be done.


Support for training lm_head in freeze finetuning_type

What does this PR do?
Fixes #3694
Before submitting
A ValueError occurs because lm_head is not included in model.named_parameters().
https://github.com/hiyouga/LLaMA-Factory/blob/main/src/llamafactory/model/adapter.py#L124-L131
[rank6]:   File "/llama-factory/src/llamafactory/model/adapter.py", line 135, in _setup_freeze_tuning
[rank6]:     raise ValueError(
[rank6]: ValueError: Module lm_head is not found, please choose from norm, embed_tokens

If embed_token and lm_head are tied together like gemma2, a KeyError occurs in model.named_parameters() where lm_head cannot be found.

We do not need to specify lm_head to trainable if the model ties word embeddings, since the input/output embeddings share the same weights. Just use freeze_extra_modules: embed_tokens


Search bar should handle case where multiple accounts have the same public key

Problem
Same public key can be assigned to multiple accounts.
Today HashScan displays first account returned by mirror node request.
It should list resulting accounts and let user choose.
Solution
Display an intermediate page which enables user to walk through the account list and navigate to a specific one.
Alternatives
No response

Some test data on mainnet.
Public key:

0x00032e74f928e6d5cf9ec40f554cf2172f379776e4591f75973c8c013295dd41

Associated accounts:

0.0.1188864-wbhys
0.0.1188869-vvukz


Make the safe-string-interpolation faster

Would you consider to serialize faster and with less allocations?
I have a couple of projects that have routines or tricks which can be reused here:

fast-string-interpolator that uses pre-allocated thread local instances of java.lang.StringBuilder under hood.
jsoniter-scala that has very efficient routines for serialization of double or float primitives, java.util.UUID, and java.time.* classes.

This is a great input. Thanks for that. I will be interested in those projects to make things really faster. I am planning for a quick before the end of this year. Let’s work on it

Here are small but more efficient than standard implementations of .toString for java.time._ types.


Let config cache smoke test run :configuration-cache:embeddedIntegTest

@bot-gradle test RFM

OK, I've already triggered ReadyForMerge build for you.


Spectrum measurement tutorial sequence

This PR adds in a new sequence for remeasuring the galaxy. This adds a new table example_galaxy_table and vue component SpectrumMeasurementTutorialSequence. The vue component is c-spectrum-measurement-tutorial
Important changes
In stage one, several changes callback functions have been made to make them more general:

_on_galaxy_row_click → on_table_row_click and accepts the table associated with the row as a parameter. this way it can be used for multiple tables. It is wrapped in a lambda function when used in a callback
galaxy_table_selected_change → table_selected_change: The change dictionary, provided by the hub observer, includes an owner key which includes the table. This is now used instead of directly referencing one of the tables.
A new callback has been added for on_spectrum_click_example_galaxy which adds the ability to fill in the galaxy velocity if we have reached that point in the tutorial. The problem here was I couldn't find a way to plug directly into whatever was automatically updating the velocities for the selected galaxies.

Guidelines
So the new guidelines are in

spectrum_measurement_tutorial_sequence folder
spectrum_measurement_tutorial.[vue,py]
guidelines_spectrum_measurement_tutorial.vue contains the actual guidelines, and is included as a component in spectrum_measurement_tutorial.vue. Each window item has a value elements so it is easy to associate with a step.
spectrum_measurement_tutorial.vue has a step watcher, where sequencing can be done using state variables and vue_ functions defined in spectrum_measurement_tutorial.py

Some problems
needs input from @Carifio24

Newly added state variables are being reset to their original value. These include show_meas_tutorial, show_galaxy_table and show_example_galaxy_table. Right now, visibility is being controlled with markers
This only pulls data from the database so far as I know, it is not hooked up to send it back. This is fine for now.

needs input from @patudom

Histogram - The outlier points are no longer clear because the bar height (or if we used the dotplot, the marker size) is too small. It currently only shows a single bar clearly when zoomed all the way out.    Zoomed in view
Right now it is just one dotplot viewer. It is not too difficult to add another for the second measurement data (currently it just puts them on the same plot). But there is a space issue. The solution I see would be to put them side by side.

We discussed the ability to filter in tables yesterday, so I've got an implementation of that in https://github.com/cosmicds/cosmicds/pull/197 if it's useful for this sequence. I tried this branch out on my machine yesterday and I'm also seeing the issue with the new callback properties getting reset. Still trying to get to the bottom of that.

Merging so we can proceed with the file/stage name changes. I can provide more detailed feedback after that process is complete. Thanks!


"New" timeline buttons show all classes

OS
No response
Godot Version
3.4.stable
Plugin Version
1.4.1 (latest)
Issue description
Creating a new timeline doesn't seem to filter for the Timeline class; nor is there a way to save a timeline to an external file like there are in other parts of the Godot UI where this same UX is used.
Steps to reproduce

Create an EventManager
Click on "Timeline"
Be flooded with "New $kitchen-sink" menu entries

Workaround
No response

Yes, is an annoying issue. I'm thinking on replace it with a custom picker.
The issue is related to https://github.com/godotengine/godot-proposals/issues/18

#62 now has a custom editor that makes easy the creation of new timelines


Is it still private preview?

Last time this document was updated 3 years ago. Is it still private preview?

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

ID: ec855c91-2d07-f16b-c6ea-d94c761224aa
Version Independent ID: fbd17954-d1d3-86f2-18ed-7a7ca58bb9ff
Content: TFS/TestManagement/RestClient TestHttpClient2_2 API
Content Source: docs/extend/reference/client/api/TFS/TestManagement/RestClient/TestHttpClient2_2.md
Product: devops
Technology: devops-ecosystem
GitHub Login: @chcomley
Microsoft Alias: chcomley

Hi @SychevIgor, Thanks for reaching out. I've contacted a PM to get some clarity on this and will be in touch. Thanks!

Adding @aaronhallberg for input please. Thanks Aaron!

Hi @SychevIgor, I've just confirmed that this is no longer in private preview and is GA. We'll be working on updating this set of articles soon. Here's the official documentation you can refer to: https://docs.microsoft.com/en-us/rest/api/azure/devops/test/runs?view=azure-devops-rest-5.1
Do let us know if you have any further questions. Thank you!

Well, for a 3 months- I already find the right link. Issue was created to save some time for others.

@SychevIgor, Thanks for your intention, truly. Glad you found the right link.


Fix single_column test in pr suite

The resolution didn't get updated along with the change in the test case.

Checklist

[x] Testing comment in the PR documents testing used to verify the changes

Testing
I was able to set up the pr suite so I think this is fixed.  I'm sitting in the queue on Perlmutter.

Thanks @sbrus89!


net/http: undeclared HTTP/2 trailers not sent when response body is empty and unflushed

What version of Go are you using (go version)?

$ go version
go version go1.19 darwin/arm64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/ashah/Library/Caches/go-build"
GOENV="/Users/ashah/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/ashah/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/ashah"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/ashah/sdk/go1.19"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/ashah/sdk/go1.19/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.19"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/ashah/src/github.com/akshayjshah/trailererr/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/0h/bt5cl84j5zs2t3sbyyy8vgzw0000gn/T/go-build2678483231=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
In a net/http handler, I wrote some HTTP trailers to the response writer using http.TrailerPrefix. The handler never writes to or flushes the response body. (This is admittedly odd, but trailers without a body are necessary in the gRPC HTTP/2 protocol.)
What did you expect to see?
I expected the client to receive the trailers regardless of the HTTP protocol version in use.
What did you see instead?
Everything works as expected when the server and client use HTTP/1.1. Regardless of the HTTP protocol version, everything also works as expected if I stop using http.TrailerPrefix and instead set the "Trailer" header.
When the client negotiates up to HTTP/2 and the handler uses http.TrailerPrefix, the client doesn't receive any trailers unless the handler flushes the response body. This is a little strange - it doesn't match the HTTP/1.x behavior, and it's not obvious why flushing a response without any buffered data should do anything at all. (I assume that something in x/net/http2 doesn't send trailers unless we've sent a DATA frame, and perhaps flushing always sends a frame? I haven't dug far enough to verify that hypothesis.)
Here's a reproduction:
package main

import (
	"io"
	"net/http"
	"net/http/httptest"
	"testing"
)

func testTrailers(tb testing.TB, flush bool) {
	const trailerKey = "Foo"
	handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		w.Header().Set(http.TrailerPrefix+trailerKey, "Bar")
		if flusher, ok := w.(http.Flusher); ok && flush {
			flusher.Flush()
		}
		if _, err := io.Copy(io.Discard, r.Body); err != nil {
			tb.Errorf("discard request body: %v", err)
		}
		if err := r.Body.Close(); err != nil {
			tb.Errorf("close request body: %v", err)
		}
	})

	srv := httptest.NewUnstartedServer(handler)
	// Replacing the next two lines with srv.Start() allows both tests to pass.
	srv.EnableHTTP2 = true
	srv.StartTLS()
	defer srv.Close()

	res, err := srv.Client().Get(srv.URL)
	if err != nil {
		tb.Fatal(err)
	}
	tb.Logf("response protocol is %s", res.Proto)
	if _, err := io.Copy(io.Discard, res.Body); err != nil {
		tb.Fatalf("discard response body: %v", err)
	}
	if res.Trailer.Get(trailerKey) == "" {
		tb.Fatalf("missing trailer %q, got %v", trailerKey, res.Trailer)
	}
}

func TestTrailersFlush(t *testing.T) {
	testTrailers(t, true /* flush */)
}

func TestTrailersWithoutFlush(t *testing.T) {
	testTrailers(t, false /* flush */)
}

cc @neild

i send i wip cl.  i found rws.conn.writeHeaders return https://go.googlesource.com/go/+/refs/tags/go1.19/src/net/http/h2_bundle.go#6218, I am still researching.  hope some help

@ianwoolf Any changes to the code in h2_bundle.go would need to be made in golang.org/x/net/http2. They'll be vendored into net/http during the next release cycle.

@ianwoolf Any changes to the code in h2_bundle.go would need to be made in golang.org/x/net/http2. They'll be vendored into net/http during the next release cycle.

sorry i missed that, i am sending a new cl to golang.org/x/net/http2

The problem is that we're checking for undeclared trailers (with the magic Trailer: prefix) only after sending the response headers, which results in the headers being sent with an END_STREAM flag to finish the request. Flushing the ResponseWriter avoids the problem by sending the headers while the handler is still running.
You can probably also work around the problem by calling WriteHeader.
https://go.dev/cl/426874 contains a fix.

The problem is that we're checking for undeclared trailers (with the magic Trailer: prefix) only after sending the response headers, which results in the headers being sent with an END_STREAM flag to finish the request. Flushing the ResponseWriter avoids the problem by sending the headers while the handler is still running.
You can probably also work around the problem by calling WriteHeader.
https://go.dev/cl/426874 contains a fix.

hmm..  i get it, thank you

Thank you everyone for the discourse, for reporting the issue @akshayjshah and @neild for the fix in CL https://go-review.googlesource.com/c/net/+/426874 in September 2022 for x/net/http2 which got bundled into the  standard library(so fixed many Go cycles ago) and the repro from @akshayjshah now passes. With that I shall close this issue!


⚖ Register Validator: P2P.ORG

Network Choice
dentrite-1
Who are you?
P2P.ORG https://p2p.org
Delegator address
axone1zvt5zj2cmswe5ekl2q6fd5l2nrfqghx93ufncr
gentx
{"body":{"messages":[{"@type":"/cosmos.staking.v1beta1.MsgCreateValidator","description":{"moniker":"P2P.ORG","identity":"","website":"https://p2p.org","security_contact":"cosmos-security@p2p.org","details":"P2P Validator (P2P.org) - an early contributor \u0026 long-time Cosmos Ecosystem participant. Stake with us for a secure, reliable and simple service"},"commission":{"rate":"0.050000000000000000","max_rate":"0.200000000000000000","max_change_rate":"0.010000000000000000"},"min_self_delegation":"1","delegator_address":"","validator_address":"axonevaloper1zvt5zj2cmswe5ekl2q6fd5l2nrfqghx9pf9pzp","pubkey":{"@type":"/cosmos.crypto.ed25519.PubKey","key":"EglgXzK5EmvVwieYtghutWNJxzgUP2lGajv/vlXutR4="},"value":{"denom":"uaxone","amount":"1000000000000"}}],"memo":"a2b214a62451175b7e2017f6658a5859f9675255@<IP_ADDRESS>:26656","timeout_height":"0","extension_options":[],"non_critical_extension_options":[]},"auth_info":{"signer_infos":[{"public_key":{"@type":"/cosmos.crypto.secp256k1.PubKey","key":"AoB+PdH66wCSvBGPPXYiwXHRS2+lHQH/yeclBulAXhlH"},"mode_info":{"single":{"mode":"SIGN_MODE_DIRECT"}},"sequence":"0"}],"fee":{"amount":[],"gas_limit":"200000","payer":"","granter":""},"tip":null},"signatures":["9Khr24CcXUm/CrRLtgjDwMSaj8pFVAWAD9Pj4APwsJZSA8WVpwpdyT59U4h5aU/Fy7AYn4HsMCM47GJb6clvsA=="]}

😉 Here is the corresponding PR: https://github.com/axone-protocol/networks/pull/821


Enable lowering ttir all_reduce and mesh_shard to ttnn and flatbuffer

Overall Plan
As a second step of multi-device support plan, this PR allows to lower all_reduce and mesh_shard ops to TTNN and flatbuffer format.

Convert MLIRs from JAX/OpenXLA/PJRT to TTIR (merged)
Pass converted TTIR to TTNN MLIR and Flatbuffer format (this PR)
Parse TTNN flatbuffer and execute in TT Runtime

Implementation Details

The all_reduce op lowering needs special handling in this PR.
Although all_reduce lowering is supposed to be one to one mapping from ttir all_reduce to ttnn all_reduce, the all_reduce ops are broken down into reduce_scatter and all_gather ops because current support of all_reduce in TTNN is not stable.
In addition, the reduce_scatter op in TTNN currently does not support two dimensional tensor correctly. As a temporary workaround, we insert reshape ops front and back to make the tensor as four dimensional tensor.
Once stable code is available, the code can be removed and simple one to one mapping code can be replaced.

Changes in this PR

all_reduce breaks down into reduce_scatter and all_gather
mesh_shard is directly converted
corresponding dialect unit tests are added

The mesh shape in the output of ./build/bin/ttmlir-opt --ttir-to-ttnn-backend-pipeline="system-desc-path=ttrt-artifacts/system_desc.ttsys" test/ttmlir/Dialect/TTNN/ccl/all_gather.mlir doesn't look right:
%0 = "ttnn.get_device"() <{mesh_shape = #ttnn<mesh_shape<PHONE_NUMBER>66816x4>}> : () -> !tt.device<#device>

@gfengTT  It seems like there was some issue in getOrInsertDevice() when assigning default meshShape.

The mesh shape in the output of ./build/bin/ttmlir-opt --ttir-to-ttnn-backend-pipeline="system-desc-path=ttrt-artifacts/system_desc.ttsys" test/ttmlir/Dialect/TTNN/ccl/all_gather.mlir doesn't look right:
%0 = "ttnn.get_device"() <{mesh_shape = #ttnn<mesh_shape<PHONE_NUMBER>66816x4>}> : () -> !tt.device<#device>

@gfengTT It seems like there was some issue in getOrInsertDevice() when assigning default meshShape. Thanks for catching this. I believe fixed it. Can you double check if it is fine?

Yes, looking good now.


WIP: Drop table after test

@cheungpat, thanks for your PR! By analyzing the annotation information on this pull request, we identified @rickmak to be a potential reviewer


ReadAsString

While for the example it definitely is more clear to use ReadAsStringAsync and JsonConvert.Deserialize that json string into objects, there should maybe be a note that this can lead to poor performance and increased memory footprint, especially for large responses, since a string has to be created.  Newtonsoft.Json supports deserializing from the response stream directly without requiring the string to be created which can have other improvements as well like not requiring the response to be fully buffered.

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

ID: 155a1af2-3ce7-058b-d243-b4d7a9cd6e22
Version Independent ID: 47f0df9e-ab6a-a38b-5916-991b5110ebf1
Content: Consume a RESTful Web Service - Xamarin
Content Source: docs/xamarin-forms/data-cloud/web-services/rest.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @spelltwister
Thanks for pointing this out. I've updated the doc in a PR that will go live today or tomorrow.


LICENSE still has placeholders

Here. I'm no lawtician, but it seems like a good addition to fill these in.

Fixed.


Add the abstract method for Frame.fillna.

Adding the abstract Frame.fillna to clarify there are the methods in the sub classes.

I'd merge this since this is trivial.


Manage update 500 error

InternalError: internal error.
at get (/base/data/home/apps/s~custom-elements/manage:20161107t160919.396880592496182829/src/manage.py:108)

@dstockwell: I tried looking at this. Not sure where its failing etc, maybe its the transaction?

@dstockwell: ping


Petition to block proxi from the issues/PRs since he's spamming my inbox 😠


feat: Pairwise and composition with an equiv

Add a lemma about Pairwise being preserved by composing the arguments of the pairwise relation with an equiv.
lemma EquivLike.pairwise_comp_iff {X : Type*} {Y : Type*} {F} [EquivLike F Y X]
    (f : F) (p : X → X → Prop) : Pairwise (fun y z ↦ p (f y) (f z)) ↔ Pairwise p := by

This depends on Mathlib.Data.FunLike.Equiv and
Mathlib.Logic.Pairwise, neither of which imports the other, so to avoid increasing the imports of very basic files I put it in its own new file, but feel free to suggest a better location of this lemma (and to golf the proof).
From AperiodicMonotilesLean.

To consider in review: do we also want variants for unbundled bijective functions, and one-way implications for unbundled injective and surjective functions and for EmbeddingLike, and where should all those variants go if so?

I've now added the lemma for unbundled functions (in Mathlib.Logic.Pairwise, since they don't have any further dependencies), and one for EmbeddingLike (in the same new file as the EquivLike one; the question about the best locations still applies there, for both lemmas, and quite likely the two belong in different files). I've also used Function.Surjective.forall₂ for golfing as suggested.


Is it possible to return address of local scope variable?

https://github.com/openebs/CITF/blob/80c588255c1a37d0e953b6f530a3ceaa38aff5ac/citf_options/citf_options.go#L13-L21
@a4abhishek scope of the local variable(citfCreateOptions) ends within this function itself.

you can return the local Scope which having the global export ting type
``go
func CreateOptionsIncludeAll(configPath string) *CreateOptions {
var citfCreateOptions CreateOptions
citfCreateOptions.ConfigPath = configPath
citfCreateOptions.EnvironmentInclude = true
}
configPath := CreateOptionsIncludeAll("configPath").ConfigPath
fmt.Println(configPath) ==> you can access the values

@MuthukumarHelios can you refer some online resource to know more about it.


fix(deps): update aws-sdk-go-v2 monorepo

This PR contains the following updates:

Package
Type
Update
Change

github.com/aws/aws-sdk-go-v2
require
patch
v1.32.2 -> v1.32.3

github.com/aws/aws-sdk-go-v2/config
require
patch
v1.28.0 -> v1.28.1

github.com/aws/aws-sdk-go-v2/credentials
require
patch
v1.17.41 -> v1.17.42

github.com/aws/aws-sdk-go-v2/service/ecr
require
patch
v1.36.2 -> v1.36.3

Release Notes

aws/aws-sdk-go-v2 (github.com/aws/aws-sdk-go-v2)
v1.32.3
Compare Source

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

7 additional dependencies were updated

Details:

Package
Change

github.com/aws/aws-sdk-go-v2/feature/ec2/imds
v1.16.17 -> v1.16.18

github.com/aws/aws-sdk-go-v2/internal/configsources
v1.3.21 -> v1.3.22

github.com/aws/aws-sdk-go-v2/internal/endpoints/v2
v2.6.21 -> v2.6.22

github.com/aws/aws-sdk-go-v2/service/internal/presigned-url
v1.12.2 -> v1.12.3

github.com/aws/aws-sdk-go-v2/service/sso
v1.24.2 -> v1.24.3

github.com/aws/aws-sdk-go-v2/service/ssooidc
v1.28.2 -> v1.28.3

github.com/aws/aws-sdk-go-v2/service/sts
v1.32.2 -> v1.32.3


dummy_image in FID doesn't fit into custom feature extractors

🐛 Bug
When a custom feature extractor is passed into FID, a hard-coded dummy_image is used to find the number of features:
https://github.com/Lightning-AI/torchmetrics/blob/3e5a6ea49046d90574ae25151cebfe3b1d96496d/src/torchmetrics/image/fid.py#L298-L301
which will fail if the custom extractor does not accept input with dtype=torch.uint8, shape=(1, 3, 299, 299), and device does not match the custom feature extractor.
To Reproduce
import torchmetrics.image
import torchvision
torchmetrics.image.FrechetInceptionDistance(torchvision.models.resnet50().to('cuda'))

@asy51 mind sharing env detail fro reproducing the issue?

Python 3.10.4
torch 2.0.0+cu177
torchvision 0.15.1+cu117
torchmetrics 1.2.1
torch_fidelity 0.3.0


style: start writing actual stories for button calendar clipboard modal

CDS Report build#223.0 ✘
*

build ✘

CDS Report yarn-build-ui-kit#1461.0 ✘

build

build ✘


feat: add Rewriter

resolve #10

Codecov Report

Merging #16 (ff5fd2c) into main (b1e46b2) will increase coverage by 7.50%.
The diff coverage is 66.90%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
+ Coverage   47.43%   54.94%   +7.50%
==========================================
  Files          18       21       +3
  Lines         957     1092     +135
==========================================
+ Hits          454      600     +146
+ Misses        469      449      -20
- Partials       34       43       +9

Impacted Files
Coverage Δ

internal/discoverer/discoverer.go
74.19% <0.00%> (-14.27%)
:arrow_down:

internal/discoverer/discoverer_mock.go
0.00% <0.00%> (ø)

internal/core/comm/watch.go
55.55% <75.00%> (+55.55%)
:arrow_up:

internal/core/components/rewriter.go
76.47% <76.47%> (ø)

internal/core/entity/filter.go
76.92% <76.92%> (ø)

internal/core/storer/store.go
81.55% <85.71%> (+0.55%)
:arrow_up:

internal/core/comm/utils.go
87.50% <0.00%> (+25.00%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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


bot.on("update") method

There is a bot.on("update") method in Telegraf.js. Some examples:
bot.on("new_chat_members", ctx => ctx.deleteMessage())
this code removes "somebody Joined the group" notification when someone joins group. Another usage is:
bot.on(["new_chat_members", "left_chat_member"], ctx => ctx.deleteMessage())
this code waits for somebody to join or leave the group and when someone leave or join the group it removes the notification.

resolved in https://github.com/denogram/denogram/commit/25765b07a1e3ac49902ea6eab981adc1edc9c1cf


Pagination:fix IE onchange doesn't work Bug

Pagination:fix IE onchange doesn't work Bug
Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.
#6221
#6222
#6224

i change elementui resource in the pagination.js , but also doesn`t work in ie10


Improve view for PageTranslator

Improve auto translate section view

Move styles to css files

Maybe we should add spoiler for options?
This spoiler will remember its state


Backups created by chef-server-ctl backup command cannot be restored after version 12.9.1-1

Description
Backups created by chef-server-ctl backup command cannot be restored after version 12.9.1-1.
Chef Version
12.11.1-1
12.11.0-1
12.10.0-1
Backups created with the latest versions chef-server-core versions cannot be restored.  Chef-server-ctl restore terminates with error message:  "[ERROR] no implicit conversion of nil into String".
Platform Version
Ubuntu 14.04
~$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=14.04
DISTRIB_CODENAME=trusty
DISTRIB_DESCRIPTION="Ubuntu 14.04.4 LTS"
Chef-server-core - standalone version
Replication Case
install chef-server-core with version 12.11.1-1
chef-server-ctl reconfigure
chef-server-ctl backup -y
chef-server-ctl restore -c chef-backup.tgz
Stacktrace
Expanding tarball: /var/opt/chef-backup/chef-backup-2016-11-11-01-06-52.tgz
Restoring Chef Server from backup
Cleaning up any old files
ok: down: bookshelf: 0s, normally up
ok: down: nginx: 0s, normally up
ok: down: oc_bifrost: 1s, normally up
ok: down: oc_id: 0s, normally up
ok: down: opscode-chef-mover: 1s, normally up
ok: down: opscode-erchef: 0s, normally up
ok: down: opscode-expander: 1s, normally up
ok: down: opscode-solr4: 0s, normally up
ok: down: postgresql: 0s, normally up
ok: down: rabbitmq: 1s, normally up
ok: down: redis_lb: 0s, normally up
bookshelf disabled, not stopping
nginx disabled, not stopping
oc_bifrost disabled, not stopping
oc_id disabled, not stopping
opscode-chef-mover disabled, not stopping
opscode-erchef disabled, not stopping
opscode-expander disabled, not stopping
opscode-solr4 disabled, not stopping
postgresql disabled, not stopping
rabbitmq disabled, not stopping
redis_lb disabled, not stopping
Terminating processes running under application users. This will take a few seconds.
Your config files have been backed up to /root/opscode-cleanse-2016-11-11T01:10.
[ERROR] no implicit conversion of nil into String

From what I  saw when testing 12.10, it looks like the backup command is no longer grabbing the /etc directory, just /var. So when it tries to restore there are tons of files missing.

AFAIK there's no support for restoring a backup made by chef-server-ctl backup to a different server version or topology

Correct, however you cannot restore to the same version when /etc/opscode is not backed up.

Are you sure that /etc/opscode isn't being backed up?   The code is a bit opaque, but doesn't appear to have changed much since I last looked at it:  https://github.com/chef/chef_backup/blob/master/lib/chef_backup/strategy/backup/tar.rb#L84

That's what I saw on 12.10 a couple weeks ago. I'll validate it again on this end and report back my findings.

Thanks!   when you do, take a look at the manifest file that gets written out in the tarball for confirmation on what was grabbed.

@irvingpop I can confirm what @wduncanfraser said. On Ubuntu 14.04, backed up from, and restoring with, chef-server-core-12.11.1-1, there is no /etc/opscode present:
ubuntu@ip-172-31-33-44:~$ tar tvf chef-backup-2016-12-05-20-19-43.tgz | wc -l
9528
ubuntu@ip-172-31-33-44:~$ tar tvf chef-backup-2016-12-05-20-19-43.tgz | grep etc
drwxr-x--- opscode/opscode                      0 2016-12-05 19:43 var/opt/opscode/upgrades/etc/
-rw-r--r-- opscode/opscode                    631 2014-12-06 01:07 var/opt/opscode/upgrades/etc/config.rb

I found a similar issue filed a few days ago at https://github.com/chef/chef_backup/issues/26 with the same [ERROR] no implicit conversion of nil into String issue as OP.

👍

Anyone who needs a quick fix on this, see https://github.com/chef/chef_backup/issues/26#issuecomment-268919926

@bdwyertech's Fix was integrated in Chef Server 12.12 so this should now be fixed.  Thanks @bdwyertech !


Web -  Editing and deleting simultaneously ends up in deleted messages showing up again

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a workspace Where User A is an admin
Add User B to the workspace in step 1 and open the same Workspace from User B on another bowser
Send a message from Use B to the workspace.
Open the message from User A in the workspace
Hover on the message from User A & Click on the Delete Icon and leave the pop up confirmation as opened
Hover on the same message from user B and click on edit message, edit the message and leave it unsaved
Go to your network and disable connection (get offline)
From user A screen on the confirmation pop up click on delete
Go to your network and get back online
Now immediately click on save changes from user B edit text field
The deleted message will reappear on both user A and User B screen as edited message

Expected Result:
The message will be deleted
Actual Result:
The deleted message reappears
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/b130e7c2-46b3-425f-bb64-dec19aa652c1
https://github.com/Expensify/App/assets/93399543/a501be7f-23e3-4648-bc19-a26a07245c14
Expensify/Expensify Issue URL:
Issue reported by: @lidiyakelay
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1693253911482689
View all open jobs on GitHub

Couldn't reproduce on v1.3.64-2, closing.

@kbecciv or @garrettmknight  the issue is still occurring in the latest version i have been able to reproduce
https://github.com/Expensify/App/assets/59802743/4abac286-716d-4c42-a7a6-0f76b7d32db5

@kbecciv or @garrettmknight can you please check this


next/previous controls are not working

I have a multi page site that I would like to navigate with the next/previous controls in the page. For unknown reasons, it only works from the chapter to the first page, subsequent pages do not connect to the controls:

What am I doing wrong?
P.S. one can observe the behavior here: https://stillwater-sc.github.io/domain-flow/introduction/

For now, the best way is, to change your layouts/_default/single.html to:
{{- partialCached "page-meta.hugo" . .RelPermalink }}
{{- $outputFormat := partial "output-format.hugo" . }}
{{- partial "output-partial.hugo" (dict "base" "header" "page" . "parameter" . "outputFormat" $outputFormat) }}
{{- if not .File }}
  {{- partial "output-partial.hugo" (dict "base" "body" "page" . "parameter" (dict "page" . "content" (partial "output-partial.hugo" (dict "base" "initial" "page" . "parameter" . "outputFormat" $outputFormat))) "outputFormat" $outputFormat) }}
{{- else }}
    {{ if .Params.WebGLRenderTarget }}
        {{ if .Params.SpacetimeVisualization }}
            {{ partial "spacetime.html" . }}
        {{ end }}
        {{ if .Params.IndexSpaceVisualization }}
            {{ partial "indexspace.html" . }}
        {{ end }}
        {{ if .Params.FreeScheduleVisualization }}
            {{ partial "freeschedule.html" . }}
        {{ end }}
        {{ if .Params.LinearScheduleVisualization }}
            {{ partial "linearschedule.html" . }}
        {{ end }}
        {{ if .Params.WebGLAnimation }}
            {{ partial "webgl.html" . }}
        {{ end }}
    {{ end }}
  {{- partial "output-partial.hugo" (dict "base" "body" "page" . "parameter" (dict "page" . "content" (partial "output-partial.hugo" (dict "base" "content" "page" . "parameter" . "outputFormat" $outputFormat))) "outputFormat" $outputFormat) }}
{{- end }}
{{- partial "output-partial.hugo" (dict "base" "footer" "page" . "parameter" . "outputFormat" $outputFormat) }}

@McShelby that worked:

But it also highlights the fact that I don't understand how the layout/partials anymore. Is layouts/partials/content.html now the default single page layout spec?

For the theme's documentation there is definetly room for improvement on how to customize certain partials.
In your specific case a call to page-meta.hugo at the beginning of your single.html was missing. This is done internally for the shipped partials. Using content.html in your case frees you from that burden and also asserts that you can use the integrated "print chapter" feature, as this collects the content of several pages and puts it into one document.

thank you for thinking ahead on the "print chapter" feature, definitely want that!
Thank you for your insights, this theme is magnificent!


Request for option to display subfolders in left-hand list

What's the Problem?
Unable to display subfolders in the left-hand list
Solution/Idea
Windows Explorer allows subfolders to be displayed in the left-hand column. I rely on that extensively to move files from one place to another. I'm not finding any such option with Files, though.
Alternatives
Windows Explorer
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

This proposal will allow end users to accomplish X
Should

This proposal will allow developers to accomplish Y
Could

This proposal will allow end users to accomplish Z
Won't

Files Version
2.0
Windows Version
10.0.18363.1801
Additional comment
No response

Same as #1928

I understand there is currently no plan to show a tree view in the sidebar. Instead we'll add a TreeView layout in the main pane (#1928).


Online event is misleading - should always poll when "online"

From MDN:

In Chrome and Safari, if the browser is not able to connect to a local area network (LAN) or a router, it is offline; all other conditions return true. So while you can assume that the browser is offline when it returns a false value, you cannot assume that a true value necessarily means that the browser can access the internet.

This is the most comprehensive implementation I've found but it still doesn't account for that fact.
True connectivity checking would:

when "offline" return offline
when "online", still check with ping and return that status

Really there are three statusses: offline | connected | online

closing


What would I change in bootstrap.properties to have values loaded from different consul folder

Hello , As of now the values are loaded from http://localhost:8500/ui/#/dc1/kv/config/ what should I do to have it look at http://localhost:8500/ui/#/dc1/kv/private? Any config property?

spring.cloud.consul.config.prefix=private

Add javadocs to ConsulConfigProperties and call out configuration in docs.


cmake: do not check for Catkin when it is not needed

This is for compilation issues in the CI process.
@wxmerkt Is it fine for you? Could you make your ROS building code compliant this new way of checking for Catkin?

This likely is a regression for us and makes things more complicated.
What is the compilation issue with a quiet find_package? Perhaps we can fix that issue instead.
Practically speaking, we could get rid off this catkin-type search entirely if Pinocchio uses the correct Python package path (i.e., whatever is standard on the distro and not forcing site-packages for all).

Practically speaking, we could get rid off this catkin-type search entirely if Pinocchio uses the correct Python package path (i.e., whatever is standard on the distro and not forcing site-packages for all).

Ok. We can go towards this solution. Could you provide me more details?

That would be ideal - we could totally go away with the catkin-wrappers then by using a simple package.xml and the jrl cmake modules.
The key here is to detect whether the system is debian-flavoured and set PYTHON_DEB_LAYOUT (exposed by jrl cmake modules in [2]) accordingly. [3] was attempting to automatically set it, but got blocked. It is now up to the individual user projects to add that logic (i.e., we'd need here, hpp-fcl, and eigenpy).
References:

https://github.com/jrl-umi3218/jrl-cmakemodules/issues/38, issue discussing incorrect Python-paths on debian-flavoured distributions.
https://github.com/jrl-umi3218/jrl-cmakemodules/pull/39, added Python-install location option
https://github.com/jrl-umi3218/jrl-cmakemodules/pull/47, would activate the correct flag automatically on a debian-based distribution - yet, blocked as it would affect too many downstream and suggested to handle on a per-project basis.
https://github.com/stack-of-tasks/pinocchio/issues/790, an ill-fated work-around suggestion to achieve the same thing.

@wxmerkt Thanks for this quick reply.
I would go towards #790, but the problem is that if catkin is present on the current machine, it means that Pinocchio will be installed in dist-packages while maybe this not intended by the user.
Is there a way to detect that Catkin is trying to build Pinocchio and then set on the respective flags?

For the records:
Traceback (most recent call last):
File "/root/robotpkg/math/py-pinocchio/work.runner-7gqkM7SY-project-5-concurrent-35/pinocchio-2.1.2/catkin_generated/generate_cached_setup.py", line 20, in <module>
from catkin.environment_cache import generate_environment_script
ImportError: No module named catkin.environment_cache
CMake Error at /opt/ros/kinetic/share/catkin/cmake/safe_execute_process.cmake:11 (message):
execute_process(/usr/bin/python2.7
"/root/robotpkg/math/py-pinocchio/work.runner-7gqkM7SY-project-5-concurrent-35/pinocchio-2.1.2/catkin_generated/generate_cached_setup.py")
returned error code 1
Call Stack (most recent call first):
/opt/ros/kinetic/share/catkin/cmake/all.cmake:198 (safe_execute_process)
/opt/ros/kinetic/share/catkin/cmake/catkinConfig.cmake:20 (include)
CMakeLists.txt:93 (find_package)

In https://gepgitlab.laas.fr/stack-of-tasks/pinocchio/-/jobs/21970
I have no time to allow to this right now, sorry.

For the records:
Traceback (most recent call last):
File "/root/robotpkg/math/py-pinocchio/work.runner-7gqkM7SY-project-5-concurrent-35/pinocchio-2.1.2/catkin_generated/generate_cached_setup.py", line 20, in <module>
from catkin.environment_cache import generate_environment_script
ImportError: No module named catkin.environment_cache
CMake Error at /opt/ros/kinetic/share/catkin/cmake/safe_execute_process.cmake:11 (message):
execute_process(/usr/bin/python2.7
"/root/robotpkg/math/py-pinocchio/work.runner-7gqkM7SY-project-5-concurrent-35/pinocchio-2.1.2/catkin_generated/generate_cached_setup.py")
returned error code 1
Call Stack (most recent call first):
/opt/ros/kinetic/share/catkin/cmake/all.cmake:198 (safe_execute_process)
/opt/ros/kinetic/share/catkin/cmake/catkinConfig.cmake:20 (include)
CMakeLists.txt:93 (find_package)

In https://gepgitlab.laas.fr/stack-of-tasks/pinocchio/-/jobs/21970
I have no time to allow to this right now, sorry.

@nim65s I think we need to deactivate the search of catkin under robotpkg. This is easy to do.

I would go towards #790, but the problem is that if catkin is present on the current machine, it means that Pinocchio will be installed in dist-packages while maybe this not intended by the user.

Having thought more about it, I don't like the approach I initially proposed in #790 and the switch should be distro-dependent, not catkin-dependent.
I'd like to remove all catkin-related modifications that are required workarounds from the CMakeLists. The correct way would be to follow the standard of the platform, and only do non-standard paths when absolutely necessary (i.e., if you choose to install always to site-packages in robotpkg, activate that flag only during robotpkg builds).

Is there a way to detect that Catkin is trying to build Pinocchio and then set on the respective flags?

I tried and it got overcomplicated/wasn't possible. Ideally, resolving the Python package path situation would be best and would completely remove all catkin from the CMakeLists (which are a required workaround to deal with non-standard Python package paths).
My hunch to the error above is that Python35 does not have catkin installed - but CMake triggered it and forces alias python=python3. So the failures likely are only on Python3 builds.

(i.e., if you choose to install always to site-packages in robotpkg, activate that flag only during robotpkg builds).

That sounds good to me. I like the rule: if the user did not specify anything, use the target OS settings.
Sadly, the t-OS (target OS), different from the b-OS (build OS) cannot be specified with CMake, can it ?
Does CMake assume that the t-OS is the b-OS ?
I believe the best way to answer the question @wxmerkt raised, and that has been regularly raised, is to find a good solution (and implementation in the JRL cmake module) to the problem:
We need a way of knowing what the target OS is.
Here are some suggestions:

assume the t-OS is the b-OS, except if a particular CMake variable set the t-OS to something else.
do target specific choices only if a CMake variable is set (otherwise, do not do any specific, not even for b-OS).

Maybe the good solution is to rely on CMAKE_SYSTEM_NAME: https://gitlab.kitware.com/cmake/community/wikis/doc/cmake/CrossCompiling
This requires us to write a Platform/${CMAKE_SYSTEM_NAME}.cmake for each platform we want to support. I believe this is a hack which may break things. Maybe a PROJECT_SYSTEM_NAME, with a file in the JRL cmake module, is better.
Any thoughts or suggestions ?

Hi @jcarpent, sorry I have been out of the loop, what is the outcome of your conversations/discussions to package according to platform-standard unless deactivated?

@jcarpent I will provide a new PR for that.
So, the idea is to make Pinocchio looking for the current OS policy as you mentioned with @jmirabel.
We will force the flag if needed when using robotpkg.
Is it fine for you?

I would even suggest to use Python directly to set PYTHON_PACKAGES_DIR. What do you think?

Thank you very much for the change to cmake upstream, I made a comment there :-)

With the update on the jrl-cmake-modules, @wxmerkt do you think we can now remove the references to Catkin?

Yes, I will make a series of pull requests to adjust. I want to first do a release to the ROS buildfarm of those so we can then adjust the dependent packages (but it will be a breaking changing of that version for some). E.g., eigenpy is currently (but will be removed because of the packaging issues) part of MoveIt and we also have dependent packages.
Is it okay if I make the PR to remove it in a few days? And can we tag new releases with the updated cmake-module synced into hpp-fcl, eigenpy, and pinocchio?

Ok. I can make the PR for Pinocchio if you want?

with @lesteve, we will work soon on the packaging of Pinocchio inside anaconda. So we need to proceed as follows for eigenpy too.

If you can sync the cmake submodule and tag releases, I will test the new ROS packaging (without the catkin-cmake wrapper) and test its impact

Actually fine to remove catkin right now, we have released, tagged versions

If you can sync the cmake submodule and tag releases, I will test the new ROS packaging (without the catkin-cmake wrapper) and test its impact

Ok sure. I've made a first proposal in #826.


About download the models

Maybe you could provide download links for these models?
I only found some of them on the official website https://github.com/tensorflow/models/tree/master/research/slim.
Especially those adversarial models. Thank you very much!
--checkpoint_path_inception_v3=inception_v3.ckpt
--checkpoint_path_adv_inception_v3=adv_inception_v3_rename.ckpt
--checkpoint_path_ens3_adv_inception_v3=ens3_adv_inception_v3_rename.ckpt
--checkpoint_path_ens4_adv_inception_v3=ens4_adv_inception_v3_rename.ckpt
--checkpoint_path_inception_v4=inception_v4.ckpt
--checkpoint_path_inception_resnet_v2=inception_resnet_v2_2016_08_30.ckpt
--checkpoint_path_ens_adv_inception_resnet_v2=ens_adv_inception_resnet_v2_rename.ckpt
--checkpoint_path_resnet=resnet_v2_101.ckpt \

The models can be downloaded here

Oh, thanks!!


Web Cmdlets Session Cookies not working on macOS

Reported to me this week at the PowerShell summit was that the Cookies are not working on macOS when passing a session object.
I have yet to get a repro or the exact details of the issue, but I'm documenting it here for further investigation.

Not sure how useful it'll be, but since I'm here anyway, this might come in handy (aka "it's dangerous to go alone, take this"):
function Get-WebSessionCookies {
    [CmdletBinding()]
    param(
        [Parameter(Position = 0, Mandatory, ValueFromPipeline)]
        [Alias('Session', 'InputObject')]
        [ValidateNotNull()]
        [Microsoft.PowerShell.Commands.WebRequestSession]
        $WebRequestSession
    )
    begin {}
    process {
        $CookieContainer = $WebRequestSession.Cookies
        try {
            [hashtable] $Table = $CookieContainer.GetType().InvokeMember("m_domainTable",
                [System.Reflection.BindingFlags]::NonPublic -bor
                [System.Reflection.BindingFlags]::GetField -bor
                [System.Reflection.BindingFlags]::Instance,
                $null,
                $CookieContainer,
                @()
            )
            Write-Output $Table.Values.Values
        }
        catch {
            $PSCmdlet.ThrowTerminatingError($_)
        }
    }
    end {}
}

(Because it's low-key a bit of a pain to pull all the cookie data from a websession at times). :)

I was able to reproduce this with Invoke-RestMethod, but not Invoke-WebRequest.  It appears that Invoke-RestMethod doesn't play well with object scopes or something, as the session variable is always null unless I'm referencing it from within the same function.
My code is a bit more involved than the above, as I wanted to make the actual REST API request into a reusable function, so I'm passing in headers, body, etc. to that function, and returning a hashtable with both the session and response objects.  If you are running code all in one shot, it seems to work.
Like, this works:
`
$session = $null
$response = Invoke-RestMethod -Uri 'http://www.google.com' -method 'GET' -SessionVariable session
$session.cookies
Capacity Count MaxCookieSize PerDomainCapacity

 300     2          4096                20

`
However, creating a PowerShell script and directly calling functions in the script to do this, will not.
The resulting object you'd return from a function containing Invoke-RestMethod does not have a session object with cookies in it.
Obviously the workaround to this would be to use Invoke-WebRequest and then process the response body for JSON/XML, but it's not simple as Invoke-RestMethod automatically doing that.

Is this still an issue in the latest Powershell version?


Add "Silent All Notifications" Settings.

Add "Silent All Notifications" Settings pls. Or just in zen mode. I know the "Zen Mode: Silent Notifications" setting, but error notifications will still pop out. So I want the feature that can silent all notifications in zen mode even error notifications will not pop out. Thank you🙏

/duplicate https://github.com/microsoft/vscode/issues/144324


Web API - reshoot postman  Key -> ID

Key was replaced by ID, need to re-shoot.
also a bunch of the JSON code has key. Search of Key and check all the images.
related #3463

fixed.


[network] added inbound/outbound connected peers dashboard

This will allow us to see if certain nodes won't allow inbound
connections due to improper configurations vs outbound connections.
TBD run cluster test to verify the dashboard

@bors-libra r+

:pushpin: Commit a4f9eba has been approved by davidiw

:hourglass: Testing commit a4f9ebafb2c6964edd3d71b29e3dfa373517930a with merge c08c8e875296b848f10cd72fd34f74d7bd153a82...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing c08c8e875296b848f10cd72fd34f74d7bd153a82 to master...


Move generating webhook certificate to operator

Description
Changes proposed in this pull request:

Generate web hook certificate and patch configuration in telemetry-manager
Remove obsolete make targets

Related issue(s)

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

/hold

/retest

/re-test

/unhold

/retest

/retest


set up requested mimetype of response

set up the requested format as response.outputs[name].data_format
if the requested format is not in response.outputs[name].supported_formats - raise InvalidParameterValue error
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[ ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage increased (+0.7%) to 72.57% when pulling c63b1c491a52f2e93bec22df59eb1ef67327eddd on eran-pinhas:master into d52910cf5022f188a99d6a27b650791a57691749 on geopython:master.

@cehbrecht - not an enhancement, it's bug (unless i got it all wrong).
when the request specifies strictly what response mimetype it is expecting - the handler's response object is not being effected and therefore the handler is unable to answer with the right data

btw, why does the coveralls check fails?

@eran-pinhas

btw, why does the coveralls check fails?

I suppose it needs to be configured to be less restrictive on the expected coverage. I would ignore it for now.
@jachym Can you check the coverage config?

I have tested this PR with my Emu service using the following request:
http://localhost:5000/wps?version=1.0.0&service=WPS&request=Execute&identifier=wordcounter&DataInputs=text=@xlink:href=http://bird-house.github.io&ResponseDocument=output=@asReference=True@mimetype=application/json
When I choose an invalid like mimetype=image/png I get with PR an exception (as expected):
MimeType image/png not valid
It looks like the mimetype attribute is not case-insensitive ... probably another issue.

@eran-pinhas Thanks for fixing this. Maybe you could also extend the test-suite for this case? Otherwise I will pull it in.

It looks like the mimetype attribute is not case-insensitive ... probably another issue.

I don't understand, what is the issue?

if you use mimeType instead of mimetype in the above request to the wordcounter process then the mimetype is not recognized. It is not related to this PR ... just noticed while testing.

Done

yes it is sound simpler, thank you


S2U-38 <IP_ADDRESS> Resources: Add 'Download ZIP' event when downloading z…

…ip files from resources

Could you please read the Jira? The acceptance criteria has been clarified.


Make properties correspond

designDoc and designDocs do not correspond, this way it should work

@ssured Thank you very much for the PR! I will be working on a new release over the next couple weeks and will be handling all pull requests then.

I checked the code and designDocs is correctly plural because it is an Array. So instead I will replace the singular version with the plural version. Thank you very much for making a pull request to point out this issue.


Click to download files on summary page

Description
Applicant can click file names to download files on the North Star summary page.
Checklist
General
Read the full guidelines for PRs here

[x] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[x] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[x] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[x] Removed the release notes section if the title is sufficient for the release notes description, or put more details in that section.
[x] Created unit and/or browser tests which fail without the change (if possible) (Better browser tests are blocked by #8143)
[x] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[x] Extended the README / documentation, if necessary. For user-facing features, consider updating the user docs. For "under-the-hood" changes or things more relevant to developers, consider updating the dev wiki.

User visible changes

[x] Followed steps to internationalize any new strings

[x] Added context strings to new messages
[x] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[x] Wrote browser tests using the validateAccessibility method
[x] Made equivalent changes in Thymeleaf for applicant-facing features and changes (or created an issue in the Northstar epic to track that it's missing from Thymeleaf)
[x] Tested on mobile view. See mobile device mode
[x] Manually tested at 200% size
[x] Manually evaluated tab order
[x] Manually tested with a screen reader if the feature is applicant-facing. See screen reader testing

Issue(s) this completes
Fixes #8985

#9105  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ssandbekkhaug and the rest of your teammates on  Graphite

Exciting update! Can you add tests to cover this new functionality in northstar?

Unfortunately not yet. Uploading files in browser tests in blocked by #8143


Helm Common Values

We should audit and support the common helm values other controllers support.

Going to pick this one up next since it should help test the helm chart release automation.


Whitelist anchors for updates creation

Whitelisted anchors:

a
h1
h2
strong
img
ul
li

Fix for https://github.com/opencollective/frontend/issues/462

Thanks for creating a PR! :clap: The code looks good. I'll test it out locally and see if we have any test coverage we can add to that function.

@Betree Thank you, I'll check the DEBUG=email issue.

@Betree strip_tags appears to called in the UpdateType (/server/graphql/types.js) and parses the markdown text before being handled by the createUpdate function, so the h1 and h2 tags are stripped before your change.
To fix that issue, I think it's fine to modify the strip_tags function to add h1 and h2 as default allowed tags. That change should also allow those tags after editing an update as well.

@Betree sorry about the delayed response. Those should allowed by default in for the sanitize-html package (https://www.npmjs.com/package/sanitize-html#what-are-the-default-options).

@Betree any updates on this? Can you help with anything?

Hi @HipsterBrown, I'm still unable to connect with an account in dev - event with DEBUG=email. Maybe I missed something. I'd like to visually confirm that it works before pushing :wink:

@Betree Makes sense. 😄 Did you set the DEBUG environment variable in the docker-compose.yml, or somewhere else?

I just retried with setting DEBUG in the docker-compose.yml and it worked, I probably missed something the other day :wink:
It now works as intended :

Anyway the short resume from collective's homepage doesn't look very nice. Should we open another issue for this?

@Betree Great work!

Anyway the short resume from collective's homepage doesn't look very nice. Should we open another issue for this?

Yes, can you create a separate issue for this bug? Then we can merge your work. :smile:
Do you have a Twitter we can mention when we thank you for this fix?

New issue in #1327. My twitter handle is @Betree83 :wink:


Gate.control() method rasies the error 'Cannot make controlled gate with id instruction'

Information

Qiskit Terra version:
0.12.0
Python version:
Python 3.7.3
Operating system:
WIN10

What is the current behavior?
Converting circuit into gate using the QuantumCircuit.to_gate property, and then applying the Gate.control method raises the error 'Cannot make controlled gate with id instruction'. This actually means that you cannot create a control gate if you have an id instruction in your circuit. However, when the circuit with no IdGate is used, the Gate.control work just fine.
Steps to reproduce the problem
CIRCF_2.iso(B2, [0, 1, 2, 3],[]) # works fine
GATEB2=transpile(CIRCF_2, basis_gates=['u1', 'u2', 'u3', 'cx', 'id']).to_gate() # works fine
CGA=GATEB2.control(1) # crashes with error 'Cannot make controlled gate with id instruction'

What is the expected behavior?
The Identity gate IdGate should not compromise the control instruction.
Suggested solutions

I believe this should be resolved by pr #3950.

Update for Santa:elf:- QGoGP
:no_entry_sign: I am not sure what isometry "B2" refers to but I tried out an arbitary isometry and it seems that the issue is fixed. Have attached the code I used and screenshots for the same! :>
Screenshot 1 -
from qiskit import QuantumCircuit, QuantumRegister, execute, Aer, transpile

circuit = QuantumCircuit(3)
circuit.iso([[0,1],[1,0]],[1],[]) # works fine
GATEB2=transpile(circuit, basis_gates=['u1', 'u2', 'u3', 'cx', 'id']).to_gate() # works fine
CGA=GATEB2.control(1) # works fine too!

test_circuit = QuantumCircuit(4)
test_circuit.append(CGA,[0,1,2,3])
test_circuit.draw("mpl")

Will update comment/add more comments if cause is found/fixed!

Python version 3.9.7
qiskit-terra version: 0.19.1

Funnily enough, this one is essentially the same issue as #7399.

Nice! I just confirmed that #7403 fixes this issue.

This was fixed by a combination of #7146 and #10607.


Not responsive

Describe the bug
A clear and concise description of what the bug is.
website losses it's responsiveness in small devices whole screen is not visible easily it is zoomed.
To Reproduce
open website in mobile  devices you see the error
Expected behavior
A clear and concise description of what you expected to happen.
website content should fit viewport as we open it different devices.
Screenshots
If applicable, add screenshots to help explain your problem.

in these screenshots you see elements are cropped.
Additional context
Add any other context about the problem here.
@Priyaaa1   I can fix all these issues i have good knowledge  of media queries and responsive design,
assign this issue to me under GSSoC'24

I am interested to contribute in this section, Please assign me the issue.

I agree to follow this project's Code of Conduct
I'm a GSSOC'24 contributor
I want to work on this issue


-a flag and all: true are ignored in v13.0.0

Expected Behavior
When running nyc with all: true in the nyc configuration all files should be included in coverage
This issue appears if all: true is removed from the package.json config and the -a flag is used in the npm test script.
Observed Behavior
Only tested files are included
Bonus Points! Code (or Repository) that Reproduces Issue
https://github.com/ajb3ck/nyc-test
Forensic Information
Operating System:  macOS High Sierra
Environment Information: https://gist.github.com/ajb3ck/4c1a1147df03c9633ec798ed26bb5a1d
nyc version: 13.0.0
npm test script: cross-env NODE_ENV=test nyc mocha
nyc config in package.json:
"nyc": {
    "include": [
      "src/**/*.js"
    ],
    "require": [
      "babel-register"
    ],
    "reporter": [
      "text",
      "lcov"
    ],
    "all": true,
    "sourceMap": false,
    "instrument": false
  }

It appears this issue is due to mixing of babel 6 and babel 7.  Your repository is using babel 6 (and babel-plugin-istanbul@4).  NYC uses babel 7.  I was able to get your repository working by upgrading babel dependencies to v7 and upgrading babel-plugin-istanbul to v5.  So I'd recommend sticking with nyc 12 until you are ready to upgradfe to babel 7.

Thanks for the issue @ajb3ck. Since we haven't heard back I'm going to close this issue for now, but feel free to reopen it once you've tried Corey's suggestion.


☆(𝚉𝚆𝚊𝚝𝚌𝚑≦𝚅𝙸𝙳𝙴𝙾𝚂•FULL) Elif Karaarslan Viral ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ Now Trending On Social Media X Twitter #150

10 sec ago Elif Karaarslan ʟᴇᴀᴋᴇᴅ ᴠɪᴅᴇᴏ Viral On Social Media. ☆(𝚉𝚆𝚊𝚝𝚌𝚑≦𝚅𝙸𝙳𝙴𝙾𝚂•FULL) Elif Karaarslan Viral ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ Now Trending On Social Media X Twitter #150
𝖶𝖺𝗍𝖼𝗁 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤►𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
Elif Karaarslan's latest video has taken the internet by storm, sparking lively discussions across various social media platforms. The rising digital influencer, known for her captivating content—particularly her recent Spiderman-themed video—has experienced a tremendous surge in popularity, with fans eagerly awaiting the full version. Audiences are flocking to Instagram, X (formerly Twitter), and Discord to join the conversation and engage with her latest creation.
As the excitement continues to build, fans are actively sharing their thoughts online, underscoring the pivotal role social media plays in propelling creators like Elif Karaarslan into the limelight. With her distinctive style and creative approach, Nila has made a significant impact on the digital scene, leaving her audience eager to see what she will do next.  The rising digital influencer, known for her captivating content—particularly her recent Spiderman-themed video—has experienced a tremendous surge in popularity, with fans eagerly awaiting the full version. Audiences are flocking to Instagram, X (formerly Twitter), and Discord to join the conversation and engage with her latest creation.
As the excitement continues to build, fans are actively sharing their thoughts online, underscoring the pivotal role social media plays in propelling creators like Elif Karaarslan into the limelight. With her distinctive style and creative approach, Nila has made a significant impact on the digital scene, leaving her audience eager to see what she will do next.


Make the backend work inside a Docker container

Background
Due to complex nature of my Python projects that are often deployed in the cloud and involve external non-Python dependencies I usually prefer to isolate my environments using Docker rather than through Python's native mechanism of virtual environments.
Issue
The backend doesn't really work inside a Docker container. I tried to add kitty as a dependency (which I'd try to avoid regardless) but it didn't really help.
Solution
Use kitty's native protocol instead of kitty +kitten icat to make it work both inside and outside Docker containers.

Adding native protocol support was initially planned, but I ended up never really working on it, mainly because using the icat kitten works well enough for local workflows, and because I had some concerns about how stable the image protocol would end up being. Stability has turned out fine however, and I'd definitely prefer using the native protocol preferable to shelling out to the icat kitten, particularly in a usecase like yours. So it's pretty cool that you worked on integrating this.


Fix table/list parsing ambiguity

Fix parsing of
a | b
- | -
1 | 2

Markdown-it parses it as a table, but it should not be one, because there is a list on the second line that intercepts the table.
The GFM spec says:

A leading and trailing pipe is also recommended for clarity of reading, and if there’s otherwise parsing ambiguity.

Also GitHub's own parser parses this as a list, not a table.

This actually introduces a regression for everybody who was writing their table without a leading pipes. See here for example. Is this change really necessary? If there's a parsing ambiguity, then this ambiguity still exists after this change, and the only difference is that it broke a lot of existing Markdown documents.

@laurent22 answered in https://github.com/laurent22/joplin/issues/7858#issuecomment-1563853366


Calling rehash is required after installing new python version

I'm running on OS X and am observing that calling rehash is required 100% of the time after a new version of Python is installed with pyenv. My impression is that shouldn't be needed. Is this expected behavior?
$ python -V
Python 2.7.13
$ pyenv install 3.6.0
Installing Python-3.6.0...
Installed Python-3.6.0 to /Users/dan/.pyenv/versions/3.6.0

$ pyenv global 3.6.0
$ python -V
Python 2.7.13
$ pyenv rehash
$ python -V
Python 3.6.0

I see a similar issue after uninstalling a version. This is run immediately after the above commands:
$ pyenv uninstall 3.6.0
pyenv: remove /Users/dan/.pyenv/versions/3.6.0? y
$ python -V
bash: /Users/dan/.pyenv/shims/python: No such file or directory
$ pyenv versions
pyenv: version `3.6.0' is not installed (set by /Users/dan/.pyenv/version)
  system
$ pyenv rehash
$ python -V
Python 2.7.13

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/40889723-calling-rehash-is-required-after-installing-new-python-version?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

Both install/uninstall are implemented to call pyenv-rehash automatically.
https://github.com/yyuu/pyenv/blob/v1.0.6/plugins/python-build/bin/pyenv-install#L276
https://github.com/yyuu/pyenv/blob/v1.0.6/plugins/python-build/bin/pyenv-uninstall#L83

Yeah, I see that but it didn't really help me in the case above. Maybe it would make sense to do it any time the environment changes?


Use FirstLevelParseJSON, but the return data always contains metadata, response_errors, attributes and etc.

Hi sir,
When i use the latest version of her on my rails project,  i follow the document to set up and use her. Now i have two questions about the return value from her.
Background:
OS: Mac
gem 'rails', '4.1.5'
gem 'rails-api'
Example:
I have an Affiliate model and initialize her like:
Her::API.setup url: 'http://localhost:3000' do |c|
  # Request
  c.use Faraday::Request::UrlEncoded

  # Response
  c.use Her::Middleware::DefaultParseJSON

  # Loger

  # Adapter
  c.use Faraday::Adapter::NetHttp
end

And in the controller, the code is
affiliate = Affiliate.find(9991104)
render json: affiliate

And get a result like this:
{
metadata: { },
response_errors: { },
destroyed: false,
attributes: {
name: "1",
affiliate_code: "1",
affiliate_url: "1",
description: "1",
is_deleted: true,
created_date: "2013-12-17T07:34:00.000+00:00",
created_by: "shuang",
modified_date: "2015-11-06T02:50:27.567+00:00",
id: 9991104
},
changed_attributes: { }
}

So how can I get the response only include attributes like
{
name: "1",
affiliate_code: "1",
affiliate_url: "1",
description: "1",
is_deleted: true,
created_date: "2013-12-17T07:34:00.000+00:00",
created_by: "shuang",
modified_date: "2015-11-06T02:50:27.567+00:00",
id: 9991104
}

And another question is when i call Affiliate.all in controller
affiliates = Affiliate.all
render json: affiliates

And the result is
{
message: "stack level too deep"
}

Dose anyone encounter the same problem like me?
Thanks for all of you help.

# config/initializers/her.rb
Her::Model.class_eval do
  def as_json(options = nil)
  attributes.as_json(options)
  end
end

# controllers - no change here in theory
affiliate = Affiliate.find(9991104)
render json: affiliate


DELETE returns error 405 method not allowed

I'm following this tutorial on Visual Studio (8.8.10) on my Mac (11.2 , Apple M1) but I have get the error 405 method not allowed when I try to use the DELETE method.

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

ID: 583ae3e7-5915-f22b-e54c-6c100cf2a7da
Version Independent ID: 95f2549e-29f9-e31a-b4d0-432b971643ff
Content: Tutorial: Create a web API with ASP.NET Core
Content Source: aspnetcore/tutorials/first-web-api.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter

Potential bugs in the ASP.NET Core framework and product feedback can be reported at dotnet/aspnetcore issues (include full repro steps and cross-link to a repro project in GH when appropriate). For Visual Studio, use the Report a Problem gesture or Suggest a Feature gesture within VS, which open internal issues for the VS team. For Visual Studio Code, you can open an issue at microsoft/vscode issues.


Change daemonset imagePullPolicy from Never to IfNotPreset

With image imagePullPolicy set to IfNotPreset the user doesn't need to pull the image manually

@zshi-redhat @ahalim-intel Can you please review the PR?


Back button is broken

If you navigate away from Stack#show and then click the back button, you are not returned to Stack#show.

I am seeing this behavior again on master (as of 585a3d3258c0b20ca431bb8bfd5a747affc4810c).
Example of broken behavior:

visit /stacks/:id/apps
click an app -> /apps/:app_id/services
click back button -> same page (/apps/:app_id/services)

Confirmed in Chrome and FF

The above comment no longer breaks for me, but this is another broken case:

visit /stacks/:id/apps
click on an app that has never been deployed ( -> /apps/:app_id/deploy)
click back button -> ends up on same page

Right. After diving into this, we determined the issue is resolved in Ember 1.11 (it seems to have been a bug in 1.10).
#155 updates us to Ember 1.11. Closing this.


Programatically toggle a toggle

I'm working in an integration with my ember.js application. As such I need to code the variable bindings in both ways.
The DOM to Model part i accomplished through the callbacks illustrated in the docs and #23.
But, if the the toogle's model changes in the app, I need to trigger a change in the DOM as well.
I created 3 dependent toggles, i.e all of them are observing a single variable. That mean that if I trigger one, the others should trigger as well, because their model changed.
When the model changes, right now I'm just adding or removing the active css class. Is this the correct way?
That seems to work for the green color of the toggle. And, if I toggle only one of the toggles, it works.
However, after I toggle other buttons, the handles start not moving and I get to states like this:

What I did to get to that state was:

Toggled on 1st toggle, all got correctly on
Toggle off 2nd toggle, and the 1st handle didn't move but changed color

Sorry for the specifics, but maybe these toggles depend on something they shouldn't. I'll keep the debugging meanwhile.
Maybe touchend isn't triggering on the other toggles?

Hello, the solution is to add the declaration !Important in CSS as these dynamically adding a class to a DOM element
CSS
.toggle.active .toggle-handle {
border-color: #5cb85c;
-webkit-transform: translate3d(44px, 0, 0) !important;;
-ms-transform: translate3d(44px, 0, 0) !important;;
transform: translate3d(44px, 0, 0) !important;
}
JS
element.classList.toggle('active');


Cleaning file system

Could you create public method in FakeSftpServerRule for clean file system ?
Example of code

 public void clearFileSystem() {
        Iterable<Path> rootDirectories = fileSystem.getRootDirectories();
        if (rootDirectories != null) {
            stream(rootDirectories.spliterator(), false)
                    .forEach(path -> {
                        try {
                            walkFileTree(path, new SimpleFileVisitor<Path>() {

                                @Override
                                public FileVisitResult visitFile(Path file,
                                                                 BasicFileAttributes attrs) throws IOException {
                                    deleteContent(file);
                                    return CONTINUE;
                                }

                                private void deleteContent(Path file) throws IOException {
                                    if (file.getNameCount() != 0) {
                                        Files.delete(file);
                                    }
                                }

                                @Override
                                public FileVisitResult postVisitDirectory(Path dir, IOException exc)
                                        throws IOException {
                                    deleteContent(dir);
                                    return CONTINUE;
                                }
                            });
                        } catch (IOException e) {
                            log.error("Error during delete content", e);
                        }
                    });
        }
    }

Hi @OArtyomov,
thanks for the feature request. Can you explain your use case, please. I like to understand why this method is needed.