DFE with physical simulations

Attempt to have direct fidelity estimation without using the density matrix. Instead, use simulator.run().
Also, while writing the code, I found two mistakes. First is that the second run of compute_characteristic_function() did not use the 'noise' parameter. Second, the density matrix was re-computed 4**n_qbit times, which I think is wasteful. Sorry about these two mistakes, I think they are now fixed.
Also, sorry for the delay. I had decided to re-read my textbook, because my first PR required a lot of help from the reviewers.
I have been working with Matthew Harrigan, but happy to receive feedback from anyone.

Hello,
Do you have any comments on this PR? No hurry at all, but I am still interested in working on it.

Sorry for the noise in this review. I think what happened is that I replied to your comments a few minutes before I had pushed the changes. I did the reverse here, and I do see the call to PauliSumCollector on line 105. Sorry again.

Thanks. PTAL.

Friendly ping. No hurry at all.
In the meantime, would it make sense for me to study this?
https://arxiv.org/abs/0907.2833
That would be to address the TODO that this PR adds.

For a reference, I'd check out cirq.sim.clifford both for the code (which you will likely use) and the references it contains

Thanks for the review. I tried to address your comments.
Do you have advice on how to resolve Travis errors? They seem to both come from asyncio.run() which apparently doesn't exist for it (but it runs fine for me)?
PTAL.

asyncio.run was introduced in python 3.7 as syntactic sugar. To support python 3.6, change it to
loop = asyncio.get_event_loop()
# Blocking call which returns when the hello_world() coroutine is done
loop.run_until_complete(hello_world())
loop.close()

see also https://docs.python.org/3.6/library/asyncio-task.html#example-hello-world-coroutine vs https://docs.python.org/3.7/library/asyncio-task.html#coroutines

Thanks. I also made the Travis test pass.
PTAL.

Automerge cancelled: A required status check is not present.
Missing statuses: ['cla/google']

any ideas @Strilanc ?

Hello,
Not urgent, but is there an update on the merging? It seems that the CLA was missing, which is strange but appears to have been resolved.
Thanks in advance.

Thanks. Changed the comments.

Link to abstracts instead of pdfs. (eg, https://arxiv.org/abs/1104.3835 instead of https://arxiv.org/pdf/1104.3835.pdf)

I added some comments and the docstring. I am nowhere near an expert, so please let me know if anything is inaccurate. Thanks for helping me.

Fixed the

Fixed the check errors


Private channel can't receive despite being notified as active.

Background
Private channel that is opened by a remote node is not ready for payment and not listed in ListChannels RPC output despite a "CHANNEL_ACTIVE" notification dispatched.
Your environment

lnd 0.8.0-beta with neutrino backend running on mobile.

Steps to reproduce

Open a private channel from a remote node (B) to a local node (A)
Track channel activeness on node A via SubscribeChannelEvents
When active:
** Generate invoice - no routing hints exist in the invoice.
** Call GetChanInfo RPC for this specific channel - error returned [ERR] RPCS: [/lnrpc.Lightning/GetChanInfo]: edge not found
** The call:

client.ListChannels(context.Background(), &lnrpc.ListChannelsRequest{
   	ActiveOnly:  true,
   })

returns empty list.
Expected behaviour
Should have routing hints on the invoice so the channel can receive payment, should be listed in ListChannels and get no error from GetChanInfo
I think the reason is that the CHANNEL_ACTIVE event is dispatched immediately after channel reestablishment https://github.com/lightningnetwork/lnd/blob/master/htlcswitch/link.go#L992
And the channel_update message which is required for the channel to be usable only received later.

Update: When generating invoices right after channel active event is dispatched the routing hints are not added since the remote node announcement haven't been received yet.
I deduce that from the logs running in trace:
2019-11-01 15:21:14.901 [INF] IRPC: Skipping channel 6064b72e280933a652e85efae0e16e5e0d0fa3404a85cf69017a16ba11463f3d due to counterparty 031015a7839468a3c266d662d5bb21ea4cea24226936e2864a7ca4f2c3939836e0 being unadvertised

2019-11-01 15:21:14.901 [INF] IRPC: Skipping channel 6064b72e280933a652e85efae0e16e5e0d0fa3404a85cf69017a16ba11463f3d due to counterparty 031015a7839468a3c266d662d5bb21ea4cea24226936e2864a7ca4f2c3939836e0 being unadvertised

Is the node that the channel is being opened to aware of the other channels of the node opening the channel? Hints are only added for public nodes.

I see that there is a gap between the channel reestablishment that triggers the CHANNEL_ACTIVE to the time the node announcement (of B) is received at the local node (A). During this gap, routing hints are not generated.
Now thinking again this behavior makes sense.
The ListChannels with ActiveOnly filter still returns empty list despite the event dispatched. Do you think it is related to the missing node in the graph as well?

2019-11-01 15:21:14.901 [INF] IRPC: Skipping channel 6064b72e280933a652e85efae0e16e5e0d0fa3404a85cf69017a16ba11463f3d due to counterparty 031015a7839468a3c266d662d5bb21ea4cea24226936e2864a7ca4f2c3939836e0 being unadvertised

Is the node that the channel is being opened to aware of the other channels of the node opening the channel? Hints are only added for public nodes.

yes

Update: Did some more tests and I see that ListChannels does not depend on the node announcement but only on the channel_update that lands shortly after the channel is reestablished.
So in fact I think it is reasonable for now, thanks for the clarifications.
Closing the issue.

On second thought the current behavior is still problematic for being able to receive immediately after a private channel is opened as right now the hints are only added if the remote node is seen as public which takes minutes...
What is the reason for this restriction? In my case I know the node is public and it is only a matter of time until the graph will be updated to reflect that...
As this use case is crucial for good on-boarding experience I believe it is necessary to either let the user pass routing hints manually to AddInvoice or ability to skip the restriction for publicity of certain nodes.
Curious to hear others opinions here and if there are better approaches to handle this case.

There's actually already an issue and PR tracking this, see https://github.com/lightningnetwork/lnd/issues/2963 and https://github.com/lightningnetwork/lnd/pull/3022. Will close this as a duplicate of https://github.com/lightningnetwork/lnd/issues/2963.


update @webassemblyjs

What kind of change does this PR introduce?
dependencies

Did you add tests for your changes?
existing tests

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

@sokra So I did some digging into this failing on macOS. It appears that the should allow to create a WebWorker test case is causing the issue for custom-worker and web-worker tests in the configCases/worker category.
I noticed that for for node-worker cases the worker is being .terminate()d, but web-worker and custom-worker don't have a terminate statement at the end. So I added await worker.worker.terminate() at the end and the issue does not happen anymore.
Hope this helps!

@sokra So I did some digging into this failing on macOS. It appears that the should allow to create a WebWorker test case is causing the issue for custom-worker and web-worker tests in the configCases/worker category.
I noticed that for for node-worker cases the worker is being .terminate()d, but web-worker and custom-worker don't have a terminate statement at the end. So I added await worker.worker.terminate() at the end and the issue does not happen anymore.
Hope this helps!

Awesome, thanks for figuring that out...

Awesome, thanks for figuring that out...


Union estimate matrix space optimization

Currently, all the union_estimates for chopper pack with the -u flag are stored in a vector<vector<uint64_t>>. The total number of needed entries is n*n / 2 where n is the number of user bins. The matrix would hence consume 4 TB of RAM for around 1,000,000 user bins:
n * n | 1<<20 * 1<<20 = 1<<40
  / 2 | 1<<39
  * 8 | 1<<42

The matrices of the hierarchical binning itself need much less memory when the number of technical bins is reasonable. The clustering of chopper pack with the -r flag generally needs even more memory than the union estimates, but we can set the --max-ratio parameter (a.k.a. r in my thesis) to something close to 1 to reduce the amount of memory needed. This is achieved by performing multiple smaller clustering runs instead of a single large run.
It follows that the union estimate matrix is likely the memory bottleneck for chopper pack. There are a few ideas to mitigate this problem:

Don't store all union estimates, only the ones that will likely be used (e.g. we usually don't really need to know the cardinality of the union of all user bins and other large intervals of the user bin sequence). This might already be enough for our needs.
Use smaller data types as entries since we usually don't need the full range of a 64 bit unsigned integer.
Sophisticated matrix compression techniques from literature.

This was fixed by #100. Instead of computing the full matrix, we only store what is needed to compute a column in the DP matrix. This has implications on the runtime that were not benchmarked but should not be too high as it is not in the hot loop of the DP algorithm.


[0.5pt] Find external API - group task

Find an API to base the development of the webapp around it. The API should allow you to

Get a list of items with a unique item id (or generate the unique id).
For a given item, get detailed information about it.
choose an APIs that doesn't need authentication. if you select an API that requires authentication, you should implement it on your own. Also, if you select an API that provides image resources your webapp will be more visual.

Hello @MussieTeka
Great job on implementing the requirements.


Dice qc

This branch provides an additional QC metric by comparing hippocampal segmentations obtained from nnunet to those obtained by a much simpler method: f3d registration to the CITI168 template. This considers only the entire hippocampal volume. A dice score <0.7 is likely indicative of an issue with the nnunet segmentation.

I believe as of 7a568f15da79e739b58b6d4450ee682bab374e29 all the code is in place, but something still seems to be broken.
It seems snakemake cannot find the input for rule qc_nnunet_dice, which should be generated by rule qc_nnunet_f3d. When you get the chance, do you think you could have a look, @akhanf ?

@jordandekraker looks like this also includes commits from the other custom_density branch -- that might cause issues when merging things — to keep things clean do you mind creating a fresh branch from master that just has your commits? (ie 1441e61 and 7a568f1)?  Also, I noticed #75 has the same change as is here, so if that's the case can either remove that PR to include the change only here, or keep them separate?

Ahh sorry when I created this branch I had commits from various other branches in my path still. I'll try to clean it up.

I just reverted the changes from other branches, so the commit history is a bit messier here but it should be easy to see the Files changed. Let me know if this is still to hard to work with and I can delete the branch and try again

I just reverted the changes from other branches, so the commit history is a bit messier here but it should be easy to see the Files changed. Let me know if this is still to hard to work with and I can delete the branch and try again

Thanks! This works fine now, and we can squash the commits when the time comes to keep the master history clean.. Will take a look now!

@jordandekraker just made some commits to fix the workflow, should dry-run properly now (haven't tested it further myself)..
The edits that were causing it to not find the files were:

replaced input functions with string directly (could have also used
input functions, but would have needed to use the wildcards object for
string formatting, so this is more compact..)
removed wildcard constraints (was Lflip|R) for one of the rules, this was limiting the rule to only be run on Lflip or R, so wasn't able to produce the L output..

made some other cosmetic changes too, details in the commit message. Once you've tested it let me know, and I'll squash and merge it in.

Thanks for the help, Ali! Got things working on my end now.
Not sure if it belongs in the PR, but we could also add a group-wide QC report which would include a histogram of dice scores and a list of those with <0.7.

can add the group-level workflow in another PR


StrawberryResolver.annotation removed in release 0.115.0

I, and possibly others, am relying on the annotations of the resolver in order to "inherit" resolvers.
It would be great if we could have that information returned in a future release.

@nrbnlulu I'm sorry that you were inconvenienced by the removal of annotations. I've re-implemented the property in #1990. The output should be identical to how it was prior to Release v0.115. However, it makes use of the new implementation to determine reserved parameters. Please let me know if you experience any bugs once #1990 is released 😊


carla-autoware don't support carla version 0.9.13

Environment :
Ubuntu 18.0.4
Carla  version : 0.9.13
Ros : Melodic
Issue:  carla-autoware (0.910) don't  support carla version 0.9.13
Error Description :
[rosout][INFO] 2022-03-30 05:36:33,776: Trying to connect to localhost:2000
[rosout][WARNING] 2022-03-30 05:36:33,782: Version mismatch detected: You are trying to connect to a simulator that might be incompatible with this API. **Client API version: <IP_ADDRESS>. Simulator API version: 0.9.13-1-g8854804f4**

It is just a warning, does it work or not?

The environment that  can work normally,I did
Environment :
Ubuntu 18.0.4
Carla version : 0.9.13
Ros : Melodic

The environment that can work normally,I did
But I use the docker to build.
Environment :
Local -
Ubuntu 20.0.4
Docker -
Ubuntu 18.0.4
Carla version : 0.9.13
Ros : Melodic


Add an option on atom-text-editor to make it read-only

Currently in Nuclide, we create a subclass of TextEditor named ReadOnlyTextEditor to try to make a read-only version:
https://github.com/facebook/nuclide/blob/1fdf2b98/pkg/nuclide/ui/atom-text-editor/lib/ReadOnlyTextEditor.js

This is pretty hacky and tough to maintain because it's hard to be sure that we have blocked off all reasonable entry points that could mutate things.
Currently on master, the new refactoring of TextEditor that frowns on using the constructor directly and strongly pushes for the use of atom.workspace.buildTextEditor() breaks our existing use of extends TextEditor in implementing ReadOnlyTextEditor.

We can work around this using a new hack (will link once the diff is up), but it would be nice to have proper support for this in Atom. We use TextEditor in a number of places as a UI element to show things like console logs.
I'll try to put up my own fix for this when I get a chance, but I'm filing this to have a placeholder for now.
/cc @nathansobo @maxbrunsfeld

For reference, this is how we made a quick fix for this in Nuclide: https://github.com/facebook/nuclide/commit/c7dabd840e415973f23218fbf45d89a992dcb730

@bolinfest Were you thinking this would be instead of, or in addition to https://github.com/atom/atom/issues/6880?

Thanks @bolinfest. This is definitely something we've needed as well in a couple cases and have resorted to hacks to achieve. It would be good to do a solid, supported implementation.

+1
It is something I find useful in Emacs, the "read-only-mode".


GA not sending Completed Order as an event

From @gonegreen on May 3, 2014 19:17
I may be wrong here but it doesn't appear that GA sends the "Completed Order" track as an event. It correctly sends the ecommerce purchase but fails to actually send a standard event which requires the addition of another track() call with another event such as "Completed Purchase". Is this intentional or am I missing something?
All my transactions track in GA and all events are there but that one.
Copied from original issue: segmentio/analytics.js-integrations#134

From @chiplay on May 23, 2014 17:38
I'm also curious about this - currently send a seperate event, but that seems sloppy.  We need that event call for GTM rules.

From @chiplay on May 23, 2014 17:51
@ianstormtaylor As far as I can tell - it seems that any of the custom ecom methods ('Completed Order, Added Product, Removed Product, Viewed Product) that successfully call a method on the integration, bailout and dont call the original track event:
exports._wrapTrack = function(){
  var t = this.track;
  this.track = function(track){
    var event = track.event();
    var called;
    var ret;

    for (var method in events) {
      var regexp = events[method];
      if (!this[method]) continue;
      if (!regexp.test(event)) continue;
      ret = this[method].apply(this, arguments);
      called = true;
      break;
    }

    if (!called) ret = t.apply(this, arguments);
    return ret;
  };
};

Although currently, only Fox Metrics takes advantage of the add/remove/view methods.  What is the disadvantage to have the original 'track' still get called after the custom ecom method?

From @chiplay on July 8, 2014 17:33
@ianstormtaylor @yields Any thoughts regarding this issue?  We're starting to build out our analytics infrastructure in more detail and I'm still not crazy about having to make seperate track calls for every ecom specific event.  Thanks for your insight!

From @chiplay on July 8, 2014 17:42
@ianstormtaylor @yields one thing that is especially frustrating is that the behavior only happens when the ecom method is succesfully called. That mean for our other integrations (eg. Keen) we are sending two events for each checkout just so we can track conversion in GA.  Events don't grow on trees :)  What is the downside to having it always send the track event?

From @yields on July 8, 2014 17:54
will it hurt if we start sending them ? cc @DirtyAnalytics

That mean for our other integrations (eg. Keen) we are sending two events for each checkout just so we can track conversion in GA.

for that case you can just do
analytics.track('Completed Order', { ... }, { All: false, 'Google Analytics': true });

and it will only send to GA

From @yields on July 8, 2014 17:35
maybe we should add it as an option ? cc @ianstormtaylor

From @ianstormtaylor on July 8, 2014 18:39
@yields I think if we decide to do this, we could implement it pretty easily by just having a this.track() in the first part of the ecommerce events? That way we can make it an integration-level choice? Since some we wouldn't want to do it for.

From @chiplay on July 15, 2014 14:46
@ianstormtaylor I'm sure there is a better way to write this - but this seems like a clean approach and would allow for a single track call:
analytics.track('Completed Order', { ... some properties ... },{ track: true });

and in https://github.com/segmentio/analytics.js/blob/master/analytics.js#L6749
exports._wrapTrack = function(){
  var t = this.track;
  this.track = function(track){
    var event = track.event();
    var shouldTrack = track.obj.options && track.obj.options.track; // probably a better way to do this
    var called;
    var ret;

    for (var method in events) {
      var regexp = events[method];
      if (!this[method]) continue;
      if (!regexp.test(event)) continue;
      ret = this[method].apply(this, arguments);
      called = true;
      break;
    }

    if (!called || shouldTrack) ret = t.apply(this, arguments);
    return ret;
  };
};

cc @yields @DirtyAnalytics

From @ianstormtaylor on July 8, 2014 18:39
Oh but I guess that's where we'd have the circular problems you mentioned

From @chiplay on July 8, 2014 17:57
@yields nice work around - thanks!  Still would be great to remove all the duplicated calls if that doesn't effect other integrations.

From @sperand-io on September 30, 2014 17:52
Giving this a bump.
It's come up a few times in the past month and sucks for goal attribution / conversion funnel analysis in GA. From a customer:

"Commerce events are not de facto mapped as "regular" goals.  Personally I like the this.track() approach in the GA completedOrder method
@yields @ianstormtaylor @DirtyAnalytics

From @karamilo on October 4, 2014 19:16
Same here. It would be great to send a commerce event and a standard event as well directly.
I'm not excited about having 2 track() calls for each ecommerce events...

closing due to inactivity please reopen if it's still an issue!

Was this ever fixed? Segment doesn't seem to send Order Completed events to GA as events?


single choice breaks if choices > 5

sample test
self.single_choice("select?", ["lol", "lol2", "lastISA (FreeTFT) - ends in 30 minutes", "FreeNet (FreeTFT) - ends in 30 minutes",  "pub.play.grid.tf (FreeTFT) - ends in 11 months", "lol5"])

fixed. thanks..


Unknown fields in the Configuration API are silently ignored

What happened:
If you start Kueue with a Configuration containing fields that don't exist, Kueue starts normally.
What you expected to happen:
Kueue startup to fail, guarding users from typos.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Kueue version (use git describe --tags --dirty --always):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/assign @IrvingMg


[AAP-11474] Use singular form for event_source/event_filter

https://issues.redhat.com/browse/AAP-11474
Move

plugins/event_filters => plugins/event_filter
plugins/event_sources => plugins/event_source

Replaces #116


[GSSoC'23] Script for Scrabble game using GUI

Aim
Building the script for a Scrabble game using GUI
Details
This will produce a Scrabble game which can be played between 2- 4 users.
Please assign this to me under GSSoC contribution.

Go Ahead @nashira26


Lorenz96

There's only one build that failed, and only because it timed out. So, I'm gonna merge this into master 😬


Access Plain Text

As an Assistant Professor of History,
I need to access the plain text of all of our Civil War diaries
so that I may topic model them.
As a Professor of Art History,
I want to have access to the plain text of out of copyright scholarly European art periodicals
So that I may topic model the journals to see patterns of art scholarship.
John R.
Related: #90
Dependent: #153

This is for a single work. Automatic OCR - additional ticket. Give them access to the dirty, raw OCR. Maybe an alternative view. Kind of an annotation sort of thing. Conversion and download? Move to initial


go install of exectable does not work

This line appears to cause go install a bad day:
$ go install github.com/SaveTheRbtz/zstd-seekable-format-go/cmd/zstdseek@main
go: downloading github.com/SaveTheRbtz/zstd-seekable-format-go v0.7.1-0.20240727192458-ceaa236e14f4
go: github.com/SaveTheRbtz/zstd-seekable-format-go/cmd/zstdseek@main (in github.com/SaveTheRbtz/zstd-seekable-format-go/cmd/zstdseek@v0.0.0-20240727192458-ceaa236e14f4):
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

It returns with an exit code of 1, and does not install.

Should work now, thanks for reporting!


[ Proposal ] add «kill» after «Run current scene»

I just learning stuff via godot console, so I have readjusted pop-up game window to 1x1px and now I'm looking to my vscode terminal output
so it would be nice to close it after running the script
something like that
/bin/godot --path $workspaceRoot && pkill -f -n godot

extends SceneTree
↪ godot -s script.gd
sorru ~_~


Player GUI API

This is a very WIP API. I'm open to suggestions on how the API should be designed.
If I have missed and open-able GUIs then let me know.
API Issue: #696

I think it should be called Window as that's what it's called internally and is known by the packet name.

I suggest to name it like AnvilGui

@liach what about AnvilWindow

Anyways name like Window or Gui should be suffixes, as the names of AITasks and Minecarts.

The GUIs should expose their inventory if they have one.

@ST-DDT Actually I think it's better to relate some Guis with their Container inventories, like forge does.

If I have missed and open-able GUIs then let me know.

Also there's a demo screen:

I dunno about the use case (text override via resource packs?), but it's possible.

One problem I can see is detecting when the player opens and closes a window. It could be possible to detect when a player right-clicks on blocks/items that cause windows to open, though I'm not sure whether that would be a great idea.
So either getActiveWindow will only return a window that's been opened by calling showWindow, I remove the method, or I try to detect when a window will be shown.
The other issue is that not all Windows report when they are closed. Containers do, but for example the BookWindow (GuiScreenBook) doesn't. Not sure on a good way to get around this constraint.

Status?

You can create a class called ServerWindow which server-controllable windows extends from.

Closing in favor of https://github.com/SpongePowered/SpongeAPI/pull/1258

@Aaron1011 not really? This is for displaying windows on the client (could be inventory, could be others).
Custom inventory would be something that provides the backing data for an inventory, a Window (the view) is what it gets displayed on

@simon816: No, the custom inventory PR I linked handles all aspects of the creation process - including the underlying window type that's used.

You're not going to create a BookWindow or a WinGameWindow from an inventory system.
The Window API is more to do with showing GuiScreens to a client with full control of the things displayed (e.g. for a horse inventory window you can customize the model horse in the gui)

I want to make sure I don't overlap and conflict with the current proposal for a custom inventory API.
Feedback would be appreciated.
My take on the design is that a custom inventory defines the backing model for an inventory, kind of like a database model.
This GUI API is used to display that to the client. The custom inventory implementation will encompass the ContainerWindow-based GUIs and handle interaction from the player to the backend.

@simon816
This isn't really needed with InventoryArchetypes in that branch...you can build and display any Vanilla container in the game....in what scenario would you open the Window to the client and have no backing inventory?

To add further, you can figure out which Container you have open by calling getOpenInventory (in the branch) as that returns a Container and then call getArchetype... You can THEN also figure out the carrier as well :P.
What does this offer us? Not trying to stomp on your work, just want to make sure it all makes sense.

Ah well all that didn't exist when I made this PR. I'll go over the current custom inventory proposal and see whether there's anything this window API can add to it.

OK so I think most inventory-related Windows could be handled by InventoryArchetype, but not completely (more details below).
There are still some Windows that aren't covered by inventories.
My original inspiration for this was way back when the method editSign() was proposed (#682) and the subsequent issue (#696) that I based this PR on.
Here is a list of Windows that are non-inventory based:

BookWindow
CommandBlockWindow
DemoWindow
SleepingWindow
WinGameWindow
SignWindow

All others are types of InventoryArchetype
I can definitely see usecases for SignWindow and BookWindow because you might not want to have a physical object in the world in order to show these to players.
In terms of the inventory-based Windows, I'd like it to be possible to control as much of the Window as possible, for example a virtual furnace that can cook items in your backpack. You'd need to be able to offer FurnaceData to the Window to update the display, another example could be a virtual brewing table.
I can't see any way to actually offer this data back to an inventory instance and I don't think it makes sense to do so. It's not part of the inventory, rather, it's the state of the virtual furnace.
I'm not really sure how best to create something like this without creating a Window type for each InventoryArchetype.

Please update the target branch to bleeding (Edit this PR and change the target branch) and reopen.

This should probably be the preferred way to show inventory menus to clients as well right?
Where the current inventory API is better used for virtual inventories.
The main problem is potential dupe issues if events arn't cancelled correctly, where as if the inventory is only a window/gui rather then actually backed by something means that items can't be mistakenly taken onto another slot / container / dropped / cursor right?

@simon816 any news?

@simon816 Are you still planning to work on this? If so, you can also take a look at some changes I am making in Lantern which could be useful for the implementation of this API. I seperated a container implementation into two layers, a client one (which controls the window contents) and a server one (which handles clicking logic, number keys, etc.)
You can find it here: https://github.com/LanternPowered/LanternServer/pull/37
Just know that it still WIP, but it is almost finished.

Yes, I will update this soon™. I was discussing with a few sponge devs yesterday about whether this is redundant now that we have the custom inventory API.
I will re-do this PR to extend the custom inventory API, since a lot is already covered by it.
There's some pretty cool things I want to be able to do, e.g. custom control of the furnace/brewing stand progress bars, customising the horse model in the horse inventory gui etc.

Updates on this?

@simon816 When you eventually come back to this, will the PR be able to cover controlling repair costs in anvils as well as the name changing field? Or is that not relevant to this?


Fixes buggy location of feature 'ClausalCoreFile'

In the location of feature ClausalCoreFile, there is a small bug in the specified line.

@Keller-1998 Did you check that it's not a revision problem? We recently introduced a way to specify location with regard to the revisions they are valid in. Maybe the line number is correct for a different revision range?
Adding @bnico99 to double check this, as he initially introduced the picosat model

@vulder @Keller-1998 is indeed right, there was a typo in one of the locations I added.


Add e2e test for blob transaction

Describe the feature
ref
https://github.com/alloy-rs/alloy/issues/403#issuecomment-2023843596
alloy examples: https://github.com/alloy-rs/examples
TODO

use alloy for tx building and signing
send tx to the node
retrieve it over rpc
mine a block with it
ensure the sidecar of that tx is still present
reorg that block, ensure tx is reinjected into the pool, then mine block again (bonus for this PR)
finalize that block
ensure sidecar no longer exists

Additional context
No response

May I?


JSON Lines serializer tests are unreliable

I often get this test failure locally when running gulp test with Node.js 4.2.4:
  1) serializers/jsonl can write no points to a provided stream:
     Error: timeout of 10000ms exceeded. Ensure the done() callback is being called in this test.

Deeper in the log, there is this error:
  serializers/jsonl
    ✓ can instantiate a jsonl serializer
[16:22:51] 'test' errored after 44 s
[16:22:51] Error in plugin 'gulp-mocha'
Message:
    ENOENT: no such file or directory, open '/var/folders/zp/7fg2ysbj4g9drhgxdd32dk2c0000gn/T/tmp-31317JQgtQ5Zfaqxi'
Details:
    errno: -2
    code: ENOENT
    syscall: open
    path: /var/folders/zp/7fg2ysbj4g9drhgxdd32dk2c0000gn/T/tmp-31317JQgtQ5Zfaqxi
    domain: [object Object]
    domainThrown: true
Stack:
Error: ENOENT: no such file or directory, open '/var/folders/zp/7fg2ysbj4g9drhgxdd32dk2c0000gn/T/tmp-31317JQgtQ5Zfaqxi'
    at Error (native)
    at Object.fs.openSync (fs.js:549:18)
    at Object.fs.readFileSync (fs.js:397:15)
    at /Users/dmajda/Jut/Code/juttle/test/adapters/serializers/jsonl.spec.js:25:23
    at doNTCallback0 (node.js:419:9)
    at process._tickDomainCallback (node.js:389:13)
    1) can write no points to a provided stream
    ✓ can write out a few points correctly

This also happens on Travis CI, see e.g. this job.
@rlgomes Could you have a look?

I'll have a look at this today

Fixed by #157.


docs: HDP0015: Virtual memory

Signed-off-by: Dan Cross<EMAIL_ADDRESS>

@andrewth might you have some time to look at this?


Fix historical instances of one-space indentation.

No functional change.
Use-case: I was looking into #265 and ran into the boundary between the even-number-of-spaces and odd-number-of-spaces indentation, so I figured I'd fix this once and for all.
Review with git show -w --word-diff.
@phy1729 Would this cause conflicts to any outstanding PR of yours?

Use-case: I was looking into #265 and ran into the boundary between the even-number-of-spaces and odd-number-of-spaces indentation, so I figured I'd fix this once and for all.

See #700.


Fix input labeling on FF/Fedora

Per these codepens: http://codepen.io/anon/pen/KrALvb vs http://codepen.io/anon/pen/dXQEkO, on FF48 and Fedora 23, labels are misaligned with their input boxes.

This was also cross tested on a live Ubuntu 16.04 image with FF45:

Thanks,
Alex

Same problem on FF47, and no problems on Chrome 52

This should be merged BMO

Duplicate https://github.com/Dogfalo/materialize/pull/3821


Add Functionality and Documentation for DIY Camera Builds

Code changes to generalise camera support (Including tweaks to the menuconfig menus which you may want to change)
Generalise the IP5306 PMU as well (Previously was hardcoded to the M5Fire and Basic devices)
Add some options to enable a "Single Button UI" that basically just forces wrapping on every menu.
Config profiles for ESP32-CAM, ESP32-Wrover-CAM and T-Camera-Plus (All based around the "custom" device type)
Adjust profiles for M5Fire and M5Basic to use the full screen. (No scaling of elements or anything clever, just allowing the GUI window to take up the full space which spreads things out a bit)
Add hardware build documentation

The first item does touch a bit of code that is also used in the retail Jades, as well as make some changes that you might want to do differently in terms of menuconfig. Other than that the general approach has been to stick as closely as possible to the hardware found in the retail devices. (Everything seems to work quite nicely with the exception of the PinQR login workflow on T-Camera-Plus when on Battery and the ESP32-CAM, which may be power related but I haven't looked into it in too much depth)
Rather than just add endless profiles for component based builds, I just stuck with the "Custom" Jade type, expanded the functionality a bit and added some config profiles for three ESP32 boards that work quite well.

Hey thanks for this.
I have tried to integrate this into our internal CI - I have pushed this: diy_camera_jcd.  I have made small changes and squashed commits etc. and pushed here.
The additional commit shows an issue - our ci runs the script tools/check_default_configs.shand expects to see no diffs.  If it's just re-ordering it's easy enough to just check in the changes - but the changes here seem more significant.
I think its because the sdkconfig files as given are inconsistent with the menuconfig/Kconfig files ?
btw loving the 'full screen' m5stack devices ;-)

The fullscreen M5 stack is certainly nice and I am curious to play around with some options for a larger QR on those ones once this initial stuff is merged.
It all looks good and just gave it a quick test and it played nicely with an esp32-wrover-dev board, I can double check the other DIY hardware configs over the weekend as well.
My only comment is that leaving some kind of console feedback for button presses is probably worthwhile when debug is enabled, at least when the hardware type is "custom", as otherwise it might be challenging for folk to troubleshoot their hardware assembly. (This wasn't really relevant for the previous DIY options)

Do you think my branch version is good ?
I was worried the tools/check_default_configs.sh had materially affected your defaults files - I guess the acid test is whether the final generated 'sdkconfig' file is any different.
I'll check that now - should have checked before (d'oh!)

Ok, I have checked the generated sdkconfig files are indeed unchanged by the changes made to the defaults files - so that's all good.  Sorry for the noise.
I have pushed this branch to our internal CI so we'll see how it fares ... should be cool.
(I've put the changes it's hassling me to make into the separate commit, possibly for squashing later.)
Two thoughts really:

in all the places where we have 'HAS_CAMERA', are they all ok as just that ?  I'm thinking do any need to be 'HAS_CAMERA && HAS_PSRAM' ??     Jade devices not only have a camera they have 4Mb external ram, so the checks before on 'BOARD_TYPE_JADE[v1.1]' would have been implying camera and lots of ram.
It looks like the devices you've added all have camera and psram, so not a huge deal ... but I might make some tweaks to these ifdef checks in one or two places ... shouldn't affect the new devices.
If you're happy just leave it to me to make educated guesses!  (should anyone ever have a camera and low ram [if that even works!])
The main one I can think of is where it 'exports' a QR code for you to copy and scan back, but really I think all the places need a bit of consideration.

As much as I love the m5stack full-screen stuff maybe we need to hive that off into a separate PR, as will need a bit of an audit that all the screens look good (enough).  Maybe we want to make it a Kconfig option ?
(An advantage of having the m5 screen 'bordered' to look like a Jade screen is you can easily test UI changes on an m5stack knowing it will look the same on a jade proper/any-other-hw...)
I am a fan though - don't get me wrong!  ;-)
And I really appreciate the contributions mate - sorry it's taking so long to integrate.

Yea I checked the config files and it had mostly removed comments and other whitespace that I added.
In terms of your points

You are correct that there is no other check for the presence of psram, and you are right that I basically considered it implied in the code if you enable the feature. The camera initialization puts the framebuffer in psram, so it won't ever work unless folk are tweaking that code (and even then I couldn't get it to work without psram). If run without the psram, it at least visually throws an error when the user attempts to initialize the camera, rather than failing silently. (So basically it comes completely down to your preference as to how these kinds of dependencies are represented in the code)
Not a problem, easy to to and let people enable it themselves if they like without having to formally support it as a default :)

I saw the changes on the branch and it looks good, particularly switching things like the screen layout to relative rather than absolute, so well caught.

Cool.
I'll try to get a colleague to also review and will look to get it merged.
Thanks.

All good.  We're cutting a small release with some bugfixes and requested enhancements atm, and once that's tested, tagged and out the door this change will get merged to master.  Cheers.

No worries, no rush :)

Apologies for the delay but this has now been merged to master.
Some other config-related changes [eg. update to base idf version, update to button lib] have also been merged - I do not have the devices you have added to check they are working as expected - if you could that'd be great.
If you're happy with my rebasing/merging of your stuff and it's working as you intended, we can close this PR.
Many thanks, as always.

Great, thanks :)


postgresql diffChangelog dropping sequences erroneously

Environment
Liquibase Version: 4.7.1
Liquibase Integration & Version: CLI
Liquibase Extension(s) & Version:
Database Vendor & Version: postgresql 11.12
Operating System Type & Version:  macOS 12.2
Description
When using diffChangelog between postgresql sequences where the owner varies, liquibase generates a DROP statement instead of just ALTER SEQUENCE OWNED BY.
Steps To Reproduce
Using docker
`#Download sample database
wget https://raw.githubusercontent.com/morenoh149/postgresDBSamples/master/worldDB-1.0/world.sql
docker run -d -p 5433:5432 --name test1 -e POSTGRES_HOST_AUTH_METHOD=trust postgres:11.12
docker run -d -p 5434:5432 --name test2 -e POSTGRES_HOST_AUTH_METHOD=trust postgres:11.12
sleep 5
psql -h localhost -p 5433 -U postgres -d postgres -f world.sql
psql -h localhost -p 5434 -U postgres -d postgres -f world.sql
psql -h localhost -p 5433 -U postgres -d postgres -c 'CREATE SEQUENCE public.city_id_seq
AS integer
START WITH 1
INCREMENT BY 1
NO MINVALUE
NO MAXVALUE
CACHE 1;'
psql -h localhost -p 5434 -U postgres -d postgres -c 'CREATE SEQUENCE public.city_id_seq
AS integer
START WITH 1
INCREMENT BY 1
NO MINVALUE
NO MAXVALUE
CACHE 1;
ALTER SEQUENCE public.city_id_seq OWNED BY public.city.id;'
#acquire postgresql-42.3.1.jar and store in the same directory
liquibase diffChangeLog --referenceUrl jdbc:postgresql://localhost:5434/postgres --referenceUsername postgres
--url jdbc:postgresql://localhost:5433/postgres --username postgres
--classpath postgresql-42.3.1.jar --changeLogFile test.postgresql.sql
`
Actual Behavior
test.postgresql.sql will contain
`-- liquibase formatted sql
DROP SEQUENCE "public"."city_id_seq" CASCADE;`
Expected/Desired Behavior
test.postgresql.sql should contain
ALTER SEQUENCE public.city_id_seq OWNED BY public.city.id;

I simplified the setup a bit. After running your docker run commands, I just ran docker exec -it {NAME} psql -U postgres against both names, and ran:
create table city (id int);
CREATE SEQUENCE public.city_id_seq
AS integer
START WITH 1
INCREMENT BY 1
NO MINVALUE
NO MAXVALUE
CACHE 1;

against both, and  ALTER SEQUENCE public.city_id_seq OWNED BY public.city.id; against just test2.
But, the fix for this is going to have to be part of a larger project around postgresql and squences/auto-increment logic.
If you run liquibase snapshot against the two databases, you get
Catalog & Schema: postgres / public
    liquibase.structure.core.Sequence:
        city_id_seq
            cacheSize: 1
            dataType: integer
            incrementBy: 1
            maxValue:<PHONE_NUMBER>
            minValue: 1
            startValue: 1
            willCycle: false

    liquibase.structure.core.Table:
        city
            columns:
                id
                    nullable: true
                    order: 1
                    type: int4
            default_tablespace: false

for test1, but only
Catalog & Schema: postgres / public
    liquibase.structure.core.Table:
        city
            columns:
                id
                    nullable: true
                    order: 1
                    type: int4
            default_tablespace: false

for test2.
The sequence isn't included in the snapshot for test2 because the "owned by a column" is part of what we use to determine that it's not a "real" sequence but just an implementation detail of an auto-increment column.
But, in this case even though it's owned by the column, we've not set the column up to be auto-increment and so it just gets lost.
At some point we need to take a good look at how we handle "internal" objects like sequences that back sequences, but at this point I don't know of a solution to this issue that's not going to send us down that road before we're ready.
So for now, it's another example of "Generated snapshots are not always right. They are there as a convenience when possible, but can't handle everything so expect that you may have to fix them up before you try to apply them".

Thank you very much for the detailed response @nvoxland !

In fact, this issue causes a lot of problems and manual work, so it should not be just closed. Moreover, often creating an "OWNED BY" sequence is a recommended way when you use the "sequence per table" scenario because their lifecycles become unified: e.g. by deleting the table/column you delete the sequence.
Suggesting that "OWNED BY" sequences are only used as an internal technical detail of the autoincrement id implementation is wrong. Take a look at the Postgres user manual, which shows it as a totally legit way to use OWNED BY for non-identity sequences declaration: https://www.postgresql.org/docs/current/sql-createsequence.html
.
P.S. We are getting quite a bit of feedback on that as a vendor of JPA Buddy.

Talking with @aleksey-stukalov a bit, I think I may have a way to address at least some of this general problem in a straightforward enough way to handle it on it's own vs. as part of a larger project.
Reopening this issue to investigate that more.


Unable to Resolve Class From Parent Object

A working project demo is attached

[inheritance-issue.zip](https://github.com/gradle/gradle/files/7580973/inheritance-issue.zip)
Expected Behavior
Gradle should be able to compile the project
Current Behavior
Gradle build complains that Builder cannot be resolved
Context
The project cannot be built even though the IDE recognizes and resolves the class
Steps to Reproduce
-- A working project with all details implemented is attached

Create a new Gradle project with Groovy framework support
Create a new package called 'parents'
Create 2 new abstract classes
The second abstract class should extend the first
The first should have a generic Builder pattern
Create a third object that is dependent on the second abstract class

Your Environment
Build scan URL:
https://scans.gradle.com/s/7dm6d6j6qj3o6

This is a bug in the Groovy compiler, not Gradle. Please report it to them.
Proof:
$ groovyc src/main/groovy/**.groovy
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
src/main/groovy/ThirdObject.groovy: 5: unable to resolve class Builder <ThirdObject>
 @ line 5, column 33.
       protected ThirdObject(final Builder<ThirdObject> builder) {
                                   ^

1 error


Documentation for security analysis preprocessors.

Please check if the PR fulfills these requirements (please use '[x]' to check the checkboxes, or submit the PR and then click the checkboxes)

[X] The commit message follows our guidelines

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Documentation for security analysis preprocessors feature.

@sylvlecl, @MioRtia: This PR seems ready to merge: are we waiting for something in powsybl-core? It seems that this PR is linked to a merged PR in powsybl-core.

@sylvlecl, @MioRtia: This PR seems ready to merge: are we waiting for something in powsybl-core? It seems that this PR is linked to a merged PR in powsybl-core.

Yes, I am merging it!


pmem-csi: avoid false negative liveness probe

During a long-running DeleteVolume for a 200GB volume the LVM lock is held and
the liveness probe for metrics data timed out after a minute, killing the
container, because metrics retrieval must take the LVM lock.
As operation duration is non-deterministic, the solution is to probe only
metrics data which can be served immediately when the process is up and
running. This is less precise (= doesn't really check any functionality), but
that's an inherent problem of liveness probes: they simply cannot cover the
full functionality.
Fixes: #1046

@avalluri does this look alright?

I tried out the fix on the cluster with the 200GB volume that was pending deletion.


Implement basic http backend

This pull request implements a basic HTTP backend. It introduces a new http package that includes a Service struct with a requestFactory method. The requestFactory method creates an HTTP request based on the provided routing key. The Service struct also includes a handler field that uses the backend package to handle requests. This implementation lays the foundation for further development of the HTTP backend functionality.

Codecov Report
Attention: Patch coverage is 0% with 18 lines in your changes missing coverage. Please review.

Project coverage is 88.45%. Comparing base (261015f) to head (1b277db).

Files with missing lines
Patch %
Lines

pkg/prov/http/svc.go
0.00%
18 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   90.81%   88.45%   -2.36%
==========================================
  Files          21       22       +1
  Lines         675      693      +18
==========================================
  Hits          613      613
- Misses         43       61      +18
  Partials       19       19

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

@ksysoev not sure why the test is flaky. I hope the mutex implementation was correct.

@ksysoev not sure why the test is flaky. I hope the mutex implementation was correct.

Implementation is fine, maybe only suggestion is to use RWMutex, i got the idea of your test, but I'm not sure that we reliably can test this.


docs: fix variable name

🔗 Linked issue
none
📚 Description
The api endpoint is called comments. The data variable name should match that.

Thank you ❤


Inference on GPU?

Is inference on GPU is supported? As I understand lpcnet_demo is CPU only? Is there any measurements are avaliable on speed of lpcnet_demo?

Related https://github.com/mozilla/LPCNet/issues/38


Update Coder and DirectRunner documentation

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @lukecwik @kennknowles

Changes Unknown when pulling 970c58c9e35d7b9879f41527eadb98085a0dcb35 on tgroh:public_docs into ** on apache:master**.


service.dial.no_terminators

For network troubleshooting efficiency, it would be beneficial to have a specific dial fail metric for 'service XXXXX has no terminators' type events (service.dial.no_terminators).  These failures are currently rolled into service.dial.error_other with other causes.

Instead of adding a new service counter, we're going to add 'reason' enumeration to the new circuit failed events, which will give us more information/granularity than an interval counter. Closing this as a duplicate of #419


Base Bootstrapping of Navigation with TCA

This is the first task

[ ] remove all the Swift Router pattern stuff
[x] step up as a pure TCA project
[x] prepare what you consider is needed to bootstrap navigation

Swift Router pattern stuff will be removed in next PR (kept for sanity right now)


Flawed Application.isInitialized

Describe the bug
Application.isInitialized returns true before Application initialization is complete. Furthermore, correcting this breaks everything, even the build because Application.checkAppInitialized is used to ensure the Application is "initialized" during initialization.
To Reproduce
Steps to reproduce the behavior:

Change https://github.com/NationalSecurityAgency/ghidra/blob/5ab72bf4f254773b12c3c9566d14c42e5eb2495f/Ghidra/Framework/Generic/src/main/java/ghidra/framework/Application.java#L109-L112 to the following to correct it

Application app = new Application(layout, configuration);
app.initialize();
configuration.initializeApplication();
Application.app = app;
firstCreationThrowable = new AssertException("First call to initialize Application...");

Expected behavior
The documented behavior is is the expected behavior
https://github.com/NationalSecurityAgency/ghidra/blob/5ab72bf4f254773b12c3c9566d14c42e5eb2495f/Ghidra/Framework/Generic/src/main/java/ghidra/framework/Application.java#L120-L127
Additional context
I can just hack around it and use Reflection to check Application.firstCreationThrowable for now.

Making the changes to this got internally rejected.  The official guidance is to not use multithreading on this class.

Making the changes to this got internally rejected. The official guidance is to not use multithreading on this class.

No problem. I didn't think it was worth fixing, it just didn't behave as documented.


⚠ Bofrost.de has degraded performance

In 7774ea3, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1134 ms

Resolved: Bofrost.de performance has improved in e310ccb after 6 minutes.


Generalized expectation derivative

Implement a custom gradient for _sampled_expectation to enable derivatives of Hamiltonians with general BitstringEnergy diagonals.
Resolves #148

It accepts every Hermitian operator, the restriction is only on the representation of the operator


Fix broken release page links

The release page links in the README.md leads to 404 Not Found Error Page.
At the moment it looks like this:
Line#41: releases page
Line#132: * Java client library release notes
Proposed fix:
Line#41: releases page
Line#132: * Java client library release notes

Thanks for fixing the links!


Install the Optional SAP HANA Automated Predictive Library Package for SAP HANA, express edition (Native Linux Machine)

Tutorial URL: https://developers.sap.com/tutorials/hxe-ua-apl-binary.html
Step 1, when trying to run HXEDownloadManager_linux.bin, I receive the error seen below (Using HXE Docker 2.4):
hxeadm@hxe:/usr/sap/HXE/home> ls -lart
total 64
drwx------ 2 hxeadm sapsys 4096 Sep 21  2014 .local
drwxr-xr-x 2 hxeadm sapsys 4096 Sep 21  2014 .fonts
drwx------ 2 hxeadm sapsys 4096 Sep 21  2014 .config
-rw-r--r-- 1 hxeadm sapsys 6043 Aug 19  2016 .muttrc
-r-xr-xr-x 1 hxeadm sapsys  159 Mar  1 09:16 .sapsrc.sh
-r-xr-xr-x 1 hxeadm sapsys  176 Mar  1 09:16 .sapsrc.csh
-r-xr-xr-x 1 hxeadm sapsys 1791 Mar  1 09:16 .profile
-r-xr-xr-x 1 hxeadm sapsys 2495 Mar  1 09:16 .cshrc
-r-xr-xr-x 1 hxeadm sapsys 2256 Mar  1 09:16 .bashrc
-rwxr-x--- 1 hxeadm sapsys 3021 Jul 30 00:40 .sapenv.sh
-rwxr-x--- 1 hxeadm sapsys 2986 Jul 30 00:40 .sapenv.csh
drwxr-xr-x 4 hxeadm sapsys 4096 Jul 30 00:40 ..
drwxr-xr-x 6 hxeadm sapsys 4096 Jul 30 00:40 .
-rw------- 1 hxeadm sapsys   50 Jul 30 00:41 .bash_history
drwxr-xr-x 2 hxeadm sapsys 4096 Jul 30 00:41 bin
hxeadm@hxe:/usr/sap/HXE/home> cd bin/
hxeadm@hxe:/usr/sap/HXE/home/bin> ls
HXECheckUpdate_linux.bin  HXEDownloadManager_linux.bin  change_key.sh  hxe_gc.sh
hxeadm@hxe:/usr/sap/HXE/home/bin> ./HXEDownloadManager_linux.bin linuxx86_64 installer apl.tgz
which: no java in (/usr/sap/HXE/HDB90/exe/krb5/bin:/usr/sap/HXE/HDB90/exe/krb5/sbin:/usr/sap/HXE/HDB90/hxe:/usr/sap/HXE/HDB90:/usr/sap/HXE/HDB90/exe:/usr/sap/HXE/HDB90/exe/mdc:/usr/sap/HXE/HDB90/exe/Python/bin:/usr/sap/HXE/HDB90/exe/dat_bin_dir:/usr/sap/HXE/home:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
which: no java in (/usr/sap/HXE/HDB90/exe/krb5/bin:/usr/sap/HXE/HDB90/exe/krb5/sbin:/usr/sap/HXE/HDB90/hxe:/usr/sap/HXE/HDB90:/usr/sap/HXE/HDB90/exe:/usr/sap/HXE/HDB90/exe/mdc:/usr/sap/HXE/HDB90/exe/Python/bin:/usr/sap/HXE/HDB90/exe/dat_bin_dir:/usr/sap/HXE/home:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
This program is generated by unregistered Jar2Exe and it has expired to run for DEMO use.

Just an FYI - Someone is now complaining about this on the SAP forums:
https://answers.sap.com/questions/12857573/hxecheckupdatejar-jar2exe-has-expired.html

Thanks, Mike. Engineering is working on it

Tutorial URL: https://developers.sap.com/tutorials/hxe-ua-apl-binary.html
Step 1, when trying to run HXEDownloadManager_linux.bin, I receive the error seen below (Using HXE Docker 2.4):
hxeadm@hxe:/usr/sap/HXE/home> ls -lart
total 64
drwx------ 2 hxeadm sapsys 4096 Sep 21  2014 .local
drwxr-xr-x 2 hxeadm sapsys 4096 Sep 21  2014 .fonts
drwx------ 2 hxeadm sapsys 4096 Sep 21  2014 .config
-rw-r--r-- 1 hxeadm sapsys 6043 Aug 19  2016 .muttrc
-r-xr-xr-x 1 hxeadm sapsys  159 Mar  1 09:16 .sapsrc.sh
-r-xr-xr-x 1 hxeadm sapsys  176 Mar  1 09:16 .sapsrc.csh
-r-xr-xr-x 1 hxeadm sapsys 1791 Mar  1 09:16 .profile
-r-xr-xr-x 1 hxeadm sapsys 2495 Mar  1 09:16 .cshrc
-r-xr-xr-x 1 hxeadm sapsys 2256 Mar  1 09:16 .bashrc
-rwxr-x--- 1 hxeadm sapsys 3021 Jul 30 00:40 .sapenv.sh
-rwxr-x--- 1 hxeadm sapsys 2986 Jul 30 00:40 .sapenv.csh
drwxr-xr-x 4 hxeadm sapsys 4096 Jul 30 00:40 ..
drwxr-xr-x 6 hxeadm sapsys 4096 Jul 30 00:40 .
-rw------- 1 hxeadm sapsys   50 Jul 30 00:41 .bash_history
drwxr-xr-x 2 hxeadm sapsys 4096 Jul 30 00:41 bin
hxeadm@hxe:/usr/sap/HXE/home> cd bin/
hxeadm@hxe:/usr/sap/HXE/home/bin> ls
HXECheckUpdate_linux.bin  HXEDownloadManager_linux.bin  change_key.sh  hxe_gc.sh
hxeadm@hxe:/usr/sap/HXE/home/bin> ./HXEDownloadManager_linux.bin linuxx86_64 installer apl.tgz
which: no java in (/usr/sap/HXE/HDB90/exe/krb5/bin:/usr/sap/HXE/HDB90/exe/krb5/sbin:/usr/sap/HXE/HDB90/hxe:/usr/sap/HXE/HDB90:/usr/sap/HXE/HDB90/exe:/usr/sap/HXE/HDB90/exe/mdc:/usr/sap/HXE/HDB90/exe/Python/bin:/usr/sap/HXE/HDB90/exe/dat_bin_dir:/usr/sap/HXE/home:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
which: no java in (/usr/sap/HXE/HDB90/exe/krb5/bin:/usr/sap/HXE/HDB90/exe/krb5/sbin:/usr/sap/HXE/HDB90/hxe:/usr/sap/HXE/HDB90:/usr/sap/HXE/HDB90/exe:/usr/sap/HXE/HDB90/exe/mdc:/usr/sap/HXE/HDB90/exe/Python/bin:/usr/sap/HXE/HDB90/exe/dat_bin_dir:/usr/sap/HXE/home:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin)
This program is generated by unregistered Jar2Exe and it has expired to run for DEMO use.

the download can be run easily with the downloader jar if some java is installed, instead of the executable:
java -jar HXEDownloadManager.jar -d ./ztmp linuxx86_64 installer apl.tgz
Connecting to download server...

SAP HANA, express edition version: <IP_ADDRESS>.20190729.1

Downloading "SAP HANA Automated Predictive Library"...
apl.tgz : 100%
Concatenate download files to ./ztmp/apl.tgz...
./ztmp/apl.tgz created.
Verify ./ztmp/apl.tgz file checksum...
./ztmp/apl.tgz file checksum is OK.

HXEDownloadManager.jar is available where HXEDownloadManager_linux.bin can be found.
but that does not help with the updater, as there is no jar available, only /usr/sap/HXE/home/bin/HXECheckUpdate_linux.bin
Any progress with the updater?

This issue indeed looks like as it has been dropped... has anyone taken over Lucia's work after she left SAP?

@rbrainey Can you help here?

As per [this answer on SAP support] (https://answers.sap.com/questions/12857573/hxecheckupdatelinuxbin----jar2exe-has-expired.html)
you need to first reset the systemdate, because apparently the bin was built with a demo-version and has already exipered...
sudo timedatectl set-time 2019-07-31
after that, invoke the HXEDownloadManager_linux.bin command.

Ok, this does not look right.
First off, changing the local time/date back to when the demo license for the [Jar2Exe](https://jar2exe.com/) tool was still valid is a breach of the license agreement.
It's not simply cheating and therefore bad in a moral sense but depending on jurisdiction illegal.
The fact that this "solution" was offered on an SAP support site makes it look as if SAP is endorsing this.
While the fine-print for SAP Community makes it clear, that SAP is not liable for the stuff people post on the user forum, the fact that SAP packages up and delivers a time-limited demo software and let's a "solution" stand uncommented/unmoderated that basically leads to a license breach is not a good look (at the very least).
Looking further into the Jar2Exe license agreements, the packaging of the demo-software is problematic further. SAP explicitly allows and advertises the productive use of the SAP HANA Express Edition for memory configurations up to 32GB.
The Jar2Exe license, however, explicitly only allows that executable binary files created with the demo license to be used for testing and evaluation purposes.
The kicker is that the "biggest" license (the one that allows for productive usage within a workgroup, e.g. the team that releases the HXECheckUpdate_linux and HXEDownloadManager tools) for Jar2Exe costs just US$145.
At that price, somebody in that team should have considered whether just buying that tool would not be the right idea or if the tool is required at all.

Totally agree with you Lars, concerning the bad morality of the solution, but hey, we're trying to get the Express Edition and some of the addons running.
As this is currently the only available working solution, we just have to live with the fact that we're kinda cheating.
Unless ofcourse you can convince your SAP-collegues in de HXE-development team to purchase the jar2exe license ;-)

Ha! I would love to do that, but the SAP-colleagues are my former SAP-colleagues (I left the company a while ago) and I really don't have a "private line" to them.

I really can understand why everybody who is being asked to help here left the company.
It's such a big shame, that this problem is still not fixed yet.

Disclaimer: I'm not an SAP employee any more and I'm done with that community, but this thing with the dates is hitting rock bottom.
You can download a working version of HXEDownloadManager_linux.bin from the download page and upload it into your VM. No need to get shady with the dates.

Go here: https://www.sap.com/cmp/td/sap-hana-express-edition.html?infl=3d6178c3-dded-49c7-b799-32726fc8f17d - Follow the reg process

Choose "Linux DM" in the toast that says " Get the Download Manager (DM) for SAP HANA 2.0, express edition below." This should be easier than the JRE one, but pretty much the same and you can get that one from this toast too.

Upload the .bin file into the VM. Grant permissions to execute and run

As for the update checker, if you only want to see if a new version is out, I'd check the HXE site where releases are posted together when an update is released - this used to happen with a couple of hours of difference: https://developers.sap.com/topics/sap-hana.html#releaseNotes . You could also use the same downloader with a non-existent target directory and you'll get the date of the last update from there too.

I'm not saying this is good or convenient or anything, just better than the dates "hack".

I've also taken a similar approach as @Lsubatin -- Just copy the binary from the VM image.  Does it work?  Yes.  Is it silly and annoying?  Also yes.  Is this surprising given that it's SAP?  No.
PS - This technique (borrowing files from VM image) is also how I brought XS Classic delivery units into my HXE docker container.  It also works with a bit of elbow grease.
https://github.com/entmike/hana-dpagent/blob/master/BootstrappingHXE.md

Thank you for your feedback. The issue seems to be resolved (however not perfect it is), so I am closing the issue.
If you still have questions, feel free to reopen the issue.
FYI to @jung-thomas @JCurrie27

@thecodester please tell me, that some SAP guy really fixed the root of the problem and that you don't close the issue because of some really nasty workaround... Otherwise I really loose my last trust into SAP...

I'm sorry for your frustration. I meant to close the issue here because (a) it seems the issue is technically not about the tutorial but the product, and this forum is about the tutorials; (b) a question is already opened in the SAP Community (see above).
But of course the discussion can continue here, and we can provide more information when it is available.


Document pronunciation of "Submariner"

We should document how we say "Submariner", as in "Submarine-er", not the famous Rolex. This was raised in a comment on one of our YouTube videos.
https://www.youtube.com/watch?v=2EL5Md5hszA&lc=Ugz0rw9NJlEHfzWtPP54AaABAg

We follow UK pronunciation ;-). (At least according to the dictionaries I checked, both pronunciations are encountered in the US, there’s nothing wrong about our way of saying “Submariner”.)

@dfarrell07 is it OK if we close this?


SD TTNN: TtCrossAttnDownBlock2D

Implement subgraph TtCrossAttnDownBlock2D and make two tests: one for 256x256 image size and one for 512x512 image size.
Have one test file named something like test_cross_attn_down_block_2d.py and inside it two tests test_cross_attn_down_block_2d_256x256 and test_cross_attn_down_block_2d_512x512. Tests should be parametrized with following input shapes:

[ ] Shape 1:  256x256

[ ] hidden_states case 1: [2, 320, 32, 32]
[ ] hidden_states case 2: [2, 320, 16, 16]
[ ] hidden_states case 3: [2, 640, 8, 8]

[ ] Shape 2:  512x512

[ ] hidden_states case 1: [2, 320, 64, 64
[ ] hidden_states case 2: [2, 320, 32, 32]
[ ] hidden_states case 3: [2, 640, 16, 16]

For all the 6 cases mentioned above:

temb: [1, 1, 2, 1280]
encoder_hidden_states: [1, 2, 77, 768]
attention_mask: None
cross_attention_kwargs: None

Implementing TtCrossAttnDownBlock2D using ttnn for the Stable diffusion model is Completed.
Commit Link : Link
For the following cases we are getting a PCC < 0.16 and for other 4 test cases we are getting PCC =~ 0.8.
Shape 1: 256x256

hidden_states:  case 3: [2, 640, 8, 8]
PCC : 0.12622983116773254

Shape 2: 512x512

hidden_states case 3: [2, 640, 16, 16]
PCC: 0.167559617771942

PR has been created for the TtCrossAttnDownBlock2D sub-module .
For the following cases we are getting a PCC < 0.16 and for other 4 test cases we are getting PCC =~ 0.8.
Shape 1: 256x256
hidden_states: case 3: [2, 640, 8, 8]
PCC : 0.12622983116773254
Shape 2: 512x512
hidden_states case 3: [2, 640, 16, 16]
PCC: 0.167559617771942
Corresponding PR : #5086

PR is merged, closing the ticket.


Refactoring / defenitions of gaps

The gap and missing obstype modules are becoming too complex. It is time to refactor them. The idea is to merge the gaps and missing obstype together into the new gap$ class.
Gap attributes:

[ ] startdt, enddt (in original import freq)
[ ] obstype
[ ] station
[ ] gapdf (only the gap records!)
[ ] anchordf (only the observation records used, trailing/leading)

gapdf (and anchordf)  is a dataframe with datetime-name index (easier for merging), and the following columns:

"obstype.name" column is always filled with Nan's
"fill": the filled values
"fill method" : method for filling
"msg": error/info messages per record

The following issues must be resolved here

#431
#417
#179
coming requests/remarks by @amberJ99

obsolete (duplicate of #467)


Add flag: devspace enter --package / -p [PACKAGE]

After adding a package via devspace add package [PACKAGE], users should be able to start a terminal for the containers within the pods started by such a package. Therefore, we need a --package / -p flag for the devspace enter command. Using this flag will let the enter command select the deployment/statefulset etc. based on the labels and then select the pod created by the selected resource.
Label selection could should combine the most often used "chart" label name as well as "helm.sh/chart" as recommended in https://github.com/helm/helm/blob/master/docs/chart_best_practices/labels.md
If the number of identified pods != 1, the enter command should print a fatal error.

Closed in favor of #345


Modified the description to optimise marketplace search (#31)

modified description to optimise search

modified wording

added the AKS keyword

@N-Usha Can you approve this cherry-pick? We will need to update the release to see the changes in the marketplace.


Version 2.0.12 fails on Windows due to postinstall using bash syntax

Describe the bug
Bug was introduced in 6d946e1594d094dbdf6cfec331b6301e352da440, namely the line:
    "postinstall": "[ \"$npm_package_name\" = \"storybook-addon-react-router-v6\" ] && sh scripts/announcement.sh || :"

Environment
Share the output of the following command :
  System:
    OS: Windows 11 10.0.22631

Hi @ldufresnegs , thanks for reporting this.
It is now fixed, can you try v2.0.14 and tell me if it ok for you ?

Hi thanks a lot for looking into this.
I've tried again, and this time it complains for an other reason. Looking a bit more into it, it seems that the announcement.js file is missing.
Presumably the issue is here: https://github.com/JesusTheHun/storybook-addon-remix-react-router/blob/2fefd707269014366a535cab214325eecff433eb/package.json#L38-L43

@ldufresnegs thanks again for checking. 2.0.15 is out, with the said script ;)

It works, thanks a lot :)


SecurityInsights: Investigation swagger updates

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Can one of the admins verify this patch?

You can add the extra words to https://github.com/Azure/azure-rest-api-specs/blob/master/custom-words.txt if they're correct

@shahabhijeet please review, thanks :)

@shahabhijeet is there any update?

@shahabhijeet @praries880 can someone please review the PR? thanks

@MyronFanQiu go ahead merge the PR if you are satisfied with responses to your comments.

LGTM. I just want to confirm. Lot's of parameters set readOnly field to true. Usually, readOnly: true means these parameters should not appear in the PUT/PATCH methods. These parameters are set by design. Right?

@MyronFanQiu One of our resource types, called entities, supports only Get/Post operations in the server-side, and creation/modification does not allowed as part of the API. We have 15 different kinds of this resource, and they all share the same behavior. This is why you see many readOnly properties set to true.

@dosegal  Thanks!


Documentation concerning tsrc apply-manifest

Environment

Output of tsrc version: 2.5.0
Operating system: Linux 64 debian

Hi ! Close question to this topic https://github.com/dmerejkowsky/tsrc/issues/279
The documentation could be updated to clarify the following incertitude. I was questioning myself concerning https://dmerejkowsky.github.io/tsrc/guide/manifest/ with sections "Using the apply-manifest command to avoid breaking developers' workflow" and "Additional notes".
Is:
It is common to place the manifest repo itself in the manifest - so it's easy to edit or read

(described in Additional notes)
Compatible with this command:
cd /path/to/work # holding .tsrc refering to manifest/manifest.yml
tsrc apply-manifest manifest/manifest.yml

? (described in Using the apply-manifest command to avoid breaking developers' workflow
In other way: is the command tsrc apply-manifest works fine when the git repository holding the manifest is inside the workspace installed by tsrc init ?
The answer seems to be yes, but this is not obvious because manifest/manifest.yml is dirty iand it will update .tsrc/xxx/manifest.yml which will try to update manifest/manifest.yml (but since dirty tsrc will not try to update ... or failed).
I tried two things:

I added new git repos in the manifest/manifest.yml: they were added.
But today I wanted to remove branch: dev-foo and call tsrc apply-manifest manifest/manifest.yml no error but tsrc did not switch to the branch to master. Not sure why !

The workflow looks like this. Say you want to add a new repo, named bar and you already have:
repos:
  - url: git@github.com:dmerejkowsky/foo.git
    dest: foo

  - url: git@github.com:dmerejkowsky/dummy-manifest.git
    dest: manifest

Run tsrc init so that 'foo' and 'manifest' are cloned in their respective folders
Edit the file in<workspace>/manifest/manifest.yml to add the new repo:

  - url: git@github.com:dmerejkowsky/foo.git
    dest: foo
+
+  - url: git@github.com:dmerejkowsky/bar.git
+    dest: bar

Run tsrc apply-manifest <workspace/manifest/manifest.yml to make sure your changes are correct
Create a commit with your changes in the manifest repo
Push the commit into a branch in the manifest, say add-bar
Make someone review the changes
Merge the add-bar branch into the main branch of the manifest
Run tsrc sync.
The repo in .tsc/manifest is updated
Since 'bar' was already cloned when you ran apply-manifest no new repo is cloned
The repo in manifest/manifest is not updated because it's on the branch add-bar and it should be an master
You can then manually run cd <workspace>/manifest ; git checkout master

tsrc did not switch to the branch to master. Not sure why !

That's expected. See the relevant documentation

@dmerejkowsky thank you for confirming that we can call apply-manifest inside the workspace but not necessary have to be applied from a manifest outside the workspace. The documentation was unclear on this point and you should include your example to remove any confusion.
Sorry for my poor English. In fact, I wanted to say that I tried two different things:
First thing:
Inital workspace/manifest/manifest.yml:
repos:
  - url: git@github.com:dmerejkowsky/foo.git
    dest: foo

If I add a new repo bar:
repos:
  - url: git@github.com:dmerejkowsky/foo.git
    dest: foo

  - url: git@github.com:dmerejkowsky/bar.git
    dest: bar

The command tsrc apply-manifest workspace/manifest/manifest.yml will git clone the bar/ repo.
Ok so far so good :) but :
Second thing:
Inital workspace/manifest/manifest.yml. Initial repo (repo foo with branch dev-foo):
repos:
  - url: git@github.com:dmerejkowsky/foo.git
    branch: dev-foo
    dest: foo

Change the branch of repo foo to follow master:
repos:
  - url: git@github.com:dmerejkowsky/foo.git
    dest: foo

The foo repo is not dirty ! Even if I commit (or not) workspace/manifest/manifest.yml in local (your step 3.) when I call tsrc apply-manifest workspace/manifest/manifest.yml the branch is not switched. Do you consider that I'm on the case described by your documentation in case the repository is on an incorrect branch, the fast-forward merge will still be attempted, but an error message will be show in the end because tsrc does not print error (echo $? is 0) and there is no possible conflict because the repo is clean and the switch to the branch is possible (git checkout works). But it stays to the dev-foo branch.


feat: port Data.Matrix.Basic

UPDATE: I restarted the port using @eric-wieser 's modifications in leanprover-community/mathlib#18696
After adding some simp Lemmas it worked quite fine (see review comment below).
Also, #3024 addresses some problems marked with porting note for now.

[x] depends on: leanprover-community/mathlib#18696

I discussed this privately with @j-loreaux yesterday. I think we need to do a mathlib3 refactor before porting matrices to avoid downstream pain. Here's what I said there:
I think it will end up making a mess downstream if we don't either change simp, or refactor in mathlib3 first

For instance, right now transpose is really hard to unfold in lean3 unless applied, which is a good thing. But lean4 will happily unfold it to a lambda, and now you've lost the information that it's a matrix
So the refactor would be:

Redefine as transpose (A) := of (λ i j, A j i) so that if it is unfolded the type is still correct, and remove simp
Add a transpose_apply lemma that is simp

I think the current design is clever though, and would prefer to keep it if we expect lean to change in the next few months. I think it's widely viewed as a bug.
docs#matrix.of has a reasonable summary of how things currently are

Having said that, it sounds like you found some other things we might want to worry about at the same time!

it seems that mathlib3 ext i j doesn't quite work.

Asked about this on Zulip

I discussed this privately with @j-loreaux yesterday. I think we need to do a mathlib3 refactor before porting matrices to avoid downstream pain. Here's what I said there:
I think it will end up making a mess downstream if we don't either change simp, or refactor in mathlib3 first

For instance, right now transpose is really hard to unfold in lean3 unless applied, which is a good thing. But lean4 will happily unfold it to a lambda, and now you've lost the information that it's a matrix
So the refactor would be:

Redefine as transpose (A) := of (λ i j, A j i) so that if it is unfolded the type is still correct, and remove simp
Add a transpose_apply lemma that is simp

I think the current design is clever though, and would prefer to keep it if we expect lean to change in the next few months. I think it's widely viewed as a bug.
docs#matrix.of has a reasonable summary of how things currently are

Having said that, it sounds like you found some other things we might want to worry about at the same time!

Yea I just wanted to get an idea what kind of issues you run into. Thinking about how to fix them instead of working around them sounds very reasonable👍

This PR/issue depends on:

leanprover-community/mathlib4#3024
By Dependent Issues (🤖). Happy coding!

This PR/issue depends on:

leanprover-community/mathlib#18696
By Dependent Issues (🤖). Happy coding!

I think you'll still need your ext workaround from the previous iteration

I think you'll still need your ext workaround from the previous iteration

Oh I see, it compiled locally because apparently I had the changes in ext from std4#111 applied inside my lake-packages/std🤦

I think my only remaining comments are:
* I preferred the `ext` hack you had before that introduced a weird `ext` lemma rather than making all the proofs use `rename_i`

* The porting comments would be clearer if they appeared at the relevant point mid-proof rather than at the end.

done

bors r+


Change BGR to RGB in docstrings

Docstrings indicate that it expects BGR when it expects RGB.

No, it expects how opencv loads which is BGR.


Dag tracking

[Short description explaining the high-level reason for the pull request]
Changes
How I tested this
Notes
Checklist

[ ] PR has an informative and human-readable title (this will be pulled into the release notes)
[ ] Changes are limited to a single goal (no scope creep)
[ ] Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
[ ] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future TODOs are captured in comments
[ ] Project documentation has been updated if adding/changing functionality.

Moving this to separate Repo as this isn't related to the core of hamilton


fix error analysis for RAITextInsights displaying all incorrect instances for text labels

Description
On the DBPedia dataset, fix error analysis for RAITextInsights displaying all incorrect instances for text labels:

Note this is only happening for a particular multiclass text classification case where the predicted label is a numpy int type.
Checklist

[x] I have added screenshots above for all UI changes.
[x] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2325 (3043de9) into main (61c70d0) will decrease coverage by 13.99%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #2325       +/-   ##
===========================================
- Coverage   88.42%   74.43%   -13.99%
===========================================
  Files         133       14      -119
  Lines        7575     1240     -6335
===========================================
- Hits         6698      923     -5775
+ Misses        877      317      -560

Flag
Coverage Δ

unittests
74.43% <ø> (-13.99%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 120 files with indirect coverage changes
:loudspeaker: Have feedback on the report? Share it here.


[Testing] Can not run integration-test command use release version rooch

cargo install --path crates/rooch
Run
rooch move integration-test --named-addresses rooch_examples=0x42

at examples/counter
error[E03002]: unbound module
  ┌─ /Users/jolestar/opensource/src/github.com/rooch-network/rooch/examples/counter/sources/counter.move:2:8
  │
2 │    use moveos_std::account_storage;
  │        ^^^^^^^^^^^^^^^^^^^^^^^^^^^ Invalid 'use'. Unbound module: '(moveos_std=0x2)::account_storage'

Seam like the integration-test does not load the project's dependency.
@shelmesky @pause125

After some local testing, it does appear that the integration-test is missing certain dependencies.


Consider moving Asp.Net Identity into the separate projects

I'd like to create the project template via dotnet new that doesn't include the part with AspNet Identity therefore I'd like to move Asp.Net Identity into the separate projects.
BusinessLogic:

Skoruba.IdentityServer4.Admin.BusinessLogic
Skoruba.IdentityServer4.Admin.BusinessLogic.AspNetIdentity

EntityFramework:

Skoruba.IdentityServer4.Admin.EntityFramework
Skoruba.IdentityServer4.Admin.EntityFramework.AspNetIdentity

There is need to separate AdminDbContext to AspnetIdentityDb, ConfigurationDb, OperationalStorageDb.
In our solution we aim to keep them separately in different databases to backup only configuration db. When using AdminUI component AdminDbContext is forces to use single Db for all 3 contexts.

Done on master - #99


Error refactor, part 2: Remove Error enum.

Finishes work started on #617, see there for motivation.
Depends on #726

bors r+


Make app free for dev stores

I have created a paid plan in db with 7 days trial.
On dev store, after completing 7 days trial it redirects to billing and shows popup to upgrade plan.
Please help me in order to make free app for all dev stores.
Plan values are:
id => 1
type => 1
name -> Basic
price => 0.99
capped_amount => NULL
terms => Basic terms
trial_days => 7
test => 1
on_install =>1

You can use the AfterAuthenticateJob to check the store plan type and then set the grandfathered column in the shop's table to true.
This would then mean all dev stores are "free". There is a list somewhere (can't find it atm) where it tells you all the different shop types.

Hi @faisal-saeed-brainx
@Kyon147  is correct, a contributor made an article several months ago detailing how to do this.
https://github.com/ohmybrew/laravel-shopify/wiki/Free-Access-Based-on-Shopify-Plan
Given this is not a bug or issue I am currently closing this, if you feel there is a bug, please reopen. Thank you.

Hi @ohmybrew @Kyon147
Thanks for your support. I have applied this approach and it is  working fine.
I have to discuss 1 test scenario with you, When we installed the app on a dev store, the AfterAuthenticateJob updates the grandfathered value as true. then after that each time it skips the api call due to grandfatherd value is true. After that user upgrades the store plan to other paid plan.
My question is for above scenario:
Now the store plan has been changed on the shopify_plan, on which it should be charge. But the grandfathered column is still true and will work as free.


rx.input(value=State.nested.var) is not updated when nested is set to a new object containing the same content.

Describe the bug
We have some validation checks in setters that do not let through some changes from the frontend. Unfortunately the reload button does not reset the UI.
To Reproduce
Steps to reproduce the behavior:

Use below app
Type an "a" into the input
Because your input is invalid, the state is not updated in the setter.
Click "Create new Object". This will overwrite the object in the state.  # Here I would expect an update in the input.
The input still shows the old value, containing the "a".
Click "Create other Object". This time the input changes to "12345"

import reflex as rx

class Object(rx.Base):
    name: str = "1234"

class State(rx.State):
    """The app state."""
    message: str = "Default"
    object: Object = Object()

    def create_new_object(self):
        self.object = Object()
        self.message = self.object.name
        print(self.object.name)

    def create_new_object_2(self):
        self.object = Object(name="12345")
        self.message = self.object.name
        print(self.object.name)

    def set_name(self, value: str):
        if not "a" in value:
            self.object.name = value
            self.message = ""
        else:
            self.message="You can't use the letter 'a' in the name"

def index() -> rx.Component:
    return rx.container(
        rx.text(f"Your message: ", State.message),
        rx.text(f"Object name: ", State.object.name),
        rx.input(value=State.object.name, on_change=State.set_name),
        rx.button("Create new object", on_click=State.create_new_object),
        rx.button("Create other object", on_click=State.create_new_object_2),
    )

app = rx.App()
app.add_page(index)

Expected behavior
The input should show the variable value after pressing the button.

I believe the issue is https://github.com/nkbt/react-debounce-input/issues/130
The internal state of the debounce is not getting updated because the bound state var does not actually change value [1].
I'm not sure if there's actually a fix we can make here 😔, but there are some potential workarounds:

set key prop on the rx.input to some value in the state that you increment/update when the object updates. This will force the input to be re-rendered with the correct value showing... Note however, reflex 0.5.6 currently has a bug where key is not passed through, so this would also need to be fixed in the framework before it's a real solution
trigger a state update in self.object.name to a different value, then set it back to the original value in the set_name function. The problem here is that the user gets their input swiped out from under them and reset to the last "valid" value, which is not a great UX.
in the create_new_object handler, if the new object name matches the current object name, first yield a state update that blanks out the self.object.name field, then update self.object to the new object. This is probably the cleanest workaround, but if these objects are getting created and assigned in multiple places, it's kind of a hassle, so you might need a helper function to handle the logic.

Thanks for the fast reply @masenf .
We have implemented the last workaround and yield the entire object as None and yield the new value afterwards. It causes a tiny flicker depending on the state size, but at least it works now as expected.

We could try https://github.com/xnimorz/use-debounce which seems to be better maintained


Please upgrade libvirt

Vendored libvirt wasn't updated for a while. Please update to latest.

Or at least backport the RNG device without which it's impossible to use libvirt efficiently.


Facets should be case insensitive

Having case sensitive sorting of facets may not be the best way to do it.
Also how to handle a clever reordering of names yielding meaning

Change the solr facet result to be white space, punctuation and case agnostic.
"Patricia  Hswe" ~= "Patricia Hswe"
"PATRICIA HSWE" ~= "Patricia Hswe"


Ability to configure cookies using a json object

Hi Peter,
I have a use case where I want to add cookies to the API being invoked just like how we can add headers using custom javascript function.
I see that its not possible right now to do it. (Please correct me if I'm wrong)
I also have an idea around the code change to make it happen.
Tell me what you think about the below capability and I can raise a pull request with the changes
configure cookies = read('classpath:custom-cookies.js')

perhaps the new cookies keyword that takes JSON may solve for your use-case: https://github.com/intuit/karate#cookies-json - and the JSON could also come from a custom function.
also cookies returned by the server are retained and auto-sent for all subsequent requests, in case that helps.
feel free to re-open if needed

I wanted specifically a js function to return the map which can be used to set the cookies. So that my feature file definition looks clean. I have to use an object from the previous step and do some extraction to set the cookies.  Let me know if that makes sense.

got it. a few points:

I'm trying to reduce the amount of syntax one has to learn as far as possible ! hence the push-back. hope that makes sense !
configure headers clearly makes sense, this is something almost all projects need to do. this is a rare area where karate has an 'overlap' both configure headers, header and headers do the same thing kinda
do you really have a situation where every request needs to dynamically set cookies based on the previous response ?
if not, what I was saying is that you can define a function that uses the current 'state' to form cookies, yes you may need to call this multiple times in a test, but I would lean towards this instead of creating a new keyword.

for example:
* def cookieCutter =
"""
function() {
  def c = karate.get('cookies');
  def temp = <some json after your custom logic>;
  return temp;
}
* cookies cookieCutter()
"""

but I'm open to be convinced otherwise - and will be happy to merge a pull request !


Fix arity error wording for a keyword

It looks like in Clojure 1.11 there is no longer a special case for arity exception on a keyword:
babel.middleware=> (:a 1 2 3 4)
The function :a cannot be called with four arguments.

In Clojure interactive session on line 1.
Call sequence:
[Clojure interactive session (repl)]

babel.middleware=> *e
#error {
 :cause "Wrong number of args (4) passed to: :a"
 :via
 [{:type clojure.lang.ArityException
   :message "Wrong number of args (4) passed to: :a"
   :at [clojure.lang.Keyword throwArity "Keyword.java" 108]}]

Add a special case for a function that's a keyword.

This now corresponds to https://github.com/Clojure-Intro-Course/babel/issues/58 in the migration repository.


KNOX-2155 - KnoxSSO should handle multiple cookies with the same name

This commit moves getting cookies by name to a
new utility class. It forces callers to look
through multiple cookies returned and handle
that case.

Merged in 1b019617d408312ac33bf8f07f0ec862f7b33382


Environment based loading of route.js file

Lets load non-minified file only when APP_ENV = local otherwise load route.min.js.

Thanks for all 3 PRs man! Things are looking good!

Guys... We should get this documented.

Good idea, lemme write it up.


For the latest Windows .zip version(s), should I be replacing the entire folder or just overwriting the existing files?

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2022.06.29 (update instructions) or later (specify commit)
[X] I've searched the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Instead of using the compiled .exe version, I normally use the .zip version and extract it to a folder. Should I delete the old contents of the folder, or is it ok (or preferred) to just overwrite the files that are contained in the .zip version.
Thanks!
Bobby G.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Don't think it matters, but if you want to be on the safe side, delete everything first


Adaptive lighting: 'Characteristic Value Transition Control': characteristic was supplied illegal value: undefined

With homebridge deconz and deCONZ 2.14.01, when adaptive lighting is enabled, I pick up a bunch of errors from a set of lights that all follow the same pattern.
Homebridge Deconz is running as a child bridge.
Bulbs are different manufacturers (Tuya and Ikea at least two of them) - might possibly relate to 'special effects' options on them?
[07/03/2022, 18:23:50] [homebridge-deconz] This plugin generated a warning from the characteristic 'Characteristic Value Transition Control': characteristic was supplied illegal value: undefined. See https://git.io/JtMGR for more info.
[07/03/2022, 18:23:50] [homebridge-deconz] Muservatory Drums Light: Adaptive Lighting: disabled
[07/03/2022, 18:23:50] [homebridge-deconz] Muservatory Drums Light: error: TypeError: Cannot read properties of undefined (reading 'generateControl')
at CharacteristicDelegate.getter [as _getter] (/usr/lib/node_modules/homebridge-deconz/lib/DeconzService/Light.js:209:40)
at CharacteristicDelegate._onGet (/usr/lib/node_modules/homebridge-deconz/node_modules/homebridge-lib/lib/CharacteristicDelegate.js:354:26)
at CharacteristicValueTransitionControl.emit (node:events:520:28)
at /usr/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1592:14
at new Promise ()
at CharacteristicValueTransitionControl. (/usr/lib/node_modules/homebridge/node_modules/hap-nodejs/src/lib/Characteristic.ts:1590:12)
at step (/usr/lib/node_modules/homebridge/node_modules/tslib/tslib.js:143:27)
at Object.next (/usr/lib/node_modules/homebridge/node_modules/tslib/tslib.js:124:57)
at /usr/lib/node_modules/homebridge/node_modules/tslib/tslib.js:117:75
at new Promise ()

What version of Homebridge deCONZ are you on?  I think I saw something similar in v0.0.11, but that should have been fixed in v0.0.12.  Could you double-check if the problem still occurs in v0.0.14?  Also for a light that was un-exposed and re-exposed after updating to the latest Homebridge deCONZ version.  There might be some residual junk from an old version in cachedAccessories, even though there shouldn't be.


removed from the reference node toc, external links

@TomArcher,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-services/activedirectory.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

sign-off