periodic boundary of cKDTree

Is there a plan to include a periodic boundary in cKDTree?

Not that I know of. Not even sure what that means exactly. Can you explain?

As in a toroidal space? I am not even sure a kd-tree is a valid partition for such a geometry, but it might be (I have to check).

I mean where x + nL and x are the same location. (not sure what a toroidal space is) In cosmology simulations we use KD-Tree to look for neighbours in a periodic box all the time. Here is an example: https://github.com/N-BodyShop/fof/blob/master/kd.c
The simplest solution is probably merging the results of all 3Ndim images. But that's not very efficient (e.g. 33 = 27)

Yes, then we mean the same thing. Consider a 2 dimensional universe. If both dimensions are periodic, you actually have a doughnut shaped universe, i.e. a torus.

I see. What is a better partition for such a space? We cosmologists may have been using the wrong data structure for too long!
anyhow, I recalled that I implemented this toroidal boundary on a KD-Tree, with a modified 'distance' function. I only used this for euclidean distances, but it seems to be able to extend to other norms in cKDTree. The bottom line is that using a KD-Tree on a 'torus' space is possible.
The idea is to use a four segment function to estimate the min/max distance between two nodes in a direction rather than a two segment function (for non-periodic). the distance between two points is also wrapped with the classical half box trick.
The relevant code pieces are here:
https://github.com/rainwoodman/kdcount/blob/master/kdcount/kd_count.h#L42
https://github.com/rainwoodman/kdcount/blob/master/kdcount/kdcount.h#L376
and
https://github.com/rainwoodman/kdcount/blob/master/kdcount/kd_count.h#L147
Will it be possible to put something like this in the current cKDTree code?

If it is just a modification of the distance function, I think we could, unless it incurs a performance regression. I don't think it should, because of NPY_LIKELY and NPY_UNLIKELY and branch prediction. It would then mean that your periodic kd-tree would take the performance it of having the CPU pipeline flushed whenever the distance function is called. This is possibly not very nice, so you might prefer to have this in a separate kd-tree object instead.
If it also means that we have to change the logic that adds kd-tree nodes to the priority queues and trackers, then I would not recommend it in the current cKDTree. It would be a terrible mess.
There is also a data structure called ball-tree, which is faster to search than a kd-tree (but slower to construct) and also allows more arbitrary distance functions. kd-trees are limited to Minkowski distances, as far as I know. I have a plan of proposing that for scipy.spatial later on.  😊

I have a feeling that your proposal will be easier to implement with a ball-tree, because we might need to rearrange the nkde tracing code in cKDTree to make a periodic distance function work.

Closing this -- Implemented by #5091.


Provide guidance on microblocks UX for app integration

Currently we have a bit of stub guidance for how apps should integrate microblocks into their UX here:
https://docs.stacks.co/understand-stacks/microblocks#application-design

The state of microblock transactions should be carefully communicated to users. No transaction is final until it's included in an anchor block, and your application design should reflect this.

I wonder whether we could start providing more specific guidance here that would be actionable for developers such as @yknl who was asking about this for Xverse.
I also notice that we're suggesting the UX should not reflect finality of transactions until inclusion in anchor blocks whereas @jasperjansz has expressed the opposite approach may be best (that apps can consider transactions final upon inclusion in microblocks and before anchor blocks). So we may need to decide more clearly just what approach we want to recommend.
We're also planning to release our own UX implementations in the wallets and explorer soon, which consider transactions as rather final upon inclusion in microblocks – but also distinguish between the two states with visual treatment in the explorer only. Perhaps we could simply lay out our two distinct approaches for the wallets and explorer, explaining our rationale for each.
See https://www.figma.com/proto/i2H9felRYWAZQgsfX7EpZz/microblocks?node-id=39%3A4125
cc @pgray-hiro @jasperjansz

the opposite approach may be best (that apps can consider transactions final upon inclusion in microblocks and before anchor blocks)

This is really something that I would want to get @diwakergupta to weigh in on.

the opposite approach may be best (that apps can consider transactions final upon inclusion in microblocks and before anchor blocks)

This is really something that I would want to get @diwakergupta to weigh in on.

Happy to chime in. Agreed we should provide more specific guidance: the biggest challenge here will be to provide simple, clear, concise and actionable guidance while taking care to set expectations / capture relevant nuance for microblock behavior.
It's fine to point to Hiro's wallets as one way apps could design the UX for microblocks. Other apps could make different choices if they want to.
Note that we now have some empirical data for microblocks, and we should use that to inform the guidance.

No transaction is final until it's included in an anchor block, and your application design should reflect this.

Aside: The original "no transaction is final until" language could be softened IMO -- as I've mentioned previously, using "final" in the colloquial sense (i.e. permanently irreversible) is technically inaccurate in any blockchain that permits forks. In theory, even a transaction included in an anchor block has some non-zero probability of being reversed (that decreases exponentially as more blocks build on top of that anchor block)

Thanks @diwakergupta 🙏 I spoke with @jasperjansz about this earlier today and he's going to chime in with his thoughts as well.

If I were to advise a developer I'd say:

There's no "final", finality is always probabilistic. More block confirmations leads to a higher probability.
For your app, what is more important: speed or finality? Coinbase used to wait 6 block confirmations before crediting an incoming transaction to your account — that's very cautious and not fast.

If you're dealing with an app where the value of a transaction is low and speed is important, such as Heystack, it makes sense to consider inclusion in a microblock "confirmed". It would lead to a much better user experience. Now let's say you're an exchange where value can be really high and speed is not that important, you'd probably want to wait for at least one anchor block.

Apologies in advance for a brain dump in (perhaps) the wrong place, but to follow up on some of the discussion here and expand a bit on @jasperjansz's response, which I agree with:
Dr. Microblocks or: How I Learned to Stop Worrying and Love Reorgs
First things first, what are microblocks, and why are they so scary?
Microblocks are merely blocks of transactions included by a miner after they mined their block, but before the next block has been selected. Transactions included in these microblocks have been processed by the network, and their results are known. But these transactions are as confirmed as the transactions in the preceding block.
The way to think about this is that transactions in microblocks are really “pending” state, not terribly different from a transaction that is still in the mempool. It’s more likely to get confirmed than a transaction just sitting in the mempool, but the security status of the transaction isn’t different.
If you think of the world purely in terms of the number of “confirmations” a transaction has, you can compare the lifecycle of transactions in microblocks and transactions in anchor blocks as follows:
Transaction 1 is broadcasted to the mempool. It has 0 confirmations.
Transaction 1 is included in a microblock. It still has 0 confirmations, but the results of the transaction are know “known”.
The next Stacks block confirms the prior microblock. Transaction 1 has 1 confirmation.
The next Stacks block confirms the prior block. Transaction 1 has 2 confirmations.
The next Stacks block confirms the prior block. Transaction 1 has 3 confirmations.

Now instead, think about a transaction included in an anchor block directly:
Transaction 2 is broadcasted to the mempool. It has 0 confirmations.
The next Stacks block includes Transaction 2. Transaction 2 has 1 confirmation.
The next Stacks block confirms the prior block. Transaction 2 has 2 confirmations.
The next Stacks block confirms the prior block. Transaction 2 has 3 confirmations.

The lifecycles of the two transactions are very similar. So what’s the point of the microblocks? The answer is in computing pending state.
In Bitcoin wallets, it’s very common for wallets to display 0-confirmation balances: i.e., your wallet balance with any mempool transactions affecting your wallet applied. This is useful, because it tells you when you’ve sent a transaction or received one. Many applications which aren’t so worried about transactions being reversed can even operate with a 0-confirmation transaction. However, in blockchains with smart contracts, displaying pending state is much harder. A given transaction isn’t just transferring inputs to outputs, it could be calling smart contracts, emitting events, etc. A transaction processed in a microblock generates all of that information.
So how do I work with microblocks?
In most cases, information from microblocks should be treated like information from any other block. Wallets and explorer should display the consequences of microblock transactions as the current state of the network. However, this state should be acknowledged as tentative somehow. Below is how blockchain.com displays an address. The balance includes pending state. That mempool transaction can easily be replaced and double-spent.

Furthermore, a microblock transaction can end up being “reorganized” into the next block rather than just confirmed as is — this happens roughly 25% of the time according to our statistics so far (see: https://github.com/blockstack/stacks-blockchain-api/issues/640#issuecomment-880142249). Because of this, the UI should be able to explain to users what “happened” if the outputs of the transaction changed, or if the transaction’s associated block changed (even a tooltip like: “this transaction was originally included in a microblock, but ended up moving into an anchor block instead”). This is the exact same outcome as if a 1-block fork occurred.
But really, how do I work with microblocks?

APIs

State querying endpoints (like say a balance lookup) should come in two varieties (or, to reduce API calls, include two sets of results): “confirmed” and “unconfirmed”. We can hard code what “confirmed” means (i.e., 1 confirmation), or it could be made configurable (someone could specify that “confirmed” means 3 confirmations). Unconfirmed would always return state with all known microblocks applied.
Transaction lookups would return information about the transaction in the canonical fork. They should also return information about whether or not the transaction was previously included in another block in another canonical fork.

Explorers

Display pending state, but warn that a transaction is still pending. Indicate in the balance that it depends on pending state.

Wallets

Display pending state, but warn that a transaction is still pending. Indicate in the balance that it depends on pending state.

Exchanges

Continue to count confirmations like before — microblocks shouldn’t make any difference here.

Applications

This is super-app-dependent. In many applications, 0-confirmations is fine! BNS is almost certainly one of these. In other applications, 3-confirmations is probably prefered. But like exchanges, microblocks don’t change that calculus at all!

Thanks @kantai, this is a great level of detail to provide for developers. Something like the last section will be especially helpful for articulating the "it depends" nature of this UX challenge, for different types of interfaces with different needs.
@jasperjansz perhaps we should add an indicator alongside any balance in the wallets or explorer that has proactively incorporated pending / microblock state, as Aaron suggests here?
The Figma linked above doesn't exhibit balance UX for the wallets or explorer in general, so perhaps we should add it (not only to clarify implementation for these products but also to serve as material for this documentation)?
Note as well that we've recently decided on the UserX team to distinguish all microblock-included transactions in the wallets in addition to the explorer (e.g. with a ⚡ symbol). This step also aligns with Aaron's guidance above.

@pgray-hiro @agraebe not sure if we've already used this, but Aaron and Ludo had put together this helpful state diagram for microblocks a while back:


sos: 404 download error

hi, I try TOR on Kali Linux and Parrot OS but said: error 404
I try on terminal command sudo … same error. any suggest.
many thanks and a hug for all best community

hotfix: #660
basically, follow this
build from source
however, you just have to change the first command here to
git clone https://github.com/tensornova/torbrowser-launcher.git
tell me if there is an error related to 'float' when you follow this.

yup
Traceback (most recent call last):
  File "/home/relected/Downloads/git/tor-fuckup/torbrowser-launcher/./torbrowser-launcher", line 30, in <module>
    torbrowser_launcher.main()
  File "/home/relected/Downloads/git/tor-fuckup/torbrowser-launcher/torbrowser_launcher/__init__.py", line 92, in main
    gui.move(
TypeError: arguments did not match any overloaded call:
  move(self, QPoint): argument 1 has unexpected type 'float'
  move(self, int, int): argument 1 has unexpected type 'float'

Issue #628
then
(1) if you've already installed it, please uninstall it or delete a directory torbrowser-launcher you downloaded before
(2) follow build from souurse
but try git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git
instead of the first command

hi, sorry for the long wait i was asleep
the spftware starts, but its stuck at 0% at downloading signatures

Hi, I have the same problem, do you know a way to make it work with flatpak? Do I just need to wait for the flatpak to be updated? Thanks you

That's exact. First we need to have it being merged via #661, and then, that a new build is pushed to flatpak.

Issue #628
then (1) if you've already installed it, please uninstall it or delete a directory torbrowser-launcher you downloaded before
(2) follow build from source but try git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git instead of the first command

Hey none of this worked for me at all. Take a look. I'm on Ubuntu 22.04. When will TOR be fixed?
computer@computer-System-Product-Name:~$ git clone https://github.com/tensornova/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone https://github.com/tensornova/torbrowser-launcher.git
[sudo] password for computer:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package clone
E: Unable to locate package https://github.com/tensornova
computer@computer-System-Product-Name:~$ git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git
E: Command line option 'b' [from -b] is not understood in combination with the other options.
computer@computer-System-Product-Name:~$ git clone https://github.com/micahflee/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone https://github.com/micahflee/torbrowser-launcher.git
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package clone
E: Unable to locate package https://github.com/micahflee
computer@computer-System-Product-Name:~$

I am also getting the "Download Error: 404"
When is this going to get fixed?

hi, sorry for the long wait i was asleep
~the software starts, but its stuck at 0% at downloading signatures~
Edit: nvm it works, thanks
EDIT 2: it deleted all of my data! :(

um, sorry, I didnt considering update that is not a new installation. It's a guess, but it seems that the data has been deleted because the program path has changed even though the data exists. This command remove does not usually delete user configuration files and the data. I think it will be okay if you downgrade it.

Issue #628
then (1) if you've already installed it, please uninstall it or delete a directory torbrowser-launcher you downloaded before
(2) follow build from source but try git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git instead of the first command

Hey none of this worked for me at all. Take a look. I'm on Ubuntu 22.04. When will TOR be fixed?
computer@computer-System-Product-Name:~$ git clone https://github.com/tensornova/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone https://github.com/tensornova/torbrowser-launcher.git
[sudo] password for computer:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package clone
E: Unable to locate package https://github.com/tensornova
computer@computer-System-Product-Name:~$ git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone -b hotfix --single-branch https://github.com/tensornova/torbrowser-launcher.git
E: Command line option 'b' [from -b] is not understood in combination with the other options.
computer@computer-System-Product-Name:~$ git clone https://github.com/micahflee/torbrowser-launcher.git
Command 'git' not found, but can be installed with:
sudo apt install git
computer@computer-System-Product-Name:~$ sudo apt install git clone https://github.com/micahflee/torbrowser-launcher.git
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
E: Unable to locate package clone
E: Unable to locate package https://github.com/micahflee
computer@computer-System-Product-Name:~$

not sudo apt install git clone but sudo apt install git

@tensornova

um, sorry, I didnt considering update that is not a new installation. It's a guess, but it seems that the data has been deleted because a path to the data has changed even though the data exists. I'll check it out and let you know if I can. and i recommend a downgrade if you are able to do it

thats reassuring, BTW what was the old location of the browser profile, maybe I can recover it manually?

I failed with torbrowser-launcher from Debian Stable too.
I think an update should be released through OS distributions.

it will slowly happen, just wait for it slowly
also Debian Stable will take a pretty long time to upate torbrowser-laucher so i think the bestthing to do rn is to install it maually

This is no solution, but info towards one.
Tor seems tor has moved to a universal language package in 12.0.1.
So the launcher should not use the language to composite the download path, instead it should use "ALL".
open the /usr/lib/python3/dist-packages/torbrowser_launcher/common.py file
Change the discover_arch_lang function to simply set self.language to "ALL", i.e. delete all the complicated language guessing code that is there and replace it with one line (I also show the comment of the next function, so it becomes clear how far it has to be replaced):
    # discover the architecture and language
    def discover_arch_lang(self):
        # figure out the architecture
        self.architecture = "x86_64" if "64" in platform.architecture()[0] else "i686"

        # figure out the language ... with 12.0.1 this has become a universal language package
        self.language = "ALL"

    # get value of environment variable, if it is not set return the default value

Now it downloads alright and tries to install ... where it hangs.
Running /home/username/.local/share/torbrowser/tbb/x86_64/tor-browser_ALL/start-tor-browser.desktop
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/torbrowser_launcher/launcher.py", line 442, in success
    self.run_task()
  File "/usr/lib/python3/dist-packages/torbrowser_launcher/launcher.py", line 301, in run_task
    self.run()
  File "/usr/lib/python3/dist-packages/torbrowser_launcher/launcher.py", line 477, in run
    if not self.check_min_version():
  File "/usr/lib/python3/dist-packages/torbrowser_launcher/launcher.py", line 465, in check_min_version
    for line in open(self.common.paths["tbb"]["changelog"], "rb").readlines():
FileNotFoundError: [Errno 2] No such file or directory: '/home/markus/.local/share/torbrowser/tbb/x86_64/tor-browser_ALL/Browser/TorBrowser/Docs/ChangeLog.txt'

Some sort of unfulfilled promise or whatever the contraption is called in Python.
So back to square one.
_Mark

Okay, I've done this step as a workaround for an installation:
sudo sed -i 's|self.language =.*|self.language = "ALL"|g' /usr/lib/python3/dist-packages/torbrowser_launcher/common.py

After this, on launching torbrowser-launcher it downloads successfully application's package, but it breaks with:
FileNotFoundError: [Errno 2] No such file or directory: '/home/user/.local/share/torbrowser/tbb/x86_64/tor-browser_ALL/Browser/TorBrowser/Docs/ChangeLog.txt'

Then, to complete workaround at user profile this must be run in a terminal:
mkdir -p ~/.local/share/torbrowser/tbb/x86_64
cd ~/.local/share/torbrowser/tbb/x86_64
ln - s tor-browser tor-browser_ALL

And it just works.
All this expecting torbrowser-launcher incorporates new fixes.

Watching this issue also

when using flatpak i get this error:

qt.qpa.xcb: could not connect to display
qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Available platform plugins are: eglfs, minimal, minimalegl, offscreen, vnc, wayland-egl, wayland, wayland-xcomposite-egl, wayland-xcomposite-glx, xcb.

also when i try tensornova's solution it requires my github username and password and then says that password authentication has been ended since august 2021.

You have to generate a token and use that as your password. Either way, when you do that you are (or I was) notified that the repository doesn't exist

i tried to use source and debian repository but i got error 404. also when i try tensornova's solution it requires my github username and password and then says that password authentication has been ended since august 2021.

My workaround also works with Debian packaging source (first part to common.py , next part to postinst & skel)

its fixed, @narcisgarcia you just have to wait until the software is updated in your repo
or you can just manually install it

just build from source code and install the package created https://github.com/micahflee/torbrowser-launcher/blob/main/BUILD.md

Getting the 404 error after installing from ubuntu official repo
Build from source worked for me

Getting the 404 error after installing from ubuntu official repo

What version of torbrowser-launcher did you get from this repo?


[RFC][VTA] Support Intel FPGA in VTA

Objective

To make VTA available for intel FPGA.
To demonstrate the quantized resnet18 on DE10-nano.

Interface Changes

tvm/vta/hardware/intel_fpga would be created.
Interface design of the instructions would be kept consistent with current implementation.
Avalon ST interface would be used in Intel FPGA for data streaming

Actionable Items

[x] Implement HLS based fetch, load, compute, store functions.

[x] load instruction
[x] gemm instruction
[x] alu instructions
[x] store instruction
[x] implement test functions for the IP cores

[ ] Adopt existing PYNQ board demonstration on DE10-nano

[ ] implement system integration scripts
[ ] implement a set of driver utilities (adopt tvm/vta/src/pynq)

[x] contiguous memory allocation and de-allocation
[x] functions to read/write to memory-mapped registers
[ ] function to program the bitstream on the FPGA

[ ] migrate python script for DE10-nano

We welcome further discussions on this topic.

This is a great initiative. Please let me know if you need any help on understadning VTA design elements. Down the road we may want to merge sources and have vendor specific interfaces and pragmas is we rely on HLS. This will ensure that our source remains consistent across different vendors. A DE10 nano reference design is the right start. We may purchase one on our end to validate your design.

@tmoreau89 I'm wondering why your current design is clocked at 100MHz, and it still achieves 7ns execution cycle, as it should N times of 10ns.

@liangfu good question. The 7ns target is passed to HLS to insert pipeline register in the synthesized module in order to close timing. It turns out that if you set the target period to 10ns, the design won’t close timing at 100MHz. Setting a more aggressive target in HLS will insert more registers and therefore allow us to close timing at 100MHz more easily. It’s a bit of a hack but it works. I’m working on changes to the design to ensure improved timing closure. For the intel back end, I would say that this parameter wouldn’t matter if the intel HLS compiler and P&R flow did a better job at closing timing at the specified target.

@tmoreau89 I'm quite curious about how fast VTA can run on top of PYNQ. Can you tell the exact timing in performing inference on the quantized resnet-18 with the designed instructuions? How much time in total are consumed to perform the gemm instruction?

@liangfu when we released VTA, our ResNet-15 inference time was around 0.4 seconds. Upon software and hardware optimizations we can bring inference closer to 0.2 seconds (I'll be releasing an update in the next 10 days or so). Note that this is for a 100MHz design with 256 FMAs, and that we are mostly just offloading GEMM on the FPGA. You can obtain the time breakdown for running each layer individually by running the unit tests on a Pynq FPGA. Do you have one handy?

@liangfu Actually, I too have a plan to port VTA to Intel FPGA, but because you opened this issue, I've been waiting and seeing.  Shall I help you with something?

@tmoreau89 Thank you for your answer. I have no PYNQ board at hand, but I do have many kinds of Intel FPGA boards, including de10-nano. Therefore, I have strong willingness to enable VTA on Intel FPGAs. On the either hand, aside from FMA instructions, I think VNNI would be more effective for nn inference.
@ktabata You're always welcome to contribute. I would submit an initial PR later on, so that we can discuss cooperate on this topic.

@ktabata @liangfu It sounds like there is a willingness to bring VTA support to Intel FPGAs. It makes sense to combine efforts if possible to facilitate this process as much as possible.
In terms of tasks needed to support a new FPGA platforms here are the high-level bits:

Porting the IP source from Xilinx HLS to Altera HLS/OpenCL with the Altera specific pragmas
Implementing test functions for those IP cores
Implementing a system integration .tcl script that wires those IP cores together to form a complete FPGA design (here we'd use Avalon-based system integration)
Implement a set of driver utilities that allow for

contiguous memory allocation and de-allocation, with functions to obtain physical address of the allocated buffer (assuming shared memory system)
functions to read/write to memory-mapped registers for accessing control registers of IP blocks
function to program the bitstream on the FPGA

This should be enough to run a TVM benchmark that targets an Intel FPGA. Essentially we could subdivide this into (1) Altera HLS port, (2) IP integration with Avalon, and (3) software driver stack.
Are there tasks that you consider to be already mostly completed on your end @liangfu ? I will be ordering a de10-nano to reproduce and validate your results.

It would be great if things can be broken down into a checklist of small items, so @liangfu and @ktabata @tmoreau89 can collaborate on some of the perspectives(either code review or commit changes). Feel free to coordinate on this issue

@tmoreau89 @tqchen Good point to split the task into each workable subtasks.
@tmoreau89 I've been mostly working on migrating the IP cores from Xilinx HLS to Intel HLS (I think OpenCL based implementation would be less efficient and incompatible with VTA.) And I think I can work on the testing and simulation of those functions as well. However, as I got my hands on the driver part. I found mmap based device communication unfamiliar to me at the moment. Can anyone take the driver migration and system integration task?

@ktabata would you be interested in bringing driver support for Intel FPGAs? I am not very familiar with the software stack on Intel FPGAs, but I'd be happy to provide some guidance.

I have some experience in AOCL, but I'm new to Intel HLS Compiler.
Currently I'm finding out which is suitable for VTA (AOCL or HLS).
I'm curios to know whether or not host(CPU) SDRAM access from HLS code is possible.
@liangfu Do you have information about that?

@ktabata - we could attempt to port VTA to AOCL, but we may not be able to implement latency hiding under this programming model  - having separate IP blocks that can operate independently from one another via dependence queues was difficult to achieve with SDAccel - the programming model did not allow us to express "continually executing" modules, which is why we went for HLS + IPI (tcl scripting) flow.
The takeaway here is that we could use AOCL to get a design up and running for Intel FPGAs, but it's likely that if we want to have high efficiency design with latency hiding, we may have to stick to HLS for now.

@ktabata Actually Intel HLS Compiler is new to me as well. It is the high efficiency design it could generate that draws my interest. The techniques include possibilities to implement dependency queues and pipelined parallel execution.
AFAIK, if we need to access SDRAM from FPGA side, we only need to add SDRAM Controller in Qsys. Please refer to GHRD for DE10-nano.

@liangfu - by SDRAM are you referring to accessing DDR? Indeed I am less familiar with how to connect the DMA master to a memory controller in Qsys. This depends on the SoC organization - on Zynq we connect our DMA Axi master to the ACP port of the ARM SoC which enables coherent memory access to the SoC's main memory.
It would be good to maintain the same coherent interface if possible with the Intel SoC as we do for the Xilinx SoC (this will make it easier to keep the TVM runtime consistent between the manufacturers).

@tmoreau89 I completely agree to have coherent design between Intel SoC and Xilinx SoC, and I would utilize AXI master bridge to support VTA. I was just answering to @ktabata 's question on whether it is possible to have Intel HLS access SDRAM.

@tmoreau89 As I'm implementing the driver for the modules, I'm not quite clear about whether the fetch module loads instruction streams from DDR via cache coherent ACP port, or it could load instructions through the lightweight AXI port. In addition, if it loads instructions via the ACP port, does it require any additional on-chip RAM store the instruction queue?

@liangfu the fetch module grabs the instructions from DDR via DMA over the ACP port. It pushes the instruction it streams in into FIFO queues, that all take 1 block ram.
If you have vivado installed on your machine, you can build the design, and visualize the board diagram by opening the .xpr project file in the Vivado GUI, and then open the vta_wrapper.bd board design. This will show you how all of the modules are integrated within the Zynq SoC chip.
Hope this helps!

@tmoreau89 Thank you for your help.
Now, I'm aware that the fetch module uses DMA to grab the instruction streams in DDR, and I have implemented that on de10-nano. However, it is defined in the Xilinx HLS code that the interface is m_axi, which should be a axi master interface. As Intel HLS compiles pointers in the argument list into memory mapped interface, do you think I shall transform that into ihc::stream_in and feed the stream with mSGDMA ?

@tmoreau89 As we have making progress to enable VTA for Intel FPGA, I found it is difficult to debug the generated hardware of the compute component, and there are several features not yet supported in the Intel HLS. Therefore, I'm wondering whether there is any plan to decompose the compute component into several smaller components. At least, it could be reasonably constructed with decoders, ALUs and GEMM modules.

@liangfu what are the features that aren't supported in Intel HLS that are making the process challenging? We could consider breaking modules into smaller components to make testing and debugging more easy. What did you have in mind?

closed in favor of the most recent chisel RFC


Allow helpers to update nomination reasons

Sometimes, you want to add information to a nomination, possibly after getting to know the nominee better. This is currently not possible for helpers.
When a solution involving allowing staff to nominate the same member multiple times was proposed in #nominations, @wookie184 pointed out that the !tp edit command can be used to edit nomination reasons, but that it is currently restricted to moderators and admins. I propose that helpers should be allowed to use the command to update their own nominations as well.
It might also be worth considering allowing helpers to delete their own nominations (if this is not already possible), if they change their minds on a candidate.

Definitely agree, helpers are already allowed to add people to the talent pool so why not edit. If we really wanted to we could only allow helpers to edit their own nominees. But that's just an idea.


add e2e test for cacert rotation

Please provide a description of this PR:
follow up: https://github.com/istio/istio/pull/47424

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test lint

gencheck_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test lint

gencheck_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test gencheck

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test lint

gencheck_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test gencheck

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test lint

gencheck_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test gencheck

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test gencheck

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

lint_istio
9befc87d026a9aa4156e6ca986dfd6d151957a68
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

lint_istio
9befc87d026a9aa4156e6ca986dfd6d151957a68
link
true
/test lint

gencheck_istio
9befc87d026a9aa4156e6ca986dfd6d151957a68
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-istiodremote

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

unit-tests_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

unit-tests_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test unit-tests

integ-security-istiodremote_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
d392d0676b9689a9381f33602d4404e113c16f42
link
true
/test integ-security-multicluster

unit-tests_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test unit-tests

integ-security-istiodremote_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-istiodremote

integ-security_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test unit-tests

integ-security-istiodremote_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-istiodremote

integ-security_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security

integ-security-multicluster_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security

integ-security-multicluster_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-multicluster

integ-security-istiodremote_istio
8ca77a96b5cfe92f91dc2be4b24918006f8821f8
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
fc1efefb446b068b68555aa6909f40815da543b3
link
true
/test integ-security-multicluster

integ-security-istiodremote_istio
8ca77a96b5cfe92f91dc2be4b24918006f8821f8
link
true
/test integ-security-istiodremote

integ-security_istio
8ca77a96b5cfe92f91dc2be4b24918006f8821f8
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

can we test the case, one with old cert and the other new cert

It's hard to find such a precise moment now.
Alternative: Keep sending traffic, all traffic result should be 200 all the time(before and after cert rotation).

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security-istiodremote

integ-security_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-istiodremote_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security-istiodremote

integ-security_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security

integ-security-multicluster_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
efd72034a95ee58b5418eb389d6bd438f7a9f4ad
link
true
/test integ-security-multicluster

integ-security-istiodremote_istio
599a31d4a4b5518f2c030342765c77d7bdc6b2f1
link
true
/test integ-security-istiodremote

integ-security_istio
599a31d4a4b5518f2c030342765c77d7bdc6b2f1
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@zirain If #47424 was backported to v1.20 then the e2es should also be backported.

/cherry-pick release-1.20

@hzxuzhonghu: new pull request created: #47952

In response to this:

/cherry-pick release-1.20

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Add bcos version check logic

Codecov Report

Merging #81 into release-rc4 will increase coverage by 0.21%.
The diff coverage is 78.94%.

@@                Coverage Diff                @@
##             release-rc4      #81      +/-   ##
=================================================
+ Coverage          36.27%   36.49%   +0.21%
- Complexity           358      360       +2
=================================================
  Files                 49       49
  Lines               3691     3710      +19
  Branches             325      327       +2
=================================================
+ Hits                1339     1354      +15
- Misses              2260     2262       +2
- Partials              92       94       +2

Impacted Files
Coverage Δ
Complexity Δ

...ebank/wecross/stub/bcos/BCOSConnectionFactory.java
69.11% <78.94%> (+3.81%)
7.00 <2.00> (+2.00)

Continue to review full report at Codecov.

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


LiveKit implementation exceptions

The whole LiveKit ecosystem has some unhandled Exceptions that might break the flow during sudden disconnections or the disposal of the application, we should carefully handle those exceptions gracefully.
Some of those throws were ignored here https://github.com/decentraland/unity-explorer/pull/1127 as an effort to increase stability but we need to review the whole package.

KeyNotFoundException: The given key 'SIGNER' was not present in the dictionary.


Unable to install on Mac

The installation fails with this message
Mac OS 10.10.3 (14D136)

The same issue exists with docker-machine.

@mannickutd Are you connected through VPN?
Mind opening up virtualbox and removing the dev VM in it. Please restart Kitematic again after.
Thanks

Hi Michael,
I am not connected through a VPN but I do have VPN software installed on
the machine.
I removed the dev VM and restarted but got the same problem.
I have no problems with docker, and boot2docker.
Thanks
Nick
On Tue, Jun 16, 2015 at 6:46 PM, Michael Chiang<EMAIL_ADDRESS>wrote:

@mannickutd https://github.com/mannickutd Are you connected through
VPN?
Mind opening up virtualbox and removing the dev VM in it. Please restart
Kitematic again after.
Thanks
—
Reply to this email directly or view it on GitHub
https://github.com/kitematic/kitematic/issues/643#issuecomment-112510711
.

@mannickutd Thanks Nick. This is likely a conflict between boot2docker and docker-machine.
cc/ @nathanleclaire to see if he wants any specific logs to help us continue to troubleshoot this.

This is with Machine rc3 right?

@nathanleclaire Judging by this created 2 days ago. Yes, it would be with Machine RC3.
@mannickutd Can you confirm which version of Kitematic you are running?

The different versions of the library are below
Last login: Thu Jun 18 08:21:37 on ttys003
Nicholass-MacBook-Air:~ nicholasstaples$ docker-machine -v
docker-machine version 0.2.0 (8b9eaf2)
Nicholass-MacBook-Air:~ nicholasstaples$ docker -v
Docker version 1.5.0, build a8a31ef
Nicholass-MacBook-Air:~ nicholasstaples$ boot2docker -v
Boot2Docker-cli version: v1.5.0
Git commit: ccd9032
On Thu, Jun 18, 2015 at 12:56 AM, Nathan LeClaire<EMAIL_ADDRESS>wrote:

This is with Machine rc3 right?
—
Reply to this email directly or view it on GitHub
https://github.com/kitematic/kitematic/issues/643#issuecomment-112982445
.

merging with #1029


hide crop, download and play/pause buttons while sound loads to close #89

Closes #89 by hiding crop, download and play/pause buttons while a sound is loading.
This was a simple change: I just grouped together all the elements that we don't want to appear until the sound loads into a single <section> element and had that <section> element only appear if sound.previews was not falsy.
This branch also adds /* eslint-disable */ to /freesound-player/cypress/plugins/index.js to quell error messages that appeared in the terminal for me after initially trying to make the sole commit in this branch and another commit in a branch I created earlier.

🥳 Good stuff Hamir!

Thanks, Amila!


libevdev: 1.9.1 -> 1.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libevdev/versions.
meta.description for libevdev is: "Wrapper library for evdev devices"
meta.homepage for libevdev is: "http://www.freedesktop.org/software/libevdev/doc/latest/index.html"
meta.changelog for libevdev is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.10.0 with grep in /nix/store/f74cbfk32j89q5zgvdn3rl5lgyxmmk5d-libevdev-1.10.0

directory tree listing: https://gist.github.com/090a90d31c0f5c478e316f494515eb2d

du listing: https://gist.github.com/9b3cbfcd613430d33d68028426bd258c

Rebuild report (if merged into master) (click to expand)

6961 total rebuild path(s)

2423 package rebuild(s)

2423 x86_64-linux rebuild(s)
2270 i686-linux rebuild(s)
16 x86_64-darwin rebuild(s)
2252 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
AusweisApp2
adapta-gtk-theme
adwaita-qt
aj-snapshot
albert
amarok
amarok-kf5
ams
android-file-transfer
anki
ankisyncd
antimicroX
antimony
ao
apitrace
apmplanner2
appcsxcad
aqemu
arx-libertatis
asymptote
audacious
audaciousQt5
audiality2
avidemux
awesomebump
bambootracker
barrier
bcompare
beamerpresenter
beets
bibletime
bino3d
birdtray
bitcoin
bitcoin-abc
bitcoin-classic
bitcoin-gold
bitcoin-unlimited
bomi
boomerang
brewtarget
cadence
cage
cagebreak
calamares
calaos_installer
calibre
calibre-py2
calibre-py3
calligra

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/f74cbfk32j89q5zgvdn3rl5lgyxmmk5d-libevdev-1.10.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A libevdev https://github.com/r-ryantm/nixpkgs/archive/f48578f636f24aa5dd721b379d26c40000afd526.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/f74cbfk32j89q5zgvdn3rl5lgyxmmk5d-libevdev-1.10.0
ls -la /nix/store/f74cbfk32j89q5zgvdn3rl5lgyxmmk5d-libevdev-1.10.0/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings
cc @AndrewMorsillo for testing.

Result of nixpkgs-review pr 102550 run on x86_64-darwin 1

16 packages marked as broken and skipped:

clutter
gnome3.clutter
haskellPackages.evdev
haskellPackages.evdev-streamly
libinput
libmanette
libretro.dolphin
python37Packages.libevdev
python38Packages.libevdev
remarkable-mouse
sway-contrib.grimshot
waybar
wshowkeys
xorg.xf86inputevdev
xorg.xf86inputlibinput
xorg.xf86inputsynaptics


Update recipe to 1.16.0

Supersedes #14

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b0of28zkm7/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps and --no-build-isolation.
ERROR: /tmp/abs_b0of28zkm7/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_b0of28zkm7/clone/recipe/meta.yaml:32: python_build_tool_in_run: The python build tool packaging is in run depends
Errors were found


error running the pysunspec_test.py file

Receive following error on running the test file:
sudo python pysunspec_test.py
pySunSpec version: 1.0.8
Test device path: /usr/local/lib/python2.7/dist-packages/sunspec/core/test/devices
Traceback (most recent call last):
File "pysunspec_test.py", line 13, in
test.test_all()
File "/usr/local/lib/python2.7/dist-packages/sunspec/core/test/test_all.py", line 58, in test_all
(count_run, count_passed, count_failed) = module.test_all(pathlist, stop_on_failure)
File "/usr/local/lib/python2.7/dist-packages/sunspec/core/test/test_device.py", line 577, in test_all
if test(pathlist) is True:
File "/usr/local/lib/python2.7/dist-packages/sunspec/core/test/test_device.py", line 418, in test_device_from_pics
d1.from_pics(filename='pics_test_device_1.xml', pathlist=pathlist)
File "/usr/local/lib/python2.7/dist-packages/sunspec/core/device.py", line 127, in from_pics
raise SunSpecError('Error loading PICS: %s' % str(e))
sunspec.core.util.SunSpecError: Error loading PICS: Block index out of range: 1
What could be the reason ?

The models directory is a sub-repository and so an additional command would be needed to acquire that when downloading the code. Please see the pySunSpec subsection of the Installation instructions on the README.


Support Zulip hosted Instance.

Zulip has a hosted instance, it would be great if you support it. Since slack already has that, it should be possible with free tier i hope.
zulipchat.com is the URL for hosted instances
eg: hackinthenorth.zulipchat.com and anna.zulipchat.com

Is there anyone who can provide an update on what is needed to support this?

Cc: @hackerkid

Also https://github.com/adambirds/recipe-zulip has a few bugs that I'd be happy to help fix; I'm not sure what the right way to get that integrated, though, since @adambirds seems inactive.


vs2015

If https://github.com/loqu8/tesseract-vs2015 is placed parallel to the tesseract repo, you can use the vs2015 sln to compile tesseract in Visual Studio with giflib, libpng, libjpeg, libtiff, libwebp and zlib. Note that the solution will look for includes and lib in ../../../tesseract-vs2015/release rather than ../../../ like the VS2010 sln.

-1. It's better to improve cmake build system rather than each solution per VS version.

@zdenop said in #101:

Future is cmake. There is no sense to make and maintain build system for each version of cmake. cmake correctly detect installed compiler and use it. We are just waiting for leptonica transfer to github, there cmake will be presented to.

So, this PR should be closed.

Use cmake for any Visual studio builds.


added headers: compare, concepts, cstdarg, climits, cfloat.
Where does this code come from?

Where does this code come from?

I went through all the classes, structs and concepts on cppreference, if there was a "possible implementation" provided, I copied it, if not, I made my own.
That comment was from my kernel, where I copied  from.
Then I thought that including gnu code in cxxshim was a bad idea so I removed all of it (but missed the comment) and wrote my own.

Code snippets from cppreference.com are problematic if they are non-trivial. They are CC-BY-SA, but CC-BY-SA and MIT are not compatible.

In this case, however, it might be that the snippets are small enough to not be copyrightable.

@ArsenArsen splitted it into multiple commits

will this be merged?


Attribute values being wrapped in parentheses while typing

Recently, Volar started to wrap some parts of attribute values in parentheses and I don’t recall a situation in which this was appropriate.
In the following examples, I expect no transformations to happen, especially not while I’m typing.
Example 1
Place the cursor after the i in (value, i), type x.
<template>

    Test

</template>

Result:
<template>

    Test

</template>

Example 2
Place the cursor after the > in "3 >", type 1.
<template>

">
    Test

</template>

Result:
<template>

1)">
    Test

</template>

The wrapping also happens to me, it’s kinda annoying..

@johnsoncodehk it seems like the original issue was a vscode-specific syntax issue so maybe those workarounds should specifically target vscode.

@rchl Yes, I will make it default disabled to other IDEs. Do you plan to implement Auto Insertion for https://github.com/sublimelsp/LSP-volar?

Probably not anytime soon since ST doesn't make it easy. Just have been thinking about other editors that would likely not need those workarounds.


[Bug] Response Format of json is not working in openai

While using openai.WithResponseFormat with the response format set to JSON,  i could see the req.ResponseFormat as nil coz of improper handling in code
Upon inspecting the code, I noticed that there is a check for an unused key in the JSON format at the following line:
Upon inspecting the code, I noticed that there is a check for an unused key in the JSON format at the following line:
https://github.com/tmc/langchaingo/blob/7eb662b22a5919b8e9daaa4c20100f26b9096b69/llms/openai/openaillm.go#L114,
ctx := context.Background()
responseFormat := &openai.ResponseFormat{Type: "json_object"}
client, err := openai.New(openai.WithModel("gpt-3.5-turbo"), openai.WithResponseFormat(responseFormat))

	content := []llms.MessageContent{
		llms.TextParts(llms.ChatMessageTypeSystem, "You are a company branding design wizard."),
		llms.TextParts(llms.ChatMessageTypeHuman, "What would be a good company name a company that makes colorful socks?"),
	}

	resp, err := client.GenerateContent(ctx, content)

	fmt.Println(resp.Choices)

	for _, message := range resp.Choices {
		fmt.Println(message.Content)
	}

	return err

Whereas its working as expected in llm package
client, err := openai.New(openai.WithModel("gpt-3.5-turbo"), openai.WithResponseFormat(responseFormat))

	completion, err := llms.GenerateFromSinglePrompt(ctx,
		client,
		"Who was first man to walk on the moon? Respond in json format, include `first_man` in response keys.",
		llms.WithTemperature(0.0),
		llms.WithJSONMode(),
	)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Println(completion)

ctx := context.Background()

client, err := openai.New(openai.WithModel("gpt-3.5-turbo")))

	content := []llms.MessageContent{
		llms.TextParts(llms.ChatMessageTypeSystem, "You are a company branding design wizard."),
		llms.TextParts(llms.ChatMessageTypeHuman, "What would be a good company name a company that makes colorful socks?"),
	}

//add type in request
	opts := llms.CallOptions{
		JSONMode: true,
	}

	resp, err := client.GenerateContent(ctx, content,opts)

	fmt.Println(resp.Choices)

	for _, message := range resp.Choices {
		fmt.Println(message.Content)
	}

	return err
	``

@devalexandre Yes, I was able to achieve it with the following code:
response, err := client.GenerateContent(ctx, content, llms.WithJSONMode())
However, I noticed that there are unused keys for every OpenAI LLM, specifically "openai.WithResponseFormat(responseFormat)", which isn't being utilized. I wanted to report this issue here.

@devalexandre Yes, I was able to achieve it with the following code:
response, err := client.GenerateContent(ctx, content, llms.WithJSONMode())
However, I noticed that there are unused keys for every OpenAI LLM, specifically "openai.WithResponseFormat(responseFormat)", which isn't being utilized. I wanted to report this issue here.

this is not BUG as we have option base but for each LLM, use some options, never use all.
maybe we could return a error if some option aren't used.


Changes in "Add New Project"

[ ] Instead of typing in the "Project Level", Adding in a dropdown to select the Level Eg:"Intermediate"," Beginner" etc....
[x] Addition of additional non mandatory fields to add links for reference eg: Github Repo , Working Website Demo for appropriate projects

Please Assign @rockingrohit9639
This Issue is a part of my contribution towards LGM-SOC'21

I'ld like to work on this issue

@Hardikpoudel Actually I am working on this issue.

ok i'll choose next one then

@Hardikpoudel bro, I will assign you this issue when you complete the previous one. And yeah, don't add code of both issues in one PR.

@Rutuj-Runwal  bro, I will assign you this issue when you complete the previous one. And yeah, don't add the code of both issues in one PR.

@rockingrohit9639 Can I work on this now?

@rockingrohit9639 Can I work on this now?

Yes @Rutuj-Runwal bro, you can start working on this issue. Dropdown for project level is already added but you can add your second suggestions. You are given 6 days to complete it.

Hey @Rutuj-Runwal bro, are you working on it?

Yes I am working.....I was unable to push the changes as I was focused on other projects will do it ASAP

Hey @Rutuj-Runwal bro, are you working on it?

Yes I am working.....I was unable to push the changes as I was focused on other projects will do it ASAP

Hey @Rutuj-Runwal bro, are you working on it?

Okay, no problem

Hey, @Rutuj-Runwal I have fixed that issue, now you can word on it. For now, you can just change the frontend we can change the backend later.

@Rutuj-Runwal any updates?


Issue telemetry bugs

This PR fixes:

issues in gNMI-1.10 and gNMI-1.11 wherein the DUT Out was being compared with ATE Out, whereas the wiring is ATE Out <-> DUT IN. There was a symmetrical problem with DUT In and ATE In.
An issue where the configured PPS was lower than the UHD could support.

Issue #1 could have been missed in a virtual environment where ATE OUT == ATE In, but in a real hardware environment this is not the case.
Issue #2 was unique to the UHD, but other OTG implementations should have no trouble generating the 40kps instead of 6kbps PPS for the original test.

Pull Request Functional Test Report for #2886 / 639fc50568a133b49b539198f4cb2a037007981c
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Cisco 8000E

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Cisco XRd

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Juniper ncPTX

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Nokia SR Linux

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Openconfig Lemming

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Cisco 8808

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Juniper PTX10008

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Nokia 7250 IXR-10e

gNMI-1.10: Telemetry: Basic CheckgNMI-1.11: Telemetry: Interface Packet Counters

Help


testnet network support - cwz0upawtyjluru - validator,rpc,api

My Moniker : Army IDs
My Validator Address : akash19j9s2wmc2xqg37afusv2vdf62ne626u7c6mr6x

@armyids , Your Moniker and validator addresses are required, please update the PR with those details

@armyids , Your Moniker and validator addresses are required, please update the PR with those details

Hi @akhilkumarpilli,
My Moniker : Army IDs
My Validator Address : akash19j9s2wmc2xqg37afusv2vdf62ne626u7c6mr6x

Hi @akhilkumarpilli,
My Moniker : Army IDs
My Validator Address : akash19j9s2wmc2xqg37afusv2vdf62ne626u7c6mr6x


code.google.com/p/goconf no longer works. Use a github fork instead.

As of January 25th, code.google.com no longer serves git.

http://google-opensource.blogspot.com/2015/03/farewell-to-google-code.html

Created a mirror ifwe/goconf on github with no plans to modify it.
Feel free to export https://code.google.com/archive/p/goconf/ to the uniqush
project later.

:+1: Thank you!


Improve error context for user defined federated authenticators

Task issue:

https://github.com/wso2/product-is/issues/21216

Currently, a server exception is always thrown when attempting to perform endpoint management using the action component.

PRs:

https://github.com/wso2/carbon-identity-framework/pull/6213


DeprecationWarning: Creating a LegacyVersion has been deprecated and will be removed in the next major release

I can reproduce the failure locally with nox.

This seems to be related to https://github.com/pypa/setuptools/issues/2885.
For now I'm going to pin setuptools<59.1.0 to resolve our CI failure.

As of this writing, setuptools version 51.1.0 has been yanked from pypi. As such, the pin implemented by this PR is not necessary.


recompose dependency has been removed but the documentation still uses it

I have been using recompose/compose exactly like in the doc:
https://material-ui.com/components/hidden/
Only the import was from 'recompose/compose' not 'docs/src/modules/utils/compose' (I have tried this too)
But don't work anymore maybe because of Remove recompose dependency (#14479)
What is the alternative of that?

@Bouncue Nice finding. We can migrate the demo to use the style hook API, it will remove the need to import a private module of the documentation.
https://github.com/mui-org/material-ui/blob/b3cace62fac1e0edd9b81b169fa731ebd738bf41/docs/src/pages/components/hidden/GridIntegration.js#L9
Do you want to work on it? :)

I am afraid I can't, I don't know about hook API. what I know is just copy and paste the code from doc

@oliviertassinari I looked into this yesterday but didn't finish as there is no hook version of withWidth yet. Should we just continue using the HOC version of withWidth for now?

@merceyz We are using compose to handle two HOCs, we don't need this helper function with a single HOC. We can remove it by migrating withStyles to makeStyles.

Yeah, just asking if I should convert it to hooks now with withWidth as a HOC, or wait for a hook version of withWidth

@merceyz Removing compose would provide value on its own. I don't think that we should wait for #15678.

@oliviertassinari Alright, done


Works in PIE Standalone but not packaged project

Hello,
I left some messages on your YouTube video and also going to try here.  This works great! in PIE Standalone but not packaged project.  Not for sure what is wrong, Have tried a lot of stuff but it will not work in a packaged windows game.   Tried in a fresh project but same results, this is 4.22.3 so maybe not compatible, can you think of anything to try?
Thank you,
John

Same as you !

I've not tried this in a packaged build yet but looking on the forums here (https://docs.unrealengine.com/4.26/en-US/ProductionPipelines/Plugins/) i see this:
At present, Plugin configuration files are not packaged with projects. This may be supported in the future, but currently requires manually copying the files to the project's Config folder.
So perhaps try copying the config file fromthe plugin to the config folder in your project


Reference cluster security group from node security group

Description
I would like to utilize the actions-runner-controller module which requires the ability for the control plane to access a service within the cluster on port 443.  By default this is not allowed.
Versions

Terraform: 1.0.1
Provider(s):
Terraform v1.0.1
on darwin_amd64

provider registry.terraform.io/gavinbunney/kubectl v1.13.1
provider registry.terraform.io/hashicorp/aws v3.73.0
provider registry.terraform.io/hashicorp/cloudinit v2.2.0
provider registry.terraform.io/hashicorp/helm v2.4.1
provider registry.terraform.io/hashicorp/kubernetes v2.7.1
provider registry.terraform.io/hashicorp/null v3.1.0
provider registry.terraform.io/hashicorp/tls v3.1.0

Module:

Reproduction

Attempt to reference the cluster_primary_security_group_id from within node_security_group_additional_rules.

module "eks" {
  # https://github.com/terraform-aws-modules/terraform-aws-eks
  source  = "terraform-aws-modules/eks/aws"
  version = ">= 18.2.3, < 19.0.0"

  node_security_group_additional_rules = {
   ...
    https_ingress = {
      description              = "Allow HTTPS"
      protocol                 = "-1"
      from_port                = 443
      to_port                  = 443
      type                     = "ingress"
      source_security_group_id = [module.eks.cluster_primary_security_group_id]
    }
   ...

Code Snippet to Reproduce
Expected behavior
I would like to reference the cluster_primary_security_group_id from within the node_security_group_additional_rules.
Actual behavior
❯ terraform apply
╷
│ Error: Cycle: module.eks.var.node_security_group_additional_rules (expand), module.eks.aws_security_group_rule.node, module.eks.aws_eks_cluster.this, module.eks.output.cluster_primary_security_group_id (expand)

Terminal Output Screenshot(s)

Additional context

Replace source_security_group_id = ... with source_cluster_security_group = true

I ran into the same issue when trying to install the AWS Load Balancer Controller and needed to enable communication to the nodes on port 9443.
I adjusted the variable description in: https://github.com/terraform-aws-modules/terraform-aws-eks/compare/master...Phylu:patch-1


[ENUM Request] Table 1022 - Job Planning Line Invoice - Document Type

Hello,
if its possible i would like the field "Document Type" in the 1022 table to be a Enumeration.
Kind regards.

I would also like to be able to extend it by one extension

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


add CSM.relabel function

Changes
add CSM.relabel function to rename nodes in CSM
This should be safe right.. @vhirtham ? ;)
More of a utility so no test for now
Related Issues
Closes # (add issue numbers)
Checks

[x] updated CHANGELOG.md
[x] updated tests
[x] updated doc/
[x] update example/tutorial notebooks

This should be safe right.. @vhirtham ? ;)

Have to review the corresponding code sections, but I am pretty sure that it isn't safe in its current form when combined with subsystems.  If you rename the common node of the parent and child system, you need to update the subsystem data accordingly. Additionally, you might need to store the original name. That might get a little bit ugly. That's why we skipped this functionality back then.

This should be safe right.. @vhirtham ? ;)

Have to review the corresponding code sections, but I am pretty sure that it isn't safe in its current form when combined with subsystems. If you rename the common node of the parent and child system, you need to update the subsystem data accordingly. Additionally, you might need to store the original name. That might get a little bit ugly. That's why we skipped this functionality back then.

Ah yes, I see this could be dangerous on already merged subsystems ... maybe we can catch this and disable in this case?
However on a "clean" CSM before merging this would be safe? I need to rename a node to make merging possible

More of a utility so no test for now

You are living dangerous...

😈

Ah yes, I see this could be dangerous on already merged subsystems ... maybe we can catch this and disable in this case?
However on a "clean" CSM before merging this would be safe? I need to rename a node to make merging possible

Yeah, we can disable it for now as a quick fix
if self._subsystems:
    raise("CFabry did this :p")

Test don't really seem to work from forks 😞


feat: add dark mode to video gallery and prefooter

Overview
See storybook for examples. The mode prop only affects text and cta colors, as the background colors will be handled by the dynamic renderer. If that is potentially problematic I'm open to discussion!

PR Checklist

[x] Related to designs: landing page components, career page v2
[x] Related to JIRA ticket: REACH-569
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-labs@8.1.2-alpha.26e04b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://60188e63e1109918070361e6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@8.1.2-alpha.7bf28b.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6019e16366dde217256cf8e8--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@8.3.1-alpha.92381c.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6019f1ba66dde231856cf740--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@8.3.1-alpha.838989.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://601b332ef46e1438f261cc55--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@8.3.1-alpha.3966b4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://601c7b35e05aae1293c296ee--gamut-preview.netlify.appDeploy Logs

gonna go ahead and let this merge to unblock some other PRs, if https://github.com/Codecademy/client-modules/pull/1362 gets merged I'll circle back and consume it here in another PR

📬Published Alpha Packages:@codecademy/gamut-labs@9.0.2-alpha.2a1b41.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://601c7dbbafeea71071a4c826--gamut-preview.netlify.appDeploy Logs


Allow requesting addition/removal of IAM Permission boundaries to IAM roles

Permission boundaries are similar to IAM managed policies, except they don't grant additional permissions to a role. Rather, they specify the maximum permissions a role can have.
A role may have a maximum of one permission boundary. Administrators should be able to add/remove a permission boundary via the role's policy editor page. End-users should be able to request the addition or removal of a permssion boundary.
It might be worth referencing the Managed Policy UI  component, as the permission boundary component might look similar.

This feature is complete.


To add ability to set timeouts for test suite execution

To make control of tests execution more flexible it will be helpful to have additional timeout on test suite execution which will break suite execution once the timeout is reached.

The property itself already exists in config but does not take effect for now.

Details
The default timeout is 40 minutes (could be set explicitly or using config file)
The process
Once timeout is reached next actions should be processed:
the suite is immediately stopped
all already executed tests remain in statuses they got after execution
currently executed test is failed due to SuiteTimeoutException
all remain not yet executed tests are not executed and 'Skipped' status is set

done in 00ed2ec


<EMAIL_ADDRESS>

<EMAIL_ADDRESS>not work with<EMAIL_ADDRESS>Found<EMAIL_ADDRESS>npm ERR! node_modules/@capacitor-community/barcode-scanner
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Conflicting peer dependency<EMAIL_ADDRESS>npm ERR! node_modules/@capacitor/core
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@capacitor-community/barcode-scanner
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

See #240


dotnet run System.ArgumentNullException: for a simple program with new installation of .netcore 2.1.401

From @Petermarcu on August 31, 2018 18:56
@noorrocks commented on Wed Aug 29 2018
Issue Title
dotnet run System.ArgumentNullException: for a simple program with new installation of .netcore 2.1.401
General
Trying to get started with .netcore programming. Followed the basic tut from asp.net website but failed.
https://www.asp.net/get-started
I get the error (attached) after dotnet run command.
Powershell output
This is my first ever program after a fresh installation of the latest SDK (as of now). I have tried console app and it works fine. Even a simple app with 'web' template works ok and display 'Hello World' in the browser. However, as soon as I add 'services.addMVC' to the app it breaks with the same error. I have done all my research and tried a lot of things, but not sure what is happening.
Program files:
Files

@noorrocks commented on Thu Aug 30 2018
After wrestling with this problem for two days, finally, I have caught the issue.
if you see the 'powershell output', my project directory is ' Directory: C:#NIN\dotnetcore'
Somehow, it doesn't like the '#' in a folder name. I created a new folder without '#' and it worked like a charm.
I was also able to reproduce this problem on a different PC.
Please note that it only bothers the '#' if it's at the root folder name. Down the chain, it works OK.
This issue can be marked as a known issue, I think.
Copied from original issue: dotnet/cli#9932

From @livarcocc on August 31, 2018 20:56
@peterhuene can you take a look?

The exception is coming from ASP.NET MVC:
Parameter name: path1
   at System.IO.Path.Combine(String path1, String path2)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.RelatedAssemblyAttribute.GetRelatedAssemblies(Assembly assembly, Boolean throwOnError, Func`2 fileExists, Func`2 loadFile)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.RelatedAssemblyAttribute.GetRelatedAssemblies(Assembly assembly, Boolean throwOnError)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationAssembliesProvider.GetRelatedAssemblies(Assembly assembly)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationAssembliesProvider.GetAssemblyItem(Assembly assembly)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationAssembliesProvider.ResolveFromDependencyContext(DependencyContext dependencyContext)
   at Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationAssembliesProvider.ResolveAssemblies(Assembly entryAssembly)+MoveNext()
   at Microsoft.AspNetCore.Mvc.ApplicationParts.ApplicationPartManager.PopulateDefaultParts(String entryAssemblyName)
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.GetApplicationPartManager(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcCoreServiceCollectionExtensions.AddMvcCore(IServiceCollection services)
   at Microsoft.Extensions.DependencyInjection.MvcServiceCollectionExtensions.AddMvc(IServiceCollection services)
   at try1.Startup.ConfigureServices(IServiceCollection services) in C:\#NIN\dotnetcore\tryx\try1\Startup.cs:line 35
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Hosting.ConventionBasedStartup.ConfigureServices(IServiceCollection services)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.EnsureApplicationServices()
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Initialize()
   at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
   at try1.Program.Main(String[] args) in C:\#NIN\dotnetcore\tryx\try1\Program.cs:line 17

Having  C:\#NIN\ as the root of this path appears to be tripping up the MVC RelatedAssemblyAttribute.
I suspect this line might be treating # as an anchor, resulting in a root directory returned, which then gets converted to null by Path.GetDirectoryName and passed to Path.Combine, resulting in the exception.  Just a guess, though.
I'll move this to ASP.NET MVC.

@pranavkm, can you please look into this?

@noorrocks, this seems to indeed be caused by the # character in the path. Simply moving bits to a different folder would solve the issue. Feel free to send us a PR with the fix.

Hey, I am a first-timer looking for open source tasks to start off. I found this issue pretty interesting. Could I claim it? Is there anyone I can reach out to when I need help to understand the code and the issue?

Could I claim it?

Sure, feel free to send us a PR. Our wiki has some instructions for contributing code.
With regards to this specific exception, the issue lies here - https://github.com/aspnet/Mvc/blob/release/2.2/src/Microsoft.AspNetCore.Mvc.Core/ApplicationParts/RelatedAssemblyAttribute.cs#L118-L121. The method is provided a URI to an assembly that looks like so: file://c:/#NIN/dotnetcore/tryx/try1 and it's attempting to turn it into a path: C:\#NIN\dotnetcore\tryx\try1. The problem is Uri.LocalPath contains only the segment C:\, and the rest of the path is stashed away in the fragment.
One way to fix it might be to try to reconstruct the separate components. Another alternative might be to not use the URI path if it happens to contain a query or fragment component. Hope this helps.

Sounds great! I will look into that!

Could I claim it?

Sure, feel free to send us a PR. Our wiki has some instructions for contributing code.
With regards to this specific exception, the issue lies here - https://github.com/aspnet/Mvc/blob/release/2.2/src/Microsoft.AspNetCore.Mvc.Core/ApplicationParts/RelatedAssemblyAttribute.cs#L118-L121. The method is provided a URI to an assembly that looks like so: file://c:/#NIN/dotnetcore/tryx/try1 and it's attempting to turn it into a path: C:\#NIN\dotnetcore\tryx\try1. The problem is Uri.LocalPath contains only the segment C:\, and the rest of the path is stashed away in the fragment.
One way to fix it might be to try to reconstruct the separate components. Another alternative might be to not use the URI path if it happens to contain a query or fragment component. Hope this helps.

I think the correct path is file:///c:#NIN/dotnetcore/tryx/try1
According to this article:

Incorrect: file:////applib/products/a%2Db/ abc%5F9/4148.920a/media/start.swf
Correct: file://applib/products/a-b/abc_9/4148.920a/media/start.swf
https://blogs.msdn.microsoft.com/ie/2006/12/06/file-uris-in-windows/

@mkArtakMSFT  I think this problem is really hard and should not be labled "up-for-grabs"!
Also if you can help me with something: I tried to fix it but don't know how to actually test it by running "dotnet run". how can I use the newly built mvc DLLs?

Yeah I was looking at the code but had no idea how to start. Sorry I don't think I can work on this.

Hey! I think its up for grabs?
Would like to take a chance =] if so, let me know
Thanks

@sqdavid all yours to try 👍

yay! hope can find another issue I can help with =]

@sqdavid  is it ok to just ignore assemblies containing the # character?

how can I use the newly built mvc DLLs?

@obayit, you could update the dependencies of the https://github.com/aspnet/Mvc/blob/release/2.2/test/Microsoft.AspNetCore.Mvc.FunctionalTests/Microsoft.AspNetCore.Mvc.FunctionalTests.csproj project and that would give you a fully functional MVC app with your custom dependencies.

how can I use the newly built mvc DLLs?

@obayit, just noticed your comment. Just to make sure I answered your question, you could update the dependencies of the https://github.com/aspnet/Mvc/blob/release/2.2/test/Microsoft.AspNetCore.Mvc.FunctionalTests/Microsoft.AspNetCore.Mvc.FunctionalTests.csproj project and that would give you a fully functional MVC app with your custom dependencies.

Do you mean that I should write a test function to make sure my fix worked?

@sqdavid is it ok to just ignore assemblies containing the # character?

My first implementation was reconstructing Codebase using the fragment, but, as @pranavkm commented in my PR, usually we can trust LocalPath =]
https://github.com/aspnet/Mvc/pull/8556#discussion_r223122587

how can I use the newly built mvc DLLs?

@obayit, just noticed your comment. Just to make sure I answered your question, you could update the dependencies of the https://github.com/aspnet/Mvc/blob/release/2.2/test/Microsoft.AspNetCore.Mvc.FunctionalTests/Microsoft.AspNetCore.Mvc.FunctionalTests.csproj project and that would give you a fully functional MVC app with your custom dependencies.

Do you mean that I should write a test function to make sure my fix worked?

I built 3 new Unit Tests to validate my PR, but if you do a "dotnet run" you could test this specific implementation =]


feat(rate-limit) Add the X-RateLimit-Reset header

NOTE: Please read the CONTRIBUTING.md guidelines before submitting your patch,
and ensure you followed them all:
https://github.com/Mashape/kong/blob/master/CONTRIBUTING.md#contributing
Summary
While there's no standards on rate limiting headers, that I can find, a convention does exist
across a number of popular APIs (e.g. Github, Twitter, Vimeo, Reddit) using variations of the following.

X-RateLimit-Limit – The limit that you cannot surpass in a given amount of time
X-RateLimit-Remaining – The number of calls you have available until a given reset time stamp, or calculated given some sort of sliding time window.
X-RateLimit-Reset – The timestamp in UTC formatted to HTTP spec per RFC 1123 for when the limits will be reset.

The Retry-After header is mentioned in the IETF RFCs (e.g. RFC-7231 and serves
a very similar purpose to the X-RateLimit-Reset header, the only functionality that's missing from the current set of headers.
Full changelog

Add X-RateLimit-Reset header
Add test

Issues resolved
Fix #233

@salazar Thanks, updated with your comments :)

This has become stale and I am in favor of closing it. Any updates @grahamar?

Closing due to lack of updates. Anybody reading this is free to give another stab at this, as we would be very interested in adding such a feature to the rate-limiting plugin!


Bug: watch task does not exist

If you edit the package.json you will see this piece of code:
  "scripts": {
    "dev" : "gulp watch"
  },

watch task does not exists in the generated Gulpfile.

Yep, we change the name of the task to "serve" in the gulpfile and forgot to update the package.json.


Impossible to build a project without "factory" partition (IDFGH-12760)

Answers checklist.

[x] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.2.1
Operating System used.
Linux
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
What is the expected behavior?
I expected to be able to create a project having only ota_0 and ota_1 partitions (as described in the documentation).
What is the actual behavior?
Build process throws an error:
Executing action: all (aliases: build)
Running make in directory /data/src/intern/esp/esp32-io-bridge-s3/build
Executing "make -j 34 all"...
[  0%] Generating ../../partition_table/partition-table.bin
[  0%] Built target _project_elf_src
[  0%] Built target memory_ld
Error at line 6: Value 'factory' is not valid. Known keywords: ota, phy, nvs, coredump, nvs_keys, efuse, undefined, esphttpd, fat, spiffs, littlefs
Please check extra_partition_subtypes.inc file in build/config directory
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/esp_psram.c.obj
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/mmu_psram_flash.c.obj
make[2]: *** [esp-idf/partition_table/CMakeFiles/partition_table_bin.dir/build.make:75: partition_table/partition-table.bin] Error 2
make[1]: *** [CMakeFiles/Makefile2:2993: esp-idf/partition_table/CMakeFiles/partition_table_bin.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/esp32s3/esp_psram_impl_quad.c.obj
[  1%] Linking C static library libesp_psram.a
[  1%] Built target __idf_esp_psram
make: *** [Makefile:136: all] Error 2
make failed with exit code 2, output of the command is in the /data/src/intern/esp/esp32-io-bridge-s3/build/log/idf_py_stderr_output_62218 and /data/src/intern/esp/esp32-io-bridge-s3/build/log/idf_py_stdout_output_62218

Steps to reproduce.

Create a partition CVS file without a "factory" partition
run "idf.py build"

Build or installation Logs.
gaia erik:~/src/esp/esp32-io-bridge-s3 $ idf build
Detecting the Python interpreter
Checking "python3" ...
Python 3.12.2
"python3" has been detected
Checking Python compatibility
Checking other ESP-IDF version.
Adding ESP-IDF tools to PATH...
Checking if Python packages are up to date...
Constraint file: /home/erik/.espressif/espidf.constraints.v5.2.txt
Requirement files:
 - /home/erik/.espressif/esp-idf/tools/requirements/requirements.core.txt
Python being checked: /home/erik/.espressif/python_env/idf5.2_py3.12_env/bin/python
Python requirements are satisfied.
Added the following directories to PATH:
  /home/erik/.espressif/esp-idf/components/espcoredump
  /home/erik/.espressif/esp-idf/components/partition_table
  /home/erik/.espressif/esp-idf/components/app_update
  /home/erik/.espressif/tools/xtensa-esp-elf-gdb/12.1_20231023/xtensa-esp-elf-gdb/bin
  /home/erik/.espressif/tools/riscv32-esp-elf-gdb/12.1_20231023/riscv32-esp-elf-gdb/bin
  /home/erik/.espressif/tools/xtensa-esp-elf/esp-13.2.0_20230928/xtensa-esp-elf/bin
  /home/erik/.espressif/tools/riscv32-esp-elf/esp-13.2.0_20230928/riscv32-esp-elf/bin
  /home/erik/.espressif/tools/esp32ulp-elf/2.35_20220830/esp32ulp-elf/bin
  /home/erik/.espressif/tools/openocd-esp32/v0.12.0-esp32-20230921/openocd-esp32/bin
  /home/erik/.espressif/tools/xtensa-esp-elf-gdb/12.1_20231023/xtensa-esp-elf-gdb/bin
  /home/erik/.espressif/tools/riscv32-esp-elf-gdb/12.1_20231023/riscv32-esp-elf-gdb/bin
  /home/erik/.espressif/tools/xtensa-esp-elf/esp-13.2.0_20230928/xtensa-esp-elf/bin
  /home/erik/.espressif/tools/riscv32-esp-elf/esp-13.2.0_20230928/riscv32-esp-elf/bin
  /home/erik/.espressif/tools/esp32ulp-elf/2.35_20220830/esp32ulp-elf/bin
  /home/erik/.espressif/tools/openocd-esp32/v0.12.0-esp32-20230921/openocd-esp32/bin
  /home/erik/.espressif/python_env/idf5.2_py3.12_env/bin
  /home/erik/.espressif/esp-idf/tools
Done! You can now compile ESP-IDF projects.
Go to the project directory and run:

  idf.py build

Executing action: all (aliases: build)
Running make in directory /data/src/intern/esp/esp32-io-bridge-s3/build
Executing "make -j 34 all"...
[  0%] Generating ../../partition_table/partition-table.bin
[  0%] Built target _project_elf_src
[  0%] Built target memory_ld
Error at line 6: Value 'factory' is not valid. Known keywords: ota, phy, nvs, coredump, nvs_keys, efuse, undefined, esphttpd, fat, spiffs, littlefs
Please check extra_partition_subtypes.inc file in build/config directory
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/esp_psram.c.obj
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/mmu_psram_flash.c.obj
make[2]: *** [esp-idf/partition_table/CMakeFiles/partition_table_bin.dir/build.make:75: partition_table/partition-table.bin] Error 2
make[1]: *** [CMakeFiles/Makefile2:2993: esp-idf/partition_table/CMakeFiles/partition_table_bin.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  0%] Building C object esp-idf/esp_psram/CMakeFiles/__idf_esp_psram.dir/esp32s3/esp_psram_impl_quad.c.obj
[  1%] Linking C static library libesp_psram.a
[  1%] Built target __idf_esp_psram
make: *** [Makefile:136: all] Error 2
make failed with exit code 2, output of the command is in the /data/src/intern/esp/esp32-io-bridge-s3/build/log/idf_py_stderr_output_62218 and /data/src/intern/esp/esp32-io-bridge-s3/build/log/idf_py_stdout_output_62218
[1]    61932 exit 2     idf build

More Information.
Documentation says this should work.

Partition table. For testing purposes I renamed "factory"/"app" to "other"/"data".
# name		type		subtype		offset		size		flags
nvs,		data,		nvs,		0x9000,		0x4000,
otadata,	data,		ota,		,			0x2000,
phy_init,	data,		phy,		,			0x1000,
spiffs,		data,		spiffs,		,			0xf0000,
other,		data,		factory,	,			0x100000,
ota_0,		app,		ota_0,		,			0x100000,
ota_1,		app,		ota_1,		,			0x100000,

This is interesting. When I change the partition table to this, it works. Apparently IDF didn't like my oversight where I left "subtype" = "factory" on a "type" = "data". Build works and it runs.

Problem between ears and keyboard of user. User fixed, issue gone ;-)


Playground: ctrl+left/ctrl+right to move the caret between words doesn't work in firefox

same with ctrl+shift+left and ctrl+shift+right

This is an upstream monaco issue I believe - https://microsoft.github.io/monaco-editor/index.html


enhancement not use thread pool to handle exchange block

…
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


Add --erase-all to probe-run invocations

Newer models of the nrf52840 need this in order to flash.

@amanjeev Can you please test if now works for you out of the box?

@Urhengulas thank you! could you please link to probe-run fix PR here as well please for the sake of connecting the two?

Sure, https://github.com/knurling-rs/probe-run/pull/380


Want to learn: "@sveltejs/kit": "npm<EMAIL_ADDRESS>where did this package pull from? Do you have an article?

Thanks thank you

I used Google translate and the result is "thanks thanks".

想学习一下： "@sveltejs/kit": "npm<EMAIL_ADDRESS>这个是从哪里拉取的包呢？有文章么？

The main content is the title, not just the content. Hey hey

Ah OK, please write this in English so I can understand better.
You asked where the package is from, so the short answer is:
https://www.npmjs.com/package/@tal500/kit
This is a fork of SvelteKit 'kit' package I'm maintaining of PR sveltejs/kit#6265, until hopefully legacy support will be merged.
You can see in this page that the "homepage" field refer to the branch 'legacy-builds-versioning', which is a fork of the branch 'legacy-builds' for "temporary" clean fork NPM version purpose.
Hope it answers, you're welcome also to maintain your own fork of PR sveltejs/kit#6265 if you wish to use it carefully since it's unofficial.
Additionally, I'll edit and translate your messages for now.

Oh, oh, OK. Thank you for your answer. It turned out that it could not be sent to the npm warehouse. I am just wondering why we need to specify an npm:?

Oh, oh, OK. Thank you for your answer. Originally, it was also published to the npm repository. I am just wondering why we need to specify an npm:?

Oh, oh, OK. Thank you for your answer. Originally, it was also published to the npm repository. I am just wondering why we need to specify an npm:?

This line of code:
https://github.com/Tal500/sveltekit-legacy-demo/blob/565de0610f9e051e3ce175fdd4fe9c6b471863c4/package.json#L18
tells to NPM: "Install the package<EMAIL_ADDRESS>but name it as a package named "@sveltejs/kit"".
This makes all the import ... from "@sveltejs/kit/*" to work, even thought the NPM package name is now forked to be "@tal500/kit".
An example of such import, in:
https://github.com/Tal500/sveltekit-legacy-demo/blob/565de0610f9e051e3ce175fdd4fe9c6b471863c4/vite.config.js#L3

Oh, I see. Thanks for clarification