re-running detailed routing

Describe the bug
Once I triggered detailed router to resume routing after stopping at some iteration where there was still routing violations and it didn't completely finish. By resuming I mean a separate call to detailed_routing. I am trying to recreate this scenario but running into issues. I re-ran detailed routing multiple times with 1 iteration and the violations count is diverging. Tried to increase droute_end_iter to 1 but still couldn't get it to work.
set_thread_count 20
read_db ./spm.odb
read_liberty ./sky130_fd_sc_hd__tt_025C_1v80.lib
read_sdc ./spm.sdc
set min_layer met1
set max_layer met5
set i 0

detailed_route\
    -bottom_routing_layer $min_layer\
    -top_routing_layer $max_layer\
    -droute_end_iter 0\
    -or_seed 42\
    -save_guide_updates\
    -verbose 1

while {$i < 10} {
    puts "DEBUG iter $i"
    detailed_route\
        -bottom_routing_layer $min_layer\
        -top_routing_layer $max_layer\
        -no_pin_access\
        -disable_via_gen\
        -droute_end_iter 0\
        -or_seed 42\
        -save_guide_updates\
        -verbose 1
    set i [expr $i + 1]
}

Expected Behavior
Clarity on whether this is supported and how to correctly do it.
Environment
Git commit: 87af90f72f3f9be1fdfa1d886f0dd8d8b8f34694
kernel: Linux 6.11.10-1-liquorix-amd64
os: Ubuntu 22.04.5 LTS (Jammy Jellyfish)
cmake version 3.25.1
-- The CXX compiler identification is GNU 11.4.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- OpenROAD version: v2.0-17585-g87af90f72
-- System name: Linux
-- Compiler: GNU 11.4.0
-- Build type: RELEASE
-- Install prefix: /usr/local
-- C++ Standard: 17
-- C++ Standard Required: ON
-- C++ Extensions: OFF
-- The C compiler identification is GNU 11.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Performing Test C_COMPILER_SUPPORTS__-Wall
-- Performing Test C_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive - Success
-- Performing Test C_COMPILER_SUPPORTS__-x
-- Performing Test C_COMPILER_SUPPORTS__-x - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-x
-- Performing Test CXX_COMPILER_SUPPORTS__-x - Failed
-- Performing Test C_COMPILER_SUPPORTS__c++
-- Performing Test C_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__c++
-- Performing Test CXX_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17 - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17 - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- TCL readline library: /usr/lib/x86_64-linux-gnu/libtclreadline.so
-- TCL readline header: /usr/include/x86_64-linux-gnu
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.0.2", minimum required is "4.0")
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- boost: 1.81.0
-- Found GTest: /home/karim/.local/lib/cmake/GTest/GTestConfig.cmake (found version "1.13.0")
-- GTest: 1.13.0
-- Found Python3: /usr/include/python3.10 (found version "3.10.12") found components: Development Development.Module Development.Embed
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.11")
-- Found Threads: TRUE
-- spdlog: 1.8.1
-- Found BISON: /usr/bin/bison (found version "3.8.2")
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- STA version: 2.6.0
-- STA git sha: aa598a2f14c5c142e90391a69988523505e7db3d
-- System name: Linux
-- Compiler: GNU 11.4.0
-- Build type: RELEASE
-- Build CXX_FLAGS: -O3 -DNDEBUG
-- Install prefix: /usr/local
-- Found FLEX: /usr/bin/flex (found version "2.6.4")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- TCL readline library: /usr/lib/x86_64-linux-gnu/libtclreadline.so
-- TCL readline header: /usr/include/x86_64-linux-gnu/tclreadline.h
-- CUDD library: /usr/local/lib/libcudd.a
-- CUDD header: /usr/local/include/cudd.h
-- SSTA: 0
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.0.2", minimum required is "3.0")
-- STA executable: /home/karim/work/OpenROAD/src/sta/app/sta
-- Found re2: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/re2/re2Config.cmake (found version "9.0.0")
-- Found Clp: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/Clp/ClpConfig.cmake (found version "1.17.7")
-- Found Cbc: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/Cbc/CbcConfig.cmake (found version "2.10.7")
-- Found SCIP: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/scip/scip-config.cmake (found version "8.0.1")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- Found OR-Tools: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/ortools (version: 9.5.2237)
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- Found OpenMP_C: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- GUI is not enabled
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization
-- Could NOT find VTune (missing: VTune_LIBRARIES VTune_INCLUDE_DIRS)
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found suitable version "1.81.0", minimum required is "1.78")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization system thread
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- Found Eigen3: /usr/share/eigen3/cmake/Eigen3Config.cmake (found version "3.4.0")
-- TCL readline enabled
-- Tcl Extended disabled
-- Python3 enabled
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/tmp.3GXOpVUvbb

To Reproduce
test-case.tar.gz
Extract, cd into extracted directory and run openroad -exit run.tcl
Relevant log output
No response
Screenshots
No response
Additional Context
No response

Unfortunately, this functionality is not currently supported in DRT. In fact, it should have displayed a warning and avoided performing any routing during the reruns (I am working on a PR to address this).
Currently, DRT supports incremental detailed routing, but not in the way you described. Incremental detailed routing focuses on routing only the unrouted nets, while rerouting nets with violations is not yet implemented.
If your goal is to capture snapshots of the design after each iteration, you can achieve this by adding the following line before calling the detailed_route command:
set_debug_level DRT snapshot 1

Additionally, include the following option in the detailed_route command:
-drc_report_iter_step 1

This approach will generate a database file and a DRC report for each iteration, allowing you to track the design's progress.


code outline in explorer view

How do I make the outline view to show in the explorer panel. Right now it shows as a separate panel.
I want to show it in the explorer panel just like the "open editors"
Thanks
Anand

https://github.com/patrys/vscode-code-outline/issues/120#issuecomment-387975553


Show connector and task status in a colored way

RUNNING should be show green.
FAILED should be shown red.
It seems the table library stumples upon ANSI color code though.

Implemented for RUNNING and FAILED; should show PAUSED in yellow once implemented.

Hey @karim-elngr, perhaps something for you to pick up now that pause/resume has been implemented?

Done.


Translated text to french

for files disclosure.php and docs.php.

Oh no - you didn't GPG sign your commits :(


Embark demo output seems to be out of order

Overview
TL;DR
It seems that the output generated by embark demo (probably also embark new) arrives out of order in stdout. Here's what it looks like when running it:
❯ embark demo
Initializing Embark template...
Installing packages...
-------------------
Next steps:
-> cd embark_demo
-> embark run
For more info go to http://embark.status.im
Init complete

App ready at embark_demo

While this looks fine at a first glance, I believe Init complete should be shown before Next steps.
I assume this is due to the commands asynchronous nature.
It'd be nice if we first create and install and then output the next steps.

Fixed by #928.

Ah! I totally missed that. Thanks @michaelsbradleyjr !
Closing this one. :)


Add UI interaction for music album

Add UI interaction for music album

can add music album
can choose genre and source
can enter date

I'm reviewing this PR


How to get context?

plz help me to understand how to select the next word to be predicted and context for it.
thank you

This program is based on the encoder-decoder model. I am not sure exactly if you want to know the whole thing or what I am exactly doing in the decoder part. The encoder encodes the input sentences - usually uses the context of the previous words to encode the current word. A bi-directional encoder encodes by taking context from both sides. That is, once it encodes a word by taking the context of past encoded words, and once it encodes a word by taking the context of future encoded words. Then it combines the two versions of encoding into one by some fashion (addition, average or something). As a result, for each input word (or word-vector) you will end up with its encoded version.
The prediction happens in the decoder part. The predicted word in case of summarization should depend on the input (in this case the encoded version of the input) which is being summarized, and the context of the previous predicted words. When predicting the first word there is no previous predicted word. But we usually a word vector which may be symbolized  (representing start of sentence) as the first word to provide context for the first prediction. This is done to use to same formula for predicting each of the words. In addition to the previous context provided by the previous words, we also take the context from the encoded input (that which is to be summarized). Attention mechanism is used to select a relevant portion of encoded input to serve as a context. Both the context from previous words, and the encoded input are combined and used as a hidden state for the decoder RNN to finally predict the next word.
The dataset I used comes with example inputs and their corresponding summaries (outputs that we want the algorithm to predict given the inputs). The summaries from the dataset can be considered as target outputs...and what the model actually produces are the predictions. The model's prediction is compared with the target output, and then some learning algorithm is used to make the machine predict summarizations similar to the target outputs.

Thank you for taking time to answer.
Also, does using encoder decoder with attention eliminate the need  to check grammar..... for eg john died vs john died.... will this model be able to distinguish between dies and died?
what happens when we have to get output like : "the book is great" how to identify the and is?

john dies vs john died vs john dead... will this model be able to distinguish between dies and died?
Yes, the model can distinguish (superficially, at least) 'died' and 'dead, because normally 'died' and 'dead' will be represented by different word vectors when they are fed to the model. But, that doesn't necessarily mean it will understand their grammatical meaning and significance.

What the model learns to predict depends on the dataset. For example, the model may learn from the data that it is not very likely that 'dead' will follow after a noun....and 'dead' is more likely to follow after 'is' or something like that. The model may learn that if the previous words in the sentence were in past tense, then from the context of the past words 'john dead' is more likely that 'john dies'. If the input mention 'john dies' then the model may learn that the output is more likely to be 'john dies'...and so on.
Similarly, the model should learn that 'the' often starts at the beginning at the sentence or before a noun like word, and almost never after a noun, where as 'is' often start after a noun. So the model will give higher probability to 'is' than 'the' given that the previous words are "the book".
The model basically start to learn the patterns of our usage of words from the huge chunk of data it is fed. Thus, if the data has poor quality grammar, the model output will probably also be of poor quality grammar.
Other machine learning models are mostly similar in this respect.
Does it totally eliminate the need to check grammar? Well, if you do check the grammar, or write some program at the end to enhance the grammar of the output you may get better result. Abstractive summarization is a work in progress. It's not yet perfect. And in practice, the grammar of the output may not be perfect either even if trained from high quality dataset. I am not sure about the exact quality of the output of the current state of the art abstractive summarization programs. The target is to eliminate the need to write specific grammatical rules and make the model learn for itself.

thanks a lot for the detailed explanation. Please bear with me for  2 more questions.
Have you implemented noun identification? Is it possible for me to use any other big dataset(say news article) and say train it little more time using the same code and expect similar or little more better accuracy in prediction(if its a big dataset).
thank you

I haven't implemented noun identification. But I have used NLTK library for POS (part-of-speech) tagging i.e tagging words as NN (singular noun), VB (verb base form) and so on (http://www.ling.upenn.edu/courses/Fall_2003/ling001/penn_treebank_pos.html).
You may find some open source code for POS tagging somewhere.

Is it possible for me to use any other big dataset(say news article) and say train it little more time using the same code and expect similar or slightly higher accuracy in prediction(if its a big dataset).

It's possible. However, I will recommend using mini-batching which I didn't use in this code.
I used mini batching in my later codes (like: https://github.com/JRC1995/Machine-Translation-Transformers) but those are mostly different models.
You may try using this (https://github.com/JRC1995/Machine-Translation-Transformers) code for summarization too. It is also essentially an encoder-decoder model but without RNNs and sequential processing.
Also to objectively compare the quality of an output, evaluation metrics like ROGUE\BLEU\METEOR etc. can be used.
Also note, I haven't used the validation and test set. Validation accuracy and the evaluation of predictions of validation input is important to understand where to stop the training or which state of the model to use for prediction. And, test accuracy is usually the best reflection of the model's performance.

I will try with mini-batching. Thanks a lot for your help


Edited help command and help parser to reflect undo command

Context
Edited Help Command and Help parser to work for undo command
Edited Exit Command Message_Status.
-> syntax error found in the previous work.
Links

Looks good, can we merge this already?


Accessibility pane is empty

Repro step:

Visit bing.com in Edge browser.
Click Accessibility pane

Expected: Accessiblity Tree, ARIA Attributes, Computed Properties to be displayed as in Edge DevTool.
Actual: Accessibility pane is empty

This no longer repros.  Closing issue.


Add compound phrases (fuckbook)

The title

...was quite long 😉
Sure, that’s possible. But it’s not a regex. Would you be interested in PRing?
The right places are profanities and cuss.

No problem!
Yup, it won’t be easy to add total support for compounds, but I also think it would be very viable. think rapist, which is profane, but not in therapist.
To keep false-positives low, I’d prefer just listing words in the aforementioned libs

I added it to profanities; will add to cuss in a second. If you’d like to add more, please PR there!


The menu icon on the right is always visible

The menu icon on the right is always visible, even on big screen (>= 1280 pixel), see image below:

Solved by PR #31


Duplicate Tags when multiple @Api have same value.

I have an Abstract controller for all my controllers that handle users.  I have an @Api declaration on the abstract class so that all subclasses will be grouped under the same tag.  It looks like the swagger doc has multiple tags with the sane value of users though, so i am failing validation on swagger-ui
"tags": [
{
"name": "questions"
},
{
"name": "modifiers"
},
{
"name": "users"
},
{
"name": "users"
},
{
"name": "items"
},
{
"name": "recommendations"
},
{
"name": "terms"
},
{
"name": "caches"
},
{
"name": "users"
},
{
"name": "applicationinformation"
},
{
"name": "users"
}
],

Thanks for reporting!

I still have the same issue when upgrading to 2.0.3.  Is this part of 2.0.3 or do I need to wait for another verison?

2.0.4


Sub generators

I couldn't find this in the documentation, but are there sub-generators for creating various app components like directives services or components?

Work in progress

Wip Angular1

noted


added support for releasing specific version

now it is possible to grunt release:1.2.3 in case you already know the version number that you would like to publish. unit test infrastructure needed to be enhanced in order to test this properly.

Wow. Such good work. Thanks so much. :)

Thanks, I'm glad you like it :)
When do you think this can be merged?

@drublic any news about this?

Sorry for the delay. This is now part of master and will be released with the new version.
Thanks again for your work.


Build fix for alloca on *BSD

Build fix for alloca on *BSD.
For alloca() the alloca.h header should be used for ancient glibc, modern versions of glibc include alloca.h from stdlib.h anyway, and Solaris. malloc.h is not portable and is more of a Linux-ism and has been long deprecated.

Seems okay to me but alloca headers are infamously inconsistent... will wait for @icculus to +1 just in case.

No problem. Thanks.

It looks okay, if it broke somewhere else we'll keep adjusting until everything is happy.


refactor pipe params

Pipe arguments are provided as parameters since April 2016. It is also possible to pass objects - so there is no need to parse a string.
For backward compatibility we keep the functionality.

Coverage remained the same at 100.0% when pulling 4016b83519cdc58a2cae9e9936aa8a8bd15fc966 on fix-pipeArguments into 4e225be712989e8d19bed3aaee6b69e1cea9c290 on master.


Tests coverage reports are broken on CI

Looking at the generated cobertura.xml files they are empty:
<?xml version="1.0" encoding="utf-8" standalone="yes"?>
<coverage line-rate="0" branch-rate="0" complexity="0" version="1.9" timestamp="0">
  <packages />
</coverage>

Digging through older runs on CI, this is the first build where it broke. And the diff from the previous good run was 02b5a479...c5f1b953 which includes code-coverage nuget update - PR #2506 .
Once this is fixed we should also add a check in the yml to complain if the reports are empty.
cc @eerhardt @joperezr

@fhnaseer any thoughts? Suspecting that the upgrade to dotnet-coverage v17.10.3 caused it.

Please run dotnet-coverage with log parameters --log-level Verbose --log-file log.txt and share these logs?

I tried to run this locally with:
./dotnet.sh dotnet-coverage collect --log-level Verbose --log-file log.txt --output local.cobertura.xml "dotnet test $PWD/tests/Aspire.MySqlConnector.Tests/Aspire.MySqlConnector.Tests.csproj -bl "
Test run for /Users/ankj/dev/a2/artifacts/bin/Aspire.MySqlConnector.Tests/Debug/net8.0/Aspire.MySqlConnector.Tests.dll (.NETCoreApp,Version=v8.0)
Microsoft (R) Test Execution Command Line Tool Version 17.9.0 (arm64)
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[xUnit.net 00:00:00.40]     Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource [SKIP]
  Skipped Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource [1 ms]
[xUnit.net 00:00:00.50]     Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource_Keyed [SKIP]
  Skipped Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource_Keyed [1 ms]
Results File: /Users/ankj/dev/a2/tests/Aspire.MySqlConnector.Tests/TestResults/_infinity_2024-03-12_15_28_38.trx
Html test results file : /Users/ankj/dev/a2/tests/Aspire.MySqlConnector.Tests/TestResults/TestResult__infinity_20240312_152838.html

Passed!  - Failed:     0, Passed:    34, Skipped:     2, Total:    36, Duration: 152 ms - Aspire.MySqlConnector.Tests.dll (net8.0)

Workload updates are available. Run `dotnet workload list` for more information.
No code coverage data available. Profiler was not initialized.
Code coverage results: local.cobertura.xml.

log.txt
$ cat local.cobertura.xml
PCH%

I was however able to run it with:
dotnet test --collect "Code Coverage;Format=cobertura" -p:VSTestResultsDirectory=$PWD/artifacts -bl $PWD/tests/Aspire.MySqlConnector.Tests/Aspire.MySqlConnector.Tests.csproj
Test run for /Users/ankj/dev/a2/artifacts/bin/Aspire.MySqlConnector.Tests/Debug/net8.0/Aspire.MySqlConnector.Tests.dll (.NETCoreApp,Version=v8.0)
Microsoft (R) Test Execution Command Line Tool Version 17.9.0 (arm64)
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[xUnit.net 00:00:00.44]     Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource [SKIP]
  Skipped Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource [1 ms]
[xUnit.net 00:00:00.54]     Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource_Keyed [SKIP]
  Skipped Aspire.MySqlConnector.Tests.ConformanceTests.TracingEnablesTheRightActivitySource_Keyed [1 ms]
Results File: /Users/ankj/dev/a2/artifacts/_infinity_2024-03-12_15_32_13.trx
Html test results file : /Users/ankj/dev/a2/artifacts/TestResult__infinity_20240312_153213.html

Passed!  - Failed:     0, Passed:    34, Skipped:     2, Total:    36, Duration: 172 ms - Aspire.MySqlConnector.Tests.dll (net8.0)

Workload updates are available. Run `dotnet workload list` for more information.

Attachments:
  /Users/ankj/dev/a2/artifacts/05e5c536-d9eb-42c8-baef-2bf291ec580e/ankj_infinity_2024-03-12.15_32_10.cobertura.xml

The generated xml is ~6M. I couldn't find a way to specify the filename for the xml with this method though.

FWIW, by reverting to 17.9.3 I got the code coverage work again --> https://dev.azure.com/dnceng-public/public/_build/results?buildId=600868&view=codecoverage-tab
https://github.com/dotnet/aspire/pull/2825


chore(download): remove body download range recovery

Brief description
The body downloader has range recovery logic in case body stage logic is changed and it may be able to re-request block ranges. e.g. bodies stage calls set_downloader_range(10..100), consumes 20 bodies from the stream, has some recoverable error during insert, and on the next iteration calls set_downloader_range(10..100) again. since 20 bodies were already returned from the stream, the dl needs to figure out whether there is anything to re-request in the new download range. It does it by checking if the blocks at the start of the range are 1) in progress (being requested from peers), 2) in buffer (waiting to be added to the queue), or 3) in the queue (waiting to be returned).
Bug
With the introduction of the task downloader, some bodies might be in the intermediary buffer (neither in the downloader nor consumed by the bodies stage). The recovery logic doesn't account for that.
Performance Impact
The downloader would send out redundant requests (possibly dozens) and keep active connections processing those. This would prevent the downloader from progressing forward and keep the network component busy.

@mattsse yes, since previous download range reset logic should be currently unreachable


New fork with extended functionalities.

Hi @robfig,
I made this fork from your work called Agenda with extended options. It is still under evaluation, but I would appreciate some input from you, if possible.
There is at least one function that I added and could possibly be a PR for your cron too:
func (c *Cron) Parse(spec string) (sch Schedule, err error) {
	return c.parser.Parse(spec)
}

Thanks in advance.

Cool, neat project. For me, I'm not sure it's useful enough to justify the additional API surface area. I think if the caller cares to call Parse, they should initialize the Parser and provide it to cron on construction rather than fetching the one it creates by default. Thanks for the offer though!


Improve documentation

Documentation here and in CE.jl is insufficient. For example, it's not obvious to users that PythonCall or RCall need to be loaded in order for the package extensions to be loaded.

cc @MaritRadder


How to remove | bar at the beginning of child rows in tree .

You should modify the styling. See here for defaults. Instead of
.has-parent:not(.has-children):before {
  padding-right: 0.5em;
  content: "\0007C";
  font-size: large;
}

you probably want a rule without content.

Yes , I got that point and have removed \0007C from style.css and modified style sheet is
.has-parent:not(.has-children):before {
padding-right: 0.5em;
content: "";
font-size: large;
}
but this changes is not taking any effect , still i am getting that | bar

I cannot help without seeing more code. You should check out browser inspector very carefully. Some rule might override that (could be an order, specificity issue). | is controlled by CSS and there's not much I can change about it code-wise.

Ok thanks for your time , I resolved it .


Hello, the following error occurred while running the program. How can I solve it?

PS C:\gmtest\Agentverse-master\Agentverse-master> node index --daemon
node:internal/modules/cjs/loader:1078
throw err;
^
Error: Cannot find module 'socks-proxy-agent'

Maybe you can run npm install first?
I've wrote socks-proxy-agent in package.json, so maybe the install failed?


Entity Component System

At the moment, the engine is very "hierarchy" based when it comes to game object. This should eventually be moved to a more flat Component based design.

Starting work on this in the develop branch

Please consider two features for yours:

Dependency resolution during runtime, not every frame. Every ECS I've looked at filters events every time you call world.update() instead of caching them and it's slow.
Support polymorphic systems

Decided to integrate Ontology


Bug: [OnchainKitProvider] chain type incompatible

Describe the bug and the steps to reproduce it

Checkout https://github.com/sergical/ossummer/pull/17
Check web/src/OnchainProviders.tsx
See type error

Full TS error
Type '{ blockExplorers: { readonly default: { readonly name: "Basescan"; readonly url: "https://sepolia.basescan.org"; readonly apiUrl: "https://api-sepolia.basescan.org/api"; }; }; contracts: { ...; }; ... 10 more ...; readonly network: "base-sepolia"; } | { ...; }' is not assignable to type 'Chain'.
  Type '{ blockExplorers: { readonly default: { readonly name: "Basescan"; readonly url: "https://sepolia.basescan.org"; readonly apiUrl: "https://api-sepolia.basescan.org/api"; }; }; contracts: { ...; }; ... 10 more ...; readonly network: "base-sepolia"; }' is not assignable to type 'Chain'.
    Types of property 'fees' are incompatible.
      Type 'ChainFees<undefined> | undefined' is not assignable to type 'ChainFees<ChainFormatters | undefined> | undefined'.
        Type 'ChainFees<undefined>' is not assignable to type 'ChainFees<ChainFormatters | undefined>'.
          Types of property 'baseFeeMultiplier' are incompatible.
            Type 'number | ((args: ChainFeesFnParameters<undefined>) => number | Promise<number>) | undefined' is not assignable to type 'number | ((args: ChainFeesFnParameters<ChainFormatters | undefined>) => number | Promise<number>) | undefined'.
              Type '(args: ChainFeesFnParameters<undefined>) => number | Promise<number>' is not assignable to type 'number | ((args: ChainFeesFnParameters<ChainFormatters | undefined>) => number | Promise<number>) | undefined'.
                Type '(args: ChainFeesFnParameters<undefined>) => number | Promise<number>' is not assignable to type '(args: ChainFeesFnParameters<ChainFormatters | undefined>) => number | Promise<number>'.
                  Types of parameters 'args' and 'args' are incompatible.
                    Type 'ChainFeesFnParameters<ChainFormatters | undefined>' is not assignable to type 'ChainFeesFnParameters<undefined>'.
                      The types of 'block.transactions' are incompatible between these types.
                        Type '`0x${string}`[] | ({ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; ... 11 more ...; transactionIndex: number | null; } | { ...; } | { ...; } |...' is not assignable to type '`0x${string}`[] | ({ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; ... 11 more ...; transactionIndex: number | null; } | { ...; } | { ...; } |...'. Two different types with this name exist, but they are unrelated.
                          Type '({ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; typeHex: `0x${string}` | null; ... 10 more ...; transactionIndex: number | null; } | { ...; }...' is not assignable to type '`0x${string}`[] | ({ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; ... 11 more ...; transactionIndex: number | null; } | { ...; } | { ...; } |...'.
                            Type '({ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; typeHex: `0x${string}` | null; ... 10 more ...; transactionIndex: number | null; } | { ...; }...' is not assignable to type '`0x${string}`[]'.
                              Type '{ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; typeHex: `0x${string}` | null; ... 10 more ...; transactionIndex: number | null; } | { ...; } ...' is not assignable to type '`0x${string}`'.
                                Type '{ type: "legacy"; chainId?: number | undefined; yParity?: undefined; from: `0x${string}`; gas: bigint; hash: `0x${string}`; input: `0x${string}`; nonce: number; r: `0x${string}`; s: `0x${string}`; to: `0x${string}` | null; typeHex: `0x${string}` | null; ... 10 more ...; transactionIndex: number | null; }' is not assignable to type '`0x${string}`'.ts(2322)
types.d.ts(31, 5): The expected type comes from property 'chain' which is declared here on type 'IntrinsicAttributes & OnchainKitProviderReact'

Also issue with QueryClientProvider

This is when signed in and Avatar component is rendered (I think) - client stack trace below
Warning: Cannot update a component (`HotReload`) while rendering a different component (`Avatar`). To locate the bad setState() call inside `Avatar`, follow the stack trace as described in https://reactjs.org/link/setstate-in-render at Avatar
QueryClientProvider.js:21 Uncaught Error: No QueryClient set, use QueryClientProvider to set one
    at useQueryClient (QueryClientProvider.js:21:15)
    at useBaseQuery (useBaseQuery.js:28:91)
    at useQuery (useQuery.js:13:74)
    at useName (useName.js:26:73)
    at Avatar (Avatar.js:48:78)

What's the expected behavior?
Render without errors
What version of the libraries are you using?
"@coinbase/onchainkit": "0.22.3",
"viem": "2.x",
"wagmi": "^2.9.2",
"@privy-io/react-auth": "^1.71.0",
"@privy-io/server-auth": "^1.7.3",
"@privy-io/wagmi": "^0.2.10",

I pulled your PR locally and can't reproduce this error.
Try removing the entire node_modules folder and yarn install again.
I am using node 18.18.2 btw

whaaaaaa :/
I tried that and I am getting this in my server stack too
 GET / 200 in 7ms
 ⨯ ../src/QueryClientProvider.tsx (18:11) @ useQueryClient
 ⨯ Internal error: Error: No QueryClient set, use QueryClientProvider to set one
    at useQueryClient (./node_modules/@coinbase/onchainkit/node_modules/@tanstack/react-query/build/modern/QueryClientProvider.js:19:15)

Added an Avatar component to the homepage https://github.com/sergical/ossummer/pull/17/commits/2d86dbb100bb17226b0aaaa7bdc7e673239165a6
Also are you not seeing the type error either in the OnchainProviders file?

What command are you running to get this?
Hmmm let me try checking the typescript. I am using neovim + biome. One sec

neovim + biome
Wouldn't expect any less from you

What command are you running to get this?
yarn dev

What do you get when you run yarn check?
This is what I get
./app/paymaster-bundler/_components/PaymasterBundlerDemo.tsx
32:22  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop

./src/OnchainProviders.tsx
25:11  Error: 'theme' is assigned a value but never used.  @typescript-eslint/no-unused-vars

./src/components/layout/header/AccountConnect.tsx
38:30  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop

./src/components/layout/header/ProfileNavigation.tsx
45:22  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop
54:20  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop

./src/components/layout/header/ThemeToggle.tsx
28:36  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop
29:36  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop
30:36  Warning: JSX attribute values should not contain functions created in the same scope  react-perf/jsx-no-new-function-as-prop

info  - Need to disable some ESLint rules? Learn more here: https://nextjs.org/docs/basic-features/eslint#disabling-rules

Derp. I was on the main branch lol
One sec

LOL
https://www.loom.com/share/299135368cfb4d8893ea4a176c4eb520

Yup I see it now

OK, I think this is a problem from OnchainKit bundling the viem and wagmi in the bundle directly.
Can you stay on for like 15 min? I can push a new release without those bundled in.

For you, I'll pull an all nighter

(kinda wanna make sure all of this works before buildathon deadline at the end of the month lol)
Thank you very much <3

on it

Aight 0.22.4 released. Think it's working now.

TS error went away. Still complaining about queryclientprovider :/

Where is that at? From yarn dev?

yeah, yarn dev and hit localhost:3000

latest on that branch has 0.22.4 and the Avatar component in HomePage.tsx. removing Avatar component doesn't render an error.

Yup see it

I think this is the same problem as we bundle up @tanstack/react-query as well

Forgot this isn't slack and i can't do /giphy but cool to know github is beta testing slash commands

Verified this by removing @tanstack/react-query in the package.json and recompiling after using no cache.

Aight let me move that to peer dependencies as well

Not using graphql but wondering if that might cause an issue down the line too just inspecting the package.json for onchainkit

Hmm yes it might. We need to move away from getting eas attestations via graphql

v0.22.5 released
I'm gonna tackle the graphql stuff tomorrow. Clocked in 12+ hours already today lol

@Zizzamia this man is 🐐 just fyi
gn fren!

everything works as expected now

I think this problem is happening again with last versions of viem (2.21.55) and wagmi (2.14.1).
Type 'import("/Users/xxx/xxx/xxx/node_modules/@coinbase/onchainkit/node_modules/viem/_types/types/chain").Chain' is not assignable to type 'import("/Users/xxx/xxx/xxx/node_modules/viem/_types/types/chain").Chain'.
Also I am using the  which throws the following error:
WagmiProviderNotFoundError: useConfig must be used within WagmiProvider. The above error occurred in the <FundButton> component. It was handled by the <ReactDevOverlay> error boundary. Error Component Stack


Two undefined reference issues with slave code

Hi, I´m trying to implement your library in my code but keep getting errors about undefined references using Visual Studio Code and PlatformIO.
I´m not that into C++ and can´t figure out how to solve this. Am I doing something wrong or is there a bug in your library?
C:\Users\Martin\AppData\Local\Temp\ccmrTEpa.ltrans0.ltrans.o: In function main: <artificial>:(.text.startup+0x2c2): undefined reference to CTWI::SetLocalDeviceAddress(unsigned char) <artificial>:(.text.startup+0x2ce): undefined reference to CTWI::SetSlaveReceiveHandler(void (*)(unsigned char const*, unsigned char)) collect2.exe: error: ld returned 1 exit status *** [.pio\build\nanoatmega328\firmware.elf] Error 1
Thanks in advance!

It's not explained, but there is a define statement which controls whether the slave functions are enabled:
CTWI_ENABLE_SLAVE_MODE: https://github.com/nitacku/nI2C/blob/master/nTWI.h#L29
Simply uncomment this line to enable slave mode.


test_assets() should not check "collection-assets" for 0.x.x versions

test_assets() method is checking existence of "collection-assets" in "stac_extensions" for versions starting with 0.x. However, this is not correct as collection-assets extension was defined only for versions 1.0.0-beta.1 and 1.0.0-beta.2. It doesn't exists e.g. for 0.9.0 (there is a 'assets' extension proposal for that version, but not collection-assets).
Here are the relevant links:
https://github.com/radiantearth/stac-spec/blob/v0.9.0/extensions/collection-assets/README.md (page not found, i.e. extension doesn't exist)
https://github.com/radiantearth/stac-spec/blob/v1.0.0-beta.1/extensions/collection-assets/README.md (page exists)
https://github.com/radiantearth/stac-spec/blob/v1.0.0-beta.2/extensions/collection-assets/README.md (page exists)
https://github.com/radiantearth/stac-spec/blob/v1.0.0-rc.1/extensions/collection-assets/README.md (page not found, i.e. extension doesn't exist - in fact, it is not necessary, as assets become part of the collection specification)
One example collection for which the test fails: https://openeo.dataspace.copernicus.eu/openeo/1.2/collections/SENTINEL1_GRD

https://github.com/Open-EO/openeo-test-suite/blob/4d03e80ac1bc93f2738e18a64aed16eecaed4690/src/openeo_test_suite/tests/collections/openeo_api_collection_tests.py#L111-L122
This test checks for this part of the openEO spec (https://github.com/Open-EO/openeo-api/blob/c5a45b4647b06e313a4f099e9119bfa3cca5c6a3/openapi.yaml#L4514-L4521) about the "assets" property in collection metadata:

Implementing this property REQUIRES to add collection-assets to the list of stac_extensions in STAC < 1.0.0-rc.1.

But indeed,  "collection-assets" is not an valid core extension in STAC 0.0.9 or earlier, as it was only introduced in STAC 1.0.0-beta.1.
@m-mohr  does this mean that the openEO spec should actually state something like

Implementing this property REQUIRES to add collection-assets to the list of stac_extensions when 1.0.0-beta.1 <= stac_version < 1.0.0-rc.1.

In any case, the current implementation of test_assets is a bit off as it naively checks if the version starts with 0., while it should also include 1.0.0-beta.1 and 1.0.0-beta.2 (the only 1.x versions below 1.0.0-rc.1)

looping @clausmichele  too

That's all correct. But I'd like to note that we should deprecate STAC < 1.0.0 in openEO anyway. With that we may also just already remove those < 1.0.0 requirements completely.

Thanks for the fix @soxofaan!


Failures on [BUILD] - Initial CL to unfork the NNBD Dart SDK...Remove ElementSuggestionBuilder

There are new test failures on [BUILD] - Initial CL to unfork the NNBD Dart SDK...Remove ElementSuggestionBuilder.
The tests

vm/dart/causal_stacks/async_throws_stack_non_symbolic_test RuntimeError (expected Pass)

are failing on configurations
dartkp-linux-product-x64

stdout:
RegExps for expected stack:
"^#0      throwSync \(.*/utils.dart:16(:3)?\)$"
"^#1      noYields3 \(.*/utils.dart:54(:3)?\)$"
"^#2      noYields2 \(.*/utils.dart:50(:9)?\)$"
"^#3      noYields \(.*/utils.dart:46(:9)?\)$"
"^#4      doTestAwait "
"^#5      doTestsCausal "
"^<asynchronous suspension>$"
"^#6      main "
"^#7      _startIsolate.<anonymous closure> "
"^#8      _RawReceivePortImpl._handleMessage "

Non-symbolic actual stack:
Warning: This VM has been configured to produce stack traces that violate the Dart standard.
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
pid: 9036, tid:<PHONE_NUMBER>03296, name DartWorker
isolate_instructions: 7fa9315ba000 vm_instructions: 7fa9315b6000
    #00 abs 00007fa93164bb42 _kDartIsolateSnapshotInstructions+0x91b42
    #01 abs 00007fa93164bc33 _kDartIsolateSnapshotInstructions+0x91c33
    #02 abs 00007fa93164bde3 _kDartIsolateSnapshotInstructions+0x91de3
    #03 abs 00007fa931642a58 _kDartIsolateSnapshotInstructions+0x88a58
    #04 abs 00007fa931650050 _kDartIsolateSnapshotInstructions+0x96050
    #05 abs 00007fa93162e442 _kDartIsolateSnapshotInstructions+0x74442
    #06 abs 00007fa9316526f3 _kDartIsolateSnapshotInstructions+0x986f3
<asynchronous suspension>
    #07 abs 00007fa931651931 _kDartIsolateSnapshotInstructions+0x97931
    #08 abs 00007fa931651e05 _kDartIsolateSnapshotInstructions+0x97e05
    #09 abs 00007fa9315f6d70 _kDartIsolateSnapshotInstructions+0x3cd70

Actual stack:
#0      throwSync (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:16)
#1      noYields3 (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:54)
#2      noYields2 (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:50)
#3      noYields (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:46)
#4      doTestAwait (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:224)
#5      doTestsCausal (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/utils.dart:311)
#6      _RootZone.runUnary (dart:async/zone.dart:1450)
#7      _FutureListener.handleValue (dart:async/future_impl.dart:143)
<asynchronous suspension>
#8      main (file:///b/s/w/ir/runtime/tests/vm/dart/causal_stacks/async_throws_stack_non_symbolic_test.dart:16)
#9      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299)
#10     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168)

Expected line 7 to be `^<asynchronous suspension>$` but was `#6      _RootZone.runUnary (dart:async/zone.dart:1450)`

stderr:
Unhandled exception:
Expect.isTrue(false) fails.
Warning: This VM has been configured to produce stack traces that violate the Dart standard.
*** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
pid: 9036, tid:<PHONE_NUMBER>63488, name Dart_Initialize
isolate_instructions: 7fa9315ba000 vm_instructions: 7fa9315b6000
    #00 abs 00007fa931642f56 _kDartIsolateSnapshotInstructions+0x88f56
    #01 abs 00007fa93164942c _kDartIsolateSnapshotInstructions+0x8f42c
    #02 abs 00007fa93162e442 _kDartIsolateSnapshotInstructions+0x74442
    #03 abs 00007fa9316526f3 _kDartIsolateSnapshotInstructions+0x986f3
<asynchronous suspension>
    #04 abs 00007fa931642b66 _kDartIsolateSnapshotInstructions+0x88b66
<asynchronous suspension>
    #05 abs 00007fa931650050 _kDartIsolateSnapshotInstructions+0x96050
    #06 abs 00007fa93162e442 _kDartIsolateSnapshotInstructions+0x74442
    #07 abs 00007fa9316526f3 _kDartIsolateSnapshotInstructions+0x986f3
<asynchronous suspension>
    #08 abs 00007fa931651931 _kDartIsolateSnapshotInstructions+0x97931
    #09 abs 00007fa931651e05 _kDartIsolateSnapshotInstructions+0x97e05
    #10 abs 00007fa9315f6d70 _kDartIsolateSnapshotInstructions+0x3cd70

There are no failures in the current results, though some tests are skipped in dartkp mode.
@cskau-g Could you ensure our stack trace tests cover JIT/AOT and are not skipped except on configurations that really don't make sense. Possibly the status file needs some cleanup.

I've cleaned up the status files and it looks like things are running as intended.


Consider renaming Codemirror.js to Codemirror.jsx

I'm using webpack, because Codemirror.js extension is js but it contains JSX code, I need to pipe node_modules through babel. Maybe consider renaming this file to .jsx so the webpack config can ignore .js files in node_modules.

Sorry @sporto - that was actually a mistake in the package.json, which should be pointed at the lib folder instead (see #3)
Fixed now.


Dynamically evaluating conditions

Hi,
No issue with the article, just a question regarding the evaluation of conditions.
In Azure Devops you have conditions like the following:
condition: and(contains(variables['build.sourceBranch'], 'refs/heads/master'), succeeded())
I'm looking to implement something similar to this in a project and wondered if this is a standard syntax?
Is this part of a library that can be re-used elsewhere to dynamically evaluate conditions or is this something unique to Azure Devops?
It sort of looks like Powershell, but not sure how the condition and variables are evaluated.
Anyone mind providing some insight if allowed?
Thanks.

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

ID: 3f151218-9a11-0078-e038-f96198a76143
Version Independent ID: 09c4d032-62f3-d97c-79d7-6fbfd89910e9
Content: Conditions - Azure Pipelines
Content Source: docs/pipelines/process/conditions.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

@philip-reed Thank you for the question.
Please see if expressions has the information you need.
In particular, the example $[eq(variables['Build.SourceBranch'], 'refs/heads/master')] will allow the use of variables to bring about a conditional True or False.


Codegen uses in-memory file system

Closes #294
This makes the codegen use a simple in-memory file system. This makes the test have no side-effect on the code, and it's easier to write the changed files with the correct line-endings.
Implementation wise:

[x] Wrote simple file system class in  new files.py module. A singleton is created.
[x] Use that file system instead of the real file system in all codegen modules.
[x] Move main() and its subfunctions from __main__.py  to __init__.py, so it can be easier imported.
[x] The main() does not write to disk.
[x] The __main__.py  calls main() and then writes to disk.

I renamed "file system" to "file cache", which sounds less intimidating.

@FirefoxMetzger could you check that with this change, running python codegen does not change the line endings? (i.e. an empty git status?)

Yep, that appears to do the trick. I ran the testsuite and now there are no modifications upon completion:
(.venv) PS C:\Users\Sebastian\Documents\Coding-Projects\wgpu-py> git status
HEAD detached at upstream/codegen
nothing to commit, working tree clean


Heroku preparations

I've made a couple of things to prepare the deployment to heroku:

add spring security to redirect all http requests to https
add bucket4j to  limit the requests per minute
don't log.info idle connection clean up
refactor .properties files to .yml
remove context path because it's redundant with heroku subdomain
introduce preview profile
Integrationtests now implement interface WithIntegrationTestProfile

 PoC Heroku Deployment


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1469

Questions ?
Please refer to the Backport tool documentation


False positives

Hi there, lots of 921110 errors...
ModSecurity: Warning. Matched "Operator `Rx' with parameter `(?:get|post|head|options|connect|put|delete|trace|track|patch|propfind|propatch|mkcol|copy|move|lock|unlock)\s+(?:\/|\w)[^\s]*(?:\s+http\/\d|[\r\n])' against variable `REQUEST_BODY' (Value: `<?xml version="1.0" ?>\x0a<d:propfind xmlns:d="DAV:">\x0a  <d:prop>\x0a    <d:getetag/>\x0a  </d:pro (23 characters omitted)' ) [file "/etc/modsecurity/owasp/coreruleset/rules/REQUEST-921-PROTOCOL-ATTACK.conf"] [line "33"] [id "921110"] [rev ""] [msg "HTTP Request Smuggling Attack"] [data "Matched Data: propfind xmlns:d="dav:">\x0a found within REQUEST_BODY: <?xml version="1.0" ?>\x0a<d:propfind xmlns:d="dav:">\x0a  <d:prop>\x0a    <d:getetag/>\x0a  </d:prop>\x0a</d:propfind>\x0a"] [severity "2"] [ver "OWASP_CRS/3.3.2"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/210/272/220/33"] [hostname "xxxxxxxxx"] [uri "/remote.php/dav/files/a.xxxxxxxxx/"] [unique_id "166756463217.934518"] [ref "o26,25v844,105t:urlDecodeUni,t:htmlEntityDecode,t:lowercase"]
just detecting:
---fIkF7ECV---C--
<?xml version="1.0" ?>
**<d:propfind xmlns:d="DAV:">**
  <d:prop>
    <d:getetag/>
  </d:prop>
</d:propfind>

Maybe could be good to include an exception for this.
Thanks.

Same issue as https://github.com/coreruleset/coreruleset/issues/2973, but for the new plugin.

@feanorknd I created a pull request https://github.com/coreruleset/nextcloud-rule-exclusions-plugin/pull/42 to fix FPs relating to file uploads, could you let me know if this fixes your issue?


Practice using issues on Github

Problem
HRB knows how to create issues just not in GitHub.
Solution
HRB is creating this issue in Github as practice.

[x] does this checklist work?
[x] can issues be edited?

Looks to me like HRB knows how to create issues in GitHub now.

Why did you add the "good first issue" label? Is that standard practice? @psokolhessner

Doesn't do much functionally here, but bigger picture & longer term it can help organize issues. Click "Labels" on right hand side to see all the options (there are many; and these are configurable so if we want to change the possible issue labels for a repository, we 100% can). I think of them like 'tags'.

Alright, @hayleyrbrooks given that you appear to be comfortable opening issues, worth closing one too. Add a comment to close this thing out.

Issues unlocked! Closing this one out.


Segmentation fault: 11 in macOS Catalina 10.15.7 using grpcio==1.34.0 and grpcio-tools==1.34.0

What version of gRPC and what language are you using?
grpcio version: 1.34.0
grpcio-tools version: 1.34.0
Python version: 3.8.5
What operating system (Linux, Windows,...) and version?
macOS Catalina 10.15.7
What runtime / compiler are you using (e.g. python version or version of gcc)
tanjuntao@mbp:~$ python3 --version
Python 3.8.5
tanjuntao@mbp:~$ gcc --version
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/4.2.1
Apple clang version 12.0.0 (clang-1<IP_ADDRESS>)
Target: x86_64-apple-darwin19.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin
tanjuntao@mbp:~$ which python3
/usr/local/bin/python3
tanjuntao@mbp:~$ which gcc
/usr/bin/gcc
tanjuntao@mbp:~$

What did you do?
Here is a snippet of my project hierarchy:
.
├── protos
│   └── lr.proto
├── service
│   ├── __init__.py
│   ├── lr_pb2.py
│   ├── lr_pb2_grpc.py
│   └── tst.py
└── test.py

My .proto file is defined in protos directory and inside service directory, I use grpc_tools.protoc to compile the lr.proto. Then I new an __init__.py in service to make it a python package.
Here is the problem, the content of service/tst.py is :
import lr_pb2
import lr_pb2_grpc

When I run python3 tst.py in iTerm2 terminal, I got the Segmentation fault: 11 error.
I tried to debug this error, and the image below showed which line threw the error:

I could not figure it out, so then I tried to add a import grpc sentence before import lr_pb2 as reported in this issue #24897
import grpc
import lr_pb2
import lr_pb2_grpc

The error disappeared.
Below are some other things I have done.
1. Test the same code in a Linux system
I tested the same code in Ubuntu 16.04 LTS with grpcio==1.34.0 and grpcio-tools==1.34.0, the result was whether I import grpc or not in service/tst.py, no segmentation fault error was reported.
2. Degrade grpcio and grpcio-tools version
I uninstalled grpcio==1.34.0 and grpcio-tools==1.34.0 with pip3 and re-installed grpcio==1.33.2 and grpcio-tools==1.33.2. Then I re-generate the pb2 and pb2_grpc files under service directory.
Then I run python3 tst.py in terminal, the result was: whether import grpc was included in tst.py or not, there was nosegmentation fault: 11 error.

BTW, there is another question I am really curious about. I guess it's my misuse of gRPC instead of its own bug.
Here is what it is:
Outside service, the content of test.py is:
from service import lr_pb2
from service import lr_pb2_grpc

Whether I am using grpcio==1.33.2 or grpcio==1.34.0, when I run test.py I always get this error:
Traceback (most recent call last):
  File "test.py", line 3, in <module>
    import service.lr_pb2_grpc as lr_pb2_grpc
  File "/Users/tanjuntao/Desktop/lr_grpc/service/lr_pb2_grpc.py", line 5, in <module>
    import lr_pb2 as lr__pb2
ModuleNotFoundError: No module named 'lr_pb2'

When working with grpcio==1.33.2, I modify lr_pb2_grpc.py and change the import sentence from:

import lr_pb2 as lr__pb2

to:
import service.lr_pb2 as lr__pb2

And the error disappears.

When working with grpcio==1.34.0, I need to modify the lr_pb2_grpc.py as above, I should also add a import grpc sentence before import lr_pb2 in test.py. The test.py becomes:

import grpc
from service import lr_pb2
from service import lr_pb2_grpc

Only in this way, the error would disappear. If I do not include the import grpc sentence, segmentation fault: 11 still arises.
What did you expect to see?
No segmentation fault :11 in macOS Catalina 10.15.7 when working with grpcio==1.34.0 and grpcio-tools==1.34.0.
What did you see instead?
Segmentation fault: 11
Anything else we should know about your project / environment?
Nope.

Duplicate of https://github.com/grpc/grpc/issues/24897. 1.34.1 was released earlier today, which should resolve this issue.

Duplicate of https://github.com/grpc/grpc/issues/24897. 1.34.1 was released earlier today, which should resolve this issue.

I see you added a usage question as well. Try adding the service directory to sys.path before attempting the import.

I see you added a usage question as well. Try adding the service directory to sys.path before attempting the import.

Thanks for your help, it works!

Thanks for your help, it works!


Cannot read property 'length' of null

I get this error with io.js v1.0.3:
$ npm test

><EMAIL_ADDRESS>test ./crx
> node ./test/index.js

TAP version 13
# it should pack the test extension
buffer.js:170
      length += list[i].length;
                       ^
TypeError: Cannot read property 'length' of null
    at Function.Buffer.concat (buffer.js:170:24)
    at null.<anonymous> (./crx/src/crx.js:220:29)
    at emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:404:10)
    at emitReadable (_stream_readable.js:398:7)
    at onEofChunk (_stream_readable.js:381:3)
    at readableAddChunk (_stream_readable.js:126:7)
    at Readable.push (_stream_readable.js:106:10)
    at Transform.push (_stream_transform.js:120:32)
    at done (_stream_transform.js:183:17)
npm ERR! Test failed.  See above for more details.

Yep, same issue as per Node v0.11 as well.
I have added iojs support in CI to see how it goes.


Add metascore integration

So, we have this project to be a leaderboard hub / aggregator for IC games. This PR integrates the reversi game in Metascore. This is more of a proposal than a ready-to-ship PR: canister ids shouldn't be overwritten in any final PR, and I haven't tested how the upgrade would affect state.
It would be great to get this integration in, so that everyones' existing high scores could make their way into Metascore!

This is an interesting PR. Thanks!
The changes to deliver scores to Metascore looks a bit invasive to me. Will it be possible to use a pull model instead of push?
Also it seems to rely on a critical assumption "a user has the same principal across multiple games" to make this work. Am I reading it correctly?

Yes, we support pull and push. At the moment pulls are very infrequent, so
push is just better for being up to date.
Yes common principal across cans is critical. The Stoic wallet provides
that in this PR. Metascore also supports plug.
We are currently running this code in our own canisters, which you can see
by going to metascore.one
Not sure if it would make sense to merge these changes to your upstream,
but wanted to give you a heads up, and would be happy to help get this PR
in a better state if you wanted to merge!
On Wed, Sep 22, 2021 at 4:09 PM Paul Liu @.***> wrote:

This is an interesting PR. Thanks!
The changes to deliver scores to Metascore looks a bit invasive to me.
Will it be possible to use a pull model instead of push?
Also it seems to rely on a critical assumption "a user has the same
principal across multiple games" to make this work. Am I reading it
correctly?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ninegua/reversi/pull/14#issuecomment-925392137, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AOQ5SUMUNB6ZQTMCGQWF2LDUDJOZTANCNFSM5EFUE2LA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


SpaceDock catch-up 15

KSP-SpaceDock/SpaceDock#188 is not fixed.
These are the mods added since #7136.
ckan compat add 1.4 1.6

#rebuild

#rebuild


Fix custom emojis for statuses not loaded on page refresh

Summary

Fetch custom emoji for custom status only if not loaded.

Ticket Link

https://mattermost.atlassian.net/browse/MM-33210

Related Pull Requests

https://github.com/mattermost/mattermost-webapp/pull/7592

Refer the conversation here: https://github.com/mattermost/mattermost-webapp/pull/7592#discussion_r583879528

Hello @chetanyakan,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

How does this behave if you have multiple CustomStatusEmoji mounted for the same user? Does it send out multiple requests at once for the same emoji? I'd be worried about that with loading them in the component.

Is that even possible?

Yeah, if you've made multiple posts in a channel, this component appears next to each of them in the channel, so theoretically it could send out multiple requests as long as those aren't deduplicated somehow.

Yeah, if you've made multiple posts in a channel, this component appears next to each of them in the channel, so theoretically it could send out multiple requests as long as those aren't deduplicated somehow.

In theory ... yes, you are right!
although it only dispatches an action... should'nt redux handle this in case of duplicate requests?

although it only dispatches an action... should'nt redux handle this in case of duplicate requests?

Nope, unfortunately not. A few routes used to have that logic, but they were always one-off things. I think that those might have been removed when switching to redux too. That is something that would be very useful though.

The way I suggested doing it in the previous PR (loading emojis for statuses when receiving user profiles) should still work though.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @jfrerich @emilyacook

The way I suggested doing it in the previous PR (loading emojis for statuses when receiving user profiles) should still work though.

@hmhealey Apart from the users visible in LHS (for DMs and Group chats), the other user profiles are fetched on a need-to-know basis. We keep getting new user profiles on switching channels or even scrolling up in a channel and also, the users' status emojis also keep changing.
I can't find a single place where a loadCustomEmojisForUsers common API can be called to effectively ensure all custom emojis are always loaded.
Can you elaborate on how you were suggesting to solve this?

Unfortunately, it's not as easy as I originally thought since we load profiles/statuses in a few different ways which makes this tougher. I was hoping it was more like posts where every new post goes through one function...
So the first place is loading profiles in bulk for certain components. I think if you look at where loadStatusesForProfilesList is used and call loadCustomEmojisForUsers at the same time, that should cover most cases. There'll be changes to three components (the More DMs modal and channel/team members modals) and then adding to a few functions in status_actions.jsx. That should cover all cases where a component is mounted and needs some stuff.
Next, we'd need to handle profiles loaded from posts. Thankfully, those are all done in one place like I mentioned, so you just need to add it to getProfilesAndStatusesForPosts. That could be done in the same call where custom emojis are loaded for posts, but I don't think that's actually necessary since posts come with their custom emojis included now.
Lastly, we'd need to handle the websocket event for a user being updated. That's another simpler one though since it's just handleUserUpdated though.
I think that's all of them. We definitely need to look into a better way of doing this sort of incidental loading logic since it's obviously not great how we do it currently. We got away with a lot of that previously be passing down a lot more data with post requests (post metadata), but we don't have this for other objects like users, channels, etc, so it makes it tough to add additional info that's not directly in the object

@chetanyakan friendly ping on this PR, are next steps clear or should we sync up on it?

@amyblais if this gets completed in time I'd like to cherry pick it to v5.35

Gentle ping @hmhealey @ogi-m on this one

@ogi-m Can you review/test this PR?

@ogi-m @jgilliam17 gentle ping on this


test: investigate the metrics test flakiness

We should deflake or at very least investigate and improve the metrics test, which after #194 landed is flaking a bit more (but this is fine because #194 is such a massive improvement that it's well worth the price tag)

Fixed by https://github.com/k8stopologyawareschedwg/resource-topology-exporter/pull/195 adding Eventually around the flaky test, which is fine and what I'd have done.

If memory serves, metrics just always take some time before to be populated (that's expected, documented and bening, but of course I don't have docs handy). If the metrics test executes too early, because ginkgo random ordering, that could explain the flakiness

also addressed by #201


[IDE] Implement completion-like cursor info for ValueDecls

This brings up the ability to compute cursor info results using the completion-like type checking paradigm, which an reuse ASTContexts and doesn’t need to type check the entire file.
For now, the new implementation only supports cursor info on ValueDecls (not on references) because they were easiest to implement. More cursor info kinds are coming soon.
At the moment, we only run the new implementation in a verification mode: It is only invoked in assert toolchains and when run, we check that the results are equivalent to the old implementation. Once more cursor info kinds are implemented and if the SourceKit stress tester doesn’t find any verification issues, we can enable the new implementation, falling back to the old implementation if the new one didn’t produce any results.

@swift-ci Please smoke test

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test macOS

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test macOS

@swift-ci Please SourceKit stress test

Note to self: The stress tester failure at offset 300 of modification concurrent-359 of rows/MovieRow.swift has the following reduce reproducer:
// RUN: %sourcekitd-test -req=cursor -req-opts=retrieve_symbol_graph=1 -pos=5:12 %s -- %s

let formatter: DateFormatter = {
  struct MovieRow {
    struct Props

The solver-based implementation reports Props as conforming to Sendable in symbol graph, the AST-based implementation does not report this. I think this change is acceptable and I’ll just mark this discrepancy as an XFail in the stress tester.

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test

@swift-ci Please smoke test macOS

@swift-ci Please SourceKit stress test

@swift-ci Please smoke test


Prisma migrate dev: Error: not reachable

Hi Prisma Team! The following command just crashed.
Command
migrate dev --schema [redacted]
Versions

Name
Version

Platform
windows

Node
v18.16.1

Prisma CLI
4.10.0

Engine
ca7fcef713137fa11029d519a9780db130cca91d

Error
Error: unreachable

@Mukeshkumar9787 Could you provide a reproduction for this?
Also, did you try using 4.16.2 or the latest version 5.1.1 to see if that's already fixed maybe?

@Mukeshkumar9787  did you see the message from @Jolg42 ? Without additional information to be able to reproduce, we would later close this issue as part of our issue process.

Hey @Mukeshkumar9787 ,
With the lack of information and our inability to reproduce this issue, we're now closing it. Feel free to comment if you have any updates about it.
Thank you 🙏


comments update

Coverage decreased (-0.1%) to 50.904% when pulling c9c1354467dffa06958b95416c7c69aeef6c5539 on lcestou:patch-14 into 1a140d21e2b9403b54ed0d793608c817f9f6c15d on TimothyLuke:master.


fix: fix fouc for drilldown menus #11770

The .no-js fix for preventing FOUC in other components doesn't seem to have been applied to the drilldown component. This results in FOUC for drilldown sub-menus. This applies the .no-js fix to child menus of the drilldown component to prevent the FOUC.

Description

I added the .no-js FOUC fix to target the child menus of the drilldown component.

Closes #11770

Types of changes

[ ] Documentation
[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (anything that would change an existing functionality)
[ ] Maintenance (refactor, code cleaning, development tools...)

Checklist

[X] I have read and follow the CONTRIBUTING.md document.
[X] The pull request title and template are correctly filled.
[X] The pull request targets the right branch (develop or develop-v...).
[X] My commits are correctly titled and contain all relevant information.
[X] I have updated the documentation accordingly to my changes (if relevant).
[X] I have added tests to cover my changes (if relevant).

Hi @noahheck,

The .no-js fix for preventing FOUC in other components doesn't seem to have been applied to the drilldown component

I checked some scss files and did not see .no-js there. Which fix do you mean? Your change would probably preent that the content is not displayed for users without JS which is bad for the accessibility.

Hi @noahheck,
thanks for your contribution to Foundation Sites.

The .no-js fix for preventing FOUC in other components doesn't seem to have been applied to the drilldown component

I checked some scss files and did not see .no-js there. Which fix do you mean? Your change would probably preent that the content is not displayed for users without JS which is bad for the accessibility.

@DanielRuf
Here is how the .no-js is applied in the dropdown-menu component:
https://github.com/zurb/foundation-sites/blob/develop/scss/components/_dropdown-menu.scss#L165

Ah ok.
We should still check if we can do it this way or if we have to adjust some attributes.
https://github.com/zurb/foundation-sites/blob/develop/js/foundation.dropdown.js#L82
cc @Owlbertz

So the potential issue here is if JS doesn't load, the menu is completely hidden.
We've generally had users handle this type of behavior on their own, for example: https://foundation.zurb.com/sites/docs/responsive-navigation.html#preventing-fouc
@DanielRuf, thoughts?

If we document it, it should be fine @JeremyEnglert.

Closing as this should be handled by users depending on their requirements.


Make sure paddle-paddle works (on totally looks alike dataset)

Closed due to dropping of tf and paddle in near future


Feature/1148 area of plants planting element

This PR adds a AreaOfPlantings component that represents an area of plantings created by dragging during planting mode.
The AreaOfPlanting differs from a usual Planting by having an isArea property that is set to true in case of the AreaOfPlantings.
Basics

[ ] I added a line to changelog.md
[ ] The PR is rebased with current master.
[ ] Details of what you changed are in commit messages.
[ ] References to issues, e.g. close #X, are in the commit messages and changelog.
[ ] The buildserver is happy.

Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines

Amazing job, we both love it! Follow-up: #1154


 And it doesnt involve you either.
And it isnt useless, its so dan doesnt need to change the files later.  So dont reply to something which doesnt involve you

wtf is ur username

wtf is ur username

You're really trying to start a fight in github comments 💀

Do u mean my github username or my discord username.
As my github username is quite clear, CurryBoi29. Not my fault if you cant read english

shut it

shut it

According to world population studies, approximately 108 billion people have lived on this planet assuming that the average lifespan of all of these people was 25, there has been around 2.7 trillion years of life, if we multiply this by the number of days in a year which is (365) there is a total of 985,500,000,000,000 days of life ( 985.5 trillion days).  If you all didn't know already.

Shit posting on this repo gets you banned by dan. If you all didn't know already.

WHO ASKED? 🖕

Shit posting on this repo gets you banned by dan. If you all didn't know already.

 If you all didn't know already.

 Its not you who needs this. Its me who needs it.

Lgtm, but wouldn't dis let curry generate as much premium codes as he wants

Lgtm, but wouldn't dis let curry generate as much premium codes as he wants

Yes that's why I don't like it, it would be an unfair advantage  above other users

Lgtm, but wouldn't dis let curry generate as much premium codes as he wants

Yes that's why I don't like it, it would be an unfair advantage above other users

Wdym it’s so curry can do giveaways he not gonna abuse it as dan would know

Lgtm, but wouldn't dis let curry generate as much premium codes as he wants

Yes that's why I don't like it, it would be an unfair advantage above other users

Wdym it’s so curry can do giveaways he not gonna abuse it as dan would know

True but why do giveaways  if he does too many giveaways  of prem server yk what can happen?

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

Because Dans busy most of the time and dosnt respond to dms

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

When was the last time Dan replied to your DMs .__.

Around 700 years ago

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

Because Dans busy most of the time and dosnt respond to dms oftern

He can dm Dan 10 or 20 days before the game and he can always ask Dan for checking  dms

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

When was the last time Dan replied to your DMs .__.

Around 700 years ago

Exactly that’s why this pr

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

Because Dans busy most of the time and dosnt respond to dms oftern

He can dm Dan 10 or 20 days before the game and he can always ask Dan for checking dms

That still dosnt work why do u think normaly there is a lot of tickets ect open because Dan dosnt have time to do them

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

Because Dans busy most of the time and dosnt respond to dms oftern

He can dm Dan 10 or 20 days before the game and he can always ask Dan for checking dms

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

When was the last time Dan replied to your DMs .__.

Around 700 years ago

Exactly that’s why this pr

Why not just dm Dan and ask for around 10 - 15 premium codes rather then thus pr

When was the last time Dan replied to your DMs .__.

Around 700 years ago

You seemed to have answered your own question "why not DM dan for the codes"

astolfo is pretty cute ngl

Yes that's why I don't like it, it would be an unfair advantage above other users

as ive said around 50000 times. Dan gave me rules on how many times i can use the commands and the amount of giveaways i can host. Like why would i abuse it, im not a dickhead.
And if dan was so worried why would he have given me the option in the first place?
It also isnt unfair, i have -1000 servers so it wouldnt be beneficial to me at all. Like do yall not want giveaways or something? I can perfectly stop doing them if u really want me to.


Add an error when trying to attribute a function to a variable

Summary or solution description
The compilation did not stop when you tried to attribute a function to a variable, only when you were trying to call it later.
Now you get a NotSupported error.
How to Reproduce
https://github.com/CityOfZion/neo3-boa/blob/fbbb3c7edfc163c87a01ad3489b096305c4b9231/boa3_test/test_sc/variable_test/AssignFunction.py#L1-L12
Tests
https://github.com/CityOfZion/neo3-boa/blob/fbbb3c7edfc163c87a01ad3489b096305c4b9231/boa3_test/tests/compiler_tests/test_variable.py#L962-L964
Platform:

OS: Windows 11 x64
Python version: Python 3.7

Task linked: CU-86a0b40vw Add an error when trying to attribute a function to a variable

coverage: 91.696% (-0.3%) from 91.954% when pulling fbbb3c7edfc163c87a01ad3489b096305c4b9231 on CU-86a0b40vw into 3bd6bed56f84099b1256798fabe975c143ed1466 on development.


[Timestampable] on change of relation in join table

What's the correct set up for updating timestamp when deleting a relationship on a join table?
class Contact {

    /**
     * @var \DateTime $listUpdated
     *
     * @Gedmo\Timestampable(on="change", field="lists")
     * @ORM\Column(type="datetime")
     */
    private $listUpdated;

    /**
     * @ORM\ManyToMany(targetEntity="AppBundle\Entity\Lists", inversedBy="contacts", cascade={"persist"})
     * @ORM\JoinTable(name="contact_list",
     *      joinColumns={@ORM\JoinColumn(name="contact_id", referencedColumnName="id")},
     *      inverseJoinColumns={@ORM\JoinColumn(name="list_id", referencedColumnName="id")}
     *      )
     */
    private $lists;

I do not think timestampable can be triggered when the many to many relation changes. because the Contact table does not change and the change is only applied to the other table - many to many association one. I do not think it would be a simple task to support this, since all objects in such case would need to be analyzed having relation with the watched timestampable object.