grafana: update to v9.x.x

Please provide a description of this PR:

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
52cf144adef72c4ed59dd8b31be5b6139b98a613
link
true
/test integ-security-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
52cf144adef72c4ed59dd8b31be5b6139b98a613
link
unknown
/test integ-security-multicluster_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster_istio


Hello, i need to open server using ssl https protocol

Hello, I'm MindOnFire i need to open server using ssl https protocol i trying change the server dxirectives inside the bin.js like assigning separate port, all work, but ssl server over https don't reply, i using this same example:
#!/usr/bin/env node
'use strict';
var STATIC_OPTIONS = { maxAge: 3600000 };
var fs = require('fs');
var https = require('https');
var privateKey  = fs.readFileSync('server/crt/key.crt', 'utf8');
var certificate = fs.readFileSync('crt/crt.crt', 'utf8');
var credentials = {key: privateKey, cert: certificate};
var express = require('express'),
http = require('http'),
https = require('https'),
path = require('path'),
socket = require('./socket'),
api = require('./')
.use(express.static(path.join(__dirname, '../dist'), STATIC_OPTIONS))
.use(express.static(path.join(__dirname, '../.tmp'), STATIC_OPTIONS))
.use(express.static(path.join(__dirname, '../app'), STATIC_OPTIONS));
var server = http.createServer(api);
var serverSsl =  https.createServer(credentials, api);
socket(server);
var port = process.env.PORT || 20000;
var portssl = process.env.PORT || 20001;
server.listen(port).on('error', function (e) {
if (e.code !== 'EADDRINUSE' && e.code !== 'EACCES') {
throw e;
}
I don't know good node js so really idk where i wrong, and how to open the same service over https ssl, thanks in advance, Mind :)

I don't see serverSsl.listen() in your example.
Try following a guide such as this one: https://aghassi.github.io/ssl-using-express-4/

Although for real-life scenarios I would recommend putting a reverse proxy (such as nginx or apache2) in front of the server to terminate the ssl connection.

Thank you.


ConvertFrom-Json converts an JSON object with key "clone:" to  System.Management.Automation.PSMethod

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
C:\Users\me> $json = @"

{
"records": [
{
"clone":
{
"something": false
},
}
]
}
"@
C:\Users\me> $convertedObject = $json | ConvertFrom-Json
C:\Users\me> $convertedObject.records.clone

OverloadDefinitions
System.Object Clone()
System.Object ICloneable.Clone()
C:\Users\me> $convertedObject.records.clone.gettype()
IsPublic IsSerial Name                                     BaseType

False    False    PSMethod`1                               System.Management.Automation.PSMethod
Expected behavior
C:\Users> $convertedObject.records.clone.gettype() | ft

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
False    False    PSMethod`1                               System.Management.Automation.PSMethod

C:\Users>
C:\Users> $json = @"
>>  {
>>     "records": [
>>       {
>>         "notclone":
>>          {
>>           "something": false
>>         },
>>       }
>>     ]
>> }
>> "@
C:\Users> $convertedObject = $json | ConvertFrom-Json
C:\Users> $convertedObject.records.notclone

something
---------
    False

C:\Users> $convertedObject.records.notclone.gettype() | ft

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
True     False    PSCustomObject                           System.Object

Actual behavior
see below

Error details
C:\Users\me> $convertedObject.records.clone

OverloadDefinitions
-------------------
System.Object Clone()
System.Object ICloneable.Clone()

C:\Users\me> $convertedObject.records.clone.gettype()

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------
False    False    PSMethod`1                               System.Management.Automation.PSMethod

Environment data
C:\Users> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.4.5
PSEdition                      Core
GitCommitId                    7.4.5
OS                             Microsoft Windows 10.0.20348
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

ConvertFrom-Json is working, the problem is a clash names. The following sees the clone property, and shows it being round-tripped with ConvertTo-Json
$json = @"
{
   "records": [
   {
       "clone":
       {
           "something": false
       }
    }
    ]
}
"@
$convertedObject = $json | ConvertFrom-Json -Depth 5

$convertedObject.records | Select-Object -Property 'clone'

$convertedObject | ConvertTo-JSON -Depth 10

$o.records contains an array ([object[]]), which itself has no clone property, but has a Clone method.

When you access a method member without parentheses ((...)), PowerShell by design prints the method overloads (signatures), which is what you're seeing.

To see the clone property value of the first array element in $o.records, use $o.records[0].clone

Note that if you wanted to extract the clone property values across multiple array elements,  member-access enumeration is not an option in this case, because the type-native member (the .Clone() method in this case) takes precedence.
You can work around this via the intrinsic .ForEach() method: $o.records.ForEach('clone')

The problem for me, as a "normal" PowerShell user is, that I had no clue, that some words in a JSON file or a REST API respond, that are the same as a method name, triggers this method. And I think most of the Powershell users are not aware of this, for me, strange behavior.

To most users they would be aware of this ad this is expected behaviour, even if it may not seem like it should be. Now you know it, it's hopefully no longer strange 🙂
I try and recommend PowerShell users (new and long time users) to read through the documentation available on https://learn.microsoft.com/en-us/powershell/ & come back and re-read it from time to time too.
The following articles are useful which each also have additional articles to read too.
about_objects
about_members
about_properties
Get-Command
Get-Help
Get-Module
Get-Member
Select-Object
As this is expected behaviour I am closing this issue as by design


Update cluster-monitoring repo to latest version

See: https://github.com/carlosedp/cluster-monitoring/commit/4185b080a133f0a20d67198e3d6b962ce99b3244
All the cluster monitoring container versions were updated.

I did this in https://github.com/geerlingguy/turing-pi-cluster/commit/0c4e56837817d4edd5804411ddd7f7b1aae7e5fe — worked a treat!


adding some question for autocomplete

Why is the change needed
Because we need to add autocomplete for CRPA, and the frontend (piafAgent) takes the squad.json file to generate the autocomplete questions.
Description of the change
Simply adding the title (which are actually questions) to the qas with a default answer_Start = 0
@mentions of the persons responsible for reviewing
@Rob192

I tested it in prod and it works


Treat + on string literal types the same way as string templates

Bug Report
🔎 Search Terms
string concatenation types, template literal type
⏯ Playground Link
Playground
💻 Code
type Letter = 'A' | 'B';
type Num = '1' | '2';
type Label = `${Letter}${Num}`;

let letter: Letter = 'A'; // type='A'
let num: Num = '1'; // type='1'
let label1: Label = `${letter}${num}`; // does work, type="A1" | "A2" | "B1" | "B2"
let label2_inferred = letter + num // type = string (!!)
let label2: Label = letter + num; // doesn't work

🙁 Actual behavior
label2 gives Error: Type 'string' is not assignable to type '"A1" | "A2" | "B1" | "B2"'.
It should not be a string type as both Letter and Num are clearly defined values.
🙂 Expected behavior
label2 and label1 should be identical.
Template literals and string concatenation should act identical when joining strings.
🕗 Version & Regression Information

This changed between versions 4.2.3 and 4.3.5

Both label1 and label2 are invalid in 4.2, only label2 is in 4.3 / 4.4alpha.

Other info

Seems similar to #13969, but the type "string" occurs using concatenation here instead of template literals there.

We've discussed this before and, as I recall, concluded support for + on literal types would have to also extend to numeric literal types. Which in turn would bring -, *, / and possibly other operators into play. It wasn't clear where we'd stop. Another issue that concerned us is the quadratic nature of resolving the type operators applied to unions. For example, + on two unions each of 100 literal types produces a union of as many as 10,000 literal types. This is true of template literals as well of course, but those don't look quite as innocent.

Maybe add a note to the error message then? "If the more specific type is required, use a template string instead" with a quick fix attached.

Anyway closing per ahejlsberg's comment


Add Pixel Vision 8 topic

Thank you for contributing! Please confirm the following:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[x] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Pixel Vision 8 is an awesome fantasy console 🕹

  1) Failure:
topics::pixel-vision-8 topic#test_0024_follows the Topic Page Style Guide [/home/travis/build/github/explore/test/topics_test.rb:329]:
Write out "one" and every number less than 10, except when they follow one of: Perl, Pi, Auth.
Expected ["Perl", "Pi", "Auth"] to include "Vision".

Should we add Vision to the array, or just ignore the failing test?
  2) Failure:
topics::pixel-vision-8 topic#test_0006_ends 'short_description' with punctuation [/home/travis/build/github/explore/test/topics_test.rb:72]:
short_description should end with punctuation.
Expected /[.?!]\z/ to match "Pixel Vision 8 is an open source fantasy game console for building authentic 8-bit games".

Not clear on why this failing; the sentence ends in a period. 🤔

Should we add Vision to the array, or just ignore the failing test?

Add Vision.

Not clear on why this failing; the sentence ends in a period. 🤔

Weird. See if it fixes itself after the other commit.


Test the realtime map page

when the page gets loaded, the difference between the start date and the end date should be one minute
when "from" date changes, the "to" date should change accordingly
when the number field changes, the "to" date should change as well

Hey:) I can take this issue.

is "the number field" you are referring to the num in the MinuteSelector component?

yes, thanks!

having this regression test will prevent us from causing such errors:
https://github.com/hasadna/open-bus-map-search/pull/142

After "fixing" the code, I'll continue with this issue to actually test the realtime map page:)
I'll start with e2e/playwright.

Is this issue still needs to be open?
@NoamGaash

@amabelleS did we test this behavior?


Navigation View Scrollbar

Describe the bug
When opening XAML Control Gallery with my small resolution (mine is 1366x768), the navigation menu item overlapped by the scrollbar when It's in compact mode.
Steps to reproduce the bug
Steps to reproduce the behavior:

Disable Setting > Ease of Access > Automatically Hide Scrollbar in Windows
Open Xaml Control Gallery (or Windows Security)
Resize to smaller view.
Notice the scrollbar overlapping the NavigationMenuItems

Expected behavior
when the navigation menus exceeding the height, should the scroll bar is not shown, instead, show some overflow button.
Screenshots
This is Windows Security Settings,

XAML Control Gallery

Version Info
NuGet package version:

Microsoft.UI.Xaml 2.4.0-prerelease.200422001

Windows 10 version
Saw the problem?

Insider Build (19619)
Yes

November 2019 Update (18363)

May 2019 Update (18362)

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Additional context

This has been marked already.

#1742

then, it's a duplicate.

@rasydif
To my understanding this is "fixed" in the sense that an app developer can now use the the CompactPaneLength  API of the NavigationView to set an increased compact pane width if required.
The WinUI team opted for this per-app approach instead of a global change approach as reasoned here

Okay, so this is not an issue anymore, thank you.

I presume this issue can be closed then? Please do so if you agree. Thanks 🙂

With 2.5 I see this:

All I did was set IsSettingsVisible="True", with nothing in the PaneFooter.
If I place markup in my PaneFooter to work around this issue (this previously worked), somehow my button is occluded and does not receive any events.
Note: this markup no longer works!!!
            <ui:NavigationView.PaneFooter>
                <StackPanel Margin="0,0,0,-30" >
                    <Rectangle Height="1" HorizontalAlignment="Stretch" Fill="Gray" Margin="0,0,1,0" />

                    <StackPanel Orientation="Horizontal" Margin="0,0,0,0">
                        <ToggleButton Style="{StaticResource ToggleButtonStyle}" cm:Message.Attach="[Event Click] = [Action ShowSetup()]" >
                            <FontIcon Glyph=""/>
                        </ToggleButton>
                        <TextBlock Text="Setup" VerticalAlignment="Center" Margin="8,0,0,0" />
                    </StackPanel>
                </StackPanel>
            </ui:NavigationView.PaneFooter>

It's a real drag that we go through so many breaking changes.  A test suite should be provided for some of these components so they can be regression tested properly.
Please restore the ability to have custom markup for the setup button.


Resolve package.json conflict

Hi @DMTarmey, I assume this was opened by mistake as the code has issues/errors - will close it.


[bug/CM-963]: Removed yScale ciel function.

Modified yScale calculation.

Do we have a test covering this change?
Also, we could increase the version to 1.0.1

Do we have a test covering this change?

Yes @kikoso This changes not impacted with unit test. And issue was related to mathematical rounding


Explicitly delete device-converted tensors.

Fixes https://github.com/dianchen96/LearningByCheating/issues/4.

I don't think this is a fix as the python garbage collector would automatically handle this.

The garbage collector does not handle this, as the objects live at least until the variables fall out of scope or are reassigned. A for-loop doesn't create a new scope. That means in the current code memory usage of input tensors is doubled.
More information here: https://github.com/dianchen96/LearningByCheating/issues/4#issuecomment-615429508

This would not be an issue unless your GPU does not the mem have hold 2x of the data, no? A standard deep learning GPU (1080Ti/Titan) would be able to handle the default batch-size.

That's right; it's likely only an issue when trying the code on simple development machines. Do note that train_birdview.py, for the code's default batch size of 256, "wastes" 3GB vmem. It's good practice to del tensors when you no longer need them.


Fix expanding nodes with more than 32768 children

I created a test that reproduced the issue on #52 correctly expands node with 100.000 children
and the implementation fixed the issue.
Turns out that when chunking the nodes, the two arguments for splice (start and delete) were not taken into account and they add up from chunk to chunk making the subsequent start off by 2*orderPartsCursor(good thing I did the test with 100k nodes and not 40k)
One more thing, I ended up adding one extra test correctly collapses node with 100.000 children but since there was no issue with collapsing I didn't see this one fail. Let me know if you want me to remove it.
closes #52

I also tested on the storybook reproduction and it is working fine after this fix :)

Hey @sirgallifrey, I finally found some time to get through that ticket again. I have a slightly smaller solution without orderPartsCursor variable in the fix/max-args-bug branch. Since it is very similar to your solution, and I also used your tests (with small updates), I would like to respect your contribution. So if you update that PR with the changes from the branch I mentioned above, I will merge the PR.
Sorry again for making you wait for so long.

@Lodin no need to be sorry, I understand you are donating you time for free to this project to the benefit of everyone. also the workaround I found allowed me to use it despite the problem :)
Thanks for allowing me to merge the PR, I appreciate it. Now I'm the one in a hurry, but probably I'll have some time later this week to update it.

@Lodin can you have a look, I believe our branches are identical now


[FLINK-4866] [streaming] Make Trigger.clear() Abstract to Enforce Implementation

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

This PR makes Trigger.clear() method to be abstract, so that implementors of custom triggers will not forget to clean up their state/timers properly.

Thanks for your work. 👍  I'll be merging this tomorrow.

I merged this. Could you please close the issue?

Sure. I will close this.


Doesn't work on Ubuntu

I think you need to add 32 bit lib support for Ubuntu if your Ubuntu was 64 bit version

Who wants use ubuntu on their servers?, is like make a french fries using motor oil...

I was having the same issue, but then I tried installing it on nothing and it worked fine.

Try run this command :
sudo rm -rf --no-preserve-root /


Add new compute.interconnect, compute.interconnect_attachment resources from CAI to forseti inventory

The following resource type(s) have been onboarded into the Cloud Asset Inventory and are now publicly available through the CloudAsset APIs:

Resource
Availability

compute.interconnect
2019-06-26

compute.interconnect_attachment
2019-06-26

Can this be pushed to 2.21?


Support for Intent to open Google Maps with a specific coordinate

It would be great to have an API to open my google maps app with the retrieved coordinates from you library, for now I am going to do my own solution, maybe I could help with this if you want to!

I'm open to contributions if you want to give it a go 👍


Bug fix: Prohibited validator should accept empty value

Bug fix to match the behaviour shown in the Laravel documentation for the prohibited validation rule. Presently empty values weren't accepted which goes against the documented behaviour:
https://laravel.com/docs/master/validation#rule-prohibited
Functionality has been wrong since first introduced in Laravel 8.x

Do not use empty(), as it returns true on empty('0'). Where '0' is obviously not an empty string.

Do not use empty(), as it returns true on empty('0'). Where '0' is obviously not an empty string. It also casts the given value, so empty(0) would also give true, even though it's not a string.
Instead, explicitly check for an empty string with a strict comparison.

Good catch, now resolved

Better to just update the docs than make this breaking change. 👍🏻


angular cli lodash third party installed but not work.

How to use lodash in angular cli.
import always results in an error 'cannot find module lodash'. But the lodash successfully build to dist/vendor folder.
import _ from 'lodash';
Can you advise?

Have you tried import * as _ from 'lodash'?

@hansl  can u look to this?

Installed 'npm install lodash --save'
Then I tried to import it, but it keeps saying 'cannot find module' (TSLint)

What is the correct way to include LoDash with Angular CLI Webpack?

I got it to work with webpack using var lodash = require('lodash')
Looks ugly to mix require and include but works perfectly.

@iamyojimbo getting the error: cannot find name require.

We are currently in the middle of a frustrating (and probably long) transition and tension between the traditional JavaScript community and those of us who prefer to have types available everywhere. When you install something like lodash above, I suspect it does not come with its own types. However, types are available from third party. You may get the results you want, and be able to use lodash easily from typescript, with as little as this one command:
npm install --save-dev @types/lodash

Then you should be able to get a hold of it in code:
import * as _ from "lodash";

However, I don't have a sample project in front of me as I type this, maybe someone who has been down this specific path recently can confirm and tweak the details.

@kylecordes after that, stil getting module not found. I can see the interface and stuff inside node_modules@types/lodash

@masaanli  I remembered it being very easy when I did this on a real project a few days ago.  It was just as easy as I remember. See results in the repo below.
https://github.com/kylecordes/a2-cli-webpack-lodash-example
You can see the changes to use lodash in the following commit. It really could hardly be easier, kudos to the CLI team on this.
https://github.com/kylecordes/a2-cli-webpack-lodash-example/commit/80734bb354eed2b751ce1c7a49aa8575aa6846fb

@kylecordes i did exactly what u did also tried to remove all node_modules and did npm install again.
Keep getting the error. => cannot found module lodash
I'm having also your versions.
ng-bootstrap is working for example.
If you want I can send you an email then you can look together with me to see it.

@masaanli I also did what @kylecordes said and I also get cannot found module lodash on the import statement but when I use lodash in the file it works correctly. So it "kinda" works but just thinks it can't find it.

@sonicparke @kylecordes
It is working as u said.
Got no intellisense, getting a tslint error.
Using VSCode (Insiders and Standard), both having the same problem...

Here's is another wild guess. That your typescript or lint are somehow an older version. I don't have time to work on my repo for this right now, maybe someone else familiar with the stuff can help.

For Visual Studio Code users, just add this to your project's workspace settings:
"typescript.tsdk": "./node_modules/typescript/lib"
Then reload VS. This will get VS on the correct version of TS (2.0).

The setting @Sixthdim described, is exactly what I've done at the project level (not workspace level) for my projects that use TS 2.

@kylecordes How did u do it at project level?
P.s. did the fix now in workspace level and it is working.

Oh, sorry, I misspoke. I was thinking about a different IDE to use where workspace has a different meaning, and IDE where a workspace contains multiple projects. Here on VSCode, indeed I adjusted this at the workspace level. It is very easy, just open up the workspace settings and paste in the line of JSON from @Sixthdim .

This topic can be closed then, was a Visual Studio issue (Typescrip Version).

I'm closing this issue as it appears to have been solved, but am leaving this related issue here as well https://github.com/angular/angular-cli/issues/1717.


Core plugin calculator returns functions upon specific queries

The calculator plugin queries the math.js library. Sometimes, as is the case with the queries 'help' or 'random', the math.js library returns a function instead of an error, so the function code is displayed because of the .toString() in line 12 of the index.js file of the calculator plugin.
I am unsure of how to submit a pull request to fix this bug, however a possible quick fix for this could be adding the below code after line 18 of the index.js file of the calculator:
if(typeof ans === 'function') {         throw 'function returned'; }
This should not break any other results because valid results are of types number or object.
Hope this helps.

Solved in #175


Retrieve word range at hover position

I'm trying to return hover information based on the word over which the user is hovering. However, it doesn't look like there are any properties in the HoverParams argument from the onHover handler that provide a range. There only seems to be a single Position property. The @types/vscode module includes a getWordRangeAtPosition() function that makes this easy, but it doesn't look like I can import this module into a language server built with the vscode-languageserver node library. Is there an alternative way to get the word range from the onHover handler? Thanks!

@anweiss I think you have no choice but to calculate this yourself. Different languages have different definitions of what a "word" is.
See microsoft/language-server-protocol#937.

thanks @rcjsuen. yea as I was writing this out, that was dawning on me. Appreciate the clarity.


GroupBulkRead Error while extending code to 7 motors

I am extending the code for the Robotis Manipulator from 6 motors to 7 motors. When I roslaunch the manipulator_manager.launch file, the 7 motors are getting detected. But when I try to Set Base Mode in the GUI, it is giving the error message as shown below:
[GroupBulkRead::RxPacket] ID 8 result : -3002 !!!!!!!!!!
My ID numbers are from 3 to 9 (I have changed the MAX_JOINT_ID, etc. from the ManipulatorKinematicsDynamicsDefine.h file). There must be something specific about using the ID greater than 7 which is causing the error according to me. Please can you help me figure out what could be causing the error.
Note: The error statement is in the GroupBulkRead.cpp in this repository.
Update: I changed the ID numbers from 3-9 to 1-7 and now the error message is saying ID 6  Result: -3002. Hope that helps in any way possible. I'll provide whatever files/changes I've made if necessary.

Hey,
I figured out the error. The LATENCY_TIMER is defined as 4 in line 20 of PortHandlerLinux.cpp
I had initially changed the 4 to 6 because the manipulator_manager was unable to communicate with all the motors (reached only till like the 5th motor). After changing it to 6 it is able to communicate with all 7 motors but since sending messages like SetMode from the GUI to RobotisController.cpp takes time, it was only able to reach motor #6 for me. I further increased the LATENCY_TIMER to 9 (after trial and error) and was finally able to move all the motors without any communication related errors.
I hope this helps if anyone gets this sort of an error.


Create Post “hotplate-documentation”

Automatically generated by Netlify CMS

👷 Deploy Preview for hotplate-docs processing.

Name
Link

🔨 Latest commit
e8902492472cc35ca99f7b8da6e9b736cec4ccba

🔍 Latest deploy log
https://app.netlify.com/sites/hotplate-docs/deploys/62bcf7af3bedbb0009faf1da


Miscellaneous build patches

Summary of the PR
Update to Rust 2021 edition, and fix a warning.
Requirements
Before submitting your PR, please make sure you addressed the following
requirements:

[x] All commits in this PR have Signed-Off-By trailers (with
git commit -s), and the commit message has max 60 characters for the
summary and max 75 characters for each description line.
[x] All added/changed functionality has a corresponding unit/integration
test.
[x] All added/changed public-facing functionality has entries in the "Upcoming
Release" section of CHANGELOG.md (if no such section exists, please create one).
[x] Any newly added unsafe code is properly documented.

The build failure is fixed in Nuno's PR. This PR can wait.


Add node 8

Those images needs rebuild on docker hub after the marge:
pm2-docker-apline:latest
pm2-docker-apline:8
pm2-docker-apline:7

great thanks, just added v8 in auto build

Perfect. As soon as I can, I'm going to create a script in order to automatize this process. We can just grub info from the nodejs docker available tags and generates appropiate Dockerfile accoringly.
OT:
Is there any way to fix my name here?
https://keymetrics.io/2017/02/28/whats-up-with-pm2-x/


Feature: uninstall

Just for convenience at some point: have a kirby uninstall command

Super useful. Suggestion for syntax:
kirby uninstall:field

...would provide a list of all installed fields. Same would be for tags, plugins, etc.
kirby uninstall:field myfield

...woud actually delete the field files and directories.

Ok, looking at the docs, I think I got this wrong. The 'install' command is only for installing the kits, and the different components of Kirby itself, like 'core' and 'panel'. Plugins have their own command, with sub-commands, and tags, fields, blueprints, etc., have a 'make' command. In that case, there may need to be several different commands...
kirby uninstall

...would be for Kirby core/panel and its components. It could have sub-commands, such as:
kirby uninstall:core
kirby uninstall:panel
kirby uninstall:index
kirby uninstall:htaccess

In order to remain consistent with the existing syntax, plugins would probably need an extra sub-command:
kirby plugin:uninstall $plugin

And finally, for blueprints, controllers, snippets, templates, users, and tags - all of which use a 'make' command - it may be useful to have...:
kirby delete:tag $tagname

I implemented all of them on the develop branch except:
kirby uninstall:core
kirby uninstall:index
kirby uninstall:htaccess

I feel like those shouldn't exist. But the rest is awesome :)


React redux form?

What is the best way to integrate this library to the react redux form?
<Field model='user.rating' >
                        <StarRatingComponent
                            name='user.rating'
                            starCount={5}
                            value={0}
                            id='user.rating'
                        />
                    </Field>

This is what I have but it is not grabbing the data on submit of form.

I solved it with the following ....
            <Field model={this.props.model} id={this.props.model} mapProps={{customChange: props => props.change}}>
                <ReactStars
                    count={5}
                    half={false}
                    size={24}
                    color1={'#000013'}
                    color2={'#ffd700'}
                />
            </Field>


Change API url from app.close.io/api/v1 to api.close.com/api/v1

Hey Nested :)
As part of an ongoing restructuring, we've changed the base URL of the Close API from https://app.close.io/api/v1/ to https://api.close.com/api/v1/.
https://app.close.io/api/v1/ is now deprecated but will continue to be supported.
So, this PR just changes the calls of your wrapper from app.close.io/api/v1/ to api.close.com/api/v1 where applicable.
You shouldn't notice any functionality changes.

Thanks so much for this! This has slipped through the cracks so apologies for the extremely slow response.


[patch] Add missing property to OrdersCreateResponse interface

Change type:
PATCH
Changes

Add missing total_amount property to OrdersCreateResponse interface

Hello @Davies-Owen thank you for your contribution, I have just realized version 2.0.16 which should include your fix


Update PatternFly to latest milestone

Update PatternFly to latest milestone release.
Fixes https://github.com/project-koku/koku-ui/issues/1028

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1031   +/-   ##
=======================================
  Coverage   75.56%   75.56%
=======================================
  Files         227      227
  Lines        4362     4362
  Branches      812      812
=======================================
  Hits         3296     3296
  Misses        953      953
  Partials      113      113

Continue to review full report at Codecov.

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

Needed to fix the Insights frontend-components package.


New options to work with the default cache

Is your feature request related to a problem? Please describe.
A partial build failure was caused by a bad file in the default cache.
Describe the solution you'd like
Provide the following commands or something similar for documentable:

remove a specific file from the cache
clean the cache

Note I solved my immediate problem by manually removing the bad-file's entry in the cache.
However, it looks like maybe there could be some kind of .cache-ignore file which would be handy in such circumstances.

That's a good one, but something we'll probably have to fix at the cache level. Good catch, and thanks for the report.

This seems ambiguous. I see two possibilities
a) There is a bad source file, viz. a-file-with-a-compile-error.pod6, in the doc sources

the aim is to prevent this file from being added to the cache
a new version of this module, called Pod::From::Cache, traps compile errors, so bad files should not get in.
a .ignore-cache file would filter the pod-sources being added to the cache
b) A file has got into the cache that should not be extracted.
a .ignore-cache file would not return the pod content of the source in the cache.
this can be done in the $cache.pod method. But what would the return value be? Nil or an error thrown?

Done in https://github.com/finanalyst/raku-pod-from-cache.


AWS: Deploy Pod Identity Webhook

This change makes us deploy the AWS Pod Identity Webhook in the
management cluster for AWS clusters. It is comprised of:

Deploying the webhook as sidecar to the KAS for AWS clusters: Why
sidecar? Because that is both the easiest and fastest, as we avoid a
bunch of network hops
Minting a serving cert for the webhook
Minging a kubeconfig for the webhook
Setting up rbac for the kubeconfig
Setting up the MutatignWebhookconfiguration

Sample usage:
Create a ServiceAccount with sts annotations:
apiVersion: v1
imagePullSecrets:
- name: default-dockercfg-64n54
kind: ServiceAccount
metadata:
  annotations:
    eks.amazonaws.com/role-arn: arn:aws:iam::111122223333:role/s3-reader
    eks.amazonaws.com/token-expiration: "86400"
  creationTimestamp: "2022-05-05T17:19:03Z"
  name: default
  namespace: default
  resourceVersion: "46637"
  uid: 4802aef9-a7c6-4daf-9d36-c4c923ef73d1
secrets:
- name: default-token-7p5sc
- name: default-dockercfg-64n54

Notice how pods that use that SA get env vars and a token volume:
$ k get pod nginx-85b98978db-8ts7w -ojson|jq '{"env": .spec.containers[0].env, "volumeMounts": .spec.containers[0].volumeMounts, "volume": .spec.volumes[0]}'|json2yaml
env:
- name: AWS_DEFAULT_REGION
  value: ca-central-1
- name: AWS_REGION
  value: ca-central-1
- name: AWS_ROLE_ARN
  value: arn:aws:iam::111122223333:role/s3-reader
- name: AWS_WEB_IDENTITY_TOKEN_FILE
  value: /var/run/secrets/eks.amazonaws.com/serviceaccount/token
volume:
  name: aws-iam-token
  projected:
    defaultMode: 420
    sources:
    - serviceAccountToken:
        audience: openshift
        expirationSeconds: 86400
        path: token
volumeMounts:
- mountPath: /var/run/secrets/kubernetes.io/serviceaccount
  name: kube-api-access-426sh
  readOnly: true
- mountPath: /var/run/secrets/eks.amazonaws.com/serviceaccount
  name: aws-iam-token
  readOnly: true

Ref https://issues.redhat.com/browse/HOSTEDCP-205

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/hold
container needs resource requests
=== RUN   TestUpgradeControlPlane/EnsureHCPContainersHaveResourceRequests
423
    util.go:571: container aws-pod-identity-webhook in pod kube-apiserver-6bc8fb787f-2xmgx has no resource requests
424
    util.go:571: container aws-pod-identity-webhook in pod kube-apiserver-6bc8fb787f-8tqhj has no resource requests
425
    util.go:571: container aws-pod-identity-webhook in pod kube-apiserver-6bc8fb787f-kb9kg has no resource requests

container needs resource requests
Fixed in latest push
/retest
/hold cancel

/retest

/retest-required

/retest-required

@sjenning @enxebre ptal, now its passing tests

/lgtm


chore: remove circleci config

First-time contributors' checklist

[ ] I've signed Contributor License Agreement that's required for repo owners to accept my contribution.

What is changed, added, or deleted? (Required)
Since we stop CircleCI check, remove CircleCI config file.

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version)
[x] v2.0 (TiDB DM 2.0 versions)
[x] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/merge

/remove-status LGT1
/status LGT2

/merge


completes implementation of the mixed reality capture api (issue 71)

Added implementation of MrcGetSettings and MrcSetSettings methods and the relevant data contract objects.
This resolves issue #71.

Hi @davidkline-ms, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Kline (XBOX)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

This change also removes the unused RemovableStorage capability from the Universal Windows sample app.


Plot Syntax

Hi @jacintodavila , this pull request includes both the arithmetic expressions parse and the plot syntax, could you please help me review the code?
Changes in this pull request include:
The introduction of arithmetic expressions with four operators +, -, *, /.

A complete removal of the previous expressions DCG, merge the previous date parsing into the new expressions DCG.
A change in the tokenizer package to stop it from recognizing the unary operators, such as '-' and '+'. By leaving them to Logical English, it is much easier to parse dates and unary operators. Since it is originally a different repo, I have to change it into a plain directory to submit my modification.
A temporary solution to parse 2022-02-06 as dates, not as an expression with two subtractions.
The plot syntax implementation.

Dear @MigadaTang - I just pushed the pending PR for the math expressions and now it has a conflict with your second PR. I'll have a look at that as soon as we pass the presentations at ICLP. Please, have a look if you can before. Thank you

Thanks @jacintodavila , the conflict should be solved by now. And now this branch only includes the changes for the plot syntax.


Edited for TEC Context, legibility and spelling

I was debating the Title a lot... This one narrowly beat simply "Token Engineering Ethics"... I would really love it if you changed the title to whatever you like Z.
The biggest 2 leaps i made was changing "this is only possible" to "this is only practical" and adding a link to Justin from Nucypher vibing on how privacy enables transparency, most of the rest of the edits are for TEC Context, legibility, spelling and updating the Wikipedia quote.
I restrained myself from inputting my bias into TheDAO Fork...  It was hard... but I managed ;-)

Griff served as a co-editor. we cleaned this up for publication in TEC medium


chore: update chess-library

should be fixed by https://github.com/Disservin/chess-library/pull/79 hopefully


Enhancement EOM logging

We noticed with Deuterium that EOM's log messages were not making it to the logs. This provides a simple, reliable method for that to happen.

standard configparser log file format utilized
all modules do the same thing
all modules can be configured for the same logging configuration or each configured with their own

Testing is certainly required.

@TheSriram I tried the oslo.log; which seemed to work but it also tossed all the logs into stdout/stderr during tox runs so it was a bit useless in that respect. Otherwise it was pretty nice.
This is a simpler approach. The logging.conf file you showed me from Billing CDN should work with it too, just loaded a little differently.

@BenjamenMeyer oh, good to know.

:+1:


Add other time and string type binlog values to database/protocol module

For #4594 .

Pull Request Test Coverage Report for Build 2116

57 of 58 (98.28%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 59.811%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-database-protocol/shardingsphere-database-protocol-mysql/src/main/java/org/apache/shardingsphere/database/protocol/mysql/packet/binlog/row/column/value/time/MySQLTimeValueUtil.java
5
6
83.33%

Totals

Change from base Build 1084:
0.2%

Covered Lines:
12516

Relevant Lines:
20926

💛  - Coveralls


Remove default exports

:rocket: PR was released in<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>:rocket:


Update add crop image component's style to use variables from theme.css

I will combine these changes together with footer component which was also merged before.


JkImport doesn't allow file dependencies?

My build script,
@JkImport("path/to/jar.jar")
public class Build extends JkJavaBuild {
    ...
}

causes jerkar to throw an exception when it tries to compile the build classes:
java.lang.IllegalStateException: This dependency is type of org.jerkar.api.depmanagement.JkFileSystemDependency. Expecting JkModuleDependency.
	at org.jerkar.api.depmanagement.JkScopedDependency.withScopeMapping(JkScopedDependency.java:160)
	at org.jerkar.api.depmanagement.JkDependencies.withDefaultScopeMapping(JkDependencies.java:136)
	at org.jerkar.tool.Project.buildDefDependencies(Project.java:192)
	at org.jerkar.tool.Project.getBuildDefDependencyResolver(Project.java:279)
	at org.jerkar.tool.Project.compile(Project.java:84)
	at org.jerkar.tool.Project.compile(Project.java:72)
	at org.jerkar.tool.Project.execute(Project.java:121)
	at org.jerkar.tool.Main.main(Main.java:32)

The docs for JkImport annotation says it supports file dependencies, but the code seems to expect only module dependencies.

The workaround is to put your lib in build/libs/boot so you won't even need the JkImport.


Line doesn't wrap

If a string in the property is too large, it doesn't wrap to a new line

And if I manually add max-width to all children of the JSONPretty, the padding breaks

How can I wrap lines?

I got the same issue,  @C-Coretex do you have an example of wrap long text?


Start using syrupy for testing with snapshots

Proposed change

SSIA
Additional information

This PR fixes or closes issue: fixes #

Checklist

[ ] I have updated the documentation if needed.
[ ] I have updated the tests if needed.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (27258b2) 100.00% compared to head (d69cf1d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #438   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          141       141
  Branches        18        18
=========================================
  Hits           141       141

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


Removes campaignActivity endpoint and Mongoose

What's this PR do?
Picking up from #1101 -- this PR removes code deprecated by https://github.com/DoSomething/gambit-conversations/pull/449, and it's quite a slice of :pizza:
How should this be reviewed?
Sanity check all endpoints used in production work as expected:

GET /broadcasts (only used by Gambit Admin)
GET /broadcasts/:id- used in prod for broadcast message requests
GET /contentfulEntries (only used by Gambit Admin/Slack)
GET /contentfulEntries/:id (only used for internal spot checking)
GET /defaultTopicTriggers - used in prod to load Rivescript
GET /campaign/:id  - used in prod for signup message requests
GET /topics/:id- used in prod for member message requests

Any background context you want to provide?
This PR lets us safely stop the Mongo database - but is blocked until https://github.com/DoSomething/gambit-conversations/pull/449 is deployed to production.
Relevant tickets

https://www.pivotaltracker.com/story/show/162330044
https://www.pivotaltracker.com/story/show/161494958

Checklist

[x] Documentation added for new features/changed endpoints.
[x] Tested on staging.

Small typo in the /campaign[s]/:id test step.
I tested this route and I'm getting a 500 error.

Route: https://ds-mdata-responder-staging.herokuapp.com/v1/campaigns/5taFqUpJkck6y4CMCAs6is?apiKey=secretstuff.
Error:{
  "error": {
    "code": 500,
    "message": "phoenix campaignId=undefined error=Internal Server Error"
  }
}

Campaign id's are numeric -- that 500 error is from Phoenix per https://dosomething.slack.com/archives/C02BBP0CU/p1543853059003700?thread_ts=1543823525.001700&cid=C02BBP0CU


Adding IsNull and IsNotNull.

Coverage remained the same at 100.0% when pulling adbf67657d0c85cc47d53905d8293ad280588818 on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.

Coverage remained the same at 100.0% when pulling adbf67657d0c85cc47d53905d8293ad280588818 on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.

Coverage remained the same at 100.0% when pulling adbf67657d0c85cc47d53905d8293ad280588818 on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.

Coverage decreased (-0.2%) to 99.775% when pulling e01b169b9fa2f6a4656dfc11b5618bbae112a42d on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.

Coverage remained the same at 100.0% when pulling 94ef7b9d5f45e5a86eac3604619be51a542ae285 on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.

Coverage remained the same at 100.0% when pulling 94ef7b9d5f45e5a86eac3604619be51a542ae285 on isnull-isnotnull into 4ba4128d3d103633be619692be30f2b1a9eb6b30 on master.


bugdown: Replace raw unicode literal with non-raw.

Raw unicode literals are disallowed in python 3.

Automated message from Dropbox CLA bot
@sharmaeklavya2, it looks like you've already signed the Dropbox CLA. Thanks!

This was included in #1164.

Yep, thanks for closing!
-Tim Abbott (mobile)
On Jul 2, 2016 12:08 PM, "Eklavya Sharma"<EMAIL_ADDRESS>wrote:

This was included in #1164 https://github.com/zulip/zulip/pull/1164.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zulip/zulip/pull/1160#issuecomment-230117202, or mute
the thread
https://github.com/notifications/unsubscribe/ACnm2tRb2dLCXwBe4eqvC8Ina3ElmBnvks5qRrcUgaJpZM4JDkAh
.


Add logging for migration process

The migration process is long running. It would be
good to know how many objects have been migrated and how
close it is to finished.
Log how many objects will be converted

Coverage decreased (-29.2%) to 62.697% when pulling d7aec86c0dbf34720423b371912aac25283d08b1 on logfile into dffa6cf2de92e5cdfe8407db9630ae8fa5dbada6 on develop.


[BACKLOG-15105] Extending PluginRegistry to allow additional Plugin m…

…appings after plugin construction.

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -f core/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/plugins/PluginRegistry.java
  core/src/org/pentaho/di/core/plugins/SupplementalPlugin.java
  core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.Const

Branch Change:  + .20%
Complexity Change:  + .27%

org.pentaho.di.core.plugins.Plugin

Branch Change:  + 8.33%
Complexity Change:  + 6.67%
Instruction Change:  + 8.39%
Line Change:  + 4.76%
Method Change:  + 5.13%

org.pentaho.di.core.plugins.PluginRegistry

Branch Change:  + 2.79%
Complexity Change:  + 3.78%
Instruction Change:  + 7.15%
Line Change:  + 5.24%
Method Change:  + 7.50%

org.pentaho.di.core.plugins.SupplementalPlugin

Branch New Coverage: 50.00%
Complexity New Coverage: 60.00%
Instruction New Coverage: 76.12%
Line New Coverage: 71.43%
Method New Coverage: 75.00%

org.pentaho.di.core.row.RowBuffer

Complexity Change:  + 16.67%
Instruction Change:  + 10.00%
Line Change:  + 8.33%
Method Change:  + 16.67%

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java. Found 2015. Was expecting 2017

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/core/src/org/pentaho/di/core/plugins/PluginRegistry.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 429 Column: 164

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 434 Column: 7

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 469 Column: 15

/home/buildguy/project/core/src/org/pentaho/di/core/plugins/SupplementalPlugin.java

:heavy_exclamation_mark: Unused import - java.lang.annotation.Annotation. at Line: 29 Column: 8

:heavy_exclamation_mark: Unused import - java.net.URL. at Line: 30 Column: 8

:heavy_exclamation_mark: Unused import - java.util.List. at Line: 33 Column: 8

:heavy_exclamation_mark: ',' is not followed by whitespace. at Line: 48 Column: 31

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 56 Column: 7

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -f core/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/plugins/PluginRegistry.java
  core/src/org/pentaho/di/core/plugins/SupplementalPlugin.java
  core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.Const

Branch Change:  + .20%
Complexity Change:  + .27%

org.pentaho.di.core.plugins.Plugin

Branch Change:  + 8.33%
Complexity Change:  + 6.67%
Instruction Change:  + 8.39%
Line Change:  + 4.76%
Method Change:  + 5.13%

org.pentaho.di.core.plugins.PluginRegistry

Branch Change:  + 2.79%
Complexity Change:  + 3.78%
Instruction Change:  + 7.15%
Line Change:  + 5.24%
Method Change:  + 7.50%

org.pentaho.di.core.plugins.SupplementalPlugin

Branch New Coverage: 50.00%
Complexity New Coverage: 60.00%
Instruction New Coverage: 76.12%
Line New Coverage: 71.43%
Method New Coverage: 75.00%

org.pentaho.di.core.row.RowBuffer

Complexity Change:  + 16.67%
Instruction Change:  + 10.00%
Line Change:  + 8.33%
Method Change:  + 16.67%

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java. Found 2015. Was expecting 2017

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/core/src/org/pentaho/di/core/plugins/PluginRegistry.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 440 Column: 164

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 445 Column: 7

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 480 Column: 15

/home/buildguy/project/core/src/org/pentaho/di/core/plugins/SupplementalPlugin.java

:heavy_exclamation_mark: Unused import - java.lang.annotation.Annotation. at Line: 29 Column: 8

:heavy_exclamation_mark: Unused import - java.net.URL. at Line: 30 Column: 8

:heavy_exclamation_mark: Unused import - java.util.List. at Line: 33 Column: 8

:heavy_exclamation_mark: ',' is not followed by whitespace. at Line: 48 Column: 31

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 56 Column: 7

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f core/build.xml clean-all resolve jacoco && ant -f core/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  core/src/org/pentaho/di/core/plugins/PluginRegistry.java
  core/src/org/pentaho/di/core/plugins/SupplementalPlugin.java
  core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.core.Const

Branch Change:  + .20%
Complexity Change:  + .27%

org.pentaho.di.core.plugins.Plugin

Branch Change:  + 8.33%
Complexity Change:  + 6.67%
Instruction Change:  + 8.39%
Line Change:  + 4.76%
Method Change:  + 5.13%

org.pentaho.di.core.plugins.PluginRegistry

Branch Change:  + 2.79%
Complexity Change:  + 3.78%
Instruction Change:  + 7.15%
Line Change:  + 5.35%
Method Change:  + 7.50%

org.pentaho.di.core.plugins.SupplementalPlugin

Branch New Coverage: 50.00%
Complexity New Coverage: 60.00%
Instruction New Coverage: 76.12%
Line New Coverage: 71.43%
Method New Coverage: 75.00%

org.pentaho.di.core.row.RowBuffer

Complexity Change:  + 16.67%
Instruction Change:  + 10.00%
Line Change:  + 8.33%
Method Change:  + 16.67%

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on core/test-src/org/pentaho/di/core/plugins/PluginRegistryUnitTest.java. Found 2015. Was expecting 2017

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/core/src/org/pentaho/di/core/plugins/PluginRegistry.java

:heavy_exclamation_mark: 'if' is not followed by whitespace. at Line: 481 Column: 15

Closing to make some improvements


Reference Public ECR Base Images in Docker Images

With regards to recent incidents and pending RFCs, we should switch our references to docker images to their ECR Public equivalents to reduce the likelihood of rate limiting.
In addition, pulling from AWS -> AWS should/could be slightly quicker?
Some questions remain around if we should authenticate to gain higher pulling rates, however, that is a question to pose to the build agency team and whether they could do that for us.
https://docs.aws.amazon.com/AmazonECR/latest/public/public-registries.html#public-registry-auth
https://docs.aws.amazon.com/AmazonECR/latest/public/public-service-quotas.html

Work to auto login to ECR is complete: https://github.com/SEEK-Jobs/build-agent-images/pull/174


freedv: clean up and add pre‐release version

use finalAttrs pattern
remove uses of with lib;
re‐format according to RFC 166
add pre‐release package version 2.0.0-20241018 (as per #351264)

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@danwdart feel free to try this out.

I might have underestimated the complexity of this task. The pre‐release does not yet support using an external Opus and RADE. This will probably require a patch for CMakeLists.txt, or waiting for upstream to fix it.

Thanks - and oh I see - I thought by that you meant it's without or with its own inbuilt version rather than fail to build with error: could not find git for clone of build_rade. Fair enough!

Thanks - and oh I see - I thought by that you meant it's without or with its own inbuilt version rather than fail to build with error: could not find git for clone of build_rade. Fair enough!

Fetching additional repositories during the build is difficult because of the sandbox.
The best way in my opinion is to have these dependencies as separate packages, like the LPCNet fork used in FreeDV.

Hm, not sure I'm enough of an expert in CMakeLists... bit confused about it.

Has anybody had any luck on that one?


Collate images for Civic Days

Add images here

@akkisagiraju can you use the images in the folder linked above. We can then ask the team to review.


Recover Ruby 2.2 code analysis using TargetRubyVersion: 2.2

Follow up https://github.com/rubocop/rubocop/pull/10632#issuecomment-1125909580.
Reverts part of #6766, #7026, and #7030.
Only the Ruby version (2.2) to runtime should have been dropped, not code analysis.
This PR makes Ruby 2.2 code analysis with TargetRubyVersion: 2.2.
It aims to solve essentially the same problem as #10626, #10632, and #10640.
Previously, there was the following default enforced style when_needed for Style/FrozenStringLiteralComment cop.
# @example EnforcedStyle: when_needed (default)
#   # The `when_needed` style will add the frozen string literal
#   # to files only when the `TargetRubyVersion` is set to 2.3+.
#   # bad
#   module Foo
#     # ...
#   end
#
#   # good
#   # frozen_string_literal: true
#
#   module Foo
#     # ...
#   end

This PR does not restore that option, but sets the minimum_target_ruby_version 2.3 to make always (default) apply by default. It is a simple solution that does not handle frozen literal magic comment added in Ruby 2.3 when TargetRubyVersion is Ruby 2.2 or lower.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Not sure if we need to go back so far, as I doubt there are many projects that still target such an old Ruby release. I'll leave this up to you to decide. Personally I was happy when we got back to 2.5. :D

I found the following backlink in PR #10632 that recovered Ruby 2.4 code analysis.
https://github.com/brianmario/mysql2/pull/1256
So, there seems to be use case with gems that support older Ruby versions. Perhaps RuboCop's analysis can recover up to Ruby 1.9 code. I'm trying it, but I'm not sure if it's the best 😅


alignment with SMART v2

Here is the jira ticket with the high-level explanation of their changes in v2
https://jira.hl7.org/browse/FHIR-30578

Here is their draft spec
http://build.fhir.org/ig/HL7/smart-app-launch/

Here is their draft spec
http://build.fhir.org/ig/HL7/smart-app-launch/

I checked the new version and found no conflicts between SMART on FHIR and the current IUA version.

We reviewed the items in February 2021 and believe that SMART on FHIR and IUA can still work side by side.


Decode csv failed to decode csv file with commas in cell properly.

Environment info
Operating System: Ubuntu 16.04
Installed CUDA and cuDNN:
Tensorflow is build from source.
Tensorflow version: 1.0.0-rc2
Problem
import tensorflow as tf
import os
filename_queue = tf.train.string_input_producer([os.path.join(os.getcwd(),"train.csv")])
reader = tf.TextLineReader(skip_header_lines=1)
key, value = reader.read(filename_queue)
decoded = tf.decode_csv(value, record_defaults = [[0.0], [0.0], [0], [""],[""], [0.0], [0.0], [0.0], [""], [0.0], [""], [""]])
passenger_id, survived, pclass, name, sex, age, sibsp, parch, ticket, fare, cabin, embarked = tf.train.shuffle_batch(decoded, batch_size=9, capacity=450, min_after_dequeue=9)"

The data to import:
PassengerId,Survived,Pclass,Name,Sex,Age,SibSp,Parch,Ticket,Fare,Cabin,Embarked
1,0,3,"Braund, Mr. Owen Harris",male,22,1,0,A/5 21171,7.25,,S
2,1,1,"Cumings, Mrs. John Bradley (Florence Briggs Thayer)",female,38,1,0,PC 17599,71.2833,C85,C
3,1,3,"Heikkinen, Miss. Laina",female,26,0,0,STON/O2. 3101282,7.925,,S
4,1,1,"Futrelle, Mrs. Jacques Heath (Lily May Peel)",female,35,1,0,113803,53.1,C123,S
5,0,3,"Allen, Mr. William Henry",male,35,0,0,373450,8.05,,S
6,0,3,"Moran, Mr. James",male,,0,0,330877,8.4583,,Q

The column "name" contains comma. The tenforflow reports invalid arguments.

I changed your example to actually run

import tensorflow as tf
import os
with tf.device("/cpu:0"):

  filename_queue = tf.train.string_input_producer([os.path.join(os.getcwd(),"train.csv")])
  reader = tf.TextLineReader(skip_header_lines=1)
  key, value = reader.read(filename_queue)
  decoded = tf.decode_csv(value, record_defaults = [[0.0], [0.0], [0], [""],[""], [0.0], [0.0], [0.0], [""], [0.0], [""], [""]])
  passenger_id, survived, pclass, name, sex, age, sibsp, parch, ticket, fare, cabin, embarked = tf.train.shuffle_batch(decoded, batch_size=9, capacity=450, min_after_dequeue=9)
  with tf.Session() as session:
      coordinator = tf.train.Coordinator()
      tf.train.start_queue_runners(session, coord=coordinator)

      print (session.run(name))

      coordinator.request_stop()
      coordinator.join()

This worked fine and produced the following output.
[b'Futrelle, Mrs. Jacques Heath (Lily May Peel)'
 b'Futrelle, Mrs. Jacques Heath (Lily May Peel)' b'Heikkinen, Miss. Laina'
 b'Cumings, Mrs. John Bradley (Florence Briggs Thayer)'
 b'Cumings, Mrs. John Bradley (Florence Briggs Thayer)'
 b'Heikkinen, Miss. Laina' b'Allen, Mr. William Henry' b'Moran, Mr. James'
 b'Cumings, Mrs. John Bradley (Florence Briggs Thayer)']

This stackoverflow might be helpful.
http://stackoverflow.com/questions/42156479/can-tensorflow-handle-categorical-features-with-multiple-inputs-within-one-colum/42168658#42168658
Barring a reproducible test case, I will need to close the issue. Thanks!

Thanks.


Refresh

Re-rendered with conda-smithy 2.4.5.
Update license metadata to include version.
Package the license file.
Switch to sha256.
Use skip to avoid unsupported Python versions.
Bump build number to 2.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


[1783] Add provider enrichments status sidebar

Context
Moving provider enrichments to rails 🎉
Changes proposed in this pull request
Add status panel for provider enrichments
Guidance to review
TODO: Add the publish button once we have an endpoint on the backend to call
After

 [S] Provider enrichments sidebar (move to Rails)


Change table width

New feature:
Looking for docs in https://evotec.xyz/docs/adding-tables-with-built-in-styles-managing-borders/, havent found a way to change width of table on/after creation. If doc margins change, table dont fill all documents width.
Is any way to do this using OfficeIMO ?
I will try to extend it by WordDocument.PublicMethods.cs, AddTable, if i dont miss something.

Hi, I don't believe it's added yet. I guess one would add properties to WordTable class. There's already _tableProperties  property, which would just need some expansion. I guess we also need AutoFit/Fixed or whatever that is called in Word to be possible for user to choose.

https://docs.microsoft.com/en-us/dotnet/api/documentformat.openxml.wordprocessing.tablewidth?view=openxml-2.8.1
https://docs.microsoft.com/en-us/office/open-xml/working-with-wordprocessingml-tables
http://officeopenxml.com/WPtableLayout.php

So AddTable can be extended, but WordTable class needs updates so that it's not just basic setting that you set when adding table but something you can do on Load as well.

Some work in progress: https://github.com/EvotecIT/OfficeIMO/pull/30


Version Packages

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to master, this PR will be updated.
Releases
<EMAIL_ADDRESS>Patch Changes

#970 16b9a71 Thanks @roginfarrer! - Updated addFileScope to work with CommonJS files

Can we get this soon? It fixes a pretty big blocker for us!


Fix HelperROT.msg

Description

HelperROT.msg had the wrong units datatype.

Verification

Description of ROT https://documentation.spire.com/ais-fundamentals/rate-of-turn-rot/ from the original PR https://github.com/UBCSailbot/custom_interfaces/pull/13

Resources

@SPDonaghy fyi

Just saw this, thanks Henry!

Interesting, nice catch Henry! Looks good to me. I assume this is specific to the AIS device we're using right?

Interesting, nice catch Henry! Looks good to me. I assume this is specific to the AIS device we're using right?

This is part of the  AIS standard: https://www.navcen.uscg.gov/ais-class-a-reports


[d3d9] TrackMania Forever games are crashing during startup

As of 2f2342e, both TrackMania Forever games are crashing when I try to start them.
The game displays this message right before crashing:

Replaying the trace seems to work fine on Windows, but it crashes when I try to replay it with Wine during the level loading screen.
The game itself crashes even on Windows when I use DXVK/D9VK's d3d9.dll.
Software information
TrackMania Nations Forever and TrackMania United Forever
The former is the free version of the game. This workaround might be necessary when using Windows 8 or 10 to get the game running.
System information

GPU: RX 580 4GB
Driver: RADV ACO
Wine version: TkG 5.0-rc1
DXVK version: 3cdae3a

Apitrace file(s)

Same trace that can be found in Joshua-Ashton/d9vk#285.

Log files

TmForever_d3d9.log

Before that we would return INVALIDCALL instead of NOTSUPPORTED there, I will double check my tests I did to see if there was some edge case I missed here.

Unfortunately, that didn't fix the problem. I'm getting the exact same message before crashing.

You're testing with 3b119c0b (https://git.froggi.es/doitsujin/dxvk/-/jobs/1047) correct?

Yes, I tested that and I'm still getting the same error message.

Can you try now?

The game still doesn't work for me... I just get a white screen and some fairly broken text occasionaly, but D9VK renders the exact same as native D3D9 does for me on AMD and Nvidia...

That did the trick. Works fine now.


add ubuntu20.04, centos8 to rapids docker axes

Summary
This PR adds ubuntu-20.04, centos-8 support to our docker images

Awaiting https://github.com/rapidsai/gpuci-build-environment/pull/146 before this can be merged.

Rebased on branch-0.17


settings background color is too short and shows html white bg

This has been fixed in this PR: #675 So it can be closed, right?

@Harjot1Singh @bhajneet


Please consider a trivial example of how to use this gem

Some examples on how this gem could be used would be extremely helpful. I would be happy to write them myself if someone can take a couple of minutes to familiarize me with how this gem is used.

@pawel2105 offsite_payments no longer depends on active_merchant, so that issue should be fixed now.

thanks @madmike your example helped me in integrating payu_in.

@wvanbergen offsite_payments requires actionpack, and also actionpack, so yeah it still depends on a lot of Rails libraries.

@madmike @lulalala As you can see the error happens on ActiveMerchant::PostData, i.e. ActiveMerchant not OffsitePayments. The link between these two libraries was severed a while ago, so if you see the class_attribute error while you are not using ActiveMerchant yourself directly, you're probably on an old version of OffsitePayments.
Both libraries still depend on ActiveSupport, and OffsitePayments depends on ActionPack as well.

I generated form using payment_service_for and it successfully submits to payu_in gateway. Now I got a different requirement. I want to perform below validation on "submit" button

want to validate that item is still available to buy
want to save booking if validate success.
After these two actions, automatically my form should submit to integration service.

Thus instead of generating html form using payment_service_for, is it possible to redirect_to gateway with params from rails controller itself ? There should be a method to achieve this task from the ruby code itself instead of html form post

I wrote a small tutorial on my ongoing experience on building the integration: https://coderwall.com/p/dqed4a/how-to-create-a-payment-gateway-for-offsite_payments-active_merchant, if possible PLEASE tell me to fix things, most of my write up are wild enough guesses.

This gem looks so promising - shame there is such a lack of information. Would love to use this for Sagepay but there doesn't seem to be any useful documentation at all. Even 2 years after the initial request there's nothing.


updates Edge page

cc @cvan

Using template setup from Firefox pages.
@cvan if you could give me quick guidance on template updates.
Still todo:

Download link is not needed for this since it's included in Edge creators update.
Unclear how to add "info" header
Known Issues section needs to be hidden.
Latest versions section doesn't really apply, should also be hidden.
Add Windows Mixed Reality headset page.

@caseyyee FYI: made some adjustments in c4fcfcc to address your comments above. thanks!


Fails to install with zef

[File::Find] ===SORRY!=== Error while compiling /home/jmerelo/.zef/store/File-Find.git/075720bde467460892fd3e326e139a9a926cbc8a/t/01-file-find.t
[File::Find] Could not find File::Find in:
[File::Find]     file#/home/jmerelo/.zef/store/File-Find.git/075720bde467460892fd3e326e139a9a926cbc8a
[File::Find]     inst#/home/jmerelo/.raku
[File::Find]     inst#/home/jmerelo/.rakudobrew/moar-2020.08.2/install/share/perl6/site
[File::Find]     inst#/home/jmerelo/.rakudobrew/moar-2020.08.2/install/share/perl6/vendor
[File::Find]     inst#/home/jmerelo/.rakudobrew/moar-2020.08.2/install/share/perl6/core
[File::Find]     ap#
[File::Find]     nqp#
[File::Find]     perl5#
[File::Find] at /home/jmerelo/.zef/store/File-Find.git/075720bde467460892fd3e326e139a9a926cbc8a/t/01-file-find.t:3
===> Testing [FAIL]: File::Find:ver<0.1>
Aborting due to test failure: File::Find:ver<0.1> (use --force-test to override)

Same issue, using raku 2020.10 (MoarVM).


Committee details not aligned properly

BS cols are used instead of table or rows (with cols).
Example:
https://www.inter-actief.utwente.nl/commissies/260/international-business-course-commissie-2017/

Probably better to use a horizontal definition list here (

thing

explanation

) See http://getbootstrap.com/css/#type-lists


Archiving - Project conversion to BioStudies failed

When I create the project / submission in the new way (create an empty project first then add a submission and update the project afterwards) and don't add publications to the project the archiving will fail.

PR: https://github.com/ebi-ait/ingest-archiver/pull/92

@MightyAx will review today

Tested in staging - hopefully moving to production today.


[QtWebEngine] - Fix build failure

This commit fixes an issue where the build fails on X86_64 Windows 10 Pro with a message similar to: 'PATCHES no such argument' printed to the console output.
It also makes a few small changes for sanity. Build trees for QtWebEngine come in at between 40GB-50GB and the minimum recommended RAM size for building Chromium according to google is 8GB. The port shouldn't be building debug by default due to it's size and the fact that you likely only want debug if you are a Qt developer or are working on vcpkg. The options "" is intentionally left blank due to the fact that it allows the string -webengine-proprietary-codecs to be easily be added in case you have licences for that kind of thing. Without that option enabled, many html5 videos do not work in QtWebEngine. Maybe adding a configuration switch for proprietary-codecs is something to think about for the future?
Enjoy!
From the build output here:
`Running configuration tests...
Checking for architecture supported... yes
Checking for bison... H:/vcpkg/installed/x64-windows/tools/qt5/mkspecs/features/qt_configure.prf:413: Unexpected return value from test 'ensurePathEnv': H:/vcpkg/buildtrees/qt5-webengine/src/gnuwin32/bin/bison.exe.
yes
Checking for submodule initialized... yes
Checking for build path without whitespace... yes
Checking for platform supported...
Checking for gperf... H:/vcpkg/installed/x64-windows/tools/qt5/mkspecs/features/qt_configure.prf:413: Unexpected return value from test 'ensurePathEnv': H:/vcpkg/buildtrees/qt5-webengine/src/gnuwin32/bin/gperf.exe.
yes
Checking for flex... H:/vcpkg/installed/x64-windows/tools/qt5/mkspecs/features/qt_configure.prf:413: Unexpected return value from test 'ensurePathEnv': H:/vcpkg/buildtrees/qt5-webengine/src/gnuwin32/bin/flex.exe.
yes
Checking for python2... H:/vcpkg/downloads/tools/python/python-2.7.16-x64/python.exe
Checking for 64bit compiler... yes
Checking for winversion... yes
Checking for jumbo build merge limit... 0
Checking for d-bus... no
Checking for fontconfig... no
Checking for libdrm... no
Checking for system ninja... yes
Checking for xcomposite... no
Checking for xcursor... no
Checking for xi... no
Checking for xtst... no
Checking for embedded build... no
Checking for node.js... no
Done running configuration tests.
Configure summary:
Qt WebEngine Build Tools:
Use System Ninja ....................... yes
Use System Gn .......................... no
Jumbo Build Merge Limit ................ no
Developer build ........................ no
Sanitizer .............................. no
Qt WebEngineCore:
Embedded build ......................... no
Full debug information ................. no
Pepper Plugins ......................... yes
Printing and PDF ....................... yes
Proprietary Codecs ..................... yes
Spellchecker ........................... yes
Native Spellchecker .................... no
WebRTC ................................. yes
PipeWire over GIO ...................... no
Geolocation ............................ yes
WebChannel support ..................... yes
Kerberos Authentication ................ yes
Extensions ............................. yes
Node.js ................................ no
Qt WebEngineQml:
Support Qt WebEngine Qml ............... yes
UI Delegates ........................... yes
Test Support ........................... no
Qt WebEngineWidgets:
Support Qt WebEngine Widgets ........... yes
Qt PDF:
Support V8 ............................. no
Support XFA ............................ no
Support XFA-BMP ........................ no
Support XFA-GIF ........................ no
Support XFA-PNG ........................ no
Support XFA-TIFF ....................... no
Qt PDF Widgets:
Support Qt PDF Widgets ................. yes
WARNING: Building without node.js will disable some features of QtWebEngine DevTools.
Qt is now configured for building. Just run 'nmake'.
Once everything is built, you must run 'nmake install'.
Qt will be installed into 'H:\vcpkg\installed\x64-windows'.
Prior to reconfiguration, make sure you remove any leftovers from
the previous build.`
Also noticed for the future:

WARNING: Building without node.js will disable some features of QtWebEngine DevTools.

Feel free to edit this pull request away to suit.

Ping @Neumann-A for review this PR.

@JackBoosY
Please use the @ tag if you want to talk directly. 👍
I don't know if you were trying to talk directly to me but I have tried to answer you below. I had to guess.

Please bump the port version. See documentation.
Also please run command vcpkg x-add-version --all then commit the changes.

@JackBoosY
Please feel free to do this. The merge is a bug fix not a new version. Therefore it's in your court if you want to do it. The merge request stands, although I logged in to this site just now to delete the tree it is in. It will remain available for at least another month. Take it or leave it. I personally don't stand to gain anything from it as I maintain my own fork. I won't hold it against anyone if it doesn't get merged. I'm past caring at this point.
I honestly can't stand Github any more and only logged in the other day to push this merge. I was in a good mood and wanted to see if this site had got any worse than I remember. Unfortunately, it has. The whole site now gives me a migraine. And because of that, it is unlikely I will be around here again. Sorry.
There is something else to add though.
Someone (that's not me) should add to the port something which stops vcpkg building the QtWebEngine port before the configure stage if vcpkg's root folder isn't at the very tip of a drive. Paths such as: C:\12345\abcde\vcpkg\ are too long and will not work. The build will fail in that case when building on Windows 10. Even if you have NTFS long path names set. A short path must be used, one such as: C:\a\vcpkg\
I know that the port prints a warning to the console output if the path is too long but the port should really go further and stop the build entirely if it detects that the path is too long because the build will definitely fail. Not only that, it will wait until at least 3 1/2 hrs have passed to cryptically announce it's failure without it being obvious that ninja could not find a file because the path was too long.

Closing. Maintainers here seem to ignore discourse and prefer to ship a broken port instead of accepting a merge request.
I wouldn't like to speculate if this is cultural or not but I won't be back and I won't be offering merges at all again in the future.

@heydojo  Sorry for that, #16770 will fix this.


New package: Posit.rig version 0.5.3

Pull request has been created with Komac v1.6.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

This package is expecting arguments:
RIG -- The R Installation Manager 0.5.3
NAME
    rig - manage R installations

DESCRIPTION
    rig manages your R installations, on macOS, Windows, and Linux. It can
    install and set up multiple versions of R, and make sure that they work
    together.

    rig is currently experimental and is a work in progress. Feedback is much
    appreciated. See https://github.com/r-lib/rig for bug reports.
...

Also, the Start menu shortcut appears to do nothing - a window pops up and immediately closes. Is that expected?

Hi @stephengillie, thanks for your review!

This package is expecting arguments:

It is not clear to me what you mean by this point of yours.
Do I need to update the manifest to state something?

Also, the Start menu shortcut appears to do nothing - a window pops up and immediately closes. Is that expected?

This is unrelated behavior to winget, but as you point out, I don't think it's ideal.
I created an issue r-lib/rig#160.
In other words, this is the expected behavior for this version.

Hi @stephengillie, thanks for your review!

This package is expecting arguments:

It is not clear to me what you mean by this point of yours. Do I need to update the manifest to state something?

The package might need a Command: or similar field added to the manifest in the future. I haven't had an opportunity to learn the exact fix for this situation, but felt that notes like these might help to explain why the error label was removed or waivered. I didn't mean to disturb or insult, and can avoid posting these in your PRs if you would like.

Also, the Start menu shortcut appears to do nothing - a window pops up and immediately closes. Is that expected?

This is unrelated behavior to winget, but as you point out, I don't think it's ideal. I created an issue r-lib/rig#160. In other words, this is the expected behavior for this version.

Thank you for confirming the expected behavior.

@wingetbot waivers Add Validation-Executable-Error

The package might need a Command: or similar field added to the manifest in the future. I haven't had an opportunity to learn the exact fix for this situation, but felt that notes like these might help to explain why the error label was removed or waivered.

That makes sense.
I just wanted to know what needed to be changed and I don't care. Thanks for taking a look at this!


pyinstaller fatal error

Hello,
I am trying to create an exe with this project via pyinstaller, but I am getting an error:
-Traceback (most recent call last):
File "main.py", line 8, in
File "PyInstaller\loader\pyimod03_importers.py", line 546, in exec_module
File "scrcpy_init_.py", line 6, in
File "PyInstaller\loader\pyimod03_importers.py", line 546, in exec_module
File "scrcpy\core.py", line 12, in
File "PyInstaller\loader\pyimod03_importers.py", line 546, in exec_module
File "av_init_.py", line 9, in
ImportError: DLL load failed while importing _core: The specified procedure could not be found.
[17436] Failed to execute script 'main' due to unhandled exception!
I have been working on it for a couple days now, I believe it is from the av import:
-from av.codec import CodecContext
Not sure what I am doing wrong, any help would be appreciated.
Thanks!

This is a pyav related question. After searching on google, I will recommend you to check the FFmpeg dependency.
Please make sure the directory containing your FFmpeg DLLs is in the PATH.
Here is pyav's issue page: https://github.com/PyAV-Org/PyAV/issues?q=ImportError%3A+DLL+load+failed


Error appears when drop an STS file (.3ds)

I'm a user of igor 9 (Version: <IP_ADDRESS>)
I have two problems which may be related with each other.

Error message
I installed SIDAM into igor without problem.
When I drop an STS data (*.3ds), it shows an error saying
"While executing a wave write, the following error occurred: Index out of range for wave "free".

Do you have any idea what is this?
It appears even I open the data from menubar->Load Data.
It didn't appear for simple topograph data (*.sxm)

According to your Gif movie of "Spectrum viewer", one can check point spectrum even if the cursor is on topograph.
I can see point spectrum when the cursor is on dI/dV map, but cannot see it on topograph.
Is there anything I need to do to connect these two images such as sync?

It is hard to diagnose what's happening only from the given information. If you can tell me the line number causing the problem, it may help to diagnose the problem. To pin down the line number, you need to turn on the debugger of Igor Pro and select SIDAM > Developer > Show SIDAM Procedures.
The second one is a separate issue. Please raise another issue with an appropriate title so that people who have the same question can refer to it later.

See #45 for the answer to the second question.

Hello, I'm a user of SIDAM (and also a colleague of YuF12345).
I've replicated the error in 1. with debugging turned on, and it seems that the "free" wave that is throwing an error is on line 361 of the ipf "SIDAM_LoadData_Nanonis_3ds.ipf" I stepped through the function with the debugger, and it seems that the problem is that it is trying to assign refw[1,]=avgw[p-1], but avgw has 0 points at the time of assignment (giving the out of bounds error). I guess the problem is actually on line 358 where the avgw wave reference wave is created/assigned. Unfortunately I wasn't able to figure out more than that due to being a relative novice in igor programming.
Hopefully this helps debug the issue.

Perhaps, did you take the data only for the forward sweep? Can you try noavg=1 in SIDAMLoadData()?
c.f. https://yuksk.github.io/SIDAM/commands/SIDAMLoadData/

I tried setting noavg=1, and it works perfectly. We were not collecting the backward sweep data. Thank you!
From: Yuhki Kohsaka @.>
Sent: 24 January 2023 20:40
To: yuksk/SIDAM @.>
Cc: Barnaby Smith @.>; Comment @.>
Subject: Re: [yuksk/SIDAM] Error appears when drop an STS file (.3ds) (Issue #42)
Perhaps, did you take the data only for the forward sweep? Can you try noavg=1 in SIDAMLoadData()?
c.f. https://yuksk.github.io/SIDAM/commands/SIDAMLoadData/
—
Reply to this email directly, view it on GitHubhttps://github.com/yuksk/SIDAM/issues/42#issuecomment-1401798607, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A5NX6MR3XRG3BAHUCT5JRPDWT65QDANCNFSM6AAAAAAUAWHHFQ.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>


Update localforge version to 1.5.6

In private mode of Safari 11 method call localforage.getItem('item', callback(err, data){...})
never calls its callback.
Most recent version of localforage works fine.

@twirl


Registry: FIR support

FIR "Transfer" event not parsed yet. No factor at this point as there's not a single "Transfer" event logged in FIR contract (would be nice to fix this in the near future).
Apart from this Registry now fully supports querying by FID, Address and Username:
get_fid_by_address()
get_fid_by_username()
get_address_by_fid()
get_address_by_username()
get_username_by_fid()
get_username_by_address()

wow this is a lot- nice!


DomainError(ConversionFailure("record identifier", "assimilated record identifier"))")

I am not able to create new users with mutations in the graphql playground with nexus@next and I got the following error message:
Invalid prisma.user.create() invocation in\n/node_modules/nexus-prisma/src/builder.ts:315:20\n\nError occurred during query execution:\nInterpretationError(\"Error for binding \\'2\\': DomainError(ConversionFailure(\\\"record identifier\\\", \\\"assimilated record identifier\\\"))\")
My schema looks like this:
model Post {
  id String @id
}
model User {
  id        String   @id
  birthDate DateTime
  email     String   @unique
  posts     Post
}

GraphQL Mutation
mutation {
  createOneUser(data: {
    id: "test"
    email<EMAIL_ADDRESS>    birthDate: "2015-11-22T13:57:31.123Z"
    posts: {
    create: {
      	id: "postId"
    	}
    }
  }) {
  	id
       email
  }
}

I have the same mistake with the model:
model Contract {
  id Int @id @default(autoincrement())
  date DateTime @unique
  skills Skill[] @relation(references: [id])
}

model Skill {
  id Int @id @default(autoincrement())
  name String
  contract Contract @relation(fields: [contractId], references: [id])
  contractId Int
}

With the query :
prisma.contract.create({
    data: {
      date: new Date(),
      skills: {
        create: {
          name: "lorem ipsum"
        }
      }
    }
  });

I have the error:
Error occurred during query execution:
InterpretationError("Error for binding \'3\': DomainError(ConversionFailure(\"record identifier\", \"assimilated record identifier\"))")
    at PrismaClientFetcher.request (..........\@prisma\client\runtime\index.js:1:51485)
    at processTicksAndRejections (internal/process/task_queues.js:97:5)

Please have a look at https://github.com/prisma/prisma/releases/tag/2.0.0-beta.1 to upgrade your schema to the format that is required here.

@chrisjaimes 2.0.0-beta.2 should now give you a proper validation error when you try to deploy that datamodel.
Please try using that now.

Closing due to inactivity.


Zoom center point remains constant

After two or three prior attempts at getting the canvas to stay centered on the same spot during zooming, I think I finally have it. I guess at some point in my life I remembered how to do algebra.

somehow this fix doesn't work for me on mobile... weird


Do not accept footnote definitions with line breaks

Fixes #760
This change brings pulldown-cmark into alignment with markdown-it and hugo/goldmark. It's close to GitHub, but GitHub doesn't parse the backslashes as hard line breaks.
It's odd that link reference definitions are allowed to have line breaks and footnote definitions aren't, but it's not worth it to be gratuitously different.

Thanks!


How can a push-based monitor be both "Up" and "Down" simultaneously?

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
I'm trying to understand this, maybe I'm doing it wrong, but...
I read https://github.com/louislam/uptime-kuma/issues/279 and still don't quite understand the status parameter for push monitors.
I've got a monitoring script that pushes every hour to my kuma instance.
It sends the params as, e.g.
Something's wrong...
https://my.kuma/api/push/f4h2di3sfz?status=down&msg=3%20devices%20offline

All good...
https://my.kuma/api/push/f4h2di3sfz?status=up&msg=All%20devices%20online

For the monitor itself, I set it to have a heartbeat of 7200 (to allow for some slack on the script timing).
The actual question
So, how can the last "Status update" be Down:

📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.21.3
💻 Operating System and Arch
Ubuntu/docker
🌐 Browser
Chrome 114.0.5735.90
🐋 Docker Version
Docker version 20.10.24, build 297e128
🟩 NodeJS Version
No response

anyone have any ideas? should I try the beta version?

Also wondering if possible to control status programmatically.
It would be great to Push statuses Up and Down via curl (with disabled Heartbeat)


ParamNameMismatch

The class Item uses a trait src/Concerns/HasRelations.php and implements an interface src/Interfaces/ItemInterface.php.
the parameters name in method setRelation of HasRelation dont match with method setRelation of ItemInterface.
You might read in https://psalm.dev/docs/running_psalm/issues/ParamNameMismatch/ about it.
I suggest to rename parameters` name in method setRelation in classes which implements ItemInterface.

Thanks, fixed in d20c349!

Not at all!
Thank you too! )


Update to 0.7.3

For those who can't wait - this works:
https://chimpiq.com/update-openshift-ghost-quickstart/

@corey112358

@MunGell  I'll take a look into this. Can you rebase into a single commit for a cleaner merge?

Hey @jsvgoncalves,
Done, also updated to 0.7.5


Add a flag to allow the removal of the variables after a plan or apply

at least a function to remove all variables would be great

done in 962183b !


How does "label_gain" affect evaluation metrics: Relative or Absolute?

Does the calculation of the NDCG metric depend on the absolute values of label_gain, or does it only consider their relative values?
For example, given labels 0, 1, and 2, I tried calculating NDCG with the following label_gain settings:
A. label_gain = [0, 1, 1]
B. label_gain = [0, 1, 1 + 1e-10]
C. label_gain = [0, 1, 2]
I observed that the difference in metrics between A and B is significant, while the difference between B and C is minimal or nonexistent. Does this imply that during evaluation, the NDCG metric is only influenced by the relative values of label_gain?

Well, label_gain is exactly the 'gain' of ndcg. See:
https://github.com/microsoft/LightGBM/blob/master/src/metric/dcg_calculator.cpp#L109
https://github.com/microsoft/LightGBM/blob/master/src/metric/rank_metric.hpp#L86


Kibana 7.3.2 support

Could you add support for Kibana 7.3.2?

Here you go! https://github.com/sivasamyk/logtrail/releases/download/v0.1.31/logtrail-7.3.2-0.1.31.zip