Beast message interface and middleware

I am searching C++11 library for embedding http server inside my application, and I found Beast as a perfect solution. I have found that current message interface is very good and offers full control over every parth of message
What I dont like:

I can forget to fill some fields (status, reason...) and message will be sent with no errors on server side
Too many steps to fill data. Current message is good for low-level operations, but after I tried Nodejs, I want to write less code... This can be done with constructing messages from pre-defined templates or setting some missed fields with default values. I would like to send a file/json answer/custom html or any other typical message with using only one line of code. Yes, I can make such templates myself and use them, but I think that Beast should show users how to make it the right way
And do we really need an explicit call of prepare() ?

One more question is not clear for me about message construction and its structure. How to make middleware? For example, something like connect-multiparty
This middleware should be able to add new field "files" into message structure, check request path and intercept messages. Should I derive from request_v1 and write overloaded functions for reading/wiriting? Do you have any plans about middleware affecting message structure? I think, Beast should offer a simple way how to write and use middleware

First of all thank you for your interest in Beast! Those are some great questions. Answers:
Adding checks or defaults for missing fields would incur a run-time penalty. It is presumed that callers correctly fill out the message structure before invoking HTTP algorithms. I could add some asserts though, but that would only help debug builds.
Beast HTTP is intentionally a low level library. From http://vinniefalco.github.io/beast/beast/http/scope.html :
"This library is designed to be a building block for creating higher level libraries. It is not designed to be end-user facing. There is no convenient class that implements the core of a web server, nor is there a convenient class to quickly perform common operations such as fetching a file or connecting and retrieving a document from a secure connection. These use-cases are important, but this library does not try to do that. Instead, it offers primitives that can be used to build those user-facing algorithms."
I agree that very common use cases require more lines than usual. It is my hope that other developers will use Beast as a building block to create those user-facing algorithms. For example, someone suggested that having a C++ library modeled after Python Requests (http://docs.python-requests.org/en/master/) would be helpful. I hope that someone will build that library on top of Beast.
The explicit call to beast::http::prepare is not required. It is just a convenience function that fills in some of the fields for you. For example, it will set the Content-Length field, Transfer-Encoding, and Connection fields appropriately. If you want to set those fields yourself and skip the call to prepare, you can do it.
To extend the message structure, it is best to use composition. Create a new struct or class that has the message as a data member. Custom algorithms that operate on this new type can be in their own namespace, and call Beast algorithms as required. I'm not sure if an example of "using middleware" is warranted - such an example would serve no purpose other than to inform readers of how to use object composition, which is assumed knowledge.


flutter gallery crashes on launch in profile on ios 32-bit device

https://github.com/flutter/flutter/pull/41213 reverted two engine rolls that were related to the failure
cc @gaaclarke

cc @rmacnak-google

Seems to have been introduced in https://dart-review.googlesource.com/c/sdk/+/117683

Is not reverting cleanly unfortunately

https://dart-review.googlesource.com/c/sdk/+/118661 with revert

Is this the same as https://github.com/flutter/flutter/issues/41010 that just got closed? Or is this a separate issue?

This is separate issue from #41010

Ryan was able to reproduce similar (looking) issue in standalone dart sdk as well.

Revert landed, this now needs to be rolled to flutter engine as part of dart sdk roll.


Redundant bracket warning for unary unboxed tuple

The current master (9f18b433b3c7f4e7976aa3983289edbcf2e0d84b) incorrectly warns about redundant brackets on unary unboxed tuples in Data.HashMap.Internal, e.g. here:
Data/HashMap/Internal.hs:970:59-62: Warning: Redundant bracket
Found:
  (v')
Perhaps:
  v'

I've haven't been able to reproduce the issue on a smaller example though, so maybe hlint is getting confused by some aspect of that module.

@sjakobi I think (based on the column number) what HLint warns is the | otherwise -> Leaf h (L k (v')) on line 971, which is a legitimate warning. Perhaps you are using a different version of Internal.hs where that line is 970?

Oh, yes! :man_facepalming: Sorry for the noise, and thanks for the help!


ÆØÅ in "name" breaks form

Hey there
We have a large form with FormEditor but we are seeing some wierd bug, i dont know if it is focused around ÆØÅ in the name/alias of a field.
The error is :

The field that it fails at looks like this:

If it is the æøå we should maybe lock the alias field and autogenerate it with an option to unlock and overwrite it.

Their might still be something with øæå but we found something else with ng-show that one of our angular gurus fixed for us.
when you validate you use the css class hide/show but if your(read our) markup dosnt have any handling for the 2 css classes, it will not work. But if you instead use ng-show it would swap the hide/show classes and it would work.

Hey @rasmusfjord,
The field aliases aren't supposed to be able to have ÆØÅ in them... clearly something's wrong with the regex'es that should prevent this, as you can see the lower cased "æ" is removed but not the upper cased "Å". I'll have a look at that.
With regard to the hide/show part, FormEditor was created to play nice with the bootstrap CSS out of the box. I agree, ng-show or ng-if would be a more angular-ish approach. I will have a look at that as well as soon as I can.

As always Kenn you rock and keep up the good work.
Yeah one would argue that the bootstrap everything era is ending so its properly a good idea to go for a more pure appoarch where its just shown/not shown with angular.
Say if you need testers, Alt. you could lean up against the umbraco doctype alias namer where it also locks the field, because our backoffice editors(customers) are a bit confused over the 2 top fields when adding a new field in formeditor. Because in danish its "navn"(name) and "Tekst"(text) where you could argue that "navn"(name) should be refrased alias, and the "navn"(name) should replace the current "Tekst"(text) field. If you get my point :)

@rasmusfjord thanks.
I've replicated the issue - it only happens if you have special characters in the very beginning of the field name, that's why I haven't caught it before. I'll submit a fix for it soon, but if it's not urgent for you then I'll hold off the next release until I've verified everything with the Umbraco 7.6 RC.
Also I've moved your other comment about ng-show/hide to #123 and will solve that asap as well.


Export Not Working for Feature Management Key/Value Pairs

I am using App Configuration as a store for feature flags (FeatureManagement).  I noticed that when I try to export the configuration to a configuration file - it does not recognize any key/value pairs.  Once I add a simple key/value pair to the existing feature, it only exports out the 1 key/value pair.  But from the picture below, it seems that App Config recognizes that there are 3 key/value pairs - 2 being of a feature management content type.

Is this the expected behavior?
We are wanting to use this as a way to promote feature flags (for instance for a beta rollout) through non-production to production.

@toddmeinershagen The import/export of feature flags to a Configuration file is not supported through the Azure Portal. There are two ways you could export all settings including feature flags for your scenario.

Use Azure CLI to export all key-value pairs and feature flags with null label to a configuration file. More details on the supported parameters can be found here.
az appconfig kv export -n MyAppConfiguration -d file --path D:/abc.json --format json

Export the configuration directly to another App Configuration resource through Azure Portal

The support for importing/exporting feature flags from/to files in portal is tracked in #190.


Optimise Neo4j writes to reduce number of updates needed

Change EquivalenceWriter writes to create resourceRef nodes where
needed rather than executing a separate query to write them before
writing equivalence assertions
Change HierarchyWriter writes to create contentRef nodes where
needed rather than delegating to ContentWriter to create them
Use ON CREATE SET clauses in ContentWriter to avoid repeatedly
updating data that cannot change
Remove unnecessary metrics from Neo4jContentStore. The Neo4j driver
does most of the work when the transaction is closed so that is where
most of the execution time is concentrated. The metrics of the
other execution steps are showing unrealistic low values that are
not useful
Remove methods to write contentRef and resourceRef from
ContentWriter as they are no longer needed

Kinda lgtm I guess.


External modules excluded from bundles

Coverage increased (+0.008%) to 32.855% when pulling 7d839b6c5f2a13c65d712d0932fa4fa5669536f7 on bundle_no_babel into 012a7e3c334a2baabba7f742b07eef44a5e652a6 on master.


Change profile picture URL to userphoto.aspx

Category

[x] Bug Fix
[ ] New Feature

What's in this Pull Request?
This PR changes the profile picture URL path of userphoto.aspx.
Currently it uses the URL from the user's profile picture value present under the https://tenant-my.sharepoint.com path. Because of that, there is a cross-domain issue which causes the image to be not loaded.
By fetching it from userphoto.aspx path, we are able to fix the issue.
I also thought about using the outlook service url, but the issue with that is that if the profile picture doesn't exist, we get a very small  1*1 image which is not useful whereas in case of userphoto.aspx we get a proper blank image which is useful.

thx @gautamdsheth, appreciated.


Drop images on window navigates to it

If drop image onto Etcher window then window will navigate and load this image instead of Etcher html.
Could be fixed with this:
document.addEventListener('dragover',function(event){
  event.preventDefault();
  return false;
},false);

document.addEventListener('drop',function(event){
  event.preventDefault();
  return false;
},false);

Hi @amka ,
Thats very funny, I'd never have though of dragging an actual image to the window :) Your snippet looks good, however consider that we support drag and drop on the first step, so that code might get in between that feature.
I believe Electron allows us to inspect the file name on certain drag events, so we could deny the drag depending on the extension.
Happy to take a PR if you feel like working on it, otherwise I can handle it myself.

Hi @jviotti, I'm just writing app with drag-n-drop and made some tests with Etcher.
I'm novice in JS and don't think can make PR in reasonable period.

Hi @amka ,
No worries, I'll take over on it, thank you very much for the heads up! I'm actually thinking we could PR this fix straight to Electron.


Projektmanagement

[ ] Rollen, Verantwortlichkeiten
[ ] Risiken
[ ] Sprints

This issue was moved to osm2vectortiles/osm2vectortiles-thesis#5


Gettransaction

I add getTransactionByInvoice and getTransactionDetailByInvoice for checking
transaction status after complete paynow request.
It really necessary to check status in case callback request has problems
but currently not implemented.

Thanks for this. There appears to be a typo in the class name though - "PaysbuyGetTrasaction"

Ok I will fix the typo one.

Hi there - did you fix it?

I'll merge it in and fix it 😄


:PlugInstall gives (random, spurious) PlugClean required messages

When calling :PlugInstall, I get seemingly random indications that I need to call :PlugClean, e.g.:
 10 x vim-over
 11     master
 12     https://github.com/osyo-manga/vim-over.git
 13     PlugClean required.

Note that the package(s) indicated change from run to run; ie, the culprit is not vim-over but that vim-plug seems to randomly suggest that some vim plugins need cleaning. :PlugStatus seems to consistently return OK, however, and does not report any problems. Similarly, :PlugClean doesn't actually do any cleanup.
If I had to guess, somehow the concurrent use of git to update plugins is causing problems on my system, or perhaps some weirdness with Vim 8 is going on? This is on an Ubuntu 16.04 VM.
:PlugInstall1 seems to execute without any issue.

VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Oct 15 2016 17:44:16)
Included patches: 1-39
Compiled by kevin@KBOX
Huge version with GTK2 GUI.  Features included (+) or not (-):
+acl             +conceal         +file_in_path    +libcall         +mouse_sgr       +profile         +tag_old_static  +viminfo
+arabic          +cryptv          +find_in_path    +linebreak       -mouse_sysmouse  +python          -tag_any_white   +vreplace
+autocmd         +cscope          +float           +lispindent      +mouse_urxvt     -python3         -tcl             +wildignore
+balloon_eval    +cursorbind      +folding         +listcmds        +mouse_xterm     +quickfix        +termguicolors   +wildmenu
+browse          +cursorshape     -footer          +localmap        +multi_byte      +reltime         +terminfo        +windows
++builtin_terms  +dialog_con_gui  +fork()          +lua             +multi_lang      +rightleft       +termresponse    +writebackup
+byte_offset     +diff            +gettext         +menu            -mzscheme        +ruby            +textobjects     +X11
+channel         +digraphs        -hangul_input    +mksession       +netbeans_intg   +scrollbind      +timers          -xfontset
+cindent         +dnd             +iconv           +modify_fname    +num64           +signs           +title           +xim
+clientserver    -ebcdic          +insert_expand   +mouse           +packages        +smartindent     +toolbar         +xpm
+clipboard       +emacs_tags      +job             +mouseshape      +path_extra      +startuptime     +user_commands   +xsmp_interact
+cmdline_compl   +eval            +jumplist        +mouse_dec       -perl            +statusline      +vertsplit       +xterm_clipboard
+cmdline_hist    +ex_extra        +keymap          -mouse_gpm       +persistent_undo -sun_workshop    +virtualedit     -xterm_save
+cmdline_info    +extra_search    +lambda          -mouse_jsbterm   +postscript      +syntax          +visual
+comments        +farsi           +langmap         +mouse_netterm   +printer         +tag_binary      +visualextra
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/usr/local/share/vim"
 f-b for $VIMRUNTIME: "/usr/share/vim/vim74"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK  -pthread -I/usr/include/gtk-2.0 -I/usr/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/incl
ude/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/libp
ng12 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng12 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/gli
b-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/freetype2   -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc   -L. -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -fstack-protector -rdynamic -Wl,-export-dynamic  -L/usr/local/lib -Wl,--as-needed -
o vim   -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lglib-2.0 -
lfontconfig -lfreetype -lSM -lICE -lXpm -lXt -lX11 -lXdmcp -lSM -lICE  -lm -ltinfo -lnsl  -lselinux  -ldl  -L/usr/local/lib -llua  -L/usr/lib/python2.7
/config -lpython2.7   -lruby-2.3 -lpthread -lgmp -ldl -lcrypt -lm

Type:

[x] Bug
[ ] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[x] Linux
[ ] OS X
[ ] Windows

Vim:

[x] Terminal Vim
[ ] GVim
[ ] Neovim

If I had to guess, somehow the concurrent use of git to update plugins is causing problems on my system

Looks like so, but I haven't experienced the problem. Are the errors repeatable, same error on same plugins? Or are they completely random? Can you show the output of the following commands?
type git
git --version

I guess it is Vim8 since I had this issue once I upgrade vim to v8.0.0041. So I revert it back to v8.0.0032 and everything works fine.

@yufanlu Thanks, I just confirmed that the Ruby installer fails with the latest Vim :(
https://travis-ci.org/junegunn/vim-plug/builds/167548429

If I had to guess, the most recent job-control related patch was this:
commit 97792de2762cc79cc365a8a0b858f27753179577
Author: Bram Moolenaar<EMAIL_ADDRESS>Date:   Sat Oct 15 18:36:49 2016 +0200

    patch 8.0.0036
    Problem:    Detecting that a job has finished may take a while.
    Solution:   Check for a finished job more often (Ozaki Kiichi)

Perhaps this inadvertently regressed something that vim-plug depends on? (https://github.com/vim/vim/commit/97792de2762cc79cc365a8a0b858f27753179577)

I can confirm that (https://github.com/vim/vim/commit/472e85970ee3a80abd824bef510df12e9cfe9e96) works just fine, so (https://github.com/vim/vim/commit/97792de2762cc79cc365a8a0b858f27753179577) must be the culprit here.

@kevinushey Thanks for the investigation, I'll see what I can do.

Found the code that is no longer compatible with the latest Vim and replaced it with an alternative method. The incompatibility might be a bug of Vim itself, but the new code should be safer so I just decided to use it. The problem with it is that it doesn't work on Ruby 1.8 but I believe Ruby 1.8 is no longer relevant.
Thanks for the help!

Awesome, thanks! Just want to confirm as well that everything appears to be fine now too.
And thank you for vim-plug!


Feedback form

Closes https://sce.myjetbrains.com/youtrack/issue/TD-309
Added a new feedback form component to capture customer feedback inside our web apps.
The form is launched from a floating action button.

First user selects the feedback type.

Next they can enter title and detailed description and click submit. onSubmit callback prop is exposed to allow the componet to be integrated into our apps and values user entered are returned in the callback data.

I think would look better if the sentiment feedback dialog is not shown behind the title and description dialog. I think also would be good if the height of the two dialogs was the same so user feels like it is just one dialog, which to be honest it could be.

Description should be multi-line text field. Maybe default to something like 3 rows.

The open prop doesn't seem to be working. I would expect dialog to open when I set to true. Please also add a new test for this.

i think would be good if the feedback dialog was anchored to the position of the button rather than in middle of screen like it as at the moment. Maybe also there needs to be prop to control which position it is anchored to. In the app suite we probably will have the floating button in bottom right of the screen and then I would want the feedback form to also appear in bottom right.

Rather than 'message' I think 'sentiment' is a better property name. Looks like the MUI icons already use sentiment also.

index.js file at top level should be updated to export your component so it can be imported by apps that use this library.

Could you please rename BootstrapDialogTitle to just DialogTitle?

@nikitharani I've pushed updates for following changes we discussed earlier. Please check if they look OK to you.

Fixed the tests for dialog open/close.
Added parameterised test for the 3 different sentiment types.
Fixed issue with sentiment not being returned in onSubmit.
Re-factored the component to just contain one dialog rather than two.

Please rename to DialogTitle not Dialogtitle. We try to use CamelCase for multi-word names.

I think would look better if the sentiment feedback dialog is not shown behind the title and description dialog. I think also would be good if the height of the two dialogs was the same so user feels like it is just one dialog, which to be honest it could be.

I made these changes in my last set of commits.

DialogTitle
renaming to DialogTitle has a conflict with built-in react DialogTitle component. so I am renaming it to DialogeTitle.

You can resolve this by aliaising the MUI DialogTitle when you import it.
import {DialogTitle as MuiDialogTitle} from '@mui/material'.
You will see I already do this for the Dialog itself when importing inside FeedbackForm.

Please can you add a story and test for the DialogTitle component?
The story would probably need to create a dialog so you can show the dialog title component.
I think 3 tests.

Check title is correctly shown.
Check the close button is shown when onClose is provided.
Check the close button is not shown when onClose is not provided.

Your new position property needs to be exposed as a property which user can change. Should also define a default.
Should appear here

and here


snowflake_grant_privileges_to_role all_privileges=true not updating the DB state correctly

Terraform CLI and Provider Versions
terraform {
required_providers {
snowflake = {
source  = "Snowflake-Labs/snowflake"
version = "0.70.1"
}
}
}
Terraform Configuration
terraform script #1:

resource "snowflake_grant_privileges_to_role" "warehouse_grant" {
  role_name = ROLE_1
  on_account_object {
    object_type = "WAREHOUSE"
    object_name = snowflake_warehouse.warehouse.name
  }
  all_privileges    = true
  with_grant_option = true
}

after apply successfully creating the warehouse all_privileges for the role.
now some other script removed ["USAGE", "MONITOR"] privileges  from the role in the DB

now when we again plan/apply above terraform script we expect again all_privileges  in this case missing ["USAGE", "MONITOR"] should be created.but its not creating these privilages db still missing ["USAGE", "MONITOR"]

Expected Behavior
create all missing privileges in the DB after reapply the terraform script
Actual Behavior
Not creating missing privileges in the DB after reapply the terraform script
Steps to Reproduce

terraform apply

2.Remove/Revoke  any privilege's  e.g ["USAGE", "MONITOR"] from the DB directly by script or any ways.
3.Again plan and apply
terraform plan
terraform apply
How much impact is this issue causing?
Medium
Logs
No response
Additional Information
if we specify the privilages explicitly them its working fine instead of  all_privileges
resource "snowflake_grant_privileges_to_role" "warehouse_grant" {
role_name = local.role_name
on_account_object {
object_type = "WAREHOUSE"
object_name = "DEV_TEST_WH"
}
privileges        = ["APPLYBUDGET", "MODIFY", "USAGE", "MONITOR", "OPERATE"]
with_grant_option = true
}

Hey @mohideenbilal 👋
Is there a possibility to try the latest version of the provider (0.85.0 at that time) to see if the issue has been resolved or not? In the latest versions, there were a lot of refactors in grant resources, which could resolve the issue. Also, in the latest versions we provided a newer version of snowflake_grant_privileges_to_role named snowflake_grant_privileges_to_account_role. We prepared a guide that may help you with migration in case where snowflake_grant_privileges_to_account_role would work for you. The migration guide.

We're having a similar issue that I think might be related. If its not I'd happily open another bug.
Following resource is unable to be. imported
resource "snowflake_grant_privileges_to_account_role" "XX_warehouses" {
  privileges = ["USAGE"]
  account_role_name = "AD_HOC_USR"
  always_apply      = false
  with_grant_option = false
  on_account_object {
    object_type = "WAREHOUSE"
    object_name = "XX_WH"
  }
}

terraform import snowflake_grant_privileges_to_account_role.ad_hoc_warehouses "AD_HOC_USR|false|false|USAGE|OnAccountObject|WAREHOUSE|XX_WH"
import command just returns "killed"
Turning the logs higher, seems to give no indicator as to whats going on.
The import will lock the workspace, but not unlock the workspace.
No SQL commands are run against the db by the user when checked.

@mohideenbilal Hey again,
Sorry, but I didn't understand the issue at first. The all_privileges option is not tracking the privileges on the Snowflake side for a few reasons. In the future, we're planning to go through this issue and decide what the solution should be. For now, If you would like Terraform to be aware of privileges in Snowflake and re-apply privileges that could be removed by another config, you have to explicitly pass all of the privileges for that object type individually in the privileges field.
@ike-lacey "XX_warehouses" is the name of the resource and in the command, you refer to it as "ad_hoc_warehouses". Is that it or this was changed before putting it into a comment? (I think you can create a new issue for that if that's not the case and you are not able to import the resource)

Hey @mohideenbilal
In the latest version, we provided a flag in the newer resource snowflake_grant_privileges_to_account_role called always_apply which will apply all_privileges on every terraform apply and possibly terraform plan which could solve your issue for now. As I previously said the option for tracking privileges is not yet decided, but we'll have this conversation in the future and change the resource accordingly. If the provided workaround (always_apply flag) is working for you and you're fine with using it, please close this issue.

@sfc-gh-jcieslak I am using the 0.87.2 version and using always_apply flag. according to the docuemtntaion this flag should cause a plan everytime terraform runs, but that is not doing that for me.
Here is an example of terraform config file I have:
resource "snowflake_grant_privileges_to_account_role" "db_full_arl_8" {
  count             = length(local.db_role_mapping_1)
  with_grant_option = false
  all_privileges    = true
  always_apply      = true

  account_role_name = local.db_role_mapping_1[count.index]["role"]
  on_schema_object {
    future {
      object_type_plural = "VIEWS"
      in_database        = local.db_role_mapping_1[count.index]["db"]
    }
  }
}

@bob-zarkoob I'll take a look at it as soon as possible

Thank you so much @sfc-gh-jcieslak.

Hey as the issue got a little bit off topic, @bob-zarkoob could you open another issue describing your issue with always_apply? It would be much easier to manage on our end. I've checked this case and it looks like the always_apply is not read from the configuration during resource creation. That means you have to unset it and set it back to make it work (or create it with always_apply set to false and set it to true in the next terraform apply). Please use this workaround for now, and create another issue, so I'll put it into our backlog to work on it as soon as possible.


CCAP-252 Handle illegal arguments

🔗 Jira ticket
CCAP-252
✍ Description

Handle IllegalArgumentException

With this global adviser, when an illegal character is passed to an argument, in this case lang=../.., it is handled by redirecting to the /error screen, thus handling the exception.

@coltborg This looks great.
Is there anything else we want to communicate to devs when errors of this type occur?
This is what we see in dev:

@coltborg This looks great. Is there anything else we want to communicate to devs when errors of this type occur?
This is what we see in dev:

Ya, that would be nice! Looking into how to pass the error message.


vendor: Update libvirt.org/go/libvirt

What this PR does / why we need it:
Update the libvirt Go bindings to match the C library, which was recently updated as part of https://github.com/kubevirt/kubevirt/pull/10050
Release note:
NONE

@victortoso FYI

/lgtm

Update the libvirt Go bindings to match the C library

Minor updates on the Go module but looks reasonable to keep it in sync.

/approve

/retest


Menu for best practices on administering service instances with ODB

Signed-off-by: Maria Ntalla<EMAIL_ADDRESS>

Thanks, tinygrasshopper


FF-Perf: Request failed with status code 429 in tokens connector

When running a performance test of 300 token mints, when the go routines first start sending API requests, the Nest.js token connector gets overwhelmed and returns 429: Too Many Requests for all but 65 requests; hinting that the connector is rate limiting requests. Maybe there's a way to increase the limit?
tokens-connector.log
ff-perf.log
log_ethconnect_0.log
log_firefly_core_0.log.zip

This is probably ethconnect rejecting the requests. Because we're using the direct webhook plugin of ethconnect and not a more robust message queue, it has a finite configurable limit of transactions in flight. See https://github.com/hyperledger/firefly-ethconnect/issues/185
Assuming we don't want to use a "better" message queue in ethconnect for this test, this is probably a matter of tuning MaxInFlight on ethconnect to be a bit higher.

So I don't think this one is necessarily to fix in FireFly core. In core there is retry configurable to handle 429s, if you have a hosted copy of ethconnect running in a cloud environment with rate limiting, where that limit is too low (likely the case here).
Having configurable retry in tokens connectors would be a good thing to raise against those individually, if retry is not configurable there.


Fix flaky e2e tests

In e2e tests like TestProxyHairpinIPv4:

A test Service is created
A test Pod for the Service is created
Probe the Service using code data.RunCommandFromPod(data.testNamespace, pod, container, []string{"wget", "-O", "-", url, "-T", "5"})

When using the command wget -O url -T 5 to connect to a Service without an Endpoint, which is implemented by AntreaProxy, a rejection message will be received, and no retry will be attempted. There is a chance that even if the test Pod is ready, the Service may not be fully synced, resulting in a rejection message and a failed test. This PR fixes the flaky test by adding retry logic.

/test-e2e
/skip-conformance
/skip-networkpolicy

/test-e2e

/test-e2e

/test-e2e


Add custom context name support

Defaulting to Patient context to preserve backward compatibility
todo

[x] address Corey's feedback on adding documentation

References

WFFHCOHORT-861

@deekim -- I do not have a model in mind. I just saw that there is a model-info argument in the usage block and didn't remember that CohortCLI is FHIR based only.

@csandersdev would you happen to know?

I don't have a firm answer. There is a contextInfo node in the modelinfo, but I'm not sure whether the translator requires those for a valid using statement or not.
The Unspecified and Population names are hard-coded into the FHIRModelResolver. There is also some hard-coding in the SearchParamFhirRetrieveProvider that will check for a context named Patient.

I do not have a model in mind. I just saw that there is a model-info argument in the usage block and didn't remember that CohortCLI is FHIR based only.

👍 Gotcha, I think that just if you had a custom FHIR-based IG, but I could be wrong there.


Issue while installing composer

Q
A

Bug report?
[ ]

Feature request?
[ ]

Support request?
[ ]

Hello i'm trying to install the composer but i have this error.
An error occurred when executing the "'cache:clear --no-warmup'" command:
In DocParser.php line 995:
  Notice: Trying to access array offset on value of type null
Any suggestion ?

Hello there!
You should change the content of this line to:
while ($this->lexer->lookahead!==null && $this->lexer->lookahead['position'] === ($this->lexer->token['position'] + strlen($this->lexer->token['value']))
The full explanation to do that may be found on https://github.com/doctrine/annotations/issues/273

Thanks a lot. but i get this error if i change the content of the line   [Syntax Error] Expected Doctrine\Common\Annotations\DocLexer::T_NAMESPACE_S
EPARATOR, got '(' at position 25 in class Cocorico\CoreBundle\Document\List
ingAvailabilityTime.

getting this error, facebook info is filled out.  Not sure what would cause it.  The FB set up instructions seem to be a little different then current FB UI but we id enter Valid OAuth Redirect URIs.  Any help would be great.  Thanks
#Facebook
cocorico.facebook.app_id: '26xxxxxxxxxxxxxxxxxx'
cocorico.facebook.secret: '25eded44xxxxxxxxxxxxxxxxxxxxe'
In VariableNode.php line 85:
The path "hwi_oauth.resource_owners.facebook.client_id" cannot contain an e
mpty value, but got null.

Hello,
It can be related to your PHP version. For now Cocorico has been tested and works with php 7.1 and 7.2.

Thanks i fixed it by changing PHP version.


Vulnerable to: RUSTSEC-2024-0370

Nativelink uses the proc-macro-error crate which is unmaintained for longer then 2 years.
Observed in Vulnerabilities: https://securityscorecards.dev/viewer/?uri=github.com/TraceMachina/nativelink
RUSTSEC-2024-0370

This is pulled in by aws-smithy-protocol-test as a test-only dependency. As such it's not pretty in the vuln logs but (currently) doesn't affect runtime code.
The fix here would be to bump/change these dependencies in the AWS Rust SDK upstream (which initially seems surprisingly straightforward) or to remove our dependence on the affected crate (which initially seems like a nontrivial effort).

In aws-smithy-protocol-test, the dependency on assert-json-diff was originally set to ^1.1, which brought in extend as a dependency (^0.1.0). However, starting with assert-json-diff 2.0.2, extend is no longer required. Additionally, extend itself resolved this issue in version 1.2 by removing its dependency on proc-macro-error. So, updating either assert-json-diff to 2.0.2 or extend to 1.2+ will clear the vulnerability entirely


Add Smalt dataset

@fredeBio do you have a run number for S.malt. ?

Yes, they are in the MS.
_1a8l7wu
mknhxpv8
pgfmaxx5
yy72i755
78eu9zl0

I must have deleted them when I cleaned up old runs. I'm re-running them now and will save them in the old run directories so that all links remain valid.

I keep getting mailer demons. You might have put in a wrong email address.
----- Original Message -----
From: Carsten Fortmann-Grote @.>
To: mpievolbio-scicomp/rarefan @.>
Cc: fredeBio @.>, Mention @.>
Sent: Tue, 06 Sep 2022 17:34:28 +0200 (CEST)
Subject: Re: [mpievolbio-scicomp/rarefan] Add Smalt dataset (Issue #4)
I must have deleted them when I cleaned up old runs. I'm re-running them now and will save them in the old run directories so that all links remain valid.
--
Reply to this email directly or view it on GitHub:
https://github.com/mpievolbio-scicomp/rarefan/issues/4#issuecomment-1238316907
You are receiving this because you were mentioned.
Message ID: @.***>

yes, wrong email (and multiple reruns ...)
On Tue, 2022-09-06 at 10:14 -0700, fredeBio wrote:

I keep getting mailer demons. You might have put in a wrong email
address.
----- Original Message -----
From: Carsten Fortmann-Grote @.>
To: mpievolbio-scicomp/rarefan @.>
Cc: fredeBio @.>, Mention @.>
Sent: Tue, 06 Sep 2022 17:34:28 +0200 (CEST)
Subject: Re: [mpievolbio-scicomp/rarefan] Add Smalt dataset (Issue
#4)
I must have deleted them when I cleaned up old runs. I'm re-running
them now and will save them in the old run directories so that all
links remain valid.

New runs successfully finished and linked to old dataset URLs.


Draft of CSR switching in DynamicFragment

What do these changes do?

Fix a condition bug in the Mutate function
Give a demo on switching the CSR structure from MutableCSR to DeMutableCSR in CopyFrom

Some questions need to be decided:
(1) the ID of vertices is bounded by ivnum_ + ovnum_
(2) In identical mode, the edges are copied by add_forward_edges method
(3) the copy method in reversed mode,  the in/out edges constructed created by reversing the in/outer edges of the source fragment (add_reversed_edges)
Related issue number
#1602

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #1799 (e164c95) into main (2695783) will decrease coverage by 32.86%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1799       +/-   ##
===========================================
- Coverage   72.99%   40.12%   -32.87%
===========================================
  Files          89       89
  Lines        9804     9816       +12
===========================================
- Hits         7156     3939     -3217
- Misses       2648     5877     +3229

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_cython_ast.py
0.00% <0.00%> (-100.00%)
:arrow_down:

...ython/graphscope/tests/unittest/test_serailaize.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/analytical/udf/patch.py
3.47% <0.00%> (-96.53%)
:arrow_down:

python/graphscope/tests/unittest/test_lazy.py
0.00% <0.00%> (-96.30%)
:arrow_down:

...thon/graphscope/tests/unittest/test_scalability.py
0.00% <0.00%> (-96.16%)
:arrow_down:

python/graphscope/tests/unittest/test_app.py
0.00% <0.00%> (-95.66%)
:arrow_down:

...hon/graphscope/tests/unittest/test_create_graph.py
0.00% <0.00%> (-92.00%)
:arrow_down:

python/graphscope/tests/unittest/test_graph.py
0.00% <0.00%> (-85.57%)
:arrow_down:

python/graphscope/tests/unittest/test_context.py
0.00% <0.00%> (-81.04%)
:arrow_down:

... and 40 more

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 2695783...e164c95. Read the comment docs.


2020-08-13 - Add recaptcha to B001243

Looks like the recaptcha was omitted from the recent rewrite on this form.  Here's a PR that adds it!
Thanks, Jason C.

Hello jason44406,
I am looking to use this YAML file to contact my reps, can you guide me on how to use it?


Allow client reconnection

(Implements #147)
I've set a poll to gather what you guys think about this feature being enabled by default or not. What do you think?

Things to consider when reconnection is enabled:

The rooms will live for a bit longer (15 seconds by default), even if there're no more clients connected to it and autoDispose is set to true.
Reconnecting clients will skip requestJoin method, since it was authorized already before.

Usage
class MyRoom extends Room {
  onJoin (client) {
    this.state.setupClient(client.sessionId);
  }

  async onLeave (client) {
    this.state.inactivateClient(client.sessionId);

    try {
      // allow client to reconnect within 20 seconds
      const reconnectedClient = await this.allowReconnection(client, 20);
      this.state.reactivateClient(client.sessionId);

    } catch (e) {
      this.state.removeClient(client.sessionId);
    }
  }
}

Will we be able to change allowReconnection dynamically?

Hey @sandrosc, yes, it's possible to enable and disable allowReconnection at any time.

@endel awesome!

It turns out I've overlooked this feature, and the implementation will need to be slightly different. I've updated the usage example in the description of this pull-request.
You may call allowReconnection() per client, and it will return a promise, that is resolved whenever the user has reconnected. The promise is rejected after the timeout specified. (15 seconds by default)
This way you have full control of the events, and you can flag users as inactive accordingly.

if i would upgrade to the version. do i need to change something so that everything works the same?

@Wenish onAuth will continue to be called before joining. I think requestJoin won't be necessary since this method is used to determine which room the client will join, and we already know exactly which room the client should connect to.
If you're using version 0.9.x there's nothing else to update! allowReconnection is totally optional.


Tooltip and body {height: 100%}

If I write body {height: 100%}, tooltip position wrong,
This option: 'boundariesElement: 'viewport' in Popper.js, resolve this issue,
Please fix it

Related to this Popper.js issue https://github.com/FezVrasta/popper.js/issues/302 not to our hand but it can be useful to allow users to choose how Popper.js will prevent the overflow

This has been fixed in 1.12.3, note that there's still https://github.com/FezVrasta/popper.js/issues/331 that could cause some headaches, PRs are welcome!

Fixed by : https://github.com/twbs/bootstrap/pull/23626 and the awesome work of @FezVrasta


Add new "routed" backend.

Add a new "routed" backend that forwards traffic using regular IP
routes.  Useful on private layer2 networks (or dedicated virtual
networks).

@anguslees yeah, something along subnet lines. flat or l2flat? Or maybe emphasize that the host is used the gateway: host-gw?
/cc @philips you're good with names.

@anguslees yeah, something along subnet lines. flat or l2flat? Or maybe emphasize that the host is used the gateway: host-gw?
/cc @philips you're good with names.

I'm +1 on host-gw. I think we should roll with that name for now and mark this backend as a tech preview until it matures a bit.


add sshfs to tunnel script for storage deals

(see commit message)

maybe spit out a warning / error if sshfs isn't installed?

The script should now exit if either of the first two commands fails. The third one is inside a command substitution, so that's trickier.

It would be nice to have a convenient way to shutdown sshfs and the tunnel, but not necessary for now.

You're right; for now I've found that killall ssh is enough for my needs. I'm not sure if there could be a cleaner way. I guess we could teach the script to stay running in the foreground, and stop all the sub-processes on ^C, but then you lose all the source goodness with env vars.


iptables cookbooks breaks at load time if it's in the dependency tree on non-linux platforms

Cookbook version
6.0.0
Chef-client version
irrelevant
Platform Details
anything not in https://github.com/chef-cookbooks/iptables/blob/master/attributes/default.rb#L41-L48
Scenario:
Run chef.
Steps to Reproduce:
Use a cookbook, such as openssh, which depends on the iptables cookbook but runs on a platform that iptables doesn't support.
Update to iptables 6.0.0 in a policy (or whatever) and run chef.
Expected Result:
Chef converges with no major changes.
Actual Result:
Chef blows up at load time with
================================================================================
Recipe Compile Error in C:/chef/cache/cookbooks/iptables/resources/chain.rb
================================================================================
Chef::Exceptions::ValidationFailed
----------------------------------
Property config_file must be one of: String!  You passed nil.

This points to https://github.com/chef-cookbooks/iptables/blob/master/resources/chain.rb#L23 , which specifies that the default value is an attribute, which comes from https://github.com/chef-cookbooks/iptables/blob/master/attributes/default.rb#L41-L48
Suggested Fix
Change it to default: node['iptables']['persisted_rules_iptables'].to_s?
Obviously the cookbook doesn't support these other OSes and doesn't run on them, but it can get in the dependency tree with shared cookbooks and blow up at load time, which is awful.
We've observed this on Windows, AIX, and Solaris, example error above from Windows. (Note that AIX is supported by the openssh cookbook, so this technically breaks that!)

Fixed via https://github.com/chef-cookbooks/iptables/commit/4a536f241f14b930ba4cd3610717f2aeaabe0a49


fix: new arch android interface

Summary:
fixes #652
Resolved incompatible interface generation for the codegen source.

Test Plan:

@vonovak Thanks, I have removed the tmp solution from the description as after further investigation this issue is related to measure function from ViewManager not being called in some setups like RNTA

Me and @vonovak made #657 - which might solve the root cause of the problem.


model loading fails due to breaking changes in 1.20dev?

i am running https://github.com/microsoft/webnn-developer-preview/blob/main/demos/sd-turbo (or my img2img fork https://github.com/eyaler/webnn-developer-preview/blob/main/demos/sd-turbo), and trying to upgrade the ORT dev version to test performance. The last version that works for me is 1.20.0-dev.20240913-904b850b44. the next version 1.20.0-dev.20240914-c63dd0234b, as well as any version after that i tested up to 1.21.0-dev.20241018-b4cb937440, result in failure to load models. e.g:
[Load] UNet failed, TypeError: Cannot read properties of undefined (reading 'a')
    at $b (https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.mjs:84:209)
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[732]:0xa6273
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[12416]:0x1266e92
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[12455]:0x127db24
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[12477]:0x128c7a7
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[2398]:0x2c389d
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[3366]:0x4691ac
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[3369]:0x46d475
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[3398]:0x47bedd
    at https://cdn.jsdelivr.net/npm/onnxruntime-web@1.20.0-dev.20240914-c63dd0234b/dist/ort-wasm-simd-threaded.jsep.wasm:wasm-function[2104]:0x2545ae

well my issue was on chrome 129. once upgraded to 130 all mentioned versions work!


ci: download a test model to serve during the test session

@z103cb I think you had in mind a small model just for this usecase?


D8: Program type already present: io.flutter.plugins.webviewflutter.BuildConfig when youtube_flutter_player added with webview_flutter[BUG]

Hi , when I am adding plugins:
youtube_player_flutter
webview_flutter
together then there is an error in building the apk
D8: Program type already present: io.flutter.plugins.webviewflutter.BuildConfig
com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
Learn how to resolve the issue at https://developer.android.com/studio/build/dependencies#duplicate_classes.
Program type already present: io.flutter.plugins.webviewflutter.BuildConfig
at com.android.builder.dexing.D8DexArchiveMerger.getExceptionToRethrow(D8DexArchiveMerger.java:131)
at com.android.builder.dexing.D8DexArchiveMerger.mergeDexArchives(D8DexArchiveMerger.java:118)
at com.android.build.gradle.internal.transforms.DexMergerTransformCallable.call(DexMergerTransformCallable.java:102)
at com.android.build.gradle.internal.tasks.DexMergingTaskRunnable.run(DexMergingTask.kt:444)
at com.android.build.gradle.internal.tasks.Workers$ActionFacade.run(Workers.kt:335)
at org.gradle.workers.internal.AdapterWorkAction.execute(AdapterWorkAction.java:50)
at org.gradle.workers.internal.DefaultWorkerServer.execute(DefaultWorkerServer.java:47)
at org.gradle.workers.internal.NoIsolationWorkerFactory$1$1$1.create(NoIsolationWorkerFactory.java:65)
at org.gradle.workers.internal.NoIsolationWorkerFactory$1$1$1.create(NoIsolationWorkerFactory.java:61)
at org.gradle.internal.classloader.ClassLoaderUtils.executeInClassloader(ClassLoaderUtils.java:98)
at org.gradle.workers.internal.NoIsolationWorkerFactory$1$1.execute(NoIsolationWorkerFactory.java:61)
at org.gradle.workers.internal.AbstractWorker$1.call(AbstractWorker.java:44)
at org.gradle.workers.internal.AbstractWorker$1.call(AbstractWorker.java:41)
at org.gradle.internal.operations.DefaultBuildOperationExecutor$CallableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:416)
at org.gradle.internal.operations.DefaultBuildOperationExecutor$CallableBuildOperationWorker.execute(DefaultBuildOperationExecutor.java:406)
at org.gradle.internal.operations.DefaultBuildOperationExecutor$1.execute(DefaultBuildOperationExecutor.java:165)
at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:250)
at org.gradle.internal.operations.DefaultBuildOperationExecutor.execute(DefaultBuildOperationExecutor.java:158)
at org.gradle.internal.operations.DefaultBuildOperationExecutor.call(DefaultBuildOperationExecutor.java:102)
at org.gradle.internal.operations.DelegatingBuildOperationExecutor.call(DelegatingBuildOperationExecutor.java:36)
at org.gradle.workers.internal.AbstractWorker.executeWrappedInBuildOperation(AbstractWorker.java:41)
at org.gradle.workers.internal.NoIsolationWorkerFactory$1.execute(NoIsolationWorkerFactory.java:56)
at org.gradle.workers.internal.DefaultWorkerExecutor$3.call(DefaultWorkerExecutor.java:215)
at org.gradle.workers.internal.DefaultWorkerExecutor$3.call(DefaultWorkerExecutor.java:210)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at org.gradle.internal.work.DefaultConditionalExecutionQueue$ExecutionRunner.runExecution(DefaultConditionalExecutionQueue.java:215)
at org.gradle.internal.work.DefaultConditionalExecutionQueue$ExecutionRunner.runBatch(DefaultConditionalExecutionQueue.java:164)
at org.gradle.internal.work.DefaultConditionalExecutionQueue$ExecutionRunner.run(DefaultConditionalExecutionQueue.java:131)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:64)
at org.gradle.internal.concurrent.ManagedExecutorImpl$1.run(ManagedExecutorImpl.java:48)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at org.gradle.internal.concurrent.ThreadFactoryImpl$ManagedThreadRunnable.run(ThreadFactoryImpl.java:56)
at java.lang.Thread.run(Thread.java:748)
Caused by: com.android.tools.r8.CompilationFailedException: Compilation failed to complete
at com.android.tools.r8.utils.t.a(:55)
at com.android.tools.r8.D8.run(:11)
at com.android.builder.dexing.D8DexArchiveMerger.mergeDexArchives(D8DexArchiveMerger.java:116)
... 34 more
Caused by: com.android.tools.r8.utils.AbortException: Error: Program type already present: io.flutter.plugins.webviewflutter.BuildConfig
at com.android.tools.r8.utils.Reporter.a(:21)
at com.android.tools.r8.utils.Reporter.a(:7)
at com.android.tools.r8.utils.t.a(:36)
... 36 more
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:mergeDexRelease'.

A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
com.android.builder.dexing.DexArchiveMergerException: Error while merging dex archives:
Learn how to resolve the issue at https://developer.android.com/studio/build/dependencies#duplicate_classes.
Program type already present: io.flutter.plugins.webviewflutter.BuildConfig

I need to add webview for specific feature in my app.
How can this be solved.
Somewhere i heard that adding: android.enabledR8 = true in gradle.properties might fix this but it is already there.
What might be the fix?

+1
I have
webview_flutter: ^0.3.22+1
youtube_player_flutter: 6.1.1
(some videos are not playing in 7.0.0+3, hence i m using 6.1.1)

@prashantspandey  This package already depends upon flutter_inappwebview. You may consider using that for webview instead of webview_flutter.

@mirkarpude can you mention some video id for videos that's not playing ?

@mirkarpude can you mention some video id for videos that's not playing ?

sure,
https://www.youtube.com/watch?v=Hy7DrXJTRiY
one of a link, in some devices like, Samsung J1- and J7, POCO F2, Vivo v13, Mi note 5pro, mi 6 (checked personally with 4G and Broadband speen),
but on some devices its working like my phab 2 pro and one+ 7 t.
the code was same copied as in 7.0.0 example

@mirkarpude the video is working fine for my devices. Please check with these apks.


Update Admin CLI shard operations to support SQL databases

What changed?
Update Admin CLI shard operations to support SQL databases

Why?
To address https://github.com/uber/cadence/issues/2777

How did you test it?
local test with mysql

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build a41916e3-b25f-4182-8133-81a8059f8111

1 of 62 (1.61%)  changed or added relevant lines in 4 files are covered.
26 unchanged lines in 9 files lost coverage.
Overall coverage increased (+0.02%) to 59.921%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tools/cli/adminCommands.go
0
2
0.0%

tools/cli/adminTimers.go
0
25
0.0%

tools/cli/adminDBScanCommand.go
0
34
0.0%

Files with Coverage Reduction
New Missed Lines
%

common/membership/rpServiceResolver.go
2
73.58%

common/persistence/cassandra/cassandraPersistenceUtil.go
2
92.62%

common/task/weightedRoundRobinTaskScheduler.go
2
89.64%

service/history/handler.go
2
46.67%

service/history/queue/transfer_queue_processor.go
2
57.37%

tools/cli/adminTimers.go
2
0%

common/types/history.go
4
24.69%

service/history/historyEngine.go
4
70.86%

service/history/execution/context.go
6
68.39%

Totals

Change from base Build 9a0bc4bb-9b40-4612-b797-04903e0b7c03:
0.02%

Covered Lines:
88224

Relevant Lines:
147233

💛  - Coveralls


Suppress garbled output of dynamic-colors init in termite terminal

Except for #23, dynamic-colors works for, thanks for it !
But dynamic-colors init gives me an garbled output like this:
» dynamic-colors init
]708;#fdf6e3%

It's easy to work around using dynamic-colors init > /dev/null but it would still be good to fix it.

btw, this is using the termite terminal. On xterm or urxvt it works fine.

The compability check works fine in my termite terminal btw.

Ugh, just realized that all outputs are garbled in termite:
» dynamic-colors switch solarized-light
]708;#fdf6e3%

@varac
You can just comment out the "border" case statement in dynamic-colors. It looks like escape code 708 isn't supported by some terminals (such as xfce4-terminal).
diff --git a/bin/dynamic-colors b/bin/dynamic-colors
index f40c6ec..d2cc992 100755
--- a/bin/dynamic-colors
+++ b/bin/dynamic-colors
@@ -80,8 +80,8 @@ change_color_osc () {
       send_osc 14 "$2" ;;
     highlight)
       send_osc 17 "$2" ;;
-    border)
-      send_osc 708 "$2" ;;
+    #border)
+      #send_osc 708 "$2" ;;
   esac
 }


Charms Notches

The charms Dont have Notches, all new charms cost 0 Notch

Did you give yourself the Transcendence charms with debug on an existing file, or create a new file with Transcendence installed? If the former, this is a known issue (see #24).


Parameterize search (use dropdowns)

This should be a bit more friendly than a long text command.

See #679. v2 is no longer being maintained.


[mapping template] [request property type check]create address validation and get address by id

a/c:

request property type check are supported for create address validation and get address by id

regression on address

streetName not set in mapping file
not provide the priority, still check on kraken side,
should remove this check:

pass


DBZ-5076 Fix typo: archive_log_target to archive_lag_target

https://issues.redhat.com/browse/DBZ-5076

Applied to main (=2.0) and branch 1.9.


[BUG] new view ... get_account shows wrong results

Describe the bug

near view zavodil.poolv1.near get_account '{"account_id": "my.near"}' --nodeUrl https://rpc.mainnet.near.org

bad result
View call: zavodil.poolv1.near.get_account({"account_id": "my.near"})
{
account_id: 'my.near',
unstaked_balance: '0',
staked_balance: '0',
can_withdraw: true
}

near call my_lockup.lockup.near refresh_staking_pool_balance '' --accountId my.near --nodeUrl https://rpc.mainnet.near.org

good result:
Fetching total balance from the staking pool @zavodil.poolv1.near
The current total balance on the staking pool is xyzpt (correct amount staked a couple of days ago)
Expected behavior
the first command should show correct amount of staked tokens xyzpt like the second one does
I don't know if this is related or not but the staked amount showing in https://wallet.near.org/ it is also zero.

@iamveritas Possibly your issue is not configuring your NEAR_ENV to mainnet?
Try:
NEAR_ENV=mainnet near view POOL_NAME.poolv1.near get_account '{"account_id": "YOUR_ACCOUNT.near"}'
Let me know if that works.

I am passed that at the moment, not really able to go back to try it anytime soon...

ok, no worries! I'm going to close this issue then.  Please feel free to reach out or submit another issue if this does not work for you.


Update README.md

Please change the README as 'images', 'attachments' and many other mandrill properties are packed into 'message' object. The example is confusing and leads to waste of time when trying to figuring our what doesn't work. Thanks!

Coverage remained the same at 100.0% when pulling 55b9bb5b3a5524a873da47416ce9899a41550a13 on triclozan:patch-1 into fd55199960f8bf05d25153a0b110020d9af56373 on Rebelmail:master.

Please merge this... I went through a phase in my life trying to figure out what was wrong with my code...


Capture pipeline jobs as experiments, fix typos

REQUIRED: Add a summary of your PR here, typically including why the change is needed and what was changed. Include any design alternatives for discussion purposes.

- Capture pipeline runs as experiments
- Fix minor typos

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[x] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[x] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

If you are opening a PR for Community Notebooks under the notebooks/community folder:

[ ] This notebook has been added to the CODEOWNERS file under the Community Notebooks section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

If you are opening a PR for Community Content under the community-content folder:

[ ] Make sure your main Content Directory Name is descriptive, informative, and includes some of the key products and attributes of your content, so that it is differentiable from other content
[ ] The main content directory has been added to the CODEOWNERS file under the Community Content section, pointing to the author or the author's team.
[ ] Passes all the required formatting and linting checks. You can locally test with these instructions.

@andrewferlitsch can you ptal?

I will split this into two PRs.


Suggestions for improving documentation (JOSS review)

Here are some suggestions on how to improve the docs, again with reference to the JOSS review https://github.com/openjournals/joss-reviews/issues/7334,
This is not yet complete, but as far as I got today.
I added "[X]" to some of these points with X = A, B, C, ... so you can easily
reference them if you want to comment. The headings just follow the structure of your documentation.
Overview
Summary
[A] After "DiscreteEntropy implements a large collection of entropy
estimators.", add a list of all implemented estimators, so the user can get a
quick, central overview (otherwise one has to scroll through the "Estimators"
section where there is no table of contents that lists all estimators). Also,
list the measures that one can calculate with your package (cross entropy, KL
divergence, ...,  mutual information, conditional entropy, etc.) and link to
their specific API documentation.
In general, the documentation is very good in terms of how comprehensive
it is, but since most of it is API documentation, it is lacking guidance
beyond the "Overview" page.
Installing Discrete Entropy
Add spaces before "higher are supported. ..."
to make the markdown list render correctly:
1. If you have not done so already, install [Julia](https://julialang.org/downloads/). Julia 1.8 and
   higher are supported. Nightly is not (yet) supported.

[B] Mention the current limit to Julia "<=1.10".
Basic usage
[C] Add a new subsection (##) "Contributing" and link to CONTRIBUTING.md.
This would be a good bonus:
If a contributor would want to add their own estimator, how would that work?
(What is the interface, or what functions would need to be implemented?)
Data
CountData
[D] Why is the total number of samples CountData.N represented with a
floating point number? Is it not always an integer? Is the same true for the
multiplicity matrix?
Below the matrix, change "We have 1 bin with a 4 elements" with
"We have 1 bin with 4 elements". This one "a" can make it really
confusing :-)
from_data
"A 0 value in the histgram histogram ..."
Estimate_H
estimate_h
"Return the estimated in nats of Shannon entropy of data in nats
using estimator."
"Note
While most calls to estimate_h take a CountData struct, this is not true for
every estimator, especially those that work directly over samples, or need the
original structure of the histogram.
For a complete list of methods of the function, try
julia> methods(estimate_h)

"
[E] The call to methods returns a long list of method calls that take a CountData
object and only a single call that takes a SampleVecor:
...
[25] estimate_h(data::SampleVector, estimator::Type{T}, ::Type{Bootstrap}; seed, reps, concentration) where T<:AbstractEstimator
...

However, in this case every estimator seems to be allowed
(T<:AbstractEstimator), so it does not get clear at this point
which estimators require which kind of data.
[F] Put
"This function is a wrapper indended to make using the libary easier. For finer
control over some of the estimators, it is advisable to call them directly,
rather than through this function."
right under
"Return the estimate in nats of Shannon entropy of data using estimator."
(or the revised version).
Estimators
[G] I would advise putting the API reference for the abstract base types
at the end, since they are mainly of interest for contributors.
On the other hand, you should put the types for the concrete estimators here
(and in the overview as a list). Since you advertised estimate_h which needs
them as inputs earlier in the docs, you should provide these types now, since
one cannot always guess from the function names how the corresponding types are
called exactly (e.g. LaPlace --- is this  really how it should be written, by
the way?).
grassberger
The link in
"Equation 13 from Finite sample corrections to entropy and dimension estimate"
yields a 404.
schurmann
"This There is no one ideal ..."
schurmann_generalized
The link underneath the function signature should go to the end and
have "External Links" above it.
[H] "Computes the generalised Schurmann entropy estimation, given a countvector
data and a vector of xi values."
This says "vector of xi values" but the function signature above it says
xis::Distribution.
bub
It would be good to explain the optional keyword arguments.
The external link goes to a blank page saying:
"Your session has timed out. Please go back to the article page and click the PDF link again."
(The URL contains a token that is invalid.)
chao_wang_jost
The external link does not open for me.
unseen
The docs do not render correctly for this one, seems a "`" is missing.
The external link requires an google account and access permission.
In general it would be good to give dois of the cited papers,
so people can find them even when the links are not working.
minimax
The math is not rendered correctly.
nsb
The function signature does not include type information.
ansb
Should say what keyword argument undersampled is (N/K?).
pym
[I] This function does not conform to the standard API.
What are the arguments mm and icts? Why does it not
take CountData?

Here is the rest of my suggestions.
@kellino I am pinging you here, in case you did not see this issue.
Mixed Estimators
pert
I would not use verbatim for explaining the parameter a,b,c here, because
usually it is only used for code examples.
[J] Explain the arguments. What is swapped with estimator, a or b or c?
What about e1 and e2, a and c? Why can one not choose a, b and c
freely?
jackknife
What is corrected?
bayesian_bootstrap
Put keyword arguments seed and concentration in description in backticks.
The link is behind a paywall. Is there no paper that describes how
Bayesian bootstrapping works?
Utility Functions
logx
[K] I do not understand why returning 0 instead of -Inf (like the
standard Julia function) is a good idea. You use this function in your
code, so perhaps you can explain why this is fine in the context of
entropy estimation?
bins
Use verbatim for x.multiplicities
from_csv
*CSV.File() → CSV.File()
You can pass all further keyword arguments on to CSV.File() by changing the
function signature to
from_csv(file::String, field, ::Type{T}; remove_zeros=false, header=nothing, kw...)

and swapping for row in CSV.File(file; header=header) in the source code
with for row in CSV.File(file; header=header, kw...)
Then, users will have more options before falling back to using CSV.jl
directly.
Divergence and Distance
cross_entropy
[L] In the function signature, should it really be {T<:MaximumLikelihood}?
Looking at the source code, this should be a typo.
kl_divergence
"Both distributions needs to be defined over the same space, so length(p) == length(q)."
Use the same statement as in the description of cross_entropy: "P and Q must be the same length."
"If the distributions are not over the same space (their cross entropy is < 0), then itthe function returns Inf."
(cross entropy < 0 according to source code, I find this easier to understand)
jensen_shannon_divergence
[N] The description says "P and Q, as represented by their histograms."
Why does the function take AbstractVectors and not CountVectors?
jensen_shannon_distance
[O] Arguments are called P and Q like for cross_entropy and
kl_divergence, where they are of type CountData, but here they
are AbstractVectors. Should they be CountVectors, like for
jensen_shannon_divergence?
uncertainty_coefficient
[P] What is joint in the equations in the description? I(X;Y)?
Mutual Information and Conditional Information
mutual_information
Description has some typo, does not render correctly:
"using Optim: estimatemaxstep mutualinformation(joint::Matrix{I}, estimator::Type{T}) where {T<:AbstractEstimator, I<:Real}"
(looks like a surplus or misplaced using Optim: estimate_maxstep in the docstring)
[Q] What is XY and where would it come from? I think this one
would benefit from an example.
conditional_entropy
Explain the keyword arguments.
"and estimator estimatorestimator" (verbatim)

Hi @nluetts, this is an extraordinary effort on your part. Thank you very much indeed! I have pushed (to paper) and hope that I've covered every point that you've raised directly in the docs. I'm going to close this now issue now, but please reopen if you think I've missed something or not done something very well.


[Dropdown Menus] Dropdown Menus links won't click

I have noticed an issue when using Dropdown Menus in Foundation 6.
When hovering/clicking/tapping on the Dropdown Menu menu link it opens fine, but then the links contained within this dropdown menu will not work. You can click but that appears to close the menu rather than follow the link.
Again, this behaviour is also apparent on the Foundation 6 Docs Page.
http://foundation.zurb.com/sites/docs/dropdown-menu.html
The links on the Foundation Page above are admittedly onto to "#", but that should move to the top of the screen - which it does not.

duplicated with #7129 and #7143 , looks like @zurbchris is working on a whole new branch to fix dropdown menu issues b9f581c3c09e1c82e3f7dfa41b45c7993d7820a7


PyPI not updated with most recent release

The Python Package Index is still showing v0.0.1a0 when the current released version is v0.0.2.
https://pypi.org/project/spcqe/

workflow failed here, I didn't catch it at the time (3 months ago): https://github.com/cvxgrp/spcqe/actions/runs/10014751354

Repository was missing the PyPI credentials. I added them as environment secrets to the "deployment" environment. This should fix this. If it fails at next deployment we can reassess, but closing this for now.


Support Rails 5.1

This is a port of @matthewrudy's pull request #134, with a few extra commits to clean up the Travis configuration to see if we can get all the JRuby builds 💚!

Oh boy, I'm probably making things worse, but I think that .travis.yml needed to be brought up to date with the latest available Ruby versions (and to kill the EOL Rubies). What I'm not totally sure about is losing specificity on the version of JRuby we use.

@albus522 estimation on when this can be released?

How can I use this gem on rails 5.1.1?

@lionel02218 I just use my forks
gem 'delayed_job', github: 'gogovan/delayed_job', branch: 'rails-5.1'
gem 'delayed_job_active_record', github: 'gogovan/delayed_job_active_record', branch: 'rails-5.1'

but you can just switch that to use @laserlemon's fork if that suits you.

@matthewrudy @laserlemon. Error with new migrations:
Trace:
==<PHONE_NUMBER>5206 CreateConversions: migrating ================================
-- adapter_name()
   -> 0.0000s
-- adapter_name()
   -> 0.0000s
-- create_table(:conversions, {:id=>:integer})
   -> 0.0013s
==<PHONE_NUMBER>5206 CreateConversions: migrated (0.0014s) =======================

rails aborted!
StandardError: An error has occurred, this and all later migrations canceled:

Directly inheriting from ActiveRecord::Migration is not supported. Please specify the Rails release the migration was written for:

  class CreateDelayedJobs < ActiveRecord::Migration[4.2]

When will an update for this be available, am not able to upgrade to rails 5.1 because of it

We're going to give it our best shot to have this wrapped and released tomorrow. Thank you for your patience!

Version 4.1.2 released! 🎉  🎊 🍾

@laserlemon Thank you for all your hard work!  I've been watching all the commits go through CI and it was really cool to see you break down the problems and get to green! 💚


HttpData source and HttpProxy destination

Bug Report
Describe the Bug
We want to pass data from consumer to provider the case is consumer create an asset with policy and contract details and during asset creation the type of data address is HTTPDATA. Now provider want to send some data to consumer and doing the contarct negotiation process and initiate the transfer process with data address type HTTPPROXY.
During the transfer process initiate we face the below error.
SEVERE 2022-09-09T11:51:48.068826591 Fatal error processing transfer request: 73fb10b0-bf0a-4976-8204-4a49082a535b. Error details: Failed to delegate data transfer to Data Plane: Cannot find a transfer Service that can handle HttpData source and HttpProxy destination

Expected Behavior
Data must be pass to Consumer to provider.

@dilipdhankecha2530 Could you please provide a complete bug report (referring to the template) covering the used version, used extensions, executed API calls etc.?

@juliapampus
So we want to pass data from the consumer side to the provider side but when we use the HttpData at that time transfer has been initiated from the provider side to the consumer side (First hit the rest endpoint from the asset data address and once we got the response hit the destination data address API).
But now the use case is the consumer wants to pass the data to the provider so here I am confused about how to do this part with HttpData?
or
Do we need to use another data address type for this type of senarios?

Hello,
the problem description is very confusing.
First, the consumer does not pass data to the provider. It is the other way around.
Also this is the provider who creates the Asset, not the consumer.
If you want to use the Data Plane as a http proxy to query the data, which is typically what you want to do when the provider exposes its data through a REST API, then the consumer will have to set the data type as HttpProxy in the DataRequest.
This type of data exchange is covered in this integration test: https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/blob/cebe2cf3295796255d81239247009ca7706ec1c2/system-tests/e2e-transfer-test/runner/src/test/java/org/eclipse/dataspaceconnector/test/e2e/AbstractEndToEndTransfer.java#L48

@bscholtes1A
Yes, we are using HttpProxy for consumer DataRequest and use HttpData for the provider DataRequest while creating the asset.
Now How the flow would be in EDC that not able to understand could you please explain or give the idea behind this?

I am going to convert this to a discussion as it is not an issue.

Now How the flow would be in EDC that not able to understand could you please explain or give the idea behind this?

Please ask more specific questions so that we can try and answer them. General "how-do-things-work" questions can usually be answered by looking at the documentation, samples, or tests. In your case, set a breakpoint on where the error is happening and perform some investigation so you can provide us with more detail and make it easier for us to help you.


feat: updating implementation of 104 zalando guideline (#610)

Closes #610.

@maxim-tschumak could you please take a look on integration test?
--- FAIL: TestIntegrationWithZallyApiDefinition (0.67s)
    --- FAIL: TestIntegrationWithZallyApiDefinition/integrationWithZallyApiDefinition (0.67s)
	Error Trace:	integration_test.go:140
	Error:      	"SHOULD
	            	======

SHOULD

Set Flow to 'application' When Using OAuth 2.0
	            		OAuth2 security definitions should use application flow
	            		https://zalando.github.io/restful-api-guidelines/#104

SHOULD

Use Specific HTTP Status Codes
	            		Operatons should use specific HTTP status codes
	            		https://zalando.github.io/restful-api-guidelines/#150
	            			/api-violations POST default
	            			/review-statistics GET default
	            			/supported-rules GET default

	            	Summary:
	            	========

	            	MUST violations: 0
	            	SHOULD violations: 2
	            	MAY violations: 0
	            	HINT violations: 0
	            	" does not contain "SHOULD violations: 1"

Codecov Report

Merging #611 into master will not change coverage.
The diff coverage is 62.5%.

@@            Coverage Diff            @@
##             master     #611   +/-   ##
=========================================
  Coverage     84.15%   84.15%
  Complexity      883      883
=========================================
  Files           145      145
  Lines          2279     2279
  Branches        307      307
=========================================
  Hits           1918     1918
  Misses          223      223
  Partials        138      138

Impacted Files
Coverage Δ
Complexity Δ

...zalando/zally/rule/zalando/SecureWithOAuth2Rule.kt
85.41% <62.5%> (ø)
42 <1> (ø)
:arrow_down:

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 1d00566...fc1e435. Read the comment docs.

:+1:

:+1:

👍

👍


unable to create the react app

If you have a general question about Create React App or about building an app with Create React App we encourage you to post on our Spectrum community instead of this issue tracker. The maintainers and other community members can provide help and answer your questions there: https://spectrum.chat/create-react-app
If you're looking for general information on using React, the React docs have a list of resources: https://reactjs.org/community/support.html

i'm unable to create the app the whole process is getting stopped after "run npm audit fix "like shown in the image above.Please kindly provide me a solution for this and let me know how to fix this issue......
If you've discovered a bug or would like to propose a change please use one of the other issue templates.
Thanks!

Where are you running this at? How are you running it? Does the user running command actually have permissions in the folder?


[mvn] Update maven wrapper

Coverage remained the same at 100.0% when pulling ee89dac8db25beaa944c5fb0b326537de6e92c25 on hazendaz:master into ca60107443de45bed9bf16fb7de91d154dfcfd7d on mybatis:master.


Modify repoName to PLUGIN for docker plugin create

- What I did

Modify repoName to PLUGIN for docker plugin create
Fix the unconsistency between code and document for docker plugin subcommands.

- How I did it
- How to verify it
- Description for the changelog

Modify repoName to PLUGIN for docker plugin create which keeps consistency among plugin subcommands.
Add  "(rootfs + config.json)"  to the document of docker plugin create
Modify NAME to PLUGIN in docker plugin push.

Signed-off-by: yuexiao-wang<EMAIL_ADDRESS>

Please clarify the reason for this PR?
e.g. consistency
(If consistency is your reason, probably you may want to modify push.go as well: https://github.com/yuexiao-wang/docker/blob/1e2f1f816e0e83b258de5e156011daa72fff0cc5/cli/command/plugin/push.go#L17)
PTAL @tiborvass @anusha-ragunathan

@AkihiroSuda  PTAL
Thanks

@AkihiroSuda PTAL
Done!

LGTM


Performance issues using powershell versions

@majormoses requested I open this issue
We ran into performance issues using the powershell scripts for metrics and checks. As we started really growing in number of checks, and number of nodes.  Get-WMIObject isn't the fastest thing on the block.  I've rewritten the checks and metrics using CIM and the performance greatly increased. around 75% faster execution time, and lower CPU load during execution.
I've combined all of the metrics into a single script, because a single instantiation of powershell worked out better, and we gather all of this data from all windows nodes.
https://github.com/csabo/sensu_win_checks_cim

@csabo Thanks for your hard work! I know next to nothing about powershell so I am not much help in actual porting but I am more than happy to review the PRs, run tests, (I have a windows desktop I use for gaming and do not have access to a windows server), help with changelog, etc.
Regarding the metrics scripts I think we would want to keep them separate as different people have different needs. If I knew what I was talking about in powershell I would say we could implement the logic in whatever they call a library and create individual scripts that just included it. If you wish to avoid the penalty cost of running each one as its own instantiation maybe we can still have a meta check to run all the checks but it should pull from the shared logic. I know in ruby and most interpreted languages that is easy to do without a performance penalty but I am not sure if that is possible here as I have not used windows for anything but playing games in many years.

I would be interested in helping to test or work it into a library.  I can't promise "timely" necessarily - but I've run into the same performance issues and I was going to tackle a rewrite, as well.

@csabo that sounds good to me @pdpvandyk awesome no worries, this is open source and something like 99.99% of all plugin development is done by people (such as myself) who do not work for sensu and donate their time, expertise, etc  to better the sensu and larger monitoring community.

@pdpvandyk @majormoses I've gotten started on a branch that will include the CIM replacements of WMI where applicable.

@pdpvandyk @majormoses Alright the branch is up, Ive checked each with powershell 5 and 6, they seem good to me, a couple of the metrics I felt were a little complex in their code, so I replaced them with what I did for us in house, that I feel is a little more straight forward. The only ones I didnt get to yet were the "network" "uptime" and "system" metrics, I'll try to get them done tomorrow or this weekend. In the mean time plenty here to test/validate/discuss!
https://github.com/csabo/sensu-plugins-windows/tree/cim


Indexing fails when triggered from a background job

I have a background job in my Umbraco solution that is supposed to publish content pages in bulk with the expectation that FullTextSearch re-index them.

But what happens is this (from the Umbraco logs)

The indexing of the page in question fails with SocketException when the HttpPageRenderer in FullTextSearch tries to make a request to a controller for the page HTML.
What could be the reason for that ?

Is it only when triggered by your background task? The error message suggests, that it can't make the HTTP request for rendering the page.


Update/Add OpenSSL

Tool name
OpenSSL
Tool license
Apache-2.0 license
Add or update?

[X] Add
[ ] Update

Desired version
1.1.1 for windows x86
Approximate size
No response
Brief description of tool
I am testing 32bit compatibility on Windows, I use find_package(OpenSSL) in cmake to find the openssl library, but I find the x64 architecture version, which causes problems with the link, can I install the x86 architecture version of openssl into the image?
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[ ] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
Tried to use choco, after successful installation, cmake still can't find the path

Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

is it possible to install the x86 architecture version of openssl into the default image?

Hello! We generally have no plans to provide x86 software on the runners and try to remove some leftovers of it (like 32bit python on setup-python GitHub action).


feat: improve status display when cluster is fenced

This patch makes it stand out in the output of cnpg status plugin command when the cluster especially when the primary instance of the cluster is fenced.
Closes #841
Signed-off-by: Hai He<EMAIL_ADDRESS>

/ready-to-test

/ready-to-test

/ok-to-merge

/ready-to-test


Inform users when pcscd is required

Closes str4d/age-plugin-yubikey#83.

Codecov Report

Merging #86 (d763801) into main (aaa445c) will decrease coverage by 0.03%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main     #86      +/-   ##
========================================
- Coverage   1.34%   1.31%   -0.04%
========================================
  Files          6       6
  Lines        372     381       +9
========================================
  Hits           5       5
- Misses       367     376       +9

Impacted Files
Coverage Δ

src/error.rs
0.00% <0.00%> (ø)

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


Regression: Type using conditional and mapped types is no longer assignable  in 4.3

Bug Report
🔎 Search Terms
assignability regression
🕗 Version & Regression Information

This changed between versions 4.2  and 4.3-beta

⏯ Playground Link
Playground link with relevant code
💻 Code
export type UnrollOnHover<O extends object> = O extends object ?
    { [K in keyof O]: O[K]; } :
    never;

export type Schema = Record<string, unknown>;
class Table<S extends Schema>  {
    __schema!: S;

    // Removing this line, removes the error
    getRows<C extends keyof S>(): Array<UnrollOnHover<Pick<S, C>>> {
        return null!
    }
}

class ColumnSelectViewImp<S extends Schema> extends Table<S> { }

const ColumnSelectView1: new <S extends Schema>() => Table<UnrollOnHover<S>> = ColumnSelectViewImp;
const ColumnSelectView2: new <S extends Schema>() => Table<UnrollOnHover<S>> = Table;

🙁 Actual behavior
Assignment to ColumnSelectView1 failed
🙂 Expected behavior
Assignment to ColumnSelectView1 succeeds as it did in 4.2
Workaround
Wrap S in UnrollOnHover as well Playground link

Yep, like I suspected, this was caused by a change in inference.  Namely, we used to erase local type parameters to any, while now we map them into their constraints. So in earlier versions of TS, we'd replace C with any, resolve the Pick into an object with some index signatures, and match up the index signature type S[any] against UnrollOnHover<S>[string] (getting S = UnrollOnHover<S>, same as the other inference site). Now, UnrollOnHover remains unexecuted, because C is only replaced with keyof S., so instead, we're matching {[k: string]: S[any]} to UnrollOnHover<Pick<UnrollOnHover<S>, keyof UnrollOnHover<S>>>, which, when we look at the mapped type in UnrollOnHover, looks like {[K in keyof Pick<UnrollOnHover<S>, keyof UnrollOnHover<S>>]: Pick<UnrollOnHover<S>, keyof UnrollOnHover<S>>[K]} and wouldn't ya know, that template matches up to the index signature type and we match S to Pick<UnrollOnHover<S>, keyof UnrollOnHover<S>>.
The new inference is definitely valid (certainly, it's an expression of the same type, even if we don't always recognize it as such) - it's an inference we always could have made (albeit with more contortions), it's just really unfortunate, since it's at the same priority as the other inference site (and last-in-wins). I have put up a PR (#44126) that lowers the priority of the secondary inference site in structures like this one, so the mapped-over structure is a lower priority than the plain type parameter, whcih gets back the old inference and thus assignability behavior.

@DanielRosenwasser @RyanCavanaugh  This does not appear fixed in 4.3.2 (Playground Link) but works in nightly (Playground Link). Will there be a 4.3.3 where this will be included ?


Create Reference annotation and remove @ff.ref

Describe the issue
Right now we are using the @ff.ref javadoc tag. This however should be an annotation with the classname as value.
When changing the classname the reference should update automatically.

The following files have unused imports that where used for referenced classes;

HttpSenderBase.java
BatchTransformerPipeBase.java
JdbcIteratingPipeBase.java
JmsTransactionalStorage.java
JsonXsltPipe.java
FtpFileSystemSender.java
SendGridSender.java


Minor edits to the sparsity RFC

A few edits to make the proposal more consistent with the updated MLIR doc

Thanks! Can you merge this for me (we don't seem to have write access)?


Timeline feature enhancement

As a user:
I want to: use the timeline feature to change the color, shape and size of nodes.
So that I can: demonstrate the changing state of individuals (susceptible, infected, recovered) over time.
Acceptance Criteria
Given a node list with a column for each state transition (susceptible to infected, infected to recovered, infected to dead) and field values being the date when the node made that transition...
When that node list is used in the timeline function:
1 - the stylistic attributes of the nodes can be mapped to these columns
2 - changes can occur when the timeline slider reaches the "threshold date"
3 - if there is no date entry for that state transition column then the node never changes state (color, size, shape, etc.)

Potential to add ability to modify links over time later.


[PostgreSQL] New UI

We want to develop a new UI for the existing Database connectors in the Import API. The idea is to generate some screens so the user can connect to the desired database, run an SQL query and import the resulting dataset into the CARTO Dashboard. We are going to start developing the UI for the Postgres connector, but it should be easy to make it extensible to MySQL and to SQL Server connectors.

[x] First screen: Connection. This is the screen where we gather the connection params (https://github.com/CartoDB/cartodb/issues/15396)
[x] Second and third screens: SQL Query and Selected dataset. This is where we import the desired dataset into the Dashboard from an SQL query. (https://github.com/CartoDB/cartodb/issues/15415)
[x] Error message in First screen (https://github.com/CartoDB/cartodb/issues/15437)

Related info

Design: https://invis.io/QUULMAVESAW#/390965275

Acceptance
Connectors enablement

Login with dbconnectorsui user
[x] Check that all db connectors are disabled (PostgreSQL, MySQL, SQL Server)
Edit account and enable carto-connectors in the Feature flags section
[x] Login again and check that all db connectors are enabled

Connectors' tests
Connection parameters are in this document https://docs.google.com/document/d/1qnkvzJPHeBTvDYbe27S1v4AnCsem9RfDcd3mpvxep3E/edit?usp=sharing

Click on the PostgreSQL connector button, it will take you to connection params page.
[x] Check that all copies and logos refer to PostgreSQL.
[x] Try to connect with wrong params and check than an error message appears.
[x] Connect with correct params and check that it takes you to SQL query screen.
[x] Type a query without from keyword and run query. An error should appear.
[x] Type a correct query and import a table. Sync options cannot be tested in dedicated instances.
Repeat with other connectors
Repeat from Add new layer option from a map in Builder

Note: Back button goes back to Connectors list page

@MireyaD do you mind checking with @alejandraarri if the PostgreSQL design is ready to go and make the changes that are needed? Some issues where I can see we need some more work:

Assets
It's the design feasible? I notice that in the BigQuery one we use a single column while the MySQL here uses a double-column, would that be a problem?
I would remove the links to the help articles until we tackle them
Are the copies final? Did you check them with @vdiego?

Thanks!
/cc @urbanphes

Sure! @alejandraarri and I are having a meeting today to check if something else is needed :)

Hey! After yesterday meeting we have some open questions:

Should we include in the first screen a Contact us link? Can we manage the amount of messages? Or we would have the same situation as in the request connector case?
IP are correct?
@vdiego, could you check the copies?

Let me know what you think. Thanks!

@MireyaD I left comments in invision :)

Deployed into production! 🚀
In the end, working for all DB connectors: PostgreSQL, MySQL and SQL server


Don't display the vouch button if your account is expired

But still display the "remove vouch" one.

Done


Fixes & tests for non-raw kernels

For #1215

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).

Codecov Report

Merging #4431 (0760790) into main (91ebb08) will increase coverage by 0%.
The diff coverage is 90%.

@@          Coverage Diff          @@
##            main   #4431   +/-   ##
=====================================
  Coverage     75%     75%
=====================================
  Files        396     396
  Lines      25861   25867    +6
  Branches    3710    3711    +1
=====================================
+ Hits       19521   19563   +42
+ Misses      4813    4794   -19
+ Partials    1527    1510   -17

Impacted Files
Coverage Δ

src/client/telemetry/index.ts
80% <ø> (ø)

src/client/datascience/jupyter/kernels/kernel.ts
69% <85%> (+<1%)
:arrow_up:

...ascience/raw-kernel/rawNotebookSupportedService.ts
91% <100%> (+<1%)
:arrow_up:

.../client/datascience/notebook/kernelWithMetadata.ts
87% <0%> (-2%)
:arrow_down:

...client/datascience/kernel-launcher/kernelFinder.ts
85% <0%> (-2%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
75% <0%> (-1%)
:arrow_down:

...lient/datascience/jupyter/kernels/cellExecution.ts
76% <0%> (ø)

...ient/datascience/editor-integration/codewatcher.ts
69% <0%> (+<1%)
:arrow_up:

...ent/datascience/jupyter/kernels/kernelExecution.ts
81% <0%> (+<1%)
:arrow_up:

...datascience/jupyter/liveshare/hostJupyterServer.ts
85% <0%> (+<1%)
:arrow_up:

... and 11 more

Codecov Report

Merging #4431 (0760790) into main (91ebb08) will increase coverage by 0%.
The diff coverage is 90%.

@@          Coverage Diff          @@
##            main   #4431   +/-   ##
=====================================
  Coverage     75%     75%
=====================================
  Files        396     396
  Lines      25861   25867    +6
  Branches    3710    3711    +1
=====================================
+ Hits       19521   19563   +42
+ Misses      4813    4794   -19
+ Partials    1527    1510   -17

Impacted Files
Coverage Δ

src/client/telemetry/index.ts
80% <ø> (ø)

src/client/datascience/jupyter/kernels/kernel.ts
69% <85%> (+<1%)
:arrow_up:

...ascience/raw-kernel/rawNotebookSupportedService.ts
91% <100%> (+<1%)
:arrow_up:

.../client/datascience/notebook/kernelWithMetadata.ts
87% <0%> (-2%)
:arrow_down:

...client/datascience/kernel-launcher/kernelFinder.ts
85% <0%> (-2%)
:arrow_down:

src/client/datascience/baseJupyterSession.ts
75% <0%> (-1%)
:arrow_down:

...lient/datascience/jupyter/kernels/cellExecution.ts
76% <0%> (ø)

...ient/datascience/editor-integration/codewatcher.ts
69% <0%> (+<1%)
:arrow_up:

...ent/datascience/jupyter/kernels/kernelExecution.ts
81% <0%> (+<1%)
:arrow_up:

...datascience/jupyter/liveshare/hostJupyterServer.ts
85% <0%> (+<1%)
:arrow_up:

... and 11 more


 If so, could you produce a conda .yml file and post it here?
Background: we're working on including an environment file for each chapter so that these things can be better encapsulated, particularly where different chapters are using conflicting versions of the same python packages.

We actually ran all of our code in Google Colab, so we didn’t need to manage the python environment it ran in. I’d be happy to create an environment file for the chapter’s packages though!

That would be great if you could do that! I've been doing it for some other chapters and it's really tricky when I only have a tenuous grasp of the details.


Add info and faqs on how to add custom fields to dbt generated docs

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
Users have asked if there is a way to expand dbt generated docs with custom fields. there is! but it's don't clearly documented.
Answer
There’s a meta config that exists on models and columns that can be defined in yaml. Believe they’ll all be surfaced through the docs (certainly model level stuff is, but I’m less positive on column level)
What part(s) of the page would you like to see updated?
Proposing adding this helpful piece:

Docs FAQs
build and view your docs page as faq_src
link faq in meta config page as faq_src
dbt docs reference page as faq_src

Additional information
No response

@mirnawong1 I'm going to close this because of efforts needing to be put into dbt Explorer instead. Please let me know or reopen if you disagree!


The cmdlet not worked

I've tried to execute this cmdlet, but only created an empty backend pool resource without any backend ip configurations.

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

ID: 45300feb-159a-6671-75d1-4bd72c291aa1
Version Independent ID: d5adde6a-0571-8095-8086-88b08abea686
Content: New-NetworkControllerLoadBalancerBackendAddressPool (networkcontroller)
Content Source: docset/windows/networkcontroller/New-NetworkControllerLoadBalancerBackendAddressPool.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

@pengzhisun. Sír could you tell me command you are typed. if possible give screenshot of error

@pengzhisun. Sír could you tell me command you are typed. if possible give screenshot of error

Hi @RAJU2529 , there is no error after this command, I've tried to add an IPConfig  object for one VM NIC to a new load balancer backend pool, but after the command execution, only an empty backend pool created without the IPConfig to be added.
The sample scripts to re-pro:

# Get IP Config for one VM NIC with IP Address "<IP_ADDRESS>" in Virtual Network "test-vnet"
$VNETName = "test-vnet"
$NICIpAddress = "<IP_ADDRESS>"
$VNET = Get-NetworkControllerVirtualNetwork -ConnectionUri $Uri -Credential $Creds -PassInnerException | Where-Object { $_.ResourceMetadata.ResourceName -eq $VNETName }
$NIC = Get-NetworkControllerNetworkInterface -ConnectionUri $Uri -Credential $Creds -PassInnerException | Where-Object { $_.Properties.IpConfigurations.Properties.Subnet.ResourceRef -eq $VNET.Properties.Subnets[0].ResourceRef -and $_.Properties.IpConfigurations.Properties.PrivateIpAddress -eq $NICIpAddress }
$IpConfig = $NIC.Properties.IpConfigurations[0]

# New SLB Backend Pool for an existed SLB in the same vnet
$BEPool = New-Object Microsoft.Windows.NetworkController.LoadBalancerBackendAddressPoolProperties
$BEPool.backEndIpConfiguration = $IpConfig

New-NetworkControllerLoadBalancerBackEndAddressPool -ConnectionUri $Uri -LoadBalancerId "{some-slb-id}" -ResourceId "test-pool" -Properties $BEPool

@pengzhisun
The issues section of this repository is intended for documentation problems.
Please consider checking any relevant resources and asking the community about your problem via the following link.
PowerShell Community
Also feel free to reply if there is anything else that could be done to improve the document itself.
Thanks.