DataActions logs

Are data operations (which are specified in the DataActions and NotDataActions properties of a role definition) logged somewhere? Where can I find it?
Thanks,
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: ad021df2-1df2-9537-6037-adc0df8cd297
Version Independent ID: 9b54361a-eabe-bb96-1b03-4d9010a48b6e
Content: Overview of Azure platform logs - Azure Monitor
Content Source: articles/azure-monitor/platform/platform-logs-overview.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@GitelGeiger Thanks for your comment! We will review and provide an update as appropriate.

@GitelGeiger All the data operations will be logged in AzureActivity table of log analytics.

@SwathiDhanwada-MSFT just to clarify, is it something that's already happening, or future-planned?
I'd also like to ask if AzureActivity table of log analytics contains logs not found in the Activity Log of Azure Portal?
I'm asking because I don't see these Data Operation logs in the Activity log, and they're not indexed for operation filter.

Apologies for my previous comment @GitelGeiger . To get insights on the operations that are performed at data plane, you need to enable diagnostic logs for the azure resource. You can view the logs based on type of collection you have set
Azure diagnostics - All data written is to the AzureDiagnostics table.
Resource-specific - Data is written to individual table for each category of the resource.

@GitelGeiger Apologies for my previous comment. I presumed it is logged in Activity Log as I tested for Storage Account.  I verified with engineering team, if the request goes to ARM , then it is logged in Activity Logs. Otherwise, it is up to
Resource Provider to implement logging in the resource logs. Currently, there is no document available which gives you clear idea where Data Actions of azure resources will be logged.  However, engineering team is working on indexing these logs and there isn't any ETA as of now.

@GitelGeiger I would recommend you to navigate here and share your feedback or suggestions directly with the responsible Azure feature team and clicking the vote button of your suggestion to raise visibility and priority on it.
We will close the issue for now. If there are further questions , please revert and we will be glad to assist you.

Thank you


Seperate examples

@Rahat-ch Tried the new version, find some problems:

Fix the env examples Morph -> MORPH problem, this caused when deploy the url identified as invalid.
Hardhat examples works for me now.
I can't forge build Foundry because it says missing forge-std file, probably remapping problems in config file.
So i can't make foundry example work, please check to fix thanks!


Convert iDynTree objects in FloatingBaseEstimators to manif/Eigen types in the API

With the changes introduced in https://github.com/robotology/idyntree/pull/736/files and possibly an extension to also the iDynTree::Model class, we need to reflect the changes in FloatingBaseEstimators, so that the public API is free of iDynTree objects.
Instead, we will rely on manif and Eigen types.

iDynTree objects to be changed here

https://github.com/dic-iit/bipedal-locomotion-framework/blob/216d2b0503302d8628129490406cb9652a77b0a2/src/Estimators/include/BipedalLocomotion/FloatingBaseEstimators/FloatingBaseEstimatorIO.h#L48


[prim_xilinx_ultrascale_pad_wrapper] Implement physical input disable using I/O buffers

prim_xilinx_ultrascale_pad_wrapper supports input disabling (since PR #23042) but for bidirectional pads currently implements it with a logic assignment. (For input-only and analog pads, physical input disable is implemented with an IBUF_IBUFDISABLE primitive.) For bidirectional pads, there is an IOBUF_DCIEN primitive for pads in high-performance banks and an IOBUF_INTERMDISABLE primitive for pads in high-range banks, which both have an IBUFDISABLE input. To instantiate the correct primitive, prim_xilinx_ultrascale_pad_wrapper needs to know which bank the pad is connected to, and currently this information is not available to that module.
To resolve this issue, the information about the I/O bank of a pad needs to be passed to prim_xilinx_ultrascale_pad_wrapper (and for that, the information probably needs to move from SDC files to an Hjson), and then prim_xilinx_ultrascale_pad_wrapper needs to instantiate the correct primitive based on the I/O bank parameter.
Note that this issue is specific to UltraScale devices, whereas issue #23094 is specific to 7 Series devices.

Putting into backlog until we see the need to implement this.


ERROR: UndefVarError: DiscreteVariable not defined

julia> include("setup.jl")
9-element Array{ByteString,1}:
 "/Applications/Julia-0.4.5.app/Contents/Resources/julia/local/share/julia/site/v0.4"
 "/Applications/Julia-0.4.5.app/Contents/Resources/julia/share/julia/site/v0.4"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/src"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/src/GraphPlot"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/src/HMM"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/src/PotentialInference"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/Demos"
 "/Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine"

julia> demoBurglar()
ERROR: UndefVarError: DiscreteVariable not defined
 in demoBurglar at /Users/zhuxiaohu/projects/Julia0p4ProbabilisticInferenceEngine/Demos/demoBurglar.jl:9

julia> using brml

julia> demoBurglar()

ERROR: UndefVarError: DiscreteVariable not defined
totally lost. :(

The cwd must be the folder where you cloned the repo, since brml.jl uses cwd relative paths. I've managed to use the code as a library by  replacing the cwd() calls in brml.jl to Base.source_path() calls, but there are still some functions that are not correctly exported.


Ensure that we set the auth token on signIn/createEcosystem/loginConfirm

We should set the authToken on login in every langauge. We do it some places, we need to do it everywhere. This will make things easier for our users.
Languages:

[x] .NET
[x] CLI
[ ] Go @geel9

SignIn and LoginAnonymous already implemented; LoginConfirm not implemented

[ ] Java @geel9
[ ] Dart
[x] Python
[ ] Ruby

SignIn implemented; LoginConfirm and LoginAnonymous not implemented

[ ] Swift
[ ] Typescript

We also need to ensure that we don't cache old copies of the token when using trinsic_service. This might require us to add support for the .WithToken() function call?


Add publish options for commonjs modules

Change babel with babel-cli to compile the code

remove the extensions from the require calls, to be able to import
those files in /lib.

fixed some lint errors

add /lib for commonjs modules and require calls

@randallknutson can you test this and check that everything is working ok?


[bazel] Unable to use workspaces with underscores in name?

I am debugging a performance regression in ROS 2 Debians.
As I'm doing this, I'm using timeshift to backup / restore things, so I have an "old and good" install of /opt/ros/humble, and a "new and bad" install of /opt/ros/humble.
I want to swap back and forth by changing the workspace path.
On latest main (955245e), the following works:
cd drake-ros/ros2_example_bazel_installed
export ROS2_DISTRO_PREFIX=/opt/ros/humble
bazel fetch @ros2//...

However, the following fails on latest main (955245e):
sudo ln -s /opt/ros/humble /opt/ros/humble_old
cd drake-ros/ros2_example_bazel_installed
export ROS2_DISTRO_PREFIX=/opt/ros/humble_old
bazel fetch @ros2//...

The failure states:
...
        File "{output_base}/external/ros2/BUILD.bazel", line 11, column 16, in <toplevel>
                share_filegroup(
        File "{output_base}/external/ros2/tools/common.bzl", line 8, column 45, in share_filegroup
                srcs = [path for path in native.glob(
Error in glob: pattern cannot be absolute

Looking at $(bazel info output_base)/external/ros2/BUILD.bazel, I see this around line 11:
share_filegroup(
    name = "ament_cmake_core_share",
    share_directories = ["/opt/ros/humble/share/ament_cmake_core", "/opt/ros/humble/share/ament_index"],
)

Somehow, we're just ignoring the user specified directory?

Ah, from looking at code, looks like it's from CMAKE_PREFIX_PATH, which may be baked into the installation.


suggestion, add command to create voucher codes easy.

for example:
/voucher create CODE CodeType LimitToUse
/voucher create ABC123 redeemMoney 1
the user can use the code /voucher redeem ABC123 and obtain command of redeemMoney type, and just one use.

Added to tracker.


MFA requirement breaks m365 login

I set up a new tenant today Microsoft 365 Developer Program and if I understand it correctly, all accounts seem to require MFA set up on them after 14 days.
m365 login with username and password doesn't support MFA and I wondered if you had any recommendation from the other m365 login authentication options e.g. cert, app registration, managed identity etc.

Not at this time... unless something changed, you can't deploy apps via apps or MSI's yet. I've seen where someone wrote a blog post about using certs, but I've yet to see it work.
AFAIK, using a service account without MFA enabled is the only option today.

Thanks, Andrew


Add Google Analytics to generated documentation

Signed-off-by: Michael Hall<EMAIL_ADDRESS>

@mhall119 Rebase this PR branch and we can pull it in.

@tsconn23 rebase done


Added support for array index and corresponding unit test

That was trivial and a need on my side.

Thanks for your contribution. I'll try to push a new release to nuget soon.

I just released a new version. It would be great if you could check that your fixes works now :)

Thanks, I was able to test that the new package includes arrays and I removed the temporary dependency on my build.
By the way, your tool was invaluable in supercharging what I think is a cool project: linq to z3.
This is my tree with your last addition of binary array indexes used in the meal planner, which is a draft project for my students.
However, the partial evaluator and interpretor are used in several places all over the theorem engine to maximise the types of lambdas that can be ultimately translated into Z3 constraints to be solved or optimized.
My upgraded version of Sudoku and MissionariesAndCannibals are probably good entry points to the flexibility that your library brought to the original prototype.
Would you mind having a look and tell me whether you think this is the correct approach to integrating your tool and if you can think of other magic it could add?

That's a cool project. I wasn't aware of Z3 being that open by now.
Sure, I'll have a look at the integration. @Khazuar might also be curious to see how this is being used now.

Thanks for your comments.
I believe I have addressed your first suggestion about scoping as can be seen in the Theorem class and illustrated in my upgraded version of Sudoku. I found out about your library when I was specifically trying to solve that issue, and I'm very glad I did.
The result is much less verbose than the original Sudoku example, for which the tedious manipulation of Expressions kind of takes away the original interest for the provider.
Now I'm not sure to understand your second suggestion about inlining. I think this is the place where I wasn't so sure how to best further integrate your lib to handle mixed Member expressions. I only took care of a few cases with collections array indices,  indexed properties and Enumerable functions. I wouldn't be surprise if you have got better suggestions to cover a whole range of LINQ uses here.
Also, what is the provider you originally wrote that lib for?


NotFoundError when load model from different device

i got an error when load model using keras.models.load_model from different devices. This is my complete code:
from tensorflow import keras
model_path = '/content/drive/MyDrive/saved_model/my_model'
imported = keras.models.load_model(model_path)
I got an error like this:
NotFoundError: Op type not registered 'SimpleMLCreateModelResource' in binary running on 135bfc4bd927. Make sure the Op and Kernel are registered in the binary running in this process. Note that if you are loading a saved graph which used ops from tf.contrib, accessing (e.g.) tf.contrib.resamplershould be done before importing the graph, as contrib ops are lazily registered when the module is first accessed.
The training and evaluation was completed successfully

Thanks for the report.
Solution
Can you try importing TF-DF before loading the model?:
import tensorflow_decision_forests as tfdf
from tensorflow import keras

model_path = '/content/drive/MyDrive/saved_model/my_model'
imported = keras.models.load_model(model_path)

Explanation
TF-DF relies on a couple of TensorFlow custom ops. These ops need to be loaded manually (which happen when importing TF-DF).

Its working!! Thanks

Hello @achoum, i am getting the same errror when i save the model in python using model.save and then load the model using  tensorflow C API. Can you please let me know if we can use tensorflow decsion forest model in C API?


[11.x] fix: PHPDoc for loading nested relations

Hello!
See https://github.com/laravel/framework/pull/52754#issuecomment-2346033141
All of these relations can be recursive definitions, so I just added the array type as a stop-gap measure for the time being.
Thanks!

Thank you sir!


Jupyter workflow demo in Makefile

What is the goal / desired outcome?
One-click setup of a Jupyter workflow demo
If there is a way to accomplish this today via workaround, what does that require?
(Optional) What is your proposal for a feature to solve this?

Dupe of https://github.com/pachyderm/pachyderm/issues/3681


[Suggestion] Backup world with temporary file extension and rename after

I've run into situations where the app crashes during a backup for some reason or another (no logs, sorry...) and I think it would be a good idea to backup to a temporary filename and then rename on success to the final name.
This would allow looking at a list of files and knowing which ones were incomplete right off the bat and would also prevent the trimmer from potentially trimming good backups and leaving an incomplete one instead.
So maybe something like My World - Redux.20220302-112042.mcworld.inprogress and just strip off the .inprogress extension upon successful backup.

I like this. Let me see what I can do.

This just got merged, so should show up in the test tag soon.


[Bug Report] Dockerfile fails to build from master and v0.26.2 tag on Windows

Describe the bug
Attempting to build a docker image fails with error:
#13 100.8 Using sanitize (0.1.0 at e09520e)
#13 100.8 crystal build src/mango.cr --release --progress --static --error-trace
#13 101.0 In src/mango.cr:1:1
#13 101.0
#13 101.0  1 | require "./config"
#13 101.0      ^
#13 101.0 Error: while requiring "./config"
#13 101.0
#13 101.0
#13 101.0 In src/config.cr:50:58
#13 101.0
#13 101.0  50 | puts "The config file #{cfg_path} does not exist. " \
#13 101.0                                                            ^
#13 101.0 Error: unknown token: '\r'
make: *** [Makefile:18: static] Error 1

To Reproduce
Clone the repo and run docker build -t XYZ .
Environment (please complete the following information):

OS: Windows 11, using Docker with WSL 2 backend
Mango Version: master, v0.26.2

Docker (if you are running Mango in a Docker container)

I've copied the env.example file and renamed to .env, no modifications

Additional context
I'm attempting to build the image myself so I can add an EXPOSE 9000 line to the Dockerfile; explicit EXPOSE is needed in the docker image for Traefik which auto-detects ports for containers and exposes them behind HTTPS.
I've not tried older tags.

Windows line breaks strike again :) Can you give https://github.com/hkalexling/Mango/issues/306#issuecomment-1147999115 a try?

@hkalexling Thank you for that note and quick response, I've instead cloned and built the repo using Ubuntu on Windows (instead of via Windows terminal) and I was able to build the image without issue


Card padding changes when RHP is opened

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.80-5
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @dubielzyk-expensify
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1717646276509929
Action Performed:

Go to "Profile" (Avatar in the bottom navigation)
Click "Preferences"
Click "Language"

Expected Result:
RHP (right hand panel) should slide out
Actual Result:
RHP slides out, but the card under the scrim (overlay) changes its padding.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/95152228-60e4-4937-b0cb-df128b92b926

https://github.com/Expensify/App/assets/38435837/a2cf1ed1-c4dc-40a7-9d86-41447f579765

View all open jobs on GitHub

Looks like it's already reported here https://github.com/Expensify/App/issues/43031

Seems like it's been fixed already


Release 2024-09-03

#790
#791
#792
#793

:tada: This PR is included in version 6.47.1 :tada:
The release is available on:

GitHub release
v6.47.1

Your semantic-release bot :package::rocket:


Prototype - Explore the possibility to have page templates

What would that feature address
When creating a new application, developers usually design it iteratively, page by page. In such cases, the application templates ( nor the blueprint templates) don't help very much.
The purpose is to have the capability to scaffold pages on top of an existing application (or not).
The following questions must have an answer:

How to insert new modules in existing applications?
How to modify existing files?
How to handle the singleton configuration if it exists?
How to manage the application version?

Acceptance Criteria

[x] Create a prototype to evaluate the technical feasibility.
[x] Append the requirement document

There is a prototype which shown that the feature is feasible in either community or enterprise version.
We need to discuss about which template to add to this feature and where the page is located.

After discussion with the CS team, we agree that there are some page templates that can have some value in enterprise.
There will be an Epic created to follow up with the discussion.


Hero component - macro issues

Writing jest tests for the hero component highlighted a number of issues with the macro:

~The parameter wide is not documented in macro options.~
We should move away from using things like censusTheme and just use something like branded: true and let css theme settings handle the look and feel.
Also...

{% if params.censusThemeDark is defined and params.censusThemeDark %}
    {% set skinName = "dark" %}
{% else %}
    {% set skinName = "light" %}
{% endif %}

The above adds a suffix string to the image src of a pre title image. We should update the variable keys to make this clear. Or implement as a ternary inline in the src (perhaps).

First point fixed in #2126

Branding has been removed as part of wider piece of work


Cannot install flatpaks

im on fedora 38
flatpak version: 1.15.4-1
doing a quick reinstall didn't work
Warning: While pulling runtime/com.github.Eloston.UngoogledChromium.Codecs/x86_64/stable from remote flathub: Writing content object: Need more input
Error: While pulling app/com.github.Eloston.UngoogledChromium/x86_64/stable from remote flathub: Writing content object: Need more input
error: Failed to install com.github.Eloston.UngoogledChromium: While pulling app/com.github.Eloston.UngoogledChromium/x86_64/stable from remote flathub: Writing content object: Need more input

seems like other are having the same issue on the fedora subreddit, ill close this since a fix is expected soon


HTML can't be beautified

Description
HTML won't beautify due to error
Input Before Beautification
This is what the code looked like before:
<html>
<head><meta generator="asimov.js">   <meta http-equiv="content-type" content="text/html;charset=UTF-8">   <meta name="viewport" content="width=device-width; initial-scale=1.0; user-scalable=no">   <link rel="shortcut icon" type="image/x-icon" href="/site/images/favicon.ico">   <meta name="p:domain_verify" content="6dee83ae94807c7e2d7a14084a73a176">   <meta name="apple-itunes-app" content="app-id=406644151">   <meta name="google-site-verification" content="Ra-CDN32q8k5C3YzXgNxRj75pvM1qyp5UXna3XJ5qHo">   <meta property="fb:app_id" content="208559595824260">   <meta property="fb:admins" content="513485681">   <meta property="og:title" content="Wunderlist | To-do list, Reminders, Errands - App of the Year!">   <meta property="og:type" content="website">   <link rel="stylesheet" href="https://uhf.microsoft.com/mscc/statics/mscc-0.1.12.min.css">       <meta property="og:image" content="//dr0wv9n0kx6h5.cloudfront.net/664cb69d34d0ef040ff8a446e429bce8feb54b41/site/images/logo-big.png">     <meta property="og:image:type" content="image/png">     <meta property="og:image:width" content="265">     <meta property="og:image:height" content="265">   <title>Wunderlist | To-do list, Reminders, Errands - App of the Year!</title>   <link href="site/styles/font_lato_default.css" rel="stylesheet" type="text/css">   <link href="site/styles/main.css" rel="stylesheet" type="text/css">   <link rel="alternate" href="https://www.wunderlist.com/" hreflang="en"><link rel="alternate" href="https://www.wunderlist.com/de/" hreflang="de"><link rel="alternate" href="https://www.wunderlist.com/ja/" hreflang="ja"><link rel="alternate" href="https://www.wunderlist.com/zh/" hreflang="zh"><link rel="alternate" href="https://www.wunderlist.com/zh_TW/" hreflang="zh-TW"><link rel="alternate" href="https://www.wunderlist.com/ru/" hreflang="ru"><link rel="alternate" href="https://www.wunderlist.com/es/" hreflang="es"><link rel="alternate" href="https://www.wunderlist.com/fr/" hreflang="fr"><link rel="alternate" href="https://www.wunderlist.com/pt/" hreflang="pt"><link rel="alternate" href="https://www.wunderlist.com/ko/" hreflang="ko">  <style>@media print {#ghostery-purple-box {display:none !important}}</style><script async="" type="text/javascript" src="//6wkblog.disqus.com/count.js"></script><style type="text/css">.fancybox-margin{margin-right:17px;}</style></head>

Expected Output
The beautified code should have looked like this:
Nicely formatted HTML that I Can't get

Actual Output
The beautified code actually looked like this:
SAME AS ORIGINAL CODE

Steps to Reproduce

Add code to Atom editor
Run command Atom Beautify: Beautify Editor
Receive error

Debug
Im not going looking for the file sorry
Checklist
I have:

[x ] Tried uninstalling and reinstalling Atom Beautify to ensure it installed properly
[x ] Reloaded (or restarted) Atom to ensure it is not a caching issue
[x ] Searched through existing Atom Beautify Issues at https://github.com/Glavin001/atom-beautify/issues
so I know this is not a duplicate issue
[x ] Filled out the Input, Expected, and Actual sections above or have edited/removed them in a way that fully describes the issue.
[ ] Generated debugging information by executing Atom Beautify: Help Debug Editor command in Atom and added link for debug.md Gist to this issue

When you use the debug feature the file should be copied into your clipboard so that you can just paste the contents.  We really need that information to look into this problem.  For instance this project contains multiple beautifiers for HTML and if we do not know which one we cannot know where to start troubleshooting.


docs: export KUBECONFIG for cilium-cli with k3s

Previously, commit 606b5fe9f49f ("docs: KUBECONFIG for cilium-cli with
k3s") and commit 606b5fe9f49f ("docs: KUBECONFIG for cilium-cli with
k3s") changed cilium install commands to set KUBECONFIG explicitly.
However, successive cilium-cli commands (e.g. cilium status or
cilium connectivity test in the getting started guide) will need the
corresponding kubeconfig as well. Thus, suggest to export KUBECONFIG
at the top of the k3s specific guides.
For https://github.com/cilium/cilium-cli/issues/696

LGTM. I'm looking at the dedicated k3s doc and wondering if we even need a dedicated k3s page if k3s is one of the supported tabs under the main "k8s-install-default" page... but that's unrelated to the goal of this PR.

Good point about the dedicated k3s doc. It looks like most of it is already covered in the k3s-specific part of the quick start guide. It looks like the only part missing is the step to install additional k3s nodes which don't looks specific to Cilium, so one could argue this is already covered by the k3s upstream docs.


Mitigate workQueue race condition (#240)

Move workQueue length check out of setTimeout
Quickly store a reference to workQueue[0] in a local variable in case workQueue[0] changes

Running Async tasks more frequently or increasing the setTimeout delay in onComplete increases the frequency of the exception described in #240, which suggests that the bug is indeed caused by workQueue[0] being overwritten with a proceeding Async task before runTask() can run.
I haven't observed the exception with this patch in place, even under extreme conditions with Async tasks being run one after another, milliseconds apart. Without the patch, there is a blizzard of exceptions.
Is this enough to fix the race condition? Not sure exactly how async JS works.

Great sleuthing!
Based on this, I think the current issue comes from a bad interaction between onComplete and register. Before, it was possible for a task to complete in onComplete bringing the number of active tasks down to zero. Then register would add a task and start running it. Then the setTimeout would run, checking if there's more work to do, which now there is, and then starts running the first task again. This repeats all the work until it gets to where the first run paused, causing the crash.
That's my understanding at least. And under that theory, your fix should avoid this.


Update dynamodb to 2.16.37

Updates software.amazon.awssdk:dynamodb from 2.16.5 to 2.16.37.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk", artifactId = "dynamodb" } ]

labels: library-update, semver-patch

Superseded by #89.


20NEWS dataset not available

Hi,
I wonder whether you could upload the 20NEWS dataset that you ended up using, the one shared on Matthew Kusner's github is incomplete, the words are missing inside.
Also it would be nice if you could share the script you used for splitting the Gutenberg data (since you mentioned that you split the data by year).
I would appreciate your help.
Kind regards,
Sabine

Hi Sabine,
Below is the script for loading 20news data. It is based on the sklearn.datasets.fetch_20newsgroups:
def load_20news():
    from sklearn.datasets import fetch_20newsgroups
    from sklearn.feature_extraction.text import CountVectorizer
    newsgroups_train = fetch_20newsgroups(subset='train', remove=('headers', 'footers', 'quotes'))
    newsgroups_test = fetch_20newsgroups(subset='test', remove=('headers', 'footers', 'quotes'))

    tf_vectorizer = CountVectorizer(max_df=0.95, min_df=5, stop_words='english')
    X_train = tf_vectorizer.fit_transform(newsgroups_train.data)
    vocab = sorted(tf_vectorizer.vocabulary_.items(), key = lambda x: x[1])
    vocab = [v[0] for v in vocab]
    vocab_start = 1520 # removing weird words
    min_doc_length = 20 - 1 # removing short docs
    vocab = vocab[vocab_start:]
    X_train = X_train.toarray()[:,vocab_start:]
    y_train = newsgroups_train.target[X_train.sum(axis=1)>min_doc_length]
    X_train = X_train[X_train.sum(axis=1)>min_doc_length]

    X_test = tf_vectorizer.transform(newsgroups_test.data)
    X_test = X_test.toarray()[:,vocab_start:]
    y_test = newsgroups_test.target[X_test.sum(axis=1)>min_doc_length]
    X_test = X_test[X_test.sum(axis=1)>min_doc_length]

    X = np.vstack((X_train, X_test)).astype(np.int)
    y = np.concatenate((y_train, y_test))

    train_idx = np.arange(X_train.shape[0])
    test_idx = np.arange(X_test.shape[0]) + X_train.shape[0]

    return vocab, X, y, train_idx, test_idx

For Gutenberg we used 80/20 train-test split in the order of document appearance (it is not necessarily by year - just the order it appears in the dataset).

Hello again,
Thank you very much, it worked :) You may close this issue.
Best,
Sabine


Increase default text field limit of 255

Hi, can the current limit of 255 characters for the default value of a text field be increased? Thanks!

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

ID: 89a8f6f1-da40-5a76-6336-dc04ad779d85
Version Independent ID: 1c39c3fe-aa40-c8d7-d52a-3f4d967cbaa5
Content: Add and manage fields to an inherited process - Azure DevOps Services
Content Source: docs/organizations/settings/work/customize-process-field.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@FunnyFry -- Josh, thank you for your question.  You may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

Also, the product teams aren’t always watching this repository, so to make sure it gets in front of them for triage, please submit your idea here.
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


Publish to JCenter

So that Spectrum can be more easily incorporated into Maven, Gradle, and Ivy projects.

Still waiting on acceptance into JCenter proper. Bintray publishing works, so this is available at my personal repo at https://bintray.com/greghaskins/maven .

Spectrum is live on JCenter as of v0.5 (com.greghaskins:spectrum:0.5.0)


Exit current command, not the whole shell

Add possibility to exist current command (c.Stop) without exiting the whole shell(current behaviour).

I think returning anywhere in the command function gives same behaviour.
func (c *ishell.Context) {
  if someCondition {
    return
  }
  // do something else
}

In fact, I was trying to stop command whitin a goroutine. context.Stop() stops the whole shell.
Example:
...
func (c *ishell.Context) {
  go myfunc(c, mydata)
}
...
func myfunc(c *ishell.Context, mydata data) {
  if someCondition {
    c.Stop()
    return
  }
  ...
}

You should use channels if that is your use case. And you should structure the code in a way that you can terminate it and not have an isolated goroutine. i.e. myfunc(c, mydata) should be stoppable.
If you give a clearer example of what you want to achieve (or show contents of myfunc), I am willing to help with a snippet.


Separate column default constraints from EF store-generated values

Issue #9627 talked about the different possible reasons for having columns with default constraints. We discussed this some more and decided to look at adding more separation of these concepts in the 3.0 timeframe. This would make it easier to specify that a column has a default constraint without that meaning that EF should use a sentinel value to decide whether or not to include a value for the column.

Related to https://github.com/aspnet/EntityFrameworkCore/issues/701


Ender IO spamming console

I just updated to ender io  1.7.10-<IP_ADDRESS>5_beta and ender core-1.7.10-<IP_ADDRESS>_beta off of jenkins and as soon I open my world the console gets spammed with this message.
[17:29:35] [Server thread/WARN] [EnderIO]: NetworkPowerManager: Exception thrown when updating power network java.lang.ArrayIndexOutOfBoundsException: 8
This happening  several times a second.
Any reason this might be happening?

https://github.com/SleepyTrousers/EnderIO/issues/1998

maybe, it happen when updated it. maybe an outdated rf api somewhere.

Is there a way to avoid spamming log with this message? I mean, I know there is a problem, but not everybody is able to debug and fix problems like this. But the fml logs can grow to insane sizes if you spam console with error messages in every tick.
Btw, for me this is happens if Calculator mod's Flux Point/Flux plugs works with EIO cables (in my case IIRC there is a flux point dropped to a EIO cable).

Not really. this is actually the program crashing hard. It's just that we abort the termination of the server and write out a message about it.
What could be done was to add the information to the message that is needed to find out what actually happened there.

Yeah, I experienced it now, Calculator and ImmersiveEngineering does not have a protection against this and this exception is crashed my client.

After the change #2930 has gone live, please post one of the extended error outputs here.

@HenryLoenwind please inform me which build contains the fix and we will try it as soon as it is possible. I have to talk with our server owner, but I think he will be open for it.

@hron84 That'd be the next one, 418. I'll try to remember to post in here when it's made.

I'll keep this page open anyway.

having this as well with "crazypants.enderio.machine.generator.stirling.BlockStirlingGenerator@a55560e" being repeated endlessly.

Please post the log. Every tenth error message should have a full stack trace.

clarifying, fml-server-latest.log?

I need the log lines that read "NetworkPowerManager: Exception thrown when updating power network" and have a stack trace after them.

I am not finding that string anywhere in a .log I can search. My console I have to search manually.

Then you have another problem than the one this issue is about. Please make a new issue and post the part of the log you have an issue with.

will do

@Divineaspect if you see it in your client console, you should also check fml-client-latest.log on client too. It contains log messages displayed on the in-game console.


feat(#229): Add Type for ClassName

In this PR I fixed the problem with ClassName type. Now ClassName provides its type.
Also I configured 'spring-fat' integration test. Now this test can be run in many different modes.
Related to #229
History:

feat(#229): specify different profiles for all cases
feat(#229): show how we can configure invoker properties
feat(#229): configure input and output directories for all steps in spring-fat integration test
feat(#229): add one more 'spring-fat' configuration
feat(#229): addd type for ClassName
feat(#229): fix all qulice suggestions

PR-Codex overview
This PR updates the properties files and adds exception handling to the compiler methods. It also includes a new test and additional properties in the pom.xml file.
Detailed summary

Added invoker.profiles in invoker.properties
Updated invoker.profiles in invoker.jeo.opeo.properties
Updated invoker.profiles in invoker.jeo.only.properties
Added exception handling in JeoCompiler.java
Added @ToString annotation in ClassName.java
Added new test in DefaultCompilerTest.java
Updated pom.xml with new properties for disassembling, decompiling, and compiling files

The following files were skipped due to too many changes: src/it/spring-fat/pom.xml

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 8min)


Time utilities

Fix style issues
Round seconds to hours util
Deepen docstring of round_seconds_to_hours function
Test for round_seconds_to_hours
set_timezone util
utcoffset_is_aliquot_to_hours util
Use SECONDS_IN_HOUR constant instead of 3600
Fix bug in set_timezone
utcoffset_difference util

This change is

Pull Request Test Coverage Report for Build 2863

26 of 26 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.446%

Totals

Change from base Build 2862:
0.2%

Covered Lines:
2014

Relevant Lines:
2780

💛  - Coveralls

Pull Request Test Coverage Report for Build 2863

26 of 26 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 72.446%

Totals

Change from base Build 2862:
0.2%

Covered Lines:
2014

Relevant Lines:
2780

💛  - Coveralls


How to define parameter types in pairs?

Let's say I have some function x(param1, param2). If param2 is true, then param1 is a number. If param2 is false, then param1 is a string.
x("valid", false);
x(12, true);
x("invalid", true); // Expected: error
x(11, false); // Expected: error

I feel like this is easy with declare:
declare function x(string, false);
declare function x(number, true);

However, declare is for definitions of functions I am using, not functions I am defining.
Maybe something like
const x: (a: string, b: false) | (a: number, b: true) = (a, b) => etc;

But that is not working. No matter what I try, it demands that the types of parameters a and b be specified, but I do not now how to specify them in pairs. Type of A is dependent on type of B, but not equivalent to. I would know how to do <T: string | number>(a: T, b: T) to make them the same, but I am at a loss at making them related. :(
Any help appreciated? Or feature requested if it's not possible.
Most specifically, I'm doing x: (a: MyType<X>, b: true) => MySecondType<X> | (a: MyType<X & Y>, b: false) => MySecondType<X & Y>.

You can do this:
type Fn = {
  (a: number, b: true): number;
  (a: string, b: false): string;
}

const x: Fn = ((a, b) => { /* etc */ }: any);

(x("valid", false): string);
(x(12, true): number);
x("invalid", true); // Expected: error
x(11, false); // Expected: error

What does the any do?  Doing this without the any tells me that I need to define my parameters.

In that case I do not believe it is possible. The any is the type that can be inhabited by any other type and can also inhabit any other type. It is simply an escape hatch. It is similar to the mixed type which can be inhabited by any other type but cannot inhabit any other type itself.

I'm not sure I understand what Flow is expecting to be there? Why does any fix it? Is it the definition for the function? If so, why doesn't Fn work? Or (a: one) =>two | (a: three) => four? I feel like conceptually there must be a non-any value that satisfies it, no?

If you place your cursor on the x on line 6 on Try Flow you can see the type it is expecting at the bottom of the page:
((a: number, b: true) => number) & ((a: string, b: false) => string)

Which is an intersection of two functions and AFAIK is not a type that could actually be satisfied by flow's inference of javascript code (hence the need to cast to any beforehand).

With that definition, I have this giving me "no errors":
const x: Fn = (<T: number | string>(a: T, b: boolean): T => { return a; }: ((a: number, b: true) => number) & ((a: string, b: false) => string));

Is this valid?
And if so, for my initial case, it would be something like: (<T: X | X & Y>(a: MyType<T>, b: boolean): MySecondType<T>): (a: MyType<X>, b: false): MySecondType<X> & (a: MyType<X & Y>, b: true): MySecondType<X & Y>)
Where it accepts a type T and returns U or accepts T<X&Y> and returns U<X&Y>.
More specifically, this is a HOC accepting a Component with props X and appending props Y if the flag parameter is set.
But I can't say I fully understand what flow means by intersection of two functions or how I would know to do that in the future. Which is why I'm not sure if the code I came up with, though it gives no errors in Try Flow, is valid or what I'm looking for.

Yeah I am running into this as well.  Seems like it is definitely a bug in Flow itself.  In the end I have to use FlowIgnores because it appears to get the right type but gives errors that are impossible to get rid of.
This should be a really easy thing to do IMHO.  It is also very important.
To make a simple example of this:
function foo(b: boolean) {
  if (b) {
  	return '1'
  }
  return 1
}

const str = foo(true); // number | string

const num = foo(false); | number | string

This should know exactly what each value is.
As described above, you can intersect functions (also dont know what that is doing, but it seems to work) to get it to know the type correctly.  However it seems it will always produce errors.
[Try Flow}(https://flow.org/try/#0C4TwDgpgBAShDOBXANsKBeKAKLAjAXFMAE6IQCUGAfFPCQJYB2A5pQGTZ6EBmAhsvArUojRAFtcEYuQDcAKDndEjAMbB6Ae0ZRuGjQB4AQoVx7kEXoypcoh8oQAkAYX7J9cJKgA0tmgG85KCh6bmxcSgCggEhiCGBEYm0AcgBGJMCoAF8M2PjEqBS5bIUVLTpaEgwdPSwSMlkoAHpGiuImZhKytFExKt0NLD4BChkmlp7JYiA)
type Result = ((b: true) => string) & ((b: false) => number);

function foo<B: boolean>(b: B): $Call<Result, B> {
  if (b) {
  	return '1'
  }
  return 1
}

const str = foo(true); // string

const num = foo(false); // number

3: function foo<B: boolean>(b: B): $Call<Result, B> {
                                   ^ function call. Function cannot be called on any member of intersection type
1: type Result = ((b: true) => string) & ((b: false) => number);
                  ^ intersection
Member 1:
1: type Result = ((b: true) => string) & ((b: false) => number);
                  ^ function typeError:
3: function foo<B: boolean>(b: B): $Call<Result, B> {
                   ^ boolean. Expected boolean literal `true`
1: type Result = ((b: true) => string) & ((b: false) => number);
                      ^ boolean literal `true`
Member 2:
1: type Result = ((b: true) => string) & ((b: false) => number);
                                          ^ function typeError:
3: function foo<B: boolean>(b: B): $Call<Result, B> {
                   ^ boolean. Expected boolean literal `false`
1: type Result = ((b: true) => string) & ((b: false) => number);
                                              ^ boolean literal `false`
7:   return 1            ^ number. This type is incompatible with the expected return type of
3: function foo<B: boolean>(b: B): $Call<Result, B> {
                                   ^ string

See also: #5383


serfdom: 0.9.8 -> 0.10.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/hashicorp/serf/releases.
meta.description for serfdom is: "Service orchestration and management tool"
meta.homepage for serfdom is: "https://www.serf.io"
meta.changelog for serfdom is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.10.0 with grep in /nix/store/qpy850w8xi3b4rm5pj4254wa5dzzdds5-serf-0.10.0
found 0.10.0 in filename of file in /nix/store/qpy850w8xi3b4rm5pj4254wa5dzzdds5-serf-0.10.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
serfdom

Instructions to test this update (click to expand)

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

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

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

serfdom

Maintainer pings
cc @pradeepchhetri for testing.


org.redisson.tomcat.RedissonSessionManager.findSession Session can't be found

Hi,
I was trying to deploy simple webapp using Redisson manager on AWS Elastic Beanstalk. The redis that I have configured, is also from AWS Elasticache.
Context.xml:
<Manager className="org.redisson.tomcat.RedissonSessionManager"
          configPath="${catalina.base}/conf/redisson.yml" readMode="REDIS" updateMode="DEFAULT"/>

redisson.yml:
singleServerConfig:
  idleConnectionTimeout: 10000
  connectTimeout: 10000
  timeout: 3000
  retryAttempts: 3
  retryInterval: 1500
  password: null
  subscriptionsPerConnection: 5
  clientName: null
  address: "redis://{ClusterName}.cache.amazonaws.com:6379"
  subscriptionConnectionMinimumIdleSize: 1
  subscriptionConnectionPoolSize: 50
  connectionMinimumIdleSize: 24
  connectionPoolSize: 64
  database: 0
  dnsMonitoringInterval: 5000
threads: 16
nettyThreads: 32
codec: !<org.redisson.codec.FstCodec> {}
transportMode: "NIO"

I have moved these 2 files to conf folder of Tomcat8, and also the required JARs to lib folder. If I login to box, I can see these files correctly updated in Tomcat directory.
Still when the deployment is complete, if I try to access the application, it does not come up. Even though I am able to see the war getting deployed in Tomcat.
Upon observing the logs, I saw below :
INFO [http-nio-8080-exec-3] org.redisson.tomcat.RedissonSessionManager.findSession Session 729345036D7A8E80685303969D2DA502 can't be found
Am I making any mistake here? Redisson version is : 3.12.4

Porbably affected by this issue https://github.com/redisson/redisson/issues/2589. Try 3.12.5 version


[FEATURE] Display Session Summary / Analysis

Description
The session analysis / summary page should include all annotations created by all users. The annotations should be shown as in the session screen on a timeline per marker. The annotations of a user are grouped together and then the next user's annotations are shown underneath.
The analysis settings offer to switch on/off markers, users and sample solution.
Tasks

#239 follows up with the participants and marker switch on/off in analysis page.
@TS-Stefan can we close this having the separate issue for the managing analysis page display grid?

Missing parts such as the marker switch on/off functionality are handled in separate issues. Closing this issue, because in general the analysis page exists and can be used


Add developer documentation for the Asgardeo Android SDK.

Description:
Currently the developer documentation for the Asgardeo Android SDK is very limited, which makes it difficult for the developers to develop on the Asgardeo Android SDK. Hence it is recommended to improve the developer documentation on the Asgardeo Android SDK.

The developer documentation for the Asgardeo Android SDK can be found here

https://github.com/asgardeo/mobile-ui-sdks/blob/main/android/README.md

and the details on how to contribute and release each package can be found here

https://github.com/asgardeo/mobile-ui-sdks/blob/main/CONTRIBUTING.md


Provide a way to sign up for a free OpenShift cluster on DevSandbox

i.e. link to https://developers.redhat.com/developer-sandbox/get-started

Test adding a button from the login dialog:

@slemeur @mohitsuman any comments? other/better ideas?

Looks great to me to get that in the extension quickly!

We should also add Learn More button and point to https://console.redhat.com/openshift/create, to see all the offerings supported by Red Hat.

@mohitsuman agreed on principle but https://console.redhat.com/openshift/create access requires creating a Red Hat account 1st, so probably a no go. Need a publicly accessible URL

@mohitsuman does https://www.redhat.com/en/technologies/cloud-computing/openshift work for you?

I agree console.redhat needs RH authentication. We can use this then, https://www.redhat.com/en/technologies/cloud-computing/openshift/try-it


save() can insert two objects

First of all, thank you for this awesome lib!
Unfortunately, I've encountered an issue. Let's say you have a class defined like this:
  public class Task extends BaseModel implements Parcelable {
      @Column(columnType = Column.PRIMARY_KEY_AUTO_INCREMENT)
      public int id;
 }

If you create yourself an object with an specific id, it is inserted twice:
  Log.e("Test", "first count=" + new Select().from(Task.class).queryList().size()); // display 0
  Task test = new Task();
  test.id=23;
  test.save(false);
  Log.e("Test", "second count=" + new Select().from(Task.class).queryList().size()); // display 2 !!

In my project, I need to specify the id to restore the data from the server and keep the relations between this object and the others.

@FredJul what version of the app are you using? also, what is it doing underneath, is it updating or inserting in SqlUtils.save()? Just curious.

Hi!
I'm using the 1.8.0 version of your lib. And when I do this test, I've an empty database, so the save is inserting two items.
Do you think that's a possible thing or did I missed something?

@FredJul I would not try to keep autoincrementing column ids for previously stored data. I'd rather just make it a general primary key. No matter what happens, if you call save() on that object where id=23 does not exist, its going to insert that item into the database and get a new id value.


is it possible get after re-sized width and height?

    public function resize($path, $height)
    {
        try {

            //Image size is 480x550
            //Given Height = 200
            $img = ImageManagerStatic::make($path);
            $img->resize(null, $height, function ($constraint) {
                $constraint->aspectRatio();
                $constraint->upsize();
            });
            header('Content-Type: image/jpg');

            //Resized Output is 175x200
            echo $img->response('jpg', 70);

        } catch (\RuntimeException $e) {
            echo 'Caught exception: ', $e->getMessage(), "\n";
        }
    }

Is it posbile to get resized width and height of the given image..  which is in this case is 175x200

Do width() and height() work?
$img->resize(null, $height, function ($constraint) {
    $constraint->aspectRatio();
    $constraint->upsize();
});

$width = $img->width();
$height = $img->height();


Entity-path order is not stable with new blueprint

The real root cause of:

https://github.com/rerun-io/rerun/issues/2176

Is that the entity-path query-order can change on storing/retrieving the space-view blueprint from the store.
This allows for numerous "unstable" bugs that result in flickering, etc.
We should define a reasonable order and always use it stably.

Easy to reproduce/test of a behaviour that's likely due to this issue:


[BUG][GUI][Rancher Integration] Load Harvester UI extension on Rancher 2.10 shows temp error

Describe the bug
On Rancher 2.10, after setup harvester ui extension repo in App Repository, click Install harvester ui extension shows error.
The harvester ui extension shows below error after click Reload. But refresh the page again. The error dismiss.

The error reason
Failed to load script from 'https://localhost:8005/v1/uiplugins/harvester/1.0.0/plugin/harvester-1.0.0.umd.min.js'\n

To Reproduce
Steps to reproduce the behavior:

Setup a Rancher-2.10 (master-head)
In local -> App Repository  -> Create a harvester repo with

Git Repo URL : https://github.com/harvester/harvester-ui-extension
Git Branch : release-test or release-harvester-test

Go to extension page, click install harvester ui extension.
Click reload on top right corner.
The ui extension shows error.

Expected behavior
No error ocurred after install harvester ui extension
Ref: https://github.com/rancher/dashboard/blob/702bb57b237f253ede0946fb06de70fc9de56610/shell/plugins/plugin.js#L79

Use https://github.com/rancher/dashboard/issues/12592 to track this error.


PDF View

Find a way to add a PDF view for blueprints, so that we can print as plain document with elements full body and footer displaying.

I added print view, which we can use to generate a PDF -- not sure the PDF view is needed, and print view might be enough for now.


Add docker deploy

Optimize deployment methods to better enhance ease of deployment and stability.

Thanks for your contribution.


[BUG] load annotations exception while deleting labels

Describe the problem
load annotations results in an exception
Code to reproduce issue
>>> gs =fo.load_dataset('test')
>>> gs.list_annotation_runs()
['test1', 'test12']
>>> gs.load_annotations('test12')
Downloading labels from Label Studio...
Download complete
Loading labels for field 'ground_truth'...
Traceback (most recent call last):
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/core/odm/database.py", line 807, in bulk_write
    coll.bulk_write(batch, ordered=ordered)
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/pymongo/_csot.py", line 107, in csot_wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/pymongo/collection.py", line 569, in bulk_write
    bulk_api_result = blk.execute(write_concern, session)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/pymongo/bulk.py", line 576, in execute
    return self.execute_command(generator, write_concern, session)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/pymongo/bulk.py", line 446, in execute_command
    _raise_bulk_write_error(full_result)
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/pymongo/bulk.py", line 157, in _raise_bulk_write_error
    raise BulkWriteError(full_result)
pymongo.errors.BulkWriteError: batch op errors occurred, full error: {'writeErrors': [{'index': 0, 'code': 2, 'errmsg': 'Cannot apply $pull to a non-array value', 'op': {'q': {'ground_truth.detections': {'$exists': True}}, 'u': {'$pull': {'ground_truth.detections': {'_id': {'$in': [ObjectId('65578a3b7499471472bb40e1'), ObjectId('6551158dfd0a50d1decf13be'), ObjectId('65bd13ddb36066900d2c2d9b'), ObjectId('65511598fd0a50d1decf3849'), ObjectId('6557445e59d3f439ba79cd5d'), ObjectId('65bd1531500399d9ed879671'), ObjectId('65577f7f66df65c20c6de81f'), ObjectId('65bd13ddb36066900d2c2cfc'), ObjectId('65bd13ddb36066900d2c2cfb'), ObjectId('65bd13ddb36066900d2c2cfd'), ObjectId('6551158dfd0a50d1decf13ba'), ObjectId('65bd13deb36066900d2c434a'), ObjectId('65bd13ddb36066900d2c2d0d')]}}}}, 'multi': True, 'upsert': False}}], 'writeConcernErrors': [], 'nInserted': 0, 'nUpserted': 0, 'nMatched': 0, 'nModified': 0, 'nRemoved': 0, 'upserted': []}

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/core/collections.py", line 8837, in load_annotations
    return foua.load_annotations(
           ^^^^^^^^^^^^^^^^^^^^^^
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/utils/annotations.py", line 1079, in load_annotations
    _merge_labels(
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/utils/annotations.py", line 1486, in _merge_labels
    dataset.delete_labels(ids=_del_ids, fields=label_field)
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/core/dataset.py", line 3230, in delete_labels
    foo.bulk_write(sample_ops, self._sample_collection)
  File "/home/testuser/git/testrepo/model/venv/lib64/python3.11/site-packages/fiftyone/core/odm/database.py", line 816, in bulk_write
    raise ValueError(msg) from bwe
ValueError: Cannot apply $pull to a non-array value

System information

opensuse tumbleweed
python 3.11.8
FiftyOne v0.23.7
installed from pip

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another
member of your organization be willing to contribute a fix for this bug to the
FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently
[x] Yes. I would be willing to contribute a fix for this bug with guidance
from the FiftyOne community
[ ] No. I cannot contribute a bug fix at this time

Hi @spaced, if you add reproduction steps we will be able to diagnose 🙏

I am experiencing this same issue as well. I loaded a detection dataset from a file, called annotate with classes=dataset.get_classes("gt_bounding_boxes") + ["small_blurry"] (the only difference from when I last did this), labeled 500 images in labels studio, and then called load_annotations.

Same error here...
I created an annotation job for CVAT from fiftyone. Calling load_annotations from there shows this error
---------------------------------------------------------------------------
BulkWriteError                            Traceback (most recent call last)
File ~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:807, in bulk_write(ops, coll, ordered, progress)
    [806](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:806) batch = list(batch)
--> [807](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:807) coll.bulk_write(batch, ordered=ordered)
    [808](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:808) if batcher.manual_backpressure:
    [809](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:809)     # @todo can we infer content size from bulk_write() above?
    [810](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:810)     # @todo do we need a more accurate measure of size here?

File ~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/_csot.py:107, in apply.<locals>.csot_wrapper(self, *args, **kwargs)
    [106](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/_csot.py:106)             return func(self, *args, **kwargs)
--> [107](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/_csot.py:107) return func(self, *args, **kwargs)

File ~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/collection.py:569, in Collection.bulk_write(self, requests, ordered, bypass_document_validation, session, comment, let)
    [568](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/collection.py:568) write_concern = self._write_concern_for(session)
--> [569](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/collection.py:569) bulk_api_result = blk.execute(write_concern, session)
    [570](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/collection.py:570) if bulk_api_result is not None:

File ~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:576, in _Bulk.execute(self, write_concern, session)
    [575](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:575) else:
--> [576](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:576)     return self.execute_command(generator, write_concern, session)

File ~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:446, in _Bulk.execute_command(self, generator, write_concern, session)
    [445](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:445) if full_result["writeErrors"] or full_result["writeConcernErrors"]:
--> [446](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/pymongo/bulk.py:446)     _raise_bulk_write_error(full_result)
...
    [814](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:814) except BulkWriteError as bwe:
    [815](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:815)     msg = bwe.details["writeErrors"][0]["errmsg"]
--> [816](https://file+.vscode-resource.vscode-cdn.net/Users/mg/dev/kuckuck-ai/construction-site-object-detection-ml-project/fiftyone/notebooks/09-14%20kuckuck-v3.6/~/dev/kuckuck-ai/construction-site-object-detection-ml-project/venv/lib/python3.9/site-packages/fiftyone/core/odm/database.py:816)     raise ValueError(msg) from bwe

ValueError: Cannot apply $pull to a non-array value


mitisuibishi MUT 2 (kline 5 baud init)

Do you think you could add support for pre obd2 MUT 2 protocol for Mitsubishi vehicles?
There is protocol information here:
https://web.archive.org/web/20161029144911/http://evoecu.logic.net/wiki/ECU_Development
it uses a OBD2 connector but a slightly modified Kline protocol to the standard.

Do you think you could add support for pre obd2 MUT 2 protocol for Mitsubishi vehicles?

If you want that, you have to do it yourself :) I don't own a Mitsubishi vehicle to test on, and I don't want to do the 'I write code, you test it' approach right now, I have many things on my hands.
If you want to take a stab at it, enable the debug prints in the header file, change the baudrate and try to see if you can receive those Receive C0 55 EF 85 bytes they mention are sent after the 5 baud init.


Tooltip causes crash when navigating away from page

Describe the bug
Tooltip services causes a Javascript crash when navigating to another page
To Reproduce
Open a tooltip by hovering over it with your mouse,
Navigate to another page. The blazor yellow issue bar will show up at the bottom. In the event viewer it says:
Exception:  System.AggregateException: One or more errors occurred. (Error: No element is currently associated with component 1752)  ---> System.InvalidOperationException: Error: No element is currently associated with component 1752    at Microsoft.AspNetCore.Components.RenderTree.Renderer.InvokeRenderCompletedCallsAfterUpdateDisplayTask(Task updateDisplayTask, Int32[] updatedComponents)    --- End of inner exception stack trace ---
Expected behavior
Navigating to another page should work with a tooltip open.
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 104.0.5112.81

*Screenshots
Open tooltip

Navigate to other page shows me the yellow bar:

Event viewer:

JS console output:

Additional context
The issue happens at the JS level on the destroyPopup function it tries to destroy the popup while it didn't exist anymore. I didn't have this issue before it could be that it was introduced during one of the last updates to Radzen. I was able to make a quick fix my implementing my own CustomTooltip component in Blazor and commenting this line out in the Dispose() Method:

If you are using  this fix make sure to replace the Radzen Tooltip in your MainLayout with your custom one.

Duplicate to https://github.com/radzenhq/radzen-blazor/issues/571


Improve readability of NTLMAuthenticationGrantHandler

Here[1] we have written while loop with always true and it breaks with some next level API condition. So in the method, we can't see the logical break of the loop.
We need to improve the readability of this code segment.
[1] https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/0e61801108184e1f3be82f294fe0abe47f09df16/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth2/token/handlers/grant/iwa/ntlm/NTLMAuthenticationGrantHandler.java#L94-L130
Moved from https://wso2.org/jira/browse/IDENTITY-6911

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Fix: lnd arm version from arm64 to armv7 for RPi

What
lnd compatibility with Raspberry Pi devices. The correct version for RPi devices is armv7 build.
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[x] simple bug fix

The RaspiBolt v3 guide assumes you run a Raspberry Pi 4 with the latest Raspberry Pi OS 64-bit:
https://raspibolt.org/operating-system.html
The armv7 binaries are compiled for 32-bit operating systems. The arm64 binaries (also called armv8) are the correct ones when used on a 64-bit system.

Ah ok gotcha, I need to double-check my setup then because I thought had Raspberry Pi OS on my Pi 4 but still ran into this issue 🙈

Raspbian Buster 10, looks like I set my Pi up quite a while back, hadn't realized 64-bit was now the standard 🙈
PRETTY_NAME="Raspbian GNU/Linux 10 (buster)"
NAME="Raspbian GNU/Linux"
VERSION_ID="10"
VERSION="10 (buster)"
VERSION_CODENAME=buster
ID=raspbian
ID_LIKE=debian
HOME_URL="http://www.raspbian.org/"
SUPPORT_URL="http://www.raspbian.org/RaspbianForums"
BUG_REPORT_URL="http://www.raspbian.org/RaspbianBugs"


⚠ Samsung has degraded performance

In 3c67fa6, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3050 ms

Resolved: Samsung performance has improved in fedbd46.


Fix etcd_host default

Removes outdated config; see discussion

Coverage remained the same at 11.278% when pulling 9824ddae5ed9f53c973b4ba2631f0f586326e348 on fix-etcd-host-default into 97d1bde5b68c598b1c040b3ebc98b34e3a7c7a2d on master.

:+1:

👍


Modal components are broken

Same issue as in #73, examples/modal_components (https://github.com/iced-rs/iced_aw/commit/175b928380110bbf4d61361865f0dbe13ed8e108) can be used to reproduce:
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:139:38
stack backtrace:
   0: rust_begin_unwind
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/std/src/panicking.rs:584:5
   1: core::panicking::panic_fmt
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/core/src/panicking.rs:67:14
   2: core::panicking::panic
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/core/src/panicking.rs:117:5
   3: core::option::Option<T>::unwrap
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/core/src/option.rs:949:21
   4: iced_lazy::component::Instance<Message,Renderer,Event,S>::rebuild_element_if_necessary::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:139:38
   5: iced_lazy::component::ouroboros_impl_state::State<Message,Renderer,Event,S>::new
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:94:1
   6: iced_lazy::component::ouroboros_impl_state::StateBuilder<Message,Renderer,Event,S,ElementBuilder_>::build
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:94:1
   7: iced_lazy::component::Instance<Message,Renderer,Event,S>::rebuild_element_if_necessary
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:146:18
   8: iced_lazy::component::Instance<Message,Renderer,Event,S>::with_element_mut
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:206:9
   9: iced_lazy::component::Instance<Message,Renderer,Event,S>::with_element
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:199:9
  10: <iced_lazy::component::Instance<Message,Renderer,Event,S> as iced_native::widget::Widget<Message,Renderer>>::layout
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_lazy-0.6.1/src/component.rs:258:14
  11: <iced_native::widget::container::Container<Message,Renderer> as iced_native::widget::Widget<Message,Renderer>>::layout::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_native-0.10.3/src/widget/container.rs:169:17
  12: iced_native::widget::container::layout
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_native-0.10.3/src/widget/container.rs:309:23
  13: <iced_native::widget::container::Container<Message,Renderer> as iced_native::widget::Widget<Message,Renderer>>::layout
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_native-0.10.3/src/widget/container.rs:158:9
  14: <iced_aw::native::overlay::modal::ModalOverlay<Message,Renderer> as iced_native::overlay::Overlay<Message,Renderer>>::layout
             at /Users/turai/Development/iced_aw/src/native/overlay/modal.rs:81:27
  15: iced_native::overlay::element::Element<Message,Renderer>::layout
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_native-0.10.3/src/overlay/element.rs:62:9
  16: iced_native::user_interface::UserInterface<Message,Renderer>::update
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_native-0.10.3/src/user_interface.rs:208:30
  17: iced_winit::application::run_instance::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_winit-0.9.1/src/application.rs:424:44
  18: iced_winit::application::run::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_winit-0.9.1/src/application.rs:251:24
  19: <winit::platform_impl::platform::app_state::EventLoopHandler<T> as winit::platform_impl::platform::app_state::EventHandler>::handle_nonuser_event::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/app_state.rs:106:17
  20: winit::platform_impl::platform::app_state::EventLoopHandler<T>::with_callback
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/app_state.rs:80:13
  21: <winit::platform_impl::platform::app_state::EventLoopHandler<T> as winit::platform_impl::platform::app_state::EventHandler>::handle_nonuser_event
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/app_state.rs:101:9
  22: winit::platform_impl::platform::app_state::Handler::handle_nonuser_event
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/app_state.rs:209:21
  23: winit::platform_impl::platform::app_state::AppState::cleared
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/app_state.rs:394:9
  24: winit::platform_impl::platform::observer::control_flow_end_handler::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/observer.rs:184:21
  25: winit::platform_impl::platform::observer::control_flow_handler::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/observer.rs:145:9
  26: std::panicking::try::do_call
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/std/src/panicking.rs:490:40
  27: std::panicking::try
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/std/src/panicking.rs:454:19
  28: std::panic::catch_unwind
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/std/src/panic.rs:140:14
  29: winit::platform_impl::platform::event_loop::stop_app_on_panic
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/event_loop.rs:265:11
  30: winit::platform_impl::platform::observer::control_flow_handler
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/observer.rs:143:5
  31: winit::platform_impl::platform::observer::control_flow_end_handler
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/observer.rs:179:9
  32: <unknown>
  33: <unknown>
  34: <unknown>
  35: <unknown>
  36: <unknown>
  37: <unknown>
  38: <unknown>
  39: <unknown>
  40: <unknown>
  41: <unknown>
  42: <() as objc::message::MessageArguments>::invoke
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc-0.2.7/src/message/mod.rs:128:17
  43: objc::message::platform::send_unverified
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc-0.2.7/src/message/apple/mod.rs:27:9
  44: objc::message::send_message
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc-0.2.7/src/message/mod.rs:178:5
  45: winit::platform_impl::platform::event_loop::EventLoop<T>::run_return::{{closure}}
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc-0.2.7/src/macros.rs:133:15
  46: objc::rc::autorelease::autoreleasepool
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/objc-0.2.7/src/rc/autorelease.rs:29:5
  47: winit::platform_impl::platform::event_loop::EventLoop<T>::run_return
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform_impl/macos/event_loop.rs:212:25
  48: <winit::event_loop::EventLoop<T> as winit::platform::run_return::EventLoopExtRunReturn>::run_return
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/winit-0.27.5/src/platform/run_return.rs:62:9
  49: iced_winit::application::platform::run
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_winit-0.9.1/src/application.rs:870:28
  50: iced_winit::application::run
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced_winit-0.9.1/src/application.rs:226:5
  51: iced::application::Application::run
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced-0.9.0/src/application.rs:209:12
  52: iced::sandbox::Sandbox::run
             at /Users/turai/.cargo/registry/src/index.crates.io-6f17d22bba15001f/iced-0.9.0/src/sandbox.rs:153:9
  53: modal_component::main
             at ./src/main.rs:9:5
  54: core::ops::function::FnOnce::call_once
             at /rustc/f5559e338256f17ada6d82b429acc2dbd8facc9c/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

yeah I know they are still broken but I still do not know Why it is broken as its been really hard to trace the reason for this. The example I have in there is to help trace and to know when it is finally fixed or not.

My very shallow investigation suggests this doesn't really have anything to do with this crate, ran into the exact same panic while messing with my own components, but I'd rather keep this issue open to warn others.

yeah that's fine. I suspect it has something to do with something Iced itself is doing but again its hard to track down. Like I can see where it error-ed but i cant see why it had an error.  Made me feel like something thread wise is missing data. Or something  ran before something else which made the data not yet appear. I will keep this comment open for others and ill point @hecrj  here too.

I ran into this some days ago and created an even more minimal example before discovering this issue. Just in case it might help, it can be found here.

Preliminary analysis:

Modal::new calls the into() implementation of MyComponent.
into() is implemented via iced_lazy::component.
component() calls Element::new() in component.rs, setting tree to None.
I'm not sure how, but then diff() in modal.rs is called.
This in turn calls state() in component.rs, setting tree to Some. So far so good.
Then iced_nativ::user_interface::UserInterface::update calls Modal::overlay.
Here we call ModalOverlay::new(). One of the arguments is (self.content)(). I do not yet know that syntax, but it seems to call iced_lazy::component again. Is this some kind of typecast?
In any case, component(), calls Element::new() again and resets tree to None.
Still in UserInterface::update, we run into Element::layout().
This eventually calls Instance::rebuild_element_if_necessary in component.rs.
And now this function expects a tree to present, but because state was never called again since step 8, it is None.

this seems like a iced issue then if it has to do when the state is being called. I wonder if there is something we can do to ensure the state is made and set.

This is an iced issue. Component currently breaks the Widget contract. A proper fix isn't straightforward. I'll think about it.
In the meanwhile, we can call Widget::diff for the content before passing it to the ModalOverlay to ensure the widget tree has the proper state set.

@turai  and @TheComamba can you please try the latest Main branch of iced_aw. I believe the new push fixes this issue.


MVT tiles draw splotchy on dev v0.5-pre

All over NYC and elsewhere, when I set the source to be dev in Eraser Map scene file I get:
/cc @rmarianski @bcamper @sensescape

Looks like it may be another multi-polygon encoding issue? We should
compare tiles in GeoJSON and MVT to see how they diverge.
On Wed, Nov 11, 2015 at 6:56 PM, Nathaniel V. KELSO <
<EMAIL_ADDRESS>wrote:

All over NYC and elsewhere, when I set the source to be dev in Eraser Map
scene file I get:
/cc @rmarianski https://github.com/rmarianski @bcamper
https://github.com/bcamper @sensescape https://github.com/sensescape
[image: screen shot 2015-11-11 at 15 47 29]
https://cloud.githubusercontent.com/assets/853051/11106653/92edef20-888c-11e5-9d5d-ec274d082e82.png
[image: screen shot 2015-11-11 at 15 42 30]
https://cloud.githubusercontent.com/assets/853051/11106652/92ed9458-888c-11e5-9e65-3b730eb2510e.png
—
Reply to this email directly or view it on GitHub
https://github.com/mapzen/vector-datasource/issues/406.

I was able to verify that it's something in the recent mapbox-vector-tile changes. I'll dig into it.

This issue gets exposed the moment that multipolygons are encoded. I dug into the encoding, and things look right from what I can tell in the encoded data. I also see the issue regardless of the winding order that gets used. Fwiw I grabbed Varun for a few minutes and we stepped through the decoding in tangram-es, and it seems like what's in the tile is at least what I was expecting.
To me it seems like the decoder is just not detecting the new polygon, and thinks it's an inner ring from the previous geometry. Is the mapbox decoding library in use the latest that supports this behavior?
Can we set aside some time to debug this together? I can set up a dev server to always return back a fake tile that has a single simple MultiPolygon to expose this.

Aha, I think I may have to add a decoding step for this in Tangram! It
looks like the current version of Mapbox's decoder may not support this --
but I have code I can reuse to add that. Will try it and get back to you.
On Thu, Nov 12, 2015 at 3:13 PM, Robert Marianski<EMAIL_ADDRESS>wrote:

This issue gets exposed the moment that multipolygons are encoded. I dug
into the encoding, and things look right from what I can tell in the
encoded data. I also see the issue regardless of the winding order that
gets used. Fwiw I grabbed Varun for a few minutes and we stepped through
the decoding in tangram-es, and it seems like what's in the tile is at
least what I was expecting.
To me it seems like the decoder is just not detecting the new polygon, and
thinks it's an inner ring from the previous geometry. Is the mapbox
decoding library in use the latest that supports this behavior?
Can we set aside some time to debug this together? I can set up a dev
server to always return back a fake tile that has a single simple
MultiPolygon to expose this.
—
Reply to this email directly or view it on GitHub
https://github.com/mapzen/vector-datasource/issues/406#issuecomment-156221427
.

Fixed in this branch, when we are ready to merge: https://github.com/tangrams/tangram/compare/decode-multipolygon

Nathaniel to make sure Tangram release version has this support now, and to update whatever demos on the site use those.

Reference: https://github.com/mapzen/mapbox-vector-tile/issues/20

@nvkelso also to followup with John about communicating this change.

TilesBRB: We need another PR in the MBVT repo that allows us to configure if the multi polygon is enabled, and disable that by default (but keep the winding switch already accomplished). This should allow to fix the splotchiness in old Tangrams, and still work with new Tangrams.

@rmarianski anything more to do here?

This had to do with multipolygon encodings, which should be all good now on all fronts. Closing.


Create Github Webhook fail

Rancher versions:
21/3 master
Steps to Reproduce:
Create a Pipeline and try to set webhook trigger.
Results:
Fail to create Pipeline in Github repository.

It works, but there was just push event, I set both PR and push.

Verified in 31/3 master


Disable C++ models from being compiled without explicit request

This has been added for testing C++ models in CI, but it shouldn't be part of the binaries.
A follow-up PR / commit will add proper CI for testing the models.

Codecov Report

Merging #1535 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1535   +/-   ##
======================================
  Coverage    64.6%   64.6%
======================================
  Files          87      87
  Lines        6742    6742
  Branches     1034    1034
======================================
  Hits         4356    4356
  Misses       2085    2085
  Partials      301     301

Continue to review full report at Codecov.

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


Remove WorkerInterface

Fix config and tests accordingly.

@xepozz Check please if you need this interface for #154.

We won't remove this interface.


Deprecate CONF API, make conf_method_st opaque

Rewrite _CONF_xxx API in terms of new internal conf_xxx functions,
which are basically the old ones renamed.

Note that the old _CONF_xxx API was not exported in shared libraries because the functions did not exist in libcrypto.num because conf_api.h wasn't scanned?  By moving it into conf.h, they now show up as deprecated.
Because they weren't documented and weren't available (only in static libraries), could/should we just remove them?

The _CONF_xxx API was intended to be internal, at a time when we didn't have the concept of internal headers.  That's why they aren't exported.  And that's why they should be made internal entirely, and should not appear in include/openssl/conf.h.

Ping for a second review.

More than two weeks.  Ping for a second review (anyone, not OMC)

Extra ping


Added 'last_updated' and 'last_changed' into Title Variables

This pull request includes the following changes:

Made it possible to use 'last_updated' and 'last_changed' in the Title and Labels

In order to achieve this, the following has been done:

Made the 'stateAttributes' Variable also contain the 'last_changed' and 'last_updated' Data from the State Object itself.

Thank you very much (again :))


Source code errors

I just installed Starscream with cocoapods and there are a lot of errors in the source files which makes Xcode complain a lot. Errors such as "can't invoke init with....", "expected ',' separator" and so on. Am I building it wrong or did someone goof something up?
Here's a paste with all the errors
http://pastebin.com/Gn2L56x3

Edited the source code and got it to build at least, no clue if it will run as intended.
http://pastebin.com/Qm5q2aQp

What version of Xcode are you on? These sound like Swift version errors.

Xcode 6.1.1

@nuclearace thanks for narrowing this down. xCode 6.3 is when swift 1.2 was released, which the last few releases of Starscream are targeted for. I would recommend installing the latest version xCode which should resolve the issue.


Added new flag --require-https

Added new CLI/Config File flag --require-https.
When it's presented, lychee additionally checks whether HTTPS encryption is
available for HTTP URLs. If so, it's treated as errors.
Closes #192

Thanks for tackling this. Added a few minor comments. Otherwise looks good. 👍

I rebased the code.
I renamed it to --require-https. (Forgotten to change commit message)
And replaced unsafe code.
But I didn't extract it to a separate function.
I just cannot come up with a meaningful name for this section. And I think this is quite straightforward already.

I renamed it to --require-https. (Forgotten to change commit message)
And replaced unsafe code.

Thanks, that's much more intuitive now. Also cheers for removing unsafe!

I just cannot come up with a meaningful name for this section. And I think this is quite straightforward already.

That might be the case, but still, the code is getting quite dense, and harder to reason about.
I think it would make sense to handle the entire self.require_https portion in a separate function, for instance, another branch of the if/else chain here: https://github.com/lycheeverse/lychee/pull/195/files#diff-c67c22c2f52695b3cdf26f7e6ba2fab2f49e4605941a7d87a005653d1c568e7aR178-R195
Could be changed to something like:
        let status = if self.filter.is_excluded(&uri) {
            Status::Excluded
        } else if uri.is_mail() {
            self.check_mail(&uri).await
        } else if self.require_https {
            // check website requiring HTTPS
            // (possibly a new method call, which would end up calling self.check_website)
        } else {
            self.check_website(&uri).await
        };

This is because the logic is no longer "just return the status".

Well, the logic is different only when the URL intially returns Ok. Then we need to check whether it's HTTP and whether HTTPS is required. If so, change its scheme to HTTPS and check again. Should it returns Ok this time, we throw an error.
If create a separate function, it inevitably checks whether HTTPS is required for every URL. then in the body of new function, it either calls check_website twice, or repeat that parts of code. That's to say, it either created unnecessary function calls, or it increases redundancy.

That's to say, it either created unnecessary function calls, or it increases redundancy.

I don't think that's an issue. The compiler will likely optimize the code anyway. And even if there were no optimizations at all -- I believe the readability is worth it. Handling the "require HTTPS" case in the else branch simply obscures the "normal" case where we simply want to execute the request without any additional logic, specific to the "require HTTPS" case.
Moving it into a dedicated function would make it much more readable, easier to reason about (the reader would go "aha! the normal checking is not the same as checking when HTTPS is required"), and remove a few levels of code nesting. Also, any changes to the "require HTTPS" case would become more "localized" -- e.g. you'd only touch one function, without thinking much about it.
But that's, of course, all a bit subjective. I don't like nested if statements, as they make "executing the code in your mind" much harder, exponentially harder with every level of nesting.

I agree with @pawroman here. Then again it's not really a blocker.
Added two tests and the pipeline is green, so I'm gonna merge this.

Great work guys! I just recently recognised a bunch of unnecessary plain HTTP links on our website, mostly leading to forced HTTPS redirects, but still, can be seen as unresponsable nowadays to lead visitors onto a non-encrypted track 🙂.


Errors at --as step

Spotted this error in the --as step. It states "invalid version format". Is this referring to the .json format? If so, shouldn't the --val step catch this?
2019-03-12 17:29:54,835 mrtarget.common.Redis ERROR - Error processing job b6a82577027147c5bb154a9105f18771:
Traceback (most recent call last):
  File "mrtarget/common/Redis.py", line 300, in _inner_run
    job_results = self.process(data)
  File "mrtarget/modules/Association.py", line 329, in process
    available_evidence = self.es_query.count_evidence_for_target(target)
  File "mrtarget/common/ElasticsearchQuery.py", line 448, in count_evidence_for_target
    'size': 0
  File "/usr/local/lib/python2.7/site-packages/elasticsearch/client/utils.py", line 73, in _wrapped
    return func(*args, params=params, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/elasticsearch/client/__init__.py", line 632, in search
    doc_type, '_search'), params=params, body=body)
  File "/usr/local/lib/python2.7/site-packages/elasticsearch/transport.py", line 312, in perform_request
    status, headers, data = connection.perform_request(method, url, params, body, ignore=ignore, timeout=timeout)
  File "/usr/local/lib/python2.7/site-packages/elasticsearch/connection/http_requests.py", line 90, in perform_request
    self._raise_error(response.status_code, raw_data)
  File "/usr/local/lib/python2.7/site-packages/elasticsearch/connection/base.py", line 125, in _raise_error
    raise HTTP_EXCEPTIONS.get(status_code, TransportError)(status_code, error_message, additional_info)
RequestError: TransportError(400, u'illegal_argument_exception', u'invalid version format:
"CATEGORIES": {"CLINICAL_PRECEDENCE": 1.0, "DISCOVERY_PRECEDENCE": 0.0, "PREDICTED_TRACTABLE": 0.0}, "ENSEMBLE": -0.73153363, "HIGH_QUALITY_COMPOUNDS": 0, "SMALL_MOLECULE_GENOME_MEMBER": FALSE, "TOP_CATEGORY": "CLINICAL_PRECEDENCE"}}}}')
2019-03-12 17:29:54,841 mrtarget.common.ElasticsearchLoader WARNING - push to elasticsearch failed for chunk.  retrying in 5s...

No, I think it refering to the "version" of the document in Elasticseach - usually something used to avoid dropped updates. That's still a problem because I don't think the association step should be doing any updates to documents! Was this with public 19.02 data? Can we reproduce it?

Found this one as well:
2019-03-13 11:05:24,219 mrtarget.common.Redis DEBUG - ========> Processing evidence: target[ENSG00000185527], disease[EFO_0000311]
2019-03-13 11:05:24,279 mrtarget.common.ElasticsearchLoader WARNING - push to elasticsearch failed for chunk.  retrying in 5s...
2019-03-13 11:05:24,668 mrtarget.common.Redis ERROR - Error processing job<PHONE_NUMBER>ba412aa8d5f94c44cf4d15: hits
Traceback (most recent call last):
  File "mrtarget/common/Redis.py", line 300, in _inner_run
    job_results = self.process(data)
  File "mrtarget/modules/Association.py", line 329, in process
    available_evidence = self.es_query.count_evidence_for_target(target)
  File "mrtarget/common/ElasticsearchQuery.py", line 451, in count_evidence_for_target
    return res['hits']['total']
KeyError: 'hits'
2019-03-13 11:05:24,696 mrtarget.common.Redis DEBUG - ############# found 64 evidences for target ENSG00000113721
2019-03-13 11:05:24,821 mrtarget.common.Redis DEBUG - ############# found 1 evidences for target ENSG00000154678

@afaulconbridge as agreed, moved second issue to a separate ticket: https://github.com/opentargets/platform/issues/482

I've reviewed our logfiles for and seen no evidence of this problem. Therefore I expect it to be resolved with 19.04. I'll leave this ticket open for now, and it should be re-examined with 19.04 to ensure the problem is solved.
I wonder if the underlying cause was the an issue similar to opentargets/platform#177 with fork() affecting file descriptors.

will assume to be solved in 19.04 as I have not seen this error while running 19.04.5 on my latest test set :+1:


seperator between frames NULL or NULL+LF?

This client works with ActiveMQ, now the official spec doesn't say end of frame is
NULL+LF
but just NULL.
If we have a server that adheres to the exact standard, then the client will not work, because it will never see a ^@+\n
Was this done on purpose or incorrectly interpreted or copied ?
Or should it handle cases of just a NULL between frames?

This was fixed in issue #13


[BUG] [timeseries] Some forecasting models fail during fit if an S3 path is used

Bug Report Checklist

[x] I provided code that demonstrates a minimal reproducible example.
[x] I confirmed bug exists on the latest mainline of AutoGluon via source install.
[x] I confirmed bug exists on the latest stable version of AutoGluon.

Describe the bug
Setting TimeSeriesPredictor(path="s3://my-bucket/my-predictor") leads to some forecasting models failing during training.
Related issue: https://github.com/awslabs/gluonts/issues/3171
Expected behavior
All models train successfully, same as if a local path was used.
To Reproduce
pred = TimeSeriesPredictor(path="s3://my-bucket/my-predictor").fit('https://autogluon.s3.amazonaws.com/datasets/timeseries/m4_hourly_tiny/train.csv')

Open question: Should we support s3 paths?
In early versions of AutoGluon I supported s3 paths for TabularPredictor. I eventually found that once I mixed in ray and started using optimized model saving techniques that it became hard to ensure the code worked properly in both scenarios.
Because of this difficulty, I eventually stopped supporting S3 paths for predictor artifacts.
One work-around could be to use a local directory for all training / prediction, and then upload the local files to the s3 location after fitting is complete. Similarly, download the artifact if called via .load. I haven't implemented this approach but it could work.
If you have found a way to make s3 paths work with relative ease, I'd be interested in knowing how you did it.

@Innixma I assumed that TabularPredictor already supported S3 paths since it worked fine with medium_quality presets, but now I understand that this might not be the case for high_quality / best_quality, where ray is used.
I will check how much work it would take to make S3 paths work in all scenarios. At the very least, we should raise some informative error message explaining that only local paths are supported. Currently for the TimeSeriesPredictor we just observe some model failures in the middle of the training process, which is a bad customer experience.