[Feature Request] getChanges() not supported

Call to a member function getChanges() on int
 $update = User::where('_id', $request->userId)->update($validatedData);
 return $update->getChanges();

La function Illuminate\Database\Query\Builder::update() returns an int in Laravel (see the code).
The method getChanges() exists and is correctly updated when called on a Model instance.
$user = User::create(['name' => 'John Doe']);

$updated = User::where('name', 'John Doe')->update(['name' => 'Jane Doe']);
var_dump($updated); // 1

$updated = $user->update(['name' => 'Jimmy Doe']);
var_dump($updated); // 1
var_dump($user->getChanges()); // array(name => Jimmy Doe, updated_at => UTCDateTime)

Call to a member function getChanges() on int
 $update = User::where('_id', $request->userId)->update($validatedData);
 return $update->getChanges();

I dont understand why this doesnt work, do I need to do first() then update?
With your example you
why not
$updated = User::where('name', 'John Doe')->update(['name' => 'Jimmy Doe']);
var_dump($updated); // 1
var_dump($updated->getChanges());

The update method returns an int, this will not change and we can't add a method getChanges on an int.
Yes, you must get the model instance first.
$user = User::where('name', 'John Doe')->first();
$updated = $user->update(['name' => 'Jimmy Doe']);
var_dump($updated); // 1
var_dump($user->getChanges());

The update method returns an int, this will not change and we can't add a method getChanges on an int.
Yes, you must get the model instance first.
$user = User::where('name', 'John Doe')->first();
$updated = $user->update(['name' => 'Jimmy Doe']);
var_dump($updated); // 1
var_dump($user->getChanges());

Thanks


Changing the dataset

Hi there!
I hope you are doing well.
I was trying to understand the data acquisition pipeline of the project here. I am a student and for me it is not possible to download the whole kitti dataset at one go(limited data in one day) and hence the download_kitti.py ends up with broken data and I get the error of none type has no attribute shape. Is it possible to somehow train the network on a smaller part of the dataset?
I know it might affect the performance of the network but at the end I am doing this as a personal project to learn working with pytorch and depth estimation.
If it is possible to change the size of train test val dataset please guide me how can I do it.
Many thanks in advance!
Regards,
Bansi

Hi there, I think in this case, it is pretty hard to change the sets, because they are pretty fixed defined in the dataloader.
However, if I understand correctly, the only problem is the amount of data downlaoded. I think I would recommend to download and extract the raw data from kitti manually (or in smaller portions). Afterwards, it should be possible to apply the rest of the download_kitti.py script and to run the code as intended.
Regards,
Marvin


Admin rights

The 64bit Windows download requires admin rights to install, even though the documentation says it doesn't require admin rights for a user to install it into their own context. They do. Version 1.2.23220.
[Enter feedback here]

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

ID: 09ab98fc-fea2-a11a-e5ca-de1430211d97
Version Independent ID: 4d9943c3-fef7-ca86-bd03-1241a04b8135
Content: Get started with the Windows Desktop client
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/windowsdesktop.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @Heidilohr
Microsoft Alias: helohr

When will this be fixed.  This client is updated constantly.  We need a way to install in user context via INTUNE. The switches that are supposed to do that do not work and it installs in admin context regardless.

Closing the loop on this - this is fixed in the client and it can be installed without admin rights.
#please-close


Update dependencies

/cc @DustinCampbell

Change looks good, though it failed to get some of the packages.

Lgtm once ci passed 😃

investigating the build errors. may be some packages are still being shifted.

fingers crossed!

My mistake, duplicate feed names causes one feed ignored.

fixing a new build error now. some breaking api change in hosting.

it was fixed on dev branch. vscode branch is actually behind.

Cool!

oh, no. the test will fail because a mismatch in dotnet-test-xunit on aspnetcirelease and cli. working on a solution.

@hal-ler something's wrong with cake script build on mac environment:
Successfully installed 'Cake 0.11.0' to /Users/travis/build/OmniSharp/omnisharp-roslyn/.tools
Analyzing build script...
Processing build script...
Installing addins...
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated

@david-driscoll We saw similar behavior earlier as NuGet getting into an infinite loop on OSX. It was fixed, by pinning the NuGet binary to v3.3.0. Did something else also change, like the Mono version?

Can we I merge this without Mac testing?

I'll merge to the vscode branch anyway. we should keep looking into the mac build environment.

Oh you were behind dev? Then pull in https://github.com/OmniSharp/omnisharp-roslyn/commit/bba128a507ca4587cf6c0b411ce3d0fe528daa3a. That fixes the issue.

Yeah, it looks like that change isn't there. I'll cherry-pick it.


Add cook to calendar export

Originally created by Tom Smyth at 2016-10-03 13:50, migrated from redmine-#5244

Version changed from 2.4 to 2.5

By Tom Smyth at 2016-10-24 16:30

Version set to 2.4

By Tom Smyth at 2016-10-23 03:17

Status changed from New to Closed

By Tom Smyth at 2016-10-26 16:31


Flagged as "unsafe" by Symantec Endpoint Protection

Hi there,
My work laptop has Symantec Endpoint Protection installed, it's part of internal security policies, I cannot uninstall it. Unforunately it's flagging wslgit.exe as "unsafe" every time I use it. I tried to add an exception but it's not working, it popups the following every time:

Is there anything that can be done on your side to prevent this from happening?

Similar false positives were reported in #7 and #35 . They were resolved by filing false positive reports for the respective software (including a link to the Github issue).
I just reported wslgit as a false positive on Symantec's site, so hopefully this will be handled soon.

I just got a response from Symantec that wslgit.exe will not be flagged anymore (after a database update within the next 24 hours).
Can you please confirm that it works now and that this issue can be closed?

I don't think the database on my system is yet updated, give me a day or two and I'll report back, sounds good?

@andy-5 It's no longer flagged, closing this. Thanks :)

I'm having this issue again... :/

Which binary is affected? The v0.8.0 release or the v0.9.0 alpha build?

Which binary is affected? The v0.8.0 release or the v0.9.0 alpha build?

All of them.

I'm getting this again all the time:

Isn't there a way to sign this file with some sort of Microsoft signature that would make Symantec ignore them or something?

FYI, one can submit false positives at https://submit.symantec.com/false_positive/
I've now submitted the v0.8.0 release for checking.
I'm not sure if signing the executable would help. It may increase the reputation a little bit, but Symantec seems to report it because the file is very new and it hasn't seen it very often.

FYI, one can submit false positives at https://submit.symantec.com/false_positive/
I've now submitted the v0.8.0 release for checking.

Good to know about this :)
However, v0.8.0 was not flagged for me, that one was fine. The problematic ones were the v0.9.0 alphas.
I've decided to compile my own binary and use that instead. Trying to see if it helps and avoids flagging by Symantec, we'll see how it goes.

Closing this (again) since there's nothing else that can be done to improve this. Thanks @andy-5 .


[CARRYOVER] Return pipeline errors to user by raising exceptions from…

… the client

/azp run Azure.azure-iot-sdk-python

/azp run Azure.azure-iot-sdk-python


Cannot get DependencyPropertyDescriptor

I'm using Bindables.Fody 0.4.0 with Fody 2.1.2, decided to upgrade to 4.0 today.
To get a Bindables DP at runtime, I used to get it like this :
DependencyPropertyDescriptor dependencyPropertyDescriptor =
    DependencyPropertyDescriptor.FromName(
        nameof(Test),
        typeof(MainWindow),
        typeof(MainWindow));

It worked well before, but now it always return null. Maybe your code is not to blame as I can reproduce this behavior when leaving Bindables.Fody at 0.4 and updating Fody to 3.0.
Do you know why this happens?
Any other way to get a reference to the DP at runtime?
I've tried the code from a MSDN post and it results in the same behavior.
DependencyProperty dp = (DependencyProperty)typeof(MainWindow).GetField("TestProperty").GetValue(this);

Here's a sample project with Bindables.Fody 0.4 and Fody 2.0, it runs correctly, updating either Bindables or Fody will result in a custom exception being thrown.
BindablesDependencyPropertyDescriptor.zip

Have you tried to match the Fody version? These are the Fody versions required by each version:
 - 0.3.* -> Fody [1, 2)
 - 0.4.* -> Fody [2, 3)
 - 0.5.* -> Fody [3, 4)
 - 4.*   -> Fody [4, 5)

If it does not solve the problem you can check the FodyWeavers.xml file. There should be a Bindables entry in that file.

You're absolutely right, by trying to isolate things in a dummy project, I forgot to include Bindables in FodyWeavers, everything works as expected with Bindables 4.x and Fody 4.x.
Thanks for pointing me to the obvious!


Build: switch to webpack

Switching to webpack, it allows us to use ES6 import in source file.

I'm closing now since I think current build steps work fine.


Support Swift 3 and Xcode 8.

Migrated the project to support Swift 3 and Xcode 8.
This pull request consists of

https://github.com/yoichitgy/SwinjectPropertyLoader/pull/1 by @gasthana0
Some cherry-picked commits of #3

Need to fix .travis.yml.

Thanks @jakubvano, I guess I added the AnyObject check by my misunderstanding to use an objc associated object. I'll check.

Removed the commented out code by be48d02 because the type check was just misunderstanding about AnyObject, NSObject and requirement of objc_get/setAssociatedObject.


[DO NOT MERGE] Add sync checker

Our existing migration sync checks don't support verifying correct unpublishing behaviour. Unfortunately, due to the way unpublishing works we need to check both content stores and have the document rather than the edition(s) available for the checks, so... back to the drawing board.
This is a new sync check framework that will allow us to compose new checks from some basic 'building blocks' (e.g. TopLevelCheck, DetailsCheck). It also abstracts away the unpublishing behaviour so as we don't need to consider it again for every Edition based format that we migrate.
Non-edition based formats can be supported too once we extract a common interface for a 'check' from DetailedGuideCheck

👍 for the bits I didn't do

@steventux short review please :)


Is it possible to execute javascript using WinAppDriver?

When I try to execute Javascript I get an exception that the Command is not implemented.  I am trying to do something like below.
How can I execute JavaScript ?
String script = "";
JavascriptExecutor js = (JavascriptExecutor) driver;
js.executeScript(script);

Yes its not supported


Support promise, async/await for some important actions like sign, verify jwt.

Describe the problem you'd like to have solved

I usually use sign, verify function for my authentication process by synchronous syntax. I don't know the process is heavy or not but better to use asynchronous for it.

In documentation, I see we have callback syntax but it is not popular on modern code these days.

Duplicate of #111


ImportError: cannot import name

Hi, I am new to pyflux and as of yet I havnt been able use it. How can I fix ImportErrors: cannot import name __check_build; ImportError: cannot import name families?

Taking a look at this today!

Need more information to resolve this issue; which version of Python are you using? How are you installing?


Media Library: added global media permissions

New global media permissions:

ViewMediaContent
ImportMediaContent
EditMediaContent
DeleteMediaContent

All these permissions imply ManageOwnMedia and they refer to all media folders.
See issue #7620 .

Hi, any news on this PR?

I added it to the topics for tomorrow's meeting. I need confirmation from others.

Sorry, I forgot to update the permission properties. Now they should be consistent with the new name.


Set Min Launch

Set a lower bound such that player does not get super speed.
Also include upper bound if possible


chore(data/equiv): use has_coe_to_fun

To-function coercion now works much more reliably.  I'm not sure if this makes a big difference, but at least we can save two notations.

Thanks!


Kafka: include Kafka broker hook to log connection error reasons

Reason for this PR
The current metricsHook struct that collects metrics based on Kafka hooks is not equipped to also report details of why connection errors are caused.
Details
We provide a new struct that implements a (single, for now) kgo Hook dedicated to logging errors during the broker connection phase.
The struct can be iterated in the future to implement other monitoring hooks, providing logging utilities where the metrics would have too high cardinality.
Additional context
Currently, the proposed design adds the new logger hook behind the CommonConfig.DisableTelemetry flag (similarly to the metrics hook).
This can be improved and it is not a perfect solution, but it has to be done this way until a bigger refactor is done to expose the CommonConfig.hook fieldin some way.
The more idiomatic, but also more expensive, solution would be to allow exporting franz-go logs into the Logger.
This is mentioned in https://github.com/elastic/apm-queue/issues/540

Unpleasant discovery: the newly added test is currently flaky because of franz-go implcitly retrying to reconnect at least once, failure here.
https://github.com/elastic/apm-queue/actions/runs/10324596296/job/28584375960?pr=541#step:4:8413
I think we should disable the retry to make the test consistent but I am unsure how to pass that option to the client 🤔

I think we should disable the retry to make the test consistent but I am unsure how to pass that option to the client 🤔

Not seeing an option to control the connection retry, I ended up changing the assertion: we now validate that there is either a single or 2 log lines containing the error


Future: use @testsetup to reduce time to run tests on GPU

Feature Request
For the Buildkite CI set up in #411 , we are currently calling Pkg.add("") in each test item for whichever GPU backend we are testing. This is to avoid having all four backends as a dependency in test/Project.toml, and the Pkg.add() step must be done in each test because of the way TestItemRunner.jl runs test items in separate modules.
However, calling Pkg.add() each time a test item is run introduces overhead that could possibly be avoided. It looks like there is a @testsetup macro under development (https://github.com/julia-vscode/TestItemRunner.jl/issues/71) which would allow only adding the packages once, so we should look into using this once it is complete.

Update: The previous approach of calling Pkg.add() in the middle of the tests led to some version dependency issues. We are now doing something closer to DiffEqGPUs approach here: https://github.com/SciML/DiffEqGPU.jl/blob/master/.buildkite/runtests.yml, so the overhead mentioned above with running the tests is no longer an issue.


Fix #3952

Coverage decreased (-0.96%) to 86.59% when pulling 08b4b60675a7b81309e5b990fe455265e9963133 on yakky:feature/fix_3952 into 0055edadb481233eef180aba22a5fec875817d45 on divio:support/3.0.x.


how to update labels?

Hi nick,
I followed up this issue https://github.com/nnnick/Chart.js/issues/13 which at the end of it you have added a fantastic solution to be able to update the data series and then update the chart.
However I can't figure out how can I update the labels if the length of the data series and label values are different from the original dataset?
Many thanks

+1 how to update labels

I've now just changed my own use to just replace the chart with a fresh one, instead of trying to animate and append updates.

+1 Does anyone have a workaround that doesn't include redrawing the entire chart? I can update the chart.datasets[0].bars[0].label value, but the canvas keeps the old label even after calling update().

I have a workaround, but it's cumbersome. We're working on a nice PR that will likely fix any issues anyone is having with updating an existing chart and likewise animating those changes.

@tannerlinsley Thanks for the update. I've implemented a workaround that removes the animation only when I need updated labels so that users don't see the re-animation when I redraw the chart.

Still can not update the labels?
Looks like have to redraw the whole thing.

Can someone post a simple workaround for this issue?

Doesn't my solution posted on Oct 2 work?

@rocketblr
Thanks, missed that and it indeed seems to work!
Are there any gotchas when updating scale.xLabels and scale.yLabels this way?

Be careful when you need to change number of labels. In that case you should combine updating labels with adding or removing data to chart.

Ah, ok. I'll assume I need to call .addData() or .removeData() as many times as necessary to get the right number of labels. Would also be nice if there was a cleaner way to do this, but that works for now.

+1

+1. Any updates on this?

Have you tried using 2.0 alpha?

Hi Tanner,
Can we use Alpha in production? If not, do you have an ETA for production ready version?
Thanks.

Alpha is not production ready.  It is coming as soon as possible though.  I can’t give an ETA :( But it’s going to be amazing!

We just want to assess as many issues in 2.0 as possible.  So we are asking that everyone try their luck at it and help us with feedback.

As @rocketblr has explain in his Oct. 2nd post how to do this, and it is much more easily supported in 2.0, we're closing this issue for now. Thanks for all of the feedback and be sure to help us get 2.0 ready for release!

@tannerlinsley could you provide a link to this October 2nd post? Is it in a PR or on a blog of some kind?

Incase anyone is wondering how to do this in 2.0, this works for me:
lineChart.config.data.labels = newLabels
lineChart.update()

I have the same problem. I am creating a chart by dynamically assigning value to both labels and dataset. But at certain point, label are not showing. label data is generating but not showing.

myChart.data.labels = intervals;
myChart.update();

working for me. Besides i'm using chart with React.js and pan/zoom plugin shown below.
this.state.myChart.data.labels = intervals;
this.state.myChart.update();
this.state.myChart.resetZoom();


Upgrade to latest patch editions of libraries

We're a bit behind on a few that don't otherwise require code-changes on our end and they generally address some security or critical bug fixes.

That should be done, right?


Support CNI Overlay

Hi,
In current version of this IaC, the networking models Kubnet and Azure CNI is supported.
Microsoft do not recommend to use Kubnet in production environment and do not support Kubnet in new functionality like "Application gateway for containers". Azure CNI requires a lot of IPs (that cause ip exhaustion) and is not suited to connect with OnPrem (expressroute).
Since we need to connect our AKS, with viya-4, to OnPrem I would like to request support for "CNI Overlay"

https://azure.microsoft.com/en-us/updates/azurecnioverlay/

  network_profile {
    network_plugin = "azure"
    network_plugin_mode = "overlay"
    ebpf_data_plane = "cilium"                                // Optional
    outbound_type = "userDefinedRouting"
  }

CNI Overlay exist in two versions 'Azure CNI Overlay' and 'Azure CNI Overlay Powered by Cilium'

@runemy, We had explored Azure CNI Overlay support early this year in May and it resulted in intermittent failures, networking and DNS issues. That led us to backlog the work around this. It's been a while that this feature is live, hoping it performs better now. This work is planned to be reinvestigated early next year.

Changes for Azure CNI Overlay have been released and available in the main branch.


[tools/replay] Replay tool SIGSEV on simple vkcube with AMDGPU/Mesa

Hi,
Thanks for your work, it's appreciated!
I'm trying to capture a trace for a program that can crash the amdgpu driver.
I tested the GFXReconstruct Capture Layer on VKCUBE first and though the capture file was created, I cannot replay it:
System Information
[...]
     Graphics:
       Device-1: AMD Navi 10 [Radeon RX 5600 OEM/5600 XT / 5700/5700 XT]
       vendor: Sapphire Limited driver: amdgpu v: kernel bus ID: 0c:00.0
       Display: x11 server: X.Org 1.20.8 driver: amdgpu
       resolution: 2560x1440~144Hz
       OpenGL: renderer: AMD Radeon RX 5700 XT (NAVI10 DRM 3.36.0
       5.6.11-zen1-1-zen LLVM 10.0.0)
       v: 4.6 Mesa 20.0.6 direct render: Yes

[...]

Segfault Trace
(gdb) run gfxrecon_capture_20200510T160914.gfxr
Starting program: /home/_USER_/dev/gaming/gfxreconstruct/_build/tools/replay/gfxrecon-replay gfxrecon_capture_20200510T160914.gfxr
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/usr/lib/libthread_db.so.1".

Program received signal SIGSEGV, Segmentation fault.
0x0000000000000000 in ?? ()
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x00007ffff797c7da in ?? () from /usr/lib/libvulkan.so
#2  0x00007ffff795a8ab in ?? () from /usr/lib/libvulkan.so
#3  0x00007ffff796ffee in ?? () from /usr/lib/libvulkan.so
#4  0x00007ffff7976962 in vkCreateInstance () from /usr/lib/libvulkan.so
#5  0x0000555555b1041c in gfxrecon::decode::VulkanReplayConsumerBase::OverrideCreateInstance (this=0x7fffffffd110, original_result=VK_SUCCESS, pCreateInfo=0x555555f0c7c0, pAllocator=0x0, original_instance=..., pInstance=0x7fffffffcb48)
    at ../framework/decode/vulkan_replay_consumer_base.cpp:1439
#6  0x0000555555bf2b97 in gfxrecon::decode::VulkanReplayConsumer::Process_vkCreateInstance (this=0x7fffffffd110, returnValue=VK_SUCCESS, pCreateInfo=..., pAllocator=..., pInstance=...) at ../framework/generated/generated_vulkan_replay_consumer.cpp:44
#7  0x0000555555b6a182 in gfxrecon::decode::VulkanDecoder::Decode_vkCreateInstance (this=0x7fffffffcf00, parameter_buffer=0x555555f0e1a0 <incomplete sequence \342>, buffer_size=1025) at ../framework/generated/generated_vulkan_decoder.cpp:57
#8  0x0000555555b997f0 in gfxrecon::decode::VulkanDecoder::DecodeFunctionCall (this=0x7fffffffcf00, call_id=gfxrecon::format::ApiCall_vkCreateInstance, call_info=..., parameter_buffer=0x555555f0e1a0 <incomplete sequence \342>, buffer_size=1025)
    at ../framework/generated/generated_vulkan_decoder.cpp:8048
#9  0x0000555555afae2b in gfxrecon::decode::FileProcessor::ProcessFunctionCall (this=0x7fffffffd050, block_header=..., call_id=gfxrecon::format::ApiCall_vkCreateInstance) at ../framework/decode/file_processor.cpp:409
#10 0x0000555555afa557 in gfxrecon::decode::FileProcessor::ProcessBlocks (this=0x7fffffffd050) at ../framework/decode/file_processor.cpp:193
#11 0x0000555555afa181 in gfxrecon::decode::FileProcessor::ProcessNextFrame (this=0x7fffffffd050) at ../framework/decode/file_processor.cpp:86
#12 0x0000555555ad94e9 in gfxrecon::application::Application::PlaySingleFrame (this=0x555555f0dbe0) at ../framework/application/application.cpp:88
#13 0x0000555555ad9415 in gfxrecon::application::Application::Run (this=0x555555f0dbe0) at ../framework/application/application.cpp:62
#14 0x0000555555ac9a8f in main (argc=2, argv=0x7fffffffdf08) at ../tools/replay/desktop_main.cpp:138

Could someone help me figure out how to solve this so I can submit a bug report?

From the stack trace, the failure looks to be happening at the first Vulkan call, but I can't tell exactly why.
Could you answer the following?

Was the capture layer enabled during replay?
When capturing vkcube, was an environment variable such as VK_ICD_FILENAMES set to specify the path to a driver?  If so, was the same environment variable set for replay?
What version of the Vulkan loader is installed on the system?

You might also try replay with a build from the dev branch.  Switching from master to dev will require a submodule update, so the commands would look something like this:
git checkout dev
git submodule update

Was the capture layer enabled during replay?

Not that I can tell.
I tried to manually enable it because I thought it was necessary actually but it appears it's not because there's an explicit warning when it's enabled:
env  VK_LAYER_PATH=$HOME/dev/gaming/gfxreconstruct/_build/layer VK_INSTANCE_LAYERS=VK_LAYER_LUNARG_gfxreconstruct GFXRECON_LOG_LEVEL=debug  ./gfxreconstruct/build/bin/gfxrecon-replay gfxrecon_capture_20200510T160914.gfxr
[gfxrecon] WARNING - Replay tool has detected that the capture layer is enabled
fish : Tâche , 'env  VK_LAYER_PATH=$HOME/dev/ga…' terminée par le signal SIGSEGV (Erreur de frontière d'adresse)

was an environment variable such as VK_ICD_FILENAMES set to specify the path to a driver? If so, was the same environment variable set for replay?

No, I didn't change ICD loaders in-between the capture and the replay. Both uses the default Mesa one.

What version of the Vulkan loader is installed on the system?

local/vulkan-icd-loader 1.2.137-1

Sorry for the delay.  I have tried reproducing the issue locally but have been unable to create a vkcube capture that will crash on replay.  If you could attach your capture file to this issue, I can try to use it to reproduce the problem.


Cannot find module error when building

I'm running into an error saying that a module cannot be found during the build process.

Nexe v1.1.0 is being run on Windows with the basic command
nexe -i myapp.js -o myapp.exe
The module that's causing the error is being required like so:
module.exports.VariableIds        = require("lib/opcua_node_ids").VariableIds;
It seems like the require path isn't being resolved correctly. After changing the require path to include ./ ( require("./lib/opcua_node_ids").VariableIds;)this particular message disappears but other messages of the same type crop up.

I believe that apps need to refer to local paths as ./ so not sure if this is a design issue with browserify or not.

Yes, its trying to find the module lib, using ./ indicates its in the cwd not a npm module.

Ah ok. Thanks for the info.


Integrate the filebrowser rock into main

This PR integrates the rock built from https://github.com/canonical/filebrowser-rock.
Closes #126
Testing instructions
note: at time of writing, to test the volumes webapp UI you need to deploy the full kubeflow bundle as we have not defined a minimum bundle that makes volumes function properly. Deploying what the integration CI deploys here is not enough
juju deploy kubeflow --channel 1.8/stable --trust

# wait for everything to settle

# Migrate to the newest charm version
juju scale-application kubeflow-volumes 0
# Wait for it to scale down
sleep 10
juju refresh kubeflow-volumes --channel latest/edge/pr-127
juju trust kubeflow-volumes --scope=cluster
juju scale-application kubeflow-volumes 1

# Configure everything else
juju config dex-auth static-username=user123
juju config dex-auth static-password=user123
juju config dex-auth public-url="http://<IP_ADDRESS>.nip.io"
juju config oidc-gatekeeper public-url="http://<IP_ADDRESS>.nip.io"

# Wait for everything to settle

# log into <IP_ADDRESS>.nip.io using user123/user123
# go to volumes web app
# create a volume named testvol
# click the "Open PVC Viewer" button (small folder icon on the right side of the row)
# filebrowser window should open (although there will be no files since pvc is empty - you can create some though!)

# confirm we got a filebrowser with the new image:
kubectl -n user123 get pod $(kubectl get pod -n user123 --selector=app.kubernetes.io/instance=pvcviewer-testvol --output=jsonpath={.items..metadata.name}) -o yaml | grep filebrowser:
# Should return:
filebrowser:
    image: docker.io/library/charmedkubeflow/filebrowser:v2.27.0
    image: docker.io/library/charmedkubeflow/filebrowser:v2.27.0

# NOT
filebrowser:
    image: docker.io/library/filebrowser/filebrowser:v2.23.0
    image: docker.io/library/filebrowser/filebrowser:v2.23.0

Ready as soon as https://github.com/canonical/filebrowser-rock/pull/2 merges

Now ready


Temporal error refactor

I've been looking at the Temporal Java client documentation to see how that handles workflow and activity errors.  We'll want the Temporal .NET client to look more like this than what the GO client is doing.  Here's what I'm seeing:
TemporalException - This is the base class for all of these errors.

It has the equivalent of Message and InnerException properties but nothing else.

ActivityCompletionException : TemporalException - This looks like the base class for a handful of activity related exceptions.

This adds the equivalants of these properties: ActivityId, ActivityType, RunId, WorkflowId
Here are the derived exceptions:

ActivityCanceledException - Usually indicates that activity was already completed (duplicated request to complete) or timed out or workflow is closed.
ActivityCompletionFailureException - Unexpected failure when completing an activity.
ActivityNotExistsException - Usually indicates that activity was already completed (duplicated request to complete) or timed out or workflow is closed.
ActivityWorkerShutdownException Indicates that WorkerFactory.shutdown() or WorkerFactory.shutdownNow() was called. It is OK to ignore the exception to let activity to complete. It assumes that WorkerFactory.awaitTermination(long, TimeUnit) is called with a timeout larger than the activity execution time.

TemporalFailure : TemporalException - This adds the equivalant of an OriginalMessage property.  This is probably a way to include the "inner" message in an error without actually serializing the inner exception expicitly.
Here are the derived exceptions:

ActivityFailure - Contains information about an activity failure. Always contains the original reason for the failure as its cause. For example if an activity timed out the cause is TimeoutFailure. This adds these properties:

ActivityId`
ActivityType
Identity ???
RetryState
ScheduledEventId ???
StartedEventId ???

ApplicationFailure - Represents failures that can cross workflow and activity boundaries.  Any unhandled exception thrown by an activity or workflow will be converted to an instance of ApplicationFailure.  This adds these properties:

Type
IsNonRetryable
NewFailure - New ApplicationFailure with isNonRetryable() flag set to false.
newNonRetryableFailure - New ApplicationFailure with isNonRetryable() flag set to true.

CanceledFailure - The Java code documentation kind of sucks: I'm guessing this happens when a workflow or activity is canceled.
ChildWorkflowFailure - Thrown by a child worklow for uncauch exceptions?  This adds these properties:

Execution
InitiatedEventId ???
Namespace
RetryState
StartedEventId ???
WorkflowType

ServerFailure
TerminatedFailure
TimeoutFailure
ServerFailure - Exceptions originated at the Temporal service.  Properties:
*IsNonRetryable
TerminatedFailure - Maybe we'll actually see these now?
TimeoutFailure - This looks like its used for activity timeouts due to missing a heartbeat.  This applies to workflows exceeding their run limits as well.  Properties:

LastHeartbeatDetails
TimeoutType - enum: ScheduleToStart StartToClose Heartbeat

WorkflowException : TemporalException - Base class for workflow related exceptions. This adds these properties:

Execution
WorkflowType

Here are the derived classes:

CancelExternalWorkflowException - Exception used to communicate failure of a request to cancel an external workflow.
SignalExternalWorkflowException - Exception used to communicate failure of a request to signal an external workflow.kCompletedEventId`
WorkflowExecutionAlreadyStarted -
WorkflowFailedException - Indicates that a workflow failed.  Properties:

RetryState
WorkflowTask

WorkflowNotFoundException - Thrown when workflow with the given id is not known to the Temporal service.
WorkflowQueryException -
WorkflowServiceException - ???

Here's a first cut at some thoughts on the .NET side:

[ ] Temporal really went to town on changing their error handling.  It looks like the .NET client will need to convert any uncaught workflow or activity exceptions into ApplicationFailure exceptions and probably allow workflow/activity code to throw this directly.  This looks like it maps to the old custom error.

[ ] We'll have to think about how we'll name these types on the .NET side.  Should we stay with names like TimeoutFailure or stick with .NET conventions and go with TimeoutFailureException.  Java seems to have a similar covention of appending Exception to excepion type names, so we can probably assume that Temporal went this way for a good reason.  We should probably do the same.

[ ] Temporal retained the RetryOptions type with the RetryOptions.DoNotRetry array property.  I read somewhere that the DoNotRetry array matches a Type property which will default to the fully qualified name of the .NET exception type.  I'm not sure where this property is defined though.

[ ] We'll need to define all of these types and remove the old ones.

[ ] This is the first time I've seen a mention of event IDs as in the InitiatedEventId and StartedEventId error properties.  I did a quick search in the GO client code and these are present there too, so I assume that you'll see these in errors returned by Temporal.

Here's a first cut at some thoughts on the GO side: source code

[ ] It looks like ApplicationError encodes inner exceptions?  We'll need to support this on both the .NET/GO sides.  We could do this via JSON and I assume that this is not intended to be cross-platform compatible (i.e. we wouldn't expect a Java client executing a .NET workflow to be able to desrialize an inner exception.

[ ] We had a relatively simple (in comparision) way for the .NET/GO side to communicate errors for Cadence.  We'll have to figure something simple out for this wave of new error types.  I'm thinking perhaps of a simple error class that includes an error type property (that would map to one of the specific types above along with a string property that will hold the serialized JSON for the error.  Something like:
internal class TemporalError
{
   public string ErrorType { get; set; }
   public string ErrorJson { get; set; }
}

This is not too different from what we have, but we'd replace the error string property with the JSON property and then both sides would need to switch on the error type and deserialize the JSON into the property type.

CLOSING: Temporal is deprecated.  We're waiting for an official client.


training hours

Hi, in your paper, you mentioned that "The fourteen painters dataset, for example, ran 1400 epochs in 220 hours on our nVidia Titan X GPU. Note that pairwise CycleGAN instead would have taken about 2860 hours, or four months."
How you calculated the training hours, i.e. 2860 hours for CycleGAN?

Hi,
It was sort of a back of the envelope calculation. That assumes no parallel trainings.
[Time to train CycleGAN on 2 domains on our GPU] * (14 choose 2 domains) = approx. 2860 hrs


Normalize trailing slash on Uri values

While working on rust-etcd I ran into a problem with hyper::Uri where it did not normalize trailing slashes as I expected. I'm not sure if this is intended behavior or not, but it definitely tripped me up and I spent some time determining it was the cause of my bug, so I thought I'd bring it up.
Here is the behavior I'm seeing:
extern crate hyper;

use hyper::Uri;

fn main() {
    let no_path: Uri = "http://example.com".parse().unwrap();
    let root_path: Uri = "http://example.com/".parse().unwrap();

    println!("{}", no_path);
    println!("{}", no_path.path());

    println!("{}", root_path);
    println!("{}", root_path.path());
}

Output:
http://example.com
/
http://example.com/
/

I have an internal method that looks roughly like this:
fn build_url(uri: &Uri, path: &str) -> String {
    format!("{}v2/keys{}", uri, path)
}

If the string the Uri was built from ended in a slash (e.g. "https://example.com/"), this would work as expected. If it didn't (e.g. "https://example.com"), the output of my build_url function would come out as "https://example.comv2/keys/whatever", which is wrong.
I can work around this now that I know it's happening, but I'm wondering if it'd be appropriate for hyper::Uri to do some normalization here. If the current behavior is expected, it's probably worth talking about it in the documentation, as I doubt I'm the last person who will be surprised by this.
Probably also worth mentioning that I'm only doing this string manipulation because of continued pain with #1102—if I could simply do uri.set_path(path) that would be much better. I do understand why it was decided to use a separate Uri type rather than using the url crate, but as a user, having to deal with both types is unpleasant.

A URI can have an empty path segment per spec. In general care must be taken not to confuse URIs and URI-References and the issues involved with resolving a URI reference against a context URI. In addition are the rules for HTTP request URIs. Then also, URIs and proxy requests can be unintuitive but exceptional cases be very useful (eg. doing a proxy requests to a non HTTP URI)
Looking at the current implementation briefly it looks not quite right but seems to focus on HTTP Requests instead of full URI spectrum.
In general, I think the best advice is to stick to the RFCs very strictly and let them guide the architecture of the various pieces (structs). Scala's Akka-HTTP did a good job, AFAIK and it certainly will be helpful to run the design by an expert briefly. I'll try to get one of them interested.
Sorry, I have never myself implemented an HTTP lib, just know that non-strict implementations cause a lot of pain down the line if you want to do real HTTP.


Cleanup build rpm deb

ping @tianon @mikedougherty

oh nice -- i think eventually we can use this to build in parallel :)

ping @tianon PTAL

Nice, LGTM :+1:

This will really speed up building packages! Thnx


Update dependency file-loader to v4.3.0

This PR contains the following updates:

Package
Type
Update
Change

file-loader
devDependencies
minor
^4.2.0 -> 4.3.0

Release Notes

webpack-contrib/file-loader
v4.3.0
Compare Source
Features

new esModules option to output ES modules (#340) (9b9cd8d)

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

Checkov Scan Findings
Potential security misconfigurations were detected.
Please reach out to the #infra-team-support/@security team if any assistance is required.
terraform scan results:

Passed checks: 12, Failed checks: 7, Skipped checks: 0

Check: CKV_GCP_14: "Ensure all Cloud SQL database instance have backup configuration enabled"
	FAILED for resource: google_sql_database_instance.master_instance
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/big_data.tf:2-20
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_general_2

		2  | resource google_sql_database_instance "master_instance" {
		3  |   name             = "terragoat-${var.environment}-master"
		4  |   database_version = "POSTGRES_11"
		5  |   region           = var.region
		6  |
		7  |   settings {
		8  |     tier = "db-f1-micro"
		9  |     ip_configuration {
		10 |       ipv4_enabled = true
		11 |       authorized_networks {
		12 |         name  = "WWW"
		13 |         value = "<IP_ADDRESS>/0"
		14 |       }
		15 |     }
		16 |     backup_configuration {
		17 |       enabled = false
		18 |     }
		19 |   }
		20 | }

Check: CKV_GCP_11: "Ensure that Cloud SQL database Instances are not open to the world"
	FAILED for resource: google_sql_database_instance.master_instance
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/big_data.tf:2-20
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_networking_4

		2  | resource google_sql_database_instance "master_instance" {
		3  |   name             = "terragoat-${var.environment}-master"
		4  |   database_version = "POSTGRES_11"
		5  |   region           = var.region
		6  |
		7  |   settings {
		8  |     tier = "db-f1-micro"
		9  |     ip_configuration {
		10 |       ipv4_enabled = true
		11 |       authorized_networks {
		12 |         name  = "WWW"
		13 |         value = "<IP_ADDRESS>/0"
		14 |       }
		15 |     }
		16 |     backup_configuration {
		17 |       enabled = false
		18 |     }
		19 |   }
		20 | }

Check: CKV_GCP_6: "Ensure all Cloud SQL database instance requires all incoming connections to use SSL"
	FAILED for resource: google_sql_database_instance.master_instance
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/big_data.tf:2-20
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_general_1

		2  | resource google_sql_database_instance "master_instance" {
		3  |   name             = "terragoat-${var.environment}-master"
		4  |   database_version = "POSTGRES_11"
		5  |   region           = var.region
		6  |
		7  |   settings {
		8  |     tier = "db-f1-micro"
		9  |     ip_configuration {
		10 |       ipv4_enabled = true
		11 |       authorized_networks {
		12 |         name  = "WWW"
		13 |         value = "<IP_ADDRESS>/0"
		14 |       }
		15 |     }
		16 |     backup_configuration {
		17 |       enabled = false
		18 |     }
		19 |   }
		20 | }

Check: CKV_GCP_15: "Ensure that BigQuery datasets are not anonymously or publicly accessible"
	FAILED for resource: google_bigquery_dataset.dataset
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/big_data.tf:22-38
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_general_3

		22 | resource google_bigquery_dataset "dataset" {
		23 |   dataset_id = "terragoat_${var.environment}_dataset"
		24 |   access {
		25 |     special_group = "allAuthenticatedUsers"
		26 |     role          = "READER"
		27 |   }
		28 |   labels = {
		29 |     git_commit           = "83661b5e88dd3768c10dbcfa2050c04d85b62fa8"
		30 |     git_file             = "terraform/gcp/big_data.tf"
		31 |     git_last_modified_at = "2020-07-09 13:34:36"
		32 |     git_last_modified_by =<EMAIL_ADDRESS>		33 |     git_modifiers        = "nimrodkor"
		34 |     git_org              = "bridgecrewio"
		35 |     git_repo             = "terragoat"
		36 |     yor_trace            = "2560d883-bc3a-4cb6-b9fc-fb666edf626e"
		37 |   }
		38 | }

Check: CKV_GCP_62: "Bucket should log access"
	FAILED for resource: google_storage_bucket.terragoat_website
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/gcs.tf:1-14
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_logging_2

		1  | resource "google_storage_bucket" "terragoat_website" {
		2  |   name          = "terragot-${var.environment}"
		3  |   force_destroy = true
		4  |   labels = {
		5  |     git_commit           = "ff3ee8387837a499665630ebb0371be39ce35fb6"
		6  |     git_file             = "terraform/gcp/gcs.tf"
		7  |     git_last_modified_at = "2020-07-08 12:02:14"
		8  |     git_last_modified_by =<EMAIL_ADDRESS>		9  |     git_modifiers        = "nimrodkor"
		10 |     git_org              = "bridgecrewio"
		11 |     git_repo             = "terragoat"
		12 |     yor_trace            = "bd00cd2e-f53f-4daf-8d4d-74c47846c1cc"
		13 |   }
		14 | }

Check: CKV_GCP_29: "Ensure that Cloud Storage buckets have uniform bucket-level access enabled"
	FAILED for resource: google_storage_bucket.terragoat_website
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/gcs.tf:1-14
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_gcs_2

		1  | resource "google_storage_bucket" "terragoat_website" {
		2  |   name          = "terragot-${var.environment}"
		3  |   force_destroy = true
		4  |   labels = {
		5  |     git_commit           = "ff3ee8387837a499665630ebb0371be39ce35fb6"
		6  |     git_file             = "terraform/gcp/gcs.tf"
		7  |     git_last_modified_at = "2020-07-08 12:02:14"
		8  |     git_last_modified_by =<EMAIL_ADDRESS>		9  |     git_modifiers        = "nimrodkor"
		10 |     git_org              = "bridgecrewio"
		11 |     git_repo             = "terragoat"
		12 |     yor_trace            = "bd00cd2e-f53f-4daf-8d4d-74c47846c1cc"
		13 |   }
		14 | }

Check: CKV_GCP_28: "Ensure that Cloud Storage bucket is not anonymously or publicly accessible"
	FAILED for resource: google_storage_bucket_iam_binding.allow_public_read
	File: /infra/gcp/permutive-playground/test-checkov-vulnerable-iac/gcs.tf:16-20
	Guide: https://docs.bridgecrew.io/docs/bc_gcp_public_1

		16 | resource "google_storage_bucket_iam_binding" "allow_public_read" {
		17 |   bucket  = google_storage_bucket.terragoat_website.id
		18 |   members = ["allUsers"]
		19 |   role    = "roles/storage.objectViewer"
		20 | }


Possible to hide all IoT devices?

I have IoT devices exposed to my LG TV's Home Dashboard for the convenience of controlling them while the TV is on. These devices all appear in the TV's Accessory Details view in HomeKit. Would it be possible to prevent them from showing at all so that the accessory details view is cleaner and less cluttered?

I do not quite understand what you mean, but I guess you could use the configuration to disable the stuff which you do not want to appear.


QuickPick : Ordered Selection

There are currently two modes of operation for QuickPick: Single selection and Multiple selection.
With multiple selection, the returned selected items are in the order in which they are listed in the dropdown menu.
In some cases, the order in which the selected items are used is also important and I see no mechanism for the user to specify this information.
Allowing ordered selection actually introduces two new modes of operation for QuickPick :

All the items must be used and can be ordered as the user wishes
Any number of items can be selected and these selected items can be ordered as the user wishes

A click and drag mechanism for ordering the items would be the most user friendly; however, I imagine that this is incompatible with the current selection menu code.
For the first of the two new modes of operation, I cannot see any other method in which the user could provide information about the order.
However, for the second of the two new modes of operation, the most straightforward approach would be to keep track of the order in which the items are selected.
Prefixing the label with a number would indicate the order in which it would be used.

@DTHaydon this would need an API addition which we usually only do if there's a large demand for it. Did you have a specific scenario in mind that you wanted to make but couldn't because this API doesn't exist?

@TylerLeonhardt
Of the two new selection modes I mentioned:

the first (use all items but can reorder) I have not encountered a need for and would probably require an API addition.
the second (select and order) I have wanted to use (and have been able to implement in some form)

For my original implementation of the ordered selection, I tracked the order in which items were selected.
However, this was inconvenient if an item was forgotten and where a click and drag feature would have been helpful.
With release 1.63 (individual QuickPickItem buttons) and the API Proposals: quickPickSeparators and quickPickSortByLabel it is possible to implement something more user friendly:
https://user-images.githubusercontent.com/15086793/145851629-c332b03e-447c-481d-b398-fb271b452591.mov

quick pick list chaos sort


Throw/catch native errors where possible

Closes https://github.com/refined-github/refined-github/issues/7750

Demo

The first one is handled during init, the id is known an passed to the logger
The second one is caught via unhandledrejection and the id is parsed via the error stack

Can be seen at https://github.com/refined-github/refined-github/commits/main

Tasks

[x] Ensure that GitHub errors are not caught
[x] Self-review code

Good testing ground: https://togithub.com/refined-github/refined-github/issues/3686
There are currently a lot of nameless errors:

And https://github.com/refined-github/refined-github/issues/6000#issuecomment-2185499249

Also catch API errors in include/exclude arrays:
https://github.com/refined-github/refined-github/blob/f3d06808fba45ba2200070b3ebc441eaea0c08e2/source/feature-manager.tsx#L194-L196
Test URL: https://github.com/refined-github/refined-github/actions

Also it should ignore these issues

[ ]  Ensure that GitHub errors are not caught

Done: See native error followed by caught RGH error (and temporary {promise} log for all errors) https://github.com/user-attachments/assets/53569068-bf2a-4f1a-8064-cb652392f0fe"

There are currently a lot of nameless errors:

Before
	After

Many nameless errors:
One deduplicated log:

Also fregante/webext-storage-cache#49

Also caught:

Before
	After

Also catch API errors in include/exclude arrays:

Before
	After

Also it should ignore these issues

Before
	After

Final touch: restore Refined GitHub in the title since errors now all come from error.js


(TWILL-238) Change TwillController restartInstances API

Implemented https://issues.apache.org/jira/browse/TWILL-238
New method in TwillController to restart instances of runnable

Thank you, serranom, for your comment. Is there any way to set "right" order of imports in IDEA? I've imported settings from HowToContribute page, but it seems, that it still wants to move imports.

Reverted reformatting

LGTM.


Allow distrusting system certificate store

Currently when using CA pinning, you always also trust the system certificate store. This is often not wanted.

I believe this'll require a small feature in schannel-rs, but should be straightforward to do.

Any news about this? I'd like to be sure in some situations that I'm only using the one root certificate I've added myself.

We used to be able to do this in native-tls 0.1 with the following code:
        let mut ssl =
            openssl::ssl::SslConnectorBuilder::new(openssl::ssl::SslMethod::tls()).unwrap();
        // Server authentication
        ssl.set_ca_file(ca_certificate.clone()).unwrap();
        let builder: native_tls::TlsConnectorBuilder =
            native_tls::backend::openssl::TlsConnectorBuilderExt::from_openssl(ssl);

but this method no longer works in nativetls 0.2


Verbose option is ignored for statistics message

Imagemin ignores it's "verbose" option for statistical messages, such as "[18:44:52] gulp-imagemin: Minified 17 images (saved 6.11 MB - 88.3%)". Should fix or add another option "stats" to control statistics output.

Pardon - wrong project.


Fix minor typo in link to Reinforcement paper

This fixes a minor typo in the link to the "Continuous control with deep reinforcement learning" paper

@hamishdickson Please review CONTRIBUTING section.

update


Implement the Error trait for the ConfigError type

Seems to have just been an oversight that this was missed. This makes it easier to convert ConfigErrors to other error types, like std::io::Error or failure::Error.

@Diggsey good eye, and thanks for taking the time to contribute! Mind updating the changelog here (https://github.com/railgun-rs/actix-raft/blob/master/CHANGELOG.md#04) with a new 0.4.4 section with a bullet point underneath describing this change?
After that, I'll cut a new release.

Updated 👍


Showcase Submission for webgl/2-3d-geometries

Thank you indicodeswitch for your contribution! A member of the Coding Train team will review it shortly.

Rapid Auntie Jen Dance
indicodeswitch

Thank you for sharing!


int main () doesn't let the code running

The code I run
#include <iostream>

int main () {
    jdisplay::YouTubePlayer("https://www.youtube.com/watch?v=BwewsbFMIcs", "1280", "720");
    return 0;
}

The error it returned:
/tmp/tmpu6u6r47y.cpp: In function ‘int main()’:
/tmp/tmpu6u6r47y.cpp:8:13: error: a function-definition is not allowed here before ‘{’ token
    8 | int main () {
      |             ^

[C++ 14 kernel] Interpreter exited with code 1. The executable cannot be executed

The replacement of:
220 if not re.search(r'int\s+main\s*\(', code):

by this new RegEx:
if not re.search(r'int\s+main\s*\(\s*\)', code):

has extended the code functionality. I will review this line whenever the PR is ready.


Contributed to the list of names

Had to close the former pull request and undergo the processes all over again.

Thank you! As soon as we treat the pull requests prior to this, we will merge.

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.


Fix click js in ie

fixes https://github.com/selenide/selenide/issues/1406
had to use a different JS code to trigger "click" event in IE.

@asolntsev js click script changes LGTM


Disable cookies for clients and public routes

Describe the bug
When using the /clients or /public gateway routes you still get cookies set which is not required for both.
To Reproduce

Call a /clients-Route (i.e. /clients/digitalwf-backend-service/rest) or /public-Route
You get a Set-Cookie-header back with the auth and/or csrf cookie contained

Expected behavior
You can call both routes as before but there is no csrf or auth cookie set
Screenshots

Additional context

Internal docs: https://git.muenchen.de/ccse/refarch-security/-/wikis/vorgaben/kommunikation#2-aufrufe-aus-anderen-anwendungen-immer-über-das-api-gateway

Will be done within https://github.com/it-at-m/refarch/issues/135


Figure 8.10 text mismatch

P177 - Organizing scene files
The instructions are:
Add the following folders in the root folder of our project:
• parts:
� environment
� player
• screens:
� game
In figure 8.10 the folder is called arena, instead of environment.

The name Arena in general is misused within that chapter. Thanks for letting us know!


Issue: Buttons unresponsive in the wifi manager on RG351v

RG351v ArkOS 2.0 (03112023)
After an update, the wifi manager menu changed and became like this:

The buttons are unresponsive, I assume a physical keyboard is required to use this menu.
Select+Start does not work to exit the menu. I can only exit with a hard shutdown using the power button.
Workaround: go to retroarch menu > settings > wifi to update the wifi settings

See here for the fix:  https://github.com/christianhaitian/arkos/issues/652#issuecomment-1446652500


Replace std::unordered_map with F14FastMap (#328)

Replace std::unordered_map with F14FastMap in velox/type/tz/TimeZoneMap.cpp
Fixes #328

@odilov linux-build and macos-build are failing. Would you take a look?

@mbasmanova sure, I'll fix it

@funrollloops I've updated CMakeLists.txt, macos-build is ok, but linux-build is still failing. What is format-check ? Could you please give me some advice ?
CC: @mbasmanova

@odilov Try running "make format-fix" from the root directory. It should fix formatting. Then, update the PR to confirm.

@odilov linux-build fails because of a test failure: DateTimeFunctionsTest.fromUnixtimeWithTimeZone
Could you run this test locally to see if it succeeds?

96: [ RUN      ] DateTimeFunctionsTest.fromUnixtimeWithTimeZone
96: unknown file: Failure
96: C++ exception with description "Unknown time zone: +01:00" thrown in the test body.
96: [  FAILED  ] DateTimeFunctionsTest.fromUnixtimeWithTimeZone (1 ms)

@mbasmanova it's not working :(
I can't run project locally, because of M1 chip
even "make format-fix" is not working
Installing Rosetta 2 on this system is not supported.
Do you have any idea ?
CC: @funrollloops

I can't run project locally, because of M1 chip

That's a bigger problem. We need to fix that first.

even "make format-fix" is not working

Can you share full output of "make format-fix"?

@mbasmanova

Can you share full output of "make format-fix"?

scripts/check.py format branch --fix Fix  : velox/type/tz/CMakeLists.txt /bin/sh: cmake-format: command not found Error: velox/type/tz/CMakeLists.txt Fix  : velox/type/tz/TimeZoneMap.cpp error: cannot find executable "clang-format" Error: velox/type/tz/TimeZoneMap.cpp make: *** [format-fix] Error 1
by running cmake --version
I'm getting:
cmake version 3.21.3
that means I have latest cmake

@mbasmanova
I'm trying to run sh setup-macos.sh from scripts folder
I'm getting these:
Error: Cannot install under Rosetta 2 in ARM default prefix (/opt/homebrew)!
To rerun under ARM use:
arch -arm64 brew install ...
To install under x86_64, install Homebrew into /usr/local.

brew upgrade --formula ninja
Error: Cannot install under Rosetta 2 in ARM default prefix (/opt/homebrew)!
To rerun under ARM use:
arch -arm64 brew install ...
To install under x86_64, install Homebrew into /usr/local.

CC: @mshang816

/bin/sh: cmake-format: command not found

Can you try installing cmake-format?
pip3 install --user cmake-format regex

@mbasmanova
I've installed it before:
pip3 install --user cmake-format regex
Requirement already satisfied: cmake-format in /Users/marshall/Library/Python/3.9/lib/python/site-packages (0.6.13)
Requirement already satisfied: regex in /opt/homebrew/lib/python3.9/site-packages (2021.9.30)
Requirement already satisfied: cmakelang>=0.6.13 in /Users/marshall/Library/Python/3.9/lib/python/site-packages (from cmake-format) (0.6.13)
Requirement already satisfied: six>=1.13.0 in /opt/homebrew/lib/python3.9/site-packages (from cmakelang>=0.6.13->cmake-format) (1.16.0)

I think you need to do something like this:
# To add cmake-format bin to your $PATH, consider adding this to your ~/.profile:
export PATH=$HOME/bin:$HOME/Library/Python/3.7/bin:$PATH

Hey Odilov, just FYI none of us have ARM macs. We use assembly instructions in some parts of Velox; ARM support is not planned.
The format-fix CI failure actually prints out the command to apply the patch locally:
patch -p1 <<EOF
diff --git a/velox/type/tz/CMakeLists.txt b/velox/type/tz/CMakeLists.txt
index 257b32a..aba00e3 100644
--- a/velox/type/tz/CMakeLists.txt
+++ b/velox/type/tz/CMakeLists.txt
@@ -14,4 +14,5 @@
 add_subdirectory(tests)
 add_library(velox_type_tz TimeZoneMap.h TimeZoneDatabase.cpp TimeZoneMap.cpp)

-target_link_libraries(velox_type_tz ${Boost_REGEX_LIBRARIES} ${FMT} ${FOLLY_WITH_DEPENDENCIES})
+target_link_libraries(velox_type_tz ${Boost_REGEX_LIBRARIES} ${FMT}
+                      ${FOLLY_WITH_DEPENDENCIES})
EOF

Copy-pasting that into your terminal should fix the formatting automatically.
Can you run docker x86 locally? If so, you should be able to reproduce the CircleCI failure and drop into an interactive terminal like so:
# Starting from the root of your velox checkout
docker run -it \
    --mount type=bind,source=$(pwd),destination=/root/velox,readonly \
    --workdir /root/velox \
    $(grep -o -E 'prestocpp/velox-circleci:[^[:space:]]+' .circleci/config.yml) \
    bash --init-file /opt/rh/gcc-toolset-9/enable
make debug

I would install the pip formatting deps in a virtualenv in your checkout, so you can run it from within the docker instance.


doc add pg and sqlserver database sync

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

run buildall


"Nothing selected." - Guide

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3509
Originally created by @vischous on 2022-05-19 13:36:10

This isn't a Meltano issue as it's a Singer common issue. Not sure where the right place is for this issue
When does "Nothing Selected" happen?

This happens when someone is trying to run a tap themselves, (on a non SDK tap)

pip install .
tap-jira --config config.json --discover > catalog.json
tap-jira -p catalog.json --config config.json
You'll see "Nothing Selected"

Possible Solutions for folks that hit Nothing Selected

Write up an "Explanation" for Nothing Selected.  it'll come up in Google search results pointing folks to use Meltano to solve this issue for them!

View 1 previous comment from the original issue on GitLab


Hi, there are some issues about the infoNCEloss

Hi, there are some issues about the infoNCEloss:
when we define the info_nce_loss in the code as follows:
info_nce_loss = InfoNCELoss(temperature=0.1,
batch_size=args.batch_size * 2,
flat=('flat' in args.crl),
n_views=8 if args.mcrop else 2)
the labels size in the function of infoNCELoss is batch_size x 2 x n_views, while the size of the features are batch_size x n_views. Since the sizes of these two params are not matched, the calculation of the infoNCEloss is not correct.

The features are comprised of two parts: labeled and unlabeled, each with a size equal to batch_size*n_views.

The features are comprised of two parts: labeled and unlabeled, each with a size equal to batch_size*n_views.

I know, but when this function is used to perform warm-up training, it will be incorrect. If possible, you can check it.

 Thank you.

If possible, could you hare me a training log about the PLRemix on webvision, it seems i met some problems when training on this set.


Messung - Das Messfenster erscheint nicht

Strangely this only occurs in the release demo version and not when developing locally:

I have unfortunately have to re-open this issue, as this was not solved. This ticket was confused with another issue where the entire sketching container was not sized properly:

I think I found the problem. In the SketchingWidget.vue component a component call 'measurement-toggle' was defined with a v-if directive (v-if="measurement") was assigned a variable that does not exist:
https://github.com/conterra/mapapps-sketching-enhanced/blob/84cc48812569f57f7b9d70064be4a010fd755010/src/main/js/bundles/dn_sketchingenhanced/SketchingBase.vue#L61


Exploits to the agent on testnet

Malicious Fredo exploits the agent
I could not keep my hands still over the weekend. I was simply to hyped by what we have achieved and wanted to try out the protocol on testnet. I don't like to bring the bad news but I found some exploits to the protocol and the agent. The good news are: We have an awesome team and I think we can fix those issues in a reasonable amount of time. And it's better that we found out about that before announcing it to the public.
Attack scenario
Bob is the honest agent and gets tricked by Malicious Alice (M'Alice) to not follow the protocol anymore. This will ultimately lead to the protocol losing funds or DOSing the protocol.
Assumption: Events from one chain come in at chronological order.
Links and helpful information
Explorers
Request Manager on Boba
Fill Manager on Metis
Function selectors
For a better understanding of transaction see the methods' selectors
Request Manager

createRequest: 0x2f4212b8
claimRequest: 0xb1743c5b
challengeClaim: 0x92c824d8
withdraw: 0x2e1a7d4d

Fill Manager

fillRequest: 0xefd39926

Exploits
Fill data not checked #336

M'Alice sends wrong fill (i.e. amount = 1)
Bob does not check the metadata of the fill and will assign M'Alice as the filler to the request
M'Alice claims
Bob will not challenge as M'alice is assigned as the filler

Successful exploit on testnet

Request
Fill
Claim
Withdraw
M'Alice filled the request with 1 Wei of TST. Bob did not check the requests' metadata and assigned M'Alice as the filler. Thus, when M'Alice claimed the request, Bob did not challenge.

Severity
High: Tokens can get stolen
Claimed requests will not be filled

M'Alice will claim an incoming request immediately
Bob will receive the request and claim event in the same cycle
The request is in the state "claimed" and will not be filled anymore and eventually expire

Successful exploit on testnet

Request
M'Alice's claim
Bob received the request and claim event in the same fetch. The request will transition to claimed before it is trying to be filled. Bob did not fill.

Severity
Medium: Requests will expire. DOS Attack on agent
Claims will be dropped during fill unconfirmed

Bob sees a request and fills it -> State filled_unconfirmed
In the same time M'Alice will claim the request. The claim arrives before the fill.
The claim will stay unprocessed as filled_unconfirmed.claim() is an invalid transition.
Bob's fill event arrives.
Bob's fill event and M'alice's claim will be processed and transition filled_unconfirmed->filled->claimed.
Processing the request will not trigger a claim by bob.

Successful exploit on testnet

Request
M'Alice's Claim
Bob's fill
Bob did not claim his fill

Severity
Medium/High: Agent will be unable to claim its tokens, but dishonest claims will still be challenged.
Cause of the issues
It took me quite a while to get behind what's going on in the agent. Instead of fixing the issues described above with some sort of special conditions, there is a more fundamental issue with the agent and I suggest to fix the cause root of the problem directly.
To be more precised, the state machine of the agent is somewhat flawed and there are a couple of bad things coming together which make the agent open to exploits.
I'd like to distinguish between three problems:

Order of incoming state changes
Overlaying to many different states into one
skipping outputs of state changes

Order of incoming state changes
We must acknowledge that the order of incoming state changes for one particular request is not fix. This is due to the fact that we have two different sources of state changes (source and target chain). We cannot rely on the delivery time of those events. Due to network slow downs events can come in in unforseen orders. Note, that not all messages can have any order. It is i.e. not possible that a claim of a request arrives before the request was created. This is due to the fact that the two events come from the same emitter (source chain) and the contract forbid claims for non-existing request ids.
Secondly, and this is also related to the next fundamental issue, state changes for a request can be triggered by external parties. I.e. anybody can create a claimEvent inbetween the request was created and expired/withdrawn. That any malicious actor is able to trigger state changes introduces a wide range of exploitation opportunities and should be strictly avoided!
Overlaying to many different state changes
If one looks at the state machine, it describes very much the happy case if the agent is the filler. But the agent can also take on a different role in the protocol, "the watcher". Unfortunately, the state machine does not take the different roles into account. A request in the state "claimed" does not distinguish if the state is claimed by the agent or someone else. In state "claimed_unconfirmed" an incoming state will transit to "claimed", independently if it's the agent's claim or someone elses claim. While I understand that overlaying states is convinient and keeps the codebase small and clean, it introduces fundamental flaws to the functionality as those are in fact not the same states. In my opinion we must distinguish between those states, and the agent must realize on what he needs to receive to transit in state.
I think one simple solution would take claims by other addresses out of the equation and handle them separately. This would also remove the problem that other actors can influence the state transition of the agent.
Skipping outputs of state changes
It is possible that multiple events regarding one request can be processed at once. I.e. (RequestCreated, RequestClaimed). The agent is build in a way that it processes all processable events at once and after it, it applies actions created from current state. Information get lost because two events can cause two state transitions and the action of the first transition is not applied.
I think changing the state machine as described in the previous point could natively avoid this problem but I'm not 100% sure yet.
What to do next
I strongly suggest that we as the dev team find a proper solution which systematically increases the robustness of the agent instead of fixing one particular exploit after the other. I was able to find three exploits but I'm sure I have not thought through all possible combinations of how the agent can be influenced by incoming events, order of events, and timing issues.

closed in favor of related subissues


(PUP-5212) Pip package honors HTTP proxy

The pip package provider previously did not work behind a http proxy. It
will now honor system proxy settings for querying latest packages from
pypi.python.org.

Hmm, the travis fail on the first commit on ruby 1.9.3 (https://travis-ci.org/puppetlabs/puppet/builds/97052540) looks like it may be https://github.com/bundler/bundler/issues/3558, which (I think) @DavidS mentioned running into elsewhere.
I didn't run this down so don't know why it's intermittent, or really what's going on.
Fix might be to force a newer version of bundler with:
before_install:
  - gem install bundler -v 1.10

Mayyyyybe. Not sure.

I'll give it a try.

Yes, I ran into this and workarounded it the same as here (adding a link to the bug report). It worked for me, but it is a HACK.


[-wATCH-]— Baltasar Ebang Engonga Video Original Video Link Baltasar Ebang Engonga Video Viral On Social Media X Trending Now

16 seconds ago Balthazar Baltasar Ebang Engonga L𝚎aked Video Viral On Social Media. Θ(ＷＡＴＣＨ VIDEO𝕊) Balthazar Baltasar Ebang Engonga Viral ᴠɪᴅᴇᴏ ʟᴇᴀᴋᴇᴅ Update Files Trending On Social Media X Telegram
𝖶𝖺𝗍𝖼𝗁 🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅 𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄) 𝖶𝖺𝗍𝖼𝗁
🟢 ➤ ➤ ➤ 🌐 𝖢𝗅𝗂𝖼𝗄 𝖧𝖾𝗋𝖾 𝖳𝗈 𝗅𝗂𝗇𝗄 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺 𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇𝗄)
🔴 ➤► 𝖣𝖮𝖶𝖭𝖫𝖮𝖠𝖣👉👉 (𝖥𝗎𝗅𝗅 𝖵𝗂𝗋𝖺𝗅 𝖵𝗂𝖽𝖾𝗈 𝖫𝗂𝗇 𝗄)
L𝚎aked Video Balthazar Baltasar Ebang Engonga Original Video Viral Video L𝚎aked on X Twitter Telegram
Balthazar Baltasar Ebang Engonga Dark Video Female Balthazar Baltasar Ebang Engonga Dark Video Link Viral On Social Media X TikTok Trending Now
Balthazar Baltasar Ebang Engonga Original Video video took the internet by storm and amazed viewers on various social media platforms. Balthazar Baltasar Ebang Engonga , a young and talented digital creator, recently became famous thanks to this interesting video.
L𝚎aked Video Balthazar Baltasar Ebang Engonga Original Video Viral Video L𝚎aked on X Twitter
Balthazar Baltasar Ebang Engonga Original Video video oficial twitter
L𝚎aked Video Balthazar Baltasar Ebang Engonga Original Video Viral Video L𝚎aked on X Twitter.
.
.
.
.
.
Balthazar Baltasar Ebang Engonga's latest video has taken the internet by storm, sparking lively discussions across various social media platforms. The rising digital influencer, known for her captivating content—especially her recent Spiderman-themed video—has experienced a massive surge in popularity. Fans are eagerly awaiting the full version, while audiences flock to Instagram, X (formerly Twitter), and Discord to engage with her latest creation.
As excitement continues to grow, fans are actively sharing their thoughts online, highlighting the crucial role social media plays in propelling creators like Balthazar Baltasar Ebang Engonga into the spotlight. With her unique style and creative approach, she has made a significant impact on the digital scene, leaving fans excited for what’s to come next. The viral success of her video further underscores the power of social media in boosting visibility and fostering stronger connections between creators and their audiences.
.
..
In the digital age, content spreads at an unprecedented speed. Social media has transformed how we communicate, share, and consume information. From trending topics to viral videos, platforms like Twitter, TikTok, Instagram, and Facebook have become central to modern discourse. Amid this environment, the name "Balthazar Baltasar Ebang Engonga" has gained significant attention following the leak of a controversial video. The so-called "Balthazar Baltasar Ebang Engonga Leaked video" became viral, making waves on various social media platforms and sparking a debate on privacy, online safety, and the implications of going viral.
This article aims to delve into the phenomenon of the Balthazar Baltasar Ebang Engonga Leaked video, examining its origins, how it became viral, and the broader implications of such content in the age of social media virality. It also explores the ethical and legal dimensions surrounding Leaked videos and the influence of digital culture on personal privacy.zxvs.
.
..//

cvbcvb

sdfsdf

cvc


Base css class for Jupyter widgets should not have 'ipy'

Removing references to IPython in jupyter-js-widgets Javascript.

Thanks for doing this


Cannot change default enviroment variables

I'm trying to setup a mongo container in a docker-compose file, here is the definition
mongodb:
        container_name: "mongo-test"
        image: mongo
        restart: always
        ports:
            - 27017:27017
        env_file:
            - docker.mongo.env
        networks:
            - dbnet
        volumes:
            - volume-test:/data/db

My env file
MONGO_INITDB_ROOT_USERNAME=user
MONGO_INITDB_ROOT_PASSWORD=1234

The thing is that when I try to connect with Mongo Compass with the following connection string mongodb://user:1234@localhost:27017 I get an authentication failed error. I can only connect with the instance if I use the default credentials root/example.
I have tried moving the env variables inside de .yml file but it still doesn't work. Running it raw it seems to work
docker run -d --name mongo-test -p 27017:27017 -e MONGO_INITDB_ROOT_USERNAME=user -e MONGO_INITDB_ROOT_PASSWORD=1234 mongo

https://docs.docker.com/compose/environment-variables/#the-env_file-configuration-option

By using this option, environment variables declared in the file CANNOT be referenced in the Compose file or used to configure Compose.

For further help you could also try asking over at the Docker Community Forums, Docker Community Slack, or Stack Overflow. Since these repos aren't really a user-help forum


Fix documentation of X509_VERIFY_PARAM_add0_policy()

The function was incorrectly documented as enabling policy checking.
Fixes: CVE-2023-0466
Checklist

[x] documentation is added or updated

Rebased to fix the CHANGES.md conflict. Assuming approvals hold.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.


asp update PACKAGE does not recreate the local branch

If $ASPROOT (~/.cache/asp) has been purged for some reason, asp update PACKAGE does not recreate the local branch for the pkgbase. This creates a problem because any existing asp checkouts become broken until another (pointless) checkout has been made:
$ asp checkout linux
remote: Enumerating objects: 9771, done.
remote: Counting objects: 100% (1912/1912), done.
remote: Compressing objects: 100% (781/781), done.
remote: Total 9771 (delta 1583), reused 1210 (delta 1131), pack-reused 7859
Receiving objects: 100% (9771/9771), 2.28 MiB | 2.20 MiB/s, done.
Resolving deltas: 100% (3994/3994), done.
From https://github.com/archlinux/svntogit-packages
 * branch            packages/linux -> FETCH_HEAD
 * [new branch]      packages/linux -> packages/packages/linux
Cloning into 'linux'...
done.

$ (cd ~/.cache/asp; git --no-pager branch)
  packages/packages/linux

$ (cd ~/.cache/asp; git --no-pager branch --all)
  packages/packages/linux
  remotes/packages/packages/linux

$ (cd linux; git pull)
Already up to date.

After purging the cache:
$ rm -rf ~/.cache/asp

$ asp update linux
remote: Enumerating objects: 9771, done.
remote: Counting objects: 100% (1912/1912), done.
remote: Compressing objects: 100% (781/781), done.
remote: Total 9771 (delta 1583), reused 1210 (delta 1131), pack-reused 7859
Receiving objects: 100% (9771/9771), 2.28 MiB | 8.86 MiB/s, done.
Resolving deltas: 100% (3994/3994), done.
From https://github.com/archlinux/svntogit-packages
 * branch            packages/linux -> FETCH_HEAD
 * [new branch]      packages/linux -> packages/packages/linux

$ (cd ~/.cache/asp; git --no-pager branch --all)
  remotes/packages/packages/linux

$ (cd ~/.cache/asp; git --no-pager branch)

$ (cd linux; git pull)
fatal: couldn't find remote ref refs/heads/packages/packages/linux

$ (mkdir tmp; cd tmp; asp checkout linux)
mkdir: created directory 'tmp'
Cloning into 'linux'...
done.

$ (cd linux; git pull)
Already up to date.

Well, this is sort of unfortunate. The difference between 'update' and 'checkout' is such that I only commit to keeping around a copy of the blobs when the user requests the checkout. Everything else asp does is lighter weight and only needs the branches and histories. Whereas the current asp cache is only a few kilobytes, such a change could potentially make the cache hundreds of megabytes.

I apologize for not getting back in over a year, but...
I'm not sure what the problem is. Asp already fetches all the blobs on asp update, it simply does not create a local branch out of that history (only a remote one).
https://github.com/intelfx/asp/commit/36ba405952ca1316211622663c88be2a99efb527 fixes the issue for me. Does this make any more sense?


[Feature] Options dialog

Reported by aulo on 2004-01-06 14:38 UTC
Create application options dialog allowing application
user to change application properties without a need to
modify the properties file manually.

Updated by aulo on 2004-02-12 11:35 UTC

status: open --> closed


class_embeddings_h

Good job! But I have a question. How did you get the class_embeddings_h in the dataset?

Hi, You may refer to the reply here.


AN-364: Fix docker build by adding missing library

Further investigation reveals that the reason the libraries solc relies on are incompatible in the docker build process is because Alpine links binaries against "musl" whereas Go binaries link against glibc (like most Linux distributions).
The changes in this commit add those missing libraries. They don't verify the checksum of the added library - but neither does HardHat.

Closed in favour of a swap out of the base image.


Missing requirements.txt file

I have cloned my forked repo of this project locally but when I try to run  pip install -r requirements.txt I cant find the file in the folder of the backend

fixed in the "update" branch


Reproducing two object scene evaluation results

I was wondering if you could please provide some more information concerning the evaluation protocol for the two object scene evaluation results. I see that there's a file named viewlist/2obj_eval_views.txt with some indices in it, but it doesn't appear to be in the format used for the other evaluations.
For these tests, are you using 1-3 specific input view indices to render a specific target viewpoint? Or are you measuring the results when rendering all non-input views?
I also noticed that the paper says that there are 50 testing views, but the test data split appears to only have 20 views, as in the training data (Sec. B.2.4). Is the data that's provided the same as what's being used for these evaluations?

Same here, I tried the official ckpt, and test with view '10', the PNSR is around 16, which is significantly different from the 20.15 PSNR reported in the paper. Kind of confusing.


Manager crashes after downloading of chunk fails

When I download large assets (In this case it is the free City Sample) the Asset Manager often crashes.

OS: Linux Mint 21.1 (Ubuntu 22.04)

Here is the debug output:
$ flatpak run --env=RUST_LOG=debug io.github.achetagames.epic_asset_manager
<epic_asset_manager> - [DEBUG] - /app/share/epic_asset_manager
<epic_asset_manager> - [DEBUG] -
<epic_asset_manager> - [DEBUG] - 3.8.3
<epic_asset_manager::application> - [INFO] - Epic Asset Manager (io.github.achetagames.epic_asset_manager)
<epic_asset_manager::application> - [INFO] - Version: 3.8.3 ()
<epic_asset_manager::application> - [INFO] - Datadir: /app/share/epic_asset_manager
<epic_asset_manager::application::imp> - [DEBUG] - GtkApplication<EpicAssetManager>::startup
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<egs_api::api::types::download_manifest> - [ERROR] - No header magic
<epic_asset_manager::ui::widgets::download_manager> - [ERROR] - Unable to remove chunk file: No such file or directory (os error 2)
<epic_asset_manager::ui::widgets::download_manager> - [ERROR] - Unable to remove chunk file: No such file or directory (os error 2)
<epic_asset_manager::ui::widgets::download_manager> - [ERROR] - Unable to remove chunk file: No such file or directory (os error 2)
<egs_api::api::types::chunk> - [ERROR] - No header magic
thread 'File Pool' panicked at 'called `Option::unwrap()` on a `None` value', src/ui/widgets/download_manager/asset.rs:1067:26
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
<epic_asset_manager::ui::widgets::download_manager> - [ERROR] - Unable to remove chunk file: No such file or directory (os error 2)
<egs_api::api::types::chunk> - [ERROR] - No header magic
thread 'File Pool' panicked at 'called `Option::unwrap()` on a `None` value', src/ui/widgets/download_manager/asset.rs:1067:26
<epic_asset_manager::ui::widgets::download_manager::asset> - [ERROR] - Failed to start chunk download, trying again later: error sending request for url (https://epicgames-download1.akamaized.net/Builds/Rocket/Automated/CitySample/CloudDir/ChunksV3/15/15C21EC8CBCAD2FB_79D23BD24883F18061C948B6A0169DC6.chunk): error trying to connect: dns error: failed to lookup address information: Name or service not known
thread 'main' panicked at 'cannot sample empty range', /run/build/epic_asset_manager/vendor/rand/src/rng.rs:134:9

Most of those errors are not errors, the Panic is the concerning one(that is the actual crash. I will take a look at it. The City Sample was a complicated assets since the introduction of it so I am not even surprised it fails.
Good news is that if you hit download again it should resume and you might hit a different mirror and finish the download(the mirror selection is somewhat random).

Resuming the download works but I had to do it over 10 times for the download to finish. When the download was finally over, I encountered another minor bug which I will describe in another issue.

Took a look at this, it is slightly concerning considering where it fails. I will need to download the asset myself and see the structure of the Chunk that failed. Something is wrong and it is not really clear what. Fixed the crash in an upcoming commit.


add tags and loading state support

Adding a command getMetaTags that will return all imeta tags present in this extension.

Adding a loading state, handy with immediateUpload where you want your file to be uploaded before allowing a submit.

Recomputing the node position instead of using the pos parameter in updateNodeAttributes, when mixing this extension with other, the pos attribute sometimes was incorrect.

I am no sure why we need getMetaTags or tags on storage function as I already put the tags on each file, concatenating all of them seems a not so useful abstraction

I am no sure why we need getMetaTags or tags on storage function as I already put the tags on each file, concatenating all of them seems a not so useful abstraction

oh yes absolutely!
Actually I only used the storage tags, but then realised I could not track which are deleted and added this getMetaTags action. I'm using just that now

I mess up and forgot to commit some error handling, can you rebase?

I am also not so sure about the loading flag, it doesn't help much the control flow as we still need to listen to the onComplete callback to make sure if all files were upload correctly, ideally the uploadFiles command would just return a promise but we can't, tiptap/prosemirror give us very little to no guidelines on how to properly do async stuff within the editor, since the loading flag can't be used as state on react or svelte, i think it's better for users control their own state by using the onStart, onDrop and onComplete callbacks, how you using the flag?

the onComplete trigger when using uploadFiles.
If you are using selectFile and immediateUpload you would have no way to know if the upload is ongoing or not.
To be honest, the loading state is a svelte store in my use case. Here I can't make any assumption so loading can be useful fo you to cancel a submit if some files are still loading.

I just got some interesting improvements here.
I moved the Uploader instance to the storage, then we could trigger a upload and receive a proper promise, and also have more flexibility on access code on the Uploader class.
const handleUpload = async () => {
  try {
    const files = await editor.storage.fileUpload.uploader.start()
  } catch (error) {
    console.log(error)
  }
}

You were right about the onComplete, It wasn't working with immediateUpload true, I'm trying to get it all working at once.
My apologies for making more conflicts in this pull-request, I would suggest you to not push any further changes until I push this, sorry.

Just pushed new improvements and fixes.

onComplete now should trigger on both uploadFiles and immediateUpload true.
Now you can trigger upload with await editor.storage.fileUpload.uploader.start() which returns a promise with the node attributes, but it will throw an error if any file were rejected.
I removed files from storage in favor of getFiles() which get's you the current state of the files in the editor.

let me know what you think.

yes this should cover all use cases

For another PR, having all the tags present in the current editor content from a single command would be amazing too. ie nprofile, imeta tags, all sort of tags to be added in a Nostr event.

@cesardeazevedo we have some code you can borrow if you want to: https://github.com/coracle-social/flotilla/blob/master/src/lib/editor/util.ts

Thanks, just added getEditorTags() function. unit tests can be found here.
https://github.com/cesardeazevedo/nostr-editor/blob/main/src/__tests__/storage.test.ts#L59-L102
Hope it's correct

I think opinions differ on whether to refer to non-replaceable events using an address, my recommendation would be to do that only if the event is replaceable. I also think the a should probably be a q, otherwise the note will show up as a reply.

The a tag is from the naddr embeds, nevent embeds results in q tags, I agree naddress only makes sense for replaceable events, I embed a naddress from a kind 1 event on my tests for simplicity only

Ah, ok I see. I think my second comment still applies.

Hm, not so sure if I understood, how can we embed naddress as q tags? or it's possible to do ["q", "30023:<pubkey>:<identifier>]?

I don't think there's a convention for it right now. Flotilla uses a q tag of the same form as the event id-based version. It might be more correct based on the current spec (which I edited a week or two ago) to convert the address to an event id for the q tag, but in some situations that may not be possible. So I think ["q", "address", "relay"] is probably the way to go.


implement brand updates

What this does
This update implements the minimum changes expected to stay consistent with incoming brand guidelines:

adds new brand color palettes and typography
applies updated colors to app shells, buttons, form inputs, and some select components

Notes

The use of font-display is entirely optional and should be used sparingly.  It works well for large headings in CTA banners and should typically be set alongside font-extrabold with a line-height that is the same or very close to the font size.
The default color weight of xy-green changes from a 600 to a 300 and does not meet accessibility requirements as text.  It should primarily be used as a background color with text-xy-black or a similar 800+ color weight.  Some exceptions will apply to icons or other presentation role interface items where legibility is less of a concern.
The spinning indicator has a heavy branding baked into it.  It has been prepped to deploy alongside logo changes at a later date.  It will for a short period maintain old brand colors and marks.

To Preview In An Existing App
npm install @xy-planning-network/trees@dev
npm install --save-dev tailwindcss@latest
If You Want to Demo What A Full Gray Replacement Would Look Like
Modify the apps tailwind.config.js like so:
/* eslint-disable */
const colors = require("tailwindcss/colors")
const defaultTheme = require("tailwindcss/defaultTheme")
const treesConfig =<EMAIL_ADDRESS>const config = {
  ...treesConfig,
  content: [...treesConfig.content, ...["./src/**/*.vue"]],
}

config["theme"]["extend"]["colors"]["gray"] = colors["neutral"]

module.exports = config

Breaking Changes

xy-btn-green has been removed
xy-navy has been removed (portal is likely to continue to stub this into its config to ease the refactor)
xy-dawn has been removed
xy-lime has been removed

Major Considerations

xy-green is a very different default color, best to review existing uses

@profsmallpine There's a @dev version up if you want to see how this feels in Archive.  I left details in the PR note about setting the neutral palette in the app config as well.  I could use a second opinion on whether not we do a full replacement and tweak as needed, or leave it intact and implement neutral via search and replace or as we see fit here and there.
Ergonomically, it's quite a bit nicer to type bg-gray, text-gray etc.  Though autocompletion does most of the work anyway on text-neutral.


Detect and handle bad keys in yaml definition.

@joachimmetz

Style nit otherwise looks fine. When you fix the style I'll have another look.

LGTM


Enable-PodeOpenApi should allow for alias in iis

Describe the Change
As a developer, I want Enable-PodeOpenApi to support swagger servers so that i can use an IIS alias application folder like https://pode.domain.com/myService.
I don't know what you want to call it, but maybe something like this:

Enable-PodeOpenApi -Title 'Data Warehouse' -Version <IP_ADDRESS> -RouteFilter '/api/*' -alias '/myService'
or
Enable-PodeOpenApi -Title 'Data Warehouse' -Version <IP_ADDRESS> -RouteFilter '/api/*' -servers '/myService'

Related Issues
When using the 'Try it out' button, swagger will does not know there is a server alias.
Additional Context
Example of a good open api configuration file:
{
  "openapi": "3.0.1",
  "info": {
    "title": "Data Warehouse",
    "version": "v1"
  },
  "servers": [
    {
      "url": "/myService"
    }
  ],
  "paths": {
    "/api/test-kerberos": {

This would then build the url with the alias in the request url when using swaggers try it out button.
No alias
Example of Pode's open api file:
	"openapi": "3.0.2",
	"info": {
		"title": "Self Service Catalog API",
		"version": "<IP_ADDRESS>"
	},
        "paths": {
		"/api/test-kerberos": {

Hi @Omzig,
Are you referring to running Pode as an Application under another IIS Website? If so, Pode actually supports this already, and uses the path during route matching. It just needs appending on the OpenAPI URLs 😃

Hi @Omzig,
Are you referring to running Pode as an Application under another IIS Website? If so, Pode actually supports this already, and uses the path during route matching. It just needs appending on the OpenAPI URLs 😃

Can you tell me what documentation i missed to include the the proper routing in swagger?
Start-PodeServer -StatusPageExceptions Show -Threads 10 {
Add-PodeEndpoint -Address localhost -Protocol Http
Add-PodeAuthIIS -Name 'IISAuth' -Sessionless

Add-PodeRouteGroup -Path '/api' -Routes {
        Enable-PodeOpenApi -Title 'Self Service Catalog API' -Version <IP_ADDRESS> -RouteFilter '/api/*'
        Enable-PodeOpenApiViewer -Type Swagger -Path '/swagger' -DarkMode -OpenApiUrl '/myService/api/openapi'

        # Test Kerberos authentication
        Add-PodeRoute -Method Get -Path '/test-kerberos' -Authentication 'IISAuth' -ScriptBlock {
            Write-PodeJsonResponse -Value @{ User = $WebEvent.Auth.User }
        }
}

Hi @Omzig,
Sorry, it's not that you've missed some in the docs, it's that Pode does have support for IIS Application paths, but that logic happens to be missing from Pode's OpenAPI logic. So it'll need adding internally.

So it'll need adding internally.

Thank you for checking on that, can we fix this in the next release? Please and Thank you!


v0.26.3 release: normalizes exact word

release-v0.26.3 is originated from v0.26.2 and not from main
Add the commits of

https://github.com/meilisearch/milli/pull/505
https://github.com/meilisearch/milli/pull/514

And update the version in every concerned files
The tag v0.26.3 will be created once this PR is merged and will be done on the release-v0.26.3 branch

bors try

It's ok kero, you can approve!

Waiting for https://github.com/meilisearch/milli/pull/514 to be merged into main. Once it's done I will put the commit of this PR into this branch in order to release v0.26.3

bors try

I've added the https://github.com/meilisearch/milli/pull/505 commit, can you approve again @Kerollmops?

bors merge


Updated table drag-n-drop logic

now the file can be dropped everywhere
"unfiled files" instead of "root" in popups
old stuff removed

:+1:


Fix rake db failing when no database exists yet

Nice, the same should be applied to refresher too 👍

@KrauseFx https://github.com/fastlane/refresher/pull/18


Migrate to manifest v3

Blocked by tab capture API support in manifest V3. See https://bugs.chromium.org/p/chromium/issues/detail?id=1214847

https://github.com/GoogleChrome/chrome-extensions-samples/issues/627


Add option to include variant error messages when deserializing untagged enum fails

Currently, when the deserialization of an untagged enum fails, the error message contains no information why this is the case. It would be really helpful if there'd be an option to include the error message of each variant.
So instead of
data did not match any variant of untagged enum <ENUM>

something like
data did not match any variant of untagged enum <ENUM>. Errors:\n<ERR1>\n<ERR2>

Duplicate of #773.


Stop words and string comparison

In spelunking through the code, I noticed that the StopWordFilterBase is required to return a Lunr.ISet<string> and that the default Lunr.Set<T> implementation of that interface is backed by a HashSet<T> (https://github.com/bleroy/lunr-core/blob/main/LunrCore/Set.cs#L22). However, the HashSet<T> used by Lunr.Set<T> does not have the option of specifying a StringComparer.
If I'm understanding the flow correctly, that means that the StopWordFilterBase.IsStopWord(string word) method, which calls StopWords.Contains(word) by default, will be case-sensitive with no way to easily modify the behavior to be case-insensitive. That seems like it verges on a bug when it comes to stop words, which I think are generally expected to be case-insensitive.
In general, the requirement for StopWordFilterBase implementations to expose a Lunr.ISet<string> StopWords property seems like it should be an implementation detail given the actual determination of a stop word goes through the StopWordFilterBase.IsStopWord(string word) method.
I've got two concrete suggestions:

Add overloads to the Lunr.Set<T> implementation similar to the HashSet<T> constructors that allow specifying a StringComparer and flow that through to the backing HashSet<T>.
Remove the StopWordFilterBase.StopWords property and make the StopWordFilterBase.IsStopWord(string word) method entirely virtual.

That would also require some changes to EnglishStopWordFilter to store it's own Lunr.ISet<T>, but those would be minor. An intermediate abstract StopWordSetFilterBase could also be created to avoid changing EnglishStopWordFilter if that's desired instead.

If you're amendable to either (or both) of these suggestions, I can work up a PR. I might go ahead and work up one anyway since they're fairly minor code changes and would get me familiar with the code base.
Thanks!

Good catch, thanks! Haven't looked at the details yet, but it sounds like option 1 should be easily doable and perfectly adequate.

Cool - PR incoming!