./nant fails on Compiler and booc

I would really love to start using Boo and try to contribute to the community, but, it seems that the build process has some issues. I'll divide this into 2 parts, the Compiler and the booc.
Compiler
When I first started the nant script, the output was:

NAnt 0.93 (Build 0.93.5019.0; dev; 28.9.2013)
Copyright (C) 2001-2013 Gerry Shaw
http://nant.sourceforge.net

Buildfile: file:///E:/Boo/boo/default.build
Target framework: Microsoft .NET Framework 4.5
Target(s) specified: rebuild

 [property] Read-only property "skip.antlr" cannot be overwritten.

clean:

[clean delete] Deleting directory 'E:\Boo\boo\build'.

rebuild:

[rebuild mkdir] Creating directory 'E:\Boo\boo\build'.

init:

sign:

Boo.Lang:

[Boo.Lang csc] Compiling 59 files to 'E:\Boo\boo\build\Boo.Lang.dll'.

generate-ast:

   [delete] Deleting directory 'E:\Boo\boo\src\Boo.Lang.Compiler\Ast\Impl'.
    [mkdir] Creating directory 'E:\Boo\boo\src\Boo.Lang.Compiler\Ast\Impl'.
     [exec] E:\Boo\boo\default.build(1060,5): Attribute 'useruntimeengine' for <exec ... /> is deprecated.

Boo.Lang.Ast:

Boo.Lang.Compiler:

[Boo.Lang.Compiler csc] Compiling 592 files to 'E:\Boo\boo\build\Boo.Lang.Compiler.dll'.
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(58,32): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(58,44): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(60,27): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(60,37): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(62,28): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(62,36): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(64,23): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(64,29): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(66,33): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(66,47): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(68,31): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(68,43): error CS1519: Invalid token ';' in class, struct, or interface member declaration
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(70,39): error CS1002: ; expected
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(70,47): error CS1519: Invalid token ';' in class, struct, or interface member declaration

BUILD FAILED - 0 non-fatal error(s), 2 warning(s)

E:\Boo\boo\default.build(506,4):
External Program Failed: C:\Windows\Microsoft.NET\Framework64\v4.0.30319\csc.exe (return code was 1)

Total time: 2.6 seconds.

So I fumbled around with the code in UnmanagedResourceHelper.cs, which helped it pass to the next sections of the builder (although it still showed some errors).
booc
When it went through to the next sections after the changes made in the UnmanagedResourceHelper.cs file, I received this error which i can't really change taking into consideration that it says that the problem is inside the .exe (or atleast I didn't catch where it breaks). The log is:

NAnt 0.93 (Build 0.93.5019.0; dev; 28.9.2013)
Copyright (C) 2001-2013 Gerry Shaw
http://nant.sourceforge.net

Buildfile: file:///E:/Boo/boo/default.build
Target framework: Microsoft .NET Framework 4.5
Target(s) specified: rebuild

 [property] Read-only property "skip.antlr" cannot be overwritten.

clean:

[clean delete] Deleting directory 'E:\Boo\boo\build'.

rebuild:

[rebuild mkdir] Creating directory 'E:\Boo\boo\build'.

init:

sign:

Boo.Lang:

[Boo.Lang csc] Compiling 59 files to 'E:\Boo\boo\build\Boo.Lang.dll'.

generate-ast:

   [delete] Deleting directory 'E:\Boo\boo\src\Boo.Lang.Compiler\Ast\Impl'.
    [mkdir] Creating directory 'E:\Boo\boo\src\Boo.Lang.Compiler\Ast\Impl'.
     [exec] E:\Boo\boo\default.build(1060,5): Attribute 'useruntimeengine' for <exec ... /> is deprecated.

Boo.Lang.Ast:

Boo.Lang.Compiler:

[Boo.Lang.Compiler csc] Compiling 592 files to 'E:\Boo\boo\build\Boo.Lang.Compiler.dll'.
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\TypeSystem\Generics\GenericConstructedType.cs(49,15): warning CS0659: 'Boo.Lang.Compiler.TypeSystem.Generics.GenericConstructedType' overrides Object.Equals(object o) but does not override Object.GetHashCode()
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(38,33): warning CS0649: Field 'Boo.Lang.Compiler.Resources.Win32Resource._typeName' is never assigned to, and will always have its default value null
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(279,34): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.pstringType' is never assigned to, and will always have its default value null
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(280,34): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.pstringName' is never assigned to, and will always have its default value null
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(282,24): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.DataSize' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(283,24): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.HeaderSize' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(286,24): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.DataVersion' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(287,23): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.MemoryFlags' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(288,23): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.LanguageId' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(289,24): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.Version' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(290,24): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.Characteristics' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(291,25): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE.data' is never assigned to, and will always have its default value null
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(296,23): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE_STRING.Ordinal' is never assigned to, and will always have its default value 0
[Boo.Lang.Compiler csc] e:\Boo\boo\src\Boo.Lang.Compiler\Resources\UnmanagedResourceHelper.cs(297,25): warning CS0649: Field 'Boo.Lang.Compiler.Resources.RESOURCE_STRING.theString' is never assigned to, and will always have its default value null

generate-errorpatterns:

     [echo] WARNING: A recent version of Boo is required!
     [echo] Run again Nant (without clean or rebuild) to actually generate the patterns

Boo.Lang.Parser:

[Boo.Lang.Parser csc] Compiling 125 files to 'E:\Boo\boo\build\Boo.Lang.Parser.dll'.
[Boo.Lang.Parser csc] e:\Boo\boo\src\Boo.Lang.Parser\WSABooParsingStep.cs(33,27): warning CS0672: Member 'Boo.Lang.Parser.WSABooParsingStep.ParseModule(string, System.IO.TextReader, Boo.Lang.Parser.ParserErrorHandler)' overrides obsolete member 'Boo.Lang.Parser.BooParsingStep.ParseModule(string, System.IO.TextReader, Boo.Lang.Parser.ParserErrorHandler)'. Add the Obsolete attribute to 'Boo.Lang.Parser.WSABooParsingStep.ParseModule(string, System.IO.TextReader, Boo.Lang.Parser.ParserErrorHandler)'.
[Boo.Lang.Parser csc] e:\Boo\boo\src\Boo.Lang.Parser\BooParsingStep.cs(92,26): (Location of symbol related to previous warning)
[Boo.Lang.Parser csc] e:\Boo\boo\src\Boo.Lang.Parser\BooParser.cs(65,11): warning CS0612: 'Boo.Lang.Parser.BooParser.CreateParser(int, string, System.IO.TextReader, Boo.Lang.Parser.ParserErrorHandler)' is obsolete
[Boo.Lang.Parser csc] e:\Boo\boo\src\Boo.Lang.Parser\BooParsingStep.cs(74,7): warning CS0612: 'Boo.Lang.Parser.BooParsingStep.ParseModule(string, System.IO.TextReader, Boo.Lang.Parser.ParserErrorHandler)' is obsolete

corecs:

booc:

 [booc csc] Compiling 3 files to 'E:\Boo\boo\build\booc.exe'.
[booc copy] Copying 1 File to 'E:\Boo\boo\build'.
[booc copy] Copying 1 File to 'E:\Boo\boo\build\booc.exe.config'.

Boo.NAnt.Tasks:

[Boo.NAnt.Tasks exec] E:\Boo\boo\default.build(312,4): Attribute 'useruntimeengine' for <exec ... /> is deprecated.  Use the managed attribute and Managed property instead.
[Boo.NAnt.Tasks exec] Boo Compiler version <IP_ADDRESS> (CLR 4.0.30319.42000)
[Boo.NAnt.Tasks exec] BCE0011: An error occurred during the execution of the step 'Boo.Lang.Compiler.Steps.EmitAssembly': 'Object reference not set to an instance of an object.'.
[Boo.NAnt.Tasks exec] 1 error(s).
[Boo.NAnt.Tasks exec] 5 module(s) processed in 700ms after 40ms of environment setup.

BUILD FAILED - 0 non-fatal error(s), 8 warning(s)

E:\Boo\boo\default.build(312,4):
External Program Failed: E:\Boo\boo\build\booc.exe (return code was 127)

Total time: 7.3 seconds.

I would really appreciate the help since I really love both Python and C#, and I would really like to try implementing Boo where I can, or maybe create some OSS for it.

On the first part, what version of the C# compiler are you using?  The errors you're seeing in UnmanagedResourceHelper makes it appear that it doesn't support expression-bodied properties, which were added in C# 6.0.
For the second part, having that error come up is quite annoying.  I don't see that when I run it locally; is there any way you can reproduce it under the debugger?  Without a stack trace, it's very difficult to guess what's going wrong.  I've seen it a few times and it usually means something screwy is going on with generics, but that's just a wild guess.

It seems, from what I can tell, that the problem is with NAnt, it's taking the Windows built-in .NET Framework 4.5 which doesn't use C# 6.0. I will try to tweak it bit by bit, so as to make it work with what I got with NAnt.
If there is anyway to set the path differently, and set all of the packages it might need from the .NET 4.6 through 4.8 (i tried working on some of the Nant.exe.config, but there were some variables that i don't know where they came from) do tell me.
Sorry for the late reply btw!

Have a look at https://stackoverflow.com/questions/15116265/how-to-specify-msbuild-version-in-nant-script and see if any of the answers offered there are helpful.

Looks like i missed the mark with the C# Versioning, i changed the small pieces that were with C# 6.0 to be compatible with the one i have, it works (it shows a couple of errors, but they also showed when i ran the file with the csc of the .NET Framework 4.6 which u can see in the booc log that was originally attached under the Boo.Lang.Compiler)
Now the only problem left is with the booc, already tried:

Running the same booc command through powershell
Searched through bits of the code to see what exception propped up the error (with no luck)

I am lost as to what to do next.
I plan on spinning up a Linux VM in Hyper-V and see how the installation behaves there.

I am lost as to what to do next.

At this point, what I'd do is modify your local copy of CompilerErrorFactory.StepExecutionError to include the stack trace of the exception in the reported error.  Then you'd have some more context about what's going wrong inside the compiler.

Managed to fix it thanks to the full stack trace, it built successfully! The problem seemed to be that i changed UnManagedResources so that it would be compatible with the C# that Windows is using, with which in turn I've set some Assembly parameters to null (VersionResource if i'm correct). I just set all of those to " ", and it compiled without errors.
Now I'm running into an issue with the powershell script for the nunit, but I'll try a simple hello world and see if it works. Will post an update.

Happy to hear it's making progress for you!

It seems to be working. Thanks a lot! Couldn't have done it without you. One final question, are there any libraries/frameworks made specifically for Boo, and if so, are there any backend ones since I'm a backend dev.
Will close the issue after the answer :)

If backend web dev is your thing, have a look at Web.Boo, a server framework I've been working on for the last few years that uses metaprogramming to set up routing and make it quick, simple, and Reflection-free.  It's still very much a work in progress, but it works well.

As an addendum, I was suffering from the first error, but I recently discovered a way to fix it without modifying any of Boo's source itself. in /build-tools/tools/nant/NAnt.exe.config, I replaced the sdkdirectory and frameworkdirectory attributes of <framework name="net-4,5" ... > (line ~900). Changing them from ${path::combine(installRoot, 'v4.0.30319')} to C:\Program Files (x86)\Microsoft Visual Studio\2019\Community\MSBuild\Current\Bin\Roslyn seems to convince NAnt to use the updated compiler that you can get with Visual Studio (note: this requires Visual Studio and the Visual C# stuff to be installed).
I didn't change frameworkassembydirectory as it complained it could not find System.dll. I left it as is ${path::combine(installRoot, 'v4.0.30319')}. It may cause booc to break, but booi and booish work fine (as far as I can tell). The CI script passed without failures. I probably just need to find the right directory...


[RESOLVED] High ICMP ping response time

Problem started at 12:47:29 on 2023.08.04
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Operational data: Value: 205ms
Original problem ID: 34600
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23268&eventid=34600

Problem has been resolved in 2m 0s at 12:49:29 on 2023.08.04
Problem name: High ICMP ping response time
Host: HP_iLO
Severity: Warning
Original problem ID: 34600


Embed C Library Directly into project

Hi @stephencelis,
I'm not sure the reasons against this (so forgive me please (; ) and it might have already been discussed (couldn't find the discussion) but this would solve a few problems:

The issues with cocoapods and the sqlite3 header file
iOS SQLite version is very much behind.
SQLite 3.8-3.12.0 has many speed improvements.
FTS5 is becoming more stable and useful (bm25 ranking, speed). JSON1 extension is cool too. If this project really wanted to utilize these features without requiring iOS 11 then embedding the C library would be required anyways.

FMDB does this using a subspec (Cocoapod). Couldn't this be done in this project as well?

FYI, I am completely for this, we've effectively abandoned using the SQLite from iOS because we require a stable version to develop against.

I definitely like the idea behind this, but I have a few concerns with the current implementation:

As @kdubb touched upon, can we use a git submodule and CocoaPods subspec instead of embedding everything directly in the repo? Using a git submodule makes it much easier to keep things in sync as the upstream SQLite library sees new releases and keeps the commit diff/noise down.
I also would prefer not to require every developer to bundle SQLite with their app. I haven't done the math, but I imagine the app payload to increase by a nontrivial amount.

I've tried a few other options but CocoaPods is so far the best way to handle this. #431 adds a subspec which downloads the latest stable version of sqlite and includes it in the build.
Not sure how this could be done elegantly with submodules / Carthage – it would probably require separate targets (linking/framework) which would complicate the build even further.

SQLite can be integrated via CocoaPods now. closing.


Child of bone with zero scale at rest pose doesn't get exported visibly?

Describe the bug
In a Blender file with one armature containing one bone and a mesh object which is a child of that bone, the mesh object does not get exported correctly if the export is run while the active NLA track has the bone at zero scale.
Distinct from #1087, I think, because the export does run correctly and the file does contain most of the expected data— I think something's just wrong with the transforms in it?
To Reproduce
Steps to reproduce the behavior:

Get a fresh file.
Delete everything.
Add an armature.
Add a Suzanne.
Parent the Suzanne to the armature's single bone.
Create an NLA track on the armature pose with bone scale keyframes and no changes.
Create a second NLA track on the armature pose with the bone at zero scale.
Solo the second NLA track on the armature pose with the bone at zero scale.
Export with default settings.

(Zero-scale state at export.)

(Default-scale NLA.)

Expected behavior
The mesh object is invisible in the zero-scale animation and visible in the one-scale animation, like when the export is run from the default-scale state:

Observed behaviour
The mesh is invisible in both animations:

.blend file/ .gltf
.blend file.
GLTF_ZEROVIS.blend.zip
.gltf export from zero-scale NLA (failure case).
GLTF_ZEROVIS.gltf.zip
.gltf export from default-scale NLA (working case).
GLTF_ONEVIS.gltf.zip
Version

Blender Version 2.83.4.
three.js r116.

Additional context
Does GLTF store bone orientation separately from head and tail position?
I encountered this while trying to make a smaller reproducer for a similar apparent issue where the child meshes are visible but have wrong rotations.

Are you sure the Three.js viewer just isn't centering the camera wrong? You can test it by sticking some Cube nearby in the scene to use as a reference.
If I add a cube to your .blend and export I get this

@scurest
Ah, oops. 😅
In my defence, I was trying to narrow down another apparent issue that seems to happen with zero-scale bones (where rotation rather than visibility gets messed up), so I figured broken visibility might be a special case that happens when there's nothing else in the scene. Although, I probably should have at least opened the exported file in Blender to double-check that something was wrong.


Remove button border-radius in Chrome 62+ on macOS.

Chrome 62 buttons have been restyled to resemble macOS buttons:
https://www.chromestatus.com/features/5743649186906112
Other browser vendors haven’t signalled interest.
This leads to cross-browser inconsistencies in existing UIs.
This patch matches developer expectations of buttons not having border-radius set.

Why did you close it?

Sorry, false alarm. See #697


Workaround windows builds suddenly failing

I'm going to temporarily comment the line in the travis.yml file causing the problem. We can later re-enable it when whatever is causing the breakage is actually fixed. The line relates to turning off antivirus slowdown.
Part of https://github.com/quantumlib/Cirq/issues/2283

Automerge cancelled: A status check is failing.

Comment out all the power-shell commands?

I think it just flaked from the thing @c-poole is looking into. It made it past the commands.


Upgrade Android Studio and gradle-profiler

To dogfood support for Android Studio Jellyfish.

@bot-gradle test APT AST

I've triggered the following builds for you. Click here to see all build failures.

AllPerformanceTestsReadyForRelease build
AllSmokeTestsPullRequestFeedback build


docs: fix url for full test report

The test coverage page didn't quite work because I used the wrong URL. This new one should work.

what about publishing this on https://github.com/uutils/coreutils-docs too?

How does that work exactly? Does that happen automatically? Or do we need to trigger the action?

yeah, a cron job https://github.com/uutils/coreutils-docs/blob/main/.github/workflows/docs.yml#L11


Zinc API (Java API)

Java API (
https://github.com/sbt/zinc/blob/1.0/internal/compiler-interface/src/main/java/xsbti/compile/IncrementalCompiler.java) should be able invoke incremental compilation from soup to nuts like - https://github.com/sbt/zinc/blob/1.0/incrementalcompiler/src/test/scala/sbt/inc/IncrementalCompilerSpec.scala

migrating comments from duplicate issue #124.
Currently, I find it extremely hard to setup Zinc for incremental compilation, especially from Java.
I was pointed to this example for setting up Zinc: https://github.com/sbt/zinc/blob/1.0/zinc/src/test/scala/sbt/inc/IncrementalCompilerSpec.scala
The first problem I have is that there while IncrementalCompilerUtil.defaultIncrementalCompiler() has return type IncrementalCompiler (which is fine) the IncrementalCompiler interface has no useful methods to create a compiler for Java sources.
Second problem is that while the IncrementalCompiler interface does provide a method for compiling (compile) it does so by means of param type 'Inputs'. That in turn drives me to the internal implementation class IncrementalCompilerImpl which I really don't want to know anything about. Trying to fill in all the blanks here utilizing, in examples the inputs(), setup() and compilers methods, drives me nuts, and I finally gave up trying to make it work.
There should really be a simple API for all of this, perhaps even something similar to that of what Zinc originally provided in the Typesafe repos.
Ideally, you would provide something that would allow easy setup for Java-only mode also, but that's not a hard requirement.

We're covering this one here -> https://github.com/sbt/zinc/issues/300.
Closing.


Get DB exposure exporting working

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 86 (11.63%)  changed or added relevant lines in 6 files are covered.
7 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.04%) to 77.332%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

mirar/pipelines/winter/models/_exposures.py
0
1
0.0%

mirar/processors/sqldatabase/database_exporter.py
0
2
0.0%

mirar/io.py
0
5
0.0%

mirar/pipelines/winter/load_winter_image.py
1
69
1.45%

Files with Coverage Reduction
New Missed Lines
%

mirar/pipelines/winter/models/_exposures.py
1
88.31%

mirar/processors/sqldatabase/database_exporter.py
1
53.7%

mirar/io.py
2
61.33%

mirar/pipelines/winter/load_winter_image.py
3
11.06%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
8010

Relevant Lines:
10358

💛  - Coveralls


fix: debezium apicurio properties name in .properties file

Done in https://github.com/bf2fc6cc711aee1a0c2a/cos-fleetshard/pull/435


[Radio][Joy] support componentsProps as a function

follows the MUI Base componentsProps pattern (support callback) which remove the need for classes to style between states.
update typings
update demos to use the callback

Before:
<Radio sx={theme => ({
  [`&.${radioClasses.checked}`]: {
    [`& .${radioClasses.action}]: { ... }
  }
}) />

After:
<Radio componentsProps={{
  action: ({ checked }) => ({
    sx: theme => ({
      ...checked && { ... }
    })
  })
}} />

[x] I have followed (at least) the PR section of the contributing guide.

@michaldudak If the changes look good, I will apply with all of the Joy components.


Update pom.xml

normally, an extension is not type pom, but type jar, commented out
latest parent is 2.5
new extension project should start with SNAPSHOT version

Thanks Jan!


help: send doip packet only on UDP

Hi jacob,
I would like to know how to send doip packet to the udp port 13400 and only the UDP one, not on the tcp port that would be the default.
BR.

My query is more oriented network question.
When i initiate a doipclient under a TCP connection it not possible to test if the ECU is available or not (answer or not) and the connexion exeed after 2 min (expiration from TCP normalisation)


Fix @JmsListener handling for all payload types.

This should fix @JmsListener handling.
@pavolloffay @malafeev ^^

Does it make sense to move most of these classes to
java-jms-spring ?
There is an propagation issue
which can be fixed only using this approach.

@malafeev yeah, I guess we could move them there ...
In the long run, if all goes well, there should be a nicer solution to all of this -- one it gets in Spring core.

it would be really great to have spring instrumentations in core modules without dependency on spring cloud or spring boot
It would solve problem to add instrumentation (spring jms, kafka, rabbit etc )to projects which don't use cloud and boot.

@malafeev just moved JMS classes to java-jms. This PR should be opened against that repository.

I will keep this as pointer until it's resolved in java-jms. Or @malafeev would you do the fix, It seems that you have more free loops on tracing :)

all classes from this PR moved to java-jms.
java-jms misses only  different variants of JmsListener tests.
But this PR can be closed.

@malafeev thanks then I will close this.


fix: indefinite spinner triggered by error in add operation

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update OpenCore to 0.8.6 w/ macOS Ventura fixes

(Latest update): https://dortania.github.io/hackintosh/updates/2022/11/07/acidanthera-november.html

https://dortania.github.io/hackintosh/updates/2022/11/07/acidanthera-november.html

TODO:

[ ] Remove CPUFriend - i7 10510u and i5 10210u CPUs have native CPU power management on macOS.
[ ] Tune config for Apple GuC Firmware (-igfxfw boot arg)

Removed IMEI ACPI patch quirks

Will need to do a OC clean snapshot in propertree; was using an older version of propertree without 0.8.6 serials/etc.

Currently investigating a kernel panic around Booter > Quirks:
https://dortania.github.io/OpenCore-Install-Guide/troubleshooting/extended/kernel-issues.html#kernel-panic-on-invalid-frame-pointer

Working on updating kexts, but I'm able to get HDMI working so far after updating Lilu & WhateverGreen w/ the updated config.

Working on updating kexts, but I'm able to get HDMI working so far after updating Lilu & WhateverGreen w/ the updated config.

Hey! Good to see you again @Qonfused  :)
Sorry for not participating too much in the bug search ^^' I'm studying Law (History of administration, what a bore...), and literally, it takes me DAY AND NIGHT, Weekend included...
I've been trying to figure out the whole conversation, so for now, the Screenpad is still "buggy", but now, the HDMI connection would work on the INTEL? chip (I'm talking about Intel because Nvidia is not supported...?) on an external monitor without any difficulty? that would be awesome to hear!
Is that right?
UsedDiscord / Magik (my new pseudo as well)

Yeah that's correct. I also noticed that both trackpad buttons also work but are mapped to the wrong functions (can be fixed).

Yeah, that's good news!
Concerning the trackpad, I had noticed that, but is it true? or did I use my trackpad wrong? It's up to you to check it too, so I don't say anything stupid...

There were some additional plugins that needed to be imported from the voodoo kexts that weren't explicitly imported after doing a propertree clean snapshot; this may have affected this behavior since beforehand I noticed the same behavior as you (right click button not working).
As long as both buttons function it shouldn't be difficult to fix.

Indeed, it tells me something about this story of snapshot...

A lame fix would be to use karabiner, which works but adds unnecessary additional work for people to do.

Exactly, [https://karabiner-elements.pqrs.org/docs/help/how-to/mouse-button/](Karabiner Doc Mouse Button) but for a first preview, that is good i think, so i think try that when i have the time...


Tagging items in bank / inventory colors for Barrows & other degrading gear should consider the other %'s

Basically, if I tag a degradable item like barrows gear and it degrades, I have to retag the other versions (75%, 50%, 25% broken). I believe tagging the gear should be default consider the other versions so we don't have to retag the gear. This would be useful for raiders especially because sometimes we can forget the exact armor we had if it degrades and it's missing from the bank.

Shift drag the item to a bank tag and it will tag all %s. Alternatively, tag it with a *


Add doap.rdf for DS to be listed in https://projects.apache.org

Codecov Report

Merging #14883 (8f6b55c) into dev (3fa69d2) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Current head 8f6b55c differs from pull request most recent head 7833349. Consider uploading reports for the commit 7833349 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14883      +/-   ##
============================================
- Coverage     38.88%   38.87%   -0.02%
+ Complexity     4585     4583       -2
============================================
  Files          1232     1232
  Lines         43243    43211      -32
  Branches       4772     4755      -17
============================================
- Hits          16817    16798      -19
+ Misses        24566    24554      -12
+ Partials       1860     1859       -1

see 24 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove session based functionality

A session is a non-standardized concept that is mostly implemented by a framework or the standard lib of a programming language. Play uses a stateless approach by using a cookie to store session data on the client. There exists also a stateful approach which stores session data on the server side and a reference (ID) to this data on the client side. Silhouette already supports this concepts with cookies and stateful or stateless authenticators. There is also no necessity to support a custom session approach like Play does. This would bring only complexity to the Silhouette API.

Coverage decreased (-0.5%) to 96.062% when pulling 7df8de02a0a50c41409d0a911d75538694b0fccf on akkie:remove-session into e1425d2a094220500e29eb5bec7f19c757b4bbc9 on minutemen:master.


Any plans to make this reactive?

Was wondering if you had any plans to make the data reactive?

Yes, I will do it.


Add dialog component, add missing line in core component generator

Issue https://github.com/basemate/matestack-ui-core/issues/185: Add dialog component
Changes

[x] Added tested & documented dialog component
[x] Added closing line to HTML snippet in core component generator

@marcoroth @michaelrevans happy new year & thanks for the review, m8s!


Mempool buy problem

Sukiyaki Mempool Buy
Fail Reason:
insufficient funds for gas * price + value
Kept getting Trade Disabled, even though I had Auto-Gas Limit.
63e3c8d365e52dabbf3d556e
63e3c6b7b8cb40932151411c
63e3cc22b8cb409321514121
63e3c897f0e176d576361543

First time i hear this, didn't knew. So yeah, better to disable it.


[1.9.4] - Waila capacitor block name and icon missing

Issue Description:
Icon for capacitor block when placed in the world and block description missing. :
http://i.imgur.com/Mnj6vZb.png
Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.9.4-<IP_ADDRESS>_alpha
EnderCore: EnderCore-1.9.4-<IP_ADDRESS>-alpha
Minecraft: 1.9.4
Forge: forge-1.9.4-<IP_ADDRESS>7-universal.jar

That's a bug in WAILA. One of many.


Replace tabs with spaces

Tabs were used not just for indenting to the correct scope for but lots of other instances of vertical alignment. This is why the first parameter of a function may be offset from the (. It allowed for the parameters on subsequent lines to vertically align. Eventually we'll start using clang-format.

Looks good :)
Out of interest, are tabs the equivalent of 3 or 4 spaces?

 I picked 4 spaces except in a few places where I couldn't tell what the author's intent was like in code comments or strings that made up shader code.


WritableBytesConverter always falls back to JdkBytesConverter

What kind an issue is this?

[X] Bug report. If you’ve found a bug, please provide a code snippet or test to reproduce it below.
The easier it is to track down the bug, the faster it is solved.
[ ] Feature Request. Start by telling us what problem you’re trying to solve.
Often a solution already exists! Don’t send pull requests to implement new features without
first getting our support. Sometimes we leave features out on purpose to keep the project small.

Issue description
WritableBytesConverter always falls back to JdkBytesConverter. I would expect a return after invoking the safeWritableConverter, or a else statement. Previously, this was working correctly, but below commit introduced the behavior of always using the fallback.
https://github.com/elastic/elasticsearch-hadoop/commit/08b2986f5f2759aa9d01a4701cb53b7b6d87a85b
Steps to reproduce
Code:
Below code is where the unexpected behavior is:
@Override
public void convert(Object from, BytesArray to) {

    if (safeWritableConverter != null)
        safeWritableConverter.invoke(from, to);

    super.convert(from, to);
}

Strack trace:
N/A
Version Info
ES-Hadoop   :  7.17.9

Thanks for the report, and that looks to me like a bug in the commit you linked, too.

@jasperiq what are you doing to hit this (I want to test it as much like your use as possible)? Thanks.

I think this sums it up:
MultipleOutputs<NullWritable, Text> outputs = new Multiple ;
String namedOutput = "some named output";
String documentJson = "any kind of json";
outputs.write(namedOutput, NullWritable.get(), new Text(documentJson));

It goes wrong when wrapping the documentJson in a Text writable. Also when I wrap byte[] in Text it goes wrong.
This actual issue I have is that because of it does the toString() in the JdkBytesConverter it becomes inefficient. I start with String, in Text this is translated into a new byte[], JdkBytesConverter creates another String, and because of this String, BytesArray needs to make sure it's in UTF-8 and creates a byte[] again. This all makes me run into heap when writing larger documents.
I worked around this by directly write byte[] to the MultipleOutputs for now. However, looking at the code, I think the intention was to support Text writables in an efficient way as well.
I hope this explains my issue enough. Let me know if you have more questions.

OK thanks. I think my change in #2075 will fix that.

Indeed!


Update and rename settings.yml.sample to settings_sample.yml

added support for generic LDAP. changed AD to AD/LDAP

@gabrielsroka thanks for opening this one up! I've incorporated the idea here into #33, and I've also added additional logic to detect the differences between OpenLDAP and Active Directory. Once that merges I'll close this as superseded 🙇


[Actionmenu] Reduced example heights

Description
Component Checklist 📝

[ ] JSDoc
[x] Examples
[ ] Documentation
[ ] Storybook
[ ] Style mappings (@navikt/core/css/config/_mappings.js)
[ ] Component tokens (@navikt/core/css/tokens.json)
[ ] CSS class deprecations (@navikt/aksel-stylelint/src/deprecations.ts)
[ ] Exports (@navikt/core/react/src/index.ts and @navikt/core/react/package.json)
[ ] New component? CSS import (@navikt/core/css/index.css)
[ ] Breaking change? Update migration guide. Consider codemod.
[ ] Changeset (Format: <Component>: <gitmoji?> <Text>. E.g. "Button: :sparkles: Add feature xyz.")

If only it was possible for CodeExamples to know if the example had variant=static... Maybe we need a separate prop for padding in CodeExamples. 🤔

I guess we could look for the "examples_containerStatic__*" class name, but that feels hacky 😅

Adding a data-example="static"-attrb could work


fix(cli): fix require error issue related to update-notifier import

Closes: #392

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

v5.2.2
GitHub release

Your semantic-release bot :package::rocket:


[Camera2Video] Pixel 5 4K cannot record

Device: Pixel 5
System: Android 12 (S2B1.211112.006)
Phenomenon 1:
Without any modification, select "Back(0) 3840x2160 60 FPS" and the camera has a black screen
Phenomenon 2:
After adding set(CaptureRequest.CONTROL_AE_TARGET_FPS_RANGE, Range(args.fps, args.fps)) in here, "Back(0) 3840x2160 60 FPS" is displayed normally, modifiedFPS is 30 cameras Black screen
Notice:

Pixel 6 Pro performs as expected
Add CaptureCallback#onCaptureFailed in here to get CaptureFailure#REASON_ERROR

Thank you for reporting the issue:  will take a look at it (I have not get chance to look at Camera2 examples).


split repository.asciidoc into multiple files

This one file repository.asciidoc contained almost the whole spec, so I moved some of that material into an unused file that already existed, and some into a new file.
I and split chapter 3 into three sections, instead of hanging everything off section 3.1.

@njr-11 do you have any objection to making this change, before it starts running into conflicts?

@njr-11 do you have any objection to making this change, before it starts running into conflicts?

I forgot about this one. I remember seeing it and not wanting to spend time comparing the files to ensure nothing was left out, so I set it aside for later.  If you or another reviewer has already verified that, I'm fine with it.

Well, the difference in linecount should be 13+2+2-1 = 16 lines (a copyright header, two includes, the one added line of text, minus a heading. And 669-653 = 16, so that checks out.

I double checked in a diff tool and confirmed nothing is left out. Just the change from Repository to Architecture, the new text mentioned above, and the heading level changes to split up chapter 3.

Gracias, caballeros.


ircevent wish list

[ ] handle CAP LS 302 (cap values, multiline responses)
[ ] save the 005 output in a user-visible way
[ ] potentially reevaluate the timeout system (the behavior I introduced in the fork is draconian: a PING is sent every KeepAlive, and must be replied to within Timeout or else we kill the connection)

[ ] support BATCH (unclear what the right API is)

With 005 parsing, we could automatically set the new bot mode!

This is done (I think the timeout behavior is fine).


Refactor GitHub Actions

This PR refactored the GitHub actions with the goal to make them work properly again
For this all extensions have been removed from the php setup as they are not needed, removing with them all php 8.1 incompatibilities.
The used php version has also been upgraded to php 8.1 as this is the minimum version for this package.
The spatie/ray package has been removed as it requires ramsey/uuid which is not yet compatible with php 8.1 and makes static analysis with psalm fail.

Thanks!


Race condition with onClick and onFocus on date/time inputs

Input fields have onFocus and onClick set to override the default behaviors, so that only the text input itself is enriched.
This possibly is related to a race condition where sometimes the timestamp or range is not initialized correctly when clicking on the text input first.

Fixed by initializing once in the keydown event


[media][Encode]Green block appeared when using VEGA tools to playback H264 CQP 4K encoded bitstream,set qp=7

1.Testing environment:
Libva: (master)70b80c0dd2effb4956b208775641f7c68a67a9df
Libva_intel_driver: (master)611d8ea9d75dc026c203e3ebe53b434769d4587c
libyami: (master) 0cfc3a48e6eb4af3adc437879f082351a0cff2b2
ffmpeg: (master) da8b70b810bb2703f976d737af8c68d35716f5ea
2.Test command:
1.$/home/yami/libyami/libyami/tests/yamiencode -i /media/encoderbitstreams/4K_2013_3840_2160_300frames.yuv -s I420 -W 3840 -H 2160 --qp 7 -c AVC --rcmode CQP -o /home/yami/h264/h264_cqp/4K_2013_3840_2160_300frames.yuv_7.h264
2.$/home/yami/libyami/libyami/tests/yamidecode -i /home/yami/h264/h264_cqp/4K_2013_3840_2160_300frames.yuv_7.h264 -m 0 -f I420 -o ./h264_cqp_I420/

I try to use libva test case "avcenc" to encode this yuv video, and set qp=7:
avcenc 3840 2160 4K_2013_3840_2160_300frames.yuv avcenc_4k_qp7_level52.h264 qp=7 mode=1
The encoded frame_number of 187 also have green block.
So I think it's a libva-intel-driver bug, not libyami. My suggestion is closing this issue and firing a bug to libva-intel-driver.

Hi @keqiaox @wenxin2 : How about close this issue and file a bug to libva project?

hi zhong,
I can also duplicated this issue with libva intel driver, so I will close this bug and open a new one for libva driver.
Thanks~


Filename conflict: Option for renaming in dialogue

What's the Problem?

Duplicating a file/folder prompts the user with a dialogue they have to act upon.
Often users want to duplicate a file/folder to create a different version of it. This often goes with renaming the duplicated file/folder afterwards.
Solution/Idea
Instead of letting the user act twice, first for the renaming dialogue and after by going into renaming mode, another option could be added. The user could give a name to the duplicate directly in the dialogue, making duplicating a way smoother experience.
Alternatives
/
Priorities

Capability
Priority

This will allow users to rename files/folders directly in the dialogue
Should

This proposal will check again for a naming conflict after the user renamed
Must

Files Version
Version: <IP_ADDRESS> OS Version: 10.0.19043.1110
Windows Version
Windows 10 21H1
Additional comment
No response

Merging with #5645


[BUG] NetworkProfile CIDR <IP_ADDRESS>/16 overlaps with reserved VNET CIDR <IP_ADDRESS>/8

Describe the bug
{"code":"DeploymentFailed","target":"/subscriptions/xxx-xxx/resourceGroups/rg_graphrag/providers/Microsoft.Resources/deployments/aks","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":[{"code":"NetworkProfileCIDROverlapVNET","target":"/subscriptions/xxx-xxx/resourceGroups/rg_graphrag/providers/Microsoft.ContainerService/managedClusters/aks-graphrag","message":"NetworkProfile CIDR <IP_ADDRESS>/16 overlaps with reserved VNET CIDR <IP_ADDRESS>/8"}]}
To Reproduce
Steps to reproduce the behavior:

Downloaded release 0.3.3
Setup configuration values required in infra/deploy.parameters.json
Ran deploy.sh

Expected behavior
Deployment should proceed without any errors.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11 - WSL2

Please help with solution.

I updated the code to the latest version on master and got past this issue.


Export of Oracle 11.2 to sqllite creates unnecessary primary key

System information:

Windows 10
DBeaver 6.2.2
Oracle ojdbc6
sqlite <IP_ADDRESS>

Connection specification
Source

Oracle 11.2g
ojdbc6
VPN connexion

Target

Sqlite
jdbc <IP_ADDRESS>
local file

Describe the problem you're observing:
I'm trying to export remote Oracle tables to a local sqllite database for testing purposes. The export functionnality creates the table in the sqllite database, but it also creates a primary key on the first column, while there is no primary key in the source table.
The export then fails, because the primary key constraint fails.
On the other hand, when there's a primary key on the source table, everything seems to run fine.
Steps to reproduce, if exist:
Oracle DDL
This is the original DDL for the table :
  CREATE TABLE "SCHEMA"."THE_TABLE"
   (	"OFFER" VARCHAR2(30) NOT NULL ENABLE,
	"TARGET" VARCHAR2(255) NOT NULL ENABLE,
	"TYPE" NUMBER(3,0) NOT NULL ENABLE,
	"FORMAT" VARCHAR2(200),
	"SOURCE" VARCHAR2(500),
	"NUM_VALUE" NUMBER(20,0),
	"BOOL_VALUE" NUMBER(1,0)
   ) SEGMENT CREATION IMMEDIATE
  PCTFREE 10 PCTUSED 40 INITRANS 1 MAXTRANS 255
 NOCOMPRESS LOGGING
  STORAGE(INITIAL 65536 NEXT 1048576 MINEXTENTS 1 MAXEXTENTS<PHONE_NUMBER>
  PCTINCREASE 0 FREELISTS 1 FREELIST GROUPS 1
  BUFFER_POOL DEFAULT FLASH_CACHE DEFAULT CELL_FLASH_CACHE DEFAULT)
  TABLESPACE "SPACETABLE" ;

SQL-LITE DDL
CREATE TABLE THE_TABLE(
	OFFER VARCHAR2(30) NOT NULL,
	TARGET VARCHAR2(255) NOT NULL,
	TYPE NUMBER(3,0) NOT NULL,
	FORMAT VARCHAR2(200),
	"SOURCE" VARCHAR2(500),
	NUM_VALUE NUMBER(20,0),
	BOOL_VALUE NUMBER(1,0),
       PRIMARY KEY (OFFER)
)

Thank you for the bug report.

Fixed.
In fact the problem was in Oracle extension and in a way dbeaver determine identifying constrains.

verified


Write a Ruby program to convert string to constantcase

Description
Write a Ruby program to convert string to constantcase
Constantcase is a style of writing in which all letters are uppercase and spaces are replaced with underscores.
Input  : "hello world"
Output : "HELLO_WORLD"

How to contribute

Save the solution in program/ConvertStringToConstantcase.rb file
Add ConvertStringToConstantcase.rb file in convert-string-to-constantcase folder

!assign


Failed to retrieve list of speedtest.net servers

Testing script off fresh download:
wget -O speedtest-cli https://raw.githubusercontent.com/sivel/speedtest-cli/master/speedtest_cli.py
chmod +x speedtest-cli
When run I am getting:
Retrieving speedtest.net configuration...
Retrieving speedtest.net server list...
Failed to retrieve list of speedtest.net servers:
However on another device this is working correctly.

Try https://raw.githubusercontent.com/sivel/speedtest-cli/modularize-2/speedtest.py
I believe this issue is already fixed there.

That worked thank you
Sent from my iPhone

On Aug 30, 2016, at 4:30 PM, Matt Martz<EMAIL_ADDRESS>wrote:
Try https://raw.githubusercontent.com/sivel/speedtest-cli/modularize-2/speedtest.py
I believe this issue is already fixed there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


wip

Signed-off-by: John Schnake<EMAIL_ADDRESS>What this PR does / why we need it:
Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer:
Release note:
release-note

I dont know if we needed this or not. Just closing for now.


Remove strnlen usage

Remove the strnlen usage in cellular_pktio.c cellular_at_core.c and cellular_bg96_api.c to support more compilation environment #59.

strlen() is not safe enough if the buffer is not that clean as expected. Maybe we need to figure out another way to resolve this issue.


Made typings plugin friendly & add typings for every package

What's Changing and Why
Previously, there was a single index.d.ts file in jimp. However, if you wanted to use any of the @jimp files, you were unable to get the typings for them individually.
This meant a few things:

You couldn't get typing info if you were using @jimp/custom
The definition files were often in different places from their logic, making maintaining them much harder

The second point was made even clearer as this went on as there were a few plugins that had no type information previously (thusly making the jimp package type info incomplete). As a result, the following plugins now have typings on the jimp package that were previously not present:

Circle
Fisheye
Shadow
Threshold

What else might be affected
This will likely be a breaking change for jimp (unless we want to export the same typings as before using export * from '@jimp/core or something similar)
Tasks

[ ] Add tests
[ ] Update Documentation
[x] Update jimp.d.ts
[ ] Add SemVer Label

@crutchcorn  please rebase so the CI runs

@hipstersmoothie done

Wanted to check on the status of this. Seems that tests have passed

@crutchcorn sorry! I'm the only maintainer on this and really don't use jimp at all in my work. So I'm a little slow at getting to the issues. I haven't had time to take a proper look at this, but I'll look at it tomorrow

No worrier in the slightest. I know how maintaining something like this that we don't use ourselves goes. Just wanted to give a small bump cuz I know I need reminders on the one I maintain 😁

Really liking what I'm seeing though.
I'd prefer to make the change as breakingless as possible. Would that be easy?

Actually, if I did this properly this is not a breaking change in terms of the main Jimp typings. If you'd like, I can export the same other typings (ala the font options, etc) and then it will be 100% non-breaking change

Yeah do that and I think this is good to merge
On Sat, Aug 31, 2019 at 9:39 PM Corbin Crutchley<EMAIL_ADDRESS>wrote:

Actually, if I did this properly this is not a breaking change in terms of
the main Jimp typings. If you'd like, I can export the same other typings
(ala the font options, etc) and then it will be 100% non-breaking change
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oliver-moran/jimp/pull/770?email_source=notifications&email_token=AAJDEBG4G6CMEQ2WSE6DT6TQHNBRVA5CNFSM4IJ3DVZ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5T2I2Y#issuecomment-526885995,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJDEBCGXOJJS7V4I3AWEQ3QHNBRVANCNFSM4IJ3DVZQ
.

Actually, you know what? I was just about to make that "fix", but then realized that none of the other types in that file were actually exported, so this PR in it's current state is already not a breaking change.

how can i convert .jpg to .png?
i want to convert .jpg image before color picking.
because .jpg's color picking is not exact.

Mmm, apologies. I'm going to reject my own PR until I can fix some issues with it. I'll try to have them fixed in the next day or two. Huge apologies for bumping when things aren't fully working >~<

So sorry once again for bumping before being ready.
I have now double and tripple checked that this should work in my own project which uses the configure function. I have also checked that the original jimp package exports the same things in it's typings now. This PR should be GTG and does not introduce any breaking changes

Actually, while this can totally be a patch for jimp, I might suggest making it a minor bump as it technically introduces the feature of typings to all of the non jimp mainline package

Merge conflicts resolved, scanIterator added to the @jimp/core package (and as a result the jimp typings)
Still not a breaking change :)

:rocket: PR was released in v0.8.0 :rocket:


Enable building TorchArrow with PyTorch in CI

This PR enables building TorchArrow with PyTorch in CI. This also allows us to test the functionality of some of the torchtext operators that were added in https://github.com/facebookresearch/torcharrow/pull/271 and https://github.com/facebookresearch/torcharrow/pull/287

@wenleix any suggestions on fixing the CI error? From the error message, the build thinks we're using CUDA even though I'm explicitly installing the cpu version of torch.
CMake Error at /home/runner/.local/lib/python3.7/site-packages/torch/share/cmake/Caffe2/Caffe2Config.cmake:90 (message):
  Your installed Caffe2 version uses CUDA but I cannot find the CUDA
  libraries.  Please set the proper CUDA prefixes and / or install CUDA.
Call Stack (most recent call first):
  /home/runner/.local/lib/python3.7/site-packages/torch/share/cmake/Torch/TorchConfig.cmake:68 (find_package)
  csrc/velox/CMakeLists.txt:73 (find_package)

@wenleix any suggestions on fixing the CI error? From the error message, the build thinks we're using CUDA even though I'm explicitly installing the cpu version of torch.
CMake Error at /home/runner/.local/lib/python3.7/site-packages/torch/share/cmake/Caffe2/Caffe2Config.cmake:90 (message):
  Your installed Caffe2 version uses CUDA but I cannot find the CUDA
  libraries.  Please set the proper CUDA prefixes and / or install CUDA.
Call Stack (most recent call first):
  /home/runner/.local/lib/python3.7/site-packages/torch/share/cmake/Torch/TorchConfig.cmake:68 (find_package)
  csrc/velox/CMakeLists.txt:73 (find_package)

cc @mthrok

The change list looks okay, and the CI log indeed says it is installing CPU version. And I expect this kind of issue from CUDA-enabled torch installation. So I am not sure why.


CI: Cilium IPsec upgrade - Git credentials not removed, aborting now.

CI failure
Link: https://github.com/cilium/cilium/actions/runs/8387397247/job/22969525847
Run # For private repositories requiring authentication, check that we
  # For private repositories requiring authentication, check that we
  # can no longer fetch from the repository.
  if ! curl -sL \
    -H "Accept: application/vnd.github+json" \
    -H "X-GitHub-Api-Version: 2022-11-28" \
    "https://api.github.com/repos/cilium/cilium" | \
    jq --exit-status '.private == false'; then
    echo 'Checking whether "git fetch" succeeds'
    if git fetch origin HEAD; then
      echo "::error::Git credentials not removed, aborting now."
      false
    fi
  fi
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    cilium_cli_ci_version:
    job_name: Setup & Test
    QUAY_ORGANIZATION: cilium
    QUAY_ORGANIZATION_DEV: cilium
    QUAY_CHARTS_ORGANIZATION_DEV: cilium-charts-dev
    EGRESS_GATEWAY_HELM_VALUES: --helm-set=egressGateway.enabled=true
    CILIUM_CLI_RELEASE_REPO: cilium/cilium-cli
    CILIUM_CLI_VERSION: v0.16.3
    PUSH_TO_DOCKER_HUB: true
    GCP_PERF_RESULTS_BUCKET: gs://cilium-scale-results
    KIND_VERSION: v0.22.0
    KIND_K8S_IMAGE: kindest/node:v1.29.2@sha256:51a1434a5397193442f0be2a297b488b6c919ce8a3931be0ce822606ea5ca245
    KIND_K8S_VERSION: v1.29.2
false
Checking whether "git fetch" succeeds
From https://github.com/cilium/cilium
 * branch              HEAD       -> FETCH_HEAD
Error: Git credentials not removed, aborting now.
Error: Process completed with exit code 1.

Maybe this problem stems from an unexpected response from the GitHub API? If the private key doesn't exist, then jq will fail and we'll trick the step into thinking that the repo is private:
$ if ! echo '{}' | jq --exit-status '.private == false'; then echo "private"; else echo "public"; fi
false
private

We could add a check for the existence of the .private key to see if that helps, something like this:
if echo '{}' | jq --exit-status '.private != null and .private == true'; then echo "private"; else echo "public"; fi
false
public
if echo '{"private": true}' | jq --exit-status '.private != null and .private == true'; then echo "private"; else echo "public"; fi
true
private
if echo '{"private": false}' | jq --exit-status '.private != null and .private == true'; then echo "private"; else echo "public"; fi
false
public

Do we know how often this has been occurring? If it's very rare, it's maybe acceptable as a false positive?
I fear that loosening the constraint might cause us to miss a change in the way the checkout works. For example, with Ryan's workaround, if the GitHub API stops providing the .private field at some point, we'll never detect a change in credentials and will proceed even though the credentials remain available. On the other hand, we explicitly ask for a given version of the GH API, so this is maybe OK.

Do we know how often this has been occurring? If it's very rare, it's maybe acceptable as a false positive?

I've personally seen it a couple of times now.

OK, thank you.
How about we display more information from the curl request, first? I'm curious to understand what's going wrong with the check, and this might help find the right workaround: https://github.com/cilium/cilium/pull/31652

Looks like this is no longer happening (as far as I know), can we close the issue?
Un-assigning myself as I won't have a chance to look into it again anytime soon.

The same step failed again, this time the issue was due to rate limiting.
https://github.com/cilium/cilium/actions/runs/11118933394/job/30893337939
Run # For private repositories requiring authentication, check that we
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100   279  100   279    0     0   5808      0 --:--:-- --:--:-- --:--:--  5812
{"message":"API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://docs.github.com/rest/overview/resources-in-the-rest-api#rate-limiting"}
false
Checking whether "git fetch" succeeds
From https://github.com/cilium/cilium
 * branch              HEAD       -> FETCH_HEAD
Error: Git credentials not removed, aborting now.
Error: Process completed with exit code 1.


Feature/71 - Distance nodes in graph

@ddelpiano @zsinnema
To distance the nodes, I have been playing with different layouts and a 'radial' layout is working better than a 'tree' layout. Some datasets have lots of folders and files, which are making the lower levels very crowded in a 'tree' layout. Using a 'radial' layout we have a better space graph without doing arranging of the node's X positions, which is something we would have to do with a 'tree' layout. Before we invest time on distancing the nodes on the 'tree' layout graph, we should check if 'radial' is an acceptable layout instead, it came up before as a request in one of the sprint meetings and it might be okay to use it.
Radial Layout

Tree Layout

Having both will likely be helpful, for small datasets or a focused subset of a large dataset it will be useful, the radial is a good use of space for big datasets.


secure by default?

Why not make 'secure' be the default? It seems like the web is heading towards a secure-by-default design with HTTP/2 being based on SSL.

@StephenHynes7  is there any reason we should not change this?

Bump

sorry but this careless to not enable by default.

It should be secure by default as of  1.4.0


VS Code can't find nested types with prisma plugin

Hey, thanks for providing this great library, I really like it!!
I have a problem with the generated types, a few nested Prisma types doesn't get recognized by VS Code.
This is the following output:
The namespace ""/Users/felixhaberle/Desktop/git/project/node_modules/.prisma/client/index".Prisma" doesn't have an exported member "UserWhereUniqueInput".ts(2694)

I have already deleted node modules, reinstalled all libraries, and checked my node version which is v14.17.0 – still the same error.
Do you know what can help in such a case? Thank you very much!!

Nevermind, I fixed it with looking into the VS Code settings, there was a custom Typescript reference. When I removed that reference, the problem was gone. Thanks very much!

glad you got it figured out!


modify access controll level

I want to use this module as a framework.

I was missing modify access control level to use such as CocoaPods.
Thanks for your pull request.


Tax - Selection highlight remains at previous default tax rate when the default rate changes

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

Found when validating PR : https://github.com/Expensify/App/pull/36784
Version Number: v1.4.44-7
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Action Performed:
Precondition:

User is an employee of Collect workspace.
The Collect workspace has some tax options.

[Employee] Go to workspace chat.
[Employee] Open request money page > Manual.
[Employee] Enter amount and proceed to confirmation page.
[Employee] Click Show more > Tax rate.
[Admin] On Old Dot, change the default workspace tax rate.

Expected Result:
The selection highlight will move when the default tax rate changes.
Actual Result:
The selection highlight remains at the previous default tax rate when the default rate changes.
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/bab04821-750a-48cc-84bf-fce741063b5f

View all open jobs on GitHub

@slafortune I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think that this bug might be related to #wave6-collect-submitters
CC @greg-schroeder

Feature is WIP, so we can put this HOLD until we finish the implementation as it is being rolled out in phases https://github.com/Expensify/Expensify/issues/305057

I'll move this to weekly in the meantime.

PR is being reviewed - 33927

Still on hold

Still on hold

The likelihood of someone running into this is in practice is so unlikely. You'd have to be creating an expense and pausing on the tax input selector exactly at the time someone on the workspace settings edits the default. @MonilBhavsar if you haven't easily solved this in a PR that's coming for Track Tax, I'd close this.


⚠ Bofrost.be has degraded performance

In 078d254, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1121 ms

Resolved: Bofrost.be performance has improved in db3132a after 6 minutes.


Fix: grpc test may fail

fix grpc sample may fail

Codecov Report

Merging #251 (915e1b9) into develop (1052eaa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #251   +/-   ##
========================================
  Coverage    40.18%   40.18%
========================================
  Files           50       50
  Lines         2541     2541
========================================
  Hits          1021     1021
  Misses        1410     1410
  Partials       110      110

Continue to review full report at Codecov.

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


Improve breakpoints and add variable for max-width

This is an enhancement or feature.

Summary
I really don't know if you are interested in adding this to the theme. It comes to resolve a problem that is mainly there when you change the size of the browser in computers, o you have a small screen computer or an iPad. It felt that sometimes in smaller browser sizes some content was bigger, wider, than in bigger ones for no reason.
Create a new breakpoint in between "large" and "x-large" so the size changes are a little bit more fluid.
Tune-up how the right and left sidebars and the post/page content behaves. It feels more fluid now and usually when the size of the browser decreases the content decreases. Before at some breakpoints the content was bigger than in the wider breakpoint.
Create a $max-width variable, so know the content can be wider than the larger breakpoint. This variable is applied to page, search and masthead "__inner-wrap".

It felt that sometimes in smaller browser sizes some content was bigger, wider, than in bigger ones for no reason.

Can you post some screenshot examples of these? I'd like to see how bad things can look before committing to adding another breakpoint.

Since Luis took the initiative to add the breakpoints, I expect he will have a better idea of the boundary conditions he fixed.  I have, however, observed jumpy behavior as boundaries appear to be crossed.  I'll see what I can put together.  If your interested in trying some of this for yourself, you can check out the non-trivial pages in https://michaeljcaruso.github.io/vision/ to see if anything jumps out at you.  I'm not using Luis' fork, but I am using a fork that provides for separate override of $max-width (called $x-large-max-width in my world).

This is how the default theme behaves:

And this is how behave with the PR in my web.

Ok, so the content isn't really breaking. This is more of "I'm the sort of person who goes to websites and resizes the browser to see how responsive they are" rather than a normal use case of reading the page as present. :wink:
Still not completely convinced this is a problem worth solving for by adding complexity of another breakpoint.

Ok, so the content isn't really breaking. This is more of "I'm the sort of person who goes to websites and resizes the browser to see how responsive they are" rather than a normal use case of reading the page as presented. :wink: ...
LOL!! 😂

Yeah it’s totally that.
To be honest I told you upfront that I really didn’t know if this was going to be useful for you. For me the main issue here is the content should be always smaller / narrower as the browser window get smaller / narrower.
It’s true that one usually doesn’t change the browser window size, but I sometimes do... (to fit more than one window in the screen) and screens  computers and resolutions really varies.
The aim of the pull is consistency in the way smaller screen always smaller content.
I understand that you want to keep the theme as simple as possible, though.
Sent with GitHawk

Fair enough.
When flexbox and display: grid gain even more traction I can move away from floats and media queries for most of the layout. That will greatly improve how fluid the content is and remove all the hard stops based on a arbitrary breakpoint widths.

If I can chime in as an interested third party, while in the process of deploying a significant GitHub pages documentation site using minimal-mistakes, tuning the presentation of my own material led me to Luis' PR.  While change dynamics are always going to be a matter of aesthetics and boundary behavior, one aspect of Luis' change that attracted my attention was its simple $max-width parameterization of screen width utilization.  While we can always customize that in a fork (and I have), I'd like to upvote the inclusion of that feature in minimal-mistakes, especially given the way approaches to the correct solutions have changed in response to major rewrites of libraries like _sass.  My two cents, for what it's worth (-: yes, I know, 2 cents :-)
Cheers

I see value in adding the $max-width Sass variable so you can better control container's width with max-width: $max-width;.
The other adjustments to fine tune the sidebar widths for large-wide were more of what I was directing my statements against.
@luispuerto If you want to retool the PR (or submit another one) with just the $max-width additions and leave out the new sidebar variables and large-wide breakpoints, I'd be happy to merge that in.

@mmistakes we can use this PR, I don't have a problem.
The modifications have been done 🙂

Perhaps for reference sake I'm going to drop here a link to the commit on my fork where I did the changes
https://github.com/luispuerto/luispuerto.net/commit/da7744ce79f1e0efe2fc08b2308eee1de3361ffa

Looks good @luispuerto, thanks.


bumped the version up to {1.11.1}

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

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

Everything has passed except 2/4 of the ppc64le builds, which are currently awaiting available machines. The 2 ppc64le jobs that completed successfully did so in under 30 minutes. Let's see if the remaining 2 jobs run by tomorrow, and if not, I'd say we should go ahead and merge.

It looks like the re-render step failed because the PR was issued from an organization repository. I guess we will have to maintain the feedstock from personal repositories. Do you have any ideas on how to proceed? Perhaps close the PR and re-issue it?

Oh right, I didn't even notice that. Yeah, I think that is the best way to proceed.


WalletConnect is hidden in mobile phones

Ref: https://open-rsk-dev.slack.com/archives/C02JVHMHC49/p1646916400849289

I was able to temporary workaround this issue by adding this to my css:
.rlogin-providers-mobile {
display: flex !important;
}

There is a variable hideMobile={true} set on providers https://github.com/rsksmart/rLogin/blob/6e31694ae4ca39ef67945f7357c5c6637d650508/src/ux/step1/WalletProviders.tsx#L89
Testing: see https://docs.cypress.io/api/commands/viewport#Syntax

@ilanolkies not sure what to do with hit information, where can I pass "false" here? It doesn't make sense that wallet connect will not be shown on mobile by default..

We need to do a PR to fix it. It needs to be false by default, we agree on that

Great, then this could arrive in the next release of rLogin I believe as it's such a small change :)

@jessgusclark / @ilanolkies - I tested it and it does not work. Wallet connect does not appear by default on mobile.
In addition, it seems that @tkeidar 's workaround is not working anymore.
That means we cannot upgrade to this new version of rLogin as users won't have walletconnect on mobile view.
Can you please take a look at it? If I'm correct then this issue needs to be reopened.


Some additional environment variables

Context
We are migrating from gableroux docker images on circleci.
Suggested solution
Could you please add the following environment variables on images similar to unityci/editor:ubuntu-2019.4.14f1-android-0.9.0?

export JAVA_HOME=/opt/unity/Editor/Data/PlaybackEngines/AndroidPlayer/OpenJDK
export ANDROID_HOME=/opt/unity/Editor/Data/PlaybackEngines/AndroidPlayer/SDK
export ANDROID_NDK_HOME=/opt/unity/Editor/Data/PlaybackEngines/AndroidPlayer/NDK
export PATH=$JAVA_HOME/bin:$ANDROID_HOME/tools:$ANDROID_HOME/tools/bin:$ANDROID_HOME/platform-tools:$PATH

Also, could you run the following to accept latest android tool updates: yes | $ANDROID_HOME/tools/bin/sdkmanager --licenses || true
Considered alternatives
Also worth considering, could you either change UNITY_PATH environment variable
from: /opt/unity
to: /opt/unity/Editor/Unity
Or alternatively create a new environment variable for this.
Additional details
We can set these ourselves in our own yml builds but it might useful for others.
Thanks!

Could you please elaborate on the rationale behind these changes?
Please assume I don't know anything about the inner workings of Unity editor.
We're willing to add each line if there's a sound rationale behind it.
Also note that some of these paths may not really exist on images other than those with android as a target platform.

No problem. Yeah, the java ones are for android target platforms.
We build a unity sdk that leverages native code on android which is coded in java. To build the unity android app we need to compile (and run tests) on the native android layer, which then gets packaged into unity, and then we build/package the whole lot together.

This might also increase general reliability for Android builds.
Quick details of each var:
JAVA_HOME - Canonical way to specify system path to Java JRE/JDK
ANDROID_HOME - Canonical way to specify system path to Android SDK
ANDROID_NDK_HOME - Canonical way to specify Android NDK path
The PATH env var should also be updated to add all the Android command-line utilities to the default search path, to facilitate usage of those tools in pre- or post-Unity build steps.

@dominicmarmionswrve Check out https://github.com/game-ci/docker/pull/88 - just merged, should set those environment variables for you.  You might still need to accept the license agreement on your own, though.

We might still wanna accept the license for the sdk manager, as we're making images for CI, and the license has been accepted on peoples computers already.
I recently excluded 2018.2 builds and below from using this fix because sdkmanager was not recognized. Perhaps we should undo the change using your suggestion:
yes | $ANDROID_HOME/tools/bin/sdkmanager --licenses

We could then apply it to both 2018 and 2019+ images.


Add NoOp support

Adds support for the NoOp (no-operation) operation. This does nothing except await a completion.
I'm trying to understand the performance aspects of various bits of my system, and understanding the runtime overhead is useful. This makes that a lot easier.
It doesn't clearly fall into fs, net, or buf, so I've placed it at the top level.

@ollie-etl can you add a unit test for this?

Sure

@ollie-etl the test fails

@ollie-etl the test fails
Arg! I refactored before committing, obviously. Sorry about that. I'll fix it up


update gpt url and fix key error in gpt generate and train main

address issue #317 and #337

Fix https://github.com/asyml/texar-pytorch/issues/317, Fix https://github.com/asyml/texar-pytorch/issues/337


typo "iPod Touch"

Description
(en) It should be "iPod Touch", but it is "iPad Touch".
(ja) "iPod Touch"とすべきところ、"iPad Touch" になっています。
Version
5.6.0
Diff
581
-            return "iPad Touch (unknown)"
+            return "iPod Touch (unknown)"

Thank you for pointing it out! We will release the corrected version over the weekend.
Thank you for your cooperation!

Closed because it has been resolved


Add an Apache Zeppelin Logo section to the website

What is this PR for?
This PR is bringing both PNG and SVG format of the logo to the website, as well as a few other infos about color and font used.
To do that we add a new page and navigation menu named: More
What type of PR is it?
Documentation
How should this be tested?
Build the website and visit the More menu
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Cool!
It's a small thing, but would you consider move "Apache Zeppelin Logo" into "Apache" menu?
No special reasons for me, but it can keep number of top level menu small.

@Leemoonsoo just changed the position, what do you think?

Merging if there is no more discussion


[TASK] Write documentation for MASP transaction protocol

https://www.notion.so/hicommonwealth/Transaction-Circuit-v1-9a0f2c6480f94a0c91c64d76dd4c068f?pvs=4
Expand on following topics:

[ ] What is a multi-asset shielded pool?
[ ] What are features of MASP?
[ ] Main functionality (what are private asset transfers).
[ ] How is privacy-preserved via ZKPs and relayers?
[ ] What types of tokens are supported?

Features

[ ] Join and split private transactions
[ ] Internal shielded transfers, withdrawing
[ ] Private swapping between different types of assets via RFQ, relayers, and ZKPs.
[ ] Rollup Functionality via Batched Deposits
[ ] Support for fungible and non-fungible tokens
[ ] Yield generation via anonymity mining

Outline

[ ] Overview
[ ] Lifecycle (user interacts with privacy protocol by shielding assets, describe user stories broken down by user type/persona, etc.)
[ ] Implementation Details (smart contract calls, event watching, etc.)
[ ] Fees and Incentives
[ ] Privacy and Security
[ ] Links (code links, repo links, etc.)

Draft complete: https://www.notion.so/hicommonwealth/MASP-Protocol-for-docs-site-8e2c060c880b4f7c90bc692ef4c44f87?pvs=4


Getting Failure when testing connection

Version report
Jenkins and plugins versions report:
Jenkins Version Jenkins 2.319.1
Slack Plugin Version: 2.49

What Operating System are you using (both controller, and any agents involved in the problem)?

Ubuntu 20.04

Reproduction steps

Installed Slack Plugin, when doing Test Connection trough configure option getting "Failed". With this showing in jenkins.log

2021-12-07 23:20:40.422+0000 [id=16]    WARNING j.p.slack.StandardSlackService#postToSlack: Slack post may have failed. Response: null
2021-12-07 23:20:40.422+0000 [id=16]    WARNING j.p.slack.StandardSlackService#postToSlack: Response Code: 400
Also configured a log recorder for "org.apache.http" which is showing:
'http-outgoing-6 << "HTTP/1.1 400 Bad Request[\r][\n]"'
and some lines below
'http-outgoing-6 << "invalid_channel[\r][\n]"'
which is weird because I just copied/pasted
channel name from slack
Results
Expected result:
Successful message sent to Slack
Actual result:
Failed status on Test Connection

What configuration did you do in the slack side?

i used this app  on slack

also got the  "added an integration to this channel" message on the channel
i double checked  leading or trailing white spaces on Default channel / member id and there is non, tried with the # before channel name and without it , and got the same result

Generally this is a better way to set it up (bot user mode):
https://github.com/jenkinsci/slack-plugin#bot-user-mode


open-interpreter: 0.1.3 -> 0.1.4

Description of changes
Update to latest version. No changes that effect build.
Diff: https://github.com/KillianLucas/open-interpreter/compare/v0.1.3...v0.1.4
Changelog: https://github.com/KillianLucas/open-interpreter/releases/tag/v0.1.4
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.


typo in chapter 16 jupyter notebook line 39

(X_train, y_test), (X_valid, y_test) = keras.datasets.imdb.load_data()
instead of y_train, y_valid, y_test is used at both the places. However its not a problem since its not used ! just FYI in case if you want to change it.

Good catch, thanks! Just fixed.


feat: fix p3.9, p3.10 Dockerfiles. Deprecate non-"all" files for now

Fixes #488

Thanks - both images build successfully with the change to .['modules'].
We might want to look into the image size, esp. for the 3.10 version:
       compressed    uncompressed
3.9:     3.9 GiB       11.5 GiB
3.10:    6.5 GiB       8.78 GiB


Unified RNN interface

This PR fixes #1271
I implement the function create_stateful_rnn, which creates stateful RNN units from stateless RNN ones and change the implementation of LSTM, StatefulGRU, StatefulSGU, StatefulDSGU, PeepholeLSTM and StatefulPeepholeLSTM to use this function.
As you can see, this PR includes #1101 and #1115 to test the behavior. So I would like to merge them first.

Current implementation breaks backward compatibility because StatefulGRU is not a subclass of GRUBase.

We discussed how we should proceed this PR and concluded that first we need the design document on the ideal interfaces of RNN modules and see the discrepancy from the existing ones.

Another comment is that generation of RNN classes is too complicated to realize the feature. It is better if we can do it in more handy way.

We are redesigning the interface of RNNs and planning to send another PR. So we close this PR.


Android 11 has new requirements regarding background location permissions

Which platform(s) does your issue occur on?
-Android version 11+
-emulator / device
For Android 11+ a new location permission was added: "ACCESS_BACKGROUND_LOCATION"
In order to work in the background (e.g from a background service) you have to add that permission to the manifest.
Additionaly it is necessary to ask the user to grant the permission to run in background, so user has to grant "Allow all the time"!
But this grant is not visible in the plugin.
https://developer.android.com/training/location/permissions#request-background-location
says you can get this label by processing: getBackgroundPermissionOptionLabel()
Is it possible to do that with this plugin?
Additionally when checking
geolocation.isEnabled() I assume it is necessary to distinguish between the different types of location grants.
Currently my app checks if it is enabled and returns TRUE, but then getCurrentLocation freezes as it does not have the correct grant ("Allow all the time")
KR Klaus

NativeScript 8 uses another package for geolocation: https://github.com/NativeScript/plugins/tree/main/packages/geolocation  .
The 'always' flag is already present there.


"Add Configuration..." only works if launch.json has configurations

VS Code version: Code 1.31.0 (7c66f58312b48ed8ca4e387ebd9ffe9605332caa, 2019-02-05T22:35:56.624Z)
OS version: Windows_NT x64 10.0.17134
Steps to Reproduce:

Create a launch.json with this content:
{

}

Click the "Add Configuration..." button in the lower right corner.

Other than the cursor, flashing, nothing happens.

If there's an empty configurations array, it works as expected. I'm not sure if it's always been like that or if it's a recent problem.

This is still an issue for me in version 1.36.0 (not Insider's). I have to manually create a launch.json file, because VSCode won't create one at all.


Deprecate k8s skaffold scripts and move helm to project root directory

https://github.com/apache/pinot/issues/11647

Codecov Report

Merging #11648 (c9f2ed3) into master (bc07eb8) will decrease coverage by 48.65%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #11648       +/-   ##
=============================================
- Coverage     63.10%   14.45%   -48.65%
+ Complexity     1120      201      -919
=============================================
  Files          2335     2335
  Lines        125208   125208
  Branches      19209    19209
=============================================
- Hits          79013    18103    -60910
- Misses        40559   105581    +65022
+ Partials       5636     1524     -4112

Flag
Coverage Δ

integration
0.00% <ø> (-0.01%)
:arrow_down:

integration1
?

integration2
0.00% <ø> (ø)

java-11
14.45% <ø> (-48.64%)
:arrow_down:

java-17
14.45% <ø> (-48.50%)
:arrow_down:

java-20
14.45% <ø> (-35.55%)
:arrow_down:

temurin
14.45% <ø> (-48.65%)
:arrow_down:

unittests
14.45% <ø> (-48.65%)
:arrow_down:

unittests1
?

unittests2
14.45% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1511 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Storeutl keys doc

Fix storeutl doc wrt to -keys selector
Checklist

[x] documentation is added or updated

A low-hanging fruit, ping @openssl/committers for the 2nd approval

This pull request is ready to merge

Merged to all branches. Many thanks!


React Scripts Left Out of Github Pages Deployment

Hi, I am trying to deploy my react app to github pages, and I'm running into an issue where the url I provide loads the main index.html, but none of my react scripts. When I open view source on my github pages site I see the main index.html.
I followed the directions given in: create-react-app/packages/react-scripts/template/README.md, and everything worked up to this point. I also notice in my build folder that none of my react scripts are included, so perhaps this is the issue. Please advise.
Here is my git repo
Here is the github pages site
Thanks!

you need to provide homepage in your package.json where you deploy.

Hi @themre ,
I was facing the same issue a few days ago and decided to stop messing up with github pages since it's not that friendly with single page apps. I succeeded to use surge.sh without extra dancing around, here is my react.js repo: . Check out package.json, predeploy and deploy sections. That's all you're going to need. Optionally, you can add CNAME and .surgeignore to you public folder.

Closing this as answered by https://github.com/facebookincubator/create-react-app/issues/2418#issuecomment-305125293.


refactor: better resource cleanup handling

What this PR does / why we need it:
This PR improves resource cleanup handling.

moves cleanup code closer to the resource creation code
reuse discovery information used at resource creation time to make creation/cleanup more robust
only cleanup a resource if kuttl created it (don't register cleanup if it was an update)

for awareness... I commonly run the following as a starting point of review:  make lint test integration-test
make lint is failing at this point.   Likely missed based on the integration build tag

I will fix this tomorrow. How about running this a gh workflow ?

yeah... when I noticed the passing CI... I had the same thought.  I will look into that.

well... now I remember some issues we had in the past but haven't been resolved.   For some reason CircleCI wouldn't run PRs from non-committers.   So the full CI isn't run (just the DCO).  I worked with circle to help resolve that but didn't get it resolved.   We were looking to move to Github Actions in part to solve that... but it also reduces the number of CI dependencies.   Probably need to refocus on that.

that's what https://github.com/kudobuilder/kuttl/pull/337 was intended to solve.


Group add/del/set permission nodes

I think the commands

pex user <user> group add <group> [world] [lifetime]
pex user <user> group remove <group> [world]
pex user <user> group set <group> [world]

Should be managed by different permission nodes, or at least add/remove should be different node of set. I'm using an invitation system so my citizens users can set new people in Citizens group, but in this way also they can remove them, that could be problematic if the citizens fight between them. My idea is what citizens can add their friends if they enter the server for first time but a moderator intervention be required to remove the user from citizens group.
One way what I've thought to workaround the issue is create visitor group and use promote/demote commands but that's much more messy to people, who should performance several commands.
What do you think? thank you for the interest.

I've just discovered the flag default to groups.

It's still valid, because citizens cannot promote to citizens the people in visitors default group.

This has already been suggested multiple times and is on the list of things @zml2008 might do if he gets bored.

@Stormbow what do you recommend me to manage this?

The very blunt answer is: Don't give out admin-level commands to non-admins.
You're actually better off using plugins like AutoRank to let users promote themselves without having any access to PEX commands.  AutoRank will let users see what they need to do to be promoted to the next rank and they can use an AutoRank command to accomplish their own promotion once they qualify.

You can choose from an almost limitless number and variety of requirements for the user to be promoted.
For example, I required users to have played on the server for a month, voted 30 times, and they had to own a stack of gold blocks (which they could acquire in multiple ways— like mining, nether farming, buying gold nuggest/ingots/blocks from the admin shop) to promote themselves to "Noble" (where they could then own land and start their own town) on my server.

@Stormbow thank you so much :)

You're welcome.  :)  Have fun with it!


[Feature] Forks filtering

New env var GHS_FILTER_FORKS.
Tests for new functionality.

@vladkens, that was actually my initial idea to use !fork 😆 Personally it felt odd and confusing at first, from the user perspective. But on the other hand, maybe it makes more sense to keep filtering in one var. Go ahead and update the PR. Thanks!


Lazy runtime model loading from disk

Is your feature request related to a problem? Please describe.
NOTE This is an exploratory issue! We very well may decide that this isn't the right way to go.
When running caikit.runtime in a cloud environment, it will be replicated for throughput and resilience. Currently, there are two ways to ensure that an inference call for a given model will be satisfied correctly:

Run caikit.runtime as a kserve ServingRuntime using model-mesh to handle dynamic model routing
Know all of the necessary models at boot time and place them "on disk" (likely a mounted volume) in runtime.local_models_dir.

(1) has significant drawbacks in some cases because model-mesh is grpc-only , does not support streaming, and its heuristics are designed for very lightweight models which does not apply to large foundation models.
(2) is similarly problematic in that it doesn't allow a given replicaset of caikit.runtime servers to change the set of loaded models dynamically.
Describe the solution you'd like
In the simplest replication case, it is sufficient to ensure that all models are loaded on all replicas of caikit.runtime. The easiest way to accomplish this without the replicas being aware of siblings is for the replicaset to share a mounted volume and for an inference request with an unknown model-id to fall back to looking for the model "on disk" in the mounted volume and loading it if needed.
This solution would not have any of the sophistication of model-mesh's model rebalancing and it would still be prone to race conditions if the initial load operation were made concurrently with an inference request to the same model, but with appropriate application safeguards to ensure that a model was properly loaded before allowing inference requests, this should handle the case for extremely lightweight models well.
Describe alternatives you've considered

Extend model-mesh to support REST and streaming
Build a Foundation-Model specific component that serves the same purpose as model-mesh, but is geared towards large singleton models with lightweight prompts

Closed by #341


Updated README documentation

Update README.md
Update README_CN.md

@cutelolly sorry we didn't forget about your PR. The problem we are facing is that we actually use the Readme.rst in the documentation. Moving to md could be an issue. with @MaxJPRey we are looking for solutions


Image.set_pixel() not working properly on Mac

Godot version
v4.0.beta15.official [4fa6edc88]
System information
macOS ventura 13.1, M1 Pro
Issue description
Image.set_pixel() is not working properly when creating a new image in GDScript.
Steps to reproduce
I am trying to create an Image with code and then use it to create an ImageTexture to set in a Sprite2D.
The code I'm using is:
func _ready():
	# creates image
	var img = Image.new()
	img.create(10, 10, false, Image.FORMAT_RGB8)

	# setting values pixel by pixel
	for i in range(10):
		for j in range(10):
			img.set_pixel(j,i, Color.RED)

	# creates and sets texture
	var tex = ImageTexture.new()
	tex.create_from_image(img)
	$Sprite2D.set_texture(tex)

Although I'm following the example code available in the documentation available here, an error message appears for every img.set_pixel() like the following:
E 0:00:01:0571   _ready: Index p_x = 8 is out of bounds (width = 0).
  <C++ Source>   core/io/image.cpp:3274 @ set_pixel()
  <Stack Trace>  node_2d.gd:10 @ _ready()

I thought that maybe it was because I wasn't using Image.lock(), but it was removed in v4.0.
Because of this error, later when I do $Sprite2D.set_texture(tex), another error appears like:
E 0:00:01:0572   _ready: Invalid image
  <C++ Error>    Condition "p_image.is_null() || p_image->is_empty()" is true. Returning: Ref<ImageTexture>()
  <C++ Source>   scene/resources/texture.cpp:171 @ create_from_image()
  <Stack Trace>  node_2d.gd:14 @ _ready()

It looks like a bug to me, can anyone else reproduce it on their machines?
Minimal reproduction project
Test_Texture.zip

Create returns an image, not updating the original


fix for notebooks opening as json

This PR fixes #10320
Opening notebooks from books viewlet started opening them as json after last weeks vscode merge (dbb40d820ca2b490ce5111029bc5801d22602995).
Fix: Instead of opening them as text documents, using azdata.nb.showNotebookDocument.

Coverage increased (+0.004%) to 33.398% when pulling 8ca664f1de3836f346faee491fe3dbd3e4ee8115 on fix/notebooksLaunchAsJson into 10ac5fdfab4cca7136701f14aa12c0a1947d7e6d on master.


Fix Warnings

Fix a few more warnings I found on the continuos integration build bots logs
Signed-off-by: Ettore Tiotto<EMAIL_ADDRESS>

Jenkins Linux amd64 Build #3156 [push] Fix Warnings (#1054)  Si... started at 17:50

Jenkins Linux s390x Build #3147 [push] Fix Warnings (#1054)  Si... started at 18:50

Jenkins Linux ppc64le Build #2263 [push] Fix Warnings (#1054)  Si... started at 18:54

Jenkins Linux s390x Build #3147 [push] Fix Warnings (#1054)  Si... passed after 44 min

Jenkins Linux ppc64le Build #2263 [push] Fix Warnings (#1054)  Si... passed after 53 min

Jenkins Linux amd64 Build #3156 [push] Fix Warnings (#1054)  Si... passed after 59 min


[front] reduce table header detection temperature

Description
Reduce model temperature in dust app to avoid result variance
https://dust.tt/w/78bda07b39/spaces/vlt_rICtlrSEpWqX/apps/Hllp6rDlNo
Risk
Deploy Plan
deploy front

@tdraier ooc what's the current status of table detection?


Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
Originally posted by @welcome[bot] in https://github.com/github/docs/issues/18947#issuecomment-1178557362

This issue appears to have been opened accidentally. I'm going to close it now, but feel free to take a look at the contributing.md to learn how to contribute in this repo. You can open a discussion if you have any questions or check out issues with the help wanted label. :sparkles:
Thanks for your passion and interest in improving our docs! 💛


MB-4433 | Removing AWS Commercial accounts from CircleCI Config file.

Description
Commercial AWS is going away, prod already had. Staging as well. This removes experimental.
References

Jira story for this change

Messages

:book:
:link: MB-4433

  Generated by :no_entry_sign: dangerJS against 498793b284eae107e2b7b75a5602f36fea08afcd


Basic support for Kubernetes MCS host

This is the first phase of work for #33949.
This PR adds clusterset.local as an alias for cluster.local and resolves to the same IP. Since we're using the same IP, this change only impacts HTTP listeners and their routes.
The MCS host is also supported for DNS lookup.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

@nmittler: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
0fa6909f92d0bb26c3351bd9b8fbe0d5cf039968
link
/test integ-pilot-multicluster-tests_istio

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

@stevenctl

LGTM – curious how things would work for non http though

Since the clusterset.local host resolves to the same IP, there should be effectively no change for TCP traffic. I can expand the test to verify.

Since the clusterset.local host resolves to the same IP, there should be effectively no change for TCP traffic. I can expand the test to verify.

might still need an SNI match? Although I think we only do that with explicit virtual service sniHosts, in which case its explicitly list, not expanded. so maybe not.


Update golang package

This PR updates the version of golang to 1.13

@cf-london Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

new PR gets created for golang 1.13.1