[BUG] cni_compatible fails when podman is being used instead of docker

Describe the bug
The cni_compatible test depends on docker and kind to run the test, locally on the client running the testsuite. Currently is seems kind will not create clusters when a system is using podman (RHEL 8 OS installs podman by default instead of Docker).
When podman is installed and has docker wrapper commands, you may see the following errors when manually creating a kind cluster:
# kind create cluster
enabling experimental podman provider
Creating cluster "kind" ...
 ✓ Ensuring node image (kindest/node:v1.24.0) 🖼
 ✗ Preparing nodes 📦
ERROR: failed to create cluster: command "podman run --name kind-control-plane --hostname kind-control-plane --label io.x-k8s.kind.role=control-plane --privileged --tmpfs /tmp --tmpfs /run --volume
450d0455de77541f0eccfd3c73f462b272517ccf43255407c3d154aa42841098:/var:suid,exec,dev --volume /lib/modules:/lib/modules:ro -e KIND_EXPERIMENTAL_CONTAINERD_SNAPSHOTTER --detach --tty --net kind --lab$
l io.x-k8s.kind.cluster=kind -e container=podman --publish=<IP_ADDRESS>:41533:6443/tcp -e KUBECONFIG=/etc/kubernetes/admin.conf docker.io/kindest/node@sha256:0866296e693efe1fed79d5e6c7af8df71fc73ae45e$
679af05342239cdc5bc8e" failed with error: exit status 126
Command Output: time="2022-08-25T10:04:26-04:00" level=warning msg="Failed to load cached network config: network kind not found in CNI cache, falling back to loading network kind from disk"
time="2022-08-25T10:04:26-04:00" level=warning msg="1 error occurred:\n\t* plugin type=\"bridge\" failed (delete): cni plugin bridge failed: running [/usr/sbin/iptables -t nat -D POSTROUTING -s 10.$
9.0.6 -j CNI-03a5bee804494f7000307447 -m comment --comment name: \"kind\" id: \"7550fdb452c5283a6308053918658d44ad43d931e661412e81b4facff16ebfb3\" --wait]: exit status 2: iptables v1.8.4 (nf_tables$: Chain 'CNI-03a5bee804494f7000307447' does not exist\nTry `iptables -h' or 'iptables --help' for more information.\n\n\n"
Error: plugin type="bridge" failed (add): cni plugin bridge failed: failed to set bridge addr: could not add IP address to "cni-podman1": permission denied

To Reproduce
Install cnf-testsuite on a client running podman and attempt to run the cni_compatible test.
Expected behavior
We should only support official docker so our checks for docker installed should verify it's not podman posing as docker and skip the test, perhaps with a message claiming the test requires official docker to run.
Other Possible Solutions

Troubleshoot kind working with podman to officially support podman for this test
Change the cni_compatible to test the CNF on a docker image that is loaded onto their k8s cluster that can support kind clusters rather than installing kind cluster on their local client machine

Work around solution on RHEL:
dnf remove podman -y
dnf config-manager --add-repo=https://download.docker.com/linux/centos/docker-ce.repo

Now show the versions of docker available to choose which one you want to install:
dnf list docker-ce --showduplicates | sort -r

Now install the version, enable and start docker:
dnf install docker-ce-3:20.10.17-3.el8
systemctl enable docker
systemctl start docker

You can verify docker is running doing docker ps and then test creating a kind cluster with: kind create cluster and it should not fail after configuring the nodes.
Now you can run the cni_compatible test.

https://github.com/cncf/cnf-testsuite/wiki/CNF-Testsuite-Source-Installation-for-Red-Hat-Enterprise-Linux-8

Proposed solution: Skip the test if docker is not found for this test to run.
kind seems to have some issue creating clusters with podman.

Setup the testsuite on RHEL8 with docker installed and attempted to run the cni_compatible test (notes below).
Notes on the issue

When podman is installed, the kind cluster creation fails.

This means the setup cannot be setup on the machine (because we cannot set KUBECONFIG without a kind cluster).

When podman is uninstalled and docker is installed, then the kind cluster creation works as expected.

This also results in the testsuite running as expected when cni_compatible test is run.
The cni_compatible test already checks for docker version. If it does not exist, then the test would be skipped.

@agentpoyo @lixuna:
For this ticket, I don't think we have to make any changes to the codebase. It would be useful if we document anywhere that kind has issues with podman. So anyone on RHEL8 should use Docker instead of podman. @agentpoyo's docker installation instructions on this wiki page can act as reference.

@agentpoyo @lixuna: For this ticket, I don't think we have to make any changes to the codebase. It would be useful if we document anywhere that kind has issues with podman. So anyone on RHEL8 should use Docker instead of podman. @agentpoyo's docker installation instructions on this wiki page can act as reference.

I think if podman is installed and docker isn't, it should SKIP the test and not FAIL the test. @HashNuke @lixuna @denverwilliams @wavell @taylor Thoughts?

@agentpoyo

I think if podman is installed and docker isn't, it should SKIP the test and not FAIL the test

I agree with your opinion. And that is the current behaviour (ref). If docker is not installed, the test is skipped.

[Update] Had a conversation with @agentpoyo about this. We will try to reproduce the issue again and then post an update on this ticket.

hi @HashNuke and @agentpoyo is this issue still relevant to cnti?

Yes, this is likely still relevant.

Notes for contributors:
In the comments above, I was recommending documenting that we do not support podman in the interest of time/priority. But the ideal solution for this issue would be to support podman.

Still valid, "cni_compatible" skips when docker is not installed. INSTALL.md contains docker as a prerequisite specifically for this test.


Site credits update

added Al to product team
moved Sarah to past team
changed job title for Ginger

coverage: 33.231%. remained the same when pulling f0db34d87e88596015f4bc2396c20939f6492168 on ginger/credits into 628da407d1c08bb1e4e531a40befe61051483785 on dev.


support custom analyzer

#12
added a custom widget class to support custom analyzer, which looks quite primitive
(have corrected the typo in the following picture)

@luceos, could you review please?

I'd rather see the dropdown completely removed to make it a free input type text one. This custom component, including its design (sorry), will not make things simpler for people. If you are able to take care of that, then obviously I would have no issue merging.

I'd rather see the dropdown completely removed to make it a free input type text one. This custom component, including its design (sorry), will not make things simpler for people. If you are able to take care of that, then obviously I would have no issue merging.

I thought the dropdown is designed for a reason so I decide to keep it and allow custom string at the same time. As you say so, things have become quite simple.

Idea: A simpler way is to add a text field named Custom analyzer and add a help message that if the field is not empty, the custom analyzer will be used instead of the selected language,
https://github.com/blomstra/flarum-ext-search/blob/f981c27e00b943335eb8d2c30827ff7eb14576f9/src/Commands/BuildCommand.php#L94
And in PHP something like this:
- 'type' => $settings->get('blomstra-search.analyzer-language') ?: 'english',
+ 'type' => $settings->get('blomstra-search.custom-analyzer') ?: $settings->get('blomstra-search.analyzer-language') ?: 'english',

Idea: A simpler way is to add a text field named Custom analyzer and add a help message that if the field is not empty, the custom analyzer will be used instead of the selected language.
implemented in frontend

@rafaucau  @luceos

Although I appreciate the effort and the flexibility of listening to the ideas provided, I do think that multiple controls to manage one setting is going to cause issues.

Using only a text field and linking to an article that shows the most common analyzers; this is necessary so that people can see their own locale/language pack reflected in a list.
Showing the dropdown, but adding an option "Custom" that, once selected, will make the Custom analyzer visible.

In terms of UX similarity to Flarum option 2 would be best. In terms of simplicity option 1 would be best.
The current solution however, will guarantee cause confusion and as such raise support workload, so I'm not comfortable merging it in the current state.


Alchemist stuck on "Loading..."

Hello,
I'm trying to use the development version of Alchemist. I've enabled Developer Mode in Photoshop 23.1.1 for Windows 11. I downloaded the alchemist-master.zip. I'm installing it using the uxp folder's manifest.json file. When I load the UXP from the Adobe UXP Developer Tool, the plugin stops at "Loading...".

This is the App Log:
[2022-02-10 08:31:56:820] [console] Templates are not initialized yet!
[2022-02-10 08:31:58:465] [console] Rules are not initialized yet!
[2022-02-10 08:31:58:465] [console] Trying to run predict when model was not loaded.```

The issue was I was installing the plugin from the uxp folder rather than the dist folder. I believe that this should be documented in the readme.md file. Thank you.


Translate function on images behaving differently between C and Javascript versions (and I think the JS one is incorrect)

When I've got an image and I call x.translate[] on it passing in non-integer values e.g. x.translate[(1.5,2)] the native version of Decker still does the translation, rounding the decimal values to the nearest integer. But the web version blanks out the image when this happens. I'm guessing the behaviour in the native version is probably what should be correct?
Have tested in current versions of both Firefox and Chromium.

You're correct; the intended behavior is an implicit floor of the offset pair, as there isn't really a sensible way to perform sub-pixel translation. Thanks for the report!


isNotNull may trigger hit-index in any condition

I find in most cases, a condition will be added a isNotNull filter. But maybe the original filter is not suitable for index.
for Example.
string a;  select  xxx from xxx where a > 12;
the SQL's corresponding Spark-Plan:  isNotNull(a) && cast(a#3 as double) > 12. Cast will be filtered, and isNotNull will trigger index. it seems unreasonable.

Is this a bitmap index??
In theory, statistics check should skip IsNotNull as it has 100% hit rate.

I'll pick this issue, since I implemented Filter optimize and process

@Moonlit-Sailor the solution we just discussed offline, should better target 0.4.0. for 0.3.0 I think we can use the workaround.

alright

@yueawang You are right. task won't use index for statistic check. But when for ParquetFileFormat, it can't use VectorizedColumnReader, and will run slower.

I see @TigerSong , I did not consider this kind of issue ago. It's OK to remove this support, as the loss is just a corner case which rarely happens.

Thanks @TigerSong for reporting this, since #558 is merged, would you mind closing #557 ?

#558 has fixed it.


Fix bug on window state save

Fixes #433

Please note the PR you're referencing was closed: https://github.com/tauri-apps/tauri-plugin-window-state/pull/52#issuecomment-1369853832

Hey :wave: Thank you for your contributions! We don't accept PRs for the individual plugin repos anymore since we're moving all of them to https://github.com/tauri-apps/plugins-workspace. It would be much appreciated if you could re-open it against that repo.

Yeah. I know, I got the notification XD


Minor bug fix to minimal example in README

Hi, that's not a bug.
From v0.2.0 the index and index_file methods return (or at least they should return) the retriever, so that it's possible to build a fully working search engine in a single line of code.
Is it not working on your end?

Hi @AmenRa, sorry it's probably an issue at my end. PIP didn't update my library due to the Python 3.8 constraint.


Deparse with indentation?

For a while now we're looking for a gem that can format a given (PostgreSQL) query with indentation. There are a couple of them out there, but they can't handle PostgreSQL specifics, have errors in the outputted SQL or can't be found by me.
So my question, do you think it's feasible to use/extend your gem to parse and deparse with indentation? Or even better, did someone already did this?

@Willianvdv Can you clarify a bit what you mean by "with indentation" - do you mean essentially an opinionated deparse that formats the query a certain way?
As far as I know nobody has done it yet on the basis of this gem (and not familiar with any other efforts besides generic ones that don't support all Postgres syntax), but it wouldn't be too difficult to implement.
A good start for that discussion would maybe be if you can provide an example of what you think would make sense as output rules? I'm thinking we could teach the deparser different styles of output that can be set optionally (i.e. default to the current minimal formatting, but ship one or more better looking output options)

Yeah! I'm not sure what format flavors there are in the SQL world, but the first hit in Google brought me to http://www.sqlstyle.guide/ which describes some best practices how to make your query readable. I can try to add an optional style parameter to deparse to use this style guide.

@Willianvdv Sounds good - a PR would be very welcome!

Closing this, since the issue has become stale, and the deparser-based efforts described here unfortunately didn't pan out at the time.
If one were to attempt this, please be aware that the deparser has moved to the C library (context: https://pganalyze.com/blog/pg-query-2-0-postgres-query-parser#turning-parse-trees-back-into-sql-using-a-deparser), and it would need to be implemented there.
That said, an alternative method would be to utilize the new scanner method to guide whitespace formatting: https://github.com/pganalyze/pg_query/blob/main/spec/lib/scan_spec.rb
(i.e. don't parse and deparse at all, just use the scanner to understand where keywords are located in the source string, and then format based on that)


Updated the README and removed the setup script.

Note that these updated instructions presume #115.

@rimey: Overall, the readme looks good. Just a couple of comments. It makes more sense to address the API related comment (for containers/images) in a separate PR.

I added a Preliminaries section with some tips on configuring kubectl (e.g., how to use kubectl config use-context).


Slim down product images

We might have some potential to slim down product images. This can reduce build time, image size and attack surface. For example, the Hive Dockerfile has a comment about Hadoop: https://github.com/stackabletech/docker-images/blob/1965d50dc436552d4c7e06363f4b1ed46deac29c/hive/Dockerfile#L102
Now that we build from source, it might be worth digging into the build processes to:
a) Limit which components we build. It doesn't make sense to build stuff that's never copied to the final image.
b) Revalidate if all the components that are copied into the final image are really needed in production. With Hive, for example, we switched the build to only build the metastore, which significantly reduced the attack surface. Some products consist of multiple components and plugins, which might not all be needed to run the platform.
c) While we're at it, try to generate an SBOM for each component that is copied into the final image (next to the component itself). For most components that should already be the case, see https://github.com/stackabletech/docker-images/pull/814
We want to focus on products that are mostly affected by vulnerabilities right now:

[ ] Trino
[ ] Hive
[ ] HBase
[ ] Druid

Acceptance criteria:

[ ] Document what could be removed and the impacts of the removal
[ ] Document what can't be removed and why it can't be removed

I'm fairly certain that there is another similar issue already and there is at least one draft PR for Hadoop on this already.

Hive analysis regarding the Hadoop question in the issue description:
The components copied into Hive from the Hadoop-Image are already slimmed down by some degree since we are only building part of the Hadoop components https://github.com/stackabletech/docker-images/blob/main/hadoop/Dockerfile#L66
Hive therefore only contains a subset of Hadoop (contents shown with dive):

Someone with experience and product knowledge might slim it further down by removing jars that are not needed from the shown folders (and under each lib folder in there).

HBase
The first attempt to remove unused components focused on Phoenix. After realising that > 50% of CVEs in the HBase image come from the jackson-databind 2.4.0 the focus was shifted to removing this dependency.
This PR #820 removes it from the phoenix-server component and the number of CVEs is reduced from 502 to 229.
CI for PR #820   https://testing.stackable.tech/view/02 Operator Tests (custom)/job/hbase-operator-it-custom/

HBase - replace htrace with the noop version
Looked into replacing the htrace dependency with it's no-op version as done in the Omid image by @soenkeliebau . Htrace 3.5.0 brings in the offending jackson-databind as a transitive dependency into Phoenix.
Unfortunately it is not possible to replace htrace 3.5.0 with htrace-noop 3.5.0 . The noop version is only a drop in replacement for htrace-core4 :(
On the upside, Phoenix will hopefully replace htrace with opentelemetry soon: https://github.com/apache/phoenix/pull/1282


CODEXUploader not working when the source data is on a network drive

For some reason, when the source data is on the network-mapped drive, the Uploader isn't working: it is able to open the folder and parse the folder contents to populate the ExperimentView, but the 'Start' button doesn't do anything and there are no exceptions shown in the console.  At the same time, when Driffta is started manually from a BAT file, it works

Looking at it right now.

Hi all,
The network drive issue looks like it happens because of access denied error:
Caused by: java.io.FileNotFoundException: \Win-j530bs39qlf\e\180223_FFPE_tonsil_organoid_Ron\Experiment.json (Access is denied)
at java.io.FileOutputStream.open0(Native Method)
Thats why nothing happens when you try to click on start.
I will print this message out on the textArea.
Thanks,
Vishal

Fixed this issue. available in latest commit. But not released yet.


⚠ Processors has degraded performance

In 9423d21, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 69 ms

Resolved: Processors performance has improved in 731b6d9.


DW1000 library and M0 board (ATMEL SAMD21G18)

Hello,
I would like to use the library DW1000 on M0 board.
With some little change on example:

BasicConnectivityTest run
BasicSender doesn't run: DW1000Class::handleInterrupt() is never called and checking SPI library I don't understand how it could run???

On the other side I don"t understand on schematics why IRQ from the DW1000 is connected to Arduino IRQ input???
Thanks a lot for your suppport

Could the IRQ line be hooked up to any digital line on the m0 since they all can be interrupts?

I change interrupt pin from 2 to 3 because on 2 it was not running.
I test caalback with this SW and LED blink when I do a pin change state:
const byte ledPin = 13;
const byte interruptPin = 3;
volatile byte state = LOW;
void setup() {
pinMode(ledPin, OUTPUT);
pinMode(interruptPin, INPUT_PULLUP);
//  attachInterrupt(digitalPinToInterrupt(interruptPin), blink, CHANGE);
attachInterrupt(interruptPin, blink, CHANGE);
}
void loop() {
digitalWrite(ledPin, state);
}
void blink() {
state = !state;
}

I made some tests to sole the issue (I pool SYS_STATUS to track the emission) and in fact:

first send I receive no interrup from the DW1000
other I receive interrupt

Why?

Hi @sloubiou ,
I had the same problem with Arduino Due and now same problem with my arduino Zero.
Can you share your code (your test to track emission)?

Finally I found what is the issue.
Issue is coming from SPI configuration...
Because clock is 48MHz on M0, 16MHz SPI fast and 2MHz SPI slow doesn't run.
You should configure (DW1000.cpp,) fast SPI (_fastSPI) to 12MHz ans slow SPI (_slowSPI) to 1.5MHz.
Now sender and receiver have discussion with interrupts.
Good luck

@sloubiou Did you have to change any code in addition to the spi to get it to work on the zero?

I change a lot of thing to understand the issue but finally only SPI is enought for me.
I'm working on M0 / M0 pro not on Zero but should be the same id micro frequency is 48MHz.

@sloubiou Hello, thanks a lot for your advices about the Arduino M0 Pro. I have this same board and I followed the testboard pins of the wiki : reset on the pin 9, MOSI on the pin 11, etc. But I do not manage to validate the BasicConnectivityTest sketch.
I only get :
Device ID: FFFF - model: 255, version: 15, revision: 15
Unique ID: FF:FF:FF:FF:FF:FF:FF:FF
Network ID & Device Address: PAN: FFFF, Short Address: FFFF
Device mode: Data rate: 6800 kb/s, PRF: 16 MHz, Preamble: 128 symbols (code #4), Channel: #5
I changed the IRQ Pin to 3 and also changed the SPI max and min speed but nothing is working.
Could you please share your wiring diagram to be sure that I am not doing anything wrong ?
Thanks a lot in advance ;)

Hello,
Please add a look on my web site :
http://www.aiill.fr/Geolocalisation_et_Integration_Arduino_DW1000.html
Sincères salutations
Simon Loubiou
AiiLL Loubiou EIRL
Tel : 07-82-76-05-47
Tel : +33 782-760-547
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.aiill.fr http://www.aiill.fr/
De : GARCIA Clément<EMAIL_ADDRESS>Envoyé : mardi 17 janvier 2017 18:25
À : thotro/arduino-dw1000<EMAIL_ADDRESS>Cc : sloubiou<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [thotro/arduino-dw1000] DW1000 library and M0 board (ATMEL SAMD21G18) (#117)
@sloubiou https://github.com/sloubiou  Hello, thanks a lot for your advices about the Arduino M0 Pro. I have this same board and I followed the testboard pins of the wiki : reset on the pin 9, MOSI on the pin 11, etc. But I do not manage to validate the BasicConnectivityTest sketch.
I only get :
Device ID: FFFF - model: 255, version: 15, revision: 15
Unique ID: FF:FF:FF:FF:FF:FF:FF:FF
Network ID & Device Address: PAN: FFFF, Short Address: FFFF
Device mode: Data rate: 6800 kb/s, PRF: 16 MHz, Preamble: 128 symbols (code #4 https://github.com/thotro/arduino-dw1000/issues/4 ), Channel: #5 https://github.com/thotro/arduino-dw1000/issues/5
I changed the IRQ Pin to 3 and also changed the SPI max and min speed but nothing is working.
Could you please share your wiring diagram to be sure that I am not doing anything wrong ?
Thanks a lot in advance ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-273237177 , or mute the thread https://github.com/notifications/unsubscribe-auth/AUWYKDJ-Wk_zpe2uC6WPiPWCSocClXoAks5rTPl-gaJpZM4Jwh03 .  https://github.com/notifications/beacon/AUWYKO2rsqZzHqJWowC5KxmoSOD1joa0ks5rTPl-gaJpZM4Jwh03.gif

Simon
Looks interesting. Do you have a  English version?
On Jan 24, 2017 10:27 AM, "sloubiou"<EMAIL_ADDRESS>wrote:

Hello,
Please add a look on my web site :
http://www.aiill.fr/Geolocalisation_et_Integration_Arduino_DW1000.html
Sincères salutations
Simon Loubiou
AiiLL Loubiou EIRL
Tel : 07-82-76-05-47
Tel : +33 782-760-547
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.aiill.fr http://www.aiill.fr/
De : GARCIA Clément<EMAIL_ADDRESS>Envoyé : mardi 17 janvier 2017 18:25
À : thotro/arduino-dw1000<EMAIL_ADDRESS>Cc : sloubiou<EMAIL_ADDRESS>Mention <
<EMAIL_ADDRESS>Objet : Re: [thotro/arduino-dw1000] DW1000 library and M0 board (ATMEL
SAMD21G18) (#117)
@sloubiou https://github.com/sloubiou Hello, thanks a lot for your
advices about the Arduino M0 Pro. I have this same board and I followed the
testboard pins of the wiki : reset on the pin 9, MOSI on the pin 11, etc.
But I do not manage to validate the BasicConnectivityTest sketch.
I only get :
Device ID: FFFF - model: 255, version: 15, revision: 15
Unique ID: FF:FF:FF:FF:FF:FF:FF:FF
Network ID & Device Address: PAN: FFFF, Short Address: FFFF
Device mode: Data rate: 6800 kb/s, PRF: 16 MHz, Preamble: 128 symbols
(code #4 https://github.com/thotro/arduino-dw1000/issues/4 ), Channel:
#5 https://github.com/thotro/arduino-dw1000/issues/5
I changed the IRQ Pin to 3 and also changed the SPI max and min speed but
nothing is working.
Could you please share your wiring diagram to be sure that I am not doing
anything wrong ?
Thanks a lot in advance ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-273237177>
, or mute the thread <https://github.com/notifications/unsubscribe-
auth/AUWYKDJ-Wk_zpe2uC6WPiPWCSocClXoAks5rTPl-gaJpZM4Jwh03> . <
https://github.com/notifications/beacon/AUWYKO2rsqZzHqJWowC5KxmoSOD1jo
a0ks5rTPl-gaJpZM4Jwh03.gif>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-274836148,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AExz3UPg32aIUKC4uAum-Yu4xb-_Wehfks5rVhh0gaJpZM4Jwh03
.

No sorry
Sincères salutations
Simon Loubiou
AiiLL Loubiou EIRL
Tel : 07-82-76-05-47
Tel : +33 782-760-547
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.aiill.fr http://www.aiill.fr/
De : az-z<EMAIL_ADDRESS>Envoyé : mardi 24 janvier 2017 19:06
À : thotro/arduino-dw1000<EMAIL_ADDRESS>Cc : sloubiou<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Objet : Re: [thotro/arduino-dw1000] DW1000 library and M0 board (ATMEL SAMD21G18) (#117)
Simon
Looks interesting. Do you have a English version?
On Jan 24, 2017 10:27 AM, "sloubiou" <<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

Hello,
Please add a look on my web site :
http://www.aiill.fr/Geolocalisation_et_Integration_Arduino_DW1000.html http://www.aiill.fr/Geolocalisation_et_Integration_Arduino_DW1000.html
Sincères salutations
Simon Loubiou
AiiLL Loubiou EIRL
Tel : 07-82-76-05-47
Tel : +33 782-760-547
<EMAIL_ADDRESS><EMAIL_ADDRESS><<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.aiill.fr http://www.aiill.fr < http://www.aiill.fr/ http://www.aiill.fr/>
De : GARCIA Clément [<EMAIL_ADDRESS><EMAIL_ADDRESS>Envoyé : mardi 17 janvier 2017 18:25
À : thotro/arduino-dw1000 <<EMAIL_ADDRESS><EMAIL_ADDRESS>Cc : sloubiou <<EMAIL_ADDRESS><EMAIL_ADDRESS>Mention <
<EMAIL_ADDRESS><EMAIL_ADDRESS>Objet : Re: [thotro/arduino-dw1000] DW1000 library and M0 board (ATMEL
SAMD21G18) (#117)
@sloubiou < https://github.com/sloubiou https://github.com/sloubiou> Hello, thanks a lot for your
advices about the Arduino M0 Pro. I have this same board and I followed the
testboard pins of the wiki : reset on the pin 9, MOSI on the pin 11, etc.
But I do not manage to validate the BasicConnectivityTest sketch.
I only get :
Device ID: FFFF - model: 255, version: 15, revision: 15
Unique ID: FF:FF:FF:FF:FF:FF:FF:FF
Network ID & Device Address: PAN: FFFF, Short Address: FFFF
Device mode: Data rate: 6800 kb/s, PRF: 16 MHz, Preamble: 128 symbols
(code #4 < https://github.com/thotro/arduino-dw1000/issues/4 https://github.com/thotro/arduino-dw1000/issues/4> ), Channel:
#5 < https://github.com/thotro/arduino-dw1000/issues/5 https://github.com/thotro/arduino-dw1000/issues/5>
I changed the IRQ Pin to 3 and also changed the SPI max and min speed but
nothing is working.
Could you please share your wiring diagram to be sure that I am not doing
anything wrong ?
Thanks a lot in advance ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-273237177 https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-273237177>
, or mute the thread < https://github.com/notifications/unsubscribe- https://github.com/notifications/unsubscribe-
auth/AUWYKDJ-Wk_zpe2uC6WPiPWCSocClXoAks5rTPl-gaJpZM4Jwh03> . <
https://github.com/notifications/beacon/AUWYKO2rsqZzHqJWowC5KxmoSOD1jo https://github.com/notifications/beacon/AUWYKO2rsqZzHqJWowC5KxmoSOD1jo
a0ks5rTPl-gaJpZM4Jwh03.gif>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
< https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-274836148 https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-274836148>,
or mute the thread
< https://github.com/notifications/unsubscribe-auth/AExz3UPg32aIUKC4uAum-Yu4xb-_Wehfks5rVhh0gaJpZM4Jwh03 https://github.com/notifications/unsubscribe-auth/AExz3UPg32aIUKC4uAum-Yu4xb-_Wehfks5rVhh0gaJpZM4Jwh03>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/thotro/arduino-dw1000/issues/117#issuecomment-274885717 , or mute the thread https://github.com/notifications/unsubscribe-auth/AUWYKGmwl-lfDQtf4te8PBFRD_N24Vpiks5rVj2QgaJpZM4Jwh03 .  https://github.com/notifications/beacon/AUWYKEw6BCbWKS_GCL7uruXU0tZZepXGks5rVj2QgaJpZM4Jwh03.gif


Alarm app

Description
I will make an alarm app using HTML, CSS, and javascript.
Project Title
Alarm app
Stack:

HTML
CSS
Javascript

I am a participant at GSSOC'22 .

@Ayush7614 Please assign this task to me @Ayush7614


Cannot read properties of undefined (reading 'secondaryTokens')

Current Behavior
Hey guys,
Can u help me understand how to resolve this issue.
I'm monitoring my app with Sentry, and I am seeing that some users of the site are having this console.error in the terminal.
TypeError: Cannot read properties of undefined (reading 'secondaryTokens')

at Sentry report, i see that comes from:

It may be that there are no major problems, but I would like to treat this to avoid any side effects.
Ps.:
Expected Behavior
errors like "read properties of undefined" aren't very cool, and it seems to be something that wasn't handled on purpose, so it would be good to see if there are any points of improvement.
Steps To Reproduce
I confess I don't know.
What package is effected by this issue?
@web3-onboard/core
Is this a build or a runtime issue?
Runtime
Package Version
2.18.0
Node Version
18.x
What browsers are you seeing the problem on?
Chrome, Microsoft Edge
Relevant log output
TypeError: Cannot read properties of undefined (reading 'secondaryTokens')
  at updateAccountDetails(./node_modules/.pnpm/@web3-onboard+core@2.18.0/node_modules/@web3-onboard/core/dist/index-89f66b35.js:7895:1)
  at next(./node_modules/.pnpm/@web3-onboard+core@2.18.0/node_modules/@web3-onboard/core/dist/index-89f66b35.js:7943:1)
  at ConsumerObserver.prototype.next(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/Subscriber.js:96:1)
  at Subscriber.prototype._next(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/Subscriber.js:63:1)
  at Subscriber.prototype.next(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/Subscriber.js:34:1)
  at cb(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/Subject.js:41:1)
  at errorContext(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/util/errorContext.js:19:1)
  at Subject.prototype.next(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/Subject.js:31:9)
  at BehaviorSubject.prototype.next(./node_modules/.pnpm/rxjs@7.8.1/node_modules/rxjs/dist/esm5/internal/BehaviorSubject.js:31:1)
  at setStep(./node_modules/.pnpm/@web3-onboard+core@2.18.0/node_modules/@web3-onboard/core/dist/index-89f66b35.js:7951:1)
  at connectWallet(./node_modules/.pnpm/@web3-onboard+core@2.18.0/node_modules/@web3-onboard/core/dist/index-89f66b35.js:7840:1)

Anything else?
No response
Sanity Check

[X] If this is a build issue, I have included my build config. If this is a runtime issue, I have included reproduction steps and/or a Minimal, Reproducible Example.

This is solved in the latest versions of core/react/vue - closing as this issue has been handled. Please feel free to continue the conversation or discussion if there is still an issue. Thanks!


Column level lineage is  not captured for databricks merge statements

I have observed that column level lineage is not being captured for databricks delta merge. I believe it has to do with difference in implementation of delta commands in databricks. Could you check and confirm. Attaching query and the generated plan.
execplan.txt
SampleQueryWithNestedWindowFunction.txt

same issue. no input neither output

https://github.com/OpenLineage/OpenLineage/pull/2348 this has been merged recently

Should we close this issue as resolved?


update(build): bump libs to latest master

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

What this PR does / why we need it:
As per title
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

update(build): update libs to 026ffe1d8f1b25c6ccdc09afa2c02afdd3e3f672

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.38.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: eec81e81a4b77c0f7738ef8fe7b57d0f8a64f610

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,LucaGuerra]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Develop the Election List View

Visit https://localhost:3000/elections and Connect your wallet.
We need a more appealing list/tile view instead of the list of addresses that are currently been shown on the page.
Even if you are not able to do the smart contract integration, give us an appealing UI based on the current look of the application.

@mikehammond can I take this issue

@Webnai Did you say you wanted to take this one?

@mikehammond yes can I?

@Webnai When you run the app you will see that there is already some data from the blockchain getting displayed but not in a nice way so take it from there.
Your Metamask should be connected to Sepolia Network

well noted sir


[Request] Environment Variable to control pre-created buckets

Describe the feature
Cryostat3 attempts to connect to the S3 instance and create the buckets it needs at startup, however, if Cryostat3 comes up before its S3 instance is available, then this logic fails. A fairly simple workaround in the case where the S3 instance is a cryostat-storage instance from this repository is to add an environment variable with a list of bucket names which should be created on startup.
https://github.com/seaweedfs/seaweedfs/discussions/2461
It would also be possible to add logic to Cryostat3 so that it attempts to create buckets whenever it hits an exception that the requested bucket does not exist, or to always check if buckets exist before trying to access them, but that's all extra overhead and complexity that is best not done.
Anything other information?
No response

This problem might also be solvable in k8s by using an init container that sets up the buckets in the shared PVC before allowing the main storage container to start up and inherit the pre-configured data volume, but that only helps us in k8s/OpenShift. This proposed solution may be a little less elegant but is also more universal.


Physical properties

Hi. Thank you for a great work. I have a question about plugin. Does it converts physical properties for both directions? For example float: right into float: left. Or it's only for logical properties?
Thank you, Pasha

It only works on logical properties and values.

That’s correct. Consider rtlcss for something sugary. This only implements what the spec defines.


When app is closed, notifications don't appear on system tray

When app is closed, notifications do not appear on system tray on android 7 and above devices

Hi @4tKnight!
Have you registered a background handler? You need to create a background handler using the configure method as explained on the pub.dev page. Also there is some platform specific work mentioned which you have to do.

Ok, will check it out. Thanks

Hi @4tKnight
This platform is not met for assistance on personal code.
Please see https://firebase.google.com/support for resources for asking questions like this
and check our guides;
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with FlutterFire itself,
if you disagree please write in the comments and I will reopen it
Thank you


Support AK 5.3

This removes the AK_API calls and makes this repo work w ak 5.3

Guessing since the tests are failing on other commits too this is a known thing?
I haven't checked the tests locally yet

This was already on develop, I just didn't merge into main correctly.

Try now.


[SUPPORT REQUEST] 3D Convolution

Hi.  Seems you can run deep learning inference on AMD iGPUs with this on Pytorch (on wsl) ! Who would have thought ? Well, you guys.
First of all : great job.
I noticed that 3d conv isn't yet compatible with this. Unlocking 3D convolutionnal neural networks would be of great value to me. Are there any plans ? If not, could you provide pointers on what one would need to do to make it happen, pretty please ? I am willing to devote some time, but I really don't know this code.

I should say that I'm also willing to do the work if it's on pytorch repo if it should happen over there, but if you got pointers, please share.

Sorry for the delayed reply! I'm going to ping some folks here working on the PyTorch implementation to see if this is already planned or if it can be added.

@Adele101 @smk2007 - can you respond, since this is related to PyTorch, not DirectML?

I have the same request.An error occurred when I tried to run 3D convolution on GPU.
Could not run 'aten::slow_conv3d_forward' with arguments from the 'UNKNOWN_TENSOR_TYPE_ID' backend. This could be because the operator doesn't exist for this backend, or was omitted during the selective/custom build process (if using custom build). If you are a Facebook employee using PyTorch on mobile, please visit https://fburl.com/ptmfixes for possible resolutions. 'aten::slow_conv3d_forward' is only available for these backends: [CPU, BackendSelect, Named, AutogradOther, AutogradCPU, AutogradCUDA, AutogradXLA, AutogradNestedTensor, UNKNOWN_TENSOR_TYPE_ID, AutogradPrivateUse1, AutogradPrivateUse2, AutogradPrivateUse3, Tracer, Autocast, Batched, VmapMode].

Thanks for the request! We will add 3D convolution to our backlog for operator support and will update this thread when more information is available.

I did this : is it enough ?


Spacing around references

Hi,
It appears that generally there are no spaces around references (ref) in the generated XML.
For example (generated):
...presented in<ref type="table">Tables II, III</ref>the...

Which should be:
...presented in <ref type="table">Tables II, III</ref> the...

Hello - this is normally already fixed in the latest branch version.

Do you mean master?

oups yes sorry latest master !

Didn't mean to correct you, just to make sure. I made sure I updated master and did even clean before a rebuild. I removed the old XML file just to be sure but the issue appears to be still there. (e.g. first PubMed manuscript)
Anything I should be looking at?

No no I was wrong ;)
I tried the first PubMed central document of the PMC 1942 sample, I obtain this:
... <ref type="table" target="#tab_0">Tables II, III,</ref> the mice with adequate precision for all dose levels, as can be seen in the plots in <ref type="figure" target="#fig_1">Fig. 2</ref> which show both...

Apparently I've committed everything:
lopez@smallbook:~/grobid$ git status
On branch master
Your branch is up-to-date with 'origin/master'.
nothing to commit, working directory clean

Hmmm how very odd. On that example I am seeing now a space to the right but not to the left. i.e.:
presented in<ref type="table" target="#tab_0">Tables II, III,</ref> the mice

That is running it via the PubMed evaluation itself. i.e.:
mvn compile exec:exec -PPubMedCentralEval -Dpmc=... -Drun=1
When I run Grobid directly it seems to exhibit a different behaviour and continues with a different text (I guess it is picking up a different config) but still no space to the left.
java -Xmx1024m -jar .../grobid-core/target/grobid-core-0.4.2-SNAPSHOT.one-jar.jar -gH .../grobid-home -gP .../grobid-home/config/grobid.properties -dIn ... -dOut ... -r -exe processFullText
presented in<ref type="table" target="#tab_0">Tables II, III,</ref> </p></div>

(Doubting myself I did also look at it using different file editors / viewers)
What are of code / commit should have fixed that? I could make sure that it is run.

I also see the missing spaces on the left with the current master! I overlooked them because they are not apparent with the grobid console (there is a new line inserted before) and I did not check with a file editor. I try to fix that quickly, thanks.

Phew... I was starting to question what I was doing.
I also noticed missing spaces at page boundaries. For example (same PMC paper):
only at the highest toxic doseof 200 mg qd
instead of:
only at the highest toxic dose of 200 mg qd
(dose is on page 237 and of is on the following page 238, separated by a figure)
I am not sure if that is caused by the same issue.

Both issues are indeed related but with different models - segmentation in this particular case of page boundary and fulltext for the <ref> elements. Basically the way we create and iterate among labelled sequences (the nice TaggingTokenClusteror class), in some cases, does not preserve easily some starting and ending spacing.
The missing spaces for the <ref> should be fixed with commit f5d5cd600ebf6f29b7dd7eff93692dc01911aeaa, I'll work on the other problem in the next days.
Thanks a lot for notifying all this problem, this is very useful !

Normally the spacing problems around references and in relation to your page boundary example are now all fixed.

Great, thank you.


[darwin] Fix libiconv41 hash

The libiconv41 sha256sum seemed to be incorrect. I guess the upstream archive was changed recently?
patching script interpreter paths in /nix/store/xmivkll9gpswgy4s0kl8qpq8p5x4zsg5-patch-2.7.5
building path(s) ‘/nix/store/5p2cv430dlag356ryjawxvflizjhn1qm-libiconv-41.tar.gz’
trying http://www.opensource.apple.com/tarballs/libiconv/libiconv-41.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 4532k  100 4532k    0     0   261k      0  0:00:17  0:00:17 --:--:--  265k
output path ‘/nix/store/5p2cv430dlag356ryjawxvflizjhn1qm-libiconv-41.tar.gz’ should have sha256 hash ‘10q7yd35flr893nysn9i04njgks4m3gis7jivb9ra9dcb77gqdcn’, instead has ‘0sni1gx6i2h7r4r4hhwbxdir45cp039m4wi74izh4l0pfw7gywad’

Rename commit message as it said in CONTRIBUTING.md.

Fair enough, will open up a "proper" pull request

You don't need to open new PR. https://github.com/NixOS/nixpkgs/blob/master/CONTRIBUTING.md#hotfixing-pull-requests


no version info available within app

No "about" page to tell what version I'm using.

It's in the upper right of the Help screen or at the end of the "Terms & Conditions."

OK I found the former, but not the latter.


chore(main): release 1.2.0

:rocket: New release :bookmark:
1.2.0 (2023-05-10)
:sparkles: Features

add support for comments (#44) (685ebf6)

:hammer: Miscellaneous

repo: add info regarding changelog (82eee54)
repo: update CONTRIBUTING.md (4abd77b)
repo: update README.md (225237f)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/hugo-sid/hugo-blog-awesome/releases/tag/v1.2.0 :sunflower:


Starting Countdown

This PR adds a countdown to the sequence when you /start so gamers aren't getting caught off guard.
Also I added a few lines in the gitignore to be friendly to IntelliJ IDEA.
Tested on 1.16.1 Spigot.

https://user-images.githubusercontent.com/26975085/107450551-5aa40900-6afa-11eb-800a-71f6555507ff.mp4

that moment when i was changing around the sounds and color a little but


Update project_idea.md

Hi Oriol, I reviewed and addressed the comments I made in the original repo. I also made some edits to the doc where I felt it was necessary. Kindly review, thanks.
I am sorry for being a tad late with this.

Thanks! Can you still keep the original line breaks? As this is markdown, line breaks without an empty line are not taken into account when rendering, and it makes reviewing and finding where exactly are the changes much easier.
You also have to be careful with the spacing within each line. For example, the lack of a space between the # characters and the text doesn't create a title, or similarly, the lack or a space between the - and the bullet point text doesn't create a bullet point either.
This is the link to the preview of the rendering build from this pr: https://arviz-gsod--9.org.readthedocs.build/en/9/2022/project_idea.html and this the link to the current rendered version: https://arviz-gsod.readthedocs.io/en/latest/2022/project_idea.html (which I guess is what you read originally)

Thanks!


Server Side Rendering

Thought I'd open an issue to kick off the discussion. It's already possible because of the way we build on top of Glamor, but we haven't exposed it as an API. But basically, if you did this:
import styleSheet from 'styled-components/lib/models/StyleSheet'

/* before each render */
styleSheet.flush()

/* after each render */
styleSheet.rules().map(rule => rule.cssText).join('\n')

Then you should get the chunk of CSS you need for each request. Can someone with a server-rendered setup take a look and confirm this works, and maybe show how they'd be invoking it? I'd be happy enough to export something like:
import { serverStylesheet } from 'styled-components'

/* before each render */
serverStylesheet.reset()

/* after each render */
serverStylesheet.getCSS()

What do people think?

I'll gladly help test this out! We currently avoid using styled components for most styles because of the unclear path to support SSR, would be great to move the inline styles into styled components if we can generate the CSS on the server.
Here is what I have setup so far:
import styleSheet from 'styled-components/lib/models/StyleSheet';

styleSheet.flush();

let markup = renderToString(
  <Root store={store} Router={ServerRouter} routerProps={{ location, context }} />
);

console.log(styleSheet.rules());

Unfortunately, I'm getting back an empty array []. I can confirm that one of the components nested inside Root is wrapped by styled and generates a className when the client loads.

I'm currently implementing SSR on a project and I managed to make this work without styleSheet.flush() (if I call this, the result is the same as @pheuter), only the styles from injectGlobal weren't included.
Once I finish it I will post the complete code and a demo here. (I'm trying to make the app work completely without javascript enabled on client).

only the styles from injectGlobal weren't included.

Huh, that's interesting. I think I might accidentally use another styleSheet for that? /cc @geelen

Demo: https://arc.diegohaz.com (disable javascript, even the form works).
Code: https://github.com/diegohaz/arc/blob/universal-redux/src/server.js#L74
It doesn't work with injectGlobal (I removed).
The serverStylesheet.getCSS() API would be nice. 👍

I noticed something.
I'm using Grid from 'grid-styled' and using styleSheet.rules().map(rule => rule.cssText).join('\n') seems to go just one level deep. It never rendered code of Grid, but rendered my code that used media queries.
code is like this:
const Wrapper = styled(Grid)`
  position: fixed;
  display: flex;
  justify-content: center;
  align-items: center;
  height: 40px;
  z-index: 0;
  top: 90px;
  z-index: 1;
  width: 100%;
  left: 0px;
  @media screen and (min-width: 52em){
    left: 16.66%;
  }
  @media screen and (min-width: 64em){
    left: 25%;
  }
`

Invoked as <Wrapper xs={1} sm={1} md={2/3} lg={1/2}>{children}</Wrapper>
gets rendered as
.iMsaJZ {
  position: fixed;
  display: -webkit-box;display:flex;display:-webkit-flex;display:-ms-flexbox;
  -webkit-box-pack: center;
  -ms-flex-pack: center;
  -moz-justify-content: center;
  -webkit-justify-content: center;
  justify-content: center;
  -webkit-box-align: center;
  -ms-flex-align: center;
  -webkit-align-items: center;
  align-items: center;
  height: 40px;
  z-index: 0;
  top: 90px;
  z-index: 1;
  width: 100%;
  left: 0px;
 }
@media screen and (min-width: 52em){
  .iMsaJZ {
    left: 16.66%;
  }
}
@media screen and (min-width: 64em){
  .iMsaJZ {
    left: 25%;
  }
}

The element receives the correct classname on SSR, wich are two classes, one for my code and one for Grid's.
But there is no mention to the classname designed to Grid in the initial server response.
When I turn JS on then Grid's code appears to the browser.
I hope I made myself clear :)

Based on @pheuter test, I get the same result,(the class is injected in the string markup) but not calling stylesheet.flush() before calling styleSheet.rules()  produces TypeError: Cannot read property 'cssRules' of undefined

Ah, this is so great. I was able to get it working with gatsby in a no time
  render () {
    const styles = styleSheet.rules().map(rule => rule.cssText).join('\n');

    return (
      <html lang="en">
        <head>
          <style>
            {styles}
          </style>
        </head>
        <body>
          <div id="react-mount" dangerouslySetInnerHTML={{ __html: this.props.body }} />
          <script src={prefixLink(`/bundle.js?t=${BUILD_TIME}`)} />
        </body>
      </html>
    )
  }

I'm all for exposing a simpler api via serverStylesheet.getCSS()

@gbozee I ran into that problem. Turns out when (f.e.) webpack 2 is used, it loads the es bundle, due to the package.json js:next/module fields.
Until there is an API for the stylesheet that we can use, you should work around it using webpack's resolve.alias to load the lib folder instead.

I think I might accidentally use another styleSheet for that?

@mxstbr I'm interested in solving this. Can you give me a way?

Please get in touch when somebody start to handle this. I would like to get this to work out of the box in Phenomic (I just integrated Glamor and Aphrodite rendering and would like to cover all populars CSS in JS solutions).
I was thinking glamor integration will be enough but @philpl just told me it won't work :(

@aesopwolf did you get the same problem as @diegohaz of not having styles included from injectGlobal calls?
I think its a general consensus that this is working out of the box.  The only minor caveat is styles from injectGlobal isn't included.  But fixing that is trivial I guess?

@thisguychris I just ran a quick test with injectGlobal and I didn't get the same problem. My document head included a single style tag with everything expected.
Input:
const Title = styled.h1`
  font-size: 1.5em;
  text-align: center;
  color: palevioletred;
`;

injectGlobal`
  body {
    background: red;
  }
`;

Output:
<style>
    .kZQsGZ {
        font-size: 1.5em;
        text-align: center;
        color: palevioletred;
    }
    body {
        background: red;
    }
</style>

Yeah the implementation for injectGlobal uses the same instance of StyleSheet under the hood so what @aesopwolf is getting looks right..

@geelen so do we get a green light for the exposed api? aka serverStylesheet.getCSS()?
Also, @aesopwolf would you mind giving me a test repo of your SSR to test my PR with?

I would love an exposed API, would love for you to look into that @thisguychris!

You guys are right, injectGlobal is working. It was a problem in my code.
I was using it inside componentDidMount (which doesn't run in ssr).
Sorry for the mess.

Now this repo is working with styled-components, injectGlobal, SSR and everything: https://github.com/diegohaz/arc/tree/fullstack
@thisguychris

It would be great if .flush() worked. The problem may be somewhere here:
https://github.com/styled-components/styled-components/blob/master/src/vendor/glamor/sheet.js#L88
     this.sheet  = {
        cssRules: [],
        insertRule: rule => {
          const serverRule = { cssText: rule }
          this.sheet.cssRules.push(serverRule)
          return {serverRule, appendRule: (newCss => serverRule.cssText += newCss)}
        }
      }

flush() on server does this:
this.sheet.cssRules = []

So after flushing appendRule will be appending text to a dangling serverRule that is not in this.sheet.cssRules anymore.

@diegohaz tried fullstack branch getting this error
[0] MongoDB connection error: MongoError: failed to connect to server [<IP_ADDRESS>:27017] on first connect

@thisguychris It's because that branch requires the MongoDB service to run. You can use the universal-redux branch as well (it doesn't require mongod).

@diegohaz looks like its working 👍
http://take.ms/r9QiC
So we don't need flush() / reset() for SSR?

Currently styled components will inject the styles even though they are rendered on the server and there is no need for that. Are you guys planning to do something about it? Ideal solution would be to reuse the prerendered styles and inject only these that are necessary.
Maybe it would be possible to render the generated style tag(s) within <body/> with React? Then React would update their contents only when the change.

Being relatively new to styled-components I started adopting SSR for a hack project. For some reason I'm getting TypeError: Cannot set property 'cssRules' of undefined with or without .flush(). Execution will actually fail to this error already when calling flush() prior to renderToString.
Here's what logging out styleSheet yields:
e {
  isSpeedy: false,
  sheet: undefined,
  tags: [],
  maxLength: 40,
  ctr: 0 }

and here's my sample code for sanity check:
import express from 'express';
import React from 'react';
import { renderToString } from 'react-dom/server';
import fs from 'fs';
import App from './App.js';
import template from './template.js';
import styleSheet from 'styled-components/lib/models/StyleSheet';

const config = JSON.parse(fs.readFileSync('./build/asset-manifest.json', { encoding: 'utf8' }));

const jsPath = config ? config[Object.keys(config).find(fileName => fileName.endsWith('.js'))] : '/bundle.js';
const cssPath = config ? config[Object.keys(config).find(fileName => fileName.endsWith('.css'))] : null;

styleSheet.flush();
const markup = renderToString(<App />);
const styles = styleSheet.rules().map(rule => rule.cssText).join('\n');

const server = express();
server.use('/static', express.static('build/static'));
server.get('*', (req, res) => res
  .send(template(markup, jsPath, cssPath, styles)));

const port = process.env.PORT || 3000;
server.listen(port);

SSR is working fine aside from styled components.

I am trying to use hedron, but it seems the styles from it are not present. But my custom styles are rendered fine. Any ideas why this is happening/how to work around?

Don't know if it's better to create a new issue, but I'm seeing this kind of warning on server when passing custom props to a styled component:
Warning: Unknown props `transparent`, `light` on <a> tag. Remove these props from the element. For details, see https://fb.me/react-unknown-prop

It happens only on server (terminal), not on browser.

@jorilallo I had quite similar issue. In my case I solved it by adding 'styled-components/lib/models/StyleSheet' in externals in webpack 2.
externals: {
'styled-components/lib/models/StyleSheet': 'commonjs styled-components/lib/models/StyleSheet',
}
it was not there because I was adding only the node_modules root folder to externals.

I'm not sure why this is so hard using styled-components. It's already possible with glamor, which it seems this is based on, right? Can't we just simply use the underlying glamor API or parts of it and re-expose?

@code-punkt see #214 :)

Curious about why both .reset and .flush are required? Wouldn't the flush after the render call be sufficient, since it returns the styles and resets the state for the next call?

@aesopwolf thanks for the code snippet, worked with Gatsby for me too. But I think it should be <style dangerouslySetInnerHTML={{ __html: styles }} />, otherwise special characters will get escaped out.

@tomazy You can always remove your SSR style block once the client-side rendering is done. That way, your page will look great while the app is loading and you will not get duplicate styles.
ReactDOM.render(<App />, document.getElementById('app'));
document.getElementById('ssr-style').remove();

@exOn yes, but styled-components could also just stop injecting it on the client-side again so you don't have to hack stuff like that together ;-)
See #214 for more discussion about this.

@tomazy I used something similar to remove the server rendered critical-css block when react and styled-components kicked in on the client side.
The problem with this approach is that it will also remove any styles you injected with injectGlobals,Those styles, if rendered on the server, will end up in the ssr block.
It is also feels like a dirty hack compared to the rest of the experience offered by styled-components 😛
Of course there are simpler ways to stick some global css in the pre-rendered page's head section but the point is that it would be nice to have proper support for it.

PSA: I was seeing TypeError: Cannot read property 'cssRules' of undefined when using this line of code:
const styles = styleSheet.rules().map(rule => rule.cssText).join('\n');
The fix: If you are seeing this as well, make sure you are rendering at least one styled component in your app.
For me, I was just installing styled-components for the first time, and hadn't used one yet, and was therefore seeing this error.

I have submitted a PR to fix this issue here. Hope this helps some!

I'm using 'hedron' (which uses styled-components) as well and my stylesheet rules do get mapped. However, I would like the 'hedron' stylesheet rules to get mapped as well. At the moment this doesn't seem to be happening, since the following error is being thrown:
Warning: React attempted to reuse markup in a container but the checksum was invalid. This generally means that you are using server rendering and the markup generated on the server was not what the client was expecting. React injected new markup to compensate which works but you have lost many of the benefits of server rendering. Instead, figure out why the markup being generated is different on the client or server:

What would be the best way to solve this issue? Should I submit a PR for 'hedron' and add a method that returns styleSheet.rules().map(rule => rule.cssText).join("\n");? And invoke hedron's method in my page head, where I run this method for my own styleSheet rules as well?

I am trying to use hedron, but it seems the styles from it are not present. But my custom styles are rendered fine. Any ideas why this is happening/how to work around?

@dignifiedquire - Could you please tell me how you worked around this? I'm having the exact same issue with hedron.
@diegohaz - Using the universal-redux branch for this, is there anything that I should look into particularly?

@karthikiyengar Hedron is using styled-components which I believe means you need to import both your own server markup and theirs.

@kristojorg - Thanks for your answer.
Please disregard, my vendor assets weren't being generated properly, leading to the issue.

This discussion has now been moved to #386.


Addendum for #214: coerce strings as string nodes

I like the new parser impl namespace, makes things clearly internal.
Thought maybe the changelog would need an update, but it does not.

LGTM!


Add Cashu wallet backend

Adds Cashu wallet backend through the REST API.
Depends on https://github.com/cashubtc/nutshell/pull/318

@callebtc the required PR seems to be merged, can you take a look again?

@callebtc

is this PR completed
is there a cashu docker image that we can use in our regtest

Bump @callebtc. Lets get this out of draft and ready to merge

I won't have time to finish this PR unfortunately


Documentation on plugin API?

Hi, i'm currently building a pdebuild plugin for our CD infrastructure (https://github.com/HuayraLinux/strider-debian) but it seems like the only real documentation on the API.
For example the type of the parameters config, job and context, when to call done, etc.
¿It's out there documentation or a file from strider sourcecode with relevant comments? The plugin template was helpful but didn't cover all my needs
BTW: Sorry if my wording sounds strange, im trying my best but english isn't my passion :disappointed:

Thanks! It seems brief but descriptive :smile:


Added test for Candidate Pair and Request for Connectivity Check

Description of changes:
This PR is for adding additional unit tests to improve coverage, by thoroughly testing various ICE API's .
Also, The Unit-Tests are written from start and keeping the format same as the previous written unit-tests of other files.
testHmacFxn was improved.
The following test cases have been deleted because of no added value:

test_iceAddHostCandidate_Multicast

The following test cases have been added:

test_iceAddCandidatePair
test_iceCreateRequestForConnectivityCheck

Test Steps
git submodule update --init --recursive --checkout test/CMock
cmake -S test/unit-test -B build/ -G "Unix Makefiles" -DCMAKE_BUILD_TYPE=Debug -DBUILD_CLONE_SUBMODULES=ON -DCMAKE_C_FLAGS='--coverage -Wall -Wextra -Werror -DNDEBUG -DLIBRARY_LOG_LEVEL=LOG_DEBUG'
make -C build all
cd build
ctest -E system --output-on-failure
make coverage

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Can we have a test case that adding remote candidate first following by local candidate?

Please raise a separate PR for this case.


Possible typo in Manipulator Kinematics notebook?

Should ET13 be rtb.ET.Rx(pi) instead of rtb.ET.Rx(pi / 2)?

Great catch! Thanks for letting me know


#187 - perimeter prj name is at 30 char limit

/lgtm

will do the readme perimeter to per as part of another - forgot it
could have done "perim" or "ptr" - later as a broader adjustment


Create a doc that explains how the rate limiting settings affect the Mattermost instance

Original Jira ticket here: https://mattermost.atlassian.net/browse/PLT-4406
Corey and Chris can help with technical details

👋 @jasonblais and @lfbrock: I look forward to addressing this my work simulation. Please confirm the accuracy of my audience analysis to inform my content strategy.

audience: Who are we writing the topic for?
Admins and end users
version(s): Does the situation apply to previous releases, for the current release and going forward, or a future release?
The information need is a combination of conceptual and procedural content. The conceptual information about ways that rate limit settings affect a Mattermost instance and guidelines for recommended values are appropriate for all versions. Procedures or methods for rate limiting by IP address and user authentication token should be written generically so that users can follow instructions across any version.
platforms: Does this situation apply to every platform or only specific one(s)?
The content applies to all platforms.
related topics: Are there other existing topics that the reader should consider? Do parallel or other topics need to be written? I envision this as a total of four topics - two conceptual and two procedural; with one set (concept and procedure) in the User's Guide and a second set in the Admin's Guide.

Thanks for answering my questions in our channel @jasonblais and @lfbrock. Here is the content strategy for the new topic:
Summary
New topic that provides an overview of rate limiting. This topic will be a combination of conceptual and procedural content to support administrators in setting rate limits as it relates to Mattermost.
New content
Topic title: “Setting Rate Limits“

Explain what a rate limit is
Explain implications and benefits of rate limit settings to a Mattermost instance
Provide three (basic, intermediate, and advanced or low, medium and high) scenarios and recommended rate limit values for each one
Explain rate limiting methods

Limit by IP address
Limit by User Auth token

Outstanding questions

Should we mention that users need admin credentials or is that a given?
Warn against any particular actions?

Great, thanks @mirhonda!

Should we mention that users need admin credentials or is that a given?

We'll include it in the Admin's guide so we can take Mattermost admin credentials as given.

Warn against any particular actions?

If there's something admins should be aware of before taking some actions, that would be helpful to include.

We have a section on rate limiting in the config guide: https://docs.mattermost.com/configure/configuration-settings.html#rate-limiting. Would you say that covers this request?

@justinegeffen It would be good to have a one-pager of what are the implications of changing the rate limit settings. E.g. something what's described in https://github.com/mattermost/docs/issues/894#issuecomment-380805387

Replaced by https://github.com/mattermost/docs/issues/6591


helm dependency update with inline helm values file

Hi guys
I am using the following cm plugin config map with my side car to access hashi vault to get secrets for my helm charts.
  avp-helm.yaml: |
    ---
    apiVersion: argoproj.io/v1alpha1
    kind: ConfigManagementPlugin
    metadata:
      name: argocd-vault-plugin-helm
    spec:
      allowConcurrency: true
      discover:
        find:
          command:
            - sh
            - "-c"
            - "find . -name 'Chart.yaml' && find . -name 'values.yaml'"
      generate:
        command:
          - bash
          - "-c"
          - |
            helm template $ARGOCD_APP_NAME -n $ARGOCD_APP_NAMESPACE -f <(echo "$ARGOCD_ENV_HELM_VALUES") . |
            argocd-vault-plugin generate --verbose-sensitive-output -
      lockRepo: false

Its not working obviously as it doesnt do the dependency update - and I cant figure out how I cd down the path to the folder do the dependancy update and then do the helm templating with the inline variables from the argocd application.
I can use a simpler helm chart with no dependancies and no requirements for inline arguements and its working fine.

sorted it all out.

I used to have a similar pb
Unable to save changes: application spec for vault is invalid: InvalidSpecError: Unable to generate manifests in applications/vault/helm: rpc error: code = Unknown desc = plugin sidecar failed. error generating manifests in cmp: rpc error: code = Unknown desc = error generating manifests: sh -c "helm template $ARGOCD_APP_NAME -n $ARGOCD_APP_NAMESPACE ${ARGOCD_ENV_HELM_ARGS} . |\nargocd-vault-plugin generate -\n"failed exit status 1: Error: An error occurred while checking for chart dependencies. You may need to runhelm dependency build to fetch missing dependencies: found in Chart.yaml, but missing in charts/ directory: vault Error: Must provide a supported Vault Type, received Usage: argocd-vault-plugin generate <path> [flags] Flags: -c, --config-path string path to a file containing Vault configuration (YAML, JSON, envfile) to use -h, --help help for generate -s, --secret-name string name of a Kubernetes Secret in the argocd namespace containing Vault configuration data in the argocd namespace of your ArgoCD host (Only available when used in ArgoCD). The namespace can be overridden by using the format <namespace>:<name> --verbose-sensitive-output enable verbose mode for detailed info to help with debugging. Includes sensitive data (credentials), logged to stderr
`
In fact It was a side effect of argocd upgrade with use of sidecar
Before for a helm chart application located in : applications/app/helm
apiVersion: argoproj.io/v1alpha1 kind: Application metadata:   name: app   namespace: argocd spec:   destination:     namespace: myapp     server: https://kubernetes.default.svc   project: myapp   source:     path: applications/app/helm     repoURL: https://xxx.myrepo.git     targetRevision: main
works but I doesn't anymore
you must be more precise to avoid kustomise vault plugin:
apiVersion: argoproj.io/v1alpha1 kind: Application metadata:   name: app   namespace: argocd spec:   destination:     namespace: myapp     server: https://kubernetes.default.svc   project: myapp   source:     helm:                                     // <====== specify that it's a helm       parameter: []                      // <====== specify that it's a helm       releaseName : myapp        // <====== specify that it's a helm       valueFiles:                           // <====== specify that it's a helm       - values.yaml                       // <====== specify that it's a helm     path: applications/app/helm     repoURL: https://xxx.myrepo.git     targetRevision: main


some demo controls not working

increment, change height and change color are throwing errors in the console and not working.
when clicking on increment
TypeError: ngProgress.status is not a function
    at Object.$scope.count (http://victorbjelkholm.github.io/ngProgress/:369:47)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:6129:19
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12496:13
    at Object.Scope.$eval (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7770:28)
    at Object.Scope.$apply (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7850:23)
    at HTMLAnchorElement.<anonymous> (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12495:17)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1930:10
    at Array.forEach (native)
    at forEach (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:110:11)
    at HTMLAnchorElement.eventHandler (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1929:5)

$scope.count = function($event) {
  $event.preventDefault();
  $scope.progressbar.set(ngProgress.status() + 9); // .status() is not a function
}

when clicking on change height
TypeError: Cannot read property 'preventDefault' of undefined
    at Object.$scope.new_height (http://victorbjelkholm.github.io/ngProgress/:378:19)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:6129:19
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12496:13
    at Object.Scope.$eval (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7770:28)
    at Object.Scope.$apply (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7850:23)
    at HTMLAnchorElement.<anonymous> (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12495:17)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1930:10
    at Array.forEach (native)
    at forEach (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:110:11)
    at HTMLAnchorElement.eventHandler (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1929:5)

$scope.new_height = function(new_height, $event) {
  $event.preventDefault(); // $event is not defined
  $scope.progressbar.height(new_height);
}

when clicking on change color
TypeError: Cannot read property 'preventDefault' of undefined
    at Object.$scope.new_color (http://victorbjelkholm.github.io/ngProgress/:373:19)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:6129:19
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12496:13
    at Object.Scope.$eval (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7770:28)
    at Object.Scope.$apply (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:7850:23)
    at HTMLAnchorElement.<anonymous> (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:12495:17)
    at https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1930:10
    at Array.forEach (native)
    at forEach (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:110:11)
    at HTMLAnchorElement.eventHandler (https://ajax.googleapis.com/ajax/libs/angularjs/1.0.1/angular.js:1929:5)

$scope.new_color = function(color, $event) {
  $event.preventDefault(); // $event is not defined
  $scope.progressbar.color(color);
}

i could take a deeper look later

Hm, seems like some of the interface of the library have changed while the website doesn't reflect this. Thanks for checking it out, I'll take a look later today at it.
Again, thanks for reporting it!

Fixed in 99055a328fffea032d6b18d8ce5664f338594cfe, cd178d5927437f2d88ac65579f8c32d38a051928 and 1ebcd66727ba1e78e7adfedb285f8c4f4314668c
Thanks again for reporting!


Line breaks (new line) are not working

Describe the bug
2 consecutive lines are being displayed in the same line, rather then in a separate lines
In some cases Slides does not honor new line as one expects, e.g. on lists.
To Reproduce
Steps to reproduce the behavior:
$ echo '
line 1
line 2
' | slides

outputs
line 1 line 2

and
echo '
- list 1

- list 2
' | slides

outputs
• list 1
• list 2

Expected behavior
A new line needs to be displayed in a new line, e.g. giving the inputs above, the outputs should look like (respectively)
line 1
line 2

and
• list 1

• list 2

Desktop (please complete the following information):

OS: MacOS
Version: 13.3.1

With regular text, the behavior you expect appears to be consistent with hard line breaks, but I found that slides does not seem to respect the two extra whitespaces or backslash methods mentioned in the GFM spec. I found an open issue for the lack of line breaks on whitespace on an upstream repo: https://github.com/charmbracelet/glamour/issues/84. I found that inserting an "extra" line of whitespace worked got the desired result.
I can confirm the odd list behavior on slides 0.9.0 (Ubuntu 22.04.02). The list rendering behavior does not appear consistent with GFM - any number of empty lines between list items does not result in a line break between the two lists.
There are several issues in glamour that mention broken lists (https://github.com/charmbracelet/glamour/issues/239, https://github.com/charmbracelet/glamour/issues/167) but they do not seem to describe our issue here exactly. Perhaps another issue upstream is warranted?

@electriquo Try this:
```
Line one
Line two
Line three
```

It won't render anything special like bold or emphasis, but if you just want text (like an address block for example) it looks fine.

@electriquo Same. I find it very disappointing that Glamour, a library whose only job is to render markdown in the terminal, falls so short so quickly (in my first use of Slides, I started running into glamour issues basically on line 2 of my markdown file).
Makes me wonder if a better approach would be a spec-compliant markdown to HTML converter pipelined to a terminal HTML renderer.
Don't want my disappointment to reflect badly on @maaslalani -- I'm loving every part of this project that you've put together!


integrate this plugin into my grapejs already in place

Hello,
I am looking to integrate this plugin into my grapejs already in place.
Only I can't get him to add his panel without destroying the others
If I do :
//...
  panels: {
    defaults: [
      {
        buttons: [
          //...
          {
            attributes: { title: 'Open Code' },
            className: 'fa fa-code',
            command: 'open-code',
            id: 'open-code'
          }
          //...
        ],
        id: 'views'
      }
    ]
  },
  //...

That code is present, but the other tab disappears that resets the config.
How to tell him to add it?

You can add it to the panels using the same method in index.html.
https://github.com/Ju99ernaut/grapesjs-component-code-editor/blob/5a85578279355836081a1aaabcf50a9ddaa4e900/index.html#L107-L119


Initial commit for ETL-3803: remove topic conception from compaction

@zliu41 Can you review?

Updated with some fix. Tested in Azkaban with regular compaction, late data movement, and recompaction.

Please resolve conflicts.

@zliu41 Updated.

@zliu41 Updated.

@zliu41 Updated.

@zliu41 Updated.

@ydai1124 can you please describe the high-level design in the PR description?

@chavdar Changed both title and description.

@chavdar @zliu41 Moved dataset and datasetFinder to gobblin-api. @ibuenros Can you also take a look on the last commit?

LGTM


[Request] Support bungeecord/velocity with /server lobby ?

is it possible for this to support a Bungee/Velocity environment ?
so portals can send players to different servers ?

Is this for 1.18 or 1.19? This requires a separate mod to be installed to add the velocity functionality.

Indeed, I meant, could this be used to once you have the '/server' function in place ...

I do already have a mod for 1.18 that does this. I don't think I ported it to 1.19 yet.
The /server commands exists on the proxy not the server where the mod exists, so the mod must communicate with the proxy.

I do already have a mod for 1.18 that does this. I don't think I ported it to 1.19 yet.
The /server commands exists on the proxy not the server where the mod exists, so the mod must communicate with the proxy.

Is that a totally separate mod ?
Could that be incorporated into this mod ?..
Would be magic if possible....

I released 1.2.0 for 1.19 which includes support for portals running commands built in. If you're on 1.18 this used to be its own addon  mod to datapack portals.
See the example portal.
Note you'll also need to use the other mod you linked for the server switching command itself. It has also been updated to 1.19

We are running latest, so this would actually work then !!...
Will give this a go ! Thankyou ..

well tried to get this working, the velocityserverswitch mod works..
but cannot get this DatapackPortals to work :( using datapackportals-1.2.0-1.19.jar there are no errors in logs...
does it have any  dependences ? as in do i need the customportalapi ? (as i cannot see find that as a download)
have used your basic example, and my custom one, added the data pack, and thats showing up ok in /datapack list... and datapack is loading correctly looking at logs...

No it does not require any dependencies to be installed, if you check the logs customportalapi should already be there.
The example portal requires a frame to be made out of dirt and then lit with a stick. That didn't work?

Feel free to message me on discord:
Kyrptonaught#4239

No it does not require any dependencies to be installed, if you check the logs customportalapi should already be there.
checking logs, yes its present :) that ticks that off...

The example portal requires a frame to be made out of dirt and then lit with a stick. That didn't work?
i used you end portal example.. but indeed, doesnt do a thing, no errors or anything...

discord friend/message sent :) ...

just to close this issue....
to fix above issues, you needed mod on client...
but with this provided mod it works for vanilla clients (using nether/end portal data visuals)
also update have has fixed fabric proxy issues to :)  so all my additional requests been answered too !
problems clossed


Adding tac Command

I want to add the tac command to this project please assign me this issue under HacktoberFest 2023.

Push and open a pull request Pls...


x/build/cmd/gomote: OpenBSD workdir is owned by root

/tmp/workdir on the openbsd-386-60 gomote (and probably the other openbsd gomotes, but I didn't check) is owned by root and not writable by user gopher. gomote push is able to write the Go tree there, but it's all owned by root, so if you then gomote ssh and run make.bash it will fail because you're logged in as gopher and can't write to the Go tree.
The work-around is to cp -R /tmp/workdir/go $HOME/go and build from gopher's home directory.
/cc @bradfitz @andybons

Duplicate of #23043


Option to select all logical CPU cores.

Is your feature request related to a problem? Please describe.
It is frustrating to have to change the max_no_of_threads setting every time you run the same test on systems with a vastly different number of cores.
If you don't do this, you either incur additional overhead on the system with fewer cores or you don't use all the cores that the stronger system has available.
Example:
We have two systems, a server and a laptop, and we want to run a configuration on both.
The server has 128 CPU cores.
Let us now assume that we want to use all cores of the server.
Therefore, we set max_no_of_threads to 128.
If we now run the same config on a laptop with 12 threads (logical cores) we still create 128 threads of which only 12 can be executed in parallel.
For each additional specified thread that is also spawned, we generate unnecessary overhead.
The alternative is to set max_no_of_threads to 12.
This might be the best option to run the test with the laptop, but with regard to the server, 116 available cores now remain unused.
Describe the solution you'd like
Allow to set max_no_of_threads to the keywords all  to select all logical cores of the current system..
Additionally, the keyword half could be used to select the half of all available logical cores.
Describe alternatives you've considered
An alternative to introducing the all keyword is to select all cores when max_no_of_threads is set to 0.

Hello @FelixUhle ,
I am interested in contributing to your project, specifically in the area of enhancing thread management to adapt dynamically to different system capabilities.
Before I begin, I would like to gather some additional information to ensure my contributions align well with the project's existing standards and practices:
Could you please provide more clarity or any additional specifics regarding the feature for dynamic thread allocation based on system capabilities? Understanding the exact requirements and expectations will help me tailor my approach effectively.
Does the project adhere to any specific coding standards or practices that I should be aware of? Knowing these would help me ensure that my contributions maintain consistency with the existing codebase.
Also, since the feature involves multithreading, I am particularly interested in understanding any existing guidelines or best practices you follow for testing multithreaded applications. This information will be crucial for me to write effective and comprehensive tests for the new feature.
Thank you for your time and consideration. I am eager to contribute and look forward to hearing from you soon.
Best regards,

Hey @gmottajr,
Thank you for your interest. We would be delighted if you could participate in PROLEAD.
As @FelixUhle has already mentioned, we have a bit of information about the current issue. So far, we have been statically specifying the max_no_of_threads by setting an integer in the config file, which means that the number of threads is not dependent on the actual available threads on the target system. Now, we would like to dynamically determine the number of threads available on the system so that dynamic specifications such as all or half could be used for max_no_of_threads. Parsing these new keywords is not a problem; I can take care of that. PROLEAD would then need to determine the number of threads for these keywords and internally set max_no_of_threads back to an integer.
Regarding coding standards, I recently uploaded a style file that can be easily used in VS Code. In general, we plan to adopt the Google coding style in the future. However, we haven't implemented this in all files yet. For guidance, you can take a look at probing.cpp.
We are currently thinking about testing. So far, we haven't established a guideline or best practices for it. I have uploaded an initial file for unit tests, which we will gradually supplement. As for testing multithreading, we can collaborate on finding a solution together.
Regards,
Nico

Thank you very much, Nico (@nicolaimueller) for you feedback, I do appreciate the details you provided me. I'll carefully take a look into the file you mentioned.
Shall I have this issue assigned to me, please, so that I can start to work on it?
Thank you very much, once again!

Hey guys awesome! 🙌
I could enjoy an free window  🤔 here during this holydays I have something for you as a starting point 💡.
I create 📝 a ThreadManager class designed to manage and optimize thread allocation in C++ applications, especially useful when running the same application on different systems with varying numbers of CPU cores 💻 . It provides a flexible way to determine the best number of threads to use based on the system's hardware and user-defined preferences.
Here is a description of my initial solution for the problem you described:
Enum: ThreadOption
The ThreadOption enum defines various options for thread allocation:

All: Utilize all available logical cores of the system.

Half: Use half of the available logical cores.

Third: Allocate one-third of the available logical cores.

Quarter: Use one-quarter of the available logical cores.

Specific: Use a specific number of threads, defined by the user.

Public Static Method: getOptimalThreadCount
static unsigned int getOptimalThreadCount(ThreadOption option, unsigned int specificCount = 0);

This method calculates the optimal number of threads based on the selected ThreadOption. It takes two parameters:

option (ThreadOption): The thread allocation strategy as defined in the ThreadOption enum.
specificCount (unsigned int): The specific number of threads to use when ThreadOption::Specific is chosen. Defaults to 0.

Parameters

option: The strategy for determining the number of threads.
specificCount: An optional parameter that specifies the exact number of threads to use when the Specific option is chosen.

Return Value
Returns the calculated optimal number of threads as an unsigned integer.
Behavior

All: Returns the total number of logical cores available on the system.
Half/Third/Quarter: Calculates the respective fraction of the total number of cores.
Specific: Returns the lesser of specificCount or the total number of cores, ensuring that the thread count does not exceed the number of available cores.
Default Case: In an undefined scenario, it safely returns 1 as a fallback.

Usage Example 🚀
unsigned int threads = ThreadManager::getOptimalThreadCount(ThreadOption::Half);
// This will set 'threads' to half the number of cores available on the system.

Implementation Details 📝

The class uses std::thread::hardware_concurrency() to determine the number of logical cores available on the system.
Error handling should be implemented in cases where hardware_concurrency() returns 0, indicating an inability to detect the number of cores.

Notes 🌟

The class does not store state and is designed to be used statically.
Ensure thread safety if used in a multi-threaded context.

Here you can see the class ThreadManager 📚:

Ease of Understanding and Adaptability for Developers 😎 :
The ThreadManager class provides a patterned solution that is straightforward and intuitive. This design choice ensures that any contributing developer, regardless of their familiarity with the project, can easily understand and work with the class. It streamlines the learning curve and fosters effective collaboration, making it simpler for new contributors to adapt and contribute enhancements or fixes.
Focused on SOLID Principles 🏗:
Single Responsibility Principle (SRP): Dedicated solely to thread management, ensuring clarity and reducing complexity.
Open/Closed Principle (OCP): Easily extendable for new thread allocation strategies without modifying existing code, enhancing flexibility.
Liskov Substitution Principle (LSP): Compatible for integration in any relevant context without affecting program correctness.
Dependency Inversion Principle (DIP): Relies on abstractions (e.g., enum for thread options), not specific implementations, paving the way for future enhancements.
**Open to Modifications 🔧 😁: **
I am completely open to any modifications or adjustments you deem necessary. My goal is to ensure that this solution meets our project's needs effectively and efficiently.
Location of the Class File 🤔:
Additionally, I would like your suggestion on where you prefer this class file to be placed within our project's repository. Your guidance on its optimal location would ensure seamless integration with our existing codebase.
I am looking forward to your valuable feedback and suggestions. Your input is crucial to refining this solution and ensuring it fulfills our project's requirements. Please feel free to reach out if you need any further information or clarifications.
Once again, thank you for considering my contribution 🙏.
Best regards 😊,
Gerson Jr.

Hi @gmottajr,
We sincerely appreciate your prompt response and commitment to addressing the issue. We are thrilled to have you on board and fully endorse your proposals. Let's proceed as follows:
Location of the Class File
To seamlessly integrate your ThreadManager into PROLEAD and PROLEAD_SW, we propose placing all relevant files under inc/Util and src/Util. This aligns with our convention of organizing components not directly tied to hardware or software versions in these directories. Additionally, for enhanced readability, we suggest creating two distinct files: ThreadManager.hpp and ThreadManager.cpp, each containing a single class.
For unit testing, we recommend adding a new file ut/Util/ThreadManager.cpp. Please keep in mind that we currently use Catch2 as unit testing framwork. The only reason for that decision was that we think that using google test is a bit of overkill for or small project. However, if converting the test cases from Google Test to Catch2 poses any challenges, please don't hesitate to reach out.
We genuinely appreciate your dedication to this endeavor. Your contribution is invaluable, and we're delighted to have such capable individuals actively participating in our project. 😊
Once again, thank you for your efforts.
Best regards,
Nico


feat!: rename event templates to 'cloudevents'

Changes
:broom: This commit renames the templates for cloudevents to be named "cloudevents".
BREAKING CHANGE
/kind enhancement
Fixes: https://github.com/knative-sandbox/kn-plugin-func/issues/578
Release Note
action required
When creating new function projects using the built in templates, you must now specify "cloudevents" instead of "events" to create an event based project.

// test/_e2e/trigger_events_test.go:61
func DefaultFunctionEventsTest(t *testing.T, knFunc *TestShellCmdRunner, project FunctionTestProject) {
	if project.Template == "events" && project.IsDeployed {

In Makefile recipe for pkged.go: there is also occurrence of /events/
https://github.com/lance/func/blob/lance/578-rename-events-template/Makefile#L66-L77


Fix ZCU102 support in Vivado shell project

The ZCU102 board is officially supported, but the board part setting was missing in the respective Tcl script template.

Thanks Felix!

Have you ever ran tfc_end2end_example.ipynb with ZCU102 board setting?

If I run the notebook, there is a stitch IP synthesis error which was not generated with ZCU104 setting.

I believe I tested this. Which error message are you seeing exactly?
This might be a Vivado license issue, as the ZCU102 part is not included in the WebPACK license.

Yes, you are right. It is a license issue.
Is there a way to replace the license to a purchased one?
Since I set VIVADO_PATH to my tool location, I thought Vivado uses my license.

Personally, I use the XILINXD_LICENSE_FILE environment variable to define the address of my license server, so I added the following line to my run-docker.sh script to set the same variable within the FINN docker environment:
DOCKER_EXEC+="-e XILINXD_LICENSE_FILE=$XILINXD_LICENSE_FILE "

Hi Felix,
Thanks for your reply.
I added the following lines on the run-docker.sh(please find the attached
file).
: ${XILINXD_LICENSE_FILE="/home/samuel/Downloads"}  #location of my license
file
DOCKER_EXEC+="-e XILINXD_LICENSE_FILE=$XILINXD_LICENSE_FILE "
However it didn't solve the license issue.
Stitch IP synthesis failed with the same reason.
Since I am using a USB dongle key(MAC address) for my license, I doubt that
Docker can't see the USB dongle key properly.
Sorry not familiar with Docker.
Regards,
Samuel
2021년 4월 30일 (금) 오후 7:06, Felix Jentzsch @.***>님이 작성:

Personally, I use the XILINXD_LICENSE_FILE environment variable to define
the address of my license server, so I added the following line to my
run-docker.sh script to set the same variable within the FINN docker
environment:
DOCKER_EXEC+="-e XILINXD_LICENSE_FILE=$XILINXD_LICENSE_FILE "
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Xilinx/finn/pull/280#issuecomment-829988708, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATZU3NKHWCJUKARQFYY74J3TLJ6K3ANCNFSM4XOOFV7A
.

Hi FADU-Sam, using USB dongle keys isn't something we support with FINN, but you may be able to work around this by either:

generating all the necessary project files etc inside Docker, then running actual synthesis on them manually on the host (all generated files are normally on /tmp/finn_dev_<username> mounted both in container and on the host)
trying to make the USB device directly visible inside the Docker container by adding more flags to the Docker launch commandline a la https://stackoverflow.com/questions/24225647/docker-a-way-to-give-access-to-a-host-usb-or-serial-device

Hi Maltanar,
Thanks for your suggestions.
I ran actual synthesis on my host under /tmp/finn_dev_samuel and the bitstream generation was successful.
However I don't know how to skip the below process and run the next instruction.

I also tried to make USB device visible to the Docker by adding following flag to the Docker launch sh file.
DOCKER_EXEC+="--device=/dev/bus/usb/003/004 --volume=/dev/bus/usb/003/004:/dev/bus/usb/003/004 "
Unfortunately it didn't work out too(maybe it was not a right flag).
Since Docker allows to change MAC address of its network bridge, I added the below flag to the sh file.
DOCKER_EXEC+="--mac-address xx:xx:xx:xx:xx:xx "
I expected the flag would make the Docker bridge having the same MAC address that my USB dongle has.
docker network inspect   reported that MAC address correctly got replaced.
But it didn't solve the synthesis issue and got the same license complain.
Vivado expects to see its license under /home//.Xilinx according to Vivado license manager.
I think somehow I need to include my license inside Docker and make Vivado to search the license under a specific directory.


Fix the URL to openbeer repository

This PR will fix the URL that was pointing to the wrong link.

@fredy-gadotti  Thanks for noticing and the patch. Cheers. Prost. Salud.


Use appropriate function call for chef 12

In chef12 the function locate_config_file in Chef::Knife was renamed
to chef_config_dir.  This patch adds a conditional to check for chef12
and apply the appropriate function call.

looks good to me

:+1:

@jkeiser How's this looking, anything blocking a review?

The code itself looks good, but:

Why are the 1.9.3 tests being removed?  We are trying to support the same versions as Chef, and AFAIK we haven't dropped support for it (I could be wrong, happy to be :)
Is GreenAndSecure included with Chef?  It looks like a plugin ... perhaps it's better to use Chef::VERSION so it'll run everywhere.

The problem I was seeing with the 1.9.3 test was:
Gem::InstallError: ohai requires Ruby version >= 2.0.0.
Has ohai dropped support for Ruby < 2?

OK.  Will merge and update travis to not run 1.9.3 anymore.  Thanks :)


Dropping an image into my form submits the form

I have an inplace summernote widget for leaving comments. If I drag and drop a picture onto the form, it appears in the widget, cool.  But in addition to uploading the image, it's also posting my form in the background and that's causing duplications of comments.  It's shows 404, but the comment is still appearing somehow (why is it submitting the form anyway?)
[08/May/2019 21:53:14] "POST /summernote/upload_attachment/ HTTP/1.1" 200 185
Not Found: /quests/submission/15/complete/
[08/May/2019 21:53:15] "POST /quests/submission/15/complete/ HTTP/1.1" 404 1772
[08/May/2019 21:53:15] "GET /media/django-summernote/2019-05-08/3485d494-cc80-4438-b769-67345ad9bdd8.png HTTP/1.1" 200 7085

My form doesn't visibly change except for the picture appearing.  But if I submit, it ends up being submitted twice (or if I cancel without submitting, it's already been submitted once).
Actually, If I keep dropping new images into the form, it submits the form (in secret) for every image.  So I end up with tons of extra comments!
Any idea why this might be happening or how I can prevent it?

I was able to work around the problem by checking the value of the submit button (which doesn't exist when summernote submits the form).  However, I still don't understand why summernote is submitting the form at all.

Sorry for bothering you. Could you record a screencast for this situation? (with logs of django runserver together)

https://youtu.be/Th1upRLjdic

Note that I am still stuck on django 2.0.x and django-summernote==<IP_ADDRESS> due to this problem: https://github.com/summernote/django-summernote/issues/344