catkin_make err in ros kinetic version

In file included from /usr/include/X11/Xlib.h:44:0,
from /usr/include/FL/fl_utf8.h:63,
from /usr/include/FL/Fl.H:30,
from /opt/ros/kinetic/lib/cmake/Stage/../../../include/Stage-4.1/stage.hh:57,
from /home/mi/ws_motion_plan_hw/src/upo_stage_ros/src/upo_stageros.cpp:38:
/opt/ros/kinetic/include/ros/callback_queue_interface.h:56:5: error: expected identifier before numeric constant
Success,   ///< Call succeeded

Hello,
as indicated in Github, upo_stage_ros was developed under ROS Indigo. It is
an old package for some finished projects that is no longer in
development.
Anyway, I will take a look at the problems in the Kinetic distro asap.
Best regards,
Noé Pérez
El vie., 17 abr. 2020 a las 9:45, dbdxnuliba<EMAIL_ADDRESS>escribió:

In file included from /usr/include/X11/Xlib.h:44:0,
from /usr/include/FL/fl_utf8.h:63,
from /usr/include/FL/Fl.H:30,
from
/opt/ros/kinetic/lib/cmake/Stage/../../../include/Stage-4.1/stage.hh:57,
from /home/mi/ws_motion_plan_hw/src/upo_stage_ros/src/upo_stageros.cpp:38:
/opt/ros/kinetic/include/ros/callback_queue_interface.h:56:5: error:
expected identifier before numeric constant
Success, ///< Call succeeded
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/robotics-upo/upo_stage_ros/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADUK36BLLEB3ATVYAYM4WKTRNACKZANCNFSM4MKQ3FAA
.

--
NOE PEREZ HIGUERAS
Postdoctoral Researcher - RoboticsLab
Dept. Ingenieria de Sist. y Automatica
Universidad Carlos III de Madrid
c/ Butarque, 15 28911 Leganes, Madrid
España - SPAIN

ok ,thanks a lot ,


fix(userspace/libsinsp): solve wrong type conversion

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area libsinsp
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This solves compilation issues related to a wrong type conversion occurring in the Falco musl static build:
error: converting to 'bool' from 'std::nullptr_t' requires direct-initialization [-fpermissive]

Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.12.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: fb33a658e2c22a99b1fd1e577d609f8c680b2ad9

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, jasondellaluce, Molter73
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,Molter73,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Custom Scalar support for gql

Perceived Problem
Currently custom scalars are only supported for the graffle interface.
Ideas / Proposed Solution(s)

[x] gql outputs
[ ] gql inputs

Whoops, duplicate of https://github.com/jasonkuhrt/graffle/issues/1161


Remove "this" calls from Create Instance

Dev Home version
No response
Windows build number
No response
Other software
No response
Steps to reproduce the bug
TODO found in DevHome.SetupFlow.RepoConfigTaskGroup.cs
Expected result
No response
Actual result
No response
Included System Information
No response
Included Extensions Information
No response

@AmelBawa-msft The task may be outdated but there's still a link to this issue in the code (https://github.com/microsoft/devhome/blob/29c3ce67aa81adc3e2c79c2ea4debf2a7687dd6e/tools/SetupFlow/DevHome.SetupFlow/TaskGroups/RepoConfigTaskGroup.cs#L35). Can you just remove that comment in an upcoming PR?


"Balaciu", "Copuzu", "Crăsanii de Jos" and "Crăsanii de Sus" villages are listed in the drop-down menu.

Description:
When user writes ” Balaciu” and "Copuzu", "Crăsanii de Jos" and "Crăsanii de Sus" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Balaciu".

Expected results
The location is listed in the drop-down menu as "Sat Balaciu IALOMIȚA (Balaciu)".

Step 2
Write in the search bar "Copuzu".

Expected results
The location is listed in the drop-down menu as "Sat Copuzu IALOMIȚA (Balaciu)".

Step 3
Write in the search bar "Crăsanii de Jos".

Expected results
The location is listed in the drop-down menu as "Sat Crăsanii de Jos IALOMIȚA (Balaciu)".

Step 4
Write in the search bar "Crăsanii de Sus".

Expected results
The location is listed in the drop-down menu as "Sat Crăsanii de Sus IALOMIȚA (Balaciu)".


Using Panda with Stencil.js: Module X has already exported member

Description
When I attempt to use panda with a stencil.js init template, using postcss, I get build errors relating to exported types.
I was attempting both the CLI and PostCss approaches of panda css
[ ERROR ]  TypeScript<EMAIL_ADDRESS>ph.d.ts:5289:127
           Type 'CompilerNodeToWrappedType<InstanceOf<T>["compilerNode"]>' does not
           satisfy the constraint 'Node<Node>'.Type 'SourceFile | ImportDeclaration |
           ExportDeclaration | ImportEqualsDeclaration | Identifier | ... 198 more ... |
           Node<...>' is not assignable to type 'Node<Node>'.Type 'SourceFile' is not
           assignable to type 'Node<Node>'.The types returned by 'forgetDescendants()'
           are incompatible between these types.Type 'SourceFile' is not assignable to
           type 'CompilerNodeToWrappedType<InstanceOf<T>["compilerNode"]>'.

   L5288:        expression?: ts.Expression;
   L5289:    }>;
   L5290:  export declare function BaseExpressionedNode<T extends Constructor<Expressioned
NodeExtensionType>, TExpression extends Node = CompilerNodeToWrappedType<InstanceOf<T>["co
mpilerNode"]>>(Base: T): Constructor<BaseExpressionedNode<TExpression>> & T;

[ ERROR ]  TypeScript: node_modules/.pnpm/@pandacss+types@0.41.0/node_modules/@pandacss/ty
pes/dist/index.d.ts:14:1
           Module './composition' has already exported a member named 'Recursive'.
           Consider explicitly re-exporting to resolve the ambiguity.

     L13:  export type * from './runtime'
     L14:  export type * from './shared'
     L15:  export type * from './static-css'

[ ERROR ]  TypeScript: node_modules/.pnpm/@pandacss+types@0.41.0/node_modules/@pandacss/ty
pes/dist/index.d.ts:14:1
           Module './pattern' has already exported a member named 'LiteralUnion'.
           Consider explicitly re-exporting to resolve the ambiguity.

     L13:  export type * from './runtime'
     L14:  export type * from './shared'
     L15:  export type * from './static-css'

[ ERROR ]  TypeScript: node_modules/.pnpm/@pandacss+types@0.41.0/node_modules/@pandacss/ty
pes/dist/index.d.ts:14:1
           Module './pattern' has already exported a member named 'Primitive'. Consider
           explicitly re-exporting to resolve the ambiguity.

     L13:  export type * from './runtime'
     L14:  export type * from './shared'
     L15:  export type * from './static-css'

[ ERROR ]  TypeScript: node_modules/.pnpm/@pandacss+types@0.41.0/node_modules/@pandacss/ty
pes/dist/index.d.ts:19:1
           Module './composition' has already exported a member named 'Token'. Consider
           explicitly re-exporting to resolve the ambiguity.

     L18:  export type * from './theme'
     L19:  export type * from './tokens'
     L20:  export type * from './utility'

[ ERROR ]  TypeScript: styled-system/types/style-props.d.ts:3041:54
           Property 'initialLetterAlign' does not exist on type 'CssProperties'.

   L3040:  initialLetter?: ConditionalValue<CssProperties["initialLetter"] | AnyString>
   L3041:  itialLetterAlign?: ConditionalValue<CssProperties["initialLetterAlign"] | AnySt
   L3042:   /**

Link to Reproduction
https://github.com/JollyGrin/stencil-panda
Steps to reproduce
Option 1
git clone https://github.com/JollyGrin/stencil-panda
cd stencil-pencil
pnpm i
pnpm prepare
pnpm panda
pnpm start

Option 2

pnpm create stencil
pick component from the stencil init
pnpm i -D @stencil-community/postcss autoprefixer
update stencil.config.ts

import { postcss } from '@stencil-community/postcss';
import autoprefixer from 'autoprefixer';

export const config: Config = {
  namespace: 'stencil-panda',
  ...
  plugins: [
    postcss({
      plugins: [autoprefixer()],
    }),
  ],
};

install panda

pnpm install -D @pandacss/dev
pnpm panda init -p

update package.json scripts

 "scripts": {
    "prepare": "panda codegen",
  }

update src/components/my-component/my-component.css by adding @layer reset, base, tokens, recipes, utilities;
run panda prep

pnpm prepare
pnpm panda

try adding some css to src/components/my-component/my-component.tsx

import { css } from '../../../styled-system/css';
...
 return (
      <div class={css({ color: 'red' })}>
        Hello, World! I'm {this.getText()}
      </div>
    );

pnpm start

JS Framework
Stencil.js (using ts)
Panda CSS Version
^0.41.0
Browser
Arc
Operating System

[X] macOS
[ ] Windows
[ ] Linux

Additional Information
Mostly likely is I did something wrong with the install/configuration, but after searching through:

open/closed issues
toggling config options
googling
chatgpt

I still couldn't find the solution to overcome this build step so figured I'd try to make an issue with an example repo of my progress.
https://github.com/JollyGrin/stencil-panda

Please set skipLibCheck: true in your tsconfig to resolve this.
I recommend using the CLI when integrating with Stencil until we develop a doc/example.
Feel free to contribute to the docs when you get all this working.


.net 8.0

Hi, just wanted to ask, if you plan to switch to new .NET (8.0 is now LTS) instead of .NET framework.

Hi @mexmer,
Thanks for the question. I like .NET 8 a lot, but I wasn't planning on doing this.
Summary: nothing wrong with it, it is feasible, I just don't see a benefit, so it would be a bit of work for no gain.
Here are the reasons I think this:

.NET Framework is preinstalled on all targeted Windows versions, whereas users would have to separately download and install .NET Runtime 8

It also requires fewer steps to update .NET Framework with Windows Update on Windows Server, because there's a little-known and poorly-documented registry key to enable this for .NET

This program wouldn't be improved much by .NET 8's many nice benefits

Cross-platform runtimes
More built-in libraries like System.Text.Json and WebSockets
AOT compilation
Minor runtime performance improvements
Minor BCL additions like new types (DateOnly is cool, Span and Memory if you have RAM bottlenecks) and methods (Task.WaitAsync is nice), and simpler support for language features like init properties and more features of record types
Even though Task.Delay allows longer durations, which this program needs, it's still not long enough, so Task chaining is still required for arbitrary delays
TimeProvider is cool, but these automated tests already work without it
Not having to prepend each .cs file with #nullable enable
Self-contained single-file executables

However, if there is a specific reason to target a .NET runtime like 6 or 8, I would consider multitargeting this project and providing separate downloads for both .NET Framework 4.8 and .NET 8. I won't be removing the .NET Framework build for backwards compatibility with existing installations, and because zero prerequisites is always helpful.
Let me know if there is a feature of the .NET 8 runtime that you are missing in this project.

Thanks for answers. Indeed most of you mentioned makes sense.
Only problem with .NET framework i see, it is not pre installed (unless we talking .NET 2.0, but your app needs 4.8) ... here your assumption is wrong, it's optional component, you need download and install and requires computer restart (just did that few days ago, on windows 2016 .NET 4 feature installs only 4.6 and 2019 server .NET  features is version 4.7)
only Windows Server 2022 has 4.8, but even there it's optional feature, that is disabled by default and requires restart after installing.
since i'm writing services for windows server in .NET LTS (now 8, before 6), i'm somewhat familiar with it's downsides too.
for deployment it depends on client, i either created selfconainted or framework dependent, it depends on client environment. Becuase we also provide app updates, i'm not much stressed with lack of windows update support for .NET LTS deployment, we just deliver app update after testing with new .NET release.
Anyways i was just curious, why project stays on .NET Framework, thanks for answers.

Oh yeah I forgot that Server 2016 only starts with .NET Framework 4.6.2 instead of 4.8 before updates. I think I was getting it confused with Windows 10 which has a shorter support lifecycle than Server.
I could have sworn it was preinstalled and didn't require a restart. Are you referring to Server Core/Nano/no-Desktop-Experience? I will have to test this out in a VM to refresh my memory. I don't remember having to install it in either of my server machines, otherwise how would PowerShell work?
Thanks!

.NET Framework 4.7.2 was preinstalled in Server 2019 Desktop Experience, and installing the 4.8 update did require a restart. I must have gotten that via Windows Update before attempting to run this program on my real servers.
Thanks again, I know a little bit more about this now.

Oh yeah I forgot that Server 2016 only starts with .NET Framework 4.6.2 instead of 4.8 before updates. I think I was getting it confused with Windows 10 which has a shorter support lifecycle than Server.
I could have sworn it was preinstalled and didn't require a restart. Are you referring to Server Core/Nano/no-Desktop-Experience? I will have to test this out in a VM to refresh my memory. I don't remember having to install it in either of my server machines, otherwise how would PowerShell work?
Thanks!

FYI feature is automatically installed, if you go with "Desktop Experience" setup (this is available for standard, and datacenter edition) or install Windows Server Essentials Edition, because MMC modules are written in .NET Framework. But will still require update, unless you installing Windows Server 2022

Seems like a fresh installation of Server 2019 Core (without Desktop Experience) also has .NET Framework 4.7.2 preinstalled. If it didn't, PowerShell 5 wouldn't run. But yeah, an update would be required to get .NET Framework 4.8 on here.

You are right, i forgot about powershell. so yes, core is installed, and for your service it would be enough, except it targets 4.8


You can't organise editors into tabs using Edit Placement

From version 1.9 (or 10?) you can move editors into tabs using #tab_name in your module's Placement file. However, you can't control this from the admin area using Edit Placement. That screen should be revamped to allow tabs to be created and editors moved between them

@Hazzamanic @sebastienros Is it possible to have this feature mergeable in 1.10.x? I have tried to merge it by myself but it seems like if the fix was branched from dev and not from 1.10.2; am I right?


Eas build :app:checkReleaseDuplicateClasses

Summary
I tried to run eas build on android. If I use stanalone build everything works correct. All of pipelines complete successfully and everything crashed on Run gradlew.
Environment
expo-env-info 1.0.5 environment info:
System:
OS: Windows 10 10.0.22000
Binaries:
Node: 16.17.1 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.17 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.19.2 - ~\node_modules.bin\npm.CMD
SDKs:
Android SDK:
API Levels: 26, 27, 28, 29, 30, 31, 32, 33
Build Tools: 29.0.0, 29.0.1, 29.0.2, 29.0.3, 30.0.0, 30.0.1, 30.0.2, 31.0.0, 33.0.0
System Images: android-29 | Google Play Intel x86 Atom, android-30 | Google APIs Intel x86 Atom
IDEs:
Android Studio: Version     2<IP_ADDRESS> AI-211.7628.21.2111.8309675
npmPackages:
expo: ^45.0.6 => 45.0.6
react: 17.0.2 => 17.0.2
react-dom: 17.0.2 => 17.0.2
react-native: 0.68.2 => 0.68.2
react-native-web: 0.17.7 => 0.17.7
Expo Workflow: managed
Please specify your device/emulator/simulator platform, model and version
Andoid 11, API 30
My dependcies:
"dependencies": {
"@expo-google-fonts/amatic-sc": "^0.2.2",
"@expo-google-fonts/roboto": "^0.2.2",
"@react-native-async-storage/async-storage": "~1.17.3",
"@react-native-community/cli-server-api": "^9.0.0",
"@react-navigation/drawer": "^6.3.1",
"base-64": "^1.0.0",
"expo": "^45.0.6",
"expo-app-loading": "~2.0.0",
"expo-dev-client": "~1.0.1",
"expo-network": "~4.2.0",
"expo-notifications": "~0.15.4",
"expo-radio-button": "^1.0.8",
"expo-splash-screen": "~0.15.1",
"expo-status-bar": "~1.3.0",
"expo-updates": "~0.13.4",
"moment": "^2.29.4",
"react": "17.0.2",
"react-dom": "17.0.2",
"react-native": "0.68.2",
"react-native-device-country": "^1.0.2",
"react-native-expo-viewport-units": "^0.0.8",
"react-native-gesture-handler": "~2.2.1",
"react-native-google-pay": "^2.1.0",
"react-native-intersection-observer": "^0.0.8",
"react-native-reanimated": "~2.8.0",
"react-native-safe-area-context": "4.2.4",
"react-native-select-input-ios": "^2.0.5",
"react-native-share": "^7.6.1",
"react-native-web": "0.17.7",
"react-native-webview": "11.18.1",
"react-redux": "^7.2.6",
"redux": "^4.1.2",
"redux-thunk": "^2.4.1"
},
"devDependencies": {
"@babel/core": "^7.12.9",
"@react-navigation/native": "^6.0.8",
"@types/base-64": "^1.0.0",
"@types/jest": "^27.4.1",
"@types/react": "~17.0.21",
"@types/react-native": "~0.64.12",
"@types/react-redux": "^7.1.23",
"jest": "^27.5.1",
"ts-node": "^10.7.0",
"typescript": "~4.3.5"
}
Error output

Task :app:mergeReleaseResources
Task :app:bundleReleaseResources
Task :app:compileReleaseAidl NO-SOURCE
Task :app:generateReleaseBuildConfig
Task :app:javaPreCompileRelease
Task :app:checkReleaseDuplicateClasses FAILED
Task :app:checkReleaseAarMetadata
w: Detected multiple Kotlin daemon sessions at build/kotlin/sessions
Deprecated Gradle features were used in this build, making it incompatible with Gradle 8.0.
You can use '--warning-mode all' to show the individual deprecation warnings and determine if they come from your own scripts or plugins.
See https://docs.gradle.org/7.3.3/userguide/command_line_interface.html#sec:command_line_warnings
Execution optimizations have been disabled for 6 invalid unit(s) of work during this build to ensure correctness.
Please consult deprecation warnings for more details.
568 actionable tasks: 568 executed
[stderr] FAILURE: Build failed with an exception.
[stderr] * What went wrong:
[stderr] Execution failed for task ':app:checkReleaseDuplicateClasses'.
[stderr] > A failure occurred while executing com.android.build.gradle.internal.tasks.CheckDuplicatesRunnable
[stderr]    > Duplicate class com.google.android.gms.common.api.internal.zza found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.api.internal.zzb found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.api.internal.zzc found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.api.internal.zzd found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.api.zza found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.api.zzb found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzb found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzc found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzd found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzf found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzg found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzk found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzl found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzm found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.internal.zzn found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]      Duplicate class com.google.android.gms.common.zza found in modules jetified-play-services-base-15.0.1-runtime (com.google.android.gms:play-services-base:15.0.1) and jetified-play-services-basement-17.0.0-runtime (com.google.android.gms:play-services-basement:17.0.0)
[stderr]
[stderr]      Go to the documentation to learn how to Fix dependency resolution errors.
[stderr] * Try:
[stderr] > Run with --stacktrace option to get the stack trace.
[stderr] > Run with --info or --debug option to get more log output.
[stderr] > Run with --scan to get full insights.
[stderr] * Get more help at https://help.gradle.org
[stderr] BUILD FAILED in 6m 14s
Error: Gradle build failed with unknown error. See logs for the "Run gradlew" phase for more information.

Reproducible demo or steps to reproduce from a blank project
eas configure
eas build -p android

It looks like it's an issue with react-native-google-pay it depends on com.google.android.gms:play-services-base:15.0.1 which is one of the conflicting versions from the error message. I see the last commit in that repo was over 2 years ago, so it does not seem that package is maintained,

It looks like it's an issue with react-native-google-pay it depends on com.google.android.gms:play-services-base:15.0.1 which is one of the conflicting versions from the error message. I see the last commit in that repo was over 2 years ago, so it does not seem that package is maintained,

Thank you. Now it works


update  tutorial.md

add priority-class

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/assign @hzxuzhonghu

/lgtm
/approve


Brooklin having issue connecting with zk server with java 21 runtime

Subject of the issue
Describe your issue here.
Brooklin unable to connect to the zk server with java 21 runtime.  whereas it is able to connect to zk server with java 11 runtime.
Your environment

Operating System Linux
Brooklin version 4.1.0
Java version 21
Kafka version
ZooKeeper version 3.5.4

Steps to reproduce
Tell us how to reproduce this issue.
Deploy the brooklin with java 21 runtime
Expected behaviour
Tell us what should happen
Brooklin unable to connect to zk server and giving below error
2023-10-17 12:16:56.628  INFO 1 --- [main-SendThread(zk-cs:2181)] org.apache.zookeeper.ClientCnxn$SendThread : Opening socket connection to server zk-cs/:2181. Will not attempt to authenticate using SASL (unknown error)
2023-10-17 12:16:56.629  WARN 1 --- [main-SendThread(zk-cs:2181)] org.apache.zookeeper.ClientCnxn$SendThread : Session 0x0 for server zk-cs/:2181, unexpected error, closing socket connection and attempting reconnect
java.nio.channels.UnresolvedAddressException
at java.base/sun.nio.ch.Net.checkAddress(Unknown Source)
at java.base/sun.nio.ch.Net.checkAddress(Unknown Source)
at java.base/sun.nio.ch.SocketChannelImpl.checkRemote(Unknown Source)
at java.base/sun.nio.ch.SocketChannelImpl.connect(Unknown Source)
at org.apache.zookeeper.ClientCnxnSocketNIO.registerAndConnect(ClientCnxnSocketNIO.java:277)
at org.apache.zookeeper.ClientCnxnSocketNIO.connect(ClientCnxnSocketNIO.java:287)
at org.apache.zookeeper.ClientCnxn$SendThread.startConnect(ClientCnxn.java:1021)
at org.apache.zookeeper.ClientCnxn$SendThread.run(ClientCnxn.java:1064)
I able to hit the zk server from brooklin pod. so it must not be connection issue and with same setup i am able to connect with java 11 runtime.
Actual behaviour
Tell us what happens instead
Brooklin should able to connect zk server with java 21 runtime.

Hey,
We have not certified java 21 with Brooklin. My recommendation is to use java 11 for production Brooklin instances at this time
Are you running this under docker or some sort of container? If yes, double check the networking is configured properly.
Did you share zk-cs as the hostname for the purposes of this issue, or is that the actual hostname?
Can you share the output of running the command hostname zk-cs from within the environment running Brooklin?

deployed the zookeeper cluster in the k8s. And below is my zk service yaml
apiVersion: v1
kind: Service
metadata:
name: zk-cs
spec:
ports:
- port: 2181
name: client
The same set of configuration works fine when the brooklin running in java 11 run time. whereas when we try to run the brooklin in jdk17 or jdk21 runtime. it failing to connect with zk service.

Can it run on Java 17?


INT server : Item details layout html issue.

Map alignement is not proper. Make it according to layouts.

Fixed.


Drop R2DBC BOM after 0.9 release

Going forward, R2DBC 1.0 is expected to become the "done" version and a baseline for drivers for the foreseeable future. Having a bill of materials during the specification evolution helped to correlate driver versions to the specification version to ensure a consistent dependency arrangement.
With version 1.0 of the specification, we expect all available drivers to be based on that version which removes the need for version coordination. Additionally, BOM releases enforce a strong synchronization across participating drivers and BOM releases require constant release effort.
Therefore, I propose to stop releasing the BOM after the last 0.9 R2DBC release train, likely during the next year.

We did not release a BOM, so we can close this ticket now.


Incorrect method stated for creating remoteDesktopSecurityConfiguration

When you post to this endpoint, the API returns "Specified HTTP method is not allowed for the request target." from testing, to enable remoteDesktopSecurityConfiguration, a patch is required instead. The response is also a 204.

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

ID: 5ea3ab4f-2eab-6e98-ba1b-59776b74dc41
Version Independent ID: 7d8b9961-37d4-e325-c3df-918a7c5a4f1d
Content: Create remoteDesktopSecurityConfiguration - Microsoft Graph v1.0
Content Source: api-reference/v1.0/api/serviceprincipal-post-remotedesktopsecurityconfiguration.md
Product: applications
Technology: microsoft-graph
GitHub Login: @SanDeo-MSFT
Microsoft Alias: MSGraphDocsVteam

Thanks for this feedback. We are updating docs to fix this.


synchronized() fix in SerializerTracker

Issue spotted by @pburka

closing all PRs and issues in big switch to BeakerX.


Fixing group messaging

Group messages have a negative message.chat.id
This fixes send_response() to be able to handle sending messages back
Before it was trying to edit them and causing server errors:
aiogram.exceptions.TelegramBadRequest: Telegram server says - Bad Request: message can't be edited

Thanks for the PR!


Taking stills

Hello Dave,
Your PiStreaming is working superbly :-) thank you.
I would like to be able to take a still image/photo, would that be possible?
Using Raspistill doesn't want to work, presumably because other services have snaffled the resource.
I don't mind if the video stream flickers/pauses while the configuration modes are changed in order to get a higher resolution than 640x480 - in fact for a still image it is preferable.
Many thanks,
Brian.

Just to add to this one. I've been reading in more and by modifying main() in the while loop for camera.wait_recording(1).
while True:
                camera.wait_recording(1)
                if (os.path.isfile("/home/pi/camphoto")):
                    os.remove("/home/pi/camphoto")
                    my_file = open('/home/pi/camphoto.jpg', 'wb')

#                    camera.stop_recording()
#                    camera.resolution = (2592, 1944)
                    camera.capture(my_file, format="jpeg", use_video_port=True)
#                    camera.resolution = (WIDTH, HEIGHT)
                    my_file.close()

                    f = open("/home/pi/camphoto.done", "a")
                    f.close()

#                    camera.start_recording(output,'yuv')

The use of 'camphoto' files are notifications from one program to another, in this case if 'camphoto' is present, remove it then take a photo.
This works like a charm, but only works at 640x480 - as long as I don't change the resolution.
If I remove the comments to stop recording, increase the resolution, take the photo, put the resolution back and restart recording then it fails.
The photo does get taken and at the correct resolution, but pistreaming then fails.
From the output it appears to complain at BroadcastOutput.write stating that the file is closed.
I assume it's because there are other threads running while I've got the camera recording stopped that is causing the issue here, wondered if there's a way around it.
Many thanks,
Brian.

Following. ..interested in doing similar.

Also interested. I am trying to implement a button to the webpage that would take a picture.


[Console-IoT] Add meaningful ID to components

Type of change

Refactoring

Description

Adding meaningful id to components for address module
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

Can one of the admins verify this patch?

All comments have been resolved after the latest commit. This PR is ready for merge

@suyash-naithani some suggestions are not resolved yet. toggleId is missing in some of the DropdownWithToggle comopents

Resolved all comments. Looks good to merge


The validity of the way PIN-SLAM get the SDF gradient？

Some tensor operation is Non-differentiable in neural_points.py.
How to make sure the validty of the way pin slam gets the SDF gradient?
The input coordinates are first discretized, neighbor searched it's 6 neighbors neural points, following the fusion of the neural points feature, and then fed into neural decoder.
Strictly speaking, the analytical gradient obtained by pytorch should not be the gradient of the SDF to the input position.

Thanks for your interest in our work.
The voxel-based neighborhood searching is not differentiable. However, it only acts to index the surrounding neural point in the nearby voxels and is not involved in the computational graph. All the steps afterward are differentiable. The relative coordinate between the query point and each surrounding neural point (whose coordinate is a constant) is used for latent feature concatenation and inverse distance weighting, thus involved in the computational graph of the final SDF prediction. You may read the paper for more details.

Thank your for detailed answer and take up your valuable time.
And later I will re-edit my question and make my question more clearly after my recently busy days.
I think the unstanding how to get SDF gradient is very important, which significantly effect the localization accuracy.Intutively I think the Sdf decoder and the feature fusion way is not natural but the neural network is very powerful so Pin slam can work very well.
I mean the scan-to-implicit map registration can provide a more refined way to estimate pose compared to point-to-plane residual, which is the meaning of neural sdf map.
Photo slam also uses the LM optimization to tracking. The common way that pin slam and phone slam use these classic optimization methoda instead of the neural network backpropagation
arise my thinking.Because the pose is only 6dof, different from the map geometry or other property, it doesn't have to updated by backpropagation due to its low dimension.
As you mentioned in pin slam paper and other papers, geometry loss is easy to optimize but color loss or      photometry loss is hard to optimize by the graph solver.  And
My thinking for this question is stopped here. I think that why visual odometry such as ORB slam and Vins always recover the 3d point.

Thanks for your insightful comment. Now it seems the discussion is on the research concept instead of technical details. Therefore, I'd like to close this issue but we can still have the discussion under this thread.


Geographic Coord functionality in plotcurv3D

Added geographic coord functionality to plotcurv3D as an optional argument.

Thanks it's been a busy week, I will try to integrate this next week. Thank you!


List Slug and Rename Redirect Support

Lists and Collection store a slug field, auto-computed from title
Lists can be retrieved via slug
When changing List and Collection title, the old slug is also stored to avoid breaking links
When retrieving List or Collection object, slug_matched is true is using the canonical slug, otherwise should redirect
Bookmark ids use incremental, per list counter

Superceded by #506


Documentation bug: mxnet.ndarray.contrib.foreach doesn't accept name

Description
This is a documentation bug. In the document of mxnet.ndarray.contrib.foreach, the Parameter section stated a parameter name. But it is not in the signature and not accepted by the function.
Link to the document: https://mxnet.apache.org/versions/1.6/api/python/docs/api/ndarray/contrib/index.html#mxnet.ndarray.contrib.foreach

Error Message
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: foreach() got an unexpected keyword argument 'name'

To Reproduce
import mxnet as mx
step = lambda data, states: (data + states[0], [states[0] * 2])
data = mx.nd.random.uniform(shape=(2, 10))
states = [mx.nd.random.uniform(shape=(10))]
outs, states = mx.nd.contrib.foreach(step, data, states, name='op')

Environment
We recommend using our script for collecting the diagnositc information. Run the following command and paste the outputs below:
curl --retry 10 -s https://raw.githubusercontent.com/dmlc/gluon-nlp/master/tools/diagnose.py | python

# paste outputs here

OS: ubuntu 18.04
Python: 3.7.6
pip: 20.0.2
numpy: 1.18.5
mxnet: 1.6.0

This looks like a copy-paste error. The place for fixing it is here:
https://github.com/apache/incubator-mxnet/blob/3c4ac19daa3e645d918692b864ea19640f7e0314/python/mxnet/ndarray/contrib.py#L178-L179

Hello,
I see a PR merged for this issue.
Is this still open?


Add Connect 4 Game

I want to add Connect 4 Game.

Please Assign me under JWOC 2024, if approved.

@Surajit0573 Go for it!


python311Packages.google-cloud-compute: 1.16.1 -> 1.17.0

Changelog: https://github.com/googleapis/google-cloud-python/blob/google-cloud-compute-v1.17.0/packages/google-cloud-compute/CHANGELOG.md
Description of changes

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.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 292385 run on x86_64-linux 1

8 packages failed to build:

python311Packages.skrl
python311Packages.skrl.dist
python311Packages.textnets
python311Packages.textnets.dist
python312Packages.google-cloud-compute
python312Packages.google-cloud-compute.dist
tts
tts.dist

20 packages built:

python311Packages.dalle-mini
python311Packages.dalle-mini.dist
python311Packages.fastai
python311Packages.fastai.dist
python311Packages.google-cloud-compute
python311Packages.google-cloud-compute.dist
python311Packages.k-diffusion
python311Packages.k-diffusion.dist
python311Packages.spacy
python311Packages.spacy-loggers
python311Packages.spacy-loggers.dist
python311Packages.spacy-lookups-data
python311Packages.spacy-lookups-data.dist
python311Packages.spacy-transformers
python311Packages.spacy-transformers.dist
python311Packages.spacy.dist
python311Packages.textacy
python311Packages.textacy.dist
python311Packages.wandb
python311Packages.wandb.dist


Looback 3/NodeJS intellisense not working

Not able to get intellisense on Loopback 3 nodeJS framework projects.
Example repo has Account model, I've added "test" function:

Trying to get intellisense for Account.test from another Model file doesn't work:

Or, trying via app.models.Account as per LB3 conventions:

I've tried everything I can think of, any advice here? This seems like it could be a bug in intellisense?

VSCode Version: 1.3.9
OS Version: Mac OS 10.14.6

Steps to Reproduce:

clone https://github.com/rlyttle/loopback-example-relations
Try to use intellisense between Model javascript files within common/models folder (Account.js and Address.js used as examples).

Does this issue occur when all extensions are disabled?: Yes

Update: I've pushed new code that shows intellisense working in Webstorm IDE, but not in VS Code
Webstorm screenshot of Account module:

Webstorm screenshot of Address module (provides intellisense for Account functions in webstorm, but not VS code):

Any help would be appreciated, this seems like a bug in VS Code


Fix typo

python -> Python

Learn Build status updates of commit 0c2ada7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/machine-learning/migrate-to-v2-managed-online-endpoints.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@changeworld Thanks for catching this!
Looks good to me.

#sign-off


[dv:tTL UL agent] updated tl_ul to select transctions ID from fifo ti support b2b transactions

This PR contains The update that I was tasked with in the TL UL meeting and addresses the problem described in #3303  + a fix to zero delays.
A queue is added to the tl_agent_cfg. The queue contains available transaction IDs from [0 : Max_outstanding_req-1] and is randomized before the test starts.
when the driver sends a transaction it pops an available ID from the queue,
when the Transaction is completed by the device the monitor will push the ID back into the queue.
Q: currently Ids are Popped from the front and pushed in the back.
this means all ID's will be cycled. another option is to push and pop on the same side of the queue which will only use the first few IDs
Q: currently the queue is randomized before the test starts, I don't think this hurts us - but realistically a host will probably start with id=0 and increment.
with this PR I also added a knob to control the pct of which zero_delays will be set
this allows a designer to control the value from a test or a higher level config item.
there is also a little AES clean up that came about in the process of debugging an issue with controlling zero_delays.
should we merge this until we have a better suggestion or should we wait for @sriyerg and @weicaiyang solutions?
Aes is not blocked but scribbled as long as B2b is not an option

Oh, and the failing CI doesn't look like it's anything to do with you. I'm going to have a bit of a dig to understand that now.

@rswarbrick I think all your comments should be addressed by my latest update

Oh, and the CI failures aren't spurious this time :-) You'll need to re-format your commit messages to the standard style (one short header line, a blank line, the body text).

Hi ~Magnus~ Rasmus (sorry!)! Some nitty stuff, plus a proper question about replacement strategy into the queue. Also, could you split out the AES changes into a separate PR? As far as I can see, these aren't related to the ID queue?

Agree with @rswarbrick to make this PR focus on tl_agent only (submit AES's stuff in another PR). This is because tl_agent is widely adopted and would affect the other IPs.

@rswarbrick @weicaiyang @sriyerg @tunghoang290780
I am holding back on fixing the review comments here as we only need one of the 3 solutions (mine, @sriyerg or @weicaiyang )
and I think we will most likely go with @sriyerg suggestion (I have yet to review it) but we all agreed that it would be the cleanest in the meeting.
so unless we decided that my approach which was meant to be quick is the way we want to go I see no point in spending time updating it.
some good points were made though and we should make sure these are considered in the other two suggestions


Streaming reads

Hi! Thanks for writing this lib!
Have you thought about the case of streaming reads? It seems the API is centered around Mp4Reader::read_sample(track_id, sample_id) which in turn delegates to Track::read_sample. Reading the code it seems the latter assumes a  reader.seek(SeekFrom::Start(sample_offset))?;
With very large files, or continuous TS-streams, it's not necessarily practical to assume the whole things can be wrapped in a BufReader.
Would it be possible to extend the API somehow to have more of a "read-next-sample-whatever-it-is" approach? Happy to contribute a PR.

Hey @lolgesten,
Thanks for the input and suggestion. I admit most of the samples I have worked with have been small enough to be wrapped into a buffer, but I agree it can be improved.
I would appreciate a PR if you have the time for it. :)

Just to not: generally, I think, mp4 is not really designed to be streamable =(
There are some things that can be done to make it so: https://trac.ffmpeg.org/wiki/HowToCheckIfFaststartIsEnabledForPlayback


Revenant (1999) Error, Almost Working

Revenant (1999) PC (16-Bit Graphics)
https://www.pcgamingwiki.com/wiki/Revenant/
https://www.gog.com/game/revenant/
https://en.wikipedia.org/wiki/Revenant_(video_game)/
Hi I've tested Revenant using the Nov version DXWrapper. I've test also with DDrawCompat EXPTest 2020.19.Narzoul version and its loading and working. The good thing is i can see my 2nd GPU nVidia when im using DXWrapper, Hope you could fix it someday
-Using only DDraw.dll , DXWrapper is Disabled
dxwrapper-revenant.log

-Using DDraw.dll , DXWrapper is Enabled

-Using DDrawCompat EXPTest 2020.19 By Narzoul, DDraw.dll

Thanks for the report.  This issue is because Revenant uses Direct3D functions and I have not yet added support for most of the Direct3D functions.  This issue is related to #101, #71, #65.
For status on Direct3D see this wiki:  (DirectDraw-to-Direct3D9-Conversion-Status)[https://github.com/elishacloud/dxwrapper/wiki/DirectDraw-to-Direct3D9-Conversion-Status]

Thanks for the report.  This issue is because Revenant uses Direct3D functions and I have not yet added support for most of the Direct3D functions.  This issue is related to #101, #71, #65.
For status on Direct3D see this wiki:  (DirectDraw-to-Direct3D9-Conversion-Status)[https://github.com/elishacloud/dxwrapper/wiki/DirectDraw-to-Direct3D9-Conversion-Status]

Can you try with the latest build?  This issue should be fixed in the latest build.


direct session: catch SIGINT and SIGTERM

The direct session uses a child process for privileged operations. However the child process doesn't exit cleanly when it receives SIGINT or SIGTERM.

related bug #1583

The direct session has been dropped in favor of libseat.


git bash clear command not working on windows again!

[x] I am on the latest Hyper.app version

[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Windows 10 v1809

Hyper.app version: 3.0.0

Link of a Gist with the contents of your .hyper.js: https://gist.github.com/seriiix/4d1a4750b08f6355f0fb9b727d082e36

Issue
There was a well-known issue with clear command with bash on windows. It seems with v3 there is another issue related to this.

When I write clear it does clear some upper parts of the terminal but not all of it. I was using this config with v2 and everything was working as expected.
shell: 'C:\\Program Files\\Git\\git-cmd.exe',
shellArgs: ['--command=usr/bin/bash.exe', '-l', '-i'],
env: { TERM: 'cygwin' },

I suspect this is because Hyper is using the default config now. Hyper 3 uses a different config location and ships with a default config, effectively ignoring settings from the old config. https://github.com/zeit/hyper/issues/3605

I set up my .hyper.js setting from scratch and get the same issue.

@spences10 can you confirm that you've updated the config in the new location? I'm using Hyper 3 with Git Bash, and clear seems to work after I've re-applied my config.

@SidneyNemzer yes, confirmed. So setting up from scratch would be using the shortcut Ctrl+, to open the configuration file and edit it from there, like you would if you were setting up from scratch (like a fresh install with no preconfigured options).

I suspect this is because Hyper is using the default config now. Hyper 3 uses a different config location and ships with a default config, effectively ignoring settings from the old config. #3605
Try copying your old config to the new location as described in that issue.

No, as you can see (themes and fonts are not default), I already re-applied my old config.

Do you guys have the Web GL render enabled?
The clear command works for me, I'm on Windows 10, Hyper 1.0.2, using Git Bash. I've got the WebGL renderer enabled.

Yes. I have WebGL enabled. Sharing  .hyper.js file is part of the issue template. You can easily see my config in the mentioned gist. Also, we are talking about Hyper V3, not V1.

I noticed when I'm facing this problem, Resizing the terminal will clear the bottom parts as well.

Do you guys have the WebGL renderer enabled?

I haven't touched that setting so mine is whatever the default is.

I did a fresh install today. No issues with clear
My Settings with win10 and Hyper.app version: 3.0.2:
    shell: 'C:\\Program Files\\Git\\git-cmd.exe',
    shellArgs: ['--command=usr/bin/bash.exe', '-l', '-i'],
    env: { TERM: 'cygwin'},

Honestly i still can't clear properly.

What versions of Git Bash is everyone using? I just remembered that I'm using an older version because the newer one had issues. (I'll edit this with my version when I get to my home PC).

What versions of Git Bash is everyone using? I just remembered that I'm using an older version because the newer one had issues with Hyper. (I'll edit this with my version when I get to my home PC).

@SidneyNemzer I believe it's not related to git bash because when I force re-render the terminal by minimizing and maximizing the window or resizing it, It'll get entirely clear.

This is happening to me with Bash on Windows 10. I'm using Hyper 3.0.2. I uninstalled and reinstalled Hyper and the problem still persists.

My main problem is the residue after executing htop.

But when I merged this code change, it worked fine.

https://github.com/xtermjs/xterm.js/pull/2149

When it detects my input, it draws the full screen.

Now, if you don't compile by yourself, you need to wait for upstream updates.

This doesn't happen if the windowSize config is exacly [540, 380] (the default value).
Of course, this size is very small for daily operations like edit files and long/wide lists viewing.

Test again with a canary CI build please


Time reporting format

Documentation reference (URL): https://github.com/ess-dive-community/essdive-csv-structure/blob/master/csv_quick_guide.md; https://github.com/ess-dive-community/essdive-hydrologic-monitoring/blob/main/template_DataFile.csv; https://github.com/ess-dive-community/essdive-soil-respiration/blob/main/data_reporting_format_guide.md; https://github.com/ess-dive-community/essdive-leaf-gas-exchange/blob/master/docs/definedVariables.md; https://github.com/ess-dive-community/essdive-sample-id-metadata/blob/master/guide.md#collection-date
Submitter: Kristin Boye
Justification (why is this change necessary?): there is no consistency in the format for how date and time is reported between the various "reporting formats"

I suggest the following changes: Could we agree on an "ESS-DIVE central" format for how to report date and time? Looking through the currently uploaded instructions (CSV, Sample ID, Soil Respiration, Hydrologic Monitoring, Leaf Gas exchange) there are as many ways of reporting date and time as there are reporting formats (although all recommend UTC and request UTC offset if using local time). I personally like (and think it would make things more script-friendly as we are trying to limit the use of special characters????) to keep delimiters out of it (as proposed in the soil respiration reporting format) and simply use up to 12 digits (YYYYMMDDHHMMSS) to the level of appropriate resolution for the data, but as long as we agree on what we should require/recommend I think that would help a lot...

Agreed upon this format with a space between date and time: YYYY-MM-DD hh:mm:ss
I can add this information to the instructions.

@tvelliquette , when we get all of the wording updated, we can close out this issue.

@robcrystalornelas I added to quick guide and detailed guide info about it being okay to use two fields for date and time under temporal data plus added the space in the ISO standard format
Under temporal range I added text about naming fields "("dateTime_start and dateTime_end" or "time_start and time_end")

I just finished reviewing the guides for this term. Everything looks good. I added in some quotes to the example you provide above, so that people know "dateTime_start" and "dateTime_end" are two different columns.

close if done

Closing


Sample Solution for Client Side (Excercise 1)

This is my first take on the sample solution. Obviously, it is not aligned with the sample solution for the server side, yet.

:+1: thanks!
I will get through it as early as possible.


Add iOS 10 usage description

If you're running iOS 10 (currently in beta) and the app crashes when interacting with HealthKit please update to 0.5.1.


related fixes `get_vaccine_cohorts_at_date:

date argument same name internally created date variable. argument renamed as target_date.
Later, days_ago variable is created from Sys.Date() - date (variable) so only creates correct value if target_date is date run.
changed to target_date - date

good to merge?


(SERVER-3016) Filter out EC params when parsing private keys

When OpenSSL generates an elliptic curve key, it writes the EC params in
a separate block above the key in the output file. When we parse this
file, we load both objects in the file, the EC params, and the key. But
we only want to return the key and ignore the params, so this commit
adds a filter to the reader function, so that EC param objects are
ignored.

holding off on merging in case @justinstoller wants to take a last look


Oauth2 Client_secret on frontend is always a security issue

Hello,
You asked to fill an issue if we think the Oauth2 app *client_secret  is security risk in frontend.
Just to let you know that your client secret can be use to manipulate other GitHub  resources using other mechanism than Oauth2 authorization_code flow.
They can be use in GitHub authorization APIs to gain informations about your end-user for example (https://developer.github.com/v3/oauth_authorizations/).

Hello @kadary,
I've been thinking about the same when reading about client_secret to be used on the front-end. After reading your ticket about it the thing that made it even more funnier that we even think alike with our profile pictures. :laughing:

Well, when I saw this, I jumped out from my seat. I expected some kind of hashing or something to hide it.
So, I've moved to https://utteranc.es/.


Upgrade actions/cache, actions/checkout, and actions/setup-java

Upgrade actions/cache, actions/checkout, and actions/setup-java to versions 3, 4, and 3.13.0

/easycla


ReadMeCard no longer supports GitHub Enterprise

Expected Behavior
The ReadMeCard should query against GHE instances correctly to retrieve the ReadMe for an entity
Current Behavior
The ReadMeCard defaults to hitting public github.com and fails for Backstage instances configured with GHE integrations
Steps to Reproduce
View the ReadMeCard in an entity that is registered to a GHE source.
Possible Solution
Refactor GithubClient.getContent to allow the base url of octokit to be configured based on the entity useEntityGithubScmIntegration hostname
Context
This looks like it was caused by this recent commit which removed usage of the useRequest hook in favor of the new GithubClient which looks like it doesn't correctly configure the octokit base url to account for GHE endpoints resulting in defaulting to hitting github.com
Your Environment

@roadiehq/backstage-plugin-github-insights version: 2.3.0
Backstage release v1.9.1

Sorry about that @kuangp that was my fault, Ill take a look

I hope I have the fix coming. It would be great if you could test it for me @kuangp . I dont have a ghe deployment.

This should be fixed in version 2.3.1

Fix looks good! Thanks @punkle !


Align spinoso-string#make_binary to be more alike the String#b

Fixes #1720
As talked about in the issue, the implementation in spinoso-string can be a little confusing when looking at how it is implemented in artichoke-backend. Specifically, spinoso-string is changing the encoding, however String#b is actually cloning the String before actually changing, so we might as well just do this directly and return a new value.

@lopopolo I believe it's unrelated to this specific change, but am I correct in assuming we can't actually check anything with relation to encoding on the String ruby class? https://github.com/artichoke/artichoke/blob/trunk/artichoke-backend/src/extn/core/string/string.rb#L514
Specifically, trying to test this above change like so:
artichoke 0.1.0-pre.0 (2022-04-02 revision 5484) [x86_64-unknown-linux-gnu]
[rustc 1.58.1 (db9d1b20b 2022-01-20) on x86_64-unknown-linux-gnu]
>>> "a".b.encoding
=> #<Encoding:UTF-8>

^-- not what I was expecting, and dug into String#encode and found that statement.

Yea Encoding as a Ruby object isn't exposed from Artichoke right now. The comments in the Ruby stubs appear to be outdated tho.
Artichoke does have an Encoding class, but it is essentially a stub to get Ruby spec to run. It so not wired up to the String internals.
I should file some tickets.

Would you believe that Documentation Workflow error run can't be reproduced locally?
➜  artichoke git:(make-binary-to-binary) rustup -Vv
rustup 1.24.3 (ce5817a94 2021-05-31)
info: This is the version for the rustup toolchain manager, not the rustc compiler.
info: The currently active `rustc` version is `rustc 1.58.1 (db9d1b20b 2022-01-20)`
➜  artichoke git:(make-binary-to-binary) rustc -Vv
rustc 1.58.1 (db9d1b20b 2022-01-20)
binary: rustc
commit-hash: db9d1b20bba1968c1ec1fc49616d4742c1725b4b
commit-date: 2022-01-20
host: x86_64-unknown-linux-gnu
release: 1.58.1
LLVM version: 13.0.0
➜  artichoke git:(make-binary-to-binary) cargo version --verbose
cargo 1.58.0 (f01b232bc 2022-01-19)
release: 1.58.0
commit-hash: f01b232bc7f4d94f0c4603930a5a96277715eb8c
commit-date: 2022-01-19
host: x86_64-unknown-linux-gnu
libgit2: 1.3.0 (sys:0.13.23 vendored)
libcurl: 7.80.0-DEV (sys:0.4.51+curl-7.80.0 vendored ssl:OpenSSL/1.1.1l)
os: Ubuntu 20.04 (focal) [64-bit]

And cargo +nightly doc --workspace appears to "just work". I see you had some failure of this recently, any tips?

Oh sorry I ran into this in #1753 too. The current nightly rust doc has an ICE in spinoso-array. It's fixed in master. I'll run the workflow again tonight when the next nightly is out.
https://github.com/rust-lang/rust/issues/95712#issuecomment-1090020369

And cargo +nightly doc --workspace appears to "just work". I see you had some failure of this recently, any tips?

this command uses whatever version of nightly you happen to have installed. The breakage is only on the current nightly though, which you can get by running rustup update nightly.
rake doc and rake fmt:rust will automatically pull down a nightly if you don't have one, but they won't update it.

new nightly is out, I rebuilt the docs workflow and build is green. we're ready to merge


request to move out of src of folder

I am using the plugin, but as the components are inside src folder, I can't use it using antigen. I found a workaround(hardlinking the contents), but that feels like a sub-optimal solution. You can also consider making it available like oh-my-zsh plugin or prezto plugin. But at the end of day it's your wish.

since I am not an oh-my-zsh user so I didn't make it into omz plugin, you're welcome to submit pull request to make it happen, I am open to accept it :)


Fix broken image link in USERS.md

Kind
/kind cleanup
Release Note
NONE

/override pull-cert-manager-master-make-test
Obviously a flake for a change like this


Feature Request: Display Branch Names in Merge Conflict Resolution UI

During a rebase or merge operation in Git, it can sometimes be challenging to remember what current and incoming refer to. This can make it difficult to understand which version of a file belongs to which branch.
Current UI

I appreciate the UI in Git Fork, which I find more intuitive.

In addition to displaying the branch names, it also shows the Git user for each branch, which can be quite helpful.
Proposed Enhancement
I propose enhancing the merge conflict resolution UI in Visual Studio Code by displaying the branch names. I believe this would make it easier for users to understand the context of the merge or rebase operation they are performing.

We currently do show branch names, next to incoming/current.
I wonder why they don't show in your screenshot.
Can you try to reproduce in a fresh git repository?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Chance details, Chance color coding is not working on homepage

As mentioned, chance details are not working on homepage, but are working on page 2

Is the Enter button also not appearing on the first page? Looks like nothing is being added to giveaways, any error in the console (Ctrl + Shift + J)?

i do not understand these, but here you go, (if i am sharing some private data, pls tell me, i will delete the pic)

You're not sharing any data. But yeah, this looks like the same issue as #1511, so I'll close this as duplicate for now. If it's not fixed with the next version, I'll reopen.


Activate 'Post Comments'?

Hi Guys,
I see that there is already a 'Post Comments' feature integrated here. Could you tell me how to activate it again? Or is there some critical flaw, which lead to it being disabled?
Many thanks!
 Just went back and found the commit where you disabled the comments and rolled back those changes on my install.
Thanks,
Kyle

@cantuket may you say me where I can enable PostComments?

@vadimk-design I'm not sure what commit they removed PostComments, but here is the code that should make it work, though I've never tested it...
Post Comment Model
var keystone = require('keystone'),
	Types = keystone.Field.Types;

/**
 * Post Comments Model
 * ===================
 */

var PostComment = new keystone.List('PostComment', {
	nocreate: true
});

PostComment.add({
	post: { type: Types.Relationship, ref: 'Post', index: true },
	author: { type: Types.Relationship, ref: 'User', index: true },
	date: { type: Types.Date, default: Date.now, index: true },
	content: { type: Types.Markdown }
});

/**
 * Registration
 * ============
 */

PostComment.defaultColumns = 'post, author, date|20%';
PostComment.register();

Post Route (w/ comments)
var keystone = require('keystone');

var Post = keystone.list('Post');
	PostComment = keystone.list('PostComment');

exports = module.exports = function(req, res) {

	var view = new keystone.View(req, res),
		locals = res.locals;

	// Init locals
	locals.section = 'blog';
	locals.filters = {
		post: req.params.post
	};

	view.on('init', function(next) {

		Post.model.findOne()
			.where('slug', locals.filters.post)
			.populate('author categories')
			.exec(function(err, post) {

				if (err) return res.err(err);
				if (!post) return res.notfound('Post not found');

				// Allow admins or the author to see draft posts
				if (post.state == 'published' || (req.user && req.user.isAdmin) || (req.user && post.author && (req.user.id == post.author.id))) {
					locals.post = post;
					locals.post.populateRelated('comments[author]', next);
					locals.page.title = post.title + ' - Blog - SydJS';
				} else {
					return res.notfound('Post not found');
				}

			});

	});

	// Load recent posts
	view.query('data.posts',
		Post.model.find()
			.where('state', 'published')
			.sort('-publishedDate')
			.populate('author')
			.limit('4')
	);

	view.on('post', { action: 'create-comment' }, function(next) {

		// handle form
		var newPostComment = new PostComment.model({
				post: locals.post.id,
				author: locals.user.id
			}),
			updater = newPostComment.getUpdateHandler(req, res, {
				errorMessage: 'There was an error creating your comment:'
			});

		updater.process(req.body, {
			flashErrors: true,
			logErrors: true,
			fields: 'content'
		}, function(err) {
			if (err) {
				locals.validationErrors = err.errors;
			} else {
				req.flash('success', 'Your comment has been added successfully.');
				return res.redirect('/blog/post/' + locals.post.slug);
			}
			next();
		});

	});

	// Render the view
	view.render('site/post');

Post View (w/ comments)
extends ../../layouts/inner

block content
	.container-sm
		a(href='/blog').text-muted ← Back to the blog
		article.full-post
			if !post
				h2 Invalid Post.
			else
				header
					if post.state != 'published' && user.isAdmin
						.mt-1: .alert.alert-danger This Post has not been published yet.
							a(href='/keystone/posts/' + post.id).alert-link Open Keystone
							|  to publish it.

					h1.blog-post__title= post.title
					p.blog-post__byline Posted
						if post.author
							|  by
							a(href='/members', rel='author')=post.author.name.full
						if post.publishedDate
							|  on #{post._.publishedDate.format('MMMM D, YYYY')}
						if post.categories && post.categories.length
							|  in
							each cat, cat_i in post.categories
								a(href='/blog/' + cat.key)= cat.name
								if cat_i < post.categories.length - 1
									| ,
								else
									|

					//- sharing buttons
					.share-buttons
						span.share-button
							.fb-like(data-href="http://www.sydjs.com/blog/post/" + post.slug, data-colorscheme="light", data-layout="button_count", data-action="like", data-show-faces="false", data-send="false")
						span.share-button
							a(href="https://twitter.com/share", data-via="SydJS", data-hashtags="sydjs", data-count="button", data-dnt="true").twitter-share-button Tweet
						span.share-button
							.g-plusone(data-size="medium")
				if post.image.exists
					.post-image.mv-3: img(src=post._.image.fit(720,420)).img-responsive.img-thumbnail
				!= post.content.full

				//- Comment
					.comments
						h4.mb-1=post.comments.length == 0 ? 'Be the first to reply' : plural(post.comments.length, '* comment', '* comments')
						for comment in post.comments
							if comment.author
								.comment.media
									.pull-left
										img(src=comment.author.photo.exists ? comment.author._.photo.thumbnail(60,60) : '/images/placeholders/user-60x60.jpg', alt=comment.author.name.full, width=40, height=40).media-object.img-circle
									.media-body
										h6.mt-0
											=comment.author.name.full
											span.text-muted  · #{moment(comment.publishedOn).fromNow()}
										!=comment.content.html
					if user
						//- Reply form
						form(method='post').comment-form.mt-5
							input(type='hidden', name='action', value='create-comment')
							.media
								.pull-left
									img(src=user.photo.exists ? user._.photo.thumbnail(60,60) : '/images/placeholders/user-60x60.jpg', alt=user.name.full, width=40, height=40).img-responsive.img-circle
								.media-body
									textarea(placeholder='Add your comment...', name='content').form-control
									.mv-1
										button(type='submit').btn.btn-success Submit
block js
	script(src='/js/common/share.js')
	script(src="https://apis.google.com/js/plusone.js")

Hope this helps!

@cantuket Thank you very much!!! I'll try!!! Thank you!


use tree-sitter-systemrdl

Have you considered using tree-sitter for syntax highlighting in VS code?
https://github.com/SystemRDL/tree-sitter-systemrdl

I haven't played around with tree-sitter yet. I'd be open to it.
I'm not particularly attached to the current implementation, as long as the tree-sitter parser you made handles the full feature-set (including Verilog preprocessor)

It handles preprocessor but no syntax highlighting of Perl 5 in it.

For Verilog preprocessor, we can borrow it from this project: https://github.com/tree-sitter/tree-sitter-verilog
In general, parsing any language with preprocessor in one go is hard


Replaced deprecated justDoIt() method with minify()

see https://github.com/css/csso/commit/4d06d099ea09d7cff947279c04459bd14ce5ebbd

Im pretty sure that test failing due to upstream update, and not related to my changes

What's the status on getting this merged? as I'm having to manually fix this after deployments...

@t32k Can this be a new release version (0.7.2)? I would rather not refer to master or a hash in my package.json :smile:


Windows fixes

Fix windows/mingw build issues identified in  #502
Also fix resource search path issues identified during testing.
Should also pull matching PR in scopehal-apps and update the submodule.

Curly braces go on their own line
Why dirname() after PathRemoveFileSpec()? That shouldn't be necessary.
Don't use raw printf(). Use LogTrace(), LogDebug(), LogVerbose(), LogNotice(), LogWarning(), etc. which allow for verbosity level control, IO redirection, pretty printed indentation, etc. These wrappers all have the same semantics as normal printf.

Fixed 1 and 3. Printf was leftover debug.
For 2, this mirrors how this is done on the linux side of the codepath. On mingw64, glscopeclient.exe ends up at
/mingw64/bin/glscopeclient, and it's resources at /ming64/share/glscopeclient
So starting from the binary path, we need to pop the executable name and one more layer of hierarchy (bin) before appending share/glscopeclient.


Tabbed UI in editing page are not marked up as tabs for screen readers

In the editing UI, Wagtail makes it possible to use panels to separate page fields into tabs. Unfortunately those tabs are not marked up as such for screen reader users (tab / tablist / tabpanel ARIA roles).

I was also noticing this the other day! Glad someone opened an issue for it.

Fixed in #8266.


Cannot launch Metatron distributed Druid

Hi,
I'm trying to install Metatron for testing. When I extract the Druid and try to launch it, as it says on the instructions, I get this message:
OpenJDK 64-Bit Server VM warning: INFO: os::commit_memory(0x0000000540000000,<PHONE_NUMBER>, 0) failed; error='Cannot allocate memory' (errno=12)

I don't understand why this error is showing, not even with sudo the command runs.
Could someone give me some light on the subject?
Druid version: druid-0.9.1-SNAPSHOT.201911201314-hadoop-2.7.3
Java: 1.8.0_252
Thanks in advance

@tiago-as In my opinion, you need to compare the memory size of your machine with the memory size set in the druid. If you set the memory size in the druid configuration to be larger than the free memory in your machine, you have to reduce the memory size of druid nodes. Please refer to this link

@kyungtaak thanks so much for the quick answer.
There are 6 files inside the druid directory with the name "runtime.properties" . I believe these files are where those configurations like the "memory size of the druid nodes". Although I'm not really sure which file I am supposed to change in order to lower the druid memory usage.
Any guidelines in this regard?

@kyungtaak changed the machine resources and I was able to launch. Closing the issue.


Why data report no buxj?

Hi, I am using cvxpy to solve a linear optimization problem under some constraints, the problem is the iterating solving procedure does not always remain stable, even there shall be a solution that exists theoretically. I am using MOSEK as the solver,  here is the MOSEK infeasibility report and info after calling MOSEK to analyze the cvxpy output problem file (**.gz).

MOSEK DUAL INFEASIBILITY REPORT.

Problem status: The problem is dual infeasible
The following constraints are involved in the infeasibility.
Index    Name             Activity         Objective        Lower bound      Upper bound
The following variables are involved in the infeasibility.
Index    Name             Activity         Objective        Lower bound      Upper bound
495                       1.000000e+00     1.454947e-01     0.000000e+00     NONE
                Analyzing the problem

*** Structural report
Dimensions
Constraints   Variables     Matrix var.   Cones
358           2346          0             1
           Constraint and bound types
           Free          Lower         Upper         Ranged        Fixed

Constraints:  0             0             0             0             358
Variables:  359           1987          0             0             0
           Cone constraint types
           Quad          R. quad       Primal exp    Dual exp      Primal pow
    Cones  1             0             0             0             0

*** Conic structural report
Quad
Dimension   #
[256,511]:  1
All:  1
*** Data report
Nonzeros      Min           Max
|cj|:  2034          5.7e-03       1.0e+03
|Aij|:  7764          1.7e-05       2.0e+00
           # finite      Min           Max
  |blci|:  358           0.0e+00       1.0e+00
  |buci|:  358           0.0e+00       1.0e+00
  |blxj|:  1987          0.0e+00       0.0e+00

*** Done analyzing the problem
Interior-point solution summary
Problem status  : DUAL_INFEASIBLE
Solution status : DUAL_INFEASIBLE_CER
Primal.  obj: 1.4549473543e-01    nrm: 1e+00    Viol.  con: 0e+00    var: 0e+00    cones: 0e+00
Optimizer summary
Optimizer                 -                        time: 0.00
Interior-point          - iterations : 0         time: 0.00
Basis identification  -                        time: 0.00
Primal              - iterations : 0         time: 0.00
Dual                - iterations : 0         time: 0.00
Clean primal        - iterations : 0         time: 0.00
Clean dual          - iterations : 0         time: 0.00
Simplex                 -                        time: 0.00
Primal simplex        - iterations : 0         time: 0.00
Dual simplex          - iterations : 0         time: 0.00
Mixed integer           - relaxations: 0         time: 0.00
The problem is, why the data report shows me that there is no |buxj| (upper bound for xj), even I define constraints using cvxpy as
cost = cp.sum_squares(lhs_m @ x - rhs_v)
prob = cp.Problem(cp.Minimize(cost),
                  [
                      0 <= g @ x - h,
                      g @ x - h <= 1,
                      z @ x >= 0,
                      x >= -1000,
                      x <= 1000
                  ])

Am I wrong somewhere? any suggestion would be helpful for me.

CVXPY reformulates the problem, canonicalizes, dualizes, so everything ends up elsewhere than you thought and with different signs.
As a simple example, even the older versions of CVXPY which did not dualize, would write a bound such as x>=c for a variable x in the constraint section as constraint of the form -x<=-c, so there would be -c in buc rather than c in blx. Or something along these lines.
If you want any such information to make sense you must save the problem MOSEK actually gets in readable format (for example save_file="something.opf", see https://docs.mosek.com/latest/faq/faq.html#cvxpy) and then relate the infeasibility reports, problem analyzer output, etc., with respect to that file.
If you need full control over this kind of things then use the solver's API directly.

And if you want more stability then consider minimizing cp.norm instead of cp.sum_squares - equivalent problem, but often better numerical properties.

big thanks for your quick response, really useful advice. Calling MOSEK API directly do shall be the better way to debug infeasibility.

CVXPY reformulates the problem, canonicalizes, dualizes, so everything ends up elsewhere than you thought and with different signs.
As a simple example, even the older versions of CVXPY which did not dualize, would write a bound such as x>=c for a variable x in the constraint section as constraint of the form -x<=-c, so there would be -c in buc rather than c in blx. Or something along these lines.
If you want any such information to make sense you must save the problem MOSEK actually gets in readable format (for example save_file="something.opf", see https://docs.mosek.com/latest/faq/faq.html#cvxpy) and then relate the infeasibility reports, problem analyzer output, etc., with respect to that file.
If you need full control over this kind of things then use the solver's API directly.

@aszekMosek Hi, sorry to bother you again, I want to ask, is '-x<=-c' and 'x>=c' exactly the same in modelling a problem(even they are exactly the same thing theoretically)? does add lower bounds explicitly performs better than modelling all constraints as upper bounds? Can the conversion stabilize the computation in some degree when calling MOSEK as a solver, am I right?

If the conversion has any numerical effect then it is accidental and solver-dependent. The conversions are for cvxpy's convenience.
MOSEK has its own presolve that will anyway convert everything into some internal problem according to its own rules.
If you would like to discuss something specifically about MOSEK then I suggest https://groups.google.com/g/mosek may be a better place.


Move JsonObject from vertx-core to its own library

I like JsonObject. It is simple, intuitive, very light-weight and is widely used by the entire Vert.x community. I have seen examples of Vert.x projects that only depend JsonObject and therefore require the one dependency on vertx-core.
While these projects could potentially be valuable to a much broader audience this vertx-core dependency adds way too much overhead for it to be interesting to them.
It would be great having a separate small vertx-json library, that is used by core, but also stands on its own.
I also have 2 of these kinds of projects coming up (near future): vertx-graphql-json-marshaller (to marshal/unmarshal GraphQL schema's to JSON) and vertx-jwt-codec (a more versatile and less opinionated JWT module).
From a marketing perspective: JSON is so widely used these days, that vertx-json could have many users as well. A good way to attract new users to the rest of toolkit!

In order to do this first we need to transform jsonobject/jsonarray into interfaces so we can replace implementations at runtime first.

Can't it just be pulled in by default in vertx-core as a 'system' dependency, so nothing effectively changes on that end?

how would you define a contract for different implementations?

Ah, you meant its needed for polyglot support reasons.

no, i mean say that we have:
io.vertx.json.JsonObject is provided by vertx-json.jar we then reference the class form vertx-core.jar if now you want to replace vertx-json.jar with myown-json.jar you can't break the contact otherwise the JVM will fail to run with ClassNotFound, LinkageException, etc...
We need to have a base contract usually in java it is a interface. So if we have a JsonObject interface then we can replace jars and as long the implementations are correct they should not affect the runtime.

Java 9 modules don't allow the same package name to span on two modules, the io.vertx.core package would span on 2 difference modules, how do you solve that ?
Concerning @pmlopes suggestion about making JsonObject an interface, new JsonObject() would not break.

I'd even say that the default/fallback impl should remain on core since it is a runtime requirement to have always 1 impl present unlike the cluster managers which are not really required to be present to run an application.

So there would be 2 libraries. One with the api which is also referenced by core impl. classes and one lib with the separated implementation classes.

@pmlopes doesn't this imply to replace all occurences of io.vertx.core.json.JsonObject by io.vertx.json.JsonObject in method signatures ?

i was typing out of my head, the signature should remain the same but as you pointed out this would be a breaking change since one cannot instantiate a interface the new JsonObject() will break

Yes. That should not change. Would be too big a break I would think

For my understanding: If JsonObject was moved as-is to its own library with codegen support, with package name unchanged, and pulled in as dependency in core, then there would be no issue, right (other than having small part of core package live in its own jar)? Or are there more things at play?

the constructor would break all the things. Even if we would rename the class to an interface and keep it on the same package, all code like:
new JsonObject()

Would break since you can't instantiate an interface. You would need something like:
JavaObject.create()

Yes sure, that's what I described, but that is not as big a break as changing the package name. It should be easy to migrate to.

Regarding #164 we should be careful since this would become a circular dependency:
core > depends on > common > depends on > core

Yes, sure. But it would be an either / or thing: either not using core, and core may include you, or depend on core and that's it.

This would be a big break imo since i can expect that all existing modules would break. Just grep for new JsonObject in any vert.x related project and if there is 1 occurrence it means that it won't work out of the box on the "next" release.

And my other suggestion of ripping it out as-is, and with separate codegen/docgen support?
Regarding circular dependency: I added suggestion to break into separate topic-specific common modules. Some may use core, others may not. Having as single commons module would mean it would get a lot of transitive dependencies and less attractive to include.

@aschrijver it would cause problem with Java 9 modules because you cannot have the same package name spanning over two different modules (io.vertx.core).

Ah. That's a pity. So it would be either a breaking change or leave as it is now..

Looking through new Java 9 features I see it will also have a light-weight JSON API in java.util. So seems a better idea to investigate its usability as a replacement. That will also be breaking, but makes a lot more sense.

@aschrijver i think it got dropped from JDK9:
http://mail.openjdk.java.net/pipermail/jdk9-dev/2014-December/001670.html
I haven't followed up it latelly.

Yes it seems so. I found this: https://jaxenter.com/json-api-dropped-java-9-113028.html
Many recent feature lists still have it as a bullet point (maybe because don't want to make it even less attractive).

@aschrijver @pmlopes @vietj
Kindly take a look at json-b.net

@EmadAlblueshi it looks nice on first glance, but what is your proposal with regards to vert.x and your project?

@aschrijver I'm thinking out loud if we can utilize it in the future. It looks easier.

just a note: i just looked at the json-b pom and it seems to pull a couple of packages from javaee and CDI which will be big and brings lots of unnecessary code i guess (i did not try it, so i could be totally wrong here).

@pmlopes true

This could be revived for 4.x but for now we can do it since it would break pretty much all existing code.

Marked it as wontfix - understand won't fix for now


Use a different presentation style for virtual properties

Today "real" properties and "virtual" properties (like array ranges) are always rendered in the same style:

I propose to use a different style for virtual properties to make it clear what is a real property of the debugged data structure and what is an artificial node introduced by VS Code.
This style should be used for the virtual properties that are introduced by the generic VS Code UI and for Variables that have a presentationHint.kind === "virtual". The latter makes it possible for debug adapters to flag a Variable to be a "virtual property".

@weinand makes sense. I would simply go with a different color. Or you had something else in mind?

yes, I agree. I was thinking of a different more neutral colour, e.g. grey

I've added a 'virtual' value to the VariablePresentationHint type.

@weinand I have decided to go with opacity: 0.5 to make those properties fade out a bit to make them less important. Did not want to introduce a new color since we already have a bunch. Though I can change all of this easily so let me know what you think

I've verified that this works for a presentationHint.kind: 'virtual' by adding this for mock-debug:


Revive circle

I had to look in an older project to remind myself how to do it. There's a special method, pointToLayer, in the geojson function that allows you to customize how points get added to the layer.

Nice!  Thanks for figuring that one out.

No problem @micheal-w-wells . Let me know if you wanna go through what I did.


Handle bundle artifact naming changes

What this PR does / why we need it:
This PR will handle the newer BYOH Bundles naming convention i.e. byoh-bundle-ubuntu_20.04.1_x86-64_<bundleType>:<k8sVersion> where
bundleType - k8s
k8sVersion - example - v1.22.3
This PR introduces

bundleType type to support future use cases of various bundle types
handles --list-supported output for installer cli

The corresponding bundles (particular to a patch version) should be pushed to the OCI registry of choice
By default, BYOH uses projects.registry.vmware.com

Note: It may happen that a specific patch version of a k8s minor release is not available in the OCI registry

OS                      K8S Version     BYOH Bundle Name
---                     -----------     ----------------
Ubuntu_20.04.*_x86-64    v1.21.*        byoh-bundle-ubuntu_20.04.1_x86-64_k8s:v1.21.*
Ubuntu_20.04.*_x86-64    v1.22.*        byoh-bundle-ubuntu_20.04.1_x86-64_k8s:v1.22.*
Ubuntu_20.04.*_x86-64    v1.23.*        byoh-bundle-ubuntu_20.04.1_x86-64_k8s:v1.23.*

Which issue(s) this PR fixes
fixes: #448 #414
Special notes for your reviewer

This is a backward-incompatible change.
Naming still uses the ubuntu patch version i.e. 20.04.1  to map any ubuntu OS 20.04 patch version to the mentioned artifact.

lgtm. great stuff


Misc doc changes

Besides other documentation changes, this commit includes all documents
to the generated HTML and EPub doc.

@kianmeng Thanks 🙇♂


HTTPONLY setting

In version django-cookie-consent==0.3.2.dev1 the cookie defaults to HTTPONLY = True, which means you can't manipulate the cookie via javascript.
Would be good to be able to override from settings.py with, for example,
COOKIE_CONSENT_HTTPONLY = False
Needless to say I tried this and it didn't work

@kiwipedro Please share with us your javascript or anything else that we might find useful like a traceback or settings.py.
I was able to test django-cookie-consent==0.3.2.dev1 and in the testapp conf.py file https://github.com/jazzband/django-cookie-consent/blob/master/cookie_consent/conf.py I was able to change the value of HTTPONLY False in this file with the testapp and it worked.
You should be able to use COOKIE_CONSENT_HTTPONLY = False though when installing django-cookie-consent into your main project.

Thanks for the reply.  I worked around it by installing it as an app, rather than package. I do realise you can set HTTPONLY server-side, but once it's set and deliver to the browser you can't view it/change it via "document.cookie" property in JS, if HTTPONLY is set to True.
My point was more along that if you release this as a new version, anyone who installs this as a package (not an app) will not be able to set HTTPONLY to False, and it may break websites (like mine) that need access to the cookie from JS.
An override, available in the settings.py of the WGSI project, would be ideal.

Looking at the docs: https://django-cookie-consent.readthedocs.io/en/latest/settings.html#settings, you should be able to configure this with the COOKIE_CONSENT_HTTPONLY setting.
The version on PyPI may not support this setting yet - currently I am unable to publish new versions of this package due to JazzBand restrictions that are not being resolved, see https://github.com/jazzband/django-cookie-consent/issues/64#issuecomment-1256945503

@sergei-maertens that issue was marked as closed. Are you now able to publish the new version?

Yes, 0.4.0dev1 is on PyPI as pre-release. There's one more open issue blocking the full 0.4.0 release!

django-cookie-consent 0.4.0 is now on PyPI, where this setting is available.

Thanks for fixing!


Project/namespace quota not working

Setup

Rancher version: 2.6.3-rc1 (3 node k3s KVM setup, 2 node k3s downstream cluster v1.21.5)
Browser type & version: Chrome 89.0.4389.90 (Official Build)

Describe the bug
Namespace quota for CPU Reservation is not followed.
To Reproduce
Create a project.
Set quota.

Create a namespace. Quota configuration is displayed correctly.

Create a deployment.

Result
The deployment is created without quota setting to be obeyed.
Expected Result
Error about the quota limit.
Screenshots

Additional context

Detailed Description

Context

It looks like the workload does error out eventually, but there's no validation prior to creating the workload, we'll have to decide if we want to warn or prevent this from the UI side

The backend does have a better validator in the web hook that we can use (different than the mutating web hook).

@mantis-toboggan-md @gaktive
Has the code changed in the 2.6.3 rcs? My experience is the quotas worked with 2.6.0-2.6.2.

It looks like..

the project has the correct defaults set spec.containerDefaultResourceLimit(limitsCpu|requestsCpu),
the namespace has the correct max's set annotations['field.cattle.io/containerDefaultResourceLimit] {\"limitsCpu\":\"200m\",\"requestsCpu\":\"300m\"}"
the app.deployment has the correct container specific max set (in post) spec.template.spec.containers[].resources requests: {cpu: "500m"}

So I think either

We set inputs to have max values relative to the namespace available
We do visual validation up front  (just like we show in Namespace when the project has max's set)

We request the backend returns an error when invalid values are supplied (don't think this exists in Steve world?), or make use of some kind of validation hook as mentioned by Gary (@gaktive Not quite sure what that means?)

@vincent99 do you recall how the web hook validator mentioned earlier worked?

@codyrancher Can you take a look?

@maxsokolovsky can you take a look at this? did we address this already?

@peteingithub, what are the values for the resource quota itself on the project? I'm not seeing them on the screenshots. I see container default resource limits, but that's something else. These limits are the default container limits applied, if none are specified. But this is relevant for non-blank quotas. From the docs:
When setting resource quotas, if you set anything related to CPU or Memory (i.e. limits or reservations) on a project / namespace, all containers will require a respective CPU or Memory field set during creation.
But you don't seem to have any resource quota limits.
If you did not set any limits (in the Resource Quotas tab), then this behavior makes sense - you don't have limits in the project's namespaces, so a namespace and a deployment in it can have any resource amounts.

@maxsokolovsky
I'm quite positive the screenshots I added would be the only ones I set (have not been working with Rancher in months).
I guess I misunderstood this. I had the feeling, that the default limits would be the ones the system sets for you unless there were other values already set, hence filling the blanc, overriding non-blanc.
Thanks for clarifying!

Ok, sorry I haven't chimed in sooner... I have a case where this is not working as expected (on Rancher v2.6.3).
Project created with the following:

Resource Quotas

Project Limits

CPU Limit: 4000
CPU Reservation: 4000
Memory Limit: 4096
Memory Reservation: 4096

Namespace Default Limits

CPU Limit: 1000
CPU Reservation: 1000
Memory Limit: 1024
Memory Reservation: 1024

Container Default Resource Limits

CPU Limit: 500
CPU Reservation: 250
Memory Limit: 256
Memory Reservation: 128

One Namespace Created in Project with the following:

Resource Quotas

CPU Limit: 1000
CPU Reservation: 1000
Memory Limit: 1024
Memory Reservation: 1024

Container Default Resource Limits

CPU Limit: 500
CPU Reservation: 250
Memory Limit: 256
Memory Reservation: 128

One Deployment created with the following to override container defaults:

Resource Limits and Reservations

CPU Limit: 10000
CPU Reservation: 5000
Memory Limit: 16384
Memory Reservation: 8192

That deployment SHOULD immediately fail since all of its defined resources exceed every limit set correct? As a test, the exact same setup in Rancher 2.5.12 fails immediately with the typical quota exceeded message.
Am I missing something about how this should work now? Thanks!
Screenshots:
Project:

Namespace:

Deployment:

Deployment YAML:

Deployment Running:

@seanwcom, I just tried your scenario in v2.6.4 and could not reproduce. Kubernetes did not let me create that deployment, since the container resource failed to meet the quota limits (in this case, the namespace default limits).

Oh, great! I'll try to get my lab updated this week, but glad to hear its working there. Thanks for the update.

A coworker just tested this on v2.6.4 and also had the expected error message, so this looks great to me. Thanks all!


Document job conditions

This is a...

[x] Feature Request
[ ] Bug Report

Problem:
I have no idea how to check if a Job has failed and will no longer retry.
JobConditions looks promising, but I don't understand why it is a list.
If there are multiple conditions, which one is current?
I'm guessing that there will only be one condition with status: "True", but no documentation backs up this guess.
I asked a question on stack overflow and haven't received an answer.
Proposed Solution:
Include documentation in the job docs to explain what status.conditions is, why it is a list, and how to interpret the various fields.
Page to Update:
https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion/#job-termination-and-cleanup

/remove-lifecycle stale

/remove-lifecycle stale


[Agreements] Add back XmacsLabs.Mogan

cc @denelon @stephengillie
Installer prompt:

Is the AgreementLabel necessary? I think it is.

Strictly speaking, the CLI will work without the Label, but it is probably a best practice to include it


SW-5926 Create Google Drive folders on demand

Currently we require an admin to configure the Google Drive folder to use for
uploaded documents for projects, but that doesn't work well for applications since
they won't have been reviewed yet at the time they need to start uploading
documents.
Add a config option that specifies a Google Drive folder in which new project
folders will be created on demand for projects that don't currently have folders.

#2346  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @sgrimm and the rest of your teammates on  Graphite


feat(lighthouse-service): Add compared value to payload (#5496)

This PR

When receiving an SLI result each metric is compared with previous values, those are aggregated following a certain aggregation strategy defined in the SLO. In case of comparison criterias this values stays the same for all comparisons and influence the target values used to decide whether the metric passes or fails.
In this PR we store the aggregated value in each evaluation result as ComparedValue, to facilitate understanding of why the metric passes or fails a comparison criteria.
For criterias with fixed targets the pass or fail is dependent only on the TargetValue, anyhow ComparedValue is calculated.

unit tests modified
integration test checked

Related Issues

Fixes #5496

LGTM!


event names are not upgraded

when a core-signals element is found, it is correctly converted to iron-signals, but the on-core-signal-??? is left as on-core-signal-???

Should be possible to add event names to elementMapping.js and then apply them.


all administrative units excel sheet

Dear @njoguamos,
Great job on making this data publicly available. I would like to know if it is possible to get the complete list of administrative units. The excel sheet in the repo ( specifically the 5th sheet) is incomplete and only lists Mombasa data only.

@gicharu thanks. I have been following up with Kenya National Bureau  of Statistics on complete data set. I have not been successful. You can check later.


How to weave a Jupyter notebook to pdf?

I tried to use the documentation example weave("notebook.ipynb") but the result was notebook.md.

On Weave master, try to run Weave.notebook("notebook.ipynb"), this should be fixed by #196

weave("notebook.ipynb", docformat = "md2pdf") should work.


Convert providers to use wit-bindgen-wasmcloud-provider

Once the providers w/ wasmcloud-provider-sdk usage have stabilized, providers should be converted to use wit-bindgen-wasmcloud-provider.

Not stale, this is in progress

@vados-cosmonic are any providers left here, or can we close this as completed?

I don't think we can close this as completed just yet since the sqldb-* providers are still left -- they do require some fundamental work though, because of how they were implemented originally. A straight forward port would be fastest, but the contract really needs to change.
Maybe we can close this as completed since those other providers require fundamental changes?

Ah right. Up to you. I'm okay closing and filing separate tasks for the sql providers, since they'll use new contracts

Filed an issue for the other contracts, closing this for now!