Jump searching in cpp

kindly issue me the code of jump searching in C++ language.
Description:-
A jump search or block search refers to a search algorithm for ordered lists. It works by first checking all items Lₖₘ, where k\in \mathbb {N} and m is the block size, until an item is found that is larger than the search key.

could you explain with the help of some diagram

And would you like to implement this algorithm in all four languages

Yes, surely I can do this in java and c language too

This is the diagram representation of the jump search.

Assigned!
@ADITYA0714 : C, C++, Java, Python

I Have made the code :
what all should i include before pull request.
it would be grt if  you tell in simplier manner as i'm  new to these things

Hey, @Kumar-laxmi is this issue available in C++? If not, please assign me this issue in C++.

I only know c++. But if you want me to implement ot in all languages I will have to take help of Google. Are you open to it?
It would be Ok for me until it is directly not plagiarised from Google

Assigned!
@sumitkr2000 : C, C++, Python, Java

Ok!


Removed duplicate "the" from documentation

Related issues:

https://github.com/Microsoft/TypeScript/pull/27526
https://github.com/Microsoft/TSJS-lib-generator/pull/572

This is fixed and can be closed.


Add method to get the list of languages using in repository

Hi kohsuke
I used your API to call Github API but I found that there isn't any method lists all languages using in repository. Therefore, I ask you to add it into your master code.
Thanks

Kohsuke Kawaguchi » github-api #285 SUCCESS
This pull request looks good
(what's this?)

Kohsuke Kawaguchi » github-api #286 SUCCESS
This pull request looks good
(what's this?)

Hi KostyaSha ,
Thanks for your feedback. I already changed the code as you mentioned

Thanks, looks good for me, let's wait @kohsuke

Merged, thanks!


Release 4.4.0

Questions
Answers

Description?
Release 4.4.0.

Type?
release

BC breaks?
no

Deprecations?
no

Fixed ticket?
Resolves PrestaShop/Prestashop#21381Resolves PrestaShop/Prestashop#21361Resolves PrestaShop/Prestashop#19230Resolves PrestaShop/Prestashop#15858Resolves PrestaShop/Prestashop#24394Resolves PrestaShop/Prestashop#24639Resolves PrestaShop/Prestashop#26767Resolves PrestaShop/Prestashop#25921Resolves PrestaShop/Prestashop#28632Resolves PrestaShop/Prestashop#30431Resolves PrestaShop/PrestaShop#16563

@sowbiba I updated the description and put Resolves correctly :) thanks for the PR, let's wait for QA now 😎

Perfect. Thanks @kpodemski


Value change within a token or tag

Ability to use a value replacement in a parameter or in between tokens
For example
<input value="[$last_bought]" />
Results in
<input value="Nothing yet!">
Because there is just no concept yet of a value being 'within' a tag. We currently default a value just being placed anywhere and wrapping it with a span, but that causes issues like this.
This also ties into html titles. I believe we do specific parsing now to remove the span if in a title: https://github.com/medusa-ui/medusa/blob/fef17c4e17e19defacd3ab25cc240fb785272776/src/main/java/io/getmedusa/medusa/core/injector/HTMLInjector.java#L93-L95
So we should default to not wrapping values with a span tag and remove special html title removal.
But the issue is that we do need a way to refer back to this value because when this value gets an incoming change event, we need to be able to do a lookup (ideally a from-value lookup) and swap the values.

First iteration, still not done with it: https://github.com/medusa-ui/medusa/commit/caa5552ae2ba4d395a11266ec3309aa69ecc9954

Updated with second iteration which fixes these problems: https://github.com/medusa-ui/medusa/commit/c5a5e71ac6a063398afb6d8400334d104376f65d


Adding the ability so that in tabs indicatorColor it can be a hex, rgba code or simply string primary/secondary

this is an initial commit change, I have just added a prop validation so that indicator color can accept a string also with secondary and primary.

@adeelibr I think that there is a misunderstanding. The feature you want to introduce in this pull-request was removed in #10999. In the issue, #11085, I was proposing to introduce a TabIndicatorProps property to the Tabs component so people can mess around with it. The dynamic inline-style is one use-case. What do you think about this plan?

@oliviertassinari Oh okay that makes sense, my bad! let me make some changes to my PR.

I have added a new prop in the Tabs component called tabIndicatorProps which I pass down to TabIndicator component. This prop is basically an object that takes in style params to customize the TabIndicator Component
Examples of using it
<Tabs
    value={value}
    onChange={this.handleChange}
   tabIndicatorProps={{
        height: 10, // change height of the bottom indicator
        backgroundColor: 'green',
       // etc, ..
   }}
>
    <Tab label="Item One" />
    <Tab label="Item Two" />
    <Tab label="Item Three" href="#basic-tabs" />
/Tabs>

@adeelibr It's a great first pull request on Material-UI 👌🏻. Thank you for giving it a shot!

@oliviertassinari I am super excited, this was my first open source PR. Thank you for helping me out with this PR. I intend to be more active in open source. Is there a possibility that if I stay active in contributing here that I can become an official member of the team? (Although I know I am new at this, but the experience has just been awesome, The learning and growth working is huge)
Is there a road map for this, also can I work in improving docs in material ui as well? I mean I can start working on that as well.

I reviewed the changes that you made to the PR, can you be so kind enough to tell me how is that you are passing TabIndicatorProps without using the style prop in the span element in TabIndicator component.
Also how is it that you are using type script in material ui? Can you share some resources please so that I can study on it, that way I can contribute better next time.
Thank you.

Also I tried updating my local forked branch to get it synced with the latest material-ui with the following commands
git remote add upstream git@github.com:mui-org/material-ui.git
git checkout v1-beta
git pull upstream v1-beta

This gives me an error
Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

This project works through meritocracy. The more value one contributor brings to the project, the more responsabilites he gets. Yes, people who joined the core team have been people undertaking important improvements, people deeply caring about the futur of the project, people sharing the same vision for the project.
There is no fixed roadmap. There is opened issue. There is the closed issue history. There is issue that brings context.  There are the documention vision, api, roadmap, governance etc. There are milestone where we try to share a sens of what's important.
But more importantly, contributing to open source tests people resilience. A lot gives up. You won't learn from me, you will learn from others.


CompositeAuth does not respect the $user of authMehods

What steps will reproduce the problem?
When developing authorization mechanism for REST API there is an option to use CompositeAuth to support several authorization mechanisms. In my case, I have two user components in the app and I would like to authenticate with either one of the methods. I would like to use ONE user for HttpHeaderAuth and ANOTHER (default) user for JwtHttpBearerAuth methods. However, compositeAuth does not respect the user set in the configs of authMethds. This is due to CompositeAuth using it's own $user property (which is set by default to Yii::$app->user).
'compositeAuth' => [
    'class' => CompositeAuth::class,
    'authMethods' => [
       [
         'class' => HttpHeaderAuth::class,
         'user' => Yii::$app->terminal,
          'header' => 'terminal',
        ],
        JwtHttpBearerAuth::class,
     ],
],

The problem is in authenticate(..) function of CompositeAuth class. It passes CompositeAuth::$user object instead of passing respected authMethod $user object.
//Current code
 $identity = $auth->authenticate($user, $request, $response);

//Proposed fix
 $identity = $auth->authenticate($auth->user, $request, $response);

Additionally, $request and $response should also be probably taken from the config of the $auth object and not from CompositeAuth properties.
What is the expected result?
What do you get instead?
Additional info

Q
A

Yii version
2.0.?

PHP version

Operating system

Agree. Do you have time for PR with this?

But the first param $user would be obsolete then, see https://github.com/yiisoft/yii2/blob/9fc8a2d37d600fed8229c8a7833d9aaff8c0b13e/framework/filters/auth/CompositeAuth.php#L65-L95
It only used in line 88 again. This would actually be an API change.
This change needs to be "triple" checked, looks to me like there could be very strange side effects.
PS: If changed, this should go into 2.2.
CC: @handcode @eluhr

I would rather use the CompositeAuth user when user in auth method is not set. The same for request and response.
Not sure about 2.2 though but definitely it needs to be checked throughout.

If it is a bug, it should be fixed.

It is a change, but also kind of a bug. My expectation is that $user of authMethods should be used and if null (not set), then CompositeAuth::$user should be used and if that one null (not set) then Yii::$app->user should be used.
This way it will maintain compatibility with existing/deployed code.

Whatever fix is implemented, it should also address CompositeAuth::challenge(...) function. It has the same problem as authorize(...).

The more I think about it the more I don't understand how this whole authenticate function is supposed to be used. On one hand it is supposed to be implemented as part of the interface with $user, etc being passed to it by some external code. But on the other, $user is a member of the class implementing this same interface. So which $user is supposed to be used? One being passed by external code or the one defined in the class itself??

I did some meditation on this and I don't think this is a bug anymore. CompositeAuth should be used in order to provide several ways to authenticate a user based on one specific model. If one of the methods is using different model it should use a different entry point for this (action). Overall I think the only thing missing here is a doc statement that configuring additional parameters for auth methods in CompositeAuth is pointless.

What would be a realistic use case for this function then? When would you develop two different authentication methods for the same front end? Why would you ever develop two different front ends for the same app that would use two different authentication strategies??
Your proposed solution of using a different controller/action will result in code duplication in my case because the code is exactly the same for two different users.

Just because this is your use case it does not mean that it is the general way of doing things. I changed my mind about this being a bug because I believe that what I described was a principle behind the design here - that is why I recommend separate action for your case in 2.0 (DRY is not something we must obey no matter what) and we should change the way it behaves but in 2.2 as @schmunk42 said, to be more user friendly.

But maybe you are right and the Yii documentation needs to be updated and say that there is only ONE user application component that is allowed. If you want to have a different user component, you need to develop a new app.

Very interesting. I was not aware of Yii 2.2. I thought everyone was working on Yii3. This will make sense. After you elaborated on your previous response, explaining the idea of re working this in 2.2 it makes sense to me.


EIP-7691 blob sidecar subnet computation not updated

Looks like this was missed in https://github.com/ethereum/consensus-specs/pull/4023
The electra spec only states that MAX_BLOBS_PER_BLOCK_ELECTRA should be used instead of MAX_BLOBS_PER_BLOCK
https://github.com/ethereum/consensus-specs/blob/e6bddd966214a19d2b97199bbe3c02577a22a8b4/specs/electra/p2p-interface.md?plain=1#L87
However in the gossip validation there is a check which uses BLOB_SIDECAR_SUBNET_COUNT but we should use BLOB_SIDECAR_SUBNET_COUNT_ELECTRA here as well
https://github.com/ethereum/consensus-specs/blob/e6bddd966214a19d2b97199bbe3c02577a22a8b4/specs/deneb/p2p-interface.md?plain=1#L180
It might be required to update compute_subnet_for_blob_sidecar (or simply extend the note in p2p-interface.md)
https://github.com/ethereum/consensus-specs/blob/e6bddd966214a19d2b97199bbe3c02577a22a8b4/specs/deneb/validator.md?plain=1#L183

How about opening a PR for that?

How about opening a PR for that?

Can do that, was debating if a note is sufficient on p2p-interface, but updating compute_subnet_for_blob_sidecar might be better / more explicit


Node event listener and logger

A few more tests are still missing...

Issue/Task Number: #1, #2, #3, #4, #7, #16
Closes #1
Closes #2
Closes #3
Closes #4
Closes #7
Overview
This PR contains the base code for the project and rudimentary ethereum event listeners and loggers.
With this PR, you can start using Potterhat for logging events from multiple nodes. Interfacing with Potterhat via JSON-RPC and Pub/Sub will be coming in the next few PRs.
This PR has been trialed with Geth, Parity, Pantheon, Infura.
Changes

Created base code
Implemented PotterhatNode subapp that connects to Ethereum clients
Implemented PotterhatOrchestrator subapp that will coordinate between the listeners (right now it manages starting of nodes from env variables and maintain a registry of active nodes).

Implementation Details
See omisego/OIP#15.
Usage
Setup the following variables, for example:
POTTERHAT_NODE_1_ID: "local_geth"
POTTERHAT_NODE_1_LABEL: "My Local Geth"
POTTERHAT_NODE_1_CLIENT: "geth"
POTTERHAT_NODE_1_RPC: "http://localhost:8645"
POTTERHAT_NODE_1_WS: "ws://localhost:8646"
POTTERHAT_NODE_1_PRIORITY: 10
...
POTTERHAT_NODE_2_ID: "local_parity"
POTTERHAT_NODE_2_LABEL: "My Local Parity"
POTTERHAT_NODE_2_CLIENT: "parity"
POTTERHAT_NODE_2_RPC: "http://localhost:8745"
POTTERHAT_NODE_2_WS: "ws://localhost:8746"
POTTERHAT_NODE_2_PRIORITY: 20

Run with mix run --no-halt, then you would be greeted with something like this.
18:17:10.523 [debug] Skipped HOME from node configurations.
18:17:10.538 [debug] Skipped HOSTNAME from node configurations.
18:17:10.537 [debug] Skipped LANG from node configurations.
18:17:10.537 [debug] Skipped PATH from node configurations.
18:17:10.537 [info] POTTERHAT_NODE_3_ID parsed as index: 3, key: id.
18:17:10.537 [info] POTTERHAT_NODE_3_CLIENT parsed as index: 3, key: client.
18:17:10.537 [info] POTTERHAT_NODE_1_PRIORITY parsed as index: 1, key: priority.
18:17:10.537 [info] POTTERHAT_NODE_2_CLIENT parsed as index: 2, key: client.
18:17:10.537 [info] POTTERHAT_NODE_3_RPC parsed as index: 3, key: rpc.
18:17:10.537 [info] POTTERHAT_NODE_3_WS parsed as index: 3, key: ws.
18:17:10.537 [info] POTTERHAT_NODE_2_RPC parsed as index: 2, key: rpc.
18:17:10.537 [info] POTTERHAT_NODE_1_WS parsed as index: 1, key: ws.
18:17:10.537 [info] POTTERHAT_NODE_1_RPC parsed as index: 1, key: rpc.
18:17:10.537 [info] POTTERHAT_NODE_2_PRIORITY parsed as index: 2, key: priority.
18:17:10.537 [info] POTTERHAT_NODE_1_LABEL parsed as index: 1, key: label.
18:17:10.537 [info] POTTERHAT_NODE_2_WS parsed as index: 2, key: ws.
18:17:10.537 [info] POTTERHAT_NODE_3_LABEL parsed as index: 3, key: label.
18:17:10.537 [info] POTTERHAT_NODE_1_CLIENT parsed as index: 1, key: client.
18:17:10.537 [info] POTTERHAT_NODE_2_ID parsed as index: 2, key: id.
18:17:10.537 [info] POTTERHAT_NODE_2_LABEL parsed as index: 2, key: label.
18:17:10.538 [info] POTTERHAT_NODE_3_PRIORITY parsed as index: 3, key: priority.
18:17:10.538 [info] POTTERHAT_NODE_1_ID parsed as index: 1, key: id.
18:17:10.696 [warn] Local Pantheon (#PID<0.273.0>): Failed to connect: econnrefused. Retrying in 5000 ms.
18:17:10.696 [warn] Local Geth (#PID<0.270.0>): Failed to connect: econnrefused. Retrying in 5000 ms.
18:17:12.068 [debug] Registered node: #PID<0.269.0>. Active nodes: 1.
18:17:13.278 [info] Local Parity (#PID<0.269.0>): Listening for new heads started...
18:17:13.417 [debug] Local Parity (#PID<0.269.0>): New block 7856057: 0x2994db63a4cd8b8c1ddade461be04852314c315f520ba5c958b4fedc1829c6c8
18:17:15.605 [debug] Local Parity (#PID<0.269.0>): New block 7856058: 0x07c95fb9baf519693b151899b308bb78f5214d19d8890da79748b6a42ad8f4d5
...

Impact
Shiny new stuff should only break itself.
Oh, it will also write logs to /var/log/potterhat.log if it has enough permission to do so.

Oh, it will also write logs to /var/log/potterhat.log if it has enough permission to do so.

I think a better approach would be to log to STDOUT. This is a pattern for containers and there's plenty of agents/consumers of this that can process the logs and send it off to another service for consumption. If you're just wanting to try this out stand up fluentd and Elasticsearch on the cluster. Then you get searchable logs with indices etc.

First quick scroll... Hands down, looks amazing. WiIl dig into details as well.

Ugh. Configured the wrong email so all the commits show unverified signature. Fixed from now on.

As a general rule, I would try and avoid pulling supervision trees between applications.
As it seems from the current implementation, the separation of potterhat_orchestrator and potterhat_node applications is not really needed.
What was your intention?
PotterhatNode.Node as a worker process has a limitation. Namely, a ethereum client node can do a lot more then 1 connection. I would add a :poolboy layer between them ASAP, because it'll shape the way you think what a node represents!
Currently, if a child event process dies, you don't try to restart the child from the PotterhatNode.Node process, is that intentional and if yes, whats the plan for maintaining valid performance stats?
Why does the PotterhatNode.Node have pub/sub functionality?

Also deferred_config ... if this goes through, try not to use it too much :-)
ethereumex is 0.5.4.
I don't know how you plan to implement the API layer... I guess if you're going the Ethereum jsonrpc route (and not phoenixes websockets) you might want to look at https://github.com/mana-ethereum/mana/tree/master/apps/jsonrpc2 and https://github.com/Azolo/websockex/pull/64

@InoMurko

As a general rule, I would try and avoid pulling supervision trees between applications.
As it seems from the current implementation, the separation of potterhat_orchestrator and potterhat_node applications is not really needed.
What was your intention?

The original intention was that the orchestrator would be the one determining which node would be the leader. But yes it looks like in the end I don't utilize potterhat_orchestrator at all, and all can be done within potterhat_node. Removing.

PotterhatNode.Node as a worker process has a limitation. Namely, a ethereum client node can do a lot more then 1 connection. I would add a :poolboy layer between them ASAP, because it'll shape the way you think what a node represents!

Not sure I get this. Enlighten me!

Currently, if a child event process dies, you don't try to restart the child from the PotterhatNode.Node process, is that intentional and if yes, whats the plan for maintaining valid performance stats?

Good point. Currently a child event process's death would restart the whole node (and so do the other event processes). Will update it so only the child event process is restarted.

Why does the PotterhatNode.Node have pub/sub functionality?

In #23 the exposed Pub/Sub websockets will relay the events by subscribing to the node via Node.subscribe/2

Tasks:

[ ] Remove file logging
[ ] Remove PotterhatOrchestrator
[ ] Implement poolboy
[ ] Isolate event process restarts

@InoMurko To answer

PotterhatNode.Node as a worker process has a limitation. Namely, a ethereum client node can do a lot more then 1 connection. I would add a :poolboy layer between them ASAP, because it'll shape the way you think what a node represents!

and

Currently, if a child event process dies, you don't try to restart the child from the PotterhatNode.Node process, is that intentional and if yes, whats the plan for maintaining valid performance stats?

would this work?

Node 1 and Node 2 GenServers on top are only responsible for node health monitoring, feeding its status to ActiveNodes.
A pool of RPC.Handler processes take care of the entire relay, from receiving the user request to making the actual call to the ethereum node, all within each own process.
The only bottleneck left is ActiveNodes.

cc @sirn @T-Dnzt

Looks good @unnawut !


Update onboard-windows-multi-session-device.md

No longer a limitation https://github.com/MicrosoftDocs/microsoft-365-docs/commit/d2711856ff71e4d912bde4b81e3be98e3f877181


fluid.grid~ does not appear in the reference search

is there somewhere we need to add it manually? everything else seems to be found in Max7 and 8, I've rebuilt the database, and the ref is definitely there in the folder..

Are you getting a new
build/interfaces/flucoma-obj-qlookup.json
build/interfaces/max.db.json
generated when you build?

They should have an up to date list of all the objects
They should be copied up into /interfaces as part of the install step.

IIRC from doing FrameLib the reference should work automatically if there is a .maxref.xml with the corresponding name inside Package/docs.
You don't need the qlookup or the max.db.json stuff afaik but I'm ready to be corrected 🤓

qlookup, iirc, gets you entered into the object launcher, or whatever it's called
max.db.json makes sure the objects appear in the search in the reference interface (e.g. if you start typing the name into the address bar)

That sounds right. I think if you right click the object and ask for the reference it just looks for the .maxref so its worth pinning down which bit of the docs feels broken @tremblap

I'm interpreting "does not appear in the reference search" as to do with the json files, because IIRC that was the feature request that led to them getting generated in the first place

the files are there. the stuff is new and put in interface. I can even find the object's reference by right-clicking the object.
What doesn't work is, as I said in the title, the search of reference. Add to this that autocomplete does not find fluid.grid~
That is all true for Max 7 only now. Now max 8 finds it after a restart of the computer. This is surreal.

I can even find the object's reference by right-clicking the object.

This depends on whether the .maxref exists, not on whether there's an entry in the max.db.json (which is what Max 7, but not 8, depends on for searchability).
Have you rebuilt the Max 7 db?

A new 'feature' of Max8 is that since I recompiled the doc (and the new cmake, it might be that) Max refuses to quit. I have to finish with a force-quit. I wonder if that is the new doc, or the new cmake compile messing something. I will switch to the latest public, reboot, and see if that happens. If not, I'll replace the doc with the old one and see if that happens. If not we kno it's the doc.

ok, this is fun.
Something breaks max8 silently (refusing to finish to quit) and max7 (not finding the ref in the search nor autocomplete) in the tip of doc+max - everything works fine with the last public release. Next step, replace the doc only from the release. I expect grid not to be found but at least Max to quit properly.

so, new compiles with old doc is working and quiting fine. Now, I'll recompile the doc put it at the right place and see

ok now it is a little saner. Max7 does not autocomplete fluid.grid~ nor find it in the reference search, but max8 behaves. I wonder if the construction of interfaces/flucoma-obj-qlookup.json and its buddy max.db.json is not faulty. I find grid in the former but not in the latter.

The problem here is that the code that generates max.db.json only tries (and needs) to gather together those objects in fluid.libmanipulation.mxo and was testing for the existence of a dump message as a common trait of all these objects, but fluid.grid~ doesn't have a dump message. So I'll need to go back to the drawing board for how to capture that.
The reason we need this is because in Max 7 (not 8), the reference search can't find objects mapped to a different external without the aid of this additional json file.
Happy enough for this to be labelled a bug, if the title is changed to say what needs fixing now that we know.

changed, made explicit.
Another option is to add  to grid, which would make sense. I am also tempted to add the max values for each dimension and dump would be handy there too. Nevertheless, other objects are likely to be missed so this renamed bug still stands.


To run BRS tests FS(X):EFI\BOOT\brs\SCT>SCT -s <BRSI.seq>

This should be:
To run BRS tests FS(X):EFI\BOOT\brs\SCT>SCT -s BRSI.seq

Thanks for the comments, fixed in 46425ef, Initially, I included angled brackets in the file name to indicate that other file names could be used within the brackets. Removing the angled brackets makes it clearer and more concise for users.

fixed in 46425ef


🐛 Fix chart type and prep for release 0.1.10

Summary
This PR fixes a typo in the Helm chart's values file, tracks the latest ks/ks release, and prepares for the next ks/OTP release.
Related issue(s)
Fixes #

@dumb0002: Thanks for the review. BTW, this repo is not hooked up to Prow. GitHub is waiting for you to use GitHub's approval mechanism.

I am going to merge this based on @dumb0002's informal LGTM.


Ability to specify port used by peer

Is your feature request related to a problem? Please describe.
When using netbird behind a gateway which also have a wireguard server set up on default 51820 port, netbird peer connexions are difficult if possible at all.
At home I have my router set up as a Wireguard server (not netbird, plain wireguard) to be able to remote in or remote exit from home.
I noticed I randomly couldn't reach some netbird peers on my laptop when at home and the command netbird status -d helped me remember peers are trying to handshake my pubic_ip:51820 which probably makes my gateway confused about it is supposed to handle the handshake or pass it to my laptop.
I just changed my gateway's wireguard server to listen to 51821 and I can now quickly connect to every availble peers.
I happened to visit a company the other day that probably also have a wireguard server listening on the default port 51820, I couldn't reach any netbird peers from this location and obviously could not ask those guys to change their wireguard setup...
Describe the solution you'd like
It'd be great if we could specify the port used by a peer from the UI, or have some kind of port range that a peer would try after some kind of connect timeout.
Describe alternatives you've considered
Changing the gateway wireguard server listening port is a workaround but not always under my will...

The netbird cli has a wireguard-port, example:
netbird down && netbird up --allow-server-ssh --wireguard-port 51821


Batched shading crashes in pointcloud_search()

Problem

Expected behavior: pointcloud_search() returns the same result in batched shading mode as in regular mode.
Actual behavior: pointcloud_search() crashes in batched shading mode with the following stack trace :
#0  0x00007fff312a32de in OSL_v1_12::b16_AVX512_noFMA_pvt::(anonymous namespace)::default_pointcloud_search(OSL_v1_12::BatchedShaderGlobals<16>*, OpenImageIO_v2_4::ustring, void const*, OSL_v1_12::Wide<float const, 16>, int, bool, OSL_v1_12::BatchedRendererServices<16>::PointCloudSearchResults&)::{lambda(OSL_v1_12::ActiveLane)#1}::operator()(OSL_v1_12::ActiveLane) const () from /home/john/dev/build/gaffer-1.3/lib/lib_b16_AVX512_noFMA_oslexec.so
#1  0x00007fff312a947f in osl_b16_AVX512_noFMA_pointcloud_search_masked () from /home/john/dev/build/gaffer-1.3/lib/lib_b16_AVX512_noFMA_oslexec.so
#2  0x00007fffd58002ad in ?? ()
#3  0x0000000000000001 in ?? ()
#4  0x0000000000000000 in ?? ()

Steps to Reproduce

Build OSL with the default Partio-based pointcloud services (I used Partio 1.14.6).
Write a shader that uses pointcloud_search() and run it in batched mode.

(These repro steps are a bit lame - I can probably come up with something more specific if the bug doesn't immediately jump out at you).
Versions

OSL branch/version:  1.12.9
OS: CentOS 7
C++ compiler:  GCC 9.3.1
LLVM version: 11.1.0
OIIO version: <IP_ADDRESS>

Did a bit more digging. The crash only occurs when not providing the "index" output array which the language docs imply is optional (and which can be omitted fine in non-batched mode), but which BatchedRendererServices claims is mandatory. The crash then occurs when writing via an invalid pointer in default_pointcloud_search().

@AlexMWells Can you take a look at this?

I added an additional unit test and can confirm it does indeed crash:
#### Error: this command failed:
2/2 Test #999: pointcloud.batched.opt ...........***Failed    3.47 sec
/bin/sh: line 1: 138828 Aborted                 (core dumped) testshade --center --vary_pdxdy -g 256 256 -t 1 -param radius 0.1 -od uint8 -o Cout out_rdcloud_varying_no_index.tif rdcloud_varying_no_index >> out.txt 2>> out.txt

I'll look at the implementation...

I created
https://github.com/AcademySoftwareFoundation/OpenShadingLanguage/pull/1753
to address the crash


updated evernote (6.5_453172)

Thank you for the contribution. It was merged directly as 120236d1fed31ef5e1e974a0266c2e62040e2994 to keep commit history cleaner. Your contribution is still credited to you.


java.io.IOException: Could not read whole file

Occurs when loading from Wildfly 10 with any inner jar on the VFS (lib)
12:10:07,238 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	-- Exception while opening file jar:file:/C:/java/devstudio/runtimes/jboss-eap/standalone/tmp/vfs/deployment/deployment231e896dadb0dd92/hazelcast-3.8.3.jar-c36cfdefd452a91d/hazelcast-3.8.3.jar!/hazelcast-full-example.xml
12:10:07,239 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- java.io.IOException: Could not read whole file
12:10:07,240 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ScanSpec.readAllBytes(ScanSpec.java:1506)
12:10:07,241 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ScanSpec.access$300(ScanSpec.java:72)
12:10:07,243 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ScanSpec$15.processMatch(ScanSpec.java:1551)
12:10:07,243 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ClasspathElementZip.openInputStreamAndProcessFileMatch(ClasspathElementZip.java:319)
12:10:07,244 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ClasspathElement.callFileMatchProcessors(ClasspathElement.java:314)
12:10:07,245 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.scanner.ScanSpec.callMatchProcessors(ScanSpec.java:497)
12:10:07,246 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:1379)
12:10:07,247 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:1440)
12:10:07,248 ERROR [stderr] (MSC service thread 1-4) 2017-10-02T12:10:05.543+0200	FastClasspathScanner	---- 	at io.github.lukehutch.fastclasspathscanner.FastClasspathScanner.scan(FastClasspathScanner.java:1463)

Hi, thanks for this report. Can you please test the master version, with the latest push? If it still crashes, please can you attach the whole log? (The latest version logs the actual and expected filesize, and fixes an issue with how read() was being called, but I also want to see the log lines about decompressing temporary files.) Thank you!

There was definitely a bug in how read() was being called. I don't know if it will fix your particular issue, but it would probably impact somebody eventually, so I pushed out version 2.7.1 with the latest change. It may be easier for you to try 2.7.1 rather than have to build your own version from git master. Please let me know if you still see the problem.

@GedMarc PS if version 2.7.1 or later doesn't fix this issue, can you please provide a small test project that demonstrates the issue? I tried and failed to get WildFly up and running once before for another issue.

Looking good :) Let's close this guy off!
Fantastic Library Luke!

@GedMarc Thanks, glad it fixed the problem for you!


jackson: Handle joda exception in FinatraCaseClassDeserializer

Problem
parseConstructorValues method doesn't not handle joda
IllegalFieldValueException, this causes that the exception is
re-thrown leading to 'Internal Server Error' while deserializing
a request into a case class.
This happens when the case class have a LocalDate field and the
request argument is an invalid LocalDate.
Solution
Handle joda IllegalFieldValueException to not return an
'Internal Server Error' with meaningless feedback but 'Bad request'
having enough feedback for the client.

Current coverage is 94.54% (diff: 25.00%)

Merging #372 into develop will decrease coverage by 0.17%

@@            develop       #372   diff @@
==========================================
  Files           210        210
  Lines          3048       3062    +14
  Methods        2830       2843    +13
  Messages          0          0
  Branches        200        196     -4
==========================================
+ Hits           2887       2895     +8
- Misses          161        167     +6
  Partials          0          0

Powered by Codecov. Last update bdf2b40...3f8ef86

@kevinoliver do you have any suggestion for writing the tests? I've tried to find a proper file without success.
I could write a new test class but I think it's not the proper way. Also I think that it would be better to catch 'IllegalArgumentException' instead of the joda exception (which inherits from IllegalArgumentException), what do you think?
Thanks.

+@bryce-anderson and @ryanoneill to see if they can better answer

@AlexITC you can add your case to the ExampleCaseClasses and put a route in the DoEverythingController, then a test in the DoEverythingServerFeatureTest. Thanks.

@cacoco I have updated the code to handle IllegalArgumentException and added a test for verifying the change. Thanks.

@AlexITC thanks, looks good. We'll work on getting this merged. Thanks!

This has been merged internally and will go out with the next release, thanks!


how to handle web API limits

In getting search results, recently played, songs in a playlist, etc., the API provides pagination limits. We decided to get rid of this for 99% of the use cases because we don't want to put the burden of restriction on the user. However, player.recently_played() cannot get more than 50. We are considering making the default playback limit 50 (the max), but this is kind of inconsistent with the rest of the API.

duplicate, see issue #16


Create fallback redirect on valoraapp.com for iOS users who land there when using WC

Problem: ios users land to valoraapp.com/wc?.....  instead of Valora mobile app, when trying to connect Valora to some dapp in the browser.
Expected behavior: when someone lands at valoraapp.com/wc we should automatically redirect them to the Valora app via deep link using the parameters from the URL where they landed.
there should be a copy saying that the user now will be redirected to the Valora app + button to download Valora if it is not installed on the device.

Closing in backlog spring clean May 2022.
The in-app browser reduces the impact of this


batch_size

You say that the the effective batchsize equals to batch_sizeiter_sizen_gpu, to be 128. Your parameter is 32*1(iter_size)*4(GPU), i have to set it to 16(bath_size)*4(iter_size)*2(GPU) make batchsize to be 128 due to my poor server,what about other parameter?such as the stepvalue and max_iter, keep it still or multiple 4 because the iter_size become 4 rather than 1? Do i need to change other parameter？ i do the work on the ucf101 spilt1.
i also have another question, you set the new_length to 5 in the tsn_bn_inception_flow_train_val.prototxt,
but the better parameter should be 10 according to the 2 stream paper(NIPS14).Is it a bug or 5 is better for TSN?
Thanks!

Generally, as long as you use the same effective batchsize, no other parameter need to be altered.
For the new_length, 5 is working reasonably well with TSN and leads to faster training. So we use 5 instead of 10. It is not a bug.


Remove getopt, try 2

Remove the unused call to getopt and getopt.

LGTM
Make sure you try this with make clean too.

Yes, I thought the Travis failure was unrelated since it looked so bizarre.

Still LGTM
When you bizarre Travis failure you can restart the build and see if it happens again.


Issue on docs: link redirects to main page

Path: /oss/guides/generate-report-ui
Under the section Install Elementary CLI, in this row:

If you're receiving command not found: edr please check our troubleshooting guide.

when I click the link, it redirects me to https://docs.elementary-data.com/introduction instead of a troubleshooting page.

Hi @LePeti !
Thanks for surfacing, we'll look into it.


Lifecycle callback related to route change?

Hi, my application is using web socket to receive some data, and I want to achieve the following:
The registration and unregistration of data streams is done by sending events to the socket. Only 1 connection is allowed for a single user.
Page A is registered to some streams, when I push any new routes, I need to unregister from those streams.
Then Page B comes in, and register other streams.
When Page B is popped, Page A is back and re-register to its streams.
The order of register and unregister is very important.
I tried writing the logic of unregistration in the dispose method of Page B, then put a callback of button in Page A like:
FlatButton(
  onTap: () async {
    _unregisterFromStreams();

    await Navigator.of(context).push(/* Page B*/);

    _registerFromStreams();
  }
)

But the _registerFromStreams() of Page A executes before the _unregisterFromStreams()
of Page B. Also if I do Navigator.of(context).pushReplacement() in Page B, the future gets resolved.
Since the order of register and unregister is critical in my application, this doesn't work.
Is it possible to run a callback before a route is pushed and after a route is popped? Or is it possible to get the widget in NavigatorObserver?
I looked at RouteAware but it is not suitable, since I may have Page A>Page B>Page C, I don't want the _registerFromStreams() of Page A to be executed when I popped Page C and go back to Page B.
Any help is appreciated!

This question should be asked in stackoverflow.

Or is it possible to get the widget in NavigatorObserver?

Which widget are you referring to?

I looked at RouteAware but it is not suitable, since I may have Page A>Page B>Page C, I don't want the _registerFromStreams() of Page A to be executed when I popped Page C and go back to Page B.

_registerFromStreams should not be triggered when popped page C in this case if you use didPopNext on page A.
It seems like RouteAware or NavigatorObserver should work in your use case

Thanks, I will give it a try on Monday.

RouteAware and RouteObserver did the job, thanks.


add Partition method

add Partition method.
Example:
dao.User.Ctx(ctx).Partition（“p1","p2").All()
change set -> sets

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (569cbb0) 77.28% compared to head (53fcc63) 77.23%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2989      +/-   ##
==========================================
- Coverage   77.28%   77.23%   -0.06%
==========================================
  Files         645      638       -7
  Lines       54287    54032     -255
==========================================
- Hits        41955    41730     -225
+ Misses      10226    10214      -12
+ Partials     2106     2088      -18

Flag
Coverage Δ

go-1.18-amd64
?

go-1.19-386
?

go-1.19-amd64
?

go-1.20-386
77.23% <57.14%> (?)

go-1.20-amd64
?

go-1.21-386
?

go-1.21-amd64
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

database/gdb/gdb_model.go
58.79% <57.14%> (-16.21%)
:arrow_down:

... and 15 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bugged heart

The bug is that you have won my heart with this software.
Steps to reproduce the behavior:

Go to group chat
See a meme of a flutter not paid package
Check pubspec.yaml if it's true
Your heart explodes coz it is unexpected, witty and bloody amazing

Expected behavior
Don't be so coollll
Additional context
I just wanted to say, your package made my night by my knowing it exists, and someone even thought of making it a package. Thanks a lot, you're amazing


Documentation for in_place is incorrect.

The documentation for the in_place function of path.py has incorrect syntax (at least on Python 3.4).

Thanks!

Released as 7.7.1.


Webhook thumbnail for items with NBT data

So there doesn't seem to be way to get poke ball thumbnails to appear in the webhooks since they are all specified with nbt.
Perhaps there can be some kind of nbt support in the item-replacement-urls section similar to the custom model data?
Such as -> pixelmon:poke_ball:PokeBallID:ultra_ball: "urlhere"
Or -> pixelmon:poke_ball{PokeBallID:ultra_ball}: "urlhere"
I can only think if this affecting poke balls atm but it could happen again if other items get the same nbt change.

Bump? :)

Added for latest version


Update recipe

This PR presents some relatively minor changes to update this recipe

patch out cython from runtime python requirements
patch out gslcblas from library links
a few other pedantic changes

Checklist

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

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


[vim] The black hole register ( "_ ) is not handled correctly

In Vim, when writing to the black hole register (_), for example with "_dd, the default register should not be written to. Likewise, when reading from _, the default register should not be read from.
The CodeMirror behavior does not match this.
Tested on https://codemirror.net/demo/vim.html against VIM 8.1:
VIM (expected):

Issuing "_dd, then p will paste the whatever was in the default register before the "_dd command.

CodeMirror (actual):

Issuing "_dd, then p will paste the deleted text from the "_dd command.

The docs on the black hole register can be found with :help change and searching "black hole". Web mirror for these docs is here.

I think a quick fix would be to add _ to the list of valid registers:
https://github.com/codemirror/CodeMirror/blob/6fcc49d5321261b525e50f111f3b28a602f01f71/keymap/vim.js#L419
Then not actually treat it as a black hole, but I think this would fix the default register being written to. I think that if you pass a register that is not in the list of valid registers then codemirror vim will fall back on the default register


Clarify Field.name in relation to the RFC

Hi there, this is not a bug per se but, I would like to clarify the function Field.name() which returns an option but, on the pointed doc it says that for a header for multipart body it is mandatory. The question is, on multer is it optional because can be used to represent both the multipart body's and the HTTP headers?

No, I don't think that's why. This library tries to strike a good balance between faithfully following the spec while also being aware of what happens in the real world to ensure applications using multer actually work in the wild. It's not unheard of for clients to sent fields without field names, and so outright rejecting such fields would mean not functioning with that class of clients. In that case the field would be None, of course. What's more, this allows applications to choose whether to reject fields without names (such as Rocket does) or not by simply filtering out fields without name.is_none().


Done with modal component

Added the modal component

Not able to see close icon in live app. Except that good to go

Bro its looking fine to me . Did you checked it from mobiles devices ?


Symmetry: Accept 'foreign' build artifacts without .ninja_log entry

Not sure about the condition in line 345 - we might want to disable that check too.


Corrupts Saves in Remaster! :(

This corrupts my remaster saves! :(
I want my extra quest items GONE! lol :(

I'm working on fixing this in #55 .
If you're building from source you can try that branch, but it's incomplete at the moment.
There are a few differences between saves in the original and the remaster:

The remaster treats the autosave, the tutorial, and the quicksave as special files and keeps them in memory once the starts. So realistically, the editor can now only mess with files of the format svd_fmt_5_x.sav
The remaster uses a new container that allocates a 6kb header and 4mb body for save data. Once that 4mb is exceeded it switches to zlib compression. I don't know what this looks like yet.
The remaster uses a CRC32 check on both the header and the save file.
The format for stashed items is different.
The data files I use are for the original, not the remaster. There is no current way to regenerate these files for the remaster.


[FSDE-20459] Prepare release of Chi 5.14.0

You can check this PRs instance in https://2a1d5a1810dc5143585fd52f5f8c2b8dc5aa3753-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-392-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-392/2/. :x:

The CI pipeline has run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-392/3/. :white_check_mark:

You can check this PRs instance in https://8e30e4bb833888c05d1e46a0aa9930f45d2e5782-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-392-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline has run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-392/4/. :white_check_mark:

You can check this PRs instance in https://a58847fe81397dfe32acb111794499e616b313cd-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-392-ux-chi.kubeodc-test.corp.intranet/ (internal)


@NgModule issue with Angular 4 and ngx-modialog

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
A bug report.

What is the current behavior?
I'm trying to create a DialogService, which can open modals. I've added an Modal as dependency for my service. I've also imported a ModalModule.forRoot() and BootstrapModalModule from ngx-modialog into my AppModule. But I've got an error when building in AOT mode.

ERROR in Unexpected value 'BootstrapModalModule in C:/Users/XRM/Documents/repairPayments/micro/Xrm.Micro/Xrm.Micro/node_modules/ngx-modialog/plugins/bootstrap/src/bootstrap.module.d.ts' imported by the module 'AppModule in C:/Users/XRM/Documents/repairPayments/micro/Xrm.Micro/Xrm.Micro/ClientApp/app/app.module.ts'. Please add a @NgModule annotation.

What is the expected behavior?
Everything is ok, no exceptions are thrown.

Please tell us about your environment:

Angular version: 4.3.6
Webpack: 3.x

Same issue
Uncaught Error: Unexpected value '[object Object]' imported by the module 'AppModule'. Please add a @NgModule annotation.
    at syntaxError

Is this still relevant? there is a new version (5) that works with angular v5.
Also, make sure you are working with the supported TypeScript version for your angular version, the error you have hints that you are using a TS version not supported by angualr.


 Frequently pts are taking this long-term and will experience withdrawal if they are stopped suddenly
Do we want to add all of the other common withdrawal syndromes? (like seratonin syndrome that is an outcome of stopping SSRI)

should we create a class for medication withdrawal syndrome? Substance withdrawal has the connotation of addictive or illegal drugs
do we need a grouping class 'withdrawal syndrome', add 'substance withdrawal' as a child + 'medication withdrawal syndrome' as children
consider making existing term broader substance withdrawal syndrome

I reviewed the definition of the parent term "substance-related disorder", and ensured that it includes alcohol, drugs, medications
I updated the definition of "substance withdrawal syndrome" to make the term broad and ensure that the definition includes alcohol, drugs, and medications (note that the original definition was a bit complicated and might have included medication)
created the new "baclofen withdrawal syndrome"

Note for the future: other withdrawal syndrome terms (e.g. "serotonin syndrome") will be created on an "as needed" basis.

Thank you @sabrinatoro and @nicolevasilevsky


Safari/Chrome: Height of overflow-hidden item in flex item in absolute flexbox container

I have following simple HTML snippet which has different result in Chrome and Safari

CSS

The content is shown perfectly in Chrome while in Safari, #3 height is 0. But if I do one of following

Set an explicit height to #1
Remove height: 100% on #2
Remove overflow:hidden on #3
remove disaplay:flex on #1 or #2

then the height of all elements are calculated correctly in Safari.

Can you create a CodePen demo? When I load your HTML I don't see the issue.


Blazor support

Hello,
I use your library in my Blazor project server side.
With RenderMode.ServerPrerendered mode the Blazor will load 2 times, at the first time I can get the tenant name but at the second time the TenancyContext is null.
But when I switch  to  RenderMode.Server, it will load 1 time and the TenancyContext is always null.
Can you give me any suggestion to fix it?
Thanks.

I resolve by adding ITenancyProvider into my constructor and getting content name from it. I check if the context is null I will get the tenant from TenancyProvider
if (tenancyContext.Tenant != null)
_tenancyContext = tenancyContext;
else
{
var task = Task.Run(async () => await tenancyProvider.GetCurrentTenantAsync());
_tenancyContext = new TenancyContext
{
Tenant = task.Result
};
}


Add Listview sample to showcase adding a context menu

Description

Adds a new listview sample that shows how to add a context menu.
Motivation and Context

Closes #519
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Very good point regarding the old brushes, I've updated the other samples too, so they don't include the old brushes too (and the samples all use the same border).

/azp run


In-Shop Clothes Retrieval

Not an issue of the code, just wanted to point out this dataset is available for download:
The DeepFashion page has a Google Drive link, and following that link you can find the retrieval dataset:
https://drive.google.com/drive/folders/0B7EVK8r0v71pVDZFQXRsMDZCX1E

Ok great, thanks for the info! I will include this in the README.


Extra space in pseudo-elements and pseudo-classes selectors

Description
When using rule with a pseudo-elements or pseudo-classes selectors inside a :not() selector and this rule is nested in another rule, less beautify add extra space after the first colon
Input
.one {
    color: #FFF;
    span:not(*::selection) { // pseudo-element
        margin-top: 0;
    }
}

.two {
    color: #000;
    span:not(*:active) { // pseudo-class
        margin-top: 0;
    }
}

Expected Output
The code should have looked like this after beautification:
.one {
    color: #FFF;
    span:not(*::selection) {
        // pseudo-element
        margin-top: 0;
    }
}

.two {
    color: #000;
    span:not(*:active) {
        // pseudo-class
        margin-top: 0;
    }
}

Actual Output
The  code actually looked like this after beautification:
.one {
    color: #FFF;
    span:not(*: :selection) {
        // pseudo-element
        margin-top: 0;
    }
}

.two {
    color: #000;
    span:not(*: active) {
        // pseudo-class
        margin-top: 0;
    }
}

Steps to Reproduce
Environment
OS:
Ubuntu, MacOSX
Settings
default one

I have the same problem


More test (unit test, integration test and doc test)

An exhaustive unit tests for serde_amqp might be necessary. Fixing a bug with AmqpValue lead to another bug with AmqpSequence


install failed

my installation log:
https://pastebin.com/pNDrHtPH
npm --version
6.14.16
node --version
v14.19.0

Duplicate of #201: NPM < 8 is not supported as installer due to how it forces SSH on repos. This has been observed in https://github.com/matrix-discord/matrix-discord-parser/runs/5239492006, and fixed by upgrading to NPM 8 https://github.com/matrix-discord/matrix-discord-parser/pull/24/commits/27262e1cb6c860882e2fad8ee7ff14917dd72a31


Homogenize API arguments

This PR causes breaking changes to API
Resolves the rest of #432.

[x] Remove dst_
[x] Replace rst with raster
[x] grid_size to size
[x] The various forms of sample (see below)
[x]  Always use points, never pts

Grammar of sampling
This PR proposes to regularise the grammar around the word sample and its derivatives.

When the intention is to return a (random) sample, the term sample is used. Previously, subset or subsample were both used to denote this purpose.
downsample or upsample defines a change in a Raster's resolution.  The more generic term is resample, but the use of a specific direction is preferred. Try not to use resample as this has connotations of towards reproject.
The terms subset and subsample refer to an contiguous portion of data (e.g. an x*y window of a Raster, specific bands of a Raster, or row n...row m of a Vector), without a change in resolution. I am regularising to subset.

It looks like subset is used only in split_bands, we could even rename into something specific to that function like bands to avoid any kind of confusion?

For consistency, we could use the same terminology as in load which is indexes. Although I'm not a huge fan of that term, I always forget the name...

I also don't find indexes that intuitive. From the xarray side of life, index can be along any dimension. How about the keyword band/bands instead, for both load() and split_bands()?

I also don't find indexes that intuitive. From the xarray side of life, index can be along any dimension. How about the keyword band/bands instead, for both load() and split_bands()?

I would be fine with bands. But this would just be undoing what was done in a previous PR: https://github.com/GlacioHack/geoutils/pull/346. Any thoughts @rhugonnet ?

+1 for grid_size and for bands!


Debug Top Strip

Issue Type: Bug
when debugging, the top "little debug status strip" is on top of the tab names, so you can't select the tab under the debug status strip. And since the (other bug) windows menu doesn't list other open documents, you can't even switch to some of the files you are editing while debugging.
VS Code version: Code 1.41.1 (26076a4de974ead31f97692a0d32f90d735645c0, 2019-12-18T14:57:51.166Z)
OS version: Darwin x64 18.7.0

You can see there's now other documents in the windows menu (separate bug) and the document under the little "debug gizmo" on top of the tab obscures some files you can't even get to now.

The debug.toolBarLocation setting allows you to have it docked rather than floating. And even when floating you can drag it downward to move it out of the way of the tabs.

You can see there's no other documents listed in the windows menu even though they are open (separate bug)

#88066 - closed, as designed

Closing as @gjsjohnmurray provided the answer.
For more flexible layout of the debug toolbar please file that issue and you can upvote it


⚠ Bofrost.fr has degraded performance

In cbb6889, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1235 ms

Resolved: Bofrost.fr performance has improved in 5ffe977 after 15 minutes.


Activity Feed privacy is not updated with repo privacy

Gitea version (or commit ref): 1.0.1
Git version: 2.7.4
Operating system: Ubuntu 16.04
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
When I change my project to private I expect that my activity related to that project should also become private.
However, Activity Feed items remain public after changing the project to private.
Workaround:
UPDATE action SET is_private = 1 WHERE repo_name = 'private-project';

Screenshots
Change repo to private:

Activity still shows public:

Yes but it's showing only for those who has access for this repository. Nobody who don't have access to this repository should not be able to see that repository activities on dashboard. At least I'm not able to reproduce that so that I can see activities when logged in with user who has no access to that repository

I can confirm the behavior is as you describe on try.gitea.org, which is the expected behavior.


ESPN basketball league initialization fails to find start date for 2023

Sport
Basketball
Summary
My overall use case right now is that I want to be able to pull down projections for the 2023 season in order to perform pre-draft value over replacement player calculations. I can't do this with the package as-is because my 2023 season league fails to initialize.
Steps to replicate as follows. This code does not run successfully:
from espn_api.basketball import League league = League(league_id=8675309, year=2023, espn_s2='foo', swid='bar')
I get an error that reads:
min() arg is an empty sequence
I do not get this error if I run it for 2022.
I traced this back to line 31 of league.py:
self.start_date = datetime.datetime.fromtimestamp(min([i[1][1]/1000 for i in self._get_pro_schedule(1).items()])).date()
I'm inferring that get_pro_schedule is returning an empty sequence, although I have not verified that. I can initialize a league object and return a player list if I comment out the above line and hard-code the start_date to the actual start of the NBA season, which I looked up online.
I don't know if this can be fixed or only documented as a limitation. I did not go deeper into get_pro_schedule to determine exactly what the cause of this is.
Thank you for your hard work on this! I've used this package for a few years and it's extremely useful.
Logs/Data
No response

Looks like you were correct, the _get_pro_schedule response was empty. start_date was not used anywhere so removing for now.


Connection URI string still requires setting of a DATABASE variable.

I am attempting to connect with a mongolab uri provided by heroku. I store it as an evironment variable.
MONGOLAB_URI="mongodb://heroku_tchBR549:saaaalute@aa111111.mongolab.com:57934/heroku_tchBR549"

In my config.py, I declare it like so:
MONGOALCHEMY_CONNECTION_STRING = os.environ["MONGOLAB_URI"]

... whch results in the following:
File "/home/clownshoes/project/venv/local/lib/python2.7/site-packages/flask_mongoalchemy/__init__.py", line 116, in init_app
"(the %s setting)." % key('DATABASE'))
flask_mongoalchemy.ImproperlyConfiguredError: You should provide a database name (the MONGOALCHEMY_DATABASE setting).

The database name is the final value after the port number in the URI.
When I actually put the database name in like so:
MONGOALCHEMY_CONNECTION_STRING = os.environ["MONGOLAB_URI"]
MONGOALCHEMY_DATABASE="heroku_tchBR549"

I can successfully connect.
init_app actually gets the uri at line 119. It seems that the DATABASE variable check might do it's job in _get_mongo_uri just as well, and avoid this problem by declaring DATABASE by snipping it off the end of CONNECTION_STRING.  (unless I'm reading it wrong, and brother, I am prone to that . . .)

Similar issue 3 years later. Anyone know a fix ? Problem with my set up maybe?


Crash 'Escape From The Orc Lair'

eamon.terranok.com-1492211092251.log.txt
Browser log attached. Game went into bad state after crash and would no longer accept commands.  Looks like crash happened after processing a monster fumble in one of the combat routines:
attack orc
Sauron swings at Orc
-- parried!
Mork tears at Orc
-- a fumble!
History ends.
Sauron is player character.
The offending line that threw the exception is:
if (wpn.type === artifact_1.Artifact.TYPE_MAGIC_WEAPON) {
Mork (friendly) was attacking Orc (28 Group Monster). I can't tell from the code if wpn was NULL or if the artifact_1 var is NULL. Based on the combat verb (tears) I don't think Mork was using an artifact weapon but was using a natural weapon. Prior to the block of the crash, the code does null ref checking since it possible a monster is using natural weapons. I think your miss/fumble block in monster.js might need to check wpn for NULL and exit out since a natural weapon can't fumble... hope that helps!
Very nice work! Had fun playing this module!

Thanks for the detailed report. I have seen errors before in the fumble routine, when a monster with natural weapons fumbles and the fumble roll is "damage weapon." I think that's what happened in this case. Shouldn't be too hard to fix.

Fixed in commit faa2813.


[Feature Request] Infinite scroll for el-select component

Existing Component
Yes
Component Name
el-select
Description
I would like to have an ability to enable infinite scrolling for el-select to be able to display a huge list of options using API.

Any updates?


User should be able to log in

Is your feature request related to a problem? Please describe.
In the current configuration, we have a bit of a bootstrap problem where you either need to build the ~/.config/epinio/settings.yaml file or have access to the cluster to run epinio settings update.
Describe the solution you'd like
A user should be able to run epinio login that would ask the user for the api url, username, and password.

The settings file also needs the CA for the epinio server which can't be fetched through the epinio API unless we expose that API through plain http too. The idea was that the operator give the user the full settings.yaml file not just the credentials. Pretty much like it happens with kubeconfig files.

The more important part of this is what is stored in kubernetes.
The password field there should be stored salted and hashed in a non-reversable way. Then when an auth request comes, you compare using the verify function instead of equality.
For the human side of this, the admin would hash a password, store the output while giving the input to the developer. Or the developer would give the hashed password to the admin to store.
This weirdness is why we need to eventually work with an upstream auth provider (such as Dex) and not handle passwords ourselves. That and people are going to ask how to hook into their existing LDAP server.

@agracey thanks, I think that the comment was related to the https://github.com/epinio/epinio/issues/1361 issue though.
This login part is going to affect only the developers settings.
I will address the hashed password issue in a separate PR.

You are right... I had both open next to each other

Since the change to hashed passwords will break epinio settings update and likely force this log in flow, the two likely need to release at the same time.
We will need to document this change as soon as both are released:

https://epinio.io: https://github.com/epinio/epinio.io/blob/main/content/_index.html
https://github.com/epinio/helm-charts/blob/main/chart/epinio/templates/NOTES.txt
https://github.com/epinio/homebrew-tap/blob/main/Formula/epinio.rb
https://github.com/epinio/extension-docker-desktop/blob/main/ui/src/epinio/Pusher.js
Digital Ocean scripts, not sure where they are?
https://github.com/epinio/docs/blob/main/docs/tutorials/quickstart.md (should add a note about logging in)
https://github.com/epinio/docs/blob/main/docs/howtos/install_epinio_on_rancher_desktop.md
https://github.com/epinio/docs/blob/main/docs/references/authorization.md
https://github.com/epinio/docs/blob/main/docs/references/cli/epinio_settings_update.md

Any other place referencing epinio settings update
This workflow change should also be noted in the release notes as it's a breaking change for anyone already using the platform.


Add testing for large write transactions

The largest testing we currently do regularly is with TPC-C. We should set up regular nightly tests with large writes at 4k and 64k to understand our performance and allow room for improvement.

opps wrong repo


(autohotkey.portable) Add note about AV false positives and where to ask questions

When this PR is merged, please push a new 2.0.3 version of the autohotkey.portable package.
Description
Add a Notes section to the description confirming where to find information on AV false positives and where to ask questions or raise issues for the package.
Motivation and Context
The latest 2.0.3 version of autohotkey.portable has 6 AV detections, and we need to add an official explanation to the description.
How Has this Been Tested?
This is just a change to the markdown / package description.
Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[ ] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[x] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] I have updated the package description and it is less than 4000 characters.
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:white_check_mark: Package verification completed without issues. PR is now pending human review


Bump sdk to v2.18.0

Signed-off-by: Nana Essilfie-Conduah<EMAIL_ADDRESS>Description:
Bump sdk to v2.18.0
Related issue(s):
Fixes #469
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #474 (e81fbc7) into main (512e1b8) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #474   +/-   ##
=======================================
  Coverage   76.46%   76.46%
=======================================
  Files          12       12
  Lines         922      922
  Branches      143      143
=======================================
  Hits          705      705
  Misses        165      165
  Partials       52       52

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


training in only specific categories of mvtec

added the functionality to use only certain categories of mvtec data

Looks good. Thanks for adding the additional flag and cleaning up the code.


Failed to parse guest name

Hii,
I am trying to install vmtop but when i run the command: # sudo ./vmtop.py --vm --limit 10 --sort vcpu_steal it shown just one vm and appear this errors:
Failed to parse guest name
Unexpected error on VM creation: (<class 'subprocess.CalledProcessError'>, CalledProcessError(1, ['pgrep', '852']), <traceback object at 0x7f9e7d30e388>)
where is the issue? @nanzhong @simondeziel @jdesfossez @jbwyatt4 @diunox

sorry for the delay !
any way you still have access to a reproducer case and you could share the output of that pgrep command ?


Binary counter.

Hello,
I'v test the binary counter feature.
It's seems that the binary counter don't take in count the html tags only the text content.
Have a nice day
Shaan1974

Yeah, That is the intended behavior.
I will add an option.
"charCounterType" : "char", "byte", "byte-html"

Ok thanks.

The 2.28.1 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


mobile: Make the mobile-remote-ci-cc config use C++17

The Mobile/Compile time options CI workflow runs for all Envoy changes, not just mobile, and so should catch mobile build errors (see mobile-compile_time_options.yml).
However, it was not set to use C++17, which mobile requires, so we didn't catch a PR that failed to build for Envoy Mobile because it used C++20 constructs (https://github.com/envoyproxy/envoy/pull/33067).
This change ensures that the Mobile/Compile time options CI workflow uses C++17 compiler options, so it can catch changes like this in upstream Envoy code.

This PR failed the Mobile/Compile time options CI job when introducing some c++20 specific code (consteval) into source/common/common/logger.h: https://github.com/envoyproxy/envoy/actions/runs/8442527013.
So the PR does what it was supposed to: catch c++20 in Mobile/Compile time options.
Now that https://github.com/envoyproxy/envoy/pull/33124 was merged, I'll sync to tip of tree and this PR will be good to go.


Notifications still say RIOT.IM

I had that until I rebooted the device.

yep as @karthanistyr said :/


Transformation error: crashes on TSInstantiationExpression

When I run the code on one of my codebases, it crashed with the error:
Transformation error (did not recognize object of type "TSInstantiationExpression")
Error: did not recognize object of type "TSInstantiationExpression"

I think it's related to a new TS feature like Map<string, Error>:
function makeBox<T>(value: T) {
    return { value };
};

const makeStringBox = makeBox<string>; // <- This is a "TSInstantiationExpression"
const stringBox = makeStringBox('abc');

const ErrorMap = Map<string, Error>; // <- This is a "TSInstantiationExpression"
const errorMap = new ErrorMap();

See the following code on astexplorer.net (view snippet) with JS and recast (jscodeshift's engine):

Should be fixed in 1.3.0.

Thanks!


Update README.md

Just to document how anymatch works with directories...geez why no docs before :(
thanks 👍 :)

Coverage remained the same at 100.0% when pulling d785e22c5ae9bbda5f93d2b6c5a84baa587ce471 on the1mills:patch-1 into 43f99be50105b504c18e1b51d1f8c1651623a2aa on es128:master.

This is incorrect, those examples would match as true (when being used for an ignore setting you want to positively match on the things you want excluded).
It also seems like an example that wouldn't be all that helpful to people with typical use cases.
I've opened #16 with my take on how this could be presented for now.


Two bugs with tactics

Hi all,
We've run into two tactic-related issues with Vale.  In both cases, we're trying to use the int canonicalizer.  You can reproduce them by checking out the fast_mul branch of the Vale repo (https://github.com/project-everest/vale) at commit b477a46.  In the base of the repo, run scons, which will take a few minutes (note that if you haven't built Vale before, you'll first need to run nuget.exe restore ./tools/Vale/src/packages.config -PackagesDirectory tools/FsLexYacc).
With the version that's checked in, you should see a tactic failure:
obj/crypto/aes/aes-x64/X64.FastMul.fst(83,3-83,19): (Error 23) user tactic failed: apply_lemma: Cannot instantiate lemma CanonCommSemiring.semiring_reflect CanonCommSemiring.const_last ...
The hypothesis is that it's failing due to an inability to unify with an if-else expression present.
To test this hypothesis, we tried changing src/crypto/aes/aes-x64/X64.FastMul.vaf to wrap the if-else in a function.  You can try this by commenting out line 128 and uncommenting line 129 and rerunning scons.  This version fails with an F* internal error:
<dummy>(0,0-0,0): (Error 230) Variable "__fname__mem#303637" not found
@nikswamy thought @mtzguido and @catalin-hritcu might be interested in this issue.  Adding @Chris-Hawblitzel as well, since this came up in the context of our Vale meeting this morning.
Please let me know if you need any help with the repro or with understanding what the code is trying to do.
-Bryan

Thanks for taking a look and fixing the bug, Guido.
I'm surprised that this code is using pointwise, which is very memory hungry especially in a large local context. Where is the pointwise coming from? I would suggest revising it to use topdown_rewrite, if possible.
Also, it may be possible that the memory explosion is coming from a normalization perf bug that I've been trying to fix on nik_whnf_scrutinee ... But haven't completed.
Nik
Get Outlook for Androidhttps://aka.ms/ghei36

From: Guido Martínez<EMAIL_ADDRESS>Sent: Friday, March 30, 2018 4:22:15 PM
To: FStarLang/FStar
Cc: Nikhil Swamy; Mention
Subject: Re: [FStarLang/FStar] Two bugs with tactics (#1415)
Thanks for the report Bryan.
The first bit, the unification failure, is indeed due to the if-then-else, which is desugared as a match and we don't have good unification for them. I tried to add that before but ran into trouble and moved on.
The second thing is an actual bug where I forgot to update an environment when descending through pointwise. It should be fixed now in master. Your example now goes further for me, but it seems to take a huge amount of RAM and I couldn't finish running it. It goes over 16 GB, are things expected to be this huge or is it maybe a separate bug?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FFStarLang%2FFStar%2Fissues%2F1415%23issuecomment-377644584&data=02|01|nswamy%40microsoft.com|f42e131aa0954c64233108d596951359|72f988bf86f141af91ab2d7cd011db47|1|0|636580489379096534&sdata=p5yRCb%2FR529Xs6FH2G6bbO3pUdzup3Wqj72BDJ7nsX0%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAF4Ln85ZEwRZakHYns7Ah6nb4TNDMIHFks5tjr4ngaJpZM4TB6qh&data=02|01|nswamy%40microsoft.com|f42e131aa0954c64233108d596951359|72f988bf86f141af91ab2d7cd011db47|1|0|636580489379096534&sdata=n6JNyxc56BfacKjz%2FLy07lgTC9RreMgjHn%2F1AHl9QG0%3D&reserved=0.

@mtzguido Thanks for the bug fix!  I do expect the tactic to finish in a reasonable amount of time, as I've tried in a similar, slightly cleaner context, and it succeeds there.  This could be the normalization bug @nikswamy is working on, or perhaps it's something tactic-specific.

Actually it was calling topdown_rewrite, I was confused since a debug message was saying Pointwise.
As for the bug, now I see this is actually looping, and I think it's somehow building an infinite term (through uvars). When I run it with --debug_level Norm it completely halts at one point and starts eating up RAM. Without, it doesn't stop and keeps showing typechecking debug messages, again eating RAM unboundedly. My guess is that in the first case it gets stuck at printing the term.

Hmm, sounds troublesome. A missing occurs check?

Sorry, actually I was wrong. It was taking an enormous amount of time while resugaring but I'm not sure it was actually looping. (FWIW, I did some measurements and found that compress calls were taking a really long time, multiple seconds, for some Tm_delayed-resolved nodes; it might be interesting to see if these can be improved.)
Also the huge terms were indeed coming from pointwise, via unfold_def, which has already been troublesome. To get around this I added a new normalizer step called UnfoldFully which is similar to UnfoldOnly but will continue unfolding the thing recursively.
This seems to work much better. But too much better it seems, since it's causing some failures in X64.FastMul:
obj/crypto/aes/aes-x64/X64.FastMul.fst(83,3-83,19): (Error 23) user tactic failed: trefl: not a trivial equality (Memory_i_s.buffer_read dst_b 0 (Mkstate?.mem b) +
0x10000000000000000 * Memory_i_s.buffer_read dst_b 1 (Mkstate?.mem b) +
0x100000000000000000000000000000000 *
(Mkstate?.regs b (X64.Machine_s.R11) +
  X64.FastMul.my_bool_to_nat (X64.Vale.Decls_i.overflow (Mkstate?.flags b))) vs Memory_i_s.buffer_read dst_b
  0
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = b in
    __fname__mem) +
0x10000000000000000 *
Memory_i_s.buffer_read dst_b
  1
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = b in
    __fname__mem) +
0x100000000000000000000000000000000 *
((let { ok = _ ; regs = __fname__regs ; xmms = _ ; flags = _ ; mem = _ } = b in
    __fname__regs) (X64.Machine_s.R11) +
  (if
      X64.Vale.Decls_i.overflow (let
            { ok = _ ; regs = _ ; xmms = _ ; flags = __fname__flags ; mem = _ }
          =
            b
          in
          __fname__flags)
    then 1
    else 0))) (see also /home/guido/r/fstar/ulib/FStar.Tactics.Effect.fst(86,32-86,42))
1 error was reported (see above)

due to the projectors being unfolded on one side and not the other. It's hard to pinpoint the discrepancy, since the unfolding logic is a bit unclear to me in the normalizer. @nikswamy, there's currently an unfold_until : option<S.delta_depth> in fsteps and a delta_level : list<Env.delta_level> in the cfg, and I'm not sure on what each of them represents; should there be a single one, maybe?
Also this should succeed if we properly unified matches, which maybe we can tackle next week?
For now I tried slapping a compute () before the trefl calls in CanonCommSemiring, and the tactic then succeeds (but a resulting query fails afterwards). So maybe that can get you guys going...
(merging to master in a few minutes)

@mtzguido, i haven't seen the complete code changes yet. but Unfold_only also does recursive unfolding, right?

To clear up the difference:

UnfoldOnly will descend into the definition of the symbol and continue normalizing as usual, possibly unfolding more things.
UnfoldFully will descend into the definition, but changing the configuration to normalize absolutely everything.

So for
let x = 2
let y = x + x

we have that
norm_term [delta_only [%`y]] (`y)   ==  `(x + x)

and
norm_term [delta_fully [%`y]] (`y)  == `(2 + 2)

Thanks for the explanation, that helps!
But can't we do it in the user-land itself, using topdown_rewrite? E.g.
module Test

open FStar.Tactics

let foo (l:list int) = List.Tot.length l
let bar (l:list int) = List.Tot.length l

//fully unfold the application of term t to n arguments
let unfold_fully (t:term) (n:int) =
  let should_rewrite (s:term) :Tac (bool * int) =
    let hd, tl = collect_app s in
    (term_eq hd t) && List.Tot.length tl = n, 0
  in
  let rewrite () :Tac unit =
    compute ();  //does delta for all
    trefl ()
  in
  topdown_rewrite should_rewrite rewrite

//unfolds foo but not bar
let _ = assert_by_tactic (foo [1;2] == 2 /\ foo [1; 2; 3] == 3 /\ bar [1] == 1) (fun () -> unfold_fully (`foo) 1; dump "A")

Or may be I misunderstood? (Just trying to see if we can avoid another normalizer flag.)

We could, that's in fact similar to the old unfold_def (expect that was using pointwise).
But the trouble is that topdown_rewrite doesn't descend to every subpart of a term (such as lets, matches, and a lot more), and it's also quite expensive. We could (and should!) make it descend to everything, but it will still be pretty expensive due to all the extra goals, typechecking, and unification.
I'd also like to not add a normalizer flag, but it seems reasonable to me here. I expect it to be orders of magnitude faster.

I guess it's fine, I was just curious to understand more.
I buy the incompleteness argument, but a bit fuzzy on the performance one. topdown_rewrite, unlike pointwise, won't generate redundant unification problems since it uses the callback, which I agree are additional reduction steps. Even for the unification problems t = ?u in this particular case of unfold_fully, t may not have nested uvars(?) and once we are in the rewrite above, it's just normalizing.

I made the changes to make matches unify, and some changes to the delta depths of projectors and their unification, and both alternatives now seem to be working (with an assertion failure at the end, which I'm assuming is unrelated). I noticed the second one is much slower for some reason, by the way.
The bad news is that they cause a bunch of regressions (so this hasn't hit master). We just spoke a bit in the developer's meeting about a possible fix. I'll come back with news soon.

Cool, thanks for the update!  Sounds like you're making progress.

I think I finally have a (partial..) fix, sorry this took SO long. It required changes in the unifier that took me a lot to get right. I think the current logic makes sense, and it passes existing regressions.
For this file, the tactic runs as expected (with both alternatives) but I still get the failing assertion at the end. I now see this seems to be due to the new unfolding, since the query in question is:
obj/crypto/aes/aes-x64/X64.FastMul.fst(32,0-92,16): (Info) Before calling solver VC=
(*Could not prove goal #1*)
Memory_i_s.buffer_read dst_b
  0
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = b in
    __fname__mem) +
0x10000000000000000 *
Memory_i_s.buffer_read dst_b
  1
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = b in
    __fname__mem) +
(0x100000000000000000000000000000000 *
(let { ok = _ ; regs = __fname__regs ; xmms = _ ; flags = _ ; mem = _ } = b in
  __fname__regs) (X64.Machine_s.R11) +
0x100000000000000000000000000000000 *
(if
    X64.Vale.Decls_i.overflow (let
        { ok = _ ; regs = _ ; xmms = _ ; flags = __fname__flags ; mem = _ } =
          b
        in
        __fname__flags)
  then 1
  else 0)) ==
Memory_i_s.buffer_read inA_b
  0
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = va_s0 in
    __fname__mem) *
Memory_i_s.buffer_read inB_b
  0
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = va_s0 in
    __fname__mem) +
0x10000000000000000 *
(Memory_i_s.buffer_read inA_b
  0
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = va_s0 in
    __fname__mem) *
Memory_i_s.buffer_read inB_b
  1
  (let { ok = _ ; regs = _ ; xmms = _ ; flags = _ ; mem = __fname__mem } = va_s0 in
    __fname__mem))

which contains a bunch of unfolded projectors (note this query is an unsolved tactic goal).
I guess the SMT chokes when presented with all this, but I'm not really sure. I'll keep looking and reply.

Pretty sure it's CanonCommSemiring that's unfolding these things, but I couldn't find the exact spot...

Where are your changes checked in?  Perhaps @Chris-Hawblitzel  and I should take a look at the SMT-side of things.

In master now

How did you arrive at such a compact description of the goal?  I tried changing my local definition of the tactic to be let int_canon = fun _ -> canon_semiring int_cr; dump "final", but it dumps a huge amount of info, starting with:
Location: /Users/parno/git/FStar/examples/tactics/CanonCommSemiring.fst(327,12-327,23)
ACTIVE goals (0):

SMT goals (3):
(va_b0: X64.Vale.Decls_i.va_code), (va_s0: X64.Vale.Decls_i.va_state), (dst_b: X64.Machine_s.buffer64), (inA_b: X64.Machine_s.buffer64), (inB_b: X64.Machine_s.buffer64), (p: Prims.pure_post (X64.Vale.State_i.state * X64.Vale.Decls_i.va_fuel)), (_:
Prims.squash (X64.Vale.Decls_i.va_require_total va_b0 (X64.FastMul.va_code_fast_multiply ()) va_s0 /\
    Mkstate?.ok va_s0 /\
    x:
    Prims.unit
...

and ultimately it reports Could not prove goal #1.  It doesn't seem to be a timeout, in that increasing the rlimit doesn't cause it to take any longer than before.  I also tried enabling Z3's NLarith, but that doesn't seem to help either.
Looking at your output above, it appears to be the correct, canonical expression, and it more or less lines up with the version as stated in the lemma in FastMul_helpers_i.fst, except for all of the let expansions, but the lemma goes through and the in situ version still doesn't.  Not sure if it's a tactic issue at this point, or just Z3 being unhelpful.

I get that output from --debug_level SMTQuery, --debug_level Rel or --debug_level Tac, any of those will print that bit. Z3 / our encoding should probably prove this one, but hopefully we should be able to control unfolding as well. I'll work a bit more on it tonight and report.

Wow, this is proving to be quite a doozy.  Thanks for sticking with it!

Fixed the unification issue, now it's unfolding too much again! But at least I have a better clue:
Somehow, the vmap in CanonCommMonoid is getting way too unfolded. The bad part is that it's out of the control of tactics since the vmap is a pure function from values to values, instead of returning syntax. If we could make them have term for a codomain, this issue would disappear completely since we would be in full control of normalization, so that's something we could explore.
Nevertheless, it's weird that it's normalizing so much, I think. So I'll look some more and see if something is fishy.

This actually seems to work in nik_rel, which should be landing into master soon, so this might just go up and running then! I think there's still a lingering issue, but we might be able to continue for now. I'll keep you posted.

Interesting that it's working with Nik's fixes.  That's good enough to let us keep making progress on our side at least.

Everything is now in master, btw.

For the record, we're now looking at this from the z3 side. The extra unfolding doesn't really seem to make a difference, and it's really hard to remove it given how CanonCommSemiring works, maybe plain  impossible.
The resulting query seems to involve some NL arith, which might be stumping z3 given the big context (even if pruning with the prune tactic). According to Nikolaj Bjørner, z3 might get improved support for (real) NL arith which we could leverage, but that's down the line...

Do you think we can massage the context further (possibly with something Vale-specific) to arrive at something isomorphic to the helper lemma context?  Or is it time to give up and decide that F*/Z3 can't handle this kind of query at this point, and so we should fallback to manually writing out the full version of the helper lemma?

This turned out to be a weird encoding issue, and was fixed by 63fabb782639c0a93de8c40eacf1ec74a8f8413b. We just forgot to close this.

Cool, thanks for the update.


Elements rebase

This changes all of the examples under /public to use Recurly.js Elements to initialize, style, and attach fields rather than through calling recurly.configure.
This also includes some changes to CSS selectors to accommodate the changes.

The card input fields are not styled correctly on the 3D Secure page

The card input fields are not styled correctly on the 3D Secure page

I think this is a caching issue. Can you try opening the page in an incognito window?


[ISSUE #1665] Add more unit test for Validators

What is the purpose of the change
Add more unit test for Validators
Brief changelog
XX
Verifying this change
UT passes
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.09%) to 50.559% when pulling ca7e458f307f0ad35e8c442bad212ab1e505e875 on jamesxuhaozhe:develop into f8f6fbe4aa7f5dee937e688322628c366b12a552 on apache:develop.

I'm not sure why some of the checks failed. But doesn't look they failed because of my change. Can someone kindly review this pr? Thanks!

@jamesxuhaozhe the Travis failed because the coverage decreased, you can ignore this temporarily, or add more unit tests to improve coverage.

@jamesxuhaozhe the Travis failed because the coverage decreased, you can ignore this temporarily, or add more unit tests to improve coverage.

Thank you. However, I am still confused. Because if you take a look at the code change I made in this pr. The change did nothing but adding more unit tests. Ideally, the test coverage should increase a little bit. Let me know how should I fix this and if I am missing anything. Thanks a lot!

@duhenglucky Not sure what is going on, I added even more unit tests like you suggested, but the coverage still dropped. Much appreciated if you can help to take a look at. Just trying to get this pr to merge. :)

@jamesxuhaozhe Would you like resole conflicts first?


Test failures on macOS 10.11.6 with Vim 8.1.0450

From test_terminal.vim:
Found errors in Test_terminal_scrape_123():
function RunTheTest[40]..Test_terminal_scrape_123[14]..WaitForAssert[2]..<SNR>6_WaitForCommon[11]..<lambda>89 line 1: Expected 3 but got 0
function RunTheTest[40]..Test_terminal_scrape_123[15]..Check_123 line 6: Expected True but got 0
Caught exception in Test_terminal_scrape_123(): Vim(call):E684: list index out of range: 0 @ function RunTheTest[40]..Test_terminal_scrape_123[15]..Check_123, line 7
TEST FAILURE
make[2]: *** [report] Error 1
make[1]: *** [scripttests] Error 2
make: *** [test] Error 2

From test_terminal.vim:
Found errors in Test_terminal_scrape_123():
function RunTheTest[40]..Test_terminal_scrape_123[14]..WaitForAssert[2]..6_WaitForCommon[11]..89 line 1: Expected 3 but got 0
function RunTheTest[40]..Test_terminal_scrape_123[15]..Check_123 line 6: Expected True but got 0
Caught exception in Test_terminal_scrape_123(): Vim(call):E684: list index out of range: 0 @ function RunTheTest[40]..Test_terminal_scrape_123[15]..Check_123, line 7
TEST FAILURE
make[2]: *** [report] Error 1
make[1]: *** [scripttests] Error 2
make: *** [test] Error 2

I don't think anything changed in this area.  Is this just flaky or does
it fail consistently?  Note that the CI on Travis doesn't have this
problem.
--
hundred-and-one symptoms of being an internet addict:
138. You develop a liking for cold coffee.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I don't think anything changed in this area.  Is this just flaky or does it fail consistently?

I suppose it's flaky.  I just tried again and everything passed.  Note that I'm rarely sitting at the machine that has 10.11.6 on it, so it might have started flaking a long time ago and I wouldn't know.


Remove: pnpm.pnpm.7 version 7.9.2

Replaced by pnpm.pnpm

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.2 schema?

@wingetbot run


Lint

Closes #167 #127 #91
Todo:

[x] make warnings pretty
[ ] write tests for rules
[ ] run lint on shep deploy
[ ] update readme

@chris-olszewski we had a little pow-wow and this got brought up. Can we rename the lint concept to doctor. We've got so much linting running around internally and constantly discuss it and use that word for another purpose, so want to appropriate something else here

Good with doctor. I don't want this on by default. I think shep deploy
should have the current behavior and we should add shep release to do
doctor and run tests
On Mon, Mar 6, 2017 at 9:18 PM, Zachary Foster<EMAIL_ADDRESS>wrote:

@chris-olszewski https://github.com/chris-olszewski we had a little
pow-wow and this got brought up. Can we rename the lint concept to doctor.
We've got so much linting running around internally and constantly discuss
it and use that word for another purpose, so want to appropriate something
else here
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bustlelabs/shep/pull/203#issuecomment-284535609, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAcxaFrooKJdHPND29_QCgDea0Mx1vODks5rjHgggaJpZM4MO0lj
.

@southpolesteve I believe this is ready to go. No longer addresses #127 as it sounds like that will be handled by shep release


Reference to Gameobject in USD-Prefab gets lost at runtime

USD 1.0.3.preview2 / Unity 2019.4.7f1
Steps to reproduce

Import 'ImportMesh' USD sample file
import Assets\Samples\USD\1.0.3-preview.2\ImportMesh\mesh.usd as prefab
Open mesh_prefab.prefab for editing.
Create a new MonoBehaviour

public class TestReference : MonoBehaviour
{
    public GameObject someObject;
}

Drag the TestReference behaviour onto the root USDAsset
Drag one of the cube gameobjects from the graph onto the field 'someObject' to reference it
Drag the prefab into the scene
Press Play

Observed Behaviour
The reference of field 'someObject' on the Behaviour is missing
Expected Behaviour
The behaviour is still referencing the cube gameobject
Workaround
At least what works for me now...
Convert
Reload(true)
in USDAsset.cs line 247
into
if(!Application.isPlaying){
   Reload(true);
}

Hi halifacx, this bug has been fixed in the dev branch already (https://github.com/Unity-Technologies/usd-unity-sdk/pull/159) and is lined up for the next release.
Thanks for reporting.


Cancel Return Request: A confirmation message perhaps?

Environment Details
Spartacus: epic/order-return-cancel branch
Browser: Chrome
Device: mac
Happens on: http://dev-com-10.accdemo.b2c.ydev.hybris.com:4200/electronics-spa/en/USD
Steps to Reproduce

Place an order
Go to Order History
Select a return request
Click on "cancel return"

ACTUAL RESULT
The return is cancelled.
Expected Result
Would be nice to have a confirmation message before it happens? To make sure that is what the user wants to do? But worst case, he can resubmit another return request....

yeah by design we just left it without a confirmation since you can just start a new return request
closing


Remove later in favor of sleep

The signature of later no longer serves any semantic purpose. I think we should remove it in favor of sleep :: Int -> Aff eff Unit.

Data.Time.Duration.Milliseconds?

Is sleep the correct name? Not to say it's not, but it might suggest a synchronous sleep, another term that is used for an async wait is delay.


Support MongoDB as an alternate data store

Not everyone has access to AWS. MongoDB is free and Open Source, so R2-D2 should be extended to allow the use of either data stores. This won't be straightforward but could be a pretty interesting task : )

hi there @ghostwriternr  I'd like to help this as a participant of Hacktoberfest.
Does this enhancement mean to

add mongodb image in docker-compose for local development
create connection to mongodb from the golang app

Please correct me or add more detail if my opinion above is incorrect. Thanks

Hi @feildrixliemdra! The goal of this issue is to enable R2-D2 to use MongoDB as a datastore instead of DynamoDB, since DynamoDB is AWS exclusive. Support for MongoDB can enable deployment of R2-D2 completely in non-AWS environments.
So, this would require writing a clean abstration over the DB layer that can be extended by Mongo/Dynamo and have a config that controls which DB will be used. Adding docker-compose support would be an addition after this is done, so that local development becomes easy.
Let me know if you have any further questions! 😄

i got it, i think i can help this but i need some time. I will update this through this pr #18 .
I tried to run the apps but the documentation only tell to copy the env and docker-compose up while it return errors. If you can provide steps to run R2-D2 apps locally that would be great! thanks @ghostwriternr