Auto install not working

Hi,
Thanks for sharing this plugin.
I have tried to install it following your instructions but it seems that cordova auto install is not recognizing your plugin.
Even though no error message is thrown by the cli, the plugin is not added to my project.
Could it be due to the config.xls ?

Thanks @unihockey, I have updated the readme;
Installing the plugin has changed from ""cordova plugin install../cordova-plugin-discovery" to
"cordova plugin add ../cordova-plugin-discovery"

Awesome, I should have been more careful. I'll try it out this evening. Have a good day.


A truly no .rda object solution as originally intended

Thank you for your Pull Request! We have developed this task checklist from the Development Process Guide to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your code as well as making sure the admiral codebase remains robust and consistent.
Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

[ ] Place Closes #<insert_issue_number> into the beginning of your Pull Request Title (Use Edit button in top-right if you need to update)
[ ] Code is formatted according to the tidyverse style guide. Run styler::style_file() to style R and Rmd files
[ ] Updated relevant unit tests or have written new unit tests, which should consider realistic data scenarios and edge cases, e.g. empty datasets, errors, boundary cases etc. - See Unit Test Guide
[ ] If you removed/replaced any function and/or function parameters, did you fully follow the deprecation guidance?
[ ] Update to all relevant roxygen headers and examples, including keywords and families. Refer to the categorization of functions to tag appropriate keyword/family.
[ ] Run devtools::document() so all .Rd files in the man folder and the NAMESPACE file in the project root are updated appropriately
[ ] Address any updates needed for vignettes and/or templates
[ ] Update NEWS.md under the header # admiralpeds (development version) if the changes pertain to a user-facing function (i.e. it has an @export tag) or documentation aimed at users (rather than developers)
[ ] Build admiralpeds site pkgdown::build_site() and check that all affected examples are displayed correctly and that all new functions occur on the Reference page.
[ ] Address or fix all lintr warnings and errors - lintr::lint_package()
[ ] Run R CMD check locally and address all errors and warnings - devtools::check()
[ ] Link the issue in the Development Section on the right hand side.
[ ] Address all merge conflicts and resolve appropriately
[ ] Pat yourself on the back for a job well done! Much love to your accomplishment!

@Pierre-Wallet something to think about before our meeting, I think this was @rossfarrugia 's actual original intention, where the data is being created on the spot in the function itself, I just didn't like the idea of having the thousands of lines in the /R/xxx.R function file itself, but I didn't realize you can source data-raw files in the function body
This solution gets rid of all .rda objects in its entirety, we won't need an internal dataset, we don't need to export any, the "exists in code" is the tribble scripts that will be housed in the data-raw folder


ScrollViewer with ImageScrollBar

Help needed as in https://forum.babylonjs.com/t/customize-scrollbar/6956/18?u=johnk
Thank you for saying you will help.

So what problems do you face so far?

Gijs came up with a solution for image scroll bars in https://www.babylonjs-playground.com/#13CF95#66
It has something I liked, the thinness of the images within the bars, plus a couple of issues

It calls Image.drawImage directly
It only works for a vertical bar.

When attempting to modify the ScrollViewer code I had the following problems

getting a thin image
working with one image in both a vertical and horizontal orientation.

Here is a working example of what I have so far  Scroller
You will notice that the thumb fills the height of the background unlike the Gijs example. Also the background image is more stretched across the height then the Gijs example.
What I have done it so far
i. Created a new ImageScrollBar class
ii. Added an extra parameter to ScrollViewer which when true allows images for the scroll bars see Line 97
iii. Forced the thumb to be in proportion to the image used and then extend the width using the thumbSize variable lines 378 / 379 in ScrollViewer
iv. Set the measures for the bar background and bar (thumb) in lines 92 - 117 in the ImageScrollBar which then calls Image._draw
v. The Image._draw call goes to Image.ts at line 615 where the drawImage parameters are set and called depending on a switch state line 648. The ScrollViewer for both ScrollBar and ImageScrollBar go through Line 653 STRETCH_FILL and the private function _drawImage is called Line 598
I experimented with creating different switch calls named IMAGE_VERTICAL, IMAGE_HORIZONTAL but couldn’t work out the parameter values to use to get the same effect as Gijs, so deleted this attempt.
I tried using transform and rotate on the canvas image but got completely lost and could not work out the values to use.
Now Gijs has come up with some further code though this again calls drawImage directly rather than the private _draw but perhaps this is sufficient. His use of rotate only partially solves the vertical horizontal bar issue as it does not take into account whether width > height or vice versa.
I will keep experimenting and would be very grateful for any advise you could give.

Am close to a solution of sizing the images for the thumb and the bar. No where near for rotating on vertical or horizontal.

Here are some thoughts:

I'm not sure we should have a background image. Or it should be optional.
Your scroller example looks really good to me.
The thumbImage width should be set to bar width when vertical and height should be set to bar height when horizontal
_getThumbThickness should return bar.width * image aspect ratio when vertical (and same thing for height when vertical)

Am making progress.
So far I have that the background image is optional, along the horizontal or vertical it fills the space available it height across the space can be varied with barImageHeight, range 0 to 1.
There are also two new properties, thumbLength and thumbHeight (range 0 to 1) that controls the controls the size of the thumb.
A little more checking and I should be able to PR this stage and give examples.
What I really would like to do is if your image for the thumb (or background) is say 100 x 200 then it fits best in the vertical bar space and I would like to rotate the image to put it into the horizontal bar space and vice versa for an image 200 x 100. This I am have major problems with.

Ok for that you can call something like that before rendering the image:
https://github.com/BabylonJS/Babylon.js/blob/master/gui/src/2D/controls/control.ts#L1283
It does a central rotation (do not forget to call context.save() before and context.restore() after drawing the image)

Have this new problem on npm run build following the last pull
[17:34:04] Finished 'typescript-libraries-importLint' after 8.02 s
[17:34:04] Finished 'importLint' after 8.03 s
[17:34:04] Starting 'circularDependencies'...
[17:34:04] Starting 'core-circularDependencies'...
[17:35:12] 'core-circularDependencies' errored after 1.12 min
[17:35:12] TypeError: Cannot read property 'summary' of undefined
at dependencies (C:\Users\User\Documents\GitHub\Babylon.js\Tools\Gulp\tasks\gulpTasks-dependencies.js:40:29)
at C:\Users\User\Documents\GitHub\Babylon.js\Tools\Gulp\tasks\gulpTasks-dependencies.js:64:9
at taskWrapper (C:\Users\User\Documents\GitHub\Babylon.js\Tools\Gulp\node_modules\undertaker\lib\set-task.js:13:15)
at bound (domain.js:395:14)
at runBound (domain.js:408:12)
at asyncRunner (C:\Users\User\Documents\GitHub\Babylon.js\Tools\Gulp\node_modules\async-done\index.js:55:18)
at process._tickCallback (internal/process/next_tick.js:61:11)
[17:35:12] 'circularDependencies' errored after 1.12 min
[17:35:12] 'default' errored after 1.58 min
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: gulp --max-old-space-size=8192 --tsLintFix
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\User\AppData\Roaming\npm-cache_logs\2019-11-19T17_35_12_665Z-debug.log
Packing up for now will check back later.

Run npm install in tools/gulp folder

If you are happy with what I have done so far then I am happy to have this merged. I will work on the rotation issue and do a new PR  later, when I have it working.
Working example (now showing properties lines 90 to 95) https://tender-murdock-275855.netlify.com/view/test1.html


[WIP] Add docs for mutable scheduling directives on gated Pods

Placeholder PR for docs on mutable pod/node affinity/selector (implemented in https://github.com/kubernetes/kubernetes/issues/116160)

/retitle [WIP] Add docs for mutable scheduling directives on gated Pods
/sig scheduling

@alculquicondor This is ready for a technical review. I referred to the KEP for the language describing the rules for modifying node selector, node affinity etc.

@alculquicondor Thanks for the review, I've addressed your comments and this is ready for another look

@alculquicondor I included your most recent suggestions

Since this PR is associated with the 1.27 release, this should target the dev-1.27 branch. The author should edit the PR and change the target to dev-1.27
/milestone 1.27

Thanks @reylejano I updated the base to dev-1.27

/label tide/merge-method-squash
/lgtm

/approve


[CAMEL-14066] Split route parsing from the main CamelContext api

Can we not make it possible to use ModelCamelContext for this?, so you can do
context.adapt(ModelCamelContext.class).addRoutesDefinitions

Not sure that would really help.  However, we could easily go for
ModelHelper.addRoutesDefinitions(context, inputStream);

instead of the current solution
context.adapt(ModelCamelContext.class).addRoutesDefinitions(inputStream);

or the one in this PR actually:
RoutesDefinition routes = ModelHelper.loadRoutesDefinition(context, inputStream);
context.adapt(ModelCamelContext.class).addRoutesDefinitions(routes.getRoutes());

Hmm then I guess the current PR is the best IMHO.


Error after generation

Hello.
I have this issue.
When I try to generate an image or a pdf, in the output of the generated file I can read this string
{"code":30000000,"message":"worker run process failed.job request timed out","data":null}
What type of error is this? What can I do to solve it?
Thanks.

Hello.
I have this issue. When I try to generate an image or a pdf, in the output of the generated file I can read this string
{"code":30000000,"message":"worker run process failed.job request timed out","data":null}
What type of error is this? What can I do to solve it?
Thanks.

Maybe worker poll full.


python3Packages.shiv: init at 1.0.0

Motivation for this change

new package: a command line utility for building fully self-contained Python zipapps as outlined in PEP 441, but with all their dependencies included

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.shiv

@GrahamcOfBorg build python3Packages.shiv

@GrahamcOfBorg build python3Packages.shiv


Ability to specify absolute windows path in devon.properties

For variables such as DEVON_SOFTWARE_PATH you want to be able to specify an absolute path on your machine. On Linux and MacOS there is a simple and clear concept for a path. However, Windows is more like Mr. Jeckyl & Doctor Hyde and is therefore more "flexible".
# CMD/Windows native
DEVON_SOFTWARE_PATH=C:\software
# MINGW
DEVON_SOFTWARE_PATH=/c/software
# Cygwin
DEVON_SOFTWARE_PATH=/cygdrive/c/software

IMHO the only reasoable syntax for windows is the first one as this is what windows users expect. However, the devon.properties file tries to be platform agnostic and also shall run on windows in the following environments:

CMD
PowerShell
MINGW (git bash)
Cygwin

This means that parsing the devon.properties would require some magic to transform the path such that it fits to the environment. This is especially tricky as we do not know if a variable is a path or not. We could therefore do the following:

Check if the variable value starts with a letter followed by a colon and a slash/backslash (e.g. "C:/" or "C:" or "D:/"). The do the according transformation.
Define a variable ROOT or DEVON_ROOT that is pointing to the absolute path of the root disc where devon is currently launched. This can be determined and normalized for the current environment easily. Then users could use ${ROOT}/software instead what would even be more portable in case a project wants to have an absolute path in the devon.properties shared via settings GIT repo (as otherwise linux and mac users would run into issues, but could also overwrite the variable locally).

As writing stuff like "C:\software" is still the most intuitive way how windows users would specify an absolute path we should go for this and considere UX even though the transformation stuff will be complex and may introduce new bugs temporary.

Since NTFS is POSIX compatible the forward slash syntax for filenames works almost everywhere. Prefix this with the mount point of the filesystem in front - and your are done.
"/software" prefixed with  "C:" or "/c" or "/cygdrive/c" or even "\?\C:"
O.k. - a path name C:/software needs quoting in Windows CMD in order to work. But since proper quoting in scripts is anyway a must in any OS - who cares.
A user will need to handle these things not that often -. no need to be intuitive IMHO. Just put a good comment into devon.properties about the expected path names.

@markusschuh thanks for your hint. Works for cd:
Cygiwn
hohwille@laptop /cygdrive/c/projects/cool-project/
$ cd c:/projects
hohwille@laptop /cygdrive/c/projects

MINGW/MSYS
hohwille@laptop MINGW64 /c/projects/test
$ cd c:/projects

hohwille@laptop MINGW64 /c/projects

BUT:
hohwille@laptop MINGW64 /c/projects/test
$ mv C:\Projects\test\update\scripts C:\Projects\test
mv: cannot stat 'C:Projectstestupdatescripts': No such file or directory

So it is not as easy.

However, I discovered that:

cygdrive can convert a windows path to unix and vice versa for the current bash shell on windows
cygdrive is also available for MINGW/MSYS

That means I only need to do a cygdrive transformation for variables starting with letter plus dot inside the bash world.

fixed with PR #241 and #242


30-days free trail Azure account link is broken

https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-arduino-iot-devkit-az3166-translator
The link of "free 30-day trial Microsoft Azure account" is broken.
It should be https://azure.microsoft.com/en-us/free/

v-mavick submitted PR "Corrected broken link to Free Azure subscription page. #36343" to correct this link in the following three documents:
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-arduino-iot-devkit-az3166-devkit-remote-monitoring
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-arduino-iot-devkit-az3166-translator
https://docs.microsoft.com/en-us/azure/iot-hub/iot-hub-arduino-iot-devkit-az3166-retrieve-twitter-message

Reopen it until the PR have been merged to msdocs.


Some extra sample (or test case) for Deedle C# application

Hi,
Is it possible to include extra sample in the Deedle C# document or test cases for Row based processing? In area like economics often time series are combined into newer time series (set based combination). When I read the Deedle C# API signatures and document page there is few mention on such application. however after some poking around its surprisingly easy to do such calculation. I feel it is even easier to do such process in C# than F# (sorry to say that). The following is an example I hope i could have found on the documentation page:
Things to pay attention to are:

The rows function didn't remove the row while both columns' value are missing
The mean function is smart enough to calculate mean based on available data point.

Thanks for opening source such a great library
using System.Text;
using System.Threading.Tasks;
using Deedle;

namespace TestDeedleRowProcessWithMissingValues
{
    class Program
    {
        static void Main(string[] args)
        {
            var s1 = new SeriesBuilder<DateTime, double>(){
                 {DateTime.Today.Date.AddDays(-5),10.0},
                 {DateTime.Today.Date.AddDays(-4),9.0},
                 {DateTime.Today.Date.AddDays(-3),8.0},
                 {DateTime.Today.Date.AddDays(-2),double.NaN},
                 {DateTime.Today.Date.AddDays(-1),6.0},
                 {DateTime.Today.Date.AddDays(-0),5.0}
             }.Series;

            var s2 = new SeriesBuilder<DateTime, double>(){
                 {DateTime.Today.Date.AddDays(-5),10.0},
                 {DateTime.Today.Date.AddDays(-4),double.NaN},
                 {DateTime.Today.Date.AddDays(-3),8.0},
                 {DateTime.Today.Date.AddDays(-2),double.NaN},
                 {DateTime.Today.Date.AddDays(-1),6.0}
             }.Series;

            var f = Frame.FromColumns(new KeyValuePair<string, Series<DateTime, double>>[] {
                KeyValue.Create("s1",s1),
                KeyValue.Create("s2",s2)
            });

            s1.Print();
            f.Print();

            f.Rows.Select(kvp => kvp.Value).Print();

//            29/05/2015 12:00:00 AM -> series [ s1 => 10; s2 => 10]
//            30/05/2015 12:00:00 AM -> series [ s1 => 9; s2 => <missing>]
//            31/05/2015 12:00:00 AM -> series [ s1 => 8; s2 => 8]
//            1/06/2015 12:00:00 AM  -> series [ s1 => <missing>; s2 => <missing>]
//            2/06/2015 12:00:00 AM  -> series [ s1 => 6; s2 => 6]
//            3/06/2015 12:00:00 AM  -> series [ s1 => 5; s2 => <missing>]

            f.Rows.Select(kvp => kvp.Value.As<double>().Mean()).Print();

//            29/05/2015 12:00:00 AM -> 10
//            30/05/2015 12:00:00 AM -> 9
//            31/05/2015 12:00:00 AM -> 8
//            1/06/2015 12:00:00 AM  -> <missing>
//            2/06/2015 12:00:00 AM  -> 6
//            3/06/2015 12:00:00 AM  -> 5

            //Console.ReadLine();
        }
    }
}

Closed and merge my commets to #308

Exception.ToString()
I don't know why I copied a lot of examples, but they all failed to run with Unhandled Exception.

@liugaocn can you provide more information? What exactly raises Unhandled Exception?


Fix link to release notes

Why was this closed? A reason should be posted.

Not sure about that. I didn't intentionally close the issue.

Ah, I deleted my fork.


⚠ Fortnite has degraded performance

In 86d814d, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1602 ms

Resolved: Fortnite performance has improved in 60d3ec9.


Fix "make dist" after recent changes to parser

In particular, it is now possible to either build ModSecurity with pre-generated parser, or use --enable-parser-generation configure option to rebuild parser from sources.

Merged! thanks ;)
Just to notice: the Travis failure has nothing to do with the commit, it was an internal error.


IndexError: list index out of range (piratebay search)

flexget -V
2.0.21

2016-05-15 00:00 CRITICAL task          get-movies      BUG: Unhandled error in plugin discover: list index out of range
Traceback (most recent call last):
  File "c:\python27\lib\site-packages\flexget\task.py", line 442, in __run_plugin
    return method(*args, **kwargs)
  File "c:\python27\lib\site-packages\flexget\event.py", line 23, in __call__
    return self.func(*args, **kwargs)
  File "c:\python27\lib\site-packages\flexget\plugins\input\discover.py", line 272, in on_task_input
    return self.execute_searches(config, entries, task)
  File "c:\python27\lib\site-packages\flexget\plugins\input\discover.py", line 143, in execute_searches
    search_results = search.search(task=task, entry=entry, config=plugin_config)
  File "c:\python27\lib\site-packages\flexget\plugin.py", line 120, in wrapped_func
    return func(*args, **kwargs)
  File "c:\python27\lib\site-packages\flexget\plugins\urlrewrite\piratebay.py", line 137, in search
    entry['title'] = link.contents[0]
IndexError: list index out of range

Config? Full log? Kinda need to see what it failed on. The crash log says nothing.

The bug is just that link.contents does not have an index 0.. which kills the task. I would assume this stacktrace was enough to locate and put a try/catch or length test and safely error the search, instead of letting it interrupt the entire task.

The relevant/trimmed part of my config (though I don't think its actually helpful):
    discover:
      what:
        - movie_list: movies
      from:
        - piratebay:
            category: 207
            sort_by: date

I should add, it doesn't seem to happen all the time. It happened once. so far. I was simply reporting the existence of a crasher. Was probably a transient error.

Well, I'm not just gonna throw a blanket try-except in the code without knowing the exact situation it's required. I need to know which movie caused it.
If I can't reproduce the error, I can't fix it.

i wasn't suggesting a blanket try/catch. i was merely suggesting it be written defensively such that if the server (or a proxy) responded with malicious/bad/unknown data this plugin would handle logging an error and letting the task move on instead of crashing.
I don't know what it was looking for at the time. As i said above, i believe this to be rare and has only occurred the one time so far. this isn't blocking me. i just submitted this as a courtesy for general improvement. if it happens again, ill try to gather more info.

You don't have the logs preceding the crash log?


Specifying Asset Handles without an Instance of the AssetCollection from File / Const time

I've got a set of const custom materials that I need to swap out with the standard materials that are loaded from GLTF's like so:
These materials are basically a copy of the StandardMaterial with a Thermal glow blending emissivity layered on top. I need a way to indicate that I need a specific field from within the AssetCollection.
The two things I can come up with for this are either some macro wizardry to secretly do const handles internally, or using something the key attribute of the asset macro to generate some form of index.
Additionally, you could probably get somewhere with Reflect / get_field, but I have a very strong preference to not litter duplicate &str's around. That specifically would be less of an issue if we could reference Const &str's in macro attributes.
My use case for the key solution would be the ability to reference the handle from a file or  a const Material at compile time.
{
    "base_color_texture": "GameAssets::colored_paint_albedo",
   <other JSON fields>
}

Here is an example of where I'm forced to do lookups on the asset_server directly and create a bunch of adidtional consts:
pub fn hydrate_material(
    desc: ThermalMaterialDescriptor,
    server: &Res<AssetServer>,
) -> ThermalPbrMaterial {
    //Using asset_server.load is alright as if the texture is already loaded it will just return the handle at the cost of a lookup
    let base_color_texture = desc.base_color_path.map(|path| server.load(path));

    let thermal_texture = desc.channel_pack_path.map(|path| server.load(path));

    let emissive_texture = desc.emissive_path.map(|path| server.load(path));

    let normal_map_texture = desc.normal_map_path.map(|path| server.load(path));

    let metallic_roughness_texture = desc.metallic_roughness_path.map(|path| server.load(path));

    let occlusion_texture = desc.occlusion_path.map(|path| server.load(path));

    ThermalPbrMaterial {
        base_color_texture,
        emissive_texture,
        normal_map_texture,
        metallic_roughness_texture,
        occlusion_texture,
        thermal_accent_boost_texture: thermal_texture,
        base_color: desc.uniform.base_color.into(),
        thermal_color: desc.uniform.thermal_color.into(),
        emissive: desc.uniform.emissive.into(),
        reflectance: desc.uniform.reflectance,
        perceptual_roughness: desc.uniform.roughness,
        metallic: desc.uniform.metallic,
        min_radiant_temp: desc.uniform.min_radiant_temp,
        max_radiant_temp: desc.uniform.max_radiant_temp,
        temp: desc.uniform.temp,
        thermal_boost_scalar: desc.uniform.thermal_boost_scalar,
        thermal_pulse_frequency: desc.uniform.thermal_pulse_frequency,
        thermal_pulse_intensity: desc.uniform.thermal_pulse_intensity,

        flip_normal_map_y: desc.flip_normal_map_y,
        double_sided: desc.double_sided,
        cull_mode: desc.cull_mode,
        unlit: desc.unlit,
        fog_enabled: desc.fog_enabled,
        alpha_mode: desc.alpha_mode,
        depth_bias: desc.depth_bias,
    }
}

Here is an example of where I'm forced to do lookups on the asset_server directly and create a bunch of adidtional consts:
What additional consts do you mean? There is no const in the code example.

Have you looked into custom dynamic assets for this issue? Maybe that could help reducing the direct usage of the asset server.
Just FYI: Bevy v0.12 (in~3 months) will likely support properly typed const asset handles. That might make it easier.

I will close this for now, but feel free to open the issue again with some more explanation.


Drag Selection appears to be broken

2D network
   +   mouse drag should produce shadow box to select nodes

issue with D3 behavior update: https://github.com/d3/d3-drag/issues/62
have to bypass default brush filter

applying explicit filter exposed other problems with brush when using drag selection so reverted back to previous version of 'brush' and 'end' callback functions


document how to use delete_similar

Documentation link
https://pan-os-python.readthedocs.io/en/latest/howto.html#optimize-with-bulk-operations
Describe the problem
doc mentions delete_similar as an alternative to delete to perform bulk delete, but it's not clear how this feature should be used.
if calling fw.remove(obj) on an AddressObject, next call to obj.delete_similar() fails with an exception
No PanDevice set for object tree

Suggested fix
can the documentation include an example of usage of delete_similar ?

Hi, @freedge were you able to find some details about it ?
I'm looking for a way to bulk-delete huge amounts of objects (which actually takes a very long time with unit API calls), but I don't really understand if delete_similar is the right way to proceed, neither how to use it.


x/net/{ipv4,ipv6}: some tool fails with batch.go

x/net/ipv4 and ipv6 batch.go has synatx errors
What version of Go are you using (go version)?
1.8
What did you do?
src/golang.org/x/net/ipv4/batch.go:59:14: expected type, found '='
src/golang.org/x/net/ipv4/batch.go:78:2: expected declaration, found 'if'
src/golang.org/x/net/ipv6/batch.go:56:14: expected type, found '='
src/golang.org/x/net/ipv6/batch.go:69:2: expected declaration, found 'if'

As the build constraint line in the source files say it requires the new language feature called type alias introduced in Go 1.9. You didn't mention what's the tool, I guess that the tool might be kinda syntactic tool and does not support build constraints which are lines start with +build:. FYI, released Go toolchains from the Go team support the build constraints: see https://build.golang.org.
So this is not an issue for x/net/{ipv4,ipv6} packages. Perhaps fixing the tool you are using might be the best. If it's gonna happen you need to tweak the stuff that invokes the tool to avoid from the undesirable behavior of the tool.

Perhaps worth noting (just had a coworker run into this): gofmt in 1.8.3 has this same behavior (does not support build constraints) and we see the errors manifest when it attempts to format vendored dependencies.

@mastercactapus,
Then it's better to file a new issue for cmd/gofmt.


Add ability to exclude

Having the ability to exclude certain files or folder would be lovely. Not usable for me until then :(

Exclude from the crawling or exclude from the indexing?
Crawling in the sense that none of it's links will be crawled either, while indexing means that it will be excluded but it's children will be crawled (unless specified otherwise).

A fair question. I suppose my use case is crawling. Optimally I guess you'd want both to be offered.

Sorry for the delay, am very busy.
Will send out an update in a couple minutes

The is a new array option named $blacklist


Flush ./dist on compile

Running npm run compile multiple times results in error:

⨯ unlinkat [project root]/dist/linux-unpacked/locales: directory not empty
github.com/develar/go-fs-util.EnsureEmptyDir
/Volumes/data/go/pkg/mod/github.com/develar/go-fs-util@v0.0.0-20190620175131-69a2d4542206/fs.go:98
github.com/develar/app-builder/pkg/electron.UnpackElectron.func1.1
/Volumes/data/Documents/app-builder/pkg/electron/electronUnpack.go:38
github.com/develar/app-builder/pkg/util.MapAsyncConcurrency.func2
/Volumes/data/Documents/app-builder/pkg/util/async.go:68
runtime.goexit
/usr/local/Cellar/go/1.16/libexec/src/runtime/asm_amd64.s:1371
  ⨯ [project root]/node_modules/app-builder-bin/linux/x64/app-builder exited with code ERR_ELECTRON_BUILDER_CANNOT_EXECUTE  failedTask=build stackTrace=Error: [project root]/node_modules/app-builder-bin/linux/x64/app-builder exited with code ERR_ELECTRON_BUILDER_CANNOT_EXECUTE
at ChildProcess.<anonymous> ([project root]/node_modules/builder-util/src/util.ts:249:14)
at Object.onceWrapper (events.js:422:26)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1048:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:288:5)

There's other package options like del-cli that boast some nicer features so you might want to take this choice with a grain of salt

I have not encountered such a problem on Windows. As far as I know, electron-builder cleans the directory itself. I assume that the reason for this problem is something else. For example, you try to compile an application when it is running.

I think you're right, I'm on Linux and encountered some difficulty deleting a dir that had an app running in it after filing this, closing the Electron app solved it


SKU not configurable

What I'm trying to achieve
I'm trying to add stock to my product
Steps to reproduce the problem

I added the title, description, etc and clicked on pusblish (dashboard)
When trying to add the product to the cart, the option isn't available
When checking the product, the Inventory section must be 0 due to the SKU number, which isn't available to change

What I expected to happen

Just being able to add products to the cart and have them in my inventory

Screenshots
System information
Operating system: Win 10 pro
Browser: Chrome

Don't worry, we're on it: #390

Fix merged


CTC-1535 - add JS code samples for asset renditions

Motivation
CTC-1535
Checklist

[ ] Code follows coding conventions held in this repo
[ ] Tests are passing
[ ] Documentation has been updated (if applicable)
[ ] Temporary settings (e.g. project ID used during development and testing) have been reverted to defaults

How to test
No testing needed

Looks good. I appreciate the corrections in non-JS code samples for consistency! Merging.


stable/prometheus: Update default version to 2.2.1

What this PR does / why we need it:
This PR bumps the default Prometheus version from 2.1.0 to 2.2.1. There are important fixes in the 2.2.x series that make operating a higher-traffic Prometheus setup more viable.
Special notes for your reviewer:
See the release notes for 2.2.0 and 2.2.1.

/ok-to-test

/assign

/lgtm


Metadata-based photo clustering

This is a more complex/far-off feature idea, but a very useful one. Currently, the app only supports tagging one taxon or observation at a time. These features would make it suitable for processing many at once.

[ ] After loading images, "cluster" them by taxon/observation (if tagged)
[ ] Group related images together in a (collapsible/expandable) box

Something like the iNaturalist web UI for observation uploads

[ ] Display a group's (condensed) taxon/observation info in a layout to the right
[ ] Clicking a taxon/observation from that list will go to its search/info page
[ ] Display remaining ungrouped/untagged images in a separate layout to the left
[ ] Drag ungrouped images into an existing group

Moved to #227


Candle Auction Milestone-3: Workshop to substrate.dev

Project Abstract
This is a proposal to continue the project with Milestone-3: adding a Workshop to substrate.dev

Hi David. The reason I haven't added a license to Milestone-3 is that the substrate-docs repo doesn't specify the license itself. So I'm not sure if Apache 2.0 would be ok here, not to contradict with the target site (substrate.dev) license.
Please, kindly advise on it. If Apache 2.0 wouldn't be an obstacle there, I'll add just it. Thx.

I think this applies here, i.e. it assumes the substrate-docs license when you submit a PR.

Please find added license terms.


Added changelog for fixing KubeletConfig issue

Added change log for PR: https://github.com/ComplianceAsCode/compliance-operator/pull/58

/lgtm


[Bug]: After executing SQL add statements within the result set, it may not be displayed within the result set

ODC version
odc423
OB version

What happened?

What did you expect to happen?

How can we reproduce it (as minimally and precisely as possible)?
Add data
Anything else we need to know?
No response
Cloud
No response

The front end will poll the queryData interface until the latest result set is obtained.


Unblock ads

Hello. I need a configuration where Google ads are not blocked. How can I edit it?

Fork this repo for yourself and remove related lines from script.

Can you advise which lines should be removed in the forked version?

for the repo, inform me to change the code on your fork

https://github.com/USTOIR/reality-ezpz-adsallowed

https://github.com/USTOIR/reality-ezpz-adsallowed/pull/1


ES6 Constant Re-assignment

This is valid according to JSHint with "esnext" on:
const x = 0;
x++;

It should instead warn "Attempting to override 'x' which is a constant.", like the following does:
const x = 0;
x += 1;

Agreed!


Trying to install package editable but no documentation available

I'm trying to install a package in editable mode which I gather is done by passing the -e flag to install but the site documentation is so sparse (both here and on the official docs site) that honestly I don't know whether that is the correct option or what it is supposed to do.

You might find some examples in https://docs.pipenv.org/en/latest/basics/#a-note-about-vcs-dependencies.

Oh, super. Weird how I didn't find this…
https://docs.pipenv.org/en/latest/basics/#editable-dependencies-e-g-e


example: update niscope measurement to run when trigger source isn't present in measure pins

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?

The trigger_source configuration is also reserved along with pin_names.
Renamed the input configuration name from pin_names to measurement_pins and trigger_source to trigger_pin for better clarity.

AB#2683394
Why should this Pull Request be merged?
The existing NI-SCOPE example holds a software constraint that the trigger pin must always be present within the pin names (measure pins) because only the pins in the pin_names are reserved and not the trigger_source. However, there could be use cases where the trigger pin need not be within the measure pins.
LabVIEW PR containing a similar update: https://github.com/ni/measurementlink-labview/pull/506
What testing has been done?
Manually verified the example execution.
Before:

Now:

@dixonjoel @bkeryan The example is working fine with the latest argo installer. Hence, I'm submitting this PR.


Bump React-Redux to 7.1.3

Description
This version of Redux gives us some hooks, like the useSelector we can use in place of connect. The typings changed quite a bit.

Fixed a conflict in yarn lock (as redux version had been deduped in lockfile)


[3.x] Add tab Metadata to Tabs & TabContainer

Implements https://github.com/godotengine/godot-proposals/issues/6664
This PR adds two new methods to TabContainer & Tabs for getting and setting metadata on individual tabs.

For Tabs, I simply mirrored the way metadata is handled in ItemList & TreeItem, and added a Variant metadata; variable to the Tab struct.
For TabContainer, metadata is saved on the tab control itself as meta, which is in line with how methods such as set_tab_title() and set_tab_icon() work.

Thanks!


[!] Unable to find a specification for React depended upon by RNDeviceInfo

Summary

Version
0.22.6

React Native
0.56

macOS Version
High Sierra

Current behavior
I followed the instructions, ran react-native link, then went to do a pod install inside the ios folder and i'm getting the following error
[!] Unable to find a specification for `React` depended upon by `RNDeviceInfo`

Expected behavior
No error?!

I get a different, but related problem:
https://github.com/rebeccahughes/react-native-device-info/issues/515

Try this, hope it helps!
https://stackoverflow.com/questions/52737251/how-do-i-prevent-react-native-device-info-causing-react-ambiguity-on-my-setup/52776146#52776146


Allow navigation from Add API Backend wizard header

The Add API Backend wizard shows the steps in the header area of the template. Currently, the steps do not allow users to navigate backwards when clicking. Adding backward navigation capabilities to the steps in the header may improve usability.

+1

This should be possible using the {{ pathForStep stepId }} helper.

Issue using pathForStep helper, https://github.com/forwarder/meteor-wizard/issues/72
Did not work as expected.

Got useful link how to use Wizard with Iron Router, https://github.com/forwarder/meteor-wizard#using-routers-


bugs on drop_action

I can't use drag and drop in the playlist.
Maybe it is because layerY is deprecated.

use e.originalEvent.layerY

In dragging nowplaying-music, icon gets bigger


Normalized Severity operator typo

Describe the bug:
Normalized Severity should be one word in the CSE signal generation syntax.  The screen captures under the Scheduling the Search steps are wrong; they show normalized severity as two words.  It is correct (one word) under Required fields.
Doc URL -
https://help.sumologic.com/docs/alerts/scheduled-searches/generate-cse-signals/#scheduling-the-search
Steps to reproduce the behavior:

Go to Generate CSE Signals With a Scheduled Search (url above).
Scroll down to Scheduling the Search
Look for: Normalized Severity in step 1 & 2 screen caps

Screenshots
If applicable, add screenshots to help explain your problem.

Replaced the image with the typo, in https://github.com/SumoLogic/sumologic-documentation/pull/1566


Imrpove grapheme.split testing

Quoted from Luke:

coverage of the unicode data file test points is pretty average, like it might only have a test that covers an emoji at the start of a string, but not the middle or end or before a CLRF or after a Hangul sequence... etc.
So I'm reasonably confident there are a couple of edge cases we haven't caught, and could end up crashing someone's code. It would be nice to get that to a point where we are reasonably confident that is not going to happen.

This is a fun project for anyone who has the time and interest. Brendan's https://github.com/bhansconnect/roc-fuzz platform would be a great way to test this.
I have had plans to do this, but haven't had an opportunity yet.


Set up generate of API type docs using Spectacle

We should start generating an API docs site so users can easily learn what types are supported by cert-manager.
Spectacle generates static HTML, and looks great so I'm going to try getting it set up. For now, the documentation will live inside this repo. In future we should push it to a website somewhere.

This went pretty badly after trying to generate docs with the openapi-gen tool. Going to close this now in favour of plain old markdown docs :smile:


Type check with pytype

Description

Add type checking with pytype to the Travis CI builds; update Dockerfile accordingly.
Add py.typed file to declare PEP 561 compliance.
Fix a few minor bugs caught by pytype: mostly wrong variable names.
Suppress some false-positives or adapt the code so that it can be type checked.

Note what I have not done is added any type annotations (except where it was needed to make code type check). I don't have the bandwidth to do this right now, although it'd be great if we added annotations to some of the core classes.

Motivation and Context

Closes #561.

[X] I have raised an issue to propose this change (required for new features and bug fixes)

Types of changes

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

Checklist:

[X] I've read the CONTRIBUTION guide (required)
[X] I have updated the changelog accordingly (required).
[x] My change requires a change to the documentation.
[ ] I have updated the tests accordingly (required for a bug fix or a new feature).
[ ] I have updated the documentation accordingly.

@araffin can you rebuild the Docker image to include pytype? (Or give me access to DockerHub so I can push a new image.)

@AdamGleave sorry for that. I'll try to do that (not sure if i can with the free version)
in the mean time, you can change the travis config to point to your image

@AdamGleave what is your docker username?

@AdamGleave what is your docker username?

I'm https://hub.docker.com/u/adamgleave. Thanks!

Travis is now passing. I'm now done with this PR (except any review comments).
One thing we shouldn't forgot is to change .travis.yml to point to araffin/stable-baselines-cpu since I changed it to point to my account temporarily.
Codacy has some false positives for unused imports that are used in type hints in comments. I think we'll just have to ignore these: it'd probably understand if I used variable type annotations, but that's only supported in 3.6+.

but that's only supported in 3.6+.

Maybe a good time to drop 3.5 support? This would allow to use f-strings and faster dict implementation that keeps the insertion order. I have to check when is the official end of support of 3.5...

I think we'll just have to ignore these

yes, otherwise, it is not a big deal to ignore those (you have now the rights to do that normally ;) )

Regarding Python 3.5: Many older linux distros come with Python 3.5 by default and it is difficult-ish to upgrade the pre-packaged one to newer one, so that may be bit of a headache for some (E.g. They need the older distribution/packages to run some specific software). Then again, they could just install minoconda and go with that, so I would vote for moving to Python versions +3.6.

Maybe a good time to drop 3.5 support? This would allow to use f-strings and faster dict implementation that keeps the insertion order. I have to check when is the official end of support of 3.5...
End of life for 3.5 is 2020-09-13, although I'd be OK dropping support earlier.

@araffin thanks for the review, I've addressed the comments.

While we're making some changes to the CI, wondering if we should disable Travis branch builds, and just build PRs? Right now our build queue in Travis is often quite long.
We'd need to change scripts/run_tests_travis.sh since right now the Codacy upload only happens on branch builds. But this is just becuase CODACY_PROJECT_TOKEN may not be available on forks. We could instead directly check for the presence of that environment variable.

I agree on both ;)

Almost there, still some edit needed in the doc: https://stable-baselines.readthedocs.io/en/master/guide/install.html#build-the-docker-images

Sorry for missing, updated.

maybe a line about running type check?

I've added this in CONTRIBUTING.md, couldn't see a better place for it.

EDIT: I ignored the issues in codacy related to typing

Thanks :)

EDIT: I'm afraid only @hill-a has the right to change that (for the build)

Travis seems to have let me disable branch builds so I've done that for now (and updated scripts/run_tests_travis.sh accordingly).

We seem to have both a travis-ci.com and travis-ci.org instance! The travis-ci.com is the one producing "Travis CI - Pull Request " and travis-ci.org is the one with "continuous-integration/travis-ci/pr" and "continuous-integration/travis-ci/push".
travis-ci.org is deprecated so it's probably best if we just get rid of that one. I've disabled builds on it but can't seem to delete it. @hill-a or someone else with admin access to this repo on GitHub should follow this to remove the status checks beginning with "continuous-integration/"

Unfortunately, i think only @hill-a  has the rights to do that. Will ping him again if he does not see this message ;)


Scp 847 - playground endpoint for ACTUS

I duplicate PR from here (because github stopped tracking my branch): https://github.com/input-output-hk/plutus/pull/2163
That one has a large discussion with lots of comments.

This PR introduces:

ACTUS as a library
server-side endpoint for generating ACTUS contract from contract terms

The purpose is to have experimental Blockly editor of ContractTerms on a Playground that generates Marlowe contracts.
Notes:

"Model" package in this PR doesn't follow usual Haskell conventions in order to closely reflect original ACTUS specification:
https://www.actusfrf.org/techspecs, direct link. It is actively updated in their repo, so technically I can't refactor it without loosing the track of the original spec on github
Other modules (Definitions.*, Ops, Generator, MarloweCompat) should follow standard Haskell conventions. I'll fix it if their not.
I heavily rely on type-classes from "Ops" module because in research branch we use them to "switch" between pure Haskell code and Marlowe code implementation of arithmetics and date operations
I had to create a separate "actus-marlowe" project because marlowe project doesn't work with HIE and "actus-marlowe" is a lot of code to maintain (this PR is just a tip of the iceberg). They also likely to have different versioning with Marlowe semantics

TODO:

[x] fix warnings
[x] fix NIX build

Hi @dk14, please read CONTRIBUTING and make sure you do what it says, this broke things for cabal users: https://github.com/input-output-hk/plutus/pull/2174


Prd readme merge

Changed the tables to images instead of markdown tables because the .md editor deleted all of them.

The Readme looks good !


chore(templates): update default curlversion

fixing 4 critical CVE's
update curl v8.1.1

Please ensure your pull request adheres to the following guidelines:

[ ] explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
[ ] Document your Changes in the CHANGELOG.md file of the respected chart as well as the Chart.yaml

curl v7.76.1 has the following critical CVE's

https://avd.aquasec.com/nvd/cve-2021-36159
https://avd.aquasec.com/nvd/cve-2022-37434
https://avd.aquasec.com/nvd/cve-2021-3711
https://avd.aquasec.com/nvd/cve-2021-3711

Dear @githubcdr,
thanks for your contribution! I noticed we have a few inconsistencies in the helm chart. Sometimes we pinpoint the version, sometimes not. To fulfill your goal could you please update every occurrence of the curlimages/curl image to use 8.1.2? It would be nice to update all occurrences both in sonarqube and sonarqube-dce charts (please make sure to update the README file as well).
Please let me know if you can help further on that. Otherwise, I will define a task and we will tackle it in one of the next hardening sprints.

We'll work on this PR in the context of SONAR-19950


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 1486 ms

Resolved: Bofrost.fr performance has improved in b2f2e67.


[lint-config] Enable import/no-nodejs-modules rule

Related to #11350. This lint rule helps prevent taking dependencies on node libraries, and given our browser-first focus, we should enable it in our default config. I made it a warning to make it easier to adopt.

Looks good to me! Longer term, I kind of wonder if we want our lint config package to be more of a factory, rather than a home for static configs. We could then have packages call some function to generate the appropriate config for them. They could request strict vs minimal (etc.), whether or not they are intended to be used in a browser or in a node environment, etc., and get the appropriate linter config.
This was a direction I wanted to take things in Whiteboard, but never got a chance. Interested in your thoughts on this.
But regardless, for now this seems like a good call. Tooling packages can turn this off if it becomes obnoxious.

Longer term, I kind of wonder if we want our lint config package to be more of a factory, rather than a home for static configs. We could then have packages call some function to generate the appropriate config for them. They could request strict vs minimal (etc.), whether or not they are intended to be used in a browser or in a node environment, etc., and get the appropriate linter config.

I love the direction! The question to me always comes down to cost/benefit. If there are really only 2-3 practical configs in play (which I believe is the case for our repo), then may not be worth it. But I wouldn't say no if we started moving that way. 😄

Longer term, I kind of wonder if we want our lint config package to be more of a factory, rather than a home for static configs. We could then have packages call some function to generate the appropriate config for them. They could request strict vs minimal (etc.), whether or not they are intended to be used in a browser or in a node environment, etc., and get the appropriate linter config.

I love the direction! The question to me always comes down to cost/benefit. If there are really only 2-3 practical configs in play (which I believe is the case for our repo), then may not be worth it. But I wouldn't say no if we started moving that way. 😄

Yeah, exactly.

Now that I think of it, it might also be worth preremptively disabling this new rule in some of the tooling packages, so we don't forget.


Single sequencer crashes while creating NewExchange

Since go-header 0.2.5, the behavior seems to be changed in terms of NewExchange where the peers is mandatory at the constructor:
if len(peers) == 0 {
		return nil, fmt.Errorf("no trusted peers")
}

So, for single sequencer, do we need to adopt this new behavior, where we do not create exchange and syncer and just keep the p2p server up?
The behavior before was that, we would start exchange and syncer for every node (sequencer, fullnode). Is this no longer the case?

Fixed


Add Getting Started section to README

README Update
Please, add a getting started section to the README so folks know how to start with this project

Added Getting Started section to README file


Add Car Assistant Natural Language example

A set of two examples focused on leveraging typechat to improve the performance of in vehicle AI assistants.

@microsoft-github-policy-service agree company="Microsoft"


A new Test issue with no mentions

This issue does not mention anyone

A new comment with no mentions


Update the 'Monitor and notify Microsoft 365 health status' sample

This sample in it's current state does not work, due to changes in the command surface of the CLI for Microsoft 365.
It needs to be rewritten slightly, using the new commands and incorporating the new data structure that the Microsoft 365 APIs now return.

Any comments @arjunumenon?

That is a good find @martinlingstuyl. Did not realize that the return data structure has got changed. Nothing I can think of to be added. Just that need to execute the script and need to see where it is getting failed and may need to update script accordingly.
If none takes it till end of October (Hactoberfest season), I will take it up and quickly get it pushed since I had written the initial script sample.

Can I work on it?

All yours @SmitaNachan!


Name folder containing tests 'tests' or 'test', not just 't'

Yes, short paths are nice, but readable folder names that clearly communicate what's inside are nicer.

"t" is a historical porpoises standard. We should pick either "test" or "tests" as the standard.

I am using "tests", but "test" is very common in the Java/Maven world, "spec" is used a lot with Ruby - there are probably others. Whatever you pick for this project itself is somewhat up to your personal taste I guess. For measuring other modules I'd say either is acceptable, because all those folder names are common and understood.

"t" is also common in the perl world. But... I think we'll go with "test", as it seems to be the most commonplace.

Yeah, but perl is write-only code ;)


Add optional machine name parameter to ProcessWatcher

Pull request for #111.
I'm not completely sure the tests look nice. Perhaps setting a test MachineName?

Looks good, as for the tests I'll need a time to rewrite them using new version of the library, therefore do not worry that much about it :).
I'll review the PR once I get back home in a few hours. Thanks!


Create rule: The ids of the StudyProtocolDocumentVersions associated with a StudyProtocolDocument are all distinct.

Create rule defined for #113

Changed DDF99986 to DDF00026. Modified the description and output message to refer to "id values" instead of "ids" to make it clear that it's only the value of the id attribute that are being checked.
Ready for review.

Note that the imported dataset is named "STUDYPROTOCOLDOCUMENTVERSIO" (without an "N" at the end).  I have raised issue #126 about this.

The rule seems okay, but test data seems to be missing.

I'm getting the message "unit-test-coreid-DDF00025-negative Not implemented" in a box with the title "File Upload" I have to leave now, but perhaps we can sort this out Thursday morning.

Matt helped me synch the test data, which I think will resolve my issues.
I ran the test data for DDF00026 and got the expected results.

Rule deleted in DDF Phase 4. Saving a copy of the definition:
Authorities:
  - Organization: 'CDISC'
    Standards:
      - Name: USDM
        References:
          - Citations:
              - Cited Guidance: 'USDM 2.6'
                Document: 'USDM v2.6'
            Origin: USDM Conformance Rules
            Rule Identifier:
              Id: 'DDF00026'
              Version: '1'
            Validator Rule Message: 'No two versions of a study protocol document have the same id.'
            Version: '1.0'
        Version: '3.0'
Check:
  all:
    - name: id
      operator: is_not_unique_set
      value:
        - parent_id
Core:
  Id: CORE-000418
  Status: Draft
  Version: '1'
Description: 'The StudyProtocolVersions associated with a StudyProtocolDocument
  must all have different id values'
Executability: Fully Executable
Outcome:
  Message: 'The id values of the StudyProtocolVersions associated with a
    StudyProtocolDocument include duplicates.'
  Output Variables:
    - parent_entity
    - parent_id
    - id
    - protocolVersion
    - protocolStatus.decode
Rule Type: Record Data
Scope:
  Entities:
    Include:
      - StudyProtocolDocumentVersion
Sensitivity: Record


improving sanity check pipeline

Codecov Report

Merging #407 (e7aa287) into main (9a73bf7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #407   +/-   ##
=======================================
  Coverage   61.60%   61.60%
=======================================
  Files          19       19
  Lines        3487     3487
=======================================
  Hits         2148     2148
  Misses       1234     1234
  Partials      105      105

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


[Do not merge] Update to govuk frontend 7.0.0

This is an attempt to do a cleaner merge of the changes from the https://github.com/alphagov/govuk-prototype-kit-private-beta.
Includes:

Removal of Elements, Template, Toolkit
Migration of all code
Updates to guidance relating to Elements, Template, Toolkit
New cookie banner component

If you want to review all changes without  whitespace: https://github.com/alphagov/govuk_prototype_kit/pull/512/files?w=1
The other parts that are missing are being tracked with this milestone: https://github.com/alphagov/govuk_prototype_kit/milestone/1
https://trello.com/c/SrxrezjZ/1061-add-govuk-frontend-to-the-govuk-prototype-kit

@joelanman

changed all examples that users will interact with to avoid macros
squashed the commits you mentioned
fixed an issue where there was a stray deleted file in a commit it didnt belong.

Sorry I meant small changes plus reverting macros, not that that was small, I wasn't very clear.

Wondering if the Cookie banner work would be better as a separate PR? Is it part of 'Update to govuk frontend'?

@joelanman yeah, I think it makes sense since otherwise this PR would break cookie support.


Curve448 primitives

This adds all the primitives required for Curve448 support (X448 and Ed448).
This is based heavily on original work by Mike Hamburg (thanks @bitwiseshiftleft) and contributed under CLA.
This is code that originally appeared in #4829, but I've pulled out just the primitives (i.e. none of the EVP changes). The EVP layer will come in a different PR once this is in. I've also stripped out all commits that were updating the temporary Makefile I had in #4829 , so that doesn't appear here.
At the moment only the 32-bit reference code is used. There are a number of files for other architectures that I have imported but done very little else with at this stage. Again later PRs can look at integrating the platform specific code in.

Checklist

[ ] documentation is added or updated
[x] tests are added or updated

New commits pushed addressing all comments.

New commit pushed addressing comments. I also tweaked the constant_time_lookup() function to add an explicit cast of the mask to avoid an AppVeyor failure.

Sorry for so many commments...
I am gonna take a break now :)

Perhaps it is time to just remove all of the define tricks and write straightforward C code.  The only reason I can think to keep it is if we want to cherry-pick improvements that Mike makes to his code.  That seems unlikely.

32-byte alignment is required for avx2 operations.  On x86 at least, the library may be using avx2 even if the caller isn’t compiled with -mavx2.  We could instead change it so all the vector ops assume unaligned.
Sent from my phone.  Please excuse brevity and typos.

On Feb 2, 2018, at 08:27, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/field.h:

+#ifndef GF_H
+# define GF_H

+# include "internal/constant_time_locl.h"
+# include <string.h>
+# include <assert.h>
+# include "word.h"
+
+# define NLIMBS (64/sizeof(word_t))
+# define X_SER_BYTES 56
+# define SER_BYTES 56
+
+# if defined(GNUC) || defined(clang)
+#  define INLINE_UNUSED inline attribute((unused,always_inline))
+#  define RESTRICT restrict
+#  define ALIGNED attribute((aligned(32)))
this requests 32 byte alignment. The alignment is not measured in bits here.
Was that the intention?
That is more aligned that what we usually get from malloc.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@richsalz Perhaps it is time to just remove all of the define tricks and write straightforward C code. The only reason I can think to keep it is if we want to cherry-pick improvements that Mike makes to his code. That seems unlikely.

I disagree.  For highly-repetitive blocks of code with only minor parametric changes between repetitions, it becomes much more difficult to spot subtle errors by visually comparing the blocks than to just check the correctness of the #define and the parameters to each macro call. Sometimes readability of the correctness of the algorithm trumps readability of C control flow. If we had a real meta-compiler that could generate the C code from a higher level algorithm description we'd go with that, but the C-preprocessor is here in a way a poor-man's version of such a compiler.

I pushed a couple of commits to address the latest round of comments. I have either responded to all your comments above - or if not they were just style issues that are fixed.

New commits pushed addressing all the latest feedback. I either responded to your feedback or just fixed the style issues.

Some fixup commits pushed to address some travis/appveyor issues and your feedback as well as a commit to address your question about the clamp function.

The reason for ignore_result() was that casting to void isn’t sufficient to suppress attribute((warn_unused_result)) in some compilers.  If you’re removing that attribute too, then you don’t need ignore_result().

On Feb 6, 2018, at 11:43 PM, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/word.h https://github.com/openssl/openssl/pull/5105#discussion_r166538145:

/* On most arches this will be optimized to a simple cast. */

mask_t ret = 0;
unsigned int i;
unsigned int limit = sizeof(c448_bool_t) / sizeof(mask_t);

if (limit < 1)

   limit = 1;

for (i = 0; i < limit; i++)

   ret |= ~word_is_zero(m >> (i * 8 * sizeof(word_t)));

return ret;
+}

+static ossl_inline void ignore_result(c448_bool_t boo)
+{

(void)boo;
yes, agreed. I was just unsure if this was supposed to force the compiler to not optimize the
ignored expression away.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/5105#discussion_r166538145, or mute the thread https://github.com/notifications/unsubscribe-auth/AARlXLzcuOnGqq6o7C_0hHHf65LWKbxLks5tSVQtgaJpZM4Ri79d.

The _nr skips coefficient reduction.  /* 2+e */ is saying that the coefficients are at most 2+epsilon times the reduction limit.  An older version of the code used _nr more frequently and was littered with this sort of comment, but the current version uses it only in one or two functions to make checking easier.

On Feb 7, 2018, at 12:50 AM, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/curve448.c https://github.com/openssl/openssl/pull/5105#discussion_r166550317:

   /* Scalar conditioning */

   if (t / 8 == 0)

       sb &= -(uint8_t)COFACTOR;

   else if (t == X_PRIVATE_BITS - 1)

       sb = -1;

   k_t = (sb >> (t % 8)) & 1;

   k_t = 0 - k_t;             /* set to all 0s or all 1s */

   swap ^= k_t;

   gf_cond_swap(x2, x3, swap);

   gf_cond_swap(z2, z3, swap);

   swap = k_t;

   gf_add_nr(t1, x2, z2);  /* A = x2 + z2 *//* 2+e */

what does this comment /* 2+e */ mean?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/5105#pullrequestreview-94628160, or mute the thread https://github.com/notifications/unsubscribe-auth/AARlXP6IUlpx3cwJmDLErk2qknp8pmYjks5tSWPGgaJpZM4Ri79d.

Could use -! instead.

On Feb 7, 2018, at 1:56 AM, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/eddsa.c https://github.com/openssl/openssl/pull/5105#discussion_r166567439:

   secret_scalar_ser[EDDSA_448_PRIVATE_BYTES - 2] |= 0x80;

} else {

   secret_scalar_ser[EDDSA_448_PRIVATE_BYTES - 1] &= hibit - 1;

   secret_scalar_ser[EDDSA_448_PRIVATE_BYTES - 1] |= hibit;

}
+}

+static c448_error_t hash_init_with_dom(EVP_MD_CTX *hashctx, uint8_t prehashed,

                                  uint8_t for_prehash,

                                  const uint8_t *context,

                                  size_t context_len)

+{

const char *dom_s = "SigEd448";
uint8_t dom[2];

dom[0] = 2 + word_is_zero(prehashed) + word_is_zero(for_prehash);
It is a bit surprising to use constant_time_is_zero_32 here.
Does this need the constant time hammer at all?
If yes, at least use constant_time_is_zero_8?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/5105#pullrequestreview-94647994, or mute the thread https://github.com/notifications/unsubscribe-auth/AARlXAwKQ9GSuJ9XCVgWuYDuKLzUnSKRks5tSXNpgaJpZM4Ri79d.

The NEON version of the code permutes the limbs within a gf for efficiency reasons.  The other arches don’t.

On Feb 7, 2018, at 2:22 AM, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/field.h https://github.com/openssl/openssl/pull/5105#discussion_r166573958:

+void gf_sub(gf out, const gf a, const gf b);
+void gf_mul(gf_s * RESTRICT out, const gf a, const gf b);
+void gf_mulw_unsigned(gf_s * RESTRICT out, const gf a, uint32_t b);
+void gf_sqr(gf_s * RESTRICT out, const gf a);
+mask_t gf_isr(gf a, const gf x); /** a^2 x = 1, QNR, or 0 if x=0.  Return true if successful */
+mask_t gf_eq(const gf x, const gf y);
+mask_t gf_lobit(const gf x);
+mask_t gf_hibit(const gf x);

+void gf_serialize(uint8_t *serial, const gf x, int with_highbit);
+mask_t gf_deserialize(gf x, const uint8_t serial[SER_BYTES], int with_hibit,

                 uint8_t hi_nmask);

+# include "f_impl.h"            /* Bring in the inline implementations */
+
+# ifndef LIMBPERM
why is this ifndef needed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/5105#pullrequestreview-94655990, or mute the thread https://github.com/notifications/unsubscribe-auth/AARlXOICSDisdYWrZQJj0kll1YOVqIMHks5tSXlogaJpZM4Ri79d.

The NEON version of the code permutes the limbs within a gf for efficiency reasons.  The other arches don’t.

Right. We're only using the 32-bit reference version of the code in this PR so we don't have the architecture specific optimisations here.

More fixes pushed as described in my response to your comments.

The reason for ignore_result() was that casting to void isn’t sufficient to suppress attribute((warn_unused_result)) in some compilers. If you’re removing that attribute too, then you don’t need ignore_result().

Hmmm, well, maybe this attribute was not really helpful then.
Should we remove it?

Hmmm, well, maybe this attribute was not really helpful then.
Should we remove it?

Well it seems we only use it in two places anyway. And in both of those cases the functions in question do not have that attribute against them anyway. I'll remove it - although I will still cast the result to void to make it clear that ignoring the result is intentional.

Travis failures appear unrelated. This is the afalg engine issue that is in master at the moment.

Latest fixes pushed.

I would recommend against Duff’s device.  It’s a hard-to-read hack which isn’t especially useful on a modern compiler anyway.

On Feb 7, 2018, at 10:47 PM, Bernd Edlinger<EMAIL_ADDRESS>wrote:
@bernd-edlinger commented on this pull request.
In crypto/ec/curve448/curve448utils.h https://github.com/openssl/openssl/pull/5105#discussion_r166844252:

       switch((_end) - (_i)) { \

       default: \

       case 8: {_x}; (_i)++; /* fall through */ \

       case 7: {_x}; (_i)++; /* fall through */ \

       case 6: {_x}; (_i)++; /* fall through */ \

       case 5: {_x}; (_i)++; /* fall through */ \

       case 4: {_x}; (_i)++; /* fall through */ \

       case 3: {_x}; (_i)++; /* fall through */ \

       case 2: {_x}; (_i)++; /* fall through */ \

       case 1: {_x}; (_i)++; continue; \

       case 0: break; \

       } \

       break; \

   } while(1); \

} while(0)
+#endif
This is an old-school technique called a Duff's Device.
I would expect to see a compare against 9 and conditional branch to
the default label, followed by an indirect branch via call table here.
What would you expect to be generated here?

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/openssl/openssl/pull/5105#discussion_r166844252, or mute the thread https://github.com/notifications/unsubscribe-auth/AARlXBNpWWqWu9A92H3Eun_beX8kCh19ks5tSpiTgaJpZM4Ri79d.

I would recommend against Duff’s device.

That's ok for me, then we should rip out the FOR_LIMB stuff.

I rebased this, fixed a travis failure (complaining about an unused result from curve448_scalar_decode), resolved the outstanding nits above, and removed all of the FOR_LIMB stuff (replacing all those commits with a single one which removes it all).

All issues above resolved, and a new commit pushed.

New commits pushed addressing latest comments.

Constant time tests updated as requested by @kroeckx.
@bernd-edlinger, please reconfirm!

appappveyor is relevant:
..\test\constant_time_test.c(105): warning C4244: 'function': conversion from 'uint64_t' to 'const unsigned int', possible loss of data
..\test\constant_time_test.c(141): warning C4244: 'function': conversion from 'uint64_t' to 'uint32_t', possible loss of data

travis as well:
test/constant_time_test.d.tmp -MT test/constant_time_test.o -c -o test/constant_time_test.o test/constant_time_test.c
test/constant_time_test.c: In function 'test_32values':
test/constant_time_test.c:349:18: error: iteration 9ul invokes undefined behavior [-Werror=aggressive-loop-optimizations]
         uint32_t f = test_values_32[j];
                  ^
test/constant_time_test.c:348:5: note: containing loop
     for (j = 0; j < sizeof(test_values_32); ++j) {
     ^

Fixup commit pushed to address the Travis and Appveyor problems. Lets see if they are happy...

Travis is still not working:
       # ERROR: (bool) 'constant_time_is_zero_32(a) == CONSTTIME_FALSE_32 == true' failed @ ../_srcdist/test/constant_time_test.c:104
        # false
        not ok 10 - iteration 10
    not ok 2 - test_binops

CIs seem happy now.

Pushed!!! Thanks @bernd-edlinger for all your work reviewing this!


Inputstream.adaptive is missing?

I updated Libreelec to 11.O.1 (Kodi 20.1 Nexus) on a Raspberry Pi 4.
If I try to watch something with an add-on that used to work (VRT of NetFlix), I get that Inputstream.adaptive is misisng. I deleted it, because it wasn't working and it tried unsuccessfully to re-install Widevine.
But now I cannot find the inputstream anymore in the addons and inputstreamhelper doesn't work anymore.
I checked and I have the latest version of those addons.
Apparently I'm the only one, because I'm searching the web for answers, but I can't find anything.

https://dl.google.com/widevine-cdm/versions.txt is gone (404)
Probably has to do with the fact that Google is deprecating Widevine Content Decryption Module?

Some dependencies are missing in the Nexus repository. When trying to install the Internet Archive Game Launcher
I see "vds.rar" and "script.module.infotagger" and "vfs.libarchive" are missing. I found that is indeed the case. So maybe the inputstream/widevine need one of these as well, or maybe other parts are missing as well?

That seems unrelated to me.

OK. So now I have two problems in stead of one :-)

I did a fresh install of LibreELEC and Kodi, now there was no problem with inputstream or WideVine.
I think maybe my SD-card was to full and that caused installation troubles.

I get that Inputstream.adaptive is misisng. I deleted it, because it wasn't working

So you delete inputstream.adaptive and then you wonder why addons relying on it don't work anymore?

I checked and I have the latest version of those addons.

You obviously don't, since you uninstalled one of them...

https://dl.google.com/widevine-cdm/versions.txt is gone (404)

It's not. I don't know, why you can't accesss it, but it is still available.

Probably has to do with the fact that Google is deprecating Widevine Content Decryption Module?

No, they don't. Google only deprecated older versions of the CDM a few months ago when they released a new version.
I'm closing this because of too much nonsense. I have no idea what you guys are doing to your systems, but it has nothing to do with inputstreamhelper.

Hey Horstle,
You're right, it was incomplete and incorrect information. But the inputstream.adaptive already didn't work, deleting it was to install it again, but that didn't work, I couldn't find it anymore.
It was some very idiosyncratic problem, probably because of changing settings, adding and deleting addons, ... . Because on a fresh install, everything worked.


[CU-86er1h8z5] Integrate the value format feature into sub-command line pull

Target

The target why you modify something.
Task ticket: CU-86er1h8z5
About the process integrate value formatting feature into current source code, it would has multiple sub-tasks need to do for dividing the development to be more clear and simple to review and record.

[x] Add new random generators for the new value format URI
[x] Add new properties about the new value format and integrate with the new random generator.
[x] Integrate the value format feature into sub-command line pull.
[ ] Add more tests to cover the issue about converting incorrect between API document config and PyMock-Server config.

Effecting Scope

Breaking changes at the result of subcommand line pull.

Description

Refactor to change the data type hint of property value_format from dict to be BaseFormatModelAdapter of data model adapters for converting logic.
Implement the setting logic at property value_format.
Implement the converting logic at property value_format.
Fix all broken tests because of supporting to set and convert property value_format.
Add note (TODO flag) about bug which cannot convert the value format setting from outside data.

Task linked: CU-86er1h8z5 [SubCmd - pull] Integrate the value format feature into sub-command line pull.

About the SonarQube issue:
The FIXME flag will be resolved in the next PR.


Fixed error in path substitution for naked domains

Fixes #14

There is another error in this line:
rewrite                ^PATHTOCHANGE$ PATHTOCHANGE/ permanent;
That line shouldn't be there when path is /.

Sorry for not getting in touch sooner, I adress the first part of the fix in the next update (your fix was incomplete it need to be changed both in install and upgrade script) and plan to adress the second part soon.

Fixed and tested in dev branche


No way to gracefully free allocated resources/connections

It seems that pjsua2 keeps some resources allocated even when all accounts are unregistered etc
Would it be possible to implement support for libDestroy function? Or is it somehow already accessible?
http://www.pjsip.org/pjsip/docs/html/classpj_1_1Endpoint.htm#a5ee37daaeb8f2c00d2adc8fd756fc656
http://www.pjsip.org/docs/book-latest/html/endpoint.html#shutting-down-the-library

Anything is possible. There currently is no binding to libDestroy() if that's what you're looking for. The other objects (e.g. Transported, Account, Call) will eventually have close() methods added too.


Dr. Memory tool (http://www.drmemory.org/) reports UNADDRESSABLE ACCESS errors within TBB code

Please refer to https://github.com/DynamoRIO/drmemory/issues/2004 for a detailed description.
The errors are reported for memory addresses returned by task::prefix() method (usually within constructors for the task class).
Will appreciate your opinion on whether these are likely to be false positives reported by Dr. Memory, or if this is indeed an issue within TBB.
Thanks!

I suppose it is a false positive because a pointer to a task is not a pointer to the first byte of allocated memory. You can see in scheduler.cpp:322 that a task pointer is shifted to the right on the task_prefix_reservation_size number of bytes. Therefore, when we calculate a prefix pointer in task.h:904 we shift to the left safely (sizeof(task_prefix)<=task_prefix_reservation_size scheduler_common.h:163)

Response from Dr. Memory developers:

Yes, a custom allocator that pads can result in problems tracking which addresses are addressable. The typical solution is to annotate the allocator so that the tool knows which range of memory it allocated. That feature is not yet present in Dr. Memory (#107). Do your custom allocators already have Valgrind annotations? If so the plan is for Dr. Memory to support those.

Alexey, could you please disclose some info on how NFS_Allocate() and NFS_Free() work in TBB code; in particular, whether they have "Valgrind annotations", and, if not, if there are plans do add them?

I have answered in the Dr.Memory related thread.


showCustom small icon

Is there a way to have bigger icon for showCustom? I tried increasing the size of the image but still no. Help

any new?


Cannot compare dynamic values without explicit cast

The main example:
print extract_all(@"(\d+)", "a set of numbers: 123, 567 and 789") == dynamic(["123", "567", "789"])
throws the error: Cannot compare dynamic values without explicit cast

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

ID: df27c2b2-99c9-ed18-5b94-85e506132632
Version Independent ID: d77fe688-fe1d-1c4f-eacc-d5d52b2d130f
Content: extract_all() - Azure Data Explorer
Content Source: data-explorer/kusto/query/extractallfunction.md
Service: data-explorer
GitHub Login: @orspod
Microsoft Alias: orspodek

@evilches thanks for your feedback.
@yonileibowitz - can you assist re: this question?

thanks @evilches, we'll have the example in the docs updated

resolved by https://github.com/MicrosoftDocs/dataexplorer-docs-pr/pull/125

@yonileibowitz thanks for the super quick fix.
@evilches hope this resolves the issue. I am closing the issue. Feel free to reach out with any additional questions.


WOPI Host url error

Usage & Examples
http://[owas.domain]/we/wordeditorframe.aspx?WOPISrc=http%3a%2f%2flocalhost%3a8080%2fwopi%2ffiles%2fword.docx&access_token=[token]&ui=zh-CN
http://[owas.domain]/p/PowerPointFrame.aspx?WOPISrc=http%3a%2f%2flocalhost%3a8080%2fwopi%2ffiles%2fppt.pptx&access_token=[token]&ui=zh-CN
http://[owas.domain]/x/_layouts/xlviewerinternal.aspx?WOPISrc=http%3a%2f%2flocalhost%3a8080%2fwopi%2ffiles%2fBook1.xlsx&access_token=[token]&ui=zh-CN
==>
http://[owas.domain]/x/_layouts/xlviewerinternal.aspx?WOPISrc=http%3a%2f%2flocalhost%3a8080%2fwopi%2ffiles%2fBook1.xlsx?access_token=[token]&ui=zh-CN
&access_token   =====> ?access_token
-- Example site url --
https://github.com/rchcomm/WopiHost/blob/master/README.md

how to create access_token

access_token, generally, is the credential, for accessing the file, such as encode(base64) from username and password

Thanks for reply
I need a sample script for editing in asp
and I don't saw any thing where I can pass username and password

for example: access_token=base64(u=userid;p=password)

Can I get any asp sample please

you can use this project "marx-yu/WopiHost". what your problem?
for asp sample, I am sorry, I have not any help.

Thanks for reply
on what file in your project i have to edit the code for passing my user credentials.
do i need to pass user credentials like domain\user or just user
can you please tell me what is the best way for building asp for this ???

Now I can view document in the browser.I need to edit without sharepoint in my web application by PHP.I cannot learn C# before.Does the access_token is necessary to create it?

I just want to install Office web apps server 2013 without any integration like sharepoint, exchange or lync.And i need to edit the office files in browser.

Access token is something that you provide to WOPI Client  (OWA) and WOPI Client uses it to authenticate against WOPI Host. marx-yu/WopiHost is example of such WOPI Host.
If you have a look at CobaltServer.cs (lines 64,65)  you'll see that it only checks whether the access_token is set in the query string. In real scenario you would probably want to extract user credentials from it, perform authentication and based on requested file perform also authorization check.
To use marx-yu/WopiHost in web application just replace IAsyncResult result with HttpContext context and adjust the code accordingly. Then create IHttpHandler and pass the context to CobaltServer.
I'm currently working on a generic WOPI host that will be compatible with web apps as well as windows apps. It'll be on GitHub in few weeks.

@petrsvihlik thanks for so detail explain and bright work of WOPI host

@marx-yu,  @petrsvihlik
i really cant understand what to edit i dont have much programming skills
can you please tell me what modifications has to be done

how i can create access_token of my own to enter in an url
do i need to make any changes in this code as the petrsvihlik say in cobaltserver.cs 64, 65 lines one of them is empty line

my process until now
i can edit all documents using sharepoint

@marx-yu , @petrsvihlik
i edited code in cobaltserver.cs
i edited cobaltserver.cs
var fileExt = filename.Substring(filename.LastIndexOf('.') + 1);
if (fileExt.ToLower().Equals(@"xlsx"))
editSession = new FileSession(filename, m_docsPath + "/" + filename, @"administrator", @"admin"<EMAIL_ADDRESS>false);
else
editSession = new CobaltSession(filename, m_docsPath + "/" + filename, @"administrator", @"admin"<EMAIL_ADDRESS>false);
CobaltSessionManager.Instance.AddSession(editSession);
and uploaded the files in https://defuse.ca/checksums.htm to create sha256
http://wopi.mydomain.tk/we/wordeditorframe.aspx?WOPISrc=http%3A%2F%2Flocalhost%3A8080%2Fwopi%2Ffiles%2FTest.docx&access_token=45406b1543d8762192ae1a6a5a8de6d878af48939eb1343849f5ddc5785ef0ee&ui=en-US
http://wopi.mydomain.tk/x/_layouts/xlviewerinternal.aspx?WOPISrc=http%3A%2F%2Flocalhost%3A8080%2Fwopi%2Ffiles%2FTest.xlsx&access_token=2702ac89018e16436f8830ba58fad13bb68485be7b79ab5a2ddb86a696f547f5&ui=en-US
but i am getting error on both

what error are you getting? have you tried to debug the CobaltServer? does the request even reach the code? i see you are passing some kind of token - that should be enough for WopiHost as it only checks whether it's set or not. so the problem will be elsewhere

@petrsvihlik can i message you in twitter ???

hi, i'd prefer github or stackoverflow as those are the right places to discuss dev topics.

@petrsvihlik
ok but i want to show everything by giving remote desktop of mine

@kmkmahesh maybe later in the evening (i'm gmt+1). i also have work to be done :D

ok when ever you are free i am ready i just emailed to your organization email


Report for calls statistics

There is info about all calls made.
report name: calls_statistics
alpha version here, includes support for calls list for older iOS:
https://github.com/timotoots/iphonebackuptools/commit/57e1da7b7b88bf4bfae1cafdd88e1eded294276a

That sounds like a good report to include. From my testing, It only supports earlier iOS Versions. The newer ones (approx 9.0+) utilize the Calls2 database for which the report would work like this: SELECT * from ZCALLDBPROPERTIES. I can implement the newer iOS Support if need be.

Is the output as key/value pairs okay?
I don't know how easy it is to get to know every iOS version and the formatting. Have you thought of auto-discovery of the databases? For some it would be easy. If calls exists then it's newer function, if not then older function. Might be faster to get on track than mapping the exact history of iOS versions. Just a thought..

That's a good point and I'll explore this for the future. When I was doing my initial research on this, sometimes iOS would leave databases either completely empty or leave them just with empty tables, instead of deleting them altogether.

Added in #19


Shader compilation failed

Trying to compile generated shader code and got this error:
@"[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (Error code 3):\r\n"
@"Compilation failed: \r\n"
@"\r\n"
@"program_source:378:27: error: as_type cast from 'long' to 'float' is not allowed\r\n"
@"            float _1181 = as_type<float>((as_type<int>((_1172.x + _1172.y) + _1172.z) & (-2147483648)) |<PHONE_NUMBER>);\r\n"
@"                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\r\n"

Original bug: https://github.com/KhronosGroup/MoltenVK/issues/1476

It's the classic INT_MIN problem: because<PHONE_NUMBER> cannot be represented as an int, it gets promoted to long instead. That's why most definitions of INT_MIN use some oblique expression like ((-2147483647)-1). SPIRV-Cross will need to do the same for constants that happen to be the minimum representable signed value.

Thanks for the clarification. Is there any workaround for this?

This should already be fixed on master.


update xmldom to use @xmldom/xmldom

xmdom 0.6.0 is subject to the  Misinterpretation of malicious XML input vulnerability
https://npmjs.com/advisories/1769
To update xmldom to @xmldom/xmldom (0.7.2)
https://www.npmjs.com/package/@xmldom/xmldom
similar issue for xml-encryption and xml-crypto
for xml-crypto, can update to 2.1.3
and for xml-encryption, could resolve it to use the @xmldom/xmldom version

Thanks for writing in! We're aware of this issue, and are waiting for auth0/node-xml-encryption#88 to be merged and published before acting on this.


fix: Examples azsadmin\resource-manager\storage\Microsoft.Storage.Admin

start is an integer
count is an integer
StartIndex is an integer

Automation for azure-sdk-for-python

Automation for azure-sdk-for-node

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java

Automation for azure-sdk-for-ruby

Can one of the admins verify this patch?

@nschonni Thanks for submitting these changes. Since these are non description changes, I am looping in  @shanswmicrosoft who originally created this file.
@shanswmicrosoft Please take a look at these changes. If they are valid changes, Please create a new PR.
I will be closing this PR without merging.


response in json , template it does not effective

moco version： moco-runner-0.12.0-standalone.jar

json:

mock file content :

mock result:

====================================================
if  response in text, template is ok

text:

mock file content : __

mock result:

====================================================
my app need json response , and i need different req.uri to proxy different real uri
how to config template funtion in json response
3ks

+1

This configuration can solve my problem.


⚠ Bofrost.es has degraded performance

In 7843c5c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1660 ms

Resolved: Bofrost.es performance has improved in 61b6e30 after 46 minutes.


Add server side function wkt("polygon

This was created accidentally. See the real issue with the longer name.


Concurrent problems in Hooks

Public func RegisterStartHook and RegisterShutdownHook will be used concurrently by users and type Hooks []func() is not a concurrency-safe object.
If I could, I would pr = =。

In server package(server/hooks.go)，RegisterShutdownHook write the Hooks global variables and server.Run(server/hooks.go:191) func read Hooks  global variables.


Homematic IP PS-2 Device not showing up

I agree to the following

[X] I have read the documentation
[X] I am aware of the latest release notes
[X] The Backend (CCU/Homegear/...) is working as expected and connected devices are controllable by its own UI.
[X] I am running the latest version of the custom_component (and Home Assistant)

The problem
I have added a Homematic IP device PS-2 to my CCU3. It is not showing up in Home Assistant and the logging shows the following problem mainly "missing paramsets"
What version of HomematicIP (local) has the issue?
1.45.0
What was the last working version of HomematicIP (local)?
1.45.0
What type of installation are you running?
Home Assistant OS
What type of installation are you running for your homematic backend?
CCU3
Which version of your homematic backend are you running?
3.71.12
What hardware are you running for your system?
NAS with virtual system
Which config details do you use

[X] Username and password
[ ] TLS
[ ] callback data (see)

Which interfaces do you use?

[X] Homematic IP
[X] Homematic (Bidcos-RF)
[ ] Groups (Heating-Group)
[ ] BidCos-Wired (HM-Wired)

Diagnostics information
No response
Anything in the logs that might be useful for us? The log (full log) is the best source to support trouble shooting!
2023-10-08 19:42:01.345 DEBUG (MainThread) [hahomematic.platforms.device] __INIT__: Initialized device: ccu3-webui-HmIP-RF, 00351F29B5F00C, HmIP-PS-2, UG_Keller_ESP32_Strom
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:0, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:1, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:2, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:3, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:4, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:5, missing paramsets
2023-10-08 19:42:01.346 DEBUG (MainThread) [hahomematic.platforms] CREATE_ENTITIES: Skipping channel 00351F29B5F00C:6, missing paramsets

Additional information
No response

Run service homematicip_local.clear_cache and restart HA.

Run service homematicip_local.clear_cache and restart HA.

Did not solve the problem.

Are there any data in the paramset file (cache) related to the above mentioned channels 00351F29B5F00C:0 to 00351F29B5F00C:6?
If empty then it looks like the ccu is not providing the required data.
Try remove the device from ccu, and add it again.
The device is supported by the integration an mine works just fine.

Please add the integration diagnostics(click on the integration,…).

Thanks for your support. Meanwhile I did a second clear cache, followed by a hard reboot (power off and reboot of the virtual maschine). Now the device is visible! Maybe something went wrong at the first clear cache.
Sounds like the problem is on my side. Thanks for your very quick response. I'll close the issue.


JSON contains method not working with URLs

Laravel Version: 8.37.0
PHP Version: 8.0
Database Driver & Version: MariaDB 10.3.25

Description:
The whereJsonContains method of the database builder does not work if the value is an URL.
I think the issue here is that Laravel doesn't add the required quotes (") at the beginning and at the end of the URI.
Steps To Reproduce:

Create a table with a JSON field
Populate the JSON field with a JSON array (example: `["a","b","https://laravel.com"]
Try the following statements ($db must be an instantiated DB call, such as DB::table('db_table'):

DB::enableQueryLog();
$res1 = $db->whereJsonContains($query, 'a')->get();
$res2 = $db->whereJsonContains($query, 'https://laravel.com')->get();
$res3 = $db->whereRaw("JSON_CONTAINS(`$query`, '\"https://laravel.com\"')")->get()
dd(DB::getQueryLog(), $res1, $res2, $res3);

Notice how the first statement worked, the second no, and the third worked

You need to escape the slashes because whereJsonContains() runs the value through json_encode(): "https:\/\/laravel.com"

You need to escape the slashes because whereJsonContains() runs the value through json_encode(): "https:\/\/laravel.com"

The escaped string works in the raw statement like the non escaped one. What changes are the quotes

I think the issue here is that Laravel doesn't add the required quotes (") at the beginning and at the end of the URI.

whereJsonContains() does add the quotes, they are coming from json_encode().

Hey @maicol07. Please see the responses by @staudenmeir above.

Seems that the value in the DB must be escaped as well to get it work


Remove rococo runtime dependencies

In order to obtain events on the rococo relay chain, the order pallet introduces the rococo runtime dependency.
In order to submit signed transactions to the rococo relay chain, the node code introduces the rococo runtime dependency.
We should remove this dependency, otherwise upgrading rococo or switching to connecting to polkadot will be affected.
The work that needs to be completed is:

The OnDemandOrderPlaced event on the relay chain can be parsed in the order pallet. The source code that needs to be replaced is as follows:
let v_price: Vec = head_data
.iter()
.filter_map(|item| {
if let rococo_runtime::RuntimeEvent::OnDemandAssignmentProvider(
parachains_assigner_on_demand::Event::OnDemandOrderPlaced {
para_id: pid,
spot_price: sprice,
},
) = item.event
{
if pid == para_id {
Some(sprice)
} else {
None
}
} else {
None
}
})
.collect();
Able to complete offline signature in node without relying on rococo runtime, for example, you can use subxt

Import runtime Call / Storage metadata without import the whole runtime
https://github.com/paritytech/substrate/issues/8158


Rename CRD ServiceGroup

Current name ServiceGroup conflicts with concept of a group in Habitat, as well as concept of a Service in Kubernetes, we need to come up with a better name for CRD.
List of ideas:

HabitatService
Habitat

What changed since we decided to close #30?

We decided to close that PR as we couldn't come up with anything better then ServiceGroup, but we agreed that ServiceGroup is not the best name from what I remember?
In one of our last meetings I believe we sort of agreed on Habitat, but I wanted your input before changing everything to that, so I opened the issue to not forget and also to discuss/collect ideas.

A pro of using "Habitat" would be that we would follow the logic of other operators:

PrometheusOperator -> Prometheus
ReplicaSetController -> ReplicaSet
...

Closed by https://github.com/kinvolk/habitat-operator/pull/85


Snap scheduler not getting configured properly

There exists some sort of race when setting up the snap scheduler. When I installed east-2a, while it looked like the snap schedule got configured properly, it didn't snap accd to the cron job. When I previously wrote the ansible for this, I had some trouble re snap_scheduler.py init happening on multiple nodes simultaneously causing initialization failures. I fixed that via a random delay... poor choice.
Hopefully moving to a truly serial initialization of the schedulers will fix this. See https://github.com/ansible/ansible/issues/12170#issuecomment-283950548 for an approach to having a serial task w/in a role. Also 2 comments down suggests using play_hosts instead of marker. Hopefully we can put this in a block: with a sync and pause to fix the race.

As it turns out, the above referenced comment didn't work, but the following one does: https://github.com/ansible/ansible/issues/12170#issuecomment-372263149


Add obj-reader and json-reader and allow to set the reader

For more details, open the Copilot Workspace session.

Superseded by #2 . Closing this one.


#2218 Align navbar with recent style changes

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[ ] Bugfix
[ ] Feature
[x] Refactoring

What's new?

[x] PR closes #2218
[ ]

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address:
Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

pay 100 usd

😍 Perfect, I’ve sent the payout
💵 $100 @ 186.01 USD/KSM ~ 0.538 $KSM
🧗 EzGc4s9PgCPx1YnF3fqzhLzVHpHMTL4LWPScwpDrR8JKgSU
🔗 0xcbcd3c022fc2e65bcb15f437d965b9866015339c00169fb807fbd9e3903d29a9
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


There is a type in the Versions Information page

Describe the bug
There is a typo in the Spanish translation of the Version Information page: it reads "durante 18 o 30meses a partir de la fecha de lanzamiento,". There is a space missing after the 30 number.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://docs.microsoft.com/es-es/windows/release-information/'
Read the first paragraph.

Expected behavior
Text must be "durante 18 o 30 meses a partir de la fecha de lanzamiento..."
Screenshots

Desktop (please complete the following information):

OS: Windows 2010 Pro
Browser: Firefox
Version 78.0.1 (64-bit)

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
I am happy to help. :-)

@lizap do you know the loc contact for this repo? Seems like a manual translation issue - missing space.
@ShenLanJohn could you assist?