Fix storageClassName in generated PV example

I was confused by the example, because it appeared to create a PV for itself, rather than for the EFS CSI Driver.
Looking at the source, it appears that the generated PV's storageClassName is efs-sc (for the EFS CSI Driver), not efs (for this CSI Driver).
Signed-off-by: Paul "Hampy" Hampson<EMAIL_ADDRESS>

The example in the README is an example of a PVC for this provisioner, not directly for the EFS CSI driver. In response to the PVC with StorageClass efs, this provisioner will create a PV with StorageClass efs-sc, bound to the requesting PVC and handled by the EFS CSI driver.

That's correct. The example I'm fixing is the second one, showing the generated PV.

You are absolutely right and I was confused. 🤦♂


Working with multiple scopes

I'm trying to work with multiple scopes at the same time and it works great for tagging and toggling the panel but it seems incomplete for other features like Telescope and status line.
Also, doesn't seem to work well when cycling with keymaps :Grapple cycle_forward scope=xxx. I couldn't make it to cycle with  2 different scopes using different keymaps.
Is this an expected flow? Am I doing something wrong? What's the original idea behind scopes?

I'm trying to work with multiple scopes at the same time and it works great for tagging and toggling the panel but it seems incomplete for other features like Telescope and status line.

The telescope integration is relatively simple atm. I don't use grapple with telescope personally, but would gladly accept a PR to improve the existing integration.

Also, doesn't seem to work well when cycling with keymaps :Grapple cycle_forward scope=xxx. I couldn't make it to cycle with 2 different scopes using different keymaps.

Would you mind giving a more concrete example? I can't seem to reproduce this on my side. If I have two scopes git and global, I am able to cycle through their tags just fine with:
Grapple cycle forward scope=git
Grapple cycle forward scope=global

Is this an expected flow? Am I doing something wrong? What's the original idea behind scopes?

Have you had the change to read through the Scopes documentation (or, :h grapple.nvim-scopes)? I've seen some confusion on this topic. I'd love to hear what does not make sense so that I can improve the documentation 🙂

I think I messed up with git and git_branches. My bad, it works fine.
Thanks for the quick reply and sorry for the inconveniences.


Init git-flow repo for non-default configuration

Getting this error when choosing production and next releases branches:
14:51:26.317: [live-portal] git -c core.quotepath=false flow init
Which branch should be used for bringing forth production releases?
   - master
   - staging
14:51:26.654: master
Branch name for production releases: [master]
Which branch should be used for integration of the "next release"?
   - staging
14:51:26.767: staging
Production and integration branches should differ.
Branch name for "next release" development: [master]

I actually already sorted this out in cbc70bb3a7f448257ea9e80c0b5d6155f21267db- the fix will be available in the upcoming version

Thanks @OpherV , will be waiting!

Available in 0.6.1

Updated, thanks!


Add loongarch64 support

Add loongarch64 support.  While it's not yet released, it won't hurt to merge this in advance. Thanks.


Add the project to the listing at https://openapi.tools/

Hi @larshp,
what do you think about adding the project to:
https://openapi.tools/
This is done as described in their CONTRIBUTING.md.
CU
Gregor

sure, feel free if you have time

Got this suggestion from https://twitter.com/ajmrdg who will give a SAP TechEd talk on "OpenAPIs in the SAP Ecosystem". I've brought this project to his attention.

cool 👍

Based on their categories.yml I would think the best fit is:

SDK Generators
Server Implementations

What do you think about:
- name: ABAP OpenAPI Client & Server Generator
  category:
    - sdk
    - server
  language: ABAP
  github: https://github.com/abap-openapi/abap-openapi
  link: https://abap-openapi.github.io/web-openapi-client/
  description:
    ABAP OpenAPI Client & Server Generator written in ABAP.
    Runs also in the Browser and on command line using NodeJS.
    Creates one self-contained global class + interface per OpenAPI definition.
  v2: true
  v3: true

v2: true, as of now there is no v2 support
"Runs also in the Browser and on command line using NodeJS." -> "Code generation runs also in the Browser and on command line using NodeJS."

Please cross-check https://github.com/gregorwolf/openapi.tools/blob/add-ABAP-OpenAPI/_data/tools.yml#L2052

looking good 👍

https://github.com/apisyouwonthate/openapi.tools/pull/363

It was merged on 4th of June 2023.


Fix contribute doc link

Two commits:

Because pnpm install was not creating node_modules in documentation folder.
Hotfix for broken links in docs for contribution github.

For point 2:

This solving issue #134
Idea for variable was from PR #138
It's not perfect solution, but working well for all scenarios for now.
The main rootcouse is in astro implementation itself probably. But I was not able to spend more time to dig into.
In past there was similar issue but with different method: https://github.com/withastro/astro/issues/4252
When looking for proper fix, comment my hotfix, replace variable in path to Astro.url.pathname and compare contribution links in docs pages in astro dev with astro build & astro preview

Have hope this is good PR for you ;)

Thank you!


Add the ability to right click in the VisualScript editor to open the generic search

This was in the original patch for VS, but there was some complaints about it. I don't remember the exact reasons. Please comment.

I am in favour of it.

This was in the original patch for VS, but there was some complaints about it. I don't remember the exact reasons.

At a guess, some might have wanted a contextual menu on right click, IMO should be an option for users to configure.

What @Megalomaniak said could be done later, this is just to fix the issue I always faced where nodes would just spawn wherever they wanted.

Should the existing node windows block the search?

This isn't up to date anymore, since #28447.

@Jummit Is this still desired? If so, it needs to be finished and rebased on the latest master branch.
If not, abandoned pull requests will be closed in the future as announced here.

I think I won't port it to master.


add Galaxy integration

The aim is to integrate PileOMeth into Galaxy and the Galaxy ToolShed.
@dpryan79 feel free to assign me to this issue.

I guess we can mark this as done now (I'll update the README here to reflect that). Thanks for writing it!


Problem getting Currents value with Hycomm grib 2 file

Hi! Great Library!
I can process most of my grib 1 and 2 files! This is working great!
But I came under a problem today as I was processing a grib2 file. To be more precise, I am extracting a message from a grib2(Hycomm)  and I am trying to get the Sea Current.
The problem is that the value seems to increase for no reason as the coordinates change in GeoSpatialValues and it even doesn't make any sense, the Sea Current is supposed to be in meter per second!
Here is the file I have a problem with...it contains only the U component of the current but I have the same problem with V component:
20170526120000.zip
I hope you can find the problem! If you want me to test anything or send you other files, just let me know!
Thanks!
Here is a snapshot of the output with the attached file...
 However, I found when working with U and V components I needed to set MultipleFieldMessages = true.
using (var file = new GribFile(ctx.GribFile))
{
file.Context.EnableMultipleFieldMessages = true;
}

@IdahoSixString thanks for the answer but it's not related to my problem. I already set this flag to true and I have both U and V components.
it's just that the data makes no sense as the value are increasing without any reason and is really too big.

This may be a bug in grib_api. I'll check.

Thanks a lot! I also asked my grib provider to check the data, just in case!

Hi!
Just  quick update to my problem. I received, as a test, the same data but in GRIB1 format and the data looks all good!
Unfortunately, the GRIB1 file is like 1.2GB !!! So for me it's really not an option for me!
But the problem seems to be related to GRIB2 only....
Still investigating...

Hi,
is there any way to create GRIB1 file by reading GRIB2 file?

@RajeshManam please create a new issue with your question. This makes it easier to manage and avoids spamming subscribers to this issue.
(Sorry for spamming the subscribers to this thread)

Created new issue, thank you for guidance.

@silv2015 Is this still an issue for you?

Sorry for the delay! Actually, I switch from Hycomm to MyOcean which gives me the same data that I need and the format I received from MyOcean is GRIB1.
So the problem is still there but it's no longer affecting me as I'm using another source.
Thanks a lot


Post Training quantization knowledge

Hi!
First of all, i'm running distiller on a GTX 750 Ti, i7 and 8gb ram.
I had made some tests on distiller (more specif on quantization) and i'm having some troubles to understand the results that you guys can see in #137, more specif about the test time on quantized model:

Time on pre-trained: 0.405s
Time on quantized: 6.814s

As we can see, the test time on pre-trained model is much slower than quantized, while distiller is proposed to increase inference speed.
In one research that my team had made, we found something very interesting that Neta wrote about one of his researches on distiller:

We understood that post training is on step 2, considering  redundant operations and thats cause the large time difference between the two model above.
That image can be applied to post training? And if the answer is positive, distiller already suport it? Or that is for coming next?
Thanks!

Hi @joaolcaas,
As it is today, Distiller only simulates post-training quantization. This allows us to study the effect of quantization on accuracy. It does not, unfortunately, provide insights on the actual runtime of the quantized model.
We did not implement low-level specialized operations that utilize 8-bit capabilities of the CPU/GPU.
Specifically, the way post-training quantization is implemented, supported layers are wrapped with quantize/de-quantizer operations, but the layers themselves are unchanged. They still operate on FP32 tensors, but these tensors are restricted to contain only integer values.
So, as you can see, we are only adding operations in order to simulate quantization. Therefore it isn't surprising at all that you're getting slower runtime when quantizing.
We do not have plans to implement "native" low-precision operations within Distiller. We do, however, plan to support exporting of quantized models using ONNX, once quantization is published as part of the standard. Then you'll be able to export a model quantized in Distiller and run it on a framework that supports actual 8-bit execution on CPU/GPU. In addition, 8-bit support in PyTorch itself is also in the works (it's already implemented in Caffe2). Once that's released, we'll see how we can integrate Distiller with that.
I'm sorry for the mis-understanding, and I hope this clears everything.

I see. Thank you!

Closing this issue as it's been use for a long time.


Bump the sample project to the new kotlin version

Coverage remained the same at 93.585% when pulling f1b3e734a0f69d783fcc07df6e3c07d434dfb4fa on UlfS:feature/bump-kotlin-version into 4f7451aaafacd1541abedc1bdfd362c738b505fe on ePages-de:master.


Allow to configure notebooks in AutoReload by default

Often i use a notebook in order to test a library that i'm developping.
It's very convenient when we have to display something etc....
The problem is that everytime i need to remember the way to enable autoreload mode in notebook. And everytime i have to search on the internet and do the right copy/past in order to enable this behavior.
It would be great if in the UI it was possible to enable it for the whole notebook :

It's a quite simple but very interesting feature !
Thanks a lot !

Thanks for filing thi sissue,
Are you referring to he autoreload magic command?

Yes exactly, it’s always two lines to add at the beggining of the cell but i have to search it everytime on the web.
Beeing able to set it up by default would improve debogging experience.
Envoyé à partir de Outlook pour iOShttps://aka.ms/o0ukef

De : Don Jayamanne @.>
Envoyé : Wednesday, November 1, 2023 4:49:09 AM
À : microsoft/vscode-jupyter @.>
Cc : François Ponchon @.>; Author @.>
Objet : Re: [microsoft/vscode-jupyter] Allow to configure notebooks in AutoReload by default (Issue #14478)
Thanks for filing thi sissue,
Are you referring to he autoreload magic command?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vscode-jupyter/issues/14478#issuecomment-1788364047, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGG5F7E65WZQZTGVOKFAI5LYCHBDLAVCNFSM6AAAAAA5ZDMOUGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBYGM3DIMBUG4.
You are receiving this because you authored the thread.Message ID: @.***>

Please use this setting to add the necessary commands that will be sent to the kernel when it starts.


EXPLORATION: Drop kernels not needed for mobilenet and measure bundle size

DO NOT MERGE.
Before:
$:~/Code/core/tfjs-core$ yarn bundle-size
...

~~Minified bundle size~~
112K dist/tf-core.min.js.gzip
465K dist/tf-core.min.js

~~Unminified bundle size~~
231K dist/tf-core.js.gzip
1.4M dist/tf-core.js

~~ESM bundle size~~
118K dist/tf-core.esm.js.gzip
513K dist/tf-core.esm.js

✨  Done in 39.71s.

After:
~~Minified bundle size~~
81K dist/tf-core.min.js.gzip
325K dist/tf-core.min.js

~~Unminified bundle size~~
184K dist/tf-core.js.gzip
1.1M dist/tf-core.js

~~ESM bundle size~~
86K dist/tf-core.esm.js.gzip
367K dist/tf-core.esm.js

Difference of .min.js.gzip = 31K = ~28% reduction in bundle size.

This change is

Dropping this PR.


Update user after tracking and being notified

@keybase/react-hackers
We want the latest state after a user hits track.
The most obvious approach is asking the service the about the user (through identify) after the user hits track. But that's racy because their is a whole notify flow that happens on the service side and so the latest info isn't available as soon as track returns.
Instead we wait for NotifyUsers.usersChanged and update the tracker information from there.
Edit: meant to put this in here:
before
after

Does usersChanged come from the 60 second timer or somewhere else? What's the approx usual latency between clicking Track and having the popup change?

@cjb forgot to add the gifs

I may have found a bug in the identify / notify stuff, or may be misunderstanding it.
but if I run KEYBASE_RUN_MODE=staging ./keybase untrack max && KEYBASE_RUN_MODE=staging ./keybase id --ui max
a couple of times without restarting the service, the identify call I make after being notified of a user change doesn't do any of the identify_ui calls like reportLastTrack or proof lists.
If I restart the service in between untrack and showing the tracking popup it works like the gifs above.
When it breaks, it looks like this (not updating any user state):

@patrickxb  / @maxtaco
Can I rely on being able to call identify.identify2 after receiving keybase.1.NotifyUsers.userChanged, or is there some other way I should be asking for the latest user info?

@MarcoPolo it might be a question of caching, does that seem right?  If the identification is cached, then it short-circuits the whole spew of messages you might be expecting. What if you set needProofSet=true?

@maxtaco even with needProofSet=true (also tried forceRemoteCheck=true just in case), but neither of those call any messages on the delegate UI.
Shouldn't I be at least getting the reportLastTrack message even if it short circuits?
I do get the user plus keys after the identify call though, but I need the other stuff.

that's a surprise. urg
On Fri, Mar 11, 2016 at 2:00 PM, Marco Munizaga<EMAIL_ADDRESS>wrote:

@maxtaco https://github.com/maxtaco even with needProofSet=true (also
tried forceRemoteCheck=true just in case), but neither of those call any
messages on the delegate UI.
Shouldn't I be at least getting the reportLastTrack message even if it
short circuits?
I do get the user plus keys after the identify call though, but I need the
other stuff.
—
Reply to this email directly or view it on GitHub
https://github.com/keybase/client/pull/2291#issuecomment-195499109.

Can you show me a service log from when this is happening?
On Fri, Mar 11, 2016 at 2:01 PM, Maxwell Krohn<EMAIL_ADDRESS>wrote:

that's a surprise. urg
On Fri, Mar 11, 2016 at 2:00 PM, Marco Munizaga<EMAIL_ADDRESS>wrote:

@maxtaco https://github.com/maxtaco even with needProofSet=true (also
tried forceRemoteCheck=true just in case), but neither of those call any
messages on the delegate UI.
Shouldn't I be at least getting the reportLastTrack message even if it
short circuits?
I do get the user plus keys after the identify call though, but I need
the other stuff.
—
Reply to this email directly or view it on GitHub
https://github.com/keybase/client/pull/2291#issuecomment-195499109.

Definitely have to figure out this lag, but the electron code looks fine :+1:


decl_event

in progress here: https://hackmd.io/s/rkzKBt7K4

Actually, Guillaume has kindly already done this: https://crates.parity.io/srml_support/macro.decl_event.html
Marking as complete


GO annotations being "projected" by Ensembl back onto same species?

Should IEA GO annotations be "projected" by Ensembl back onto same species? e.g.  Q4VAE3
Tmem65 (mouse) GO:0003231
cardiac ventricle development IEA
GO_REF:0000107
with/from field: UniProtKB:Q4VAE3|ensembl:ENSMUSP00000071984
Which appears to be the same gene in the same species, apparently inferred from an IMP annotation of the same gene?
Shouldn't these projections only be inferred for homologs in a different species? Or am I missing something?

Please move this to the annotation tracker.
https://github.com/geneontology/go-annotation

@Pauldenny - you can close this one when you've opened the item in the annotation tracker.  Thanks.

Moved to the annotation tracker. https://github.com/geneontology/go-annotation/issues/1488


I2S on STM32H7

Currently I2S is not supported on STM32H7 chips. The driver i2s_ll_stm32.c does not build for stm32h7 and throws a few linker errors as mentioned in #45955
Describe why you are asking for this support?
I am using a custom board with STM32H753IIK. The i2s2 bus on the board is connected to an audio codec chip. I would like to be able to record and playback audio in i2s slave mode. I have modified the i2s driver based on the comments available in #45955. The driver builds successfully but when I run a sample application I am facing the below problem.
For the moment I am trying to only playback audio but dma_tx_callback does not get called. I checked this issue #37038 where the problem was not using the PLL clock on a F4 chip but in my case I have enabled the PLL clock also.
I am using Zephyr 3.4.99
Below are the changes made to the i2s_ll_stm32 driver
#if 1  // <---- enabled this
#define DCACHE_INVALIDATE(addr, size) \
...

#define SPI_STM32_DMA_TX	0x01
#define SPI_STM32_DMA_RX	0x02
static inline uint32_t *ll_func_dma_get_reg_addr(SPI_TypeDef *spi, uint32_t location)
{
#if (CONFIG_SOC_SERIES_STM32H7X)
	if (location == SPI_STM32_DMA_TX) {
		/* use direct register location until the LL_SPI_DMA_GetTxRegAddr exists */
		return &(spi->TXDR);
	}
	/* use direct register location until the LL_SPI_DMA_GetRxRegAddr exists */
	return &(spi->RXDR);
#else
	ARG_UNUSED(location);
	return LL_SPI_DMA_GetRegAddr(spi);
#endif /* st_stm32h7_spi */
}

static int rx_stream_start(struct stream *stream, const struct device *dev)
{
...
	ret = start_dma(stream->dev_dma, stream->dma_channel,
			&stream->dma_cfg,
			/*(void *)LL_SPI_DMA_GetRegAddr(cfg->i2s),*/
			ll_func_dma_get_reg_addr(cfg->i2s, SPI_STM32_DMA_RX),
			stream->src_addr_increment, stream->mem_block,
			stream->dst_addr_increment, stream->fifo_threshold,
			stream->cfg.block_size);
...
}

static int tx_stream_start(struct stream *stream, const struct device *dev)
{
...
	ret = start_dma(stream->dev_dma, stream->dma_channel,
			&stream->dma_cfg,
			stream->mem_block, stream->src_addr_increment,
			/*(void *)LL_SPI_DMA_GetRegAddr(cfg->i2s),*/
			ll_func_dma_get_reg_addr(cfg->i2s, SPI_STM32_DMA_TX),
			stream->dst_addr_increment, stream->fifo_threshold,
			stream->cfg.block_size);
...
}

static int rx_stream_start(struct stream *stream, const struct device *dev) and static int tx_stream_start(struct stream *stream, const struct device *dev)
{
...
#if (CONFIG_SOC_SERIES_STM32H7X)
	LL_SPI_EnableIT_UDR(cfg->i2s);
	LL_SPI_EnableIT_OVR(cfg->i2s);
	LL_SPI_EnableIT_FRE(cfg->i2s);
#else
	LL_I2S_EnableIT_ERR(cfg->i2s);
#endif
...
}

static void rx_stream_disable(struct stream *stream, const struct device *dev) and static void tx_stream_disable(struct stream *stream, const struct device *dev)
{
...
#if (CONFIG_SOC_SERIES_STM32H7X)
	LL_SPI_DisableIT_UDR(cfg->i2s);
	LL_SPI_DisableIT_OVR(cfg->i2s);
	LL_SPI_DisableIT_FRE(cfg->i2s);
#else
	LL_I2S_DisableIT_ERR(cfg->i2s);
#endif
...
}

Here is my dts
/ {
	soc {
		i2s2: i2s2@40003800 {
			compatible = "st,stm32-i2s-custom";
			#address-cells = <1>;
			#size-cells = <0>;
			reg = <0x40003800 0x400>;
			clocks = <&rcc STM32_CLOCK_BUS_APB1 0x00004000>,
					<&rcc STM32_SRC_PLL1_Q SPI123_SEL(0)>;
			interrupts = <36 0>;
			status = "disabled";
		};
	};
};

&pinctrl {
	/omit-if-no-ref/ i2s2_mck_pc6: i2s2_mck_pc6 {
		pinmux = <STM32_PINMUX('C', 6, AF5)>;
		slew-rate = "very-high-speed";
	};

	/omit-if-no-ref/ i2s2_sdi_pi2: i2s2_sdi_pi2 {
		pinmux = <STM32_PINMUX('I', 2, AF5)>;
		slew-rate = "very-high-speed";
	};

	/omit-if-no-ref/ i2s2_sdo_pi3: i2s2_sdo_pi3 {
		pinmux = <STM32_PINMUX('I', 3, AF5)>;
		slew-rate = "very-high-speed";
	};
};

i2s_rxtx: &i2s2 {
	status = "okay";
	pinctrl-0 = <&i2s2_ck_pi1 &i2s2_mck_pc6 &i2s2_sdi_pi2 &i2s2_sdo_pi3 &i2s2_ws_pb12>;
	pinctrl-names = "default";
	dmas = <&dmamux1 0 40 (STM32_DMA_PERIPH_TX | STM32_DMA_PRIORITY_HIGH)
			&dmamux1 1 39 (STM32_DMA_PERIPH_RX | STM32_DMA_PRIORITY_HIGH)>;
	dma-names = "tx", "rx";
};

Below are the sample application codes. It was made by modifying the zephyr/samples/drivers/i2s/echo sample. This application along with my codec driver has been tested on nrf5340dk_nrf5340_cpuapp board. The application reads audio data from a file and send to the i2s driver.
...
// codec init and settings are done here
...
...
    config.word_size = SAMPLE_BIT_WIDTH;
	config.channels = NUMBER_OF_CHANNELS;
	config.format = I2S_FMT_DATA_FORMAT_I2S;
	config.options = I2S_OPT_BIT_CLK_SLAVE | I2S_OPT_FRAME_CLK_SLAVE;
	config.frame_clk_freq = SAMPLE_FREQUENCY;
	config.mem_slab = &mem_slab;
	config.block_size = BLOCK_SIZE;
	config.timeout = TIMEOUT;
	if (!configure_streams(i2s_dev_rx, i2s_dev_tx, &config)) {
		return -1;
	}
...
...
// Loop
				size_t block_size = BLOCK_SIZE;
				void *buf = calloc(1, block_size);
				if (buf == NULL) {
					LOG_ERR("calloc failed ad %d bytes!!", block_size);
					break;
				}
				rc = fs_read(&fi->file, buf, block_size);
				if (rc <= 0) {
					free(buf);
					LOG_INF("no more data in file");
					break;
				}

//				ret = i2s_buf_write(i2s_dev_tx, buf, block_size);
				ret = i2s_write(i2s_dev_tx, buf, block_size);
				if (ret < 0) {
					free(buf);
					LOG_ERR("Failed to write data: %d\n", ret);
					err = 1;
					break;
				}

				free(buf);
...

My requests are:

Can stm32h7 support be added to the i2s driver? I have limited experience in the lower layer drivers.
Can someone tell me what am I doing wrong and what else I should be trying?

Hi,
We have no short term plan to extend I2S support, but here are some hints:

Did you enabled dmamux and dma nodes in dts ? See SPI example
Have a check on SPI driver history (spi_ll_stm32.c, spi_ll_stm32.h)
Check dts pll configuration. You're using PLL1_Q as clock input. You need to be sure it is correctly configured vs your expectations.
Ask @endian-benjamin. You already did it on #45955 but let's continue the discussion here if he jumps in.

Good luck

Hi @erwango,

Yes, the dmamux and dma nodes are enabled

&dma1 {
	status = "okay";
};
&dma2 {
	status = "okay";
};
&dmamux1 {
	status = "okay";
};

Below is the PLL configuration. I believe it is correct

&clk_hse {
	hse-bypass;
	clock-frequency = <DT_FREQ_M(24)>; /* 24MHz clock */
	status = "okay";
};
&pll {
	div-m = <2>;
	mul-n = <80>;
	div-p = <2>;
	div-q = <16>;
	div-r = <2>;
	clocks = <&clk_hse>;
	status = "okay";
};
&rcc {
	clocks = <&pll>;
	clock-frequency = <DT_FREQ_M(480)>;
	d1cpre = <1>;
	hpre = <2>;
	d1ppre = <2>;
	d2ppre1 = <2>;
	d2ppre2 = <2>;
	d3ppre = <2>;
};

I tried running the spi_loopback sample using nucleo_h753zi dts on my custom board, but got the error code -14. The buf_in_nocache function returns false. Any idea what is the problem?
Thanks

I tried running the spi_loopback sample using nucleo_h753zi dts on my custom board, but got the error code -14. The buf_in_nocache function returns false. Any idea what is the problem?

You should enable CONFIG_NOCACHE_MEMORY=y


Merge overlapping candidate spans in DictionaryMatcher

Similar to longest_matchflag, can we have a merge_contiguous_match flag to deal with (1) overlapping matches and (2) contiguous matches. This happens pretty frequently.
Overlapping Example
left shoulder
      shoulder blade
left shoulder blade
Contiguous Example
hypothalamic
                       abnormalities
hypothalamic abnormalities

@jason-fries The first example seems good to cover, although slightly non-trivial to add... the latter technically is covered with Concat, but agreed as well


injectedModule bug: "Cannot assign to read only property 'request' of object '[object Object]'"

Current Behavior
The dApp crashes. The log output is below.
Expected Behavior
No response
Steps To Reproduce

I am using react-create-app with Craco
Injected is the only module being used

import { init, useConnectWallet } from "@web3-onboard/react";
import injectedModule from '@web3-onboard/injected-wallets';

const apiKey = process.env.REACT_APP_BLOCKNATIVE_KEY;

const injected = injectedModule();

init({
  apiKey,
  wallets: [injected],
  chains: [
    {
      id: 1,
      token: 'ETH',
      label: 'Ethereum Mainnet',
      rpcUrl: CONSTANTS.RPC_URLS[1],
    },
    {
      id: 8453,
      token: 'ETH',
      label: 'Base',
      rpcUrl: CONSTANTS.RPC_URLS[8453],
    }
  ]
});

What package is effected by this issue?
@web3-onboard/injected
Is this a build or a runtime issue?
Runtime
Package Version
2.10.12
Node Version
18.17.1
What browsers are you seeing the problem on?
Chrome
Relevant log output
Cannot assign to read only property 'request' of object '[object Object]'
TypeError: Cannot assign to read only property 'request' of object '[object Object]'
    at createEIP1193Provider (http://localhost:3000/static/js/bundle.js:337890:20)
    at http://localhost:3000/static/js/bundle.js:349171:99
    at checkFor6963Providers (http://localhost:3000/static/js/bundle.js:349178:10)
    at injected (http://localhost:3000/static/js/bundle.js:349186:86)
    at ./src/config/Layout/index.js (http://localhost:3000/static/js/bundle.js:11284:93)
    at options.factory (http://localhost:3000/static/js/bundle.js:370923:31)
    at __webpack_require__ (http://localhost:3000/static/js/bundle.js:370311:33)
    at fn (http://localhost:3000/static/js/bundle.js:370580:21)
    at ./src/config/App.js (http://localhost:3000/static/js/bundle.js:10770:65)
    at options.factory (http://localhost:3000/static/js/bundle.js:370923:31)

Anything else?
This is my craco.config.js:
const webpack = require('webpack')

module.exports = {
  webpack: {
    configure: {
      resolve: {
        fallback: {
          path: require.resolve('path-browserify')
        },
        alias: {
          assert: 'assert',
          buffer: 'buffer',
          crypto: 'crypto-browserify',
          http: 'stream-http',
          https: 'https-browserify',
          os: 'os-browserify/browser',
          process: 'process/browser',
          stream: 'stream-browserify',
          util: 'util'
        }
      },
      experiments: {
        asyncWebAssembly: true
      },
      plugins: [
        new webpack.ProvidePlugin({
          process: 'process/browser',
          Buffer: ['buffer', 'Buffer']
        })
      ]
    }
  }
}

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.

@BenAzlay please have a look at our react demo project that utilizing craco - https://github.com/blocknative/react-demo

@Adamj1232 Your react demo uses React App Rewired and not Craco. Nevertheless I tried replacing Craco with React App React on my dApp and now the behavior is as follows:

The dApp crashes for a split second with the "Cannot assign to read only property 'request' of object '[object Object]'" error before it goes back to normal
That error still appears in the console

@BenAzlay ah you are correct, we used to have a CRACO application but we did update to rewired on the react-demo project. If you look at the config-overrides here https://github.com/blocknative/react-demo/blob/master/config-overrides.js that should get your app up an running.
@ayepRahman Im not sure why you have a ?. there but please also have a look at the react project for proper initialization or checkout our examples folder - https://github.com/blocknative/web3-onboard/tree/develop/examples

@Adamj1232 initially I followed the example without the ?.. The example shows nextjs using page routing instead of app routing.

downgrading from 2.10.12 to 2.8.5 doest throw the error

@ayepRahman is that the max version before errors present?

@Adamj1232 I am using the exact same config-overrides.js as you. Here is my services.js:
import { init } from '@web3-onboard/react'
import injectedModule from '@web3-onboard/injected-wallets'
import walletConnectModule from '@web3-onboard/walletconnect'
import coinbaseModule from '@web3-onboard/coinbase'
import gnosisModule from '@web3-onboard/gnosis'
import metamaskModule from '@web3-onboard/metamask'
import { CONSTANTS } from '../utils/constants'

const apiKey = process.env.REACT_APP_BLOCKNATIVE_KEY;

const injected = injectedModule();

const coinbase = coinbaseModule();

const walletConnect = walletConnectModule({
    projectId: process.env.REACT_APP_WALLETCONNECT_PROJECT_ID,
    // dappUrl: 'https://reactdemo.blocknative.com/'
});

const gnosis = gnosisModule()
const metamask = metamaskModule({
    options: {
        extensionOnly: false,
        i18nOptions: {
            enabled: true
        },
        dappMetadata: {
            name: 'Web3Onboard React Demo'
        }
    }
})

export const initWeb3Onboard = init({
    connect: {
        autoConnectAllPreviousWallet: true
    },
    wallets: [
        metamask,
        injected,
        walletConnect,
        coinbase,
        gnosis,
    ],
    chains: [
        {
            id: 1,
            token: 'ETH',
            label: 'Ethereum',
            rpcUrl: CONSTANTS.RPC_URLS[1],
        },
        {
            id: 8453,
            token: 'ETH',
            label: 'Base',
            rpcUrl: CONSTANTS.RPC_URLS[8453]
        },
    ],
    appMetadata: {
        name: 'My dApp Name',
        description: 'My dApp description',
        recommendedInjectedWallets: [
            { name: 'Coinbase', url: 'https://wallet.coinbase.com/' },
            { name: 'MetaMask', url: 'https://metamask.io' }
        ],
    },
    accountCenter: {
        desktop: {
            enabled: true,
            position: 'topRight',
        },
        mobile: {
            enabled: true,
            position: 'topRight'
        }
    },
    apiKey,
    theme: 'dark'
})

Then in my layout.js I'm using init like this:

  const [{ wallet, connecting }, connect, disconnect] = useConnectWallet();

  const [web3Onboard, setWeb3Onboard] = useState(null);

  useEffect(() => {
    setWeb3Onboard(initWeb3Onboard)
  }, []);

  const onClickConnect = () => {
    if (wallet) disconnect(wallet);
    else {
      connect();
    }
  }

Again, now the dApp doesn't crash anymore, and connection with injected wallet works as well, but the error is still in the console:

@BenAzlay what version of node? Are you using npm, yarn?

@Adamj1232 I'm using npm, and Node v18.17.1 (as mentioned in my initial comment)

I am experiencing the same problem - it works fine, but throws the error in the console.

@BenAzlay which wallets do you have added to your browser?

There is a fix for this available at the latest alpha version of the injected module. Thank you for reporting!

There is a fix for this available at the latest alpha version of the injected module. Thank you for reporting!

I confirm this fixed it (updating @web3-onboard/injected-wallets to version 2.10.14-alpha.1)! Thank you so much!


display_name of Workflow Title overwritten by logged User's display_name on LIVE project

I believe this is the correct project to report this issue against. If not, please advise.
While waiting an extremely long time to hear back on our request for beta approval, I've observed an issue which is significant.
I've seen the use of display_name across classes result in overwriting of the User email setting and during workflow editing the Workflow Title is being regularly overwritten. This overly aggressive automatic field update is annoying but as long as I change the Workflow Title field to the intended name and quickly leave the page before it is rewritten, no problem. Since this workaround let me get on with our project prototype, no problem.
With our workflow ready for a shakedown cruise we set it to LIVE and applied for Beta status. That's been going on 10 days ago with no action on the approval.
When a test user reported an issue, I poked around our projects to confirm everything was as it should be. In doing this I visited the DISABLED Workflow page in our Project and watched as the display_name field was overwritten by my User display_name. Okay, I thought... probably cosmetic. But to be sure, I did a quick classification and as you see in this screenshot, the ACTUAL workflow name was changed on a live and uneditable workflow!?

I can do any requested investigation needed. I am not yet running a local development instance as I am on Windows and don't see any clear instructions on how to set up a Windows-based dev box.
-: Jim :-

Hi @Jim-Salmons, the best repo is probably https://github.com/zooniverse/Panoptes-Front-End as this repo is the API backing the front end client.
I can't reproduce the autosave problem you are experiencing. I'm on Chrome OS X and Linux and the update only fires if there are changes and i change the focus away from the text box. I wonder if this is a bug on windows. Can you please report the OS and browser combo you are experiencing the problem under? Pinging @brian-c for thoughts on this.
Also @mrniaboc, @aliburchard any timeline on the factminer beta review?

@camallen, thanks for the reply. Will leave this issue here for the moment. You core folks can decide if it needs to move to or be referenced from the front-end repo. (The repo descriptions are very minimal BTW, so a bit of a comment in the respective readme files might make the distinction between between front- and back-end repos more clear for issue reporting.)
I am on a fully updated Win 8.1. And I observed the display_name overwriting in both up-to-date Chrome and Firefox. While the overwriting on a supposed disabled webform was an eye-opener, this behavior was driving me nuts during initial workflow authoring.
@mrniaboc and @aliburchard, I have canceled the FactMiners beta for the moment to nail down some interaction issues and streamline the instructions with a "before you begin" video piece. I'll let you know directly when I resubmit for the beta.

I bet the title field is being overwritten by an autofill plugin. @Jim-Salmons it looks like your LastPass plugin has identified a few matches on zooniverse.org. Could one of them have been associated with the wrong field somehow? Mind turning it off for a moment to see if the problem persists?

@brian-c It appears that LastPass was the issue! :+1: I just made the tweak and it appears to be settled down. Will update if this is not the case, but here's what I did.
In LastPass I located my Zooniverse logons -- my project-wise admin account, FactMiners, and my personal one -- and went to the Advanced Settings (default hidden) for each logon and checked the "Disable AutoFill" checkbox and saved the entry.
Since the logged-on User account field uses the display_name field which is the same fieldname as a variety of Zooniverse objects, LastPass sees log-in fields where they are not intended.
Considering how many users might have password managers running, it might be reasonable to store the User account's display_name with that database column name, but consider using 'screenname' or 'nickname' in the upper-right logged-on User display and log-on widget-ids. Something that would reduce the chance of to these overwrites happening. If this 'gotcha avoidance' approach is not doable, perhaps a FAQ entry or Wiki comment would be useful. I can write something up w/ screenshots to submit if there is a proper place for such Project Builder docs. (Sorry, I'm new to the community and there are LOTS of repos here, so I don't know the ropes yet.)
Here's a screenshot to clarify...


JSON parser errors with scoop

Bug Report
duplicati-client<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>groff, msys2@2016-10-25, <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>processing<EMAIL_ADDRESS>python@2, <EMAIL_ADDRESS>python@3, <EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>python@mini, resourceHacker<EMAIL_ADDRESS>rsync<EMAIL_ADDRESS><EMAIL_ADDRESS>Current Behaviour
JSON errors if I do a "scoop search x" with this bucket enabled.

Expected Behaviour
No JSON errors.

Additional context/output
WARN  Error parsing JSON at C:\Users\Pulaski.scoop\buckets\anderlli0053_DEV-tools\bucket\duplicati-client.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>+x32.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at C:\Users\Pulaski.scoop\buckets\anderlli0053_DEV-tools\bucket\groff.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at C:\Users\Pulaski.scoop\buckets\anderlli0053_DEV-tools\bucket\processing.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at C:\Users\Pulaski.scoop\buckets\anderlli0053_DEV-tools\bucket\resourceHacker.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at C:\Users\Pulaski.scoop\buckets\anderlli0053_DEV-tools\bucket\rsync.json.
WARN  Error parsing JSON at<EMAIL_ADDRESS>WARN  Error parsing JSON at<EMAIL_ADDRESS>
Possible Solution

System details
Windows version: [e.g. 7, 8, 10] 10
OS architecture: [e.g. 32bit, 64bit] 64bit
PowerShell version: [output of "$($PSVersionTable.PSVersion)"]
PS C:\Users\pulaski> $($PSVersionTable.PSVersion)
Major  Minor  Build  Revision

5      1      19041  1682
Additional software: [(optional) e.g. ConEmu, Git]
Scoop Configuration

//# Your configuration here

@shakeyourbunny  Hmm.. thanks for the report.
When i try to search with command scoop search x it doesn't return any warning nor error while parsing / searching in this bucket.  It returns warnings in other ones. I checked some of the mentioned manifests in my bucket and all looks fine, maybe there are some foreign path names and other things in the manifests that cause this. I tried to install some of the mentioned apps and they work as they should.

EDIT:  I found out that since this bucket fetches some manifests from other buckets, the said "warning corruption" happens there also.
So there is nothing wrong with the apps / manifests that you are warned about, they still work just fine, it just happens because of some comments inside JSON manifests, usually starting with //.  Now that comments are what different devs use to write down what the code does, and since this bucket is self-updating and fetching, even if i remove all the comments in the manifests, it will only work until the next BOT update (every 6 or so hours) or even when the new version of the specified erroneus app is released and then detected and fetched from the upstream BOT, which will overwrite all the comments and erroneus elements in manifest.

Well, I found that in groff there IS a comment.
https://github.com/anderlli0053/DEV-tools/blob/840057c09e6caa54be9374204bdf7dbdf3a0cce3/bucket/groff.json#L13
My guess is that if this would be filtered out by your update checker, would be neat.
On the shell, I could catch these comments which produces the errors with:
grep "//" *.json | egrep -v "http(|s)://|note:|regex|checkver|UpdateUrl|appxbundle" | egrep "\{|\}|\[|\]"

There is also a Github project which directly strips off all comments, making it valid JSON again.
On the first look it looks empty, but you have to select the branch with the language of your choice to get it.
https://github.com/getify/JSON.minify/tree/master

I'm trying to achieve something with "grepWin" tool on Windows. I'll see what can i do with RegEx 🙂

I wrote a small python script, which cleans up the JSON files and does a syntactical check / linting of them, you can integrate it if you want.
https://gist.github.com/shakeyourbunny/48716963e2eca4adebb129b8995f2c39
Note that you have to install the json_minify python module (pip3 install json_minify) and edit the source path on line 26.
The script ran through and cleaned up the comments, but it also has found some other things that made the JSON not usable:

there are some JSONs with weird encoding or are even incomplete/truncated (cabal.json, processing.json which made the files unusable, ignored by the script)
there is a single JSON ALAC.json which has a weird byte order mark (corrected by the script)
a certain author called Andrew Pozenel thinks that a string in JSON may go over multiple lines and tabulators are allowed (which is not allowed). Some of the files are listed in the python script, but is automatically corrected now.

After all that cleaning up the output of scoop now is more or less fixed:
 scoop search obs
WARN  Error parsing JSON at C:\Users\Pulaski\.scoop\buckets\zzz_anderlli0053\bucket\cabal.json.
WARN  Error parsing JSON at C:\Users\Pulaski\.scoop\buckets\zzz_anderlli0053\bucket\processing.json.
Results from local buckets...

Name               Version Source           Binaries
----               ------- ------           --------
obs-browser-plus   3.22.5  zzz_anderlli0053
obs-studio-small   21.1.2  zzz_anderlli0053
obs-studio         27.2.4  zzz_anderlli0053
obsidian           0.15.8  zzz_anderlli0053
OBSPortable        27.2.4  zzz_anderlli0053
OBSStudio-Portable 27.2.4  zzz_anderlli0053
obsutil            5.4.6   zzz_anderlli0053
streamlabs-obs     1.10.0  zzz_anderlli0053

It would be great if your update scripts ("robot") would do a cleanup after rounding up the updates :-)

@shakeyourbunny
Hey thanks for the Python script, although i get an error running python3 cleanup-manifests.py, inside "bucket" folder and got these errors
Traceback (most recent call last):
  File "C:\Users\ander\Desktop\DEV-tools\bucket\cleanup-manifests.py", line 60, in <module>
    cleanup_json(files.path)
  File "C:\Users\ander\Desktop\DEV-tools\bucket\cleanup-manifests.py", line 53, in cleanup_json
    json_dict = json.loads(json_clean)
  File "C:\Program Files\Python310\lib\json\__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "C:\Program Files\Python310\lib\json\decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "C:\Program Files\Python310\lib\json\decoder.py", line 353, in raw_decode
    obj, end = self.scan_once(s, idx)
PS C:\Users\ander\Desktop\DEV-tools\bucket>

I'm guessing it kind of worked, since when i run it it comes to the " -><EMAIL_ADDRESS>and then the script errors out for some reason.. Hmm..

The JSON is invalid and could not be processed.
https://github.com/anderlli0053/DEV-tools/blob/1a9c920e92526c711a12201d82962689ef3fe7ab/bucket/perl%4<IP_ADDRESS>%2Bgcc.json#L8
see https://codebeautify.org/jsonvalidator/y22e3bc37
In a nutshell:
	"hash": "sha256:9da0cc11166e13ce0a9077cb0b42c4909ed849647c094f71aebb2eaf7550bd68",}
````
There is a , at the end of the line. which is not legal in JSON.

@shakeyourbunny  I had to modify your script a little as i was getting escape sequence error and also others, no big deal
i had to explicitly specify/import the path from the "os" module:
from os import path
( I don't know why the interpreter doesn't want to import the whole module classes, at least for me and for the path thing)
and set the clean path like so clean_directory = (r"C:\\Users\\ander\\Desktop\\DEV-tools\\bucket") so it takes raw input and double backslashes to work correctly.
I've even reset the python3 by completely clean-reinstalling it but it still works strangely like that... at least it works 😄
Console throws no more warnings for this bucket at least, please run scoop update and scoop search x and then test if there are some other warnings that show on your PC, but not on mine.
Thank you for your script
P.S.:
I'm keeping this open, until you report back about the status.

seems to work for the most part.
PS C:\Windows\System32\WindowsPowerShell\v1.0> scoop update
Updating Scoop...
Updating 'main' bucket...
 * df663b2cd simplex-chat: Update to version 3.1.0                       5 minutes ago
Updating 'zzz_anderlli0053' bucket...
Updating files: 100% (3021/3021), done.
 * 0f391276 Update cleanup-manifests.py                                  3 minutes ago
 * c8b30a69 Various fixes                                                26 minutes ago
Scoop was updated successfully!
PS C:\Windows\System32\WindowsPowerShell\v1.0> scoop search obs-studio
WARN  Error parsing JSON at C:\Users\Pulaski\.scoop\buckets\zzz_anderlli0053\bucket\processing.json.
Results from local buckets...

Name             Version Source           Binaries
----             ------- ------           --------
obs-studio-small 21.1.2  zzz_anderlli0053
obs-studio       27.2.4  zzz_anderlli0053

the thing with processing.json is to be expected, but I'm curious why your Python installation freaked out.
Thank you for your efforts!

seems to work for the most part.
PS C:\Windows\System32\WindowsPowerShell\v1.0> scoop update
Updating Scoop...
Updating 'main' bucket...
 * df663b2cd simplex-chat: Update to version 3.1.0                       5 minutes ago
Updating 'zzz_anderlli0053' bucket...
Updating files: 100% (3021/3021), done.
 * 0f391276 Update cleanup-manifests.py                                  3 minutes ago
 * c8b30a69 Various fixes                                                26 minutes ago
Scoop was updated successfully!
PS C:\Windows\System32\WindowsPowerShell\v1.0> scoop search obs-studio
WARN  Error parsing JSON at C:\Users\Pulaski\.scoop\buckets\zzz_anderlli0053\bucket\processing.json.
Results from local buckets...

Name             Version Source           Binaries
----             ------- ------           --------
obs-studio-small 21.1.2  zzz_anderlli0053
obs-studio       27.2.4  zzz_anderlli0053

the thing with processing.json is to be expected, but I'm curious why your Python installation freaked out.
Thank you for your efforts!

I will look at the processing.json to see what the problem is and no,
Thank you for writing the Python script and even opening the issue about multiple erroneus manifests at once, and not one-by-one.


Processing history

Add processing history to criterias. This will help to see how many iterations a criteria has been through before it was accepted.

Released in v1.1.0


Transpilation of async uses es6 Promise without polyfill

Async/await code doesn't provide the Promise polyfill when transpiling into ECMASCRIPT5 or older.
For example, Internet Explorer breaks when it tries to run over transpiled async with the following error
ReferenceError: 'Promise' is undefined
    at $jscomp.executeAsyncGenerator (foo.min.js:120)

You can see from the transpiled code here that the polyfill is not provided.

@patstuart
Thanks for the report.
You should specify a language input mode of ECMASCRIPT_NEXT and --rewrite_polyfills in order to get the polyfills needed for async/await.
If you look at the warnings tab on your example link you'll see this.
JSC_PARSE_ERROR: Parse error. this language feature is only supported for ECMASCRIPT8 mode or better: async function at line 2 character 0
async function foo() {
^
JSC_PARSE_ERROR: Parse error. this language feature is only supported for ECMASCRIPT8 mode or better: async function at line 3 character 2
  await bar();
  ^

Brad,
Thanks. You'll note that I am already compiling with the ECMASCRIPT8, and the service produces the same error occurs with your alternative syntax. I don't know why, since I do not get any warnings on my own machine. I am using the latest compiler.
Regardless, the problem is not that the compiler can't handle async. It's that it handles async, but it doesn't automatically add the polyfill: $jscomp.polyfill("Promise", ...
My temporary workaround is to make reference to a promise elsewhere in the code, which forces the compiler will add the polyfill. E.g., window.a = new Promise();
Here are my compiler options as proof:
--language_in ECMASCRIPT_NEXT
--language_out ECMASCRIPT5
--rewrite_polyfills
--compilation_level SIMPLE

Let me know if there is anything else I can add which could help.

@shicks Could some of your recent changes be preventing the Promise polyfill being included here?

I'm trying to reproduce this with a runtime test.

thank you!


Fixed "fraction of crops for animal products"

The new equation, if correct, requires recalibration of the model.

This pointed the way to the real issue with Feed based inflation not making sense.  Closing this so that we can fix directly.


[FLINK-18671][docs] Update upgrade compatibility table for 1.11.0

This is a simple documentation update. The documented behaviour is ensured via all migration tests that we have in Flink.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Thanks for the review. Yes, the table looks good in the html version. Columns are getting a bit narrow, but I think it's still ok - for a next version we will need to consider changing something here.


Check for log directory on startup if file logging is enabled and create it if it doesn't exist

Using the scaffold utility to build an app creates the /app/log directory, but if a dev creates the directory structure from scratch using just the repo contents, it's easy to forget /app/log and end up with errors during attempted file writes.
At app startup, check for the existence of the app log directory (as specified in the config) if file logging is enabled and create the directory if it doesn't exist. If attempts to create the directory fail, throw an error and quit startup.

Fixed.


ListView text within <> not displayed

I have defined a gui.ListView and when I append a message with a value in pointy brackets, then the text is not displayed.
The example below gives as output:
Message =xxx=
Message
instead of
`import remi.gui as gui
from remi import start, App
class MyApp(App):
def init(self, *args):
super(MyApp, self).init(*args)
def main(self):

    #tb = gui.TabBox(width='80%')
    list_view = gui.ListView(width='100%', height='100%')
    string = 'xxx'
    message = 'Message ={}='.format(string)
    list_view.append(message)
    message = 'Message <{}>'.format(string)
    list_view.append(message)

    return list_view

if name == "main":
start(MyApp, title="ListView-test", standalone=False)`

Hello Andries,
There is already a fix for this, you have to use the version on master branch. I will make a release on pypi soon. Try the version on master branch and let me know if solves your problem.
I made this update just yesterday.

Hello Davide,
Indeed that solved the problem.
Thanks


Respect optimize parameter in zeros_like

This fix address the issue in #12436 where optimize flag was not respected when shape is fully defined for zeros_like.
This fix fixes #12436.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Can one of the admins verify this patch?

The PR has been rebased to fix the merging conflict and pushed. Please take a look.

Jenkins, test this please

Jenkins, test this please.

Error on MacOS was a lot larger than on other platforms:
[ RUN      ] NNGradTest.MaxPoolGradHelper
tensorflow/cc/gradients/nn_grad_test.cc:39: Failure
Expected: (max_error) < (2e-4), actual: 0.134928 vs 0.0002

Yay! All green.


feat: chapter 11 - creating user interfaces with layouts and views

Chapter 11: Creating User Interfaces with Layouts and Views
This pull request adds a new feature to the application.
Topics Covered

Introducing ConstraintLayout
Introducing the Layout Editor
Using ConstraintLayout
Styles, Themes, and Theme Attributes

How to Test
To test the new feature, you can do the following:

Clone the repository.
Install the dependencies.
Run the application.
The new feature should be available.

Screenshots

Known Issues
There are no known issues with this pull request.
References
Reviewers
Thanks!

Looks good to me.


about the predictor

I'm a beginner . May I ask that  can your code predict the yield_strength of the material?

Hi @yangyizu, yep we can! Please take a look in the data/matbench folder, there is the steels_yield dataset. Is this the dataset you want to predict on?
Or do you have your own data that you want to train / predict with?

In fact, I have been training the  steels_yield.csv dataset you provided, and I am trying to obtain a regression model with the method of deep learning to predict the yield_strength property. However, due to the small amount of data(the data set has just over 300 rows), the effect of my model is poor, and the MSE value is too large. So is there any way to solve my problem or is there any source code I can refer to? Thank you very much indeed！

Could you please provide more details with what you are trying do?
In our source code and in the paper, we show how to train/evaluate CrabNet on the material properties (including steels_yield), so you can refer to the code there for those steps: https://github.com/anthony-wang/CrabNet/blob/4c46703d5f6e2e55e7280a507db117a586ad9af3/matbench_crabnet.py#L127-L134. One of the benefits of CrabNet is that it can also train and do well on small datasets like this one here.
Are you training steels_yield with CrabNet (then you can refer to the above), or with another neural network that you created?

I was given a task assigned by my teacher to predict the yield_strength of metal. The data set my teacher gave me is exactly the same as the data set you provided, maybe because our data sets are all from the same website. But the teacher's request is to only use the steels_yield.csv file for deep learning training, because we only need to predict the yield_strength. I tried a lot of common models, but the results were not good because the amount of data was too small. I will try your CrabNet model later. Thank you so much for your teaching！

Hey, that sounds like a cool exercise!
So if the requirement is to use deep learning then you can either try a very simple MLP (multilayer perceptron, or a series of fully-connected layers) or you can also try using CrabNet.
In fact, we already have a trained CrabNet model using the steels_yield dataset, see: https://zenodo.org/record/4633866.
But if you can use other machine learning model, maybe you can even use a simpler model like linear regression, random forests (like our comparison benchmark in the paper), gradient boosting, etc. These models typically do pretty well when the amount of data is small.
Let me know if you need more help!

thanks a lot！非常感谢！

沒問題 🙂


Implement a user aware service

For paths that may or may not be authenticated, but don't care if you are (say, a homepage but you're logged in or not), UserAwareService might be useful here.

Closed by #144


Dropping Flutter Web sample.

Fixes https://github.com/dart-lang/stagehand/issues/632

PTAL @kwalrath

Removing this broke the Flutter: New Web Project in VS Code, since it was hard-wired to use this template.
I think this was a bit of an edge case (we don't usually reference templates by ID, but here we had to), but I thought it worth mentioning - the IDEs do use stagehand so changes like this could (though usually aren't) breaking.
I'll remove the command from the next release, but I won't bother hot-fixing it unless people start raising issues .
@pq FYI - I'm not sure if it works the same way in IntelliJ - if you're just showing the full list it probably doesn't matter, but just in case - FYI!

Thanks for the info, @DanTup. I didn't know that any IDEs were hardwired to use individual templates! I think/hope IntelliJ just shows the full list.

Yeah, it's not very common for us to have something this specific (for this reason) but it was confusing to have Flutter's web projects appear under Dart: New Project (which uses stagehand) when there's a Flutter: New Project command alongside it (using flutter create) so we added some specific handling (in VS Code, at least).
I actually carefully coded it so if additional templates were added with a flutter-web prefix, it would handle them gracefully - allowing the user to select from a list - I didn't think about it being removed though 🤦♂! Though to be honest, it probably wouldn't have helped - for how we need to set the command up, there's no way we could dynamically hide the command (VS Code limitations) so the best we could've done is shown an appropriate error instead of a weird empty list until it was removed in another release.

Related: #637


[DERCBOT-901] Assembling - Sentence generation feature

@morgandiverrez you can do a review also of this one as it impact some of your developments


Execution can't recover after crash

While running trin execution, it happened that era1 deserialization failed (irrelevant to this issue).
When I tried to resume running it, it would fail very soon afterwards with error:
Error: database error: not found database error block_hash
After looking a bit more into it, I found the problem.
The BlockExecutor::manage_block_hash_serve_window modifies the db directly after every processed block. If the execution crashes (like it happened to me) and we try to resume it, the stored block hashes will not be the correct ones (we will have 256 blocks from the moment of crash, not the saved checkpoint).
Possible solutions:

(preferred) Keep track of block hashes in memory and flush them to this when the rest of state is flushed.
Before execution starts, make sure we have all required block hashes in db (and seed them if that's not the case)

Alternatively, we can just never delete block_number->block_hash from the db. Clearly, not most optimized solution, but definitely the easiest one.
It's only ~64 bytes per block, so it's not the end of the world (total of ~1.2 GB for entire chain at the moment).

https://github.com/ethereum/trin/pull/1451#issuecomment-2351083111
https://github.com/ethereum/trin/pull/1451#issuecomment-2351083985
Additional comments I made on this problem, and why switching to an ACID database solves them

Why can't we use RocksDB? Instead of using rocksdb::DB, we can use rocksdb::TransactionDB or rocksdb::OptimisticTransactionDB.
Difference between transaction and optimistic transaction can be found here: https://github.com/facebook/rocksdb/wiki/Transactions .
I think in our case, we can even use rocksdb::DB::write. Might be the simplest solution.

Erigon has a write up here
https://github.com/erigontech/erigon/wiki/Choice-of-storage-engine
They tried like 5 different database solution then ended up with MDBX.
They say it isn't ACID,

Why can't we use RocksDB? Instead of using rocksdb::DB, we can use rocksdb::TransactionDB or rocksdb::OptimisticTransactionDB. Difference between transaction and optimistic transaction can be found here: https://github.com/facebook/rocksdb/wiki/Transactions .
I think in our case, we can even use rocksdb::DB::write. Might be the simplest solution.

This looks like a good initial start, as it seems to have higher reliability than our current solution, but because various projects have pointed out issues, I am inclined to think it is a bad choice long term.

https://github.com/cberner/redb here is an interesting KV store based in RUST. It looks like nothing can compare with the performance of LMDB or MDBX which is better then LMDB


Replace tests that depend on TaskHandler

Motivation
After #427, we now want to make sure, we can eventually remove now unused code. Some tests depend on TaskHandler, that actually test other functionality. Work that needs to be merged before #443.
Changes

Replicate four test that depended on TaskHandler

Result

Code can be removed, but tests continue to test.

swift-nightly fails because of sendable requirements.


JSON: Intellisense appears and disappears every keystroke

Issue Type: Bug
This bug occurs when in JSON, while typing a key. On every keystroke, the intellisense menu appears and disappears with the suggestion to quote the key. So this feature only works when typing keys that consist of an uneven number of characters, since intellisense disappears at every even character.
I've made a recording:
intellisense in json.zip
VS Code version: Code 1.29.1 (bc24f98b5f70467bc689abf41cc5550ca637088e, 2018-11-15T19:13:36.375Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-8950HK CPU @ 2.90GHz (12 x 2904)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
31.74GB (17.11GB free)

Process Argv

Screen Reader
no

VM
67%

Extensions (19)

Extension
Author (truncated)
Version

overtype
ada
0.2.0

Bookmarks
ale
9.3.0

html-class-suggestions
And
1.0.7

color-info
bie
0.5.0

mustache
daw
1.1.1

vscode-eslint
dba
1.7.0

vscode-babel-coloring
dza
0.0.4

EditorConfig
Edi
0.12.5

file-icons
fil
1.0.16

auto-close-tag
for
0.5.6

duplicate
gee
1.0.1

beautify
Hoo
1.4.7

docthis
joe
0.7.1

vscode-duplicate
mrm
1.2.1

vscode-docker
Pet
0.4.0

vscode-sort-json
ric
1.13.0

code-settings-sync
Sha
3.2.0

rewrap
stk
1.9.1

vscode-svg-previewer
vit
0.2.0

@mjbvz, should general intellisense issues go to you? Feel free to re-assign to me if not!

Yep, it sure does.

That's the expected behaviour of quick suggest: code completions are automatically shown when typing at the end of a word. You cam configure/disable this in the settings.


Building for nRF52810

Hi,
I'm trying to compile for nRF52810 but failing. I cannot find any example so I am trying to bend 52832 example.
I have found this patch from @alvarop : https://github.com/alvarop/mynewt-core/commit/9fe0f2b96322a48824418bc005db42f1c52d0a22#diff-5f33e3c2dbee3aecfa37c7f194a190d72331e43ecddc3f2a6d7fac768408e997
But I am getting hardfault.
This is part of my syscfg.yml:
syscfg.defs:
    BSP_NRF52:
        description: 'Set to indicate that BSP has NRF52'
        value: 1
    ENC_FLASH_DEV:
        description: 'Encrypting flash driver over interal flash for testing'
        value: 0
    RAM_RESIDENT:
        description: 'Compile app to be loaded to RAM'
        value: 0

syscfg.vals:
    # Enable nRF52832 MCU
    MCU_TARGET: nRF52810

    UART_0_PIN_TX: 14
    UART_0_PIN_RX: 15
    UART_0: 1
...

When debugging, I can see that here https://github.com/apache/mynewt-core/blob/master/hw/mcu/nordic/nrf52xxx/src/system_nrf52.c the code for NRF52 is executed instead of NRF52810_XXAA.
How do I undefine NRF52 and define NRF52810_XXAA ?

Yes, thank you, that is an improvement. But I am still getting hardfault. An example bsp would be best.


Datashader Kibana Layer Bug - Resolved In Newer Kibana Version

Creating this issue by request.
in Kibana Version 7.10.* if you setup a datashader layer:
If you type invalid KQL in the search bar (i.e. ‘())’) it crashes the map
when testing this in version 7.13 - 8.0 the error was caught and no longer crashed the map.

leaving this issue as open just so that it is more easily visible

Are there still 7.10 instances being used on related projects?  It looks like this repo's latest branch uses 7.13.2.

Closing this issue as we are now targeting much newer versions.  Can re-open if necessary.


yarn check-types builds a production bundle and creates untracked files

Since we merged https://github.com/ChainSafe/lodestar/pull/6685, running yarn check-types has become really slow and at the same time also creates untracked files (see https://github.com/ChainSafe/lodestar/pull/6685#discussion_r1586631639)
My suggestion is to either revert the PR or come up with a proper solution as of right now this negatively affects developer experience.
It also fails randomly locally, and even in the CI (see https://github.com/ChainSafe/lodestar/pull/6685#discussion_r1586635384)

[x] fix build error
[x] no more untracked files as side effect of running build
[ ] yarn lint should consistently pass, irrespective of running yarn check-types before
[ ] move build out of yarn check-types to speed up time to run yarn check-types by 2x (or more)

Noticed something else, running yarn lint before yarn check-types causes no issues. But if check-types is executed once locally, any lint runs will fail with the following
/home/nico/projects/ethereum/lodestar/packages/light-client/test/unit/webEsmBundle.browser.test.ts
  1:1  error  Unused eslint-disable directive (no problems were reported from '@typescript-eslint/no-unsafe-call')

yarn lint should consistently pass, irrespective of running yarn check-types before

That depends upon what lint is doing. We are doing linting on both src and tests. And when you lint test which are depending upon the build artifacts then you have to build first before linting.

move build out of yarn check-types to speed up time to run yarn check-types by 2x (or more)

That's for one package where needed and not for all packages.

lint test which are depending upon the build artifacts

this is such a bad design, lint should never depend build...and it never has until now.
Also right now it's vice versa, just running yarn lint will pass but if you run yarn check-types once locally, any yarn lint afterwards will fail (just give this a try yourself locally)

So in paractivce it's not slowing down 2x.

I meant by 50% but it's more like 25-30%, this is still bad enough imo

Issue seems to be resolve, having a smooth DX on unstable branch now, thanks @nazarhussain :heart:


CATALOG OF REQUIREMENTS

We have to develop the catalog of requirements

Software Requisites Specification - Autoguider Camera_mz_with_comments.docx

SRS - Autoguider Camera - Final version.docx
This is my proposal for the final version of the catalog of requirements


web error 'Oops...  Something went wrong! '

After I completed the deployment, the agent was ok and kubeshark check was OK, but the web interface was opened with this error

This errors is usually caused from some compatibility check. Are you using the latest version? Are there any ingresses in the cluster?

此错误通常是由某些兼容性检查引起的。您使用的是最新版本吗？集群中是否有入口？

kubeshark version 40.3, Is it related to the kubenets version? My version of kubenets is v1.20.6

此错误通常是由某些兼容性检查引起的。您使用最新版本吗？集群中是否有入口？

kubeshark版本40.3，与kubenets版本有关吗？我的kubenets版本是v1.20.6
cluster no ingress

Can you first upgrade to the latest version? you can use this: sh <(curl -Ls https://kubeshark.co/install)

Same here

Lastly, I assume you have connection to the outside?
Can you dump the logs by running: kubeshark logs and upload here? Alternatively you can do via Slack.

I suggest trying V50.0 (latest)

我建议尝试V50.0（最新）

ok, I'll try


fix(compatibility): Alternative approach to IE compatiblity

(removed polyfill)

:tada: This PR is included in version 2.3.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Loads font-awesome asynchronously

This is an enhancement or feature.
Summary
Loading font-awesome asynchronously allows to display the site faster. This change is advised by google pagespeed insights

To be honest, I was thinking that Michael would take this as "too minor".


Condition of sale

Hitas Pull Request
Description
Adds the "condition of sale" model and endpoints.

Does not yet add the ability to filter apartments based on if they have conditions of sale of not.

Pull request checklist
Check the boxes for each DoD items that has been completed:

Frontend

[ ] Changes have been tested

Backend

[x] Changes have been tested
[x] Automatic tests have been added
[x] OpenAPI definitions have been updated
[x] Database migrations will work in the test environment
[ ] Oracle migration has been updated
[x] initial.json work with current migrations

Test plan

[x] Automatic tests
[ ] Check logic for edge cases

Tickets
This pull request resolves all or part of the following ticket(s): HT-173, HT-175

New condition of sale create endpoint with household instead of additional_ownerships is now available. Tests have also been written with given-when-then to make them easier to read.


Add udp+local dns server

Codecov Report

Merging #1467 (3ea6ac5) into master (fa61371) will decrease coverage by 0.19%.
The diff coverage is 13.63%.

@@            Coverage Diff             @@
##           master    #1467      +/-   ##
==========================================
- Coverage   39.52%   39.33%   -0.20%
==========================================
  Files         598      591       -7
  Lines       35068    34975      -93
==========================================
- Hits        13862    13758     -104
- Misses      19678    19684       +6
- Partials     1528     1533       +5

Impacted Files
Coverage Δ

app/dispatcher/system.go
0.00% <0.00%> (ø)

app/dns/nameserver.go
49.20% <11.11%> (-3.34%)
:arrow_down:

app/dns/nameserver_udp.go
81.59% <100.00%> (ø)

app/proxyman/outbound/errors.generated.go
0.00% <0.00%> (-100.00%)
:arrow_down:

common/task/task.go
88.88% <0.00%> (-7.41%)
:arrow_down:

testing/servers/tcp/tcp.go
81.81% <0.00%> (-5.46%)
:arrow_down:

testing/scenarios/common.go
73.07% <0.00%> (-4.62%)
:arrow_down:

app/proxyman/outbound/handler.go
38.05% <0.00%> (-4.48%)
:arrow_down:

transport/internet/udp/hub.go
73.61% <0.00%> (-4.17%)
:arrow_down:

transport/internet/kcp/dialer.go
80.39% <0.00%> (-3.93%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


(MWPW-130268): add link markup creation.

Link markup creation.
Resolves: MWPW-130268
Before: https://main--milo--adobecom.hlx.page/drafts/a_vkniaziev/document1
After: https://mwpw-130268--milo--adobecom.hlx.page/drafts/a_vkniaziev/document1

@VKniaz : can you please add before and after test urls? Thx

Just noticed in your after test url that the placeholder replacement for a CTA Buy now button is still not properly replaced.

@honstar Document updated, now before on top.

@VKniaz  the CTA- prefix is still there.

@VKniaz the CTA- prefix is still there.

@yesil : this is expected in the BEFORE section from @VKniaz's test document where an old CTA link is still in use. The AFTER section shows the correct Buy now CTA button.

@honstar I was checking it at https://mwpw-130268--milo--adobecom.hlx.page/drafts/ilyas/merch
I updated the CTA text and it looks good.
Thanks @VKniaz

@VKniaz is it too much ask to update the unit tests?
e.g: https://github.com/adobecom/milo/blob/main/test/blocks/merch/mocks/body.html


Appium : How to run the UWP Desktop app in background for couple of seconds and launch the app from the same state?

driver.runAppInBackground(30);
This command doesn't work for UWP.

Pls check this ticket I opened before, it should solve your problem
https://github.com/Microsoft/WinAppDriver/issues/184

Hi @gouthamsam,
Windows Application Driver doesn't currently support that Appium specific command. Can you provide the scenario you are trying to replicate and the spec on what the command does?

I am testing a UWP app, where I want to run the app in background for couple of minutes to check some screen customizations and bring it back to the same state. In general, for iOS and android driver, it supports that command where the app will stay in background for couple of minutes and come back to the previous state of activity.
Scenario:
Button color customization.

Clicking on the button and change it's color to some random color. We defined a timer in the UWP app and after couple of minutes. It will revert back to the same color.
I want to test this scenario, by running the UWP App in background. I tried with the Sendkeys to minimize the App and Restore it after couple of minutes.
getAppiumDriver().getKeyboard().sendKeys(Keys.META,String.valueOf('\u006D'),Keys.META );
sleep(30);
//Restoring the app from background      getAppiumDriver().getKeyboard().sendKeys(Keys.META,Keys.SHIFT,String.valueOf('\u006D'),Keys.META);
The problem I'm facing with sendkeys is somehow the pressed state of the hot keys still remains selected and other tests were getting failed.
For example:
getStaticAnswer() is a textbox field.
for (int i = 0; i < 3; i++) {
getStaticAnswer(i).click();
getStaticAnswer(i).sendKeys(Integer.toString(i));
}
Now instead of "0" it types "(" in the first field and so on.

Thank you

Hi @gouthamsam,
I see. From your description above, you are simulating your test scenario by minimizing all windows using Alt +m keyboard shortcut and restoring them with Alt + Shift + m. Looks like you have most of the code in place with just a missing Shift key release to stop the key from being held down indefinitely.
In the code snippet you sent, the Shift key was sent once and was never released. As a result 0 is typed as ) and so on. Quoting JSON Wire Protocol SendKeys API document:

The modifiers are not released at the end of the call. Rather, the state of the modifier keys is kept between calls, so mouse interactions can be performed while modifier keys are depressed.

Below is a sample snippet code in C# to do what you are attempting to do. Notice the extra Keys.Shift after the m to release the key modifier.
// Press Windows keys followed by 'm' to minimize all windows
session.Keyboard.SendKeys(Keys.Meta + 'm' + Keys.Meta);

// Press Windows keys followed by Shift and 'm' to restore the windows
session.Keyboard.SendKeys(Keys.Meta + Keys.Shift + 'm' + Keys.Shift + Keys.Meta);

I would also consider only minimizing the particular window instead of all windows if you are interested about your particular app. For example:
// Save the current window size
var windowSize = session.Manage().Window.Size;

// Press Windows keys followed by two Arrow Down keys to minimize current active window
session.Keyboard.SendKeys(OpenQA.Selenium.Keys.Meta + OpenQA.Selenium.Keys.ArrowDown + OpenQA.Selenium.Keys.ArrowDown + OpenQA.Selenium.Keys.Meta);

// Wait for couple minutes or do any other useful operation
System.Threading.Thread.Sleep(TimeSpan.FromMinutes(2));

// Restore the minimized window to the saved window size
session.Manage().Window.Size = windowSize;


feat: support retry integration with istio

What type of PR is this?

feat
What this PR does / why we need it (en: English/zh: Chinese):

en:
zh:
Which issue(s) this PR fixes:

https://github.com/kitex-contrib/xds/issues/26

dep: https://github.com/cloudwego/kitex/pull/1363

@ppzqh @PureWhiteWu 再看下～依赖更新好了

/lgtm


Add icon for Intellijidea toolbox

Description
The PR adds icon (was taken from https://opensearch.org/brand.html) which will be shown in Intellijidea toolbox or Intellijidea recent projects like

or

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Nice devs usability enhancement, thanks @snuyanzin !

Codecov Report

Merging #4882 (2978e18) into main (eb33015) will decrease coverage by 0.06%.
The diff coverage is n/a.

:exclamation: Current head 2978e18 differs from pull request most recent head 7ebf034. Consider uploading reports for the commit 7ebf034 to get more accurate results

@@             Coverage Diff              @@
##               main    #4882      +/-   ##
============================================
- Coverage     70.73%   70.67%   -0.07%
+ Complexity    57884    57819      -65
============================================
  Files          4689     4689
  Lines        277296   277296
  Branches      40363    40363
============================================
- Hits         196137   195970     -167
- Misses        64873    64999     +126
- Partials      16286    16327      +41

Impacted Files
Coverage Δ

...ch/transport/ReceiveTimeoutTransportException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

...opensearch/index/reindex/BulkByScrollResponse.java
48.38% <0.00%> (-37.91%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

...java/org/opensearch/threadpool/ThreadPoolInfo.java
56.25% <0.00%> (-37.50%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
56.25% <0.00%> (-35.00%)
:arrow_down:

...ensearch/client/indices/DetailAnalyzeResponse.java
20.54% <0.00%> (-34.25%)
:arrow_down:

...rc/main/java/org/opensearch/ingest/IngestInfo.java
51.72% <0.00%> (-27.59%)
:arrow_down:

...rch/index/reindex/BulkByScrollResponseBuilder.java
57.14% <0.00%> (-23.81%)
:arrow_down:

.../search/aggregations/bucket/range/ParsedRange.java
67.50% <0.00%> (-23.75%)
:arrow_down:

...ndex/query/functionscore/DecayFunctionBuilder.java
33.45% <0.00%> (-22.19%)
:arrow_down:

... and 460 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Thunderbolt hotplug

Hi, I checked your commits. there is a patch for the usb-c, does it work or not?
Thanks

I tested it tonight, the thunderbolt 3 hot swap is working


Set caching globally for all queries?

Is it possible to set the cache for all queries?
Thanks

And the reversed method (.NoCacheable) would be nice as well.
It's cool that spatial datatypes can now be cached :). Unlike EF6

Added it via #https://github.com/VahidN/EFCoreSecondLevelCacheInterceptor/commit/eec9104efbba5aa62f1e228374bf3f90d0607610


[DevTool]Call result() on LROPoller returned by begin_delete()

This is a possible fix for pipeline failure https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1076986&view=results

@scbedd this change looks good to me -- do you want to verify?


fix: adds stream-browserify as stream polyfill for browser

@nchanged - I've made the changes to the stream polyfill (domain is already fixed in #1438 ). I'm currently requiring in stream-browserify but let me know if you want this as inline code instead.
I haven't been able to run tests properly as I've ran into issues running gulp dev on my machine. Essentially it boils down to this: https://stackoverflow.com/questions/53146394/node-app-fails-to-run-on-mojave-referenceerror-internalbinding-is-not-defined
Ultimately, I would need to upgrade gulp to v4 (and get a new vinyl-fs and new natives).
Guessing this would have breaking changes so I've held off on this. But let me know if you want some PR effort on this.

Please can this go into release soon?

@riglar sorry, I though I merged it ;-))


Change string to bool

Change string to bool because the string type is not a primitive type
Summary
Describe your changes here.
Fixes #Issue_Number (if available)

Build failed. Trying again.

Thanks, @jahbenjah, for making the additional change. We'll merge your PR once we've updated our live branch. Your changes should appear live on docs.microsoft.com in the next day or two.


AppRegistry

I need to change the app registry, but when I do so, I get an error. Where else do I have to change it?

@sidneylopes have you tried the solution from the docs https://github.com/troublediehard/magento-react-native/wiki/Rename-Project ?

sorry, I didn't see it. It worked, thanks


[Wallets] Increase init account preverification gas

Problem solved
Short description of the bug fixed or feature added
Changes made

[ ] Public API changes: list the public API changes made if any
[ ] Internal API changes: explain the internal logic changes

How to test

[ ] Automated tests: link to unit test file
[ ] Manual tests: step by step instructions on how to test

/release-pr

/release-pr


Bump build number to trigger an R 3.6 build Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

The cf-scripts R 3.6 migration script got merged. So the bot PRs should appear soon anyway.

Great, I'll hold off then.


 tag is converted into useless output

 is converted to [command]text which isn't any valid element in Asciidoc. This should just be treated like literal and be converted to single backtick.
<command>ls -la</command>

`ls -la`

Inside  this should not be converted at all and just output the text inside the tags.

Needs example. I believe that asciidoctor support this ?!

You can not have text formatting in code blocks. It says it's supported but it doesn't work for most highlighters.
Input:
https://github.com/SUSE/doc-caasp/blob/adoc/xml/admin_software.xml#L126
Output:
https://github.com/SUSE/doc-caasp/blame/adoc/adoc/admin_software.adoc#L87
The double backticks should not be there.

So the only syntax highlighter that actually supports this in the PDF output is pygments but that also strips all the colors from the actual highlights making them less useful.
In my opinion allowing formatting text in the code examples causes all kinds of follow-up problems like which substitutions to enable and unforseen substitutions with raw outputs etc.. I would make this a styleguide item to not use formatting inside code examples so we can use proper highlighting and avoid using different highlighters for the two outputs.


'javascript:' in string bug

Consider this simple app:
var s = 'javascript: is so cool!'
console.log(s)

Causes standard to report:
Error: Use JavaScript Standard Style (https://github.com/feross/standard)
  app.js:1:8: Script URL is a form of eval.

Looks like this came up as an issue on the eslint side too:
https://github.com/eslint/eslint/issues/1448
Suggested fix was:

To close the loop on this, I'd say your best bet is to disable the rule around the area in question via /eslint-disable no-script-url/

@jprichardson Out of curiosity, what were you trying to do where this rule was an issue?

@feross Sure, cleaning up this old project: https://github.com/jprichardson/node-linkscrape Specifically, this file https://github.com/jprichardson/node-linkscrape/blob/master/lib/linkscrape.js
It seems to me that this behavior is pretty odd and unexpected, specifically in the context of Node.js (albeit rare). Would it make sense for standard to disable no-script-url?

Heh its the exact same use case in that eslint issue... Interrogating a string.

Agreed. This is a bit odd.


Fix getitem for metric collection when prefix/postfix is set

What does this PR do?
Fixes #2399

Before submitting

[ ] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?
Make sure you had fun coding 🙃

📚 Documentation preview 📚: https://torchmetrics--2430.org.readthedocs.build/en/2430/

seems like PT 1.13 does not like it :)
FAILED unittests/bases/test_collections.py::test_metric_collection_prefix_postfix_args[None-None] - AttributeError: 'str' object has no attribute 'removeprefix'
FAILED unittests/bases/test_collections.py::test_metric_collection_prefix_postfix_args[prefix_-None] - AttributeError: 'str' object has no attribute 'removeprefix'
FAILED unittests/bases/test_collections.py::test_metric_collection_prefix_postfix_args[None-_postfix] - AttributeError: 'str' object has no attribute 'removeprefix'
FAILED unittests/bases/test_collections.py::test_metric_collection_prefix_postfix_args[prefix_-_postfix] - AttributeError: 'str' object has no attribute 'removeprefix'


Add Phoxy swapnil0709

Description

Phoxy is a website that uses pexels API to bring you the best collection of high-quality images, that are free to use. You can search for your favorite images and also open the full resolution images and save them.

Add GIF/ScreenShot

Create a GIF of your Working Project and Add it for a quick PR review.
https://github.com/swapnil0709/Hacktoberfest-Projects/blob/phoxy-swapnil0709/Phoxy-swapnil0709/ezgif.com-gif-maker.gif

Task Checklist
Put x in between the square brackets without any spaces to tick your Task.

[x] You have created a new branch for contribution.
[x] Added README.md for your project in a specified folder.
[x] Add Demo in GIF or Screenshot form.
[x] You are Registered Officially for HacktoberFest 2021 as Partcipant.
[x] Followed Contribution Guidelines.

Issue
Mention your issue no. #106

Thank you  for contributing @tenet-coding


Consider publishing weekly status update

We had a discussion in ASP.NET leads sync around reporting of the current status of EF Core. This would include things like:

What is currently being worked on
What was checked in since the last status update
What our immediate priorities are
Burn-down for current release

I was thinking about this over the weekend, and realized that this would likely be useful to many, not just internal people. Therefore, this issue is to consider ways of doing this publicly.
/cc @divega

@ajcvickers curious what you have in mind. Twitter? Announcements repo? Both? Something else?

@divega I don't know. I have considered:

Announcements, but they will be mixed in with everything else, and while we are in the ASP.NET org, it still seems a bit weird for EF to announce in ASP.NET Announcements.
A new docs page, that we update with the latest status at the top, with everything below turning into history. Or with some form of archiving.

I think we can Tweet and Facebook it anyway, but I don't think the content should be there.
Do you have any ideas?

A Github issue, locked for comments - makes it easy to follow (adding comments for each update) ??


Character encoding of the HTML - firefox

Running the extension on Firefox will give the following error

The character encoding of the HTML document was not declared. The document will render with garbled text in some browser configurations if the document contains characters from outside the US-ASCII range. The character encoding of the page must be declared in the document or in the transfer protocol.

Probably, Google Chrome is able to set a default encoding while Firefox doesn't do it

#70 Resolves this issue.


Add Witness count in new Explorer

Describe the bug
In Old Explorer there is Total witness count when you go to a Gateway statistics.
In New Explorer when you get to Gateway statistics there is a list of Witnesses without a Number (so needs manually scrolling and counting)  Please Add Witness count in Witness details of Gateway.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://explorer-beta.helium.com/hotspots'
Click on 'Any gateway with Witness'
Go to 'Witness information'
There is no total number of witnesses like in old explorer

Expected behavior
There should be count like in old Explorer
Screenshots
2 1x Old 1x New

Desktop (please complete the following information):
ON all Desktops
Smartphone (please complete the following information):
-On all IOS
Additional context
Add any other context about the problem here.

dupe of #457


Select specific regions for Service Health Alerts

Would it be possible to have this as an option . To specfiy only certain regions to be alerted on for any service health alerts, possibly by using parameters to overvide all regions.

@gerrynicol thank you for submitting this and its a valid ask, I'm adding to the backlog for consideration and will keep you posted.

@gerrynicol I have been having conversations with numerous product teams on this and there are a couple of things to consider:

Whilst in the current configuration for AMBA you enable all regions you only actually are notified on regions where you have resources in the event of a notification
With that in mind it is not recommended to select specific regions as this could cause issues when new regions come onboard and you haven't updated this manually in your configuration.

With the above in mind I don't feel we should provide the ability to select specific regions balancing out the pros and cons.
Hope that helps share my conclusion and thanks for bringing this up, feel free to follow-up if I have missed something.


CTRL+L doesn't redisplay line

Normally, CTRL+L will both clear the screen and re-display the line currently being typed, but this implementation clears the screen and the currently entered line.

Fixed in devel branch.


Error During Import: ValueError: Required parameter name not set

View details in Rollbar: https://rollbar.com/Azavea/RasterFoundry/items/591/
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/utils/exception_reporting.py", line 24, in func_wrapper
    return func(*args, **kwargs)
  File "/opt/raster-foundry/app-tasks/dags/geotiff/import_geotiff_scenes.py", line 55, in import_geotiffs
    created_scenes = [scene.create() for scene in scenes]
  File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/uploads/geotiff/factories.py", line 67, in generate_scenes
    scene.thumbnails = create_thumbnails(local_tif.name, scene.id, self.organizationId)
  File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/uploads/geotiff/create_thumbnails.py", line 127, in create_thumbnails
    s3_bucket = boto3.resource('s3').Bucket(s3_bucket_name)
  File "/usr/local/lib/python2.7/site-packages/boto3/resources/factory.py", line 474, in create_resource
    client=self.meta.client)(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/boto3/resources/base.py", line 119, in __init__
    'Required parameter {0} not set'.format(identifier))
ValueError: Required parameter name not set

Additional logging context:
Apr 27 17:12:55 <IP_ADDRESS> staging/airflow-worker:  2017-04-27 21:12:50,749 - rf.uploads.geotiff.create_footprints - INFO - Beginning process to extract footprint for image:/tmp/tmpB2tsF_
Apr 27 17:12:55 <IP_ADDRESS> staging/airflow-worker:  /usr/local/lib/python2.7/site-packages/rasterio/__init__.py:160: FutureWarning: GDAL-style transforms are deprecated and will not be supported in Rasterio 1.0.
Apr 27 17:12:55 <IP_ADDRESS> staging/airflow-worker:    transform = guard_transform(transform)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:  Traceback (most recent call last):
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/bin/airflow", line 15, in <module>
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      args.func(args)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/airflow/bin/cli.py", line 237, in run
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      pool=args.pool,
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/airflow/utils/db.py", line 53, in wrapper
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      result = func(*args, **kwargs)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/airflow/models.py", line 1245, in run
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      result = task_copy.execute(context=context)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/airflow/operators/python_operator.py", line 66, in execute
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      return_value = self.python_callable(*self.op_args, **self.op_kwargs)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/utils/exception_reporting.py", line 24, in func_wrapper
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      return func(*args, **kwargs)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/opt/raster-foundry/app-tasks/dags/geotiff/import_geotiff_scenes.py", line 55, in import_geotiffs
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      created_scenes = [scene.create() for scene in scenes]
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/uploads/geotiff/factories.py", line 67, in generate_scenes
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      scene.thumbnails = create_thumbnails(local_tif.name, scene.id, self.organizationId)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/rf-0.1.0-py2.7.egg/rf/uploads/geotiff/create_thumbnails.py", line 127, in create_thumbnails
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      s3_bucket = boto3.resource('s3').Bucket(s3_bucket_name)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/boto3/resources/factory.py", line 474, in create_resource
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      client=self.meta.client)(*args, **kwargs)
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:    File "/usr/local/lib/python2.7/site-packages/boto3/resources/base.py", line 119, in __init__
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:      'Required parameter {0} not set'.format(identifier))
Apr 27 17:12:57 <IP_ADDRESS> staging/airflow-worker:  ValueError: Required parameter name not set
Apr 27 17:13:03 <IP_ADDRESS> staging/airflow-worker:  INFO:root:The key airflow/import_geotiff_scenes/import_geotiffs/2017-04-27T21:12:20.350266 now contains 3356 bytes

Closed by https://github.com/azavea/raster-foundry-deployment/pull/48


Create separate binary for APIs

Description

We should create a separate binary (potentially even an own crate) for the API part of Chronicle. This is straightforward now that all our configuration is docker-compose.yml-based.
Motivation

This would make versioning of Chronicle a lot simpler.

We would likely have to maintain 3 separate crates vs the one right now.

The more I think about it the more disadvantages I see with this, which is why I'm closing this for now.


Replace twitter with X

The share-button
https://github.com/opera7133/tella/blob/master/layouts/blog/single.html#L39
still shows the text and the CI of twitter. Color, logo and URL should be changed to to X.

Replaced logo and text in latest commit.

Thanks!


Add initial ballerina service implemetation

Purpose
$Subject

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


[Security] Workflow main.yml is using vulnerable action actions/checkout

The workflow main.yml is referencing action actions/checkout using references v1. However this reference is missing the commit a6747255bd19d7a757dbdda8c654a9f84db19839 which may contain fix to the some vulnerability.
The vulnerability fix that is missing by actions version could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider to update the reference to the action.

41a21089c309b2c96d134ab949716bf1566ff337


Allow aliases for selected values

This can come in handy when there is a complicated select value that doesn't require fragment and is referenced in order_by and/or group_by.
Corresponding Ecto PR: https://github.com/elixir-ecto/ecto/pull/3966

Looks like SQL Server doesn't like aliases in group by/order by =(. I will update the tests later today.


undefined reference to V4l2Decoder::outputPulse

If --enable-X11 --enable-v4l2 --disable-wayland --disable-egl, then V4l2Decoder::outputPulse is declared but undefined.

Hi @uartie ,
The --enable-X11 in the command line need change to --enable-x11. I tried them both, it's no problem on my system ubuntu 16.10. Could you double check it.
thanks

Sorry, I can reproduce it now, it fixed by #802

yes, I meant --enable-x11... sorry for the typo