icetime: No path found

when generating an icetime rpt file like this:
icetime -d hx8k -P tq144 -p syn/myfile.pcf -mtr myfile.rpt myfile.asc

icetime returns an error: No path found!; The myfile.asc from nextpnr exists, and same error occurs regardless if myfile.rpt already exists or not.
// Reading input .pcf file..
// Reading input .asc file..
// Reading 8k chipdb file..
// Creating timing netlist..
// Timing estimate: 0.00 ns (inf MHz)
No path found!

The source code shows that message in double report(std::string n = std::string()), acting as if no rpt file string was specified, but there must have been as a the myfile.rpt is generated despite the errors.
In this a known problem? There does not appear to be an open issue - but the only other google search result for icetime "No path found!" was found on a bounty page here.

Is it possible that this is an empty design? That would be the normal cause of this message, and corresponding "infinite" maximum frequency.
Otherwise, can you post sufficient information to reproduce this message (asc, verilog and pcf).

This all started with me learning FPGA programming and looking at an AD/DA converter, and specifically this project mentioned here as a passthru example to get started. (I have more details for the AD/DA board on my blog).
My nextpnr output is here. There are a lot of zeros, so perhaps indeed it is empty? I renamed everything to myfile to ensure the dashes in the filenames were not getting confused with commandline parameters.
If indeed this is due to an empty asc file - that would be a great feature to add to icetime say that. :)
The specific constraint file they are using is here with top.v, but copied here for this issue:
verilog:
/*
 * Copyright (c) 2018 Joel Holdsworth<EMAIL_ADDRESS> * All rights reserved.
 *
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions are met:
 *
 * 1. Redistributions of source code must retain the above copyright notice,
 *    this list of conditions and the following disclaimer.
 * 2. Redistributions in binary form must reproduce the above copyright
 *    notice, this list of conditions and the following disclaimer in the
 *    documentation and/or other materials provided with the distribution.
 * 3. Neither the name of copyright holder nor the names of its
 *    contributors may be used to endorse or promote products derived from
 *    this software without specific prior written permission.
 *
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
 * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
 * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
 * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
 * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
 * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
 * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
 * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 * POSSIBILITY OF SUCH DAMAGE.
 */

module top(clk_100mhz, adclk, addb, daclk, dadb);
parameter ClkFreq = 50000000; // Hz

input clk_100mhz;
output adclk;
input [7:0] addb;
output daclk;
output [7:0] dadb;

// Clock Generator
wire clk_50mhz;
wire pll_locked;

SB_PLL40_PAD #(
  .FEEDBACK_PATH("SIMPLE"),
  .DELAY_ADJUSTMENT_MODE_FEEDBACK("FIXED"),
  .DELAY_ADJUSTMENT_MODE_RELATIVE("FIXED"),
  .PLLOUT_SELECT("GENCLK"),
  .FDA_FEEDBACK(4'b1111),
  .FDA_RELATIVE(4'b1111),
  .DIVR(4'b0000),
  .DIVF(7'b0000111),
  .DIVQ(3'b100),
  .FILTER_RANGE(3'b101)
) pll (
  .PACKAGEPIN(clk_100mhz),
  .PLLOUTGLOBAL(clk_50mhz),
  .LOCK(pll_locked),
  .BYPASS(1'b0),
  .RESETB(1'b1)
);

wire clk = clk_50mhz;

// Reset Generator
reg [3:0] resetn_gen = 0;
reg reset;

always @(posedge clk) begin
  reset <= !&resetn_gen;
  resetn_gen <= {resetn_gen, pll_locked};
end

endmodule

and pcf:
#
# Copyright (c) 2018 Joel Holdsworth<EMAIL_ADDRESS># All rights reserved.
#
# Redistribution and use in source and binary forms, with or without
# modification, are permitted provided that the following conditions are met:
#
# 1. Redistributions of source code must retain the above copyright notice,
#    this list of conditions and the following disclaimer.
# 2. Redistributions in binary form must reproduce the above copyright
#    notice, this list of conditions and the following disclaimer in the
#    documentation and/or other materials provided with the distribution.
# 3. Neither the name of copyright holder nor the names of its
#    contributors may be used to endorse or promote products derived from
#    this software without specific prior written permission.
 IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE
# LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
# CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
# SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
# INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
# CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
# ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
# POSSIBILITY OF SUCH DAMAGE.
#

set_io clk_100mhz R9

set_io adclk E6
set_io addb[0] D1
set_io addb[1] D2
set_io addb[2] C1
set_io addb[3] C2
set_io addb[4] B1
set_io addb[5] B2
set_io addb[6] A1
set_io addb[7] C5

set_io daclk H4
set_io dadb[0] H3
set_io dadb[1] J1
set_io dadb[2] J2
set_io dadb[3] K1
set_io dadb[4] L1
set_io dadb[5] H5
set_io dadb[6] M1
set_io dadb[7] M2

here's the modified Makefile I created (adapted from an example) to isolate the error:
PROJ = otl-modem
PIN_DEF = syn/otl-modem.pcf
DEVICE = hx8k

ARACHNE = arachne-pnr
ARACHNE_ARGS =
ICEPACK = icepack
ICETIME = icetime
ICEPROG = iceprog

src=

all: $(PROJ).bin

synthesize: otl-modem.bin

%.blif: syn/top.v $(src)
	yosys -p 'synth_ice40 -top top -blif $@' $<
	# yosys -q -p "synth_ice40 -blif otl-modem.blif" syn/top.v $(src)

%.asc: $(PIN_DEF) %.blif
	$(ARACHNE) $(ARACHNE_ARGS) -d $(subst up,,$(subst hx,,$(subst lp,,$(DEVICE)))) -o $@ -p $^

# otl-modem.asc: otl-modem.blif syn/otl-modem.pcf
	# arachne-pnr -d 8k -p syn/otl-modem.pcf $< -o $@
	# echo "---Running icetime!" $<
	# icetime -c 50 -p syn/otl-modem.pcf  -tm -d hx8k $@
	# icetime -d hx8k -c 50 otl-modem.asc
	# echo "---Done with icetime!"

%.rpt: %.asc
	$(ICETIME) -d $(DEVICE) -P tq144 -p syn/otl-modem.pcf -mtr $@ $<

%.bin: %.asc
	$(ICEPACK) $< $@
#otl-modem.bin: otl-modem.asc
#	echo "Running icepak!"
#	icepack $< $@

simulate-ft2232h_interface: ft2232h_interface.vcd
	gtkwave $< >/dev/null 2>/dev/null &

ft2232h_interface-sim: \
	sim/ft2232h_interface.v \
	src/ft2232h_interface.v
	iverilog -o $@ $^

%.vcd: %-sim
	cd $(dir $<); ./$(notdir $<)

clean:
	rm -f $(PROJ).blif $(PROJ).asc $(PROJ).rpt $(PROJ).bin

.SECONDARY:
.PHONY: all prog clean

.PHONY: \
  synthesize \
  simulate-ft2232h_interface \

This is indeed an empty design for the purposes of icetime (all it has is some disconnected input and output ports after dangling cells are swept away).

FYI, the "no path found" message was added to clear up this case in #45.

Thank you for your help. I'm going to leave this issue open, with the suggestion that the No path found error (which sounds like a file system warning) be changed to the next noted in https://github.com/cliffordwolf/icestorm/issues/45:
This design is empty. It contains no paths.
I created https://github.com/cliffordwolf/icestorm/pull/205 for this.

Merged #205.


ignore SSL feature not working

I set the URL and token then the splunk plugin warned me"Could not enumerate saved searches. self signed certificate in certificate chain".
I checked the closed issue and found that you guys updated a feature about SSL serveral days ago,and I unchecked splunk.commands.enableCertificateVerification.
But it seems that nothing happend, it still required me the warning above.

https://github.com/splunk/vscode-extension-splunk/issues/63

Fixed in version 0.3.0


Added new 4 spesific messages

Added new 4 spesific messages for this tools/commands:

mv : You often move files and folders. Are you tired?
rm : Don't delete everything! They can be important :)
zoom : Wow, are you like meetings? Good meetings!
neofetch : Are you like neofetch? I think, you'll like CLIwrap too!

Hi, planning on pulling this after a possible python rewrite. I noted your PR for these tools.


Update README.md

The main reason why I did this is that I wanted to make sure that this app was originally developed by you (Baptiste Gelez). For example, since now the repository was transferred, this sentence sounds as if I (Ryo Nakano) have created this app:

One day I was lost on the Internet, I found this great mockup of a spreadsheet app, and I decided to make it real.

This is not a code update, but is a little big modification, so I would be grateful if you could review this PR when you have a chance.
Changes Summary

Use 4 spaces for bash commands
Grammar fixes & improvements
Use upper case when appropriate (AP Style)
Mention elementary HIG
Remove contact on mastodon
Write not only how to build but also how to install
Use elementary style instruction in 'Build and Installation' section
Fix the example of how we name our namespaces is not existing file
Add link to create a new issue
Update the URL of the mockup

Thank you!


When I click on a editable cell in a row that is already selected, the row is deselected.

Hi, thank you for the help with this project.
Well as the title say, when I have a selected row, and I click on an editable cell of the same row, the row is deselected. This is not a expected behavior, since I click in something which is editable, it's means that it need to be selected.
This example from the project Storybook, have the same unexpected behavior I'm facing.
Thanks you

@jpramos123 that is not default behavior, the behavior you faced is we enable the selectRow. clickToEdit.  So actually, it's expected behavior.


[sram/dv] Sign off V2S

Signed-off-by: Weicai Yang<EMAIL_ADDRESS>

All the SRAM_CTRL related AIs in #11039 are done.
There are 2 other items to update the checklist, which need the coming RFC this Friday for approval. I don't think we need to wait for that before signing off this block.

Didn't we add a few more V2S items? I suggest making a PR updating the checklists first.

yes, I made a RFC #11090 for it. Need to go over it in the TC meeting this Friday and then update the final checklist at #11053
I feel this block may not need to wait for the checklist update, but I'm fine to wait until the checklist is landed.


Microsoft container entry are not the newest release

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [x] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

docker pull mcr.microsoft.com/iotedge/opc-plc

Any log messages given by the failure

 OpcPlc V1.0.0+1 starting up...

Expected/desired behavior

 OpcPlc V1.1.0 starting up...

OS and Version?

Linux 18.04 LTS

Mention any other details that might be useful
The image on mcr.microsoft.com/iotedge/opc-plc should be updated to the newest release

Thanks! We'll be in touch soon.

Hi! I noticed the same issue on my end. Basically i would like to use the --Nodesfile parameter but it is not available on the lattest docker image. Would be Possible to make thsi version public?

@pcbl Until they decide to publish a newer version, you can set up your own container registry in azure, and publish this module there yourself form VSCode or VS.
It don't seem like the are looking at issues atm..

@Swampen , sure thing. Already working on it. It would just be so convenient when it was already there. :-)

PR #23 fixed this

There was an issue in the build scripts which is fixed and latest is now the latest version (1.1.6)


OOM to train translation models with very large multilingual dataset.

❓ Questions and Help
Before asking:

search the issues.
search the docs.

What is your question?
I try to train a multilingual translation model with several languages dataset. But fairseq will load all the data into the memory so the training failed as I have about 1T datasets after preprocess by fairseq. I want to do the language resampling in a dynamic way so I do not prefer to concat all the data together and split it data1,data2,data3 ... (It can also consume a very large amount of time everytime I changed the dataset of one language)
So, is there a way to train a multilingual translation model throught task translation_multi_simple_epoch and load the dataset in a streaming way to reduce the RAM requirements.

What's your environment?

fairseq Version (e.g., 1.0 or main): main
PyTorch Version (e.g., 1.0) 1.10
OS (e.g., Linux): Linux
How you installed fairseq (pip, source):
Build command you used (if compiling from source):
Python version: 3.8
CUDA/cuDNN version:
GPU models and configuration:
Any other relevant information:

Instead of having one folder as a $data_dir, have more more folders like chat1:chat2:chat3:....:chatN
and make sure each chat_i folder has a train.bin,idx.
Then fairseq will load only one folder among them at the same time, in which is called "robin-round" fasion.
The point is using : to seperate folders (containing .bin and .idx)
For validation, only the first training folder's valid,bin,.idx is used. So you have to put valid.bin,idx inside the chat1 (for this example)
chat1,chat2 are just random folder names.
Since you want a multilingual model, you may want every chat folder to have every language data, because fairseq reads folders in the given order.
In this example, it trains with chat1's train.bin then chat2's and no folder shuffle is done by fairseq.

Instead of having one folder as a $data_dir like fairseq-train /data-bin/, have more more folders like fairseq-train chat1:chat2:chat3:....:chatN and make sure each chat_i folder has a train.bin,idx. Then fairseq will load only one folder among them at the same time, in which is called "robin-round" fasion. The point is using : to seperate folders (containing .bin and .idx)
For validation, only the first training folder's valid,bin,.idx is used. So you have to put valid.bin,idx inside the chat1 (for this example)
chat1,chat2 are just random folder names. Since you want a multilingual model, you may want every chat folder to have every language data, because fairseq reads folders in the given order. In this example, it trains with chat1's train.bin then chat2's and no folder shuffle is done by fairseq.

Hi, @gmryu  I have split the dataset into shards, and the size of each shard is 170G. I try to train a model with 2 nodes with 8 A100 and 700GB memory in each node. And it consumes almost 600+ GB of memory to load one shard of data. When loading the next shard, it seems the memory used for the first one is not released and the out-of-memory error occurs as it needs much more than 1.2T memory.  So should I split the dataset to a smaller size?

Are you using like --num-shards --shard-id? They are different from : .
If you use :, previous memory should be released.
Anyway, in this case, 170GB per epoch is way too much. I would say 1GB per epoch is already too much as well.

Are you using like --num-shards --shard-id? They are different from : . If you use :, previous memory should be released.
Anyway, in this case, 170GB per epoch is way too much. I would say 1GB per epoch is already too much as well.

Yes, I use : to link all the data shards together. So, if the dataset is 4TB, then I need to split it into 4k data shards, right?


Pdf

Coverage decreased (-0.1%) to 36.275% when pulling cdf8f6d330b32c407f7eb4a96717e28e2720fa38 on pdf into 1ee2e907d77e0f1b04857f3a8fe506079f559227 on master.


Camembert Vocab Issue

Hi @louismartin,
I have a question about Camembert Model that you provide from there (http://dl.fbaipublicfiles.com/fairseq/models/camembert.v0.tar.gz).
Why in vocab.txt all the tokens have a 999 next to the token? Is vocab.txt the vocab that goes out from sentencepiece training on large corpora?
Thank you

Hi @simonefrancia ,
The 999 is a dummy placeholder to make the sentencepiece vocab work with fairseq vocab format.
Thanks

Thanks, so you don't use temporal vocab created by fairseq proprocessing at LM inference-level ? that is only needed for LM Training ?

Ok I think I understood. The sentence piece vocab is an input of the fairseq preprocess, so you had to convert sentencepiece notation with fairseq notation. Is that right? Thanks

Yes exactly!

Hi @louismartin ,
I've inserted the vocab trained with SentencePiece in fairseq preprocessing, but a strange thing happens.
This is the command:
python3 preprocess.py \
    --only-source \
    --srcdict sentencepiece.bpe.vocab \
    --trainpref train.bpe \
    --validpref valid.bpe \
    --testpref test.bpe \
    --destdir $DATA_DIR \
    --workers 60

The output of this command:
Vocab Dimension: 32000
Dictionary: 32003 types
train.bpe: 7000000 sents, 167070745 tokens, 91.6% replaced by <unk>
Dictionary: 32003 types
Dictionary: 32003 types
valid.bpe: 1500000 sents, 35775360 tokens, 91.6% replaced by <unk>
Dictionary: 32003 types
test.bpe: 1500000 sents, 35748293 tokens, 91.6% replaced by <unk>
| Wrote preprocessed data to data-bin

I've trained SentencePiece Tokenizer on an entire corpus ( sampling a subset of sentences ), then I have converted the SP vocab in the FAIRSEQ vocab format, then splitted the initial entire corpus in train, valid and test.
The strange thing is that for train.bpe, valid.bpe and test.bpe 91.6% of the tokens are replaced by <unk>.
Do you have any suggestion about this?

Possibly the format of the vocab is off somehow. You can try running fairseq-preprocess again without the --srcdict option, that way it will generate a large/fresh dictionary. You can then adapt the sentencepiece vocab to match the format of that dictionary.

Thanks for you reply @myleott .
But if I don't insert sentencepiece vocab inside fairseq preprocessing, fairseq creates its own vocab, and with that creates also preprocessed data, am I wrong? ( Maybe I am missing something).
Thanks

fairseq creates its own vocab and with that creates also preprocessed data

Yes, exactly :) Since you have a 91.6% unknown rate, I suspect your sentencepiece.bpe.vocab is formatted incorrectly somehow. So to figure out the correct format you should let fairseq generate its own vocab (dict.txt). Then compare the generated dict.txt to the sentencepiece.bpe.vocab you created and confirm that the format is correct.

ok I think that the difference is that in sentencepiece.bpe.vocab the separator is \t, and in vocab.txt the separator is  .
I already noticed this difference and I made a script that converts one to the other in pandas:
import pandas as pd

df = pd.read_csv("sentencepiece.bpe.vocab", sep="\t", header=None,  index_col=False,  quotechar=None, quoting=3, encoding="utf-8")
df[1] = 888
print("Vocab Dimension: " + str(df.shape[0]))

df.to_csv("sentencepiece.bpe.fairseq.vocab", sep=" ", header=None,  index=False, encoding="utf-8")

But I see also that fairseq makes a dict like this:
10 100517
1 99999
2 99999
21 68345
28 56706

while sentencepiece is like this:
<unk>	0
▁,	-3.11256
▁.	-3.49813
▁'	-3.68534
▁di	-3.70167
......

So I don't understand what are the numbers that are in the first column of fairseq vocab.

In the fairseq dictionary the first column is the token and the second column is the frequency of the word in the training set, but the actual value doesn't matter, you can just use 12345.
What's interesting is that the fairseq dictionary seems to be based on IDs instead of Pieces. Did you use the scripts/spm_encode.py script? If so, did you use --output_format=piece or --output_format=id?
What does your training file look like? Can you share the first few lines of train.bpe?
$ head train.bpe

OUTPUT of $ head train.bpe:
1 1256 700 2
1 1880 919 2388 5028 2
1 47 28 2071 20 15473 56 22 3459 28<PHONE_NUMBER> 6640 2406 597 1313 3993 31 17291 10 25 1261 3 28 2060 102 1313 6 18320 8637 70 3341 17640 21 2
1 83 31 128 31 2
1 18967 20 953 590 10 3 28 3430 293 11147 10 6611<PHONE_NUMBER> 20 31770 3193 196 28 2
1 1999 8779 52 5670 71 2190 21 5 28 95 4104 10442 85 754 1493<PHONE_NUMBER> 446 28 3242 102 1243 462 41 2
1<PHONE_NUMBER> 20 2979 3113 2
1 16960 6862 6 257 30746 5229 20 257 30746 2
1 83 31 309 2
1 83 31 128 31 28 28 2

My spm_train command is this:
spm_train \
    --input=$model.raw \
    --max_sentence_length=4192 \
    --model_prefix=sentencepiece.bpe \
    --vocab_size=32000 \
    --model_type=bpe \
    --shuffle_input_sentence=true \
    --pad_id=-1 \

And my spm_encode command is this:
spm_encode \
    --model=sentencepiece.bpe.model \
    --extra_options=bos:eos \
    --output_format=id \
    < $model.raw \
    > $model.bpe

Ok I think I understand: my error is that I am using --output_format=id , so I encode my data transforming in codes; but I feed the fair-processing with sentencepiece.bpe vocab ( that has not codes, but it has tokens as first column, and so look-up in this dict failed in 91.6% of the cases ).
So I think that changing with --output_format=piece and add --srcdict sentencepiece.bpe.vocab should fix the problem. Am I wrong?
Thank you very much

Exactly. If you use --output_format=piece when encoding then it should output pieces. Later you can use --srcdict sentencepiece.bpe.vocab and it should be able to look up the tokens and you should have a (much) lower unknown rate.
Please reopen if this is still an issue after doing the above, thanks!


Fixed typographical error, changed archetecture to architecture in README.

@mono, I've corrected a typographical error in the documentation of the mono-upnp project. You should be able to merge this pull request automatically. However, if this was intentional or if you enjoy living in linguistic squalor, please let me know and create an issue on my home repository.

Thanks!


Reimplement Identity to Owner mapping

Currently the whole identity -> owner mapping is incomplete: user can map his identity to the owner just one time on the start screen.
We need to implement the whole management logic of identity -> owner mapping with UI accessible for admin roles (#707).

Decided to have two options of how users can connect themselves to owners:

some attribute in the OAuth provider with the name of an owner, e.g. ODD_OWNER_NAME = <Owner name>.  will be matched with an existing one in the platform.
Start screen window now offers users to associate themselves with some owner in the system. We must rework this so that this screen would create a request that is visible to admins to approve/decline (https://github.com/opendatadiscovery/odd-platform/issues/902).


Change Region

Hi,
Testing the tool for konnect. When used with kongctl connect login, the process works smoothly, but it defaults to US, and I cannot see EU resources. Is there a flag to switch or is there a plan to add it?

The regions are not codified at this time, however, you can modify the base URL with the base-url flag for konnect based commands.
Try using : https://eu.api.konghq.com


Panic on CTRL+C while zeus start is loading

panic: send on closed channel

goroutine 52 [running]:
panic(0x416aae0, 0xc82000ef30)
    /usr/local/Cellar/go/1.6.2/libexec/src/runtime/panic.go:481 +0x3e6
github.com/burke/zeus/go/filemonitor.(*fsEventsMonitor).Add(0xc820010fc0, 0xc82029a150, 0x65, 0x0, 0x0)
    /Users/andrew/code/go/src/github.com/burke/zeus/go/filemonitor/filemonitor_darwin.go:40 +0x5b
github.com/burke/zeus/go/processtree.(*SlaveNode).handleMessages(0xc8200ba380, 0xc8200e0088)
    /Users/andrew/code/go/src/github.com/burke/zeus/go/processtree/slavenode.go:414 +0x251
created by github.com/burke/zeus/go/processtree.(*SlaveNode).SlaveWasInitialized
    /Users/andrew/code/go/src/github.com/burke/zeus/go/processtree/slavenode.go:124 +0x38d
panic: send on closed channel

goroutine 37 [running]:
panic(0x416aae0, 0xc8200e62d0)
    /usr/local/Cellar/go/1.6.2/libexec/src/runtime/panic.go:481 +0x3e6
github.com/burke/zeus/go/filemonitor.(*fsEventsMonitor).Add(0xc820010fc0, 0xc8200fc7e0, 0x6a, 0x0, 0x0)
    /Users/andrew/code/go/src/github.com/burke/zeus/go/filemonitor/filemonitor_darwin.go:40 +0x5b
github.com/burke/zeus/go/processtree.(*SlaveNode).handleMessages(0xc8200ba1c0, 0xc820124018)
    /Users/andrew/code/go/src/github.com/burke/zeus/go/processtree/slavenode.go:414 +0x251
created by github.com/burke/zeus/go/processtree.(*SlaveNode).SlaveWasInitialized
    /Users/andrew/code/go/src/github.com/burke/zeus/go/processtree/slavenode.go:124 +0x38d

I'm not really sure what to make of this @thedrow any chance you could elaborate?

Yes I started Zeus and then typed Ctrl+C in order to stop it.

Mh.. @thedrow is this reproducible for you? I tried on both mac an linux but can't see it happening, what would potentially help is:

what version of zeus are you using
what os are you using and what version (I assume Mac due to directory structure, but not sure which version)
anything else that might help?

Thanks for your help!

I no longer have access to that machine but I did see it happening from time to time.
I was using the latest Zeus version and OS X Sierra.


translate config-rule-deprecated.md #368

What is the purpose of the change
Translate config-rule-deprecated.md #368
Brief changelog
docs/zh-cn/user/demos/config-rule-deprecated.md

Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Test your code locally by running docsite start, and make sure it works as expected.
[x] Make sure no files under build directory is added.
[x] If this contribution is large, please follow the Software Donation Guide.

Pls read the guide for adding new document in the index. A new blog need following steps:
Add new .md file under blog/en-us or blog/zh-cn.
Update site_config/blog.js, add a new entry to the blog in either en-us or zh-cn.
Send the pull request contains the .md and blog.js only.

Pls read the guide for adding new document in the index. A new blog need following steps:
Add new .md file under blog/en-us or blog/zh-cn.
Update site_config/blog.js, add a new entry to the blog in either en-us or zh-cn.
Send the pull request contains the .md and blog.js only.

yes,but this document is a little special. a new entry of this config-rule-deprecated.md should be added to config-rule.md and has been added.And there`s something wrong with config-rule.md.

I see, my fault.


cast election date to right format

the election tables use dates in a few places so i need to do this a bit more thoughfully than just casting everything to date at the end in a polish stage.. https://github.com/labordata/nlrb-data/blob/main/scripts/tally.sql#L103


[nrf fromlist] modules: tfm: add option for selecting crypto modules

These are needed to disable not needed crypto modules in
the trusted-firmware-m module.
Signed-off-by: Håkon Øye Amundsen<EMAIL_ADDRESS>Upstream PR: https://github.com/zephyrproject-rtos/zephyr/pull/35901

Added DNM label as this should be included as part of https://github.com/nrfconnect/sdk-nrf/pull/4156

Force pushed, to bring down the small change that @oyvindronningstad proposed.


tools: New CMake based Wasm build system

Summary
Introduce a new CMake based build system for Wasm. And target the Wasm ABI to wasm32-wasi, it should
be a more commnly used and standard ABI for Wasm.
Please refer to README.md for more detail.
Impact
New build system
Testing
CI and local machine

What do you thing about moving the README.md to nuttx/Documentation as it is done for other component in apps ?

What do you thing about moving the README.md to nuttx/Documentation as it is done for other component in apps ?

I'm fine with it, but I'm not sure should it be placed at nuttx/Documentation since all codes is in apps side, and only part of apps can be compiled to Wasm module.
BTW, I put it here first because apps/Documentation is not exist.

Yes, all documentation is in nuttx repo, also documentation for apps https://github.com/apache/nuttx/tree/master/Documentation/applications which later can be found here https://nuttx.apache.org/docs/latest/applications/index.html

Thanks, I'll move the README to nuttx/Documentation if it's ready.


Fix missing peripheral::itm export, prepare v0.6.7

This fixes the issue where cortex-m 0.5.10 re-exports 0.6's peripheral::itm module; in principle it's possible other crates could have been using this module directly too (it is public) but in practice it doesn't contain anything useful (all the useful things are in the top-level itm module).
 Please double check that you specified the right target!

:warning: Warning :warning:

Pull requests are usually filed against the master branch for this repo, but this one is against v0.6.x. Please double check that you specified the right target!


Create an intro

Add an intro about yourself: who are you, what is your strength, why are you here.

Intro based on my transition from hospitality to IT


Use Valheim UI Assets

Currently the dropdown and "Map" button use the Unity default UI textures, so it looks a little out of place. Need to change it to use Valheim UI sprites instead. This should be done dynamically at runtime, not by packaging Valheim assets into the mod.

Done in 1.0.4


Updated to v10

Hi.
I finished updating to version 10. There was a focus on type correctness. So after the update, many unknown/undefined types must be checked/dealt with.
After these changes + library updates, only one library is left, which throws errors. It is hav-components. I append the error message, so it can be changed too.
I would like to continue with updating, until 12 or 13 as based on https://angular.io/guide/releases
'Angular versions v2 to v11 are no longer under support.'. However, in my opinion, this should be resolved first
ERROR in ./node_modules/hav-components/fesm2015/hav-components.js
Module not found: Error: Can't resolve '@angular/cdk/layout' in 'Z:\TUKE\ING\2ZS\ESS\FORKED\MSP-Symphony\frontend\node_modules\hav-components\fesm2015'
ERROR in ./node_modules/hav-components/fesm2015/hav-components.js
Module not found: Error: Can't resolve '@angular/cdk/overlay' in 'Z:\TUKE\ING\2ZS\ESS\FORKED\MSP-Symphony\frontend\node_modules\hav-components\fesm2015'
ERROR in ./node_modules/hav-components/fesm2015/hav-components.js
Module not found: Error: Can't resolve '@angular/cdk/portal' in 'Z:\TUKE\ING\2ZS\ESS\FORKED\MSP-Symphony\frontend\node_modules\hav-components\fesm2015'

Thank you for your response. Okay, I will fix the indentation -> should I run npm command
"format": "prettier "src/**/*.{js,json,ts,html,css,less}" --write",
? to fix everything? or just files, where I have changes?

I did run prettier formatting with local .prettierrc:
However, changes (specific angular.json and package.json) have changed even more... Do you use something else?
{
"printWidth": 100,
"singleQuote": true,
"useTabs": false,
"tabWidth": 2,
"semi": true,
"bracketSpacing": true,
"endOfLine": "lf",
"overrides": [
{
"files": "*.json",
"options": {
"tabWidth": 4
}
}
]
}

<EMAIL_ADDRESS>helped, however<EMAIL_ADDRESS>had to be installed too.

Hello!
Thank you for this contribution. I will look into this branch and verify with a thorough hands-on test that the UI is intact following the upgrade.
There are some conflicts stemming from my previous feature branch merge into develop. We can expect that yet more conflicts will be introduced owing to my next planned contribution today (#65). I want to propose that we hold off with merging this at least before that is completed, @hav-gerdint .
If it is confirmed that everything looks good I'd like to tend to those conflicts myself.
It'd be great to have this enhancement in the next release! 🤞

@Sonajos Yeah, angular-cdk is used by hav-components, so I would expect it to need to be upgraded. But I think @angular/cdk only needs to be included as a devDependency, no?

@hav-gerdint. I put it in devDependency... However, in my opinion, it would be better, if hav-components has @angular/cdk as its own dependency. This way, projects which do not use it (like this one), will not have to include it too.

Just a heads-up that I'm currently doing a rebase of the branch mentioned above, if you should have it checked out.
@Sonajos @gerdint

I'm sorry, I seem to have botched it. 🤦
I'll have to delete the branch entirely. Hopefully I haven't inconvenienced anyone!

@ann-hav  @hav-gerdint
What do you think? Should I update the project to the next version, Or should this be merged and I will fork a new branch?

@Sonajos
Hi! I think this is promising work.
If you're inclined, please continue your work with this issue on the new branch 54-upgrade-angular-to-a-more-recent-version--sonajos


Release v0.8.30

Thought to release the next version before merging other big changes. Can I have your thoughts? @muga

Thanks. Will be merging today.


Reproduce Deep representation learning of electronic health records to unlock patient stratification at scale

https://www.nature.com/articles/s41746-020-0301-z
I am contacting the authors for the data. Last author left the institute. Ewww.

Impossible because they can't / are not willing to provide the data. Welcome to academia.


Fix/minor color changes

Added some more dark mode colors for tables

Turns out it was a bit more variables that needed changing


Pallindrome Checker issue resolved

Program to check if the string is pallindrome or not is done using python

Hey @LogicDecode I solved the issue #90 kindly check it if possible and merge it label it hacktoberfest2023 and following the guidelines of hacktoberfest2023


dsp: missing Polynomial-Maclaurin.html

Pinging @thielema on this one:
haddock: internal error: dist/doc/html/dsp/../dsp-<IP_ADDRESS>/Polynomial-Maclaurin.html:
openFile: does not exist (No such file or directory)

https://www.stackage.org/package/dsp

On Tue, 6 Jun 2017, Luke Murphy wrote:

Pinging @thielema on this one:
haddock: internal error: dist/doc/html/dsp/../dsp-<IP_ADDRESS>/Polynomial-Maclaurin.html:
openFile: does not exist (No such file or directory)

What does this error mean? I do not have this Haddock problem locally.

I'm suspicious that this might be a false positive. I've seen 3 errors of this type and I also found a bug in the haddock package (https://github.com/haskell/haddock/issues/634).

I have never seen those; Maybe some disagreement in passing destination paths?

Thing is, many, many packages which have made new releases are failing with this error.


Implement all events and arguments to properly track a call as it proceeds through the changes

Add an "up" event for when a conversation is started.
Add a "hangup" event for when a conversation ends.
Add a call id to track a call across stages (similar to Linkedid in Asterisk).

Moved to Reviewboard.


Space missing between "Continue with" and "Google" on Cognito Page

It is unclear if this is controlled by this project or another Amazon Service, but the following should be fixed. There is a space missing between "Continue with" and "Google" on Cognito Login page. See image below (red circle highlighting the issue)

Thanks for reporting @expatiating, it's service side and we'll fix with the next deployment.

👍  Thanks for the quick reply - look forward to it!


Predicted Class Name not plottted in Ch2-what-does-my-neural-network-think.ipynb

Hi,
Thanks for the amazing book. I recently finished the book and started trying out exercises.
In this notebook [1], the predictions like "10% cardigan" are not plotted in the -output.jpg image. What am I missing?
Thank you.
[1] https://github.com/PracticalDL/Practical-Deep-Learning-Book/blob/master/code/chapter-2/2-what-does-my-neural-network-think.ipynb

Hi @CloudaYolla you may consider making a small PR as suggested in https://github.com/PracticalDL/Practical-Deep-Learning-Book/commit/47fd74b58a5b6e909aad5d72d46672748cd5a69a#comments which will add the predictions line.
The missing line somehow got dropped while upgrading with newer PRs.


proposal: spec: require tagged literals for external structs

It would aid compatibility and evolution if a package writer knew that all clients
creating structs with types exported from the package use the form
  pkg.Struct{A: a, B: b}
rather than
  pkg.Struct{a, b}

It may be worth making this a language restriction

At present there is a vet check for this:
b.go:5:9: pkg.Struct composite literal uses unkeyed fields

Is there an advantage to promoting this check to the language level?

There might be scenarios where it's tedious to have to repeat struct field names over and over again; e.g., for small structs that are frequently created and won't change. Example
p := graphic.Point3D{1, 2, 3}

But for cases like these it's always easy (and probably better) to provide any factory function instead.

There is a trick to force users of an exported struct to write field names:
type Struct struct {
	A int
	B string
	_ struct{}
}

And for any usage of Struct{1, "2"} a simple compile error will appear:
./prog.go:4:16: too few values in Struct literal (Playground https://play.golang.org/p/sPgtcxAoOxd)
(source: https://groups.google.com/forum/#!topic/golang-nuts/NSjVW82i0mY)

As this is checked by vet today, it does not seem necessary to add to the language proper.
(Perhaps at some future point we should consider promoting some vet checks into the language proper.)


can i add some example scholarships

like this static data

@devxMani sure go ahead, you may add this feature above the footer

Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr

Note: the level of the issue may change based on the quality of the PR code

Its been more than two days and no new pull request was made so we are closing this issue.


Docs: show CJS workaround for #169

What:

Why:

How:

Checklist:

[ ] Documentation
[ ] Tests
[ ] Ready to be merged

[ ] Added myself to contributors table

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


INDEX

I have the strong suspicion that the index causes some troubles.
It say in in the docs
        index : dict
            indicating the varied Parameters as
            {position in parameter_combinations: name}
            is needed for post processing to create the
            multi index containing the variable Parameters

but the code (experiment_handling.py l103-106 )
        self.kwparameter_combinations = \
            [{index[i]: params[i]
              for i in range(min(len(params), len(index.keys())))}
             for params in parameter_combinations]

assumes that i starts with 0. However, the first varying parameter could be not the first one, right?
Second: As far is I understand correctly, the index is needed for the resaving routing. So I suggest to give the index to the resaving routine as a parameter - instand of an object parameter.

Hmm.. so far, the index dictionary did not cause me any trouble. But that might be because the parameters that I listed there were indeed only those that I actually varied. All others, I specified as keyword parameters with default values in the interface of the run_function of the experiment.
I assume, something like
self.kwparameter_combinations = \
            [{index[i]: params[index[i]]
              for i in range(min(len(params), len(index.keys())))}
             for params in parameter_combinations]

would make it work in more general cases (also supplying non-varied parameter values in the parameter combination tuple)?

(TEST)
in your suggestion i still starts at 0. I'll get back to this later when I have more time

I don't understand the Issue with i starting at 0 as long as parameter combinations only consist of parameters that are actually varied and the index dictionary is of the form
index = {0: 'name of 1st paramter in index', ..., n: 'name of n+1st paramter in index'}

After a second look I changed the expression to
        self.kwparameter_combinations = \
            [{index[i]: params[i]
              for i in index.keys()}
             for params in parameter_combinations]

though. This should also cover the general case of having fixed and varied parameters in the parameter combinations and then picking them out.

Will test this and push to master, if it proves to be stable.

Looks good, thumps up. I thought of something like this as well.


[VS Code] Unable to join the host due to an authentication error. Verify the host is online, then sign out and sign in again and retry.

Error:
Unable to join the host due to an authentication error. Verify the host is online, then sign out and sign in again and retry.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.869

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.28.2

platformName
MacOS

platformVersion
18.0.0

Thank you for reporting this. I am going to resolve this as a duplicate of #433. Can you please run the command "Live Share: Export Logs" and drag-and-drop the zip here, on #433, or send it to<EMAIL_ADDRESS>Did signing out and signing in again resolve this for you?


Exception thrown when reading a XLSX or XLSB workbook with a chart sheet

Thanks for the great library!
When I was trying to create a reader for an Excel file (XLSX) with a chart sheet a KeyNotFoundException was getting thrown. This also seems to be happening for XLSB files as well. In this case I wasn't trying to parse the chart sheet itself, instead I wanted to parse another worksheet that happened to be in the same workbook as the chart sheet.
Here's an example file: ChartSheet.xlsx
I took a shot at fixing the issue in this PR: #150

Thanks for providing the fix.


[mgmt] Fix duplicated query parameters in request

related issue: https://github.com/Azure/azure-sdk-for-python/issues/23561.
fix feature: https://github.com/Azure/autorest.python/blob/autorestv3/ChangeLog.md#2022-03-03---5130
we need:
(1) upgrade autorest.python to 5.13.0+
(2) add reformat-next-link: false in readme.python.md for service which has similar problems
candidate services:
resources, consumption, advisor

related issue: https://github.com/Azure/azure-sdk-for-python/issues/22823, https://github.com/Azure/azure-sdk-for-python/issues/21932, https://github.com/Azure/azure-sdk-for-python/issues/23175

resources: https://github.com/Azure/azure-rest-api-specs/pull/18591


RFC: add sideEffects: false to package.json

Webpack 4 now supports a sideEffects: false value in package.json which indicates to the tree shaking algorithms that there are no side effects from loading modules. This is fairly important to semantic-ui-react because defaultProps are essentially side effects as far as babel is concerned. BUT, if that's the only side effects semantic-ui-react uses, since they're only relevant to the component itself, they should be safe to shake out. Not a great explanation, but the TL:DR; is that we should add sideEffects: false to the package. Agreed? I can make a PR if that's desired.
https://github.com/webpack/webpack/tree/master/examples/side-effects

@djMax Thanks for suggestion, it's one of the most expected things. It will allow to omit babel-plugin-lodash.
However it should be tested before we will add this param to package.json.

I would be happy to help test this whenever available!

@goenning PRs are welcome if you'd like to submit something and help us start the testing process. 🎉

Yes, please!  I've been wanting to add an analytics build step to our PRs as well.  I want to see at least bundle size in something like a markdown file so we can easily see diffs in the PRs.  We could track full library size as well each component.
I did this last year in #1233 and tracked doc site bundle stats as well as library stats in well formatted markdown.  Have a look at those to see what I'm after here.
IIRC the final engineering hurdle here was figuring out how to generate these so that PRs include them.  Looking back, I think I'd just use husky to fire this on pre-push.  This way, PRs are updated only when pushing to the remote.
Thoughts?

Webpack 4 in production mode, without sideEffects

Webpack 4 in production mode, with sideEffects

Tested JSX
import React from 'react'
import { Button, Ref } from 'semantic-ui-react'

const App = () => (
  <Ref innerRef={(n) => console.log(n)}>
    <Button>Example</Button>
  </Ref>
)

Option was added in #2941.


vm/adb: have adb work with android emulator by default

Android emulator generates an invalid android ID for syzkaller. Solved by removing android ID check in adb.go

Android emulator generates an invalid android ID for syzkaller

Could you please share what that ID looks like?

Android emulator generates an invalid android ID for syzkaller

Could you please share what that ID looks like?

sure thing!
I've been working on a different PR of the code which checks for Android Emulator generated IDs as well (in case that's what y'all would prefer)!
typically the format for the ID is: emulator-<port number>

just noticed a couple adjustments I could make to the code to make it more legible and inline with what's there. Going to make some more changes

okay. should be okay now

This PR has 14 commits, when you really need just one :)

This PR has 14 commits, when you really need just one (or two -- one for Contributors, one for code) :)

I think those contributors got added by accident. so definitely only one needed. I will be keeping this mind in the future. was fumbling for a bit there.

Now it's 15 commits.

Now it's 15 commits.

once again, thank you so much for your patience!

CI tests are now failing with Error: Please limit commit description line length to 120 characters.
Please fix that and then I'll be able to merge this PR :)


chore(release): 2.27.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 72cf8494dd2075cfaa5c0d20cdeca4f5ec754901
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: a32d7c4d68ee5e8b68ad91e3c1cc966d9ca441dd
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: e850446a0614f043fa1c29ead897c62ebb7baa3b
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Question

The bots is machine learning ?

No
And ask questions on discord next time.

@Luka967 Ok ,sorry ?

(Close this)

And ask questions on discord next time.
Says in the discord rules to not ask questions.

WaitWhat

https://i.imgur.com/qNyPHqu.png - Where?

Rule #3: "Don't ask around everybody for custom code or roles"

^^

How does mentioning code or roles imply questions

Don't ask around everybody for custom code
ask

Don't ask for custom code or roles

Don't ask any type of question, not even how this or that works

?????

Asking questions about the project is a question. Asking questions about people helping you adding custom code is not condoned, I don't know how much clearer this needs to be.


⚠ Samsung has degraded performance

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

HTTP code: 200
Response time: 2268 ms

Resolved: Samsung performance has improved in aaa6042.


Example "multiwindow" does not close on OSX

It seems that the close button in the corner is not activated when hovering the mouse button over it. I can't press the button but get a color wheel instead.

Do you use osx 10.11?

OSX 10.9.5.

Same on 10.10.2

Closing for https://github.com/tomaka/winit/issues/48


Implement loop widget

This PR adds a new special widget: for.
This widget allows you to iterate over a json list of data, rendering a new widget child for each element.
This could be used like this, for example:
(defwidget workspaces []
  (box
    (for wsp in workspaces
      (button :class {wsp.state}
              :onclick "wmctrl -s ${wsp.id}"
        "[${wsp.name}]"))))

To achieve this, we restructure the WidgetUse struct to be an enum that may be a basic widgetuse
(which is what prior to this PR, all widgetUse instances where), or some special WidgetUse such as Children or Loop
Still todo:

[ ] some testing
[ ] Documentation

Coolest PR

This is awesome!!
I am interested though, why you decided on doing this as a widget? I would have expected this more naturally in s-exptessions, no?

It took me a bit to it figure out, but this works fine for me now:
(defwidget workspaces []
    (box :class "workspaces"
         :orientation "h"
         :space-evenly true
         :halign "center"
        ; (literal :content {wm-tags})
      (box
        (for tag in '${wmstate.workspaces[0].tags}'
          (button
            :class {tag.mine ? "ws-button-mine" :
                    tag.visible ? "ws-button-visible" :
                    tag.busy ? "ws-button-busy" : "ws-button"}
            :onclick "leftwm-command \"SendWorkspaceToTag 0 ${tag.index}\""
            {!tag.mine && !tag.busy && !tag.visible ?  "·" : "${tag.name}"})))))

So what is keeping this in draft?

Read the latest README update. Afaik, eww maintainance is on hold for a bit while Elkowar works on his thesis

Read the latest README update. Afaik, eww maintainance is on hold for a bit while Elkowar works on his thesis

Ohh, I .obviously missed that, sorry.
I guess I can live a while with this current build as this appears to run quite fine.

@druskus20 if you wanna write some docs I'm down to merge this, lol


Allow to configure template of dropdown button in ngx-dropdown-treeview

I want to make cutom template for button, for example, I want to show not only list of selected items separated by commas but blocks with item title and remove icon.

@ximikavt @leovo2708
hi
is any way to implement it?


reduce the size of default handlers

this commit replaces the two default handler (DefaultDefaultHandler and
DefaultUserHardFault) by a single handler named EndlessLoop. This results in
one less symbol being generated.
Also this new handler uses compiler_fence to avoid the "infinite loops w/o
side effects are undef values" bug in LLVM. compiler_fence is guaranteed to
generate no code so this reduces the size of the handler (when compiler in
release).

As far as I could test this new handler doesn't generate abort instructions when
optimized so it seems like a safe replacement. If we are feeling paranoid then
once #95 we could implement EndlessLoop in assembly.
r? @rust-embedded/cortex-m (anyone)

Couple of comments

Does this mean you can no longer tell the difference between a HardFault and any other unhandled exception, without defining your own handlers? Not the biggest deal since most debuggers will catch HardFault before it triggers the exception handler anyway, but worth considering. Worth the small reduction in binary size IMO.

EndlessLoop vs InfiniteLoop / InfiniteLoopDefaultHandler?

Does this mean you can no longer tell the difference between a HardFault and any other unhandled exception, without defining your own handlers?

With or without this change you can put your breakpoint on the HardFault symbol and that will halt your program before it jumps into the user hard fault handler. (So no ExceptionFrame info at that point). Being able to break after that trampoline requires knowing whether the default user hard fault handler has been overridden or not. If it's the default then break on EndlessLoop; if it has been overridden break on UserHardFault.
But in any case once you are in EndlessLoop (or in DefaultUserHardFault) you can't inspect the exception frame using print *ef because ef is missing from the function arguments. That is a disadvantage. To preserve the ef argument we would need two default handler with different signatures.

EndlessLoop vs InfiniteLoop / InfiniteLoopDefaultHandler?

InfiniteLoopDefaultHandler seems too long
I'll try with two handlers and the compiler_fence.

OK, I have changed to two separate default handlers. This still saves 18 bytes (before it was 20 bytes) and one now can inspect ExceptionFrame from the default user hard fault handler -- without this commit that was not possible because DefaultUserHardFault had the wrong signature.
I have named the default handlers DefaultHandler_ and UserHardFault_ so they show when autocompleting for b Default... and b User....

Thanks! I guess the only question is if the extra two bytes is worth the marginally easier debugging hard faults when using the default handlers. On balance I think it's probably worth it. The new names seem good.
0800069a <UserHardFault_>:
 800069a:	e7fe      	b.n	800069a <UserHardFault_>

0800069c <DefaultHandler_>:
 800069c:	e7fe      	b.n	800069c <DefaultHandler_>

(I'm holding off on bors to check you're happy that this is done and ready, feel free to r= me if so).

bors r=adamgreig


fix: image list

Problem:
Missing airgap deployer image in list
Solution:

Add airgap image to list.
Release full image list in asset.

Related Issue:

Do not need to change anything in the original Rename CLI Assets step.
Rename the Release CLI To Repository Asset step. to Release Asset
Append one line to the Release Asset step: hack/mirror/full_image_list.txt.

3. hack/mirror/full_image_list.txt

updated


Update Composer dependencies (2023-01-24-00-12)

Visual regression test passed!
View the visual regression test report


Migrate to Ionic@4

Ionic 4 will be a major change as the framework will be completely rewritten as web components with the new v4 version.
It's currently available for testing, so we must make sure that once v4 is out we are ready to migrate to it.

Hi.
I followed the migration guide to ionic 4 on a virgin generated ngx-rocket and made a PR against it to get an idea of the big picture of what would be involved for this issue.
I tried to use my best judgment whenever it was unclear whether a given change was contrary to your goals for this project, but there were still a few instances that I wasn't sure about (e.g. styling structure).
It's not fully done yet, but now that it loads, I wanted to run it by you first to see if it would be helpful to you for me to take it to completion.  If not, it's no biggie - I wanted to use this as an exercise in the port anyway.
Let me know if you think it's worth the trouble of continuing :)

Lol forgot to paste the link :P
https://github.com/captaincaius/ngxrocket501toionic4beta/pull/1/files

Hi @captaincaius, I'll take a look but you can also try comparing/helping @iocaste on this PR as he also started working on it: https://github.com/ngx-rocket/generator-ngx-rocket/pull/358

Thanks! :) Will do.

PR here: #369

Closing this issue as the PR was merged. Thanks for your work 👍


Performance -  Encoding relationships

When relationships are encoded, it's using the full schema (attributes & relationships) despite the relationship is not included. Only type & id is needed for those cases.
Example:
Endpoint /users/1
UserSchema defines a relationship which loads an ImageSchema with all its properties & relationships but "?included=image" is not even set by default.
This performance issue is visible and severe when using lazy loading ORMs or when having external dependencies like API calls.
Let me know if you need more details. Perhaps I'm not explaining myself correctly.
Affected version: v0.5.13
Regards

@acmenna Since version 0.5.X we've spent a lot of time to optimize the code at this department. I'm 99,9% the problem is already solved in current release.
If you post a small example of how you work with schema I can answer how it works in current release.

an example of that we test such behaviour

if you compare Schema wiki between 0.5 and the current you will see that getRelationships has now additional parameter array $includeList which would contain a list of relationships to be included. With this data you can omit relationships which won't be included anyway.

https://github.com/neomerx/json-api/wiki/Upgrade-Notes

@acmenna I feel I have answered how to work with 'lazy' relationships and I haven't got any feedback from you for a while. If you have any further questions please don't hesitate to ask.

Thanks @neomerx. We'll be upgrading to 0.8.* this week. I will let you know if we find anything.
Thank you


Hiding env variables from script file after build

Versions

nuxt: 2.15.8
node: v14.18.3
dotenv: 10.0.0

Reproduction

Additional Details

I use require('dotenv').config() in nuxt.config.js.
And this
modules: [
    '@nuxtjs/dotenv',
  ],
  dotenv: {
    systemvars: true
  },
 env: {
    baseURL: process.env.BASE_URL,
    baseUrlWS: process.env.BASE_URL_WS,
}

Steps to reproduce

Use env object in nuxt.config.js
Build the project
Open the Chrome DevTools
Open the Network tab
Look through all the .js files and search for "env:"

What is Expected?
Hidden from prying eyes env variables
What is actually happening?
I can get the values of evn variables from the project

Check out https://nuxtjs.org/docs/configuration-glossary/configuration-env#automatic-injection-of-environment-variables for an explanation of why using .env means these values are encoded in your JS. Consider migrating to runtimeConfig, which allows private config to remain on your server. (Of course, if you need these values on client-side, they will be accessible to your end user.)


Log Location

Hi,
I have set Log Location and Env shows correctly but the logs are still placed in default location.
Any idea?
Regards,
Vishal

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

ID: 911b355c-91b0-cfba-6960-f33d9391c0d9
Version Independent ID: e1d4c6d0-f79a-8931-b5fc-279f6185c522
Content: Configure, optimize, and troubleshoot AzCopy with Azure Storage
Content Source: articles/storage/common/storage-use-azcopy-configure.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@vishalaj1 , Could you please post your question in either MSDN storage forums or Stackoverflow since this issue is not related to the documentation.
We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Manual Code Review Csaba

Implementation

[ ] Does this code change do what it is supposed to do?
[ ] Can this solution be simplified?
[ ] Does this change add unwanted compile-time or run-time dependencies?
[ ] Was a framework, API, library, service used that should not be used?
[ ] Was a framework, API, library, service not used that could improve the solution?
[ ] Is the code at the right abstraction level?
[ ] Is the code modular enough?
[ ] Would you have solved the problem in a different way that is substantially better in terms of the code’s maintainability, readability, performance, security?
[ ] Does similar functionality already exist in the codebase? If so, why isn’t this functionality reused?
[ ] Are there any best practices, design patterns or language-specific patterns that could substantially improve this code?
[ ] Does this code follow Object-Oriented Analysis and Design Principles, like the Single Responsibility Principle, Open-Close Principle, Liskov Substitution Principle, Interface Segregation, Dependency Injection?

Dependencies

[ ] If this change requires updates outside of the code, like updating the documentation, configuration, readme files, was this done?
[ ] Might this change have any ramifications for other parts of the system, or backward compatibility?

Logic Errors and Bugs

[ ]  Can you think of any use case in which the code does not behave as intended?
[ ] Can you think of any inputs or external events that could break the code?

Performance

[ ] Do you think this code change will impact system performance in a negative way?
[ ] Do you see any potential to improve the performance of the code?

Usability and Accessibility

[ ] Is the proposed solution well designed from a usability perspective?
[ ] Is the API well documented?
[ ] Is the proposed solution (UI) accessible?
[ ] Is the API/UI intuitive to use?

Readability

[ ] Was the code easy to understand?
[ ] Which parts were confusing to you and why?
[ ] Can the readability of the code be improved by smaller methods?
[ ] Can the readability of the code be improved by different function/method or variable names?
[ ] Is the code located in the right file/folder/package?
[ ] Do you think certain methods should be restructured to have a more intuitive control flow?
[ ] Is the data flow understandable?
[ ] Are there redundant comments?
[ ] Could some comments convey the message better?
[ ] Would more comments make the code more understandable?
[ ] Could some comments be removed by making the code itself more readable?
[ ] Is there any commented out code?

Testing and Testability

[ ] Is the code testable?
[ ] Does it have enough automated tests (unit/integration/system tests)?
[ ] Do the existing tests reasonably cover the code change?
[ ] Are there some test cases, input or edge cases that should be tested in addition?

Experts Opinion

[ ] Do you think a specific expert, like a security expert or a usability expert, should look over the code before it can be committed?
[ ] Will this code change impact different teams? Should they have a say on the change as well?


Unable to create a management group in bicep

Bicep version
Bicep CLI version 0.12.40
Describe the bug
Unable to create single management group
Unable to create parent and child management group
To Reproduce
targetScope = 'tenant'

resource parentManagementGroup 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'parent'
  properties: {
    displayName: 'parent'
  }
}

resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'child'
  properties: {
    displayName: 'child'
    details: {
      parent: {
        id: parentManagementGroup.id
      }
    }
  }
}

CLI command to deploy this code: az deployment tenant create --location westus --template-file mg3.bicep
If I give tenant level am getting error: I guess I dont have access to create tenant level
{"code": "AuthorizationFailed", "message": "The client<EMAIL_ADDRESS>with object id does not have authorization to perform action 'Microsoft.Resources/deployments/validate/action' over scope '/providers/Microsoft.Resources/deployments/mg3' or the scope is invalid. If access was recently granted, please refresh your credentials."}

Code: target scope = Management Group
targetScope = 'managementGroup'

resource parentManagementGroup 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'parent'
  scope: tenant()
  properties: {
    displayName: 'parent'
  }
}

resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'child'
  scope: tenant()
  properties: {
    displayName: 'child'
    details: {
      parent: {
        id: parentManagementGroup.id
      }
    }
  }
}

CLI Command:  az deployment mg create --location westus --management-group-id parent --template-file mg3.bicep
Error BCP038: The property "scope" is not allowed on objects of type "Microsoft.Management/managementGroups". Permissible properties include "dependsOn".
Error BCP119: Unsupported scope for extension resource deployment. Expected a resource reference.
Error BCP038: The property "scope" is not allowed on objects of type "Microsoft.Management/managementGroups". Permissible properties include "dependsOn".
Error BCP119: Unsupported scope for extension resource deployment. Expected a resource reference.

any update on this

Can you double check your version of bicep that the CLI is using with az bicep version? I am not able to repro the second case.

az bicep version - 0.2.212
I am getting an error while deploying these two codes. am i doing in the correct way or not? if this is correct then what about the error.
if the code is wrong, then can you help me with the correct code of parent and child management group.

This is what am creating but I got stuck in the management group itself. Can you guide me on this.
I want to create parent, child management group and need to associate the subscriptions.

I'm 99% sure the issue is because you are using a very old version of bicep with AZ CLI. Can you try running az bicep upgrade?
BTW, is VS code giving you any errors when you are editing these files? If no, then upgrading bicep to latest should fix it.

I am getting this error when I run az bicep upgrade:
Error while attempting to retrieve the latest Bicep version: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /repos/Azure/bicep/releases/latest (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:997)'))).
I am getting two version when I run this command:

I am working on the corporate laptop. when I run the command where bicep am not getting any path to find the file.

I have upgraded the version.
az bicep version
Bicep CLI version 0.12.40 (41892bd0fb)
bicep --version
Bicep CLI version 0.12.40 (41892bd0fb)
Still am facing the issues. am unable to create the management group(parent and child)
Can you correct whether its a right way to create management group.
Code:
targetScope = 'managementGroup'
resource parentManagementGroup 'Microsoft.Management/managementGroups@2020-05-01' = {
name: 'parentmg'
scope: tenant()
properties: {
displayName: 'parent-mg'
}
}
resource childManagementGroup 'Microsoft.Management/managementGroups@2020-05-01' = {
name: 'childmg'
scope: tenant()
properties: {
displayName: 'child-mg'
details: {
parent: {
id: parentManagementGroup.id
}
}
}
}

I am trying to create the structure. but I am unable to

any update on creation of parent and child management group?

need to create child management groups( production, non prod, dev) for parent management group. any updates

main.bicep:
targetScope = 'tenant'
resource parentgroup 'Microsoft.Management/managementGroups@2021-04-01' = {
name: 'main-roots'
}
// module deployed at management group level but in a different management group
module childmodule 'module.bicep' = {
scope: parentgroup
name: 'newmg'
params: {
scopeName: parentgroup.name
}
}
module.bicep:
targetScope = 'managementGroup'
param scopeName string
module mg1 'mg3.bicep' = {
scope: managementGroup(scopeName)
name: 'Sandbox'
}
module mg2 'mg3.bicep' = {
scope: managementGroup(scopeName)
name: 'Services'
dependsOn: [
mg1
]
}
mg3.bicep
targetScope = 'managementGroup'
Error:
{"code": "AuthorizationFailed", "message": "The client xxxx.com with object id 'fd920e35-41d9-4fcc-8546-9793d2369ede' does not have authorization to perform action 'Microsoft.Resources/deployments/validate/action' over scope '/providers/Microsoft.Resources/deployments/mgg1' or the scope is invalid. If access was recently granted, please refresh your credentials."}
help me to create parent and child MG

It looks like the code is correct, but you do not have permissions to deploy to the tenant scope. If you are not able to get those permissions, then you need to deploy at the management group scope. The trick is your deployment command must target a pre-existing management group. So you would need to use this deployment command:
az deployment mg create -f main.bicep -m <yourMGID> -l <yourlocation>

Then this code should work, which will create a management group called main-roots at the tenant scope, then create a child MG called Sandbox which will be a child of main-roots.
targetScope = 'managementGroup'

// will create a new management group at tenant
resource parentgroup 'Microsoft.Management/managementGroups@2021-04-01' = {
  scope: tenant()
  name: 'main-roots'
}

resource childMg 'Microsoft.Management/managementGroups@2021-04-01' = {
  scope: tenant()
  name: 'Sandbox'
  properties: {
    details: {
      parent: {
        id: parentgroup.id
      }
    }
  }
}

If you are continuing to run into issues, it may be more productive to open a support case as they can look closer into the deployment logs to check your permissions, etc.


Copy Dolphin Emulator instructions to the NAND dump guide

Description

i like the idea, though i dont think it should just be a notice, there would be proper steps to do this

we need to stop abusing notices in general


How to move a dashboard from one project to another?

Dear engineers,
thanks for your great job, now i have a problem I want to move a dashboard from one project to another, what should I do?

Hello, davinci's dashboard doesn't seem to support cross-project movement. You can try to modify the database table data to achieve this effect.


More readable fix for issue #7957

Replaces HTML entity-escaped text with Markdown-escaped text where necessary.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

/assign @chenopis

/lgtm
/approve

closes #7957


Remove ACL Migration Code

ACL migration code was added when we moved from RBAC to ACL in lakeFS.
I think it is about time we remove this migration path from future lakeFS versions.

@N-o-Z I think this is done with #8129, right? closing..


auto stringify object to json

is it reasonable to add json to an html data attribute?
and is it reasonable to auto stringify object to json?
e.g.
var src = "
{{foo}}
";
var tpl = Handlebars.compile(src);
var html1 = tpl({foo:123}); // -->
123

var html2 = tpl({});        // -->

var html3 = tpl(null);      // -->

 or nothing or error

Handlebars is agnostic to the output format. You'll want to either format the data prior to sending into the template or use a helper to perform the proper formatting.


aoc-cli 0.5

aoc-cli release v0.5.0 4 days ago and rename the --file command flag by --input-file.
It can be solved by either :

install aoc-cli v0.4 with cargo install aoc-cli --version 0.4
patch src/bin/download.rs with
https://github.com/fspoettel/advent-of-code-rust/blob/a6405f2edb7b626696dc72da7e81176c304536c0/src/bin/download.rs#L66-L72

    cmd_args.append(&mut vec![
-      "--file".into(),
+      "--input-file".into(),
        tmp_file_path.to_string_lossy().to_string(),
        "--day".into(),
        args.day.to_string(),
        "download".into(),
    ]);

Ugh, second update in a row that broke. I implemented both of your suggestions - updated to v0.5.0 and pinned in the readme command. Thanks!


Google

Hi
Can you add Google login and authentication to your great project?

I will try

it's done, you can follow this link  https://github.com/neozhu/CleanArchitectureWithBlazorServer

THX


Handle 400 status correctly

Port of https://github.com/adobe/helix-sidekick-extension/pull/830

:tada: This issue has been resolved in version 2.1.1 :tada:
The release is available on:

GitHub release
v2.1.1

Your semantic-release bot :package::rocket:


[23686] Cache implemented with initial service extension

Ticket

[23686]

Description

TwilioVerifyDemoCache package created in order to share data between Notification extension and Demo app.
Notification Extension created in order to test implementation
Demo app updated in order to support custom Factor model

Screenshot

Testing

[ ] Added unit tests
[x] Ran unit tests successfully
[ ] Added documentation for public APIs and/or Wiki

          1 Warning

:warning:
Big PR

Code coverage
Total coverage: 95.07%

Powered by Slather

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Big PR

Code coverage
Total coverage: 95.07%

Powered by Slather

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Big PR

Code coverage
Total coverage: 95.07%

Powered by Slather

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Big PR

Code coverage
Total coverage: 95.07%

Powered by Slather

  Generated by :no_entry_sign: Danger

          1 Warning

:warning:
Big PR

Code coverage
Total coverage: 94.68%

Powered by Slather

  Generated by :no_entry_sign: Danger


[Archetype builder] Clean up PotentialArchetype a bit.

Clean up the representation of PotentialArchetype in a few small ways:

Eliminate the GenericTypeParamType* at the root, and instead just
store a GenericParamKey. That makes the potential archetypes
independent of a particular set of generic parameters.

Give potential archetypes a link back to their owning
ArchetypeBuilder, so we can get contextual information (etc.) when
needed. We can remove the "builder" arguments as a separate step.

Also, collapse getName()/getDebugName()/getFullName() into
getNestedName() and getDebugName(). Generic parameters don't have
"names" per se, so they should only show up in debug dumps.
In support of the former, clean up some of the diagnostics emitted by
the archetype builder that were using 'Identifier' or 'StringRef'
where they should have been using a 'Type' (i.e., the type behind the
dependent archetype).

@swift-ci please smoke test and merge


Added new decoder to support Plain String Events from Kafka

Added support to read plain String events.
To use this decoder set the property camus.message.decoder.class to
camus.message.decoder.class=com.keedio.camus.etl.kafka.coders.StringMessageDecoder

The events will be stored in HDFS as plain Strings, the timestamp used in the events will be the current timestamp of the system.

Can one of the admins verify this patch?

It looks like @mvalleavila hasn't signed our Contributor License Agreement, yet.
Appreciation of efforts,
clabot

[clabot:check]

@confluentinc It looks like @mvalleavila just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

Camus is also deprecated from CP 3.0.0 on and we won't be making further changes. This looks like a change that might have been better targeted at the upstream repository before it was deprecated, but also looks like there was a mismatch on the target branch anyway.


Compatibility with Flax

Will diffrax.diffeqsolve work inside a Flax linen Module? How would you set up the initialization to use Flax inside of ODETerm instead of Equinox?

Thanks. I'll give it a go.

Hmm, I want to do something like the following:
import diffrax
import jax
import jax.numpy as jnp
from flax import linen as nn

class NeuralODE(nn.Module):
    derivative_net: nn.Module

    @nn.compact
    def __call__(self, coords):
        def f(t, y, args):
            return self.derivative_net(y)

        term = diffrax.ODETerm(f)
        solver = diffrax.Dopri5()
        solution = diffrax.diffeqsolve(term, solver, t0=0, t1=1, dt0=0.1, y0=coords)
        return solution.ys

coords = jnp.ones((1, 4))
model = NeuralODE(derivative_net=nn.Dense(4))
rng = jax.random.PRNGKey(0)
params = jax.jit(model.init)(rng, coords)

Yet, this gives me:
UnexpectedTracerError: Encountered an unexpected tracer. A function transformed by JAX had a side effect, allowing for a reference to an intermediate value with shape (4,) and dtype float32 to escape.
JAX transformations require that functions explicitly return their outputs, and disallow saving intermediate values to global state.
The function being traced when the value leaked was diffeqsolve at ./.venv/lib/python3.10/site-packages/equinox/jit.py:25 traced for xla_call.
------------------------------
The leaked intermediate value was created on line ./.venv/lib/python3.10/site-packages/flax/core/scope.py:767 (param).
------------------------------
When the value was created, the final 5 stack frames (most recent last) excluding JAX-internal frames were:
------------------------------
...
./.venv/lib/python3.10/site-packages/flax/core/scope.py:767 (param)
------------------------------

To catch the leak earlier, try setting the environment variable JAX_CHECK_TRACER_LEAKS or using the `jax.checking_leaks` context manager.Detail: Can't lift sublevels 2 to 1
See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.UnexpectedTracerError

Any ideas @patrick-kidger?

I'm afraid not. This looks like something to do with Flax, which I'm not familiar enough with that I can help debug this.
That said I'm sure it's possible. Both Equinox and Diffrax operate at "the samel level as" normal JAX. (In contrast Flax is a wrapper around JAX.)

Thanks for the help. It’s at moments like these when I wish I was still at Google :)
I wonder if diffrax is calling jit inside diffeqsolve. That would explain the error. Is there a way to disable that?

Otherwise, is there an way to use Equinox and Flax together? Do you have any examples?

Ah, I figured out a (slightly hacky) way to do this:
class NeuralODE(flax.struct.PyTreeNode):
    """A simple neural ODE."""

    encoder: nn.Module
    derivative_net: nn.Module
    decoder: nn.Module

    def init(self, coords):
        coords, encoder_params = self.encoder.init_with_output(rng, coords)
        coords, derivative_net_params = self.derivative_net.init_with_output(rng, coords)
        coords, decoder_params = self.decoder.init_with_output(rng, coords)

        return {
            "encoder": encoder_params,
            "derivative_net": derivative_net_params,
            "decoder": decoder_params
        }

    def apply(self, params, coords):
        coords = self.encoder.apply(params["encoder"], coords)

        def f(t, y, args):
            return self.derivative_net.apply(params["derivative_net"], y)

        term = diffrax.ODETerm(f)
        solver = diffrax.Euler()
        solution = diffrax.diffeqsolve(term, solver, t0=0, t1=1, dt0=0.1, y0=coords)
        coords = solution.ys
        coords = self.decoder.apply(params["decoder"], coords)
        return coords

rng = jax.random.PRNGKey(0)
coords = jnp.ones((1, 4))

model = NeuralODE(
    encoder=nn.Dense(10),
    derivative_net=nn.Dense(10),
    decoder=nn.Dense(4))
params = jax.jit(model.init)(coords)

Then you can simply use this like any other nn.Module:
@jax.jit
def compute_loss(params, coords, true_coords):
    preds = model.apply(params, coords)
    return jnp.abs(preds - coords).sum()

grads = jax.grad(compute_loss)(params, coords, jnp.zeros_like(coords))

Hurrah! I'm glad you figured this out.

@patrick-kidger I'm using haiku and also faced with the leaked tracer issue.
I was almost clueless for about 1 hour, until I find that comparing to my other programs, this "buggy" program uses the haiku model (a simple MLP) only in the ODETerm , thus the haiku model initialization happens inside diffrax frames.
I workarounded by calling the model with fake compatible data once before calling diffrax (thus the model is already initialized when calling diffrax).
I understand that the haiku or flax way of bridging Jax-style pure function with Pytorch-style module has certain degree of "dark" magic inside, but I don't think there are simple ways to trigger frightening leaked tracer exception.
Could you suggest the root cause of this exception? If it's infeasible to prevent such leak, is it possible to provide a more reasonable error or warning at least?

So Haiku (like Flax) was implemented as a wrapper around JAX, and by-and-large isn't compatible with other libraries in the JAX ecosystem. That's really the root cause -- Haiku assumes that it's only being used in particular ways.
My top recommendation is just to use Equinox instead. This provides a PyTorch-style module without the "dark magic".
If you really want to use Haiku, then probably the best thing to do is to pass your MLP through hk.transform before using another library. This should transform the Haiku DSL into "normal JAX".


BlizzPatchNotes rewrite so Pandoc is not required.

Pandoc and pypandoc are used to convert Blizzard patch notes HTML to plaintext.
Pandoc is not a python package and cannot be installed automatically via the downloader cog and pip.
With some code improvements, the cog itself can handle the text formatting so the requirement for pypandoc and Pandoc can be removed.
Also, some patch notes are too dang long. Add a confirmation dialogue when patch notes will span 5+ messages and include a link to the web page.

DONE!


Volume for Quest Users

I use this as a cross-platform video player and it works extremely well; however, the volume difference between PC and Quest is insane (the volume level on Quest is far quieter), is there a way to mitigate or fix this?
For details I use direct .mp4 links from bunny.net CDN

A comment from a quest user was this - the volume slider does not work for them, whether at 0 or 100 the volume is the same

Are you sure the volume slider works at all in your map? 0 volume on the slider is muted, so if it's not changing then something went wrong and crashed the script. For a volume difference in general I'd just say let them adjust the volume if it's too low, or upload the Quest version of the map with the default volume set higher. 100% on the volume slider is 100% volume on the audio sources, so it's not going to get higher than that.

I see, yes I can confirm the slider works correctly for PC but not for the questies. I suppose I can try making the default volume 100%. I will test this more and update you to see how it goes. The only thing I can think of at the present moment is that Udon is acting weird when people on Quest try to interact with the scripts.
For instance in my world I have buttons that fill the InputField automatically with a predefined VRCUrl and then sends the command to the video controller script OnURLInput. This also does not work for Quest but does on PC

Input fields on Quest are weird, I'm not sure if they would cause issues when you're attempting to use them like this. https://vrchat.canny.io/quest-creators/p/864-ui-inputfields-are-not-functional-on-quest Also make sure you don't have any custom scripts that could interfere with the video player.
Use the URLButton example script if you want a clickable video change. https://github.com/MerlinVR/USharpVideo/wiki/examples#url-button

Thank you that's actually really useful, after I change the buttons I'm expecting the world will run more smoothly. I hope that is also the issue with the audio indirectly somehow. If not I'll report back and re-open this thread


Lazy loading not working until scroll

I have 2 pages:

Feed of songs
Song page

In feed, the lazy loading working good, then I go to song page, and the lazy loading not working until I scroll. I tried ionicScrollDellegate.resize() - not helping.
It's happenes only on the device, on pc it's ok.
Video:
https://drive.google.com/file/d/0B_2AM8ZhXhE9Zk5aMlRyalplYUk/view

I fixed it somehow. I just wrote there again (deleted code and wrote again).

Same issue here. MatanYed solutions doesn't help.

For me no image is shown until I touch the screen, I can wait 30 seconds and nothing will show, then as soon as I touch the screen they start loading and showing... Any idea why?


Improve NGC compatibility

On running ngc there are a lot of errors with visibility and methods params.
This pull request change some inputs visibilities and fix the method params to match with the html call.

I will try to solve the conflict by tomorrow evening so we can merge it and create a new release at the end of the week

@cleever have you tried with Ionic?
iOS? thoughts?
I'm just looking for a workaround because I can't get cordova-plugin-crop working on iOS

@st1s for Ionic you should consider using Cropperjs as described in the ionic forum:
https://forum.ionicframework.com/t/image-cropping-plugin/51206/27

@cleever I am using angular2-img-cropper with ionic with no problems.


Error message "couldn't find values to plot" is not helpful

There are some cases in which chart doesn't know what to do with the input. In those cases, it'd be better to explain briefly why the data cannot be plotted.

In the current implementation, the error output is a lot more useful.
$ cat <<EOF | chart
> 1
> 2
> 3
> EOF
Error trying to chart: I don't know how to plot a dataset with this line format
Lines read	3
Line format inferred	f
Lines used	3
Float column count	1
String column count	0
Date/Time column count	0
Chart type	pie
Scale type	linear
Separator	[tab]


Active project?

Hi there !
TUI Calendar is on active development?

I'm afraid we can assume it is not maintained anymore :cry:

its a great project but i don't see any pushes recently and open issues..
sadly another great project is not being maintained


[Improve] Improve streampark-flink-client module base on [3.1 Naming Style]

[Improve] Improve streampark-flink-client module base on [3.1 Naming Style]

What changes were proposed in this pull request
Issue Number: close #3299

Brief change log

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

Dependencies (does it add or upgrade a dependency): (yes / no)

cc @RocMarshal


Merge State list of portals

I've been keeping an unofficial list of open data portals in WA for the last year or so:
https://kerika.com/m/wWFJ
I'd prefer to contribute to this project and drop my own.
Could you add my links to the directory?

which section of nwdata.org should this be under?

Ophelia –
How about just adding links to the Washington section?
Current:
Seattle

data.seattle.govhttps://data.seattle.gov/
boundaries.seattle.iohttp://boundaries.seattle.io/

Proposed:
Seattle

data.seattle.govhttps://data.seattle.gov/
boundaries.seattle.iohttp://boundaries.seattle.io/
OpenBudget.Seattle.govhttps://openbudget.seattle.gov/

Wilford “Will” Saunders | Open Data Guy
Washington State OCIO (WaTechhttps://watech.wa.gov/)
360.890-2580 m        |   360.40[7.8693] desk
<EMAIL_ADDRESS>|  SIP<EMAIL_ADDRESS>| Yammer https://www.yammer.com/opendatainitiative/  |  data.wa.govhttps://data.wa.gov/
From: Ophelia Yin<EMAIL_ADDRESS>Sent: Sunday, April 23, 2017 12:22 PM
To: sethvincent/nwdata.org<EMAIL_ADDRESS>Cc: Saunders, Will (WaTech<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [sethvincent/nwdata.org] Merge State list of portals (#61)
Importance: Low
which section of nwdata.org should this be under?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sethvincent/nwdata.org/issues/61#issuecomment-296482069, or mute the threadhttps://github.com/notifications/unsubscribe-auth/APkhuCos2atPtMsCrG6qGGLqOUM9KLQcks5ry6S8gaJpZM4M10lc.

Hello Will,
the following are already under Seattle section.  I'll put the OpenBudget under Seattle section as requested.

data.seattle.govhttps://data.seattle.gov/
boundaries.seattle.iohttp://boundaries.seattle.io/

are you going to be at meetup April 26?
ophelia

please approve/merge PR#63

this issue is closed via PR#63


$relatedQuery shouldn't mutate loaded relations

I find that $relatedQuery.findById and $relatedQuery.insert will mutate the relations loaded on a model instance.
I find that this isn't particularly useful & tends to create more problems than it avoids.
Consider e.g

Loading a model with a filtered relation. Any insert or find operation on the same relation using $relatedQuery breaks the filter.
A 'shortcut' relation that is defined only for syntactic sugar (e.g often this is the inverse of a more common access pattern - e.g find genre related to this artist, where you usually access genre->artist & don't really want genre  nested under artist when serialized etc.) - querying with $relatedQuery you now have an unwanted piece of data non-obviously nested in your model

My proposal would be to disable this behaviour as I don't think a complete solution exists here.

Why would we disable findById but not for example where('foo', 'bar')?

Is there some mechanism at play that copies back results from $relatedQuery onto the relation on the model instance? If so I would propose to disable that entirely.
But yes it would follow that .where etc. would also not modify relations.

Every time anything is fetched through a $relatedQuery the corresponding relation is assigned the result value. I agree that this should be disabled entirely, but I don't know how many people rely on this feature. It's one of those things that seem really scary to remove, but then again it may be that everyone has just learned to live with that.
I really don't know what to do with this.

It is annoying when you want to serialize some model graph to return in JSON that if you do any related queries on a model beforehand, those will get included in the output all the way down. However, it probably will break existing applications, including mine in a few places. 😅
That said, it's not super hard to fix them with $loadRelated where needed. The current behavior is not surprising, but it is quite tedious to get around if you want to avoid it.

Is there some way to reduce impact of such a change?
i.e leave default behaviour in place, implement strict $relatedQuery strategy enabled similar to the way defaultEagerAlgorithm etc are set...

We could add a static config flag to Model that would be false by default. Then just mention something like this in the changelog If things blow up, set Model.theNewConfig = true 😄. I would really like to remove that stuff from the codebase too though.

I was thinking to do the opposite by leaving it disabled by default for a couple versions with a 'deprecated in future' warning logged when the existing $relatedQuery algo is used.
The message would be something more like Get your code working with Model.theNewConfig  = true soon because pretty soon it will be the default. :)

The only problem with that is that the next version is 1.0 and we'd be stuck with that option for a long time.

Actually we are stuck with the code and the option either way. The only way to get rid of this by 1.0 is to just remove it in 1.0 and force everyone to make the update. Not the most pleasant update.

@koskimas versions before 1.0 are expected to have breaking changes in api so it's possible to tag the next version as 0.10, introduce the change there with a warning and prepare users for changed behavior in 1.0.

@VladShcherbin Yes I'm aware of that, but I'd really like to move to 1.0 as soon as possible.

The problem with @drew-r's suggestion is that everyone who uses $relatedQuery would get their logs filled with deprecation warnings even if they never used the relation property.
Actually the breaking change from just removing the whole behaviour is not that bad. You just need to search for all your $relatedQuery queries and do this:
const children = person.$relatedQuery('children');
person.children = children;

or
const insertedChild = person.$relatedQuery('children').insert(child);
person.children.push(insertedChild);

if you really want the current behaviour. If $relatedQuery is used solely to fetch a relation, it can be replaced with eager or $loadRelated.
I think I'll just remove this feature in the next version without deprecating it first.

What if $relatedQuery(...).insert() assigned the property, but just await-ing to do a SELECT didn't? I think it's unintuitive and very duplicative to have to assign a prop/push an array when writing, but for reads, $relatedQuery is duplicating $loadRelated.
Also, what person.children hasn't been fetched? Aside from being pretty ugly, person.children = [await person.$relatedQuery('children).insert(child)] implies that all of person's associated children are being set here. I understand the effect of the current behavior (create and push an array) is no different, but the code doesn't imply that it's the only associated record in the array.
I think the issue with $relatedQuery is that it's not intuitive for reads, given that eager and $loadRelated exist. However, removing its behavior for writes means an extremely common problem (and knowledge of the relationship type, as checking for existing arrays) has to be spread throughout userland code.

(Edited the comment above with a more complete example.)

@drew-r @jordansexton We have two people on this thread and already two different opinions. I think everyone will have their own about this one.
I think its even more inconsistent if $relatedQuery('foo').insert adds the result, but no other operation does. Now the reasoning is add the values whenever we can. I think the only consistent change is to remove the mutation altogether.
@jordansexton What if we add helpers like this:
const insertedChild = await person.$relatedQuery('children').insert(child)
person.$setRelated('children', insertedChild);

const insertedChildren = await person.$relatedQuery('children').insert([child1, child2, child3])
person.$appendRelated('children', insertedChild);

Those helpers would do the right thing based on the relation type and whether the relations are empty or not.

I agree with you. What about an option (defaulting to falsy) to $relatedQuery that performs the current behavior (and could use $setRelated/$appendRelated under the hood)? This allows 1.0 to break compatibility, but also makes the upgrade quite easy/obvious.

@jordansexton @drew-r What if I add two booleans:
Model.relatedFindQueryMutates = true;
Model.relatedInsertQueryMutates = true;

and leave them to true by default now?

Yeah, no opposition here. While I think an option arg/context prop to locally override when calling $relatedQuery might be useful, it also seems like this global option covers most cases easily.

@jordansexton There is no mechanism for passing options to relatedQuery and I don't want to add a third argument for it. A separate options method would be no less code than using the new setRelated or appendRelated methods.

I figured as much, and that makes sense.


expose signal that killed the process

this may be as simple as introducing a getter for signalCode which is a pass through to process.signalCode

when a spawn fails, it won't be a signal but will be a regular JS exception (e.g. if the command doesn't exist)
when the command does exist but exits for some reason, process.signalCode will tell you why


sendgrid add support for scheduling emails

For sending scheduled emails through sendgrid, two body fields were missing. This PR adds send_at and batch_id to sendgrid adapter provider_options_body_fields

Hi, thanks for the PR. Could you also update the module docs to reflect the added new options? Cheers!

@princemaple hey, updated the documentation. please review again

Thank you


chore: sync workflows

Changes

chore: sync workflows (e9ca4959af659f0b15b43221cbcc8e7ea1c0e532)

Base PullRequest
default branch (https://github.com/technote-space/ga-framework-template/tree/master)
Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

cp -f .github/workflows/ci.yml .github/workflows/.tmp/ > /dev/null 2>&1 || :

bash .github/workflows/.tmp/workflows/ts-repo/copy.sh release

sed -i "s/cron:.\+$/cron: ${MINUTE} ${HOUR} * * ${DAY}/" .github/workflows/update-dependencies.yml

cp -f .github/workflows/.tmp/ci.yml .github/workflows/ci.yml > /dev/null 2>&1 || :

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflows/broken-link-check.yml

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

cp -f .github/workflows/ci.yml .github/workflows/.tmp/ > /dev/null 2>&1 || :

bash .github/workflows/.tmp/workflows/ts-repo/copy.sh release

sed -i "s/cron:.\+$/cron: ${MINUTE} ${HOUR} * * ${DAY}/" .github/workflows/update-dependencies.yml

cp -f .github/workflows/.tmp/ci.yml .github/workflows/ci.yml > /dev/null 2>&1 || :

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflow-settings.json

:octocat: Repo | :memo: Issues | :department_store: Marketplace

Command results

Details:

add path
/home/runner/work/_actions/technote-space/create-pr-action/v2/node_modules/npm-check-updates/bin

rm -rdf .github/workflows/.tmp

mkdir -p .github/workflows/.tmp

git clone --depth=1 https://github.com/technote-space/github-actions-workflows.git .github/workflows/.tmp/workflows
stderr:
Cloning into '.github/workflows/.tmp/workflows'...

cp -f .github/workflows/ci.yml .github/workflows/.tmp/ > /dev/null 2>&1 || :

bash .github/workflows/.tmp/workflows/ts-repo/copy.sh release

sed -i "s/cron:.\+$/cron: ${MINUTE} ${HOUR} * * ${DAY}/" .github/workflows/update-dependencies.yml

cp -f .github/workflows/.tmp/ci.yml .github/workflows/ci.yml > /dev/null 2>&1 || :

rm -rdf .github/workflows/.tmp

Changed files

Changed file:

.github/workflows/toc.yml

:octocat: Repo | :memo: Issues | :department_store: Marketplace